get_nearest_node function with a list of values

Hi,
Is there a way to use get_nearest_node function for a list of values?
The current syntax is get_nearest_node(graph, point, return_dist=False).
How can I modify the points variables to something like a list or tuple of values (say [point1, point2,....] where point1 = (latitude, longitude)] to avoid looping over all points.

Thanks for using OSMnx! Please re-read the contributing guidelines and the new issue template you saw when you opened this issue.
As the template noted, your issue will be automatically closed if you didn't provide all of the information requested in the template, because we need all of this information to address your issue.
Also note: per the contributing guidelines, please don't use the issue tracker to ask "how-to" or usage questions. If asked here, they will be closed automatically. Instead, ask "how-to" and usage questions on StackOverflow.


Two Sum in JavaScript

Information about Algorithm

Given an array of integers nums and an integer target, return indices of the two numbers such that they add up to target.
You may assume that each input would have exactly one solution, and you may not use the same element twice.
You can return the answer in any order.
Have you read the Contributing.md and Code of conduct

[x] Yes
[ ] No

Other context

This is a competitive platform problem from leetcode

I want to solve this issue and contribute in it

i want to work on this plz assign to me


Upgrade terraform-provider-gitlab to v3.9.1

Fixes #144
Fixes #143

@stack72 Part of this PR was to update to V2. Could you give me some tips on resolving the breaking changes?


Allow marking node unschedulable in case of kubelet pushing node updates to master

kube-controller-manager has '--sync_node_status' flag and when this is set,
node controller probes kubelet for updates and when this is unset(default),
kubelet pushes updates to master. Ability to mark node unschedulable works
in the former case but is broken in the latter case. This change will fix the issue.
Record an event on node ready or schedulable changes
Related change: pull #6155

Covered under Red Hat corporate CLA

Assigning to @davidopp, feel free to re-assign or rebalance.
@pravisankar looks like missing gofmt btw.

@gmarek

Thanks for doing this. Looks basically fine (didn't check the tests yet). Could you please also change the scheduler so it checks unschedulable condition from spec rather than status (I dont't think it's good to require a round-trip to the node before we enact operator change to schedulable condition for a node). Or to be more precise, scheduler should require !spec.unschedulable && nodeready==true, in order to schedule on a node.
Also, please mention "closes #6077" in the commit description.

Hey, so on second thought and after discussion with @yujuhong I'm not sure this PR is a good idea. Maybe we should just remove NodeSchedulable condition from NodeStatus. (And make scheduler use schedulable condition from Spec.) Yu-Ju pointed out that in #6165 we're removing Host from PodStatus because it is already in Spec. That redundancy elimination is somewhat analogous to removing NodeSchedulable from NodeStatus.
In #6077 @bgrant0607 mentioned two reasons for leaving NodeSchedulable in NodeStatus:
1. To represent an ACK from Kubelet, since it's the last line of defense.
2. To enable clients to just look at NodeStatus and not look at NodeSpec when determining properties they need to reason about, such as schedulability.

I'm not sure either of these are compelling. For (1), I'm fine with Kubelet rejecting new pods if node is not schedulable, but it can just look at schedulable from the Spec for this--there's no reason it needs to be reflected back in the NodeStatus. With respect to (2), I don't think it's really that hard for clients to look at both Spec and Status. The client is going to be fetching the whole Node object anyway, so having it look at both Spec and Status seems only trivially harder than having it look at just Status.
In sum, I think removing NodeSchedulable condition from NodeStatus would simplify the system and not lose anything important.

I'm fine with @davidopp's proposal.

Closing this pull request based on feedback from @davidopp , @bgrant0607

FYI the simplified version of this feature was implemented in #6411. Thanks for your contribution, and sorry we didn't think through the requirements thoroughly enough before filing the initial issue.


Fix for error in un-compressing file loaded using XMLHttpRequest

Scenario:
I am using js-unzip in a project with js-epub and booktorious. I am loading the epub files using an XMLHttpRequest. I am developing for a mobile platform hence can not do Base64 decoding on the client end as it is fairly slow.
Problem:
If I use Base64 encoding then there is no problem in un-compressing the file. However when I load an .epub file using a simple  XMLHttpRequest and then pass it to js-epub it fails during "Post processing" phase in the function JSEpub.uncompressNextCompressedFile
Example of Fail Case:
code used to load .epub file downloaded from http://www.epubbooks.com using XMLHttpRequest
//serving the file from local server to avoid cross domain error.
var uri = "epubs/finn.epub";
var req = new XMLHttpRequest();
req.onreadystatechange = function () {
  if (req.readyState == 4 && req.status == 200) {

 //from booktorious
  ebookReader.create(ebookReader.loading).initialize(req.responseText);

  } else if (req.readyState == 4 && req.status < 400 && req.status > 299) {
    console.log('error finding file!');
  } else if (req.readyState == 4) {
    console.log('cross domain loading error');
  }
};
req.overrideMimeType('text/plain; charset=x-user-defined');
req.open("GET", uri);
req.send(null);

Solution:
Used what you suggested in the comment for getByteAt: function on line 101. I think using "& 0xFF" should be the default case as it would not be so obvious to everyone.
Without the fix people will runs into this problem while loading a file using XMLHttpRequest that is not Base64 encoded. And those who do not need this can opt out.

Hi, and thanks for the pull request! I've gotten numerous reports about this issue but I'm yet to see a unit test that reproduces the issue. After some mucking about, I pushed 7b1a4f8eb6c865017baec8b18a215140da334eca which hopefully resolves this issue. It's now aware of multibyte strings. Simply masking out the bits means data loss, which the test case in that commit also proves. Making the loop aware of multibyte strings seems to work fine, though. Please let me know if this fixes your problems.


FutureWarning: The default matplotlib

Environment

Qiskit version: 0.46

What is happening?
C:\Users\Steve\PycharmProjects\pythonProject4.venv\Scripts\python.exe C:\Users\Steve\PycharmProjects\pythonProject4\main.py
C:\Users\Steve\PycharmProjects\pythonProject4.venv\Lib\site-packages\qiskit\visualization\circuit\matplotlib.py:266: FutureWarning: The default matplotlib drawer scheme will be changed to "iqp" in a following release. To silence this warning, specify the current default explicitly as style="clifford", or the new default as style="iqp".
self._style, def_font_ratio = load_style(self._style)
Process finished with exit code 0
How can we reproduce the issue?
from qiskit.circuit.library import MCXGate
gate = MCXGate(4)
from qiskit import QuantumCircuit
circuit = QuantumCircuit(5)
circuit.append(gate, [0, 1, 4, 2, 3])
circuit.draw('mpl')
What should happen?
It should not die here \matplotlib.py:266
Any suggestions?
Let me know how to fix the problem

The warning message explains what is happening and how to fix/suppress the warning.  If you require the old style in QuantumCircuit.draw, specify a style='clifford' argument, or style='iqp' if you're happy to use the new style (which will become the default in Qiskit 1.0).
The code should not be "dying", it's just issuing a warning.

Qiskit 1.0 is currently expected to be released on the 15th of February.  In the mean time, you could install qiskit<0.46, use one of the remedies suggested in the message or using a Qiskit configuration file, filter the warning out using the standard Python tools, or just ignore it.
I'm going to close this issue as "resolved", but feel free to re-open if there's more that needs discussing.

Qiskit 1.0 is currently expected to be released on the 15th of February.
do I need to uninstall Qiskit and re-install it?

We will be recommending that you create a new virtual environment to install Qiskit 1.0 in, rather than trying to upgrade in-place, once it is released.

It will be just as easy to pip uninstall Qiskit and then pip install qiskit matplotlib pylatexenc
Regards,
Steve Beaver
@.***
Office :<PHONE_NUMBER>
Cell      :<PHONE_NUMBER>
No man is a true believer unless he desires for his brother that, what he desires for himself.

This electronic mail (including any attachments) may contain information that is privileged, confidential, and/or otherwise protected from disclosure to anyone other than its intended recipient(s). Any dissemination or use of this electronic email or its contents (including any attachments) by persons other than the intended recipient(s) is strictly prohibited. If you have received this message in error, please notify us immediately by reply email so that we may correct our internal records. Please then delete the original message (including any attachments) in its entirety. Thank you
From: Jake Lishman @.***
Sent: Friday, February 9, 2024 8:24 AM
To: Qiskit/qiskit
Cc: Steve Beaver; Author
Subject: Re: [Qiskit/qiskit] FutureWarning: The default matplotlib (Issue #11762)
We will be recommending that you create a new virtual environment to install Qiskit 1.0 in, rather than trying to upgrade in-place, once it is released.
—
Reply to this email directly, view it on GitHub https://github.com/Qiskit/qiskit/issues/11762#issuecomment-1936026074 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AR3TRYQ7HVMEW4IVUMH7ZFTYSYWQLAVCNFSM6AAAAABDBQKG6GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMZWGAZDMMBXGQ .
You are receiving this because you authored the thread.  https://github.com/notifications/beacon/AR3TRYQJHQ3W6MSETWIMF53YSYWQLA5CNFSM6AAAAABDBQKG6GWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTTTMVU5U.gif Message ID: @.***>


refactor: move range command

[ ] I am sure that the code is update-to-date with the dev branch.

close #

🉑

A test failed.


"Apply config" -> Error: Did not receive valid response from server

Whenever I click "Apply config" I get the following error,
Error: Did not receive valid response from server
XHR response code: 504 XHR responseText: undefined jQuery status: error
The error is when reaching /admin/ajax.php?command=reload.
It gets a 504 error.

I tracked it down to:
libraries/BMO/Framework.class.php
public function doReload($passthru=false) {
The following codes simply hangs and does not continue:

            $process = new \Symfony\Component\Process\Process($AMPSBIN.'/fwconsole reload --json');
            $process->setTimeout(1800);
            $process->run();

Alot of times we have been seeing these upstream errors from Freepbx - I believe I am going to need to switch to a different method and start pinning stable releases to this image as oposed to just pulling from their git repositories.
I am moving and will be away for a week, once I return I will look into this.

Hi @barhom - I cannot recreate on a fresh install. My test scenario involved updating all modules and adding an extension.

cant reproduce my problem anymore. closing the issue.

@tiredofit actually I was able to reproduce it if I used tag "15-3.3.1". Works good if I dont specify a tag.

Hi Guys, I installed a fresh new FreePBX <IP_ADDRESS>
But then still got this issue.
How to fix this, why there FreePBX always having this issue on all of its new version?


Logging Improvement

Fixes Issue
Closes #7437

Changes proposed

Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[ ] I have updated the documentation accordingly.
[ ] My change requires changes to the documentation.
[X] All new and existing tests passed.
[X] This PR does not contain plagiarized content.
[X] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

@akash19coder :  I was trying to test your logger code but it doesn't seem to be working for me. It is throwing an exception for the log file. Have a look at the screenshot.

Looks ok to me.

Looks ok to me.

we didn't implement the file appender functionality as Vercel doesn't support it.

Broke production with the following error...
2023-07-01T02:59:03.343Z	undefined	INFO	- info Loaded env from /var/task/.env
The server environment variables are valid!
Error: unable to determine transport target for "pino-pretty"
    at fixTarget (/var/task/node_modules/pino/lib/transport.js:140:13)
    at transport (/var/task/node_modules/pino/lib/transport.js:114:22)
    at normalizeArgs (/var/task/node_modules/pino/lib/tools.js:311:16)
    at Function.pino (/var/task/node_modules/pino/pino.js:87:28)
    at 3640 (/var/task/.next/server/chunks/3640.js:39:44)
    at __webpack_require__ (/var/task/.next/server/webpack-api-runtime.js:25:42)
    at 2251 (/var/task/.next/server/chunks/6164.js:14:65)
    at __webpack_require__ (/var/task/.next/server/webpack-api-runtime.js:25:42)
    at 6164 (/var/task/.next/server/chunks/6164.js:63:71)
    at __webpack_require__ (/var/task/.next/server/webpack-api-runtime.js:25:42)
RequestId: 58215cbd-ea22-40b5-b6b6-b4f397737a32 Error: Runtime exited with error: exit status 1
Runtime.ExitError

Runs locally fine in development and production mode

I couldn't fix in preview, so I have reverted this PR https://github.com/EddieHubCommunity/LinkFree/pull/7894
A new PR will need to be raised

I am not sure why it didn't work in production


feat(Storage): Add retry decider function for GCS

7 errors and 2 failures in total for the conformance test.
There were 7 errors:

Google\Cloud\Storage\Tests\Unit\RetryConformanceTest::testOps with data set #1 ('storage.bucket_acl.list', null, array('return-503', 'return-503'), array('BUCKET'), true, false, 0)
Google\Cloud\Core\Exception\NotFoundException: {"error":{"code":404,"message":"{"error": {"errors": [{"domain": "global", "message": "Bucket my-bucket does not exist."}]}}"}}

Google\Cloud\Storage\Tests\Unit\RetryConformanceTest::testOps with data set #26 ('storage.bucket_acl.list', null, array('return-reset-connection', 'return-reset-connection'), array('BUCKET'), true, false, 0)
Google\Cloud\Core\Exception\NotFoundException: {"error":{"code":404,"message":"{"error": {"errors": [{"domain": "global", "message": "Bucket my-bucket does not exist."}]}}"}}

Google\Cloud\Storage\Tests\Unit\RetryConformanceTest::testOps with data set #51 ('storage.bucket_acl.list', null, array('return-reset-connection', 'return-503'), array('BUCKET'), true, false, 0)
Google\Cloud\Core\Exception\NotFoundException: {"error":{"code":404,"message":"{"error": {"errors": [{"domain": "global", "message": "Bucket my-bucket does not exist."}]}}"}}

Google\Cloud\Storage\Tests\Unit\RetryConformanceTest::testOps with data set #92 ('storage.objects.insert', null, array('return-reset-connection', 'return-reset-connection'), array('BUCKET'), true, true, 0)
Google\Cloud\Core\Exception\ServiceException: Connection refused for URI http://localhost:9000/upload/storage/v1/b/bucket-63999747ede6d/o?uploadType=multipart

Google\Cloud\Storage\Tests\Unit\RetryConformanceTest::testOps with data set #93 ('storage.objects.insert', null, array('return-reset-connection', 'return-reset-connection'), array('BUCKET'), true, true, 1)
Google\Cloud\Core\Exception\ServiceException: Connection refused for URI http://localhost:9000/upload/storage/v1/b/bucket-63999747f1106/o?uploadType=multipart

Google\Cloud\Storage\Tests\Unit\RetryConformanceTest::testOps with data set #103 ('storage.objects.insert', null, array('return-reset-connection', 'return-503'), array('BUCKET'), true, true, 0)
Google\Cloud\Core\Exception\ServiceException: Connection refused for URI http://localhost:9000/upload/storage/v1/b/bucket-6399976446ce2/o?uploadType=multipart

Google\Cloud\Storage\Tests\Unit\RetryConformanceTest::testOps with data set #104 ('storage.objects.insert', null, array('return-reset-connection', 'return-503'), array('BUCKET'), true, true, 1)
Google\Cloud\Core\Exception\ServiceException: Connection refused for URI http://localhost:9000/upload/storage/v1/b/bucket-6399976449b8c/o?uploadType=multipart

There were 2 failures:

Google\Cloud\Storage\Tests\Unit\RetryConformanceTest::testOps with data set #114 ('storage.objects.insert', null, array('return-503'), array('BUCKET'), false, false, 0)
Failed asserting that exception of type "Exception" is thrown.

Google\Cloud\Storage\Tests\Unit\RetryConformanceTest::testOps with data set #115 ('storage.objects.insert', null, array('return-503'), array('BUCKET'), false, false, 1)
Failed asserting that exception of type "Exception" is thrown.

ERRORS!
Tests: 318, Assertions: 294, Errors: 7, Failures: 2, Skipped: 17.

@dwsupplee  Could you please tell me which constructor you are talking about?
I have checked the StroageClient constructor and the available arguments are the following:
/**
 * Create a Storage client.
 *
 * @param array $config [optional] {
 *     Configuration options.
 *
 *     @type string $apiEndpoint The hostname with optional port to use in
 *           place of the default service endpoint. Example:
 *           `foobar.com` or `foobar.com:1234`.
 *     @type string $projectId The project ID from the Google Developer's
 *           Console.
 *     @type CacheItemPoolInterface $authCache A cache used storing access
 *           tokens. **Defaults to** a simple in memory implementation.
 *     @type array $authCacheOptions Cache configuration options.
 *     @type callable $authHttpHandler A handler used to deliver Psr7
 *           requests specifically for authentication.
 *     @type FetchAuthTokenInterface $credentialsFetcher A credentials
 *           fetcher instance.
 *     @type callable $httpHandler A handler used to deliver Psr7 requests.
 *           Only valid for requests sent over REST.
 *     @type array $keyFile The contents of the service account credentials
 *           .json file retrieved from the Google Developer's Console.
 *           Ex: `json_decode(file_get_contents($path), true)`.
 *     @type string $keyFilePath The full path to your service account
 *           credentials .json file retrieved from the Google Developers
 *           Console.
 *     @type float $requestTimeout Seconds to wait before timing out the
 *           request. **Defaults to** `0` with REST and `60` with gRPC.
 *     @type int $retries Number of retries for a failed request.
 *           **Defaults to** `3`.
 *     @type array $scopes Scopes to be used for the request.
 *     @type string $quotaProject Specifies a user project to bill for
 *           access charges associated with the request.
 * }
 */
 ```

 Please let me know what am I missing here?

 Other comments has been resolved, thank you!

Resolved all the comments, Thank you!


Allow SVG to be treated as an image or XML doc in ImageViewer, with extension to override

Discovered while working on https://github.com/mozilla/thimble.webmaker.org/pull/654#issuecomment-118167503 -- an SVG opens as an XML file in the editor vs. an image.  For our users, they will mostly want to see SVG as an image.  This defaults to SVG-as-Image, and allows you to override by including the optional SVGasXML extension.

r+

Merged with 8e204d52b6df018930f29cb21bcd642f1dce93ca


Imp-Launcher - Update DL from GH not S3

Change the imp-launcher to download the latest binaries from GH instead of s3

imp launcher 0.2.1 now pulls from GH as opposed to s3


passing locale through NextIntlProvider

Hey guys
I'm using next-intl for integrating multi langs in my site but i faced a little issue which is having the locale from the route
because i need it to determine what language i want to show
this is my data type  {     "home": {     "ar": "الرئيسية",     "en": "Home"     },     "donate": {     "ar": "تبرع",     "en": "donate"     } }
as you see u have [en - ar] inside the object
so how im solving this is in every compenent i want to show translation i import the t method useTranslations and locale from useRouter like so
`const t = useTranslations();
const { locale } = useRouter();
return (

{t(`home.${locale}`)}

)
`
so i dont want to import locale on every component that i want to translate so is there any way i can extract the locale from the
NextIntlProvider ??

next-intl intends to split messages by locale. This allows for smaller payloads, as you only load the messages for the locale that the user really needs.
If this is not an option for you, you could think about wrapping the useTranslations hook to apply the locale automatically when the t function is called.

I'll close this due to inactivity. I hope my previous message helps you.


Fixed broken link in settings docs

Should be merged to 1.8 branch too.

merged in ca51c559154fe11008c6b59d908e1c957c30cac6, thanks.


fix: Remove MDCTextfield Constant External

Remove MDCTextfield Constant External from webpack.config.js
fixes #802

Codecov Report

Merging #803 into rc0.12.0 will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           rc0.12.0     #803   +/-   ##
=========================================
  Coverage     95.14%   95.14%
=========================================
  Files            73       73
  Lines          2924     2924
  Branches        449      449
=========================================
  Hits           2782     2782
  Misses           46       46
  Partials         96       96

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 242203a...72c3b1f. Read the comment docs.


Do the benchmark on a more realistic schema

The current benchmark is run on a subset of the tests in the official JSON-schema test suite.
Although the results seem very much in line with what other stable benchmarks produce (like themis), and exercising as much of the standard as possible is good to catch any slow corners in a validator. It might be a good idea to do a benchmark on data that is more like what you see in production.
Perhaps an performance benchmark on one simple and one advanced schema.
To keep things simple for anyone wanting to chose a validator, the benchmark should still produce one single number for each validator like it does now. Nitty gritty details about performance are for schema validator authors to do in their efforts.

I was thinking of machine-generating a bigger schema from the schemas in the official tests, to make sure that the test touches as many parts of the spec as possible. Just to make sure to hit any troublesome parts of a validator.

ok, so the bad news is I don't have as much time as I'd like, but the JSCK benchmarks use five schemas, and you're more than welcome to use them.
here's a categorization of those 5 schemas:

draft 3, trivial
draft 3, medium
draft 4, trivial
draft 4, medium
draft 4, complex

so you can use them to benchmark a range of complexities and draft support.

Advanced benchmark idea: Validating the draft 4 schema with the draft 4 schema.

@ebdrup The more I think about it, the more I believe that using test-suite for performance benchmarking is completely wrong.
With the exception of some really old and slow libraries, where slow validation may affect UX in the browser, validation performance only really matters for server-side validation. This is where the performance advantage achieved by compiling validators is becoming important... Server-side validation passes 99+% of the time (excluding the cases of DOS attacks and or broken API consumer - but performance is not going to help too much in these cases).
So I think to be relevant, benchmark should only test passing cases on some data samples of small/average/large size.
Tests can still be used to benchmark compliance, separately from performance.
What do you think?


feat: add customData property to OAuthRedirectConfiguration

https://magiclink.atlassian.net/browse/M2PB-403
https://www.notion.so/magiclabs/Decentraland-OAuth-v2-dynamic-redirect-URIs-132ad65b2b7080e5b349f0584e90c7b8

📦 Published PR as canary version: Canary Versions

:sparkles: Test out this PR locally via:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

:rocket: PR was released in<EMAIL_ADDRESS>:rocket:


Migrate ActivateRelations

AntV Open Source Contribution Plan（可选/Optional）

[x] 我同意将这个 Issue 参与 OSCP 计划 / I agree to participate in the OSCP plan

Issue 类型 / Issue Type
初级任务 / Primary Task
任务介绍 / Task Description
迁移激活相邻关系交互 Demo

Migrate Activate Relations behavior Demo
参考说明 / Reference Description

⚠ 如果您额外实现了：高亮相邻边的能力，我们会将这个任务类型提升至中级任务

您可以参考旧代码并考虑复用其中的部分逻辑。但请注意以下几点：

需要调整配置项的命名，以保持与当前交互命名风格的一致性。
根据您的理解，您可以对部分逻辑进行修改或优化。
不要迁移 History 相关的逻辑（例如：graph.startHistoryBatch）

在正式开始之前，建议您简要了解新版交互实现的逻辑和配置项命名风格，例如：Drag Canvas和Drag Element。
在迁移过程中，如果遇到任何问题，欢迎在Issues区留言寻求帮助。

⚠ If you add the ability to: 'Highlight adjacent edges', we will raise this task type to' Intermediate Task '

You are encouraged to consult the old code and consider reusing some of its logic. However, please keep in mind the following points:

The naming of configuration items needs to be adjusted to remain consistent with the current interaction naming conventions.
Based on your understanding, you are welcome to modify or enhance parts of the logic as you see fit.
Do not migrate History related logic (e.g. graph.startHistoryBatch)

Before you begin, it is advisable to familiarize yourself briefly with the logic and naming conventions of configuration items used in the newer versions of interaction implementations, such as: Drag Canvas and Drag Element.
Should any issues arise during the migration process, you are welcome to seek assistance by posting a message in the Issues section.

认领~


Vulkan Abstraction - Change from VkWaitIdle to VkFence

Overview
Currently our vulkan abstraction layer uses vkQueueWaitIdle. Might be better for using VkFence for various reasons from what I've learned. This is something that I would eventually want to look into.
Reasons

Wait on completion of work without having to wait on work submission after the work you are waiting on.
VkFence allows other threads to push command buffers to queue without interfering with wait.
Flexibility on how command buffers get submitted to the render pass when dealing with multiple threads to the renderer thread.

Stack Overflow - Here is the link for more information https://stackoverflow.com/questions/39537176/in-what-situations-is-vkfence-better-than-vkqueuewaitidle-for-vkqueuesubmit

For right now im closing this because it has already been added as part of synchronization integrations into the Vulkan abstraction layer/
VkFence is a much finer grained solution then vkQueueWaitIdle. Since VkFence allows to wait on tasks without having them needing to be submitted first.
Allowing flexibility in how we wait for specific threads to finish with tasks. Which can also allow your CPU to be open for doing other tasks while finishing it's current task prior to submission.


Bump kubebuilder to 3.15.0 (with merge conflicts resolved)

Contains the changes from the below PR along with merge conflicts resolved.
Ref: https://github.com/operator-framework/ansible-operator-plugins/pull/77

@acornett21 / @joelanford can you help fix the test failures?

@Neo2308 We can probably drop this right? Since we need to go to kubebuilder v4 to get k8s 1.30, correct?

@acornett21 we could drop this, but since this already has some of the major changes that are needed when moving to kubebuilder v4, it would be easier to get this merged and make the extra changes on top of it.

I'm pretty sure @Neo2308's responses to the review questions are correct, but my memory is fuzzy. I had some conversations with @camilamacedo86 in the upstream kubebuilder channel after I made my original PR and I think further changes came in 3.15.1 and 4.x releases.
To be honest, I've long since evicted this from my cache, and I don't have time right now to load all this back in.

I think further changes came in 3.15.1 and 4.x releases.

@joelanford further changes for sure, we are trying to layer them on in smaller chunks. Thanks for the reply.


refactor: change some attributes of class "Commit" to lazy loading

Some attributes: The attributes other than "name" and "commit_id".

Task linked: CU-2jact3q 将 commit_id + name 作为 Commit 生成的必要条件，其他属性设置为懒加载

commit type should be feat


Refs #27808 -- Added test for Nested ArrayField with nullable base field.

Test for Ticket 27808

Do we want to pin psycopg2-binary>=2.7.5 given that this was an issue fixed in Psycopg2?

@felixxm, @pope1ni Thanks for the review.
Don't know PSYCOPG2_VERSION checking is good enough or not.

@hramezani Thanks :+1: I pushed minor edits.


[IOTDB-1678] client-cpp session bug: can cause connection leak.

Description
[IOTDB-1678] client-cpp session bug: can cause connection leak.

Coverage decreased (-0.006%) to 67.369% when pulling 48a90222da400cd5bade008158f7f1e4fc8c1274 on jamber001:IOTDB-1678 into 184fd6fc4176987bb8bdb1426644f00d126f0ed6 on apache:master.


null-safety migration

Adding null-safety to allow sound null safety when using this package.
Solves #41.

Hi @tejainece, are you the best person to contact for reviewing this PR?

Waiting for this. Please give us an update.

Thanks for the PR. null safety enabled duration package is now published to pub.


terraform null-resource provider

Is there an easy way to enable additional terraform providers e.g. null-resource?

@EikeHens If you use null provider in your Terraform configuration, the provider will be enabled by default when the Terraform Controller pod is running. Could you please give more specific scenarios?


Improve the config validation

properly end the scalarNode so that the indentation of ->end() calls is not confusing
mark the node as required, so that the code assuming that $config['app'] is configured is safe.

Thanks stof!


How to train target datasets, such as UAV123 ?

Thanks

Thanks for your attention. We don't recommend you train on UAV123 because compared with LaSOT and other datasets, it is smaller. If you want to do, you can use the par_crop.py to crop the dataset before training. After that, it will produce the JSON files via gen_json.py. Those .py files are located at train_dataset.


Add a warning text to explain that icon & splash screen cannot be edited

Add a warning text or a reminder to explain that the icon & splash screen cannot be edited after the app submission to the stores.

done 👍


UpdateRepo API hangs on POST request

The API for POST /api/repos/ hangs infinity on requests with valid payload: URL-encoded form with repo and config parameters.
Initialize reported as https://github.com/artipie/artipie/issues/896
Reproduced with integration test: https://github.com/artipie/artipie/pull/897

@rultor release, tag=0.4.3

@rultor release, tag=0.4.3

@olenagerasimova OK, I will release it now. Please check the progress here

@rultor release, tag=0.4.3

@olenagerasimova Done! FYI, the full log is here (took me 8min)

@rultor release, tag=0.4.4

@rultor release, tag=0.4.4

@olenagerasimova OK, I will release it now. Please check the progress here

@rultor release, tag=0.4.4

@olenagerasimova Done! FYI, the full log is here (took me 8min)


⚠ PrivateHD has degraded performance

In 5722c01, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 1912 ms

Resolved: PrivateHD performance has improved in fabf2a8.


JustText

For those who do not want any text formatting except new lines

@jaytaylor, it looks like this would close #52. Is there a reason it hasn't been merged yet?

I need this feature too, hopefully this could be merged. :)
btw, is it better to rename JustText to TextOnly?

TextOnly definitely makes more sense.

@copoer Happy to work together to get this merged!  Are you able to (pretty please) squash it into a single commit?

Are you able to (pretty please) squash it into a single commit?

GitHub offers a button to squash the commands and merge, that means you can do it directly by one click. :)

This is true, but would this lead to a commit history polluted with WIP?  OP - please clean up and squash the commits so we can merge this baby

This is true, but would this lead to a commit history polluted with WIP?

No. Only one commit after merged. :)

By the way, could you tag a new version after merged this PR?
It would be great to use Go module too, see #42

Beautiful!  Thank you @copoer !!!!!

By the way, could you tag a new version after merged this PR? It would be great to use Go module too, see #42

@iredmail I'm on it, haven't created tags for my go libraries before but let's give it a shot.  I will follow up in #42.


Validation of arguments to prevent unexpected behaviour

For example, if I run:
iohyve start

I run into:
# iohyve start
Usage: grep [OPTION]... PATTERN [FILE]...
Try `grep --help' for more information.
Usage: grep [OPTION]... PATTERN [FILE]...
Try `grep --help' for more information.
[: -: unexpected operator
Starting ... (Takes 15 seconds for FreeBSD guests)
Usage: grep [OPTION]... PATTERN [FILE]...

After this the console becomes unresponsive.
Advice (when I have the time I'll look into it) is to validate if the argument contains valid data.

Hi,
started that already, but I am currently waiting for my PR #142  to be merged until I continue working on it.

To me it looks like this can simply be build into the __parse_cmd function. I'll add it to the current version, later on merge it into the lib separated version if it will be merged. Personally I don't mind the one-file solution like it works now.

Yeah it looks like @moogle19 had already done some work on this previously. I'm going to close this issue for now, and we can discuss in the PR once #142 get's merged.
Again, thanks for your input!


docs(crashlytics): Crashlytics docs added

Added the information on how to set up crashlytics for iOS
Added the crashlytics docs link to Readme

Thanks AhsanAyaz for documenting this. Is there anyway to automate this pods installation? I'm pretty sure I'm going to forget to do it at some point in a release.

@AhsanAyaz why was this closed and not merged? Is it not the correct thing to do? I also have the same problem you outlined it https://github.com/arnesson/cordova-plugin-firebase/issues/924


Gcc 7 1 0 / Qt 5.8 fixes

Description
The following are small fixes caught by the GCC 7.1.0 compiler when used in combination with Qt 5.8.
As it stands there are still a number of warnings being emitted, but their scope was much greater than my drive-by edit had time to fix.
Checklist:

[x] I have read the contribution guidelines.
[x] If this is more extensive than a small change to existing code, I
have previously submitted a Contributor License Agreement
(individual, and if there is any way my
employers might think my programming belongs to them, then also
corporate).
[ ] I have updated the documentation, if applicable.
[ ] I have ensured that the change is tested somewhere in the testsuite
(adding new test cases if necessary).
[x] My code follows the prevailing code style of this project.

LGTM. As soon as I try it at work to make sure nothing breaks there, I will merge it.

There are actually a whole bunch more compiler warnings from GCC 7.1.0, though I'm not able to address them all at the moment.

Do you mean you don't have time to address them, or you don't know HOW to address them? If the latter, feel free to just post the error messages, and I may be able to make a good guess about how to fix the warnings.
In the mean time, I'll see if I can set up Travis to add a gcc7 compile to the mix, that'll get things cleared up fast.

These changes seem fine both on my laptop and work machine, so I will merge now. Thanks for the fixes!


Fix for #650

If the where clause is on the same line as the impl, { is put on a new line. Fixes issue #650

Thanks for the PR! I think that, given this comment, the way to address this is to always put the where clause on a newline unless the whole impl fits on one line (i.e., if the body is empty).
E.g.,
impl<T> ToOwned for T where T: Clone {}

impl<T> ToOwned for T
    where T: Clone
{
    fn foo() {}
}

@nrc alright, I will see if I can do that! Thanks for taking a look!

I just pushed a new commit, it now prefers to put where on a new line.
The one exception is if impl_empty_single_line is enabled, the impl is empty, and the where clause is a single line, the whole impl is put into a single line.

looks good thanks! r+ with the minor refactoring

I refactored the if clause into a separate function, and I squashed all the commits.

Sweet! Thanks for the PR!


Iso

Code: http://pasterack.org/pastes/28373
Pasterack doesn't display the plot for some reason, so here it is:

thank you


FrontEnd: DNS & Working of DNS?

Description 📜
I want to work on making documentation of DNS & Working of DNS with some DNS basics and how its a role in the web.

Domain of Contribution 📊

[x] Frontend Dev (HTML, CSS, JS)

/assign

I want to contribute to this issue : FrontEnd: DNS & Working of DNS?
Name-Arya Gupta
Domain-Frontend with HTML CSS
Batch-16
Serial No-3172
Issue Number-1343
Type-Audio
Contribution Approaches-What is DNS, full form of DNS, how does it work, relation of DNS with web development

I want to work in the documentation part


Show cumulative sum data in area charts

Describe the feature you'd like to request
The GitHub stars (among other KPIs) should display a cumulative sum chart

Describe the solution you'd like
Precomputed data in data provider
Describe alternatives you've considered
na

Completed with the following code snipped:
(() => {
  let counter = 0;
  const result = [];
  for (const x of $ctx.eventData.data) {
    counter += x.Starred;
    result.push({
      ...x,
      cumulative: counter
    });
  }
  return result;
})()


Show more detailed error message when a partner user attempts to login to the public site

QA Notes: Attempt to login to the public site with a partner user with MFA enabled. If you create a new user, by default they should have MFA enabled.

lgtm!


Should this library check for document.startViewTransition before doing anything else?

https://github.com/paoloricciuti/sveltekit-view-transition/blob/c04966d1b0555e05e267b300e81ba79628bf16fd/src/lib/sveltekit-view-transition.ts#L337-L340

Mmm it might be a good idea...I'm trying to think if there's a downside to it but I can't seem to find one...I'll let the night bright my reason and eventually fix that tomorrow

Closed by #32


Prefer brewed Clang over gcc-6 for needs :openmp

[?] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[x] Have you successfully run brew tests with your changes locally?

Clang has fully implemented OpenMP support as of LLVM 3.7, so if OpenMP
is required by a formula gcc is no longer the only choice of compiler.
Clang should be preferred over gcc because using gcc meant linking
against libstdc++, which is ABI incompatible with libc++. This may be
unnoticeable for some users, but it causes other builds to fail, e.g.
pstoedit when imagemagick was built with OpenMP. pstoedit is required
for the octave formula, so for some users this could be a significant
problem.
Unfortunately, these changes appear to have broken the *.cc files used
by brew to log changes to the command emitted by Make/etc -- those files
only show changes for calls to ld.
In addition, the changes made to determine_cc are rather hacky, and
should be replaced with a cleaner solution.

This is a work in progress. Hopefully will have more commits coming in in the next few days.

Just to let you know: I'm broadly in favour of this change but it'll take a while to figure out the best approach.

Alright, changed the strategy up a bit. Hope it's better.
brew install imagemagick --with-openmp actually gets past the configuration stage now; the previous issue with a missing ltdl.h appears to have gone away on its own. The build still fails, however, with linker issues.
Gist here
For example, this part of the build log looks a bit suspicious:
Undefined symbols for architecture x86_64:
  "std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >::copy(char*, unsigned long, unsigned long) const", referenced from:

Something to do with the standard library not being found? Or the wrong one being used? Don't remember what libc++ uses as its namespace vs libstdc++...

And imagemagick --with-openmp now builds. No idea what I changed to fix things.

CC @mistydemeo any naming thoughts here?

And pstoedit builds, so at least one problem is fixed

Something to check out for later: brew install fftw --with-fortran --with-mpi --with-openmp fails:
configure:13302: checking for mpicc
configure:13329: result: /usr/local/bin/mpicc
configure:13347: checking for MPI_Init
configure:13347: /usr/local/bin/mpicc -o conftest -g -O2   conftest.c  >&5

Failed to execute brew_clang -pipe -Os -o conftest -g -O2 conftest.c -I/usr/local/Cellar/open-mpi/2.0.1/include -L/usr/local/opt/libevent/lib -L/usr/local/Cellar/open-mpi/2.0.1/lib -lmpi -isystem/usr/local/include -isystem/usr/include/libxml2 -isystem/System/Library/Frameworks/OpenGL.framework/Versions/Current/Headers -L/usr/local/lib -L/System/Library/Frameworks/OpenGL.framework/Versions/Current/Libraries -Wl,-headerpad_max_install_names

Xcode and/or the CLT appear to be misconfigured. Try one or both of the following:
  xcodebuild -license
  sudo xcode-select -switch /path/to/Xcode.app

configure:13347: $? = 1
configure: failed program was:
| /* confdefs.h */
| #define PACKAGE_NAME "fftw"
| #define PACKAGE_TARNAME "fftw"
| #define PACKAGE_VERSION "3.3.5"
| #define PACKAGE_STRING "fftw 3.3.5"
| #define PACKAGE_BUGREPORT<EMAIL_ADDRESS>| #define PACKAGE_URL ""
| #define PACKAGE "fftw"
| #define VERSION "3.3.5"
| #define FFTW_ENABLE_ALLOCA 1
| #define FFTW_SINGLE 1
| #define BENCHFFT_SINGLE 1
| #define HAVE_SSE2 1
| #define STDC_HEADERS 1
| #define HAVE_SYS_TYPES_H 1
| #define HAVE_SYS_STAT_H 1
| #define HAVE_STDLIB_H 1
| #define HAVE_STRING_H 1
| #define HAVE_MEMORY_H 1
| #define HAVE_STRINGS_H 1
| #define HAVE_INTTYPES_H 1
| #define HAVE_STDINT_H 1
| #define HAVE_UNISTD_H 1
| #define HAVE_DLFCN_H 1
| #define LT_OBJDIR ".libs/"
| /* end confdefs.h.  */
| /* Define MPI_Init to an innocuous variant, in case <limits.h> declares MPI_Init.
|    For example, HP-UX 11i <limits.h> declares gettimeofday.  */
| #define MPI_Init innocuous_MPI_Init
|
| /* System header to define __stub macros and hopefully few prototypes,
|     which can conflict with char MPI_Init (); below.
|     Prefer <limits.h> to <assert.h> if __STDC__ is defined, since
|     <limits.h> exists even on freestanding compilers.  */
|
| #ifdef __STDC__
| # include <limits.h>
| #else
| # include <assert.h>
| #endif
|
| #undef MPI_Init
|
| /* Override any GCC internal prototype to avoid an error.
|    Use char because int might match the return type of a GCC
|    builtin and then its argument prototype would still apply.  */
| #ifdef __cplusplus
| extern "C"
| #endif
| char MPI_Init ();
| /* The GNU C library defines this for functions which it implements
|     to always fail with ENOSYS.  Some functions are actually named
|     something starting with __ and the normal name is an alias.  */
| #if defined __stub_MPI_Init || defined __stub___MPI_Init
| choke me
| #endif
|
| int
| main ()
| {
| return MPI_Init ();
|   ;
|   return 0;
| }
configure:13347: result: no
configure:13354: checking for MPI_Init in -lmpi
configure:13379: /usr/local/bin/mpicc -o conftest -g -O2   conftest.c -lmpi   >&5

Failed to execute brew_clang -pipe -Os -o conftest -g -O2 conftest.c -lmpi -I/usr/local/Cellar/open-mpi/2.0.1/include -L/usr/local/opt/libevent/lib -L/usr/local/Cellar/open-mpi/2.0.1/lib -lmpi -isystem/usr/local/include -isystem/usr/include/libxml2 -isystem/System/Library/Frameworks/OpenGL.framework/Versions/Current/Headers -L/usr/local/lib -L/System/Library/Frameworks/OpenGL.framework/Versions/Current/Libraries -Wl,-headerpad_max_install_names

Xcode and/or the CLT appear to be misconfigured. Try one or both of the following:
  xcodebuild -license
  sudo xcode-select -switch /path/to/Xcode.app

configure:13379: $? = 1
configure: failed program was:
| /* confdefs.h */
| #define PACKAGE_NAME "fftw"
| #define PACKAGE_TARNAME "fftw"
| #define PACKAGE_VERSION "3.3.5"
| #define PACKAGE_STRING "fftw 3.3.5"
| #define PACKAGE_BUGREPORT<EMAIL_ADDRESS>| #define PACKAGE_URL ""
| #define PACKAGE "fftw"
| #define VERSION "3.3.5"
| #define FFTW_ENABLE_ALLOCA 1
| #define FFTW_SINGLE 1
| #define BENCHFFT_SINGLE 1
| #define HAVE_SSE2 1
| #define STDC_HEADERS 1
| #define HAVE_SYS_TYPES_H 1
| #define HAVE_SYS_STAT_H 1
| #define HAVE_STDLIB_H 1
| #define HAVE_STRING_H 1
| #define HAVE_MEMORY_H 1
| #define HAVE_STRINGS_H 1
| #define HAVE_INTTYPES_H 1
| #define HAVE_STDINT_H 1
| #define HAVE_UNISTD_H 1
| #define HAVE_DLFCN_H 1
| #define LT_OBJDIR ".libs/"
| /* end confdefs.h.  */
|
| /* Override any GCC internal prototype to avoid an error.
|    Use char because int might match the return type of a GCC
|    builtin and then its argument prototype would still apply.  */
| #ifdef __cplusplus
| extern "C"
| #endif
| char MPI_Init ();
| int
| main ()
| {
| return MPI_Init ();
|   ;
|   return 0;
| }
configure:13388: result: no
configure:13396: checking for MPI_Init in -lmpich
configure:13421: /usr/local/bin/mpicc -o conftest -g -O2   conftest.c -lmpich   >&5

Failed to execute brew_clang -pipe -Os -o conftest -g -O2 conftest.c -lmpich -I/usr/local/Cellar/open-mpi/2.0.1/include -L/usr/local/opt/libevent/lib -L/usr/local/Cellar/open-mpi/2.0.1/lib -lmpi -isystem/usr/local/include -isystem/usr/include/libxml2 -isystem/System/Library/Frameworks/OpenGL.framework/Versions/Current/Headers -L/usr/local/lib -L/System/Library/Frameworks/OpenGL.framework/Versions/Current/Libraries -Wl,-headerpad_max_install_names

Xcode and/or the CLT appear to be misconfigured. Try one or both of the following:
  xcodebuild -license
  sudo xcode-select -switch /path/to/Xcode.app

configure:13421: $? = 1
configure: failed program was:
| /* confdefs.h */
| #define PACKAGE_NAME "fftw"
| #define PACKAGE_TARNAME "fftw"
| #define PACKAGE_VERSION "3.3.5"
| #define PACKAGE_STRING "fftw 3.3.5"
| #define PACKAGE_BUGREPORT<EMAIL_ADDRESS>| #define PACKAGE_URL ""
| #define PACKAGE "fftw"
| #define VERSION "3.3.5"
| #define FFTW_ENABLE_ALLOCA 1
| #define FFTW_SINGLE 1
| #define BENCHFFT_SINGLE 1
| #define HAVE_SSE2 1
| #define STDC_HEADERS 1
| #define HAVE_SYS_TYPES_H 1
| #define HAVE_SYS_STAT_H 1
| #define HAVE_STDLIB_H 1
| #define HAVE_STRING_H 1
| #define HAVE_MEMORY_H 1
| #define HAVE_STRINGS_H 1
| #define HAVE_INTTYPES_H 1
| #define HAVE_STDINT_H 1
| #define HAVE_UNISTD_H 1
| #define HAVE_DLFCN_H 1
| #define LT_OBJDIR ".libs/"
| /* end confdefs.h.  */
|
| /* Override any GCC internal prototype to avoid an error.
|    Use char because int might match the return type of a GCC
|    builtin and then its argument prototype would still apply.  */
| #ifdef __cplusplus
| extern "C"
| #endif
| char MPI_Init ();
| int
| main ()
| {
| return MPI_Init ();
|   ;
|   return 0;
| }
configure:13430: result: no
configure:13469: error: could not find mpi library for --enable-mpi

openblas --with-openmp also fails, although this time it is OpenBLAS' own Makefile that is failing the build:
make
CC=brew_clang
FC=/usr/local/bin/gfortran
libs
netlib
shared

Makefile.system:409: *** OpenBLAS: Clang didn't support OpenMP yet..  Stop.

I wonder if mixing clang and gfortran would cause issues, too...

Fixed, I think.

@aw1621107 A few more comments. After that can you let me know what formulae you've tested this with? Thanks!

I think we may want homebrew science's opinions here. Since Linux builds with gcc not clang, switching the openmp default compiler may cause problems for their CI due to the divergence, where currently there is unity.

switching the openmp default compiler may cause problems for their CI due to the divergence, where currently there is unity.

Good point. Let's make this PR prioritise llvm_clang last for now to ease getting support in and merged sooner rather than later and then we can have an immediate follow-up PR which changes the default priority.

This looks good to me now. Will want to do some local testing with a few formulae and then hopefully we can get this shipped.

Alright, made llvm_clang lowest priority.
The formulae with OpenMP that I have installed using this patch are:

fftw
imagemagick
llvm (not sure if this counts?)
metis
openblas
pngquant
suitesparse
superlu

I don't use many of these directly, so I don't think I can really say anything besides that they compiled.

brew style failing: https://bot.brew.sh/job/Brew/2945/version=any_sierra/testReport/junit/brew-test-bot/sierra/style/. After that's fixed I'll test locally and if it looks good: merge.

Alright, pushed the style fixes and cleaned up the commit message a bit.
Just finished building the newest imagemagick locally, and it finishes compiling. However, the logs say that imagemagick was not built with OpenMP despite the presence of with-openmp. Seems the linker cannot find the OpenMP library. Pretty sure this didn't happen before, but it's something I need to fix regardless.

Ok, cool, will wait for that.

I think the issue is that gcc is linked by default, while llvm isn't. Thus, the linker can find libgomp just fine, but cannot find libomp. In addition, the imagemagick build actually uses OpenMP if brew link --force llvm is run.
Seems like this could be an issue. Linking formulae that use OpenMP against LLVM might not play well with all the other formulae that use Apple's LLVM, and trying to link just libomp with Apple LLVM doesn't sound like it'd work too well. Thoughts?

You'll probably need to make sure the relevant paths are added in superenv so they can link against libomp.

Linking formulae that use OpenMP against LLVM might not play well with all the other formulae that use Apple's LLVM

I think we'll need to accept or fix this or this whole concept is a non-starter, unfortunately.

You'll probably need to make sure the relevant paths are added in superenv so they can link against libomp.

If I'm understanding this correctly, superenv already has a mechanism to add library search paths for keg only dependencies, right? If so, would it be possible to make needs :openmp also add a dependency on LLVM? Or will depends_on "llvm" have to be added manually to the relevant formulae? Or is that not a good path to take?
In terms of LLVM libs that may conflict with Apple's LLVM libs, seems like that main culprits would be libc++, libLTO, and libunwind. Looks to me that the other ones are more compiler-specific libraries that most formulae wouldn't use. I'd hope that those are reasonably API/ABI stable across versions, so with any luck formulae shouldn't have too much of an issue...

Also, going through the OpenMP-using formulae I have installed:

fftw: Something weird broke and configure can't find MPI_Init for OpenMPI. Compiling with gcc also fails, saying "OpenMP enabled but not using an OpenMP compiler".
imagemagick: compiles fine
metis: compiles fine
openblas: Needs editing to Makefile.system to remove check for clang, and build fails in line 1849 kernel/x86_64/dgemm_kernel_4x8_sandy.S with "error: unexpected token at start of statement". The "unexpected token" is a series of #s used as comments, so not really sure what's going on. Build with gcc also fails (with linker errors), so looks like there's something up with my installation...
pngquant: compiles fine
suite-sparse: compiles fine
superlu: compiles fine

So I guess the change didn't break anything that wasn't already broken.

@BrewTestBot test this please

Thanks for your contribution to Homebrew!  You rock!

This PR broke homebrew/science/trinity with or without llvm installed.
g++-6 -DHAVE_CONFIG_H -I. -I..    -ansi -pedantic -fopenmp -Wall -Wextra -Wno-deprecated -m64 -I/Library/Java/JavaVirtualMachines/jdk1.8.0_92.jdk/Contents/Home/include -I/Library/Java/JavaVirtualMachines/jdk1.8.0_92.jdk/Contents/Home/include/darwin -fopenmp -MT IRKE.o -MD -MP -MF .deps/IRKE.Tpo -c -o IRKE.o IRKE.cpp
IRKE.cpp: In member function 'void IRKE::populate_Kmers_from_kmers(const string&)':
IRKE.cpp:89:62: error: 'omp_set_num_threads' was not declared in this scope
     omp_set_num_threads(IRKE_COMMON::MAX_THREADS_READ_PARSING); // place limit here because performance deteriorates beyond this value.
                                                              ^
IRKE.cpp: In member function 'void IRKE::compute_sequence_assemblies(KmerCounter&, float, unsigned int, unsigned int, bool, bool, std::__cxx11::string)':
IRKE.cpp:461:53: error: 'omp_set_num_threads' was not declared in this scope
         omp_set_num_threads(IRKE_COMMON::NUM_THREADS);
                                                     ^
IRKE.cpp:464:30: error: 'omp_set_num_threads' was not declared in this scope
         omp_set_num_threads(1); // turn off multithreading for the contig building.
                              ^
make[2]: *** [IRKE.o] Error 1
make[1]: *** [install-recursive] Error 1
make: *** [inchworm_target] Error 2
/usr/local/Homebrew/Library/Homebrew/debrew.rb:11:in `raise'

It also doesn't work as designed. Even if I remove the dependency on gcc and explicitly add a dependency on llvm, g++-6 is still used.

It also doesn't work as designed. Even if I remove the dependency on gcc and explicitly add a dependency on llvm, g++-6 is still used.

That is intentional. It's the lowest priority so should require being manually specified with --cc or fails_with every other compiler for now.

Ah OK. Then the bug is straightforward: it filters out fopenmp for anything that isn't llvm-clang now, including gcc.
Oh, and If I uninstall gcc, I get
Error: undefined method `null?' for "3.9.0":String
Please report this bug:
  https://git.io/brew-troubleshooting
/usr/local/Homebrew/Library/Homebrew/compilers.rb:132:in `block in find_compiler'
/usr/local/Homebrew/Library/Homebrew/compilers.rb:120:in `each'
/usr/local/Homebrew/Library/Homebrew/compilers.rb:120:in `find_compiler'
/usr/local/Homebrew/Library/Homebrew/compilers.rb:113:in `compiler'
/usr/local/Homebrew/Library/Homebrew/compilers.rb:96:in `select_for'
/usr/local/Homebrew/Library/Homebrew/extend/ENV/shared.rb:167:in `compiler'
/usr/local/Homebrew/Library/Homebrew/extend/ENV/super.rb:42:in `setup_build_environment'
/usr/local/Homebrew/Library/Homebrew/extend/os/mac/extend/ENV/super.rb:89:in `setup_build_environment'
/usr/local/Homebrew/Library/Homebrew/build.rb:85:in `install'
/usr/local/Homebrew/Library/Homebrew/build.rb:189:in `<main>'

With

https://gist.githubusercontent.com/ilovezfs/21a8649d5ba53a4e9dba1868989f91e0/raw/c6f3bbb5b33d97c5336d583dad8e6fd4e76b55b0/gistfile1.txt
gcc not installed
llvm installed
https://github.com/Homebrew/brew/pull/1546 pulled locally

I get
checking whether make sets $(MAKE)... yes
checking for C++ compiler default output file name...
configure: error: in `/private/tmp/trinity-20161121-33709-5fj2js/trinityrnaseq-Trinity-v2.3.2/Inchworm':
configure: error: C++ compiler cannot create executables
See `config.log' for more details.
make: *** [inchworm_target] Error 77
/usr/local/Homebrew/Library/Homebrew/debrew.rb:11:in `raise'
BuildError: Failed executing: make all

if I run
brew install -dvs trinity

@aw1621107 only way I've found to fix the above is to an explicit depends_on "llvm" to the trinity formula.

Then the bug is straightforward: it filters out fopenmp for anything that isn't llvm-clang now, including gcc.

Oops. That was quite the oversight.
@ilovezfs The error is due to the linker being unable to find OpenMP, because LLVM isn't linked by default. brew link --force llvm would also fix that.
I would guess that means that needs :openmp needs to also implicitly set depends_on "llvm", since linking LLVM isn't a great option, but not sure what else can be done. Thoughts?

@aw1621107 I think the best solution would be for OpenMP's library to be added by superenv if needs :openmp is used.

@MikeMcQuaid Sounds good to me! Only thing is that LLVM's libomp is in the same directory as everything else... Would creating a temp directory with only a symlink to libomp work?

Yes, something like that sounds sensible assuming you want to avoid e.g. libc++.


Python 3.11 support

Is there any reason that Python 3.11 isn't yet supported? I was trying to get a simple cognee example up and running but it's surprising to me that it only supports up to Python 3.10.

Unstructured library which we use asked for python 3.10
I will check if we can remove it as a dependency


distinguish derived variables coming from same version

Currently we're treating all variables that can be potentially calculated in the same way. This forces user to have to check all of them but we could assume that the ones that have exactly same version and frequency are correct. And should be listed with the perfect matches.
The ones which aren't fully matching should only be shown if not already included in perfect match list, this would avoid useless repetition.

This has been done by

removing requirement of calculation =='' for first selection, rather we can exclude based on mappings having more than 1 input_vars.
This brings lots of the perfect matches to be listed together with or without calculation.
The others that need more than 1 input vars are still dealt with separately. This time we return again any mapping containing the variable but we require more than one input variable
then we check if version and frequency are the same as the mapping we're considering and if yes we add it to a separate list.
finally we write this list right after the perfect matches


Building multiple trees

For a library I am making I would like to build a tree for the library distribution and another tree with modules defined purely for testing the library. Example modules used purely for testing are DS.Model, Ember.Object and Ember.Controller instances that are used to test the library API surface. Is there any documentation on how to build a library tree as well as a  tree for testing with Broccoli?
This is how I currently use Broccoli:

https://github.com/digitalplaywright/ember-declarative-authorization

An example of a module I'd define purely for testing (will define more once I know how to do this):

https://github.com/digitalplaywright/ember-declarative-authorization/blob/master/test/rules/rules.js

This can be accomplished 2 ways:

by utilizing broccoli as a library and having two separate build pipelines.
by creating a broccoli-plugin, that explicitly writes to two separate directories.

I don't think this is an issue with broccoli, although some community plugin can likely make this much more ergonomic.


WIP Upgrade to LibCImGui

start generating function wrappers for cimplot v0.9

Call me if you need something fixed outside generator code.

@sairus7 the wrappers are fixed up so that they load, but the demo is broken because it looks like CImGui has been restructured on the development branch. Since I think you've been using the latest versions can you try to update the Renderer script and see if you can get the demo running?
The generated wrappers should be very close to what we need. If something is busted ping me--it's probably just a small edit here or there that's needed. For the moment the methods from implot_internal.h are filtered out because I'll need some special casing to handle it and I want to focus on getting the main API and demo working first.

How should I deal with warning like this?
WARNING: both ImPlot and LibCImGui export "ImPlotPoint"; uses of it in module MyImPlot must be qualified?
For now I just add module name.

Looks like every name libcimplot should be replace to libcimgui in generator. I've replaced it manually for now. Also fixed some api function names.
Demo working until Drag and Drop, except of Queries:
no method matching unsafe_convert(::Type{Ptr{ImPlotLimits}}, ::ImPlotLimits)

Looks like every name libcimplot should be replace to libcimgui in generator. I've replaced it manually for now. Also fixed some api function names.

Ah yeah. It was hardcoded in the generator script. When we're sure it's all working I need to once again refactor generator.jl to remove redundant code blocks and fix things like that.

Demo working until Drag and Drop, except of Queries:
no method matching unsafe_convert(::Type{Ptr{ImPlotLimits}}, ::ImPlotLimits)

Awesome!

Also, since we dropped CImGui from ImPlot env, to run examples you need to activate separate env in the demo folder, and have dev both ImPlot and CImGui in your system.

@sairus7 removed the submodule (and Manifest.toml for good measure). Does that fix things for you? Any reason to not just merge this to main? We can decide to tag it at a later time if needed?

Yes, thanks, now I can just add CImGui#master ImPlot#upgrade-v0.9, and it works.
I agree we should merge this PR into main!


max_tokens's default in docstring of InferenceClient::chat_completion's  is 20 but that of TGI is 100

Hi there!🤗
InferenceClient::chat_completion's docstring states that max_tokens's default value is 20 but it seems that of the TGI is 100.
https://github.com/huggingface/huggingface_hub/blob/8a99deb4e1c2c70ed180d4429e9da48902cd39bf/src/huggingface_hub/inference/_client.py#L586-L587
https://github.com/huggingface/text-generation-inference/blob/6e3220529df5906ae586031873b7865e9923040b/router/src/lib.rs#L939

Hello @sadra-barikbin , thanks a lot for reporting this! 🤗 I just opened a PR #2653 to fix the docstrings.


Validate metadata during entry using nwbinspector

fix (1) and (2) on #93
This PR introduces a way to validate metadata against the nwbinspector and implements this feature on the project-wide metadata form.

@CodyCBakerPhD Finished adding your suggestions. Should be good to review!

As I test this out in the actual app, should it be preventing me from moving to the next page if there's a metadata issue? Or should it be allowing non-valid entries to these fields of the metadata form to begin with?
Currently I can enter lots of non-compliant stuff in these fields and it allows me to proceed

As I test this out in the actual app, should it be preventing me from moving to the next page if there's a metadata issue? Or should it be allowing non-valid entries to these fields of the metadata form to begin with?
Currently I can enter lots of non-compliant stuff in these fields and it allows me to proceed

Nevermind, as per meeting today I checked and ensured that functionality is working well on the project level metadata side

@garrettmflynn This is really, really great. Can't wait to see it integrated with all the other metadata fields!


[*] Contributing on the TODO LIST

Hello! @reveng007
First of, nice project I have to say! Many great links and well structured 👏🏻.
Overview
I plan on contributing to this project, that is - I saw (both the Issues, and the TODO-List) you have made, some of them:

system() like function (but in kernel)
sockets (this one was a bit broad/confusing) - do you want it to be a reverse shell..?
Among others.

I plan on contributing to all the TODO's, (as by the time writing, each Issue and TODO you have listed does not really seem impossible for me to achieve). Like, the adding a system() like function.
TODO 4# Sockets
This TODO I need to request some details before I dive in. It is unclear what you want to achieve.

Adding Linux Kernel Sockets to this LKM rootkit, so that this rootkit acts as an all-rounder. Both, as a LKM rootkit as well as stealthy C2 Server.

on both of these, what exactly do you want to achieve here?:

"stealthy C2 Server."
"this rootkit acts as an all-rounder."

TODO 5# Persistence
As well as, this one got me specifically interested into contributing, since this would be a great exercise:

Surviving system reboot

Additional Idea
I have thought of some things to add to your project, some of them is included (but not lim. To):

Cryptography, Obfuscation and Steganography.

Cryptography - encryption & decryption (like, encrypting the sockets so it's not sent in plain text..)
Obfuscation - This is pretty self explanatory.
Steganography - if required by the user (that controls the rootkit), he or she will be able to choose to use Steganography as a extra-additional step for security. That is, when exfiltrating something (data, which might be a reverse shell, or something on those lines) It could work by hiding it in, normal pictures or just something else.

Note these would take longer time to make but , still wanted to point them out.
Edit: I also thought of adding something related to kernel-mode shellcode execution, as it is something I have been working on now for some time. I will see if I have the time; if this would be something of interest.

have a great day! If any comments,thoughts,problems or concerns rise - please point them out!

Of course, it goes without saying - if 1 or more 'todos' is already done, please point them out so I don't reinventing the wheel.
Thank you.

Of course, it goes without saying - if 1 or more 'todos' is already done, please point them out so I don't reinvent the wheel.
Thank you.

No TODO list is actually done till now :)

Of course, it goes without saying - if 1 or more 'todos' is already done, please point them out so I don't reinvent the wheel.
Thank you.

No TODO list is actually done till now :)

Of course, it goes without saying - if 1 or more 'todos' is already done, please point them out so I don't reinvent the wheel.
Thank you.

No TODO list is actually done till now :)

Okay, thank you! have a great continuing day!

Thanks a lot for the appreciation! 💯
Thanks again to you for showing interest in contributing to my project 😃
Overview:

system() like function (but in kernel) just to open a new bash/sh prompt as I have mentioned earlier in the README file.
sockets (this one was a bit broad/confusing) - do you want it to be a reverse shell..? Yes, a revshell but as kernel mode ;)

TODO 4# Sockets:
Rootkit will also act as a revshell (rather than kernel mode revshell) just to nullify the use of usermode revshell for long time just for being stealthy.
Allrounder meaning:  funtionality of revshell +  stealthy as LKM rootkit
TODO 5# Persistence:
Actually, I have no idea how to achieve this till now ;(
Ig some assembly language will be needed to achieve this stuff...
Cryptography and Obfuscation are really dope idea :fire: but Steganography will be dependent upon the Scenario/environment of the Victim Machine, right?
"Note these would take longer time to make but , still wanted to point them out."
=> Not a problem at all man!!
Again thanks for providing a helping hand to me :)
"I also thought of adding something related to kernel-mode shellcode execution, as it is something I have been working on now for some time. I will see if I have the time; if this would be something of interest."
You are most welcome to add those if you want to.
That would really help this project to attract more public attention!!
Thank you @loneicewolf 🤗 , have a nice day!!

No TODO list is actually done till now :)

edit: I will begin with the easiest one, the system() like function(but in kernel);
Thank you for fast reply and have a great continuing day!

I'm also excited about this 😄
Not a problem at all

No problems at all, with the Persistence "exercise" as I called it, I was meaning that I myself would do it; and make (I guess, another Pull Request when it's done)

On the Steganography step, I was a bit vague too, broadly speaking, it "can" be a picture, but it can also be just, a covert channel; whatever that might be can be another question (there is loads of methods to achieve this).
Then I guess I have nothing more to add - if there is anything else just write a new todo element or reply to me in any issue;
Have a nice day! And keep it up :)

Btw, do u know how to make custom logger ??
Like when we inject it creates some logs right?
So I wanted to create a custom logger which can log our kernel print, instead of syslog logging our rootkit's behaviour. Targeting for more OPSEC.
Or another idea was to use macros like, KERN_EMERG, KERN_ALERT from printk.h header which has lower log lvl than console log level, so that those logs will not be logged by syslog.
But we have to some how perform proxying of those output from console to our current terminal.
Did I make any sense?

Hi again!
Thanks for additional reply;

Btw, do u know how to make custom logger ?? Like when we inject it creates some logs right? So I wanted to create a custom logger which can log our kernel print, instead of syslog logging our rootkit's behaviour. Targeting for more OPSEC.
Or another idea was to use macros like, KERN_EMERG, KERN_ALERT from printk.h header which has lower log lvl than console log level so that those logs will not be logged by syslog, rather it will be shown in the console, like ctrl + shift + f5/f4 (i don't really remember that particular tty).
But we have to some how perform proxying of those outputs from console to our current terminal, that means those kernel prints will work like printf (in c) or like echo in bash
Did I make any sense?

Uhm, the last sentence is flawless - sorry I am having a bit trouble following what you try to say; I will try (below):

You want to make a "logger" of some sort, (that, logs everything from what the operator does with the rootkit, what the rootkits output is, and so on) right?
macros, like KERN_ALERT to (according to your text - again, if I understand you correctly) - is to hide the output..?) if you want to hide the output, why not just comment it out? (or if you need the outputs but still don't want it to be "so plaintext" maybe log them somewhere else? Perhaps encrypted?)
"Proxy the outputs from console to current terminal" here I do not follow at all I think.. Do you want to redirect the output from something to something else..? Sorry. Please clarify

And, soon I will have to head home for the night, ill come back online tomorrow.

I think I do understand the - logging the output to another tty - so you have to do the ctrl+shift+fX key (where X is 5,4,...,2,1.) to see the output, correct? If this is not a correct understanding I think you have to elaborate, sorry :)
Have a good night!

I think I do understand the - logging the output to another tty - so you have to do the ctrl+shift+fX key (where X is 5,4,...,2,1.) to see the output, correct? If this is not a correct understanding I think you have to elaborate, sorry :)
Have a good night!

You are right, exactly !
But the thing is, that will be quite hectic right?
Searching for the value of X in ctrl+shift+fX  for every machine once it is injected into kernel.

You want to make a "logger" of some sort, (that, logs everything from what the operator does with the rootkit, what the rootkits output is, and so on) right?

Yes

macros, like KERN_ALERT to (according to your text - again, if I understand you correctly) - is to hide the output..?) if you want to hide the output, why not just comment it out? (or if you need the outputs but still don't want it to be "so plaintext" maybe log them somewhere else? Perhaps encrypted?)

If I comment out the Comment lines, then how will I know that those instructions got successfully executed in the kernel?
Am I right?
According to this link,
Any log messages with levels less than this (that is, of higher priority) will be printed to the console, whereas any messages with levels equal to or greater than this will not be displayed.

I'm talking about this!
3.  "Proxy the outputs from console to current terminal" here I do not follow at all I think.. Do you want to redirect the output from something to something else..? Sorry. Please clarify

Exactly. I want to redirect the output from Console to current terminal we are having as Operator. Does this make sense now?

Not a problem 👍🏼

Can you perhaps make an example of each of them (draw a diagram, or some example code) maybe in another language to demonstrate..? If it is difficult explaining. :)

If more explanation is needed, pls feel free to ask me 👍🏼

Goodmorning!
Yes, thank you @reveng007 Ill check that link, I'm on train now so it will take awhile but; thanks for being patient and so (very) quick responding!

Goodmorning!
Yes, thank you @reveng007 Ill check that link, I'm on train now so it will take awhile but; thanks for being patient and so (very) quick responding!

Hey, not a problem.
You are only helping me out by contributing to my project. So thanks to u :) @loneicewolf

Hello, @reveng007 now I have made up a kinda plan, I will as said be slow on this, not only because of various factors such as the diff.kernel version the rootkit is using, safe coding (since, even if we code in the kernel a rootkit, we should still strive to make it as "stable" and "reliable" ironically speaking as possible); and so on;
The Persistence
Regarding the Persistence step, a naive approach (I guess you & everyone else think of this first when thinking about rootkit persistence) is just, to "load it at startup" like any other loadable kernel module, (which, as a start might be just, placing a copy in a folder that has modules to load at boot); so I think the actual persistence wouldn't be so - complex - but the "exercise" here would be the steps required to make it harder to detect (relatively speaking).
Blog feedback
And I have checked your blog (comprehensive I have to tell you - keep it up! One minor improvement could be to make the subtitles more "visible" because I had to scroll quite lots to get to a part you refereed me to but other than that it's really well structured). :)
Wrap up
So, basically I will at times prepare (not at the same time , but one by one):

[ ] The system() like function in the kernel;
[ ] The Sockets (reverse shell, ...)
[ ] The Persistence
[ ] I will take on the Additional ideas (cryptography,obfuscation,steganography) to the last.

I wish you a nice continuing day! This will be a fun project to contribute to! 👍🏻

Yes, Safe coding is required but I guess before that we need our concepts to work. After that we should stabilize  our code based on that.
Persistence
This can be helpful, although I don't have a good knowledge in asm, in this case it may require us the usage of kernel mode asm.
link: https://programmer.group/inject-your-rootkit-code-into-a-linux-kernel-module.html
Yes, Our Target will be more stealth!
" "load it at startup" like any other loadable kernel module" is easy:
We can leverage `initramfs` to set up rootkit to load up automatically at boot time.

With sudo:
1. adding rootkit (without .ko) to /etc/modules
2. Copying rootkit LKM to: /lib/modules/5.15.0-58-generic/kernel/drivers/<any driver/module folder>
3. depmod -a
4. update-initramfs -u

Blog feedback:
Thanks a lot!
Actually at that time I was totally noob, I didn't know anything about kernel, LKM, even the concept of OS and usermode C was unknown to me.
I have made that comprehensive just to log my learning path. But at last I decided to make that public so that someone like me can learn creating rootkit from scratch even if they don't really have a knowledge of all those basics, without getting intimidated by other high level blogs/posts.
I have to find that out, that how to add those feature so that one click over a topic enables user to easily navigate to that particular topic.
Wrap Up:
In this section, I don;t really have to add anything, everything that we said is perfect!
Except one thing, although it is my query.
Do u have any idea how to do this?

Hiding process files completely. Our hidden process file can be accessed to open/read. If someone does, ls , they can easily open them.

I wish you a nice continuing day! This will be a fun project to contribute to! 👍🏻

Same to you my Man! 👍🏼

regarding the last 2 points, it depends, because "completely" is not possible. But I assume you mean as closest as possible to "complete".
But - I mean, you could do some checks, instead of only "hiding a file that has a prefix" you can, "hide the file iff it contains some specific data" like, some metadata to match - for example the file is In a particular location, at a particular time, made by a particular user(and so on), or have some keyword inside of it. (this can be, whatever from a first line of random letters to some cipher or what it now might be)
of course this is a extremely broad, thing - because there is loads of ways you can program a rootkit, or even just program 1 "thing";
so it depends on the situation very much.
Regarding the blog

Actually at that time I was totally noob, I didn't know anything about kernel, LKM, even the concept of OS and usermode C was unknown to me.

respect! Hard worker ^_^ we all begin somewhere and I don't like to use the word "noob" by the way; but
Have a nice night! ☕

Hmmm!!
Got you point....
Thanks a lot 😊
Good night to u as well!

Update:
@reveng007  Hi!
this was quite the delay, I have to admit. I will hopefully begin very soon with the proof of concepts(as we always start with), then make it more stable. I have researched some topics, and looked into your blog (Because of various factors like  as it was long and, a bit all-consuming to read, and so on - I had to re-write certain parts, on my side that is. I don't really have anything to contribute to the blog post, as I think it's quite well structured. And as you point out in this repo, this is not for the latest kernels, but a specific one.).
Briefly speaking, I have reviewed some stuff I would need in order to make all these contribution "points" (persistence, kernel-mode system() function, etc). And will begin with POCS very shortly.
You can close this if you want, as I will probably (excluding time) make new pull/issues for each "part". As I always do.
Wishes!

No worries...
Take ur time...
I also haven't got time to put in to Linux kernel...
Mostly spending time windows...
Thanks again!
All the very best :)


Dummy commit

JIRA link (if applicable)
PRL-2372
Change description
Standard direction order
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[x] No

Non needed


2nd Monitor: After restart/wake up, 2nd monitor not centered.

After a restart, wake up the second monitor is not centered and i have to open settings for it to fix it self. Main monitor stays centered.
This is on the payed Microsoft Version, this didnt happen on the free zip version i had until 2 days ago. It only happens to me on the paid version.

@ChrisAnd1998 - I think you need to refresh the position after connecting or disconnecting another monitor.

@shayvaturi That's what i tried but, it doesn't work. The UIAutomation API does not recognize a new taskbar without restarting the application.


Can't parse valid expression "string".property

This is valid expression: "string".property, but could not be parsed with jsep

Could you provide more details? This unit test is very similar to your example and seems to be working as expected


[Xamarin.Android.Glide] add multitargeting for .NET 6

Install latest .NET 6 & stable Xamarin.Android on CI
Use Xamarin.Legacy.Sdk.
Build with dotnet build
Use [assembly: AssemblyMetadata("IsTrimmable", "True")]
Use the same pattern as AndroidX, putting .aar/.jar files loose inside the NuGet package
Set legacy compatibility properties mentioned here:

https://github.com/xamarin/xamarin-android/blob/5f408ebb22df51b598c62f776ee0f82f92661b58/Documentation/guides/OneDotNetBindingProjects.md
This also updates any AndroidX dependencies so that they are using the
new versions that also multitarget for .NET 6.

Glide is new dependency of some of GPS-FB-MLKit artifacts and it has MonoAndroid10.0 TFM.

Adding multitargeting and MonoAndroid9.0 TFM
#1270
#1271

Per this comment, I don't think we can remove MonoAndroid90.
Also, what do we need to get CI to run on this? Bump the version number?  (cc @moljac)
I would like a chance to examine the final NuGet package before we merge since there are a lot of changes to this.

CI ran previously... because I downloaded artifacts...

/azp run

Also, what do we need to get CI to run on this? Bump the version number? (cc @moljac)

for publishing?
tagging. But leave it to me. Tags in XC repo are messy and I never got to clean them up.

Per this comment, I don't think we can remove MonoAndroid9.0.

Yes. This is why I wrote that.
OTOH it might take a while for google to make some dependencies releases, until then MonoAndroid9.0 might become obsolete.
Directory.Build.props has MonoAndroid9.0, so it should be OK if I'll be necessary to add those dependencies.

I would like a chance to examine the final NuGet package before we merge since there are a lot of changes to this.

Now I see MonoAndroid9.0 is not packaged.

@moljac would you be able to release the Glide packages later this week? Thanks!

@moljac would you be able to release the Glide packages later this week? Thanks!

Tagged. Waiting for CI. By the time you get up I think I will publish it.

The latest release is working fine: https://github.com/dotnet/maui/pull/2563


Close Caption SEI messages in H.264/H.265 coded stream

For IS-04, video/smpte291 media_type is defined as a Flow of format urn:x-nmos:format:data. It is defined as an allowed media_type for the generic receiver_data.json schema and a the more specific flow_sdianc_data.json schema.
In general I would assume that if close caption data is available in an NMOS system, it will appear as a Source/Flow of urn:x-nmos:format:data format and media_type video/smpte291 or any other standard way of transporting such close caption data.
Supporting in-band close caption as SEI messages implies that the H.264/H.265 Flow is associated to a Source providing a video and a data Flow that result in the H.264/H.265 Flow with embedded close caption. The H.264/H.265 Flow then has a parent video Flow and a parent data Flow. It makes the model a little more complex but it seems acceptable.
                                              parent
               +---------- Flow_video/raw <-----+
               |                                |
               v              parent            |
      +-- Source_video <--------+               |
      |                         |               |
SDI --|                   Source_video <-- Flow_video/H264 <-- Sender
      |                         |               |
      +-- Source_data  <--------+               |
               ^              parent            |
               |                                |
               +----- Flow_data/smpte291 <------+
                                              parent

It seems that the embedding of the close caption information in SEI messages of the coded H.264/H.265 stream does not add anything to BCP-006-02 and BCP-006-03 as long as there is no specific information/control about how the embedding is performed. Simply looking at the coded Flow attributes does not indicate that there is such embedding. A Controller/User must look at the Flows/Sources parents relationship to realize that such embedding is performed.

Wouldn't the main H264 flow be of urn:x-nmos:format:mux format if that were the case?

I don't think that needs to be the case. The parents relation indicates this Flow is derived from the referenced Flows in some manner but doesn't indicate how.


resize by max value

suggest adding the function, which resize by one(max) value:
public static class SizeHelper
    {
        public static Size Resize(this Size oldSize, int maxValue)
        {
            var aspect = oldSize.Width / oldSize.Height;

            int newW;
            int newH;

            if (aspect > 1) // landscape
            {
                newW = (int)Math.Min(oldSize.Width, maxValue);
                newH = (int)Math.Truncate(newW / aspect + 0.5);
            }
            else
            {
                newH = (int)Math.Min(oldSize.Height, maxValue);
                newW = (int)Math.Truncate(newH * aspect + 0.5);
            }
            return new Size(newW, newH);
        }
    }

example:
all images in my database has max size(width and height) = 1500
i have image 3000x2000. i need 1500:1000
i have 2000x3000. i need 1000:1500
Size newSize = origSize.Resize(1500);
..... Resize(newSize.Width, newSize.Height);
Such a function can be useful :)

@protokovich
Thank you for this nice suggestion too.
I'm going to consider how to incorporate this method to method chaining.


Make immutable path matching exact

Previously Javalin would mark any file with path starting with "/immutable" as immutable; this patch requires files to be in a directory named exactly "immutable" as the docs specify.
Fixes #369.

Thanks @akunohomu!

This has been released as part of 2.3.0 now: https://javalin.io/news/2018/10/06/javalin-2.3.0-released.html


Add support for VHDL configurations

When having multiple architectures for a given design, VHDL configurations can be used to swap architectures across entity interfaces.
Configurations are used to link the desired architecture to the entity's interface. When using configurations for simulation, the configuration design unit is what becomes the top-level, not the test-bench entity itself. If a configuration design unit is found in the top-level test-bench entity file, rename the @VHDL-SIM-TOP to be the configuration name associated with that entity on export.

Full VHDL configuration has now been added, both in-line within an architecture's declaration section and as configuration design unit's for entities and their architectures.


BN-16 feat: A seller should be able to delete a specific item

PR Description
As a Seller, I should be able to delete a specific item in my colletion
Such that I can remove items I no longer deal in/were errornously added
Description of tasks that were expected to be completed
List all the tasks and sub-tasks that were assigned to you Make sure you check every completed task. list them in this style

[x] Create delete request to delete a specific item in a given seller's collection
[x] When the requesting user/token is a seller and the target item does not exist in the particular seller's collection and the request body is validated
Then:
A json object with a status: 4xx, error: the appropriate error message should be sent
[x] When the requesting user/token is a seller and the target item already exist in the particular seller's collection and the request body is validated
Then:
The item should be deleted
A json object containing status: the appropriate code, message: appropriate response message, and item: an object of the deleted item should returned
[x] When the request body fails any validation checks or the token is not of the role seller
Then:
The appropriate 4XX response message should be sent to the requesting client describing the validation error

Description of tasks that were expected to be completed and that are not yet completed

[x] none

How this may be tested?
Please describe the tests that you ran to verify your functionalities. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration.

run npm i to install all required dependencies
run npm run dev
step 3
step 4...

Screenshots (If appropriate/applicable)
Provide screenshots related to your PR in case it's appropriate and applicable
Please check this Checklist before you submit your PR:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented on my code, particularly in hard-to-understand areas
[x] My code generates no warnings
[x] There are no vulnerabilities

I will work on Swatch the commit and also removing . ignore file soon as
possible but i will not be able to attend standup for today I'm in person
case that take me more time than I expected and I don't have laptop for now.
On Wed, May 3, 2023, 14:50 Emmanuel Dushime @.***>
wrote:

@.**** commented on this pull request.
If you can be able to swatch commits, this PR is ready to be merged.
In .gitignore
https://github.com/atlp-rwanda/ecommerce-app-cogito-bn/pull/35#discussion_r1183638938
:

@@ -1,2 +1,3 @@
.env
node_modules
+src/routes/vendor/userRoutesV.js

why did you added this file in .gitignore?
—
Reply to this email directly, view it on GitHub
https://github.com/atlp-rwanda/ecommerce-app-cogito-bn/pull/35#pullrequestreview-1410818307,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYGIFYDDJWPQILA6MTADBKLXEJICZANCNFSM6AAAAAAXK6U7EU
.
You are receiving this because you were mentioned.Message ID:
@.***>


SAK-30924 Drag and Drop upload has'Overwrite existing files' function…

…ality.
Added overwrite boolean for 'doDragDropUpload' method which checks if user
wants to overwrite files.

I commented on the JIRA.
This seems like unnecessary clutter to me .... we could add 20 checkboxes on this page .... not sure why this is important enough to deserve a checkbox.

didnt we establish that we DID want this feature (on a separate mail list conversation). This was because the default behaviour is to NOT overwrite existing files which is counter intuitive.

so could this be merged then?
Everybody's in agreement that it's desirable arent they.
I'd even go so far to say the current behaviour is a bug (but that's just me all over isnt it?)

so is this going to be merged. All parties seem to think it's a good idea

actually, we need to include https://jira.sakaiproject.org/browse/SAK-31794 in this PR. I'll get Nick to update this PR

This PR is now ready to merge, can somebody do it?

Check to see if old file can be moved to trash.

the idea is that the display of the option is behind a property, the default value of the property would be "dont display the option in the UI".
People would like a pop up list of files that will be overwritten with "continue" & "cancel" buttons AND / OR people would also like the overwritten file to be moved to Trash.
If the above is done the there's no need to protect via a property
NB: the same should be done with "upload new version" in Resources > Actions as well (maybe on a separate Jira)
this is also loosely related to https://jira.sakaiproject.org/browse/SAK-31947 (WL-2521 @ Oxford0

@ern We've just had a discussion here and noted that when you do an "Upload New Version" via Resources > Action menu, the file isnt moved to Trash, so why should it be done here. The improvement is a non trivial fix. I am very happy for this to me merged (pretty much) as is and will create a new Jira to say that both "Upload New Versions" should move the original file to Trash.

given my above comments, can this be merged and a new Jira created to say when a file is overwritten either in resources > upload new version  OR drag and drop upload the previous copy should be moved to Trash?
I agree this extra feature is a great improvement but is separate from the issue being covered here.

Needs rebasing...

I would recommend a property to enable the visibility of the checkbox and another property to set the default status of the checkbox (checked or not checked)

I feel like if this got added there's also no point for the "Upload New Version" feature anymore, so that could be removed to simplify the workflow. Upload new version just is the same as upload.
I'd also feel like refactoring all of this into kernel "addResource" with a parameter for "boolean overwrite" makes more sense than all this code duplication is a better idea.

Shawn: we'll do the requested properties.
Matt: I tend to agree that upload new version should be the default, however, in the case of "single file upload" (Resources > Actions > Upload new version" it may make sense to have a confirmation check "Are you sure you wish to overwrite this file". It is fairly easy to accidentally select "Upload New Version" alongside the wrong file. The idea of making 'overwrite' the default action is totally separate issue though - I think it needs a Jira of its own.

Rebased branch and opened PR #3975


Mismatch vnode.children & node.data when use tsx

Environment

Node Version:     v16.20.0
Nuxt Version:     3.6.5
Nitro Version:    2.5.2
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      devtools
Runtime Modules:  -
Build Modules:    -

Reproduction
https://stackblitz.com/edit/github-jztfdl?file=pages%2Findex.vue
Describe the bug
We will get two warnings like the following logs
When we use jsx synax,even and we don't wirte any code that can effect server render:
import { defineComponent } from 'vue';

export const testComponent = defineComponent(
  () => {
    const templateText = 'my text';
    return () =>
hello wrold {templateText}
;
  },
  {
    name: 'test-component',
  }
);

but still mistach.
Additional context

I don't know why they are different.
(vnode.children !== node.data).
Maybe it's a error of render.
Logs
[Vue warn]: Hydration text mismatch:
- Client: "hello wrold my text"
- Server: "hello wrold "
  at <TestComponent>
  at <Index onVnodeUnmounted=fn<onVnodeUnmounted> ref=Ref< undefined > >
  at <RouteProvider key="/" vnode= Object route= Object  ... >
  at <RouterView name=undefined route=undefined >
  at <NuxtPage>
  at <App key=3 >
  at <NuxtRoot>

[Vue warn]: Hydration children mismatch in
: server rendered element contains fewer child nodes than client vdom.
  at <TestComponent>
  at <Index onVnodeUnmounted=fn<onVnodeUnmounted> ref=Ref< undefined > >
  at <RouteProvider key="/" vnode= Object route= Object  ... >
  at <RouterView name=undefined route=undefined >
  at <NuxtPage>
  at <App key=3 >
  at <NuxtRoot>

I found the question when i debug.
The vnode children has two elements,but the vchildren only attach the first child:
(createTextVNode('你的名字是: '))
It will ignore the scecond child.

This is not a Nuxt issue and is likely a core Vue SSR bug with TSX. Would you check to see if it already exists in https://github.com/vuejs/core and if not, raise an issue?
You can use https://stackblitz.com/github/nuxt-contrib/vue3-ssr-starter to create a reproduction. 🙏

This is not a Nuxt issue and is likely a core Vue SSR bug with TSX. Would you check to see if it already exists in https://github.com/vuejs/core and if not, raise an issue?
You can use https://stackblitz.com/github/nuxt-contrib/vue3-ssr-starter to create a reproduction. 🙏

Sure,let me try.

This is not a Nuxt issue and is likely a core Vue SSR bug with TSX. Would you check to see if it already exists in https://github.com/vuejs/core and if not, raise an issue?
You can use https://stackblitz.com/github/nuxt-contrib/vue3-ssr-starter to create a reproduction. 🙏

you're right,the problem is same when i don't use nuxt.
i will close this issue,thanks bro.

No problem! Glad I could help 😊


Add additional logs in NvdApiRetryStrategy

PR #162

If you create the Issue first - then reference the issue in the PR, the issue will automatically close when the PR is merged. example:
git commit -am "fix: resolves #161"

See https://docs.github.com/en/get-started/writing-on-github/working-with-advanced-formatting/using-keywords-in-issues-and-pull-requests#linking-a-pull-request-to-an-issue


Report result with same command is not constant

Describe the bug
When you run tern with Scancode extension repeatedly,
small parts of the results are different from each other.

To Reproduce
Steps to reproduce the behavior:

pull fossology/fossology:3.10.0
Run command tern report -f json -x scancode -i debian:buster -o debian_buster_img_scancode.json

Error in terminal
no error occurred
Expected behavior
The report on a same docker image should be same.
Environment you are running Tern on
Tern at commit 273e3c8
Python 3.6.13
Docker version 19.03.13, build 4484c46d9d
Distributor ID: Ubuntu
Description: Ubuntu 16.04.7 LTS
Release: 16.04
Codename: xenial

Thanks for the issue @JustinWonjaePark -- Is the parts of the code that are differing file-related output from Scancode or package metadata from Tern?

Can you also confirm which version of scancode you have installed?

Hello @rnjudge , I am sorry for late response.
The version of scancode that I use is 21.3.31

@rnjudge , it actually looks part that returned from Scancode.
(copyright and license information by files)

@JustinWonjaePark it looks like this is an issue with scancode then? or is it specific to Tern? Tern simply captures scancode output and reports on it. Are the differences you're seeing after back to back runs on the same image or running with a fresh cache for the same image? I can try to reproduce but a little confused by your steps that say to pull fossology image and then run a tern+scancode report for debian:buster. Can you clarify what you did to produce this error?

Here are the steps to run scancode without Tern:

tern -k report -i debian:buster. The -k option keeps the containers mounted filesystem so you can run scancode on it independently of Tern.
Find the mounted filesystem by runningls ~/.tern/temp/. The directory under ~/temp for me and the debian:buster image was e63992559353a522e62a73bc137fd023ee97e1226b96947df8e00e73d1a9d627. Whatever the directory there is, use that.
Run scancode on the mounted filesystem. sudo /home/rjudge/ternenv/bin/scancode -ilpcu --quiet --timeout 300 -n 4 --json - /home/rjudge/.tern/temp/e63992559353a522e62a73bc137fd023ee97e1226b96947df8e00e73d1a9d627/contents

The output of scancode will print directly to the terminal.

Hi @JustinWonjaePark any followup from your investigation?


Fix avoidCss3dTransform test

small typo that caused issues on older browsers, fixes part of #926 .

pretty straightforward fix, merging


Patch documentation Cosmos Db raw json support for Spark Connector

Patch documentation  raw json support for Spark Connector

@seesharprun
Because this PR submits changes that include tabbed content and also content that seems unintended for the end user (line 319), the content must be moved to the private repo before merging. Let our team know if you need help with that.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@tangerinous thanks for the changes, looks good to me.

#sign-off

@tangerinous  - looks like you need to run a PR against the private repo of this, please reach out to me for more information, thanks!

Moved PR to target repository