2.1.0 broke test.mock.num.brokers

Description
After upgrading from 2.0.2 to 2.1.0 unit tests utilising test.mock.num.brokers are failing with error Confluent.Kafka.ConsumeException: Subscribed topic not available: UnitTest: Broker: Unknown topic or partition
How to reproduce
public class Example
{
    public void SampleReproduction()
    {

        var adminClient = new AdminClientBuilder(new ClientConfig(new Dictionary<string, string> { {"bootstrap.servers", "localhost:9200"}, {"test.mock.num.brokers", "3"} })).Build();
        var bootstrapServers = string.Join(",", adminClient.GetMetadata(TimeSpan.FromSeconds(1)).Brokers.Select(b => $"{b.Host}:{b.Port}"));

        var producer = new ProducerBuilder<Null, string>(new ProducerConfig { BootstrapServers = bootstrapServers }).Build();
        var consumer = new ConsumerBuilder<Ignore, string>(new ConsumerConfig { BootstrapServers = bootstrapServers, GroupId = "ExampleUnitTest", AutoOffsetReset = AutoOffsetReset.Earliest }).Build();
        var message = new Message<Null, string> { Value = "Test" };
        var topic = "UnitTest";

        producer.Produce(topic, message);
        consumer.Subscribe(topic);

        var actual = consumer.Consume(TimeSpan.FromSeconds(10)).Message.Value;

        actual.Should().Be(message.Value);
    }
}

stacktrace contains nothing interesting
Confluent.Kafka.ConsumeException: Subscribed topic not available: UnitTest: Broker: Unknown topic or partition

Confluent.Kafka.ConsumeException
Subscribed topic not available: UnitTest: Broker: Unknown topic or partition
   at Confluent.Kafka.Consumer`2.Consume(Int32 millisecondsTimeout)
   at Confluent.Kafka.Consumer`2.Consume(TimeSpan timeout)
   at Tests.ExampleUnitTestForConsumersAndOrProducers.SampleReproduction() in /Users/mac/github.com/rabotaua/kafka/Tests/ExampleUnitTestForConsumersAndOrProducers.cs:line 43

I have added log handler to admin client, but there is only one record:
[thrd:app]: Mock cluster enabled: original bootstrap.servers and security.protocol ignored and replaced with <IP_ADDRESS>:55554,<IP_ADDRESS>:55556,<IP_ADDRESS>:55557

Attempt to manually create topic:
await adminClient.CreateTopicsAsync(new[] { new TopicSpecification { Name = "UnitTest" } });

fails with error
Confluent.Kafka.KafkaException: Failed while waiting for controller: Local: Timed out

Confluent.Kafka.KafkaException
Failed while waiting for controller: Local: Timed out
   at Tests.ExampleUnitTestForConsumersAndOrProducers.SampleReproduction() in /Users/mac/github.com/rabotaua/kafka/Tests/ExampleUnitTestForConsumersAndOrProducers.cs:line 44
   at Xunit.Sdk.TestInvoker`1.<>c__DisplayClass48_0.<<InvokeTestMethodAsync>b__1>d.MoveNext() in /_/src/xunit.execution/Sdk/Frameworks/Runners/TestInvoker.cs:line 264
--- End of stack trace from previous location ---
   at Xunit.Sdk.ExecutionTimer.AggregateAsync(Func`1 asyncAction) in /_/src/xunit.execution/Sdk/Frameworks/ExecutionTimer.cs:line 48
   at Xunit.Sdk.ExceptionAggregator.RunAsync(Func`1 code) in /_/src/xunit.core/Sdk/ExceptionAggregator.cs:line 90

[thrd:app]: Mock cluster enabled: original bootstrap.servers and security.protocol ignored and replaced with <IP_ADDRESS>:55662,<IP_ADDRESS>:55663,<IP_ADDRESS>:55664

Checklist
Please provide the following information:

[x] A complete (i.e. we can run it), minimal program demonstrating the problem. No need to supply a project file.
[x] Confluent.Kafka nuget version.
[x] Apache Kafka version.
[x] Client configuration.
[x] Operating system.
[x] Provide logs (with "debug" : "..." as necessary in configuration).
[x] Provide broker log excerpts.
[ ] Critical issue.

The same is true for newly released 2.1.1

Use ProduceAsync instead.
public class Example
{
    public async Task SampleReproduction()
    {

        var adminClient = new AdminClientBuilder(new ClientConfig(new Dictionary<string, string> { {"bootstrap.servers", "localhost:9200"}, {"test.mock.num.brokers", "3"} })).Build();
        var bootstrapServers = string.Join(",", adminClient.GetMetadata(TimeSpan.FromSeconds(1)).Brokers.Select(b => $"{b.Host}:{b.Port}"));

        var producer = new ProducerBuilder<Null, string>(new ProducerConfig { BootstrapServers = bootstrapServers }).Build();
        var consumer = new ConsumerBuilder<Ignore, string>(new ConsumerConfig { BootstrapServers = bootstrapServers, GroupId = "ExampleUnitTest", AutoOffsetReset = AutoOffsetReset.Earliest }).Build();
        var message = new Message<Null, string> { Value = "Test" };
        var topic = "UnitTest";

        await producer.ProduceAsync(topic, message);
        consumer.Subscribe(topic);

        var actual = consumer.Consume(TimeSpan.FromSeconds(10)).Message.Value;

        actual.Should().Be(message.Value);
    }
}

The broker has 0 topics initially, so the consumer has to wait for the topic to be created, as the Producer has the default AllowAutoCreateTopics property as true. I don't know why the previous versions of the library didn't have errors, it could be some change made in librdkafka 2.1.0.

Thank you for clarification
Indeed seems like that's our case
We use sync version of Produce exactly because how it works internally and its fire and forget kind of nature
Changing it to ProduceAsync immediately solves broken test
Also we may prove your suggestion by just putting some small delay after producing our message, then everything will also work
So indeed there is something deep inside librdkafka changed and now sync produce method becomes even more fire and forget than before :)


Unable to specify include and lib dirs for OpenSSL on Windows and on Linux

There are bugs in CMakeLists.txt. Please fix, e.g.
$ git diff CMakeLists.txt
diff --git a/CMakeLists.txt b/CMakeLists.txt
index b6fd4ee9..312ac8bc 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -200,11 +200,11 @@ IF(${CMAKE_SYSTEM_NAME} MATCHES "Windows")
        set(CMAKE_CXX_FLAGS_RELEASE "${CMAKE_CXX_FLAGS_RELEASE} /MP /EHsc /bigobj ${HZ_C11_FLAGS}")

        if (${HZ_COMPILE_WITH_SSL} MATCHES "ON")
-               if ("x${HZ_OPENSSL_INCLUDE_DIR}" MATCHES "x")
+               if ("x${HZ_OPENSSL_INCLUDE_DIR}" STREQUAL "x")
                        set (HZ_OPENSSL_INCLUDE_DIR C:\\OpenSSL-Win64\\include)
                endif()

-               if ("x${HZ_OPENSSL_LIB_DIR}" MATCHES "x")
+               if ("x${HZ_OPENSSL_LIB_DIR}" STREQUAL "x")

The HZ_COMPILE_WITH_SSL section is missing for Linux. I copied 18 lines for Darwin and pasted for Linux. How do we eliminate in link:
-Wl,-rpath,${HZ_OPENSSL_LIB_DIR}

README.md is missing dashes:

You can provide your openssl installation directory to cmake using the following flags:

DHZ_OPENSSL_INCLUDE_DIR="Path to open installation include directory"
DHZ_OPENSSL_LIB_DIR="Path to openssl lib directory"

Hello @justinpa-asg
The reason that I did not add directory for Linux was because it always installed it in the same system library folder and the build finds it. Which environment did you try in? How did you install openssl? If you used a custom folder for openssl, that may be the reason for such a need. I can add it if we really need it but initially it was not put to Linux intentionally since the openssl related include and libs were already in the path we already use for build.

Hi @ihsandemir
Our team builds our own version of OpenSSL with FIPS support. Thus we want to specify an alternative location for it. Copying the 18 lines from Darwin section to Linux section allowed me to use the flags. Why have the flags if they can't be used?

ok, i see your need. I will add this to Linux section as suggested.

Pull 334 did not resolve any of the problems reported in this issue. For Linux, I get:
-- Using HZ_OPENSSL_LIB_DIR: /builds/openssl-1.0.2n/lib
-- Using HZ_OPENSSL_INCLUDE_DIR: /builds/openssl-1.0.2n/include

But then:
$ ldd libHazelcastClient3.9.1_64.so
        linux-vdso.so.1 =>  (0x00007fffc23ff000)
        libstdc++.so.6 => /usr/lib64/libstdc++.so.6 (0x00007f2e3af3f000)
        libm.so.6 => /lib64/libm.so.6 (0x00007f2e3acba000)
        libgcc_s.so.1 => /lib64/libgcc_s.so.1 (0x00007f2e3aaa4000)
        libc.so.6 => /lib64/libc.so.6 (0x00007f2e3a710000)
        /lib64/ld-linux-x86-64.so.2 (0x0000003522400000)

There's no libssl.so or libcrypto.so linked. I see -isystem /builds/openssl-1.0.2n/include but I do not see -lssl or -lcrypto.
Regarding Windows, MATCHES was not replaced with STREQUAL.

I found cmake documentation that states: "Specify libraries.. when linking any targets created... below by... add_library()". So link_libraries must precede add_library in CMakeLists.txt. I attached a patch for issue 330 and 374.
issue330.txt

@ihsandemir When will patches be applied? We must continue to create our own patches for new releases.


Add Entity to existing Incident

Is your feature request related to a problem? Please describe.
When investigating an incident, we can find new items related to incident that can be useful to analyse.respond to/pivot against when new incidents will occur.
Describe the solution you'd like

Having a buttun in Entities section of an incident to add new Entity, adding a right pane form asking for entity Type and associated entity
Having also Logic App action to add it directly with automation.

Describe alternatives you've considered
Workaround :

Adding new alerts associated with hardcoded Entities and forced linked to this incident. Any ideas could be greatly appreciated.

Additional context
None.

Hey @FormindGMO, Could you please provide more details about the issue your facing, it would be great if you could attach screenshots as well. It helps us to investigate on it. Thanks!

Hello! thanks for fast reply. It's not an issue but rather a feature request. Let's illustrate w/ an example.
Assuming an incident of a computer having a malware that was run on it. In this incident we can assume that we had FileHash entity pre-positioned, and targeted host.
Analyzing targeted host, we can see that malware reached Command&Control <IP_ADDRESS>. I want to :

Add for the record this new finding in Sentinel incident with new IP Entity
Enabling this can maybe leverage other playbooks (like : IP blockage on Firewall)
And if in the future, other incidents spawns having this IP as Entity, it's linked in Sentinel Investigation tab, which is very useful for historical analysis.

So my feature could be to manually (or via Logic Apps) add entity to Incidents.

This is a similar requirement for the currently in preview create incident feature in sentinel.
What purpose is there to create an incident manually in sentinel where you can't assign entities to it.
During investigation you could also find assets gents that are not 'explicitly' bound by the analytic rule inside of the incident and best case we should be able to add that entity to the incident as it enriches the incident with that particular asset and are relevant to the case.
Similarly as @FormindGMO mentions, this can be utilized alongside existing playbooks that utilize said entities for actions such as:

bulk reset MFA
reset auth
block IP
block user
add entities to known threat actors/ add ips to known threat actors

The feature should allow a user within the tenant to at the least - assign entities from the tenant incident is in.

Thanks @FormindGMO / @Kaloszer, sharing information with us, we have reach out to concern team for this issue, Once we get any info from team, will update you.

+1

Hi @FormindGMO, We have received a response from the concerned team. The Sentinel entities linking to an existing incident is not yet supported in the portal. And team has noted down this feature request into their queue, However, Sentinel allows adding entities to an existing incident using the bookmarks feature. For more information, please refer to the following document: https://learn.microsoft.com/en-us/azure/sentinel/bookmarks
So, closing this issue from GitHub. If you still need support for this issue, feel free to re-open it any time. Thank you for your co-operation.


Can't make the code it work

Thanks for creating the safe API timer, I have been hit by the sudden crash hundreds of times in the past.
I tried both methods, but unfortunately both of them didn't seem to work.  I used the latest Office 365 64-bit on Windows 10 22H2 64-bit.  I tried uninstalling Office 365 and do a fresh installation, still didn't work.
For the API method, I can see that the new Excel instance and file was created in the task manager, but when the code executed to the end of InitTimer function and returned to SetTimer function, the new Excel instance was killed and therefore no timer can be initialised by the next few lines of code. I checked the temp directory and found the Excel file was properly initialised and saved.
For the native method, once the Demo code is run, the Excel just stuck in infinite loop.
Is there any thing or dependency I need to setup before either method works? Thank you in advance for your advice.

Hi @dleunghk ,
There should be nothing to setup (no dependency). It should work directly. Well, it worked for all of the computers I tested on. It might be that something is specific to your configuration or maybe an antivirus is in the way.
Have you downloaded the Demo book and ran the demo or did you import the modules into a bigger project? Just trying to isolate where the problem is.

Forgot to ask, where do you use the timers? What do you need them for? I ask because I might suggest alternatives based on the specific need.

Dear Cristian,
Thanks for the advice, I downloaded both Demo books, and it seems that the native one worked, while the API one still didn't.  I may need some time to compare with the two files I created through .bas and .cls import.
Actually I have an Excel which streams real time stock data and I need a timer to run every 20 seconds to do some calculations on the data.
I am currently using the API callback which crash Excel often whenever I edited the VBA code when the timer was running.  And whenever I debug and break any VBA code, the timer cannot be stopped by KillTimer.
Will let you know why my files didn't work.  Thanks again for your help and your excellent safe timer codes.
Rgds,Darik    Cristian Buse @.***>) 在 2022年12月12日 星期一 GMT 下午12:19:38 寫道：
Hi @dleunghk ,
There should be nothing to setup (no dependency). It should work directly. Well, it worked for all of the computers I tested on. It might be that something is specific to your configuration or maybe an antivirus is in the way.
Have you downloaded the Demo book and ran the demo or did you import the modules into a bigger project? Just trying to isolate where the problem is.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Hi @dleunghk ,
If you don't need millisecond accuracy and you're already using Excel then I wonder why don't you just use Application.OnTime.


run: fix sandbox-side metadata service to comply to appc v0.8.1

This fixes coreos/rkt#2621 by bringing the metadata service in compliance with the Appc spec v0.8.1.
It also fixes a few small encoding issues in the sandbox-side service.

There is an active failing test. This is because the annotations spec has changed in a non-backwards-compatible manner.
Before this is merged, is it OK to break the API?

I see we are inheriting a versioned (v1) API endpoint from ACE spec which has not been bumped for v0.8.1, so I fear there is not much we can do here except breaking API. We are arguably just aligning to an errata for the original spec.
But I understand this will probably break some consumers. I guess for the future we may consider adding an endpoint returning appc version, to let consumers gracefully handle this kind of quirks.

Something is racy with this test. I'm investigating. The poststop command is failing to execute.

@squeed for some reason, merged rkt commits are visible in this PR

The Jekins failures are due to #2870. The question, then, is should we stop testing on poststop?

The Jekins failures are due to #2870. The question, then, is should we stop testing on poststop?

I guess that's better than not testing anything...

OK, once this is reviewed, it is ready to merge.

@squeed can you change the title of https://github.com/coreos/rkt/pull/2863/commits/e35b6410607f8e77d2895b6c4a76af552367e053 to comply to the contributing guidelines?

LGTM after the nits are fixed.

Test failure due to #2287, merging.


server: use IDAllocator to let balancer easy test.

Now  Balancer can only depend on ClusterInfo, we can test whole
balancer mechanism independently.
@qiuyesuifeng @disksing @shenli

LGTM


Reviews import revisited

Adapted reviews importer tool to new reviews schema.

Security audit should be cleared in an external Pr


Ignore duplicates on import_from_iterable

Hello,
is there a way to ignore/not insert entries with duplicate primary keys when using import_from_iterable?
Best regards
Niklaus

@kst83 , i think the easiest way to do this is to import into staging table first. And apply DISTINCT on Exasol side afterwards. It's normally much faster than sorting anything in Python.
C.set_autocommit(False)
C.execute("CREATE OR REPLACE TABLE staging_my_table LIKE my_table")
C.import_from_iterable(src, 'staging_my_table')

C.execute('INSERT INTO my_table SELECT DISTINCT * FROM staging_my_table')
C.commit()

@kst83 , alternatively, for PRIMARY KEY column you may do this:
C.import_from_iterable(src, 'my_table')
C.execute('''
DELETE FROM my_table
WHERE rowid IN (
	SELECT rowid
	FROM (
		SELECT rowid, ROW_NUMBER() OVER (PARTITION BY <primary_key> ORDER BY <ts_col> DESC) AS row_number WITH INVALID PRIMARY KEY (<primary_key>)
		FROM my_table
	) a
	WHERE row_number>1
);
''')

It finds rows with duplicate PRIMARY KEY values, keeps the row with latest <ts_col> and deletes all other duplicate rows.

@kst83 , one of my colleaguse suggested a pure Python solution for small lists.
my_list = [(2006, "White Pony", 3490349),(2006, "White Pony", 9492423),(2009, "White Pony", 4342342),(2006, "Red Pony", 3490349),(2006, "White Swan", 9492423),(2009, "White Swan", 4342342)]

seen = set() #< keep track of what we have seen as we go
unique_list = [x for x in my_list if not (x[1] in seen or seen.add(x[1]))]

print unique_list
# [(2006, 'White Pony', 3490349), (2006, 'Red Pony', 3490349), (2006, 'White Swan', 9492423)]

Credits: https://stackoverflow.com/questions/31482286/semi-unique-tuples-in-python-aka-tuple-primary-keys
Of course, it may run out of memory and much slower than any Exasol-based solution. But it should be fine for small lists less than 10k rows.

I guess, we may close this issue.


Moved: Pirinel.GitBlade version <IP_ADDRESS>

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

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[X] Does your manifest conform to the 1.5 schema?

Microsoft Reviewers: Open in CodeFlow

Manual Validation ended with:

Is a dependency missing?

idk it works all fine on my VM even without Git installed.


Unable to find installed packages

Hi @gerbenoostra !
Thanks for the plugin, but currently I am unable to get this to work and would appreciate your help.
I am working in a polyglot monorepo with the following simplified layout:
monorepo/
  -- libs/python/utils
  -- services/pythonapp

I have two pyproject.toml files in each of the lib and service, the service is referencing via paths the lib and works ok locally. The issue is during my docker build:
FROM python:3.11-slim AS build

WORKDIR /app

COPY services/test-py-app ./app
COPY lib/python/utils ./utils

RUN pip install poetry && poetry config virtualenvs.create false && pip install poetry-plugin-mono-repo-deps
RUN cd utils && poetry build && poetry install
RUN cd ../app && poetry install

The issue is on the last line where I get:
Path /usr/lib/python/utils for ag-utils does not exist
Skipping virtualenv creation, as specified in config file.
Replacing path dependency ag-utils @ file:///usr/lib/python/utils in group main with ag-utils (>=0.0.1,<0.1.0)
Installing dependencies from lock file

Package operations: 0 installs, 1 update, 0 removals

  - Updating ag-utils (0.0.1 /app/utils -> 0.0.1): Failed

  RuntimeError

  Unable to find installation candidates for ag-utils (0.0.1)

  at /usr/local/lib/python3.11/site-packages/poetry/installation/chooser.py:74 in choose_for
       70│
       71│             links.append(link)
       72│
       73│         if not links:
    →  74│             raise RuntimeError(f"Unable to find installation candidates for {package}")
       75│
       76│         # Get the best link
       77│         chosen = max(links, key=lambda link: self._sort_key(package, link))
       78│

Cannot install ag-utils.

Which makes sense as how would poetry know this package is already installed on the machine?
It seems to me I am not quite grasping the correct usage of this plugin :(

The goal of the plugin is to have poetry build create wheels and tar.gz files that are pip installable. Thus a poetry build doesn't have any effect to a subsequent  poetry install.
You could have a 2-stage Dockerfile, where you do the following in the build step (or you build the tar.gz/whl files outside docker in your CI/CD script):
FROM python:3.11-slim AS build
WORKDIR /app
COPY services ./services
COPY lib ./lib

RUN pip install poetry && poetry config virtualenvs.create false && pip install poetry-plugin-mono-repo-deps
RUN cd services/test-py-app && poetry build
RUN cd lib/python/utils && poetry build

And in the following step you then install those wheels:
FROM python:3.11-slim AS app
WORKDIR /app
COPY --from=build /app/services/test-py-app/dist/*.whl /tmp/
COPY --from=build /app/lib/python/utils/dist/*.whl /tmp/
RUN pip install --pre --no-cache-dir --find-links /tmp/ test-py-app


2FA login support

Fix #13

Pull Request Test Coverage Report for Build 418604931

150 of 223 (67.26%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.5%) to 33.682%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

telegram/create_auth_key.go
0
1
0.0%

internal/crypto/bytes.go
22
39
56.41%

telegram/srp.go
0
21
0.0%

internal/crypto/srp.go
128
162
79.01%

Totals

Change from base Build 415608790:
1.5%

Covered Lines:
1770

Relevant Lines:
5255

💛  - Coveralls

Looks good


Exponent for right hand Y Axis disappears if left-hand Y Axis doesn't have one.

An exponent on the right hand Y axis isn't detected properly by export_fig, and so removed by the set_tick_mode function. Here's a minimal code to reproduce this error:
x = 1:0.1:10;
y = sin(x).^2;
figure(1)
clf(1)
h_ax = gca();
plot(h_ax,x,y,'-')
hold(h_ax,'on')
yyaxis(h_ax,'right')
plot(h_ax,x,y*1e6,'--')
hold(h_ax,'off')
export_fig(1,'test.png','-png')

In this example, data exists on both left and right axes, but only that on the right is scaled by an exponent on the axis.
When set_tick_mode is called for 'Y', only the left axis is looked at, and so the exponent on the right is missed. This could be fixed by changing the check of hAxes.([ax 'Ruler']).SecondaryLabel.String (line 1368) to looking at get(h_ax.YAxis,'Exponent'). This returns a single number if there is only one Y Axis, or a cell array if there are two.
However, as this is in R2017b, I don't know how badly this would break any backwards compatibility. I think this is one area in which Matlab has been changed significantly in recent versions.

A fix appears to be changing line 1368 from:
if isempty(strtrim(hAxes.([ax 'Ruler']).SecondaryLabel.String))

to:
c = get(hAxes.YAxis,'Exponent');
if (~iscell(c)&&eq(c,0)||iscell(c)&&all(cellfun(@(x)(eq(x,0)),c)))

This oneliner handles both the single number and cell inputs using short circuit logicals.

Fixed (in a different manner than yours, but presumably just as effective)


Merge main -> google

9c3282cc Make cmake-bazel output color for the CI (#4220)
1a8a0f37 Merge pull request #4241 from NatashaKnk/google-to-main
552d19b6 Some tweaks to Python and e2e benchmarking docs (#4249)
6f633625 Merge pull request #4225 from google/benvanik-flow-workgroups
2e7a68a3 Updating -iree-hal-materialize-interfaces to support workgroup ops.
2eec75ce Adding hal.interface.binding.subspan op.
8dbf60ef Adding -iree-flow-outline-dispatch-regions2 pass.
415c28e7 Adding -iree-flow-inject-dispatch-tracing pass. This will replace the existing..
204c599d Cleanup renaming/includes/todos.
700a7714 Adding flow.dispatch.workgroups op family. Progress on #4140.
6f1afa5e Delete llvmjit compiler target and HAL backend. (#4152)
ff93f5e3 [NFC] Remove unused functions and unneeded dump. (#4234)
65435727 Rename MobileNetV3 to MobileNetV3Small in docs (#4240)

Was just debugging clang-format weirdness for Natasha


TypeError: 'NoneType' object is not iterable (graphite-web 9.12)

I was running into this error with certain graphs. The solution was to edit /opt/graphite/webapp/graphite/render/views.py
if cachedData is not None:
  requestContext['data'] = data = cachedData
else: # Have to actually retrieve the data now
  for target in requestOptions['targets']:
    t = time()
    seriesList = evaluateTarget(requestContext, target)
    log.rendering("Retrieval of %s took %.6f" % (target, time() - t))
    # added this conditional to avoid broken graphs
    if seriesList is not None:
            data.extend(seriesList)

Fixed by #1341.


Fix ASAN error with uninitialized core_dump_enabled_ in test constructor

Risk Level: Low
Testing: Unit Tests
Docs Changes: N/A
Release Notes: N/A
Platform Specific Features: N/A
Signed-off-by: Yan Avlasov<EMAIL_ADDRESS>[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Deprecated:]
[Optional API Considerations:]

/retest


fix: cross namespace gateway test cases

This PR fix issuse #45865 by deploy the cross namespace secrets into the correct namespace, and then add two cases for tls gateway routing.

Fix cross namespace secrets deployment and add a test case for cross namespace

use ingressutil.CreateIngressKubeSecretInNamespace func to create credential into correct namesapce
add a ReferenceGrant to allow cross namespace secrets reference

Add a test case for same namespace

[ ] Ambient
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[X] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

Hi @w12379564. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/ok-to-test


xIPAddress: get-netipaddress erroraction stop

xipaddress will not assign a new ipv6 ip where dhcp is enabled. Error action is set to stop. If set to silentlycontinue the test-targetresource takes 30 seconds to complete.
$GetNetIPAddressSplat = @{
InterfaceAlias = $InterfaceAlias
AddressFamily = $AddressFamily
ErrorAction = 'Stop'
}
$currentIPs = @(Get-NetIPAddress @GetNetIPAddressSplat)
PowerShell DSC resource MSFT_xIPAddress  failed to execute Set-TargetResource functionality with error message: The
running command stopped because the preference variable "ErrorActionPreference" or common parameter is set to Stop: No
matching MSFT_NetIPAddress objects found by CIM query for instances of the ROOT/StandardCimv2/MSFT_NetIPAddress class
on the  CIM server: SELECT * FROM MSFT_NetIPAddress  WHERE ((InterfaceAlias LIKE 'Ethernet 2')) AND ((AddressFamily =
23)). Verify query parameters and retry.
+ CategoryInfo          : InvalidOperation: (:) [], CimException
+ FullyQualifiedErrorId : ProviderOperationExecutionFailure
+ PSComputerName        : localhost
Windows 2012 R2
xnetworking <IP_ADDRESS>

Hi @danSpotter - I see what you mean:

In Test-TargetResource the cmdlet Get-NetIPAddress is called with EA silentlycontinue.
In Set-TargetResource the cmdlet Get-NetIPAddress is called with EA stop.

I'm trying to think if there might be any consequence to using EA of silentlycontinue to the cmdlet Get-NetIPAddress in Set-TargetResource. The 30 second delay does sound a bit concerning as well. Does this 30 second delay only occur when first converting the IPv6 address from DHCP to static or does it occur every time?

I have a similar problem. I have a static IP address of <IP_ADDRESS>/24.  When I try to add a list of addresses, I get the message that the first address already exists and the error behavior is to stop.  When I change the behavior in the Set-TargetResource to Continue it works.
PS C:> invoke-dscresource -ModuleName xnetworking -name xipaddress -method set -Property @{ addressfamily  = "IPV4"; InterfaceAlias = "Ethernet0"; IPAddress = "10.
110.194.33/24", "<IP_ADDRESS>/24", "<IP_ADDRESS>/24", "<IP_ADDRESS>/24", "<IP_ADDRESS>/24", "<IP_ADDRESS>/24", "<IP_ADDRESS>/24"; }
AVERTISSEMENT : L'état du Gestionnaire de configuration local est modifié par des opérations non DSC. Si vous souhaitez modifier l'état du Gestionnaire de configuration local,
utilisez l'applet de commande Remove-DscConfigurationDocument.
La ressource DSC PowerShell MSFT_xIPAddress n'a pas pu exécuter la fonctionnalité Set-TargetResource. Message d'erreur: L’exécution de la commande s’est arrêtée, car la variable
de préférence «ErrorActionPreference» ou le paramètre courant a la valeur Stop: Instance MSFT_NetIPAddress already exists
+ CategoryInfo          : InvalidOperation : (root/Microsoft/...gurationManager:String) [], CimException
+ FullyQualifiedErrorId : ProviderOperationExecutionFailure
+ PSComputerName        : localhost

Hi @yvesgermain - have you tried the new IPAddress in NetworkingDsc 6.0.0 (xNetworking has been renamed to NetworkingDsc)? This fix (https://github.com/PowerShell/NetworkingDsc/pull/331) may have resolved your issue - but it went out in 6.0.0.


ArangoDB install error on raspberry pi of arm cpu

I'm using the latest ArangoDB(3.1.11) on raspberry pi of arm cpu.
Install is error.
The following is error information
# dpkg --install arangodb3-3.1.11-1_armhf.deb
Selecting previously unselected package arangodb3.
(Reading database ... 20813 files and directories currently installed.)
Preparing to unpack arangodb3-3.1.11-1_armhf.deb ...
Unpacking arangodb3 (3.1.11) ...

Processing triggers for man-db (<IP_ADDRESS>-5) ...
Errors were encountered while processing:
 arangodb3

i try command
# /etc/init.d/arangodb3 restart
[....] Restarting arangodb3 (via systemctl): arangodb3.serviceJob for arangodb3.service failed. See 'systemctl status arangodb3.service' and 'journalctl -xn' for details.
 failed!

and try
# systemctl status arangodb3.service`
arangodb3.service - LSB: arangodb
   Loaded: loaded (/etc/init.d/arangodb3)
   Active: failed (Result: exit-code) since Fri 2017-03-31 11:10:43 UTC; 6s ago
  Process: 946 ExecStart=/etc/init.d/arangodb3 start (code=exited, status=139)

Mar 31 11:10:43 Docker arangodb3[946]: Starting arango database server: arangodSegmentation fault
Mar 31 11:10:43 Docker arangodb3[946]: database version check failed, maybe you need to run 'upgrade'? ... failed!
Mar 31 11:10:43 Docker systemd[1]: arangodb3.service: control process exited, code=exited status=139
Mar 31 11:10:43 Docker systemd[1]: Failed to start LSB: arangodb.
Mar 31 11:10:43 Docker systemd[1]: Unit arangodb3.service entered failed state.

and try
# journalctl -xn
-- Logs begin at Fri 2017-03-31 09:17:01 UTC, end at Fri 2017-03-31 11:10:43 UTC. --
Mar 31 11:10:32 Docker sudo[927]: pam_unix(sudo:session): session opened for user root by pirate(uid=0)
Mar 31 11:10:32 Docker su[931]: Successful su for root by root
Mar 31 11:10:32 Docker su[931]: + /dev/pts/0 root:root
Mar 31 11:10:32 Docker su[931]: pam_unix(su:session): session opened for user root by pirate(uid=0)
Mar 31 11:10:41 Docker systemd[1]: Starting LSB: arangodb...
-- Subject: Unit arangodb3.service has begun with start-up
-- Defined-By: systemd
-- Support: http://lists.freedesktop.org/mailman/listinfo/systemd-devel
--
-- Unit arangodb3.service has begun starting up.
Mar 31 11:10:43 Docker arangodb3[946]: Starting arango database server: arangodSegmentation fault
Mar 31 11:10:43 Docker arangodb3[946]: database version check failed, maybe you need to run 'upgrade'? ... failed!
Mar 31 11:10:43 Docker systemd[1]: arangodb3.service: control process exited, code=exited status=139
Mar 31 11:10:43 Docker systemd[1]: Failed to start LSB: arangodb.
-- Subject: Unit arangodb3.service has failed
-- Defined-By: systemd
-- Support: http://lists.freedesktop.org/mailman/listinfo/systemd-devel
--
-- Unit arangodb3.service has failed.
--
-- The result is failed.
Mar 31 11:10:43 Docker systemd[1]: Unit arangodb3.service entered failed state.

Is there any idea that would cause this error?

Sorry, we currently can't offer support for the pi packages, they're discontinued again.

Can run arangodb on the docker of arm cpu ?

If you choose to run it via x86-64 emulation (does qemu/bochs support that nowadays?) - maybe.

There is a way to break this problem.
I tried on raspberry and then run docker.
Installed arm debian, in the installation arangodb3-3.1.11-1_armhf.deb
is fine.is work.
root@e0aa9471bb00:/# arangod
2017-03-31T15:43:44Z [3226] INFO ArangoDB 3.1.11 [linux] 32bit, using VPack 0.1.30, ICU 54.1, V8 <IP_ADDRESS>, OpenSSL 1.0.1t  3 May 2016
2017-03-31T15:43:44Z [3226] INFO using SSL options: SSL_OP_CIPHER_SERVER_PREFERENCE, SSL_OP_TLS_ROLLBACK_BUG
2017-03-31T15:43:44Z [3226] INFO Starting up with role SINGLE
2017-03-31T15:43:44Z [3226] INFO file-descriptors (nofiles) hard limit is 1048576, soft limit is 1048576
2017-03-31T15:43:44Z [3226] INFO Authentication is turned on
2017-03-31T15:43:44Z [3226] INFO Authentication system only
2017-03-31T15:43:44Z [3226] INFO Authentication for unix sockets is turned on
2017-03-31T15:43:44Z [3226] WARNING fcntl on lockfile '/var/lib/arangodb3/LOCK' failed: not implemented. a possible reason is that the filesystem does not support file-locking
2017-03-31T15:43:44Z [3226] FATAL database is locked, please check the lock file '/var/lib/arangodb3/LOCK'
root@e0aa9471bb00:/# arangosh
Please specify a password:

                                       _
  __ _ _ __ __ _ _ __   __ _  ___  ___| |__
 / _` | '__/ _` | '_ \ / _` |/ _ \/ __| '_ \
| (_| | | | (_| | | | | (_| | (_) \__ \ | | |
 \__,_|_|  \__,_|_| |_|\__, |\___/|___/_| |_|
                       |___/

arangosh (ArangoDB 3.1.11 [linux] 32bit, using VPack 0.1.30, ICU 54.1, V8 <IP_ADDRESS>, OpenSSL 1.0.1t  3 May 2016)
Copyright (c) ArangoDB GmbH

Pretty printing values.
Connected to ArangoDB 'http+tcp://<IP_ADDRESS>:8529' version: 3.1.11 [server], database: '_system', username: 'root'

Please note that a new bugfix version '3.1.16' is available
Type 'tutorial' for a tutorial or 'help' to see common examples
<IP_ADDRESS>:8529@_system>

Uf wiederluege! Na shledanou! Auf Wiedersehen! Bye Bye! Adiau! ¡Hasta luego! Εις το επανιδείν!
להתראות! Arrivederci! Tot ziens! Adjö! Au revoir! さようなら До свидания! Até Breve! خداحافظ!
root@e0aa9471bb00:/# exit
HypriotOS/armv7: pirate@Docker in ~
$

I share my way after a few days.
Hardware
raspberry Pi 3

As stated above, these debs currently won't work, and we currently can't offer support for arm binaries.


Why is isEmpty false even when all bits are set to 0?

a = new bs(new Uint8Array(32 / 8 - 1)).flip()
for (i = 0; i < 32; ++i) {
  a.set(i, 0);
}
a.isEmpty() // false?

It seems like it will work if I set the range on the flip: flip(0, 31).
I think I'm accustomed to fixed sizes bitsets and not arbitrary sized ones, I'm not understanding the usage  of the '_' property. It appears to change the behaviour of some functions.

The '_' property is simply an internal flag to store the MSB state on inversion. If for example .set(10, 1).not().get(10000) is executed, you would expect the result to be 1, since all the bits on higher indexed were turned to 1 from 0.

That means for your actual problem: The initial invocation of .flip() inverts '_' (using .not()) (BTW: I added a small note on the source file for the meaning of this flag). You then set the range from 0-31 to zero, using .set(), which doesn't alter the flag though. The same is true for flip, using an interval as parameter (flip(0, 31)). This does not change the infinity flag obviously, as only the small range has to be changed.


Preserve dataframe column order for relevant client operations

Linked issue(s):
What change does this PR introduce and why?
Preserve dataframe column order for relevant operations that client sdk perform for users. The one piece where order cannot be preserved is when snowflake is involved. Admittedly this impacts most of the meaningful operations in client sdk, however I think it still has value to maintain order as much as we can.
Also introduced some small refactor to reduce serde boilerplates.
Please check if the PR fulfills these requirements

[ ] Include reference to internal ticket and/or GitHub issue "Fixes #NNNN" (if applicable)
[x] Relevant tests for the changes have been added
[ ] Relevant docs have been added / updated

Do you have any potential strategies in mind for preserving ordering at the persistence layer? This is a pretty major rough edge already, and its impact will be worsened when values aren't split across e.g. test sample/ground truth/metadata objects.

Persisting a field_order column and applying this order when reading datapoints seems like a pretty clean solution, actually. Any hesitations with that?

Persisting a field_order column and applying this order when reading datapoints seems like a pretty clean solution, actually. Any hesitations with that?

Can put up a PR for that and check how it goes.

Can put up a PR for that and check how it goes.

Amazing. Longstanding ticket for reference: KOL-2959


Initial implementation of random content library

Initial implementation of the gem to generate random content for a schema.
See it in action here: https://github.com/alphagov/govuk-random-content-store/pull/1

Closing temporarily, picking this up next week.


chore(release): publish packages

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

To be honest, I'm still very unsure about the changes in #620. On heartbeat timeout, it just emits a CHANNEL_ERROR and CLOSED event, with the data always being null. So you have no idea why it closed.

@Vinzent03 At least that is how the JS client library works as well, and emitting the error there was causing unwanted errors to be thrown. Maybe we can look at the particular issue you are pointing out in another PR, but for now, this version ships more bug fixes than creating bugs.

Hmm okay ...


Strange port forwarding behavior

Environment
Windows build number: Microsoft Windows NT 10.0.19041.0
Your Distribution version: On a self-built WSL image, based on Fedora Rawhide
Whether the issue is on WSL 2 and/or WSL 1: WSL 2

Steps to reproduce

In Linux side, echo linux side > test in a certain folder
In Windows side, echo "Windows Side" > test in another folder
run python -m http.server or python -m http.server --bind <IP_ADDRESS> in Linux Side
run python -m http.server or python -m http.server --bind <IP_ADDRESS> in Windows side (you need python on Windows)
open another window in Windows Side, run curl <IP_ADDRESS>:8000/test the output indicates the request went to which side
kill those python webservers and restart them in different order, different command (overall 2x2x2=8 tries) and redo 5

Expected behavior
Emmmm. I don't mean the current behavior is wrong, but programs listening on Linux side can override Windows side in some cases, which used to confuse me a lot.
I noticed this because I run proxies on both WSL and Windows, listening on same port number, I sometimes find my chrome on Windows connects to proxy in WSL side. I have to kill the proxy instance or change port to fix this.
Actual behavior
The behavior is strange

When you run python -m http.server --bind <IP_ADDRESS> first, Linux side python -m http.server or python -m http.server --bind <IP_ADDRESS> second, you will connect to Windows side
All other case, you get to Linux side

The behavior is strange because you've bound the same port to the same address.
/dupe #4619


RayCast2D and CollisionPolygon2D collisions are unreliable

Godot version:
3.2.1
Issue description:
The Sprite becomes red when the raycast's is_colliding() returns true.

As you can see, raycast will detect collision only with the edge of polygon's quad and will fail inside.
Minimal reproduction project:
ReproductionProject.zip
EDIT:
Also tested with rectangle CollisionShape2D and it works correctly, so it's indeed a bug with polygons.

CollisionPolygon2D decomposes into multiple ConvexShape2D shapes when in "solids" mode:
https://github.com/godotengine/godot/blob/master/scene/2d/collision_polygon_2d.cpp#L49-L59
To determine whether a shape intersects a raycast, CollisionSolver2DSW::solve_raycast uses Shape2DSW::intersects_segment:
https://github.com/godotengine/godot/blob/00949f0c5fcc6a4f8382a4a97d5591fd9ec380f8/servers/physics_2d/collision_solver_2d_sw.cpp#L96
Unfortunately, ConvexPolygonShape2DSW::intersect_segment does not check if both points are inside:
https://github.com/godotengine/godot/blob/master/servers/physics_2d/shape_2d_sw.cpp#L559
Probably such a check could be done after failing to intersect any of the lines using Geometry::is_point_in_polygon. Given that is_point_in_polygon does the check using any point definitely outside of the polygon, maybe some part of it can be merged with the loop in intersect_segment. Maybe even doing something like further_away = get_aabb().position - Vector2(1, 1), as any point outside the AABB of the polygon is definitely outside the polygon.

See also #38343, #35845.

Any workarounds?

@knightofiam Area2D or KinematicBody2D and test_move()

Still valid for v4.0.dev.custom_build [a0d800e96]
That could be said to be a perfomance feature, but then the mode probably should not be called a solid.


A bug in the 'group' transform

In the documentation, the syntax of group is group [{key_columns}] {pipeline}.
However, I found it only supports the inner pipeline in aggregate type. I tried derive, select, sort and filter, I found group doesn't work. Here is an example:
Given prql query like this:
from tb1
group [c2] (derive a1 = c0 + c1)

The SQL query I get from both online prql playground and prql-python is shown below:
SELECT
  *,
  c0 + c1 AS a1
FROM
  tb1

-- Generated by PRQL compiler version 0.4.2 (https://prql-lang.org)

I think the right answer should be like this:
SELECT
  *,
  c0 + c1 AS a1
FROM
  tb1
GROUP BY
  c2

The same bug exists when I replace derive with select, sort, filter, and take.
Given another example:
from tb1
group [c2] (select t1 = 3 + c0)

The SQL generated is like this:
SELECT
  3 + c0 AS t1
FROM
  tb1

-- Generated by PRQL compiler version 0.4.2 (https://prql-lang.org)

The right answer should be:
SELECT
  c2
  3 + c0 AS t1
FROM
  tb1
GROUP BY
  c2

I think it only supports the inner pipeline in the type of aggregate currently.

The version I tested is 'PRQL compiler version 0.4.2'

This is intended behavior.
In SQL, adding a GROUP clause signals aggregation for each of the groups.
In PRQL, group does not do aggregation by default.
Your example:
group [c2] (derive a1 = c0 + c1)

... would mean:

for each group,
compute a new column a1 by adding c0 and c1,
result for each group would be the initial group, but with an extra column,
in the end, combine all groups back into a single relation.

Now, computing a new column is not affected by group at all, since it operates on each row separately.
Does that make sense?

Thanks for the issue @YuxiLing

SELECT
*,
c0 + c1 AS a1
FROM
tb1
GROUP BY
c2

What would this SQL be doing? Generally GROUP BY would be using some aggregation like SUM.
As @aljazerzen 's points out, this is an area where PRQL makes a deliberate change from SQL, in that aggregations are explicit. But possibly we should be raising errors rather than producing SQL that is unlikely to work. I can see why the current behavior is confusing.

Thanks for your clarification.
Just one final tip from my side: maybe you could mention this difference between SQL group by and PRQL group in your PRQL Language Book.
Nevermind. Thank you so much.

Just one final tip from my side: maybe you could mention this difference between SQL group by and PRQL group in your PRQL Language Book.

Yes, strongly agree, we put something in aggregate here: https://prql-lang.org/book/transforms/aggregate.html#aggregate-is-required, and have a long section in our FAQ under "What’s going on with this aggregate function?"
If we had something similar in our group docs, would that help @YuxiLing ? Or is it still confusing?

Is there ever a case for GROUP BY without a function in the SELECT statement? If not, possibly we should error? Particularly in these areas where we're a bit different to SQL, raising an error gives us a chance to explain what's going on in an error message.

Thanks. FAQ clarifies this problem clearly.

@YuxiLing if it's OK, I'll reopen this for this question. I still think PRQL could do a better job here.

Is there ever a case for GROUP BY without a function in the SELECT statement? If not, possibly we should error? Particularly in these areas where we're a bit different to SQL, raising an error gives us a chance to explain what's going on in an error message.

Yes, there is a case:
from tracks
group album_id (take 1)

There was even discussion in #1457 about how this:
func no_op rel -> rel

from tracks
group album_id no_op

... should just sort the rows so that all album_ids are next to each other. This could be accomplished with sorting by album_id.
Also, if you'd have:
from tracks
group album_id (sort title)

... I believe this should produce ORDER BY album_id, title.

Yes interesting. Arguably (take 1) is an aggregation function. But the other cases are at least ambiguous enough that this isn't an easy situation where we can just raise.
So I'll close...


Deploy HDInsight using REST API

If I need CLI to login and perform other tasks before I could call REST API. Whats the point using REST API to deploy anything? Why not simply use CLI to perform everything.
If we want to deploy HDInsight (or anything) using REST API, provide a simple way to:

Login to Azure (REST POST)
Deploy using JSON (REST POST)
Check Status (REST GET)

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

ID: 49d3c731-c2cc-bda7-4d93-89d17cf54ccf
Version Independent ID: 8a91fb59-2355-8e20-948d-405ffb6b0af7
Content: Create Apache Hadoop clusters using Azure REST API - Azure
Content Source: articles/hdinsight/hdinsight-hadoop-create-linux-clusters-curl-rest.md
Service: hdinsight
GitHub Login: @hrasheed-msft
Microsoft Alias: hrasheed

Thanks for the ask  . At this time we are reviewing the ask and we will reply back as soon as we can .

Thank you!

@eyedia : You can use the bearer token to get your call authenticated and use the token in all susequent API calls . This has been the design which I have seen on many the API calls on Azure and many  of the other cloud providers . Some code is placed here which calls that out .
https://docs.microsoft.com/en-us/azure/hdinsight/hdinsight-hadoop-create-linux-clusters-curl-rest#get-an-authentication-token
Ay this time I am going ahead and closing the issue . You can always re-open that if you think thats appropiate .

Thanks @himanshusinha-msft , let me check that out


Fix black screen after game exit

Credit : @jbdemonte

do you have a link on the orignal commit please ?
do you have tested it ?

Link : https://gitlab.com/recalbox/recalbox-emulationstation/merge_requests/80


Disable arm64 container signing

Disable arm64 container signing to align with Hydra PR 9769272.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2400

Relevant Lines:
3967

💛  - Coveralls


UTF-8 encoding support for credential attributes

When submitting a credential offer using uncommon characters (such as an emoji, as in the provided example), the Traction tenant responds with a 400 error and hangs the issuance process.
Private Zenhub Image
This might be indicative of incorrect handling of character encoding, and should be verified. In particular with Traction, things to check are:

that the tenant-proxy is not modifying the charset encoding for requests being proxied
that ACA-Py supports UTF-8 as (default?) encoding for plaintext credential attributes
that Traction plug-ins support UTF-8 as (default?) encoding when handling requests

This issue is to validate and understand the current situation, and determine next steps.
It is a feature required, specifically, by BC Wallet: https://github.com/bcgov/bc-wallet-mobile/issues/1503

I used the Traction Tenant UI to issue myself a credential and assigned one of the attributes as 😅  - everything went as expected and I was able to correctly store the credential in the BC Wallet.

Was just trying this myself too, at least through the Tenant UI > Traction Proxy > My BC Wallet I got other characters transferred through in both the Attribute Name and the Attribute values

Closing as it doesn't seem to be an issue, please feel free to reopen if necessary @jleach


Presentation

I just stumbled across your new library and like the humor. :) Good work so far!
Are you going to implement some kind of "presentation layer"? What I mean is: A user can choose a "flavor" like "Bootstrap" or similar and the lib just adds the appropriate class names to the elements. I know that this is not much fun, but I think it would help to create good looking forms with nearly no effort.
Edit: I think the "flavor" would / should / could make use of already existing methods like $x->setBeforeEach('<div class="form-row">')
You could also name it "topping" so it makes more sense if the user calls it at the end of the form creation. :D

Yes, this is something I want to do.
We already have Blends e.g. https://github.com/soatok/cupcake/blob/master/src/Blends/MultiCheckbox.php
Before I get to the higher-level presentation stuff, I wanted to make sure the low-level functionality covers 100% of the HTML5 Form specification. Then, I want to make it easy to dive in and manage these elements (think jQuery versus 2008-era vanilla JavaScript). Once the utility is completely implemented, my next focus was going to be on convenience methods.

This is great news! I am interested in replacing my own FormBuilder, and this is the first alternative I like. :D
A special feature of my own form builder is the possibility to store the configuration of the form completely as an array. A field definition may look like this:
    array(
        'type'        => 'select',
        'label'       => 'Update frequency',
        'name'        => 'sitemap_update_freq',
        'options'     => array(
            'always', 'hourly', 'daily', 'weekly', 'monthly', 'yearly', 'never'
        ),
        'selected'    => 'weekly',
        'class'       => 'fbleave',
    )

There are more options like "required", "value" (set a default value), "allow" (attach validation) or even "title" (some description to be added as "title" attribute). Should be easy to create importers for Cupcake, first from JSON and PHP array. Later there can be importers from database and other sources, too. Would you prefer this to be a separate project? Are you interested anyway?


Missing VK_EXT_physical_device_drm

Swaywm fails to run its vulkan renderer. According to the error displayed, AMDVLK is missing 'VK_EXT_physical_device_drm'.
Logs on the Related issue.

The extension is enabled in 2023.Q1.2


Adding environment variables

Hi,
It seems that config/dev.env.js or config/prod.env.js is not used for adding environment variables anymore! Is my understanding true? Am I supposed to use chainWebpack in vue.config.js to add them?
BEST
Afshin

Hi Afshin,
Thanks for telling me your thoughts. Franky, i haven't worked on this repo for quite a while. I totally forgot what I did, LOL. It's just a pet project. Don't be too serious. Please feel free to update what you need.
Cheers,Harry
Sent from Yahoo Mail on Android
On Wed, 17 Feb 2021 at 9:30 pm<EMAIL_ADDRESS>wrote:
Hi,
It seems that config/dev.env.js or config/prod.env.js is not used for adding environment variables anymore! Is my understanding true? Am I supposed to use chainWebpack in vue.config.js to add them?
BEST
Afshin
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


heat.resize() not working

The canvas element when you do heat.resize() works, but the heatmap points still with the same coordinates as origin, and do not update. Any ideas?
In detail:
In origin, canvas size is 300x150, but then when dinamically image is loaded, I resize heatmap to 700x700, the canvas is successfully updated but the heatmap points are drawed in a 300x150 canvas.

Hi, I have submitted a pull request to solve this bug
https://github.com/mourner/simpleheat/pull/36


Symmetric key sharing

Allow users to securely share symmetric keys over the grd.me servers (assume the grd.me servers are available but otherwise untrusted).

@arnottcr @floryst  do you think it's important to let the sender know whether or not the recipient specifically accepted their key versus acknowledging that their key had been received? The latter is considerably easier to implement and technically the receiver could accept and then delete their key, and the sender would just get the notification that they'd accepted their key, so I feel like it's safe to go with the latter. What are your thoughts?

For documentation's sake: I don't (though I can) notify the user that their key has been received.

That should be alright, we can see if it causes an issue.

This issue was moved to GrdMe/GrdMe#15

This issue was moved to GrdMe/GrdMe#15


No Success Test for CFE_TBL_Update

Is your feature request related to a problem? Please describe.
From #548. No Success Test for CFE_TBL_Update.
Describe the solution you'd like
Add a success test for CFE_TBL_Update in cFE/modules/tbl/ut-coverage/tbl_UT.c.
Requester Info
Ariel Adams, ASRC Federal

I'll work on this.


Azure Publish Error - bundle.js

Hi,
I've published my app to azure app service like it described in the documentation.
I've also setup all environment parameters in the "application settings" section on azure portal

NODE_EV: production
TOKEN_SECRET: xxxx
MONGODB: mongo://xxxxx

When I open my web app I am not able to login into the app. I know there can be a issue with the mongodb connection but the strange thing is that I get no server message response in my web app when I leave the sign up fields empty. It should show me the messages "Name cannot be blank" and so on.
I've tried to find out if the service for login available and yes it is avail when I request it via postman.
POST/ http://boiler-with-db.azurewebsites.net/login
[
  {
    "param": "email",
    "msg": "Email is not valid"
  },
  {
    "param": "email",
    "msg": "Email cannot be blank"
  },
  {
    "param": "password",
    "msg": "Password cannot be blank"
  }
]

What I also observed is an error in the developer console:
bundle.js:1 Uncaught SyntaxError: Unexpected token <
Failed to parse SourceMap: http://boiler-with-db.azurewebsites.net/css/vendor/bootstrap.css.map
It looks like the bundle.js is empty and only included the 404 page code.
Can anybody help me with that issue?

my configuration:
Platform: node
Framework: express
Template Engine: jade
CSS Framework: bootstrap
CSS Preprocessor: css
JavaScript Framework: react
Build Tool: webpack
Unit Testing: none
Database: MongoDB
Authentication: email, facebook, twitter, google
Deployment: azure

I suspect you are having the same issue that I had with megaboilerplate site when it was hosted on Azure.
The reason why you are getting bundle.js:1 Uncaught SyntaxError: Unexpected token < is because it cannot find bundle.js file. It cannot find it because the file is never being generated on Azure.
I'll take a look at the azure deployment script again, but meanwhile you should compile bundle.js locally via npm run build and commit it to Git.

Many thanks for your help. Will do it like you described!


cmd/lncli: add unused flag

Change Description
Added unused flag to NewAddr command
Steps to Test
lncli  newaddress --unused=true
Pull Request Checklist
Testing

[ ] Your PR passes all CI checks.
[ ] Tests covering the positive and negative (error paths) are included.
[ ] Bug fixes contain tests triggering the bug to prevent regressions.

Code Style and Documentation

[ ] The change obeys the Code Documentation and Commenting guidelines, and lines wrap at 80.
[ ] Commits follow the Ideal Git Commit Structure.
[ ] Any new logging statements use an appropriate subsystem and logging level.
[ ]  There is a change description in the release notes, or [skip ci] in the commit message for small changes.

📝 Please see our  Contribution Guidelines for further guidance.

Needs an entry in the release notes (0.15.0) too.

@guggero sorry what does this mean?

Please go through the checklist in the pull request body. There's a link there.

@ellemouton, @guggero is there more I should fix on this?


Obligations of Partner Codes

Should there be lists of light to full obligations for partner codes, both on the JSON-producing and JSON-consuming side?
for example,

outputs QM portion of Molecule into JSON (QC program)
can run a geometry optimization from a JSON input (QC program)
tests its JSON input/output alongside usual testing scheme (ALL)
plots orbitals from JSON schema basis sets and densities (Viz program)
returns JSON vibrational modes from JSON Hessian (vibrational analysis program)

It would be great to develop a notion on the level of support offered, maybe basic, intermediate, advanced, and complete or similar.

I like the idea of constructing a list of thing that a program must, do and then attaching them to tiers. Something like:

A: Verifies input JSON for completeness
B: Exports energy/gradient/Hessian quantities
C: Runs QM "workflows" (optimization, response, spectra, etc)
D: Exports Wavefunction quantities (orbitals, eigenvalues, densities)
E: Convert atomic-orbital quantities to a common representation
F: Adhere to a common set of input and output variables

To meet the various levels of interoperability QM programs must do:
Level 1 - A, B, C
Level 2 - A, B, C, F
Level 3 - A, B, C, D, E, F
The two definition approach could be useful as some programs could offer items of their "interoperability level".

This sounds like the right direction to me, maybe reorder so that each level is ABC, ABCD, ABCDEF :-)


Using require to inject file contents into JS files causes issues with unit tests

We cannot write unit tests for some files if using code as follows:
const addRunCellHook = require('../../pythonFiles/vscode_datascience_helpers/kernel/addRunCellHook.py');

We might want to consider removing such webpack specific code from our code and use some other way to inject the file contents.
Or we need way another way to write unit tests for such files.
Or we could have a simple provider that provides the contents of these files.
We end up duplicating the files (its in the source code as well as vsix)

Not sure why this breaks anything? Does the require fail in a unit test?

Does the require fail in a unit test?

Yes.
Can we not inject he IFileSystem to read the file contents using the Uri instead of using require.

Hmm, I think the file system could read it from the extension URI. I don't think these need to be embedded.

Yes agreed,
was writing some tests for IKernelProvider and found bugs (bugs in the actual code),
however the web scenario isn't covered by tests due to this issue
I would feel lot more comfortable knowing the web version too has some tests.

@sadasant's other fix in this area should be applicable.


fix(tests): nested produce calls

Add tests for nested produce calls with draft objects. Previously, only non-drafts were tested.

Coverage increased (+0.7%) to 97.049% when pulling f1d7ec2428336c395a64442e674c349d20585902 on aleclarson:patch2 into 4613b0685e7dcadf1aa1bde04cb1ac3f4e724cb4 on mweststrate:master.


The live demo is broken

https://googlecodelabs.github.io/polymer-offline-weather/final/index.html
when I open the console I see:
The FetchEvent for "https://query.yahooapis.com/v1/public/yql?format=json&q=select * from%2… WHERE text%3D"37.4209397%2C-122.0934599" and gflags%3D"R")" resulted in a network error response: an object that was not a Response was passed to respondWith().
https://query.yahooapis.com/v1/public/yql?format=json&q=select * from%2…20WHERE text%3D"37.4209397%2C-122.0934599" and gflags%3D"R") Failed to load resource: net::ERR_FAILED

up, can anyone fix this?

@jeffposnick Can you have a look?

Thanks for the ping—somehow I wasn't already Watching this repo 😦
I believe something was deprecated or otherwise changed with the underlying API calls. I'll try the approach mentioned at http://stackoverflow.com/questions/34904938/yahoo-yql-query-with-gflags-returns-nothing to see if that helps.

Glad to help, I'll try that

I've tried using geoplaces(1) but it returns useless data for what the purpose is here. There has to be a better query using lat-long or maybe we need to first get the location and then query that later.

Right, I think it might need to happen in two separate API calls, as described in this answer: http://stackoverflow.com/a/34920208/385997

actually nevermind, I didn't remove gflags. It now returns
https://query.yahooapis.com/v1/public/yql?format=json&q=select * from weather.forecast where woeid in (SELECT woeid FROM geo.places(1) WHERE text%3D"(51.504500699999994%2C-0.035201199999999995)")
correctly but it needs to reassign all the variables with this one.

sent you guys a pull request


[Bug] Wallet auto-login not working when using an encrypted keyring

What happened?
When configuring a wallet to auto-login while using an encrypted keyring the auto-login is not working. Instead after starting the chia client and entering the keyring passphrase the wallet selection screen is shown always.
Version
1.8.1
What platform are you using?
Windows
What ui mode are you using?
GUI
Relevant log output
No response

Bug still exists

@seeden can you take a look? I'm able to repro using these steps:

Set "Enable Auto Login" to on
Set or change the passphrase for your keyring -- make sure that the "Save passphrase" checkbox is unchecked
Quit Chia and let all services stop
Launch Chia

As mentioned in the report, after the keyring passphrase is provided, the SelectKey screen is displayed.
The core issue is that the GUI is starting the wallet (via start_service) before the keyring has been unlocked. If you launch chia services from the CLI, the daemon waits for the keyring to become unlocked before starting any services. So for example:
Daemon not started yet
Starting daemon
(Unlock Keyring) Passphrase:
Unlocking daemon keyring
chia_harvester: started
chia_farmer: started
chia_full_node: started
chia_wallet: started```

Arguably the wallet service should wait for the keyring to be unlocked before starting, since the WalletNode can't be fully  instantiated without the wallet key, but that would likely be a significant change.

I verified everything in GUI, and here is the current state:

GUI is waiting until keyringStatus.isKeyringLocked === false
GUI will run services only when isKeyringLocked === false

I also tried to wait 30 seconds after the keyring unlock but nothing changed.
My second thought was to invalidate loggedInFingerprint but it is always null.
I think that GUI is doing a correct job and there is no issue.
Maybe it is somehow related do the backend and it is doing something else because we are using only RPC.
@paninaro do you have any idea what can be wrong?

Fixed via https://github.com/Chia-Network/chia-blockchain-gui/pull/1982


Improve code block higlighting colors in docs

Currently the color scheme for code block highlighting just doesnt look right to me:

while i dont think red and green should never be seen its probably not the best visual contrast and is known to be troubling to people with color blindness and/or dyslexia
i also think red should generally not be used for anything other than errors

Agree

Also agree. Not good from an accessibility and color blindness point of view. And not great aesthetically

Preview of various styles:
https://www.florian-diesch.de/doc/sphinx/themes-and-pygments-styles/index.html#theme-default

https://dt.iki.fi/pygments-gallery


Can not hide checkboxes for certain nodes in KryptonTreeView

After changing a regular TreeView to a KryptonTreeView, hiding check boxes for nodes does not work.
This is code that I use http://dotnetfollower.com/wordpress/2011/05/winforms-treeview-hide-checkbox-of-treenode/.
Does anyone have idea why this does not work?

https://github.com/Wagnerp/Krypton-NET-5.470
https://github.com/Wagnerp/Krypton-NET-5.470#2019-05-20-build-1103---bugfixes
Fixed issue #179, KryptonTreeView with CheckBox and Image does not leave a "Gap"
Fixed issue #176, Can not hide checkboxes for nodes in KryptonTreeView


Missing non-dependent browser integration test for botframework-streaming

Describe the bug
botframework-streaming should have its own browser integration tests that are not reliant on any dependent libraries (e.g. botframework-directlinejs, botframework-webchat) or dependent's tool chains.
Expected behavior

[ ] browser-based app using botframework-streaming should build

[ ] without/with the use of browserify, parcel, rollup, webpack

[ ] no runtime errors in app from botframework-streaming

tool
test exists?

nothing
❌

browserify
❌

parcel
❌

rollup
❌

webpack
❌

[bug]

With the current CI configuration and streaming E2E test, this is no longer a P0 and has been placed on the backlog for now.


ARC-33 Becoming an xGov

This ARC will be used to standardize the way to become an xGov.
Questions that need to be answered have been listed in the Rationale part.
Please refer to this part and give your thought.

Hey governors,
Benedetto Biondi here, CEO of Folks Finance.
Thanks to the AF for opening this forum for discussions. At Folks we have been reasoning about a potential pilot program to start and run xGov quickly.
The idea is very straightforward.
Algofi and Folks Finance run a decentralized voting system:

https://app.folks.finance/algo-liquid-governance
https://gov.algofi.org/

FF and Algofi provide DeFi governance participation solutions, so the protocols communities are already directly interested in AF governance measures.
I recently discussed with John Clarke, Algofi Co-Founder, and we both agree that we could run a pilot program that would work as follows:

Select an xGov manager for each protocol that collects proposals and feedback.
Having our internal forum discussion with relative communities.
Run a vote on the most relevant proposals on the respective voting system.
Approved proposals will be sent to AF for a global communitarian vote.

This simple solution could be quickly implemented right after the beginning of G6.
Direct Advantages:

decentralizing the AF governance
the community would have the chance to be more active
fast execution

Multiple community proposals could be collected and provided to AF for the global vote, increasing the number of decisions and actions to be taken within the AF governance.
Every proposal given directly from AF will have a certain amount of complaints because it is difficult to manage each single vertical of such a large ecosystem. Indeed, having different communities' votes managed by a single protocol under xGov would remove the AF's responsibility of proposing new measures.
In our opinion, the AF has the right to evaluate each xGov final proposal before being posted to the global. In case of a clear dangerous xGov-approved proposal, the AF should be denying the global vote.
We believe that the one-year commitment requirement can be independent then the running of xGov to existing liquid governance solution provider as Folks and Algofi.
We are looking forward to discussing this further.

There should be a few simple rules that can create a consistent and long term pool of committed xGovs:

min stake: 100k or potentially 50k Algo in order to bring more decentralization;
min locked up period of stake: 6 months or more, with longer periods bringing higher APRs;
participate in consensus: altough I see this as a must in the long term, I would not roll it out from the first xGOV session, but include it as being mandatory fron 2nd period onwards and give xGovs time to get up to speed with requirements;
participate in all mandatory measures, else not eligible for rewards - but would not include as mandatory to participate in projects/DAOs hand picked by the AF, only whoever wants that for any additional incentive granted by that particular project;
last but perhaps the most important aspect, mandatory past governance track record: imo xGovs address should prove that it has successfully participated (voted&rewards claimed) in past governance rounds (at least 2 or 3 past periods);

All these measures would ensure that participants are committed and have a long-term vision for their governor status.

This document has a bunch of gendered terms (he, etc.). This needs to be fixed to be gender neutral.


Added Lynfield college from NZ

School Name: Lynfield College
Website: https://www.lynfield.school.nz/
The domain is: @lynfield.school.nz

@ruskindantra Pull request merged. Thank you!


added morpheus as character

Updated the character for Morpheus.
#hacktoberfest2023

Changes made . Please check @roannav

@Naman73598  Thank you very much for your contribution! 🏆

@all-contributors please add @Naman73598 for code and content


Navbar needs a @navbarBackgroundHover

Actually navbar background color is transparent for hover state.
.navbar .nav > li > a:hover {
    background-color: transparent;
    color: @navbarLinkColorHover;
    text-decoration: none;
  }

For a better control/customization please add a @navbarBackgroundHover color definition...
However i'm not sure if it's more consistent to have a @navbarBackgroundHover (same pattern as @navbarBackgroundHighlight),  or a @navbarLinkBackgroundColorHover as it's more close to what it is really...

The navbar doesn't have a background on link hover on purpose—to better distinguish active and hover. However, I've made the change to improve customization and used @navbarLinkBackgroundHover as the variable.
I didn't merge this in automatically since it wasn't a 1:1 merge into 2.0.1-wip.
Thanks!


Pr nncf jpqd with bert example only

What does this PR do?

Fixes # (issue)
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Reduce FPs when handling static inline functions

See #55, #82, and #83.
abi3audit currently considers all symbols when auditing. This is generally correct, but results in false positives when a static inline function (such as Py_XDECREF) doesn't get inlined, but instead remains as a local/private symbol.
The cause below this is nuanced: static inline functions like Py_XDECREF are part of the limited API but not the stable ABI; they're expected to be inlined into code that is part of the stable ABI. In other words, static inline functions are referentially opaque: their expansion is compatible with the stable ABI, but function identifiers themselves are not.
In practice this is a non-issue, and abi3audit should not flag local-only symbols for static inline functions.
To do this, the audit phase probably needs two things:

A list of static inline functions to ignore
Each Symbol's visibility, to know whether to ignore it

For (1), we can just start with Py_XDECREF. For (2), I think we'll need to extend the abi3info Symbol model to include a visibility: Visibility | None attribute, which will need to be populated as appropriate from each supported symbol table/object file.
CC @nicholasjng, who helped triage this and has graciously offered to help out 🙂

Judging from that thread, the Python folks could use a collection tool for those inline methods and macros. Sounds like a fun project for getting into the cPython codebase.
fyi, you can also assign me directly here, I'm planning to work on this soon (until end of week).

#87 fixes this in practice, but I'll leave this open for the longer-term potential fix of checking symbol visibility.

So regarding the symbol visibility, how do we fetch/infer the visibility from the stable_abi.toml file? That I would like to understand before starting the work on part 2).

So regarding the symbol visibility, how do we fetch/infer the visibility from the stable_abi.toml file? That I would like to understand before starting the work on part 2).

I was thinking about it from the opposite direction 🙂 -- rather than trying to infer each function's intended visibility, we can look at actual symbol visibilities in the symbol table and apply the following rules:

Symbol is abi3: pass
Symbol is NOT abi3 but has local/private visibility: pass
Symbol is NOT abi3 and has public visibility: fail

That way we'd end up allowing Py_XDECREF when it appears as a private symbol, but would still reject it if it (somehow) ends up being a public symbol. Thoughts?

Just for my understanding, public visibility means "linked in from an external lib", and "local/private" means from the project itself? How do I even get Py_XDECREF as a local/private visibility symbol?

Just for my understanding, public visibility means "linked in from an external lib", and "local/private" means from the project itself? How do I even get Py_XDECREF as a local/private visibility symbol?

Yep, pretty much. The way you end up with a local/private Py_XDECREF is via a static member defined in a header, like CPython does.
As a contrived example:
// myheader.h
static inline int foo() { return 42; }

// mytool.c
#include <myheader.h>

expands roughly to:
// mytool.c
// myheader.h
static inline int foo() { return 42; }

In effect, this means that every translation unit (i.e. .c file, approximately) has its own local copy of the foo symbol, each of which is local (i.e. private). Normally the C compiler then inlines each of these foos (since they're inline) so the local-only foo goes away, but this is not guaranteed (especially with GCC's -Os, apparently).
TL;DR: this all happens a "public" header (like those in CPython) can define static members, which then become "private" in each C file they get expanded in. One of C's many joys 😅

Done.


A Question on paper about hyperparamter setting

In the paper, the final objective is defined as L = Ls + α * Lc, where Ls is to increase the contrast between two augmented embeddings and Lc is for feature decorrelation. In Table 3, an ablation study is conducted by removing individual losses. The results suggest that Lc is a more important loss than Ls, as only keeping Lc results in a higher score of 66.48 in the STS dataset compared to a score of 35.70 when only applying Ls loss. Intuitively, this makes sense as the two augmented embeddings are likely to be semantically similar, and Lc helps to decorrelate the features. However, in the experiment setting, the value of α is set to 0.005 (for BERT), which may indicate that Ls is more important when combined with Lc. Can you share your thoughts on why Lc appears more important in the ablation study but Ls appears more important when combined with Lc in the final objective? Thank you.

Thanks for your interest in our work. Minimizing Ls without regularization is susceptible to reaching one of the many local minima with suboptimal performance on the tasks with the solution space less restricted. On the other hand, the space of Lc is more restricted as it reduces redundancy. Redundancy reduction produces good features. But low redundancy is not sufficient.


The poster image should appear after the video is finished.

Hi Plyer,
I think after the video is finished the player should bring back the poster image.
Right now I just see the last frame of the video. (In your example it's just an empty white screen.)
Warm regards,
Jack

I did look into this but I've not found a suitable solution yet. The "native" way seems to be to seek to 0 and then call .load() but the problem with that is .load() will basically make another request for the video. Not ideal. I will keep looking. It looks like video.js and the like load the poster image into a div, which I'd like to avoid that if possible.

video.src = ''; unload the video on chrome... got the tip fromp MDN.

Hey, there's a bug fix for this in v1.6.2


[BUG] fail on file make program unusable

If the json list becomes corrupted (e.g. uncorrect schema), program will never be able to retrieve it back.
At least it could be useful to destroy that file (and maybe make it .bak) so it can be created back and used as a blank one instead.

delete scrt-data folder and a new one will be generated


Add storyshot-puppeteer?

We used to have storyshots-puppeteer set up so that we could generate images of the components which is nice to refer to in the README.md files of the components. That way GitHub users can see what the components look like.
But there seems to be some bugs when setting it up and as long as our documentation is up to scratch (and we could link to that instead) perhaps this isn't very important. Perhaps it's not even worth the burden of maintaining?

See: https://github.com/storybookjs/storybook/tree/master/addons/storyshots/storyshots-puppeteer


Search highlight in wrong position

Hi guys,
in this pdf: http://www.ics.com/files/docs/kd/KDChart1refman.pdf
There is wrong highlight position in page 22.
This is a know issue?
I'm using Nightly, pdf.js development and Ubuntu 12.04

@SamiChoudhury I tested using current Nightly in Flatpak, and it seems this is solved. As I cannot reproduce the issue anymore, I'm closing the bug. If you can still reproduce, please reopen the bug.

The bug seems not to be fixed for Android WebView. May you have a look? It works fine in desktop browsers, mobile browser and ios WKWebview, but not in Android WebView.

same large offset for android webview,

The bug seems not to be fixed for Android WebView. May you have a look? It works fine in desktop browsers, mobile browser and ios WKWebview, but not in Android WebView.

Try wv.getSettings().setTextZoom(100)

another problem —— the browser  has a minimal supported font-size for an expample it is 13px right now on my laptop.
anything smaller is "invalid",  to fix that, the css transfromation value of the text layer spans should be smaller :

"fix"ed:


Creating issue is not working

I tried using the issue api.
var github = new Github();
var options = {
title: "Found a bug",
body: "I'm having a problem with this.",
assignee: "samprity",
milestone: 1,
labels: [
"Label1",
"Label2"
]
};
var issues = github.getIssues("samprity", "JSLint1");
issues.create(options, function(err, issue) {});
But I am getting this error :

Hi.
The screenshot doesn't really help us since it doesn't show the error. What's the exact error are you receiving? How did you use the library?

its says that axios is not a function. I have used the library in index.html :

I'm pretty sure your issue is related to the fact that you included the wrong file as specified https://github.com/michael/github/issues/285#issuecomment-180782156. In regard of your question, website hosted on github.io have an underlying repo, so the answer is yes.


Make Score.clip include the ongoing tempo/time signature/key signature at start

Hello,
I have a small feature request for the clip method: make it include the current tempo, time signature and key signature at the tick start. The current implementation keeps the events of these types occurring within the clip section (i.e. changes), but we might not know what are the values of these features at the beginning of the clip.
Describe the solution you'd like
Make clip determine the latest tempo/time signature/key signature occurring before the start tick, and add Tempo/TimeSignature/KeySignature elements in the returned Score chunk with time values at start_tick.
This could set as an option controlled by a method attribute.
Describe alternatives you've considered
Here is how I implemented it in Python:
def extract_chunk_from_midi(
    midi: Score, tick_start: int, tick_end: int, clip_end: bool = False
) -> Score:
    """
    Extract a chunk of a ``Score``.

    The returned chunk will have a starting time at tick 0, i.e. the times of its
    events will be shifted by ``-tick_start``.

    :param midi: object to extract a chunk from.
    :param tick_start: starting tick of the chunk to extract.
    :param tick_end: ending tick of the chunk to extract.
    :param clip_end: if given ``True``, the chunk at ``tick_end + 1`` and thus include
        the events occurring at ``tick_end``. (default: ``False``)
    :return: chunk of the ``midi`` starting at ``tick_start and ending at ``tick_end``.
    """
    # Get the tempo, time sig and key sig at the beginning of the chunk to extract
    # There might not be default key signatures in the Score
    tempo, time_signature, key_signature = None, None, KeySignature(0, 0, 0)
    for tempo_ in midi.tempos:
        if tempo_.time > tick_start:
            break
        tempo = tempo_
    for time_signature_ in midi.time_signatures:
        if time_signature_.time > tick_start:
            break
        time_signature = time_signature_
    for key_signature_ in midi.key_signatures:
        if key_signature_.time > tick_start:
            break
        key_signature = key_signature_
    tempo.time = time_signature.time = key_signature.time = 0

    # Clip the MIDI and append the global attributes
    midi_split = midi.clip(tick_start, tick_end, clip_end=clip_end).shift_time(
        -tick_start
    )
    midi_split.tempos.append(tempo)
    midi_split.time_signatures.append(time_signature)
    midi_split.key_signatures.append(key_signature)

    return midi_split

I finally got back doing some c++ and got a glimpse of nanobind. If you want, I can try to implement this.
Additional context
That's something I'm implementing for this MidiTok PR: https://github.com/Natooz/MidiTok/pull/148

I am recently preparing for the 3.24 gre exam. Sorry for not having time to read your requirements carefully. But you can try to modify the function if you are happy to do so.

@Natooz I understand this issue. This probably won't involve a lot of modifications. I'll make the changes as soon as I can.

I agree with you

@Natooz I have modified the clip function. Does this work as what you expected?

Absolutely! Thank you for these changes!


Safety actor-critic

Baseline PPO agent:

Critic represents total reward
Actor is trained to maximize critic

CBF PPO agent:

Base critic represents nominal reward
CBF critic represents safety reward
Actor is trained to maximize base critic + lambda * CBF critic

Remarks

If lambda is set large enough, then the combined reward function defines a SVF, so optimal actor is guaranteed to be safe.
Additionally, because the CBF critic is trained on the 0/1 safety  reward, it has an analytic form, so we can apply our tricks / metrics.

Experiments

Baseline PPO
PPO + penalty
PPO + penalty + CBF tricks

Concrete tasks

[ ] Implement separate safety reward tracking in replay buffer
[ ] Implement multi-critic actor-critic architecture
[ ] Modify training loop to also train the second critic


Monero zmq crash takes down monero-lws-daemon. (Probably)

monerod is running, all good.
tcp        0      0 <IP_ADDRESS>:38080      <IP_ADDRESS>:*               LISTEN      7425/monerod
tcp        0      0 <IP_ADDRESS>:38081    <IP_ADDRESS>:*               LISTEN      7425/monerod
tcp        0      0 <IP_ADDRESS>:38082      <IP_ADDRESS>:*               LISTEN      7425/monerod
tcp        0      0 <IP_ADDRESS>:38083      <IP_ADDRESS>:*               LISTEN      7425/monerod
tcp        0      0 <IP_ADDRESS>:38084      <IP_ADDRESS>:*               LISTEN      7425/monerod

monerod is still running, but zmq is gone.
tcp        0      0 <IP_ADDRESS>:38080      <IP_ADDRESS>:*               LISTEN      6527/monerod
tcp        0      0 <IP_ADDRESS>:38081    <IP_ADDRESS>:*               LISTEN      6527/monerod
tcp        0      0 <IP_ADDRESS>:38084      <IP_ADDRESS>:*               LISTEN      6527/monerod

monero-lws-daemon is running on a different host.
After some time, after zmq is gone, this happens.
2022-11-11 21:34:43.847 I Retrieving current active account list
2022-11-11 21:34:43.847 I No active accounts
2022-11-11 21:34:53.848 I Starting blockchain sync with daemon
2022-11-11 21:35:24.315 W Failed to connect to daemon at tcp://<IP_ADDRESS>:38082
2022-11-11 21:35:24.315 I Retrieving current active account list
2022-11-11 21:35:24.315 I No active accounts
2022-11-11 21:35:34.356 E thrown at scanner.cpp:800: Too many open files

There's a shitload of Failed to connect to daemon at tcp://<IP_ADDRESS>:38082 before the crash, so how long after, I don't know.

I'm starting to think the lws crash was unrelated to zmq server was down.
Without a log-file on monero-lws-daemon it's impossible to say which one went down first.
The monero-lws-daemon does not have that many file-handles open. 430-450 including events.
I shut monerod down
at 2022-11-12 01:28:18.875 W Failed to connect to daemon at tcp://<IP_ADDRESS>:38082
monero-lws-daemon  is still running
at 2022-11-12 05:47:02.075 W Failed to connect to daemon at tcp://<IP_ADDRESS>:38082
Could it be something it received from zmq ?
This is when zmq crashed on monerod:
2022-11-11 11:25:59.635 [P2P1]  INFO    net.p2p.msg     src/cryptonote_protocol/cryptonote_protocol_handler.inl:550     [<IP_ADDRESS>:52902 INC] Received NOTIFY_NEW_FLUFFY_BLOCK <e95a3a4ea3960a337d047fa778a540fd3fe56e89fa46fa4623d07dbe34d8c285> (height 1220696, 0 txes)
2022-11-11 11:25:59.635 [P2P4]  INFO    perf.blockchain src/common/perf_timer.cpp:120   PERF             ----------
2022-11-11 11:25:59.635 [P2P4]  INFO    perf.blockchain src/common/perf_timer.cpp:156   PERF        7      get_long_term_block_weights
2022-11-11 11:25:59.635 [P2P4]  INFO    perf.blockchain src/common/perf_timer.cpp:156   PERF      125    update_next_cumulative_weight_limit
2022-11-11 11:25:59.636 [P2P4]  INFO    blockchain      src/cryptonote_core/blockchain.cpp:4524 +++++ BLOCK SUCCESSFULLY ADDED
2022-11-11 11:25:59.636 [P2P4]  INFO    blockchain      src/cryptonote_core/blockchain.cpp:4524 id:     <e95a3a4ea3960a337d047fa778a540fd3fe56e89fa46fa4623d07dbe34d8c285>
2022-11-11 11:25:59.636 [P2P4]  INFO    blockchain      src/cryptonote_core/blockchain.cpp:4524 PoW:    <ea79e4acc1b0c5289d896a971b107a2a54a2ffe563bf25b90493c84f8a510000>
2022-11-11 11:25:59.636 [P2P4]  INFO    blockchain      src/cryptonote_core/blockchain.cpp:4524 HEIGHT 1220695, difficulty:     111144
2022-11-11 11:25:59.636 [P2P4]  INFO    blockchain      src/cryptonote_core/blockchain.cpp:4524 block reward: 3.538476387438(3.538476387438 + 0.000000000000), coinbase_weight: 85, cumulative weight: 85, 34(0/33)ms
2022-11-11 11:25:59.637 [P2P5]  INFO    net.p2p.traffic contrib/epee/include/net/levin_protocol_handler_async.h:57      [<IP_ADDRESS>:38080 OUT] 187 bytes received for category command-2008 initiated by peer
2022-11-11 11:25:59.637 [P2P5]  INFO    net.p2p.msg     src/cryptonote_protocol/cryptonote_protocol_handler.inl:550     [<IP_ADDRESS>:38080 OUT] Received NOTIFY_NEW_FLUFFY_BLOCK <e95a3a4ea3960a337d047fa778a540fd3fe56e89fa46fa4623d07dbe34d8c285> (height 1220696, 0 txes)
2022-11-11 11:25:59.638 [P2P4]  ERROR   net.zmq src/rpc/zmq_pub.cpp:538 Unable to send ZMQ/Pub - ZMQ server destroyed
2022-11-11 11:25:59.638 [P2P4]  ERROR   net.zmq src/rpc/zmq_pub.cpp:529 Unable to send ZMQ/Pub - ZMQ server destroyed
2022-11-11 11:25:59.641 [P2P0]  INFO    net.p2p.traffic contrib/epee/include/net/levin_protocol_handler_async.h:57      [<IP_ADDRESS>:54532 INC] 187 bytes received for category command-2008 initiated by peer

Okay, here we go again.
monerod
2022-11-12 11:18:47.711        I Found block <ae713803b7f463493a421939c51a6bc7f322db359393260ea219736ba282e1c2> at height 1221412 for difficulty: 106982
2022-11-12 11:26:20.469        I Found block <f93d44351b7fb02f7e8af8a6b5554556fe5defa67f376e950ae894c3baeaeb98> at height 1221418 for difficulty: 106276
2022-11-12 11:47:00.976        I Found block <7612be135b7d7ff85a3caab7e2d286ac5f3b1498821ddfd88390b7249412b8c0> at height 1221434 for difficulty: 107536
2022-11-12 12:10:56.926        E ZMQ RPC Server Error: thrown at zmq_server.cpp:161: Resource temporarily unavailable
2022-11-12 12:11:18.102        I Found block <b50745860f385ff1a2a12c88deec69af22cab6e04951c10aaeda7a31398337d1> at height 1221447 for difficulty: 106169
2022-11-12 12:11:18.152        E Unable to send ZMQ/Pub - ZMQ server destroyed
2022-11-12 12:11:18.153        E Unable to send ZMQ/Pub - ZMQ server destroyed
2022-11-12 12:11:49.473        E Unable to send ZMQ/Pub - ZMQ server destroyed
2022-11-12 12:11:49.473        E Unable to send ZMQ/Pub - ZMQ server destroyed
2022-11-12 12:13:19.091        E Unable to send ZMQ/Pub - ZMQ server destroyed
2022-11-12 12:13:19.091        E Unable to send ZMQ/Pub - ZMQ server destroyed
...
...
2022-11-12 18:26:02.081        E Unable to send ZMQ/Pub - ZMQ server destroyed
2022-11-12 18:26:02.081        E Unable to send ZMQ/Pub - ZMQ server destroyed
2022-11-12 18:26:52.542        I Found block <ace0338ea93db5f7d9c24dc7300e0b2a55e2fee48c3fd5f2b28754853adae3b6> at height 1221669 for difficulty: 114610
2022-11-12 18:26:52.583        E Unable to send ZMQ/Pub - ZMQ server destroyed
2022-11-12 18:26:52.583        E Unable to send ZMQ/Pub - ZMQ server destroyed
2022-11-12 18:27:40.895        E Unable to send ZMQ/Pub - ZMQ server destroyed
2022-11-12 18:27:52.347        E Unable to send ZMQ/Pub - ZMQ server destroyed
2022-11-12 18:27:52.347        E Unable to send ZMQ/Pub - ZMQ server destroyed

monero-lws
2022-11-12 18:25:37.079 W Failed to connect to daemon at tcp://<IP_ADDRESS>:38082
2022-11-12 18:25:37.079 I Retrieving current active account list
2022-11-12 18:25:37.079 I No active accounts
2022-11-12 18:25:47.103 I Starting blockchain sync with daemon
2022-11-12 18:26:17.451 W Failed to connect to daemon at tcp://<IP_ADDRESS>:38082
2022-11-12 18:26:17.451 I Retrieving current active account list
2022-11-12 18:26:17.452 I No active accounts
2022-11-12 18:26:27.459 I Starting blockchain sync with daemon
2022-11-12 18:26:57.702 W Failed to connect to daemon at tcp://<IP_ADDRESS>:38082
2022-11-12 18:26:57.702 I Retrieving current active account list
2022-11-12 18:26:57.702 I No active accounts
2022-11-12 18:27:07.740 E thrown at scanner.cpp:800: Too many open files

And, here's the testnet crash.
2022-11-12 12:10:56.683     7f48f57fa700        ERROR   net.zmq src/rpc/zmq_server.cpp:174      ZMQ RPC Server Error: thrown at zmq_server.cpp:161: Resource temporarily unavailable
2022-11-12 12:12:25.187 [P2P6]  WARNING cn      src/cryptonote_core/cryptonote_core.cpp:2048    There were 17 blocks in the last 90 minutes, there might be large hash rate changes, or we might be partitioned, cut off from the Monero network or under attack, or your computer's time is off. Or it could be just sheer bad luck.
2022-11-12 12:13:55.304 [P2P8]  WARNING cn      src/cryptonote_core/cryptonote_core.cpp:2048    There were 16 blocks in the last 90 minutes, there might be large hash rate changes, or we might be partitioned, cut off from the Monero network or under attack, or your computer's time is off. Or it could be just sheer bad luck.
2022-11-12 12:15:25.436 [P2P3]  WARNING cn      src/cryptonote_core/cryptonote_core.cpp:2048    There were 16 blocks in the last 90 minutes, there might be large hash rate changes, or we might be partitioned, cut off from the Monero network or under attack, or your computer's time is off. Or it could be just sheer bad luck.
2022-11-12 12:16:55.655 [P2P0]  WARNING cn      src/cryptonote_core/cryptonote_core.cpp:2048    There were 16 blocks in the last 90 minutes, there might be large hash rate changes, or we might be partitioned, cut off from the Monero network or under attack, or your computer's time is off. Or it could be just sheer bad luck.
2022-11-12 12:18:25.831 [P2P4]  WARNING cn      src/cryptonote_core/cryptonote_core.cpp:2048    There were 16 blocks in the last 90 minutes, there might be large hash rate changes, or we might be partitioned, cut off from the Monero network or under attack, or your computer's time is off. Or it could be just sheer bad luck.
2022-11-12 12:19:55.908 [P2P1]  WARNING cn      src/cryptonote_core/cryptonote_core.cpp:2048    There were 15 blocks in the last 90 minutes, there might be large hash rate changes, or we might be partitioned, cut off from the Monero network or under attack, or your computer's time is off. Or it could be just sheer bad luck.
2022-11-12 12:20:41.110 [P2P1]  ERROR   net.zmq src/rpc/zmq_pub.cpp:538 Unable to send ZMQ/Pub - ZMQ server destroyed
2022-11-12 12:20:41.110 [P2P1]  ERROR   net.zmq src/rpc/zmq_pub.cpp:529 Unable to send ZMQ/Pub - ZMQ server destroyed

I got a little more info from the latest crash.
2022-11-13 19:42:00.245        W Failed to connect to daemon at tcp://<IP_ADDRESS>:38082
2022-11-13 19:42:00.245        I Retrieving current active account list
2022-11-13 19:42:00.246        I No active accounts
2022-11-13 19:42:10.252        D Problems at ssl shutdown: uninitialized (SSL routines, SSL_shutdown)
2022-11-13 19:42:10.253        D Problems at cancel: Bad file descriptor
2022-11-13 19:42:10.253        D Problems at shutdown: Bad file descriptor
2022-11-13 19:42:10.258        D Destructing connection #0 to <IP_ADDRESS>
2022-11-13 19:42:10.258        E thrown at scanner.cpp:800: Too many open files

Turning log-level up to 4 and restarting....

Restarting monero-lws-daemon got me this:
2022-11-13 23:58:37.170   I Using monerod ZMQ RPC at tcp://<IP_ADDRESS>:38082
2022-11-13 23:58:37.170        I Starting blockchain sync with daemon
2022-11-13 23:59:07.715        D Problems at ssl shutdown: uninitialized (SSL routines, SSL_shutdown)
2022-11-13 23:59:07.715        D Problems at cancel: Bad file descriptor
2022-11-13 23:59:07.715        D Problems at shutdown: Bad file descriptor
2022-11-13 23:59:07.716        E Connection failed with daemon

Restarting monerod, ZMQ was down, and restarting monero-lws-daemon.. waiting...

@trasherdk Here is a monerod binary with the vtnerd's ZMQ fix, please test it if it changes something: https://github.com/selsta/monero/suites/9282067270/artifacts/435113240

@selsta I need a static build to test.
$ ./monerod --version
./monerod: /lib64/libc.so.6: version `GLIBC_2.27' not found (required by ./monerod)
./monerod: /lib64/libc.so.6: version `GLIBC_2.25' not found (required by ./monerod)

It is a static build but your glibc is too old. Will make another one.

@selsta It's up and running on stagenet. Thanks.

Okay. Both v<IP_ADDRESS>-release (testnet) and v<IP_ADDRESS>-41ca021ee (stagenet) has been running since yesterday, without any zmq noise. There has been quite a bit of traffic on stagenet zmq the last 24 hours.
It must have been some nastiness on the wire causing the zmq crash.
I'll keep @selsta's version run on testnet for now.

So you mean both didn't crash? Patched and unpatched?
Also seems I compiled v<IP_ADDRESS> instead of v<IP_ADDRESS>, just keep that in mind if you get the "wrong version" error message.

@trasherdk I'm leaving this open for now, but we should have the monerod side fixed. Hopefully there's no further issues on the monero-lws-daemon side.

For what it's worth, I'm still getting this error relatively constantly - it's just a matter of time after a restart.

@paulshapiro just to confirm, did you apply the monerod zmq fixed manually? They aren't merged yet.

Yep, I have tried 2-3 of the candidate fixes from vtnerd patched manually onto source for a monerod build and watched them over a handful of the past months but I've still seen the issue from the monero-lws code. And to confirm (apologies for not having checked before I made my prior post) it looks like what I'm seeing is all somehow related to the "get_dynamic_fee_estimate" zmq client .send(). But I can only see that the error that too many files are open is emanating from the endpoint itself currently, due to the fact the log is coming from here. My system settings as far as open file limits go seem ample too.

The monerod side is fixed, and early reports for the LWS 0.3 branch are suggesting that the too many open files issue is fixed too.

This thing seems fixed.


[Kush] On null value in message, option to skip or fail.

Currently if Kafka topic has a null message, the beast fails with null pointer exception. We should allow users to skip null messages or fail based on configuration.

Codecov Report

Merging #62 into master will decrease coverage by 0.99%.
The diff coverage is 37.03%.

@@             Coverage Diff              @@
##             master      #62      +/-   ##
============================================
- Coverage     82.23%   81.23%   -1.00%
  Complexity      277      277
============================================
  Files            53       53
  Lines           895      906      +11
  Branches         87       89       +2
============================================
  Hits            736      736
- Misses          128      136       +8
- Partials         31       34       +3

Impacted Files
Coverage Δ
Complexity Δ

...main/java/com/gojek/beast/converter/RowMapper.java
90.74% <0.00%> (ø)
18.00 <0.00> (ø)

src/main/java/com/gojek/beast/sink/bq/BqSink.java
68.00% <0.00%> (-4.00%)
8.00 <1.00> (ø)

...gojek/beast/converter/ConsumerRecordConverter.java
76.19% <37.50%> (-23.81%)
4.00 <0.00> (ø)

.../gojek/beast/protomapping/ProtoUpdateListener.java
53.84% <41.17%> (-3.79%)
5.00 <1.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 ed9c047...f832e0e. Read the comment docs.

changes looks good to me