SyntaxError: Unexpected token at abstract class default export

While parsing Rollup's src/ast/nodes/shared/FunctionBase.ts:
export default abstract class FunctionBase extends NodeBase {
                        ^
        declare async: boolean;
        // ...
}

SyntaxError: Unexpected token (36:24)
    at pp.raise (file:///mnt/chromeos/GoogleDrive/MyDrive/blik/haverbeke_2012_acorn/0/acorn/src/location.js:15:13)
    at TypeScriptParser.raiseCommonCheck (file:///mnt/chromeos/GoogleDrive/MyDrive/blik/tyrealhu_2023_acorn_typescript.js:5008:44)
    at TypeScriptParser.raise (file:///mnt/chromeos/GoogleDrive/MyDrive/blik/tyrealhu_2023_acorn_typescript.js:5014:29)
    at pp.unexpected (file:///mnt/chromeos/GoogleDrive/MyDrive/blik/haverbeke_2012_acorn/0/acorn/src/parseutil.js:111:8)
    at pp.semicolon (file:///mnt/chromeos/GoogleDrive/MyDrive/blik/haverbeke_2012_acorn/0/acorn/src/parseutil.js:88:59)
    at TypeScriptParser.parseExport (file:///mnt/chromeos/GoogleDrive/MyDrive/blik/xtuc_2020_acorn_importassertions/0/src/index.js:96:16)
    at TypeScriptParser.parseExport (file:///mnt/chromeos/GoogleDrive/MyDrive/blik/tyrealhu_2023_acorn_typescript.js:3392:34)
    at pp.parseStatement (file:///mnt/chromeos/GoogleDrive/MyDrive/blik/haverbeke_2012_acorn/0/acorn/src/statement.js:137:69)
    at TypeScriptParser.parseStatement (file:///mnt/chromeos/GoogleDrive/MyDrive/blik/tyrealhu_2023_acorn_typescript.js:3582:30)
    at pp.parseTopLevel (file:///mnt/chromeos/GoogleDrive/MyDrive/blik/haverbeke_2012_acorn/0/acorn/src/statement.js:22:21) {
  pos: 1277,
  loc: Position { line: 36, column: 24 },
  raisedAt: 1282
}

acorn expects a semicolon after the "abstract" keyword.

I parse the same code, but it didnt throw an error. this is my case

code

export default abstract class FunctionBase extends NodeBase {
  declare async: boolean;
}

parse options

acorn.Parser.extend(tsPlugin({
  jsx: {
    allowNamespaces: true
  }
})).parse(input, {
  sourceType: 'module',
  ecmaVersion: 'latest',
  locations: true
})

,okay interesting i looked around, i'm using one additional plugin, https://github.com/xtuc/acorn-import-attributes, and taking that out fixes this. It actually has a newer version than the one i'm using (an upgrade to import attributes instead of import assertions), besides an open issue regarding an export declaration, so i'll take a test with it. really hope it won't interfere cuz these compositions of overriding get nasty to track for me.

well, the mentioned plugin's new version causes other interferences when turned on... extending acorn with it before acorn-typescript causes .d.ts files to fail ("SyntaxError: Missing initializer in const declaration: export const VERSION;"), while after it's line break between regular importnamedspecifiers that become "unexpected token".

All right, maybe we can support import-assertions in acorn-typescript in the future

I think this issue could be a feature request

support in version<EMAIL_ADDRESS>

has been supportted


Allow the authentication scheme to be specified for MsalInterceptor

Core Library
MSAL.js (@azure/msal-browser)
Wrapper Library
MSAL Angular (@azure/msal-angular)
Public or Confidential Client?
Public
Description
I would like to be able to specify the authentication scheme that the MsalInterceptor passes in the authentication header. To support API's that can handle multiple authentication schemas as documented here https://github.com/AzureAD/microsoft-identity-web/wiki/multiple-authentication-schemes
Source
External (Customer)

Thanks for the feature request! This is similar to #6795 which we are tracking on our backlog though it hasn't been prioritized yet. I'll add a +1 to that feature request and close this in favor of that issue. When we have updates they will be shared there.


useMoney perf

Description
WIP
Additional context

Before submitting the PR, please make sure you do the following:

[ ] Read the Contributing Guidelines
[ ] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123)
[ ] Update docs in this repository according to your change
[ ] Run yarn changeset add if this PR cause a version bump based on Keep a Changelog and adheres to Semantic Versioning

@frehner Thanks for checking this! Yeah the readability is not the best right now.

Is it still slow on the first request (where it has to initialize all the formatters) but fast on subsequent requests when it can reuse the formatters? Just curious.

Does React's useMemo memoize stuff across requests? My assumption was that it is scoped to the current rendering tree to avoid memory leaks. Therefore, it would start anew in different requests. But I might be wrong, I'll give it a try.
I have the feeling that the Proxy is what makes this efficient because it allows lazy-loading the formatters on demand (and not all of them are always needed).

@frehner So it seems that the useCallback+Proxy version is about 3x times faster  that useMemo/no-proxy
One of the problems with the useMemo/no-proxy version is that some of the formatters are created upfront but they are never used (and apparently they are not reused across requests).
I've simplified the code a little bit and I think it's more readable now. It only wraps Intl constructors in useCallback, not results (even though we could also do it but it would be less readable). Thoughts?


CLI doesn't install express when installing a functions template.

CLI doesn't install express when creating a new project with a functions template.
Do you want to install functions templates: y
✔  Selected: Node
✔  Your app is running at http://localhost:1337 (Ctrl+C to stop)
ℹ  Emails will be sent to http://localhost:8818
⚠ Run `npm init && npm i && npm i express` to use functions

✘ Failed to initialize functions server
✔ See you later, grasshopper!%

I think the decision to not run terminal commands, like npm i, on behalf of the user automatically was taken consciously while I was in the team.
But maybe we can add an additional consent [Y/N] to ask the user whether to run it for them.

The concept of templates has been removed in the latest release.


Safe Domains Ignored by Outlook in EXO

Based on my past commit this year, https://github.com/MicrosoftDocs/microsoft-365-docs-pr/pull/14839/commits/a710fbf1b2df388eb59e5fb14c7b0bbd2ba28fa6 - there should have been a sentence in this doc that Safe Domains are ignored in EXO, when defined by end-users in Outlooks. I do not see my change in this doc.
Current text:
"By design and for increased security of Exchange Online mailboxes, only the junk email settings for safe senders and safe domains, blocked senders, and blocked domains are recognized. Safe mailing list settings are ignored."
Suggested text
"By design and for increased security of Exchange Online mailboxes, only the junk email settings for safe senders, blocked senders, and blocked domains are recognized. Safe domains settings are ignored."

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

ID: c900a9d1-897d-f7e9-6c58-a510edf56a9b
Version Independent ID: bc59ac94-233b-e558-4455-a4c7166d4023
Content: Create safe sender lists - Office 365
Content Source: microsoft-365/security/office-365-security/create-safe-sender-lists-in-office-365.md
Product: m365-security
Technology: mdo
GitHub Login: @chrisda
Microsoft Alias: chrisda

@hudish Thank you for your feedback. We have made changes to the document via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/9611
Once the author approves, the changes will be live.
Thanks
Sri

Issue is more subtle. It's more about 'Move to JMF' vs 'Quarantine'. Everything explained in https://github.com/MicrosoftDocs/microsoft-365-docs-pr/pull/17670.
Thanks for the feedback.
Closing


Trying to lunch after downloading it and closing it

im trying to open YAAGL on the file I chose after downloading it but it keeps saying {"code":"NE_FS_FILRDER", "message":"Unable to open file: /Applications/genshin/GenshinImpact_Data/globalgamemanagers"}

macOS: Version 14.5
Device: mcbook air m3
CPU: Apple M3
RAM: 16 GB
I downloaded it and clicked launch but I forgot to choose cross over on wine so it dint work, now idk how to reopen it to fix it

Where are the game files located?

on applications

thats the problem, don't put game file sin Applications, put them in ur $Home in a Folder

I opened one and moved the files there, still not working its making me download it all over again 🙁

run integrity check in Yaagl settings

I'm sorry, can I ask where the integrity option is I can't find it, or is it inside the file?


systemd Not Booting Up in WSL 2 Despite Configuration in wsl.conf

Windows Version
Microsoft Windows [Version 10.0.21996.1]
WSL Version
WSL 2
Are you using WSL 1 or WSL 2?

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>
Distro Version
Ubuntu-22.04
Other Software
No response
Repro Steps
1. Install WSL and  Ubuntu 22.04:

Followed the official WSL installation guide to install WSL 2 and the Ubuntu 22.04 distribution.

2. Configure wsl.conf:

Edited the /etc/wsl.conf file to include the following configuration:

[boot] systemd=true
3. Restart WSL:

Executed the following command in PowerShell as an administrator to shut down WSL:

wsl --shutdown
4. Start WSL and Verify systemd:

Opened the Ubuntu terminal and ran:

ps -p 1 -o comm=
Expected Behavior
The command should output systemd, indicating that systemd is running as the init system.
Actual Behavior
The output is not systemd, and running systemctl status returns the following error:
System has not been booted with systemd as init system (PID 1). Can't operate. Failed to connect to bus: Host is down
Diagnostic Logs
No response
### Tasks

I can't get the log

Closing since this issue doesn't have logs

Here I've collected the logs as requested.
WslLogs-2024-06-12_11-38-46.zip

I've edited my last comment to include the new log file as requested. Please let me know if there are any further details needed.

Can this issue be open? I’ve added the log

@OneBlue please can you reopen the issue as I've provided the log as requested

If you open the wsl.conf with vim, what is the format showing in the bottom left corner?
If it's showing [dos] you need to convert it to UNIX format by using a tool like dos2unix or by deleting and rewriting it inside the wsl with vim or nano.


Try all available view types during Artifact.import_data

I'll use a FeatureData[Taxonomy] import (gg_13_8) as an example, but currently Artifact.import_data('FeatureData[Taxonomy]' path_to_gg_taxa) will fail due to not being a TSVTaxonomyFormat file, but it is a valid HeaderlessTSVTaxonomyFormat file.

It would be useful to have importing iterate through all available formats and then fail if it doesn't match any of the registered variants, as I only knew where to look up and find the Headerless... variant due to being familiar with the codebase.

:+1:, this feature would be great to add when sniffers are leveled up (#201)


Styling Custom Timer Form

Adding interactive animation to the custom timer form will improve and enhance the user interface and user experience.
https://github.com/avinash201199/stopwatch/assets/111856910/096b1784-7592-4156-9bad-0b91d249e7ff

#213

#213


[CHEC-1357] Add functionality to the image manager component

There's two key updates here:

You can now specify a list of existing images which powers the functionality of a "choose existing" button.
You can provide additional "actions" that can be performed on each image. Providing these will change the existing delete button into a more options menu

There's one more part to the image management UI in the designs - relating to providing fields for updating attributes on an individual image. I've been debating with myself whether this is a good idea to add to the UI library though. It's true that we can ecapsulate the functionality of showing the modal and displaying the image, but we'd need to provide a slot for the fields.
I'll discuss a little more with the team and follow up with another PR

I'll fix the linting error in a moment. I'm just going to turn it off as I think the order of the definitions in the CSS makes more sense than alphabetical.


Output comparison testing

Hello!
I would like to propose that “output comparison” testing (aka snapshot testing) be discussed.
At its minimum, it would only require for write and ioctl functions to be customizable via internal setters or a new Application.init.
Another option would be enabling the injection of a custom Renderer (either using a protocol or subclassing the existing one). Size would need to be customizable too.
@rensbreur have you thought about this already?

Hi, I'm open for concrete suggestions. I don't think snapshot testing should be part of SwiftTUI, but if another framework can be supported without requiring big changes in how SwiftTUI works I'd be open to help out.

At its minimum, it would only require for write and ioctl functions to be customizable via internal setters or a new Application.init.
Another option would be enabling the injection of a custom Renderer (either using a protocol or subclassing the existing one). Size would need to be customizable too.

I'm not sure it's that easy to get snapshot tests to work like this, because of things like escape sequences used to redraw specific parts of the screen. Working with layers and controls might be easier but I'm not sure.


Decorators for ask.intent

I've been scratching my head on this one for a bit now. I'm trying to make a decorator for my intents for basic things like checking access tokens and doing debug logging.
One issue I get is TypeError: some_intent() missing 4 required positional arguments: ...
My decorator looks like this:
def debug_intent(func):
    @wraps(func)
    def wrapped(*args, **kwargs):
        intent_type = request['type']
        print(intent_type, session)
        return func(*args, **kwargs)

    return wrapped

Decorated ask route:
@ask.intent('SomeIntent', mapping={'var1': 'Var1'})
@debug_intent
def some_func(var1, var2, var3, var4)

My regular flask routes work but I just cant seem to decorate the ask routes. I couldn't find any issues related to this

Try this:
@debug_intent
@ask.intent('SomeIntent', mapping={'var1': 'Var1'})
def some_func(var1, var2, var3, var4)

I think @ask.intent sets context and lets you access things like session.
However, I was having trouble getting the decorator to output statement:
return statement(render_template('link_account')).link_account_card()
I struggled with decorators too and gave up as I was in a hurry.
A simple example would help.

Did you ever figure this out? It's driving me crazy

Got it working by doing this:
def preflight_check(f):
  @wraps(f)
  def decorated_function(*args, **kwargs):
    if not session.get('user', {}).get('accessToken'):
      response_text = render_template('not_logged_in').encode('utf-8')
      return statement(response_text).link_account_card()

    slots = request.get('intent', {}).get('slots')
    for key, value in slots.iteritems():
      kwargs.update({key: value.get('value')})

    return f(*args, **kwargs)
  return decorated_function


support HTTP routing based on first path component, and strip that component from the request

We've seen a couple use cases where we want to map a call to localhost:4140/a/b/c to a call to /b/c on service a. This is useful when setting the Host header is difficult, but when explicitly setting the destination service to localhost:4140/a is easy.
We can either have a general path rewriting mechanism or we can explicitly support the "route on first path component" case, which may capture the majority of usage.

Currently, it should not be assumed that identifiers can mutate requests, the contract is effectively Request => Id.  We need to be careful as to where request modification occurs, as it gets tricky in the face of retries, etc.  I'll take some time to write this up...

One possibility would be changing the identifier contract to Request => (Id, Request).  This allows the identifier to output a modified request.  Retries in the path stack would simply be retrying the modified request (the identifier wouldn't be re-run).

This reminds about what we did in out Finagle proxy PoC that was designed to allow service overrides with Dtabs. We were stripping the part of the request path and as it's mentioned above and were setting it as the SCRIPT_NAME HTTP header - the one from CGI spec (and propagated to its derivatives such as WSGI). Given that apps/app servers honor the SCRIPT_NAME header properly we were able to 're-mount' our apps on arbitrary sub-paths with 0 changes to apps themselves. In other words, cool feature!

+1 this feature would be awesome - otherwise we're mounting all our apps to listen on /servicename instead of just /.

@endzyme and others: thanks for the feedback. Is setting a Host header (but still connecting to locahost:4140) prohibitive?
I ask because in order to get fancy things like distributed tracing (à la https://blog.buoyant.io/2016/05/17/distributed-tracing-for-polyglot-microservices/) and per-request routing to work, you'll need to pass some other headers along at request time anyways. So I'm trying to get a sense of whether Host: is different for some reason, or if this mainly an ease-of-integration feature.

@wmorgan - Thanks for the rapid reply!
So we can handle the forwarding of headers fairly easily from our end. It's a little easier to reconcile than having to override a Host header. Overriding the Host header doesn't 100% jive with RFC for http headers. We get a lot of feedback from developers concerned about having to retrofit lots of existing applications in order to use linkerd. This is a barrier to entry for us and, from a product perspective, I can see it being an obstacle to adoption for others, even just to POC.

@endzyme (and others): Thanks for the feedback. I'm not sure Host header overwriting is really violation of the RFC (at least not by my reading), but the barrier to entry point is a good one. We're taking a look at this.

Thanks @wmorgan! I appreciate your groups diligence and attention to this!


set_temps is always setting both

The set_temps interface is working, but when I set it, both driver and passenger temps are equal.
Headers:
Content-Type: application/json
My result:
    "driver_temp_setting": 17.0,
    "passenger_temp_setting": 17.0,

Am I doing something wrong here?

This has been an unwritten thing for a bit now. The app only displays one temperature setting, despite the API accepting two inputs. I'll add a note about it.


Updated version of chefspec required from '~> 4.2.0' to '~> 4.5.0'

Updated version of chefspec required from '~> 4.2.0' to '~> 4.5.0'.
See: https://github.com/hw-cookbooks/postgresql/issues/321

Thanks @roblperry I will review and get this merged to develop in the next week or so.


[Editor] Init the default highlight color before creating the first editor instance

We want to be able to draw an highlight with the default color but without having an instance of the HighlightEditor.

/botio integrationtest

/botio integrationtest

/botio integrationtest


Fixed alertmanager_http_config setting failing deploy

alertmanager_http_config was duplicated, thus template would fail to
copy because of verification

Oh, wow. We should probably add an e2e test for this...
Thanks! :+1:


Toolbar refactoring: History buttons

The history buttons have already had their own button class implemented. Only the mentions of history buttons have been deleted from #hide_button? and #disable_button methods. Except for one case. I could not delete it because then it would hide the buttons due to the Rbac check despite of the HistoryItem button class inheriting from ButtonWithoutRbacCheck.
The whole block should be refactored first, in a separate PR.
The #disabled? method got implemented and the old spec examples also got deleted. The HistoryItem spec file got refactored and extended by new examples as well.
Links
Parent issue: #6259
Related issue: #6554
Pivotal Tracker: https://www.pivotaltracker.com/story/show/135531855

@miq-bot add_label ui, refactoring

@romanblanco @PanSpagetka could you review, pls?

Looks good :cookie:


If element is deleted, it must be removed from selection

add an element
select an element
undo.
the selection stay in place

Normally fixed in commit #76

Ok this is on the root only. I will add events for updating selection.


[Discussion] Future of Important Repos in the cloudfoundry-community Organisation

What
The most important repositiories that are consumed and maintained in the cloudfoundry-community org must be protected.
Why:
With the recent unavailability of the cloudfoundry-community due to its deletion/attack many of the repositories such as go-cfclient & cf-terraform-provider that were important to developers/users were unavailable.
From discussions, it is understood that this org is not in the purview/control of the CloudFoundry Foundation. (Admins are people from Stark & Wayne?)
How
Therefore, I request the KeyMembers/Drivers/Administrators of the CloudFoundry Foundation to either take control of the org OR fork the most important and popular repositories (or all) to the foundation so that it will be protected, standardized and well maintained.

Right after the incident, we've stripped down to few people that are trustworthy.
The current reduced set of admins are from the CF Foundation, the Gstack (myself) and the FiveTwenty consultancies, which are companies actively involved in maintaining the community's projects.
We are working on re-organizing the way we give access in the future, so that we eliminate risks of similar incidents.
Long story short, this is being worked on. A blog post will also do a full recap of facts and outcomes.

There is a summary of the incident and taken actions to restore trust in the cloudfoundry-community org here. Additionally, we discussed this topic during the TOC meeting on 13th of February (recording).

Therefore, I request the KeyMembers/Drivers/Administrators of the CloudFoundry Foundation to either take control of the org OR fork the most important and popular repositories (or all) to the foundation so that it will be protected, standardized and well maintained.

The driving party here should be the maintainers of those repositories. They should initiate discussions with the CFF and the TOC. From the TOC side we can help to execute the move when we have an agreement.

The driving party here should be the maintainers of those repositories. They should initiate discussions with the CFF and the TOC. From the TOC side we can help to execute the move when we have an agreement.

Makes sense. I believe we should try to get in touch with the maintainers (at least for the active projects that are maintained and heavily used) and work with them to move their repos to the CF Foundation.
For eg: https://github.com/cloudfoundry-community/go-cfclient/issues/379

We discussed this during the TOC meeting today. Our advice for the maintainers willing to move their repositories into the cloudfoundry org is to start the discussion by opening an issue in this repository.


⚠ GitLab Server has degraded performance

In 59e0c89, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 9150 ms

Resolved: GitLab Server performance has improved in 46117a3.


Add documents to CreateCase request

Looks like I've written too many ways to create a case with documents here?

lgtm


XCode 10 Swift 4.2 can't archive

Report
What did you do?
I archived my project (release config)
What did you expect to happen?
I excepted successful build
What happened instead?
I have the error: abort trap:6 and in the bottom of  error log
2.	While running pass #1768320 SILFunctionTransform "SpeculativeDevirtualization" on SILFunction "@$S25SkyFloatingLabelTextFieldAAC4TwimE18configureForSignIn33_604294689B813157BF26F70B546FAE81LL4withySS_tF".
 for 'myFuncThatConfigSFLT(with:)' at /Users/aleksandrkravcenkov/twim-ios/Twim/User Stories/Login/EnterApiKey/View/EnterApiKeyViewController.swift:194:5
3.	While deserializing SIL vtable for 'SkyFloatingLabelTextField' in module 'SkyFloatingLabelTextField'

In debug builds successfully
Environment
Xcode Version: 10.0 (10A255)
SkyFloatingLabelTextField Version: v3.6.0
Deployment Target: iOS 10.3
Base SDK: iOS 12
Method of Integration: CocoaPods
Project that demonstrates the issue
n/a

I also have the same issue but with another third party library called 'UICircularProgressRing'. I think the problem is not related to the particular library but may be a bug with Xcode 10.

Yes. I reincluded all my source code files and this error dissappeared

Where is Valid architectures setting in Xcode 10? I couldn't find it.

It might help others. I did following things - don't know what helped but I was able to archive my app

Remove armv7 from valid architectures of a private dynamic framework that I was using.
Deintegrate/Integrate Pods again.

I still don't have an option of setting valid architectures for my Xcode project.

@sandeepbol , you need to select the All tab in the build settings (right next to the Basics tab that is selected in the screenshot)


Fix removes key Entitas.VisualDebugging.Unity.Editor.SystemWarningThreshold

However, it appears that visual debugging is still using it.  I get an exception when the key is missing.
Here's the stack trace:
System.Collections.Generic.KeyNotFoundException: The given key was not present in the dictionary.
  at System.Collections.Generic.Dictionary`2[System.String,System.String].get_Item (System.String key) [0x000a2] in /Users/builduser/buildslave/mono/build/mcs/class/corlib/System.Collections.Generic/Dictionary.cs:150
  at DesperateDevs.Serialization.Properties.get_Item (System.String key) [0x00000] in <filename unknown>:0
  at DesperateDevs.Serialization.Preferences.get_Item (System.String key) [0x00000] in <filename unknown>:0
  at Entitas.VisualDebugging.Unity.Editor.VisualDebuggingConfig.get_systemWarningThreshold () [0x00000] in <filename unknown>:0
  at Entitas.VisualDebugging.Unity.Editor.EntitasHierarchyIcon..cctor () [0x00000] in <filename unknown>:0
UnityEditor.EditorAssemblies:ProcessInitializeOnLoadAttributes()

The properties file can contain custom keys and other Entitas Addons like VisualDebugging also use that file to store some settings. jenny fix offers to remove unused keys. These keys are considered unused in the context of the code generator but might be required by you or Addons like VisualDebugging. If asked by jenny, please select the option (i)gnore instead of removing them, this will tell the code generator to not ask you again about those keys and that they are marked as used by you or other Addons like VisualDebugging. To automatically restore keys that you accidentally removed, simply open the Entitas Preferences Windows, and the lost VisualDebugging keys are restored with default values.

How are we supposed to know what other parts of Entitas needs other than trial and error?  Shouldn't the system be able to maintain itself without issue?


learning ping by intercepting the response

I am new to tunnels and network level coding, however there are some great examples using gopacket specifically i found this one Decoding Packet Layers here.
I am interested in understanding PING specifically so I slightly adjusted the code to
	ipLayer := packet.Layer(layers.LayerTypeIPv4)
	if ipLayer != nil {
		fmt.Println("IPv4 layer detected.")
		ip, _ := ipLayer.(*layers.IPv4)

		if ip.Protocol == layers.IPProtocolICMPv4 {
			fmt.Println("icmp")
			fmt.Println("payload ", string(ip.Payload))
			fmt.Printf("From %s to %s\n", ip.SrcIP, ip.DstIP)
			fmt.Println()
			fmt.Printf("packet %+v\r\n", packet.Dump())
		}
		return
	}

What I'm interested in is manipulating the ping response in this case. I.e when my computer is pinged from another device on the network (or the internet if thats configured on the router) can I hijack the response that my machine makes, i.e maybe by changing the payload or the latency (time it takes to respond). I.e can I get the pinger to receive data that is incorrect.
I tried adding a time.Sleep(1 * time.Second) in the above, however it doesn't slow the response down - so what I figure is I am reading the incoming data, but not hijacking the network to manipulate the returned data.
Can gopacket do this? What would the interception of the response look like so that I can add in an arbitrary delay and get ping latency of 1s +
Thanks for all/any help.

@amlwwalker
gopacket only monitors the network traffic, and does not take control any packet
if you want learn something about network,
suggest to use cisco packet tracer,
you can use it to observe how ping packets are transmitted frame by frame


fix(doc-search): unable to prevent default

What is it?

[ ] Feature / enhancement
[x] Bug
[ ] Docs / tests

Description
The doc search component's {meta|ctrl}k keybinding was not working correctly. The default browser action was triggered along with the opening of DocSearch component.
Use cases and why
To prevent the default browser behavior while opening DocSearch component.

Current behavior - the DocSearch component opening along with the default browser behavior. (for eg, focusing of address bar in chromium based browsers)

Expected behavior - prevent default browser behavior while opening DocSearch component.

Checklist:

[x] My code follows the developer guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] Added new tests to cover the fix / functionality

Hey @zanettin 👋🏻
Thank you for reviewing my pr. My bad, now only the {meta|ctrl}+k keybinding's default action is prevented.
Also, the issue is that in linux or windows machines, the DocSearch component's keybinding triggers the default browser behavior along with the opening of DocSearch component. See:
https://user-images.githubusercontent.com/70624701/222329626-2e57ea58-8bf2-459f-9dfe-42253a0ce9f5.mp4

Hi @vedantnn71 👋
The solution is working. I just pushed the mentioned changes regarding useRef => useSignal changes. I guess it's ready to merge but would love to get a feedback from anybody else because of the other changes. Thanks once again 🙏

ty for approving the changes @zanettin, I tried it running locally and it is working fine. :)

Hi @vedantnn71 👋
Just had an idea of how we could keep the current handling w/o leveraging useVisibleTask:
  return (
    <div
      class="docsearch"
      preventdefault:keyDown={!state.isOpen}                   // <= PREVENT DEFAULT the qwik way
      window:onKeyDown$={(event) => {
        function open() {
          // We check that no other DocSearch modal is showing before opening
          // another one.
          if (!document.body.classList.contains('DocSearch--active')) {
            state.isOpen = true;
          }
        }

        ...

can you pls try that on your end as well and if it works update the PR? looked good on my end so far 👍

Hello @zanettin, tested it out in the qwik way like you suggested, everything seems to be working fine with that.
pushed the changes :)


Allowing uppercase in control column names

Love this project.  Thank you very much for doing it.  I tried using this to import from SAP Hana and it seems that it converts all result set column names to uppercase.  This patch will allow it to still work.

Interesting fact about SAP Hana JDBC driver. Thanks for the solution!


OpenTimestamp client install & LND timestamp verification

What
This PR is a proposal to:

Install the OpenTimestamp client as a preparatory step before installing LND
Verify the LND binary timestamp with the above-mentionned client
Re-organise both the Bitcoin Core and LND install sections by following the following logical sequence:

Download -> Signature check -> timestamp check -> checksum check -> Installation
(New, following reviews): Download -> Checksum check -> Signature check -> Timestamp check -> Installation

Download -> to download the required binary, checksum, signature and timestamp files
Checksum check -> to verify the checksum of the binary
Signature check -> to verify the signature of the checksum file just used above
Timestamp check -> to verify the timestamp of the checksum file just used above
Installation -> to install the program once the integrity of the downloaded binary file has been verified

Note: if the PR is too confusing I could remove the modifictions done on bitcoin-client.md and focus this PR on LND only, let me know.
Why
Verifying the timestamps adds an extra layer of security.
Scope

[x] significant change to core configuration
[ ] independent bonus guide
[ ] simple bug fix

Fixes #1043
Test & maintenance

Follow new instructions to install client and verify the timestamp of LND
Check that the instructions are flowing nicely and are not too long while still explaining what's going on.

Is there reason to do signature check before checksum check? Recently I changed JoinMarket installer to do the opposite, otherwise user got confused with signature validation error, when reason was incomplete download. https://github.com/JoinMarket-Org/joinmarket-clientserver/pull/1339

Is there reason to do signature check before checksum check?

Just what I thought was more logical, i.e. before using the checksum file to verify the binary it seems ogial to verify first the authenticity of the checksum file.
But if it leads to potential confusions with edge cases (e.g. incomplete downloads), then we can do the checksum check first if you prefer.

Not sure what you mean.., As it is now, it's not for "lnd" user but the "admin" user. Would adding sudo make it available to other users than admin?

Yes, right. LND and bitcoind is installed from admin user not their respective users. But no so with JoinMarket and probably others in future.
When you do pip3 install opentimestamps-client, it will install in /home/admin/.local/bin/, when you do sudo pip3 install opentimestamps-client, it will install in /usr/local/bin/ and will be available to all users.
Also, I think that OTS client installation should be moved to Bitcoin guide. For now it is added mainly because of LND, but this will change in future.

When you do pip3 install opentimestamps-client, it will install in /home/admin/.local/bin/, when you do sudo pip3 install opentimestamps-client, it will install in /usr/local/bin/ and will be available to all users.

Thanks for the clarification.

Also, I think that OTS client installation should be moved to Bitcoin guide. For now it is added mainly because of LND, but this will change in future.

Where exactly in the Bitcoin guide would you like to have the instructions on how to install the client? and why would it make more sense since we can't use it at first install?
For the Bitcoin Core updates, we could simply add a note and the command to use in the Update section at the end.

Where exactly in the Bitcoin guide would you like to have the instructions on how to install the client? and why would it make more sense since we can't use it at first install?

Could be between "Running bitcoind" and "Bitcoin Core is syncing" maybe. My idea is to have it installed at the beginning, directly after bitcoind, so that we have it installed system-wide and it is available for any following guide. Not only LND, but for anything else that may start timestamping releases (or actually RaspiBolt users might use it to timestamp any documents they may want for themselves too). No point in doing separate guide IMHO, as it  basically is just some description what it is and single command sudo pip3 install opentimesamps-client. If we add this to LND guide, some people may choose to use CLN instead and later, in JoinMarket guide, they will have OTS client missing. I don't say that my idea is the best approach, I'm open to other ideas, just this today seems to me the most logical one.

If we add this to LND guide, some people may choose to use CLN instead and later

Excellent point about CLN, I didn't think about that.
Moving it at the end of the Bitcoin Core chapter sounds good indeed. The other solution that would also makes sense is to move it at the start when we install a bunch of program that we will need later:

We could add a line after that one to install the OTS client. However, I wonder if installing it before Bitcoin Core is running might mess up the configuration of the OTS client?
Anyways, will move it to the Bitcoin Core section now. PTAL.

[...]

Ok, I updated the following:

I moved the OTS client installation to the Bitcoin Core guide, at the end, just before the user moves to the next section
I shifted the order of the checks, now starting with the checksum check, followed by the signature and ots check
I added a note in the "Update" section of Bitcoin Core to explain how to use the client instead of the website
I tired to explain a bit what's going on and the rationales of why we do this

PTAL and let me know if things can be simplified or clarified.
Feel free to reword entire sentences or sections, I don't mind. Clear wording is not my best skill.

FYI; more information on how OpenTimestamps work (from 2016 but still informative): https://petertodd.org/2016/opentimestamps-announcement#fn:hello-world-address

On top of the on-going discussion about package requirements to install ots (see above), I have 2 questions:
1. Virtual environment or pipx
It is often recommended to install a Python program in a virtual environment rather than globally to avoid conflicts.
However, in their documentation, they rather recommend pipx for standalone command line tools.. which seem to fit ots.

Python “Virtual Environments” allow Python packages to be installed in an isolated location for a particular application, rather than being installed globally. If you are looking to safely install global command line tools, see Installing stand alone command line tool. (source)

Wdyat? should we use pipx to install the opentimestamp-client package? or a virtual env?
2. How does OTS talk to our Bitcoin node..??!
How come we don't have to provide information in a conf file about where our Bitcoin Core node runs on the host? How does OTS intreact with our Bitcoin node..? can't find much information so far, it's puzzling me.

$ sudo pip3 install opentimestamps-client works for me.

👍

@VajraOfIndra, I appreciate wanting to not make the guide more complex if not necessary.

I'm personally for including it, but didn't want to merge my PR until there were a bit more support from the community and the other maintainers.

Prior to merge, the lightning-client.md section should be removed or updated to work with v0.15.3-beta. Not sure how to best integrate this with existing LND v0.15.3-beta pull request.

Good point, I'll wait for the LND v0.15.3 PR to be merged and will update this PR accordingly.
@sig-noi Wdyt about the OTS client running as is without any confoguration.. does it seem normal to you? I feel we might be missing a few install/configuration steps. Heck, how does the OTS client knows where our Bitcoin Core client is? and how can it talk to it without special permissions? :)

Heck, how does the OTS client knows where our Bitcoin Core client is?

It looks at ~/.bitcoin/bitcoin.conf - https://github.com/petertodd/python-bitcoinlib/blob/c1eca007724dc12fa1abfbc6df06965c703ad748/bitcoin/rpc.py#L148.

and how can it talk to it without special permissions?

User using OTS should be in bitcoin group so that /data/bitcoin/.cookie is accessible.

It looks at ~/.bitcoin/bitcoin.conf - https://github.com/petertodd/python-bitcoinlib/blob/c1eca007724dc12fa1abfbc6df06965c703ad748/bitcoin/rpc.py#L148.

User using OTS should be in bitcoin group so that /data/bitcoin/.cookie is accessible.

Thank you for the clarifications. So that's what it works out of the box with our setup 👍

It looks at ~/.bitcoin/bitcoin.conf - https://github.com/petertodd/python-bitcoinlib/blob/c1eca007724dc12fa1abfbc6df06965c703ad748/bitcoin/rpc.py#L148.

User using OTS should be in bitcoin group so that /data/bitcoin/.cookie is accessible.

I opened an issue/help about this I opened an issue/help about this: https://github.com/opentimestamps/opentimestamps-client/issues/130
I'm waiting for a response: Wdyt?

Thank you for the clarifications. So that's what it works out of the box with our setup 👍

It looks at ~/.bitcoin/bitcoin.conf - https://github.com/petertodd/python-bitcoinlib/blob/c1eca007724dc12fa1abfbc6df06965c703ad748/bitcoin/rpc.py#L148.

User using OTS should be in bitcoin group so that /data/bitcoin/.cookie is accessible.

I opened an issue/help about this: opentimestamps/opentimestamps-client#130
I'm waiting for a response

No need for issue, it is working, I even pointed out to the exact lines of code that supports it.

It looks at ~/.bitcoin/bitcoin.conf - https://github.com/petertodd/python-bitcoinlib/blob/c1eca007724dc12fa1abfbc6df06965c703ad748/bitcoin/rpc.py#L148.

User using OTS should be in bitcoin group so that /data/bitcoin/.cookie is accessible.

I opened an issue/help about this: opentimestamps/opentimestamps-client#130
I'm waiting for a response

No need for issue, it is working, I even pointed out to the exact lines of code that supports it.

We must avoid queries to public calendars if we already have our own node. Therefore, avoid the following logs:
$ ots verify manifest-roasbeef-v0.15.3-beta.sig.ots -f manifest-roasbeef-v0.15.3-beta.sig
> Got 1 attestation(s) from https://bob.btc.calendar.opentimestamps.org
> Got 1 attestation(s) from https://finney.calendar.eternitywall.com
> Got 1 attestation(s) from https://alice.btc.calendar.opentimestamps.org
> Got 1 attestation(s) from https://btc.calendar.catallaxy.com
> Success! Bitcoin block 759141 attests existence as of 2022-10-17 UTC

I misunderstand you?

We must avoid queries to public calendars if we already have our own node.

You cannot avoid querying calendar servers, Bitcoin node does not have OTS merkle trees. Best you can maybe do here - use Tor for that. These are two different things - for ots verify you need both to query calendar servers and have full node.

ok, how can we use tor for this?

ok, how can we use tor for this? e.g:  ots verify manifest-roasbeef-v0.15.3-beta.sig.ots -f manifest-roasbeef-v0.15.3-beta.sig --socks5-proxy <IP_ADDRESS>:9050
??

Yes, that's the idea probably, but doesn't work for me. :( Problem could be that it tries to use SOCKS5 proxy both for connecting to calendar servers and Bitcoin node.
 ₿ ots --socks5-proxy <IP_ADDRESS>:9050 verify manifest-roasbeef-v0.15.0-beta.sig.ots -f manifest-roasbeef-v0.15.0-beta.sig
Got 3 attestation(s) from cache
Traceback (most recent call last):
  File "/home/admin/.local/lib/python3.8/site-packages/socks.py", line 809, in connect
    negotiate(self, dest_addr, dest_port)
  File "/home/admin/.local/lib/python3.8/site-packages/socks.py", line 443, in _negotiate_SOCKS5
    self.proxy_peername, self.proxy_sockname = self._SOCKS5_request(
  File "/home/admin/.local/lib/python3.8/site-packages/socks.py", line 533, in _SOCKS5_request
    raise SOCKS5Error("{:#04x}: {}".format(status, error))
socks.SOCKS5Error: 0x01: General SOCKS server failure

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/admin/.local/bin/ots", line 8, in <module>
    sys.exit(main())
  File "/home/admin/.local/lib/python3.8/site-packages/otsclient/ots.py", line 34, in main
    args.cmd_func(args)
  File "/home/admin/.local/lib/python3.8/site-packages/otsclient/cmds.py", line 490, in verify_command
    if not verify_timestamp(detached_timestamp.timestamp, args):
  File "/home/admin/.local/lib/python3.8/site-packages/otsclient/cmds.py", line 412, in verify_timestamp
    block_count = proxy.getblockcount()
  File "/home/admin/.local/lib/python3.8/site-packages/bitcoin/rpc.py", line 489, in getblockcount
    return self._call('getblockcount')
  File "/home/admin/.local/lib/python3.8/site-packages/bitcoin/rpc.py", line 234, in _call
    self.__conn.request('POST', self.__url.path, postdata, headers)
  File "/usr/lib/python3.8/http/client.py", line 1256, in request
    self._send_request(method, url, body, headers, encode_chunked)
  File "/usr/lib/python3.8/http/client.py", line 1302, in _send_request
    self.endheaders(body, encode_chunked=encode_chunked)
  File "/usr/lib/python3.8/http/client.py", line 1251, in endheaders
    self._send_output(message_body, encode_chunked=encode_chunked)
  File "/usr/lib/python3.8/http/client.py", line 1011, in _send_output
    self.send(msg)
  File "/usr/lib/python3.8/http/client.py", line 951, in send
    self.connect()
  File "/usr/lib/python3.8/http/client.py", line 922, in connect
    self.sock = self._create_connection(
  File "/home/admin/.local/lib/python3.8/site-packages/otsclient/args.py", line 127, in create_connection
    sock.connect(address)
  File "/home/admin/.local/lib/python3.8/site-packages/socks.py", line 47, in wrapper
    return function(*args, **kwargs)
  File "/home/admin/.local/lib/python3.8/site-packages/socks.py", line 814, in connect
    raise GeneralProxyError("Socket error", error)
socks.GeneralProxyError: Socket error: 0x01: General SOCKS server failure

P.S. We could use Tor SOCKS5 proxy also for all downloads from GitHub.

Same for me...
Can you think of what could be the solution? open an issue to https://github.com/opentimestamps/opentimestamps-client maybe?

ok, how can we use tor for this? e.g:  ots verify manifest-roasbeef-v0.15.3-beta.sig.ots -f manifest-roasbeef-v0.15.3-beta.sig --socks5-proxy <IP_ADDRESS>:9050
??

Yes, that's the idea probably, but doesn't work for me. :( Problem could be that it tries to use SOCKS5 proxy both for connecting to calendar servers and Bitcoin node.
 ₿ ots --socks5-proxy <IP_ADDRESS>:9050 verify manifest-roasbeef-v0.15.0-beta.sig.ots -f manifest-roasbeef-v0.15.0-beta.sig
Got 3 attestation(s) from cache
Traceback (most recent call last):
  File "/home/admin/.local/lib/python3.8/site-packages/socks.py", line 809, in connect
    negotiate(self, dest_addr, dest_port)
  File "/home/admin/.local/lib/python3.8/site-packages/socks.py", line 443, in _negotiate_SOCKS5
    self.proxy_peername, self.proxy_sockname = self._SOCKS5_request(
  File "/home/admin/.local/lib/python3.8/site-packages/socks.py", line 533, in _SOCKS5_request
    raise SOCKS5Error("{:#04x}: {}".format(status, error))
socks.SOCKS5Error: 0x01: General SOCKS server failure

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/admin/.local/bin/ots", line 8, in <module>
    sys.exit(main())
  File "/home/admin/.local/lib/python3.8/site-packages/otsclient/ots.py", line 34, in main
    args.cmd_func(args)
  File "/home/admin/.local/lib/python3.8/site-packages/otsclient/cmds.py", line 490, in verify_command
    if not verify_timestamp(detached_timestamp.timestamp, args):
  File "/home/admin/.local/lib/python3.8/site-packages/otsclient/cmds.py", line 412, in verify_timestamp
    block_count = proxy.getblockcount()
  File "/home/admin/.local/lib/python3.8/site-packages/bitcoin/rpc.py", line 489, in getblockcount
    return self._call('getblockcount')
  File "/home/admin/.local/lib/python3.8/site-packages/bitcoin/rpc.py", line 234, in _call
    self.__conn.request('POST', self.__url.path, postdata, headers)
  File "/usr/lib/python3.8/http/client.py", line 1256, in request
    self._send_request(method, url, body, headers, encode_chunked)
  File "/usr/lib/python3.8/http/client.py", line 1302, in _send_request
    self.endheaders(body, encode_chunked=encode_chunked)
  File "/usr/lib/python3.8/http/client.py", line 1251, in endheaders
    self._send_output(message_body, encode_chunked=encode_chunked)
  File "/usr/lib/python3.8/http/client.py", line 1011, in _send_output
    self.send(msg)
  File "/usr/lib/python3.8/http/client.py", line 951, in send
    self.connect()
  File "/usr/lib/python3.8/http/client.py", line 922, in connect
    self.sock = self._create_connection(
  File "/home/admin/.local/lib/python3.8/site-packages/otsclient/args.py", line 127, in create_connection
    sock.connect(address)
  File "/home/admin/.local/lib/python3.8/site-packages/socks.py", line 47, in wrapper
    return function(*args, **kwargs)
  File "/home/admin/.local/lib/python3.8/site-packages/socks.py", line 814, in connect
    raise GeneralProxyError("Socket error", error)
socks.GeneralProxyError: Socket error: 0x01: General SOCKS server failure

P.S. We could use Tor SOCKS5 proxy also for all downloads from GitHub.

Same for me...
Can you think of what could be the solution? open an issue to https://github.com/opentimestamps/opentimestamps-client maybe?

I updated this PR to v0.15.3. PTAL.
It's ready to be merged but they are some conflicts.. i tried to resolve them but I'm stuck. Could someone please resolve the conflicts in order for us to merge that PR? @kristapsk?

Closing because conflicting files.
Will reopen a new PR.


Plugin displays the cli login information in output

While executing the plugin, it does a cli login and the spn details including the secret are also displayed in the output. Also there should be a way to suppress the cli command output on jenkins as people would run cli commands to list publish profiles etc and that should not be displayed

Disable print login #4 .


Some apps display the icons tiny

In my case, the app "Files" and photo album creation tool "Hofmann" are displaying the icons tiny.

Not sure if this is a fault at their end or in the switcher.
https://files.community/
https://www.hofmann.es/album-de-fotos/crear/editor

FWIW, I noticed this with Windows 10's builtin help viewer hh.exe, which many apps (e.g. 7-zip, AutoHotKey) use to display their help files.
It probably wouldn't look very nice, but maybe it would be possible just to upscale icons of this smaller size instead of just drawing them at their original size?
Also not sure if it's relevant, but I am using a high-DPI display with scaling set to 175% (although I see the same behavior on a regular display with 100% scale).


[RichEditBox] 'More options' button is not working properly

Describe the bug
On RichEditBox, selecting any written text opens up a 'Text Editing Bar' that allows you to modify the selected text's visuals. It also has an Ellipsis button that would display more options of actions to be done by the user, but currently it is not possible to access those options as they close rapidly on open.
Steps to reproduce the bug

Go to 'RichEditBox' component
Type any Text on a RichEditBox
Highlight that text to open up a Text Editing Bar
Click on the 'Ellipsis' Button to show 'more options'
See Error

Expected behavior
More Options should correctly stay open, allowing the user to interact with it.
Screenshots

NuGet package version
WinUI 3 - Windows App SDK 1.4.2: 1.4.231008000
Windows version
Windows 10 (21H2): Build 19044
Additional context
No response

Additional note: This only happens when the flyout opens automatically on selecting the text. Manually opening via right-click works as expected.

More additional note: This also happens to WinUI2 with WinUI2 gallery

Several issues here.

The control was listening for text changes and dismissing the flyout when that happened.  However, it didn't limit it based upon change type and so it was dismissing when a format change occurred (as well as actual text change).  Hence when clicking on one of the format buttons, that would incorrectly dismiss the flyout.
The more button was incorrectly taking focus when clicked causing the RichEditBox to dismiss the flyout.
The more popup menu would also take focus (again causing RichEditBox to dismiss the flyout).

This fix is in and should be in 1.5


Change the images of Partners & Sponsors

A new list of images will added, and some current removed or resized

Sent via Slack!


ci: enable basic linters in golangci-lint

This PR enables a few basic linters in golangci-lint.
fixes https://github.com/csi-addons/volume-replication-operator/issues/128
Signed-off-by: Madhu Rajanna<EMAIL_ADDRESS>

@Madhu-1 The PR looks good to me, Was CI complaining to run tests parallely?

@Madhu-1 The PR looks good to me, Was CI complaining to run tests parallely?

Yes, linter complains about it that's the reason for running tests parallelly.

@mergifyio refresh

@Mergifyio refresh


Microsoft Teams v<IP_ADDRESS>59

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

Microsoft Reviewers: Open in CodeFlow

Tiny weird issue, but it installs fine


Folder hierarchy is broken with gulp-typescript 2.13.0

The gulp-typescript should output .js files in respective folders (maintain hierarchy).
The folder hierarchy is lost and all the files are dumped in one location.
Here is the snippet of my gulpfile.js
var gulp = require('gulp');
var ts = require('gulp-typescript');
var sourcemaps = require('gulp-sourcemaps');
var path = require('path');

var tsProject = ts.createProject('tsconfig.json');

gulp.task('transpile', function(){
  return tsProject.src()
  .pipe(sourcemaps.init())
  .pipe(ts(tsProject))
  .js
  .pipe(sourcemaps.write('.', {
    includeContent: false,
    sourceRoot: function(file){
      return file.cwd;
    }
  }))
  .pipe(gulp.dest('./build'));
});

Here is the snippet of tsconfig.json
{
    "compilerOptions": {
        "module": "commonjs",
        "moduleResolution": "node",
        "target": "es5",
        "noImplicitAny": false,
        "noEmitHelpers": false,
        "experimentalDecorators": true,
        "sourceMap": true
    },
    "exclude": [
        "node_modules",
        "build",
        "config",
        "typings/browser",
        "typings/browser.d.ts"
    ]
}

The problem is occurring with gulp-typescript version 2.13.0. If I switch to older version 2.12.1, the folder hierarchy is maintained while generating js file.

Can you post your directory structure?

Thanks for your quick response.
Please check below image generated after using gulp-typescript 2.12.1

And here is the image for gulp-typescript 2.13.0

I just switched from older version to newer and started seeing this behavior.

It looks that the current behavior is correct. Older versions had a bug, which caused that the root directory was not calculated correctly. Older version considered the directory containing the tsconfig.json file as the project root, but tsc uses the common directory of all sources as the root. This bug has now been fixed. In your case, all sources are in api/controllers/, thus that is the root directory. gulp.dest works relative to this directory, so api/controllers/search.js is written as build/search.js. You can fix this by adding TypeScript files in other directories or by setting "rootDir": "./" in the compiler options.

Soon I will be having another directory adjacent to the controllers having similar names for ts files (modules). I guess, in that case the files will be overwritten with the gulp-typescript 2.13.0.
Also on the home page https://github.com/ivogabe/gulp-typescript, the flag you have listed is said to be non-supported. Are you still suggesting the use of the flag.
In addition to above api/controllers is not the root directory for my code, in fact the file which contains tsconfig.json is the root. The folders specified in exclude section of tsconfig.json are actually adjacent to it.

The readme is outdated, thanks for pointing that out. It should say that it is only supported for usage with tsProject.src(). You can find more details on the bug in #253 and #276, it was fixed in 2.11.0 and another fix (related to projects that have definition files) in 2.12.2.


ListBlobs: Dependency issue

It has been reported that using ListBlobs-task cannot find dependency Microsoft.Identity.Client version 4.39.0.

Fixed in version 3.0.2.


Pramen-Py fails on Spark 2.4.3

Describe the bug
PySpark 2.4.* does not have SparkSession.getActiveSession attribute.
Code and/or configuration snippet that caused the issue
  File "/home/sys_lba/syncwatcher-py/lib/python3.6/site-packages/pramen_py/utils/__init__.py", line 151, in get_or_create_spark_session
    maybe_active_session = SparkSession.getActiveSession()  # type: ignore
AttributeError: type object 'SparkSession' has no attribute 'getActiveSession'

The line that throws is:
https://github.com/AbsaOSS/pramen/blob/e9cf9b8355641c30356a59af08274d92549e0fca/pramen-py/src/pramen_py/utils/__init__.py#L151-L151
Expected behavior
Make sure Pramen-Py does not crash when running on Spark 2.4.*
Context

Pramen/pramen-py version: 1.1.0
Spark version: 2.4.3
Scala/Python version: 2.11
Operating system: Linux

hasMethod = 'getActiveSession' in dir(SparkSession)
if hasMethod == False, then consider maybe_active_session = session.getOrCreate()


Add calendar component

What is the plan for adding calendar component?

This project is a wrapper around mdc-web. That means we follow the official component development. Actually there is no calendar component.

We'll add it once mdc-web add it. Please check https://github.com/material-components/material-components-web/issues/55


Decide on, and document, our Kubernetes support policy

The release of Kubernetes 1.23 raises the issue of what versions of Kubernetes we test against, in relation to the Kubernetes support matrix. Currently we test against 1.19 through 1.22; https://github.com/submariner-io/shipyard/pull/688 adds support for 1.23 but doesn’t change the tests (except for the “consuming e2e” Shipyard test).
We should document our criteria for our Kubernetes test matrix: do we test only against currently-supported versions, or are there reasons to (a) continue testing against older releases and (b) delay testing against newer releases?

I think we should support 4 latest versions. So with 1,23 released, it would be 1.20 through 1.23. As per kubernetes release cadance, each year k8s has 3 releases. So somebody starting today (say v1.23) would have a whole year to use Submariner with 1.23 and upgrade to 1.27 (in future) after a year and still have Submariner working. This would avoid need for frequent upgrades.

Currently, it looks like our oldest K8s cluster is v1.19.8.

The Kubernetes project maintains release branches for the most recent three minor releases (1.23, 1.22, 1.21). Kubernetes 1.19 and newer receive approximately 1 year of patch support.

https://kubernetes.io/releases/
Maybe we should parallel the K8s policy, test any versions less than a year old.

I quite like @Jaanki’s proposal as well, and it fits @asanderson’s usage too.
I am wondering whether we could be more lenient, or rather try to determine what the basic requirements are for “supporting” (or at least testing against) various Kubernetes versions:

we want to ensure that we test against the latest and greatest, if only to be sure that we don’t depend on objects or methods which are removed (e.g. CRD v1beta1, removed in 1.22 — we were ready for this in subctl but our charts weren’t);
we want to at least keep track of any risks attached to the -2/+2 policy, e.g. on the clients;
we want to ensure that we test against the oldest still-supported release, to ensure that we don’t require objects which aren’t available yet.

The availability of objects we need is the main issue to watch out for. The introduction/deprecation/removal rules mean that we always have some time to deal with changes, but they also mean that we can’t support old releases indefinitely without adding code to deal with that (e.g. trying to install v1 CRDs and falling back to v1beta1 as an example, although we never actually had to do that). Perhaps that this constraint should be the driver: as long as we don’t need to drop support for a given version, we can leave it in the test matrix.
Right now this would mean that we keep v1.20-v1.23 (those are the current non-EOL releases), and since v1.19 still works (and has users), we leave that in too. When v1.20 reaches EOL in February, if Submariner devel still works with v1.19, we keep v1.19, add v1.24 (probably), and remove v1.20 to keep our test matrix sane — given that all this is driven by the object deprecation cycle, if v1.19 still works, v1.20 will too. When v1.19 stops working, or no longer makes sense, we bump the minimum up to whatever makes sense at that point; this may mean reintroducing v1.20 to the test matrix, or going straight up to v1.21 or v1.22 depending on how far down the road we are.

We talked about this on the Submariner dev call today, here's my summary of the take-away TODOs:

[ ] Run all CI against the lastest K8s version by default.
[ ] Run one defaults-only E2E job for each of the currently-supported, non-latest K8s versions.
[ ] Add a GHA workflow that runs all CI for all of the currently-supported, non-latest K8s versions.

[ ] Add a label to trigger running this workflow against a PR.
[ ] Run this workflow periodically.
[ ] Run this workflow on releases.

[ ] Run one defaults-only E2E job in submariner-operator repo with the oldest K8s version we think works with Submariner.

[ ] Document that this K8s version should equal the K8s version that subctl refuses to attempt installing Submariner on without force-flag.

[ ] Create user-facing docs summarizing our K8s support.
[ ] Create dev-facing docs summarizing our CI version policies and how to keep things up-to-date.

I didn't get a chance to comment on this before .... I appreciate 1 year of support .. more than that may be a bit too cumbersome. We in the field have to learn to upgrade more regularly than every 2 years or so...

I think this is done @skitt (https://github.com/submariner-io/submariner-website/issues/624#issuecomment-992808761).


feat(backend): Testnet rafiki integration

IMPLEMENTS
https://github.com/interledger/testnet/issues/268
Changes

Introduced a new env variable, which holds the ewallet of the settlement account.
fix dev-backend script
fix db structure for Users, PaymentPointers, Accounts
Removes redundant jwt strategy expired check
Fixes auth bug that led to only keeping tokens for 15 minutes
add mutations for withdrawLiquidity and depositLiquidity in Rafiki
add Rapyd requests for DepositLiquidity, HoldLiquidity, ReleaseLiquidity, Transfer and setTransferResponse

This PR implements the Rafiki integration, providing:

A create asset endpoint
A prices endpoint, which rafiki uses for currency exchange rates
A quote endpoint, which provides rafiki with information about the digital wallet fee.
A webhook endpoint, that rafiki will call with different event types, upon which testnet will react.

Flow of funds:
Testnet uses the webhook events in order to implement the following flow of funds:

OutgoingPaymentCreated:

Hold .sendAmount on Rapyd wallet (sendAmount accounts for the transaction itself + interledger protocol fees + wallet fees)
Deposit liquidity on rafiki

OutgoingPaymentCompleted:

Release .sendAmount from Rapyd wallet
Transfer .sendAmount from sender to settlement account as a Rapyd wallet to wallet transfer
Accept the wallet to wallet transfer on behalf of the settlement account
Withdraw liquidity from Rafiki

OutgoingPaymentFailed

Release .sendAmount from Rapyd wallet
Withdraw liquidity from Rafiki

IncomingPaymentCompleted  /  IncomingPaymentExpired

Transfer .receivedAmount from settlement account to receiver as a Rapyd wallet to wallet transfer
Accept the wallet to wallet transfer on behalf of the receiver
Withdraw liquidity from Rafiki

Update of transaction states still needs to be added, but I need to discuss that and will make it a separate PR


minor opt: minor optimization to the orca parser

Commit Message: minor opt: minor optimization to the orca parser
Additional Description:
By this way, the parser needn't to scan the whole header value if the header value has invalid format. And the we needn't create a copy of the header value for json format now.
Risk Level: low.
Testing: n/a.
Docs Changes: n/a.
Release Notes: n/a.
Platform Specific Features: n/a.

/wait-any on CI

/wait-any on CI

(yeah I just invited blake to the org but I think it hasn't gone through yet =P)

block by https://github.com/envoyproxy/envoy/pull/36525

gently ping @blake-snyder cc @alyssawilk

cc @adisuissa @blake-snyder thanks for the approvals 🌷

cc @alyssawilk friendly ping for an approval from senior maintainer because this change the core code. :)

LGTM but general question is why is this in core espeically if it's not being used? May be a question for @blake-snyder

I think this should has been used by the orca related features?

LGTM but general question is why is this in core espeically if it's not being used? May be a question for @blake-snyder

This is currently used during ORCA header processing: https://github.com/envoyproxy/envoy/blob/main/source/common/router/router.cc#L2131


fixed some typos in the README.md and CONTRIBUTING.md files

I fixed some typo errors in the README.md file and the Contributing.md files

fixed some typo errors in the README.md file and the Contributing.md files


Add bash autocompletion to embassy cli tools

Create Tab completion scripts for various commands in embassy cli tools (cli/sdk)

Stretch goal: ZSH plzz  :)

Would also be nice to have "targets" like the embassy-publish tool, and have these tab-complete.
Stretch goal obv :)

Potential fix: clap v4


Markdown Table Broken

Bottom Markdown table is broken on the attached report:

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

ID: d25d9de3-9362-0c4a-5eb4-b831b51e8d62
Version Independent ID: 47555a66-659f-3101-ee25-1ab859ae3238
Content: Report Data Type - Business Central
Content Source: dev-itpro/developer/methods-auto/report/report-data-type.md
Service: dynamics365-business-central
GitHub Login: @SusanneWindfeldPedersen
Microsoft Alias: solsen

Thanks for the feedback. This is fixed with the recent doc refresh.


Extending RenderPhotoProps

Describe the bug
AFAICT, type PhotoProps should be RenderPhotoProps under Next.js 13 example:
https://react-photo-album.com/examples/nextjs:
const NextJsImage: React.FC<PhotoProps> = ({
    imageProps: { src, alt, title, sizes, className, onClick },
    wrapperStyle,
}) => (

Also, I intend to pass add'l props to the custom render function (e.g., a badge overlay), but am hitting typescript friction -- shouldn't be so difficult and I'm prob in the weeds. Do you have any suggestions on how to best accomplish?
Thanks!!
-Roland.
Expected behavior
Doc bug -- see description.
How to reproduce
Read docs.
Screenshots / Logs
No response
Additional context
No response

Thank you for catching this error. PhotoProps type was renamed to RenderPhotoProps in 2.0.0. I'll go ahead and update the documentation.

Also, I intend to pass add'l props to the custom render function (e.g., a badge overlay), but am hitting typescript friction -- shouldn't be so difficult and I'm prob in the weeds. Do you have any suggestions on how to best accomplish?

I'm assuming your additional props are coming in the photos array. In that case you can use one of the following approaches to access custom photo attributes and keep TypeScript happy.

Use inline render function.

const photos = [
  {
    src: ... ,
    width: ... ,
    height: ... ,
    badge: ...  // <-- custom attribute
  }
];

// ...

<PhotoAlbum
  photos={photos}
  renderPhoto={({ photo: { badge } }) => {
    // ...
  }}
/>

Extend Photo interface.

import * as React from "react";
import PhotoAlbum, { Photo, RenderPhotoProps } from "react-photo-album";

interface CustomPhoto extends Photo {
    badge?: React.ReactNode;
}

const NextJsImage: React.FC<RenderPhotoProps<CustomPhoto>> = ({
    photo: { badge }
}) => {
    // ...
};

Use global augmentation.

Save the following react-photo-album.d.ts file under your project root or types folder.
// react-photo-album.d.ts

import * as React from "react";

declare module "react-photo-album" {
    interface Photo extends Photo {
        badge?: React.ReactNode;
    }
}

export {};

import * as React from "react";
import PhotoAlbum, { RenderPhotoProps } from "react-photo-album";

const NextJsImage: React.FC<RenderPhotoProps> = ({
    photo: { badge }
}) => {
    // ...
};

Works like a charm. You saved me a lot of time, I'm sure. Thank you so much!!


Doing Counterfactual Experience Replay

I would like to try and do some counterfactual experience replay (specifically I want to use PPO and do CER on the critic). Is there something that exposes the replay buffer or something that allows me to add custom trajectories while training somehow?
Also, how do I access the trajectories taken by the model during training?

You can try overriding the postprocess_fn in PPOTOrchPolicy. In the MARLlib, such an example may be: https://github.com/Replicable-MARL/MARLlib/blob/368c6173577d0f9c0ad70fb5b4b6afa12c864c15/marllib/marl/algos/core/CC/coma.py#L116-L125
the signature for postprocess_fn is fixed, which is
postprocess_fn(
policy: Policy, sample_batch: SampleBatch,
other_agent_batches: Optional[Dict[AgentID, SampleBatch]] = None,
episode: Optional[MultiAgentEpisode] = None
) -> SampleBatch

As displayed in the build_policy_class in https://github.com/ray-project/ray/blob/55fc0710d8472a9abaf244ed6567eb3b13136531/rllib/policy/policy_template.py#L40-L43
You can manipulate the SampleBatch instance provided in the signature to add custom trajectories.

Is this function called before or after both the policy gradient and value function gradient update in ippo or in between?

@nikhil-pitta it is called before both the policy gradient and value function gradient. The pipeline is basically: extra_action_out_fn → postprocess_fn → loss_fn → compute_gradients → apply_gradients.

@Aequatio-Space Thanks so much for the info! We would like to add counterfactual experience to DDPG before adding to the replay buffer or when pulling from the replay buffer to evaluate. Specifically, we want to augment our current step/collected experiences and add to the replay buffer for the algorithm to use. What callback function could we do to use this, and would it be the same for every offline policy? If not a callback, what would you suggest we do?

@nikhil-pitta You mentioned "augment our current step/collected experiences and add to the replay buffer", and it sounds exactly what postprocess_fn do, as our earlier discussion. This extra function applies to all MARLlib algorithms since the interface is identical.

@Aequatio-Space Thank you for the response, and yes that function is exactly what I need! I tried using it for PPO by overriding the postprocess_fn for the IPPOTorchPolicy []((https://github.com/Replicable-MARL/MARLlib/blob/master/marllib/marl/algos/core/IL/ppo.py); line 31).
I wanted to do the same thing for IQL, and saw this file (), but I was unsure how to add a postprocess_fn or any other callback because it looks like a new policy is written up within this file that doesn't accept callback functions like the PPOTorchPolicy.
Do you know what I can do to add callback functionality for IQL?

@nikhil-pitta Note that JointQPolicy inherits the Policy class, which has a function postprocess_trajectory.
https://github.com/ray-project/ray/blob/55fc0710d8472a9abaf244ed6567eb3b13136531/rllib/policy/policy.py#L361-L366.
Directly overriding this function may help.

@Aequatio-Space Thanks for the response again! So now I am trying use IQL, but an issue I'm facing is that I want each of my agents to have separate policies, but I run into the exception ValueError("joint Q learning does not support individual function"), which comes from setting share_policy="individual."
This is not my desired behavior because I would like each agent to have their own policies, which is how I expected IQL to be implemented. Is there any way to get around this?

I remember joint Q learning supports share_policy=all, you can see the related logic at here.
https://github.com/Replicable-MARL/MARLlib/blob/368c6173577d0f9c0ad70fb5b4b6afa12c864c15/marllib/marl/algos/run_vd.py#L105-L118
Try to adapt the code under this setting.


Chapter 1 Training and running a linear model using Scikit-Learn

I got error on chapter 1 Example 1-1,
when I run the code it show KeyError - 'INEQUALITY'.

@RohitKumar23-11 that means oecd_bli variable does not contain the INEQUALITY key. Try display it in the notebook, or try restarting the kernel.

I also have this error. The printed data is correct and I tried restarting the kernel, but this error still comes up.
Bummer. I sure hope this book's code is up to date.

Hi @RohitKumar23-11 ,
As @Praful932  said, this error means that the oecd_bli DataFrame does not have a column called INEQUALITY. Looking at your code, it looks like your code is using files that probably have a different structure than the ones I used when writing the book. So I suggest you use the same files as I did:

https://raw.githubusercontent.com/ageron/handson-ml2/master/datasets/lifesat/oecd_bli_2015.csv
https://raw.githubusercontent.com/ageron/handson-ml2/master/datasets/lifesat/gdp_per_capita.csv


Test normal user trying to access support page

Context
Test normal user trying to access support page
Changes proposed in this pull request
Claims and placements Sign in specs
Guidance to review
Review
Do tests pass?

 [Cov] As an unauthenticated User, I should be redirected to the Sign In page when trying to access authenticated pages


MRPHS-3169 : Added code for getting vm tool status in vm info api

Symptom:
[<JiraID>](https://apporbit.atlassian.net/browse/Mrphs 2779)
Description:
Need to send CPU, memory, disk usage, VM tools installed, network information, etc.
Resolution:
Added code for sending this info
Testing:
Unit Testing:
root@deepali-dev3 vmware-cli]# go run vsphereclient.go

{"VMId":"vm-5121","IpAddress":["<IP_ADDRESS>"],"ToolsRunningStatus":"guestToolsRunning"}

Added more information in #36..so closing this one.


[backport] Support Python scalars in iscomplexobj

Backport of #1991

@toslunar Could you take a look at this backport?


Add game page stays up beyond 10 minutes without regenerating the code

It should go back to menu screen after 10 minutes have expired as the QR code is stale now.

Regenerating the code is another option but not required as it will do unnecessary server request in case the kiosk is abandoned.

Tracking this bug for adding more than one game ID in the backend. This should be displayed until the expiry of 10 minutes.


Contributions are not working on dashboard

The contribution part of the dashboard is right now not working?

The GitHub API is unstable and sometimes doesn't return these data

I assume these numbers are rebuild during the weekly refresh, maybe let it fail if the numbers are not generated?


Support awaiting all on-going, deferred tasks to complete before shutting down ProcessorSubscription

Despite shutting a processor down before async task completes isn't a critical problem thanks to later re-process, it is a common requirement to await all async task's completion to avoid unnecessary errors/warns from middleware clients or so.

Fixed by #79


A0-2827: Add an option to disconnect the signer in the account settings

The EditAccountMenu options have different left paddings / margins. Not sure if this is related to this MR:

I think on approve path in AccountManagement should also be changed to '..'. Now:

click disconnect account in account setting
disconnect the account in account management
after submit you land on connected apps / pages list

The EditAccountMenu options have different left paddings / margins. Not sure if this is related to this MR:

I don't think so - some options are trying to display an icon which apparently doesn't work and just leaves the padding.

The EditAccountMenu options have different left paddings / margins. Not sure if this is related to this MR:

I don't think so - some options are trying to display an icon which apparently doesn't work and just leaves the padding.

This is how it looks fixed. Not sure if I like it more than without icons 😅

Btw I can also add an icon to the new menu item:

if that's what we want. Wdyt?

Btw I can also add an icon to the new menu item:

if that's what we want. Wdyt? I think it looks neat.

I like it :rocket:

Current result:


Should have top-level groups for the IESG and IRTF

Description
In the "Groups" menu in the Datatracker, there is a top-level group for the IAB (https://datatracker.ietf.org/group/iab/about/), but not for the IESG or IRTF. Even just pointers to the IESG and IRTF home pages would be useful, but a page like the IAB top-level group would be more helpful.
Code of Conduct

[X] I agree to follow the IETF's Code of Conduct

See #6239


The Session cookie can become too large for most browsers to handle

When testing the IHC page I received the following error, which seemed to prevent submitting the page from actually modifying the json. If this happens it effectively renders the website unusable.

After looking up the issue it appears that too much data may be being pushed to the user, which is filling the 4kb limit for session cookies. Multiple sources suggest using the server's storage instead of that of the user for a session, although it is currently unclear to me the exact consequences of doing so or how much effort it would take.
https://github.com/Azure-Samples/ms-identity-python-webapp/issues/63
https://stackoverflow.com/questions/53551637/session-cookie-is-too-large-flask-application
https://www.appsloveworld.com/coding/dataframe/112/userwarning-the-session-cookie-is-too-large-the-value-was-14773751-bytes-but

Fixed by making sure that the check_metadata strings are loaded to server memory instead of being offloaded to the client's cookies. This also ensures that these strings are only loaded once on server run instead of repeatedly being loaded from file on every page load.


Uncaught TypeError: Cannot read property 'seeing-is-believing' of undefined

[Enter steps to reproduce below:]

...
...

Atom Version: 1.3.1
System: Mac OS X 10.11.1
Thrown From: seeing-is-believing package, v1.7.0
Stack Trace
Uncaught TypeError: Cannot read property 'seeing-is-believing' of undefined
At /Users/danielwise/.atom/packages/seeing-is-believing/lib/seeing-is-believing.coffee:89

TypeError: Cannot read property 'seeing-is-believing' of undefined
    at Object.module.exports.SeeingIsBelieving.getVars (/Users/danielwise/.atom/packages/seeing-is-believing/lib/seeing-is-believing.coffee:89:5)
    at Object.module.exports.SeeingIsBelieving.annotateDocument (/Users/danielwise/.atom/packages/seeing-is-believing/lib/seeing-is-believing.coffee:139:17)
    at atom-workspace.subscriptions.add.atom.commands.add.seeing-is-believing:annotate-document (/Users/danielwise/.atom/packages/seeing-is-believing/lib/seeing-is-believing.coffee:28:58)
    at CommandRegistry.module.exports.CommandRegistry.handleCommandEvent (/Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:260:29)
    at /Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:3:61
    at KeymapManager.module.exports.KeymapManager.dispatchCommandEvent (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/atom-keymap/lib/keymap-manager.js:536:16)
    at KeymapManager.module.exports.KeymapManager.handleKeyboardEvent (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/atom-keymap/lib/keymap-manager.js:359:22)
    at WindowEventHandler.module.exports.WindowEventHandler.handleDocumentKeydown (/Applications/Atom.app/Contents/Resources/app.asar/src/window-event-handler.js:97:36)
    at HTMLDocument.<anonymous> (/Applications/Atom.app/Contents/Resources/app.asar/src/window-event-handler.js:3:61)

Commands
     -0:00.9.0 seeing-is-believing:annotate-document (atom-text-editor.editor.is-focused)

Config
{
  "core": {}
}

Installed Packages
# User
seeing-is-believing, v1.7.0

# Dev
No dev packages

Should be fixed now: Make sure you have the latest Atom and update your SiB.


redirect is displaying redirect message on page

5.2.39 worked as expected, when I update to 5.2.41 then redirects are displaying redirect message on screen before redirecting.
using "return redirect()->route('route-name')->send(); in a custom php file that the controller calls.

Can you show the http response that causes this behavior, including headers and content?

I have narrowed the issue down to \symfony\http-foundation\Response->send()
before composer upgrade ( when it worked as expected and if I manually add it back in it works again ) it was….
public function send()
{
    $this->sendHeaders();
    $this->sendContent();

    if (function_exists('fastcgi_finish_request')) {
        fastcgi_finish_request();
    } elseif ('cli' !== PHP_SAPI) {
        static::closeOutputBuffers(0, true);
    }

    return $this;
}

after composer upgrade ( showing redirect message ) it is…
public function send()
{
    $this->sendHeaders();
    $this->sendContent();

    return $this;
}

it looks like it is the "static::closeOutputBuffers(0, true);” statement that is missing that is causing the issue.
it looks like they have reverted back to the working code in the git repository ( https://github.com/symfony/http-foundation/commit/997cab63d81d61c6421c4dd49cc0fa588662b340 ), but it does not seem to be merged into the released version composer downloads as I just tried upgrading again and it is still not showing.
Thanks!!

On Jul 28, 2016, at 1:27 AM, Simon Svensson<EMAIL_ADDRESS>wrote:
Can you show the http response that causes this behavior, including headers and content?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

This appears to be an issue with Symfony and not laravel.


Update failing tests in the python ecosystem

What are you trying to accomplish?

3 tests are failing on CI in the python ecosystem.
Failed examples:
rspec ./spec/dependabot/python/file_parser/setup_file_parser_spec.rb:178 # Dependabot::Python::FileParser::SetupFileParser for setup.cfg parse length is expected to eq 15
rspec ./spec/dependabot/python/file_parser/setup_file_parser_spec.rb:219 # Dependabot::Python::FileParser::SetupFileParser for setup.cfg parse a tests_require dependencies has the right details
rspec ./spec/dependabot/python/file_parser_spec.rb:999 # Dependabot::Python::FileParser parse with a setup.cfg length is expected to eq 15
It looks like we are running a python script in a subprocess to parse a setup.cfg file here:
https://github.com/dependabot/dependabot-core/blob/main/python/helpers/run.py#L13 .
I'm not sure if something has changed in the python parser but for some reason it's
not including the dependencies under tests_require and that's causing some assertions
in the tests to fail.
The dependencies missing / being ignored are here:
https://github.com/dependabot/dependabot-core/blob/main/python/spec/fixtures/setup_files/setup_with_requires.cfg#L25

https://github.com/dependabot/dependabot-core/issues/10388
How will you know you've accomplished your goal?
Tests will be green again

Checklist

[x] I have run the complete test suite to ensure all tests and linters pass.
[x] I have thoroughly tested my code changes to ensure they work as expected, including adding additional tests for new functionality.
[x] I have written clear and descriptive commit messages.
[x] I have provided a detailed description of the changes in the pull request, including the problem it addresses, how it fixes the problem, and any relevant details about the implementation.
[x] I have ensured that the code is well-documented and easy to understand.

It looks like this just updates the tests to reflect that those dependencies aren't being included?
Let's instead figure out what caused this to change, determine if it is a regression and fix it if so.

I agree though my concern was that we'll be blocked from merging and shipping.

./spec/dependabot/python/file_parser/setup_file_parser_spec.rb:178

@amazimbe : The issue is that the two dependencies (pytest==2.9.1, responses==0.5.1) are not present in the parsed output after the method parse_setup in the native helper (parser.py) is executed.
The code takes all the dependencies from the fixture setup_with_requires.cfg which is 15 and write to temp setup.cfg which is correct but when the native helper parse the temp setup.cfg at this place there only 13 dependencies and two of the above stated dependencies is missing.

./spec/dependabot/python/file_parser/setup_file_parser_spec.rb:178

@amazimbe : The issue is that the two dependencies (pytest==2.9.1, responses==0.5.1) are not present in the parsed output after the method parse_setup in the native helper (parser.py) is executed.
The code takes all the dependencies from the fixture setup_with_requires.cfg which is 15 and write to temp setup.cfg which is correct but when the native helper parse the temp setup.cfg at this place there only 13 dependencies and two of the above stated dependencies is missing.

@amazimbe I should had seen your initial description. You already got this.


Support Regex Literal Syntax

We want to support JavaScript style regular expression with the expression between /s.
For example,/^Hello World$/
It needs to support escaping / with a \.

JS doesn't allow division without a numerator. I am having a hard time seeing how in Enso we would lexically distinguish this syntax from operator sections.
I don't think we can make all these work as I would expect they should at the same time:
match_some_words_surrounded_by_spaces = / hello world /
add_half_foo_to_reciprocal_bar = / 2 + 1 /

my_favorite_math_functions = [ /2, +1, 1/ ]
match_some_comma_separated_numbers = /2, +1, 1/


cli: Add rewards:show

Description
Add rewards:show command that shows rewards for the last N epochs.
If --address is supplied: voter, validator, and validator group rewards applying to address are filtered.
Tested
Tested that rewards are correctly displayed by the CLI on my testnet.
Related issues

Fixes #787 and #786

Ok. I've removed the helpers and any "any:".

I'm going to be gone on vacation. I have no more concerns. Thanks for all the iterations on this! 👍


gem build vagrant-hostsupdater.gemspec produces invalid gem

gem build vagrant-hostsupdater.gemspec
mv vagrant-hostsupdater-0.0.11.gem /tmp/vagrant-hostsupdater-0.0.11.gem
vagrant plugin uninstall vagrant-hostsupdater
vagrant plugin install /tmp/vagrant-hostsupdater-0.0.11.gem

produces empty ~.vagrant.d\gems\gems\vagrant-hostsupdater-0.0.11 folder
unlike other vagrant plugins
that makes it impossible to build custom gems from plugin forks and makes git useless, unless you merge #41

ok, nevermind, i just realized it now happens with all the plugins, so it's my system


Include subitems when refreshing items from compendiums

Is it safe to remove the expandObject calls? From a glance, it seems the point is to avoid conflicts, so that "system.location" and "system": { "location": ??? } aren't in the same call.

export function mergeObject(original, other={}, {
    insertKeys=true, insertValues=true, overwrite=true, recursive=true, inplace=true, enforceTypes=false,
    performDeletions=false
  }={}, _d=0) {
  other = other || {};
  if (!(original instanceof Object) || !(other instanceof Object)) {
    throw new Error("One of original or other are not Objects!");
  }
  const options = {insertKeys, insertValues, overwrite, recursive, inplace, enforceTypes, performDeletions};

  // Special handling at depth 0
  if ( _d === 0 ) {
    if ( Object.keys(other).some(k => /\./.test(k)) ) other = expandObject(other);

Neato, already handled.


Data: project addition by Abhinav kumar

Add a new project to the list
Game-Of-Thrones-Personality-Matcher ⚔
This is a project based on data science and machine learning which used TSNE algorithm and the main aim of this project is to match the personality of the game of thrones characters and this app is hosted on streamlit framework in pycharm IDE
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC'23 contributor
[X] I want to work on this issue

you can add multiple projects in a single PR

Fine @priyankarpal

Completed. Thanks for your contributions 🎉 ❤


Typo in httpd warning (minor)

Just something minor but it's worth fixing.
https://github.com/NixOS/nixpkgs/blob/master/nixos/modules/services/web-servers/apache-httpd/default.nix
line 659
The warning mistakenly prints an extra double quote at the end of it's line.

Thanks! Fixed in b9c95c7d600e9a10e40a8a1de2504725602241b7


  says Property 'to' doesn't exist on type 'IntrinsicAttributes'

When I'm using   ; it always throws an error that Type '{ to: string; }' is not assignable to type 'IntrinsicAttributes & IntrinsicClassAttributes<HashLink> & Readonly<HashLinkProps> & Readonly<{ children?: ReactNode; }>'.   Property 'to' does not exist on type 'IntrinsicAttributes & IntrinsicClassAttributes<HashLink> & Readonly<HashLinkProps> & Readonly<{ children?: ReactNode; }>'.ts(2322)
Unable to find a reason for the same. I've imported {HashLink} from 'react-router-hash-link'; Also used lerna add --scope=common-components react-router-hash-link to add the package. Not sure if I'm missing something here.

Hi, do you have a failing example I can look at on codesandbox? Thanks.


Feature/kick

There is still some work to be done and considerations to keep in mind. If you notice anything, please let me know so I can address it over the weekend.
This is IRC v3. Parameters: <channel> <user> *( "," <user> ) [<comment>]. I'm using IRC v3 because in IRC RFC 2812, the parameters are defined as <channel> *( "," <channel> ) <user> *( "," <user> ) [<comment>]. I chose IRCv3 because that's the way irssi handles it.  we can discuss about this.
Second I will test on the weekend if Handling Self-Kick. and see how does irssi does it
thanks for the checking

Going to have to fix the conflicts to make any kind of merge possible.... Exploding the commits on this branch seems like the only way, I will add some kind of commits for some history...


Add testdata padding for .vf4 and .vf8 instructions

I'm not sure if this is the prettiest solution but it prevents the accesses to unitialized memory described in https://github.com/ksco/riscv-vector-tests/issues/26.

Thanks for the report! I think this is a bug and the solution should not be a workaround.
.. After looking at the code, I think the issue is in genCodeVdVs2VmP3() function when initializing vd register group with random values at line 44, which should be c.vdEMUL1 instead of c.LMUL1:
44		builder.WriteString(i.gWriteRandomData(c.LMUL1))
45		builder.WriteString(i.gLoadDataIntoRegisterGroup(vd, vdEMUL1, c.SEW))

.. But, looking at v-spec, a vsext.vf4 v8,v16 with LMUL=2 should write v8-v15, since we're extending the source register group from v16-v17 with LMUL=2 to dest register group v8-v15 with LMUL=8, right?
So IIUC this is indeed a mistake in the test suite, but I think there is something wrong with your emulator too.

Yes, I also think there is something wrong in the lmul calculation but I'm not sure if your proposed fix would be the right one either. Instead I think it should be https://github.com/ksco/riscv-vector-tests/pull/28.

.. But, looking at v-spec, a vsext.vf4 v8,v16 with LMUL=2 should write v8-v15, since we're extending the source register group from v16-v17 with LMUL=2 to dest register group v8-v15 with LMUL=8, right?

The way I read the following lines in the spec is that source eew (=sew/4=16) and emul (=lmul/4=1/2) are smaller than the destination eew (=sew=64) and emul (=lmul=2).

The EEW of the source is 1/2, 1/4, or 1/8 of SEW, while EMUL of the source is (EEW/SEW)*LMUL. The destination has EEW equal to SEW and EMUL equal to LMUL.


chore(deps): :arrow_up: update all non-major dev dependencies

This PR contains the following updates:

Package
Change
Age
Adoption
Passing
Confidence

@commitlint/cli (source)
^18.1.0 -> ^18.2.0

@commitlint/config-nx-scopes (source)
^18.1.0 -> ^18.3.0

aws-cdk-lib
^2.102.0 -> ^2.103.0

Release Notes

conventional-changelog/commitlint (@commitlint/cli)
v18.2.0
Compare Source
Note: Version bump only for package @commitlint/cli

conventional-changelog/commitlint (@commitlint/config-nx-scopes)
v18.3.0
Compare Source
Features

config-nx-scopes: support latest nx version (#3728) (22e6f74)

aws/aws-cdk (aws-cdk-lib)
v2.103.0
Compare Source
Features

update AWS Service Spec (#27650) (d1c36c6)
cli: support Fn::ImportValue intrinsic function for hotswap deployments (#27292) (a54ea0f)
ec2: support for c7i and r7iz instance types in aws-ec2 (#27611) (382a0ed), closes /docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-properties-ec2-instance.html#cfn-ec2
ec2: support for i4g instance types in aws-ec2 (#27639) (039071d), closes /docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-resource-ec2-instance.html#cfn-ec2 #27631
lambda: add grantInvokeCompositePrincipal method (#27529) (447b63c), closes #27503
opensearch: OpenSearch 2.9 engine version (#27612) (1f7075f), closes #27574
secretsmanager: validate maximum value of automaticallyAfter in RotationSchedule (#27592) (99740b3), closes /github.com/aws/aws-cdk/pull/27570#pullrequestreview-1682163803 /github.com/aws/aws-cdk/pull/27570#pullrequestreview-1684311044
synthetics: syn-python-selenium-2.0 runtime (#27565) (1a89ef9), closes #27566

Bug Fixes

cdk: Add AppSync:Api_Key as hot swappable and fix a bug with AppSync.function (#27559) (12ff023)
ec2: naming collisions when using ec2.InitFile.fromAsset() on multiple instances in the same stack (#27468) (fc3be31), closes #16891 /github.com/aws/aws-cdk/blob/main/packages/aws-cdk-lib/aws-ec2/lib/cfn-init-elements.ts#L427
pipelines: using the same source (s3, connection, or codecommit) in a pipeline causes duplicate id error (#27602) (70acc84), closes #23916 #19875
region-info: facts only returned from constant region list (#27506) (7121c7e), closes #27260
stepfunctions-tasks: cloudwatchlogs service generates wrong action in role policy (#27623) (de7fb04), closes #27573

Alpha modules (2.103.0-alpha.0)
Features

schedule-alpha: support customer managed KMS keys (#27609) (df24d22), closes #27543

v2.102.1
Compare Source
Bug Fixes

region-info: adding missing regions for kinesisfirehose (#27672) (46372e0), closes #27656

Alpha modules (2.102.1-alpha.0)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

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

This PR has been generated by Renovate Bot.

⚠ Artifact update problem
Renovate failed to update an artifact related to this branch. You probably do not want to merge this PR as-is.
♻ Renovate will retry this branch, including artifacts, only when one of the following happens:

any of the package files in this branch needs updating, or
the branch becomes conflicted, or
you click the rebase/retry checkbox if found above, or
you rename this PR's title to start with "rebase!" to trigger it manually

The artifact failure details are included below:
File name: package-lock.json
npm WARN cli npm v10.2.1 does not support Node.js v20.2.0. This version of npm supports the following node versions: `^18.17.0 || >=20.5.0`. You can find the latest version at https://nodejs.org/.
npm ERR! code ERR_SOCKET_CONNECTION_TIMEOUT
npm ERR! errno ERR_SOCKET_CONNECTION_TIMEOUT
npm ERR! request to https://registry.npmjs.org/telejson failed, reason: Socket connection timeout

npm ERR! A complete log of this run can be found in: /tmp/renovate/cache/others/npm/_logs/2023-10-29T09_05_56_823Z-debug-0.log


Add range for auto renew period

Problem
Users should only be able to select a value for auto renew period that is within the defined range creating a transaction.
ledger.autoRenewPeriod.maxDuration=8000001
ledger.autoRenewPeriod.minDuration=6999999

Solution
Only allow users to select an auto renew value within the provided range.
Alternatives
No response

There is already a range check on the app. The issue reported by Alex stems from a bug in the Account Update Transaction creation


Add /interfaces endpoint

This endpoint retrieves the available network devices from the UCI config.

[ ] read out devices assigned to lan

[ ]  read out devices assigned to wan

[ ] read out devices assigned to mesh

[ ]  read out wireless devices assigned to lan

[ ]  read out wireless devices assigned to mesh

in progress here https://github.com/althea-mesh/althea_rs/pull/263


Ensure we do not leak open handles

.dir uses up a filehandle handle until its Seq is exhausted, so
currently we use up a handle per directory level we recurse into.
This causes the routine to crash on systems with low open files
limit, or in programs that open up a lot of handles prior to calling
our routines.
Fix by evaluating .dir eagerly, so it closes its handle before recurse.
I also included a test covering this bug, but it needs ulimit -n 100 set to test the stuff properly.

Looks legit to me.
In retrospect, the .nots look a little funny, but hey if it's a feature, someone should use it.  Might as well be my code.


at-cc: Support HSC and DC-to-DC power module device

Summary:

Support HSC (ADM1272) device.
Support DC-to-DC power module (Q50SN120A1) device.

Test Plan:

Build code: Pass

Hi @GoldenBug ,
Thanks for your comments.
Based on your suggestions to modify the code, rebase main and force update this branch.
Please help to review it again, thank you.