[Forms] Extracting Defaults form a schema

Hello, I'm the creator and maintainer of https://github.com/logaretm/vee-validate and I'm very much interested in this project.
I have a few ideas from the perspective of a long-time maintainer of a popular form library that makes use of schema validation for Zod, Valibot, Yup and others.
I will keep each idea to one issue at a time to not muddy the discussion.
I have come up a Typed Schema concept with similar aims to what you have here to address some of the issues you mentioned in the README.
What
One thing comes to mind is defaults and how can form libraries take advantage of that information to initialize fields with default/initial values.
Take the following snippets as an example, we can define a few fields with Zod and Valibot with default values like this:
Zod
z.object({
  name: z.string().default('test'),
  age: z.number().default(11),
});

Valibot
v.object({
  age: v.optional(v.number(), 11),
});

AFAIK Valibot offers utilities to extract those values like we do here, we also do that with Zod with a hacky approach.
In vee-validate at least there are a few hard requirements for being able to resolve initial values, but the most important one is schemas must be sync. Since forms need to have their values initialized on mount rather than at unknown point in the future.
How (Suggestion)
I believe extending the interface with an optional ~defaults might prove very useful for form libraries.
export interface StandardSchema<Input = unknown, Output = Input> {
  // ...

  /**
   * Resolves the defined initial values for the schema.
   */
  readonly "~defaults": StandardDefaults<TInput>;
}

What is hard to answer here is:

what is the type of ~defaults return type? is it TInput or TOutput?

I honestly don't know, in reality it is a mix of both in real-world forms. But usually I would say it fits TInput more as it is usually a partial of it and not necessarily a full representation of a valid input, in many cases they overlap with TOuput because a number is still assignable to a nullable number. It breaks when there is a transformation taking place then it becomes confusing, but i would say this is rare with forms.
What do you guys think? Should Standard schema accommodate such cases or should there be an extended version of the standard that caters more to form libraries?
Some FAQs

Why not use ~validate as a parser?

Empty forms rarely pass the schema for an output to be produced. Schema providers usually have the parse output and errors as mutually exclusive values. But here with a ~defaults extractor we are more interested in the defined values rather than how they can be produced via parse/validate calls.

In ArkType, defaults must conform to the input type so that any transformations can be applied to them as if the value were provided directly, so I'd be in favor of that.
What I think is the tougher question here is how ~defaults would be organized. One option would be to just mirror the shape of the props on the schema, but it duplicates those names and critically, is incompatible with unions.
Realistically, we'd need to address https://github.com/standard-schema/standard-schema/issues/7 and have an established representation of a schema's AST before we can make good decisions about how to associate default values with it.

In ArkType, defaults must conform to the input type so that any transformations can be applied to them as if the value were provided directly, so I'd be in favor of that.

I agree. In Valibot we have the concept of defaults and fallbacks. Defaults are used when no input is given and they always have to match the input type for the same reason as in ArkType. Fallbacks always match the output as they are returned directly in case of validation errors.

Realistically, we'd need to address #7 and have an established representation of a schema's AST before we can make good decisions about how to associate default values with it.

I think I agree. We would include defaults elsewhere in this AST similar to how it works in JSON Schema. Valibot already includes a custom AST, probably similar to how many other schema libraries do it. However, it will be much harder to agree on a common structure specifically for Valibot. If we can't natively integrate the AST into our library code, it's probably not an option for us, as it would increase the bundle size a lot if we were forced to add it on top.
Going back to your initial suggestions with ~defaults, I think the idea is great, but probably very form library specific. Personally, I would prefer that the user provide initial form values separately and not as part of the schema. I don't have a good example right now, but I remember that there are some edge cases where it's not a good idea to mix the two. Also, there are probably many questions about how to implement the ~defaults property. How to handle dynamic defaults that use a generator function? How to handle unions? Should ~defaults contain the recursively summed defaults of the whole schema (this would again be a problem in terms of bundle size for valibot) or just the default of a single schema as explicitly passed to .default in Zod and optional in Valibot, if defined.
Standard Schema is intentionally not able to cover every use case, so for more complicated and individual cases it will still be necessary to use adapters. If Standard Schema becomes too specific, we would block innovation in the long run, as new schema libraries that can't apply to this specification have no chance of being widely adopted if one day every third-party library only accepts Standard Schema.

and critically, is incompatible with unions.

Yep, I have no idea how would that work with a ~defaults kind of behavior. While libraries like mine can afford to do best effort here, I think a standard spec will need to do better.
I didn't see a lot of unions/intersections with forms in the wild, so I'm unable to suggest a path forward with this point in particular.

Realistically, we'd need to address https://github.com/standard-schema/standard-schema/issues/7

I just took a look at this and I agree. Having a standard AST will go along way in making things like this user-land or library-land in my case, as i can go through the AST and decide what I want to do for my users and can make certain compromises that a standard spec cannot.

there are probably many questions about how to implement the ~defaults property...

Yep, the way we do them at the moment is different for each schema provider given the different tools available for each. For Zod we walk the AST and collect what we can. I totally agree from a standard standpoint this would be hard to implement. But given an AST, then you can leave those questions to the implementer.
ASTs also will solve an issue I have in mind with what I can path descriptions, which are simple flags/info about a given field. I will leave more details there on that issue.
Thanks for the feedback everyone, I will be annoying you with more Form specific stuff, this project is something I have been dreaming about for a while so I appreciate the time and work you are putting in here.


Closes #2, Allows paths to be excluded and included.

@larzconwell Very sorry I just found this PR, can you rebase it on master branch?

This uses glob.sync. I would opt for minimatch.
Just my 2 cents...

I agree @bwin. I don't really write Node anymore so I'm just gonna close this PR.
IIRC I opted for glob.sync at the because createPackage didn't have a callback at the time and I didn't want to break api.

Thanks for contributing anyway.


VC-JWT and broader JWT support

Currently the lib is going towards JsonWebSignature2020 as the first cryptosuite.
For wider interop we should support JWTs and their spec-specific flavors, such as VC-JWT.

Completed in #47


ietf-comments fails to generate issues on github

dooku-[lwork/rats/architecture](2.6.6) mcr 5869 %ietf-comments -g ietf-rats-wg/architecture 197.txt
Warning: Unrecognised h2 section comment:.
Warning: Unrecognised h2 section notes.
Document: draft-ietf-rats-architecture
Revision: 21
CC: @larseggert
Github Issue Creation Error: Not Found - restart with --start=1
[1]    246718 exit 1     ietf-comments -g ietf-rats-wg/architecture 197.txt

where 197.txt comes from
https://mailarchive.ietf.org/arch/msg/rats/pLZ4oeYgStKiygg_KA5NMJN6HQg/

Could you:

Run pip show ietf-comments
cd to the path indicated by Location shown in the results from (1)
cd ietf-comments
edit github.py to append github.enable_console_debug_logging() on a line before class GithubRepo:
run the command as you do above
paste the results here


Stub clog causes unused variable errors if the variables are only used in logs

inline
void
QuicTraceEventStubVarArgs(
    _In_ const void* Fmt,
    ...
    )
{
    UNREFERENCED_PARAMETER(Fmt);
}
#define QuicTraceEvent(Name, ...) QuicTraceEventStubVarArgs("", __VA_ARGS__)

In msquic, we use the following trick to make sure the compiler sees all logs as referenced, rather then just defining away the macro.

CLOG could/should be smarter than the above by individually UNREFERENCED_PARAMETER'ing each arg. One problem with the inline function approach is that on Linux that requires the function then to be declared in a different C file.

We have a c file in clog, so theoretically thats fine. I don't know if its actually possible to just reference any variable number of parameters in strictly a macro context, especially since we can't generate different macros for different number of args but with the same name.


'fstp tbyte ptr' -> should be 'fstp tword ptr' ?!

0x0069A807 - DB BD C0 FD FF FF
x32dbg:

Capstone:

iced:

Which one is correct?

tbyte ptr is what masm uses and that's why you see iced's masm formatter use it. The t in tbyte means ten and that's the size of the memory location.

so tword and tbyte are identical? Or does x32dbg uses something different than masm and that's what it makes it a tword instead of a tbyte ?

tword is what nasm uses but it doesn't use ptr so it seems to be a mix of masm and nasm.


Bump dependency compatibility with laravel 10

Upgrading dependenies:

"inertiajs/inertia-laravel": "^0.6.9"
"mockery/mockery": "^1.5.1"
"orchestra/testbench": "^6.24.1 || ^7.22.1"
"phpunit/phpunit": "^9.6.4",

to be compatible with laravel 10

@ksassnowski hi...thank you for this amazing package. I was wondering, will you be merging this soon or are we better off forking it and making the necessary changes ourselves. I am having difficulties upgrading my app to use inertial laravel. Thank you.

Can someone verify for me if the changes in this PR allow you to upgrade to Laravel 10?

@alslimany Can you please also update the GitHub action so it also runs on Laravel 10? https://github.com/laravel-arcanist/inertia-response-renderer/blob/main/.github/workflows/run-tests.yml

@ksassnowski looks like that was the last time @alslimany was active on github 😆

Hi all, i'm here.
Just waiting to get back home.

Quick heads up that I’m going to be on vacation for a week. I’ll get this merged once in back.
Sorry for the delay.


chore(deps): update devdependency vite to ^5.4.5

This PR contains the following updates:

Package
Type
Update
Change

vite (source)
devDependencies
patch
^5.4.4 -> ^5.4.5

Release Notes

vitejs/vite (vite)
v5.4.5
Compare Source
Please refer to CHANGELOG.md for details.

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Hywax Assistant.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (70b139e) to head (1d6d24c).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##              main       #68   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            3         3
  Lines          146       146
  Branches        26        26
=========================================
  Hits           146       146

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


/apps/blockchain-cpp - CLI Runner

What would you like to be added:
/apps/blockchain-cpp that export a CLI to start a blockchain. With additional utility to take snapshot and load snapshot. Easy interoperability with packages/jellyfish-api-core.

[x] Take Snapshot
[x] Load Snapshot from ...
[ ] Interoperability with packages/jellyfish-api-core.
[ ] Export an additional Docker Image as an Artifact.
[ ] Automatically get binary when library is installed via postinstall script with https://docs.npmjs.com/cli/v8/using-npm/scripts

/triage accepted
/area apps
/sig ecosystem

@fuxingloh
Just for clarification, does the binary in "Automatically get binary" task refer to DeFiD? Or is it something else?


Add a built version as a github release

To simplify usage, some users would like to download a plain file. We could achieve this w/ a github release using their API.

Once #167 is fixed, this won't be a problem, will it?

Agreed, can just download from cdn


Use XDG Base Directory standards

For *nix systems, it may be better to default to checking in ${XDG_CONFIG_HOME} for config files, but this adds some complexity and doesn't apply to Windows-based environments.
https://standards.freedesktop.org/basedir-spec/basedir-spec-latest.html

Yep, I was thinking the same, I'll do after #7 :)

@tonyskapunk thanks! I think os.UserConfigDir() might be good for this.


Update Ldap.php to allow LDAP over SSL/TLS connections

This is a very crude workaround to allow LDAPS connections. A better solution would be the one explained in the comments.

@fredericmohr Thanks for this and sorry it's taken me a while to getting round to merging your pull request.
I have followed your comments and updated the ldap in 598b07b53d5e3aa5ed43dad419ce2499c5493943 so that a protocol can be optionally specified without breaking. Port 636 or 389 will then be used, depending on protocol type, unless a port is specified by the user.


Handle errors

This version will handle errors when loading file a bit better.
This commit also will add some data validation checks to the
test file for the Checker module.

Possibly fixes issue #5 . Needs more time to check.

I think it helps as much as possible. Closing now as think kind of thing would be best in a beta branch.


Moved JAVA_OPTS_APPEND out of Dockerfile

What this PR does:
Moved JAVA_OPTS_APPEND out of Dockerfile so we can pass jvm args as required from docker compose file, k8s yaml, etc.
Which issue(s) this PR fixes:
Related to #662
Checklist

[X] Changes manually tested
[X] Automated Tests added/updated
[X] Documentation added/updated
[X] CLA Signed: DataStax CLA

Docker compose files are now broken. Log from JSON API container:
2023-11-22 13:03:07 INFO exec -a "java" java -XX:InitialRAMPercentage=50.0 -XX:MaxRAMPercentage=75.0 -XX:+UseG1GC -XX:MinHeapFreeRatio=10 -XX:MaxHeapFreeRatio=20 -XX:GCTimeRatio=4 -XX:AdaptiveSizePolicyWeight=90 -XX:+ExitOnOutOfMemoryError "-Dquarkus.http.host=<IP_ADDRESS> -Djava.util.logging.manager=org.jboss.logmanager.LogManager" -cp "." -jar /deployments/quarkus-run.jar
2023-11-22 13:03:07 INFO running in /deployments
2023-11-22 13:03:07 Error: Could not find or load main class "-Dquarkus.http.host=<IP_ADDRESS>
2023-11-22 13:03:07 Caused by: java.lang.ClassNotFoundException: "-Dquarkus.http.host=<IP_ADDRESS>

My bad, fixed it in this PR https://github.com/stargate/jsonapi/pull/678 Can I get a review/approval for that please?


How to load a pre trained classifier params file in auto gluon for inferencing ?

Hi,
Once I've trained a classifier in Autogluon and I have the params file how can I load this params file in another script for inferencing (with my code below for training) ?
dataset = task.Dataset('DataSet')
classifier = task.fit(dataset)
network = classifier.model
network.save_parameters('network.params')
test_dataset = task.Dataset('DataSet/test', train=False)
indice, resultat, probs_all = classifier.predict(test_dataset)
In the classification examples you do both training and inferencing in the same script and I've found nothing in the documentation to load params in a autogluon classifier.
Is there like a hidden task.load function ?
Thx for your help. (hope this is understandable I'm new here)

@zhreshold Could you look into this?

@zhreshold @Jerryzcn  Any idea what happened to the old example scripts that used to live in examples/image_classification/?  I vote we add those back into this directory (and move the blog-stuff to examples/image_classification/kaggle/ subdirectory).  I think example scripts are mainly supposed to help newcomers see what functionality is possible & provide an easy test-script for contributors to trial their new code, but the Kaggle blog example-scripts seem overly complex and do not demonstrate basic functionality like save/load.

Hi, so with the code :
import autogluon
from autogluon import ImageClassification as task
classifier = autogluon.task.image_classification.Classifier.load('network_98_4.params')
indice, resultat, probs_all = classifier.predict('Test')
Test is my folder of test pictures
I got the following error :
/usr/lib/python3/dist-packages/apport/report.py:13: DeprecationWarning: the imp module is deprecated in favour of importlib; see the module's documentation for alternative uses
import fnmatch, glob, traceback, errno, sys, atexit, locale, imp
Traceback (most recent call last):
File "ClassifierLoad.py", line 6, in
classifier = autogluon.task.image_classification.Classifier.load('network_98_4.params')
File "/home/penta/.local/lib/python3.8/site-packages/autogluon/task/image_classification/classifier.py", line 56, in load
state_dict = load(checkpoint)
File "/home/penta/.local/lib/python3.8/site-packages/autogluon/utils/serialization.py", line 79, in load
return _load(f, map_location, pickle_module, **pickle_load_args)
File "/home/penta/.local/lib/python3.8/site-packages/autogluon/utils/serialization.py", line 361, in _load
magic_number = pickle_module.load(f, **pickle_load_args)
_pickle.UnpicklingError: invalid load key, '\x12'.

@Pentar0o You are mixing save/load that's designed for different purposes.
Refering your original code snipet
dataset = task.Dataset('DataSet')
classifier = task.fit(dataset)

You can save/load the complete state of the estimator by
classifier.save('cls.states')
new_classifier = autogluon.task.image_classification.Classifier.load('cls.states')

or simply save the network parameters if you don't necessarily need to resume the full state of fitting but want to revert or copy the weights from certain checkpoint:
classifier.model.save_parameters('network.params')
classifier.model.load_parameters('network.params')

For you usecase I'd suggest you stick with the first example so you don't have to instantiate a new classifier.

Question : how to load the params without having to start the learning process first ? because the task.fit(dataset) will launch the training.
Is there something like :
new_classifier = autogluon.task.image_classification.Classifier.load_parameters('network.params')

@zhreshold Could we add an example of saving, loading, and predicting on the loaded model to the image classification tutorial for quick reference?

The current api for serializing the estimator is incomplete so I think it only make sense when the save/load functionalities are complete, I will update the corresponding tutorial after merging the new estimator refactorization.


allow npm imports

Added index.js to allow one-line imports in npm based builds

This change is

Thank you @anlGT , did you check if it works?

Yes, just have to load jQuery first, otherwise it throws Error: [$compile:ctreq] Controller 'mdtTable', required by directive 'mdtHeaderRow', can't be found!


Expected entry point (lib/index.js) missing in v1.0.2 bundle

It appears this release wasn't built before npm publication.
Thanks for your time!

Oops. Looks like the file got gitignored. Let me know if v1.0.4 works for you.

Yep, thanks!


fix(types): restrict regular styles function call to one argument

What's included
fix: #37
fix: #15
Tag functions are properly typed and regular function calls are restrict to one argument (see: https://github.com/solidjs/solid-styled-components/pull/38#discussion_r988537281)

keyframes
glob
css
createGlobalStyles
The return of the styled(...)

We can now override the types to mark the theme as always available on the props. (see: https://github.com/solidjs/solid-styled-components/pull/38#discussion_r988531112)

Thank you


HDH 212 fixing feedback plugin

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the CODEOWNERS.
What Type of PR is This?

[ ] Issue
[ ] Feature
[x] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[x] Tested Locally
[ ] Optional Screen Shoot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://658d37532575f28e0e6bca41--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://658d377ee363928b014bce84--harness-developer.netlify.app


AKS Learn feedback: Incorrect or Missing Information in AKS Quickstart Documentation

Type of issue
Missing information
Feedback
The documentation does not fully explain the additional resource groups created when deploying an AKS cluster using the New-AzAksCluster command. This omission can lead to confusion when users try to clean up resources after completing the quickstart tutorial.
Detailed Description:
When deploying an AKS cluster with the New-AzAksCluster command, two additional resource groups are automatically created:

Network Watcher Resource Group: This is created for monitoring purposes in the region where the virtual network resides.
Managed Cluster Resource Group: This resource group contains all the infrastructure components required for the AKS cluster, such as Virtual Machine Scale Sets, Load Balancers, and Public IP addresses. The naming convention for this resource group follows the pattern: MC_<ResourceGroupName>_<AKSClusterName>_<Region>

The documentation should:

Include details about these additional resource groups and their purpose.
Clarify that users must delete these resource groups to fully clean up after testing AKS.

Suggested Update to Documentation:
Update the "Delete the cluster" section to include a note about these additional resource groups and their cleanup process. For example:
In addition to the main resource group, the deployment creates two additional resource groups:

- A Network Watcher resource group (used for monitoring).
- A Managed Cluster resource group containing AKS infrastructure resources. Ensure you delete these additional resource groups to avoid ongoing Azure charges.

Provide an example command to delete the managed cluster resource group:
Remove-AzResourceGroup -Name MC_myResourceGroup_myAKSCluster_westus

Why This Matters:
Leaving these additional resource groups undeleted can result in unexpected Azure charges. Including this information will improve the overall accuracy of the documentation and help users better manage their Azure resources.
Thank you for considering this feedback. Let me know if further clarification is needed!
Page URL
https://learn.microsoft.com/en-us/azure/aks/learn/quick-kubernetes-deploy-powershell
Content source URL
https://github.com/MicrosoftDocs/azure-aks-docs/blob/main/articles/aks/learn/quick-kubernetes-deploy-powershell.md
Author
@tamram
Document Id
c71f4c52-7711-69d4-36a1-c43bd140b15d

#reassign:schaffererin

hi, @Latzox. thank you for your feedback. i updated the callout about the second resource group. the pr was merged, and the updated article will be live shortly. i will now close out this github issue. thanks again!
#please-close


Day

Codecov Report
Base: 53.46% // Head: 52.39% // Decreases project coverage by -1.07% :warning:

Coverage data is based on head (d2f9a7e) compared to base (f63c5b1).
Patch coverage: 10.00% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master      #78      +/-   ##
============================================
- Coverage     53.46%   52.39%   -1.08%
  Complexity      415      415
============================================
  Files            96       96
  Lines          1762     1798      +36
  Branches        184      188       +4
============================================
  Hits            942      942
- Misses          779      815      +36
  Partials         41       41

Impacted Files
Coverage Δ

.../main/java/seedu/waddle/commons/core/Messages.java
0.00% <ø> (ø)

...rc/main/java/seedu/waddle/model/item/Duration.java
0.00% <0.00%> (ø)

src/main/java/seedu/waddle/model/item/Item.java
0.00% <0.00%> (ø)

src/main/java/seedu/waddle/model/item/Day.java
12.50% <9.67%> (-87.50%)
:arrow_down:

...n/java/seedu/waddle/model/itinerary/Itinerary.java
78.46% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


feat: add a plugin system

Current dependencies on/for this PR:

main

PR #386

PR #402   👈

PR #403

This comment was auto-generated by Graphite.


Right Angle Triangle Mode

It would be great to have a third mode, called RAT Mode  or Right Angle Triangle Mode.
Horizontal mode will be renamed to Isomatric Mode and the Vertical one will be renamed to Classic Mode.

Horizontal and Vertical mode have now be renamed.


[multitenant mvp] transactionalId isolation

Is your feature request related to a problem? Please describe.
Following on from #190 this feature is to further extend the multi-tenant filter to support the mapping of transactionalIds.
Two tenants must be able to use the same transactionalId, without collision.  RPCs that handle the transactionalids must be intercepted: requests must add the tenant prefix, responses must strip the tenant prefix.
Describe the solution you'd like
As tenant A, my application must be able to create transaction foo using transactionId bar, independently of any other tenant, who happens to chose the same transaction id.
Describe alternatives you've considered
N/A
Additional context

Those that require request mapping (AddOffsetsToTxn, AddPartitionsToTxn, EndTxn, TxnOffsetCommit, DescribeTransactions, InitProducerId, Produce)
Those that require response mapping: (ListTransactions)

Hey @racheljpg
For 1. break it into the smallest steps possible. I'd be perfectly happy to see 1 PR per request or response mapping and a further PR to add integrations tests.
Having said that it might be easier to start writing a the failing integration/acceptance test and then add the simplest thing you can to make it pass. Then add the next failing case, and then what ever is needed to make it pass.
for example.
First test

beginTransaction
produceMessage
commitTransaction

Second test:

beginTransaction
produceMessage
abortTransaction

Third test:

beginTransaction
describeTransaction

There also needs to be some coverage of the consumer side, thought I'm not quite sure if what that would look like.
For 2. That's a good question see KIP-98 for all the details.

The TransactionalId to use for transactional delivery. This enables reliability semantics which span multiple producer sessions since it allows the client to guarantee that transactions using the same TransactionalId have been completed prior to starting any new transactions. If no TransactionalId is provided, then the producer is limited to idempotent delivery.

n.b. I had miss read 2. initially and thought you were referencing Transactional.id not Trasaction.id. So the same transactional.id will be used across (also note that is the ID this issue is referencing as needing re-written). So we would need to ensure the transactional.id header/field is always re-written. I'm somewhat more on the fence when it comes to error message bodies and the like as the costs of determining where the ID occurs and replacing them seem quite high. I need to do some more research on that point.
Digging into the RPC definitions I'm not seeing a transaction.id so I maybe my first answer was right, transactional.id is reused and thus needs re-wrting. If I've missed what you were getting at then let me know and I'll look further.


[authoring] scroll position improperly maintained across documents

If you're in the middle of a long document and switch to another document, it attempts to preserve your scroll position inside that new document, opening that document scrolled to approximately you where in the other document.
You need a document taller than your window size in order to see this.
Expected Behavior
Either each document should have a separate saved scroll position, or switching between documents should reset the scroll position to the top.

@jessmartin looks like scroll position resets between docs now!
I'm going to leave this issue around with lower priority for preserving scroll position of a document during a session though.


CLI: README improvements

After we set an initial version of the README for CLI and Foundry (cli branch) with #442, would be nice to add a few more sections to it:

[ ] Add arguments explanations (projectName, --install .. )
[x] Adapt guide of how to contribute, explaining how to set the environment in local to test changes on the CLI

Completed in #422, We can create a new issue soon on point 1. once we maybe get CLI branch in new repo 🙌


[GURPS] Macro Template Reference

With the default template I can simply add {{Field=Value}} and it will appear in the template in chat. I'm trying to add additional notes to a macro for rolling damage, and I can't seem to get another inline field to appear.
Example:
@{selected|roll}&{template:skillRoll} {{type=Damage Roll}} {{characterName=@{selected|character_name}}} {{skillName=Neck Snap}} {{damageRoll=[[(@{selected|swing})*1.5]]}} {{damageType=Crushing}} {{someNote=This is a test where I may expand upon the nuances of neck snapping}}

someNote Does not appear. What I see is this:

Is it possible to explain/expand on the template usage?

Hi @trev-dev
Here's a link to a document that explains the available fields for the skillRoll template. There are some new fields I need to add but it should be helpful.
https://1drv.ms/b/s!AnMvSmU-IUW2jJsp-Eip6ahs8bFMhQ?e=84RXmx
Here are a set of new fields for custom results.
Text Box One
noteBoxOneShow = 1 or empty. If 1, the note box will display in the chat area
noteBoxOneLabel = The label of the row. This should contain a value
noteBoxOneContent = The content you want to display
Text Box Two
noteBoxTwoShow = 1 or empty. If 1, the note box will display in the chat area
noteBoxTwoLabel = The label of the row. This should contain a value
noteBoxTwoContent = The content you want to display
Text Box Three
noteBoxThreeShow = 1 or empty. If 1, the note box will display in the chat area
noteBoxThreeLabel = The label of the row. This should contain a value
noteBoxThreeContent = The content you want to display
Text Box Four
noteBoxFourShow = 1 or empty. If 1, the note box will display in the chat area
noteBoxFourLabel = The label of the row. This should contain a value
noteBoxFourContent = The content you want to display

@MadCoder253 For whatever reason I didn't get notified of your reply, and saw it as my issue was closed. Thank you so much for the doc!

You're welcome @trev-dev !


Remove special chars in CodeInput

Fixes #839
Replaces any special characters as user types/copy and pastes text into the CodeInput
In TalkBack on Android, it will announce the deletion as the user types a special char (e.g. if they type a dollar sign it will say 'dollar sign deleted')

This is auto removing spaces
This is auto removing -

My concern here is someone needs to "sign-off" on what pattern we're looking for here.
https://github.com/cds-snc/covid-shield-mobile/issues/837#issuecomment-662522258

Format of the codes will be up to 12 characters including spaces.

@JuliannaR @smcmurtry
If you folks can test this one as well - I want to ensure we're not causing issues for anyone inputting codes.

On hold

I would also recommend that the decided upon logic/ReGex pattern for sanitizing these key strings should be placed inside of their own utility function, and perhaps placed in a place like src/shared.
The function name should ideally serve to tell us what the code is doing.

@nickmly can you make the suggested change https://github.com/cds-snc/covid-alert-app/pull/844#issuecomment-667491424 today?

Closes this pending further discussion we'll re-open as needed.


Setting sensible default for optimizer

It is better to make a sensible default for optimizer and use this rather than declare in manually when it is needed.
https://github.com/nnstreamer/nntrainer/pull/308#discussion_r454772686

:octocat: cibot: Thank you for posting issue #311. The person in charge will reply soon.

This is handled with props update


Flaggie doesn't understand make.conf as used by gentoo-lto

Failure stack trace:
❯ sudo flaggie --sort-flags --drop-ineffective --drop-unmatched-pkgs --drop-unmatched-flags
Password:
Traceback (most recent call last):
  File "/usr/lib/python-exec/python3.6/flaggie", line 25, in <module>
    sys.exit(main(sys.argv))
  File "/usr/lib64/python3.6/site-packages/flaggie/cli.py", line 195, in main
    porttree, cache)
  File "/usr/lib64/python3.6/site-packages/flaggie/makeconf.py", line 537, in __init__
    self.parse(mf, path)
  File "/usr/lib64/python3.6/site-packages/flaggie/makeconf.py", line 582, in parse
    self.files[fn] = self.MakeConfFile(fn, path)
  File "/usr/lib64/python3.6/site-packages/flaggie/makeconf.py", line 455, in __init__
    f = codecs.open(path, 'r', 'utf8')
  File "/usr/lib64/python3.6/codecs.py", line 897, in open
    file = builtins.open(filename, mode, buffering)
NotADirectoryError: [Errno 20] Not a directory: '/etc/portage/make.conf/make.conf.lto'

Version:
❯ flaggie --version
flaggie 0.2.1

make.conf support was removed from master branch and won't return.

I take it that you are referring to this commit.
If support for make.conf is removed, why is the file being parsed?

You're using an old version.


feature request: add a compute argument to to_hdf5

All is in the title, I would like to use to_hdf5 as I use store: passing it a compute argument to false if I don't want to compute it directly. It is just a matter of two lines of code to implement I think. If you want I can do a pull request for that.
Regards,

to_hdf already has a compute keyword:
https://github.com/dask/dask/blob/7b4f90f0a708cbb97ebb498c78f7931d82ec99db/dask/dataframe/io/hdf.py#L45

Ah you probably mean the dask array to_hdf5 method, not the dask dataframe one. Adding a compute= keyword seems like a nice addition (e.g. to_zarr has a compute= keyword)

I can do a pull request for that

That would be welcome, thanks!

Oh cool thank you, yes I meant the one in dask array I should have precised that.
Thank you for your answer, I will do it.

Hi again,
So I am reopening this issue just to get some help on how I should implement the modification.
I understand why it has not been done before: inside the to_hdf5 function, there is a context manager for the hdf5 file. The process is as follows:
with h5py.File(filepath, mode) as f:
    store(**kwargs)

The problem is, I implemented the "compute=False" possibility, but now if I do ".compute" on the object returned by my custom function dask scheduler fails because the file I am trying to write into has been closed.
Therefore, I see two solutions:

I could store the opened file inside some config object or something
I could create a second function to_hdf5_delayed or something like that that would take a hdf5 file object as input such that the user opens the context manager outside dask code and just passes the file to the to_hdf5 method in order to create the task graph.

Do you have any guideline/advice on how I should do ?

I would either ...

include the h5py file/dataset object in the graph and assume that the user won't close that object
take a filename/dataset name and include those in all of the tasks and open-write-close the file in every task

But there are probably other ways here and it could be that neither of those ways are optimal.  Probably the hard part of resolving this issue is doing some work to figure out the best approach.  The Xarray folks have a lot of experience doing this.  You might consider looking at their solution.

Ok I will do it thank you very much for your answer !


Populate full TaxonInfo

with all vernacular names, distributions & references for 1 taxon
https://sp2000.github.io/colplus/api/staging.html#model-TaxonInfo

https://github.com/Sp2000/colplus-backend/commit/7f928957934600b79b71e97d88408ef83f5521ae

33df835b505f608356c65afde86020e6cc52289c


Changes to rgb stream alignment and addition of rgb2* functions.

Taking a break from the mechanical design of my camera, so I should have more time to contribute in the future.
I was getting reduced size images when creating full size images on the IMX477. I changed align_stream to estimate the stream size to be high so that no pixels are lost.
The .npy is one of many ways to capture RGB values in a non-visual format. Will probably not be used by the majority of users, but is nice to have.
It might be worth it to move the converters back into picamera2.py and within the main class. Additions of metadata as exif and tiff tags could utilize some of the private information in the main class. This may require some reworking of some of the capture functions. The alternative is to require the user to explicitly pass metadata.
Please ignore Picamera2 to PiCamera2 commit. It was untested and reverted back for the sake of this PR.
Signed-off-by: Ian Black<EMAIL_ADDRESS>

Hi, thanks for these contributions. I'm trying to be more careful now about testing and reviewing changes now, I think things were a little bit haphazard at the very start. Anyway, I put some guidance on the README page (here). With that in mind, might you be able to:

Fix up the style checker complaints. I'm trying stick with PEP-8 wherever possible, though there will be occasions when different formatting is tidier in which case I'm happy to ignore it.
Look at any failing tests.
I'm trying to stick with "one change per pull request", so I'm wondering if changes to image sizes and new conversions should be separate PRs.
Also could you remove any unwanted commits, just so that what is left is as clear as possible?

Thanks!


Import Syntax Issues With Streamlit 1.12.2

Trying to build a website using Hydralit, and encountered ModuleNotFound errors immediately upon importing the library.
It seems like at some point Streamlit changed around their code schema; changing two lines in sessionstate.py seemed to do the trick:
# from
from streamlit.scriptrunner.script_run_context import get_script_run_ctx

# to
from streamlit.runtime.scriptrunner.script_run_context import get_script_run_ctx

and
# from
from streamlit.server.server import Server

# to
from streamlit.web.server import Server

I'm pretty new to Streamlit/Hydralit, but hopefully these changes won't break anything else? Seems to be working just fine from the testing I've been doing.

After some more digging, it seems like these changes would also introduce issues around getting the sessionstate for a specific session_id (perhaps the functionality of Server has changed?); working on potential solutions now.

Yes, the way the streamlit server imports worked before was that it used a hack to obtain a state, which is now deprecated. The Server class no longer has a method called  get_current, which breaks Hydra anyway. See this forum post on Streamlit 1.12 breaking functionality: https://discuss.streamlit.io/t/streamlit-script-run-context-gone-with-1-8/23526/5. Fishing the Server instance from the GC is hacky.
I've been working on refactoring Hydralit to use the built-in session_state, which removes the need for the sessionstate.py file entirely.

Update:
I've put in a PR for a rewrite that works with Streamlit >=1.12. You can check it out at #45. I've only tested it with hydralit_example though.

please update to the latest package (1.0.14), as a fix has been created and deployed.
pip install -U hydralit
Thanks very much for your hard work saikumarmk!


If you do not receive mail, how can we send mail from the new

If you have not received an activation message, how to send mail from a new message
How to create a  tag click to resend

so you'll need to get the id of the user then fire off the send code:
$id = 2;//enter the id of the user

//send email
$to = $_POST['email'];
$subject = "Registration Confirmation";
$body = "
Thank you for registering at demo site.

To activate your account, please click on this link: ".DIR."activate.php?x=$id&y=$activasion

Regards Site Admin
";

$mail = new Mail();
$mail->setFrom(SITEEMAIL);
$mail->addAddress($to);
$mail->subject($subject);
$mail->body($body);
$mail->send();


No apply on homescreen button if the app is installed on SD Card

It didnt show the apply on screen option for me, i moved the app to internal storage and it got fixed.

It's working fine, just tried it

i think disabling the downloads causes this to happen sometimes. I now tried and on my new app it has the same problem. it's sometimes appearing in some places but sometimes it just doesn't. I had the same problem on my other app which had downloads disabled.

I just sent you a few screenshots of the bug to you on hangouts


Failure To Load Capture For Simple Vulkan App

Description
I'm writing a custom 2D Vulkan renderer and at the moment it's pretty simple, with only seven different shaders and everything on screen is either a textured quad or a solid colored quad.  I've waited for about 15 minutes before I gave up. I can't even close the app normally; I have to end the task in task manager.

Steps to reproduce
Here is the failing capture file.
failing_capture.zip

Open RenderDoc
Hit "Open Capture" in the menu
Click on the failing capture file
It should now be stuck in the loading process and never finish

If you need a copy of the exe/source, please let me know. I'd be happy to email it. The source is proprietary so I'd rather not share it publicly.

Environment

RenderDoc version: v1.18
Operating System: Windows 10
Graphics API: Vulkan
GPU: NVidia GTX 1060 6GB
Driver Version: 511.65

I'm not able to reproduce this unfortunately, the capture loads successfully for me. However I have noticed recently that nvidia's drivers no longer reliably report device lost and instead tend to hang, so I suspect this is a device lost error.
I fixed a couple of validation errors that were found when loading the capture (they didn't seem to be your fault, but mine). Can you test again on the next nightly build to see if that fixes it? if not we can go from there, though if you have a different GPU you're able to test on that would be good to try.

I don't think your commits are in the nightly build yet. When the Feb, 24 build goes on the website I'll download and report back.

Yes sorry, nightly builds go up at the end of the day UTC (UK time) around 4AM or so.

Downloaded the nightly and also upgraded my driver version to 511.79 in case that would fix the device error you were talking about.
The application no longer hangs, but I get a new message saying "Replaying the capture failed at the API level"
I've attached the log output
failing_capture.log.txt

Looks like there's a minor bug that doesn't properly report device lost on open as device lost, I've fixed that but going by the log that's still what's happening so it doesn't change much but the text of the dialog message.
Unfortunately I'm still not able to reproduce this at all. I don't have an exact match for your GPU and driver, partly because the closest GPU I have a 1080 is in my primary development machine on windows 7 so it doesn't have the same driver available.
Without any validation errors and without being able to reproduce there's not much I can do directly. Do you have a different machine or GPU you're able to test this on? Are you able to capture other vulkan programs OK e.g. Sascha's demos? if so can you try simplifying your program down to the smallest possible case that still fails to load - if you can get it down to a single triangle draw I can try and inspect what's going on by hand to see if I see anything suspicious, or compare to a working program to see what's different.

Narrowed it down to the text renderer. Trying to reproduce a minimum case

Ok, I was able to narrow it down to two draw calls with two quads. It has something specifically to do with rendering the texture of the little red enemy and then a font texture. I have no idea why.
If you replace the enemy texture with anything other texture the capture loads fine. If you don't draw the font quad, just the enemy texture the capture loads fine, and vice versa.
failing_capture2.zip

Unfortunately that capture still loads fine for me. I'm not quite sure what you mean with

If you replace the enemy texture with anything other texture the capture loads fine.

Do you just mean changing the contents of the texture, or since I see you're using descriptor indexing do you mean indexing a different descriptor?
On that note I see you're using descriptor indexing without enabling the physical device feature. I don't know if nvidia does anything with them, but you should be enabling shaderSampledImageArrayDynamicIndexing. In fact both of the draws use a fragment input as a descriptor index which is not dynamically uniform, so this usage would require the EXT_descriptor_indexing extension and the shaderSampledImageArrayNonUniformIndexing feature, and the access would need to be marked as NonUniform in the shader.
This also suggests a further avenue of simplification: what happens if you pass that index via a dynamically uniform source such as from a uniform buffer or push constant? Or if that doesn't work, if the index is hardcoded to a constant? What if you don't use descriptor indexing at all and only bind a single non-arrayed texture?

All textured quads are drawn with a single draw call and a large instance count with the array of textures index passed as a member of the per-instance data SSBO to reduce descriptor set rebinding calls. So using push constants or uniform buffers for the index into the array of textures wouldn't work in this case.
I'll take a look at your other suggestions and get back to you.

Made your suggested changes and renderdoc is still going into an infinite loop. It was hard to find resources on nonuniformEXT, and I'm a graphics API newbie so I might have made an obvious mistake in the code:
failing_capture3.zip
Per your suggestion I hard coded the texture indexes just to see what happens. Still hangs:
failing_capture4.zip
Is there some sort of debug build I can download to get a stack trace of what it's getting stuck in?

That first one looks fine now. Though in the hardcoded case the second draw still uses a non-uniform instance index to look up the texture, so it's not a complete test of that factor.
If hardcoding both shader indices doesn't fix it though I'd suggest then trying to simplify further and remove the descriptor array entirely and sample directly from a single texture binding, see if that fixes it.
You don't need a debug build to see the stack trace, you can attach VS to renderdoc directly and use the symbol server at https://renderdoc.org/symbols - though you can build the source from this repository if you want a true debug build. However I'm 99.9% sure that you will find it is hanging inside the nvidia driver code, as I have found that very often when a device lost error happens they don't report it properly and instead hang inside the next call that might need to block. That's especially likely since sometimes you see a device lost error actually reported.

This may be unrelated by the latest version (1.18) fails to load captures as well for me - tried it on 2 different Vulkan apps, it simply exits when I close the app to check the captured frame. Downgraded to 1.17 and it seems to work fine there.

Please do not comment on existing bugs with an unknown problem, please open a new bug and fill out the template to provide the relevant information for me to investigate.

I apologize, but I can't devote any more time to debugging this. I need to rewrite the text rendering for the app anyway, and I got nvidia's debugger working so I need to move on. Thanks for your time.


dep ensure -add does not recursively introduce sub-dependencies from projects that do not vendor their imports

What version of dep are you using (dep version)?
dep:
 version     : v0.4.1
 build date  : 2018-01-27
 git hash    : 37d9ea0
 go version  : go1.9.3
 go compiler : gc
 platform    : darwin/amd64

What dep command did you run?
dep ensure -add google.golang.org/grpc
What did you expect to see?
imports from within google.golang.org/grpc also vendored.
vendor dir should have both:

google.golang.org/grpc
github.com/golang/protobuf/proto (imported by google.golang.org/grpc/encoding/proto/proto.go)

What did you see instead?
only google.golang.org/grpc was vendored (sub-dependencies were not introduced).

@marcusljx What kind of [prune] settings are you using in your Gopkg.toml? Setting unused-packages = True would exclude github.com/golang/protobuf/proto if you aren't using the google.golang.org/grpc/encoding/proto package (directly or indirectly).

@mattayes github.com/golang/protobuf/proto is in use by google.golang.org/grpc/encoding/proto, which is in use by my project.
For context, my prune settings is
[prune]
  go-tests = true
  unused-packages = true

@marcusljx I think I've realized what the issue is: google.golang.org/grpc and google.golang.org/grpc/encoding/proto are two different packages. Only the latter package has a dependency on github.com/golang/protobuf/proto, which is why you're not seeing it after you run dep ensure -add google.golang.org/grpc. Runing dep ensure -add google.com/golang.org/grpc/encoding/proto will add github.com/golang/protobuf/proto to vendor/, as expected.
You can find more information on this in the Daily Dep guide.

@marcusljx Good catch! If that answers your question can you close this issue?


Mitigate AI SDK's problem with numeric in properties

This is requested by VS business insights. The application insight SDK will convert numeric value in the properties values to string type. To allow these values types unchanged, numeric values should be saved through measurements argument on track_event method. It is a odd design, but it is what it is.

Codecov Report

Merging #2397 into master will decrease coverage by -0.04%.
The diff coverage is 7.69%.

@@            Coverage Diff             @@
##           master    #2397      +/-   ##
==========================================
- Coverage   72.33%   72.29%   -0.04%
==========================================
  Files         323      323
  Lines       18273    18284      +11
  Branches     2701     2704       +3
==========================================
+ Hits        13217    13218       +1
- Misses       4223     4233      +10
  Partials      833      833

Impacted Files
Coverage Δ

.../azure-cli-core/azure/cli/core/telemetry_upload.py
21.95% <7.69%> (-4.72%)
:x:

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 4c0fdcf...15dd2b0. Read the comment docs.

@derekbekoe?


404 to access iOS API docs

Idea Title
https://docs.microsoft.com/en-ca/xamarin/ios/
links to
https://developer.xamarin.com/404?link=%2Fapi%2Froot%2Fmonotouch-lib%2F
for iOS API docs (bottom)
Issue Details
404

A correct link is https://developer.xamarin.com/api/root/ios-unified/

@Khairunj Can you correct the link?

@conceptdev is already on the case! 🎉
Thanks @spouliot for the report 👍

fixed!

@conceptdev Thanks for quickly fixing. I verified that it works. So close it.
@spouliot Again thanks for the report.


Updating terra-date-picker to use DisclosureManager v3

Summary
The terra-date-picker has examples/tests that include date pickers in modals. I'm updating those examples to use the new APIs provided by DisclosureManager v3.

@bjankord Regarding a major version bump, only examples and tests are impacted here; the DatePicker API has not been altered in any way. The way this change would be non-passive is if someone was incorrectly and transitively dependent on the modal manager dependency in this package.

Regarding a major version bump, only examples and tests are impacted here; the DatePicker API has not been altered in any way. The way this change would be non-passive is if someone was incorrectly and transitively dependent on the modal manager dependency in this package.

Ok, I think I'm fine with that. My main concern is around apps pulling in / bundling 2 versions of modal manager without them really knowing it.
e.g. App defines modal-manager ^3.0.0 in package.json, and date-picker now pulls in modal-manager ^4.0.0, the bundle now ends up with 2 versions of modal manager.
Node modules directory ends up looking something like this after install:
my-app/
  node_modules/
    terra-modal-manager (v3)
    terra-date-picker (v2)
      node_modules/
        terra-modal-manager (v4)

If we roll out this change as a major release, consumers can still get mixed versions by simply pulling modal manager v3 and datepicker v3 together, but if they are concerned about the bundle size, they can either roll back to datepicker v2 (requires no development work) or update to modal manager v4 (requires development work).
With rolling this change out as a passive release in datepicker, if consumers have the example above, the only way to remove the mixed versions of modal manager is to update their main depdency on modal manager to v4, which requires some additional development time.
I don't think it's a big concern, but something worth thinking through.

I think my wires got crossed somewhere. I updated the Changlog for the DatePicker but made changes for the DateTimePicker. There's also a few DatePicker examples that need updating, so I'll go through and do that real quick.


Player can move while "Dead"

During the implementation of the #2 Enemy AI System. I found out that when the AI kills the player. The player can freely move when in death animation.
Player should stay at the location, where he/she died. (Resurrection will be available later in an up coming release)

Another bug, just caught my attention. Enemy also does the same behaviour with the player. I'll also fix this in this issue.
Screenshot:

Bugfix: d072609
Fix has been implemented.


Documentation: Max & Min

Do the checklist before filing the issue:

[ ] Do you have Node.js and all the relevant dev-dependencies installed?
[ ] Is this a bug fix?
[x] Is this an enhancement?
[ ] Is this a feature request?

NOTE: Provide a clear and concise description of the feature that needs to be added! Or if its a bug, then provide the necessary steps to reproduce it along with screenshots.

Give your answer below:

Provide brief documentation on both M.Max() & M.Min() inside the docs folder!
A blank index.md is already provided for each module
Required fields: a brief intro, Syntax, Arguments, Return, Examples, Info
For exact formatting, refer here!

@pbiswas101 I would like to work on this issue.

@vsharma20v  you're assigned!


Bug: Fix Text alignment in the cards

Describe the bug
The text is not properly aligned in the card. It would look more presentable if the description and bottom border will have a little space in between them.
Add Screenshots

Steps To Reproduce

Go to 'Home page'
Scroll to the second section

The expected behavior

Good amount of space between the text and the bottom border.
Add colors and animations to the cards.

Can you please assign me this issue under ggsoc'24?

Thank you for assigning. Will start the work right away.


The autoload file path is something like "../../../../../.emacs.d/straight/build/company/company"

What's wrong
Today, when I open my emacs, some errors happened like:
Cannot open load file: No such file or directory,
../../../../../.emacs.d/straight/build/nasy-tools/nasy-tools-company-function

When I searched for the reason, I found some file paths of the auto-generate autoloads files inside straight/build/xx is like ../../../../../.emacs.d/straight/build/xx. E.g.:
 (custom-autoload 'global-company-mode "../../../../../.emacs.d/straight/build/company/company" nil)

  (autoload 'global-company-mode "../../../../../.emacs.d/straight/build/company/company" "\
  Toggle Company mode in all buffers.
  With prefix ARG, enable Global Company mode if ARG is positive; otherwise,
  disable it.  If called from Lisp, enable the mode if ARG is omitted or nil.

  Company mode is enabled in all buffers where `company-mode-on' would do it.

  See `company-mode' for more information on Company mode.

  \(fn &optional ARG)" t nil)

  (autoload 'company-manual-begin "../../../../../.emacs.d/straight/build/company/company" nil t nil)

My .emacs.d is a soft link of ~/.config/nasy-emacs.  When using the relative position of going to the parent folders and returning, it looks for ~/.config/.emacs.d/ instead of ~/.emacs.d/ or ~/.config/nasy-emacs/
Currently, I use gsed -i 's#../../../../../.emacs.d#/Users/Nasy/.emacs.d#g' stright/build/*/*.el to do a temporary fixing.
Is there any way to force it to generate an absolute path or a relative path without "../.."?
Directions to reproduce
For "../../../../../.emacs.d/xxx", I am not sure what caused this to happen.  Some of them still keep (autoload 'xx "xx").
And you could try:
git clone https://github.com/nasyxx/emacs.d.git ~/.config/nasy-emacs
ln -s $HOME/.config/nasy-emacs $HOME/.emacs.d
cd $HOME/.emacs.d/
make generate -j
emacs

Version information

Emacs version: emacsGit build with https://github.com/nix-community/emacs-overlay
Operating system: macOS 11.2.1

What is the output of M-x straight-version?

If you could please provide the output of this as well (prior to applying your gsed workaround):
(straight--autoloads-file "company")

M-x straight-version
prerelease (grafted, HEAD -> develop, origin/develop) c94516d 2021-02-23
(straight--autoloads-file "company")
"/Users/Nasy/.emacs.d/straight/build/company/company-autoloads.el"

I'm actually having the same problem and can verify that specific packages are generating autoloads with a hardcoded path. I've had this problem for several months now, but it goes away when I switch back to Emacs 27 from Emacs 28.

It would be helpful if you could reproduce the problem via straight-bug-report.

Okay, after narrowing down the problem it seems to me that there is an issue with (setq find-file-visit-truename t). Similar to OP, I have my emacs symlinked from my dotfiles folder to ~/.config/emacs. Here is the output of straight-bug-report.

Test Case
(straight-bug-report
  :pre-bootstrap
  (setq-default straight-repository-branch "develop")
  (setq-default straight-fix-org t)
  (setq-default straight-fix-flycheck t)
  (setq-default straight-use-package-by-default t)
  (setq-default straight-check-for-modifications
		'(check-on-save find-when-checking))
  :post-bootstrap
  (setq-default use-package-verbose nil)
  (setq-default use-package-always-defer t)
  (setq-default use-package-compute-statistics nil)
  (setq-default use-package-expand-minimally t)
  (setq-default use-package-enable-imenu-support nil)
  (straight-use-package 'use-package)
  (use-package files :straight
    (:type built-in)
    :init
    (setq-default find-file-visit-truename t))
  (use-package editorconfig :straight t :hook
    (emacs-startup . editorconfig-mode))
  (message "bye"))

Test run at: 2021-09-02 15:07:04
system-type: darwin
straight-version: prerelease (HEAD -> develop, origin/develop) 45eb611 2021-08-13
emacs-version: GNU Emacs 28.0.50 (build 1, x86_64-apple-darwin19.6.0, NS appkit-1894.60 Version 10.15.7 (Build 19H1323)) of 2021-09-02

Output
Bootstrapping straight.el...
Bootstrapping straight.el...done
Rebuilding all packages due to build cache schema change
Looking for gnu-elpa-mirror recipe → Cloning melpa...
Looking for gnu-elpa-mirror recipe → Cloning melpa...done
Looking for emacsmirror-mirror recipe → Cloning gnu-elpa-mirror...
Looking for emacsmirror-mirror recipe → Cloning gnu-elpa-mirror...done
Looking for emacsmirror-mirror recipe → Cloning el-get...
Looking for emacsmirror-mirror recipe → Cloning el-get...done
Looking for straight recipe → Cloning emacsmirror-mirror...
Looking for straight recipe → Cloning emacsmirror-mirror...done
Building straight...
Building straight...done

Test run with version: prerelease (HEAD -> develop, origin/develop) 45eb611 2021-08-13
Cloning use-package...
Cloning use-package...done
Building use-package...
Building use-package → Building bind-key...
Building use-package → Building bind-key...done
Building use-package...
Building use-package...done
Cloning editorconfig-emacs (for editorconfig)...
Cloning editorconfig-emacs (for editorconfig)...done
Building editorconfig...

Save the above init-file to someplace like ~/dotfiles/emacs/init.el
Run ln -s ~/dotfiles/emacs ~/.emacs.d
Run emacs

This will give the error Cannot open load file error. What's strange is, if you symlink just the init.el it won't cause a problem. But it doesn't seem to like when the entire directory is symlinked from somewhere else.

Also, I can only reproduce this error on Emacs 28. I just tried downgrading to 27.2 and do not get an error when launching emacs.

Thanks for the additional info, @dylanjm.
This likely means something has changed in Emacs 28.
Looking through anything related to symlinks I see the following:
https://github.com/emacs-mirror/emacs/commit/657fe8b9d87dd17d9b50dd8e15bfd917c6c630b2
d18b1c8c7583b981d4f3d687e49e82250d51634f
d23723cfb24b87170f53ce615bff083ef2d7e76a
It looks like this may also be Mac specific. Not sure what the differences, if any, are on that OS.

I see someone emailed the Emacs dev thread about this, but it did not get a response. If either of you are still experiencing this issue, I would recommend joining in on that thread, as it may help the maintainers develop a solution.
https://lists.gnu.org/archive/html/emacs-devel/2021-11/msg00511.html

@progfolio I've CCed @larsmagne in latest mail list, Hope we can get this fixed in upcoming Emacs-28.

Hi all, this happens to me as well, on Arch linux, using emacs28-git from AUR.
You can check this Reddit thread where I was describing the issue: https://www.reddit.com/r/emacs/comments/rcg4r2/help_transitioning_to_emacs_28/
Somebody amazing left a comment and I ended up in this thread.
Please ask me anything.

@croqaz Could you file a bug via report-emacs-bug and post the link to this mail thread.

@croqaz Could you file a bug via report-emacs-bug and post the link to this mail thread.

They needn't file a separate bug report. They can reply to the existing thread if they have new information to add.

Closing this issue for now. The upstream bug report shows the error is reproducible without straight.el. Unfortunately, it does not seem like it has received any attention from the maintainers. Anyone affected by this, I recommend you try the repro test provided here: https://lists.gnu.org/r/bug-gnu-emacs/2021-12/msg00376.html
If you can reproduce the error with that, it may be worth adding to that thread.

I also ran into this on Windows with Emacs 28 just now (but not on Linux with Emacs 29.0). Uninstalling 28 and reinstalling 27.2 resolved it right away.
Note that the linked bug reporter mentions this workaround albeit for package-install:

Currently I use this snippet to patch the issue up:
(defun faerryn-patch-package-find-file-visit-truename (oldfun &rest r)
  (let ((find-file-visit-truename nil))
    (apply oldfun r)))
(advice-add #'package-install :around
            #'faerryn-patch-package-find-file-visit-truename)

I imagine because it's package-install that directly/indirectly causes the autoload generation, when using the stock package manager.
Does anyone know if there's a similar straight.el function we can advise for now to temporarily unset find-file-visit-truename when autoload generation occurs, then re-activate it? It seems like doing so would work around the issue which would at least allow us to use straight.el + Emacs 28 + symlink .emacs.d while the maintainers figure out the issue.

Does anyone know if there's a similar straight.el function we can advise for now to temporarily unset find-file-visit-truename when autoload generation occurs, then re-activate it?

https://github.com/raxod502/straight.el/blob/af5437f2afd00936c883124d6d3098721c2d306c/straight.el#L4979
However,  advising the function is not a good solution. That function should be patched to lexically bind find-file-visit-truename.  Pull requests welcome.
I would recommend bumping the linked Emacs bug report first.  This should be solved upstream.

Thank you @progfolio!
I suppose I'll try advising it just to see if that workaround works first.

That function should be patched to lexically bind find-file-visit-truename. Pull requests welcome.

By this, do you mean that we should add find-file-visit-truename to the let binding list? Then its value would be nil if emacs 28, otherwise take the "outer"/global value? Or would we unconditionally set it to nil?
I am definitely happy and willing to bump that bug report but admittedly I have zero idea how to go about doing that. My understanding is that this is a mailing list, so if I subscribe to the mailing list now, I won't get that email since it already happened in the past, so how can I reply to it? It's been a long while since I worked with mailing lists. Would you be kind enough to share some directions?
It seems like I'm supposed to click on this button:

and then also add<EMAIL_ADDRESS>to the To: field based on this answer.

I can confirm this workaround works:
(defun my-patch-package-find-file-visit-truename (oldfun &rest r)
  (let ((find-file-visit-truename nil))
    (apply oldfun r)))

(advice-add #'straight--build-autoloads :around
            #'my-patch-package-find-file-visit-truename)

But I acknowledge you're saying that's not the ideal solution, so I'm also happy to both:

Contribute a PR for the "lexical binding" once I hear back from my earlier question and
Bump that bug report once I figure out how to do so

By this, do you mean that we should add find-file-visit-truename to the let binding list? Then its value would be nil if emacs 28, otherwise take the "outer"/global value? Or would we unconditionally set it to nil?

Yes, that's the general idea. Hopefully, we can unconditionally bind find-file-visit-truename to nil here:
https://github.com/raxod502/straight.el/blob/e2de88ea0e7ccf9f4846e9e756f86e0dcf469c44/straight.el#L5038-L5053
I'll have to do some testing/research to see if this is OK for versions prior to 28, or if we need to condition the value on that version.
Let's hold on that part for now and focus on getting this bug fixed upstream first.
You have the correct idea about replying to that bug. Especially the part about manually adding<EMAIL_ADDRESS>as a recipient.
You can put a note in your message that mentions you're not subscribed to the mailing list and to please CC you.
Thank you for taking this on.

Thank you! I did post a reply/bump yesterday, I think/hope I did it correctly:
https://lists.gnu.org/r/bug-gnu-emacs/2022-04/msg00444.html
I also subscribed to the mailing list just now.

As a further note for myself, I just learned here that the bug report is visible here as well, and here.
Also added some of this information to that same wiki page. I'm not sure what "official" affiliation it has with emacs, if any (I doubt it does), but figured I'd add it there since it came up high in Google search results.

Hi all - I'm experiencing a similar issue. Note this is a cross-post from my stackexchange.
Note: I'm posting here, because all comments point back to this thread, and my SE question is still not resolved.
I'm running emacs (v28.1) on my macOS (Montery v12.2.1) and I get the following message when trying to use magit:
command-execute: Cannot open load file: No such file or directory, ../../../../../../../.emacs.d/straight/build/magit/magit-status

My init file is here. Importantly I have  my ~/.emacs.d is symlinked to my dropbox folder:
.emacs.d -> ~/DROPBOX/REPOS/ss_dotfiles/.emacs.d

I tried the following suggested workarounds:

@kisaragi-hiu in #944 in my init file here
@blaenk in this patch in my init file here

Neither of these workarounds fix this issue with straight.el loading projectile.el. I've tried restarting emacs, the macOS etc, but the same problem persists.
@blaenk @progfolio @kisaragi-hiu - based on my init file, am I implementing your suggestions correctly?

Hi all, I also have the exact same problem with Emacs stable 28.1 on Arch.
I installed go-mode and I'm seeing the exact same error as above.
I eventually deleted the ~/.emacs.default/straight/build folder and restarted Emacs and now it's working, after straight + Emacs rebuilt all the repos.

I eventually deleted the ~/.emacs.default/straight/build folder and restarted Emacs and now it's working, after straight + Emacs rebuilt all the repos.

@croqaz - that worked! Many thanks.
To be more precise I did the following:

Backup ~/.emacs.d dir
Deleted the entire ~/.emacs.d/straight dir.

This is more than what @croqaz suggested, but I just wanted to be quite cautious to resolve this issue.
Re-installing everything from scratch took less than a minute for me. Thanks to straight.el 👍🏾

Ensure that (setq find-file-visit-truename nil) is included in my init file after bootstrapping straight.el.
Do a quick search for setq find-file-visit-truename and check that we have set it to nil and not t anywhere else.   -

I had it set to t in my dired, which I removed.
This did cause me issues, but were resolved after removing it and setting it to nil only once as per above.