flaw in stalling logic(Hazard detection unit)

An unwanted stall occurs in when there are multiple load instructions.

hazard detection unit is used to stall the pipeline when an instruction that is dependent on previous load instruction arrives. as load instructions don't have RS2(source register 2 ) field so there may be an issue which we have faced while testing. the logic is as,
when(io.ID_EX_MEMREAD === "b1".U && ((io.ID_EX_REGRD === rs1_sel) || (io.ID_EX_REGRD === rs2_sel)))
{
io.inst_forward := 1.U
io.pc_forward := 1.U
io.ctrl_forward := 1.U
io.inst_out := io.IF_ID_INST
io.pc_out := io.pc_in
io.current_pc_out := io.current_pc
}
.otherwise
{
io.inst_forward := 0.U
io.pc_forward := 0.U
io.ctrl_forward := 0.U
io.inst_out := io.IF_ID_INST  // Doesn't matter if we pass the old instruction forward because it won't be selected by the mux
io.pc_out := io.pc_in         // Doesn't matter if we pass the old pc value forward because it won't be selected by the mux
io.current_pc_out := io.current_pc
}
according to this logic whenever an instruction followed by load instruction comes in pipeline the logic will compare RD of execute stage with RS1 or RS2 of decode stage and stall the pipeline if instruction in execute stage is load instruction.

but a situation may arrive when both instructions in execute and decode are load instructions and value of RS2 field of decode matches with the value of RD in execute, then there will be an unwanted stall because load instruction don't have RS2 field.
Solution is to set a check point on stalling condition. that is to check the instruction in decode stage is load instruction if it is then compare only RS1 field with RD of execute.
when(io.IF_ID_MEMREAD === "b1".U)
{
when(io.ID_EX_MEMREAD === "b1".U && (io.ID_EX_REGRD === rs1_sel))
{
stalling_conditions()
}
.otherwise
{
default_conditons()
}
}
.otherwise
{
when(io.ID_EX_MEMREAD === "b1".U && ((io.ID_EX_REGRD === rs1_sel) || (io.ID_EX_REGRD === rs2_sel)))
{
stalling_conditions()
}
.otherwise
{
default_conditons()
}
}


Minor edit of basic_auth

Reimplemented RequestBuilder::basic_auth to use Base64Encoder; this prevents
unnecessary intermediate allocations

Thanks!


Add coveralls support for coverage tracking

Added in 2bebf480b517f236022743ee4a5b1ff381545acd
But used https://codecov.io/ instead since coveralls had dependency issues


Topic Recognition (s47605790): StyleGAN on the ADNI dataset (Task 8)

Hello teaching team,
This is my pull request for my project which is implementation of StyleGAN model on the ADNI dataset using PyTorch. This model is used for generating synthesis brain images similar to those in the ADNI dataset.
Files and folders included:

dataset.py: preparation for the data
modules.py: contains components of this project
train.py: is for training model, generating loss plot
predict.py: for testing the model and generating UMAP embedding plot
README.md: documentation of this project
readme_materials: assets for README.md

Details of the project are all documented in the README.md file
Thanks,
Tin Trung Nguyen, Student ID: 47605790

Images are not shown in the README.md file, I will redo the pull request


Confusing example in Cafe Menu Step 21

Describe the Issue
I am seeing this over and over again in the forum. The example in the instructions uses the selector #example-id and for whatever reason people think they need to add -id to the end of the #menu selector in their solution. And the hint doesn't appear to help much because half the time they will paste the hint verbatim along with their code.
Affected Page
https://www.freecodecamp.org/learn/2022/responsive-web-design/learn-basic-css-by-building-a-cafe-menu/step-21
Your code
Example:
#example-id {
  width: 250px;
}

Camper solution:
#menu-id {
  width: 300px;
}

Expected behavior
I think we need to get rid of the dash in the example. I also think we could improve the instructions by making it clearer how to use the id selector.

The goal now is to make the menu div not take up the entire width of the page. The CSS width property is perfect for this.

You can use the id selector to target a specific element by its id attribute. An id selector is defined by placing a hash symbol in front of the element's id value. For example, if an element has the id of cat then you would select it by its id like this:

#cat {
  width: 250px;
}

I also think it might be helpful to answer the question "Why use an id?" so people know why we are introducing this selector. We do mention that it allows you to "target a specific element" but that doesn't explain why you would want to do that. But I understand that we want to keep reading down to a minimum so I'm not insisting :slightly_smiling_face:
Screenshots
No response
System
N/A
Additional context
No response

I understand your concern. It seems that there might be some confusion due to the use of the #example-id selector in the instructions, which may lead learners to think that they should use a similar naming convention in their code.
To clarify, you should use the selector that matches the actual element you want to style. In your provided HTML, the element you want to style has the id attribute "menu." So, you should use the #menu selector to target that specific element or clarify with telling we are using instead of  id which is given in line no "10" of index.html. you can clarify with give reference of code you use.

Hi there,
I'm excited to contribute to this project, and I'd like to work on this issue. It seems like a great opportunity to dive into the codebase and make a meaningful contribution.
I'll start by reviewing the issue and the provided information. If there are any specific requirements or suggestions, please let me know. I'll be happy to follow the project's coding standards and guidelines.
Looking forward to your feedback and guidance. Thanks for the opportunity!

I wouldn't use an example that is the answer though.

I think the proposed update sounds reasonable.
Maybe we update the instructions as well.

Create a #menu selector and give the selected element a width of 300px.

And also update the test

The #menu selector should set a width of 300px on the element.

Maybe we also add # to the sentence?

...by placing a hash symbol # in front of the...

I like the proposed changes. No question - what's the current convention for using a symbol in a sentence in the curriculum? In mathematical writing, that would be offset by parenthesis or commas, so 'the hash symbol (#)' or 'the hash symbol, #,' is the grammatically correct usage.

I don't really know, but personally, I'm not a fan of when we use parentheses.
I have seen them being included in the code posted too often. It seems a lot of people can not tell the difference between the syntax inside the code block and the surrounding parentheses. For example (a) I have seen multiple times show up in the code as (a).

Yeah, I don't think it's the best option either. I see three choices

parenthesis

commas

just ignore the grammar rules

We should officially pick one and document it

To Address this Issue .You could Consider the Following Improvement
Update the Example : Instead of Writing examle-id We can write idName so that it simple and more understandable.
Original :
#example-id{
width:250px;
}
Updated :
#idName{
250px;
}

To Address this Issue .You could Consider the Following Improvement
Update the Example : Instead of Writing examle-id We can write idName so that it simple and more understandable.
Original :
#example-id{
width:250px;
}
Updated :
#idName{
width:250px;
}

I think we have already settled on a fix. See the discussion above.
To summarize the proposal as I understand it:

Step 21
The goal now is to make the div not take up the entire width of the page. The CSS width property is perfect for this.
You can use the id selector to target a specific div element.

Good catch, I forgot to add that part:

Step 21
The goal now is to make the div not take up the entire width of the page. The CSS width property is perfect for this.
You can use the id selector to target a specific element with an id attribute. An id selector is defined by a name with a hash symbol # directly in front of the element's id value. For example, if an element has the id of cat then you would target that element like this:
#cat {
  width: 250px;
}

Use the #menu selector to give your element a width of 300px.

I'd be okay with that change.

This issue is not a bug. @bbsmooth I would urge you to close this issue now, not worth the time of other contributors.

@leftover19, we will be making a wording change. Please do not spam this Issue with additional requests that we do not clarify the wording.

Going to go ahead and open this for contribution. An acceptable PR will implement Jeremy's proposed fix.


GDScript: Fix missing warning for shadowing of built-in types

func _ready():
  var Array := 3 # before: ok, after: warning

Extracted from #74618.

Thanks!
PS. Now that it returns nothing, the name of GDScriptAnalyzer::is_shadowing is not great. It should probably be something like check_if_shadowing or test_shadowing, or some version of these.


Clarification Regarding E2E Latency Numbers

I wanted to clarify the way the E2E latency numbers are calculated and posted on this page:
https://github.com/Xilinx/Vitis-AI/tree/master/models/AI-Model-Zoo#performance-on-zcu104
Specifically, in the case of ZCU104, there are two DPU engines instantiated and the benchmarking is done with a batch size of 2 by default based on this message:
https://github.com/Xilinx/Vitis-AI/issues/275#issuecomment-766762849
However, the E2E latency seems to be calculated simply by inverting the FPS, which would be incorrect in this case since the batch size is not one and the latency will be the same whether batch size is 1 or 2 in this case since each frame is processed independently by either of the two DPU engines, and batch-1 performance will NOT scale across multiple DPU engines.
As an example, in the case of Resnet 50, the single-threaded FPS is reported as 75.1 which is with a batch size of 2. The E2E latency in this case would be 1000/(75.1/2)=26.63 ms, and not the 13.29 ms that is in the table.
I would appreciate it if someone clarifies this since these latency numbers can be outright incorrect and quite misleading the way they are currently being calculated.

The E2E latency number is using a single thread (one DPU, batch size =1), and in this case the frame rate is inverses of the latency.
The multithreaded throughput value uses 2 DPUS, batch size =2.

@jimheaton Based on the the reply from @qianglin-xlnx, that is not the case. Single-threaded is still batch size of 2 and number of threads is independent of batch size, unless the person doing the benchmarking modified the Xilinx benchmark code to enforce batch size of 1 as mentioned under that Github issue.

@zohourih. Issue #275 was a question about the Alveo U50 DPU.
For Zynq and ZynqUltraScale+ DPUs: One thread = 1 DPU, batch size 1.

Hi @zohourih
Just as @jimheaton  said, for DPU on edge, the batch size is 1.
For ZCU104, we can integrate 2 DPU core, that batch size of each DPU core is 1. The number of DPU core is not the batch num.

@jimheaton @qianglin-xlnx All right, I understand your point. I will close the issue, then.


Simplify and update comments in core/allocators.h

Updated code to provide:

minor performance improvements,
add and update comments to be more descriptive,
reformatting to increase readability.

In regards to the git blame noise vs. code noise issue, I believe it's better to have git blame noise if it makes the code more consistent. After all, you aren't limited to only the HEAD when doing a git blame.

In regards to the git blame noise vs. code noise issue, I believe it's better to have git blame noise if it makes the code more consistent and easier to read. After all, you aren't limited to only the HEAD when doing a git blame. You can git blame from any commit.

It depends on the merit of the "consistency".
Variable naming conventions etc? Yes
Swapping ++i for i++? No

The issue with valuing git blame noise so highly, is that you limit the iterative process of refining the code. A better solution would be to find a way to make git blame noise less of a problem, so that it doesn't turn away positive code improvements. I believe, this is especially important in an open source project, as you will have developers of all skill types (from beginners to experts) potentially looking at the code.
Small changes like ++i to i++ for consistency, can stop you second guessing yourself when debugging, as well as reducing the barrier for understanding the code.

The issue with valuing git blame noise so highly, is that you limit the iterative process of refining the code. A better solution would be to find a way to make git blame noise less of a problem, so that it doesn't turn away positive code improvements. I believe, this is especially important in an open source project, as you will have developers of all skill types (from beginners to experts) potentially looking at the code.
Small changes like ++i to i++ for consistency, can stop you second guessing yourself when debugging, as well as reducing the barrier for understanding the code.

Small changes can make an improvement yes.
This is not one of them. i++ and ++i as a lone statement is not confusing, no one is confused. It is a change for the sake of a change.
Git blame is right down at the bottom of priorities when it comes to reasoning about changes. There very little that's less important, but one thing that definitely is, is arbitrarily fiddling with stuff.
This change alone is not much of a problem, but if it's your intent to go through many files in the code base and make similar changes lines to logical equivalents for no increase in clarity, it's going to be a source of irritation.

I never said that i++ or ++i are confusing. Just that the rest of the file uses the i++ notation. Having this instance use a different notation can cause headaches down the line when debugging. I'd prefer debugging a file that uses one style, then flips to the other where necessary, rather than a file that uses the styles interchangeably. If this kind of stuff doesn't affect you, you're going to see this type of change as arbitrary.
I can see how this can be a source of irritation, with having to weigh up git blame and the git history with small subtle changes that might be arbitrary to some, but more meaningful to others. The programmer in me is fine with either style, but the designer in me cares deeply about making choices (no matter how small) with purpose and meaning.

I was not aware that this change could have a potential performance impact.
The Google Style Guide (https://google.github.io/styleguide/cppguide.html#Preincrement_and_Predecrement) identifies the pros and cons of prefix vs postfix. In this case, they imply that either may be used as the variable is 'a simple scalar (non-object)'.
I made this particular change to maintain the style consistency within the file. If the advantages of prefix are more desirable, then changing the postfix operators to prefix would be an option to maintain a consistent style.


[ErrorException] Undefined property: LEtudiant\Composer\Installer\SharedPackageInstaller::$binaryInstaller

Hi,
I should preface that I'm trying to install a proprietary system from my work. I'll try to provide as much information as possible without divulging their code.
 [ErrorException]
 Undefined property: LEtudiant\Composer\Installer\SharedPackageInstaller::$binaryInstaller

I'm currently running Windows 7 x64, PHP 7.0.2. This error originally appeared on an older build of Composer, but had worked in the past (as early as last week, same PHP build, same composer build.) In an attempt to fix the solution I entered composer install --no-custom-installers which allowed the system to continue installation, but the underlying application could not continue.
I'm not entirely sure what this error means, could you expand on it?
After upgrading composer to the latest, I'm getting the following error:
 [ErrorException]
 symlink(): Could not fetch file information(error 3)

I'm not entirely sure what this error means either, but I get the feeling it's PHP or composer related, and not necessarily related to your library. Have you encountered this before?
I'm able to get past this part of the installation by entering composer install --no-plugins which allows me to continue.
Any ideas what might be going on?

Hello, the first error is a BC change in the last Composer version. Just update to the last Composer version like you done. Otherwise, require the last 2.x version of this plugin.
For the symlink error, it appears your folder or symlink is not readable or the path is wrong.
Is your dependencies folder exists ?

Hi Divi,
The dependency folder exists. If I run the application with --no-plugins flag, then it downloads and puts everything in the correct shared folder.
The application does run, but I'm worried that whatever should run during the installer script is not.

Divi,
Is this a windows symlink problem? It appears to work fine on unix environments.

Yep, symlinks are pretty broken on Windows :(

The symlink appears to be needed, but I'm not sure how to make that symlink work properly to satisfy this plugin without it throwing an error.
If the reason why this error is being displayed (symlink(): Could not fetch file information(error 3)) is because I'm using Windows, then my recommendation would to be add the Windows incompatibility to the documentation to let others know of the same problem should they encounter it.
Thank you for all of your help.

No, it worked on my Windows a few months ago, I said it pretty broken because relative symlinks doesn't not work.
Try with this configuration : "symlink-base-path": "/foo/bar/composer-dependencies".
See : https://github.com/Letudiant/composer-shared-package-plugin/blob/master/docs/all-available-configurations.md
Can you paste me your composer.json file please ?


DevelopmentCodeFragment does not catch fully qualified functions

PHPMD version: 2.9.1
PHP Version:7.4.12
Installation type: composer
Operating System / Distribution & Version: OSX 10.14.6

Current Behavior
\print_r not caught
Expected Behavior
The method foo::bar() calls the typical debug function print_r() which is mostly only used during development.
Steps To Reproduce:
function bar($baz)
{
  \print_r($baz);    // not caught by DevelopmentCodeFragment rule
  print_r($baz);    // caught by DevelopmentCodeFragment rule
}

Checks before submitting

[x] Be sure that there isn't already an issue about this. See: Issues list
[x] Be sure that there isn't already a pull request about this. See: Pull requests
[x] I have added every step to reproduce the bug.
[x] If possible I added relevant code examples.
[x] This issue is about 1 bug and nothing more.
[x] The issue has a descriptive title. For example:  "JSON rendering failed on Windows for filenames with space".

Confirmed. Actually, this rule doesn't work when there is an active namespace so I actually never had it working in my projects. But allowing starting \ in function names would be a good start.

Could I please be assigned this issue?

With pleasure :) 🥇


Want to "throw" an exception, but are not in the HANDLER for that exception anymore.

[Enter feedback here]
Hi, what if you want to preserve the callstack, BUT you are not in the handler anymore (you saved off the exception and want to throw it later)?

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

ID: 65b444d4-49e0-6d5a-ef71-1235478066b5
Version Independent ID: 13c37e4f-453b-3578-1a3a-de6a890da280
Content: CA2200: Rethrow to preserve stack details (code analysis) - .NET
Content Source: docs/fundamentals/code-analysis/quality-rules/ca2200.md
Product: dotnet-fundamentals
GitHub Login: @gewarren
Microsoft Alias: gewarren

I'm not certain that the code analysis description would be the correct place to document this, but it really should be documented somewhere (but isn't right now), and linking to it from this code analysis as a form of "see also" hint would be sensible.
In the interim, if you want to capture an exception for possibly rethrowing later, with a preserved callstack, you should save an ExceptionDispatchInfo object created like var edi = ExceptionDispatchInfo.Capture(ex);.
You can then later rethrow the exception with intact stack trace by calling edi.Throw(), which will rethrow the exception while preserving the stack trace, and other dispatching details. You can also inspect the exception so captured via: edi.SourceException.
This really should be documented somewhere other than documentation for the ExceptionDispatchInfo class, since for example, developers creating a custom awaitable will likely want to use it.

Perhaps use of https://learn.microsoft.com/en-us/dotnet/api/system.runtime.exceptionservices.exceptiondispatchinfo?view=net-7.0 should be mentioned here? https://learn.microsoft.com/en-us/dotnet/standard/asynchronous-programming-patterns/task-based-asynchronous-pattern-tap#exceptions
cc @AaronRobinsonMSFT

@gewarren The suggestion from @KevinCathcart is correct.

Perhaps use of ExceptionDispatchInfo should be mentioned here?

I'm not sure adding it to the task patterns is appropriate. I agree we should add it somewhere, but I'm not sure where. The task model has different ways of handling exceptions and I'd like to avoid of confusing those two scenarios. Is there a best-practices for general Exception usage?

@AaronRobinsonMSFT Yes, we have best practice for exceptions here: https://learn.microsoft.com/en-us/dotnet/standard/exceptions/best-practices-for-exceptions. I'll add something there, thanks.


Test for the greening algorithm

Codecov Report

Merging #33 (9b984cb) into master (bb8450b) will increase coverage by 0.83%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master      #33      +/-   ##
============================================
+ Coverage     47.43%   48.26%   +0.83%
- Complexity     3916     3979      +63
============================================
  Files          1448     1448
  Lines         21068    21068
  Branches       3921     3921
============================================
+ Hits           9993    10169     +176
+ Misses         8948     8754     -194
- Partials       2127     2145      +18

Impacted Files
Coverage Δ

...org/uast/uast/generated/tree/green/Identifier.java
79.31% <0.00%> (+3.44%)
:arrow_up:

...java/org/uast/uast/generated/tree/js/Property.java
67.34% <0.00%> (+4.08%)
:arrow_up:

...va/org/uast/uast/generated/tree/green/Program.java
82.92% <0.00%> (+7.31%)
:arrow_up:

...st/uast/generated/tree/green/ClassDeclaration.java
79.16% <0.00%> (+8.33%)
:arrow_up:

...uast/uast/generated/tree/green/IntegerLiteral.java
75.75% <0.00%> (+9.09%)
:arrow_up:

...st/uast/generated/tree/green/FieldDeclaration.java
80.95% <0.00%> (+9.52%)
:arrow_up:

.../org/uast/uast/generated/tree/green/ClassBody.java
82.92% <0.00%> (+9.75%)
:arrow_up:

.../uast/uast/generated/tree/green/ModifierBlock.java
73.17% <0.00%> (+9.75%)
:arrow_up:

...uast/uast/generated/tree/green/DeclaratorList.java
73.17% <0.00%> (+9.75%)
:arrow_up:

...org/uast/uast/generated/tree/green/Declarator.java
69.38% <0.00%> (+10.20%)
:arrow_up:

... and 6 more

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


[NEW] Implement notification settings

Your Rocket.Chat.Android version: 1.0.20

On the web I have the option of disabling notifications and unread status.

I would like the mobile app to respect that option. I chose not to see any #general notifications, even the unread status, but on the mobile it shows up on my channels list as unread.

I'm closing this in favor of the same issue https://github.com/RocketChat/Rocket.Chat.Android/issues/1203
referencing the new app.


Instrument Everything?

Are there any videos or posts that better details "Instrument Everything", or how you team "instruments everything"? I'm interested in any guidance, tools, tricks, etc.

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

ID: 15f1fadb-d2a0-f671-af1e-0707ef6e385d
Version Independent ID: 64faa90d-7e2a-465a-89dc-06f40ccf612c
Content: Moving to Cloud Cadence - Azure DevOps
Content Source: https://mseng.visualstudio.com/DefaultCollection/TechnicalContent/_git/devops-learn-pr?path=%2fdocs%2fdevops-at-microsoft%2fmoving-cloud-cadence.md&version=GBlive&_a=contents
Product: devops
Git Login: samgu
Microsoft Alias: loril

I'm still interested in a response.

@MisinformedDNA -- Dan, you may find answers here:

Azure DevOps Support Bot
Azure DevOps on Stack Overflow

My apologies for the delay in responding.


MD MSU MD+ audio support

Hi, Is it possible and how can i implement the MD MSU MD+ audio in games?

Hi,
SGDK does not provide any assistance with the MD MSU / MD+ features of some flash carts nor MCD in general so you should handle that by yourself. You can find some documentation here about how use the MSU MD:
https://github.com/krikzz/msu-md
I think it can be used with SGDK, at least you can use SGDK compiler to compile anything from the MSU-MD sdk :)

Hi, thanks for the reply.
We are players who dreamed of having arcade versions like SNK / KOF 9x ,
Marvel fighting and Capcom / Street Fighter Alpha games series for genesis.
Please, could you know the work of my brazilian fighting game development
team for genesis?
I am responsible for the development of the audio area.
We are also developing tools that speed up sprite production and game
physics.
Here's our youtube channel for you to check it out.
https://www.youtube.com/playlist?list=PLO-5vY5uaWRt8VUzMrCrAj_BeCtK3PDya
We would be very happy if you could leave us comments and mention our
projects in your portfolio.
Thank you very much!!  YOU and the SGDK has opened up new avenues for sega
genesis lovers. :))
Em ter., 2 de nov. de 2021 às 05:50, Stephane Dallongeville <
@.***> escreveu:

Hi,
SGDK does not provide any assistance with the MD MSU / MD+ features of
some flash carts nor MCD in general so you should handle that by yourself.
You can find some documentation here about how use the MSU MD:
https://github.com/krikzz/msu-md
I think it can be used with SGDK, at least you can use SGDK compiler to
compile anything from the MSU-MD sdk :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Stephane-D/SGDK/issues/227#issuecomment-957230913,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWJXHL3E4HIPYJY4B234YPLUJ6Q47ANCNFSM5HEZFIHA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Your projects looks quite promising, it would be really nice to see a complete game out of it though :)
In the current state it's more of a POC than a real game (it misses important parts as collisions) and to be honest lot of that exists so i cannot reference all them.
Lately you had this POC also showing vs fighters animations (ok, that is far less advanced but animation is impressive by itself) :
https://twitter.com/masterlinkueibr/status/1460054520969449476
https://twitter.com/masterlinkueibr/status/1459884704702816260

Thanks for the feedback!
Em seg., 15 de nov. de 2021 às 07:55, Stephane Dallongeville <
@.***> escreveu:

Your projects looks quite promising, it would be really nice to see a
complete game out of it though :)
In the current state it's more of a POC than a real game (it misses
important parts as collisions) and to be honest lot of that exists so i
cannot reference all them.
Lately you had this POC also showing vs fighters animations (ok, that is
far less advanced but animation is impressive by itself) :
https://twitter.com/masterlinkueibr/status/1460054520969449476
https://twitter.com/masterlinkueibr/status/1459884704702816260
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Stephane-D/SGDK/issues/227#issuecomment-968771013,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWJXHL7LONMQP3FWDGOTRMLUMDRJ5ANCNFSM5HEZFIHA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Sharing a map with a teammate email link raises 500 error when not logged in

Context
Please explain here below what you were doing when the issue happened
I made a map with an org user, and shared with second org user. When I opened email sent to 2nd org user with "view your map" link in a separate browser, I received a 500 error rather than being asked to log in.
Here is the link I tried from the email: https://smoke712-org.carto-staging.com/u/andrew2ndsmokeuser/viz/andrewsmokeorguser.664ca6e4-a5a7-11e6-a407-040111176f01/map
So the map is in the "andrewsmokeorguser" account but shared with the "andrew2ndsmokeuser" account.
Note that once I am logged in to "andrew2ndsmokeuser" I can see and view the map just fine, no errors.
Steps to Reproduce
Please break down here below all the needed steps to reproduce the issue

Make a map, share with another org user
Open email link, in a browser in which you are not logged in
view 500 error

Current Result
Please describe here below the current result you got
see above
Expected result
Please describe here below what should be the expected behaviour
Rather than a 500 error, I would have expected a login prompt, to then see the map.
Browser and version
What internet browser (Chrome, Firefox, etc) and version was you using and version
Chrome on the "andrewsmokeorguser"; Safari on the "andrew2ndsmokeuser"
.carto file
Don't forget to rename .carto file to .zip file to be able to upload to github
Additional info
Please add any information of interest here below
SMOKE_MM07

Hey @andrewbt
Do you get an actual 500 error?

Or an embed is private error?

I am trying to reproduce but I get a private embed error, not the 500.

I got the actual 500.
On Tue, Nov 8, 2016 at 12:56 PM, Javier Torres<EMAIL_ADDRESS>wrote:

Hey @andrewbt https://github.com/andrewbt
Do you get an actual 500 error?
[image: screenshot_20161108_125613]
https://cloud.githubusercontent.com/assets/7569673/20098171/bf6ffd42-a5b2-11e6-99a0-6c793f08f631.png
Or an embed is private error?
[image: screenshot_20161108_125544]
https://cloud.githubusercontent.com/assets/7569673/20098177/c2332680-a5b2-11e6-9224-3f161d9ba46f.png
I am trying to reproduce but I get a private embed error, not the 500.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/CartoDB/cartodb/issues/10628#issuecomment-259118195,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB65cyHvU7gcQIkwAO6TaZpefTBfl7Yjks5q8GN9gaJpZM4KsVdU
.

--
Andrew Thompson, Solutions Engineer
CARTO | 201 Moore St, Brooklyn, NY, 11206
<EMAIL_ADDRESS>| T +1<PHONE_NUMBER> <+15083337265>
Web carto.com http://cartodb.com/ | Twitter @CARTO
http://twitter.com/carto
Predict Through Location
Live sessions to learn CARTO http://carto.com/webinars/

Ahh crap, maybe it was 400-series error? Like a 403? It was definitely the
blue page with the spaceship. Let me see if I can get it again.
On Tue, Nov 8, 2016 at 12:58 PM, Andrew Thompson<EMAIL_ADDRESS>wrote:

I got the actual 500.
On Tue, Nov 8, 2016 at 12:56 PM, Javier Torres<EMAIL_ADDRESS>wrote:

Hey @andrewbt https://github.com/andrewbt
Do you get an actual 500 error?
[image: screenshot_20161108_125613]
https://cloud.githubusercontent.com/assets/7569673/20098171/bf6ffd42-a5b2-11e6-99a0-6c793f08f631.png
Or an embed is private error?
[image: screenshot_20161108_125544]
https://cloud.githubusercontent.com/assets/7569673/20098177/c2332680-a5b2-11e6-9224-3f161d9ba46f.png
I am trying to reproduce but I get a private embed error, not the 500.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/CartoDB/cartodb/issues/10628#issuecomment-259118195,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB65cyHvU7gcQIkwAO6TaZpefTBfl7Yjks5q8GN9gaJpZM4KsVdU
.

--
Andrew Thompson, Solutions Engineer
CARTO | 201 Moore St, Brooklyn, NY, 11206
<EMAIL_ADDRESS>| T +1<PHONE_NUMBER> <+15083337265>
Web carto.com http://cartodb.com/ | Twitter @CARTO
http://twitter.com/carto
Predict Through Location
Live sessions to learn CARTO http://carto.com/webinars/

--
Andrew Thompson, Solutions Engineer
CARTO | 201 Moore St, Brooklyn, NY, 11206
<EMAIL_ADDRESS>| T +1<PHONE_NUMBER> <+15083337265>
Web carto.com http://cartodb.com/ | Twitter @CARTO
http://twitter.com/carto
Predict Through Location
Live sessions to learn CARTO http://carto.com/webinars/

So I get the embed error with that link now because the map is set to
private / not shared. Let me re-enable private sharing with one user.
 Like a 403? It was definitely the
blue page with the spaceship. Let me see if I can get it again.
On Tue, Nov 8, 2016 at 12:58 PM, Andrew Thompson<EMAIL_ADDRESS>wrote:

I got the actual 500.
On Tue, Nov 8, 2016 at 12:56 PM, Javier Torres<EMAIL_ADDRESS>wrote:

Hey @andrewbt https://github.com/andrewbt
Do you get an actual 500 error?
[image: screenshot_20161108_125613]
https://cloud.githubusercontent.com/assets/7569673/20098171/bf6ffd42-a5b2-11e6-99a0-6c793f08f631.png
Or an embed is private error?
[image: screenshot_20161108_125544]
https://cloud.githubusercontent.com/assets/7569673/20098177/c2332680-a5b2-11e6-9224-3f161d9ba46f.png
I am trying to reproduce but I get a private embed error, not the 500.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/CartoDB/cartodb/issues/10628#issuecomment-259118195,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB65cyHvU7gcQIkwAO6TaZpefTBfl7Yjks5q8GN9gaJpZM4KsVdU
.

--
Andrew Thompson, Solutions Engineer
CARTO | 201 Moore St, Brooklyn, NY, 11206
<EMAIL_ADDRESS>| T +1<PHONE_NUMBER> <+15083337265>
Web carto.com http://cartodb.com/ | Twitter @CARTO
http://twitter.com/carto
Predict Through Location
Live sessions to learn CARTO http://carto.com/webinars/

--
Andrew Thompson, Solutions Engineer
CARTO | 201 Moore St, Brooklyn, NY, 11206
<EMAIL_ADDRESS>| T +1<PHONE_NUMBER> <+15083337265>
Web carto.com http://cartodb.com/ | Twitter @CARTO
http://twitter.com/carto
Predict Through Location
Live sessions to learn CARTO http://carto.com/webinars/

--
Andrew Thompson, Solutions Engineer
CARTO | 201 Moore St, Brooklyn, NY, 11206
<EMAIL_ADDRESS>| T +1<PHONE_NUMBER> <+15083337265>
Web carto.com http://cartodb.com/ | Twitter @CARTO
http://twitter.com/carto
Predict Through Location
Live sessions to learn CARTO http://carto.com/webinars/

Hmmm, I can't duplicate either anymore. /shrug
On Tue, Nov 8, 2016 at 1:00 PM, Andrew Thompson<EMAIL_ADDRESS>wrote:

So I get the embed error with that link now because the map is set to
private / not shared. Let me re-enable private sharing with one user.
 Like a 403? It was definitely
the blue page with the spaceship. Let me see if I can get it again.
On Tue, Nov 8, 2016 at 12:58 PM, Andrew Thompson<EMAIL_ADDRESS>wrote:

I got the actual 500.
On Tue, Nov 8, 2016 at 12:56 PM, Javier Torres<EMAIL_ADDRESS>
wrote:

Hey @andrewbt https://github.com/andrewbt
Do you get an actual 500 error?
[image: screenshot_20161108_125613]
https://cloud.githubusercontent.com/assets/7569673/20098171/bf6ffd42-a5b2-11e6-99a0-6c793f08f631.png
Or an embed is private error?
[image: screenshot_20161108_125544]
https://cloud.githubusercontent.com/assets/7569673/20098177/c2332680-a5b2-11e6-9224-3f161d9ba46f.png
I am trying to reproduce but I get a private embed error, not the 500.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/CartoDB/cartodb/issues/10628#issuecomment-259118195,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB65cyHvU7gcQIkwAO6TaZpefTBfl7Yjks5q8GN9gaJpZM4KsVdU
.

--
Andrew Thompson, Solutions Engineer
CARTO | 201 Moore St, Brooklyn, NY, 11206
<EMAIL_ADDRESS>| T +1<PHONE_NUMBER> <+15083337265>
Web carto.com http://cartodb.com/ | Twitter @CARTO
http://twitter.com/carto
Predict Through Location
Live sessions to learn CARTO http://carto.com/webinars/

--
Andrew Thompson, Solutions Engineer
CARTO | 201 Moore St, Brooklyn, NY, 11206
<EMAIL_ADDRESS>| T +1<PHONE_NUMBER> <+15083337265>
Web carto.com http://cartodb.com/ | Twitter @CARTO
http://twitter.com/carto
Predict Through Location
Live sessions to learn CARTO http://carto.com/webinars/

--
Andrew Thompson, Solutions Engineer
CARTO | 201 Moore St, Brooklyn, NY, 11206
<EMAIL_ADDRESS>| T +1<PHONE_NUMBER> <+15083337265>
Web carto.com http://cartodb.com/ | Twitter @CARTO
http://twitter.com/carto
Predict Through Location
Live sessions to learn CARTO http://carto.com/webinars/

--
Andrew Thompson, Solutions Engineer
CARTO | 201 Moore St, Brooklyn, NY, 11206
<EMAIL_ADDRESS>| T +1<PHONE_NUMBER> <+15083337265>
Web carto.com http://cartodb.com/ | Twitter @CARTO
http://twitter.com/carto
Predict Through Location
Live sessions to learn CARTO http://carto.com/webinars/

I'm looking at the error log and I cannot see anything :confused: This could be related to recent changes that we did to hide maps that have not been published yet.
There are some circumstances where you could get a 404 or 403, specially if the map has not yet been published and is password protected (or not shared). Did you publish the map before sharing it?

It was set as private and shared with one user (which I believe
automatically published it; if not, I feel like had published it also after
sharing, but perhaps no...). When the other user tried to use the link,
that was when the error page came up.
It's definitely not happening anymore. Might have just been a "fluke".
On Tue, Nov 8, 2016 at 1:16 PM, Javier Torres<EMAIL_ADDRESS>wrote:

I'm looking at the error log and I cannot see anything 😕 This could be
related to recent changes that we did to hide maps that have not been
published yet.
There are some circumstances where you could get a 404 or 403, specially
if the map has not yet been published and is password protected (or not
shared). Did you publish the map before sharing it?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/CartoDB/cartodb/issues/10628#issuecomment-259121795,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB65cwAWfORQQ_mSz0mygbQgfodHzCX5ks5q8GgWgaJpZM4KsVdU
.

--
Andrew Thompson, Solutions Engineer
CARTO | 201 Moore St, Brooklyn, NY, 11206
<EMAIL_ADDRESS>| T +1<PHONE_NUMBER> <+15083337265>
Web carto.com http://cartodb.com/ | Twitter @CARTO
http://twitter.com/carto
Predict Through Location
Live sessions to learn CARTO http://carto.com/webinars/

@javitonino @andrewbt closing then! Thanks!


Remove PATH from .bazelrc action_envs

One thing i have noticed is that my local RBE use (which uses a different container env to Envoy's CI) tends not to share caches with envoy CI, at least sometimes
as my usage doesnt appear to affect envoy CI or vice-versa i didnt worry about it, it seemed a feature - altho i did wonder
trying to track caching issues i noticed that we currently have PATH set as an action_env , about which the docs say...

For example, environments with different $PATH variables won't share cache hits. Only environment variables explicitly whitelisted via --action_env are included in an action definition. Bazel's Debian/Ubuntu package used to install /etc/bazel.bazelrc with a whitelist of environment variables including $PATH. If you are getting fewer cache hits than expected, check that your environment doesn't have an old /etc/bazel.bazelrc file.

not sure if we have it set for legacy reasons or llvm reasons, but i see llvm has PATH specified elsewhere so it doesnt seem necessary, and removing it seems like a generally good idea

cc @keith

somewhat relatedly the @envoy_repo repo rule exposes PATH and VERSION which im wondering about in relation to caches being blown
in that case PATH (which is only meant for no-remote jobs) is the path to WORKSPACE not executable path and is derived from the WORKSPACE but im wondering the effect

the problem with PATH is us generally supporting any directory for compilers and things. because of that i think we have to have it unless we solve that another way?

if we dont need for it CI, perhaps just documenting that you should set it if you need xyz from PATH with a warning about how it will affect caches i guess

another possible approach is to standardize the PATH between types of machine

thinking about this further - its not an issue for CI because no matter the host it alwasy runs in the ~same container - still not sure this is desirable as it essentially leaks the local env but not a prio in terms of CI


[Feature] Abstract-Particle-Swarm-Background

Description
JavaScript and HTML canvas generate a vibrant particle swarm animation. Colorful particles move randomly, bouncing off canvas edges, creating dynamic visual effects.
Screenshots

Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[ ] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[X] I am a SWOC'24 contributor.

Code of Conduct

[X] I agree to follow this project's "Code of Conduct".

Proceed @kom-senapati


[flutter_tools] reland remove flutter view cache

Description
Fixes #56194
Remove caching of FlutterView and poll forever if the list of flutter views is empty.
Added test for missing release mode logic.

NNBD when!


Prepare for releasing v0.8.3.

A month has passed since v0.8 was released and it contained a significant issue with the mismatch of image crate that was fixed in #2352. Thus it is time for another minor release - v0.8.3.
This PR gets us ready for publishing:

Finalized the changelog.
Updated all the relevant links.
Bumped druid and druid-shell to 0.8.3.

Consider using today's date, if you are anticipating finishing the publication today.

I didn't anticipate such a fast review cycle, but yes I updated the date and will move forward with it today.


Can't to import DateRangePickerInput

Hello
I can't to import component DateRangePickerInput from @devfamily/admiral
There is no in library 😢

Hello, Gordy!
Thanks for your issue.
You are right.
We added the changes to the master, but didn't update our package.
We have just updated the @devfamily/admiral to 5.9.4 version. You should update the version, if you wanna use the DateRangePuckerInput component.
Have a nice day!


Can't pass down styled props to an extended styled system + rebass component

I'm creating a Position component like so:
import styled from 'styled-components'
import { position } from 'styled-system'
import { Box as Base } from 'rebass/styled-components'

const Position = styled(({ position, ...props }) => <Base {...props} />)`
  ${position}
`

export default Position

I'm expecting to be able to use it like so after importing into another file:
const Header = styled(Position)` color: red `;

<Header
  as="header"
  mb={4} // Box styled props
  pt={3} // Box styled props
  position="sticky"
  top={0}
  left={'auto'}
  right={'auto'}
  zIndex={100}
>

Do I need to be utilizing forwardRef ? Is there a way to pass down both Box props and Position props from an extended component?

At first glance, this looks like it should work. Can you provide a codesandbox link with a reproduction?

@coreybruyere @jxnblk it seems that styled-components has some magic going on starting 4.0.0 whereby it doesn't render extended components, it only inherits their styles (I guess for speed?). Obviously then, you can't pass down props to a component that doesn't get rendered. Weirdly it is not documented in their migration guide or anywhere. Replace styled-components with either >v3.0.0 or @emotion/styled and it works.

No idea about the community in general, I mainly just prefer emotion’s css prop instead of creating one shot components for every styled div. Plus emotion seems to be react concurrent mode ready, but that shouldn’t be a concern for now I think.


Unexpected type exception during the assigning Future variable to FutureOr.

Dart SDK Version: 2.1.0-dev.4.0
OS: Windows 10 (64 bit)
The following test example creates Future object and assign it to the variable of Future  type in different ways:
import "dart:async";

main() {
  Future f = new Future(() => 12345);
  Future<FutureOr> f1 = f;

  FutureOr fo = f;
  f1 = fo;
}

Dartanalyzer passes with it without errors.
Dart passes here on the line 5 (Future<FutureOr> f1 = f;) and fails on the line 8 (f1 = fo;) with the following error:

Unhandled exception:
type 'Future' is not a subtype of type 'Future<FutureOr>'
#0      main (file:///D:/DART/sdk/tests/co19/src/LanguageFeatures/Instantiate-to-bound/Future/test.dart:8:3)
#1      _startIsolate. (dart:isolate/runtime/libisolate_patch.dart:289:19)
#2      _RawReceivePortImpl._handleMessage (dart:isolate/runtime/libisolate_patch.dart:171:12)

It seems like such behavior is incorrect: f0 and f1 actually points to the same Future object, so assigning f1 to f and fo should cause the same result.

The static and run-time type of f is Future<dynamic>.
The static type of f1 is Future<FutureOr<dynamic>>.
The static type of fo is FutureOr<dynamic>.
The assignment of fo to f1 is statically valid because FutureOr<dyamic> is a super-type of Future<FutureOr<dynamic>> (most easily recognized by seeing that the type Future<dynamic> is between the two and obviously related to them).
The assignment is a down-cast (assigning from super-type to sub-type), so at run-time, an extra check is added to check that the value has type Future<FutureOr<dynamic>>.
Since the value has type Future<dynamic>, that check should succeed (like it did in line five). The VM fails it for some reason. The program runs without problem in dart2js.
So, definitely a bug.

Seems related to the subtype relationship implementation in the VM. Tentatively assigning to @crelier

Thanks for the report. A fix is on its way.


TW Sizing plugin - add position attributes (bottom/left/right/top)

Are we sure? I mean... position attributes such as top|bottom|left|right seem much more related to space scale than size one, don't you think? Happy to discuss.


x/build/cmd/gomote: 502 Bad Gateway error

I often encounter this 502 Bad Gateway error while actively working with a buildlet created using gomote. I guess this is because I was holding the buildlet too long (~30min) and there is a hidden time limit on each buildlet. If so, we need a better error message than this:

$ gomote ls user-hakim-windows-amd64-2016-0
Error running ls: 502 Bad Gateway:

And, gomote list still shows the lease is not yet expired. That's misleading.

$ gomote list
user-hakim-windows-amd64-2016-0	windows-amd64-2016	host-windows-amd64-2016	expires in 29m49.611152102s

I'm seeing this frequently too. I don't think it's actually a time limit, but it's opaque enough that I'm not sure.
CC @golang/osp-team

I ran into the same problem this morning. Very mysterious error.

Deployed coordinator with new logging. Please report any errors you see. We can also search the coordinator's logs for it too.

New error just now:
Error running puttar: 502 Bad Gateway; body: (golang.org/issue/28365): gomote proxy error: Put http://<IP_ADDRESS>/writetgz?dir=: net/http: request canceled while waiting for connection now run: gomote run user-thanm-android-386-emu-0 /bin/bash build-on-gomote.sh Error running run: Error trying to execute /bin/bash: buildlet: HTTP status 502 Bad Gateway: (golang.org/issue/28365): gomote proxy error: Post http://<IP_ADDRESS>/exec?: net/http: request canceled while waiting for connection

Error: 502 Bad Gateway; body: (golang.org/issue/28365): gomote proxy error: Get http://macstadium_host04a/tgz?dir=pkg&pargzip=0: EOF

@andybons, I suspect that's a different problem but not sure.

My comment was a local issue. Sorry for the spam.

As @bcmills noted in https://github.com/golang/go/issues/37001#issuecomment-581691898, there's a chance this is the same issue as #37001, which has been resolved on Feb 6, 2020. /cc @toothrot
The last comments here are from 2019. Has this happened to anyone again since then?

I spent some quality time with a windows-amd64-2016 gomote instance for #39183 over the past week, and haven't seen any disconnects. I think it's fixed.

I just saw this on an iOS gomote.
Log:
$ gomote run user-mknyszek-ios-arm64-corellium-0 go/src/all.bash
...
2021/12/03 18:22:32 user-mknyszek-ios-arm64-corellium-0: timeout after 10s waiting for headers for /status
2021/12/03 18:22:52 user-mknyszek-ios-arm64-corellium-0: timeout after 10s waiting for headers for /status
2021/12/03 18:23:12 user-mknyszek-ios-arm64-corellium-0: timeout after 10s waiting for headers for /status
2021/12/03 18:23:12 Buildlet https://farmer.golang.org:443 failed three heartbeats; final error: timeout waiting for headers
Error running run: Error trying to execute go/src/all.bash: Buildlet https://farmer.golang.org:443 failed heartbeat after 10.000250219s; marking dead; err=timeout waiting for headers
$ gomote run user-mknyszek-ios-arm64-corellium-0 go/bin/go test -short runtime
Error running run: Error trying to execute go/bin/go: buildlet: HTTP status 502 Bad Gateway: (golang.org/issue/28365): gomote proxy error: Post "http://iPhone/exec?": revdial.Dialer closed
$ gomote list
user-mknyszek-ios-arm64-corellium-0	ios-arm64-corellium	host-ios-arm64-corellium-ios	expires in 29m57.358647306s
$ gomote run user-mknyszek-ios-arm64-corellium-0 go/bin/go test -short runtime
Error running run: Error trying to execute go/bin/go: buildlet: HTTP status 502 Bad Gateway: (golang.org/issue/28365): gomote proxy error: Post "http://iPhone/exec?": revdial.Dialer closed
$ gomote run user-mknyszek-ios-arm64-corellium-0 go/bin/go test -short runtime
Error running run: Error trying to execute go/bin/go: buildlet: HTTP status 502 Bad Gateway: (golang.org/issue/28365): gomote proxy error: Post "http://iPhone/exec?": revdial.Dialer closed

I've been running stress tests on a pool of 8 windows-amd64-longtest builders and over the course of about an hour I've gotten dozens of these failures:
2022/01/19 13:06:00 creating windows-amd64-longtest buildlet
2022/01/19 13:06:28 created buildlet user-austin-windows-amd64-longtest-3
2022/01/19 13:06:28 installing go1.4
Error running push: 502 Bad Gateway; body: (golang.org/issue/28365): gomote proxy error: Post "http://<IP_ADDRESS>/writetgz?dir=go1.4": read tcp <IP_ADDRESS>:49068-><IP_ADDRESS>:80: read: connection reset by peer
One odd thing is that they seem to happen in clusters. My stress testing tool will retry setting up a new gomote 5 times before giving up, and many of the logs show the buildlet failing part way through make.bat and then failing several times in a row with "502 Bad Gateway":

2022/01/19 13:03:05 creating windows-amd64-longtest buildlet
2022/01/19 13:03:33 created buildlet user-austin-windows-amd64-longtest-3
2022/01/19 13:03:34 installing go1.4
2022/01/19 13:03:41 Remote doesn't have "src/syscall/zsysnum_freebsd_386.go"
2022/01/19 13:03:41 Remote doesn't have "test/fixedbugs/issue19548.go"
2022/01/19 13:03:41 Remote doesn't have "src/cmd/compile/internal/ssa/deadcode_t
est.go"
2022/01/19 13:03:41 Remote doesn't have "src/cmd/go/internal/modfetch/sumdb.go"
2022/01/19 13:03:41 Remote doesn't have "src/cmd/vendor/golang.org/x/sys/windows
/race0.go"
2022/01/19 13:03:41 Remote doesn't have 10994 files (only showed 5).
2022/01/19 13:03:41 Remote lacks a VERSION file; sending a fake one
2022/01/19 13:03:47 Uploading 10995 new/changed files; 40413585 byte .tar.gz
Building Go cmd/dist using C:\workdir\go1.4
Building Go toolchain1 using C:\workdir\go1.4.

Building Go toolchain2 using go_bootstrap and Go toolchain1.
Building Go toolchain3 using go_bootstrap and Go toolchain2.
Error running run: Error trying to execute go/src/make.bat: missing Process-Stat
e trailer from HTTP response; buildlet built with old (<= 1.4) Go?
2022/01/19 13:06:00 setup command failed: exit status 1
2022/01/19 13:06:00 creating windows-amd64-longtest buildlet
2022/01/19 13:06:28 created buildlet user-austin-windows-amd64-longtest-3
2022/01/19 13:06:28 installing go1.4
Error running push: 502 Bad Gateway; body: (golang.org/issue/28365): gomote prox
y error: Post "http://<IP_ADDRESS>/writetgz?dir=go1.4": read tcp <IP_ADDRESS>:490
68-><IP_ADDRESS>:80: read: connection reset by peer
2022/01/19 13:06:29 setup command failed: exit status 1
2022/01/19 13:06:29 creating windows-amd64-longtest buildlet
2022/01/19 13:07:08 created buildlet user-austin-windows-amd64-longtest-3
2022/01/19 13:07:08 installing go1.4
Error running push: 502 Bad Gateway; body: (golang.org/issue/28365): gomote prox
y error: Post "http://<IP_ADDRESS>/writetgz?dir=go1.4": read tcp <IP_ADDRESS>:383
34-><IP_ADDRESS>:80: read: connection reset by peer
2022/01/19 13:07:10 setup command failed: exit status 1
2022/01/19 13:07:11 creating windows-amd64-longtest buildlet
2022/01/19 13:07:48 created buildlet user-austin-windows-amd64-longtest-3
2022/01/19 13:07:48 installing go1.4
Error running push: 502 Bad Gateway; body: (golang.org/issue/28365): gomote prox
y error: Post "http://<IP_ADDRESS>/writetgz?dir=go1.4": read tcp <IP_ADDRESS>:419
10-><IP_ADDRESS>:80: read: connection reset by peer
2022/01/19 13:07:50 setup command failed: exit status 1
2022/01/19 13:07:50 creating windows-amd64-longtest buildlet
2022/01/19 13:08:31 created buildlet user-austin-windows-amd64-longtest-3
2022/01/19 13:08:31 installing go1.4
Error running push: 502 Bad Gateway; body: (golang.org/issue/28365): gomote prox
y error: Post "http://<IP_ADDRESS>/writetgz?dir=go1.4": read tcp <IP_ADDRESS>:333
78-><IP_ADDRESS>:80: read: connection reset by peer
2022/01/19 13:08:33 setup command failed: exit status 1
2022/01/19 13:08:33 giving up after 5 retries
exited: status 1

To add to my previous message, I've noticed something very strange. I had spun up 25 windows-amd64-2016 builders and they ran fine for a few minutes. Then all but 3 of them died with errors like in my previous message. My gopool program kept trying to recreate the 22 failed builders, but I never saw one that lasted more than 2.1 seconds after creation. All but the 3 kept failing with 502 errors. I believe exactly the same thing had happened in a previous run, though I wasn't watching it as closely.

As a data point, I tried running
VM=$(gomote create windows-amd64-2016)
gomote push $VM
gomote destroy $VM

in 12 parallel loops. I expected this to quickly produce 502 errors, but I let it go for about an hour and didn't get any 502 errors.


Connect-PnPOnline : Invalid URI: The hostname could not be parsed

I downloaded the latest and I meet all of the prerequisites for installation. I completed Step 1 without issue. My credentials are current and correct.
I execute the following script is here:
.\deploy.ps1 -TenantName "CMI Compas" -TenantID "8f361bbf-7452-4793-" -RequestsSiteName "Request a Team" -RequestsSiteDesc "Used for admins to create a Microsoft Team after approval." -ManagedPath "Teams" -SubscriptionId d20c05ac-2955-4c89-b11a- -Location "eastus" -ResourceGroupName "teamsgovernanceapp-rg" -AppName "Requestateamapp" -ServiceAccountUPN<EMAIL_ADDRESS>-UseMSGraphBeta $false -IsEdu $false -KeyVaultName "TeamsAutomationKey"
The script starts, then the first error is:
Creating Teams Requests SharePoint site...
Error occured while creating of the SharePoint site: The remote server returned an error: (401) Unauthorized.
Connect-PnPOnline : Invalid URI: The hostname could not be parsed.
At C:\files\requestateam-master\deployment\scripts\deploy.ps1:749 char:15

$pnpConnect = Connect-PnPOnline $requestsSiteUrl -UseWebLogin

          ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : NotSpecified: (:) [Connect-PnPOnline], UriFormatException
FullyQualifiedErrorId : System.UriFormatException,SharePointPnP.PowerShell.Commands.Base.ConnectOnline

REQUESTS SPO SITE CONFIGURATION
Configuring SharePoint site...
Applying provisioning template...
Apply-PnPProvisioningTemplate : The remote server returned an error: (401) Unauthorized.
At C:\files\requestateam-master\deployment\scripts\deploy.ps1:334 char:9

    Apply-PnPProvisioningTemplate -Path (Join-Path $packageRootPa ...

    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : WriteError: (:) [Apply-PnPProvisioningTemplate], WebException
FullyQualifiedErrorId : EXCEPTION,SharePointPnP.PowerShell.Commands.Provisioning.Site.ApplyProvisioningTemplate
Applied template
Error occured while configuring the SharePoint site: Exception calling "ExecuteQuery" with "0" argument(s): "The remote server returned an error: (401) Unauthorized."

The script continues and then I use the service account credentials and the scripts complete with this:
Finished deploying logic apps
Azure resources deployed
AZURE RESOURCES DEPLOYMENT COMPLETE
LOGIC APP CONNECTIONS AUTHORISATION
Starting authorisation for Logic App Connections
Please authenticate with the Service Account -<EMAIL_ADDRESS>SharePoint Connection
Connection  teamsautomate-spo  now  @{status=Connected}
Office 365 Outlook Connection
Connection  teamsautomate-o365outlook  now  @{status=Connected}
Office 365 Users Connection
Connection  teamsautomate-o365users  now  @{status=Connected}
Microsoft Teams Connection
Connection  teamsautomate-teams  now  @{status=Connected}
LOGIC APP CONNECTIONS AUTHORISATION COMPLETE
DEPLOYMENT COMPLETED SUCCESSFULLY
Any help here would be awesome. Thanks

This can be closed. It was my ID10T error.


Add Dispose and IsConnected to IDataQueueHandler

Description

The IDataQueueHandler interface has been extended to include Dispose and IsConnected methods

Related Issue
Closes #4348
Motivation and Context

IDataQueueHandler implementations should be able to perform cleanup (e.g. closing connections) and also have a method to test for connectivity.

Requires Documentation Change
No.
How Has This Been Tested?

Deployed live algorithms in QC cloud

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Refactor (non-breaking change which improves implementation)
[ ] Performance (non-breaking change which improves performance. Please add associated performance test and results)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Non-functional change (xml comments/documentation/etc)

Checklist:

[x] My code follows the code style of this project.
[x] I have read the CONTRIBUTING document.
[x] All new and existing tests passed.
[x] My branch follows the naming convention bug-<issue#>-<description> or feature-<issue#>-<description>

I think LiveTradingDataFeed should now probably be calling Dispose on the IDataQueueHandler it owns?

I think LiveTradingDataFeed should now probably be calling Dispose on the IDataQueueHandler it owns?

This might be tricky because the IDataQueueHandler instance is usually the brokerage instance.


Custom database parser does not check for missing fields

The result of a missing field usually ends up being a broken question or a vague error in console.
In warning about missing fields, the user should also be cautioned about fields being mis-named.

This has since been fixed -- final changes to address it have been made in 7cb80fc


Server-side data fetching with RLS - token in local.user instead of s…

Testing the example I can use supabaseServerClient(locals.user.accessToken) successfully.
But the supabaseServerClient(session.accessToken) does not work for me in the endpoint.

Can be closed as soon as PR #245  is merged.

PR #245 solved tis.


Could not able to create database in mongo

from chatterbot import ChatBot
from chatterbot.trainers import ChatterBotCorpusTrainer
from pymongo import MongoClient
import logging
logging.basicConfig(level=logging.INFO)
chatbot = ChatBot(
"Terminal",
database_uri='mongodb://localhost:27017/chatterbot_test_database',
storage_adapter='chatterbot.storage.MongoDatabaseAdapter',
logic_adapters=[
"chatterbot.logic.BestMatch"
],
input_adapter="chatterbot.input.TerminalAdapter",
output_adapter="chatterbot.output.TerminalAdapter"
)
chatbot.set_trainer(ChatterBotCorpusTrainer)
chatbot.train("chatterbot.corpus.english")
print("Type something to begin...")
while True:
try:
bot_input = chatbot.get_response(None)
except (KeyboardInterrupt, EOFError, SystemExit):
break

from you code looks everything ok to me, could you post the stack trace of the error you got?

1.This program is not throwing any error. this code simply get executed but does not create any database.
2. As far as virtual environment is concerned.
i just created environment using: virtualenv cbtenv.
virtualenv cbtenv
New python executable in /home/vaibhav/cbtenv/bin/python
Installing setuptools, pip, wheel...done.


Fleet RBAC: Existing UI works with user with limited permissions

Stage 1
User with access to gitrepos only via global role
Create a global role, e.g. Fleet User, give permissions:

Create a new user with 'user base' and add the 'Fleet User' role to them

Dashboard page should render - currently requires bundles.
Should be able to add a gitrepo - adds okay, but then fails with an error message - must be computing a property or something.
Should be able to access detail for a GitRepo - with and without bundle resource access
BUG: Refreshing on the git repo page shows no repos, but if you go to dashboard and back again, it does - check this is not due to requiring bundle access

Stage 2
Can we give a user access to a single project that contains a namespace for a fleet workspace and check they can only do things to that fleet workspace
Update the global role 'Fleet User' and remove the gitrepos resource from it.
Create a project/namespace role and give access to gitrepos (as above image, but only gitrepos resource - call it `Fleet Project Access'.
As an admin user, create a project FleetTestProject and create a fleet workspace. Go into the local cluster and move the fleet workspace to the project. Edit the project and give our test user permissions the `Fleet Project Access' on that project.
With these permissions, the user we created should be able to access Fleet, but only be able to access the resources in the single Fleet Workspace we created.
We now need to remove the global permissions on the Fleet Workspace resource.
We need to:

Update the UI so that if the user does NOT have access to the Fleet Workspace resource, that we get the list of workspaces that the user can access by looking at the namespaces they have access to and filtering those that have the annotation "objectset.rio.cattle.io/id": "workspace" - this appears to be preset on namespaces that are fleet workspaces.

With this done, our user will not be able to manage or see fleet workspace resources, but will be able to see the list of fleet workspaces that they have access to (via the underlying namespace) and they will be able to create git repos in those workspaces.

What's the current state here?
Shouldn't we close this issue?

Where can I find this in the UI ? Continuous Delivery?
I am on v2.7-head with dashboard release-2.7.2 4065709cf
Fleet is v0.6.0-rc.5
@nwmac

I got this clarified in an offline conversation. Please disregard.

Validated on Rancher vv2.7-head over multiple commits latest: 0c76b3f
Latest validated Dashboard: release-2.7.2 94650da49
Fleet: v0.6.0-rc.5
Validated based on documented test plan scenarios.
Separate issues during validation:
https://github.com/rancher/dashboard/issues/8623
https://github.com/rancher/dashboard/issues/8624
https://github.com/rancher/dashboard/issues/8468
https://github.com/rancher/dashboard/issues/8467


Study details and status endpoints throw 403 error instead of 400 when querying a study that doesn't exist

This is related to the recent change to use claims in the token to look up if a researcher has access to the study they are querying. It seems like the checks for whether the user has the required claim is run before checking if the study exists at all, so we get a 403 error. This is less useful for error handling - it now says "Permission denied" instead of something like "the study doesn't exist"

Yes, study existence check happens at service layer; Claim check happens at controller layer, which is before service layer. We should either find a way to check study existence without using a lot of resources and also enforce modulith, or just combine the error message to "study doesn't exist or permission denied"

Suggested solution:
1.Remove the @PreAuthorize annotation from the getParticipantGroupStatus method.
2.Check if the study exists before performing the authorization check.
3.Return a 400 error if the study does not exist, otherwise proceed with the authorization (check the identity of the user) using the canManageStudy function from ProxiesMethodSecurityExpressionRoot.
4.If the user does not have permission, return a 403 error.
it's a bad practice to inject CoreParticipantRepository with ProxiesMethodSecurityExpressionRoot(participantRepository, authenticationService)
ProxiesMethodSecurity is a part of SecurityConfig class, therefore I suggest creating a securityService and calling it from the service layer to check canManageStudy()

We decided not to change the endpoints but instead make a better textual description in the error message.


SnapshotBatchSubmitted events not properly being processed

Describe the bug
The SnapshotBatchSubmitted events emitted by the ProtocolState contract are being missed by the processor_distributor causing the projects to never be flagged as finalized. This causes issues with aggregate snapshot generation downstream.
To Reproduce
Affected versions: main
Steps to reproduce the behavior:
1.
2.
3.
Expected behavior
The system_event_detector should catch the SnapshotBatchSubmitted events and broadcast a message to the processor_distributor.
Proposed Solution
The event signature for SnapshotBatchSubmitted is incorrect and needs to be updated.
Caveats
N/A
Additional context
N/A

Implemented here https://github.com/PowerLoom/snapshotter-core/tree/fix/24

Moving to current iteration to allow for review.


Where to get this theme?

hey mate!
Really like this theme! Any chance on getting it via platforms like opendesktop.net or the AUR?

Thanks :D
You can get it from here by using the install instructions or from - https://www.pling.com/p/1615057/
I can see about opendesktop.net but I don't know how to package stuff for AUR so will leave that to someone more experienced.


Add standard error redirection

…e in case of no internet connection

It helps to prevent the syntax error messages of sed.
But the script is still broken if there is no connection.
We should do more work on this script.

Mine works fine. It just does not show rrt number.
https://i.imgur.com/IKPWZXq.png

Yes, thats the point. Something like Error: I could not ping the target. Would be better.
Another question is how to handle ping timeouts.
I will think about it.


Linux "In App" Titlebar coloring

I have noticed that the "In App" titlebar wasn't themed. I fixed that
note that I have only tested this with macchiato, but I see no reason why it shouldn't work with the other themes.

I should've probably said this earlier, but this only applies to the th-ch YoutubeMusic client.

LGTM


Hero System 5R: Initial Commit

Changes / Comments
New sheet for Hero System 5th Ed Revised
Based on Criss's updates to the 6th edition
verified all 5R calculations
added a sheet version
Hit location and Targeting tables added.
lift attribute removed
allowed for costs in the powers/equipment field
Bolded the power/equip names
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[x] Does the pull request title have the sheet name(s)? Include each sheet name.
Hero System 5R.html
Hero System 5R.css
[ ] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[x] Does this add or change functional aesthetics (such as layout or color scheme) ?
updates to include slight format changes, added additional lookup tables (hit location, targeting) , added
[x ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
removed lift attribute, nothing to preserve, this was generated from STR attribute
[x ] If this is a new sheet, did you follow Building Character Sheets standards ? Yes, except for a chrome vs Firefox check, did that and now will need to do slight update to have looking correct in both

If you do not know English. Please leave a comment in your native language.

Sorry, this is just 1 character sheet folder

I've mistakenly included a warewolf and have reverted this pull


lint fixes

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

19 of 34 (55.88%)  changed or added relevant lines in 23 files are covered.
30 unchanged lines in 2 files lost coverage.
Overall coverage remained the same at 66.58%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

dataplane/forwarding/fwdaction/actions/evaluate.go
0
1
0.0%

dataplane/forwarding/fwdport/ports/group.go
2
3
66.67%

dataplane/forwarding/fwdtable/flow/flow.go
2
3
66.67%

dataplane/internal/kernel/kerneltest/kernel.go
0
1
0.0%

sysrib/server.go
0
1
0.0%

dataplane/forwarding/fwdaction/actions/decap.go
0
2
0.0%

dataplane/forwarding/fwdaction/actions/swapoutput.go
0
2
0.0%

dataplane/forwarding/fwdport/ports/kernel.go
0
2
0.0%

dataplane/forwarding/fwdport/ports/tap.go
0
2
0.0%

gnmi/gnmi.go
0
2
0.0%

Files with Coverage Reduction
New Missed Lines
%

sysrib/server.go
14
54.66%

dataplane/forwarding/fwdtable/flow/flow.go
16
79.29%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
4114

Relevant Lines:
6179

💛  - Coveralls

this pr wasn't ready for review, i'm still looking into the test failure


[1.1.x] Fix carbonlink queries for tagged series. | flake8 cleanup | With PR #2295 the top nav dashboard fails to render the default tree | Also have to override the input box value of '' with '*' because que

Backports the following commits to 1.1.x:

Fix carbonlink queries for tagged series. (#2370)
flake8 cleanup (#2371)
With PR #2295 the top nav dashboard fails to render the default tree (#2372)
Also have to override the input box value of '' with '*' because query='' is a "bug" (#2372)
fix rrd paths (#2383)

added avg_zero support for consolidate function (#2380)

added avg_zero to tests (#2380)

added test for test_readers_util (#2380)

fixed avg_zero readers_util test (#2380)

add avg_zero consolidation support, match whisper behavior when there are no non-null values (#2380)
docs: add link to regex documentation (#2381)
remove unused import (#2383)
remove separate rrdtool environment in travis (#2383)
fix typo in comment (#2383)
Add a minValue option to nonNegativeDerivative and perSecond (#2375)
Add coverage for composer's send_email function  (#2174)

Codecov Report

Merging #2399 into 1.1.x will increase coverage by 0.17%.
The diff coverage is 82.85%.

@@            Coverage Diff             @@
##            1.1.x    #2399      +/-   ##
==========================================
+ Coverage   79.62%   79.79%   +0.17%
==========================================
  Files          85       85
  Lines        8892     8910      +18
  Branches     1899     1906       +7
==========================================
+ Hits         7080     7110      +30
+ Misses       1551     1537      -14
- Partials      261      263       +2

Impacted Files
Coverage Δ

webapp/graphite/render/evaluator.py
100% <100%> (ø)
:arrow_up:

webapp/graphite/readers/utils.py
100% <100%> (ø)
:arrow_up:

webapp/graphite/finders/standard.py
71.42% <100%> (-4.33%)
:arrow_down:

webapp/graphite/readers/rrd.py
45.16% <60%> (-1.51%)
:arrow_down:

webapp/graphite/render/datalib.py
96.55% <75%> (-2.02%)
:arrow_down:

webapp/graphite/render/functions.py
96.13% <85.71%> (-0.09%)
:arrow_down:

webapp/graphite/settings.py
77.61% <0%> (ø)
:arrow_up:

webapp/graphite/composer/views.py
55.55% <0%> (+30.86%)
: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 8820eb8...e012af4. Read the comment docs.

Strange, that went fine. Merging too.


[video_player] a way to get the thumbnail from the video

I know there's couple of flutter plugin that does this for iOS and Android. But I'm developing an app for FLUTTER WEB, and I can't find a solution to this. If we initialized() video_player, it provides a thumbnail, but there's no way we can use that as BYTE DATA

Hi @markgrancapal
I see there's an open issue addressing the case you described.
Please follow up on that issue, I'm closing the current one as duplicate.
If you disagree, please write in the comments and I will reopen it.
Thank you

Hi @markgrancapal
I see there's an open issue addressing the case you described.
Please follow up on that issue, I'm closing the current one as duplicate.
If you disagree, please write in the comments and I will reopen it.
Thank you

Oh great! thanks @iapicca


peer: resume partial writes due to timeouts

This PR modifies the behavior of the peer writes messages to the wire, such that we are able to gracefully handle timeout errors and resume partially sent messages. As of #2819, if a message is partially written to the wire due to a timeout error, we will try to write the full message again. In turn, this will produce an authentication error on the remote side (after it is able to read the number of bytes specified in the header) since the middle of the latter ciphertext will not pass as the MAC check.
We resolve this by splitting the Write call into two subroutines, WriteMessage  and Flush. WriteMessage encrypts the plaintext and buffers the ciphertext on the underlying connection. A subsequent call to Flush will then attempt to write the bytes out on the wire. If a timeout error is encountered during flush, we can safely resume the byte stream by calling Flush again. In addition to being able to resume partial writes, this also has a yuge benefit in not blocking the write pool with network operations. This fully decouples the number of write pool workers from the number of peers, since the blocking operations now take place in each peer.
This is an alternative, and more complete fix, to #2901, which only offers the ability to fail faster if a partial write takes place.

@Roasbeef @wpaulino ptal


[T5A3][W14-A2]HonKeanWai

Accepting this PR without a review because another of your PRs for this tutorial was reviewed in detail.


Using pre-trained SimplE Wikidata5m model files

Hello,
I just saw the docs and github pages but I couldn't find how to get all connected nodes in the knowledge graph for one particular node.
I am new to this arena and GraphVite, so can you explain the difference between model and app?
Like this model object
with open("simple_wikidata5m.pkl", "rb") as fin:
    model = pickle.load(fin)

and the app object in the Colab tutorial provided.
And lastly how to get all nodes connected to a particular entity in the graph from the model object.
Thanks!!

They almost refer to the same thing, except that model is a non-executable dump of the parameters in the application. To turn a model into an application, create an application with the same hyperparameters, and then use app.load_model(model).
The model dump only contains the embeddings and doesn't contain the original graph. You can manually parse the file gv.dataset.wikidata5m.train to get entity neighbors. See here for the format of Wikidata5m.

Thanks a lot for the clarification.
The code that I ran just now:
import pickle
import graphvite.application as gap

with open("transe_wikidata5m.pkl", "rb") as fin:
    model = pickle.load(fin)
app = gap.KnowledgeGraphApplication(dim=512)
app.load_model(model)

And here it starts printing tons of IDs (Qxxxxxxxx), is that normal?
Does app have that parsing functions for the graph for obtaining entity neighbours?

No need to load the model back to an application if you don't want to further finetune the model. You can directly access model.solver.entity_embeddings or model.solver.relation_embeddings.
app doesn't have a direct interface to the graph. You need to parse the original training file to manipulate the graph structure.
For example, a minimal parsing code may look like
triplets = []
with open(gv.dataset.wikidata5m.train, "r") as fin:
    for line in fin:
        triplets.append(line.strip().split("\t"))

The entities and relations are encoded in as "Qxxxx" and "Pxxxx" respectively. To access their corresponding embeddings, please use model.graph.entity2id and model.graph.relation2id to convert them to indexes.


tighten hamiltonian application a bit

So I played around with avoiding allocations a bit, it doesn't actually change much in terms of performance. Still the f_real .= 0 and f_fourier .= 0 show up on profiling, but I wonder where it really comes from (the OS figuring out we actually do need the memory, probably). I think calloc is faster than malloc+bzero, and so we might benefit from changing the f_real = similar(); G_to_r!(f_real, ...) pattern to something like f_real = zeros(); G_to_r!(f_real, ...; zeroed_output = true). I haven't done that yet because I think we should clear up the FFT functions first, and because I can't find something generic that does what similar does but also allocates zero. Might just special-case to Array for a start if there's an actual performance difference.

This PR is just saving an alloc. @mfherbst for this kind of commits, is it OK with you if I pass the tests locally and just merge directly into master, or do you prefer PRs? Happy to do either.

Hah, so I took the trouble to avoid the fill by using zeros, only to figure out that it actually allocates an uninitialized array and then calls fill!. This is julia issue 130.

It does seem to make a (small) difference though, so let's wait until 130 is fixed and revisit this. Apart from this there isn't much more to be gained in this kind of micro-optimization (in particular saving allocs is basically useless as far as runtime is concerned).

This PR is just saving an alloc. @mfherbst for this kind of commits, is it OK with you if I pass the
tests locally and just merge directly into master, or do you prefer PRs? Happy to do either.
Generally I prefer PRs, because it's sometimes surprising what works on your computer, but not somewhere else. For fixing a typo in the docs or a comment or so it's of course a different thing.


Glob path works strange

Hi folks,
I have an issue with unit tests when I run the tests with:
helm unittest -f tests/**/*_test.yaml
My folder structure looks like this (I want to put the yaml files in a separate folder for each configuration aka default, test, dev, prod, ...):

So, when I execute it, I got this:

task: [unittest] helm unittest -f tests/**/*_test.yaml .
Error:  file 'tests/default/service_account_test.yaml' seems to be a YAML file, but expected a gzipped archive
Error:  file 'tests/default/service_test.yaml' seems to be a YAML file, but expected a gzipped archive
Chart [ bla bla ] .
PASS  test default deployment  tests/default/deployment_test.yaml
Charts:      2 failed, 2 errored, 1 passed, 3 total
Test Suites: 1 passed, 1 total
Tests:       5 passed, 5 total
Snapshot:    0 passed, 0 total
Time:        113.48075ms
Error: plugin "unittest" exited with error
task: Failed to run task "unittest": exit status 1
task: Failed to run task "unittest": exit status 1

It seems that the glob pattern is faulty. How can I fix that? Or is that a GO behavior?
Thx

I have solved the issue by myself.
It's important to surround the path with quotes...
Now it's working.

@pmoscode could you please give an example? I am facing the same issue.

@swathinsankaran
Sure, for me worked:  helm unittest -f 'tests/**/*.yaml' .
Like I mentioned above, I used single quotes around the path to get it working.

Thanks @pmoscode for the reply.
I ended up adding the below docker command to solve the issue.
helmtests:
	sudo docker run \
	-v $(PWD)/helm/ako:/apps \
	-v $(PWD)/tests/helmtests:/apps/tests \
	helmunittest/helm-unittest:3.11.1-0.3.0 .

Perhaps it's worth to mention, that my I'm using a Mac for development. So, perhaps my solution isn't working on other operating systems... I didn't check.

Hello @swathinsankaran,
Based on your docker command it seems you are retrieving the test from a folder which is not part of the helmchart itself.
Although the glob path can work like that, the current implementation starts the globbing with the helmchart as basepath.
It seems you are looking for a feature similar to #186.
Greetings,
@quintush


#184, Added Identity switch; Added ParameterSets; Configured default …

…for Graph to make non-breaking; Added Azure
This provides a framework for beginning to add additional APIs as part of the connection process. Ref option 2 in #184. For ref option 1, I will build off this.

If this looks good, I can update the MDX files too before merge.

I think this one still needs some work, so I am closing for now until I can test further.


appear the windows is not responding, but I still can use it.

but I can use it success.
vscode version 1.77
install only C/C++ v1.14.5 extension

but I can use it success.

This means VS Code hangs only temporarily.

install only C/C++ v1.14.5 extension

Please could you disable the C++ extension and try again,
If the issue doe not exist with C++ extension disabled, then please file an issue on the C++ extension as its caused by that extension
If you still run into this issue then Please see https://github.com/Microsoft/vscode/wiki/Performance-Issues for creating actionable performance problem issues.
Once you have that information please feel free to re-open this issue or ping here and we'll re-open this, optionally please feel free to create a new issue
/label info-needed

Hi. I have the same issue. Usually, it takes only a second or two to open vscode in my system. Since yesterday, it takes atleast 15-30 seconds to open. And after it opens, "The window is not responding dialog is shown". I was able to close that dialog and start using vscode, but it has become so slow that it seriously affects my workflow. I'm searching for a solution for hours. Please help.


Release v0.6.3

Release proposal https://github.com/steemit/steem-js/releases/tag/v0.6.3

@sneak all changes are visible on the release doc here: https://github.com/steemit/steem-js/releases/tag/v0.6.3 . I will create a branch staging so next update will includes changes directly on the PR by merging staging to master.


Handling templates

This boilerplate looks really cool, so big thumbs up for it! :) I have one question, how to handle loading templates from a file (I'm not a big fan of inline templates). When I try to do import template from './template.html' I'm getting ts error, that it didn't find any module there.

Hey I'm glad you like our scaffold 🙂
Think is that we are kind of fans of inline templates a it's because this way you tent to keep components small and simple.
Anyway if you really really don't want to go inline you can either upgrade to typescript 2 and use declare module "*.html" or simply use require("./yourfile.htm").
If you will upgrade to typescript 2 PR is welcome 😉

It would be super neat if the templateUrl syntax worked, as per the angular 2 docs.
@Component({
  moduleId: module.id,
  selector: 'toh-heroes',
  templateUrl: 'heroes.component.html'
})

You mentioned, declare module "*.html", although this does not work, Something is missing from the syntax?

@blowsie another problem with templateUrl in angular 1 is that it make your component async, quite frankly I'm not sure how does that work in angular 2 as I said we prefer inline 😉
Trick I was mentioning is called wildcard module declarations I'm not sure what is the problem I would check typescript version you are using and related typescript thread https://github.com/Microsoft/TypeScript/issues/6615
It should definitely work in our scaffold see https://github.com/ngParty/Angular1-scaffold/commit/6e91cfd8e938d97a8879a9bd5e13158e17b6fe68

Has anyone tried using angular2-template-loader with ng-metadata? That would load both templates and style url automatically.

@blowsie

It would be super neat if the templateUrl syntax worked, as per the angular 2 docs.

@Component({
moduleId: module.id,
selector: 'toh-heroes',
templateUrl: 'heroes.component.html'
})
You mentioned, declare module "*.html", although this does not work, Something is missing from the syntax?

this works, but with SystemJS :) (module.id is specific for systemJS )
https://github.com/ngParty/ng-metadata/blob/master/playground/app/components/lifecycle/do-check.component.ts#L80

@aciccarello

Has anyone tried using angular2-template-loader with ng-metadata? That would load both templates and style url automatically.

I haven't


public API for creating/accessing subsets

This will likely need to be addressed by an upstream PR to jdaviz.

related upstream jdaviz PR: https://github.com/spacetelescope/jdaviz/pull/3104


Generated interfaces for event messages or command messages should extend corresponding interfaces

When I write something like in an events.proto file:
message OrderCreated {
    option (is) = { java_type: "OrderEvent", generate: true };
    string order_id = 1;
}

I expect that the generated interface OrderEvent implements EventMessage since we are in the events.proto file and all those types are EventMessages by convention.
Instead, the generated code simply extends Message.

This might be the desired behaviour.
In the example, OrderCreated implements both OrderEvent and EventMessage.
If a user wants OrderEvent to extend EventMessage, they should create the interface manually.

As discussed, leaving current behaviour as is.


KeyError: 215.0

I try to run your h5 model on [Ver 2.0] Super Chamber Piano and i got this error

Thank you for your work

@EKGD
Thank you for your interest and for bug reports.
This repo had been depreciated long time ago so some stuff may not work right.
Please refer to https://github.com/asigalov61/asigalov61 for the list of current/supported projects.
Thank you.
Alex.


Move OpenAPI schemas into Python package

Fixes #1093
Changelog

Add OpenAPI schema under mlserver.rest.openapi package as symbolic links (i.e. files in root repo remain source-of-truth)

Closing PR to re-open from master


Explain a subtlety of client handling for "download" batch action

This may not be worth pulling or the best method or place to express the idea, but the behavior wasn't self-evident to me from the documentation while implementing an LFS server, even though it makes sense now that I understand the interaction.

Looks good to me :+1:

Hmm, I'm not sure this is correct, it implies that the server must respond with a download link when the operation is upload in order for the upload to be skipped, when in fact simply omitting the upload link in the response does that. You don't have to provide a download link instead and doing so makes no difference vs omitting - git-lfs just doesn't find a link of type upload in both cases. I don't think we should be telling server implementors that they have to provide a download link in upload requests.

Ah, I must have misunderstood the prePushCheckForMissingObjects() / NewDownloadCheckQueue() interaction -- and, in retrospect, my test case would have worked for either behavior. I'll fix this.

#1087 reverts this and properly attributes the behavior to the upload action instead. Sorry about that!

Thanks for the fast turnaround!


Musical notes need some explanation

[Enter feedback here]
I think that the musical notes animation needs some explanatory text about what the musical notes actually represent.

Is this referring to writes of different documents or writes to the same document?
If it is referring to different documents then does this apply cross partition? I imagine that different physical partitions can have different replication lags

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

ID: 2b3fb7da-5483-160d-1781-d4abff9d798d
Version Independent ID: d5e51b7e-7f34-c7df-ce4f-ee4bf5c8059a
Content: Consistency level choices - Azure Cosmos DB
Content Source: articles/cosmos-db/consistency-levels.md
Service: cosmos-db
GitHub Login: @seesharprun
Microsoft Alias: sidandrews

Thank you for your feedback, @asos-martinsmith
We considered your feedback along with other actions we are taking to improve documentation. At this time, we have elected to prioritize other documentation improvements and cannot act on this feedback. For now, we are closing this issue. If you feel this in error, please open a new issue.
#please-close


"Database upgrading" Forever!!!

The k9mail client does not go beyond "Database upgrading" screen even after a day! This is similar to https://github.com/k9mail/k-9/issues/1947 but still prevalent in v5.205 and also v5.206 after upgrading k9mail from fdroid repo.
Expected behavior
Should display the email list in the k9mail client
Actual behavior
After an update of the k9mail client, it stops working with only displaying "Database updating..." forever. Screenshot at http://picpaste.com/geeDAaIT.png
Steps to reproduce

Upgrade to v5.205 or v5.206 of k9mail client from fdroid repo

Environment
K-9 Mail version: v5.205 and v5.206
Android version:
Account type (IMAP, POP3, WebDAV/Exchange): IMAP

It's a duplicate. We didn't close it because it's not fixed.
Closing this.


Remove type refinements section

We recently devoted significant effort to get this in a pretty good state where it works intuitively. As a result, this section is outdated. I'm deleting this because I think it's redundant to explain the same thing that intuition would also say.

I would feel better if we replaced this section instead. It's not obvious that type refinements even exist (esp. to users of programming languages without gradual typing); we recognize specific ways to refine a type with some limitations (for example you can't refine the type to be a function right now); etc.
So I would rather replace this section with up to date information.

You can refine a type to be a function if the thing being refined beforehand had function types now. 🙂 You just can't pick the one particular function type from the union.
But sure, I'll replace this section instead.


deploy command not working on Apple Silicon

Using version 1.1.4
Describe the bug
When running swa deploy on my Apple Silicon Macbook I get the following error:
✖ Deployment Failed :(
✖ Deployment Failure Reason: spawn Unknown system error -86

Desktop (please complete the following information):

OS: MacOS
Version: Ventura 13.5

Additional context
Are there plans for the deploy command to support Apple Silicon based Macbooks?

I'm having the same issue

Same issue here

Same issue here .. there is any workaround or resolution?

Good morning all!
-86 usually means a mismatch between the binary architecture and your system CPU architecture.
I was having the same issue on my M3 and solved it by installing Rosetta.
softwareupdate --install-rosetta

Hope it helps!


[FLINK-2558] Add Streaming Connector for Elasticsearch

This adds an ITCase, an example and documentation.

I'm currently in the process of setting up a cluster to test it with an actual Elasticsearch cluster. Once I have the results of that I'll post them here.

I'm happy to report that I managed to set up a distributed Elasticsearch cluster and the sink did work with it :smile:

This looks pretty nice :-) Docs, tests, good comments, cluster tests!
+1 to merge this!

Added two minor comments on the docs/interface, but treat those as optional, they are and matter of taste anyways.

I updated the doc, if no-one objects I would like to merge this tonight.


Bump dependencies to pull in security patches

Description
Updates indirect dependencies that have vulnerability alerts from dependabot.

TODOs
Read the Gruntwork contribution guidelines.

[x] Update the docs.
[x] Run the relevant tests successfully, including pre-commit checks.
[x] Ensure any 3rd party code adheres with our license policy or delete this line if its not applicable.
[x] Include release notes. If this PR is backward incompatible, include a migration guide.

Release Notes (draft)

Updated transient dependencies to pull in security patches.

Thanks for review! Merging now.


Complex mesh for electrostatics

This should make a nice test case for large scale 2D simulation, have a look here
https://en.wikipedia.org/wiki/Comb_drive

The mesh is complex to do in Gmsh, so SALOME will be used.

![Screenshot from 2024-02-08 15-47-34](https://github.com/arcaneframework/arcanefem/assets/52162083/05373ba4-6a02-4726-b84c-a62d3

33bcdcf)


Add explanation of "standard" and "environmental" sensor values to docs

The library returns two sets of concentrations: "standard" and "environmental" but the docs don't explain the difference between them. After some searching, I figured out that "standard" refers to what the concentration would be at standard pressure (i.e. sea level) whereas "environmental" refers to measured values that depend on ambient pressure. It would be nice to add this info to the docs.
(There's a discussion about this here: https://publiclab.org/questions/samr/04-07-2019/how-to-interpret-pms5003-sensor-values)

ooh thats useful info!
@kattni plz add to learnguide / readme :)

Added to Learn guide: https://learn.adafruit.com/pm25-air-quality-sensor/usage-notes#standard-vs-environmental-concentration-3140491


Update debugging-absolute-beginners.md

(You can replace all of this text with your description.)
Before creating your pull request, please check your content against these quality criteria:

Did you consider search engine optimization (SEO) when you chose the title in the metadata section and the H1 heading (i.e. the displayed title that starts with a single #)?
For new articles, did you add it to the table of contents?
Did you update the "ms.date" metadata for new or significantly updated articles?
Are technical terms and concepts introduced and explained, and are acronyms spelled out on first mention?
Should this page be linked to from other pages or Microsoft web sites?

For more information about creating content for docs.microsoft.com, see the contributor guide at https://docs.microsoft.com/contribute/.

@HeMinzhang : Thanks for your contribution! The author, @mikejo5000, has been notified to review your proposed change.

@HeMinzhang : Thanks for your contribution! The author, @mikejo5000, has been notified to review your proposed change.


Website and api not updating

Where is the right place to report web and api update issues?
It seems both showing old data now, this screenshot was taken at 6pm.

yep - this is the right place! it's up to date now but i'll have a look at where the delay came from

we've added some healthchecks on the API endpoint but I believe this may have been a delay on some of our data sources - we're handling this a bit better #266
Thanks again for reporting this


Functional Tests need in-VR description

Implement a system that adds a multi-line description for each functional test.
Fill it out explaining the test's purpose, how to manually execute test, and pass/fail criteria.

Deferring in-vr descriptions for the moment, descriptions of current Functional Tests have been collated in and published here in packages\functional-tests\docs\README.md


Support alterntive combination of notes

The syntax C3002xx could mean:
E|-3
B|-0
G|-0
D|-2
A|-x
E|-x

Originally, I wrote this as L3002xx but I don't remember what the L is for ("direct (L)ist"?).
The C would be for Chord but I think this creates an odd parsing rule where, if it starts with C, it might still be a set of notes together or it might be a C chord, like C or C#. Avoiding the parsing problem can be achieved by using any character that isn't a chord letter. L achieves that and is also readable. T also works.

Done.


Refactor to use Hikari

With discord.py development being ceased, Hikari seems the sensible library to migrate to.

Closing for now with the resumption of discord.py development.


Disable failing Windows CyCache test

See #5825 for a description of the failure.

I'll merge this fairly quickly unless there are major objections, since I didn't realize it was still failing when I removed allow_failures for 3.12.
I believe it's a problem local to the CI since the test works on my laptop.
Needs backporting to 3.0.x since I also removed allow_failures for 3.12 there

Fine with me.

3.0.x 16d31ef77


Stretch module to display border

My display is smaller than my mirror. Between the display border and the module border is a lot of unused space on the display. Is there a way to let the modules float to the display border?

you have to put the following lines of code into ~/MagicMirror/css/custom.css
body {
  margin: 0;
  height: 100%;
  width: 100%;
}

Thanks!

Hi @micherlinman, Please close the issue.
Thanks.