$query not a reserved property anymore

https://segment.zendesk.com/agent/tickets/34866
$query is no longer a reserved property.

LGTM


Add Metronome (MET)

Add support for the Metronome (MET).
Website: https://metronome.io/
Telegram: https://t.me/metronometoken
Twitter: https://twitter.com/MET_Token
Etherscan: https://etherscan.io/address/0xa3d58c4e56fedcae3a7c43a725aee9a71f0ece4e
GitHub: https://github.com/autonomoussoftware/metronome
Description: Metronome ("Metronome" or "MET") is  focused on making greater decentralization possible and delivering institutional-class endurance. Metronome is founded on three key design principles:

Self-governance with no undue influence from founders after initial launch and public access — contract governance starts at launch.

Reliability and predictability where issuance and supply are immutable

Portability to enable maximum decentralization, even across different blockchains. The first 'chainhop' took place in summer 2019, between Ethereum and Ethereum Classic

Additionally, Metronome reportedly offers advanced payment features as programmable money.

Mass pay: The ability to send MET to multiple addresses in a single transaction to save on transaction fees.

Subscriptions: The ability to authorize a set amount of MET to move between two addresses on a recurring basis.

Hi, @patidarmanoj10 could you solve conflicts? After that I can merge this request - thanks.

@xrn . Conflict resolved.   Thanks


Windows 7 gives a bad base64 error

When using the generated .ovpn file on windows I got the following error windows PEM_read_bio:bad base64 encode
I have googled around alot and it seems to be that some people got this error with problems with line endings not being unix, I have done everything that was suggested and cannot fix the encoding. Really stuck on what to do about this one

Looks like a problem in your end. Can you please try to download the certificate from the server directly to Windows? Alternatively, you can put it in a .zip to ensure it is not modified.

I fixed it in the end by recreating the whole file by hand, something is
happening with the unix file endings when taking the generated .ovpn file
over to windows 7. I found several posts about this online from other
users, completely unrelated to this. To fix it you just need to make sure
the file is ASCII character encoding, and for windows the inline certs dont
work so you have to make the ca.cert and ca.key files. Then it's all good.
Thanks again for this superb script, I have donated :)
On Thu, Jul 2, 2015 at 6:49 PM, Nyr<EMAIL_ADDRESS>wrote:

Looks like a problem in your end. Can you please try to download the
certificate from the server directly to Windows? Alternatively, you can put
it in a .zip to ensure it is not modified.
—
Reply to this email directly or view it on GitHub
https://github.com/Nyr/openvpn-install/issues/50#issuecomment-118007277.

I am having to the problem on Win7-64. liveforfree mentioned
"for windows the inline certs dont work so you have to make the ca.cert and ca.key files"
Can you please give the instructions how to do this exactly? or better yet, build this as an option in the script.

Ok mate, you simply gotta goto the openvpn directroy on your linux box
where you are running this install shell script and copy out the .ca and
.key files to your local machine, then ensure they reside inside of your
locals openvpn/config directory (same directory the .ovpn fille be)
next inside the .ovpn file that is generated you must remove the inline
  and  tags and put it back to the default samples and also
turn on the tap adapter because it is windows not linux - after that it
worked very well. but yeh certainly cost me a hour or so and alot of energy
figuring out wtf is up with shitty windows.
it's the same story again and again, our elders at microsoft decide to
release a unfinished product and cash out on it - so the shame rains on
their heads really because i have no respect for them and i would spit on
them if i saw them in the street for their pathetic abilities with
computers (not to mention internet explorer)
I hope u got it fixed an i hope windows is over soon cos it truely is the
worst operating system i ever encountered in my life
On Sun, Jul 5, 2015 at 5:40 AM, dafx<EMAIL_ADDRESS>wrote:

I am having to the problem on Win7-64. liveforfree mentioned
"for windows the inline certs dont work so you have to make the ca.cert
and ca.key files"
Can you please give the instructions how to do this exactly? or better
yet, build this as an option in the script.
—
Reply to this email directly or view it on GitHub
https://github.com/Nyr/openvpn-install/issues/50#issuecomment-118560860.

I am not able to reproduce this on Windows 7 with the current version of OpenVPN. In-line certificates are working fine for me.

Here is the screenshot of what I see.
http://content.screencast.com/users/miraxlabs/folders/Jing/media/c4ebbc64-06ff-46fa-bae0-5a0af8040f15/2015-07-06_2148.png

Sorry but I still can't reproduce.

Lucky you :)
On Tue, Jul 7, 2015 at 4:33 PM, Nyr<EMAIL_ADDRESS>wrote:

Sorry but I still can't reproduce.
—
Reply to this email directly or view it on GitHub
https://github.com/Nyr/openvpn-install/issues/50#issuecomment-119137057.

I really appreciate what you have done here. But my QA would start screaming if I say "sorry, works for me".

I just said I can't reproduce.
From what I know, this could have nothing to do with the script itself: if someone can help in proving the opposite and I can reproduce the problem, I'll reopen the issue and attempt a fix :)


How to host image using onImageUploadBefore

Hey there, I'm trying to understand how I can start hosting images. I've looked around and saw that I can use onImageUploadBefore to hit an api that returns a url, but I am not sure what to pass back. I was expecting to see an encoded64 string but I do not see anything on the File object
I've provided a codesandbox with a console log to show what I'm saying.
Version
suneditor-react: 2.16.5
Additional context
https://codesandbox.io/s/sunedtior-react-host-image-p7ydw?file=/src/App.tsx

https://github.com/mkhstar/suneditor-react#about-core
In "suneditor-react", there is no "core" object in the argument value.
onImageUploadBefore={(files, info, uploadHandler) => {

https://github.com/mkhstar/suneditor-react#about-core
In "suneditor-react", there is no "core" object in the argument value.
onImageUploadBefore={(files, info, uploadHandler) => {

I'm a little confused. What does the core object have to do with my question? Does it contain the encoded64 string? I'm wondering what I need from the File object to host the image

Um.. "base64" is the default.
Do not use the "imageUploadBefore" event to if you want render the image as "base64".

@JiHong88 I think I am just confused and lacking knowledge. What am I using in the File object to save the image in my server API?
On line 17 in my code sandbox: const hostedUrl = apiToHostImage(files[0]); . What am I actually using in the File object to save the image?

If you want to upload a file to the server,  just use the imageUploadUrl option.
Just upload a file to the server and send a response in the specified format.
imageUploadBefore is used in a special case. https://github.com/JiHong88/SunEditor/issues/782

@JiHong88 I don't see how I can use that to do my server call. These are the threads that I'm referring to:
https://github.com/JiHong88/SunEditor/issues/414#issuecomment-660418723
https://github.com/JiHong88/SunEditor/issues/551
https://github.com/mkhstar/suneditor-react/issues/128

Without using "imageUploadBefore", you can simply use "imageUploadUrl" to upload an image to the server.
Is there any reason why I can't use "imageUploadUrl"?

@JiHong88 Maybe an example can clear up what you're trying to suggest. From what you've linked me, imageUploadUrl only accepts a string that maps to the url (honestly not entirely sure how to use it either).
Where do I do the server call to get the url back after uploading the image from my files with imageUploadUrl?

@changyeamoon  Create upload api or use aws, etc your self.
You must be set image upload api url to the "imageUploadUrl" .
And set response data at the API.
request format: {
  "file-0": File,
  "file-1": File
}

response format: {
  "errorMessage": "insert error message",
  "result": [
    {
      "url": "/download/editorImg/test_image.jpg",
      "name": "test_image.jpg",
      "size": "561276"
    }
  ]
}

@JiHong88 I'm using a special api mechanism for salesforce so I don't think I can use this approach with the upload api. I just ended building a custom plugin to better suit my needs. Thank you for the help!
Closing as my question was answered here (https://github.com/mkhstar/suneditor-react/issues/128#issuecomment-863564684)

Can't read file-0 from request object with typescript. how to do this? where is this? In body? Please write few line of codes.


error: No such file or directory (os error 2)

> rye toolchain register /usr/bin/python3
error: No such file or directory (os error 2)

With https://github.com/mitsuhiko/rye/pull/26 I get:
> cargo run --  toolchain register /usr/bin/python3
    Finished dev [unoptimized + debuginfo] target(s) in 0.05s
     Running `target/debug/rye toolchain register /usr/bin/python3`
Error: could not symlink interpreter

Caused by:
    No such file or directory (os error 2)

Ah, I think the directory where the symlink is to be created does not exist.

@WhyNotHugo this should be resolved with 01fd9601e5f1ee5a5e708f6a43ceed1fe0d42de2

Yup, we reached the same fix I see. Thanks!


mark the database password as sensitive

When doing a plan or when an error log is produced the database password is exposed. This should not be the case and I believe marking this as sensative will solve this according the current docs.

@mattyjones would you mind rebasing to fix merge conflicts when you get a chance?

@mattyjones v2.8.0 has been just released with this minor fix.


[BUG] After upgrade to 1.2.0, longhorn-manager fails to start properly after reboot

Describe the bug
After upgrading from 1.1.2 to 1.2.0 via Helm Chart, I had to reboot a few of the worker nodes today. Afterwards, the longhorn-manager did not come up properly and seem to be stuck in a CrashLoop.
Logs show the following:
time="2021-09-03T10:30:11Z" level=error msg="Upgrade failed: upgrade CRD failed: upgrade from v1.1.0 to v1.2.0: upgrade recurring jobs failed: upgrade from v1.1.0 to v1.2.0: translate volume recurringJobs to volume labels failed: failed to update mariadb-0 volume: Operation cannot be fulfilled on volumes.longhorn.io \"mariadb-0\": the object has been modified; please apply your changes to the latest version and try again"
time="2021-09-03T10:30:11Z" level=info msg="Upgrade leader lost: k3os-agent-pve01-amd64-e6cda727"
time="2021-09-03T10:30:11Z" level=fatal msg="Error starting manager: upgrade CRD failed: upgrade from v1.1.0 to v1.2.0: upgrade recurring jobs failed: upgrade from v1.1.0 to v1.2.0: translate volume recurringJobs to volume labels failed: failed to update mariadb-0 volume: Operation cannot be fulfilled on volumes.longhorn.io \"mariadb-0\": the object has been modified; please apply your changes to the latest version and try again"

To Reproduce
Sorry, no idea.
Expected behavior
Longhorn comes up properly.
Log
n/a
Environment:

Longhorn version: 1.2.0
Installation method (e.g. Rancher Catalog App/Helm/Kubectl): Helm
Kubernetes distro (e.g. RKE/K3s/EKS/OpenShift) and version: k3s 1.21.4

Number of management node in the cluster: 3
Number of worker node in the cluster: 6

Node config

OS type and version: k3os
CPU per node: 4
Memory per node: 8192
Disk type(e.g. SSD/NVMe): SSD
Network bandwidth between the nodes: 1gb

Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal): Baremetak
Number of Longhorn volumes in the cluster: 25

Additional context
Add any other context about the problem here.

Same as https://github.com/longhorn/longhorn/issues/2961.

Guess I didn't find it because the error message was saying 1.1.2 -> 1.2.0 instead. Thanks.

Is there anything I can do until we get v1.2.1? I need to move some hardware pretty soon and suspect that the cluster won't come up with this problem afterwards,


support distribute ddl

Distributed tables need to be built for a large number of tables. At present, there are 20 nodes in the cluster. The workload of table structure management is very large. It is hoped that the ability of distributed management can be optimized, especially for ddl.
Because of the previous use of elasticsearch, the distributed management of ES is very perfect, and some excellent experience can be used for reference.
thanks.

Distributed DDL feature has already been implemented pretty long time ago: https://clickhouse.yandex/docs/en/query_language/create/#distributed-ddl-queries-on-cluster-clause


[nmstate-1.0] ovs: Do not validate on non-desired interface

Currently nmstate is showing unmanaged ovs vxlan with empty parent which
failure the apply() even it is not mentioned in desire state.
The fix is skip all the validations on non-desired interfaces.
Integration test case included.

The failuer is becasue we changed to keyfile in Dockerfile.c8s-nmstate-dev. Included these two patches via cherry-pick:

nm.connection: do not rename existing connections
CI: Use NetworkManager keyfile by default


Raw view doesn't store ELF section divisions

ELF linear view starts with:

and subsequently has visual indicators where sections begin and end as you scroll through the binary.
I think it would be really helpful to see these in raw linear as well (this will also allow us to show where non-mapped sections begin and end):

This is currently possible through adding user sections after analysis, and would be easy to add as a plugin, but I want it in the core.

Fixed in 2.2.2529-dev.


CNN.py

Hello, tahanakabi
When I ran the CCNN.py file, some errors appeared. May I ask if you are willing to answer questions?
Is it a parameter setting problem? Thanks.


Add a LICENSE

I know it's a simple piece of code, but would you care to release it using an OSS license?

done


Add sessionpool CRUD

This checklist is used to make sure that common guidelines for a pull request are followed.
Related command

General Guidelines

[ ] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
[ ] Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)
[ ] My extension version conforms to the Extension version schema

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

Thank you for your contribution! We will review the pull request and get back to you soon.

Please fix CI issues

Recommend title:
[Container app] az containerapp sessionpool: Support sessionpool command.
And please update history.rst


Fix twitter link and minify generated html pages to save a few kB

@lgeiger do you want me to carry on adding commits to this or review it? As it is assigned rather then being requested as a reviewer?

@lgeiger do you want me to carry on adding commits to this or review it? As it is assigned rather then being requested as a reviewer?

Sorry, I meant to assign you as a reviewer. Assignments and review requests are still not in my muscle memory  for GitHub 😇


Corne Version 4

Summary
This PR details the design changes in the latest version of the Corne Keyboard. I intentionally abandoned backward compatibility to achieve technological advancement and design updates. Changes include fine-tuning the key pitch, MCU alteration, TRRS jack and LED design improvements, the addition of expansion options, and the introduction of a new dedicated case.
Images
Corne Cherry

Corne Chocolate

Major Design Changes

Changes from v3
Abandonment of Backward Compatibility
The new version deliberately abandons backward compatibility with past models to enhance design freedom and aim for technological progress and design updates. This decision, while difficult due to its incompatibility with third-party cases, was deemed necessary for future development.
Major Design Changes

Change in Key Pitch
Departure from ProMicro
Change in TRRS Jack Parts
Abolition of Underground LEDs

In the new version, the key pitch has been adjusted from 19mm to 19.05mm to align with the dimensions of typical keyboards. Initially, the Corne design deviated from standard dimensions due to a lack of knowledge, but I have finally been able to correct this. Furthermore, replacing ProMicro with direct implementation of RP2040 on the PCB has improved the appearance and functionality of the product. The change in TRRS jack parts has also simplified the overall look of the keyboard. Additionally, the abolition of Underground (downward-facing) LEDs has been implemented for cost reduction and simplification.
Expansion
The space created by the removal of ProMicro has led to enhanced expandability for users. Continuous use of OLED displays, along with the possibility to add up to two expansion switches or rotary encoders, is now available. This allows users to customize their keyboards according to their needs, creating more unique and functional keyboards.
Dedicated Case
With advancements in 3D printing services, the new version also focuses on the design of a dedicated case. This case has a simple design and can be manufactured at a low cost. Users can easily customize the case to create their unique keyboards. These designs are published as open source, serving as references to foster creativity in the community and meet diverse needs.
Corne's Concept and Gratitude to the Community
Finally, the concept of Corne is to share the joy of creating keyboards and the possibility of designing them oneself, aiming to provide a foothold for users to design their original keyboards. I am deeply grateful that this project has continued much longer than imagined and has received support from many people. It is a great pleasure as the creator to see various individuals take an interest in this keyboard, assemble it themselves, and create new designs.

Summary
Corne Keyboardの最新バージョンの設計変更について詳述しています。後方互換性を意図的に放棄し、技術的進歩とデザインのアップデートを行いました。キーピッチの微調整、MCUの変更、TRRSジャックとLEDのデザイン改善、拡張オプションの追加、そして新しい専用ケースの導入が含まれます。
Changes from v3
後方互換性の破棄
この新バージョンは、過去のモデルとの後方互換性を意図的に捨てることで、設計の自由度を高め、技術的進歩とデザインのアップデート目指しました。これはサードパーティ製のケースとの互換性がなくなることでもあり苦渋な決断になりましたが、今後の発展に必要なものだと判断しました。
主要な変更

キーピッチの変更
ProMicroとの決別
TRRSジャックの部品変更
Underground LEDの廃止

新バージョンでは、キーピッチを19mmから19.05mmに変更し、一般的なキーボードの寸法に合わせています。Corneを設計した当初は一般的な寸法を知らなかったため異なっていましたがやっと修正できます。また、ProMicroの使用をやめ、RP2040をPCBに直接実装することで、製品の外観と機能性を向上させています。またTRRSジャックの部品変更により、キーボードの全体的な見た目がさらにシンプルになりました。さらに、Underground(下向き)のLEDを廃止することで、コスト削減と簡易化を行いました。
拡張性
ProMicroの廃止によって生まれたスペースは、ユーザーにとっての拡張性の向上を意味しています。OLEDディスプレイの継続的な使用に加え、最大2つの拡張スイッチやロータリーエンコーダーの追加が可能になりました。これにより、ユーザーは自分のニーズに合わせてキーボードをカスタマイズし、より個性的で機能的なキーボードを作成することができます。
専用ケース
3Dプリントサービスの進歩を背景に、新バージョンでは専用ケースの設計にも注力しています。このケースはシンプルなデザインで、低コストでの製造が可能です。またユーザーはケースを容易にカスタマイズし、自分だけの独特なキーボードを作成することができます。そして、これらのデザインはオープンソースとして公開され、コミュニティの創造性を促進し、さまざまなニーズに応えるためのリファレンスとして機能します。
Corneのコンセプトとコミュニティへの感謝
最後に、Corneのコンセプトは、キーボード作成の楽しさと自ら設計する可能性を共有することであり、ユーザーが自分だけのオリジナルキーボードを設計するための足掛かりとなることを目指しています。このプロジェクトが想像以上に長く続き、多くの人々に支持されていることに対して、深く感謝をしています。様々な方々がこのキーボードに興味を持ち、自ら組み立て、新しいデザインを生み出すことは、作者として嬉しい限りです。

the new version looks amazing.
are the pcb design finalized? I'm eager to make some and try

Is there reason why you stick to TRRS for split connection?
USB-C seems to be the saver (shortage while plug/unplug) alternative.

@hisea The PCB design is almost fixed and I'm adjusting firmware and creating a prototype with the specific case.
You can try it but, the design may change yet.
@BenRoe Thank you for your question.
Type-C functions as a part that connects the left and right sides. However, there is a slight risk in using the same parts as those used for connecting to a PC. I don't think anyone would make a mistake, but since it can be connected, it is necessary to ensure the safety of the electrical circuit.
I also don't have the skills to do so, and I believe there are no best practices in the industry yet. I might change it again if I have a chance to make v5.

regarding the Type-C connection between the sides, in this build it is solved like this https://github.com/weteor/3W6 and the creator said in an interview he saveguarded against the danger of plugging it in to the PC connection, maybe you could reach out and he could help with that

Thank you for sharing.
I'll reference if next time.

Hey @foostan - Thank you for all the work you've done with Corne! I've used Corne variants as my daily driver for a couple of years now.
Other than the change in key pitch and the new extension points where the pro micro were, are there any other physical changes? Does the pcb have the same dimensions and stand-off placemetn as v3?

Since the key pitch has changed, the overall size has changed. The position of the through hole for the screw has also changed.

@astrolemonade I added it. Thank you for confirming.


Inconsistency get_robot_mode service call

Issue Description :
Managed robot mode in hardware interface is 1(autonomous mode) while service call response with 0(Manual Mode) where the server presents in a controller node.

TODO :
The issues could be addressed in drfl library. it handles the status at global data section. by putting it into the proper section, the issue seems to be resolved. Similarly, Some might have same inconsistent issues.

On https://github.com/doosan-robotics/doosan-robot2/pull/134 :


Give a link to the README directly on the info page

The Game Directory Server greeting page displays useful info. Now that the client has merged into Evennia, I think it would be useful to expand the "How to add your game to this list" section with a direct clickable link to the README.md file in the contrib folder.

Done!


allow disabling of plaintext notifications on desktop

Allow users to toggle if desktop notifications show a preview of an incoming message. When disabled notifications look like this (on macos):

setting option:

relates to:
https://github.com/keybase/client/issues/15620
https://github.com/keybase/client/issues/8731
https://github.com/keybase/client/issues/7991
https://github.com/keybase/client/issues/7269
https://github.com/keybase/keybase-issues/issues/3348
cc @heronhaye

[ ] after release

merging over systests timeout


fix(iteration): handle undefined workItem error in WI iteration modal…

… for board view

@vikram-raj fabric8/fabric8-ui:SNAPSHOT-PR-2193-1 is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-2193-fabric8-planner.badger.fabric8.io


Variant D: no export after top-level await (does not block execution)

What about:
Variant D: no export after top-level await (does not block execution)
await can be used in main module body, but module will resolve synchronously as it's the case now (await will have same effect as initializing async execution using any other mean).
export could be declared only before first await, if used afterwards module load should crash with TDZ-like error.

In summary: it'll give possiblity to use an await syntax in top-level context, but it would not change how modules resolve otherwise.

Reopening as it was mentioned by @wycats at TC39 meeting, so probably is worth further discussion

Can you link to those notes? I recall mentioning a D but not necessarily this one.

Can you link to those notes? I recall mentioning a D but not necessarily this one.

https://github.com/tc39/tc39-notes/blob/master/es8/2018-01/jan-24.md#13iiil-top-level-await-for-stage-0 second comment of YK (@wycats)
Still @wycats doesn't state (or it's not included in the comments) whether the top level await is expected to block evaluation of dependents in that scenario. If it's the case, then indeed it'll be a different D

why await FooBarPolyFill[Symbol.moduleLoaded] when you could await import('./FooBarPolyfill.js') (after statically importing it)?

Because it's even more boilerplate if you have to export a .then yourself.
Perhaps it wasn't clear but I don't believe this variant would guarantee await import('./FooBarPolyfill.js') means the entire module has actually executed and it'd resolve potentially before the entire TLA is executed.
If await import('./FooBarPolyfill.js') did await the TLA of the imported module then yes we don't really have a major issue, we could just replace static import with dynamic and avoid pretty much all of the boilerplate.
Note that without a builtin one you need to add even more boilerplate yourself in every single module that might depend on a module with TLA.
e.g. The above example could be done already like this with no TLA, but it's rather tedious boilerplate to have to write in every single:
// math.js
import loadClassicScript from './loadClassicScript.js'

let math = null

export { math as default }

let resolve
let reject
const ready = new Promise((_resolve, _reject) => {
  resolve = _resolve
  reject = _reject
})

export const then = (...args) => ready.then(...args)

async function load() {
  await loadClassicScript('./math.js')
  math = window.math
  delete window.math
}

load()

import math from './math.js'

export default function someMathFunction() {
  // use math
}

let resolve
let reject
const ready = new Promise((_resolve, _reject) => {
  resolve = _resolve
  reject = _reject
})

export const then = (...args) => ready.then(...args)

async function load() {
  await import('./math.js')
}

load()

The champion group is pursuing proposal based on module exports permitted together with top-level await. The choice was presented at the March 2019 TC39 meeting. Therefore, I'm closing this issue.


archive: enhance parallel operations

This adds a few improvements to the way archive module works:

Support pixz as the parallel version of xz when creating an archive
Support lbzip2 as a better alternative to pbzip2 (lbzip2 works twice as fast in my tests)
Make unarchive helper also use parallel alternatives, and make it behave exactly the opposite to archive helper in terms of how commands are executed.

Looks good to me. Thanks for your contribution!


Weapon: grenades implementation

Halo-style grenades could be a useful addition for taking down forts and dealing splash damage.
Ideal implementation:

Grenades are stored separately from inventory, almost as if they were a resource.
The G key (+iv_drop / InputButton.Drop) should be used to throw a grenade.
When throwing a grenade, a first-person throwing animation should be played, and the actual grenade throw should be delayed until a point in this animation where the grenade leaves the viewmodel's hand.
Players are given two grenades upon spawning. They can be replenished through ammo packs.

Possible references:

https://github.com/Facepunch/dm98/blob/master/code/Entities/Weapons/GrenadeWeapon.cs
https://github.com/Facepunch/dm98/blob/master/code/Entities/HandGrenade.cs

Done, needs proper integration


Add example for Prettier ESM config

Thought this might be useful for folks since we are in a shift towards an ESM config future.

Hey thanks! I think actually just going to simplify this to the JSON configuration since there are about 500 ways to configure Prettier and think that's the most common, updated it directly in this PR 👍


MINOR: Update version for doc to 2.0.0

Committer Checklist (excluded from commit message)

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

@dguy Thanks for the review, merging to trunk and 2.0.


haxe: add 4.1 and 4.0 compiler versions

Motivation for this change
Fixes #95638. cc @OPNA2608
Arguably the extlib 1.7.7 override is not ideal, but I don't really want to be stuck with it in ocamlPackages as long as we don't need it for anything else. The compatibility patch from 4.2 unfortunately doesn't even apply on 4.1.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
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 nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 118112 at 108cd8e8 run on aarch64-linux 1

2 packages built successfully:

haxe_4_0
haxe_4_1

Sorry for the late response! Tested haxe_4_1 and haxe_4_0 with a game project, both worked fine.

@ofborg build haxe_4_0 haxe_4_1 ocamlPackages.ocaml_extlib ocamlPackages.ocaml_extlib-1-7-7


drivers: can: shell: print device name in RX path

Include the device name when printing received CAN frames. This improves the user experience when working with multiple CAN controllers via the CAN shell.

Question: is the shell always in privileged mode?

Question: is the shell always in privileged mode?

Yes.


using crate2nix with nixpkgs and flake: error: attribute 'currentSystem' missing

After adding a package in nixpkgs with crate2nix, I'm experiencing some problems using it with flakes.
When I try to add it, I get the following stacktrace
at /nix/store/dxdvm5d0bmxhldkvmxl24xbac9xhq4x7-source/pkgs/top-level/impure.nix:15:1:

           14|
           15| { # We put legacy `system` into `localSystem`, if `localSystem` was not passed.
             | ^
           16|   # If neither is passed, assume we are building packages on the current

       … from call site

       at /nix/store/dxdvm5d0bmxhldkvmxl24xbac9xhq4x7-source/pkgs/servers/search/meilisearch/Cargo.nix:7:1

            6| { nixpkgs ? <nixpkgs>
            7| , pkgs ? import nixpkgs { config = {}; }
             |          ^
            8| , lib ? pkgs.lib

       … while evaluating 'buildRustCrateWithFeatures'

it looks like in the call to import nixpkgs, the system attribute is not passed. I'm wondering how to best solve this issue.
currently trying a few things.
You can test this in nixpkgs master by running
nix build .#meilisearch -L it will give you the whole stackstrace.

You need to pass in the pkgs to Cargo.nix while you are importing it in default.nix in the same folder. So adding pkgs to the arguments and inherit pkgs; in import Cargo.nix { ... } works fine.

Thanks a lot of the help on this!
I tried modifying the system attribute inside the import, but the best thing was indeed to pass pkgs directly.
I tried removing the nixpkgs = ... line as well since it didn't seem needed.


fix map length example

Purpose
this will fix a small bug in map length example.

@xlight05 shall we also update the expected output to reflect this correction?

@MaryamZi Yea, sorry forgot :laughing: .

@xlight05, thank you!
Would it also be possible to update the commit messages? :)
The convention followed is to capitalize the first letter of the commit message and to use imperative mood ("Fix" instead of "Fixed"). For more details you could refer to https://chris.beams.io/posts/git-commit/.
e.g., - Fix map length test

@MaryamZi  Done.
Thank you for instructions.


Distro aware checksum

This fixes the first problem described in issue #1342. Otherwise the checksum would always be failing on Alpine

...and FYI the final piece in the puzzle: https://github.com/itzg/docker-minecraft-server/pull/1349

@itzg might be fun to know that the get distro line was written by GitHub Copilot 😆

That is a fun fact especially since I was impressed it can be determined in one shot like that. Oh...I might be thinking of further retrieving the distro version.

@itzg This is what Copilot makes of it:
# Get distro version
distro_version=$(cat /etc/os-release | grep -E "^VERSION_ID=" | cut -d= -f2 | sed -e 's/"//g')

Basically same strategie

Aha, it's the pre systemd days that I'm remembering...and didn't realize systemd also included os-release
http://0pointer.de/blog/projects/os-release


Validate that selected routes are not obsolete

in helper processsms

helper processsms is cleaned up, the obsolete modem is removed from the possibleroutes list. But the modemid is used to find the correct modem in the routing list. Thats bad because if the modem is registering to fast, there may be tow route entries with the same modemid available. The routing will fail.
As quickfix I will remove this routing entry also on watchdog run but it will be necessary to refactor the code to routingid

Temporary fix pushed. This fix may break some things under some circumstances. This is not resolved.

The logic should be refactored to use routingid instead of modemid inside the smstransfer object to pin the route to use.


feat: Design Gnone 43

Hi, really nice project
This is based on gnone 42 design, right?
Gnone 43 design really looks good and better, so I would suggest to move this to v43 when you've time
happy coding :)

Thanks for your interest in the project.
Yeah this is based on gnome 42. Unfortunately, upgrading to gnome 43 caused some weird bugs where my brightness adjustment bar and the touchpad settings disappeared.
So, I can't try it and get the icon set, but I would be glad if you could help me find the icon set and give some UX suggestions :)


Kandarin Hard Diary Wiki --> CHarge Water Orb --> Needs check to ensure on standard spell book

Got there was on ancients, rip.

fixed in https://github.com/Zoinkwiz/quest-helper/commit/f25955c844987836d86b22755b26df95ce6c21d5.


Introduce MongoFullTextSearchQueryUsage check

What?
A simple check which discourages the usage of Filters#text API in Mongo queries.
Why?
We have observed at Picnic that using MongoDB to serve full text search queries can lead to major cluster outages (Mong server OOMs) when query strings are tokenised into even just a handful of tokens, invoking an index scan (IXSCAN) for each individual token in a query string. As Mongo does not gracefully handle such queries, and it is not possible to ban the usage of this filter on the (Mongo) server-side, our best effort is to introduce a Bug Check. MongoDB advises against implementing FTS queries on Mongo Atlas, and to use alternative solutions like Atlas Search, and other search engine databases like Elastic Search, or by simply implementing bespoke queries.
Implementation notes

Description "Usage of the Mongo full-text search ($text) queries is discouraged" doesn't actually hold true, perhaps there are other reasons why you would need to use the Filters API to obtain the Bson query and not use it in the Java driver - this Bug Check would still flag this. I can add the driver dependencies and match on the execution of a query using the drivers, or we can settle on this being good enough.
~Named enum value MONGO, like others in ThirdPartyLibrary unsure why we don't refer to the specific name of the FQC but likely we can revisit later when we need to rename in order to encompass more than just the Filters API in this case.~
Excluding the org.mongodb:bson-record-codec artifact , not required for testing, but contains classes violating compatibility with byte code enforcement limited by the max JDK version (11). The other option is to (see commit) ignore the class via Maven byteCodeEnforcement ( org.bson.codecs.record.* ) but I think this would be more disruptive.
Perhaps a more elaborate summary is desired to help others better understand the underlying problem, but typically we keep these pretty succinct?

Suggested commit message
Introduce `MongoFullTextSearchQueryUsage` check (#649)

Discourage the inefficient usage of `mongodb-driver-core`'s `Filters#text`
for constructing full-text search queries.

If I understood @Stephan202 correctly from an offline discussion we can downgrade this to a SUGGESTION and for our fork of error-prone utilise this upcoming feature to https://github.com/PicnicSupermarket/error-prone/pull/33. However this would apply to 25 other SUGGESTIONs if we were to utilise this flag internally.
Question, would there be an option to add an additional tag from an enum to indicate the Picnic escalation on a per check basis using our fork's processing of the annotation yet still ensuring compatibility with error-prone? Something like:
enum PicnicSeverityEscalation {
    WARNING_TO_ERROR,
    SUGGESTION _TO_ERROR,
    SUGGESTION_TO_WARNING
}

or simply just
enum PicnicSeverity {
    ERROR,
    WARNING,
    SUGGESTION
}

such that for usage in @BugPatterns:
@BugPattern(
    summary =
        "Usage of Mongo for full-text search queries via the `$text` operator is discouraged.",
    link = BUG_PATTERNS_BASE_URL + "MongoFullTextSearchQueryUsage",
    linkType = CUSTOM,
    severity = WARNING,
    tags = {PERFORMANCE, PicnicSeverityEscalation.SUGGESTION_TO_WARNING})
...

then in our fork modify BugCheckerInfo to iterate over the tags and override the BugCheckerInfo#defaultSeverity where necessary?
Interested to hear your thoughts!

An extra thought: we could've also used a Checkstyle module here to flag this method. However, I imagine this check being generalized when we find another method we want to flag.
We can have a check that flags multiple specific methods and have a custom message for every thing that we want to flag. Additionally, we should see if we can provide an optional fix for the methods that do have a simple replacement. So the check will contain methods without possible replacement, or simpler replacements then the ones from the FluxFlatMapUsage check.

Rebased.

Rebased.

Rebased, will merge!


TypeError: argument of type 'NoneType' is not iterable ('trigger.entity_id' is not a field in automation)

I've added an automation based on your example "reset manual_control after 1 hour", but it fails due to trigger.entity_id not being defined - has this been updated/changed because I cannot find any reference to the adaptive_lighting switch in the event
Thanks a lot

The entity_id should be included: https://github.com/basnijholt/adaptive-lighting/blob/17ac3951240a65e75e79a548b85d50d7319d9f9b/custom_components/adaptive_lighting/switch.py#L260
Perhaps you changed the trigger type?

Actually, you are right:

This is a bug! I will fix it.

Thanks for directing me here @basnijholt. Good to know it's a bug and you've identified it. Looking forward to getting this working!

This should be fixed now:

You should make the change like https://github.com/home-assistant/home-assistant.io/pull/14877/files#r546235892


chore: fix optionalDevDependencies regression

Fixing a regression with optionalDevDependencies.
@IgorTodorovskiIBM please comment on this to let me know if issue #509 is still relevant.   The optional-dev-dependency util is a bit of a footgun because of how it mutates package.json unexpectedly, and how easy it is to forget that it's part of our dev environment.  That, and the fact this regression has gone unnoticed for four months, has me wondering if we could revert #510 to simplify our build setup... which would be really nice.
I'm fixing this for now, but if I don't hear anything to the contrary  I expect I'll do that reversion the next time this issue pops up.

Actually, this PR would help as our z/OS uuid builds continue to fail because wdio is added as a devDependencies. Moving it out of devDependencies to only in optionalDevDependencies would help.

Thanks a lot for all the recent cleanups @broofa!
Just noticed one thing: are we able to submit PRs without approval from a second maintainer now?


Alias option doesnt work , throws error:040A1079:rsa routines:RSA_padding_check_PKCS1_OAEP_mgf1:oaep decoding error

Hi ddimitrioglo,
aws-saml version 1.1.2
as mentioned in the documentation I created a config.json file with the aliases section like below :
{
"profile": "saml",
"username"<EMAIL_ADDRESS>"password": "***",
"directoryDomain": "https:///adfs/ls/IdpInitiatedSignOn.aspx?loginToRp=urn:amazon:webservices",
"aliases": {
"ACCOUNTID":"nonprod"
}
}
I ran

aws-saml login --alias nonprod and got error:040A1079:rsa routines:RSA_padding_check_PKCS1_OAEP_mgf1:oaep decoding error

same issue when i run aws-saml login --alias
Thanks in advance!

Hi @aashrit93 ,
To fix the issue faster, could you please answer several questions?

Is usual (interactive) flow working? (aws-saml login)
Are you trying to configure password via CLI or adding manually to the ~/.aws-saml/config.json?
Is your password value in ~/.aws-saml/config.json a valid base64 string?

Thank you.

Hi Dmirtri,
Thank you so much for your prompt reply. I realized I was storing the
password as a regular UTF-8 string in the ~/.aws-saml/config.json file.
Making the string base64 worked. I really appreciate the effort you took to
build this package, it makes life easier for me . Cheers :)
Regards,
Aashrit
On Sun, Nov 10, 2019 at 8:14 PM Dmitri Dimitrioglo<EMAIL_ADDRESS>wrote:

Hi @aashrit93 https://github.com/aashrit93 ,
To fix the issue faster, could you please answer several questions?

Is usual (interactive) flow working? (aws-saml login)
Are you trying to configure password via CLI or adding manually to
the ~/.aws-saml/config.json?
Is your password value in ~/.aws-saml/config.json a valid base64
string?

Thank you.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ddimitrioglo/aws-saml/issues/17?email_source=notifications&email_token=ABM7CDSZHKHO4RARLPMJMXTQTAM6DA5CNFSM4JKVEQOKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDU6WVY#issuecomment-552201047,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABM7CDQCOEYF6MPN5EWWHK3QTAM6DANCNFSM4JKVEQOA
.

Hi @aashrit93 !
Glad to hear that you could configure and make it working!
Thank you for the feedback 😉


Type checking

TSConfig added in https://github.com/projectwallace/css-code-quality/commit/95b35df7aa7345357cfd61660dea0b8f22ffe5c4


Consider improving console transparency

As reported on this twitter thread, consider improving how the Windows Console renders background transparency by making the background transparent, but not the text itself.
Here's an example of where the text is a little difficult to read since its slightly affected by the luminance of the background image:

Probably be nice to also be able to do the same effect as text on the Windows desktop; white on black, to ensure contrast whatever the background.

The title bar should be opaque, too (IMHO)

It already is Marcogroppo.

Thanks for the ask. Closing since this is now on our feature backlog which will be dealt with when we get around to overhauling the Console rendering engine.
Closing. If you have further asks/issues, please file new issues on our Console GitHub Repo.


Query string in authorize request of /? format is not treated as OAuth request

Describe the bug
If a client calls the spring authorization server /oauth2/authorize endpoint with a query string following a trailing slash, they do not access the OAuth resource. Easy to work around, but passing it on to handle as you see fit.
To Reproduce
Call an implementation of the spring authorization server with eg.
curl --verbose --url "{{HOST}}/oauth2/authorize/?response_type=code&client_id={{valid-client}}&scope=scope-a&state=anyState&redirect_uri={{valid-redirect-uri}}"
Server will respond with login prompt/401 unauthorized
Expected behavior
302 response code and redirect following behaviour without trailing slash
Sample
N/A
Reports that include a sample will take priority over reports that do not.
At times, we may require a sample, so it is good to try and include a sample up front.

@townsece /oauth2/authorize/?response_type... is not a valid authorization request. The authorization endpoint URI should be followed by the query string (without trailing slash). See example in 4.1.1. Authorization Request


UnlockedFlash::write memory safety

The function UnlockedFlash::write (and write_native) let you write to arbitrary memory locations, bypassing Rust's ownership checks, but are currently not marked as unsafe.
I recently made some changes to some code that was using this function and accidentally ended up passing an address that was effectively a pointer to the stack rather than a pointer to flash. Needles to say, this didn't end well.
I feel like these functions as they are, should be marked as unsafe, since they rely on the caller to check that the address to be written is (a) part of flash and (b) not part of the program currently being executed.
A safe alternative API would be to accept an &mut [u8] for the part of flash that is to be written.
Thoughts?

You are absolutely correct about this. Those must be marked as unsafe.
About alternative API how do you make sure that &mut [u8] is part of the flash memory and not something else?
Also have to recheck what clever tricks other HALs are using. Maybe it is already solved somewhere.

Address could be checked if those are correctly defined for every MCU: https://github.com/stm32-rs/stm32g0xx-hal/blob/main/src/flash/mod.rs#L10-L11

If the &mut [u8] wasn't part of flash (e.g. it was in RAM), I think it would end up as a somewhat slow, somewhat weird form of memcpy - i.e. it would overwrite what's in the destination slice, but that's OK and wouldn't violate memory safety (since you could have overwritten it via other simpler means anyway).


unable to parse scientific notation numbers (i.e 1.234E-4)

I cannot parse WKTs that use scientific notation, so something like: 'POINT (3.234E-2 10)' doesn't work, even though this seems proper.  I didn't have access to the ISO, but it is allowed in the OpenGIS implementation specification.
Here is a jsfiddle that demonstrates my problem:
http://jsfiddle.net/qfxxLx95/4/

Hi Tom,
I am having a similar issue at the moment. I know it was a while ago, but I wondered if you ever found a solution to the problem?
Thanks,
Oliver

We ended up working around the issue by having the tool that generated the WKT not produce scientific notation.

I looked a little closer.  I think it should be a pretty small change.  I can't try it now, but maybe tonight.
This file defines the grammar:  terraformer-wkt-parser/src/wkt.yy
I think we can just change line 10 to be:
"-"?[0-9]+("."[0-9]+)?([eE][-+]?[0-9]+)? return 'DOUBLE_TOK'
The DOUBLE_TOK tokens seem to be passed into Number which supports sci-notation.
I'll try it soon, but feel free to try it and submit the pull request if you don't want to wait on me Oliver.

closed by your own f11b0e6e8df1a885b6fab0b28d9838cd01b45fe4.  thank you!


VirtualBox no longer starts after updating to Windows11 and using WSL2 and WSLg

Windows Version 21H2 (Build 22000.318)
Windows Subsystem for Linux WSLg Preview 1.026 (17.11.2021)
Oracle VirtualBox 6.1.28

After updating and using WSL2, VirtualBox no longer starts. Is this error known?
I use few Linux distros under WSL2. But I have installed 20 different machines for network testing under VirtualBox. I can no longer start these virtual machines.
I have already reinstalled VirtualBox as well. That did not bring any solution.

folks seem to hit similar/same issue ( MicrosoftDocs/WSL#798 ) on VirtualBox 6.1.28, downgrading to .26 might solve it. And well, VB and Windows Hypervisor doesnt go well, you will end up using quirky workarounds at the end :)

Virtualbox 6.1.30 has fixed a regression in 6.1.28 (Changelog)
VMM: Fixed 6.1.28 regression preventing VMs starting when using Hyper-V mode on Windows 10
I'm able to boot Vagrant / Ubuntu images that I was using prior to enabling WSL2, they also work with Hyper-V enabled. They're about 10-12% slower than before, but perfectly usable for my needs.

Enable Windows  Hypervisor Platform component

Thank you for the information. There are probably some dependencies between the different virtualization options that I don't completely understand. I have reset Microsoft Windows 11 Home and reinstalled it cleanly. Then I installed Oracle VirtualBox version 6.1.30. I can now start my virtual machines again. Since the focus of my work is a lab environment with many Linux virtual machines and networks, I will wait to test WSL2 and WSLG further.

Virtualbox 6.1.30 has not fixed the regression for me. The newest working version as of right now is 6.1.26

Thank you for the information. There are probably some dependencies between the different virtualization options that I don't completely understand. I have reset Microsoft Windows 11 Home and reinstalled it cleanly. Then I installed Oracle VirtualBox version 6.1.30. I can now start my virtual machines again. Since the focus of my work is a lab environment with many Linux virtual machines and networks, I will wait to test WSL2 and WSLG further.

Tested here and it is working together.
OS:

Edition: Windows 11 Home
Version 21H2
OS build 22000.434

Virtual box

Version 6.1.30 r148432 (Qt5.6.2)

WSL:

Name: Ubuntu
State: Running
Version: 2

Windows Feature

FeatureName : VirtualMachinePlatform
State       : Enabled

an old windows XP ova, that has been runing fine in 10+ years, also on my latest WIndows 11 Home (on AMD Ryzen 7 5800 U), does not work any longer after installing and activating WSL 2. Gets stuck at boot up. Oracle, what's wrong, why haven't you been able to steer clear of this in almost 2 years?

re-confirmed on Windows 10 machine, 12 year old i7 mobile ,with WSL2 and VB 6.1.16
the same old XP OVA as mentioned above, that ran fine before install/update of WSL2, does get stuck on boot screen.
I think the VB VMs do not get the resources they wish to claim, RAM allocation in task manager shows peanuts!

addtional info on my last two posts: 6.1.26 does not help at all, no difference.
It must be the latest updates that have to do with WSL2.
How can Oracle let it slip so much for so long (first reports I found were spring 2020).
I would NEVER choose Oracle for any professional solution, with their product strategies, tbh...

Me thinks the Virtualization Based Security is the culprit... must be being activated during WSL2 setup?


Refactored CODEOWNERS file

This PR includes a refactor of the CODEOWNERS file.

Task linked: CU-869510n4t Standardize GitHub permissions


Fix KMIP file upload error message

Turns out that the drop is also rejected if the file extension is not '.pem' (even though PEM encoded files can be in many names, e.g. cert-manager uses .crt in its secret keys). Let's make sure the error message represents this, to avoid wasting more time from other people.

/ok-to-test

/lgtm
/assign sanjalkatiyar

Thanks a lot for pointing this out and raising a PR :)

Though we need to make change at one more place: https://github.com/red-hat-storage/odf-console/blob/master/packages/odf/modals/advanced-kms-modal/advanced-vault-modal.tsx#L68
Also, no matter how small the change is, any textual changes are verified by UX designers and Doc team, I will ping them for the same and will let you know.

Thanks a lot for pointing this out and raising a PR :)

Though we need to make change at one more place: https://github.com/red-hat-storage/odf-console/blob/master/packages/odf/modals/advanced-kms-modal/advanced-vault-modal.tsx#L68
We need to run yarn i18n command, after any textual changes for the parser to make respective change in the json locales file.

Thanks for the reply!
I will fix items 1 and 2 and look forward to the response of 3!

Thanks for the reply! I will fix items 1 and 2 and look forward to the response of 3!

Hi, I checked with them and Invalid file: File size exceeds 4MB limit or incorrect file format (only .pem files allowed) is their suggestion.

@SanjalKatiyar thanks. Will update now.

Though technically the file format doesn't have to be wrong, the error is in the file extension.
I can absolutely have a PEM-encoded file in a file with a .crt extension (again, look at the default cert secrets used by openshift - those call it ca.crt, cert.crt and ca.key).

Though technically the file format doesn't have to be wrong, the error is in the file extension. I can absolutely have a PEM-encoded file in a file with a .crt extension (look at the default cert secrets used by openshift - those call it ca.crt, tls.crt and tls.key).
So would this also be fine: Invalid file: File size exceeds 4MB limit or incorrect file extension (only .pem files allowed). ?
(replacing format with extension.)

looks good to me !

/approve
/lgtm


There is a background color for the last property value on 'Properties' panel after switching to the next item by pressing 'Tab'

Storage Explorer Version: 1.20.0-dev
Build Number: 20210519.5
Branch: main
Platform/OS: Windows 10/ Linux Ubuntu 18.04/ MacOS Big Sur 11.3
Architecture: ia32/x64
How Found: Exploratory testing
Regression From: Not a regression
Steps to Reproduce

Select one item which has properties on its Properties panel (Like one queue).
Press 'Tab' to switch among the current items.
After switch from the last property value on the Properties panel to the next item.
Check there is no background color for the last property value.

Expected Experience
There is no background color for the property value.
Actual Experience
There is a background color for the property value.

The property values are displayed in <input> elements, which are in part styled by the system. Try as I might, I can't modify the color of the selection in an obvious manner, nor can I programmatically remove the selection range.
I noticed the text highlight goes away when focus wraps back around into Cloud Explorer from the top and comes back after Shift-tabbing out again. It's possible this is a weird quirk with the Cloud Explorer iframe.

Turns out the weird behavior is somehow related to the fact that the inputs are read-only. However, the selection color can be fixed with a combination of input::selection and input:focus::selection CSS selector.

Verified this issue on the build main/20220410.3. Fixed,


help me guys

[x] I have read the FAQ.
[x] I have searched in existing issues.

guys what is this

What is the result of adb devices?

https://github.com/Genymobile/scrcpy/blob/master/FAQ.md#device-not-detected


Add temporary holiday to xkrx exchange

Korea has declared Aug 18, 2020 as temporary holiday, and the KRX market does not operate on that day.
See http://global.krx.co.kr/contents/GLB/05/0501/0501110000/GLB0501110000.jsp

@sjquant - Thanks! Looks good to me.


⚠ Cioos Atlantic CKAN has degraded performance

In 25eeb6f, Cioos Atlantic CKAN (https://catalogue.cioosatlantic.ca/) experienced degraded performance:

HTTP code: 503
Response time: 18533 ms

Resolved: Cioos Atlantic CKAN performance has improved in 61253a1.


feat: add plugin to remove first h1 header

When writing markdown, it's customary to have a H1 header at the top of the file, e.g.
---
title: Page Title
tags: ["test", "custom", "title"]
---

# Page Title

Some text

## H2 Title

More text

This can for example be enforced by the great Obsidian Linter extension.
Since Quartz uses the frontmatter title (or filename as a fallback) to display the page header, the result is that there are two headers:

This plugin removes the additional Markdown title:

I don't think this should be the behaviour.
If they want to remove the title, they can. Otherwise, this is not WYSIWYG.

Yes, this should not be the default. But quite some people might like it and it’s not a one-liner. That‘s why I made it an optional Plugin.
What criteria do you have for including (optional) plugins?

Feel free to post this in the community plugins. I'm sure there are ppl would find this useful. I'm going to just close this for now.
I think there was a heads-up that we will not currently accept plugins atm.

People like to browse close PR, so if there is anyone ask for this you can just reference this implementation to them.

Thanks for the link. I think this could be added to the „making a plugin“ page, so people know that no PRs are accepted but there is another publishing option for custom built plugins.


Read this if you experience PDF printing hanging on Windows

With M125 Chrome uses sandboxes for printing (https://crbug.com/338553158) which may require additional permissions on the Chrome installation folder:
See https://pptr.dev/troubleshooting#chrome-reports-sandbox-errors-on-windows for how to set permissions.

Thanks for this explanation!
Instead of icacls $HOME/.cache/puppeteer/chrome /grant "ALL APPLICATION PACKAGES:(OI)(CI)(RX)", the command should be icacls %USERPROFILE%/.cache/puppeteer/chrome /grant *S-1-15-2-1:(OI)(CI)(RX).
%USERPROFILE% is the windows equivalent of $HOME and S-1-15-2-1 is the SID – the user friendly name does not work in non-english Windows.

I need

Will this be fixed? Could it at least print a warning on installation with link to workaround? Thanks.

The fix will be available once Puppeteer ships Chrome 127.

My puppeteer version is win64-127.0.6533.72, I used icacls $HOME/.cache/puppeteer/chrome /grant "ALL APPLICATION PACKAGES:(OI)(CI)(RX)", nothing changed, and still have this problem!
The only way I can start it working is to start my app in debug terminal!!!!
I am using Windows 11.


Add precondition guards to pthread_mutex calls

While in the current configuration lock/unlock shouldn't ever fail, it seems polite to guard against basic misuse in debug builds.

I think you may want to pull the result of the lock/unlock calls out of the precondition(). According to the docs:

In -Ounchecked builds, condition is not evaluated, but the optimizer may assume that it always evaluates to true. Failure to satisfy that assumption is a serious programming error.

So in -Ounchecked builds, there will be no locking at all in these paths.

err good call @kperryua; will adjust that


Nice work!

Spasibo! Complete implementation and well-designed tests! I've learned a lot from this repo and your articles. ✨

@ManiaciaChao,
It's nice to hear. I was glad to help.

@ManiaciaChao,
It's nice to hear. I was glad to help.


hassio: Component failed to initialize

Hello,
I have an issue hith hassio, from few days it doesn't apper on the webui.
Home Assistant release with the issue: 0.89.2
Operating environment (Hass.io/Docker/Windows/etc.):
docker on debian 9
Description of problem:
Hassio doesn't appear on homeassistant webui. I tried a complete reinstall bu the problem is still here.
Traceback (if applicable):
2019-03-21 06:25:02 WARNING (MainThread) [homeassistant.setup] Setup of hassio is taking over 10 seconds. 2019-03-21 06:25:02 ERROR (MainThread) [homeassistant.components.hassio.handler] Timeout on /supervisor/ping request 2019-03-21 06:25:02 ERROR (MainThread) [homeassistant.components.hassio] Not connected with Hass.io 2019-03-21 06:25:02 ERROR (MainThread) [homeassistant.setup] Setup failed for

/move to home-assistant/hassio


Update all component demo page references

General Support Request
We need to keep the naming convention for all demo pages consistent across all components. In this case, we all files for the component landing page to be named index and the api examples page to be named api.
Possible Solution
Update all demo.md or demo.html references to index.md or index.html and update all apiExamples.md or apiExamples.htmltoapi.mdorapi.html`.
Additional context
These file names are being updated across all components.

Note: Will need to also update other components to actually used this updated script.  Doc Site will also need to be updated to take into account the new names.

This work is all completed by this PR that will be merged shortly


Website monitor not exists option ignore_cert_err

Hello,
I find the option ignore_cert_err exists when we create a website monitor by using Site24x7 API but it does not exist in Terraform parameter. Could you advise if this option is available in the future?

@DarkWandererOne Thank you for raising this issue. We have fixed it in the latest version.


Abnormal exit

thread '' panicked at C:\Users\Administrator.cargo\registry\src\github.com-1ecc6299db9ec823\clipboard-rs-0.1.2\src\platform\win.rs:72:60:
Open clipboard: OSError(5)
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace
Recurrence method : When copying text in windows Terminal

@zibo-chen
Could you please provide me with the code you run? I'll do a test.

I see your error description is that the clipboard handle cannot be opened. I think this is a random situation. Can you try a few more times to check? Failure to obtain the clipboard handle is normal, so the code retries 10 times by default. If you still can't get it, is it possible that another program of yours is occupying the clipboard handle?

I frequently encounter this issue within Excel, but I have managed to resolve it by introducing a delay in the process. It seems that Excel might be writing to the clipboard multiple times in quick succession, which could cause the program to fail in obtaining the clipboard handle. By adding a pause, it allows enough time for the clipboard to become available again before the program attempts to access it. This workaround has been effective for me.

@zibo-chen
I understand the problem you mentioned. Your guess is correct. I will try to see if it can be optimized. When I have the handle, I don’t need to get the handle again. This should solve your problem.


Few More UI Changes (Kaldarasha Requests)

Refer to the image below for the number references:

Take this side bar out for the Public release, and only keep it for the Debug release.

Move these buttons to the right-side when Compact List is unchecked (disabled). Keep them in the same place when Compact List is checked (enabled). Not sure if this can be done. Can they be dragged somehow instead of having to click an up/down button for each move? If you have any suggestions, let me know.

In the Active Mods tab, get rid of the Installed box next to the mods. It's obviously redundant.

Get rid of the Custom GL Config button. It doesn't really serve a purpose for anyone, therefore it is never used.

okay will take a look
all seems doable

sorry but was busy
here are the changes
Debug.zip
Release.zip

Hmm, I prefer the previous version tbh. The side buttons now have horrible clipping when you use Compact List in the Active Mods tab. Plus, you can't resize the left-pane window, so there is a lot of empty space between the mod details on the left and the buttons on the right.
Let's just revert back to the old style, but have the Installed button removed in the Active Mods tab.
If you have any ideas for it, please share.

I'm thinking about a full ui rewrite
if you could draw a draft with some buttons or somthing...
that would be great

This is something that I was working on

Not sure about the buttons yet, but this is a 7H logo made by someone on the forums. We should replace the Midgar backdrop with it.
https://drive.google.com/open?id=0B-Q_AObuWRSXUUt6bkVzVXdRclE

somethin I have done :D

Sorry I've been absent. I have a couple weeks of summer with more free time, so let's pick this back up shall we? =)
Can we get anything working with the new GUI?

sadly my windows HD died (nothing lost) and I don't have any way to use visual studio for now
also busy with other stuff....
sorry


rowCount length

Hi there,
Noticing that the rows returned from the worksheet.getRows is different to the value returned from logging the worksheet object.
spreadsheet.useServiceAccountAuth(creds, function (err) {
        // Return the spreadsheet
        spreadsheet.getInfo(function (err, info) {
            // Loop through all the worksheets
            async.eachSeries(info.worksheets, function (worksheet, cb) {

                console.log(worksheet);
                // ... worksheet.rowCount = 49 here ...

                // Loop through the rows
                worksheet.getRows({'max-results': 300}, function (err, rowData) {
                    if (err) return next(err);

                    console.log(rowData.length);
                    // ... rowData.length = 16 here ...

                    cb();
                });
            }, function (err) {
                db.end(function(err) {
                    return next(null, sheetData);
                });
            });
        });
    });

Any thoughts?

I dug into this more. The getRows method (or somewhere within) halts when it finds a row that is empty. Since my sheet has empty rows (to help format visually), this breaks. Looking into it and may have a PR soon.

Blergh.

Hi,
the property rowCount in SpreadsheetWorksheet object provides an whole row count in a google spread sheet. In our sheet we have only 300 odd rows populated.
I need to get rows length of where data is present. Is there any way to achieve it?


[WIP] Prefix lm add BOT token to separate input and target

This doesn't seem to change the loss, which suggests there just might not be any impact at all.


Windows 8.1 Instalation failed

Description
When attempting to install chef-dk it fails when extracting files
ChefDK Version
The chef installer version is 2.4.17-1-x64
Platform Version
Windows 8.1 Enterprise
Replication Case
Download the installer and run, setup behaves normally until it actually begins the installation and gets to the "Extracting files" bit where it then hangs for about 30 seconds before displaying an installation failed error.
Stacktrace
chefdk-install.log
The problem seems to be around this area
Starting FastUnzip
FASTZIPDIR = C:\opscode
FASTZIPAPPNAME = chefdk
Starting extraction
Exception thrown by custom action:
System.Reflection.TargetInvocationException: Exception has been thrown by the target of an invocation. ---> System.UnauthorizedAccessException: Access to the path 'C:\opscode\chefdk\embedded\lib\ruby\gems\2.4.0\gems\thor-0.19.1\spec\fixtures\bundle\main.thor' is denied.
at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)
at System.IO.FileStream.Init(String path, FileMode mode, FileAccess access, Int32 rights, Boolean useRights, FileShare share, Int32 bufferSize, FileOptions options, SECURITY_ATTRIBUTES secAttrs, String msgPath, Boolean bFromProxy, Boolean useLongPath, Boolean checkHost)
at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize)
at ICSharpCode.SharpZipLib.Zip.FastZip.ExtractFileEntry(ZipEntry entry, String targetName)
at ICSharpCode.SharpZipLib.Zip.FastZip.ExtractEntry(ZipEntry entry)
at ICSharpCode.SharpZipLib.Zip.FastZip.ExtractZip(Stream inputStream, String targetDirectory, Overwrite overwrite, ConfirmOverwriteDelegate confirmDelegate, String fileFilter, String directoryFilter, Boolean restoreDateTime, Boolean isStreamOwner)
at CustomActionFastMsi.CustomActions.FastUnzip(Session session)
--- End of inner exception stack trace ---
at System.RuntimeMethodHandle.InvokeMethod(Object target, Object arguments, Signature sig, Boolean constructor)
at System.Reflection.RuntimeMethodInfo.UnsafeInvokeInternal(Object obj, Object parameters, Object arguments)
at System.Reflection.RuntimeMethodInfo.Invoke(Object obj, BindingFlags invokeAttr, Binder binder, Object parameters, CultureInfo culture)
at Microsoft.Deployment.WindowsInstaller.CustomActionProxy.InvokeCustomAction(Int32 sessionHandle, String entryPoint, IntPtr remotingDelegatePtr)``

Do you have permission to install other applications? It says access denied

I do, so that was the bit that threw me, it has extracted a lot of files into C:\opscode and then for some reason decided it couldn't do the particular file mentioned.


Add more variables

Is it possible to add more variables?
If yes, then which letters/characters should not be used?
Very interesting code you have here,
Thanks

It's possible to do this by using the setValidVariables() method. Please see the examples of the section 4 (Test valid variables) in tests/FormulaParserTest.php
Any letter other than "e" can be used as a variable. The letter "e" is always processed by the parser as a constant = 2.718.
Thank you!


Adding maven Snapshots repo for SDK

Signed-off-by: Sarat Vemulapalli<EMAIL_ADDRESS>Description
Adding snapshots maven publish to publishing tasks.
Publishing tasks
----------------
generateMetadataFileForMavenJavaPublication - Generates the Gradle metadata file for publication 'mavenJava'.
generatePomFileForMavenJavaPublication - Generates the Maven POM file for publication 'mavenJava'.
publish - Publishes all publications produced by this project.
publishAllPublicationsToSnapshotsRepository - Publishes all Maven publications produced by this project to the Snapshots repository.
publishMavenJavaPublicationToMavenLocal - Publishes Maven publication 'mavenJava' to the local Maven repository.
publishMavenJavaPublicationToSnapshotsRepository - Publishes Maven publication 'mavenJava' to Maven repository 'Snapshots'.
publishToMavenLocal - Publishes all Maven publications produced by this project to the local Maven cache.

Issues Resolved
Part of #92
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

cc: @gaiksaya

Codecov Report

Merging #311 (10bfd37) into main (7ceebb7) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main     #311   +/-   ##
=========================================
  Coverage     72.97%   72.97%
  Complexity      120      120
=========================================
  Files            25       25
  Lines           518      518
  Branches         15       15
=========================================
  Hits            378      378
  Misses          131      131
  Partials          9        9

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Posting the query here as well. I see 2 different group id in build.gradle
https://github.com/opensearch-project/opensearch-sdk-java/blob/10bfd3784a3bb249c74099fea92c41851b2207a0/build.gradle#L39-L46
> Task :publishMavenJavaPublicationToSnapshotsRepository FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':publishMavenJavaPublicationToSnapshotsRepository'.
> Failed to publish publication 'mavenJava' to repository 'Snapshots'
   > Could not PUT 'https://aws.oss.sonatype.org/content/repositories/snapshots/org/opensearch/opensearch-sdk-java/1.0.0-SNAPSHOT/maven-metadata.xml'. Received status code 401 from server: Unauthorized


Add note on Docker Official Images

Add a new note about distributing Docker Official Images.
Also contains a minor style update

@nebuk89 Could you PTAL?

@sudo-bmitch Discussed this internally and removed the clause on distributing images that are created using a Docker Official Image. Publicly mirroring Docker Official Images without an agreement with Docker would still constitute a violation of Docker ToS.
Hope that clarifies your question.


No space before colon for requireAlignedObjectValues if disallowSpaceAfterObjectKeys is set

Hi,
I'm wondering is there a way to implement such behavior. Looks like this change would require to pass current configuration (or just a list of rules) to the rule's configure method. Is it a bad thing to do?
Example configuration:
"requireAlignedObjectValues": "all"

Invalid
var x = {
    a : 1,
    bcd : 2,
    ef : 'str'
};

Without "disallowSpaceAfterObjectKeys": true (current behavior):
Valid
var x = {
    a   : 1,
    bcd : 2,
    ef  : 'str'
};

With "disallowSpaceAfterObjectKeys": true:
Valid
var x = {
    a  : 1,
    bcd: 2,
    ef : 'str'
};

Here's the relevant piece of code from the requireAlignedObjectValues rule:
errors.assert.spacesBetween({
    token: keyToken,
    nextToken: colon,
    // + 1 is responsible for this extra space
    exactly: maxKeyEndPos - keyToken.loc.end.column + 1,
    message: 'Alignment required'
});

The reasoning behind that may seem to be a local issue: I'm trying to configure JSCS plugin to work with built-in Intellij IDEA formatter and there's no option to allow space before the colon only for multiline-aligned objects and not inline.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/19199190-no-space-before-colon-for-requirealignedobjectvalues-if-disallowspaceafterobjectkeys-is-set?utm_campaign=plugin&utm_content=tracker%2F281640&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F281640&utm_medium=issues&utm_source=github).

I think it's better for most if not all rules to be independent of each other.
I'm not sure I understand what you're asking for - do you just want the last code snippet when both rules are true? Although maybe we could just change requireAlignedObjectValues to have that last version by default (a more stricter check)?

Thanks for contributing! Yes, it's possible for these two rules to exist together by having the requireAligned rule to only verify alignment and not location of the colon.

@mikesherov , @hzoo
Please see my attempt to solve this under #1428.


Added support to memcached

Added support to Memcached cache system;
Fixed some typos on FileCache.php

Coverage remained the same at 100.0% when pulling 1bb4f005d97a9f2397b62e8089b14c78d64c8b97 on williamokano:master into 7017c61c1c51e3c07b379827051a1abe5686b57c on nielsenmg:master.

Coverage remained the same at 100.0% when pulling 5869202b3cb7610eb9c35afdf43ad0f8ae983ba5 on williamokano:master into 7017c61c1c51e3c07b379827051a1abe5686b57c on nielsenmg:master.

Coverage remained the same at 100.0% when pulling a855d87acf9221e4d5eaf8e3b4ed9b4add578943 on williamokano:master into 7017c61c1c51e3c07b379827051a1abe5686b57c on nielsenmg:master.


Index out of bounds error when trying to init viewer

I'm attempting to run the example scripts, and have created an environment on my local machine that has everything the viewer needs. However I get the following error when attempting to load any atlas:
Traceback (most recent call last):
  File "/home/berk/mambaforge/envs/iblviewer/lib/python3.9/threading.py", line 973, in _bootstrap_inner
    self.run()
  File "/home/berk/mambaforge/envs/iblviewer/lib/python3.9/threading.py", line 910, in run
    self._target(*self._args, **self._kwargs)
  File "/home/berk/Documents/iblviewer/iblviewer/qt_application.py", line 139, in _initialize_viewer
    self.viewer.initialize(plot=self.plot, dark_mode=self.dark_mode, **self.kwargs)
  File "/home/berk/Documents/iblviewer/iblviewer/mouse_brain.py", line 444, in initialize
    self.ibl_model.get_atlas_model(mapping)
  File "/home/berk/Documents/iblviewer/iblviewer/mouse_brain.py", line 191, in get_atlas_model
    self.atlas_volume.data = self.get_mapped_volume(self.atlas.label, atlas_mapping)
  File "/home/berk/Documents/iblviewer/iblviewer/mouse_brain.py", line 350, in get_mapped_volume
    volume_data = self.atlas.regions.mappings[map_id][volume]
IndexError: index 312782632 is out of bounds for axis 0 with size 2655

Have tried reinstalling ibllib, the viewer itself, and a new environment with no solution. Not sure how this could happen, as the error occurs only on my home machine while the work machine (on ubuntu with the exact same environments I think) works fine.

Hi Berk, I've tested installing iblviewer on new conda envs, one with python 3.8 and another with python 3.9. Both work for me. However, in the error message, it seems that you're running iblviewer from a cloned repo, not from a pip install.
When working with IBL data, the viewer loads the Allen atlas, computes the mapping and stores it locally in the flatiron cache (defined in the related IBL config file). If you have multiple iblviewer installed, they all rely on the same cache. Could you try to clear the cache and run the viewer again? It will take some time rebuilding the cache when you load it for the first time.

Yep this was indeed the issue, had to dig around to find the files and delete them but it worked afterwards.


rsync command not found with 23.11 nixos image

nixos-anywhere fails at copy extra file step using as installation target a machine booted with nixos 23.11 minimal iso image:
https://channels.nixos.org/nixos-23.11/latest-nixos-minimal-x86_64-linux.iso
nixos-anywhere command:
nix run github:numtide/nixos-anywhere -- --extra-files "$temp" --flake .#server02<EMAIL_ADDRESS>This is the error:
+ echo '### Copying extra files ###'
### Copying extra files ###
+ rsync -rlpv -FF -e 'ssh -i "/tmp/tmp.eYUCcg96wX"/nixos-anywhere -o UserKnownHostsFile=/dev/null -o StrictHostKeyChecking=no ' /tmp/tmp.Ol4VOcuUb6/ root@<IP_ADDRESS>:/mnt/
Warning: Permanently added '<IP_ADDRESS>' (ED25519) to the list of known hosts.
bash: line 1: rsync: command not found
rsync: connection unexpectedly closed (0 bytes received so far) [sender]
rsync error: error in rsync protocol data stream (code 12) at io.c(231) [sender=3.2.7]
+ rm -rf /tmp/tmp.eYUCcg96wX

Using the 23.05 image installation works without any issue.

We should probably start making our own ISOs that have all needed tools.

Is there a workaround?

Another work around is building your own installer. nixos-generators works well for that.

maybe we can do a fallback with nix-shell?

Editing /etc/nixos/configuration.nix and adding rsync to systemPackages with a nixos-rebuild test also works on current minimal iso.
  environment.systemPackages = with pkgs; [
    rsync
  ];

hmm, but thats a lot of hassle. Maybe we should just port the rsync code to scp?

We also create a new nixos-anywhere optimized installer image that has rsync installed and enables sshd by default: https://github.com/nix-community/nixos-images?tab=readme-ov-file#iso-installer-images

We also create a new nixos-anywhere optimized installer image that has rsync installed and enables sshd by default: nix-community/nixos-images#iso-installer-images

I saw this a few days ago and could not for the life of me figure out where the ISOs are actually hosted. I'm new to nix and figure I must misunderstand how it works.

To anyone who is looking at this, #316 (or #325) has been merged and it solves the problem.


Fix: Pundit adapter blindly ignores existing pundit_user definition

As per the pundit readme, there are instances where I need to provide additional context about the user. I do this by defining pundit_user, however the pundit rails_admin adapter ignores this and replaces it with it's own pointing at current_user. This breaks policies that rely on the additional context.
This patch addresses this.
In addition to this I have another issue with the pundit adapter, and I'd like to include some fixes for it as well, but before just doing them I'd like to see if there's a good reason why things have been done the way they are.
Typically the method name for checking with pundit for "can I do X" will be suffixed with a "?". Currently the pundit adapter does not do this.
I see 2 problems with this:

In scenario #1, it means aliasing a bunch of methods (i.e. alias_method :index, :index?) - a minor incovenience
In scenario #2, I may want different permissions for rails_admin than my standard application policies, for some reason. Currently I'm able to do this because it calls different methods (those without a question mark), but it's not particularly clear that these are specifically for rails_admin. This has confused a junior developer here already.

I'd like to put a patch together that basically either prefixes the methods with rails_admin, or works more like sudosu/rails_admin_pundit where it runs through a specific method, which in turn can use a case statement.
I'd appreciate any input on that before I write a patch and start depending on it.

For the scenario 1, that's my fault. I'll change all pundit calls to be suffixed with ?.
For the scenario 2, the recommended way is to set custom authorization key through action configuration.
https://github.com/sferik/rails_admin/pull/2399#discussion_r39257553

Thanks for the pointer, @mshibuya. Much appreciated <3


Use https-scheme for request URIs if secure TLS is used

$request->getUri()->getScheme() will now return https if secure TLS is used and http otherwise.
Builds on top of #146 and #158

Updated to work around PHP < 7 not giving access to stream crypto parameters :shipit:


When in high contrast mode and navigating with the keyboard the focused tab is barely visible

Describe the bug

In High Contrast mode, the keyboard-focused NavigationView item is barely discernible from non-focused NavigationView items.
Steps to reproduce the bug

Steps to reproduce the behavior:

Turn on HC
Open Xaml Controls Gallery, press Tab to move keyboard focus
Use arrow keys to continue moving focus within the list of items

Expected behavior

It's easy to see which item has keyboard focus.
Screenshots

(focus is currently on "Collections")
Proposed fix
Remove the borders around NavigationViewItems. They don't seem necessary (MenuFlyoutItems don't draw borders in HC) and detract rather than improve the user experience.
Additional context

Copied from an internal bug

Agree with @yulikl's proposed fix.

I would be happy to fix this :)


Error in R script in single-reference model

After I installed the virtual environment of pseudoR by conda, I removed the -3 option and directly called the pseudoR_single-reference.sh script.
The command is as follows, but it resulted in an error, seemingly within the R script. Below are the command, the generated files and log files.
Could you help me identify where the problem might be?
source pseudoR_single-reference.sh -s list.strain -d  /home/user/software/pseudoR -c reads -m s  -r ref.fasta -t 8  -1  '.read1.fq.gz' -2 '.read2.fq.gz'
Thank you.
log.txt
result.tree.txt

Hi! Thanks for trying out pseudoR and I'm sorry about the errors.. Can you possibly attach C8092.unmapped.contig_mapping.filtered.sam, C8092_pre_mapping_output.csv, and list.strain ? Thanks!

Thank you for your response！
Attached, please find the document you mentioned. I hope it proves helpful.
AttachedFile.zip

Thanks for these! I've gone through the files you provided and I unfortunately think the reason you are getting this error is that there aren't any strongly supported IS insertions in this sample. There are a number of trimmed reads that were remapped with 3' terminus of ISOSDB6215 or ISOSDB4244. However, there were no trimmed reads with the 5' terminus of either of these IS elements. To ensure that IS insertions are bonafide, only loci where trimmed reads with the both 3' and 5' termini (some reads have 3' termini and some reads have 5' termini) are reported as insertions. During my development of the program, I found that reporting loci where only one type of termini mapped lead to spurious results. I will update the repo at some point in near future to make this explained as an error in program. Thanks again for trying pseudoR and let me know if there is anything else I can help with.

Thank you for your response.
It appears that indeed my current sample didn’t detect the IS insertions. I tried another sample and was able to detect them. This tool is designed to detect genomic IS insertions based on metagenomic data.
Could you please provide insight into whether it’s possible to enumerate the types and quantities of IS elements within the metagenomic or illumina data? This is a matter of particular concern for me.
Thank you！

Glad it worked! This tool is designed primarily to identify sites where both an un-inserted allele and an IS inserted allele exist. For instance, this tool can find a novel, unidentified IS insertion in a gene that was previously assembled without an IS insertion. In the tool's output, you will be given the IS identified (ISOSDBXXX) and the likely IS family. The ISOSDB number is unique identifier for that IS element. You can profile these further if you want by pulling out the sequence of the IS element from the ISOSDB database. Let me know if there is anything else I can help with!

Thank you for your response and advices！


Enable support for AWS LB

The (non-AWS-specific) change here is that we pass []*api.ServicePort into the LoadBalancer provider.
We then use this to implement AWS Load Balancers.  AWS needs the full ServicePort because we need to see the NodePort for a LoadBalanced service on AWS, because we can't match on source IP, because ELB (apparently) works by relaying TCP connections.
This also paves the way for e.g. UDP load balancing.

I saw that had accidentally been using this branch as my integration branch.  I cleaned it up - should hopefully be much easier to review now!  Thanks & sorry!

@justinsb looks like you have merge conflicts now. I'm excited to see this merged soon!

needs rebase, but LGTM otherwise

Rebased and looking good on my e2e testing on AWS!


linker: add _image_ram_size symbol to linker symbol definitions

Add _image_ram_size symbol to linker definitions. This symbol
corresponds to the total RAM used by the image.
This PR also adds a usage of _image_ram_size. The reason this symbol is introduced is because GCC will not compile the expression ((uint32_t)&_image_ram_end) - ((uint32_t)&_image_ram_start), with the error error: initializer element is not constant.

This PR in particular is harmless, being just an optimization on top of an existing API. But I don't really like the API, FWIW. Zephyr apps that can assume they run in a contiguous region are becoming a smaller and smaller subset of the platform list. Seems like the framework needs to grow to keep up.

This is a good point- the specific use case this PR enables is executing from a RAM region on the iMX RT series. I think zephyr_code_relocate is probably part of what we want to enable an application to have something like a "base" region where the system boots from, and then relocate large portions of the code to different memory regions


Json ahdoc

This PR improves the speed of JSON parsing with the string_parser module. The new version of parse_json_string is heavily inspired by Python's implementation, available here
Time is usually either as performing as before or better, Valgrind also gives the new version an advantage, sometimes significant.
On large_escaped (~120M, lots of escaped characters), we have:

Before: 13 748 436 458 Ir
After: 11 687 104 643 Ir

i.e. an improvement of ~18%
Time is:

Before: 0m4.428s
After: 0m3.932s

i.e. an improvement of ~13%
Note that on this particular test, although the number of allocations is limited as much as we can, Boehm still cannibalizes the runtime, seeing that the real time is ~2.6s.
Perf gives a total time of ~38% in GC_mark and ~7% in GC_cache_miss.
Some more optimizations can be thought, but this shows how important a good GC is necessary for the future, this program can be a good metric of how GC usage can be improved.
For further reference, Python3 on the same test gives the following output on time and Valgrind:

Time (user): 0m1.760s
Time (real): 0m2.177s
Valgrind: 8 845 966 867 Ir

Are you better than go now?

Most definitely yes.
We're closer than ever to Python 2 and with a bit of optimization (I suspect some work could be done on our HashMap, especially considering the high number of MapNode objects spawned on this test), Python 3 could be within our reach.
Here are the last results from the json benchmarks:

+1 I do not get why Python is still so efficient. But it is an improvement.

Reroll: reduced the number of allocations of Buffer in parse_json_string and made append_substring more robust.
With the new optimizations, we're doing better than Python 2 as proven by the following graph:

nice. +1


[Feature] Adding proper answers to the FAOs in the homepage. [GSSoC'23]

Description
The FAOs present in the home page don't have appropriate answers. I can add proper answers to those FAQs.
Screenshots

Additional Information
Please assign me this issue under GSSoC'23

Please assign me this issue under GSSoC'23.

hey @1Shubham7 this issue has already been stated check #136

Ok @himanshu1221 . I got it.


trace_call API

Signed-off-by: Daniel Lehrner<EMAIL_ADDRESS>PR description
trace_call API: Executes the given call and returns a number of possible traces for it.
Fixed Issue(s)
fixes #2885

[x] I thought about the changelog and included a changelog update if required.

Hey @daniellehrner  I looked at the two tests trace_call_12_1_trace and trace_call_12_1_vmTrace today, pushed a fix for the trace one and I'll look into the vmTrace one tomorrow, looks like the revert reason is not part of our vmTrace body so we might need to change the code a bit more.

LGTM, other than some unit test fallout


Add documentation for v1.3 features

Codecov Report

Merging #203 (233c325) into master (f94c558) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #203   +/-   ##
=========================================
  Coverage     67.11%   67.11%
  Complexity      886      886
=========================================
  Files           139      139
  Lines          3065     3065
  Branches        339      339
=========================================
  Hits           2057     2057
  Misses          864      864
  Partials        144      144

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 f94c558...233c325. Read the comment docs.


Cut Inside First - Cut optimization feature.

Cut inside first needs to be performed on cut sequences. Even without a more fully fleshed out cut planner.

Looking at the requirements for that optimization though it's not too hard. Take the primary points and some basic linearization of shapes in all contiguous loops of the cut shapes and check whether all the points of that loop is found completely within another contiguous loop.

This issue was corrected in 0.6.0. Which is now in full release.


New Installs Broken (Debian Buster & Docker i386 Drop)

Two issues prevent a successful build/deploy:
1
With Debian Buster being left behind as the old stable, Raspberry Pi OS fails all apt-get update, including ones that the ansible playbook needs to complete successfully.
sudo apt-get update --allow-releaseinfo-change
In the scripts would correct this.
See: https://beta.raspberrypi.org/forums/viewtopic.php?f=66&t=318302&sid=ba1f80edd81c3a18bbb63d1c52e511c1&start=50
2

TASK [Run Docker install convenience script.] fails

fatal: [<IP_ADDRESS>]: FAILED! => {"changed": true, "cmd": ["/tmp/get-docker.sh"], "delta": "0:00:04.121966", "end": "2021-10-19 22:57:45.918384", "msg": "non-zero return code", "rc": 100, "start": "2021-10-19 22:57:41.796418", "stderr": "+ sh -c apt-get update -qq >/dev/null\n+ sh -c DEBIAN_FRONTEND=noninteractive apt-get install -y -qq apt-transport-https ca-certificates curl >/dev/null\n+ sh -c curl -fsSL \"https://download.docker.com/linux/debian/gpg\" | gpg --dearmor --yes -o /usr/share/keyrings/docker-archive-keyring.gpg\n+ sh -c echo \"deb [arch=i386 signed-by=/usr/share/keyrings/docker-archive-keyring.gpg] https://download.docker.com/linux/debian buster stable\" > /etc/apt/sources.list.d/docker.list\n+ sh -c apt-get update -qq >/dev/null\n+ sh -c DEBIAN_FRONTEND=noninteractive apt-get install -y -qq --no-install-recommends  docker-ce-cli docker-scan-plugin docker-ce >/dev/null\nE: Unable to locate package docker-ce-cli\nE: Unable to locate package docker-scan-plugin\nE: Package 'docker-ce' has no installation candidate", "stderr_lines": ["+ sh -c apt-get update -qq >/dev/null", "+ sh -c DEBIAN_FRONTEND=noninteractive apt-get install -y -qq apt-transport-https ca-certificates curl >/dev/null", "+ sh -c curl -fsSL \"https://download.docker.com/linux/debian/gpg\" | gpg --dearmor --yes -o /usr/share/keyrings/docker-archive-keyring.gpg", "+ sh -c echo \"deb [arch=i386 signed-by=/usr/share/keyrings/docker-archive-keyring.gpg] https://download.docker.com/linux/debian buster stable\" > /etc/apt/sources.list.d/docker.list", "+ sh -c apt-get update -qq >/dev/null", "+ sh -c DEBIAN_FRONTEND=noninteractive apt-get install -y -qq --no-install-recommends  docker-ce-cli docker-scan-plugin docker-ce >/dev/null", "E: Unable to locate package docker-ce-cli", "E: Unable to locate package docker-scan-plugin", "E: Package 'docker-ce' has no installation candidate"], "stdout": "# Executing docker install script, commit: 93d2499759296ac1f9c510605fef85052a2c32be", "stdout_lines": ["# Executing docker install script, commit: 93d2499759296ac1f9c510605fef85052a2c32be"]}

If we re-run apt-get update we can see what's wrong:
pi@raspberry:~/internet-pi $ sudo apt-get update
Hit:1 http://security.debian.org buster/updates InRelease
Hit:2 https://download.docker.com/linux/debian buster InRelease
Hit:3 http://ftp.debian.org/debian buster InRelease
Hit:4 http://archive.raspberrypi.org/debian buster InRelease
Hit:5 http://ftp.debian.org/debian buster-updates InRelease
Reading package lists... Done
N: Skipping acquire of configured file 'stable/binary-i386/Packages' as repository 'https://download.docker.com/linux/debian buster InRelease' doesn't support architecture 'i386'

It seems debian buster i386 is dropped for docker repo now.

doesn't support architecture 'i386'

It seems like something, somewhere, is trying to force an install of i386... that's definitely not what the Pi should be reporting.


⚠ Avatar API Endpoint has degraded performance

In 06ebfd6, Avatar API Endpoint (https://avatar.roblox.com/v1/avatar-rules) experienced degraded performance:

HTTP code: 200
Response time: 1065 ms

Resolved: Avatar API Endpoint performance has improved in 4955639 after 7 minutes.


Build and publish API documentation with docfx

Description

Add docfx config
Add workflow to build and publish to GitHub pages
Fix malformed XML docs in code

Example: https://xtansia.github.io/opensearch-net/api/OpenSearch.Client.html
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@VachaShah do you want to review this?

@dblock & @Yury-Fridlyand Would you be able to review this please?


Can you make an alternative, pure python version of Tiktoken? (no Rust dependency)

Can you make an alternative, pure python version of Tiktoken, with no Rust dependency?
For those who cannot compile and run Rust binaries on their system (for various reasons: package managers support, company policy, intranet or local machine security, docking containers limitations, vm restrictions, environment virtualization, lack of Rust support in jupyter notebooks remote hosting, etc).
Of course it would be slower than the Rust version, but as a fallback it would allow Tiktoken to be used in a much wider set of platforms and working environments.
Thanks!

@hauntsaninja ?

I have no current plans to support this.

I'm closing the issue as per^. Note that the latest release of tiktoken includes aarch64 wheels, in case that was the issue you were running into.

@Emasoft I want to actually run tiktoken in code interpreter but it isn't available. so I had to tar up the git repos and run it by adding it to sys path. This approach won't work due to the rust binary. And it looks like one of case you need a pure python impl. I don't know RUST but GPT4 did a pretty good job helping me translating it (plus a few minor debugging and re-trials), you can consult:
https://github.com/kechan/tiktoken.git
I added the option to Encoding class, and you will have to instantiate it a different way to invoke the pure python tokenizing. Or you can hard code and hack it up.
I tested quite a few symbols and sentences, and it seems ok. (GPT4 is awesome).

@kechan This is great! Thank you! I'll try this asap.

@Emasoft I used it like this locally (but there may be easier/better way):
import tiktoken.registry as registry
from tiktoken.registry import _find_constructors
from tiktoken.core import Encoding
_find_constructors()
constructor = registry.ENCODING_CONSTRUCTORS['cl100k_base']
params = constructor()
enc = Encoding(**params, use_pure_python=True)
enc.encode("hello world")
For code interpreter, I sort of pickle the params (a dict), upload it and then feed in **params like above, and able to call the encoder. it is quite handy for experimentation, so I didn't do exhaustive test. if you find any issue, let me know. Likely will enlist gpt4's RUST skill to fix.

@kechan Can you publish the pure Python wheel in your repo? I struggled to build it.

@MRYingLEE Instead of trying to build or pip install, just tar up the tiktoken repository, untar it on the target machine (with limited ability or restriction), and use
sys.path.append(os.path.join(extract_path, 'tiktoken'))
to add it to your path. On the more capable machine with normal install, run the code I posted to get & pickle the params for cl100k_base, and use that same file on the target machine.
in short, I don't have time to separate out the rust dependency from setup.py and build process, I just use sys.path.append the location where I untar the repos. maybe I will come around later. Hope this helps.

Note that the educational submodule in tiktoken contains pure Python code for using and training BPE encodings. https://github.com/openai/tiktoken/blob/main/tiktoken/_educational.py
It has an underscore prefix since I don't want to promise any API stability, but could be useful for folks finding this issue

Must be a simpler way to do this? Count tokens without rust?

Note that the educational submodule in tiktoken contains pure Python code for using and training BPE encodings. https://github.com/openai/tiktoken/blob/main/tiktoken/_educational.py

@hauntsaninja  Since the pure python code version it's already available, why do not add to the pip setup the option to install the pure python version without Rust dependencies on platforms without Rust installed? More and more people are in need of such version, for a variety of reasons. Please! 🙏


Remove bluepill support

I no longer use bluepill, and I'd like to clean up the additional complexity supporting a third process supervision suite brings to this cookbook.

Done in 2.0


[core] Don't download monorepo packages

Causes false positives in docs type-checking and manipulates download stats. We want downloads to approximate usage. But we're not using the downloaded packages.

I might be missing something, will these changes be reverted the next time we run yarn release:version?

I might be missing something. Will these changes be reverted the next time we run yarn release:version?

I hope not. Will keep an eye on it.