Removed iterative from walsh

Closes #385

Do you use pre-commit?
If not yet, this is how to:
do a pip install -e ".[test]" in the  directory of your local clone of mrpro.
This will install "pre-commit", a tool that runs ruff, mypy etc.
Now you can call pre-commit run --all-files to check the files, let it apply some automatic formatting. Maybe you need to change some things manually, such that pre-commit does not raise any issues anymore. This would solve the "pre-commit" action failing.

If you use vscode, you can  use the "test" tab on the right left to see all tests and run them.
All of them, especially those related to your change, should pass.
Currently, there is an issue with the
 from tests.algorithms.csm.test_iterative_walsh import multi_coil_image
E   ModuleNotFoundError: No module named 'tests.algorithms.csm.test_iterative_walsh'

as you forgot the change the name there :)
ps: if you already know all these things, ignore my comments.
pps: if you did not know them and need more help -> slack.

Good job getting your first change into mrpro! 🎉
One final tiny remark: Git commit messages are mostly in imperative mode.
So, the final merged git commit should have been "Remove 'iterative' from walsh' -- not "Removed ... "
Not really important, just a small detail for the future :)
@bernharb


Check assignments

we can check history of checks made for a given Project & Reminder combination
we can now assign person to be responsible for a given check
cleanup in slim templates
updated  rubocop settings

I would say that we need a migration for adding a history object based on ProjectCheck and last_check_user_id & last_check_date fields

thanks @angouleme ! great job! :+1:


Spark raise_error function support

This support for spark raise_error function
document: https://spark.apache.org/docs/latest/api/sql/#raise_error
spark code https://github.com/apache/spark/blob/branch-3.5/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/misc.scala#L85C1-L92C1
raise_error also been re-used in spark assert_true funciton, so after the PR,  assert_true would also support to offload.
issue https://github.com/apache/incubator-gluten/issues/5991

@rui-mo / @PHILO-HE help review

ping @rui-mo

I need @rui-mo for some input, @PHILO-HE  please ignore my review refresh request before i update the code

Could you run the fuzzer test for this function to make sure it works? Thanks.

Could you run the fuzzer test for this function to make sure it works? Thanks.

Could you run the fuzzer test for this function to make sure it works? Thanks.

I have verified locally end to end with gluten, @rui-mo  what do you means for run fuzzer test for this function , does i need add a new fuzzer test or something , any reference ?

@gaoyangxiaozhu The command is like below, and you could use --only raise_error to test this function only.
https://github.com/facebookincubator/velox/blob/main/.github/workflows/scheduled.yml#L508-L512

--only raise_error

got! so it should be fine , a example local fuzzer run

@rui-mo any new comments ?

@gaoyangxiaozhu Could you try the fuzzer test again to make sure it works? Thanks.

@gaoyangxiaozhu Could you try the fuzzer test again to make sure it works? Thanks.
done.

ping @rui-mo / @PHILO-HE  for follow up if has any new comments ？

@gaoyangxiaozhu I assume we need to rebase to https://github.com/facebookincubator/velox/pull/10274 and returns status in raise_error function after it gets merged. Thanks.

@gaoyangxiaozhu I assume we need to rebase to #10274 and returns status in raise_error function after it gets merged. Thanks.

sure, so could you help sign the PR or we need people from meta to review and help sign ?

@gaoyangxiaozhu Will continue to review this PR after the code is rebased to #10274. Thanks.

@gaoyangxiaozhu https://github.com/facebookincubator/velox/pull/10274 has been merged. Could you rebase to it and update the code? Thanks.

@gaoyangxiaozhu #10274 has been merged. Could you rebase to it and update the code? Thanks.

Done, and also local test with fuzze test. should be fine. @rui-mo


ChoicePrompt handes null locales

Fixes: #2942
I'm waiting on CX to confirm the issue, but I did some troubleshooting and intentionally setting Activity.Locale to null does indeed break the same way as in that issue.
Changes

ChoicePrompt: Ensures that locale defaults to English and handles mapping to the nearest locale more gracefully
PromptCultureModels: Implements our own version of Recognizers-Text's MapToNearestLanguage -- They added a bunch of other supported languages that will break this without doing this.
ChoicePromptTests: Added a test to specifically set the Locale to null

Other Notes

ConfirmPrompt already handled this fine and had tests for it
Currently working on JS implementation
I'll also work on some tests for PromptCultureModels, but I figured this should get out the door, first.
Since Recognizers-Text added a bunch of supported languages, I'll see about adding them here in a future PR

Pull Request Test Coverage Report for Build 89132

0 of 0   changed or added relevant lines in 0 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.2%) to 76.637%

Files with Coverage Reduction
New Missed Lines
%

/libraries/Microsoft.Bot.Builder.Dialogs/Prompts/PromptCultureModels.cs
4
96.08%

Totals

Change from base Build 88872:
-0.2%

Covered Lines:
4366

Relevant Lines:
5697

💛  - Coveralls

:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.Luis.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.QnA.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.ApplicationInsights.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Azure.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Dialogs.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.TemplateManager.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Configuration.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Connector.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Schema.dll compared against version 4.5.3

:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.Luis.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.QnA.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.ApplicationInsights.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Azure.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Dialogs.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.TemplateManager.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Configuration.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Connector.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Schema.dll compared against version 4.5.3


Custom DOCVQA training

How can i train this model for VQA suing my custom document image dataset. Is there any any training script available?

You may check this link:
https://github.com/NielsRogge/Transformers-Tutorials/tree/master/UDOP
Thanks to @NielsRogge for creating this.


Upgrade autorest.csharp to net7.0

https://dev.azure.com/azure-sdk/internal/_build/results?buildId=3268492&view=logs&j=110ffb37-f6b6-5d2e-4482-cee28f4a1c07&t=c1e6de2a-7bc7-5f8d-32b9-8690377ad59f

i am curious that why when there is net8.0 released and we update to net 7.0? could we just directly bump to net8.0?

i am curious why when there is net8.0 released and we update to net 7.0? could we just directly bump to net8.0?

@ArcturusZhang azure-sdk-for-net adopts up to net7.0. So better to align with that.

Making this change here https://github.com/Azure/autorest.csharp/pull/4055 so closing this PR.


python311Packages.losant-rest: 1.19.1 -> 1.19.2

Diff: https://github.com/Losant/losant-rest-python/compare/v1.19.1...v1.19.2
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 266426 run on x86_64-linux 1

4 packages built:

python310Packages.losant-rest
python310Packages.losant-rest.dist
python311Packages.losant-rest
python311Packages.losant-rest.dist


create-microcosm-app

Yarn has recently added a new feature: yarn create. It can be used to do some pretty neat stuff:
https://yarnpkg.com/blog/2017/05/12/introducing-yarn/
Let's make a create-microcosm-app or just create-microcosm to help people get started right away. Instead of cloning in the quick start, what if we said:

After yarn finishes installing, create a microcosm:
yarn create microcosm

absolutely 👍

Threw this together today leaning on some work from create-react-app: https://github.com/vigetlabs/create-microcosm
A lot of work to still be done, but it works with yarn create microcosm project-name. We can move conversation to that repo for next steps (of which there are a lot...)


Unclear and redundant statement regarding environment variable definition prior to Deployment creation

Before creating a Deployment define an environment variable for a container.

What is the purpose of this statement here. We are able to create a Deployment from sample provided below it without executing above statemet.

Page related to issue: https://kubernetes.io/docs/concepts/workloads/controllers/deployment/
/language en

/retitle Unclear and redundant statement regarding environment variable definition prior to Deployment creation

It appears that the statement in question was included as a solution to address issue #40735. To eliminate confusion and improve clarity, one possible option could be to move the link to the "What's Next" section at the end, thereby reducing confusion with the existing content on the page. I will allow others to share their thoughts on this matter.

Thanks Dipesh. Appreciate it.
Regards
Shridhar
On Fri, Jun 23, 2023 at 2:22 AM Dipesh Rawat @.***>
wrote:

It appears that the statement in question was included as a solution to
address issue #40735 https://github.com/kubernetes/website/issues/40735.
To eliminate confusion and improve clarity, one possible option could be to
move the link to the "What's Next" section
https://kubernetes.io/docs/concepts/workloads/controllers/deployment/#what-s-next
at the end, thereby reducing confusion with the existing content on the
page. I will allow others to share their thoughts on this matter.
—
Reply to this email directly, view it on GitHub
https://github.com/kubernetes/website/issues/41735#issuecomment-1603299966,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BAN57GEBFHVEWD5CWWOXCH3XMSWCVANCNFSM6AAAAAAZQTZXLY
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hi.. I'd like to work on the issue.. Can you please elaborate on what could be the fix?

I recommend removing

Before creating a Deployment define an environment variable for a container.

from the page. That's a small fix that doesn't need us to work on the rest of https://github.com/kubernetes/website/issues/12082
/triage accepted
/language en
/priority backlog

/assign

As PR #42155 fixes the issue, so we can close it


Update harness-self-managed-enterprise-edition-overview.md

Added the FG and NG support for CD.
Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[ ] Feature
[ ] Maintenance/Chore

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

Issue(s)

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

[ ] Tested Locally
[ ] Optional Screen Shoot.

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

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


Space between bullet and text inside table inside figure

Description
A picture says more than 1000 words. I believe the picture is fully descriptive, let me know if you need any details.

The final list, inside a table inside a figure, has a weird space between the bullets and the bullet content. Is this ever desired?
Reproduction URL
No response
Operating system
Linux
Typst version

[X] I am using the latest version of Typst

Seems to be because it's centered. Seems related to what I fix in #1034. Use table(align: start) (or align: left) to override.
Perhaps align: start should be made the default for tables (instead of inheriting from the context)?

Thanks. With set table(align: horizon + start), the tables look great.
I will keep this issue open, as the whitespace introduces would be much better after the text, than between the text and bullet.

#set table(align: horizon + start), as suggested above, works fine.
But then why this is not something default? What are the pitfalls, drawbacks?


Support for selection based on row clicks

It would be great to support multi selection by clicking on rows instead of checkboxes as an alternative way of selecting items. Selecting by using keyboard and presing space would be also desirable.

Both are actually missing features in Grid (https://github.com/vaadin/vaadin-grid-flow/issues/196, https://github.com/vaadin/vaadin-grid-flow/issues/908 ) and both provide a good UX.
If we are going to implement any of them in the addon, it makes sense to track its progress as separate issues.

Good idea, I've created #13 and #14 for continuing with this. Closing this one.


GNOME 42: On-screen keyboard broken

Describe the bug
In GNOME 42 in Tablet mode the on-screen keyboard (OSK) is very broken. On Gnome's own UI it shows up but was not reacting to touches (that was https://gitlab.gnome.org/GNOME/gnome-shell/-/issues/5273, https://github.com/NixOS/nixpkgs/pull/169429), but on normal applications (tried GTK3, GTK4 and QT5) it doesn't even show up. When manually enabling it through accessibility options it doesn't show up either. The gesture is broken upstream (https://gitlab.gnome.org/GNOME/gnome-shell/-/issues/5335), but it not working at all is AFAIK a NixOS issue.
This should really be fixed for 22.05 as otherwise this release is basically unusable on 2-in-1s and tablets.
Steps To Reproduce
Steps to reproduce the behavior:

Use a Tablet or a 2-in-1 (with SW_TABLET_MODE working) in tablet mode
Open a GTK3 (known-working) application and tap into a text field
Use the swipe gesture to pull up the text field

Expected behavior
The On-screen keyboard should appear and take touch input.
Notify maintainers
@bobby285271
@hedning
@jtojnar
@dasj19
@maxeaubrey
Metadata
Not currently on the machine, was running nixos-unstable from ~7 days ago with my backport for the first issue.

GTK 3 might also be affected by #172406.

Still doesn't explain why it doesn't even come up by manually enabling it from the accessibility settings.

So for me it maybe works in the session manager, but sometimes I have to enable it in the accessibility settings. Once in gnome, it works for anything gnome related, but I can't swipe up to bring it up (and I believe the other gestures haven't worked for me as well). It doesn't work in any QT5 app I have (kate, dolphin, konsole), but it does work in Firefox and Thunderbird provided I export MOZ_USE_XINPUT2=1 MOZ_ENABLE_WAYLAND=1. Xinput will allow firefox to respond to touch gestures (pinch to zoom) and enable wayland will allow the keyboard to appear for almost all text boxes within firefox (but it has issues with extensions: bitwarden).
Sometimes even in the gnome apps, and firefox, I have to tap the text box a few times until the keyboard appears, but as a 2-in-1 user myself, the lack of OSK makes gnome unusable with any programs that aren't provided by gnome in tablet mode.
Finally, exporting QT_QPA_PLATFORM=wayland in environments nor commandline seems to have an affect on the KDE QT5 apps.

Has long been fixed in GNOME 42.2 with https://github.com/NixOS/nixpkgs/pull/175424.

Seems it's broken again now in gnome 46 now, Exactly the same issue.

That will be a different issue then, since this was resolved upstream.


OpenGL ES 3.0+ requires integer signed/unsigned 2d textures...

... But glium only checks for OpenGL 3.0+, and thus thinks they're unsupported.

Actually, now that I've patched Glium locally, it now seems DepthFormat ignores GLES too.


fix(prepare): not handle dev dependencies

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Remove path prefix only if $path contains prefix

I am testing a webdav implementation with Nginx on Debian 10. I have encountered an error trying to list the contents of a directory. The problem is that in the listContents function it removes as many characters as the prefix of the path, even if the path returned by the webdav server does not contain the prefix. This causes the necessary part of the returned file paths to be removed.
In my case the following happened:
Prefix: http://<IP_ADDRESS>:20000/
Route returned by webdav: /uploads/user/8546/2020-06-12
Result: 06-12

Looks good to me :+1:
And if I understand https://tools.ietf.org/html/rfc4918#section-8.3 correct relative urls are allowed. Would be nice to have a test case for it ;)


Refactor "active" property

The active property is currently unused.
Refactor to either make use of property of remove.

Not particularly related to the demo.

This was part of GameWorldObject.
GameWorldObjects are long extinct.


BUG - Templates Not Included in files Array of fiori-generator-shared Package json

Related Feature
#2019
Description
Templates Not Included in files Array of fiori-generator-shared Package. Currently, the fiori-generator-shared package is missing the inclusion of templates in its files array within the package.json. This omission prevents the templates from being included when the package is published or used as a dependency, which affects the usability of the package.
Expected results
The files array in package.json should include the templates directory to ensure that all necessary templates are included when the package is installed.
Actual results
The templates directory is missing from the files array, causing templates to be excluded from installation.

Fix provided in https://github.com/SAP/open-ux-tools/pull/2113
Read me templates available in published npm version.


Fixed an issue where opening files wouldn't work when the directory was sorted by size or by file type

DefaultQuery doesn't work because it's always overwritten(?) with null property* from sortEntry.PropertyName.
Might be necessary to write custom replacement in the future to achieve full functionality.
*(or just adding null causes problems)
Fixes #2475

Should solve #2475 issue.

@Lubieerror Great job!


Failed assertion: boolean expression must not be null

Failed assertion: boolean expression must not be null
[SEVERE] flutter_data:data_repository_builder on lib/mypath/user_model.dart  (cached)::

sdk: ">=2.11.0 <3.0.0"
My build_runner process is failing, I believe that was because I ran dart migrate --skip-import-check
Do you have some advice how to solve it?
Thanks

Are you trying to migrate to null-safety?

I was trying to,
Running dart migrate --skip-import-check
Changing sdk target
But I rollback and now I'm getting this error

Please follow #90 for developments on null safety. I will start that in about 2 weeks.


Time zone

Hi, first of all thank you for this great gaming solution :) Second, I want to set my timezone correctly. Currently it is showing like this:

which is two hours late compared to my local time.
How can I change time zone (Europe/Poland) using docker compose file ?

Timezone should be as simple as setting the TZ env variable, ex:
docker run -e TZ=Europe/Warsaw --rm -it --entrypoint date ghcr.io/games-on-whales/base-app:edge
Wed Oct  9 21:21:56 CEST 2024

You'll have to set this additional env variable in all the apps defined in your config.toml, see the docs linked below on how to do it.
The us there in your screenshot refers to the keyboard layout instead, you can see how to change that in the docs.
Feel free to re-open if this doesn't work!

Hi, thanks for quick reply. I followed your instructions but it worked half part. Below is my definition of Steam application with added TZ env var and screenshot when it was running with opened shell window.

As you see shell is recognizing changed time zone but Sway is not. I'm guessing TZ env variable is not passed to Sway correctly.
[[apps]]
start_virtual_compositor = true
title = "Steam"

[apps.runner]
base_create_json = """
{
  "HostConfig": {
    "IpcMode": "host",
    "CapAdd": ["SYS_ADMIN", "SYS_NICE", "SYS_PTRACE", "NET_RAW", "MKNOD", "NET_ADMIN"],
    "SecurityOpt": ["seccomp=unconfined", "apparmor=unconfined"],
    "Ulimits": [{"Name":"nofile", "Hard":10240, "Soft":10240}],
    "Privileged": false,
    "DeviceCgroupRules": ["c 13:* rmw", "c 244:* rmw"]
  }
}
"""
ports = []
devices = []
env = [
  "PROTON_LOG=1",
  "TZ=Europe/Warsaw",
  "RUN_SWAY=true",
  "GOW_REQUIRED_DEVICES=/dev/input/* /dev/dri/* /dev/nvidia*",
]
mounts = []
image = "ghcr.io/games-on-whales/steam:edge"
name = "WolfSteam"
type = "docker"

@ABeltramo Cannot reopen this issue. I see only comment button.

Sorry about that, I'll look into it. We are probably missing a call to tzselect somewhere..

This turned out to be a specific issue with Waybar clock, I've applied this workaround for now.
You should be able to test it out by just updating the images using:
docker pull ghcr.io/games-on-whales/steam:edge


Add safeArray.

Useful when you hate undefined and you just want an empty array. :)
Anything other than null or undefined gets wrapped in an array and returned that way. It makes sense to me that if I'm calling .safeArray, I expect it to always return an array AND I don't want to lose the value of the path just because it happens to be a different type initially.

@mlkirkpatrick Thanks for this PR. Looks good. I've merged the code already. Will run some local checks and release a new version to npm sometime this week or the next.

v3.1.0 published to npm with the changes. Thanks again. I have added you to the contributors list.


Add support for zalando platform IAM

Our default authorization provider ZignAccessTokenProvider.java is outdated and the usage of zign is not recomended anymore by default.
In this PR I have introduced support for Zalando's platform IAM and we could replace ZignAccessTokenProvider with this new integration.
See https://github.com/zalando-nakadi/fahrschein/issues/381

Codecov Report
Base: 66.94% // Head: 67.05% // Increases project coverage by +0.11% :tada:

Coverage data is based on head (438cacf) compared to base (60b45ce).
Patch coverage: 66.66% of modified lines in pull request are covered.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff              @@
##               main     #382      +/-   ##
============================================
+ Coverage     66.94%   67.05%   +0.11%
- Complexity      403      407       +4
============================================
  Files            74       75       +1
  Lines          1688     1700      +12
  Branches        147      148       +1
============================================
+ Hits           1130     1140      +10
- Misses          454      457       +3
+ Partials        104      103       -1

Flag
Coverage Δ

unittests
67.05% <66.66%> (+0.11%)
:arrow_up:

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

Impacted Files
Coverage Δ

...rg/zalando/fahrschein/ZignAccessTokenProvider.java
0.00% <ø> (ø)

...alando/fahrschein/PlatformAccessTokenProvider.java
66.66% <66.66%> (ø)

...ain/java/org/zalando/fahrschein/Preconditions.java
18.51% <0.00%> (+7.40%)
:arrow_up:

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

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

(Addressed review comments by @otrosien from https://github.com/zalando-nakadi/fahrschein/pull/380)

👍

👍


Add a Market-Making section to the sidebar overviewing Opportunities, Key Concepts, and Resources

Please see here for a Notion page that details what this should look like. Note that this should be a new sidebar section, called "Market Making", with three sub-pages, Opportunities, Key Concepts, and Resources. Note also that hyper links should be added throughout, where necessary.

Fixed in #36


Revamp the about us page

The about us page currently looks out of place. I would like to revamp it, add navbar, footer and change the body section to sync it with the website theme

@JigarJoshi04 Please assign this

someone already working on it so at this time I can't assign it to you


Bump pandas from 1.2.3 to 1.2.4

Currently getting these warnings whenever Pandas calls SQL library functions
=============================== warnings summary ===============================
repos/delphi/delphi-epidata/integrations/acquisition/covidcast_nowcast/test_csv_uploading.py::CsvUploadingTests::test_duplicate_row
repos/delphi/delphi-epidata/integrations/acquisition/covidcast_nowcast/test_csv_uploading.py::CsvUploadingTests::test_duplicate_row
repos/delphi/delphi-epidata/integrations/acquisition/covidcast_nowcast/test_csv_uploading.py::CsvUploadingTests::test_uploading
  /usr/local/lib/python3.8/site-packages/pandas/io/sql.py:1429: SADeprecationWarning: The Engine.run_callable() method is deprecated and will be removed in a future release.  Use the Engine.begin() context manager instead. (deprecated since: 1.4)
    return self.connectable.run_callable(

Bumping to pandas==1.2.4 removes these warnings.

Oops this is a duplicate of #992.


Fix 404.html to be deployed via Netlify

Describe the bug
The 404.html is currently WIP.
Only index.md is successfully being deployed. Trying the 404.html will direct to default Netlify 404 instead of the designated one.
To Reproduce
Steps to reproduce the behavior:

Go to 'https://otp-verification-system.netlify.app/404'
See error 404 shown is the default version by Netlify.

Expected behavior
It is expected that the designated 404.html is shown when wrongly type the website address.
Screenshots
This is Netlify default 404.

Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
This issue is intended to be beginner friendly.
Please find the possible solution in this link for implementation.

Hi, To overwrite the default 404 error page provided by netlify, create a netlify.toml file in the root directory and add below lines of code. refer here
[[redirects]]
  from = "/*"
  to = "/something.html"
  status = 404


Do not populate resource-related fields from span attributes in Recei…

…veResourceSpansV2

What does this PR do?
JIRA link
In ReceiveResourceSpansV2, we want to deprecate that resource-related fields are only populated from resource attributes, not span attributes. The fields may be copied to "Meta", but will not be used in the top-level fields.
Ensure that the following keys are not used from span attributes:

datadog.host.name
container.id
k8s.pod.id
env
service.name

Add unit tests for this (see in particular the last test case in testOTelSpanToDDSpan).
Once this PR is in the nightly build, I'll flesh out receive_resource_spans_v2_test E2E tests to test all the above attributes in a follow-up PR.
Motivation
Describe how to test/QA your changes
Possible Drawbacks / Trade-offs
Additional Notes

/merge

/merge

/remove

/merge


Updating to use the new GET call for obtaining mater key

Updating to use the new GET api instead of the POST that is creating race conditions when querying for a  new masterKey

this PR will have to wait until the runtime is deployed, or it'll completely break next portal until that happens.

Yes i will wait for the runtime changes to be deployed before i push this - i just wanted to get the PR started.


How to configure the resources to start with the server

I never understood how the MOD to run

mtaserver
<resource src="?" startup="1" protected="0" /

Assuming you have the Multi Theft Auto server software available there are two ways to start the mod, first you need to copy all the folders into mods/deathmatch/resources/ (from server root), then you start the server. (Use /refresh if it's already running). And then "/start NameOfResource". What you have there is the file "mtaserver.conf" in mods/deathmatch/ allowing you to set up which resources should start together with the server.
All you need to add in the src property is the name of the resources you would like to start, that reminds me that we should make an installation guide as it takes some time writing all those config lines for this whole project. For now you may use this:
`

<!-- GTW resources -->
<resource src="GTWantispam" startup="1" protected="0" />
<resource src="GTWchat" startup="1" protected="0" />
<resource src="GTWfarmer" startup="1" protected="0" />
<resource src="GTWfastfood" startup="1" protected="0" />
<resource src="GTWfisher" startup="1" protected="0" />
<resource src="GTWhelp" startup="1" protected="0" />
<resource src="GTWmechanic" startup="1" protected="0" />
<resource src="GTWphone" startup="1" protected="0" />
<resource src="GTWplayerblips" startup="1" protected="0" />
<resource src="GTWrob" startup="1" protected="0" />
<resource src="GTWsmoke" startup="1" protected="0" />
<resource src="GTWtrain" startup="1" protected="0" />
<resource src="GTWtrainhorn" startup="1" protected="0" />
<resource src="GTWturf" startup="1" protected="0" />
<resource src="GTWturnsignals" startup="1" protected="0" />
<resource src="GTWupdates" startup="1" protected="0" />
<resource src="GTWvehicleshop" startup="1" protected="0" />
<resource src="GTWweather" startup="1" protected="0" />`

Startup="1" force the resource to start together with the server and protected="1" prevent you from stopping the resource. Hope that helped you.

Thank you

only the strange black screen

maps where to put?

Maps works like resources, just keep the project file tree structure from this page.

Please use a proper and informative title that may help others with the same issue. I'm leaving this open now for further questions.

Information has been added in the file Installation.md in main, consider this as resolved.


wishlist 2 ?

Hey guys ! I just found out about your project and I think it is awesome ! I had posted on several discord servers an idea very similar to nana about 6 month ago so I wanted to share it with you. Maybe it can inspire you ! I know you do it more for macos and windows but personnaly I think it would be awesome to do it for linux. We could be the first to have an awesome 2022 new terminal, easier than before and more accessible to the noobs. Heres the post I did 6 month ago :+1:
Hey guys I had an crazy idea, I'm not a developper and I know it would be very hard to do but I say it anyway here :
A terminal emulator who is showing gui things. Not for the old bastard like us that loves the black terminal but for young people that discover linux.
When you enter commands, it will have a scrolling menu (like in excel) that recommands commands, and custom simple commands with html/css output (like ls would be able to show a dolphin like screen)
and if you write it in rust everybody will LOVE you ahahah
I would love to help creating it, I'm not a developper but I can design maybe or idk
Heres a concept image :

I don’t believe your concept image is visible

I still see the images.

Thanks for reaching out!


Support icons column

Ingest icons column - if present in the CSV.
Allow setting the icons column.
Allow deleting contents of the icons column.
Write out icons column.

Might be necessary to limit to, say, the first 20 values in the iThoughts palette.

Done in 1.8.


Fix sample-amp.css , font-wight -> font-weight

font-wight -> font-weight

@livekn
Thank you for your advice. I have update npm version . Please update npm.


cannot npm install nodegit with gcc-4.8.5

My gcc-version is 4.8.5 but during installation I got the following error:
Error: /usr/lib64/libstdc++.so.6: version `GLIBCXX_3.4.20' not found (required by /home/app/node_modules/nodegit/build/Release/nodegit.node)
    at Error (native)
    at Object.Module._extensions..node (module.js:434:18)
    at Module.load (module.js:343:32)
    at Function.Module._load (module.js:300:12)
    at Module.require (module.js:353:17)
    at require (internal/module.js:12:17)
    at Object.<anonymous> (/home/app/node_modules/nodegit/dist/nodegit.js:11:12)
    at Module._compile (module.js:409:26)
    at Object.Module._extensions..js (module.js:416:10)
    at Module.load (module.js:343:32)
]
  killed: false,
  code: 1,
  signal: null,
  cmd: '/bin/sh -c node /home/app/node_modules/nodegit/dist/nodegit.js' }

It looks like there is no GLIBCXX_3.4.20 in gcc4.8. Wouldn't nodegit support gcc4.7+ ?  How do I configure my gcc to make it include GLIBCXX_3.4.20 ?

You're missing the libstdc++-dev dependency mentioned in the README.

Since the docs are targeted for Ubuntu whereas I'm using Centos (centos-release-6-8.el6.centos.12.3.x86_64) and a libstdc++-dev does not seem to be available in centos's domain. Do you have ideas how I can upgrade my libstdc++.so.6 to include the GLIBCXX_3.4.20 version ?

Urgh, I wish! CentOS is something I have very limited experience with. I'd search for whatever provides libstdc++ headers in the package manager, should be a relatively easy issue to Google I'd hope. Sorry I closed the issue so quickly, but we can discuss this issue better in realtime with Gitter. If we figure something out, I'd definitely be okay with updating the README to contain instructions for Cent OS.

ok so the solution is to install gcc-4.9


Build problem related to opencl on OSX 10.11, libfreenect2 installed fine with C+11 and OpenCL

I tried building on El Capitan, and it fails at kinect2_registration.
At first, it failed because it cannot find cl.hpp file in #include < OpenCL/cl.hpp >.
I fixed it with cl.hpp from https://www.khronos.org/registry/cl/ version 1.2, which is the same cl.hpp header libfreenect2 use.
Running the build again and I run thru this problem.
Felixs-MacBook-Pro:catkin_ws FelixJonathan$ catkin build kinect2_registration
--------------------------------------------------------------------------------
Profile:                     default
Extending:          [cached] /Users/FelixJonathan/ros_catkin_ws/install_isolated
Workspace:                   /Users/FelixJonathan/catkin_ws
Source Space:       [exists] /Users/FelixJonathan/catkin_ws/src
Build Space:        [exists] /Users/FelixJonathan/catkin_ws/build
Devel Space:        [exists] /Users/FelixJonathan/catkin_ws/devel
Install Space:     [missing] /Users/FelixJonathan/catkin_ws/install
DESTDIR:                     None
--------------------------------------------------------------------------------
Isolate Develspaces:         False
Install Packages:            False
Isolate Installs:            False
--------------------------------------------------------------------------------
Additional CMake Args:       None
Additional Make Args:        None
Additional catkin Make Args: None
Internal Make Job Server:    True
--------------------------------------------------------------------------------
Whitelisted Packages:        None
Blacklisted Packages:        None
--------------------------------------------------------------------------------
Workspace configuration appears valid.
--------------------------------------------------------------------------------
Found '50' packages in 0.1 seconds.
Starting ==> kinect2_registration

[kinect2_registration] ==> '/Users/FelixJonathan/catkin_ws/build/kinect2_registration/build_env.sh /usr/bin/make --jobserver-fds=3,4 -j' in '/Users/FelixJonathan/catkin_ws/build/kinect2_registration'
Scanning dependencies of target kinect2_registration
[ 50%] Building CXX object CMakeFiles/kinect2_registration.dir/src/kinect2_registration.cpp.o
[ 50%] Building CXX object CMakeFiles/kinect2_registration.dir/src/depth_registration_opencl.cpp.o
warning: unknown warning option '-Wunused-but-set-variable'; did you mean '-Wunused-const-variable'? [-Wunknown-warning-option]
warning: unknown warning option '-Wunused-but-set-parameter'; did you mean '-Wunused-parameter'? [-Wunknown-warning-option]
warning: unknown warning option '-Wunused-but-set-variable'; did you mean '-Wunused-const-variable'? [-Wunknown-warning-option]
warning: unknown warning option '-Wunused-but-set-parameter'; did you mean '-Wunused-parameter'? [-Wunknown-warning-option]
/Users/FelixJonathan/catkin_ws/src/iai_kinect2/kinect2_registration/src/depth_registration_opencl.cpp:165:3: error: expected expression
  CHECK_CL_ERROR(err, "cl::Platform::get");
  ^
/Users/FelixJonathan/catkin_ws/src/iai_kinect2/kinect2_registration/src/depth_registration_opencl.cpp:38:62: note: expanded from macro 'CHECK_CL_ERROR'
#define CHECK_CL_ERROR(err, str) do { if(err != CL_SUCCESS) {OUT_ERROR(FG_BLUE "[" << CL_FILENAME << "]" FG_CYAN "(" << __LINE__ << ") " FG_YELLOW << str << FG_RED " failed: " << err); return false; } } while(0)
                                                             ^
/Users/FelixJonathan/catkin_ws/src/iai_kinect2/kinect2_registration/include/kinect2_registration/kinect2_console.h:51:24: note: expanded from macro 'OUT_ERROR'
#define OUT_ERROR(msg) OUT_AUX(FG_RED, FG_RED, ROS_ERROR_STREAM, msg)
                       ^
/Users/FelixJonathan/catkin_ws/src/iai_kinect2/kinect2_registration/include/kinect2_registration/kinect2_console.h:46:78: note: expanded from macro 'OUT_AUX'
#define OUT_AUX(FUNC_COLOR, MSG_COLOR, STREAM, MSG) STREAM(FUNC_COLOR "[" << OUT_FUNCTION(__PRETTY_FUNCTION__) << "] " MSG_COLOR << MSG << NO_COLOR)
                                                                             ^
note: (skipping 3 expansions in backtrace; use -fmacro-backtrace-limit=0 to see all)
/Users/FelixJonathan/ros_catkin_ws/install_isolated/include/ros/console.h:394:43: note: expanded from macro 'ROS_LOG_STREAM_COND'
      ROSCONSOLE_PRINT_STREAM_AT_LOCATION(args); \
                                          ^
/Users/FelixJonathan/ros_catkin_ws/install_isolated/include/ros/console.h:357:56: note: expanded from macro 'ROSCONSOLE_PRINT_STREAM_AT_LOCATION'
    ROSCONSOLE_PRINT_STREAM_AT_LOCATION_WITH_FILTER(0, args)
                                                       ^
/Users/FelixJonathan/ros_catkin_ws/install_isolated/include/ros/console.h:352:64: note: expanded from macro 'ROSCONSOLE_PRINT_STREAM_AT_LOCATION_WITH_FILTER'
    __rosconsole_print_stream_at_location_with_filter__ss__ << args; \
                                                               ^
/Users/FelixJonathan/catkin_ws/src/iai_kinect2/kinect2_registration/src/depth_registration_opencl.cpp:169:5: error: expected expression
    OUT_ERROR("no opencl platforms found.");
    ^
/Users/FelixJonathan/catkin_ws/src/iai_kinect2/kinect2_registration/include/kinect2_registration/kinect2_console.h:51:24: note: expanded from macro 'OUT_ERROR'
#define OUT_ERROR(msg) OUT_AUX(FG_RED, FG_RED, ROS_ERROR_STREAM, msg)
                       ^
/Users/FelixJonathan/catkin_ws/src/iai_kinect2/kinect2_registration/include/kinect2_registration/kinect2_console.h:46:78: note: expanded from macro 'OUT_AUX'
#define OUT_AUX(FUNC_COLOR, MSG_COLOR, STREAM, MSG) STREAM(FUNC_COLOR "[" << OUT_FUNCTION(__PRETTY_FUNCTION__) << "] " MSG_COLOR << MSG << NO_COLOR)
                                                                             ^
/Users/FelixJonathan/catkin_ws/src/iai_kinect2/kinect2_registration/include/kinect2_registration/kinect2_console.h:39:29: note: expanded from macro 'OUT_FUNCTION'
#define OUT_FUNCTION(NAME) ([](const std::string &name)\
                            ^
note: (skipping 2 expansions in backtrace; use -fmacro-backtrace-limit=0 to see all)
/Users/FelixJonathan/ros_catkin_ws/install_isolated/include/ros/console.h:394:43: note: expanded from macro 'ROS_LOG_STREAM_COND'
      ROSCONSOLE_PRINT_STREAM_AT_LOCATION(args); \
                                          ^
/Users/FelixJonathan/ros_catkin_ws/install_isolated/include/ros/console.h:357:56: note: expanded from macro 'ROSCONSOLE_PRINT_STREAM_AT_LOCATION'
    ROSCONSOLE_PRINT_STREAM_AT_LOCATION_WITH_FILTER(0, args)
                                                       ^
/Users/FelixJonathan/ros_catkin_ws/install_isolated/include/ros/console.h:352:64: note: expanded from macro 'ROSCONSOLE_PRINT_STREAM_AT_LOCATION_WITH_FILTER'
    __rosconsole_print_stream_at_location_with_filter__ss__ << args; \
                                                               ^
/Users/FelixJonathan/catkin_ws/src/iai_kinect2/kinect2_registration/src/depth_registration_opencl.cpp:176:3: error: expected expression
  OUT_INFO("devices:");
  ^
/Users/FelixJonathan/catkin_ws/src/iai_kinect2/kinect2_registration/include/kinect2_registration/kinect2_console.h:49:23: note: expanded from macro 'OUT_INFO'
#define OUT_INFO(msg) OUT_AUX(FG_GREEN, NO_COLOR, ROS_INFO_STREAM, msg)
                      ^
/Users/FelixJonathan/catkin_ws/src/iai_kinect2/kinect2_registration/include/kinect2_registration/kinect2_console.h:46:78: note: expanded from macro 'OUT_AUX'
#define OUT_AUX(FUNC_COLOR, MSG_COLOR, STREAM, MSG) STREAM(FUNC_COLOR "[" << OUT_FUNCTION(__PRETTY_FUNCTION__) << "] " MSG_COLOR << MSG << NO_COLOR)
                                                                             ^
/Users/FelixJonathan/catkin_ws/src/iai_kinect2/kinect2_registration/include/kinect2_registration/kinect2_console.h:39:29: note: expanded from macro 'OUT_FUNCTION'
#define OUT_FUNCTION(NAME) ([](const std::string &name)\
                            ^
note: (skipping 2 expansions in backtrace; use -fmacro-backtrace-limit=0 to see all)
/Users/FelixJonathan/ros_catkin_ws/install_isolated/include/ros/console.h:394:43: note: expanded from macro 'ROS_LOG_STREAM_COND'
      ROSCONSOLE_PRINT_STREAM_AT_LOCATION(args); \
                                          ^
/Users/FelixJonathan/ros_catkin_ws/install_isolated/include/ros/console.h:357:56: note: expanded from macro 'ROSCONSOLE_PRINT_STREAM_AT_LOCATION'
    ROSCONSOLE_PRINT_STREAM_AT_LOCATION_WITH_FILTER(0, args)
                                                       ^
/Users/FelixJonathan/ros_catkin_ws/install_isolated/include/ros/console.h:352:64: note: expanded from macro 'ROSCONSOLE_PRINT_STREAM_AT_LOCATION_WITH_FILTER'
    __rosconsole_print_stream_at_location_with_filter__ss__ << args; \
                                                               ^
/Users/FelixJonathan/catkin_ws/src/iai_kinect2/kinect2_registration/src/depth_registration_opencl.cpp:179:5: error: expected expression

... Truncated ...

2 warnings and 3 errors generated.
make[2]: *** [CMakeFiles/kinect2_registration.dir/src/kinect2_registration.cpp.o] Error 1
make[1]: *** [CMakeFiles/kinect2_registration.dir/all] Error 2
make: *** [all] Error 2
[kinect2_registration] <== '/Users/FelixJonathan/catkin_ws/build/kinect2_registration/build_env.sh /usr/bin/make --jobserver-fds=3,4 -j' failed with return code '2'

Failed   <== kinect2_registration [ 3.9 seconds ]
[build] There were '1' errors:

Failed to build package 'kinect2_registration' because the following command:

# Command to reproduce:
cd /Users/FelixJonathan/catkin_ws/build/kinect2_registration && /Users/FelixJonathan/catkin_ws/build/kinect2_registration/build_env.sh /usr/bin/make --jobserver-fds=3,4 -j; cd -

# Path to log:
cat /Users/FelixJonathan/catkin_ws/build/build_logs/kinect2_registration.log

Exited with return code: 2

[build] Runtime: 4.1 seconds

Note: I truncated the error log, but basically it repeating the same error related to opencl call in depth_registration_opencl.cpp.
I can run protonect just fine. Any suggestion would be great.
Thanks

Hi,
libfreenect2 includes their on OpenCL headers. Maybe that is why it is working for libfreenect2 but not for the kinect2_bridge. You can try to copy the CL folder to kinect2_registration/include/ and change the include in depth_registration_opencl.cpp like they are in opencl_depth_packet_processor.cpp. Maybe that helps.

Any updates?

Hi, just changing include file from CL folder in libfreenect2 does not fix the building problem.
(It still has the same exact problem).

What exactly did you change? How does you include statements now look like?

Actually the openCL might not be the problem anymore after replacing it with the libfreenect opencl.
Here is the include statement I have. I don't modify anything in it.
(Note: I had add cl.hpp from libfreenect openCL to the standard location of OpenCL/cl.hpp in OSX. That's the only modification I made.)

#include <fstream>

#include <kinect2_registration/kinect2_console.h>

#ifdef __APPLE__
#include <OpenCL/cl.hpp>
#else
#define CL_USE_DEPRECATED_OPENCL_1_1_APIS
#include <CL/cl.h>
#undef CL_VERSION_1_2
#include <CL/cl.hpp>
#endif

#ifndef REG_OPENCL_FILE
#define REG_OPENCL_FILE ""
#endif

#include "depth_registration_opencl.h"

The problem is still about error: expected expression on CHECK_CL_ERROR(err, "cl::Platform::get");  when calling this macro
#define CHECK_CL_ERROR(err, str) do { if(err != CL_SUCCESS) {OUT_ERROR(FG_BLUE "[" << CL_FILENAME << "]" FG_CYAN "(" << __LINE__ << ") " FG_YELLOW << str << FG_RED " failed: " << err); return false; } } while(0)

Maybe something in the macro is not supported by your compiler. Try to change it to:
#define CHECK_CL_ERROR(err, str) do { if(err != CL_SUCCESS) {OUT_ERROR(str << " failed: " << err); return false; } } while(0)

Still no luck, it expanded to the other macro in kinect2_console.
I agree that it might be compiler problem, since I can build this in linux without any problem.
The compiler I use is the standard clang from the osx. clang --version result:
Apple LLVM version 7.0.2 (clang-700.1.81)
Target: x86_64-apple-darwin15.2.0
Thread model: posix

The error from building
  CHECK_CL_ERROR(err, "cl::Platform::get");
  ^
/Users/FelixJonathan/catkin_ws/src/iai_kinect2/kinect2_registration/src/depth_registration_opencl.cpp:38:62: note: expanded from macro 'CHECK_CL_ERROR'
#define CHECK_CL_ERROR(err, str) do { if(err != CL_SUCCESS) {OUT_ERROR(FG_BLUE "[" << CL_FILENAME << "]" FG_CYAN "(" << __LINE__ << ") " FG_YELLOW << str << FG_RED " failed: " << err); return false; } } while(0)
                                                             ^
/Users/FelixJonathan/catkin_ws/src/iai_kinect2/kinect2_registration/include/kinect2_registration/kinect2_console.h:51:24: note: expanded from macro 'OUT_ERROR'
#define OUT_ERROR(msg) OUT_AUX(FG_RED, FG_RED, ROS_ERROR_STREAM, msg)
                       ^
/Users/FelixJonathan/catkin_ws/src/iai_kinect2/kinect2_registration/include/kinect2_registration/kinect2_console.h:46:78: note: expanded from macro 'OUT_AUX'
#define OUT_AUX(FUNC_COLOR, MSG_COLOR, STREAM, MSG) STREAM(FUNC_COLOR "[" << OUT_FUNCTION(__PRETTY_FUNCTION__) << "] " MSG_COLOR << MSG << NO_COLOR)
                                                                             ^
/Users/FelixJonathan/catkin_ws/src/iai_kinect2/kinect2_registration/include/kinect2_registration/kinect2_console.h:39:29: note: expanded from macro 'OUT_FUNCTION'
#define OUT_FUNCTION(NAME) ([](const std::string &name)\
                            ^
note: (skipping 2 expansions in backtrace; use -fmacro-backtrace-limit=0 to see all)

so clang is having a problem with the lambda function. Maybe there are some flags that you have to set when using clang, or maybe the function has to be written differently.
Or if nothing works, replace the line 39 - 46 with this:
const std::string getFunctionName(const std::string &name);
#define OUT_AUX(FUNC_COLOR, MSG_COLOR, STREAM, MSG) STREAM(FUNC_COLOR "[" << getFunctionName(__PRETTY_FUNCTION__) << "] " MSG_COLOR << MSG << NO_COLOR)

and paste this to the end of kinect2_registration.cpp:
const std::string getFunctionName(const std::string &name)
{
  size_t end = name.rfind('(');
  if(end == std::string::npos)
  {
    end = name.size();
  }
  size_t begin = 1 + name.rfind(' ', end);
  return name.substr(begin, end - begin);
}

I got it compiling and submitted pull request #219 .

Note: It runs fine too.
Thanks @wiedemeyer for your help inspecting the problem. The last tips really help me pass the kinect2_registration macro related problem. I just need to fix some minor stuff after that to make it build.


[docs] Add more docker-compose.*.yaml examples

There are lots of things people can do with docker-compose.*.yaml, but unless familiar with docker-compose it's hard for them to figure out.
The current examples are geared toward extra services, but there's so much more you can do.

Adding an extra server inside web container with HTTP_EXPOSE
Overriding variables
Various items from Stack Overflow examples

The many ddev get recipes in ddev get --list --all are serving this need quite nicely.


Feature request: Group by type

Hello,
First of all, gitree is a pretty neat tool!
##I have a feature request.
It could be nice to group folders together and then files in the output.
Lets have an exemple:
└── api
    ├── assets
    │   └── img
    ├── index.js
    ├── package-lock.json
    ├── package.json
    └── twig
        ├── _layouts
        │   └── default.twig
        ├── index.twig
        └── partials
            ├── header.twig
            └── socials.twig

What if folders were at the top ?
└── api
    ├── assets
    │   └── img
    └── twig
    │   ├── _layouts
    │   │   └── default.twig
    │   └── partials
    │   │   ├── header.twig
    │   │   └── socials.twig
    │   └── index.twig
    ├── index.js
    ├── package-lock.json
    └── package.json

Does it make sense ?
Am I the only one looking for this kind of output ?

This is an interesting one! The order is only in its current state as the git only tracks files which we order alphabetically. Specifying different ordering via a flag would be a cool addition.
I can see three types of ordering?

Default (files and interwoven with folders, ordered alphabetically)
Folders first (folders displayed first, all ordered alphabetically)
Files first (files displayed first, all ordered alphabetically)

Definitely taking this on as an issue, but will happily take PRs!


Large bundle size, currently 165KB

Describe the Bug
Bundling using Webpack in a Next.JS application currently generates a bundle size of 165KB.  This is larger than the rest of our application combined.
Your Example Website or App
No response
Steps to Reproduce the Bug or Issue
Build any ReactFlow application and inspect the bundle size.
Expected behavior
The bundle size is expected to be small.
Screenshots or Videos
No response
Platform
N/A
Additional context
No response

Hey @a3957273
we have react-draggable and d3-zoom as dependencies. The large bundle size is on our list and we would like to drop the d3-zoom dependency but haven't found a good alternative or the time to try to solve this problem internally (and I am also still not sure if this makes sense). However this is not a bug.. Minified and gzipped it should be around ~47 kb which is OK.

The 165KB size was taken pre-compression.  The 'over the wire' size isn't too much of an issue, but on low-end devices the load time of the page can be hindered by the amount of JavaScript that needs to be parsed.
It's fair that this isn't a bug, apologies for classifying it as such.


Add better commands for passing mine, region, and zone flags

So , the current way is this :
make run region=1 zone=2
Right ? or there is some other way , like setting env ?

@imtherealnaska it would be nice to have something like  --region 1 --zone 1 (flags) which follows the standard from go-quai/go-ethereum


Fixes #289 :- Footer overflowing in contributor page fixed

Fixes Issue🛠
Closes #289
Description👨💻
The footer was over lapping with contributors' cards due to which the content visibility was compromised. The footer has been fixed and restored to its original place
Type of change📄

[x] Bug fix (non-breaking change which fixes an issue)

How this has been tested✅
Major checks have been made using Chrome developer tools in all sizes of screens.
Checklist✅

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have added demonstration in the form of GIF/video file
[x] I am an Open Source Contributor

Screenshots/GIF📷
Before:-

After:-

Hi, @Rakesh9100 PLease verify the changes and merge the pr into main branch
Thanks

Pls remove the spaces below the footer @pb0126

Okay

Hi @Rakesh9100, Please check now, also assign me issue #291

Not fixed, check again and resolve @pb0126

@pb0126

Please check again

Yes done, netlify didn't updated  that I think @pb0126


Extend manifest CR tests

Description
This PR extends existing tests for Manifest CR reconciliation.
It adds one new suite that uses CustomResourceReadyCheck for state checking instead of a simplified implementation ExistsReadyCheck. Usage of CustomResourceReadyCheck should be preferred, because this is the one the production code is using.
Some improvement in timing configuration of the test suites is proposed. This should be fine-tuned in subsequent PRs, but a significant improvement in execution times is already observed.
Changes proposed in this pull request:

add checks for oci-sync-ref annotation
add checks for status.synced resource list
add checks if all resources are deleted when the Manifest CR is deleted
add suite for Manifest CR following state transition in the Resource CR: Ready -> Warning -> Ready
configure more aggresive timing for test suites (faster execution)

Related issue(s)
See: #702

/test pull-lifecycle-mgr-tests


Can not reproduce results in your paper

Hi, I downloaded your codes, and train ATSS-R50 and RankLoss-ATSS-R50, the results are:

algorithm
exp1 mAP
exp2 mAP

ATSS
0.396
0.393

RS-ATSS
0.393
0.392

The mAP of RS-ATSS is lower than ATSS, however, in the paper,  RS-ATSS is 0.399 , ATSS is 0.393。
I wonder why can not reproduce the results, is there something wrong?

Hi,
Thanks for your interest in our work. I will try to assist reproducing the results.
What is your batch size (how many GPUs and how many images on each GPU) during training? Also, just to confirm: have you made any changes in the code or config file?

@kemaloksuz  I only change workers_per_gpu, nothing changed else.
 55 data = dict(
 56     samples_per_gpu=4,
 57     workers_per_gpu=2)

What about number of GPUs?, i.e. How many GPUs do you use for training? You can also directly share your training command to see the number of GPUs.
8 V100  16G GPU were used to train.

What about number of GPUs?, i.e. How many GPUs do you use for training? You can also directly share your training command to see the number of GPUs.

8 V100 16G GPU were used to train.

We used 4 GPUs for training ATSS and the learning rate is tuned for 4 images/GPU, that is 16 images in batch. So, that's the mismatch with our configuration.  Hence, I think decreasing the number of GPUs to 4 (and keeping 4 images/GPU) should solve your reproduction problem.

Just to update: Upon your worries, we tried to reproduce ATSS and obtained 39.8 AP, which aligns with the reported result 39.9 AP. If you use 4 GPUs for training and 4 images/GPU, you should get ~39.9AP.

Using the provided info above, have you been able to reproduce our results? If so, could you please close the issue?


npm run ingest return 429 error

~/gpt4-pdf-chatbot-langchain/scripts/ingest-data.ts:46
    throw new Error('Failed to ingest your data');
          ^

Error: Failed to ingest your data
    at run (~/gpt4-pdf-chatbot-langchain/scripts/ingest-data.ts:46:11)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at <anonymous> (~/gpt4-pdf-chatbot-langchain/scripts/ingest-data.ts:51:3)

Node.js v18.16.0

creating vector store...
error Error: Request failed with status code 429
    at createError (~/gpt4-pdf-chatbot-langchain/node_modules/axios/lib/core/createError.js:16:15)
    at settle (~/gpt4-pdf-chatbot-langchain/node_modules/axios/lib/core/settle.js:17:12)
    at IncomingMessage.handleStreamEnd (~/gpt4-pdf-chatbot-langchain/node_modules/axios/lib/adapters/http.js:322:11)
    at IncomingMessage.emit (node:events:525:35)
    at IncomingMessage.emit (node:domain:489:12)
    at endReadableNT (node:internal/streams/readable:1359:12)
    at process.processTicksAndRejections (node:internal/process/task_queues:82:21) {

it was a small pdf with only 3 pages and mainly text.

did u create a index in pinecone?

yes I did created index in pinecone.

did you solve it?

dimension needs to be 1536 i think, but i still have the same error

It could be related to your credits in OpenAI.
Check that your credits didn't expire here - https://platform.openai.com/account/usage

I also have a small pdf and face the same error. I still have enough OpenAI credit.

is it possible to share this pdf. Sometimes the pdf isn't converted.

hi I resolved the issue, it turned out to be not enough openAPI balance


DriveItem is missing from finished upload session

It seems that driveItem is not anymore delivered when the final byte range of the file is PUT to the upload URL. Final response contains the same expected range than previously send bytes. Response code is 201 and file is uploaded successfully. From the documentation it is not obvious whether driveItem should be in final response or not if deferCommit is not used. For me it sounds logical to include the item data and before it did work that way. If I want to get driveItem after upload is finished do I need to handle upload session with deferCommit set as true?
Response after sending final bytes:
{"expirationDateTime":"2021-09-21T07:02:17.624Z","nextExpectedRanges":["10485760-15334441"]}

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

ID: d409f771-fd62-e549-d581-aac009e79e8c
Version Independent ID: fe646b3a-06f5-a269-47e6-b4ee693e5410
Content: Resumable file upload - OneDrive API - OneDrive dev center
Content Source: docs/rest-api/api/driveitem_createuploadsession.md
Product: onedrive
GitHub Login: @JeremyKelley
Microsoft Alias: JeremyKe

That response suggests the service is expecting you to upload another range. Can you show the Range headers for each of the Put requests you've made?

Here is an example:
Content-Range: bytes 0-5242879/15846246
-> 202 {"expirationDateTime":"2021-09-22T05:21:57.637Z","nextExpectedRanges":["5242880-15846245"]}
Content-Range: bytes 5242880-10485759/15846246
-> 202 {"expirationDateTime":"2021-09-22T05:21:57.637Z","nextExpectedRanges":["10485760-15846245"]}
Content-Range: bytes 10485760-15728639/15846246
-> 202 {"expirationDateTime":"2021-09-22T05:21:57.637Z","nextExpectedRanges":["15728640-15846245"]}
Content-Range: bytes 15728640-15846245/15846246
-> 201 {"expirationDateTime":"2021-09-22T05:21:57.637Z","nextExpectedRanges":["15728640-15846245"]}

That is weird - the fact that it's a 201 implies that the service did think it's completed. To drill in further I'll need a few more details:

Are you calling personal or business OneDrive?
Are you providing an access token (via the Authorization header) or relying on the fact the upload URL is preauthorized?
Getting a request id would let us look at telemetry:

If hitting business OneDrive directly the SPRequestGUID and Date headers in the the 201 response

We are using Microsoft Graph SDK for Java. And as said we have used this interface successfully before.
In this particular case:

Personal OneDrive
Access token is provided in header

I noticed that when deferCommit is used and upload is finished with PUT request then response contains DriveItem as described in here: https://docs.microsoft.com/en-us/graph/api/driveitem-createuploadsession. But I'm bit confused whether deferCommit can be used with both personal and business accounts.
Documentation in https://docs.microsoft.com/en-us/graph/api/driveitem-createuploadsession says:

deferCommit
Boolean
If set to true, final creation of the file in the destination will require an explicit request. Only on OneDrive for Business.

But later on the same documentation page:

If deferCommit is true, you can explicitly complete the upload in two ways:

After the final byte range of the file is PUT to the upload URL, send a final POST request to the upload URL with zero-length content (currently only supported on OneDrive for Business and SharePoint).
After the final byte range of the file is PUT to the upload URL, send a final PUT request in the same way that you would handle upload errors (currently only supported on OneDrive Personal).

From: akih @.>
Sent: Monday, September 20, 2021 8:00 AM
To: OneDrive/onedrive-api-docs @.>
Cc: Subscribed @.***>
Subject: Re: [OneDrive/onedrive-api-docs] DriveItem is missing from finished upload session (#1531)
We are using Microsoft Graph SDK for Java. And as said we have used this interface successfully before.
In this particular case:

Personal OneDrive
Access token is provided in header

I noticed that when deferCommit is used and upload is finished with PUT request then response contains DriveItem as described in here: https://docs.microsoft.com/en-us/graph/api/driveitem-createuploadsession. But I'm bit confused whether deferCommit can be used with both personal and business accounts.
Documentation in https://docs.microsoft.com/en-us/graph/api/driveitem-createuploadsession says:
deferCommit
Boolean
If set to true, final creation of the file in the destination will require an explicit request. Only on OneDrive for Business.
But later on the same documentation page:
If deferCommit is true, you can explicitly complete the upload in two ways:

After the final byte range of the file is PUT to the upload URL, send a final POST request to the upload URL with zero-length content (currently only supported on OneDrive for Business and SharePoint).
After the final byte range of the file is PUT to the upload URL, send a final PUT request in the same way that you would handle upload errors (currently only supported on OneDrive Personal).

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/OneDrive/onedrive-api-docs/issues/1531#issuecomment-922654153, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABLIQTVQ2QKUWG5DHODBBW3UC3EWRANCNFSM5EECWWUQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

re: defer commit, I believe it works on both so the fact that it calls out OneDrive for Business specifically is probably a documentation bug.
However, it really shouldn't be needed in this case and I'm at a loss to explain how you're seeing what you're seeing. The code that sets the status code to 201 will only serialize the item metadata to the response stream.
Are you using the LargeFileUploadTask from the Java SDK or are you executing the upload session manually?

We are using version 2.10.0 from the Java SDK and can not upgrade at the moment because our device has 23 Android SDK. So the class that I'm using is  ChunkedUploadProvider. I looked deeper to our Java SDK sources and it seems that there is a bug in ChunkedUploadResponseHandler which is causing this behavior; previous response is cached and used when trying to serialize item metadata.. So the problem was in our end, sorry for bothering!


Documents & Images Search - Searching within Collections

While looking at issue #3997 I have realised that the searching within collections UI needs some work and has some confusing behaviour.
Issue Summary
Images
A URL like /admin/images/?collection_id=4&q=pa will actually cause an error: 'DatabaseSearchResults' object has no attribute 'filter'
Both
Typing in a search will clear the selected collection in a non-intuitive way, the selected collection stays but the results are from all collection. Inversely, selecting a collection will clear the existing search.
Steps to Reproduce
Search then change Collection

Open up the Wagtail Bakery Demo application
Log into Admin, navigate to 'Images' on the left sidebar
Type p into search, around 6 images show
Try to narrow down the search by selecting 'BreadPage images'
5a. Expected: You will now be on a page of BreadPage images that have the letter p in them
5b. Actual: Your typed in search is cleared and you are seeing all images in the collection

Change Collection then Search

Open up the Wagtail Bakery Demo application
Log into Admin, navigate to 'Images' on the left sidebar
Select the collection BreadPage Images (URL will change to /admin/images/?collection_id=4)
Type p into search field
5a. Expected: I will see all the BreadPage Images with the letter p in title.
5b. Actual: Collection dropdown does not change, but my search results are from all collections.
Click backspace to clear the search
7a. Expected: I should be back at the collection BreadPage Images as it is still now selected in the collection dropdown.
7b. Actual: I am now seeing all images from all collections, even though the selected collection is still selected in dropdown, to get back to this collection I have to select a different one, then select the original again (to trigger the change).

Technical details

Python version: 3.4.3
Django version: 1.11.6
Wagtail version: 1.13.1
Browser version: Chrome 62

Notes for Resolution
Client considerations
We will need a standard way to update the URL for multiple parameters, so that if we add a search parameter q we do not clear the collection_id parameter and visa verser.
Jquery has a nice helper $.param to convert js objects to URL params, when in object form it is easy to update the object with any changed params. However, getting the current URL params into an object is not part of Jquery. Ideally we would use the URLSearchParams Web API but IE does not support it yet, maybe we could use a polyfill..
Server considerations
The above mentioned issue with images needs to be resolved to ensure that multiple URL params can be handled.
Pagination also needs to be considered here, what happens if you are on page 2 of a search and change the collection, should pagination reset?
Need to handle mixes of search/collection in the request.is_ajax() response, making sure the right data is sent back and handled on the client.
Relevant Files

wagtailadmin/js/core.js - when searching on images/documents index
wagtail/images/templates/wagtailimages/images/index.html - changes on collection, currently posts a form so need to check behaviour on posting
/wagtail/documents/templates/wagtaildocs/documents/index.html - same as the above for images
/wagtail/documents/views/documents.py - view for documents
/master/wagtail/images/views/images.py - view for images

Partial work was done on a fix for this a while ago https://github.com/wagtail/wagtail/pull/4664 but we closed as it had been a long time and had not been reviewed.

For anyone looking at this, it might be best to pick up a small part of this first - maybe just the images only.

https://github.com/wagtail/wagtail/pull/9206 might offer some insights how to fix this kind of problem simply.


Can we support Visual Basic apps in the .NET Core Paketo Buildpack?

We should investigate how hard it would be to support VB in the dotnet-core-cnbs, it may be just a simple as recognizing *.vbproj during detect.

After some preliminary testing, this appears to be as simple as changing to the detection criteria to allow all *.*proj files to trigger detection true. Beyond that .Net takes care of the rest.

This should be documented once complete in https://github.com/paketo-buildpacks/paketo-website/issues/85.

This should be documented once complete in https://github.com/paketo-buildpacks/paketo-website/issues/85.

As per a discussion, we should check for a restricted subset of the .Net project files. Namely csproj, fsproj, and vbproj. This is to prevent potential conflicts with other software and language ecosystems that may use a file that matches the *.*proj file structure.
As a note you are able to generate apps that use VisualBasic through the dotnet new command.

As per a discussion, we should check for a restricted subset of the .Net project files. Namely csproj, fsproj, and vbproj. This is to prevent potential conflicts with other software and language ecosystems that may use a file that matches the *.*proj file structure.
As a note you are able to generate apps that use VisualBasic through the dotnet new command.

Looks great! We verified this by doing the following:

create a Visual basic console app.

dotnet new console -lang vb
The template "Console Application" was created successfully.

Processing post-creation actions...
Running 'dotnet restore' on /Users/kashyap/workspace/paketo-samples/dotnet/visual-basic/visual-basic.vbproj...
  Determining projects to restore...
  Restored /Users/kashyap/workspace/paketo-samples/dotnet/visual-basic/visual-basic.vbproj (in 70 ms).
Restore succeeded.

Build it

pack build --buildpack ~/Downloads/dotnet-core-0.0.10.cnb visual-basic-console-app
full: Pulling from paketobuildpacks/builder
Digest: sha256:5e314f45b48c9e2c015cbb49559c7cf0f15c4a65831674af80cbdf7ca7e4f542
Status: Image is up to date for paketobuildpacks/builder:full
full-cnb: Pulling from paketobuildpacks/run
Digest: sha256:341f2f77add7c099df0d5365d8fc4880934e197377c8e30b99df542ada8e4117
Status: Image is up to date for paketobuildpacks/run:full-cnb
===> DETECTING
5 of 8 buildpacks participating
paketo-buildpacks/dotnet-core-runtime 0.1.3
paketo-buildpacks/dotnet-core-sdk     0.1.3
paketo-buildpacks/icu                 0.0.100
paketo-buildpacks/dotnet-publish      0.1.2
paketo-buildpacks/dotnet-execute      0.3.1
===> ANALYZING
Previous image with name "visual-basic-console-app" not found
===> RESTORING
===> BUILDING
Paketo .NET Core Runtime Buildpack 0.1.3
  Resolving Dotnet Core Runtime version
    Candidate version sources (in priority order):
      visual-basic.vbproj -> "5.0.0"
      <unknown>           -> "*"
      <unknown>           -> "*"

    No exact version match found; attempting version roll-forward

    Selected dotnet-runtime version (using visual-basic.vbproj): 5.0.2

  Executing build process
    Installing Dotnet Core Runtime 5.0.2
      Completed in 6.881s

  Configuring environment for build and launch
    DOTNET_ROOT -> "/workspace/.dotnet_root"

  Configuring environment for build
    RUNTIME_VERSION -> "5.0.2"

Paketo .NET SDK Buildpack 0.1.3
  Resolving .NET Core SDK version
    Candidate version sources (in priority order):
      RUNTIME_VERSION     -> "5.0.102"
      visual-basic.vbproj -> "5.0.*"
      <unknown>           -> "*"

    Selected .NET Core SDK version (using RUNTIME_VERSION): 5.0.102

  Executing build process
    Installing .NET Core SDK 5.0.102
      Completed in 16.841s

  Configuring environment
    DOTNET_ROOT -> "/workspace/.dotnet_root"
    PATH        -> "/workspace/.dotnet_root:$PATH"

Paketo ICU Buildpack 0.0.100
  Executing build process
    Installing ICU
      Completed in 1.126s

Paketo .Net Publish Buildpack 0.1.2
  Executing build process
    Running 'dotnet publish /workspace --configuration Release --runtime ubuntu.18.04-x64 --self-contained false --output /tmp/dotnet-publish-output785745243'
      Completed in 4.3161658s

  Removing source code

Paketo .NET Execute Buildpack 0.3.1
  Assigning launch processes
    web: /workspace/visual-basic --urls http://<IP_ADDRESS>:${PORT:-8080}

===> EXPORTING
Adding layer 'paketo-buildpacks/dotnet-core-runtime:dotnet-core-runtime'
Adding layer 'paketo-buildpacks/dotnet-core-sdk:dotnet-env-var'
Adding layer 'paketo-buildpacks/icu:icu'
Adding 1/1 app layer(s)
Adding layer 'launcher'
Adding layer 'config'
Adding layer 'process-types'
Adding label 'io.buildpacks.lifecycle.metadata'
Adding label 'io.buildpacks.build.metadata'
Adding label 'io.buildpacks.project.metadata'
Setting default process type 'web'
*** Images (562005a586c6):
      visual-basic-console-app
Adding cache layer 'paketo-buildpacks/dotnet-core-runtime:dotnet-core-runtime'
Adding cache layer 'paketo-buildpacks/dotnet-core-sdk:dotnet-core-sdk'
Adding cache layer 'paketo-buildpacks/icu:icu'
Successfully built image visual-basic-console-app

Run it. It works.

docker run -e PORT=3000 -p 3000:3000 visual-basic-console-app
Hello World!

cc: @thitch97

Looks great! We verified this by doing the following:

create a Visual basic console app.

dotnet new console -lang vb
The template "Console Application" was created successfully.

Processing post-creation actions...
Running 'dotnet restore' on /Users/kashyap/workspace/paketo-samples/dotnet/visual-basic/visual-basic.vbproj...
  Determining projects to restore...
  Restored /Users/kashyap/workspace/paketo-samples/dotnet/visual-basic/visual-basic.vbproj (in 70 ms).
Restore succeeded.

Build it

pack build --buildpack ~/Downloads/dotnet-core-0.0.10.cnb visual-basic-console-app
full: Pulling from paketobuildpacks/builder
Digest: sha256:5e314f45b48c9e2c015cbb49559c7cf0f15c4a65831674af80cbdf7ca7e4f542
Status: Image is up to date for paketobuildpacks/builder:full
full-cnb: Pulling from paketobuildpacks/run
Digest: sha256:341f2f77add7c099df0d5365d8fc4880934e197377c8e30b99df542ada8e4117
Status: Image is up to date for paketobuildpacks/run:full-cnb
===> DETECTING
5 of 8 buildpacks participating
paketo-buildpacks/dotnet-core-runtime 0.1.3
paketo-buildpacks/dotnet-core-sdk     0.1.3
paketo-buildpacks/icu                 0.0.100
paketo-buildpacks/dotnet-publish      0.1.2
paketo-buildpacks/dotnet-execute      0.3.1
===> ANALYZING
Previous image with name "visual-basic-console-app" not found
===> RESTORING
===> BUILDING
Paketo .NET Core Runtime Buildpack 0.1.3
  Resolving Dotnet Core Runtime version
    Candidate version sources (in priority order):
      visual-basic.vbproj -> "5.0.0"
      <unknown>           -> "*"
      <unknown>           -> "*"

    No exact version match found; attempting version roll-forward

    Selected dotnet-runtime version (using visual-basic.vbproj): 5.0.2

  Executing build process
    Installing Dotnet Core Runtime 5.0.2
      Completed in 6.881s

  Configuring environment for build and launch
    DOTNET_ROOT -> "/workspace/.dotnet_root"

  Configuring environment for build
    RUNTIME_VERSION -> "5.0.2"

Paketo .NET SDK Buildpack 0.1.3
  Resolving .NET Core SDK version
    Candidate version sources (in priority order):
      RUNTIME_VERSION     -> "5.0.102"
      visual-basic.vbproj -> "5.0.*"
      <unknown>           -> "*"

    Selected .NET Core SDK version (using RUNTIME_VERSION): 5.0.102

  Executing build process
    Installing .NET Core SDK 5.0.102
      Completed in 16.841s

  Configuring environment
    DOTNET_ROOT -> "/workspace/.dotnet_root"
    PATH        -> "/workspace/.dotnet_root:$PATH"

Paketo ICU Buildpack 0.0.100
  Executing build process
    Installing ICU
      Completed in 1.126s

Paketo .Net Publish Buildpack 0.1.2
  Executing build process
    Running 'dotnet publish /workspace --configuration Release --runtime ubuntu.18.04-x64 --self-contained false --output /tmp/dotnet-publish-output785745243'
      Completed in 4.3161658s

  Removing source code

Paketo .NET Execute Buildpack 0.3.1
  Assigning launch processes
    web: /workspace/visual-basic --urls http://<IP_ADDRESS>:${PORT:-8080}

===> EXPORTING
Adding layer 'paketo-buildpacks/dotnet-core-runtime:dotnet-core-runtime'
Adding layer 'paketo-buildpacks/dotnet-core-sdk:dotnet-env-var'
Adding layer 'paketo-buildpacks/icu:icu'
Adding 1/1 app layer(s)
Adding layer 'launcher'
Adding layer 'config'
Adding layer 'process-types'
Adding label 'io.buildpacks.lifecycle.metadata'
Adding label 'io.buildpacks.build.metadata'
Adding label 'io.buildpacks.project.metadata'
Setting default process type 'web'
*** Images (562005a586c6):
      visual-basic-console-app
Adding cache layer 'paketo-buildpacks/dotnet-core-runtime:dotnet-core-runtime'
Adding cache layer 'paketo-buildpacks/dotnet-core-sdk:dotnet-core-sdk'
Adding cache layer 'paketo-buildpacks/icu:icu'
Successfully built image visual-basic-console-app

Run it. It works.

docker run -e PORT=3000 -p 3000:3000 visual-basic-console-app
Hello World!

cc: @thitch97


newest CMDMess. to platformio.ini/newest Servo lib

Moved the adapted CMDMessenger lib to platformio.ini and added the required definitions to platformio.ini.
Futhermore updated arduino-libraries/Servo to newest version in platformio.ini.
At the time when this repo is moved to Mobiflight, I would highly appreciate if Sebastian could fork the CMDMessenger lib and adapt the following changes in CmdMessenger.h:
#ifndef MAXCALLBACKS #define MAXCALLBACKS        50   // The maximum number of commands   (default: 50) #endif #ifndef MESSENGERBUFFERSIZE #define MESSENGERBUFFERSIZE 64   // The length of the commandbuffer  (default: 64) #endif #ifndef MAXSTREAMBUFFERSIZE #define MAXSTREAMBUFFERSIZE 512  // The length of the streambuffer   (default: 64) #endif #ifndef DEFAULT_TIMEOUT #define DEFAULT_TIMEOUT     5000 // Time out on unanswered messages. (default: 5s) #endif
The platformio.ini should also changed accordingly.  Just to have all required things in one reposority.

I've opened #18 as a reminder to move the github repo for CmdMessenger over.

@elral since the checked in CmdMessenger is no longer needed can you update this pull request to also delete the CmdMessenger folder under lib/?

Never mind 🤦♂ I see you already deleted them. Time for more ☕.

@elral This PR needs to have the ProMicro merge conflict resolved as well.


Support live visualizations with Tyche, via the OpenPBTStats log format?

Tyche is a VS Code extension designed to help users understand what their property-based tests are doing - with nice features like live-updating graphs, and lots of explorable raw data if you want to dig deeper.
I've been collaborating with the author since late last year (as a Hypothesis maintainer), as described in our paper.  Notably, this helped us find several performance problems and outright bugs in Hypothesis itself, as well as being useful for our users.
But the reason I'm opening this issue is that Tyche is not Hypothesis-specific; it's instead based on a simple json-lines format which could be emitted from any PBT library.  If you want to support it from CsCheck, emitting the type, run_start, property, status, and representation is enough to enable a substantial portion of Tyche’s features.

This looks pretty cool -- thanks for letting us know. Will look into it


Assertion failed: file JTAppleCalendar/JTACMonthActionFunctions.swift, line 43

(Required) Version Number: 8.0.3
Description
Error this should not be nil. Contact developer Jay on github by opening a request
There was an error in this code section. Please contact the developer on GitHub
Steps To Reproduce
I called layout subviews on my view controller.
I've been using the calendar for a while but it's the first time I see this happening. Hard to replicate!
Expected Behavior
Additional Context
Stack trace https://pastebin.com/fuAc8Xb0

Its a assert error that i have  in the code. It will not crash in prod because its simply an assert and not a fatal error.
However, it does alert the developer that something is potentially wrong.
This usually happens when the Calendar's frame is .Zero, or .hidden = true
Are you doing something like this?
If there error happens again, then can you go up the stack trace until you see the .frame of the calendar.
To find the error instantly, can you put a breakpoint inside the dateOwnerInfoFromPath function and see why it is returning nil?  Because this is why it is causing as Assert error. The dateOwnerInfoFromPath function is returning nil.

@patchthecode I am having the same issue. It has something to do with iOS 14.1 because the crash is non-existent on pervious iOS versions (tested 13.6 and below). Accordingly, it only appeared after upgrading to Xcode 12 + iOS 14.1 simulators.

I'll be updating to iOS 14.1 soon. I am currently on iOS 14.0

this should have been fixed now. latest version is 8.0.5


Fix MuJoCo.Humanoid action description

Description
The old description appears to have been a copy and paste of MuJoCo.Ant description

Sources:
https://github.com/Farama-Foundation/Gymnasium/blob/main/gymnasium/envs/mujoco/assets/humanoid.xml#L103
https://github.com/Farama-Foundation/Gymnasium/blob/main/gymnasium/envs/mujoco/assets/humanoidstandup.xml#L103

pre-commit does nothing for me
(also note the files, that fail the pre-commit on GitHub workflow are unrelated to my PR)

Ok, you need to update your fork and merge with main.

Ok, pc passes now


-1 precision will now use default to max of 17, double precision

Anything beyond that is useless, though people are free to use higher if they'd like.

@gir-bot lgtm


material design enhenced

Hi, this theme adds many enhencements to your fork. You can preview it here : http://jsfiddle.net/djibe89/yvbts5e9/
Thanks for your work and help.

Hi, try my pull request here : http://jsfiddle.net/djibe89/yvbts5e9/
many many new enhencements


[BUG] Named client component export Cart clashes with Hydrogen Cart component

Describe the bug
I think we have a naming issue in our ClientMarker wrapping logic. If a userland component is named the same thing as a Hydrogen component (and is not a default export, seemingly), the names clash and we get this error in the server console:
5:00:20 PM [vite] Internal server error: /Users/joshlarson/src/github.com/half-helix/shopify_supply/src/components/Cart.client.jsx: Identifier 'Cart' has already been declared. (5:13)

  3 |
  4 | export default wrapInClientMarker({ name: 'Cart', id: '/Users/joshlarson/src/github.com/half-helix/shopify_supply/src/components/Cart.client.jsx', component: Cart, named: false });
> 5 | export const Cart = wrapInClientMarker({ name: 'Cart', id: '/Users/joshlarson/src/github.com/half-helix/shopify_supply/src/components/Cart.client.jsx', component: namedImports['Cart'], named: true });
    |              ^
  6 | export const CartIcon = wrapInClientMarker({ name: 'CartIcon', id: '/Users/joshlarson/src/github.com/half-helix/shopify_supply/src/components/Cart.client.jsx', component: namedImports['CartIcon'], named: true });
  7 |
  Plugin: vite:react-babel
  File: /Users/joshlarson/src/github.com/half-helix/shopify_supply/src/components/Cart.client.jsx
  3  |
  4  |  export default wrapInClientMarker({ name: 'Cart', id: '/Users/joshlarson/src/github.com/half-helix/shopify_supply/src/components/Cart.client.jsx', component: Cart, named: false });
  5  |  export const Cart = wrapInClientMarker({ name: 'Cart', id: '/Users/joshlarson/src/github.com/half-helix/shopify_supply/src/components/Cart.client.jsx', component: namedImports['Cart'], named: true });
     |               ^
  6  |  export const CartIcon = wrapInClientMarker({ name: 'CartIcon', id: '/Users/joshlarson/src/github.com/half-helix/shopify_supply/src/components/Cart.client.jsx', component: namedImports['CartIcon'], named: true });
  7  |
      at Object._raise (/Users/joshlarson/src/github.com/half-helix/shopify_supply/node_modules/@babel/parser/lib/index.js:541:17)
      at Object.raiseWithData (/Users/joshlarson/src/github.com/half-helix/shopify_supply/node_modules/@babel/parser/lib/index.js:534:17)
      at Object.raise (/Users/joshlarson/src/github.com/half-helix/shopify_supply/node_modules/@babel/parser/lib/index.js:495:17)
      at ScopeHandler.checkRedeclarationInScope (/Users/joshlarson/src/github.com/half-helix/shopify_supply/node_modules/@babel/parser/lib/index.js:1688:12)
      at ScopeHandler.declareName (/Users/joshlarson/src/github.com/half-helix/shopify_supply/node_modules/@babel/parser/lib/index.js:1654:12)
      at Object.checkLVal (/Users/joshlarson/src/github.com/half-helix/shopify_supply/node_modules/@babel/parser/lib/index.js:11086:24)
      at Object.parseVarId (/Users/joshlarson/src/github.com/half-helix/shopify_supply/node_modules/@babel/parser/lib/index.js:14044:10)
      at Object.parseVar (/Users/joshlarson/src/github.com/half-helix/shopify_supply/node_modules/@babel/parser/lib/index.js:14019:12)
      at Object.parseVarStatement (/Users/joshlarson/src/github.com/half-helix/shopify_supply/node_modules/@babel/parser/lib/index.js:13836:10)
      at Object.parseStatementContent (/Users/joshlarson/src/github.com/half-helix/shopify_supply/node_modules/@babel/parser/lib/index.js:13421:21)
Error: entry-server.jsx could not be loaded. This likely occurred because of a Vite compilation error.

To Reproduce

In Cart.client.jsx in the starter template, change export default to export
Check the server console

Expected behaviour
Names should not clash.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here. eg.

Hydrogen version: 0.7.0

@jplhomer I think the actual problem is when having a component exported twice, once as default and once as named export:
export const MyComponent = function() {}
export default MyComponent

This would get compiled to something like:
import MyComponent, * as namedImports from 'xyz'

export const MyComponent = namedImports.MyComponent
export default MyComponent

Which is obviously wrong because the identifier MyComponent is declared twice. At least is not related to using the same name as Hydrogen components, which would be harder to fix I think 😅

Ahh good catch!


Removed knowledge about "main source" in pane and price scale

Type of PR: enhancement
Overview of change:
It's a bit strange to have "main source" term in pane or price scale, so I decided to remove it, but I'm not sure about that.

The tests need to be fixed...


Update README, adding instruction for extracting messages using wildcard

Resolves #65

@googlebot I signed it!

Sorry to tag you @alan-knight @davidmorgan. May I know whom should I reach out to review this PR? Thanks!

I'm afraid we don't currently have a process for accepting contributions to intl_translation. If/when we do I'll try to include this change. Sorry about that!


add coveralls

Changes Unknown when pulling bc252741403b855fbe0b9c387866a61604c664fd on pre-build into ** on master**.

Changes Unknown when pulling bc252741403b855fbe0b9c387866a61604c664fd on pre-build into ** on master**.


502 Bad Gateway response to requests for statistics endpoint

Description
Request to statistics endpoint is giving 502 Bad Gateway
Logs/Alerts

Also causing an error in the console in the explorer node stats in dashboard

duplicate to: https://github.com/threefoldtech/tfgrid-sdk-go/issues/104


Unexpected return type of useTemplateRef with native HTML element

Vue - Official extension or vue-tsc version
2.1.4
VSCode version
null
Vue version
3.5.0
TypeScript version
5.5.4
System Info
System:
    OS: Linux 5.0 undefined
    CPU: (8) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 0 Bytes / 0 Bytes
    Shell: 1.0 - /bin/jsh
  Binaries:
    Node: 18.20.3 - /usr/local/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 10.2.3 - /usr/local/bin/npm
    pnpm: 8.15.6 - /usr/local/bin/pnpm

package.json dependencies
{
  "dependencies": {
    "vue": "^3.5.0"
  },
  "devDependencies": {
    "@vitejs/plugin-vue": "^5.1.3",
    "typescript": "^5.5.4",
    "vite": "^5.4.2",
    "vue-tsc": "^2.1.4"
  }
}

Steps to reproduce
Add foo.vue file with a code below. This is almost same as the example in Vue official docs.
<script setup lang="ts">
import { useTemplateRef, onMounted } from 'vue';

// the first argument must match the ref value in the template
const input = useTemplateRef('my-input');

onMounted(() => {
  input.value?.focus();
});
</script>

<template>
  <input ref="my-input" />
</template>

What is expected?
Running vue-tsc shouldn't raise an error, input.value should be inferred as HTMLInputElement | null (same as actual runtime behavior).
What is actually happening?
Raises an error below.
src/App.vue:8:16 - error TS2551: Property 'focus' does not exist on type 'InputHTMLAttributes & ReservedProps'. Did you mean 'onFocus'?

8   input.value?.focus();
                 ~~~~~

Link to minimal reproduction
https://stackblitz.com/edit/vitejs-vite-mmua2t?file=src%2FApp.vue
Any additional comments?
No response

It has been fixed by #4786


Refactor curlatLng to currentLatLng

Hi @nicolas-raoul, I'm a newbie to open source and I would like to work on this. Hope it is okay

@aahlad2000 Of course, that's a perfect task to begin. :-)

hi @nicolas-raoul I am new to open source community, I am not sure how it works, can I still take this task, or do I move on to next task?

HI @krishnathakuralen, I have already picked this up. Can you please pick another task up, thanks.

ok sure :)

@nicolas-raoul raise a new PR for this:
Refactor curlatLng to currentLatLng #5646
Please help to review and approve this PR.


Decide if metadata should be managed independently of tags

Right now, nup update extracts each song's metadata from its ID3 tag and sends it to the server.
I tend not to update songs' tags after I've imported them since it makes gsutil sync upload the updated files to GCS as new objects, which is slow and can cost money (the Coldline storage class has a minimum storage duration of 90 days). Releases are also often merged in MusicBrainz, which makes their album IDs change, which breaks their cover art.
Updating a song might also cause problems if the Android client had previously cached part of it pre-update (the client doesn't save the ETag header, and I think it might not even check that the content-length is the same).
I'm filing this bug to brainstorm different ways I can improve this.

The nup tool could support per-song overrides. A new subcommand could download updated metadata from MB for a given file and save it to a file (e.g. ~/.nup/overrides/<sha1>.json), and update could use it in place of the tag's metadata when sending the song to the server. If the song's album ID changes, update can look for cover art for both the old and new IDs. Adding this might also help with #32.

The server could automatically fetch updated metadata from MB periodically and use it to update the objects in datastore. However, MB sometimes has bad data (e.g. I submit edits to fix something before importing a song but it takes a week for the edits to be applied), and it may create a lot of churn in datastore and excess updates when syncing the Android app.

I think I don't like the idea of the server changing things automatically, because it'll be annoying if an album becomes hard to find or suddenly has typos or weird capitalization due to a bad edit sneaking through.

I think I'll probably go with the first approach. It takes about two hours to scan my whole collection. There are around 300 songs that I'll need to manually retag (actually fewer, because there are a few [non-album tracks] that for some reason have release MBIDs that are 404s now; those should just be looked up as standalone recordings). It looks like a bit over 10,000 songs will be updated!

One thing I'm still thinking about is how the JSON override files should be named and how they should reference the underlying song files. It seems like they need to be named after (or at least include) the underlying song's filename, since otherwise there's no way to figure out where to go for the base metadata when nup update sees a change to an override file.
Probably override files ought to be named after song files, e.g. artist/album/01-song.mp3.json. If they're named after SHA1s, then there's no way to look for override files when calling files.ReadSong() with onlyTags set to true, since the SHA1 won't be computed. Naming them after song IDs is even worse, since those are assigned by the server.
This feels a bit hokey since an override file would need to be renamed if the underlying song file is renamed, but that's probably not the end of the world -- the whole point of this is to avoid needing to make changes to song files since it results in full uploads to GCS.
I should probably add a new check to nup check to make sure there are no override files that no longer have corresponding song files.

I think that this works well enough now:

nup metadata -scan to get updated metadata from MusicBrainz and write override files
nup metadata -set-album-id <id> <dir> to update the MB release associated with a directory of songs


fix: Add types of GroupChat and OpenChat for SendbirdChat

Description Of Changes
Issue

Customer had to use the type assertion for removing type error

const globalState = useSendbirdStateContext()
const getSdk = sendbirdSelectors.getSdk(globalState) as SendbirdGroupChat;
getSdk.groupChannel.getTotalUnreadMessageCount();

Fix

Specify the getSdk returns SendbirdGroupChat or SendbirdOpenChat

@HoonBaek
FYI, the type of module added to the SDK can be defined as follows!
import type { ModuleNamespaces } from '@sendbird/chat/lib/__definition';
import type SendbirdChat from '@sendbird/chat';
import type { GroupChannelModule } from '@sendbird/chat/groupChannel';
import type { OpenChannel } from '@sendbird/chat/openChannel';

export type SendbirdChatSDK = SendbirdChat & ModuleNamespaces<[GroupChannelModule, OpenChannelModule]>;

@bang9 Thanks for good suggestion, I'm curious
ModuleNamespaces<[GroupChannelModule, OpenChannelModule]>

is it okay in the case that the SendbirdChat contains only one of the modules
I guessed this is more preferred
ModuleNamespaces<[GroupChannelModule | OpenChannelModule]>

I couldn't be wrong, so point it out, I really appreciate

@HoonBaek The module namespace type accepts an array of modules as generics.
If an SDK instance should only contain one module, it is correct to receive only one.
If two SDK instances each have only one module, one for open channel and one for group channel, the types should also be separated. (Typically, types should follow the actual code.)


findOne/findOneOrFail should work without where param or with empty where param

Is your feature request related to a problem? Please describe.
In my unit tests, I commonly set up some stuff like a single user for my web app. I know for certain that there is only one record in a given table. I often don't want or need to write logic for picking the exact record based on where statements, I just want to retrieve the first (and only) record in the most straightforward manner possible.
To achieve this goal, it would be nice if I could trivially run em.findOneOrFail(User) or even em.findOneOrFail(User, {}) in my tests, but I get the You cannot call 'EntityManager.findOne()' with empty 'where' parameter error.
I'm not sure why this error exists. Is it to prevent users from mistakenly constructing a query that doesn't have a where parameter?
em.findOne/em.findOneOrFail without a where parameter intuitively makes sense to me. It means "find me one _____, whichever one comes first," similar to JS iterables' find method.
If preventing users from shooting themselves in the foot is the main goal, forcing them to pass the where parameter but allowing an empty object--as in, em.findOne(User, {})--would be a good compromise.
Describe the solution you'd like
The ability to use findOne and findOneOrFail without a where parameter or with an empty object for a where parameter. The behavior would be defined as select `t0`.* from `table` as `t0` limit = 1.
Describe alternatives you've considered
I know that I can use find to get around this limitation, but it's cumbersome to use find, pull the first element from the array, and test that it's not undefined.

I'll add one more thing, the current behavior is inconsistent with the TS typings. The method signature allows you to pass an empty where param object; em.findOne(User, {}) is valid TS. The error only comes up at runtime.
If the intention is to prevent empty findOne/findOneOrFail queries, then this issue is actually less of a feature request and more of a bug report because the types should ensure that the where param object is populated.
That said, I would prefer that the requirement is loosened to allow for empty where param objects or perhaps no where param at all.