Create a library for table object like in Lua

At the moment there are some problems with the event model because it hits an internal compiler error https://github.com/PistonDevelopers/event/issues/311 so we had to use the backup strategy and make GenericEvent inherit all the other events to keep stuff from breaking.
We need to come up with something new in order to support custom events. The problem is: How to communicate something from A to B without depending on the medium to restrict what can be communicated. When foo calls bar which calls baz then bar should not know the generic constraints.
Lua uses a table object implemented as a hash table which can use any value, including itself as a key. This means you can build any data structure out of this one object, including multi-dimensional ones. If we had an enum Value which could contain String, bool, usize, u64 and f64 and Table where Table maps from Value to Value, then you can construct anything you like and write a trait that maps to the native type.
The purpose is to support custom events that are not part of formal input structure, but it can also be used to get dynamical typed object functionality in Piston.
If we use Table(Arc<Table>) in Input then we can avoid memory allocations and reuse the same objects in the window back-end.
This requires wrapping f64 and make it implement Eq. If you take NaN and use it as a key, then you can not read it back because NaN != NaN. One can have a method Value::is_nan which you check manually.

If we used it for update and render, we would not need to wrap Input in Event<Input>. We could drop it entirely, rename Input to Event and drop GenericEvent. Then we could implement the traits directly for Event and use a concrete type for the event model.
Instead of:
fn foo<E: GenericEvent>(&mut self, e: &E) { ... }

we get
fn foo(&mut self, e: &Event) { ... }

The "input" repo could contain only mouse and keyboard while what is Input now gets moved to the "event" repo.

Alternative is to use Arc<Box<Any>>.

The problem with Arc<Box<Any>> is that you can't call make_unique, so you can't reuse the object.

Created new repo https://github.com/PistonDevelopers/table

Using Any instead.


feature: Replace PNG icons with vectorized image

This PR replaces the sharp pngs with smooth svgs. The clock svg has been reused from my metrics-plugin PR.
Before:

After:

[X] Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
[X] Ensure that the pull request title represents the desired changelog entry
[X] Please describe what you did
[X] Ensure you have provided tests - that demonstrates feature works or fixes the issue

Released in 2.85.


Error: Microsoft.AspNetCore.Razor.OmniSharpPlugin fails to load

Issue Description
One day, the following error came to occur at startup. What is the cause?
System.TypeLoadException: VTable setup of type Microsoft.AspNetCore.Razor.OmniSharpPlugin.OmniSharpPluginStrongNamedRoslynServiceProvider failed
  at (wrapper managed-to-native) System.RuntimeType.GetPropertiesByName_native(System.RuntimeType,intptr,System.Reflection.BindingFlags,bool)
  at System.RuntimeType.GetPropertiesByName (System.String name, System.Reflection.BindingFlags bindingAttr, System.Boolean icase, System.RuntimeType reflectedType) [0x0001b] in <98fac219bd4e453693d76fda7bd96ab0>:0
  at System.RuntimeType.GetPropertyCandidates (System.String name, System.Reflection.BindingFlags bindingAttr, System.Type[] types, System.Boolean allowPrefixLookup) [0x00010] in <98fac219bd4e453693d76fda7bd96ab0>:0
  at System.RuntimeType.GetProperties (System.Reflection.BindingFlags bindingAttr) [0x00000] in <98fac219bd4e453693d76fda7bd96ab0>:0
  at System.Reflection.RuntimeReflectionExtensions.GetRuntimeProperties (System.Type type) [0x00006] in <98fac219bd4e453693d76fda7bd96ab0>:0
  at System.Composition.TypedParts.Discovery.TypeInspector+<DiscoverPropertyExports>d__7.MoveNext () [0x00030] in <9aac21a4a3f24063b2b8b59143acddfc>:0
  at System.Composition.TypedParts.Discovery.TypeInspector+<DiscoverExports>d__5.MoveNext () [0x000dc] in <9aac21a4a3f24063b2b8b59143acddfc>:0
  at System.Composition.TypedParts.Discovery.TypeInspector.InspectTypeForPart (System.Reflection.TypeInfo type, System.Composition.TypedParts.Discovery.DiscoveredPart& part) [0x00061] in <9aac21a4a3f24063b2b8b59143acddfc>:0
  at System.Composition.TypedParts.TypedPartExportDescriptorProvider..ctor (System.Collections.Generic.IEnumerable`1[T] types, System.Composition.Convention.AttributedModelProvider attributeContext) [0x00037] in <9aac21a4a3f24063b2b8b59143acddfc>:0
  at System.Composition.Hosting.ContainerConfiguration.CreateContainer () [0x00042] in <9aac21a4a3f24063b2b8b59143acddfc>:0
  at OmniSharp.CompositionHostBuilder.Build () [0x001ed] in <c228582629e34359bfdbc0fc855c1d2f>:0
  at OmniSharp.Stdio.Host..ctor (System.IO.TextReader input, OmniSharp.Stdio.Services.ISharedTextWriter writer, OmniSharp.IOmniSharpEnvironment environment, Microsoft.Extensions.Configuration.IConfiguration configuration, System.IServiceProvider serviceProvider, OmniSharp.CompositionHostBuilder compositionHostBuilder, Microsoft.Extensions.Logging.ILoggerFactory loggerFactory, System.Threading.CancellationTokenSource cancellationTokenSource) [0x0007e] in <19355510af164339b7bccd4ca09bc44c>:0
  at OmniSharp.Stdio.Driver.Program+<>c__DisplayClass0_1.<Main>b__1 () [0x00117] in <ba7fbbf8e8fb4043b79656a74082347f>:0
  at OmniSharp.CommandLineApplication+<>c__DisplayClass11_0.<OnExecute>b__0 () [0x0000b] in <c228582629e34359bfdbc0fc855c1d2f>:0
  at McMaster.Extensions.CommandLineUtils.CommandLineApplication.Execute (System.String[] args) [0x00039] in <344a1e8cd6594b17b1e01f8df6ae8827>:0
  at OmniSharp.CommandLineApplication.Execute (System.Collections.Generic.IEnumerable`1[T] args) [0x00042] in <c228582629e34359bfdbc0fc855c1d2f>:0
  at OmniSharp.Stdio.Driver.Program+<>c__DisplayClass0_0.<Main>b__0 () [0x00028] in <ba7fbbf8e8fb4043b79656a74082347f>:0
  at OmniSharp.HostHelpers.Start (System.Func`1[TResult] action) [0x0001c] in <c228582629e34359bfdbc0fc855c1d2f>:0
[ERROR] Error: OmniSharp server load timed out. Use the 'omnisharp.projectLoadTimeout' setting to override the default delay (one minute).

Environment information
VSCode version: 1.32.3
C# Extension: 1.19.0
Mono Information
        OmniSharp using global mono :5.16.0
Dotnet Information
.NET Command Line Tools (2.1.2)
Product Information:
Version:            2.1.2
Commit SHA-1 hash:<PHONE_NUMBER>
Runtime Environment:
OS Name:     Mac OS X
OS Version:  10.13
OS Platform: Darwin
RID:         osx.10.12-x64
Base Path:   /usr/local/share/dotnet/sdk/2.1.2/
Microsoft .NET Core Shared Framework Host
Version  : 2.0.3
Build    : a9190d4a75f4a982ae4b4fa8d1a24526566c69df

Visual Studio Code Extensions

Extension
Author
Version

bracket-pair-colorizer
CoenraadS
1.0.61

classynaming
reflectiondm
1.2.0

code-spell-checker
streetsidesoftware
1.7.1

csharp
ms-vscode
1.19.0

dart-code
Dart-Code
2.25.1

docomment
k--kato
0.1.6

flutter
Dart-Code
2.25.1

plantuml
jebbs
2.10.9

unity-debug
Unity
2.7.2

view-in-browser
qinjia
0.0.5

vscode-language-pack-ja
MS-CEINTL
1.33.2

can you try updating your global Mono? Latest extension 1.19.0 requires minimum 5.18 and the latest available on https://www.mono-project.com/download/stable/ is already 5.20.1.

Thank you.
Updated to 5.18.1. But the same error occurs.
OmniSharp Full log
Starting OmniSharp server at 4/24/2019, 6:10:54 PMvir
    Target: /Users/(user_name)/(project_name)/(project_name).sln

OmniSharp server started with Mono 5.18.1 (/Library/Frameworks/Mono.framework/Versions/5.18.1/).
    Path: /Users/(user_name)/.vscode/extensions/ms-vscode.csharp-1.19.0/.omnisharp/1.32.3-beta.9/omnisharp/OmniSharp.exe
    PID: 61808

[info]: OmniSharp.Stdio.Host
        Starting OmniSharp on MacOS 10.13.6 (x64)
[info]: OmniSharp.Services.DotNetCliService
        DotNetPath set to dotnet
[info]: OmniSharp.MSBuild.Discovery.MSBuildLocator
        Located 2 MSBuild instance(s)
            1: Mono 15.0 - "/Library/Frameworks/Mono.framework/Versions/5.18.1/lib/mono/msbuild/15.0/bin"
            2: StandAlone 15.0 - "/Users/(user_name)/.vscode/extensions/ms-vscode.csharp-1.19.0/.omnisharp/1.32.3-beta.9/omnisharp/msbuild/15.0/Bin"
[info]: OmniSharp.MSBuild.Discovery.MSBuildLocator
        Registered MSBuild instance: Mono 15.0 - "/Library/Frameworks/Mono.framework/Versions/5.18.1/lib/mono/msbuild/15.0/bin"
            CscToolPath = /Users/(user_name)/.vscode/extensions/ms-vscode.csharp-1.19.0/.omnisharp/1.32.3-beta.9/omnisharp/msbuild/15.0/Bin/Roslyn
            CscToolExe = csc.exe
System.TypeLoadException: VTable setup of type Microsoft.AspNetCore.Razor.OmniSharpPlugin.OmniSharpPluginStrongNamedRoslynServiceProvider failed
  at (wrapper managed-to-native) System.RuntimeType.GetPropertiesByName_native(System.RuntimeType,intptr,System.Reflection.BindingFlags,bool)
  at System.RuntimeType.GetPropertiesByName (System.String name, System.Reflection.BindingFlags bindingAttr, System.Boolean icase, System.RuntimeType reflectedType) [0x0001b] in <b4d1a66727f44ba888b54966284fabb4>:0
  at System.RuntimeType.GetPropertyCandidates (System.String name, System.Reflection.BindingFlags bindingAttr, System.Type[] types, System.Boolean allowPrefixLookup) [0x00010] in <b4d1a66727f44ba888b54966284fabb4>:0
  at System.RuntimeType.GetProperties (System.Reflection.BindingFlags bindingAttr) [0x00000] in <b4d1a66727f44ba888b54966284fabb4>:0
  at System.Reflection.RuntimeReflectionExtensions.GetRuntimeProperties (System.Type type) [0x00006] in <b4d1a66727f44ba888b54966284fabb4>:0
  at System.Composition.TypedParts.Discovery.TypeInspector+<DiscoverPropertyExports>d__7.MoveNext () [0x00030] in <9aac21a4a3f24063b2b8b59143acddfc>:0
  at System.Composition.TypedParts.Discovery.TypeInspector+<DiscoverExports>d__5.MoveNext () [0x000dc] in <9aac21a4a3f24063b2b8b59143acddfc>:0
  at System.Composition.TypedParts.Discovery.TypeInspector.InspectTypeForPart (System.Reflection.TypeInfo type, System.Composition.TypedParts.Discovery.DiscoveredPart& part) [0x00061] in <9aac21a4a3f24063b2b8b59143acddfc>:0
  at System.Composition.TypedParts.TypedPartExportDescriptorProvider..ctor (System.Collections.Generic.IEnumerable`1[T] types, System.Composition.Convention.AttributedModelProvider attributeContext) [0x00037] in <9aac21a4a3f24063b2b8b59143acddfc>:0
  at System.Composition.Hosting.ContainerConfiguration.CreateContainer () [0x00042] in <9aac21a4a3f24063b2b8b59143acddfc>:0
  at OmniSharp.CompositionHostBuilder.Build () [0x001ed] in <c228582629e34359bfdbc0fc855c1d2f>:0
  at OmniSharp.Stdio.Host..ctor (System.IO.TextReader input, OmniSharp.Stdio.Services.ISharedTextWriter writer, OmniSharp.IOmniSharpEnvironment environment, Microsoft.Extensions.Configuration.IConfiguration configuration, System.IServiceProvider serviceProvider, OmniSharp.CompositionHostBuilder compositionHostBuilder, Microsoft.Extensions.Logging.ILoggerFactory loggerFactory, System.Threading.CancellationTokenSource cancellationTokenSource) [0x0007e] in <19355510af164339b7bccd4ca09bc44c>:0
  at OmniSharp.Stdio.Driver.Program+<>c__DisplayClass0_1.<Main>b__1 () [0x00117] in <ba7fbbf8e8fb4043b79656a74082347f>:0
  at OmniSharp.CommandLineApplication+<>c__DisplayClass11_0.<OnExecute>b__0 () [0x0000b] in <c228582629e34359bfdbc0fc855c1d2f>:0
  at McMaster.Extensions.CommandLineUtils.CommandLineApplication.Execute (System.String[] args) [0x00039] in <344a1e8cd6594b17b1e01f8df6ae8827>:0
  at OmniSharp.CommandLineApplication.Execute (System.Collections.Generic.IEnumerable`1[T] args) [0x00042] in <c228582629e34359bfdbc0fc855c1d2f>:0
  at OmniSharp.Stdio.Driver.Program+<>c__DisplayClass0_0.<Main>b__0 () [0x00028] in <ba7fbbf8e8fb4043b79656a74082347f>:0
  at OmniSharp.HostHelpers.Start (System.Func`1[TResult] action) [0x0001c] in <c228582629e34359bfdbc0fc855c1d2f>:0

the error shows you are running OmniSharp 1.32.3-beta.9 which is really old (not the one that ships with 1.19.0 extension)
please check in your VS Code settings what is your "omnisharp.path" setting? if you have anything set, please remove it and reinstall the extension, this will pull the latest stable OmniSharp.

It was set to "omnisharp.path": "1.32.3-beta.9", so delete it and solve the problem.
Thank you for your help.

great! 👍👍


Figure out a way to make scripts run when a Bugzilla change is made

IIRC @kyoshino was working on something to notify clients of changes in Bugzilla.
There are some scripts (e.g. component assignment) which would be nice to run as soon as a bug is filed / changed, rather than hourly.

There once was https://wiki.mozilla.org/BMO/ChangeNotificationSystem, but I think it has been decommissioned.

Yes, we don’t have any notification system now. Our hope is GraphQL: Bug 1281087 - Re-implement Bugzfeed-like change notification system using GraphQL Subscriptions

#1107 is the way.


Remove variables::VpiSigned?

variables have typespec and typespec also have VpiSigned. Is variables::VpiSigned remnant from time before typespec? Can this be removed?

Sure. But does it do any harm?


Automatize the package publishment

Closes #129
In the build.gradle :

Update from the deprecated plugin maven to maven-publish
Signing is now linked to the Publishing task (variables have changed)
Add nexusStaging task in order to automatize the closing and release in the nexus repository

Other :

Add github action to publish the package on the Sonatype OSSRH on each release
gradle.properties has been deleted and we now use environment variables
Update CONTRIBUTING.md

@curquiza we will need to add the 'Check release validity'
@eskombro do we still need all the publishing steps in the CONTRIBUTING.md?
The last step of the  GitHub Action ossrh-publish.yml has not been tested  "Close and release Nexus Repository" as we don't want to publish a new version yet.
Also the Secrets need to be added.
:blush:

Bors merge


Notebook related rule updates

This is an open discussion around linting Jupyter Notebooks using certain rules that doesn't map well with the notebook model.
For some context, Notebooks are made up of cells, each cell can be assumed to act as a single Python file but that's not exactly true as anything defined in a cell is available throughout the notebook via the global scope.
There are certain lint rules which doesn't map well with notebooks and this issue is to keep track of them in a central place. We can discuss on possible solutions here.

[ ] E402: module-import-not-at-top-of-file - Usually imports definition in a notebook is scattered across multiple cells. The current workaround is to ignore this rule but one could argue that this rule should work at cell level i.e., "Module level import not at top of cell".
[x] B018: useless-expression, B015: useless-comparison - Expressions in a cell are not useless if they're at the end because the result of that expression is the output of the cell. So, these rules should be updated to avoid raising any violation if it's the last expression in the cell.
[x] E703: useless-semicolon - On the other hand, semicolons are used to avoid the output mentioned in the previous point. If there's a trailing semicolon for the last expression in a cell, it's probably best to avoid raising a violation for it. The formatter does a similar thing: https://github.com/astral-sh/ruff/pull/8590
[x] D100: undocumented-public-module - This is irrelevant for Notebooks.

I think another rule that makes less sense in notebooks is D100. Module docstrings are there most likely in markdown.
And I'm also not shure about T201, because you need this from time to time if something within the cell should also be output and not just the last line?

Do you mean that the content of the module docstring will be as a separate markdown cell?

Yes, I would put everything I write in a module in the module docstring in a notebook in a markdown cell, which is why it makes no sense to have a module docstring.

If so, then I think yeah we should avoid triggering this rule. Also, I think D100 in general isn't relevant for Notebooks as they can't be imported.

That's what I basically wanted to say. That it makes no sense to me in notebooks that the rule triggers.

I'm a bit torn on how to go for E402. There are a few solutions for this:

We recommend disabling them for notebooks
By default we disable them for notebooks
For notebooks, we'd check the rule for each cell so "Module level import not at top of cell"

I'm thinking of going with (3) in preview.

@redeboer Yeah, I think we can discuss this in a separate issue.


Use Link outside of a router

This should resolve #3889.
I added a note to the documentation listing the restrictions on this, which are:

to must be a string. If to is an object, then the url will be /[object%20Object].
If any active aware props are used, then a location prop must be included in the <Link>'s props.

I'm actually considering not supporting all the location from props instead of context stuff. The idea was for redux support but I think ControlledRouter is better.
So, that leaves testing as the only use-case for rendering any of these components outside the context of a router. In v2 that was nice, but in v4 you can just wrap your unit test in a MemoryRouter and be done with it. So I'm going to be removing all the "prop instead of context"

Hi RR friends!
I'm using RR4 and although wrapping the component under test in <MemoryRouter> removes the error, I've now lost the ability to test the state of the component since enzyme's state() method only works on the root. For context, I'm writing an integration test on a form-based flow, so I'm using mount rather than doing shallow rendering. Any ideas?


Cli tool for Golte

Hey there! I made a cli tool for Golte.
https://github.com/TimLai666/golte-cli
With this tool, you can:

create a Golte project, automatically run npm init, go mod init, etc.
build a Golte project.
run a Golte project.

Awesome! I've added your project to the README (03c65fc69e028aa43d16a145ba1875a6d207bd7b).


Missing data for 2018/10/21,22 hours 23,00,01

Request returns 404 for following urls:
url: http://data.gharchive.org/2018-10-21-23.json.gz
url: http://data.gharchive.org/2018-10-22-0.json.gz
url: http://data.gharchive.org/2018-10-22-1.json.gz

GitHub was down during that time, hence no activity files:
https://status.github.com/messages


Fix FLAnimatedImage import when using CocoaPods

New Pull Request Checklist

[X] I have read and understood the CONTRIBUTING guide
[X] I have read the Documentation
[X] I have searched for a similar pull request in the project and found none
[ ] I have updated this branch with the latest master to avoid conflicts (via merge from master or rebase)
[ ] I have added the required tests to prove the fix/feature I am adding
[ ] I have updated the documentation (if necesarry)
[ ] I have run the tests and they pass
[ ] I have run the lint and it passes (pod lib lint)

This merge request fixes / reffers to the following issues: ...
Pull Request Description
This fix the import issue when trying to use SDWebImage/GIF with CocoaPods.

By the way, I've read the current CHANGELOG. When it comes to FLAnimatedImage support, It says

requires iOS 8+ (it's the only way due to FLAnimatedImage requirements)

However, FLAnimatedImage's podspec requires only iOS 6. And I just installed with Podfile that requires iOS 7 and it compiles just as it should do. Am I misunderstanding this or is this a mistake in the changelog?

This somehow messed up the tests. I'll look into it tonight.

@skyline75489 looks good. I tested all combinations (Swift, Objc with and without use_frameworks) and they all work.


fix(docs): corrects a typo in product documentation

Scope of change is restricted to docs only.

Thanks for helping us out! 🙌


Can China Futures CTP interface be supported

Similar implementations include vnpy, which supports many exchanges, as well as China's futures market and stock market. Unfortunately, the efficiency of the Python based framework is not very satisfactory.
I have been looking for a Rust based framework. I was lucky to find Barter, but only support crypto exchanges.
Development Documentation：CTP Client Development Guide.pdf
CTP_Client_Development_Guide.pdf
c++ code & dll is:v6.6.8_traderapi.zip
(I found the trust encapsulation of the ctp interface, but I don't know how to combine it with barter:https://github.com/WiSaGaN/ctp-rs)
test account is :https://www.simnow.com.cn/   (MarketPrice not need account)

Hello @kingmo888 !
Thanks for your question / suggestion!
This project is still very small and is gaining traction...
Currently most of the contributors are focused on enhancing the crypto exchanges support,
even though the design goal of the project is indeed to be agnostic of different types of assets traded.
So, my advice is that you wait a little bit until we organise the project & update docs regarding contributing, project architecture, small guides for adding new data sources etc. and then wait for either an adopter of this issue, or start working on it on your own.
If you have any addition questions, it is best that you hop on our discord server (https://discord.gg/wE7RqhnQMV),
and chat there with the rest of community.
Sincerely,
@carpetmaker

By the way, taking VNPY as an example,

This project looks amazing! It seems to have a very advanced feature set, is well-documented and seems to be very popular / very actively worked on. There is however an unfortunate language barrier (most of the resources are in Chinese), but that is where I will definitely use your occasional help of translating the resources used for the potential reference implementation later on.
Feel free to join the discord server here, so you can chat with the rest of the community.
(hopefully you don't have any geo-location / regional restrictions)

Closing this now we've migrated this repo to the new barter-rs monorepo:
https://github.com/barter-rs/barter-rs


[sei-cosmwasm] update order data type and example contract

Add status_description to Order data structure
Use the latest bulk order placement response structure in the example contract

Did we do testing for this? Generally for testing we should be instantiating testing contract on local chain and verifying behavior

Yes


chore: bdd-spring-1574819600 to 0.0.1

chore: Promote bdd-spring-1574819600 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Ability to migrate from a HA airgapped install to HA helm managed install

What kind of request is this (question/bug/enhancement/feature request):
enhancement
Description:
For users who have installed Rancher HA using the airgapped instructions here - https://rancher.com/docs/rancher/v2.x/en/installation/air-gap-high-availability/ there is no migration path to move to a Helm managed installation that is described here - https://rancher.com/docs/rancher/v2.x/en/installation/ha/
Develop a process to do this migration.
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI):
v2.1.1

Installation option (single install/HA):
HA airgapped

Is it possible that the same or similar instructions for moving from HA RKE to HA Helm could address this scenario? Referring specifically to https://rancher.com/docs/rancher/v2.x/en/upgrades/upgrades/migrating-from-rke-add-on/


Validate initializer arguments before sending any transactions

deployProxy and deployBeaconProxy calls getInitializerData in hardhat or truffle, which tries to encode the initializer call and gives an error if that failed.
The fix for this issue may be to just ensure getInitializerData is called before any deployments occur, and to add corresponding tests.


Different loss functions for training and validation

I am having a strange problem where I am unable to use different loss functions for pytorch lightning. So, if I use something like this, it is completely fine:
class SegmentationModel(pl.LightningModule):
    def __init__(self, hparams: dict):
        self.lossfn = GeneralizedDiceLoss()

    def training_step(self, batch, batch_idx):
        inputs, targets = self.prepare_batch(batch, output_folders=False)
        logits = self.model(inputs)
        loss_val = self.lossfn(logits, targets)
        return {'loss': loss_val}

    def validation_step(self, batch, batch_idx):
        inputs, targets = self.prepare_batch(batch)
        logits = self.model(inputs)
        loss_val = self.lossfn(logits, targets)
        return {'val_loss': loss_val}

Now, as soon as I want to use a different function (even if it is the same function!), things do not work. In particular, the validation loss does not move. So, if I do something like this in my class definition:
class SegmentationModel(pl.LightningModule):
    def __init__(self, hparams: dict):
        self.train_lossfn = GeneralizedDiceLoss()
        self.val_lossfn = GeneralizedDiceLoss()  # same loss!

    def training_step(self, batch, batch_idx):
        inputs, targets = self.prepare_batch(batch, output_folders=False)
        logits = self.model(inputs)
        loss_val = self.train_lossfn(logits, targets)  # use train loss
        return {'loss': loss_val}

    def validation_step(self, batch, batch_idx):
        inputs, targets = self.prepare_batch(batch)
        logits = self.model(inputs)
        loss_val = self.val_lossfn(logits, targets)  # use validation loss
        return {'val_loss': loss_val}

This makes the model validation completely stuck and it seems the model does not generalise at all. I have a feeling that both the loss functions get called in sequence because the output shows something like:
 | Name         | Type                | Params
-----------------------------------------------------
0 | model        | UNet2D              | 2 M
1 | train_lossfn | GeneralizedDiceLoss | 0
2 | val_lossfn   | GeneralizedDiceLoss | 0

Perhaps I am using this wrong and I probably need to detach the validation loss function but is there a way to achieve this?

@pamparana34 can you describe more about things do not work

 | Name         | Type                | Params
-----------------------------------------------------
0 | model        | UNet2D              | 2 M
1 | train_lossfn | GeneralizedDiceLoss | 0
2 | val_lossfn   | GeneralizedDiceLoss | 0

As per my knowledge, since the loss function is for computing how much different model's predictions and ground truth are, so it doesn't have parameters to learn, it will be zero.
But I'm not sure, I could be wrong.
the validation loss is properly fluctuating in this example

could you point us the the GeneralizedDiceLoss so we can try to reproduce.

Looks like it's coming from here


How to quickly create an application skeleton for TypeScript?

I use application generator tool, express-generator, to quickly create an application skeleton for my nodejs application.
I am finding application generator tool to quickly create an application skeleton for TypeScript to easily use with express.js.

This is not a support forum.
Questions should be asked at StackOverflow or on Gitter.im.


Tolerate failures to delete federated credentials during deletion

Which issue this PR addresses:
Fixes no JIRA

What this PR does / why we need it:
Our federated credential built-in role doesn't yet have permissions to clean up federated credentials, so currently wi/mi installs are failing to clean them up. For the time being, we need to tolerate failures to clean up federated credentials rather than fail the whole install.

Test plan for issue:
Tested this locally on deletion of a wi/mi cluster. Deletion succeeded.

Is there any documentation that needs to be updated for this PR?

How do you know this will function as expected in production?

/azp run ci


[ISSUE#2176] Fix the issue of redundancy parameters in method

What is the purpose of the change
fix the issue #2716
Brief changelog
XX
Verifying this change
XXXX
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Coverage increased (+0.03%) to 51.272% when pulling 1ec6a39e970aecc332a1a781641efa21dd65bd9d on fantow:fix_issue#2176_ into c91a2f4cb084c19112e43febabe4824c35ff202d on apache:develop.


speaker_embedding_dimension

Line 220 in model.py:
self.speaker_embedding_dim = hparams.encoder_embedding_dim

Should be
self.speaker_embedding_dim = hparams.speaker_embedding_dim

??

Yes, you're right @tshmak . I probably did that because I wanted to add/point-wise multiply the speaker embeddings to the the encoder output, rather than concatenate them.
But it shouldn't lead to a problem, as I don't think that attribute(self.speaker_embedding_dim) is being used elsewhere.
Do you want to open a pull request? Or shall I correct ?

Please correct, as my repository contains other modifications to your program also. Just to clarify, your program concats rather than add/multiply the speaker embedding right? (cf. here). Is it because you tried add/multiply and in the end decided to stick with concat?

@tshmak Yes, the program concats right now. It is because the papers says to concatenate. Although, I am yet to try multiplying/adding , but i lost server access, so couldn't do.


[docs] Fix outdated link to next/link docs

[x] I have followed (at least) the PR section of the contributing guide.

While I was checking out the examples for setting up MUI in Next.js project I've found that the urls to next/link are outdated (since the version 8 of Next.js I believe)

We had it already right in https://mui.com/guides/routing/#next-js. @radlinskii Thanks!


Define lifecycle hooks in the model

Currently you can define lifecycle hooks at actions. I think it's nice if we could define them inside model as well especially when we finish implementing #60.

This is now possible with Vuex ORM 0.29.0! 🎉  However, I needed to introduce breaking change where you can no longer define lifecycle hook at actions... Please see the release note for more detail!
https://github.com/vuex-orm/vuex-orm/releases/tag/v0.29.0


Issues fixes

https://github.com/NUS-ALSET/achievements/issues/297#issuecomment-415249451

@FeynmanDNA updated code of HomeV2. Could you review your notes?

@Brainenjii the update looks awesome!
I think the new solvesPySkills and unSolvedPySkills are work in progress from commit by @walkwel


Fail import tensorflow if rules_python is installed.

Issue type
Bug
Have you reproduced the bug with TensorFlow Nightly?
Yes
Source
source
TensorFlow version
tf 2.17.0
Custom code
No
OS platform and distribution
Ubuntu 24.04
Mobile device
No response
Python version
3.11
Bazel version
No response
GCC/compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current behavior?
When I installed both TensorFlow and Mesop simultaneously, I encountered an error while importing TensorFlow. It seems that installing Mesop also installs rules_python, which causes TensorFlow to malfunction.
Specifically, at the following code:
https://github.com/tensorflow/tensorflow/blob/bc90265931a0ca90ee2e7c2ef988ad6634733961/tensorflow/python/platform/resource_loader.py#L117-L119
r becomes None, leading to an error.
According to rules_python:
https://github.com/bazelbuild/rules_python/blob/0.26.0/python/runfiles/runfiles.py#L40
It seems possible that r = runfiles.Create() can indeed return None.
While it seems the issue could be linked to Mesop installing rules_python, there might also be an underlying problem with TensorFlow. I would appreciate your help in investigating further.
Standalone code to reproduce the issue
$ pip install "tf-nightly==2.18.0.dev20240906" "mesop==0.12.3"
$ python -c "import tensorflow"

# I run this in docker image of python:3.11.

Relevant log output
2024-09-06 21:45:07.412511: I external/local_xla/xla/tsl/cuda/cudart_stub.cc:32] Could not find cuda drivers on your machine, GPU will not be used.
2024-09-06 21:45:07.415758: I external/local_xla/xla/tsl/cuda/cudart_stub.cc:32] Could not find cuda drivers on your machine, GPU will not be used.
2024-09-06 21:45:07.425482: E external/local_xla/xla/stream_executor/cuda/cuda_fft.cc:477] Unable to register cuFFT factory: Attempting to register factory for plugin cuFFT when one has already been registered
WARNING: All log messages before absl::InitializeLog() is called are written to STDERR
E0000 00:00:1725659107.441013      78 cuda_dnn.cc:8322] Unable to register cuDNN factory: Attempting to register factory for plugin cuDNN when one has already been registered
E0000 00:00:1725659107.445948      78 cuda_blas.cc:1418] Unable to register cuBLAS factory: Attempting to register factory for plugin cuBLAS when one has already been registered
2024-09-06 21:45:07.463219: I tensorflow/core/platform/cpu_feature_guard.cc:210] This TensorFlow binary is optimized to use available CPU instructions in performance-critical operations.
To enable the following instructions: AVX2 FMA, in other operations, rebuild TensorFlow with the appropriate compiler flags.
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/usr/local/lib/python3.11/site-packages/tensorflow/__init__.py", line 53, in <module>
    from tensorflow._api.v2 import compat
  File "/usr/local/lib/python3.11/site-packages/tensorflow/_api/v2/compat/__init__.py", line 8, in <module>
    from tensorflow._api.v2.compat import v1
  File "/usr/local/lib/python3.11/site-packages/tensorflow/_api/v2/compat/v1/__init__.py", line 30, in <module>
    from tensorflow._api.v2.compat.v1 import compat
  File "/usr/local/lib/python3.11/site-packages/tensorflow/_api/v2/compat/v1/compat/__init__.py", line 8, in <module>
    from tensorflow._api.v2.compat.v1.compat import v1
  File "/usr/local/lib/python3.11/site-packages/tensorflow/_api/v2/compat/v1/compat/v1/__init__.py", line 47, in <module>
    from tensorflow._api.v2.compat.v1 import lite
  File "/usr/local/lib/python3.11/site-packages/tensorflow/_api/v2/compat/v1/lite/__init__.py", line 9, in <module>
    from tensorflow._api.v2.compat.v1.lite import experimental
  File "/usr/local/lib/python3.11/site-packages/tensorflow/_api/v2/compat/v1/lite/experimental/__init__.py", line 8, in <module>
    from tensorflow._api.v2.compat.v1.lite.experimental import authoring
  File "/usr/local/lib/python3.11/site-packages/tensorflow/_api/v2/compat/v1/lite/experimental/authoring/__init__.py", line 8, in <module>
    from tensorflow.lite.python.authoring.authoring import compatible # line: 263
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/tensorflow/lite/python/authoring/authoring.py", line 42, in <module>
    from tensorflow.lite.python import convert
  File "/usr/local/lib/python3.11/site-packages/tensorflow/lite/python/convert.py", line 151, in <module>
    _deprecated_conversion_binary = _resource_loader.get_path_to_datafile(
                                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/tensorflow/python/platform/resource_loader.py", line 118, in get_path_to_datafile
    new_fpath = r.Rlocation(
                ^^^^^^^^^^^
AttributeError: 'NoneType' object has no attribute 'Rlocation'

@n-kats,
As a Temporary fix, Could you please try to add python definitions in the Project's {project}/WORKSPACE file, and it might  work and the Project can be build succesfully.
Thank you!

In my project, I'm not using Bazel or a WORKSPACE file. Should I still create a WORKSPACE file in this case?

In my project, I use TensorFlow and Mesop (installed via pip), and there is no build process involved.
However, in the Mesop project, the developers use Bazel and rules_python, and the Mesop wheel file includes rules_python (I believe this is causing the issue).


Secure/Harden MongoDB

https://www.owasp.org/index.php/Testing_for_NoSQL_injection#Example_1

https://docs.mongodb.com/manual/core/security-encryption-at-rest/

https://stackoverflow.com/questions/31151644/using-pymongo-with-public-ssl-key

We don't need no databases.


[BUG] Failed generation when translation string contains \ and $

I have a string in my .arb files that contains a \, something like:
{
  "my_string": "a\\b c\\$d",
}

Note that the \is already escaped, to be treated as a normal character.
When I run  intl_utils:generate it fails with the error:
Error in parsing [redacted]/generated/l10n.dart, no messages extracted.
ERROR: Failed to generate localization files.
Invalid argument(s): Parsing errors in [redacted]/generated/l10n.dart

The generated/l10n.dart file is actually generated, but none of the generated/intl/messages_*.dart.
Here is the relevant part of the generated file:
/// `a\b c\$d`
String get my_string {
  return Intl.message(
    'a\b c\\$d',
    name: 'my_string',
    desc: '',
    args: [],
  );
}

There are two separate issues here:

The first \ in the dart string in line 4 is not properly escaped (should be \\) and treated as a control sequence.
The second \ is properly escaped, however the following '$' is not, and therefore treated as a string interpolation symbol. This one actually leads to the above error.

Hi @schultek,
Thanks for reporting this!
The fix is available in the latest version - intl_utils 2.2.0.


Archive all files level log every day

Hi Guys!
I want to archive all files level log every day but It is only archive one file in list files level log
Please help me!
My config:

<target name="LogFile" xsi:type="File" keepFileOpen="true"
  layout="${longdate} ${uppercase:${level}} ${newline} ${message} ${newline}"
  fileName="${basedir}/../logs/App-test.${shortdate}.${level}.log"
  archiveFileName="${basedir}/../logs/archives/App-test.{#}.gz"
  archiveEvery="Day"
  archiveNumbering="Date"
  archiveDateFormat="yyyyMMdd"
  maxArchiveFiles="30"
  enableArchiveFileCompression="true" />

Please fill in the template, so we could help you with your issue, thanks!

`

`

You mean fill full file Nlog.config ?
<?xml version="1.0" encoding="utf-8"?>
<nlog xmlns="http://www.nlog-project.org/schemas/NLog.xsd" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" throwExceptions="true" internalLogFile="c:\nlog.txt" internalLogLevel="Debug">
  <extensions>
    <add assembly="NLog.Extended" />
  </extensions>

  <targets async="true">
    <target name="LogFile" xsi:type="File" keepFileOpen="true"
      layout="${longdate} ${uppercase:${level}} ${newline} ${message} ${newline}"
      fileName="${basedir}/../logs/App-test.${shortdate}.${level}.log"
      archiveFileName="${basedir}/../logs/archives/App-test.{#}.gz"
      archiveEvery="Day"
      archiveNumbering="Date"
      archiveDateFormat="yyyyMMdd"
      maxArchiveFiles="30"
      enableArchiveFileCompression="true" />
  </targets>

  <rules>
    <logger name="*" levels="info,warn,error,debug,trace" writeTo="LogFile" />
    <logger name="Name.Space" minlevel="Debug" maxlevel="Error" final="true" />
  </rules>
</nlog>

https://github.com/NLog/NLog/issues/new

@pisvthanh The problem is the {shortdate} in fileName=

fileName="${basedir}/../logs/App-test.${shortdate}.${level}.log"

This means it rolls automatically, so the archive/compression-logic gets confused.

I will be implement differently
Thanks!

I assume your question has been answered, if not, please let us know!


Table 12104 "Withhold Code" - field 5 "770 Code" lenght to increase

Hi,
as per new Italian regulatory update on 2022 Withholding Tax Export, field "770 Code" can be used with 2-digit values.
Actually field 5 "770 Code" is Text[1]
Can you please increase field lenght to be compliant with the above regulatory update?
Thanks

Hi @AlexanderYakunin, this issue is still in status Open. 7 months has been passed without an answer.
Could you please let me know your plan for this issue?

We are not working on Regulatory changes. This is another team, and they have this request in their backlog. Please let me know which legal update you are talking about, and I will check with them. Going forward please use support requests to query about legal updates if there is any urgency. This forum is not suitable for that.

In this link you can find istructions, in italian, for 2022 declaration (page 75-76 you can see codes can be 2 digit).
Next declaration is expected in January 2023.
Morover, you can take into consideration that in technical specification, this field is exported in a non-positional record. So potentially in future it can be higher than 2 chars.

Hello!
Thank you for this information!
We investigated this case and see that this field is not used anywhere except table and page in current implementation. Our Regulatory Features team is willing to investigate it further as we may miss something important in this declaration export. In order to get it done we need to have technical support experts working on this case, please open service request instead of this GitHub requests, using one of possible methods:
•	On premises - https://support.microsoft.com/en-us/supportforbusiness/productselection?sapId=93d37907-ad94-d591-22e9-593cfa09dd3f
•	On line – contact your partner, or if you are a partner open a support incident through Partner Center - https://partner.microsoft.com/en-us/pcv/dashboard/overview


Node module update

Update node modules and removes the explicit 10.4.1 node build.
Fixes #13 #14 #15 #20 #22

Coverage remained the same at 77.273% when pulling e819a23e056863c3b84f256f684ebcbcc7610314 on node-module-update into 4b5659a15b1f80dc77bb5badc07c391af62a29b0 on master.


Strange truncation of fastq files when demultiplexing short Illumina inserts libraries

Hi there,
Have been using your library to demultiplex libraries - 384 well illumina libraries per ONT run - which have a similar construction to Tru-seq paired indexes.
The first sets tested worked very well - library insert size range 200  - 600 bp - targeted PCR library with intentional bias to see if the input ratio would be picked up and was.
However, we are also interested in the performance of the indexing plates - both for cross contamination and total library read counts per index - then compare back to the original Illumina library run. Using a 90 bp insert (x 4 - one per quadrant of the plate) and final amplicon 165 bp long. The combined demultiplexing report of 800 plus files provides a relatively realistic number of index pairs found - however the demux fastq.gz files - one per 384 well, provides only short reads of mean 26 bases +/- 40 bp. Strangely.  The filtered fastq files were filtered for 180 to 400 bp ranges and pre-mapped to verify that number quantity of mappable reads. Post Anglerfish demux  results in low or no alignment (bowtie2 or Minknow medeka) - looking through with fastqc showed they were too short - BLAST did no find many that aligned with insert or the i5 or i7 region.
I have attached a zipped file with: output of Anglerfish / text with settings used (have tried with and without -lenient) / fastq filtered file / the resulting demux reads from this file / a copy of the samples sheet - 384 indexes. I am using v0.6.1 - Ubuntu on WSL - Python 3.10 - minimap2 is installed. Unsure if very short amplicons causes problems with the pipeline. The fragment range includes singles and duplicated reads - expected.
anglerfish_test.zip
Thanks very much for this pipeline! We are hoping to pipe to this for real-time demultiplexing.

Hi!
Thanks for sharing the result files, and for testing out anglerfish. Sorry if I'm misunderstanding something about your library or sequencing setup.

the combined demultiplexing report of 800 plus files provides a relatively realistic number of index pairs found - however the demux fastq.gz files - one per 384 well, provides only short reads of mean 26 bases +/- 40 bp. [...] Post Anglerfish demux results in low or no alignment (bowtie2 or Minknow medeka) - looking through with fastqc showed they were too short - BLAST did no find many that aligned with insert or the i5 or i7 region.

Anglerfish will only output in the demuxed.fastq.gz files the library inserts or more accurately, the sequence between the two matching i7 and i5 templates. I'll give an example using paftools view for the minimap2 alignment file you included, for one read (the query sequence)
(base) remi.olsen@remiombp anglerfish_test % paftools view truseq_dual.paf | grep "002774fa-5542-4b67-a3da-42f289e20508" -A 3
>002774fa-5542-4b67-a3da-42f289e20508	182	128	181	+	truseq_dual_i7	65	0	53	45	45	42	NM:i:8	ms:i:262	AS:i:262	nn:i:8	tp:A:P	cm:i:10	s1:i:36	s2:i:0	de:f:0.1509	rl:i:0
Ref+:          1 =================================nnnnnnnn============ 53
                 |||||||||||||||||||||||||||||||||        ||||||||||||
Qry+:        129 =================================tatgcacg============ 181
--
>002774fa-5542-4b67-a3da-42f289e20508	182	33	102	+	truseq_dual_i5	70	0	70	61	62	60	NM:i:9	ms:i:352	AS:i:352	nn:i:8	tp:A:P	cm:i:13	s1:i:46	s2:i:0	de:f:0.1286	rl:i:0
Ref+:          1 =============================nnnnnnnn======t========================== 70
                 |||||||||||||||||||||||||||||        |||||| ||||||||||||||||||||||||||
Qry+:         34 =============================gcagacga======-========================== 102

We see that the first 33 bases of this 182 bp long read does not match the template (but probably still is adaptor), the second not mathcing area is between bp 102 and 129, which anglerfish counts as the insert. And indeed the majority of the 4000 reads - it's very fortunate that all except 2 reads map to either i5 or i7, because then I know the length distribution - seem to be this short (190 +- 31 bp):
cut -f1,2 truseq_dual.paf | sort | uniq | datamash mean 2 sstdev 2 count 2
190.48724362181	31.057328130074	3998

This is all to say that, if you expected longer inserts (90 as you say, in all these wells or only 4 of them?) I am not able to explain why these are so short from the anglerfish output alone. Even the few "long" ones, say over 300 bp
cut -f1,2 truseq_dual.paf | sort | uniq | awk '{if($2 > 300){print $0}}' | wc -l
      61

Seem to be mostly be concatenated shorter fragments, e.g. i5-insert-i7-i5-insert-i7 constructs. Which is not uncommon to see.

We are hoping to pipe to this for real-time demultiplexing.

That's really cool. It's something I've had in the back of my mind, but never implemented because it doesn't really fit with the workflow in our lab.

Yes.. figured it out - but thank you for getting back to me. About 3% of reads are conatamers. It is finding the indexes - and trimming the seqence 'mappable' read) between the flanking i7 i5 region (the intended behaviour).  A reference for bowtie or medeka with just central insert which is 25 bp long works.  My bad should have looked at the sequences more closely - more than 99% of the reads map to the trimmed reafernece insert alone.
Pile up inserts

contig alignment to central (unique) region.

thanks again for getting back to me. Yes... these are all 384 plates beinig tested. Example 4000 read output - with conditions is about 1 minute to process - 32 Gb ram  /intel i7 laptop or about 12 hours for a full run or work well as the reads come off the nanopore and merge all fastq per well at the end.
Thanks again.


State dropdown is still active if user clicks out of the dropdown.

Describe the bug
State dropdown is still active if user clicks out of the dropdown. If user selects all state and scrolls the page the value in dropdown changes to some other value, .i.e. other state.
To Reproduce
Steps to reproduce the behavior:

Go to Homepage
Click on All State from the state ropdown
Scroll down to the page
See error (The value changes to some other state)

Expected behavior
Once after selection from the dropdown the value should not change id user scrolls the page.
Screenshots
If applicable, add screenshots to help explain your problem. (X)
Desktop (please complete the following information):

OS: [e.g. iOS] (Windows 8.1)
Browser [e.g. chrome, safari] (Chrome)
Version [e.g. 22] (Version 80.0.3987.163 (Official Build) (64-bit))

Smartphone (please complete the following information): (X)

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Is it because you've hovered over the states table or the map?

Closing due to inactivity.


Setup on all IPFS repos

PR Done

[ ] https://github.com/ipfs/js-ipfs
[x] https://github.com/ipfs/js-ipfs-api
[x] https://github.com/ipfs/js-ipfs-data-importing
[x] https://github.com/ipfs/js-ipfs-repo
[ ] https://github.com/ipfs/js-ipfsd-ctl
[x] https://github.com/ipfs/js-ipfs-unixfs
[x] https://github.com/ipfs/js-ipfs-blocks
[x] https://github.com/ipfs/js-libp2p-ipfs
[x] https://github.com/diasdavid/js-peer-id
[x] https://github.com/diasdavid/js-peer-info
[x] https://github.com/diasdavid/js-libp2p
[x] https://github.com/diasdavid/js-libp2p-swarm
[x] https://github.com/diasdavid/js-libp2p-ping
[ ] https://github.com/diasdavid/interface-connection
[x] https://github.com/diasdavid/js-libp2p-utp
[ ] https://github.com/diasdavid/interface-stream-muxer
[ ] https://github.com/diasdavid/js-libp2p-spdy
[ ] https://github.com/diasdavid/js-libp2p-kad-routing
[ ] https://github.com/diasdavid/js-libp2p-mdns-discovery
[ ] https://github.com/diasdavid/js-libp2p-railing
[ ] https://github.com/diasdavid/js-libp2p-record
[ ] https://github.com/diasdavid/interface-record-store
[ ] https://github.com/diasdavid/js-libp2p-distributed-record-store
[x] https://github.com/diasdavid/js-libp2p-kad-record-store
[x] https://github.com/diasdavid/js-libp2p-websockets
[x] https://github.com/diasdavid/js-multistream
[ ] https://github.com/diasdavid/js-ipld
[x] https://github.com/diasdavid/webcrypto
[x] https://github.com/whyrusleeping/js-mafmt
[x] https://github.com/vijayee/js-ipfs-merkle-dag
[ ] https://github.com/xicombd/ipfs-multipart
[x] https://github.com/jbenet/js-multiaddr
[ ] https://github.com/jbenet/js-multihashing
[x] https://github.com/jbenet/js-multihash

Merged and Released

[ ] https://github.com/ipfs/js-ipfs
[ ] https://github.com/ipfs/js-ipfs-api
[ ] https://github.com/ipfs/js-ipfs-data-importing
[ ] https://github.com/ipfs/js-ipfs-repo
[ ] https://github.com/ipfs/js-ipfsd-ctl
[ ] https://github.com/ipfs/js-ipfs-unixfs
[ ] https://github.com/ipfs/js-ipfs-blocks
[ ] https://github.com/ipfs/js-libp2p-ipfs
[ ] https://github.com/diasdavid/js-peer-id
[ ] https://github.com/diasdavid/js-peer-info
[ ] https://github.com/diasdavid/js-libp2p
[ ] https://github.com/diasdavid/js-libp2p-swarm
[ ] https://github.com/diasdavid/js-libp2p-ping
[ ] https://github.com/diasdavid/interface-connection
[ ] https://github.com/diasdavid/js-libp2p-utp
[ ] https://github.com/diasdavid/interface-stream-muxer
[ ] https://github.com/diasdavid/js-libp2p-spdy
[ ] https://github.com/diasdavid/js-libp2p-kad-routing
[ ] https://github.com/diasdavid/js-libp2p-mdns-discovery
[ ] https://github.com/diasdavid/js-libp2p-railing
[ ] https://github.com/diasdavid/js-libp2p-record
[ ] https://github.com/diasdavid/interface-record-store
[ ] https://github.com/diasdavid/js-libp2p-distributed-record-store
[ ] https://github.com/diasdavid/js-libp2p-kad-record-store
[ ] https://github.com/diasdavid/js-libp2p-websockets
[ ] https://github.com/diasdavid/js-multistream
[ ] https://github.com/diasdavid/js-ipld
[ ] https://github.com/diasdavid/webcrypto
[x] https://github.com/whyrusleeping/js-mafmt
[ ] https://github.com/vijayee/js-ipfs-merkle-dag
[ ] https://github.com/xicombd/ipfs-multipart
[ ] https://github.com/jbenet/js-multiaddr
[ ] https://github.com/jbenet/js-multihashing
[ ] https://github.com/jbenet/js-multihash

all the listed on https://github.com/diasdavid/js-libp2p
https://github.com/jbenet/js-multiaddr
https://github.com/jbenet/js-multihashing
https://github.com/jbenet/js-multihash

one more https://www.npmjs.com/package/mafmt :)

Use https://gist.github.com/dignifiedquire/1e907226b2eea6242cc2 for easier setup

@dignifiedquire let's not forget to prioritise https://github.com/ipfs/community/pull/111

I would really appreciate if we could focus a day and push all of this together. There are community members and contributors that will have their life a ton more easier when we finish all of these, like @upfs and https://github.com/ipfs/js-ipfs/issues/93.
I've been pushing more of these in the last days, but haven't been able to go through all of them alone. @xicombd @jgantunes @ngginever @noffle could you give us a hand?
Important note: when adding dignified, also update the installation docs, example: https://github.com/ipfs/js-ipfs-blocks#installation

I don't mind helping at all. In fact I was working precisely on that for js-libp2p-ipfs but it would probably be better to have #5 handled first for this one.

@JGAntunes @diasdavid I thought js-libp2p-ipfs was supposed to work in the browser as well, or is that a temporary thing?

@dignifiedquire we have https://github.com/ipfs/js-libp2p-ipfs-browser to be the one that will run in the browser, since transports are different.

Thanks to @JGAntunes 1.0.1 is now released with exactly this feature :)

awesome, thank you @JGAntunes :D

All these need updating, please help everyone who has time, they need to depend on<EMAIL_ADDRESS>now

WIP, but should make it easier to update things: https://github.com/dignifiedquire/aegir-format

Closing as most of this work has been done now.


Optimise SemanticModel::match_builtin_expr

Short-circuit more aggressively if builtins has not been imported

Performance is [a wash overall on Codspeed, but some benchmarks have a 1% speedup](Performance is a wash overall on Codspeed, but some benchmarks have a 1% speedup). But maybe it helps with the perf problems in #10965?


Organize Testing around Policies, not Centralized

As currently designed, all testing would consolidate around the single script, Policy.Tests.ps1. This makes testing an individual policy difficult. Please consider organizing tests around individual policies.
This has the side benefit of allowing CI/CD whenever there's a Add/Update to Policies automatically.

@NickFadz as discussed makes total sense to split the tests per policy to also allow easy scaling towards a more complex hierarchy of management groups as described here. Will gather some more feedback, prioritize the issues and implement the changes. Keep you posted once done. Many thanks upfront for your superb feedback 🥇


Allow sort of mappings to minimize line-crossing with mapping columns displayed

when displaying the mapping column, the lines can cross two times although it can be avoided. There is no way to change order to "uncross" manually
for instance in this case, by reverting order there would be no crossing:

Sorting wouldn't solve the issue for all cases, and it's also counter-intuitive to rely on sorting in this app, where actions performed on the document have an effect on the final output. Like when sorting transformations.
But to be honest I don't get what the problem is here.

in the column of the middle, the 2 mappings are in the other order, there will be no line crossing.
It won't be possible to avoid all line-crossing but it can help to minimize them.
if there is a different output depending on the order of mapping, it becomes even more important to be able to change order of the mappings (but the goal won't be to avoid line crossing, just to change the order)

How would sorting help in this case?

And I haven't said that sorting the mappings changes the outcome. It doesn't, else they would be sortable.
Anyhow, better loop @atlasmap/uxd so they can keep track of this feedback.

This provided case is awesome to illustrate it. If we put the third block on top, if I count well, it will allow to move from 22 crossed points to 3.

What's currently driving the order of the middle column? Another idea could be to allow manual drag/drop along the y-axis for the middle column. So instead of trying to detect which is the optimal order for least overlapping edges (agree it will probably never be perfect, and can never fully untangle certain combinations of relationships), we could use the existing strategy (in case that's important to preserve) and leave it in users control. Maybe that's what you were already talking about by "sorting"?
I suppose a maximum complexity case could have more than a single "best" order to illustrate different aspects of a dataset, so giving the user control would free us of having to "get it right" out of the box. Just an idea, take it or leave it 🙂

I had both in mind manual and automatic.
Ideally, having both would be the best :-)
Agree that an automatic layout won't have a "single best" order. Even if i would expect that some libraries exists to do this kind of work proposing different heuristics.
Starting with manual order allows to put the responsibility of providing the "best" order in user hands. It sounds a good idea.


Mangata dev v0.9.16

TO BE MERGED AFTER mangata-dev-v0.9.16 BRANCHES are merged :
https://github.com/mangata-finance/substrate/pull/22
https://github.com/mangata-finance/open-runtime-module-library/pull/6
https://github.com/mangata-finance/moonbeam/pull/6
https://github.com/mangata-finance/cumulus/pull/2
https://github.com/mangata-finance/snowbridge/pull/2


feature: SaaS 适配动态 IP 安装校验，统一业务逻辑 IPv6 表示法 (closed #926 closed #927 #787)

Codecov Report

Merging #928 (deca180) into V2.2.X_IPV6 (5c7f812) will increase coverage by 0.06%.
The diff coverage is 94.35%.

@@               Coverage Diff               @@
##           V2.2.X_IPV6     #928      +/-   ##
===============================================
+ Coverage        72.01%   72.07%   +0.06%
===============================================
  Files              303      303
  Lines            22141    22200      +59
===============================================
+ Hits             15944    16000      +56
- Misses            6197     6200       +3

Impacted Files
Coverage Δ

apps/backend/subscription/steps/agent.py
57.93% <ø> (ø)

apps/node_man/constants.py
97.17% <83.33%> (+0.05%)
:arrow_up:

apps/node_man/serializers/host.py
77.27% <87.50%> (-0.15%)
:arrow_down:

apps/node_man/handlers/validator.py
97.20% <89.18%> (-1.25%)
:arrow_down:

apps/backend/subscription/tools.py
56.29% <90.90%> (+0.53%)
:arrow_up:

...components/collections/agent_new/query_password.py
91.83% <100.00%> (+3.26%)
:arrow_up:

apps/backend/subscription/serializers.py
88.67% <100.00%> (+0.44%)
:arrow_up:

apps/node_man/handlers/host.py
88.67% <100.00%> (+0.59%)
:arrow_up:

apps/node_man/handlers/job.py
86.32% <100.00%> (+0.19%)
:arrow_up:

apps/node_man/handlers/meta.py
93.24% <100.00%> (+0.03%)
:arrow_up:

... and 7 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 5c7f812...deca180. Read the comment docs.


Added tests for sdk-type bindings

Issue describing the changes in this PR
Added unit tests for sdk-type bindings grpc changes. Related Epic - https://github.com/Azure/azure-functions-dotnet-worker/issues/1081#issuecomment-1276566098
Pull request checklist

[ ] My changes do not require documentation changes

[ ] Otherwise: Documentation issue linked to PR

[ ] My changes should not be added to the release notes for the next release

[ ] Otherwise: I've added my notes to release_notes.md

[ ] My changes do not need to be backported to a previous version

[ ] Otherwise: Backport tracked by issue/PR #issue_or_pr

[ ] I have added all required tests (Unit tests, E2E tests)

Additional information
Additional PR information

/check-enforcer evaluate


[Feature] Adding Zip Code Distance Calculator

Description
A Zip Code Distance Calculator is a tool used to determine the distance between two locations based on their zip codes. This type of calculator is commonly used in logistics, travel planning, and real estate to estimate the distance and travel time between two points.
Screenshots
Features :
Input Fields: Users enter the starting and destination zip codes.
Distance Calculation: The tool calculates the distance between the two zip codes, often using the Haversine formula to determine the great-circle distance between two points on the Earth’s surface.
Units: Results are typically provided in both miles and kilometers.
Travel Time Estimation: Some calculators also estimate travel time based on average speeds for driving, walking, or biking.
Maps Integration: Advanced tools might integrate with mapping services (like Google Maps) to provide a visual representation of the route.
Checklist

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

Code of Conduct

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

@Rakesh9100 @jessicachawla , please assign it to me.


fix crontabs -> cronTabs in tests/k8s/crd.nix

must have been missed in recent refactor (086780088cce40c1bda897136ebdf06b2f684d1c)
discovered while trying to run tests with:
nix-build release.nix -A test-results --show-trace

after this fix I can see another error (below), but don't have the time to investigate further.
...
while evaluating anonymous function at /Users/adriangierakowski/SourceCode/3nder/kubenix/modules/k8s.nix:132:64, called from /nix/store/hvf28w7kjlmsxysfpwjx39sghchswbvv-nixpkgs-20.03pre199926.d901b961a95/nixpkgs/lib/attrsets.nix:234:16:
while evaluating the attribute 'attrName' at undefined position:
while evaluating anonymous function at /nix/store/hvf28w7kjlmsxysfpwjx39sghchswbvv-nixpkgs-20.03pre199926.d901b961a95/nixpkgs/lib/modules.nix:75:45, called from undefined position:
while evaluating the attribute 'value' at /nix/store/hvf28w7kjlmsxysfpwjx39sghchswbvv-nixpkgs-20.03pre199926.d901b961a95/nixpkgs/lib/modules.nix:336:9:
while evaluating the option `kubernetes.customTypes.stable.example.com/v1/CronTab.attrName':
The option `kubernetes.customTypes.stable.example.com/v1/CronTab.attrName' is used but not defined

Thanks!


(v2.0.0) Supporting nvidia localization for nav2 following REP 105

The current implementation of isaac_ros_map_localization does not publish the output transform into tf following the REP 105.
"The goal of a localization system is keep a continous track of the position of the robot (not just finding some initial position - amcl is itself able to converge by its own even without any initial position)"  (answering https://github.com/NVIDIA-ISAAC-ROS/isaac_ros_map_localization/pull/22#issuecomment-1791695390)
The previous implementation publishes the robot position in the /localization_result topic (using the global /map reference frame) but nothing is published in /tf according (to REP 105)
To support nav2 it is required to publish the /map -> /odom transform via a tf2 broadcaster.
From my point of view, without these proposed changes we could not claim that this package is integrated with nav2.

"The goal of a localization system is keep a continous track of the position of the robot (not just finding some initial position - amcl is itself able to converge by its own even without any initial position)"
Initializing AMCL with uniformly distributed particles is an option as you suggested. But in order to converge, the robot has to be manually moved around. With this package setting the initial pose, you can start autonomous navigation immediately.
The reason this package isn't a great replacement for AMCL for all cases is that this localization is expected to fail if there are repeating patterns in the global map.
But if there aren't repeating patterns in your specific map, it might be a good replacement for AMCL.
As stated before, we believe setting the initial position is the best use for this package in a general scenario. And that is what we meant by Nav2 integration.
But feel free to use this package in anyway that is appropriate for your use case.

Hi @pabloinigoblasco
Is the commits you made suitable to replace AMCL for Nav2 usage? I am looking for alternative to replacing the default nav2 amcl.
Best,
Samuel


Vulnerability roundup 76: flex-2.6.4: 1 advisory

search, files

[ ] CVE-2019-6293 (nixos-19.09)

Scanned versions: nixos-19.09: e34ac949d1b. May contain false positives.

See https://github.com/NixOS/nixpkgs/issues/55386

See https://github.com/NixOS/nixpkgs/issues/55386#issuecomment-683792976


Texture not loading

Hello,
Ive made the skin in PS and imported it as a DDS file but when I link it in the texture parameters the gun remains black, not sure what I have missed. Hoping someone is able to help out. Thanks

this is not a sollumz bug


wildfly15/jdk11 potential issue in future?

Today I tried the latest agent version 1.2.0 together with wildfly15/jdk11. To run it we use docker image jboss/wildfly:15.0.0.Final-1 as base.
On startup some warnings are shown in the logs you probably should take a look at:
=========================================================================

  JBoss Bootstrap Environment

  JBOSS_HOME: /opt/jboss/wildfly

  JAVA: /usr/lib/jvm/java/bin/java

  JAVA_OPTS:  -server -Xms64m -Xmx512m -XX:MetaspaceSize=96M -XX:MaxMetaspaceSize=256m -Djava.net.preferIPv4Stack=true -Djboss.modules.system.pkgs=org.jboss.byteman -Djava.awt.headless=true -javaagent:/opt/jboss/wildfly/elastic-apm-agent-1.2.0.jar  --add-exports=java.base/sun.nio.ch=ALL-UNNAMED --add-exports=jdk.unsupported/sun.misc=ALL-UNNAMED --add-exports=jdk.unsupported/sun.reflect=ALL-UNNAMED --add-modules=java.se

=========================================================================

OpenJDK 64-Bit Server VM warning: Sharing is only supported for boot loader classes because bootstrap classpath has been appended
2018-12-20 12:39:20.695 [apm-server-healthcheck] INFO co.elastic.apm.agent.report.ApmServerHealthChecker - Elastic APM server is available: {"ok":{"build_date":"2018-11-09T17:48:30Z","build_sha":"2c46a2919dbe06767caeacbffd0055aab1493991","version":"6.5.0"}}
2018-12-20 12:39:20.732 [main] INFO co.elastic.apm.agent.configuration.StartupInfo - Starting Elastic APM 1.2.0 as pst on Java 11.0.1 (Oracle Corporation) Linux 4.4.0-138-generic
2018-12-20 12:39:20.733 [main] WARN co.elastic.apm.agent.configuration.StartupInfo - To enable all features and to increase startup times, please configure application_packages
2018-12-20 12:39:21,891 INFO  [org.jboss.modules] (main) JBoss Modules version 1.8.7.Final
...
WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by co.elastic.apm.agent.shaded.bytebuddy.dynamic.loading.ByteArrayClassLoader$SynchronizationStrategy$ForJava7CapableVm to method java.lang.ClassLoader.getClassLoadingLock(java.lang.String)
WARNING: Please consider reporting this to the maintainers of co.elastic.apm.agent.shaded.bytebuddy.dynamic.loading.ByteArrayClassLoader$SynchronizationStrategy$ForJava7CapableVm
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release
...

could you please consider to check this?

@sschmiedleitner Thanks for reporting! Is it working otherwise? Any other symptoms? Are you using security manager?

in general it is working, so it is transmitting data to apm-server/elasticsearch and I can see results in the APM UI. But just to be prepared for the future, I reported this.

The warnings above are since Java 9 modules were introduced- they warn about code making accesses that break the module dependency setup. Currently, the default value used for the --illegal-access flag is permit, which emits a warning during the first fault access, but this default will be changed in future releases.
I tried to reproduce but I got other warnings from WildFly classes and even when used --illegal-access=debug couldn't reproduce, however- seems like the wrong SynchronizationStrategy is chosen in Java 11, for which I opened an issue in ByteBuddy repo - https://github.com/raphw/byte-buddy/issues/603.
Since everything seems to be working properly, apart from these warnings, I am closing this issue and will follow the ByteBuddy issue.

@eyalkoren the ByteBuddy issue covers the illegal access, but what about

OpenJDK 64-Bit Server VM warning: Sharing is only supported for boot loader classes because bootstrap classpath has been appended


Add a python function that given a date-range, downloads audio in a range from Orcasound's S3

There is a class to do this, see DateRangeHLS
https://github.com/orcasound/aifororcas-livesystem/tree/main/InferenceSystem/src/hls_utils
See https://github.com/orcasound/aifororcas-livesystem/blob/main/InferenceSystem/src/PrepareDataForPredictionExplorer.py for usage.
Add support + error handling to make sure only date ranges < a certain time period are allowed.
Add support for randomly sampling audio segments within the date range.

Before this is tackled in the 2022 hackathon, it may be worth consulting with @valentina-s et al. about new code written by Google Summer of Code contributors that could be leveraged.


Button to find scene changes

A neat feature may be to, when you click a button, run wwxd/scxvid on the current output node and iterate over the entire clip. Alternatively, a helper function the user can import that allows you to pass a clip and an enum determining the mode (wwxd, scxvid, both, etc. Check the old lvsfunc or vsencode code for one such example, I forgot which one has it).
Additionally, hotkeys to jump from scene to scene might be neato as well if they do not exist yet. If they do, the user can just use those binds.

Feature has been added to vstools, you can just import the generated file into vspreview.


Wiki uses incorrect data model to create new Simulations

This page:
https://github.com/Azure/device-simulation-dotnet/wiki/[API-Specifications]-Simulations#create-default-simulation
uses the following data model to POST a new simulation:
{
  "Enabled": true,
  "DeviceTypes": [
    {
      "Id": "truck-01",
      "Count": 3
    },
    {
      "Id": "truck-02",
      "Count": 1
    },
    {
      "Id": "elevator-01",
      "Count": 10
    }
  ]
}

However, property DeviceTypes is misnamed. The correct name is DeviceModels.

Fixed. Thanks for catching this.


Avoid panic when blending two pixels with zero alpha.

I ran into a situation where trying to overlay two rgba images led to a panic.
The root cause was a division by zero.
Here's a minimal reproducible example program run against image = "0.18.0":
extern crate image;
use image::{ImageBuffer, Rgba, RgbaImage};
use image::imageops::overlay;
fn main() {
    let mut a: RgbaImage = ImageBuffer::new(1, 1);
    a.put_pixel(0, 0, Rgba([0, 0, 0, 0]));

    let mut b: RgbaImage = ImageBuffer::new(1, 1);
    b.put_pixel(0, 0, Rgba([0, 0, 0, 0]));

    overlay(&mut a, &b, 0, 0);
}

I don't have a ton of graphics experience, so I'm not sure if there's a "correct" way to blend two completely transparent pixels in terms of retaining the color components.
This patch just uses the blended colors and leaves alpha at zero.
This patch is the simplest thing I could think of to fix, but I'd be happy to change to something more idiomatic (num::CheckedDiv trait and explicit handling of the None condition?)

It seems that the final_alpha is set to 1.0? Should this not be zero?

Doh, I wasn't thinking and was only testing against images with completely opaque pixels. I'll amend the commit.
That said --- is there any reason to do anything with the color values at all when blending two transparent pixels?
If not, then presumably it'd be more efficient to return a transparent white/black pixel rather than doing calculations on the color components.

I think blending is good, because it preserves the information in the image to neighboring pixels. For example, if you are super-sampling textures, white/black pixels can cause trouble.

Thanks for the quick reply!
I did some reading, and I don't think this kind of compositing is well defined when both pixels have zero alpha.
The referenced stack overflow answer and SVG spec, defines compositing in terms of "premultiplied" color components (as defined in the original Porter and Duff paper).
But since alpha is in the domain [0, 1],
let alpha_final = bg_a + fg_a - bg_a * fg_a;

must always be greater than the alpha of either pixel.
Which means that it can only be zero when both pixel's alpha are zero.
But in that case, the "premultiplication" step
let (bg_r_a, bg_g_a, bg_b_a) = (bg_r * bg_a, bg_g * bg_a, bg_b * bg_a);
let (fg_r_a, fg_g_a, fg_b_a) = (fg_r * fg_a, fg_g * fg_a, fg_b * fg_a);

will set all of the color components to zero.
Or am I missing something in my logic there?

Premultiplied color components do not conserve the color information in pixels when making them transparent. This is a different way of encoding images, which is not used here.

The code I quoted wasn't hypothetical, it's from the RGBA blend implementation and has the comment "We premultiply our channels bu their alpha, as this makes it easier to calculate".
Or are you suggesting that we switch to a different blending algorithm in the case when both source alphas are zero?

@lynaghk I messed this up, sorry.

I was not aware of premultiplying colors being used here. Not sure if this is the right algorithm.

@bvssvni No worries. I'm not too familiar w/ the tradeoffs for color blending and graphics stuff like this, so I'm not sure what the appropriate thing to do would be --- all I wanted from my changes was to avoid a panic.
If you do find another way through and want a hand, let me know.
But no worries if it'd be easier to just do the changes yourself too.

Another issue that there is no conversion to sRGB before blending. This algorithm is very opinionated. However, that is a design decision that could be dealt with in a later version.

Can you add a unit test that blends two transparent pixels and see what happens to the alpha channel? At least the resulting alpha channel should be zero.

I think it should simply return instead of setting the alpha to 1.0.
Closing in favor of https://github.com/PistonDevelopers/image/pull/759
Thanks for pointing this out!

Sorry for the delay on unit tests, I was away from the Internet.
Returning makes sense, given that the blend operation mutates. Thanks for the fix and your work on this library!


Add Heading field to Link blob organism

A heading field is now included in the Link blob organism.
Test

Go to http://localhost:8000/landing-page/ and review the bottom where the organism is.
Log into the admin and go to a page with the organism in a StreamField to find a new field (Heading)

@kave
@jimmynotjim

There's some weird visual layout issues that FEWDs will need to solve in another PR, but it looks like once you fix that 25-75 as 50-50 import this should be good.

:+1:


[New Port Request] libfins

Library name
libfins
Library description
Libfins is a multi platform library with routines to communicate with Omron PLCs over the FINS protocol.
Source repository URL
https://github.com/lammertb/libfins.git
Project homepage (if different from the source repository)
https://www.libfins.org
Anything else that is useful to know when adding (such as optional features the library may have that should be included)
Libfins is a multi platform library with routines to communicate with Omron PLCs over the FINS protocol. FINS is an inter-PLC protocol which allows hardware layer independent exchange of data between PLCs. Omron PLCs support the exchange of data over serial lines, TCP, UDP and bus systems. This library currently implements the FINS/TCP communication.
FINS is a client server protocol where a client sends requests to a remote server and processes their answer. The libfins library contains almost all of the publically documented functionality. The only functionality currently missing is using a PLC as gateway to execute Compoway/F and Modbus commands over SCU serial units. These features will be implemented and tested in the near future though.
The libfins library is licensed with the MIT license. This should give maximum flexibility when using the library in both open en closed source applications.

[ ] add cmak support (ing)
https://github.com/lammertb/libfins/pull/19
[ ] add vcpkg port

no uwp, no osx and no android 🤣

Given that the FINS communication is running in the background without a user interface and only needs a network connection, what would be the challenges to add uwp, osx and android support?

@lammertb What a surprise, glad to meet you :)

@lammertb You may get CI failure logs here: https://dev.azure.com/vcpkg/public/_build/results?buildId=92230&view=artifacts&pathAsName=false&type=publishedArtifacts

I am observing these logs.

@lammertb The error occurs on the following platforms:

x64-osx
x64-uwp
arm-uwp
arm-neon-android
arm64-android

But there are only three reasons:

Compile Error (x64-osx)
The compilation log is as follows
FAILED: CMakeFiles/fins.dir/src/fins_utils.c.o /Library/Developer/CommandLineTools/usr/bin/cc  -I/Users/vagrant/Data/buildtrees/fins/src/f27ee7c714-84fae67962.clean/include -fPIC -g -arch x86_64 -isysroot /Library/Developer/CommandLineTools/SDKs/MacOSX12.1.sdk -Wall -Wstrict-prototypes -Wshadow -Wpointer-arith -Wcast-qual -Wcast-align -Wwrite-strings -Wnested-externs -Wredundant-decls -Werror -funsigned-char -MD -MT CMakeFiles/fins.dir/src/fins_utils.c.o -MF CMakeFiles/fins.dir/src/fins_utils.c.o.d -o CMakeFiles/fins.dir/src/fins_utils.c.o -c /Users/vagrant/Data/buildtrees/fins/src/f27ee7c714-84fae67962.clean/src/fins_utils.c
/Users/vagrant/Data/buildtrees/fins/src/f27ee7c714-84fae67962.clean/src/fins_utils.c:522:2: error: implicit declaration of function 'usleep' is invalid in C99 [-Werror,-Wimplicit-function-declaration]
   usleep( 1000 * msec );
   ^
   1 error generated.

Accroding to Stackoverflow issue, we can use -std=gnu99 instead of -std=c99 to resolve the problem.

Compile Error (x64-uwp/arm-uwp)
The following is an excerpt from the compilation log
[1/68] C:\PROGRA~1\MICROS~1\2022\ENTERP~1\VC\Tools\MSVC\1436~1.325\bin\Hostx64\arm\cl.exe    -ID:\buildtrees\fins\src\f27ee7c714-84fae67962.clean\include /DWIN32 /D_WINDOWS /D_UNICODE /DUNICODE /DWINAPI_FAMILY=WINAPI_FAMILY_APP /D__WRL_NO_DEFAULT_LIB__ /nologo /Z7 /MP /GS /Gd /Gm- /W3 /WX- /Zc:wchar_t /Zc:inline /Zc:forScope /fp:precise /Oy- /EHsc  /utf-8  /D_DEBUG /MDd /Od /RTC1  -MDd -Ot -GT -DWINVER=0x0501 -D_WIN32_WINNT=0x0501 -volatile:iso -nologo -J -sdl -Wall -WX -wd4464 -wd4668 -wd4710 -wd4711 -wd4201 -wd4820 -Wv:18 /showIncludes /FoCMakeFiles\fins.dir\src\fins_04_02.c.obj /FdCMakeFiles\fins.dir\fins.pdb /FS -c D:\buildtrees\fins\src\f27ee7c714-84fae67962.clean\src\fins_04_02.c
FAILED: CMakeFiles/fins.dir/src/fins_04_02.c.obj
C:\PROGRA~1\MICROS~1\2022\ENTERP~1\VC\Tools\MSVC\1436~1.325\bin\Hostx64\arm\cl.exe    -ID:\buildtrees\fins\src\f27ee7c714-84fae67962.clean\include /DWIN32 /D_WINDOWS /D_UNICODE /DUNICODE /DWINAPI_FAMILY=WINAPI_FAMILY_APP /D__WRL_NO_DEFAULT_LIB__ /nologo /Z7 /MP /GS /Gd /Gm- /W3 /WX- /Zc:wchar_t /Zc:inline /Zc:forScope /fp:precise /Oy- /EHsc  /utf-8  /D_DEBUG /MDd /Od /RTC1  -MDd -Ot -GT -DWINVER=0x0501 -D_WIN32_WINNT=0x0501 -volatile:iso -nologo -J -sdl -Wall -WX -wd4464 -wd4668 -wd4710 -wd4711 -wd4201 -wd4820 -Wv:18 /showIncludes /FoCMakeFiles\fins.dir\src\fins_04_02.c.obj /FdCMakeFiles\fins.dir\fins.pdb /FS -c D:\buildtrees\fins\src\f27ee7c714-84fae67962.clean\src\fins_04_02.c
cl : Command line warning D9025 : overriding '/WX-' with '/WX'
C:\Program Files (x86)\Windows Kits\10\include\10.0.22621.0\shared\ws2def.h(360): error C2061: syntax error: identifier 'SOCKADDR_STORAGE'
C:\Program Files (x86)\Windows Kits\10\include\10.0.22621.0\shared\ws2def.h(360): error C2059: syntax error: ';'
C:\Program Files (x86)\Windows Kits\10\include\10.0.22621.0\shared\ws2def.h(361): error C2143: syntax error: missing '{' before '*'
C:\Program Files (x86)\Windows Kits\10\include\10.0.22621.0\shared\ws2def.h(361): warning C4431: missing type specifier - int assumed. Note: C no longer supports default-int
C:\Program Files (x86)\Windows Kits\10\include\10.0.22621.0\shared\ws2def.h(361): warning C4431: missing type specifier - int assumed. Note: C no longer supports default-int
C:\Program Files (x86)\Windows Kits\10\include\10.0.22621.0\shared\ws2def.h(361): warning C4218: nonstandard extension used: must specify at least a storage class or a type
C:\Program Files (x86)\Windows Kits\10\include\10.0.22621.0\shared\ws2ipdef.h(845): error C2061: syntax error: identifier 'SOCKADDR_STORAGE'
C:\Program Files (x86)\Windows Kits\10\include\10.0.22621.0\shared\ws2ipdef.h(846): error C2059: syntax error: '}'
C:\Program Files (x86)\Windows Kits\10\include\10.0.22621.0\shared\ws2ipdef.h(854): error C2061: syntax error: identifier 'SOCKADDR_STORAGE'
C:\Program Files (x86)\Windows Kits\10\include\10.0.22621.0\shared\ws2ipdef.h(855): error C2061: syntax error: identifier 'gsr_source'
C:\Program Files (x86)\Windows Kits\10\include\10.0.22621.0\shared\ws2ipdef.h(855): error C2059: syntax error: ';'
C:\Program Files (x86)\Windows Kits\10\include\10.0.22621.0\shared\ws2ipdef.h(856): error C2059: syntax error: '}'
C:\Program Files (x86)\Windows Kits\10\include\10.0.22621.0\shared\ws2ipdef.h(864): error C2061: syntax error: identifier 'SOCKADDR_STORAGE'
C:\Program Files (x86)\Windows Kits\10\include\10.0.22621.0\shared\ws2ipdef.h(867): error C2061: syntax error: identifier 'gf_slist'
C:\Program Files (x86)\Windows Kits\10\include\10.0.22621.0\shared\ws2ipdef.h(867): error C2059: syntax error: ';'
C:\Program Files (x86)\Windows Kits\10\include\10.0.22621.0\shared\ws2ipdef.h(867): error C2059: syntax error: '['
C:\Program Files (x86)\Windows Kits\10\include\10.0.22621.0\shared\ws2ipdef.h(868): error C2059: syntax error: '}'
C:\Program Files (x86)\Windows Kits\10\include\10.0.22621.0\um\ws2tcpip.h(832): warning C4431: missing type specifier - int assumed. Note: C no longer supports default-int
C:\Program Files (x86)\Windows Kits\10\include\10.0.22621.0\um\ws2tcpip.h(832): error C2143: syntax error: missing ')' before '*'
C:\Program Files (x86)\Windows Kits\10\include\10.0.22621.0\um\ws2tcpip.h(832): error C2143: syntax error: missing '{' before '*'
C:\Program Files (x86)\Windows Kits\10\include\10.0.22621.0\um\ws2tcpip.h(833): error C2059: syntax error: ')'
C:\Program Files (x86)\Windows Kits\10\include\10.0.22621.0\um\ws2tcpip.h(834): warning C4431: missing type specifier - int assumed. Note: C no longer supports default-int
C:\Program Files (x86)\Windows Kits\10\include\10.0.22621.0\um\ws2tcpip.h(834): error C2054: expected '(' to follow 'SourceList'
C:\Program Files (x86)\Windows Kits\10\include\10.0.22621.0\um\ws2tcpip.h(880): error C2143: syntax error: missing ')' before '*'
C:\Program Files (x86)\Windows Kits\10\include\10.0.22621.0\um\ws2tcpip.h(880): error C2081: 'SOCKADDR_STORAGE': name in formal parameter list illegal
C:\Program Files (x86)\Windows Kits\10\include\10.0.22621.0\um\ws2tcpip.h(880): error C2143: syntax error: missing '{' before '*'
C:\Program Files (x86)\Windows Kits\10\include\10.0.22621.0\um\ws2tcpip.h(881): error C2059: syntax error: ')'
C:\Program Files (x86)\Windows Kits\10\include\10.0.22621.0\um\ws2tcpip.h(882): warning C4431: missing type specifier - int assumed. Note: C no longer supports default-int
C:\Program Files (x86)\Windows Kits\10\include\10.0.22621.0\um\ws2tcpip.h(882): error C2054: expected '(' to follow 'SourceList'

At present, I don't have time to solve this problem, but the initial guess may be that the macro definition and the system header file have collided.

CI host network problem
The following is an excerpt from the compilation log
# arm-neon-andorid

ERROR: The command failed with an unexpected error. Here is the traceback:
ERROR: HTTPSConnectionPool(host='andcontainerregistry.azurecr.io', port=443): Max retries exceeded with url: /oauth2/exchange (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x7f8ec176bd60>: Failed to establish a new connection: [Errno 111] Connection refused'))
Traceback (most recent call last):
File "/opt/az/lib/python3.10/site-packages/urllib3/connection.py", line 174, in _new_conn
     conn = connection.create_connection
File "/opt/az/lib/python3.10/site-packages/urllib3/util/connection.py", line 95, in create_connection
     raise err
File "/opt/az/lib/python3.10/site-packages/urllib3/util/connection.py", line 85, in create_connection
     sock.connect(sa)
ConnectionRefusedError: [Errno 111] Connection refused

# arm64-andorid

WARNING: Uppercase characters are detected in the registry name. When using its server url in docker commands, to avoid authentication errors, use all lowercase.
WARNING: Error response from daemon: Get "https://andcontainerregistry.azurecr.io/v2/": dial tcp <IP_ADDRESS>:443: connect: connection refused
ERROR: Login failed.

Accroding to Stackoverflow issue, we can use -std=gnu99 instead of -std=c99 to resolve the problem.

Given Apple clang, I wouldn't bet on gnu99. But there is the mention of _XOPEN_SOURCE.

3. CI host network problem

The android build seems to be scripted in a different way.
All network access should be in the portfile via maintainer functions (implying proper asset caching), and dependencies should be acquired via vcpkg.

Better to fix the problem where it occurs and not playing around with compiler settings until compilation accidentally succeeds.
The compiler stops at the usleep() call because the proper include file is not loaded. I have now included <unistd.h> for OS-X compilation. That should fix the usleep issue. I have no OS-X system to test it, so maybe the macro OSX is not the right one but if another macro is needed, it is pretty easy to fix.

Regarding the x63-uwp /arm-uwp compiler problem, I have a VS2022 envirnonment available here. Will spend some a few spare hours on it today to try to trace the problem and fix it.

The uwp compilation issue is not unique to this library, looking at https://github.com/godotengine/godot/issues/16830
Currently scanning that issue to find a solution.

Removed the macro definition upstream https://github.com/lammertb/libfins/commit/03a9afaf5bff1134dd0b8da729a2cbf62b4908c3

Had a few happy hours with Visual Studio hacking the code to get it smoothly running without a single issue for 32 and 64 bit targets.

Added a Visual Studio 2022 project file to the repository.
Suppression of non-maskable warnings is now done with compiler pragmas in the source
Code compiles for Debug and Release for x86 and x64 without warning with warning setting -Wall
All warning suppression options in the CMake file have been deleted
Reduced warning level to -W3 for the CMake and VS Release builds
Debug mode in the Visual Studio project is still -Wall to catch problems while developing
Removed the "treat warning as errors" flag for CMake to let it happily compile in slightly different environments

The code compiles now without warnings on

Windows x86 and x64
Ubuntu 20.04 and 22.04
MacOS 11,12 and 13
Android ARM, ARM64, x86 and x64.

MacOS and Android builds use MS-build as I don't have enough knowledge of CMake yet to add cross-platform compilation.
MS-Build configurations and YAML files are all included in the repository and CI Github Actions for all platforms have been set up.

@lammertb It would be good to add a CMake setup to your CI. It can generate msbuild files, and it supports multiple platforms and and cross builds. For vcpkg, it is better to use the same build system for all platforms, to ensure the same set of artifacts.

@lammertb Today, I directly incorporated the current cmake project into the CI workflow and found that it can compile normally. Previous compilation errors may have been caused by compilation flags. In other words, perhaps we don't need to add Msbuild. I'm very sorry for wasting your valuable time. 🥺
Perhaps we can delete the msbuild file sometime.

PR #32795 is merged, nice work, thanks you all.


 Enable gulp task detection..."
See error "Unable to write gulp.autoDetect to Workspace Settings. This setting can be written only into User settings."

I can repro. Happens when the configuration scope is application. Weird because I don't think this has changed recently 🤔
https://github.com/microsoft/vscode/blob/b44d1fb989750ebd168ca7b05f373cd0e264906c/src/vs/workbench/services/configuration/common/configurationEditing.ts#L522-L524

Doesn't reproduce in stable, so seems like a regression from https://github.com/microsoft/vscode/issues/175821

Does not reproduce with grunt

grunt and gulp have the same scope here, so not sure what is going on

Oh, I think that's because there aren't any grunt tasks

Reverting that PR does not fix it

gulp task detection is working for me now. @alexr00 pls lmk if you can still repro


Adds mocking/testing example

I used the library for a project and tried to come up with unit tests for it by mocking the name server tho I failed. It would be nice to have one example for that. I could come up with a PR for that if someone could help me to understand why my approach failed.

this might help: https://github.com/coredns/coredns/tree/master/plugin/pkg/dnstest


Modified the state of the themes dropdown

fixed #16 Modified the state of the themes from outlined to filled for default view so that icons does not change on search

I have fixed the issue by adding a new theme in #24 . So, this is creating conflict issues.
But now due to approved 24 PR the issue is fixed. We can close this now.

Please resolve conflicts!

Sir I have pushed the commits for this in the merged PR so I am closing this issue as it has been resolved with https://github.com/EOS-uiux-Solutions/eos-icons-adobe-xd/pull/24.


Fix Mistake in Resource Bin Packing for Extended Resources

An "and" is missing and this is grammatically incorrect.
Feel free to keep this PR pending until after Hacktoberfest. I do not wish this to be grouped in with those type of contributors.

CLA signed!

Kubernetes is not participating in Hacktoberfest.
We do of course aim to include everyone and welcome everyone who wants to contribute to Kubernetes.
/kind cleanup
/lgtm
/approve


Unable to list the connected table's fields in has many relationship

I'm trying to connect two tables with @connection(name: "ConnectionName"). But I can't able to list the connected table's data in a single list query. In get query I can get all the fields.
Example:
Two tables - Authors and Books
One Author has many Books
I have the below code in my schema.graphql file
type Author
@model(subscriptions: null)
@auth(rules: [{
    allow: owner,
    operations: [read, create, update, delete]}]) {
  id: ID!
  authorName: String!
  books: [Book] @connection(name: "Books")
}

type Book
@model(subscriptions: null) {
  id: ID!
  title: string!
  authorId: Author @connection(name: "Books")
}

From the above schema the queries and mutations will be generated automatically as below.
getAuthor Query
export const getAuthor = /* GraphQL */ `
  query GetAuthor($id: ID!) {
    getAuthor(id: $id) {
      id
      authorName
      books {
        items {
          id
          title
          createdAt
          updatedAt
        }
        nextToken
      }
      createdAt
      updatedAt
    }
  }
`;

listAuthor Query
export const listAuthors = /* GraphQL */ `
  query LisAuthors(
    $filter: ModelAuthorFilterInput
    $limit: Int
    $nextToken: String
  ) {
    listAuthors(filter: $filter, limit: $limit, nextToken: $nextToken) {
      items {
        id
        authorName
        books {
          nextToken
        }
        createdAt
        updatedAt
      }
      nextToken
    }
  }
`;

Here my expectation is to get the list of authors along with their books in a single query which is listAuthors. But I can get only nextToken from the books table in that listAuthors query.

Hi @Priyangacdi if this is a new project, I would highly recommend you use the new transformer which uses @hasMany and @belongsTo to accomplish the connection you are trying to set up: https://docs.amplify.aws/cli/graphql/data-modeling/#setup-relationships-between-models
If you need to use the transformer v1, see the docs here on how to set up a has-many <=> belongs-to relationship: https://docs.amplify.aws/cli-legacy/graphql-transformer/connection/#belongs-to


Updating to Vapor 3.0

This PR aims to:

[x] Update SwiftyBot to Vapor 3.0
[x] Make a complete refactoring to easily allow adding other platforms
[x] Add a full suite of tests
[x] Add sender name for Messenger platform
[x] Add support to Sender Actions for Messenger platform
[x] Set a custom Messenger profile at startup
[x] Add support to Google Assistant bot platform
[ ] Add multi language support

Is it safe to test Developer for Vapor 3 ?

Currently, there is a coverage of 98%, but I didn't try it in production yet.
I'm waiting for the new Facebook Review approval to release it in production, while I'm integrating a new bot platform.
If you want I can update this PR when I will release it in production, to be sure that it fully works 😉
However, if you just want to test it, you can, let me know if it works 😄

I'll give it a "manual testing" shot if we know that it works. Since the guide is for 2.0, anything specific I should without for?
ps: thanks for the great refactoring... it's neat!

Thanks! The refactoring was needed to be able to add new platforms easily.
I'm also writing a new tutorial on how to create a bot for the new supported platform, but it will cover only the new platform.
Maybe I should update the first 2 tutorials for Telegram and Facebook Messenger as well
The 3.0 version covers the same 2.0 things and adds some little things to Facebook Messenger like the sender name, you will find the list on top of this PR

A lot of old Hound CI's comments has been removed

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 36

Complexity increasing per file
==============================
- generate_linux_tests.rb  8

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 36

Complexity increasing per file
==============================
- generate_linux_tests.rb  8

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 54

Complexity increasing per file
==============================
- generate_linux_tests.rb  9

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 54

Complexity increasing per file
==============================
- generate_linux_tests.rb  9

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 54

Complexity increasing per file
==============================
- generate_linux_tests.rb  9

See the complete overview on Codacy