Support incremental compilation

Currently, Reflekt does not support the incremental compilation process. We have to get all compiled instances per module and merge old results with new ones

Let me tell about the current situation with this issue.
Incremental compilation is not directly a compiler API. This means that currently, the compiler API does not provide an opportunity to interfere with the incremental compilation process, namely to do something with files that have not actually changed but must be recompiled in the plugin.
Consider an example, since we are actually replacing the Reflekt queries and replacing reflection to IR generation, then we do not contain Reflekt calls in the compiled code. Let's say, before changing the project, we were looking for classes with annotation B, and after the first round of compilation, we added another class C with annotation B to the project. But since the IR for the first Reflekt call has already been replaced, and in fact, the file with the Reflkt call itself has not changed, we do not have its access during the second round of the compilation process. In the result, we will not change the replaced IR and the result will be incorrect since it will not contain the new class C.
We tried to find weak points to change this, but unfortunately, this is not possible at the present time.
However, this functionality can be useful not only in Reflekt, but also to other plugin developers. Therefore, the ability to interfere with the incremental compilation process from a compiler plugin will be added in the next Kotlin releases.
At the moment, we have plans to add the necessary changes in Kotlin 1.7.0 (firstly for Gradle and JVM target, next, support for JPS and other targets like JS).
To summarize, we are doing everything possible to make Reflekt available for use with incremental compilation, but we cannot support it before version 1.7.0 (or maybe a little later)


Move Udacity courses to includes & add structured data

All of the Udacity courses should be available as includes and include the structured meta-data.
https://developers.google.com/search/docs/guides/search-gallery
/cc: @PaulKinlan

Done!


fails to connect after restoring peripheral from state

I have noticed at times that when I run the example app with the same transmitter ID as a previous session, the app fails to scan or connect. Rather than seeing the expected log messages (Scanning for peripherals or Reconnecting to known peripheral ... in 180.0 s) I just see the message Restoring peripheral from state .... This is on latest dev and iOS 11.2.5.

This is intended behavior; state restoration is spinning up correctly.  Are you actually seeing issues if you wait for up to the 5 minute cycle?

Yeah, I’ve tried waiting for several 5 minute cycles and it never connects.

I investigated a bit further, and found that when the peripheral is restored from state, the CBPeripheralState is .connecting. So it falls over at this guard statement and we don't end up scanning.
I tried disconnecting from the peripheral in deinit (i.e. like before #72) but no difference. I'm not sure what the best solution is; would it work to add .connecting to the list of acceptable states?

Guessing in the dark: there could be an issue where a restored CBPeripheral has a state of .connecting but that connect request needs to be re-instated. This sounds like a potential Core Bluetooth bug.
A fix could be to allow BluetoothManager to re-ask a peripheral in the .connecting state to connect; I can't imagine any negative consequences of that.
See https://github.com/LoopKit/xDripG5/tree/allow-reconnect-when-connecting for a diff that tweaks the guard statement slightly for that case.

Should do the trick. Thanks.

Haven't had any issues since #82, which implements the new guard statement. Thanks. Closing.


Linux capabilities

Address #80.

This feels very Linux-specific. What do you think about adding a new argument flag that skips the check, and then it is up to the user to make it work anyway

If we do it like this I think it makes sense to exit only after all checks are done, so that the user does not keep getting new errors after fixing the first one.
capng_get_caps_process failure should fall back to the old uid 0 check also I think

This feels very Linux-specific. What do you think about adding a new argument flag that skips the check, and then it is up to the user to make it work anyway

That can be an additional argument that does not intersect with proposed changes. Proposed changes allow more granular capabilities checking and reporting.

If we do it like this I think it makes sense to exit only after all checks are done, so that the user does not keep getting new errors after fixing the first one.

Thank for the suggestion, forced-pushed with this change, please check.

capng_get_caps_process failure should fall back to the old uid 0 check also I think

Nope. Having UID 0 doesn't guarantee having all the required capabilities. On Linux root (UID 0) can also be restricted.


Added package for using Cli tools from MsBuild like a NSwag.MSBuild

Hi!
I've added a new package for using CLI tools from MsBuild for make swagger.json after every build. It's necessary for CI\CD in my project.
It uses same idea as https://github.com/RicoSuter/NSwag/wiki/NSwag.MSBuild

@domaindrivendev
Hi!
Could you please review these changes?

Hi, can you port this PR to my fork please https://github.com/Havunen/DotSwashbuckle

Thanks for contributing - if you'd like to continue with this pull request, please rebase against the default branch to pick up our new CI.


Implement E2E Tests to prove that SDK works with Engine

Add End to End Tests to ensure that everything is working:

[x] ApplySimulation
[x] GetSimulation
[x] ListSimulation
[x] DropSimulation
[ ] Step Simulation
[ ] Get/Set Attributes.

Scope of E2e Tests:

All attributes with data types except char.
No implicit type conversion.
Multiple Systems with if/elif/else statements

E2E tests have been scoped down as #45 has been reprioritized to v0.2.
These E2E tests should only verify v0.1 functionality.

Blocked by PR #51 due to #53.


fix: add prettire cache

fix: add prettire cache

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


feat: create versioned purls from OSV

Re-open for #897

LGTM though we might consider an assert to agitate this in existing test.

I'll create a follow up PR.


[REF] Refactor tests

See also the following PR review:

https://github.com/neurobagel/federation-api/pull/30

Changes proposed in this pull request:

This PR contains some refactoring of tests which was suggested in the review for the linked PR, but was not successfully pushed to the PR branch before it was merged 🙃😬 (see files changed for differences)

Checklist

[x] PR has an interpretable title with a prefix ([ENH], [FIX], [REF], [TST], [CI], [MNT], [INF], [MODEL], [DOC]) (see https://neurobagel.org/contributing/pull_requests for more info)
[ ] PR links to GitHub issue with mention Closes #XXXX
[x] Tests pass
[x] Checks pass

For new features:

[ ] Tests have been added

For bug fixes:

[ ] There is at least one test that would fail under the original bug conditions.

:rocket: PR was released in v0.1.0 :rocket:


fix: wrong artifact env path in dockerfile

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[ ] N
[x] Y

fix https://github.com/kcl-lang/kcl/issues/1239

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
121

Relevant Lines:
327

💛  - Coveralls


ArangoDB Starter --ownAddress ignores port argument

My Environment

ArangoDB Version:        3.3.19
Storage Engine:              MMFiles
Deployment Mode:        Cluster
Deployment Strategy:    ArangoDB Starter
Configuration:               3 virtual servers
Infrastructure:               own
Operating System:        CentOS 7
Used Package:              SUSE or RedHat .rpm

Steps to reproduce

arangodb --ownAddress localhost:4000

Problem:
arangodb binds to localhost:8528 as the following snippet from the output shows:
2018-11-07T10:43:30-05:00 |INFO| Starting arangodb version 0.13.7, build bdac926 component=arangodb
2018-11-07T10:43:30-05:00 |INFO| Relaunching service with id '176d3806' on localhost:4000:8528... component=arangodb
...
Expected result:
arangodb should bind to localhost:4000 as explained in https://docs.arangodb.com/3.3/Manual/Deployment/ArangoDBStarter.html#automatic-native-clusters

Hi @bgaechter,
Thanks for opening an issue. Unfortunately, the starter option --ownAddress that you are using to start the cluster is currently obsolete. Please use the latest option --starter.address and --starter.port to specify the address and port respectively.
arangodb --starter.address localhost --starter.port 4000

Please have a look to latest options by using arangodb --help. For more info you can refer to the latest documentation.
Thanks!

Hi @bgaechter
Just sending a followup, to check if you had the chance to read above Vinay's message? Did you try to use --starter.address instead ?
Please provide a feedback
Thanks

Hi @OmarAyo
Yes, I used --starter.address together with --starter.port and it works but it doesn't solve my problem. I need the coordinator to be available on that specific port while setting the port for the arangoDB starter only changes the port for the initial setup of the cluster. It would be nice to be able to do these kind of things with the helper.
However, I am running arangod with some parameters for now, which works for me.

Closing this as outdated with the availability of ArangoDB 3.7.


cumulocity-freshdesk and included global error handling

Please verify and approve

Added new example dropbox to send email

@mangatrai Would you be able to help review this as well?

Thanks @maam1, will you change the filenames to lowercase and replace the _ with -?

In Step 11 - if possible please correct spellings of freshdesk.

I have made the required changes. Can you please verify and approve the changes

Looks good to me.. @peterlamar i believe branch can be merged unless you see any other issue.

Looks good to me as well. @mangatrai. I just invited you to the repo and granted merge privileges, merge when ready.

Completed.


Update textwrap to version 0.13.0

The newly released version has a new wrapping algorithm which tries to optimize output in narrow columns. Please see https://github.com/mgeisler/textwrap/releases/tag/0.13.0.

Thanks for the update!

Thanks for the update!


Feature(front): Style GUI

Add some colors and fonts from v1
Add (messy) logic for stone styles
Show suggestion
Tile colors ok
Closes #3

Stone alignment bug fixed :) (normally)


allow named unused arguments

When using any kind of polymorphism, it's nice to define all the arguments; even if some aren't used on a specific implementation.
function rect:area(width, height)
  return width * height
end

function square:area(width, height)         -- warning: unused argument height
  return width ^ 2
end

but luacheck would complain that height isn't used.  Some editors insert a "rename to '_'" suggestion, but that eliminates the argument's documentation value.
Proposal:  consider any name starting with _ as "unused", suppressing the warning when it's not used, and even adding a warning if it's actually used:
function square:area(width, _height)         -- ok
   return width ^ 2
end

function triangle:area(width, _height)
  return width * _height / 2                 -- warning: using argument with "unused" hint
end

I like this idea, although my approach is a little different. Instead of including all args and prefixing them I just name them all the same thing, _, which can be re-used in arguments for as many fields as you want to ski. It's a bit less tempting to use an "unused" variable this way:
function triangle:area(width, _)
  return width ^ 2
end

Yes, using _, or just omitting the unused args (if they're the last ones) do work; but as mentioned, the names have a useful purpose in making the code readable.  The alternative that I've seen used (besides just silencing the warning) is to comment the names, like
function square:area(width, _ --[[ height ]] )
   return width ^ 2
end

but it's ugly, much harder to read, and syntax highlighting makes it worse.

@javierguerragiraldez Happy to merge a patch if you implement the feature! Nobody is currently actively developing luacheck, but I'm shepparding it with maintenance releases as people send in PRs, while we don't have a new maintainer.
We should probably move it to @lunarmodules, btw!

I agree, @lunarmodules is the likely home for this. I'm still willing to chip in on maintenance duties. See thread about repo ownership.
Since GitHub isn't cooperating with migrating the repository ownership I propose we post a new (non-fork) repository on @lunarmodules and work on transferring issue and PRs there manually. I can get that ball rolling now.

I take that back, I guess the best steps would be to migrate this repo to @lunarmodules and then ask GitHub to disconnect it from the original so it isn't marked as a fork any more.


feat: transformer embed support

Related to this. in Finetuner we want to delegate embed to docarray. While transformers is not supported yet

as was discussed with @davidbp will fix weaviate storage errors after #93


test: Speed up test_viewer tests, split out pyqt5/pyside2 tests again

Description
Been playing with test speed locally and found that one of the biggest offenders was test_viewer, specifically add_layers.  It had a 3-factor parametrization (looked like two, but was three due to the keybindings for loop), meaning it was running many hundreds of times, each creating a new viewer (expensive).  When running tests locally with xdist, all of my CPUs would be on full blast right up until it hit test_viewer::add_layer, at which point something choked, and all the processes crawled to a halt.  By removing that parametrization it sped right through the test.
I know it's slightly nicer to be able to test each key binding it isolation, but i think we need make_napari_viewer to be better in order for that to work here.  This compromise (in addition to parallelizing pyside2/pyqt5 envs) has been getting tests back down to around 10 mins in testing over in my fork.  lets' see how it does here
Type of change

[ ] Bug-fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

References

How has this been tested?

[ ] example: the test suite for my feature covers cases x, y, and z
[ ] example: all tests pass with my change
[ ] example: I check if my changes works with both PySide and PyQt backends
as there are small differences between the two Qt bindings.

Final checklist:

[ ] My PR is the minimum possible work for the desired functionality
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] If I included new strings, I have used trans. to make them localizable.
For more information see our translations guide.

I know it's slightly nicer to be able to test each key binding it isolation, but i think we need make_napari_viewer to be better in order for that to work here.

Why cannot use the for loop inside the test? If the only problem is creating a viewer then we could perform all tests in one function.

Why cannot use the for loop inside the test? If the only problem is creating a viewer then we could perform all tests in one function.

that's exactly what I do here.

as an aside, this PR includes a couple changes to let pytest-xdist work.  However, I don't think we should turn it on.  For posterity, here is a run with it on.  If anything it shaves no more than about 10% off the time (there's only 2 cpus available on GHA), and it comes at the cost of possible concurrency failures.
Using locally on my computer though, it flies!  with the entire test suite pytest -n auto napari wrapping up in 1m34sec ... compared to 3:47 without xdist.  So it's definitely nice to have as a local option

Ha, good occasion to use https://pytest-viewer.fly.dev/gh/napari/napari/pull/4646

Test add_layers goes from using 28% of the test suite to 5%, next offender (after test_examples, being test_mgui).


lxqt.libfm-qt: 2.0.1 -> 2.0.2

Description of changes
Diff: https://github.com/lxqt/libfm-qt/compare/2.0.1...2.0.2
Things done

Built on platform(s)

[x] 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/)
24.05 Release Notes (or backporting 23.05 and 23.11 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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 310175 run on aarch64-linux 1

6 packages built:

lxqt.libfm-qt
lxqt.lximage-qt
lxqt.lxqt-archiver
lxqt.lxqt-qtplugin
pcmanfm-qt (lxqt.pcmanfm-qt)
lxqt.xdg-desktop-portal-lxqt


Explain our guidance on managing .NET Core SDK Installers

Over time, users can have a large variety of .NET Core SDKs installed on  a single machine.
We need to explain which to keep installed, and which can be safely removed.
In summary, the .NET Core SDK is backwards compatible and the newest one on the box is used unless there is a global.json file that indicates another version of the SDK.
So, the short answer is:

You need the one you want to run (latest suggested and previews should be fine)
And, any that are specified in global.json

If a SDK version is in a global.json on your machine, and you do not have that version, you will roll forward across patch, but not feature versions of the SDK. Except that the roll forward behavior is overly aggressive in the the 2.1.x -> 2.1.300 range. For that range, we recommend you not do roll forward but have the specified version available.
/cc @richlander @KathleenDollard

If I have a .Net Core 2.0 app, is there any benefit (not just features but also fixes, especially if there will be future security fixes as there were already in the past) to use the newer .Net Core 2.1 SDKs like e.g. 2.1.300 SDK or should I stick to the old 2.1.200 SDK for .Net Core 2.0?

@bergmeister There is a difference between "upgrading your application from 2.0 to 2.1" and "using newer tooling to build a 2.0 application".
Both are possible, you can use the 2.1.300 SDK to build applications for 1.0, 1.1, 2.0 and 2.1.
Using the new SDK will likely give you incremental performance boosts when used locally for development purposes.
If you only install the 2.1.300 SDK on a machine, you will also need to install the 2.0 Runtime (+ ASP.NET Core Runtime Store, depending on the platform) because otherwise you won't be able to run the application (e.g. dotnet run will fail but dotnet build should work). This is only because the new SDK installer bundles a 2.1 runtime but no 2.0 runtime.

This item should focus only on:

which versions I need to keep on my machine
how do I uninstall SDKs I don't need anymore per OS

Related to #5304

As a suggestion, maybe the SDK could have a --prune command to remove all but the latest version.
Or specify the minimum version number to keep. E.g. --prune 2 would delete all the v1.x SDKs I have on my machine. Or --prune 2.1 would remove all the 2.0 ones and the 2.1 preview ones.
For example, my SDK list currently looks like this:
Bart$Barts-MacBook-Air ~ ➜  dotnet --list-sdks
1.0.0-preview2-003131 [/usr/local/share/dotnet/sdk]
1.0.0-preview2-003148 [/usr/local/share/dotnet/sdk]
1.0.3 [/usr/local/share/dotnet/sdk]
1.0.4 [/usr/local/share/dotnet/sdk]
2.0.0 [/usr/local/share/dotnet/sdk]
2.1.101 [/usr/local/share/dotnet/sdk]
2.1.200 [/usr/local/share/dotnet/sdk]
2.1.300-preview2-008533 [/usr/local/share/dotnet/sdk]
2.1.300-rc1-008673 [/usr/local/share/dotnet/sdk]
2.1.300 [/usr/local/share/dotnet/sdk]
2.1.302 [/usr/local/share/dotnet/sdk]

Can someone enter this, or up vote if it exists, this issue in the dotnet/cli repo. This should definitely be on our radar.

@bgever See comment above. Do you want to make that issue on dotnet/cli?

@BillWagner @KathleenDollard  See https://github.com/dotnet/cli/issues/9736


Integration tests with SQL Server

Equivalent to existing MySQL, MariaDB, PostgreSQL and SQLite at https://github.com/prisma/prisma/blob/master/.github/workflows/test.yml

Done by @eemmiillyy and @madebysid


THU/scenario_management_scripts

Add a python script to make scenarios removal easier.
Note: this script does not handle the change of scenarios' parent id yet.

These scripts are DevOps tools and will be placed in a different repo to keep this one focused on the webapp.
Closing the PR.


add logo image

Thanks for opening this PR!
A deployment should be available at https://30.qrcode.yaoss-collection.com/ shortly.
Have a great day!


Implement retry logic for Push Notifications

When sending Web Push Notifications, the status code of the endpoint might return some unsuccessful status code (e.g. TooManyRequests).
Many of those unsuccessful requests will return a Retry-After header to resent the notification at a later time.

I would love to know if one is still able to receive a push even if it has been send while the device has been powered off (or had no network of some kind)

I guess it depends on the device platform. On windows with Firefox I noticed that I got the notification when I turned on the Pc. I would guess on iOS you will not receive it, but I am not sure.


improvement(semantic/cfg): better control flow for switch statements.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#3547  👈
#3519 : 1 other dependent PR (#3238 )
#3473
#3454
#3472
#3462
#3453
#3452
#3451
#3381
#3407
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @rzvxa and the rest of your teammates on  Graphite


SC settings for ApiServerSource's Receive Adapter's container/deployment

For 4.12+ we apply the SCC, and on 4.11 and lower we do ignore this

/retest

/retest

/retest

/hold

/unhold

/retest

/cherry-pick release-v1.8

/cherry-pick release-v1.9


October cms

so it can be used with octobercms?
and if so can you give a small example how.
thank you.

You can try)
But it developed for native laravel auth. If octobercms used it all will work.

For authentication using this plugin.
https://github.com/rainlab/user-plugin

Can you just try this bridge?) If something will not work I will try to fix.


progress callback only returns 100%, progress is not updated in .NET

It seems progress is only reported when file is uploaded, If  console.log progress I get 100% only, never 5%,10%...Backend is ASP.NET web api. angular version 1.3.19

You might need to configure your server to report progress event.

If the same code you have is working when uploading to http://angular-file-upload-cors-srv.appspot.com/upload then it is a server issue.

@imperat0r Have you included the shim? We were experiencing the same issue and found that including the shim fixed it.


Odd historical behavior in US Data

3T days to double in February?

when there is no cases, time to double is infinitely. :)

Oh yeah, makes sense! Do you think it’s an odd UI, since it flattens the current data to the button of the graph?
Perhaps “infinity” could be represented without a line on the graph...  a mouse hover in that region would reveal the infinite days to double, but the graph itself wouldn’t re-scale? Or we could not prevent the date slider from going back that far? I find it odd to be scrolling back, looking for data trends, then all of a sudden the line goes flat b/c scale is now 0 - 3T

I'll say we can wait on this. Plenty of larger features to build.  see this:
https://github.com/xunhuang/covid-19/issues/121
On Sat, May 9, 2020 at 11:02 AM tumbleshack<EMAIL_ADDRESS>wrote:

Oh yeah, makes sense! Do you think it’s an odd UI, since it flattens the
current data to the button of the graph?
Perhaps “infinity” could be represented without a line on the graph... a
mouse hover in that region would reveal the infinite days to double, but
the graph itself wouldn’t re-scale? Or we could not prevent the date slider
from going back that far? I find it odd to be scrolling back, looking for
data trends, then all of a sudden the line goes flat b/c scale is now 0 - 3T
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/xunhuang/covid-19/issues/118#issuecomment-626213939,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFSGFYV6Z2Z6LC5ARE5CVTRQWLEVANCNFSM4M44HQCA
.


136 add support for adding a batch of users to a group

This closes #136. @rafatower CR this, please.

essentially ok with it, changes at your discretion

@rafatower thanks, merging!


Request Body is Always Empty?

I'm attempting to POST a message body to a Sinatra app running under Puma but the body is always reported in Sinatra as malformed/empty.  Making the same request via curl works as expected.  I've also tried with GET as my verb with the same result.
ex. hey -c 1 -n 1 -m POST -d "some text" http://localhost:4567/post-test
I get the same result when I try to pass a file as the body with -D
ex. hey -c 1 -n 1 -m POST -D "test.json" http://localhost:4567/post-test
My Sinatra app ~=
class App < Sinatra::Base

  post 'post-test' do
    puts request.body.read
  end

end

Sinatra logs the following:
2016-11-14 16:30:14 -0800: HTTP parse error, malformed request (): #<Puma::HttpParserError: Invalid HTTP format, parsing fails.>
2016-11-14 16:30:14 -0800: ENV: {"rack.version"=>[1, 3], "rack.errors"=>#<IO:<STDERR>>, "rack.multithread"=>true, "rack.multiprocess"=>false, "rack.run_once"=>false, "SCRIPT_NAME"=>"", "QUERY_STRING"=>"", "SERVER_PROTOCOL"=>"HTTP/1.1", "SERVER_SOFTWARE"=>"puma 3.5.2 Amateur Raccoon Rocketry", "GATEWAY_INTERFACE"=>"CGI/1.2"}
---

Am I missing something obvious?

I’m seeing extraneous characters inserted into the request body. E.g. if I run this against a simple server that logs requests
$ hey -n 1 -c 1 -m POST -d 'hey world' http://<IP_ADDRESS>:50000

I see this come in:
POST / HTTP/1.1
Host: <IP_ADDRESS>:50000
User-Agent: Go-http-client/1.1
Transfer-Encoding: chunked
Content-Type: text/html
Accept-Encoding: gzip

1
h
8
ey world
0

How to close "Transfer-Encoding: chunked"?

Hello there,
I meet the same problem, the post request body is empty. Has it been fixed ?

For me -d worked as specified, but no ruby on the server side

The same problem:
hey -m POST -T "application/json" -H "Authorization: Key XXX" -d '{"x": 0, "y": 1}' server-url

But the server receives following header and body:
Header:
'Authorization': 'Key XXX'
'Transfer-Encoding': 'chunked'
'User-Agent': 'Go-http-client/1.1'
'Host': 'server-url'
'Content-Type': 'application/json'
'Accept-Encoding': 'gzip'

Body:
'' (empty string)

Why is the body not sent to the server? Also working with curl.


[DEVOPS-421] re-sync configuration.yaml

revert 6f211f359faa985c584b5dc6c6646f045cf81930
configuration.yaml came from cardano 824b2303a5dd0058e617a351a1667fa66973d919:node/configuration.yaml

Superseeded by https://github.com/input-output-hk/daedalus/pull/513


How do i add botplay to the settings?

Describe your problem here.
I need to know this, please help lmao
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No response

okay, where do you want you option to be? gameplay, visuals, etc.

okay, where do you want you option to be? gameplay, visuals, etc.

gameplay's the best otpion for me

		var option:Option = new Option('Botplay',
			"If checked, botplay is turned on.",
			'botplay',
			'bool',
			false);
		addOption(option);

put this in GameplaySettingsSubstate.hx

`public static var botPlay:Bool = false;`

put this in ClientPrefs.hx

public static var botPlay:Bool = false;
FlxG.save.data.botPlay = botPlay;
		if(FlxG.save.data.mechanics != null) {
			mechanics = FlxG.save.data.mechanics;
		}

put these bad boys in ClientPrefs.hx

souce/Clientprefs.hx:149: characters 4-13: unkown indentifier : mechanics
it appears that

Oop I forgot to change that 💀

Oop I forgot to change that 💀

well if i reemplaze "mechanics" with "botplay!", it'll show but, by SOME reason, i cant check it, it doesn't let me

You forgot to do something/did it wrong

Oop I forgot to change that 💀

well if i reemplaze "mechanics" with "botplay!", it'll show but, by SOME reason, i cant check it, it doesn't let me

You have to replace it with botPlay, not botplay

Oop I forgot to change that 💀

well if i reemplaze "mechanics" with "botplay!", it'll show but, by SOME reason, i cant check it, it doesn't let me

You have to replace it with botPlay, not botplay

So yeah, now i can check it but, when you check it, it does nothing like, it stills without the botplay

That was just the option part 💀

Ok

Let me create a tutorial

Let me create a tutorial

alright

Do you have vs code?

Do you have vs code?

yes, i code with that

Tutorial: In case that you have Vs Code (VSC), go to PlayState.hx, go to line 301 and copy and paste this code:
cpuControlled = ClientPrefs.botPlay;

Wither362

stills without working for some reason

well i'll discover later, thank you either way guys


Re-enable disabled op-e2e tests

Updates e2e tests to point to https://github.com/bobanetwork/v3-erigon/pull/83

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (77d4417) 41.71% compared to head (00a09d0) 28.30%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff              @@
##           develop     #117       +/-   ##
============================================
- Coverage    41.71%   28.30%   -13.41%
============================================
  Files           74      167       +93
  Lines         4898     7270     +2372
  Branches       765     1307      +542
============================================
+ Hits          2043     2058       +15
- Misses        2749     5106     +2357
  Partials       106      106

Flag
Coverage Δ

cannon-go-tests
82.20% <ø> (ø)

chain-mon-tests
27.14% <ø> (ø)

common-ts-tests
26.72% <ø> (ø)

contracts-bedrock-tests
0.63% <ø> (?)

contracts-ts-tests
12.25% <ø> (ø)

core-utils-tests
44.03% <ø> (ø)

sdk-next-tests
38.59% <ø> (ø)

sdk-tests
38.59% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 93 files with indirect coverage changes


CellularPdnConfig_t has const data members.

In cellular_types.h the CellularPdnConfig structure has three const char fields making it impossible to use this structure without violating constness.
typedef struct CellularPdnConfig
{
    CellularPdnContextType_t pdnContextType;                   /**< PDN Context type. */
    CellularPdnAuthType_t pdnAuthType;                         /**< PDN Authentication type. */
    const char apnName[ CELLULAR_APN_MAX_SIZE + 1 ];           /**< APN name. */
    const char username[ CELLULAR_PDN_USERNAME_MAX_SIZE + 1 ]; /**< Username. */
    const char password[ CELLULAR_PDN_PASSWORD_MAX_SIZE + 1 ]; /**< Password. */
} CellularPdnConfig_t;

apnName, username, and password all need to be changed to char.

@thirtytwobits
Thank you for reporting this issue. We create a PR #168 to address this issue. Once it is merged, we will update in this thread again.

The PR #168 is merged. Thank you for reporting this issue.

Thanks so much!


[ERROR:flutter/lib/ui/ui_dart_state.cc(157)] Unhandled Exception: FormatException: Unexpected character (at character 1)

I'm getting this error, using version 1.0.4 of the package. I managed to fix it locally, but from what I saw it looks like it has already been fixed. I would like to clear that doubt.
If it hasn't been fixed, I would like to help.

@DheCastro
I got this error.
That's why i can't build the application.
Can you please help me with this. :)

@Spidy-crypto
Make the request using the Dart http package, like this:
`Future carregarInsta(String username) async {
  Map<String, String> headers = {
    "Cookie":"ig_did=37B5ACB6-3D52-4B98-B16A-6A5F24B07270; mid=XyRQmgALAAHq4JxQuJjmyZYe3PsS; ig_nrcb=1; csrftoken=yhlS5bgCs2Bi7EhrBHfqzUwabtqlXDZK; ds_user_id=1365574483; sessionid=1365574483:PeSeLxzrCayFK3:29; shbid=16784; shbts=1613586527.8306825; rur=PRN"
  };

  var res = await http.get(Uri.encodeFull(url + username + "/?__a=1"), headers: headers);
  var data = json.decode(res.body);
  var graphql = data['graphql'];
  var user;

  }`

Thus, you will have access to the following profile data:
followers:  user['edge_followed_by']['count'].toString()
following: user['edge_follow']['count'].toString()
website: user['external_url']
bio: user['biography']
profileimage: user['profile_pic_url_hd']
Instagram changed some details in the graphql public API call and sending the Cookie attribute in the header, solved my problem.

@Spidy-crypto
Make the request using the Dart http package, like this:
`Future loadInstaProfileData(String username) async {
  Map<String, String> headers = {
    "Cookie":"ig_did=37B5ACB6-3D52-4B98-B16A-6A5F24B07270; mid=XyRQmgALAAHq4JxQuJjmyZYe3PsS; ig_nrcb=1; csrftoken=yhlS5bgCs2Bi7EhrBHfqzUwabtqlXDZK; ds_user_id=1365574483; sessionid=1365574483:PeSeLxzrCayFK3:29; shbid=16784; shbts=1613586527.8306825; rur=PRN"
  };

  var res = await http.get(Uri.encodeFull(url + username + "/?__a=1"), headers: headers);
  var data = json.decode(res.body);
  var graphql = data['graphql'];
  var user;

  }`

Thus, you will have access to the following profile data:
followers:  user['edge_followed_by']['count'].toString()
following: user['edge_follow']['count'].toString()
website: user['external_url']
bio: user['biography']
profileimage: user['profile_pic_url_hd']
Instagram changed some details in the graphql public API call and sending the Cookie attribute in the header, solved my problem.

@Spidy-crypto
Make the request using the Dart http package, like this:
Future<void> loadInstaProfileData(String username) async {

      Map<String, String> headers = {
        "Cookie":"ig_did=37B5ACB6-3D52-4B98-B16A-6A5F24B07270; mid=XyRQmgALAAHq4JxQuJjmyZYe3PsS; ig_nrcb=1; csrftoken=yhlS5bgCs2Bi7EhrBHfqzUwabtqlXDZK; ds_user_id=1365574483; sessionid=1365574483:PeSeLxzrCayFK3:29; shbid=16784; shbts=1613586527.8306825; rur=PRN"
      };

      var res = await http.get(Uri.encodeFull(url + username + "/?__a=1"), headers: headers);
      var data = json.decode(res.body);
      var graphql = data['graphql'];
      var user;

      }

Thus, you will have access to the following profile data:
followers:  user['edge_followed_by']['count'].toString()
following: user['edge_follow']['count'].toString()
website: user['external_url']
bio: user['biography']
profileimage: user['profile_pic_url_hd']
Instagram changed some details in the graphql public API call and sending the Cookie attribute in the header, solved my problem.
Greetings and a hug!
Hope this helps.

Getting the same error with version 0.1.4 think it could be a request throttling error from instagram is it worked the first few times i tried and then i started getting this issue.

Yes, I have the same error with version 0.1.4.
It work only for few times. Now this error.
Can you fix it?

User_token does not exist in the api connection used. Instagram only allows a few shows. When I examine the code in the library, there is no auth key or user_token. I am thinking of reviewing the Instagram Api documentation and making a contribution.

The old public API is deprecated (on Oct 2020). We should use the oEmbed API.
It seems we have to do something like
https ://graph.facebook.com/v10.0/instagram_oembed?url=https ://www.instagram.com/p/abc/&html&access_token=12345

Haven't looked into the Packages' code so far but it seems we have to add the following to eliminate such request limitations:

Facebook Developer Tokens
Authentication for Instagram Users to get access_tokens and view details of private Profiles

According to this: Instagram Basic Display API
@yusuftogtay are you already on it? Maybe i would look into it too if i find time.
@viralvaghela would you consider merging a pr with these changes? Or do you have another scope for this package?

Hello, I have looked into package code, and that public API is still working fine

@viralvaghela Yes, it's true. But after few requests from the app where we have implemented your package, we cannot get the information anymore. It seems due to new Instagram policy about API

Yes, that still works.
But Instagram has a limitation for, i believe 100 Requests per hour. It depends on the type of App one is developing if that is enough.
But while developing, where hot reload may be triggering alot of requests that limit is reached very quickly.
Maybe a clear Error Exception or a note in the Readme would be enough, so that new devs using the package remember to not trigger getProfileData() on each widget build but instead i.E. on App open and then on manual reload.

Thanks, @noelli  , if you want you can implement  oEmbed API and contribute to this package :)

Thank you @viralvaghela but I don't have enough time right now. I'm busy with my full-time job and other small project :(

Hello!
I had a similar problem while programming a instagram bot. When running the url in Chrome, with all your own data, it will work, but if you run it with Postman or in Tor (without your data, cookies), it will send you to the log in view.
How to solve it?
I just collected the cookies of my Chrome Browser and implemented it in the request.
So I checked which cookies are needed and found out, that only the sessionId, rur and shbid are needed.
rur=FTW; shbid=480
The sessionId is almost unique, but sending the same again, does not produce any damage.

@viralvaghela
could you edit the getProfileData method to something like this:
Future<void> getProfileData(String username, String sessionId) async {     var res = await http.get(Uri.encodeFull(url + username + "/?__a=1"), headers: {"rur":"FTW","shbid":"480","sessionid":sessionId});     var data = json.decode(res.body);     var graphql = data['graphql'];     var user = graphql['user'];     var biography = user['biography'];     _bio = biography;     var myfollowers = user['edge_followed_by'];     var myfollowing = user['edge_follow'];     _followers = myfollowers['count'].toString();     _following = myfollowing['count'].toString();     _website = user['external_url'];     _imgurl = user['profile_pic_url_hd'];     _feedImagesUrl = user['edge_owner_to_timeline_media']['edges'].map<String>((image) => image['node']['display_url'] as String).toList();     this._username=username;   }
The change is in 2nd line

This solved the issue for me...

@ValentinAhrend
Thank you for sharing this.
However can you please share from where do you get the Session ID ?
Can it be generated Client Side or has to be a Instagram Server Side generated Session ID ?
Thanks,
Deep

I received the sessionId using a heroku spring boot application, that uses selenium to login into Instagram and then getting the cookie. You need a logged in account to get this cookie. For me heroku web server was the best solution, because it is free. The users tries to use the currently uploaded sessionId and if it does not work, the sessionId will be updated. The sessionId is available for multiple tries.

@ValentinAhrend
Thanks for the explanation.
So did you use a headless browser ? Can you share the Heroku code with me ?
It did be great if you can so I will not have to troubleshoot Selenium and can focus on better things on the platform I will be working on.
Thanks mate

I made a repo about it. @ProjectAdApp is basically me, so here: https://github.com/ProjectAdApp/isdp
I deleted some stuff for security reasons. No Problem!

hey i just wanted to download some reels in my app and faced the same error is it fixed or do i need to help?

It seems like each network IP is limited X number of requests. Have we confirmed that it's 100 requests? Anyone come up with a workaround?

oh could you please help me find some docs related to api?

oh could you please help me find some docs related to api?

I think this could be good:

thanks for the help

hello ! how did you fix it ? can you please help ?

from where i will get session id??


New Feature: Vertical CarouselRecyclerView 🚀

WORK

Fix #7
Made necessary changes where X coordinates are used
As using both directions requires more lines of code, I refactored those parts making them reusable
Not suitable for ReflectionViewContainer as it is also vertical oriented custom view

DEMO

Thanks a lot @egemenhamutcu, will review the MR.


Adding new things to classes or creating new classes?

Is there any way to add a new variable, method or class to the existing class by using any part of Recaf?

Its on the TODO list

This will be added in 2.11.0

Is there a to add a new field in table/disassembly mode only? I'm working with some class files which do not decompile correctly in any backend, plus I would prefer to work with the disassembly where possible here.

Oh yeah, thats an oversight on my part. At the moment table mode doesn't have the ability to add new items, but it should be an easy fix.


feature: show tooltip titles on touch screens

Description

Adds the touched class to the html body, allowing custom css for touchscreens.
Show tooltips on touchscreens, based on the title attribute of html elements.
I added this as core FM functionality in base.html so plugins can use it, too (this was my primary concern).

Look & Feel

How to test
ngrok http 5000

Use your phone to visit pages with tooltips, such as table items on the /users,  /assets and /asset/<id> pages.

Still needed a changelog entry.


Intégration doesn't work for me

HA core version
2024.3.3
Intergration version
0.3.22
Device type and model
Heatpomp
Used App
NetHome Plus v5.19.0219
The description of problem
2024-04-03 14:12:59.730 WARNING (MainThread) [homeassistant.components.climate] Entity midea_ac_lan.15063309337xxxx_climate (<class 'custom_components.midea_ac_lan.climate.MideaCFClimate'>) does not set ClimateEntityFeature.TURN_OFF but implements the turn_off method. Please create a bug report at https://github.com/georgezhao2010/midea_ac_lan/issues
2024-04-03 14:12:59.730 WARNING (MainThread) [homeassistant.components.climate] Entity midea_ac_lan.15063309337xxxx_climate (<class 'custom_components.midea_ac_lan.climate.MideaCFClimate'>) does not set ClimateEntityFeature.TURN_ON but implements the turn_on method. Please create a bug report at https://github.com/georgezhao2010/midea_ac_lan/issues
2024-04-03 14:12:59.731 WARNING (MainThread) [homeassistant.components.climate] Entity midea_ac_lan.15063309337xxxx_climate (<class 'custom_components.midea_ac_lan.climate.MideaCFClimate'>) implements HVACMode(s): off, auto, cool, heat and therefore implicitly supports the turn_on/turn_off methods without setting the proper ClimateEntityFeature. Please create a bug report at https://github.com/georgezhao2010/midea_ac_lan/issues
2024-04-03 14:13:59.212 WARNING (MainThread) [homeassistant.components.climate] Entity midea_ac_lan.15063309337xxxx_climate (<class 'custom_components.midea_ac_lan.climate.MideaCFClimate'>) does not set ClimateEntityFeature.TURN_OFF but implements the turn_off method. Please create a bug report at https://github.com/georgezhao2010/midea_ac_lan/issues
2024-04-03 14:13:59.213 WARNING (MainThread) [homeassistant.components.climate] Entity midea_ac_lan.15063309337xxxx_climate (<class 'custom_components.midea_ac_lan.climate.MideaCFClimate'>) does not set ClimateEntityFeature.TURN_ON but implements the turn_on method. Please create a bug report at https://github.com/georgezhao2010/midea_ac_lan/issues
2024-04-03 14:13:59.213 WARNING (MainThread) [homeassistant.components.climate] Entity midea_ac_lan.15063309337xxxx_climate (<class 'custom_components.midea_ac_lan.climate.MideaCFClimate'>) implements HVACMode(s): off, auto, cool, heat and therefore implicitly supports the turn_on/turn_off methods without setting the proper ClimateEntityFeature. Please create a bug report at https://github.com/georgezhao2010/midea_ac_lan/issues

The logs
No response

#420 should fix it


Page Won't Show!

I tried loading your site in the latest versions of Chrome, Firefox, and Safari. On all three browsers, the title, favicon, and page background showed, but nothing else. I really want to use your website! Does it work for you? Screenshot of console:

Fixed, sorry for the long, long delay.


[Fix] Analyze logs with single val iteration

Motivation
Make analyze_log.py works with single validation iteration. Currently, the analyza_logs.py use iters = iters[:-1] to get rid of the last validation iteration, but in a very small val set, there maybe only one validation step. In such case, the analyze_log.py won't work, it use print out a blank figure. This PR fixed that.
Modification
When there is only one item in the iters list, then don't slice it.
Checklist
Before PR:

[ ] Pre-commit or other linting tools are used to fix the potential lint issues.
[x] Bug fixes are fully covered by unit tests, the case that causes the bug should be added in the unit tests.
[ ] The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
[ ] The documentation has been modified accordingly, like docstring or example tutorials.

After PR:

[ ] If the modification has potential influence on downstream or other related projects, this PR should be tested with those projects, like MMDet or MMSeg.
[ ] CLA has been signed and all committers have signed the CLA in this PR.

Test with this log file.
python tools/analysis_tools/analyze_logs.py plot_curve 20211028_115735.log.json --keys accuracy_top-1

20211028_115735.log

Test with this log file.
python tools/analysis_tools/analyze_logs.py plot_curve 20211028_115735.log.json --keys accuracy_top-1

20211028_115735.log

Thanks for your contribution.
Here, I think you want to put the log json file, but put a log file by mistake.

20211028_115735.log.json.txt
@mzr1996 Please remove the .txt suffix because github not supported raw json.

I have read your modification.
First, let me confirm your need. You want to analyze the log and plot the accuracy curve, right?
Let me show the code:
            if 'mAP' in metric:    # <-- Here, it's the condition to find the metric in val steps
                xs = np.arange(1, max(epochs) + 1)
                ys = []
                for epoch in epochs:
                    ys += log_dict[epoch][metric]
                ax = plt.gca()
                ax.set_xticks(xs)
                plt.xlabel('epoch')
                plt.plot(xs, ys, label=legend[i * num_metrics + j], marker='o')
            else:                  # <-- Else, it will find the metric in train steps
                xs = []
                ys = []
                num_iters_per_epoch = log_dict[epochs[0]]['iter'][-1]
                for epoch in epochs:
                    iters = log_dict[epoch]['iter']
                    if log_dict[epoch]['mode'][-1] == 'val':
                        iters = iters[:-1]
                    xs.append(
                        np.array(iters) + (epoch - 1) * num_iters_per_epoch)
                    ys.append(np.array(log_dict[epoch][metric][:len(iters)]))
                xs = np.concatenate(xs)
                ys = np.concatenate(ys)
                plt.xlabel('iter')
                plt.plot(
                    xs, ys, label=legend[i * num_metrics + j], linewidth=0.5)

The problem is not the last eval step is skipped, but we forget to add "accuracy" to the condition to find the metric in val steps.
So I think a better solution is to modify the first line to:
            if 'mAP' in metric or 'accuracy' in metric:

And the original code can only work when evaluate in every epoch. So please change the original code to:
            if 'mAP' in metric or 'accuracy' in metric:
                xs = []
                ys = []
                for epoch in epochs:
                    xs.append(epoch)
                    ys.extend(log_dict[epoch][metric])
                ax = plt.gca()
                ax.set_xticks(xs)
                plt.xlabel('epoch')
                plt.plot(xs, ys, label=legend[i * num_metrics + j], marker='o')

Yeah, you are right. I have modified the code according to your suggestion. I think the iters thing still need a sentinel value to prevent it fall into an empty list.


Django 3 support

fixes issues with migrations and on_delete (fixes #25, #26)
adds support for 2.2 and 3.0 (fixes #39)
drops Python 2 and Django 1 (fixes #27)

Supersedes and closes #28, #29, #32, #33, #37

I got an error with the setup encoding in windows with python 3.8, it gets fixed adding the encoding to the descriptions in the file similar to #35
setup(     . . .      description=open('DESCRIPTION', encoding="utf-8").read(),     long_description=open('README.rst', encoding="utf-8").read(),     . . .  )

Coverage decreased (-0.2%) to 85.342% when pulling 9d8929989d84fd2b30daea40691a37ced6fe4944 on fmalina:master into 6bad47509a14d65a3a5a08777455f4cc8b4961fa on arteria:master.

@fmalina The last commit, https://github.com/fmalina/django-openinghours/commit/9d8929989d84fd2b30daea40691a37ced6fe4944, broke my poetry based project which was using fmalina:master.
@arteria, could you please take a look at this, and fast-forward all except the last commit in this PR.

@jayvdb I am using pip3 install git+git://github.com/fmalina/django-openinghours.git#egg=openinghours and it breaks for me without the last change

@jayvdb I reverted it and started to use pip3 install git...#egg=django-openinghours and that seems to work


Google OAuth2 issue

Google OAuth2 not working correctly. I'm getting 401 Client Error: Unauthorized.
SOCIAL_AUTH_GOOGLE_OAUTH2_KEY
SOCIAL_AUTH_GOOGLE_OAUTH2_SECRET
SOCIAL_AUTH_GOOGLE_OAUTH2_SCOPE

keys are set!

Hi, I have some suggestions to help solving this issue!
The following users made pull-requests to related issues:
Maybe it's a good idea to assign them to this issue? Or at least ask them some tips?
Some background now. Here is a list of similar issues:

Problems connecting Google OAUTH2

Backend not found error for google oauth2 after two weeks of inactivity

Unable to refresh google oauth2 token after update python social auth to 0.2.1

Also, these are some files that might need to change:
Hope it's useful!
Cheers,
CinnaBot.

Hi, I have some suggestions to help solving this issue!
The following users made pull-requests to related issues:
Maybe it's a good idea to assign them to this issue? Or at least ask them some tips?
Some background now. Here is a list of similar issues:

Problems connecting Google OAUTH2

Backend not found error for google oauth2 after two weeks of inactivity

Unable to refresh google oauth2 token after update python social auth to 0.2.1

Also, these are some files that might need to change:
Hope it's useful!
Cheers,
CinnaBot.

Hi, I have some suggestions to help solving this issue!
The following users made pull-requests to related issues:
Maybe it's a good idea to assign them to this issue? Or at least ask them some tips?
Some background now. Here is a list of similar issues:

Problems connecting Google OAUTH2

Backend not found error for google oauth2 after two weeks of inactivity

Unable to refresh google oauth2 token after update python social auth to 0.2.1

Also, these are some files that might need to change:
Hope it's useful!
Cheers,
CinnaBot.

Hi, I have some suggestions to help solving this issue!
The following users made pull-requests to related issues:
Maybe it's a good idea to assign them to this issue? Or at least ask them some tips?
Some background now. Here is a list of similar issues:

Problems connecting Google OAUTH2

Backend not found error for google oauth2 after two weeks of inactivity

Unable to refresh google oauth2 token after update python social auth to 0.2.1

Also, these are some files that might need to change:
Hope it's useful!
Cheers,
CinnaBot.

Hi, I have some suggestions to help solving this issue!
The following users made pull-requests to related issues:
Maybe it's a good idea to assign them to this issue? Or at least ask them some tips?
Some background now. Here is a list of similar issues:

Problems connecting Google OAUTH2

Backend not found error for google oauth2 after two weeks of inactivity

Unable to refresh google oauth2 token after update python social auth to 0.2.1

Also, these are some files that might need to change:
Hope it's useful!
Cheers,
CinnaBot.

cxzsjhdshjadshajsd


Bug: doubleing the path

Thanks for the library @buttercubz , suiper helpful.
I think I discovered a bug. When I use env.config() I get the following error:

I think the issue is in line 43 of mod.ts
  const env = Deno.readFileSync(join(Deno.cwd(), path));

The issue is that the path = defaultPath and default Path looks like this:
const defaultPath = join(Deno.cwd(), ".env");

So basically it's joining Deno.cwd() twice...
maybe if we replace line 43 with:
path === defaultPath ? defaultPath : join(Deno.cwd(),path)

what do you think?
Happy to submit a PR

@viztastic for sure any pr is welcome


RangeError: Maximum call stack size exceeded

context: {
			env: {
				app_debug: true,
				app_env: 'local',
				laravel_config_cached: false,
				laravel_locale: 'de',
				laravel_version: '10.47.0',
				php_version: '8.3.3',
			},
			user: {
				first_name: 'Daniel',
				last_name: 'Düsentrieb',
				email<EMAIL_ADDRESS>......
			}
		}

JavaScript Error
RangeError: Maximum call stack size exceeded
    at si (https://...:231:2670)
    at Al (https://...:232:546)
    at bf (https://...:228:186649)
    at https://...:228:221181
    at yp (https://...:228:222533)
    at HTMLUnknownElement.y (https://...:228:103635)
    at Object.Hn (https://...:228:103960)
    at Kn (https://...:228:104921)
    at Ih (https://...:228:270720)
    at uh (https://...:228:261922)

Reproduction Steps
Please tell us what you were doing when this error occurred, so we can more easily debug it and find a solution.

Throw an error in php code.
Wait a very long time for the browser to understand it is in an endless loop.
Ignition page shows with stack trace and everything, but in the context section the user is not showing. Instead, the prompt to report this issue is given:

User Agent
Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36

Check recursive functions
Make sure all recursive functions have proper termination conditions.
Add log output to view the depth and parameters of function calls.
In addition, you should use servbay to troubleshoot development environment bugs, which is easier to use on Mac than tools such as docker and mamp


Feature request: ability to specify ssh port

I would rather not have the ssh on the default 22 port if possible.
I know that i can restrict the source ips via the firewall, but my ip changes often.
Thank you

I just released 1.1.3 which allows you to set the ssh_port option.


Language Server <IP_ADDRESS> - variable info

When hovering over a list variable, this is what I got on version <IP_ADDRESS>:

Which is similar to Jedi's behavior:

However, on version <IP_ADDRESS>, this is what I get:

Is that expected?

Expected in that our current hover implementation doesn't show type documentation, but it can be added.


[T2A4][W13-A4]Lang Yanbin

Ready for review

Some comments added. Please close the PR after reading comments.


Track non-minified source

I suggest this repository contain both the human-readable distribution (after combining) and the minified distribution (which it already does). This would follow convention and make debugging easier.

This project is using Closure Compiler and non-minified versions aren't that useful (since they require the whole Closure compiler setup). They simply won't work. I could add a debug version, but that'll have mangled variable names which makes debugging hard.

This is a serious problem for us because currently I'm getting some strange unhandled Promise rejection error inside of the minified code, and there's no way for me to interpret the problem and solve it :/
Paused on promise rejection: ''. somewhere after Q.start


Added notebook example for Chinese - English

Built a notebook to learn Chinese-English word embeddings with your method. Might be useful to developers/researchers because of the non-availability of common words such as 'london' in the embedding spaces (different alphabets), thus requiring some form of supervision.
As discussed in our Twitter conversation (https://twitter.com/sebastian_wag/status/938716176842141696).

Stale PR, older than 3 years old, automatically closed.


Accessing ChromaDB Embedding Vector from S3 Bucket

Accessing ChromaDB Embedding Vector from S3 Bucket
Issue Description:
I am attempting to access the ChromaDB embedding vector from an S3 Bucket and I've used the following Python code for reference:
# Now we can load the persisted database from disk, and use it as normal.
vectordb = Chroma(persist_directory=persist_directory, embedding_function=embedding)

However, I'm uncertain about the steps to follow when I need to specify the S3 bucket path in the code.
Questions/Clarifications:

What specific modifications or additions are required to access the embedding vector from an S3 Bucket?
Are there any configuration changes needed to integrate S3 access seamlessly?

Additional Information:

Name: chromadb
Version: 0.3.21
Summary: Chroma.
Home-page:
Author:
Author-email: Jeff Huber<EMAIL_ADDRESS>Anton Troynikov<EMAIL_ADDRESS>License:
Location: c:\users\ibm26.conda\envs\llms\lib\site-packages
Requires: clickhouse-connect, duckdb, fastapi, hnswlib, numpy, pandas, posthog, pydantic, requests, sentence-transformers, uvicorn
Required-by:

Your assistance and guidance on this matter would be greatly appreciated. Thank you!
Suggestion:
No response

Any luck on how to do this?


Question: NPM Install Fails (Peer Deps)

What version of drizzle-orm are you using?
<EMAIL_ADDRESS>Question
Just started playing around around with this lib and wow! Great job! I have a strong affinity for type-safe dx-first tools (I'm the author of Type Route ) and this checks a lot of boxes. Really well done!
I'm evualuating this tool for a new project I'm working on. I see the peer deps for drizzle-orm-sqlite specify "@cloudflare/workers-types": ">=3 <4". Just curious why <4 is necessary?

(Didn't mean to add the bug label - just used the issue template)

Hi there! Thanks for the interest in our ORM!
We restrict the versions of our peer dependencies to protect against potential breaking changes in the future versions. We'll bump the dependency version range once we confirm that we support v4.

@bradenhs this should now be fixed in latest 🎉 (0.14.5)


save peptidome analysis state between sessions

store data in local storage
check out if we have to save after every change, or only on page leave
on loading, check if a previous session is stored and offer to restore
in the future, we can maybe offer to save (and name) sessions

Original issue by @bmesuere on Wed Sep 17 2014 at 14:04.
Closed by @bmesuere on Thu Jul 02 2015 at 15:20.

We can use the beforeUnload event to save the session: https://developer.mozilla.org/en-US/docs/WindowEventHandlers.onbeforeunload
The saves should be versioned in case we want to change the save format
Things to save

Loaded genomes (should be easy)
Order ( a bit harder, since we can only set the order after all genomes are loaded and the user could start reordering the genomes himself before everything is loaded)
Selected lines on the pancore graph

Original comment by @bmesuere on Thu Sep 18 2014 at 19:18.

The order part could be simplified if the order set by the main pancore class was the final order.
Currently, the genomes are reordered by the pancore-graph based on the loading/drawing speed.
If we stop doing this, things would become a lot easier. While we're at it, the pancore graph shouldn't access the table object directly.
Original comment by @bmesuere on Thu Mar 19 2015 at 15:13.


Websocket support

Just saw a tweet about how nuxt currently supports websockets! I'm not sure if there is a good standard way of hooking up to it with trpc, but maybe we could get some guidance/support on this, thanks!
Related docs: https://nitro.unjs.io/guide/websocket

This is the existing demo without the new websockets feature.
I looked into nitro websocket and I think it's easy to implement as long as we have access to the websocket object since it's required in the applyWSSHandler function when creating a WS server in tRPC.


Svelte 3.46 gives "Uncaught Error: Function called outside component initialization"

for "framework7-svelte": "^6.3.16" version I have a very simple application here https://github.com/yakworks/framework7-svelte-simple
master works when its tied to "svelte": "3.38.3"
this branch https://github.com/yakworks/framework7-svelte-simple/tree/svelte-3.46
shows the error when upgrading to 3.46. Tried changing from import { App,  View } from 'framework7-svelte'; to import { App,  View } from 'framework7-svelte/esm/framework7-svelte-src'; and still get index.mjs?v=6582916b:948 Uncaught Error: Function called outside component initialization
is it possible to upgrade to svelte or is it tied to 3.38?

fixed in https://github.com/framework7io/framework7/commit/feda228523ba672cac2df49b1d6c6ce7d45e91fd


Support for Azure Active Directory login in MSSQL/Tedious

Would it be possible to support Azure Active Directory (password) logins in the extension?
As per the tediousjs documentation (http://tediousjs.github.io/tedious/api-connection.html), tedious and the version used by node-mssql already supports it.
Where would i look to if i wanted to add a PR for this?

Hey @ransagy is possible for sure.
But I believe we already support this, it's just not documented.
Did you try to add it those options to mssqlOptions like this:
"mssqlOptions": {
  ...
  "authentication": {
    "type": "azure-active-directory-password",
    "options": {
      "userName": "<userName>",
      "password": "<password>"
    }
  },
}

@mtxr You're right, it does work; but only if the password is embedded in the configuration. if i try to use "askForPassword": false and only leave the username part of that, authentication fails.

@ransagy wouldn't you want "askForPassword" to be true if you're not embedding it in the json?

@BobbyMcWho that's just the problem; it doesn't input the password into the specific aad password field in this case, vut in nto

@ransagy I just did a commit. I don't have a way to test it. Would you mind testing and reaching me back?

@mtxr I'll try grabbing the repo and building to test, will get back to you.

@mtxr confirmed working, thanks!

Awesome!
On Sat, Sep 28, 2019, 05:09 Ran Sagy<EMAIL_ADDRESS>wrote:

@mtxr https://github.com/mtxr confirmed working, thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mtxr/vscode-sqltools/issues/375?email_source=notifications&email_token=AAFMX2NSUJRMVTS24EXQRILQL4GKNA5CNFSM4IVFQTX2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD72TSLA#issuecomment-536164652,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAFMX2IDZPST3UDB3SEDSCTQL4GKNANCNFSM4IVFQTXQ
.


Update QJIT VQE demo to use value_and_grad and Catalyst printing

When this demo was originally written, these features weren't available. Adding these features to the demo makes it more efficient (we aren't computing the cost function twice), and also shows users how to print from inside qjit.

Instead, we can use Optax <https://github.com/google-deepmind/optax>__, a library designed for optimization using JAX, as well as the :func:~.catalyst.grad function, which allows us to differentiate through quantum just-in-time compiled workflows.

Should we point to the [value_and_grad](https://docs.pennylane.ai/projects/catalyst/en/latest/code/api/catalyst.value_and_grad.html#:~:text=value_and_grad,-value_and_grad(fn%3DNone&text=A qjit() -compatible transformation,computed within the compiled program.) function here instead?

fixed, thanks @justinpickering!


Usage with Capistrando and capistrano/rbenv plugin

Hi,
I just installed fullstaq-ruby on a new server. Everything worked out fine. I followed the instructions regarding the rbenv integrations, got it working ok. When I write rbenv versions the output shows me the correct fullstaq-ruby installed version:
$ rbenv versions
* 2.7.1-jemalloc (set by /home/emeral/.rbenv/version

I added the line indicating the version in the deploy.rb file:
# config/deploy.rb
set :rbenv_ruby, '2.7.1-jemalloc'

When I try to run cap deploy it looks like it still tries to check if the ruby version is installed on the .rbenvhome folder:
00:00 rbenv:validate
      WARN  rbenv: 2.7.1-jemalloc is not installed or not found in /usr/local/rbenv/versions/2.7.1-jemalloc

I've tried setting the rbenv_custom_path to the fullstaq folder
set :rbenv_custom_path, '/usr/lib/fullstaq-ruby/'

It works initially (it fools this initial check) but then it fails when capistrano tries to use that path to execute rbenv
/usr/lib/fullstaq-ruby//bin/rbenv

Is there something else I'm missing? Thanks!!

Fullstaq Ruby uses a modified rbenv which adds support for system-wide installed Rubies. It looks like capistrano-rbenv is written with the assumption that Rubies are always located in RBENV_DIR/versions, which is not the case in fullstaq-rbenv. You should submit a feature request to capistrano-rbenv.

Ok. I'll explore a solution with the capistrano/rbenv plugin and collaborate with a PR if there's something to add to the documentation here, thanks!!

Hi, I think I've found a way to do it. There's a "hidden" option (at least it was not documented) to set the rbenv_ruby_dir in the capistrano/rbenv plugin.
That, combined with altering a little bit the rbenv_prefix option, so it point to the correct rbenv bin, works ok.  I've already submitted a PR on the capistrano/rbenv plugin to have that option documented (https://github.com/capistrano/rbenv/pull/91).
I've just sumitted a PR to document the neccesary options to add for the capistrano/rbenv plugins https://github.com/fullstaq-labs/fullstaq-ruby-server-edition/pull/47
Thanks :)

Thanks for the pull request @guillehorno, and thanks for figuring out this issue!
I've also done some investigations, and I think the issue can be solved in a better way. Instead of setting rbenv_ruby_dir, we could set:
set :rbenv_custom_path, '/usr/lib/rbenv'

This is the path to fullstaq-rbenv's system-wide installation directory. /usr/lib/rbenv/versions/* are symlinks to /usr/lib/fullstaq-ruby/versions/*. This will, capistrano-rbenv will end up checking whether /usr/lib/rbenv/versions/#{rbenv_ruby} exists.
However, this requires a change on the Fullstaq Ruby side. With the above option, we capistrano-rbenv will end up mapping 'ruby' to /usr/lib/rbenv/bin/rbenv exec ruby. However, /usr/lib/rbenv/bin/rbenv does not exist: we don't package it. So we need to modify the fullstaq-rbenv packaging script (container-entrypoints/build-rbenv-{deb,rpm}) to package /usr/lib/rbenv/bin.
We could make things even better if we modify capistrano-rbenv as well. Imagine that, instead of setting rbenv_custom_path, we could set:
set :rbenv_type, :fullstaq

This would make capistrano-rbenv autodetect that rbenv_path should be /usr/lib/rbenv.
Would you like to give making these changes a try? If not, let me know, and I'll do it instead.
If you do choose to contribute these changes, please don't forget to add a test case as well.

Thanks @guillehorno! I've posted some feedback in the PR.

This has been implemented.


UI code reusability: pagination

I saw there are some pieces of code that are repeated multiple times for Paginating tables. The following piece of code is repeated across:

https://github.com/trustification/trustification/blob/main/spog/ui/src/components/vulnerability.rs#L110
https://github.com/trustification/trustification/blob/main/spog/ui/src/components/vexination.rs#L98
https://github.com/trustification/trustification/blob/main/spog/ui/src/components/advisory.rs#L105

We could enhance the code and allow re-usability of certain logic common across tables.

Thanks for the good description! I agree, it would be nice to better reuse pagination. It could be the number of pages going up or down, but I think this common behavior is good to separate out.


cache schema for selected models

resolves #4688

Description
Cache schema for selected models only instead of caching all schemas in a project when --cache_selected_only option is provided in cli or config. This will help improve startup time when a project has many databases and schema but only few models are selected.
This behavior is similar to how schemas are created currently. Dbt only creates schema (if they don't exists) for selected models only rather than all schema defined in a project.

Checklist

[x] I have signed the CLA
[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[x] I have added information about my change to be included in the CHANGELOG.

@jtcohen6, @ChenyuLInx , I am thinking of adding test to check cached relation after supplying this new flag. Do you have existing integration test to verify that caching is working properly for project with multiple database/schema. I can add a new test to that.

verify that caching is working properly for project with multiple database/schema
@karunpoudel  I don't think we do. Thanks for adding those! Let me know if there's anything I can help with! We haven't made the annocement yet but we are swithing to new testing framework, you can find examples here. And here are some initial docs. This is not fully done so if you feel more comfortable using the original test framework that also works for me. But I highly recommand checking out the new ones.

I'd be happy to have this change as is, as an optional/experimental flag, included in v1.1.0-rc1. We already have a follow-on issue open to add more rigorous testing and tracking (https://github.com/dbt-labs/dbt-core/issues/4961), ahead of this being something we'd want to turn on by default.

Rebased + merged in https://github.com/dbt-labs/dbt-core/pull/5036


Add example Phoenix application

Plus update the gatekeeper-auth example to use the new Electric.Phoenix apis

Awesome! Can you add a note/section to the Phoenix docs linking to and flagging up this example?

Awesome! Can you add a note/section to the Phoenix docs linking to and flagging up this example?

https://github.com/electric-sql/electric/pull/1948/commits/ada89413d82e5ab6e0d4cdf7c6fb8f9af9b4e648
@thruflo how's that?

@magnetised nice, ta. Two niggly suggestions:

rename the example folder to electric-phoenix, the -example prefix just adds noise and we're removing them
move this down to the page, above the help wanted section

So make it a:
## Examples

### electric-phoenix

...

<HelpWanted ...

So we can add multiple examples over time. Ideally with some description of what the example shows (i.e.: the scope of the integration / functionality).
As per https://electric-sql.com/docs/integrations/next#examples (but actually up a heading level).

@thruflo I pulled some of your work out of the gateway auth example and made them part of Electric.Phoenix - specifically the Electric.Phoenix.shape_from_params/2 is based on your Api.Shape.from/1 function and I've added ShapeDefinition.matches?/2 so that our code supports most of the core functionality used in the token matching.

@magnetised great 👍


[small] Use GetValues for SqlServer reverse engineering on Mono

Fixes failing reveng tests on Mono.
cc @lajones

:shipit:


** (RuntimeError) precompiled NIF is not available for this target: "armv7l-unknown-linux-gnueabihf"

I am getting this error while trying to install uuidv7 in Raspberry PI 3

Hi! Thanks for the report. The issue is that armv7l-unknown-linux-gnueabihf isn't enabled by default in the rustler_precompiled library, so the solution would be to add it to the list of target builds

Hi! This issue is fixed by https://github.com/martinthenth/uuidv7/pull/62, and I'm going to release this change as v1.0.0 probably this week.


Auto-tag and publish to NPM via Travis

https://trello.com/c/e1AYHybj/672-put-together-the-copy-and-paste-work-as-a-package-that-can-be-included
This configures Travis to automatically push a new tagged version when
the package version in package.json changes, as well as push the new
version to NPM if it isn't already there. We use different checks for
each function to ensure they can run independently if one of them fails.

Ooh exciting! Nice one wading through all the complexities of this @alex-ju and @benthorner
I see that all the effects kick in on master - is there much that'd be useful to look at before then or shall we yolo merge it? Looks good to me from reading it. Only thing I'd like as an extra is recording github releases too but that can be treated as a later enhancement.

@kevindew I've already switched the branch names to check it works, and all is well :-). We started off by using the releases provider, but a plain old tag creates a release anyway, in the familiar style of other GOV.UK releases, so that seemed preferable. (The main reason for avoiding full releases was that it kept sending me an email every time, and you can't turn it off without sacrificing other notifications.)


Sentry.CrashError without much information to work with

Environment
How do you use Sentry?
Self-hosted/on-premise: 21.8.0 e1ddac6
Which SDK and version?
sentry-elixir 8.0.6

Context
We are getting a lot of Sentry.CrashError ** (exit) killed error reports, but we can't really figure out why those crashes are happening.

Here is the exception section

Sentry.CrashError ** (exit) killed

Sentry.CrashError: ** (exit) killed
  File "gen_server.erl", line 811, in :gen_server.handle_common_reply/8
  File "proc_lib.erl", line 236, in :proc_lib.wake_up/3

(Sentry.CrashError ** (exit) :hb_timeout)

(DBConnection.ConnectionError tcp recv (idle): closed)

As you can see, not much to work with, in some cases, no stack trace at all so it is quite hard to figure out what the issues are.
I setup
config :logger,
  handle_otp_reports: true,
  handle_sasl_reports: true

Which was my last line of defense in figuring this out.
Any thoughts?

I tried setup some metadata to see how it works, but no success so far
config :sentry,
  metadata: [:ancestors]

Commanded user I guess ;)
I think I figured it out, it's because the logger_backend doesn't handle reason {:killed, []} properly, it ends up going to https://github.com/getsentry/sentry-elixir/blob/master/lib/sentry/logger_backend.ex#L105-L107, which results in a call to Exception.format_banner(:exit, :killed, []), which gives "** (exit) killed".
However, I'm not familiar enough with erlang errors and reasons to apply a proper fix here, I'll need to investigate more.

Okay, basically doing this

fixes the problem (includes the full message)
But I have no idea if this is a valid fix. Thoughts?

Is there any way to improve the pattern matching implementation with {other, stacktrace}? it should fallback into that one

Is there any way to improve the pattern matching implementation with {other, stacktrace}? it should fallback into that one

Yeah I guess the if check on line 106 in the above screenshot could be moved into the {other, stacktrace} fallback.

I'm also getting many Sentry.CrashError ** (exit) time out. Is there a way to stop them?

You can try using an event filter (https://github.com/getsentry/sentry-elixir/blob/master/lib/sentry/event_filter.ex) that excludes Sentry.CrashError, but then, depending on what the source of you're errors is, you might exclude too much.

That could be an option, thanks!
Do you know if the cause of the Sentry.CrashError ** (exit) time out will be removed anytime soon?

Do you know if the cause of the Sentry.CrashError ** (exit) time out will be removed anytime soon?

Not sure if my solution is appropriate, maybe someone on the team can have a look and I can cook up a PR.

Okay, basically doing this
fixes the problem (includes the full message)
But I have no idea if this is a valid fix. Thoughts?

@whatyouhide is this a solution to this problem? If so, I can open a PR so we can get rid of this one :)

We're reworking some of the internals, but yes, this is something we can improve. Just need to get a few things re-shaped first 🙃


Drones do not spawn

Hello,
When I start beacon.sh, drones fail to spawn. It looks as below;

This is being addressed in here


Consistently display the preset Theme name in System Settings

Describe the feature
Low priority... make sure the preset Theme names are shown consistently in System Settings.
To Reproduce
Steps to reproduce the behavior:

Go to System Settings.  Select a desktop preset theme, e.g. candy with a single click.
Close System Settings. Then open System Settings again. Note the preset theme is correctly highlighted.
Now double-click on a preset theme. This opens the theme editor (themed.p64).
Without changing any settings, close the editor.
The named Theme is now [custom].
Ideally the original, preset Theme name should still be displayed.

Expected behavior
The preset Theme name should be preserved, if the theme was not edited.
Screenshots

Software (please complete the following information):

OS: macOS
Version: v0.1.0b

Re-tested with release v0.1.0d on macOS. Issue still exists.

Re-tested with release v0.1.0g on macOS. Issue still exists.


Provide parsing of all types of subjectAltName

Currently, we have a pile of custom X.509 parsing that sits on top of mbedTLS, in order to extract otherName subjectAltNames from certificates (Microsoft's ActiveDirectory certificates put the user names in the otherName format).
Even if mbedTLS only provides "full" parsing for dNSName and iPAddress subjectAltNames, it would help just to provide a way to extract the raw ASN.1 bytes of the names. Currently, using the public functions you have to actually manually parse the entire list of extensions in the cert to hunt for subjectAltName, then parse that to hunt for each individual name, which is code that mbedTLS already ships.
If there's appetite for it, I'd be happy to contribute the code.
The suggested API would be to extend the mbedtls_x509_sequence subject_alt_names in mbedtls_x509_crt to contain all the names, rather than stripping the non-dNSName ones. Alternatively, you could just make the local/static parsing functions in x509_crt.c public, so that there's a way to extract the names from v3_ext if you want them.

Hi @NWilson!
I'd welcome the contribution, but as you know we still have a backlog of PRs to absorb, so it may take a little time to accept.

For clarity, @yanesca has just pointed out to me that IP addresses in subjectAltNames is a feature we support and will be merged in the next day or two.

@sbutcher-arm Any idea when support for ipAddresses in the subjectAltNames will be merged?

I'd love to see support for rfc822Name as well.

+1
All names should be stored in subject_alt_names. Filtering printable names could be done in x509_info_subject_alt_name but at least rfc822Name, dNSName and uniformResourceIdentifier can be printed.

I believe this is addressed in #2530

I'm closing this issue, as this is possible with the current state of code. Please see https://github.com/Mbed-TLS/mbedtls/blob/74d6e59e152dda5d4b5e85d2b9efca83ad32619c/tests/suites/test_suite_x509parse.function#L419 for an example.
Further improvements for SAN parsing will be comming in the following PR's:

https://github.com/Mbed-TLS/mbedtls/pull/6882 (bugfix/tests);
https://github.com/Mbed-TLS/mbedtls/pull/6475 (IP address parsing);
https://github.com/Mbed-TLS/mbedtls/pull/6900 (directoryName parsing);
https://github.com/Mbed-TLS/mbedtls/pull/6847 (URI parsing);
If there's something else to request or you would like to add something, please reopen this issue or create a new one.
Thanks for bearing for so long!


[5.6] access to parent model from withDefault closure

Sometimes, you may wish to return default value of relations using advanced logic and condition base of parent values
This already support but not work for lazy eager loading for example
// Message model
public function user()
{
        return $this->belongsTo(User::class)->withDefault(function ($user) {
            $user->name = $this->getAttribute('username');
            $user->email = $this->getAttribute('email');

            return $user;
        });
}

// single record.
$message = Message::first();
$message->user->name; // return username from message model as except.

// but i we try access to user info use lazy eager loading
$messages = Message::with('user')->get();
$messages->first()->user->name; // return null

This PR support pass parent model to closure to make sure the parent model available in all cases.
public function user()
{
        return $this->belongsTo(User::class)->withDefault(function ($user, $parent) {
            $user->name = $parent->getAttribute('username');
            $user->email = $parent->getAttribute('email');

            return $user;
        });
}

thanks

@taylorotwell is there any chance we can backport this to Laravel 5.5? This PR is actually a workaround for a bug, and not a new feature.
I have a HasOne relationship with a withDefault callback, which is accessing the parent model using $this. This is working fine when I use the relationship on a single model.
But when I'm eager loading this relation from an eloquent collection, all the results are wrong.
The load method in the eloquent collection is using the first() item to eager-load the relationships,
so in the withDefault callback, $this is bound to the first item in the collection, which is returning wrong results every time the withDefault callback is called.
public function salary()
{
    return $this->hasOne(Salary::class)->withDefault(function ($salary) {
        return $salary->fill([
            'currency' => $this->currency,
        ]);
    });

It works fine on single models:
$userA = User::create(['currency' => 'USD']);
$userA->salary()->create(['amount' => 100', 'currency' => $userA->currency]);
$userA->salary->currency === 'USD'; // this is true

$userB = User::create(['currency' => 'EUR']);
$userB->salary->currency === 'EUR'; // this is true

But when we eager-load eloquent collections, things are getting messy:
$users = User::with('salary')->get();
$users->find($userA)->salary->currency === 'USD'; // this is true

// Here the salary currency is USD, because when we used `$this->currency` in the withDefault callback,
// `$this` was bound to the $userA, and not the matching parent user instance.
$users->find($userB)->salary->currency === 'EUR'; // this is false


Order Online page issue

Issue: #1456
Description
Improved UI of online order page to match website theme.
Added form validation.
Made the page responsive too for all screen sizes.
Type of PR

[ ] Bug fix
[X] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
https://github.com/user-attachments/assets/fa546802-2210-44f2-9286-ac4c15d051c2
Responsive page,
https://github.com/user-attachments/assets/e8f5b0b6-3962-44b3-9ef2-141718d7d449
Checklist

[X] I have gone through the contributing guide
[X] I have updated my branch and synced it with project main branch before making this PR
[X] I have performed a self-review of my code
[X] I have tested the changes thoroughly before submitting this pull request.
[X]  I have provided relevant issue numbers, screenshots, and videos after making the changes.
[X] I have commented my code, particularly in hard-to-understand areas.

@sunny0625 @khushi-joshi-05 ,
Please review this PR. Thank You!

@sunny0625 @khushi-joshi-05 ,
Please review this PR. Thank You!

@sunny0625 Sir please review and merge this PR.

@sunny0625 @khushi-joshi-05 ,
Please review this PR as GSSoC'24 contribution closing soon.


Add Coverity Scan to CI

Coverity is a static code analysis tool which can find and warn about many hard-to-find faults.
This PR adds a CI job that triggers a scan, and a correction of the last existing issue.
Results will be available at:
https://scan.coverity.com/projects/r3
Fixes #110

Coverage remained the same at 90.228% when pulling b8b36c4d88263cbc31b9a424ad7489d8788a7c3f on Nordix:coverity-in-ci into 76e277030494a01147198dd4ff874bcc912a7de6 on c9s:2.0.

@c9s If you also think this would be a good tool to run in CI then I will update the PR before it can be merged.
Updates are needed to use secrets.COVERITY_TOKEN and to update the email to yours before merging this. Then I will add you as admin in the Coverity project, so that you can regenerate a token on the Coverity project webpage by the click of a button and add that as a Github secret.

@bjosv sure! let's do this

@bjosv I can add you as a collaborator of this repo ;)

@bjosv I can add you as a collaborator of this repo ;)

It's a badge of honor, thanks :)
I now added you as owner of the project in https://scan.coverity.com/projects/r3
The coverity job failed after the last change due to the missing Github secret COVERITY_TOKEN that needs to be configured in the Github project (Settings/Secret which I cant access) and it needs to be set to the Project token got from: https://scan.coverity.com/projects/r3?tab=project_settings

Sorry for the slow update! I just add the token


Sticky table header

Hello Beer people,
is there an easy trick to get an table header sticky when scrolling bellow ?
I'm doing a data table and I would love the fixed property to work properly for the <th>.
And every time I try to get something fixed the element on the right is pushed under the fixed element.
I guess I'm doing something wrong but I can't put a finger on it. 🤔
Cans someone throw me a ball ?
As a bonus, a way to enforce columns width on the table column would be grate 🙏

Hi @guidouil ,
Nice add and thanks for that 🍺
We will support fixed helper on <th>, <thead> and <tfoot> elements.
At moment you can do like this codepen https://codepen.io/leonardo-myrp/pen/JjxvXKE (until next release).
About "collumn width", it's hard to standarize something like that.
Maybe we can do the job, using our max, min, small, medium, large helpers on <th> and <td> elements.
We need to think more about it.


[SPARK-29037][CORE][SQL] Fix the issue that spark gives duplicate result and support concurrent file source write operations write to different partitions in the same table.

What changes were proposed in this pull request?
For InsertIntoHadoopFsRelation operations.
Case A:
Application appA insert overwrite table table_a with static partition overwrite.
But it was killed when committing tasks, because one task is hang.
And parts of its committed tasks output is kept under /path/table_a/_temporary/0/.
Then we rerun appA. It will reuse the staging dir /path/table_a/_temporary/0/.
It executes successfully.
But it also commit the data reminded by killed application to destination dir.
Case B:
Application appA insert overwrite table table_a.
Application appB insert overwrite table table_a, too.
They execute concurrently, and they may all use /path/table_a/_temporary/0/ as workPath.
And their result may be corruptted.
In this PR, we set a staging output path for InsertIntoHadoopFsRelation operation.
The output path is a multi level path and is composed of specified partition key-value pairs formatted .spark-staging-${depth}/p1=v1/p2=v2/.../pn=vn.
We can detect the conflict by checking the existence of relative staging output path.
For example:
table_a(c1 string, a string, b string,...) partitioned by (a, b , ...)
When writing to partition a=1, we need to check the existence of
.spark-staging-0
.spark-staging-1/a=1
.spark-staging-2/a=1
...
.spark-staging-numPartitions/a=1
When we found relative staging output path is existed, an exception would be thrown.
Besides, we propose an operation liveness detection mechanism to automatically detect whether an operation is running based on hdfs lease mechanism.
Thanks @advancedxy for providing the prototype of solution and valuable suggestions.
Why are the changes needed?
Data may be corrupted without this PR.
Does this PR introduce any user-facing change?
User can see the exists of staging output path and may need clean up these path manually, when this path is belong to a killed application and has not been cleaned up gracefully.
How was this patch tested?
Added unit test.

just cc @advancedxy

Can one of the admins verify this patch?

Can one of the admins verify this patch?

We propose an operation liveness detection mechanism to automatically detect whether an operation is running based on hdfs lease mechanism, and automatically clean up the conflicted staging partition path, which has been cleaned up gracefully.

Besides, In this PR, we just set an output dir for FileOutputCommitter and does not invoke any methods of FileOutputCommitter.

Gently ping @cloud-fan .

About HDFS lease and soft limit period
HDFS is an distributed file system, it has a lease mechanism to maintain lots of DFSClients to access files.
There is an constant filed LEASE_SOFTLIMIT_PERIOD, whose value is 60 seconds.
DFSClient get a lease of an file and it would be added into the dfsclient list of LeaseRenewer
LeaseRenewer would refresh the leases with a period LEASE_SOFTLIMIT_PERIOD/2
When DFSClient closed or aborted, it would be removed from the dfsClient list of LeaseRenewer and has no chance to refresh its leases
If a dfsclient want to write a file, this file must be in immutable status.
When a dfsClient begin to write a file, this file moved to being written status.
If a dfsCilent begin to write a file, but its relative lease has not been refreshed for a LEASE_SOFTLIMIT_PERIOD,  nameNode would move this file that is being written to be immutable.
It means that,  another dfsClient can begin to write this file, such as append operation.
Operation liveness detection mechanism
So, we propose an operation liveness detection mechanism based it:
For each operation, we create a LOCK file under its staging path and keep this LOCK file being written status.
After this operation completed, deleing this staging path and the lock file under it would be released.
Even this operation aborted and could not cleanup this staging path gracefully, the LOCK file would be moved to immutable status after 60 seconds(LEASE_SOFT_LIMIT_PERIOD).
When detecting some conflicted staging path, we try to get relative LOCK under staging path, if successful, delete this staging path automatically, otherwise, the operation owns this staging path is still running and we need wait it to complete.
When we try to get relative LOCK under staging path, if this LOCK file is not existed, we create it and get the LOCK; otherwise, we try to append this LOCK file, if append successfully, we get the LOCK successful, otherwise, it is owned by another DFSClient(operation).


Missing build step for linux

In explanation of how to setup build environment on linux (master/docs/linux-build-environment),
before 7) "Build qrencode QR Code C library (libqrencode) for linux", one must install libpng, e.g. using sudo apt-get install libpng-dev

I had to do the following steps to build for Linux:

Downgrade gcc to 5.1.1 in order to install odb.

The DB was not recognized unless I do something like this:
ln -s /usr/lib64/libodb-sqlite.so /usr/local/lib/libodb-sqlite.so

In order to link the application I had to add the -fPIC option to the C_FLAGS and CXX_FLAGS in c_flags.mk and cxx_flags.mk in the deps/mk directory.
Version:   afc6cf2e79059fcfef500143699bfd8ee9d731df
System    Fedora 23 (x86_64) system.


Remove fd_filestat_get.rs

Because it seems testing something very specific to wasmtime.
cf. https://github.com/bytecodealliance/wasmtime/pull/4531

I propose we keep this test, and document that fd_filestat_get on stdio fds returns a zero size and zero timestamps. This would prevent WASI programs from depending on the entity that stdin and stdout are connected to, making virtualization more robust. And it would making portability to Windows more robust, where stdin and stdout aren't files and don't support GetFileInformationByHandle.

I propose we keep this test, and document that fd_filestat_get on stdio fds returns a zero size and zero timestamps. This would prevent WASI programs from depending on the entity that stdin and stdout are connected to, making virtualization more robust. And it would making portability to Windows more robust, where stdin and stdout aren't files and don't support GetFileInformationByHandle.

while it's reasonable to allow the implementations to return 0,
i'm not sure if it's a good idea to require implementations to return 0.

I propose we keep this test, and document that fd_filestat_get on stdio fds returns a zero size and zero timestamps. This would prevent WASI programs from depending on the entity that stdin and stdout are connected to, making virtualization more robust. And it would making portability to Windows more robust, where stdin and stdout aren't files and don't support GetFileInformationByHandle.

while it's reasonable to allow the implementations to return 0, i'm not sure if it's a good idea to require implementations to return 0.

https://github.com/WebAssembly/WASI/pull/531

Actually, I've got some thoughts about the change introduced in https://github.com/WebAssembly/WASI/pull/531 - it says:

This can be 0 if the underlying platform doesn't provide suitable timestamp for this file.

I wonder if the documentation should be updated saying that this must be zero for stdio? Otherwise, there'll be inconsistency across multiple implementations, unless there are good reasons to keep those timestamps non-zero for stdio. Any thoughts?

Actually, I've got some thoughts about the change introduced in WebAssembly/WASI#531 - it says:

This can be 0 if the underlying platform doesn't provide suitable timestamp for this file.

I wonder if the documentation should be updated saying that this must be zero for stdio? Otherwise, there'll be inconsistency across multiple implementations, unless there are good reasons to keep those timestamps non-zero for stdio. Any thoughts?

my thinking is that there is little reason to treat stdio that special. for example, there are filesystems which doesn't have timestamps at all. allowing zeros can benefit such filesystems/platforms.
i dunno how useful (or useless) stdio timestamps are. but i prefer to have less special cases.

I don't have a strong preference here. But if the spec doesn't explicitly say that timestamps for stdio should be zero, I agree the test should be removed. If we want to keep the test, the doc must be updated accordingly.

I don't have a strong preference here. But if the spec doesn't explicitly say that timestamps for stdio should be zero, I agree the test should be removed. If we want to keep the test, I think the doc must be updated accordingly.

i'm not sure what you are talking about.
the doc has been updated in the PR you mentioned.

The doc says:

This can be 0

which means it might or might not be zero. But the test expects that the size/time is always zero. The test is more specific than the documentation, which means that for some of the runtimes the test might fail.

The doc says:

This can be 0

which means it might or might not be zero. But the test expects that the size/time is always zero. The test is more specific than the documentation, which means that for some of the runtimes the test might fail.

ok. i was misreading your comment. thank you for explanation.
yes, i'm still in favor of removing this test.

@sunfishcode , unless you have any concerns, I'll merge the PR

I'm going to merge this PR as there are no concerns raised. Please open an issue for further discussion if needed.