DRILL-6570: Fixed IndexOutofBoundException in Parquet Reader

Reserving same size intermediary buffers to handle the case of false-positive; that is, a column is first thought to be fixed length (after sampling), then reverted to variable length as we read more values.

@kkhatua, can you please review this fix?
Thanks!

@sachouche  The change looks simple, but I might be missing the bigger picture. For e.g., ColumnPrecisionInfo.precision is passed as an unused argument. Not clear as to why you would not need it still.
Perhaps someone else who is more familiar could review this.

The reason I didn't remove the ColumnPrecisionInfo.precision is that I am allowing myself another chance, in the future, to optimize the fixed length path; at this time, the savings are very less (about 2kb). Though, I can remove this input object if it helps code clarity.

The IDE (or Maven, itself) might report this as unused. It might be worth mentioning as a that this is a placeholder for the future.
That said, I'd still advice @Ben-Zvi  or someone else familiar with the original PR review this.


Feature/remove dev docker

What does this PR do?
Removes the unnecessary docker-dev task from CI

:shipit:


It used the wrong field to determine whether notification has been seen

Resolves #94
Can't test cuz Idunno how to send notifications. But pretty sure this solves it because:
To mark a notification as viewed, the route PATCH /anvandare/notifikationer/<notificationId>/visit are called by both web and app upon clicking on a notification.
Looking at the web, it used the visited field of the notification to mark as viewed or not.
App used seen which seems to be an old field as seen in app/controllers/api/notifications_controller.rb

This is consistent with the old app. LGTM


[stable/rabbitmq]: adds podLabels to values.yaml

Signed-off-by: Michael Hill<EMAIL_ADDRESS>
What this PR does / why we need it:
This PR adds the ability to add custom pod labels to the pods created in this chart. This can be useful when defining custom network policies that need to be applied to consumed charts.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

@juan131 @desaintmartin minor version bumped!

/ok-to-test
/lgtm


Question : 2nd Latent output

Can you provide more information on the 2nd latent output from the brushnet node?  It (and the negative output) don't appear in the example images in the readme.md

I am working on PowerPaint implementation. These outputs will be needed there. They are conditioning, not latents. You should delete Brushnet node from workflow, then add it again and you will see them correctly. Now they are just copied from input, so you can feed them directly to KSampler to lessen the number of connecting lines.


Problem description
I am using python 3.9.10. I cannot use any later version. The whole software I am using is this python version dependent... and I do not want to go back using pybind11 (which is awesome but nanobind is awesomer)
I did this that compiled nanobind:
conda create -n my_env python==3.8.16 pypy=7.3.11
pip install -e .

But wouldn it break the compatibility with C API?
The nanobind examples requires 7.3.10 PyPy version, but the 3.9.10 python can only install 7.3.8 version.
Because of this I get error:
  133 | #    error "nanobind requires a newer PyPy version (>= 7.3.10)"
Reproducible example code
Default nanobind example with conda environment with python 3.9.10

I tried changing this
#if defined(PYPY_VERSION_NUM) && PYPY_VERSION_NUM < 0x07030800 // 0x07030a00

but then I get this:

5/13] Building CXX object CMakeFiles/nanobind-static.dir/home/petras/anaconda3/envs/tvf/lib/pypy3.9/site-packages/nanobind/src/common.cpp.o
      FAILED: CMakeFiles/nanobind-static.dir/home/petras/anaconda3/envs/tvf/lib/pypy3.9/site-packages/nanobind/src/common.cpp.o
      /usr/bin/c++ -DNB_COMPACT_ASSERTIONS -I/home/petras/anaconda3/envs/tvf/lib/pypy3.9/site-packages/nanobind/ext/robin_map/include -I/home/petras/anaconda3/envs/tvf/include/pypy3.9 -I/home/petras/anaconda3/envs/tvf/lib/pypy3.9/site-packages/nanobind/include -O3 -DNDEBUG -fPIC -fvisibility=hidden -ffunction-sections -fdata-sections -fno-strict-aliasing -MD -MT CMakeFiles/nanobind-static.dir/home/petras/anaconda3/envs/tvf/lib/pypy3.9/site-packages/nanobind/src/common.cpp.o -MF CMakeFiles/nanobind-static.dir/home/petras/anaconda3/envs/tvf/lib/pypy3.9/site-packages/nanobind/src/common.cpp.o.d -o CMakeFiles/nanobind-static.dir/home/petras/anaconda3/envs/tvf/lib/pypy3.9/site-packages/nanobind/src/common.cpp.o -c /home/petras/anaconda3/envs/tvf/lib/pypy3.9/site-packages/nanobind/src/common.cpp
      /home/petras/anaconda3/envs/tvf/lib/pypy3.9/site-packages/nanobind/src/common.cpp: In function ‘PyObject* nanobind::detail::obj_vectorcall(PyObject*, PyObject* const*, size_t, PyObject*, bool)’:
      /home/petras/anaconda3/envs/tvf/lib/pypy3.9/site-packages/nanobind/src/common.cpp:314:26: error: ‘PyObject_VectorcallMethod’ was not declared in this scope; did you mean ‘PyObject_VectorcallDict’?
        314 |     res = (method_call ?

      *** CMake build failed

Please don't open issue tracker tickets for things that aren't obviously bugs with nanobind.
Regarding your questions: Older versions of PyPy segfault when used with nanobind. This check warns you about API incompatibility and (if you were to fix the API issues) undefined behavior. There is no solution to your request, you will either need to give up PyPy, change the Python version requirement, or switch back to pybind11.

Thank you for answer, I will keep my questions in discussions.
For now I see there is no solution for my questions.


Last and next holidays in channel list

X days since last holiday (holiday name and date)
X days until next holiday (holiday name and date)

http://www.elproximoferiado.com/

https://github.com/trhura/clojure-humanize#datetime


Backport same PRs as done in bitcoin#11445

This PR backports the same PRs that were originally backported in bitcoin#11445. All PRs are backported from the original PRs and not from the commits found in bitcoin#11445. This ensures that we don't miss something.

@UdjinM6 dropped the TX_WITNESS tests


Delete TEHIK_VACC12.png

How is it possible that this pull request gets merged even if the checks are failing?


How to install Elasticcrawler?

I installed Elasticcrawler.But it's failed.Can you give me a detailed course about installing Elasticcrawler.

Hanasian, Elasticcrawler configuration is described on the project home page at https://github.com/Seegnify/Elasticcrawler. I can update it once I know a bit more about your case. Can you tell what commands you executed and what output you got? Can you provide logs?


Question about license and contribution

Hello, I see there's no license in this repository. I made a fork and added MIT license like stated in .gemspec, because I would like to refactor it and to add a few more methods, so i can use it. Is that ok?

Hi @alx3dev! Thanks for the question.
Sure thing. When you finish you can also make a PR to add these changes to this repo.
Cheers

Sure, I will. For now, I simplified message.rb and info.rb. I will upload it once I am done, and make PR once I test it, since I need it for production use.

Here you can see what I did so far. I removed modules and made classes. Client is responsible to handle token, name avatar and to send viber request. RubyVibe extend Client, so user methods stay there, and they call viberize with arguments.
Method viberize is private, responsible for:

first argument is viber API url -> stay in URL module (URL::SEND_MESSAGE)
it also accept options hash - that is added to payload hash
if info: true argument is added, payload is created without name and avatar

All that looks good. (y)
On Tue, Jan 11, 2022 at 12:15 PM alx3dev @.***> wrote:

Here you can see what I did so far. I removed modules and made classes.
Client is responsible to handle token, name avatar and to send viber
request. RubyVibe extend Client, so user methods stay there, and they call
viberize with arguments.
Method viberize is private, responsible for:

first argument is viber API url -> stay in URL module
(URL::SEND_MESSAGE)
it also accept options hash - that is added to payload hash
if info: true argument is added, payload is created without name and
avatar

—
Reply to this email directly, view it on GitHub
https://github.com/ademdc/ruby-vibe/issues/1#issuecomment-1009861258,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEZ5U4UL2UA5G772DKNIHRTUVQGOZANCNFSM5LUGS4LQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you commented.Message ID:
@.***>


upgrade to django 4.2.14 (from 2.2.20)

Rationale

Django 2.2 has been eol for a long time, while 4.2 has extended support until early 2026

Changes

update django to 4.2 and other libraries such as social-auth-app-django so that they're compatible with the new version
change code so that it's compatible with django 4.2

[!IMPORTANT]
#79 will need to have merge conflicts resolved after this PR is merged

coverage: 53.359% (+0.2%) from 53.129%
when pulling ccce200d85340e426d4440b46a20e8e28035f44c on aarushtools:upgrade-django
into f908c9a8e03e036571b34abf6732a47a0abd5841 on tjcsl:master.


New command: m365 pp pipeline get

Usage
m365 pp pipeline get
Description
Gets information about the specified Power Platform pipeline
Options

Option
Description

-i, --id<id>
The id of the pipeline to retrieve.

-e, --environmentName <environmentName>
The name of the environment where the pipelines are located.

--asAdmin
Run the command as admin and retrieve Power Pages pipelines for environments where you do not have explicitly assigned permissions.

Examples
Retrieves a pipeline by ID the given Power Platform environment.
m365 pp pipeline get --id d87a7535-dd31-4437-bfe1-95340acd55c5 --environmentName Default-d87a7535-dd31-4437-bfe1-95340acd55c5

Retrieves a pipeline by ID the given Power Platform environment as admin.
m365 pp pipeline get --id d87a7535-dd31-4437-bfe1-95340acd55c5 --environmentName Default-d87a7535-dd31-4437-bfe1-95340acd55c5 --asAdmin

Default properties
N/a
Additional Info
You can have a look at the logic behind: https://learn.microsoft.com/en-us/power-platform/developer/cli/reference/pipeline and/or https://learn.microsoft.com/en-us/power-platform/developer/pipelines/table-reference

if we would be able to specify the exact API endpoint which should be used then we could add this as 'good first issue'
other than that lets open it up 👍
awesome work 👏


Emag be quiet

About the PR

Closes #20641
Makes the text that shows up when you emag something only visible to the emagger
Why / Balance

See #20641
Technical details

Changes the PopupClient (shows up on all clients) to PopupEntity (shows up on one person only)
Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:cl:

tweak: EMAGs no longer show the emag text to literally everyone

Yeah this is not an issue, the emag isn’t supposed to be completely silent and unnoticeable

not an issue as in it doesnt happen

apparently my eyes decieveth

This screenshot is with two side-by-side clients. I confirm that "nothing happens".


API toggle for assertions

https://docs.oracle.com/cd/E19683-01/806-7930/assert-4/index.html
This should be fairly straightforward to implement; implement a method on the JVM object that takes a class name, and returns true/false if assertions should be enabled. Change java.lang.Class.desiredAssertionStatus0 to call this method.
Once done, we should see if it's feasible to enable system assertions during our unit tests, as the JDK has a number of assert statements scattered about its class files.

OK, the assertion flag now works.


Update migrating-oauth-apps-to-github-apps.md

These changes are to include the 'Delete OAuth App' step in 'Migrating OAuth Apps to GitHub Apps'

Why:
Closes [issue link]

What's being changed:

Check off the following:

[x] I have reviewed my changes in staging (look for the latest deployment event in your pull request's timeline, then click View deployment).
[x] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

``
Duplicate of #

These changes are to include the 'Delete OAuth App' step in 'Migrating OAuth Apps to GitHub Apps'
Why:
Closes [issue link]
What's being changed:
Check off the following:

[x]  I have reviewed my changes in staging (look for the latest deployment event in your pull request's timeline, then click View deployment).
[x]  For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ]  This pull request impacts the contribution experience

[ ]  I have added the 'writer impact' label
[ ]  I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

Hey I m mike

Hello mike

I'm need coins free in application sango please


No admin interface for profile

There appears to be no admin interface to view/edit the telephone number associated with the user via the user/profile url.

Fixed this and put it in a pull request for django_classified
https://github.com/slyapustin/django-classified/pull/51


raftstore: calculate slow score periodically

What problem does this PR solve?
Problem Summary:
Address #10539.
What is changed and how it works?
What's Changed:

Add a tick to pd worker, inspect raftstore latency and calculate slow score periodically.

Related changes

Need to cherry-pick to the release branch

Check List
Tests

Unit test
Integration test

Release note
Introduce a mechanism to detect slow tikv store

/run-all-tests

LGTM.

/merge

/merge cancel

/merge

/merge


Does this work with Twig files?

It doesn't seem to.
As an example:
    <div class="relative flex items-center "
         x-data="youtube()"
         x-init="init('{{ youtube_id }}',
				{
					autoplay: true,
					remember: false,
					aspect_ratio: {{ aspect_ratio }}
				})"

attribute x-data is not allowed here
attribute x-init is not allowed here
Plugin version 0.0.6 installed.

Hm. I don't use Twig so I don't really know. Feel free to submit a PR!

Old thread, but i am noticing the same thing. No experience with plugins to open a PR


Fix environment variable name

This PR contains:

[ ] New features
[ ] Changes to dev-tools e.g. CI config / github tooling
[ ] Docs
[x] Bug fixes
[ ] Code refactor

What is the current behavior? (You can also link to an open issue here)
#246
What is the new behavior?
The correct name should be INSPECT_EVAL_MODEL_BASE_URL, not INSPECT_MODEL_BASE_URL.
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
No
Other information:

Thanks very much - an update to the VSCode extension including this fix will be available later today!


[Feature] Threaded collection and parallel envs

Description
This PR proposed to automatically adapt the number of threads of parallel envs and collectors based on the number of workers that are required.
For each, the workers are set to num_workers on the parent env and to 1 (if not specified otherwise) on the children.
It solves the problem that all workeres were asking for as many threads as the number of CPUs on the machine, thereby maxing the cpu workload.
For a parallel env (N workers) within a collector (M workers), what will happen is that the main process will have M processes, and create workers that all have 1 thread. Each worker will then reset its number of threads to N, and create sub-processes that each have 1 thread only.
cc @skandermoalla

Quick check: what would be the default behavior of a SyncDataCollector with a ParallelEnv passed to its constructor?

Quick check: what would be the default behavior of a SyncDataCollector with a ParallelEnv passed to its constructor?

Good question
Open your script, import torch -> process has X threads (X = num of cpus)
create parallel env -> process threads reduced to N (N = num workers of the parallel env)
sync data collector -> no effect, still N threads
launches N procs, each with 1 thread

@matteobettini @skandermoalla I don't really see a proper way of testing this, except with (pointless?) mocks
Agree to leave it as it is?

Ok, I implemented some tests!

Thanks so much for this. Will test with my setup and give feedback!

@skandermoalla @matteobettini @Vittorio-Caggiano
With my setup, I get roughly these numbers in terms of throughput (on #1539 + this)
Always 32 procs, a machine with 96 cpus and 1 A100
Atari (Pong-v5):
TorchRL ParallelEnv.rollout: 10281.7697
TorchRL ParallelEnv within SyncDataCollector (to test collector overhead): 8217.6762
TorchRL GymEnv + gym async env + env.rollout: 9554.7727
TorchRL ParallelEnv (N=4/worker) across 8 collectors, async: 22023.7621
TorchRL GymEnv + gym async env (N=4/worker) across 8 collectors, async: 17997.9470
TorchRL ParallelEnv (N=4/worker) across 8 collectors, async: 10806.8827
TorchRL GymEnv + gym async env (N=4/worker) across 8 collectors, async: 10141.7132
Myo:
TorchRL ParallelEnv.rollout: 12346.7140
TorchRL ParallelEnv within SyncDataCollector (to test collector overhead): 10127.3826
TorchRL GymEnv + gym async env + env.rollout: 7444.5473
TorchRL ParallelEnv (N=4/worker) across 8 collectors, async: 18115.6079
TorchRL GymEnv + gym async env (N=4/worker) across 8 collectors, async: 14534.9367
TorchRL ParallelEnv (N=4/worker) across 8 collectors, async: 12997.6912
TorchRL GymEnv + gym async env (N=4/worker) across 8 collectors, async: 11727.7253
I can (and will) run some more extensive benchmarks.
To me the most instructive take is that (beyond the fact that we're now way more competitive) the sync data collector is slower than plain env.rollout so there is some overhead there that needs to be taken care of. The other takeaway is that sync data collectors don't really help here
Anyhow, good progress, team!

Thanks a lot for this! It'd be great if the code of the benchmarks is also made available so that we can compare.


remove "credentialed client" from terminology section

The latest versions of OAuth 2.1 no longer use the term "credentialed client". The term "credentialed client" shall be removed from the bullet list in section <IP_ADDRESS> Terminology.
This does not affect any specifications given in the IUA Trial Implementation and can safely be removed.

Done in branch CP 1288 branch.


Integrated ESLint workflow

Closing pull request due to Github Actions having everything that we need to run eslint out of the box now


The chatCompletion operation does not work with the specified model appears when I use AzureChatOpenAI, but it does exist

Issue you'd like to raise.
openai.error.InvalidRequestError: The chatCompletion operation does not work with the specified model, text-embedding-ada-002. Please choose different model and try again. You can learn more about which models can be used with each operation here: https://go.microsoft.com/fwlink/?linkid=2197993.
Suggestion:
No response

Answer generated by a 🤖
Answer
The error message you received indicates that you are trying to use the text-embedding-ada-002 model with the chatCompletion operation, which is not supported. The chatCompletion operation currently supports only gpt-3.5-turbo and gpt-3.5-turbo-0301 models.
To fix the issue, you should change the model in your code to one of the supported models. For example, you can replace "model": "text-embedding-ada-002" with "model": "gpt-3.5-turbo" in your code.
Here's the corrected code snippet:
const completion = await openai.createChatCompletion({
  model: "gpt-3.5-turbo",
  messages: [{role: "user", content: "Hello world"}],
});
console.log(completion.data.choices[0].message);

Sources

docs/extras/modules/chains/additional/openai_openapi.yaml

This response is meant to be helpful and save users time. It is not meant to be a precise solution.
Please leave a 👍 if this is helpful and 👎 if it is irrelevant

My model is usable in completion, but it is no longer usable when integrating with Langchain. I am not sure if Azure cannot use text embedding ada-002, as I have read the official statement

Even we are getting the error with  gpt35 InvalidRequestError: The completion operation does not work with the specified model, gpt-35-turbo. Please choose different model and try again. You can learn more about which models can be used with each operation here: https://go.microsoft.com/fwlink/?linkid=2197993.
Running below code:
from langchain.agents import create_csv_agent, create_pandas_dataframe_agent
llm=AzureOpenAI(deployment_name="XXX",model_name="gpt-35-turbo",openai_api_version="2023-05-15",openai_api_base="https://{name}.openai.azure.com/", openai_api_key=token.token,openai_api_type = "azure_ad")
agent = create_pandas_dataframe_agent(llm, df, verbose=False)
agent.run("Question")


GH-35086: Upgrade cyclonedx-maven-plugin to 2.7.6

Rationale for this change
To fix the reported arrow-vector module cyclonedx error,
$ mvn clean package -DskipTests --pl vector --am
...
[INFO] --- cyclonedx:2.7.3:makeBom (default) @ arrow-vector ---
[INFO] CycloneDX: Parameters
[INFO] ------------------------------------------------------------------------
[INFO] schemaVersion          : 1.4
[INFO] includeBomSerialNumber : true
[INFO] includeCompileScope    : true
[INFO] includeProvidedScope   : true
[INFO] includeRuntimeScope    : true
[INFO] includeTestScope       : false
[INFO] includeSystemScope     : true
[INFO] includeLicenseText     : false
[INFO] outputReactorProjects  : true
[INFO] outputFormat           : all
[INFO] outputName             : bom
[INFO] ------------------------------------------------------------------------
[INFO] CycloneDX: Resolving Dependencies
[ERROR] An error occurred attempting to read POM
org.codehaus.plexus.util.xml.pull.XmlPullParserException: UTF-8 BOM plus xml decl of ISO-8859-1 is incompatible (position: START_DOCUMENT seen <?xml version="1.0" encoding="ISO-8859-1"... @1:42)

What changes are included in this PR?
Upgrade cyclonedx-maven-plugin to the latest one, 2.7.6.

https://github.com/apache/spark/pull/40726 ([SPARK-42382][BUILD] Upgrade cyclonedx-maven-plugin to 2.7.6)
https://github.com/apache/orc/pull/1463 (ORC-1407: Upgrade cyclonedx-maven-plugin to 2.7.6)

Are these changes tested?

Are there any user-facing changes?

Closes: #35086

Ah, sorry. I was already trying this: #35092

Oh, sorry. Thank you for fixing that!

Then, please let me know if you need further assitance. Good luck. :)

FYI, as I wrote in the PR description, Apache ORC and Spark verified this plugin and are using it now.

Thanks!!!


update option.java, Account.java and README.txt

I have added customer name option when creating account so it will be used when a customer want to exit the app.

Can you rebase please and resolve merge conflicts.


OpenSSL 1.1.1b no longer accepts an empty string as PSK identifier

We use OpenSSL with PSK+AES256. As we don't use multiple preshared keys, the relevant callback (SSL_set_psk_client_callback()) returns an empty string for the PSK identifier, which worked fine with OpenSSL 1.1.0h and earlier.
When using an empty string as PSK identifier with OpenSSL 1.1.1b, SSL_do_handshake() will result in ERR_R_INTERNAL_ERROR from tls_construct_ctos_early_data() (in line 819 of .\ssl\statem\extensions_clnt.c). This error is queued because the PSK identifier is duplicated with OPENSSL_memdup(id, idlen), which can't allocate a memory block of size 0, 0 being the length of the empty string excluding the terminating '\0'.
Considering the documentation for SSL_CTX_set_psk_client_callback(), which states:

... a buffer identity of length max_identity_len bytes where the resulting NUL-terminated identity is to be stored, ...

the PSK identifier is expected to be a NUL-terminated string. To me this means that OPENSSL_memdup() should be called with idlen+1, to duplicate the string including the terminating '\0'. Using an OpenSSL 1.1.1b built with this change worked as expected, at least for manual tests.

It does look like we should be pairing memdup with strlen(.) + 1, yes.  I think we do want to keep the recorded length as-is, though, so your proposal to just change the memdup call seems correct.

The spot in the code that is failing is related to TLSv1.3 handling. TLSv1.3 explicitly states that an identity must be at least 1 byte long. But in TLSv1.2 an identity was allowed to have a zero length. So it's not clear what we should if an old style callback is used (as in this case) and a zero length identity is returned.

To me this means that OPENSSL_memdup() should be called with idlen+1, to duplicate the string including the terminating '\0'.

This is not correct. There is no need to duplicate the \0 here because we are going to handle this as binary data with a separate length from that point on (in other codepaths the identity has come from the new style callbacks where the identity is allowed to be binary data containing 0 bytes in the middle of it).

Possibly we should handle this as if the callback had returned 0 (indicating that no PSK could be found) - which is true - there is no PSK capable of working in TLSv1.3 available. That would mean the handshake would continue. The PSK would continue to be used if the connection eventually goes on to negotiate TLSv1.2  but not if TLSv1.3 is negotiated. This might be quite unexpected for the application - so another possibility is for us to fail (probably with a more helpful error than "internal error") in this scenario. An application would then have to either (1) explicitly disable TLSv1.3 or (2) amend the code to not use a zero length identity.

Is there a security-relevant reason to enforce the language in the TLS 1.3 spec? Specifications sometimes have minor bugs that impact on usability, and if there's no other downside, we could (on the server side) simply allow the empty PSK even with TLS 1.3. This would be my preference if the TLS 1.3 requirement is a mere technicality.  It is hard to imagine how a 1 byte PSK is substantively different from a zero byte PSK.
On clients, given a zero-byte PSK, we don't have that option, so we need to either fail, or perhaps cap the protocol level at TLS 1.2.  Sending zero byte PSKs while advertising TLS 1.3 support, s sadly not viable given the TLS 1.3 spec...
Thoughts?

My guess is that we would encounter interoperability issues if we did this. Actually on the server side I believe we do currently tolerate a 0-byte PSK identity from the client (although we will never produce one). I would be not at all surprised if implementations immediately abort a handshake on encountering such a thing with a decode_error alert - and they would be entirely within their rights to do so. The TLSv1.3 message syntax defines the minimum length of a PSK identity as being 1, so strictly speaking this is a syntax error if you send a 0 length identity.


linux-yocto-onl/5.15: update 5.15.35

Update linux to 5.15.35 and meta to the appropriate revision.
Unfortunately yocto kernel-meta upstream enabled various subsystem
automatically when enabling virtio, some of which we don't need,
like DRM - there are no GPUs to pass through on a headless system.
Lacking a way to patch the file, add a pre-change version of virtio.cfg
so we can keep DRM disabled.
Signed-off-by: Jonas Gorski<EMAIL_ADDRESS>

Boot tested on as4610 and as4630.

According to the commit message for 917043019b46d8f1d0a9808fe3b0b815f14e9e60, the patch added prerequisites for the virtio  drivers. So what are the consequences of reverting it? No virtio?

According to the commit message for 917043019b46d8f1d0a9808fe3b0b815f14e9e60, the patch added prerequisites for the virtio drivers. So what are the consequences of reverting it? No virtio?

No, the prerequisites for (some) virtio drivers aren't automatically enabled, so e.g. before if you e.g. wanted PCI virtio, you needed to have it enabled in your base config, with this commit everything is automatically enabled, whether you want it or not.
In our case it forces DRM (= video) support on, which doesn't make sense for us because we don't have any GPUs to virtualize (at least on the ARM platforms).

I am wondering how we maintain this as it evolves. Could it be easier to disable virtio.scc (or push an empty virtio.cfg) and then add our own virtio config files when we need them? Looks like we'll have to do that anyway. Or are our use cases covered by the old virtio.cfg that this commit now uses to override the new one?

It keeps the status-quo from before the changes. The ideal solution is probably to prod upstream to split the virtio into multiple smaller snippets (so you can selectively enable the subsystems you care about).
Currently virtio gets enabled included as a side effect of enabling the kernel feature virtualization, so we don't have a direct control over it.

From an outcome-based perspective, yes, we are preserving the status quo. But we are effectively forking a file that is maintained upstream, and I don't see how we are going to maintain it (i.e., according to which rules). If we freeze it the way it is now, that seems quite arbitrary. But if we are ever going to change it, what are the rules? When would we add or remove something?

From an outcome-based perspective, yes, we are preserving the status quo. But we are effectively forking a file that is maintained upstream, and I don't see how we are going to maintain it (i.e., according to which rules). If we freeze it the way it is now, that seems quite arbitrary. But if we are ever going to change it, what are the rules? When would we add or remove something?

There are no rules. This is (hopefully) only a temporary stop gap solution until we have a better one/worked with upstream to fix this.

What explanation? I really am against trying to "document" yocto stuff in our recipes, and litter docu there.


Unable to save form submitted image file

Using both the following code examples, i am unable to convert the MultipartFormEntry.Value into a valid image file. Note that these lines have been added to the example FileUpload class.
Example 1
public void HandleRequest (Request request, Response response)
{
    var mpf = MultiPartEntry.Parse(request);
    var filePath = Path.Combine(Application.dataPath, "Image.png");
    var bytes = System.Text.Encoding.UTF8.GetBytes(mpf["fileToUpload"].Value);
    File.WriteAllBytes(filePath, bytes);

    response.statusCode = 200;
    response.message = "OK.";
    response.Write(html.text);
}

Example 2
public void HandleRequest (Request request, Response response)
{
    var mpf = MultiPartEntry.Parse(request);
    var filePath = Path.Combine(Application.dataPath, "Image.png");
    File.WriteAllText(filePath, mpf["fileToUpload"].Value);

    response.statusCode = 200;
    response.message = "OK.";
    response.Write(html.text);
}

This the original file on the left, and the saved file on the right. Will continue to test.

Hello, have you solved this problem?

I never found a fix.

Have you later found any way for unity to act as an http server and send files to unity  via the web client?
I want players to be able to upload their pictures and mp3 files to unity via the web.

Nothing public, it was 3 years ago i was working on this, and i can't even remember how i solved it, or even if i solved it. Sorry.

Nothing public, it was 3 years ago i was working on this, and i can't even remember how i solved it, or even if i solved it. Sorry.

Ok,thank you for this reply.Good day.


Does not correctly calculate volume percentage

I've been shamelessly stealing referencing your PA code for my own project and I've come across a bug in the way it's displaying volumes currently.
Using the libpulse LinearVolume seems to result in some weird not-actually-percentage value. I assume it's something to do with the log scale, and it's supposed to be linear to human hearing.
The issue with that is that just about every other volume program calculates the percentage differently, causing Mixxc to show very different values. For example, I'm using ncmpcpp for music and that allows controlling the sink input volume directly, and is always out of alignment. pavucontrol seems to agree with ncmpcpp and not Mixxc.
I have found another Rust mixer project which has a couple of helper methods for calculating to/from a percentage which seem to line up with the other mixers. I believe you could grab these and stick them in to correct this:
https://github.com/jantap/rsmixer/blob/a35039adaaf9dcf5e27d939df58d83335f76dea3/src/util.rs#L3-L25

I've not managed to get anywhere with a PA client before now and I've been searching months for an example, so this has been a lifesaver. I hugely appreciate the amount of work put into creating this and I will definitely give the project a shout-out.

I think i was aware of this at some point of development. Being in sync with other applications that show volume was never a point for me and it might be impossible in some cases (check out how YouTube treats volume on different videos.....). I personally don't like touching volume controls in applications, because they sometimes don't even sync with audio server's volume slider and tend to remember their own stuff and I tend to forget that I touched something and it's this whole mess of forgotten settings, i think you get my point.
Currently at 2% in Mixxc's I can barely hear anything (as i expect), but at the same time pavucontrol displays like 30% at that point and anything that's <20% I would never even set, because I just can't (distinctively) hear there. Of course my custom EQ with bass enhancer in Easyeffects and inexpensive Sennheisers without amp don't help the experiment much, but for me personally Mixxc's percentages make more sense.
Thanks for your interest and I'm glad that you found this project useful for your own cool thing.
I'll leave this issue opened, but as i understand you're not interested in using this Mixxc by itself.
If you or someone else actually needs a compile flag or CLI flag to make Mixxc's behavior more consistent with pavucontrol or other solutions they can come here and ask or propose a PR. It should be easy to solve.

I think that's fair enough. I can see why Pulse gives volumes in the way it does, and it does almost seem strange that the standard seems to be different. It sounds like our workflows do differ slightly around this, which might be why we prefer it the other ways around.
I'm not currently looking to use this, but that doesn't mean I won't be in future. If I do, I'll definitely open a PR (assuming nobody beats me to it). My personal take would be that a CLI flag (something like --non-linear or --logarithmic) could be used to switch the percentage algorithm.

I got a chance to revisit this while making implementation for the master slider that controls sink volume.
From my understanding, I'm doing almost* everything correctly.
https://docs.rs/libpulse-binding/latest/libpulse_binding/volume/index.html

The Volume in PulseAudio are cubic in nature and applications should not perform calculations with them directly. Instead, they should be converted to and from either dB or a linear scale.

This explains why i can barely hear <30% mentioned by me in pavucontrol. Because raw values for software volume are cubic, and linear for sinks and sources.

Don’t use the above functions on sink and source volumes, unless the sink or source in question has the DECIBEL_VOLUME or DECIBEL_VOLUME flag set.

Where "above functions" refers to Volume::multiply() ChannelVolumes::sw_multiply() and conversions through VolumeDB VolumeLinear. (that's how I read it)
b0815ce34793597cbb0b43b4205fdfedad3a754b makes it Ctrl-C Ctrl-V level of simple to change if someone really really needs it. I'm not a fun of doing things the wrong way to be compatible with stuff that does things the wrong way and since there's not much of an interest here, I'm gonna close this issue.
Feel free to reopen if you think that I misunderstand something.


Naming: PodcastReleaseDate

Summary
The corresponding file tags are not restricted to podcasts. Why is the item named PodcastReleaseDate instead of just ReleaseDate? The specific use case for tagging podcast episodes should only be mentioned in the docs.
Since this item was only used for MP3 files until v0.17.1 renaming it now before extending the support for more file types might be the last chance without causing too much trouble and confusion.
API design
No response

Related: #327
Maybe a good opportunity to change multiple namings at once.


Using a struct with static let strings for cleaner field name access

Awesome PR! Can you just rebase your changes onto the latest of vapor2 branch so the build will go through? Thanks!

Should be okay now!


Compiler support errors are inconvenient

Instead of whitelisting compilers you should blacklist them. What's the point of compiler whitelisting anyway? Shouldn't the code be portable in the first place?
This is my compiler:
g++ (Ubuntu 4.8.5-2ubuntu1~14.04.1) 4.8.5
I received the compiler error, I couldn't easily upgrade my compiler because seems that my system uses 4.8.5 by default and getting around it would have required me to change the Makefile of my project which I didn't want to do (to specifically call g++4.9 instead of just g++). So I changed the header file to this
// exclude unsupported compilers
#if defined(__clang__)
    #if (__clang_major__ * 10000 + __clang_minor__ * 100 + __clang_patchlevel__) < 30400
        #error "unsupported Clang version - see https://github.com/nlohmann/json#supported-compilers"
    #endif
#elif defined(__GNUC__)
    #if (__GNUC__ * 10000 + __GNUC_MINOR__ * 100 + __GNUC_PATCHLEVEL__) < 40800
        #error "unsupported GCC version - see https://github.com/nlohmann/json#supported-compilers"
    #endif
#endif

and everything compiled just fine, I ran my program and it works. Not sure what the big issue is. Why wouldn't nlohmann::json compile correctly with g++ 4.8.5 ???
For the record, I use linux mint 17.1 (is it really obsolete already? I don't think so).

From the page mentioned in the error message:

GCC 4.8 does not work because of two bugs (55817 and 57824) in the C++11 support. Note there is a pull request to fix some of the issues.

The test suite fails to compile with GCC 4.8. Whereas a subset of the features may work, I do not want to provide support for this compiler version.

oh ok, C++11 related issues I guess I should really upgrade my compiler then :D


Keep on top feature

Summary of the Pull Request
Makes a button on the title bar to keep up on top of other windows. Calculator has a similar feature.
Please note, as of 06/13, the button design is finished but the code is not even started. This is a work in progress.

References
Connected to #3038.

PR Checklist

[X] Closes #3038
[X] CLA signed. If not, go over here and sign the CLA
[X] No documents

For some reason, there are always commits unrelated to this. So, please ignore the commits from Apr 17th, to May 24th. Only beginning on 30 June 2020.

Also, can someone help with the placement of the button? It looks good in the designer but it actually is placed on top of the "New Tab" split button

For some reason, there are always commits unrelated to this PR. So, please ignore the commits from Apr 17th, to May 24th. Only beginning on 30 June 2020.

Squash changes and force push :)

Also, this button should preferably be a part of the minmaxclosecontrol so it can reuse the styles and maybe even code

We might even want to make a separate control, so that (optional?) features like #5426 and #3038 can be implemented together

Alright, thanks!

We might even want to make a separate control, so that (optional?) features like #5426 and #3038 can be implemented together

Alright.

If I set the value for the button for Grid.column="2" and for the MinMaxClose Grid.Column="3", the button overlaps the minimize one.

Thanks for the effort! I think I'd prefer if we started by coming at this from another direction. I think it would be better to prove that we can get the Terminal to be set to "keep on top" mode first, then add the UI for enabling that setting. IMO, I'd rather this just be a setting that's enabled in the settings UI (#1564), or toggled with a keybinding/command, rather than have a dedicated button for it. So, if I were implementing this, I'd probably do it in the following steps:

First prove that I can get the Terminal to stay on top of all windows.
Then I'd add a setting to enable this fuctionality
THen I'd make sure that the setting can be hot-reloaded, so the value can be turned on/off at runtime.
If a follow-up PR, I'd add a ShortcutAction to manually toggle that setting.

IMO, I'd rather this just be a setting that's enabled in the settings UI (#1564), or toggled with a keybinding/command, rather than have a dedicated button for it.

I would prefer it like the calculator with a dedicated button in titlebar to change always on top status along with a keyboard shortcut to toggle the always on top flag. IMHO, you should rather add a setting to set whether or not to show the "always on top" button in titlebar

Also, it is interesting to think how this feature would interact with tab tearoff when it is implemented...

Also, it is interesting to think how this feature would interact with tab tearoff when it is implemented...

That's exactly what my question was when this issue was first posted:

I suppose my next question would be: what happens when multiple applications want to be "always on top" - who wins?

and I'm not sure I've gotten a totally satisfactory answer to that question yet.

> I would prefer it like the calculator with a dedicated button in titlebar to change always on top status along with a keyboard shortcut to toggle the always on top flag. IMHO, you should rather add a setting to set whether or not to show the "always on top" button in titlebar
I've been highly reluctant to add more buttons to our already pretty minimalistic UI. It starts a slippery slope - "if this button is okay, then why can't I have a button dedicated to each profile"? I worry that adding more dedicated UI elements like this would leave the UI looking cluttered. Hence why I'm trying to drive the Command Palette (see #2046) for adding commands in a toggleable menu, and also why I'm driving to allow the user to add actions directly to the "New Tab Button dropdown" (see spec in-progress #5888). That seems like a much better place to stash this button, if the user so chooses. Hence my push to have this be a ShortcutAction the user can toggle 😉.

I just did some playing around using the Keep on Top feature in Calculator. I'll be honest, I'm not a huge fan of it. There's an ugly button right next to the Calculator Mode title and it always goes to the right. I actually prefer having it done exactly as zadjii-msft requests. Another thing to think about in regards to that "Shortcut Action": how will that play into Quake Mode? Will we be able to instantly place Terminal on top (toggable setting) when we launch it?

Also, should there be a maximum size limit in AOT mode?...

Given PowerToys has a prototype of "always on top" tool, we might not need to implement it by ourselves. So we can keep our UI minimalistic. I also think they can figure out a solution for cases like multiple windows.

In some world where somebody chooses to use Windows Terminal but not Power Toys, we'd need this feature available as an option that defaults to false. I am a bit worried about conflicts between trying to use the same feature via both Terminal settings and PowerToys but I think that's simply a doc update that recommends users not to enable it in both but one or the other.

Thanks for the contribution! Since this PR seems to be a bit stale now, we're going to close this in favor of #6903. Thanks!

Alright, thanks
From: Mike<EMAIL_ADDRESS>Sent: July 13, 2020 12:14
To<EMAIL_ADDRESS>Cc: Eric<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Re: [microsoft/terminal] Keep on top feature (#6494)
Thanks for the contribution! Since this PR seems to be a bit stale now, we're going to close this in favor of #6903https://github.com/microsoft/terminal/pull/6903. Thanks!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/terminal/pull/6494#issuecomment-657741209, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALKX3RXBCKQWYFWCASFEUVLR3NMHXANCNFSM4N5EDUFA.


Delimiter not considered during preview unless default (tab character)

[x] I have read through the manual page (man fzf)
[x] I have the latest version of fzf
[x] I have searched through the existing issues

Info
Only when the --delimiter flag is omitted (and therefore the default, "tab") does fzf allow for tab-delimited field separation in the --preview command. Interestingly, neither other -d values nor explicitly setting -d to $'\n' work as expected.

OS

[ ] Linux
[x] Mac OS X
[ ] Windows
[ ] Etc.

Shell

[ ] bash
[x] zsh
[ ] fish

Problem / Steps to reproduce
"CSV" and therefore newline delimited input with comma delimiter flag supplied to fzf.
echo -e 'header1,header2\nfield1,field2'                | fzf -d=','   --header-lines 1 --preview='echo {1}'

"CSV" / newline delimited input transformed to be tab separated, with tab delimiter flag supplied to fzf.
echo -e 'header1,header2\nfield1,field2' | tr ',' $'\t' | fzf -d=$'\t' --header-lines 1 --preview='echo {1}'

"CSV" / newline delimited input transformed to be tab separated, with no delimiter flag supplied to fzf.
echo -e 'header1,header2\nfield1,field2' | tr ',' $'\t' | fzf          --header-lines 1 --preview='echo {1}'

When you write -d=,, =, becomes the delimiter.
These are the correct ways to write it:

-d,
-d ,
--delimiter ,
--delimiter=,

Thanks for the prompt reply!

When you write -d=,, =, becomes the delimiter.
These are the correct ways to write it:

-d,
-d ,
--delimiter ,
--delimiter=,

I was not able to replicate this
FZF version:


Support XMLUnit placeholders in EqualToXmlPattern

When doing stub request body XML equality matching, I need the ability to ignore certain field values. For example, I have two requests
<message>
  <id>12345</id>
  <content>Hello</content>
</message>

<message>
  <id>67890</id>
  <content>Hello</content>
</message>

I want to define a stub mapping with request body like
<message>
  <id>#{xmlunit.ignore}</id>
  <content>Hello</content>
</message>

so that both requests match.
Using MatchesXPathPattern is technically doable, but its user experience is not so good as with the placeholder feature. XPath is a logical indicator while the placeholder is a visual indicator, and XPath might involve specifying namespaces while placeholder does not need them.

Would like this in a new version please 👍

Hi @tomakehurst , would you be able to have a check on this PR and see whether it can be accepted?

I pulled down your latest changes last night, so I'll see if I can find a few minutes today to take a look.

Thanks.

Rebased into 7f92d26039398ef22697443a2a059244ac61b41a and merged


Question about butteraugli's threshold value

Dear all:
There is something we are not certain about. We test  butteraugli on this image.
Original:

Compressed JPEG compression quality 90

According to butteraugli's result, the distmap's value is above good threshold and even bad threshold, so the difference can be visible. But according to our experiment, human cannot observe the difference at all. And the butteraugli' results keep the same even when the quality reached 98. Is there any problems with our understanding. Many thanks.

How do you conduct the human rating experiment?
I see some difference in the darker areas where the pattern disappears into the black. There is higher saturation and the pattern disappears more quickly into the dark in the lower images. Also, the lower image has some ringing in the background pattern.

The images were compared in a side-by-side experiment on a calibrated
sRGB display with the peak luminance of 220 cd/m^2, dark room. The
viewing distance was controlled so that the angular resolution was 60
pixels per visual degree.

The issue with the samples you posted isn't jpeg compression artifacts; there's a very noticeable difference in the gamma or the color range.


[release-1.6.5] CI cherry picks

Change type

[ ] New feature
[ ] Bug fix
[ ] Build
[ ] CI
[ ] Documentation
[ ] Performance
[ ] Refactor
[ ] Style
[ ] Unit tests
[ ] Subsystem tests

Checklist

[ ] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.

/lgtm

/approve


Keep column metadata/provenance for joined catalogs

Having extra metadata with the columns for a joined catalog to track which tables they came from.

Additional points from meeting:

when performing additional operations, only require the added prefix/suffix if required for disambiguation
if we need to extend/change the _hipscat_index, we can add metadata about the version it was generated with and determine compatibility matrix
we could include support for VOField (e.g. VOTable-as-parquet) metadata in this effort

This could do with a design doc for defining the scope.


Add testing for union-find

Add union-find test
Update UnionFind to have memory_space typedef and a size() function

retest this please

So the undefined reference to
desul::atomic_min(int*, int, desul::MemoryOrderRelaxed, desul::MemoryScopeDevice)' is still a problem with the latest Kokkos develop`.

Workaround have worked.


import bigger file to project directory

Hello,
I imported csv file to orchest project directory but a huge file (> 500MB ) is not imported. I didn't get any errors. This feature needs an improvement.
And Is there any way to do this ? I need to import huge files to my projects.

We'll investigate limits here, it shouldn't "inherently" be impossible but you might be hitting certain limits we've overlooked (we're not capping the upload size ourselves).
An alternative you can use in the meanwhile is using Python or the JupyterLab terminal to download a file from a URL using something like wget/curl or a Python package like requests.
E.g. run cd /data && wget https://<somefile> in the JupyterLab environment shell terminal.

Hi @nuhyurduseven, it looks like the max body size is not being correctly set in some cases, as a temporary workaround you could patch the object through this command:
kubectl patch -n orchest ingress orchest-webserver -p '{"metadata":{"annotations":{"nginx.ingress.kubernetes.io/proxy-body-size":"0"}} }'

Unfortunately, this needs to be reapplied when restarting or updating Orchest. The solution to this issue should be fairly simple so we'll likely release a fix pretty soon.

Hi @fruttasecca, Thanks for the solution. I fetched the data from database in orchest yesterday Now I met my need. But I'll try your solution or I think I can wait your update.

@nuhyurduseven happy to hear that. I'll keep the GH issue open in order to keep track of it.

@nuhyurduseven : Thanks for reporting this, this issue has been addressed in #1362 , and will be fixed in the next release.


server/model dir should be emptied after each build (?)

When building a new docker image, the model directory is copied to server/model, but the user has to remove the contents manually to build from a different model dir

Yes that's by design. Imagine the case where actually the user doesn't have a backup of the model and the script goes on to delete it. Its better be annoying and make the user make the conscious decision to delete their model.
We could include a cmd line flag (-y) to bypass this, or make the script ask for permission interactively. Feel free to post a pr

TODO: Add a flag to remove old models during model building


Show non-held scrolls in Spellcasting Activations tab

Currently you must be holding a scroll for it to show there, which makes sense RAW, but makes it difficult to assess your options if you have multiple scrolls, as you have to switch back and forth between tabs. Perhaps non-held scrolls could show but with their cast button greyed out and a quick button to change its carry type, akin to the one in the inventory.

It's planned; just kicked out MVP first


Add Android Q unit tests once JUnit / Robolectric testing is fixed

Currently, JUnit tests are blocked for tests requiring Android SDK 29 (Q) because of issues with Robolectric, our Android testing framework. To make progress on high priority tickets, we will land PRs now and land tests as a follow-up. This issue is created to track where such tests need to be added once the issues with Robolectric have been resolved.
Issue tracking implementation of Robolectric support of Q: https://github.com/flutter/flutter/issues/38471
Tests required:

[ ] FlutterView systemGestureInsets tests - https://github.com/flutter/engine/pull/10413

What specific Robolectric issues were you referring to? We added a number of system inset tests to FlutterViewTest recently. Were they what you were referring to?
https://cs.opensource.google/flutter/engine/+/master:shell/platform/android/test/io/flutter/embedding/android/FlutterViewTest.java

I might get the details of this wrong since it's been a while since I worked on this, but if I recall correctly, the tests missing are system gesture inset tests in FlutterViewTest, similar to the ones on the link you shared.
I tried writing them originally with https://github.com/flutter/engine/pull/10413, but at the time, Roboelectric did not have support for the system gesture insets API. I think we tracked this with https://github.com/flutter/flutter/issues/38471.

I close that bug #38471 :)
Can you see if there's anything left to do here?

Sure thing! Let me rename the issue to be more particular to the system gesture insets tests in particular, since it looks like we already support the later versions of Robolectric

Since this is still blocked by https://github.com/flutter/flutter/issues/57655, I will remove my assignment for now as I am not actively working on the issue


Triangle

Nice work! I would recommend resubmitting only because it would be nice for you to see some of these style changes early on in Level-Up.


Return custom message for app not found 404

For context, see deis/workflow-cli#77
This would allow the cli to give a specific error message if an application isn't found, such as Error: Application foo not found rather than just the generic Error: Not Found that is currently logged.

This is not going to be all that easy iirc, we use a get_object_or_404 which raises a generic 404,  we'd have to add our own logic to replace that if we wanted the custom message


Bot spamming on group messages

Hey there =)
Sorry for annoying you again
I just updated the bot to the newest release, and it works fine with scrobbles now, but its spamming when receiving any group messages with "Ops, something went wrong"
 lastgram_1  |    at Lastgram.Commands.CommandHandler.ExecuteCommandAsync(Message message, Func3 responseFunc) in /src/Lastgram/Commands/CommandHandler.cs:line 23
lastgram_1  |    at Lastgram.Bot.ExecuteCommandAsync(MessageEventArgs eventArgs) in /src/Lastgram/Bot.cs:line 62
lastgram_1  | System.NullReferenceException: Object reference not set to an instance of an object.
lastgram_1  |    at Lastgram.Commands.CommandHandler.TryParseCommandType(Message message, Type& type) in /src/Lastgram/Commands/CommandHandler.cs:line 73
lastgram_1  |    at Lastgram.Commands.CommandHandler.ExecuteCommandAsync(Message message, Func3 responseFunc) in /src/Lastgram/Commands/CommandHandler.cs:line 23 lastgram_1  |    at Lastgram.Bot.ExecuteCommandAsync(MessageEventArgs eventArgs) in /src/Lastgram/Bot.cs:line 62
This is what i get on console.

I can see the issue in the code. Just confused about it responding to every message... It should only be able to read messages that are the specified commands

Ah, I think you have disabled Group Privacy for the bot.
See: https://stackoverflow.com/questions/50204633/allow-bot-to-access-telegram-group-messages
Enable it, kick the bot, and then invite it again. Should solve it. I will fix the issue in the code later today

Should be fixed with https://github.com/djohsson/Lastgram/commit/beb6121612b02f041a5b270f756b9bd7826d76b0


Typo "resouce"→"resource"

https://learn.microsoft.com/en-us/azure/developer/java/sdk/azure-sdk-library-package-index
https://github.com/MicrosoftDocs/azure-dev-docs/blob/main/articles/java/sdk/azure-sdk-library-package-index.md
https://github.com/MicrosoftDocs/azure-dev-docs/blob/main/articles/includes/java-all.md
#PingMSFTDocs

Thanks, @hyoshioka0128 !


New Port: TowerBag

New Port for TowerBag
Description: Play as a little lonely skull, carry your TowerBag and face the evil creatures of the forest. This is a Tower Defense Game, but with only one Tower. Arrow Keys/WASD to walk, Spacebar to grab the tower. Made by Matheus Cunegato and Rodolfo Salazar.
Instructions:
Downloade the game from itch https://matheuscunegato.itch.io/towerbag and extract the data.win into the game folder.
CFW Tests
[x] AmberELEC
[x] ArkOS
[x] JELOS / ROCKNIX
[x] muOS
Architecture
[X] aarch64
[] Panfrost (Mainline)
Resolution Tests
Test all major resolutions:
[x] 480x320 (Optional)
[x] 640x480
[x] 720x720 (RGB30)
[x] Higher resolutions (e.g., 1280x720

Missing libcrypto, this package worked on all cfw's?

All good then :)


Queue trying to perform pref-sync in Incognito

Added a video to a queue in logged-out or Incognito:

Should force sign in for queue mode.


Bump transformers to 4.25.1

What does this PR do?

Fixes # (issue)
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Could you bump the version to 0.6.0 at the same time?

It was bumped yesterday by James.


add .txt endpoint to etext resources

the .txt would be reconstituted by concatenating the things (or maybe not, we'll see)

this has been done


virtual: give VWs an order in order to control precedence

To allow shadowing one VW another, e.g. a by-default APIBinding VW shadowed by a PermissionClaim VW: if the user does not claim APIBindings, just the bindings for their own exports are visible. But through a claim all bindings in those workspaces become visible. This needs an order of VW layers.

/lgtm

/approve