Some examples are still stating they work without auth

Some examples (eg search or fetch related artists) used to work without authentication, but this is no longer the case. They should be modified to obtain a token and set it before making the request.

I've just fixed search.


Scope and time scheduling of the meeting (issue #353 related)

Goal of this issue is to schedule a meeting:
Time: 19:45
Date: 14.12.2023

Issue finished.


Changing the name of a group should not recalculate group2groupcache table

References

Fixes #8155

Description
Remove the line that set to true the property groupsChanged when the name of the group is changed.
Instructions for Reviewers
It is a small change.
I have tested it in a DSpace 6x instance with many groups relations (1000 entries in group2group table).

Go to group administration
Create a new group and change the name
The response should be quicker than adding a new subgroup.

Checklist

[x] My PR is small in size (e.g. less than 1,000 lines of code, not including comments & integration tests). Exceptions may be made if previously agreed upon.
[x] My PR passes Checkstyle validation based on the Code Style Guide.
[x] My PR includes Javadoc for all new (or modified) public methods and classes. It also includes Javadoc for large or complex private methods.
[x] My PR passes all tests and includes new/updated Unit or Integration Tests based on the Code Testing Guide.
[x] If my PR includes new, third-party dependencies (in any pom.xml), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation.
[x] If my PR modifies the REST API, I've linked to the REST Contract page (or open PR) related to this change.

@tdonohue I have added the suggested change. I have also done a quick test enabling the TestConsumer to confirm that it triggers a MODIFY_METADATA event.


Update bmi

The Dakota BMI now requires the basic_modeling_interface package. I also added the BMI time methods.

Coverage increased (+0.03%) to 94.721% when pulling ed076e9e3e59de7ec7a7c231c9e53f97141562c1 on mdpiper/update-bmi into f162387a2bb7d1adaea03174b899a58823da7b2f on master.


⚠ TorrentLeech has degraded performance

In fe4b059, TorrentLeech ($TL) experienced degraded performance:

HTTP code: 200
Response time: 1783 ms

Resolved: TorrentLeech performance has improved in 8a992e1.


Add missing filterable default value

filterable was missing in some widgets.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage remained the same at 70.415%

Files with Coverage Reduction
New Missed Lines
%

packages/react-ui/src/theme/carto-theme.js
4
61.54%

Totals

Change from base Build<PHONE_NUMBER>:
0%

Covered Lines:
1155

Relevant Lines:
1544

💛  - Coveralls

👍🏻


[Icon Request] Support for 'stryker'

[x] I'm sure this issue is not a duplicate.

[x] I'm submitting an icon request:

Type: filename
Icon Name: stryker
Sample original Icon:
Filenames: stryker.conf.json, stryker.conf.js, stryker.conf.*.json, stryker.conf.*.js, stryker-config.json, stryker4s.conf, stryker-config.yaml, stryker-config.yml

More info: https://stryker-mutator.io

Closed by #2896


Update rewards sql query to properly compute jit CoW AMM orders surplus

Completed with the merging of PR #7


Heap profiler consumes a lot of memory

Originally reported on Google Code with ID 671
Hello,

I am testing the heap profiler to track memory leak in my application.

It works well but when I activate the heap profiler on my runtime, the memory usage
of the app grows strongly (1 MB/sec) until my machine ends in OOM (over 15 GB RAM used).

I use the profiler like this (link with tcmalloc at runtime):

LD_PRELOAD=libtcmalloc.so.4.2.4 HEAPPROFILE=<path_to_output_files> <MyBinary>

Thank you!
Chris

Reported by lebars.christophe on 2015-01-30 09:53:15

pmap -X mypid print:

fanally, MallocExtension::instance()->ReleaseFreeMemory(), and malloc_stats() print:

also, I am testing the heap profiler to track memory leak in my application.
and linked libtcmalloc.so , then i run it with the command below:
HEAPCHECK=normal \
HEAP_CHECK_RUN_UNDER_GDB=1 \
HEAP_CHECK_IDENTIFY_LEAKS=1 \
HEAP_CHECK_MAX_POINTER_OFFSET=-1  \
PERFTOOLS_UNITTEST=0 \
TCMALLOC_RELEASE_RATE=10 \
HEAP_PROFILE_INUSE_INTERVAL=52428800 \
TCMALLOC_LARGE_ALLOC_REPORT_THRESHOLD=32768 \
HEAPPROFILE=heapprof/heapprof \
HEAP_CHECK_IGNORE_GLOBAL_LIVE=0 \
HEAP_CHECK_IGNORE_THREAD_LIVE=0 \
HEAP_CHECK_MAX_LEAKS=0 \
TCMALLOC_TRACE=1 \
./diagent

and use a singnal SIGUSER2 to release and display tcmalloc memory usage,  (my application is runing  backgroud):
void sigusr2(int signum) {
    MallocExtension::instance()->ReleaseFreeMemory();
    malloc_stats();
}
main() {
     signal(SIGUSR2, sigusr2);
}

is the 700mb  high address memory （which address is 0x7fdfc3160000） consume by  Heap profiler?


Unity 5.3 support

Hi I tried using this plugin in Unity 5.3 and there were compile errors (which I fixed) and a bunch of null refs (StickerPackEditor.cs, line 446, 363).
Is there any point in me trying to get this working in Unity 5.3?  Or is there some fundamental, unfixable problem that I will eventually encounter?
Thanks.
Colin

If you just fix the compile errors it should work fine. I'll make sure it
works for 5.3 as well now that I know someone needs it.

Is there any point in me trying to get this working in Unity 5.3? Or is
there some fundamental, unfixable problem that I will eventually encounter?
Thanks.
Colin
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/agens-no/iMessageStickerUnity/issues/12, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAitDeVfms7sGoPT_YwbN2BoPs4sBYw2ks5sIn_1gaJpZM4OIN4s
.

That's great news, thank you.  Will you reply back on this thread when you have made it 5.3-compatible?

Sure. Currently on holiday so might be a few weeks though
On Thu, 29 Jun 2017 at 10:05, colinamuzo<EMAIL_ADDRESS>wrote:

That's great news, thank you. Will you reply back on this thread when you
have made it 5.3-compatible?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/agens-no/iMessageStickerUnity/issues/12#issuecomment-311893274,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAitDW5ocFbJj_2xM76I-W6VMUI70oTzks5sI1qtgaJpZM4OIN4s
.

Ok, no worries.

Fixed the compile errors on 5.3 now. Please give it a try :)
On Thu, Jun 29, 2017 at 10:15 AM, colinamuzo<EMAIL_ADDRESS>wrote:

Ok, no worries.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/agens-no/iMessageStickerUnity/issues/12#issuecomment-311895500,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAitDSGNGpatgZUhxFwCueWq9XrkWlIwks5sI10PgaJpZM4OIN4s
.


license & contributing

There are two distinct sections for Contributing & License in the README, one pointing to "Open-2 Contributing" (an out of date contributing template that I made some time ago) and an ISC License, the other to Esri/contributing and Apache 2.0. It would be good for users to clarify licensing and contributing and make sure that information is accurately reflected in the README and package.json.

Changed this with John!


Upgrade gradle-intellij-plugin to latest version

It just worked. They must have improved the multi-module support.
I had to also update the publish section since the old method is now deprecated.
TODOs

Test the plugin publish to make sure it still works
pluginId from the 'publish' section is now deprecated. Instead, the plugin id is pulled from the ID of plugin.xml. Does this mean we need to update the ID in the plugin.xml for the publish to work (and what are the effects of doing that)?

Tested out publish to alpha and it works properly.

@patflynn I'll go ahead and merge since things look good to me but if you can recall the other reasons we delayed this then we can revisit those or roll this back if needed


Set BP_JVM_VERSION=17 for maven sample

Fixes #493
Summary

Adding BP_JVM_VERSION=17 since the example needs java 17 and the default is (still) 11
Adding --builder paketobuildpacks/builder-jammy-base because the sample might or might not work depending on what pack commands you tried out before (e.g. using buildpackless builder)

Use Cases

Samples should work exactly as described without the user to find out missing pieces.
Checklist

[X] I have viewed, signed, and submitted the Contributor License Agreement.
[X] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[ ] I have added an integration test, if necessary.
[X] I have reviewed the styleguide for guidance on my code quality.
[X] I'm happy with the commit history on this PR (I have rebased/squashed as needed).

Also, the branch needs to be updated & Github won't let me update it. Sorry, I hate to always ask about that.

@ForestEckhardt - Are you able to update this branch or force merge?
The samples repo is so busy, it's hard to have @c0d1ngm0nk3y update the branch and get to it in time to merge.

OK.
@c0d1ngm0nk3y can you update the branch one more time please, then ping @ForestEckhardt and myself so that we can merge it ASAP. Make sure to ping as Github doesn't seem to notify us when the branch is updated. Thanks!


cli: support Uint160 hashes with 0x prefix

Close #1753 .

Same thing happens with importNEP17Token and importDeployed. Similar code is used in calcHash. So maybe we should provide some common utility function to convert parameter into uint160 using plain hash, prefixed hash and address and then use it everywhere (allowing addresses everywhere hashes are allowed (and updating documentation if needed))?


Quick Latex for Mobile

I love your plugin on my desktop it makes writing latex a million times easier. I really wish it would work on mobile and don't know if theres some reason that it can't or if its just an option that you could enable.

hi!
thank you for the suggestion!
I have tried enabling the mobile option, however it seems that the way this plugin is written does not work in the mobile environment. I may need some serious try and error to see where the issue may be.

As the documentation for the isDesktopOnly key in manifest.json seems to suggest, plugins don't work on mobile because they use some NodeJS or Electron API. Just throwing browserify at the problem might solve the issue by virtue of providing browser alternatives to Node APIs, which should also work on the Obsidian mobile app.


The 30fps 1080p lazy patch includes the DOF targeting mod in ver 1.1.1

There are massive differences between the 1.1.0 and 1.1.1 version patches, with entirely different mods between them.
This is not specified and seems like a bug.

I'm assigning Hover on that, it's his pack

I'm assigning Hover on that, it's his pack
Otherwise you can made a pull request with a fix too

The lazy pack patches for 1.1.0 has not changed since the previous distribution.
The new 1.1.1 lazy packs contain different patches, because not all patches from 1.1.0 are available in 1.1.1 yet, and vice versa.
The DOF removal is present in the 1.1.1 patches because it improves the fidelity of the presented image, by removing a low-res DOF effect.
This patch is not available for 1.1.0, but should this change, then it will be added at a later date.
This is the first complaint of the DOF mod, so I will consider it acceptable, unless a larger amount of users complain of missing, low-quality DOF.


Removing dump to json option at node level

dump to json at node level is not really needed as 'add_from_json' option is supported at only leaf level.

Are you going to removing dump for BFNode too?

No, 'dump' command at the node level is useful to dump entries of all its children.


Custom Skill calling Azure Function with EasyAuth

The Azure Function example uses ?code host key. What is the setup necessary to use System Managed Identity of EasyAuth enabled Azure Function?
System Managed Identity should is supported but setup is unclear: documenation

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

ID: 872a816d-1bfb-16fc-7589-de9598836723
Version Independent ID: bb057815-fa8b-db5c-d359-7e619e4450e3
Content: Custom skill example using Bing Entity Search API - Azure Cognitive Search
Content Source: articles/search/cognitive-search-create-custom-skill-example.md
Service: cognitive-search
GitHub Login: @LiamCavanagh
Microsoft Alias: liamca

@vdimov-gcmlp Thanks for your comment. We will review the issue and get back to you shortly.

Resolved by #1077850318

Hi vdimov-gcmlp, thanks for resolving.  We are scheduled to update the custom skill examples (4) in the next month and we can address the authentication options then.
In the mean time, the article you found is the best resource for using system identity:  https://docs.microsoft.com/en-us/azure/search/cognitive-search-custom-skill-interface#set-the-endpoint-and-timeout-interval
#please-close


Non-async version of rust bitcoin client

Right now the rust client crate provides only async API. While it is indeed used in many projects, there are cases when async is not desired. For instance, I always put tasks into a dedicated thread, communicating to it using channels; and I avoid using any async runtimes.
Is it possible to have a non-async version of the library? I am willing to work on the PR adding it, but are there any chances of getting it merged?

There is already a non async client, let me know if you have any issues using it - I definitely care about keeping it working.


pinning node to version 7.9, because of problems with zombie on 7.10

See #933
I've no idea how to fix the problem with zombie and the commit history on master for zombie shows many failing build, therefore i propose to pin the node version for now to keep the builds passing.
This can be reverted when a proper solution can be found for the failing browserify tests with zombie.

Coverage remained the same at 92.686% when pulling a5d059100f29bb2ce3533ef9415010ab7840d244 on LarsHassler:fix/failing-browserify-test-on-node-7 into 17d7c25682ea520b0a255e933b64d5c7f355330f on node-nock:master.

Now an other test fails. No idea why


Image standardization, cleanup; adding 6.10.3

General Dockerfile cleanup and standardization
Moved proprietary server-side startup command generation logic to
generateStrtupCommand.js, run on container startup.
Same for env vars (PM2HOME)
Standardizing on port 8080 as default; correctly handling customer
setting of PORT appsetting
Taking dependency on HTTP_LOGGING_ENABLED, which will now be set
for all containers (built in and custom)
Default site is now a static express site hosted out of
/home/site/wwwroot, in line with standard

Merging immediately. Sending to you guys as FYI if you want to have a look.

@nickwalkmsft Awesome! This change should enable us to enhance the core containers to detect startup scripts via package.json right?
https://docs.npmjs.com/cli/start

I believe that was always available, simply by putting "npm start" in the app startup command - the logic you see in generateStartupCommand.js is roughly what we've always had, it's just visible now. We have some work to do regarding the portal UX and what sane values are for "app startup command"/"startup file".
You can see the startup logic in generateStartupCommand.js, but just to sum it up here:

If your appCommandLine is two or more tokens, we run it directly, so you can do stuff like "npm start" or "node myapp.js myarg1 myarg2".
If your appCommandLine is a single token, we assume you're supplying a "startup file" script and we try to execute it with PM2 for robustness.
If your appCommandLine is empty, we hunt for a few conventional app names in wwwroot. We'll run one if we find it;  if we don't, we run the static page to show the preinstalled "Welcome to App Service" page.

We're trying to maintain a balance between "easy to get started" and "too much magic." Adding a check for a scripts.start in package.json might make a good addition to #3.
Please note that these images are pre-production - they are still subject to some review and testing, and they haven't yet been deployed.

Apologies, I was referring to an addition to #3. The auto-detection of a start script in package.json is already pretty well established in other products, so it would be a nice behavior to allow existing apps to be deployed, without needing to explicitly set the app command line flag.

Yep, that's a good point, and a good idea. At that point we're looking around for unspecified files to run, we might as well check for a start script first. I should be able to get this in in a day or so. Thanks!

@nickwalkmsft Awesome! Thanks so much

🎉 👍
This is fantastic!


Find Occurences output is empty in Find tab

Please read contributing guide first. Please read and remove this header before issue submit

IDE build number: 1641
Java version: 1.8.0_102
OS name and version: MacOS Sierra 10.12.4

Issue description. In most cases need provide reproduce example.
Every time I search using "find in path", the results are showing blank in the Find tab. This does not happen for other types of search, such as "Find Usages", etc.
The results show up fine in the preview tab of the find dialog, but after choosing "Find", the Find output is always blank.
Screenshots below:

Hi. Tested and fixed


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_3b4993ff-f34b-4d46-8ca9-8bc56e8d2bcb/pull/1 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-archive-test-target_3b4993ff-f34b-4d46-8ca9-8bc56e8d2bcb.git

Docs Build status updates of commit 892bddb:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-GitBranchDeletedOrForcePushed] Cannot sync git repo to specified commit because branch Release_Archive_master_2021-06-07-08-34-32 has been deleted or has been force pushed

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


StyleGAN Model ADNI Dataset 47492347

Hello team, this is my working implementation of a StyleGAN generative model trained on the ADNI brain dataset using PyTorch. Please have a read through my README file for a detailed description of my implementation details.
Summary of files:

dataset.py ~ contains data loading preprocessing functionalities
embedding.py ~ functions used to plot UMAP embeddings
modules.py ~ collection of modules and models that make up StyleGAN architecture
predict.py ~ test script used for generating images using a pre-trained model
settings.py ~ config file used for tweaking train hyperparameters
train.py ~ main model training script
utils.py ~ contains global imports

Feel free to inquire about any issues to fix or potential improvements!
Jun Woo (Jason) Jung
SN: 47492347

Recognition problem: StyleGAN on ADNI ---- Hard Difficulty

Algorithm solves the problem: (4 Marks)
Clear generated images;
Losses of the generator are unusual but left unexplained -0.5;
UMAP plotted on training data instead of the style space -0.5

Algorithm demonstrated and functions: (3 Marks)

Design: (0.5 Mark)
Not very modularized design -0.5

Commenting:
Sufficient docstrings and comments (1 Mark)

Algorithm above Normal Difficulty: (5 Marks)

Algorithm is Hard difficulty: (5 Marks)

Suggestions:

I can see you intend to achieve modularization, the problem is, however, only 5 scripts need to be uploaded (including utils.py, see Section 4 on task sheet). In your code, other scripts are not really necessary, e.g. functions in embedding.py should either go to dataset.py or utiles.py, so are the parameters in settings.py;
Embedding should be plotted on the style space according to the reference paper, and by doing so hopefully you can see it makes more sense;
The losses, especially ones from the generator, are abnormal --- this is fine since GANs are tricky. But insteading of displaying the loss plots, you should also attempt to justify how come the losses.

This is an initial inspection, no action is required at this point

Solves problem: Nice documentation and clear explanation of the network. The generated images are reasonable and have clear structures. Reasonable loss plots and nice records of all experiment steps. UMAP does not show clear clusters, but it is a good attempt to explore the reason.
Implementation: Good usage of ArgParse. Settings could be args in the training script. Overall nice
Design: Great
Comment: Sufficient and well-structured.
Difficulty: Hard

Marking

Good/OK/Fair Practice (Design/Commenting, TF/Torch Usage)
Adequate design and implementation. -1
Spacing and comments.
Header blocks.
Recognition Problem
OK solution to problem. Loss unexplianed.-1
Driver Script present.
File structure present.
Good Usage & Demo & Visualisation & Data usage. -1
Module present.
Commenting present.
No Data leakage found.
Difficulty : Hard. StyleGAN (Hard Difficulty)
Commit Log
Good Meaningful commit messages.
Some/Adequate Progressive commits. Somewhat minimal.-1
Documentation
Readme :Good.
Model/technical explanation :Good.
Description and Comments :Good.
Markdown used and PDF submitted. PDF Checked.
Pull Request
Successful Pull Request (Working Algorithm Delivered on Time in Correct Branch). ON time.
No Feedback required.
Request Description is good.
TOTAL-4

Marked as per the due date and changes after which aren't necessarily allowed to contribute to grade for fairness.
Subject to approval from Shakes

Good work!


Also allow links to Drive materials.

fixes: #30981

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

see: https://github.com/apache/beam/issues/30981


Add a 'bitwiseOr' resource (| binary operator)

There are some operators in AskVM, e.g. 'lessThan' resource (< operator) (src/askvm/resources/math/lessThan.ts), but we are missing bit operators, like 'bitwiseOr' resource (| operator).
Please note that bitwiseAnd (& operator) is listed here: https://github.com/xFAANG/askql/issues/374
Dear new contributor,
If you have any questions on how to start, please write a comment with a tag: @czerwinskilukasz1 or @mhagmajer, or join our Discord Community and let us know there. We will be very happy to help you start! :)
Cheers,
Łukasz (@czerwinskilukasz1)
AskQL Core Developer

Hello @czerwinskilukasz1 , I am new and want to look into it. How do I start?

Hello @subedigaurav , welcome to AskQL project.
Thanks for asking how to start.  First how about you get familiar with the part of code you were going to work on.
The description mentions an already existing operator (lessThan, implemented in src/askvm/resources/math/lessThan.ts). A good start would be to take a look at its code, understand more or less how it works and try to write bitwiseOr in a similar manner.  After that find out how < and lessThan were bound together and do a similar thing for |.
And the last thing would be to write tests which check whether your bitwiseOr resource and | operator work as intended. You could write the tests similar to the ones for > and greaterThan (we do have tests for < but they are not as great example):
https://github.com/xFAANG/askql/blob/master/src/askscript/__tests__/09-operators/operators-07-greater_than-oper.test.ts
https://github.com/xFAANG/askql/blob/master/src/askscript/__tests__/09-operators/operators-07-greater_than-res.test.ts
Let me know if you have more questions.

@czerwinskilukasz1 thank you for the reply. I have figured out the implementation for the bitwiseOr. But I am confused about the tests. Do I need to write tests and test them myself or is it done automatically?

Maybe helpful https://github.com/xFAANG/askql/issues/135 and pr for this

thanks @km4 . I get it now. I will be creating my first pr soon. :)

@subedigaurav , if you were wondering how to run the tests after you write them, please run:
npm run build
npm test

Of course re-running npm run build before npm test is only needed after you change files.


Merging v1alpha2 concepts docs into unversioned docs

What type of PR is this?
/kind cleanup
/kind documentation
What this PR does / why we need it:
I thought I was going crazy because I couldn't find docs I knew I'd added earlier (https://github.com/kubernetes-sigs/gateway-api/pull/863#discussion_r707650322). It looks like we missed merging some files over when we made the concepts docs unversioned. As far as I can tell, the unversioned api-overview page was most up to date so I left that unchanged. The other v1alpha2 pages were more up to date so I moved them over to the unversioned docs without any changes.
Does this PR introduce a user-facing change?:
NONE

/cc @hbagdi @youngnick

/lgtm
/hold

/lgtm
/hold cancel


How to apply half face texture

I wish to apply texture(hairsegmentation, iris coloring) to one side of the face.
I tried

as per this graph.
My app getting crashed when input_frame passing ImageCroppingCalculator
In case of Hair Segmentation, I wish to apply texture to either right side or left side
In case of Iris Tracking.  I wish to track only one side of the face.
Is it possible to achieve or am getting in the wrong way ?
Can anyone help

Expected Result achieved using this Calculator


[REF-2889] reflex-web benchmarking is failing

Describe the bug
Looks like the reflex-web benchmarking action is checking out the reflex-ci ref, which probably was not updated for the required radix 3.0 changes in 0.5.0
To Reproduce
Any merge to main.
See https://github.com/reflex-dev/reflex/actions/runs/9180410481/job/25244764598
REF-2889

Seems like we also started losing performance size data for mac after we pinned the macos version for reflex runs

Pinning the macOS version was mostly a hotfix to unblock the CI, we should probably look into using newer images if we can make them work.

@masenf can you add this on linear and assign me to it? Adding the linear tag has no effect when I do it


Incorporate josh Comeau's CSS reset

[x] Disable MacOS browsers' default subpixel antialiasing

body {
  -webkit-font-smoothing: antialiased;
}

source: https://www.joshwcomeau.com/css/custom-css-reset/#four-font-smoothing-5

[x] Hard-wrap long words like URL (which can fix #475 )

p, h1, h2, h3, h4, h5, h6 {
  overflow-wrap: break-word;
}

source: https://www.joshwcomeau.com/css/custom-css-reset/#seven-word-wrapping-8

[x] Avoid the z-index mess with a popup

#__next {
  isolation: isolate;
}

source: https://www.joshwcomeau.com/css/custom-css-reset/#eight-root-stacking-context-9

Disabling subpixel font smoothing requires one more vendor specific property for Firefox: -moz-osx-font-smoothing: grayscale;.


add hartmann

added hartmann and tested the technical parts with small iterations.
currently running longer tests to also see if it gflownet converges

I restructured the surrogate component to custom setups with DKL, MCD, probabilistic nn etc.


Bug: Unable to start server without OpenAI API Key

Affected components(s)

[ ] Chat Agents
[x] Chat Endpoints
[ ] RAG
[ ] LLM Models
[ ] Vector Stores
[ ] Chat History
[ ] Response Caching
[ ] API Key Authentication
[ ] Endpoint Deployment
[ ] Other

Describe the bug
Running the server throws an error when no value is set for the OPENAI_API_KEY variable in the .env file, which should not happen.
Ideally, only at least one of the API keys should be set: Google Gen AI API Key or the OpenAI API Key.

fix implemented


Cant Access the graph using foxx microservices

I am writing a Foxx microservice to execute an AQL query for my graph
'use strict';
const createRouter = require('@arangodb/foxx/router');
const router = createRouter();
const joi = require('joi');
const db = require('@arangodb').db;
const errors = require('@arangodb').errors;
const aql = require('@arangodb').aql;
const ipColl = db._collection('IPvertices');
const DOC_NOT_FOUND = errors.ERROR_ARANGO_DOCUMENT_NOT_FOUND.code;
module.context.use(router);
router.get('/traverse/:ip', function (req, res) {
    try {
        const data = db._query(aql`
                FOR v,e,p IN 1..2 ANY
                "IPvertices/${req.pathParams.ip}"
                GRAPH "test"
                RETURN p
            `);
        res.send(data);
    } catch (e) {
        if (!e.isArangoError || e.errorNum !== DOC_NOT_FOUND) {
            res.send(e);
            throw e;
        }
        res.throw(404, 'The entry does not exist', e);
    }

})
    .pathParam('ip', joi.string().required(), 'Your IP.')
    .response(joi.object().required(), 'Graph Traverse')
    .summary('Graph Traverse')
    .description('Assembles a list of graph traverse data for given ip');

But the API responses with error code 400
please note I have created a graph (name = test) in Arangodb WebUI and my AQL query is working fine in the UI
My AQL query

FOR v,e,p IN 1..2 ANY
    "IPvertices/<IP_ADDRESS>"
    GRAPH "test"
    RETURN p

What might be the reason for this?

Hi @Wathsara
Can you please share your environment information, such as:
ArangoDB Version:
Storage Engine: RocksDB
Deployment Mode: Single Server
Deployment Strategy:
Configuration:
Infrastructure:
Operating System:
Total RAM in your machine:
Disks in use:
Used Package:
Dataset:
Size of your Dataset on disk:
Replication Factor & Number of Shards (Cluster only):

Many Thanks

Hi @Wathsara,
This ticket is being closed as inactive due to the date of the last activity on it.
If you believe this was done in error, please comment or create a new issue.


slot couldn't add a 'slot' attribute on itself

Vue.js version
2.1.4
Reproduction Link

http://codepen.io/anon/pen/GNGMGw
Steps to reproduce
run the demo directly
What is Expected?
slot content in component foo should be displayed correctly
What is actually happening?
i encounter a problem when  upgrading my project to 2.x，slot couldn't add a 'slot' attribute on itself in<EMAIL_ADDRESS>works well in 1.x， the upgrading guide has not mentioned the issue yet，finally ，i solve it by wrapping slot into a template tag

http://vuejs.org/v2/guide/migration.html#slot-Attribute-Styling-removed


Add comment support

It is now possible to extend each model with comments using use CommentTrait; and CommentList.vue component.

Codecov Report

Merging #353 (ddca479) into 0.8-release (1093819) will decrease coverage by 0.82%.
The diff coverage is 73.25%.

@@                Coverage Diff                @@
##             0.8-release     #353      +/-   ##
=================================================
- Coverage          89.82%   88.99%   -0.83%
- Complexity           908      950      +42
=================================================
  Files                 46       50       +4
  Lines               3361     3526     +165
=================================================
+ Hits                3019     3138     +119
- Misses               342      388      +46

Impacted Files
Coverage Δ
Complexity Δ

app/AttributeValue.php
100.00% <ø> (ø)
10.00 <0.00> (ø)

app/Http/Controllers/NotificationController.php
0.00% <0.00%> (ø)
9.00 <9.00> (?)

app/Notifications/CommentPosted.php
0.00% <0.00%> (ø)
3.00 <3.00> (?)

app/User.php
92.00% <ø> (ø)
13.00 <0.00> (ø)

app/Traits/CommentTrait.php
91.17% <91.17%> (ø)
9.00 <9.00> (?)

app/Attribute.php
100.00% <100.00%> (ø)
6.00 <0.00> (ø)

app/Comment.php
100.00% <100.00%> (ø)
3.00 <3.00> (?)

app/Entity.php
79.31% <100.00%> (ø)
31.00 <0.00> (ø)

app/Http/Controllers/EntityController.php
82.33% <100.00%> (+3.84%)
117.00 <14.00> (+18.00)

app/Http/Controllers/HomeController.php
84.37% <100.00%> (+1.04%)
8.00 <0.00> (ø)

... and 6 more

Continue to review full report at Codecov.

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


fix for overlapping partitions

Hoyt, this is the fix for this issue: https://github.com/dato-code/SFrame/issues/312
whenever we merge this fix, then I can start building for linux/mac and collecting cy_spark_unity.so artifacts and generate a new spark_unity.jar.


Search sign group Incompatible with "auto-nohl" snippet

This is kind off more a request for support than a bug report, but since a bunch of people have such a "auto-nohl" snippet, it is probably relevant to more people than just me
So I use the following snippet to automatically set :nohl when I am using a non-search key (not n / N etc.) and automatically enable search highlights when I do a search:
vim.on_key(function(char)
	local searchKeys = { "n", "N", "*", "#" }
	local searchConfirmed = (fn.keytrans(char) == "<CR>" and fn.getcmdtype():find("[/?]") ~= nil)
	if not (searchConfirmed or fn.mode() == "n") then return end
	local searchKeyUsed = searchConfirmed or (vim.tbl_contains(searchKeys, fn.keytrans(char)))
	if vim.opt.hlsearch:get() ~= searchKeyUsed then vim.opt.hlsearch = searchKeyUsed end
end, vim.api.nvim_create_namespace("auto_nohl"))

however, when I have nvim-scrollview as well as that snippet active, pressing n results in the highlights (and the search signs in the scrollbar) briefly appearing and than disappearing.
Would it be possible to make this plugin compatible with the auto-nohl snippet or give some advice on how the snippet has to be changed to work together with this plugin?

yes, the issue also occurs with nothing but nvim-scrollview and the snippet in my config :/

I can now reproduce. I was originally only looking at the scrollbar signs, which still are displayed for me. But I can now see that the search pattern is not highlighted.

nvim-scrollview calls normal! while updating scrollbars, which triggers the vim.on_key callback.
I've added a global variable, vim.g.scrollview_refreshing, that gets set to v:true while scrollbars are being refreshed (a0ae5f9, 02d0880).
With that update, a workaround is to ignore keys while scrollview is refreshing.
vim.on_key(function(char)
    if vim.g.scrollview_refreshing then return end
    ...

thanks! can confirm that this does fix the issue 😊


[flagd-in-process] Add targeting validation and log warning

The Go/in-process, Java/in-process and standalone versions of flagd log a warning message with invalid targeting rules when they are fetched. We should implement this in the JS/in-process.
Targeting rules can be validated with the flags.json schema in conjunction with the targeting.json schema. Both will have to be loaded into AJV to fully validate. See similar examples in Go and Java
The in-process provider should be modified to use both of these schemas, and validate against them, only logging a warning if validation fails. Only a failure to parse the definition into a FeatureFlag instance should cause an abortive error (this means relaxing the existing validation to be consistent with all other impls which just do basic parsing).

I am not sure if I understand this task correctly, but https://github.com/open-feature/js-sdk-contrib/blob/main/libs/shared/flagd-core/src/lib/parser.ts#L8 seems to me we're already doing validation, and only throw an exception if the property is set, else it is a warning.
Did I miss something here, @toddbaert? or should we do something differently?

specifically due to https://github.com/open-feature/js-sdk-contrib/pull/878

You are right. I or somebody else must have done this!


library machine type  'x86' conflicts with target machine type 'x64'

i want to build onnx mlir from source  on windows x64 . i was following windows build readme of onnx-mlir. Protobuf and LLVM-Project build part are correctly done but when i run "call cmake --build . --config Release --target onnx-mlir" then get warning LNK4272: library machine type 'x86' conflicts with target machine type 'x64' and got some error.
how do i solve this problem ? thanks in advance.

@sstamenova any suggestions for @sazzad15-1779 ?


Documentation/Example code

Thank you for this initiative to bring JDBI to Quarkus! I'm coming from Dropwizard and I'm new to Quarkus...
and without a SQL centric database approach like JDBI I don't know if it is going to work out for me.
Can you please provide an example of how to use this extension?
And possibly clarify if it working well together with

reactive drivers?
connections pools?

Hi,
You need to inject AgroalDatasource:
public class JdbiProvider {
    @Inject
    AgroalDataSource ds;

    @Singleton
    @Produces
    public Jdbi schedJDBI() {
        Jdbi jdbi = Jdbi.create(ds);
        jdbi.installPlugin(new SqlObjectPlugin());
        return jdbi;
    }

and you can use it everywhere:
public class UserDAO {
    @Inject
    Jdbi jdbi;

    public Optional<User> findUserById(Jdbi jdbi, long id) {
        return jdbi.inTransaction(transactionHandle ->
                transactionHandle.createQuery("SELECT * FROM users WHERE id=:id")
                        .bind("id", id)
                        .mapTo(User.class)
                        .findFirst());
    }
}

Connection pool is working per default.
Jdbi does not support reactive drivers.

Thank you for the helpful example!
Concerning the reactive drivers: do you think this is a fundamental issue or likely to be supported at one point?

I don't think it will support it soon:
https://github.com/jdbi/jdbi/issues/1454


Test P-384

This just tests P-384 support and builds on top of https://github.com/RustCrypto/elliptic-curves/pull/600.
We still have to wait for new versions to be released.
Builds on top of https://github.com/facebook/voprf/pull/101.
Fixes #81.

Rebased on #85.

Rebased after #85.

Everything upstream was merged now, so we are only waiting for #101.

This is good to go now too!


network fee change

Hi,
why network fee change after transfer confirm?

could you please share your transaction hash?

Thanks for following up.
This is the second time this has happened to me.
They are exactly the same.
https://blockchair.com/dogecoin/transaction/f2b59cfd4930922bd451a44f2115dbaea748d3fa53e4886e2ba2a4bbf4f31f82?from=trustwallet

https://blockchair.com/dogecoin/transaction/b95b7deb2138dc06cdbf8818d8cb94a3ae77ccf9c454983b8eba949d276f86b1?from=trustwallet

hi, please check this.
https://github.com/dogecoin/dogecoin/issues/3018#issuecomment-1172976618

Why is no one doing anything to solve this problem?

@nonomaxxis We're investigate, will post update soon


Use reference to bytes data

To avoid the unnecessary copy.

Sorry, but I want Context to own the data, especially in combination with FFI.

For FFI, I think you can own the data from stack of FFI function. Since you are not exposing Context to outside, I don't think it makes sense to have it own the data.

One reason for doing so is that, with this change, callers would have a better control over the data it passes in. That said, if they want to erase the sensitive data after computing the hash, they can do. But if the crate copies the data and owns the data itself, callers would not be able to control the location of sensitive data without more complicated technique.

I find your last argument very convincing so I agree with this change. However, I just pushed the Config changes creating conflicts with this pull request . Do you want to update this pull request or should I write the changes and close this pull request?

If you are going to write the changes, please go ahead and close this pull request :)
I don't think I have time to get to it today... but if you prefer me to update the pr, you can keep it open, and I'll do it tomorrow.

I'll try to write it today.

I committed a slightly modified version in 1018dfcc5444be7076cb92135d4e479f8c16954a. Thanks for the suggestion.


JJK_quote_Generator-Yashas Singh

Hi my project is a quote generator which essentially takes a quote from a show called Jujutsu Kaisen and draws an image of the person the quote is associated to.
I don't have too much experience with coding in js so it was definitely was a bit harder to work on.

Thanks for the submission! Your folders seem to be duplicated. Can you fix that?

yup done @leomcelroy

I haven't heard anything so I'm going to go ahead and close this for now so it's not in my re-review list. If you want to keep working on it just let me know and we can re-open this PR for now!


Make group(by:) generic for all Collections

🚀
This change allows the function to be used on String, for example.
Checklist

[x] I checked the Contributing Guidelines before creating this request.
[x] New extensions are written in Swift 5.0.
[x] New extensions support iOS 8.0+ / tvOS 9.0+ / macOS 10.10+ / watchOS 2.0+, or use @available if not.
[x] I have added tests for new extensions, and they passed.
[x] All extensions have a clear comments explaining their functionality, all parameters and return type in English.
[x] All extensions are declared as public.
[x] I have added a changelog entry describing my changes.

          1 Warning

:warning:
Consider adding tests for new extensions or updating existing tests for a modified SwifterSwift extension

          4 Messages

:book:
Thank you for submitting a pull request to SwifterSwift. The team will review your submission as soon as possible.

:book:
iOS: Executed 607 tests, with 0 failures (0 unexpected) in 12.170 (14.195) seconds

:book:
tvOS: Executed 581 tests, with 0 failures (0 unexpected) in 5.667 (8.054) seconds

:book:
macOS: Executed 434 tests, with 0 failures (0 unexpected) in 1.076 (1.392) seconds

  Generated by :no_entry_sign: Danger


Fixed #28104 -- Prevented @condition decorator from setting ETag/Last-Modified headers for non-safe requests.

https://code.djangoproject.com/ticket/28104

Since this is a bugfix I'm not even sure if we need the release note? It's a "backwards incompatible change" in so much as the current behavior is wrong.
I have updated the docs and am pretty happy with them now I think.

Just rebased, any further thoughts by anyone or steps I should take?

@timgraham i updated some, thoughts?. having a bit of a problem explaining why we don't set the headers generally. it's partially the design of the decorator and partially rfc compliance and partially simply that we can't divine it automatically. i think we might be able to set last-modified for POST without issue but am not 100% sure.

I don't have much expertise to offer here.


producer: add warning about RequiredAcks>1

Per https://groups.google.com/d/msg/kafka-clients/4dR2FeN9GKE/_I3WRMoF8AsJ
Setting it to a value > 1 will raise an exception in <IP_ADDRESS> and later, so
update the godoc and log a warning when validating such a configuration.
@wvanbergen @snormore @drdee
cc @mkobetic @yagnik

LGTM.


Formdata attributeerror

I updated to v11 and when I started my bot was spammed with an attribute error, I pinpointed the issue and was able to fix it for myself, don't know how universal it is though. I'm on debian stretch, python3.5, aiohttp2, telepot v11
https://github.com/nickoala/telepot/blob/master/telepot/aio/api.py#L45 is the line in question, I had to change it to
aiottp.FormData()

For it to work proper.

Exactly which version of aiohttp are you using? Mine is aiohttp 2.0.7

Well when I updated it said it was updating it to 2.0.7 and an attempt to install --upgrade says the requirement is already satisfied, aiohttp 2.0.7, easy_install says 2.0.7, but pip freeze says 1.0.5. So this looks to be a problem on my end, I'll close the issue when I get the solution so anyone else with the problem has it too.

I had a
~/.local/lib/python3.5/

With some packages, specifically the old aiohttp one, so I had to move any there that would have been missing and then moved it to lib.bak, looks to have fixed the issue.

You remind me of something though, that aiohttp makes available a lot of classes from the top level. Instead of using aiohttp.formdata.FormData(), making it aiottp.FormData() will be more robust and less sensitive to module changes. Maybe I will do that in the next update.


[bazinga-tools] Deprecate/delete the extention

Extension
https://www.raycast.com/luin/bazinga-tools
Raycast Version
1.74.0
macOS Version
14.3.1 (23D60)
Description
Seems that the domain name that this extension uses (bazinga.tools) is not available anymore:

Steps To Reproduce
I just was browsing and checking the source code and found this.
Current Behaviour
No response
Expected Behaviour
No response

Yeah sorry I forgot it here. Bazinga tools is not maintained anymore so should be removed.


[Doc-Improvement][Update teams manifest schema][2689677]

Update manifest schema with messageHandlers.value.supportsAnonymizedPayloads.

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/resources/schema/manifest-schema-dev-preview.md
90
28
link
:white_check_mark:

msteams-platform/resources/schema/manifest-schema.md
94
21
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Learn Build status updates of commit 1b5a85c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/resources/schema/manifest-schema.md
:white_check_mark:Succeeded
View

msteams-platform/resources/schema/manifest-schema-dev-preview.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/resources/schema/manifest-schema-dev-preview.md
89
29
link
:white_check_mark:

msteams-platform/resources/schema/manifest-schema.md
94
21
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Learn Build status updates of commit 2b6fd28:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/resources/schema/manifest-schema.md
:white_check_mark:Succeeded
View

msteams-platform/resources/schema/manifest-schema-dev-preview.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/resources/schema/manifest-schema-dev-preview.md
89
29
link
:white_check_mark:

msteams-platform/resources/schema/manifest-schema.md
94
21
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Learn Build status updates of commit 0c103bb:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/resources/schema/manifest-schema.md
:white_check_mark:Succeeded
View

msteams-platform/resources/schema/manifest-schema-dev-preview.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/resources/schema/manifest-schema.md
94
21
link
:white_check_mark:

msteams-platform/whats-new.md
92
36
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/resources/schema/manifest-schema.md
94
21
link
:white_check_mark:

msteams-platform/whats-new.md
92
36
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Learn Build status updates of commit 608e094:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/resources/schema/manifest-schema.md
:white_check_mark:Succeeded
View

msteams-platform/whats-new.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/resources/schema/manifest-schema.md
94
21
link
:white_check_mark:

msteams-platform/whats-new.md
92
36
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Learn Build status updates of commit 33c9bd9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/resources/schema/manifest-schema.md
:white_check_mark:Succeeded
View

msteams-platform/whats-new.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/resources/schema/manifest-schema.md
94
21
link
:white_check_mark:

msteams-platform/whats-new.md
92
36
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Learn Build status updates of commit 57ef2a1:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/resources/schema/manifest-schema.md
:white_check_mark:Succeeded
View

msteams-platform/whats-new.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit f88aa63:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/resources/schema/manifest-schema.md
:white_check_mark:Succeeded
View

msteams-platform/whats-new.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Remove Mentors & Requests tables from UI

Feature
Why is this feature being added?
The current Mentors table does not provide valuable data to users. It's a list of mentor names and zip codes. Currently, logged in users can still easily request a mentor from their profile page.  We can remove this from the UI and regroup around other ways to improve the mentee / mentor interface (see https://github.com/OperationCode/operationcode/issues/758)
The Requests table can also be removed - it's an empty table that is not currently displaying anything.
What should your feature do?
Remove the unneeded components (everything in 'scenes/home/mentor')
Remove 'Mentors' as a navigation option while logged in
Remove any related code that will be unused
Screenshots

I would like to work on it if possible

@tal87 it's yours!


CveDB loadCpeEcosystemCache loads cache with (vendor,vendor) pairs

Describe the bug
While looking at how to incorporate the changes for Oracle I stumbled upon a bug in CveDB. https://github.com/jeremylong/DependencyCheck/blob/9780493d25791e3fa938d1c77c3b52a82af830ea/core/src/main/java/org/owasp/dependencycheck/data/nvdcve/CveDB.java#L865
When loading the EcoSystemCache from the DB it loads the vendor in both Left and Right of the pair used as the map key, rather than loading vendor in Left and product in Right
Version of dependency-check used
The problem occurs using version 6.X of the core
Log file
N/A spotted on code inspection.
Expected behavior
Left of the pair gets filled with vendor, Right of the pair gets filled with product
Additional context
If no-one else gets to it to code a fix I'll work on creating one after finishing the Oracle support for the DC v6 database changes.

omg - thank you for noticing this.  I'll get it updated shortly.


Multiple folders support

Question
Description
Hi,
I'm facing a problem very similar to this one.
I'm trying to implement docz in my react-native project, this is how my root folder is structured
|- android
|- ios
|- docs
|- src

I don't want docz to parse android and ios folders otherwise it will run into errors.
This discussion  says I should specify the folders docs and src in the files config.
This is my doczrc.js but it still parses android and ios folders.
{
	title: 'React Native',
	files: '{src,docs}/**/*.{md,markdown,mdx}',
	src: './',
};

I've ran into this exact issue. Anything in src is still parsed even if you are more specific with files. The docs don't really explain the distinction. It would seem that only having a files option should suffice? Thanks!


koord-runtime-proxy: Introduce envs injecting under contianerd scenario

Signed-off-by: honpey<EMAIL_ADDRESS>Ⅰ. Describe what this PR does

Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews
V. Checklist

[ ] I have written necessary docs and comments
[ ] I have added necessary unit tests and integration tests
[ ] All checks passed in make test

/lgtm
/approve

/lgtm


deploy a rendezvous to IPFS infra

@lgierth @VictorBjelkholm did cloud.ipfs.team changed? I'm being unable to deploy, always with this error:
remote: App container failed to start!!
=====> ws-star web container output:
       npm ERR! Linux 4.4.0-98-generic
       npm ERR! argv "/app/.heroku/node/bin/node" "/app/.heroku/node/bin/npm" "start"
       npm ERR! node v6.12.0
       npm ERR! npm  v3.10.10
       npm ERR! missing script: start
       npm ERR!
       npm ERR! If you need help, you may report this error at:
       npm ERR!     <https://github.com/npm/npm/issues>
       npm ERR! Please include the following file with any support request:
       npm ERR!     /app/npm-debug.log

Which gives really no useful information as there is a npm script start and works flawless locally.

The failure was caused by what I expected, the Dockerfile was confusing Dokku and although I renamed it, I failed to switch branches when pushing.


Add option at Cluster level logging config to include RKE K8s system components or not

When configuring cluster level logging, we need to provide an option to exclude the RKE system components from the main log stream.
An admin that configures the system project logging, can then route the RKE system components to a log target of their choosing.

@tfiduccia We use exclude because we want the default is same as before, before is we will include the system component by default, so now also the include is by default, if a user doesn't want system component log, they click the exclude checkbox to exclude them.

@aiwantaozi This has nothing to do with what the default is.  If the default should be enabled,  then make that the default. If you have existing entries missing the field, you treat nil from the backing store as enabled and return enabled in the API.  We have this pattern in a bunch of places.
Options are presented in the API as "enable thing: true/false", not "disable thing: true/false". "thingDisabled: false" or "don't disable thing" is a counterintuitive double-negative that is hard to read and understand.

@aiwantaozi - Also either way the radio button and the field are mismatched.  When the radio button is enabled, which is the default, it's setting the "disabled" field to true.

Thanks！we will do that

Sure, will update them, thanks

Sure, will update them, thanks

probably makes sense to hold off on testing until the UI issue is to test as well https://github.com/rancher/rancher/issues/18026

@aiwantaozi @loganhz  Project level logging should not have the Include System Log option and should be removed.

@kingsd041 we mention it before, @n313893254 will submit a pr

Version: Master 2.2 (2/14/19)
Confirmed that for project level logging the System logging option is not available for each target type including custom. I confirmed I can only see the "Include System Log" checkbox option for each target type when we are at cluster level logging.
This function was already tested in another issue around cluster logging. But I did briefly smoke test to ensure it still works as expected from UI perspective. In the API this is includeSystemComponent: <boolean>


add releationships to the show / index view template

even if it's just a simple id of the relationship, it will add some basic scaffolding for users to then modify the view to what they want.

on relationships that have many, add a dynamic "add item" button. also, remove item button on existing items.

ids of related items on index page, and full related items from show controller method when viewing parent show page


Error installing insomnia-inso

Describe the bug
Cannot install Insomnia CLI (insomnia-inso) due to font-manager error
To Reproduce
Following installation guidance here: https://support.insomnia.rest/article/109-cli-installation

Install npm
Install npm install -g insomnia-inso or sudo npm install -g insomnia-inso
Receive following error:
gyp ERR! clean error
gyp ERR! stack Error: EACCES: permission denied, rmdir 'build'
gyp ERR! System Darwin 19.5.0
gyp ERR! command "/usr/local/bin/node" "/usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /usr/local/lib/node_modules/insomnia-inso/node_modules/font-manager
gyp ERR! node -v v12.18.2
gyp ERR! node-gyp -v v5.1.0
gyp ERR! not ok
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: node-gyp rebuild
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.

Expected behavior
A clean install of insomnia-inso
Screenshots
n/a
Desktop (please complete the following information):

OS: 10.15.5 (MacOS)
Installation Method: npm
App Version: for Insomnia Designer -> Version 2020.3.3 (2020.3.3)

Additional context
Add any other context about the problem here.

Oops, it looks like we added font-manager to the CLI dependencies by mistake. We will remove it for the next release.

As a workaround for now, you can use Node v10, I'll make a note in the readme. We should, however, fix this. 😄


Uncaught Error: Cannot find module "react/lib/ReactComponentTreeHook"

Building this project gives me the following error:
Uncaught Error: Cannot find module "react/lib/ReactComponentTreeHook"
This results in a blank window.

Does anyone have a solution to this?

Seems like the version of react in this project is not compatible with the version of react-dom. React does not contain ReactComponentTreeHook at all.

To be more clear: It builds as it should, but when I open up the browser I see that error.

Hi. Sorry for the delay. I think I manage to fix your issue. Please pull latest commit and check if it works now. The react-dom version should be fixed at 15.0.1.

Yessir it works! Perfect! Thanks a lot!

OK. Thanks for reporting this bug.
I should probably update react and react-dom versions but ATM I have no spare time to do this.


fix(JSONPointer): rework handling of expected value to be more powerful

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 99.848%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
9189

Relevant Lines:
9203

💛  - Coveralls


where with a string that contains '/' add an escape char

Hello,
when you do :
Model::where('a', '=', 'abc/def')->get();

You would like to do something like that :
db.model.find({a: 'abc/def'});

But instead it does something like that
db.model.find({a: 'abc\/def'});

That's very problematic. The only whay I find to execute my query is tu use this :
Model::where('a', 'not regexp', 'abc/def')->get();

Hi,
i have the same problem! i have to search in a field that contains "/" this slash or other value like "$".
How can i search?


How should/would the cache be used remotely?

Originally posted by @choldgraf in https://github.com/ExecutableBookProject/jupyter-cache/pull/6#issuecomment-590100257
Maybe a use-case to consider here.
A team has a really big book, it takes 2 hours to complete. An author forks the book, clones it locally, edits one page. They want to contribute the page back. A few questions:

Do they need to run the entire 2-hour build process locally before seeing what the page looks like? --> seems like this could be handled by letting cache execution step be configurable only to specific files
When they make a PR, does the entire book need to re-build top to bottom on the CI/CD job? --> here the cache could probably be stored as a build artifact in a CI/CD job independent of the .git repository
Is there any way for a "master cache" to be bundled with the book?

If so, then is that a pattern we want to encourage?

I could see a benefit of committing the cache, in the sense that then git would keep track of changes to the cache and diffs to the pages would propagate through github, clones, etc. However, I worry about a few things:

The cache would probably become gigantic for non-trivial projects, unless it could be incrementally-updated and have some kind of "shallow clone" behavior.
It would require sub-moduling a book repository, so I think it would only work for fairly power-users.
The cache diffs themselves would be binary (I think?) so they wouldn't make any sense in github which would make it hard to know what has changed in the cache.

Our Python lectures take around 1.5 hours to build from scratch, so this is our scenario.
For 99% of our PRs, we just make the edits in RST, generate the ipynb for that one page and then run it manually to see if it looks OK.  This is fine for most edits, which typically adjust language or tweak code.
If we're concerned about how this looks in the PDF, say, we generate that one page locally.  Sometimes RAs will include an image in the PR to show that the PDF looks fine.
These are imperfect systems but they work OK for the most part.  So my vote would be for us to favor simplicity, at least initially, but not committing the cache.  (Plus, I'm a reasonably sophisticated user, but submodules still confuse me.  My instinct is to fear and distrust them.)

@jstac you can never trust two things: politicians, and sub-modules.
I wonder if one potential way to address this would involve meeting another use-case: building single-page documents. If we make the CLI easy for building the HTML or PDF of one page and letting users quickly preview what it looks like, the same machinery could be re-used for people that only want to build a single page and not an entire book...

Yep, that seems like a good idea.  Two birds with one stone, etc.  And the single-page use case is certainly important.
Such tools are available in jupinx for reviewing edits to QE lectures. I suppose cross references involving other pages won't work.  But, for 99% of cases, it's perfectly fine.

Glad you have mentioned this @jstac. It will be really important to support rendering of single pages for usability. We currently do this using environment variable FILES= and passing that through to SPHINX. I agree the CLI tool needs to cater to this and make it easier :-)

An approach I was playing around with for the jupyter book CLI was to use jupyter-book page: https://jupyterbook.org/features/page.html
perhaps we could use the same pattern, but also allow for PDF output with a kwarg or something?

As discussed with @mmcky, jupinx currently uses a static cache, housed in the Sphinx _build folder on an Amazon server. The build is persisted for all execution triggers (cron jobbed every hour), which run a 'git-pull' then sphinx-build. For this use case, the (just merged) hash implementation of jupter-cache should work fine.
@mmcky also noted that their current (sphinx based) cache implementation doesn't work on Travis CI; presumably because the cache is compressed/un-compressed, changing the file mtime's that sphinx uses to determine re-builds (matching to a dictionary stored in the pickled environment object). This wouldn't be an issue for jupyter-cache since it is hash based.
It would also be interesting to think how it might work with GitHub actions, CircleCI and ReadTheDocs builds.

Just a note to self, in case this is issue is encountered (sqlite on NFS):  jupyter/notebook#1782

Another related note: for jupyter book I was starting to collect a repository with several CI/CD patterns that could be used to deploy books: https://github.com/choldgraf/jupyter-book-deploy-demo
I think it'd be helpful if we replicated that repository for the new build system, ideally with multiple levels of complexity that users may want (e.g. vanilla build w/o execute then host online, execute and build, and execute+cache and build


Fix for cursor and cluster tests on OS X

This reduced the map size and stack depth to make tests work on OS X.

Thank you! I really should find a way to test on OS X myself, but I don't have any Apple devices currently.

I can confirm that latest master installs fine on Mac OS X whereas v0.4.3 does not. Would be great to publish to npm.

@mhart Can you please check whether the tests run correctly there as well?


Add Judicial Appointments Commission (JAC) agency

Changed the intranet data structure to allow a new agency. One thing to note, is that there was an old JAC entry here that deviated from the standard agency prefix, so instead of JAC it was the full judicial appointments commission title. So I've removed that who old entry and created a new one using just JAC. Now it is inline with how we approach the other agencies.
Added JAC to the menu registry
Added JAC header colour
Added JAC email (and email to comment allowlist)
Add ACF fields for agency homepage
Removed JAC from appearing in switcher until site goes live

I've manually added back in the ACF fields I could see have changed. Hopefully that's got them.


256 fxblox app api

@ehsan6sha @matiqorb @mehdibalouchi
I need more context, I added some sample interfaces but that is not enough. what operation needs to be done on the formidable interfaces.

Hi @farhoud , Great stuff, I think It is better to define APIs as an Async method

Work move to https://github.com/functionland/Blox-fula


#22  Seperated input and output from step 4 so it is clear for noobs …

…what to paste in and what to expect out
This is potentially the smallest PR ever

FYI you can add us as reviewers in the upper right. We will be notified in slack when there are PR's awaiting review.


feat: add example react app to test wallet plugin

based on Dan's https://github.com/web3/create-web3js-dapp

Why doesn't the example dApp actually use the plugin?

@danforbes thanks for the review, I will apply all your comments in a follow up PR https://github.com/web3/web3-wallet-rpc-utils/pull/5 as I'm afraid the changes might get lost as I restructured a lot of files later. The usage of a plugin is coming in later PRs as well.


Update README.md

@Fenscode Can you please add it in alphabetical order.

I've rearranged my detail according to alphabetical order.


⚠ DesiTorrents has degraded performance

In b838642, DesiTorrents ($DST) experienced degraded performance:

HTTP code: 200
Response time: 1719 ms

Resolved: DesiTorrents performance has improved in 0e4ae54.


composer (and /usr/local/bin) need to be writeable in web image

Describe the bug
composer (and /usr/local/bin) need to be writeable in web image
To Reproduce
$ ddev exec composer self-update --1
Warning: You forced the install of 1.10.17 via --1, but 2.0.4 is the latest stable version. Updating to it via composer self-update --stable is recommended.
Upgrading to version 1.10.17 (1.x channel).

  [Composer\Downloader\FilesystemException]
  Filesystem exception:
  Composer update failed: "/usr/local/bin/composer" could not be written.
  rename(/usr/local/bin/composer): Failed to open stream: Permission denied

Randy
I got exactly the same error when attempting to upgrade composer on my Ubuntu 20.04 system running:
Docker version 19.03.13, build 4484c46d9d and
DDEV-Local version	v1.15.3
commit            	v1.15.3
db                	drud/ddev-dbserver-mariadb-10.2:v1.15.1
dba               	phpmyadmin/phpmyadmin:5
ddev-ssh-agent    	drud/ddev-ssh-agent:v1.15.1
docker            	19.03.13
docker-compose    	1.25.0-rc4
os                	linux
router            	drud/ddev-router:v1.15.1
web               	drud/ddev-webserver:v1.15.3
Here is the cli response when I tried to self-update Composer:
$ ddev exec composer self-update --stable
Updating to version 2.0.6 (stable channel).
Downloading (100%)
[Composer\Downloader\FilesystemException]
Filesystem exception:
Composer update failed: "/usr/local/bin/composer" could not be written.
rename(/usr/local/bin/composer): failed to open stream: Permission denied
self-update [-r|--rollback] [--clean-backups] [--no-progress] [--update-keys] [--stable] [--preview] [--snapshot] [--1] [--2] [--set-channel-only] [--] []
Failed to execute command composer self-update --stable: exit status 1

You can easily use ddev exec sudo composer self-update --stable if you'd like. It has always worked. You can also update it with a custom Dockerfile, which is easier. See https://github.com/drud/ddev/issues/2383#issuecomment-712175038

Yes, your suggestion worked a treat.
Thanks for responding :)


Unable to re-build projections manually. Projection agent is stuck

I've encountered issues that make projections re-building via CLI to be stuck/deadlock. Only after a while, I see the error message.

🐛 After further investigation, I discovered it was related to type mismatch in the event model. So the issue occurred during the development/design time.
It would be great to see errors reported when something goes wrong. Is this something that can be added? Do you accept PRs?

@NikiforovAll I'm certainly happy for any and all pull requests to improve the logging. There's quite a bit of logging in the daemon, so where is an exception happening that you don't see in logs?

@jeremydmiller the console app halts when I run the projections command. It is like a deadlock or something. When I spin up a visual studio without "Just My Code" option I can see that JSON deserialization exception is thrown but is not propagated to the client code.
Steps to reproduce:

Store event
Make breaking change to the event model
Rebuild projects

@NikiforovAll I think I'm going to describe this as "Marten can't rebuild projections where there's a poison pill message". I think the error handling that's supposed to stop the rebuilding when there's an invalid event is kicking in and stopping it.
I think I can mimic the serialization problems

Notes:

In rebuild mode, automatically skip any event that has serialization failures for now, and automatically make them dead letter events
Assert no dead letter events in rebuild command? Make that opt in, opt out??? Fail fast???
May need to change the Rebuild mode to also pass in rebuild settings


Fix "-e" option for "azure network net create".

Command line option should use "-"(ASCII: 0x2d) instead of "-"(Unicode: 0x2013: Em-Dash).
This causes following error when  this script is copied and pasted to command line.
info:    Executing command network vnet create
error:   The name can contain only letters, numbers and hyphens with no more than 63 characters. It must start with a letter or number

Hi @ksasagit, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@ksasagit Thank you for this contribution.  I'll have the author review.
@dlepow Would you please review this contribution and let me know if I should merge it?  Thank you.

@ksasagit - Thanks for catching this! If you have additional feedback, feel free to contact me. I've had other suggestions for improvement that I'll incorporate soon.
@deneha - Hi Dene, please merge. Thanks!


Fixed error in example code, fixed typos and grammar

The error I'm referring to is in The basics. Line 1: import { GoogleSpreadsheet } = from 'google-spreadsheet';. The = should not be there.

Thanks - I'll merge these changes into the main README.md, as the docs readme index is a copy


Is it possible to document benchmark

Hi @paul-gauthier, thank you for such wonderful job with aider! I am currently trying to run the benchmark tests but the problem is that there is no documentation on how to do it. Specifically, there is no doc on the arguments of benchmark.py and on how environment variables should be set. Documentation on the tree structure of python exercises would also be nice.  Would it be possible to add some ?

Here you go!
https://github.com/paul-gauthier/aider/tree/main/benchmark

Thanks @paul-gauthier !


EnvelopedEvent is no longer exported with v3.21.2

@slack/bolt version
3.21.2
Your App and Receiver Configuration
import { type EnvelopedEvent } from '@slack/bolt';

const parseEventJson = (body: string) => {
  const slackEvent: EnvelopedEvent = JSON.parse(body);
};

Node.js runtime version
v20.10.0
Steps to reproduce:

My application works fine with @slack/bolt 3.21.1
Upgrade @slack/bolt to 3.21.2
tsc throws the following error: error TS2305: Module '"@slack/bolt"' has no exported member 'EnvelopedEvent'.

Expected result:
Backward compatibility should be maintained, and EnvelopedEvent should be importable.
Additionally, EnvelopedEvent was explicitly exported in #1334.
Actual result:
EnvelopedEvent is not exported at 3.21.2.

Hi @nekoya, thank you so much for reporting this issue, and we're sorry for the disruption by a patch release.
I have identified the PR that introduced this change: https://github.com/slackapi/bolt-js/pull/2223/files#r1753234778. Other members of my team will look into this issue later during North American timezone hours and will likely release a patch to resolve it.

My apologies, that was my mistake. I will cut a patch release to fix.

bolt v3.21.3 is live and should address this issue. Once again, my sincere apologies for the disruption 🙇


Loading Gzipped version css/js in the document

Hi @Stolz
As i can see, we have gzip version enabled and we are able to create .gz file on my local. I would like to know how does it actually load in the document. After checking the source code, i found it loads only the non gzipped file. Can you please help with this?

Since I requested this feature....
This is an NGINX feature.  If it receives a request for file.css, and it would send a compressed version of this file, then it looks in the filesystem for a pre-compressed version file.css.gz.
If the file is found, it will send this, rather than compressing file.css.

As Greg mentioned, this is a feature of the browser and the webserver, not the assets library itself.
In a nutshell: Some webservers, if properly configured, before serving a file some.file to a client that has included the Accept-Encoding: gzip, deflate in the request headers, will first look if the file some.file.gz exists. If it does, then the file some.file.gz is returned in the response along a Content-Encoding: gzip,deflate response header. If the file does not exist, then some.file will be returned instead.
The assets library just creates the file for your, but how it used depends on the browser and the webserver.
More info: https://en.wikipedia.org/wiki/HTTP_compression


Feature Request: Independently Scrolling "Card View" with Unlimited Cards

It would be nice if the user could scroll the "Card View" without scrolling the whole page as well as being able to increase the maximum number of displayed cards in the "Card View" to minimize the number of pages (or allow all cards to be on a single scrolling page).

That is something which I had at the beginning and would add again if I would have more funding, or a sponsor :)
95% of the server cost is coming from loading images for the users, so I try to keep the cards that are loaded to a minimum.
When the current user-pool grows and the site has a bigger audience cardmarket considers sponsoring the site, then i can add this.


Value in PLN is incorrect

I have 139,18 PLN which is 37,26 USD (around 0,1237 ETH)

Other thing is that we should't have a comma before decimal fractions for PLN value where the ETH has a period before decimal fractions 0.1237 ETH = 37,26 USD. One or another depending if this is Australia, UK, US, most of Asia (period) OR European/South American country.
https://en.wikipedia.org/wiki/Decimal_separator#/media/File:DecimalSeparator.svg

@tomekalphawallet The API gets the values in USD only so far

@James-Sangalli on iOS it displays me PLN for an unknown reason


Fixed DoS vulnerability in graphql-go package

Currently there is DoS vulnerability in graphql-go package. This has been fixed in the latest release v1.3.0 of the package
I would like to upgrade the package to latest version in this PR.
Link to vulnerability

Codecov Report

Merging #133 (fa04c6e) into main (13268f2) will increase coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #133      +/-   ##
==========================================
+ Coverage   69.51%   69.55%   +0.03%
==========================================
  Files          24       24
  Lines        2624     2624
==========================================
+ Hits         1824     1825       +1
+ Misses        669      668       -1
  Partials      131      131

Impacted Files
Coverage Δ

auth.go
87.42% <0.00%> (+0.62%)
: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 13268f2...fa04c6e. Read the comment docs.

Thanks @azadasanali
Dependabot thought the vulnerability was fixed but the commit it is pointing to is from 2020 so I'm not sure how that counts as being up to date.


[HOW-TO] Reduce Rolling Shutter with HQ-Camera

We want to use the HQ-Camera attached to a slightly moving object.
Currently we're facing blurry parts in the image, due to rolling shutter of the camera.
How can we reduce this effect as good as possible, operating with a RPi 3A+?
Is the ExposureTime in metadata the only value we have to consider to reduce?
Hope to find some tipps on how to make the most out of it :)
Our Code:
# Limit ExposureTime to reduce blur due to GlobalShutter by daytime
if (timeofday == "day"):
   AeExposureMode = controls.AeExposureModeEnum.Short
   FrameDurationLimits = (50, 5000)
else:
   AeExposureMode = controls.AeExposureModeEnum.Long
   FrameDurationLimits = (100, 10000000)

with Picamera2() as picam2:
   picam2.start(show_preview=False)

   picam2.set_controls({
      "AeFlickerMode": controls.AeFlickereModeEnum.FlickerOff,
      "AeMeteringMode": controls.AeMeteringModeEnum.Custom if metering else controls.AeMeteringModeEnum.Spot,
      "AeExposureMode": AeExposureMode,
      "FrameDurationLimits": FrameDurationLimits
   })

   # auto exposure might take a few moments settle
   time.sleep(0.5)

   ctrl={}
   capture_config = picam2.create_still_configuration(ctrl)
   time.sleep(2)

   metadata = picam2.capture_metadata()

   picam2.switch_mode_and_capture_file(capture_config, "file.jpg")

   picam2.stop()

Yes, I think the exposure time is the only thing that will affect the blurriness of the final image. You can keep this short by setting it directly (and the analogue gain will adapt according to the exposure time chosen), or by selecting an appropriate exposure mode.
I notice that you've chosen the "short" exposure mode which is an appropriate thing to do. If you wanted, you are free to edit the parameters of the "short" exposure mode in the camera tuning file (search for imx296.json on the Pi's filesystem).
FrameDuration can limit the exposure time, so it's a reasonable way to constrain what the AE algorithm will do, but won't directly affect blurriness (except by forcing a different exposure time to be used).

For a deeper understanding of what is happening: Does it make a difference if we use a faster RaspberryPi? Or is the speed limited by the camera sensor?

No. It's all down to the camera sensor. I think the only factor is the exposure time.
You mentioned that you're using the HQ camera. I just wanted to add that the HQ cam is not a "global shutter camera". It employs a "rolling shutter", like most of our cameras (except the "Global Shutter Camera" that we also sell). The rolling shutter can have an effect on an image, in that the top of the image is not exposed at the same moment as the bottom, so you can get some strange motion artefacts. However, the blurriness of any part of the image is not affected by when the exposure happened, only how long it was.

I'm so sorry, I've been talking about rolling shutter and wrote global shutter in the beginning... I changed the title here for anyone searching for this...
Thanks for the informations! I'll give it a try and report back :)

The exposure time in your metadata is extremely short, so I don't think you can improve that. The capture should have the same exposure time, but of course you may be subject to increased rolling shutter effects. I guess I'm a bit confused about what the problem really is - motion blur, rolling shutter effects, or lens problems? Could you maybe clarify that?
You can certainly capture a stream of images, there's no problem with that. Though if single images are showing a problem, then you may end up with a stream of images with the same problem. Capturing a stream of images is usually limited by the time taken to write them out. The HQ cam can run up to 10fps in the full resolution mode, though to achieve that you need fast SSDs - a micro SD card is unlikely to keep up.

Here's an example
If you click the last day in the timeline at the bottom and hover over those images you can see, that little movements of the crane cause parts of the image to be blurry and distorted.
I'd except capture time to be less than 100ms, but then this seems to be too much of distortion...

Okay you nailed it to the lens and you're right. It seems to be just a little little bit loose! And there seems to be some erruption, combined with the rolling shutter to make this effect.
I'm sorry I was on the wrong track. Thank you so much for your time and the final hint!


Add optional parameter support.

Optional parameters crash it.

Hello michael, excuse my late response. This project was a playground for me to learn the inner workings of parsing & compiler technology and I never planned on supporting it. As of now I can't afford to provide adequate support even if I'd like to.


Multiple Runs Result in IndexError

I found that running Melissa multiple times results in IndexError. Here's the traceback that I received:
Traceback (most recent call last):
  File "main.py", line 7, in <module>
    main()
  File "main.py", line 5, in main
    stt()
  File "/Users/tanay/Desktop/Melissa-Core/melissa/stt.py", line 39, in stt
    brain.query(speech_text)
  File "/Users/tanay/Desktop/Melissa-Core/melissa/brain.py", line 81, in query
    scoring_row['group'][scoring_row['order']]:
IndexError: list index out of range

@neilnelson ping.

Unable to reproduce again.


get env vars from file instead of redefining in script

alternate to #122
this approach would allow us to change values in /etc/environment and re-run the deploy script instead of needing to keep env vars in the deploy script up to date

Ignoring the snyk check for now. This was tested on the test server with ...


Wrong GET endpoint in "get web template by template id" function

Test Case/s To Reproduce Issue
# path to test case
tests/robot/_resources/keywords/composition_keywords.robot:657

# robot command to execute related test case(s) in your terminal/console

robot -v SUT:EHRSCAPE -d results/composition -e future -e circleci -e TODO -e obsolete -e libtest -L TRACE --noncritical not-ready --name COMPO robot/COMPOSITION_TESTS

# Valid test suite folder names
COMPOSITION_TESTS

Actual Result
GET Response : url=http://localhost:8082/rest/openehr/v1/template/genericFHIRTemplate
status=404, reason=Not Found
headers={'Cache-Control': 'must-revalidate,no-cache,no-store', 'Content-Type': 'application/json', 'Content-Length': '118'}
body={
"servlet":"default",
"message":"Not Found",
"url":"/rest/openehr/v1/template/genericFHIRTemplate",
"status":"404"
}
Expected Result
GET Response : url=http://localhost:8082/rest/openehr/v1/definition/template/adl1.4/genericFHIRTemplate
status=200, reason=OK
...

In openEHR specification:
https://specifications.openehr.org/releases/ITS-REST/latest/definitions.html#definitions-adl-1.4-template-get-1
you can see the GET endpoint is different from the one in "get web template by template id" function.

Hello @gregorberger,
As per my analysis, seems to be not a bug, as I executed all test cases under COMPOSITION_TESTS folder and found this one:

This is actually the endpoint for ECIS.
I will rename this keyword to contain ECIS name, as this is confusing.
I see that the expected result in covered in keyword, retrieve OPT by template_id, located under template_opt1.4_keywords.robot

Please let me know if you still face any other issues.
CC: @stefanspiska

@gregorberger - Execution was done using the command you pasted in above.
robot -v SUT:EHRSCAPE -d results/composition -e future -e circleci -e TODO -e obsolete -e libtest -L TRACE --noncritical not-ready --name COMPO robot/COMPOSITION_TESTS
Results can be found below.
I see no errors there.
In case you still face any issues, just let me know the test case or the .robot file with test case, where the error is reproducible.
compositionResults.zip

@vladislavploaia thank you for your response.


mention which curve simpleEcKeyRing is using + consider changing to Curve25519

    /**
     * Creates a key ring consisting of an ECDSA master key and an ECDH sub-key.
     * The ECDSA master key is used for signing messages and certifying the sub key.
     * The ECDH sub-key is used for encryption of messages.
     *
     * @param userId user-id
     * @param password Password of the private key. Can be null for an unencrypted key.
     *
     * @return {@link PGPSecretKeyRing} containing the key pairs.
     *
     * @throws PGPException
     * @throws NoSuchAlgorithmException
     * @throws InvalidAlgorithmParameterException
     */
    public PGPKeyRing simpleEcKeyRing(@Nonnull String userId, String password)

I think the comment should mention this. I digged in the code and it seems that P256 is the chosen curve.
Separately, would it be ok to settle on Curve25519 as the default one?

Yeah this should be fixed and I guess curve25519 would be indeed a good candidate for the default curve.

Yeah this should be fixed and I guess curve25519 would be indeed a good candidate for the default curve.

Excellent - thanks!

Excellent - thanks!

However there seems to be just one curve - does this still need to be implemented?
public enum EllipticCurve {
    _P256("P-256"),
    ;

    private final String name;

    EllipticCurve(@Nonnull String name) {
        this.name = name;
    }

    public String getName() {
        return name;
    }
}

However there seems to be just one curve - does this still need to be implemented?
public enum EllipticCurve {
    _P256("P-256"),
    ;

    private final String name;

    EllipticCurve(@Nonnull String name) {
        this.name = name;
    }

    public String getName() {
        return name;
    }
}

I do see master has more but not yet curve25519

public enum EllipticCurve {
    _P256("P-256"),
    _P384("P-384"),
    _P521("P-521"),
    ;

    private final String name;

    EllipticCurve(@Nonnull String name) {
        this.name = name;
    }

    public String getName() {
        return name;
    }
}

I do see master has more but not yet curve25519

public enum EllipticCurve {
    _P256("P-256"),
    _P384("P-384"),
    _P521("P-521"),
    ;

    private final String name;

    EllipticCurve(@Nonnull String name) {
        this.name = name;
    }

    public String getName() {
        return name;
    }
}

I might be wrong, but I read that curve25519 for signing is known as ed25519 (which is found here on current master) and for encryption as XDH (which is found here).
I planned ti interview viktor about this soon :D

I might be wrong, but I read that curve25519 for signing is known as ed25519 (which is found here on current master) and for encryption as XDH (which is found here).
I planned ti interview viktor about this soon :D

Let's ping @wiktor-k in case he has thoughts

Let's ping @wiktor-k in case he has thoughts

What would the keygen parameters then look like? Is it something like this?
I'm also not too familiar with this terminology, just trying to get an idea of what it will look like.
// encryption
KeyType.EDDSA(EllipticCurve._25519)

// signing
KeyType.XDH(EllipticCurve._25519)

What would the keygen parameters then look like? Is it something like this?
I'm also not too familiar with this terminology, just trying to get an idea of what it will look like.
// encryption
KeyType.EDDSA(EllipticCurve._25519)

// signing
KeyType.XDH(EllipticCurve._25519)

It would be like
// signing
KeyType.EDDSA(EdDSACurve.ED25519)
// encryption
KeyType.XDH(XDHCurve.X25519)

A real example can be found here

It would be like
// signing
KeyType.EDDSA(EdDSACurve.ED25519)
// encryption
KeyType.XDH(XDHCurve.X25519)

A real example can be found here

I see. That means that curve25519 support seems to just be awaiting a release. Then once we are sure that this is the right setup, we could switch it to the default.
Is the master ready for a release that includes this functionality? A relase with this to experiment with would be very helpful :)

I see. That means that curve25519 support seems to just be awaiting a release. Then once we are sure that this is the right setup, we could switch it to the default.
Is the master ready for a release that includes this functionality? A relase with this to experiment with would be very helpful :)

The master should be more or less ready to go. I will try to create a release until the end of the week, however I can give no guarantee as I'm currently busy with sad family events :/

The master should be more or less ready to go. I will try to create a release until the end of the week, however I can give no guarantee as I'm currently busy with sad family events :/

I'm very sorry to hear that - I send my best wishes your way.
No worries, code can wait. Family first
On Tue, 12 Jan 2021, 18:44 Paul Schaub<EMAIL_ADDRESS>wrote:

The master should be more or less ready to go. I will try to create a
release until the end of the week, however I can give no guarantee as I'm
currently busy with sad family events :/
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/pgpainless/pgpainless/issues/46#issuecomment-758825953,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABQDZEJWANLWZEXJPKWUFGLSZSDABANCNFSM4V7J4LTA
.

I'm very sorry to hear that - I send my best wishes your way.
No worries, code can wait. Family first
On Tue, 12 Jan 2021, 18:44 Paul Schaub<EMAIL_ADDRESS>wrote:

The master should be more or less ready to go. I will try to create a
release until the end of the week, however I can give no guarantee as I'm
currently busy with sad family events :/
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/pgpainless/pgpainless/issues/46#issuecomment-758825953,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABQDZEJWANLWZEXJPKWUFGLSZSDABANCNFSM4V7J4LTA
.

Looks good @tomholub and @vanitasvitae :+1:
I see on the horizon that curve 448 (goldilocks, Ed448) is being considered for inclusion in OpenPGP and on first sight it seems it would fit your design well Paul so that's nice :)

Looks good @tomholub and @vanitasvitae :+1:
I see on the horizon that curve 448 (goldilocks, Ed448) is being considered for inclusion in OpenPGP and on first sight it seems it would fit your design well Paul so that's nice :)

I just published version 0.2.0-alpha5 :)

I just published version 0.2.0-alpha5 :)

Excellent, thank you!

Excellent, thank you!

I switched the simpleEcKeyRing() methods over to use curve25519 instead of p256.
Additionally there is now modernKeyRing() which generates a key ring consisting of an ed25519 EdDSA primary key used only to certify another ed25519 EdDSA signing subkey and an X25519 XDH encryption subkey.
I'm not yet 100% satisfied with the name, so if you have any suggestions, let me know :)


⚠ Bofrost.lu has degraded performance

In ca5e34a, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1572 ms

Resolved: Bofrost.lu performance has improved in aa9158d after 6 minutes.