Crashing when taking lighting sword

Crashing when I do /customitems and choose lighting sword

crash log pls?

THIS CRASH WAS CAUSED BY A PLUGIN
BAD PLUGIN: CustomItem
Thread: Main
Error: An error occurred while adding an enchantment to the item.
File: plugins/CustomItem-dev/src/Manager/ItemManager
Line: 74
Type: InvalidArgumentException
Backtrace:
#0 plugins/CustomItem-dev/src/Manager/CustomItemsManag

thanks for reporting!.. I'll resolved it soon..

fixed... Commit


Implementation guide and template for plugins

I would like to propose an implementation guide and template for official plugins.
The template for plugins could contain a reference for directory layout, build, testing and documentation.
The plugin system is so good and that would let the svg.js core to remain small and place some of the sugar in plugins instead.

Want to back this issue? Place a bounty on it! We accept bounties via Bountysource.

@Fuzzyma where is the guide?

The "Extending functionality" section from the docs explains how to do it.
There is nothing more to say because it's just that simple.


How to make wider?

I want to make the content of the page and the buttons toolbar wider.
Is it possible to do this with this theme ?
If not then I propose this as a new feature request as the pages are way too narrow.

I'm new to github themes and I selected this theme and also wanted content of the page to be wider.
I have created file: /assets/css/style.scss and inside is this:

@import "{{ site.theme }}";
.inner {
width: 80%;
}


[Python] Converting data frame to Table with large nested column fails Invalid Struct child array has length smaller than expected

Hey,
I have a data frame for which one column is a nested struct array. Converting it to a pyarrow.Table fails if the data frame gets too big. I could reproduce the bug with a minimal example with anonymized data that is roughly similar to mine. When I set, e.g., N_ROWS=500_000, or smaller, it is working fine.

import pandas as pd
import pyarrow as pa
N_ROWS = 800_000
item_record = {
    "someImportantAssets": [
        {
            "square": "https://some.super.loooooooooong.link.com/withmany/lorem/upload/"
            "ipsum/stilllooooooooooonger/lorem/{someparameter}/156fdjjf644984dfdfaera64"
            "/specificLink-i15348891"
        }
    ],
    "id": "i15348891",
    "title": "Some Long Item Title i15348891",
}
user_record = {
    "userId": "faa4648-4964drf-64648fafa648-4648falj",
    "data": [item_record for _ in range(24)],
}
df = pd.DataFrame([user_record for _ in range(N_ROWS)])
table = pa.Table.from_pandas(df)

Traceback (most recent call last):
  File "/.../scratch/experiment_pq.py", line 23, in <module>
    table = pa.Table.from_pandas(df)
  File "pyarrow/table.pxi", line 3472, in pyarrow.lib.Table.from_pandas
  File "pyarrow/table.pxi", line 3574, in pyarrow.lib.Table.from_arrays
  File "pyarrow/table.pxi", line 2793, in pyarrow.lib.Table.validate
  File "pyarrow/error.pxi", line 100, in pyarrow.lib.check_status
pyarrow.lib.ArrowInvalid: Column 1: In chunk 0: Invalid: List child array invalid: Invalid: Struct child array #1 invalid: Invalid: List child array invalid: Invalid: Struct child array #0 has length smaller than expected for struct array (13338407 < 13338408)

The length is always smaller than expected by 1.
Expected behavior:
Run without errors or fail with a better error message.
System Info and Versions:
Apple M1 Pro but also happened on amd64 Linux machine on AWS

arrow-cpp                 7.0.0           py39h8a997f0_8_cpu    conda-forge
pyarrow                   7.0.0           py39h3a11367_8_cpu    conda-forge
python                    3.9.7           h54d631c_3_cpython    conda-forge

I could also reproduce with

 pyarrow 8.0.0

Reporter: Simon Weiß
Related issues:

[Python] Converting data frame to Table with large nested column fails Invalid Struct child array has length smaller than expected (is cloned by)
[Python] List child array invalid (duplicates)
[Python] List child array invalid (is duplicated by)

Note: This issue was originally created as ARROW-17137. Please see the migration documentation for further details.

The same error appears when converting a List of Dicts in Python (JSON data) to a pa.Table -.-
I will test the workaround now, is there any possibility that this will be fixed anytime soon?

The issues is going to happen anytime a single string column ends up with more than 2^31 characters.  So, in OPs reproduction the column square has 161 characters per string and 800,000 * 24 strings which is 3,091,200,000 characters.  2^31 is 2,147,483,648.  At this point we have to split the resulting array into chunks (or use the large_string data type but that has issues of its own).
This "breaking unexpectedly large columns into chunks" behavior is rather tricky and it appears we are doing something wrong when working with lists of struct arrays.  Here's a compact reproducer (that only has 3 rows):
import pyarrow as pa
import pandas as pd

x = "0" *<PHONE_NUMBER>
df = pd.DataFrame({"strings": [x, x, x]})
tab = pa.Table.from_pandas(df)
print(tab.column(0).num_chunks)

struct = {"struct_field": x}
df = pd.DataFrame({"structs": [struct, struct, struct]})
tab = pa.Table.from_pandas(df)
print(tab.column(0).num_chunks)

lists = [x]
df = pd.DataFrame({"lists": [lists, lists, lists]})
tab = pa.Table.from_pandas(df)
print(tab.column(0).num_chunks)

los = [struct]
df = pd.DataFrame({"los": [los, los, los]})
tab = pa.Table.from_pandas(df)
print(tab.column(0).num_chunks)

It seems the struct array has length 3.  Meanwhile, it's child, the string array, has length 2 (because it had to be broken into 2 chunks.  The first chunk has the first 2 values and the second chunk has the third).
So if someone wanted to investigate this I would recommend starting by looking at the conversion from pandas code and see how the struct array and list arrays are handling the case where their children is converted into multiple chunks.

👀 Any updates for this issue?

The workaround works, but it definitely slows things down

I think the root cause is here:

https://github.com/apache/arrow/blob/main/cpp/src/arrow/util/converter.h#L368
https://github.com/apache/arrow/blob/main/cpp/src/arrow/util/converter.h#L377
https://github.com/apache/arrow/blob/main/cpp/src/arrow/util/converter.h#L86
https://github.com/apache/arrow/blob/main/cpp/src/arrow/array/array_base.cc#L272
https://github.com/apache/arrow/blob/main/cpp/src/arrow/array/data.cc#L139
https://github.com/apache/arrow/blob/main/cpp/src/arrow/array/util.cc#L317
https://github.com/apache/arrow/blob/main/cpp/src/arrow/array/util.cc#L65

In the blanket implementation of Array::Slice, only the offset and length of the underying ArrayData is modified.
For e.g. ListArrays, this means that when looking at its length and traversing its internal value offsets (its ArrayData) things will look as expected. But when we check the underlying child_data (the actual values ArrayData) we won't have the same sliced view.
I did a janky sanity test to make Array::Slice virtual and override in ListArray to slice the offset/length of the child_data, which seemed to work. But that's probably not what we want as an actual fix.
I'm guessing this will be the same for all nested types. Hence, we get the validation error which probably isn't expecting Slices that have extra data.
I'm not sure what the best path forward is here. It's not clear to me:

whether the Slice is actually what we want (do we want to keep the extra data around?)
how to support "deep slicing" for all the other less common types I don't have experience with (Unions, Dictionary, etc)

@westonpace can you advise if this sounds like the right place, and what next steps should look like?

@mikelui

In the blanket implementation of Array::Slice, only the offset and length of the underying ArrayData is modified.
whether the Slice is actually what we want (do we want to keep the extra data around?)

If memory serves this is correct (I am not quite 100% certain of this paragraph but "fairly confident" :)  maybe someone else will confirm) and intended behavior.  The length of a parent array may be shorted than the length of a child array.  Offsets are cumulative so if a parent array has offset 10 and a child array has offset 5 then the memory location that is looked at for the 0th value will be the 15th slot.

EDIT: Actually I think I can make it work by overriding ToArray(length) in the PyConverters and doing some logic to check if shrinking is necessary. It still would be essentially a custom Slice implementation. But, it at least constrains the scope to just the python -> arrow conversions. I'll work on a PR soon.

I'm not very familiar with the python converters.  However, I think we have a good amount of regression tests.  If you can get them to pass while passing this case then you are probably on the right track.


[bug] QR Code connection issue after updating to version 2 and setting chains to crossbell

Link to minimal reproducible example
https://github.com/runjuu/crossbell-wallet-connect-v2-example
Summary
Hi, I encountered an issue when updating to version 2 and setting the chains to [crossbell] (import from @wagmi/core/chains). Unfortunately, I am unable to connect using QR Code. I attempted to use both Trust Wallet and Spot Wallet, but neither were successful. Interestingly, version 1 worked without any issues.
Trust Wallet

Spot Wallet

List of related npm package versions
"@wagmi/core": "0.9.6",
"@web3modal/ethereum": "2.1.3",
"@web3modal/html": "2.1.3",
"ethers": "5.7.2"

Does v2 work with our wallet example? https://react-wallet-eip155.vercel.app

Hi @0xAsimetriq , I'm unsure of how to change the network to crossbell on https://react-wallet-eip155.vercel.app. Additionally, I was wondering if there are any Git repositories that I could clone to test the crossbell network?
Thank you for your help!

@runjuu Could you deploy your app to somewhere like vercel so we can test it (dapp) against our wallet https://react-wallet-eip155.vercel.app
My hunch is that Trust is not handling combo of v2 / crossbell properly, so I want to test against our demo wallet which allows connection with any chain. If it works out there, issue is likely with trust wallet.

Closing due to innactivity


Better communicate testnet reset

We need to better communicate to devs working with testnet that the ledger will be reset every 3 months. It may also be important to change the network passphrase and avoid any kind of accidental mixing between nodes & clients during testnet reset.
Ideas:

update Horizon errors on testnet to state the date of the last reset and the date of the next one, and link out to our testnet best practices
change the network passphrase, and update SDKs to auto-detect the current passphrase
modify Stellar lab to state reset information when in testnet mode

Any other ideas?

That seems fine.
For the date in the passphrase, does it need to be fully specified or just have the month/Quarter (and have a link to get the actual date)?
Something like Stellar Test Network Q3 2019; See http://www.stellar.org/testnet for more information
Do we need to use templates somehow for all the sample configurations that we have for Horizon/Core - also, some of those, I think, are packaged/tagged with releases

I like updating the passphrase with date + time, it will be a good motivator for automating the reset, which ultimately is the goal here.
With regards to configs, the stellar-core-postgres package has a sample Testnet configuration which would need to be updated every quarter. We could easily modify the NETWORK_PASSPHRASE during package build and then simply schedule a build of this package every 3 months.
Caveat: Package end-users utilising a modified config will still need to update the passphrase manually.

Yeah, the advantage of including the date + time is that https://horizon-testnet.stellar.org/ becomes the one source of truth for when the reset is happening.

Below are the 3 configs with a Testnet passphrase that end up in the .deb packages:

https://github.com/stellar/packages/blob/master/stellar-core-postgres/stellar-core_postgres.cfg
https://github.com/stellar/stellar-core/blob/master/docs/stellar-core_testnet.cfg
https://github.com/stellar/stellar-core/blob/master/docs/stellar-core_standalone.cfg

The standalone config should probably be updated to use "Standalone Network ; February 2017" in the same way that the docker quickstart standalone config does here.
For public docker images, we need to update:

https://github.com/stellar/docker-stellar-core-horizon/blob/master/testnet/core/etc/stellar-core.cfg
https://github.com/stellar/docker-stellar-core-horizon/blob/a2f1f5ff4717520e6a2c28aa468d1d8758fd7d8b/stat#L77
https://github.com/stellar/docker-stellar-core/blob/1ea7fa8c7b99732574ab5a20166f8eee1171ff5b/examples/compat_complete.env#L22
https://github.com/stellar/docker-stellar-core/blob/1ea7fa8c7b99732574ab5a20166f8eee1171ff5b/examples/compat_minimal.env#L20
https://github.com/stellar/docker-stellar-core/blob/1ea7fa8c7b99732574ab5a20166f8eee1171ff5b/examples/local.env#L34
https://github.com/stellar/docker-stellar-core/blob/1ea7fa8c7b99732574ab5a20166f8eee1171ff5b/examples/single.env#L36

There is also a bunch of documentation that needs updating as part of the process, either permanently or on each reset.
https://github.com/stellar/docs/search?q=Test+SDF+Network+%3B+September+2015&unscoped_q=Test+SDF+Network+%3B+September+2015
We wouldn't have to update all the configs/docs if we set the passphrase to NETWORK_PASSPHRASE="Test SDF Network ; Testnet is reset every 3 months - backup your data", or something along those lines.
But then new quickstarts might still get bad messages from non reset peers.
@MonsieurNicolas is there anything else we can do to ignore messages from stale peers ?

As for ignoring other peers without changing the passphrase I don’t think we can do a lot more than what we’re doing

Resolving this as testnet resets seem to be going smoothly lately, thanks to Tom's work and better announcements.

@tomquisel do you think this issue can be closed?


[Format][C++] Recommended/required value for ArrowDeviceArray.device_id int in case of CPU data

In https://github.com/apache/arrow/pull/40717#pullrequestreview-1958505514, @paleolimbot noted that the current implementation of the C Device interface in Arrow C++ is using a device_id of -1 when exporting CPU data (while nanoarrow is using 0).
The spec itself doesn't say anything about this (https://arrow.apache.org/docs/format/CDeviceDataInterface.html#c.ArrowDeviceArray.device_id):

Mandatory. The device id to identify a specific device if multiple devices of this type are on the system. The semantics of the id will be hardware dependent, but we use an int64_t to future-proof the id as devices change over time.

Should we recommend or require a specific value for this case?
I noticed that the DLPack specification, from which the device type/id structure was taken, does specify to use 0 (https://dmlc.github.io/dlpack/latest/c_api.html#c.DLDevice.device_id):

The device index. For vanilla CPU memory, pinned memory, or managed memory, this is set to 0.

Should we adopt the same guideline for our spec?
cc @pitrou @zeroshade @kkraus14

Since the device_id should never be utilized or have any meaning in the case of CPU memory I'm fine with it either being unspecified or being explicitly specified to be 0. In my opinion it's an arbitrary decision / value

Perhaps we could add something like: "If the device type does not have a notion of device id, it is recommended to return 0 as a convention".

Keeping aligned with dlpack and using 0 probably makes the most sense but there is some downsides.
If we wanted to optionally denote which NUMA node CPU memory is on for example, being able to use numa node 0 would be good. Using -1 as the default would be more ideal in this situation.
For managed memory, if that memory has been touched on a specific GPU then it would be local to that GPU and while it could be accessed by any device, it would be expensive to do so where it would be ideal to use an actual device id for that. If it hasn't been touched yet then again something like -1 would be ideal to indicate that it's not known if the memory is local anywhere.

Given @kkraus14's comments, I think we'd be better served with recommending -1 then

My understanding is that this is a doc clarification, should we add it for 16.0.0?

Issue resolved by pull request 41101
https://github.com/apache/arrow/pull/41101


Markdown headers with links in them create unwieldy ID values

The new auto ID functionality from php markdown (courtesy of @evert) is pretty nice but it starts to break down once you create links in the headers:
### [Sculpin](https://sculpin.io) in a Nutshell

I would assume (or hope) that the above ID would be '#sculpin-in-a-nutshell' but it is instead:
#%5Bsculpin%5D%28https%3A%2F%2Fsculpin.io%29-in-a-nutshell

This may be a case of misunderstood expectations on my part. I see a few potential things:

I seem to recall that there was some rule about not making block level elements clickable. I think that what I'm doing here does not fall under category (this is
stuff
 vs
stuff
) but if it does, maybe what I'm doing shouldn't be done anyway.
Maybe it is expected that the full HTML be included here to distinguish it from other similar headers that do not have links.
We should striptags the header before making it clickable.
What happens if other things are included in the header, like ? Is that included in the header ID or...?

Hi!
So I think we can fix this on the sculpin-side. the id="" tag generator just gets the entire contents of the header. So, basically anything that comes after the ### in plain markdown (so before conversion to html).
There's a few things i could potentially do:

I can use striptags to take out any html. I believe striptags is safe as long as we don't use allowable_tags.
I can automatically remove all markdown links with a regex.
I can remove all special characters from the resulting string.

That would at least cover all the cases that you mention. If you are happy with that approach, I can prepare a patch. This is indeed 100% on the sculpin side based on the way I've implemented it in the markdown library.

@evert Interesting. OK, so, I guess the question comes down to whether the links are already rendered at the time that this function is called. Right? I'd say if they are already rendered, let's do striptags. Otherwise, remove the markdown links with regex.

They are not yet rendered. My suggestion was actually not a multiple choice, I was gonna go for all three, in that order =)

@evert then by all means, have at it. (i misunderstood :) )


How use proxy?

r.url('https://2ip.ru/', proxy = "http://<IP_ADDRESS>:3128")

Hi @BlondinkaQ for setting proxy, the underlying browser is Chrome. You can use visual automation and keyboard automation to control Chrome to set the proxy using the same steps as you would set them manually (from Chrome Settings --> proxy).


Link to standard Java and Android classes/documentation

Hi Ishan,
Firstly, loved the plugin. Thanks again!
One suggestion I had - while looking into someone else's source code on github, I sometimes come across android framework classes which I'm not particularly familiar with. It would be helpful if I could directly link to the either the source code or documentation of these classes. (Similar can be done for java classes as well) This could be maybe be enabled by default or via an option/setting.
Let me know what are your views on this.
If you think this makes sense, I could give it a shot (I haven't understood the source code completely, but I think it shouldn't be too difficult - Need to match for android.* packages and make changes around browseableClass I think - correct me if I am wrong)

@ManojMadanmohan Sounds like a nice idea to me! Let the PRs rolle in ;)

Another thought -
We could move this behind an interface actually, with the implementations returning the true/false if they can handle a package/class, and the url to link to if they can. Or instead, maybe move this information inside a configuration file.
This way, people from the community can add in the required links for packages, and we could maybe have a settings/option on the UI to enable/disable each of these packages.
This is offcourse a bit complex; might take a lot of time.
For now, I've done the work for linking to android documentation; raising a PR shortly

Fixed in #8


[Feature]: Support Model Driven Application

Is your feature request related to a problem? Please describe.
The preview version of Power Apps Test Engine only supports canvas applications. Extend the Test Engine support to also include the ability to write tests for Model Driven applications so that they can be included in testing of Power Platform solutions
Describe the solution you'd like
Provide the ability to:

Navigate between pages
Navigate between view and detail pages
Execute actions like Save and Save & Close
Query the JavaScript Object Model of the page and create Power Fx variables that represent the current state
Allow updates to the Power Fx variables and update the JavaScript object model

Describe alternatives you've considered
No response
Additional context?
No response

Yes! This is a game changer.

Thanks @filcole for the feedback. This is something we are actively looking as it is a natural evolution of Introduce the Extensibility Framework
The plan is to make use of XRM SDK and wrap existing JavaScript Object Model to a provide Low Code reprensentation of grids and forms that can be interacted with via Test Engine:

GridEntity - To get grid view for current grid
GridRow - To use code like gridContext.getGrid().getRows()[0].data.entity

For the kits that the Power CAT tools team creates and maintains we have many Model Driven Applications that we need to also test. The proposed extensions to support MDA address a key need for our test automation evolution.

@Grant-Archibald-MS  we were preparing to spend a rather large effort to use the older Easyrepo approach with our MDA solutions, however I see a lot of commits around the MDA functionality is it close enough to warrant pausing for some weeks? I know you cannot give specific dates but just a general scope before the end of the month?, year?

@Grant-Archibald-MS
Hi there. We are also interested in an update regarding test-engine for model driven apps. We were using EasyRepro too before, in combination with the SpecFlow library implemented by Cap Gemini (which is heavily based upon EasyRepro). But both libraries have not been updated for 2 than years. And with all the changes that have been done to the Power Platform GUI design, but also the fact that EasyRepro is still based on Selenium 2 we decided to stop putting any effort into UI tests based upon the SpecFlow library and EasyRepro.

@sdhsynsci Model Driven Application support is progressing and if you wanted early look at what is working we have integration branch, However this is using a build from source strategy using the MIT licence not included as new features of pac test run.
@pvillads as owner of the Test Engine and the release schedule for updates to the pac test run command.

@JoehannusApg thanks for the interest in automated testing of Model Driven Applications using Test Engine and it is an active area of interest for the team to work towards covering a range of MDA scenarios across entitylist, entityrecord and custom pages.
Looking at the architecture it is based in Playwright and take advantage of Power Fx as the method. It also has a number of features around authentication handling cases like MFA and execution in the context of CI/CD process.
As an early adopter of the Test Engine we are using the custom page support of Test Engie to test our tests for the CoE Kit. We covered some of this during our recent community office hours https://www.linkedin.com/feed/update/urn:li:activity:7262550764605104138/ (Slides 6 - 9) discuss our ALM process and how we are using Test Engine for our testing work

@Grant-Archibald-MS @pvillads
Can you give an estimate on a possible Public Preview date? And perhaps even what you are aiming at for a possible General Availability Date?


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_eb8f8e27-b0a8-49a9-8155-a06acc32cdd8/pull/1 is merged into live branch.
Auto archive content to https://github.com/VSC-Service-Account/docs-archive-test-target_eb8f8e27-b0a8-49a9-8155-a06acc32cdd8.git

Docs Build status updates of commit c14dac1:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-UserAccountBuildPermissionInsufficientError] Owner v-alji no longer has Read permission of the repository https://github.com/VSC-Service-Account/docs-archive-test-source. It might be due to recent GitHub changes on SSO. Please ask the owner v-alji to logon to Docs Portal https://ops.microsoft.com once again. If it is not the case, please ask anyone of Admin permission of the repo to assign a new owner via Docs Portal https://ops.microsoft.com

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit c14dac1:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-UserAccountBuildPermissionInsufficientError] Owner v-alji no longer has Read permission of the repository https://github.com/VSC-Service-Account/docs-archive-test-source. It might be due to recent GitHub changes on SSO. Please ask the owner v-alji to logon to Docs Portal https://ops.microsoft.com once again. If it is not the case, please ask anyone of Admin permission of the repo to assign a new owner via Docs Portal https://ops.microsoft.com

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Conflicting python dependencies, e.g., onnxruntime

Hi all,
when looking into the current dependency handling of python packages, I realized that there are conflicting dependencies between the integrated tools, e.g., onnxruntime==1.5.2 vs onnxruntime==1.3.0.
The only reason that the tests have not failed until now is that parsing the python dependencies is done manually line by line (see install.sh line 18, and hence silently overwriting previous package versions, instead of using a smart way of resolving conflicts.
Is a reason for the manual parsing instead of using pip install -r python_dependencies.txt?

Is a reason for the manual parsing instead of using pip install -r python_dependencies.txt?

No, probably there is no particular reason other that the manual parsing is needed for the system dependencies and can probably be changed.
Please feel free to go ahead fixing this issue.
Otherwise I will handle it when I will have some time.

@stefaniapedrazzi, thanks for your reply. Please see PR #112 for the further discussion.


[WIP] add oblique slicing

I am scoping oblique slicing to be sampling an arbitrary 2D plane from a 3D space. Additional axes can be selected/sliced, but those are axis aligned.
This slice is defined in a slice request:
@dataclass(frozen=True)
class Oblique2DWorldSliceRequest:
    """Data to request an oblique 2D slice in world coordinates.

    For convenience, I am using numpy arrays - may use tuples in
    the future for serializabiltiy.

    Parameters
    ----------
    point: np.ndarray
        The point in nD world coordinates the plane is centered on.
        This will be the (0, 0) coordinate of the resulting 2D slice.
    normal_vector: np.ndarray
        The normal unit vector of the plane.
        This is in the 3D spatial coordinate system described by spatial_dims.
    up_vector: np.ndarray
        The up unit vector of the plane. This will be the (0, 1) direction
        of the resulting 2D slice.
        This is in the 3D spatial coordinate system described by spatial_dims.
    spatial_dims : Tuple[int, int, int]
        The axis indices of the nD world coordinate system for the
        3D spatial axes comprising the spatial coordinate system.
    slice_positive_extent : float
        The extent of the slice in the positive direction of the normal vector.
    slice_negative_extent  : float
        The extent of the slice in the negative direction of the normal vector.
    """

    point: np.ndarray
    normal_vector: np.ndarray
    up_vector: np.ndarray
    spatial_dims: Tuple[int, int, int]
    slice_positive_extent: float
    slice_negative_extent: float

This slicing operation then proceeds as follows:

convert the slice request to data coordinates
extract the 3D spatial subvolume from the data (Defined by spatial_dims in the slice model)
construct the plane in data coordinates (from point, normal_vector, and up_vector)
sample using the plane and the slice_positive_extent/slice_negative_extent
determine the offset of the data origin from the point. this will be used to set the transform. the point in world coordinates is set to (0, 0) in the rendered scene.

I haven't started adding code to the library, but I have starting writing an example script that demonstrates the slicing process: examples/oblique_slicing.py`

This is still pretty rough, but I want to get this initial infrastructure in so I can test with images in a separate PR. Definitely not fit for consumption yet 😬


translator: remove IssuingValidator #352

Signed-off-by: Angelo De Caro<EMAIL_ADDRESS>

Thanks @mbrandenburger , comment addressed.

comment addressed.

I guess you missed one :D

While the changes look good to me, it is unclear why this is removed. Can you please add some explanation why this is removed.

Sorry, @mbrandenburger . I added it to the git issue. Essentially, the validator is responsible of checking that the identity of an issuer is valid 😅


aktualizr: Update to a8e9847

Update Aktualizr to the latest version.
Signed-off-by: Leon Anavi<EMAIL_ADDRESS>

Blocked waiting for a clean end-to-end test run

I have closed this pull request because this version of Aktualizr is not stable.


Create CP Notifier Script

The Competitive Programming Notifier Script is a Python tool that alerts users about upcoming coding competitions from selected websites. It offers three notification methods:
Desktop Notifications: Instant alerts on your computer. Email Notifications: Detailed emails with competition info. SMS Notifications: Alerts sent directly to your mobile phone. Key Features:

Configurable Settings: Easily enable or disable notification methods and set up your email and SMS configurations.
Competition Monitoring: Fetches upcoming competitions, which can be customized based on user preferences.
Regular Checks: Runs continuously to ensure timely updates.

This script is perfect for competitive programmers and coding enthusiasts, helping them stay informed and manage their schedules effectively.
Pull Request Template
script name -
Brief about script
Issue no.(must) -
Self Check(Tick After Making pull Request)

[ ] This issue was assigned to me.
[x] One Change in one Pull Request
[ ] My file is in proper folder (Name of folder should be in lowercase with no space in between) (E.g. meet_schedular)
[ ] I am following clean code and Documentation and my code is well linted with flake8.
[ ] I have added README.md and requirements.txt (Include version numbers too e.g. pandas==0.0.1) with my script
[ ] I have used REPO README TEAMPLATE (Necessary)
[ ] Just including required dependencies in requirements.txt (Don't include Python version too)

If issue was not assigned to you Please don't make a PR. It will marked as invalid.

@pawangeek , I've uploaded my PR


.spec.region is not being used

Th EKSCluster schema has a spec.region field to specify the region where to create all the resources (cluster, VPN, VPC, etc.) that is not currently being used and the AWS_DEFAULT_REGION environment variable is being picked instead.
furyctl shall use the value specified in the configuration file instead of the default region environment variable.
Tasks:

[ ] update infrastructure phase terraform templates
[ ] update cluster phase terraform templates
[ ] update distribution phase terraform templates
[ ] update dependency check (don't check for AWS_DEFAULT_REGION presence)
[ ] update the schema and make .spec.region mandatory if it's not that way

related issue #293

@ralgozino I think so, we probably just forgot to do so


Default configuration can never load following documentatation

Currently, following the documentation, the default configuration will never load and will never cast an error when it fails.
Additionally when loading a custom configuration file, an error is never fired if there is an error in loading due to an invalid or incorrect path
this PR adds an error when the provided configuration file does not load, it also fixes the broken path that causes the default file to never load. It is only a partial fix because if someone moves the compiled binary outside of the root path, the default file will no longer be found (but at least it returns an error)
I also fixed linting errors

Seems like the pr-tests failed can you check and fix please?

I just merged another PR to handle this issue and fixed up the release process to do the same thing.  All versions should pull the default properly now I think.
Let me know if you have any trouble with versions v0.4.0 or greater.  Closing this as a dupe.


Error: No intercept evaluator

Hello,
I downloaded the master branch and compiled with CMake. Everything worked great but, when running a game, the agent with the ball is constantly printing the following error:
Helios_base 11: ERROR: no intercept evaluator.
Helios_base 11: ERROR: no intercept evaluator.
Helios_base 11: ERROR: no intercept evaluator.

Curiously, this error is not printed when compiling with automake, so the problem might be related to CMake compiling, maybe.
Unfortunately, I couldn't find the source of the issue...

Thanks for reporting the problem.
The missing source file has been added to the CMake build target.
Please check the latest master branch.


Use model string in ros_gz_spawn_model.launch.py

🦟 Bug fix
Replace world_string with model_string. This was renamed by mistake in https://github.com/gazebosim/ros_gz/pull/602 .
cc @Amronos
Summary

Checklist

[x] Signed all commits for DCO
[ ] Added tests
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] Consider updating Python bindings (if the library has them)
[ ] codecheck passed (See contributing)
[ ] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

Can you also update the gz_spawn action?
I think a good follow-up would be to add tests to catch these types of mismatches.

Updated in 9e63f09


Bad input position - React

As it seen on image, input its rendered in bad postion (every time same postion).
index.tsx
import React from "react";
import ReactDOM from "react-dom";
ReactDOM.render(
      <CanvasDatagrid />,
  document.getElementById("root")
);

CanvasDatagrid
const CanvasDatagrid = () => {
  const grid = useRef<HTMLDivElement>(null)

  useLayoutEffect(() => {
    if (grid.current) {
      const dataGrid = canvas({
        parentNode: grid.current,
        data: [
          { col1: 'foo', col2: 0, col3: 'a' },
          { col1: 'bar', col2: 1, col3: 'b' },
          { col1: 'baz', col2: 2, col3: 'c' }
        ]
      })
    }
  })

  return (

    <div  id="grid" ref={grid} style={{height: 300, marginTop:300}}>

    </div>
  )
}

export default CanvasDatagrid

"react": "^16.8.6",
"canvas-datagrid": "0.22.12",

I figured it out, it only happens when feature " Toggle device toolbar" is on in chrome


mod is compatible with optifabric

According to your testing, does the issue occur with ONLY Immersive Portals mod and Farbic API?
Yes
Describe the issue
my mc set with fabric ,optifine, and optifabric. and the immersive portals error with optifabric.

A potential solution has been determined:
	 - Replace mod 'Immersive Portals' (immersive_portals) 2.0.5 with any version that is compatible with:
		 - optifabric 1.13.5
Unmet dependency listing:
	 - Mod 'Immersive Portals' (immersive_portals) 2.0.5 requires version 0.28.1 or later of fabric, which is missing!
	 - Mod 'Immersive Portals' (immersive_portals) 2.0.5 is incompatible with any version of mod 'OptiFabric' (optifabric), but a matching version is present: 1.13.5!
	 - Mod 'Immersive Portals Core' (imm_ptl_core) 2.0.5 requires version 0.28.1 or later of fabric, which is missing!
	 - Mod 'QMiscUtil' (q_misc_util) 2.0.5 requires version 0.28.1 or later of fabric, which is missing!
	at net.fabricmc.loader.impl.FabricLoaderImpl.load(FabricLoaderImpl.java:190) ~[fabric-loader-0.14.8.jar:?]
	at net.fabricmc.loader.impl.launch.knot.Knot.init(Knot.java:148) ~[fabric-loader-0.14.8.jar:?]
	at net.fabricmc.loader.impl.launch.knot.Knot.launch(Knot.java:68) [fabric-loader-0.14.8.jar:?]
	at net.fabricmc.loader.impl.launch.knot.KnotClient.main(KnotClient.java:23) [fabric-loader-0.14.8.jar:?]

https://qouteall.fun/immptl/wiki/Miscellaneous.html#optifine-compatibility


The question about commit

Hello, is it possible to use jgit to commit local changes to another specified branch without switching to the specified branch?

not as far as I know of, and please use stackoverflow for questions about jgit usage as specified in the issue-guidelines:
Note: Please use sites such as http://stackoverflow.com for general questions about JGit usage, not issues in this project. Issues should be used for problems with snippets and suggestions of missing snippets. Snippets from good answers on stackoverflow can then be included here, naturally.


Documentations website not working

The documentations website not working:
http://yajrabox.com/docs/laravel-datatables

@yajra
This happens all the time, and we have asked you many times to take the community offers being given to you for others to host your documentation.
As was the case last time, it is clear that you do not have the time, or technical capability to maintain your own documentation website. This package is very difficult to use without documentation.
This time, I hope you actually take those offers. It is very frustrating for the people that use this package to have to complain on Github every time you fail to maintain your documentation website.

Thank you! Please see #2223.

i have hosted the doc in netlify if anyone need a quick browsing
https://laravel-datatables-docs.netlify.com/


Gosec changes

I think travis file needs to be updated here https://github.com/IBM/ibmcloud-storage-utilities/blob/master/.travis.yml#L34. Refer https://github.ibm.com/alchemy-containers/armada-storage-file-plugin/pull/234/files#diff-354f30a63fb0907d4ad57269548329e3L34
Also check why make block-storage-attacher is needed. If its used or not.

The travis invokes the "make block-storage-attacher"  thatsy making changes there .


Successful node pushes aren't getting logged properly

In one of the recent pushes of managed tokens, the experiment uboone had successful pushes that aren't getting logged properly, and aren't showing up in the metrics:
Successful push example from log:
time="2023-10-16T10:01:30-05:00" level=info msg="Success copying file to destination" destinationFilename=/tmp/vt_u45225-uboone_production experiment=uboone node=uboonebuild01 role=production sourceFilename=/tmp/vt_u47535-uboone_production

Aggregation message that's wrong:
time="2023-10-16T10:01:30-05:00" level=info msg="Successful nodes: uboonegpvm01, uboonegpvm02" service=uboone_production

Notice that uboonebuild01 (and other nodes) are missing from this message.
The prometheus query also shows that the successful push metric doesn't appear in prometheus:
managed_tokens_last_token_push_timestamp{experiment="uboone"}:

managed_tokens_last_token_push_timestamp{experiment="uboone", job="managed_tokens", node="uboonegpvm01", role="production", service="uboone_production"} |<PHONE_NUMBER>.7236297
managed_tokens_last_token_push_timestamp{experiment="uboone", job="managed_tokens", node="uboonegpvm02", role="production", service="uboone_production"} |<PHONE_NUMBER>.7479558

This shows that there may be a correlation between which nodes get logged as "successful" and which get metrics pushed.
ETA:  This seems to only affect cases where there was one legitimate failure (in this case uboonegpvm03).

The source is here:
https://github.com/shreyb/managed-tokens/blob/46135bbccb9cdc0f085ce0a312f350a992f86923/internal/worker/pushTokensWorker.go#L223
In this case, this clause gets executed for every successful node UNTIL there is a single failure.  Then this line:
https://github.com/shreyb/managed-tokens/blob/46135bbccb9cdc0f085ce0a312f350a992f86923/internal/worker/pushTokensWorker.go#L196
changes the pushSuccess struct value to false, which causes the first clause to NEVER get executed any longer for the worker.Config in question.
In truth, the code in that former clause should not depend on the value of pushSuccess's bool.  The latter is a whole-Config indicator, whereas the code that's not running is a per-node indicator.  So we should really be running that if there are no errors with the current node.


mosquitto image pull policy set to never?

While trying to get this working on AKS, I noticed that in the mosquitto manifest, you have imagepullpolicy set to 'Never'.
This will prevent the cluster from EVER pulling the image.
https://github.com/EdwinVW/dapr-traffic-control/blob/cd397cdc8f52758c3b5c426b5f43f795433d3168/src/k8s/mosquitto.yaml#L24

Hi @larryclaman. Thanks for your interest in this repo!
The reason why the imagePullPolicy is set to Never is because you need to build the Mosquitto Docker image locally first. This will make sure the correct Mosquitto configuration is used. The default Mosquitto image (when pulled from Docker Hub) does not work out of the box. See this paragraph in the readme.

To differentiate between the default Mosquitto image and the local one, I've added dapr-trafficcontrol as repository (see the Mosquitto K8S manifest file). Because this repo does not exist, I've added the imagePullPolicy.
For an AKS implementation, you could indeed bake your own Mosquitto image (including the config) and put it in an ACR.

Hi Edwin,
I looked into this a little more (as I continue to develop my AKS-ready version of this project) and I'm curious as to why you are creating your own local mosquitto image.  Looking at the Dockerfile, it uses the vanilla mosquitto image, but simply bakes in the conf file:
FROM eclipse-mosquitto:latest

# override default config-file
COPY ./mosquitto.conf /mosquitto/config/mosquitto.conf

Curious as to why you chose this route (using a custom image)?  eg, why not simply pass the conf file as a mounted volume (per the instructions here) as docker run -it -p 1883:1883 -p 9001:9001 -v mosquitto.conf:/mosquitto/config/mosquitto.conf eclipse-mosquitto
And for a K8S deployment, you can achieve the same thing by passing the conf file as a configmap.

Hi Larry. Yes that's also a possibility. But I want to keep it as simple as possible, because the focus of this sample is to learn Dapr. With the custom container image with the config baked-in, users of the sample can use this image in self-hosted mode as well as in K8S. They don't have to switch between specifying a volume mount in self hosted mode and using a K8S config-map when running in K8S. I also don't have to explain this in the instructions and (maybe) confuse people. This is also the reason I don't include a version running on AKS (or Azure Container Apps or ...) in the sample. Those other ways of hosting the sample are cool to have. But the focus should stay on learning Dapr. I hope you understand my considerations in this area.


Order of find keywords matters

Stated in UG:

The order of the keywords does not matter. e.g. Hans Bo will match Bo Hans
Current behaviour: order of keywords matter
https://github.com/AY2324S1-CS2103T-T11-3/tp/assets/122242919/e3482f1f-a151-47ea-bad5-bc7e523ccef7

Other behaviour that is not intended (based on ug):

Only full words will be matched e.g. Han will not match Hans
Command: find dav

closing - changing behaviour of find command


Disable commits in simple consumers

closes #90

Alright, this is it. I think it (currently) is not possible to bypass the group authorization mechanism with the confluent consumer. Therefore the Kafka user still needs to be authorized to use the corresponding group ids.
We might also want to add a "--resumable" keyword to the "consume" command in the future in order to make use of committed offsets if desired.
Shall we consider renaming the default group id a breaking change? The users need different authorization after all... we could also think about making this configurable via the esque config.

@hfjn you okay if we merge this? :)

Nice one! Merge away.


Add image building profile

With this I can build an image using buildkit.
It's requesting the host networking folks because i just ran make protoc and it updated their protos as well. I can drop those hunks from the patches if it's annoying.

I'm file with the change in vpc/api/vpc.pb.go. Nothing but a a version bump. Will leave the rest of the review to Kyle.

I hope you didn't spend too much time fighting overlay.
I had already known what does and does not work from here: https://stash.corp.netflix.com/projects/TN/repos/titus-integration-tests/pull-requests/514/overview
But you came to the same conclusions, which is great!

I had already known what does and does not work from here: https://stash.corp.netflix.com/projects/TN/repos/titus-integration-tests/pull-requests/514/overview

Yeah, I knew it as well in principle. In practice it took a while to remind myself because the error from buildkit was so bad. But I hadn't seen your PR there at all, I made a very similar one, just using a tmpfs:
https://stash.corp.netflix.com/projects/TN/repos/titus-integration-tests/commits/173b02319e67bf6a9863f39cf83d73ccf4a4acf5
https://stash.corp.netflix.com/projects/TN/repos/docker-files/commits/baaf48b703b4daec051d0a9b5d8300687f59efec
I did use figuring this out as an excuse to write: https://github.com/tych0/kernel-utils/blob/master/why-did-mount-fail.bt which is a tool I've always wanted.


etcdctl v3: Not able to verify transaction example via CMD

I am not able to figure out how to verify the transaction example given in Readme.md of etcdctl via the command prompt.
On trying to execute the example i am getting the below error:

Can someone suggest how to verify it.

@sinsharat txns need newlines between the condition/success/failure blocks (this is given in the documentation).
$ ETCDCTL_API=3 ./bin/etcdctl txn -i
compares:
mod("key1") > "0"

success requests (get, put, delete):
put key1 "overwrote-key1"

failure requests (get, put, delete):
put key1 "created-key1"
put key2 "some extra key"

FAILURE

OK

OK

@heyitsanthony I had tried that as well, but the moment i put a new line it gives the same error.

Please let me know what i am doing wrong.
Thanks.

@sinsharat Seems like we need Windows-specific code to read new line character
https://github.com/coreos/etcd/blob/master/etcdctl/ctlv3/command/txn_command.go#L76
\n is not going to work in Windows, I think.
Do you want to try fixing it? Think we just need to define readCompares separately in txn_command_windows.go.

@gyuho  sure will look into it and try and fix the issue.


Error when selecting a folder

When I select a folder in the dialog, app crashes with:

Caused by: java.lang.ClassNotFoundException: Didn't find class "org.apache.commons.io.FilenameUtils" on path: DexPathList[[zip file "/data/app/com.myApp-F0_sdB_QRzZKIKky8gOrAA==/base.apk"],nativeLibraryDirectories=[/data/app/com.myApp-F0_sdB_QRzZKIKky8gOrAA==/lib/arm64, /data/app/com.myApp-F0_sdB_QRzZKIKky8gOrAA==/base.apk!/lib/arm64-v8a, /system/lib64, /vendor/lib64]]

I have apache commons library imposed in my project.
Any ideas what could cause that?

Sorry, had wrong commons imported

PLease add to description that library also depends on 'org.apache.commons:commons-io:1.3.2'

thank you i have added the library dependency in the description


Update intro-notebook to allow figure window resizing in colab

Description
The fix for #21 should be incorporated in the intro-notebook.
Source code
I have added the following in the notebook and will submit it in new pull request.
## Run this cell once and then
## comment it out after first run
import leafmap
leafmap.update_package()
print("Restarting notebook server...")
import os
os.kill(os.getpid(), 9)

Suggested change

leafmap_update_package() is only needed for new features that have been released to PyPI and conda-forge. Running leafmap_update_package() in every notebook is discouraged as it will reinstall all the dependencies and this could result in issues for some users.


Sanitize stat strings by removing curlies

@Raynos @kriskowal
including curlies like {uuid} in stat key makes Graphite sad.

This explains why there was no data in graphite because the curlies make graphite sad.

recommended: patch version
recommended: update typed-request-client on master of sirvice & patch version sirvice.


Added ChipTemplate to DropDownDataGrid

Chips only had effect if no Template was present (line 41).  And the markup for when Chips was active was trying to use template (line 50).
Fixed the confusion by adding a new template to personalize the rendering of the chips...

Thanks for the pull request @stlufred! I think that we should fix this to work like the DropDown component where ValueTemplate and Template are used if defined for chips: https://github.com/radzenhq/radzen-blazor/blob/master/Radzen.Blazor/RadzenDropDown.razor#L44-L55
I'll push the relevant code and I'll close this.


Add DnsCrypt Protocol

Check out the documentation and faq.

This would be nice to have alongside Pi-Hole. Maybe one for the app store, I wouldn't want this hard-coded like TOR.
https://github.com/getumbrel/umbrel/blob/master/apps/README.md#1-containerizing-the-app-using-docker


Fatal error activating plugin.

When I install any to the plugins after version 5, I get the following error.
Fatal error: Can't use function return value in write context in /var/www/html/testsite/wp-content/plugins/static-html-output-plugin/library/StaticHtmlOutput.php on line 352
This means I am stuck on version 2.4.

Using http://www.meandeviation.com/tutorials/learnphp/php-syntax-check/v5-3/, I'll allow activation for php 5.3 but warn to the user

fixed in version 5.8 (allow activation, but present user with notice to upgrade PHP)


Use filters.json to generate a type-safe Swift CIFilter library

This has been suggested by several people - since we have all the filter data, we could autogenerate Swift code for type-safe applications of the filters, instead of the current solution of string typing.
A good way to go about this could be to put the generated files in a new folder in this repo, and have a TypeSafeCIFilter.podspec in the root which references them

Hi, thanks a lot for building this tool! A full-featured library is a massive work, how about providing simple code syntax-highlighted snippets right on the website?

Definitely! Are you thinking of something like in #40?

Sounds great, yes!

As of iOS 13+, there seem to be classes available for each filter, e.g. this one for CIBokehBlur. Though the documentation is still in beta, it's reasonable to assume that these classes will take the place of any type-safe library that we would generate. Going to close this issue for now.


feat: support rollup plugin this.load in plugin container context

Description
This adds support for calling this.load (added in Rollup 2.60) in a rollup plugin when using a plugin context via Vite's plugin container (namely, when using the dev server).
Resolves #6810
Additional context
This fixes the issue I have in my specific use case (rollup-plugin-typescript2 needing to call this.load, though not using any of its output - I tested this PR locally with that test case), and is strictly more functional than the previous lack of support. However, there are some limitations which I don't know if it's appropriate to address or otherwise present issues:

The return of this.load from Vite will be a ModuleInfoProxy, which doesn't include most of the properties from ModuleInfo. I imagine some plugins are likely to be calling this.load expecting some of these parameters to work. That said, this is already an existing limitation of getModuleInfo, so I assume that is out of scope.
Further, this.load is supposed to trigger transform and moduleParsed - this does not (in accordance with vite's on-demand nature)
The resolveDependencies parameter is left unused, as since the modules are not actually transformed/parsed, from my understanding dependency information is unavailable/irrelevant
ModuleOptions properties other than meta are unused (though as far as I know, that's the responsibility of updateModuleInfo, and so out of scope, and presumably unnecessary in the on-demand situation)

I don't have much experience working at this level, so feedback is definitely appreciated, and I'm happy to make any adjustments

What is the purpose of this pull request?

[ ] Bug fix
[x] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the PR Title Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[x] Ideally, include relevant tests that fail without this PR but pass with it.

Sorry for attending this 2 months late 😬 Let's get this in 4.2


Logical operations between QueryDescriptor objects are not allowed

Elastic.Clients.Elasticsearch version:8.11
Elasticsearch version:8.11
.NET runtime version:4.8
Operating system version:Win 11
Description of the problem including expected versus actual behavior:
In previous versions (7.x) the QueryContainer objects could be used for logical operations like && and || example:
QueryContainer container = container1 || container2;
In 8.x this is not working. The QueryContainer seems to be replaced with QueryDescriptor, which is fine, but the logical operations implementation is missing. Is this a bug, or something that will be added in the future, or is it replaced with some other approach?
Steps to reproduce:

Try to construct a QueryDescriptor from two other queries with logical operations

Expected behavior
The operation to be allowed, or documentation to be available on how to migrate such code.
Provide ConnectionSettings (if relevant):
Provide DebugInformation (if relevant):
### Tasks

Hi @bonny-bonev, some of the convenience features are not yet ported over from 7.x to 8.x. My primary focus is on endpoint support at the moment (there is still a lot of functionality missing in 8.x). Afterwards I definitely plan to bring back some of these usability imrovements.

And operators overloads*

Is there any eta when this might be completed?


Make data set available in Python package

Hi - this is a feature proposal for the Python package portion of xgboost. If there is interest, I would be happy to try implementing and provide a PR.
Details:
The Getting Started documentation shows an example of loading the "agaricus" data set by using a file path.
Link: https://xgboost.readthedocs.io/en/stable/get_started.html
I believe the use of a local-specific file path for the data provides unnecessary friction for Python users when initially trying out xgboost. Hence, the enhancement idea is to include the "agaricus" data set in the python package. This might work similar to how sklearn provides data sets, e.g. data = sklearn.datasets.load_diabetes(). For xgboost the interface might look something like data = xgboost.datasets.load_agaricus().
I would look forward to any feedback on this feature proposal.

I think it might be better to change that getting started script to use some other easier-to-access datasets instead of copying the data into XGBoost Python wheel.

@trivialfis  Thanks for your thoughts! So long as there is no specific reason to use the "agricus" data set, that sounds good to me.
Two implementation options might be:
Option 1: Just use sklearn.datasets directly to get a data set that works well for showcasing xgboost.
Option 2: Download a data set from the web, possibly by using something like wget via Python
Additional Thoughts:
Option 1 is probably the best/quickest way to do it. However, if we want to keep the data set in sync for all examples (now or perhaps down the road), then we would also have to make sure there is an analogous approach for the other programming language options for the chosen data set. I'll try to look into that.
Option 2 has the disadvantage that it requires having a reliable data source, and for new users to trust that data source. Also, it ends up using the user's file path which will likely require the user to do a slight modification before running the code. However, it's probably a more programming language-agnostic method.

I think sklearn.datasets is an excellent choice. As for other languages, the XGBoost R package has the data packaged already so it should be fine.


On macOS when launching multiple times focus win

I dug into this and narrowed the issue down to the macOS app bundle
Info.plist file. So, I spun up a native macOS app real quick and tested
it by launching the binary directly and launching it via the app bundle.
When launching from the command line directly, it created multiple
windows & instances of the app. However, when launching via the app
bundle it behaved as I normally expect a macOS app to behave, that is
when launched multiple times to simply focus the already existing window
and instance.
This informed me that it wasn't something in code as much as it was
something in the app bundle configuration. Hence, I reworked the
Info.plist file based on the one that was created by XCode when I made
the native macOS app and it started behaving as expected.
The downside is that the ALACRITTY_LOCALE, LANG, and LC_CTYPE are not set as
they would be via the launcher script. Also, the PWD starts out at / rather than
the users home directory. The LANG, LC_CTYPE environment variables can be set in
your config under the env section. It isn't dynamic which still isn't quite as
nice as the bash + applescript approach. However, it make the behavior work
correctly which is critical for anyone using an app launcher like Spotlight to
switch apps.

@jwilm I made the requested changes, back merged master into my branch, resolved the conflict, and testing things. They seem to be good.

Thanks!

np

So, now, the only way to open more than one window of Alacritty is... open -n /Applications/Alacritty.app (assuming I've "installed" it)?

You can also do open -a -n ‘Alacritty’. But, yes, this is the
intended behavior as this is how macOS apps behave. If an app supports
multiple windows, it is a feature of the app not that the app is
actually running multiple times.
On 16 Dec 2017, at 19:51, Steven Lu wrote:

So, now, the only way to open more than one window of Alacritty is...
open -n /Applications/Alacritty.app (assuming I've "installed" it)?
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/jwilm/alacritty/pull/941#issuecomment-352230023

open -a -n Alacritty does not work. it tries to look for the app in the cwd, so -a seems superfluous

Sorry Alacritty needs to follow the -a and I think be in single quotes or double quotes

Very interesting. When I do that, it tries to run /Users/me/Documents/alacritty/assets/osx/Alacritty.app which fails as it is missing its executable. Seems like lots more can go wrong so I will just use the absolute applications path without -a :)

Sounds like Spotlight has indexed your the Alacritty.app in /User/me/Documents/alacritty/assets/osx/Alacritty.app which you don't want.

indeed!
spotlight is generally not helpful (eating up your CPU indexing your FS) except for the times that it does turn out to be helpful, so I would never use open -a personally, after seeing how it behaves...
my advice would be to use the open -n /Applications/Alacritty.app bound to a key to do it, now that using spotlight to spawn Alacritty no longer works. Or, if you don't install Alacritty to Applications, just point it to the absolute path to where it lands in target/release/osx after you make app.

so... how does one toggle between multiple open instances of Alacritty on MacOS.
cmd + ~ doesnt work, I'm assuming I have to bind that to something?

multiple open instances must be switched as though they're different apps. when alacritty properly supports multiple windows in one instance, then cmd + ~ will work.

This is related to #607, #401, #674. I would like to announce that I have a functioning workaround for cmd + backtick.
You may be able to understand why this works. It's not elegant, but works a damn treat.
#!/bin/bash
# open the next program that matches a given process name, if pid of program
# owning the active window is found in the pgrep list, go to the next one in
# the list.
# ME is the pid corresponding to the process owning the focused macos window.
ME=$(osascript -e "tell application \"System Events\" to get unix id of first process whose frontmost is true")
# List of targeted pids
TARGETS=$(pgrep $1 | tr '\n' ' ')
echo $ME -- $TARGETS
TARGETS="$TARGETS$TARGETS"
# construct doubled list to ensure the first instance of ME will not be the
# last. Then focus the very next one.
STRIPFRONT="${TARGETS#*$ME }"
TARGETPID="${STRIPFRONT%% *}"
osascript -e "tell application \"System Events\" to set frontmost of the first process whose unix id is ${TARGETPID} to true"

I stash this in my shell scripts dir. You can try it out, it basically will tell OS X/macOS to focus the next alacritty window. One instance of the process corresponds to one window (the way winit works now) Simple as that. My keybinding, as an example:
  - { key: Grave,    mods: Command, command: { program: "/Users/slu/util/macos-open-next-program.sh", args: ["alacritty"] } }

I'm not sure right now if its something I messed up on my end or not but when I recompiled master this trick in my config file no longer works. Will report back when I get around to fixing this.


Jbuilder not building bytecode and native executable at the same time

My jbuild is as follows
$ cat jbuild
(jbuild version 1)
(executable
  ((name   hello)
    (public_name hello)
    (modes   (byte native))
    (flags (:standard -safe-string))))

$ cat hello.ml
let () =
   print_string "hello world\n"

$ cat hello.opam
opam-version: "1.2"
build: [
     ["jbuilder" "build" "--only-packages=odocopt" "--root" "." "-j" jobs "@install"]
]

depends: [
  "jbuilder"        {build & >= "1.0+beta8"}

However, executing command jbuilder build only builds hello.exe and not hello.bc. Shouldn't having (modes (byte native) create both hello.exe and hello.bc in the same jbuilder build invocation? I can generate hello.bc but only after removing native from the modes list.
The manual seems to suggest this is the expected functionality, i.e. when there is (modes (byte native) jbuilder will build both and not favor native at the expense of byte.

If you look at the .install file generated, I think you'll find that it's only installing hello.exe? If you want jbuilder build [@install] to build hello.bc as well, I'm fairly sure you need to add an install stanza to your jbuild file.

Basically (modes ...) is used to describe what can be built. Maybe the doc is not very clear. The idea is that if you write (modes (byte)) then jbuilder will not try to build the native version and will fallback to the byte-code version.
Regarding building both at once, you can ask for it explicitely:
$ jbuilder build hello.{exe,bc}

The @install target will build only things that are to be installed and only one of the native or byte-code version gets installed for executables. If you want to install both, you indeed need to add an install stanza.

$ jbuilder build hello.{exe,bc}

Exactly what I was looking for. Thanks. 👍


Ability to print

VSCode Version:
OS Version:

Steps to Reproduce:

As a workaround, you could copy-paste to e.g. Word and paste from there (the syntax highlighting will be maintained).

Yes good idea... but it would be better off course to print directly from vs code

I have to say this is an idiotic trend with newer text editors. SublimeText, VScode, and Atom all do not have the ability to print. How the hell is this not considered during implementation of a text editor??? People us text editors for notes and other reasons. Not having the feature to print is completely idiotic. The software is incomplete. How about releasing software when it is actually finished. Printing is a standard across most modern software. Hell even Notepad++ has a print feature. Notepad it self has a print feature. Is this really that hard to understand???  No, I'm not going to install some half-assed plug in to fix this issue. The Printer should be native. Copying from one text editor to another in order to print is completely retarded.

https://github.com/Microsoft/vscode/issues/5953


Won't intall with Luks selected

It won't install with Luks.  Always get stuck after partitioning and won't write to the encrypted volume.
Thanks for all the work put in.
Branko

Yeah - the issue is that, after the luksFormat the system is still being strapped to ${partition3}, rather than the cryptdevice.
@ChrisTitusTech If luks is being used, the ${partition3} invokes should be switched to /dev/mapper/ROOT in scripts/0-preinstall.sh on a number of lines.


Add syntax highlighting to code examples

Add syntax coloring so examples are more readable

Added in 0.1.0


[FEATURE] Writing frames to content store even before kinesis video stream initialization.

Is your feature request related to a problem? Please describe.
Writing frames to hybrid content store won't work if during start of the application, there is network offline.
Describe the solution you'd like
We are using the kvs gstreamer sample application for the video streaming application.
We do want to leverage hybrid content store mechanism in our application.
Currently we are able to utilize hybrid content store and it is working fine.
Suppose at the start of application, there is no network connectivity then kinesis video initialization fails.
Kinesis video SDK needs a unique pointer to write frames to content store. Due to this limitation we will loose the frames.
Can we bypass this limitation and write frames to content store from start of the application?

@AnilNS Can you elaborate more on what you are using for the hybrid content store mechanism.
I am looking to utilize RAM / Disk for buffering when the device is offline as well.
I have experience with C++ and could make these changes to this library if we define the requirements properly.

This is a relatively easy feature to implement with the main hinderance being the state machinery. It would be great to perhaps capture the full details and what edge cases are with this feature and why this is important to implement in the SDK rather than the application layers. For example, the SDK has no way of knowing when the network is back on/available as it's application and platform specific. I remember a while back one of the patrons had a mobile application which would create a temp storage and store the frames (with the metadata). The application was also listening on the airplane mode and would reply the collected frame buffer into the SDK with their timestamps faster than realtime. Perhaps sub-optimal for sure but nevertheless a solution.


fix: timestamping

Due to implementation needs, we must provide a time.Time when verifying TSA certificate chain validity after signing. Originally, this time value was taken from the timestamp countersignature envelope itself.
This PR switches to use the current time when verifying TSA certificate chain validity since the tsa cert chain is expected to be valid during signing.

This PR switches to use the current time when verifying TSA certificate chain validity since the tsa cert chain is expected to be valid during signing.

Is this true for verification ?

Is this true for verification ?

This PR is for signing only. We will have another PR for verification in notation-go once this PR gets merged.


[!] No streaming URL has been found...

I get results after a search but trying to play results in the error [!] No streaming URL has been found.... My youtube-dl version is 2020.05.29 (which is latest). Was working till few days ago, didn't change anything.

With what version of youtube-viewer?

3.7.5. Which sadly isn't the latest.
I haven't got the updates. I am using Manjaro KDE.

It should be fixed in 3.7.6. (https://github.com/trizen/youtube-viewer/commit/b2ac969756ed1e7d3d81c9c4d4ec3a740deadc1f)

I'll just get it from aur. That should fix the issue.