[docs-only] Various docs edits

Description
Tidy some things in docs.
How Has This Been Tested?
CI
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Technical debt
[ ] Tests only (no source changes)

Checklist:

[ ] Code changes
[ ] Unit tests added
[ ] Acceptance tests added
[ ] Documentation ticket raised:

@phil-davis Info, you many not have known it...
Do not correct stuff in /docs/services/<name>/_index.md
These files get created automatically by the build process and have their source in /services/<name>/README.md
You can see that in each page of _index.md on top where the source file to edit is described.

OK - I just let the IDE show me typos etc. Hopefully I made consistent corrections!


Normalize Cvar Values

If I'm I'm setting a boolean cvar and do:
]/set g_bot_level4 0
CVAR VALUE CHANGED: g_bot_level4 = 0 (1)
]/g_bot_level4
"g_bot_level4" - "0" - bool - whether bots use Tyrant - default: "on"
]/set g_bot_level4 off
CVAR VALUE CHANGED: g_bot_level4 = off (1)
]/g_bot_level4
"g_bot_level4" - "off" - bool - whether bots use Tyrant - default: "on"

0 and off are equivalent values yet a string compare won't catch them as such when doing Cvar::GetValue.
I'm sure similar problems exist for other cvar types too (maybe floating point?).
Currently, all of our normalization logic is only when you call the new C++ API in the engine.
We should normalize all cvars on set. I propose we do this by modifying the semantics of OnValueChanged to also return a normalized value and store that instead of the direct value the user passed in.

We should probably solve the float round trip problem first. You need a complicated algorithm to normalize a float without either losing precision or producing very ugly strings
    std::string SerializeCvarValue(float value) {
        // You'd need %.9g to guarantee a round trip but with 8 or 9 digits of precision you get
        // stuff like 0.65f -> "0.649999976"
        return Str::Format("%.7g", value);
    }

For the case of reading cvar values in Lua, I would suggest adding more APIs like Cvar.getBool, Cvar.getNumber in addition to the current one which gets a string. We don't have to care about the type, if any, the cvar is registered as; just parse it as the requested type. Similar to RocketConditionalElement

Yes that's my workaround for now


feat: update pyproject.toml

This PR aims to minimize the number of configuration files for the project. Instead of a separate .pylintrc and .style.yapf, we now would have all the configurations within the pyproject.toml file leading to a overall minimal code structure.

In 60c8cea, I added the pylint configuration to pyproject.toml using the output generated after running:
pylint --generate-toml-config

@Conchylicultor Request for Review


Do not forward syslog to logstash

Syslog events should be handled completely by httpqueue, don't forward them to logstash

handled directly by queue


Medallia customization

Hi, I'm doing a pull request to check if we can include into your branch some little changes that I made to the library:

Customise the way that point values are being rendered
Be able to zoom programatically the chart based on the data being displayed
Be able to know when the chart changed the size in order to update the zooming

It also contains the fix of the bug https://github.com/lecho/hellocharts-android/issues/105 that I copied from the master branch
Thanks for your time and for your work! It's impressive!!!!
David.

Thanks, I will look into that, probably next week:)


Add a "distribution_time" method

Note: New PR for #200 with tests added.

Add a "distribution_time" method to facilitate measuring timing for
a yielded block and report it as a distribution metric instead of as a
histogram metric.

@remeh @djmitche here's a new PR for #200 with tests added now.  Apologies for letting this drop off my radar!
If you'd prefer to re-open #200 that's fine, just lmk.

Are there other specs besides spec/statsd_spec.rb that you'd like me to add?
Is something wrong w/ the CircleCI specs? They failed due to this error:

No configuration was found in your project. Please refer to https://circleci.com/docs/2.0/ to get started with your configuration.

https://app.circleci.com/pipelines/github/DataDog/dogstatsd-ruby/121/workflows/fd72df13-728d-4c65-9d50-4d09c96506c5/jobs/889

It looks like the patch is based on a version of master that still used circleci (ef41904b914346bcd6b3a2c682c8d4c7f6e6fe10).  A rebase should help.

@djmitche thanks, I've rebased on latest master and force-pushed.

@remeh does this look ok?

Great! I left a follow-up comment above. Anything else before someone can merge this?  Thanks agian.

Yep absolutely, latest change + the additional tests are great! Sorry for the latency @jordan-brough


Logical operators in Quick Measures

Wondering if Logical Operators (AND, OR, XOR, NOT) would be part of the Quick Measures calculations?
It would be helpful to be able to process a list of input of "true/false" and "yes/no" values into a summarized format.

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

ID: 0614115b-602d-cab2-307e-618f6a3dbd2f
Version Independent ID: 09e23d6d-c1f8-ca45-3ad0-47a6ff5ec213
Content: Use Quick measures to easily perform common and powerful calculations in Power BI - Power BI
Content Source: powerbi-docs/desktop-quick-measures.md
Service: powerbi
GitHub Login: @davidiseminger
Microsoft Alias: davidi

Hi @avnish19
That's a good question.  I've forwarded it to the author who will get back to you soon.
Take care,
Michele

Adding another assignee to look into this question.

It's a good suggestion, but not available in the product today. You can submit it on ideas.powerbi.com if you want.
#please-close @mgblythe


IHttpConnectionOptions and IHubConnectionOptions aren't exposed

There is no way to new up a class that implements either IHttpConnectionOptions or IHubConnectionOptions because they aren't exported. Looks like we just need to add them to the index.ts file.

Also IConnection and IHubProtocol aren't exposed.

I am not sure if IConnection and IHubProtocol need to be exposed.

How do you write your own BetterMessagePack protocol then? Or implement SuperFastUDP connection?

For now we treat protocols as an internal thing (that's why IHubProtocol is in the Internal namespace)

In general we need to rethink our exporting strategy. At the moment it is quite inconsistent. In some cases we export all types from a file/module while in other cases we re-export. We re-export if there are types in the file/module which should not exposed publicly but we should review if this is consistent and if there is a better way of doing this.

+1, need this as well

@andreimcristof - I don't think this issue is related to headers too much. Even if you created a custom IHttpClient it won't really  - it would basically work only for the OPTIONS request. Keep in mind the neither WebSocket nor EventSource allow passing headers. I think you will need to use querystring for passing authorization header. Also, see: #888 and #686

@moozzyk thank you for the detailed explanation. cheers


Fix: strip sensitive headers on redirect to different origin

I originally sent a PR for this as #271, but it fixed a number of issues related to the redirect handling which could be considered breaking - and also added a dependency. After giving @joeybaker's review some thought, I am really eager to get a patch release out for this issue, as I don't want to leave the entire 2.x range with a security issue.
This PR attempts to address only the security issue, and does so in a way that should be backwards compatible. I still think we should improve the redirect handling overall, so I have renamed #271 to be more descriptive about this.
In other words, I want to merge this PR first, release a patch release on the 2.x range (potentially also backporting it to 1.x). Later on, I want to merge #271 and do a major release for that - but there is no rush with that one.

Hi @rexxars! Thanks for the hard work on this one.
I saw you published v1.1.1 also. Does that mean you backported this fix to support 1.x too? I'm asking because Snyk and the vulnerabilities database say to upgrade to version 2.
Thanks again :)

Hi @rexxars! Thanks for the hard work on this one. I saw you published v1.1.1 also. Does that mean you backported this fix to support 1.x too? I'm asking because Snyk and the vulnerabilities database say to upgrade to version 2.
Thanks again :)

Yes, I backported it to 1.1.1 in case people are still on 1.x and cannot upgrade for some reason. Generally we don't maintain the 1.x range any longer, but felt like patching this was worthwhile.

Yes, I backported it to 1.1.1 in case people are still on 1.x and cannot upgrade for some reason. Generally we don't maintain the 1.x range any longer, but felt like patching this was worthwhile.

Thanks for thinking about that :)
I reached out to both Snyk and the vulnerabilities database to update that 1.1.1 also contains a fix.

Hi. is there any patch file available for this change? I don't want to upgrade to 2.0.2 yet. Just want this fix. Could you please help?

Thanks @rexxars

@rexxars eventsource v1.1.1 does not fix the vulnerability. It still fails security scans.

It fixes this vulnerability (exposure of sensitive headers when redirecting to different origin), but yes, 1.x still has a different vulnerability because of the url-parse dependency.  I have opened https://github.com/EventSource/eventsource/pull/281 to address this. It will likely be the last release on the 1.x range.

v.1.1.2 is now released, dropping the original dependency and thus also the url-parse dependency.


KAFKA-6141: add disabling of useless embedded zookeeper error logs

Created a function which sets  zkShutdownHandler filed of a ZooKeeperServer object via reflection(because it field and ZooKeeperServerShutdownHandler is not accessible).
So it is a solution because useless logs is eliminated.

Thanks for the PR. It's annoying that there's no way to do this without accessing internal APIs. I suggest we fix this in ZooKeeper instead of using reflection to access internals.

@ijuma, okay, but two questions for you:

Сan we use it unless it is not fixed in ZooKeeper?
Is there are any issue for that is ZooKeeper? I d like to work on that in ZooKeeper

The is a ticket on ZooKeeper (ZOOKEEPER-2795). It looks like it will not provide you what you are looking for.
Why not simply add the following entry to the log4j.properties:
log4j.logger.org.apache.zookeeper.server.ZooKeeperServer=FATAL

ZOOKEEPER-2795 would be fine as we can filter that log message without filtering important log errors (which is why we would not want to set the log level of ZooKeeperServer to FATAL).

Thanks for raising this since it contributed to the upstream issue being issued. Since it's been fixed upstream, we can close the PR. When 3.4.12 is released, we will upgrade and get the fix.


Feature: Flag pets adopted.

If pets are flagged as adopted, they shouldn't appear on the website or apps.

This can be added at the user's discrimination since 2b58e6c0f3e15416fcc8f5fa6315c74235f57461. It would be up to the app developers to honor this flag/field.


Use of single quote instead of back quote in C11 parser is intentional ?

 I've update the C11 parser fixing that, along with some other issues I've found last months when putting the C11 AST to real use. Thanks for reporting.

Also I just made a Lua-5.4.4 , lpeglabel and lpegrex online playground here

Nice! Very useful to quickly prototype grammars! Can I add a link for it in the LPegRex readme?

Thank you !
Yes of course !

How do you debug lpegrex grammars ?
I'm looking to something like https://nmap.org/nsedoc/lib/lpeg-utility.html debug function.

Also like this https://lua-users.org/lists/lua-l/2009-10/msg00774.html

How do you debug lpegrex grammars ?

Well usually I don't need to, because I usually prototype grammars incrementally, by creating small tests to cover each grammar rule change or addition. As seen in https://github.com/edubart/nelua-lang/blob/master/spec/syntaxdefs_spec.lua . When some change does not workout I can usually reason about why.

Also like this https://lua-users.org/lists/lua-l/2009-10/msg00774.html

Something like that is probably doable with some patches in lpegrex.lua, it would be an enhancement for the library to add debug support like that.

Could you help me with a grammar I'm trying to use with lpegrex ?
I'ts a an already working grammar (by working I mean it does parse several input sources) that works with https://chrishixon.github.io/chpeg/playground/ and https://yhirose.github.io/cpp-peglib/ but  with lpegrex I'm getting:
lua gmpl-ast.lua /home/mingo/dev/lp/GLPK-dad-master-if/examples/diet.mod
nil	fail	3630
lua: gmpl-ast.lua:731: syntax error: /home/mingo/dev/lp/GLPK-dad-master-if/examples/diet.mod:100:0: fail

^
stack traceback:
	[C]: in function 'error'
	gmpl-ast.lua:731: in local 'parse_gmpl'
	gmpl-ast.lua:762: in main chunk
	[C]: in ?

The grammar seems to be compiled fine and the input text diet.mod is parsed till the end but fails.
I'm not ready to publish the grammar yet but if you could give a hand on it I could email it to you.

Instead of debugging for you, I've added experimental support for a simple debug mode, inspired by the link you have shared. Also added in the README how to use it. Let me know if that helps you debugging.
When you see fail error label, I think it's a misuse of lpeglabel, maybe checking its sources could give a hint why.

I also just updated the https://mingodad.github.io/lua-wasm-playground/ to use the latest lpegrex version.
And the lpegrex.debug mode can be used there.

I was trying to improve a bit the debug info with this:
    if lpegrex.debug then
      for k, patt in pairs(G) do
        if k ~= 1 then
          local enter = lpeg.Cmt(lpeg.P(true), function(s, p)
            local lineno, colno = lpegrex.calcline(s, p)
            io.stderr:write(string.format('ENTER %s (%d:%d)\n', k, lineno, colno))
            return true
          end)
          local succeded = lpeg.Cmt(lpeg.P(true), function(s, p)
            local lineno, colno = lpegrex.calcline(s, p)
            io.stderr:write(string.format('OK %s (%d:%d)\n', k, lineno, colno))
            return true
          end)
          local failed = lpeg.Cmt(lpeg.P(true), function(s, p)
            local lineno, colno = lpegrex.calcline(s, p)
            io.stderr:write(string.format('FAILED %s (%d:%d)\n', k, lineno, colno))
            return false
          end)
          G[k] = enter * (patt + failed) * succeded
        end
      end
    end

But I'm getting this error:
lua gmpl-ast.lua /home/mingo/dev/lp/GLPK-dad-master-if/examples/diet.mod
>lua gmpl-ast.lua /home/mingo/dev/lp/GLPK-dad-master-if/examples/diet.mod
lua: gmpl-ast.lua:765: rule 'Primary_expression' may be left recursive
stack traceback:
	[C]: in function 'error'
	gmpl-ast.lua:521: in field 'compile'
	gmpl-ast.lua:732: in function 'parserGmpl'
	gmpl-ast.lua:765: in main chunk
	[C]: in ?

I understand the consequence of left recursion but I can't see why it's been detected here.
Any idea of a workaround ?

With the suggestion of @roberto-ieru I extended a bit the debug output and it can be seen on https://mingodad.github.io/lua-wasm-playground/ with the Json parser** where I inlined lpegrex.lua to test/show it.

And here is the output replacing ENTER by -->, FAILED by <-- and OK by =:
--> Json (1:1)
--> SKIP (1:1)
= SKIP (1:1)
--> Object (1:1)
--> { (1:1)
<-- { (1:1)
<-- Object (1:1)
--> Array (1:1)
--> [ (1:1)
--> SKIP (1:2)
= SKIP (1:2)
= [ (1:2)
--> Value (1:2)
--> String (1:2)
<-- String (1:2)
--> Number (1:2)
<-- Number (1:2)
--> Object (1:2)
--> { (1:2)
--> SKIP (2:0)
= SKIP (2:5)
= { (2:5)
--> Member (2:5)
--> String (2:5)
--> SKIP (2:13)
= SKIP (2:13)
= String (2:13)
--> : (2:13)
--> SKIP (2:14)
= SKIP (2:15)
= : (2:15)
--> Value (2:15)
--> String (2:15)
--> ESCAPE (2:21)
= ESCAPE (2:22)
--> SKIP (2:27)
= SKIP (2:27)
= String (2:27)
= Value (2:27)
= Member (2:27)
--> , (2:27)
--> SKIP (3:0)
= SKIP (3:5)
= , (3:5)
--> Member (3:5)
--> String (3:5)
--> SKIP (3:14)
= SKIP (3:14)
= String (3:14)
--> : (3:14)
--> SKIP (3:15)
= SKIP (3:16)
= : (3:16)
--> Value (3:16)
--> String (3:16)
<-- String (3:16)
--> Number (3:16)
<-- Number (3:16)
--> Object (3:16)
--> { (3:16)
<-- { (3:16)
<-- Object (3:16)
--> Array (3:16)
--> [ (3:16)
<-- [ (3:16)
<-- Array (3:16)
--> Boolean (3:16)
--> NAME_SUFFIX (3:20)
<-- NAME_SUFFIX (3:20)
--> SKIP (3:20)
= SKIP (3:20)
= Boolean (3:20)
= Value (3:20)
= Member (3:20)
--> , (3:20)
--> SKIP (4:0)
= SKIP (4:5)
= , (4:5)
--> Member (4:5)
--> String (4:5)
--> SKIP (4:13)
= SKIP (4:13)
= String (4:13)
--> : (4:13)
--> SKIP (4:14)
= SKIP (4:15)
= : (4:15)
--> Value (4:15)
--> String (4:15)
<-- String (4:15)
--> Number (4:15)
--> SKIP (4:22)
= SKIP (4:22)
= Number (4:22)
= Value (4:22)
= Member (4:22)
--> , (4:22)
--> SKIP (5:0)
= SKIP (5:5)
= , (5:5)
--> Member (5:5)
--> String (5:5)
--> SKIP (5:11)
= SKIP (5:11)
= String (5:11)
--> : (5:11)
--> SKIP (5:12)
= SKIP (5:13)
= : (5:13)
--> Value (5:13)
--> String (5:13)
<-- String (5:13)
--> Number (5:13)
<-- Number (5:13)
--> Object (5:13)
--> { (5:13)
<-- { (5:13)
<-- Object (5:13)
--> Array (5:13)
--> [ (5:13)
<-- [ (5:13)
<-- Array (5:13)
--> Boolean (5:13)
<-- Boolean (5:13)
--> Null (5:13)
--> NAME_SUFFIX (6:0)
<-- NAME_SUFFIX (6:0)
--> SKIP (6:0)
= SKIP (6:1)
= Null (6:1)
= Value (6:1)
= Member (6:1)
--> , (6:1)
<-- , (6:1)
--> } (6:1)
--> SKIP (6:2)
= SKIP (6:2)
= } (6:2)
= Object (6:2)
= Value (6:2)
--> , (6:2)
<-- , (6:2)
--> ] (6:2)
--> SKIP (7:0)
= SKIP (7:0)
= ] (7:0)
= Array (7:0)
= Json (7:0)
= Rule counters:
	Total	Succed	%	Failed	%	Rule
	74	52	70.27	22	29.73	all rules
	20	20	100.00	0	0.00	SKIP
	9	5	55.56	4	44.44	String
	5	5	100.00	0	0.00	Value
	5	3	60.00	2	40.00	,
	4	4	100.00	0	0.00	Member
	4	1	25.00	3	75.00	Object
	4	1	25.00	3	75.00	Number
	4	4	100.00	0	0.00	:
	4	1	25.00	3	75.00	{
	3	1	33.33	2	66.67	[
	3	1	33.33	2	66.67	Array
	2	0	0.00	2	100.00	NAME_SUFFIX
	2	1	50.00	1	50.00	Boolean
	1	1	100.00	0	0.00	Null
	1	1	100.00	0	0.00	]
	1	1	100.00	0	0.00	}
	1	1	100.00	0	0.00	Json
	1	1	100.00	0	0.00	ESCAPE
Array
| Object
| | Member
| | | "string"
| | | "some\ntext"
| | Member
| | | "boolean"
| | | true
| | Member
| | | "number"
| | | -150.0
| | Member
| | | "null"


Interactive Window: Always scroll on new cell doesn't work with jupyter notebook cells started by shift+enter

This is similar to #4475 and #11127. Looks like Interactive Window: Always scroll on new cell doesn't apply to cells started by shift+enter
Environment data

Vs Code version:

Version: 1.78.2
Commit: b3e4e68a0bc097f0ae7907b217c1119af9e03435
Date: 2023-05-10T14:44:45.204Z
Electron: 22.5.2
Chromium: 108.0.5359.215
Node.js: 16.17.1
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 20.6.0
Sandboxed: Yes

Jupyter Extension version (available under the Extensions sidebar): v2023.4.1011241018
Python Extension version (available under the Extensions sidebar): v2023.8.0
Python and/or Anaconda version: 3.8.16
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): N/A
Jupyter server running: Local

Expected behaviour
Both pressing "running" button or using "control+enter" gives the expected behavior as shown:

Actual behaviour
But pressing "shift+enter" gives the incorrect behavior:

Steps to reproduce:

See above.

the notebook view is different than the interactive window, so that setting won't change anything for what you are attempting to do.
It sounds like you are looking for the configurability requested in this issue
Let me know if that's not the case

the notebook view is different than the interactive window, so that setting won't change anything for what you are attempting to do.
It sounds like you are looking for the configurability requested in this issue
Let me know if that's not the case

I simply can't understand why control+enter works but shift+enter not while they are almost doing the same thing: running a cell.

They execute different commands - you could update your keyboard shortcuts to make only execute the command that you like though.

They execute different commands - you could update your keyboard shortcuts to make only execute the command that you like though.

Oh, I got it. This perfectly solves my issue, thanks!

I'd like to vote to re-open this bug.
The default behavior of Jupyter Lab in the browser when Shift-Enter is pressed is to both execute the current cell and jump to the next cell, without autoscrolling the window
The fact that VS Code scrolls the window with this behavior means that the cell that just executed is constantly getting cut off, requiring manual scrolling up.
This behavior, execute then jump to next, is probably one of the most common things a data scientist does while working with a Jupyter Notebook.
Having to use Ctrl-Enter to execute just the current cell, then manually click to jump to the next cell, is maddening.
These kinds of UX quality of life things are very important for people in their code editors. I just made the jump to VS Code from Jupyter Lab in the browser and its driving me crazy.
My proposal: allow Shift-Enter to execute the current cell and select the next cell, without also autoscrolling the VS Code window. This means Shift-Enter should respect "interactiveWindow.alwaysScrollOnNewCell".
Moving to Cntrl-Enter is not a suitable replacement.

The interactiveWindow.alwaysScrollOnNewCell setting is for the interactive window and does not effect notebooks.
I agree that the current scroll on execute behavior is off, and that should get fixed as part of the other issue you commented on..

I guess this one is still not fixed? It is not a big problem if the the cell execution result is comparatively small. However, whenever cell output is taking half or more of the screen, it becomes a real headache. Would love to see it fixed, but don't know if I can contribute in any useful way.


[nginx] Support for server-side events

Right now server-side events are impossible to support since setting the appropriate values in an ingress.kubernetes.io/configuration-snippet block results in duplicate directives, and there is no option on the controller to explicitly support them on a single ingress resource.
See https://github.com/drone/docs/issues/230 for a configuration example.

@rochdev please provide a test case. This should work ootb


Add Review Commons reviews

Import Review Commons reviews from Hypothesis.

Is there any duplication with the eLife editorial community (reviews that appear on both sides)? According to the Hypothesis group all the 111 annotations have been created by the EMBOpress account so no overlap.

I guess we are missing a name, description, logo, etc for the new editorial community page? And is it Review Commons or EMBOpress?

However there is also a Reply to the reviewers. Should that be included somehow?

Permalink to one review: https://hypothes.is/a/-8oO9LDGEeqx599D5Swz_Q This should probably be where the Read more links points to?

The markdown from the text property should be converted to HTML and truncated for the summary shown in our page?

The date from the created property will be shown, which is the date of the annotation creation on Hypothes.is, as we don't have anything more specific.

Should any of the articles be endorsed? Probably not.

At the moment we add ReviewReferences manually, what is the relevant set to add here? All of them up to the point in which we do this work? This is about 111 reviews.

Notes
Loaded with curl https://api.hypothes.is/api/annotations/-8oO9LDGEeqx599D5Swz_Q | jq .
Example of reviewed article:
https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>971v1
Has 2 reviews from EMBOpress, which appears to be in the Review Commons Hypothesis group

No contention
Review Commons (eLife will not be part of this group)
Skip this (somehow), and push the decision of what to do with it later on outside of this ticket.
Yes that's the link.
Yes, convert to HTML and truncate
Yes, use created.
No. Let's worry about that later. It may even coming from another journal, and they would endorse another version of the article (that they accepted) rather than the v1 of the preprint.
First 10 as they appear on the Hypothesis group as of 2020-06-24.

Bit hidden, but https://web.hypothes.is/help/formatting-annotations-with-markdown/ suggests that they use GitHub-flavoured Markdown.


About metrics reporter

After integrating metrics-aspectj, but there is no reporter.
If not use metrics annotation, it can be used by
ConsoleReporter.forRegistry(metricRegistry).build().start(15, TimeUnit.SECONDS)
can i get the metricRegistry from metrics-aspectj?
please offer a proposal, thx

I understand.. write like this:
    public AnnotationService(MetricRegistry registry) {
        this.registry = registry;
        ConsoleReporter.forRegistry(this.registry).build().start(10, TimeUnit.SECONDS);
    }


fix: OS integrations for quick file import

This PR refactors code to fix bugs caused by js api changes.
The sneaky one: the way we pass template changed in a way that failed silently:
https://github.com/ipfs/js-ipfs/blob/8cb8c73037e44894d756b70f344b3282463206f9/packages/interface-ipfs-core/src/object/new.js#L33-L40
so instead of unixfs, we were patching untyped object, which gateway was unable to render.
@rafaelramalho19
I've pushed 1 line fix to your branch and created this PR to include our fixes.
How to test
Same code is used for different variant on each OS, so we need to test both.
macOS

[ ] drag&drop multiple files or a file and a directory to desktop's icon

Windows

[x] right click on a file and select "Add to IPFS" from context menu

Closes #1684

Looks great! Thanks for picking up from my first bug fix 😄
🚢  itttttttt


cannot find matplotlib

It always report cannot find matplotlib in the box for function visualize_images.
I have installed matplotlib in host, in launcher, I even add installation script in ipynb files, but still cannot solve the problem.

I have the same error during the step of simple grid visualizer

I install in host outside of launcher, no error and it s working


feat: add into_cow_str for LazyValue

To make it cheap for converting LazyValue to str without borrowing iterator, here I add a into_cow_str method for it.

 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.


fixing broken link

Change Summary
The link to the debugger is broken.
Merge Checklist

[ ] PR has a Changeset
[ ] PR includes documentation if necessary
[ ] PR updates the boilerplates if necessary

thank you!


Link prominently to book in readme

I think the book is easy to miss, so add an additional link to it in the readme.

Done!


[Modified] UI improvements - round 1

Continuation from #137 changes

@Jay-o-Way Honestly, I am sorry, this is very harder than I imagined because there are too many changes.
As I am also worried that this would lead to wasted efforts from your end, I will still try to figure out ways I can scavenge the few changes I want from this PR but for the last time and further from now on, I have to enforce that I won't merge/review any PR that is not focused on a particular change.

Introduced some changes (#140) from here, related to Controls.BrowserList (now, its BrowserButton)


Chat - User lands in duplicated concierge chat if login via concierge link

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open https://staging.new.expensify.com/concierge
Perform a sign in
Navigate to a different chat from concierge
Return back to previous concierge chat
Send a link https://staging.new.expensify.com/concierge
to any chat apart from concierge
Click on the sent link

Expected Result:
At step 2 user lands in completely functional concierge chat. Only 1 chat is displayed in LHN.
At step 6 user lands in primary concierge chat (that is displayed at the top of LHN).
Actual Result:
At step 2 user lands in non-functional duplicated concierge chat with an error
At step 4 functional chat is displayed.
At step 6 user lands in duplicated concierge chat.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/3290526&group_by=cases:section_id&group_id=229066&group_order=asc
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/93399543/228046883-ff48ad0e-7d14-4792-8fb7-0ec5c3a38e4b.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

Triggered auto assignment to @arielgreen (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Bug0 Triage Checklist (Main S/O)

[ ] This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
[ ] This bug is not a duplicate report (check E/App issues and #expensify-bugs)

If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead

[ ] This bug is reproducible using the reproduction steps in the OP. S/O

If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
If the reproduction steps aren't clear and you determine the correct steps, please update the OP.

[ ] This issue is filled out as thoroughly and clearly as possible

Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.

[ ] I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

Cannot reproduce.
https://user-images.githubusercontent.com/37308300/228282469-5c93f000-9c3a-44d5-87a2-9ec89514c947.mp4

@arielgreen Can you please try again using this TR steps

Issue is reproductible for QA team.

@kbecciv I still cannot reproduce.


IP Forwarding setup

Hi guys,
I noticed there's an enable_ip_forwarding recipe which is not yet documented in the README.md which seems to help me achieve the same (?) thing as using the server.up script does?
In particular I see this recipe sets the net.ipv4.conf.all.forwarding and net.ipv6.conf.all.forwarding using the sysctl_param resource. The up script, if used, sets /sbin/sysctl -w net.ipv4.ip_forward=1, so I am somewhat unsure as to which method is the preferred one to use here.. ?
If someone could help explain this, I'd be happy to tidy it up in a PR that helps document this.
Thanks!

The server recipe includes the enable_ip_forwarding recipe to ensure IP forwarding is enabled but also so it's managed on a chef resource level in terms of persistence (too). The server.up script ensures its turned on when the script is run on the system level (there is somewhat a portability case here where this is not dependent on the chef cookbook but more a generic choice). One shouldn't rely on the other and there is no harm in doing it in both places.
As its just a trivial recipe included by a main recipe, there may not be any need to document in the readme, but no probablem if you'd like to submit a PR for the it on any of the above. Hope this kind of helps :)


Addon configuration loading

Just some docs for now, code will come soon.

@tiberiuichim first shot on it, let me know what do you think!!

@tiberiuichim I have to test it yet on production, it would be great if you could do that as well.

This branch still needs a lot of work. The current problem is related to webpack's dynamic imports. See https://webpack.js.org/api/module-methods/#dynamic-expressions-in-import
I don't think it's worth pursuing dynamic imports, even if we make it work, the generated bundles would be sub-optimal (we'll no longer benefit from tree-shaking).
I'm working on an approach where we generate a js file that has the addon loading code. This ensures that webpack gets to "see" the package that we want to include in the bundle and keep its static import mechanism.

The PR is ready to be merged. @tisto @robgietema

@tiberiuichim awesome!!! Will try to find some time during the next days to look into it...

@tiberiuichim just wondering: do you use this in production?

@tisto no, not yet. There's a complete Volto project that you can clone and test it:
https://github.com/eea/volto-slate-project
git clone https://github.com/eea/volto-slate-project
cd volto-slate-project
yarn install
yarn start

I can't promise I can put this into "staging" pretty soon, as we're quite busy developing features.

@tiberiuichim can you please rebase? @robgietema approved the PR, so I will just have another look and then merge and release that thing!

@giuliaghisini @pnicolli @nzambello @cekk if possible I'd like to have a review from you folks as well. Just to make sure you are ok with that. That's a big and important decision. :)

The builds have passed, I'm not sure why their status is yellow.

@tiberiuichim I guess Travis hiccups. I will do another release, then merge this one tomorrow and do another release.

@tiberiuichim thank you Tiberiu. You rock! That's a big step forward for Volto!!!

@tisto thanks!
Volto's future is very exciting. I'm happy to be able to work on that.

@tiberiuichim which PR is next? :)


Using against a private repo

Is your feature request related to a problem? Please describe.
When running this against a private repo what permissions does my person token need ?
It also seems that not all the py scripts take the option of the org e.g ./list_sprintplanning_columns.py
Describe the solution you'd like
To be able to run against a private repo as this tool is very useful
Describe alternatives you've considered
We had to cut and paste this morning.. Worked fine but not very Dev friendly :-)
Additional context
Perhaps an uber command line to bring all the parts together would be nice in time..
Maybe the functionality already exists ? I just could not find it before the cut/paste beat me to it ?

list_projects.py was hard coded to atsign_foundation, but I've modified it to take the org as an argument.
We now also have:

list_company.py which is hard coded to atsign-company
list-foundation.py which is hard coded to atsign-foundation

So to get the csv for the marketing board PR13 column:
./list_company.py
5640050 1 @ Production Infrastructure Build out and Go LIve!
5640153 2 @protocol features roadmap
5640288 3 @persona features roadmap
5640319 4 @demos
5640333 5 @libraries
6016491 6 Sprint Planning - Jagan and Murali
6065970 7 Master Bug Project Board
11705603 8 Atmosphere Board
11705635 9 Web Registrar Board
12388324 11 Wavi Rebranding and v2
12478765 12 App Issues
12553473 14 FHIR @hack : July 9-11 2021
12587318 15 Marketing and GTM

./list_project_columns.py 12587318 atsign-company
14564544 Events | Comms
14566749 Social Media | Community
14564545 Apps | Content
14566719 PR13 18 SP's | 29"s

./dump_project_cards.py 14566719
cat 14566719.csv

Issue Key,Summary,Description,Acceptance Criteria,Story Points
4,Company Messaging and Strategy,,,
5,Enhanced Content / SEO Strategy,,,
7,Educational Content,,,
2,Update Nomanclature,,,
13,Social Media Tool Research,,,
10,Metrics/Analytics,,,
8,@buzz,,,
11,PR,,,
15,Target Training Organizations,,,
14,Early Access Program,,,


Andrew testing

Before issuing a pull request, please see the contributing page.

It is not working so I have to close it


[Wait for #1499] [CAPI] Expose l2norm layer

[Pending commit: #1499]

[CAPI] Expose l2norm layer

**Changes proposed in this PR:**
- Change name of l2norm layer to PreprocessL2Norm Layer
- Move l2norm layer to the main code base

**Self evaluation:**
1. Build test: [X]Passed [ ]Failed [ ]Skipped
2. Run test: [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: Jihoon Lee<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #1500. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnstreamer.mooo.com/.

:octocat: cibot: @zhoonit, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://nnstreamer.mooo.com/nntrainer/ci/repo-workers/pr-checker/1500-202108181819320.44649291038513-94dc6fd0779653991b9af9eb22ebc416083b30a9/.


Prefixed credentials

As originally discussed here
Ability to use multiple credentials in a single application easily.
I can add examples to the README file, but wanted to get feedback on the
implementation itself first.

Yesterday I have worked on the 6.0.0 branch and force pushed it. So it might be a good idea to rebase on that. If you don't know how that works it's fine. As soon as I am able to spend more time I will integrate this change.

I've pulled down your force-pushed 6.0.0 branch. Saved my old branch to a different name, branched off of the new 6.0.0 branch, and cherry picked the commits with the prefix credentials feature. Then force pushed my own prefixed-credentials branch so that it shares the common base with your new 6.0.0.

Awesome.

I have intergrated your result in the 6.0.0 version


added my data

Please fix the merge conflicts and update the PR @yash-seth

I have fixed the merge conflicts @Rajdip019

Good job @yash-seth
Congrats 🎉, your PR got merged. To be eligible for the swags from ClueLess. Follow this steps.

SignIn / SignUp in ClueLess Website .

Go to profile section from the top right hand corner.

Comment your profile URL under this comment tagging the person who merged your PR.

That’s it. Follow the leaderboard !
And don’t forget to share your achievement with your friends.


BBT Does Not Load: "Better BibTeX failed to load: <Error: update: validation failed…"

[x] I have verified in the Zotero addons screen that I am on the latest release of BBT (https://github.com/retorquere/zotero-better-bibtex/releases/latest) and Zotero
[x] I am posting a single question, bug or feature request
[x] I have included a descriptive subject of the problem
[x] I am available for follow-up questions and testing

Report ID: PJUECUD8
I'm having a problem with Better BibTeX not starting. The Initializing message says "Loading citation keys" and never vanishes. It started occurring after I upgraded to the newest version of the extension (for some reason I was using a year-old version previously, as Zotero has not been automatically updating extensions).
In looking through the report I submitted, I notice the error I copied+pasted below. It's particularly weird because I have deleted that citation from my database (and emptied the trash) but the error is still occurring.
Clicking on reset cache and re-scan citation keys in the preferences does not seem to do anything, either.
I have removed and reinstalled the extension multiple times, which has not helped.
Report Error
(3)(+0000016): {better-bibtex} update: validation failed for {"itemID":2096,"libraryID":1,"pinned":false,"citekey":"BernalWorldFleshDevil2018","extra":"","meta":{"revision":0,"created":1527102495788,"version":0},"$loki":1067} [{"keyword":"additionalProperties","dataPath":"","schemaPath":"#/additionalProperties","params":{"additionalProperty":"extra"},"message":"should NOT have additional properties"}] <Error: update: validation failed for {"itemID":2096,"libraryID":1,"pinned":false,"citekey":"BernalWorldFleshDevil2018","extra":"","meta":{"revision":0,"created":1527102495788,"version":0},"$loki":1067} ([{"keyword":"additionalProperties","dataPath":"","schemaPath":"#/additionalProperties","params":{"additionalProperty":"extra"},"message":"should NOT have additional properties"}]) @chrome://zotero-better-bibtex/content/common.js:8121:21 exports.DB.init@chrome://zotero-better-bibtex/content/common.js:5542:13 >

(3)(+0000000): {better-bibtex} Better BibTeX failed to load: <Error: update: validation failed for {"itemID":2096,"libraryID":1,"pinned":false,"citekey":"BernalWorldFleshDevil2018","extra":"","meta":{"revision":0,"created":1527102495788,"version":0},"$loki":1067} ([{"keyword":"additionalProperties","dataPath":"","schemaPath":"#/additionalProperties","params":{"additionalProperty":"extra"},"message":"should NOT have additional properties"}]) @chrome://zotero-better-bibtex/content/common.js:8121:21 exports.DB.init@chrome://zotero-better-bibtex/content/common.js:5542:13 >

A year ago that's probably still a beta; the beta has stuff in the DB that transitional versions cleaned up, but that cleanup is now no longer present.
I can either clean up the better-bibtex.sqlite database manually if you get me a copy (not hard), or you can remove it altogether (all keys will be regenerated and you will lose auto-export). Either of these will let BBT start up normally.

The early beta's didn't update automatically, so it's very likely you're on a pre-release.

Deleting the DB worked! Thanks so much!


Clarify why X-Registry-Auth header can't be empty

Move comment to more noticeable place.
Make DefaultDockerClient.authRegistryHeader() return null
if auth is empty.
If DefaultDockerClient.authHeader() returns null,
PushPullIT's testPushImageToPrivateAuthedRegistryWithoutAuth()
and testPushImageToPrivateUnauthedRegistryWithoutAuth() would fail.
resolves #763

Codecov Report

Merging #815 into master will not change coverage.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master     #815   +/-   ##
=========================================
  Coverage     66.22%   66.22%
  Complexity      730      730
=========================================
  Files           165      165
  Lines          3103     3103
  Branches        353      353
=========================================
  Hits           2055     2055
  Misses          893      893
  Partials        155      155

I couldn't find any docs on this. Ryan left the original comment.


add PodMigrationJob CRD proposal

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

This proposal defines a CRD-based Pod migration API, through which the descheduler or other automatic fault recovery components can evict or delete Pods more safely. At the same time, the proposal also describes the specific implementation details of the API.
Ⅱ. Does this pull request fix one issue?
#357

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

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

/lgtm
/approve


fix: totalSize should be paddingStart + paddingEnd when size is 0

This needs to be fixed up to remove the breaking changes that were rolled back. @piecyk, I'll leave that to you, or we can close this issue and fix it in next. You're call.

@tannerlinsley has been rebased onto master

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Forms are not responsive

The register donor form and register organisation forms are not responsive for small screen devices
Screencast from 2023-06-26 23-35-43.webm

@manuel12 Thanks, it's now yours


newbie

hey Rachid
can i ask for the tutorial for this installation?
https://github.com/rin1015/animecenter

There's already a tutorial for that here: https://github.com/rin1015/animecenter#setup

Is it compatible with version 5.3 of the framework? Cheers!

Haven't tested it yet. But ti should work.


tetris -> interactive

@ProbablePrime, thanks for your PR! By analyzing the blame information on this pull request, we identified @SimonSchick and @UnwrittenFun to be potential reviewers

lgtm

Keep in mind to release this as a major.

lgtm

According to Art & Unwritte, type definitions are not breaking.


Problems when updating to 1.9.0

Restarting after an 1.9.0 update gives a warning for "Invalid config". The log is attached. Am I doing something wrong?
home-assistant_2024-11-13T15-45-38.743Z.log

Can you see what version of gql you got in your Home Assistant environment?

Looks like a corrupt file somewhere.

How can I find the version of gql?
My system is fully updated, so I assume I have the latest available version.

How can I find the version of gql? My system is fully updated, so I assume I have the latest available version.

You should have the latest version, but for some reason the downloaded files are corrupt. Are you running in a container?

I'm running a virtual machine, VirtualBox. Would you have an idea on how to replace the corrupted files?

Are you running HassOS, a container or plain Python inside your VM? For both HassOS and container you should be able to reinstall keeping your configuration and data. Plain Python needs some more work.

Just got the problem resolved after updating the core to 11.2.


ARC-2922 update copy button logic

What's in this PR?
Update the button appearance logic.

The user can only clicked all the required buttons to continue to next step.
All button states get reset when user switch to another tab, either 'i am a jenkin' or 'a jenkin admin in my team'

Why
Jenkins design update 1.1
Affected issues
ARC-2922
How has this been tested?
yarn test & manual UI testing

PR review pause as I just talked with Sarah. Some design needs to be updated


Issue uploading file greater than 50GB

rclone v1.32-18-gf7562dcβ
Ubuntu Server 16.04.1
ACD
rclone sync -v --stats=10s --low-level-retries=1 --acd-upload-wait-time=2m0s --max-size=70G --log-file=/home/... --transfers=2 /tmp/.PlexMedia/" "MediaServerBackup:/PMS/
Log
Now before we continue, I realize the stated max file size has been 50GB. However, that was established over 9 months ago, and I'm wondering if there has been a recent change in this.
I only ask because I stumbled upon this post from an Amazon rep that stats there is no limit on max upload size via the API. This post is nearly a year old, but I found it intriguing that they would say this.
My upload of a roughly 54GB file fails, but only during the time-out period. Question I have is this: is rclone limiting my file size and then deleting it at the end of the failed upload, when it perhaps uploaded correctly, or is this still being limited by Amazon?

No, rclone doesn't limit your file upload size.  It is possible Amazon have raised the limit - let me know if you succeed with files > 50GB.

Wasn't able to succeed with a filer larger than 50GB. Don't know if Amazon is actually failing tho, because it looks similar to other large files in that it starts through the --acd-upload-wait-time processing and then just ends with no issues being stated regarding size save for the rclone warning.
I've attached the log. The entire log is for just one file; apparently it kept trying to upload it, even after failing.
Any idea? or is this Amazon's limitation it's now hitting. The log doesn't seem to provide the most detail here.

Checking the log out, each time where it managed to upload 100% of the file, it gave a "500 Internal Server Error" which I think is Amazon's way of saying it isn't supported :-(

I have a feeling you're right. Do you know the best way to get an answer regarding this? I've tried to reach out to the API support via email, but they never responded. Just looking to get a firm answer on where they currently are sitting at.

Amazon have been strangely coy about publishing file size limits.  50GB is the limit most talked about though.  Try asking customer support rather than API support.


CycleGAN Error: One of the variables needed for gradient computation has been modified by an inplace operation

RuntimeError                              Traceback (most recent call last)
 in ()
1 n_epochs = 200 # keep this small when testing if a model first works, then increase it to >=1000
2
----> 3 losses = training_loop(dataloader_X, dataloader_Y, test_dataloader_X, test_dataloader_Y, n_epochs=n_epochs)
2 frames
/usr/local/lib/python3.6/dist-packages/torch/autograd/init.py in backward(tensors, grad_tensors, retain_graph, create_graph, grad_variables)
130     Variable.execution_engine.run_backward(
131         tensors, grad_tensors, retain_graph, create_graph,
--> 132         allow_unreachable=True)  # allow_unreachable flag
133
134
RuntimeError: one of the variables needed for gradient computation has been modified by an inplace operation: [torch.cuda.FloatTensor [1, 512, 4, 4]] is at version 4; expected version 3 instead. Hint: the backtrace further above shows the operation that failed to compute its gradient. The variable in question was changed in there or anywhere later. Good luck!

Though we are a little late to respond, we suggest you reach out to Udacity support for user-specific issues.
Thanks


Switching framegraph crash (Fallback Layer)

Describe the bug
Wisp crashes when running Wisp on a machine that doesn't support DXR (so only supports the Fallback Layer) and switching framegraph to any framegraph that is using an Acceleration Structure.
To Reproduce
Steps to reproduce the behavior:

Run Wisp
Press F2 to switch framegraph
Experience the crash

Expected behavior
Just switch framegraph. No crash should be involved.
Screenshots

Deepest error location

Deepest 'high level' error location

Callstack
System (please complete the following information):

OS: Windows 10
NVIDIA GeForce GTX 1050
RAM 16GB

I think this is ray-tracing/fallback crashing, not the framegraph switching.

just tested it, when using hybrid as the startup framegraph it crashes as well, so it's an issue in ray-tracing/fallback, not the framegraph.


added currentAccumulator support

closes #51
Signed-off-by: blu3beri<EMAIL_ADDRESS>

CI is failing 🙃

Yeah it's almost finished :D


Loss of gender field on csv entry edit

The gender field setting is being lost on en edit

@johnt-freecen @geoffj-FUG Fixed in test3

Hi Kirk,
Thanks for the update, the system is displaying the value but can you please check what is there. The Gender field now displays four values M m F f-. I am not certain what the last character is.
Regards
John
Sent from Mail for Windows 10
From: Kirk Dawson
Sent: 25 September 2020 15:37
To: FreeUKGen/FreeCENMigration
Cc: johnt-freecen; Mention
Subject: Re: [FreeUKGen/FreeCENMigration] Loss of gender field on csv entryedit (#982)
@johnt-freecen @geoffj-FUG Fixed in test3
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

@johnt-freecen clarified selection box entries. TY

Kirk
Now working OK
This item can be closed
Geoff


Prevent double payments for expenses

Fix https://github.com/opencollective/opencollective/issues/3502 and removes the duplicates.
Note: the way expense locking has been implemented is currently being revamped & not ready for review

Closing in favor of https://github.com/opencollective/opencollective-api/pull/4605


[Question] - Focus on a textfieldWithFloatingLabel

Is there a way to set the focus to a textfieldWithFloatingLabel? Similar to React Native TextInput with a ref and then using this.refs.InputName.focus().

Thank you for reporting, I'll look into it.


reduce resize duration for each frame

is it possible to reduce resize duration by using goroutines for processing each frame? but it will affect cpu/memory usage

that's a great idea... yeah, that should be totally possible.  We'd need to make sure that the frames get reassembled in the right order, but that shouldn't be too difficult.  I probably won't have time to try this myself for a little while, but if you want to give it a shot, I'd be happy to assign the issue to you.  Otherwise, we can just leave it open for now.

Looking at this more closely, I don't think we can process frames in parallel.  The way gif disposal methods work, each frame potentially needs access to the previous processed frame, so they have to be handled sequentially.
I'm going to go ahead and close this, since I don't think it's possible, but please feel free to reopen if an alternative comes to mind.


Clarify usage

I struggled a bit to figure out how to actually get a hold of my queued message. Maybe clarify in docs (not sure if what I got is the best way...) what to do with a Memory<byte> object? Maybe my lack of c# knowledge but I think a clarification might be useful?!
so I 'd add a message as usual:
 await batchQueue.AddMessageAsync(new CloudQueueMessage(JsonConvert.SerializeObject(myDTO)));

and then consume it in my trigger
public static async Task RunAsync([QueueTrigger("batch-queue")]MyDTO myDTO, ILogger log)
{
        // use myDTO here ...
}

using BatchQueueTrigger
public static async System.Task RunAsync([QueueBatchTrigger("batch-queue")]IMessageBatch batch,  ILogger log)
{
        foreach (var msg in batch.Messages)
        {
                MyDTO dto = JsonConvert.DeserializeObject<MyDTO>(System.Text.Encoding.Default.GetString(msg.Payload.ToArray()))
        }
        batch.MarkAllAsProcessed();
}

is there a better way?

The IMessageBatch provides only bytes by design as QueueBatch is meant to be used with deserialization that should be able to work with data in-situ, without need of allocating or trying to get a string out of it. Preferable way to use it is to use it with some binary serialization/deserialization.
In other words, this is not a generic batch for Azure Storage Queues but rather specific, for formats that can deal with binary data. I hope this clarifies the purpose of this project @EricAndreas. If yes, feel free to close the issue.

That's great, I was not aware of that purpose (maybe state that and provide a usage example? I know it would have helped me...) I found this project when looking for batching options (rather than scale out) with queue triggers and this seemed like a perfect fit - if it also has better performance it's even better!  Would you say QueueBatch.Tests/IntegrationTests.cs shows "formats that can deal with binary data"? I got usage from that but not quite sure what (other?) usage this project aims for (any samples)?
But (more importantly), great work!

If your serialization supports reading in zero-copy mode, or something similar, you could allocate an instance once, and use it for all the cases. Or if every message in a batch represents an array, you could read it one after another. Thinking about it as loop (batch) over loop (array) of items. These would be scenarios.
The project was tested with the last approach (batches of arrays of items) but this case is unfortunately not public 😢.
If you agree, let's keep this issue open @ErikAndreas. I don't promise to address this within a week, but it'll be at least visible.


Reconcile the minikube install page with best practices

Update the minikube installation document to be in-line with the guidance we provide users at https://minikube.sigs.k8s.io/docs/start/
Preview URL: https://deploy-preview-20717--kubernetes-io-master-staging.netlify.app/docs/tasks/tools/install-minikube/
Specific changes:

fix capitalization of "minikube"
remove obsolete guidance:

virtualization is no longer required by minikube
neither is kubectl (though handy)
Linux packages are no longer experimental

Remove the non-installation steps, as they are covered by the "next steps" document (which will be further improved in a subsequent PR)

Related issues:

https://github.com/kubernetes/website/issues/15969
https://github.com/kubernetes/minikube/issues/5686

/cc @zacharysarah  @afbjorklund @prasadkatti  @medyagh @priyawadhwa @sharifelgamal

/sig minikube

/assign @sftim

@tstromberg, Changes look okay.
I am wondering if you need to say more about what happens when you run,
minikube start.
What is the reason for removing the cleanup instructions?

@tstromberg, Changes look okay.
I am wondering if you need to say more about what happens when you run,
minikube start.

Do you have any suggestions? From my vantage point it seems self explanatory given the surrounding context, both from the documentation and the command output.

What is the reason for removing the cleanup instructions?

This is just an installation doc, the guide for managing your cluster should be captured by the  "whatsnext" link (which I also intend to update once this doc is approved)

@tstromberg ,
What you have is okay, just thought some text would help tie in the start command and the
driver text. Here are some ideas:
## Starting your first cluster
minikube will automatically select an appropriate driver, but will mention alternatives if available.
To start your cluster, run:
minikube start

To stop your cluster, run:
minikube stop

For additional information or assistance about minikube, reach out on the
minikube Slack channel #minikube.
## What's next

* [Run Kubernetes locally with minikube](...)
* Read the [minikube documentation][minikube.sigs.k8s.io/docs)

@tstromberg . Is this pull request ready? If so, please squash your commits. Thanks!

Okay to close? @tstromberg

@tstromberg if you want to update that page and open a new PR from your branch, go ahead (and please mention that the new PR is a continuation of #20717).
You'll need to rebase your branch against master BTW.

/close


Full document support?

What do you mean by this? Can you maybe describe what you want to do and how you think diesel should help you with this?

Document in diesel.rs is so simple and not fully, it's just a guide.

Oh, you mean the documentation. Yeah, the guide doesn't describe everything.
You have seen the API documentation, right? It can be confusing in the
beginning, but contains a lot of detailed information on most of diesel's
features.
What are you missing the most? What are you trying to do that is not
documented? The plan is to add more guides for specific use cases of diesel
in the future.
sackery<EMAIL_ADDRESS>schrieb am So. 18. Dez. 2016 um 06:01:

Document in diesel.rs is so simple and not fully, it's just a guide.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/diesel-rs/diesel/issues/545#issuecomment-267803643,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABOX-Ox61CMdMprTHPB5hStu3J1pLShks5rJL4igaJpZM4LOwKo
.

I haven't heard back, so I'm gonna close this. I have opened a new issue for implementing #[deny(missing_docs)], though: #563.

I haven't heard back, so I'm gonna close this. I have opened a new issue for implementing #[deny(missing_docs)], though: #563.


Performance is extremely bad

When many instances of this component are on the page, the page becomes noticeably less performant.  Performance could be made better by (1) limiting the number of scroll/resize event handlers are attached to window to just 1 and (2) throttling the event handler so that it's only executed once per time interval (250ms is a good place to start).

For me, scrolling quickly past a LazyLoad module with an onContentVisible callback results in:
Uncaught RangeError: Maximum call stack size exceeded

I think it can't handle calling that function so many times so quickly. For instance, when I put a breakpoint in the function and scrolled, it got called a bunch of times. The page also appears to slow down.
Not sure if I'm doing something wrong implementation wise, or if something needs to be fixed.

Well I fixed my problem by putting an if statement before any heavy logic to prevent it from doing too much on each scroll... Oddly enough, after adding this if statement (and changing nothing else?), it also seemed to be calling the callback less frequently.... Not sure how that happened.

@gavacho Can you test version 3.0.0, is there some performance improvements for you?


zack team update

Description

adding info and photo to website
Related Issue(s)

Checklist

[ x] I have read the contribution guidelines
[x ] I have considered the performance impact of these changes
[x ] Suitable unit/system level tests have been added and they pass
[x ] Documentation has been updated

Hi @zackwasli, I need to retouch the photo. It should be a specific size/proportion and have a transparent background to ensure consistency with the other cards. Could you please share the original photo with me?


Dictionary and Sets Lab Completed

I understand that the data set for the File Lab assignment is not clean, but I can't figure out why the spaces are not being stripped completely.  The only thing I can think is that it's some character other than a space.

.strip() should strip all whitespace. but you may have to call it after splitting the text:
In [2]: s = "this, that"

In [3]: s.split(',')
Out[3]: ['this', ' that']

note how the space after the comma is now with " that"


add node-red terminology page

Description

Related Issue(s)

Checklist

[ ] I have read the contribution guidelines
[ ] I have considered the performance impact of these changes
[ ] Suitable unit/system level tests have been added and they pass
[ ] Documentation has been updated

@Hasmin-AC, @ZJvandeWeg, @Yndira-E, I need your brief feedback on the terminology page.

Hi @sumitshinde-84 you need to rebase to the latest version of the main branch.
This was introduced today: https://github.com/FlowFuse/website/pull/2113
Please check the description to see how you can add this page as an item to the left nav and let me know if you need any assistance 😉

Thanks, @Yndira-E. I'll rebase to the latest main branch and add the page to the left nav as described in #2113. I'll reach out if I need any help :)

@sumitshinde-84 Is this a draft with a reason? I think it might be fairly ready?

@sumitshinde-84 Is this a draft with a reason? I think it might be fairly ready?

I have added terminology into the left nav; it's ready for review now.


Support .NET Framework 4.8

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 28.3%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/EasyPDF/ClassEasyPDF.cs
0
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
201

Relevant Lines:
698

💛  - Coveralls


[d3d9] black squares in dead space

Black squares in intro in dead space (steam version), https://github.com/ValveSoftware/Proton/issues/293#issuecomment-592952438
Important: When reporting an issue with a specific game or application, such as crashes or rendering issues, please include log files and a D3D11/D3D9 Apitrace (see https://github.com/apitrace/apitrace) so that the issue can be reproduced.
In order to create a trace for D3D11/D3D10: Run wine apitrace.exe trace -a dxgi YOURGAME.exe.
In order to create a trace for D3D9: Follow https://github.com/Joshua-Ashton/d9vk/wiki/Making-a-Trace.
DO NOT use DXVK together with apitrace!
Software information
Dead space, maximum settings, acrhlinux, aco enabled.
System information

GPU: RX 560
Driver: X.Org Radeon RX 560 Series (POLARIS11, DRM 3.36.0, 5.5.6-17-tkg-pds, LLVM 9.0.1)
Wine version: latest wine-tkg
DXVK version: dxvk from git

Apitrace file(s)

https://mega.nz/#!MIAGgIKY!eRB1tJwK7KPijL8_Hkb_kf2K71L6zNsUiwTLoOa9PlQ

Log files

d3d9.log: Dead Space_d3d9.log
d3d11.log:
dxgi.log:

Does that also happen with ACO? I dont remember seeing that problem on Nvidia?

I tested with aco only.

Ok, I thought LLVM because of that:

Driver: X.Org Radeon RX 560 Series (POLARIS11, DRM 3.36.0, 5.5.6-17-tkg-pds, LLVM 9.0.1)

Should be fixed now.

Yes, fixed, thanks!


Postfix maps generated incompletely

Since the breaking change from v1.5 to v1.6 to generate_postfix_maps, one issue has arosen, which revealed another.
When our Discourse wanted to send its notifications via SMTPS, it lately failed with some thousands of email jobs unfinished, resulting in the following logs at the involved MTA:
Dec  1 00:44:00 email postfix/submission/smtpd[13211]: connect from static.<IP_ADDRESS>.clients.your-server.de[<IP_ADDRESS>]
Dec  1 00:44:00 email postfix/submission/smtpd[13211]: Anonymous TLS connection established from static.<IP_ADDRESS>.clients.your-server.de[<IP_ADDRESS>]: TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)
Dec  1 00:44:00 email postfix/submission/smtpd[13275]: warning: pgsql:/etc/postfix/sql-sender-login-mailboxes-extra.cf lookup error for<EMAIL_ADDRESS>Dec  1 00:44:00 email postfix/submission/smtpd[13275]: NOQUEUE: reject: RCPT from static.<IP_ADDRESS>.clients.your-server.de[<IP_ADDRESS>]: 451 4.3.0<EMAIL_ADDRESS>Temporary lookup failure<EMAIL_ADDRESS><EMAIL_ADDRESS>proto=ESMTP helo=<localhost.localdomain>
Dec  1 00:44:00 email postfix/submission/smtpd[13275]: disconnect from static.<IP_ADDRESS>.clients.your-server.de[<IP_ADDRESS>]

It seems the original /etc/postfix/sql-sender-login-mailboxes.cf query SELECT email FROM core_user WHERE email='%s' AND is_active does not sanitize enough für recipient_delimiter = + enabled setups (standard with modoboa-installer), while there should be another rule which picks up the last failing one in smtpd_sender_login_maps.
# cat /etc/postfix/sql-sender-login-mailboxes-extra.cf
...
query = SELECT mb.address || '@' || dom.name) FROM admin_mailbox mb INNER JOIN admin_senderaddress sad ON sad.mailbox_id=mb.id INNER JOIN admin_domain dom ON dom.id=mb.domain_id WHERE sad.address='%s'

Running the query manually in psql quickly also reveals a syntax error, which can be avoided easily. Unfortunately I was not able to find the pieces of code which are invoked upon calling generate_postfix_maps, as GitHub search only reveals tests and documentation.

I guess you encountered this issue: https://github.com/tonioo/modoboa/issues/940 and I don't think it is related to recipient_delimiter.
@almereyda can you confirm this please, I'd like to release 1.6.2 today.

@tonioo Yes, this seems to fix it. I had manually removed the ) from the .cf file and soon the message log started to empty, while tail -f /var/log/mail.log revealed successful transfers.

Great :)


fix window size issue

Change WindowDialog to PopupPanel to fix window size issue as reported in #15 and #19 .
Unfortunately, when changing to PopupPanel, the window title is lost. However, I don't feel this is a big of a deal.
Before fix:

Fix:

Not sure if I like this solution. As no one else reported same problem I will wait until a next release comes, to see if it is fixed.

Hi! Just wanted to note that I ran into this same issue (window improperly sized, leaving the configuration button unusable, so I can't set my asperite executable path) and it's preventing me from using this plugin :(

Thanks for letting me know @conradhappeliv . Which operational system do you use? I'll look into this again.
As a workaround for now, you can use TAB and ENTER to navigate the buttons, so you can change the path. I  believe you are also able  to resize the screen. This may help.

That did the trick - thanks for the tip!
I'm on Windows 10. I see the issue on both computers I've tried it on, with the monitors I'm using being 1440p and 4K (laptop, so highly scaled).

Interesting. Thanks for the extra info. That does make sense. I saw the issue happening in a retina display, with probably higher DPI than my normal monitors. I knew it was a scaling issue, but at least now I have a pattern. Cheers

Hi I have the same problem on my Macbook pro retina screeen.

Thanks @justdaft . Good to know. I'm fixing this issue on #25  by moving the wizard screen to the dock.
I reopened the old issue #15 until the new version is released in the asset library. Cheers

Thanks, I am also struggling to get the plug in to work, I can't seem to
get the path to aseprite correctly configured, I am on a mac, do you have
any suggestions?
Billy
On Thu, 10 Jun 2021 at 13:36, Vinicius Gerevini @.***>
wrote:

Thanks @justdaft https://github.com/justdaft . Good to know. I'm fixing
this issue on #25
https://github.com/viniciusgerevini/godot-aseprite-wizard/pull/25 by
moving the wizard screen to the dock.
I reopened the old issue #15
https://github.com/viniciusgerevini/godot-aseprite-wizard/issues/15
until the new version is released in the asset library. Cheers
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/viniciusgerevini/godot-aseprite-wizard/pull/22#issuecomment-858583404,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AALQ4B2XXSHHEPCAMJWOSKDTSCWTFANCNFSM43H56G3Q
.

Hi Billy. Try /Applications/Aseprite.app/Contents/MacOS/aseprite. This seems to be the default PATH on MacOS.
You can test it by running it on the terminal.

Fantastic, that works, and your latest version of the addon works
perfectly, no more dialog box issues.
Many thanks
On Fri, 11 Jun 2021 at 05:40, Vinicius Gerevini @.***>
wrote:

Hi Billy. Try /Applications/Aseprite.app/Contents/MacOS/aseprite. This
seems to be the default PATH on MacOS.
You can test it by running it on the terminal.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/viniciusgerevini/godot-aseprite-wizard/pull/22#issuecomment-859259249,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AALQ4BZIOYR3BN3VXJZQA23TSGHUXANCNFSM43H56G3Q
.

Awesome. Thank you


There's no check to prevent creating a recursive entity hierarchy

Description
Currently there's nothing to prevent me from doing:
this.entity.reparent(this.entity);

Creating a recursive entity hierarchy and causing game-freezing stack overflow exceptions to be thrown every frame.
Ideally there should be a check to make sure the child entity isn't already present in the new parent's ancestry. Or at the very least a check to make sure we aren't parenting an entity to itself.
Steps to Reproduce

See above

Hey @Grimeh, I have a fix coming for the self parenting issue. I don't think that reparenting a child to the same original parent should be checked or throw an error though. It seems something like entity.reparent(sameParent, newIndex) could be a valid use of "reparenting to the same parent". Let me know your thoughts though. I may be missing something (cc: @willeastcott @vkalpias).


classify: add dv installation name to api forms

update classify forms

related checkins:

https://github.com/IQSS/geoconnect/commit/7a63a0b87cf309d9d6eed15722ede87fe4fefbdc
https://github.com/IQSS/shared-dataverse-information/commit/09c33c941c9ab32789003c5c84acf4e377148509
https://github.com/cga-harvard/cga-worldmap/commit/5e45e57dd1d0915db3cb4f0709f6c317a66fcf08


Gopkg.toml version vs revision

Hello
I've noticed the Gopkg.toml in the repo does not pin a specific version of the dependencies it uses.
Accordingly to dep documentation:
Note: When you specify a version without an operator, dep automatically uses the ^ operator by default. dep ensure will interpret the given version as the min-boundary of a range, for example:

1.2.3 becomes the range >=1.2.3, <2.0.0
0.2.3 becomes the range >=0.2.3, <0.3.0
0.0.3 becomes the range >=0.0.3, <0.1.0

Is this by design? This means that projects that import github.com/envoyproxy/protoc-gen-validate could also import github.com/golang/protobuf version 1.3.1 but the version defined in the protoc-gen-validate Gopkg.toml file is 1.2.0.
cc @rodaine

PGV is a binary and doesn't expose any runtime dependencies for Go; there's not really anything here to import wrt Go. I'm curious, can you tell me why you are importing it and how you're using it?
That said, the looseness of the definition is controlled by the lockfile (which specifies the exact SHA to use when building).

PGV is a binary and doesn't expose any runtime dependencies for Go; there's not really anything here to import wrt Go. I'm curious, can you tell me why you are importing it and how you're using it?
That said, the looseness of the definition is controlled by the lockfile (which specifies the exact SHA to use when building).

@rodaine We import github.com/lyft/protoc-gen-validate as it's present in imports of the files created by pb.go files with protoc-gen-go. And so we need to pin a specific version for github.com/golang/protobuf.
We are using it here https://github.com/cilium/proxy/blob/master/Gopkg.toml and the project is imported, for example, here:
https://github.com/cilium/proxy/blob/fd8566c74481c22cfe2805ac1c445e312ed08a83/go/envoy/data/accesslog/v2/accesslog.pb.go#L14
Thanks for the reply

I think it's worth pinning a minimum version for these up to a major release (assuming our deps are following semver).


Cannot read .txt file?

Hello,
I'm using this action but i don't know why it can't be read. I'm passing a .txt file to the path and i'm getting this message:

Can you help on this, please?

https://github.com/jaywcjlove/github-action-read-file/blob/8498b54a70094fa5eaa02caca1af2da7e84121b4/src/index.ts#L24-L28
@SirPhemmiey Maybe you need to check if your configuration is correct, if the file exists, or if the file is on the correct Git branch.

I'll be closing this. I used another action and it worked.
Thanks.


Add compatibility with xmlsec 1.3

Minimal changeset to allow unit tests to pass on both xmlsec 1.2 and 1.3.
This includes disabling the key search strict mode that 1.3 enabled by default, since encrypt/decrypt tests fail without it. In the future, it probably makes sense to enable strict mode by default and provide a method to disable it, but I'm not confident enough in my C skills to build it myself, sorry.
Fixes #254

@hoefling Would you be the right person to ask regarding running GitHub workflows and getting feedback on this PR?

I can confirm that this PR builds fine on an Apple Silicon machine, and works (at least for my use case, signing SOAP requests).

Can we please fold this in? This has been an issue for quite awhile and the suggested "work-arounds" are no longer feasible.

On my  x86_64 linux machine, I get the following results
libxmlsec versions:
1.3.1: Success
1.3.2: Success
1.2.37: Failure
1.2.38 Failure
For the failures:
ImportError: .../.env/lib/python3.11/site-packages/xmlsec.cpython-311-x86_64-linux-gnu.so: undefined symbol: xmlSeckeyDataEcGetKlass


closes issue389, select2 uses jquery mousewheel to prevent scrolling …

…out of bounds, fixes shifting of page to the right. @notmessenger alex is going to add testing for this in issue 230 https://huboard.com/softlayer/sl-ember-components/#/issues/56565066 , please merge thanks.

Setting aside whether or not the jquery-mousewheel dependency is need or not, do you not also need to make an addition to https://github.com/softlayer/sl-ember-components/blob/develop/blueprints/sl-ember-components/index.js?

How were you able to replicate this problem?  Visiting http://softlayer.github.io/sl-ember-components/#/demos/sl-select using Chrome 43.0.2357.130 (64-bit) on OS X 10.10.4 (14E46) I am unable to recreate it.

@notmessenger are you looking at develop branch? this issue was introduced by https://github.com/softlayer/sl-ember-components/pull/404

@notmessenger https://github.com/softlayer/sl-ember-components/blob/develop/blueprints/sl-ember-components/index.js I honestly don't know if this file has any bearing on injecting dependencies ( or what it really is to be honest). I have removed dependencies in the past from this file on purpose and built the project and it worked fine(to test what this file actually does). I will add the jquery-mousewheel dependency here if needed however I did not find a need for it as it worked fine with my current changes here.

Issue https://github.com/softlayer/sl-ember-components/issues/230 is no longer a valid issue, so any tests it was going to provide against these code changes needs to be incorporated into this PR.


weechatScripts.signal: init at unstable-2024-01-06

Description of changes
Add Signal bridge. Closes #279160.
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

Wondering if the name should be signal-bridge instead? I've never used WeeChat, so I'm unsure.

Not interested in maintaining this anymore and there seems to be little interest in this package. Feel free to snag the contents of this PR if this package is needed.


Mouse cursor doesn't visually move on GNOME.

This issue was brought up by this StackOverflow post: https://stackoverflow.com/questions/53231971/pyautogui-does-not-move-the-actual-mouse-pointer
The moveTo() function will "move" the cursor (as determined by subsequent calls to position()) but the displayed cursor remains in the same place. This happens on GNOME in Ubuntu 18.04.1 LTS

#279

This issue was brought up by this StackOverflow post: https://stackoverflow.com/questions/53231971/pyautogui-does-not-move-the-actual-mouse-pointer
The moveTo() function will "move" the cursor (as determined by subsequent calls to position()) but the displayed cursor remains in the same place. This happens on GNOME in Ubuntu 18.04.1 LTS

Was a fix ever found for this issue? I am running Crostini on a chromebook and am having the exact same issue.

I did not find a fix

I did not find a fix

In my case I have been informed that Crostini simply does not have access to mouse and screen functions. This is a security feature, so I don't think it pyautogui will be working anytime soon. Have you tried any other methods (maybe another programming language)?

I ended up working around pyautogui's failure by using the PIL library and manually programming in checks for various pixels in order to identify images. This is obviously not the greatest solution, and is prone to break in the future, but so we my game bot was just a side project of mine, it's not as bad as using this method in production.
I do hope sometime in the near future that this issue will be resolved though. It's clear this is something that will cause many issues for many users, and I would like to implement the robust solution that pyautogui promises with it's image recognition library

Switching to xorg worked for me
https://stackoverflow.com/a/65528590/14599124


Security > Socket.IO: Check the token validity

It's already done. The issue, if confirmed, is elsewhere.

It's already done. The issue, if confirmed, is elsewhere.

Ok, let me know.

@jerome-benoit : Shall we build unit test for this?

@jerome-benoit : Shall we build unit test for this?

Yes but that's not an easy unit test to write, we have to add a bunch dependencies to ev-server to implement it.


Blank screen instead of monitor in kde5 kscreen settings window

I can't modify my kscreen configuration as I only see white space instead of my current monitors in kscreen settings.
I tried to run kscreen settings from command line:
$ kcmshell5 kcm_kscreen
kf5.kservice.sycoca: ERROR: KSycoca database corruption!
kf5.kservice.sycoca: ERROR: KSycoca database corruption!
kf5.kservice.sycoca: ERROR: KSycoca database corruption!
kf5.kservice.sycoca: ERROR: KSycoca database corruption!
Recreating ksycoca file ("/home/user/.cache/ksycoca5_en_wraF0soOHIZSqb5ctlvumxL025k=", version 303)
Menu "applications-kmenuedit.menu" not found.
Saving
kf5.kservice.sycoca: ERROR: KSycoca database corruption!
LOAD
kscreen: Primary output changed from KScreen::Output(Id: 638 , Name: "DVI-I-0" ) ( "DVI-I-0" ) to KScreen::Output(Id: 638 , Name: "DVI-I-0" ) ( "DVI-I-0" )
kscreen: Primary output changed from KScreen::Output(Id: 638 , Name: "DVI-I-0" ) ( "DVI-I-0" ) to KScreen::Output(Id: 638 , Name: "DVI-I-0" ) ( "DVI-I-0" )
QQmlComponent: Component is not ready
"file:///run/current-system/sw/share/kcm_kscreen/qml/Output.qml:20 module \"QtGraphicalEffects\" is not installed\n"
Failed to create QMLOutput
QQmlComponent: Component is not ready
"file:///run/current-system/sw/share/kcm_kscreen/qml/Output.qml:20 module \"QtGraphicalEffects\" is not installed\n"
Failed to create QMLOutput
QQmlComponent: Component is not ready
"file:///run/current-system/sw/share/kcm_kscreen/qml/Output.qml:20 module \"QtGraphicalEffects\" is not installed\n"
Failed to create QMLOutput
QQmlComponent: Component is not ready
"file:///run/current-system/sw/share/kcm_kscreen/qml/Output.qml:20 module \"QtGraphicalEffects\" is not installed\n"
Failed to create QMLOutput
QQmlComponent: Component is not ready
"file:///run/current-system/sw/share/kcm_kscreen/qml/Output.qml:20 module \"QtGraphicalEffects\" is not installed\n"
Failed to create QMLOutput
QQmlComponent: Component is not ready
"file:///run/current-system/sw/share/kcm_kscreen/qml/Output.qml:20 module \"QtGraphicalEffects\" is not installed\n"
Failed to create QMLOutput

cc @ttuegel

I'm attaching a screenshot for better understanding:

/cc @ttuegel who might know KDE5 best here.

This was fixed; maybe there was a regression when I updated KDE Frameworks recently. Are you using the unstable channel?

Are you talking about #11244? This is different from that one. In that one I got error that no kscreen backend is available, but there is no such error at this time.
I'm on unstable channel, but I think I experienced this on KDE Frameworks 5.17 too, but my configuration somehow was working and I ignored it. After today's update my configuration broke and I needed to modify kscreen configuration manually (in .local/share/kscreen). So kscreen actually is available and works. It just doesn't show monitors in settings.

OK, I'm on it. Looks like it's just a missing dependency.

Yup, just missing dependencies. Should be working in master.

It works. Thanks! :)


[Feature]: It would be great if the Package target framework version is also displayed in the Versions tab

Related Problem
Displaying the package target framework version is useful.
But it would be great if the Package target framework version is also displayed in the Versions tab
So that no need to traverse multiple times to find the target framework of a particular version.
Displaying the target framework as an additional column would help to locate it even more quickly.
The Elevator Pitch
It helps in finding the appropriate version of the package in a particular target framework version.
Target framework versions can be cached in the backend so that they need not be queried again and again.
Additional Context and Details

A sample versions page, here multiple versions of the package targeting different framework versions, have to traverse each version of the package to find out the exact version of the package to reference.
Displaying the target framework as an additional column would help to locate it even quicker.

In the case of a multi-targeted package, at least displaying the major version helps a lot like .NET 8.0.


adding Taco Bell

Taco Bell excludes using the pound sign (#) in passwords for some reason.

Thank you for the contribution!


chatbot integration in website

@himanshu007-creator, For customer support chatbot we can use Customerly. It is well suited to integrate with WordPress or any Web Based projects.

@himanshu007-creator, For customer support chatbot we can use Customerly. It is well suited to integrate with WordPress or any Web Based projects.

Good! so try to  get familiar with the requirements and be ready to contribute from 1st march

We can also add chatbot using dialogflow as its easy to integrate with website.

@ashi-sharma that is also a nice solution, I will look into it as of now integrating customerly into the website is simple, but their setup is taking  quite large time.

@ashi-sharma , @aniketsingh98571  Good going!,
you both should start working with your idea and lets see who makes the best, fastest solution.

@himanshu007-creator  can you some intents for the Dialogflow? I have integrated dialogflow with the help of communicate plugin on my website and I am planning to do the same for your website/

Have a look here first of all https://github.com/himanshu007-creator/SFC-foundations/issues/12

@himanshu007-creator Can I work on this issue as a part of Gssoc'21?

@himanshu007-creator As I have discussed previously regarding this issue, I would like to go with it.

@Jaya singh /mu-593 Can i work on this issue ,I would create AI based chatbot to your website as a part of Gssoc'21

@himanshu007-creator sir , we can merge chatbot for customer help like this is a NGO based website we can give option to them so that they will not write only they have to select a option .....
We can make a json file for do so .
I have implemented it before for a hotel website.
I would like to implement it .
Thanks

@aniketsingh98571  please make a new PR or modify the old PR and push changes to new website code

@aniketsingh98571  Please let us know the progress !

hey!! @himanshu007-creator  ,Can i use this as my contribution toward the open source project.

i have integrated chat bot using  chatbot.com api ,review the changes and merge the PR soon


respond_to? predicate needs to be renamed

As discussed in https://github.com/dry-rb/dry-logic/pull/55, its current name overrides Object#respond_to? in an incompatible fashion.
We can rename it to responds_to?, for example. Just like includes? delegates to include?.

The same issue exists with eql?.
h = {}
o = Object.new
def o.hash
  Dry::Logic::Predicates.hash
end
h[o] = true
h[Dry::Logic::Predicates]
# ! => lib/dry/logic/predicates.rb:181:in `eql?': wrong number of arguments (given 1, expected 2) (ArgumentError)


Fix confusing documentation typo.

Yikes, thanks for catching this! I agree this is very confusing!


MultiplayerManager's JoinLobby call may unexpectedly fail with HTTP_E_STATUS_FORBIDDEN (0x80190193).

Hello,
I'm seeing an unexpected HTTP_E_STATUS_FORBIDDEN when I use JoinLobby() to process an accepted invite. The steps to reproduce are:

Engage two users, A and B, on Xbox 1 where user A's XboxUserId compares less than B's XboxUserId.
Engage user C, who is not following user A, on Xbox 2.
Create a MPSD session containing user C with the "followed" joinRestriction.
Send an invite from user C to user B.
Accept the invite on machine 1.

In step 5 users A and B are passed to the JoinLobby() function along with the IProtocolActivatedEventArgs. It appears that things go wrong because the loop in multiplayer_lobby_client::commit_lobby_changes_helper() attempts to add the users to the MPSD session in the order that they appear when iterating the "xboxLiveContextMap" object. This is an std::map<> with the XboxUserId as the key type. Since A's XboxUserId compares less than B's the commit_lobby_changes_helper() function attempts to add user A to the MPSD session before user B (the invited user). This fails because user A does not have permission to write to the document.
The MPSD request returns status 403 and the following error message:

The requested session cannot be accessed. The calling user must have the multiplayer privilege and must be a member of the session if the session either isn't open or has a join restriction on it that the user doesn't satisfy, multi-user requests aren't allowed for large sessions, banned xuids can't access the session, users must have the communicate permission to access sessions that require it, users must qualify to access club sessions based on the club rules, and devices other than an Xbox One can only access sessions with user-style authorization.

I suspect that if the joinRestriction used in step #3 was "local" then it wouldn't matter whether user C is following user A, but I have not tested this.
I think the correct behaviour here would for MultiplayerManager to ensure that the user who was invited is added to the MPSD session first. This would enable the other local users on the invited user's machine to join as well. This is what happens when the invited user's XboxUserId, by chance, happens to compare lowest amongst the users attempting to join.
I also posted this bug here back on March 13th.

Hello, Bergerer.
I can see the issue is still investigating per engineering team
We will be closing this thread as the duplicate, for now, please check with "https://forums.xboxlive.com/questions/75492/index.html" for the further update of this issue.
Thanks for your patience.
Young


Why doesn't driverless copy remote PPD options?

I realize that perhaps this is not the best forum for this question, but I tried the mailing list and given past activity there, I'm not sure if there is the right place for this kind of question either.
I have read https://wiki.debian.org/CUPSDriverlessPrinting and surrounding wiki pages. I think I've managed to get a good grasp about how a modern CUPS setup should work, and the differences between CUPS' everywhere and cups-filters' driverless PPD generators.
I'm still confused about a particular detail, though.
I have an old printer (meaning: not supporting AirPrint or IPP everywhere) locally connected to a CUPS server (2.2.10). I use the Gutenprint printer drivers. The shared CUPS queue supports all the types necessary for a everywhere or driverless setup in clients. My desktop computer is running cups-browsed (1.27.4-1) which auto-setups a queue using drvless.ppd. I can print through it. So far so good.
However, what I don't understand is why aren't the PPD options available in the server (the ones exposed by Gutenprint, and very comprehensive) copied over and available in my local queue? I only see generic (and fewer) options for paper size, resolution, etc. When using macOS as a client I noticed that it does copy the server options in the client PPD, which is great. So I am wondering why driverless does not do the same? Is this a deliberate design decision? Or is my set-up broken somehow?
This current behaviour decreases the usefulness of cups-browsed quite a lot, because if I need to control those non-exposed options I'm forced to print directly to the remote queue via a print dialog that does discovery.

I just tested this with the following:
*Linux CUPS 2.3.3 server running latest cups-filters release 1.27.5 with a hp color LaserJet 3700dn setup with HP's OEM PPD.
*macOS catalina client (CUPS 2.3.1)
*Linux CUPS 2.3.3 client running latest cups-filters release 1.27.5
The macOS client pulled the PPD settings as expected. Going into the CUPS web interface on macOS and adding printer has the same result. IPP-Everywhere was available as a driver option in the web interface, but Apple hides this on the native System Preferences pane.
The Linux client too worked as expected. Automatically detected and added printer queues on the CUPS server and pulled all the advanced PPD settings (duplexing on/off, paper sizes, etc.). I checked the generated PPD and the client is using drvless.ppd.
Of note, cups-filters on the Linux client also detected the printer itself and automatically added a queue for it. The printer has a HP JetDirect 635N card in it that supports IPP 1.0 and Bonjour, but too old for IPP-Everywhere/Airprint. CUPS automatically added the queue and the generated PPD specifies the input language as PCL, Postscript, or PDF. Advanced settings were NOT available in this case and the default output resolution was set to 300dpi (printer can do 600dpi). The PPD also had a note in it that the printer didn't respond to IPP media size queries.
On macOS, if I directly added the printer itself, Apple pulls driver software from hp, which includes stuff like a proper picture of the printer for the queue's icon. Advanced options and querying of supply levels (toner and fuser life) is supported. Supply levels did not work via the CUPS queues.

The IPP Everywhere/IPP driverless protocol cannot convey arbitrary options, only options which can be represented by standard IPP attributes. Gutenprint has a huge range of options and many are actually not representable as IPP options and so do not get visible to the client. In contrary the options of most typical PostScript laser printers have their IPP equivalents and so make it to the client via IPP, this is why @NJRoadfan is more successful with his client PPD files.
@NJRoadfan note that due to the fact that your printer is only IPP 1.0 which means pre-driverless-era you get only a very rudimentary PPD when trying to set up your printer as driverless IPP printer. This is also most probably the reason why CUPS cannot poll toner levels from it but a proprietary MacOS driver can.
All this is expected behavior, no bug of cups-filters. Therefore I will close this issue here.
Note also that Gutenprint will soon be moved to the new printer driver format, the Printer Application. This is an emulation of a driverless IPP printer on the local machine, which makes CUPS and cups-browsed set up a print queue fully automatically. The developers of Gutenprint will also find a way for the user to access the vast amount of adjustment parameters.

@tillkamppeter Thanks for the response. I'm aware of the direction of moving into "Printer Applications" instead of printer drivers and PPDs, but I'm guessing that it's still months/years away from being available in practice (packaged in mainstream distros)?
If you don't mind, could you tell me if there's any manual way in which I can set up a local printer queue connected to the remote printer queue that has all the options from the remote queue, but without installing Gutenprint locally and sending "raw" to the server? Basically I just want to use the everywhere/driverless model, and also be able to controll all the options in the client. I understand that this is not possible by default (even though macOS seems do to it), but is there a way I can "force" it, even if I need to copy PPDs around?


Allow building as a standalone library

Separate qmake projects for tests and lib
Set up CI for testing

Hm, just out of curiosity, how does this allow building as a standalone library? It's a header-only library, so you can't create a library from it :wink:
For this reason, the qmake file was there only to build tests since the library itself can't be compiled separately (and thus doesn't need a qmake file).
However, adding CI is nice! :smiley:

It's a header-only library, so you can't create a shared or static library from it wink

Haha you are right, I didn't actually realize that :smile: The setup generates a 16 KiB .so library, I have no idea what it actually contains.
However, the qmake setup is still useful for the make install target and in order to generate pkg-config files, right?

The setup generates a 16 KiB .so library, I have no idea what it actually contains.

Okay, funny :grin:

However, the qmake setup is still useful for the make install target and in order to generate pkg-config files, right?

Ah, good point, probably yes :+1:

I gave up on the pkg-config file. Couldn't get it generated without also generating useless .so files. Furthermore, the pkg-config file would contain -lfontobene-qt5 which is explicitly not required in a header-only library.


New google-cloud-bigquery dependency versions not supported

Problem
With the new google-cloud-bigquery dependency versions, I keep getting None as a result for every query I run.
How to reproduce
I have tested with the code of the very first example of the readme:
from sqlalchemy import *
from sqlalchemy.engine import create_engine
from sqlalchemy.schema import *
engine = create_engine('bigquery://project')
table = Table('dataset.users', MetaData(bind=engine), autoload=True)
print(select([func.count('*')], from_obj=table).scalar())

For this combination of dependencies:
google-cloud-bigquery==2.8.0
pybigquery==0.5.0

Result: None
For this other combination of dependencies:
google-cloud-bigquery==1.24.0
pybigquery==0.5.0

Result: 10

@pedro-iatzky Do you have the BigQuery Storage API enabled?


Onboarding presentation placeholder

Fixes #15

Shall I do some more steps to test this one to test locally? Links are failing on redirection in index.html to both sections.

Shall I do some more steps to test this one to test locally? Links are failing on redirection in index.html to both sections.

That's not going to work without HTTP server running (so no access using file:// protocol). To test it locally you can simply start a Python-based HTTP server in that directory (depending on Python version it's either python -m SimpleHTTPServer or python3 -m http.server) and go to http://localhost:8000. There these links should work properly.