[Storage] _queue_client_async.py mypy fixes

_queue_client_async.py:203: error: Incompatible types in "await" (actual type "None", expected type "Awaitable[Any]")  [misc]
_queue_client_async.py:266: error: Incompatible types in "await" (actual type "List[SignedIdentifier]", expected type "Awaitable[Any]")  [misc]
_queue_client_async.py:409: error: Incompatible types in "await" (actual type "List[EnqueuedMessage]", expected type "Awaitable[Any]")  [misc]
_queue_client_async.py:475: error: Incompatible types in "await" (actual type "List[DequeuedMessageItem]", expected type "Awaitable[Any]")  [misc]
_queue_client_async.py:670: error: Incompatible types in "await" (actual type "None", expected type "Awaitable[Any]")  [misc]
_queue_client_async.py:743: error: Incompatible types in "await" (actual type "List[PeekedMessageItem]", expected type "Awaitable[Any]")  [misc]

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-storage-queue

/azp run python - storage - ci


Logs not being exported

This should have been solved in #191, and will be in v0.1.0-rc61.
The log export implementation failed where the file did not exist.


[Feature Request]: Adding an active Nav link Scroll when clicked on pages

Is there an existing issue for this?
NO

[X] I have searched the existing issues

Feature Description
Adding an active Nav link
Hey @Harish-2003
When the user clicks on home, about us, FAQ's and Contact Us tabs the website is directly moving into that tab without any scroll animation
Watch this :
https://github.com/user-attachments/assets/7e79eb11-4f12-4d48-bac5-e7593b014b13
Use Case
So let me add a scrolling animations to all the tabs so that the user can able to visulaize the scolling effect smoothly instead of direct moving
After :
https://github.com/user-attachments/assets/941732eb-362b-4cb7-9c7c-2acc8060a6ae
Assign me this issue as a GSSOC and Hacktoberfest Contributor
Priority
High
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC contributor
[X] I want to work on this issue
[X] I'm willing to provide further clarification or assistance if needed.

Hey @OpenTekHub @Harish-2003 , let me know why you have still not assigned this issue

@Harish-2003
I hope you will add the labels and assign the issue to @SumithThota .
I will be looking forward for your response .
Regards ,
Community Manager
Gssoc-Extended Core Team


Unable to get Attribute name when creating property method

System: Windows11
Labview version: LV2020 64-bit
OpenGDS version: 1.2.72/1.2.86

As shown in the figure, when I try to use GOOP to create a property method, it cannot get the property name, but outputs the %s format character. This problem was found in both versions 1.2.72 and 1.2.86. I searched [GLA Summit 2021: OpenGDS with Interface support], and found that the attribute name can be obtained correctly in the video. Is there any place that needs to be set?

The screenshot of the video appeared at 10:20

/resource/Framework/Providers/Open_GDS/ClassProviders/Provider_EndevoGOOP400/ClassWriterGOOP400_class/GetClassProperties.vi

the value of ChildTag is output as ObjectAttribute?s,
I checked the code not sure if the problem is here, The code here may not have changed in many years.

I now suspect that it is a problem with the escape character '\A7' when processing strings. This mysterious escape character is used in many places in the code to format and split strings. But I failed to use it to format the string here, and the second string became '? ’, which leads to other problems later on. Perhaps this is a feature of Labview, not a code problem? ? ?
I will try on other versions of Labview later to see if the same problem occurs.

I realize now that it has something to do with my OS language setting, I am using Windows11, the system language is Chinese, and I am using the English version of Labview2020-64 bit, when I change the system language to English it works fine. I'm not sure if I need to close this issue now, I'll modify the separator '\A7' to make it work on my system.


initial commit. split from vokal-ng-lib

@jrit
This PR is essentially just pulling the requisite items from ng-lib, with the addition of the grunt task for creating the /dist files.
Due to issues with npm install for karma-phantomjs-launcher, I couldn't get tests to work for this.

We'll want to run these builds against https://drone.io/ instead of vokal's drone server since this is public. That can come in the next PR. Should we mark this as 1.0?

Where is it set, which drone server this runs against? And yes, this should probably be set as 1.0.

You would register the hook on the drone.io site, not in the repo. We should figure out if we have a vokal account next week and go from there.


Add Traffic Ops 304 Not Modified Support

At least for the large, common endpoints hit by Traffic Portal: Delivery Services and Servers.
When using Traffic Portal, every time you navigate back to the DS or Servers page, it re-requests the entire file from TO.
If we add 304 Not Modified support to these endpoints, Traffic Portal will be drastically faster, every time you click back from a DS to the list of all DSes, or Servers.

If it helps, the DS Snapshots PR adds ETag support to the CRConfig. It has a set of functions for adding and handling last-modified requests:
https://github.com/apache/trafficcontrol/blob/5e88f741d355945f9fc3961570ec506c2f085daa/traffic_ops/traffic_ops_golang/crconfig/cachecontrol.go
I don't object to anyone copying those functions, and then I'll just change the DS Snapshots PR to use whatever package they end up in (lib/go-util?).

@rob05c want to rename this to Traffic Portal? This has been implemented for TO.

@rob05c want to rename this to Traffic Portal? This has been implemented for TO.

This probably requires TP to send if-modifed-since headers to fully take advantage of this.

This probably requires TP to send if-modifed-since headers to fully take advantage of this.


feat: add skywalking_agent 0.6.0 release

update skywalking_agent with release 0.6.0

:tada: This PR is included in version 1.183.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


ISPN-14084 MergePolicyCustomTest - Inbound transfer was cancelled

https://issues.redhat.com/browse/ISPN-14084
~An initial PR trying to debug the problem. At this point I am trying to use the CI to debug the problem.~
After some debugging, there are some cases when a rehash event is issued during the tests. The rehash listener will start canceling all current segment transfers and complete the future with a CacheException. Since the InboundTransferTasks are canceled before completing the future, the InboundTransferTask's callback would invoke cancel again, which could lead the future completion with a CancellationException instead of the CacheException.
I only moved to first complete the future and only then cancel the tasks.
Also, includes a change to the DefaultConflictManager, because, in the scenario above, the CancellationException is wrapped in a CompletionException. I've changed it so we can handle this case too.
I did not find out why the rehash event happened in the test. I've run the CI multiple times, but the problem did not occur.

Thanks @jabolina


Updates README to explain why we are using this fork

This is a README update that would explain why we are using this fork.
This should be merged together with https://github.com/jetstack/cert-manager/pull/3850 which actually starts using this fork- currently we still use the one from Maartje's account.
Signed-off-by: irbekrm<EMAIL_ADDRESS>

Holding till https://github.com/jetstack/cert-manager/pull/3850 gets merged.
/hold

/lgtm
This looks like an unambiguously good thing to me.

/approve

/hold cancel

I guess I need to add an OWNERS file to this repo 😄

I have added an OWNERS file https://github.com/cert-manager/crypto/blob/cherry_pick_altcert/OWNERS
(I did not make a PR for that as it would have needed to be force-merged anyway)
I guess perhaps it wants another approve 😅

/approve

/approve

https://user-images.githubusercontent.com/1972547/114210273-7a6b7800-9957-11eb-95b4-2e6bf2acb206.mp4

Haha, it does feel weird to be changing things in this repo :)
I almost accidentally PR-ed this against upstream as that came up first in the list 😆


Use config branch

Checklist

[ ] Update changelog in CHANGELOG.md.
[ ] Make sure values.yaml and values.schema.json are valid.

Shouldn't be needed anymore.


Running time column is blank on dashboard

this happens in a rare case when "create time" equals to "finish time"

Done!


Random Password Generator Added

Related Issue
-Created a password generator that generates a random password containing when length and criteria is selected.
Checklist:

[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have made corresponding changes to the documentation.
[x] New and existing unit tests pass locally with my changes.

Screenshots

Extremely sorry, I'll make the changes ASAP

@Rizwan-S please have a look

I think you have removed the wrong file

@Rizwan-S please have a look, git pull might have caused addition of some updated information


Feature/news menu

Coverage increased (+0.4%) to 90.895% when pulling d34caa9258ce3bf95ebe3183c272524425713082 on feature/news-menu into 1ad94e30ecdb29d5436a1c2c4311fb69a7acd09c on master.


Roller height calculation or label position wrong for theme mono

Hi!
I have seen a problem with the height calculation when using the theme mono.
The upper roller border is too close to the highest option. Especially bad when the roller has a thicker border because then the highest option cut off. Also the selected option is positioned too high on the "selected frame".
See my example below.
BR
Stephan

Hi,
As roller and ddlist are very similar do think it is related to this? https://github.com/littlevgl/lvgl/issues/174

Uh, no idea. I was not able to understand what is causing the problem so far.
Sorry!

Hi,
I check it. It seams like a theme issue instead of roller issue. I added "Ág" characters to the roller ("Á" to see top of the characters and "g" to see the bottom) and I saw that the position is correct.
It seems to me only a little bit more line space is required in the mono theme. I added an update to beta
Oroginal:

More line space:

Hi!
It is better, but still there is not enough space between the top label and the border.
This is a screen shoot after your latest update. You can still see that the top text is too close to the border. Also the selected option is too close to the "selected" frame.
BR
Stephan

Hi,
I tested it with "A|g" characters and looked like this for me (using the built in 10 px font):

Which font have you used? And can you send a screenshot with "AIg" texts on the roller?

Hi!
Sorry, I was not able to get the special character printed.
Maybe I am doing something wrong, but I don't have the time right now...
Have you tried a 3pt border? That is the default 'focused' style.
BR
Stephan

Thank for the image. It seems there is a few pixel error. I will examine it deeper.

I checked it with 7px and 8px monochrome fonts on Font converters website. And found that there was zero (8px font) or one pixel (7px font) error.
Is it possible that there a few empty line in your font below the letters. Can you tell which font do you use? Or even send the font's "C" file?

Hi
I am using the lv_font_dejavu_10.
Here the source files.
RollerTest.zip
BR
Stephan

Hi,
Thank you!
I found a minor issue which caused 1 px error when calculating the size of the "selected" area.
However using only numbers still looks ugly because number doesn't have lower parts like some letters have.
I suggest to use an other font which is designed to be small for low resolution displays. There are two on the font converter's website (namely: Syncronizer and Unscii)
With Syncronizer I got the this (a 2 px border in style mod function might be better):

Hi!
Yes, I am already looking for decent 8pt and 16pt fonts. This "syncronizer font" looks fine for 7&8pt. However, I was not able to find a decent larger font. Do you have a tip for me?
BR,
Stephan

Hi,
You can convert the 7px and 8px fonts in 14px and 16 px size. They will remain pixel perfect.
However this way the the glyphs are really just doubled which means for example a "/" character will be built from 2x2 blocks instead of smoother 1x1.

Or you can simply find 16px pixel fonts in google. Like this: https://fontstruct.com/fontstructions/show/1123988/properhitboxglobal

Hi,
Pixels fonts need to be converted in exactly the same height as they designed.
I played with it and found that it is a 9px font. I attached the result.
px_font_9.c.zip

As the original issue is solved I close this.


Recreating Data Processing in another disassembler

Thanks for the awesome work. As the title suggests, I don't have a Binary Ninja license and am looking to potentially recreate the data processing steps in something like Ghidra.
I understand the format for the CFG output but it's very unclear what the output should look like for the DFG.
Would it be OK for you to release the output from processing something like /bin/ls using your scripts to provide folks who don't have Binary Ninja a guide for what the expected format is?

The format of CFG and DFG are the same. Maybe I will find a sample file including both CFG and DFG data samples in the repo later.

Hi, I am wondering if there are any update on this issue? A sample file for the CFG/DFG (both are preferred) or some support for Ghidra (instead of BinaryNinja) is appreciated.
Thanks in advance.

We have the sample in README. It is pretty simple.
https://github.com/palmtreemodel/PalmTree#dataset-format


What would it take to support zsh ?

Hi, I find your plugin very useful !
Do you think it's possible to implement easily zsh support within your plugin ?

@bric3
It makes sense to allow extensions to support other similar shells in BashSupport.
I haven't used zsh yet. Is there a list of differences to Bash? Which feature/change would you like to see the most?

Hi,

There's of course the shebang
I don't know if that matters yet, but zsh offers support for some form of expansion in the prompt (or print -P) using %.
The grammar should be almost the same
The redirection operators may be cause problems to the parser.

But also I think I got confused by some other errors that are unrelated to zsh :

local usage can be highlighted as an error
function oh_my_git_info {
    local oh_my_git_string="";

'local' must be used in a function less... (⌘F1)
A global variable declared as local is invalid.

and thus other errors appears, such as invalid token

Some variable substitution seem to cause trouble to the syntax highlighter
on="${omg_on:-%B}";
off="${omg_off:-%b}";
red="${omg_red:-%F{red}}";
green="${omg_green:-%F{green}}";
yellow="${omg_yellow:-%F{yellow}}";
violet="${omg_violet:-%F{magenta}}";
reset="${omg_reset:-%{%f%k%b%}}";

The string appears not finished, hence the rest of the function is not parsed correctly. The problem might be located in the parameter expansion expression.

For reference :

http://zsh.sourceforge.net/Doc/Release/Expansion.html#Parameter-Expansion
http://zsh.sourceforge.net/Doc/Release/Prompt-Expansion.html
http://zsh.sourceforge.net/Doc/Release/Shell-Grammar.html#Shell-Grammar
http://zsh.sourceforge.net/Doc/Release/Redirection.html#Redirection

I have commit a pull request https://github.com/jansorg/BashSupport/pull/488
support "#!/bin/zsh" file as Shell file


[UI] Updated the UI of confirmation prompt

Signed-off-by: Meghana-12<EMAIL_ADDRESS>Description
options prop is passed to component PromptComponent. The first option, options[0], is considered as the primary text and is placed on the right with the current background(primary) as the button color and the second option, options[1], is considered as secondary text and is placed on the left with transparent background.

On hover :

This PR fixes #4209
Notes for Reviewers
@Lee I've attached the screenshot of the update. Is this what you were expecting ?
Signed commits

[x] Yes, I signed my commits.

Codecov Report

Merging #4266 (92f9161) into master (ebacbc5) will increase coverage by 0.81%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4266      +/-   ##
==========================================
+ Coverage   27.07%   27.89%   +0.81%
==========================================
  Files          59       59
  Lines        5015     5037      +22
==========================================
+ Hits         1358     1405      +47
+ Misses       3240     3215      -25
  Partials      417      417

Flag
Coverage Δ

gointegrationtests
10.14% <ø> (+0.15%)
:arrow_up:

integrationtests
∅ <ø> (?)

unittests
20.15% <ø> (+0.80%)
:arrow_up:

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

Impacted Files
Coverage Δ

internal/store/store.go
76.47% <0.00%> (-10.20%)
:arrow_down:

mesheryctl/pkg/utils/platform.go
3.84% <0.00%> (-0.08%)
:arrow_down:

mesheryctl/pkg/utils/helpers.go
19.50% <0.00%> (ø)

mesheryctl/internal/cli/root/system/start.go
54.96% <0.00%> (+0.66%)
:arrow_up:

mesheryctl/internal/cli/root/system/config.go
5.20% <0.00%> (+4.07%)
:arrow_up:

mesheryctl/pkg/utils/sse-client.go
92.50% <0.00%> (+92.50%)
:arrow_up:

Continue to review full report at Codecov.

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

@leecalcote Should I create a new PR to add the changes @warunicorn19 suggested ?


Stop excessive blank lines in SEGSAdmin output

Change use of function from appendPlainText() to insertPlainText()
Issues closed
Closes #761

Okay this is ready for merge if build passes and reviewers happy, cheers

Will squash and merge once CI clears


Fix SAP client compilation failure with Update 10

Description:
The HTTP package has introduced a change in the client invocation return type(http:Response|anydata -> http:Response|anydata|stream<http:SseEvent, error?>. This change is breaking the SAP client.
Describe your task(s)

[x] Fix the breaking change
[ ] Release a new version for Update 10

Tested the following connector repos with the fix:

[ ] module-ballerinax-sap.s4hana.qm
[ ] module-ballerinax-sap.s4hana.sales
[ ] module-ballerinax-sap.s4hana.procurement
[ ] module-ballerinax-sap.s4hana.inventory
[ ] module-ballerinax-sap.s4hana.delivery


[Bug] Crash after clicking "Settings"

Which version of the app?
WinUI 3 Gallery
Description
Win10 22h2 (19045.4291), installed in Microsoft Store, VS 2022 has been installed with .net and cpp desktop target.
How could I debug it or upload the log files as a user who is not an expert in programming?
Screenshots
https://github.com/microsoft/WinUI-Gallery/assets/63829496/3bf31042-ab07-42ab-bd3f-6cb2d93d1cb0
Windows version
Windows 10 22H2 (19045)
Additional context
No response

There seems to be an issue with the video - at least for me, at the moment. Can you check it. And also give steps to reproduce in text please. Thanks :)

This should be resolved with #1480 soon

Duplicate of #1477

@Mikachu2333 To debug a store app, you can try attach to process with Visual Studio: https://learn.microsoft.com/en-us/visualstudio/debugger/attach-to-running-processes-with-the-visual-studio-debugger?view=vs-2022
Let me know if this helps.


Setting position using x, y, z and rotation using Quaternion and Euler angle

When trying to set position and rotation using rotation matrix obtained from OpenCV, there is unexpected sudden changes orientation (usually when rotation matrix values flip signs)
As per my observation the transform matrix used in filament have translation as 4th row
1.0f,  0.0f, 0.0f, 0.0f,
0.0f,  1.0f, 0.0f, 0.0f,
0.0f,  0.0f, 1.0f, 0.0f,
x,       y,      z,    1.0f
Some other libraries have translation as fourth column
1.0f,  0.0f, 0.0f,  x,
0.0f,  1.0f, 0.0f,  y,
0.0f,  0.0f, 1.0f,  z,
0.0f,  0.0f, 0.0f,  1.0f
Requesting feature to set rotation as Quaternion or Euler angles, and position as x, y, z.
It is a lot easier than using matrices, is there any other way to set the orientation and position of an entity?

Filament uses column major matrices, not row major matrices. Which means the 4th column is the translation.. Our matrix API provides several utility functions to create matrices from Euler angles, quaternions, etc. I recommend you use those (for instance mat4::rotation(angle, rotationAxis) or mat4(myQuaternion)).

This is from gltf-bloom sample of android to rotate model,  modified fourth row's first value to get translation in x direction and similar results for y and z.
 tm.setTransform(tm.getInstance(filamentAsset.root),
                    floatArrayOf(
                            cos(v),  0.0f,    -sin(v),    0.0f,
                            0.0f,      1.0f,     0.0f     0.0f,
                            sin(v),   0.0f,    cos(v),    0.0f,
                            v,        -1.7f,     0.0f,    1.0f
                    ))

Where as if you change the fourth column it is resulting in scale change in respective axis.

You are mistaking the ASCII column for the in-memory column. Filament expects matrices in column-major order, so when you pass a matrix by writing floatArrayOf(a, b, c, d...) the values a, b, c and d are those of the first column, not the first row. If you are using Filament from Kotlin you could try to use the math library I wrote: https://github.com/romainguy/kotlin-math They'll give you the features you want and it uses column major order as well.


GH-33631: [R] Rewrite JIRA ticket numbers in pkgdown documents to GitHub issue numbers

Rewrite the JIRA issue numbers to the GitHub issue numbers, so that the GitHub issue numbers are automatically linked to the issues by pkgdown's auto-linking feature.
Issue numbers have been rewritten based on the following correspondence.
Also, the pkgdown settings have been changed and updated to link to GitHub.
I generated the Changelog page using the pkgdown::build_news() function and verified that the links work correctly.

ARROW-6338	#5198
ARROW-6364	#5201
ARROW-6323	#5169
ARROW-6278	#5141
ARROW-6360	#5329
ARROW-6533	#5450
ARROW-6348	#5223
ARROW-6337	#5399
ARROW-10850	#9128
ARROW-10624	#9092
ARROW-10386	#8549
ARROW-6994	#23308
ARROW-12774	#10320
ARROW-12670	#10287
ARROW-16828	#13484
ARROW-14989	#13482
ARROW-16977	#13514
ARROW-13404	#10999
ARROW-16887	#13601
ARROW-15906	#13206
ARROW-15280	#13171
ARROW-16144	#13183
ARROW-16511	#13105
ARROW-16085	#13088
ARROW-16715	#13555
ARROW-16268	#13550
ARROW-16700	#13518
ARROW-16807	#13583
ARROW-16871	#13517
ARROW-16415	#13190
ARROW-14821	#12154
ARROW-16439	#13174
ARROW-16394	#13118
ARROW-16516	#13163
ARROW-16395	#13627
ARROW-14848	#12589
ARROW-16407	#13196
ARROW-16653	#13506
ARROW-14575	#13160
ARROW-15271	#13170
ARROW-16703	#13650
ARROW-16444	#13397
ARROW-15016	#13541
ARROW-16776	#13563
ARROW-15622	#13090
ARROW-18131	#14484
ARROW-18305	#14581
ARROW-18285	#14615

Closes: #33631

@github-actions crossbow submit preview-docs

Thanks for building preview docs and reviewing!


Refactor __init__.py, mostly by moving Parameters to parameters.py

Fixes https://github.com/holoviz/param/issues/648
This is also adding __all__ to the __init__.py.
When you start to touch so many lines it's tempting to do some formatting and even more refactoring, styling, imports sorting, etc.. I've decided to stick to address the limitations exposed in the issue; this can be done later.
I think Github supports ignoring specific commits on blame, I'll enable that once this is merged.

Thanks. The large-scale moves here make sense, but are hard to review, so I'll just trust you didn't insert anything beyond what was already there. :-)
Are there special techniques to preserve git blame in a case like this? E.g. if the git history includes git mv __init__.py parameters.py followed by recreating __init__.py, will github know that the history of the classes in parameters.py can be found in older __init__.py versions? Now would be the time to be careful about that, and I don't know how best to achieve that.


Matrix for environment variables

Is there an existing feature for managing environment variables in a similar way to module dependencies?
For example, I'd like to run my benchmarks WITH_FOO=1 and WITH_FOO=0, then have the benchmarks call a function with this as an argument, e.g.:
if os.environ['WITH_FOO'] == '1':
    # do something
else:
    # do something else

def time_something():
    # usual benchmark stuff

The purpose of this is that the Shapely module has an optional "speedups" module that uses compiled cython instead of ctypes. The changes are global (i.e. many functions benefit from the speedups module), and I'd like to be able to benchmark with and without them, without having to duplicate a lot of code.

There isn't currently a way to do this, but it's a very good idea.  I'd definitely accept a patch, and if I find the time for this, I'll self-assign first.

I would love to see this feature implemented too!  My endless kudos to the one who implements it ;)
In our case we would love to benchmark @datalad under different default versions of git-annex repositories.


Doc comments for --node-taints option.

If you create a nodepool with X number of tags, and later update them to Y number of tags using the "--node-taints" flag they do get updated, but if you scale up the new nodes will go back to using X number if tags. Should we add a "note" section explaining or noting this behavior?

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

ID: 39fce7ab-ab56-44c6-b9a8-405444247a0b
Version Independent ID: dabbb6c6-c616-a6ec-61f6-1b9d3bb069e8
Content: az aks nodepool
Content Source: latest/docs-ref-autogen/aks/nodepool.yml
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

route to CXP team

@pecrow Thank you for reaching out, we are looking into it.


[Identity] OnBehalfOfCredential, credential-only approach

This PR is a way to approach the requirements for the OnBehalfOfCredential from a credential-only perspective.
While “swaping credentials” approach here: https://github.com/Azure/azure-sdk-for-js/pull/16924 is on the ambitious and generic side, this approach is on the most minimal, less obtrusive side. You can read more about the underlying problems and two solutions we’ve discussed (neither on this PR) going to this link: https://gist.github.com/sadasant/ece98248cb8cd2df5edd03ab92749f35
This PR allows:

Keeping the MSAL clients per user-assertion.
Avoiding problems with parallel requests with this credential.

If this PR seems favorable, it would need:

Finishing documentation and tests.
It will need a separate design change from .Net: extending the AccessToken type with a “refreshOn” property, to control our tokenCycler from the credentials directly.
⚠ Important: To re-use the context, we will need to return an opaque “context” object from the context call and allow users to pass it through the userAssertionContext call in order to re-use that same context in the future.

The public API from this PR aims to be as closed as possible to .Net’s code. .Net’s code looks as follows:
var cred = new OnBehalfOfCredential(TenantId, ClientId, Secret);
var client = new Client(Uri, cred);

using (var _ = UserAssertionContext(CurrentUserContextToken)) {
  // do request in the context of CurrentUserContextToken
  return Ok(client.GetItem());
}

This code above declares the OnBehalfCredential with three of the four required values to authenticate, then uses a UserAssertionContext to alter the internals of the requests that take action within the using scope, specifying the last piece for the On-behalf-of authentication flow to finally succeed: the user assertion — which is contained in a class instance, to allow more things to be stored with it.
In this PR, I’m proposing an approach that would make things look as follow:
const credential = new OnBehalfOfCredential();
const client = new SecretClient("https://key-vault-name.vault.azure.net", credential);

// Must be called within a userAssertionContext(), so the following lines will fail:
// await credential.getToken() // will fail.
// await client.getSecret("secret-name"); // will fail.

// This will work:
await credential.userAssertionContext("current-user-context-token", async () => {
  await client.getSecret("secret-name"); // will succeed.
});

// Any calls outside of `userAssertionContext` will fail.
// Attempts to set multiple parallel userAssertionContexts will fail.

This approach works by replacing the credential’s getToken with a good getToken that can only work in the scope that executes the callback of userAssertionContext. Any call outside of that scope will fail. Any attempt to repeat a userAssertionContext while there’s one already running will fail. We give full control to the users and we prevent problems with parallelisms, we keep MSAL clients per user assertion, and we also automatically clean any caching once the user returns on their callback scope.
How does this look? Feedback appreciated.

Closed on favor of: https://github.com/Azure/azure-sdk-for-js/pull/16995


refactor: Make compose.js more testable.

This series of commits should make the code a bit more organized, and it will also make it easier to stub out certain methods when we test compose.start().
Some followups to this will be to actually extract compose_actions.js (there's a shim now) and to actually write unit tests. :)
These are all refactorings.

Automated message from Dropbox CLA bot
@showell, it looks like you've already signed the Dropbox CLA. Thanks!

The test failure is the same DB query count flake I pointed @umairwaheed at earlier today.  Merged, thanks @showell!


[SPARK-26807][DOCS] Clarify that Pyspark is on PyPi now

What changes were proposed in this pull request?
Docs still say that Spark will be available on PyPi "in the future"; just needs to be updated.
How was this patch tested?
Doc build

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8402/
Test PASSed.

Test build #102931 has started for PR 23933 at commit 859d3ba.

Test build #102931 has finished for PR 23933 at commit 859d3ba.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102931/
Test PASSed.

Merged to master.


fix(bin) : fix overflow and allow max for rpc max response size

Ref : https://github.com/paradigmxyz/reth/issues/5376
if checked_mul returns None (meaning an overflow would have occurred), unwrap_or will return u32::MAX
this would prevent overflow by using checked_mul and would return u32::MAX on overflow

whoops, sorry I sniped this one...

do you want to add support for "max"?
similar to this type:
https://github.com/paradigmxyz/reth/blob/e26bc175316a77038900dd9d198608d300b7a545/bin/reth/src/args/types.rs#L5-L7

whoops, sorry I sniped this one...

oh really sorry, didn't saw you already opened pr for it!
Nice !

similar to this type:

sure, ty lot for this offer ! i'm taking this


[stable/wordpress] Add option to use HTTPS for readiness and liveliness

The healthchecks are hardcoded to use the http port, which prevents users from setting up Wordpress to serve over https. If you change your wp_config to serve over https, the readiness and liveliness check fail and the container gets into a crashloop backoff.
This PR adds a simple flag to move these checks to https + removes a secret value that was unused.

/ok-to-test

/ok-to-test


Implement new default action to set slots after each user turn

Proposed changes:

Closes #9364

Status (please check what you already did):

[x] added some tests for the functionality
[ ] updated the documentation
[ ] updated the changelog (please check changelog for instructions)
[X] reformat files using black (please check Readme for instructions)

It looks good, just wrote a couple of comments.


Loading fixtures in tests. Documentation wrong?

The documentation shows how to load fixtures in tests: https://github.com/comfy/comfortable-mexican-sofa/wiki/Working-with-CMS-fixtures#using-fixtures-in-tests. This didn't work for me. The task doesn't run when doing rake test. Maybe this is because rake db:test:prepareis gone in Rails 4? Also the code snippet is missing the environment constraint on the task definition so Comfy wouldn't be defined. Also Site creation is missing the path attribute.
Anyway, what about just a little test helper like this?
require 'rake'

def load_comfy_fixtures!(from='comfy-photocase', to='photocase')
  Comfy::Cms::Site.create!(:identifier => to, :hostname => host, :path => '/')

  Rake::Task.clear
  Photocase::Application.load_tasks
  ENV['FROM'] = from
  ENV['TO']   = to
  silence_stream(STDOUT) do # If you don't want to see the import task log messages in your test output...
    Rake::Task['comfortable_mexican_sofa:fixtures:import'].invoke
  end
end
```

You could load fixtures in individual testes via ```load_comfy_fixtures!```or via ```setup```-block.

I know I can change documentation but I wanted to check with you first :)

I've moved the code from a rake task to test/test_helper.rb and it's importing the fixtures just fine

What you mean?

The code you posted:
  Comfy::Cms::Site.create!(:identifier => to, :hostname => host, :path => '/')

  Rake::Task.clear
  Photocase::Application.load_tasks
  ENV['FROM'] = from
  ENV['TO']   = to
  silence_stream(STDOUT) do # If you don't want to see the import task log messages in your test output...
    Rake::Task['comfortable_mexican_sofa:fixtures:import'].invoke
  end

remove it from the rake task and place it in test/test_helper.rb

I would hope so :D


allow users to configure basic authentication credentials

@joemasilotti
Hi Joe, my staging app is behind basic auth and I need a way to configure TurboNavigator for that. Here is how I did it in my fork. Please let me know if this is something we can add to the library or If you have something else in mind.
Thank you!

Oh, interesting! I've never used basic auth before.
What's the downside of leaving a blank implementation in the extension? I'm wondering if Xcode's exception is a better indication that something needs to be done vs. it silently not working.
Thinking a bit, does it make sense to log/warn the developer that they need to implement it? Curious to hear what you think?


Spike: Identify Options for Hard Bounces

Goal
We see hundreds of Hard bounces per day. This is absolutely not good for our rating with Amazon. Investigate and present ways we could reduce the number of hard bounces to @k-macmillan  and @mjones-oddball .
Checklist

[ ] Reasons for hard bounces identified, tracked in a document
[ ] Scheduled discussion
[ ] Document updated with decision

Timebox
2 days
Assumptions

All hard bounce data is in Cloudwatch

Additional Info/Resources

SES Hard Bounces Cloudwatch query: "message" "Status Hard bounced"

Still relevant.


DL: Naming improvements for dependent and independent varname

JIRA: MADLIB-1324
Previously, the name/value of 'dependent_varname' and 'indenpendent_varname'
in madlib_keras_fit() is a bit confusing. This commit does the following:

remove dependent_varname and independent_varname from the fit()
interface since we know what these are from the minibatch summary table,
and user must use minibatching for DL
use original column names from the source table (i.e., pre minibatch) in
fit summary table, not the generic names dependent_var and independent_var
This change will also change the predict summary table to have the
original column names instead of dependent_var and independent_var

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/871/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/873/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/879/

LGTM
SELECT madlib.madlib_keras_fit('mnist_train_packed',  -- source_table
                               'model1', -- model
                               'model_arch_library',  -- model_arch_table
                               1,  -- model_arch_id
                               $$ loss='categorical_crossentropy', optimizer='adam', metrics=['acc'] $$,  -- compile_params
                               $$ batch_size=128, epochs=4 $$,  -- fit_params
                               5,  -- num_iterations
                               FALSE,  -- use_gpu
                               NULL,  -- validation_table
                               'Frank',  -- name
                               'A test model'  -- description
                              );

produces a model

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/886/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/891/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/892/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/893/


View not found with root finder // if app has a theme

What steps will reproduce the problem?
1: Add theme config
      'view' => [
          'theme' => [
              'basePath' => '@app/themes/abc',
              'baseUrl' => '@web/themes/abc',
              'pathMap' => [
                  '@app/views' => '@app/themes/abc',
              ],
          ],
      ],

2: Call inside any module
module: report
controller: student
action: xyz
$this->render('//base-view/xyz', ['var_name' => 'some_value']);
3: File should be like below:
theme-dir/abc/base-view/xyz.php
What is the expected result?
The view should be found from the following path:
app-dir/theme-dir/abc/views/base-view/xyz.php
What do you get instead?
**View not Found – yii\base\ViewNotFoundException**
The view file does not exist: app-dir/modules/report/views/student/base-view/xyz.php
Additional info

Q
A

Yii version
2.0.23

PHP version
7.1

Operating system
Linux - ubuntu

When tried to give an example with fresh yii2 repo, it worked. It seems to some config issue with my project.


type 'int' is not a subtype of type 'String'  while playing a  video

Is there an existing issue for this?

[X] I have searched the existing issues

Package
youtube_player_flutter
What happened?
I have used youtube_player_flutter for playing youtube videos with videoId(String). There are 200 above videos which are being played, except one Video and that video is giving error type 'int' is not a subtype of type 'String', although the data from api and in my model is STRING only.
What is the expected behaviour?
Video should be able to play while passing the videoId in string.
How to reproduce?
youtube_player_flutter: "8.1.2" with flutter version 3.10.6
Flutter Doctor Output
No response

I have the same issue with trying to play videos by videoId. Once I call play it gives this error and the video won't start

same issue here, only one specific video, I tried using url to id and I tried using the id directly but that specific video wont play

same issue.
How to fix this?

Same error here


v1.18: Use updated branch for curve25519-dalek

Problem
RUSTSEC-2024-0344 was announced so update to a branch that contains the commits that were created in response to the advisory.
We must do this manually as the v1.18 branch is built against curve25519-dalek 3.2.1; this is not the latest major release and the maintainers have chosen not to push changes to their older release branches
Summary of Changes
Update to a branch that contains the zeroize commit, as well as the new security advisory commits

Also, FWIW, the change to ignore the advisory in CI is present in v2.0; it landed before we cut the branch:
https://github.com/anza-xyz/agave/commits/v2.0/ci/do-audit.sh
Technically, master is currently building against a version of ed25519-dalek that does NOT have the commits to address the security advisory. That is, master is built against this branch:
https://github.com/anza-xyz/agave/blob/b97fa99c050df8e1817b63e688d07e8b5c0ac0d9/Cargo.toml#L533-L535
which corresponds to this branch:
https://github.com/anza-xyz/curve25519-dalek/tree/3.2.1-unpin-zeroize
This PR branch is proposing to build v1.18 against this curve25519-dalek branch:
https://github.com/anza-xyz/curve25519-dalek/commits/3.2.1-fix-audit/
The branch here cherry-picked two additional commits, the commits that address the security advisory that we're ignoring

Approved for merging over red CI. It's just the downstream projects check that's failing. See #1960 and #releng discussion for context


test: Updated GPG pub key ID

CoreOS has changed GPG keys for new releases. If we don't use 48F9B96A2E16137F here and there is already imported old GPG key, it will be not updated and GPG check will fail.

LGTM.
Per OOB discussion with @kayrus , I verified like that:
$ curl -s https://coreos.com/security/image-signing-key/CoreOS_Image_Signing_Key.asc \
  | gpg --import --keyid-format LONG
$ gpg --list-keys --keyid-format LONG "CoreOS Buildbot (Offical Builds)"
pub   4096R/50E0885593D2DCB4 2013-09-06
uid                          CoreOS Buildbot (Offical Builds<EMAIL_ADDRESS>sub   4096R/07FA9ED31CB5FA26 2015-08-31 [expires: 2017-08-30]
sub   4096R/48F9B96A2E16137F 2016-05-16 [expires: 2017-05-16]

So here the new GPG key is the last subkey, 48F9B96A2E16137F. Checking only the primary key 50E0885593D2DCB4 is not sufficient. That's why we need to change key in the script to the new subkey.

Merged. Thanks.


Fix the build

The travis build has been failing for 2 reasons recently

pylint takes > 10 minutes without outputing
bad merge confict auto resolve in controls.jsx

Codecov Report

Merging #5403 into master will decrease coverage by 15.76%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master    #5403       +/-   ##
===========================================
- Coverage   77.15%   61.38%   -15.77%
===========================================
  Files          44      373      +329
  Lines        8892    23522    +14630
  Branches        0     2725     +2725
===========================================
+ Hits         6861    14440     +7579
- Misses       2031     9069     +7038
- Partials        0       13       +13

Impacted Files
Coverage Δ

superset/assets/src/explore/controls.jsx
47.32% <ø> (ø)

...c/dashboard/deprecated/v1/components/Dashboard.jsx
0% <0%> (ø)

...ts/src/messageToasts/containers/ToastPresenter.jsx
100% <0%> (ø)

...set/assets/src/SqlLab/components/ColumnElement.jsx
100% <0%> (ø)

...erset/assets/src/explore/actions/exploreActions.js
69.23% <0%> (ø)

...ashboard/components/gridComponents/ChartHolder.jsx
93.18% <0%> (ø)

.../assets/src/visualizations/parallel_coordinates.js
26.31% <0%> (ø)

...et/assets/src/explore/components/controls/index.js
100% <0%> (ø)

...c/dashboard/deprecated/v1/components/CodeModal.jsx
0% <0%> (ø)

...set/assets/src/dashboard/actions/dashboardState.js
41.98% <0%> (ø)

... and 320 more

Continue to review full report at Codecov.

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


revamp Extension and ExtParam traits

The existing code made Descriptor generic over ExtParam rather than Extension. This made the Extension trati basically useless since every Descriptor always had CovenantExt as its extension type.
It also caused me a lot of confusion because I tried to call the address method on a Descriptor<Pk, NoExt>, which I was able to parse, just not call any methods on. I pulled on this string and 1000 lines later here we are.

cc @sanket1729

@apoelstra , the goal of Extension was so that downstream projects can implement their own extensions. ExtParam is a parameter for the data type used in Extension. For example, if CheckSigFromStack were an extension, then

The struct CheckSigFromStack would implement Extension
CsfsMsg and CsfsKey would implement the ExtParam .

It probably might be best to abandon the goal for downstream projects to implement custom extensions. Will offer detailed comments after reviewing this PR

@sanket1729 we can abandon that goal, but then we should drop the Extension trait, the NoExt object, and probably a bunch of other stuff. I can alternately open a PR do that if you want.
But what then will we do for Simplicity? Fork the library a second time?

@apoelstra, l will actually try to write a simple downstream library. Or perhaps an example in this repo itself that implements the extension trait. Will identify a lot of pain points. So far, the PR is looking great. Will comment more after review of this PR in about 30 mins

@apoelstra I was able to create APO + SighashAll extension by implementing Extension in a downstream repo. I think this should do for simplicity translation.


Adds command spo commandset remove, Closes #4333

This PR adds command spo commandset remove.
Closes #4333

ready to merge 🚀

thanks @nicodecleyre for your quick turn around 👍 I will have some 'CLI time' tomorrow so may aim is to merge this one 👍

ready to merge 🚀

merged manually


nodePackages: update to latest versions

Motivation for this change
Update all node packages to the latest version.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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/)
22.05 Release Notes (or backporting 21.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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

@ofborg test thelounge

Might be a duplicated  of #153586. Not sure whey the amount of changed lines differ so much.

Closing, packages were updated at #156173
@lo1tuma I suggest you do the same on #153586, coc-eslint was updated too


GitHub Action for Linting Hyperlink Redirection

Resolves #70
What I Did
I created a GitHub Action that asserts that links to external sites redirect to a new tab using the regular expression defined in #70. Refer to ./.github/actions/lint for the main logic behind the linter. Also check out the latest run where the linter fails because of links that got matched by the regular expression.
What to do once approved

Change all failing links so that linting passes
Remove line 7 in ./.github/workflows/lint.yml

@patrick-5546 The annotations work! Here is the latest action run and its annotations

@patrick-5546 I suggest that we tackle the problem that you mentioned more generally by adding the choice for a config file. That way, users can specify to ignore specific patterns. What do you think?

Can you give an example of what this would look like for the case I outlined?

We would add an ignore pattern for links that match the following regular expression:
https://ubcsailbot.github.io/docs/.*

This would be one ignore pattern that we would add to ignore links back to our site.

I think you misunderstood the case I was trying to outline. Let me explain it in more detail.
By internal links, I mean when you link to another file in the repository using its relative path, for example:
[homepage](../index.md)

With that definition out of the way, here are some error cases that I propose we should add:

External link to site used instead of internal link; for example if we are in docs/:[homepage](https://ubcsailbot.github.io/docs/index.md) -> [homepage](./index.md)

Internal link opens in new tab instead of in the same tab[homepage](./index.md){target=_blank} -> [homepage](./index.md)

This could be configurable

Ignore pattern ike you suggested

Configurability Options

In workflow (easiest if there are only a few options)
https://github.com/UBCSailbot/docs/blob/fe56aa3b940bddfb6f9d36ae789664494e374e4d/.github/workflows/lint.yml#L20-L21
Configuration file

When deciding whether to implement these additional error cases, please consider whether it is worth the effort.

Let's make the pattern configuration into a separate issue because this one is already getting very large.


[Discussion] Use git submodule to fetch valid and invalid CycloneDX BOM tests files from the github.com/cyclonedx/specification repo

As of today, I noticed cyclonedx-go/testdata only provides valid-* sample test BOM files in XML and JSON. It provides no  invalid-* files, and no protobuf files.
$ git clone https://github.com/CycloneDX/cyclonedx-go.git
$ cd cyclonedx-go/testdata
$ broot --cmd ":pt"

cyclonedx/cyclonedx-go/testdata
 ├──snapshots …
 ├──valid-annotation.json
 ├──valid-annotation.xml
 ├──valid-assembly.json
 ├──valid-assembly.xml
 ├──valid-bom.json
 ├──valid-bom.xml
 ├──valid-component-hashes.json
 ├──valid-component-hashes.xml
 ├──valid-component-omniborId.json
 ├──valid-component-omniborId.xml
 ├──valid-component-ref.json
 ├──valid-component-ref.xml
[...]

And files are not sorted by CycloneDX specification version, which makes it
harder to list and to maintain.
But I noticed the github.com/CycloneDX/specification repository provides both
valid-* and invalid-* BOM sample files in XML, JSON and PROTOBUF formats.
These files are useful to tests implementation of the CycloneDX, such as
github.com/CycloneDX/cyclonedx-go.
$ git clone https://github.com/CycloneDX/specification.git
$ cd specification/tools/src/test/resources
$ broot --cmd ":pt"    # https://dystroy.org/broot/documentation/usage/#export-a-tree

cyclonedx/specification/tools/src/test/resources
 ├──1.0
 │  └──2 unlisted
 ├──1.1
 │  ├──invalid-component-ref-1.1.xml
 │  ├──invalid-component-type-1.1.xml
 │  ├──invalid-empty-component-1.1.xml
 │  ├──invalid-hash-alg-1.1.xml
 │  ├──invalid-hash-md5-1.1.xml
 │  ├──valid-bom-1.1.xml
 │  └──23 unlisted
 ├──1.2
 │  ├──invalid-bomformat-1.2.json
 │  ├──invalid-component-ref-1.2.json
 │  ├──invalid-component-ref-1.2.xml
 │  ├──invalid-component-swid-1.2.json
 │  ├──invalid-component-swid-1.2.xml
 │  ├──valid-assembly-1.2.json
 │  ├──valid-assembly-1.2.xml
 │  └──82 unlisted
 ├──1.3
 │  ├──invalid-bomformat-1.3.json
 │  ├──invalid-component-ref-1.3.json
 │  ├──invalid-component-ref-1.3.xml
 │  ├──invalid-component-swid-1.3.json
 │  ├──valid-assembly-1.3.json
 │  ├──valid-assembly-1.3.xml
 │  └──122 unlisted
 ├──1.4
 │  ├──invalid-bomformat-1.4.json
 │  ├──invalid-component-ref-1.4.json
 │  ├──invalid-component-ref-1.4.xml
 │  ├──invalid-component-swid-1.4.json
 │  ├──valid-assembly-1.4.json
 │  └──129 unlisted
 ├──1.5
 │  ├──invalid-bomformat-1.5.json
 │  ├──invalid-component-ref-1.5.json
 │  ├──invalid-component-ref-1.5.xml
 │  ├──invalid-component-swid-1.5.json
 │  ├──valid-annotation-1.5.json
 │  ├──valid-annotation-1.5.xml
 │  ├──valid-annotation-1.5.textproto
 │  └──149 unlisted
 ├──1.6
 │  ├──invalid-bomformat-1.6.json
 │  ├──invalid-component-ref-1.6.json
 │  ├──invalid-component-ref-1.6.xml
 │  ├──invalid-component-swid-1.6.json
 │  ├──valid-component-hashes-1.6.json
 │  ├──valid-component-hashes-1.6.xml
 │  ├──valid-component-ref-1.6.textproto
 │  └──178 unlisted
 └──ext
    ├──invalid-depgraph-1.0.xml
    ├──valid-component-depgraph-1.0.xml
    └──valid-depgraph-1.0.xml

I suggest using a git submodule to fetch tests files from the CycloneDX specification repository.
git clone https://github.com/CycloneDX/cyclonedx-go
cd cyclonedx-go
git submodule add -f https://github.com/CycloneDX/specification specification

This would give access to the full list of valid-* and invalid-* BOMs files
in *.xml, *.json and *.textproto with no efforts.
This would be available at this path:
cyclonedx-go/specification/tools/src/test/resources.
And if there are tests files dedicated to one particular language (Go, python, or Java),
then either keep a folder for custom test files, or contribute to the tests files in the
CycloneDX/specification repo.
All CycloneDX Implementation Python, Golang, Java could do the same
The same principle could be applied to any CycloneDX implementation:

github.com/CycloneDX/cyclonedx-go
github.com/CycloneDX/cyclonedx-core-java
github.com/CycloneDX/cyclonedx-python-lib

                    ┌────────────────────────────────────────────────────┐
                    │  github.com/CycloneDX/specification commit 59e7d88 │
                    │      └── tools/src/test/resources (local)          │
                    └─────┬─────────────────┬───────────────────┬────────┘
                          │                 │                   │
                 fetch submodule    fetch submodule      fetch submodule
                          │                 │                   │
 ┌────────────────────────▼───────────────┐ │                   │
 │ github.com/CycloneDX/cyclonedx-go      │ │                   │
 │ └── specification @59e7d88 (submodule) │ │                   │
 └────────────────────────────────────────┘ │                   │
                                            │                   │
                  ┌─────────────────────────▼────────────────┐  │
                  │ github.com/CycloneDX/cyclonedx-core-java │  │
                  │  └── specification @59e7d88 (submodule)  │  │
                  └──────────────────────────────────────────┘  │
                                                                │
                                ┌───────────────────────────────▼──────────┐
                                │github.com/CycloneDX/cyclonedx-python-lib │
                                │   └── specification @59e7d88 (submodule) │
                                └──────────────────────────────────────────┘

Use the git submodule for Schema Files as well?
Once the git submodules for CycloneDX/specification are available in both
cyclonedx-go, cyclonedx-core-java and cyclonedx-python-lib to fetch
test files, this could also be used for fetching schema files (JSON Schema,
XSD and protobuf).
This would prevent issues such as this one where I noticed CycloneDX schema
files are different from one CycloneDX implementation to another.
https://github.com/CycloneDX/specification/pull/479#issuecomment-2180940956

Note: I start this discussion for the cyclonedx-go project. But we could
open the same discussion in others CycloneDX implementations (python and
java).

CC @jkowalleck & @nscuro who helped me figure out how to explain this idea :smile:

@Nicolas-Peiffer Do you know how well Git submodules play with Go modules? For example, consider we want to embed schema files from the submodule via //go:embed. Will consumers of cyclonedx-go still be able to use the library (because Go automatically clones submodules), or will this cause issues (because Go doesn't automatically clone submodules)?

@nscuro you make a very good point that I completely missed. Thank you very much for this :+1:  .
Indeed, after a quick search with ChatGPT4 and the web, I came to the conclusion that GoLang packages on Github.com does not support git submodules natively.
There is an open issue since 2016 somehow similar to this topic:
https://github.com/golang/go/issues/17522
And the workarounds based on git clone --recurse-submodules [...] are not satisfactory, since they are not supported by go install and go get.
Fetching files and commit them instead of using a submodule?
I think the only viable short term solution (outside of patching GoLang to add support for git submodules) it to mimic what is already done on the python implementation:

test files: https://github.com/CycloneDX/cyclonedx-python-lib/tree/main/tests/_data/schemaTestData

fetching tests files from github.com/CycloneDX/specification with a bash script tests/_data/schemaTestData/fetch.sh ;
adding complementary tests files under tests/_data/own;

schema: https://github.com/CycloneDX/cyclonedx-python-lib/tree/main/cyclonedx/schema/_res

What do you think?


propose mapping to search for keyword

I think h can be used for :Help in python files, there's still :h[elp] and :tab h.
Propose a mapping for faster lookup.

I don’t have a preference. Maybe “PyHelp”
On Wed, Oct 23, 2024 at 5:28 PM Enno @.***> wrote:

I think h can be used for :Help in python files, there's still :h[elp]
and :tab h.
Propose a mapping for faster lookup.
You can view, comment on, or merge this pull request online at:
https://github.com/girishji/pythondoc.vim/pull/3
Commit Summary

a2dd6ac
https://github.com/girishji/pythondoc.vim/pull/3/commits/a2dd6ac9a26c1c432aafc716c0447ff2a482ea11
propose mapping to search for keyword

File Changes
(2 files https://github.com/girishji/pythondoc.vim/pull/3/files)

M README.md
https://github.com/girishji/pythondoc.vim/pull/3/files#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5
(12)
M ftplugin/python_doc.vim
https://github.com/girishji/pythondoc.vim/pull/3/files#diff-c564554cd8b905992e7cc56e379b3a052ad716b094af358602bc522e2afc1068
(9)

Patch Links:

https://github.com/girishji/pythondoc.vim/pull/3.patch
https://github.com/girishji/pythondoc.vim/pull/3.diff

—
Reply to this email directly, view it on GitHub
https://github.com/girishji/pythondoc.vim/pull/3, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD6DOYASWPU5KPQHNOORKGLZ466AXAVCNFSM6AAAAABQPDEEBOVHI2DSMVQWIX3LMV43ASLTON2WKOZSGYYDSMBSGM2TENA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

There's  a missunderstanding as I find a filetype local :Help for Python not that bad, since one rarely searches for Vim in Python files and, if so, can still use, say :he. This PR is mainly about daring :h and a mapping :dK.


Fix documentation about unused variables

Some environment variables are not used:

PASV_ADDRESS_RESOLVE: see PASV_ADDR_RESOLVE
PASV_ADDRESS_ENABLE: see PASV_ENABLE

Note : default values are different between these variables and the ones which are used

Hello @jibidus , thanks for noticing this.
The actual variables used are PASV_ENABLE and PASV_ADDR_RESOLVE, updated README.md.
Thanks!


Thread profiling screen, selected thread color similar to other color

TIMED_WAITING, when pressed, has a very similar color to WAITING

Even less distinguishable when contrasted by unpressed TIMED_WAITING (Above is WAITING, below is pressed TIMED_WAITING


Lotus should works without i18n

Is your feature request related to a problem? Please describe.
Actually Lotus is not really ready to work on a non i18n project. More precisely, it is almost required to be mount on a i18n url pattern.
Describe the solution you'd like
Not everyone needs to use translated article or to serve different languages. Although it is a major feature from Lotus, it may be expected to be used without using i18n on url pattern or else.
I don't know yet how to do this technically, since all querysets require a language prefix, something will need to simulate the language middleware which set language to request context and just use the defaut language from settings.
Also, a settings could exists to disable the translation behavior like to hide the "original" field in admin.
Describe alternatives you've considered
Not  any.

Done in 0.5.3-pre.3, views have been changed so they use the language code from either request if it have related attribute LANGUAGE_CODE, else fallback on settings.LANGUAGE_CODE. This work automatically so no need to configure anything special.
However, the language is still showing from admin forms.


Node server to node cluster

Replace wording about nodeServer with nodeCluster.

Fixes issue #502.

Hi John, @John-A-Davies
The following occurrences were also found. Wondering if they should also be replaced. Could you take a look?
cc @1000TurquoisePogs

docs-site\docs\extend\extend-desktop\mvd-dataservices.md (1 hit)

Line 71: An example is available in sample-app/nodeServer/ts/helloWorld.ts

docs-site\docs\extend\extend-desktop\mvd-plugindefandstruct.md (4 hits)

Line 24: #### nodeServer
Line 26: When an application plug-in has router-type dataservices, they are interpreted by the Zowe Application Server by attaching them as ExpressJS routers. It is recommended that you write application plug-ins using Typescript, because it facilitates well-structured code. Use of Typescript results in build steps because the pre-transpilation Typescript content is not to be consumed by NodeJS. Therefore, keep server-side source code in the nodeServer directory. At runtime, the server loads router dataservices from the lib directory.
Line 38: The lib directory is where router-type dataservices are loaded by use in the Zowe Application Server. If the JS files that are loaded from the lib directory require NodeJS modules, which are not provided by the server base (the modules zlux-server-framework requires are added to NODE_PATH at runtime), then you must include these modules in lib/node_modules for local directory lookup or ensure that they are found on the NODE_PATH environment variable. nodeServer/node_modules is not automatically accessed at runtime because it is a dev and build directory.
Line 54: location: This is a directory path that is relative to the server's executable (such as zlux-app-server/bin/nodeServer.sh) at which a pluginDefinition.json file is expected to be found.

C:\GitFolder\docs-site\docs\user-guide\mvd-configuration.md (1 hit)

Line 407: - Zowe Application Server: zlux-app-server/log/nodeServer-yyyy-mm-dd-hh-mm.log

Yes, all these instances of nodeServer should be retained and not changed to nodeCluster.


[Funimation] Dont download fragments with aria2c

Checklist

[X] I'm reporting a bug unrelated to a specific site
[X] I've verified that I'm running yt-dlp version 2021.12.01. (update instructions)
[X] I've checked that all provided URLs are alive and playable in a browser
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Description
I implemented the aria2c downloader to my configuration file to download videos from funimation and the following happened.
The subtitle files I downloaded were downloaded with aria2c. But after that the video fragments are downloaded with hlsnative even though I set yt-dlp to download with aria2c.
Note: In the verbose log I cancelled the download, just to show you the result.
My config file
#Video Format Option
-f "(bv[height<=720]+ba/b[height<=720])[format_note*=Uncut]/
(bv[height<=720]+ba/b[height<=720])[format_note*=Simulcast]"

#Subtitles Options for Anime Video
--sub-lang all,-en_Uncut_CC,-en_CC,-pt,-pt_Uncut --embed-subs --sub-format srt

#Output Template
-o "%(extractor_key)s/%(series)s/%(series)s S%(season_number)sE%(episode_number)s - %(episode)s.%(ext)s"

#Cookies for Info User
--cookies cookies-funimation-com.txt -n

#Extractor Options
--extractor-args funimation:language=japanese

#More Options to yt-dlp
--add-metadata -v

#Downloader Options
--downloader aria2c

Verbose log
C:\youtube-dl>yt-dlp --config-location funimation.conf https://www.funimation.com/v/mushoku-tensei-jobless-reincarnation-p2/the-birth-of-my-little-sister-the-maid
[debug] Command-line config: ['--config-location', 'funimation.conf', 'https://www.funimation.com/v/mushoku-tensei-jobless-reincarnation-p2/the-birth-of-my-little-sister-the-maid']
[debug] Custom config file: funimation.conf
[debug] Custom config: ['-f', '(bv[height<=720]+ba/b[height<=720])[format_note*=Uncut]/\n(bv[height<=720]+ba/b[height<=720])[format_note*=Simulcast]', '--sub-lang', 'all,-en_Uncut_CC,-en_CC,-pt,-pt_Uncut', '--embed-subs', '--sub-format', 'srt', '-o', '%(extractor_key)s/%(series)s/%(series)s S%(season_number)sE%(episode_number)s - %(episode)s.%(ext)s', '--cookies', 'cookies-funimation-com.txt', '-n', '--extractor-args', 'funimation:language=japanese', '--add-metadata', '-v', '--downloader', 'aria2c']
[debug] Encodings: locale cp1252, fs utf-8, out utf-8, err utf-8, pref cp1252
[debug] yt-dlp version 2021.12.01 [91f071af6] (source)
[debug] Lazy loading extractors is disabled
[debug] Plugins: ['SamplePluginIE', 'SamplePluginPP']
[debug] Git HEAD: e3c7d4957
[debug] Python version 3.9.7 (CPython 64bit) - Windows-10-10.0.19042-SP0
[debug] exe versions: ffmpeg n4.4.1-1-gb5b784d5ad-20211031 (setts), ffprobe n4.4.1-1-gb5b784d5ad-20211031
[debug] Optional libraries: Cryptodome, mutagen, sqlite, websockets
[debug] Proxy map: {}
[funimation:page] Logging in
[debug] [funimation:page] Extracting URL: https://www.funimation.com/v/mushoku-tensei-jobless-reincarnation-p2/the-birth-of-my-little-sister-the-maid
[funimation:page] mushoku-tensei-jobless-reincarnation-p2_the-birth-of-my-little-sister-the-maid: Downloading JSON metadata
[debug] [Funimation] Extracting URL: https://www.funimation.com/player/3982661
[Funimation] 3982661: Downloading player webpage for 3982661
[Funimation] the-birth-of-my-little-sister-the-maid: Downloading Simulcast japanese (3982661) JSON
[Funimation] the-birth-of-my-little-sister-the-maid: Downloading Simulcast japanese (3982661) m3u8 information
[debug] Sort order given by extractor: lang, source
[debug] Formats sorted by: hasvid, ie_pref, lang, source, quality, res, fps, hdr:12(7), vcodec:vp9.2(10), acodec, filesize, fs_approx, tbr, vbr, abr, asr, proto, vext, aext, hasaud, id
[debug] Downloading subtitles: en, es
[info] 2637663: Downloading 1 format(s): 3982661-hls-3858+3982661-hls-audio-aacl-256-Japanese
[info] Writing video subtitles to: Funimation\Mushoku Tensei - Jobless Reincarnation\Mushoku Tensei - Jobless Reincarnation S2E20 - The Birth of My Little Sister, the Maid.en.srt
[debug] Invoking downloader on "https://d33et77evd9bgg.cloudfront.net/FunimationStoreFront/V2593209/d2e2adbc-9ddd-476a-9b8f-22f32e81503e_english_full.srt"
[download] Destination: Funimation\Mushoku Tensei - Jobless Reincarnation\Mushoku Tensei - Jobless Reincarnation S2E20 - The Birth of My Little Sister, the Maid.en.srt
[debug] aria2c command line: aria2c -c "--console-log-level=warn" "--summary-interval=0" "--download-result=hide" "--file-allocation=none" -x16 -j16 -s16 --min-split-size 1M --header "User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/72.0.3626.114 Safari/537.36" --header "Accept-Charset: ISO-8859-1,utf-8;q=0.7,*;q=0.7" --header "Accept: text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8" --header "Accept-Encoding: gzip, deflate" --header "Accept-Language: en-us,en;q=0.5" "--check-certificate=true" "--remote-time=true" --dir ".\Funimation\Mushoku Tensei - Jobless Reincarnation\" --out ".\Mushoku Tensei - Jobless Reincarnation S2E20 - The Birth of My Little Sister, the Maid.en.srt.part" "--auto-file-renaming=false" -- "https://d33et77evd9bgg.cloudfront.net/FunimationStoreFront/V2593209/d2e2adbc-9ddd-476a-9b8f-22f32e81503e_english_full.srt"
[aria2c] Downloaded 22020 bytes
[download] 100% of 21.50KiB in 00:01
[info] Writing video subtitles to: Funimation\Mushoku Tensei - Jobless Reincarnation\Mushoku Tensei - Jobless Reincarnation S2E20 - The Birth of My Little Sister, the Maid.es.srt
[debug] Invoking downloader on "https://d33et77evd9bgg.cloudfront.net/FunimationStoreFront/V2593209/d2e2adbc-9ddd-476a-9b8f-22f32e81503e_spanishlatinamerica_full.srt"
[download] Destination: Funimation\Mushoku Tensei - Jobless Reincarnation\Mushoku Tensei - Jobless Reincarnation S2E20 - The Birth of My Little Sister, the Maid.es.srt
[debug] aria2c command line: aria2c -c "--console-log-level=warn" "--summary-interval=0" "--download-result=hide" "--file-allocation=none" -x16 -j16 -s16 --min-split-size 1M --header "User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/72.0.3626.114 Safari/537.36" --header "Accept-Charset: ISO-8859-1,utf-8;q=0.7,*;q=0.7" --header "Accept: text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8" --header "Accept-Encoding: gzip, deflate" --header "Accept-Language: en-us,en;q=0.5" "--check-certificate=true" "--remote-time=true" --dir ".\Funimation\Mushoku Tensei - Jobless Reincarnation\" --out ".\Mushoku Tensei - Jobless Reincarnation S2E20 - The Birth of My Little Sister, the Maid.es.srt.part" "--auto-file-renaming=false" -- "https://d33et77evd9bgg.cloudfront.net/FunimationStoreFront/V2593209/d2e2adbc-9ddd-476a-9b8f-22f32e81503e_spanishlatinamerica_full.srt"
[aria2c] Downloaded 21557 bytes
[download] 100% of 21.05KiB in 00:06
[debug] Invoking downloader on "https://vmfst-api.prd.funimationsvc.com/FunimationStoreFront/V2593209/d2e2adbc-9ddd-476a-9b8f-22f32e81503estrnv-hl152_streaming_video_1280_720_4500000_index.m3u8?Key-Pair-Id=APKAIHNXECY27H4O6NIA&Policy=eyJTdGF0ZW1lbnQiOlt7IlJlc291cmNlIjoiaHR0cHM6Ly9kMzNldDc3ZXZkOWJnZy5jbG91ZGZyb250Lm5ldC9GdW5pbWF0aW9uU3RvcmVGcm9udC9WMjU5MzIwOS8qIiwiQ29uZGl0aW9uIjp7IkRhdGVMZXNzVGhhbiI6eyJBV1M6RXBvY2hUaW1lIjoxNjM5MTcwNTU5fX19XX0_&Signature=L8JR3NU4-cEA0i8X-4W3UlXn2U3O0b3oELsfHfj3Z4I0F5B3dJz~bNZjN5TV2ZmdZe5MCG-ioKupT~wFoOvtD1jHkb7yHRad06y0eb2iICSGSaSypROLgSYXkWAwHh775kfVHAWj9oi0Fnswtd8XmGsBJ6boUN2-GcJTHcdHcY~U8TldSnYGXJu6qLhFQITDM9G8rQppZTD7IZkRojwiQU-FzvFKVl9e5EYAUWD5zPj~lzJ9jFkF7qBZxDhwKI5W6AG0mmLSZCp5EHpHR2npd9meQZNFSP~Y4rzwgbUL4wKTFN3syADcJAX7KPFfLLDPPYJsPyLdy~kAn5FgNK6-aA__&rt=1420044"
[hlsnative] Downloading m3u8 manifest
[hlsnative] Total fragments: 713
[download] Destination: Funimation\Mushoku Tensei - Jobless Reincarnation\Mushoku Tensei - Jobless Reincarnation S2E20 - The Birth of My Little Sister, the Maid.f3982661-hls-3858.mp4
[download]   0.3% of ~926.80MiB at 124.98KiB/s ETA 02:06:16 (frag 2/713)
ERROR: Interrupted by user

Aria2c does not support downloads with custom byterange. See #1507


Adds bulk code generation to staff-dashboard

Pre-Flight checklist

[X] Screenshots and design review for any changes that affect layout or styling

[X] Desktop screenshots

[X] Migrations

[X] Migration is backwards-compatible with current production code

[X] Testing

[X] Code is tested
[X] Changes have been manually tested

What are the relevant tickets?
Closes #1623
What's this PR do?

Ups the maximum discount code length to 100 characters (since bulk codes use UUIDs, we were left with 13 characters for the bulk code prefix, which has been sort of a problem)
Refactors bulk code generation into api.py
Adds a Create Bulk Discount form to the staff dashboard to allow staff users to create codes on their own

The form works like the existing Create Discount form, with a couple of small modifications to support bulk generation
Generated codes are displayed after creation and the option is given to download the codes as a CSV file

How should this be manually tested?
Note that both your watch and refine containers should probably be restarted for testing this as there are new JavaScript dependencies.

Load the staff dashboard and go to Discounts.
Click the Bulk Create button.
Fill out the form appropriately.
4. Note that the prefix does not automatically append a separator between it and the code itself, so if you want one (our convention is to use a -) you need to specify it explicitly.
5. If you fill out the date fields, make sure to also specify a time. Times are in UTC.
Click Save to create.
You should see the list of discount codes.
Click Download CSV to download the generated codes in a CSV file.
The CSV file should be correct.

Screenshots (if appropriate)

(I changed the button title from Create Batch to Bulk Create for consistency but did not redo the screenshot.)

Added some validation rules in https://github.com/mitodl/mitxonline/pull/1631/commits/6b48b1e037f30da518f2ce46ce85c96842e5ca10 . I updated the discount code count field to require a minimum of 2 - if you're only making 1 you should just be using the regular form, I think; it wasn't worth refactoring the api for this to accommodate it.


Use pkg:lints and pkg:flutter_lints when analyzing code

To promote good coding practices and for a consistent experience across the dart ecosystem, dartpad should use the analyzer options defined in the new flutter_lints and lints packages when analyzing Flutter code or pure Dart code, respectively.
/cc @mit-mit @pq @devoncarew

For Dart code please also use recommended.yaml (dart create has been updated).

Being worked on in https://github.com/dart-lang/dart-services/pull/722


Broken Link - ICETransformer Example

Describe the bug
In these release notes, you have this statement: "Learn about how to use the ICETransformer through an example with the Adult Census dataset". I get a "Page not Found" from that link. I also don't see an ICE Explainer when I manually navigate to the Interpretability section of the docs.
To Reproduce
Click the link in the release notes
Expected behavior
The link should take users to some example page.
Info (please complete the following information):
N/A
** Stacktrace**
N/A
If the bug pertains to a specific feature please tag the appropriate CODEOWNER for better visibility
Additional context
N/A

Thank you its here and updating the release notes:
https://microsoft.github.io/SynapseML/docs/next/features/responsible_ai/Interpretability - PDP and ICE explainer/


Empty type field in VariableDefinition

Description
For a given apollo_parser::ast::VariableDefinition v, calling v.ty() returns None.
Steps to reproduce
Execute the following program:
use apollo_parser::Parser;
const QUERY: &str = r#"
query MatchupAnalysisQuery(
    $homeTeamAbbrev: String!
    $awayTeamAbbrev: String!
    $sportType: GameSportTypeEnumType!
    $poolId: ID
    $weekNumber: Int
    $gameInstanceUid: String
  ) {
    matchupAnalysis(
      homeTeamAbbrev: $homeTeamAbbrev
      awayTeamAbbrev: $awayTeamAbbrev
      sportType: $sportType
      poolId: $poolId
      weekNumber: $weekNumber
      gameInstanceUid: $gameInstanceUid
    ) {
      id
    }
}"#;
fn main() {
    let parser = Parser::new(QUERY);
    let ast = parser.parse();
    assert!(&ast.errors().is_empty());
    let doc = ast.document();
    for def in doc.definitions() {
        match def {
            apollo_parser::ast::Definition::OperationDefinition(operation) => {
                let vars = operation.variable_definitions().unwrap();
                let fmt = format!("{}", &vars);
                dbg!(&vars, fmt);
                for var in vars.variable_definitions().into_iter() {
                    let name = var.variable().unwrap().name().unwrap().to_string();
                    let ty = match var.ty() {
                        Some(ty) => ty.to_string(),
                        None => "NO TYPE".to_owned()
                    };
                    println!("{}: {}", name, ty);
                }
            },
            _ => unimplemented!()
        }
    }
}

Expected result
Output:
homeTeamAbbrev: String!
awayTeamAbbrev: String!
sportType: GameSportEnumType!
poolId: ID
weekNumber: Int
gameInstanceUid: String

Actual result
Output:
homeTeamAbbrev: NO TYPE
awayTeamAbbrev: NO TYPE
sportType: NO TYPE
poolId: NO TYPE
weekNumber: NO TYPE
gameInstanceUid: NO TYPE

Environment

Operating system and version: MacOS BigSur Version 11.6
Shell (bash/zsh/powershell): zsh
apollo-rs crate: ??
Crate version: apollo-parser 0.1.0

@kpeters-cbsi ah! let me publish a new version for you -- we fixed this issue in #127, but it just hasn't been published.

@kpeters-cbsi I just published a new version, 0.2.0. It has a fix for the problem you're running into above.
The release comes with a breaking change - ast.errors() now returns an Iterator - so you will have to change your errors assert to assert!(ast.errors().len(), 0). You can read up on it in more detail in the Changelog.
Let me know if you're still running into trouble!


Add THead to continue row header on next print page With Firefox

Hi,
we've had a problem with pivottable using Firefox's print: If there was a pivottable whose print had several pages, we had the effect that the row header was not rendered on the next page.

We were able to fix this problem by changing Pivot.js the following way:
@@ -737,7 +737,7 @@
     Default Renderer for hierarchical table layout
      */
     pivotTableRenderer = function(pivotData, opts) {
-      var aggregator, c, colAttrs, colKey, colKeys, defaults, i, j, r, result, rowAttrs, rowKey, rowKeys, spanSize, td, th, totalAggregator, tr, txt, val, x;
+      var aggregator, c, colAttrs, colKey, colKeys, defaults, i, j, r, result, rowAttrs, rowKey, rowKeys, spanSize, td, th, thead, totalAggregator, tr, txt, val, x;
       defaults = {
         localeStrings: {
           totals: "Totals"
@@ -778,6 +778,7 @@
         }
         return len;
       };
+      thead = document.createElement("thead");
       for (j in colAttrs) {
         if (!__hasProp.call(colAttrs, j)) continue;
         c = colAttrs[j];
@@ -814,7 +815,7 @@
           th.setAttribute("rowspan", colAttrs.length + (rowAttrs.length === 0 ? 0 : 1));
           tr.appendChild(th);
         }
-        result.appendChild(tr);
+        thead.appendChild(tr);
       }
       if (rowAttrs.length !== 0) {
         tr = document.createElement("tr");
@@ -832,8 +833,9 @@
           th.innerHTML = opts.localeStrings.totals;
         }
         tr.appendChild(th);
-        result.appendChild(tr);
+        thead.appendChild(tr);
       }
+      result.appendChild(thead);
       for (i in rowKeys) {
         if (!__hasProp.call(rowKeys, i)) continue;
         rowKey = rowKeys[i];

I hope this helps you to improve your great tool.
Regards,
Mathias

Thanks for the contribution! I'll see about merging it in :)

Merged in in another PR.


srb init fails with an existing rails app when generating symbol table

I've solved all the obvious issues with my project, and now I've been stumped for a while:
Generating /tmp/d20190710-5951-1ohisfh/reflection.rbi with 8423 modules and 141 aliases
Printing your code's symbol table into /tmp/d20190710-5951-1ohisfh/from-source.json
/home/docker/.gem/ruby/2.4.0/gems/sorbet-0.4.4413/lib/hidden-definition-finder.rb:117:in `write_constants': Your source can't be read by Sorbet. (RuntimeError)
You can try `find . -type f | xargs -L 1 -t bundle exec srb tc --no-config --error-white-list 1000` and hopefully the last file it is processing before it dies is the culprit.
If not, maybe the errors in this file will help: /tmp/d20190711-155-11qm7ib/from-source.json.err
	from /home/docker/.gem/ruby/2.4.0/gems/sorbet-0.4.4426/lib/hidden-definition-finder.rb:47:in `main'
	from /home/docker/.gem/ruby/2.4.0/gems/sorbet-0.4.4426/lib/hidden-definition-finder.rb:38:in `main'
	from /home/docker/.gem/ruby/2.4.0/gems/sorbet-0.4.4426/bin/srb-rbi:229:in `block in make_step'
	from /home/docker/.gem/ruby/2.4.0/gems/sorbet-0.4.4426/bin/srb-rbi:121:in `init'
	from /home/docker/.gem/ruby/2.4.0/gems/sorbet-0.4.4426/bin/srb-rbi:196:in `main'
	from /home/docker/.gem/ruby/2.4.0/gems/sorbet-0.4.4426/bin/srb-rbi:234:in `<main>'

I've tried running the suggested command (find . -type f | xargs -L 1 -t bundle exec srb tc --no-config --error-white-list 1000). No individual bundle exec srb tc exits with errors:
docker@11b635da953a:/usr/src/app$ find . -type f | xargs -L 1 -t bundle exec srb tc --no-config --error-white-list 1000 2>&1 | grep -v 'No errors' | grep -v 'bundle exec'
docker@11b635da953a:/usr/src/app$

Here's the output of from-source.json.err:
You must pass either `-e` or at least one folder or ruby file.

Typechecker for Ruby
Usage:
  sorbet [OPTION...] <path 1> <path 2> ...

  -e, string     Parse an inline ruby string (default: )
  -q, --quiet    Silence all non-critical errors
  -v, --verbose  Verbosity level [0-3]
  -h,            Show short help
      --help     Show long help
      --version  Show version

I've traced the failure down to this file. What I don't understand is this command that's getting forked:
File.realpath("#{__dir__}/../bin/srb"),
        'tc',
        '--print=symbol-table-full-json',
        '--stdout-hup-hack',
        '--silence-dev-message',
        '--no-error-count',

Running this command in a known good environment works. But with our rails application, the same command fails. It insists I pass in a path or -e expression to type check, per the error message above. Is it somehow looking for something that's missing in this project?
I've been dead in the water for a while, and feel like I'm missing something obvious.

What is your OS and hardware versions?

Tell us what is in this file: /tmp/d20190711-155-11qm7ib/from-source.json.err

This is the condition that was true to trigger the You must pass either -e or at least one folder or ruby file. message:
if (raw.count("e") == 0 && opts.inputFileNames.empty() && !opts.runLSP && opts.storeState.empty()) {

What happens when you run "srb" ?

Also what version of Rails?

What is your OS and hardware versions?

This is on Dinghy, a docker OSX wrapper.

I get the same error message as before:
docker@9de2da163cf3:/usr/src/app$ bundle exec srb
You must pass either `-e` or at least one folder or ruby file.

Typechecker for Ruby
Usage:
  sorbet [OPTION...] <path 1> <path 2> ...

  -e, string     Parse an inline ruby string (default: )
  -q, --quiet    Silence all non-critical errors
  -v, --verbose  Verbosity level [0-3]
  -h,            Show short help
      --help     Show long help
      --version  Show version

Also what version of Rails?

docker@9de2da163cf3:/usr/src/app$ ruby -v
ruby 2.4.6p354 (2019-04-01 revision 67394) [x86_64-linux-gnu]
docker@9de2da163cf3:/usr/src/app$ grep 'rails' Gemfile.lock
    rails (<IP_ADDRESS>)

Nothing yells at me so my next step if it was mine would be to add a few put's before the if line:
puts "1: #{raw.count("e")}"
puts "2: #{opts.opts.inputFileNames.empty() }"
puts "3: #{opts.runLSP}"
puts; "4: #{opts.storeState.empty()}"
if (raw.count("e") == 0 && opts.inputFileNames.empty() && !opts.runLSP && opts.storeState.empty()) {

to back into what is the root cause. Yes, I know I am modifying the code but I will remove it after the experiment.

It looks like that file is C++, so I don't think ruby would work in there.
Looking through the rest of the file, it's hard to imagine why the command exits early in "clean" environments. There's no -e flag, there's no input list of files, the command is not being run in language server protocol mode, and --store-state is not getting passed in.
I'm not exactly sure how much is involved in building the srb binary, but that seems to be getting a bit too far into the weeds...

I agree. Sorry for misleading suggestion.

I had this issue when I try to run bundle exec srb rbi hidden-definitions to update my hidden-definitions, but as I was writing this, apparently updating sorbet-rails fixed the problem 🤷♂

We're also seeing this with developers running Dinghy + xhyve - haven't had a chance to debug it properly yet.

Same issue and the suggested command prints a bunch of errors:
~/s/socializus> find . -type f | xargs -L 1 -t bundle exec srb tc --no-config --isolate-error-code 1000
bundle exec srb tc --no-config --isolate-error-code 1000 ./sorbet/config
ERROR:  While executing gem ... (OptionParser::InvalidArgument)
    invalid argument: -v
/Users/dorianmariefr/.rvm/gems/ruby-2.7.2/bundler/gems/sorbet-548807710587/gems/sorbet/bin/srb: line 109: /libexec/sorbet: No such file or directory
bundle exec srb tc --no-config --isolate-error-code 1000 ./sorbet/rbi/gems/rb-fsevent.rbi
ERROR:  While executing gem ... (OptionParser::InvalidArgument)
    invalid argument: -v
/Users/dorianmariefr/.rvm/gems/ruby-2.7.2/bundler/gems/sorbet-548807710587/gems/sorbet/bin/srb: line 109: /libexec/sorbet: No such file or directory
bundle exec srb tc --no-config --isolate-error-code 1000 ./sorbet/rbi/gems/easy_translate.rbi
ERROR:  While executing gem ... (OptionParser::InvalidArgument)
    invalid argument: -v
/Users/dorianmariefr/.rvm/gems/ruby-2.7.2/bundler/gems/sorbet-548807710587/gems/sorbet/bin/srb: line 109: /libexec/sorbet: No such file or directory
bundle exec srb tc --no-config --isolate-error-code 1000 ./sorbet/rbi/gems/activerecord.rbi

Going to close this in favor of Tapioca
https://sorbet.org/blog/2022/07/27/srb-tapioca


Attribute 'value' in annotation @RestController is declared as an @AliasFor nonexistent attribute 'value' in annotation @Controller

Hi, I am currently trying to build native image for one of my microservices as spring cloud client.
Attached  pom, bootstrap.yml, native-image/config jsons, and startup log. Liquibase and log4j was added to pom to remove some warning. Some config values was also removed for privacy purposes
In my configuration, I only included @EnableResourceServer, @EnableSwagger2, @SpringBootApplication, and @EnableFeignClients.
I ran my config server in jvm due to limitation of PostEnvironmentProcessor preventing the config server from starting. ProfileApplication was able to start normally and load the config from config server. But, After loading config, I got this error:
Attribute 'value' in annotation @RestController is declared as an @AliasFor nonexistent attribute 'value' in annotation @Controller
Note: image was build on wsl ubuntu focal
profile.zip

I suspect it is the processing we do not operating on some of the controllers found inherited from dependencies. Which dependency it is without a sample project, not quite sure. You could probably workaround it by manually creating an entry in src/main/resources/META-INF/native-image/reflect-config.json
looking like:
[
{"name":"org.springframework.stereotype.Controller","allDeclaredMethods":"true"}
]

I will give it a try. Thank you

Looks like that fixed it. Thanks. Closing this ticket now


[1915]: java.net.BindException happens when delay-expose-service is enabled.

What is the purpose of the change
issue#1915: java.net.BindException happens when delay-expose-service is enabled.
Brief changelog
dubbo-rpc/dubbo-rpc-dubbo/src/main/java/com/alibaba/dubbo/rpc/protocol/dubbo/DubboProtocol.java
Verifying this change
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GITHUB_issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn clean install -DskipTests & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.
[x] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #1916 into master will increase coverage by <.01%.
The diff coverage is 80%.

@@             Coverage Diff              @@
##             master    #1916      +/-   ##
============================================
+ Coverage     49.51%   49.51%   +<.01%
- Complexity     4504     4506       +2
============================================
  Files           558      558
  Lines         24347    24351       +4
  Branches       4315     4316       +1
============================================
+ Hits          12055    12057       +2
- Misses        10431    10435       +4
+ Partials       1861     1859       -2

Impacted Files
Coverage Δ
Complexity Δ

...libaba/dubbo/rpc/protocol/dubbo/DubboProtocol.java
57.75% <80%> (+0.3%)
30 <0> (ø)
:arrow_down:

...onfig/spring/extension/SpringExtensionFactory.java
73.07% <0%> (-11.54%)
8% <0%> (ø)

...om/alibaba/dubbo/rpc/filter/ActiveLimitFilter.java
83.33% <0%> (-5.56%)
6% <0%> (-1%)

...baba/dubbo/remoting/transport/mina/MinaClient.java
58.46% <0%> (+1.53%)
9% <0%> (+1%)
:arrow_up:

...rpc/protocol/dubbo/telnet/InvokeTelnetHandler.java
55.69% <0%> (+3.79%)
13% <0%> (+2%)
:arrow_up:

Continue to review full report at Codecov.

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


Create a direct spirv backend compile time perf test

We would like to be able to monitor the performance of end to end spirv compilation time  --- both slang side and driver side combined. This will allow us to know if we're making headawy [and or regressing] on compilation performance.
The scrappy start we think is right for this is

Take the 2 shaders from Falcor that have previously encountered compile time and get them to run standlone (2 shaders in material image test, @csyonghe can give specifics to whomever takes this task
Wrap a perf runner around that (this will require to designing a perf runner as we don't have one thats meaningful, suggest pulling best practices from other projects that have done this in the past)
Get the CL merged
Verify repeatability // amount of variance in the test results and ensure it gives clean signal

We can add more shaders to that runner after those two exist.

This is done.


'framework' attribute of package.json not documented in README

We don't mention the framework:true attribute in https://github.com/mesosphere/universe/blob/master/README.md#packagejson
We do have the word framework as an example of a tag - not sure if related (maybe that was an earlier approach?). Maybe clearer to remove that as a tag example.
It's definitely used in e.g. the HDFS package.json.

all use of the word framework has been depreciated.  you will not see any use in version-2.x or moving forward.


[mp4upload] Add mp4upload

add for mp4upload.com

Please follow the issue template for site support
Do you have any example URLs that don't require login? Or do you always need an account
Also please make sure the site isn't a pirate haven

Do you have any example URLs that don't require login

yes
https://www.mp4upload.com/wssnof547lsr

related: https://github.com/ytdl-org/youtube-dl/pull/19103/files

IGNORING THE ISSUE TEMPLATE WILL RESULT IN ISSUE CLOSED AS INCOMPLETE
The link you provide contains pirated material -  we do not support sites primary used for piracy. Quick search of this site also brings up that it hosts content for mostly piracy-based sites.

Also related from upstream:
https://github.com/ytdl-org/youtube-dl/pull/19103
https://github.com/ytdl-org/youtube-dl/pull/16958


Add resolveAliasAsValue to fix a stack overflow

Fixes #2933. The issue is that getTypeOfAlias needs to resolve the alias as a value, which would cause resolveAlias to detect a cycle. However, resolveAlias has no idea that it's looking for a value, so it stops early. This causes getTypeOfAlias to infinitely recurse, and each time it calls resolveAlias, it gets back an alias that is its own target.
The fix is to make a more aggressive resolveAliasAsValue, which will detect the cycle properly. And getTypeOfAlias should call that.
For a more complete explanation, see #3140.

Does htis only repro for LS scenarios?  Is there a command line exampe that would hit this?

I wasn't able to find any command line scenarios for this. I feel like it should be possible to hit in theory, but it's possible the code paths required are not reachable from the command line.

Closing in favor of #3160


Add a Gitstalk link on user profile page

Gitstalk is a nice little web app that shows a user's recent activities. Lately I've been using it a lot, so I'm wondering if it makes sense to add a link to it on user profile pages, such as at the bottom of the left side:

I like the tool but I don't like making stalking easier 😅

Hmm, that's fine. Just roll my own script...


Ingest-CanaryTokens: Unable to deploy to Azure, validation failed

Describe the bug
Unable to add this playbook to Azure. After inputting my information and proceeding to the review tab, I receive an error message and am unable to continue further.
To Reproduce
Steps to reproduce the behavior:

Go to code page for Ingest-CanaryTokens playbook (https://github.com/Azure/Azure-Sentinel/tree/master/Playbooks/Ingest-CanaryTokens)
Click on Deploy to Azure
Sign on as an account with necessary permissions (I use a global administrator)
Enter Workspace ID and Workspace Key
Click Review + Create
See error

Expected behavior
Expected no error. Should have been able to click Create and proceed in adding this playbook to Azure
Screenshots

Desktop (please complete the following information):

OS: Windows 11 23H2
Browser: Edge
Version: 129.0.2792.52

Additional context
The error seems to be indicating a syntax error at or around the "tags" path in the template. After reviewing the source, I was unable to identify the issue.

Hi @matthwiz, Thanks for flagging this issue, we will investigate this issue and get back to you with some updates. Thanks!

Hi @matthwiz,
We have updated the azuredeploy.json file, please refer below steps for deployment:

Open link - https://github.com/Azure/Azure-Sentinel/tree/master/Playbooks/Ingest-CanaryTokens

Click on deploy to azure button

fill required property value and click on edit template

Copy content from attached json file and paste the content and click on save

Check on next button

Please let us know are you able to deploy or not.

Hello, I was able to deploy successfully with the updated json file. Thanks for you help!

Hi @matthwiz, thanks for your response, closing this issue now. If you still need support for this issue, feel free to re-open it any time. Thank you for your co-operation


Deprecated libs while compiling shaka player

Have you read the Tutorials?
Yes
Have you read the FAQ and checked for duplicate open issues?
Yes
What version of Shaka Player are you using?
Master
Please ask your question
When compiling Shaka Player, to implement the corrections to buffering, following what is in https://shaka-player-demo.appspot.com/docs/api/tutorial-welcome.html , I get these warnings (as before):
~/shaka-player$ python build/all.py
npm WARN deprecated<EMAIL_ADDRESS>core-js@<3 is no longer maintained and not recommended for usage due to the number of issues. Please, upgrade your dependencies to the actual version of core-js@3.
npm WARN deprecated<EMAIL_ADDRESS>This module is no longer maintained, try this instead:
npm WARN deprecated   npm i nyc
npm WARN deprecated Visit https://istanbul.js.org/integrations for other alternatives.
npm WARN deprecated<EMAIL_ADDRESS>request has been deprecated, see https://github.com/request/request/issues/3142
npm WARN deprecated<EMAIL_ADDRESS>You can find the new Popper v2 at @popperjs/core, this package is dedicated to the legacy v1
npm WARN deprecated<EMAIL_ADDRESS>this library is no longer supported
Doesn't deprecated or no longer maintained libs cause problems?
Can I safely ignore these warnings?
Thanks.

We only use NPM packages for building and testing, not at runtime.  So long as it builds, everything is fine.  I think most of these are from other projects that we don't have any control over anyway.


Add more debug/trace logging

add delivery tag to inbound delivery logs
add trace logging to channel method calls

cherry-pick to 2.0.x, 1.7.x

Just curious what drives you to do this?
Is there really reason to back-port it till 1.7.x ?
Thanks

... and cherry-picked to 2.0.x and 1.7.x.

It should backport ok;
Some weird problem on google groups.


callout for email insight & kotlin code sample

Description of the change: Add Kotlin sample code to documentation with Java Library. Add callout about Email Insight Service.
Reason for the change: Kotlin is alternative of Java. Send grid also has Email Insight Service.
Link to original source: https://github.com/sendgrid/docs/issues/4577

[x] Geographic Statistics
[x] Global Statistics
[x] Google Analytics and SendGrid Statistics
[x] Mailbox Provider Comparison
[x] Category Statistics
[x] Device Statistics

Closes #

I added callout about email insight service. 🥇

Hello @wdharmana,Thanks again for the PR! It's HACKTOBERFEST! We want to show our appreciation by sending                                              you some special Hacktoberfest swag. If you have not already, could you please fill out                                              this form so we can send it to you? Thanks!                                             Team SendGrid DX

@wdharmana - this is an awesome PR! Super clean, and GREAT content. Thanks so much!!!

Hello @wdharmana,Thanks again for the PR!                                         You have earned 16 out of the 5 points needed to receive glorious SendGrid Hacktoberfest swag.                                         Please take a moment to checkout this link to find more issues to get you past the required threshold or to simply continue the celebration.                                         Also, please be sure you have officially registered with us here by November 1, 2018 to qualify.                                         If you have any questions you can email us at<EMAIL_ADDRESS>                                        Thank you and Happy Hacktobering!                                         Team SendGrid

@wdharmana - this is an awesome PR! Super clean, and GREAT content. Thanks so much!!!

Thank you @ksigler7 ! I really enjoy contribute here and I hope i can continue my contribution next time.

Hello @wdharmana,
Thanks again for the PR!
You have earned 16 out of the 5 points needed to receive glorious SendGrid Hacktoberfest swag.
Please take a moment to checkout this link to find more issues to get you past the required threshold or to simply continue the celebration.
Also, please be sure you have officially registered with us here by November 1, 2018 to qualify.
If you have any questions you can email us at<EMAIL_ADDRESS>Thank you and Happy Hacktobering!
Team SendGrid

@thinkingserious Cool!! Thanks. I've register and wait for cool swag from Sendgrid