Improve styling of pull-quotes

From: @choldgraf here: https://github.com/executablebooks/myst-theme/pull/328#issuecomment-1992506836
As an example, here are several examples and inspiration for pull quotes.
Can pull quotes be placed a-la columns etc? I often see pull quotes as a sidebar with the text floating around it. For example, here's one from the NYT:

Is this kind of styling possible with CSS grid by default?

Floats and Grids are somewhat incompatible in css, however, I think that we should have the grid system for the blocks, and some other elements, then have nested div that then we can use floats inside to do both the sidebar and margin content in a much easier way. That means top-level blocks are placed in a grid, and we can still get reflow with floats etc.
There are some things to do in that sketch of a plan though (including thinking it through more fully...) -- grids don't have margin collapse, so we have done a bunch of styling to fix the spacing inside of a grid (mb-8s in a bunch of places, etc.). There is also some logic that looks if classes internally have col- or column- and then placing a subgrid in the block for placement.
I will experiment when I look at your margin/sidebar pr tomorrow!:

#341


cass-operator webhook fixes

What this PR does:
Bumps cass-operator version to a new version which has the right failure behaviour in the webhook (see this PR).
Fixes issues in the e2e tests which arise from the wrong namespace being specified for the webhooks.0.clientConfig.service.namespace.
Which issue(s) this PR fixes:
No issue raised.
Checklist

[ ] Changes manually tested
[ ] Automated Tests added/updated
[ ] Documentation added/updated
[ ] CHANGELOG.md updated (not required for documentation PRs)
[ ] CLA Signed:  DataStax CLA

Subject to tests passing I believe that this addresses the issues raised in #397 and is ready for review.

OK @burmanm and @jsanda (whoever sees this first) this looks good to go IMO.
Tests failing

ClusterScoped/MultiDcMultiCluster fails everywhere. Once this is closed I can start looking into why.
MultiDcAuthOnOff, MultiDcEncryptionWithStargate, MultiDcEncryptionWithReaper  fail frequently too, I can't see that there's any necessary connection to the webhook in these failures, and have seen them in many other PRs today.

Are we happy to approve or are there too many failing tests?

This is covered in #365 so we'll close this PR and grab the change via the other.


Mavlink-routerd : command not found

Having issues just running mavlink-routerd --help. Getting mavlink-routerd: command not found. Followed the build instructions provided:
git clone https://github.com/intel/mavlink-router.git
cd mavlink-router
git submodule update --init --recursive
sudo apt install git meson ninja-build pkg-config gcc g++ systemd
sudo pip3 install meson
meson setup build .
ninja -C build
Am I missing anything?

Missing sudo ninja -C build install


Fix PR comment deserialization

Broke in https://github.com/rust-lang/triagebot/pull/1646.
request failed: WebhookError(IssueCommentEvent failed to deserialize

Caused by:
    0: at Path { segments: [Map { key: "issue" }, Map { key: "pull_request" }] }
    1: invalid type: map, expected a boolean at line 1 column 6086)

Apologies!  In hindsight that was an obvious mistake and I should have reviewed the API more closely.  Thanks for the fix!


config_path() undefined

Run:
php artisan vendor:publish --provider="PulkitJalan\GeoIP\GeoIPServiceProvider" --tag="config"
Error:
Call to undefined function PulkitJalan\GeoIP\config_path()
In GeoIPServiceProvider.

This is an error caused by using lumen. config_path does not seem to be available.

Solution 1:
Do a check in your service provider if you wanna support Lumen:
if (function_exists('config_path')) {
    $this->publishes([
            __DIR__.'/config/config.php' => config_path('geoip.php'),
     ], 'config');
}

Solution 2:
I created app/Support/Helpers.php and added:
<?php

if (! function_exists('config_path')) {
    /**
     * Get the configuration path.
     *
     * @param  string $path
     * @return string
     */
    function config_path($path = '')
    {
        return app()->basePath() . '/config' . ($path ? '/' . $path : $path);
    }
}

And loaded that in composer.json:
{
    "autoload": {
        "files": [
            "app/Support/Helpers.php"
        ]
    },
}

As for using the package with lumen you have to uncomment "// $app->withFacades();" and add class_alias(\PulkitJalan\GeoIP\Facades\GeoIP::class, 'GeoIP');

This package is not currently compatible with Lumen, I think solution 1 is probably best to make it compatible. If you can submit a PR I'll have a look and merge it. Thanks


Adding additional logging and to glfw initialization.

I was having some issues initializing glfw locally on Linux/wayland. I found some additional logs helpful.

Resolved.

Neat !


Add Operation: Harsh Doorstop

According to the guide, all three ports (including rcon port) need to be forwarded for the server to show in the in-game browser: https://docs.google.com/document/d/1JFxwBBBJoiBqQ9bgAFAGCLd_lpu8FXD6XxaKcMiZgg8/edit
Seems odd to require the rcon port, but there it is

That's a royal pain. I'm assuming there's no feasible way to make the port a regular port instead of the RemoteAdminPort so it's opened by AMP?

Huh? I think you are confusing the SFTP port

I thought RCON wasn't opened by default for some reason. I don't ever use it. Lol

No idea what's up then. Mine are auto-forwarded  and everything should be fine. But if you've already verified the connection works, I don't mind approving. Just note there may be a strange issue with this one.

Haven't verified anything with this one. Someone asked for it, I made the template, they have not given any feedback

It's free on Steam if you want to try it.

Ahhhhh hold the phone. I may have found the issue...

I know. It only runs on Windows tho

It looks like this is another case of "ARK" where they hide non-password protected servers by default and I have to tick the box to show them. Waiting to see if it shows up now though.

Of course, I had internet issues right after sending that. But it's not showing still. I'll leave it running until tomorrow to see if it's a timing issue.

I'm really not sure about this one. I've tried following some YouTube guides and matching up things, but they had false info in them as well (like put your public IP as the binding IP)... I just can't get it to show in game and therefore can't connect. I tried adding -NOSTEAM as one video suggested, no go. I tried adding a MapCycle.cfg and it reads it but no change. I'm listed as an admin, it has a password... Here's a pic of the in-game browser and filters in case I'm missing something dumb.

@IceOfWraith based on your testing, I don't think there is anything more needed on this one? As noted, logs are not useful for user joins and leaves


documentation error..

in line 2 of documentation you have written the wrong spelling of "re-imagining" you had written "re-imaginging"
assign it to me.

Thanks!


[AutoDiff] SILGen derivative function thunks must not be transparent.

SILGen generates thunks for derivative functions registered via
@differentiable and @differentiating attributes.
The thunks have a canonical, consistent naming (currently based on
the original function name and parameter indices) that TBDGen can also
generate via SILDeclRef::mangle.
These SILGen derivative function thunks must not be transparent; otherwise,
the functions will be inlined during MandatoryInlining and will not be
exposed publicly.
This bug was discovered as a DeadFunctionElimination crash:
function_ref instructions referencing transparent derivative thunks
had their referenced functions set to null by MandatoryInling and became
invalid.

More SILGen derivative function thunking documentation coming soon.

@swift-ci Please test tensorflow


Add a pure Mary mode

Note that this mode is not tested by consensus like the others, i.e., we only
test the Mary era as part of the Cardano blockchain, not on its own.

Draft because this depends on #2753.
All commits in this branch except for the last were created for other reasons. Only the last commit was needed to add a pure Mary mode. @deepfire as you can see, the changes in consensus are trivial. All the hard work will have to be done in the node by yourself or the node team.

bors merge

Thank you @mrBliss !


Abort with error message if OSD is requested with LSTM-only model

See related discussion for issue #4062.


Add Timed actor wrapper + friends for delayed/recurring messages

See module level docs and delay_actor.rs example for info.
When actor A wants to send a delayed message to actor B, there are 3 places where the queue can be:

in actor (thread) A
in some separate actor (thread) C
in actor (thread) B.

This PR choses option 3. In our downstream usage, all recurring messages "to self", so options 1. and 3. are functionally (but not API-wise) equivalent. The chosen approach has some pros and cons:

pro: modulo the preparatory "independent" core API changes, Timed<> wrapper and friends are strictly separate form the core API (could have been implemented in a separate crate).
pro: avoids having to pass some DelayActor reference around, or having it registered as some global resource.
pro: delayed messages do not cross thread boundary (in our use-case).
big con: requires wrapping each delayed-message-receiving actor in the wrapper.
con: requires ability to customize the Context type in actor a s a prerequisite.

Closes #14.
Separated into atomic commits, reviewing them individually is suggested. The last commit adds the main functionality, others are prerequisites. I had to tweak core actor API a bit (with changes needing to be done downstream) to be able to provide a nice API of the delayed/recurring functionality.
Demonstration of downstream code changes tonarino/portal#1126, specifically https://github.com/tonarino/portal/pull/1126/commits/4cfefdd45ba2ff7155c6e1943ba90d1bc20c6fe0.

Thanks for the reviews!
I'll let this simmer down for a couple of days (if @bschwind or @skywhale want to give this another read), then I'd like to merge and cut a 0.6.0 release


Delete S3 bucket data, once PR is closed

Once the PR is closed by triager, delete the precheck/generate data from the S3 bucket, to reclaim the resources.

@jjasghar @bjhargrave @russellb want to know your thoughts on it. Delete is a disruptive operation, so want to make sure that we pull the trigger on it when we think its safe. Deleting on pull request close probably is easiest, but we can also implement the mechanism, where we wait for certain duration after closure of the PR to make sure it was not accidentally closed (like we saw with migration of taxonomy to public etc).

We may want some delay because when a triager closes a PR, it may take some time for the submitter to review the closed PR and they may want to see the data to learn/protest closure/etc. So I think we may want a 2-4 week delay to allow time for such review by the submitter. If we delete the data immediately upon close, the submitter can never review the data to learn or protest.

The storage here is also pretty cheap for the amount of data we're talking about.  If we start doing ilab train for people, I would feel different.
I definitely agree a delay is necessary before cleanup. Since we want a delay anyway, a bulk cleanup process that runs occasionally would work. I just think it's pretty low priority for now.

I'm on the side of caution here, 4 weeks are probably a good number if at all. These are small text files, so I'd imagine it can't be much data at all, and in turn not much cost.

At this point of time, I am not considering the cost as a decision factor here. I am more like "this is what we need and whatever cost we need to pay, we should pay" thought 😄 . I think we can set any relaxed policy that's we need to (even 3 months after closure should be fine), I just want some mechanism to be in place to cleanup the garbage from s3 bucket because if the PR is never going to be re-opened we have some unused data sitting in the s3 bucket.


Updated orgchart to use office-ui-fabric-react and uifabric/styling

Q
A

Bug fix?
no

New feature?
yes

New sample?
no

What's in this Pull Request?
The orgchart webpart now uses office-ui-fabric-react and uifabric/styling instead of directly using css classes. This has less potential to break

thx Vardhaman, appreciated.


Several Things I dont Like..

Hello draw.io community. There are several things I encountered using draw.io I wish they were different.
1- I cant see the cursor, or what is selected by the other person.
2- I cant chat with the team as I do in google docs (using a chat box).
3- I doesnt have the google look and feel It would be good to respect it.
4- I dislike giving access to my whole google drive becoming vulnerable to insecurity.
Sorry it sounds harsh. Best regards.
Cheers.

As a note from someone who really doesn't like the Google look and feel, I feel inclined to point out that draw.io works on all sorts of platforms that aren't Google Drive as well.
Note that the permission you grant draw.io when you authorize it on your Google account is "View and manage Google Drive files and folders that you have opened or created with this app". It doesn't get access to everything you have in Drive.

In addition, in Google realtime, the selection of other users is highlighted and a chat is available via File, Chat.


Error in upgrade to PrestaShop <IP_ADDRESS> to <IP_ADDRESS>

Describe the bug
Error in the upgrade to PrestaShop <IP_ADDRESS> to <IP_ADDRESS>
Upgrade environment (please complete the following information):

Version you started from: <IP_ADDRESS>
Version you upgrade to: <IP_ADDRESS>
Way of upgrade: Web

Displayed log
Updating database schema...
06:27:07 ERROR [console] Error thrown while running command "prestashop:schema:update-without-foreign". Message: "Unknown database type bit requested, Doctrine\DBAL\Platforms\MySqlPlatform may not support it." ["exception" => Doctrine\DBAL\DBALException { …},"command" => "prestashop:schema:update-without-foreign","message" => "Unknown database type bit requested, Doctrine\DBAL\Platforms\MySqlPlatform may not support it."]
In AbstractPlatform.php line 423:
Unknown database type bit requested, Doctrine\DBAL\Platforms\MySqlPlatform
may not support it.
prestashop:schema:update-without-foreign [-h|--help] [-q|--quiet] [-v|vv|vvv|--verbose] [-V|--version] [--ansi] [--no-ansi] [-n|--no-interaction] [-e|--env ENV] [--no-debug] [--id_shop [ID_SHOP]] [--id_shop_group [ID_SHOP_GROUP]] [--]
Server (please complete the following information):

Web Server: Apache/2
PHP Version: 7.2.15
Module Version: 4.6.0

Hi there,
Thanks for your report.
This is not the first time we see this issue being reported. Please have a look at #87 to solve this specific issue.
I'm closing as we currently deal with a duplicate.
Regards


Add authentication using guestToken manually

Hi, great work on the library.
Currently for authentication, if apiKey is not provided guestToken is automatically created.
AuthCredential model from auth library has support for authentication with cookies and guestToken.
Please allow passing guestToken in Rettiwt  instance as config, together with apiKey (automatic guest token generation then is not necesary).
In some cases is useful to have ability to change guest tokens manually.

Any examples of such a use case?

Actually, currently, a new guest token is created for each request sent i.e, 1 unique guest token for 1 request. So it's already using different guest token for different requests, without being explicitly instructed to do so.
As you can see here, whenever a request is made without using any credentials to twitter account, the getGuestCredential() is called. This method generates a new guest token each time it is called, as can be seen here


Winson/feature/crowdin branching

更改 crowdin 流程，用 branching.
Download Crowdin translations 只会下载当前 branch 的翻译
Upload crowdin translations 会把当前分支当成 crowdin 分支，上传到自己分支里。

Also added a double prompt when someone is trying to upload translations from beta.


Optimized SpigotLogger for improved performance and reduced resource …

…usage while maintaining functionality and player message filtering.
Hopefully this PR (If you accept it) will help the plugin to use less resources. This PR in my opinion takes advantage of Bukkit's built in logger, static initialization and have a slightly faster speed and better intergretation.

Seeing this is a plugin maintained 6 years ago. It's likely you already abandoned it.
But it's still a cool plugin neverless.


Define jobcentre registers in alpha

registry files for checking.

@Jamie-Atkinson @nacnudus  Changes have been made. Please review.


My first attempt at updating

It hypothetically works however I can't test it

Good. Please, switch target branch to 1.18

K done


Swapping positions with keepers lead to awkward situation

I saw sometimes a GK swapping with an outfield or opponent GK and it end up swapping them on the location they were based. Keeper is suddenly so far out of his goal.

Fixed.


Missing certificate file breaks Nginx upon service start. Certificate missing due to limitations of subdomain-cert generation

The root cause of the issue is that for a subdomain the certificate was not generated correctly. After certificate generation for a subdomain, the SSL cert section points to the main domain's cert, but obviously in the subdomain's configuraton in the Plesk database a different, non-existent cert name was saved. This constellation randomly broke Nginx restarts. The issue cannot be fixed from the control panel, because it is impossible to remove the information that cert is present (should be present) from the database. The plugin should have a "clean" option, so that erroneous configurations can be removed. Currently, a certificate cannot be removed, but only unselected. When it is unselected, there is no way of replacing it with a new certificate until it expires.
Example messages:

"Unable to generate the web server configuration file on the host <elbe.bitpalast.net> because of the following errors:
Template_Exception: AH00526: Syntax error on line 49 of /etc/httpd/conf/plesk.conf.d/vhosts/blog.[domainname].net.conf:
SSLCertificateFile: file '/usr/local/psa/var/certificates/cert-Ym3wpE' does not exist or is empty
file: /usr/local/psa/admin/plib/Template/Writer/Webserver/Abstract.php
line: 75
code: 0
Please resolve the errors in web server configuration templates and generate the file again."

and

"Unable to generate the web server configuration file on the host <elbe.bitpalast.net> because of the following errors:
Template_Exception: nginx: [emerg] BIO_new_file("/usr/local/psa/var/certificates/cert-Ym3wpE") failed (SSL: error:02001002:system library:fopen:No such file or directory:fopen('/usr/local/psa/var/certificates/cert-Ym3wpE','r') error:2006D080:BIO routines:BIO_new_file:no such file)
nginx: configuration file /etc/nginx/nginx.conf test failed
file: /usr/local/psa/admin/plib/Template/Writer/Webserver/Abstract.php
line: 75
code: 0
Please resolve the errors in web server configuration templates and generate the file again."

I have the same issue with Apache. Even switching off SSL did not help, because the SSL config did not disappear from the subdomains apache config. Regenerating the web config did not help either. Someow the cert file does not get generated.

It seems this breaks everytime the certs would be renewed. The old cert get deleted but the new does not get  generated

That's really a pain. Every time certificates are renewed, I need to fix everything by checking which path is expected and renaming the files. Hope this will be fixed soon.

Hi guys,
I share the same problem. Anyone taking care for this problem or has an idea how to deal with the problem? My nginx configs of sub+domain point to not existing files and are not longer working.
Nevertheless I have a couple of certs in /usr/local/psa/var/certificates/ but I do not know which one to point to in the nginx config files.
@benohead , you seem to have fixed that somehow manually - could you give us a hint how you assign the correct certs (file names) in the nginx configs?
thx,
Jens

Propose to contact support https://support.plesk.com/hc/en-us/requests/new
It's hard to understand what's wrong without ability to reproduce the issue.

We had recently seen the initial issue "but obviously in the subdomain's configuraton in the Plesk database a different, non-existent cert name was saved." in the hundreds upon a single renewal script execution. The issue that needs to be solved is that the certificate names are not always in sync with the cert file names in the web server conf files. The renewal script must make sure that when it finishes, the web server conf file references to the cert files match the cert file names and that the cert files mentioned in the web server configuration are actually present on the system.
It is not happening always, and we have not yet figured out under which circumstances it is happening, but is had happened on one of our hosts again recently. I'd like to refer you to https://talk.plesk.com/threads/pppm-5665-lets-encrypt-extension-causing-severe-failure-due-to-certificate-filename-changes.341373/ for more information. This was not reproduceable, but the same situation has happened several times by now.
We have now moved to manually running the updater script because we never know what the outcome will be. Manual execution enables us to respond quickly if the web server fails during or after the cert renewal script was executed. Sometimes the cert file names are not in sync, sometimes httpd service failed and did not restart, sometimes everything works fine.

Here is a simple script courtesy of Bitpalast that you can use to bring all cert file names in sync with the file names listed in web server configuration files of Plesk installations.
cert_emergency_response.zip
Run with parameter --test and --verbose to see what it will do before you actually run it without --test.
Very important: This is an emergency response script only! It will enable you to immediately "repair" filenames of the cert files according to what the web server configuration is expecting. No need to manually sync them. It will make sure that the web server won't fail on reload/restart for missing cert files. However, it does not check whether the web server configuration files match the data that is stored in the Plesk psa database. Always run "httpdmng --reconfigure-all" to make sure that the web server configuration files reflect the current database setting.

First of all, do you use Plesk 12.5?
In Plesk Onyx we extended certificates API with direct update operation instead of delete + create. This should have made a renewal more robust.
Does any one have the problem with Onyx?

The cert name mismatch issue has been observed on Plesk 12.5.30 systems and not yet been seen on Plesk Onyx by us. However, we have more 12.5.30 installations than Onyx. Time will well if an upgrade to Onyx will solve the case.


Correctly document rustup defaults on Windows

rustup checks the host OS for the CPU arch

Please feel free to edit the pr to suit. I changed this to change the
target abi rather than arch because I suspect it's more common, and still
demonstrates setting one of the for options listed immediately below.
On Thu, 11 Apr 2019, 22:14 Daniel Silverstone<EMAIL_ADDRESS>wrote:

@kinnison commented on this pull request.
In README.md
https://github.com/rust-lang/rustup.rs/pull/1753#discussion_r274353893:

-$ rustup set default-host x86_64-pc-windows-msvc
+$ rustup set default-host x86_64-pc-windows-gnu

I *think* this bit was meant to be demonstrating the change in i686 vs.
x86_64 rather than msvc vs gnu which I think is addressed elsewhere in
the README.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<https://github.com/rust-lang/rustup.rs/pull/1753#pullrequestreview-225446368>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAef3pVEII7_KTnWrX82IesgTm2wsnlZks5vfwsegaJpZM4cpPBk>
.

Thanks both!


Notes on use with TypeScript

I used ember-browserify to add pkijs to a TypeScript project today. Here are some notes:

add the library: yarn add pkijs --dev
add the type declarations: yarn add @types/pkijs --dev
add "npm:pkijs": [ "node_modules/pkijs" ], to compilerOptions.paths in tsconfig.json. This tells Typescript that when I import PKI from 'npm:pkijs', I'm getting node_modules/pkijs/src/index.js
import PKI from 'npm:pkijs' in a component/route/model/...

I don't know where this information might be useful -- possibly in the README.

At a minimum it’s helpful to me as the maintainer of ember-cli-typescript – thanks for writing it up! I’ll add the note about using the paths option this way to that README later today!


Conditional Action in Alerting monitors

Is your feature request related to a problem? Please describe.
Opensearch currently allows configuring upto 10 Actions against a monitor. I have scenarios in my application where I want to perform an action based on certain condition. e.g. If num>10, send email alert else send a Teams notification.
Describe the solution you'd like
I want Opensearch Monitor UI to allow further conditioning on actions after the Trigger condition has resulted true.
Describe alternatives you've considered
Currently, I have to replicate my monitor with a different name and the only difference from first one is the final Action which could have been avoided if actions were further conditional.
Additional context
The feature was recently requested on Kibana forum as well : https://github.com/elastic/kibana/issues/158014
It would be great if we can gather some additional info or mockups from there as well.

@joshuarrrr I have moved this issue to alerting dashboards repo.


Windows Install

I was wondering if anyone could lead me to any instructions on how to get this running on a Windows 10 machine?
Thanks so much.

Hey @ferguson4848 , which step are you currently at? It was creating the credentials file in the aws directory, right?

Hopefully that gives you something to work with for now! This is helpful for me to go back and trace my steps on how I got mine working. I'm currently in the process of taking this (and other information) in an attempt to set a friend up, too. :)

Thank you for the info.
I was able to make the aws folder and install ASW CLI and seems to Test fine.
Is the AWS ID and Key from IAM Roles, DynamoDB or somewhere else entirely?

Also, I was able to install git and download the files, but what directory/program do I run the command npm install?

Just inside the 'interactive-adventure-game-tool' dir.

That is what I figured, but when I go to the directory I get this first:
GitHub\interactive-adventure-game-tool [master ≡]>
Then I try npm install, which results in:

npm : The term 'npm' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the
spelling of the name, or if a path was included, verify that the path is correct and try again.
At line:1 char:1

npm install

  + CategoryInfo          : ObjectNotFound: (npm:String) [], CommandNotFoundException
  + FullyQualifiedErrorId : CommandNotFoundException

At first glance, it doesn't look like you have npm installed, which comes with Node.

Okay, I installed Node 6.9.4 LTS and appears to have installed fine. I can open node.js and get a command prompt with a ">", but the npm command still gets a not recognized error.
Do I need to enable npm or something other than just the standard node install?

Bit of an update, I ran the npm install again and alot of stuff happened. After a very long string of commands I got ok   Install complete<EMAIL_ADDRESS>C:\Users\ and then a series of +-- items. Finally that all ended and I was back at the prompt:
...\Documents\GitHub\interactive-adventure-game-tool [master ≡ +1 ~0 -0 !]>
I tried the npm launch command, but got this result:
sage: npm
where  is one of:
access, adduser, bin, bugs, c, cache, completion, config,
ddp, dedupe, deprecate, dist-tag, docs, edit, explore, get,
help, help-search, i, init, install, install-test, it, link,
list, ln, login, logout, ls, outdated, owner, pack, ping,
prefix, prune, publish, rb, rebuild, repo, restart, root,
run, run-script, s, se, search, set, shrinkwrap, star,
stars, start, stop, t, tag, team, test, tst, un, uninstall,
unpublish, unstar, up, update, v, version, view, whoami
npm  -h     quick help on
npm -l           display full usage info
npm help   search for help on
npm help npm     involved overview
Specify configs in the ini-formatted file:
C:\Users\mferguson.npmrc
or on the command line via: npm  --key value
Config info can be viewed via: npm help config
<EMAIL_ADDRESS>C:\Program Files\nodejs\node_modules\npm

What commend did you use for "npm launch"? It should be "npm start". Try that? @ferguson4848 Then you should see the following below as the game launches in a web browser.

That did it! It opened in a web browser. Thank you so much.
I really appreciate you taking the time to help me through this.

‘’“16 verbose cwd C:\Users\wuxb3.aws\interactive-adventure-game-tool
17 error Windows_NT 10.0.14393
18 error argv "C:\Program Files\nodejs\node.exe" "c:\program files\nodejs\node_modules\npm\bin\npm-cli.js" "start"
19 error node v6.10.0
20 error npm  v3.10.10
21 error code ELIFECYCLE
22 error<EMAIL_ADDRESS>start: node node_modules/gulp/bin/gulp.js
22 error Exit status 1
23 error Failed at the<EMAIL_ADDRESS>start script 'node node_modules/gulp/bin/gulp.js'.
23 error Make sure you have the latest version of node.js and npm installed.
23 error If you do, this is most likely a problem with the interactive-adventure-game-tool package,
23 error not with npm itself.
23 error Tell the author that this fails on your system:
23 error     node node_modules/gulp/bin/gulp.js
23 error You can get information on how to open an issue for this project with:
23 error     npm bugs interactive-adventure-game-tool
23 error Or if that isn't available, you can get their info via:
23 error     npm owner ls interactive-adventure-game-tool
23 error There is likely additional logging output above.
24 verbose exit [ 1, true ]
”
help@AlBakerKY

What should I do, where I went wrong @AlBakerKY

@ginareams I have gotten everything to start in windows, but the save and upload functions do not work. I have the credentials set up in the correct location. Did you experience any issues with it?

@natemunk I did run into issues uploading, but not saving. Before I get to uploading, when you save, are you seeing errors in the command line when you're saving? If so, it's quite possibly an issue with your credentials. Does this look familiar? https://github.com/alexa/interactive-adventure-game-tool/issues/37
This page can help you get those set up: http://docs.aws.amazon.com/cli/latest/userguide/cli-config-files.html.

I am not getting anything when I try and save or publish. Just is not doing
anything. I am considering creating a VM with Ubuntu and trying to run from
there, been pulling my hair out all night trying to do it with windows.
On Mon, Jul 17, 2017 at 10:14 PM, Gina<EMAIL_ADDRESS>wrote:

@natemunk https://github.com/natemunk I did run into issues uploading,
but not saving. Before I get to uploading, when you save, are you seeing
errors in the command line when you're saving? If so, it's quite possibly
an issue with your credentials. Does this look familiar? #37
https://github.com/alexa/interactive-adventure-game-tool/issues/37
This page can help you get those set up: http://docs.aws.amazon.com/
cli/latest/userguide/cli-config-files.html.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/alexa/interactive-adventure-game-tool/issues/24#issuecomment-315953343,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOW0sBp-m0_g44K4QvTRPYAvB3HdQra-ks5sPDEigaJpZM4LrRb6
.

--
NATE MUNK
540.359.0732
Nate Nate<EMAIL_ADDRESS><EMAIL_ADDRESS>

@natemunk Hmm. Ok. Give me a few minutes. I definitely did run into a similar issue, I solved it with some sort of manual upload process at the time, but let me see if I can find how I did that. -- I meant to eventually get around to a long term fix for uploading through the button, but haven't been in the tool for a while.

I really appreciate it, I am fine with doing a manual upload if I can get
the files to save and not publish.
On Mon, Jul 17, 2017 at 10:21 PM, Gina<EMAIL_ADDRESS>wrote:

@natemunk https://github.com/natemunk Hmm. Ok. Give me a few minutes. I
definitely did run into a similar issue, I solved it with some sort of
manual upload process at the time, but let me see if I can find how I did
that. -- I meant to eventually get around to a long term fix for uploading
through the button, but haven't been in the tool for a while.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/alexa/interactive-adventure-game-tool/issues/24#issuecomment-315954122,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOW0sB5K8cbADxzvr7l6q0gWHDtK05Bcks5sPDLNgaJpZM4LrRb6
.

--
NATE MUNK
540.359.0732
Nate Nate<EMAIL_ADDRESS><EMAIL_ADDRESS>

@natemunk So it's been a long time and I've apparently messed with something that's keeping me from running the tool on my machine. So unfortunately I can't write you a detailed walkthrough, but I found my solution from this thread: https://github.com/alexa/interactive-adventure-game-tool/issues/3
It's likely you are actually saving changes, but it's not actually being zipped and uploaded. I manually zipped the files in the skills folder and uploaded it to lambda. There are several comments discussing that process in that thread above. I really hope that helps!

@natemunk It's a bit late for me and I'm heading to bed, but if you don't end up solving it I'll see if I can get mine running again and figure out those specifics.

Thank you Gina! I will go through it and let you know if I can get it
working properly!
On Mon, Jul 17, 2017 at 10:34 PM, Gina<EMAIL_ADDRESS>wrote:

@natemunk https://github.com/natemunk It's a bit late for me and I'm
heading to bed, but if you don't end up solving it I'll see if I can get
mine running again and figure out those specifics.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/alexa/interactive-adventure-game-tool/issues/24#issuecomment-315955584,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOW0sGO62_rGiTqefjlBF0LJE-0UpgRsks5sPDXOgaJpZM4LrRb6
.

--
NATE MUNK
540.359.0732
Nate Nate<EMAIL_ADDRESS><EMAIL_ADDRESS>

Looking through my files it appears as though nothing is saving. The
utterances and intent files are both still at the original values. I think
I am going to get to bed now too and take another crack in the morning.
On Mon, Jul 17, 2017 at 10:39 PM, Nate Munk<EMAIL_ADDRESS>wrote:

Thank you Gina! I will go through it and let you know if I can get it
working properly!
On Mon, Jul 17, 2017 at 10:34 PM, Gina<EMAIL_ADDRESS>wrote:

@natemunk https://github.com/natemunk It's a bit late for me and I'm
heading to bed, but if you don't end up solving it I'll see if I can get
mine running again and figure out those specifics.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/alexa/interactive-adventure-game-tool/issues/24#issuecomment-315955584,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOW0sGO62_rGiTqefjlBF0LJE-0UpgRsks5sPDXOgaJpZM4LrRb6
.

--
NATE MUNK
540.359.0732 <(540)%20359-0732>
Nate Nate<EMAIL_ADDRESS><EMAIL_ADDRESS>
--
NATE MUNK
540.359.0732
Nate Nate<EMAIL_ADDRESS><EMAIL_ADDRESS>

Hi, sorry to bring up an old problem, but I got the credentials working correctly, when I run npm install it goes through but I get this error:
**C:\Users\hansa\interactive-adventure-game-tool> npm install

<EMAIL_ADDRESS>install C:\Users\hansa\interactive-adventure-game-tool\node_modules\node-sass
node scripts/install.js

Downloading binary from https://github.com/sass/node-sass/releases/download/v3.13.1/win32-x64-57_binding.node
Cannot download "https://github.com/sass/node-sass/releases/download/v3.13.1/win32-x64-57_binding.node":
HTTP error 404 Not Found
Hint: If github.com is not accessible in your location
try setting a proxy via HTTP_PROXY, e.g.
  export HTTP_PROXY=http://example.com:1234

or configure npm proxy via
  npm config set proxy http://example.com:8080

<EMAIL_ADDRESS>postinstall C:\Users\hansa\interactive-adventure-game-tool\node_modules\node-sass
node scripts/build.js**
can anyone help me with this because I am unable to do launch it

@natemunk Did you ever end up figuring it out?

Yeah I ended up using an old Mac instead haha. You can check out my skill
though: Escape Jurassic Island.
It is a goofy game, but fun :)
On Wed, Aug 9, 2017 at 11:20 PM, Gina<EMAIL_ADDRESS>wrote:

@natemunk https://github.com/natemunk Did you ever end up figuring it
out?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/alexa/interactive-adventure-game-tool/issues/24#issuecomment-321453477,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOW0sPM7E53WhnHB_FRTznClJdOKnrukks5sWpMMgaJpZM4LrRb6
.

--
NATE MUNK
540.359.0732
Nate Nate<EMAIL_ADDRESS><EMAIL_ADDRESS>

@natemunk Sorry the windows install didn't end up working out for you! I'm not even sure how my install just stopped working... Haven't gone back to revisit it, but now I have a Mac as well. :) So I may never try to get it working on Windows again.
Will check it out! Glad you got it working in some form at the end. :)

Yeah, I appreciate your help! I may try and get it working on Windows down
the road, my PC is a much better machine. If  I get it working I will post
what I did on here so it will hopefully help others out.
On Mon, Aug 14, 2017 at 8:30 PM, gina-n3twork<EMAIL_ADDRESS>wrote:

@natemunk https://github.com/natemunk Sorry the windows install didn't
end up working out for you! I'm not even sure how my install just stopped
working... Haven't gone back to revisit it, but now I have a Mac as well.
:) So I may never try to get it working on Windows again.
Will check it out! Glad you got it working in some form at the end. :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/alexa/interactive-adventure-game-tool/issues/24#issuecomment-322360891,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOW0sHsEkz9YuNxqK7ZBLd9tCf3D10mPks5sYQLbgaJpZM4LrRb6
.

--
NATE MUNK
540.359.0732
Nate Nate<EMAIL_ADDRESS><EMAIL_ADDRESS>


Navigator puts enterView atop.

When Navigator play a transition it always add destination view as a child of NavigatorView. So it means that enterView will be always on top.
However, this imposes some restrictions on what kind of animations can be achieved with ViewTransition, so if you want to play animate backward transition from B to A you probably out of luck.
I see a quick solution here: NavigatorView should place enterView below exitView on transition.
However, I believe that it is responsibility of ViewTransition to add views whenever and however it needs.

I think I've run into this problem as well. I'm trying to animate a new view sliding in over the top of the old view on a Forward transition, and sliding back out on a Backward transition. The forward one is working great, but the backward transition doesn't seem to work at all, presumably because the entry view, which this time should be below the exit view, is instead appearing above it and thus hiding the transition effects.

@grundleborg Hi, consider to put this code before your actual animation:
if (direction == ViewTransitionDirection.FORWARD) {
  enterView.bringToFront()
} else {
  exitView.bringToFront()
}

This will bring your view to front, without removing and adding it again (thus, not triggering your view's onAttachToWindow/onDetachFromWindow callbacks)

Thanks @pepyakin - that works great.


Support Node.js v19

It seems Node.js v19 is not yet supported?
yarn install v1.22.19
[1/4] Resolving packages...
[2/4] Fetching packages...
error<EMAIL_ADDRESS>The engine "node" is incompatible with this module. Expected version "8 || 9 || 10 || 11 || 12 || 13 || 14 || 15 || 16 || 17 || 18". Got "19.0.0"
error Found incompatible module.

Thanks for raising the issue! Support has been added to the necessary packages:

<EMAIL_ADDRESS>
<EMAIL_ADDRESS>https://github.com/achrinza/event-pubsub/releases/tag/v5.0.8

<EMAIL_ADDRESS>
<EMAIL_ADDRESS>https://github.com/achrinza/node-ipc/releases/tag/v10.1.7

<EMAIL_ADDRESS>
<EMAIL_ADDRESS>https://github.com/achrinza/node-ipc/releases/tag/v9.2.6

<EMAIL_ADDRESS>
<EMAIL_ADDRESS>https://github.com/achrinza/strong-type/releases/tag/v1.1.4

<EMAIL_ADDRESS>
<EMAIL_ADDRESS>https://github.com/achrinza/strong-type/releases/tag/v0.1.10

Thx a lot :muscle:


Car Wash Props

I was having a car wash prop collision problem, so I stopped the resource and the props are from Bob74_ipl, where are they in the resource so I can try to remove them?

https://github.com/Bob74/bob74_ipl/blob/master/gtav/base.lua#L57-L58


Testmatrix #trivial

Supercedes #767
Adds make test-smoke-ls which prints out smoke test dimensions and top-level test names in copy/pastable format for GO_TEST_RUN= env var, used in combo with make test-smoke to run subset of tests.
Internally makes it easier to add more dimensions by generating all combinations automatically, and handles stable test naming, order of dimension names for GO_TEST_RUN purposes, and naming/adding additional values to those dimensions.

LGTM!
No errors, no warnings. :shipit:

  Generated by :no_entry_sign: Danger

Same happening here, TC smoke tests not working, wonder if latest changes broke the TC smoke test run output?


ROX-19234: Add conditional RBAC for Vulnerabilities Reports 2.0

Description
Set precedent to release even XL features with conditional rendering according to role-based access control.
Make only changes that are testable so not to invalidate testing of the feature itself.
What cause me to change original decision to wait until 4.3 release?

Obsolete Access requirement to clone, create, edit report.
Lack of conditional rendering for reports table row actions and report page Actions button.

Adapt from Reports 1.0 in #7433
Analysis

VulnReportingPage

canReadWriteReports includes obsolete Access resource for access scopes superseded by collections
Integration is required only for select options to write, because notifier integration name is included in report configuration response.

VulnReportsPage

Ditto the above
Table does not have conditional rendering for row actions.

ViewVulnReportsPage

Page does not have conditional rendering for Action button.

CollectionSelection:

ReportParametersForm uses permissions information only for allowCreate prop.
Collections modal is like a link to another container, therefore needs isRouteEnabled function call. In this case, collections depend not only on implicit WorkflowAdministation same as reports, but also on READ_ACCESS for Deployment to display collection results in Review step of wizard.

TechPreviewBanner unconditionally renders link to another container:

From Reports 2.0 to Reports 1.0 both routes have the same resource requirements.
From Workload CVEs to Vulnerability Management 1.0 the latter has more resource requirements.

Residue

Factor out hasWriteAccessForReport criterion.
Evaluate pro and con to add for conditional rendering of Create email notifier button an isRouteEnabled('integrations') function call, which is redundant with READ_ACCESS for Integration resource that is implied by hasWriteAccessForReport value.

Checklist

[x] Investigated and inspected CI test results
[x] ~Unit test and regression tests added~

Testing Performed

yarn lint in ui
yarn build in ui and then compute branch - master for the following

wc build/static/js/*.js
main.css: 0 = 3761815 - 3761815
total: 336 = 9969784 - 9969448
ls -al build/static/js/*.js | wc -l
0 = 80 - 80 files

yarn start in ui

Manual testing

Visit /main/vulnerabilities/reports with only READ_ACCESS for VulnerabilityAdministration resource.

See preview banner with link but no such link in sidebar navigation.

See preview banner without link because route is not enabled.

See absence of row actions.

Click a link to open a report page and see absence of Actions button.

Visit /main/vulnerabilities/reports with READ_WRITE_ACCESS for VulnerabilityAdministration also READ_ACCESS for Image and Integration resources.

See Create report button and row actions. No picture of original behavior.

Click Edit report in row actions to see report wizard.

See absence of Create collection button without READ_ACCESS for Deployment therefore route is not enabled.

See presence of Create collection button with READ_ACCESS for Deployment therefore route is enabled.

See absence of Create email notifier button with READ_ACCESS for Integration.

See presence of Create email notifier button with READ_WRITE_ACCESS for Integration.

/test ocp-4-10-ui-e2e-tests


testlapack: replace isOrthogonal with residualOrthogonal and remove unused helpers

This one's a bit bigger but the split into commits is quite reasonable and hopefully helps.
Please take a look.

PTAL


onClick not working for custom components

Is this a bug report?
Yes
Have you read the Contributing Guidelines on issues?
Yes
Environment
(OS, Node, npm, yarn)

OS: OSX
Node: 9.11.1
npm: 5.6.0
yarn: -

Steps to Reproduce

Clone my demo repository
Run npm install in the website/ directory
Run npm start in the website/ directory
Visit http://localhost:3000
You should see a link named "I was not clicked"

Expected Behavior

I would expect the onClick callback to be executed when the text is clicked. As a result the text should change to "I was clicked" and the console output should show up.
Actual Behavior

If you click the text there is no console output shown and the text does not change into "I was clicked".
Reproducible Demo
https://github.com/DanielMSchmidt/docusaurus-onclick

With Docusaurus your components get converted into static HTML at build-time, they are never mounted in the browser.

Oh, where was the piece of documentation I missed here 🙈

@DanielMSchmidt, if you would like to send a PR to add some documentation around this, that would be great. Maybe a sentence or two at the beginning of https://docusaurus.io/docs/en/installation.html?

@JoelMarcey I would have thought https://docusaurus.io/docs/en/site-creation.html might be a better place, would that be ok for you?

@DanielMSchmidt Sure. That could be a decent place too. Let's see what content will go there and then we can make the final decision. Thanks!


Suggestion: Needed a link while reading

For users reading this for the first time, would be great to have more context on what the 'Extensions and Updates' dialog is - adding a bit more context at the beginning of the article.

@ClydeDz : Thanks for your contribution! The author, @gregvanl, has been notified to review your proposed change.

@ClydeDz : Thanks for your contribution! The author, @gregvanl, has been notified to review your proposed change.

#sign-off


SCB-224 retry sub-transaction on failure

Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[x] Each commit in the pull request should have a meaningful subject line and body.
[x] Format the pull request title like [SCB-XXX] Fixes bug in ApproximateQuantiles, where you replace SCB-XXX with the appropriate JIRA issue.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Run mvn clean install to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

This PR is based on #117

@eric-lee-ltk  There are some build error in you commit, could you take a look at it?

Coverage increased (+0.3%) to 95.568% when pulling c26feb1d43adb073a5b02385ea14693b37056692 on eric-lee-ltk:SCB-224 into 44763ae7d4930d68d12212d3164b4f50036bc076 on apache:master.

Merged the patch into master branch.


Added GKE release channel support in cluster creation in cluster manager lib

Updated k8s to pick up the latest beta API for release channel.
$ dep ensure -update google.golang.org/api
$ ./hack/update-deps.sh

Testing
Test command:
go run testutils/clustermanager/prow-cluster-operation/main.go --create --release-channel=rapid --name=rapid-test --project=joyceyu-test

Output:
joyceyu@joyceyu-macbookpro:pkg(k8s: "")$ go run testutils/clustermanager/prow-cluster-operation/main.go --create --release-channel=rapid --name=rchan-test --project=joyceyu-test
2019/12/30 13:27:55 Identified project joyceyu-test for cluster creation
2019/12/30 13:27:55 Creating cluster "rchan-test" in region "us-central1" zone "" with:
&{Request:{Project:joyceyu-test GKEVersion: ReleaseChannel:rapid ClusterName:rchan-test MinNodes:1 MaxNodes:3 NodeType:n1-standard-4 Region:us-central1 Zone: Addons:[] EnableWorkloadIdentity:false} BackupRegions:[us-west1 us-east1] SkipCreation:false NeedsCleanup:false ResourceType:gke-project}
2019/12/30 13:31:03 Cluster creation completed
2019/12/30 13:31:03 Getting artifacts dir from prow
2019/12/30 13:31:03 Env variable ARTIFACTS not set. Using artifacts instead.
2019/12/30 13:31:03 Writing metadata to: "artifacts/metadata.json"
2019/12/30 13:31:03 Overriding meta "E2E:Zone":"" with new value ""
2019/12/30 13:31:03 Overriding meta "E2E:Machine":"rapid-test" with new value "rchan-test"
2019/12/30 13:31:03 Overriding meta "E2E:Version":"1.16.0-gke.20" with new value "1.16.0-gke.20"
2019/12/30 13:31:03 Overriding meta "E2E:MinNodes":"1" with new value "1"
2019/12/30 13:31:03 Overriding meta "E2E:MaxNodes":"3" with new value "3"
2019/12/30 13:31:03 Overriding meta "E2E:Project":"joyceyu-test" with new value "joyceyu-test"
2019/12/30 13:31:03 Overriding meta "E2E:Region":"us-central1" with new value "us-central1"
2019/12/30 13:31:03 Done writing metadata
joyceyu@joyceyu-macbookpro:pkg(k8s: rchan-test)$ cat artifacts/metadata.json
{"E2E:Machine":"rchan-test","E2E:MaxNodes":"3","E2E:MinNodes":"1","E2E:Project":"joyceyu-test","E2E:Region":"us-central1","E2E:Version":"1.16.0-gke.20","E2E:Zone":""}j

/cc @vaikas
Could you please take a look if you have a moment?
This change updated the google api to pick up release channel API.

/approve
For Gopkg.locl


Need a button to reload table

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

This issue does not meet our guidelines and thus is closed immediately.


Add option to allow track judging to use a different switching method

Currently, track judging does NOT allow for a separate switching method (auto vs manual). I would like this to be an option. Right now, the Options object has a field called "MainGroup" that is of type "GroupSwitchOps". This should easily abstract the functionality to this struct, and you can just make a copy of this main group attribute to something like "TrackGroup". The annoying part is adding a bunch of logic everywhere to check for the two and also add it all to the settings page.

Due to changes in the track system and an overall goal of reducing complexity, I have decided NOT to implement this feature.


SNOW-988515: Please have a command to simply return a repository URL

Description
Hi.
A common operation could be to create (or fetch) an image repo, take the URL to build and tag an image, and push it to the URL.
So getting the URL easily could make the process clearer. This is the process today:
repo_url=$(snow sql -c <connection name> --schema <repo schema> -q "SHOW IMAGE REPOSITORIES;" --format=JSON | jq -r ".[] | .repository_url")

snow registry token -c <connection name> --format JSON| docker login $repo_url -u sessiontoken0 --password-stdin

docker build --platform linux/amd64 -t $repo_url/<image>:<tag> .

docker push $repo_url/<image>:<tag>

Not even sure how this will work if I have a few repos in the same schema, maybe include a repo name parameter?
Thanks!
Context
No response

Done in #708


Library name change request

Library repository URL
https://github.com/Sensirion/arduino-upt-core

Hi @qfisch. I am confirming your request.
Unlike submissions, this type of request is still handled manually. The Library Manager index maintainer occasionally comes through to clear all the outstanding requests. They will close this issue when it is done. You will get a notification from GitHub at that time.
After that, you can monitor the indexing of the library via its dedicated logs web page.
Information about that here:
https://github.com/arduino/library-registry/blob/main/FAQ.md#can-i-check-on-library-releases-being-added-to-library-manager

Done:
2024/02/15 13:45:16 Loaded 6648 libraries from DB
Removing Sensirion_UPT_Core
Success!


Add workflow checks to recipe test workflow

Part of #2786.
Adds workflow checks to the recipe tests workflow

Merged.


Mobile landscape view Emoji Picker view gets cut and iPhone has scroll issues

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

With respect to the comment https://github.com/Expensify/App/pull/4456#issuecomment-895926828, adding this issue.
Action Performed:
For iOS and Android both:

Open chat in a mobile device landscape mode
Open the Emoji Picker

For iOS Scroll Issue:

Open the chat in iPhone Safari with two tabs (any extra tab should be fine)
Keep it in portrait mode
Open the Emoji Picker
Switch to landscape mode
You'll see that you can't scroll the Emojis and the header is cut at the top

Expected Result:

Emoji Picker view shouldn't get cut.
It should allow scrolling switching between portrait and landscape mode.

Actual Result:
Android:
Emoji Picker view gets cut. Scrolling enough refreshes the page

iOS:

Emoji Picker view gets cut with multiple tabs. Switching between portrait and landscape causes the scroll to stop working for the initial minute or two.
Works fine if we don't have multiple tabs open.

https://user-images.githubusercontent.com/3069065/129239280-2b771ed3-8429-4e91-940e-f3459b739bd6.mp4
NOTE: The still time where there are no actions is me trying to scroll in the picker.
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platform:

Where is this issue occurring?

Web
iOS
Android
Desktop App
Mobile Web

Version Number:
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:
View all open jobs on Upwork

Triggered auto assignment to @sakluger (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

Alright, I had a real hard time reproducing this one in mobile Safari on my iPhone, but I think I figured out a way to reproduce consistently. To reproduce the scrolling and header cutoff issues, it helped me to start off with one mobile safari tab, then to add a second tab and go back to the new.expensify.com tab. I was also able to reproduce scrolling issues when I started with multiple tabs, then closed all tabs but the Expensify one.
I'm going to assign one more triager to make sure that others can reproduce it.

Triggered auto assignment to @mallenexpensify (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

@sakluger  Sure. You can take your time to reproduce it properly. I am anyway working on another task before I can pick this.
As I mentioned it works fine when you have one tab. But hangs up when you've got multiple tab in Safari.

when you've got multiple tabs in Safari.
meaning... multiple tabs that are both open to new.expensify.com?
I tested in safari and chrome on iOS.  There was one time I might have had trouble scrolling but  'it fixed itself'.   Wasn't able to reproduce consistently on either but... I didn't do the two-tab thing.

@mallenexpensify it "fixed itself" for me too, and I still had trouble figuring out the exact set of steps to reproduce it consistently, but when I started opening new tabs and closing tabs (those tabs could be anything, not necessarily new.expensify.com), the issues started happening more often.

@mananjadhav @sakluger , let's put this on hold for new.  If it keeps happening and/or if we reproducible steps that are.. easier to reproduce, we can take off hold.

@mallenexpensify Not able to reproduce this in my iPhone 11 now. We've reduced the height of the Emojpicker in landscape mode with https://github.com/Expensify/App/issues/4541.
If nobody else is able to reproduce then we're good to close this?

This seems a duplicate issue and It was already worked in $4541. @mananjadhav you should be good to close this as nobody is assigned and you are the owner.

Gonna close cuz of #4541, if needed, we can reopen.


IE11 does not support const in for..in

Unfortunately, IE 11 does not support const (or let) in for..in loops (see
the kangax compat table).  With buble, const is not transformed to var for --target ie:11.  So this kind of code breaks:
$ echo 'for (const foo in bar) {}' > input.js
$ npx buble input.js --target ie:11
for (const foo in bar) {}

The most conservative change would be to flip the letConst bit for IE 11.  For example:
diff --git a/src/support.js b/src/support.js
index 2183032..7076329 100644
--- a/src/support.js
+++ b/src/support.js
@@ -46,7 +46,7 @@ export const matrix = {
                     8: 0b00000000000000000000,
                     9: 0b01000000000000000000,
                    10: 0b01000000000000000000,
-                   11: 0b01000000000100000000
+                   11: 0b01000000000000000000 // no let/const in for loops
        },
        edge: {
                    12: 0b01001010100101001101,

I'd be happy to propose a PR if this or another solution would be accepted.

Yes, please prepare a patch for this!

@adrianheine see #154 for the PR.


iOS 9 support location update in suspend

Hi!
You are plan add new param for iOS 9 allowsBackgroundLocationUpdates ?
https://developer.apple.com/library/ios/documentation/CoreLocation/Reference/CLLocationManager_Class/index.html#//apple_ref/occ/instp/CLLocationManager/allowsBackgroundLocationUpdates

+1


MGDOBR-1004 : remove minikube profile references

https://issues.redhat.com/browse/MGDOBR-1004
Changes in below files are left from previous PR(https://github.com/5733d9e2be6485d52ffa08870cabdee0/sandbox/pull/1072).
These changes are required to get successful PR check
PR owner checklist
Please make sure that your PR meets the following requirements:

[ ] Your code is properly formatted according to this configuration.
Run mvn clean verify -DskipTests so that imports are correctly set.
[ ] Your commit messages are clear and reference the JIRA issue e.g: "[MGDOBR-1] - $clear_explanation_of_what_you_did"
[ ] The layers in the kustomize folder have been updated accordingly.
[ ] All new functionality is tested
[ ] Pull Request title is properly formatted: MGDOBR-XYZ Subject
[ ] Pull Request contains link to the JIRA issue
[ ] Pull Request contains link to any dependent or related Pull Request
[ ] Pull Request contains description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket

PR reviewer(s) checklist

[ ] Check if code and Github action workflows have been modified and if the modification is safe
[ ] If the modification is safe, add the safe to test label

How to trigger pipelines and use the bots:

Run the end to end pipeline
Annotate the pull request with the label: safe to test. If you want to run the pipeline again, remove and add it again.

Rebase the pull request
Comment with: /rebase.

Deploy BOT
Comment with `/deploy <dev,stable> when the PR has been merged to deploy to a target environment.

Its PR check are failing because its expect changes which are part of this PR itself.
@robpblake @manstis @kostola Could you please review it with failing PR check itself.

closing I favour of https://github.com/5733d9e2be6485d52ffa08870cabdee0/sandbox/pull/1079


ASE: Memory Analysis is showing negative percentage

Example-
sites/luiintegrationv1-prod/detectors/memoryanalysis?startTime=2016-12-13T00:59:55.239Z

I am already getting the RD instance name from the role heartbeat table and that table also has core count. I make an assumption that physical memory size is core count * 1.75GB:

Before:

After:


how to use fasthtml under apache2 running on ubuntu

really a newbie question, but I am struggling with this, I tried many things, even added a line to print "Content-Type" and nothing works, other things are working fine (I put up a perl cgi script, php etc. they are working fine.
I am clearly not following something, so any help would be much appreciated (and maybe a section on  apache2/ubuntu or nginx/ubuntu could be added ?

fasthtml use uvicorn and similar to fastapi setup. I think search for apache + uvicorn or apache + fastapi can give you some idea.

Thanks, that was helpful, reading up on some info now
On Tue, Jul 30, 2024, 10:29 PM Cheng Yang @.***> wrote:

fasthtml use uvicorn and similar to fastapi setup. I think search for
apache + uvicorn or apache + fastapi can give you some idea.
—
Reply to this email directly, view it on GitHub
https://github.com/AnswerDotAI/fasthtml/issues/107#issuecomment-2259520169,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AENF2AVUDS6R7BRVGFTBZQTZPBDYJAVCNFSM6AAAAABLW7RO3CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENJZGUZDAMJWHE
.
You are receiving this because you authored the thread.Message ID:
@.***>

really a newbie question

All questions are good questions!
FastHTML is an ASGI app that uses the uvicorn library for local development. Fortunately for us, uvicorn includes deployment instructions in their documentation. 😄
https://www.uvicorn.org/deployment/


docs: Include example of how to populate a source table using the source model

Signed-off-by: Allison Doami<EMAIL_ADDRESS>Summary of Changes
Included example of how to populate a source table in Snowflake from an Airflow repo that uses the DAG ID as the file name.

Would love to get this merged so others can benefit!

Thanks!
Looks reasonable to me, but theres' a bunch of whitespace changes. Mind taking them out?

@markgrover is there a problem removing the end line spaces? Feel like it's better to remove them 😅


Compute shuffled model data

In this pr, the shuffled model data for each of the data splits is computed and stored. Please let me know if you have any suggestions.

Thanks @axiomcura, merging now!


Install GraphQL Client in app and use response from GraphQL Server to populate the landing page

Currently, we are just doing a regular "fetch" to grab data from the existing API endpoint.
I want to update the code to use GraphQL Client to grab data from the test GraphQL Server endpoint. Make sure to only grab data actually used on the landing page.

This work was done as part of https://github.com/CDLUC3/dmsp_frontend_prototype/pull/24 and PR reviewed and merged.


[Bugfix] The buy api is incorrect

Problem
The api needs to be tested and updated

This is fixed in v2.9.0

This is fixed in v2.9.0


Enable CONFIG_IP_VS_NFCT in 4.10.8-docker

Hi,
This module is enable on 4.10.8-docker kernel for armv7 and aarch64, but not on x86-64.
Thanks,

This module is enabled in all recent kernel releases (the latest 4.4, 4.9 & 4.12).


Keep track of layer indexing and parse more tile data

This should fix #45 and #34.
Also @mattyhall, could you go through existing issues and check on them? Most of those are already fixed, so it's just a metter of closing them.

Thanks


New Airport ZGSZ

I am trying to create the map for Shenzhen International Airport (ZGSZ) based on vhhh.json. However, I somehow cannot get any web browser to load zgsz.json in my fork. The game is fully loaded, but the map is not in the selection menu. There is no parsing error according to the Javascript console in Chrome developer tools. Can someone have a look and see what is wrong?
(The wind speed, arrivals and departures are still copied from VHHH, with the fix names updated.)

Also, how to flag the airport as "in development"?

As in "San Francisco International Airport &#9983" the unicode at the end of the human-readable name is the "flag" for "work in progress".
As far as it not showing up in the list be sure to add it to the list in airport.js

It's working for me, try opening it in an incognito window.

Yep it's up and working now. However a weird JS error pops up. "Cannot read property '0' of undefined" at line 476 in airport.js, one that governs the randomness of the arrivals. I will try to update the map with realistic arrivals tonight and see if it changes.

@Fechulo Where did you get the arrival data for VHHH (airlines served, most common STARs taken, frequency of arrival,  etc.)? I need to so the same for ZGSZ.

@siyuanr For the airlines, I looked at FlightAware and checked which ones were the most frequent at VHHH. For the STAR's, I looked at the most common destinations and checked the flight plans. From that I can see which STAR's they use. And to be honest, the frequency of the airlines was just a very rough estimate. However if you're looking for more realism, for EGLL I used and app called FlightScheduler, counted the number of arrivals per airline and entered that as the frequency.

All done! All the waypoints are here, completely drawn from the RNAV charts. Test it on my fork! A few remarks:

All the charts and manuals can be reviewed at eAIP China (2015.11.15).
SIDs are only for RWY 15 at the moment, as we wait for progress on #324.
Fixes CF33 (IF point for RWY33 Approach) and SZ101 (first point for many SIDs) share the same location; I chose to display CF33 on top because SIDs can be pre-programmed.
For the RNAV approaches we are supposed to turn RWY heading after flying by the IF point; however since we haven't been able to chain a fix and heading command together, I mapped out the ILS beacon locations so that we just can align with those.
New airlines CSZ, CHH, CXA, AXM, as well as the non-South American versions of KAL and CCA are added for sake of reality.

@siyuanr I tried it out briefly... looking and working good to me! :) And #324 is wrapping up soon here... I'll have a PR by tonight for it.

@siyuanr Same question for ZGSG... have you finished, and could it be merged in here?

The ATC repository is being migrated to it's new home at https://github.com/openscope/openscope,
and thus, all issues are being closed. If this is still an issue with the latest version of the sim
(accessible at http://www.openscope.co), or is a feature you still think we are lacking,
please reopen the issue at the new repo.
Please note that the vast majority of these issues have been copied to the new repository, or else are covered by other issues created there. See the below screenshot for what it looks like when your issue is known in the new repo:

Thank you!
Closing this issue.


SPIKE[6]: Change the underlying format of the scripture pane to improve copy/paste

[ ] What changes could be made to the current display of the USFM in the scripture pane?
[ ] If the scripture pane was changed would it actually improve formatted text in the clipboard, such that it pastes in an expected manner?
[ ] How much work will it be to make these changes?
[ ] What impact will these changes have on the app? Does it slow down the highlighting? Does it break other actions?

@RoyalSix Does issue #444 fix this so we don't need to replace copy/paste?


Distribution Errors [Bad CFBundleExecutable]

I'm trying to submit an app that uses the framework to App Store Connect, but I get the following errors.
Any idea how to solve it? I've tried to inspect and change the plist, without any luck.

App Store Connect Operation Error
ERROR ITMS-90260: "Bad Bundle Executable.
You must include a valid CFBundleExecutable key in the nested bundle SGPlayer
[maccatalyst.com.tm.streams.pkg/Payload/streamsMac.app/Contents/Frameworks/SGPlayer.framework]
property list file."
ERROR ITMS-90261: "Bad CFBundleExecutable.
Cannot find executable file that matches the value of CFBundleExecutable in the nested bundle SGPlayer
[maccatalyst.com.tm.streams.pkg/Payload/streamsMac.app/Contents/Frameworks/SGPlayer.framework]
property list file."

Please，Is it solved


Fix fasd completion (no submodule)

As @Eriner said in  #132, this is fix without adding fasd as submodule.

Fix word completion for fasd
Fix initialization script to have all fasd-related functions inside it
(functions was not called properly when they was in separate files)
Add another alias v for fasd that will access recently used files
from .viminfo

Fixes: #111

Fantastic, thanks @deathbeam, this is exactly what I was looking for!! Very much appreciated!


chore: Add sponsors to README

Prerequisites checklist

[x] I have read the contributing guidelines.

What is the purpose of this pull request?
Add sponsors info to the README

What changes did you make? (Give an overview)
Modify the generate-sponsors.js file to update the README  with the latest sponsor information.
Related Issues
#649
Is there anything you'd like reviewers to focus on?

We should include README.md here:
https://github.com/eslint/eslint.org/blob/99926fd5a9df419d44844c6b08b8652404d316ab/tools/commit-data.sh#L13

Ya, thanks I missed it will add it.


Rejected Application | 'Contact' Column Blank on Application Request page

Current Behavior
If a application is rejected then contact column shows blank in Application request
Expected Behavior
Contact column should show the value even if the application is rejected
Steps To Reproduce

Log in to the Portal Application as CX Admin.
Navigate to the Application Requests page.
Filter or locate application requests that have a 'Rejected' status.
Observe the Contact column for these entries.

RCA:
Rejected application have user in deactivated status and hence api returns null for contact field
This issue is similar to #871

good finding comment was added to the PR. small improvement would be to add a short screenshot this would easen up the understanding without the need of going into the portal and replaying your description just to understand what you are talking about.

Thanks @MaximilianHauer . Sure , Is it fine if i paste the screenshot of Cofinity-X portal as we don't have access to Catena-X portal.


Add ability to make wrapping in context optional

I am trying to use the linguify inside the calendar library I am working on. I would need to get string values for months and weekdays in different languages in typst code. Unfortunately it doesn't seem possible due to the fact that the linguify function creates context.
The simplest reproducer
#import "@preview/linguify:0.4.0": *

#let database = toml("lang.toml")

#let localize(weekday) = {
  linguify(weekday, from: database, default: weekday)
}

#set text(lang: "en")
#assert.eq("sunday", localize("sunday"))

this code fails with
equality assertion failed: value "sunday" was not equal to "context()"

Can you try to use _linguify() instead. Not sure if this function is exported though. Then you need to provide context around the function calling it.

The _linguify is private and not reachable.

Ok I will fix this in the next version. But might take a while until I find some time.

I published 0.4.1 if it is merged you can use
#context {
  lflib._linguify(...)
}

which returns an dictionary with { "ok": "result" }  or { "error" : "error_message" }
you still need to provide context around the function call, because of checking for text.lang,
maybe i will refactor it in the future to have an context free version if all parameters are passed in.


How to implement a delete message fucntion?

This code doesn't work, I have added bot to a group and make it as an admin
bot.onText(/hi/gi, msg => {
  bot.deleteMessage(msg.message_id)
})

Error
Unhandled rejection Error: ETELEGRAM: 400 Bad Request: message identifier is not specified
    at /home/usama/Documents/back-end/telebot/node_modules/node-telegram-bot-api/src/telegram.js:280:15
    at tryCatcher (/home/usama/Documents/back-end/telebot/node_modules/bluebird/js/release/util.js:16:23)
    at Promise._settlePromiseFromHandler (/home/usama/Documents/back-end/telebot/node_modules/bluebird/js/release/promise.js:517:31)
    at Promise._settlePromise (/home/usama/Documents/back-end/telebot/node_modules/bluebird/js/release/promise.js:574:18)
    at Promise._settlePromise0 (/home/usama/Documents/back-end/telebot/node_modules/bluebird/js/release/promise.js:619:10)
    at Promise._settlePromises (/home/usama/Documents/back-end/telebot/node_modules/bluebird/js/release/promise.js:699:18)
    at _drainQueueStep (/home/usama/Documents/back-end/telebot/node_modules/bluebird/js/release/async.js:138:12)
    at _drainQueue (/home/usama/Documents/back-end/telebot/node_modules/bluebird/js/release/async.js:131:9)
    at Async._drainQueues (/home/usama/Documents/back-end/telebot/node_modules/bluebird/js/release/async.js:147:5)
    at Immediate.Async.drainQueues [as _onImmediate] (/home/usama/Documents/back-end/telebot/node_modules/bluebird/js/release/async.js:17:14)
    at processImmediate (internal/timers.js:439:21)

let message_id = message.message_id;
bot.deleteMessage(chat_id, message_id);```

let chat_id = message.chat.id;
let message_id = message.message_id;
bot.deleteMessage(chat_id, message_id);

let chat_id = msg.chat.id;
let message_id = msg.message_id;
bot.deleteMessage(chat_id, message_id);

you should do like this in groups to delete the message ....

this works, thanks a lot friend


Split up calicoctl_test into separate files based on class

Files have changed too much - closing this PR.
We can re-implement as necessary.


Frank!Doc: Update compatibility.xsd to ignore the sequence of child elements

This commit changes the following:

compatibility.xsd, the XSD used by the Frank!Framework to parse configs, should not check the sequence of elements. We want this because old configurations do not respect the element sequence prescribed by strict.xsd.
Both strict.xsd and compatibility.xsd had empty sequences. These do not have any purpose so they are removed.
More integration tests are added to DocWriterNewIntegrationTest to improve code coverage.

The new integration tests introduce input class hierarchies and expected XSD documents. The expected XSD documents should be produced by the Frank!Doc when the input class hierarchies are processed. It was difficult to write the correct expected values for these XSD documents. I checked each expected XSD by using it to verify and example XML document. I had an example XML document open in Eclipse that referenced an XSD. Then I checked the errors I saw while editing the XML.

I put this pull request back to "draft" because I want more test coverage. Here are some ideas about the coverage I want to add:

When there are role names with multiple config children, I want to test that all config children go under the same , not just the config children of the role. I can test this by adding more config children.
I want to test that attributes still appear at the right place. Therefore I have to add attributes.


Offsetting is wrong even for 64 bit

Cause
If you try running any program with arguments like long, long, char or something its going to break. This is probably due to this part of the compiler code:
if (offset-osize)%8 > 0 {
   offset-=8-(offset-osize)%8;
}

It only works for very specific cases, and even then it breaks.
How to recreate the issue
The easiest recreation would be to use a library like strlib. buffers from the examples breaks because of strset
include "../libs/strlib/strlib.spl"
include "../libs/libc/stdio.spl"
func main(){
    let name: [char, 500];
    strset(name, 500, 0);
    scanf("%s",name);
    printf("Hello: %s\n",name);
}

Fixed in 0.17A


No compile-time guarantee

The purpose of this package is to provide a provable, compile-time guarantee that it holds a non-empty collection. There are valid reasons why a compile-time guarantee is better than a runtime check.
Recently the implementation was changed to a runtime check without updating the readme, so now this one-liner crashes the library:
withUnsafeBytes(of: "") { $0.load(as: NonEmptyString.self).first }

All because the library no longer provides a compile-time guarantee as it claims. A previous implementation with head is resilient to those types of crashes.
I suggest either to revert the implementation or to change the description of the library.

We believe "compile-time guarantee" is still accurate with the exposed API, though we do understand invariants could leak into the system.
We evaluated the trade-offs before the change, and decided that for performance and correctness of the collection algorithms themselves, holding onto the whole collection was important.
Your one-liner still crashes in the earlier design, just earlier (at the time of loading, not the time of first). When unsafe APIs are involved, you throw a lot of these compile-time guarantees out the window.
We've talked about introducing a "phantom" element to force the memory layout:
struct NonEmpty<C: Collection> {
  var rawValue: C
  private let phantom: C.Element  // grabs any element of C at initialization
}

But haven't gotten around to exploring it, or found a material need to do so yet with the existing APIs.

Your one-liner still crashes in the earlier design, just earlier (at the time of loading, not the time of first). When unsafe APIs are involved, you throw a lot of these compile-time guarantees out the window.

This is precisely where it should crash if it provides a compile-time guarantee. This crash means that it's impossible to construct this type in a wrong way, which is what the user of such a library would want.
Here is an example. Let's say we have a library that accepts a non-empty data type for one of it's APIs:
class Library {
  static func acceptValidInput(_ s: NonEmptyString) {}
}

With the current implementation of NonEmpty, we would be able to construct an empty version of this data type and pass it to the library. All the non-force-unwrapped APIs would work and the library would proceed to execute some effects, only crashing much later, when non-emptiness of the data type would be important.
let input = withUnsafeBytes(of: "") { $0.load(as: NonEmptyString.self)}

Library.acceptValidInput(input) // Library happily doing its effects, beleiving the type is non-empty

With a previous implementation, it would be impossible to construct this data type empty, and any input passed to the function would guarantee that it won't fail. Exactly what we want here.
let input = withUnsafeBytes(of: "") { $0.load(as: PreviousNonEmptyString.self)} // ❌ Crashes

Library.acceptValidInput(input) // Not executed

This is not a contrived example, our own public library accepts a non-empty key to work using a head implementation, and I'm evaluating if the new version of the API should use this library for it instead.

withUnsafeBytes and load are not contrived either, in another library we are interacting with a our own Swift code compiled to WASM. The interaction requires us passing pointers from Swift to WASM, and WASM loading the data types. Using the current implementation for that use case would be too dangerous.
I would even go as far as to say that the current implementation of NonEmpty is even worse than sprinkling guards around, as it lulls users into a false sense of security, guards would be a safer option.

We've talked about introducing a "phantom" element to force the memory layout:

This looks like an interesting compromize. It would guarantee that the data type holds some element, though, it doesn't prove the collection non-empty because the element is not proven to be a part of it.

I beleive correctness is more important than performance for users of this library. Users who value performance more than correctness would be satisfied with runtime checks anyway.

This looks like an interesting compromize. It would guarantee that the data type holds some element, though, it doesn't prove the collection non-empty because the element is not proven to be a part of it.

Right, it would only "prove" the NonEmpty is non-empty in that the structure contains at least one value. It makes no guarantees about that value's relation to the collection.

I beleive correctness is more important than performance for users of this library. Users who value performance more than correctness would be satisfied with runtime checks anyway.

Ideally we would be able to provide correctness and performance, so if this compromise is fruitful we'd love to hear about it. Note that one of the motivating reasons to wrap the entire base collection was for correctness. There runtime bugs lurking in the previous implementation due to index shuffling and subsequence behaviors. While we have introduced potential invariants with the migration, we were able to eliminate a whole other class of bugs.
If memory correctness is more important for you, we recommend locking to the old version for now.
If you explore the compromise I mentioned earlier and it proves fruitful, though, we'd love to take a look!


Array(T)#* of recursive type alias

When updating an old project to the current Crystal release I found a case that used to work but now fails to build and looks like a compiler bug. Here is a reduced version:
alias Type = Array(Type) | String

arr = ["aa", ["23"]] of Type

p arr # this works
p arr * 2 # this doesn't compile

Here is the compiler output:
In test.cr:6:7

 6 | p arr * 2
           ^
Error: instantiating 'Array(Type)#*(Int32)'

In /usr/share/crystal/src/array.cr:337:23

 337 | return Array(T).new(times, first)
                       ^--
Error: instantiating 'Array(Type).class#new(Int32, (Array(String) | Array(Type) | String))'

In /usr/share/crystal/src/array.cr:116:7

 116 | @buffer = Pointer(T).malloc(size, value)
       ^------
Error: instance variable '@buffer' of Array(Type) must be Pointer(Array(String) | Array(Type) | String), not Pointer(Array(String) | Array(Type) | String)

In particular, notice that the last line says Pointer(Array(String) | Array(Type) | String) is not the same as Pointer(Array(String) | Array(Type) | String). But they look pretty much the same to me. Not sure what is going on.
This used to work on Crystal 0.30.1 (https://play.crystal-lang.org/#/r/88y1) but broke on Crystal 0.31.0 (https://play.crystal-lang.org/#/r/88y0)

arr.uniq also fails to compile for a recursive type.
May be worth to mention: #5155

Possibly #8087 or #8048

The first snippet shouldn't compile at all because an Array(String) is inserted into Array(Type). This is a bug.
I hope someone removes recursive aliases in the future...

The original snippet no longer compiles due to #10224.

The original snippet now doesn't compile with a correct error message. And this example correctly compiles:
alias Type = Array(Type) | String

arr = ["aa", ["23".as Type]] of Type

p arr
p arr * 2


feat: implement definitions for markdown

This was already supported by vscode-markdown-languageservice, but not implemented by volar-service-markdown.

LGMT, thanks!


Copy feature for copying the queries

Copy feature for copying the queries

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/c341798aa21766ea6ed495dbb5ede2a6f5e93cc9/
Edited pages:

v19.1/sql-faqs.md

LGTM


fix: builderbaseimage default value in makefile

What type of PR is this?
/kind bug
Any specific area of the project related to this PR?
/area build
What this PR does / why we need it:
https://github.com/falcosecurity/driverkit/pull/186 introduced a subtle difference in the Makefile:
BuilderBaseImage makefile-set value was added a _$(uname -m) suffix.
This breaks cross builds because default value will try to use docker.io/falcosecurity/driverkit-builder:$(commit)_x86_64 image for cross build (instead of the manifest-created one, ie: the one without arch suffix), resulting in wrong image being used for arm64 cross build.
Moreover, this PR also bumps some deps.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/cc @dwindsor

LGTM label has been added.  Git tree hash: 15a352b19f92db711cb227b9cecdac28036d3810

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: dwindsor, FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP,dwindsor]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Document / Deprecate arrow_flight::utils::flight_data_from_arrow_batch

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
Something I discovered while working on code base at Coralogix and in Ballista.
arrow_flight::utils::flight_data_from_arrow_batch is used as method to convert a singleRecordBatch to FlightData.
This method extracts both the dictionaries as the RecordBatches.
While convenient, if the dictionary is the shared across batches, the caller will probably send the FlightData for these dictionaries over and over again over in the stream.
Describe the solution you'd like

Mark the method as deprecated to encourage users to use the IpcDataGenerator API instead.
https://docs.rs/arrow-ipc/28.0.0/arrow_ipc/writer/struct.IpcDataGenerator.html
Describe alternatives you've considered

Additional context

🤔  looks like we use flight_data_from_arrow_batch in IOx as well
https://github.com/influxdata/influxdb_iox/blob/061d582a9b4ca1493a875acd4d27ad5850a61659/service_grpc_flight/src/lib.rs#L423

It looks like the comments around https://docs.rs/arrow-ipc/28.0.0/arrow_ipc/writer/struct.IpcDataGenerator.html are non existent so some description there as well as some examples would also likely help

label_issue.py automatically added labels {'arrow'} from #3353

What should we use to replace flight_data_from_arrow_batch ?


feat: rename agency status fields (#734)

Ticket
https://github.com/usdigitalresponse/usdr-gost/issues/734
Description
Following some UXR feedback, there need to be some updates on the "Agency Status" fields in both the Grants modal and Grants tab
Screenshots / Demo Video

Testing

Grants Modal

Go to the Browse Grants tab
Click one of the grants
Select Agency Status in the GrantDetails
You should expect to see the updated values in both Result (now: Applied) and Rejections (now: Not Applying) sections

My Grants

Go to the My Grants tab
You should expect to see the updated tabs

Checklist

[x] Provided ticket and description
[x] Your PR title contains the ISSUE ticket number e.g "86: ..."
[x] Provided screenshots/demo
[x] Provided testing information
[x] Provided adequate test coverage for all new code
[x] Run automated tests (docker compose exec app yarn test)
[x] Added PR reviewers
[ ] Ensure at least 1 review before merging

PM review - Looks good from the frontend. Thanks!

@LieAlbertTriAdrian I spoke with @mhuang-usdr today; the work for this seems like it's good to go. Can we reopen this and move forward with review?

@TylerHendrickson I can reopen this. Will also update this during our stand up today


M3 Max 16C results added

Added results for latency test MacBook Pro M3 Max 16C

Could you please add the full log for completeness? Thanks!


Creating Helia in nextjs fails with "TypeError: Cannot assign to read only property 'Symbol(Symbol.toStringTag)' of object '#'"

This error happened in both Typescript and Javascript.
.
I am using the sample code from the Helia examples repo in a different project. When I tried to run the example repo directly, I got a different error: https://github.com/ipfs-examples/helia-examples/issues/289

I just reran my sample repo, and downgrading to 14.0.4 resolved the issue for me. Sounds like this issue and https://github.com/ipfs-examples/helia-examples/issues/289 might have the same root cause.

Nextjs had a regression but it was fixed in v14.1.3.
Please reopen this issue if you are still having problems.


Use MAVLink messages when IP network is not available

Currently the daemon streams video assuming there is a cabled or WiFi network. However, especially thinking about UAVs, the vehicle might be outside the range of WiFi. To cover such scenario, a possible way to continue streaming video is through MAVLink messages over RF.

No. The video should never be sent over mavlink. Control messages can though, like for start/stop recording, taking still photos, etc

Text updated. Thanks. BTW, I think you can edit that as well.


Weird sizes returned when using setWidth / setHeight

Hi,
I am a first time user of your library, and there's something I should be doing wrong but I can't find out. I'm testing your lib with a simple app running in a pixel 2 virtual device (1920x1080). And this is how I init the lib as per the documentation:
ScreenUtil.instance = ScreenUtil(width:1080, height:1920, allowFontScaling: true)..init(context);
When I try to use setWidth for a value of 200 in this device it returns a value of 76.19 which is wrong. Looking at the source code I see how you do:
setWidth(double width) => width * scaleWidth;
and scaleWidth is:
get scaleWidth => _screenWidth / instance.width;
This is where things get weird. _screenWidth following mediaquery documentation is measured in Logical pixels but in the ScreenUtil constructor we must provide physical pixels. So the reason I get 76 instead of 200 (or a value close to it because I'm running on a device with the designed resolution) is because the scaleWith is calculated with 683.42/1080 , and it should be 1080/1080 = 1.
I know it must be me doing something I shouldn't because I don't see any bug opened about this problem. But there's no much to it. Could you help here?
Cheers.

The unit of input is px, and the unit obtained is dp 。
maybe it is right?


Issue using Kerberos auth

I have RD setup on a RedHat server and have Kerberos authentication configured on that server for our AD domain.
When I change the plugin auth to 'kerberos' instead of 'negotiate', I fail to connect with the following error:
/usr/local/share/gems/gems/winrm-2.2.3/lib/winrm/connection_opts.rb:72:in `validate_required_fields': user is a required option (RuntimeError)
16:23:59				from /usr/local/share/gems/gems/winrm-2.2.3/lib/winrm/connection_opts.rb:61:in `validate'
16:23:59				from /usr/local/share/gems/gems/winrm-2.2.3/lib/winrm/connection_opts.rb:33:in `create_with_defaults'
16:23:59				from /usr/local/share/gems/gems/winrm-2.2.3/lib/winrm/connection.rb:66:in `configure_connection_opts'
16:23:59				from /usr/local/share/gems/gems/winrm-2.2.3/lib/winrm/connection.rb:29:in `initialize'
16:23:59				from /var/lib/rundeck/libext/cache/rd-winrm-plugin-1.7.0/winrmexe.rb:135:in `new'
16:23:59				from /var/lib/rundeck/libext/cache/rd-winrm-plugin-1.7.0/winrmexe.rb:135:in `<main>'

Why does the error output specify I'm missing a 'user' option?
I see a valid 'user' listed in the 'variables' output in my debug.
I have tested that this user can log into my RD server locally using kerberos, and it passes my kinit tests.
Am I missing something here?

I was able to rectify my problems. It seems I was running into 2 issues.

the winrm gem tries validates the user/password options, which were missing from winrmexe.rb
the rundeck user cannot access krb tickets for my winrm user.

I created a workaround for this changing the 'auth' section of winrmexe.rb to the following --
when 'kerberos'   system "echo #{pass} | kinit #{user}"   connections_opts[:user] = user   connections_opts[:password] = pass   connections_opts[:transport] = :kerberos   connections_opts[:realm] = realm
This satisfies the reqs of the gem and caches a krb ticket for my winrm AD service account.
Not sure if this is the right or best way to do things, but its working for my in my test environment.


Building 3.18-1361 with Visual Studio 2017, but projects all require tools v142...?

Not sure if I'm overlooking something, but I've got to downgrade all the projects to tools v141 in order to build on VS2017, correct?
I'm not quite ready to upgrade to VS2019 yet.

Also, after changing all projects to v141, I get build errors (or maybe one repeated), which I have posted here:
https://github.com/tsduck/tsduck/issues/313

Solution seems to be just a case of disabling warning C4623 across some/all projects.
But out-of-the-box the latest source does not build on VS2017/WIn10 without the above two changes.

Hi @oviano
Thank you for reporting this. Maintaining a generic configuration for Visual Studio is a challenge I never really mastered, despite many efforts to manually structure .vcxproj  and  .props files.
I no longer have VS 2017 and, since I use the Community edition, I do not know how to explicitly install VS 2017 side by side with VS 2019. Let's try to find a common unique configuration which equally works with my VS 2019 and your VS 2017. Any VS 2015 around ?
I have made some changes in commit 168160c8354a12ca0165edd8f52dc4f6382ae8b5 which could fix the issue. It definitely works on VS 2019. Please test on VS 2017, without modification.
There was an attempt to compute the right PlatformToolset in a configuration file but it apparently failed to address all cases. And v142 was only a fallback solution, when everything else failed. Now, I have used another method and removed the hard-coded fallback.
I have also disabled the warning from the VS header files which appeared in VS 2017 but not in VS 2019.

Thanks - I manually applied those changes to the 3.18-1361 code and I can confirm that it now correctly builds with no warnings or errors on VS2017/WIn10.

Thanks for your feedback.


kafka(ticdc): fix kafka consumer doesn't check partition and old message

What problem does this PR solve?

Issue Number: close #11482
What is changed and how it works?
Add partition check and old message check about simple protocol
Check List
Tests

Unit test
Integration test

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
None

/ok-to-test

/retest

/check-issue-triage-complete

/retest

/retest


Merge/2.0 to 2.2 for leak fixes and hashtree build issue

Hashtree cleanup - #1242
Unregister vnode stats - #1282 (RIAK-2312)
Memory leak in bitcask - #1331 (RIAK-1576)

:+1:

@borshop merge


Querier: How to log instant queries

A lot of our queriers were being OOM killed out of a sudden.
We checked incoming requests from Grafana as well as the http_requests_total metric. Didn't see any unusually heavy queries from Grafana.
However, I saw an increased rate of requests on /query endpoint for the whole duration of this incident.
I want to track and log all the instant queries which come on /query. How do I do that?
Load balancer logs show me incoming requests from Grafana (all hitting /query_range).
I tried setting the log level to debug but all I saw were range queries.
Also tried setting --query.active-query-path but then the contents of that file only showed raw queries. No way to differentiate b/w range and instant queries.

request logging https://thanos.io/tip/components/query.md

      --request.logging-config=<content>
                                 Alternative to 'request.logging-config-file'
                                 flag (mutually exclusive). Content
                                 of YAML file with request logging
                                 configuration. See format details:
                                 https://thanos.io/tip/thanos/logging.md/#configuration
      --request.logging-config-file=<file-path>
                                 Path to YAML file with request logging
                                 configuration. See format details:
                                 https://thanos.io/tip/thanos/logging.md/#configuration

tracing , query span just  like log.
https://thanos.io/tip/thanos/tracing.md

serviceName='thanos-query' and spanName in ('promql_instant_query','promql_range_query')

span has a attribute name query.

Thanks a lot for bringing that up.
I glanced through it in the flags list but didn't see the complete configuration; I had a feeling that I would be flooded with logs if I enabled this.