WIP: Events program

Opening a ticket to track development of an events plan

We should have goal and principles around a plan for events that includes both online (dev meetings, brownbags) and offline (meetups, conferences), etc.

High level:

There's an opportunity to accelerate engagement with our communities
We're probably not doing enough to organize and attend meetups around Mattermost
We should be answering more "call for proposals/speakers" for events, particularly OSCON
We should open a separate ticket to think about training programs, with conferences as part of an overall program. Community Leadership Summit should be on the list

This is covered in: https://handbook.mattermost.com/contributors/contributors/guidelines. :) Closing.


Git describe is inaccurate

Issue: git describe is inaccurate, because the last few tags do not point to any commits on master.
Expected behavior: git describe should describe the master branch with the latest tag (as of writing it should look like this v0.3.9-X-XXXXXXX).
Actual behavior: Instead git describe looks like this: v0.3.7-beta2-35-geefa28326
Steps to reproduce: pull from the master branch and run git describe
I am maintaining the git version of onivim on the AUR (https://aur.archlinux.org/packages/onivim-git/).
This makes it difficult to automate the versioning of the package.

One way to deal with this is to "merge" the dangling tags back into master, but revert all of their changes in the merge-commit. So the tags will be reachable from master without actually needing to bring potentially irrelevant changes into master. Although I don't know if git describe would find tags that aren't on the master "mainline".


Fix RuntimeError: dictionary keys changed during iteration

Changelog: Bugfix: Fix RuntimeError: dictionary keys changed during iteration
fixes #494

[X] Refer to the issue that supports this Pull Request.
[ ] If the issue has missing info, explain the purpose/use case/pain/need that covers this Pull Request.
[X] I've read the Contributing guide.
[X] I've followed the PEP8 style guides for Python code.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Jérémy Coulon seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Codecov Report

Merging #496 into develop will decrease coverage by 0.02%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop     #496      +/-   ##
===========================================
- Coverage    92.69%   92.66%   -0.03%
===========================================
  Files           34       34
  Lines         3559     3560       +1
===========================================
  Hits          3299     3299
- Misses         260      261       +1

Impacted Files
Coverage Δ

cpt/packager.py
86.39% <100.00%> (+0.02%)
:arrow_up:

cpt/ci_manager.py
92.53% <0.00%> (-0.42%)
:arrow_down:

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 3d05949...0b73b8c. Read the comment docs.


Integrate game state and views

Use relative path base on the python file with pathlib instead of hard
coded separator. Create BasePage class to unify the interface between
each view. Modify main loop to allow page switching.
What left to be done:

[x] Integrate GameState and views together.
[x] Potentially rename the folder so that the package can be imported into python. - is not a valid character for a module name.
[x] Fix the test case for the GameState.
[x] Fix linter errors.

well, still have some bug :p


remove duplicate import with different name

k8s.io/kubernetes/pkg/api is imported directly and once with the alias import name kapi so remove the original import and keep the aliased with kapi one and also replace the code bits which call it with api name.

LGTM.

@cdrage you will have to approve it to merge it!

@surajssd Got to wait for one more approval.
@surajnarwade @kadel @containscafeine @rtnpro ?

@cdrage thanks :+1:

@surajssd Try to wait until 2 people approve it :+1: before any merges :)

@cdrage sure ! I thought this is smaller one, with mainly name changes so merged it!


Parsing error if stylesheet url is wrong

I had a misspelling in the asset url:
<Style url="/assets/app.swiftu.styles"></Style>
instead of "swiftui"
and this resulted in the client erroring:

Does this work:


vscode in WSL2 attempts to access proxy in host without interpreting the address properly

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.61.0
OS Version: Windows 10

Steps to Reproduce:

Set "http.proxy" in settings, which should be a loopback address.
Reinstall c/cpp extension.
Wait for the extension to start downloading its internal packages.
In the "Output", the extension returns "Failed to establish a socket connection to proxies", where the address it tried to access is localhost.

I reckon that vscode doesn't realize the network domain of WSL2 is different from that of the host, not like WSL1.

Isn't typically http.proxy the address of a machine in the network, accessible also from WSL2?
What's the point to set it to a loopback address?
What do you expect VS Code to do in WSL? Ignore the setting?

@aeschli or it could treat WSL2 as a remote machine, allowing it set the value differently.


Hyperion only works with EXACTLY 50 leds

Hello, I have been using boblight and have decided to switch over to hyperion. I am running OSMC on a pi2 with a ws2812b led strip controlled via a arduino uno. As the headline says, I can get hyperion to work correctly, but only with 50 leds (0-49) and I need 212. I have tried everything I could think of in the /etc/hyperion.config.json file: editing the number of "indexes", changing the "id" and "leds", playing with "smoothing", editing the other config file in /opt/hyperion/config/... I'm out of ideas.
My files:
[Hyperion]
(https://drive.google.com/file/d/0B6h86LY88C4FLXZUbXl6VzBVZk0/view?usp=sharing)
[Arduino Sketch]
(https://drive.google.com/file/d/0B6h86LY88C4FV1otYmgzRDd3dDg/view?usp=sharing)
The hyperion file does not work with the exact configuration in the file above, but by deleting the "extra" led's so there are only 50 and not changing anything else does work. Doing this, however, essentially makes ~4 repeating patterns in the led strip, separated by 2 leds, since the arduino sketch calls for 212.

Google Drive says: "You need permission"

Same problem with Google Drive, but do you have modified Arduino sketch to manage 212 LEDs ?

Sorry about the links, they should be working now.

What is your physical LEDs configuration ? What is your power supply ? How many power point do you have ?

I have a 5V AC adapter powering the led strip directly. The Pi is powered from the usb port on my tv. Everything worked perfectly fine with boblight, so nothing external should be an issue. I must just be missing something in either the hyperion config or the arduino sketch.

Your Arduino code has the number of leds fixed (in the prefix) at 0x31 (+1) which is 50 in decimal. You should modify the prefix to have the correct number (or remove checking the last few bytes of the prefix).

I ended up finding another arduino sketch that worked with hyperion and any number of led's. Thank you though, @tvdzwan - your suggestion should work as well.

If its solved, close the issue.


when i install with requirements.txt cvxpy>=1.0.21,log   error in cvxpy setup command: use_2to3 is invalid.

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
A minimal code example that reproduces the bug or unexpected behavior.
Expected behavior
A clear and concise description of what you expected to happen.
Output
If applicable, include program output. If reporting a program crash, please include the entire stack trace.
Version

OS:
CVXPY Version:

Additional context
Add any other context about the problem here.

@Wangwuyi123 the title of the issue says you are trying to install cvxpy>=1.0.21, but in the screenshot, it shows cvxpy==1.0.21. Allowing for more recent versions would likely fix the issue.
If you need the exact version there are some workarounds you could try, but I suggest trying to upgrade first.

Hi @Wangwuyi123. I've changed the title of your issue based on Phillipp's comment. My guess is that changing to only require cvxpy>=1.0.21 will fix your issue. I especially encourage you to upgrade to cvxpy 1.1 as well, since we don't offer installation support for cvxpy 1.0.x anymore. Please post here again if you run into trouble with cvxpy 1.1!


Upgrade maven-jar-plugin 3.2.2 -> 3.4.1

This PR contains the following updates:

Package
Type
Update
Change

maven-jar-plugin
build
minor
3.2.2 -> 3.4.1

[ ] If you want to rebase/retry this PR, check this box

Warning
Renovate's suggested commit message is being replaced with improved initial commits to enable automerging. As a side effect, these suggested commit messages might have changed. Consider comparing the initial commit message and this message to determine the most suitable one. Please leave feedback in #sys-renovate.

Suggested commit message:
Upgrade maven-jar-plugin 3.2.2 -> 3.3.0

See:
- https://github.com/apache/maven-jar-plugin/releases/tag/maven-jar-plugin-3.3.0
- https://github.com/apache/maven-jar-plugin/compare/maven-jar-plugin-3.2.2...maven-jar-plugin-3.3.0

Edited/Blocked Notification
Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.
You can manually request rebase by checking the rebase/retry box above.
⚠ Warning: custom changes will be lost.


Firefox: use product version and differentiate channels

A Firefox release has two version numbers: for example <IP_ADDRESS>03 (I think this is the "build version") and 46.0a2 (product version). If win-detect-browsers reads the product version (with Folder.GetDetailsOf(file)) in addition to the file version, it can differentiate FF channels.
Based on Firefox release history and some Mozilla wiki pages, I wrote a simple function. Works for all product versions, from 0.1 to the latest. See the same gist for a XUL application I wrote for fun.
Tasks:

[x] Read product version
[ ] Find a good abstraction and place for browser-specific differentiation logic

The 3.0 branch now reads almost all version information from an executable:
> win-detect-browsers --verbose --lucky firefox
Found 1 browsers in 25ms
{ name: 'firefox',
  path: 'C:\\Program Files (x86)\\Mozilla Firefox\\firefox.exe',
  version: '<IP_ADDRESS>84',
  info:
   { FileVersion: '<IP_ADDRESS>84',
     LegalCopyright: '©Firefox and Mozilla Developers; available under the MPL 2 license.',
     CompanyName: 'Mozilla Corporation',
     FileDescription: 'Firefox',
     ProductVersion: '44.0.2',
     InternalName: 'Firefox',
     LegalTrademarks: 'Firefox is a Trademark of The Mozilla Foundation.',
     OriginalFilename: 'firefox.exe',
     ProductName: 'Firefox',
     BuildID: '20160210153822' } }

For firefox, each result gets a channel property, one of alpha (pre firefox 5), nightly, aurora (pre firefox 35), developer, beta, rc (pre firefox 5) or release.


Given and family names are reversed.

This definitely happens to some instances of "Bruce Bray", but only some of them. :)
In all contributor lists for all delivered artifacts, the first (given) and last (family) names of the contributors are reversed. E.g. for Bruce:
https://github.com/preston/vha-kbs-knarts/blob/612e893c65226b306a71c1b9869c39f53554a3af/content/cardiology_ep_arrhythmias_devices/b7/CDSK_KRprt_OS_B7EPSVT.XML#L478
Reversal of Angela Denietolis in o16.
Reversal of Joseph King in b39, b16, b40.
Reversal of Scott Wall in b12, b8, b7, b6, b36.
Another example from #212:
https://github.com/preston/vha-kbs-knarts/blob/7437cae9d3b455eb2795663ff53ac5331e9240f1/content/neurosurgery_cervical_radiculopathy_clinical_white_paper/b15/CDSK_KRprt_OS_B15CervRadi.xml#L199

Should now be consistent across all artifacts. All current artifacts have been revalidated.


[Async] Add a caching map Stream

The same as map: but it should cache the mappings in the provided dictionary


Resolve HTML Linter Bug

Resolve HTML Linter Bug
The linter occasionally autoformats the HTML in a way that produces an error in the code. One instance of this occurring is in the Organization Details Info Card Widget HTML file around line 35.
Fixed Line of Code:

After Save & Linting:

Why?
This must be resolved to ensure that the code may be compiled.
Acceptance Criteria

[ ] Update the linter to ensure that it autoformatting does not cause errors.

@KrisJordan, any ideas about this? Seems like it was just a bad idea to use ternary in this case - wondering if I can close this issue.

I wonder if we can get the max length for the auto formatter and the linter to be equal, @atoneyd ? Otherwise, closing is ok. Probably should not be ternary, agreed.


Set up AWS company account and claim SUS $1k credits

Submitted and waiting for their approval. In the meantime, we have the normal free tier to use.


Wheel pkg: don't static link libstdc++.

Codecov Report

Merging #91 (1bcabdf) into main (74c574e) will decrease coverage by 14.93%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main      #91       +/-   ##
===========================================
- Coverage   76.72%   61.79%   -14.94%
===========================================
  Files          52       52
  Lines        3785     3816       +31
===========================================
- Hits         2904     2358      -546
- Misses        881     1458      +577

Impacted Files
Coverage Δ

src/server/util/etcd_launcher.cc
5.12% <0.00%> (-72.75%)
:arrow_down:

src/server/services/meta_service.cc
28.76% <0.00%> (-49.02%)
:arrow_down:

src/server/server/vineyard_server.h
60.00% <0.00%> (-40.00%)
:arrow_down:

src/server/server/vineyard_server.cc
41.58% <0.00%> (-32.55%)
:arrow_down:

src/server/services/meta_service.h
47.21% <0.00%> (-28.60%)
:arrow_down:

src/server/util/meta_tree.cc
53.33% <0.00%> (-22.86%)
:arrow_down:

src/server/async/socket_server.cc
65.74% <0.00%> (-18.90%)
:arrow_down:

src/common/util/protocols.cc
75.91% <0.00%> (-15.80%)
:arrow_down:

src/server/services/etcd_meta_service.h
80.00% <0.00%> (-12.00%)
:arrow_down:

src/server/memory/stream_store.cc
56.07% <0.00%> (-8.42%)
:arrow_down:

... and 3 more


monitor-list empty

Hello everyone,
OS: Fedora31
xfreerdp: This is FreeRDP version 2.0.0-dev5 (n/a)
Two issues:
1.- xfreerdp /monitor-list : outputs empty
2.- xfreerdp /v:my_vdi /d:my_domain /g:mycompany.contoso.com /u:myUser /gu:myGatewayUser /gd:myGatewayDomain /sec:rdp :
ask for my password, then i have 2FA using mobile app, then when i authorize, should ask again for my password, which is not, and then i have this error:
freerdp_tcp_connect:freerdp_set_last_error_ex resetting error state
[ERROR][com.freerdp.core.gateway.rdg] - channel response errorCode=E_PROXY_RAP_ACCESSDENIED [0x800759DA], fieldsPresent= [0000]
[ERROR][com.freerdp.core] - rdg_process_channel_response:freerdp_set_last_error_ex E_PROXY_RAP_ACCESSDENIED [0x800759DA] [0x800759DA]
Using Remmina im able to access, but not using xfreerdp
Is there anything im doing wrong?

Please try with our nightly builds first.
Not much we can help here without further setup descriptions.
Please use mailing list or IRC for support questions. https://github.com/FreeRDP/FreeRDP/wiki/BugReporting


Error: The issuer 'https://login.microsoftonline.com/{TENANT-ID}/v2.0' is invalid"

I have followed the instructions in the document.
When I try to run the example

I'm able to to login to the sample application
I have checked the returned authorization token, and in the issuer field contains https://login.microsoftonline.com/xxxxxxxxxxx/v2.0 (where xxxxxx is my tenantid)
But, when I try to call the Wheather page, I get the error
401 The issuer 'https://login.microsoftonline.com/{TENANT-ID}/v2.0' is invalid"

I have checked in internet, and I have found similar issues in other scenarios (e.g. Angular), but I don't understand the root cause of the error
UPDATE:
I have workarounded the issue forcing the validate issuer in Server application

Edit Startup.cs
Add the IssuerValidator option to ConfigureServices (change xxxxxxx for your tenantid)

		public void ConfigureServices(IServiceCollection services)
		{
			services.AddAuthentication(AzureADDefaults.BearerAuthenticationScheme)
					.AddAzureADBearer(options => Configuration.Bind("AzureAd", options));

			services.AddControllersWithViews();
			services.AddRazorPages();
			services.Configure<JwtBearerOptions>(
				AzureADDefaults.JwtBearerAuthenticationScheme, options =>
				{
					options.TokenValidationParameters.NameClaimType = "name";
					options.TokenValidationParameters.ValidIssuer = "https://login.microsoftonline.com/xxxxxxxxxx/v2.0";
				});
		}

Detalles del documento
⚠ No edite esta sección. Se requiere para docs.microsoft.com ➟ Vinculación de problema de GitHub.

ID: 3039ff79-ee86-20b1-0082-d3899a84c823
Version Independent ID: af4a3a75-ae52-5f6d-2120-737bc2fefb07
Content: Protección de una aplicación hospedada Blazor WebAssembly de ASP.NET Core con Azure Active Directory
Content Source: aspnetcore/blazor/security/webassembly/hosted-with-azure-active-directory.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

Hello @ClimberBear ... We don't have the staffing to provide much product support, especially right now due to a high volume of priority issues. We can only react to specific doc issues at this time, and there are no reported specific problems with the topic. I recommend that you re-walk the steps carefully creating a new sample app and compare that along with the topic's sections that show the app's configuration to your app. Usually, a difference in the configuration can be found. If you need product support, you can reach out to devs on the usual support forums for help ...

Stack Overflow (tag: blazor)
ASP.NET Core Slack Team
Blazor Gitter

Good morning. I cannot agree with your decision of closing the issue. I can understand triage it, down priority, postpone or ask for clarification or simple sample, but closing a bug with the comment "we don't have resources to investigate" is, imho, no professional. Why? Because now the other people will no able to find it with standard filters or, worse, will think it is solved.  If you check the workaround (and now I understand how OAUTH2 works), the problem is that the default implementation (with no customization) is not expanding the variable {TENANT-ID}.

closing a bug

We only work bugs in the topics, not in developer applications or in the framework.

the problem is that the default implementation (with no customization) is not expanding the variable

Ah! ... That sounds like you found a bug in the framework. There are two buttons at the bottom of the topics, one to open for us for the document ... one to open a product issue, where they show their repo issue forums (https://github.com/dotnet/aspnetcore/blob/main/CONTRIBUTING.md#bugs-and-feature-requests). If you select the button for a product issue, it gets you to the right spot to report a problem with the framework. Open your issue here .......
https://github.com/dotnet/aspnetcore/issues
The reason that I didn't recommend this in the first place was that I misinterpreted your issue. I thought that you were just reporting a configuration problem with the app. Sorry that I didn't pick up on that. 🙈 Open your issue with the product unit, and they'll investigate. Give them steps to reproduce the issue.

Thank you for the quick answer. I will do this way. In my own defense, I have to say that I opened the issue in generic, and the bot moved/tagged. I'l try the link you tell me. Thanks again for your patiente.

Open as framework bug in https://github.com/dotnet/aspnetcore/issues/29874


NameError in stencil when cval is np.nan or np.inf

I was playing around with the stencil decorator and tried supplying np.nan to the cval argument. This results in a NameError. It looks like the type inference is not able to infer a type for np.nan. The docs state that any value can be supplied to cval so I assume this is a bug. The same thing happens for np.inf. The below code triggers the error on my machine with version 0.53.1. If this is intended, perhaps the documentation could be updated to indicate that NaNs and infs are not valid.
import numpy as np
from numba import stencil

@stencil(neighborhood=((-1, 1), (-1, 1)), func_or_mode="constant", cval=np.nan)
def kern(a):
    return (
        a[-1, -1]
        + a[-1, 0]
        + a[-1, 1]
        + a[0, -1]
        + a[0, 0]
        + a[0, 1]
        + a[1, -1]
        + a[1, 0]
        + a[1, 1]
    )

x = np.random.randint(100, size=(100, 100)).astype(float)
y = kern(x)

Here is the stack trace:
Traceback (most recent call last):
  File "stencil.py", line 21, in <module>
    y = kern(x)
  File "/home/$USER/anaconda3/envs/main/lib/python3.8/site-packages/numba/stencils/stencil.py", line 763, in __call__
    new_func = self._stencil_wrapper(result, None, real_ret, typemap,
  File "/home/$USER/anaconda3/envs/main/lib/python3.8/site-packages/numba/stencils/stencil.py", line 728, in _stencil_wrapper
    new_func = compiler.compile_ir(
  File "/home/$USER/anaconda3/envs/main/lib/python3.8/site-packages/numba/core/compiler.py", line 669, in compile_ir
    return pipeline.compile_ir(func_ir=func_ir, lifted=lifted,
  File "/home/$USER/anaconda3/envs/main/lib/python3.8/site-packages/numba/core/compiler.py", line 363, in compile_ir
    return self._compile_ir()
  File "/home/$USER/anaconda3/envs/main/lib/python3.8/site-packages/numba/core/compiler.py", line 422, in _compile_ir
    return self._compile_core()
  File "/home/$USER/anaconda3/envs/main/lib/python3.8/site-packages/numba/core/compiler.py", line 395, in _compile_core
    raise e
  File "/home/$USER/anaconda3/envs/main/lib/python3.8/site-packages/numba/core/compiler.py", line 386, in _compile_core
    pm.run(self.state)
  File "/home/$USER/anaconda3/envs/main/lib/python3.8/site-packages/numba/core/compiler_machinery.py", line 339, in run
    raise patched_exception
  File "/home/$USER/anaconda3/envs/main/lib/python3.8/site-packages/numba/core/compiler_machinery.py", line 330, in run
    self._runPass(idx, pass_inst, state)
  File "/home/$USER/anaconda3/envs/main/lib/python3.8/site-packages/numba/core/compiler_lock.py", line 35, in _acquire_compile_lock
    return func(*args, **kwargs)
  File "/home/$USER/anaconda3/envs/main/lib/python3.8/site-packages/numba/core/compiler_machinery.py", line 289, in _runPass
    mutated |= check(pss.run_pass, internal_state)
  File "/home/$USER/anaconda3/envs/main/lib/python3.8/site-packages/numba/core/compiler_machinery.py", line 262, in check
    mangled = func(compiler_state)
  File "/home/$USER/anaconda3/envs/main/lib/python3.8/site-packages/numba/core/typed_passes.py", line 104, in run_pass
    typemap, return_type, calltypes, errs = type_inference_stage(
  File "/home/$USER/anaconda3/envs/main/lib/python3.8/site-packages/numba/core/typed_passes.py", line 80, in type_inference_stage
    infer.build_constraint()
  File "/home/$USER/anaconda3/envs/main/lib/python3.8/site-packages/numba/core/typeinfer.py", line 1024, in build_constraint
    self.constrain_statement(inst)
  File "/home/$USER/anaconda3/envs/main/lib/python3.8/site-packages/numba/core/typeinfer.py", line 1371, in constrain_statement
    self.typeof_assign(inst)
  File "/home/$USER/anaconda3/envs/main/lib/python3.8/site-packages/numba/core/typeinfer.py", line 1442, in typeof_assign
    self.typeof_global(inst, inst.target, value)
  File "/home/$USER/anaconda3/envs/main/lib/python3.8/site-packages/numba/core/typeinfer.py", line 1542, in typeof_global
    typ = self.resolve_value_type(inst, gvar.value)
  File "/home/$USER/anaconda3/envs/main/lib/python3.8/site-packages/numba/core/typeinfer.py", line 1463, in resolve_value_type
    raise TypingError(msg, loc=inst.loc)
numba.core.errors.TypingError: Failed in nopython mode pipeline (step: nopython frontend)
NameError: name 'nan' is not defined

@fbunt Thanks for the report, I can reproduce this on mainline. The issue appears to be in the rendering of the constant np.nan in the stencil output template string, it's writing str(np.nan) in as the constant, which is "nan", but it actually needs the np module in the string for it to resolve.

Fixed in #7288

I think you might have to check for -np.inf as well.

I think you might have to check for -np.inf as well.

Good point, I guess this can just specialise for inf and nan. I think that's it for string-like numerical constants from NumPy, are you aware of any others?

I think those are it. Thanks for looking into this!

I think those are it. Thanks for looking into this!

Thanks, and no problem. https://github.com/numba/numba/pull/7288/commits/619b2782618ba2e749b3bcd14bd6cdbcd10e30a4 should fix up the case of negative str based const.


Cache and BP Disable Possible Way

Hello, I want to ask if there is a easy way to disable the I/D cache and BP? It seems some parameters of cache and bp can be adjusted in the configuration file but they cannot be totally disabled. Thank you so much :-)

Hello,
Basically, it is difficult to disable the I-cache, D-cache, or branch predictor in a simple way, and manual modification is required.
Simply disabling the I-cache and D-cache will greatly reduce the speed of RSD, since each access will reach the main memory. It is possible to replace the cache with a simple, one-cycle-accessible memory with some manual modifications.
Disabling branch prediction is more complicated. It is not obvious what is meant by disabling branch prediction. It is relatively easy to rewrite the predictor so that it always predicts untaken. It is difficult to modify the fetcher to halt instruction fetching at every branch instruction until the branch resolution, and that causes significant performance degradation.
If you can tell me what your goal is, I may be able to suggest a better alternative.

Hello,
Thanks for your reply. I am developing a performance simulator based on the timing database. The accurate cycle latency information obtained by the verilator will be used to develop this database. The estimated cycle consumption from my simulator will finally compare to the result from the verilator. Unfortunately, because of the limited time, I have no time to add the cache processing mechanism to my simulator. Thus the final performance result is not comparable with the verilator result.
Adding a memory and reconnecting some ports will be a possible way.  I am writing to ask if there are some configurations that can disable the cache simply like writing some control registers. If yes, then less effort needed for me.
Thank you so much! :-)

Thank you for your suggestion, but it doesn't work in my case because my timing database granularity is only a instruction blocks with hundreds of instructions .  It doesn't depends on the real context of the full program execution.  If you have any other ideas, please tell me. If not, I can close it :-)

Thank you for your suggestions. now I will close it :)

I'm sorry for the late reply.
Have you tried pipeline visualization with Konata? The logs for the pipeline visualizer contain most of the information for each cycle of the core pipeline.
If you want per-instruction-block statistics, analyzing the log may help you. From this log, you can see when each instruction was fetched and committed.
The following is the format of the log.
RSD's "make kanata" will generate a log for Konata; see the RSD's README.
(Be careful, the command is "kanata".)
https://github.com/shioyadan/Konata/blob/master/docs/kanata-log-format.md
By running Coremark more than once and extracting the information from the second run, as I suggested at the beginning, you may be able to determine the number of execution cycles in each instruction block when most accesses hit the caches.
By the way, I think, how to define the number of "executed" cycles consumed in a fine-grained instruction block is not easy. For example, if you use the difference in commit cycles, it may not reflect the effect of instruction cache misses.


Exact-solve for ground state in MockDWaveSampler on small problems

By default, problems of size 16 and smaller will be sampled with greedy sampler and then an ExactSolver-determined ground state will be injected.
Implements a suggestion from https://github.com/dwavesystems/dwave-system/issues/474#issuecomment-1291026958.
Close #474.

Thanks @jackraymond, I updated the docstring as suggested, except that I kept a reference to ExactSolver (IMO, it's important for users to understand why and how slow this can get), and I did not impose an upper bound on the cutoff (the default is fine, and if someone decides to modify it, they should read the docs and understand the consequences - which I now much clearer, thanks to your suggestions).


calico-cni plugin leaks /tmp/_MEIxxx directories after failure

We use pyinstaller to pack calico CNI; when the calico command is run, pyinstaller unpacks a copy of the code and libraries to /tmp/_MEIxxx.  If the calico command dies or is killed, those temporary files are left in place leaking 10-20MB.

cc/ @tomdee: Time for a golang version of the our CNI plugin?  ;-)

That sounds sensible to me, it seems to be a bit of pain to package python for coreOS.  Either that, or ship the CNI plugin inside calico-node (or its own container) and exec it inside there.
I also had a crack at running the CNI plugin as a one-dir pyinstaller build to see if it reduced its CPU load/startup time vs the one-file option.  It does seem to (load average is 15 rather than 20 in my test) and ping times are 20-30% better.  (Caveat: these are numbers captured in GCE so there's quite a big variation between runs anyway.)

v1.4.0 CNI plugin is written in Golang so this issue can be closed :)


run with webcam live

Hello,
Please how can i test it with my live webcam ?
I have tested on input videos as in showed in the tuto but i want to do live detection using my ubuntu webcam. IS there any option to do this please ?
Thank you in advance

added: https://github.com/ahmetozlu/tensorflow_object_counting_api/blob/master/webcam_counting.py


Allow setting the target address size for CIE version < 4

Partial fix for #318
It's a bit ugly that this needs to be set after section creation, but Section<R: Reader>: From<R> doesn't allow extra parameters.

Coverage increased (+0.05%) to 84.835% when pulling e32031ed370a15e2a69a8a9c3351c8a78726bdb1 on philipc:cfi-address-size into 41e68ab1f25e1f9934187071133ac62c35ee1a99 on gimli-rs:master.


Rename 'master' branch to 'main'

GitHub is already defaulting to using main for new repositories, because master isn't very intentional and has negative connotations.
Although it will mean a small change for developers with existing clones, I think we should transition the repository before the v1.0 release later this year.
More info at: https://github.com/github/renaming/#rename-existing.

Done in #1095


Migrated to Java 8, Spring 5.0, Spring Boot 2.0

looks fine to me

Hi @neumaennl, when do you release this feature?

@Pirat83 That is not for me to decide. @dilipkrish has the final word on this. Especially since your change makes this work only with the latest snapshot version of Springfox and is probably not backwards-compatible.

@Pirat83 Thanks for the PR and @neumaennl thanks for reviewing, I figured you have commiter/merge access. Do you not have that?

@dilipkrish did you read my message? I don't really know if it's okay to merge this because the change makes this work only with the latest snapshot version of Springfox and is probably not backwards-compatible.
If this is okay for you, then please merge it.

@neumaennl sorry I've been busy with my day job. I'll look at this in a bit

@dilipkrish any chance to bump this? :)


Validation refinements informed by Cold Exposure data 5 - Added AggregatedErrors to Protocols Loader

Please review #644 first
Summary Change Description

Implemented AggegatedErrors in the protocols loader and streamlined the code.
The loader before was doing another "hacky" thing where it was setting the description to an empty string, which is not searchable.  It should just be null.  So I changed the model for the Protocol table.  Same reason as the study description.
Added file context to sample and compound errors.

Affected Issue Numbers

Partially addresses todo items in #580

Code Review Notes
none
Checklist

[ ] Changes have been added to the Unreleased section in the changelog.md.
[ ] All issue requirements satisfied (or no linked issues)

Part of a larger refactor - checked off todo items ina  comment in the issue

[x] Linting passes.
[x] Migrations created & committed (or no model changes)
Tests

[x] Tests implemented (or no code changes)
[x] All tests pass
[x] All example load tests pass (remotely) (or their failures predated and are unrelated to this branch)

Pushed a change WRT migrations.  I expect everything will pass and there should have no effect on the next PR.

Closed? By me? Again? How the heck? How does this keep happening?


[Arcs] Unable to backtrack during Catapult move

Backtracking is a mechanic in a Catapult move that allows free navigation within the permitted path, dropping ships as needed (Arcs Rulebook, page 13).

In our game, Blue tries to Catapult move 2 ships from Hex 5, dropping one ship at Gate 1 before moving the remaining ship to Arrow 5. Currently, the UI doesn't support this backtracking move.

There is no issue. To support the move, need to move all ships first (i.e. to 1 Gate) before continuing the next move until the Catapult move is stopped.


add support for dynamic application registration for notebooks

Keycloak does not currently support wildcards in URL callbacks, which is needed for https://github.com/SwissDataScienceCenter/renku-notebooks/issues/1322 .
They are working on support, (see keycloak/keycloak/issues/14380 and keycloak/keycloak/discussions/9278 ) but it's unclear when this might be done.
To support this, we need to have fully defined callback URIs in keycloak, like https://session_name.sessions.dev.renku.ch/oauth2/callback, and the easiest way to achieve that is to create dynamic keycloak clients, one per session.
This can be done as described here, namely:

on session start, notebook-service creates a new dynamic client through the keycloak api (possibly with limited time to live), with the session-specific callback-URI
keycloak sends back a clientId and client secret
notebook-service patches the clientId/secret into the jupyterserver resource (or as a secret)
amalthea patches those values into the oauth2-proxy here and here
on session stop, we need to invalidate the client.

Additional thing to note here. We do not even need an authz service for this. We can simply authenticate the user (like we do now) but instead of using the oauth2proxy we would use the gateway or a similar centralized service.


Update dobby action to use reusable workflows

https://github.com/simplybusiness/dobby/issues/81

/dobby version patch


feat: Update Settings Modal to use HeadlessUI & Use HeadlessFloat for…

… Dropdown
Pull Request Template
⚠ Before Submitting a PR, read the Contributing Docs in full!
Summary
Please provide a brief summary of your changes and the related issue. Include any motivation and context that is relevant to your changes. If there are any dependencies necessary for your changes, please list them here.
Change Type
Please delete any irrelevant options.

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

Testing
Please describe your test process and include instructions so that we can reproduce your test. If there are any important variables for your testing configuration, list them here.
Test Configuration:
Checklist
Please delete any irrelevant options.

[ ] My code adheres to this project's style guidelines
[ ] I have performed a self-review of my own code
[ ] I have commented in any complex areas of my code
[ ] I have made pertinent documentation changes
[ ] My changes do not introduce new warnings
[ ] I have written tests demonstrating that my changes are effective or that my feature works
[ ] Local unit tests pass with my changes
[ ] Any changes dependent on mine have been merged and published in downstream modules.
[ ] New documents have been locally validated with mkdocs

I’m planning to hold off and see if Headless UI releases a fix since the previous version of the P.R didn’t require any additional libraries.


Hook in the theme compile function?

Running :KanagawaCompile, the following error message is returned:
Error executing luv callback:
...l/share/nvim/lazy/fzfx.nvim/lua/fzfx/commons/fileios.lua:394: failed to open(w) file "/home/user/.config/bat/themes/FzfxNvim/home/user/Shares/userCode/foo/bar/debian/roles/
dotfiles/files/nvim/lua/addons/themesLua.tmTheme": "ENOENT: no such file or directory: /home/user/.config/bat/themes/FzfxNvim/home/user/Shares/userCode/foo/bar/debian/roles/do
tfiles/files/nvim/lua/addons/themesLua.tmTheme"
stack traceback:
        [C]: in function 'error'
        ...l/share/nvim/lazy/fzfx.nvim/lua/fzfx/commons/fileios.lua:394: in function <...l/share/nvim/lazy/fzfx.nvim/lua/fzfx/commons/fileios.lua:392>

With fzfx disabled, the same command executes normally.

hi @savchenko , this issue is fixed in #606 . please pull main branch and try again.


Chrome only showing Play button for each song

Not sure if it's just because of how the website has been updated, but it's only showing PLAY next to each track title now on Chrome. There isn't a download button anymore or any way to play the Stems separately.

that's how the website works. go to platform to actually download the tracks

Thanks

@phewcharyztik wait for the new version, a lot of people have been asking me for this feature.


[WIP] Remove unused functions

Removed few functions that are never called.
@bdunne please have a look if my assumptions are right (you were last one to change them). Thanks.

I don't see any callers of those methods.  :+1: @gmcculloug ?

Two functions may be called by user's script. So I added comment about it so nobody deletes them because there is no callers.

Can we pull these methods out to a module dedicated to this purpose?

Yes, that would be a nice separation.  We would need to identify the validate_* methods (and the two default_require_* methods identified here) in all the workflow models that are used for that purpose and create a file for the validation methods of each workflow.
I'm thinking that is a different PR.

@gmcculloug Should I remove comments and let you handle it in your PR?

@ZitaNemeckova No, the comments are ok as is.  Thanks.


t.onReady is not a function

#57 # Description
After providing basic example of vue-gtag app falls with error in console
t.onReady is not a function

Sorry for bothering. I've just included two plugins in one Vue.use() expression.


Hcb docs

Please review my changes. They don't do a lot of code refactoring but add major changes to the interface. I thought about videos but then decided to create an FAQs that cover almost everything so videos would be unnecessary. And with a live chat, a user can seek more clarification.

Seems this guy has some conflicts. Please resolve

I resolved the merge conflict


Garbage Upgrade to Unity 2017

Overview
I have upgraded my Unity software and it has totally screwed up everything that I have been working on for months. Past projects that I have created are now obsolete, new projects can not even be built.
Expected Behavior
I just want to be able to build a project.
Actual Behavior
After Installing a bunch of different versions of the software I now finally do not have any compiling errors within Unity, however I cannot get VS to build the Hololens project. When I build or debug the software only a blank/black window shows up. Unity software does not even load upon build.
Steps to reproduce
Build from Unity then debug in VS2017
Unity Editor Version
Unity 2017.1.2f1
Mixed Reality Toolkit Release Version
HoloToolkit-Unity-v1.2017.2.0
VS Version 15.4.4

For Unity 2017.1.2f1 you need to use HoloToolkit-Unity-Examples-v1.2017.1.2
https://github.com/Microsoft/MixedRealityToolkit-Unity/releases/tag/v1.2017.1.2
I could not get the HoloToolkit-Unity-v1.2017.2.0 release to work for HoloLens on any Unity version I tried.

Have you double-checked the capabilities of your Unity project? You need to add the Windows Mixed Reality under XR Settings ->Virtual Reality in Player Settings.
To access player settings:  Edit -> Project Settings -> Player.

Finally got everything to start working together again with the following configuration:
Unity 2017.2.0f3
HoloToolkit-Unity-v1.2017.2.0
Microsoft VS Community 2017 Ver. 15.4.4
It still does give the following error however I am still able to compile script.
"Assertion failed: Main camera does not exist!
UnityEngine.XR.WSA.HolographicSettings:get_IsDisplayOpaque()
HoloToolkit.Unity.Boundary.BoundaryManager:RenderFloorQuad() (at Assets/HoloToolkit/Boundary/Scripts/BoundaryManager.cs:101)
HoloToolkit.Unity.Boundary.BoundaryManager:Awake() (at Assets/HoloToolkit/Boundary/Scripts/BoundaryManager.cs:90)"

@tspaddock see #1369


Use localhost instead of <IP_ADDRESS> when injecting live reload script

Short description of what this resolves:
When the dev server listens on <IP_ADDRESS>, it injects a live reload script that looks like this: <script src="//<IP_ADDRESS>:35729/livereload.js?snipver=1" async="" defer=""></script>. This PR replaces <IP_ADDRESS> with localhost in this case.

@jthoms1,
I'm not sure what affect this will have. Do you have any idea?
Thanks,
Dan

@imgx64 - where to replace this?

@MELAS007 I don't understand your question. If you're getting the <IP_ADDRESS> live reload script, you can run ionic serve --address localhost as a workaround until this gets merged.


No such file or directory: 'data/word2vec_from_glove_300.vec'

There goes wrong when I run
 preprocessing.py data/train_parsed.json --outfile data/train_data_str.pkl --include_str
IOError: [Errno 2] No such file or directory: 'data/word2vec_from_glove_300.vec'
WARNING:CoreNLP_PyWrapper:Killing subprocess 79059
INFO:CoreNLP_PyWrapper:Subprocess seems to be stopped, exit code -9

Then what is the word2vec_from_glove_300.vec, and how can I get it?

It's the same glove.840B.300d.txt file, but in word2vec
format. (Can be downloaded here: http://nlp.stanford.edu/data/glove.840B.300d.zip)
After downloading you need to unzip the file (say, in data/ directory), then run
python -m gensim.scripts.glove2word2vec –input data/glove.840B.300d.txt –output data/word2vec_from_glove_300.vec

in order to convert it to word2vec format.
Thanks for noticing that. I'll update instructions in README page ASAP.

That goes very well. Thanks!
And you have a typo on the arguments of input and output, which should have double hyphens. But that is not a big problem.

Now word2vec model is being downloaded automatically.
Closing the issue.


Add pgx dialect for github.com/jackc/pgx

Add new dialect for library github.com/jakc/pgx which claims to be faster than github.com/lib/pq.

Hi @vutung2311
Don't get why change import path from github.com/jinzhu/gorm to gorm?

Hi @jinzhu
It's because of the directory structure that I used. Correct me if I'm wrong because I think in that way tests can use the implementation regardless of directory structure.


Unable to inspect variable defined in async/await catch statement

Describe the bug
When debugging inside the catch block of an async/await try...catch block, I am unable to inspect the error object/value that is defined as part of the catch statement when the application is setup to support IE 11. Attempting to set a watch or to check the value of the error variable as assigned results in a ReferenceError. The error variable as defined does not appear in the local variable scope pane of my debugger either. I encountered this issue inside a react component, though it seems to apply anywhere async await calls are transformed by regenerator. See related issue in regenerator for more info. From my testing, the underlying cause of this issue seems to be the same as what I describe in the regenerator issue.
Should the linked issue in regenerator be resolved, that would resolve this issue as well. Otherwise, I would suggest that regenerator only be applied during a production build and not a development build to prevent this from becoming an issue when debugging (especially since most any modern browser now supports generators and async/await) if the issue cannot be fixed with a code/sourcemap change. Predictable debugging is important for development instances. Perhaps making this a configurable setting in create-react-app could also suffice.
Did you try recovering your dependencies?

A recovery of dependencies did not change this behavior. This is occurring on a new project with NPM v7 used and with the project I initially found the issue in, which uses the following NPM info: (This project is keeping to NPM 6 due to compatibility issues we are having in my organization)
npm --version: 6.14.6
Which terms did you search for in User Guide?

async catch ReferenceError regenerator
Environment

Environment Info:
  current version of create-react-app: 4.0.3
  running from C:\Users\kylern\AppData\Roaming\npm-cache\_npx\17380\node_modules\create-react-app

  System:
    OS: Windows 10 10.0.19042
    CPU: (12) x64 Intel(R) Core(TM) i7-8700 CPU @ 3.20GHz
  Binaries:
    Node: 14.6.0 - ~\AppData\Local\nvs\node\14.6.0\x64\node.EXE
    Yarn: Not Found
    npm: 6.14.6 - ~\AppData\Local\nvs\node\14.6.0\x64\npm.CMD
  Browsers:
    Chrome: 91.0.4472.77
    Edge: Spartan (44.19041.1023.0), Chromium (91.0.864.41)
    Internet Explorer: 11.0.19041.1
  npmPackages:
    react: Not Found
    react-dom: Not Found
    react-scripts: Not Found
  npmGlobalPackages:
    create-react-app: Not Found

Steps to reproduce

(Write your steps here:)

In a block of code with the following structure, set a breakpoint at the indicated spot:
Code:

async function fn() {
  try {
    const result = await doAsync();
    // ...
  catch (error) {
    // Breakpoint set at this line
    if (error.response) {
      // Do something ...
    }
  }
}

When stopped on the breakpoint, set a watch for error variable or enter the variable name error in the debug console. Either a ReferenceError will occur or the variable will evaluate as undefined.

Expected behavior

Expected to see the value of the error variable as defined in the catch statement in my debugger's console or watch window.
Actual behavior

Breakpoint set:

Console Output (Chrome DevTools):

Variables Pane (Chrome DevTools):

Watch (Chrome DevTools and VSCode):
Chrome

VSCode

This same behavior occurs in both Chrome DevTools and VSCode (expected as VSCode connects to Chrome's debugger)
It appears that the error object can be referenced when debugging using _context.t0, though the ideal behavior would be to have the sourcemap translate error (catch statement variable) to _context.t0
Reproducible demo

This example uses Node 16.3 and NPM 7.15.1
https://github.com/Kytech/create-react-app-async-catch-debug-issue
All edits needed to cause this issue from a clean create react app setup are done in the second commit (minus the nvmrc file)
To reproduce issue, start the app using npm run start and follow the directions on the page once it loads (and in the comments of the App.js file)

Update: I was able to work around the issue by adding the following in the browserlist options in package.json:
"browserlist": {
  "development": [
      "last 1 chrome version",
      "last 1 firefox version",
      "last 1 safari version"
    ]
}

This disables the regenerator-runtime transformation in development, making debugging work as expected.
This exists by default in new projects created with create-react-app, so for anyone who finds this issue because they are having a similar problem, make sure the above config info didn't get deleted from your package.json. Turns out that was the issue with the project I was working on, where someone accidentally combined the production and development settings into one option.


Fix slice index for benchmark-8 2D

Introduction
Scenario 2 is based on benchmark 8 of Aubry et al. (2022). NDK provides a 2D variant that is supposed to correspond to the slice through the transducer.
Changes
Fixes the slice index for benchmark 8's 2D variant to match the paper description:

Using 1-based indexing, the center of the source (rear of the bowl or center of the piston) relative to the output grid was positioned at... [1, 171, 191] for benchmark 8.

This also happens to be the middle slice: .shape[2] // 2. The 3-D mask.shape is (451, 341, 381)
TODO:

[ ] re-direct this PR towards main once https://github.com/agencyenterprise/neurotechdevkit/pull/122 lands.

@charlesincharge I took the liberty of rebasing your PR as I've fixed the issue that caused the CI to fail in the base branch

Thanks!


y/y_bar and u/u_bar out

The WiNDC model seems to be revealing an issue with the Demand Elasticity, possibly related to #71 .
To start, I added the MPSGE.jl WiNnat model as a test. I also had to revert JuMP back to<EMAIL_ADDRESS>Based on some testing, I think that this is the cause of the slight discrepencies between the MPSGE.jl WiNnat model and the GAMS version.
I'm guessing that it's also the cause of the failure of the MPSGE.jl WiNnat model to run with the Julia-generated WiNDC data.

Re the Julia WiNDC data, I narrowed it to fd_0[:pce] , on the demand side (fd_0 on the endowment doesn't cause a problem).

This now fixes #71 and I believe opens to changing implicitvars to @expressions.
This doesn't fix the non-solving Julia-generated data for the WiNDC national model.


rng-tools: 6.16 -> 6.17

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/nhorman/rng-tools/releases.
meta.description for rng-tools is: A random number generator daemon
meta.homepage for rng-tools is: https://github.com/nhorman/rng-tools
meta.changelog for rng-tools is: https://github.com/nhorman/rng-tools/releases/tag/v6.17
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 6.17 with grep in /nix/store/rxgy30gjbz8banzcixhivhy3v04l4n8i-rng-tools-6.17
found 6.17 in filename of file in /nix/store/rxgy30gjbz8banzcixhivhy3v04l4n8i-rng-tools-6.17

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

rng-tools

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/rxgy30gjbz8banzcixhivhy3v04l4n8i-rng-tools-6.17 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A rng-tools https://github.com/r-ryantm/nixpkgs/archive/09f2111db53905ca7ea623c3a1c4de06b434c796.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/09f2111db53905ca7ea623c3a1c4de06b434c796#rng-tools

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/rxgy30gjbz8banzcixhivhy3v04l4n8i-rng-tools-6.17
ls -la /nix/store/rxgy30gjbz8banzcixhivhy3v04l4n8i-rng-tools-6.17/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package built:

rng-tools

Maintainer pings
cc @JohnAZoidberg @c0bw3b for testing.

Result of nixpkgs-review pr 314157 run on x86_64-linux 1

1 package built:

rng-tools


Added basic highscore tracking for scrolly-tile

Daft-Freak mentioned this a while ago, that highscore tracking would be cool.
This adds basic highscore tracking to scroll-tile (ignoring the level seed).

Awesome, thank you! Will aim to get this tested & merged ASAP since I want to tag a new release SOON. Somewhat caught up in Sprite Editor :laughing:

Awesome, thank you! Will aim to get this tested & merged ASAP since I want to tag a new release SOON. Somewhat caught up in Sprite Editor :laughing:

Why did I make this awful hard game? :laughing:
Thank you! Works a treat.

Why did I make this awful hard game? :laughing:
Thank you! Works a treat.


Typing files exports

Search Terms
Typing, files, exports, default
Suggestion
Create an annotation that can allow file exports typing.
See the exemple for a better explanation of the needs.
Use Cases
Sometimes we want to create files that need to export specific things and those things MUST always be the same accross multiple files.
Examples
You want to create name-of-file.widget.ts files. All those widgets files needs to export 2 things, first a function which must return a boolean and the default export must be the widget itself.
To be extra carefull you want to ensure those files always have those exports.
Checklist
My suggestion meets these guidelines:

[x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, etc.)
[x] This feature would agree with the rest of TypeScript's Design Goals.

It sounds like you want #420. If not, I can't tell what you mean from this.

@RyanCavanaugh yeah exactly thanks !


ingress.class enhancement for debugging.

This is a suggestion for improvement.  I spent some time debugging why when running an ingress with the ingress.class annotation for class set,  why my ingresses were not being added to my ingress controller.    Receiving this error message in the logs:
7 controller.go:168] ignoring add for ingress echomap based on annotation kubernetes.io/ingress.class

didn't help because it didn't tell me what the ingress controller was actually seeing in the annotation.  As it turns out my problem was because the ingress had an annotation of:
ingress. kubernetes.io/ingress.class: "internal"
instead of
kubernetes.io/ingress.class: "internal"
I would recommend that the isValid function print what it sees as the annotion:
 ingEventHandler := cache.ResourceEventHandlerFuncs{
        AddFunc: func(obj interface{}) {
            addIng := obj.(*extensions.Ingress)
            if !class.IsValid(addIng, ic.cfg.IngressClass, ic.cfg.DefaultIngressClass) {
                glog.Infof("ignoring add for ingress %v based on annotation %v", addIng.Name, class.IngressKey)
                return
            }

because right now  the class.IngressKey just prints kubernetes.io/ingress.class

didn't help because it didn't tell me what the ingress controller was actually seeing in the annotation. As it turns out my problem was because the ingress had an annotation of:

Good point. Added for next beta


Compute type based on value

WIP
This passes go/types. I haven't tried to clean it up or tried it on other parts yet.

@rafael-atticlabs This is now ready for review.
I have one more todo that I want to see if I can figure out (grouping cycle types in simplify).
Sorry about the size :'(

Landing!!!


Add well-defined error codes

Feature request
Is your feature request related to a problem? Please describe.
Currently it appears that errors from GoTrue only contain a single property, msg, which describes the error. While I could show this message to my users, it would not be localized, and may not match the tone of my app.
Describe the solution you'd like
I would like it if there were well-documented specific error codes (beyond the HTTP error codes) like PASSWORD_TOO_SHORT or EMAIL_EXISTS included in error payloads so that I could create my own error messages to present to the user on the application side.
Describe alternatives you've considered
I could match against the existing msg strings, but that seems error-prone and likely to be broken when the language in those strings is updated.

After further development, I've discovered that some errors do include error codes (GoTrueError(message: nil, msg: nil, code: nil, error: Optional("invalid_grant"), errorDescription: Optional("Invalid login credentials")) while others do not (GoTrueError(message: nil, msg: Optional("Password should be at least 6 characters"), code: Optional(422), error: nil, errorDescription: nil).

This is a known issue and one we're very aware of. It is likely to be one of our biggest priorities in the next year. We'll keep you updated on this issue.

hello there, it would be nice if you could add the return: user or account does not exist.
Currently when user does not exist it just returns: Invalid login credentials.

Working on it, main PR here: #1377

Fixed by #1377

It appears @hf PR was merged, shouldn't this issue be closed then?

yes you can close
Em seg., 15 de abr. de 2024 às 07:58, Fatuma Abdullahi <
@.***> escreveu:

It appears @hf https://github.com/hf PR was merged, shouldn't this
issue be closed then?
—
Reply to this email directly, view it on GitHub
https://github.com/supabase/auth/issues/842#issuecomment-2056550054, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AQH3JUIDWJLDYH3OL3NCG6LY5OXG5AVCNFSM6AAAAAASRFKETGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJWGU2TAMBVGQ
.
You are receiving this because you commented.Message ID:
@.***>


update CCF (previously CSS) to 4.22.1 (duck player fixes)

Please review the release process for BrowserServicesKit here.
Required:
Task/Issue URL:
iOS PR:
macOS PR:
What kind of version bump will this require?: Major/Minor/Patch
Optional:
Tech Design URL:
CC:
Description:

Steps to test this PR:
1.
1.

OS Testing:

[ ] iOS 14
[ ] iOS 15
[ ] iOS 16
[ ] macOS 10.15
[ ] macOS 11
[ ] macOS 12

Internal references:
Software Engineering Expectations
Technical Design Template

Current dependencies on/for this PR:

main

PR #392   👈

This comment was auto-generated by Graphite.


Added webpack support

Hi!
I fix exports from module.
const angular = require('angular');
const ngProgressLite = require('ngprogress-lite');

angular.module('App', [
  ngProgressLite // <- exports string with module name
])

@SuperPaintman but it already has UMD support - https://github.com/voronianski/ngprogress-lite/blob/master/ngprogress-lite.js#L11

@voronianski https://github.com/voronianski/ngprogress-lite/blob/master/ngprogress-lite.js#L22

@SuperPaintman but if we just edit this line to:
- angular.module('ngProgressLite', []).provider('ngProgressLite', function () {
+ return angular.module('ngProgressLite', []).provider('ngProgressLite', function () {

Will it solve the issue? Or it needs to return string?

I agree, this is also a good way. But we should return the strins.

@SuperPaintman then maybe it's better to update your PR accordingly to return angular.module(..).name ?

@voronianski yeah, it's right

@SuperPaintman thanks!

@voronianski fixed. Pleas inc path version

@voronianski
package.json:
- "version": "1.0.7",
+ "version": "1.0.8",


very bad design

This is related to touchphat #5.
What is wrong with cap1xxx?

Hardware detection There is code for detecting the chip used. Why is it not used to automatically instantiate the right class?
Polling The CAP1166's state is polled constantly, if no IRQ line is configured. This happens at a pretty high rate and the sleep delay is hardcoded. The method name wait_for_interrupt is a misnomer. You never wait for an interrupt by polling. This is the complete opposite of an interrupt. What is happening here, is polling the state change latch register.
If the hardware interrupt is used, which is impossible on the touchphat without modifying the hardware, the interrupt handler does way too much. _handle_alert does everything, an interrupt handler should only set a flag and return asap. The actually processing of the assigned action as to happen on a different thread. And what is the rationale for the 1 second bouncetime? The IRQ line does not bounce! This just makes it unresponsive.
Setting the led behaviour is broken with a float as register address.

There are more deficiencies. It's a good start, but this library has to be reworked thoroughly.

some products have multiple ICs. Explicit is better than implicit.

It's waiting for the interrupt bit to flip in the IC. Semantics. Potato potato etc!

It works. There's no pressing need to change that for the sake of correctness.

Might be a good idea to raise a separate issue for this on the cap1xxx git. Granted since this library was written to underpin our own product libraries it's not an huge issue.

I appreciate your concern, and there are plenty of things that bug me about the library. Unfortunately time is precious and I can't afford to waste it fixing something that works well enough.


gltf - cannot open ( anymore ) with Blender

https://i.imgur.com/PthUBnv.png
Yesterday I installed cqparts via pip and it worked ok.
Today I installed via conda and I got this error.
I installed it via
$ sh Anaconda3-5.3.1-Linux-x86_64.sh
$ conda create --quiet --name testenv -c cadquery -c conda-forge freecad=0.18 python=3.6
$ source activate testenv
$ pip install cqparts
This is the project I am generating:
https://github.com/gnhdesign/ct1
and I am using this Blender addon
https://github.com/KhronosGroup/glTF-Blender-IO

Thanks for the feedback @KammutierSpule
Looking into it from my phone for now, this is where the exception is raised assert False
https://github.com/KhronosGroup/glTF-Blender-IO/blob/master/addons/io_scene_gltf2/io/com/gltf2_io.py#L52
But it's not root cause.

I opened a cross issue at glTF project
https://github.com/KhronosGroup/glTF-Blender-IO/issues/169#issuecomment-447560442
I didn't get a chance yet to generate a sample file for them. If you have some file to test and send to them I will appreciate.

Yeah, I didn't have much of a chance today I'm sorry.
I'm travelling for work, I'll be back in about 3 days, and then I'll have more of a chance to reproduce this fault myself.

I didn't get a chance yet to generate a sample file for them.

Oh! that's an easy one; essentially everything from the examples page is displayed using gltf.
I'll post a response in the cross-issue

We got the answer from Julien,

You have an accessor.count value of 708.0, but accessor.count must be an integer
(cf specification https://github.com/KhronosGroup/glTF/tree/master/specification/2.0#accessor)
So your gltf file is not valid.
Seems the current version of validator don't check that, I will open an issue

This may be caused by some library version differences, because as explained before, I could get it work at first when not using Anaconda.
I guess you can easily force the output to be integer only.

Hello,
For your information, answer I received from glTF-validator team:
https://github.com/KhronosGroup/glTF-Validator/issues/109

@KammutierSpule I think the reason this problem has surfaced is due to changes in python3's arithmetic returning floats.
Out of curiosity:

did the installation via pip use python 2.x?
did the installation via conda use python 3.x?

You can see my edit on original post with the steps I did for install and run it.
Python 3.6.7
I did install cqparts via pip while in anaconda testenv ( unless I did some mistake )
cadquery        1.2.0
cqparts         0.2.1
pip             18.1
I tested on two Linux machines.

I did install cqparts via pip while in anaconda testenv ( unless I did some mistake )

No, that looks good; installation should be simple. And if you can import cqparts then installation has worked.
In your original post:

Yesterday I installed cqparts via pip and it worked ok

I was wondering if that was on python 2, since you're implying that this issue wasn't present in that installation...
Anyway! It's not important, I think we've found the root cause 👍
Are you able to test it with the changes in #134?
I'm still traveling, so my resources are limited.

Yesterday I installed cqparts via pip and it worked ok

I was wondering if that was on python 2, since you're implying that this issue wasn't present in that installation...

yeah sorry I wasn't clear on that.
It means that the first time I tried, it was running on the native system (ie: not anaconda) so probably it was python2.
For the new system,  I uninstalled the native pip and run it over anaconda.

I believe you missed one cast at:
return int(self.idx_len / self.idx_bytelen)
it works after that..

@KammutierSpule good catch!
I reverted #134 in favour of #136.
In hindsight I should have waited for your green light before merging #134.
Thanks for your help @KammutierSpule ! 👍


bug: declared classes not stored in last tick

Is there an existing issue?

[X] I have searched the existing issues

Description of bug
store_block is only called during ticks as here. They aren't stored during on_block_time which causes it changes to not reflect. We should store the declared classes in on_block_time
Steps to reproduce
No response

Should be solved in #271

I think the issue you're referring to is something else: the verify_apply step of block import does the store block, we don't need it in block prod
I have fixed another thing which was related to forcing db flush for block production at every block, I believe that's what you're referring to.
Pending block in block prod does not use the block import pipeline yet, and that's why the store block is there
it will use the block improt and i'm also adding the ability to continue the last pending block on start-up, as well as the ability to save the tx pool to disk

I see. I will explain the behaviour I saw which is why I thought this was the issue

If a class is declared in any tick except the last one, I am able to get the class using starkli i.e. its stored inside the DB
If a class is declared in the last tick (I can force this to happen if tick time = block time, in my case it was actually tick time = 2*block time which shouldn't be possible, created issue #277), then getting the class from starkli was failing

Does this also explain the db flush issue?

ah, no it was because i forgot to save the declared class when closing a block. It's fixed yes

This was fixed in #271


text selection is not clearing out when mouse downed in the avoided cells during diagonal selection

When diagonally selecting the cells, the default selection of text in cells is being avoided.
But, when mouse downed in one of those avoided cells, the default selection is coming back on removing selection.
Here are the steps to

Select the cells from 0 t0 3

by default 0 1 2 3 text gets selected so, on mouse down in cell 1 or cell 2 the default selection is throwing back

However, on mouse down in cells from 4 to 8, this is not showing up because they are not in default range selection. What do you think could be a possible reason for this?

I know this behavior. However I don't know how to improve this behavior :sob: We need to do some investigation.

I think we can dive into the window selection and clear this issue. I will let you know if I can find something out.
https://developer.mozilla.org/en-US/docs/Web/API/Selection/removeRange


feat: kubectl plugin support

Is your feature request related to a problem? Please describe.
Describe the solution you'd like
make it easy to use just like kubectl plugin
Describe alternatives you've considered
Additional context

done by https://github.com/k-cloud-labs/kluster-capacity/pull/72


Up to Docusaurus v3 and TS support

#1628  👈
main

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

Join @anle-statsig and the rest of your teammates on  Graphite

@tore-statsig Could you help me on the SDK side to fix the remaining errors? This is my 2nd attempt to upgrade this whole Docusaurs 😅


Fixed buffer plugin line bug

they changed to branch "main"

there is no need to specify main branch, the default branch (main) will be used by packer.


Add an idle notifier for antifire potions!

Can you explain what you want, please?

If you feel like expanding further on your request, we'll consider it. In the meantime, I'm closing this issue.


Add documentation for authorization on the REST layer

Description
To support extensions, authorization of API actions can now be carried out on the REST layer, necessarily for extensions and optionally for new and existing plugins. This PR documents details surrounding the enhancement.
Issues Resolved
Fixes #4381
Checklist

[ ] By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Just collecting the dots at this point. Thus the draft status.

Received news that this will most likely get pushed to 2.10.

@DarshitChanpura Could you have a look at this for technical accuracy, language, and completeness? I'm not sure if I got some of the language-concept agreement right in some cases. Thanks.

The work is essentially done for this PR. I considered adding the status tag "Done but waiting to merge". But it will need documentation team review and editorial review before merging for 2.11. I'm leaving it alone for now in case the Security team wants any additional changes before then.

Stepping this up to doc review.


IInvokedMethodListener should not be called for methods which aren't invoked

Opening this new issue after correspondence in #1870
I can send the revised tests as a PR but without a fix for now... let me know if I should anyway.
TestNG Version

7.0.0-SNAPSHOT

Description
In TestNG's own tests, in the test class test.listeners.ordering.ListenerInvocationListenerInvocationDisabledBehaviorTest which validates listener calls and their order, one of the test method has an inaccurate expected result.
That method is testOrderHasOnlyFailedAndSkippedConfigAndSkippedTestMethod in which the executed test class contains 1 failed config method, which leads to skipping another config method and a test method.
This issue appears when using the alwaysrunlisteners set to false, which is not the default behavior. I also suggest that this becomes the default behavior for IInvokedMethodListener in 7.0.0
Expected behavior
IInvokedMethodListener should only be invoked for the first config method as it is the only method being invoked by TestNG
Actual behavior
IInvokedMethodListener is invoked for both config methods, but not for the test method.
This is both unexpected and inconsistent.
Is the issue reproductible on runner?

[ ] Shell
[ ] Maven
[x] Gradle
[ ] Ant
[ ] Eclipse
[x] IntelliJ
[ ] NetBeans

Test case sample
Here is the revised test method with the correct listener calls:
  @Test(description = "Test class has only 1 failed config, 1 skipped config and skipped test method")
  public void testOrderHasOnlyFailedAndSkippedConfigAndSkippedTestMethod() {
    List<String> expected = Arrays.asList(
        IEXECUTIONLISTENER_ON_EXECUTION_START,
        IALTERSUITELISTENER_ALTER,
        IANNOTATIONTRANSFORMER_TRANSFORM_3_ARGS,
        ISUITELISTENER_ON_START,
        ITESTLISTENER_ON_START_TEST_TAG,
        METHODINTERCEPTOR_INTERCEPT,
        METHODINTERCEPTOR_INTERCEPT,
        ICLASSLISTENER_ON_BEFORE_CLASS,
        ICONFIGURATIONLISTENER_BEFORE_CONFIGURATION,
        IINVOKEDMETHODLISTENER_BEFORE_INVOCATION,
        IINVOKEDMETHODLISTENER_AFTER_INVOCATION,
        ICONFIGURATIONLISTENER_ON_CONFIGURATION_FAILURE,
        ICONFIGURATIONLISTENER_BEFORE_CONFIGURATION,
        ICONFIGURATIONLISTENER_ON_CONFIGURATION_SKIP,
        ITESTLISTENER_ON_START_TEST_METHOD,
        ITESTLISTENER_ON_TEST_SKIPPED_TEST_METHOD,
        ICLASSLISTENER_ON_AFTER_CLASS,
        IEXECUTION_VISUALISER_CONSUME_DOT_DEFINITION,
        ITESTLISTENER_ON_FINISH_TEST_TAG,
        ISUITELISTENER_ON_FINISH,
        IREPORTER_GENERATE_REPORT,
        IEXECUTIONLISTENER_ON_EXECUTION_FINISH
    );
    runTest(expected, SimpleTestClassWithFailedAndSkippedConfigAndSkippedTestMethod.class);
  }

Please also see the following test class which uses the same test classes and scenarios to validate calls to the IInvokedMethodListener on its own:
package test.listeners;
import org.assertj.core.api.Assertions;
import org.testng.*;
import org.testng.annotations.Test;
import test.SimpleBaseTest;
import test.listeners.ordering.*;

import java.util.ArrayList;
import java.util.Arrays;
import java.util.List;
import java.util.stream.Collectors;
import java.util.stream.Stream;

/**
 * This test verifies that IInvokedMethodListeners before/after calls
 * are invoked if and only if a configuration/test methods are invoked
 */
public class IInvokedMethodListenerTest extends SimpleBaseTest {

    @Test(description = "Test class has only 1 passed config and test method")
    public void testClassHasOnlyPassedConfigAndTestMethod() {
        List<String> expected = createExpectedList(
                "test.listeners.ordering.SimpleTestClassWithPassConfigAndMethod.beforeClass",
                "test.listeners.ordering.SimpleTestClassWithPassConfigAndMethod.testWillPass"
        );
        runTest(expected, SimpleTestClassWithPassConfigAndMethod.class);
    }

    @Test(description = "Test class has only 1 failed config and skipped test method")
    public void testClassHasOnlyFailedConfigAndSkippedTestMethod() {
        List<String> expected = createExpectedList(
                "test.listeners.ordering.SimpleTestClassWithFailedConfigAndSkippedTestMethod.beforeClass"
        );
        runTest(expected, SimpleTestClassWithFailedConfigAndSkippedTestMethod.class);
    }

    @Test(description = "Test class has only 1 failed config, 1 skipped config and skipped test method")
    public void testClassHasOnlyFailedAndSkippedConfigAndSkippedTestMethod() {
        List<String> expected = createExpectedList(
                "test.listeners.ordering.SimpleTestClassWithFailedAndSkippedConfigAndSkippedTestMethod.beforeClass"
        );

        runTest(expected, SimpleTestClassWithFailedAndSkippedConfigAndSkippedTestMethod.class);
    }

    @Test(description = "Test class has only 1 passed test method powered by a data provider")
    public void testClassHasOnlyDataDrivenPassedMethod() {
        List<String> expected = createExpectedList(
                "test.listeners.ordering.SimpleTestClassWithDataDrivenPassMethod.testWillPass",
                "test.listeners.ordering.SimpleTestClassWithDataDrivenPassMethod.testWillPass"
        );
        runTest(expected, SimpleTestClassWithDataDrivenPassMethod.class);
    }

    @Test(description = "Test class has only 1 data provider driven test method with 1 passed and 1 failed iteration.")
    public void testClassHasOnlyDataDrivenMethodWithPassedAndFailedIteration() {
        List<String> expected = createExpectedList(
                "test.listeners.ordering.SimpleTestClassWithDataDrivenMethodPassAndFailedIterations.testWillPass",
                "test.listeners.ordering.SimpleTestClassWithDataDrivenMethodPassAndFailedIterations.testWillPass"
        );
        runTest(expected, SimpleTestClassWithDataDrivenMethodPassAndFailedIterations.class);
    }

    //Regular test methods without any configuration methods.

    @Test(description = "Test class has only 1 passed test method")
    public void testClassHasOnlyPassedMethod() {
        List<String> expected = createExpectedList(
                "test.listeners.ordering.SimpleTestClassWithPassMethod.testWillPass"
        );
        runTest(expected, SimpleTestClassWithPassMethod.class);
    }

    @Test(description = "Test class has only 1 failed test method")
    public void testClassHasOnlyFailedMethod() {
        List<String> expected = createExpectedList(
                "test.listeners.ordering.SimpleTestClassWithFailedMethod.testWillFail"
        );
        runTest(expected, SimpleTestClassWithFailedMethod.class);
    }

    @Test(description = "Test class has only 1 failed test method and uses invocation counts")
    public void testClassHasOnlyFailedMethodMultipleInvocations() {
        List<String> expected = createExpectedList(
                "test.listeners.ordering.SimpleTestClassWithFailedMethodMultipleInvocations.testWillFail",
                "test.listeners.ordering.SimpleTestClassWithFailedMethodMultipleInvocations.testWillFail"
        );
        runTest(expected, SimpleTestClassWithFailedMethodMultipleInvocations.class);
    }

    @Test(description = "Test class has only 1 failed test method and uses invocation counts but configured"
            + "to skip failed invocations")
    public void testClassHasOnlyFailedMethodMultipleInvocationsAndSkipFailedInvocations() {
        List<String> expected = createExpectedList(
                "test.listeners.ordering.SimpleTestClassWithFailedMethodMultipleInvocations.testWillFail"
        );
        runTest(expected, SimpleTestClassWithFailedMethodMultipleInvocations.class, true);
    }

    @Test(description = "Test class has passed/failed/skipped test methods")
    public void testClassHasPassedFailedSkippedMethods() {
        List<String> expected = createExpectedList(
                "test.listeners.ordering.SimpleTestClassWithPassFailMethods.testWillFail",
                "test.listeners.ordering.SimpleTestClassWithPassFailMethods.testWillPass"
        );
        runTest(expected, SimpleTestClassWithPassFailMethods.class);
    }

    @Test(description = "Test class has only 1 failed test method and is powered by factory")
    public void testClassHasOnlyFailedMethodPoweredByFactory() {
        List<String> expected = createExpectedList(
                "test.listeners.ordering.SimpleTestClassPoweredByFactoryWithFailedMethod.testWillFail",
                "test.listeners.ordering.SimpleTestClassPoweredByFactoryWithFailedMethod.testWillFail"
        );
        runTest(expected, SimpleTestClassPoweredByFactoryWithFailedMethod.class);
    }

    @Test(description = "Test class has only 1 failed test method with retry analyzer")
    public void testClassHasOnlyFailedMethodWithRetryMechanism() {
        List<String> expected = createExpectedList(
                "test.listeners.ordering.SimpleTestClassWithFailedMethodHasRetryAnalyzer.testWillFail",
                "test.listeners.ordering.SimpleTestClassWithFailedMethodHasRetryAnalyzer.testWillFail"
        );
        runTest(expected, SimpleTestClassWithFailedMethodHasRetryAnalyzer.class, true);
    }

    private List<String> createExpectedList(String... expectedMethods) {
        return Arrays.stream(expectedMethods)
                .flatMap(expectedMethod -> Stream.of(before(expectedMethod), after(expectedMethod)))
                .collect(Collectors.toList());
    }

    private static void runTest(List<String> expected, Class<?> clazz) {
        runTest(expected, clazz, false);
    }

    private static void runTest(List<String> expected, Class<?> clazz, boolean skipInvocationCount) {
        TestNG testng = create(clazz);
        testng.setSkipFailedInvocationCounts(skipInvocationCount);

        InvokedMethodListener listener = new InvokedMethodListener();
        testng.addListener(listener);
        testng.alwaysRunListeners(false);
        testng.run();

        Assertions.assertThat(listener.getListenerCalls()).containsExactlyElementsOf(expected);
    }

    private static String before(String methodName) {
        return "BEFORE_INVOCATION " + methodName;
    }

    private static String after(String methodName) {
        return "AFTER_INVOCATION " + methodName;
    }

    static class InvokedMethodListener implements IInvokedMethodListener {

        private List<String> listenerCalls = new ArrayList<>();

        @Override
        public void beforeInvocation(IInvokedMethod method, ITestResult testResult) {
            listenerCalls.add(before(method.getTestMethod().getQualifiedName()));
        }

        @Override
        public void afterInvocation(IInvokedMethod method, ITestResult testResult) {
            listenerCalls.add(after(method.getTestMethod().getQualifiedName()));
        }

        List<String> getListenerCalls() {
            return listenerCalls;
        }
    }

}

IInvokedMethodListener should only be invoked for the first config method as it is the only method being invoked by TestNG

Why is the config method is called? I expect no config method at all.

I can send the revised tests as a PR but without a fix for now... let me know if I should anyway.

It's always a good start :)


PDCL-6797: Return edge destinations in sendEvent.

PLEASE DON'T MERGE THIS PR!
Description
We are going to return the handles of type activation:pull in the sendEvent promise. This design might not be final, but we need it for a beta program. Please don't merge since we will be having more design conversations around returning additional handles from the response in the sendEvent command, without changing the SDK.
Related Issue
https://jira.corp.adobe.com/browse/PDCL-6797
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Improvement (non-breaking change which does not add functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have signed the Adobe Open Source CLA or I'm an Adobe employee.
[X] I have made any necessary test changes and all tests pass.
[X] I have run the Sandbox successfully.

I will update the Sandbox as soon as our ORGs have been provisioned for Edge Destinations.

It would be great to add a functional test too.

This was released as https://cdn1.adoberesources.net/alloy/edge-destinations-beta/2.7.0-beta.0/alloy.js
Further changes necessary for the beta should be added as PRs against the edge-destinations-beta branch.


Minimize/ Maximize Modal

https://docs.google.com/document/d/1oP7aMWRoAqKq60Ws24BtIUnwtkk1edHilal44Cq5fVk/edit
https://www.figma.com/file/vgqWcofGbZNh6wBrsXnzq6/Modus-Submissions?type=design&node-id=1-4&mode=design&t=xziDxcTq1mOpjKsz-0

This is a separate component

Write guidelines
Rename the control as it confuses the Modal Dialoge that we have (Non-Modal?)

Closing this issue as it is fixed in #2766 & #2686


chore: switch examples to secondary entry-points

Since the primary entry-point @angular/material is now deprecated
with V8, we should no longer use the deprecated entry-point in our
examples.
I'm planning a follow-up that does the same for the dev-app/e2e-app.

Caretaker: merge #15983 after merging this


Improve public_transport tagging

highway=platform was obsoleted in 2011 (see http://wiki.openstreetmap.org/wiki/Tag:highway%3Dplatform ), but the ID editor still suggests it when adding a highway.
I suggest we keep the interface as it is, but change the tag added to public_transport=platform instead.
More details on the available option in http://wiki.openstreetmap.org/wiki/Tag:highway%3Dbus_stop .

Not just public_transport=platform, but all 3-4 tags from vastly approved proposal: http://wiki.openstreetmap.org/wiki/Proposed_features/Public_Transport#Stop
IMO iD should treat older highway=platform/bus_stop/random tags as secondary.and public_transport=* family as primary.
Problem with highway=platform/bus_stop is that they are directly contradict a highway=* tag definition: importance of road in a highway graph.
Relevant discussion: https://github.com/gravitystorm/openstreetmap-carto/issues/435#issuecomment-199094591

I'm confused by this issue.. iD does not have a preset for highway=platform.
We do have one preset for railway=platform and one for public_transport=platform.

There separate paragraph you can use to quick search: http://wiki.openstreetmap.org/wiki/Proposed_features/Public_Transport#Compatibility_with_well_known_tags - it says about following mapping:
No 1 railway case
railway=platform <-> public_transport=platform
trivial info: transportation mode (bus=yes, tram=yes) is omitted because nobody would use buses/trams in railways
No 2 highway case: is slightly more complex (see first 2 lines in table)

@afmenez, @d1g, thanks for pointing me at the documentation for the public_transport tag. I'm ok with adding some of the new tags alongside the existing tags where it makes sense to do so, and adjusting the preset descriptions if it can help reduce mapper confusion, but I don't think it makes sense to remove any of the existing tags that are in widespread use.
I'll leave this issue open for now and let's try to make a list of actionable steps to improve the presets.

Would it be possible to use a new platform template and automatically add the new tags whenever a highway=platform or railway=platform is edited or inserted?

Would it be possible to use a new platform template and automatically add the new tags whenever a highway=platform or railway=platform is edited or inserted?

"Inserted" - generally yes we could add the new tags when the user adds a new feature, but we wouldn't be able to build in any special logic about e.g. whether a bus stop is on the road or next to the road.
"Edited" - no, the osm community would not support iD silently upgrading tags of existing features.

cc @JamesKingdom - want to look into this?  It might work like the recent healthcare tagging improvements.


Grouped ToC handwritten service overview pages

Hand-written service overview pages are not tracked in our source repos today. Our automation should generate service overview pages automatically but do not have a way to incorporate this handwritten content as specified.
A couple options:

Remove these pages and only use the auto-generated service overview pages?
Check these pages into the source repos and place the automatically generated portions in a known location? (e.g. a document include)

Found service level overview pages:

Preview

https://docs.microsoft.com/en-us/javascript/api/overview/azure/storage-overview?view=azure-node-preview
https://docs.microsoft.com/en-us/javascript/api/overview/azure/key-vault-index?view=azure-node-preview
https://docs.microsoft.com/en-us/javascript/api/overview/azure/cognitive-services?view=azure-node-preview
https://docs.microsoft.com/en-us/javascript/api/overview/azure/container-registry?view=azure-node-preview
https://docs.microsoft.com/en-us/javascript/api/overview/azure/resources?view=azure-node-preview
https://docs.microsoft.com/en-us/javascript/api/overview/azure/search?view=azure-node-preview
https://docs.microsoft.com/en-us/javascript/api/overview/azure/virtualmachines?view=azure-node-preview

Latest

https://docs.microsoft.com/en-us/javascript/api/overview/azure/storage-overview?view=azure-node-latest
https://docs.microsoft.com/en-us/javascript/api/overview/azure/key-vault-index?view=azure-node-latest
https://docs.microsoft.com/en-us/javascript/api/overview/azure/activedirectory?view=azure-node-latest
https://docs.microsoft.com/en-us/javascript/api/overview/azure/advisor?view=azure-node-latest
https://docs.microsoft.com/en-us/javascript/api/overview/azure/analysis-services?view=azure-node-latest

Legacy

https://docs.microsoft.com/en-us/javascript/api/overview/azure/storage-overview?view=azure-node-legacy
https://docs.microsoft.com/en-us/javascript/api/overview/azure/key-vault-index?view=azure-node-legacy

There are others. I'm writing a script to find and enumerate them.

This can be closed. We have automation that generates service-level overviews and allows handwritten service-level overviews to prevail when they exist.


Implement MAVLink arm command in SITL

.. for better MAVROS support

For basic SITL operation this is not necessary anymore since the latest simulator upstream update.
For people who arm over MAVROS it would be nice still.

Don't see the need for it anymore, closing.


If a local IPFS node is running before trying to start ipfs-desktop,  ipfs-desktop throws an unhandled error

Hi,
I'm on an updated Manjaro Linux, 64-Bit, and I have my own local ipfs node running fine.
I was able to install ipfs-desktop via "npm install", but running "npm start" opens up a dialog explaining that there was an unhandled error and that I should open an issue with the info from the error logs. Problem is, there are no error logs, I only have the npm error logs. Please see below.
I think I narrowed it down: the local node cannot be running already, or a "unhandled error" will be thrown. After I stopped the IPFS node, ipfs-desktop started up fine. I had a look through error.js which gave me the idea the cause might be a running node. It seems that the error handling doesn't work there though. (This might sound like I've got more than the most passing idea of what I'm talking about, I just clued things together though)
0 info it worked if it ends with ok
1 verbose cli [ '/usr/bin/node', '/usr/bin/npm', 'start' ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose run-script [ 'prestart', 'start', 'poststart' ]
5 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>6 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>7 verbose lifecycle<EMAIL_ADDRESS>unsafe-perm in lifecycle true
8 verbose lifecycle<EMAIL_ADDRESS>PATH: /usr/lib/node_modules/npm/node_modules/npm-lifecycle/node-gyp-bin:/home/xyz/vcs/ipfs-desktop/node_modules/.bin:/usr/local/bin:/usr/local/sbin:/usr/bin:/home/xyz/.local/share/flatpak/exports/bin:/var/lib/flatpak/exports/bin:/usr/lib/jvm/default/bin:/usr/bin/site_perl:/usr/bin/vendor_perl:/usr/bin/core_perl
9 verbose lifecycle<EMAIL_ADDRESS>CWD: /home/xyz/vcs/ipfs-desktop
10 silly lifecycle<EMAIL_ADDRESS>Args: [ '-c', 'electron-forge start' ]
11 silly lifecycle<EMAIL_ADDRESS>Returned: code: 1  signal: null
12 info lifecycle<EMAIL_ADDRESS>Failed to exec start script
13 verbose stack Error<EMAIL_ADDRESS>start: `electron-forge start`
13 verbose stack Exit status 1
13 verbose stack     at EventEmitter.<anonymous> (/usr/lib/node_modules/npm/node_modules/npm-lifecycle/index.js:304:16)
13 verbose stack     at emitTwo (events.js:126:13)
13 verbose stack     at EventEmitter.emit (events.js:214:7)
13 verbose stack     at ChildProcess.<anonymous> (/usr/lib/node_modules/npm/node_modules/npm-lifecycle/lib/spawn.js:55:14)
13 verbose stack     at emitTwo (events.js:126:13)
13 verbose stack     at ChildProcess.emit (events.js:214:7)
13 verbose stack     at maybeClose (internal/child_process.js:925:16)
13 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:209:5)
14 verbose pkgid<EMAIL_ADDRESS>15 verbose cwd /home/xyz/vcs/ipfs-desktop
16 verbose Linux 4.18.3-1-MANJARO
17 verbose argv "/usr/bin/node" "/usr/bin/npm" "start"
18 verbose node v8.11.4
19 verbose npm  v6.4.0
20 error code ELIFECYCLE
21 error errno 1
22 error<EMAIL_ADDRESS>start: `electron-forge start`
22 error Exit status 1
23 error Failed at the<EMAIL_ADDRESS>start script.
23 error This is probably not a problem with npm. There is likely additional logging output above.
24 verbose exit [ 1, true ]

Hello @thorsten-panknin! Sorry for this, it will hopefully be solved with #651.

Closing in favour of #689 and #688


Check for proxy.host if proxy is enabled

technically you can omit it if you're doing community edition (and we omit it in CI, hence all the failures). Maybe it needs to be if proxy.enabled & pachd.activateauth or something


Feature Request: Enable DfC LA/AM agent auto provisioning via module

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description
A bit of an odd one this.  We have a landing zone that's deployed and configured by the CAF module, but we're running through the DfC recommendations and we have some fails due to auto-provisioning not being enabled.  We have the log analytics agent provisioning enabled via the standard DINE policy (which works fine), but the DfC policy which checks compliance isn't checking to see if the extension is installed on the VMs, it's specifically looking to see if the "on" button is enabled via the native DfC auto-provisioning environment settings.  A couple of screenshots below to make it a little clearer:
The on/off setting in question:

The policy which is causing the failures in DfC:

Describe the solution you'd like
What I'd like to know is, is there a parameter that we're missing somewhere that would enable the on/off auto-provisioning via the module to avoid these policy fails?  If not, is this something that could be added into a future release to avoid the non-compliance issues, without having to create exemptions?

Thank you for raising this @gbr759. I have moved this to the ES repo as this is where we manage policy related matters for ALZ.
@jtracey93... this appears similar to #1121 so we can either keep open for additional detail, or close as duplicate?

@Springstone can you comment and advise if we should close one of these issues as duplicated and also if this is included in current policy refresh work?

This is enabled, but the MDFC portal config settings are not reflecting. This is another item to raise with the MDFC PG as something we'd like to be able to policy

We are working with the MDfC PG to improve consistency between Policy and MDfC portal UI. At the moment we enable all MDfC capabilities available today (with 3 preview features not enabled yet).

Please note that due to recently announced strategic change of direction for Defender for Cloud (https://techcommunity.microsoft.com/t5/microsoft-defender-for-cloud/microsoft-defender-for-cloud-strategy-and-plan-towards-log/ba-p/3883341), in which they will no longer be depending on either AMA or MMA agent, and will have alternate infrastructures providing required capabilities by April 2024. As there is nothing further we can follow up on, will be closing this issue. Feel free to re-open or open a new issue, should you still have concerns.


[FEATURE REQUEST] Remove site from domain list if down using a Github app?

A github app may be out there, to detect if the site linked in the .txt file is online or if its down, so you dont have to go through every domain every week or month and check, and so that no extra domains get inculded which are down. The github app, will create a issue saying the website is down, and a PR if possible removing that site from the list.
I think this is a good idea.

I'll keep this open as an idea. I could probably find somewhere that checks for expired website registrations. I'd probably just create a local python script to do all the work, since it probably would only need to be done every once in a while, so wouldn't be a big deal to do manually.

Actually just added it to the wiki for possible new features so the issue doesn't need to stay open

@ThioJoe I think this is possible to get done via React? I'm not sure though

@ThioJoe I think this is possible to get done via React? I'm not sure though

but python script also will work, i believe like you mentioned.

Sorry for posting on an old issue (can delete comment if that's not allowed), I wonder if PyFunceble would work?
Thanks


⚠ DesiTorrents has degraded performance

In d94c884, DesiTorrents ($DST) experienced degraded performance:

HTTP code: 200
Response time: 5304 ms

Resolved: DesiTorrents performance has improved in 88680eb.


Change Dropdown Order

This PR changes the order of the Data Source dropdown in TaxBrain so that PUF is the first option.

LGTM. Thanks @andersonfrailey!


chore: upgraded intl^:0.19.0 & removed extra navigation screen

What

Removed extra onboarding screen.
Upgraded intl package to intl^:0.19.0.

Screenshot

Fixes bug(s)

Fixes: intl package version from intl^:0.18.1 to intl^:0.19.0.

Part of

#5385

The disadvantage of 2 features in a single PR is when we only want a part of it.
To be honest, switching with intl to 0.19 will be OK once we release 4.15.0.
After that, we will probably switch to Flutter 3.22.

@sohamkolhe9829 btw I would advise you to create and use branches and not to send PR from the main develop branch.

how to do that, can you help me out?
On Mon, 17 Jun 2024 at 11:40, monsieurtanuki @.***>
wrote:

@sohamkolhe9829 https://github.com/sohamkolhe9829 btw I would advise
you to create and use branches and not to send PR from the main develop
branch.
—
Reply to this email directly, view it on GitHub
https://github.com/openfoodfacts/smooth-app/pull/5391#issuecomment-2172365095,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A4E7ETHWG63MDQPQT2WSKPDZHZ4WNAVCNFSM6AAAAABJMVBZYWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZSGM3DKMBZGU
.
You are receiving this because you were mentioned.Message ID:
@.***>

how to do that, can you help me out?

I'm afraid not, I'm not very good with github. But you can find specific help about it on the web.

how to do that, can you help me out?

I'm afraid not, I'm not very good with github. But you can find specific help about it on the web.

That's not that hard.

Fork this repo:

Clone this repo

Create a new branch

Develop / commit / push on this branch

And back on GitHub, you will see a banner to create a Pull Request

@sohamkolhe9829  I'm going to close, since @g123k had something that covers the same score. sorry for this one. Sorry for the mishap.