Weird connection issue with ESP8266

With using example of mqtt_esp8266, tested with adafruit IO. There are weird problems:

Sometimes it will not connect with unknown return code of 6 which is undocumented.
Leave it running a while and it will connect finally. But after a period of time, same rc = 6 again.
Even it can connect, client.connected always returns false in loop(). So in each loop, it needs to reconnect everything.

 This sounds like you are using a clientID that is already in use, so you are getting into a connection battle with another client.

I had just tried with very long clientID, it worked for a while and the problem repeated again.
Even I pressed the reset button to restart ESP8266, the problem is the same.
And I still see "Attempting MQTT connection...connected" every time even it is connected.
Publish message: hello world #76
Attempting MQTT connection...connected
Publish message: hello world #77
Attempting MQTT connection...connected
Publish message: hello world #78
Attempting MQTT connection...connected
Publish message: hello world #79
Attempting MQTT connection...connected
Publish message: hello world #80
Attempting MQTT connection...connected
Publish message: hello world #81
Attempting MQTT connection...connected
Publish message: hello world #82
Attempting MQTT connection...failed, rc=6 try again in 5 seconds
Attempting MQTT connection...failed, rc=6 try again in 5 seconds
Attempting MQTT connection...failed, rc=6 try again in 5 seconds
Attempting MQTT connection...failed, rc=6 try again in 5 seconds
Attempting MQTT connection...failed, rc=6 try again in 5 seconds
Attempting MQTT connection...failed, rc=6 try again in 5 seconds
Attempting MQTT connection...failed, rc=6 try again in 5 seconds
Attempting MQTT connection...failed, rc=6 try again in 5 seconds
Attempting MQTT connection...failed, rc=6 try again in 5 seconds
Attempting MQTT connection...failed, rc=6 try again in 5 seconds
Attempting MQTT connection...failed, rc=6 try again in 5 seconds
Attempting MQTT connection...failed, rc=6 try again in 5 seconds
Attempting MQTT connection...failed, rc=6 try again in 5 seconds
Attempting MQTT connection...failed, rc=6 try again in 5 seconds
Attempting MQTT connection...failed, rc=6 try again in 5 seconds
Attempting MQTT connection...failed, rc=6 try again in 5 seconds
Attempting MQTT connection...failed, rc=6 try again in 5 seconds
Attempting MQTT connection...failed, rc=6 try again in 5 seconds
Attempting MQTT connection...failed, rc=6 try again in 5 seconds
Attempting MQTT connection...failed, rc=6 try again in 5 seconds
Attempting MQTT connection...failed, rc=6 try again in 5 seconds
Attempting MQTT connection...failed, rc=6 try again in 5 seconds
Attempting MQTT connection...failed, rc=6 try again in 5 seconds
Attempting MQTT connection...failed, rc=6 try again in 5 seconds
Attempting MQTT connection...failed, rc=6 try again in 5 seconds
Attempting MQTT connection...failed, rc=6 try again in 5 seconds
Attempting MQTT connection...failed, rc=6 try again in 5 seconds
Attempting MQTT connection...failed, rc=6 try again in 5 seconds
Attempting MQTT connection...failed, rc=6 try again in 5 seconds
 ‚n?†$“Ò¶£ÿOCAAüû
Connecting to 666
...
WiFi connected
IP address:
<IP_ADDRESS>
Attempting MQTT connection...failed, rc=6 try again in 5 seconds
Attempting MQTT connection...failed, rc=6 try again in 5 seconds
Attempting MQTT connection...failed, rc=6 try again in 5 seconds
Attempting MQTT connection...failed, rc=6 try again in 5 seconds
Attempting MQTT connection...failed, rc=6 try again in 5 seconds
Attempting MQTT connection...failed, rc=6 try again in 5 seconds
Attempting MQTT connection...failed, rc=6 try again in 5 seconds
Attempting MQTT connection...failed, rc=6 try again in 5 seconds
Attempting MQTT connection...0‚n?†$“Ò¶£ÿOCAAüû
Connecting to 666
...
WiFi connected
IP address:
<IP_ADDRESS>
Attempting MQTT connection...failed, rc=6 try again in 5 seconds
Attempting MQTT connection...failed, rc=6 try again in 5 seconds
Attempting MQTT connection...failed, rc=6 try again in 5 seconds
Attempting MQTT connection...failed, rc=6 try again in 5 seconds
Attempting MQTT connection...failed, rc=6 try again in 5 seconds
Attempting MQTT connection...failed, rc=6 try again in 5 seconds

I'm experiencing the exact same issue with the WeMos D1R2 (also based on the ESP8266). Connecting to a Mosquito broker as the only client on the network.

A return code of 6 is undocumented because it is not part of the MQTT standard - the only way this client will return 6 is if that's what the broker has passed back. Would need to figure out under what condition Mosquitto returns a 6.

Adafruit has added their own connack codes, they have their own broker.
On 24 Jul 2016 05:47, "knolleary"<EMAIL_ADDRESS>wrote:

A return code of 6 is undocumented because it is not part of the MQTT
standard - the only way this client will return 6 is if that's what the
broker has passed back. Would need to figure out under what condition
Mosquitto returns a 6.
@ralight https://github.com/ralight - any ideas?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/knolleary/pubsubclient/issues/181#issuecomment-234739102,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAE22jyxQNMCz-KUfwb58Auw_JVXuCA2ks5qYn0wgaJpZM4JNDV5
.

Ah yes, original issue was adafruit Io, not mosquitto. Thanks @ralight

Connack of 6 means adafruit are rate limiting you


feat(schema): allow null fields to be merged with other datatypes

Which issue does this PR close?
Closes https://github.com/apache/arrow-rs/issues/4901
Rationale for this change
All fields can be marked nullable, so they could be easily made compatible with a field with DataType::Null
What changes are included in this PR?
Handle DataType::Null in schema field merge
Are there any user-facing changes?
Merging null fields no longer returns error when being merged

I see, in C++ API this behavior is guarded by options flag: https://arrow.apache.org/docs/cpp/api/datatype.html#_CPPv4N5arrow5Field12MergeOptionsE
(default to true).
So they question is does this PR require adding options to API to allow disabling the feature, or it could be the only behavior until options are added.


Follow Up - Make "Committing To" and "Interested In" available

Make "Committing To" and "Interested In" check boxes available in the follow up screen. On manage person, make these read only and have them display the values for the most recent entered communication card

Should the Follow Up entries be saved automatically? It seems that the users use the system in that way since the save immediately after adding 1 Follow Up. Could then move them outside of the form box which might make the form look more manageable.


Comment color is to dim.

The current color used in font-lock-comment-face is so dim
that one can barelly read any comment in a source file, please
consider improving this.

Sure, I'll look into it.

With a recent commit I made doc-comments violet and slightly brightened comment faces. Is that better?
I'd rather not go far beyond that (doom-one is supposed to be low-contrast, like Atom One's). I will have other colorschemes soon that are higher contrast, but in the meantime, you can modify those two comment faces in your configuration, like so:
(custom-set-faces
 `(font-lock-comment-face           ((t (:inherit font-lock-builtin-face))))
 `(font-lock-comment-delimiter-face ((t (:inherit font-lock-builtin-face)))))

Just introduced the doom-enable-brighter-comments option, as well as adjusted some of the bright-comment colors. Hopefully this helps!

Perfect mate! A thousand thanks.

Can a similar option be added for the molokai theme?


[proposal] Sugar syntax for constructing ratio variables from float and int

I propose the syntax i% for the construction of ratio variables from float and int. For example:
#import "@preview/cetz:0.2.2": canvas, plot, draw, coordinate, vector
#import draw: *

#let i = 50
#canvas({
  line(
      (),
      ((), i%, (rel: (x: 2.5))),
    )
})

This would be a short hand for  i*1%, but with a clearer syntax. At the best of my knowledge in Typst (I am still a beginner, have mercy), % has not other purpose in code block but defining ratio variables. So this syntax can be a good constructor for ratio's :)

I'm not sure whether I like the proposal (it feels weird but also kinda makes sense?) but * 0.01 is just a different operation, and doesn't return a ratio.

Honestly, that wasn't the wisest proposal I've made. I also have mixed felling about it.

but * 0.01 is just a different operation, and doesn't return a ratio.

Yes, but i*0.01% returns. I initially thought in i% as a good constructor for it, but now I see that this is actually quite restrictive as we cannot perform operations on i before constructing a ratio. On the other hand, the syntax i*1% allows that.


Missing scroll on tag modal

Cannot scroll on the add tag modal once the tags list got too long. Solved by zooming out, but still unpleasant:

From dev chat today: Might be worth redesigning this tag editing UI to be less cluttered.


Source Control show list of files changes, but there is none

I freshly clone the project from BitBucker, and load the project with Visual Studio Code. The Source Control showing list of images files change and deleted.
I'd googling and tried all the possible solution, and it still doesn't resolve the issue.
P/S: The original source committed are from MacOS, and I cloned the project to my local Window OS.

@gahhon, have you opened the repository in Visual Studio or Visual Studio Code?

Hey @lszomoru, this issue might need further attention.
@gahhon, you can help us out by closing this issue if the problem no longer exists, or adding more information.

@gahhon, great observation. I have tried to clone a git repository that has a file with a : in the path and I received an error message in VS Code that originated in git
error: invalid path 'hello: world.txt'
fatal: unable to checkout working tree
warning: Clone succeeded, but checkout failed.
You can inspect what was checked out with 'git status'
and retry with 'git restore --source=HEAD :/'

This is not an issue with VS Code but an issue with Windows. On Windows there are a set of characters that are reserved when it comes to paths, : being one of them. If you would like to contribute to this repository from a Windows machine I would suggest that you rename these files (ex: use - instead of :) so that the repository can be correctly cloned on a Windows machine.


Surrounds not working for embedded regex grammar

Parens and single quotes are listed in regex.language-configuration.json as surround pairs, but they're not surrounding in a tmLanguage.json file.
I was gonna open a PR adding just parens to the tmLanguage.json configuration but it looks like there's a bigger issue with the embedded language's configuration not applying.

yes, there is problems with vscode not correctly applying language settings for embedded languages
there are some bug reports
I'll add parens to the json language-configuration.json file tomorrow
is there any others you want added?

yes, there is problems with vscode not correctly applying language settings for embedded languages

Not surprising. I have an embedded language extension too and it's definitely a janky API.

is there any others you want added?

I think that's it. regex.language-configuration.json has single quotes but I can't remember ever wrapping something in single quotes in a regex.
Thanks for the quick response!

single quotes ' can be used instead of angle brackets <> for named groups and referencing
(?'name'...) \\g5 \\k'name' (?(name')...|...)` etc

Thanks!


fix: remove unnecessary pyarrow import in testutils

fix #364

Codecov Report

Merging #365 (3ee9001) into main (d4fdfec) will decrease coverage by 0.01%.
Report is 2 commits behind head on main.
The diff coverage is n/a.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff             @@
##             main     #365      +/-   ##
==========================================
- Coverage   94.57%   94.56%   -0.01%
==========================================
  Files          22       22
  Lines        2617     2613       -4
==========================================
- Hits         2475     2471       -4
  Misses        142      142

Files Changed
Coverage Δ

src/dask_awkward/lib/testutils.py
100.00% <ø> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


[TypeChecker] NFC: Revise comments for `{typeCheck, solve}ForCodeComp…

…letion`
@AnthonyLatsis Thanks for suggestions!

@swift-ci please smoke test

@swift-ci please smoke test Windows platform

Failing test/Profiler/coverage_relative_path.swift on Windows is unrelated, this is just a doc string change.


Update Carbon to 3.*

What was changed

Updated Carbon.
Removed dependency on lluminate/support.
Fixed some tests due to Carbon changes.

Why?
Upgrade to Symfony 7.
Checklist

Close #401

 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.

Thanks. LGTM
btw TestingRequest class looks dead


OSD-19729: Enables ocm backplane login to provide  default namespace as an argument

What type of PR is this?

feature
documentation
unit-test

What this PR does / Why we need it?
As a SRE, I want backplane login to prompt for a  default namespace for me to run commands like oc debug node/xxx.
Which Jira/Github issue(s) does this PR fix?
Resolves #OSD-19729
Special notes for your reviewer
Once merged,  ./ocm-backplane login --help  will display
Running login command will send a request to backplane api
...
Usage:
  ocm-backplane login <CLUSTERID|EXTERNAL_ID|CLUSTER_NAME|CLUSTER_NAME_SEARCH> [flags]
...
Flags:
     -n,  --namespace string   Sets a default namespace for SRE to run commands like oc debug node/xxx
...
Global Flags:
...

Example:
Considering cluster ID is 28lje9u3jd59j3dmf36sojgpa32qrag3 .

Without options

./ocm-backplane login   28lje9u3jd59j3dmf36sojgpa32qrag3
oc project

Output will be

Using project "default" on server "https://api.stage.backplane.openshift.com/backplane/cluster/28lje9u3jd59j3dmf36sojgpa32qrag3/".

With option ' namespace '

./ocm-backplane login --namespace openshift-sre-pruning   28lje9u3jd59j3dmf36sojgpa32qrag3
oc project

Output will be
_Using project "openshift-sre-pruning" on server "https://api.stage.backplane.openshift.com/backplane/cluster/28lje9u3jd59j3dmf36sojgpa32qrag3/"._
Please do refer to  the prior discussion in the https://github.com/openshift/backplane-cli/pull/315  .
Pre-checks (if applicable)

[x] Ran unit tests locally
[x] Validated the changes in a cluster
[x] Included documentation changes with PR

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

Project coverage is 48.27%. Comparing base (125a4cc) to head (1ef5d80).

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #374      +/-   ##
==========================================
+ Coverage   48.19%   48.27%   +0.08%
==========================================
  Files          56       56
  Lines        4293     4300       +7
==========================================
+ Hits         2069     2076       +7
  Misses       1901     1901
  Partials      323      323

Files
Coverage Δ

cmd/ocm-backplane/login/login.go
74.63% <100.00%> (+0.65%)
:arrow_up:

@Tafhim , @supreeth7  and @devppratik : Please can you review ?

@amej Can you pls squash the commits ?

/label tide/merge-method-squash

/label tide/merge-method-squash

@devppratik :  Does this label free me up from the work  of squashing commits locally on my laptop  & force pushing them to the fork's feature branch?

squash
@supreeth7 :
As my PR was showing 34 commits, on my current branch OSD-19729; I performed ' git rebase HEAD~34.
It showed me 54 items.  I 'pick' first row & rest  was 'squash'  .  I had to fix multiple merge commits during the rebase.
After rebasing my local branch OSD-19729  on top of the github based OSD-19729, I could push the changes.
Here is the current status with 35 commits : https://github.com/openshift/backplane-cli/pull/374/commits .

@amej thanks for addressing the review comments.
The commit number is too wide for it to be merged, even with the tide plugin squashing it, can you try squashing your commits without any trailing commit history?

@amej Can you pls squash the commits ?

@supreeth7 :  Please find a single commit now.  I force pushed the changes to the OSD-19729 branch.

@amej thanks for addressing the review comments.
The commit number is too wide for it to be merged, even with the tide plugin squashing it, can you try squashing your commits without any trailing commit history?

@devppratik  guided me to opt for the easy solution .
I opted to force push my changes to the OSD-19729 branch.
Now you will find a single commit .
Thank you for your patience.


WelcomeUserStateAccessors code is not consistent with github sample

WelcomeUserStateAccessors code mentioned in the documentation is not consistent with github sample.
https://docs.microsoft.com/en-us/azure/bot-service/bot-builder-send-welcome-message?view=azure-bot-service-4.0&tabs=csharp%2Ccsharpmulti%2Ccsharpwelcomeback
https://github.com/Microsoft/BotBuilder-Samples/blob/master/samples/csharp_dotnetcore/03.welcome-user/WelcomeUserStateAccessors.cs

@rlamba89 , The samples repo is updated independently of the documentation, so these will occaisonally be out of sync.


Using lua script

Lua scripts block everything as in Redis or not ?

So, if it starts with more than one thread is it possible to have multiple commands (or Lua scripts) executing at the same time?

@craigglennie yes


Plan for ruby 3.3 YJIT

Ruby 3.3 YJIT is going to be released at the end of 2023 with a lot of improvements for performance and resource optimization. Is there any plan for this new version?

It's released now.


feat: allow private PyPI releases for ".web.dev" tags

As title says

@RobPasMue created PR on interanl repo so closed this PR


Add TextAreaLong to all along with the other classes

That should fix #89

hey, there's sth weird with the checks here :thinking: Travis build passed, but there is some 2nd pending travis check

@Ashlett good that you noticed it!
I did found a solution to that here:
https://travis-ci.community/t/known-issue-travis-ci-reports-expected-waiting-for-status-to-be-reported-on-the-github-status-api-but-the-status-never-arrives/1154
After applying it disappeared, so we are good now!


Added About Us Page

Resolved issue #142
Added the About Us Section and made it responsive as well

@abhay-raj19 In the previous repo I wasn't able to resolve the conflicts so I deleted it and created a new one. Please merge it.
Thankyou !

@abhay-raj19 Why is the routing removed from here?

@Ayushhgupta39 Who removed this though ?

thats why all the deployment are failing up

 @abhay-raj19 Why is the routing removed from here?
Also the deployment preview is not working. @Hemashree21 can you check this.

I'll check it 👍

@abhay-raj19  @Ayushhgupta39  After I pulled the code when I run it, the output is showing a blank page like this. I tried debugging those errors but it's not working and I also tried commenting the code in which I made some changes.


Introduce PR and Issue Templates

Issues

state the why (background, context)
acceptance criteria
notes
not in scope
wireframes

PRs

linked to the original issue
unit tests
integration tests
documentation
if any new issues are created for debt or uncovered work

For reviews, reviewer should tests the acceptance criteria?

I would say yes to the last question, the review only makes sense in the context of the acceptance criteria.


"Bozioru" is listed in the drop-down menu

Description:
"Bozioru" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Bozioru" in the search bar.

Expected results
The location is listed in the drop-down menu as "Bozioru, Buzău".


Capacity hours only based on "Development Hours"

In my team, we are using multiple "disciplines" on the task level and hour allocation. But the DropPlan only sum the hours set to "Development" Discipline - so even though the total capacity for the person is 20 hours, only 10 hours will be displayed as these are the hours allocated for development.

Hi @Syssrn
DropPlan now adds up all the time for the person.
Can you confirm this is solved for you in the latest version?


[CustomHelp] remove U+200E character

U+200E sitting besides "uncategorised" was preventing users from executing [p]help uncategorised
before: "uncategorised(U+200E)"
after: "uncategorised"

Thanks for the PR, really appreciate it.
however, this requires additional logic to not affect existing user (involving config migration).
I've taken the liberty to also remove the blank character and it's checks, from other places as well.
here: https://github.com/npc203/npc-cogs/commit/bc45af0b1bb3920675a2527752cc5aacf22a1c08


Update tooltip/behavior for Line Draw Tool

Current "active" tooltip for line draw tool is active. It currently says "Update trend line. Double click graph to clear," but it currently toggles between those two actions.
Desired behavior should mimic the behavior of the angular size measurement tool:

When tool is inactive, tooltip should stay as is: "Draw a trend line"
Clicking on button activates tool, so student can draw.
After line is drawn, student should automatically be able to adjust line with the "grabbing hand" cursor.
When tool is active, icon should change to mdiEraser
Tooltip should change to "Clear drawn line"
When student clicks button, drawn line should go away; icon resets to pencil & tooltip changes back to "Draw a trend line"

So I took a look at the base line draw tool (in the cosmicds repo), since that's where all of the real line draw functionality lives, and was reminded why exactly we implemented the current behavior in the first place.
The issue is that bqplot only allows one interaction at time on any given Figure. In order to make the grab/drag functionality of the line tool possible, we need to disable the default interaction on the viewer figure - this means that any functionality added via add_event_callback is blocked when the drawing tool is active. Keeping this active when the tool is deactivated would mean not restoring the viewer's default interaction.  I'm not sure that we'll ever need the line draw tool on a viewer that also has callback events, so we might be able to get around this by being careful with manipulating interactions.


AppDef config option

Add an option to the config *.yaml files channel.AppDef which defines the App Definition (ethereum Address).

we discussed this offline. the rootSeed message is caused by package wire/net importing pkg/test and would not be surpressed by defining an appdef. assigning back to you.

I updated the issue to only reflect the AppDef config change, since the root seed message must be fixed in go-perun see https://git.perun.network/perun/go-perun/-/issues/222

Our payment app has a unique address now and does not need an on-chain address. Closing.


Better explanation of parameter "Value"

I tend to double check this out quite often during the years, since the intellisense and compiler is not that helpful with this...
Copied from the old C/AL help file:
Specifies what the function returns. The valid options are 1, 2, and 3.
The value 1 corresponds to day of the week (1-7, Monday = 1).
The value 2 corresponds to week number (1-53).
The value 3 corresponds to year.

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

ID: 5d363fcd-5067-f6db-afc7-33d2e8dff372
Version Independent ID: 58c55fa0-bb86-c573-c656-09bdc586ba4d
Content: Date2DWY Method - Business Central
Content Source: dev-itpro/developer/methods-auto/system/system-date2dwy-method.md
Service: dynamics365-business-central
GitHub Login: @solsen
Microsoft Alias: solsen

Hi, thanks for your feedback. The parameter descriptions have been updated, they will be available with the next doc refresh.


Add redis as celery broker

Coverage decreased (-0.02%) to 89.335% when pulling 56650241ed6f690a80f79183338d7d549afcb825 on flamingspaz:moresettings into 5087e80689a70b7b90ed1a25a9cee1519d1e7f11 on mozilla:master.


Enhance Clang component not found exit info

I change the clang-cl.exe not found error message
to make it more clearly

I am not sure if MSVC is understood by more people than Visual Studio. It seems to me, while I use that term a lot, Python beginners not familiar with C development, will not know what it is. Probably having both mentioned is a good thing, maybe Nuitka should always say "MSVC (known as Visual Studio)" when it refers to that. These things could also very well have mnemonic pages, such that we can give more information, e.g. links to User Manual where it talks about that, direct download link and what not. The mnemonic argument would be good for that.
For the LLVM I am not sure if it gets added to PATH, probably the default installation path could be added too, when searching for it. However, I believe that pure LLVM is not working for Nuitka, since Windows SDK is not included there, and needs to be adapted for it to work, at least that was when I tried to use clang.exe maybe I never tried clangcl.exe where chances are that the Windows SDK needs no changes for it. However, without finding MSVC, I don't think it finds a Windows SDK in Scons, or does it, or does LLVM contain a windows.h file now?

In my case, I install llvm manually to have the latest version, and add llvm into PATH
and install msvc with windows sdk by vs installer
which work fine with me
the point I want to change is that, you can install clang-cl.exe by VS or by llvm, so don't say "can't find Visual Studio", but tell the user what is lost, like clang-cl.exe

I am not sure, that clang can work without MSVC installed, however, so it may well be what's missing.

Actually in Scons, nothing is going to setup the environment in which clangcl.exe operates, what flags is uses, without first finding a cl.exe and having that work, and then the path is replaced. So it's misleading to say or. It seems you found out that instead of from MSVC you can install clangcl.exe from LLVM, but there is not a lot of benefit from that, except that we don't know if it's working as well as the package provided by Microsoft for that purpose.
Unless somebody is a huge LLVM fan that needs the very latest release to be used, I do not see the value in using it. If of course it worked standalone, sure, but I see no way of that happening unless we modify Scons very heavily, and I believe there is no Windows SDK in Clang distributions, so we would have to start and detect that, and, and... for what precisely?

I am not sure, that clang can work without MSVC installed, however, so it may well be what's missing.

yes, clang can't work without mingw or msvc, but the point is, clang-cl and cl.exe are different, and they can be install independently, so it is possible that a user install msvc, but didn't install clang, so it is useful to tell the user what is missing.
will this be more logical to you?

Your current text has "or LLVM" which is absolutely untrue for all I know. The mention of the installer is better, but the "Clang component" thing is not mentioned anymore. The installer installs MSVC, you need to configure it as a component there.

Is there anything forthcoming from this, the text can be enhanced, it just cannot become wrong.


model not working

I write on model and contlloer it is now wokring
		$filght = Blogs::all();
	print_r($filght);

"Model"
<?php namespace App\Modules\Blogs\Models;

use Illuminate\Database\Eloquent\Model;

class Blogs extends Model {

	//
   public static $unguarded = true;

    public $timestamps = false;

	protected $table = 'user';

}

Your problem is unclear. Is there an error? Would you please post the debug output?
How looks your controller? Is there the following line?
use App\Modules\Blogs\Models\Blogs

if not, it could be the problem. In that case $filght = \App\Modules\Blogs\Models\Blogs::all(); should work.

Thanks, This code of line resolved my issue,
May secound step is creating another controller is same module. Say my moule name is "Admin" and default contoller is admin, i want to create another controller "category".
I created it and when i hit the link it shows me blank page.
"ReflectionException in Container.php line 736: Class App\Modules\Admin\Controllers\CategoryController does not exist"
I have added in routes also
Route::resource('admin/category', 'CategoryController@index');

Nice to hear your problem is solved.
The generated Module is kind of “template“. So you can see how it works. Of course you have to define the routes which you want to use ;)

Yes I did same, but it is showing blank page, can you tell me steps to creat  a sub contoller for a module.

I think I have a clue what's wrong with your code.
If you have the preset Route::resource('admin', 'AdminController'); then you have the following routes reserved:
|        | GET|HEAD                       | admin                                                 | admin.index          | App\Modules\Admin\Controllers\AdminController@index          |            |
|        | POST                           | admin                                                 | admin.store          | App\Modules\Admin\Controllers\AdminController@store          |            |
|        | GET|HEAD                       | admin/create                                          | admin.create         | App\Modules\Admin\Controllers\AdminController@create         |            |
|        | DELETE                         | admin/{admin}                                         | admin.destroy        | App\Modules\Admin\Controllers\AdminController@destroy        |            |
|        | PATCH                          | admin/{admin}                                         |                      | App\Modules\Admin\Controllers\AdminController@update         |            |
|        | PUT                            | admin/{admin}                                         | admin.update         | App\Modules\Admin\Controllers\AdminController@update         |            |
|        | GET|HEAD                       | admin/{admin}                                         | admin.show           | App\Modules\Admin\Controllers\AdminController@show           |            |
|        | GET|HEAD                       | admin/{admin}/edit                                    | admin.edit           | App\Modules\Admin\Controllers\AdminController@edit           |            |

In that case a route like admin/categories would fire the App\Modules\Admin\Controllers\AdminController@show action. categories is here the parameter for the show action.
Take a look:

http://laravel.com/docs/5.1/controllers#restful-resource-controllers
php artisan route:list

My consern is quit diffrent,
I am trying to do following:

Admin is Module name and have AdminController.
Admin module has another controller CategoryController, ProductController.
Now i wanted to use that controller as part of admin module
How can i achive that is your Modular structure accept that.

Yes, this is possible.
As long as I have no comprehensive overview of you module, I have to guess.
I recommend to ask this question at StackOverflow with all module files in detail.
Send me the link and I will take a look and answer you.

Hi If you needed you can find my code on dropbox, https://www.dropbox.com/sh/ho6rhfbngikjusc/AADjdIQ2Q45nX1mgqZvq33DPa?dl=0
Stackoverflow Question link: http://stackoverflow.com/questions/31958379/laravel-5-1-module-with-multiple-controllers


Correcting Laureate spelling

Thanks a ton @luckymurari !
Also: Damn. I should figure how to turn on spell check for README docs.


Ensure last-wins profile order is respected for AWS Parameter Store and Secrets Manager

Fixes #2385
Note, for Secrets Manager this fix breaks existing tests, which I have corrected.  I don't think the existing tests assert the correct expected profile order, so these have been updated to reflect the changed logic.  However I don't know if this counts as a breaking change?
Also, I would be very grateful, if this is accepted, if it could be backported to the 4.0.x branch as this is the version we are currently using and where it affects us.

Fixed checkstyle issue and force pushed

@spencergibb - is it possible to get the build for this PR rerun?
Many thanks

Thanks for the PR @driseley.  The 4.0.x branch is no longer supported so we will not be cherry picking the PR back to that branch because there will not be any more open source releases of that branch.


Send a password reset email using Firebase in the ForgotPasswordActivity

This  issue  requires Firebase Implementation in our application. Provide your email address  by commenting while claiming the issue or after claiming the issue, and I will add you as a member to the Project's Firebase Portal
You can access it then here: https://console.firebase.google.com/u/3/project/superquizzer-8cb6c/overview
You have to implement Firebase Authentication in the ForgotPasswordActivity, if a valid email address is entered that is already registered, on clicking the submit button, the user should get an email to reset their password. Also add check for failure case.
Attach a screenshot of an email that you get on submitting your email in the ForgotPasswordActivity.
You can refer to the Firebase Docs for your reference here: https://firebase.google.com/docs/auth/android/manage-users#send_a_password_reset_email

Claim

<EMAIL_ADDRESS>

Claim

@anshgeez31  updates?

Claim


Switch back to released version of DataFusion and arrow-rs after Arrow Java 16 is released

What is the problem the feature request solves?
In #239, we moved to use specified forked of DataFusion and arrow-rs as a temporary workaround to the Arrow Java bug which sends invalid offset buffers through C Data Interface.
The fix to the bug was merged to Arrow Java. Once Arrow Java 16 is released, we should move back to released versions of DataFusion and arrow-rs.
Describe the potential solution
No response
Additional context
No response

Arrow 16 has now been released:
https://arrow.apache.org/release/16.0.0.html

Got it. Thanks.

I would like to hold this until #250 is merged. Assigned to myself.

@viirya Is there anything stopping us from depending on the datafusion repo directly now (once https://github.com/apache/datafusion/pull/10702 is merged) rather than your fork?

Let me try to switch to official DataFusion repo.

There are some changes from latest DataFusion repo. I'm working on to update Comet with it.

The release vote for arrow-rs 52 has started, and there is a draft PR in DataFusion to upgrade to this version:
https://github.com/apache/datafusion/pull/10765

Cool. I will update the PR accordingly once we get DataFusion and arrow-rs releases.


⚠ Bofrost.be has degraded performance

In d7523f0, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1369 ms

Resolved: Bofrost.be performance has improved in 6c73375.


Trojan/Mallware detection

OS:
All
Issue Descriptions:
Trojan / mallware detected.
Steps to Reproduce:
I'v download & installed the latest version, at that time our network guy contact me.
After installing and running he got a firewall warning that a Trojan was detected "Torpig"
Before that (when i downloaded it) he got a mallware detection.
So perhaps something got infected.. ?

It is a false positive.  Many open source applications are affected by this as they are relatively unknown installers.
https://github.com/electron/windows-installer/issues/106#issuecomment-226613329
It is detected as a Trojan because they way we install (no admin permissions required) is considered suspicious by really strict AV and Firewall tools.
Just to reiterate, this is a false positive and there is nothing I can do to make it go away.  However, you can.  If you AV or Firewall has the ability to mark as a false positive, please do so.  If enough people flag our installer as a false positive eventually people will stop getting these warnings.
Cheers 😆


[Land and See - Home] Hero carousel images should link to Article page

Current
On Mobile or Desktop, if you click anywhere on the carousel image that is not the link or arrows, the page does nothing.

Should be
On Mobile or Desktop, if you click on the carousel images / text, you should go to the Article page.

I think a good middle-ground might be to make the title into a link that goes to the Story, in addition to the READ MORE button. I can put that in my next PR, if that works.

Taking a look at this again, today, and I realized the title is already a link to the story. :)


Broken link in README.md

The readme has a link to https://docs.wasmtime.dev/stability-wasm-proposals-support.html, which is 404 currently.
This is from this part of the readme:

Standards Compliant. Wasmtime passes...

I wanted to read if wasmtime implements the webassembly 2.0 draft, but unfortunately that link doesn't have that info now

Thanks for the bug report. The page you might be looking for might be this one: https://docs.wasmtime.dev/stability-tiers.html#current-tier-status describes which WebAssembly proposals are supported. I will fix the link in the readme to point there.

Thank you, that's great!


PrusaSlicer not opening

Distro: Arch Linux
I have tried several PrusaSlicer versions (main repos, git and bin from the AUR and even flatpak) as well as SuperSlicer, none of them go past the loading screen, if I switch to Sway all of them work perfectly.
Launch log for PrusaSlicer:

I don't even know wtf that is
Have you tried forcing xwayland with GTK_BACKEND=x11?

Hi,
I'm having the same issue.
PrusaSlicer is already using x11 backend, see https://github.com/prusa3d/PrusaSlicer/issues/8284#issuecomment-1117203502

doesn't seem to me like something Hyprland should control here...
Does it crash or just never opens a window?
Also, can you launch hyprland (ONLY hyprland, no exec-onces) and then the faulty app, and provide the hyprland log?

weird, looks to me like the window never requests another window. Odd.

Well, maybe you should consider switching to a real slicer, like Cura. (I'm jocking)

I got triggered a bit, @Estebiu :)
But yeah. same issue here. splash windows shows, then nothing visible, process running in bg

hyprland uses gles 3.2, and gl 2 iirc is the oldest ver. Whut

hyprland uses gles 3.2, and gl 2 iirc is the oldest ver. Whut

Lol.. dunno, man! I've cura-5-bin from AUR installed, if that helps.

I just stumbled upon this one. Cura5 does not open either.
However it shows an error "Could not probe OpenGL. This program requires OpenGL 2.0 or higher. Please check your video card drivers."

I decided to use Cura in the meantime, I have to temporarily set QT_QPA_PLATFORM as "xcb" for it to open but it works for now.

Hi,
As a workaround, I got prusaslicer running using Xpra. You can run it using:
xpra start --attach :100
DISPLAY=:100 prusaslicer

and running xpra stop 100 once finished (or even add the xpra start command into your hyprland.conf file and letting the xpra session run in the background).
I'm not sure if this is linked at all with the initial issue, but I noticed that I couldn't start prusaslicer right away with xpra start --start-on-connect=prusaslicer (or other variants). The main window would just not open.

Just wanted to tune in here as well. The GTK2 builds of prusa-slicer and superslicer both open up fine for me, so it may be something related to GTK3 that causes issues here.

Hi, As a workaround, I got prusaslicer running using Xpra. You can run it using:
xpra start --attach :100
DISPLAY=:100 prusaslicer

and running xpra stop 100 once finished (or even add the xpra start command into your hyprland.conf file and letting the xpra session run in the background).
I'm not sure if this is linked at all with the initial issue, but I noticed that I couldn't start prusaslicer right away with xpra start --start-on-connect=prusaslicer (or other variants). The main window would just not open.

@4rnoP I was able to launch using xpra in one line with:
xpra start --start-child="prusa-slicer" --attach --exit-with-children

this way you don't have to manually stop xpra afterwards either.
thanks for the workaround!

Xpra does seem to work as a workaround for this, and can confirm both PrusaSlicer and SuperSlicer don't work by default.

I've seen the devs over at the Prusaslicer repo mention that the recent issues regarding Wayland/Linux are already fixed in their internal builds. So hopefully the next release will finally be usable without workarounds.

This is no longer happening for me on the latest version, PrusaSlicer is opening fine within Hyprland on Arch Linux.
prusa-slicer 2.5.1-1
hyprland 0.23.0beta-2

same here. it's fixed for me with the latest slicer

also fixed for me with PrusaSlicer 2.5.1 and 2.6.0 respectively


Bitbucket token dies and CMS fails

Describe the bug
I am using a Bitbucket implicit auth. Bitbucket token dies in 1 hour and CMS doesn't handle this, it just shows an error and everything starts to hang. Not the best experience for a user, maybe you can catch 401 error and redirect a user to the login page?
Maybe you know some other workarounds? Can I handle this situation by myself?
Note that Bitbucket implicit auth doesn't have a refresh token option.
Thanks.
To Reproduce
Open CMS with bitbucket implicit auth, wait 1 hour until bitbucket token dies.
Open any CMS page with content.
You will see this error:
https://monosnap.com/file/KLWhskJ5pzEBwDfsAHO5ZqbFqr9ceJ
and in the console:
https://monosnap.com/file/Xq8Yggm998xn8W0RIszbGrGjkSBafR
Expected behavior
Catch error and redirect a user to the login page for better user experience.
Applicable Versions:

netlify-cms-app 2.11.16
netlify-cms-core 2.18.1
Git provider: BitBucket

closed in https://github.com/netlify/netlify-cms/pull/3494

@erezrokah  Is this issue fixed? What is the behaviour and in which version it is available?

This is better handled in https://github.com/netlify/netlify-cms/pull/3847
The CMS periodically checks if the user has been logged out and will display a proper error.

@erezrokah  Is this released? Or when can we expect the release?

Oh, forgot to update - this is released under<EMAIL_ADDRESS>or<EMAIL_ADDRESS>


Data grid "calculateCellValue" plus grid "editing" plugin forcing Angular change detection

Specify the version of the devextreme-angular and devextreme packages you are using.
devexteme version: 18.1.3
devextreme-angular version: 18.1.3
Specify the type of the issue (check one with "x"):
[x] bug Report

Steps to reproduce:

Enable grid editing (tested with "popup" mode)
Assign non-existant "dataField" value to a column
Assign "calculateCellValue" function which returns new array/object for the same column
Run the test and click on "Edit" to edit one of the rows

Expected behavior:

It should just work

Actual behavior:

After clicking "Edit" the custom "calculateCellValue" function keeps being called until the app completely freezes

PS: yes, setting change detection to "OnPush" "fixes" this particular bug, but it also introduces some other bugs. So, no, that's not a proper solution.

@ovchinnikov Any thoughts about this?

Yes, I can reproduce this issue with the example. We will research it.


Simplify iter8ctl

Is your feature request related to a problem? Please describe the problem.
Iter8ctl needs simplification. In particular:

it needs to become a kubernetes client, so that there is no need for piping the experiment yaml from kubectl
the Tekton CLI supports a last flag, so that you do not need to know the name of the last task you ran. Similarly, iter8ctl needs to support a latest flag so that experiment name need not be specified explicitly
we also want to assert various conditions on the experiment using iter8ctl (example, completed, completed without failure, and so on)

Describe the feature/solution you'd like

iter8ctl describe -l : this command should describe the last experiment to have been created by the user in the cluster. Note the simplification over current iter8ctl usage: it eliminates the need for the user to know the experiment name. This is useful in every one of our tutorials, especially, the helm tutorial where experiment name has a random suffix.
iter8ctl is also a kubernetes client as implied by the above command. So, there is no need to run kubectl get experiment <name> -o yaml | iter8ctl describe -f - anymore. The command is substantially shorter.
iter8ctl should support assertions like this: iter8ctl assert -l -c completed -c winnerFound. It will exit with code 0 if assertions are satisfied. Else, it will exit with code 1 after printing an appropriate error message. This is useful for tutorial purposes, human interaction, as well as CI/CD/GitOps usage scenarios.

Will this feature/solution bring new benefits for Iter8 users? What are they?
The benefits to the users over current usage are described above.
How will this feature be tested?

[x] Existing tests will be modified for describe and new unit tests will be added for assert command.

How will this feature be documented?

[ ] Iter8 tutorials will showcase these commands.

Additional context
Making iter8ctl a K8s client was a piece of feedback we received after one of our recent presentations. This issue is about listening to this piece of useful feedback from community as well.

2\. `iter8ctl` is also a kubernetes client as implied by the above command. So, there is no need to run `kubectl get experiment <name> -o yaml | iter8ctl describe -f -` anymore. The command is substantially shorter.

What does this command look like when the -l option is not selected? And what does it look like with assertions.

I pulled and reviewed the help; It appears that one should use either -l or -e and perhaps -n. I suggest that we don't require the -e. That the name be an optional parameter without a flag. This will make it more like a kubectl command ... which is part of the intent of these changes (to make it a kubectl client).
I wonder also if lack of a name might imply -l.  So iter8ctl describe suffices for latest. And iter8ctl assert -n bookinfo-iter8 quickstart-exp -c completed is a more complex example for experiment quickstart-exp in ns bookinfo-iter8.

What does this command look like when the -l option is not selected?

iter8ctl describe -e exp -n ns: describe the experiment named exp in the namespace ns. The -n flag is optional; namespace is assumed to be default if not specified.
iter8ctl describe -l: describe the last experiment created in any namespace. -e and -n flags, if specified, will be ignored.
iter8ctl assert -e exp -n ns -c completed -c winnerFound: assert that the experiment  has completed and  a winner has been found. Currently, these are the only two assertions supported. This list will be extended to support a variety of other assertions in the future based on the info in the experiment object. -c can be repeated one or more times in an assert command.
iter8ctl assert -e exp -n ns -c completed,winnerFound is also valid.
iter8ctl assert -l -c completed is also valid.

Is the -l flag apply to all namespaces or just the ns specified by -n. I assume the latter. But the question came to mind as I read the help

When -l flag is used -n flag is ignored. I believe --help makes this clear.
+1 for dropping -e and -l.
However, could we please handle this as a separate issue? I need to understand how positional arguments (i.e., arguments without flags) work in cobra/spf13 in order to implement this feature. Given that it is going to require some significant learning on my part (which I am happy to do), could we please put this off until after the upcoming community meetup?

@kalantar Fixed as per your comments.

iter8ctl describe exp -n ns: describe the experiment named exp in the namespace ns. The -n flag is optional; namespace is assumed to be default if not specified.
iter8ctl describe: describe the last experiment created in any namespace. -n flag, if specified, will be ignored.
iter8ctl assert exp -n ns -c completed -c winnerFound: assert that the experiment has completed and  a winner has been found. Currently, these are the only two assertions supported. This list will be extended to support a variety of other assertions in the future based on the info in the experiment object. -c can be repeated one or more times in an assert command.
iter8ctl assert exp -n ns -c completed,winnerFound is also valid.
iter8ctl assert -c completed is also valid. The last experiment is used for assertions.

Done.


Documentation: AzureFileCopyV4 output variable documentation incomplete

Required Information
Entering this information will route you directly to the right team and expedite traction.
Question, Bug, or Feature?
Type: Bug
documentation issue
Enter Task Name: AzureFileCopyV4
list here (V# not needed):
https://github.com/microsoft/azure-pipelines-tasks/tree/master/Tasks/AzureFileCopyV4
Environment

Server - Azure Pipelines

If using Azure Pipelines, provide the account name, team project name, build definition name/build number: n/a

Agent - Hosted or Private:

If using Hosted agent, provide agent queue name: windows-latest

Issue Description
The AzureFileCopyV4 task has poor documentation on working with its output variables. In v3, the variables outputted were task inputs. In v4, they are created as output variables (and not defined in the task ui). The documentation between v3 and v4 does not change in any way to reflect this.
You have to know to dig into the task code to find out that the output variables are titled:

StorageContainerUri
StorageContainerSasToken

Following the output variables documentation, we can then reference the variables like so:
- task: AzureFileCopy@4
  inputs:
    SourcePath: 'Readme.md'
    azureSubscription: 'Test'
    Destination: 'AzureBlob'
    storage: 'azdo'
    ContainerName: 'workshops'
    BlobPrefix: ''
  name: AzureFileCopy
- script: |
    echo $(AzureFileCopy.StorageContainerUri)
    echo $(AzureFileCopy.StorageContainerSasToken)

I can submit a PR for the readme update if desired.   Thanks
Task logs
N/a
Troubleshooting
n/a
Error logs
n/a

@soccerjoshj07 Thank you for this. Been beating my head at this for the whole day :+1:

@soccerjoshj07 thanks for pointing this out. If you would like to raise PR, please go ahead, and reference the PR in this issue.

@arjgupta No problem. Just created PR #13430 with the updated documentation and an example. Let me know if there are any tweaks that are needed.  Thanks!

@N-Usha  Can you please review the PR for documentation :  https://github.com/microsoft/azure-pipelines-tasks/pull/13430/files0

Closing this issue as documentation PR is merged


fix: add fallback value for slack alarm messages

The fallback field is among other things used as the content of the corresponding Slack notification. When this field is omitted, all notifications say "no preview available".

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


bug(notifications): Distributing notifications to too many subscriptions

🐞 Bug Report
Affected Services

 The issue is located in:
support-notifications-go
Is this a regression?

 Yes, the previous version in which this bug was not present was: ....
doubtful
Description and Minimal Reproduction
It does not appear that the set of subscriptions is filtered correctly when distributing notifications.  Instead of looking for subscriptions where categories AND labels match, it seems like it ends up doing an OR instead.  As a result anyone with a subscription on the category gets all notifications sent to it, regardless of label.
https://github.com/edgexfoundry/edgex-go/blob/f60a6b9b3bbcf5933f7773f85fc76d84301e173f/internal/pkg/db/redis/notifications.go#L428-L451
🔥 Exception or Error

🌍  Your Environment
Deployment Environment:
Redis database
EdgeX Version:
1.3.0
Anything else relevant?

I am going to be working on a fix against 1.3.0 for something we need to deliver in the near term but can come back and bring a PR against master if I have time.

@cloudxxx8 , is this issue in the new V2 ?

@lenny-intel we haven't implemented the notification handling mechanism.
@FelixTing for awareness

we need to fix this in V2

@AlexCuse This issue fixed at V2, could you help verify that?
https://github.com/edgexfoundry/edgex-go/blob/3372c345ca573018b8293c0333122174fb4d3907/internal/pkg/v2/infrastructure/redis/subscription.go#L231-L245

Sure I will try to set up a test early next week @weichou1229.
This code looks right to me I remember the v1 change was to use redis SINTER command.

Sorry for the delay @cloudxxx8 I got pulled into some 1.3 stuff and by the time I got back to it my 2.0 services needed considerable updating.  I just tested and I am not getting the duplicates anymore, think this can be closed if your associated tasks are complete.

thanks for the confirmation, @AlexCuse


dx_id should be updated when component is copied on same page

right now, because dx_pageId does not change, it will not be updated, and we'll end up having twice the same id

fixed


[Bug] old change data that will be captured when the latest mode starts

Search before asking

[X] I searched in the issues and found nothing similar.

Flink version
flink 1.13.5
Flink CDC version
master
Database and its version
sqlserver2012
Minimal reproduce step

Configure from LATEST_ OFFSET Start CDC task
Observe that the captured data contains old change data.

What did you expect to see?
Change data from now on
What did you see instead?
Old change data
Anything else?
No response
Are you willing to submit a PR?

[X] I'm willing to submit a PR!

https://github.com/ververica/flink-cdc-connectors/pull/2176 merged.


Low level create table

Description
This is the current implementation for a low level create table api
Looking for feedback

I think the new() method for DeltaTableMetaData needs to take in format (just defaulted to parquet to get things working)
I think the create() method should return an updated version of the DeltaTable instead of nothing at the moment

@Smurphy000 could you help manually resolve the comments you already addressed with follow up commits? That will make it a little bit easier for us to do follow up reviews :)

Thank you @Smurphy000, I hope you had a good time hacking on rust and delta ;)


Turn on noUncheckedIndexAccess TS config

This would be very nice for type soundness: https://www.typescriptlang.org/tsconfig#noUncheckedIndexedAccess

For someone who's motivated 💀

I don't think it would be worthwhile to refactor all of this code to check for index existence. I think a better approach will be to apply non-null assertions, and––from here on out––take advantage of noUncheckedIndexAccess. @mitschabaude want to assign this to me?

@harrysolovay if i understand you correctly, I agree: in most cases the initial fix should be something like assert(item !== undefined) before using the item that was obtained with an index access -- instead of refactoring the array to a tuple type / checking that the key is in the object.
I'm happy to see you having a go at this. Since it's a big refactor, it could also be done iteratively in several PRs: make more code handle the unchecked index access without yet committing the TS config change.

By "non-null" assertion I mean the operator.
declare const value: string | undefined
value! // `string`

We can apply these (!) pretty much everywhere there's a TS error stemming from potentially-undefined value access. I believe a large portion of these will need to be applied to the generated bindings of o1js-bindings. Might make sense to tweak the tsconfig within that directory alone.

By "non-null" assertion I mean the operator.

oh I see. I don't particularly like that operator, because it's neither type-safe nor a runtime assertion. It's like as <the type without null>. Using that operator everywhere is equivalent to not toggling noUncheckedIndexAccess on, which is fine as an upgrade path but not where I want to be.

I believe a large portion of these will need to be applied to the generated bindings of o1js-bindings.

No, we don't generate any TS code apart from type definitions. Bindings artifacts are JS


Properly fix hashmap invalidation issue

This PR implemented a workaround: https://github.com/bpfkorea/agora/pull/1023
However it is inefficient, we should not use an array for all the list of clients since the array may grow too much (although we could put limitations in place, and remove unused clients).
Here is a test-case showing the issue:
import std.stdio;
import agora.utils.Test;
import agora.common.crypto.Key;

void test () @safe
{
    int[PublicKey] map;
    map[WK.Keys[0].address] = 10;
    map[WK.Keys[1].address] = 20;
    map[WK.Keys[2].address] = 30;
    map[WK.Keys[3].address] = 40;
    map[WK.Keys[4].address] = 50;
    size_t count;
    foreach (ref key, ref value; map)
    {
        writeln("In map (before blowItUp) Key: ", key, ", value: ", value);
        ++count;
        if (count == 3)
            blowItUp(map);
        writeln("In map (after blowItUp) Key: ", key, ", value: ", value);
    }
}

void blowItUp (ref int[PublicKey] map) @safe
{
    for (int v = 6; v < 512; ++v)
    {
        map[WK.Keys[v].address] = v * 10;
        writeln("Adding: Key: ", WK.Keys[v].address, ", value: ", v * 10);
    }
}

unittest
{
    test();
}

Notice the corruption here in the printouts:
n map (after blowItUp) Key: GDC22CFFKB4ZNRZUP6EMRIGVZSQEPSNH2CBMWLU5GLGKE36M3KX5YD36, value: 30
In map (before blowItUp) Key: GAEAAAAAAAAAAAGPHEVAWAIAAAAAAAAAAAAAAAAAIAA5SCQBAAAABOWV, value: 10
In map (after blowItUp) Key: GAEAAAAAAAAAAAGPHEVAWAIAAAAAAAAAAAAAAAAAIAA5SCQBAAAABOWV, value: 10
In map (before blowItUp) Key: GABQAAAAAAAAAAFKHEVAWAIAAAAAAAAAAAAAAAAAUD5NQCQBAAAAB2H7, value: 10
In map (after blowItUp) Key: GABQAAAAAAAAAAFKHEVAWAIAAAAAAAAAAAAAAAAAUD5NQCQBAAAAB2H7, value: 10
In map (before blowItUp) Key: GAFAAAAAAAAAAAGEHEVAWAIAAAAAAAAAAAAAAAAAUD65QCQBAAAAAYG6, value: 8
In map (after blowItUp) Key: GAFAAAAAAAAAAAGEHEVAWAIAAAAAAAAAAAAAAAAAUD65QCQBAAAAAYG6, value: 8
In map (before blowItUp) Key: GAFAAAAAAAAAAAGYHEVAWAIAAAAAAAAAAAAAAAAA6AA5SCQBAAAAAMIZ, value:<PHONE_NUMBER>
In map (after blowItUp) Key: GAFAAAAAAAAAAAGYHEVAWAIAAAAAAAAAAAAAAAAA6AA5SCQBAAAAAMIZ, value:<PHONE_NUMBER>

At first I suspected it was a call to _aaRehash, however this is actually the user-facing API for the .rehash() call (__aaRehash will not be called automatically). There is probably an internal call to one of the functions here: https://github.com/dlang/druntime/blob/master/src/rt/aaA.d#L126-L155
But I haven't debugged it yet.

Somewhat annoyingly I cannot reproduce this outside agora, with these two files (with multiple runs, because the agora test doesn't fail every time):
test.d: https://gist.github.com/AndrejMitrovic/935956fd25e62b17d60a91767c6e71ff
base32.d: https://gist.github.com/AndrejMitrovic/fb54de5f9679fa23978aa6ec3a72a68a
I've used the same compilation flags -extern-std=c++14 -d-debug -g -unittest -wi -oq.

If I copy paste this into a new module agora/common/What.d into agora and run it there, it reproduces:
dtest=agora.common.What dub -q test
> In map (after blowItUp) Key: GAFAAAAAAAAAAAEILS6A2AIAAAAAAAAAAAAAAAAAGARWWDIBAAAAAAAA, value:<PHONE_NUMBER>

If I change the test-suite runner to not use parallel, the issue goes away.

I am just starting out with D, so might be wrong. In C++ it is not recommended to iterate through an std::unordered_map while adding new elements to it. The reason is that as the hashmap grows and the density increases, C++ might decide to double to bucket size and reallocate the memory. This reallocation will be hidden from the ususal C++ iterators. I think something similar is going on here.
 Whatever extra elements are printed out is determined by the hash function which I think is implementation dependent.
Regarding the safeMethod2, it actually segfaults on Ubuntu

output of safeMethod2
second pass
key: 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5 5
value: 5 address: 7FFFF6EB7290
key: 6 6 6 6 6 6 6 6 6 6 6 6 6 6 6 6 6 6 6 6 6 6 6 6 6 6 6 6 6 6 6 6
value: 6 address: 7FFFF6EB7340
key: 21 21 21 21 21 21 21 21 21 21 21 21 21 21 21 21 21 21 21 21 21 21 21 21 21 21 21 21 21 21 21 21
value: 21 address: 7FFFF6EB7D90
key: 1 1 1 1 1 1 1 1 1 1 1 1 1 1 1 1 1 1 1 1 1 1 1 1 1 1 1 1 1 1 1 1
value: 1 address: 7FFFF6EA9FA0
key: 3 3 3 3 3 3 3 3 3 3 3 3 3 3 3 3 3 3 3 3 3 3 3 3 3 3 3 3 3 3 3 3
value: 3 address: 7FFFF6EB7130
key: 8 8 8 8 8 8 8 8 8 8 8 8 8 8 8 8 8 8 8 8 8 8 8 8 8 8 8 8 8 8 8 8
value: 8 address: 7FFFF6EB74A0
key: 9 9 9 9 9 9 9 9 9 9 9 9 9 9 9 9 9 9 9 9 9 9 9 9 9 9 9 9 9 9 9 9
value: 9 address: 7FFFF6EB7550
key: 24 24 24 24 24 24 24 24 24 24 24 24 24 24 24 24 24 24 24 24 24 24 24 24 24 24 24 24 24 24 24 24
value: 24 address: 7FFFF6EB7FA0
key: 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14
value: 14 address: 7FFFF6EB78C0
Thread 1 "simpleTest" received signal SIGSEGV, Segmentation fault.
0x00007ffff73e19e0 in _D4core8internal4hash__T9bytesHashVbi0ZQpFNaNbNiNeMAxhmZm () from /lib/x86_64-linux-gnu/libdruntime-ldc-shared.so.90
(gdb) bt
#0  0x00007ffff73e19e0 in _D4core8internal4hash__T9bytesHashVbi0ZQpFNaNbNiNeMAxhmZm () from /lib/x86_64-linux-gnu/libdruntime-ldc-shared.so.90
#1  0x00007ffff7405830 in _aaInX () from /lib/x86_64-linux-gnu/libdruntime-ldc-shared.so.90
#2  0x000055555564294b in _D5agora5utils10SimpleTest11safeMethod2FNfZ14__foreachbody2MFNfKxSQCmQCjQCg5ByteSKiZi (key=..., value=) at /s/source/agora/utils/SimpleTest.d:83
#3  0x00007ffff7405f1f in _aaApply2 () from /lib/x86_64-linux-gnu/libdruntime-ldc-shared.so.90
#4  0x000055555564246d in agora.utils.SimpleTest.safeMethod2() () at /s/source/agora/utils/SimpleTest.d:81
#5  0x0000555555642c05 in D main (args=...) at /s/source/agora/utils/SimpleTest.d:103

I think the correct way of iterating through would be either

foreach(const ref keys;myMap.keys()){
...here we can modify myMap directly, no problem
}

foreach(ref key, ref value;myMap)
{
...we cannot add new values to myMap, BUT we can add them to a temporary map and later merge it with myMap
}

There was already an open bug for it, so I added the new code and some informations: https://issues.dlang.org/show_bug.cgi?id=12218#c7
Also edited #567 to include it.

@ferencdg the .keys call will allocate a new array. We don't really want to do that every time we need to make a batch of requests.
The problem we're trying to solve is being able to iterate and send requests over a set of clients where the set may grow and where each time a request is sent we may end up with a context switch. To put it into pseudocode:
void add ()  // background fiber which adds new clients
{
    while (1)
    {
        auto addr = ...;
        clients[addr] = new Client();
        yield();  // may cause a context switch into run()
    }
}

void run ()  // separate fiber
{
    foreach (client; clients.byKey())
        client.request();  // may cause a context switch into add()
}

Therefore I suggest replacing the two arrays here with singly-linked lists: https://github.com/bpfkorea/agora/blob/a490dade705e5d9367e74c40793f2f430ae52b76/source/agora/network/NetworkManager.d#L328-L332
There's an implementation here: https://dlang.org/phobos/std_container_slist.html
We don't do any removals of the clients (yet), if we do we could replace it with a doubly-linked list at a later point. Let's keep it simple for now.

@AndrejMitrovic alright it makes sense, I can replace them with single linked lists
I was aware that .keys() create a temporary which might be costly, just wasn't aware of the original problem

I was aware that .keys() create a temporary which might be costly, just wasn't aware of the original problem

Indeed, I haven't described the actual underlying problem well - only the symptom.

Fixed by #1123


Add shortage to Electricity Sankey

In https://github.com/quintel/etsource/pull/2644 a number of fixes have been made to the Electricity Sankey. One of these is that shortages due to blackouts have been added to the Sankey queries. This PR adds blackouts to the Sankey.

@antw shortages have been added to the Sankey as "blackouts", but perhaps it would better visually when the flow has a gradient like the losses flow has as well. Would it be possible to add this gradient to the blackouts flow and would it be possible to add a "losses" label to the loss flow?


[main] Upgrade to latest dependencies

Cron -knative-prow-robot
/cc knative-sandbox/kperf-approvers
/assign knative-sandbox/kperf-approvers
Produced by: knative-sandbox/knobots/actions/update-deps

/retest


Not working in symfony 5.2

Here is my detailed explanation https://stackoverflow.com/questions/65649554/omines-datatables-in-symfony-5-does-not-work.
I will appreciate documentation and examples of omines datatables-bundle.
Thank you.

It works very well for me in 5.2.
Try without "minimum-stability": "dev", in your composer.json

It works very well for me in 5.2.
Try without "minimum-stability": "dev", in your composer.json

I've got a working 5.2.2 project and wanted to add this bundle. Getting exactly the same error. Starting a new project like in te Stack Overflow isn't an option. How can I troubleshoot what's missing? Already get the Sensio Extra bundle as suggested.

@shades684 we're using the bundle in multiple 5.2.6 projects right now I believe?

Hi, guys. We're using this bundle in 5.2.3 symfony projects. Everything is going perfectly

I've also tested it, no issue whatsoever and certainly no 'phantom' dependencies like Sensio.
Closing as there is no reproduction scenario.


"mg-anon-func-jquery" do not work!

Snippet mg-anon-func-jquery don't work!

Hi Rafael! Thanks for open this issue! Please fell free to pull request the fix to that :)
Check the contribution text to check how to do that.

Doesn't exist anymore? :|

I renamed it to 'iife' ;)


Discord not alerting notifications until focused

Already reported ? *

[X] I have searched the existing open and closed issues.

Regression?
Yes
System Info and Version
 System/Version info
Hyprland, built from branch  at commit 9a09eac79b85c846e3a865a9078a3f8ff65a9259  (props: bump version to 0.42.0).
Date: Wed Aug 7 19:17:10 2024
Tag: v0.42.0, commits: 5069

flags: (if any)

System Information:
System name: Linux
Node name: oliman-arch
Release: 6.10.10-zen1-1-zen
Version: #1 ZEN SMP PREEMPT_DYNAMIC Thu, 12 Sep 2024 17:17:51 +0000

GPU information:
03:00.0 VGA compatible controller [0300]: Advanced Micro Devices, Inc. [AMD/ATI] Navi 32 [Radeon RX 7700 XT / 7800 XT] [1002:747e] (rev c8) (prog-if 00 [VGA controller])

os-release: NAME="Arch Linux"
PRETTY_NAME="Arch Linux"
ID=arch
BUILD_ID=rolling
ANSI_COLOR="38;2;23;147;209"
HOME_URL="https://archlinux.org/"
DOCUMENTATION_URL="https://wiki.archlinux.org/"
SUPPORT_URL="https://bbs.archlinux.org/"
BUG_REPORT_URL="https://gitlab.archlinux.org/groups/archlinux/-/issues"
PRIVACY_POLICY_URL="https://terms.archlinux.org/docs/privacy-policy/"
LOGO=archlinux-logo

plugins:

======Config-Start======

Config File: /home/oil/.config/hypr/hyprland.conf: Read Succeeded
# Monitor
monitor = ,highrr, auto, auto

debug:disable_logs = false

$wallpaperDir = "/mnt/moredrive/Wallpapers/asus-rog-genesis-animated-wallpaper-pc.mp4"

# Startup Apps
exec-once = swaync
# exec-once = mpvpaper -o "--loop" '*' $wallpaperDir
exec-once = hyprpaper
exec-once = swaylock -C ~/.config/swaylock/config -i /home/oil/Pictures/wallpaper.jpeg #to lock on boot
exec-once = /usr/lib/polkit-gnome/polkit-gnome-authentication-agent-1
exec-once = udiskie
exec-once = waybar
exec-once = wl-paste --watch cliphist store
#exec-once = wl-paste --type image --watch cliphist store #Stores only image data
#exec-once = copyq --start-server
exec-once = wl-clip-persist --clipboard both
exec-once = /usr/bin/discord --start-minimized
exec-once = blueman-applet
exec-once = /usr/bin/steam-runtime -silent %U
#exec-once = qbittorrent
#exec-once = premid
#exec-once = discover-overlay
#exec-once = vesktop --start-minimized
exec-once = nm-applet

# Animations
bezier = overshot, 0.05, 0.9, 0.1, 1.1
animation = windows, 1, 4, overshot, popin
animation = workspaces, 1, 4, default, slide

# Hypr Variables
$terminal = alacritty
$fileManager = nemo
$menu = rofi -show drun
$browser = firefox
$suspendCmd = pidof swaylock || swaylock -C ~/.config/swaylock/config -i /home/oil/Pictures/wallpaper.jpeg &>/dev/null & disown; sleep 0.1; systemctl suspend

# env Variables
env = XCURSOR_SIZE, 24
env = QT_QPA_PLATFORMTHEME, qt5ct
env = ELECTRON_OZONE_PLATFORM_HINT, wayland

# Main Config
general {
    border_size = 2
    col.inactive_border = rgba(216633aa)
    col.active_border = rgba(42df66ff)
}
decoration {
    rounding = 5
}
input {
    accel_profile = flat
    numlock_by_default = true
}
misc {
    vrr = true
}

# Binds
$mainMod = SUPER

bind = ,Print, exec, grim -g "$(slurp)"
bind = $mainMod, Print, exec, grim

bind = $mainMod, Q, exec, $terminal
bind = $mainMod, C, killactive
bind = $mainMod, M, exit
bind = $mainMod, E, exec, $fileManager
bind = $mainMod, V, togglefloating
bind = $mainMod, R, exec, $menu
bind = $mainMod, J, togglesplit
bind = $mainMod, W, exec, $browser
bind = $mainMod, P, exec, $suspendCmd
bind = $mainMod, L, exec, swaylock -C ~/.config/swaylock/config -i /home/oil/Pictures/wallpaper.jpeg
bind = $mainMod, F, fullscreen
bind = $mainMod, K, exec, wlogout
#bind = $mainMod, D, pin window activewindow
binde = ,XF86AudioRaiseVolume, exec, wpctl set-volume -l 1.5 @DEFAULT_AUDIO_SINK@ 5%+
binde = ,XF86AudioLowerVolume, exec, wpctl set-volume -l 1.5 @DEFAULT_AUDIO_SINK@ 5%-
binde = ,XF86AudioMute, exec, wpctl set-mute @DEFAULT_AUDIO_SINK@ toggle
bindl = ,XF86AudioPlay, exec, playerctl play-pause
bindl = ,XF86AudioNext, exec, playerctl next
bindl = ,XF86AudioPrev, exec, playerctl previous
bindl = ,XF86AudioStop, exec, playerctl stop
bindl = $mainMod, A, exec, wpctl set-mute @DEFAULT_AUDIO_SOURCE@ toggle
# bind = ,XF86PowerOff, exec, $suspendCmd
bind = $mainMod, T, exec, cliphist list | rofi -dmenu | cliphist decode | wl-copy

bind = $mainMod, left, movefocus, l
bind = $mainMod, right, movefocus, r
bind = $mainMod, up, movefocus, u
bind = $mainMod, down, movefocus, d

bind = $mainMod, 1, workspace, 1
bind = $mainMod, 2, workspace, 2
bind = $mainMod, 3, workspace, 3
bind = $mainMod, 4, workspace, 4
bind = $mainMod, 5, workspace, 5
bind = $mainMod, 6, workspace, 6
bind = $mainMod, 7, workspace, 7
bind = $mainMod, 8, workspace, 8
bind = $mainMod, 9, workspace, 9
bind = $mainMod, 0, workspace, 10
bind = $mainMod, code:49, workspace, r-1
bind = $mainMod, code:23, workspace, r+1

bind = $mainMod SHIFT, 1, movetoworkspace, 1
bind = $mainMod SHIFT, 2, movetoworkspace, 2
bind = $mainMod SHIFT, 3, movetoworkspace, 3
bind = $mainMod SHIFT, 4, movetoworkspace, 4
bind = $mainMod SHIFT, 5, movetoworkspace, 5
bind = $mainMod SHIFT, 6, movetoworkspace, 6
bind = $mainMod SHIFT, 7, movetoworkspace, 7
bind = $mainMod SHIFT, 8, movetoworkspace, 8
bind = $mainMod SHIFT, 9, movetoworkspace, 9
bind = $mainMod SHIFT, 0, movetoworkspace, 10

bind = $mainMod CTRL, 1, movetoworkspacesilent, 1
bind = $mainMod CTRL, 2, movetoworkspacesilent, 2
bind = $mainMod CTRL, 3, movetoworkspacesilent, 3
bind = $mainMod CTRL, 4, movetoworkspacesilent, 4
bind = $mainMod CTRL, 5, movetoworkspacesilent, 5
bind = $mainMod CTRL, 6, movetoworkspacesilent, 6
bind = $mainMod CTRL, 7, movetoworkspacesilent, 7
bind = $mainMod CTRL, 8, movetoworkspacesilent, 8
bind = $mainMod CTRL, 9, movetoworkspacesilent, 9
bind = $mainMod CTRL, 10, movetoworkspacesilent, 10

bindm = $mainMod, mouse:272, movewindow
bindm = $mainMod, mouse:273, resizewindow

# Screen Sharing
exec-once = dbus-update-activation-environment --systemd WAYLAND_DISPLAY XDG_CURRENT_DESKTOP

======Config-End========

Description
Notifications from Discord are not showing up or even making a noise until the window is focused. I'm using swaync with the default Discord client with no mods.
How to reproduce
Unfocus Discord and just go about your business. You'll start getting mobile notifications instead of desktop ones.
Crash reports, logs, images, videos
No response

not a hyprland issue. The official discord client is a notorious shitfest, and it might freeze when invisible. You can try the renderunfocused windowrule once you update from your out-of-date install


MenuGroup missing props - expanded by default?

🚀 Feature Proposal
MenuGroup should have the ability to be expanded by default.
Example

Can you consider extending this expandable behavior into a separate component?
The existing one needs to set a Menu first, then a MenuGroup and then a MenuItem

Any updates for this?

Any updates?


[dribbblish] ui theme broken

Prerequisites
Os: Ubuntu 20.04
Spicetify version: 2.16.1
Spotify version: <IP_ADDRESS>6.gc5f8b819
Describe the bug
nowPlayingBar doesnt follow the lenght of its container. I installed many times spicetify and spotify but the ui wont fix.
I tried all the color schemes but the problem persist. Out of curiosity i tried to dribblish dynamic and everything is working fine.
Screenshots

Logs
config-xpui.txt
console log

I have the same issue. Pretty sure most of these themes need to be updated to the new spotify version for linux and will remain broken until then.

this is not resolved, as far as I can tell. Why did you close it, @harbassan ?


Prevent context update after/while rebalancing

Hi,
This method leads to the context being updated with previously assigned partitions, and to a fatal error of trying to seek to a "no longer assigned partition".
Reproduction can be done by submitting a BQ connector with one task, and update that same connector to use 5 tasks for instance.
It really often ends up with the first created task to fail with that exception :
[2019-06-05 10:19:20,150] ERROR WorkerSinkTask{id=mybigquery--0} Task threw an uncaught and unrecoverable exception (org.apache.kafka.connect.runtime.WorkerTask:177)
java.lang.IllegalStateException: No current assignment for partition mypartition-0
	at org.apache.kafka.clients.consumer.internals.SubscriptionState.assignedState(SubscriptionState.java:259)
	at org.apache.kafka.clients.consumer.internals.SubscriptionState.seek(SubscriptionState.java:264)
	at org.apache.kafka.clients.consumer.KafkaConsumer.seek(KafkaConsumer.java:1501)
	at org.apache.kafka.connect.runtime.WorkerSinkTask.rewind(WorkerSinkTask.java:601)
	at org.apache.kafka.connect.runtime.WorkerSinkTask.access$1200(WorkerSinkTask.java:70)
	at org.apache.kafka.connect.runtime.WorkerSinkTask$HandleRebalance.onPartitionsAssigned(WorkerSinkTask.java:675)
	at org.apache.kafka.clients.consumer.internals.ConsumerCoordinator.onJoinComplete(ConsumerCoordinator.java:291)

I was able to load in BigQuery millions of records with that fix, while killing the connector, adding and removing tasks while still not loose any records.
Could you please double check that behavior and merge that PR ?
Have a good day
Nicolas

@mtagle IIRC we chatted about this a while back and decided the method was basically useless at this point and safe to remove, right?

Codecov Report

Merging #161 into master will increase coverage by 0.09%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #161      +/-   ##
============================================
+ Coverage      65.1%   65.19%   +0.09%
+ Complexity      238      237       -1
============================================
  Files            33       33
  Lines          1367     1362       -5
  Branches        130      130
============================================
- Hits            890      888       -2
+ Misses          439      437       -2
+ Partials         38       37       -1

Impacted Files
Coverage Δ
Complexity Δ

...wepay/kafka/connect/bigquery/BigQuerySinkTask.java
58.75% <ø> (+0.56%)
23 <0> (-1)
:arrow_down:

I wonder if the bug is not necessarily that the connector is attempting to manage the offsets, but how it is doing it. The updateOffsets(...) is essentially trying to set the task context's offsets  to those it got directly from Connect via the flush(...) method. This definitely is unnecessary and problematic, as noted above.
But if it is desired that the connector track its own offsets, then it should use the preCommit(...) method that was added as part of KIP-89, and which first appeared in AK <IP_ADDRESS>. Connect now calls this method periodically, and passes in what it thinks the offsets are for each topic partition that have been consumed. By default, the preCommit(...) method simply calls flush(...) with the same offsets. This way, connectors that override flush(...) still work.
A connector can, however, override preCommit(...) so that it returns the actual offsets for each topic partition of the records it has completely processed. And when it does this, Connect will asynchronously commit those offsets on the consumer (or synchronously if the task is closing).

@criccomini @mtagle @jgao54 ping :)

:looking:

Based on what @rhauch  is saying, I think we want to override preCommit(). Reason being is that we need to have flushes (when we guarantee writes to BQ are done successfully) fully finished before we commit our offsets. As such, I think we should do:
preCommit() {
  flush();
  saveOffsets();
}

Talked with @mtagle about this a bit more. It actually sounds like we don't need to manage the offsets ourselves since we apparently fully flush all messages (or time out on flush) before returning. As such, I think we can offload the work to Kafka connect. The PR should suffice then. Going to let @mtagle confirm before merging.

(Also, planning to release in the next 2 weeks)

As far as I can tell, KCBQ should not need to keep track of it's own offsets, at least if it's just doing normal BQ streaming. .flush() makes sure that all the rows that we've been given have been successfully to BigQuery before it returns.
(Going through GCS is a little bit more complicated but after .flush() completes successfully you can at least be sure that the data has reached GCS. Since GCS loading is technically in beta this is something I feel comfortable with.)
So, with that being said I'm totally happy to get rid of this method, and I believe it is safe to do so.

Thanks @mtagle @criccomini!


Tcti device printf

This is a significant departure from the way we currently handle the debugging functions in the TCTIs. The way we've done this up till now was to have a global function pointer set at compile time. This was typically controlled by a pre-processor macro. The global data this approach causes a litany of problems.
The refactoring I've been doing has all been leading up to decoupling the TCTIs from the application code (removing this shared global data) while preserving the current debug messages. This PR is an experiment in allowing the application consuming the TCTI to provide a callback function and some data that the TCTI will invoke in place of the global debug function pointer. This PR only addresses the issue in the device TCTI since it's a more simple case. The socket TCTI comes next.
Very interested in feedback not just on the implementation but on the general approach (user supplied callback). This was the most flexible thing I could think up and it's a pretty common pattern (event driven etc) but that's not to say that I've anticipated every possible contingency and it may be that this approach has some flaws that I haven't anticipated.

No, I was thinking for consistency's sake, use the TCTI_LOG everywhere.  I didn't see the later use of TCTI_LOG to output the initializing message.  In that case, do you really need the first one that uses the function pointer?

I agree. But unless I'm missing something I think you're getting confused reading the diff. Your comment is on a line of code that this PR removes. This function pointer is removed completely by this PR.

My mistake...looked right at it and didn't see it.


Wiki Biome Bug

If you add the apostrophe it does this. Shika also said that it does it with pokemon like Aegislash that just don't spawn, instead of just saying "this pokemon doesn't spawn".

Please re-open if this is still an issue.


Change misleading wording in exit message

PR #688 added a message when the exit status != 0. However, when the exit status is non-zero because a test failed, the message wording is misleading because it specifically refers to SimpleCov. I personally interpreted it as an error in SimpleCov's report generation, when SimpleCov was actually working just fine!
This PR makes the wording more generic so as to not mislead one into thinking SimpleCov is misbehaving when it isn't. 😄
Also, the message was missing the trailing "\n", so added that as well.
Old output:
SimpleCov failed with exit 1$

New output:
Exiting with status 1
$

:wave:
Hi there, thanks for contributing!
I'm not sure we should drop simplecov from the message but rather improve the message as I'd wanna know where the message comes from.
Something like SimpleCov made the process exit with $reason or something :)

Sure, and yet that's just the thing--SimpleCov is also intercepting exit statuses from earlier up (eg: a test failure), and is then claiming to have generated/caused the error.
When SimpleCov actually causes the error, then I agree, it's ideal for SimpleCov to identify itself as the source of the non-zero exit. However, when SimpleCov is intercepting a non-zero status originating elsewhere, it should not then claim that as its own.
Perhaps it'd be better to entirely remove the message added in #688, and then add separate messages elsewhere in SimpleCov when SimpleCov itself sets the exit status to non-zero based on its own conditions (ie: insufficient test coverage)?

@zarqman I think it's cool for simplecov to identify itself as the originator of the message but you are right it could be more explicit like "a process failed with status x" and otherwise being more explicit about that simplecov set the exit status due to reason y I agree about that.
Thinking about it... I agree, maybe only if we set the exit status ourselves due to our logic and not because it bubbled up, good idea @zarqman !

#742 will cause a small merge conflict for you but I think it's fine :)

@zarqman any plans on pursuing this or shall I take over?

haven't had a chance to get back to this...sorry! feel free to take it over.

Taken the new approach to #906


Adding watch of Cluster operators

https://github.com/openshift-scale/cerberus/issues/43
This is a start for this issue.
I created a key that is optional in a config.yaml file. It will be set to false if it not in the yaml.
Only if the watch_operators key is set to true does it get the cluster operators and verify that the degraded status is False. If the status is not false it adds the operator to the failed list which gets returned to the start_cerberus.py file to log.
I added the key to the README.md as well.
I definitely could get rid of monitor_cluster_operators function, it is pretty much just passing the parameters around in an extra function.
Need to fix the output the I print out to be consistent with other outputs.  (Another commit to come)
Definitely might consider taking out the printing of the degraded status for each operator every iteration and just show the failed ones. (See below of large output)

Can one of the admins verify this patch?

Squashed commits

@paigerube14 Looks like there are conflicts, we need to rebase it.

I'm seeing I have no conflicts. I did have them earlier, maybe you just need to refresh?

@paigerube14  It's not allowing me to rebase and merge but can do a merge without rebase to add the commits to the base branch, so we should be good. Thanks.

I am seeing some conflicts with README.md and start_cerberus.py.

Fixed conflicts with readme and start_cerberus.


Warning in vite-plugin-svelte

When you do npm run dev this warning apears:
[vite-plugin-svelte] WARNING: The following packages have a svelte field in their package.json but no exports condition for svelte.
missing-exports-condition

It is a quite common warning, and would be a good (and usually easy) change, since the svelte field in package.json is deprecated.
Below I added the entire warning for more context.
[vite-plugin-svelte] WARNING: The following packages have a svelte field in their package.json but no exports condition for svelte.

<EMAIL_ADDRESS>
Please see https://github.com/sveltejs/vite-plugin-svelte/blob/main/docs/faq.md#missing-exports-condition for details.

I've just released a beta version that is compatible with @sveltejs/package v2.
Could you perhaps test it? @CHamalainen
npm i<EMAIL_ADDRESS>


[SourceKit] Disable automatic rebuilding of dependent-ASTs

Explanation: Previously, after a change to one open document sourcekitd would attempt to rebuild all ASTs that might be affected by that change (e.g. to update diagnostics).  While this can be useful, it create a big performance problem (both memory and time) when there are many open documents - e.g. change one file and rebuild 50 ASTs.  This behaviour assumed there would only be a few open documents corresponding to e.g. files visible at the same time in an editor, but our API forces you to keep an open document for any unsaved file changes, so that assumption doesn't hold.  While removing the automatic AST builds creates a functionality regression if any clients of sourcekitd knew about this behaviour, it seems worthwhile to avoid the huge performance issue.
Scope: Affects clients of sourcekitd that use multiple open documents.
Radar: rdar://problem/34415818
Risk: Very low; just removes a call to rebuild ASTs.
Testing: Tested manually in Xcode. We plan to introduce new API to make this more testable in the future.

@swift-ci please test


useless ui staters, text jumping

Type: Bug
1.start gnome-wayland with resolution 3840x2160
2. open big go or python file
VS Code version: Code 1.77.1 (b7886d7461186a5eac768481578c1d7ca80e2d21, 2023-04-04T23:20:37.202Z)
OS version: Linux x64 6.1.14-100.fc36.x86_64
Modes:
Sandboxed: No
Remote OS version: Linux x64 6.1.14-100.fc36.x86_64

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-3770K CPU @ 3.50GHz (8 x 3703)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: disabled_softwarevideo_encode: disabled_softwarevulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
1, 1, 1

Memory (System)
15.58GB (9.37GB free)

Process Argv
--unity-launch

Screen Reader
no

VM
0%

DESKTOP_SESSION
gnome

XDG_CURRENT_DESKTOP
GNOME

XDG_SESSION_DESKTOP
gnome

XDG_SESSION_TYPE
wayland

Item
Value

Remote
SSH: turbuntu

OS
Linux x64 6.1.14-100.fc36.x86_64

CPUs
Intel(R) Core(TM) i7-3770K CPU @ 3.50GHz (8 x 3786)

Memory (System)
15.58GB (9.37GB free)

VM
0%

Extensions (13)

Extension
Author (truncated)
Version

codespaces
Git
1.14.5

jupyter-keymap
ms-
1.1.0

remote-ssh
ms-
0.100.0

remote-ssh-edit
ms-
0.84.0

remote-explorer
ms-
0.4.0

go
gol
0.38.0

python
ms-
2023.6.1

vscode-pylance
ms-
2023.4.20

jupyter
ms-
2023.3.1201040234

jupyter-keymap
ms-
1.1.0

jupyter-renderers
ms-
1.0.15

vscode-jupyter-cell-tags
ms-
0.1.8

vscode-jupyter-slideshow
ms-
0.1.5

(1 theme extensions excluded)

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.77.3. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

I think it can be closed as it's a duplicate of #180210 and has less informations.


Ansible Task extension 01-postgis.yaml overwrites the pg_* symlinks in /usr/bin

Bug report
Describe the bug
Whilst running the playbook against a VM, I was having issues with 27-pg-repack.
One of the issues is that the makefile for pg_repack was trying to target postgres 12, whereas postgres 15 was being installed.
Using ansible pauses, I was able to confirm that pg_config --version was returning the correct version after the setup-postgres.yml tasks.
I added pauses between all the extension installs to see which was causing the issue, and the output of pg_config changed after the 01-postgis.yaml task, where it incorrectly returned postgres 12
I think something in SFCGAL or postgis is overwriting the pg_config file (and perhaps other files) from the symlinks created in setup-postgres.yml. This is then causing pg_repack to target the wrong postgres installation
Expected behavior
the pg_config file (perhaps pg_* files) in /usr/bin should not be overwritten when installing this extension
System information
running the ansible playbook against an Ubuntu20.04 VM with 16 cores
Additional context
Currently, I am duplicating the tasks from the setup-postgres.yml for re-creating the symlinks, and running that after the 01-postgis.yaml task.
Which is a bit of a hack, but I'm not sure how to find what it is in the postgis task that is replacing these files.
Running these addition steps confirms that:
/usr/bin/pgtopo_import, /usr/bin/pgtopo_export, /usr/bin/pg_config, /usr/bin/shp2pgsql, /usr/bin/raster2pgsql, /usr/bin/pgsql2shp
symlinks created during the setup-postgres.yml tasks all get overwritten during the 01-postgis.yaml task, and need to be recreated

Hi @Towerful  - thanks for reporting this!
We're investigating

closing as our build tooling has shifted away from ansible and the referenced flows are no longer maintained


commented line causing syntax errors

Reproducible on yara 3.6, however yara 3.4 handles this just fine:
$ cat testrule
rule testrule
{
// }
strings:
$ = "abc"
condition:
any of them
}
$ ./yara3.4 testrule testrule
testrule testrule
$ ./yara3.6 testrule testrule
testrule(3): error: syntax error, unexpected HEX_STRING, expecting '{'
Changing to a block comment fixes the issue:
$ cat testrule
rule testrule
{
/* } */
strings:
$ = "abc"
condition:
any of them
}
$ ./yara3.6 testrule testrule
testrule testrule

This is because comments are now allowed inside hex strings (see #455) , and YARA is identifying the sequence { // } as an hex string. Unfortunately this ambiguity doesn't have an easy solution and this is a very uncommon edge-case, so it won't be fixed.

makes sense. thanks for the quick response.


Authentication failed on dopamine jailbroken device

Authentication failed.
Try specifying -u/--username and/or -P/--password
I changed my account and password and still got the error:
./dump.py -u mobile -P xxx com.xxx
Start the target app com.xxx
unexpected error while resuming process: (os/kern) failure

you can try the below command:
./dump.py -umobile -Pxxx com.xxx


Failed to load 'BleWinrtDll.dll' because one or more of its dependencies could not be loaded.

First of all, thank you so much for this Dll! It works wonders in Unity.
I am attempting to use it on HoloLens 2 and at first it failed because of the wrong architecture, so I've built the Dll for ARM64 following this guide. Alas, now it fails because it is missing some of the dependencies. I have used Dependencies  program to check which I am missing in the BleWinrtDll built for ARM64:

MSVCP140.dll
VCRUNTIME140.dll

When I check BleWinrtDll built for x64 those Dlls are found, but I am not able to build it to HoloLens since it is wrong architecture.
This is way over my head, so I am wondering:

Is there any way to specifically define where those dependencies are?
Is there something I should look out for when I am building for ARM64 so that those Dlls are included correctly?

Any tips would be highly appreciated! As I am quite stuck.

Thank you, @adabru, the issue was that the Dll project had to be configured for UWP as @ShumWengSang so kindly pointed out in this issue.
I have created a fork of this repository and updated Unity Project to be ready for HoloLens 2 build together with the Dlls for Unity Editor and HoloLens. Also Arduino code is included if anyone will be interested: https://github.com/marianylund/BleWinrtDll.
Hope this will help the next HoloLens enthusiasts! ;D


Documents and corrects use of class conversion methods

Recently added {to,from}_{serialized,raw,dict} methods for various conversions were poorly documented, and sometimes even wrongly used. This patch adds a short summary on when to use them, and fixes some subpar bits around the code.
Fixes #1518.

Failing /tests/prepare/install should be fixed by rebasing on the latest main.


InstagramBridge error 500 "Unable to find username in search result" got unexpected answer from server

I run into the same error message described in #1419 with version: dev.2020-02-26 today and the patch I pulled manually from 7b63da522fa728811349081a0921f84bb1fa6f19 does not fix my case. I also get Error message: Unable to find username in search result.
Suspecting that it's some sort of blocking since the provided examples in this very issue do also not work while my intended request works fine on the installation by @Legalien
I run the example with wget  --header='Accept: text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,*/*;q=0.8'  --user-agent="Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/74.0.3729.169 Safari/537.36" "https://www.instagram.com/donniedhillon/" and this is the response I got:
{"errors": {"error": ["Sorry, there was a problem with your request."]}, "status": "ok", "error_type": "generic_request_error"}

Requesting the index page / works fine so they didn't block my IP per se. No idea what's going on. Mayhap it's some sort of greylisting and it starts working tomorrow.

Can confirm donniedhillon username leads to an error on my local machine, with the latest master.
XML Parsing Error: prefix not bound to a namespace
Location: http://localhost:8000/?action=display&bridge=Instagram&context=Username&u=donniedhillon&media_type=all&format=Atom
Line Number 79, Column 3:
		<media:thumbnail url="https://www.instagram.com/p/B6ArQxnD1xm/media?size=l"/>
----------------^

Works OK with Html, I think this is a weird XML bug on some PHP library.

Actually, it's weird. Last night a few hours before this thread was started I received an email about an error in my feed. But I checked the feed now and it seems to work fine.
Maybe Instagram bugged out last night for a few hours and now it's back to normal.
Everything seems to work fine on my end as far as I can see.

Can confirm the error I reported remains, only for that particular username. All other usernames I'm subscribing to are OK.

Well, 6 days later and it's still the same. Error persists. The output format doesn't matter in my case.

Same for the user pyro.keys.

Same for the user pyro.keys.

Can confirm.
Warning: Invalid argument supplied for foreach() in bridges/InstagramBridge.php on line 94

Can confirm #1531 fixes this, per #1511

@somini Sadly it doesn't fix this issue for me. I applied the patch manually but I'm not getting an Atom feed error but IG errors on the lookup already. There is no stream to be created for Atom/HTML/Json/Mrss/Plaintext at all.

@somini wrote:

Can confirm #1531 fixes this, per #1511

I don't think that's related to #1499, either. You fixed something in the output format, but #1499 is an input issue.
@bekopharm: Thanks for testing!

I don't think that's related to #1499, either. You fixed something in the output format, but #1499 is an input issue.

On my instance, I was getting a feed with those usernames, but there was a XML error. #1531 fixes it, so I can subscribe to the users mentioned above. Works on my machine...

On my instance, I was getting a feed with those usernames

You are not affected by this bug at all @somini

@teromene @SuperSandro2000  this is not fixed by<PHONE_NUMBER>cfaa10c574deabf4df3feb8e7e84af
8047041963cfaa10c574deabf4df3feb8e7e84af fixes an output issue that created an invalid RSS feed.
#1499 describes an input issue. Nothing is received that can be outputted at all. Atom feed or else.

Sorry about that. I just copied the fixes line from the original PR where I added the tests. Please reopen this.

So who do I have to bug to re-open this? @sebsauvage ?


Missing ___LiveAlgorithmSettings class for some brokerages

Expected Behavior

We can use the CreateLiveAlgorithm to deploy algorithms for all the supported brokerages.
Actual Behavior

We only have BitfinexLiveAlgorithmSettings, GDAXLiveAlgorithmSettings, InteractiveBrokersLiveAlgorithmSettings, OandaLiveAlgorithmSettings, DefaultLiveAlgorithmSettings, and TradierLiveAlgorithmSettings
Potential Solution

Add classes for the missing brokerages
Reproducing the Problem

N/A
System Information

N/A
Checklist

[x] I have completely filled out this template
[x] I have confirmed that this issue exists on the current master branch
[x] I have confirmed that this is not a duplicate issue by searching issues

[ ] I have provided detailed steps to reproduce the issue

Closing as we are going to be moving API support to the lean CLI


Unreliable play.value

It seems that the play.value sent with ioctl to start rumble keep changing between 1 and 3 back and forth, depending on the kernel version used. What's going on?

Right. This will be some compiler flag mess-up, and not value mess-up. The same code works fine if I compile it from my ide (using the yocto sdk, but with whatever cflags), and the same code stops working if it is compiled as part of the yocto image.

Solved by 7c6e46d0db732afc74d628db2bdc9c4e0912b53a
A struct went half uninitialized, and the random values were sometimes usable, sometimes not so much.