Header feature collides with pandoc tables

Both markdownreveal’s own header feature and the way pandoc generates tables use the .header CSS class. As a consequence, tables that use headers are displayed distorted.
This markdown:
% Title
% Author
% Date

# A section

## A slide

Test text.

## A second slide

Column A | Column B
---------|---------
Cell A | Cell B

leads to this output:

A resolution would be to change the class name for markdownreveal’s header (but this could break custom styles) or formulate the CSS rule more specific (e.g. div.header instead of plain .header, as pandoc tables use tr.header).

Thanks for reporting, and sorry for the late reply. :sweat_smile:
It is actually an ugly bug! I will probably find some time next week to fix it. I think I will change the class name for Markdownreveal's header/footer. Even if it breaks someone's current style, it is the cleanest/safest way. After all, Markdownreveal is still marked as alpha until 1.0.0 release. :wink:

@frederik-elwert I think it is fixed now. Could you try to pip install --upgrade markdownreveal and tell me if it is fine for you too?
I bumped the version to 0.2.0 as this change is not backwards compatible and can break other people's custom styles.

Feel free to reopen if it is not fixed or you find any problems.
Thanks again for reporting! :blush:


hooks/dbt.py references UnsetProfileConfig which no longer exists

Line 247 on hooks/dbt.py references a class called UnsetProfileConfig which does not seem to exist on the dbt-core library anymore? or maybe it never did?

Ya found this

already documented in another issue


Feature: Auto-update for RCs

Problem
Some of the users (myself included) would like to be up to date with the latest RCs. Right now the only way to get a new RC is to manually download it from GitHub.
Solution

Let users opt-in to RCs. A checkbox in Settings could work.
During the auto-updated check pay attention to the latest RC, otherwise only pay attention to stable releases.

I'd leave it open, so we could eventually implement that. Maybe we could make PR to the upstream package to support more flexible versioning.


torch-blade shape anaysis related problems

We forcely anaysis sahpe for each op, and erase shape information after failed, to get totally dynamic graph in https://github.com/alibaba/BladeDISC/pull/929 . This issue tracks realted problems occured in examples, TorchBench, and other scenes.
Occured problems:
shape anaysis

[ ] https://github.com/alibaba/BladeDISC/actions/runs/3840036856/jobs/6538601043#step:6:1651

related lowering problems

[ ] torch.ones/zeros with dynamic size input not support lowering

ones/zeros can be decomposed into aten.full


How to solve this error: No state in response

hi,I am using the latest version of oidc-client.ts, when trying to login and get the token, it prompts me an error: "No state in response". Could you please advise how to solve this issue?
The configuration in my client is as follows:
const config = {
authority: 'https://localhost:7150/',
client_id: 'client1',
redirect_uri: ${window.location.origin}/#/callback,
response_type: 'code',
scope: 'openid profile api',
response_mode: 'query'
}
and the error message like this:
Error: No state in response
at OidcClient.readSigninResponseState (OidcClient.ts:155:26)
at OidcClient.processSigninResponse (OidcClient.ts:173:48)
at UserManager._signinEnd (UserManager.ts:442:51)
at UserManager.signinRedirectCallback (UserManager.ts:174:33)
at Proxy.created (callback.vue:19:31)
at callWithErrorHandling (runtime-core.esm-bundler.js:173:36)
at callWithAsyncErrorHandling (runtime-core.esm-bundler.js:182:21)
at callHook (runtime-core.esm-bundler.js:3608:5)
at applyOptions (runtime-core.esm-bundler.js:3510:9)
at finishComponentSetup (runtime-core.esm-bundler.js:7387:9)
the callback url like this:
http://localhost:5007/#/callback?code=9F399AF8C82E4F458A5B1A37C79ADF28&state=b97ddcf6aa584df4a320e3022234bf75

Your response mode is not query but fragment: response_mode: 'query' -> 'fragment'

That was feasible and the previous error no longer occurs, but now I have received another error message:
Error: authority mismatch on settings vs. signin state
at ResponseValidator._processSigninState (ResponseValidator.ts:151:26)
at ResponseValidator.validateSigninResponse (ResponseValidator.ts:62:14)
at OidcClient.processSigninResponse (OidcClient.ts:175:31)
at async UserManager._signinEnd (UserManager.ts:442:32)
at async UserManager.signinRedirectCallback (UserManager.ts:174:22)
at async Proxy.created (callback.vue:20:13)
I have checked the corresponding source code according to the error message, and I feel confused why the authority that I specified in the configuration is inconsistent with the state.authority（I noticed that this state seems to be read from my configuration.）

You may need to enable logging, you can do like here described https://authts.github.io/oidc-client-ts/#logging. This and debugging + looking into session and local storage should help you find your issue...

I started the logging as per your guidance, and I see a lot of output in the browser. Some of it is quite confusing. One message reads as follows: "[WebStorageStateStore] remove('9ef30a7377aa42058ceaf1264fb71be5'): begin." Does this mean that the state is being removed from local storage?

Does this mean that the state is being removed from local storage?

Yes, old stale states are removed...
The state works like:

client auth request -> authz server
client remember what we sent (store state)
authz server -> request to redirect_uri (client)
client now needs to match that request with what was send (lookup/read state previously saved)
there must be a state...

是不是使用HashRouter，redirect_uri中包含字符"#“，导致不能从url中正确的读取state参数的值

I got the same issue, although I'm not using hash router. My settings is:
 const settings = {
      authority: '...',
      redirect_uri: 'http://localhost:8000/login/callback',
      client_id: '...',
      client_secret: '...',
      client_authentication: 'client_secret_basic' as const,
      scope: 'openid profile',
      metadata: {
       ...
      }
    };
    ```

response_mode: 'fragment'

Thank you for your guidance. I have modified the configuration to "response_mode: 'fragment'", but the issue still has not been resolved.

To track your issue: debug/log where the state is stored:
https://github.com/authts/oidc-client-ts/blob/ae221679d1f823abb4443146cdde90315bdde03e/src/OidcClient.ts#L134-L135
Here it is read back from the store:
https://github.com/authts/oidc-client-ts/blob/ae221679d1f823abb4443146cdde90315bdde03e/src/OidcClient.ts#L271-L277
Maybe it is useful to add response.state in to the message of the thrown error...

Could this be a browser problem?
WebKit seems to have some issues when accessing the sessionstorage to quickly before redirecting, as mentioned here.
To quote from the subsequent discussion

A workaround is for the JavaScript code to wait before accessing the storage.

If y'all are having the same issue as me - using a SPA in hash-mode router, which will make problems with the URL params parsing:

As you can see, the URLSearchParams expects the input to only contain the query part, which would be the case for a typical fragment-style Oauth callback:
https://domain.org/oidc-callback#id_token=eyJh...&state=abc123&..

but in hash mode:
https://domain.org/#oidc-callback?id_token=eyJh...&state=abc123&..

Fragment parsing is thus not appropriate for this case.
It would need a special parsing mode as part of this library

Please have a look at this issue https://github.com/authts/oidc-client-ts/issues/734#issuecomment-1298381823. Read through the thread. Summary:

Simply process the URL before passing it along to signinCallback

This issue (question) staled some time ago, closing it for now.


Does not work with reverse proxy

Im using sinus bot docker with the linuxserver/swag reverse proxy (nginx) as an ssl proxy. I set everything up just like it is shown in sinusbot documentation and when i try to access the webinterface it works normally and also the ssl encryption works. But as soon as i reload the page, try to login to the web interface or try to open the web interface in a new web browser tab, i get an network timeout. Then the timeout keeps happening and i need to wait a long time before it starts to reach the webinterface and this cycle continues...
How can i fix this?

config.ini
TS3Path = "/opt/sinusbot/TeamSpeak3-Client-linux_amd64/ts3client_linux_amd64"
ListenHost = "<IP_ADDRESS>"
DataDir = "/opt/sinusbot/data/"
ListenPort = 8087
LocalPlayback = false
EnableLocalFS = false
MaxBulkOperations = 0
LogLevel = 0
EnableProfiler = false
YoutubeDLPath = "/usr/local/bin/yt-dlp"
EnableDebugConsole = false
EnableInternalCommands = false
AllowStreamPush = false
UploadLimit = 83886080
RunAsUser = 0
RunAsGroup = 0
ExternalFileBase = ""
InstanceActionLimit = 0
UseSSL = false
SSLKeyFile = ""
SSLCertFile = ""
Hostname = "music.ejgaming.de"
HostnameMask = ""
SampleInterval = 0
StartVNC = false
EnableWebStream = false
LogFile = ""
LicenseKey = ""
IsProxied = true
DenyStreamURLs = []
Pragma = 0
UserAgent = ""

proxyconf.conf
server {

    listen 443 ssl;
    listen [::]:443 ssl;

    server_name music.ejgaming.de;

    include /config/nginx/ssl.conf;

    client_max_body_size 0;

    location / {

        set $upstream_app app;
        set $upstream_port 8087;
        set $upstream_proto http;
        proxy_pass $upstream_proto://$upstream_app:$upstream_port;

        proxy_set_header X-Forwarded-Host $host:$server_port;
        proxy_set_header X-Forwarded-Server $host;
        proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
        proxy_buffers 16 16k;
        proxy_buffer_size 16k;

        proxy_set_header Upgrade $http_upgrade;
        proxy_set_header Connection "Upgrade";

    }
}


Use EventEmitter/EventHandle; fully-implement

Description of the change

Format via purs-tidy
Add dependency on node-event-emitter
Fully implement net bindings

Added bindings for BlockList and SocketAddress
Worked around Socket TCP/IPC issue by forcing user to specify which connection type they'll use it for when creating the socket
Defined rows for connect/listen options as these are sometimes used in the http2 connect function.

Checklist:

[ ] Added the change to the changelog's "Unreleased" section with a reference to this PR (e.g. "- Made a change (#0000)")
[ ] Linked any existing issues or proposals that this pull request should close
[ ] Updated or added relevant documentation
[ ] Added a test for the contribution (if applicable)

Superceded by #16


Doesn't compile

Hi,
I'm trying to build my own version but the script is stuck on:
HEAD is now at 3fd44fd... Import l10n files
Submodule 'test/ttx/fonttools-code' (git://git.code.sf.net/p/fonttools/code) registered for path 'test/ttx/fonttools-code'
Cloning into 'test/ttx/fonttools-code'...
github --credentials get: github: command not found
Any clue?
Thanks

Tested again today with a new configuration, still hangs at:
Cloning into 'test/ttx/fonttools-code'...
Any clue?

I'm not sure why your output says github when the repository is hosted on git.code.sf.net. Are you perhaps using a nonstandard git interface, like github's scripts?

Well it seems I'm using GitHub "PortableGit" as client.
Now, if I authenticate on sf.net first with git, I'm able to clone this repo. Seems that this build script expect a same credential for different GIT repos.

$ git --version
git version 1.9.5.github.0

OK, well, seems that this cannot be built on Windows. I give up.
CMake Error at webodf/CMakeLists.txt:12 (include):
include could not find load file:
C:/DEV/GitHub/build/WebODF-build/jsfileslist.txt

-- Configuring incomplete, errors occurred!
See also "C:/DEV/GitHub/build/WebODF-build/CMakeFiles/CMakeOutput.log".
mingw32-make[2]: *** [WebODF-prefix/src/WebODF-stamp/WebODF-configure] Error 1
mingw32-make[1]: *** [CMakeFiles/WebODF.dir/all] Error 2
mingw32-make: *** [all] Error 2

Having same problem, linux min 18.2, git version 2.7.4.
Trying to build like in readme says
git clone http://github.com/kogmbh/ViewerJS.git
mkdir build
cd build
cmake ../ViewerJS
make

returns this error on the make command :
.
.
.
[ 63%] Performing download step (git clone) for 'PDFjs'
Cloning into 'PDFJS-source'...
Note: checking out 'v1.1.114'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by performing another checkout.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -b with the checkout command again. Example:

  git checkout -b <new-branch-name>

HEAD is now at 3fd44fd... Import l10n files
Submodule 'test/ttx/fonttools-code' (git://git.code.sf.net/p/fonttools/code) registered for path 'test/ttx/fonttools-code'
Cloning into 'test/ttx/fonttools-code'...
fatal: remote error: access denied or repository not exported: /p/fonttools/code
fatal: clone of 'git://git.code.sf.net/p/fonttools/code' into submodule path 'test/ttx/fonttools-code' failed
CMake Error at PDFjs-prefix/tmp/PDFjs-gitclone.cmake:67 (message):
  Failed to update submodules in:
  '/home/sergio/Desktop/javascript/build/PDFJS-source'

CMakeFiles/PDFjs.dir/build.make:89: recipe for target 'PDFjs-prefix/src/PDFjs-stamp/PDFjs-download' failed
make[2]: *** [PDFjs-prefix/src/PDFjs-stamp/PDFjs-download] Error 1
CMakeFiles/Makefile2:139: recipe for target 'CMakeFiles/PDFjs.dir/all' failed
make[1]: *** [CMakeFiles/PDFjs.dir/all] Error 2
Makefile:83: recipe for target 'all' failed
make: *** [all] Error 2

Same error in macOS Mojave 10.14
git git version 2.17.1 (Apple Git-112)

Has anybody found a solution for this?

@renierdbruyn I'm not sure if you're still trying to use this almost a year later, but the problem was that the repository for a submodule of pdf.js no longer exists. I made a pull request to that has a later version of pdf.js with the submodule fixed.
https://github.com/webodf/ViewerJS/pull/287


Create Autonomous for Red Right

Create an autonomous mode program for when we are on the red alliance and start on the right. Set filename and program name to autoRedRight

Initial auto code was merge


Additional logging

Problem
Some key data is missing from cartographer, making it impossible to analyse some of the user behavior
Impact
This will help us measure the worst-case scenario for the user - transfer gets stuck and user has to reengage multiple times
Proposed Solution
Start logging:

Action - bump relayer fee
Count - # of bumps in relayer fee, slippage

Acceptance Criteria
We reliably see this data in cartographer for all transfers

subgraph already has it. carto just needs schema update

Backlog because we can get data from subgraphs.

@just-a-node is it relevant?


popup opens  if KEY_TO_CALL_POPUP is continuously pressed instead of double tap

if you change line 43 from:
		if key_event.keycode == KEY_TO_CALL_POPUP and key_event.is_pressed():
to:
		if key_event.keycode == KEY_TO_CALL_POPUP and !key_event.is_pressed():
it works as intended.

@robatte thanks for suggestion, I fixed it


KNOX-2375 - Token state eviction should access the keystore file less…

… frequently
What changes were proposed in this pull request?
The goal of these changes is to reduce the number of times during a token state reaper iteration that the keystore file is loaded and subsequently written. This change reduces that number from (1 + 2-to-4-per-token) to (1 + 0-to-1-per-token + 1). So, for 100 expired tokens and 100 unexpired tokens, that means going from 601 loads to 2 loads (if all the token state aliases are cached) or up to 202 in the absolute worst case when none of the token state aliases is cached.
Part of the solution includes adding bulk alias/key removal method to the AliasService and KeystoreService respectively.
On the request side, I've added a delay and bulk creation of token state aliases. Because key pair generation is so slow, doing this in a background thread allows Knox token requests to complete MUCH more quickly. Now, when a token is issued, its state is cached in the unpersistedState collection. A periodic thread converts this unpersisted state into aliases in the keystore.
Finally, I've resolved an issue which was pausing KnoxToken request processing because the DefaultTokenAuthority was blocking on access to aliases for every single request. The values of the aliases being referenced are now being cached after the first access, removing the contention for the KeystoreService locks.
How was this patch tested?
I've added multiple tests to exercise these changes: AliasBasedTokenStateServiceTest, DefaultTokenStateServiceTest, DefaultKeystoreServiceTest, RemoteAliasServiceTest, and ZookeeperRemoteAliasServiceTest. I'm continuing manual testing, but wanted to get the review process started as soon as possible.

Besides my review comments below I'm not sure if I got the idea of the unpersistedState collection in AliasBasedTokenStateService. Could you please update the PR description to make it cleaner for me :)

Done.


Overwrite config/headers on update using boltdb

Postgres is overwriting it right now.
Fixes #453 (To delete single keys the client will have to handle it, that's what is happening right now in fn)

👍


saveAs() not work

saveAs() can save only one time!
Eg.
$uploadedFile=UploadedFile::getInstance($model, 'file');
$uploadedFile2=UploadedFile::getInstance($model, 'file');
$uploadedFile->saveAs('uploads/' .$uploadedFile->name); // This first time work fine
$uploadedFile2->saveAs('uploads/' .$uploadedFile2->name);//Problem! nothing happened
/------------------------ Or-------------------/
$uploadedFile=UploadedFile::getInstance($model, 'file');
$uploadedFile->saveAs('uploads/' .$uploadedFile->name); // This first time work fine
$uploadedFile->saveAs('uploads/' .'AnotherFileName');//Problem! nothing happened
| ---------------- | ---
| Yii version      | 2.0.12?
| PHP version      | 7.14
| Operating system | Mac

if you want to use the same method twice you have to set second parameter of saveAs(), $deleteTempFile to false: .

thx


Can't use queue's fake method and artisan's call in same test

Laravel Version: 5.4.19
PHP Version: 7.1.3
Database Driver & Version: MariaDB 10.1.19

Description:
The problem is - when we use queue's fake method and artisan's call method we gonna end up with this kind of error:

TypeError: Argument 1 passed to Illuminate\Queue\Worker::__construct() must be an instance of Illuminate\Queue\QueueManager, instance of Illuminate\Support\Testing\Fakes\QueueFake given

Which I believe is issue with Laravel itself. If I'm wrong, please correct me. I found simple workaround for this - I've created seperate unit test for this particular feature, which doesn't include fake method, but I think it's worth to mention this problem there.
Steps To Reproduce:
Simply use this example code:
    public function setUp()
    {
        parent::setUp();

        Queue::fake();
    }

    public function testExample()
    {
        Artisan::call('schedule:run'); // Can be literally any command
    }

Fixed in https://github.com/laravel/framework/pull/18831


raidboss: add R4S Midnight Sabbath wings and guns info

add R4S Midnight Sabbath wings and guns info, When dealing this, The previous ones didn't show the safe zone very visually. Only call:

Cardinals => Stack With Partner
Intercards => Spread

In this change, modify to

Cardinals => Avoid Line + Stack With Partner
Intercards => Donut + Spread

It's easier to visual danger zone and find safe areas in the game.

If you wanted to, I think you could consider doing something like:
        cardinals: {
          ...Outputs.cardinals,
          cn: '去正点',
        },

(and so on) so as to address the cn translation issue you flagged without requiring i18n PRs for all other languages if they don't have the same issue.  But I'm fine to merge either way once the merge conflict is resolved.

If you wanted to, I think you could consider doing something like:
        cardinals: {
          ...Outputs.cardinals,
          cn: '去正点',
        },

(and so on) so as to address the cn translation issue you flagged without requiring i18n PRs for all other languages if they don't have the same issue. But I'm fine to merge either way once the merge conflict is resolved.

Thank you, learned the usage!

I checked the historical commit and found that the outputs commit was matched with RS4 trigger commit. There are no other triggers currently using this Output, so the outputs are modified here.

I checked the historical commit and found that the outputs commit was matched with RS4 trigger commit. There are no other triggers currently using this Output, so the outputs are modified here.

Yep, you're completely right -- I added those for r4s (and then completely forgot that I did it 🙃).  LGTM, and thanks for your work on this!


Pipeline should inherit from collections.OrderedDict

When you think about it a compose.Pipeline is nothing more than a collections.OrderedDict from the standard library. I'm not sure but this might be a nice thing to have.

Done! Also TransformerUnion inherits from collections.UserDict. This makes accessing parts of a pipeline/union much nicer.


Doc page for user feedback missing

I get a "Page Not Found" error for the User Feedback doc page. Are the docs accessible from somewhere else?

User Feedback has not been implemented yet. You can find the tracking issue here:
https://github.com/getsentry/sentry-dart/issues/215

@sabetAI where did you get this link from? if it's somewhere we should fix it, please let us know.
if you just concatenated user-feedback/ yourself, then it's what @ueman said.

I clicked the "Read the docs" button on the User Feedback page in the console.

@sabetAI indeed, thanks

I think this should be under @getsentry/sentry though


only allowing array of 2 element , array with more than 2 element showing garbage values

yes, i have this problem too..

Sorry for late reply! Fixed! v1.0.3 👍


Apply ArgMin/ArgMax fix for infinity input to Min/Max

A few weeks ago, cub::DeviceReduce::ArgMin returns wrong value for INF input was reported and fixed in Fix cub::DeviceReduce::Arg{Max,Min} for inf values for ArgMin/ArgMax in DeviceReduce and DeviceSegmentedReduce. However, the same issue persists for Min and Max in DeviceReduce and DeviceSegmentedReduce. This can be shown by slightly modifying the reproducer:
#include <iostream>
#include <cub/cub.cuh>

constexpr int N = 10;
constexpr float INF = std::numeric_limits<float>::infinity();

int main() {
    float* in;
    float* d_in;
    float* out;
    float* d_out;

    in = new float[N];
    out = new float;
    std::fill(in, in + N, INF);

    size_t temp_storage_bytes = 1024;
    void* d_temp_storage;
    cudaMalloc((void**)&d_temp_storage, temp_storage_bytes);

    cudaMalloc((void**)&d_in, sizeof(float) * N);
    cudaMalloc((void**)&d_out, sizeof(float) * N);

    cudaMemcpy(d_in, in, sizeof(float) * N, cudaMemcpyHostToDevice);
    cub::DeviceReduce::Min(d_temp_storage, temp_storage_bytes, d_in, d_out, N);
    cudaMemcpy(out, d_out, sizeof(float), cudaMemcpyDeviceToHost);

    std::cout << *out << std::endl;  // Expected: inf
}

Output
3.40282e+38

Are there plans to apply a similar fix for Min and Max?

Thanks for your response, realize I misunderstood the documentation.


Private Role tip broken

Last PR attempted to fix the issue:
https://github.com/lbryio/lbry-tipbot/pull/20
[10:30 AM] Larva: !roletip @Trusted LBRYian 0.01
[10:30 AM] rouse: !roletip @Trusted LBRYian 1
[10:30 AM] Larva: Not work

Roletip works now, but the private version is still broken.

@tzarebczan Does this issue have a solution?

No, and it's not that big of a deal since it's not used often (if at all). Going to give it a shot with  https://github.com/lbryio/lbry-tipbot/pull/33

closing, Niko has fixed this


[FLINK-6138][table] Create the ListStateDescriptor with the aggregati…

…onStateType instead of a serializer.
Thanks for contributing to Apache Flink. Before you open your pull request, please take the following check list into consideration.
If your changes take all of the items into account, feel free to open your pull request. For more information and/or questions please refer to the How To Contribute guide.
In addition to going through the list, please provide a meaningful description of your changes.

[ ×] General

The pull request references the related JIRA issue ("[FLINK-6138] Create the ListStateDescriptor with the aggregationStateType instead of a serializer.")
The pull request addresses only one issue
Each commit in the PR has a meaningful commit message (including the JIRA id)

[ ] Documentation

Documentation has been added for new functionality
Old documentation affected by the pull request has been updated
JavaDoc for public methods has been added

[ ] Tests & Build

Functionality added by the pull request is covered by tests
mvn clean verify has been executed successfully locally or a Travis build has passed

+1 to merge

LGTM
merging


Volto returns Cache-Control: public, max-age=0 for static files

In production mode, Volto returns the header:
Cache-Control: public, max-age=0

On nginx, when it gets max-age=0 it doesn't cache to disk if it's configured to do so. To do this, we would have to use the additional configuration:
proxy_ignore_headers Cache-Control;
I took some time to figure this out. So I ask if it wouldn't be better not to send Cache-Control, leaving the configuration to a proxy.
Anyway, if Volto has to return a max-age, I think it should be something > 0, since they are static resources, which can be in the browser cache.
I think @mamico discussed something about this at: #2216

#2216 is now merged.
@wesleybl, I think that this could be closed. Do you agree?

@mamico yes we can close. Thanks!


feat: New sync option "default namespace"

Re-make of #77 for v2.0 - but this time, only the "default namespace" part.
My use case: monorepo where each subfolder project has its unique Tolgee namespace - which is difficult for the CLI to auto-detect.

Hey, thanks for this contribution 👍  Could you add some tests?

Hey, thanks for this contribution 👍 Could you add some tests?

You welcome! Unfortunately, I don't have the time resources for this now. Can you point me to a similar test for example? I will take a look when I have some spare time.

Also this would be an ideal candidate to have in tolgee config as well.
You can add it to schema.json and then run npm run config:type, which will add type definition as well.

As I understand (correct me if I'm wrong), the sync command is not part of the tolgee config (unlike the push and pull commands). Does it mean that the sync command is closed to new contributions until someone puts efforts to make it a first-class citizen like push and pull? We're talking about major work and breaking changes, similar to those made from 1.x to 2.x.

I'd say this might be considered part of the extractor, so it should be a general option (not part of any command, but global).
It could be added here https://github.com/tolgee/tolgee-cli/blob/main/src/test/e2e/extract.test.ts
If you don't have time, it's fine. I'm actually currently re-writing the extractors to be more managable, so I can add this feature as a part of it (https://github.com/tolgee/tolgee-cli/pull/88).


Fix the selected div bg color

Issue: When user selects any item from operation list then that item was not shown in different color, so it becomes difficult for user to identify which one is currently selected.
Changes: I have added selected as a prop to Operation style, so if an item is selected then color will be changed for that.

Thanks @prateekvijayvergiya for this wonderful PR


fragmented slide?

as I am looking for reveal.js alternatives to craft beautiful presentation, this project can be counted as one, but still lacks some important feature like #91 ; and may also include the fragmented slide?
like http://lab.hakim.se/reveal-js/#/fragments want the slide not all showing at once, but fragments to step through one by one

this is an important and elementary feature, if you consider all the competing, js and web based presentation framework projects at https://bestof.js.org/tags/presentation , most others already have;
also in the traditional desktop based microsoft powerpoint, or the Open Office, or in google office, none of them not having this feature

Let me try to say this politely. We couldn't care less of what others already have. Really. Trying to ask for a feature just because others have it even when we already said we were not going to makes little sense.
If you need that feature desperately you have a two options:

Use one of the competing presentation frameworks. I'm not sure about them but I don't think we're here to compete. We just offer different things and visions. We won't be offended if you stop using WebSlides or not use it and chose Reveal instead. It just means WebSlides isn't for you.
Fork WebSlides, then implement your feature, then rename it and finally release it.

We have already said we won't go into that direction which is what we'll do: not do it. We're sorry if you stop considering WebSlides as an "alternative to craft beautiful presentations".

don't get me wrong, I noticed from contributor graph that this project is a new project started in this year 2017, get 3K stars in just 8 months is an amazing achievement;
all issues reporters I believe are in the good belief for this project can grow into a better one, and from github open source experience, whoever reported the issue is also the very potential one can help implement; for me also did a lot of open source contribution, but ask the question first for a test if anyone is doing this feature and in progress

however once somebody suggested to bestof.js and people found it from https://bestof.js.org/tags/presentation or http://alternativeto.net/software/reveal-js/?license=opensource this is inevitably seen as alternative solutions to each other, or competing projects, whatever you think you are competing or not; people are thinking in that way.
it also depends on what do you think future of this project, just for fun without profit? or based on donations to cover some spent time, or to commercialize in future is one option? then will compete with revealjs authors' https://slides.com/about anyway
from https://medium.freecodecamp.org/introducing-webslides-fa7a9e37ff97 I read this, so far the pros of webslides is landings, portofolios, and long-forms
I hope in other cons areas webslides can catch up soon

Well, don't get me wrong but the amazing achievement was achieved with a basic set of features and we have improved ever since. We've received positive feedback and the amount of requests like this one is fairly low.
We might, of course, be wrong with our vision but we want to learn about that and follow this path we decided together. So far, contribution from outside of the project members has been fairly minimal and that's ok.
We are not aiming to the Reveal target which is why we don't want to copy any of the Reveal features. I do believe Reveal does a really good job on some areas and I also believe we do a really good job on some areas as well, not only in those you mention.
Future will tell. We're having fun and that's all that matters for us. Copying Reveal is not fun or entertaining for us so implementing some feature just because X has it won't ever happen.
We do consider WebSlides to be mostly done to the point we wanted to take it. What will happen from here, future will tell.

After some time of inactivity, closing the issue.


[HUDI-8301] Removed DataItem for Flink stream writing

Change Logs
Currently StreamWriteFunction uses buffer with DataBuckets consisting of DataItems, which are transformed HoodieRecords. We could use HoodieRecords directly to reduce number of objects in the heap, and decrease GC workload.

Before
After

GC related CPU samples
66.7K
43.6K

Memory allocation
61.17 GB
51.58 GB

Total time of 60M rows writing
4 min
3 min 50 s

GC related CPU samples:
before

after

Memory allocation in StreamWriteFunction:

Before
After

writeBuffer() > getRecords()
23.19 GB
0 GB

bufferRecord()
37.98 GB
51.58 GB

Total
61.17 GB
51.58 GB

Impact
Improved Flink stream write performance.
Risk level (write none, low medium or high below)
Low
Documentation Update
No need
Contributor's checklist

[x] Read through contributor's guide
[x] Change Logs and Impact were stated clearly
[x] Adequate tests were added if applicable
[ ] CI passed

Need some time to check failed tests.

Refactored setting of FlinkOptions.WRITE_BATCH_SIZE and FlinkOptions.WRITE_TASK_MAX_SIZE in unit tests.
Restarted CI by force push.

@danny0405 , hi!
If you don't mind, could you, please, review these changes?


Upload Test Result CI step fails

It fails when one of the tests crashes with "Restarting after unexpected exit, crash, or test timeout" message in the logs, like here f.e.
┆Issue is synchronized with this Jira Task by Unito

FYI @lawrence-forooghian

It looks like an issue in Fastlane. Please could you create a branch that just runs a single test (so that I don't have to wait for the whole test suite to execute) which consistently exhibits this issue?

It looks like an issue in Fastlane. Please could you create a branch that just runs a single test (so that I don't have to wait for the whole test suite to execute) which consistently exhibits this issue?

I think none of the tests consistently crash, so I'll just make one.


Stop depending on swift-collections-benchmark

Instead, move the benchmark suite into its own private nested package, under Benchmarks/.
The recommended way to build & run benchmarks is via the Utils/run-benchmarks.sh script, which continues to work as before with this change.
Checklist

[X] I've read the Contribution Guidelines
[X] My contributions are licensed under the Swift license.
[ ] I've followed the coding style of the rest of the project.
[ ] I've added tests covering all new code paths my change adds to the project (if appropriate).
[ ] I've added benchmarks covering new functionality (if appropriate).
[ ] I've verified that my change does not break any existing tests or introduce unexplained benchmark regressions.
[ ] I've updated the documentation if necessary.

@swift-ci test

This will also need to be cherry-picked on the release/1.0 branch.

looks great.
is checking-in the resolved file intentional?

cc @neonichu @abertelrud

is checking-in the resolved file intentional?

It is! I'd prefer if we ran the benchmarks using a consistent codebase, in as much as that's possible. (Not that I'd expect picking up a new argument parser would directly affect measurements -- but subtle function alignment changes within the CollectionsBenchmark module (such as when picking up a new version of a dependency) can measurably perturb benchmarks. The runner script tries to align modules on page boundaries, but this doesn't help if a function gets slightly offset within the module.)
(If this proves unnecessary, and/or having to manually update dependencies becomes too much of a chore, then we can always remove the lock later.)

There is no point in having the top-level resolved file though; I'll remove it.

@swift-ci test


from werkzeug.wsgi import DispatcherMiddleware

Expected Behavior

It seems to me the DispatcherMiddleware is not available when call werkzeug.wsgi on flask 1.1.1.
from werkzeug.wsgi import DispatcherMiddleware

Actual Behavior

Should be in the document:
from werkzeug.middleware.dispatcher import DispatcherMiddleware
Environment

Python version: 3.6
Flask version: 1.1.1
Werkzeug version: 0.16.0

It's still available, just deprecated. You should see a DeprecationWarning when running:

DeprecationWarning: The import 'werkzeug.wsgi.DispatcherMiddleware' is deprecated and will be removed in Werkzeug 1.0. Use 'from werkzeug.wsgi.middleware.dispatcher import DispatcherMiddleware' instead.


add transaction entity

first time... huh ?

For naming, try to name the PR in the clear way
For ex:
[Transaction] Create entity ...


Problem with Vulkan Callback issue

@HurricanKai
Summary
Unhandled exception. System.InvalidOperationException: Vulkan functions can only be used after initialization (just before the Load callback is executed)
at Silk.NET.Windowing.Internals.ViewImplementationBase.<get_VkSurface>g__Throw|106_0()
at Silk.NET.Windowing.Internals.ViewImplementationBase.get_VkSurface()
at VulkanSample.Launch.HelloTriangleApplication.InitWindow() in /home/jens/Projects/dotnet/VulkanSample/VulkanSample.Launch/Program.cs:line 81
at VulkanSample.Launch.HelloTriangleApplication.Run() in /home/jens/Projects/dotnet/VulkanSample/VulkanSample.Launch/Program.cs:line 28
at VulkanSample.Launch.Program.Main(String[] args) in /home/jens/Projects/dotnet/VulkanSample/VulkanSample.Launch/Program.cs:line 1178
Steps to reproduce

Platform: Desktop Ubuntu 20.04.2
Framework Version: .NET 5.0
API: Vulkan
API Version: Vulkan 1.0

I use VulkanTriangle from Lab directory and I tried but it throws same problem.
I have already installed and latest updated lib files under Ubuntu 20.04

Same trying no success.

Because my display card is AMD Radeon RX 480 8 GB VRAM has Vulkan 1.0 and OpenGL 4.6 - I really don't understand why I have tried. I already tested AmdGpu's rocm and vulkan are already installed.
How do I get successful VulkanTriangle on Ubuntu 20.04.2

This is a problem with the lab application regarding dependency on undefined and undocumented behaviour in our windowing system.
Applications in the Lab folder shouldn’t be relied upon as proper examples, and really shouldn’t be used unless as part of development of the library itself.
Will leave this open as an action item to eventually fix our Lab application.

In the lab application the window.Initialize() just has to be moved to above the null check, should be an easy fix.

In the lab application the window.Initialize() just has to be moved to above the null check, should be an easy fix.

The PR I have open incorporates this fix

Thanks @Perksey and @HurricanKai
I have moved _window.Initialize();
over if statement then I retry but it looks like my display card has Vulkan 1.0.0 and it doesn't support :(
Unhandled exception. System.NotSupportedException: Validation layers requested, but not available!
   at VulkanSample.Launch.HelloTriangleApplication.CreateInstance() in /home/jens/Projects/dotnet/VulkanSample/VulkanSample.Launch/Program.cs:line 258
   at VulkanSample.Launch.HelloTriangleApplication.InitVulkan() in /home/jens/Projects/dotnet/VulkanSample/VulkanSample.Launch/Program.cs:line 98
   at VulkanSample.Launch.HelloTriangleApplication.Run() in /home/jens/Projects/dotnet/VulkanSample/VulkanSample.Launch/Program.cs:line 29
   at VulkanSample.Launch.Program.Main(String[] args) in /home/jens/Projects/dotnet/VulkanSample/VulkanSample.Launch/Program.cs:line 1177

vulkaninfo | grep "AMD"
ERROR: [Loader Message] Code 0 : /usr/lib/i386-linux-gnu/libvulkan_intel.so: wrong ELF class: ELFCLASS32
ERROR: [Loader Message] Code 0 : /usr/lib/i386-linux-gnu/libvulkan_radeon.so: wrong ELF class: ELFCLASS32
		GPU id 	: 0 (AMD RADV POLARIS10 (ACO))
		GPU id 	: 0 (AMD RADV POLARIS10 (ACO))
		GPU id 	: 0 (AMD RADV POLARIS10 (ACO))
GPU id : 0 (AMD RADV POLARIS10 (ACO)):
			AMD RADV POLARIS10 (ACO) (ID: 0)
		AMD RADV POLARIS10 (ACO) (ID: 0)
			AMD RADV POLARIS10 (ACO) (ID: 0)
	deviceName     = AMD RADV POLARIS10 (ACO)
	VK_AMD_buffer_marker                      : extension revision 1
	VK_AMD_device_coherent_memory             : extension revision 1
	VK_AMD_draw_indirect_count                : extension revision 1
	VK_AMD_gcn_shader                         : extension revision 1
	VK_AMD_memory_overallocation_behavior     : extension revision 1
	VK_AMD_mixed_attachment_samples           : extension revision 1
	VK_AMD_rasterization_order                : extension revision 1
	VK_AMD_shader_ballot                      : extension revision 1
	VK_AMD_shader_core_properties             : extension revision 1
	VK_AMD_shader_core_properties2            : extension revision 1
	VK_AMD_shader_explicit_vertex_parameter   : extension revision 1
	VK_AMD_shader_fragment_mask               : extension revision 1
	VK_AMD_shader_image_load_store_lod        : extension revision 1
	VK_AMD_shader_info                        : extension revision 1
	VK_AMD_shader_trinary_minmax              : extension revision 1
	VK_AMD_texture_gather_bias_lod            : extension revision 1

That suggests you don't have the Vulkan SDK installed. Change the "EnableValidationLayers" at the top of the file to be equal to false, that should disable that.
But I'd highly suggest installing the Vulkan SDK if you are going to do any vulkan development at all.

I have already tried...
same problem like my last post.

But I'd highly suggest installing the Vulkan SDK if you are going to do any vulkan development at all.

I installed libvulkan-dev
sudo apt install -fy libvulkan-dev
Reading package lists... Done
Building dependency tree
Reading state information... Done
libvulkan-dev is already the newest version (<IP_ADDRESS>-1).
libvulkan-dev set to manually installed.
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.

Looking at The LunarG page you would have to run
wget -qO - https://packages.lunarg.com/lunarg-signing-key-pub.asc | sudo apt-key add -
sudo wget -qO /etc/apt/sources.list.d/lunarg-vulkan-focal.list https://packages.lunarg.com/vulkan/lunarg-vulkan-focal.list
sudo apt update
sudo apt install vulkan-sdk

for Ubuntu 20.04 & Latest SDK

Hey @HurricanKai  Thanks for problem. Now It works fine.

Updated SDK works fine! Close now!
Or I need buy new display card?


 One of them had been pending, and it is still pending for weeks. I'm unable to close / force close it as it is still open.
Also -- The TX that it has provided to show that it's opening can't be found, which is why this is still stuck.
Channel should open and then allow me to close it.
Actual Behavior
Possible Fix
Steps to Reproduce
Context
I'm trying to withdraw my funds, though I'm stuck on pending.
Your Environment
Zap version: Newest verison
Operating System and version: Windows 10

Hi,
It sounds like you are affected by this issue - https://github.com/LN-Zap/zap-desktop/issues/2531 - which is a bug in lnd. As mentioned in that thread, the only way that I'm aware of to rectify this is to use a debug build of lncli and call lncli abandonchannel on the channel. This is not something that's currently supported in Zap itself.

Hi,
It sounds like you are affected by this issue - #2531 - which is a bug in lnd. As mentioned in that thread, the only way that I'm aware of to rectify this is to use a debug build of lncli and call lncli abandonchannel on the channel. This is not something that's currently supported in Zap itself.

Ah, thanks for the prompt reply.
I assume that means that the funds are stuck for now? Unless Zap implements this?

Not if you build yourself a debug build of lnd and use lncli to run the aforementioned command against the lnd instance that Zap runs for you.

Not if you build yourself a debug build of lnd and use lncli to run the aforementioned command against the lnd instance that Zap runs for you.

Hm. Sounds a tad intense, but probably just cause I don't really know what any of that means. Anychance you'd be able to point me in the right direction for doing that?

If you hop into our slack channel I can probably better help you there

@mrfelton Bit of a big delay here, but wanted to do this now if you have sometime. Let me know!

DONE.
Okay. I found a way to return our funds.
if your funds are stuck like mine, I mean you send them from Zap (on-chain) wallet to Zap Lightening and is stuck on Pending.
What you need is to copy your recovery words, download new app on a different PC and recover (!) your wallet from the seed numbers you copied before. if you don't have a different PC you can use virtualization software like virtualbox (https://www.virtualbox.org) to create a virtual PC and install Zap on it.
After that, when you open your Zap on that new PC you will see all your coins on On-Chain, it means you can send it back to your original account/address or any other address


esp-idf demo sets LEDs to full white

I'm running an ESP32 Thing from Sparkfun, driving a Neopixel 12 LED ring.
The Arduino demo works without issue after configuring the "strand" to be 12 pixels long.  The esp-idf demos, though, both set all LEDs to full white brightness.  Have tried a few times to make sure I didn't screw anything up, and either I'm always screwing something up or there's something incompatible somewhere.  Any ideas?

If you're using the very latest version of these drivers as well as the very latest IDF this ought to be working... however, I'm not as familiar with the ESP32 Thing so maybe it's a timing issue?
There are certain timing defaults set in sdkconfig.defaults that might be an issue unique to that board, but I don't think that's the problem here. I do recall there being some unique things about the Thing versus the regular board so that might be part of the problem. See this thread: https://esp32.com/viewtopic.php?t=2800
So, per that thread, run make menuconfig and set Component Config -> ESP32-Specific -> Main XTAL frequency to 26 MHz and try it again (I'm pretty sure my extra defaults won't affect this but let's see how this one change goes first).

That fixed it entirely.  Great find and thanks for the help!

You're welcome - Glad it worked out!


feat(frontend): more changes in PR2

Sorry, we do not accept changes directly against this repository. Please see
CONTRIBUTING.md for information on where and how to contribute instead.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#2  👈
#1
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @olga-mcbfe and the rest of your teammates on  Graphite


WIP: Don't use PolymerGestures

Refs #222
Problems/TODO:

[ ] onTrackEnd on nodes not always called, causing move transaction to stay open.
[ ] Dragging nodes moves them too fast. The delta values are different now than before? Or the event rate?
[ ] Does not support non-Chrome mobile. Need to add the hacks in hammer.js to the non-PointerEvent backends
[ ] Update uses of PolymerGestures up and down events (in menus). Using panup, pandown?

Can we remove bower.json after this PR?

No, to remove Polymer completely also need to rewrite the tests and examples to use the React components instead of Polymer elements.

We should also remove the bower install postinstall step soonish

Some issues related to panend upstream: hammerjs/hammer.js#845 hammerjs/hammer.js#707

Removed WIP marker now. Note that there are two minor UX regressions (see CHANGES), but hopefully  these can be fixed later.


DOC: Search Results button enhancement

What's wrong with the existing documentation
Instead of putting the search button here, we can put a link saying 'Search Results' that can direct them to other Search Result page that asks for users input, with that we can add better hover effects and make button look more visible.
Add ScreenShots

we can do this similar to:

Record

[X] I agree to follow this project's Code of Conduct
[X] I have checked the existing issues
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

I would like to fix this issue.

is your feature request related to a problem? Please describe.
Currently search has not such a good style .
Describe the solution you'd like
Adding margin, padding and set allginment and change in style and changes according to search bar
Additional context
Assign me this work under GSSoC
Sir i'm Experienced in Frontend Developer , there are some my work link
https://www.shinnextream.in/
https://innovationnitrr.netlify.app/
so, please assign me

@AditiPrabhakar you wanna help with designing?

@Satyam1923 Picture this: a seamless user experience that not only enhances visibility but also streamlines navigation. Instead of a conventional search button, why not entice users with a sleek and inviting "Search Results" link? This link will seamlessly guide them to a dedicated Search Results page, where they can input their query with ease.
But wait, there's more! Alongside this intuitive link, let's sprinkle in some captivating hover effects to elevate the visual appeal and draw users in. With every hover, they'll be greeted with a delightful animation that sparks curiosity and encourages exploration.
I'm enthusiastic about taking on this task and transforming our search functionality into something truly spectacular. Please entrust me with this assignment under GSSoC'24, where I'll infuse creativity and innovation into every aspect. Together, let's create an experience that not only meets but exceeds user expectations!

Done.


Removing the unused telemetry events

This PR is the last step in reclassification of Telemetry events.
After we migrated from deprecated npm dependency "vscode-extension-telemetry" to new dependency "@vscode/extension-telemetry" the Extension Name changed from "mindaro" to "mindaro.mindaro", I classified only the events which are required (these events will be saved in the main table RawEventsVSCodeExt) and left the unwanted events as is (these events will be stored in unclassified table RawEventsVSCodeExtUnclassified)
This work was done and was released successfully in the Nov release. And made sure all the classified events are being logged in the appropriate table.
Now as the final step I'm cleaning the code so that the logs will not be logged under unclassified table which is waste of data/not required data.
Here is the consolidated list of events.
https://msazure.visualstudio.com/CloudNativeCompute/_wiki/wikis/personalplayground/594061/Deprecation-of-npm-dependency-vscode-extension-telemetry-

Codecov Report
Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (2b11e5b) 73.67% compared to head (d46ff78) 73.53%.

Files
Patch %
Lines

src/clients/KubectlClient.ts
0.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #83      +/-   ##
==========================================
- Coverage   73.67%   73.53%   -0.15%
==========================================
  Files          37       37
  Lines        3199     3132      -67
  Branches      149      149
==========================================
- Hits         2357     2303      -54
+ Misses        837      824      -13
  Partials        5        5

Flag
Coverage Δ

unittests
73.53% <0.00%> (-0.15%)
:arrow_down:

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

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


LHN - HTML encoding is visible in chat switcher

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Navigate to a conversation
Send the following message

>>> hey

Open the chat switcher and see the message.

Expected Result:
HTML should not be visible in any area of the app.
Actual Result:
HTML encoding is visible in the preview message.
Workaround:
None, visual issue.
Platform:
Where is this issue occurring?
Web ✔
iOS
Android
Desktop App
Mobile Web
Version Number: 1.0.70-0
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:

Expensify/Expensify Issue URL:
View all open jobs on Upwork

From @cead22 https://expensify.slack.com/archives/C01GTK53T8Q/p1623969778428700

Showing html encoding   in CMD+K quick switcher and LHN

Triggered auto assignment to @Luke9389 (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

Not produced

I think this may have been fixed by this PR: https://github.com/Expensify/expensify-common/pull/381

Pretty sure this is no longer happening

Reopening this issue.
@coleaeason was able to reproduce https://expensify.slack.com/archives/C01GTK53T8Q/p1627404387349800

LHN Shows HTML elements instead of stylized text

Wops this was not reproducible anymore by Cole. Will reopen if this happens again


[SPARK-45547][ML] Validate Vectors with built-in function

What changes were proposed in this pull request?
Validate Vectors with built-in function
Why are the changes needed?
with built-in function, the logic might be optimized further
Does this PR introduce any user-facing change?
no
How was this patch tested?
ci
Was this patch authored or co-authored using generative AI tooling?
no

Is this actually faster? I would be surprised

scala> import org.apache.spark.ml.linalg._
     |
     | val df = Seq.range(0, 1000000).map(i => (i,Vectors.dense(Array.fill(256)(1.0)))).toDF("i", "vec")
     |
import org.apache.spark.ml.linalg._
val df: org.apache.spark.sql.DataFrame = [i: int, vec: vector]

scala> df.count()
23/10/17 16:28:31 WARN TaskSetManager: Stage 0 contains a task of very large size (1391 KiB). The maximum recommended task size is 1000 KiB.
val res0: Long = 1000000

scala> val validateUDF = udf { vector: Vector =>
     |     vector match {
     |       case dv: DenseVector =>
     |         dv.values.forall(v => !v.isNaN && !v.isInfinity)
     |       case sv: SparseVector =>
     |         sv.values.forall(v => !v.isNaN && !v.isInfinity)
     |     }
     |   }
val validateUDF: org.apache.spark.sql.expressions.UserDefinedFunction = SparkUserDefinedFunction($Lambda$4177/0x000000b00198fa00@17640714,BooleanType,List(Some(class[value[0]: vector])),Some(class[value[0]: boolean]),None,false,true)

scala> val validatedCol = forall(unwrap_udt(col("vec")).getField("values"), v => not(v.isNaN) && abs(v) =!= expr("double('inf')"))
val validatedCol: org.apache.spark.sql.Column = forall(unwrap_udt(vec)[values], lambdafunction(and(`!`(isNaN(x_0)), `!`(`=`(abs(x_0), double(inf)))), x_0))

scala> val start = System.currentTimeMillis; df.select(bool_and(validateUDF(col("vec")))).head(); System.currentTimeMillis - start
23/10/17 16:28:47 WARN TaskSetManager: Stage 3 contains a task of very large size (176683 KiB). The maximum recommended task size is 1000 KiB.
val start: Long =<PHONE_NUMBER>779
val res1: Long = 4562

scala>

scala> val start = System.currentTimeMillis; df.select(bool_and(validatedCol)).head(); System.currentTimeMillis - start; System.currentTimeMillis - start
23/10/17 16:28:52 WARN TaskSetManager: Stage 6 contains a task of very large size (176683 KiB). The maximum recommended task size is 1000 KiB.
val start: Long =<PHONE_NUMBER>903
val res2: Long = 4637

I did a quick test, it seems there is no significant change. Especially, this validation is only performed once.
Using built-in functions can help simplify the codes and get potential benefit from SQL optimization.

Ok seems fine if you like

thanks, I happen to find another a few similar places, please hold on and let me add them all

on second thought, let's let it alone

@srowen would you mind taking another look? I think all vector validation related udfs are covered in this PR.
when we check the values, the performances are similar;
when we check the size, the new implementation is much faster since it can directly utilize the size field in VectorUDT

merged to master


test: rename integrity check index.test.ts, remove hello.yml

Can you please add an if (selectedOutputSN.studentRaws !== null) condition for every test case that uses the studentRaws so it doesn't get a warning?
Please use an unique integer RFID for each dummy record.

I found a bug where it includes the 203100000 SN when selecting the student number from 2030 to 2030. Thanks, Gab!

okay wait lang


Getting index file from existing model

Let's say I downloaded someone's model, or made one myself, but forgot the index file. Is there a way to recreate it from what was already trained, or do I have to restart from 0?

You can click "train the index" and model is not needed to train again

You can click "train the index" and model is not needed to train again

If you alaready have the feature you don't need to extract them again, or you should click data preprocess and feature extract


correct doc link in example deployment

[skip-ci]

@strokyl thank you for the PR if you could update your commit message to doc: correct doc link in example deployment I can get this PR merged.
ref: https://github.com/coreos/etcd-operator/blob/master/CONTRIBUTING.md#contribution-flow

done

@strokyl, thanks!


all arguments must be optional

@rultor release, tag is 0.0.9

@rultor release, tag is 0.0.9

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

@rultor release, tag is 0.0.9

@yegor256 Done! FYI, the full log is here (took me 7min)


Missing entry in gff file

Hey!
I've been parsing ggcaller output to get locations of genes in each genome. To do this, I look in the GFF file for each genome, and find the entry with the right ID. If I understand correctly, I can get the internal ggcaller ID for a gene within a specific genome by looking at the corresponding genome column and gene row in gene_presence_absence_roary.csv, and then from that construct the GFF ID by taking the final number in the ggcaller ID, and appending it to the genome ID. If an entry is empty in gene_presence_absence_roary.csv, then that gene is not present in the given genome.
In an example I've been looking at, there is a ggcaller ID given in gene_presence_absence_roary.csv, but when I construct the GFF ID as I described above, there is no corresponding entry in the GFF file for that genome. The gene is also marked as being present in gene_presence_absence.Rtab file (in fact, it appears to be a core gene).

Yup, looks like you're right about it being a gene that was called over a contig break - minimapping it back gives me two matches, mapping the beginning of the gene to the end of the genome, and the end of the gene to the beginning of the genome. (That clears up my concerns, so do you still want the output files?)
Does a gene getting called across a contig break get recorded anywhere in the ggcaller output?


Adding debugging tutorial

Content looks good, I like the examples!  I think we can come up with a better way to show the solutions though...I'll see if I can quickly put something together.


Homebridge won't start with this plugin.

Hi,
I got this error when I installed this plugin and configure it as in the example.
Latest Homebridge 1.1.1
I'm newbie...
Before this samsungtv-control plugin I installed Xiaomi Air Purifier2 and Desk Lamp2 and it worked fine.
I've got Samsung SmartTV UE55F6770 2013
Error loading accessory requested in your config.json at position 2
[6/30/2020, 00:26:37] Error: The requested accessory 'SamsungTV' was not registered by any plugin.
at PluginManager.getPluginForAccessory (/usr/local/lib/node_modules/homebridge/src/pluginManager.ts:198:15)
at /usr/local/lib/node_modules/homebridge/src/server.ts:347:37
at Array.forEach ()
at Server._loadAccessories (/usr/local/lib/node_modules/homebridge/src/server.ts:330:29)
at Server.start (/usr/local/lib/node_modules/homebridge/src/server.ts:156:12)
at cli (/usr/local/lib/node_modules/homebridge/src/cli.ts:80:10)
at Object. (/usr/local/lib/node_modules/homebridge/bin/homebridge:17:22)
at Module._compile (internal/modules/cjs/loader.js:1138:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:1158:10)
at Module.load (internal/modules/cjs/loader.js:986:32)

The plugin has been reimplemented completely and can be found here now. I am closing all issues in this repository therefore. Please consider testing the new plugin and help me improve it.


name modification: piecemeal microautophagy of the nucleus

Hi David
Following from the Autophagy editors comments. In the phrase 'ATG1/ULK1 ', ULK1 is the human gene symbol, but Atg1 is the symbol used in other species. As all other Atgs listed in GO term names are in the format Atg# I think it would be sensible to revise the names of all the 5 terms which have ATG1/ULK1 and instead use Atg1/ULK1. AND revise the definitions (edits not listed here)
GO:1904745 ATG1/ULK1 kinase complex assembly
change to  GO:1904745 Atg1/ULK1 kinase complex assembly
GO:1905864 regulation of ATG1/ULK1 kinase complex assembly
change to: regulation of Atg1/ULK1 kinase complex assembly
GO:1905865 negative regulation of ATG1/ULK1 kinase complex assembly
change to: negative regulation of Atg1/ULK1 kinase complex assembly
GO:1905866 positive regulation of ATG1/ULK1 kinase complex assembly
change to: positive regulation of Atg1/ULK1 kinase complex assembly
GO:1990316 ATG1/ULK1 kinase complex
change to:  Atg1/ULK1 kinase complex
Please confirm if you agree and I will revise Table S1
Thanks
Ruth

I do not disagree with having 'Atg1' instead of 'ATG1' if this is used to refer to a broader range of species (i.e. not just human, where the correct format is ATG1). So ok with me. Thanks.


Update pl.json

Thanks!

Thanks!

Some additional messaging was added to the plugins.manage.message_confirm_disable translation item to clarify what happens when a plugin is disabled.

Some additional messaging was added to the plugins.manage.message_confirm_disable translation item to clarify what happens when a plugin is disabled.


This extension interferes with terminals including those for other extensions

When this extension is installed it causes the command  C:/ProgramData/Anaconda and then "conda activate base" to be run in terminal windows, including in REPL windows for other languages.  This causes errors before the Python REPL starts.
The interesting thing is that enabling this extension causes the same behavior for other terminals and extensions in VS Code even when they have nothing to do with Python. So all shells start with these two commands whether they make sense or not. Also the first two lines of the Julia REPL are those two lines -- which obviously make no sense as Julia statements.
Turning off the vscode-python extension fixes the problem, but that's a pity.
I'll also note here that when the a Python REPL is started the name given to it is "REPL" which is a bit non-specific as there are other languages that have REPLs. "Python" would be a better name.
Environment data

VS Code version: 1.39.2
Extension version (available under the Extensions sidebar): 2019.10.44104
OS and version: Windows 7
Python version (& distribution if applicable, e.g. Anaconda): Anaconda Python 3.6.4
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): None
Relevant/affected Python packages and their versions:
Jedi or Language Server? (i.e. what is "python.jediEnabled" set to; more info #3977): XXX

Expected behaviour
Starting a terminal or REPL unrelated to python should not cause conda start-up commands to run.
Actual behaviour
Opening any new terminal, including a shell or a Julia REPL causes the conda start up commands to run in that shell or REPL, regardless of whether these make sense or can be run. Here is what I see when opening a command shell terminal example:
Microsoft Windows [Version 6.1.7601]
Copyright (c) 2009 Microsoft Corporation.  All rights reserved.
C:\Data\Projects\repos\home-dev>C:/ProgramData/Anaconda3/Scripts/activate
(base) C:\Data\Projects\repos\home-dev>conda activate base
(base) C:\Data\Projects\repos\home-dev>
And here is what I see when running Julia: Start REPL
julia> C:/ProgramData/Anaconda3/Scripts/activate
ERROR: syntax: "/" is not a unary operator
julia> conda activate base
ERROR: syntax: extra token "activate" after end of expression
julia>
Steps to reproduce:

Install an Anaconda distribution
Install the extension
Set the python interpreter for VS Code to point to the Anaconda python.
Open a terminal using New Terminal under the Terminal menu.

Disabling the python extension solves the problem and sets the behavior to normal.

Logs
Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)

Output from Console under the Developer Tools panel (toggle Developer Tools on under Help; turn on source maps to make any tracebacks be useful by running Enable source map support for extension debugging)

It seems you would like to be able to turn off environment activation, which can be done by setting "python.terminal.activateEnvironment": false. Otherwise there's no good way for us to know your intent easily when creating a terminal if you have activated the extension to make it conditional.


feat: upgrade symfony to v7

allows for symfony greater than 6
upgrades aws-sdk-php

Thank you for the contribution. I am currently on vacation and will review the changes and merge them as soon as I get back (in the next couple of days).

Any updates on the review ? I any help needed testing it ?

Any progress on this? I would be a huge help to get this merged

Hello, another one to ask but is this schedulded ?


Can't get library to run test.

Hola!
Im having a hard time getting this library to work. This code from the homepage just sits there forever and will never finish:
  var speedTest=require('speedtest-net');

  test=speedTest({maxTime:5000});

  test.on('data',function(data){
    console.dir(data);
  });

  test.on('error',function(err){
    console.error(err);
  });

And this code:
var speedTest=require('speedtest-net');

speedTest.visual({maxTime:1000,log:true,maxServers:4},function(err,data){
  console.log(data);
});

Outputs this:
Testing from <IP_ADDRESS> at Time Warner Cable, expected dl: 2.59MB/s, expected ul: 0.32MB/s

And never finishes.
Not sure what the issue is. Is there anything I can run to help? I tried logging more events and it gets the full list of servers, but never the best servers.
Thanks!

Specifying a server works, don't worry about looking into this yet, I should be able to figure it out!

I don't know why it wouldn't do anything. It works for me :). You could try adding lots of console.log's throughout the code (especially pingServers / pingServer /getHttp functions).
I'm closing this for now until there's more info.


Suggested design changes for version 0.10.

I think these changes would eventually come up anyways, so here are my suggestions on that.

Add optional abstract callback function in BaseSampler and BasePruner, which is triggered for each trial on execution of trial.report(), supplying the trial_id and reported score.

Add functionality for shared user variables across trials. The current storage.get/set_user_attr expects a trial_id. There should be additional functions to access serialized user_attr (perhaps based on trial_id or report timestamp) so that finding moving averages or last-k of user_attr is simple.

A really cool feature for the CLI to have would be to directly input an executable and a list of tunable command line arguments that the executable supports (along with the range), and wrap the executable as an optuna objective (a trial's result can be identified using a supplied regex to look for in the executable's output).
e.g. Let's say there is train_nn.py, that accepts arguments like --num_layers, --hiddens, --lr, --optimizer. Then running something like

$ optuna study optimize train_nn.py --n-trials=100 --study $STUDY_NAME --storage sqlite:///example.db \
--params num_layers:suggest_int(1,3) hiddens:suggest_int(1, 128) \
lr:suggest_uniform(0.00001,0.001) optimizer:suggest_categorical('Adam', 'MomentumSGD') \
--regex "Accuracy: ([0-1]+(?:\.[0-9]+)?)"

to run the optimization directly would be amazingly convenient.

@sile What is your opinion on the suggestions?

Thank you for your useful suggestions.

Add optional abstract callback function in BaseSampler and BasePruner, which is triggered for each trial on execution of trial.report(), supplying the trial_id and reported score.

As mentioned in https://github.com/pfnet/optuna/issues/346, we are planning to expose new sampler/pruner interfaces in the near feature. I think the new interface will meet your needs.
For example, new BaseSampler will have additional methods named before and after.
after is triggered each time when a trial execution finished, and the method takes FrozenTrial (this contains trial_id and the reported score) as the argument.

Add functionality for shared user variables across trials.

I think BaseStorage.set_study_user_attr and BaseStorage.get_study_user_attrs can be used for this purpose.

There should be additional functions to access serialized user_attr (perhaps based on trial_id or report timestamp) so that finding moving averages or last-k of user_attr is simple.

To keep storage API simple, we basically recommend calling BaseStorage.get_all_trials method and applying necessary processing to the resulting trials in your sampler or pruner (instead of adding new methods to BaseStorage).

A really cool feature for the CLI to have would be to directly input an executable and a list of tunable command line arguments ...

Yes, it's very cool! Coincidentally, we were discussing this feature last month.
The concrete syntax and interface are under consideration, but we would like to provide the feature to Optuna users.

As mentioned in #346, we are planning to expose new sampler/pruner interfaces in the near feature. I think the new interface will meet your needs.

I wasn't aware of that. That should be really helpful in adding new pruners and samplers.

To keep storage API simple, we basically recommend calling BaseStorage.get_all_trials method and applying necessary processing to the resulting trials in your sampler or pruner (instead of adding new methods to BaseStorage).

I think it would be better to compromise on simplicity a little in exchange of performance gains(via direct database queries instead of python post procrssing). Plus, having extra functions available for the user doesn't hurt in the very least.

Yes, it's very cool! Coincidentally, we were discussing this feature last month.
The concrete syntax and interface are under consideration, but we would like to provide the feature to Optuna users.

That's amazing! If possible, I would definitely like to contribute to the design.

@Rishav1

That's amazing! If possible, I would definitely like to contribute to the design.

We'll release our prototype soon. Your feedback would be quite helpful, then.

@Rishav1

I think it would be better to compromise on simplicity a little in exchange of performance gains(via direct database queries instead of python post procrssing).

If we find that the performance impact is severe and the method is beneficial for many users/developers, we will consider to implement it to the storage. However, it is hard to remove the method once implemented as a part of the public interface, so we think that it is better that you try it with individual samplers and pruners at first.

FYI: https://github.com/pfnet/optuna/pull/375 (new sampler interface)


[editor]: Support ico in preview

Please add support for .ico files to the editor preview.
Generated by adamraichu.zip-viewer
Great extension btw

Done! Should be published ~10-15 minutes after CI clears.


Plugins: ubuntu - account for silversearcher

The alias 'ag' in the ubuntu plugin causes conflicts
if the user also has The Silver Searcher installed. This patch checks
to see if the silversearcher binary is available, if so,
the aliases is not made.
This addresses issues #3633 and #3844

@KhasMek you should probably add a "toggle-on" variable to apply this alias: when an environment variable named APPLY_AG_ALIAS is "true", apply the alias. Otherwise don't (also when ag-silversearcher is installed). This way, people will be given the choice whether to use this or not in cases where they do or don't have ag-silversearcher.
wdyt?

@oryband that is certainly doable, nor am I really opposed to it. However, IMHO, all the extra characters for the flag check in the plugin, then setting the flag either via CLI or in a dotfile seems like a bit of a waste when the user could just manually set the alias via CLI or a dotfile for the occasions they wish to use it over Silver Searcher.
I think it may be best to wait to see the overall opinion of the project/plugin maintainers before any further modification to the PR is made. Sound good to you?

@KhasMek yeah sure. Alternatively we can just document this "feature" and explain how to override it. :)

@blueyed and I are working on a safe aliasing system which should take care of that kind of stuff.

ping?

Ouch it has been a while... sorry guys.
For reference, the discussion I was referring to began with https://github.com/robbyrussell/oh-my-zsh/issues/3682#issuecomment-78437079
I have to read that again and get started.


Request: update your ores with tool requirement / min mining level

I have an issue with your ores now to explain the issue in detail my pick axe is a tinkers construct pickaxe made with  paper head,paper binding/knightslime handle with 5 reinforced plates added & a obsidian sharpening kit added to the tool.
So the tool will mine Cobalt level ores and will mine ores faster if the tool is a higher minging level then the ore in question.
Your ores are not either requiring a pickaxe to mine or they don't have a min mining level set so mining each one of your ores is like mining obsidian...
was wondering if this can be corrected in a future update?

This isn't on my end, blocks don't set "min mining levels" for themselves.  It uses the same material as any other ore.


Refactor hyperdrive create and update commands to match each other's behavior and address bug with individual parameters on creation

What this PR solves / how to test
Fixes SQC-338
fix: make individual parameters work for `wrangler hyperdrive create` when not using HoA

`wrangler hyperdrive create` individual parameters were not setting the database name correctly when calling the api.

refactor: use same param parsing code for `wrangler hyperdrive create` and `wrangler hyperdrive update`

ensures that going forward, both commands support the same features and have the same names for config flags

feature: allow using a connection string when updating hyperdrive configs

both `hyperdrive create` and `hyperdrive update` now support updating configs with connection strings.

Author has addressed the following

Tests

[ ] TODO (before merge)
[X] Tests included
[ ] Tests not necessary because:

E2E Tests CI Job required? (Use "e2e" label or ask maintainer to run separately)

[X] I don't know
[ ] Required
[ ] Not required because:

Changeset (Changeset guidelines)

[ ] TODO (before merge)
[X] Changeset included
[ ] Changeset not necessary because:

Public documentation

[X] TODO (before merge)
[ ] Cloudflare docs PR(s):
[ ] Documentation not necessary because: no changes

Looks good to me from the Hyperdrive Team side of things.


为什么我没有看到性别的图标 这个很重要

Icon name
，
Use cases
，
Design ideas
，
Checklist

[X] I have searched if someone has submitted a similar issue before and there weren't any. (Please make sure to also search closed issues, as this issue might already have been resolved.)
[X] I have searched existing icons to make sure it does not already exist and I didn't find any.
[X] I am not requesting a brand logo and the art is not protected by copyright.
[X] I am not requesting an icon that includes religious, political imagery or hate symbols.
[X] I have provided appropriate use cases for the icon(s) requested.

Duplicate of https://github.com/lucide-icons/lucide/issues/560

Duplicate of https://github.com/lucide-icons/lucide/issues/560


Expose imageStyle prop

Hi, thanks for this repo. This is an awesome contribuiton.
I'm wondering if we can expose a new prop named imageStyle, which will append to the existing styles.image.
With this new prop we can customize the resizeMode, aspecRatio and other Image specific props.
I'm openining a PR to handle this.

@alexandreh92 closing this due to inactivity, feel free to reopen


Samples validation

Please, check samples - some of them look incorrect:

</td> - all samples are the same
match("ab:dcd","ab123cd"); returns 0 - should not it return 1?
match("ab: cd","ab cd"); returns 0 - should not it return 1?

"An expression that is followed by an asterisk" - asterisk is not displayed in any expression

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

ID: 27a2af11-bc8e-9ffd-802c-12b3a91a739a
Version Independent ID: dd23ab4f-2e5f-b99f-deae-28df175462ba
Content: X++ string runtime functions - Finance & Operations | Dynamics 365 | #MSDyn365FO
Content Source: articles/dev-itpro/dev-ref/xpp-string-run-time-functions.md
Service: dynamics-ax-platform
GitHub Login: @RobinARH
Microsoft Alias: robinr

The code is correct. The second one
match("ab:dcd","ab123cd"); returns 0
is correct since 123 is more than the one digit provided for. You may want to add this example to make that obvious:
match("ab:d*cd","ab123cd"); returns 1
(i.e. adding a * operator after the :d, which means "0 or more digits")
The example:
match("ab: cd","ab cd"); returns 0. Only the first space is matched
is also correct, but hinges on the fact that there should be TWO blanks after the : in the pattern.
I also noticed
A right angle bracket (>) or a dollar sign (?) at …


docker images build program does not check if doker is installed

Despite the docker not being installed in the system, the output of the program writes that the images were successfully created.

The docker images build program is not in this repo but neuropoly/sct_docker.


lambda get_function response is missing elements

Scenario
During unit testing, I create a function using create_function() and immediately query it using get_function(). I observe that not all response elements documented at get_function appear in the response. The Amazon documentation does not suggest that these elements are optional.
Moto response
Here is a representative response from get_function() using moto:
{
    "ResponseMetadata": {
        "HTTPStatusCode": 200,
        "HTTPHeaders": {},
        "RetryAttempts": 0
    },
    "Configuration": {
        "FunctionName": "CirrusScan_scoutsuite_2048",
        "FunctionArn": "arn:aws:lambda:us-west-1:123456789012:function:CirrusScan_scoutsuite_2048",
        "Role": "arn:aws:iam::123456789012:role/GD-CirrusScan-TaskRole",
        "CodeSize": 0,
        "Description": "",
        "Timeout": 900,
        "MemorySize": 2048,
        "LastModified": "2023-06-29T16:11:13.553511000Z",
        "CodeSha256": "d7eba8aaa078d6703afc83f455a89e3845c0e2c6ed67cede76d53a3c6c6188a2",
        "Version": "$LATEST",
        "VpcConfig": {
            "SubnetIds": [],
            "SecurityGroupIds": []
        },
        "TracingConfig": {
            "Mode": "PassThrough"
        },
        "Layers": [],
        "State": "Active",
        "LastUpdateStatus": "Successful",
        "PackageType": "Image"
    },
    "Code": {
        "RepositoryType": "ECR",
        "ImageUri": "123456789012.dkr.ecr.us-west-1.amazonaws.com/scoutsuite:latest:latest",
        "ResolvedImageUri": "123456789012.dkr.ecr.us-west-1.amazonaws.com/scoutsuite@sha256:d7eba8aaa078d6703afc83f455a89e3845c0e2c6ed67cede76d53a3c6c6188a2"
    }
}

AWS response
Here is an actual response from get_function() formatted the same way
{
    "ResponseMetadata": {
        "RequestId": "6b227b6f-0d93-425b-a4fa-4935958c6e61",
        "HTTPStatusCode": 200,
        "HTTPHeaders": {
            "date": "Thu, 29 Jun 2023 16:56:47 GMT",
            "content-type": "application/json",
            "content-length": "1535",
            "connection": "keep-alive",
            "x-amzn-requestid": "6b227b6f-0d93-425b-a4fa-4935958c6e61"
        },
        "RetryAttempts": 0
    },
    "Configuration": {
        "FunctionName": "CirrusScan_scanners_2048",
        "FunctionArn": "arn:aws:lambda:us-west-2:111111111111:function:CirrusScan_scanners_2048",
        "Role": "arn:aws:iam::111111111111:role/GD-CirrusScan-TaskRole",
        "CodeSize": 0,
        "Description": "",
        "Timeout": 900,
        "MemorySize": 2048,
        "LastModified": "2023-06-26T16:32:01.000+0000",
        "CodeSha256": "3b88c938a98ed9a1d198dc5b7de92bcd7a89f38d83d3f070bf29cd7d13f1869a",
        "Version": "$LATEST",
        "TracingConfig": {
            "Mode": "PassThrough"
        },
        "RevisionId": "e7985718-adc7-47b6-a6e2-47d621a85ef0",
        "State": "Active",
        "LastUpdateStatus": "Successful",
        "PackageType": "Image",
        "ImageConfigResponse": {
            "ImageConfig": {
                "EntryPoint": [
                    "/app/.venv/bin/python",
                    "-m",
                    "awslambdaric"
                ],
                "Command": [
                    "cirrusscan_scanners.wrapper.handler"
                ]
            }
        },
        "Architectures": [
            "x86_64"
        ],
        "EphemeralStorage": {
            "Size": 512
        },
        "SnapStart": {
            "ApplyOn": "None",
            "OptimizationStatus": "Off"
        }
    },
    "Code": {
        "RepositoryType": "ECR",
        "ImageUri": "111111111111.dkr.ecr.us-west-2.amazonaws.com/scanners:latest",
        "ResolvedImageUri": "111111111111.dkr.ecr.us-west-2.amazonaws.com/scanners@sha256:3b88c938a98ed9a1d198dc5b7de92bcd7a89f38d83d3f070bf29cd7d13f1869a"
    }
}

Desired outcome
Our code under test accesses the following Configuration response elements which don't presently exist in moto:

Architectures
ImageConfigResponse

I would like to see, minimally, the above keys (with valid values) added to get_function() responses.

Marking it as an enhancement @rbailey-godaddy. Welcome to Moto!

@bblommers I can take a look


Enum

https://github.com/yhara/shiika/commit/56f57ef9222e1ffd973124310a44ff6b3a27c9af

implementation is done by #286, #288, #289.
TODO: update docs


Add Support for lower case characters in the teleTAN screen

Checklist

[x] Test your changes as thoroughly as possible before you commit them. Preferably, automate your test by unit/integration tests.
[ ] If this PR comes from a fork, please Allow edits from maintainers
[x] Set a speaking title. Format: {task_name} (closes #{issue_number}). For example: Use logger (closes #41)
[ ] Link your Pull Request to an issue (if applicable)
[ ] Create Work In Progress [WIP] pull requests only if you need clarification or an explicit review before you can continue your work item.
[x] Make sure that your PR is not introducing unncessary reformatting (e.g., introduced by on-save hooks in your IDE)
[x] Check our Contribution Guidelines

Description
The current teleTAN entry screen only supports upper case characters, this is wrong according to the official teleTAN entry spec. This PR fixes these issues.

I disagree. The text clearly states that the TAN is case insensitive. That change would be quite confusing for the end users.

@tweimer good catch! The text for all screens is currenlty still in a temporary state, but obvious mismatches between what is on screen and the functionality should still not happen.
Fixed now thanks for the heads up 👍


Sana checkpoint trained with SD-3 VAE

Hi,
Thank you for open-sourcing your code and trained models. Could you release the Sana text2image model trained with either SD-XL or SD-3 VAE?

Yes, DC-AE is fast, but ruined image details, we also try f64 - same result( We try realistic, anime - https://imgsli.com/MzI0MDg3
May be take a look at auraflow vae?  it's opensourced as i know, comparable to flux/sd3 vae and significantly better DC-AE
Auraflow vae

DC-AE

Or may be train small model for converting from DC-AE to AuraFlow in latent space directly, what do you think, is this possible?

Makes sense. It's surprising that both Sana obtains better FID scores with this VAE, despite worse reconstruction results.
From the paper:
although AE-F8C16 exhibits the best reconstruction ability (rFID: F8C16<F16C32<F32C32), we empirically find that the generation results of F32C32 are superior
I wish they'd release checkpoints trained with other VAEs, allowing users to choose the one that works best for their specific dataset when fine-tuning.

@lawrence-cj Do you plan to release Sana trained with f8c4 / f8c16 VAEs?


i18n(ko-KR): update mdx.mdx

Description (required)

update mdx.mdx
#8224

Related issues & labels (optional)

Closes #
Suggested label:

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

ko
guides/integrations-guide/mdx.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


Add a silent output

When using goss as a helathchecking endpoint, it may be desirable to
suppress verbose test output so that potential malicious entities can't
scrape healthcheck endpoints for system information.
Add the silent output, which simply checks whether any tests have
failed and returns 1 or 0 accordingly, without printing any output.

Awesome! Code looks good, can you also update the documentation as part of this pull request.

Oh of course! My bad.

I've added some one-line output descriptions. They're maybe a bit long, but I thought it was worth expanding on the use-case, since it's perhaps non-obvious.

I'll merge/release with next release. Thanks :+1:


Prevent AccordionItem from toggling open/closed when typing space

Describe the feature in detail (code, mocks, or screenshots encouraged)
Hi,
I want to have an input field in an AccordionItem bar, so that users can change a title.
I have the issue though, that when a user types a space, then the accordion toggles its closed state. Even if I try to stop the propagation of the event, it still happens.
Is there a way to stop the toggling from happening when typing into the input field?
This is a svelte kit application. Below is some example code.
<script>
	import { Accordion, AccordionItem } from '@skeletonlabs/skeleton';

	let title = 'The title';
	let summary = 'The summary';
</script>

<Accordion>
	<AccordionItem open>
		<svelte:fragment slot="lead">
			<input
				type="text"
				bind:value={title}
				class="bg-transparent"
				on:keydown|stopPropagation
				on:keyup|stopPropagation
				on:keypress|stopPropagation
				on:click|stopPropagation
			/>
		</svelte:fragment>
		<svelte:fragment slot="summary">
			<input
				type="text"
				bind:value={summary}
				class="bg-transparent"
				on:keydown|stopPropagation
				on:keyup|stopPropagation
				on:keypress|stopPropagation
				on:click|stopPropagation
			/>
		</svelte:fragment>
		<svelte:fragment slot="content">The content</svelte:fragment>
	</AccordionItem>
</Accordion>

What type of pull request would this be?
Enhancement
Provide relevant links or additional information.
AccordionItem keyboard events: https://www.skeleton.dev/components/accordions?tab=keyboard

I see, the slots are inside a button element and space triggers a button for accessibility reasons.
https://github.com/skeletonlabs/skeleton/blob/master/packages/skeleton/src/lib/components/Accordion/AccordionItem.svelte

Hey @FabianHertwig so inputs or other interactive elements inside the lead element are unexpected but I could see how that would cause some issues. Typically that space is reserved specifically for the Accordion label.
It sounds like you've seen, but we implement the keyboard interaction for a11y reasons. Both enter/space should toggle the header when in a focus state:
https://www.w3.org/WAI/ARIA/apg/patterns/accordion/
stopPropagation would have been my initial suggestion for a fix. If that's not working we'll need to review further. Off the top of my head this may require a change to the Accordion source to firing the keyboard event when a child of the lead is selected.
If you're open to this, I'd welcome a PR for this, as myself and the team are fully focused on Skeleton v3 at the moment:
https://github.com/skeletonlabs/skeleton/discussions/2729
Speaking of which, we're taking a very different approach to components in v3. So if you have time it may be worth testing your scenario there as well. Off hand I'm not sure if this scenario would remain or not:
https://next.skeleton.dev/docs/components/accordion/svelte

Thanks for your comment @endigo9740
I just found that on:keyup|preventDefault solves my issue!


Make code a bit more testeable.

refactor gpt api calls to make code testeable.
load(): ability to take arbitrary number of ads.
load(...slots): load arbitrary ads even on initial call.

https://github.com/jaanauati/react-dfp/issues/110


Key Events Propagation Issue on Android TV's Modals

Summary
In our app, users can interact with the following remote control buttons:

Channel Up (CH+)
Channel Down (CH-)
Number Keys (0–9)

For Android TVs with a TV tuner (e.g., Sony Bravia), the app should capture these Key events to prevent them from propagating to the system.
In our app, we override the onKeyDown and onKeyUp methods in MainActivity to consume these key events:
override fun onKeyDown(keyCode: Int, event: KeyEvent?): Boolean {
    when (event?.keyCode) {
        KeyEvent.KEYCODE_0, ..., KeyEvent.KEYCODE_9,
        KeyEvent.KEYCODE_CHANNEL_UP,
        KeyEvent.KEYCODE_CHANNEL_DOWN -> return true
    }
    return super.onKeyDown(keyCode, event)
}

override fun onKeyUp(keyCode: Int, event: KeyEvent?): Boolean {
    when (event?.keyCode) {
        KeyEvent.KEYCODE_0, ..., KeyEvent.KEYCODE_9,
        KeyEvent.KEYCODE_CHANNEL_UP,
        KeyEvent.KEYCODE_CHANNEL_DOWN -> return true
    }
    return super.onKeyUp(keyCode, event)
}

This solution works. Except for React Native Modal (i.e. Android's Dialog).
Observed Issue
When a React Native Modal (Dialog) is presented, KEY_DOWN events are not passed to MainActivity. This causes the CH+, CH-, and 0–9 buttons to still propagate their Key events to the system.
When the Key events propagate to the system, the unexpected behavior occurs:

The app handles the Key event. ✅
The system forcefully closes the app. ❌
The system tunes the TV to a channel. ❌

Why This Happens:

In React Native, the Android Dialog's KEY_UP events are delegated to MainActivity.
However, KEY_DOWN events are not delegated when a modal is active.
https://github.com/react-native-tvos/react-native-tvos/blob/c97452ba91f8f73fc806c63026da43bbd28d5354/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/views/modal/ReactModalHostView.kt#L261

Solution
The solution is to delegate KEY_DOWN events to the MainActivity, i.e., current activity.
--- a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/views/modal/ReactModalHostView.kt
+++ b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/views/modal/ReactModalHostView.kt
@@ -280,6 +280,12 @@ public class ReactModalHostView(context: ThemedReactContext) :
                   return innerCurrentActivity.onKeyUp(keyCode, event)
                 }
               }
+            } else if (event.action == KeyEvent.ACTION_DOWN) {
+              val innerCurrentActivity =
+<EMAIL_ADDRESS>as ReactContext).currentActivity
+                if (innerCurrentActivity != null) {
+                  return innerCurrentActivity.onKeyDown(keyCode, event)
+                }
             }
             return false
           }

Impact of the Proposed Patch for our app
When this patch is applied, the app:

Properly captures the CH+, CH-, and 0–9 button events.
Prevents these events from propagating to the system, which avoids the unintended behavior of the app closing and the TV switching channels.

Considerations
Personal Perspective
I am not deeply familiar with React Native to fully assess the risks of this proposal. However, conceptually:

It seems logical for the dialog in ReactModalHostView to delegate both KEY_UP and KEY_DOWN events to the MainActivity.
I am uncertain why the original implementation only delegates KEY_UP events to the current activity.

Open Questions

Could delegating KEY_DOWN events introduce unintended side effects in other React Native use cases?
Was the decision to handle only KEY_UP events intentional to address a specific use case or constraint?

I wonder if this issue raises the question of whether it should be:
Discussed within the react-native-tvos community:

The problem is highly relevant to Android TV implementations, particularly for devices like Sony Bravia.
The solution could potentially become a react-native-tvos-specific feature.

Elevated to the broader react-native community:

If the behavior of ReactModalHostView affects non-TV platforms, the proposed change could have wider implications.

Thanks
Thank you for taking the time to review this issue. I look forward to your feedback and insights. ❤

Thanks for this detailed analysis! Which version of RNTV are you using?

Hi! I am using v0.76.2-0.

Hey, thanks for the detailed explanation! I think I wouldn't redirect the events directly to the activity's onKeyUp/Down callback since it makes is impossible anymore to detect wether the key events actually belong to the activity or not. Which might be a dealbreaker for some folks, i don't know. Extending ReactActivity with a new explicit callback for this purpose could be better, not sure.
That being said, the requested change seems harmless given that the same logic is there for the keyup events already 😄
Just to be sure, it's not a regression, right?

Hey, thanks for the detailed explanation! I think I wouldn't redirect the events directly to the activity's onKeyUp/Down callback since it makes is impossible anymore to detect wether the key events actually belong to the activity or not. Which might be a dealbreaker for some folks, i don't know. Extending ReactActivity with a new explicit callback for this purpose could be better, not sure.
That being said, the requested change seems harmless given that the same logic is there for the keyup events already 😄
Just to be sure, it's not a regression, right?

Just to be sure, it's not a regression, right?

Hi @yusufyildirim, correct; this is not a regression.

The code in question is from RN core, and appears to have been intended for Android mobile devices. We can absolutely make TV-specific changes in this file (we already have to some extent, in fact). We can wrap any changes in a check that ensures this is only being done for Android TV.
I agree with @yusufyildirim that it would make sense to add a callback to ReactActivity to handle these events, in case they need special treatment.
I also think that your patch should not delegate KEY_DOWN events to the activity if they have codes KEYCODE_ESCAPE or KEYCODE_BACK, since those are intended to be specially handled by the modal's call to onRequestClose, etc.

Hello, everyone, and thank you for your feedback! I really want to improve my React Native TV skills and implement a solution based on your suggestions. I’m currently on holiday, but I have time to work on this next month (December).


remove obsolete models, add missing info about RNN-T

@qpjaada are you satisfied with this PR now that Marek has addressed your questions?
<EMAIL_ADDRESS>could you also review/approve?

@qpjaada are you satisfied with this PR now that Marek has addressed your questions?

I am fine with these changes. If there are no objections - lets have a Speech area. Approving.


Expose information about extended Refs

When resolving references it would be useful to know when there is a schema that was an extended ref. Ie.
given the following schema
{
    type: 'object',
    required: ['a'],
    properties: {
        a: {
            $ref: '#/definitions/typeA',
            description: 'a in obj as a'
        },
        b: {
            $ref: '#/definitions/typeA',
            description: 'a in obj as b'
        }
    },
    definitions: {
        typeA: {
            type: 'number',
            description: 'a in defs'
        }
    }
}

it produces the following dereferenced output.
{
    type: 'object',
    required: ['a'],
    properties: {
        a: {
            description: 'a in obj as a',
            type: 'number'
        },
        b: {
            description: 'a in obj as b',
            type: 'number'
        }
    },
    definitions: {
        typeA: {
            type: 'number',
            description: 'a in defs'
        }
    }
}

There is no way for the consumer to know that properties a and b were once the same pointer

What would you propose as a solution?
Keep in mind that a and b aren't the same object.  They both simply get some of their properties from the same object.

@alan-cha and I are actually interested in the same thing. We would like to keep track of the original references in the resulting dereferenced object.
What if the user could provide an option to retain the original $ref property in addition to the dereferenced value? A consideration with this idea is that the resulting value may trigger additional, superfluous dereferencing in future processing.
Alternatively, what if the user could provide the name of a field to add to a and b that holds the original reference? E.g., if the user states that field to be original-$ref, the resulting data would be:
{
    type: 'object',
    required: ['a'],
    properties: {
        a: {
            description: 'a in obj as a',
            type: 'number',
            original-$ref: '#/definitions/typeA'
        },
        b: {
            description: 'a in obj as b',
            type: 'number',
            original-$ref: '#/definitions/typeA'
        }
    },
    definitions: {
        typeA: {
            type: 'number',
            description: 'a in defs'
        }
    }
}

@luma commit in his fork seems to be what I'm looking for - is it possible to include that in mainline?


⚠ ClamAV has degraded performance

In bcfaf06, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 566 ms

Resolved: ClamAV performance has improved in 023d7e7.