Subheadings do not account for combined Types (type + tags)

Just noticed this when opening the statblock for Tarrasque, which has
{
"type" : {
"type" : "monstrosity",
"tags" : ["Titan"]
}
}

As its type, and in the beta it's parsed as [Object object], since in the Svelte file for the Subheading, properties are currently treated like Strings. I have tried out a quick solution for this, but I'm unable to compile it as I'm not too well-versed in TypeScript.
Could this be a solution?
if(monster[property]['tags'] !== undefined){
 subheading.push(`${monster['type']}`);
 subheading.push('('+`${monster['tags']}`.join(', ')+')');
}
else{
 subheading.push(`${monster[property]}`);
}

granted it's pretty specific, but my current assumption is this will only be relevant for this very case.

Interesting find. Let me think about it. Maybe I need to parse properties recursively to combine things 🤔

That would of course be the more generalized solution. Going through each property in an organized manner.

Just noticed that there are a few other cases (not in the Subheading though, but actually in other descriptions) that there are sometimes objects being fed into the JSONs I receive from 5etools. Sometimes I see things like "bludgeoning, piercing, slashing damage from nonmagical sources" or things like AC having a case of normal and Mage Armor enabled AC in there.
Last thing I noticed, but I think that may just be inconsistent source material, is something I saw once or twice, is the sheer number for AC being in an Array and therefore also being recognized as a non-integer in parsing.

Yeah, 5eTools is kind of a pain to deal with. I’ll make it generalized soon

I figure! I wrote a quick parser for the source files, but I do not want to deal with cloned Sheets + Adjustments, like "this creature is an Animated Armor but with the following changes". Sheesh. :)

This should be closed in 2.2.1


Add the typeOf Deluge function

Hey! Added the page for a function to get a type of variable in the Deluge, it took me hours of work and debugging to write this Deluge function some time ago.
Deluge has very weird types, everything is converted to string at some point, e.g. when calling another function with a parameter. However, testing some operations on the variable can help determine the type.

Good idea on a type check function. Deluge functions returning strings under the hood has been quite annoying when not working directly within a Zoho app.

This is great. Thanks for this. Will use in the future.


Astro deployment delete/remove fail (CloudFront "resource in use" errors)

Unable to delete Astro website deployed with SST even after multiple attempts.
Project Setup

Monorepo project setup based of the "AWS monorepo" example

├── infra
│   ├── config.ts
│   ├── cron.ts
│   ├── email.ts
│   ├── functions.ts
│   ├── index.ts
│   ├── notifications.ts
│   ├── secrets.ts
│   ├── sites.ts
│   └── storage.ts
├── package.json
├── packages
│   ├── core
│   ├── functions
│   └── website
├── pnpm-lock.yaml
├── pnpm-workspace.yaml
├── sst-env.d.ts
├── sst.config.ts
└── tsconfig.json

Environment

SST "^3.0.15"
Astro "^4.8.6"
Node 20 LTS
PNPM 8.15.1
Monorepo project

SST CLI output
| Error Website sst:aws:Astro → WebsiteServerCachePolicy aws:cloudfront:CachePolicy CloudFront: DeleteCachePolicy, https response error StatusCode: 409, RequestID: c5460bf5-755b-44bb-8e20-353f02cef48d, CachePolicyInUse: The specified cache policy is currently associated with a cache behavior. Please disassociate the policy before deleting.
| Error Website sst:aws:Astro → WebsiteCloudfrontFunctionServerCfFunction aws:cloudfront:Function CloudFront: DeleteFunction, https response error StatusCode: 409, RequestID: a59d6594-2403-452d-aa15-522a76f80bc8, FunctionInUse: Cannot delete function WebsiteCloudfrontFunctionServerCfFunction-c076cb6, it is in use by 1 distributions
| Error Website sst:aws:Astro → WebsiteCloudfrontFunctionImageServiceCfFunction aws:cloudfront:Function CloudFront: DeleteFunction, https response error StatusCode: 409, RequestID: c237c36f-abcd-1234-9314-8202c553aa67, FunctionInUse: Cannot delete function WebsiteCloudfrontFunctionImageServiceCfFunction-62c0fb2, it is in use by 1 distributions
CloudFrontOriginAccessIdentityInUse: The CloudFront origin access identity is still being used.: CloudFrontOriginAccessIdentityInUse: The CloudFront origin access identity is still being used.
    at de_CloudFrontOriginAccessIdentityInUseRes (/Users/x/Work/y/z/.sst/platform/node_modules/@aws-sdk/client-cloudfront/dist-cjs/protocols/Aws_restXml.js:7254:23)
    at de_DeleteCloudFrontOriginAccessIdentityCommandError (/Users/x/Work/y/z/.sst/platform/node_modules/@aws-sdk/client-cloudfront/dist-cjs/protocols/Aws_restXml.js:3308:25)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async /Users/x/Work/y/z/.sst/platform/node_modules/@smithy/middleware-serde/dist-cjs/index.js:35:20
    at async /Users/x/Work/y/z/.sst/platform/node_modules/@aws-sdk/client-cloudfront/node_modules/@aws-sdk/middleware-signing/dist-cjs/awsAuthMiddleware.js:30:20
    at async __f4. (:161:46)
    at async /Users/x/Work/y/z/.sst/platform/node_modules/@aws-sdk/client-cloudfront/node_modules/@aws-sdk/middleware-logger/dist-cjs/loggerMiddleware.js:7:26
    at async __f1. (:657:9)
| Error Website sst:aws:Astro → WebsiteOriginAccessIdentity sst:aws:OriginAccessIdentity The CloudFront origin access identity is still being used.

× Failed
   WebsiteCdnSsl sst:aws:Certificate → WebsiteCdnSslCertificate aws:acm:Certificate
   ACM: DeleteCertificate, https response error StatusCode: 400, RequestID: 6706097f-d43c-abcd-bfae-45327879a926, ResourceInUseException: Certificate arn:aws:acm:us-east-1:XXXXXXXXXXXX:certificate/3aef96bd-1234-4321-abcd-d470cc8ec629 in account XXXXXXXXXXXX is in use.
   Website sst:aws:Astro → WebsiteServerCachePolicy aws:cloudfront:CachePolicy
   CloudFront: DeleteCachePolicy, https response error StatusCode: 409, RequestID: c5460bf5-755b-44bb-8e20-353f02cef48d, CachePolicyInUse: The specified cache policy is currently associated with a cache behavior. Please disassociate the policy before deleting.
   Website sst:aws:Astro → WebsiteCloudfrontFunctionServerCfFunction aws:cloudfront:Function
   CloudFront: DeleteFunction, https response error StatusCode: 409, RequestID: abcd1234-2403-452d-aa15-522a76f80bc8, FunctionInUse: Cannot delete function WebsiteCloudfrontFunctionServerCfFunction-c046cb6, it is in use by 1 distributions
   Website sst:aws:Astro → WebsiteCloudfrontFunctionImageServiceCfFunction aws:cloudfront:Function
   CloudFront: DeleteFunction, https response error StatusCode: 409, RequestID: c237c36f-7eb1-47f1-9314-8202c553aa67, FunctionInUse: Cannot delete function WebsiteCloudfrontFunctionImageServiceCfFunction-62c0fb2, it is in use by 1 distributions
   Website sst:aws:Astro → WebsiteOriginAccessIdentity sst:aws:OriginAccessIdentity
   The CloudFront origin access identity is still being used.

   ELIFECYCLE  Command failed with exit code 1.
 ```

Still facing issues with AWS SSL Certificates
|    sdk-v2/provider2.go:520: sdk.helper_schema: error deleting Cloudflare Record: Authentication error (10000<EMAIL_ADDRESS>|  Error       TrpcApiSsl sst:aws:Certificate → TrpcApiSslCNAMERecord7bfa9dd51687dbf82144618ec1c49043devtrpcapiweckercloud cloudflare:index:Rec
ord
|  Record: Authentication error (10000)

✕  Failed
   TrpcApiSsl sst:aws:Certificate → TrpcApiSslCNAMERecord7bfa9dd51687dbf82144618ec1c49043devtrpcapiweckercloud cloudflare:index:Record
     sdk-v2/provider2.go:520: sdk.helper_schema: error deleting Cloudflare Record: Authentication error (10000<EMAIL_ADDRESS>
   TrpcApiSsl sst:aws:Certificate → TrpcApiSslCNAMERecord7bfa9dd51687dbf82144618ec1c49043devtrpcapiweckercloud cloudflare:index:Record
   Record: Authentication error (10000)

Unable to delete manually

This is using Cloudflare? Can you share relevant parts of your config?

@jayair I solved this by manually deleting all the mentioned resources from AWS Console ApiGateway and Certificate Manager (ACM). It looks like it was caused by moving my domain name from AWS to Cloudflare... I remember changing the dns in config to Cloudflare DNS and I've been encountering this error ever since. Ever since I deleted the stage and these resource manually and redeployed it works

I see, closing for now then. Lmk if you run into this again.


Use new grid layout placeholders for the cookie and header bar

inner-block and outer-block have been deprecated, remove these
and replace with the new grid layout placeholders.
This affects the global header bar (the blue line underneath the
header) and the global cookie message (shown at first visit, or when js
is disabled).
Smaller screens:

Tablet:

Desktop:

This makes the code much nicer. Thanks for this @gemmaleigh!

This is :+1:, long live the new grid.
Is there a risk of anyone hanging styles/behaviour off the removed selectors? Might be worth mentioning in release notes when we do.

The only thing I can think of - %site-width-container styles are applied using the paragraph selector #global-cookie-message p  so if the cookie message was to have any other content, e.g. if <%= yield :cookie_message %>  was used and the content inside it wasn't a paragraph, that content wouldn't have the correct alignment.
I didn't want to start adding extra inner divs, but it might be to sensible to also wrap  <%= yield :cookie_message %>  in a <p> tag, but only if we expect this content to be a single paragraph.


Improve ATC Stats Tracking

In gitlab by @A-Lawrence on Dec 19, 2015, 20:54
Based on #190 - improve ATC Stats tracking such that:

All ATC sessions are tracked with minimal footprint
Making use of the queue to push this task into, so it doesn't keep running for lengthy periods.

In gitlab by @A-Lawrence on Dec 19, 2015, 21:05
mentioned in commit 63b1bb3c976837e30e25ff8269216b824e93e6e3

In gitlab by @A-Lawrence on Dec 20, 2015, 18:49
mentioned in issue #36

In gitlab by @A-Lawrence on Dec 20, 2015, 19:27
Milestone removed

In gitlab by @A-Lawrence on Dec 30, 2015, 19:45
Status changed to closed by commit 63b1bb3c976837e30e25ff8269216b824e93e6e3

In gitlab by @A-Lawrence on Dec 30, 2015, 19:45
Resolved on branch feature/atc_statis_tracking_basic and resolved in commit   3b98374e


How do I see the response headers?

See title.

Use the -t option to trace the request and response headers.


Changes to game not fully disabling.

Certain features of this mod, particularly those related to classes, don't appear to fully disable when the option is turned off in the mod manager menu. The problems it's causing me are rather minor, and I've managed to work around them for the most part, but it has implications for compatibility with other mods. I don't know how concerned you are about that, but I thought I was worth mentioning.
Details: I'm an experienced modder, but I don't know C# and I'm not experienced at using the tools used to mod this game. However, using the game's blueprints and some of the information at https://github.com/OwlcatOpenSource/WrathModificationTemplate I have been able to create mods for my own use by just modifying the game's blueprints or adding my own along with calls for my own blueprints in modified game blueprints. I've been using this to experiment with some homebrew class concepts (and just for fun, because I like fun). It works by modifying a blueprint and putting it in a 'Blueprints' folder in the bundles folder.
For example, I like insects and was rather disappointed in the way the swarm that walks was implemented (I don't do evil characters, as a rule.) So I 'borrowed' some of the art assets used for swarm that walks and for other spells and abilities in the game and created a custom insect themed alchemist class to replace the Incense Synthesizer archetype.
When I first installed your mod, I found that it interfered with my own simpler one, but that I could get around that by simply disabling the table top tweaks features that related to alchemists. I also successfully modified a couple other archetypes, feats and so forth. So long as any related tabletop tweaks features were disabled, my own blueprints would load. If they were enabled, my own blueprints wouldn't load and it would use the default blueprints with tabletop tweaks changes instead. Interestingly, I found that I could actually switch back and forth between the two without any noticeable problems.
However, I recently tried to modify an oracle archetype to use the druid spell list instead of the oracle spell list (the divine herbalist, to be specific). It didn't work. After some trouble shooting, I found that it worked just fine if I turned off tabletop tweaks base. To make a long story slightly shorter, I disabled every feature in Base but left the mod enabled, and my altered blueprint still wouldn't load into the game unless base was turned off (set not to load in the game at all as opposed to having the features disabled in base's menu). This affects ALL oracle archetypes and the oracle base class.
So far I've identified druid, hunter, bard, the bard talent list, and hunter as having the same issue.

The only things present would be some core fixes that everything depends on
that don't modify classes. I'd need logs to know what is failing and if
anything can be done.
On Sun, Oct 9, 2022, 4:52 AM Biothurge @.***> wrote:

Certain features of this mod, particularly those related to classes, don't
appear to fully disable when the option is turned off in the mod manager
menu. The problems it's causing me are rather minor, and I've managed to
work around them for the most part, but it has implications for
compatibility with other mods. I don't know how concerned you are about
that, but I thought I was worth mentioning.
Details: I'm an experienced modder, but I don't know C# and I'm not
experienced at using the tools used to mod this game. However, using the
game's blueprints and some of the information at
https://github.com/OwlcatOpenSource/WrathModificationTemplate I have been
able to create mods for my own use by just modifying the game's blueprints
or adding my own along with calls for my own blueprints in modified game
blueprints. I've been using this to experiment with some homebrew class
concepts (and just for fun, because I like fun). It works by modifying a
blueprint and putting it in a 'Blueprints' folder in the bundles folder.
For example, I like insects and was rather disappointed in the way the
swarm that walks was implemented (I don't do evil characters, as a rule.)
So I 'borrowed' some of the art assets used for swarm that walks and for
other spells and abilities in the game and created a custom insect themed
alchemist class to replace the Incense Synthesizer archetype.
When I first installed your mod, I found that it interfered with my own
simpler one, but that I could get around that by simply disabling the table
top tweaks features that related to alchemists. I also successfully
modified a couple other archetypes, feats and so forth. So long as any
related tabletop tweaks features were disabled, my own blueprints would
load. If they were enabled, my own blueprints wouldn't load and it would
use the default blueprints with tabletop tweaks changes instead.
Interestingly, I found that I could actually switch back and forth between
the two without any noticeable problems.
However, I recently tried to modify an oracle archetype to use the druid
spell list instead of the oracle spell list (the divine herbalist, to be
specific). It didn't work. After some trouble shooting, I found that it
worked just fine if I turned off tabletop tweaks base. To make a long story
slightly shorter, I disabled every feature in Base but left the mod
enabled, and my altered blueprint still wouldn't load into the game unless
base was turned off (set not to load in the game at all as opposed to
having the features disabled in base's menu). This affects ALL oracle
archetypes and the oracle base class.
So far I've identified druid, hunter, bard, the bard talent list, and
hunter as having the same issue.
—
Reply to this email directly, view it on GitHub
https://github.com/Vek17/TabletopTweaks-Base/issues/503, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAWSQPHXMRFWYRLZ6UY4L63WCKIU3ANCNFSM6AAAAAARAT7SFM
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I'm pretty sure it's base, as I had core and reworks still enabled. Of course, you know more about it than I do. In any case, here are some logs.
I made a quick modification of oracle divine herbalist with more extensive changes than my original, created a new character with that archtype, then leveled them up once using toybox. Then I repeated as closely to the same actions, once with base turned completely off and once with all the options in base turned off but base enabled.
GameLogFull_BaseOptionsEnabled.txt
GameLogFull_BaseTurnedOff.txt
GameLogFullPrev_BaseOptionsDisabled.txt
Player_BaseOptionsDisabled.log
Player_BaseOptionsEnabled.log
Player_BaseTurnedOff.log

Hi I am also seeing this issue.

Again I need logs and idealing theod you are breaking things with to
identify anything.
On Wed, Oct 12, 2022, 12:09 AM Fmk47 @.***> wrote:

Hi I am also seeing this issue.
—
Reply to this email directly, view it on GitHub
https://github.com/Vek17/TabletopTweaks-Base/issues/503#issuecomment-1275596004,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAWSQPGN2BUSBTVDGP2JP63WCZBXPANCNFSM6AAAAAARAT7SFM
.
You are receiving this because you commented.Message ID:
@.***>

Looking at these logs this is 2.3.1 on game 1.4.4g, which specifically is running into issues with bloodline mechanics. All of this should be fixed in 2.4.2 already.
If you want to stay on 1.4.4g you need to upgrade to 2.3.2 as it contains fixes for this problem https://github.com/Vek17/TabletopTweaks-Base/releases/tag/v2.3.2


Use argparse for pew workon

Use argparse only and preserve --here option
Bonus: pew workon without an envname displays all available environments, instead of an error.

cc: @pfmoore

Thanks for this! You may need to rebase #191 now to clean it up.


Check if line is hidden/shown

Hi, is there any way to check the state of the line? Hidden or shown.

No reply came, then this abandoned issue is closed.


Allow relative urls in URLBlock

Is your proposal related to a problem?

URLBlock doesn't allow to insert relative URLs.
As per #3141 there isn't any block for adding external links and internal links. As a temp solution, it would be good to allow relative URLs (/internal-url/ in the URLBlock.

Describe the solution you'd like

it would be good to allow relative URLs in the URLBlock. I know that we can customize the Validators, but it would be good to have that by default in Wagtail.
Describe alternatives you've considered

Additional context

Hi @jkevingutierrez, this is because URLBlock uses Django’s URLField. I’m not sure if this is defined explicitly anywhere but I believe there’s a strong association in people’s minds that "URLBlock means URLField". If we had a block that supported different validation rules from URLField, presumably we should call it something else?

👋 I’m going to close this now based on my last comment and the lack of activity. I believe URLBlock is too strongly tied to URLField to warrant any kind of change. We could introduce another block, but this feels like the type of thing where we’d expect a clear, specific proposal based on pre-existing patterns.


Page numbering between front matter and main body of the text

Hm, not sure that this is an issue at all, this may be perfectly legitimate, so I am just asking just to make sure it does not go unnoticed.
In the PDF generated from LaTeX, the front matter has pages numbered in Roman numerals, say I to XIX, or something similar. Then, the first chapter has pages in Arabic numerals, starting at 1 and going to 390, or whatever.
In the PDF generated from DocBook, the preface ends on page XX, and the main matter (chapter 1) starts with page number 23 (with two blank numeration pages in between). So in effect, we have the following page succession: XX, (none), (none), 23.
This is perfectly consistent and I have absolutely no problem with that, I just wonder if this is the usual policy of O'Reilly. I have just looked at several books, including two relatively recent ones from O'Reilly (such as Programming Perl (4th E)), it seems that the normal policy is to restart numbering at 1 when getting to the book main matter (that's also what Lamport and Knuth do in their LaTeX and TeX books, but, with all due respect to them, I would not mind very much departing from them on such trivia).
Frankly, I personally don't care, so I am just asking Matt if O'Reilly has a definite policy on such page numbering, or if this an artifact due to the conversion. If not, anyone please close this issue.

This is done now (will be visible in the CE PDF), so I think we can close this issue.

OK, if this is now OK, fine with me to close the issue.


Added text editors and new community links

Added Discord and forums links to README
Added text editor packages to README
Changed Slack to Discord invite on CONTRIBUTING

no idea why this wasn't accepted earlier


Native federation can not be added in ESM module

With what library do you have an issue?
native-federation
Reproduction of the bug/regression with instructions
In a new Angular18+ project that has "type": "module" in its package.json, native federation can not be initialized because it uses require() and hardcoded 'federation.config.js' file name for initialization. Exception is thrown, asking to rename .js file to .cjs.
I changed hardcoded file name in libs/native-federation/src/builders/build/builder.ts  from 'federation.config.js' to 'federation.config.cjs' and worked like a charm. I can push a PR to check for .cjs first and then for .js, if needed (or even have an optional filename parameter for config file)
Here is a Stackblitz example that throws this exception:
https://stackblitz.com/edit/stackblitz-starters-1ggp6r
Expected behavior
Expected to load with no error. If an Angular lib/app can not be initialized as ESM module due to native federation then what?
We can not change our app's nature, to commonjs just for this, it leads to a chain of changes that make this unusable.
Versions of Native/Module Federation, Angular, Node, Browser, and operating system
"@angular-architects/native-federation": "18.2.2"
Angular CLI: 18.2.10
Node: 20.13.1
OS: linux x64
Browser: Chrome 130.0.6723.70 (Official Build) (64-bit)
Other information
No response
I would be willing to submit a PR to fix this issue

[X] Yes
[ ] No

Good Point. Thanks for this. It seems like the issue only comes up if people add type: "module" to their package.json. I think we need several things here:

Respect the new file name with .cjs when loading the federation config
Generating the right filename in the ng add schematic
Updating the file name in an ng update schematic
displaying an error message on the cnsole when we see an old federation.config.js

PRs are welcomed. Please link the Pr here.

@manfredsteyer PR is ready and I am trying to test it, but schematics seem broken, I get "path" undefined errors constantly after using ng add to add host/remotes.
Reason is this:
module-federation-plugin/libs/native-federation/src/schematics/init/schematic.ts , line 352
  const main =     projectConfig.architect.build.options.main ||     projectConfig.architect.build.options.browser
which are not supported any more in the native-federation builders schematic but exist in the esbuild section of angular.json
Changing to/adding
 `projectConfig.architect.esbuild.options.main || projectConfig.architect.esbuild.options.main`

fixes everything. Should I raise a new issue for this? Or add fix in this one?

Thank you

Awesome. Big thanks. I plan for another release at the End of Nov. I will look into this PR when making this release ready.

@manfredsteyer what about the update requirement? I haven't created a new update schematic but added a new migration. Is this correct or should I add a new update schematic that updates the name? Ty


Fix tutorial restart

This PR fixes two issues:
First issue:
The tutorial for a view is supposed to run only one time when the user opens the view. If the user wants to view the tutorial again, they can click on the help button.
The tutorial for AddEditUser, AddEditClassroom and AddEditChart view keeps restarting without pressing the help button.
Steps to reproduce:

Clear Local Storage.
Open Add Classroom view. The tutorial runs automatically for the Add Classroom view.
Open Edit Classroom view. The tutorial runs automatically for the Edit Classroom view.
Open Add Classroom view again. The tutorial runs automatically for the Add Classroom view again. It is not supposed to run again automatically.

42c098 fixes this issue.
Second issue:
When the tutorial is launched automatically (for the first visit in a view) and the user refreshes the page in between of the tutorial, the tutorial continues from the last step. However, when the user presses the Help button to launch the tutorial, the tutorial ends on page refresh.
57759b fixes this issue.

Nice. Thanks.


Add stacktrace support to s2n

Description of changes:

Add stacktrace support to s2n.
This slows down the error path, and makes unit tests slower, so turn it off by default and add a mechanism to turn it on
Add documentation
Add a unit test
Fix sidetrail makefiles to account for the new functions

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

:exclamation: No coverage uploaded for pull request head (stacktrace@f2c8cde). Click here to learn what that means.
The diff coverage is n/a.

hmm looks like I've managed to trigger an error that results with no stacktrace
Running s2n_tls13_handshake_test.c                         ...
Stacktrace is:
FAILED test 4240
!(((handshake_read_io(server_conn))) == (-1)) is not true  (s2n_tls13_handshake_test.c line 714)
Error Message: 'Bad message encountered'
 Debug String: 'Error encountered in s2n_client_hello.c line 151'
 System Error: Operation timed out (60)

Update the usage guide with your new apis and add to the api header?

Done

hmm looks like I've managed to trigger an error that results with no stacktrace
Running s2n_tls13_handshake_test.c                         ...
Stacktrace is:
FAILED test 4240
!(((handshake_read_io(server_conn))) == (-1)) is not true  (s2n_tls13_handshake_test.c line 714)
Error Message: 'Bad message encountered'
 Debug String: 'Error encountered in s2n_client_hello.c line 151'
 System Error: Operation timed out (60)

Try running like this
 make S2N_PRINT_STACKTRACE=1 test

hmm looks like I've managed to trigger an error that results with no stacktrace
Running s2n_tls13_handshake_test.c                         ...
Stacktrace is:
FAILED test 4240
!(((handshake_read_io(server_conn))) == (-1)) is not true  (s2n_tls13_handshake_test.c line 714)
Error Message: 'Bad message encountered'
 Debug String: 'Error encountered in s2n_client_hello.c line 151'
 System Error: Operation timed out (60)

Try running like this
 make S2N_PRINT_STACKTRACE=1 test

Actually, it looks like the problem may be a unit-test failing without S2N_ERROR macro having been invoked.  In this case, a stacktrace wouldn't tell you much: just which line of the unit test you failed on, which you already know.

looks like kwstyle is failing in travis
/home/travis/build/awslabs/s2n/error/s2n_errno.c:471: error: Line length exceed 184 (max=177)
FAILED kwstyle
The command ".travis/s2n_travis_build.sh" exited with 255.

looks like kwstyle is failing in travis
/home/travis/build/awslabs/s2n/error/s2n_errno.c:471: error: Line length exceed 184 (max=177)
FAILED kwstyle
The command ".travis/s2n_travis_build.sh" exited with 255.

I pushed a new commit that should help with the issue.  Anyone know if there is a way to run kwstyle on the mac?

looks like kwstyle is failing in travis
/home/travis/build/awslabs/s2n/error/s2n_errno.c:471: error: Line length exceed 184 (max=177)
FAILED kwstyle
The command ".travis/s2n_travis_build.sh" exited with 255.

I pushed a new commit that should help with the issue. Anyone know if there is a way to run kwstyle on the mac?

you should be able to follow the build instructions from here https://github.com/Kitware/KWStyle. They seem to suppport MacOS as well.

Thanks for adding this!


Middlewares in route annotations

I think it would be great to have this feature.

/**
 * @Middleware\IpFilter("<IP_ADDRESS>")
 * @Middleware\JsonParser()
 * @Middleware\ResponseDataFormatter("json")
 * @Middleware\Autologin()
 */
public function action()
{
    return $response
}

Collecting annotations runtime is performance killer.
We don't want to introduce container compiler pass since it complicates development and container itself significantly.
There won't be a single place to define your routes.

Collecting annotations runtime is performance killer.

It doesn't matter for long-running applications

We don't want to introduce container compiler pass since it complicates development and container itself significantly.

I think this will be a necessary measure, otherwise you will have to give up a lot of amenities.
I suggest to find out the opinion of the developers (for example, by voting on the forum).

There won't be a single place to define your routes.

Too much developers likes annotations (hello PHP 8 attributes). Annotations/attributes is preferable way to configure routes.

It doesn't matter for long-running applications

Correct but main usage of the framework won't be long running applications.

Technically that would be

General purpose annotation collector that, given a list of directories will scan for annotations.
For runtime it can be router-specific annotation collector that calls addRoute(). That could be used in an event handler bound to ApplicationStartup event.
For production it can be a console command that will collect routes and dump them into config/routes.php.

Will wait for more requests for the feature.


Multiple fields for sorting

Hi guys.
I'm using useSortBy hook from react-instantsearch-hooks-web with typesense. Is there a way to specify multiple sortings fields for search results?
Thanks

Here's how you configure the sortBy widget: https://github.com/typesense/typesense-instantsearch-adapter#sortby
The same format of items applies for react-instantsearch too.

Here's how you configure the sortBy widget: https://github.com/typesense/typesense-instantsearch-adapter#sortby
The same format of items applies for react-instantsearch too.

Yeah, documentation is clear. But it doesn't allow specify how to sort search results by multiple fields at once.  Something like that won't work.
    items: [
      { label: "Default", value: "products" },
      { label: "Popularity and price (asc)", value: "products/sort/popularity:desc,price:asc" },
      { label: "Popularity and price (desc)", value: "products/sort/popularity:desc,price:desc" },
    ],

how to sort search results by multiple fields at once

Ah I see.
That syntax should work in theory... Could you open the browser's network tab and look for a request to multi_search, and copy-paste the request body here?

how to sort search results by multiple fields at once

Ah I see.
That syntax should work in theory... Could you open the browser's network tab and look for a request to multi_search, and copy-paste the request body here?

  It's working. I wasn't sure because this functionality wasn't mentioned in docs.
Thanks!


Cant launch Angular2 Quickstart app

Hi,
I'm a newbie at angular2. I have tried to launch the Angular 2 Quickstart application but i still cant get it to work.
Here is my code;

I have seen various solutions here such as changing  "start": "tsc && concurrently "tsc -w" "lite-server" "  to  "start": "concurrently "tsc -w" "lite-server" "
but it doesn't work for me . help needed

Heya, that's an odd error. ENOENT means "Error No Entry", which usually is related to missing files/dirs.
Let's start by making sure your install is good. Can you try deleting node_modules and running npm install again?

I'm still getting the same error.
I have created another folder and followed the steps on the Angular Quickstart page.
When i run the npm install it keeps giving the error typings install failed. When i run npm run typings install ,it fails again. Although i found  npm install typings -g works.  npm start  still failes but i have been able to launch the localhost:3000 through running the command lite-server.
please help

Can you show me the errors you are getting with npm run typings?

I have idea why the command is failing on your system.. I find it weird that it says that the file and path is D:\Windows\system32, but can't say exactly what's broken.
@Foxandxss, @wardbell any idea?

I Have two OS partitions on my PC. I am running win 7 on this one. The system path and user path both point to D:/windows/system32 . I dont think that should be a problem

I confirm same problem here on Ubuntu 16.04.

I have uninstalled everything node-related and reinstalled them back. but the error is still there .please assist

My best bet is that there's is SOMETHING that's stopping NPM from finding your local modules.
Can you see if this command works successfully from your project dir?
node_modules\.bin\typings install

set your shell env variables correctly
https://github.com/kimmobrunfeldt/spawn-default-shell
$SHELL
$COMSPEC
more comments here - https://github.com/kimmobrunfeldt/concurrently/issues/60

@filipesilva the command pops up  command shell which installs typings then disappears after two seconds. the error is still there once i run npm start

@kiiru501 can you show me the long when you ran node_modules\.bin\typings install?

@jannakha added PR to bump concurrently, maybe that helps.

@kiiru501 locally, try to do npm install<EMAIL_ADDRESS>and see if npm start works again.

it disappears after a second .cant freeze it sorry

same error still even after npm install<EMAIL_ADDRESS>I keep getting this line every time  run a command

Hm... really weird that you both get that error AND get the version right afterwards. Also weird that in that error, there are double slashes \\ in some of the directory paths.
I think you have a broken NPM/Node install somehow, maybe something wrong in the PATH? You can try asking in https://github.com/npm/npm as well.

Sorry but this really doesn't relate to QuickStart. More of a stackoverflow thing. Closing.


Fixed issue causing panic when POST request is large

When POSTing a request, if the request is larger than one chunk. The logic fails due to buffer being undefined.

@romanzy313, I noticed that this required changing the tests in https://github.com/romanzy313/trpc-uwebsockets/commit/daa1df2421113f60b68da3057a38cf53ee9c30f6
My bad, this was from patching the lib via yarn patch and then adding the commit upstream. In the future I can make sure to do the test suite.
Would you like me to open a PR with a github action that runs on each PR that runs tests?

The bug was really evident when looking at it. But I wanted to make sure with a simple test.
And if it's not much work for you, it would be nice to have it automated. Should be a simple template workflow, right?

ya should be pretty easy. I will make a PR before the end of the week! We are using this in production, so happy to lend a hand! Thank you for writing this 😄


URL to ICR is out of date

https://github.com/exercism/crystal/blob/930d0d7cc1cc9560e9a68924bdefe720e2714d6e/docs/LEARNING.md?plain=1#L17
As I was peeking through the How to Learn page of the Crystal track I noticed that the URL to the ICR is out of date.
The new location is https://github.com/crystal-community/icr
Best regards.

Good catch!
I am a bit unsure what to do with that link, since the icr repo as you said has moved. But in like 2021 the official Crystal lang got a repl. Problem is that unless you use homebrew you have to manually compile a Crystal version which supports the repl. And I am unsure if I should have a doc about that or just remove the link.

My vote is just to remove it. There are too many caveats right now and I think it will be more confusing than it's worth for someone new to Crystal.

I'm new to Crystal myself (just about to dive in :smiley:) but if there indeed exists an official repl I think that would be the more natural thing to use, rather than a stale third-party application.

I'm new to Crystal myself (just about to dive in 😃) but if there indeed exists an official repl I think that would be the more natural thing to use, rather than a stale third-party application.

Even if you need to build Crystal from source to be able to use it? I don't disagree that it has value, but I think that might be a bit overwhelming for someone that's trying to learn.

I see your point. I guess I'm just so used to building things from source myself to realize it might be a bit too involved for some people, especially if they're new to programming. :laughing:

I am personally not using a Crystal version compiled with the interpreter. Since you have to recompile every time you want to update the language.
I will leave you with a short step through tutorial if you like to install the repl:
Also I only recommend doing this if you are using Linux, if you are using mac I sorta just recommend using homebrew and windows I recommend wsl for the repl.
Step 1: Install Crystal (not from source)
Step 2: Clone the Crystal-lang  repo from the 1.9 branch or main :https://github.com/crystal-lang/crystal/tree/release/1.9
Step 3: Run make with the argument interpreter=1.
Step 4: Assign the newly compiled language version as main version
Here is a doc from 2021 explaining the repl: https://crystal-lang.org/2021/12/29/crystal-i/

Compling the language doesnt really require any crystal knowledge but you have to be used to your operating system and how to build apps from source. Since the experience isn't super user friendly


Conduct manual QC prior to upcoming release

To prepare for our next release of the Genetics Portal, can we please conduct the manual QC checks as noted in the QC check spreadsheet?

closing this one as work is done.

closing this one as work is done.


Description of features, behavior, etc

Neither the announcement in the June 2024 (version 1.91) VS Code release notes, nor the README.MD actually describe the features, benefits and behavior of this tool. Seems worth adding?

Just my name

Please, could you add some kind of documentation on how to use it in my own extension? Thanks.

yeah, README needs an update, not clear how to install and configure and use this, do we use cargo, is it built into vs code, ... looks like a solid project, please take 25 minutes to put usage instructions

This project will be consumed by the Python extension directly. You can find the code to consume pet in the Python extension source code.
For more information on JSNORPC requests/notifications for this tool, please reference /docs/JASONRPC.md.


Exchange rates for a period of time

Hey. Is it possible to add the ability to receive exchange rates for a period of time.
For example, get the data for the last month. Not very convenient to do about 30 requests for this

Would be a very nice feature. I'm currently making a side-project with charts and this feature would be very helpful.

Sounds good.
Mirroring the API of OpenExchangeRates, we'd want something like:
GET /historical/:date

... where date is in yyyy-mm-dd format, plus base and symbols as optional parameters.
I'm happy to accept a PR for this. Otherwise, I'll take a stab myself some time in the near future.

This feature would be really great and useful

Agreed, I think it would be useful for feeding JSON to a graph to display a range of data.

+1 👍

+1

+1 :+1:

+1

+1
This would also be helpful if it returned the aggregated average of the daily rates for the same period.

+1
Any progress on this?

+1

Waiting for this, I'm Python developer sorry :(

+1


Dockerless CLI - Release automation

We would like to automate our release pipeline to speed up our release process
DOD

[ ] Research how to cross-compile to CLI using pyinstaller
[ ] If we can cross-compile, add this to the pipeline and zip/pack the exec in .tar or zip
[ ] Run integration tests cross-platform in an automated fashion (on RHEL executors, macOS if they exist, etc)
[ ] Wrap the running the integration process in a bat/bash script to run on Windows/macOS/RHEL
[ ] Add sec scan to automation
More to come...

Helpful resources:

https://shanetully.com/2013/08/cross-platform-deployment-of-python-applications-with-pyinstaller/

closing because of duplication

closing because of duplication


Build Failed - no such version exists

Hi,
I've been using Malmo (0.34) for more than a month now, and suddenly this morning I get a build failed error when I run the launchClient.bat:
FAILURE: Build failed with an exception.

* Where:
Build file 'D:\Malmo\Minecraft\build.gradle' line: 42

* What went wrong:
A problem occurred evaluating root project 'Minecraft'.
> No such version exists!

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

BUILD FAILED

Total time: 6.225 secs

FAILURE: Build failed with an exception.

* Where:
Build file 'D:\Malmo\Minecraft\build.gradle' line: 42

* What went wrong:
A problem occurred evaluating root project 'Minecraft'.
> No such version exists!

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

BUILD FAILED

Total time: 4.053 secs

FAILURE: Build failed with an exception.

* Where:
Build file 'D:\Malmo\Minecraft\build.gradle' line: 42

* What went wrong:
A problem occurred evaluating root project 'Minecraft'.
> No such version exists!

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

BUILD FAILED

Total time: 4.011 secs

I tried on another computer where Malmo was running yesterday and I have the same error. Both my computers are running Windows 10 and have Java 1.8.0_201 installed.
Do you have any idea where that could come from ?
Thanks

Currently the Gradle (java package manger) for MinecraftForge (the mod of minecraft which decompiles it and lets malmo use deobfuscated game code) is down.
This is not a Malmo issue and it effects the whole Minecraft modding community. This is currently being fixed:
http://www.minecraftforge.net/forum/topic/68782-1122-setupdecompworkspace-no-such-version/
In the mean time I will look into a nethod which skips the ForgeGradle class path dependency.

Also just a note next time u get an error with malmo, do ./gradlew(.bat?) runClient --stacktrace that will help the maintainers of this project actually help u.

Pinging ms folks so they know @AndKram

In the mean time, I'll look into making a gradle script which skips rebuilding each time you run your client.

It would be great not having to build each time we run client, thanks for info!

https://github.com/MinecraftForge/MinecraftForge/issues/5477#issuecomment-464195569 For information on this issue and an updated comment with how to fix issues.

As Unnoen said: https://github.com/MinecraftForge/MinecraftForge/issues/5477#issuecomment-464195569
@AndKram this is a shitshow to patch for everyone who's already got Malmo downloaded in its released form. They may all need to modify their launchClient.bat(.sh) to call gradlew runClient --refresh-dependences or something similar.

Deleting the gradle cache for ForgeGradle is also an option, but that will require users to find their cache folder and delete something, not entirely user friendly.
But new users shouldn't have a problem, just those with the old version still cached.

Yep!
./gradlew setupDecompWorkspace --refresh-dependencies
./gradlew runClient

will also work

Hi,
It works thank you very much !


Detect change addresses from the transaction history instead of from the DB

Looks very good, except some small issues


stable/prometheus-mongodb-Exporter

Is this a new chart

NOTE: We're experiencing a high volume of PRs to this repo and reviews will be delayed. Please host your own chart repository and submit your repository to the Helm Hub instead of this repo to make them discoverable to the community. Here is how to submit new chart repositories to the Helm Hub.

What this PR does / why we need it:
This is to support TLS as currently there is no way to use a ca certification
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/ok-to-test

@steven-sheehy I've updated the readme file and made changes


locale: Add hindi language support

The commit in this pull request will enable the HIndi language support in the application.

Thank you for the translation @LordArcadius


Reduce code conflicts

Description
Reduce the code conflict with the internal code base
Files with difference:
Item count

public/components/ContentPanel/ContentPanel.tsx
public/pages/Indices/containers/Indices/Indices.tsx
public/pages/ManagedIndices/containers/ManagedIndices/ManagedIndices.tsx ContentPanel
public/pages/Policies/containers/Policies/Policies.tsx

Policy difference, Page title difference

public/utils/constants.ts
public/pages/Main/Main.tsx
public/pages/CreatePolicy/utils/constants.ts
public/pages/CreatePolicy/components/ConfigurePolicy/ConfigurePolicy.tsx
public/pages/ManagedIndices/containers/ManagedIndices/ManagedIndices.tsx Policy managed indices
public/pages/Policies/containers/Policies/Policies.tsx
public/pages/ManagedIndices/components/ManagedIndexControls/ManagedIndexControls.tsx

Indices, Managed indicex, Policy page auto refresh is removed

public/pages/Indices/components/IndexControls/IndexControls.tsx
public/pages/Indices/components/IndexControls/IndexControls.test.tsx
public/pages/Indices/containers/Indices/Indices.tsx
public/pages/ManagedIndices/components/ManagedIndexControls/ManagedIndexControls.tsx
public/pages/ManagedIndices/components/ManagedIndexControls/ManagedIndexControls.test.tsx
public/pages/Policies/components/PolicyControls/PolicyControls.tsx
public/pages/Policies/components/PolicyControls/PolicyControls.test.tsx
public/pages/ManagedIndices/containers/ManagedIndices/ManagedIndices.tsx add refresh button

page render diff

public/pages/ManagedIndices/containers/ManagedIndices/ManagedIndices.tsx

style={{ width: "100%", overflow: "hidden", textOverflow: "ellipsis" }}

Datastream security fix

public/pages/ManagedIndices/containers/ManagedIndices/ManagedIndices.tsx SECURITY_EXCEPTION_MESSAGE

Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #589 (cfbcaac) into main (7192b2b) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #589      +/-   ##
==========================================
- Coverage   59.70%   59.67%   -0.04%
==========================================
  Files         268      268
  Lines        8863     8863
  Branches     1616     1618       +2
==========================================
- Hits         5292     5289       -3
- Misses       3177     3180       +3
  Partials      394      394

Impacted Files
Coverage Δ

...icies/components/PolicyControls/PolicyControls.tsx
85.71% <0.00%> (-14.29%)
:arrow_down:

...Indices/components/IndexControls/IndexControls.tsx
90.90% <0.00%> (-9.10%)
:arrow_down:

...ents/ManagedIndexControls/ManagedIndexControls.tsx
90.90% <0.00%> (-9.10%)
:arrow_down:

...ublic/pages/Indices/containers/Indices/Indices.tsx
82.50% <0.00%> (ø)

...ic/pages/Policies/containers/Policies/Policies.tsx
84.26% <0.00%> (ø)

...dices/containers/ManagedIndices/ManagedIndices.tsx
77.27% <0.00%> (+0.17%)
:arrow_up:

...in/public/components/ContentPanel/ContentPanel.tsx
85.00% <0.00%> (+1.66%)
:arrow_up:

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


drivers/sensor: Add navigational (GPS/GNSS) channels to API

Global positioning sensors are currently not
supported by the sensor API. This addition will
add support for the basic positioning data, using
units which are intuitive and easy to work with.
The added channels are described here:
Latitude and longitude. These channels must be
in decimal degrees. A decimal degree is floating
point representation of an angle from 0 to +-180
degrees. These angles can be directly converted
to doubles when computing distances for
example.
Speed over ground in meters pr second,
Altitude in meters above sea level.
Tracking angle in in decimal degrees from 0 to
360, relative to True North (Above earths
rotational axis)
Dilution of precision for position, it can be
determined from HDOP, VDOP, GDOP, PDOP etc.
How to use the sensor API:
To check if the sensor has a fix, indicating
that its positional data is valid, the
sensor_sample_fetch() function is used. If the
data is valid, the latest positional data is
updated and 0 is returned. If data is not
valid, an error is returned.
After fetching a sample, the
sensor_channel_get() function is used as usual.
To set the update rate of the positional
sensor, the attribute
SENSOR_ATTR_SAMPLING_FREQUENCY is used. This
allows for power saving by using features like
periodic scanning if a low update rate is
sufficient.
In defence of the specified units:
The units used should be SI units, that are
represent-able by a double (signed decimal
number), which in turn can be represented by a
sensor_value struct.
The sensor driver must read, parse, validate
and convert the navigational data to the units
described above.
The application is then tasked with converting
these units to whatever format it needs.
Signed-off-by: Bjarki AA<EMAIL_ADDRESS>

fyi #30564

fyi #30564. I think a generic gnss APIs would likely need to be decided first before drivers like #46447 can be implemented

I will argue that the sensor API is that generic API, Looking at #46447 the API contains fetch, set (attr) and a bunch of getters (channels) https://github.com/zephyrproject-rtos/zephyr/pull/46447/files#diff-692be7bb6366413fb56f89a7c18f1dbba2cd06195a4fce47a54d739a4972b9ac which the sensor API is built for. The #30564 is a near copy of the sensor API, but prefixed with gnss, and with IMO to many parameters exposed to the application, that are not proven to be essential.
I believe that we should slowly add the channels and parameters that are actually useful to the sensor API over time as they are found to be useful, starting with the bare minimum channels added in this pull request.
The #46447 driver API could be rewritten to use the SENSOR API with the proposed changes, missing the TIME and SATELITES channels. I would argue that SATELITES is a less useful parameter to determine signal quality than HDOP, and TIME should use a more appropriate API to simply inform the system of the current time.

The #46447 driver API could be rewritten to use the SENSOR API with the proposed changes

You might want to have a look at #10376 as well, if you haven't already

Regarding #10376, the GNSS/GPS receiver could register itself to the network using socket offloading, using the address family AF_LOCAL (or we could add an AF_NMEA family), and simply send the raw frames as packets. I don't think this is a good idea however for the reasons that follow:
Not all GNSS/GPS sensors use the same data format, like NMEA, and sensors can have some quirks that the application should not have be aware of and forced to handle. However, all GNSS/GPS sensors provide at minimum the same basic data in the end, which is what is reflected in channels added by this pull request.
GNSS/GPS sensors are changing with time, no application should be tied to some specific protocol, like NMEA or indeed any location provider for that matter, like WIFI location, Cellular triangulation, etc. In the end, they provide the same positional data, speed, altitude, longitude and latitude along with some certainty (precision) variable for the provided location data (fx HDOP, maybe it should be changed to be in +- meters or some more general unit if possible).
The GNSS/GPS driver should be tasked with reading whatever data in whatever format it receives, and present it in a usable, unhanging and simple format (like latitude in degrees, not a device specific string format like ddmm.mmmm). The GNSS/GPS driver naturally validates the data before presenting it to the application as valid data.
For special cases where more data is required, there is the possibility of adding driver specific channels and attributes, but for the basic location data, it should just be available. If we look at the Android SDK for example, all you get is a location heading and altitude, there is no NMEA frames, no device specific parameters, it is simply handled in their drivers (and a little bit in global settings, like enable, precision etc, which could be in a Kconfig or dts). We should look to simple APIs like theirs for inspiration.

Changed DOP channel to accuracy radius in meters. It will be up to the implementer of the driver to determine the accuracy of the sensor compared to whichever accuracy/certainty metrics the sensor provides, like HDOP, and convert this to a radius in meters, within which the sensor is located within with a 68% certainty.

I see this as a quick and easy way of getting something done, tbh personally I too prefer an API dedicated for GNSS. GNSS drivers can live in its own folder (drivers/gnss) where typical NMEA gnss drivers and common libs such as NMEA parser are stored, similar to how the modem drivers work. Other things such as existing modem driver can opt to implement the gnss device driver partially to provide a simple lat/lon, over cellular triangulation for example.

The GNSS sensor/modem (L76LL) I am implementing is part of the modem library, as it uses the command handler and iface modules, GNSS sensors using AT commands for setup and NMEA frames for data should go there, but that is for another discussion.
This solution solves the issues with getting location data, at the cost of having less user configurability from the application side, this solution i find is the best, since we have Kconfig and dts to fine tune the driver configuration.
There are two other solutions to a generic location API:
The first one I find unnecessary. This is creating a new API for positional data alone, which moves the "channels" out from the sensor API, and into named functions, like location_get_latitude(), no added functionality from what would be achieved using the already existing API, but tidier.
The second is more flexible. Create a GNSS API only for GNSS sensors, not cellular triangulation or other methods of determining location. In here, tracking modes, number of satelites, the raw HDOP values could be retrieved. I still feel this is an abundance of configurations available to the application during runtime, but if it really is needed, this is where the many configurations and raw values can be found. Then, create a new subsystem, location. Here, all sensors which can provide a location implement a location API, like an offloaded socket would, then, when the application requests a location, the Location subsystem goes through all providers, uses the get_accuracy function to determine the best provider, and returns this. I will start on this solution now, so can review and compare the solutions tomorrow :)

The #30564 is a near copy of the sensor API, but prefixed with gnss, and with IMO to many parameters exposed to the application, that are not proven to be essential.

What do you mean by too many parameters exposes to the application?  I wrote the original RFC but didn't not see any comments from you on that proposal?

The #30564 is a near copy of the sensor API, but prefixed with gnss, and with IMO to many parameters exposed to the application, that are not proven to be essential.

What do you mean by too many parameters exposed to the application? I wrote the original RFC but did not see any feedback from you on that proposal before this PR?

We have not commented on the  #30564 before this pull request since we find it more efficient to show some comparative work to discuss from. In any case, later today there will be a pull request for a location subsystem/api that we will continue from, at which point this pull request will be closed.

The new proposal for location data has now been proposed here #46660, with this, this pull request is closed be prejudice :D


xdg_shell: simplify wlr_xdg_toplevel_state

the fullscreen_output and fullscreen_output_destroy fields were only
used if the state was wlr_xdg_toplevel::client_pending. Moving these
fields to wlr_xdg_toplevel reduces confusion and simplifies the code.

Guess this is a breaking change and sway does access one of these fields directly here: https://github.com/swaywm/sway/blob/4dd46f06acc520449b980a5ea52be544cc5bfb6d/sway/desktop/xdg_shell.c#L449-L452

Super stale and superceeded by #3199


Editor: Fix default preview environment sky's horizon and brightness.

Closes #99701
This addresses two issues with the default preview environment's sky. First is the horizon always appearing grey, even when both the sky and ground colors are set to black. Second is the fact that the "Sky Energy" slider only affects the top hemisphere of the sky, while the lower hemisphere's brightness cannot be modified.
The first issue occurs because the horizon color is generated automatically from the sky and ground colors, rather than being exposed to the user. This is desirably convenient of course, except that the horizon color is generated as first a 50/50 blend between the sky and ground colors, and then a 50/50 blend between the result and pure white. This second blend is why the horizon remains grey even when the user would reasonably expect it to be pitch black.
To fix this, the second blend has been replaced by a 50/50 blend with the first blend result's luminance multiplied by 3.333 (an arbitrary value, chosen to maintain roughly the same horizon brightness that the original method produced with the default sky and ground colors).
The second issue is similarly perplexing from a user's perspective; the "Sky Energy" slider modifies the preview sky's sky_energy_multiplier rather than its energy_multiplier, meaning that only the upper half of the sky is affected, while the user has no way of modifying the ground_energy_multiplier for the lower half. For the sake of keeping the preview environment's interface simple and quick to use, I've opted to make the slider alter the sky's energy_multiplier property, rather than add a second slider for the ground, as I believe the more convenient use of such a slider is to quickly modify the sky's overall brightness, as well as its associated ambient light. I also frankly wouldn't be surprised if the use of sky_energy_multiplier instead of energy_multiplier was a mistake from the start, given that its name doesn't necessarily indicate it's specifically for the top of the sky.
With sky and ground colors set to pure black:

Before
After

With "Sky Energy" slider set to 0.25:

Before
After

With default sky and ground colors:

Before
After

Looks good to me! Elegant solution to the problem.

Thanks! Congratulations on your first merged contribution! 🎉


Synchronize sessions (Server -> Client)

I've thought about this for a while.. we can manipulate the session store via a vite plugin and add an sync method on to the store which automically syncs the state from the server back to the client. This is extremely useful when using authentication and don't want to reload the page via location.reload or location.href.
The only thing i'm worried about is keeping up with svelte-kit changes and i can't come up with a solution for overwriting the types for $app/stores, we would have to cast the session store as any.  I've tried a few things with the app.d.ts file but the svelte-kit types always seem to have higher priority.
If you want to test this version it's published under the next tag<EMAIL_ADDRESS><script lang="ts">
	import { session } from '$app/stores';
	import { onMount } from 'svelte';

	async function authenticateUser() {
		await fetch('/auth/login', {
			method: 'POST',
                         body: { ... }
			headers: { Accept: 'application/json' }
		});
		await (session as any).sync(); // session store would be in sync with the server
	}
</script>

// vite.config.js
import { sveltekit } from '@sveltejs/kit/vite';
import { cookieSession } from "svelte-kit-cookie-session/vite";

/** @type {import('vite').UserConfig} */
const config = {
	plugins: [sveltekit(), cookieSession()]
};

export default config;

Worked like a charm! The cast to any is a little unfortunate but it's something I'd be willing to add by hand if there was no other option - maybe some documentation for the signature is enough for now?

The cast to any is a little unfortunate but it's something I'd be willing to add by hand if there was no other option - maybe some documentation for the signature is enough for now?
edit: Spoke too soon :( my session store does not seem to be getting the new value when i call sync. I trimmed down my example so now I have a function that logs my users in that looks roughly like the following. After navigating to /routeA, my session does not contain the new data.
export async function authorizeUser(variables: {}) {
	// trigger the mutation to authorize the user
	var sessionData = await logIn({ variables })

	// update the session with the result
	await fetch('/auth/token', {
		method: 'POST',
		headers: {
			'Content-Type': 'application/json'
		},
		body: JSON.stringify(sessionData)
	})

	await (session as any).sync()

	goto('/routeA')
}

Here's the endpoint:
export async function POST({ request, locals }) {
	// save the tokens in the local session
	locals.session.set(await request.json())

	return {}
}

Oh no! Can you see a request to /__session.json when calling sync or have some kind of error in the console?

Ahh i think you have to await locals.session.set() in your endpoint! Since switching to webcrypto those methods are async.

Oh yep! That was it 👍

I am now running into a new error when calling session.sync(): Cannot set session store before subscribing. I haven't been able to dig into it much yet but i figured I would share while I do that in case you have any ideas

You'll have to use the session store somewhere in your application via $session. This error would also appear if you'd use the unmodified version and session.set or session.update. The set and update methods are getting available after the first subscription. Don't know the exact reason behind it.


Repo Link Question

Hi,
Is the below still the best link for updating our repo's when I upgrade to V7R5. I'm seeing 7.2 references in my packages.
https://public.dhe.ibm.com/software/ibmi/products/pase/rpms/ibm.repo

Hi @ceetonice ,
Two things:

Since late 2021, we have shipped RPMs a little differently. You can read about it here: https://ibmi-oss-docs.readthedocs.io/en/latest/yum/IBM_REPOS.html.

To update, you should just have to install the new repo RPMs:
$ yum install ibmi-release ibmi-base

Then if you read the FAQ, it will tell you how to delete your existing ibm.repo file, since it is no longer needed.

You may see references to 7.2 in the RPM names. We use to build most RPMs on IBM i 7.2, so that is the earliest release that those RPMs work on. Don't worry, they still work on V7R5. There are some RPMs that may have specific 7.5 versions as well, but just know that a lower OS level in the rpm name doesn't preclude them from working on your system.

Hope that helps!

Thank you for the response.
I should have gave more information. I created a custom repo that's hosted on one of our Linux boxes. Our IBMi repos are pointed to this Linux box. Previously the repo we added to our Linux box was https://public.dhe.ibm.com/software/ibmi/products/pase/rpms/ibm.repo. Is that still the correct one to use?

No. There is now a base repo and a version-specific repo. There is https://public.dhe.ibm.com/software/ibmi/products/pase/rpms/repo-base-7.3 and then eg https://public.dhe.ibm.com/software/ibmi/products/pase/rpms/repo-7.3 https://public.dhe.ibm.com/software/ibmi/products/pase/rpms/repo-7.4 etc

Thanks for all the help. I'm going to make that change.

Just for clarity.
I should sync from both, correct? I see that repo-7.5 does not have all the packages
https://public.dhe.ibm.com/software/ibmi/products/pase/rpms/repo-base-7.3
https://public.dhe.ibm.com/software/ibmi/products/pase/rpms/repo-7.5

Correct. 7.5 will only have the packages that we have built with specific 7.5 enhancements. All other packages are still built for the "base" release, which at this time is 7.3. So your system will have a mix of 7.5 and 7.3 packages on it (and some 7.2 that were built when that was the base release)

Thanks!


Sync wallaby into xena

[ ] Build images
[ ] Promote images

Image builds:

centos: https://github.com/stackhpc/stackhpc-kayobe-config/actions/runs/4469268391
ubuntu: https://github.com/stackhpc/stackhpc-kayobe-config/actions/runs/4469274983

Promotion jobs:

centos: https://github.com/stackhpc/stackhpc-release-train/actions/runs/4477428474
ubuntu: https://github.com/stackhpc/stackhpc-release-train/actions/runs/4477433908

Merged in CI fix for centos stream.


Error when trying to use method on typed function argument

Godot version:

4.0.dev.custom_build. 93b50a62e
Issue description:

Code
func A_Object(q_Object: Object) -> void:
	q_Object.can_translate_messages()

shows error(wrong, because when clicking at the function name it redirect properly to the documentation)
res://new_script.gd:5 - Parse Error: Cannot find property "can_translate_messages" on base "Object".

Removing type, remove error and this code works as expected
func A_Object(q_Object) -> void:
	q_Object.can_translate_messages()

CC @vnen since project works before merging GDScript 2.0

Looks like Godot try to execute functions(maybe try to check something)  when have typed arguments.
Code
func alt_process(delta) -> void:
	nodeFunction(self, true)

func nodeFunction(q_CenterContainer: CenterContainer, can_reset: bool = false) -> void:
	pass

produce stack trace with only visible nodeFunction function without any info about caller function alt_process


Faculty Name Bug  🐛

Description

Faculty name toolTip must not be shown in the RECESS column of Academic Time Table.
This must be removed.

Screenshots

Additional information

All other files must be kept intact

Make sure to read the CONTRIBUTING and SETUP docs before proceeding
Happy contributing. 💐
Star my other Repositories here

@warmachine028 Can you assign me this issue? I am new to contributing to open source projects, so wanted to start with easier once.

Go ahead with it!

There's not file in server folder. How do I get it?

There's no file in server folder. How do I get it?

Yes the server isn't created yet. currently we are working with hard coded data only.
Did you git clone the repository?

Yes I forked and cloned
Also, I am facing issues with the progress packages:

Yes I forked and cloned Also, I am facing issues with the progress packages:

did you try npm install to fix it ?

Yes I did npm install in cleint folder

can you show me your package.json file

Can you try uninstalling the package @progress/kendo-react-pdf and then reinstalling it.
If that doesn't work you can comment out the admit card module and work on the TimeTable only

@warmachine028 Created a PR. Please take a look.

Very Good! Welcome to the Contributor's Club. Make sure to star this project 🥂

I guess you'll be merging the PR. I don't have write access.

Yes! I will merge it


System logger isn't unicode-safe on Linux

Yep. Needs to be investigated.

We'll be rewriting this system - See #77


Permanent description on a photo

Hi there!
I'm new to Sigal so it could be a simple question:
Is there a way to show photo description hovering the thoto with transparent background? Is it a matter of theming?

Something like this

Hi,
currently the description is already available in the info box, which is hidden by default. This is the galleria default behavior, but I agree that it would be useful to have the possibility to always display the description (maybe with a setting?).
It's just a theming thing indeed, the description is available in the media.description attribute.

Perhaps something can be done, or the theme can be arranged so that it can be done, with a user css as in #313 .

I think it would need more than a CSS change.


Press the back button while sharing an image, the app crashes.

Submit feature requests to http://www.mattermost.org/feature-requests/. File non-security related bugs here in the following format:
Summary
Press the back button while sharing an image, the app crashes.
Environment Information

Device Name:
OS Version: android 9
Mattermost App Version: 1.40.0
Mattermost Server Version: 5.32.1

Steps to reproduce

Upload 5 images.
Tap the image.
Click the Share button.
While the image is being prepared, press the back button.
When the pop-up disappears, tap another image.
The app freezes.

Expected behavior
When the back event occurs in Android, the image being prepared should be stopped.
Another image should open normally.
Observed behavior
The app freezes.
The above error does not occur for files that have already been prepared(downloaded).
Since the progress bar does not pop up for files that have already been prepared (downloaded),
other images will open normally even if you click Back.
Possible fixes
It is shown related to the prepare_files.tsx file of the gallery component.

@enahum Were you able to repro this?

@amyblais Yes

Ticket: https://mattermost.atlassian.net/browse/MM-33778


Switch from _exit() to exit(), rework fflush().

CT runs each test case and benchmark in its own process; those processes
end with a call to _exit(). However, gcov writes coverage data in an
atexit() function which doesn't get called from _exit(). So we switch to
exit() to enable test coverage analysis. Since exit() flushes files, we
had to rework our fflush() strategy as well.

I am happy to change the PR if there's a bug I didn't see yet. :-)

Nothing wrong that I know of, I just wasn't able to get to it, sorry.
Thanks!


utf8 issues

Not every stream metadata is in utf8. needs to fixed.

Any joy with this? Causing me major issues trying to sanitize against this.

@djpimp2010 not yet. I have just accept it for now. I will try to see if i can do something.


Hybrid Mode

Is your feature request related to a problem? Please describe.
Historical data loads fast for first load, but lacks accuracy.
Describe the solution you'd like

For desktop users, hybrid mode would be set as default (mobile users would default to forced clientSideProcessing=false until a much later release)
Hybrid mode first loads relays via history relay
Once complete, it switches clientSideProcessing to on.
Further checks will use clientSideProcessing

Describe alternatives you've considered
convert nostr.watch to a text file
Additional context
People like to hear their CPU fans spin up apparently, who knew.

This is now implemented by default on a user's first visit. This should still be made a special mode, where only latency is checked client-side, but it presently not a high priority.


Redundant variables (coding style)

I've noticed a lot of cases of x = 'x' in the code (e.g. backdrop = 'backdrop'), and couldn't figure out what the use of such a coding style is. I also found that it made the code harder to understand and debug, but could be a matter of taste.
Thanks.

That's a better way to compress strings.

Ah interesting. Does it have an impact when gzipping?
On Sun, Feb 3, 2019, 2:44 PM thednp<EMAIL_ADDRESS>wrote:

That's a better way to compress strings.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/thednp/bootstrap.native/issues/269#issuecomment-460044412,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABZ6Tg5yHF6fzGgrcDy3Eb3XtZKQRrHpks5vJsuOgaJpZM4afx9n
.

If your file is properly built it should be ok.

Sorry, I meant does it still affect file size when gzipping?
On Sun, Feb 3, 2019, 3:13 PM thednp<EMAIL_ADDRESS>wrote:

If your file is properly built it should be ok.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/thednp/bootstrap.native/issues/269#issuecomment-460046285,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABZ6TmS9xVpE5zCWnwzjulwzai4aPy9tks5vJtJUgaJpZM4afx9n
.

Sorry, I was curious whether the compression improvement you mentioned also applies if the file is gzipped.
I'm actually curious in any case in which such a style improves string compression. Especially as I've been trying to reproduce the improvement but couldn't.

You can do this test yourself I don't have the time to do this. From 70Kb unminified to 7.65Kb minified and gzipped, I think that's a huge deal, right?

I no longer use BSN in my projects, but came across something relevant to this discussion, and thought I'd share in case it's found beneficial:
https://github.com/mishoo/UglifyJS2/issues/185#issuecomment-16974360

For now, it seems that improvements are unlikely due to gzip's more efficient string tokenization (gzip's pointers are smaller byte-wise that we could write in javascript), and any improvement would be marginal.

I don't understand what would be the reason. but thanks for bringing that up. Again the main idea is to have something we have control over before gzip or other scripts do their stuff.

AFAIK it's because gzip (i.e. deflate) compresses recurring strings very efficiently, so it's pretty difficult to do a better job than it.

So did you test yourself?

Here's a small sample:
Minified with tokenizing:
(function(){var n=document,e=n.documentElement,i="style",t="toLowerCase",o="Transition",a="Duration",d="Webkit",r=d+o in e[i]||o[t]()in e[i],u=d+o in e[i]?d[t]()+o+"End":o[t]()+"end",c=d+a in e[i]?d[t]()+o+a:o[t]()+a})();

221 bytes, 175 bytes gzipped.

Minified without tokenizing:
(function(){var n=document,t=n.documentElement,i="WebkitTransition"in t.style||"Transition".toLowerCase()in t.style,e="WebkitTransition"in t.style?"Webkit".toLowerCase()+"TransitionEnd":"Transition".toLowerCase()+"end",o="WebkitDuration"in t.style?"Webkit".toLowerCase()+"TransitionDuration":"Transition".toLowerCase()+"Duration"})();

334 bytes, 163 bytes gzipped.

So yes, this very basic sample supports that even though the latter has more characters, it ends up being smaller gzipped. Thus, manual tokenizing isn't worth it — the compressor does a better job than us puny humans. =]

That is interesting and strong argument indeed and I thank you for your time.
Around 7% deficit on manual tokens vs gzip, my library minified is very small compared to original plugins and jQuery all together. It's hard to want to use something similar with larger size.

@mustafa0x I've been thinking about this lately, here is my thinking. I won't have a problem reverting to the previous coding style, especially because I seriously consider rewriting the code in ES6 format and this manual tokenization is making it all likely more tedious.
Somehow I still wish 7zip/Gzip devs would take into account existing code tokenization before doing it's own. This would serve greatly to those massive chunks of folks who would like to have their script delivered in super minimized size but would rather not bother with server/cms/other settings to enable and configure compression.

gzip is supported everywhere and is almost always on by default, so I don't think optimizing for a non-gzip environment is that productive.


Usage against existing plugin

Hi there,
first of all thanks for this project, it seems like you are filling a gap there cause I was not be able to find any docker image for building logstash plugins.
I am trying to build locally the logstash-output-mongo plugin and I found the following:
$ docker run --rm -it -v ${PWD}:/work cameronkerrnz/logstash-plugin-dev:7.9 bundle
/opt/jruby/bin/bundle: line 9: require: command not found
/opt/jruby/bin/bundle: line 11: version: command not found
/opt/jruby/bin/bundle: bundle: line 15: syntax error near unexpected token `('
/opt/jruby/bin/bundle: bundle: line 15: `  str = str.dup.force_encoding("BINARY") if str.respond_to? :force_encoding'

Am I doing the right thing?
Thanks again!

If I execute the command from within the shell (no bundle install appended to the docker run command), it seems to work.
My requirement though was also to be able to build and include the plugin as part of another docker image so I am trying that now.

FYI it worked great with the following,
FROM cameronkerrnz/logstash-plugin-dev:7.9 as logstash-builder

###############################
# Clone logstash-output-mongo #
###############################

USER root

ARG pluginworkdir=/work/logstash-output-mongodb

RUN git clone https://github.com/iteratec/logstash-output-mongodb \
        --branch feature/implementDifferentActions --single-branch $pluginworkdir
RUN chown builder $pluginworkdir

WORKDIR $pluginworkdir

USER builder
RUN bundle install \
    && bundle exec rspec \
    && gem build logstash-output-mongodb.gemspec

Closing this issue cause it seems the docker run is not a real blocker.
Thank you!

Cool; though I think you'll want to use chown -R builder $pluginworkdir
Admittedly I haven't really considered the use-case of running this container as a tool, but only as a dev-container, as described in https://github.com/cameronkerrnz/logstash-plugin-dev
Currently /bin/bash is the default entrypoint, which means if you want to run a tool via docker run, remember to pass it as a -c (to bash). Eg:
docker run --rm -it -v ${PWD}:/work cameronkerrnz/logstash-plugin-dev:7.9
   -c "logstash-plugin generate --type=filter --name=coolstuff --path=/work/"

Thanks for your interest.


Read only workspace: using commands from extensions that edit/format a file should trow an error

Issue Type: Bug
Testing #52573
There should be an error shown when an extension tries to edit/modify a read-only file. Right now nothing happens. See "Prettify JSON" example below.

VS Code version: Code - Insiders 1.25.0-insider (923e9ccb3fc44bfaaf4b4049015c84e42bc40f95, 2018-06-26T13:00:22.932Z)
OS version: Darwin x64 17.6.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4770HQ CPU @ 2.20GHz (8 x 2200)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: enabledrasterization: enabledvideo_decode: enabledvideo_encode: enabledvpx_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
2, 2, 2

Memory (System)
16.00GB (0.38GB free)

Process Argv
/Applications/Visual Studio Code - Insiders.app/Contents/MacOS/Electron -psn_0_4183037

Screen Reader
no

VM
0%

Extensions (4)

Extension
Author (truncated)
Version

svgo
100
0.8.0

vscode-svgo
ajh
0.1.0

gitlens
eam
8.4.1

python
ms-
2018.6.0

(4 theme extensions excluded)

Not just extensions, currently it seems only "type" commands show errors. Delete should show errors too, and builtin commands like transpose should as well.

This is a known limitation of the readonly editor that extensions can still go crazy and execute their commands.
However if they use the file system provider which is readonly it will through an error - though they can alwasy go directly to node and there is nothing we can do.
So for now pushing this to the backlog as I have no immediate plans to fix


Reduce script command when downloading latest release

To get latest release we currently retrieve all ockam releases and then grep to get latest release.
https://github.com/build-trust/ockam/blob/4725c3efe5066dc1203b712982b34a3b5eb90320/install.sh#L150-L153
We can reduce this to a one-liner link that downloads latest release of an asset
curl --proto '=https' --tlsv1.2 --location --silent --fail --show-error --output "ockam" "https://github.com/build-trust/ockam/releases/latest/download/ockam.aarch64-apple-darwin"

The above link downloads the latest binary of ockam.aarch64-apple-darwin without specifying the tag.

Want to take this up @danieled-it? 😃


Update to JupyterLab 0.15

Tested against JupyterLab 0.15.

@gnestor, ha, I made this PR and then you made #15 instead.

I saw that you did it for jupyterlab_geojson! I created a script that will bump jupyterlab versions for all mimerender extensions that I'm maintaining, so all you need to do is ping me when a new jlab version is released and I'll update the extensions.

Nice!  Will do.


Ability to specify an index routes file

This PR allows you to set:
router: {
  index: '~/my-routes'
}

Or, if there's a pages/index.js file, automatically loads them from there.
Documentation will need to properly indicate implications (and the need to define the chunkName if need be). This relates to the Use a custom routes index file section of my VueJSDevelopers article.
I believe the current method of accomplishing this is a bit too hackish, so this should make the switch more elegant for those who desire it.

Codecov Report

Merging #3646 into dev will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##              dev   #3646      +/-   ##
=========================================
+ Coverage   99.19%   99.2%   +<.01%
=========================================
  Files          18      18
  Lines        1121    1134      +13
  Branches      302     306       +4
=========================================
+ Hits         1112    1125      +13
  Misses          9       9

Impacted Files
Coverage Δ

lib/builder/builder.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

I agree with Clark, convention over configuration of what makes Nuxt so approchable. And too many configuration is not good as well (I already think we have too much option).
People can use the router-module to customize our routing system.
I appreciate the effort @galvez like always :-)


Day_8

Resuming my progress from 2nd Feb, for better tracking I am still counting the day from where I left.

@Darshnadas Great to see you continuing again. Maintain the streak.


BTree Index Compression

What changes were proposed in this pull request?
Compress / Decompress BTree Index file
How was this patch tested?
current UT

Actual changes included in commit: https://github.com/Intel-bigdata/OAP/pull/709/commits/7c23a04a24b346b16354250e223fa4ff0abb41f2

@adrian-wang
updated PR, please review this again.

Will rebase this on #711

@adrian-wang ready to review.
Thanks.


Automate creation of ACM cert and R53 domain

Going to add optional creation of a pretty name for CF.

On second thought I'm not going to make it optional. It doesn't make much sense to leave that out because without a domain pointing at this stuff it's kind of pointless..


is 'ol.events.Event.prototype.preventDefault =' a mistake ?

I start to read source code of openlayer(v4.0.0).And I find line 48 in "/src/ol/events/event.js".
/**
 * Stop event propagation.
 * @function
 * @api stable
 */
ol.events.Event.prototype.preventDefault =

I thought it might be a mistake only in this new version,so i try to find the same line in older versions.But it start to confuse me that they are the same.I can't understand.Can anyone help?

ol.events.Event.prototype.preventDefault is equal to ol.events.Event.prototype.stopPropagation defined above, see https://github.com/openlayers/openlayers/blob/v4.0.0/src/ol/events/event.js#L56-L58
Without the jsdoc blocks, it should read:
ol.events.Event.prototype.preventDefault = ol.events.Event.prototype.stopPropagation = function() {
  this.propagationStopped = true;
};

Wow,I never write jsdoc blocks between statements.It would take me long time to figure it out without you help.thank you. @fredj


Bump-up utils package

Description
Bump-up utils package.
Fixes #122
Type of change

Bump-up dependent library (no changes in the code)

Testing steps
N/A
Checklist

[ ] make before_commit passes
[ ] updated documentation wherever necessary
[ ] added or modified tests if necessary
[ ] updated schemas and validators in insights-data-schemas in case of input/output change

Codecov Report

Merging #137 (6ca1bc6) into master (c5b7388) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #137   +/-   ##
=======================================
  Coverage   18.45%   18.45%
=======================================
  Files           3        3
  Lines         439      439
=======================================
  Hits           81       81
  Misses        347      347
  Partials       11       11

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 c5b7388...6ca1bc6. Read the comment docs.


Regular reports

Users want to receive daily / weekly / monthly reports of their recent coding activity. This might be inspired by WakaTime's reports, but doesn't have to.
What would you like to receive reports and notifications for? Let me know!
Requires #123.
Originally requested by @agrrh in https://github.com/muety/wakapi/issues/82#issuecomment-774778855

In the long run, we might even integrate shoutrrr to support different notification services, like Telegram, Slack, Mattermost, Discord, ...


Remove FOM record as required by Julius

Describe the task
Julius will send us a list of FOM that they want to remove. On our side, we have to:

Remove trash icon and disable the delete functionality for FOM with finalized or expired status
Make sure ministry user can delete the FOM with commenting closed status
Make sure submitter can delete the FOM with initial status
Make sure nobody can delete the FOM with published or commenting open status
Add a DELETED status to the project status, and FOM with deleted status will be hidden and ignored at all places
Change the status of the FOM Julius wants to remove to "DELETED", and verify they are not shown to user anymore

Acceptance Criteria

[ ] Add a new status to the file api/src/app/modules/project/workflow-state-code.entity.ts
[ ] Regenerate client code for frontend to include the new status
[ ] Check the method project.service.ts - isDeleteAuthorized() and fom-detail.component.ts canWithdraw() and other methods need to adjust
[ ] Ministry user - commenting closed - can logically delete
[ ] Submitter - status initial - can logically delete
[ ] Status of finalized or expired - logical delete not allowed
[ ] Status of published or commenting open - logical delete not allowed
[ ] Change the status of the FOM Julius wants to remove to "DELETED", and verify they are not shown to user anymore

Additional context

Add any other context about the task here.
Or here

FYI, attached the email


RemoteAutocomplete component

Added RemoteAutocomplete component. Fixed frontmatter validation. Fixed RestoredDataModal text. Updated swagger model. Fixed console error messages

@Exmirai please refactor categories to category.

@Exmirai please fix merge conflicts.

There's no global, community accepted naming convention afaik


Can I use Fixtures parameters in yaml keys to set number of loops?

Hi :wave:
I'd like to use Fixture parameters as described in the docs :books:  to generate a bunch of users and other things, but I'd like to pass a quantity. For my local tests, I'm generating 100, but in the end, the desired quantity would be 15'000 and I don't want to set it everywhere each time (I have a lot of models to generate).
I created a app/Resources/fixtures/orm/parameters.yml that looks something like this:
parameters:
    fixture.load_users.users_to_create: 1..100

And I'd like to use it this way:
Project\Bundle\SomeBundle\Entity\Users:
    some_user_{<{fixture.load_users.users_to_create}>} (extends user_template):
        username: '<username()>'

But I get the following error:
Template user_template is not defined
- Entity with Id @some_user_<{fixture.load_users.users_to_create}> and Class Project\Bundle
  \SomeBundle\Entity\Users not found

Note that the user_template is defined and it just works when I replace
some_user_{<{fixture.load_users.users_to_create}>} (extends user_template):

with
some_user_{1..100} (extends user_template):

Should I use some php instead? Anyone done something like this before?

Nope. It's doable by manually looping to create the desired number of fixtures, but right now at least this is not possible. So you can either go back to PHP templates to achieve what you want, or extend the library to resolve those parameters in the name which is totally doable but does require you to dig a bit in it.


missing referenced spdx_id in merge

main.spdx.txt
other.spdx.txt
sbomasm assemble -n merge -v 1 -t application -o merge.spdx.json *.spdx

merge.spdx.json
pyspdxtools -i merge.spdx.json
ERROR:root:The document is invalid. The following issues have been found:
did not find the referenced spdx_id "SPDXRef-Package-other" in the SPDX document
did not find the referenced spdx_id "SPDXRef-Package-other" in the SPDX document

Hey @vargenau . Thanks for raising an issue.  Wanted to know the o/p of the below command:
pyspdxtools -i main.spdx.txt and pyspdxtools -i other.spdx.txt

Both files are valid SPDX.
pyspdxtools -i main.spdx
pyspdxtools -i other.spdx

give no result, meaning they are valid.
The Java Tools also say files are valid SPDX.
I had to rename the files by adding .txt in order to be able to upload them on GitHub. You should rename them back to do the tests.
I always provide valid SPDX as input in my issues.

fixed here https://github.com/interlynk-io/sbomasm/pull/84

v0.1.4 has been released with this fix.


[GLib] Add GArrowTimestampParser

Describe the enhancement requested
It's a binding of arrow::TimestampParser.
Component(s)
GLib

Issue resolved by pull request 40457
https://github.com/apache/arrow/pull/40457


Use this plugin in URP Project

When you insert URP asset into Graphics and Quality, nothing is displayed in the image. If you insert only one of the two, nothing will be displayed.

Unfortunately, the sample project only runs on the builtin render pipeline.
On the other hand, the SelfieBarracuda package (jp.keijiro.mediapipe.selfie) is compatible with URP.
Please let me know if you find any issue with the package running on URP.


Rename width and height to w and h

width and height were generating clashes with the HTML attribute, especially in OcticonComponent.
I'm renaming them to w and h when used to generate Primer Utilities.

We could remove this change if we're renaming? https://github.com/primer/view_components/blob/6437e0f927b22790a82e5c64d15f0fa1f4b3ae59/lib/primer/classify.rb#L173


Support passing inputs to built-in ci:github-actions-dispatch action

Feature Suggestion
ci:github-actions-dispatch action currently only accepts repoUrl, workflowId and branchOrTagName. Based on the API docs, it is possible to also pass inputs to the workflow

Input keys and values configured in the workflow file. The maximum number of properties is 10. Any default properties configured in the workflow file will be used when inputs are omitted.

Possible Implementation

Adding option key like workflowInputs  to https://github.com/backstage/backstage/blob/master/plugins/scaffolder-backend/src/scaffolder/actions/builtin/github/githubActionsDispatch.ts#L35
Context
I want to be able to pass inputs to the workflow using github:actions:dispatch action
  steps:
    - id: dispatch-action
      name: Dispatch action
      action: github:actions:dispatch
      input:
        repoUrl: github.com?repo=my-repo&owner=myorg
        branchOrTagName: main
        workflowId: .github/workflows/main.yml
        workflowInputs:
           x: foo
           y: bar

duplicate of #8770


Consider adding Postgres environment variables for scan backend configuration

Rather than specifying the environment variables in the ort.conf file, also allow the use of PG_ environment variables, especially for secrets.

Actually one can specify any type of environment variables, as described in https://github.com/oss-review-toolkit/ort/#ort-configuration-file
So ideally all environment variables should be forwarded to the Docker container so they can be referenced.

Not needed, as all environment variables are forwarded to the Docker container.


[Feature] Replace RAY_IP with FQ_RAY_IP after release 0.5.0

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
We plan to deprecate RAY_IP https://github.com/ray-project/kuberay/pull/938 and use FQ_RAY_IP instead. However, the configuration YAML files need to be compatible with both nightly and the latest release of KubeRay operator. Hence, we need to replace RAY_IP with FQ_RAY_IP after release 0.5.0 (i.e. revert this commit https://github.com/ray-project/kuberay/pull/938/commits/66b4581aa7ec0e2e68c662bf3d0c75ca8234545c).
Use case
No response
Related issues
#938
#940
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Closed by #1010.


composer test don't works for windows

Output:
>composer test
You are running composer with xdebug enabled. This has a major impact on runtime performance. See https://getcomposer.org/xdebug
> phpunit
Could not open input file: <my_project_folder>\vendor\bin\/../phpunit/phpunit/phpunit

Issue fixed in composer, https://github.com/composer/composer/issues/4990


[ADD] backport module pos_keyboard from v8 to v7

This pull request contains a backport of the module pos_keyboard to v7.
Besides the formal changes to work with v7, it also contains a small improvement to PosWidget.start where now local variables are being used.

Thank you.
@ufaks will review it shortly


Incomplete memory returned by core.get_memory(RETRO_MEMORY_SYSTEM_RAM)

This is a super impressive project and I've been excited to try to use it as light-weight binding to access previously installed libretro cores to train ML agents via Reinforcement Learning. (Other options for this purpose require heavy dependencies and complicated build scripts which don't work consistently across different platforms.)
I'm currently blocked in this effort due to libretro.py's core.get_memory() method which is for some reason surfacing an incomplete version of the FBNeo core's memory. What I get back from from that core's RETRO_MEMORY_SYSTEM_RAM is almost entirely zeroes with patches of real data in there but not including what I need to fetch.
This is true even if I point libretro.py to the dylib for a FBNeo core known to work for this purpose. It works to use the aforementioned heavy frameworks to access memory I need with the very same dylib, so I'm wondering if it's some issue with libretro.py.
My code below receives a value of 0 when it should be something else. Printing out the full memory for debugging reveals that it's almost all "...00 00 00..." which can't be right.
    def _get_lives(self):
        """ Retrieve the current number of lives from the emulator's memory.  """
        memory = self.session.core.get_memory(RETRO_MEMORY_SYSTEM_RAM)
        if memory is None:
            return 0
        lives = memory[self.P1_LIVES_ADDR]
        return lives

full code here
Kind of a shot in the dark, but I bring it up here in case something jumps out at you as worth checking. Would really love to make use of this if you have any ideas.

Glad you're able to get some practical use out of this (ticket notwithstanding)!
The odd part is that the memoryview returned by Core.get_memory doesn't make a copy; changes that the core itself makes should be reflected in the memoryview.
Let's try to narrow it down:

Is the size what you expect it to be?
Can you confirm that the memory updates even when you hold on to the memoryview? If the core consistently returns the same address during a session then that should be okay.
Is P1_LIVES_ADDR relative to the start of RAM? Or is it mapped differently on real hardware?
Do you encounter a similar issue when using another core (that supports retro_get_memory)?

All good leads! Will investigate.

Oh, and since this is for FBNeo...does the issue also occur if playing a game with completely different hardware?

I got it. Was a combo of errors on my part.. (plus just a lot more 00s than I expected to see in memory, which is apparently legit.) No bug on your end. Thanks for your help!


kubernetes deployment cronjob logs are not collected

output from cronjobs is not delivered.  cron jobs are creating output, and the sumo daemonset pod is detecting these files e.g.
2018-11-13 21:22:54 +0000 [info]: #0 following tail of /mnt/log/containers/ssl-monitor-svc-1537959420-v9bq2_infra-test_ssl-monitor-script-42e671eb0de33a6016afef8e64fee234219b055af7fd0a716e6ccaa7f336574b.log
however the content of these files is not sent to sumologic.
this was deployed with a chart with the following settings
sumologic:
  collectorUrl: ...==
  logFormat: text
  sourceName: '%{namespace}/%{pod}/%{container}'
  sourceCategoryPrefix: ' '
  sourceHost: '%{pod}'
  sourceCategoryReplaceDash: '-'
updateStrategy: RollingUpdate
tolerations:
  - operator: "Exists"

issue was my problem.  using helm --upgrade wo a purge or a --recreate-pods


Adding twitter option in social.

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
twitter is not mentioned in socials.
Expected behavior
Will add this option in it
Add ScreenShots

twitter not mentioned
What browsers are you seeing the problem on?
No response
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

@Anishkagupta04,
I want to fix this issue Please assign me fot this.
I am a GSSOC'24 Contributor

I would like to contribute for this project. Please assign this task to me under GSSoC'24 with required labels.