Cannot find router when the router is not hosted at <IP_ADDRESS>

My router defaulted to <IP_ADDRESS> and was not discoverable. I changed the ip to <IP_ADDRESS> and it started working.

@adotout my router is not at <IP_ADDRESS> and Open.NAT can discover it well. Also, I have UPnp test services hosted at <IP_ADDRESS> and <IP_ADDRESS> and they are discovered with no problem. I think it has to be something dífferent and no related to ip address. Could you please share the network log here?

It looks like the network log at the time it wasn't working is fairly empty. I'm no longer able to reproduce the issue regardless of the port the router is on. It seems like my router must have been having an issue that was resolved by the reboot required to change the port from <IP_ADDRESS> -> <IP_ADDRESS>.
Thanks for taking a look, closing because I can no longer reproduce.
The network log, for the record:
Open.NAT Information: 0 : Start Discovery
Open.NAT Information: 0 : Searching for: UpnpSearcher
Open.NAT Information: 0 : Stop Discovery
Open.NAT Information: 0 : Device not found. Common reasons:
Open.NAT Information: 0 : 	* No device is present or,
Open.NAT Information: 0 : 	* Upnp is disabled in the router or
Open.NAT Information: 0 : 	* Antivirus software is filtering SSDP (discovery protocol).
System.Net Error: 0 : [12372] Exception in AppDomain#12547953::UnhandledExceptionHandler - One or more errors occurred..
at System.Threading.Tasks.Task.ThrowIfExceptional(Boolean includeTaskCanceledExceptions)
at System.Threading.Tasks.Task.Wait(Int32 millisecondsTimeout, CancellationToken cancellationToken)
at System.Threading.Tasks.Task.Wait()
at Open.Nat.ConsoleTest.NatTest.Main(String[] args) in C:\Users\behrlich\Desktop\Open.NAT\Open.Nat.ConsoleTest\Main.cs:line 42
at System.AppDomain._nExecuteAssembly(RuntimeAssembly assembly, String[] args)
at System.AppDomain.ExecuteAssembly(String assemblyFile, Evidence assemblySecurity, String[] args)
at Microsoft.VisualStudio.HostingProcess.HostProc.RunUsersAssembly()
at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state)
at System.Threading.ThreadHelper.ThreadStart()


Improve the PUT /repository/:name: to update all data of a repository, not only the name

hello

hello


./build_mac.sh error

I am getting this error while trying to build on macOS
Recreating c27196f3aef3_insurance-ca ... error

Recreating dfb72131b321_orderer0     ... error
r failed programming external connectivity on endpoint insurance-ca (1a36d77bd84d1a823381ebb1f5e8f3f29df90ad56ad43c141c7daa9ef31d1e1d): Bind for <IP_ADDRESS>:7054 failed: port is already allocated'
Recreating shop-ca                   ... done
ERROR: for dfb72131b321_orderer0  Cannot start service orderer0: b'driver failedRecreating repairshop-ca             ... done
Recreating police-ca                 ... done
s already allocated'

ERROR: for insurance-ca  Cannot start service insurance-ca: b'driver failed programming external connectivity on endpoint insurance-ca (1a36d77bd84d1a823381ebb1f5e8f3f29df90ad56ad43c141c7daa9ef31d1e1d): Bind for <IP_ADDRESS>:7054 failed: port is already allocated'

ERROR: for orderer0  Cannot start service orderer0: b'driver failed programming external connectivity on endpoint orderer0 (d9ba6d0bd841d6ca8e2a4cff8641451b209a2c2f5c8c7c7073b0690c7f2e69e8): Bind for <IP_ADDRESS>:7050 failed: port is already allocated'
ERROR: Encountered errors while bringing up the project.

@raymondfx

The project is updated with the latest fabric now
run ./clean.sh to get rid of existing images and containers and then run it with ./build_*.sh.


Adding description data for party and role

Description
This PR includes the following proposed change(s):
Adding description fields
Adding demo data for front end
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Refactoring / Documentation
[ ] Version change

if your change is a breaking change, please add breaking change label to this PR
How Has This Been Tested?
Please describe the tests that you ran to verify your changes.
Does the change impact or break the Docker build?

[ ] Yes
[X] No

If Yes: Has Docker been updated accordingly?

[ ] Yes
[ ] No

Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[X] New and existing unit tests pass locally with my changes

Looks good to me.
The only comment might be to create objects for roleType and partyType instead of returning the code and description, but that's obviously a lot more work and probably unnecessary.

Yes, if we end up re-using this should be done


Update verify_linter.py

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your PR is merged into master branch, a new PR will be created to update src/index.json automatically.
The precondition is to put your code inside this repo and upgrade the version in the PR but do not modify src/index.json.

CI

Hi @v-chbhoi, May I ask you to add some specific print info other than testing ?

For checking the build trigger only, I had simply added. This PR is no more required. Could you please remove or delete this? I am not able to do.

remove


ciscoise_sgt does not permit valid "-1" value for auto-generated SGT

Prerequisites

[x] Have you tested the operation in the API directly?
[x] Do you have the latest Terraform provider version?
[x] Review the compatibility matrix before opening an issue.

Environment:

ISE version and patch: 3.1 patch 7 & 3.2 patch 4
Terraform version: 1.6.4
ISE provider version: 0.6.22-beta
OS version: MacOS 13.6.1

Describe the bug
When using the 'ciscoise_sgt' resource to create a new SGT using an auto-generated number, the following error is returned:
│ Error: "parameters.0.value" cannot be lower than 2: -1
│
│   with ciscoise_sgt.sgt_shared_services,
│   on sgt.tf line 7, in resource "ciscoise_sgt" "sgt_shared_services":
│    7:     value = -1
Expected behavior
The "-1" value is a valid input for creating an auto-generated SGT number as per the API documentation, so the resource should allow this value.
https://developer.cisco.com/docs/identity-services-engine/latest/#!sgt
"value | Integer | Yes | Value range: 2 ot 65519 or -1 to auto-generate"
Terraform code example
resource "ciscoise_sgt" "sgt_shared_services" {
  provider = ciscoise.ise31-3
  parameters {
    name  = "Shared_Services"
    value = -1
  }
}

Successful API call that returns a 201 response
{
    "Sgt": {
        "name": "Shared_Services",
        "value": -1
    }
}

@jbogarin can you please have a look into this issue? It seems like it should be a relatively quick and easy fix.
Thanks

Tested in 0.8.1-beta and it´s working.
Closing issue, please reopen if you get some new trouble.


No connection to mqtt broker. Check/edit mqtt settings

Hi,
i've got a problem to get my device connect to the mqtt broker. I'm using amqtt on the device. At home the connection works, at the office, with password protected mqtt server (mosquitto) i get
Connecting to MQTT...
Disconnected from MQTT, reason: TCP Disconnected.

in serial monitor. 2 questions: does the amqtt library in mclighting support username and password? And is there a way to change the mqtt settings after the device is out of ap-mode?
Kind regards

Yes both MQTT and AMQTT support passwords. To reset settings, go to a web browser and type http://<IP_ADDRESS>/reset_wlan , where replace IP_ADDRESS with actual IP address.

Thank you very much. It seems that it is an issue with our office network (unfortunately). But to know how to edit the settings is very helpful.


Rethink Request events

Right now Request events are tied to XMLHttpRequest but maybe they should be tied to Requests instead; cf #43 and who knows if they should be dispatched on the document or just the individual requests tbh.

Note also that right now Authorization.Request() doesn't dispatch Request events even though it makes a number of requests.


Add cadical to the list of available solvers

Description of changes:
With the upgrade to CBMC 5.77.0 which is shipped with CaDiCaL that can be selected with --sat-solver cadical, add cadical as a valid argument to the --solver option and the kani::solver attribute.
Resolved issues:
Related RFC:

Optional #ISSUE-NUMBER.
Call-outs:

Testing:

How is this change tested? Added two tests

Is this a refactor change? No

Checklist

[X] Each commit message has a non-empty body, explaining why the change was made
[X] Methods or procedures are documented
[X] Regression or unit tests are included, or existing tests cover the modified code
[X] My PR is restricted to a single feature or bugfix

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT licenses.

Cadical didn't get included in CBMC's macos package. Turning this into a draft till this is addressed.

CBMC tracking issue: https://github.com/diffblue/cbmc/issues/7555


VS2017 caches but does not updates the Directory.Build.props file

VS2017 caches but does not updates the Directory.Build.props file
https://developercommunity.visualstudio.com/content/problem/433497/visual-studio-directorybuildprops-caching.html

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

ID: d410ebeb-3841-a0a9-efb4-185823add4dc
Version Independent ID: 7e397179-4ccc-054c-8137-5e8f2430d794
Content: Customize your build - Visual Studio
Content Source: docs/msbuild/customize-your-build.md
Product: visual-studio-windows
Technology: msbuild
GitHub Login: @Mikejo5000
Microsoft Alias: mikejo

Please see the Issue below. In legacy project system, once cached in .vs, it is never updated.
https://github.com/dotnet/project-system/issues/2888
I think this limit should be documented.


Removed the extra space in expected test result

Signed-off-by: Arash Rohani<EMAIL_ADDRESS>Description
Removed the extra space in expected test result to fix the test workflow.
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[x] The quickstart code compiles correctly
[x] You've tested new builds of the quickstart if you changed quickstart code
[x] You've updated the quickstart's README if necessary
[x] If you have changed the steps for a quickstart be sure that you have updated the automated validation accordingly. All of our quickstarts have annotations that allow them to be executed automatically as code. For more information see mechanical-markdown. For user guide with examples see Examples.

LGTM, works in my local kubernetes.  Let's try it in the action now.  Thank you for the contribution @Arashrohani

LGTM, works in my local kubernetes.  Let's try it in the action now.  Thank you for the contribution @Arashrohani

Hi @paulyuk Thanks Paul, it seems that the order of messages is different sometimes, would you please re-run the workflow or do you have any suggestions on how to fix the order issue?

Hi @paulyuk Thanks Paul, it seems that the order of messages is different sometimes, would you please re-run the workflow or do you have any suggestions on how to fix the order issue?

Hey @Arashrohani - I can rerun it easily but I dont have a good suggestion for now how we can deal with the async nature of it.
I'm noticing one other thing that may be a problem in the C# test-- see how this shows info: Program[0]?  I have a theory that is causing the markdown test to think it wasn't output even though we see it here.  Could you please change the C# code and readme to use Console.Writeline( ) instead of Logger.LogInformation in all cases?
		== APP == info: Program[0] 		== APP ==       B: Message on B 		== APP == info: Program[0] 		== APP ==       C: Message on C 		== APP == info: Program[0] == APP ==       A: Message on A 		ℹ   		terminated signal received: shutting down 		✅  Exited Dapr successfully 		✅  Exited App successfully 	ERROR expected lines not found: == APP ==       B: Message on B == APP ==       C: Message on C Exited Dapr successfully Exited App successfully
One more thing do you happen to be on Discord or some other way to faster chat as we figure this out?

@paulyuk Replaced it with Console.WriteLine but it didn't fix the order issue (it broke the first test also but I think that is related to whitespaces), I think the best approach is to add a delay or update the test runner or we can just keep one of those messages in the step (I don't like this)

@paulyuk Replaced it with Console.WriteLine but it didn't fix the order issue (it broke the first test also but I think that is related to whitespaces), I think the best approach is to add a delay or update the test runner or we can just keep one of those messages in the step (I don't like this)

Ok do you want to try the delay?  The test runner update might be impactful and more than we bargained for -- for now.

Hey @paulyuk Hmm, Sure, where should I update that? in readme? I though that is going to be somewhere else?! Do you want me to change it back to use logger?

@Arashrohani - Really appreciate this contribution and finding this issue!


Utilize MarshalJSON and UnmarshalJSON interface implementations

Is your feature request related to a problem? Please describe.
Custom types can sometimes produce empty values in a RethinkDB document. I have implemented my own decimal type:
type Decimal struct {
	flags uint32
	high  uint32
	low   uint32
	mid   uint32
}

// MarshalJSON returns the decimal as a text string without quotes
func (d Decimal) MarshalJSON() ([]byte, error) { return d.MarshalText() }

// MarshalText encodes the receiver into UTF-8-encoded text and returns the result.
func (d Decimal) MarshalText() (text []byte, err error) {
	text = []byte(d.String())
	return text, nil
}

// UnmarshalJSON unmarshals the JSON value, ignoring quotes
func (d *Decimal) UnmarshalJSON(text []byte) error {
	return d.UnmarshalText(text)

}

// UnmarshalText unmarshals the decimal from the provided text.
func (d *Decimal) UnmarshalText(text []byte) (err error) {
	*d, err = Parse(string(text))
	return err
}

It implements both json.Marshaler and json.Unmarshaler. This type encodes and decodes without issue using the standard encoding/json package. So I was surprised to see the following document stored in RethinkDB
{
"candle": {
"bar_seqno": 12024547 ,
"close_price": { } ,
"high_price": { } ,
"low_price": { } ,
"open_price": { } ,
}
....
}

when using
type Candle struct {
  BarSeqno int `json:"bar_seqno"`
  OpenPrice Decimal `json:"open_price"`
  HighPrice Decimal `json:"high_price"`
  LowPrice Decimal `json:"low_price"`
  ClosePrice Decimal `json:"close_price"`
}

candle := Candle{
  BarSeqno: 12024547,
  OpenPrice: decimal.NewFromString("1.33028"),
  HighPrice: decimal.NewFromString("1.33028"),
  LowPrice: decimal.NewFromString("1.33028"),
  ClosePrice: decimal.NewFromString("1.33028"),
}

err := r.Table("candles").Insert(candle).Exec(session)

What I would expect to see is
{
"candle": {
"bar_seqno": 12024547 ,
"close_price": 1.33028 ,
"high_price": 1.33028 ,
"low_price": 1.33028 ,
"open_price": 1.33028 ,
}
....
}

Describe the solution you'd like
If this library could use the json.Marshaler and json.Unmarshaler implementations, I would get the expected value by just using
err := r.Table("candles").Insert(candle).Exec(session)

Describe alternatives you've considered
My workaround comes from this issue and is basically:
candles = []Candle{candle1, candle2, candle3}
b := new(bytes.Buffer)
	for _, candle := range candles {
		if err = json.NewEncoder(b).Encode(candle); err != nil {
			return err
		}
		if err = r(name).Insert(r(b.String())).Exec(session); err != nil {
			return err
		}
		b.Reset()
	}

This is not only more verbose but also creates separate calls to Insert instead of sending a batch which impacts performance and also eliminates the transaction-like quality of sending a slice of objects to Insert.
Additional context
Is there something about RethinkDB or this library that would prevent adding this functionality? I would be happy to give it a try but not if someone has already proven it is a bad idea.

Update on the performance implications of the workaround above. Note: These tests are from my laptop in France to a hosted server in Germany. The raw values might not reflect on-server performance, but the relative values demonstrates the performance problem.
existing behaviour, which strips out the decimal values
t0 := time.Now()
err = rethinkdb.Table(name).Insert(candles[:100]).Exec(session)
d := time.Since(t0)
log.WithField("duration", d.Seconds()).Info("test")

time to insert 100 records is 0.23339381 seconds
using above work around
t0 := time.Now()
	for _, candle := range candles[:100] {
		if err = json.NewEncoder(b).Encode(candle); err != nil {
			return err
		}
		if err = rethinkdb.Table(name).Insert(rethinkdb.JSON(b.String())).Exec(session); err != nil {
			return err
		}
		b.Reset()
	}

d := time.Since(t0)
log.WithField("duration", d.Seconds()).Info("test")

time to insert 100 records is 5.65757508 seconds
batching the encoding
I also tried the following
	b := new(bytes.Buffer)
	bufs := make([]string, 100)
	for i, task := range tasks[:100] {
		if err = json.NewEncoder(b).Encode(task); err != nil {
			return err
		}
		bufs[i] = b.String()
		b.Reset()
	}
	t0 := time.Now()
	err = rethinkdb.Table(name).Insert(bufs[:100]).Exec(session)
	d := time.Since(t0)
	log.WithField("duration", d.Seconds()).Info("test")

to see if the slow down was in the encoding, or the rethinkdb-go actions. Interestingly, no documents were created in the RethinkDB database yet no error was given.


onSwipeStart only called once per element

Describe the bug
When using a trailing and leading action on a single element, onSwipeStart will only be called once.
To Reproduce
Steps to reproduce the behavior:

Partially swipe right.
Observe that onSwipeStart will trigger
End swipe.
Partially swipe left.
Oberse that onSwipeStart will NOT trigger.

Expected behavior
onSwipeStart should trigger every time a new sequence is started.
Note: it does work when swiping on another SwipableListItem and afterwards returning to the original element.

Hi @ArondeParon - what do you mean in point 3? Do you release the click/touch or not?
I have tried here: https://marekrozmus.github.io/react-swipeable-list/ and when I go like this:

Partially swipe right.
onSwipeStart triggers
End swipe (release the mouse or touch)
Partially swipe left.
onSwipeStart triggers.

@marekrozmus the issue seems to occur only when swiping from a "locked" state (not sure what you call it?) to another "locked" state. It's a bit hard to describe, so I recorded a video showing the behavior.
https://github.com/marekrozmus/react-swipeable-list/assets/7697/31635a5e-54b0-46e6-be17-6ab14e945565

Hi @ArondeParon - what kind of case you need that specific behaviour?
I mean now it works that swipe start is triggered when user moves item beyond the threshold and the swipe end when user release mouse button.
If you enter the "locked" state you are still "inside the swipe" state.
I think that what we are missing here is the scenario when user is swiping from "locked" state in opposite direction.
Then it should go through initial state and then new swipe start with opposite direction should be triggered?


[BUG] conflicting vault names

Describe the bug
It is possible to create the same vault name when first creating a vault on device A and pairing on device B and afterwards initating on Device B and pairing with device A when no input in vault name.
To Reproduce
Steps to reproduce the behavior:

No input in vault name
create secure vault on device A and join with device B
no input in vault name
create secure vault on device B and join with device A
See error

Expected behavior
Add prefilled names for different Vault setups, to not have conflicts add random numbers #:
Fast: Fast Vault #
Active: Active Vault #
Secure: Secure Vault #
Screenshots

-v25

QA done.


Display last modified date of an annotation

Currently it displays the create date, but not the last modified date.

Closing this because it's a duplicate of informatics-isi-edu/rbk-project#144


Don't translate database state

          1 Warning

:warning:
There're changes, but no changelog.

  Generated by :no_entry_sign: Danger


Allow layered ScrollContainers to coexist on input handling

Closes #3576

This changes drag event handling and scroll event handling in ScrollContainers to be based on their ScrollDirection, to allow for layered scroll containers to coexist.
This however becomes a nuisance on two parts:

https://github.com/ppy/osu-framework/commit/20c1d72502c4f85d5e23def9a28aa02a5725dbb6: In order for scroll containers to coexist, the innermost must not OnMouseDown to let the parenting scroll containers enter the mouse-down/drag input queue, for receiving drag events.
I'm not entirely sure if there's a reason for that in the first place, checking with git blame threw me back to 4 years ago (https://github.com/ppy/osu-framework/pull/316).

https://github.com/ppy/osu-framework/commit/7bff9b96bebc982b5dfabf75cd714b41237bf25a: When a scroll container is horizontal, scrolling vertically will also be considered as horizontal, therefore this coexistence change cannot apply when the inner scroll container is horizontal (as done in test).
In addition, it also results in this kinda weird UX (inner = horizontal, outer = vertical):
https://user-images.githubusercontent.com/22781491/164593522-314825a1-8388-47c6-b9c2-090fe519fd3e.mp4

Unsure how to deal with this test:

Feels like an arbitrary choice to drag perfectly diagonally and check whether the scroll container would block the click or not, since it's going to depend on which is checked first in the early-guard.

The diagonal in that test does look arbitrary. Changing to a specific direction is probably fine.

Made some minor refactors, but looks and feels good to me. Will wait for a second opinion for merge.


Feature: Add edit code with VSCode button to repository code page

Before you start...

[X] Have you checked if a similar feature request has already been reported?

Description
Simplest idea: integrate a magnet button to open system default text-editor (Visual Studio Code, Atom, Sublime Text, etc etc).
Alternative solutions
In addition to that, we could integrate vscode.dev inside a chromium instance so final user can code directly inside FluentHub: for example, if I just had to add a link to README.md of an awesome-list, I'll click on "Edit in FluentHub" (or something similar), code and pull.
I can help you for both ideas.
Relevant Assets
No response

Want to add "Edit in FluentHub" like vscode.dev in your browser and create something similar to text editor into FluentHub?

If so, it's a good idea. but the goal up to version v0.5 is to make all queries work correctly and with as few bugs as possible. The goal up to v0.9 is to make all mutations work correctly and with as few bugs as possible. We will also do a final check to migrate all Alpha phase features to Stable by v1.

I was talking about adding an integrated instance of vscode.dev inside FluentHub.
If you wants, I could develop it, pull a request and wait until v0.5. Meanwhile waiting I could help on bug fixes.

Just don't modify GitHub data? I don't know what “integrated instance” was.

Just don't modify GitHub data? I don't know what “integrated instance” was.

I think you should read this, it is ofc more specific than me.
For integrated instance I mean I would like to have a page like vscode.dev inside FluentHub.
I think we're talking about the same thing but with different synonyms. 😁

Ah, when clicking the edit button, the app opens WebView2 windows with vscode.dev?

Yep.

Got it. It is a good idea! if you can open a PR, please do.
you think it is like this, right?

Exactly! If is okay for you, I'll work on it soon!

Wait just to make things clear.. @onein528 you want to add WebView2 with github.dev page to FH?

No this feature request is not like that.
just open github.dev with browser.

No this feature request is not like that. just open github.dev with browser.

oh. Can't we use WebView? @onein528

can use. but he not meant to. @dovatti How about?

Not yet. I think this will take some time to start working on if alone. @DeveloperWOW64 if you would like to implement that would be great, I would help in my free time. I do work every day so I do not have too much spare time.

@onein528 @DeveloperWOW64 I've got some news: I've been coding according to these guide and essentially got code working.
Currently could be possible to get an "Open in VSCode.dev" button and open the web page inside FluentHub. But I was thinking: does exist an "official" pattern/model to achieve that? I think more than one person in UWP community would like to integrate WebView2 already wrapped with it's own UI. If so we could use this coding pattern instead of my code monster 😆
Otherwise, I've been already thinking about a Fluent wrapper for VSCode UI; but I do not think would be perfect as I want.
I'll release soon all the project folder with schemas, graphs and resources, and obviously with code working for WebView at least. Don't know when because I've been working and studying very hard this months 👍🏻

I don't see why this isn't possible.
Just create a button with a link to github.dev/org/repo

Or even something like this:

Yeah ultimately i wanna adopt with VSCode UI.

I found this URL scheme:
https://github.com/codespaces/new?hide_repo_select=true&ref=main&repo=0x5bfa/fluenthub

I'll release soon all the project folder with schemas, graphs and resources, and obviously with code working for WebView at least. Don't know when because I've been working and studying very hard this months 👍🏻
@dovatti are you still active? Could you release this code at some point?


Is there any demo app? How to use with Seneca?

How would I use this with seneca? Please provide an example :)
Thanks.

Hi Kristian, I built this to replace seneca and allow microservices to directly talk to Rabbitmq. It was before Seneca rebuilt their documentations and went through a pivot, but I can cook up a quick example over the weekend if that helps! You can also see a pair of client and server sample under the Example folder.


BUG: Not using environmental variable for enabled in the runtime

Describe the bug
In the example project template there is a config named enabled as an environmental variable but it does not support environmental variables in the run time.
Reproduction
kedro neptune init will add following config in the base config
enabled: $NEPTUNE_ENABLED
exporting relevant enviroment variable
export NEPTUNE_ENABLED=false
Could be solved with: https://github.com/neptune-ai/kedro-neptune/pull/35
Expected behavior
No metrics will be passed to Neptune
Traceback
Environment
**The operating system you're using: Mac OS
**The output of python --version: python 3.9
Additional context

Hi,
Prince Canuma here, a Data Scientist at Neptune.ai,
Thank you for the feedback! We are always looking for ways to improve our services.
I'm glad you shared the solution too, I have notified the devs to review and that PR.
Meanwhile, I would like to ask what is your use case for you to want to disable Neptune tracking using an env variable? This will help me better understand and explain to the product team the use case for env variables support.

Hi @ErayGjg
I'm just checking back to see if you still need help with this issue? or if you have any other questions I can help you with?

Hi @Blaizzy
We still want to control whenever the neptune wıll be enabled or not via environmental variables rather than using hard coded parameters.

Great! We are working on that with a contributor.
You can follow the progress here. https://github.com/neptune-ai/kedro-neptune/pull/35

Let me know this solve your issue!


DOC: asv licensing

Since this is under airspeed-velocity, update the links and licenses.

Closed by #9.


Implement generate_series for Decimal128 instead of floats

Context
select * from generate_series(0.0, 1.0, 0.1);

Expected
0.0
0.1
0.2
0.3
0.4
0.5
0.6
0.7
0.8
0.9
1.0
Actual
┌─────────────────────┐
│ generate_series     │
│ ──                  │
│ Float64             │
╞═════════════════════╡
│ 0.0                 │
│ 0.1                 │
│ 0.2                 │
│ 0.30000000000000004 │
│ 0.4                 │
│ 0.5                 │
│ 0.6                 │
│ 0.7                 │
│ 0.7999999999999999  │
│ 0.8999999999999999  │
│ 0.9999999999999999  │
└─────────────────────┘

Added needs docs because I held off on documenting an example since the support isn't quite there yet.

Gonna also add a test label to ensure we have coverage for my example

Yeah, this is expected since we're dealing with floating-point numbers. We need another implementation with Decimal128 (which should be used by default instead of floats). I'll create an issue for this.

EDIT: Or we can use this issue for tracking. I'll update the title.

Appreciated!

Needs some improvements but almost there...
> select * from generate_series(0.0, 1.0, 0.1);
┌───────────────────┐
│ generate_series   │
│ ──                │
│ Decimal128(38, 6) │
╞═══════════════════╡
│ 0.000000          │
│ 0.100000          │
│ 0.200000          │
│ 0.300000          │
│ 0.400000          │
│ 0.500000          │
│ 0.600000          │
│ 0.700000          │
│ 0.800000          │
│ 0.900000          │
│ 1.000000          │
└───────────────────┘


Fix download times metrics

What
Remove the monitor.csv.download.seconds.slow counter
Why
We were sending this counter to calculate the
percentage of slow downloads.
This is no longer neccessary, as we are going,
to use percentiles of the raw metric, so the code is redundant.
What
Use GovukStatsd timing not count
refactor the call to use timing with
the number of milliseconds.
Why
I called the wrong method when I implemented the
CSV download time Statsd call.
Trello: https://trello.com/c/3Eg16r1C/1257-3-build-sli-grafana-dashboard

Review Checklist

[ ] Changes in scope.
[ ] Added/updated unit tests.
[ ] Added/updated feature tests.
[ ] Added/updated relevant documentation.
[ ] Added to Trello card.

 (3) Build SLI Grafana Dashboard


Multiple Errors on Installation

Empire Version
current
OS Information (Linux flavor, Python version)
uname -a
Linux kali 4.12.0-kali2-amd64 #1 SMP Debian 4.12.12-2kali1 (2017-09-13) x86_64 GNU/Linux
python --version
Python 2.7.13
Expected behavior and description of the error, including any actions taken immediately prior to the error. The more detail the better.
Installation script should install every dependency and compile without error.
However libssl1.0.0 can not be found and the xar path is not checked if it contains spaces.
/install.sh
--2017-09-25 11:44:15--  https://bootstrap.pypa.io/get-pip.py
Resolving bootstrap.pypa.io (bootstrap.pypa.io)... <IP_ADDRESS>
Connecting to bootstrap.pypa.io (bootstrap.pypa.io)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 1595408 (1.5M) [text/x-python]
Saving to: ‘get-pip.py’

get-pip.py                            100%[========================================================================>]   1.52M   381KB/s    in 4.1s

2017-09-25 11:44:19 (376 KB/s) - ‘get-pip.py’ saved [1595408/1595408]

Requirement already up-to-date: pip in /usr/local/lib/python2.7/dist-packages
Reading package lists... Done
Building dependency tree
Reading state information... Done
Package libssl1.0.0 is not available, but is referred to by another package.
This may mean that the package is missing, has been obsoleted, or
is only available from another source

E: Package 'libssl1.0.0' has no installation candidate

[...]

/usr/bin/ranlib lib/libxar.a
sed -e s/<EMAIL_ADDRESS>-e s/<EMAIL_ADDRESS>-e s/<EMAIL_ADDRESS>< lib/libxar.la.in > lib/libxar.la
gcc -Wall -g  -Iinclude -Iinclude -D_GNU_SOURCE -I/usr/include/libxml2 -c src/xar.c -o src/xar.o
gcc -Wall -g  -o src/xar src/xar.o -Wl,-rpath,/root/Desktop/Pentest/Tools/Windows Domain/Empire/setup/xar-1.5.2/lib -Llib  lib/libxar.a -lz -lcrypto -lacl  -lxml2
gcc: error: Domain/Empire/setup/xar-1.5.2/lib: No such file or directory
src/Makefile.inc:39: recipe for target 'src/xar' failed
make: *** [src/xar] Error 1

This will result in an unlimited generation of the following error when executing the Empire RESTApi:
 * Starting Empire RESTful API on port: 1337
 * RESTful API token: mny[.....]kk
[!] Error: cannot find certificate ./data/empire.pem

Any additional information
Seems like the "Empire/setup/xar-1.5.2/lib" can not be found because there is a space in the path ("/Windows Domain/"). Cloning and installing it in a directory without spaces will not throw this error.

Same for me... The installer seems broken on Kali 2017.2.

I'm getting the following after a fresh install:
root@kali:~# uname -a
Linux kali 4.12.0-kali1-amd64 #1 SMP Debian 4.12.6-1kali6 (2017-08-30) x86_64 GNU/Linux
root@kali:~# python -V
Python 2.7.13
[ran ./install.sh]
...
checking openssl/evp.h presence... yes
checking for openssl/evp.h... yes
checking for OpenSSL_add_all_ciphers in -lcrypto... no
configure: error: Cannot build without libcrypto (OpenSSL)
make: *** No targets specified and no makefile found.  Stop.
make: *** No rule to make target 'install'.  Stop.
Cloning into 'bomutils'...

@gerrenmurphy I fixed the libz problem by installing the zlib package.

@whoot those are actually two separate issues. The issue with the rest api and the pem file is fixed in the dev branch. Did any of the above suggestions work for you??

Pulling the current files fixed the issues. Thanks!

Getting the same issue now as @gerrenmurphy.
Fixed it by installing zlib package and replacing 'libssl1.0.0 libssl-dev' with 'libssl1.0-dev' in install.sh.
Otherwise HTTPS Listeners wont work!

Made an pull request for this issue: #811

@whoot We now have a docker build release. This should resolve issues with libcrypto. Please let me know if this works for you. #848


Migrate to Rust 1.75.0

Release notes: https://blog.rust-lang.org/2023/12/28/Rust-1.75.0.html
Main takeaways for us: -> impl Trait in traits now supported, which means we get async fn in traits as well (without async-trait crate).
There are still some limitations outlined here that discourage the usage in public APIs.

done in #83


Question: does this work Sapling?

So I'm playing around with Sapling and while I really like the interface, there are times when having tools like lazygit or gitui is a plus. So I found this tool. I was wondering if it has support for .sl :-)

[open source] Sapling does not acknowledge an .hg folder. We did a considerable amount of work to ensure users can have Mercurial and Sapling installed side-by-side such that they don't interfere with one another.
Currently, the presence of an .sl folder means it is a Sapling working copy, but you have to peek inside the .sl/config folder to see what "type" of repository it is. In practice, the repository is inevitably a Git repository because we haven't made the Sapling server "available" (well, the code is on GitHub, but it's a lot of work to set one up...).
If you want to do Git things "sometimes" when using Sapling, you likely have to git clone the repo and do the work over there. Not great, but that's the situation.
Meanwhile, I know evolving Sapling's existing UI into a TUI is something folks on the team are interested in. In general, I think we're more interested in improving the UX of Sapling rather than try to support more and more Git tools within Sapling.

Thanks!
Ok since OSS Sapling is still just a way to clone an existing git repo I think I'm going to say we're not going to support Sapling repos, yet. When the day comes that Sapling Server is ready for OSS we can re-evaluate then.


Lmdb3

New simpler interfaces for database opening
Custom comparators for tables
Protection for mdb_mapsize to avoid data file truncation
secp256k1 settings for GMP on MSVC (yet slower cause it supports only inverse fields)

Codecov Report

Merging #82 into master will decrease coverage by 0.52%.
The diff coverage is 53.84%.

@@            Coverage Diff             @@
##           master      #82      +/-   ##
==========================================
- Coverage   82.10%   81.58%   -0.53%
==========================================
  Files          51       51
  Lines        4024     4061      +37
==========================================
+ Hits         3304     3313       +9
- Misses        720      748      +28

Impacted Files
Coverage Δ

silkworm/db/chaindb.hpp
35.71% <0.00%> (ø)

silkworm/db/tables.cpp
38.46% <20.00%> (-61.54%)
:arrow_down:

silkworm/db/chaindb.cpp
54.23% <56.55%> (-1.52%)
:arrow_down:

Continue to review full report at Codecov.

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


Hide 'CHANGES SAVED' message and 'Save' button if no changes have been made

Is your feature request related to a problem? Please describe.
When the editor loads, it shows 'CHANGES SAVED' and the 'save' button before any changes are made.
Describe the solution you'd like
Instead of showing 'CHANGES SAVED', it should be blank until a change is made. After a change has been saved, this message should show. The 'Save' button should be disabled in all cases unless there are unsaved changes.
Describe alternatives you've considered
none
Additional context

Figure: Message 'CHANGES SAVED' and the 'save' button shown before anything was changed

Figure: Mockup - Proposed UI

Hi @bradystroud, this would make a nice improvement. We would need to avoid layout shifting when hiding/showing the button. For example in the UI mockup the Writing in... text is centered and in the actual UI it is pushed a bit up due to the existing Changes saved text.


Unable to log into BlueXolo

Describe the bug
When I try to log into BlueXolo Framework on the web browser, it shows me an error on the web page about django.session does not exist.
To Reproduce
Steps to reproduce the behavior:

Run 'python manage.py runserver <IP_ADDRESS>:8000' command
Go to <IP_ADDRESS>:8000 on the web browser
Click on the login
Enter credentials
See error

Expected behavior
Be able to log into BlueXolo Framework.
Desktop (please complete the following information):

OS: Centos 7
Browser: Firefox

I noticed the console shows: 'django.db.utils.ProgrammingError: syntax error at or near "WITH ORDINALITY"'.
For resolve this issue you could update PostgreSQL 9.2 to 9.6:

Go to https://www.postgresql.org/download/linux/redhat/

Select your O.S.

Run: yum install https://download.postgresql.org/pub/repos/yum/reporpms/EL-7-x86_64/pgdg-redhat-repo-latest.noarch.rpm -y

Run: yum install postgresql96 postgresql96-server postgresql96-contrib postgresql96-libs -y

Initialize PostgreSQL:
/usr/pgsql-9.6/bin/postgresql96-setup initdb

Start/Enable PostgreSQL:
systemctl enable postgresql-9.6.service
systemctl start postgresql-9.6.service

Set a password to postgres user:
sudo -i -u postgres
psql
alter user postgres with encrypted password 'My_password';

Configure file to allow us to authenticate using passwords, change ident to md5:
vi /var/lib/pgsql/9.6/data/pg_hba.conf

Restart postgres service:
systemctl stop postgresql-9.6.service
systemctl start postgresql-9.6.service

Create user and database for bluexolo:
sudo -i -u postgres
psql
CREATE DATABASE yourdbname;
CREATE USER youruser WITH ENCRYPTED PASSWORD 'yourpass';
GRANT ALL PRIVILEGES ON DATABASE yourdbname TO youruser;

Set up BlueXolo and create new super user


CAA Fixes

This merge incorporates changes that allow us to pass all CAA test suite cases.

@sciros @gcimaszewski Unless there is objection, I think we should merge this branch into main given it is stable, accomplished the originally stated goal of improving CAA parsing, and it passes all integration and unit tests. Further improvements can be done on another branch dedicated to our next objective.

The README is now updated to more accurately reflect the current state of the project.

Yep, feel free to merge it in. I'll see about spinning off a branch to look at the distance/RIR logic for building cohorts next. It might end up being the same branch as retry logic since that's one of the main drivers for how cohorts are built.

I like the idea of a new branch for RIR/distance logic which might also take into account retries.


[RESOLVED] Interface eth0(): High error rate (>2 for 5m)

Problem started at 09:00:32 on 2023.04.15
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Operational data: errors in: 0, errors out: 0
Original problem ID: 17628
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23483&eventid=17628

Problem has been resolved in 2m 57s at 09:03:29 on 2023.04.15
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Original problem ID: 17628


foxglove recordings list`

Description
Running foxglove recordings list results in the following error:
❯ foxglove recordings list
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x10 pc=0x732f82]

goroutine 1 [running]:
github.com/foxglove/foxglove-cli/foxglove/console.RecordingsResponse.Fields({{0xc000028a60, 0x10}, {0xc000312000, 0x46}, 0xeabefe, 0x36, {0xc00002c168, 0x18}, {0xc00002c180, 0x18}, ...})
        /home/runner/work/foxglove-cli/foxglove-cli/foxglove/console/api.go:232 +0x2c2
github.com/foxglove/foxglove-cli/foxglove/cmd.renderList[...]({0xb43ca0, 0xc000014018}, 0xa61e42, 0x5?, {0xa61e5b?, 0x5})
        /home/runner/work/foxglove-cli/foxglove-cli/foxglove/cmd/utils.go:100 +0x3de
github.com/foxglove/foxglove-cli/foxglove/cmd.newListRecordingsCommand.func1(0xc0001f6280?, {0xa618b3?, 0x0?, 0x0?})
        /home/runner/work/foxglove-cli/foxglove-cli/foxglove/cmd/recordings.go:38 +0x5b0
github.com/spf13/cobra.(*Command).execute(0xc0001f6280, {0xf13268, 0x0, 0x0})
        /home/runner/go/pkg/mod/github.com/spf13/cobra@v1.3.0/command.go:860 +0x663
github.com/spf13/cobra.(*Command).ExecuteC(0xc0001b3680)
        /home/runner/go/pkg/mod/github.com/spf13/cobra@v1.3.0/command.go:974 +0x3bd
github.com/spf13/cobra.(*Command).Execute(...)
        /home/runner/go/pkg/mod/github.com/spf13/cobra@v1.3.0/command.go:902
github.com/foxglove/foxglove-cli/foxglove/cmd.Execute({0xb40f50, 0x7})
        /home/runner/work/foxglove-cli/foxglove-cli/foxglove/cmd/root.go:230 +0xd16
main.main()
        /home/runner/work/foxglove-cli/foxglove-cli/foxglove/main.go:11 +0x27

API key is loaded and working with other commands such as foxglove devices list. The API key I'm using has all options enabled.

Version:

❯ foxglove version
v1.0.12

Platform:
Windows 11 running WSL with Ubuntu 22.04

Steps To Reproduce

Pretty simply running the foxglove recordings list command.
Expected Behavior
List of at least some of the recordings in the data platform.

Thank you for the report; this should be fixed with v1.0.13


bump version

Codecov Report

Merging #2 (d5087f1) into main (edf0ce7) will decrease coverage by 94.84%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #2       +/-   ##
==========================================
- Coverage   94.84%   0.00%   -94.85%
==========================================
  Files           5       4        -1
  Lines          97      75       -22
==========================================
- Hits           92       0       -92
- Misses          5      75       +70

Impacted Files
Coverage Δ

src/mgen.jl
0.00% <0.00%> (-100.00%)
:arrow_down:

src/iseltypesuperofmissing.jl
0.00% <0.00%> (-100.00%)
:arrow_down:

src/handlemissings.jl
0.00% <0.00%> (-93.94%)
:arrow_down:

src/typediterator.jl
0.00% <0.00%> (-66.67%)
:arrow_down:

src/missingstrategy.jl

Continue to review full report at Codecov.

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


Project init

Set up a Next.js application with TailwindCSS using create-next-app
Include a .gitignore template
Install and use a formatter like Prettier

Resolved by #3


ISPN-5224 Publish the server .zip distribution to maven

https://issues.jboss.org/browse/ISPN-5224

Pushed, thanks

Thanks for sorting this out @isavin. This also helps with the remote tutorial, since it makes it easier to bring in and expand the server distribution ZIP.


All right Sideboxes on slides stacking to top

All Sideboxes on the right stack up at the top instead of being next to the question that the box is about. Ideally, we would be able to place Sideboxes next to the content they are linked to.
I've looked at putting

        [--box|example--]content[--endbox--]
        Questions and text

into the editor to try to keep them together but all 'boxes' are just pulled out and put in a different column regardless
How easy is it to undo that and give us a bit more control over the placement of the boxes?

Maybe this is a problem where css grid comes into its own

This is what it looks like at the moment

Ok I have solution to this. Each question should be in a new Bootstrap .row. Within that row should be the left column with the questions and answers in. And the right column with the context box for that question. Here's a screenshot and a dump of the html I wrote to demo it.
Can we build this into the default way that the html is contstructed in slides?

					Do you handle other sensitive data?

					There are other types of data that can bring privacy, social or ethical risks but do not receive
					special protection
					in the law as such.

<input id="check-answer[1]" name="answer[1]" type="checkbox"
								value="Location data"> Location data

<input id="check-answer[2]" name="answer[2]" type="checkbox"
								value="Real life biographical data"> Real life biographical data

<input id="check-answer[2]" name="answer[2]" type="checkbox"
								value="Other"> Other

						<textarea id="multianswer-2" name="multianswer[2]" class="form-control" rows="8"></textarea>

example

Personal data is information that relates to an identified or identifiable individual. This
							will be easy to establish when you are dealing with names or other clear identifiers such as
							IP addresses or cookies. In some cases, it may be difficult to establish whether the data is
							personal, for example if you only collect sensor data without any identifiers. In this
							situation you need to consider whether that data can be linked to other information you may
							be able to access.

					Do you handle specially protected data?

					European laws give special protections to certain categories of data. It is important that you are
					clear on whether
					you are dealing with these types of data and have the appropriate measures. These data also tend to
					raise important
					ethical quesitons beyond legal compliance.

<input id="check-answer[1]" name="answer[1]" type="checkbox"
								value="Racial or ethnic origin"> Racial or ethnic origin

<input id="check-answer[2]" name="answer[2]" type="checkbox"
								value="Political opinions"> Political opinions

<input id="check-answer[3]" name="answer[3]" type="checkbox"
								value="Trade union membership"> Trade union membership

<input id="check-answer[4]" name="answer[4]" type="checkbox"
								value="Medical data or sex life and sexual orientation"> Medical data or sex life and
							sexual orientation

<input id="check-answer[5]" name="answer[5]" type="checkbox"
								value="Genetic data"> Genetic data

<input id="check-answer[6]" name="answer[6]" type="checkbox"
								value="Genetic data"> Genetic data

<input id="check-answer[7]" name="answer[7]" type="checkbox"
								value="biometric data where processed to uniquely identify a person"> biometric data
							where processed to uniquely identify a person

example

Personal data is information that relates to an identified or identifiable individual. This
							will be easy to establish when you are dealing with names or other clear identifiers such as
							IP addresses or cookies. In some cases, it may be difficult to establish whether the data is
							personal, for example if you only collect sensor data without any identifiers. In this
							situation you need to consider whether that data can be linked to other information you may
							be able to access.

Unfortunately the main page content is all rendered in the same container, with an aside column to the right.  It's quite hard to split that container at the right points in the content during the template rendering.  Moving the layout divs into the page content would also prove problematic during the render of the recap.
As a yucky short-term measure, I've added javascript to get the vertical position of the box placement in the main content, and move the sidebox content to that vertical position (more-or-less).  We'll see if that's good enough for now.  Hopefully the sideboxes won't render on top of each other, although as a similarly yucky short-term measure the main content can be padded with vertical whitespace so break them apart.
If we convert the main slide template to use the other templating language, then there might be more possibilities for intelligent rendering.
Let me know what you think during the run-throughs, and feel free to re-open this issue if needed.


Improve TUI system API (tracking issue)

Need to improve TUI api, make it flexible and powerful as many possible...

[ ] Lisboxes (in progress)

[ ] Text bars

[ ] Input fields

[ ] Dialog boxes

[ ] Basic Layouts (vertical, horizontal)

Some features implemented in adbexplorer project https://github.com/LaineZ/adbexplorer


Krew broken? error: unknown command "ai" for "kubectl"

Installs just fine
kubectl krew install kubectl-ai/kubectl-ai                                                                                 INT ✘  home ⎈

Updated the local copy of plugin index.
Updated the local copy of plugin index "kubectl-ai".
Installing plugin: kubectl-ai
W0524 21:17:37.247983   11130 install.go:160] Skipping plugin "kubectl-ai", it is already installed

Though I get an error when calling it
kubectl ai "Create a minecraft server statefulset using the image from itgz/minecraft"                                                                                 ✔  home ⎈
error: unknown command "ai" for "kubectl"

Did you mean this?
        cp
        wait

But my other plugins such as the rook-ceph one work fine
kubectl rook-ceph ceph status                                                                                 1 ✘  home ⎈
  cluster:
    id:     d191c5d8-41e0-4324-a575-3718963d1a57
    health: HEALTH_WARN
            1 pool(s) have no replicas configured

  services:
    mon: 3 daemons, quorum d,e,f (age 3d)
    mgr: a(active, since 3d)
    mds: 1/1 daemons up, 1 hot standby
    osd: 6 osds: 6 up (since 19h), 6 in (since 3d)
    rgw: 1 daemon active (1 hosts, 1 zones)

  data:
    volumes: 1/1 healthy
    pools:   18 pools, 18 pgs
    objects: 3.28M objects, 9.9 TiB
    usage:   15 TiB used, 8.6 TiB / 24 TiB avail
    pgs:     18 active+clean

  io:
    client:   446 KiB/s rd, 61 KiB/s wr, 457 op/s rd, 1 op/s wr

Upon further inspection
total 0
drwxr-xr-x 2 sam sam 125 May 24 21:13 .
drwxr-xr-x 6 sam sam  59 May 13 22:53 ..
lrwxrwxrwx 1 sam sam  49 May 13 22:54 kubectl-change_ns -> /home/sam/.krew/store/change-ns/v1.0.0/kubectl-ns
lrwxrwxrwx 1 sam sam  38 May 13 22:53 kubectl-krew -> /home/sam/.krew/store/krew/v0.4.3/krew
lrwxrwxrwx 1 sam sam  51 May 24 21:13 kubectl-kubectl_ai -> /home/sam/.krew/store/kubectl-ai/v0.0.10/kubectl-ai
lrwxrwxrwx 1 sam sam  59 May 13 22:57 kubectl-rook_ceph -> /home/sam/.krew/store/rook-ceph/v0.4.0/kubectl-rook-ceph.sh
lrwxrwxrwx 1 sam sam  52 May 13 22:56 kubectl-who_can -> /home/sam/.krew/store/who-can/v0.4.0/kubectl-who-can

Looks like to fix this

The generated binary would need to be renamed to ai instead of kubect-ai
https://github.com/sozercan/kubectl-ai/blob/main/plugins/kubectl-ai.yaml would need to target said binary instead of kubect-ai

I could make a PR later after investigating your build process

@sfxworks thanks for the report! i can repro this behavior.
It seems like krew is trying to be clever by adding a kubectl- in front of the executable and then replacing - with an underscore. Looks like normally this can be solved with specifying files to rename the binary.
https://krew.sigs.k8s.io/docs/developer-guide/plugin-manifest/
However, in this case, we are autogenerating the krew manifest using goreleaser (https://github.com/sozercan/kubectl-ai/blob/main/.goreleaser.yaml)
Unfortunately, I don't see a way in goreleaser to rename the binaries: https://goreleaser.com/customization/krew/
Quickest workaround for this seems to be  mv $HOME/.krew/bin/kubectl-kubectl_ai kubectl-ai

If you're intent on using goreleaser, you may want to raise an issue in their PR requesting this feature. It looks like homebrew offers a post install scrip though that could conduct the rename if needed. A third option could be to switch to some prewritten github actions for homebrew and krew that already offer the features you'd need here. Krew is a popular plugin manager for kubectl, so I'd at least note the current post-install step required in the readme. I just adjusted the symlink myself (Mine was a symlink rather than the binary).


feat: add verification list page and relevant api

feat: #2794
just add query api like record

@dacongda demo video?

@dacongda demo video?

https://github.com/casdoor/casdoor/assets/47297289/59d8073c-788c-4095-a54e-bb9252799b96


Fix : overlapping of icon over the text

Is your feature request related to a problem? Please describe.
The "Become a Prime Sponsor" button has an issue , the icon in that while hovering overlaps the text .

Describe the solution you'd like.
While hovering on the button the text and icon should not be overlapped but should be at a distance when hovered .
Describe alternatives you've considered.
The icon and text should be at a correct distance when hovered is the only alternate solution i have found.
Screenshot / Screenshare

Additional context
i have already worked on it , please assign this to me .
Record

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I want to work on this issue
[X] I am in Hacktoberfest

Code of Conduct

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

@Sa24miksha are you planning to work on it?

@atapas yes i will work on it , i am making a PR soon


add xml:space="preserve" for all whitespaces

Currently only spaces at begin or end of a text add the xml:space attribute.
So at the moment comments with newlines can not be added.

@sebikeller could I ask you to write a test that covers this change?

@sebikeller - I've just tried adding some integration tests to master that add white spaces to values and notes at beginning and endings of multiple lines and I can't reproduce your problem.
If you could add an integration test that fails without your change we can review this PR better.
Thanks,

I agree, with an integration test, its not possible to reproduce the problem.
But if you open the generated file with excel, you'll see the difference.
branch master

[exceljs-master.xlsx](https://github.com/exceljs/exceljs/files/3736814/exceljs-master.xlsx)
applied pr

[exceljs-pr896.xlsx](https://github.com/exceljs/exceljs/files/3736816/exceljs-pr896.xlsx)

Recreating the file in excel, following xml can be extracted:
<?xml version="1.0" encoding="UTF-8" standalone="yes"?>
<comments xmlns="http://schemas.openxmlformats.org/spreadsheetml/2006/main">
  <authors>...</authors>
  <commentList>
    <comment ref="A1" authorId="0" shapeId="0">
      <text>
        <r>
          <rPr>
            <b/>
          </rPr>
          <t>First Line:</t>
        </r>
        <r>
          <t xml:space="preserve">
</t>
        </r>
        <r>
          <t xml:space="preserve">Second Line
</t>
        </r>
        <r>
          <t xml:space="preserve">Third Line</t>
</t>
        </r>
        <r>
          <t>Last Line</t>
        </r>
      </text>
    </comment>
  </commentList>
</comments>

@sebikeller  - agreed! I'll add a unit test on the strings xforms to keep a check on this...


Monitor runit service

Hi ,
I have logstash installed on my machine and its service is managed by runit (/usr/bin/sv start logstash_server),
How do i monitor this , with monit_check resource?
i have tried like this , but didnt work,
monit_check 'logstash' do

	check_type "process"
	start_program "/usr/bin/sv start logstash_server"
	stop_program  "/usr/bin/sv stop logstash_server"
	extra [
    "alert #{node[:monit][:alert_email]} only on {nonexist}"
  ]

end

helps would be appreciated.

You would want to put monit in to passive mode because runit (like most process managers) already handles restarting the service if it crashes.

Im a beginner to monit, can you help me to understand how to put monit into "passive mode"?

No, sorry, this cookbook is not a replacement for the monit manual. If you're not familiar with Monit already, I probably wouldn't use it. Runit on its own (or Upstart or systemd if you want something built in) is much simpler and already handles the most important bit of making sure the service stays running.

Okay, Thanks.


fix String.format lack of arg which is from BroadcastClusterInvoker

What is the purpose of the change
String.format lack of arg which is from BroadcastClusterInvoker
see #8341
Brief changelog
Verifying this change

Checklist

[x] Make sure there is a GitHub_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GitHub issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Check if is necessary to patch to Dubbo 3 if you are work on Dubbo 2.7
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Add some description to dubbo-website project if you are requesting to add a feature.
[ ] GitHub Actions works fine on your own branch.
[ ] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #8348 (8b5923e) into master (41e989b) will decrease coverage by 0.19%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master    #8348      +/-   ##
============================================
- Coverage     61.26%   61.06%   -0.20%
+ Complexity      447      445       -2
============================================
  Files          1096     1096
  Lines         44212    44167      -45
  Branches       6445     6413      -32
============================================
- Hits          27085    26971     -114
- Misses        14146    14211      +65
- Partials       2981     2985       +4

Impacted Files
Coverage Δ

...o/rpc/cluster/support/BroadcastClusterInvoker.java
0.00% <0.00%> (ø)

...ng/transport/dispatcher/all/AllChannelHandler.java
68.96% <0.00%> (-20.69%)
:arrow_down:

...rg/apache/dubbo/remoting/utils/PayloadDropper.java
76.92% <0.00%> (-15.39%)
:arrow_down:

...ng/transport/dispatcher/WrappedChannelHandler.java
47.82% <0.00%> (-15.22%)
:arrow_down:

...dubbo/common/status/support/LoadStatusChecker.java
50.00% <0.00%> (-11.54%)
:arrow_down:

.../remoting/transport/netty4/NettyClientHandler.java
76.27% <0.00%> (-10.17%)
:arrow_down:

.../dubbo/remoting/transport/netty4/NettyChannel.java
54.45% <0.00%> (-9.91%)
:arrow_down:

...mmon/threadpool/support/fixed/FixedThreadPool.java
80.00% <0.00%> (-7.50%)
:arrow_down:

...apache/dubbo/common/config/ConfigurationUtils.java
61.29% <0.00%> (-6.46%)
:arrow_down:

...moting/transport/netty4/NettyEventLoopFactory.java
30.00% <0.00%> (-6.37%)
:arrow_down:

... and 36 more

Continue to review full report at Codecov.

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

LGTM


Updated Firefox image information

Codecov Report

Merging #296 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #296   +/-   ##
=======================================
  Coverage   74.13%   74.13%
=======================================
  Files           9        9
  Lines         978      978
=======================================
  Hits          725      725
  Misses        207      207
  Partials       46       46

Flag
Coverage Δ

#unittests
74.13% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Logout should clear the cached-API

Is '$facebook.logout();' in javascript sufficient to logout a user on your frontend and clear the cache?

I think you have to call refresh() in the promise.then of logout and fill the error section by emptying user data and changing isLoggedIn to false .

$facebook.logout() invoke the facebook-js-sdk logout function, which logging you out from the app and from FB website.
The cachedAPI is caching the requests results on the current session, in order to reduce the requests to facebook.
So, currently the answer is no, but you do have a valid point.
I'll work on that as soon as I can.

Dear AlmogBaku,
I am using ngFacebook in my Angular js project.
I have a problem with  $facebook.logout() , this gives an error "FB.logout() called without an access token".
I When I am using $facebook.login() then  $facebook.logout() is working fine.
But I When I am using $facebook.login() and after login
I refreshed the page $facebook.logout() is not working and rest of the functionality is working fine like.
$facebook.api('/logout?access_token='+accessToken).then(function(response){
// my stuff
}
Thanks!


Terminal tabs in small view cuts off text

Seems like we should set a min-width like in the sidebar:

/duplicate https://github.com/microsoft/vscode/issues/121406


Kura5 streams trigger repeated notifications

When a Kura5 stream is live, it triggers a new notification for that stream every few minutes.

The problem is that because this game is in two queues (Twitch and TwitchUser) for the same platform, the stream decays faster than it can be refreshed by API calls (because the TwitchUser queue is small, the game is refreshed more frequently in that queue than in the normal Twitch queue).
As a workaround I've made the platform check for stream decay a bit more strict (previously is was a substring comparison). This relies on the fact that a TwitchUser game is also in a Twitch queue, otherwise it will never decay. This was not always the case, but it is the case now, so for now this should be good enough.
Since Live Now relies on the Legacy Twitch API v5 which will be shutdown at the end of February 2022, I don't really think it's worth looking more into a long-term solution since Live Now would probably need to be rebuilt to take advantage of the new API anyway.


LoRaWAN - Provide method to register a battery level callback

Is your enhancement proposal related to a problem? Please describe.
The LoRaWAN standard allows the battery state of charge (0-100% represented by 1-255) to be sent in the MAC section of the transmission. The library used by Zephyr supports this, however the sub-system setup within Zephyr has the callback set to NULL (see $(ZEPHYR_BASE)/subsys/lorawan/lorawan.c line 480
macCallbacks.GetBatteryLevel = NULL;
Ideally there should be a way of setting this and allowing it's use in applications.
Describe the solution you'd like
I would tend to have a _weak dummy callback in $(ZEPHYR_BASE)/subsys/lorawan/lorawan.c and allow the application writer to implement a proper version if they desire.
There may want to be a CONFIG to enable this as it may use extra bytes in the LoRaWAN payload.
Describe alternatives you've considered
Alternatively, a register callback function could be used.
Additional context
For the pull request writer, you may find it easier to test with chripstack rather than TTN. I've found that when in an application on ChirpStack, it clearly shows the the battery status for each device connected.

This is already possible using the lorawan_set_battery_level_callback api.

I believe this can be closed as it's been implemented /Cc @JordanYates


improvement(mediaplayer): fix demuxer set SourceConfig logic

Signed-off-by: pingkai<EMAIL_ADDRESS>

Codecov Report

Merging #266 into develop will increase coverage by 0.32%.
The diff coverage is 25.00%.

@@             Coverage Diff             @@
##           develop     #266      +/-   ##
===========================================
+ Coverage    68.35%   68.68%   +0.32%
===========================================
  Files          207      207
  Lines        11259    11259
===========================================
+ Hits          7696     7733      +37
+ Misses        3563     3526      -37

Impacted Files
Coverage Δ

mediaPlayer/SuperMediaPlayer.cpp
76.06% <25.00%> (+0.31%)
:arrow_up:

framework/codec/ActiveDecoder.cpp
86.43% <0.00%> (-0.51%)
:arrow_down:

framework/render/audio/filterAudioRender.cpp
87.89% <0.00%> (+0.63%)
:arrow_up:

mediaPlayer/buffer_controller.cpp
57.62% <0.00%> (+0.84%)
:arrow_up:

framework/data_source/curl/curl_data_source.cpp
86.61% <0.00%> (+1.67%)
:arrow_up:

framework/demuxer/AVBSF.cpp
68.13% <0.00%> (+2.19%)
:arrow_up:

framework/demuxer/avFormatDemuxer.cpp
80.42% <0.00%> (+3.36%)
:arrow_up:

mediaPlayer/media_packet_queue.cpp
84.00% <0.00%> (+4.00%)
:arrow_up:

mediaPlayer/player_msg_control.cpp
83.33% <0.00%> (+5.55%)
:arrow_up:

Continue to review full report at Codecov.

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


Build with gym is not support localization in application name

Issue Description
When building project with gym, my application name don't support localization in app name.
(It works normally when I build application with Xcode)
Here is my project structure :
AppName
|-Data
|-Classes
|-Frameworks
|-Libraries
|-en.lproj
|-ja.lproj
I want to try adding this build flag in gym command but I don't know how to do this..
xcodebuild -exportLocalizations
Could you please suggest me how to add the support for localization name?
Thank you in advance!!

you can try xcargs: "-exportLocalizations"  and/or export_xcargs: "-exportLocalizations" (used when exporting the archive)

Thank you for your comment @hjanuschka !! I will try this and report it again :)

I still cannot built my app to application with support localization name.
But the above solution is solved my question.
Thank you so much


Allow Unset values to avoid being send during serialization

Fixes https://github.com/triaxtec/openapi-python-client/issues/205.
Introduces a new sentinel value, UNSET which becomes the new default for nullable fields which aren't required. If a field is set to this value, it will not be sent as part of the payload during serialization.
Since we still want to treat endpoint arguments and query params as they were originally without Unset, this logic has moved to to_query_string on Property.
Given a str type, a nullable property which isn't required might look something like:
my_property: Union[str, None, Unset] = UNSET

FYI @benchling/ce-team and @rswang since Daniel is out

FYI @mzhu22

One of the major drawbacks to this approach is that it applies to both request and response bodies. When really we only want it to apply to request bodies. I don't think there's a way to really control this via the spec without misusing it.
Do we think it would be better to leave the original typings as is and only change the defaults to Unset and ignore typing violations?

Do we think it would be better to leave the original typings as is and only change the defaults to Unset and ignore typing violations?

@bowenwr good point. Maybe it would be better to leave UNSET off the type. You could then lie to mypy with just cast(<Target Type>, UNSET) and mypy should behave as expected otherwise.

Do we think it would be better to leave the original typings as is and only change the defaults to Unset and ignore typing violations?

@bowenwr good point. Maybe it would be better to leave Unset off the type. You could then lie to mypy with just cast(<Target Type>, UNSET) and mypy should behave as expected otherwise.
The response objects still have None for those values, overriding the default UNSET val, right?

I liked this suggestion, I implemented it and tested it out. It seems to pass our integration tests. I still needed to_query_method_arg to preserve the original defaults but otherwise I like this change as a lot less invasive.

👏 yay! Are you going to also try to submit this PR upstream? Seems like a useful addition for others as well.

I'd like to upstream this at some point though it would probably need considerably more polish and E2E tests. I'll probably incubate it a bit internally too to make sure I haven't missed any cases as we expand our spec coverage.


Adds owning-a-home/dropdown-utils test

Short description explaining the high-level reason for the pull request
Additions

Add test for owning-a-home/js/dropdown-utils

Testing

gulp test:unit --specs=apps/owning-a-home/js/dropdown-utils-spec.js should pass and show 100% coverage for dropdown-utils.

looks good, not sure what the deal is on the failure.  Merge after that is resolved.

looks good, not sure what the deal is on the failure. Merge after that is resolved.

I see why, did a tweak and forgot something. Should be good now.


Run tectonic on vagrant

Hi,
We would like to try tectonic on the local laptop, but it seems that it is not possible to install tectonic by vagrant, any quick way to install tectonic on my laptop?

Agreed, this is something that would make sense for Tectonic to ship/maintain.

Try Tectonic Sandbox(https://coreos.com/tectonic/sandbox)!

it always print:
==> c1: Tectonic is still starting, sleeping 30 seconds
==> c1: Tectonic is still starting, sleeping 30 seconds
i can't know the process, only waiting. in china mainland, the network is not stable, i don't know  when it can done.

same with @xiaods . I have used vpn.

Agreed on @xiaods , I need more information to make sure everything goes well.


PR#174 – Error: YAML Exception reading /home/runner/work/blog/blog/_posts/2024-10-20-2025PeacefulTransferOfPower.md: (): did not find expected key while parsing a block mapping at line 2 column 1

          Error: YAML Exception reading /home/runner/work/blog/blog/_posts/2024-10-20-2025PeacefulTransferOfPower.md: (<unknown>): did not find expected key while parsing a block mapping at line 2 column 1
         Error: YAML Exception reading /home/runner/work/blog/blog/_posts/2024-10-20-AnswerToTrumpersSupportingHarris.md: (<unknown>): found unexpected end of stream while scanning a quoted scalar at line 10 column 8

Originally posted by @RalphHightower in https://github.com/RalphHightower/blog/issues/174#issuecomment-2425234435

Duplicate


ruby 3

@rultor release, tag is 0.0.40

@rultor release, tag is 0.0.40

@yegor256 OK, I will release it now. Please check the progress here

@rultor release, tag is 0.0.40

@yegor256 Done! FYI, the full log is here (took me 2min)


Porting NimBLE for CheriOS

I did a CheriOS  port of NimBLE host controller.
I can successfully run the bleprph demo.
CheriOS runs on a CHERI-MIPS CPU, which has some additional programming constraints compared to your usual ARM/AMD46 CPU. I thus had to modify some non-npl files to make the port work.
I will open a few pull requests to see what can be merged upstream.

would be great if you could add this port to mynewt-nimble repository - we can then run builds automatically on travis (same way as we do for e.g. RIOT OS) to make sure we do not break it in future.

The port-specific code is currently more a PoC than some nice code. I will need to clean it first.


Reload cellect vs retire cellect

notify the cellect client of retire events and don't force cellect to reload the whole workflow.

@marten be good to get this out asap to stop the load on cellect / db

I was trying to figure out why Cellect was running under a heavy CPU load.  This would certainly explain it.
LGTM other than the missing require 'subjects/cellect_client' in the worker.

@camallen Since this needs to go, I'll just merge and deal with the autoload errors.

Sweet thanks.


Z-Targeting: Screen blinking on first load

The Z-targeting branch do not load correctly and screen keeps on blinking for the first time. However, if we again refresh the gitub repository and load again, it loads correctly on second attempt.
https://user-images.githubusercontent.com/104460803/175121762-f56b2202-6a11-4d08-b3c8-5c6534c2a8b3.mp4

I wasn't able ti reproduce it.. Can you post the error you got?

On first load, these are the errors. After reloading it for the second time, it starts working

https://user-images.githubusercontent.com/104460803/175537421-4773103d-522a-46ff-adaa-85f607b5bb29.mp4

This seems to be a local development issue, I haven't encountered it. I think if you clear your vite stuff it will probably work ok.


Docs: multicast deprecation

Description:
This adds deprecated tags to the primary signatures of deprecated multicast, publish & publishReplay, so they are properly marked as such in the API reference.

Thanks.


Repository bulk save

I've only rebased @vhyza's stuff

I rebased it today in original branch - https://github.com/elastic/elasticsearch-rails/pull/527.
But CI failed with The command "SERVER=launch TEST_CLUSTER_COMMAND=/usr/share/elasticsearch/bin/elasticsearch TEST_CLUSTER_PARAMS='-Des.default.path.conf=/etc/elasticsearch/ -Des.default.path.logs==/var/log/elasticsearch/' bundle exec rake test:all" exited with 1. as well as the builds few days ago...

I saw something about 5 hours ago, thought it was a bug in githubs inteface or something


Can't import ops._ from both a typeclass and the typeclass it extends.

We have two typeclasses, one extending the other, and in some files we'd like to use the extension methods of both typeclasses, but the methods provided by each ops overlap, so implicit resolution fails if both are imported.
Note: we need to import both, because some values have instances for both typeclasses, and others just for the first.
We were able to work around it by defining ownOps in the derived typeclass's companion, and importing its members instead:
@typeclass trait TraverseT[T[...]] extends FunctorT[T] { ... }

object TraverseT {
  ...
  object ownOps extends ToTraverseTOps
}

See https://github.com/mossprescott/quasar/commit/1059fd25f938680d7f91a0705c96f274193255ff for that workaround in context.
Perhaps such a thing should be generated?

I like this idea. How do you feel about object nonInheritedOps ...?
Another workaround is to create an object that mixes in all ToXOps for every type class X. E.g.:
object syntax extends ToFunctorTOps with ToTraverseTOps

If backward compatibility wasn't desirable, I'd vote for allOps and simply ops, but given that it probably is, nonInheritedOps seems not bad.
The real issue was how unclear it was to me what the problem was (fortunately, @wemrysi and @sellout were around to help). But I don't know if there's anything that can be really done about that, implicit resolution being what it is.
Also, I might try your syntax alternative, thanks!

:tada: Thanks!


train data

I found in train.txt data, the input contains {source} + {target} while output {target} + {source}. My understanding correct? If so, what is the advantage of using bilingual data for both encoder and decoder?

Yes, its correct.
My personal preference :)
Main idea is to have better embeddings (+ some articles share this idea)

thanks. although it was closed, could you provide some articles?


Add the progress bar and response validation.

The project has moved to https://github.com/rakyll/hey. Closing the PRs on this legacy repo.


PWA-281 website connector

Pull Request Test Coverage Report for Build 1197

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 47.085%

Totals

Change from base Build 1196:
0.0%

Covered Lines:
1760

Relevant Lines:
3646

💛  - Coveralls


sourceCompatibility is read before configuration is complete

Versions 1.0.0-beta.16 through 2.0.0 incorrectly use the version of the Java from the JRE running Gradle when specifying the Java source level to use with Sonarlint.
For example, running ./gradlew build on this project with a Java 21 JRE enables the rules [java:S6204, java:S6126] and causes the build to fail.
Running on one of my team's projects with ./gradlew sonarlintMain --debug I observed that the sonar.java.source property was being set to 21 when I run it with Java 21, despite having a configuration like the following in build.gradle:
java {
   sourceCompatibility = JavaVersion.VERSION_17
   targetCompatibility = JavaVersion.VERSION_17
}

Looking at the history, it appears that this was broken by c875ae86126af9798e1b7f861f7c6290dd8c1aed. Moving the read of sourceCompatibility from the SonarlintAction constructor back to the analyze function fixes it: I tested publishing locally and running against my project and see that the correct version is set in sonar.java.source.
I didn't dig much deeper, but presumably the issue here is that reading sourceCompatiblity in the constructor ends up reading a default value configured by the java plugin before the project's configuration has been applied.

Thanks a lot for the finding!


Feature Request: Accessibility

To enhance the Toolkit's accessibility features comprehensively, consider integrating functionalities for keyboard navigation, accommodating color blindness, enabling voiceover capabilities, and ensuring seamless interaction with testing frameworks. By seamlessly integrating these features, users can effortlessly navigate controls and parameters, promoting inclusivity and facilitating ease of use across diverse user needs and testing scenarios.
To truly optimize accessibility within the Toolkit, it's crucial to delve into the intricacies of each feature and consider their impact on users with varying needs and preferences. Here's an expanded version:

Keyboard Navigation: Implementing robust keyboard navigation ensures that users can efficiently navigate through the Toolkit without relying solely on mouse input. This includes providing keyboard shortcuts for common actions, ensuring logical tab order for interactive elements, and supporting keyboard focus indicators for clarity and adhoc data input. Also combinations of mouse and keyboard are very nice, eg. select multiple parameters and change them relatively with mouse or keyboard.

Color Blindness Accommodation: Designing the interface with color blindness in mind involves using color schemes that maintain sufficient contrast and incorporating alternative visual cues or patterns to convey information. By adhering to accessibility guidelines such as WCAG, the Toolkit can ensure that users with color vision deficiencies can effectively interpret and interact with the interface.

Voiceover Integration: Integrating voiceover functionality enables users with visual impairments to navigate and interact with the Toolkit using screen readers. This involves implementing proper semantic markup, providing descriptive labels for interactive elements, and ensuring that all interface elements are accessible via keyboard navigation.

Testing Framework Accessibility: Enhancing accessibility extends beyond user-facing features to encompass the accessibility of testing frameworks themselves. This involves ensuring that testing tools and scripts are compatible with assistive technologies, allowing users to create and execute tests effectively regardless of their abilities.

Inclusive Design Principles: Adopting inclusive design principles throughout the development process ensures that the Toolkit caters to a diverse range of users from the outset. This includes conducting usability testing with individuals representing different demographics and abilities, gathering feedback to iteratively improve accessibility features, and fostering a culture of inclusivity within the development team.

By prioritizing accessibility at every stage of development and embracing a holistic approach that addresses diverse user needs, the Toolkit can become a truly inclusive and empowering tool for all users, regardless of their abilities or preferences.

if i'm not mistaken, accessibility is a deep rabbithole to jump into..
not sure if this is the right time to look into that..
but absolutely worth keeping in mind during development..


Fix import changes

This reverts #591 which introduced a bunch of regressions due to changing the import lookup.
Added a less intrusive fix for Bug314 that uses protection for the import aliases. It has to remain disabled though until we fix protection checks (#669).

I think this should definitely go into the beta but we shouldn't tackle protection yet.

There is an issue with this change for explictly protected selective imports.
module a;
// explicit private AND selective import
private import std.stdio : writeln;

module b;
// reexported through another module
public import a;

module c;
import b;
// with a valid access
import std.stdio;
void bar()
{
    writeln();
}

To get back to the pre-regression state we'll have to disable alias protection
for now as well, or make a switch to the fixed protection checks #665 now.

This requires D-Programming-Language/druntime#148 to be pulled as well.

Trying to fix the issue by setting protection on the import and the aliases it generates was also my first attempt at fixing the problem. It works.
There are two issues. The main one is trouble with overload sets. As far as I remember one of the issues is that if the head of a chain is public and the import inserts another element, you need to adjust overload resolution to account for protection. In fact, that was the main complication in my original patch.
And secondly it's not strictly required that 'private alias std.stdio foo;' should have the same semantics as 'private import std.stdio : foo;'. For the alias it is protection, for the import it's visibility.
Also check out the test set here: http://www.incasoftware.de/~kamm/importtests.tar.bz2
Could someone point me to the regressions #591 introduced?

There are two issues. The main one is trouble with overload sets. As far as I remember one of the issues is that if the head of a chain is public and the import inserts another element, you need to adjust overload resolution to account for protection. In fact, that was the main complication in my original patch.

Yeah, I've come to the same conclusion. We need fix both private symbols and imports at the same
time to mitigate the impact that each of them would have as a separate fix. That way we'd tighten import
checks but teach the compiler how to resolve almost all of the arising collisions. It's likely not going to affect
overload resolution but only the considered overload sets.

And secondly it's not strictly required that 'private alias std.stdio foo;' should have the same semantics as 'private import std.stdio : foo;'. For the alias it is protection, for the import it's visibility.

That's where things went wrong. While they must not be identical changing this has broken a valid existing code.

Selective imports used to overload against module symbols and were given preference over symbols from other imports. At the same time they were checked for name collisions with other module symbols.
Normal imports added the package name to the module, which prevented hijacking of the package name.

Test cases:
D-Programming-Language/dmd@a79763d
D-Programming-Language/dmd@f330b95

Okay, I'm glad you're taking over managing this fix as I don't really have the time at the moment.
One more comment though:

Selective imports used to overload against module symbols

That change was intentional. One goal with the fix was to ensure that going from a full import to a selective one and back never affects behavior like overload resolution. The idea is that you should not be able to break a program by being more strict with your imports. That way you can also keep the nice overload set rules even when using selective imports. You could still use an explicit alias to merge the overloads into the module's scope.


Voting needed

Added a section on voting community leaders, to enable consensus based rather than hierarchical decision making.

It's only getting better.

This project is not under Open-source Governance. Therefore community leaders are selected by SAP and Deutsche Telekom.


Fixed bug with empty contentLoadedCallback. Added possibility to use custom behavior for infinite-scroll. Added composer.lock.

Fixed bug with empty contentLoadedCallback.
If property contentLoadedCallback is not set then javascript error happens.
Related with https://github.com/nirvana-msu/yii2-infinite-scroll/pull/4, but IMHO my fix is better.
Added possibility to use custom behavior for infinite-scroll.
Sometimes it is needed to use custom behavior which is not included in infinite-scroll.js.
Added composer.lock.

Issue with missing contentLoadedCallback default value has been fixed. Thanks for bringing attention to it.
composer.lock isn't really necessary in the repository.

If you amend this PR with just the change allowing to add custom behavior, or submit a new PR instead, I'll merge.
Thanks! Sorry it took so long to respond.

I'll submit new PR.


"Error: Cannot find module 'knex.js'" from running test suite

Environment
Knex version: master (b59161f)
Database + version: N/A
OS: Ubuntu 20.04 with Node 12.18.2 via NodeSource
Bug
I'm trying to clone the Knex repository and run the test suite and am getting errors within the test suite.
For example:
mkdir -p ~/tmp
cd ~/tmp
git clone git@github.com:knex/knex.git
cd knex
npm i

If I then manually run one of the tests that I'm interested in:
node_modules/.bin/mocha --exit -t 10000 test/all-tests-suite.js --grep 'ignores migrationSource if migration directory is specified'

I get the following error:
 0 passing (284ms)
  1 failing

  1) CLI tests
       knexfile
         knexfile supports custom migrationSource
           ignores migrationSource if migration directory is specified:
     node /home/josh/tmp/knex/bin/cli.js migrate:latest --knexfile=test/jake-util/knexfile-custom-migration-source/knexfile-with-directory.js --knexpath=../knex.js -> FAIL.
Stdout: Working directory changed to ~/tmp/knex/test/jake-util/knexfile-custom-migration-source

Error: Error: Cannot find module 'knex.js'
Require stack:
- /home/josh/tmp/knex/bin/cli.js
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:966:15)
    at Function.Module._load (internal/modules/cjs/loader.js:842:27)
    at Module.require (internal/modules/cjs/loader.js:1026:19)
    at require (internal/modules/cjs/helpers.js:72:18)
    at initKnex (/home/josh/tmp/knex/bin/cli.js:57:16)
    at processTicksAndRejections (internal/process/task_queues.js:97:5)
    at async Command.<anonymous> (/home/josh/tmp/knex/bin/cli.js:168:26)

(I'm not sure if running individual tests like this is intended to be supported. I got similar errors from running the full test suite - but I had other problems mixed in with the larger test suite, so I was trying to focus on this particular error.)
If I edit bin/cli.js to get the full path to knex.js, it works:
function invoke(env) {
  env.modulePath = path.resolve(env.modulePath || env.knexpath || process.env.KNEX_PATH);

However, I don't understand Knex's cwd and module design well enough to know if this fix is correct or not. I'm happy to open a PR if it would help.
Thank you.

Running individual tests is mostly not supported (many of the tests are dependent on each other).

Sorry for the delayed reply.
I was probably unclear in my original issue; I got numerous errors from trying to run the whole test suite, and so I started attempting to run individual failing tests, which is where I ran into this problem.
With that said - I don't currently have a need to do development on Knex (as evidenced by how long it took me to reply to this ☹ ), so feel free to close this issue if it's not worth further attention right now.


CEL Validation: HTTPRoute Case-Insensitive Header Name Uniqueness

What happened:
Unfortunately CEL is not currently capable of validating the case-insensitive uniqueness of header names in header modifier filters. This is the only standard channel validation we are still reliant on the validating webhook for. @gauravkghildiyal already developed a CEL rule for this:
// +kubebuilder:validation:XValidation:message="Must not match the same header (case-insensitive) multiple times in the same rule",rule="self.all(h1, self.exists_one(h2, h1.name.lowerAscii() == h2.name.lowerAscii()))"

Unfortunately there is a problem with CEL cost estimation that prevents this https://github.com/kubernetes/kubernetes/issues/119749, @jpbetz is working on a fix for this.
Anything else we need to know?:
Given the Gateway API versioning policy to support the 5 latest minor Kubernetes versions, even when this is fixed upstream, it's going to be a long time before we can adopt this.
/lifecycle frozen

/cc


Can't use some weapons abilities

So i tried using the Prominence 1 modpack and found out i can't use some weapons abilities (with right click) because it instead tries casting a spell from my spellbook
How can i fix it?

Nevermind i solved it myself, they weren't all added th the spell engine blacklist
Probably something that concerns the makers of the pack, closing the issue


Switch xhtmlrequest open to be asynchronous

Synchronous requests have been depreciated in Mozilla for a long time,
unsure about other browsers.
The code was already written to support async requests, no visible
downside to going async and it avoids blocking and warnings.

Hi lod,
thanks for your message and the pull request. I thought about making the request async when I published the current version of the plugin months ago. But I recognized it did not work well in my scenario. I wanted to provide an example project anyways, so I pushed it right 5 minutes ago. In this example folder you will find another named "includes". This is where the presentation structure is built. When leaving the async-mode set to false, I surely get the console warning you described, but when I set async to true (like you suggest in your pull request), the first of each include-level is not displayed. Can you reproduce my error? Can you imagine why async is problematic here?
Jan

Thanks for looking at it. My usage was a bit more basic than your example.
The issue was that the xhr and url objects were overwritten by subsequent calls. Restructuring the code to have a second function provide a unique scope for each section seemed the easiest fix.
There were a few other issues that emerged once I had that fixed but I think I got them all squashed.

Thank you for solving the async problem and sorry for my late reply. Now everything works like a charm.


Sample feedback: display-anchor-ad/js

If not pre-filled, update the title of this issue with the name of the audit this feedback pertains to.
Provide details of any issues with the documentation or improvements you'd like to see below.

Hi @rajaneshamit, what is your feedback for this sample?


Instruct travis to test newer rubies

After starting a discussion in #158 about ruby versions I figured I should have a proper PR to continue that discussion.  The only thing I see here is that we lose jruby-19mode in favor of jruby-head.  I'm not a jruby user myself so I would be interested in hearing some opinions.

FYI, I'm joining @ejholmes on maintaining the gem now.
Ruby 1.8.7 has been end-of-lifed now and being stuck on it prevents us from taking advantage of nice things like hash syntax. Few use 1.8.7 now. I'll plan to release a new major version soon with >1.9.3 support only.

:+1:

Merged in favour of https://github.com/ejholmes/restforce/pull/189.


@remix-run/cloudflare does not export installGlobals

What version of Remix are you using?
1.4.3
Steps to Reproduce
Create a Remix app by choosing "Cloudflare Pages" and import installGlobals from @remix-run/cloudflare in the test setup file, as suggested by the documentation:
// test/setup-test-env.ts
import { installGlobals } from '@remix-run/cloudflare'

installGlobals()

Expected Behavior
Same as with @remix-run/node, I would expect this to install polyfills.
Actual Behavior
This fails because @remix-run/cloudflare doesn't export installGlobals. I thought about reproducing this as a bug-report-test.ts PR instead, but it isn't appropriate because installGlobals isn't for the app itself, just for stuff like tests. You can easily verify the bug by just looking at the code.
My current workaround is to simply use @remix-run/node instead.

2022년 5월 18일 (수) 20:50, Matija Marohnić @.***>님이 작성:

What version of Remix are you using?
1.4.3
Steps to Reproduce
Create a Remix app by choosing "Cloudflare Pages" and import
installGlobals from @remix-run/cloudflare in the test setup file, as
suggested by the documentation
https://remix.run/docs/en/v1/other-api/node#polyfills:
// test/setup-test-env.tsimport { installGlobals } from @.***/cloudflare'
installGlobals()
Expected Behavior
Same as with @remix-run/node, I would expect this to install polyfills.
Actual Behavior
This fails because @remix-run/cloudflare doesn't export installGlobals. I
thought about reproducing this as a bug-report-test.ts PR instead, but it
isn't appropriate because installGlobals isn't for the app itself, just
for stuff like tests. You can easily verify the bug by just looking at the
code.
My current workaround is to simply use @remix-run/node instead.
—
Reply to this email directly, view it on GitHub
https://github.com/remix-run/remix/issues/3222, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXPCZB7RXHVZO27WOOJO2OTVKTKPFANCNFSM5WIGG2QA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

@kentcdodds This to me seems more like a documentation problem? 🤔

We need more thought on this actually because it's weird that a cloudflare project would use the node adapter at all. Currently this is just a workaround. I think we need to discuss this further as a team.

@silvenon We only have polyfills for Node as these APIs are natively available in Cloudflare/Deno
Updated the docs in #7228 to avoid confusion


Added parquet limit tests

Abhishek,
Based on padma's request, I added some limit tests on top of parquet. In-order to make sure that there is no randomness in the list of records returned, I only used a single parquet file which is smaller than an MFS block. I had a clean functional run (with known failures). Can you review the PR?
Rahul

Looks good. +1


NPE in DefaultRateLimitKeyGenerator

In the case that someone tries to access a url without a route, NPE is thrown.
This is because it tries to get the id and the path from the route
Caused by: java.lang.NullPointerException: null
	at com.marcosbarbero.cloud.autoconfigure.zuul.ratelimit.config.DefaultRateLimitKeyGenerator.key(DefaultRateLimitKeyGenerator.java:46)
	at com.marcosbarbero.cloud.autoconfigure.zuul.ratelimit.filters.RateLimitPreFilter.lambda$run$0(RateLimitPreFilter.java:76)
	at java.util.Optional.ifPresent(Optional.java:159)
	at com.marcosbarbero.cloud.autoconfigure.zuul.ratelimit.filters.RateLimitPreFilter.run(RateLimitPreFilter.java:75)
	at com.netflix.zuul.ZuulFilter.runFilter(ZuulFilter.java:112)
	at com.netflix.zuul.FilterProcessor.processZuulFilter(FilterProcessor.java:193)

@marcosbarbero can you release a version with this fix please?

@lchayoun will do it today

@lchayoun already available on maven central v1.3.4.RELEASE

@marcosbarbero spring-cloud-zuul-ratelimit-dependencies is not published
https://search.maven.org/#search|gav|1|g%3A"com.marcosbarbero.cloud" AND a%3A"spring-cloud-zuul-ratelimit-dependencies"

@lchayoun weird, I checked it while uploading.
I'm doing it again now, let's see. Did you notice any other missing dependency?

@lchayoun it seems to be available now

@marcosbarbero working, thanks!


Replaced ':' char per '|' in Enumeration Element

Fixes this issue https://github.com/OrchardCMS/Orchard/issues/6823

Please react to the comment I made on the issue.

See issue for discussion about implementing the fix.


(HTTP code 404) unexpected - {"message":"manifest for nanori/windows-java:latest not found"}

Expected behavior
Actual behavior
Information about the Issue
Steps to reproduce the behavior

...
...

temp


285 rewards and naming pools update

Removed the  automatic adding  new pools to the Rewards SCORE distribution list on Governance Score.
And Created a separate method to add data sources to Rewards.

Closing  the pull request .


1.8.7 Path Issue?

On a fresh install of gli using Ruby 1.8.7 & REE (RVM install) i'm getting the following error from GLI.
ruby-1.8.7-p352@test  ~/documents/projects/test
→ gli init my_proj command_name
Creating dir ./my_proj/lib...
Creating dir ./my_proj/bin...
Creating dir ./my_proj/test...
Created ./my_proj/bin/my_proj
Created ./my_proj/README.rdoc
Created ./my_proj/my_proj.rdoc
Created ./my_proj/my_proj.gemspec
Created ./my_proj/test/tc_nothing.rb
Created ./my_proj/Rakefile
Created ./my_proj/Gemfile
Created ./my_proj/lib/my_proj_version.rb
ruby-1.8.7-p352@test  ~/documents/projects/test

→ my_proj/bin/my_proj help
/Users/schneems/.rvm/rubies/ruby-1.8.7-p352/lib/ruby/site_ruby/1.8/rubygems/custom_require.rb:36:in `gem_original_require': no such file to load -- my_proj_version (LoadError)
	from /Users/schneems/.rvm/rubies/ruby-1.8.7-p352/lib/ruby/site_ruby/1.8/rubygems/custom_require.rb:36:in `require'
	from my_proj/bin/my_proj:4

Any thoughts? Congrats on the book btw.

Yeah, this is a dupe of #53.  Check out the info there for an explanation.  I'll sort something out tonight, either reverting the change to GLI that introduced this issue, or updating the book.,
Thanks for the report!


zio dependency upgrade to zio2.0.0

It will block until  both zio-grpc zio-raft can upgrade  to zio2.0.0