use a pseudo-random function to determine skipped packet numbers

As Kazuho pointed out in https://github.com/quicwg/base-drafts/issues/1092#issuecomment-362574769, we can use a pseudo-random function to determine which packet numbers to skip.

If such a function is formalized, with an unpredictable key, of course, it would remove a class of attacks on the receivers indexing of received packets, for example by simply dropping a connection if the index starts to use too much space per packet on average. It would also put a bound the gap size so large gap jumps are ruled out. This would alleviate some of my concerns regarding gaps.

I thought this was a general QUIC issue, not quic-go, but I would actually recommend making this a general QUIC proposal for the above reason.

I'm not sure any more if this actually is an improvement to what we have at the moment. Evaluating a cryptographic hash function every single time we send a packet and multiple times when we receive an ACK might be slow (and cause a lot of unnecessary allocations).
The main advantage of the pseudo-random function seems to be that it doesn't require any per-connection state. However, it's not clear how we could defend against the DoS attack where a peer sends us an ACK for a large number of packets, without keeping a slice of skipped packet numbers around. If that's the case, we might as well keep using real random to generate the skips.

Using dummy-packets in the packet history (as implemented in #2753) is probably the easier solution.


collect2: fatal error: cannot find ‘ld’

This problem occurred when I was building TVM. It compiled successfully before, but now it fails to recompile again.

This seems like a problem with your host system, are you able to compile and link a simple c program to an executable? Your PATH environment variable may be changed to not have ld in it anymore either.
If you're just looking to get started right away you could use the TVM Docker images instead: https://tvm.apache.org/docs/install/docker.html

I can compile the .c file directly with gcc to generate the .so file

Please make sure ld is available in your PATH. This is not reproducible.


Learn Tool Window is not synchronized between project windows

#0.8.54
Open LearnProject and complete some lessons.
Open another project in new window.
Open Learn tool window in that project and click on any lesson.
Go back to the LearnProject.
Try to open smth in Learn tool window - it won't respond, although scratch files will open in editor.

fixed in 0.8.56


Support for upcoming AnnData 0.10.0rc1 for compatibility with the latest rapids_singlecell

Hello,
I recently tried setting up a new environment with the latest version of rapids_singlecell (0.9.0, which pulls AnnData 0.10.0rc1) and scvi, I receive the error below while importing scvi.
Steps to reproduce:
environment.yml file
name: rapids-test
channels:
 - conda-forge
 - bioconda
 - rapidsai
 - nvidia
dependencies:
 - rapids=23.08
 - python=3.10
 - cuda-version=11.8
 - leidenalg
 - louvain
 - fa2
 - jupyterlab
 - pip
 - pip:
    - scvi-tools
    - scanpy
    - rapids_singlecell

Construct environment
$ conda env create -f environment.yml
Import scvi in the environment
Python 3.10.12 | packaged by conda-forge | (main, Jun 23 2023, 22:40:32) [GCC 12.3.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import scvi
/allen/programs/celltypes/workgroups/hct/ktrav/conda/envs/rapids-test/lib/python3.10/site-packages/scvi/_settings.py:63: UserWarning: Since v1.0.0, scvi-tools no longer uses a random seed by default. Run `scvi.settings.seed = 0` to reproduce results from previous versions.
  self.seed = seed
/allen/programs/celltypes/workgroups/hct/ktrav/conda/envs/rapids-test/lib/python3.10/site-packages/scvi/_settings.py:70: UserWarning: Setting `dl_pin_memory_gpu_training` is deprecated in v1.0 and will be removed in v1.1. Please pass in `pin_memory` to the data loaders instead.
  self.dl_pin_memory_gpu_training = (
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/allen/programs/celltypes/workgroups/hct/ktrav/conda/envs/rapids-test/lib/python3.10/site-packages/scvi/__init__.py", line 11, in <module>
    from . import autotune, data, model, external, utils, criticism
  File "/allen/programs/celltypes/workgroups/hct/ktrav/conda/envs/rapids-test/lib/python3.10/site-packages/scvi/autotune/__init__.py", line 1, in <module>
    from ._manager import TuneAnalysis, TunerManager
  File "/allen/programs/celltypes/workgroups/hct/ktrav/conda/envs/rapids-test/lib/python3.10/site-packages/scvi/autotune/_manager.py", line 23, in <module>
    from scvi.data._constants import _SETUP_ARGS_KEY, _SETUP_METHOD_NAME
  File "/allen/programs/celltypes/workgroups/hct/ktrav/conda/envs/rapids-test/lib/python3.10/site-packages/scvi/data/__init__.py", line 3, in <module>
    from ._anntorchdataset import AnnTorchDataset
  File "/allen/programs/celltypes/workgroups/hct/ktrav/conda/envs/rapids-test/lib/python3.10/site-packages/scvi/data/_anntorchdataset.py", line 7, in <module>
    from anndata._core.sparse_dataset import SparseDataset
ImportError: cannot import name 'SparseDataset'

PR for new AnnData rc: https://github.com/scverse/anndata/pull/1066
This makes sense as "SparseDataset" was renamed in the RC, per https://github.com/scverse/anndata/commit/8c00a29d7b983e6be7bf437305133b422ffe8168
Want to check you're aware of the future conflict/ask if you know when you'll be able to support the new AnnData version?
Best,
Kyle

Hi, thanks for bringing up this issue. We're aware of the future conflict and are planning on supporting the new AnnData version. No specific timeline on this but we'll most likely add the changes for our v1.1 release coming up soon. If you feel comfortable, feel free to also put a PR in with a fix.


To resolve issue #29 AdobeReaderDC Failing

Changed the way to obtain the latest version from "https://armmf.adobe.com/arm-manifests/mac/AcrobatDC/reader/current_version.txt" since the current version is not always a "full" version.  Instead, I'm using https://get.adobe.com/reader always displays the latest full version.

This works great to get the full installer, thanks! However if using Installomator for patch management, then the MDM will keep installing it over and over since that full installer is not necessarily the latest published version.
I modified and tested the code block with this and it seems to do the trick for both fresh install and patch management:
    adobereaderdc)
        name="Adobe Acrobat Reader DC"
        type="pkgInDmg"
        if [[ -d "/Applications/Adobe Acrobat Reader DC.app" ]]; then
        	downloadURL=$(adobecurrent=`curl -s https://armmf.adobe.com/arm-manifests/mac/AcrobatDC/reader/current_version.txt | tr -d '.'` && echo https://ardownload2.adobe.com/pub/adobe/reader/mac/AcrobatDC/"$adobecurrent"/AcroRdrDCUpd"$adobecurrent"_MUI.dmg)
        else
        	downloadURL=$(adobecurrent=`curl -s -H "User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15)" https://get.adobe.com/reader/ | grep "<strong>Version" | sed 's/[^0-9]*//g' | cut -c 3-` && echo http://ardownload.adobe.com/pub/adobe/reader/mac/AcrobatDC/"$adobecurrent"/AcroRdrDC_"$adobecurrent"_MUI.dmg)
        fi
        expectedTeamID="JQ525L2MZD"
        blockingProcesses=( "AdobeReader" )
        ;;

I believe this has been resolved in v0.3


LNN_and_CLT_issue

Hi @jstac , have adjusted the fig size and the font size of the legend of the figures in this lecture.

thanks @Harveyt47 , looks good.

Thanks @Harveyt47 @sayaikegawa @shlff , nice PR with useful reviews and good change.
Please keep an eye out for more figures like this, that are poorly sized.  (Before you make changes, it might also be a good idea to see what they look like in the PDF.)


Quest 2 stuck at searching for server

vrserver.txt
Bug reports:
The ALVR app in the Quest 2 is stuck at the hostname and searching for sever even though ALVR is properly configured
I have already asked for support in the discord
I have already tried many troubleshooting measures, I believe this is not user error.
Environement
AMD Ryzen 7 2700X
NVIDIA Geforce GTX 1660 TI
Windows 10 2004 19041.685

I'm sorry but I cannot help you if you don't give me any kind of clue on where to search for the bug. Probably you just have a misconfigured network.

I'm sorry but I cannot help you if you don't give me any kind of clue on where to search for the bug. Probably you just have a misconfigured network.

I'm sorry but I cannot help you if you don't give me any kind of clue on where to search for the bug. Probably you just have a misconfigured network.

how may I check that

I'm sorry but I cannot help you if you don't give me any kind of clue on where to search for the bug. Probably you just have a misconfigured network.

how may I check that

I'm sorry but I cannot help you if you don't give me any kind of clue on where to search for the bug. Probably you just have a misconfigured network.

I have been able to setup ADB servers on the quest 2 in the past

I'm sorry but I cannot help you if you don't give me any kind of clue on where to search for the bug. Probably you just have a misconfigured network.

I have been able to setup ADB servers on the quest 2 in the past

Try to ping for the client. https://github.com/alvr-org/ALVR/wiki/Troubleshooting#alvr-cant-see-my-headset
Anyway, I prefer to close this issue since it's too generic and not helpful.

Try to ping for the client. https://github.com/alvr-org/ALVR/wiki/Troubleshooting#alvr-cant-see-my-headset
Anyway, I prefer to close this issue since it's too generic and not helpful.


Establish Database Schema

Set up the Prisma Database using this schema document. Make sure to document the schema in the repo as well 👍

Closed by #97


Git clone fails when downloading lfs files

We are using git lfs to store the large file system. When we download our github repository which has git lfs fails very often with the error message in console.

Error downloading object: thirdparty/boost/1.60.0/lib/mac/libboost_regex.a (876c                                                                                017c66f74baa4e86743960ac99a64a44721c661b859d3e660aeda201849b)
Errors logged to C:\GRK\GIT\temp\kslocker.git\lfs\objects\logs\20160829T153659.                                                                                9984081.log
Use git lfs logs last to view the log.
error: external filter git-lfs smudge -- %f failed 2
error: external filter git-lfs smudge -- %f failed
fatal: thirdparty/boost/1.60.0/lib/mac/libboost_regex.a: smudge filter lfs faile                                                                                d
warning: Clone succeeded, but checkout failed.

Attached the log file for your reference 20160829T153659.9984081.txt
This error happens at some random place when downloading the lfs files.
This error is happening when I'm try from my office network. When I try with other network providers I don't get any error. So from network team I was asked to get them the ports to be open. Or what else has the network engineer to do to get rid of this error.

Hi @grkkarthick90:

Post https://api.github.com/lfs/RealImage/kslocker/objects/batch: dial tcp: lookup api.github.com: getaddrinfow: This is usually a temporary error during hostname resolution and means that the local server did not receive a response from an authoritative server.

That looks like a DNS issue. You can increase the timeout by configuring lfs.dialtimeout as described on the config man page.
$ git config lfs.dialtimeout 60

Hi @technoweenie Before it was failing with the error message An existing connection was forcibly closed by the remote host. And this is the first time I'm seeing the above error message in the log file. I missed to read it before upload. Attached the log file which has the error message which I get quite frequently.
20160812T155807.9907819.txt

Yeah, looks like general network instability with S3. That 52.216 address is from amazon.

@technoweenie Yes it always fails from aws S3 I have confirmed that with tracet also. What should I do to get rid of this error?

My only suggestion for the Git LFS client is tweaking the lfs.dialtimeout setting as I mentioned above. Our GitHub support team may be able to help you out with troubleshooting your connection to GitHub.

Hi @grkkarthick90, I'm going to close this issue out since it seems like the issue may be resolved. If the issue re-appears, or you have any further questions, don't hesitate to re-open this issue!


Node cache recommendations

Recommendations for setting cache manually for machines running something in addition to CockroachDB
Mention of cache flag's default in start-a-node.md

@bdarnell – Could I get you to take a quick look? Forgot whose issue this was originally but glad to get them to look at it, as well. Thanks!

This change is

Review status: 0 of 2 files reviewed at latest revision, 2 unresolved discussions.

recommended-production-settings.md, line 24 [r1] (raw file):

## Cache Size

If you run multiple applications on the same machine as a CockroachDB node, you might consider manually setting the cache size instead of using the default 25% of the available disk space.

s/the available disk space/available memory/g

start-a-node.md, line 32 [r1] (raw file):

`--attrs` | Arbitrary strings, separated by colons, relating to node-level attributes such as topography or machine capability. These can be used to influence the location of data replicas. See [Configure Replication Zones](configure-replication-zones.html) for full details.

Topography might include datacenter designation (e.g., `us-west-1a`, `us-west-1b`, `us-east-1c`). Machine capabilities might include specialized hardware or number of cores (e.g., `gpu`, `x16c`). The relative geographic proximity of two nodes is inferred from the common prefix of the attributes list, so topographic attributes should be specified first and in the same order for all nodes, for example:

 `--attrs=us-west-1b:gpu`
`--background` | Set this to start the node in the background. This is better than appending `&` to the command because control is returned to the shell only once the node is ready to accept requests.
`--cache` | The total size for caches, shared evenly if there are multiple storage devices. This can be in any bytes-based unit, for example:

`--cache=1000000000  -><PHONE_NUMBER> bytes`
`--cache=1GB         -><PHONE_NUMBER> bytes`
`--cache=1GiB        -><PHONE_NUMBER> bytes`

**Default:** Cache size is limited to to 25% of the available disk space calculated when the node starts.

Ditto.

Comments from Reviewable

Review status: 0 of 2 files reviewed at latest revision, 2 unresolved discussions.

recommended-production-settings.md, line 24 [r1] (raw file):
Previously, petermattis (Peter Mattis) wrote…
> `s/the available disk space/available memory/g`

Done.

start-a-node.md, line 32 [r1] (raw file):
Previously, petermattis (Peter Mattis) wrote…
> Ditto.

Done.

Comments from Reviewable

Review status: 0 of 2 files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Review status: 0 of 2 files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Thanks!

Review status: 0 of 2 files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


how to get procrustes_landmark_basis and canonical_mesh in face_geometry

Hello, mediapipe is really a gread job and very useful.
I'm confused about  how to how to get procrustes_landmark_basis and canonical_mesh in face_geometry ?
Do you get them by experiment ?

Hi @1278057578,
We are revisiting to older issues, Could you please let us know if you are still seeking a resolution for this issue, or if it has been resolved on your end?
Thank you!!


cql-engine-fhir FileBasedFhirProvider does not handle interval datePath

According to clinicalexpression.xsd, dateProperty may be a DateTime or Interval. If both dateProperty and dateRange are intervals then resource should be included in results if entire dateProperty interval is included in the date range. FileBasedFhirProvider currently only handles scenario where dateProperty is a datetime, not an interval.
The following code works in my testing.
`
if (temp instanceof DateTime) {
date = (DateTime)temp;
}
else if (temp instanceof DateTimeType) {
date = toDateTime((DateTimeType)temp);
}
else if (temp instanceof Interval) {
dateInterval = (Interval)temp;
}
                if (date != null && InEvaluator.in(date, expanded)) {
                    results.add(res);
                }
                else if(dateInterval != null && (Boolean)IncludesEvaluator.includes(expanded, dateInterval)){
                	results.add(res);
                }
                else {
                    includeRes = false;
                }

`

Resolved: https://github.com/DBCG/cql_engine/commit/9400f36224a2b7c5b83c4bd250933ce69d7475d9
Get the 1.2.16-SNAPSHOT version of the engine for the fix.
Thank you =)


Assign task - Assign task tooltip is not displaying email and avatar

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open the app
Click on plus and click on assign task
Enter any title and go to next step
Assign the task to user with avatar and display name set and complete the assign task process
Hover on @ user in task message and observe that tooltip is not displaying proper email and avatar

Expected Result:
App should display proper user details on task @user hover
Actual Result:
App does not display proper email and avatar on task @user hover
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[x] Windows / Chrome
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

Version Number: 1.3.75-11
Reproducible in staging?: Yes
Reproducible in production?: NO
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/78819774/ef709946-ae80-4e08-a46b-411a9cd29fe8
https://github.com/Expensify/App/assets/78819774/1cabfd1f-a00d-4875-82a1-2346a5eecccc
https://github.com/Expensify/App/assets/78819774/74378a53-b48f-41a6-b57c-c9eeb1ec78b5
Production
https://github.com/Expensify/App/assets/78819774/ff817b1a-b2e9-455e-99c1-59d6bceb217f
Expensify/Expensify Issue URL:
Issue reported by: @dhanashree-sawant
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1696260115370229
View all open jobs on GitHub

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

I'm not 100%, but this is possibly being fixed here

I dont think this is a blocker at this point, its a minor issue but also I think it might be expected as it seems like its showing display name and I remember we wanted to update the mentions to show display name. Maybe @puneetlath could confirm

Agree not a blocker

The mention display name PR is here and deployed: https://github.com/Expensify/App/pull/26877#issuecomment-1739839551
So the question for @puneetlath is whether the profile in the tooltip should contain primaryLogin as well as displayName? I think it probably should, providing it's a "known" user and that login is revealed to the viewer.

I'm not sure about this, but it might be fixed here

This PR would not be fixing this issue.

I'm going to unassign myself since this is not a blocker. I would suggest moving this to be an external issue once @puneetlath weighs in on the behavior.

yuuuup! 👍

Hm, yes I think this is a bug. A direct mention does seem to render the tooltip properly:

But the task assignee doesn't:

Actually, I have a feeling this might have the same cause as: https://github.com/Expensify/App/issues/22293#issuecomment-1744152239

Inteeeresting. So is your preference to send it external for investigation or hold on #22293?

I think you can set it External. That's just a theory, but could be totally off.

To me it looks like we are not using the same logic for the Task assginee mention vs the chat mention. there was some updates to the metion renderer here https://github.com/Expensify/App/issues/26306
Also in your screenshot @puneetlath See that in the task assignee we are using the Display name, while in the mention in the text message it's the login.

By using the Task assignee login, the display becomes consistent
const htmlForTaskPreview =
        taskAssignee && taskAssigneeAccountID !== 0 ? `<comment><mention-user>@${assigneeLogin}</mention-user> ${taskTitle}</comment>` : `<comment>${taskTitle}</comment>`;

Proposal from @dukenv0307 provides a correct solution to solve this issue, an account is missing in attributes.
:ribbon: :eyes: :ribbon: C+ reviewed

@puneetlath can you give this a secondary review please to green light and move on?

@dukenv0307's proposal looks good. You shouldn't even need the email address actually. You should be able to just use <mention-user accountid="${taskAssigneeAccountID}"></mention-user>

[x] Link to the PR: I'm unsure if should include this PR, but the feature displaying email/avatar was not designed at all in the MentionUserRenderer component. Here is the PR
[x] Link to comment: n/a
[x] Link to discussion: n/a
[x] Determine if we should create a regression test for this bug: I don’t think we need to add a regression test here.

Determine if we should create a regression test for this bug: I don’t think we need to add a regression test here.

Mind just elaborating on the "why?" here a touch?

@trjExpensify If we don't have tests for any tooltip with username and avatar, then I suggest adding this test.
Regression test proposal.

Assign the task to any user.
Enter any title and go to the next step.
Assign the task to the user with an avatar and display the name set.
Hover on the @ user in the task message and ensure the tooltip displays the proper email and avatar.

Do we agree :+1: or :-1:

Woah, why are there so many payment holds in the title of this issue? 😅
@ArekChr here's one of the regression tests for assigning a task. Perhaps we add it where I put it in bold below?

Navigate to a Group Chat
Click on the + in the compose box and select "Assign task"
Enter a a Title and a Description
Assign a user from the group that you have access to
Create the task
Verify you're navigated to the task "conversation"/report
Verify the assignee is displayed in the assignee field
Verify the tooltip displays the email and avatar of the assignee
Navigate back to the group conversation
Verify the task is displayed in the conversation history

okay, suggested that for Applause to add to the test case. Moving on to payments:

$50 to @dhanashree-sawant for the bug report
$750 to @dukenv0307 for the bug fix & #urgency bonus as per here.

@ArekChr here's one of the regression tests for assigning a task. Perhaps we add it where I put it in bold below?

Agree!


Originial function names without "M" at the end

Add a feature to export functions with their original name if they are not defined in Julia Base

maybe create an M module that defines the non-M functions (but doesnt export anything) and do something like:
using MatLang.M
M.double(2)

Yeah, that's a good idea!


Case insensitive and partial word search function needed for all searches

Case insensitive search example:  I want to find Jeff's HydroShare postcard.  I got to DIscover, input 'postcard'.  Jeff has used the keyword 'Postcard' in caps, and Discover does not retrieve the resource.
Partial work search example:  David is looking for his UCGIS2017 keyworded demo tutorial, and inputs 'UGGIS' into Discover, no resources are discovered.
Thanks!

@ChristinaB Just pushed this to a PR. In an hour or so you should be able to test it on playground.
i was able to tackled case insensitivity in the UI and REST API, however partial search is elusive...

@ChristinaB  Actually! I fixed that too. It's up on playground. Have a look.

It works! My little test is to search for 'postcard' and find @horsburgh 'Postcard' key work with 'Postcard' in the title.   This only works from My Resources because it was private.
Are there more tests that need to be done?  Other examples?
Glitch - I started trying to do this in Discover, I had forgotten that  it was private and I didn't realized the search in My Resources and Discover worked differently.  I originally thought your caps fix wasn't working.  @Castronova is making a new issue for this.

It looks like this is working from my short test.  I'll let you close, merge, and check the box on https://github.com/hydroshare/hydroshare/issues/2166


files

added 3 .py files

Three files:
http.py -- My first test, seeing if we could check naxos links with http requests. Contains a class representing a record from Sierra; used a csv from Dustin. (Could have done this as a pandas dataframe, may be preferable on a redo.) Functions organize data from csv, check urls, and create a list of b numbers that need to be suppressed or un-suppressed.
naxos.py -- parsed xmls to one mrc file. Looks like I did not do anything as far as encoding in UTF-8; not sure if that is part of the problem. I think I remember you saying pymarc didn't do that. So maybe that's why, lol. the folder "naxos_xml" is on my computer, and contains all of the individual xml files. dirlist.csv is a way-too-long csv with all of the file names. We discussed redoing this and skipping this whole step; I will work on making a more modular approach.
test_mrc.py -- my goal was then to check the mrc file to find any issues that arose either in my encoding or in Naxos's data. So far we've come across 2:

pymarc replaces spaces in data with "/" meaning that any file names/identifiers with spaces in them could cause issues.
some of the records are longer than 9999 characters, which causes issues because of the way the leader is structured.

http.py is the cleanest as far as organization goes.
--Ian


Forms : Fix FormStateCollection get operator

Summary of changes
The FormStateCollection.get(id : Int) operator function does not check if the id is part of a BaseGroupState.fieldStates. Hence, if a FieldFormElement and its BaseFieldState are part of a GroupFormElement, then the get operator returns null.
This PR fixes the above bug.

thanks @sorenoid @eri9000


Port RoutedEvent.AddOwner from WPF

Is your feature request related to a problem? Please describe.
https://docs.microsoft.com/en-us/dotnet/api/system.windows.routedevent.addowner?view=windowsdesktop-6.0
See discussion here https://github.com/AvaloniaUI/Avalonia/pull/7422#discussion_r793795693

Just FYI, we'll have to copy/paste the discussion out. Linking to review comments doesnt work for some reason.

Thanks, fixed. Apparently, thread shouldn't be marked as resolved, otherwise link won't work to it.

Some links to save time:

Avalonia RoutedEvent.cs
Avalonia RoutedEventRegistry.cs
WPF RoutedEvent.cs
WPF GlobalEventManager.cs

Looks like Avalonia's RoutedEventRegistry needs to be modified in a non-trivial way to support this.


Enable Cross-Origin Access

I tried to access HTTP API by fetch API, then failed. Browser displayed following error message in the console.

Failed to load https://jaspervdj.be/lorem-markdownum/markdown.txt: No 'Access-Control-Allow-Origin' header is present on the requested resource.

I've added the header, it should work now?

Yes, it works! Thank you!


Added Scrollbar, removed ScrollbarPainter

Scrollbars use the ScrollNotification machinery to stay in sync with a Scrollable.
Added ScrollNotificationKind so that one can distinghuish the start, middle, and end of scrolls.
Next step: remove OverscrollPainter, add OverscrollIndicator, and remove ScrollableListListener et al.

There's a tricky case here that I've guarded against.
We dispatch the onScrollEnd() and the corresponding ScrollNotification asynchronously, after the fling animation completes. That means that by the time these things run the ScrollableState may have been disposed.
I'm guarding against that possibility in dispatchOnScrollEnd():
  void dispatchOnScrollEnd() {
    assert(_numberOfInProgressScrolls == 0);
    if (config.onScrollEnd != null)
      config.onScrollEnd(_scrollOffset);
    if (context != null)
      new ScrollNotification(this, ScrollNotificationKind.ended).dispatch(context);
  }

Perhaps Notification.dispatch() should short-circuit if its target is null, instead?

Using mounted instead of checking to see if the BuildContext is null.


enhance(workspace): Hook up engine v3 with schema store

Background
This PR hooks up the engine v3 with the schema stores from https://github.com/dendronhq/dendron/pull/3348. This will support schema related methods on the engine side such as  initSchema and CRUD methods related to schemas
Changes

Implement initSchema and add schemas to notes while init
Deprecated engine.schemas in favor of engine.getSchema. Async to follow up next
Changed return signature of getNote and getSchema to return RespV3<...>. This will allow the engine to propagate the error to the clients w/o dropping it. (consistent with rest of engine methods)

Testing

Updated engine test spec file to include schemas
Manually tested against test-workspace and confirmed same number of schemas after init compared with existing engine

Looks good overall - can we wait until #3343 goes into master before merging this one?
Should we also go ahead and move over SchemaStore stuff into 'common-all'?

Yeup, I'll move them right now


Lambda function URL doesn't get the response

Hello,
I have spring boot service (with GET end point) deployed on Lambda as a function. I am able to test it in Lambda console and it returns the response. I created "Function URL" (so that i can use that as end point) to use it further from my front end app.
But, whenever i click on this function url, it launches browser and throws {"Message":"Forbidden"} error. I created function for public access [FunctionURLAllowPublicAccess]..The same lambda function works fine when i tested using 'API gateway - aws proxy' like below on lambda console:
{
  "body": "eyJ0ZXN0IjoiYm9keSJ9",
  "resource": "/{proxy+}",
  "path": "/searchAll",
  "httpMethod": "GET",
  "isBase64Encoded": true,
  "pathParameters": {
    "proxy": "/searchAll"
  },
  "stageVariables": {
    "baz": "qux"
  },
  "headers": {
    "Accept": "text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,*/*;q=0.8",
    "Accept-Encoding": "gzip, deflate, sdch",
    "Accept-Language": "en-US,en;q=0.8",
    "Cache-Control": "max-age=0",
    "CloudFront-Forwarded-Proto": "https",
    "CloudFront-Is-Desktop-Viewer": "true",
    "CloudFront-Is-Mobile-Viewer": "false",
    "CloudFront-Is-SmartTV-Viewer": "false",
    "CloudFront-Is-Tablet-Viewer": "false",
    "CloudFront-Viewer-Country": "US",
    "Host": "1234567890.execute-api.us-east-1.amazonaws.com",
    "Upgrade-Insecure-Requests": "1",
    "User-Agent": "Custom User Agent String",
    "Via": "1.1 08f323deadbeefa7af34d5feb414ce27.cloudfront.net (CloudFront)",
    "X-Amz-Cf-Id": "cDehVQoZnx43VYQb9j2-nvCh-9z396Uhbp027Y2JvkCPNLmGJHqlaA==",
    "X-Forwarded-For": "<IP_ADDRESS>, <IP_ADDRESS>",
    "X-Forwarded-Port": "443",
    "X-Forwarded-Proto": "https"
  },
  "multiValueHeaders": {
    "Accept": [
      "text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,*/*;q=0.8"
    ],
    "Accept-Encoding": [
      "gzip, deflate, sdch"
    ],
    "Accept-Language": [
      "en-US,en;q=0.8"
    ],
    "Cache-Control": [
      "max-age=0"
    ],
    "CloudFront-Forwarded-Proto": [
      "https"
    ],
    "CloudFront-Is-Desktop-Viewer": [
      "true"
    ],
    "CloudFront-Is-Mobile-Viewer": [
      "false"
    ],
    "CloudFront-Is-SmartTV-Viewer": [
      "false"
    ],
    "CloudFront-Is-Tablet-Viewer": [
      "false"
    ],
    "CloudFront-Viewer-Country": [
      "US"
    ],
    "Host": [
      "0123456789.execute-api.us-east-1.amazonaws.com"
    ],
    "Upgrade-Insecure-Requests": [
      "1"
    ],
    "User-Agent": [
      "Custom User Agent String"
    ],
    "Via": [
      "1.1 08f323deadbeefa7af34d5feb414ce27.cloudfront.net (CloudFront)"
    ],
    "X-Amz-Cf-Id": [
      "cDehVQoZnx43VYQb9j2-nvCh-9z396Uhbp027Y2JvkCPNLmGJHqlaA=="
    ],
    "X-Forwarded-For": [
      "<IP_ADDRESS>, <IP_ADDRESS>"
    ],
    "X-Forwarded-Port": [
      "443"
    ],
    "X-Forwarded-Proto": [
      "https"
    ]
  },
  "requestContext": {
    "accountId": "123456789012",
    "resourceId": "123456",
    "stage": "prod",
    "requestId": "c6af9ac6-7b61-11e6-9a41-93e8deadbeef",
    "requestTime": "09/Apr/2015:12:34:56 +0000",
    "requestTimeEpoch":<PHONE_NUMBER>000,
    "identity": {
      "cognitoIdentityPoolId": null,
      "accountId": null,
      "cognitoIdentityId": null,
      "caller": null,
      "accessKey": null,
      "sourceIp": "<IP_ADDRESS>",
      "cognitoAuthenticationType": null,
      "cognitoAuthenticationProvider": null,
      "userArn": null,
      "userAgent": "Custom User Agent String",
      "user": null
    },
    "path": "/prod/searchAll",
    "resourcePath": "/{proxy+}",
    "httpMethod": "GET",
    "apiId": "1234567890",
    "protocol": "HTTP/1.1"
  }
}

Entire Github project has uploaded here:
https://github.com/JonathanGladwin/awstest
I uploaded this JAR file in S3 bucket and uploaded it in Lambda function.
My code flow is: Spring Boot GET end point calls -> AWS RDS database and get data -> return to spring boot GET end point.
Anyone calls this Spring boot GET end point like this: http://......./searchAll - it gets the data from RDS and return via Spring boot service.

Got it fixed


Payment options displayed twice.

I'm using Mollie in the testing environment and haven't modified the module in any way shape or form. It's a fresh installation of the module.
Expected behavior
Payment options should only be displayed once.
Actual behavior
The payment options are displayed twice.
Steps to reproduce the behavior
Visit the Checkout page in PS. <IP_ADDRESS>

Environment

PrestaShop version: <IP_ADDRESS>
Mollie module version: 3.3.1

Hi @Apury ,
This is a known problem in PS 1.7. Please Go to Modules -> Positions or Design -> Positions. Remove the Mollie module from the displayPaymentEU hook and it should be back to normal.
Please let us know if this solves your issue.
Kind Regards

Thank you! This indeed solved the problem.

Hi @Apury ,
Thank you for taking the time to inform us. Have a great evening.
Kind Regards


maintainers: create sage team

Motivation for this change
Maintaining all the packages whose updates might break some aspect of
sage.
For reference:
https://github.com/NixOS/nixpkgs/pull/103810#issuecomment-727536510
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
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 nixpkgs-review --run "nixpkgs-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)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@7c6f434c do you want to be a member of that team?

For some reason my local eval check is failing, but I don't see what is wrong. Other teams are used in the same way. Checking if ofBorg also gives an error.

@7c6f434c do you want to be a member of that team?

Right now I will not be a good one… I have forgotten all the Sage stuff from my failures, and I am usually a bit behind on reacting to things…
But could join with a hope of providing a fallback when fixing some of the dependencies.

But could join with a hope of providing a fallback when fixing some of the dependencies.

More eyes are always better. I'll probably also take on more of an advisory role, at least for the near future. The biggest risk I see with teams is dilution of responsibility, but it should be manageable.
Does https://github.com/NixOS/nixpkgs/pull/104083/commits/b7319d2d35e5f50e777723b3b1c448ad05dc9e9a look good do you @7c6f434c?
@omasanori please also explicitly approve the PR if you are fine with it. I don't want to add anybody against their will ;)

@timokau I am definitely OK with not being maintainer on those things. I am also OK as being listed in the team; I guess for now it has the benefit of me automatically being notified of changes approved by @omasanori so I can merge them (I read what I merge, but I always trust other people's claim of testing, and mostly trust some people's claims of impact evaluation). If both of us have spotty availability I guess it's better to flip the coin twice?

Sorry for my late response.

I'd say that was quite fast :)

I read what I merge, but I always trust other people's claim of testing, and mostly trust some people's claims of impact evaluation.

Similar for me.

I am also OK as being listed in the team

Alright, let's go ahead with this then!

I think you ended up not adding me, at least judging from GitHub diff; I am fine with it if it was your intent; otherwise I guess I wonder if it is mixing up what you committed and what you did not, or GitHub failing at diffs, because the latter case would be… something to know, but no longer unimaginable

Right, I didn't stage my changes to team-list.nix when making the second commit! Thanks for noticing, I'll fix that up.


links: generate self_html links here [more]

This responsibility was simply moved from records-resources
and drafts-resources.
part of https://github.com/inveniosoftware/invenio-app-rdm/issues/232

Coverage increased (+0.2%) to 90.363% when pulling fe9a14764785583aee84a10a1b5cc5414e6212fa on fenekku:232_implement_ui_link_generation into 69b3ed3e51c2fa6b00093c96413ba304e15512e8 on inveniosoftware:master.


fix: make restart can restart stopped container

Signed-off-by: Allen Sun<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
Make pouch be able to restart a stopped container.
Ⅱ. Does this pull request fix one issue?

fixes #1206
Ⅲ. Describe how you did it
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #1208 into master will not change coverage.
The diff coverage is 0%.

@@           Coverage Diff           @@
##           master    #1208   +/-   ##
=======================================
  Coverage   15.59%   15.59%
=======================================
  Files         178      178
  Lines       10441    10441
=======================================
  Hits         1628     1628
  Misses       8693     8693
  Partials      120      120

Impacted Files
Coverage Δ

daemon/mgr/container.go
0% <0%> (ø)
:arrow_up:


dataset

Dear akaraspt：
Hello，I am working on sleep stages。
Best regards!
Jianji  Zhang

You need to send a request to the authors of the dataset. Then they will advise you what are the documents you need to submit for access.
http://massdb.herokuapp.com/en/

Thank you for answering my letter so quickly.  I  will try  to get the dataset by the adress you provided.
THANK  YOU !


vSphere provider does not allow NSX-T configuration

Rancher Server Setup

Rancher version: v2.6.0

Describe the bug
The vSphere cloud provider supports NSX-T Load Balancers and is no longer feature flagged as alpha. These config options aren't well documented but you can see them here: https://github.com/kubernetes/cloud-provider-vsphere/blob/d5f062858935b9c50b0e8e53463b6985ca34494c/pkg/nsxt/config/types_common.go#L20
Currently the chart for rancher does not allow you to set any of these settings for using NSX-T load balancers: https://github.com/rancher/charts/blob/dev-v2.6/charts/rancher-vsphere-cpi/rancher-vsphere-cpi/100.0.0/templates/vsphere-cloud-config-cm.yaml
Expected Result
The Rancher vSphere provider chart should allow NSX-T configuration. Load Balancers should be able to be used.
Additional context
https://github.com/kubernetes/cloud-provider-vsphere/pull/425

Bump

Double bump

Bump

Bump

Bump if anyone can open this


Optional sync futures, async block

This change adds a virtual #[future_is[BOUNDS]] attribute for async fns so you can opt in for Sync or 'static future
Refactored to async move block instead of inner function as it removed a whole lot of complexity and enabled the 'static stuff
closes #77

Hi @dtolnay, kindly review.
Currently when I take my fork as a dependency it fails on syn::* types equality. Any idea? While all is green in async-trait.

binary operation == cannot be applied to type syn::Path

Solved thanks to @link2xt - added "extra-traits" feature for syn.
Meanwhile, added docs and tests, updated README.md

Hi @dtolnay, thanks for the feedback.
From your example, I understand that an unused value is dropped before the async block is executed. A useful optimization in the discretion of the compiler without effect on functionality. Perhaps the next version of rust will optimize the async fn similarly...
What would happen if you used the variable in the printout or explicitly dropped it after?

I think I'll close for now, since this is pretty hard to review with the unrelated changes included together. Please open a new PR without drop order changes or the readme changes.

The improvements plus better support for 'static futures are available in a fork samotop-async-trait


Is this repo still alive ?

Hey @Dadoum, unfortunately not. I currently don't have the bandwidth to maintain it. Are you interested in taking it over ?

I can try to make a PR but can't promise anything

@Dadoum sure thing, feel free to make the PR. I'll review when I get time and merge :)


Enable hide/echo during setup for passwords

Sometimes it is better when you enter passwords when nobody can see them in clear text. This is the case when you do the setup together with other people, have a screen share or screen recording running.
But most times it is better to see what you type, because then you are sure that the password is correct.
Thus, it could become a switch when running the setup command or you could be prompted whether you want to see the passwords or hide them during the setup procedure.

can be controlled with a command line switch --show-passwords on setup.cmd or setup.sh


[4083] rework missing data display for hesa records

Context
We currently suppress the 'incomplete' badge for HESA records. I'm not sure this is correct - as when you visit the record we still show the blue inset 'missing data' and highlight it in the banner.

Allow hesa records to be incomplete
Any fields on hesa records that are optional should have a message displayed in the trainee summary views
Optional hesa fields:
itt_qualification_aim
reason_for_leaving
employing_school_urn
lead_school_urn
service_leaver
trn

Guidance to review
Make a created from hesa trainee with some missing fields above.
Check that the correct error message is loaded and the record is not classed as incomplete.
Check that a created from hesa trainee with an incomplete field not depicted above can be shown as incomplete.

 Rework 'missing data' display for HESA records.

@richardpattinson can you link to some trainees in the review app where this can be demoed?

@richardpattinson can you link to some trainees in the review app where this can be demoed?

Trainee with slug: JKgFMbSjr4vxGsWskvA7dAym has 4 fields missing and is a hesa trainee

@richardpattinson can you link to some trainees in the review app where this can be demoed?

Trainee with slug: JKgFMbSjr4vxGsWskvA7dAym has 4 fields missing and is a hesa trainee

4nSvDN6NE5tjrdK81AgpP2UQ > non hesa trainee with the same missing fields


Feature request: dynamic menu bar icon

Minor feature request: make a dynamic menu bar icon that is grey (and maybe has the switch in the off position) when no proxy is active. Something like this:

I can make the icon for you if it helps.

Hi @oscu0,
Thank you for the great idea you've shared. However, I regret to inform you that since this application is developed using the Flutter framework, it doesn't have easy access to native operating system features, making this task not easily achievable. Nonetheless, I will try to implement it in the future for sure.
By the way, here's some exciting news: I've just released a new version of the application which includes the feature of bypass list. You can download it through the release page.
Cheers,
Zhia


Laggy/delayed animations

I use the latest flutter and dropdown_search version. The package runs slow like there is a delay on animations which feels bad. Here is the comparison video with flutter's own dropdown:
https://github.com/user-attachments/assets/b4d0ccf5-dc8d-470b-af25-c53c06721839

By default there is a delay of 300 milliseconds.
You can change it by setting  duration to Duration.zero in popUpAnimationStyle properties of menuProp.

Of course, the configuration and customizability you offer is unmatched. Not to mention the speed with which you help. Thank you for this wonderful library.
By the way, 'decoratorProps' in reactive_dropdown_search library is incompatible with Mode.custom. It needs to be changed like this:
decoratorProps: mode == Mode.custom
? null
: DropDownDecoratorProps(

Making the duration 0 solves the problem, but there is a general problem with the animation. I couldn't manage to fade in and fade out slowly, the animation just doesn't work. The menu opens immediately and disappears late. I don't think the setting below works:
popUpAnimationStyle: AnimationStyle(
curve: Curves.ease,
duration: const Duration(seconds: 1),
reverseCurve: Curves.ease,
reverseDuration: const Duration(seconds: 1),
),

reactive_dropdown_search library is not mine.
DropdownSearch is already reactive and no need to state management to update it's state and you can manage it programmatically using GlobalKey.
For the animation, in the current version there is no real animation feature. But this is added as a feature in the next version as mentioned here

Actually its just a wrapper for reactive_forms support. Thanks for the information.


fix: fix @tsed/logger version on each package.json

:tada: This PR is included in version 6.133.2 :tada:
The release is available on:

v6.133.2
GitHub release

Your semantic-release bot :package::rocket:


instantiateRenderEntity sets up render assets on render components

When an Entity hierarchy was created using GlbContainer.instantiateRenderEntity, the render component didn't have .asset property set up. This is now fixed, and those properties point to a render assets.

Wait, what's the type of the resource of a render asset? e.g. entity.render.asset?

Wait, what's the type of the resource of a render asset? e.g. entity.render.asset?

the code I added assigns Asset type to it created here:
https://github.com/playcanvas/engine/blob/d5bef0eed6b3d1f1c8970b6d33e089139d39773e/src/resources/parser/glb-container-resource.js#L58-L65
But internally this is stored as an id on the component, and render.asset returns asset.id

I meant, is the asset's resource a Render? Just wondering, because that type is not public API.

I meant, is the asset's resource a Render? Just wondering, because that type is not public API.

Fair point. Making the Render class public.

Are developers expected to create Render objects themselves? Or do anything with them? Is the Render API in 'final' form?

I'm hiding the constructor as well .. as there's not a big use case for its public use.
Otherwise only meshes properties is exposed .. and I think that's useful to have.

Do we need to make Render class public API?
Looks like instances of Render are created by the glb-parser at load time and then used by the render component internally (via asset.resource).
Even though the Render class seems simple an innocuous enough,  it's not actually possible to set a Render instance directly on the RenderComponent anyway. So seems like we can leave `Render private?

I'd probably agree with that assessment. Making Render public is not necessary to solve the original motivation for this PR.


cache component may cache expired data

Describe the bug

From the above figure we can see that in extreme cases the cache will cache expired data
To Reproduce
go func() {
  c.Take(key, val, func() (interface{}, error) {
    val := getFromDB(ctx)
    // Perform some time-consuming operations, or GC jitter.
    time.Sleep(time.Second)
    return val
  })
}()

go func() {
  updateDB()
  c.Del(key)
}()

Expected behavior
Dont cache expired data
Environments (please complete the following information):

OS: Macos
go-zero version 1.5.0

I talked about this scenario in my presentations. It's a rarely happened case. It's hard to solve it well without bringing the system too complicated.

Using updating placeholders can solve this problem. I submitted an MR, you can see if it is feasible.

What if the parts go into the place where arrow points to?

@kevwan


No "verb" targets found

Need a sanity check. The readme for grunt-verb says there is no configuration required. When I install and attempt to run the verb task without adding a config to my Gruntfile.js I get a "No verb targets found." response.
$ npm install grunt-verb --save-dev
$ grunt verb
...
> Running "verb" task
> [D] Task source: /Users/jd/Projects/4tr/node_modules/grunt-verb/tasks/verb.js
>> No "verb" targets found.
> Warning: Task "verb" failed. Use --force to continue.

The readme also says that I can use the Verb generator to kickstart the documentation for any project, so I tried that as well, but got the same result.
$ npm i -g generator-verb
$ npm install grunt-verb --save-dev
$ yo verb
> ? Project name? foo
> ? Description? Foo's description
> ? Author's name? John Smith
> ? Author's URL? https://github.com/jsmith
> ? GitHub username/org? jsmith
>    create LICENSE-MIT
>    create .verb.md
>   create CONTRIBUTING.md

$ grunt verb --verbose --debug
...
Registering "grunt-verb" local Npm module tasks.
Reading /Users/jd/Projects/4tr/node_modules/grunt-verb/package.json...OK
Parsing /Users/jd/Projects/4tr/node_modules/grunt-verb/package.json...OK
Loading "verb.js" tasks...OK
+ verb
Reading package.json...OK
Parsing package.json...OK
Initializing config...OK
Loading "Gruntfile.js" tasks...OK
+ build, changelog, cover, default, doc, integration, jsdocs, polish, readme, saveRevision, server, tagRevision, test, unit, verify, version, yuidocs
...
Running "verb" task
[D] Task source: /Users/jd/Projects/4tr/node_modules/grunt-verb/tasks/verb.js
>> No "verb" targets found.
Warning: Task "verb" failed. Use --force to continue.

I dug into grunt-verb task's code, and at the bottom it looks like the task's default config file is ".verbrc.md", but according to the output of yo verb it created ".verb.md".
// /node_modules/grunt-verb/tasks/verb.js
...
    grunt.config('verb', {
      readme: {
        files: [
          {src: ['.verbrc.md'], dest: 'README.md'},
          {expand: true, cwd: 'docs', src: ['**/*.tmpl.md'], dest: '.', ext: '.md'},
        ]
      }
    });
...

So, I tried changing my .verb.md to .verbrc.md, thinking that the task might now find the correct init file. No joy though. I still get the "No verb targets found." response.
There are some grunt tasks (like grunt-build-number) that require an entry in the Gruntfile.js even when you don't alter the defaults. So, I tried adding a verb block....
// Gruntfile.js
    verb: {
      readme: {}
    },

And running task again produces...
Registering "grunt-verb" local Npm module tasks.
Reading /Users/jd/Projects/4tr/node_modules/grunt-verb/package.json...OK
Parsing /Users/jd/Projects/4tr/node_modules/grunt-verb/package.json...OK
Loading "verb.js" tasks...OK
+ verb
Reading package.json...OK
Parsing package.json...OK
Initializing config...OK
Loading "Gruntfile.js" tasks...OK
+ build, changelog, cover, default, doc, integration, jsdocs, polish, readme, saveRevision, server, tagRevision, test, unit, verify, version, yuidocs

Running "verb" task
[D] Task source: /Users/jd/Projects/4tr/node_modules/grunt-verb/tasks/verb.js

Running "verb:readme" (verb) task
[D] Task source: /Users/jd/Projects/4tr/node_modules/grunt-verb/tasks/verb.js
Verifying property verb.readme exists in config...OK
File: [no files]
Options: sep="\n", ext=".md", data=["docs/*.{json,yml}"], prefixBase, cwd="/Users/jd/Projects/4tr", destBase="/Users/jd/Projects/4tr", docs="docs"

Done, without errors.

This time around, the verb task was found, and it tried to run, but it didn't find any files in its default options, so I tried to set some up...
// Gruntfile.js
    verb: {
      readme: {
        files:[
          {
            src: [ '.verbrc.md' ],
            dest: 'README.md'
          }
        ]
      }
    },

And now I get an error 'test/fixtures/comment.tmpl.md' not existing.
Running "verb:readme" (verb) task
[D] Task source: /Users/jd/Projects/4tr/node_modules/grunt-verb/tasks/verb.js
Verifying property verb.readme exists in config...OK
Files: .verbrc.md -> README.md
Options: sep="\n", ext=".md", data=["docs/*.{json,yml}"], prefixBase, cwd="/Users/jd/Projects/4tr", destBase="/Users/jd/Projects/4tr", docs="docs"
Reading .verbrc.md...OK
Warning: ENOENT, no such file or directory 'test/fixtures/comment.tmpl.md' Use --force to continue.

Aborted due to warnings.

I checked the verbrc.md and found no reference to 'test/fixtures/comment.tmpl.md', so I'm assuming that error was thrown from inside the verb module.
This is where I'm at now. Have I gone through the steps incorrectly?

I just tried to trace what happens to the defaults. Inside of /node_modules/grunt-verb/tasks/verb.js I dumped verb.options out to the console in a few different places:

At the bottom of the file where you test !grunt.config('verb')
At the top of the file just before verb.options = _.extend(verb.options || {}, options);
At the top of the file just after verb.options = _.extend(verb.options || {}, options);

The output is below. It looks like the grunt.config('verb') at the bottom isn't getting persisted or isn't getting passed to verb.options, because when the task runs verb.options is undefined.
The grunt.config('verb') call isn't being seen by Grunt for some reason, which is why the "No "verb" targets found" error gets thrown when there isn't a verb config in Gruntfile.js.
Registering "grunt-verb" local Npm module tasks.
Reading /Users/jd/Projects/4tr/node_modules/grunt-verb/package.json...OK
Parsing /Users/jd/Projects/4tr/node_modules/grunt-verb/package.json...OK
--------------------------------------------------------------------------------
>>>>>>>>> Right after if (!grunt.config('verb'))
No grunt config found using defaults.
Inspect grunt.config :: { [Function]
  data:
   { verb:
      { readme:
         { files:
            [ { src: [ '.verbrc.md' ], dest: 'README.md' },
              { expand: true,
                cwd: 'docs',
                src: [ '**/*.tmpl.md' ],
                dest: '.',
                ext: '.md' } ] } } },
  escape: [Function],
  getPropString: [Function],
  getRaw: [Function],
  get: [Function],
  process: [Function],
  set: [Function],
  merge: [Function],
  init: [Function],
  requires: [Function] }
--------------------------------------------------------------------------------
Loading "verb.js" tasks...OK
+ verb
Reading package.json...OK
Parsing package.json...OK
Initializing config...OK
Loading "Gruntfile.js" tasks...OK
+ build, changelog, cover, default, doc, integration, jsdocs, polish, readme, saveRevision, server, tagRevision, test, unit, verify, version, yuidocs

Running tasks: readme

Running "readme" task
[D] Task source: /Users/jd/Projects/4tr/Gruntfile.js

Running "verb" task
[D] Task source: /Users/jd/Projects/4tr/node_modules/grunt-verb/tasks/verb.js

Running "verb:readme" (verb) task
[D] Task source: /Users/jd/Projects/4tr/node_modules/grunt-verb/tasks/verb.js
Verifying property verb.readme exists in config...OK
Files: .verbrc.md -> README.md
Options: sep="\n", ext=".md", data=["docs/*.{json,yml}"], prefixBase, cwd="/Users/jd/Projects/4tr", destBase="/Users/jd/Projects/4tr", docs="docs"
--------------------------------------------------------------------------------
>>>>>>>>> Before  verb.options = _.extend(verb.options || {}, options);
Inspect initial verb.options :: undefined
--------------------------------------------------------------------------------
>>>>>>>>> After  verb.options = _.extend(verb.options || {}, options);
Inspect using verb.options :: { sep: '\n',
  ext: '.md',
  data: [ 'docs/*.{json,yml}' ],
  prefixBase: true,
  cwd: '/Users/jd/Projects/4tr',
  destBase: '/Users/jd/Projects/4tr',
  docs: 'docs' }
--------------------------------------------------------------------------------
Reading .verbrc.md...OK
Warning: ENOENT, no such file or directory 'test/fixtures/comment.tmpl.md' Use --force to continue.

I was able to get it to work by setting up the verb task in the Gruntfile.js and stripping the content of .verbrc.md down so that it only references items that actually exist in the project.
Gruntfile.js
    verb: {
      readme: {
        files:[
          {
            src: [ '.verbrc.md' ],
            dest: 'README.md'
          }
        ]
      }
    },

.verbrc.md
# {%= name %}

> {%= description %}

## Install
{%= include("install-npm", {save: true}) %}

wow, thanks so much for the detailed description and research into the problem. I'll try to look into it this weekend to see what the issue is.

doh, I think (hope) it's as simple as updating grunt-verb to read .verb.md files? Glaring oversight on my part

No worries. Yes, aligning .verb.md and .verbrc.md across the verb projects will resolve one of the issues.
The issue of the default verb task not being loaded was my fault. I was calling grunt.loadNpmTasks('grunt-verb'); before grunt.initConfig() in my Gruntfile. As a result, the default verb config block your task create was wiped out before the task was called and that resulted in the No "verb" targets found error. Moving the task loader below the initiConfig call resolved the problem.
Even with all of that sorted, I was still getting a compilation error. I traced the error down to the line {%= comments("index.js") %} in the .verbrc.md file that generator-verb created by default. When left in, it produces the error:
Warning: ENOENT, no such file or directory 'test/fixtures/comment.tmpl.md' Use --force to continue.

perfect, this is great feedback. thanks again for tracking everything down. I think I know what needs to be done to get all of it fixed

to get all of it fixed
@jonschlinkert so, is this fixed?


Why can't we use AugmentedFaces with Back camera?

I was excited about the release of AugmentedFaces but I don't understand the limitation around it only supporting the front camera in Unreal?  What limitations exist with the back camera that the front camera solves?
I'm building from source with v1.7, so if there's a small change I can make for prototyping (even though not supported for production) , I'd be willing to do it.

ARCore doesn't support Augmented Faces for back camera yet. It's not just
Unreal. Unfortunately you will have to wait until ARCore supports it.
On Thu, Mar 7, 2019 at 9:54 PM Alex Petrescu<EMAIL_ADDRESS>wrote:

I was excited about the release of AugmentedFaces but I don't understand
the limitation around it only supporting the front camera in Unreal? What
limitations exist with the back camera that the front camera solves?
I'm building from source with v1.7, so if there's a small change I can
make for prototyping (even though not supported for production) , I'd be
willing to do it.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google-ar/arcore-unreal-sdk/issues/54, or mute the
thread
https://github.com/notifications/unsubscribe-auth/ASYXB7Q-xe0Ivp4ueaj-oqPN-_fx3sCaks5vUfsDgaJpZM4bkszD
.

Is this a feature coming soon or not on the roadmap?  I would like faceboxes at least like ARKit.

I wish the same on Unity too.
Why is the back camera not enabled? The option is present in Unity but when running the app error message "ARCore encountered a problem connecting. Please restart the app" appears. Restarted yields same result.
I think it is to do with GPU and CPU camera configurations.
Building on Galaxy S8

may 2020  i have tested  with unreal change cam front to back and ..... always front camera

May 2020. Thank you! Will look at that.
If I am not mistaken: So far face tracking for Android in Unity : AR Core (and therefore AR Foundation) only working on front/selfie  camera. Back/world camera not enabled yet.

greatAznur- did you have to do anything special to get Unreal to track from back/world cam? What device were you building to? Cheers

@timneill40
there is an option in the project to say which camera we choose but in any case it doesn't work for "augmented face project" ..


[1624] Refer to @provider instead of course.provider in html

Context
Refactoring to prepare for validations.
Changes proposed in this pull request
The necessity of this is because when you do a @course.update, and it fails, the @course.provider is blatted away because the response from the backend API does not contain a provider, only course info. The provider info is still available in the @provider though.
This is also terser.
Guidance to review
I have not manually tested this and am relying on the fact that none of these pages are live yet and mostly have coverage.

 Display validations when saving enrichments


Microoptimizations of contraction logic code

This adds some microoptimizations to the contraction logic code. The main strategy is minimizing the number of getfield calls to the ContractionProperties mutable struct by avoiding things like props.Acstart in loops. Here is a benchmark:
using BenchmarkTools
using ITensors

N = 6
d = 1
i = Index(QN(0, 2) => d, QN(1, 2) => d)
is = IndexSet(n -> settags(i, "i$n"), N)
A = randomITensor(is'..., dag(is)...)
B = randomITensor(is'..., dag(is)...)

@btime $A' * $B samples = 5

on this branch gives:
julia> include("time.jl");
  115.340 ms (868058 allocations: 138.49 MiB)

while on master:
julia> include("time.jl");
  170.253 ms (1261274 allocations: 141.49 MiB)

This case is pretty extreme since it involves contracting many very small blocks, so the contraction logic really dominates.
Most of the rest of the time spent in this example is block sparse logic like creating the block-offset list for the output tensor.

Codecov Report

Merging #53 (6170aa5) into master (649930a) will increase coverage by 1.99%.
The diff coverage is 48.24%.

@@            Coverage Diff             @@
##           master      #53      +/-   ##
==========================================
+ Coverage   34.87%   36.87%   +1.99%
==========================================
  Files          22       22
  Lines        2523     2251     -272
==========================================
- Hits          880      830      -50
+ Misses       1643     1421     -222

Impacted Files
Coverage Δ

src/blocksparse/blocksparsetensor.jl
49.32% <0.00%> (+1.50%)
:arrow_up:

src/contraction_logic.jl
48.49% <49.10%> (-1.08%)
:arrow_down:

src/dims.jl
26.31% <0.00%> (-5.27%)
:arrow_down:

src/tensorstorage.jl
64.28% <0.00%> (-2.39%)
:arrow_down:

src/tupletools.jl
7.14% <0.00%> (-1.73%)
:arrow_down:

src/blocksparse/blockdims.jl
75.00% <0.00%> (-1.32%)
:arrow_down:

src/linearalgebra.jl
31.54% <0.00%> (-1.24%)
:arrow_down:

src/diag.jl
11.24% <0.00%> (-0.32%)
:arrow_down:

src/svd.jl
0.00% <0.00%> (ø)

... and 13 more

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 649930a...6170aa5. Read the comment docs.


CLI improvements

This PR consolidates the server and client binaries into the main mina-indexer binary and

provides server and client subcommands
adds an initial state Summary struct
adds best_tip field to IndexerState for storing the best block
expands the available client commands with best_ledger and summary
modifies the best_chain command to take a --num arg
modifies some log messages/levels
adds release LTO
removes server cli flag --log-stdout and makes this the default unless --log-dir is specified
adds server cli flag --db-override

no implementation
intention: delete the db on the path provided by --database-dir if set, defaults to initializing the indexer state from the db

To see the specific tasks where the Asana app for GitHub is being used, see below:

https://app.asana.com/0/0/1204807559875974
https://app.asana.com/0/0/1204758919833965

I replace chrono for time


Simulated drone gives bad SLAM results

Hi,
I have been using Cartographer for a few different things and want to thank you for making this great tool open source. I am trying to simulate a drone mapping a certain cube-like structure, and to send the simulated lidar data to Cartographer for SLAM. I managed to simulate the drone circling this structure, and to run Cartographer with that bag, and it worked fine.
However, now I want the drone to fly in a cross-like pattern over the structure and map it from above. And suddenly, it doesn't seem like the SLAM catches on anymore: with odometry data provided it gives a reasonable trajectory but still drifts a lot, and without odometry it does not run at all. I have tried several things: making the drone much slower (which helped a lot), giving it an unrealistically large lidar swath so that it never loses sight of the structure during mapping (which changed nothing), tuning... But the trajectory and pointcloud outputted by Cartographer still aren't good enough. Maybe the SLAM algorithm has more trouble when the plane in which the drone moves is perpendicular to the plane in which the sensor is oriented. Otherwise I'm probably doing something wrong, and I would be very grateful for any kind of help.
I ran rosbag_validate, and here is a link to the required Git repo.
The structure the drone is mapping is confidential, therefore here are two bags in which I replaced this structure with something similar found on the internet. One of the bags is a simulation of a regular swath lidar, the other of a very wide swath for debugging purposes.
Thank you for your help!

In short, Cartographer currently does not support flying robots the way it extrapolates IMU input.
Currently, PoseExtrapolator and ImuTracker use IMU acceleration input only to extract gravity direction using a low-pass filter. Velocity is estimated by the difference of the last two scan matches or by odometry input. Yes, we decided against a Kalman filter. (Perhaps this explains why it cannot detect motion perpendicular to the plane in your case.)
We found this extrapolation approach best for mobile robots, cars, and sensors that are carried/moved around.
To work with 3D laser scans from a flying vehicle, you would need to fork and make large changes to PoseExtrapolator and ImuTracker. 2D scans cannot be used with flying vehicles.
Concerning synthetic data, we found results gathered with synthetic data sometimes not very realistic. (Some simulation may be more realistic than other.)
Choosing the right algorithm and tuning should ideally be based on measurements.

Thank you for your help. If I may, it should probably be mentioned somewhere that Cartographer can only be used as is for "rolling" robots that more or less only move inside a 2D plane.
If I understood well, Cartographer does not support robots with 6 degrees of freedom because only pose and motion are estimated, but the attitude of the vehicle never is (would necessitate using tilt from odometry or full IMU data) . Is that what you think the problem is? Do you think the fact it cannot detect motion perpendicular to the plane comes from this lack of attitude estimation? Or could it rather come from the way the submaps are constructed, for example if there is an assumption somewhere stating they can only be around the robot and not under/over it? I would like to find out exactly what part of the algorithm gets lost when mapping under/over a robot, and if it does not come from the lack of attitude estimation and the lower accuracy of PoseExtrapolator then it is probably linked to scan matching.
Otherwise I agree that 2D scans cannot be used with flying vehicles (we only really use 3D) and that eventually we should use real measurements, but we were doing ok with the simulations as long as the drone moved more or less in a plane and was mapping around itself, comparable to a rolling robot.

How about adding Kalman filtering?
Cartographer did use an unscented Kalman filter in the past, it was removed in
https://github.com/googlecartographer/cartographer/pull/378 and replaced by the current PoseExtrapolator approach. Both solutions are computationally efficient, but the current gives better results in practice. So, we are very hesitant to add a Kalman filter now.

Can the 3D pipeline work with 3D scans only under the camera?
The 3D pipeline uses HybridGrid, which is not just layers of ProbabilityGrid, but a voxel grid where all three dimensions are treated equally. So the data structure has no limitation here, they are "true 3D occupancy grids".
Scan matching in 3D does compare accumulated range data with a HybridGrid and then adds it to this and the previous one. You are right there is no grid-to-grid matching, but still this is unrelated to the question.
Yes, cartographer can very well recognize up-and-down motion. I would actually argue that laser range measurements pointing down will give very precise up-and-down results.
As I said, the only limitation is that accelerometer data is not used to predict translation.
Also, the angle of the 3D scanner should be wide in both directions, not just 4 stripes close to each other.

Generally, we would be glad to receive an open-source bag file with real data collected by an aerial vehicle, try to tune it and discuss good config parameters. It is always good to receive example data that is high quality and different from our exisiting scenarios.
I'd expect that for small, shaky quadrotors, IMU measurements would need to be high frequency and in sync with lidar time stamps.

Thank you very much for your extensive answer! It is hard to know how 3D Cartographer works exactly since, to my knowledge, there is no paper explaining how it went from 2D to 3D.
After taking a closer look at the code, I can see that 2D mapping uses a 2D ProbabilityGrid but 3D uses a HybridGrid and therefore a real 3D occupancy grid, I missed that before. Great, that probably means that the use of IMU data is indeed the only thing stopping us from using Cartographer with a drone! We will implement the Kalman filter to use accelerometer data and see if that helps, we'll come back to you if any new problems come up.
As for the data, if you are interested in it as soon as we start doing real experiments mapping non-confidential structures I can post it here and we can discuss tuning and configuration!

Hi,
We now use the old UKF version of Cartographer in parallel with the current version, have aerial (drone) and underwater (rover) simulations and run Cartographer with the simulated lidar data. In the simulations the vehicle maps a certain structure, either by spiraling around it, or by going over it. Most of the time the UKF version of Cartographer performs much better.
So I was just wondering: why exactly did you give up on the UKF and went to a much more basic state estimation? You said that gave better experimental results, I suppose that's only true for slow dynamics, more or less planar movements like with the bagpack? Did you ever figure out why the simple estimation works better than the UKF for you in those cases? I would very much like to know in which cases the UKF works better than the current Pose Extrapolator and why, so if you have any ideas or explanations or can tell me precisely why you chose to give up on the UKF that would be great!
Thanks for your help!

Hi @myavari
My quick fix for using Cartographer on a drone was to go back to an earlier version of the code, where a UKF was used in Pose Tracker for estimating the state of the vehicle, instead of the current Pose Extrapolator.
I used the versions of Cartographer from approximately June 30th 2017, commit number 17a22ed of cartographer and more or less corresponding 125aee3 of cartographer_ros. It worked rather ok, not as well with 3D than with 2D motion but still ok. Certain complex scenarios had trouble and I ended up more or less giving up on it, but if you want to do rather simple scenarios where the drone looks around itself and doesn't go too fast you should be fine.
Hope that helps!


Don't throw exceptions when write() returns false

Fixes #8.
This is a quick fix that ignores write's return value. This means that the buffer may grow arbitrarily large, but I expect that this will not be a problem in practice, since requests are not submitted to the language server very quickly.

Thanks for this @wilcoxjay. I tested locally and I can confirm things work better!


The repl download messes with encodings

Describe the bug
Typing some umlauts such as äöü in the REPL's App.svelte and downloading it will mess up the umlauts to gibberish
To Reproduce
https://svelte.dev/repl/cefce44e52ee42a79123e6c23d108b54?version=3
Download, build and start to see the text
Expected behavior
öäü can be used and don't need to be fixed afterwards via encoding conversions
This looks correct:

This is what it really looks like when downloaded:

closing since the site has been rewritten. please file at https://github.com/sveltejs/svelte.dev/issues/new if the issue persists


Some workaround to get rid of of "Unused import" hints

The high number of "Unused imports" hints renders the "Dart Analysis" window in WebStorm useless. It is already of limited value without these "Unused imports" hints because it updates automatically and any selections, scrolling, expand/collapse are lost every few seconds. There are open issues to improve it though but doesn't seem to have priority.
What about adding a member like const silentImportElementName = 0; in every elements Dart file and adding a a doc-comment in the importing file like /// silentImportElementName]` where no member of the imported library is used?

I really would like to avoid adding actual code to get rid of analysis warnings, even if dart2js would be able to easily remove them in the deployed app.
The custom-element-apigen generator doesn't use the analyzer at all or do any resolution, so it doesn't know what it is or is not actually using, so that makes solving this problem difficult.
I agree this is annoying though.

I didn't assume that the analyzer is used but I assumed that the script usually should know what it adds the import for. If it is just for the HTML it should add it to the doc comment. I did something similar for generating the stubs for the demos. If it's too complicated than it's probably better to leave it as it is and wait for the analyzer to provide a solution.

The way it actually works is just by looking at the original html imports, it just converts those all to dart imports. There is some global knowledge though of what imports contain what classes/etc, so it could be smarter.

Getting rid of half the hints would already be a notable improvement.

I guess this dart-lang/sdk#25685 (already landed in BE) will solve this.


#895 helpers for PUT methods in testing framework

Providing helpers for testing PUT requests with Revel testing framework

Hi, @grsmv. Thank you for your contribution. We have already had PUT method: https://github.com/revel/revel/blob/develop/testing/testsuite.go#L121
#781 is a related PR.

Yep, definitely. But io.Reader is not actually better solution to imitate sending a form, so maybe we still need PutForm method?

On Mar 16, 2015, at 07:34, anonx<EMAIL_ADDRESS>wrote:
Hi, @grsmv. Thank you for your contribution. We have already had PUT method: https://github.com/revel/revel/blob/develop/testing/testsuite.go#L121
#781 is a related PR.
—
Reply to this email directly or view it on GitHub.

@grsmv Oh, sorry, my mistake. You also added PutForm, I didn't notice that. Yeah, I think it's a valuable thing which is worth being merged. But you will have to target the develop branch per Revel's Contributing Guidelines.

Thank you. I'll close this and reopen #897

:+1:


cast prediction col to double for LinearRegression and RandomForest

This PR keeps align with the pyspark counterparts for prediction datatype.

build


DocumentIntelligenceClient has no begin_analyze_document_from_url

Please provide us with the following information:
I am using Document AI for extract information from Invoices:

from azure.ai.formrecognizer import DocumentAnalysisClient
This one is working fine for both online and local file upload using
document_analysis_client = DocumentAnalysisClient(
        endpoint=endpoint, credential=AzureKeyCredential(key))
poller = document_analysis_client.begin_analyze_document_from_url(
                    "prebuilt-layout", document=file)
result = poller.result()

But this one for normal documents mine specific use case is for Invoices and upon using it, I am getting error saying 404 resource not found.  I am using this:
document_intelligence_client  = DocumentIntelligenceClient(
                endpoint=DOCUMENTAI_ENDPOINT, credential=AzureKeyCredential(DOCUMENTAI_KEY))

        poller = document_intelligence_client.begin_analyze_document(
        "prebuilt-invoice", AnalyzeDocumentRequest(url_source=pdf_url)
    )

Error messages given by the failure

azure.core.exceptions.ResourceNotFoundError: (404) Resource not found
Code: 404
Message: Resource not found
Expected/desired behavior

It should work, I should get the normal output

OS and Version?

WSL Linux (Ubunutu 22.04)

I have the same issue with Java SDK.
If I use old library: azure-ai-formrecognizer:4.1.4, there is no problem.
If I use new library: azure-ai-documentintelligence:1.0.0-beta.2, it returns 404 resource not found error.
Here is my code for new library:
AnalyzeDocumentRequest analyzeDocumentRequest = new AnalyzeDocumentRequest().setBase64Source(Files.readAllBytes(file.toPath()));       SyncPoller<AnalyzeResultOperation, AnalyzeResultOperation> poller = documentIntelligenceClient.beginAnalyzeDocument("prebuilt-layout",        		null, null, null, null, null, null, analyzeDocumentRequest);

Can you please check if your resource is in one of the 3 preview regions?

East US
West US2
West Europe

Can you please check if your resource is in one of the 3 preview regions?

East US
West US2
West Europe

Yes, the problem relates to region of document intelligence instance. It is OK now.


Clean up README and add docs for all new features

[x] Debugging
[x] Getting SNAPSHOT and normal builds
[x] Detailed docs for each command
[x] Do awesome-scala PR

Awesome-scala already lists sbt-robovm (link)

@Darkyenus I know the README / docs will not a lot more love, but could you please test master locally for me and proof the README?

I have actually managed to rewrite the README, for now I have pushed it only into mine fork, so you can check it out. It should be much more helpful than what we have now, but it still needs some work.
I don't have time and energy to keep improving it today, but if you feel like it, you can merge it.
Mental TODO: SBT now allows multi-project builds defined in .sbt files, yet the readme still suggests .scala buuild files, test if it works in .sbt and change the readme (and samples) accordingly
(More elaborate issue is narrowly connected with this: It it worthwhile to essentialy fake the project types only to insert a few keys? Woudn't it be better to get rid of the whole project system and just supply relevant key settings in Seq[]'s to be added to the build definition, like other projects do?)

Your README looks great! I will go through it a couple of times and merge it. Thanks for your hard work.

The docs will continue to improve, but closed by 7afd0dd2b6ceacdb691999e4267db021fd26e583


[Tooltip] Add support for CSS variables

[x] I have followed (at least) the PR section of the contributing guide.

One of https://github.com/mui/material-ui/issues/32049
Preview: https://deploy-preview-32594--material-ui.netlify.app/experiments/material-ui/tooltip/
Not sure how to tag all contributors
cc: @mui/contributors @mnajdova @siriwatknp

See the comments. If you have hard time fixing it, I can help later today or tomorrow :)

Hi @mnajdova , unfortunately this fix is quite hard for me, I'm willing help and learn also, thanks! :)

See the comments. If you have hard time fixing it, I can help later today or tomorrow :)

Hi @mnajdova , unfortunately this fix is quite hard for me, I'm willing help and learn also, thanks! :)

The token is added, please check https://github.com/mui/material-ui/pull/32594/files#diff-41f50b7e71d281679755abc4d49b2011dffdb5e08c883077a7d7d09a7e5d3425R60

See the comments. If you have hard time fixing it, I can help later today or tomorrow :)

Hi @mnajdova , unfortunately this fix is quite hard for me, I'm willing help and learn also, thanks! :)

The token is added, please check https://github.com/mui/material-ui/pull/32594/files#diff-41f50b7e71d281679755abc4d49b2011dffdb5e08c883077a7d7d09a7e5d3425R60

Thanks @siriwatknp , will check it later

Yes, It works now, thanks @siriwatknp , added arrow option on docs


TCP Conncetion?

is there any way to make this use tcp instead of udp? thanks just a question i didn't know where to put it.

Already discussed lots of times, see previous issues on this matter.

You can use project openvpn-install-advanced by @zabullet - it can configure tcp and udp server

Reminder that the fork is not up to date and doesn't include some very important security features available in my work.
For example: Blowfish is now considered insecure and should no longer be used. It also has DNS leaks in Windows 10.
Please, avoid those dangerous suggestions in the future.


iPXE install documentation

How to categorize this PR?

/area documentation
What this PR does / why we need it:
Introduce documentation for installing Garden Linux with iPXE

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: ivy:x: clyannivy seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

@clyann Before I can merge the request, we need all contributors of this PR to sign the CLA
If you have an GitHub account:
read and (if you agree) sign: https://cla-assistant.io/gardenlinux/gardenlinux?pullRequest=479
Do you know how I could contact the user ivy from the last commit of this PR? This user does not seem to have an GitHub account linked to that email address (afaik this is required to sign the CLA, unfortunately).
Thanks!

That is a mistake on my side, git was not configured properly on one of my machines. I apologize for the inconvenience. I will close this request and open a new one


Password controls don't do anything (pretty serious issue)

As long as you enter a correct email, you don't have to get your password right on initial sign in. I just entered
user<EMAIL_ADDRESS>pw: "k"
and got in.

OK, that's a really serious issue. Turns out that I commented out the auth code as a means of testing whether the finals schedule rendered properly for middle schoolers, and forgot to uncomment it.
The issue has just been fixed, four days after it was created. I also checked the logs to verify no one else discovered this, and I did not see any suspicious series of logins (i.e. the same computer logging in several times a minute). Still, this is a really bad bug, and thanks for bringing it to my attention.

😬


Allow Addition of New Testimonial (Implement backend for testimonials)

Feature request-
Through this feature a logged in user would be able to add new testimonials to our project.
Is your feature request related to a problem? Please describe. (optional)
A clear and concise description of what the problem is.
Describe the solution you'd like
In-order to implement it, I added post request for the testimonial-wall page which takes the inputted views by the logged in person then store the full name and the inputted views as two objects at the end of the testimonial array which is already there in our project.
This basically adds new objects as new elements into the testimonial array. Then it gets rendered normally as other dummy testimonials are being rendered through the testimonial-wall.ejs file.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Describe how this feature will be useful to our readers.
This will further add credibility and more functionality to our project
Additional context
I m a GSSOC'21 participant and would like to work on it.

Note:

Do mention if your are working under GSSoC'21.
If you want to work on an issue, you should check if it has already been assigned to anyone. If the issue is free you can comment /assign to get the issue assigned to you.
If you are raising a new issue and want to work on it then also you should comment /assign under the issue to get it auto assigned.
Please refrain from adding labels to your issue/pull-request on your own. It is the job of the Project Admin and the Mentors to review your issue/pull-request and add labels accordingly.
According to the level of the issue, respective time will be allotted under which the task needs to be completed/resolved by the assigned person. If the issue is not resolved within the given time, it will be assigned to the next person. In case of any discrepancies, the decision of the maintainer will be final.

level - 0: 2 days
level - 1: 3 days
level - 2: 4 days
level - 3: 5 days

/reviewed

@Pranav016
I was not able to add the video because the size was large. So added it to google drive.
https://drive.google.com/drive/folders/1q1-24nLjMVkc6ubHgTFur6gIzVC6cCZs?usp=sharing
Please check it and tell whether it is fine for our project.
Right now I have made the link to be viewed by everybody. So once you check it then please tell me then I would change it back to restricted.

@theBeginner86 assign yourself this issue by commenting the keyword otherwise someone else will assign it to them and I won't be able to help..

/assign

@Pranav016
I was not able to add the video because the size was large. So added it to google drive.
https://drive.google.com/drive/folders/1q1-24nLjMVkc6ubHgTFur6gIzVC6cCZs?usp=sharing
Please check it and tell whether it is fine for our project.
Right now I have made the link to be viewed by everybody. So once you check it then please tell me then I would change it back to restricted. I m saying this to just be on the safer side for my drive and email related to it.

Great work, maybe have a button at the top of testimonial wall as well that will scroll down to the actual text-box because even from home page when we click on Write testimonial it goes to testimonial wall and you have to scroll which some people might not know that the text-box is at the end of the page.
You can do this by adding a id to the text box and then href to the new button that you will add to the testimonial page

ok I will try to do that.


The coroutine is canceled

Traceback (most recent call last):
File "/external/src/init.py", line 62, in async_wrapper
return await original_attr(*args, **kwargs)
File "/usr/local/lib/python3.10/site-packages/peewee_async.py", line 272, in execute
return (await execute(query))
File "/usr/local/lib/python3.10/site-packages/peewee_async.py", line 432, in execute
return (await coroutine(query))
File "/usr/local/lib/python3.10/site-packages/peewee_async.py", line 580, in select
return await _execute_with_returning(query)
File "/usr/local/lib/python3.10/site-packages/peewee_async.py", line 564, in _execute_with_returning
cursor = await _execute_query_async(query)
File "/usr/local/lib/python3.10/site-packages/peewee_async.py", line 1480, in _execute_query_async
return (await _run_sql(database, *query.sql()))
File "/usr/local/lib/python3.10/site-packages/peewee_async.py", line 1460, in _run_sql
cursor = await database.cursor_async()
File "/usr/local/lib/python3.10/site-packages/peewee_async.py", line 882, in cursor_async
await self.connect_async(loop=self._loop)
File "/usr/local/lib/python3.10/site-packages/peewee_async.py", line 853, in connect_async
await self._async_wait
asyncio.exceptions.CancelledError
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/local/lib/python3.10/site-packages/sanic/http/http1.py", line 127, in http1
await self.protocol.request_handler(self.request)
File "handle_request", line 97, in handle_request
File "/external/src/monitor.py", line 58, in get_login_handler_monitor
result = await request.app.ctx.db_manager.Base_manager.execute(query_ans_sql)
File "/external/src/init.py", line 65, in async_wrapper
await self.close()
File "/usr/local/lib/python3.10/site-packages/peewee_async.py", line 307, in close
await self.database.close_async()
File "/usr/local/lib/python3.10/site-packages/peewee_async.py", line 899, in close_async
await self._async_wait
asyncio.exceptions.CancelledError
The PooledMySQLDatabase link is managed using peewee_async's Manager.
An exception occurred when the coroutine was canceled while executing execute. We can't figure out why

Something(probably sanic) cancel the task which contains your coroutine with query. peewee-async does not use cancel() in sources


Streamlit wants my email address

Issue
I ran aider --browser and got the following:

      👋 Welcome to Streamlit!

      If you’d like to receive helpful onboarding emails, news, offers, promotions,
      and the occasional swag, please enter your email address below. Otherwise,
      leave this field blank.

      Email:

I don't know what Streamlit is and do not want its swag. A party-pooper, I know ...
Version and model info
Aider v0.41.0

Agreed this is annoying. I can't find a straightforward way to suppress it.
Streamlit is the browser GUI package that aider uses for the --browser mode.


fix: add atc button to all product component in home, list, and recomendation

Home

List

List[slug]

added POW for cart page
https://drive.google.com/drive/folders/1uZPYrEbdBtWO0cgydSTIKcFs_lXWegFk?usp=share_link

Added Pop up success when add to cart in cart page


Remove memory leak when receiving messages

Each time a message was received, a new PDU was instantiated and never deleted.
To prevent this, PDU can be just constructed on the stack and passed around as const ref.

Thanks for the pull request! This was a good catch.


pip package installation failed

used pip install audiomate
And yielded this answer
Collecting audiomate
  Using cached audiomate-6.0.0-py3-none-any.whl.metadata (7.7 kB)
Collecting audioread==2.1.8 (from audiomate)
  Using cached audioread-2.1.8.tar.gz (21 kB)
  Preparing metadata (setup.py) ... error
  error: subprocess-exited-with-error

  × python setup.py egg_info did not run successfully.
  │ exit code: 1
  ╰─> [6 lines of output]
      Traceback (most recent call last):
        File "<string>", line 2, in <module>
        File "<pip-setuptools-caller>", line 34, in <module>
        File "/private/var/folders/_6/cj2y6ttx48b7332gvfxjrfjw0000gn/T/pip-install-fn_lys3z/audioread_3996d43fe0b542edb22256cc8daefedf/setup.py", line 17, in <module>
          import imp
      ModuleNotFoundError: No module named 'imp'
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed

× Encountered error while generating package metadata.
╰─> See above for output.

note: This is an issue with the package mentioned above, not pip.
hint: See above for details.

imp was removed from the default libraries in Python 3.12 in favor of importlib. Since audiomate hasn't been updated in almost four years, I'm not sure what the odds of this being corrected are, but in the meantime a potential workaround is to use pyenv to install Python 3.11, which still supports imp.


Optimize addReadmeButtons

shorten selectors
avoid unnecessary operations
skip jQuery
get user content out of innerHTML (XSS)
various code shortcuts

Tested on the pages:

base branch: https://github.com/sindresorhus/refined-github
custom branch: https://github.com/sindresorhus/refined-github/tree/badges
long branch name: https://github.com/bfred-it/sandbox/tree/what-do-you-think-the-maximum-branch-name-length-is-question-mark-i-think-there's-no-limit-so-i'm-just-gonna-keep-going-until-github-throws-up-or-i-get-tired-of-typing-nonsense-omg-why-are-you-still-reading
1-level deep readme: https://github.com/GhostText/GhostText/tree/master/browser
multi-level deep readme: https://github.com/babel/babel/tree/7.0/packages/babel-cli
commit view (no edits allowed): https://github.com/sindresorhus/refined-github/tree/83e6bd82746327654fe7a8dd089ba3a132a84c4e
tag view (no edits allowed): https://github.com/sindresorhus/refined-github/tree/1.17.0

🙌


prop to allow truncation of email

I would love to see a prop to stop truncation of email addresses.
Suggestion:
emailTruncate?: false

Using truncate={0} will disable link truncation. Or do you want to configure truncation depending on the type of link? If that's the case, my initial inclination is to suggest using renderLink as adding truncateEmail, truncateHashtag, etc. seems a bit much.
Let me know what you think.

I was more looking in to type of link adding the ability to disable it for mailto primarily.
Problem i have is that the strings are not static and tend to be different everytime and i found it little bit ugly and confusive with the truncation of email addresses. This idea of turning it off stem primarily from users started to ask why the emails were cut off in the middle reporting it as a bug.
As a side note i can also see a usage of turning it off for hashtags as well.

I'm curious why you don't just want to disable truncation entirely. Is there a type of link that you do want to truncate?
I'm actually considering disabling truncation by default in v5.

v4.0.0 (currently release candidate) disabled truncation by default (use truncate={32} for previous behavior). The truncate function can be imported and combined with renderLink for complete flexibility.