Quart and accessing AsyncLCDClient globally?

I have some experience with other languages so I should be able to find my way through the basics of python but I am really struggling with the added quart and async aspect, which has been recommended to me instead of the sync client with flask when the sync client instantiated globally was generating async errors on POST request. I have completed the Terra SDK basics video tutorial, searched github for code where AsyncLCDClient and @app.before_serving and async def startup() are used, read through the quart tutorials, watched tutorials from the quart dev and on async but I am still stuck trying to get the basic setup of instantiating the async client, and then being able to access it and methods of it in the same function and globally as will be required.
Current code:
from quart import Quart, request
# from terra_sdk.client.lcd import LCDClient
from terra_sdk.client.lcd import AsyncLCDClient
from terra_sdk.key.mnemonic import MnemonicKey
import json, config

from terra_sdk.client.lcd.api.bank import BankAPI

# Sync version which allows global access but generates an *ASync* error on POST.
# terra = LCDClient("https://bombay-lcd.terra.dev", "bombay-12")
# wallet = terra.wallet(mk)

app = Quart(__name__)

# mk = MnemonicKey(mnemonic=config.MNEMONIC) // Production
mk = MnemonicKey()

@app.before_serving
async def startup():
    app.terra = AsyncLCDClient("https://bombay-lcd.terra.dev", "bombay-12")

    # Not recognised:
    # wallet = await app.terra.wallet(mk)
    # wallet = await terra.wallet(mk)

@app.route('/webhook', methods=['POST'])
async def webhook():
    data = json.loads(await request.data)

    # Not recognised:
    # info = await terra.tendermint.node_info()
    # info = await app.terra.tendermint.node_info()

    if (data['passphrase']) != config.WEBHOOK_PASSPHRASE: #Upgrade to bcrypt.

        return {
            "code": "error",
            "message": "invalid passphrase"
        }
    else:
            # Placeholder for order function call
            return {
             "code": "success",
             "message": "authorised"
         }

# async def order():
    # Not recognised:
    # balance = await terra.bank.balance(mk.acc_address)

    # Other order logic to swap tokens using functions from AsyncLCDClient

if __name__ == "__main__":
    app.run()

I was told I should be able instantiate it to app.terra, and that I should then be able to access it but I am not able to either in the same function or globally as I will need to.
Someone else recommended today I instantiate AsyncLCDClient instead in the global scope and then use modules and methods of it in async functions:
terra = AsyncLCDClient("https://bombay-lcd.terra.dev", "bombay-12")

@app.route('/webhook', methods=['POST'])
async def webhook():
    data = json.loads(await request.data)

    info = await terra.tendermint.node_info()
    print(f"Node info {info}")

Although I could then access terra globally this resulted in the following error:
   File "/home/hashewnuts/cloud/code/VisualStudioCode/quart-test/venv/lib/python3.8/site-packages/aiohttp/helpers.py", line 701, in __enter__
    raise RuntimeError(
RuntimeError: Timeout context manager should be used inside a task

I have seen aiohttp referred to as outdated and unmaintained, and as I understand it quart should be used without it. Searching for the error I also found this thread where pgjones recommends using a startup function as I have been.
I would really appreciate if someone could tell me where I am going wrong, and if possible provide me with a very basic template of how to properly instantiate terra Async using quart and access it globally that I can follow as I progress through my project.
Dev confirms using startup: https://terra-money.github.io/terra.py/guides/async.html

I am unsure what changed but once I do not await wallet and I do await info I am now able to instantiate them in startup and in the route :)


Client crash when a block loads.

Issue description:
When anyone on my server loads at spawn, they crash never to be able to rejoin (including myself) because of the block. I have gone over with mcedit but it seems like it a common block yet im unable to know which one it is. Please help so i can get my server back up
Versions:

Minecraft: 1.12.2
Forge: 1.12.2 - <IP_ADDRESS>11
McJtyLib: 1.12 - 2.6.4
RFTools: 1.12 - 7.29

crash-2018-03-02_22.05.27-client.txt

There is an option in forge.cfg that will automatically delete tile entities that cause crashes. That might get your server back and running but your elevator block will be gone. We will investigate the crash though

Thats the problem. there was no elevator blocks in the first place

That's not possible. There has to be an elevator block because it is crashing in an elevator block. Either way. That option in forge.cfg might fix it

Alright i'll look into it. Thanks =D

Try updating to RFTools 7.30.

Also, can you upload your world save?


fix setuptools: upgrade before installing requirements

Thanks for submitting a PR! Please check the boxes below:

[x] I have run pre-commit to check linting
[ ] I have filled in the "Changes" section below?
[x] I have filled in the "How did you test this code" section below?

Changes
Upgrade setuptools before installing any packages
How did you test this code?

Should not be needed

closing for https://github.com/Flagsmith/flagsmith/pull/1757


Comprehensive Review and Enhancement of Kubernetes and Kyma UI Views

Description
As Sebastian and Matthew, I would like Busola to comprehensively review and enhance all native Kubernetes resources and Kyma modules. This includes ensuring adherence to design guidelines, completeness of required fields such as Resource Metadata and Status, attachment of Events, inclusion of Resource Graphs, Overview cards, and Monitoring and Health views where needed, to streamline development and operational tasks within the Kyma environment.
Acceptance Criteria

[ ] All native Kubernetes resources and Kyma modules are reviewed against the current design guidelines.

[ ] Each resource/module includes:

[ ] Resource Metadata and Status fields.
[ ] Events (if applicable).
[ ] Resource Graph (if applicable).
[ ] Overview cards (if applicable).
[ ] Monitoring and Health views (if applicable).

[ ] The following Kubernetes native resources are specifically reviewed and enhanced where necessary:
Kyma Resources

[ ] Check ALL Kyma Resources

Workloads

[ ] Pod
[ ] ReplicaSet
[ ] Deployment
[ ] StatefulSet
[ ] DaemonSet
[ ] Job
[ ] CronJob

Services & Load Balancing

[ ] Service
[ ] Ingress
[ ] Endpoints

Config & Storage

[ ] ConfigMap
[ ] Secret
[ ] PersistentVolumeClaim
[ ] StorageClass
[ ] Volume

Cluster

[ ] Namespace
[ ] Node
[ ] PersistentVolume
[ ] ResourceQuota
[ ] ServiceAccount
[ ] Role
[ ] RoleBinding
[ ] ClusterRole
[ ] ClusterRoleBinding
[ ] NetworkPolicy
[ ] PodSecurityPolicy

Metadata

[ ] LimitRange
[ ] HorizontalPodAutoscaler
[ ] PodDisruptionBudget
[ ] PriorityClass

Business Value

Enhances the developer and operator experience within Busola by providing a more intuitive, informative, and integrated interface.
Reduces the learning curve for new users and accelerates the development cycle for seasoned users.
Aligns Busola's representation of Kubernetes and Kyma resources with the best practices and standards, ensuring a reliable and high-quality user experience.

Reasons

A comprehensive review and enhancement of Busola's Views will ensure that users have all necessary information and tools at their fingertips, promoting efficiency and reducing the potential for errors.

Non-functional Requirements

The enhancements should be implemented without significant impacts on the performance or responsiveness of the Busola dashboard.
All new functionalities and enhancements must be compatible with the latest versions of major browsers and follow the Fiori Guidelines.

Dependencies

Coordination with the UI/UX team for design guideline updates and mockups.

Definition of Done (DOD)

Code is reviewed and merged into the main branch.
All acceptance criteria are met and functionality is verified by both product owners and UX.
Integration tests cover all new features and are passing.

Notes and Comments

Ongoing discussion and collaboration with the Kyma UX team are crucial for the success of this epic.

Useful Links

Figma Designs

Checked with k8s api what is missing:
Workloads

Pod
ReplicaSet
Deployment
StatefulSet
DaemonSet
Job
CronJob

Discovery and Network

HorizontalPodAutoscaler


Move debugger implementation

Moved the android debugger implementation to the project template leaving native calls to JsDebugger in the runtime
ping @slavchev @blagoev @Plamen5kov @atanasovg

:+1:

:+1:
Take care not to over-architect, doing it without any use case is dangerous. For example making the debugger less android dependent will not make the whole runtime less android dependent and there are currently no use cases for this.
To be clear the danger starts when no different debugger is required but we are doing the Debugger interface abstraction which might not work exactly as it is when the real use case presents itself.
Still good job on this one.


Add a guide for TTL testing.

Also did some passing-by fixes.

Preview is available here:http://developers-pr630.previews.kube001.services.stellar-ops.com


Add Python Snake Game project

Add a snake game in python which has following features:

Arena for snake to move
A snake with a finite speed
Food for snake to swallow
Snake length increases as it swallows the food
Scores should be displayed on screen.

Thanks, Gazal.
Keep contributing.

Hi @Arindam200 , I have left some comments on the PR which you recently merged related to this issue, can you please take a look at it and help me out.


Got a parsing error

<EMAIL_ADDRESS>watch
tsx ./src/watcher.ts

[i] Purging cache (PURGE_CACHE set to true)..
[i] Watching /var/tmp/e2g-input (using polling)..
[i] Cache purged!
[i] Found Transactions(1).csv!
[i] Determined the file type to be of kind 'degiro'.
[i] Found xStation5_cashOperations_2770548_31-12-2006--29-05-2024.csv!
[i] Determined the file type to be of kind 'xtb'.
[i] Restored 0 ISIN-symbol pairs and 0 symbols from cache..
[i] Processing file using DeGiro converter
[i] Restored 0 ISIN-symbol pairs and 0 symbols from cache..
[i] Processing file using XTB converter
[e] An error ocurred while processing.
[e] Error details: Error: An error ocurred while parsing!
[e] Moving file to output..
[i] Read CSV file. Start processing..
/app/src/converters/xtbConverter.ts:112
const quantity = parseFloat(match[1]);
^
TypeError: Cannot read properties of null (reading '1')
at  (/app/src/converters/xtbConverter.ts:112:45)
at Parser. (file:///app/node_modules/csv-parse/lib/index.js:102:7)
at Parser.emit (node:events:519:28)
at Parser.emit (node:domain:488:12)
at endReadableNT (node:internal/streams/readable:1696:12)
at process.processTicksAndRejections (node:internal/process/task_queues:82:21)
Node.js v20.13.1

Example File
xStation5_cashOperations_2770548_31-12-2006--29-05-2024.csv

Thanks for reporting! I think there is an issue with the example file you provided. When I click it, I get a "Not found". Could you try and upload it again?

Hello dickwolff,
please find attached a sample file exported from xstation5 (XTB).
xStation5_cashOperations_2770548_31-12-2006--29-05-2024(3).csv
Best regards,
TVB

Incan download the CSV file now, thanks! I will check it out tomorrow and see if I can find the problem. I will get back to you once I know more!

The root cause of the issue was buy/sell orders not being recognized due to language difference. I assumed it was Spanish (compra for "buy"), and also added vende for "sale". Please let me know if this is the right term to use!
I added a fix to the 0.14.0-beta image, which you can pull from Docker Hub. Please let me know if this resolves the parsing issue!

Tested and working. "vende" and "compra" are the right words. It is Portuguese.


Fix the bug that RemotingServer can be closed repeatedly.

修复AbstractRemotingServer#stop并发控制的BUG
移除了inited状态和init方法
修复了相关的单测

Fix #82 .


fix(custom): fix elements may not be removed after updates #17333

Brief Information
This pull request is in the type of:

[x] bug fixing
[ ] new feature
[ ] others

What does this PR do?

When custom series is updated, if the renderItem returns a group that contains null for an element that was previously in the group, it will not be removed, causing some of the elements leave behind by mistake.
Fixed issues
#17333
Details
Before: What was the problem?

There is an extra element after calling setOption for the second time.
After: How does it behave after the fixing?

No extra element after calling setOption for the second time.
Document Info
One of the following should be checked.

[x] This PR doesn't relate to document changes
[ ] The document should be updated later
[ ] The document changes have been made in apache/echarts-doc#xxx

Misc
ZRender Changes

[ ] This PR depends on ZRender changes (ecomfe/zrender#xxx).

Related test cases or examples to use the new APIs

added a test case custom-update.html
tested and passed the visual tests of current cases with "custom" in their names

Others
Merging options

[ ] Please squash the commits into a single one when merging.

Other information

I'm not pretty sure but I think it's by design at original. Like the logic of option merging, giving null means not updating this element in the default merge mode. If we wan't to remove it, we need to change the $mergeChildren to false, which will loose the transition. Perhaps @100pah can give more explanation on this.
But from my side. I think the change in this PR will let code much more intuitionistic. So I will give +1 on this change(and a bit more explanation on the doc will be better). The only change I need to request is that we still need to applyLeaveTransition when removing the child instead of using remove directly

@pissang @100pah Thanks for the review. I updated the comment and add a test case where a child of new group returned from the custom series is {}, which should preserve the old element. I will also update the document about this.


updates the cmd file so that if the environment is currently being emulated then we do not install the agent or WSM

Synopsis
If you start an Azure Cloud Service locally in the Compute Emulator while the New Relic .NET NuGet package is installed, the role won't start.
To reproduce:

Create a Cloud Service in Visual Studio from a template like ASP.NET Web Role > MVC.

Install the NuGet package: install-package newrelicwindowsazure

Choose Start Debugging from the Tools menu.

The Compute Emulator will start and you'll get a Windows Installer prompt and/or a warning that the role is taking a long time to start. The Windows Installer prompt continues to prompt in a loop. Clicking Yes on the warning dialog doesn't have an effect on the loop.
Key changes
This change will cause the script to noop if it detects emulation
User visible effects
Users who use the compute emulator will now be able to start the emulator up without side effects from the New Relic nuget package
Sidekick
@enelson-newrelic

:+1:

:+1:


[WIP] Implementation AC/DC (moment buffer reset & finish training on decompression epoch)

AC_DC implementation ready, now testing!

@bfineran just for completeness, will the refactor of pruning modifiers be compatible with the current unit tests? Just wondering whether it makes sense for me to write tests before we move to the new modifier structure.

@bfineran thank you for your comments, excited about my first, real pull request :) Just for completeness, will the refactor of pruning modifiers be compatible with the current unit tests? Just wondering whether it makes sense for me to write tests before we move to the new modifier structure.

the general test structure and classes will remain

serialization fixes added


Do not log error if page index can not be evaluated

Which issue does this PR close?
Closes https://github.com/apache/arrow-datafusion/issues/4236 (cc @HaoYang670 )
Rationale for this change
errors are ignored when creating page indexes but were being logged as error (to the output)
What changes are included in this PR?
reduce logging level to debug from error
Are these changes tested?
No (for logging)
Are there any user-facing changes?
Not really. cc @Ted-Jiang

Thank you, @alamb !
I am OK with the change. But I re-ran the test filter_pushdown::single_file on the master branch and there is no error reported now. Not sure if this has been fixed by https://github.com/apache/arrow-datafusion/pull/4255.

@alamb this change to good to me! But i still have a question about when to use error! in datafusion 🤔? after https://github.com/apache/arrow-datafusion/commit/d7a7fb61afe9ce2824aae737f65aec12d9513f7f merged  i think there should no error log in test.

@alamb this change to good to me! But i still have a question about when to use error! in datafusion 🤔?

I like the "I generally subscribe to the following convention:" answer from this;
https://stackoverflow.com/questions/2031163/when-to-use-the-different-log-levels
So only log errors when there is something a system adminstrator should look at / look into. If they can't do anything about it it probably shouldn't be an error (more like a warn! if it means a DataFusion bug or debug if more like an FYI)

after https://github.com/apache/arrow-datafusion/commit/d7a7fb61afe9ce2824aae737f65aec12d9513f7f merged i think there should no error log in test.

Awesome!

I think this is a reasonable change even though the specific error in datafusion was previously fixed


File extension clarification

@PiJoCoder : Thanks for your contribution! The author, @MikeRayMSFT, has been notified to review your proposed change.

@PiJoCoder - thank you for this update.
#sign-off


rejection.headers('WWW-Authenticate') returning null

When I run the application with the interceptor added it throws the following error:
TypeError: Cannot read property 'match' of null
at responseError (VM666 digest.js:39)
at angular.js:14678
at n.a.$get.n.$eval (angular.js:15922)
at n.a.$get.n.$digest (angular.js:15733)
at n.a.$get.n.$apply (angular.js:16030)
at g (angular.js:10545)
at K (angular.js:10717)
at XMLHttpRequest.z.onload (angular.js:10658)

Using the debugger I found out that the value of rejection.headers('WWW-Authenticate') was null even though the server output header is
> HTTP/1.1 401 Full authentication is required to access this resource
< Access-Control-Allow-Origin: *
< Access-Control-Allow-Methods: POST, GET, PUT, OPTIONS, DELETE
< Access-Control-Max-Age: 1800
< Access-Control-Allow-Headers: Origin, X-Requested-With, Content-Type, Accept
< Cache-Control: no-cache, no-store, max-age=0, must-revalidate
< Pragma: no-cache
< Expires: 0
< X-XSS-Protection: 1; mode=block
< X-Frame-Options: DENY
< X-Content-Type-Options: nosniff
< WWW-Authenticate: Digest realm="XXXXXXX", qop="auth", nonce="MTQ0MzQyOTc2NDQ0MDplMjRiMTBiMThkZWMzZDgxZDI5YWZmZDNkNDFhZDNmZQ=="
< Content-Type: text/html; charset=iso-8859-1
* Server Development/1.0 is not blacklisted
< Server: Development/1.0
< Date: Mon, 28 Sep 2015 08:39:24 GMT
< Content-Length: 195

This output was taken from curl tool.

This was originally written for AngularJS 1.2.0 and hasn't been updated since. Do you have a suggestion how to solve the issue?

Shouldn't WWW-Authenticate be in Access-Control-Allow-Headers ?

@iamambrus Yes. I believe that was indeed what caused this. I'm closing this issue.


bug(mat-tab): tabs are focusable even if they are disabled

Is this a regression?

[X] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
15.0.4
Description
bug(mat-tab): tabs are focusable even if they are disabled
Example - https://v15.material.angular.io/components/tabs/examples#tab-group-ink-bar
Also cdk-* classes are getting applied in disabled mode
v15.2.9 - https://stackblitz.com/edit/fpkg6a-vehkd7?file=package.json

v15.0.4 - https://stackblitz.com/edit/ippjv4?file=package.json

Reproduction
Steps to reproduce:

Go to https://stackblitz.com/edit/fpkg6a-vehkd7?file=src%2Findex.html
Try to navigate on mat-tab using keyboard as shown in attached GIF

Expected Behavior
Disabled item should not get focus
Actual Behavior
Disabled item is getting focus
Environment

Angular: 15.2.8 (exists in 16 a well)
CDK/Material: 15.2.8
Browser(s): Chrome
Operating System (e.g. Windows, macOS, Ubuntu): Windows

This is working as expected. Users need to be able to navigate to disabled tabs to know that they exist.


Playbook Generator only accepts one configmap

In testing it seems that two configmaps tarballs are not accepted, e.g.
oc create configmap -n openstack tripleo-net-config --from-file=net-config.tar.gz
oc create configmap -n openstack tripleo-deploy-config-custom --from-file=tripleo-deploy-config-custom.tar.gz

This is correct. When we implemented the OpenStackPlaybook generator we simplified the configmaps into two types. One ConfigMap is use for custom Heat ENV files. These are ordered and applied in order during playbook generation.
The second configmap is a tarball configmap used to extract generic heat templates into your t-h-t root before playbooks are generated. These can be any type of templates including network configuration (which is really just a special case we implemented before as we didn't support tarballs). As such we just dropped the special case tripleo-net-config since it is no longer needed.
So something like this should work now:
 tar -cvzf net-config.tar.gz *.yaml
 oc create configmap -n openstack tripleo-net-config --from-file=net-config.tar.gz

Note: this tarball config map can be named whatever you want, and can contain multiple tarballs.

As such we just dropped the special case tripleo-net-config since it is no longer needed.

Ok but then your example includes this?
 oc create configmap -n openstack tripleo-net-config --from-file=net-config.tar.gz

So we replaced the special case 'net-config' map with a generic tarball config map. But I left the name of the config map the same so I can understand how that is confusing.
Perhaps I should rename that config map to tripleo-template-tarballs to clarify this explicitly. We'll still use it to inject the custom network templates I think its just the naming that is causing confusion?

Yes and I suppose how the playbook generator knows which configmap to use. Does it need to start tripleo- for example?

You specify it directly on the CR for osplaybook generator :)
apiVersion: osp-director.openstack.org/v1beta1
kind: OpenStackPlaybookGenerator
metadata:
name: default
namespace: openstack
spec:
gitSecret: git-secret
imageURL: quay.io/openstack-k8s-operators/tripleo-deploy:16.2_20210309.1
heatEnvConfigMap: tripleo-deploy-config-custom
tarballConfigMap: tripleo-net-config


build: bring back flto flags

Changes
v2:
handle only -O[s12345\ ]
Rationale
This patch re enables the -flto flags based on optimization level
provided flags.
Signed-off-by: Leandro Dorileo<EMAIL_ADDRESS>

+1

self -1
we're lacking -lubsan and with that lto-wrapper will fail... I'm sending a v3 including a patch to fix the issue.


More support for availability reporting on MQTT components

Description:
Adds the ability to use alive and LWT messages to set the availability of the majority of MQTT devices and abstracts the logic for handling availability topics to a mixin MqttAvailability.
The following components now support availability:

Switch (previously supported)
Binary sensor (previously supported)
Cover (previously supported)
Light
JSON light
Template light
Lock
Fan
HVAC
Sensor
Vacuum
Alarm control panel

Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#<home-assistant.github.io PR number goes here>
Example entry for configuration.yaml (if applicable):
binary_sensor:
  - platform: mqtt
    state_topic: "home-assistant/window/contact"
    availability_topic: "availability_topic"
    payload_available: "good"
    payload_not_available: "nogood"

The last three variables are the important ones, they can be added to the configuration of any of the components mentioned above.
Checklist:
If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass

Please don't squash after PR review has started. It makes it harder to follow changes.

@MartinHjelmare Cool, I'm just getting the documentation ready now.
Also this changes the default values for availability payloads for the MQTT Switch platform, would that be enough to warrant this being a breaking change?

Btw, what's the documentation status for this feature? Do we need to add documentation?

Great!
Yes, let's add the breaking change tag to be clear. Please add a paragraph in the description what has changed that is breaking.


Feature: add feature-flag: bt enables backtrace

Changelog
Feature: add feature-flag: bt enables backtrace
--features bt enables backtrace when generating errors.
By default errors does not contain backtrace info.
Thus openraft can be built on stable rust by default.
To use on stable rust with backtrace, set RUSTC_BOOTSTRAP=1, e.g.:
RUSTUP_TOOLCHAIN=stable RUSTC_BOOTSTRAP=1 make test

fix: #241

commit-id:933ca964

This change is

LGTM, but still can't approve

Thank you:)
It seems reviewable only allows the maintain or admin to approve.
Github allows everybody to approve:
You can find the green button in the Files Changed tab.


End Of Stream

So when I join a server and I will leave a video example, It kicks me with the error End Of Stream but If I join another server It does NOT kick me... The server runs on repl and I just wanna know why it keeps kicking me :( Please get back to me as soon as possible! Thanks!

what server did u try to connect to

[https://res.cdn.office.net/assets/mail/file-icon/png/video_16x16.png]End Of Stream Error.mp4https://1drv.ms/v/s!Akp0pe-BF5PJgVPdSE0fNqT2c1gp
Here we go

From: A-Random-Person-on-Github @.>
Sent: Friday, April 14, 2023 10:20 AM
To: A-Random-Person-on-Github/Eaglercraft-Shaders-Update-HTML-FILES @.>
Cc: Glitched4Life @.>; Author @.>
Subject: Re: [A-Random-Person-on-Github/Eaglercraft-Shaders-Update-HTML-FILES] End Of Stream (Issue #2)
what server did u try to connect to
—
Reply to this email directly, view it on GitHubhttps://github.com/A-Random-Person-on-Github/Eaglercraft-Shaders-Update-HTML-FILES/issues/2#issuecomment-1508986787, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A4KVYKIBMVAIVIHQFL42VDTXBGBMRANCNFSM6AAAAAAW6TZYQQ.
You are receiving this because you authored the thread.Message ID: @.***>

I can host a zoom if you would like so you an help with this issue...

are you using the web demo or the html file

I am uzing the international html file

From: A-Random-Person-on-Github @.>
Sent: Monday, April 17, 2023 8:50 AM
To: A-Random-Person-on-Github/Eaglercraft-Shaders-Update-HTML-FILES @.>
Cc: Glitched4Life @.>; Author @.>
Subject: Re: [A-Random-Person-on-Github/Eaglercraft-Shaders-Update-HTML-FILES] End Of Stream (Issue #2)
i recomend use the html file se
—
Reply to this email directly, view it on GitHubhttps://github.com/A-Random-Person-on-Github/Eaglercraft-Shaders-Update-HTML-FILES/issues/2#issuecomment-1511633073, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A4KVYKL3GMTN3KFR52LQB7LXBVRERANCNFSM6AAAAAAW6TZYQQ.
You are receiving this because you authored the thread.Message ID: @.***>

Usebthe other pne


Generic component an inline-component tags that deffer to underlying named components

This is a first draft.
I think this fits what we talked about last week?

It took me quite a while to finally write the few lines of documentation needed but finally this PR is ready for review

Been waiting for this.  Hopefully gets merged soon.


Enable prometheus metrics scraping

Result:

/assign @DhritiShikhar

Should there be any tests for this @alebedev87 ?

@DhritiShikhar : for the e2e test we'd need a viable way of verifying the target creation in prometheus. Will try to find something.

/lgtm

As discussed with Dhriti, the e2e testing is possible but quite heavy. Prometheus targets are not exposed on the Kubernetes level (CRs or ConfigMaps/Secrets), they have to be retrieved from the prometheus API which most often is queried by exec inf into the prometheus POD.

/assign @quarterpin

/label qe-approved

/assign @xenolinux


[torchlib] Impl aten::native_group_norm with onnx::GroupNormalization

Saw in comment
    # Because onnx.GroupNorm() need size=group for weight and bias
    # But the torch's aten function's input need size=channel, the size mismatched
    # So we have to use onnx.InstanceNorm() to simulate

and previous discussion https://github.com/microsoft/onnxscript/pull/644#discussion_r1170696910.
Reviving the thread to see if something can be done to use ONNX GroupNorm. The current workaround with InstanceNorm produces way more nodes which are not ideal for performance.
E.g.:

cc @justinchuby @xiaowuhu

Setting to low pri for now since it appears the extra computation is for the extra mean/var outputs.

Graph norm is not correctly defined in ONNX. We need to fix it: https://github.com/onnx/onnx/issues/5466

We can do this when we start to use Opset 21.


Can you provide the code of ORCA-vitB16? If yes, thank you very much

Can you provide the code of ORCA-vitB16? If yes, thank you very much

Hello @zhoulixiang38 ,
Thank you for your interest! However, our repository is focused on our own method "PIM" and does not include the code of other methods such as ORCA. I recommend checking repositories specifically related to ORCA method and ViT-B model for the information you need.
Best,
Florent


Clean up deprecated kubernetes methods

What this PR does / why we need it:
With #583 (staring from 0.16.0) some methods from pkg/client/kubernetes were deprecated in favour of controller-runtime client. This PR cleans up the deprecated methods and uses controller-runtime client as replacement.
/kind api-change
/kind cleanup
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

Deprecated methods from `pkg/client/kubernetes` are now removed. Consider to switch your invocations to [kubernetes-sigs/controller-runtime](https://github.com/kubernetes-sigs/controller-runtime) client (`.Client()`) or [kubernetes/client-go](https://github.com/kubernetes/client-go) (`.Kubernetes()`).

Thank you for your review! Applied the suggestions.
Just found an issue with the PR, let me try to resolve it.
/hold

/hold cancel


Add a method on ALAddress to show the long form of the state/province/sublocality

this would just be a wrapper to state_name() with country_code specified, as a convenience

This never made it into main, and the changes are still on https://github.com/SuffolkLITLab/docassemble-AssemblyLine/compare/address-field-enhancements. Reopening.


.

...

I'm going to close this one out until it has a title and description. 👌

I'm going to close this one out until it has a title and description. 👌


Add a flag that allows increased info when debugging

The verboseDebugInfo flag will override any other user settings to increase upload frequency, disable sampling, and set logging to verbose, but only when the application is marked as debuggable in the manifest.
Implements RUMM-1773

[x] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[x] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

@0xnm Any ideas why this might have lowered coverage? I don't see anything new that's not tested but maybe I'm just missing something.


Update README.md

Updated install directions

It is right below your change :-)
Hyprpm:

You can enable or disable it via hyprpm enable hyprscroller and hyprpm disable hyprscroller or update it using hyprpm update hyprscroller.

Adding exec-once = hyprpm reload -n to your hyprland.conf will ensure all your hyprpm managed plugins will be loaded at startup.

If you are using the arch hyrpland package v0.45.0, it is broken at the moment. I am preparing a bug report to send them.  The headers in your system don't contain a valid /usr/include/hyprland/src/version.h, it is empty. It should contain the version of the build (git commit etc.). So when hyprpm tries to checkout the correct version of the headers to build the plugin, it cannot find any.

Aw man


onLeafClick() not getting a node's all properties

I build my tree with series of node and for each node I set the following fields:
                value: argTree.subtreeNodes[i].name,
                    id: argTree.subtreeNodes[i].id,
                    data: argTree.subtreeNodes[i].name,
                    type: argTree.subtreeNodes[i].name,
                    entity: argTree.subtreeNodes[i].entityId,

if I use the  onLeafClick() , I only get the "value" and "id" (see above) , and not other field. I  intentionally assigned "data' to "name" to make sure that name exists.
onNodeSelect(node) {
    var noed  = node;
    var i=0;
    i=i+1;
    alert("the node type is " + node.value); //if I change the node.value to node.type, I get undefined, even though they are assigned to the same variable.
}
render() {
    var tempTree =this.convertSubTreetoTableTree(this.props.locationSubtreeResp);

    return (
        < MuiTreeView tree = {tempTree}   onLeafClick={node => this.onNodeSelect(node)} />
)
};

Please let me know if I am missing anything here.
Thanks,
Farshid

At the moment we only expose 3 properties value, parent and id. I'll see what I can do to have onLeafClick display all passed properties.

Published in v3.3.0.

I upgraded to the latest version , and it works great. Thanks for your quick response. Have a wonderful day!


Spec reporter should display test file name

When defining tests with a help function (shared behaviours) the stack trace does not include the test file name. This makes it difficult to track down the failing test.
(Related: #1466)

FWIW, I think the technique described in that old wiki page is generally frowned upon nowadays -- better to be explicit about what each test expects even if that means repeating stuff in the tests ("don't repeat yourself" is not about code that happens to be the same but rather about information inherent in the code, but tests by their very nature validate the information inherent in the code, thus two tests aren't necessarily redundant just because they say the same thing about distinct objects); for perspective, the example on that page is full of other code smells such as using this to store state across tests (although there are ways to achieve such shared test behavior more cleanly if really needed... but I'd have to think about whether any of them would restore the stacktrace to the test file instead of the shared test behavior file).

There's been very little discussion on the spec reporter's displayed names/paths in the last few years. Closing this out as it doesn't look like many users need this functionality still. Someone please yell at me if I'm wrong!
If you do want this, it'd also be helpful to post a comparison list of other xunit reporters in equivalent tools. We should see what they generally do and consider mirroring that behavior.


"Warm up" endpoints before performance tests

The performance tests can be unpredictable when certain endpoints need to do a pull.
I don't think it's important for us to factor in pulls when doing load tests. Network performance affects it a lot.
I saw that Gatling has a way to "warm up" endpoints before starting the performance test. We should do this for all tests.

Make sure the warm up is actually working. It seems there is a timeout on gatling warm up requests to 1000ms

After a little digging it doesn't seem like there's a warm up for your requests under test. There is a warmUp() function but this is for gatling itself to warm up and isn't for this purpose.
The correct solution here would be to use "Silencing". This lets you make a request that isn't taken into account during reporting. See examples in the docs:
https://gatling.io/docs/current/http/http_request/#silencing
https://gatling.io/docs/current/http/http_protocol/#http-protocol-silencing

Closed by 65cbbd5ec0b91aef6c585c10dcac35f6515c37b6


Go all in on plain text - swap out sanitize-html with a plain text sanitizer

I reached for sanitize-html without fully thinking it through - we're actually not using html at all with user generated content - it's all plain text. So we should sanitize plain text only.
Potential options:

sanitize
html-to-text

Why do we need to sanitize the user generated content? I think that as long as we don't pass anything inside dangerouslySetInnerHTML we're safe, but I might be wrong.

Yeah, it's a fair question. Like you say, if we're not actually rendering the markup, it's not a security concern. But if someone does add tags etc, it'll look bad.


Add Simplenote to Apps Page

Adds Simplenote to the apps page. It is free and open source.

Awesome, thanks!


New version: IanWalton.JellyfinMPVShim version 2.7.0

Pull request has been created with komac v2.2.1 :rocket:
Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:
(Automated response - build 881.)


[Performance]: Slowdown compared to Gradio

Proposal to improve performance
vLLM is amazingly fast
However, when running below prompt, with meta-llama/Meta-Llama-3-8B-Instruct, Gradio takes ~4sec per prompt (one by one) while vLLM takes ~12sec by def. When setting --quantization fp8 times reduced to ~8s
Overall vLLM is much faster since it allows to process in parallel while Gradio doesn't
Tested with AWS L4, Gradio 4.43.0
What am I missing?
prompt = """You are a knowledgeable, efficient, and direct Al assistant. Provide concise answers up to 100 words
without explainations or extra notes, focusing on the key information needed. Answer in question: answer JSON format
**User:**I like the color red. Our website is www.nba.com. My age is 18.
**Assistant:**Great. Write 3 things for me to answer.
**User:**What is our website? What is my age? What kind of drink do I like to drink?
**Assistant**:
"""
Report of performance regression
No response
Misc discussion on performance
No response
Your current environment (if you think it is necessary)
The output of `python collect_env.py`

Before submitting a new issue...

[X] Make sure you already searched for relevant issues, and asked the chatbot living at the bottom right corner of the documentation page, which can answer lots of frequently asked questions.

How does Gradio host models? Could you possible share the scripts/commands you have for running vLLM vs Gradio?


"Delete playlist" pop-up white with dark theme

The text on the screenshot is in french but it's the "delete from library" pop-up that appear when you want to delete a playlist from the dropdown menu. As you can see I use the monodark theme but the button is still in white.

@Yashirow are we ready to close this issue

Yep sure, thanks for the quick fix !


wip: DE-752

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (92b5f94) 98.15% compared to head (bbeda93) 98.15%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #322   +/-   ##
=======================================
  Coverage   98.15%   98.15%
=======================================
  Files          26       26
  Lines        4219     4219
=======================================
  Hits         4141     4141
  Misses         78       78

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


[Help] Helper func. for verifying if (AWS) subnet is private?

Would that make sense to have a helper function for verifying if the subnetwork is private/public?
My use case is as follows: I have a networking module that outputs IDs of subnetworks: private-subnet-id and public-subnet-id. The public subnet has a route to IGW in its routing table, the private doesn't
I want to avoid errors like:
output "private-subnet-id" {
  value = aws_subnet.public.id
  description = "The private subnet id"
}

I want my test to verify if I can trust that private subnet is indeed private.
It could look like this:
func VerifyContainPublicRoute(routes []Route, igwId string) bool {
	found := false

	for _, r := range routes {
		if (r.DestinationCidrBlock == "<IP_ADDRESS>/0" && r.GatewayId == igwId && r.State == "active"){
			found = true
		}
	}

	return found
}

Is this a valid use case for the terratest?

Yup that seems like a good idea, although IIRC, all you need is a route to an Internet Gateway for a subnet to be public.
It would be nice if the interface was much simpler though. I think the type signature want is IsPublic(subnetID string) bool. What you have now requires:

Getting the route table
Getting Internet gateways
Iterating cross product of routes and internet gateway

which seems cumbersome.

This was addressed in https://github.com/gruntwork-io/terratest/releases/tag/v0.19.3


Observable/Promised based subscribeToMore for reacting to updates

I'm taking an initial result set from a query and subscribing to more. When new data comes in, I need to first push it through some animation before adding it to a list of results for the user to see.
Results: [0, 1, 2]
New Result: 3...... animate for a couple of seconds
Results: [0, 1, 2, 3]
Having subscribeToMore return a Promise or be an Observable chain would make this really easy cause I think it would be as easy as delay(1000) or setTimeout(() => resolve(mergedData), 1000)
Right now to delay new results it looks something like:
queryRef: QueryRef<Things> = this.apollo.watchQuery<Things>({ query });

thing$ = new Subject<Thing>();

things$: Observable<any> = mergeStatic(
  this.thing$.pipe(delay(1000)),
  this.queryRef.valueChanges.pipe(
    map(({ data }) => data.things.edges.map(edge => edge.node)),
    mergeAll(),
  ),
).pipe(
  scan((acc, node) => [...acc, node], []),
);

this.queryRef.subscribeToMore({
  document,
  updateQuery: (prev: any, { subscriptionData }) => {
    if (!subscriptionData.data) {
      return prev;
    }

    this.animate = true;
    this.thing$.next(subscriptionData.data.createThing);

    return prev;
  },
});

@intellix maybe Apollo.subscribe(...) + QueryRef.updateQuery(...)` are what you need?

Ahah, I didn't know about the subscribe method :) Documentation only referenced subscribeToMore so missed that one: https://github.com/apollographql/apollo-client/issues/2719#issuecomment-351504883

Pretty much every time I mention .subscribe() people are surprised. I'll cover that in angular docs soon.


state store: client-side encryption and query api are mutually excluded

What content needs to be created or modified?

Add information about state store: client-side encryption and query api are mutually excluded.
The associated pull request from dapr/dapr, dapr/components-contrib, or other Dapr code repos

https://github.com/dapr/dapr/pull/4127

/assign

Closing since PR that addresses this was merged: https://github.com/dapr/dapr/pull/4127


[tiktok] fix extraction

Before submitting a pull request make sure you have:

[x] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8

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

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

What is the purpose of your pull request?

[x] Bug fix
[x] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
Extractor did not work.

Extract more metadata
Can only extract latest 9 videos uploaded by user
Fixes #20809 and #22850

Read coding conventions.

Is this in the latest version?

Hi, when will this expected to be merged?
Thanks!

@dstftw

will this be merged anytime soon?

@dstftw When this can be merged ?

With the patch applied, a RegexNotFoundError for json_string results for me.

Debug information
[debug] System config: []
[debug] User config: ['--all-subs', '--fragment-retries', '10', '--no-mark-watched', '--abort-on-unavailable-fragment', '--console-title', '--write-all-thumbnails']
[debug] Custom config: []
[debug] Command-line args: ['https://www.tiktok.com/@ravirosss123/video/6773432341571456261', '-v']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2019.10.22
[debug] Git HEAD: 4b6d03ed8
[debug] Python version 3.6.9 (CPython) - Linux-5.0.0-37-generic-x86_64-with-Ubuntu-18.04-bionic
[debug] exe versions: ffmpeg 4.0.2-1, ffprobe 4.0.2-1, phantomjs 2.1.1, rtmpdump 2.4
[debug] Proxy map: {}
[TikTok]<PHONE_NUMBER>571456261: Downloading video webpage
ERROR: Unable to extract json_string; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/home/chris/github/youtube-dl/youtube_dl/YoutubeDL.py", line 796, in extract_info
    ie_result = ie.extract(url)
  File "/home/chris/github/youtube-dl/youtube_dl/extractor/common.py", line 530, in extract
    ie_result = self._real_extract(url)
  File "/home/chris/github/youtube-dl/youtube_dl/extractor/tiktok.py", line 243, in _real_extract
    webpage, 'json_string', group='json_string')
  File "/home/chris/github/youtube-dl/youtube_dl/extractor/common.py", line 1005, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
youtube_dl.utils.RegexNotFoundError: Unable to extract json_string

With the patch applied, a RegexNotFoundError for json_string results for me.
Debug information

@karottenbaum Fixed the issue in the latest commit.

Is there a way to sideload the fixed tiktok.py file into youtube-dl until it is pushed out fully?

Is there a way to sideload the fixed tiktok.py file into youtube-dl until it is pushed out fully?

@thinkpad4 you can clone this repository and run from it.
git clone https://github.com/skyme5/youtube-dl.git
cd youtube-dl/youtube_dl
python __main__.py url

@dstftw all checks have passed, can you merge?


Install failing on Kubernetes 1.17.3 'secret "cert-manager-webhook-tls" not found

Describe the bug:
After a clean install using rke up command I went through cert-manager install process.
Cluster has 3 etcd/controlplane + 1 worker node.
Note that controlplane has NoSchedule taint now, so cert-manager will deploy on a worker node
I end up with stuck in ContainerCreating:
cert-manager-57cdd66b-kvhrt                      1/1     Running             0          58m
cert-manager-cainjector-79f4496665-59hjj   0/1     Error               16         58m
cert-manager-webhook-6d57dbf4f-cgl2n     0/1     ContainerCreating   0          58m

Expected behaviour:
All pods should be running
Steps to reproduce the bug:
rke up --config rke-config.yml
kubectl create namespace cert-manager
helm repo add jetstack https://charts.jetstack.io
helm repo update
helm install \
  cert-manager jetstack/cert-manager \
  --namespace cert-manager \
  --version v0.14.0

Anything else we need to know?:
services:
cert-manager           ClusterIP   <IP_ADDRESS>    <none>        9402/TCP   58m
cert-manager-webhook   ClusterIP   <IP_ADDRESS>   <none>        443/TCP    58m

pods:
cert-manager-57cdd66b-kvhrt                      1/1     Running             0          58m
cert-manager-cainjector-79f4496665-59hjj   0/1     Error               16         58m
cert-manager-webhook-6d57dbf4f-cgl2n     0/1     ContainerCreating   0          58m

cainjector logs:
I0320 13:11:53.775397       1 start.go:82] starting ca-injector v0.14.0 (revision 6da95758a4751b20cf85b29a3252e993449660eb)
E0320 13:11:54.823577       1 manager.go:241] cert-manager/controller-runtime/manager "msg"="Failed to get API Group-Resources" "error"="Get \"https://<IP_ADDRESS>:443/api?timeout=32s\": dial tcp <IP_ADDRESS>:443: connect: no route to host"
F0320 13:11:54.823618       1 start.go:118] error creating manager: Get "https://<IP_ADDRESS>:443/api?timeout=32s": dial tcp <IP_ADDRESS>:443: connect: no route to host

webhook events:
  Type     Reason       Age                   From                              Message
  ----     ------       ----                  ----                              -------
  Normal   Scheduled    <unknown>             default-scheduler                 Successfully assigned cert-manager/cert-manager-webhook-6d57dbf4f-cgl2n to node1.example.com
  Warning  FailedMount  37m (x5 over 57m)     kubelet, node1.example.com  Unable to attach or mount volumes: unmounted volumes=[certs], unattached volumes=[cert-manager-webhook-token-74gn5 certs]: timed out waiting for the condition
  Warning  FailedMount  14m (x10 over 55m)    kubelet, node1.example.com  Unable to attach or mount volumes: unmounted volumes=[certs], unattached volumes=[certs cert-manager-webhook-token-74gn5]: timed out waiting for the condition
  Warning  FailedMount  4m51s (x35 over 59m)  kubelet, node1.example.com  MountVolume.SetUp failed for volume "certs" : secret "cert-manager-webhook-tls" not found

It seems like cainjector should inject the secret but it's waiting for the webhook which cannot start without the secret.
Environment details::

Kubernetes version : v1.17.3
Cloud-provider/provisioner (e.g. GKE, kops AWS, etc): on-premise Fedora 29 server nodes, ESX host
cert-manager version : v0.14.0
Install method: both

/kind bug

Update, after a stop firewalld then iptables -F on my worker node it's working again.
is there a specific step for firewall rules?

The error F0320 13:11:54.823618       1 start.go:118] error creating manager: Get "https://<IP_ADDRESS>:443/api?timeout=32s": dial tcp <IP_ADDRESS>:443: connect: no route to host sounds like you have some configuration issues with your cluster. This will prevent many different parts of Kubernetes not working, and is not specific to cert-manager.

Update, after a stop firewalld then iptables -F on my worker node it's working again.

I'm not too sure - I'd advise following the official kubeadm install guide to provision Kubernetes, and then ensuring it is running properly by running Sonobuoy.
Hope that helps - I'm going to close this issue as it's not something that can be addressed in cert-manager! 😄

I am using aks without any firewall rules, my certmanager version is v0.15.0

I'm having the same problem with kubernetes version 1.19.4. did anyone managed to resolve this problem?

I had the same issue on 1.21 with EKS, and found the primary cause is here  "error"="Get \"https://<IP_ADDRESS>:443/api?timeout=32s\": dial tcp <IP_ADDRESS>:443: connect: no route to host". The CA Injector pod can't start up properly because it can't reach the kubernetes api server. Double check that you can connect to that route from the nodes. In my case, kube-proxy wasn't configured properly, and after addressing that, everything began playing nicely.


TCK Challenge: com.ibm.jbatch.tck.tests.jslxml.CDITests#testCDILookup(...)

The relevant specification version and section number(s)

https://jakarta.ee/specifications/batch/2.1/jakarta-batch-spec-2.1#batchproperty-definition
https://jakarta.ee/specifications/batch/2.1/jakarta-batch-spec-2.1#method-parameter-and-constructor-parameter-injection-with-explicit-name
https://jakarta.ee/specifications/batch/2.1/jakarta-batch-spec-2.1#consequences-and-suggested-patterns

The coordinates of the challenged test(s)

com.ibm.jbatch.tck.tests.jslxml.CDITests#testCDILookup(...)

The exact TCK version
2.1.1
The implementation being tested, including name and company
Apache Tomee 10.0.0-M1-SNAPSHOT (specifcally: Geronimo BatchEE 2.0.0-SNAPSHOT), The Apache Software Foundation
A full description of why the test is invalid and what the correct behavior is believed to be
Argument 1: Undefined CDI behaviour
As dicussed in https://github.com/eclipse-ee4j/batch-tck/pull/69, the test com.ibm.jbatch.tck.tests.jslxml.CDITests#testCDILookup(...) above relies on undefined (vendor-specific) CDI behaviour.
The test stated above contains the following code:
 cdi.select(String.class, new BatchPropertyLiteral("prop1")).get();
depending on the CDI implementation (OWB or Weld), the following will happen:

OWB will invoke the producer  with a null InjectionPoint
Weld will invoke the producer with a "fake" InjectionPoint with InjectionPoint#getMember == null.

The JavaDoc of InjectionPoint or more specifically of
InjectionPoint#getMember() does not tell us, if the return value can be
null.
However, we find a code example in the interface docs:
@Produces
Logger createLogger(InjectionPoint injectionPoint) {
    return
Logger.getLogger(injectionPoint.getMember().getDeclaringClass().getName
());
}

If InjectionPoint#getMember() can be null, this code example would
throw a NullPointerException. So from our point of view, Weld violates
the current JavaDoc of InjectionPoint by injecting a "fake"
InjectionPoint without a member.
InjectionPoint#getBean can return null and it is stated, any other
method can't - intentionally cause an Instance only defines an
InjectionPoint when there is one.
This is now also  discussed in https://github.com/jakartaee/cdi/issues/779
Argument 2: Definition of @BatchProperty
The spec says the following about @BatchProperty quoting from

The @BatchProperty annotation identifies an injection as a batch property. A batch property has a name (name) and, in case of a field injection, also has a default value. @BatchProperty is used to assign batch artifact property values from Job XML to the batch artifact itself. Note that @BatchProperty is used with the standard @Inject annotation (jakarta.inject.Inject) ...

from https://jakarta.ee/specifications/batch/2.1/jakarta-batch-spec-2.1#batchproperty-definition

A batch runtime must support injection of the CDI Bean representing the batch property via method parameter and constructor parameter injection, in addition to supporting field injection.

from https://jakarta.ee/specifications/batch/2.1/jakarta-batch-spec-2.1#method-parameter-and-constructor-parameter-injection-with-explicit-name
Given those two quotes, the valid options for injecting a @BatchProperty are:

param injection
constructor injection
field injection

but no CDI.current().select(...) / dynamic CDI lookups as tested in com.ibm.jbatch.tck.tests.jslxml.CDITests#testCDILookup(...) although dynamic CDI lookups are mentioned in https://jakarta.ee/specifications/batch/2.1/jakarta-batch-spec-2.1#consequences-and-suggested-patterns - this contradiction could maybe adressed with https://github.com/eclipse-ee4j/batch-api/issues/209
Resolution / Solution
Maybe allow an exclusion of the test for now and than follow up with https://github.com/eclipse-ee4j/batch-tck/issues/70
Any supporting material; debug logs, test output, test logs, run scripts, etc.

https://github.com/eclipse-ee4j/batch-tck/pull/69
https://github.com/jakartaee/cdi/issues/779

Is there anything we need to do here? @scottkurz

Is there anything we need to do here? @scottkurz

No, I was actually just working on this.  I had to grab some time to be able to do a few related tasks, building up to getting a release out the door soon.
I will ask you to review the specification text change as well when I complete that.

Posted PR for related spec change at https://github.com/jakartaee/batch/pull/211


[Test] Add test for home_controller

Add tests for the home_controller (app/controllers/home_controller.rb)

Can I work on this? :)

@rgenchev Are you working on this?

Yup :)

I created https://github.com/Terrastories/terrastories/pull/332 with an initial HomeControllerSpec file and describe blocks for the corresponding controller actions. Since there isn't any logic in the controller, I'm not sure if we want to merge this with empty test file.

Hey @rgenchev sorry for the confusion! This issue has already been completed here. https://github.com/Terrastories/terrastories/pull/294
Thank you still for the contribution!


feat(lidar_centerpoint): yaw norm filter (cherrypicked)

Description
Added a filter to the centerpoint estimations to ignore detections that have a yaw angle norm below a certain threshold.
Centerpoint rather than estimate the rotation directly, estimates the cosine and sine of the angle.
As such the norm of the cos/sin pair should be unitary, but since the inference does not enforce this property, it may not hold.
Empirically, in those cases where the property does not hold, there is a high chance that the detection is wrong, so we filter them out.

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@yukke42
Sorry, I did not know the signing requirement for the commits, and since I had done some merges, it was difficult to perform the necessary rebases. Instead, I just cherrypicked my commits (and signed them), and resubmitted the PR.
Old PR: https://github.com/autowarefoundation/autoware.universe/pull/1728


Draft: feat: add more trace span and integrate with jaeger

ref: https://github.com/cectc/dbpack/issues/
Ⅰ. Describe what this PR did

integrate with jaeger
add more trace span

Ⅱ. Does this pull request fix one issue?
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
jaeger trace result.

Ⅴ. Special notes for reviews

Codecov Report

Merging #177 (4f4a9ee) into dev (03307dd) will decrease coverage by 0.11%.
The diff coverage is 10.76%.

:exclamation: Current head 4f4a9ee differs from pull request most recent head 6e6cbac. Consider uploading reports for the commit 6e6cbac to get more accurate results

@@            Coverage Diff             @@
##              dev     #177      +/-   ##
==========================================
- Coverage   46.40%   46.29%   -0.12%
==========================================
  Files          75       75
  Lines        8565     8591      +26
==========================================
+ Hits         3975     3977       +2
- Misses       4220     4244      +24
  Partials      370      370

Impacted Files
Coverage Δ

pkg/filter/dt/filter_mysql_prepare.go
0.00% <0.00%> (ø)

pkg/filter/dt/filter_mysql_query.go
0.00% <0.00%> (ø)

pkg/executor/read_write_splitting.go
46.34% <63.63%> (+0.52%)
:arrow_up:

Continue to review full report at Codecov.

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

To trace the entire lifecycle of a transaction, should start with the http request proxy:

Get the TraceParent from the request header, and if it exists, pass it to the downstream service. Generate a new TraceParent if it does not exist and pass it downstream.
Users can put TraceParent into sql hints, dbpack gets TraceParent from sql hints to construct SpanContext, if it does not exist, generate a new SpanContext.

LGTM


Convert some code to Kotlin

Converted Java to Kotlin for low risk / easy classes:

dto, exception, statistic, configuration packages
Application class

Took the opportunity to rename Unaccessible*Exception to Inaccessible*Exception for better English
Adapted some Java stream logic to Kotlin idioms

Avoided apireview and util packages since they're modified in an open pull request and would have conflicted.
I realise that the PR touches a lot of files but it's mostly automated conversion with zero functional change so I'm hoping for relatively quick review and merging.
Relates to #712

Codecov Report

Merging #795 into master will decrease coverage by 0.51%.
The diff coverage is 81.03%.

@@            Coverage Diff             @@
##             master   #795      +/-   ##
==========================================
- Coverage     85.52%    85%   -0.52%
+ Complexity     1022   1016       -6
==========================================
  Files           170    170
  Lines          2721   2675      -46
  Branches        395    400       +5
==========================================
- Hits           2327   2274      -53
- Misses          196    197       +1
- Partials        198    204       +6

Impacted Files
Coverage Δ
Complexity Δ

.../zalando/zally/configuration/OAuthConfiguration.kt
0% <0%> (ø)
0 <0> (?)

...ndo/zally/configuration/NoSecurityConfiguration.kt
100% <100%> (ø)
2 <2> (?)

...a/de/zalando/zally/statistic/ApiReviewStatistic.kt
100% <100%> (ø)
8 <8> (?)

...va/de/zalando/zally/exception/ExceptionHandling.kt
100% <100%> (ø)
5 <5> (?)

...ando/zally/statistic/ReviewStatisticsController.kt
100% <100%> (ø)
10 <10> (?)

...ain/java/de/zalando/zally/dto/ViolationsCounter.kt
100% <100%> (ø)
3 <3> (?)

...ally/configuration/RulesDefinitionConfiguration.kt
100% <100%> (ø)
2 <2> (?)

...zally/configuration/RulesValidatorConfiguration.kt
100% <100%> (ø)
2 <2> (?)

...y/exception/TimeParameterIsInTheFutureException.kt
100% <100%> (ø)
1 <1> (?)

...o/zally/exception/MissingApiDefinitionException.kt
100% <100%> (ø)
1 <1> (?)

... and 34 more

Continue to review full report at Codecov.

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


Custom Tags and Layers for Tilemap

Hello, first I want to tell you that I am using the Pro-Version of your asset and I really find it very interesting.
I have only had 1 problem and it is the following; There is no way to use my own tags for tilemaps.
When the map is generated, the tags that I have put to the customs rooms tile elements are not generated.

Hey!
By default, after a level is generated, new tilemaps (so-called shared tilemaps) are created and all the rooms are copied to these new tilemaps. That's the reason why if you put custom tags onto your room template tilemaps, they are not present on the generated levels.
There are basically two ways to fix that:

You can create a custom tilemaps layers handler (docs) which lets you customize everything regarding the tilemaps in your room templates and generated levels. You can change which tilemap layers you want, whether they should have colliders, etc.
You can create a custom post-processing task (docs) and there you can call level.GetSharedTilemaps() and correct manually all the tags

The second option is more like a hotfix but may be a little bit easier to do.
For the future versions of the asset, I plan to implement some auto-inference rules that would try to guess the correct settings of tilemaps and apply that to the result without needing to implement anything yourself. The only problem is that there's nothing to enforce the user to use the same setting across all the room templates. For example, if you forget to set the custom tag on one room template, I cannot know whether it's a mistake and I should ignore it or if it's by design and I cannot infer anything because it's not clear how to do that.
Let me know if you have any other questions or problems!

Closing due to inactivity. Feel free to reopen if needed.


Refactoring get JDK bash scripts

On the back of the comment https://github.com/AdoptOpenJDK/jdk9-jigsaw/pull/68#issuecomment-308574524 - some refactoring to simply script and improve readabality

Would you be okay to address this enhancement lasombra? Let me know so I can assign it to you - thanks, its simple moving blocks of code to a common script

@neomatrix369 Sure thing. I'll try to get it done during the week.

Thanks @lasombra, looking forward to seeing the solution soon, good luck.

Fixed by #70 - will revisit to see any further refactorings are needed - keeping in mind simplicity and readability


Forever loading indicator in taskbar when playback ended

Checklist:

[x] I have checked that there are no issues with similar or the same content
YOU SHOULD CHECK CLOSED ISSUES ASWELL

[x] I have checked the FAQ (https://github.com/MarshallOfSound/Google-Play-Music-Desktop-Player-UNOFFICIAL-/wiki/FAQ) and the answer I am looking for is not there

[x] I have double checked and can reproduce the issue

OS:
Windows 10
GPMDP Version:
I don't know where is this displayed. I didn't find an "about" button in the desktop settings dialog.
From shortcut properties > open file location > I found it pointing to "Update.exe" and next to it there were 2 folders : "app-4.5.0" and "app-4.6.1"
I guess it's 4.6.1
Issue Descriptions:
After finishing playing an album the taskbar loading indicator keeps loading in marquee mode.
Steps to Reproduce:
Play something and reach to the end of that list. If there is nothing left to play the loading indicator will keep loading forever.

i can confirm this issue, also, sometimes the play button just can't be accessed and the music don't play


Bugfix/fix map

Use another tool for the marker in maps
Fix map not loading (Map not load because leaflet)
Add a your location button to get user current location
Remove unnecessary label in step2 of queue creating
Implement location setting in setting page

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 6.34921% with 59 lines in your changes missing coverage. Please review.

Files with missing lines
Patch %
Lines

manager/views.py
6.45%
58 Missing :warning:

participant/views.py
0.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Flag
Coverage Δ

unittests
35.44% <6.34%> (-0.20%)
:arrow_down:

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

Files with missing lines
Coverage Δ

manager/urls.py
100.00% <ø> (ø)

participant/urls.py
100.00% <ø> (ø)

participant/views.py
34.92% <0.00%> (ø)

manager/views.py
20.55% <6.45%> (-0.23%)
:arrow_down:

🚨 Try these New Features:

Flaky Tests Detection - Detect and resolve failed and flaky tests

Good


Add Link to sample code

Please add link to sample code

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

ID: 6486232e-d58e-fe4e-1d88-2b3d3d281dc3
Version Independent ID: d5e93d37-d132-27e0-ff95-72cfde1653d6
Content: Tutorial: Sign in users in a Node.js & Express web app - Microsoft Entra
Content Source: articles/active-directory/develop/tutorial-v2-nodejs-webapp-msal.md
Service: active-directory
Sub-service: develop
GitHub Login: @mmacy
Microsoft Alias: marsma

@diberry is this the sample you would like referenced? https://github.com/Azure-Samples/ms-identity-node

Can you ask @derisen ? I think the same code is actually listed in a couple of repos.

@diberry @MarileeTurscak-MSFT that's correct. Will make a PR to update the doc.

Thanks! I made one here but feel free to close it out and use yours if you already created one. https://github.com/MicrosoftDocs/azure-docs-pr/pull/207559


plots with specific range, show both ends

run this cell:
new Plot(xBound: [7, 9], yBound: [6, 9]) << new Line(x: (1..8), y: (1..8)) << new Line(x: (8..10), y: [8, 7, 6])

in the result of the plot, "9" should be shown on both x and y axes.

I have a  fix ready for specified case, but not in general.
Ticks and labels are based on grid lines. So in a specified case I've fixed it by including right|top grid line.
But let say I change:
p = new Plot(xBound: [7, 9.4], yBound: [6, 9.4])

What then?

doesn't need to be shown in that circumstance.  Please make a PR with what you have and I'll try it out.

When i were looking if I've introduced any BC breaks i found out that:

there are no ticks on right YAxis
top label is missing

I have to investigate this

No ticks is OK since the numbers do not correspond to the lines, right?
top label should appear though.

fixed


ExitFailure at src/HIE/Bios/Cradle.hs

I've tried following blog-post https://chrispenner.ca/posts/hie-core to setup HIE in Sublime text, but setup result in hie-bios raising error without any extra info:
 ◌ :
        1:1   	compiler    	error     	(ExitFailure 1,"","")
             	            	          	CallStack (from HasCallStack):
             	            	          	  error, called at src/HIE/Bios/Cradle.hs:217:18 in hie-bios-0.1.1-7daBaOW11mp8owawn8j60h:HIE.Bios.Cradle

LSP config in sublime:
		"hie":
		{
			"command":
			[
				"stack",
				"exec",
				"hie-core",
				"--",
				"--lsp",
				"--cwd",
				"."
			],
			"enabled": false,
			"languageId": "haskell",
			"scopes":
			[
				"source.haskell"
			],
			"syntaxes":
			[
				"Packages/Haskell/Haskell.sublime-syntax"
			]
		},

Please advise how to get any extra info. I've added simple hie.yaml to root of the project, but this does not help: cradle: {stack}

Is stack on your path?
What happens if you run hie-core in the root of your project?
Perhaps you meant ghcide rather than hie-core in your sublime config?

Yes, stack ghc/ghci works fine, directly running hie-core:
$ stack exec hie-core
Starting hie-core (GHC v8.6)
/Users/kote/hsbackend
[1/6] Finding hie-bios cradle
Cradle {cradleRootDir = "/Users/kote/hsbackend", cradleOptsProg = CradleAction: stack}

[2/6] Converting Cradle to GHC session
#!/usr/bin/env bash
if [ "$1" == "--interactive" ]; then
  pwd
  echo "$@"
else
  ghc "$@"
fi

hie-core: (ExitFailure 1,"","")
CallStack (from HasCallStack):
  error, called at src/HIE/Bios/Cradle.hs:217:18 in hie-bios-0.1.1-7daBaOW11mp8owawn8j60h:HIE.Bios.Cradle

Just an idea - maybe issue happens because I use docker-based build in stack:
$ cat stack.yaml
resolver: lts-14.6

allow-newer: true
docker:
    enable: true

packages: [.]

Maybe the issue is that you don't have ghc on your path?
We need to fix this situation but for now, you need ghc on your path.

I have older GCH in $PATH:
$ ghc --version
The Glorious Glasgow Haskell Compilation System, version 8.4.3
$ stack ghc --  --version
The Glorious Glasgow Haskell Compilation System, version 8.6.5

Should I have same GHC that is used to build project ?
AFAIU with docker+stack, all dependencies will be inside of docker, so maybe some dependencies are also missing locally. Also even produced binaries can be incompatible, so that I cannot run results of stack build on local OS:
$ file .stack-work/dist/x86_64-linux-dkda49f7ca9b244180d3cfb1987cbc9743/Cabal-<IP_ADDRESS>/build/periodic/periodic
.stack-work/dist/x86_64-linux-dkda49f7ca9b244180d3cfb1987cbc9743/Cabal-<IP_ADDRESS>/build/periodic/periodic: ELF 64-bit LSB executable, x86-64, version 1 (SYSV), dynamically linked, interpreter /lib64/l, for GNU/Linux 3.2.0, BuildID[sha1]=43f95c5c3ebd6ad7f5af39f7c4687cc4f005f549, with debug_info, not stripped
$ .stack-work/dist/x86_64-linux-dkda49f7ca9b244180d3cfb1987cbc9743/Cabal-<IP_ADDRESS>/build/periodic/periodic
-bash: .stack-work/dist/x86_64-linux-dkda49f7ca9b244180d3cfb1987cbc9743/Cabal-<IP_ADDRESS>/build/periodic/periodic: cannot execute binary file
$ stack exec periodic
$ echo $?
0

I'm not sure what your exact issue is but I don't expect things to work for you right now anyway.

I have the same problem.
$ stack exec -- ghcide
Starting ghcide (GHC v8.6)
~/p/haskell/verified-leftist-heap
[1/6] Finding hie-bios cradle
Cradle {cradleRootDir = "~/p/haskell/verified-leftist-heap", cradleOptsProg = CradleAction: stack}

[2/6] Converting Cradle to GHC session
ghcide: (ExitFailure 1,"","")
CallStack (from HasCallStack):
  error, called at src/HIE/Bios/Cradle.hs:241:18 in hie-bios-0.1.1-DmIjTbjtu5iGixdjhE06h7:HIE.Bios.Cradle

$ stack exec -- ghc --version
The Glorious Glasgow Haskell Compilation System, version 8.6.5
$ ghc --version
The Glorious Glasgow Haskell Compilation System, version 8.6.5

@madgen So to be clear, it's not a normal *.cabal package but just a single file?
I'm not sure that is supported (yet?).

@mpickering Yup, just a single file. What is the minimum modification you think I'd need to make it work (given that I'm running it with stack)?

Briefly reading the stack documentation I don't see any way to query stack to tell us what options it would use to load the script. If there is such a way then we could maybe add support for stack scripts as well.

@mpickering Sorry, that's not what I meant (though clearly that's what is needed to solve it for good). I meant should I create a stack.yaml or package.yaml or a *.cabal file to get around the problem for now for this specific project?

If you create a normal stack project or normal cabal project then it should work fine.

Alright, thanks Matthew!

I'm getting the same error from hie-bios, while running stack manually gets me:
$ stack build
Cabal file info not found for katip-<IP_ADDRESS>@sha256:a130d70f7165c0a35396bed70f0f21f34fb07724f8b48b5370e30bda26d3e25f, updating
Selected mirror https://s3.amazonaws.com/hackage.fpcomplete.com/
Downloading timestamp
No package index update available and cache up to date
Package index cache populated
Could not find katip-<IP_ADDRESS>@sha256:a130d70f7165c0a35396bed70f0f21f34fb07724f8b48b5370e30bda26d3e25f on Hackage
The specified revision was not found.
Possible candidates: katip-<IP_ADDRESS>@sha256:4b30d0643e18d01a3fd264d3d75921b49b2f464336a52fa46fa049107ebbfe04,4272, katip-<IP_ADDRESS>@sha256:171304c72f39e1856709e1b575c7585d56a1774a03c15ef48c89d2b26350f4d3,4443.

https://github.com/mpickering/hie-bios/pull/75

@mpickering another hack I can think of is running stack twice, once for parsing with --silent and if that fails, again to get the error. Hoping it's idempotent.

I think a more robust way would be to bake a filepath into the wrapper when it is created so the wrapper can write the arguments to a file which is then read by hie-bios. Then the commands can run at full verbosity. What do you think?

Sounds much better.


KOGITO-7577 - SWF Editor - Parameter named 'uuid' should be not null

Hi @romartin, @ricardozanini,
The issue was that if Workflow missing name, diagram wasn't shown since Stunner uses Workflow name as a UUID for the root node.
The fix consist of two parts:

Workflow ID is used as UUID instead of Workflow Name
If Workflow ID is missing, than generate some random value to use it as UUID.

Please, tell me if it is OK to use Workflow ID as UUID and it will not break any backward compatibility (like automated tests or something else).
If you think it's better to keep Workflow Name as UUID of the root diagram node, I will revert this change and keep only second point (If Workflow Name is missing, generate some randome value to use it as UUID).
VSIX for local tests
Thank you!

@hasys using the workflow ID is ideal since it's a required property defined by the specification.
Although one can use the key instead, ID is not required in this case. I'd recommend using the same logic here. Name is an optional attribute just to a description for human eyes.

Thanks for the guidance @ricardozanini ! Good catch.
I am agree @hasys , let's use ID or key, depending on the one being present, it looks this way we'll properly follow the spec. On ther other hand, I think the key field is not yet in our client models, so maybe you have to add this string in the model classes, updated marshallers, and then do the logic that obtains one or another. Does it makes sense?
Apart from this ID/key change, everything looks good, good job! thanks @hasys

Hello @ricardozanini, @romartin,
great catch! I changed the behavior to be closer to the spec:

Use workflow.id if it's not null or use workflow.key as UUID to generate diagram
Generate random UUID if both workflow.id and workflow.key are null to be able to show the diagram anyway.

I am going to report an issue to provide some error notification if both workflow.id and workflow.key are empty.
Let me know, if it is as you expected. Thank you!

Hi @domhanak, initial issue was reproducible using unit tests, I think it should be enough. Thank you!


Attribute lookups should be case sensitive to reflect case sensitivity of XML

https://github.com/jeffkl/RoslynCodeTaskFactory/blob/afbc509790b2cc8d76cae6bf697262512bdb8b2d/src/RoslynCodeTaskFactory/CodeTaskFactory.cs#L358-L360
I see that there are case insensitive comparisons for attribute names. It would be better to require exact case matches and throw an error for incorrect case.
The equivalent code in CodeTaskFactory.cs does a case sensitive match:
https://github.com/Microsoft/msbuild/blob/216f7cd5daf8d848d1a3773cf1290ebf32be8806/src/Tasks/CodeTaskFactory.cs#L237

I'm not sure if I did that deliberately.  I wonder if I changed it now it would break anyone?  Do you feel pretty strongly about changing it?


Re-add support for XML request format

A recent change caused our support for /products.xml to break, so we
need to restore that capability.

ch5804


Turn into starter kit

I'm very excited about the hotloading and want to use your project as a starter, but your setup is confusing, given that I know nothing about webpack (some of the key config info is at the root level, and some at the /example level).
I'd rather you set up the code with a /src (rather than /example) directory and have a production script available for final deployment (that perhaps copies the compiled assets into an /dist) in the root package.json file.
I have had a go at https://github.com/simonh1000/elm-hot-loader.
What do you think?

Hi!
I don't think it's a good idea to rename /example folder here to /src, because this repository is for the webpack-hot-loader itself. If there is a '/src' folder here, I'll assume it is the source code of loader, not the source code of example(which depends on loader).
But I agree with you that the example here is confusing, so I made another repository here:
https://github.com/fluxxu/elm-hot-loader-starter
BTW, you can remove hmr.js and index.js from your fork because these two files already included in node_modules.
Thanks :)

OK, Cool, thanks


some help ? (not run)

i try to run this project and i can't...
I have been trying for 2 days but nothing is happening..
I followed all the steps from the video
I also did a new windows installation on a new ssd
but again i could not run...
can somebody help me?
I read that the code has bugs and others but to me it does not run at all like the BitShift video....

Hello,

Can you specify the error you're encountering in more detail?


Fix Windows selection based python/sage selection run (Issues #955 , Issue #729)

Windows 10
Atom 1.34.0
Script 3.18.1
There are issues when the whole script is thrown in as arg, moved selected python code to tempfile.
Fixes #955
Fixes #729

Thanks for the contribution! I'm adding you as a maintainer now. Would you like to ship the package?


recast iosrc into cloud-object model

The iosrc currently implements a subset of file system semantics then maps those onto s3.   This is upside down.  We should have a storage abstraction that implements cloud object store semantics then we should map those semantics onto a file system for local operation.    This should lead to cleaner code and fewer bugs since the cloud peculiarities are hidden behind a layer.  Also, it will then make immutable object caching trivial as a wrapper on the new storage interface.  While we're at it, we should get rid of the single mux and just have entities that want to talk to storage keep an interface pointer to a storage implementation.

Note-to-self: When verifying the changes linked to this issue, revisit the notes in https://github.com/brimdata/zed/issues/2614#issuecomment-825996315 regarding an abusive concurrency test we should re-run.

Also, I realized this is the right place to put http.  With this change we can unify proc.DataAdaptor.Open()/Get() to use the Get() method on the ported storage engine (iosrc to be renamed storage).  Then we can unify dag.HTTP and dag.File as @nwt suggeted on the hazmat PR.


The result of HRNet+OCR

I can't find the val result of 84.9%mIoU in the OCR paper. Is this the result reproduced by yours? If so ,can you describe your implementation details?

This result is reproduced by the following paper:
Hierarchical Multi-Scale Attention for Semantic Segmentation
And you can download their pretrained weights from here:
https://github.com/NVIDIA/semantic-segmentation


Update multi_a to not include n wrapper

This is an old version of rust-minsicript (July 2021). Did not make any attempts to fix the fuzz CI and hacky integration testing infra here. I think we should go ahead with this for 7.0.1


Incomplete manifest write makes database unreadable

What steps will reproduce the problem?
1.  Write a record to the MANIFEST file that spans filesystem block boundaries
2.  Experience a fatal IO error that leaves the new record partially written
3.  Try to open the database

What is the expected output? What do you see instead?

Ideally it would recover from the intact portion of the manifest (provided the
files listed in the manifest exist), but instead the database cannot be opened;
it reports "Corruption: truncated record at end of file".

What version of the product are you using? On what operating system?

LevelDB 1.9 on iOS 6.

Please provide any additional information below.

In the particular case I've been studying, the manifest file is truncated at
4096 bytes; the record started just before the 4096-byte mark is incomplete.
The set of files on disk appear consistent with the last intact manifest
record.  The exact nature of the IO failure is unclear but may be flaky
hardware.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 29 Mar 2013 at 4:08

Won't the repair function have the same problem of resurrecting old data, since
it ignores the existing manifest and re-adds all sstables and logs that exist?

It doesn't look like sstables are ever deleted except as part of a compaction,
which also adds new sstables.  If the missing manifest record added a file,
data would not disappear unless the log file containing that data were also
deleted.  As long as the log and all sstables listed in the last intact
manifest version still exist, it should be possible to do a more precise
version of repair that would restore to some consistent earlier view of the
database, instead of the riskier full repair.

Original comment by<EMAIL_ADDRESS>on 1 Apr 2013 at 3:54

It would be very dangerous to ignore the corrupted block and keep going.  That
block may have marked an sstable as deleted.  Ignoring that block would cause
stale data to get resurrected.  Somebody might be very unhappy about that.  And
there can also be a problem in the other direction: the missing MANIFEST file
block may contain references to a new sstable.  That sstable will now get
garbage collected if we allow the leveldb open to proceed.  That will cause
data to disappear.

We decided that instead of continuing normally in such a case, we make the
app/user/operator make an explicit choice.  E.g., running the repair function
on the database or restoring from a backup.

Original comment by<EMAIL_ADDRESS>on 29 Mar 2013 at 10:24

Changed state: WontFix


[logging] Fix ordering issues of logs

Overview
Logs sometimes come in out of order, because they're ordered by Elasticsearch submission time, and not Log time.  This causes possible misordering of logs between hosts, as well as some confusion.  I observed up to 2 seconds later a log's @timestamp was in comparison to it's actual timestamp.
Tasks

[x] (https://github.com/libra/libra/pull/5563) Fix timestamp field to have milliseconds for proper ordering
[ ] Either fix Logstash to properly use this timestamp, or research if the field can just be serialized to @timestamp

Related Issues
Parent Issue: https://github.com/libra/libra/issues/5441

So, it looks like because we use Logstash in premainnet and Fluent Bit in cluster test, it makes it hard to just serialize straight to @timestamp and I'm unsure myself if it can run correctly with logstash right now.
I created this issue to convert cluster test to logstash for compatibility reasons

Blocked on issue for the simplest approach


i3-swallow: init at unstable-2022-02-19

Motivation for this change
Really useful application to swallow windows using i3wm, very nice for stuff like mpv. Note that the pypi package uses a fork which only adds the ability to install it with pip.
Things done

Built on platform(s)

[X] 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
[X] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

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

[X] Fits CONTRIBUTING.md.

any change to upstream the poetry scripts ? also now that one repo has a poetry definition can't we use poetry2nix instead of a manual derivation ?
I use i3 but I can't understand anything about the gif beause it's just too small to read. How do you run the program ? I've tried
results/i3-swallow/bin/swallow mpv
i3-swallow: no process attached

any change to upstream the poetry scripts?

Yep, the fork adds them, they're not present at all in the main repository. There is an issue open which asks if upstream is interested in incorperating those changes, but it got ignored: https://github.com/jamesofarrell/i3-swallow/issues/25.
The poetry scripts are all included in 1 commit, so i tried to use the main repo and just fetch the patch that adds the poetry scripts, but I didn't have much luck:
~/nix/nixpkgs > ./result/bin/swallow
Traceback (most recent call last):
  File "/nix/store/slqk69xr82vfp51aba825cbd5rirfc45-i3-swallow-1.0.0/bin/.swallow-wrapped", line 6, in <module>
    from i3_swallow import main
  File "/nix/store/slqk69xr82vfp51aba825cbd5rirfc45-i3-swallow-1.0.0/lib/python3.9/site-packages/i3_swallow/__init__.py", line 1, in <module>
    from i3_swallow.swallow import main
ModuleNotFoundError: No module named 'i3_swallow.swallow'

Relevant lines from the diff:
-  src = fetchPypi {
-    inherit pname version;
-    sha256 = "sha256-ROHHp6yx7rjkA6G2+jULW6Vv5gh4CXy1WHPM/PVpdrg=";
+  src = fetchFromGitHub {
+    owner = "jamesofarrell";
+    repo = "i3-swallow";
+    rev = "7a9faed693b0da7df762a095fed512eca8c6414c";
+    sha256 = "1fq9cvisimgyza8c91bkill9wgyj7ac9sf9yz87npc1smlgscibd";
   };

+  patches = [ (fetchpatch {
+    url = "https://github.com/ViliamV/i3-swallow/commit/efd7af186c024ed5bf6036fc6cdb4762f08b7c55.patch";
+    sha256 = "sha256-CVW3aWAHmuQUn+FXT0UpYkXaneMPcWFwBXAYjqmaTxc=";
+  }) ];

I really don't understand how this doesn't work, since that commit contains the scripts in the same state as on the pip package, they have not changed at all. Only other change from the forked repo is some readme stuff, which we don't really care about.

also now that one repo has a poetry definition can't we use poetry2nix instead of a manual derivation ?

I'm not extremely familair with python packaging (also probably the reason I can't get the main repo to build :sweat_smile:), what benefits does the tool have? I can't find anything using it by just grepping through nixpkgs, and it's github repo mainly looks to be shell oriented.

I use i3 but I can't understand anything about the gif beause it's just too small to read. How do you run the program ? I've tried

Ah, that seems to be because mpv doesn't start an external window to swallow without an argument, since just prints some basic usage in the current terminal. When I just run swallow mpv I get the same error, but with swallow mpv /mnt/hdd/movies/... it works as expected

The poetry setup scripts have gotten merged in the main repository, so we're now using that :)
Also succesfully using poetry-core, still not sure why that didn't work out earlier but it does now.

/marvin opt-in
/status needs_reviewer

/marvin opt-in
/status needs_reviewer


Tests for overeager autolinking in raw HTML in tables

@tripodsan found an issue (regression) with regards to autolinking of raw HTML. The test shows that a link starting with http:// will be turned into an autolink, even when the parser should be in a raw HTML parsing mode
not ok 3 Tables - HTML - Autolinks (2)
  ---
    operator: deepEqual
    expected: |-
      '
\n
\n
\nfoo	\nbar	\n
\n
\n
\n
\nbaz	\nwww.example.com	\n
\n
\n
\n'
    actual: |-
      '
\n
\n
\nfoo	\nbar	\n
\n
\n
\n
\nbaz	\nhttp://www.example.com	\n
\n
\n
\n'
    stack: |-
      Error: Tables - HTML - Autolinks (2)
          at Test.assert [as _assert] (/Users/lars/Code/micromark-extension-gfm/node_modules/tape/lib/test.js:311:54)

Originally found in the context of tables, but it seems to be generalizable to all autolinks.

This current behavior is what GitHub produces.
This package is called micromark/micromark-extension-gfm because it aims to match GitHub.
You can check each of these examples here in a comment or in a Gist to see what GitHub produces: links in links.
Note that there are differences between how GitHub handles markdown in Gists/files compared to comments/issues/PRs/releases, as noted here: https://github.com/micromark/micromark-extension-gfm-autolink-literal#what-is-this.
I’ve added some extra tests to show some of your examples (tests are crawled from GitHub)