[🐛 Bug Report]: ImportError: cannot import name 'Aggregation' from 'neuralogic.core.settings'

Describe the bug
When trying to build the model from a template, using the PyG backend, it throws the following error:
Traceback (most recent call last):
  File "<input>", line 1, in <module>
    model = template.build(Backend.PYG, settings)
  File "/usr/local/lib/python3.8/site-packages/neuralogic/core/template.py", line 124, in build
    return get_neuralogic_layer(backend)(self.module_list)
  File "/usr/local/lib/python3.8/site-packages/neuralogic/nn/__init__.py", line 19, in get_neuralogic_layer
    from neuralogic.nn.native.pyg import NeuraLogic
  File "/usr/local/lib/python3.8/site-packages/neuralogic/nn/native/pyg.py", line 4, in <module>
    from neuralogic.core.settings import Activation, Aggregation
ImportError: cannot import name 'Aggregation' from 'neuralogic.core.settings' (/usr/local/lib/python3.8/site-packages/neuralogic/core/settings/__init__.py)

Steps to reproduce the behavior
This is the script that is being used:
from neuralogic.core import Relation, Dataset
from neuralogic.core import Template
from neuralogic.utils.templates import GCNConv, TemplateList
from neuralogic.core import Backend
from neuralogic.core import Settings, Optimizer

dataset = Dataset()
dataset.add_example([
    Relation.edge(1, 2), Relation.edge(2, 1), Relation.edge(1, 3),
    Relation.edge(3, 1), Relation.edge(2, 3), Relation.edge(3, 2),

    Relation.feature(1)[0],
    Relation.feature(2)[1],
    Relation.feature(3)[-1],
])

template = Template(module_list=TemplateList([
    GCNConv(in_channels=5, out_channels=5),
    GCNConv(in_channels=5, out_channels=1),
]))

settings = Settings(learning_rate=0.01, optimizer=Optimizer.SGD, epochs=100)

model = template.build(Backend.PYG, settings)

Expected behavior
The expected behaviour would be the creation of the model from the specified template.
Environment
macOS Monterey Version 12.2.1 (21D62)
python 3.8.12
neuralogic 0.1.3
torch-geometric 2.0.3
Additional context
No response

Hello, thank you for the bug report and for trying out PyNeuraLogic!
I released a new version (0.1.4) with the bug being fixed together with some other features/bug fixes.
Thanks to your issue I noticed a few discrepancies in our documentation (fixed), namely the dimension of in_channels of the first GCNConv module (it should be in_channels=1). You will also have to rename all Relation.feature to Relation.node_feature or specify feature_name as "feature" while instantiating TemplateList (it was outdated in the documentation).
I also noticed in your code that you are using a dataset in relational form for the PyG backend. PyNeuraLogic only supports using PyG backend with a tensor representation (features and edge_index being torch tensor and not python list).
I would recommend you to switch to the Java backend, as it is the primary and the most polished backend of PyNeuraLogic.
This way, you can fully utilize all possibilities of PyNeuraLogic.
If you decide to go with a different backend (other than PyG), don't forget to add some queries to the dataset; otherwise, PyNeuraLogic will not know labels and outputs. We omitted queries in the documentation for clarity.
You can also check out some of our example jupyter notebooks that might help with using PyNeuraLogic. This framework is quite different from other tensor-based frameworks, so it might take a while to get used to it.
Thanks again for the report. If you need any help or have some other issues, do not hesitate to open a topic in discussions or an issue.

Thanks for the fast reply and the corrections to my example. I have no problem with using Java backend, so I'll proceed as you mentioned.


Optimise build scripts

There's a couple of things we can do to make the build scripts run a little bit faster:

with dotnet core sdk 2.0 restore is now implicit during build. So, we can remove the explicit restore step from the build
when we run tests it will currently rebuild the project. Given that the tests depend on the compile task we don't need this, and we can prevent it by passing in the --no-build argument to the test runner.

However, a gotcha here is that the unit tests need to be built with debug symbols for the coverage reports to work. So, shouldn't pass --no-build for unit tests, as we need to explicitly build debug versions of the assemblies that the tests are covering.

@binarymash shall we close this one?

Yep.


Navbar vertical align

It seems we can improve vertical align definition of main navbar.

i did this in the https://github.com/rockandror/startup-suspender-rails4/issues/35


Adding argument to decide Cibyl's output format.

Two formats are supported:

Raw: Just text
Colored: Same as raw, but colored for easier read.

I'm not sure that it's a good idea to have model-related logic outside of the model classes. One, because models are likely to change as features are added. And two, because the model hierarchy will be dynamic depending on the configuration and user input. I think that these Printer objects will need to be too complex to take into account the possible configurations.

At the moment all I have done is extract the 'str' method for each model and join it all together in these classes. There is no new logic in here, so complexity is still the same as it was before. The classes' purpose is to generate a textual representation of a certain model, they are not meant to deal with how the models are generated. The only user input at work here is verbosity.
In the end, the printer classes are meant to be used by the CLI once all work is done and output needs to be generated. By defining an interface it should not be difficult to change the output's format as desired.

What I meant by complexity is that these Printer classes would partially overlap with the Publisher in responsibility, but that would not be a problem, the Publisher could just rely on them.
The point about the models is that the Printers will need to know the different possibilities. For example, once we decide on a model for Zuul systems it will need to know that it has Pipelines (or some alternative model).

Sure, I have not entered that yet as we still need to define how that will be, but this will need to be extended for any oncoming models.

I've had an idea for this change and I would like to know what do you think @jsanemet. What if instead of moving the str methods of the models to the printer, we keep them in the models but passing the ColorProvider object that you created. In the printer objects we store the name of the method to call (so __str__ for current printing, in the future we might want to_jsonor similar) and then we call them as we are doing until now from the environment class. This way we need less changes and keep a "single point of failure", that is, we are sure that if a the text representation of a model is wrong, we need to look at the model class.

Yes, from what I have gathered there are two approaches for this problem:

Keep text generation distributed and give each model the responsibility of printing itself.
Centralize text generation on the Printer sub-classes.

This is what I see for each of them:
Option 1:

Pros:

No need to go through the code and replace all calls to 'str'.
If there are mistakes on the representation of a model, you know that it must be on the model's class.
No need to redo the test cases.
My complaints about this option can be solved with a 'to_string()' method instead of 'str'.

Cons:

I am not a fan on adding parameters to the 'str' function. It forces you to make a explicit call, which is not the main purpose of it.
Overuse of the 'str' method. The idea of this function is to have a smallish description of the class, something you can see on the debugger.
Selecting between formats will not be as straight forward.
The model takes a responsibility when it does not really need to.
Option 2:

Pros:

Choosing output format is really straightforward.
All code related to text representation is stored on the same class. Any bug on this regard will be on the correspondent class.
Simplifies models by removing the need to handle text representation.

Cons:

Printer classes may increase in size and be difficult to follow. This can be solved though by dividing the problem is smaller chunks.
Most likely less flexible than the other approach, but that depends on requirements I am not sure about yet.
More first-time work for having to replace 'str' everywhere on the project.

I am more a fan of option 2, as I like to divide functions as much as possible, but I can see the other option as something to consider. What I would do is present these two solutions and have the team decide. I have no problem implementing whichever.

+1 for option #1
And how about chaning Color class to prnit regular and coloured text based on command line agr? That will eliminate the need in printers at all. Like this:
style = ''
style = input("type text or coloured:")
class Colors:
"""This is an implementation of a color class"""
BLUE = '\033[94m'
GREEN = '\033[92m'
RED = '\033[91m'
YELLOW = '\033[93m'
CLOSE = '\033[0m'
BOLD = '\033[1m'
UNDERLINE = '\033[4m'

def red(text):
    if (style == 'coloured'):
       return f"{Colors.RED}{Colors.BOLD}{text}{Colors.CLOSE}"
    return text

def green(text):
    if (style == 'coloured'):
       return f"{Colors.GREEN}{Colors.BOLD}{text}{Colors.CLOSE}"
    return text

class MyClass:
x = 0
y = ""
def __init__(self, anyNumber, anyString):
    self.x = anyNumber
    self.y = anyString

def __str__ (self):
    return 'MyClass(x=' + Colors.red(str(self.x)) + ' ,y=' + self.y + ')'

myObject = MyClass(12345, "Hello")
print(myObject)

@dsariel That is another good option. I believe it would be the simplest too as there is no need to modify the printing mechanism either.
We need to put all these options on the table, see what we want out of the printing routines and choose one to continue with.

I would go with option 1, and using methods like to_string, to_json in each model.

Option number 4: To have an abstract method for each format on the Model class. That way the str is not overloaded and everything stays self-contained.

I vote for option #2
I think models shouldn't have the responsibility of printing themselves in whatever format the user selects. The str works well for colored  and plain text, but I don't think it will be as straightforward for JSON or XML.
For me, separate printer classes are the way to go for this

In the end this turned out to be a huge change, but it covers everything. Both the CI and OS printings is here.


[extractor/youtube] Retry manifest refresh to avoid ending download early when live stream is temporarily offline

IMPORTANT: PRs without the template will be CLOSED
Description of your pull request and other information

Currently when downloading a YouTube live stream, if it goes offline (e.g. due to a problem on the uploader's end), yt-dlp prints a warning like:
WARNING: [youtube] Cannot find refreshed manifest for format 140; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
WARNING: [youtube] Cannot find refreshed manifest for format 299; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U

then considers the live stream over, proceeding to muxing.
This change makes the process of finding a refreshed manifest subject to --extractor-retries and --retry-sleep extractor:, so that downloading can continue without intervention as long as the live stream is back online within extractor-retries * retry-sleep seconds.
I tested this by starting a scheduled live stream, starting yt-dlp, then disconnecting the streaming software and confirming the retry messages from yt-dlp, and finally reconnecting the streaming software and observing that yt-dlp resumes downloading. I've also confirmed that the true end of the live stream is detected and yt-dlp proceeds to muxing fine.
Fixes #
Template

Before submitting a pull request make sure you have:

[x] At least skimmed through contributing guidelines including yt-dlp coding conventions
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8 and ran relevant tests

In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[x] Fix or improvement to an extractor (Make sure to add/update tests)
[ ] New extractor (Piracy websites will not be accepted)
[ ] Core bug fix/improvement
[ ] New feature (It is strongly recommended to open an issue first)

cc @Lesmiscore


Simplify installation instructions for Ubuntu

I just switched to a new Ubuntu-PC and while installing traffico I noticed that the installation process can be simplified:
Since Ubuntu 14.04 LTS, sfnt2woff is in the Ubuntu package repositories as part of the woff-tools package. Therefore it can simply be installed from packages.

Awesome!


The intersection logic of Compare has problems.

from datacompy import Compare
import pandas as pd

df1 = pd.DataFrame(dict(a=[0],b=[1]))
df2 = pd.DataFrame(dict(a=[0,0],b=[0,1]))

comp=Compare(df1, df2, join_columns=["a"])
print(comp.report())

We got this
Number of rows in common: 1
Number of rows in df1 but not in df2: 0
Number of rows in df2 but not in df1: 1

Number of rows with some compared columns unequal: 1
Number of rows with all compared columns equal: 0

It didn't capture the intersection of df1 and df2.
This is because the outer join https://github.com/capitalone/datacompy/blob/1de2f37f75064422be5f82cd8f35a0859f0fb8a0/datacompy/core.py#L272 is on a and a temp column, in df1 the first row's temp value is 0, on df2 the second row's temp value is 1. So with this implementation. we are not able to find the true intersection. And notice, sorting the two dataframes doesn't solve the problem, in this example, they are already fully sorted.
The solution may be joining on all columns, but that is expensive, so what other ways can we use to solve this issue?

I think about it twice, this is even harder than I thought.
The big challenge is the floating columns. For example 0.99 and 1 can be considered same with certain atol and rtol. So the question is should we even compute the intersections? It's so expensive to make it right.

@goodwanghan This might actually be covered in the docs: https://capitalone.github.io/datacompy/pandas_usage.html#duplicate-rows
This is sort of a known limitation and if you have a lot of dups you would really need to join on more fields to get more accurate results. I think the example in the docs sort of outlines what you have above and is the expected behaviour.

Closing this. @goodwanghan feel free to reopen but I think this is known and documented.


include skin-folder in bower release

I'd like to see the skin folder included in the bower package (right now this folder is on the ignore-list in bower.json).
The reason is, I want to build a tool where the user can select which kind of navigation buttons shall be used. The required CSS and HTML code could be easily extracted with a script from the files in the skin-folder (it's nicely enclosed in a comment like this: <!--#region ... -->). Obviously, this is only possible if the skin folder is actually installed by bower.
To resolve this issue, it would just be neccessary to delete bower.json#L75.

To keep the bower package light weight, the 'skin' folder is ignored. As you will build a tool, you can add the skin folder by yourself.

Well, of course I could add it myself, but that means updating this dependency separately, since bower cannot be used.
Could you please reconsider including it, the folder is not even 300 KB, so the package will still be small if you include it.

updated bower.json and package.json.


[WIP] re-introduce vendor to satisfy reproducible builds requirement

re-introduce vendor to satisfy reproducible builds requirement
Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)
[ ] Changelog has been updated and is aligned with our changelog requirements

Fixes #3509

/run-e2e internal*

Can't approve from mobile, > 10k files changed and more than 3kk rows. Is it okey?

Didn't you read all of them?

I wasn't able to convince mockgen to properly generate mocks from vendor directory :(
Hava tried several things, something like this:
pkg/mock/mock_scale/mock_interfaces.go: $(shell pwd)/vendor/k8s.io/client-go/scale/interfaces.go
	$(MOCKGEN) -destination=$@ -package=mock_scale -source=$^ -build_flags=-mod=vendor
pkg/mock/mock_client/mock_interfaces.go: $(shell pwd)/vendor/sigs.k8s.io/controller-runtime/pkg/client/interfaces.go
	$(MOCKGEN) -destination=$@ -package=mock_client -source=$^ -build_flags=-mod=vendor

ends with files like this (note the change in import):
diff --git a/pkg/mock/mock_scale/mock_interfaces.go b/pkg/mock/mock_scale/mock_interfaces.go
index 45c07456..22ffa192 100644
--- a/pkg/mock/mock_scale/mock_interfaces.go
+++ b/pkg/mock/mock_scale/mock_interfaces.go
@@ -1,5 +1,5 @@
 // Code generated by MockGen. DO NOT EDIT.
-// Source<EMAIL_ADDRESS>+// Source: /Users/zroubali/go/src/github.com/kedacore/keda/vendor/k8s.io/client-go/scale/interfaces.go

 // Package mock_scale is a generated GoMock package.
 package mock_scale
@@ -9,11 +9,11 @@ import (
        reflect "reflect"

        gomock "github.com/golang/mock/gomock"
+       scale "github.com/kedacore/keda/v2/vendor/k8s.io/client-go/scale"
        v1 "k8s.io/api/autoscaling/v1"
        v10 "k8s.io/apimachinery/pkg/apis/meta/v1"
        schema "k8s.io/apimachinery/pkg/runtime/schema"
        types "k8s.io/apimachinery/pkg/types"
-       scale "k8s.io/client-go/scale"
 )

 // MockScalesGetter is a mock of ScalesGetter interface.

the change ulitmately leads in the failed docker build:
#21 15.44 /workspace/bin/mockgen -destination=pkg/mock/mock_scale/mock_interfaces.go -package=mock_scale -source=/workspace/vendor/k8s.io/client-go/scale/interfaces.go -build_flags=-mod=vendor
#21 15.84 /workspace/bin/mockgen -destination=pkg/mock/mock_client/mock_interfaces.go -package=mock_client -source=/workspace/vendor/sigs.k8s.io/controller-runtime/pkg/client/interfaces.go -build_flags=-mod=vendor
#21 16.21 /workspace/bin/controller-gen object:headerFile="hack/boilerplate.go.txt" paths="./..."
#21 70.61 pkg/mock/mock_client/mock_interfaces.go:12:2: github.com/kedacore/keda/v2/vendor/sigs.k8s.io/controller-runtime/pkg/client must be imported as sigs.k8s.io/controller-runtime/pkg/client
#21 70.61 pkg/mock/mock_scale/mock_interfaces.go:12:2: github.com/kedacore/keda/v2/vendor/k8s.io/client-go/scale must be imported as k8s.io/client-go/scale
#21 70.61 Error: not all generators ran successfully
#21 70.61 run `controller-gen object:headerFile=hack/boilerplate.go.txt paths=./... -w` to see all available markers, or `controller-gen object:headerFile=hack/boilerplate.go.txt paths=./... -h` for usage
#21 70.73 make: *** [Makefile:124: generate] Error 1
------
executor failed running [/bin/sh -c VERSION=${BUILD_VERSION} GIT_COMMIT=${GIT_COMMIT} GIT_VERSION=${GIT_VERSION} TARGET_OS=$TARGETOS ARCH=$TARGETARCH make manager]: exit code: 2

💩 💩 💩 💩 💩 💩 💩 💩

/run-e2e internal*

/run-e2e internal*


Do not leak the entire request object when an error occurs.

This is causing secrets to leak to HB.
https://app.honeybadger.io/projects/1112/faults/38996292#notice-summary

👍


New blog post: Tips for Memorizing Scripture

@michsco, this blog post is not appearing for me locally. I've triple-checked the Markdown file for formatting errors, but it looks correct.

@michsco, this blog post is not appearing for me locally. I've triple-checked the Markdown file for formatting errors, but it looks correct.

It took me a while to figure out what was wrong here, @jameslex and @dwwarnock, but it was missing the *.md file extension. You may have already fixed this since it appears it was actually committed properly.
I've been traveling, so wasn't able to look at this until now.

@jameslex -- I am closing this pull request since it is likely no longer needed.


undefined closeRow() from SwipeListView

Hi.. im try to implement closeRow() when user click button from renderHiddenItem property.
when im try to access closeRow() from rowMap, its look like undefined.
im try with onPress={ _ => rowMap[rowData.item.key].closeRow()}
Any idea to solve this? Thanks
      <SwipeListView
        useFlatList
        data={notificationList}
        disableLeftSwipe
        style={{flexGrow: 1, backgroundColor: 'white'}}
        leftOpenValue={55}
        renderItem={(data, rowMap) => this._renderNotificationItem(data.item)}
        renderHiddenItem={(rowData, rowMap) => (
          <TouchableOpacity onPress={() => {this.readThisNotif(rowData)}} style={styles.swipeButton}>
            <MaterialIcons name={`markunread`} size={28} color={'#fff'} />
          </TouchableOpacity>
        )}
      />

@pisangGoreng are you calling closeRow from within readThisNotif? Because I don't see where you are passing the rowMap to readThisNotif?
Also what is undefined? The rowMap or something in rowData?
It would be super helpful if you could post the code for readThisNotif and the redbox error you're seeing as well.
Thanks!

@pisangGoreng Sorry man I'm not sure what's going on.. I've plugged in the code you're using in renderHiddenItem into my example and it's working just fine.
Can I see what notificationList looks like? That might help figure out the issue.
Thanks!

Same happening to me. rowMap[rowData.item.key] is undefined

@pgonzalez-santiago @pisangGoreng I think my response to another issue may help you here. Have a look at this and see if it solves your problems. If not please let me know:
https://github.com/jemise111/react-native-swipe-list-view/issues/220#issuecomment-398678246

Closing as inactive, please reopen if you still have issues

rowMap[rowData.item.key] gives undefined because there is no any property called "key" in your item object.
You can use rowMap[rowData.item.id] instead as my item object consist of unique "id" in it.
Worked for me.


Feature request: Allow more concise options

The current format eats up too much space for -n VALUE, --name VALUE type options.
For example:
usage:
    myprog create [options]
    myprog delete [options]
    myprog search [options]

options:
    -n NAME, --name NAME            name
    -a ADDRESS, --address ADDRESS   address
    -p PHONE, --phone PHONE         phone

could be written more concisely as:
usage:
    myprog create [options]
    myprog delete [options]
    myprog search [options]

options:
    -n/--name NAME         name
    -a/--address ADDRESS   address
    -p/--phone PHONE       phone

Thanks!

Any chance this issue is going to be considered?


Can't get an AssetRenditionDispatcher to pick up an alternative config

Describe the bug
I'm working with Asset Share Commons and I'm aiming to use an alternative rendition mapping to change how thumbnails are handled on a search page.
The URLs rendered by the search component resolve to the AssetRenditionServlet, which uses AssetRenditionDispatchers to translate the incoming parameters. There are a few underlying implementations and the one I'm trying to tweak is InternalRedirectRenditionDispatcherImpl.
I've had a look at the implementation and it seems I could achieve this by providing an alternative configuration. This is the one that comes OOTB.
<jcr:root xmlns:sling="http://sling.apache.org/jcr/sling/1.0" xmlns:jcr="http://www.jcp.org/jcr/1.0"
          jcr:primaryType="sling:OsgiConfig"
          service.ranking="-9001"
          label="Search Results"
          name="asset-share-commons-search-results"
          hidden="{Boolean}true"
          rendition.mappings="[card=${asset.path}.thumb.319.319.jpg,list=${asset.path}.thumb.140.140.jpg]"
/>

What I want to achieve is remap the card rendition to something else.
I've created a sling:OsgiConfig node under /apps that looks roughly like this:
<jcr:root xmlns:sling="http://sling.apache.org/jcr/sling/1.0" xmlns:jcr="http://www.jcp.org/jcr/1.0"
          jcr:primaryType="sling:OsgiConfig"
          hidden="true"
          label="My Search Results"
          name="my-search-results"
          rendition.mappings="[card=${asset.path}.preview.400.400.jpg]"
          service.ranking="-8000"/>

and I can see the configuration in the Felix console. However, as the Servlet attempts to load the config, the order is not what I would expect it to be.
No matter how high the ranking I set in the node, the instance of InternalRedirectRenditionDispatcherImpl backed by this config ends up second (when using negative numbers with lower module ) or last (when using positive numbers) in the list at this line. The configuration is read but due to its position in the list, it's ignored by the Servlet (see screenshot below).
Is this a bug or am I misunderstanding the way this should work? Just looking at the underlying ASC and Sling code, specifically the AssetRenditionsImpl and RankedServices class, it seems a high ranking should allow my config to be picked up as the first one.
Environment

AEM Version: <IP_ADDRESS>
Asset Share Commons Version: 1.8.0
Author and Publish

To Reproduce
Steps to reproduce the behavior:

Provide an alternative config for InternalRedirectRenditionDispatcherImpl
Open an image using a URL resolving to the AssetRenditionServlet, e.g. http://localhost:4503/content/dam/foo/en/audio/2021/3/animal_cat_meow.mp3.renditions/card/asset.rendition

Expected behavior
The suffix allows the servlet to translate the URL to one that matches the rendition mapping of the highest-ranking service.
Screenshots

Additional context
The reason I'm trying to achieve this is to override the way the default image placeholder is handled for assets that can't have renditions generated automatically. I want to limit this change in behaviour to assets (no affecting pages), only affect assets on pages based on Asset Share Commons (therefore not the Sites console or any other applications installed on the same AEM instance).
ASC ultimately falls back to com.day.cq.wcm.core.impl.servlets.ThumbnailServlet which appears to use com.day.cq.dam.commons.thumbnail.provider.AssetThumbnailProvider. These seem to remember the days of Day and they aren't configurable.
I also wouldn't want to add an extra file for every asset there is. I know it's been suggested in https://github.com/adobe/asset-share-commons/issues/526 but it sounds like it would be a pain to update the image if necessary.
I suppose a way I could hack around this would be to have my CRX package overwrite /apps/asset-share-commons/config/com.adobe.aem.commons.assetshare.content.renditions.impl.dispatchers.InternalRedirectRenditionDispatcherImpl-SearchResults.xml but that would cause tricky side effects depending on which package gets installed in which order; an ops nightmare.

Let me check this out - what you're doing looks right, and i swore i helped someone do this same thing a month ago :) ! If ranking isnt supported then def gotta fix it

Thanks @davidjgonzalez!

ok! .. that was a weird one :)
So the services are ranked using RankedServices Sling API.
RankedServices requires service.ranking to be persisted as an Integer type (it cant be a string that's coerce-able to an Integer, it must be declared AS an Integer.)
So, right now if you go into ConfigMgr and use the WebUI to create a new one, it saves it as a String, and Strings are all evaluated as 0.
So, basically, in the interim, you can make your OSGi config files declare your values as service.ranking:Integer with a > 0 value. My PR should otherwise generally fix it - however if you'd want to update any configs youve created/updated to ensure the data type is explicitly an Integer.

Contains the fix:
asset-share-commons.all-2.0.5-SNAPSHOT.zip

Good one indeed :)
Thanks for looking into this. I'll need to port this fix into ASC 1.8 as that's what my client is using and a major upgrade is off the table for the time being. Do you reckon those configuration changes could be ported 1:1?
What's the process to raise a PR against an older version?

here pr for 1.x  https://github.com/adobe/asset-share-commons/pull/633 with a snapshot build

Thanks @davidjgonzalez. I'll install the patch locally and let you know if it works.
However, in order to get this applied properly, I'll probably need an official release. My client is a financial institution and while I could easily build and install a working package, I may have a hard time getting it added as a dependency or otherwise installed in production.

So, basically, in the interim, you can make your OSGi config files declare your values as service.ranking:Integer with a > 0 value. My PR should otherwise generally fix it - however if you'd want to update any configs youve created/updated to ensure the data type is explicitly an Integer.

I've noticed that Integer doesn't seem to work when installing the configuration through a CRX package.
Using Integer in the JCR property value causes the node creation to fail
<?xml version="1.0" encoding="UTF-8"?>
<jcr:root xmlns:sling="http://sling.apache.org/jcr/sling/1.0" xmlns:jcr="http://www.jcp.org/jcr/1.0"
          jcr:primaryType="sling:OsgiConfig"
          hidden="true"
          label="Public DAM Search Results"
          name="public-dam-search-results"
          rendition.mappings="[card=${asset.path}.preview.319.319.jpg]"
          service.ranking="{Integer}2"/>

Output from package installation in CRXDE:
E /apps/foo/config/com.adobe.aem.commons.assetshare.content.renditions.impl.dispatchers.InternalRedirectRenditionDispatcherImpl-MySearchResults (org.xml.sax.SAXException: unknown type: Integer java.lang.IllegalArgumentException: unknown type: Integer)

and using Long
<?xml version="1.0" encoding="UTF-8"?>
<jcr:root xmlns:sling="http://sling.apache.org/jcr/sling/1.0" xmlns:jcr="http://www.jcp.org/jcr/1.0"
          jcr:primaryType="sling:OsgiConfig"
          hidden="true"
          label="Public DAM Search Results"
          name="public-dam-search-results"
          rendition.mappings="[card=${asset.path}.preview.319.319.jpg]"
          service.ranking="{Long}2"/>

causes the value to be created as a String 🤯
I've manually edited the config through CRXDE to set service.ranking to 2, using Long as the type

and restarted the instance but it still appears to load the value as a String without the accompanying ASC code changes from your PR.

I've also cherry-picked the changes from the PR onto 1.8.0, built the bundle and deployed it. I can see the values are represented as Integers but the order still looks wrong.

which is quite confusing.
Meanwhile, an alternative workaround has been suggested to me. The OOTB config is saved under /apps/asset-share-commons/config. I could have two copies (somewhat redundant) with the relevant value overridden maintained for a more specific runmode, e.g. /apps/foo/config/author and /apps/foo/config/publish
This appears to work as expected and the only downside is minor duplication.

sunuva gun .. https://issues.apache.org/jira/browse/SLING-2477
Looks like we have to change them from sling:osgiConfig to .config or something for 1.x line .. good call.

Anyway, it looks like I have a functioning workaround (author and publish-specific config that trumps the OOTB one unassociated with runmodes) for the time being so I can proceed with the work this was blocking.

Also, if its NOT an integer, the value defaults to 0 ... so Longs wont work

That explains the behaviour I've been seeing for the last hour or so where the order was... random, despite my efforts. All zeros apparently, despite the values I could see with the debugger. Makes me want to set this to
<?xml version="1.0" encoding="UTF-8"?>
<jcr:root xmlns:sling="http://sling.apache.org/jcr/sling/1.0" xmlns:jcr="http://www.jcp.org/jcr/1.0"
          jcr:primaryType="sling:OsgiConfig"
          label="Search Results"
          name="asset-share-commons-search-results-publish-specific"
          hidden="{Boolean}true"
          rendition.mappings="[card=${asset.path}.preview.319.319.jpg]"
          service.ranking="This isn't an actual ranking due to a bug, all numbers placed here will default to 0.

Ugh - damn - sorry :( ... i think we if we switch to config we can specify Integers. I dont think? that .cfg.json is supported for everything 1.x supports (i wish there was a support matrix for all this stuff, and their permutations) ..

Ugh - damn - sorry :(
No worries, having two config files instead of one is a very low price compared to some of the things I was considering above. Also, this has been super interesting ;)

As far as ASC itself is concerned, I still have a very limited understanding of how each individual feature works. .cfg.json sound more in line with AEMaaCS, which I understand is supported by 2.x.
For 1.x, .config files should probably be fine.

@toniedzwiedz ok - .config should be used in: https://github.com/adobe/asset-share-commons/pull/633
I put a snapshot of ui.apps if you have a second to check it out. I debugged  and looks good from there as well:

@davidjgonzalez sure, I'll try and have a look.


Blinking led not working

After following the tutorials here the led wont blink. I am using rpi3 model b+ running on a raspbian lite os. Does this package still works? Or maybe I do something wrong

Does this package still works?

Yes, this package still works and the program to blink an LED on the page linked to above should also work correctly.

Have you verified that the LED is connected up correctly?
Are you sure that the LED is connected the right way around?
Is there any other software running that could be using GPIO4 thus preventing the LED blink program from running?

I can verify that it works just did this pages guide today. What I can say though is that npm install was failing with node version 10. When I moved back to 8 it started working, also verified it in a docker file and build constantly failed until reverting to node 8. It appears epoll only works on 8 which is a dependency of this so you may need to update the readme if I am not mistaken.

@kiwdahc Thank your for confirming that it works. I just installed onoff with Node.js v8, v10, v12, v13 and v14-nightly. As can be see below onoff installed and ran successfully with all versions of Node.js.
Can you post the complete output of npm install onoff when attempting to install with Node.js v10 please as all looks ok on my side?
Node.js v8.17.0
pi@raspberrypi:~/onoff $ rm -rf *
pi@raspberrypi:~/onoff $ node -v
v8.17.0
pi@raspberrypi:~/onoff $ npm i onoff

><EMAIL_ADDRESS>install /home/pi/onoff/node_modules/epoll
> node-gyp rebuild

make: Entering directory '/home/pi/onoff/node_modules/epoll/build'
  CXX(target) Release/obj.target/epoll/src/epoll.o
  SOLINK_MODULE(target) Release/obj.target/epoll.node
  COPY Release/epoll.node
make: Leaving directory '/home/pi/onoff/node_modules/epoll/build'
npm WARN saveError ENOENT: no such file or directory, open '/home/pi/onoff/package.json'
npm notice created a lockfile as package-lock.json. You should commit this file.
npm WARN enoent ENOENT: no such file or directory, open '/home/pi/onoff/package.json'
npm WARN onoff No description
npm WARN onoff No repository field.
npm WARN onoff No README data
npm WARN onoff No license field.

+<EMAIL_ADDRESS>added 6 packages from 12 contributors and audited 6 packages in 10.13s
found 0 vulnerabilities

pi@raspberrypi:~/onoff $ node node_modules/onoff/examples/wait-for-interrupt.js
Please press the button on GPIO4...
Button pressed!, its value was 1
pi@raspberrypi:~/onoff $

Node.js v10.18.0
pi@raspberrypi:~/onoff $ rm -rf *
pi@raspberrypi:~/onoff $ node -v
v10.18.0
pi@raspberrypi:~/onoff $ npm i onoff

><EMAIL_ADDRESS>install /home/pi/onoff/node_modules/epoll
> node-gyp rebuild

make: Entering directory '/home/pi/onoff/node_modules/epoll/build'
  CXX(target) Release/obj.target/epoll/src/epoll.o
  SOLINK_MODULE(target) Release/obj.target/epoll.node
  COPY Release/epoll.node
make: Leaving directory '/home/pi/onoff/node_modules/epoll/build'
npm WARN saveError ENOENT: no such file or directory, open '/home/pi/onoff/package.json'
npm notice created a lockfile as package-lock.json. You should commit this file.
npm WARN enoent ENOENT: no such file or directory, open '/home/pi/onoff/package.json'
npm WARN onoff No description
npm WARN onoff No repository field.
npm WARN onoff No README data
npm WARN onoff No license field.

+<EMAIL_ADDRESS>added 6 packages from 12 contributors and audited 6 packages in 8.802s
found 0 vulnerabilities

pi@raspberrypi:~/onoff $ node node_modules/onoff/examples/wait-for-interrupt.js
Please press the button on GPIO4...
Button pressed!, its value was 1
pi@raspberrypi:~/onoff $

Node.js v12.14.0
pi@raspberrypi:~/onoff $ rm -rf *
pi@raspberrypi:~/onoff $ node -v
v12.14.0
pi@raspberrypi:~/onoff $ npm i onoff

><EMAIL_ADDRESS>install /home/pi/onoff/node_modules/epoll
> node-gyp rebuild

make: Entering directory '/home/pi/onoff/node_modules/epoll/build'
  CXX(target) Release/obj.target/epoll/src/epoll.o
  SOLINK_MODULE(target) Release/obj.target/epoll.node
  COPY Release/epoll.node
make: Leaving directory '/home/pi/onoff/node_modules/epoll/build'
npm WARN saveError ENOENT: no such file or directory, open '/home/pi/onoff/package.json'
npm notice created a lockfile as package-lock.json. You should commit this file.
npm WARN enoent ENOENT: no such file or directory, open '/home/pi/onoff/package.json'
npm WARN onoff No description
npm WARN onoff No repository field.
npm WARN onoff No README data
npm WARN onoff No license field.

+<EMAIL_ADDRESS>added 6 packages from 12 contributors and audited 6 packages in 7.168s
found 0 vulnerabilities

pi@raspberrypi:~/onoff $ node node_modules/onoff/examples/wait-for-interrupt.js
Please press the button on GPIO4...
Button pressed!, its value was 1
pi@raspberrypi:~/onoff $

Node.js v13.5.0
pi@raspberrypi:~/onoff $ rm -rf *
pi@raspberrypi:~/onoff $ node -v
v13.5.0
pi@raspberrypi:~/onoff $ npm i onoff

><EMAIL_ADDRESS>install /home/pi/onoff/node_modules/epoll
> node-gyp rebuild

make: Entering directory '/home/pi/onoff/node_modules/epoll/build'
  CXX(target) Release/obj.target/epoll/src/epoll.o
  SOLINK_MODULE(target) Release/obj.target/epoll.node
  COPY Release/epoll.node
make: Leaving directory '/home/pi/onoff/node_modules/epoll/build'
npm WARN saveError ENOENT: no such file or directory, open '/home/pi/onoff/package.json'
npm notice created a lockfile as package-lock.json. You should commit this file.
npm WARN enoent ENOENT: no such file or directory, open '/home/pi/onoff/package.json'
npm WARN onoff No description
npm WARN onoff No repository field.
npm WARN onoff No README data
npm WARN onoff No license field.

+<EMAIL_ADDRESS>added 6 packages from 12 contributors and audited 6 packages in 10.088s
found 0 vulnerabilities

pi@raspberrypi:~/onoff $ node node_modules/onoff/examples/wait-for-interrupt.js
Please press the button on GPIO4...
Button pressed!, its value was 1
pi@raspberrypi:~/onoff $

Node.js v14.0.0-nightly20191225398790149d
pi@raspberrypi:~/onoff $ rm -rf *
pi@raspberrypi:~/onoff $ node -v
v14.0.0-nightly20191225398790149d
pi@raspberrypi:~/onoff $ npm i onoff
npm WARN npm npm does not support Node.js v14.0.0-nightly20191225398790149d
npm WARN npm You should probably upgrade to a newer version of node as we
npm WARN npm can't make any promises that npm will work with this version.
npm WARN npm Supported releases of Node.js are the latest release of 6, 8, 9, 10, 11, 12, 13.
npm WARN npm You can find the latest version at https://nodejs.org/

><EMAIL_ADDRESS>install /home/pi/onoff/node_modules/epoll
> node-gyp rebuild

make: Entering directory '/home/pi/onoff/node_modules/epoll/build'
  CXX(target) Release/obj.target/epoll/src/epoll.o
  SOLINK_MODULE(target) Release/obj.target/epoll.node
  COPY Release/epoll.node
make: Leaving directory '/home/pi/onoff/node_modules/epoll/build'
npm WARN saveError ENOENT: no such file or directory, open '/home/pi/onoff/package.json'
npm notice created a lockfile as package-lock.json. You should commit this file.
npm WARN enoent ENOENT: no such file or directory, open '/home/pi/onoff/package.json'
npm WARN onoff No description
npm WARN onoff No repository field.
npm WARN onoff No README data
npm WARN onoff No license field.
npm WARN You are using a pre-release version of node and things may not work as expected

+<EMAIL_ADDRESS>added 6 packages from 12 contributors and audited 6 packages in 6.25s
found 0 vulnerabilities

pi@raspberrypi:~/onoff $ node node_modules/onoff/examples/wait-for-interrupt.js
Please press the button on GPIO4...
Button pressed!, its value was 1
pi@raspberrypi:~/onoff $

Hi @fivdi  thanks for the quick response, I'll try the mentioned solutions above. I'll back after I test this

@fivdi this is my code on trying to turn on/off the gpio
`const gpio = require('onoff').Gpio;
const  waterpump = new gpio(19, 'out');
const lightcontrol = new gpio(26, 'out');
s.clients.forEach(function (client) { //broadcast incoming message to all clients (s.clients)1
if (client.isAlive === false) return client.terminate();
ws.isAlive = false;
ws.ping(noop);
        var msg_parse = JSON.parse(message);
        client_data = [];
        lightLogControls(msg_parse.lightI);
        soilLogControls(msg_parse.soilM);
        // msg_parse.soilM < 150 ? waterpump.writeSync(1) : waterpump.writeSync(0);
        // msg_parse.lightI < 30 ? lightcontrol.writeSync(1) : lightcontrol.writeSync(0);

        for(x=0; x < client_connect; x++)
        {
            client_data.push({ tempc: msg_parse.tempC, humidity: msg_parse.humid, soilm: msg_parse.soilM, lightInt: msg_parse.lightI });
        }
        console.log(client_connect);
        });`

@kevinpallado Here is the original issue that you mentioned in the first post above:

After following the tutorials here the led wont blink.

My assumption was that the LED blink program on the page linked to in the first post doesn't work. I now see that this doesn't appear to be the case and the issue is something very different.
The code now posted here isn't remotely related to the original issue. Also, the code now posted isn't a complete program and contains very little code related to onoff. I'm sorry but I can debug your code. This is something that you'll have to do yourself.
I'm going to go ahead and close this issue. If it turns out that the issue is related to onoff the issue can be reopened.

Hi @fivdi It was my mistake to post my own code, anyways I tried using this code shown below this is based on the tutorials, after running the command the led wont blink. I did test the LED if it is working and it did. I use python code to test the led, did I miss something?
var Gpio = require('onoff').Gpio; //include onoff to interact with the GPIO
var LED = new Gpio(4, 'out'); //use GPIO pin 4, and specify that it is output
var blinkInterval = setInterval(blinkLED, 250); //run the blinkLED function every 250ms

function blinkLED() { //function to start blinking
  if (LED.readSync() === 0) { //check the pin state, if the state is 0 (or off)
    LED.writeSync(1); //set pin state to 1 (turn LED on)
  } else {
    LED.writeSync(0); //set pin state to 0 (turn LED off)
  }
}

function endBlink() { //function to stop blinking
  clearInterval(blinkInterval); // Stop blink intervals
  LED.writeSync(0); // Turn LED off
  LED.unexport(); // Unexport GPIO to free resources
}

setTimeout(endBlink, 5000); //stop blinking after 5 seconds

I already know the problem, probably the pin number confusion is the cause.

GPIO4 is on pin 7 of the Raspberry Pi GPIO header.
See https://www.raspberrypi-spy.co.uk/wp-content/uploads/2012/06/raspberry_pi_3_model_b_plus_gpio.jpg
Is the LED connected to pin 7?


Apache license removal

This pull request removes references to the Apache license from files copied from the starter kit into a project repository by stater kit installation scripts.  References to Apache in these files became an issue when the project repository was released under a conflicting license.
This pull request also includes a script that scans existing project already using the stater kit for copied files, and strips the references to Apache from these files.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Closing in favor of https://github.com/awslabs/aws-templates-for-cbmc-proofs/pull/68


Zb/editable only binary

! DO NOT MERGE!
Summary
This change demonstrates the error with --editable packages which depend on eachother and the --no-build flag, reported in https://github.com/astral-sh/uv/issues/3513
Test Plan
This is a test, effectively

Thanks! Got this cleaned up over in https://github.com/astral-sh/uv/pull/3561


data_preparation.py dosen´t work property

Describe the bug
When I run data_preparation.py on my console whit python data_preparation.py comand it start to work, but when is unzip the folder non_mask_photos something happend that stop to do it and show on the console the error: view on the screenshot
To Reproduce
Steps to reproduce the behavior:

Go to cmd
type python data_preparation.py
wait a few minutes
see the error

Expected behavior
My expected behavior was that the code unzip all the data complete and show me a doc whit the dataset whit the id (0 or 1) (mask or not)
Screenshots

Desktop (please complete the following information):

OS: Windows 10 pro 1909
python 3.8.1

This is happening because you are running out of RAM, and the DataFrame is too big.
I implemented it this way for simplicity.
I'll do another version that doesn't require to put the whole DataFrame in memory

OMG how much ram do you have? or better question. How is the minimun ram that I need? because I have 8 gb. Is to small?

BTW thanks for your fast answer

should be good now

Thanks men, I gone a test now


Unable to setup thunderbird with the created username

Hi

Docker container is up and running
this is my docker-compose.yml file's configurations

version: '2.0'

services:
  mail:
    image: tvial/docker-mailserver:latest
    hostname: mail # ${HOSTNAME}
    domainname: domain.com # ${DOMAINNAME}
    container_name: mail-server # ${CONTAINER_NAME}
    ports:
      - "25:25"
      - "143:143"
      - "587:587"
      - "993:993"
      - "1025:1025"
      - "8025:8025"

    volumes:
      - maildata:/var/mail
      - mailstate:/var/mail-state
      - maillogs:/var/log/mail
      - ./config/:/tmp/docker-mailserver/
      # ? if SELinux is enabled, uncomment the line below and comment the line above
      # - ./config/:/tmp/docker-mailserver/${SELINUX_LABEL}
    env_file:
      - mailserver.env
    cap_add:
      - NET_ADMIN
      - SYS_PTRACE
    restart: always

volumes:
  maildata:
  mailstate:
  maillogs:

I have also added the username by running following command
./setup.sh email add<EMAIL_ADDRESS>[Change123]
I just wanted to configure the thunderbird with the username i created here
I am not sure, if I had missed something or still something is to be configured, totally newbie here. Please help me what are the steps i need to follow to create the useremails and configure them in thunderbird in a local area network
Thank you so much
Yogesh

This is an issue tracker for bugs and feature requests. If you're new, and that's what you said, read the documentation on getting started and use https://google.de to troubleshoot your problems. Simply copy-pasting a Compose file will only work for people who have some experience. I'm sorry but this tracker is not for configuration advise.
Here is some advise for starters: https://github.com/tomav/docker-mailserver/wiki/A-mail-server's-101
Best regards and good luck:)


Move, not copy, VIP output.

Build server fragility increases when VIPB output goes somewhere above the workspace root (several packages are set to go a directory or two up to store their VIP exports).
This causes silent build errors if there is an old package build with the same version number, also harder to keep the build clean.

Fixed in perforce code for CI Server project.


Enable stricter MyPy typing for Rohmu

About this change - What it does

This PR enables various strict MyPy settings and adds/changes a lot of type hints to enable better typing support.
Some improvements can be added later on. For example, I/O typing is particularly messy in general. The current best practice would be to define a couple Protocols and use that as arguments, why leaving the return types as concrete io classes. Unfortunately this is not simple because some stubs/library use the typing.IO* stuff which is not 100% compatible. Most Rohmu entities deal with BufferedIOBase, and so I took that as most general I/O thing, with a couple of casts or Union in a few places where the typing.IO is inevitable.

Resolves: #13
Why this way

The API has not changed, except for the better types, but I expect that if you get a TypeError with the new version you probably have a bug. In a few places some plain attributes are now @propertys, so they can't be set anymore.

I rebased the PR to avoid conflicts in rohmufile.py.


Error when running example

Hello, I get the following error when running examples from the tutorial on my computer? is there anything I might be missing?
AttributeError: 'habitat_sim._ext.habitat_sim_bindings.SimulatorConfiguration' object has no attribute 'scene_id'

, explains the cause of this error. Just in case sombody else runs into the same problem.


fix(treesitter): update markup italic hl group

As per nvim-treesitter.

Finally fixed, closes #481.


Add an optional note field to the Deployment model

/cc @marchub

This PR was successfully deployed to production by CircleCI build 29 at 2017-04-05 18:24 BST.


Add matcher property to actions and pass search to handler function

This pull request adds a matcher property to actions that allows for dynamic matching of actions. The matcher property can be a function that receives the search string and returns a boolean indicating whether the action should be executed.
When the matcher function is defined, the search will only use it and not match on the title or keywords of the action.
It also passes the search string to the handler function, so that the handler can use it to perform actions.
Both additions allow actions based on the search string, like jumping to resources (user, project, …) pages. Coupled with semantic object IDs this can open the gate to a lot of possibilities.

I think this would be a great addition! Unfortunately, this repo seems a bit abandoned, so I would suggest creating the PR on this fork https://github.com/KonnorRogers/konnors-ninja-keys created by @KonnorRogers, which seems to be pretty active.

Hi @aMaze613. Thanks for letting me know! 😊


fix(profile): fix no wifi config found message on input focus

Closes #368

Codecov Report

Merging #373 (98e47ad) into master (f8dacf7) will decrease coverage by 0.27%.
The diff coverage is 44.44%.

@@            Coverage Diff             @@
##           master     #373      +/-   ##
==========================================
- Coverage   55.45%   55.17%   -0.28%
==========================================
  Files          39       39
  Lines         972      975       +3
  Branches       92       94       +2
==========================================
- Hits          539      538       -1
+ Misses        399      398       -1
- Partials       34       39       +5

Impacted Files
Coverage Δ

...rofiles/profile-detail/profile-detail.component.ts
78.03% <44.44%> (-1.04%)
:arrow_down:

src/app/shared/add-device/add-device.component.ts
88.63% <0.00%> (-4.55%)
:arrow_down:

Continue to review full report at Codecov.

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


loss data

Hello, can you give me a copy of the output file corresponding to the running path after the main.py command？

I built this output folder myself. The corresponding files in it are empty. When it runs to load params.saved, an error occurs. The error is out of input, so I suspect it is because the file is empty.

@huan-end Thank you for your interest to our work! Are you trying to train a model or load a trained model for inference? If you train a model from scratch, the output files will be saved to the out_dir folder specified in a config file.

@huan-end Thank you for your interest to our work! Are you trying to train a model or load a trained model for inference? If you train a model from scratch, the output files will be saved to the out_dir folder specified in a config file.

What do you mean by train_ Model is readme_ transformer_ In seq2seq.txt？

@huan-end Thank you for your interest to our work! Are you trying to train a model or load a trained model for inference? If you train a model from scratch, the output files will be saved to the out_dir folder specified in a config file.

Could you be more specific? I followed your readme, but it doesn't seem to mention the need for load model（T_T）.

@huan-end the out_dir folder specified in config should not exist when you run main.py, instead the script will create the out_dir for you and save output files to the folder. Please see here to have a better understanding.
Not sure if this is the cause. It will be helpful if you can paste the error message here.


List of functions in 0.9.15 documentation is missing

Seem to have disappeared.
http://graphite.readthedocs.io/en/0.9.15/functions.html

Yes, known bug. But unfortunately we can't fix that retroactively.
Maybe we should release 0.9.16 as final release in 0.9.x series...


Update to Node.js v20

Update to Node.js v20 to resolve deprecation warning.
Update @actions/* dependencies to latest versions.
Update jsonwebtoken to resolve CVE warnings.
Run npm audit fix.
Regenerate dist.

npm test isn't working for me on this branch or on main, but I've tested this change thusly:

Updated my shared workflow to use the 7e29837c83f71f29106364b932caafce00b6b558 commit: https://github.com/martincostello/update-dotnet-sdk/pull/726
Updated the consume of my workflow to use the above change: https://github.com/martincostello/github-automation/pull/329
Run the workflow manually from the pull request branch to verify a token is still successfully obtained and that there is no warning about Node.js v16: [https://github.com/martincostello/github-automation/actions/runs/7655789377](workflow run)

Here's an example of a workflow run without this change with the warning: workflow

Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: peter-murray/workflow-application-token-action@8e1ba3bf1619726336414f1014e37f17fbadf1db. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.


Hey please add your entries in main README file

Hey please add your entries in main README file
Originally posted by @kunjgit in https://github.com/kunjgit/GameZone/pull/526#pullrequestreview-1449353192
Could you elaborate please?

Hey @PoulasthaMukherjee! 🙂
It seems like you are not following proper guidelines !! 👀
Read documentation properly !!🙏
If you have any queries reach out to us on Discord🕹 .

Hey @PoulasthaMukherjee !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

Hey @PoulasthaMukherjee ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


When a non-sticky, locked session is expire()'ed the lock should be released

From martin.grotzke on June 14, 2012 20:50:12
Logs provided in issue #143 (https://gist.github.com/2931870) show that seems not to be the case.
Original issue: http://code.google.com/p/memcached-session-manager/issues/detail?id=144

From martin.grotzke on June 14, 2012 13:18:27
Fixed.
Status: Fixed

From<EMAIL_ADDRESS>on June 20, 2012 12:37:31
Tested the jars posted in #143, they definitely look better. POSTing login details takes 170ms, much better than before.
Full logs at https://gist.github.com/2961723

From martin.grotzke on June 20, 2012 17:38:23
Strange, looks as if there are two concurrent requests coming in for POST /EDX/j_spring_security_check.
Can you reproduce this log pattern, can you provide corresponding http request/response headers?
From the logs I would say you have configured sessionBackupAsync="true" (or not at all, and this is the default). Can you change it to sessionBackupAsync="false"? For non-sticky sessions this makes more sense.

From martin.grotzke on July 03, 2012 01:13:12
(No comment was entered for this change.)
Labels: Milestone1.6.2

From martin.grotzke on July 03, 2012 01:13:35
(No comment was entered for this change.)
Labels: -Milestone1.6.2 Milestone-1.6.2


Replace Furly dapr related library

Is your feature request related to a problem? Please describe.
After trying to run the publisher in local debug mode, I found the Furly related library is not open source. I cannot check the source code for dapr related code and it's hard for community to integrate more functions.
Describe the solution you'd like
We should found open source dapr related libs to implement dapr support. For example, the dapr sdk from Dapr or DaprSideKick
Just curious why choose furlysoft?

Or open source the furly library. The nuget-packages have MIT license.

Yep, the microsoft/project-furly repo will be public with the release of 2.9.3 next week.


Scale tests - set number of senders and receivers in tenant client

Type of change

Enhancement

Description
Comes from suggestion, to be prepared for representing iot usecase this PR adds ability to set the number of senders and receivers to create by the tenant client

Checklist

[ ] Update/write design documentation in ./documentation/design
[ ] Write tests and make sure they pass
[ ] Update documentation
[ ] Check RBAC rights for Kubernetes / OpenShift roles
[x] Try your changes from Pod inside your Kubernetes and OpenShift cluster, not just locally
[ ] Reference relevant issue(s) and close them after merging
[ ] Update CHANGELOG.md

@enmasse-ci run tests profile=scale testcase=ScaleTest#testScaleTestToolsWork test_only

:heavy_check_mark: Test Summary :heavy_check_mark:
TEST_PROFILE: scale
TEST_CASE: ScaleTest#testScaleTestToolsWork
TOTAL: 1
PASS: 1
FAIL: 0
SKIP: 0
BUILD_NUMBER: 176


Document usage of wildfly-camel on openshift.io

openshift.io is in preview at the moment but lets get some documentation together on running our Docker image in this environment.

The pipeline used by OSIO does not have a notion of context-dir. Using a repo that contains all quickstarts as we have it now would thus not work.
I tried to split out the CDI quickstart and see if it could work. Some changes were necessary see https://github.com/ppalaga/wildfly-camel-example-camel-cdi The quickstart in its current form can be imported as exiting code to OSIO and the pipeline execution succeeds producing a working container. There is still this blocker https://github.com/openshiftio/openshift.io/issues/2980 The problem is that the image cannot be deployed for some reason.
I am turning to other tasks, because OSIO is slow and buggy ATM.


Implement a DACP control interface

Is your feature request related to a problem? Please describe.
When using forked-daapd, it can connect via a DACP interface to the source device to enable hardware control of the source (playback controls/ volume controls). It is particularly useful when using spocon with a pipe connecting forked-daapd. Can you implement a DACP interface so that playback controls and volume can be controlled. The control details will need to be send to forked-daapd over the metadata pipe. I have included the command in the bottom section.
Describe the solution you'd like
Implement a DACP interface and share the details over the metadata pipe.
Describe alternatives you've considered
Nothing. I do not believe there are any other options currently.
Additional context
Taken from https://github.com/mikebrady/shairport-sync-metadata-reader
daid -- this is the source's DACP-ID (if it has one -- it's not guaranteed), useful if you want to remotely control the source. Use this string to identify the source's remote control on the network.
acre -- this is the source's Active-Remote token, necessary if you want to send commands to the source's remote control (if it has one).
dapo -- the payload is the port number (as text) of the source's remote control, to which commands should be sent. It is 3689 for iTunes but varies for iOS devices.

@aleszczynskig Do you have a capture (like piping the data from the DACP controller to a file) of what the input data would look like? I am struggling to understand what the device being controlled receives.

@devgianlu - sorry for the delay. Super busy at the moment. Unfortunately I do not have a capture. This project seems to offer an implementation though. I appreciate it is not in written in java however maybe this repo would provide some useful insight?
https://github.com/melloware/dacp-net

Wait a second, is the control interface a pipe or an HTTP server? I thought it was a pipe and the stuff I found on the internet didn't make sense, but they do if it's HTTP.

It's an HTTP interface. You provide the details of that interface over the metadata pipe so that forked-daapd or similar can establish a connection back to it.


MINOR: Upgrade dependencies for Kafka 2.3

Many patch and minor updates.
Scalatest and Jetty deprecated classes that we
use. I removed usages for the former and filed KAFKA-8316 for the latter (I
suppressed the relevant deprecation warnings until the JIRA is fixed). As
part of the scalatest fixes, I also removed TestUtils.fail since it duplicates
Assertions.fail.
I also fixed a few compiler warnings that have crept in since my last sweep.
Updates of note:

Jetty: 9.4.14 -> 9.4.18

https://github.com/eclipse/jetty.project/releases/tag/jetty-9.4.15.v20190215
https://github.com/eclipse/jetty.project/releases/tag/jetty-9.4.16.v20190411
https://github.com/eclipse/jetty.project/releases/tag/jetty-9.4.17.v20190418
https://github.com/eclipse/jetty.project/releases/tag/jetty-9.4.17.v20190418
https://github.com/eclipse/jetty.project/releases/tag/jetty-9.4.18.v20190429

zstd: 1.3.8-1 -> 1.4.0-1

https://github.com/facebook/zstd/releases/tag/v1.4.0
zstd's fastest strategy, 6-8% faster in most scenarios

zookeeper: 3.4.13 -> 3.4.14

https://zookeeper.apache.org/doc/r3.4.14/releasenotes.html

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

retest this please

retest this please

retest this please


The usual offset is +16 years, because that is how long I expect some software will be used in some places.
This showed up failing tests in our package build.
See https://reproducible-builds.org/ for why this matters.

Interesting...  I completely agree that future stability is important, but will struggle to test this as I don't have an openSUSE build env.
First thing to check here is how reproducible this failure is...  These tests are for deliberately randomized effects and so sometimes hit failures due to the random seed.  If so, I can tweak the warm up for this test to let it run a little before checking output.
Does this always fail like this, or was it a one off?

Hmm. I cannot reproduce it anymore, so it probably really is more of a random failure, independent of the date.
Maybe use a fixed random seed then or let the test retry on failure?

Yeah - seeding the RNG is a good idea.  I'll push something to master.

Naughty stalebot!  I'll fix this...

I added random.seed(42) in various places but still get random failures in TestParticles.test_drop_screen and test_rain, unless I do it there:
--- asciimatics-1.14.0.orig/asciimatics/effects.py
+++ asciimatics-1.14.0/asciimatics/effects.py
@@ -12,7 +12,7 @@ from builtins import object
 from builtins import range
 from future.utils import with_metaclass
 from abc import ABCMeta, abstractmethod, abstractproperty
-from random import randint, random, choice
+from random import randint, random, choice, seed
 from math import sin, cos, pi
 from asciimatics.paths import DynamicPath
 from asciimatics.screen import Screen
@@ -54,6 +54,7 @@ class Effect(with_metaclass(ABCMeta, obj
         :param stop_frame: Stop index for the effect.
         :param delete_count: Number of frames before this effect is deleted.
         """
+        seed(42)
         self._screen = screen
         self._start_frame = start_frame
         self._stop_frame = stop_frame

Interesting...  Given that thevrandom numbers are deterministic once the seed is set, the only reason it wouldn't follow the same path is because of some systemic difference.  Are you running the test with multiple threads, or something like that?

My guess is that randomness is used before tests run random.seed(42) e.g. in
https://github.com/peterbrittain/asciimatics/blob/31b3fe579af6ef08bcd8e01e33aeaf8d46805353/asciimatics/effects.py#L509
Which is why it became deterministic when adding it early in asciimatics/effects.py

That doesn't quite make sense.  Those should only get invoked once the class is constructed (i.e. in the test).  I'd also like to avoid seeding inside the real code...
Did you try using the setUp() method of the unittest class to set the seed?  Something like this:
class TestParticles(unittest.TestCase):
    def setUp(self):
        seed(42)


Fix assert_like_rnncell not found error using version check

Is it causing problems with previous versions? I'm on 1.8.0 and your rectification works fine

@Rayhane-mamah I made it for users that use previous version.

Yes, this patch is good for TF 1.7.0 .


LiteralPath parameter binding when setting location

Similar to PowerShell/PowerShell#12428
Since _currentLocationPath reflects the literal path currently set in the calling session, we should bind to LiteralPath to avoid path resolution issues with paths that might contain wildcard characters.

Thanks for the fix!


� desipte of polish special characters

Issue Type: Bug
When I wanted to print ą,ę,ł,ć,ż,ź instead of the desired char,  I saw in the output �, despite of the UTF-8 encoding.
VS Code version: Code 1.51.1 (e5a624b788d92b8d34d1392e4c4d9789406efe8f, 2020-11-10T23:34:32.027Z)
OS version: Windows_NT x64 10.0.19041
Extensions (13)

Extension
Author (truncated)
Version

django-html
bib
1.3.0

prettier-vscode
esb
5.7.2

auto-rename-tag
for
0.1.5

code-runner
for
0.11.1

beautify
Hoo
1.5.0

python
ms-
2020.11.358366026

jupyter
ms-
2020.11.358541065

remote-wsl
ms-
0.51.3

vscode-react-native
msj
1.1.1

color-highlight
nau
2.3.0

material-icon-theme
PKi
4.4.0

LiveServer
rit
5.6.1

open-in-browser
tec
2.0.0

Edit: the problem was the Code Runner extension in VSCode. Disabling it or running script in terminal does the job.

Edit: the problem was the Code Runner extension in VSCode. Disabling it or running script in terminal does the job.


better TF namespaces, so that graph makes sense

improve TF namespaces, e.g.:

the two versions of the decoder in the graph should clearly indicate their use in the namespace name: decoder_for_training, and decoder_for_running
dropouts should be grouped
overall, the graph (before Adam) should be easy to explore by gradual expansion of a reasonable number of components

The parts of the graph must not be in different namespaces because the variables are shared. I was stuck by this fact almost a month..

How do you propose to "group" dropouts?

use tf11+ name_scope instead of the variable scopes

Solved with PR #217.


Setup build environment for Linux first.

We should support for native build environment. Currently, first target is Linux platform.

Fixed by https://github.com/lunchclass/bacardi/commit/3f9759bb3a14056f6c606a02053cb7bbab853fe8


Gettext and CLDR usage to improve i18n

This PR wraps strings in i18n, standardizes some of the language being used, extracts them into a .po for translation, and adds a basic CLDR-based implementation for detecting preferred language and switching to it.
New languages that are added to the translations will need to be added to Kandesk.Cldr locales parameter.
I see that currently there is a language selector in the user admin area, but none in the user's Account settings, and that this is persisted to the database ... that probably would need to be reworked to be brought into line with the CLDR approach.

Thanks !
Yes I plan that the user be able to choose his preferred language from within the application.


Missing derivative ops

We have some of these mapped in java but no implementation yet.
SigmoidGrad, BiasAddGrad

https://github.com/deeplearning4j/libnd4j/pull/581

Implemented and merged


Hiding already added mods when editing a build

Oh, #360 wasn't an issue, but this is a little less complex so yeah ...

It's complety broken

Yep I saw it, even though I don't understand it ... yet


Add service name and optimize transforms

This PR adds support for tracking the service name in both metrics and traces, when we use system wide instrumentation.
I also refactored the code a bit to allow for different types of records from the ring buffer. The previous implementation for the http filter used two separate transformations, one to convert to HTTPRequestTrace and then it was later converted to HTTPRequestSpan.

Codecov Report

Merging #125 (47df33a) into main (af9cb47) will decrease coverage by 32.30%.
The diff coverage is 29.67%.

@@             Coverage Diff             @@
##             main     #125       +/-   ##
===========================================
- Coverage   69.90%   37.60%   -32.30%
===========================================
  Files          29       28        -1
  Lines        1927     1968       +41
===========================================
- Hits         1347      740      -607
- Misses        473     1173      +700
+ Partials      107       55       -52

Flag
Coverage Δ

integration-test
?

unittests
37.60% <29.67%> (-1.47%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/ebpf/goruntime/goruntime.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/ebpf/grpc/grpc.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/ebpf/httpfltr/httpfltr.go
0.00% <0.00%> (ø)

pkg/ebpf/nethttp/nethttp.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/ebpf/tracer.go
0.00% <0.00%> (-55.82%)
:arrow_down:

pkg/transform/spanner.go
67.28% <29.62%> (-16.05%)
:arrow_down:

pkg/export/otel/metrics.go
71.68% <50.00%> (-9.53%)
:arrow_down:

pkg/export/otel/traces.go
74.27% <60.71%> (-4.01%)
:arrow_down:

pkg/ebpf/common/ringbuf.go
80.26% <100.00%> (+0.26%)
:arrow_up:

... and 12 files with indirect coverage changes

I'm going to follow-up with unit tests for the new code in a separate PR. We are not using this new approach yet.

LGTM. My only (small) concern is whether it could slightly impact the performance, as a []HTTPRequestTrace slice allocates all the elements contiguously in memory, while a []interface{} slice stores pointers to other zones in memory, increasing the cache misses.

I tried using generics based on your suggestion and it almost worked :). I wasn't able to work around the return type of:
func TracerProvider(ctx context.Context, cfg ebpfcommon.TracerConfig) ([]node.StartFuncCtx[[]any], error) { //nolint:all

So technically the StartFuncCtx here would have two separate types, based on what we end up loading, go instrumentation or other language instrumentation. The decision is made inside TracerProvider after we inspect the executable.


To remove phone no as mandatory field, when saving client contact person details

Describe the task
When client contact person details are saved in the portal, then it is required to input Phone no of contact person. In case user don't have details phone no of contact person, the other details like name and email of contact person are not saved.

Expected behavior
The phone no of contact person should be made optional not mandatory
References
Specify any reference material / links

The issue has been fixed and tested


Feature Request: Add mouse events to PowerShell Engine

Summary of the new feature/enhancement
Whereas:

Both tmux and PowerShell are terminal-based applications
tmux supports mouse events, such as click-and-drag to resize panes, select a window, and scroll through history
PowerShell scripts could benefit from mouse interactions

Proposed technical implementation details (optional)
This might be implemented using a PowerShell "engine event."
Right-Click Event
The right-click event should include event data that describes the terminal coordinates of the click event.
Register-EngineEvent -Action { Draw-ContextMenu -At $event.Coordinates } -SourceIdentifier [System.Management.Automation.PsEngineEvent]::RightClick

Scroll Event
The scroll up/down event should include event data about the speed / length / height (???) of the scroll.
Example
Register-EngineEvent -Action { GrabAndDisplay-SomeHistoricalLogData } -SourceIdentifier [System.Management.Automation.PsEngineEvent]::ScrollUp

Click Event
The click event should support both a single click and double-click events. Both should include terminal coordinates where the event occurred.
Register-EngineEvent -Action { Perform-AnAction -At $event.Coordinates } -SourceIdentifier [System.Management.Automation.PsEngineEvent]::DoubleClick

I guess it is more for PSReadline. Could you please create issue there?

Roger, thanks for the pointer. I wasn't sure which component was responsible for handling this type of input.


Null booleans

You cannot create null booleans with the ui as it turns to false.

Moving issues into rancherio/rancher.
https://github.com/rancherio/rancher/issues/324


Completely broken

VS Code is completely broken after the new update. I can no longer position VS Code using Rectangle, I cannot open open a project in VS Code (folder selector doesn't work). Opening a terminal doesn't work, it just instantly closes before the zsh process is started.

/needsMoreInfo including platform.

Thanks for creating this issue! We figured it's missing some basic information or in some other way doesn't follow our issue reporting guidelines. Please take the time to review these and update the issue.
Happy Coding!

Nevermind, the issue solved itself after closing and opening vscode a few times.


Post an announcement for the new webpage

Let's have a post announcing the new webpage here and on the old page. This should make people aware of the upcoming migration.

Should we add it as an announcement in the new site @awvwgk Sir ?
Thanks and Regards,
Henil Shalin Panchal


Refactoring spec

Coverage remained the same at 96.347% when pulling 95de4d88a482182675112f2796b097f8aaec1624 on refactoring_spec into a541c8d05b9eba5b348dc2f59b41fcbdf161cd53 on 0.7.

🎉


feat(fiatconnect): Include network fee estimation on FiatConnect Review Screen

Description
For ACT-586. Adds in network fee estimation to the FiatConnect Review screen for cash outs.
Images
Below is an example of the diff from current behavior, for a cash-out of 0.40 cEUR, with a provider fee of 0.02 cEUR, and an estimated network fee of .047 cEUR. (cEUR was chosen as an illustrative example since it requires conversion of the cUSD-based fee estimation to the user-selected crypto type.) Note that the displayed exchange rate is unchanged, which is, I believe, the desired behavior as suggested in the ticket (since network fee doesn't contribute to the portion of total funds used to actually convert to the desired fiat currency).

Test plan
Manual and unit tested.
Related issues

Fixes ACT-586

Backwards compatibility
Yes.

Looks good to me!


[easy] No longer mark engineering liberal studies data as allow double counting

Summary
To be merged after #554 is run on prod data.
Test Plan
👀

technically can't this be merged to master and then later when we push to prod we should run the script on prod?

technically can't this be merged to master and then later when we push to prod we should run the script on prod?

yes that's right


[question]: How to mock service in the "mocks" parameter in createTestComponentFactory?

Hi.
I have a component that, onInit, will try to subscribe to a repository observable.
I have found that, inside "providers", I can use the "mockProvider" function and pass on my mocked functions behaviours. Plus, this is done before even creating the component, so I can let automatic change detection to true.
Is there any way to do this using the "mocks" parameter? Is to allow everyone to always use the same parameters for all mocks in the system.
Because "mocks" doesn't allow to set up a mock, i'm forced to set detect changes to false and manually launch detect changes on every event update :/
The below 2 test work. What I want is to be able to set up "mocks" like I set up "providers".

  let spectator: Spectator<OrderHistoryComponent>;
  const createComponent = createTestComponentFactory({
    component: OrderHistoryComponent,
    declarations: [MockComponent(OrderHistoryListComponent)],
    mocks: [OrderHistoryRepository],
    detectChanges: false
  });

  beforeEach(()=> {
    spectator= createComponent()
    spectator.get<OrderHistoryRepository>(OrderHistoryRepository).getProductList.and.returnValue(of(EXPECTED_ORDER_HISTORY))
    spectator.detectChanges()
  })

  it('should create', () => {
    expect(spectator.component).toBeTruthy();
  });

  let spectator: Spectator<OrderHistoryComponent>;
  const createComponent = createTestComponentFactory({
    component: OrderHistoryComponent,
    declarations: [MockComponent(OrderHistoryListComponent)],
    providers: [mockProvider(OrderHistoryRepository, {getProductList: () => of(EXPECTED_ORDER_HISTORY)})]
    });

  it('should create', () => {
    spectator = createComponent()

    expect(spectator.component).toBeTruthy();
  });

The problematic ngOnInit code is this btw:
@Component({
  selector: 'app-order-history',
  templateUrl: './order-history.component.html',
  styleUrls: ['./order-history.component.scss']
})
export class OrderHistoryComponent implements OnInit {

  constructor(private _service: OrderHistoryRepository) { }

  order: IOrderHistory;

  ngOnInit() {
    this._service
      .getProductList()
      .subscribe( next => this.order = next );
  }

}

Yeah, is a simple component. And I suspect that this can happen with ngOnInit, ngOnChange and others lifecycle methods as well.
I don't know if there is some way to pass custom mock to the "mocks" parameter that I haven't found. If there isn't, I think it would be a good idea to allow passing the mock as parameter like the "mockProvider" does

Hi, as you mentioned you can do this using providers. This is not possible using mocks because it is just a shortcut for creating mocks that are simple spies of the classes provided. Adding support for this would probably bloat the API for the mocks property enough as to make it pointless.
If you want to use the same API in both cases, you could use providers: [mockProvider(...)] even when using simple spy mocks.

Yeah, that as much as I discovered. I guess that if you want to keep it separate like that, its ok. Thank you for the answer
Btw, do you accept PR's for documentation?

Unfortunately the documentation is not hosted on Github - not sure if @NetanelBasal has a process for this?

It's currently hosted in Gitbook. We should migrate it to Github Wiki so that other developers could contribute.

I like Gitbook for consuming the docs - maybe we could use something like https://docs.gitbook.com/integrations/github so the source of truth is Github but the docs are still readable through Gitbook.

Thanks. I will check it out.


SurfacePosition

Started by @IDBeswick
Initial attempt at implementing SurfacePosition. Algorithm is mostly shared with AirbornePosition, so the common elements are moved to the PositionUpdate class, which maintains a cache of even/odd frames, decides whether Global or Local calculation can be used, and generally contains the complexity of the CPR system. Some further sanity checks are required, and also a way to choose between multiple possible solutions for lat/lon for Surface data.

Small changes were made mainly pre calculating the track and velocity information. Then uses the pre-calculated values to assign them to the track object when available

SurfacePositionUpdates should now be working correctly, and AirbornePositionUpdates now correctly work for both Global and Local updates. Tests have been provided, but may not cover all edge-cases.

Looking good @IDBeswick thanks for the contribution!


treewide: enable strictDeps in bootstrap packages

Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 173909 run on aarch64-darwin 1

1 package built:

hello

alright, only one more
$ nix-diff $(nix eval --raw ".#bash.drvPath") $(nix eval --raw --expr 'with import ./. { config = { strictDepsByDefault = true; }; }; bash.drvPath' --impure)
- /nix/store/1s8vhyfq82127wrkrzqnl4jrn68kc74a-bash-5.1-p16.drv:{out}
+ /nix/store/mla63jkfy61507fvbxa2x92zskxrkpsn-bash-5.1-p16.drv:{out}
• The input derivation named `bootstrap-stage4-gcc-wrapper-11.3.0` differs
  - /nix/store/npj5218170k7nqr744ji146r71cnfhnc-bootstrap-stage4-gcc-wrapper-11.3.0.drv:{out}
  + /nix/store/19l65594r0r68dyyaikpr2ckbxgw4ck9-bootstrap-stage4-gcc-wrapper-11.3.0.drv:{out}
  • The input derivation named `gcc-11.3.0` differs
    - /nix/store/6hl94g48qsxbv2w0h4v6gdxn1g1s9hsi-gcc-11.3.0.drv:{info,lib,man,out}
    + /nix/store/6j2fmb1cj46bnivyskd64dlf2xbgz8fm-gcc-11.3.0.drv:{info,lib,man,out}
    • The environments do not match:
        strictDeps=1
  • Skipping environment comparison
• The input derivation named `bootstrap-stage4-stdenv-linux` differs
  - /nix/store/2rsa0xsw9k127hq276sqy74gzpc0wcbw-bootstrap-stage4-stdenv-linux.drv:{out}
  + /nix/store/y5bcnivsn21qxi8hp9rh0r7mzzr5dgp4-bootstrap-stage4-stdenv-linux.drv:{out}
  • The input derivation named `bootstrap-stage4-gcc-wrapper-11.3.0` differs
    • These two derivations have already been compared
  • Skipping environment comparison
• Skipping environment comparison

i'll probably do gcc in a separate PR

@ofborg build bash

@ofborg build bash

@ofborg eval
the failure is unrelated
Test Summary Report-------------------
t2200-add-update.sh                              (Wstat: 256 Tests: 17 Failed: 1)
  Failed test:  15  Non-zero exit status: 1
Files=949, Tests=24140, 57 wallclock secs ( 9.29 usr  3.23 sys + 544.43 cusr 1176.85 csys = 1733.80 CPU)Result: FAILmake[1]: *** [Makefile:56: prove] Error 1make[1]: Leaving directory '/build/git-2.36.0/t'make: *** [Makefile:2949: test] Error 2error: builder for '/nix/store/ay96gmi1fn62vqjhhcr2vkpkya74k3h7-git-minimal-2.36.0.drv' failed with exit code 2;       last 10 log lines:       > Test Summary Report       > -------------------       > t2200-add-update.sh                              (Wstat: 256 Tests: 17 Failed: 1)       >   Failed test:  15       >   Non-zero exit status: 1       > Files=949, Tests=24140, 57 wallclock secs ( 9.29 usr  3.23 sys + 544.43 cusr 1176.85 csys = 1733.80 CPU)       > Result: FAIL       > make[1]: *** [Makefile:56: prove] Error 1       > make[1]: Leaving directory '/build/git-2.36.0/t'       > make: *** [Makefile:2949: test] Error 2       For full logs, run 'nix log /nix/store/ay96gmi1fn62vqjhhcr2vkpkya74k3h7-git-minimal-2.36.0.drv'.

bash was built succesfully

2 approvals
bash and sway compile successfully
bash cross-compiles
output diffs look ok (checked with diffing)
therefore i will soon merge
diffs checked by doing  git diff --name-only HEAD HEAD~6 | rev | cut -d '/' -f 2 | rev > attrlist then cleaned it up manually
and then ran diffing --force --file attrlist
nukeReferences
pkg-config
perl
python
expat
glibc
gmp
isl
libelf
libffi
libidn2
libmpc
libunistring
mpfr
zlib
autoconf-archive
autoconf
automake
binutils
gnum4
libtool
patchelf
pkg-config
texinfo
bison
linuxHeaders
bzip2
xz
file
which

https://github.com/NixOS/nixpkgs/pull/185972


Replace the to_string macro with the serialize_all macro

If to_string is used, a human error may occur because the enum name and the name to be written in to_string exist separately.
How about improving it using serialize_all?

Codecov Report

Merging #565 (dbbe408) into main (69c9e84) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #565   +/-   ##
=======================================
  Coverage   93.34%   93.34%
=======================================
  Files         176      176
  Lines       11728    11728
=======================================
  Hits        10948    10948
  Misses        780      780

Impacted Files
Coverage Δ

core/src/plan/expr/function.rs
100.00% <ø> (ø)

core/src/executor/evaluate/mod.rs
82.81% <100.00%> (ø)

core/src/executor/evaluate/stateless.rs
83.85% <100.00%> (ø)

core/src/translate/function.rs
98.00% <100.00%> (ø)

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 69c9e84...dbbe408. Read the comment docs.


[zh] Sync manage-deployment.md and labels.md

Follow en #42818 to relocate label sections:
content/zh-cn/docs/concepts/cluster-administration/manage-deployment.md
content/zh-cn/docs/concepts/overview/working-with-objects/labels.md

/approve


What is the layout being used

What is the default layout being used for html generation?

I tried to use README.md without index.md but with the output I couldn't add styles or scripts (with a layout) - maybe it renders with (layout: none)
I ended up relative include in index.md.
---
layout: mylayout
permalink: /
---

{% include_relative README.md %}


Update

1 update  : add the keyword "write" which does the same as "save" (to use the same keyword to save like Cisco)
and 1 bug fix :  better detection of duplicate address

Thanks :)


[Resiliency] component-specific retry values

Description
Add section about component-specific retries
May  need to add more to component specs?
Issue reference
PR will close: #2431

closing until we know direction here - see code issue: dapr/components-contrib#1863


Hoist redundant parallel actions

When working on the new slew of queues (FIFO, PIFO, PIFO Tree), I found myself doing the following refactoring by hand:
par (
  (
    if_case_1:
      action;
      additional_action_for_cases_1_and_2
  ),
  (
    if_case_2:
      action;
      additional_action_for_cases_1_and_2
  ),
  (
    if_case_3:
      action;
  )
) # par ends

turned into:
action;
if_case_1_or_2: additional_action_for_cases_1_and_2

And I think that's simpler.
Basically the action was the same in all the parallel branches, so I lifted it. Then the additional_action_for_cases_1_and_2 is run after that. These cannot themselves be run in parallel: the latter step depends on some state change triggered by the former.
Just making a quick issue in case Calyx would like to perform such hoisting automatically.
Originally posted by @anshumanmohan in https://github.com/cucapra/calyx/pull/1657#discussion_r1294609470

Thanks for creating this issue! One can imagine a similar pass that hoists groups out of a while loops but that would be more complicated because you'd be implementing loop invariant code motion.

An obvious but important property of this rewrite should be that action cannot affect the value of the conditions if_case_*. If it does, we cannot move it outside the if.


Patched SonarQubePreBuildTask to work with all versions of SonarQube

Task used to fail while working with SonarQube version 5.2, because it was failing to convert decimal to string.
I have fixed this by typecasting version to string.

Hi @gaurav42, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

LGTM


Tables indented by double tabs do not render properly

Version: Markdown-it v12.0.0
Description
Consider the following Markdown source: a table inside a nested list. Each line of the table is indented by two consecutive tabs.
- Level 1

	- Level 2

		| Column 1 | Column 2 |
		| -------- | -------- |
		| abcdefgh | ijklmnop |

What is expected (Github's implementation)

Level 1

Level 2

Column 1
Column 2

abcdefgh
ijklmnop

What actually happens
It seems that the first 4 characters of each line after the tabs (| Co, | ab) is removed.
Demonstration of the bug @ https://markdown-it.github.io/
Discussions
According to the GFM spec, "in contexts where whitespace helps to define block structure, tabs behave as if they were replaced by spaces with a tab stop of 4 characters."
If every tab is replaced with 4 whitespaces, Markdown-it correctly renders the table.

Fixed, thanks for reporting.


Error in microwin

Selected value '6 : Windows 11 Pro'.....
Could not remove OS package Package_for_KB502739731bf3856ad364e35amd6422621.2355.1.1
Could not remove OS package Package_for_RollupFix31bf3856ad364e35amd6422621.2861.1.6
Invoke-WPFMicrowin : An unexpected error occurred: Removal failed. Please contact your software vendor.
At line:2805 char:24
"WPFMicrowin" {Invoke-WPFMicrowin}
               ~~~~~~~~~~~~~~~~~~

CategoryInfo : NotSpecified: (:) [Write-Error], WriteErrorException
FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorException,Invoke-WPFMicrowin
Unmounting Registry...
ERROR: The parameter is incorrect.
ERROR: The parameter is incorrect.
ERROR: The parameter is incorrect.
ERROR: The parameter is incorrect.
ERROR: The parameter is incorrect.
Cleaning up image...
Cleanup complete.

Try using Powershell 7 to see if it works or not.
Sometimes Powershell 5 is buggy sometimes

It should be fixed with #2192

@Papoutsisss if this issue is fixed please could you close this issue?


title : Request for Correct Label Assignment on Issue #161 for GSSoC Contribution

Subject: Request for Correct Label Assignment on Issue #161
Hi @mansiruhil13,
I hope you're doing well! I wanted to kindly bring to your attention that the label currently assigned to issue #161 seems to be incorrect. If possible, could you please update the label to "gssoc-ext" so that it can be counted as a valid contribution towards the GirlScript Summer of Code (GSSoC)?
I truly appreciate your time and assistance with this! Thank you for your support.
Best regards,
@khurshed07
https://github.com/khurshed07

its done thanks


x509: certificate expired with signed certificate on private repository

Description:
I get a x509: certificate has expired or is not yet valid when trying to push/pull images from a private repository with a valid signed certificate from a remote machine.
If I try push/pull from the same machine as the repository it works fine.
PS F:\docker> docker push example.net:5000/test
The push refers to a repository [example.net:5000/test]
Get https://example.net:5000/v1/_ping: x509: certificate has expired or is not yet valid

I have tested the certificate by pointing a browser and CURL at the repository URL and it works fine.
CURL output:
PS F:\docker> curl https://example.net:5000

StatusCode        : 200
StatusDescription : OK
Content           :
RawContent        : HTTP/1.1 200 OK
                    Content-Length: 0
                    Cache-Control: no-cache
                    Content-Type: text/plain; charset=utf-8
                    Date: Mon, 28 Nov 2016 23:54:05 GMT

Forms             : {}
Headers           : {[Content-Length, 0], [Cache-Control, no-cache], [Content-Type, text/plain; charset=utf-8], [Date,
                    Mon, 28 Nov 2016 23:54:05 GMT]}
Images            : {}
InputFields       : {}
Links             : {}
ParsedHtml        : mshtml.HTMLDocumentClass
RawContentLength  : 0

The repository run command is:
PS D:\docker\registry> docker run -i -p 5000:5000 --restart=always --name registry --dns=<IP_ADDRESS> -v //d/docker/registry/data:/var/lib/registry -v //d/docker/registry/certs:/certs -e REGISTRY_HTTP_TLS_CERTIFICATE=/certs/cert.crt -e REGISTRY_HTTP_TLS_KEY=/cer
ts/cert.key registry:2

Output of docker version:
PS F:\docker> docker version
Client:
 Version:      1.12.3
 API version:  1.24
 Go version:   go1.6.3
 Git commit:   6b644ec
 Built:        Wed Oct 26 23:26:11 2016
 OS/Arch:      windows/amd64

Server:
 Version:      1.12.3
 API version:  1.24
 Go version:   go1.6.3
 Git commit:   6b644ec
 Built:        Wed Oct 26 23:26:11 2016
 OS/Arch:      linux/amd64

Output of docker info:
PS F:\docker> docker info
Containers: 9
 Running: 1
 Paused: 0
 Stopped: 8
Images: 128
Server Version: 1.12.3
Storage Driver: aufs
 Root Dir: /var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 124
 Dirperm1 Supported: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: host bridge null overlay
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Security Options: seccomp
Kernel Version: 4.4.27-moby
Operating System: Alpine Linux v3.4
OSType: linux
Architecture: x86_64
CPUs: 2
Total Memory: 1.919 GiB
Name: moby
ID: UOST:JTIN:EXBN:ZFAV:RKW7:XJJD:OCQY:36JR:BGPY:OLWM:VZQ2:HP7Z
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): true
 File Descriptors: 16
 Goroutines: 28
 System Time: 2016-11-28T11:29:13.0699591Z
 EventsListeners: 0
Registry: https://index.docker.io/v1/
WARNING: No kernel memory limit support
Insecure Registries:
 <IP_ADDRESS>/8
PS F:\docker> docker info
Containers: 9
 Running: 1
 Paused: 0
 Stopped: 8
Images: 128
Server Version: 1.12.3
Storage Driver: aufs
 Root Dir: /var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 124
 Dirperm1 Supported: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: bridge null host overlay
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Security Options: seccomp
Kernel Version: 4.4.27-moby
Operating System: Alpine Linux v3.4
OSType: linux
Architecture: x86_64
CPUs: 2
Total Memory: 1.919 GiB
Name: moby
ID: UOST:JTIN:EXBN:ZFAV:RKW7:XJJD:OCQY:36JR:BGPY:OLWM:VZQ2:HP7Z
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): true
 File Descriptors: 16
 Goroutines: 28
 System Time: 2016-11-28T11:29:17.512845Z
 EventsListeners: 0
Registry: https://index.docker.io/v1/
WARNING: No kernel memory limit support
Insecure Registries:
 <IP_ADDRESS>/8

Both machines run on Windows 10 and have the same version.

It seems like after a few restarts of the host machine it works.
I can no longer reproduce the problem.


Docs: move _.compact to Collections section

Update documentation and website as mentioned in #2622.

Coverage remained the same at 96.763% when pulling 1674e0f396bcdd8db60faa84467f7fe0479618a5 on wotkuni:docs-compact-array into<PHONE_NUMBER>d8df56ed9d9da7e3521239b9de804b on jashkenas:master.

Thanks!


spring-cloud-gcp-starter-config 1.1.0.RELEASE is missing

Is spring-cloud-gcp-starter-config 1.1.0.RELEASE is available in spring-cloud-gcp-dependencies-1.1.0.RELEASE in maven central. I guess it is not.
See: spring-cloud-gcp-dependencies
and
spring-cloud-gcp-starter-config

What was the actual resolution of this issue? spring-cloud-gcp-starter-config  is configured to skip deploy, but spring-cloud-gcp-config, also mentioned in spring-cloud-gcp-dependencies, doesn't even seem to exist.

Right, spring-cloud-gcp-starter-config is not in the released versions because it's not GA, and the reason it's not GA is that the underlying Runtime Config API is Beta.
Check out the new Spring Cloud Bus support, that you can use as an alternative for configuration management.
Regarding spring-cloud-gcp-logging, it not in the 1.1.x release tag because it has only been added as a module in 1.2, which hasn't been released yet outside of snapshots.


namespace dropdown looks disabled in light theme while creating a deployment

on 2.6-head commit id: b43e4d9

namespace dropdown looks disabled in light theme while creating a deployment

@nwmac Is this one ready for test, or given https://github.com/rancher/dashboard/issues/4003#issuecomment-951255327 needs to be reopened?

PASS Verified in 2.6-head Commit ID 4375161

Deployed a downstream cluster
clicked Explore
clicked Deployments > Create
Namespace is now colored same as other input fields on screen; no longer looks disabled.
checked all other sections under Workload to confirm color change
CronJobs
DaemonSets
Deployments
Jobs
Stateful Sets


fix(koa): fix issue over async_hook with Koa platform

:tada: This PR is included in version 6.128.2 :tada:
The release is available on:

v6.128.2
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 7.0.0-rc.6 :tada:
The release is available on:

v7.0.0-rc.6
GitHub release

Your semantic-release bot :package::rocket: