How to capture the Refresh event

In my project, I've implement a WOPI server. Sometimes, oos prompt the message 'Sorry, your changes weren't saved. To continue editing this document, please copy your changes and refresh the page', But if I click the Refresh button, there is nothing happend.
My question is how to capture this refresh event in my WOPI server?

Sorry, but we can't offer support through this repository. If you are integrating with Office Online as part of the Cloud Storage Partner Program, then you will receive an invitation to a Yammer group for support as part of onboarding.


feat: autoclose peek definition and finder buffers on focus loss

close rendered windows for finder and peek_definition buffers when focus is lost (e.g. switching to another window)
before changes:

after changes:

@glepnir, could you take a look please? :)

BufLeave is not valid event because the window can have the same buffer id as origin location, hence the event will not be emitted. It is necessary to close the window only if focus is lost for finder window, it has nothing to do with the buffer.
Also I fixed a bug when treesitter is not enabled for finder entries since vim api marked all the buffers as loaded, but treesitter highlighter is not started there :)

oh right in peek it will have same buffer . make sense. thanks for your contribute.


Support the function COL_NAME()

Description
Implement a function COL_NAME(table_id, column_id) which returns the Transact-SQL column name that corresponds to the provided column ID within the specified table.
For example:
Input-
CREATE TABLE test_table(c1 int);
GO
SELECT COL_NAME(OBJECT_ID('test_table'), 1);
GO
Output-
c1
Signed-off-by: Roshan Kanwar<EMAIL_ADDRESS>Issues Resolved
BABEL-3172
Test Scenarios Covered

Use case based - Added use case based tests in JDBC framework

Boundary conditions - Added edge cases where table_id and column_id both can be invalid or NULL

Arbitrary inputs -

Negative test cases - Added test cases where table_id and column_id will be negative

Minor version upgrade tests -

Major version upgrade tests -

Performance tests -

Tooling impact -

Client tests -

Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is under the terms of the Apache 2.0 and PostgreSQL licenses, and grant any person obtaining a copy of the contribution permission to relicense all or a portion of my contribution to the PostgreSQL License solely to contribute all or a portion of my contribution to the PostgreSQL open source project.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Please add this test to upgrade/latest/schedule


Incorrect item placement for iPad screen in SplitViewController on DetailViewController

The page menu indicator is not centered in SplitViewController on DetailViewController

P.D. Thank you for the library 😁

This is obviously a bug. I'll fix it sometime soon.


Add option for nightly, ci and dev test

Description

As a developer, one needs to run a fast quickcheck test to check if a commit is correct. For a CI, this number of quickcheck tests can be more. A nightly test can even be more aggressive in terms of the number of tests to run. To facilitate running different number of tests depending upon the environment, this PR adds an option "--test-env". A "--test-env" option can be set to either "nightly", "ci" or "dev". The "dev" environment is a default value.
If "nightly" environment is set, then the number of quickcheck tests to run is set to "100_000". For "ci" environment this value is "10_000". The default "dev" environment sets the number of quickcheck tests to "100". An individual test may customise the number of quick check tests proportional to the time taken by the test to run, and to suit the environment.
The goal of current PR is to provide an option to set test environment and not to tune the number of tests for different environment. "Tuning" the tests can be done as a separate task.
Checklist

Branch

[x] Commit sequence broadly makes sense
[x] Commits have useful messages
[x] New tests are added if needed and existing tests are updated
[x] If this branch changes Consensus and has any consequences for downstream repositories or end users, said changes must be documented in interface-CHANGELOG.md
[x] If this branch changes Network and has any consequences for downstream repositories or end users, said changes must be documented in interface-CHANGELOG.md
[x] If serialization changes, user-facing consequences (e.g. replay from genesis) are confirmed to be intentional.

Pull Request

[x] Self-reviewed the diff
[x] Useful pull request description at least containing the following information:

What does this PR change?
Why these changes were needed?
How does this affect downstream repositories and/or end-users?
Which ticket does this PR close (if any)? If it does, is it linked?

[x] Reviewer requested

Things to be done

Fix the number of tests for CI/Nightly
Time the tests with CI/Nightly tests
Few tests run for a long time, nightly tests adjust quick check tests assuming it to be 100. This will be done by running tests.

closes #3892

bors r+


Optical lightning sensor research

Wavelength to check? ~777nm source (related)
Ball lightning can be detected via same frequency – source

Lots of info on optical measurements of lightning, states that there are two main bands to check: 777.4 nm and 886.3 nm
From same source:

7774  and 8683 Angstroms.  These are emissions from atomic oxygen and nitrogen respectively (OI and NI denote atomic oxygen and nitrogen)

Great paper on lightning detection (in general) within planetary atmospheres

For CG lightning – source
Likely that each type of lightning has discernible spectra.
For CG strikes, cyanogen spectra (~384.0 nm) indicates continuing current, something known to cause forest fires.

Observation of the Earth and its Environment: Survey of Missions and Sensors

Narrowband filters of 10 Å should be applied for the various spectra of interest. – source


Enhancement: Add support for application/vnd.api+json (JSON API) content type to match application/json support

Currently the parsing of json strings (#19) into objects is only supported for application/json. JSON API uses a content type of application/vnd.api+json (Why, I wish I knew). When testing with fauxjax this became unmaintainable because all of our request data had to be strings which are tedious to change.
This PR just adds application/vnd.api+json as an acceptable content type to parse a string into an object. Basically making it behave exactly like application/json does currently.

Coverage remained the same at 100.0% when pulling 8d1b2562a2d4479bf9a0c6a379364443b296601f on TayHobbs:master into 9edcf61535f7947415b20886e560b2ab5c8b2e73 on JarrodCTaylor:master.


Revert "build: allow falling back to just built compiler"

This reverts commit 86ddf43790d53cfc0c2313dc77492096991ed927.
Unfortunately, the missing support in clang prevents this from actually
enabling tests on the 5.3 branch.  It however, adversely impacts the
ability to build the standalone runtime.  Reverting this until we can
have a tested version validate an end-to-end build.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@swift-ci please test

Going to workaround this in a different way.

Build failed
Swift Test OS X Platform
Git Sha - 5256b63a9a0ecbc3fc2f449d8361157a4ffe51a4

Build failed
Swift Test Linux Platform
Git Sha - 5256b63a9a0ecbc3fc2f449d8361157a4ffe51a4


Improved funcs for MFT analysis.

The main changes include making MFT API more flexible and to enable epochs handling. The exact list of changes are below:

jumeg_mft_funcs.py:

allow fwd- and data-objects as alternative to filenames
faster calc. of P-matrix
added support for epoch-data handling
minor improvements in doc
bug fix for non-zero coordinates in prbcnt

jumeg_mft_plot.py:

add'l arg for plot_cmd_data

perform_mft_surface.py:

modified args in apply_mft()-call

Thanks @ebeich

@ebeich We can keep this PR open till the semi-current status changes. :)


proto3 optional fields support

I am planning to start using proto3 optional labels. This is an experimental feature available using --experimental_allow_proto3_optional on protoc invocations: https://github.com/protocolbuffers/protobuf/blob/master/docs/field_presence.md#protoc-invocation
Before changing all my protos I would like to know if this feature is already supported for swift. I have looked around in the repo but haven't been able to find if it is done or not.
Thank you!

https://github.com/apple/swift-protobuf/releases/tag/1.9.0

Support for new Proto3 optional (this needs a protoc from protocolbuffers/protobuf v3.12.0 (or later)) #978


Wrong detection of AllowPrivilegeEscalation and others

terrascan version: 1.15.0
Operating System: ubuntu docker

Description
Trying to use terrascan-action to scan kubernetes manifests for violations
I get an abundant amount of false positives of AllowPrivilegeEscalation and others relating to secrets files
e.g.
	Description    :	Containers Should Not Run with AllowPrivilegeEscalation
	File           :	clusters/infra/services/kiam/agent.yaml
	Line           :	1
	Severity       :	HIGH

What I Did
on: [push]

jobs:
  terrascan_job:
    runs-on: ubuntu-latest
    name: terrascan-action
    steps:
    - name: Checkout repository
      uses: actions/checkout@v2
    - name: Run Terrascan
      id: terrascan
      uses: tenable/terrascan-action@main
      with:
        iac_type: 'k8s'
        iac_version: 'v1'
        policy_type: 'k8s'
        only_warn: true
        #scm_token: ${{ secrets.GH_ACCESS_TOKEN }}
        #verbose: true
        #sarif_upload: true
        #non_recursive:
        #iac_dir:
        #policy_path:
        #skip_rules:
        config_path:  /github/workspace/.github/workflows/config.toml
        #find_vulnerabilities:
        #webhook_url:
        #webhook_token:

used above GH action on a kubernetes manifest repo

Opened the issue here on advice from @gaurav-gogia as it appears policy related and not directly to do with the GH action.

I have the same problem.
In my case, this is the output and the related file:
 {
        "rule_name": "privilegeEscalationCheck",
        "description": "Containers Should Not Run with AllowPrivilegeEscalation",
        "rule_id": "AC_K8S_0085",
        "severity": "HIGH",
        "category": "Compliance Validation",
        "resource_name": "cloudwatch-agent",
        "resource_type": "kubernetes_daemonset",
        "file": "redacted\\cwagent-daemonset.yaml",
        "line": 1
      },

And checking the corresponding file, these are the contents:
# deploy cwagent as daemonset
apiVersion: apps/v1
kind: DaemonSet
metadata:
  name: cloudwatch-agent
  namespace: amazon-cloudwatch
spec:
  selector:
    matchLabels:
      name: cloudwatch-agent
  template:
    metadata:
      labels:
        name: cloudwatch-agent
    spec:
      containers:
        - name: cloudwatch-agent
          image: amazon/cloudwatch-agent:1.247350.0b251780
          ports:
            - containerPort: 8125
              hostPort: 8125
              protocol: UDP
          resources:
            limits:
              cpu:  200m
              memory: 200Mi
            requests:
              cpu: 200m
              memory: 200Mi
          # Please don't change below envs
          env:
            - name: HOST_IP
              valueFrom:
                fieldRef:
                  fieldPath: status.hostIP
            - name: HOST_NAME
              valueFrom:
                fieldRef:
                  fieldPath: spec.nodeName
            - name: K8S_NAMESPACE
              valueFrom:
                fieldRef:
                  fieldPath: metadata.namespace
            - name: CI_VERSION
              value: "k8s/1.3.9"
          # Please don't change the mountPath
          volumeMounts:
            - name: cwagentconfig
              mountPath: /etc/cwagentconfig
            - name: rootfs
              mountPath: /rootfs
              readOnly: true
            - name: dockersock
              mountPath: /var/run/docker.sock
              readOnly: true
            - name: varlibdocker
              mountPath: /var/lib/docker
              readOnly: true
            - name: containerdsock
              mountPath: /run/containerd/containerd.sock
              readOnly: true
            - name: sys
              mountPath: /sys
              readOnly: true
            - name: devdisk
              mountPath: /dev/disk
              readOnly: true
      volumes:
        - name: cwagentconfig
          configMap:
            name: cwagentconfig
        - name: rootfs
          hostPath:
            path: /
        - name: dockersock
          hostPath:
            path: /var/run/docker.sock
        - name: varlibdocker
          hostPath:
            path: /var/lib/docker
        - name: containerdsock
          hostPath:
            path: /run/containerd/containerd.sock
        - name: sys
          hostPath:
            path: /sys
        - name: devdisk
          hostPath:
            path: /dev/disk/
      terminationGracePeriodSeconds: 60
      serviceAccountName: cloudwatch-agent

Thanks in advance.


fix: Making ninja output on windows to work

I'm a little confused which PR I should review, this one or #1120?
Also, cc @kunalspathak.

@bnoordhuis This one is more complete. But requires a change in GYP, so I understand any pushback.
#1120 should be a no brainer ;)

closing due to age, hopefully this is taken up in refack/GYP and we can pull it in there?


Content Dialog - Multiple controls on page are not supported

I render multiple Content Dialog controls on my HTML site.
If I click the associated button from one of them, all message dialogs are appearing.
Is it a bug or just not possible to have multiple content dialog controls on one page?

Hi @P4trik  it is possible to have multiple Content Dialog controls on your HTML site.
Do you see this issue specific only when you use the angular-winjs adapter, or are you able to reproduce it with just WinJS as well?


I hope to be able to view container logs, just like Docker Desktop

I hope to be able to view container logs, just like Docker Desktop

Duplicate of #46. You can use the CLI for now.


BUG: Fix Icons in Footer Section

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
Icons in the footer, including social media (Facebook and Twitter) and feature icons (like "Advertise" and "Become a Seller"), are not displaying correctly.
What happened?

Facebook and Twitter icons are not rendering properly.
Feature icons do not match the design of the original website.

Expected behavior

Facebook and Twitter icons should display correctly with base64 encoded SVGs.
Feature icons should match the original website’s design.

Screenshots

Record

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

@arghadipmanna101
please accept my pull request


Create a new Sync Widget

Lead: @sonle-geekyants
Is your feature request related to a problem? Please describe.
Now that we use a progressive sync strategy, we should provide a widget that tracks progress and provides a UI component that provides feedback to the person using the app.
Describe the solution you'd like
For more details - see this document.
Describe alternatives you've considered
Otherwise, app developers need to develop this themselves.

Target UI widgets are done.
Next task:

[ ] Add integration to get sync status
[ ] test and publish the package.

No progress.

Functionality is added. Needs review, testing and publishing.

Requires documentation update and publishing.

Published: at_sync_ui_flutter: ^1.0.1


emailed issue: Remove-DbaRemoveOrphan should exclude SSISDB

Hello,
I see now with the current upgrades that the remove-DbaRemoveOrphan users is not being utilized.
It seems with this function the SSISDB permissions used to execute SSIS packages.
So I thought it might be a good idea to log it with you guys so that it can be mentioned for other people in case they, too remove the SSIS permissions needed to execute jobs (woopsy) I got a lot of interesting emails afterwards.
Thank you so much!
/cc @ClaudioESSilva

I'm not sure I follow what the issue is in that email?

i interpreted it that there was a system user that appeared orphaned that actually wasnt?

I poked in one of our SSISDB systems and it looks like AllSchemaOwner is a stock SSISDB user with no login and many of the SSIS procs have "WITH EXECUTE AS 'AllSchemaOwner'" in them.  Dropping that user would certainly cause issues from what I'm seeing.

I will take a look on this later.
But, users without login should be skipped by the filter on
https://github.com/sqlcollaborative/dbatools/blob/development/functions/
Remove-DbaOrphanUser.ps1#L172 maybe an edge case :-)
On Wed, Mar 28, 2018 at 9:30 PM, cfradenburg<EMAIL_ADDRESS>wrote:

I poked in one of our SSISDB systems and it looks like AllSchemaOwner is a
stock SSISDB user with no login and many of the SSIS procs have "WITH
EXECUTE AS 'AllSchemaOwner'" in them. Dropping that user would certainly
cause issues from what I'm seeing.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sqlcollaborative/dbatools/issues/3443#issuecomment-377025601,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ASnfIxBeCqUSSmQHr_0ZtSINrH6KbpHCks5ti_LcgaJpZM4S-S0O
.

Btw,@potatoqualitee https://github.com/potatoqualitee can you confirm
with the sender he/she was using the latest dbatools version?
Thanks!
On Thu, Mar 29, 2018 at 9:26 AM, Cláudio Silva<EMAIL_ADDRESS>wrote:

I will take a look on this later.
But, users without login should be skipped by the filter on
https://github.com/sqlcollaborative/dbatools/blob/develop
ment/functions/Remove-DbaOrphanUser.ps1#L172 maybe an edge case :-)
On Wed, Mar 28, 2018 at 9:30 PM, cfradenburg<EMAIL_ADDRESS>wrote:

I poked in one of our SSISDB systems and it looks like AllSchemaOwner is
a stock SSISDB user with no login and many of the SSIS procs have "WITH
EXECUTE AS 'AllSchemaOwner'" in them. Dropping that user would certainly
cause issues from what I'm seeing.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sqlcollaborative/dbatools/issues/3443#issuecomment-377025601,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ASnfIxBeCqUSSmQHr_0ZtSINrH6KbpHCks5ti_LcgaJpZM4S-S0O
.

hey @ClaudioESSilva - any chance you can take a look at this within the next week?

I can confirm this is indeed a bug.
You are picking up the moduleuser cert user in SSISDB that is used for signing the module within SSIS.
PS C:\Windows\system32> Remove-DbaOrphanUser -SqlInstance sqlvm -Database ssisdb -WhatIf -Verbose
VERBOSE: [13:57:27][Remove-DbaOrphanUser] Validating users on database [SSISDB].
VERBOSE: [13:57:27][Remove-DbaOrphanUser] Orphan users found.
VERBOSE: [13:57:27][Remove-DbaOrphanUser] User [ModuleSigner] does not own any schema. Will be dropped.
What if: Performing the operation "Dropping user [ModuleSigner]" on target "SSISDB".

This line appears to be what you need to address because there is no login for this user:
https://github.com/sqlcollaborative/dbatools/blob/9b43322cd5d0e7785d20b0c4a1ede412b0b5c15c/functions/Remove-DbaOrphanUser.ps1#L157
PS C:\Windows\system32> Get-DbaDbUser -SqlInstance sqlvm -Database ssisdb | where Name -eq 'ModuleSigner'

ComputerName       : cma-cac1-sqlvm
InstanceName       : MSSQLSERVER
SqlInstance        : cma-cac1-sqlvm
Database           : SSISDB
CreateDate         : 4/30/2016 12:46:16 AM
DateLastModified   : 4/30/2016 12:46:16 AM
Name               : ModuleSigner
Login              :
LoginType          : Certificate
AuthenticationType : None
State              : Existing
HasDbAccess        : True
DefaultSchema      :

Thanks for the analysis @wsmelton.
The problem here is the LoginType. I was just looking for the SQL user without login to be excluded but the user mapped to a certificate also need to be excluded from here.
I mentioned you on the PR so you can test and accept only after that.
Thanks in advance.
@cfradenburg if you would like to test it and give your "OK" too, we appreciate.


fix(Turborepo): Fix cookie directory

Description

Only create a CookieWriter once, use clone from then on to ensure we have the right directory
Fixes bug where we passed the wrong directory into a CookieWriter

Testing Instructions
Manual verification that the cookie directory is now correct
Closes TURBO-2566

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#7664  👈
#7654
#7644
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @gsoltis and the rest of your teammates on  Graphite

Appears to still be an issue with the extension. If it's enabled, any time I make edits to any package.json, random .cookie files are created. They appear all to be blank.

The above is after hours of investigating! 😓 They started at 1 and I had about 3 different runs through going form 1 to the hundreds before discovering that disabling the VS Code extension eliminates the issue!


AttributeError: 'NoneType' object has no attribute 'name' while streaming

Describe the Bug
I am getting the AttributeError: 'NoneType' object has no attribute 'name' when streaming the comments. I first chalked it off to people commenting and deleting immediately, but that is not the case.
Desired Result
comment.author should not be None if the comment is not deleted or removed
Code to reproduce the bug
main_logger.info(f"Logged into {await reddit_instance.user.me()} Account.")
    pcm_subreddit = await reddit_instance.subreddit("PoliticalCompassMemes")
    async for comment in pcm_subreddit.stream.comments(skip_existing=True):  # Comment
        if comment.author.name.lower() in [getenv("REDDIT_USERNAME", "basedcount_bot").lower(), "flair-checking-bot"]:
           ^^^^^^^^^^^^^^ throws attribute error

The Reddit() initialization in my code example does not include the following parameters to prevent credential leakage:
client_secret, password, or refresh_token.

[X] Yes

Relevant Logs
Traceback (most recent call last):
  File "/root/basedcount_bot/basedcount_bot.py", line 39, in wrapper
    await func(reddit_instance, mongo_client)
  File "/root/basedcount_bot/basedcount_bot.py", line 250, in read_comments
    if comment.author.name.lower() in [getenv("REDDIT_USERNAME", "basedcount_bot").lower(), "flair-checking-bot"]:
AttributeError: 'NoneType' object has no attribute 'name'

This code has previously worked as intended.
Yes
Operating System/Environment
Ubuntu 20.04.5 LTS
Python Version
3.10.7
Async PRAW Version
7.6.1
Asyncprawcore Version
2.3.0
Anything else?
Now technically speaking, I could check if the author is None and skip the comment, but as I mentioned, it is giving an error for comments which should have the author info. If I go the skip route, I might skip over normal comments.

Ok turns out that the user has blocked me. The comment appears in the stream but I can't read any fields.


False positive Too many lv Field of Enum module_categories, expecting at most 1 - CW242

Vanilla HoI4 has since allowed more than one field usage here:
False Positive Error example:

Actual HoI4 code:

Move this issue to https://github.com/cwtools/cwtools-hoi4-config/issues/193 instead.


znc not entering channels using key after quick reconnect

Debian 7 & znc 1.4
In znc the channels keys appear set. If I disconnect and connect znc enters the channels properly but if it reconnects quickly (by a kill for example) it throws errors that it couldn't join them and that they are now disabled.
If I just go ahead and rejoin them, it enters them normally using the key and all.
Any suggestions on how I could force znc to use the key in any scenario or how I can provide more information about this problem?

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/4402726-znc-not-entering-channels-using-key-after-quick-reconnect?utm_campaign=plugin&utm_content=tracker%2F1759&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F1759&utm_medium=issues&utm_source=github).

Does this still happen to you on 1.6.0 or could this be closed?

As far as I can recall no it hasn't happened since 1.6.0.


Support for 293xxxxxx serial logger for deye hybrid inverter

Hello,
I would like to request support for 293xxxxxx serial logger. Currently the values are all over the place, incorrect, and logging is not working. Tried multiple lookup templates.

after reinstalling the integration it is now showing correct values. I have used the deye_sg04lp3.yaml lookup.


Test installation over npm after deployment

This PR will ensure that - after Travis CI has deployed a new version of Now CLI to npm - it will automatically try to download that version. This will result in two advantages for us:

Since the npm package download is connected to the CDN, the CDN will pull the latest version of the CLI from the platform and cache it. This means that all following installations by users don't have the delay that the CDN takes to download and cache a binary.

Travis CI will automatically let us know if the npm install -g now command failed and we can immediately act to fix it, so that no users experience any problems.

😻


Organized code

What is your question?
Can you organize the code, please? I try to traverse it and it is not easy, if you could organize it with comments perhaps, I would like that.

What ide are you using.
I use vscode and all I need to do is do Ctrl+F to search one file and Ctrl+Shift+F to search every file.


Linting and other minor additions

Some small changes:

added code of conduct
add .pylintrc and implemented complaints by pylint
added transport_co2/__version__.py, so one can check module version from within Python environment

Hei, this looks good. While we are preparing another release, would it be a good opportunity to add a couple of usage examples to the README? E.g. showing the function and Enum approach to using the library?

Added a few examples. 😎
BTW, in the Code of Conduct, I used my email since I had to use some, but I'm not sure which one is best. Should it be a more generic MaaS contact email, or is it better to actually contact a human (with the risk of slower response)?


Upgraded backend + comments

More versatile (kinda?) backend
All the actions now send socket.io messages to the backend.
Frontend can respond to gamestring changes.
Changed a few variable names to better reflect them.
Added some comments

¯_(ツ)_/¯ I've looked it all, and it all seems good. We might need to switch to a Node.js enabled platform like Heroku soon because GitHub pages is just static.


Show pending status in cop status of manual

Follow up of #5979.
This PR shows pending status in cop status of manual.
The pending status is not enabled status and may lead to misreading.

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[ ] Added tests.
[ ] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Run bundle exec rake default. It executes all tests and RuboCop for itself, and generates the documentation.

Nice catch!


Kubefed get and describe giving inconsistent data

kubectl get clusters returns No resource found. Error from server (NotAcceptable): unknown (get clusters.federation)
while
kubectl describe clusters returns the description of all the joined clusters.
Possible bug in implementation?
Attaching screenshot

I observed this using kubectl version 1.11.x against a federation that was 1.9.x.  Changing the snap channel to 1.9.x or 1.10.x "solved" it for me.  (sudo snap switch --channel=1.10/stable kubectl or sudo snap switch --channel=1.9/stable kubectl)

Thanks. Works now


KAFKA-9386; Apply delete ACL filters to resources from filter even if not in cache

With the old SimpleAclAuthorizer, we were handling delete filters that matched a single resource by looking up that resource directly, even if it wasn't in the cache. AclAuthorizerTest.testHighConcurrencyDeletionOfResourceAcls relies on this behaviour and fails intermittently when the cache is not up-to-date. This PR includes the resource from non-matching filters even if it is not in the cache to retain the old behaviour.
Committer Checklist (excluded from commit message)

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

retest this please

@rajinisivaram We look for the resource directly by querying ZK?

@ijuma When deleting ACLs, we always use the latest ACLs from ZK since the updates use the version. But in order to look up those ACLs, we need to know which resources to look up. For ACL filters that specify a resource pattern (e.g testTopicA or prefixed topic test), we can directly look up that resource to ensure that we get the latest value in ZK. And we were doing that in SimpleAclAuthorizer. For ACL filters that match multiple resources, we match against the resources in aclCache since we dont want to look up all resources in ZK to perform the match. This PR does the same for AclAuthorizer.
If you do Admin.createAcls() followed by Admin.deleteAcls(), if you specify the ACL in both cases, you are guaranteed to delete the ACL regardless of which broker handles the request. If you use a matching filter that doesn't specify the resource pattern for deleteAcls, then we don't provide that guarantee. If we did want to provide that guarantee, we would need to read all resources from ZK for this case.

Thanks for the explanation. Two comments:

Have we documented this anywhere? It's a bit subtle, so it would be good to do so, if we have not.
Could we write a deterministic test for this behavior (maybe using mocks) so that we don't regress? It can be done as a separate PR.

@ijuma Opened https://issues.apache.org/jira/browse/KAFKA-9392 to document the limitation and add a deterministic test.

Thanks. We should also check that we can continue to offer this guarantee after KIP-500.

@ijuma I guess we don't know yet the exact format in which ACLs will be stored for KIP-500. But I think this guarantee is similar to supporting concurrent updates which we need to do anyway. A deterministic test will at least ensure we will notice if it doesn't work.

@omkreddy @ijuma Thanks for the reviews, merging to trunk. Will address remaining comments under KAFKA-9392 and KAFKA-8847.


[Optimization] Duplication of mapping parameters across all indices

Is your feature request related to a problem? Please describe.
When there are large number of indices on a cluster, it increases the JVM footprint on a master/data node. This can cause the master/data node to go OOM.
Simulation of Data Node OOM

A small cluster was created with 1 master and 1 data node with 1 GB heap size
A large index template was created which had around 600 mappings.
Around 600 indices were created using the template.
Data node went out of memory.

Analysis
When an index is created, its metadata gets loaded in memory on the data node where it is present. When the heap dump for the data node was analyzed, it was found that around 80% of the heap space was occupied by MapperService classes. One instance of MapperService is created for every index present on the data node. There was about 1.2 MB space taken per instance of MapperService. Under each MapperService there was an instance of DocumentMapper which contained a hashmap of field level mappers. Each field mapper contained around 10-15 parameters for the field metadata. These field parameters are taking up space even though these are not explicitly defined for the field. When aggregated across around 600 fields in the 600 indices, these parameters take up the majority of the space. These is opportunity to optimize these by referring to some default parameters.
Code Path: MapperService → mapper (DocumentMapper) → fieldMappers (Map) → builder (TextFieldMapper.Builder) → boost (ParameterizeFieldMapper.Parameter)

Simulation of Master node OOM

A small cluster was created with 1 master and 2 data node with 1 GB heap size each
A large index template was created which had around 600 mappings.
Around 600 indices were created using the template.
A single field mapping was updated across all the indices by using the wildcard * on PUT _mappings API
Master node went out of memory

Analysis
When the heap dump of the master node was analyzed, similar objects like the data node issue were noticed. MapperService is not loaded usually on the master node. But when the mapping gets updated for indices, it loads the MapperService of those indices temporarily in memory. So again an optimization around the duplication of mapping parameters would help here.
Describe the solution you'd like
Everytime an index is created, new instances of mapping parameters are created and loaded in memory even though the parameters are not explicitly defined. We can use default instance for each mapping parameter and point the reference of the mapping parameter of a new index to the default instance.

Thanks for the detailed analysis @soosinha
One key aspect when creating singleton object for those mapping parameters would be to ensure immutability of those objects as otherwise it can cause data corruption/ overwrites across indices.

Made changes to remove the builder instance from TextFieldMapper. The retained size of a TextFieldMapper object was reduced from 3300 bytes to 376 bytes (88%). So we should see great benefit in the JVM usage for cluster with lot of indices and lot of mappings.
In my test case, I created a cluster with 700 indices with each index having 600 field mappings(250 text type fields). The heap usage on a data node reduced from 900 MB to around 500 MB.
Draft PR for the changes: https://github.com/soosinha/OpenSearch/pull/1

Thanks @soosinha!  for TextFieldMapper , the gain looks great. Left a comment on PR. From the cursory look, it looks safe to remove the builder object as it is not referenced later. Lets run the whole test suite on your changes.


ipset: 6.38 -> 7.0

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/ipset/versions.
meta.description for ipset is: '"Administration tool for IP sets"'.

Checks done (click to expand)

built on NixOS
/nix/store/jjdnf3jln6kvhlppr5s045nkjvn8d158-ipset-7.0/bin/ipset passed the binary check.
1 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 7.0 with grep in /nix/store/jjdnf3jln6kvhlppr5s045nkjvn8d158-ipset-7.0
directory tree listing: https://gist.github.com/85bf18c1c64333afdcd5301da3ff437b
du listing: https://gist.github.com/109dcb7a17d4ce0fb92f7eae4e128763

Outpath report (click to expand)

Outpath difference report
9 total rebuild paths
3 package rebuilds
3 x86_64-linux rebuilds
3 i686-linux rebuilds
0 x86_64-darwin rebuilds
3 aarch64-linux rebuilds
First ten rebuilds by attrpath
ferm
firehol
ipset

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/jjdnf3jln6kvhlppr5s045nkjvn8d158-ipset-7.0 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A ipset https://github.com/r-ryantm/nixpkgs/archive/4cb41949df316003730b3284e72e8c7b648dc998.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/jjdnf3jln6kvhlppr5s045nkjvn8d158-ipset-7.0
ls -la /nix/store/jjdnf3jln6kvhlppr5s045nkjvn8d158-ipset-7.0/bin

cc @wkennington for testing.

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: ipset
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: ipset
Partial log (click to expand)
shrinking RPATHs of ELF executables and libraries in /nix/store/jjdnf3jln6kvhlppr5s045nkjvn8d158-ipset-7.0
shrinking /nix/store/jjdnf3jln6kvhlppr5s045nkjvn8d158-ipset-7.0/lib/libipset.so.13.0.0
shrinking /nix/store/jjdnf3jln6kvhlppr5s045nkjvn8d158-ipset-7.0/sbin/ipset
gzipping man pages under /nix/store/jjdnf3jln6kvhlppr5s045nkjvn8d158-ipset-7.0/share/man/
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/jjdnf3jln6kvhlppr5s045nkjvn8d158-ipset-7.0/lib  /nix/store/jjdnf3jln6kvhlppr5s045nkjvn8d158-ipset-7.0/sbin
patching script interpreter paths in /nix/store/jjdnf3jln6kvhlppr5s045nkjvn8d158-ipset-7.0
checking for references to /build in /nix/store/jjdnf3jln6kvhlppr5s045nkjvn8d158-ipset-7.0...
moving /nix/store/jjdnf3jln6kvhlppr5s045nkjvn8d158-ipset-7.0/sbin/* to /nix/store/jjdnf3jln6kvhlppr5s045nkjvn8d158-ipset-7.0/bin
/nix/store/jjdnf3jln6kvhlppr5s045nkjvn8d158-ipset-7.0

Success on aarch64-linux (full log)
Attempted: ipset
Partial log (click to expand)
shrinking RPATHs of ELF executables and libraries in /nix/store/m6rz2vwqiy4j4n03krbccpnikjfswqd5-ipset-7.0
shrinking /nix/store/m6rz2vwqiy4j4n03krbccpnikjfswqd5-ipset-7.0/sbin/ipset
shrinking /nix/store/m6rz2vwqiy4j4n03krbccpnikjfswqd5-ipset-7.0/lib/libipset.so.13.0.0
gzipping man pages under /nix/store/m6rz2vwqiy4j4n03krbccpnikjfswqd5-ipset-7.0/share/man/
strip is /nix/store/p9akxn2sfy4wkhqdqa3li97pc6jaz3r1-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/m6rz2vwqiy4j4n03krbccpnikjfswqd5-ipset-7.0/lib  /nix/store/m6rz2vwqiy4j4n03krbccpnikjfswqd5-ipset-7.0/sbin
patching script interpreter paths in /nix/store/m6rz2vwqiy4j4n03krbccpnikjfswqd5-ipset-7.0
checking for references to /build in /nix/store/m6rz2vwqiy4j4n03krbccpnikjfswqd5-ipset-7.0...
moving /nix/store/m6rz2vwqiy4j4n03krbccpnikjfswqd5-ipset-7.0/sbin/* to /nix/store/m6rz2vwqiy4j4n03krbccpnikjfswqd5-ipset-7.0/bin
/nix/store/m6rz2vwqiy4j4n03krbccpnikjfswqd5-ipset-7.0

Thanks for testing :)


Add Docker setup script for host IP detection

Running this script starts up graph-node, uses docker inspect to obtain the host IP address, injects this IP address into docker-compose.yml and stops graph-node again.
Resolves #1024.

I'm going to go ahead and merge this.


Config provider Deno support

What is the problem this feature would solve?
Deno has a different api for accessing the environment variables.
There is no process.env.
https://github.com/Effect-TS/effect/blob/ee7abb75d91d7c1c673935db254af0d4a6eb591d/src/internal/configProvider.ts#L106
What is the feature you are proposing to solve the problem?
Support Deno out of box for config provider.
I'm willing to contribute.
What alternatives have you considered?
Using the ConfigProvider.fromMap  and manually passing the Deno env is an option, but

you have to set env variables one by one
or grant access the whole env reading in Deno.

How would supporting Deno look like?

Haven't tried, but something like this should work:
diff --git a/src/internal/configProvider.ts b/src/internal/configProvider.ts
index 44af3ae58..6de804e54 100644
--- a/src/internal/configProvider.ts
+++ b/src/internal/configProvider.ts
@@ -95,6 +95,37 @@ export const fromFlat = (flat: ConfigProvider.ConfigProvider.Flat): ConfigProvid
     flattened: flat
   })

+interface Environment {
+  getVariable(name: string): string | undefined;
+  getAllVariables(): { [key: string]: string | undefined; }
+}
+
+interface DenoRuntime {
+  Deno: {
+    env: {
+      get(key: string): string | undefined;
+      toObject(): { [index: string]: string; };
+    }
+  }
+}
+
+const isDenoRuntime = (runtime: {}): runtime is DenoRuntime => 'Deno' in runtime
+
+const getEnvironment = (): Environment => {
+  const runtime = globalThis;
+  if (isDenoRuntime(runtime)) {
+    return {
+      getVariable: (name) => runtime.Deno.env.get(name),
+      getAllVariables: () => runtime.Deno.env.toObject(),
+    }
+  }
+
+  return {
+    getVariable: (name) => process.env[name],
+    getAllVariables: () => process.env,
+  }
+}
+
 /** @internal */
 export const fromEnv = (
   config: Partial<ConfigProvider.ConfigProvider.FromEnvConfig> = {}
@@ -103,17 +134,14 @@ export const fromEnv = (
   const makePathString = (path: ReadonlyArray<string>): string => pipe(path, RA.join(pathDelim))
   const unmakePathString = (pathString: string): ReadonlyArray<string> => pathString.split(pathDelim)

-  const getEnv = () =>
-    typeof process !== "undefined" && "env" in process && typeof process.env === "object" ? process.env : {}
-
   const load = <A>(
     path: ReadonlyArray<string>,
     primitive: Config.Config.Primitive<A>,
     split = true
   ): Effect.Effect<never, ConfigError.ConfigError, ReadonlyArray<A>> => {
     const pathString = makePathString(path)
-    const current = getEnv()
-    const valueOpt = pathString in current ? Option.some(current[pathString]!) : Option.none()
+    const current = getEnvironment()
+    const valueOpt = Option.fromNullable(current.getVariable(pathString))
     return pipe(
       valueOpt,
       core.mapError(() => configError.MissingData(path, `Expected ${pathString} to exist in the process context`)),
@@ -125,8 +153,8 @@ export const fromEnv = (
     path: ReadonlyArray<string>
   ): Effect.Effect<never, ConfigError.ConfigError, HashSet.HashSet<string>> =>
     core.sync(() => {
-      const current = getEnv()
-      const keys = Object.keys(current)
+      const current = getEnvironment()
+      const keys = Object.keys(current.getAllVariables())
       const keyPaths = Array.from(keys).map((value) => unmakePathString(value.toUpperCase()))
       const filteredKeyPaths = keyPaths.filter((keyPath) => {
         for (let i = 0; i < path.length; i++) {

If you want to open a PR I'd be happy to review & merge

I'm glad you are open to support Deno. I'll submit a PR then, just have to figure out some things. For example how to test the project with Deno.

cc @fubhy who's proposing a multi-runtime test matrix

I can add a PR for the test matrix later today.

superseded by https://github.com/Effect-TS/effect/issues/1806


Support replacement scans

Replacement scans allow table functions to be used easier, and the API doesn't seem to complex (although a bit hard to understand). It'd be cool to get replacement scans to be supported by Duck.Net.

I had a look at the API and replacement scans look like a slimmed version of table functions. According to the documentation The replacement scan can then insert a table function so you would still need to register a custom table function if you want to use that function in a replacement scan.

Yeah, exactly. I already have a table function, it works great. Now I am planning for the function to "pretend" that it's a table.

This functionality would be useful for the xlDuckDb project https://github.com/RusselWebber/xlDuckDb/issues/3


TypeError: Cannot read property 'type' of undefined.

hello @jk-gan , thanks for taking your time and developing this plugin. I'm getting the following error following your configuration.
if (__DEV__) {
  const createDebugger = require('redux-flipper').default;

  let reduxDebugger = createDebugger();
  let store = createStore(RootReducer, {}, applyMiddleware(reduxDebugger));
} else {
  let store = createStore(RootReducer, {});
}

I solved it just by changing how to import createDebugger to import createDebugger from 'redux-flipper'; The rest is the same.
Thanks,

I'm closing it since it's not an issue, but here's it is for future reference.

Hi, thanks for the feedback!
If you now switch back to const createDebugger = require('redux-flipper').default;, do u get the same error? I suspect the error occurs is because of the action pass into the middleware is undefined.

@jk-gan yeah, I found where the issue arises in the code. Now it is working fine 🤣 now sure why tho. Perhaps, Flipper/RN/the Simulator needed more restarts. I'd. be good to add a safety-check if (row) so it doesn't break if row is null.

ok noted

@jk-ganI still found this issue. maybe add a check for the row like ricbermo suggested would solve it.


Added Related Video For UserStoppedTypingBehavior

What changes to the docs does this PR provide?
Added related video for UserStoppedTypingBehavior
PR Checklist
Please check if your PR fulfills the following requirements:

[ ] For new pages, used the provided template.
[ ] For new features, added an entry in the Table of Contents.
[X] Ran against a spell and grammar checker.
[X] Contains NO breaking changes.

Other information

Docs Build status updates of commit 58e2995:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/behaviors/userstoppedtypingbehavior.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


M2K workflow - use K_SINK to produce event instead of rest request to broker

Use K_SINK to produce event instead of rest request to broker
The K_SINK env variable should be added to the deployment resource or injected by creating SinkBinding
Closes: https://issues.redhat.com/browse/FLPATH-934

/hold until https://github.com/apache/incubator-kie-kogito-runtimes/pull/3386 is availble

/unhold


DataObject property name

As I'm constantly dealing with JSON documents which contain Java keywords or illegal characters I always have to work around this problem. This might not be a problem for a handful of classes but gets very time-consuming, error-prone and complex if you have to support multiple versions of a data exchange protocol or similar.
There have been several attempts to add per-property naming for DataObjects from different developers, but none of those made it to master.
With this Pull Request it would be possible to annotate a specific property like so:
@DataObject.Property(":ref")
public String getRef() {
    return ref;
}

The generated converter will then pick up the specified property name instead of the default one determined by the jsonPropertyNameFormatter.

Hello @vietj. I'm sorry to bother you. I just wanted to ask if you already had a chance to take a look at the changes?


CDilated weird ddconv weight shape

Hi @noahzn,
When loading the pretrained weights for the depth encoder, I noticed that the dilated convolution layer has an unexpected shape:
stages.0.0.ddwconv.conv.weight has shape torch.Size([48, 1, 3, 3]), where I would expect it to have shape torch.Size([48, 48, 3, 3]), since the number of channels stays the same in DilatedConv. Do you have any insight on why this is the case?
Thanks!

Hi, please make sure you are using the correct pre-trained weights and mode. You can change the model using --model.

I am loading the .pth directly with torch.load for inspection, that is how I noticed.

Could you load the decoder using my code to see if it works or not?

I did some further investigation, and it turns out the dimension of 1 for the kernel is due to the use of groups in the dilated convolution. The code works fine, but I was curious why the kernel weight was shaped like that. Thanks!


Bug: Query return false if have company and firstname setted

Platform: Android
I'm trying to find contacts by Company name. works find if the contact only have Company field. but failing if the contact have first name and company fields.
I believe it's because the query is only by display name.
https://github.com/fluttercommunity/flutter_contacts/blob/b58b508d08f207dd2065c82ddd61c83d96bc134a/android/src/main/java/flutter/plugins/contactsservice/contactsservice/ContactsServicePlugin.java#L167

@simonlinj are you still experiencing this issue with the latest version 0.2.8 ?


added flag to make-dev-page so that only tree plot is created

This PR adds a --no-empire flag to the make-dev-page script which will result in a tree plot being created instead of an empire plot.

@fedarko you requested a review from yourself, would you be able to review this PR when you get a chance?

The following artifacts were built for this PR: empire-biplot.qzv, empire.qzv, empress-tree.qzv, just-fm.qzv, plain.qzv

@fedarko should be good to merge after tests finish running.


feat(dashboard): set base

set base href for dashboard deploy

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

GitHub release
v1.2.0

Your semantic-release bot :package::rocket:


fix: upgrade ci to go 1.23

Additionally, the go.mod version needs to be raised aswell, otherwise the go compiler falls back to using the new compiler, but with the old featureset


Implement the related handler methods of the transparent operations

support index file
support object downloading
support health checking
support related manipulating operations except getting chart version

Pull Request Test Coverage Report for Build 7544

0 of 0   changed or added relevant lines in 0 files are covered.
7 unchanged lines in 3 files lost coverage.
Overall coverage increased (+0.06%) to 64.597%

Files with Coverage Reduction
New Missed Lines
%

src/common/scheduler/scheduler.go
1
81.87%

src/replication/core/controller.go
2
75.93%

src/common/utils/utils.go
4
81.55%

Totals

Change from base Build 7540:
0.06%

Covered Lines:
11081

Relevant Lines:
17154

💛  - Coveralls


Fix unbounded long lines breaking log send for gui

@keybase/react-hackers @chrisnojima

has issues, fixing here

@cjb


chat with page conversations

can use this api for chat with fanpage conversations
thanks.

Yup! If you login with the option {pageId: xxxx} you'll be logged in as a page and will receive/send messages through the page. Check the pageId option here.


to_netcdf failing for datasets with a single time value

In this notebook: http://nbviewer.ipython.org/gist/rsignell-usgs/047235496029529585cc, the
ds.to_netcdf method in cell [12] is failing for this dataset with a single time value:
/home/usgs/miniconda/envs/ioos/lib/python2.7/site-packages/xray/conventions.pyc in infer_datetime_units(dates)
    185     unique_timedeltas = np.unique(np.diff(dates[pd.notnull(dates)]))
    186     units = _infer_time_units_from_diff(unique_timedeltas)
--> 187     return '%s since %s' % (units, pd.Timestamp(dates[0]))
    188
    189

IndexError: too many indices for array

When there is a single value, I guess dates must not be an array.

Thanks for the report. I believe this is fixed on master: https://github.com/xray/xray/pull/454 (I hope to get a release out shortly)

Indeed, with master, it's working.
http://nbviewer.ipython.org/gist/rsignell-usgs/047235496029529585cc
Closing....


gnomeExtensions.no-title-bar: init at 8

Motivation for this change
I do need this extension for my Gnome Shell setup.
Tested locally.
The run-time dependencies are specified as it is required here.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

This seems to be a fork of pixel-saver, which we already have packaged, but much more active. Maybe we could replace it?
cc @jonafato

@jtojnar yes, this fork seems to be active.
I suppose I should also add platforms = platforms.linux; here?

Thanks for review!

This does seem to be more actively maintained than pixel-saver (which was last updated 6 months ago and last released over a year ago). Replacing pixel-saver with no-title-bar probably makes sense.

@jonafato Do you want to be a maintainer of this?

@jtojnar I'd be happy to maintain it. I'll submit a PR to remove pixel-saver in favor of this extension as well.

Did you built this? There is missing substituteAll and even after adding it, the build still fails.

Thanks.


FPS Steaming Handler does not work at all

Script
Both of them do not work. They do not give any error.
In my understanding addSteamFood is kind of a wrong naming. Shouldn't it be addSteaming? I don't think the Furnace + Steam Heaters is limited to food.
Calling @SatanicSanta and @bonii-xx.

And you're calling me because...?

Things like Steamed Food - they can be only created by using a Furnace plus Steam Heaters. Is this "steaming" cooking limited to food only or can it be used for anything?

And you're calling me because...?

"Is this "steaming" cooking limited to food only or can it be used for anything?"

Well, this is emberassing. I mixed ya up with @xbony2! T_T
Sorry.

Lol. I wondered why you'd think I had anything to do with it. But I can see the resemblance ;P

Poor Bonii :P Adding new steamed food needs an actually food item, but adding a recipe probably shouldn't. At least I think.
Going to look at this mod's API usage and see what they're doing.

For whatever reason MT doesn't use the addSteamFood() that comes in FSP's API, does some other stuff.

@jaredlll08 and @soundlogic2236 Would you mind changing it?

I'm not sure if it's really the problem, but it would probably be better to call it then change the steamedFoods HashMap directly.

The reason I generally didn't use methods, and manipulated things directly is mods generally only have an add recipe and not a remove recipe in their apis. Plus it's easier than using different methods. Was less code xD.
I never actually tested the FSP, so not surprised it doesn't work.

FSP is not modtweaker friendly.


fix: do not use hardwire service IP

This commit removes a hardwired IP
in the nexus service.

@gmarcy this looks OK, right?

good catch !


Adding pallet-contracts sanity check

Hi,
We want to use !ink4.0 using the extended-parachain-template.
We have got no errors in extended-parachain-template for the time being. This is only to make sure that we did not miss something silly.

We have created a default "contract.rs" configuration for devnet and mainnet.
contract.rs
use crate::{
	constants::currency::deposit, Balance, Balances, RandomnessCollectiveFlip, Runtime,
	RuntimeCall, RuntimeEvent, RuntimeHoldReason, Timestamp,
};
use frame_support::{
	parameter_types,
	traits::{ConstBool, ConstU32, Nothing},
};
use pallet_contracts::{
	migration::{v12, v13, v14, v15},
	weights::SubstrateWeight,
	Config, DebugInfo, DefaultAddressGenerator, Frame, Schedule,
};
use sp_runtime::Perbill;

pub use parachains_common::AVERAGE_ON_INITIALIZE_RATIO;

// Prints debug output of the `contracts` pallet to stdout if the node is
// started with `-lruntime::contracts=debug`.
pub const CONTRACTS_DEBUG_OUTPUT: DebugInfo = DebugInfo::UnsafeDebug;

parameter_types! {
	pub const DepositPerItem: Balance = deposit(1, 0);
	pub const DepositPerByte: Balance = deposit(0, 1);
	pub const DefaultDepositLimit: Balance = deposit(1024, 1024 * 1024);
	pub MySchedule: Schedule<Runtime> = Default::default();
	pub CodeHashLockupDepositPercent: Perbill = Perbill::from_percent(30);
}

impl Config for Runtime {
	type Time = Timestamp;
	type Randomness = RandomnessCollectiveFlip;
	type Currency = Balances;
	type RuntimeEvent = RuntimeEvent;
	type RuntimeCall = RuntimeCall;
	/// The safest default is to allow no calls at all.
	///
	/// Runtimes should whitelist dispatchables that are allowed to be called from contracts
	/// and make sure they are stable. Dispatchables exposed to contracts are not allowed to
	/// change because that would break already deployed contracts. The `Call` structure itself
	/// is not allowed to change the indices of existing pallets, too.
	type CallFilter = Nothing;
	type DepositPerItem = DepositPerItem;
	type DepositPerByte = DepositPerByte;
	type DefaultDepositLimit = DefaultDepositLimit;
	type WeightPrice = pallet_transaction_payment::Pallet<Self>;
	type WeightInfo = SubstrateWeight<Self>;
	type ChainExtension = ();
	type Schedule = MySchedule;
	type CallStack = [Frame<Self>; 5];
	type AddressGenerator = DefaultAddressGenerator;
	type MaxCodeLen = ConstU32<{ 123 * 1024 }>;
	type MaxStorageKeyLen = ConstU32<128>;
	type UnsafeUnstableInterface = ConstBool<true>;
	type MaxDebugBufferLen = ConstU32<{ 2 * 1024 * 1024 }>;
	type MaxDelegateDependencies = ConstU32<32>;
	type CodeHashLockupDepositPercent = CodeHashLockupDepositPercent;
	type Migrations = (
		v12::Migration<Runtime, Balances>,
		v13::Migration<Runtime>,
		v14::Migration<Runtime, Balances>,
		v15::Migration<Runtime>,
	);
	type RuntimeHoldReason = RuntimeHoldReason;
	type Debug = ();
	type Environment = ();
}

We have added the below lines in cargo.toml for devnet and mainnet.
pallet-contracts = { workspace = true, default-features = false }
pallet-contracts-primitives = { workspace = true, default-features = false }

[features]
default = ["std"]
std = [
   (...)
   "pallet-contracts-primitives/std",
   "pallet-contracts/std",
]

runtime-benchmarks = [
    (...)
   "pallet-contracts/runtime-benchmarks",
]

try-runtime = [
  (...)
  "pallet-contracts/try-runtime",
]

And finally in the workspace root cargo.toml
cargo.toml
[workspace.dependencies]
(...)
pallet-contracts = { version = "24.0.0", default-features = false }
pallet-contracts-primitives = { version = "27.0.0", default-features = false }
(...)

I am assuming that this is it. No more configuration is need it.
Cheers,

If you're starting from scratch I don't think you need these migrations : migration::{v12, v13, v14, v15},. Just type Migrations = () should suffice. Rest of it looks about right. Have you tested by deploying a sample ink! contract?

IIRC you also need to implement the ContractsApi in the runtime (see here as an example), plus a few minor edits to the runtime's config (do a search on contracts in the aforementioned runtime).

Thank @weezy20 and @stiiifff for your help.
This template is a bit temperamental but we where able to do it.
Cheers,


Flashcards too highly positioned

Prerequisites

[x] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in QA
Followed all applicable steps in the TROUBLESHOOTING guide: TODO
Checked for relevant discussions on Discord: https://discord.gg/pPERUuv
Checked that your issue isn't already filed: https://github.com/COVID-19-electronic-health-system/Corona-tracker/issues

Description
Flashcards are displayed too highly, blocking text

Steps to Reproduce

Log in
Go to flashcards

Expected behavior:
Flashcards are more appropriately spaced in tab
Actual behavior:
Flashcards are too high
Reproduces how often:
100%
Versions
0.3.16

I can take this

All you! If you'd like to be actually assigned on the ticket, I sent you an invite to the org as well 👍

I should have been faster but time man!
@SomeMoosery @petesamper
this an issue is that more card stacked now, not padding or margin issue because if you add that the card will look normal in less long screens(say  Samsung S9, iPhone 5s) but if you keep swiping some of the lastest cards will be under the navbar.
we might need to look into the whole style/motion of the cards

I did a screenshare user test and they were having the same issue where the cards were cut off on the bottom by the bottom nav. Any way we could create a container for the cards to fit between the top text and bottom nav bar?

Also @AdhamAH do you think you could adjust the text at the top to say "Swipe to the right or left to learn more about COVID-19" and then remove that typo that's in all caps?

I'll add changing the text as an acceptance criteria for this ticket, and change the context of this ticket to fit more with making sure the flashcards are more just scaled / centered in any screen 👍

Hey all, sorry for the delay. I know this is a simple issue so I apologize.
I gave the flashcards container a margin property (changed from marginTop in my PR) with the expectation that it would prevent the flashcards from touching the navbar on bottom. I know @AdhamAH mentioned this may just be an issue with the animations themselves, particularly when the last cards in the decks are reached, but I haven't had any luck adjusting the animations, so I left them alone.
I'm having trouble logging in on mobile to test my changes there, so I haven't been able to recreate the issue on mobile to see if my changes look alright.
That said, here is a screenshot of it on my machine, on the last flashcard, with no issues:
Again sorry all, I am a noobie :sweat_smile:.

While it's not perfect, you can use Chrome Devtools to simulate mobile environments!
No worries, take your time! We'd rather have something great than something only partially fixed :)

Whoa, didn't know about that. Alright I'll use that for now and get back to you when I have results. I'll fix this! :muscle:


[22.05] Revert "python3Packages.mistune_0_8: mark knownVulnerabilities CVE-2022-34749"

This reverts commit db8c23037a5d162d2cb380300102e5ac9b049ebb (#184209/#186149).
Description of changes

Per https://github.com/lepture/mistune/issues/314#issuecomment-1223972386, Mistune 0.8.4 is not vulnerable to CVE-2022-34749, only 2.0.x versions of Mistune before 2.0.3 are. (release-22.05 now contains mistune 0.8.4 and 2.0.4, neither of which are vulnerable.)
Things done
nixpkgs-review is running, but will take a while.

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

We should probably also fix on master
pkgs/top-level/python-aliases.nix:  mistune_0_8 = throw "mistune_0_8 was removed because it was outdated and insecure"; # added 2022-08-12

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/jupyter-broken-in-22-05/21108/5


not operating…

using latest insider ObsidianMD, the plugin does not place still-open daily note to-dos (from the note created by that other plugin) into the new, next day’s, daily note (again using that same plugin) at any place in the new note, including the new note's template section as specified in the rollover plugin settings…
any suggestion?

I have the same problem.

Yep, got the same problem here as well. Was really enjoying using it as well :(

The API for Obsidian looks like it changed the way the filesystem works. Going through the code, this code will not run in new Obsidian API system. I have made update code here with this as inspiration. Feel free to use and let me know of bugs.

The API for Obsidian looks like it changed the way the filesystem works. Going through the code, this code will not run in new Obsidian API system. I have made update code here with this as inspiration. Feel free to use and let me know of bugs.

sorry to report it still does not work, under the latest insider version :^(

sorry to report it still does not work, under the latest insider version :^(

@lwb52 I have made updates to the code and installation instructions. Original ones contained error. Note that I am talking about the Iris plugin, not this one here.

@lwb52 I have made updates to the code and installation instructions. Original ones contained error. Note that I am talking about the Iris plugin, not this one here.

Closing in favor of #3 as it is the same issue.

Closing in favor of #3 as it is the same issue.


Configuration: Replace generate-vscode-configuration with static analysis

In order to export the configuration options of the product we have a gulp task which launches VS Code and gets that information in a JSON file. This could be done differently by statically analysing the source code a-la nls.localize.
@roblourens, is there any reason why we never did this?

Yeah, config registration is dynamic code and needs to be executed. It comes from variables and function calls. nls limits what you can do in code, like you can only use string literals.

Oh yeah that was it. 🙈 D'oh... let's close this.


test

@chris-gong , it just this you need commit idk how you are going to get it you may have to merge


Add an optional opinionated module

Add a module that does all of the various revision loading and SQL table checking/creation implemented by projects such as idstore and cardant.

This is now the responsibility of darco.


NIFI-1762: Use Lambda Expressions in StandardNiFIServiceFacade to simplify codebase

Changed Java dependency to 1.8 instead of 1.7 and refactored StandardNiFiServiceFacade to make use of Lambda expressions to simplify code base. Also had to address a unit test because changing to Java 8 results in calls to assertEquals to become ambiguous

Reviewing...

Looks good! Love be able to write more concise code with Java 8 features. +1


8XYE

https://en.wikipedia.org/wiki/Most_significant_bit
https://en.wikipedia.org/wiki/CHIP-8
Bitshifts VX left by 1, VF is set to the value of the most significant bit of VX before the shift is performed

Implemented in 361a8c0


Merit order chart renders prior to ETEngine response

When going to quickly(?) for the batteries in households slider, the interface hangs:
console

@ChaelKruip remembered you, @antw already fixed this? But we couldn't find the old issue, so might be redundant.

@ChaelKruip remembered you, @antw already fixed this? But we couldn't find the old issue, so might be redundant.

#2356 is what you're thinking of. I do believe this has been fixed, but as I'm intending to do some work on chart loading in the next day or two, I will keep this open to remind me to double-check.

Thanks @antw closing this in favor of #2356

Reopening because I've encountered this several times today on beta.

I've encountered this several times today on beta.

... and now I can't reproduce it. I was also using production on the same day I (supposedly) encountered it on beta; perhaps I got my tabs mixed up. 😖
I'll close this; please reopen if it occurs again.


kapp-controller docs: typo in TOC

Hi, I noticed a minor typo in the table of contents for the kapp-controller documentation: Intergration should be Integration.

Hi @vixus0 Thanks for creating the issue. Would you like to submit a PR with a fix?

Sorry, I didn't see your reply! Anyway, I see you have a PR open now 🙂


added new API proposal [DO NOT MERGE]

!DO NOT MERGE!
We only use the review functionality for discussing the API for now.

I think we should separate the epoch data from the run meta data. Currently /run would return all runs with all epochs. So I suggest the following (missing) routes:

/run//epochdata -> returns all epoch data for the run

I would have simply returned the runs with empty epochdata arrays when we call /runs (also doesn't violate the schema definition), but it's probably better to have a separate call.

PUT /view with multiple run ids to create a new view -> return new view id
I didn't add a way to set the views yet because I'm still thinking about the automatic logging.

I would have simply returned the runs with empty epochdata arrays when we call /runs (also doesn't violate the schema definition), but it's probably better to have a separate call.

An empty array for the epoch data is also fine for me.

I didn't add a way to set the views yet because I'm still thinking about the automatic logging.

Ok, take your time. 😄

Discussion moved to https://github.com/gfrogat/confusionflow/issues/1.


Fix post main e2e upgrade job

Description
Changes proposed in this pull request:

Fix post main e2e upgrade job

Related issue(s)

https://github.com/kyma-project/nats-manager/issues/84

/retest


Next.js migration

Moved the HTML and CSS to Next.js pages, and the API routes for the Slack endpoints, the migration is ready and here are some key differences:

All Slack API calls are using the new OAuth recommended by Slack except one, the only way to invite an user is by using an undocumented API and a legacy token, therefore the current setup still requires a legacy token, the only difference is that an OAuth token can be used for the other API calls
The API section and iframe were removed
The old implementation has been deleted

Slack subdomain is not defined

@tanahatas Can you expand more on what do you mean?.
To make the app work you need to follow the steps in the readme (the readme in this PR)

How to input my slack subdomain like blablabla.slack.com ?
Because your PR using https://${teamName}.slack.com and ${teamName} is undefined.

@tanahatas The team name is returned by /api/v1/team, which is fetched in getInitialProps to return the info of the team to the components that require teamName.


bump wow to <IP_ADDRESS>840

note that this does not yet have C_Debug. going to change FrameXML_Debug to C_Debug.FrameXMLDebug, which will unblock progress, but which will also mean inconsistency with the client that has more stuff in that API


Blog post 'mkdocs.structure.files.File object' has no attribute 'alternates'

Hi,
I'm trying to set up a mkdocs-material blog with mkdocs-static-i18n.
I keep banging my head against this error.
...
mkdocs_static_i18n/custom_i18n_sitemap/sitemap.xml", line 34
...
jinja2.exceptions.UndefinedError: 'mkdocs.structure.files.File object' has no attribute 'alternates'

I'm not sure what's happening, or even if I need a sitemap.
I can work around it like this
    {%- if file.alternates is defined -%}
      {%- for i18n_locale, i18n_file in file.alternates.items() %}

          {%- if i18n_file.page.canonical_url %}
              {%- set i18n_href = i18n_file.page.canonical_url | e %}
          {%- elif i18n_file.page.abs_url %}
              {%- set i18n_href = i18n_file.page.abs_url | e %}
          {%- else %}
              {%- if i18n_file.url == "./" %}
                  {%- set i18n_href = config.site_url %}
              {%- else %}
                  {%- set i18n_href = config.site_url ~ i18n_file.url %}
              {%- endif %}
          {%- endif %}
          <xhtml:link rel="alternate" hreflang="{{ i18n_locale }}" href="{{ i18n_href }}"/>
      {%- endfor %}
    {%- endif -%}

But that's probably not very convincing.
I'm pretty sure this also happened with the version I had installed before 1.0.0

@buttle hi, can you share your versions of:

mkdocs-static-i18n
mkdocs-material
mkdocs

and any other plugin you use?

mkdocs-static-i18n: 1.0.2
mkdocs-material: 9.2.5
mkdocs: 1.5.2

Here's my src
https://gitlab.com/liberaforms/blog.liberaforms.org
I'm not sure if this is involved so I mention it here instead of opening another issue.
I navigate to
http://<IP_ADDRESS>:8000/2022/06/11/version-212/ 200
switch lang
http://<IP_ADDRESS>:8000/es/posts/releases/v2.1/ 404
Setting does not help
- blog:
    post_url_format: "{categories}/{slug}"`

results in
http://<IP_ADDRESS>:8000/releases/version-212/ 200
switch
http://<IP_ADDRESS>:8000/es/posts/releases/v2.1/ 404
Cheers.

@buttle FYI we are aware with Martin of mkdocs-material that newer blog plugin versions are not compatible with the i18n plugin anymore.
I'll have to find some time to fix that in a future release and will make sure this address your problems here too.

I know it can be hard to find time, so thanks for that. Much appreciated! :)


Error: Timeout of 5000ms exceeded on create

When sending a create request to the Server, if the same application is logged with the same user it generates the error (Timeout of 5000ms exceeded calling create on comando) when sending the request.
I noticed that this happens only when there are 2 users connected with the same username.
I also noticed that the error happens on the client because the server does not receive this request, ie the client kind of loses the connection temporarily with the server socket.
To reproduce the error: Create an Angular application using SocketIO Client, perform authentication in 2 or more tabs and execute any POST (create) call to the server, one of them will hang generating the error (Timeout of 5000ms exceeded calling create on ...)

I'm getting the same issue with one of services on find method. However server gets data and executes method but clients receive events emitted in service after a long time.
I supposed issue occurred because I'd updated socketio-server and hadn't updated client. But after I've done it, problem still takes place.
N.B. Issue takes place only with one service which has some state (ex. array with data passed from clients). I can't understand why data from client to server passes immediately but clients get events from service for about a half of minute or more

This is an Angular problem and not reproducible with any other client. There are already several related issues, specifically:

https://github.com/feathersjs-ecosystem/feathers-chat-angular/issues/8
https://github.com/feathersjs/socketio-client/issues/9

Which seem to have been solved by ensuring the latest version of all packages are installed. Besides that I can only repeat what I asked @abalad before which is a link to a repository that easily allows to reproduce the issue.

Sorry @daffl I created the issue here and will later enclose an example to demonstrate

@daffl  They created an example to test the Zone: https://github.com/JiaLiPassion/zone-socketio

I added DEBUG to Feather's serever and I was able to track down the following:
Note: I tested with authentication method, But I believe the problem is affecting the entire application when sending a POST to the server.
Server Dependencies
  "dependencies": {
    "@feathersjs/authentication": "^2.1.3",
    "@feathersjs/authentication-jwt": "^2.0.0",
    "@feathersjs/authentication-local": "^1.1.1",
    "@feathersjs/configuration": "^1.0.2",
    "@feathersjs/errors": "^3.3.0",
    "@feathersjs/express": "^1.2.0",
    "@feathersjs/feathers": "^3.1.4",
    "@feathersjs/socketio": "^3.2.0",
    "compression": "^1.7.2",
    "cors": "^2.8.4",
    "debug": "^3.1.0",
    "feathers-authentication-hooks": "^0.1.7",
    "feathers-hooks-common": "^4.10.0",
    "feathers-rethinkdb": "^0.5.1",
    "helmet": "^3.12.0",
    "rethinkdbdash": "^2.3.31",
    "serve-favicon": "^2.4.5",
    "winston": "^2.4.0"
  },
  "devDependencies": {
    "eslint": "^4.19.1",
    "mocha": "^5.0.5",
    "request": "^2.83.0",
    "request-promise": "^4.2.2"
  }

Case A - Open a tab in the browser of your application it will create a connection with the server. (In this case the authentication works normally)
Case B - Open a second tab in the browser and authenticate, and instantly arrives on the server with the following messages:
  engine:ws received "423["authenticate",{"strategy":"local","email":"sad","password":"sadsada"}]" +24s
  engine:socket packet +24s
  socket.io-parser decoded 23["authenticate",{"strategy":"local","email":"sad","password":"sadsada"}] as {"type":2,"nsp":"/","id":3,"data":["authenticate",{"strategy":"local","email
":"sad","password":"sadsada"}]} +9m

But after a while (10 ~ 15 seconds) it responds again with the following messages:
  socket.io:socket got packet {"type":2,"nsp":"/","id":3,"data":["authenticate",{"strategy":"local","email":"sad","password":"sadsada"}]} +9m
  socket.io:socket emitting event ["authenticate",{"strategy":"local","email":"sad","password":"sadsada"}] +0ms
  socket.io:socket attaching ack callback to event +0ms
  socket.io:socket dispatching an event ["authenticate",{"strategy":"local","email":"sad","password":"sadsada"},null] +0ms
  @feathersjs/authentication:hooks:authenticate Attempting to authenticate using local strategy with options { name: 'local',
  usernameField: 'email',
  passwordField: 'password',
  entity: 'user',
  service: 'usuario',
  passReqToCallback: true,
  session: false } +9m
  @feathersjs/authentication:passport:authenticate Passport request object { query: { strategy: 'local', email: 'sad', password: 'sadsada' },
  body: { strategy: 'local', email: 'sad', password: 'sadsada' },
  params:
   { query: {},
     provider: 'socketio',
     headers: {},
     session: {},
     cookies: {} },
  headers: {},
  cookies: {},
  session: {} } +9m
  @feathersjs/authentication-local:verify Checking credentials sad sadsada +9m
  engine:ws received "2" +875ms
  engine:socket packet +875ms
  engine:socket got ping +0ms
  engine:socket sending packet "pong" (undefined) +0ms
  engine:socket flushing buffer to transport +0ms
  engine:ws writing "3" +0ms
  @feathersjs/authentication-local:verify a record with email of 'sad' did not exist +15ms
  @feathersjs/authentication:passport:authenticate Authentication strategy 'local' failed { message: 'Invalid login' } undefined +15ms
  @feathersjs/errors NotAuthenticated(401): Invalid login +9m
  @feathersjs/errors {} +0ms

Then in that period that the server returned no response from the client application, it returns with the following Authentication Timeout error


add additional hardware support for Hexiwear K64 module

This commit extends the support for the Hexiwear module to include more (but not yet all) of the internal components.   The red, green and blue LED pins have been added to the board/pinmux definition.  The haptic feedback motor has been defined (based on CONFIG_HAPTIC_FEEDBACK).
The docking station interfaces have been defined (based on CONFiG_HEXIWEAR_DOCKING_STATION) and includes all of the pinmux settings for the Click board interfaces (GPIO, I2C, SPI).
SPI 0-2 devices have been added (based on the schematic).   SPI0 has been tested with input and output devices on the Click interfaces.
Documentation has been extended for these new BSP settings, and links to more of the internal components have been added.
Some sample programs have been added, as tests/examples for these hexiwear specific changes.   I will note that I'm new to Zephyr, so I do not claim these examples use best practices for Zephyr apps.   Suggestions gladly welcome.

I'm withdrawing this pull request, and will soon create a new one rebased against the origin/arm branch.   I've cleaned up the commit history, and fixed up the doc/RST issues.


 Did you read @LMnet's comment about it?

@ground5hark Is there any specific reason why you decided to choose natural ordering?

Ah I merged the wrong one. Ooops. Going to fix that now.
On Wed, Dec 9, 2015 at 7:08 PM, Yuriy Badalyantc<EMAIL_ADDRESS>wrote:

@ground5hark https://github.com/ground5hark Is there any specific
reason why you decided to choose natural ordering?
—
Reply to this email directly or view it on GitHub
https://github.com/ground5hark/sbt-concat/pull/17#issuecomment-163471755
.

@ground5hark Thanks! Did you release a new version? Could you git tag it?


Chapter "text" from the knowledge model should be presented in the user interface.

Implemented in c4d1795 as chapter introduction InfoFI inside chapter items before all the questions.


Starting... 'git' is not recognized as an internal or external command, operable program or batch file. Traceback (most recent call last):   File "C:\Users\PC\Documents\GitHub\MusicBot\update.py", line 40, in main     subprocess.check_call('git --version', shell=True, stdout=subprocess.DEVNULL)   File "C:\Users\PC\AppData\Local\Programs\Python\Python39\lib\subprocess.py", line 373, in check_call     raise CalledProcessError(retcode, cmd) subprocess.CalledProcessError: Command 'git --version' returned non-zero exit status 1.  During handling of the above exception, another exception occurred:  Traceback (most recent call last):   File "C:\Users\PC\Documents\GitHub\MusicBot\update.py", line 76, in      main()   File "C:\Users\PC\Documents\GitHub\MusicBot\update.py", line 42, in main     raise EnvironmentError("Couldn't use Git on the CLI. You will need to run 'git pull' yourself.") OSError: Couldn't use Git on the CLI. You will need to run 'git pull' yourself.

Please tick all applicable boxes.

[X] I am using Python 3.5.3 or higher (run python --version on the command line)
[X] I have followed the official guides to install the bot for my system
[x] I have updated my dependencies to the latest version using the appropriate update script

Which version are you using?

[X] The latest master version (release-260419)
[ ] The latest review version

What type of issue are you creating?

X] Bug
[ ] Feature request
[ ] Question

Description of issue
Starting...
'git' is not recognized as an internal or external command,
operable program or batch file.
Traceback (most recent call last):
File "C:\Users\PC\Documents\GitHub\MusicBot\update.py", line 40, in main
subprocess.check_call('git --version', shell=True, stdout=subprocess.DEVNULL)
File "C:\Users\PC\AppData\Local\Programs\Python\Python39\lib\subprocess.py", line 373, in check_call
raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command 'git --version' returned non-zero exit status 1.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\Users\PC\Documents\GitHub\MusicBot\update.py", line 76, in
main()
File "C:\Users\PC\Documents\GitHub\MusicBot\update.py", line 42, in main
raise EnvironmentError("Couldn't use Git on the CLI. You will need to run 'git pull' yourself.")
OSError: Couldn't use Git on the CLI. You will need to run 'git pull' yourself.
Run.bat is not working and update.bat is giving me that

Log file  musicbot.log

You need to install git-scm or make sure your pc have the path on git.exe


autocomplete prevents entering value with capital letter

When saving a changeset, and adding a Sources manually, if I try to add a custom value of "LPI Imagery", I do a SHIFT+L to get a capital L, iD changes this to lowercase l to match the completion local_knowledge as I continue to type with SHIFT down to get capitals it ends up with lPI Imagery rather than LPI Imagery.
My workaround is to type lLPI Imagery then go back and delete the first character.

iD already had some code to support case sensitive combo fields, but it was only used to avoid autocompleting the user's changeset comment.  I just added an option for other fields to enable this behavior, and added it to the source=* field.
Put another way, it means that now the combo won't try to "correct" the user's typing to be lowercase.


Create a profile for a property, including photos, description and house rules (as a host)

Maybe this should be in sprint 1 ?


Access to Node Child Process Api

Its possible?

Hey, sure. either you can remote require it
or you can invoke an IPC message which can be handled by the main process which can again use child process API offered by Node.JS.


feat: Fixed eBPF Feature Detection

On working to migrate to cilium/ebpf I started to hit issues with
nil pointer dereferences that indicated that something odd was happening
between eBPF feature detection and metrics export.
I discovered that there were additional global variables in many of the
packages whose values are set before eBPF feature detection has
happened.
It appears that in these cases - through the various layers of
indirection - we may try and read/write metrics that do not exist
causing a nil pointer dereference.
There appear to be several places where we're accessing map keys
without checking they are nil - that will need to be fixed in a
follow up. This PR attempts to at least ensure that from the BPF
side that the keys we expect to exist, do actually exist in the
maps that matter.
eBPF feature detection how produces a bpf.SupportedMetrics
struct, which is now consumed in various places in the codebase
to ensure that the metrics we're exporting match those supported
by the eBPF exporter.

pls share the NPE, and steps to reproduce

NPE:
goroutine 71 [running]:
github.com/sustainable-computing-io/kepler/pkg/collector/stats/types.(*UInt64StatCollection).SumAllAggrValues(0x16d0[40](https://github.com/sustainable-computing-io/kepler/actions/runs/9114066283/job/25057276989?pr=1438#step:11:41)0?)
	/workspace/pkg/collector/stats/types/types.go:144 +0x16
github.com/sustainable-computing-io/kepler/pkg/metrics/utils.CollectResUtil(0x16d3460?, {0x179f7e0?, 0xc0000e81c0?}, {0x18f5588, 0xd}, {0x1b67f28, 0xc000066368})
	/workspace/pkg/metrics/utils/utils.go:1[41](https://github.com/sustainable-computing-io/kepler/actions/runs/9114066283/job/25057276989?pr=1438#step:11:42) +0x6ed
github.com/sustainable-computing-io/kepler/pkg/metrics/utils.CollectResUtilizationMetrics(0x1?, {0x179f7e0, 0xc0000e81c0}, 0xc0029d8180?, 0x0)
	/workspace/pkg/metrics/utils/utils.go:48 +0xc5
github.com/sustainable-computing-io/kepler/pkg/metrics/container.(*collector).Collect(0xc0025f8ed0, 0xc000061f60?)
	/workspace/pkg/metrics/container/metrics.go:105 +0xd8
github.com/prometheus/client_golang/prometheus.(*Registry).Gather.func1()
	/workspace/vendor/github.com/prometheus/client_golang/prometheus/registry.go:[45](https://github.com/sustainable-computing-io/kepler/actions/runs/9114066283/job/25057276989?pr=1438#step:11:46)5 +0x102
created by github.com/prometheus/client_golang/prometheus.(*Registry).Gather in goroutine 1
	/workspace/vendor/github.com/prometheus/client_golang/prometheus/registry.go:5[47](https://github.com/sustainable-computing-io/kepler/actions/runs/9114066283/job/25057276989?pr=1438#step:11:48) +0xbab

utils.CollectResUtil is the offending function, where it's assuming that a hashmap entry exists - which it may not.
As far as I can tell the NPE was caused by the following chain of events:

CollectResUtilizationMetrics gets called  via the prometheus collector
It iterates over several arrays of constants e.g consts.SCMetricNames and consts.IRQMetricNames and then calls down into utils.CollectResUtil

NOTE: config.IsIRQCounterMetricsEnabled() gets checked before it tries to do anything with the IRQ metrics

CollectResUtil takes an interface {} argument (in this case operating on a ContainerStats object)
It indexes directly into ContainerStats.ResourceUsage["bpf_net_tx_irq"] without checking it exists

In which circumstances would "bpf_net_tx_irq" not exist then?

We can trace NewContainerStats -> NewStats and see that the ResourceUsage map is populated from a global variable called AvailableBPFSWCounters
InitAvailableParamAndMetrics sets the global ☝ after the bpf programs are loaded - it was empty beforehand

So the only way this can occur is when config.IsIRQCounterMetricsEnabled() == true BUT  we don't have any of the IRQ keys in the array of supported eBPF software counters. I'll continue to debug here to see why that's happening.
I'm not sure what I managed to do with the cilium/ebpf port and with this patch to make that issue worse... but there is definitely something fishy happening 🐟  Still trying to untangle this to find out what's going on.

NPE:

on which commit? does main + this PR reproduce it? or main + cillium PR ?

NPE:

on which commit? does main + this PR reproduce it? or main + cillium PR ?

got it. main + cillium PR

main + cillium PR ?

That will reproduce it (for now). I'm only able to reproduce it in CI so using this PR to help debug it.

This PR is ready to go.
As for the root cause, I ended up hooking up a debugger and I found out that the issue was that in #1413 I'd forgotten to attach one of the perf_events (config.TaskClock). I've fixed this and I'll push into CI tomorrow to see if it resolves the NPE.
Either way the issue is basically because we have 2 competing definitions of "what's supported by ebpf". This PR fixes that.


Fixed #29948 -- Update last TransactionTestCase serialized_rollback behaviour

Use the rollback setting from the last test in the fixture teardown instead of always considering that we should rollback at the end of a test suite using TransactionTestCase.

buildbot, test on oracle.

The commit that caused this regression is reverted in 4c7c608a1deee37055d4a2b8a71e34def04a2a1f.


Problem with loading lua .so library when running from another .so

I have a bit complicated architecture - I'm using python to run C++ code loaded as module (with boost::python wrappers), and the C++ code loads lua with your great library (I'm really impressed how easy it was to integrate my C++ classes with lua code). It's all on Ubuntu 20.04. It all works great and objects fly between python, C++, lua and then back - without any problems.
But then I tried to add some external library to lua, in particular - rex_pcre. So I compiled it with luarocks - and when I run it from luajit interpreter, it works as expected. But if I try to load it from the script run from my C++ code, it breaks:
Error: error loading module 'rex_pcre2' from file '/usr/local/lib/lua/5.1/rex_pcre2.so':
	/usr/local/lib/lua/5.1/rex_pcre2.so: undefined symbol: lua_checkstack
stack traceback:
	[C]: in ?
	[C]: in function 'require'
	/usr/lib/logzilla/t/common_files/lua/test.lua:1: in main chunk

Scripts is just like that:
local rex = require "rex_pcre2"

I tried to add option for "export dynamic" which I found in some other bug - but it didn't change anything. My .so file (used then as a python module) is now being linked like that (with clang++ v. 10.0.1):
/usr/bin/clang++ -fPIC -std=c++17 -fno-builtin-malloc -fno-builtin-calloc -fno-builtin-realloc -fno-builtin-free -O3 -DNDEBUG -Wl,--no-undefined -Wl,-export_dynamic -shared -Wl,-soname,parser.so -o parser.so CMakeFiles/parser.dir/pym_parser.cpp.o  ../logzilla/liblogzilla.a -lboost_date_time -lboost_filesystem -lboost_program_options -lboost_python38 -lboost_regex -lboost_system ../libMurmurHash3.a ../crc32/libcrc32.a -lcrypto -ldl ../libinfluxdb.a -lleveldb -lluajit-5.1 -lpthread -lpython3.8 -lsnmp -lssl ../libutils.a -lzmq -lcurl -lboost_date_time -lboost_filesystem -lboost_program_options -lboost_python38 -lboost_regex -lboost_system ../libMurmurHash3.a ../libinfluxdb.a -lpython3.8 ../libeasylogging.a

Here's the C++ part running the script:
            sol::state lua;
            lua.open_libraries(sol::lib::base);
            lua.open_libraries(sol::lib::package);
            auto res = lua.safe_script_file(script_path, sol::script_pass_on_error);
            if (!res.valid()) {
                sol::error err = res;
                cerr << "Error: " << err.what() << endl;
            }

I don't  provide isolated test case as I hoped that maybe you could help me based on this data - but if not, then I'll try to provide some stripped down code so you an replicate it. Or maybe I'll find solution myself, while simplifying things. Just let me know.

I was able to simplify test case a bit, even not using boost::python. So, there's a function in shared library, and there's an executable loading and running this function width dlopen and dlsym:
luatest_lib.cpp:
#include <iostream>
#include <sol/sol.hpp>

using namespace std;

extern "C" { // for not mangling function name
    void external_test() {
        sol::state lua;
        lua.open_libraries(sol::lib::base);
        lua.open_libraries(sol::lib::package);

        string script_path = "test.lua";

        cout << "This is library... Running script:" << endl;

        auto res = lua.safe_script_file("test.lua",
                sol::script_pass_on_error);
        if (!res.valid()) {
            sol::error err = res;
            cerr << "Error: " << err.what() << endl;
        }
        else {
            cout << "All's fine" << endl;
        }
    }
}

and luatest.cpp:
#include <iostream>
#include <dlfcn.h>

using namespace std;

int main(int argc, char**argv) {

    void *l = dlopen("./libluatest_lib.so", RTLD_LAZY);
    if (!l) {
        cerr << "Error loading lib: " << dlerror() << endl;
        return 1;
    }

    void (*fn)() = (void(*)())dlsym(l, "external_test");
    if (!fn) {
        cerr << "Error getting function: " << dlerror() << endl;
    }

    cout << "Calling loaded function..." << endl;
    fn();
    cout << "Done" << endl;

    dlclose(l);
    return 0;
}

And CMakeLists.txt:
add_library(luatest_lib SHARED luatest_lib.cpp)
target_link_libraries(luatest_lib luajit-5.1)

add_executable(luatest luatest.cpp)
target_link_libraries(luatest dl)

After compiling and running:
$ ./luatest
Calling loaded function...
This is library... Running script:
Error: error loading module 'rex_pcre2' from file '/usr/local/lib/lua/5.1/rex_pcre2.so':
	/usr/local/lib/lua/5.1/rex_pcre2.so: undefined symbol: lua_checkstack
stack traceback:
	[C]: in ?
	[C]: in function 'require'
	test.lua:1: in main chunk
Done

But now, there's an easy fix - if I link my executable with libluajit-5.1:
add_executable(luatest luatest.cpp)
target_link_libraries(luatest dl luajit-5.1)

...then everything works fine:
$ ./luatest
Calling loaded function...
This is library... Running script:
123
OK
All's fine
Done

The only problem is, that in my real live case the executable is python interpreter - so I'd have to recompile python and link it with lua - which is probably not the best solution... So now, I'd like to fiddle with my *.so library and somehow make it load missing symbols from libluajit-5.1.so.2 somehow. Or maybe tell sol/lua to do it. Any ideas?

OK, so I solved this on myself - the answer is in the manual of dlopen:

RTLD_GLOBAL
The symbols defined by this shared object will be made available
for symbol resolution of subsequently loaded shared objects.

So in my simplified case if I change dlopen to dlopen("./libluatest_lib.so", RTLD_NOW|RTLD_GLOBAL); everything works fine. And lucky for me - there's also a way to change this behaviour in the python.  So it's not much with this library, you can remove this issue if you want, but I think it could be left here if anybody encounters similar issue in the future.


[DOCFIX] Update POSIX API doc

Improve and POSIX API docs, especially simplifying the setting due to https://github.com/Alluxio/alluxio/pull/12857/

alluxio-bot, merge this please


Build Artefact Package

It would be nice to have a dedicated output containing the library and a header file.

The make file does not copy the lib file and the header files into a out dir.


Switching to go

Hey there Inexor team. I have been watching the entity-system development for some time now, and have come to a conclusion (for myself) that it would be good to consider an implementation alternative.
I am making this proposal now, because the more time we spend into the prototype of the entity-system with one technology, the less likely alternatives will be considered.
Disclaimer: This is not meant to insult anyone's work and effort put into the current prototype, it is just an observation.
I see that even with a small amount of functionality we already have a huge code base (20k LoC) and many compilation issues, that will not go away without huge efforts.
Until now there has not been a single case with C/C++ pointer magic (because we avoid it with smart pointers) which couldn't be as good (and with the same or better performance) handled by go. However the maintenance burden with go is much smaller as we currently have it.
I have implemented a proof-of-concept in golang covering the very basic functionality of the entity-system and would like to evaluate why I think it is usefull.
Currently I have implemented:

a type system on top of go reflections
attributes and attribute instances
entities which can have attributes and entity instances
connectors

It is possible to add and list attributes and entities. Since this is only a PoC I did not implement any functionality, such as the REST-server. I am also not a full-blown gopher, so there might be more elegant ways to do things.
My recap from implementing this PoC is what I would like to present:
Pro

I was able to re-create the basic principles of the entity-system in < 400 LoC
it compiles out of the box on all platforms

no setup time

produces a single binary
we get all the batteries included that golang has, most importantly: logging, sync, reflection, http, json
our problems with concurrency would be solved as golang introduces lightweight concurrency threads: goroutines
I tested the OpenAPI generator for golang and it's quit good (much better than restbed)

Con

Go uses type switches and composition over inheritance, so our code would look slightly different (meaning that we have to adpot to these patterns)
some parts, such as graphics, will still be needed to be managed with C/C++ (cgo)
we throw away the current implementation effort

I do not suggest to fork my repo because I am sure there are better ways to do things (I did this in one day).
However it can serve as a good base for discussion and could easily replace the entity-system with a little work.
What's currently missing is:

relations (copy the code for entities)
make the connector more reactive by moving the channel inside SetValue via DI
make e.g the UUID methods more generic / DRY
add logging via go logging module
based on above we can easily add REST and renderer in < day

Using golang is not my only ultimate goal, with this issue I wanted to showcase that we can do the ES nicely in another language, saving us massive amount of work.

This was collectively dismissed as unfeasible.