Remaining strings need translation

Here are the remaining strings that need to be translated for all locales for the release:

[ ] An error occurred
[ ] Can’t reach the internet
[ ] Check location services
[ ] Enable location services
[ ] Go to Settings
[ ] It took too long finding your location.
[ ] It took too long finding your location. Setting your device’s Location Services to “High Accuracy” can sometimes fix this problem.
[ ] Server error
[ ] Services are not available offline. Please connect to the internet and try again.
[ ] To find services, please turn on Location Services for Arrival Advisor in Settings and try again.
[ ] Try again
[ ] We’re having difficulty connecting to the server. Please try again later.
[ ] No services to show

These strings are already on Weblate in the jsx_strings component. All translators need to do is go into a locale and click on:

@ArtiBPG will have these strings translated
@tomy-pg will add these strings to the client
Acceptance Criteria

[ ]  No services to show is translated in all locales

Steps for QA:

Switch between locales and click on a topic that does not have recommended services, IE: Opening a bank account
See to it that No services to show is translated.

All strings in jsx_strings and release_notes in Weblate translated and committed.

The localized string was added to Weblate 5 days prior to the release and merged into master. The tag 1.6.0 seems to be off-sync which is why we currently do not see this change. This also affects Issue #826


Processing task polling seems a bit slow when working through lots of small files that complete quickly

Fixed as part of gh-3966


onPageNav menu mobile button should not show when user did not enable onPageNav in siteConfig

Is this a bug report?
Yes

Have you read the Contributing Guidelines on issues?
Yes
Environment
Docusaurus 1.2.0
Steps to Reproduce

(Write your steps here:)

Change siteConfig.onPageNav to false or remove the value.
Navigate to a doc page.

Expected Behavior

The three dots for the mobile onPageNav menu should not appear.
Actual Behavior

The three dots for the mobile menu should not appear but they still do. Clicking on them does nothing but it's confusing.
Reproducible Demo

If no one has taken this I would like to give it a stab.

@thai56 it's yours
I've added an "issue-claimed" label so that others will know not to start work on the issue. If you change your mind about the issue, no worries! Just let me know so that I can remove the label and free it up for someone else to claim.
I'll check in with you periodically so that we can keep the task updated with the progress.

#767

Closed by #768


Suggest your own feature, add a new issue!

If you want to contribute or add a decent feature to the project, feel free to create an issue!

@Neilblaze I found that the readme is missing getting started guide like how to create a pull request, basics of git and GitHub, etc
So would like to add it
Can you please assign it to me

@Neilblaze Also we don;t have a proper HomePage so can I work for it having components like Navbar,Header, footer, About Section etc

@Tejas1510 thanks for your interest. Well, in the README, if you check under the Resources section, you'll find all you want.

@Neilblaze Also we don;t have a proper HomePage so can I work for it having components like Navbar,Header, footer, About Section etc

can you please elaborate this?

I wish to create a landing page foe the project which will have basic information about the project is actually about adding all basic a section a particular homepage has and then we can navigate to the current page we are having

I wish to create a landing page foe the project which will have basic information about the project is actually about adding all basic a section a particular homepage has and then we can navigate to the current page we are having

Cool! Do one thing, create a new issue mentioning this & mention me over there!


Merging changes between branches

👷 Deploy Preview for crafty-rero processing.

Name
Link

🔨 Latest commit
6cb27db330531c079dfc5e16688d38d9b23494a6

🔍 Latest deploy log
https://app.netlify.com/sites/crafty-rero/deploys/64fdc88b5cfc1b00087e29f9


Certain escaped characters in strings cause crashes on Windows

The following file compiles on Mac/linux, but produces an error elm-make.exe: elm-stuff\build-artifacts\0.16.0\user\projects\1.0.0\Main.elmo: user error (Bad encoding; the file must be valid UTF-8) on Windows:
nbsp = "\xA0"

The following also compiles on Mac/linux, but produces elm-make.exe: elm-stuff\build-artifacts\0.16.0\user\projects\1.0.0\Main.elmo: commitBuffer: invalid argument (invalid character): thread blocked indefinitely in an MVar operation on Windows:
nbsp = "\x9F"

This was brought to my attention via https://github.com/avh4/elm-format/issues/132

Closing, will track from #1280 meta issue instead

The \xFF syntax is no longer allowed. You can only use \uFFFF now, and it preserves the escaped version into JavaScript.


HIVE-28487: Fix outdated MetastoreSchemaTool class reference in schemaTool.sh

In a94c0306f48215d96be3b842525c4341772d26f9 MetastoreSchemaTool was moved from org.apache.hadoop.hive.metastore.tools.MetastoreSchemaTool to org.apache.hadoop.hive.metastore.tools.schematool.MetastoreSchemaTool, but it seems like schemaTool.sh was not updated.
This results in a ClassNotFoundException: org.apache.hadoop.hive.metastore.tools.MetastoreSchemaTool
What changes were proposed in this pull request?
Update outdated class reference to actual one.
Why are the changes needed?
Let schemaTool.sh work again :)
Does this PR introduce any user-facing change?
Yes, the shell script now works. See "How was this patch tested?" for the CLI output.
Is the change a dependency upgrade?
No
How was this patch tested?
Before (fails)
/stackable/apache-hive-metastore-4.0.0-bin $ bin/base --service schemaTool
Exception in thread "main" java.lang.ClassNotFoundException: org.apache.hadoop.hive.metastore.tools.MetastoreSchemaTool
        at java.base/java.net.URLClassLoader.findClass(URLClassLoader.java:476)
        at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:594)
        at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:527)
        at java.base/java.lang.Class.forName0(Native Method)
        at java.base/java.lang.Class.forName(Class.java:398)
        at org.apache.hadoop.util.RunJar.run(RunJar.java:321)
        at org.apache.hadoop.util.RunJar.main(RunJar.java:241)

After (works)
/stackable/apache-hive-metastore-4.0.0-bin $ bin/base --service schemaTool
# [...]
Failed to parse command line.
org.apache.commons.cli.ParseException: HiveSchemaTool:Parsing failed. Reason: Missing required options: [-help print this message, -info Show config and schema details, -upgradeSchema Schema upgrade, -upgradeSchemaFrom Schema upgrade from a version, -initSchema Schema initialization, -dropAllDatabases Drop all Hive databases (with CASCADE). This will remove all managed data!, -initSchemaTo Schema initialization to a version, -initOrUpgradeSchema Initialize or upgrade schema to latest version, -validate Validate the database, -createCatalog Create a catalog, requires --catalogLocation parameter as well, -alterCatalog Alter a catalog, requires --catalogLocation and/or --catalogDescription parameter as well, -mergeCatalog Merge databases from a catalog into other, Argument is the source catalog name Requires --toCatalog to indicate the destination catalog, -moveDatabase Move a database between catalogs.  Argument is the database name. Requires --fromCatalog and --toCatalog parameters as well, -moveTable Move a table to a different database.  Argument is the table name. Requires --fromCatalog, --toCatalog, --fromDatabase, and --toDatabase  parameters as well., -createUser Create the Hive user, set hiveUser to the db admin user and the hive password to the db admin password with this, -createLogsTable Create table for Hive warehouse/compute logs], dbType
Underlying cause: org.apache.commons.cli.ParseException : HiveSchemaTool:Parsing failed. Reason: Missing required options: [-help print this message, -info Show config and schema details, -upgradeSchema Schema upgrade, -upgradeSchemaFrom Schema upgrade from a version, -initSchema Schema initialization, -dropAllDatabases Drop all Hive databases (with CASCADE). This will remove all managed data!, -initSchemaTo Schema initialization to a version, -initOrUpgradeSchema Initialize or upgrade schema to latest version, -validate Validate the database, -createCatalog Create a catalog, requires --catalogLocation parameter as well, -alterCatalog Alter a catalog, requires --catalogLocation and/or --catalogDescription parameter as well, -mergeCatalog Merge databases from a catalog into other, Argument is the source catalog name Requires --toCatalog to indicate the destination catalog, -moveDatabase Move a database between catalogs.  Argument is the database name. Requires --fromCatalog and --toCatalog parameters as well, -moveTable Move a table to a different database.  Argument is the table name. Requires --fromCatalog, --toCatalog, --fromDatabase, and --toDatabase  parameters as well., -createUser Create the Hive user, set hiveUser to the db admin user and the hive password to the db admin password with this, -createLogsTable Create table for Hive warehouse/compute logs], dbType
*** schemaTool failed ***

CC @miklosgergely and @ashutoshc, as you did the original refactoring

Please create a new ticket rather than using the old ticket, just link the old ticket in that

@ayushtkn sure thing, created https://issues.apache.org/jira/browse/HIVE-28487 and update PR title + commit message

LGTM


Add okai to made_with_lush

Hey, I finally made my own colorscheme with Lush! Thank you for the great plugin :) Now i just wanted to add it to the 'Made with Lush' section :) Let me know if I have to change anything.

:tada:


[#4843] Add lettuce plugin

#4843
Separate the redis client into jedis and lettuce.

Codecov Report

Merging #4989 into master will decrease coverage by 0.06%.
The diff coverage is 44.44%.

@@            Coverage Diff             @@
##           master    #4989      +/-   ##
==========================================
- Coverage   40.85%   40.78%   -0.07%
==========================================
  Files        2646     2649       +3
  Lines       79267    79281      +14
  Branches    10716    10716
==========================================
- Hits        32383    32338      -45
- Misses      43958    44014      +56
- Partials     2926     2929       +3

Impacted Files
Coverage Δ

...rp/pinpoint/plugin/redis/jedis/CommandContext.java
21.05% <ø> (ø)

...lettuce/interceptor/AttachEndPointInterceptor.java
0% <ø> (ø)

.../plugin/redis/lettuce/LettuceMethodNameFilter.java
0% <ø> (ø)

...orp/pinpoint/plugin/redis/jedis/EndPointUtils.java
60% <ø> (ø)

...s/jedis/interceptor/AttachEndPointInterceptor.java
23.33% <ø> (ø)
:arrow_up:

...r/ProtocolSendCommandAndReadMethodInterceptor.java
0% <ø> (-9.1%)
:arrow_down:

...interceptor/RedisClientConstructorInterceptor.java
0% <ø> (ø)

...oint/plugin/redis/jedis/CommandContextFactory.java
100% <ø> (ø)

...edis/jedis/interceptor/SetEndPointInterceptor.java
33.33% <ø> (ø)
:arrow_up:

.../lettuce/interceptor/LettuceMethodInterceptor.java
0% <0%> (ø)

... and 29 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9f7ddfc...07739b4. Read the comment docs.


[TypeScript] Drop support for TS 3.1

It is my understanding that we are not currently supporting TS 3.1 well and no customer needs it. Furthermore, the guidelines is a bit vague in that it says we should support versions > 3.1 (not including 3.1 itself) but also gives tips about how to support TS 3.1. Furthermore, based on a recent discussion in our teams channel, we think it is reasonable to set the minimum version to support to 3.5.
Rationale by @xirzec:

After shipping several Track 2 libraries, we have determined that TS 3.5 is a much more reasonable minimum bar to maintain, because of downlevel-dts support and convenient language features such as Omit. As we have shipped several libraries that require 3.5 or higher without receiving any negative feedback from consumers, we feel confident in updating this requirement to reflect the reality of the ecosystem. We will continue to listen to our customers and revise this guideline in the future as necessary.

+1.
The rationale for this is that we have not in practice supported TS3.1 for most libraries for any track 2 release and we have not gotten bugs. At the same time, supporting TS3.1 would be a laborious process.

This PR seems to have come from nowhere...
@deyaaeldeen Can we have a PR description added regarding why we are doing this or which prior conversation lead us here?

Is TS 3.5 an arbitrary choice or do we have some data/observation that this version is important to support?

@ramya-rao-a done.
@willmtemple one thing to note is that downlevel-dts supports down-leveling to TS 3.4 and up, so I think it is much more expensive to support versions earlier than 3.4.

I was tempted to say 3.4 because that's what downlevel-dts is attempting to support, but 3.5 added Omit which I find quite handy.

Is there any telemetry that may help inform this decision? For Cosmos, we have smoke tests down to 3.1 that gate PRs. So it is possible this is breaking for us but I can't say for sure one way or the other.

For Cosmos, we have smoke tests down to 3.1 that gate PRs. So it is possible this is breaking for us but I can't say for sure one way or the other.

@southpolesteve I did not know that cosmos has rigorous support for 3.1, thanks for pointing this out! Well, now we are in a state where cosmos has support for 3.1 and the rest not so much, I am not sure about storage though (cc @ljian3377 @XiaoningLiu). One suggestion is to keep the cosmos support for 3.1 and create a plan to deprecate it at some point in the future after having a better understanding of the customer needs through telemetry or other means. On the other hand, we can move other packages to have official support for TS 3.5 and implement a CI check to have the support rigorous similar to what cosmos does. We already have a plan outlined here https://github.com/Azure/azure-sdk-for-js/issues/11319, so for this proposal, cosmos will be taken out of the list of packages to update.

@deyaaeldeen Makes sense. To be clear, I welcome this change. I've added it to our list for v4

I'm curious if anyone has any creative ideas for how we can detect the version of TS folks are using without surveying them. AFAIK there's no good way for us to know if the code that calls us was compiled by TS or not, much less what version of TS they may be using.

For storage SDK, I don't have accurate data. But we can check some internal customers for their status.
Another question is should we treat it as a breaking change for SDK to deprecate 3.1 and upgrade SDK major version?

@XiaoningLiu thanks for the update! Regarding your situation, I believe it depends on the situation of the package. I think if it claimed in the changelog to provide support for 3.1 before, then this is a breaking change (e.g. servicebus has this claim).

Changing supported TS version as a breaking change is interesting. I can see it meriting a minor version bump, but it's a little weird to me to make it a major version bump given that a JS-only customer would be totally immune to the "break".

@deyaaeldeen @xirzec what are the next steps for this PR?

@ramya-rao-a maybe this is a good MQ item?

@xirzec Yes. Just like we came up with the support policy for Node.js versions, we should have one for TypeScript and we can do that as part of MQ this time

Closing in favor of https://github.com/Azure/azure-sdk-for-js/issues/19843


Many machines need process clean-up

test-osuosl-ppc64le-ubuntu-16-04-1 had dozens of jdi processes left over from tests that didn't clean up properly after themselves.
An issue has been raised for this, and test-osuosl-ppc64le-ubuntu-16-04-1 is now clean, but I suspect test-osuosl-ppc64le-ubuntu-16-04-2 needs the same treatment.
I found this command cleaned up most of the issues: pkill -f "openjdkbinary"
I recommend pausing the machine on Jenkins, and running that command once there's nothing else running on it.
You'll know if it works if running "ps -aux | grep "java"" fills the screen beforehand, and does not fill the screen afterwards.
Happy hunting!

@smlambert Do you have anything in place to mitigate this? We could use a multi-configuration jenkins job to run periodically over all the machines and kill anything that looks like it's hung. With only one jenkins executor per machine it should be fairly easy to determine what has been left around (i.e. just about everything running as the jenkins user apart from the jenkins agent and the job that's checking!)

We should likely put a clean up step in the setup stage of testing to look for and kill test related processes (that may have hung due to previous test jobs that were exited in a hung/bad state).
I also agree that a separate job that cleans up stray processes and files will be good and more thorough (as it can search for a broader range of processes to terminate).  There is such a job in use at the OpenJ9 project, we can employ same/similar approach.

a job in use at the OpenJ9 project

https://github.com/eclipse/openj9/blob/master/buildenv/jenkins/jobs/infrastructure/Cleanup-Nodes
https://ci.eclipse.org/openj9/view/Infrastructure/job/Cleanup-Nodes/
https://ci.eclipse.org/openj9/view/Infrastructure/job/Sanitize-Nodes/
Note these are band-aid solutions which I don't like doing. The job has the option to run cleanup, sanitize or both. The jenkins agent is also killed in the sanitize path which we didn't like doing but found issues properly killing processes without killing the agent. Also on Windows it does a full reboot instead because of Cygwin issues.

@smlambert Quick and dirty check to identify the scope of the problem - the test machines in red here appear to have rogue java processes left around: https://ci.adoptopenjdk.net/view/work in progress/job/SXA-processCheck/

Thanks @sxa555
Your shell script does grep java, does this exclude the Jenkins agent itself which runs on all nodes?
I will edit your script to print the actual processes so we can better understand the root of the problem... - curious if processes are all leftover from openjdk tests or from other types of testing as well.  If from openjdk test jobs, I will say we should additionally be looking at why the underlying framework can not or is not killing/cleaning at the end of the run (and raising an issue against it if so).

Your shell script does grep java, does this exclude the Jenkins agent itself which runs on all nodes?

No - that one is explicitly grepped out - so in principle it would be safe to have an option on the job to kill all the processes it has detected ...

I will edit your script to print the actual processes

The job as-is will already show them in the console logs :-)

Have removed the offline openlab machines to allow the jobs to complete - killing off runs 1 & 2 which are not completing because of this :-)

Ya, I have looked more closely at the hung processes and it is very specifically the jdi tests on openj9, we should disable those tests for now, as there appear to be multiple problems.
1 major problem being that some of those tests expect to query a HotspotDiagnosticBean.
It would be good for an OpenJDK contributor to review those tests to see what would be required to make them applicable to more than just hotspot implementations.

@adamfarley Can you adjust the title of this issue please now that we've seen that it does not appear to be specific to any one machine

Have done a cleanup of most machiens over the last couple of days and we should try and keep the process check job clean now. Might move it into "production" state soon instead of work in progress

This is being maintained reasonably well through the process cleanup job at https://ci.adoptopenjdk.net/view/Tooling/job/SXA-processCheck/configure therefore closing for now.


Alternate Packages

And maybe pigz instead of gzip, but pigz requires zlib (zlib-ng works too). Cons: pigz doesn't have wrapper script such as for grep (zgrep). Can be installed for both, gzip just for wrapper script  if needed 😄 lol
Originally posted by @owl4ce in https://github.com/dslm4515/CMLFS/issues/14#issuecomment-869789668

So far....
Ninja -> Samurai
zlib -> zlib-ng
gzip -> pigz
pkg-config -> pkgconf [done]

And when prefer to build CMLFS with *BSD portable libraries instead of GNU 😸

@owl4ce how about gettext-tiny? Sometimes, autotools (autoconf+automake+m4+etc] run fine for regenerating configure for coreutils ... other times it fails. Have you had any similar experiences?

@owl4ce how about gettext-tiny? Sometimes, autotools (autoconf+automake+m4+etc] run fine for regenerating configure for coreutils ... other times it fails. Have you had any similar experiences?

Yes, its works like a charm 👍🏻

But sorry, I use toybox for replacing some packages e.g file, findutils, coreutils, tar, sed, grep. diffutils and patch are still GNU because of compability (powerful).

In my concepts, I was trying to build with less GNU utils and libraries lol smile

I may start another repo with that as the purpose
I did rebuild llvmtools to replace Ninja with Samurai to drop libffi and python3. Definitely made llvmtools smaller:
python3+libffi+ninja: ~ 140Mb ( .tar.xz )
Samurai + no python3 + no libffi + no ninja: ~ 116Mb ( .tar.xz )

I did rebuild llvmtools to replace Ninja with Samurai to drop libffi and python3

Libcxxabi requires python. Definately need to add libffi + python3 to llvmtools

Looks like zlib-ng (with zlib-compat)  isn't compatible with perl?
Perl fails to build:
In file included from Zlib.xs:34:
/usr/include/zlib.h:1781:31: error: unknown type name 'gzFile'
Z_EXTERN int Z_EXPORT gzgetc_(gzFile file);  /* backward compatibility */
                              ^
/usr/include/zlib.h:1818:13: error: unknown type name 'gzFile'
   Z_EXTERN gzFile Z_EXPORT gzopen(const char *, const char *);
            ^
/usr/include/zlib.h:1819:45: error: unexpected type name 'off_t': expected identifier
   Z_EXTERN z_off_t Z_EXPORT gzseek(gzFile, z_off_t, int);
                                            ^
/usr/include/zconf.h:159:21: note: expanded from macro 'z_off_t'
#    define z_off_t off_t
                    ^
In file included from Zlib.xs:34:
/usr/include/zlib.h:1819:54: error: expected identifier
   Z_EXTERN z_off_t Z_EXPORT gzseek(gzFile, z_off_t, int);
                                                     ^
/usr/include/zlib.h:1820:37: error: a parameter list without types is only allowed in a function definition
   Z_EXTERN z_off_t Z_EXPORT gztell(gzFile);
                                    ^
/usr/include/zlib.h:1821:39: error: a parameter list without types is only allowed in a function definition
   Z_EXTERN z_off_t Z_EXPORT gzoffset(gzFile);
                                      ^
/usr/include/zlib.h:1840:35: error: unknown type name 'gzFile'
Z_EXTERN int Z_EXPORTVA gzvprintf(gzFile file, const char *format, va_list va);
                                  ^
In file included from Zlib.xs:743:
./constants.h:370:18: warning: incompatible pointer to integer conversion assigning to 'IV' (aka 'long') from 'void *' [-Wint-conversion]
      *iv_return = Z_NULL;
                 ^ ~~~~~~

Autoconf fails using OpenBSD M4 (Ibara's)
checking for a BSD-compatible install... /usr/bin/install -c
checking whether build environment is sane... yes
checking for a race-free mkdir -p... ./build-aux/install-sh -c -d
checking for gawk... gawk
checking whether make sets $(MAKE)... yes
checking whether make supports nested variables... yes
checking build system type... x86_64-pc-linux-musl
checking host system type... x86_64-pc-linux-musl
configure: autobuild project... GNU Autoconf
configure: autobuild revision... 2.71
configure: autobuild hostname... localh3art
configure: autobuild timestamp... 20210706T073551Z
checking for a shell whose -n mode is known to work... /bin/sh
checking for characters that cannot appear in file names... none
checking whether directories can have trailing spaces... yes
checking for expr... /usr/bin/expr
checking for GNU M4 that supports accurate traces... configure: error: no acceptable m4 could be found in $PATH.
GNU M4 1.4.6 or later is required; 1.4.16 or newer is recommended.
GNU M4 1.4.15 uses a buggy replacement strstr on some systems.
Glibc 2.9 - 2.12 and GNU M4 1.4.11 - 1.4.15 have another strstr bug.

 GNU Autoconf
configure: autobuild revision... 2.71
configure: autobuild hostname... localh3art
configure: autobuild timestamp... 20210706T073551Z
checking for a shell whose -n mode is known to work... /bin/sh
checking for characters that cannot appear in file names... none
checking whether directories can have trailing spaces... yes
checking for expr... /usr/bin/expr
checking for GNU M4 that supports accurate traces... configure: error: no acceptable m4 could be found in $PATH.
GNU M4 1.4.6 or later is required; 1.4.16 or newer is recommended.
GNU M4 1.4.15 uses a buggy replacement strstr on some systems.
Glibc 2.9 - 2.12 and GNU M4 1.4.11 - 1.4.15 have another strstr bug.

fixed by defining M4
M4=/usr/bin/m4 ./configure --prefix=/usr

But.. build fails caused by autom4te that still needs GNU M4

https://github.com/openbsd/ports/tree/master/devel/autoconf/2.71

https://www.gnu.org/software/autoconf/manual/autoconf-2.69/html_node/Why-GNU-M4.html

But.. build fails caused by autom4te that still needs GNU M4.

Looks like autom4te has to be patched for Ibara's M4

I have not gotten around to porting gettext-tiny from simple Makefiles to a build system generated by GNU's autotools. The current source of gettext-tiny does not use libtool... i think i ran into a problem 1-2 years ago where a package required some sort of library archive or file made from libtool with gettext-tiny source.
I may reconsider using a different build/configure system if autotools are hard coded for GNU's autotools (M4, autoconf, automake, etc.)

I'll move this to discussions


The page:finish hook gets called before onMounted only when hash present in path

Environment

Operating System: Darwin
Node Version:     v18.12.1
Nuxt Version:     3.4.3
Nitro Version:    2.3.3
Package Manager<EMAIL_ADDRESS>Builder:          vite

Reproduction
Inside the app.vue file I have the following:

<template>

    <NuxtPage class="flex-1" />

Demo section

</template>

Describe the bug
I'm trying to build a reveal animation when the page is loaded, so I usually create the animation timeline with GSAP inside the "onMounted" hook that starts paused and only plays once the page is fully loaded (the page:finish hook).
Everything works perfectly fine if I navigate to any page of the Nuxt app and reload it, except for reloading in pages with a hash in the path AND an actual element with the hash as an id  (for example: http://localhost:8080/#about). For some reason, if the route path contains a hash of an existing element in the DOM, the page:finish hook gets called before onMounted. However, if I remove the hash from the URL or there's no element present in the DOM that matches the hash, the order goes back to normal (onMounted first and page:finish second).
I don't know why that happens.
Additional context
No response
Logs
No response

I couldn't reproduce the issue on other projects nor the starter reproduction template you provide. I will try to figure out what I might be doing wrong.


Add iPad and Catalyst support for Config quickstart

This is what the Catalyst app looks like:

See here

Please add Catalyst to the CI as well.


Input type time won't animate the label correctly.

As written above.

Same for types "date", "datetime-local", "time" and "week".

As a temporary workaround add the "active" class to the label and it'll at least not block the user's view of the input.

I experimented a bit and added the following code to style.css as a workaround:
.input-time label{     font-size: 0.8rem;     -webkit-transform: translateY(-140%);      transform: translateY(-140%); }
which is the same css code that for .input-field label.active{}
So basically the same that adding "active" class

hi dani,
i add style.css which codes, but not working.
.input-time label{ font-size: 0.8rem; -webkit-transform: translateY(-140%); transform: translateY(-140%); }


adding base tag in template

adding template and base tag

Maybe add back the lines from the old index.html:
<meta name="viewport" content="initial-scale=1, maximum-scale=1">
<meta name="keywords" content="preact, material components, preact components">
<meta name="description" content="preact-material-components is a thin opinionless wrapper around material-components-web. Use these components to add material components to your web app, without worrying about the extra code of the the components which you are not using.">
<link rel="shortcut icon" type="image/png" href="favicon.png">
<link rel="stylesheet" href="https://fonts.googleapis.com/icon?family=Material+Icons">


change requirements.txt

flask>=0.12.3
nltk
tflearn
tensorflow

this lets the bot run on almost all the Python versions without causing any problems
tensorflow==1.13.2 wont install on Python 3.8.8
json, pickle are already there and cant be downloaded from PyPi... at least I couldn't find them

Thanks, @brewwmannsna for checking that. Would you send a pull request with the changes?

Pull requested and merged here: #8
Thank you!


switch to the released mcloud v2.4.1

subj

done. need to install CE from (develop or master branch) and do basic acceptance

Verified with CE+ec2.
Tested manually and automated behaviour.


APP-3187: Spring boot auto property expansion strategy for gradle project

Ticket
APP-3187
Description
The legacy strategy for expanding property (@symphony.apiclient.version@) only works with
Maven project. Since we decided to use gradle instead, this expansion haven't worked anymore.
That's why we have to update the properties file with the gradle auto property expansion.
More details: https://docs.spring.io/spring-boot/docs/1.4.x/reference/html/howto-properties-and-configuration.html#howto-automatic-expansion-gradle
Dependencies
List the other pull requests that should be merged before/along this one.
Checklist

[x] Referenced a ticket in the PR title and in the corresponding section

Could we verify it through unit test?

I can only test it with a real bot created with the bdk project. I couldn't find any way to write unittest for this one because it's related also with the gradle build.


Updated settings files. Fixed timezone support.

I renamed the SQLite file to Django's default. Is this a problem? I think not. Users with an old db and settings won't have problems. Users with a new settings should look into it and change entry or filename. What do you think?

At the moment, we have a settings that forces to use Django 1.10.x. Should we revert some changes to support Django 1.9? I think we do not need this support but the change would not be very hard to do, of course.

Tested:

branch with fresh personal_data dir: ok

Run 2.0 to create a last stable personal_data dir (with settings.py). Then switch to this branch. Upgrade requirements. Start server (which migrates the db). Alle done successfully. Only a django warning while starting:

?: (1_8.W001) The standalone TEMPLATE_* settings were deprecated in Django 1.8 and the
TEMPLATES dictionary takes precedence. You must put the values of the following settings
into your default TEMPLATES dict: TEMPLATE_DEBUG.

Just for notice: This warning can be removed be removing the unused lines in settings.py: TEMPLATE_DEBUG and TEMPLATE_DIRS.
Manually tests in OpenSlides 2.1-dev works successfully. So ready for merge.


Add CLBlast, a wrapper for the OpenCL BLAS library

No spaces in tags next time please :)

Arg, no problem!
On Feb 18, 2018 10:51 PM, "Dominik Picheta"<EMAIL_ADDRESS>wrote:

No spaces in tags next time please :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nim-lang/packages/pull/654#issuecomment-366551072,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVr1jajxzIzK5oQs2PMt14bDSR1A-q8Yks5tWJtYgaJpZM4SJeyo
.


Error with RN 0.57

What is the problem, can you describe what is incorrect on the screenshot?

I believe you are running into this TypeScript issue: https://github.com/Microsoft/TypeScript/issues/27421


fix: MD031/blanks-around-fences

Fenced code blocks should be surrounded by blank lines

Under discussion on #11784.
I go with ...
* Item 1

  '''csharp
  some code
  '''
* Item 2
* Item 3

... to keep vertical margin (or padding) from being added to all list items. However, that should be checked because my check on the rendering difference was done last year. DocFX/build sys may be doing something different these days (i.e., not adding a lot of extra vertical space to all list items).

I don't think it affects the virtical margin, since looking at the code it renders out to the following on docsmsft

Replace UseIdentity with UseAuthentication in the Configure method:

C#
			<button class="action" data-bi-name="copy">

				Copy
			</button>

app.UseAuthentication();

The GitHub render is also similar. EX: closing
 and then the closeing
 in both cases

You can't see the classes tho, and one would need to look at both cases (line + no line) with markup+styles. It's actually quicker to just perform an internal preview test both ways ... that will settle it.
We're all a bit 🏃🏃🏃😅 at the moment tho with priority work (e.g., VS2019-related doc updates). They may want to revisit this later.
I recommend sitting on this for a bit until we can find out what's what. They'll let us know what they want to do.

@guardrex not worries, I've rebased out the conflict. The remaining code fence ones still have a bug till the next VSCode and CLI version of Markdownlint is released.


[r] Update DESCRIPTION and NEWS.md for 1.5.2

1.5.2 will deliver a Python-only bugfix on the release-1.5 branch. R is not affected, other than our commitment to keep Python and R API versions synchronized.
Please also see https://github.com/single-cell-data/TileDB-SOMA/wiki/Branches-and-releases

See code comments and also https://github.com/single-cell-data/TileDB-SOMA/issues/1818

[sc-38306]


Shade / Relocation of classes support

I couldn't find any docs nor part of the source code to do this. Any chance for a write-up or wiki on this? Built-in JarJar support for assembly would be nice! Much appreciated! Thanks for an amazing build tool.

closing due to inactivity

I know of others that would very much appreciate this feature (myself included). @lefou, @lihaoyi Is this marked wontfix only due to inactivity, or are there design decisions not to allow this? I'd be happy to attempt it if no one else wants to.

Just due to inactivity
On Sat, 1 Jun 2019 at 6:23 AM, Nik Vanderhoof<EMAIL_ADDRESS>wrote:

I know of others that would very much appreciate this feature (myself
included). @lefou https://github.com/lefou, @lihaoyi
https://github.com/lihaoyi Is this marked wontfix only due to
inactivity, or are there design decisions not to allow this? I'd be happy
to attempt it if no one else wants to.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/lihaoyi/mill/issues/355?email_source=notifications&email_token=AE5HBDBY6QRY5P7QG66A5IDPYGQPXA5CNFSM4FCMEUG2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWWQK5Q#issuecomment-497878390,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE5HBDDONG6WOZKVKDUCJJDPYGQPXANCNFSM4FCMEUGQ
.

@nvander1 I was going through old issues and pull requests and tagged them with milestone versions or labels to better indicate if implemented or not. Feel free to work on it.

I'm looking for this feature!


Problem_Statement_77

The sum of the squares of the first ten natural numbers is:
1 * 1 + 2 * 2 + ... + 10 * 10 = 385
The square of the sum of the first ten natural numbers is:
(1 + 2 + ... + 10) * (1 + 2 + ... + 10) = 55 * 55 = 3025
The difference between the sum of the squares of the first ten natural numbers and the square of the sum is:
3025 − 385 = 2640
Find the difference between the sum of the squares of the first n natural numbers (n > 0) and the square of the sum, and then return the sum of the square of its digits.
e.g 2640 --> 2 * 2 + 6 * 6 + 4 * 4 + 0 * 0 = 56

I am Anuja Limaye, I want to work on this issue, please assign this issue to me.

I am Mrunal Jambenal SY Comp A , CCEW Pune. I would like to work on this issue.

You have been assigned this problem statement. All the best.

hello, I am arham. I would like to work on the problem. please assign it to me

I am Shreya Amritkar from SY IT CCEW. Please assign me this issue


Refactor raw SQL generation to avoid direct param insertion

Return a tuple of query string and param list from Form SQL filter
constructor rather than returning a query string with the params
inserted. Also return a tuple from View SQL filter constructor
to avoid mutation of param list variable.

retest this please

retest this please

Refactoring is up for discussion. If you don't like it, can make a separate PR for the bug fixes.


Removed unnecessary data sorting before plotting

We had to sort data when we used Cassandra because it corrupted the order of rows but with Postgres there is no such a problem

We won't need these changes when we merge: https://github.com/lycantropos/alcor/pull/38 and https://github.com/lycantropos/alcor/pull/37


Speakers Page Fixes [ Do Not Merge ]

[x] Responsive Popovers
[x]  Responsive Overlay
[x] 1:1 ratio for images
[ ] Html Rendering description

Current coverage is 47.26% (diff: 100%)

Merging #611 into development will not change coverage

@@           development       #611   diff @@
=============================================
  Files                5          5
  Lines              548        548
  Methods             52         52
  Messages             0          0
  Branches            88         88
=============================================
  Hits               259        259
  Misses             289        289
  Partials             0          0

Powered by Codecov. Last update a8c7a10...48f02c7


fix: don't install kuberhealthy by default

kuberhealthy isn't working in current EKS versions

/cc ankitm123 tomhobson babadofar


networking/wireless: Restart service on wpa_supplicant.conf changes

Description of changes
#180872 accidently made the wpa_supplicant.service no longer restart when changes to the generated wpa_supplicant.conf are made.
Previously the script referenced the config directly, but #180872 made the connection indirect through /etc/wpa_supplicant.conf.
This PR adds the generated config to restartTriggers, to restore this functionality.
Using reloadTriggers was attemtped, but failed with: Failed to reload wpa_supplicant.service: Job type reload is not applicable for unit wpa_supplicant.service.
Things done

Built on platform(s)

[X] x86_64-linux
[X] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[X] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[X] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

As per my comment, this change was a mistake (I didn't really think it through), so it would be better to just revert 89eb93dc3ffda68535d6ee51a7825f1de31ae658 for the time being.

Reverted


Swap deprecated GCS lib with replacement

What does this PR change? What problem does it solve?
It swaps the deprecated GCS library (google.golang.org/api/storage/v1) with the replacement (cloud.google.com/go/storage).

Was the change discussed in an issue or in the forum before?
no

Checklist

[x] I have read the Contribution Guidelines
[x] I have enabled maintainer edits for this PR
[ ] I have added tests for all changes in this PR
[ ] I have added documentation for the changes (in the manual)
[ ] There's a new file in changelog/unreleased/ that describes the changes for our users (template here)
[x] I have run gofmt on the code in all commits
[x] All commit messages are formatted in the same style as the other commits in the repo
[x] I'm done, this Pull Request is ready for review

@gottwald Please squash the commits. Did you manually test that the GCS backend works with these changes? If yes, please also test -o gcs.connections=1. Does the GCS library upgrade provide new noteworthy features? Or is it just updating to the current library?

@MichaelEischer Squashed and rebased on master.
I ran all gs tests and benchmarks with the env vars to make it test against a real bucket in my cloud account.
I also did manual tests of a backup, restore, listing snapshots and deleting snapshots.
And now also the same manual testing set with the -o gcs.connections=1 option.
Regarding noteworthy features: I'm not aware of any.
The main difference AFAIK is that the old one is code generated and now deprecated and the new one is written by hand and a bit nicer to use.
Also did not see any significant difference in performance according to the benchmarks.


Error cloning Vertex with Edge to self - exception thrown attempting to remove same Edge twice

Attempting to createGraphCloneVertices where one or more Vertexes has an Edge to itself throws Fhaculty\Graph\Exception\InvalidArgumentException: Invalid Edge does not exist in this Graph

Thanks for reporting! It looks like this may be a duplicate of https://github.com/clue/graph/pull/149 which was fixed a while ago, but still didn't get a tagged release so far.
Also thanks for your other contributions the last weeks, I'll get to this as soon as possible! :+1:  Your contributions are very much appreciated and I've now put this higher on my todos so I'm planning to get back to this and update the whole "graphp" organization starting next week :shipit:
Just reaching out, but I'm seeing some high quality PRs and input coming from you, so if you're interesting in helping out with maintaining this project, please drop me a line via email :heart:


Fix: Push notification onyx updates not applied when notification is selected

@arosiclair
Details

Fixes an issue where OnyxServerUpdate's received from push notifications are not applied to Onyx locally, when a notification is selected/clicked.
Based on https://github.com/Expensify/App/pull/42044 because it isn't merged yet
Fixed Issues

$ https://github.com/Expensify/App/issues/40227
PROPOSAL:
Tests

[ ] Verify that no errors appear in the JS console

Offline tests

QA Steps

[ ] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[ ] Android: Native
[x] Android: mWeb Chrome
[ ] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

@arosiclair i've updated the PR to use DeferredOnyxUpdates.enqueueAndProcess() directly, which will make sure the updates are sorted and applied synchronously.
Could you test if this brings the expected output when clicking on push notifications? It's hard to test this on the simulator (as i cannot build the Expensify app with (Remote) push notifications, because i'm not in the Expensify Apple Developer organization)
I'm gonna try to mock the push notification with the onyx payload later today, but first i have to work on some other stuff.

Could you test if this brings the expected output when clicking on push notifications? It's hard to test this on the simulator (as i cannot build the Expensify app with (Remote) push notifications, because i'm not in the Expensify Apple Developer organization)

You should be able to test on Android (emulator or device are both fine). I'll give it a test on iOS though.

@chrispader Can you please update the checklist?

@arosiclair just applied the changes you suggested and tested this on Android with the notifications from staging and it work's just fine. Haven't experienced any delays and the report (including the updates from the notifications) loads immediately.
Could you confirm this also works on iOS?
cc @allroundexperts (updated the checklist now)

@chrispader Can you please update the checklist?

@allroundexperts i completed the checklist so this PR is not blocked by me, but we still have to ensure this works on Android and iOS

Calling Airship.push.getActiveNotifications() actually doesn't give me any notifications on Android, could you check if this returns any notifications on iOS?

It should work on both. That's what we use for clearReportNotifications. I'll give it a try on Android.

Also, we still have the PushNotification.onReceived callback which will apply all (reliably) applicable updates when received. This means that the updates that are clicked/selected should already be applied at that point.
Is our approach, to "re-apply" all open/active notifications, in case these couldn't be applied (yet) or are missing?

The idea is that sometimes the onReceived callback does not run and apply the onyx data after the app is killed in the background, so we would try to apply the onyx data again when you select any notification. However, I am seeing on iOS that the onReceived callback does eventually run when we re-open the app so we may be double applying right now. I'm going to do some more testing to see how we should proceed.

Since i cannot really test this on iOS, where it seems to be the bigger problem, i'm not sure if i can properly test and work on this PR...

Yeah it might be best for me to take over this PR. I'll put some work on it today

Calling Airship.push.getActiveNotifications() actually doesn't give me any notifications on Android, could you check if this returns any notifications on iOS?

It should work on both. That's what we use for clearReportNotifications. I'll give it a try on Android.

Also, we still have the PushNotification.onReceived callback which will apply all (reliably) applicable updates when received. This means that the updates that are clicked/selected should already be applied at that point.
Is our approach, to "re-apply" all open/active notifications, in case these couldn't be applied (yet) or are missing?

The idea is that sometimes the onReceived callback does not run and apply the onyx data after the app is killed in the background, so we would try to apply the onyx data again when you select any notification. However, I am seeing on iOS that the onReceived callback does eventually run when we re-open the app so we may be double applying right now. I'm going to do some more testing to see how we should proceed.

Since i cannot really test this on iOS, where it seems to be the bigger problem, i'm not sure if i can properly test and work on this PR...

Yeah it might be best for me to take over this PR. I'll put some work on it today

thanks!

I'm merging this into my own branch and continuing from there


All commits related to the demo3 deployment of the IDR

This is my branch with all changes planned for the IDR demo3 release. This is or will be split into separate PRs.
Issues

[ ]

Todo

[ ] Update to latest version of https://github.com/openmicroscopy/infrastructure/pull/155/files (redis://)

Conflicting PR. Removed from build MANAGEMENT_TOOLS-merge#1047. See the console output for more details.
Possible conflicts:

PR #150 manics 'Add external munin roles and idr munin playbook'

ansible/vendor/munin
ansible/idr-playbooks/management.yml

PR #151 manics 'Nginx idr caching config'

ansible/idr-playbooks/idr-omero.yml
ansible/idr-playbooks/group_vars/proxy-hosts.yml

PR #153 manics 'Use docker swarm with IDR JupyterHub'

ansible/idr-playbooks/idr-docker.yml
ansible/idr-playbooks/files/jupyterhub-docker-swarm.j2

PR #155 aleksandra-tarkowska 'upgrade django-redis'

ansible/idr-playbooks/files/IDR-OMERO-52-omero.j2

PR #157 manics 'omero user role'

ansible/roles/omero-user/README.md

could we add cookie name omero.web.session_cookie_name sessionid_idr030 to avoid clashes?

@aleksandra-tarkowska I've made the idr_environment var part of the cookie name: https://github.com/openmicroscopy/infrastructure/pull/159/commits/003046e91a030ed9cba13ae963bb0f75f03eb8eb

:+1: for cleaning up now inoperable playbooks.


PageView entering animation

Hi,
It seems that PageView by default animates the items from right to left when the widget first appears on the page. I'd like to remove this animation, is it possible ?
thanks in advance,

Are you using PageController ? If so just set the initialPage to 1

After some debug just found that the issue was that I was sending -1 as the initialPage value.


AWS add gRPC ruby tests.

example result: https://source.cloud.google.com/results/invocations/84bc21c8-0e7b-42f8-b374-e7d43000bf61/log

superseded by https://github.com/grpc/grpc/pull/26349


Updated some system awaiters to use the Awaiter.WaitUntil extension

Overview

Added an input system validation method in the input system global listener.
Added a teleport system validation section in the base controller pointer.

TODO:

[ ] Needs to be tested.

@SimonDarksideJ were you able to test this?

@FejZa  can you test this as my HL is in the shop :D
Would love to see this cleared

I'm actually more interested in the tests on the VR headsets

Testing this tomorrow guys!

@StephenHodgson could you give me some details on what exactly needs to be tested? Or how I am supposed to know whether it works or not? Thank you :)

Mainly if it continues to work on the WMR platforms (HL1 and WMR VR Headsets)

Really there shouldn't be any change in behavior. So if there is then there's a problem.

OK, going to retest and review this again today so we can get it cleared


Support insert_id to avoid duplicate events

I want to have an insert_id as amplitude API says it can deduplicate subsequent events sent with an insert_id we have already seen before within the past 7 days.
https://help.amplitude.com/hc/en-us/articles/360032842391-HTTP-API-V2
But I didn't see any insert_id in the source code, only see uuid in the sending event. Does it support insert_id or any others to avoid duplicate event.

Hello, we use uuid to do de-duplication, and uuid is generated automatically.


Remove any remaining class names

A few styled components use pseudo-classes and can be removed (i.e. SocialButton)

Components to change:

[ ] Contact
[ ] EditInfoPage
[ ] OauthSignInButton
[ ] OrderConfirmation
[ ] PlasmidDetail
[ ] PlasmidTable
[ ] ShoppingCart/index
[ ] StrainDetail
[ ] StrainTable

These have all been removed in refactor/update-styling and will be merged into develop after further testing.


Only allow modifying users with donation in last 10 min

I made a few comments but I wanted to say that it's great to see such thorough tests! 👍


(typeof null === 'object') is true

node == null is causing unexpected exception at node['_type'] in function task.

@danielnorberg can you review?

:+1:

👍


rollback retest action

cc @arkodg
@tmsnan PTAL, you're the original author.

LGTM, can you raise a follow up so we can re enable in the future

we can reuse https://github.com/envoyproxy/gateway/issues/2339


Refactor events documentation

This rewrites the events documentation to consolidate it and make it more clear.
I'm also sneaking in two additions to the API reference documentation as well for Rate Limits and Errors which were not documented.
Would love your feedback.
TODO:

[x] Create redirects before merging
[x] Link to the AWS event destination guides after they merge

@mkarnowski any other feedback other than spelling? would love qualitative feedback on whether this communicates well / structure / things that are missing / etc
(same goes for @nijikokun)
@jrobsonchase @warrn @jmhertlein curious for your thoughts as well as folks who have worked on the events system!

@inconshreveable the consolidated layout flows well and i find it easier to read and find relevant information.  Sets up well for the addition of new event classes; for example audit events such as tunnel_up/down, user_login/logout and api calls.

alright i think this is ready to go!

@inconshreveable the consolidated layout flows well and i find it easier to read and find relevant information. Sets up well for the addition of new event classes; for example audit events such as tunnel_up/down, user_login/logout and api calls.

thank you! makes sense although when we get to it, thoughts on if we can find another name (or names) for those events instead of 'audit events'. because i feel like all of the configuration events are also audit events?

can i get an approval on this? (ty!)


misssssssssssssssss

Describe your problem here.
I made a miss animations and a normal animations (idle, singUP, etc) but the miss animations would play after the normal animations (except idle of course) is tjis a bug or am I doing it wrong
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No (other than the accuracy 'ratingStuff'), just modded in the "example_mods" folder

add a 0 after the xml name for the animation

add a 0 after the xml name for the animation

what do you mean? the xml name? the animation name in the xml? the image path? the animation name im entering in the the animation in the character editor in the animations tab? is the miss animations supposed to be in a defferent sprite sheet?

In the engine

so basically, if the main anim on the xml is called "left' and the miss is called "left_miss" then it plays them both, idk why. just make the names on the xmls different

In the engine when you were to put the .xml anim field just put a 0 after
it like for example: left0
Em sáb, 23 de jul de 2022 19:55, Davesunavaiable @.***>
escreveu:

so basically, if the main anim on the xml is called "left' and the miss is
called "left_miss" then it plays them both, idk why. just make the names on
the xmls different
—
Reply to this email directly, view it on GitHub
https://github.com/ShadowMario/FNF-PsychEngine/issues/9878#issuecomment-1193200066,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYFVJYH7RTKALGLGRNZQ5RDVVRZ55ANCNFSM54ORBPWA
.
You are receiving this because you commented.Message ID:
@.***>

naming it that just made it wors

e

oh yeah Im ADDING miss animations to a character (that I made)

naming it that just made it wors  e

I meant on the singing anim XML name in the field in character editor

naming it that just made it wors  e

I meant on the singing anim XML name in the field in character editor

so the name in the fla dosen't matter so in the engine update singLEFT to left0 and the miss anim left_miss

Not that way
Em sáb, 23 de jul de 2022 20:15, TFLTV @.***> escreveu:

naming it that just made it wors [image: Web
capture_23-7-2022_1841_github com]
https://user-images.githubusercontent.com/105890618/180625582-b3025eb8-f1c3-4291-aa10-d10e71de8f12.jpeg
e
I meant on the singing anim XML name in the field in character editor
so the name in the fla dosen't matter so in the engine update singLEFT to
left0 and the miss anim left_miss
—
Reply to this email directly, view it on GitHub
https://github.com/ShadowMario/FNF-PsychEngine/issues/9878#issuecomment-1193202883,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYFVJYGWGSWD2BEEP3XRQULVVR4J7ANCNFSM54ORBPWA
.
You are receiving this because you commented.Message ID:
@.***>

what good information

Sorry for taking too long but like, the animation name for singing
animations are singLEFT or the other animations.
I meant as the xml field inside the game (in character editor) not on the
fla file itself
Em sáb, 23 de jul de 2022 20:34, TFLTV @.***> escreveu:

what good information
[image: Web capture_23-7-2022_18261_github com]
https://user-images.githubusercontent.com/105890618/180626120-b8c0a0d0-adfa-465d-bff8-cc44413c5562.jpeg
—
Reply to this email directly, view it on GitHub
https://github.com/ShadowMario/FNF-PsychEngine/issues/9878#issuecomment-1193206231,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYFVJYBTC76CI44LL4H6DJLVVR6QDANCNFSM54ORBPWA
.
You are receiving this because you commented.Message ID:
@.***>

is an easy solution, here it is

@BlearChipmunk youre misunderstanding.
He’s asking how to ADD miss animations he's not saying there’s a BUG

@TFLTV in the animation name, if you want the left miss to singLEFTmiss in the animation name

what does this mean?

what does this mean?

Look at the bf character data to easily figure that out


EventEmitter memory leak

Environment details

OS: Debian 9.11
Node.js version: 10.16.3
npm version: 6.9.0
@google-cloud/pubsub version: 1.0.0

Steps to reproduce
Run the following sample program with GCP project myproject and Pub/Sub topic mytopic set up and observe warnings like:
(node:22056) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 close listeners added. Use emitter.setMaxListeners() to increase limit
const projectId = 'myproject';
const topicName = 'mytopic';
const {PubSub} = require('@google-cloud/pubsub');
const pubsub = new PubSub({projectId});
const topic = pubsub.topic(`projects/${projectId}/topics/${topicName}`);

async function sleep(ms) {
       return new Promise(resolve => {
               setTimeout(resolve, ms);
       });
}

function publish() {
       const data = Buffer.from(JSON.stringify({date: Date.now()}));
       topic.publish(data, function(err, mId) {
               console.log(err, mId);
       });
}

async function main() {
       for (let i = 0; i < 1000; i++) {
               await publish();
               await sleep(20);
       }
}

main();

Watching this program run for a while it indeed grows to 300 MB resident memory quickly. Is there indeed a memory leak, and if yes, how can it be fixed?
Also tested this with older pubsub versions like 0.30.0, 0.29.0, etc. and newest release of Nodejs 12, same issue. Is this indeed a bug in the library or related to my code?

👋 @amsibamsi we've been seeing this issue pop up across several of our Node.js clients, I believe this is a grpc issue (https://github.com/grpc/grpc-node/issues/1027) which may or may not be resolved already.
Can you check to see if<EMAIL_ADDRESS>is installed? If it isn't, you may need to delete any lock files you might have and/or re-install your dependencies.

Thanks, this indeed fixed the problem.

@amsibamsi awesome, thanks for letting us know!

@callmehiphop wouldn't it be appropriate to upgrade the dependency in package.json to "@grpc/grpc-js": "^0.5.4", then?

@ctavan yep, absolutely! We'll need to make that update across the board


[Arch] Test EventStreamRuntime to ensure its feature parity with ServerRuntime

What is the problem that this fixes or functionality that this introduces? Does it fix any open issues?
Finish #2404.
We are almost done with the arch refactor for Runtime (https://github.com/OpenDevin/OpenDevin/issues/2404). Right now we need to make sure the new runtime has feature parity with existing default ServerRuntime before we are ready to switch over to the new architecture.

Give a summary of what the PR does, explaining any non-trivial design decisions

Move all the remaining test cases from test_sandbox to test-runtime
Support run as arbitrary user for EventStreamRuntime (including Bash and Jupyter) and add test case
Set a dedicated POETRY_VIRTUALENVS_PATH=/opendevin/poetry so it can be used across different users
Support PWD updating for EventStreamRuntime's Jupyter
overload add_env_vars for EventStreamRuntime to also update env var in Jupyter;
make agentskills read env var lazily, in case env var is updated in Jupyter (only the case in EventStreamRuntime).
remove test_sandbox
merge all tests into ONE ghcr.yaml workflow

Other references

@xingyaoww a thought about this: https://github.com/OpenDevin/OpenDevin/blob/235525c9dd982c43d53b9a6db6b5d667e20fe4e7/opendevin/runtime/client/runtime.py#L213
I think this line shouldn't be commented, because we need cause set for the actions that result in obs. I see it's set in the called method now, but I'm not sure it's enough: that way it still doesn't capture all possible obs, it seems to miss some error obs in that method, where the code returns directly. Why not set it for all, as it was before? Not linking them via cause will give some issues, in two places: agent controller won't reset the pending action, and history won't identify correctly the pairs (so eval trajectories will probably be a little weird for those). Is that intended?
On the other hand, the Runtime sets a _parent attribute, which is not used anywhere.
Is there supposed to be a difference between parent action of an obs, and cause action of an obs? If they mean the same thing, the action triggering the observation, maybe we can use just one, not both.

@enyst very good point! i've removed the "_parent" and uncommented "_cause". I think I copied the old version of code where these are not implemented properly. Will update this PR after the test CI finished (or errors out).


Increment uamqp version from 1.2.0 to 1.2.5

Increment UAMQP dependency version to 1.2.5 to push out a series of key fixes (segfault, large messages)

Can one of the admins verify this patch?


Control Objective - Alarm on VPC Route Table public routes

No issue linked

closes https://github.com/turbot/tdk/issues/300


parsing of non-double coordinates

Thank you for your work. It is really helpful, but I have some geojson data which sometimes contains coordinates like [10,47.7875]. The first value is not a double value; therefore, the casting e.g here: https://github.com/jozes/flutter_map_geojson/blob/836d61835d9e178362c256f085ba028d0fb6235e/lib/src/geojson_parser.dart#L279 results in the exception:
type 'int' is not a subtype of type 'double' in type cast
Expected behavior:
Correct parsing of non-double coordinates.

As per RFC 7946 for geojson  (see https://datatracker.ietf.org/doc/html/rfc7946#page-11) the point (position) is specified with decimal numbers.
A position is an array of numbers.  There MUST be two or more
elements.  The first two elements are longitude and latitude, or
easting and northing, precisely in that order and using decimal
numbers.  Altitude or elevation MAY be included as an optional third
element.
Of course it is possible to change the code and test for the type of numbers parsed: either they are decimal or integers. But this would significantly slow down the parser and that is not my intention. Besides we would be supporting something what is not according to the specification. For me, a better solution for your problem is to write a preprocessor which would actually convert all integers into decimal (double) numbers.


address summary should use printable name

Currently when the shipping address is displayed in e.g. the checkout complete 'thank you' page it uses the full official name of the country.
For UK this comes out as "United Kingdom of Great Britain and Northern Ireland".
This looks a bit weird, arguably the address summary should use printable_name instead, which in this case would read "United Kingdom"

I agree. Thank you for the PR! I will merge, and then fix the failing tests.


--no-sandbox explanation

I noticed that when the browser launches to log in I get the warning: "You are using an unsupported command-line flag: --no-sandbox. Stability and security will suffer".
I've always understood this to be a very bad thing for security. I imagine this is necessary for the authentication functionality to work? I'd appreciate it if someone could expound on why that is.
Might it make sense to put a warning about this in the README/docs? Maybe its fine if you just use that browser instance for ChatGPT only, but it might be helpful to inform users to make sure they don't do anything else in that window unless they're feeling very lucky. =)
Thanks folks!

Hi there - good point. I tried various flags, however all of them showed somewhat similar warnings. I believe it is expected since it's automated. This may be something I could add to the Readme or PRs appreciated. Note: the browser will automatically close when you close your vs-code instance and get minimized while in-progress for chatgpt conversations. So the expectation is to use the chrome instance is only reserved for vscode-chatgpt integration.

I added a disclaimer in the readme for better clarity. Thanks again for bringing this up! @cjohnson-bf


Lattice dashboard view

Fixes #68

Some screenshots:
All hosts shown

One host selected (host info card appears)

Metrics tab


Docker security scan

A test run gave me the following the "Build for scan" step:
buildx failed with: ERROR: invalid tag "ghcr.io/ScilifelabDataCentre/form-manager:sha-fa18438afff58927c5411204a0d9d70657358be2": repository name must be lowercase

A test run gave me the following the "Build for scan" step:
buildx failed with: ERROR: invalid tag "ghcr.io/ScilifelabDataCentre/form-manager:sha-fa18438afff58927c5411204a0d9d70657358be2": repository name must be lowercase

I guess this is due to ScilifelabDataCentre?

Yes.

2022-08-29T07:58:19.829Z	FATAL	image scan error: scan error: unable to initialize a scanner: unable to initialize a docker scanner: 4 errors occurred:
	* unable to inspect the image (ghcr.io/scilifelabdatacentre/form-manager:sha-9ee9f8ddb17d229e6a2d513a452a81219f45022d): Error: No such image: ghcr.io/scilifelabdatacentre/form-manager:sha-9ee9f8ddb17d229e6a2d513a452a81219f45022d
	* unable to initialize Podman client: no podman socket found: stat podman/podman.sock: no such file or directory
	* containerd socket not found: /run/containerd/containerd.sock
	* GET https://ghcr.io/token?scope=repository%3Ascilifelabdatacentre%2Fform-manager%3Apull&service=ghcr.io: DENIED: requested access to the resource is denied


DOC: More clearly describe foreground/background

The generalalization of "binary" as '0' and 'not zero'
is not used in the "Binary[Dilate|Erode]ImageFilter.
Images with values of 0 or 1 require an explicit setting
of the DilateValue or ErodeValue to 1 in order to
designate 1 as the foreground value to be dilated/eroded.
Expand the documentation to more forcefully convey this
important consideration.

PR Checklist

[X] :no_entry_sign: Makes breaking changes
[X] :no_entry_sign: Makes design changes
[X] :no_entry_sign: Adds the License notice to new files.
[X] :no_entry_sign: Adds Python wrapping.
[X] :no_entry_sign: Adds tests and baseline comparison (quantitative).
[X] :no_entry_sign: Adds test data.
[X] :no_entry_sign: Adds Examples to ITKExamples
[X] :no_entry_sign: Adds Documentation.

Refer to the ITK Software Guide for
further development details if necessary.

For the erosion operation only the BackgroundValue matters, and for dilation only the ForegroundValue matters. It would be good if the updated documentation reflected this complementary nature of the operations instead of repeating the same text.

For the erosion operation only the BackgroundValue matters, and for dilation only the ForegroundValue matters. It would be good if the updated documentation reflected this complementary nature of the operations instead of repeating the same text.

@blowekamp The above statement is not strictly true.  I'm working on documentation to clarify the situation.
For dilate, "ForegroundValue" value is used to identify voxels that need changing & "ForegroundValue" is the value used for changed voxels.
For erode, "ForegroundValue" value is used to identify voxels that need changing & "BackgroundValue" is the value used for changed voxels.

For the erosion operation only the BackgroundValue matters, and for dilation only the ForegroundValue matters. It would be good if the updated documentation reflected this complementary nature of the operations instead of repeating the same text.

@blowekamp The above statement is not strictly true. I'm working on documentation to clarify the situation.
For dilate, "ForegroundValue" value is used to identify voxels that need changing & "ForegroundValue" is the value used for changed voxels.
For erode, "ForegroundValue" value is used to identify voxels that need changing & "BackgroundValue" is the value used for changed voxels.

For dilate: If any pixels in neighborhood are FG then the value is FG
For erode: If any pixel is neighborhood is not FG then the value is BG
So these operations are only duals of each other when the image are binary.
    That is to say that I assumed that an unsigned char image with values of 1 representing the mask could be dilated without explicitly setting the DilateValue (which I also incorrectly assumed was the size of the dilation).  I was trying to infer the behavior of the filter based on python introspection results rather than reading deep into the doxygen pages.
I went so far as to make a unit test and submit a bug report:   #1394
because I was convinced that there was a python bug in the wrapping.
   As soon as I found DilateValue, that triggered my memory.


ui-sref generates the right "href" path, but goes to another

In short, I use "ui-sref" in one of my views(lets call it "parent"). When I navigate to a view which is not nested in the "parent" view, everything goes well.
But then, I click on a link from the "parent" view (which is still open, thanks to the ui-router-extras plugin), and I get errors (because some of my stateParams have gone), although the "href" attributes of this link still shows the correct path (with the missing stateParams) and will render the right state if I open it in a new tab.
How can I make the href "override" the router behavior ?

Can you please provide a plnkr showing the bug

closing due to inactivity


Making the implementation of mmap/munmap/mremap/fixupFault more time-efficient

The time complexity of several methods in SE mode related to virtual memory page management is currently O(n), where n is the amount of pages. This is because gem5 keeps track of all currently mapped virtual memory pages in a simple linked list std::list<VMA> gem5::MemState::_vmaList, so whenever gem5 needs to find a page, it needs to linearly iterate over all VMA objects in the linked list. This operation is O(n) in time complexity and can happen in system calls including mmap, munmap and mremap. (It also happens when MemState::fixupFault is called for page faults in SE mode.) Therefore, for executables that map plenty of memory pages, the total overhead is unbearable (O(nn)).
I have worked out a solution to bring the time complexity down to O(logn) for this operation. The changes are confined to just a few files and are minimally disruptive. Would the community be interested in reviewing a PR for this fix?

Thanks for creating this issue, @FJShen!
To answer your question, when I'm reviewing code I think about two main things:

What is the impact of the change? In this case, how much performance improvement would we see from the change?
How much more difficult (or easier) will the code be to maintain over the next 10+ year?

If the impact is small, then it's required that the maintenance burden doesn't increase.
Feel free to push your change and we can review it!

Sounds good. I'm running unit tests right now, and I will create a PR once things are ready.
In terms of impact, I'd say it really depends on the workload :D In my case, I am executing an ELFie that maps ~1.6GB of 4k pages. The changes shaved off tens of minutes from the simulation.

10s of minutes seems worth it!
Please make the PR and we can have more discussion there.


Minor update in Galaxy 101 for everyone tutorial

Hi! this is my first contribution to the GTN Cofest!
Galaxy 101 for everyone is an awesome tutorial and really inspiring for newcomers I learned a lot with it!
These are the tiny changes that I made to fix #2400:

Replaced the old asterisks with checkbox to mark outputs as important in the workflow editor.
Renamed the Unique default output name to outfile to match the current name in the screenshots.

Nice, thank a lot!


Using this to extract archived files in Google Drive

Not an issue per se, but I'd like to know if this project is suitable for the task of extracting .rar files that're hosted on Google Drive -- so downloading from GD, extracting on Heroku, reuploading the unarchived file back to GD.
I've tried sorting it out myself but haven't been able to install rclone.  Any ideas? Is it not something that this is capable of? Sorry for the beginner questions.

Yup, PeaZip is installed in vubuntu. Follow the steps:

Download the RAR file.
Open file manager and locate the RAR file.
Right-click > Choose PeaZip
You can do any operations from GUI.
Also you can open PeaZip from options.
Options > Utilities > PeaZip

Rclone uses FUSE to make a temporary volume to mount google drive using block device, heroku dosen't allow any third-party devices.


fix serializable

suppose there are three transactions trying to execute in parallel.
the first one sets _waitOnCurrentFunction and starts running.
the other two await _waitOnCurrentFunction. when the first transaction finishes, the other two start running, without waiting for each other.
using a while loop is the standard pattern when waiting for a condition variable, so it seems fine to use it here.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Add comment or test? Fairly subtle behavior so I can picture myself inadvertently changing it back to an if without realizing the while is important


chore: upgrade github action versions

Summary of changes
Changes introduced in this pull request:

upgrade github action versions to suppress warnings like

Node.js 12 actions are deprecated. For more information see: https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/. Please update the following actions to use Node.js 16: actions/checkout, Swatinem/rust-cache, actions-rs/cargo, Swatinem/rust-cache, actions/checkout

@lemmih Do we want to have sth like this to automatically upgrade github action versions? It requires a github token with workflow scope tho.
Reference issue to close (if applicable)

Closes https://github.com/ChainSafe/forest/issues/2057
Other information and links

Do we want to have sth like this to automatically upgrade github action versions? It requires a github token with workflow scope tho.

Yes, that would be nice. You can use the ACTIONS_PAT token. It has the workflow scope.


Allow users to configure embed_batch_size or ThreadPoolExecutor size when calling Client.embed

It looks like batching was added in #437 - thank you for implementing this, it's very helpful.
I notice that batching, as defined here, depends on a fixed batch size. This can be suboptimal for clients submitting a large number of smaller documents, as we cannot configure the ThreadPoolExecutor size to parallelize a large number of small data payloads. As a result a client might end up blocking while waiting for small network responses.
Would it be possible to allow clients to configure either the ThreadPoolExecutor size or the embed_batch_size setting when calling embed?

Hey @acompa thanks for the feedback, this will be fixed by https://github.com/cohere-ai/cohere-python/pull/536 ! you will be able to pass your own executor in. I'll ping the thread when it's released


cargo doesn't respect the maximum number of jobs

Problem
When running cargo with an explicit -jn, or via GNU make with an explicit -jn, the number of jobs can actually exceed the limit given. Presumably, the same happens without -jn, but now that I have 32 cores/64 threads, I can't find a crate that saturates them.
Steps

clone the cargo repository
run cargo build -j4 -Z timings
open cargo-timings.html

You'll notice that at some point the number of active jobs goes over 4. This report (cargo-timing.zip) goes up to 7.
Notes
Output of cargo version:
cargo 1.42.0-nightly (86134e766 2019-12-23)

This is a known bug with the cc crate where it releases its implicit process token which can be acquired by something else to spawn more jobs, but I don't think this is an issue with Cargo so I'm going to close this.

This is a known bug with the cc crate

Do you have a link to an open issue?

There is not currently an open issue on the cc crate.

Should this one be redirected there, then, with more details about the "known bug"? Because it's not clear to me how cc releases its process token, and why that ends up bumping the number of active jobs in cargo.

Er well I'm pointing out that this issue you're reporting has nothing to do with Cargo itself, so this isn't the proper place for this issue. This is likely very related to the behavior of the cc crate, rather.
I don't really consider the behavior a bug on the cc crate myself so there's not an issue for it, so calling this a "known bug" was probably not a great usage of words on my part.

I still don't understand how cc messes with the number of active job...

@glandium There is more detail at https://github.com/rust-lang/cargo/issues/7689#issuecomment-563502323 and https://github.com/rust-lang/cargo/pull/7344#issuecomment-530184088.  The jobserver protocol is a very simple queue of tokens, and there is no protection that a client pushes extra tokens into the queue.  The cc crate is releasing its implicit token to simplify its implementation.  This will result in extra jobs, but they are not consuming CPU time since they are just waiting to reacquire their implicit token.


Duplicate entries in CHANGELOG

We should figure out the root cause of this and find a fix to avoid this in the future.

Fixed in https://github.com/xlayers/xlayers/commit/845b331de78e62b22e233a6f413edf167d570825


cmake: extensions: board revisions can be optional

Add a new option to board_check_revision that can make specifying a board revision optional. This makes it easier to work with boards that can come in a base variant with extensions. For example Fanstel BLE modules that come with/without power amplifiers.

What is the use case for this, surely the default for that board should be set i.e. the one without a PA, and a board revision used to specify the PA version instead?

What is the use case for this, surely the default for that board should be set i.e. the one without a PA, and a board revision used to specify the PA version instead?

Suppose you have a board with a footprint for one of these modules: https://www.fanstel.com/bt40f-nrf5340, https://www.fanstel.com/bt40nwithnrf21540, which are all pin compatible.
The base board could be board40, with a revision of board40@N if building for PA support. Sure you could specify a default instead, but what default do you choose? Any other letter is going to be misleading (and result in build folders like build/board40@A)

This I believe would be what you are after: https://github.com/zephyrproject-rtos/zephyr/issues/55637 though it is a long term goal, the idea being that you can have a board which is used as a base by other boards so you could have the base board itself as the development kit with the pin mappings and peripherals but no CPU, then have "extended" boards which include the base and add the desired CPU.
Would need to hear @tejlmand's comment on this PR

Updated with comments about interactions with EXACT and DEFAULT_REVISION.


Privacy-preserving settings for auto-loading twitter and youtube embedded media

Is your feature request related to a problem? Please describe.
I described the idea for this feature on SN here, largely to gauge interest. I wouldn't say there was much interest, but I figured I'd log the issue here anyway, in case the SN team agrees it's a valuable addition, and someone wants to implement it.
Basically, I think it would be nice to have a user setting that would allow you to disable making network requests to twitter, youtube, etc. by showing the embedded preview on posts that link to twitter, youtube, etc.
Similar to how SN moved the call to coinbase that populates the price ticker in the page header to server-side so as to not leak user's IPs to coinbase, this would give stackers a built-in mechanism to avoid loading resources from these other platforms by default.
Describe the solution you'd like
A user setting, enabled by default for all users, that controls whether the embedded previews for Twitter/X/YouTube links are rendered when viewing an SN item. Controllable in user settings page.
Describe alternatives you've considered
In the discussion on the SN post where I originally pitched this idea, there are alternatives described like browser extensions that can do this for you. It's also possible to do it via custom DNS like pi-hole and probably also VPNs. Despite all that, I think there's still value in having this supported within the SN platform itself. Perhaps I am alone in thinking this.
Additional context
N/A

Yep, I think it makes sense!


Website not receiving pastes

Hi, it seems 0bin.net is having a problem accepting new pastes ?

Yeah it's been broken for at least a few days now

Ok, no problem. Thanks for responding.
On Tue, Aug 16, 2022 at 6:00 AM zordtk @.***> wrote:

Yeah it's been broken for at least a few days now
—
Reply to this email directly, view it on GitHub
https://github.com/Tygs/0bin/issues/155#issuecomment-1216115851, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/A2QGSRL6T2XF2QD6USUKS3LVZMG4VANCNFSM56S2RVCA
.
You are receiving this because you authored the thread.Message ID:
@.***>

So we finally found the issue for the pastes not working.
The pastes folder was filled with file named like this:
0bin-fd470740.75887-1176577701172585282
Those files were empty, and we don't know what was producing them.
Was it somebody trying to force create paste with a specific name? Was it something on our side we did wrong and didn't notice?
Anyway, deleting those files fixed the problem. Now the hard part is going to find out the root cause and make sure it doesn't happen anymore.


Last interval line is not being painted

The farthest x point I have is 45 and my interval is 7. That means I should have 6 interval lines. However the last interval line is not being painted.
Screenshots
There should be an interval line at 43

Could you please drop here a reproducible code,
It helps me to reproduce it with just a copy-paste.
Thanks!

In linechartsample8 replace gridData with:
gridData: FlGridData(
          show: true, drawVerticalLine: true, drawHorizontalLine: false, verticalInterval: 2),

You should be able to see 5 intervals since there are points up to x:11 and lines should be drawn every 2 units. However the last line does not show at 10.0

Thanks for the explanation,
it will be fixed in the new version.

regression: last interval is being painted over top of the border again

It shouldn't happen,
where did you check it?

Fixed in 0.9.0
Thanks for reporting!

Regression is still there from this change

This went from the last border not being painted, to an interval being painted over top of the last border

It's the result of sample8 with this change:
gridData: FlGridData(
        show: true,
        drawVerticalLine: true,
        drawHorizontalLine: false,
        verticalInterval: 2,
        getDrawingVerticalLine: (value) => FlLine(color: Colors.red),
      )

What is the problem exactly?

Hi Sir,
I have question or suggestion:
Is there a option to include/exclude min/max when draw gridline/titles?

Hi Sir,
I have question or suggestion:
Is there a option to include/exclude min/max when draw gridline/titles?

I found the option is
checkToShowTitle: (minValue, maxValue, sideTitles,
appliedInterval, value) =>
true,
Thank you all.


[BUILD] VDB AX SOP not included in build with other SOPs by default when building AX

When building both OpenVDB AX and the OpenVDB Houdini Plugin (without specifying any custom dso list), it no longer automatically builds the OpenVDB AX SOP. Instead, in order to build the OpenVDB AX plugin you need to specify both OPENVDB_BUILD_HOUDINI_PLUGIN=ON and OPENVDB_BUILD_SOP_OpenVDB_AX=ON.
This may be intentional but highlighting it is a change in functionality of the build system to how it used to work for building the OpenVDB AX SOP.

Hi @richhones - I agree that I would have thought the latter flag would not be required (ie OPENVDB_BUILD_HOUDINI_PLUGIN =ON and OPENVDB_BUILD_AX=ON would build the AX SOP?), but I'm a bit confused by all the AX options.
It is no doubt this PR that has changed the behaviour, see the discussion between @Idclip and myself (https://github.com/AcademySoftwareFoundation/openvdb/pull/1087#issuecomment-869641362). Would one of you be able to propose / submit a change to address this?

Ok @danrbailey thanks for verifying this wasnt intended behaviour. Have submitted a PR to fix this now.


Update main.py

Completed the code.

Done a few changes in the code done by harshita.
Key  changes

insted of storing data in a db storing in a dict after getting from a db.
added features like viewing data extraction from the db etc.


Typo in documentation

This line contains a typo:
Filter that is set to the field thaaat you want to filter,

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

ID: 67aa28dc-9be7-25ed-0b35-b9127e6f00f9
Version Independent ID: 1f49660a-3c52-d48a-2d79-4ad0b541cee1
Content: Filtering From Query Designer and C/AL Code - Dynamics NAV
Content Source: dynamics-nav/Understanding-Query-Filters.md
Product: dynamics-nav-2018
GitHub Login: @jswymer

Thanks for the feedback. The issue has been fixed and will be available soon.


Support for custom method in js to support logs list

This release contains

Refactored response returned from RunString & RunProgram methods containing Value and Logs list

type JSEvalResponse struct {
	Result      Value
	ConsoleLogs []string
}

updated Readme file
testPrintLogs.go file to actually show a running solution to this

Sorry, can't merge it. For many reasons, main of which being it would violate the ECMAScript specification. This effectively makes 'log' a keyword with implicit behaviour when used in a call context.
I'm not sure exactly what you're trying to achieve, but I guess your best shot would be defining log as a non-writable and non-configurable global property, such as this:
func TestLog(t *testing.T) {
	var logs []string
	vm := New()
	err := vm.GlobalObject().DefineDataProperty("log", vm.ToValue(func(msg string) {
		logs = append(logs, msg)
	}), FLAG_FALSE, FLAG_FALSE, FLAG_FALSE)
	if err != nil {
		t.Fatal(err)
	}

	_, err = vm.RunString(`
	// let log; would fail with SyntaxError: Identifier 'log' has already been declared
	var log; // no effect
	delete log; // no effect
	log = () => {}; // no effect
	log("I'm still here");
	`)

	if err != nil {
		t.Fatal(err)
	}

	if len(logs) != 1 {
		t.Fatal(logs)
	}
	if logs[0] != "I'm still here" {
		t.Fatal(logs)
	}
}


Major rework of Subscriptions (Memberships)

Intermediate checkin.
The latest "2020" Stripe APIs that we are now using were pretty different from the 2018 API, and allowed for a big simplification, improved reliability, but not much compatibility.  So I abandoned the donate "app" and replaced it with the stripe_donations "app"
Organizational Subscriptions had broken Donation Subscriptions and regular Donations, so Organizational Subscriptions are not implemented in the new stripe_donations app -- not critical path.
As of this checkin, still need:

a Membership can be created and deleted, but error handling on bad cards is not yet implemented
Handle login and merging logins of we_vote_ids
Mobile layout
Chip in donations

There is a lot of commented out code in stripe_donations, that I will minimize in future PRs.

👍


Allow users to customize wrong_password_hint in PasswordElement

Example:
password01 = PasswordElement(
    name="password01",
    instruction="Enter any password",
    password="friend",
    forceInput=True,
    wrong_password_hint="Speak friend and enter."
)

Implemented.


updated concat function, which has been repurposed

@sehqlr,
Fix for issue: #1807
changed concat command to variable reference in  terraform/aws/vpc/main.tf

Alright, @hkuich, I've tested this, and it also works for Terraform v0.6.12, so this can go into 1.3. Thanks so much!

@sehqlr, glad I could contribute a little... :)


Futures margin rates

Unlike equities, each future contract has a different initial and maintenance margin. Should this be incorporated somehow into Zipline?
https://www.interactivebrokers.com/en/?f=margin&p=fut
Somewhat related to #564

It's currently in the works. @jfkirk should be able to give you more details.
Personally I'd love to have the SPAN margin method implemented
http://www.investopedia.com/terms/s/spanmargin.asp

@npezolano We are planning to incorporate a full margin system that would allow you to specify rates for individual Assets or classes of Assets. This would cover your use case.
Right now we are speccing this feature out (conducting user interviews and designing the API). Then we will begin development.
Self-assigning.

until the margin feature is implemented for futures, is there any way we can limit the max leverage when running backtests with futures?


[Bug]: Query response table showing extra empty column

Is there an existing issue for this?

[X] I have searched the existing issues

Description

Query response table is showing extra empty column in the end.

Steps To Reproduce

Run a sql query to fetch a single row.
Check the data rendered as table.

Public Sample App
No response
Environment
Production
Issue video log
No response
Version
self hosted, latest as on 5 July 2023

Hey ! I want to work on this issue, can you please assign it to me? Ill fix it.

Closing it as it is not reproducible anymore.


files/ APi for resource data access

GET or PUT individual files, ranges of files, or append to a file

@pkdash Do you think this capability is already available in REST API? If so, please close this issue. I am assigning it to @jbradberry for now if it is still an issue.

@hyi Currently GET/POST/DELETE are supported for resource file. PUT yet to be implemented.

@pkdash Are we still implementing PUT

@aphelionz I would think we need to implement PUT. @alvacouch your thoughts?

This is a legacy requirement and has been superceded by new functionality. Create new issues if they arise.


Better descriptions of templates in app-group

When running this command
davsclaus:~/workspace/quickstarts/app-groups/quickstarts (527)/$ oc get templates
NAME             DESCRIPTION                                                                        PARAMETERS    OBJECTS
base             Creates an instance of the Fabric...   1 (all set)   3
cdelivery        Creates an instance of the Fabric...   8 (all set)   29
cdelivery-core   Creates an instance of the Fabric...   7 (all set)   20
ipaas            Creates an instance of the Fabric...   8 (all set)   31
kitchen-sink     Creates an instance of the Fabric...   8 (all set)   33
logging          Creates an instance of the Fabric...   1 (all set)   7
management       Creates an instance of the Fabric...   1 (all set)   9
metrics          Creates an instance of the Fabric...   1 (all set)   5
quickstarts      Creates an instance of the Fabric...   1 (all set)   6

I think I fixed this in master - haven't had chance to test it yet mind

Ah yeah I may have run with the 2.2.1 plugin.

just trying to get the vagrant image to be able to use snapshots so we can try things out before a release...

Yeah with snapshot the ipaas now says
  "description" : "## Fabric8 iPaaS\n\nProvides an [Integration Platform As A Service](http://fabric8.io/guide/ipaas.html).\n\nHere's how to [install it on an existing Kubernetes or OpenShift environment](http://fabric8.io/guide/fabric8OnOpenShift.html).\n",