[pyspark] disable repartition_random_shuffle by default

This PR resolved the issue1 xgboost detected this parameter is not used in native.
Parameters: { "repartition_random_shuffle" } are not used.

and this PR disabled repartition_random_shuffle by default and will give some prompt when detecting an empty partition.  The severe data skew mentioned in https://github.com/dmlc/xgboost/issues/8221  is really kind of rare case, so I don't think we need to change the default repartition behavior to hash partitioning from round robin partitioning. Besides, the hash partitioning will introduce an extra "project" physical plan, see

repartition_random_shuffle=False

== Physical Plan ==
Exchange RoundRobinPartitioning(1), REPARTITION_WITH_NUM, [id=#94]
+- *(1) Project [cast(class#4 as float) AS label#20, cast(sepal_length#0 as float) AS sepal_length#10, cast(sepal_width#1 as float) AS sepal_width#11, cast(petal_length#2 as float) AS petal_length#12, cast(petal_width#3 as float) AS petal_width#13]
   +- *(1) ColumnarToRow
      +- FileScan parquet [sepal_length#0,sepal_width#1,petal_length#2,petal_width#3,class#4] Batched: true, DataFilters: [], Format: Parquet, Location: xxx

repartition_random_shuffle=True

== Physical Plan ==
*(2) Project [label#20, sepal_length#10, sepal_width#11, petal_length#12, petal_width#13]
+- Exchange hashpartitioning(_nondeterministic#26, 1), REPARTITION_WITH_NUM, [id=#92]
   +- *(1) Project [cast(class#4 as float) AS label#20, cast(sepal_length#0 as float) AS sepal_length#10, cast(sepal_width#1 as float) AS sepal_width#11, cast(petal_length#2 as float) AS petal_length#12, cast(petal_width#3 as float) AS petal_width#13, rand(1) AS _nondeterministic#26]
      +- *(1) ColumnarToRow
         +- FileScan parquet [sepal_length#0,sepal_width#1,petal_length#2,petal_width#3,class#4] Batched: true, DataFilters: [], Format: Parquet, Location: xxx

@trivialfis any idea about the error?
Run python tests/ci_build/lint_python.py --format=0 --type-check=1 --pylint=0
xgboost/spark/data.py:12: error: Module "xgboost.spark.utils" has no attribute "get_logger"
Found 1 error in 1 file (checked 24 source files)
Error: Process completed with exit code 255.

Use this instead:
from .utils import get_logger   # type: ignore

The utils.py is not properly annotated.

Interesting,
I locally run this command, and there are some other failures for other files.
$ python tests/ci_build/lint_python.py --format=0 --type-check=1 --pylint=0
xgboost/core.py:451: error: Incompatible types in assignment (expression has type "BaseException", variable has type "Optional[Exception]")
xgboost/plotting.py:112: error: Unsupported operand types for + ("List[float]" and "int")
xgboost/plotting.py:112: note: Left operand is of type "Union[float, List[float]]"
xgboost/plotting.py:121: error: Unsupported operand types for * ("List[float]" and "float")
xgboost/plotting.py:121: note: Left operand is of type "Union[float, List[float]]"
xgboost/dask.py:842: error: Incompatible types in assignment (expression has type "None", variable has type "Tuple[str, int]")
xgboost/dask.py:850: error: Incompatible types in assignment (expression has type "None", variable has type "str")
xgboost/dask.py:1735: error: "sync" gets multiple values for keyword argument "asynchronous"
Found 6 errors in 3 files (checked 24 source files)

And I saw the xgboost/spark/core.py also imports get_logger without type: ignore, see https://github.com/dmlc/xgboost/blob/master/python-package/xgboost/spark/core.py#L59-L70

Most of the spark modules have this line: https://github.com/dmlc/xgboost/blob/6925b222e0dc268803b59c93b28db65cf130beb7/python-package/xgboost/spark/core.py#L1
As a result, the type checker cannot work properly with these modules.
As for the other errors, a likely cause is missing dependencies in your local env.


Thousand separator with percents

Hello,
I got a problem with the percent format on bug numbers: numeral(10).format('%') returns "1000%" whereas "1,000%" (with the comma) is expected.
Or maybe I missed something?
Thanks in advance,
Xavier

Use the format string along with the % sign
numeral(40).format('0,0%')
//=> "4,000%"


fix race conditions

First, thanks for bringing this very neat Golang Modbus stack :pray:
I faced race conditions while writing down tests where I was successively opening/closing the server and reconnecting TCP clients
Reason is: acceptTCPClients() which is running in a separate goroutine might concurrently access to some attributes.
When I tried to reproduce this error with
go test -timeout 30s  github.com/simonvetter/modbus -race`

I spot another race condition in unit tests.
I'll propose a PR, your opinion we'll be appreciated !

@simonvetter looks like I'm not allowed to create a branch/PR. I'm ready to push on my side (commit adding getters holding read locks), let me know if you're interested

Hey man, thanks for reporting it! Glad to see you around hacking on open source projects :-)
Are those races only present in tests, or is it in the server itself?
One way of proceeding would be to fork my repo, push to your fork, then create a pull request here. Would that work for you?

Hey man, thanks for reporting it! Glad to see you around hacking on open source projects :-) Are those races only present in tests, or is it in the server itself?

To my understanding, the race against access to .started occured at server level in my test suite, while for .tcpClients it was at unit test level

One way of proceeding would be to fork my repo, push to your fork, then create a pull request here. Would that work for you?

Sure ! Done :)

Sorry for the late reply, I've been fairly busy lately.
Does https://github.com/simonvetter/modbus/tree/simon/fix-test-races solve your issues? If yes, I'll merge and do a point release.

It does ! Thanks @simonvetter. Have a good day !

nice! v1.5.1 pushed. Thanks again!


Set cross tools environment automatically

LINUX_MULTIARCH_ROOT=C:\UnrealToolchains\v22_clang-16.0.6-centos7\

https://docs.unrealengine.com/4.27/en-US/SharingAndReleasing/Linux/GettingStarted/
https://dev.epicgames.com/documentation/en-us/unreal-engine/linux-development-requirements-for-unreal-engine


.github: actions-rs audit check improvements

Cache cargo-audit install and use GITHUB_TOKEN

Codecov Report

Merging #88 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop      #88   +/-   ##
========================================
  Coverage    79.28%   79.28%
========================================
  Files           14       14
  Lines          618      618
========================================
  Hits           490      490
  Misses         128      128

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 e0e202c...d28d5d9. Read the comment docs.


feat: u251 type

Feature Request
Describe the Feature Request
When developing contracts for Starknet, deciding what data type to use is not always a straightforward process. Most application don't need to use 256 bits, so using u256 - which is composed by two felts - is sometimes overkill, and not optimized. However, the closest other integer type is u128, which might be too small. Using felt252 is not a great solution, as it lacks over/underflow checks, comparison operators, etc.

Describe Preferred Solution
A u250 or u251 type which wraps a single felt252 and implements basic integer traits, like PartialEq, Div, Rem, Add, etc.

Describe Alternatives

Related Code

Additional Context

If the feature request is approved, would you be willing to submit a PR?
(Help can be provided if you need assistance submitting a PR)

[ ] Yes
[ ] No

can you please add a standard use case where u128 is actually not large enough - while u250 would have been?
(do note that multiplications and so would be very problematic with such a type).
Also note - that if you are sure your number is not too large - you can do all your operations in u256 - and than convert it to felt252 for storage and message passing.

Is the computation overhead of using u256 (composed of two felts) small enough to consider it meaningless, if it doesn't have to be stored onchain?
If yes, then indeed I could just convert it to a felt252 before storing the data onchain. Thanks

i'm mostly saying this as - let us start with a workable solution - and consider optimisations only later if we understand they are required.

From my experience:

u128 has almost always been large enough for computation
u256 operations are, not surprisingly, MUCH less performant than u128 / u64
the primary point at which the lack of a u250-ish type is problematic is when bitpacking to take advantage of as much of a storage felt252 as possible (i.e. doing a lot of bit shifts with u256 is extremely inefficient)


Issue/11 gsp script

Pull Request
Description
Add script to update gsp locations
Fixes #11
How Has This Been Tested?
ran it on live database

[ ] Yes

Checklist:

[ ] My code follows OCF's coding style guidelines
[ ] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked my code and corrected any misspellings

Codecov Report

Merging #12 (a2eb4c5) into main (a48692b) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head a2eb4c5 differs from pull request most recent head 219d270. Consider uploading reports for the commit 219d270 to get more accurate results

@@           Coverage Diff           @@
##             main      #12   +/-   ##
=======================================
  Coverage   89.76%   89.76%
=======================================
  Files           3        3
  Lines         127      127
=======================================
  Hits          114      114
  Misses         13       13

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Add more specificity to docstrings

Currently in the docstrings I see various references to sections of what I presume is the documentation - see the image below for an example. Are these outdated or am I just not able to find them? Thanks for any help!

Could you attach a link to where you are refering to? Those might be sections in the specific RFC.

@phy25 this is the section in my screenshot: https://github.com/lepture/authlib/blob/master/authlib/oauth2/rfc6749/grants/authorization_code.py#L60

You can find them on RFC6749.


cli: Better error message for too long socket path

This verify the socket path length according to
http://man7.org/linux/man-pages/man7/unix.7.html.
The proxy fails correctly when the socket path is too long, but the
message is too generic:
init: couldn't create AF_UNIX socket:
listen unix SOCKET_PATH: bind: invalid argument

Fixes #78
Signed-off-by: Jose Carlos Venegas Munoz<EMAIL_ADDRESS>

lgtm

Coverage decreased (-0.4%) to 74.504% when pulling 8fdb5f0be831a64639a9b6383161c7aed5caad24 on jcvenegas:jcvenega/topic/cli into e9f67e722cfdcb54df7a2cd2dab2e010336217b8 on clearcontainers:master.

lgtm


support json schema validation

Coverage increased (+0.1%) to 96.595% when pulling 2df7d485eecb5d42e9ab11b6ab3727c5f7f406ff on feature/json-schema-validation into 2e192e1207f07e716c84e718c02f3a59d30c3673 on master.

Coverage increased (+0.008%) to 96.464% when pulling 90ad9db78e2c5a4d1bfc6efea8fa1e559e8afc5a on feature/json-schema-validation into 2e192e1207f07e716c84e718c02f3a59d30c3673 on master.

Coverage increased (+0.008%) to 96.464% when pulling<PHONE_NUMBER>6ae68373361190f4b9f57c67653d52 on feature/json-schema-validation into 2e192e1207f07e716c84e718c02f3a59d30c3673 on master.

Coverage increased (+0.008%) to 96.464% when pulling f913367ac28466ab71494745a896af3cbb472c32 on feature/json-schema-validation into 2e192e1207f07e716c84e718c02f3a59d30c3673 on master.


Pasting text doesn't work

I tried to plug this tool in because EditorJS doesn't have text alignment by default but when pasting a text content, it doesn't paste anything except whitespaces.

@wendellestradairely
ok!
i try and fixing, thanks for the report.

https://github.com/kaaaaaaaaaaai/paragraph-with-alignment/releases/tag/v1.1.0


MM-40114 Migrate post-related components to use getIsMobileView

getIsMobileView is the more redux-friendly, more self-explanatory, and hopefully more performant version of Utils.isMobile that we're wanting to be using everywhere now.
Ticket Link
https://mattermost.atlassian.net/browse/MM-40114
Related Pull Requests
https://github.com/mattermost/mattermost-webapp/pull/9507
https://github.com/mattermost/mattermost-webapp/pull/9509
https://github.com/mattermost/mattermost-webapp/pull/9510
Release Note
NONE

/cherry-pick release-6.3

Cherry pick is scheduled.


Global search: change "Everything " to "All Data Types"

Everything is too generic!
If "All Data Types" takes too much space in the default view, we can show only "All" and then
"All Data Types" as the top item in the dropdown, selected by default....the way it is done it amazon!
-Maulik

Fixed


Improvements (todo.py + misc files) (chg-010-013)

Update sonar python version
Diagnose Sonar Analysis Warnings
Differentiate dev packages (dependencies) from user packages (dependencies)
Update workflows for previous step
Update Makefile
Add Coverage Ignore
Add lint to Makefile
Add .vscode to git ignore
Update todo.py

Update Sonar Python Version
This was a really simple change, after looking on sonar cloud I noticed there were some warnings about the scan, one of them being that sonar is assuming our project is multiple python versions cause we do not specify the version, easy fix to this was just adding the following line to the sonar-project.properties file.
sonar.python.version=3.10

Diagnose Sonar Analysis Warnings
There was another error and I am unable to recall the exact message/issue at this time. The just of it was that github stores our code in a different place then sonar is expecting, I made updates to the sonar pipeline accordingly to try and address this. The warning still appears but I have not noticed any unusual behavior or results from our scans, will unfortunately have to put this to side and just keep an eye out for any issues.
Differentiate dev packages (dependencies) from user packages (dependencies)
I updated our pipfile to use some of the functionality pipenv provides. Pipenv lets you separate developer packages from user packages and that is exactly what I did, this was very simple in just changing the location of the package name in the pipfile.
I also had to update our workfows/pipelines to use dev packages.
Update Makefile - please checkout coverage, lint and pipeline
Edited some commands with some cool stuff (coverage command being the coolest.) Added some new commands like dev-setup, setup, lint, pipeline
Removed vscode settings from repo
Did this to avoid settings being shared across different setups, we do not need it and devs might have different setups.
Todo.py
Updated todo.py with

comments
updated docstrings
functionality for exit codes
neatened up install process (More improvement in future)

Add Coverage Ignore for test files.
This was a very important change that failed to come through, this will be added in a upcoming ticket. This was very easy to implement using a coveragerc file but sonar was not happy with us including test files in the source files and therefore included tests in coverage even tho I tried many things to tell it otherwise.
Including tests in our coverage inflates our coverage, thus this is a very important improvement in future. This will not be alot of work but will change our file structure a bit, therefore I am holding off on this.


Loading Toast Cannot Be Dismissed

Describe the bug
The Loading Toast, cannot be dismissed by Clicking on The Toast but can be dismissed using the button from which the toast is popped up. All Other Toasts can be dismissed by clicking on the toast.
To Reproduce
Steps to reproduce the behavior:

Implement Loading Toast
Run The App and Start off the toast.
Click on the spinner and it would not dismiss.

Expected behavior
I want to dismiss the loading toast by clicking on the spinner like in other toasts.
Screenshots
Sorry, No Screenshots are available at this point of time. You can ask for them later.
Desktop (please complete the following information):

OS: macOS Monterey 12.3.1
Browser: Safari
Version: Latest

Smartphone (please complete the following information):

Device: iPhone 13 Mini (Simulator)
OS: iOS 15 Latest
Browser: Safari
Version: Latest

Additional context
N/A

Hey @RudraSen2 ,
A tap to dismiss on a loader would not be a very useful feature because the user should not be able to dismiss the loading alert in case there is some processing logic.
This is why I decided that the loading alert would not be possible to dismiss.
If you really need this feature, you can always fork the library and change the code inside.


unsuitable profile default in ovirt-cluster

Production should not disable ksm nor ballooning

@michalskrivanek - so change the defaults. Currently, when creating a cluster, via the API or UI, neither are enabled (I'm not sure why ballooning isn't, btw. KSM we can argue about).


feat(citext): support postgres citext

Resolves  #295
Users needed to explicitly cast all CITEXT columns to TEXT. This set of changes will cause CITEXT columns to be treated the same as TEXT columns by sqlx.
Happy to add additional details/code/comments/tests as needed.

Looks like you need to add CREATE EXTENSION IF NOT EXISTS citext; here: https://github.com/launchbadge/sqlx/blob/main/tests/postgres/setup.sql#L3

@abonander thanks for pointing that out. I've went ahead and updated the postgres setup.sql file.

@hgranthorner CI is still failing.

So my previous implementation of this was completely wrong. I assumed that because it worked for me, it would work for everyone, and due to some test setup issues (more on that below) I wasn't able to figure out how to test it beyond just running some samples. The new implementation follows the ltree implementation by dynamically looking up its own oid at run time when needed. Note that while Postgres is happy to compare citext and text, it refuses to compare citext[] and text[]. It seems like you have already bumped into that, since you there are no test cases for varchar[] -> Vec<String>, etc. It seems like this would break some of sqlx's compile time checking though. I also double checked that everything would work without the citext extension installed (seems like everything worked), but didn't know how to set up test cases for that. I wasn't sure if I should hide this functionality behind a new citext flag - happy to add that if you'd like.
In terms of the testing issues, I ran into 2. I'm fairly certain that both of them are due to me being on Windows. For the first, the tests/docker.py file attempts to inspect the running docker instances for f"sqlx_{driver}_1". On my machine, the images were named sqlx-{driver}-1. This failure was giving me a pretty inscrutable error. If someone else can confirm that this is intended docker compose functionality on Windows, I'm happy to raise a PR to fix this for others - I was unable to find any documentation on it anywhere. My other issue had to do with some weird configuration I had set up around Docker with a WSL backend - Docker tends to devour RAM with a WSL backend for some reason, so I had set memory limits for the application. During sqlx testing, the SUTs would start closing client connections with no mention of OOM or anything like that. Removing the limitations fixed that for me. I still wasn't able to run the any tests, but I think I'm just messing something up locally...
Long story short - sorry for wasting your time earlier. Things should (hopefully) be good now. Glad to take any and all feedback.

Sorry for the delay. I have a few nits, but I'm just going to commit them as suggestions to avoid any more delays in getting this merged.

@abonander Is there a timeline to release this on crates.io?


Tracker is no longer implemented in 3.0.1

This means that the error messages will not work when using Autoform on Meteor.
I checked the source code and the reactivity is no longer present.

This was an intentional change as mentioned in the release notes. Someone is welcome to figure out a solution that does not involve dependency injection, such as an afterValidate hook where you could update a reactive variable, and I will accept a PR for that or help implement if you submit a proposal.

Thankyou.  I didn't even know how the reactivity worked until I looked at the source, unfortunately I don't think I'm skilled enough to come up with a solution, so I'll probably stick to 1.x.x.  I imagine this will come up quite a bit in the community so I thought it was worth raising.


style(frontend): avoid showing empty tokens list on loading

Motivation
We aim to show a non-empty tokens list as first, and avoid showing an empty list just because it is loading, thus reducing glitches.
Before
https://github.com/user-attachments/assets/b67eacb5-4553-4882-9f98-4bd847afaaa0
After
https://github.com/user-attachments/assets/6937aa12-702f-49ab-8431-2cb31561f79a

I rephrased a little and included some videos done in staging. Basically I am breaking down PR #1796 into two main parts:

Avoid glitches but showing some list (even with balance and exchanges loading)
Disable sorting during pointer events

this PR became obsolete, closing


Create REST API to set last and next watering/caring date

Addressed by Daniel with the following commit: 71a4cb66908fc6589eeb255d86ded3b764d57454


Landing Page & Dashboard UI Update

#396 But on the correct branch this time
A lot of minor changes to the Landing Page, main Dashboard UI, and the Instance Details menu.
Mostly a cleanup to the UI:

Fix font sizes
Fix the top Navbar, the logo, and the Avatar
Fix Maintenance Mode visibility on the Dashboard
Fix menu mess on Docs page
Fix Pricing page Rare deals mess
Fix Instance details UI
Increase Log font size

etc.

🥳

Finally


Registering bower package names is not supported anymore

Hello,
I am trying to register a new Bower package. Once it asks this:

Registering a package will make it installable via the registry (https://registry.bower.io), continue?

I agree answering "Y", but I get the following error just after a few seconds:

EUNKNOWN Unknown error: 500 - Registering bower package names is not supported anymore. You can install any bower package on github with command like "bower install jquery/jquery-dist --save"

Was the register support really dropped or it is an issue?
Thank you very much in advance!
Cheers,
Joan

Docs are now updated


Updating Markov Chain

Now supports initial state probability distribution
Supports both graph-based and matrix-based (row-oriented) representation
Transitions can now relate to Failures

@marc-zeller to be clear, this is just for notifying you that there's been updates to the Markov package.
There's no obligation to provide a detailed review.

@ISorokos: Thanks for the hint. Interesting. I would have some questions.


Add FileStore and CSVStore

Adds two new stores that are helpful when handling lots of tasks with large amounts of data.
Storing everything in memory between tasks executing may not be possible, when working with large datasets. These new stores allow for data to be wrote to files and then opened later when required by the dependent tasks.

Todo

[ ] Add some example pipelines that use FileStore
[ ] Add some example pipelines that use CSVStore


Add new option to "Add final to method paramters"

I would like to add missing "final" definition to method parameters (Java)
I think that it is an accepted good practice don't modify the parameters values during method execution.
Intellij

Hello @domink, this feature is already present in the plugin using the option "Add final modifier to local variable or parameter". If you need the parameter modification, you can add a new processor in com.dubreuia.processors.java.JavaProcessor, based on localCanBeFinalExceptImplicit.
I won't have time to implement this anytime soon, but PRs are welcomed. You can check the contributing guide in the README. I'm adding the "need help" tag for this purpose. Thanks

Yes it is. Really sorry I don’t know why I was unable to see it. Sorry.
I’m reading the guides but I’m pretty new using IntelliJ and right now I don’t feel comfortable enough to contribute... but for sure I’ll try.
Thanks a lot a good work.
El 9 ene 2020 10:11 +0100, Alexandre DuBreuil<EMAIL_ADDRESS>escribió:

Hello @domink, this feature is already present in the plugin using the option "Add final modifier to local variable or parameter". If you need the parameter modification, you can add a new processor in com.dubreuia.processors.java.JavaProcessor, based on localCanBeFinalExceptImplicit.
I won't have time to implement this anytime soon, but PRs are welcomed. You can check the contributing guide in the README. I'm adding the "need help" tag for this purpose. Thanks
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Moved to ideas https://github.com/dubreuia/intellij-plugin-save-actions/projects/2


Anonymize data

Add anonymize.py to anonymize data.
Update db.sql with anonymized dump from current production database.

To do:

documentation
update Makefile

This addresses #269.

On Wed, May 13, 2015 at 05:14:33PM -0700, Greg Wilson wrote:

M db.sql (57426)

Email addresses don't seem to be anonymized.  This should be rectified
and force-pushed ASAP, to avoid leaking that information via the Git
history.
https://help.github.com/articles/remove-sensitive-data/

On Thu, May 14, 2015 at 04:54:14AM -0700, Greg Wilson wrote:

Closed #272.

Closing thte PR doesn't remove GitHub's cached copy of your branch.
For example, try:
$ git fetch git://github.com/swcarpentry/amy.git refs/pull/272/head:muahaha
$ git grep tremily.us muahaha -- db.sql
muahaha:db.sql:INSERT INTO "workshops_person" VALUES(4181,'',0,'615RYgQd',NULL,'6f3mc6tUr','wking@tremily.us','F',1,'Kcoe','4YdA12','http://IEN.Ud8b5h8G/34NWA39DX-U3BPXLDlN','tBIb4Egq5',91,'2015-04-24 16:53:03.073446');
You might be able to remove the references by re-opening this PR,
restoring your anonymize-data branch, and running:
$ git checkout anonymize-data
$ git reset --hard ${YOUR_REMOTE_FOR_SWCARPENTRY}/master  # or any reference from before 28655cb
$ git push -f ${YOUR_REMOTE_FOR_GVWILSON} anonymize-data
If that doesn't work, I guess we have to email GitHub's support team.

We've been in touch with GitHub tech support on this one - even the hard
reset/force push isn't guaranteed to do it.  I'll forward the message.


tests: Avoid deprecated TestCase method aliases

Fixes these warnings with python -Wd:
/home/circleci/zulip/zerver/tests/test_middleware.py:43: DeprecationWarning: Please use assertRegex instead.
  self.assertRegexpMatches(
/home/circleci/zulip/zerver/webhooks/travis/tests.py:40: DeprecationWarning: Please use assertNotEqual instead.
  self.assertNotEquals(msg.topic_name(), self.TOPIC)
```

https://docs.python.org/3/library/unittest.html#deprecated-aliases

I'm pretty sure we did sweeps for these at some point; they must have gotten added since.


Union Types?

Are union types supported and if so, how would I go about doing them?
So for example, if I had a request of:
query GetItems {
     item {
          ... on SuperItems {
               id
               name
          }
          ... on SuperDuperItem {
               id
               name
          }
     }
}

How would I specify this in the Query Document, and also in parsed model?
Thanks!

What you're referring to are inline fragments https://graphql.org/learn/queries/#inline-fragments
from memory I think this should work
...
Selection.field(name: "item", alias: nil, arguments: nil, directives: nil, type: .object(selectionSet: [
    Selection.inlineFragment(namedType: "SuperItems", directives: nil, selectionSet: [
        Selection.field(name: "id", alias: nil, arguments: nil, directives: nil, type: .scalar),
        Selection.field(name: "name", alias: nil, arguments: nil, directives: nil, type: .scalar)
    ]),
    Selection.inlineFragment(namedType: "SuperDuperItem", directives: nil, selectionSet: [
        Selection.field(name: "id", alias: nil, arguments: nil, directives: nil, type: .scalar),
        Selection.field(name: "name", alias: nil, arguments: nil, directives: nil, type: .scalar)
    ])
])
...

Could do Object instead of Selection.field(... type: .object...) etc. for these instead too and should work and be less boilerplate and provide default nil values for a lot of the args; this example is just what my memory served me quickly.
FYI, our code generator (written entirely in Swift) has been stable for a while now, in the process of cleaning it up and getting CI/deployments for it. Targeting before EOY so, hopefully soon, you won't need to think about these details anymore.

@rex-remind101 That's awesome, can't wait for the code generator! We had moved away from Apollo though because of it's code generation, mainly because we wanted to specify our own models. How does the code generation work for AutoGraph? Does it generate models and the request like Apollo, or will it allow us to use our own pre-built models with a generated request?

Also, the inline fragments make sense, however I was referring to something else called union types: (https://graphql.org/learn/schema/) You'll have to search the page for "Union Types"
How to specify the inline fragments in the request makes sense, but the issue I'm having is trying to specify is how to represent the union type in the response. At request time, it is not known if "item" will be of type "SuperItem" or "SuperDuperItem". It's basically a generic item type. When it is returned it will be one of the specified union types, which the response handler will know typically by the context of some other value in the parent object. In this case let's pretend the request is actually:
query GetItems {
     items(first: 10)
          type
          item {
              ... on SuperItem {
                    id
                    name
               }
               ... on SuperDuperItem {
                    id
                    name
               }
          }
     }
}

In this example "item" can be of either type SuperItem or SuperDuperItem, and the response handler will know based on the value of the "type" field.
Now I know there isn't any way to specify in Swift that "item" can be one of many types, but is there a way to specify that it is a raw JSON object that can be parsed manually after the response returns from GraphQL?

Union types are opaque to a query, you can only use Inline Fragments to access them. Therefore the shape of the JSON isn't going to be specialized for union types, it will be JSON matching the query shape.
However by "response", I think you actually mean the "mapping" portion after the response is returned by the server? In which case, "is there a way to specify that it is a raw JSON object that can be parsed manually after the response returns from GraphQL?" I think you need to clarify what you mean here.
If what you mean is "can I just get back the JSON", then yes. Make a Decodable data structure that simply captures the returned JSON on a property, and use that as your SerializedObject for your Request
If this isn't what you meant then maybe an example of what you want at the call site would help illustrate it.
"How does the code generation work for AutoGraph? Does it generate models and the request like Apollo, or will it allow us to use our own pre-built models with a generated request?"
It generates structs and maps response data directly to them in a safe way, Apollo passes around Dictionaries internally which we're not fans of. You can use those structs however you please, including mapping them to your own models. If you're asking if you can make a model and we'll adhere directly to that, it sounds somewhat like having a meta-programming abstraction that can arbitrarily map to any model, which isn't really practical since Swift doesn't support first class meta-programming (...yet). If you had something more specific in mind though I'd love to hear it.
Also, the code generator is a SPM package/binary written entirely in Swift, which is much more to our liking. Swift happens to be a great language for writing compiler-like software.

You are completely correct on the union types, my lack of GQL experience is showing ;-)
I am referring to the mapping portion as you said, and it's specifically about when you have an unknown nested type within the response object. I figured out the solution though, it was simply to set the nested type as a Dictionary. Thanks for your help and quick responses.
In regards to the code generation, I do have some thoughts. It's the auto generated structs that are concerning, because it would mean that if we want to use our own internal models, we would have to write a parser for each of the structs, especially becoming more complicated if the structs are request specific like in Apollo. My thought would be to have requests that are auto-generated, but have the response still come out as JSON so that we could map to our own internal Codable models, or allow a return type to be passed in to the generated function just like you have now.

The code generator isn't mandatory so it shouldn't affect you if you don't want to use it.

That's fair, thanks again for the library and all your work!


redux-devtools-extension became a required dependency

This is a...

[ ] feature request
[x] bug report
[ ] usage question

What toolchain are you using for transpilation/bundling?

[x] @angular/cli
[ ] Custom @ngTools/webpack
[ ] Raw ngc
[ ] SystemJS
[ ] Rollup
[ ] Other

Environment
NodeJS Version: 8.9.4
Typescript Version: 3.2.4
Angular Version: 7.2.11
@angular-redux/store version: 10.0.0
@angular/cli version: 7.3.7
OS: Windows
Expected Behaviour:
redux-devtools-extension stays as a devDevendency
Actual Behaviour:
Due to the reference to redux-devtools-extension from the code - this dependency has become required.

I second that, redux-devtools-extension is used here, though listed as a devDependency.
The build obviously fail on that very issue, we had to manually install the missing dependency.


Please update to intl 0.18.1

No, we cannot. Since this package does not use the intl package. The error you see comes from Flutter SDK flutter_localizations not from this package.


Introduce wildcard search for advanced search visibility

Is your feature request related to a problem? Please describe.
Need to generate custom attributes that can be indexed and searched for ES 7.9+
Would be worth having the same wildcard search flexibility for some of the available search attributes as well.
Proposed Solution
Introduce new custom wildcard field. ==> CustomWildcardField
Would be worth switching a bunch of search Attributes to all use wildcard search since its a drop in replacement for keyword search but that would expect different handing across ES versions below 7.9

DomainID
WorkflowID
WorkflowType

Additional context
Elastic Ref
Would also help with Issue #3962

Closing this issue, since related one was closed: https://github.com/uber/cadence/issues/3962


How to reproduce the  GTEA

Is there any trick to reproduce the result of GTEA , could you please giving config.txt for this dataset and it will be convenient for reproducing. Thank you.@Pilhyeon

the result I reproduce is about 28.8mAP on GTEA,

Hello,
I provide some hyper-parameter settings for GTEA below.
batch_size: 7
feature_fps: 15
search_freq: 1
budget: 200
lambdas: [1, 1, 1, 1, 1]
Hope this would be helpful.
Best regards.

the result I reproduce is about 40.83mAP on GTEA, there is still some gap.

请问作者给了参数后你复现出原来的结果了吗？

现在没做这个了，你试过了吗

the result I reproduce is about 40.83mAP on GTEA, there is still some gap.

请问作者给了参数后你复现出原来的结果了吗？

现在没做这个了，你试过了吗

我还没试，你怎么不做这个了？因为效果不好提升了吗？

provide some hyper-parameter settings for GTEA

Hello, can you provide some hyper-parameter settings for BEOID?Thanks!

@Pilhyeon Hello, can you provide some hyper-parameter settings for BEOID? Thanks!

the result I reproduce is about 40.83mAP on GTEA, there is still some gap.

请问作者给了参数后你复现出原来的结果了吗？

现在没做这个了，你试过了吗

the result I reproduce is about 40.83mAP on GTEA, there is still some gap.

能分享一下你复原的代码吗，我想试试能不能复现一下，感谢！

您好！由于使用作者提供的参数仍然无法复现其结果，已经中断该实验了。
------------------ 原始邮件 ------------------
发件人:                                                                                "Pilhyeon/Learning-Action-Completeness-from-Points"                                                                                @.>;
发送时间: 2023年4月2日(星期天) 晚上6:52
@.>;
<EMAIL_ADDRESS>主题: Re: [Pilhyeon/Learning-Action-Completeness-from-Points] How to reproduce the  GTEA (Issue #5)
the result I reproduce is about 40.83mAP on GTEA, there is still some gap.
请问作者给了参数后你复现出原来的结果了吗？
现在没做这个了，你试过了吗
the result I reproduce is about 40.83mAP on GTEA, there is still some gap.
能分享一下你复原的代码吗，我想试试能不能复现一下，感谢！
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

@chenrxi 可以分享一下40.83mAP的完整代码嘛，我复现的结果很差，想看一下问题在哪里


en.wikipedia.org blocked by comment in hosts file.

Hi!
I am using the hosts file in a pi-hole and it seems that line 66888 in hosts has a comment pointing to the source of sitename that is not being treated as such.
The line reads
<IP_ADDRESS> oralse.cx  # en.wikipedia.org/wiki/List_of_shock_sites
at the moment I am not certain if this is specific to the pihole engine or if there is a syntax problem.
will research further.
Steps to reproduce:
with the hosts file added to adlists in pihole, try $dig en.wikipedia.org @pihole-ip
Workaround:
Whitelist the en.wikipedia.org domain.

Please disregard. linked to a different blocklist I did not check before that for some reason explicitly includes en.wikipedia.org

Thanks John @jgfam69qy2
The # character indicates a comment...


typo fixes to networking

@brndnmtthws - would you mind taking a quick look at this? I meant to just fix some typos, but ended up making some additions to virtual-ip-addresses.md.

@sascala Some conflicts, could you resolve them? Thanks

OK, conflicts fixed, @emanic


fix: bsd support

When checking for if on a platform that uses unix style path separators
only linux and osx where take into account. This patch and BSD to that
list.
Tested on: FreeBSD

Looks good but you also need to regen the lua files. Look in CONTRIBUTING.md

Another BSD user confirming its working fine, just yesterday I had to add GitGutter because it was not working on BSDs . Thank you very much @xanderio !


Application testing

[ ] Define necessary test coverage
[ ] Implement tests
[ ] Test automation?

Related

https://github.com/zetonteam/zeton_django/pull/67

I'd be happy to write some tests. It would be nice if one day we could discuss strategies, goals, scenarios and exchange resources on this topic. I think if we keep the coverage at 80-90%, we'll be fine.

Current tests are not in use, e.g https://app.circleci.com/pipelines/github/zetonteam/zeton_django/255/workflows/cd6af1f3-05d3-4518-9216-0b659b019ba4/jobs/259/steps
Found 0 test(s).
System check identified no issues (0 silenced).

----------------------------------------------------------------------
Ran 0 tests in 0.000s

NO TESTS RAN

Unit tests are in implementation. CircleCI config is currently running unit tests, it has been implemented in #156

@MartaSien
I guess this topic can be closed. There are tickets for remaining endpoints, and coverage is not a priority right now.


Windows 10 Hyper-V kernel panic

RancherOS Version: (ros os version)
v1.0.1 - Docker 17.03.1-ce - Linux 4.9.24
Where are you running RancherOS? (docker-machine, AWS, GCE, baremetal, etc.)
Hyper-V from Windows 10. Gen 1 VM (as Gen 2 requires EFI), boot from CD
I get a kernel panic during the boot, but it seems an upstream bug in Linux kernel as other OSes crash too.

@nponeccop is there any chance you can try the v1.0.2-rc3 release? it uses linux 4.9.30. I'm working on improving VMware support atm, but I have a plan to free up my Windows box so I can get to Hyper-V too.

It's the same crash with v1.0.2-rc3 / 4.9.30

bummer.

I tested few random kernel versions. Unfortunately I only have a 32-bit virtual system to check. So far only 3.14.2 and 3.18.34 worked. I confirmed with my friend that it's not only me.
4.0.7 which worked in the past doesn't work now. So it seems to be a regression in Hyper-V caused by windows updates. It's not even clear where to submit upstream bug reports.

FYI 4.11.6 kernel works with Hyper-V now.

@nponeccop that's useful info - I don't plan to move off LTS for 1.1 (or 1.2), but I do want to change the way we build rootfs's to make the kernels swappable independently of the OS release.
I wonder if we can pinch a patchset from one of the distros.

Good news. 4.9.44 kernel (current latest LTS) works under Hyper-V as a Generation 2 guest, which is EFI-only.
So closing #844 will work as a workaround for this bug.

More good news: current Debian kernel 4.9.30-2+deb9u3 works as a Generation 1 guest in BIOS mode. So we can try building a kernel with Debian config/patches to test if RancherOS works on Hyper-V.

excellent :) now what i really need is a spare machine i don't use for other work to run Windows and Hyper-v :/

So currently, there is no way to boot v1.1.0 in latest Hyper-V at all? You either get a kernel panic in Gen1 or EFI boot error in Gen2.
Hope this will get fixed soon :hurtrealbad:.

BTW the Microsoft contributors to Linux Hyper-V support are very friendly. But they use Debian and build the kernel with default config from Debian to see if it is broken or not. So far I haven't had any time to build their suggested kernel as I couldn't have some 10GBs of disk space lol (not TBs).
So if anybody is willing to build the test kernel the way Hyper-V guys want it - I can send their instructions. You only need to be able to clean-install Debian somewhere, then I can use your kernel to test on Hyper-V.

Note that even with 1GB it doesn't kernel panic anymore, but displays another error.
It boots to shell prompt with 1280 MB of ram and displays an error with 1024.
Also, it attempts to start hyperv-vm-tools but fails with error 404. Can I load it manually?
Finally, hyperv_fb module by default makes console resolution high enough to be unusable with 1366x768 screen resolution because of constant need to scroll. This can be changed by supplying video=hyperv_fb:1024x576 kernel command line option. I wonder if there's a way to make it permanent without affecting other users.
So at least we should document the RAM and resolution issues to make experimentation easier.

You can make kernel parameters as permanent by using --append parameter on install command like this:
sudo ros install -d /dev/sda --append "video=hyperv_fb:1024x576"

Look: https://rancher.com/docs/os/v1.0/en/configuration/adding-kernel-parameters/
But probably it is better idea to use SSH than console.
What comes to that 404 on start of hyperv-vm-tools I did not see that after install to disk but I'm also not sure how to check if it has started correctly?

It is probably my particular setup - the DNS given by DHCP was invalid. So it's rather an ask for help on how to retry a failed service than a new bug report.
It's rather impossible to work and fix things in console. And SSH is up only late in the boot process, and you need to know the IP. So a console is indispensable in this case. And I get really mad with having to scroll after every command or not seeing top and bottom of vi at the same time. And just few lines of documentation will make life much easier.
Note that in case of Hyper-V it's possible to connect to a serial port using PuTTY, and you get a decent terminal (up to xterm if your getty is capable to provide it). It's a way better console than console, but again it doesn't always work so the setup is not worth it (you either get console or ssh).

I also hit this problem on hyper-v, with the latest v1.1.1-rc1 iso, rancheros wont boot in Gen2 and Gen1 still throws a kernel panic if you use memory lower than 1100 MB, so 1024 MB will still get you a kernel panic.
PS: Should the rc1 not also have docker 17.09.0-ce/17.09.1-ce in the engine list, since it was build after those versions made it to stable?

We have supported hyper-v after 1.5.0.
https://rancher.com/docs/os/v1.x/en/installation/running-rancheros/workstation/docker-machine/


the result of df is wrong when capacity of the volume is larger than 1PB

What happened: the result of df is wrong when the capacity is larger than 1PB
What you expected to happen: display the right value of used, avail and capacity.
How to reproduce it (as minimally and precisely as possible): when the capacity is larger than 1PB, run df -h
Anything else we need to know?
Environment:

JuiceFS version (use ./juicefs --version) or Hadoop Java SDK version: the latest
Cloud provider or hardware configuration running JuiceFS:
OS (e.g: cat /etc/os-release):
Kernel (e.g. uname -a):
Object storage (cloud provider and region):
Redis info (version, cloud provider managed or self maintained):
Network connectivity (JuiceFS to Redis, JuiceFS to object storage):
Others:

fixed at abcb5c652bbae8fd3862ec257d184a8d6b005b82


Limits on extended zero temps?

Do we need some sort of limit on how long oref0 will temp to zero, to avoid bad rebounds and potential ketones?

Yes. The loop can currently cause a situation where the pump is enacted to zero-temp for >2 hours, if the CWD's BG doesn't raise during the time, which causes a) ketone buildup, and b) greatly increases the risk of cannula blockages. I also suspect we should implement a "minimum basal" setting, where the loop could be configured to never zero temp for more than 30 minutes within a certain time period, after which it would set the basal to a configured minimum hourly rate, to minimize cannula blockages. These also sound like an alert state would have to be implemented in these exceptional cases, where the CWD would be alerted to eat carbs instead of relying on zero temping.

Maybe we could lower target after some period of low temping?  So we'd still temp to 0 if below 55 or some other limit, but would stop low temping when in the 70's, 80's, etc.

:+1:

I think I like the simplicity of Sulka's idea: allow a configurable limit on number of minutes of zero-temp to allow within a DIA-hour period, and a configurable minimum basal to use after that?  Would it be better to disable that by default if it's not configured, or to use some sort of default values (maybe 120m zero-temp, with a minimum of 50% of currently scheduled basal thereafter)?  Any other thoughts?

Need a volunteer to drive this, or we need to decide we're not going to do it.  Any of y'all interested?

Ketones:

Woke with heavy (70+) ketones. After this, I definitely think after some percentage of DIA has been covered, something different happens.  I usually hate on alarms, but I would be agreeable to working out some kind of alarm for this.  Maybe some notice for "ketones may be produced."

In our case since I get notifications from both openaps and nightscout via push for us if he is <= 70 we do carb correction. So alarm would work as long as it wakes u up or one of us call u.

btw any api access to nightscout for various alerts? it would be good to add if not already there so u can plug in ur method of choice pushover, azure push or ...

It could be that putting the feedback in BWP would have been enough.  The ketone thing was always lingering at the back of my mind, but I didn't realize quite what had happened until after.  (It's not displayed, bg meter 147 at 7:40pm, had four gummy bears at 6:30-ish).  The recalibration is shown ~9.

Yes. I want a configurable max time it can zero temp.

Realized that it might be more useful to redefine this as "minimum required insulin flow / time period" rather than "prevent too much zero temping". AFAIK this is easier to calculate and would serve the same purpose, right?

Gonna need one of you to volunteer to code and test this, as we never see this situation.

I'll throw my support behind this idea, as today something went very odd with my body, and I low-temped (with a small exception) for longer than my DIA.  I'm not nearly up to snuff on the code to write it though.

What was the outcome of the long low-temp? Was this overnight or daytime? What did BG and IOB look like during the low-temp period?

Well, just a few hours later, I'm now up over 200 with a bullet, so...unsure.  It was daytime, from noon on to about 6:45pm.  Here's a pic from NS:

That looks like compression low with the slow carbs catching up.  OY, unpleasant.

I don't think it's a compression low unless I'm misunderstanding what that means.  My sensor's on my stomach, and I haven't been lying down at all today or doing anything that could cause compression in that area.  But yeah, it's unpleasant - I'm definitely feeling the other side of the problem right now.  =)

Looks like the 5pm calibration may have been off. Did you do any carb corrections for the lows before the 7pm meal? Did you bolus at all for that? Wondering why it didn't high temp for the rise after the carbs...

Right around 4:00 (I think), I ate a glucose tab.  I didn't notice the lows until that point.  At 6-ish, I ate 2 glucose tabs as well.   And I just recalled - prior to that, I ate some meat and cheese.  Not to correct, but because I was a little hungry.  I suspect that's what delayed everything and caused it to shoot up a few hours later.  I've re-set my notifications to alert me sooner, so I should catch it before it gets to that point in the future.

Seems like we're converging on a consensus that the right thing to do is to notify the user they may need to eat something, and then give insulin in response to that, rather than to give insulin, force a low, and force them to eat.  If so, is that something that belongs in oref0, or elsewhere?


WebGL tutorials

Is this playlist (from 2017) still relevant enough that it would be useful to add as a side track on the website?
https://www.youtube.com/playlist?list=PLRqwX-V7Uu6bPhi8sS1hHJ77n3zRO9FR_
I could imagine redoing these and then we could replace them.
I just wrote a couple paragraphs about WEBGL for the Nature of Code book so wondering if I should reference them as an additional resource.

I think they are a nice basic intro and I can add.  Where did you add the discussion on WEBGL-- I would be interested in reading.

I am assuming that there have been updates to the library.  Adding an image as texture isn't working the same as in the tutorial (currently working on 18.6).  Ditto for graphics.  Adding text works the same (although I had to add noStroke to eliminate the wireframe, which I don't think you had.)   If I comment out  ambientLight/directionalLight, the image/graphics loads.  Also, am I correct in thinking that the issue with the createGraphics object not being to handle a non-square image has been resolved?  Am I missing something, or should I just note the changes?  (Also, do you have  the code examples somewhere?)  I have recreated one here, but wondering if you already have it in a repo. https://editor.p5js.org/kfahn/sketches/UiotWEhYM

I think they are a nice basic intro and I can add. Where did you add the discussion on WEBGL-- I would be interested in reading.

It's in the section on additive blending! https://nature-of-code-2nd-edition.netlify.app/particles/#example-49-additive-blending

I have everything done, but I wan't sure whether we were still adding the video numbers.  Are they still important?  The other thing to note is that the sketch loading the obj model gets a loadStrings warning and   I was not able to figure out why.

I don't think the numbers are necessary, no! Closing this now as it's covered by #1121! I'll look at the obj model sketch to see if I can figure that out.


BREAKING: Upgrade jest dependencies to v29

Closes #298

📦 Published PR as canary version: 0.12.0--canary.319.1d0defe.0

:sparkles: Test out this PR locally via:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

closing in favor of #348


Prepare 2.0.1 release.

Remove remaining references to prerelease status.

Coverage remained the same at 97.756% when pulling 1baeb58ec7e3bd94cf1e6de9642dc8055ef8c557 on release/2.0.1 into b2bd4b5acbcde0ae23f3a6263e0f7be6d36a9692 on master.


es2020 syntax issues when running in emulated android 9

Describe the bug
Open web3modal click on 'view more' , view more no data
SDK Version

"@web3modal/ethereum": "^2.0.0-beta.7",
"@web3modal/react": "^2.0.0-beta.7",
"wagmi": "^0.8.10"

To Reproduce
Steps to reproduce the behavior:

open web3modal
Click on 'view more'
show other wallet

Expected behavior
click view more
show other wallet
Screenshots

Smartphone (please complete the following information):

pixel 6 Pro
android 10.0 | X86
app
X86

Additional context

const { open: web3ModalOpen } = useWeb3Modal();
{!isConnected && (
<IonButton className={styles.connectBtn} onClick={() => web3ModalOpen()}>
Connect Wallet

)}

@Vivienne-CPZ do you see any error on your side when opening that view? And is PROJECT_ID populated (if it is missing, there is no way to access our explorer api).

Android 10  ABi  arm64-v8a view more is ok
but Android 10  ABi  X86 view more is blank

When you visit website on https://web3modal.com/try-it-out do you see any errors?
Judging by what you posted however Unexpected token ?, this means that your js bundler is not set up to work with es2020 syntax. Usually this is fixed via updating to lates version of tools like babel / webpack / vite etc.

visit website on https://web3modal.com/try-it-out  android 11 ok.   android 9 is error.
https://user-images.githubusercontent.com/104967115/207257506-60d5459a-6a3d-4ab9-ba74-615c4644d65f.mov

Combining all the info above, the only idea that comes to mind is that you might be using some exotic emulator browser that does not support es2020 syntax perhaps?
I also assume you are trying to run this inside Capacitor / Ionic setup, correct? Make sure you use latest versions of their webviews based on chromium and safari.
Here is one of the es2020 syntax features that we use and it's support chart as an example, is your browser within the supported range?
https://caniuse.com/?search=chainin

Android 10 ABi X86 view more will be blank， and not error message, how to fix?

Do you have a github repository I could look at to see specifics of your implementation?

https://github.com/Vivienne-CPZ/web3modal-test

We are now testing on android emulators and android has it's unique flow of how it opens wallets. Web3Modal related things seem to work great there which leads me to think that this error might be on web3modal.com page itself. We are currently rebuilding it, so will try to address, but for now closing this issue.


Create updateMoneyRequestAmountOrCurrency

Details
This PR uses the new API for updating the amount and currency of a money request.
Fixed Issues
$ https://github.com/Expensify/App/issues/32316
Tests

Have two types of money requests: manual and scan
Edit each type of request to change the amount and/or currency
Verify that those field are updated
Verify that the receipt request no longer says scanning in progress.
Start a money request from scratch and verify that you are able to select a currency and enter an amount, the correct values show up in the confirmation page and the request succeeds.

[x] Verify that no errors appear in the JS console

Offline tests

Update the amount and currency field while offline
Verify the field shows as pending (greyed out)
Go online and verify the field is updated

QA Steps
Same as tests

[ ] 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:

[x] Android: Native
[x] Android: mWeb Chrome
[x] 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 approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[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 form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label 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

Reviewer Checklist

[ ] I have verified the author checklist is complete (all boxes are checked off).
[ ] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the steps for local testing are in the Tests section
[ ] I verified the steps for Staging and/or Production testing are in the QA steps section
[ ] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] 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)

[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I included screenshots or videos for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

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

[ ] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] 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).
[ ] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[ ] I verified that comments were added to code that is not self explanatory
[ ] 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.
[ ] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[ ] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[ ] 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 approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[ ] 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.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] 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
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[ ] If any new file was added I verified that:

[ ] 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

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

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

[ ] 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.
[ ] 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)
[ ] 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.
[ ] 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.
[ ] 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.
[ ] 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.
[ ] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

@luacmartins During testing, when I try to create a manual request, I get NaN as the amount on the preview page.
https://github.com/Expensify/App/assets/4160319/622d426e-9999-431c-a569-f8e53cebf961

Ah nice catch! I updated the code to fix the issue and added test steps for that. Thank you!

Bump for review @akinwale

Bump for review @akinwale

I will review today.

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/luacmartins in version: 1.4.17-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
cancelled 🔪

🕸 web 🕸
success ✅

Just leaving a comment here to note that this PR caused this regression: https://github.com/Expensify/App/issues/33587

🚀 Deployed to production by https://github.com/mountiny in version: 1.4.17-8 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


setting eve.tag and eve.registry for eden tests for PR's build

A different repository is used for the eve PRs after they have been built. Although this value is present in the tag that Eve passes, we still need to extract the necessary data. Using the fundamental shell commands, this PR will retrieve the necessary data.

I see a no space left on device in the failure

That's interesting @eriknordmark , because we are still not using this workflows, we run them manually till they are successful (and only then we want to use them in PRs). So this patch does not affect other workflows running (including eden_setup)

I was referring to the error for this PR in https://github.com/lf-edge/eden/actions/runs/5681201583/job/15396839195?pr=881
And they are being used in EVE (and all failing). See https://github.com/lf-edge/eve/actions/workflows/eden_test.yml
I assumed that was known

Reference

Yes, the Test from lf-edge/eden WF is being used by eve to test the reusable workflow functionality. Once this PR is merged. I will raise a PR that will fix eve.


Add a Link to Mirror Site in China

China: https://oasis-engine.gitee.io/#/
Outside: https://oasis-engine.cn

Mirror site: https://oasis-engine.gitee.io/


The image automatically fits in the middle of the canvas

Hi,
I want to realize that the picture automatically adapts to the canvas, so that the picture is in the middle of the canvas, is there a way to get the width and height of the RichItemsControl or other methods to achieve this function?
whether I can do it with an extension method that changes coordinates similar to what ShouldBringIntoView does (as the MakeVisible method does)
Trouble you again,Thank you so much.
Best Wishes!

Hi,
You should be able to get the ActualWidth and ActualHeight from the RichItemsControl.
Also you can wrap your RichItemsControl in another Panel or any other object and use that size.
Anyway, I will expose a property on RichItemsControl to see the current Viewport size.
Thanks.


Project Management Console - Ascending order sort issue in View Signatures Popup Window

Launch Project Management Console
Login as Project Manager
Click on the External Project(Or Foundation)
Click on View Signatures for a Project(CLA Group) which has signatures
Click on any column for e.g Company
Data is sorted in ascending order
Now click again on the column
Data is sorted in descending order
Now click again on the column
Observed no action is performed
Now click again and data is sorted in ascending order.
Issue observed for all columns

@hnexokonkwo please review and confirm. If resolved, ask @nirupamav to re-test.

@nirupamav issue resolved. PLea re-test

Verified and observed that sort by company is not working as expected. Until this issue is fixed cannot verify this ticket
By Ascending Order

By Descending Order

@nirupamav This issue was due to the list not having company name in individual object. This has been fixed and I will send a PR soon.

@nirupamav please retest

Verified and observed that the issue is fixed


CheckList

Here are all the commands in 1.59. We can use this tracking bug to track implementation of the rest of them in the tpmdirect branch.
0: Special Case, unsure how to deal with it
1-8: Difficulty at first glance
N/A: Could not find in Part 3
RE: Response Empty
CE: Command Empty

[ ]  AC_GetCapability - N/A
[ ]  AC_Send - N/A
[ ]  ACT_SetTimeout -N/A
[ ]  ActivateCredential - 4
[x]  Certify - 2
[ ]  CertifyCreation - 3
[ ]  CertifyX509
[ ]  ChangeEPS - 0, RE 2
[ ]  ChangePPS -0, RE 2
[x]  Clear - 0, RE 2
[ ]  ClearControl - 0, RE 2
[ ]  ClockRateAdjust - 0, RE 1
[ ]  ClockSet - 0, RE 1
[ ]  Commit - 3
[x]  ContextLoad - 1
[x]  ContextSave - 1
[d]  Create
[d]  CreateLoaded
[d]  CreatePrimary
[ ]  DictionaryAttackLockReset - 0, RE 1
[ ]  DictionaryAttackParameters - 0, RE 2
[ ]  Duplicate - 3
[ ]  EC_Ephemeral - 2
[ ]  ECC_Parameters - 1
[ ]  ECDH_KeyGen - 2
[ ]  ECDH_ZGen - 2
[ ]  EncryptDecrypt - 3
[ ]  EncryptDecrypt2 - 3
[ ]  EventSequenceComplete - 3
[ ]  EvictControl  - 0, RE 1
[ ]  FieldUpgradeData - 0, RE 3
[ ]  FieldUpgradeStart - 0, RE 4
[ ]  FirmwareRead - 1
[d]  FlushContext
[d]  GetCapability
[ ]  GetCommandAuditDigest - 4
[x]  GetRandom
[d]  GetSessionAuditDigest
[ ]  GetTestResult 0, CE 1
[ ]  GetTime - 2
[x]  Hash - 3
[x]  HashSequenceStart - 3
[ ]  HierarchyChangeAuth - 0, RE 1
[ ]  HierarchyControl - 0, RE 2
[ ]  HMAC - 1
[ ]  HMAC_Start - 2
[ ]  Import - 5
[ ]  IncrementalSelfTest - 1
[d]  Load
[d]  LoadExternal
[ ]  MAC - N/A
[ ]  MAC_Start - N/A
[ ]  MakeCredential - 2
[ ]  NV_Certify - 4
[ ]  NV_ChangeAuth - 0, RE 2
[d]  NV_DefineSpace - 0, RE 2
[ ]  NV_Extend - 0, RE 1
[ ]  NV_GlobalWriteLock - 0, RE 1
[ ]  NV_Increment - 0, RE 2
[d]  NV_Read
[ ]  NV_ReadLock - 0, RE 3
[d]  NV_ReadPublic - 0, RE 3
[ ]  NV_SetBits - 0, RE 1
[d]  NV_UndefineSpace
[d]  NV_UndefineSpaceSpecial
[d]  NV_Write
[d]  NV_WriteLock
[ ]  ObjectChangeAuth - 1
[ ]  PCR_Allocate - 2
[d]  PCR_Event
[d]  PCR_Extend
[d]  PCR_Read
[ ]  PCR_Reset - 0 RE, 1
[ ]  PCR_SetAuthPolicy  - 0, RE 1
[ ]  PCR_SetAuthValue - 0, RE 1
[ ]  Policy_AC_SendSelect - NA
[d]  PolicyAuthorize
[d]  PolicyAuthorizeNV
[ ]  PolicyAuthValue - 2
[d]  PolicyCommandCode
[ ]  PolicyCounterTimer - 0, RE 3
[d]  PolicyCpHash
[ ]  PolicyDuplicationSelect - 0, RE 2
[d]  PolicyGetDigest
[ ]  PolicyLocality - 0, RE 2
[ ]  PolicyNameHash - 0, RE 2
[ ]  PolicyNV - 0, RE 4
[d]  PolicyNvWritten
[d]  PolicyOR
[ ]  PolicyPassword - 0, RE 2
[x]  PolicyPCR - 0, RE 3
[ ]  PolicyPhysicalPresence - 0, RE 2
[ ]  PolicyRestart - 0, RE 2
[d]  PolicySecret
[d]  PolicySigned
[ ]  PolicyTemplate 0, RE 2
[ ]  PolicyTicket 0, RE 4
[ ]  PP_Commands 0, RE 3
[d]  Quote
[ ]  ReadClock 0, CE 1
[x]  ReadPublic
[ ]  Rewrap 3
[ ]  RSA_Decrypt 3
[ ]  RSA_Encrypt 3
[ ]  SelfTest - 0, RE 2
[x]  SequenceComplete - 3
[x]  SequenceUpdate - 0 RE 3
[ ]  SetAlgorithmSet - 0 RE 2
[ ]  SetCommandCodeAuditStatus - 0, RE 3
[ ]  SetPrimaryPolicy - 0, RE 3
[d]  Shutdown
[x]  Sign - 2
[d]  StartAuthSession
[d]  Startup
[ ]  StirRandom - 0, RE 1
[ ]  TestParms - 0, RE 1
[d]  Unseal
[d]  VerifySignature
[ ]  ZGen_2Phase - 4

[ ] ActivateCredential
[ ]  Certify
[ ]  CertifyCreation
[ ]  Clear
[ ]  Create
[ ]  CreateLoaded
[ ]  CreatePrimary
[ ]  ECDH_ZGen
[ ]  FlushContext
[ ]  GetCapability
[ ]  GetSessionAuditDigest
[ ]  Load
[ ]  NV_Certify
[ ]  NV_DefineSpace
[ ]  NV_Increment
[ ]  NV_Read
[ ]  NV_ReadPublic
[ ]  NV_UndefineSpace
[ ]  PCR_Event
[ ]  PCR_Extend
[ ]  PCR_Read
[ ]  PCR_Reset
[ ]  PolicyCommandCode
[ ]  PolicyGetDigest
[ ]  PolicyNV
[ ]  PolicyOR
[ ]  PolicyPCR
[ ]  PolicySecret
[ ]  Quote
[ ]  ReadPublic
[ ]  Sign
[ ]  StartAuthSession
[ ]  Startup
[ ]  Unseal


A Start Of Project

A Infinity Cloud Has Been Started!

Make own portfolio without learning programming and hiring programmers! Get started with https://infi-cloud.ml

use my portfolio website as an Inspiration: davidmicahdaniels.netlify.app


In WriteCommandResult, the 'nModified' could be either Long or Integer

I have a GRAILS project based on the GRAILS 2.3.11 framework. This includes mongo-java-driver-2.14.2, and we can't easily upgrade to 3.x (where this issue seems to be fixed).
I'm build a proof-of-concept using Azure CosmosDB with MongoDB 3.2.0 API, and ran into a problem, 'java.lang.ClassCastException: java.lang.Long cannot be cast to java.lang.Integer' at getModifiedCount() in WriteResultCommandHelper.java.
This pull request fixes this particular problem. Please release java-mongo-driver-2.14.4 with this PR.

Hi @ToniA,
Thanks for the ticket, there are no planned releases for the 2.14 version of the driver, the last release was over two years ago. Please look to migrating over to the 3.x series of the MongoDB Java driver. As this not only contains bugfixes but also contains many improvements to the API as well as access to many of MongoDB's new features.
Because there are no planned 2.14.x releases and as this is an issue outside MongoDB, I'm closing this ticket as won't fix.
All the best,
Ross


Convert from async/await over to promise on query return

Convert from async/await over to promise on query return, see players.json for examples

no change needed, async/await seems to be ES7, using .exec is ES6


Test tracking will not move the 3d model in ps config tool

When trying to compute tracking poses, my HMD gets detected fine(since hipstersloth reminded me i need to have SteamVr running), but while doing the test with the move controller attached to my hmd, or even doing the calibration mat method, the camera will not detect the controller itself, i see it in the camera feed and the tool says that im holding it upright, but once it gets to taking samples, it sticks at 0/60

Have you used the color calibration tool to make sure it's capturing the psmove sphere correctly?

took me acouple re-reads, but yes, the settings i had were not showing anything while 'masked' view was on, after tinkering with it, i figured it out(:


Daily node status overview

Daily report for Node Monitor
Have a daily report sent to the node provider.
Email contents:

[x] subject: (status) Daily Node Report
[x] list how many nodes up, unassigned, down... (current overview looks good)
[ ] list how many (ghots) outages there were that day?

list how many (ghots) outages there were that day?

Moved to separate issue

[ ] error handling to make sure interval for report is always greater than interval for snapshots
[ ] if interval for report is null, don't send report


test: behaviors-message_on_lock_failure (2024-03-04 18:21:54.102048+00:00)

automatic test cleanup


Provide workflow's status command

Using an admin console/UI, we can check workflow's current execution status.

Addressed by #139 #143 #144 #147


Logical Negation(! contains, !Startswith , !endswith) in Filter Operators

Hi,
Does Filter Model supports logical negation for Contains, Startswith and endswith filter operators.
https://sapui5.hana.ondemand.com/sdk/#/api/sap.ui.model.FilterOperator/properties
I see there is no direct opertor for the above operators like Contains, Startswith and endswith.
Example : If i want to filter Product name does not endswith "ilk"
/Products?$filter=not endswith(Name, 'ilk').
Regards
Karthik S

Seems to be a duplicate of https://github.com/SAP/openui5/issues/1512

Yes, it's a duplicate - closed.


Updated unit tests and changed filepaths in parse_uw_alerts.py

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

5 of 22 (22.73%)  changed or added relevant lines in 1 file are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.6%) to 59.659%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

uw-alert-web/parse_uw_alerts/parse_uw_alerts.py
5
22
22.73%

Files with Coverage Reduction
New Missed Lines
%

uw-alert-web/parse_uw_alerts/parse_uw_alerts.py
4
32.54%

Totals

Change from base Build<PHONE_NUMBER>:
-0.6%

Covered Lines:
210

Relevant Lines:
352

💛  - Coveralls


build node.js v4, v6

Adding builds for the LTS node versions

Coverage remained the same at 94.068% when pulling 2383a3ca7cabb54a5eb5aabab0a31e8fdf1b4e6d on MarkHerhold:travis-builds into bd58e621b1fd1f927ee936199f3ccadb780dd5f3 on blakeembrey:master.


Test instructions for running WST Drupal Site

💡 Description
See emails from Paul P. re-access to GitHub repo and deployment.
⚔ Parent Epic / Related Tickets
No response

@jordanpadams could you copy/paste these emails from Paul P. into this ticket (or just forward them to my account if they're private)?
(Side note: who is Paul P.?)

Instructions should be here: https://github.com/Web-Strategy-Team/Docker-Drupal-Demo/

@nutjob4life ☝

Thanks @jordanpadams!
I'm starting with:
⬇ Step 1: Clone the Repository
The instructions state:
git clone ssh://git@bitbucket.gsfc.nasa.gov:7999/wst/docker-drupal.git

This fails outside of JPL full tunnel VPN since the host bitbucket.gsfc.nasa.gov won't resolve, which makes sense.
After turning the JPL full tunnel VPN on, bitbucket.gsfc.nasa.gov does resolve, but connections to port 7999 time out. GSFC must be blocking access from JPL to port 7999, so no git clone using ssh protocol.
On the outside chance they have their BitBucket set up similarly to GitHub (with both ssh and https) access, I tried
git clone https://bitbucket.gsfc.nasa.gov/wst/docker-drupal.git

But that just gives a 404. (You can visit https://bitbucket.gsfc.nasa.gov/ with a browser and get redirected to NASA IdMAX login; but after logging in I just get an error, presumably because I'm not at GSFC.)
Anyway, that's as far as I can get. Hope this helps!

@nutjob4life some background, the repo at https://github.com/Web-Strategy-Team/Docker-Drupal-Demo/ is a fork of that bitbucket. So anywhere you see anything in those instructions related to bitbucket, so assume they mean the github repo you are at: https://github.com/Web-Strategy-Team/Docker-Drupal-Demo/

Okay, @jordanpadams , I interpreted testing the instructions meant literally following them to a T.
But with the caveat in mind that they haven't updated them after publishing to "GitHub", here's what I get:
⬇ Step 1: Clone the Repository
Using:
git clone https://github.com/Web-Strategy-Team/Docker-Drupal-Demo.git

🔧 Step 2: Create the .env File
Easy enough:
cp .env.example .env

🔧 Step 3: Replacements (replace_me)
Replaced replace_me with various values—although the instructions give specifics for those in the next step.
🔧 Step 4: Configure .env Variables
Replaced specific .env variables as needed.
🔌 Step 5: Install Docker Drupal
Running
scripts/build.sh -l

This fails with:
Copying dummy default certs...
No settings.local.php file found. Copying project.settings.local.php to settings.local.php
1+0 records in
1+0 records out
32 bytes transferred in 0.000066 secs (484848 bytes/sec)
== Checking for existing containers...
Running: docker-compose --file docker-compose.yml --file docker-compose-services.yml --file docker-compose-dev-tools.yml --file docker-compose-local.yml  up --build --detach
scripts/docker-compose.sh: line 80: docker-compose: command not found

Instead of stopping here, let's dig deeper: apparently build.sh calls docker-compose.sh which wraps docker-compose so luckily there's one place to edit the call to the more modern docker compose.
Now … it's taking quite a bit of time! But failed with
failed to create network drupal_network: Error response from daemon: invalid pool request: Pool overlaps with other one on this address space

The README recommends increasing the subnet number by 1 in case it collides with another project, which I guess it does on my Mac Studio. Trying again!
After another long wait, there's:
…
…
…
 [success] The configuration was imported successfully.
 [success] Added administrator role to admin
 [success] Created a new user with uid 2
 --------- ----------- ----------- --------------- -------------
  User ID   User name   User mail   User roles      User status
 --------- ----------- ----------- --------------- -------------
  2         auth_user               authenticated   1
 --------- ----------- ----------- --------------- -------------
== Drupal Reset Complete...
You can login with u:admin p:F@rTmaguk3TF!hFl1nGeR
== Memory Limit...
1024M
yarn install v1.22.22
[1/5] Validating package.json...
[2/5] Resolving packages...
[3/5] Fetching packages...
[4/5] Linking dependencies...
[5/5] Building fresh packages...
Done in 36.70s.

         _/                                          _/
    _/_/_/  _/  _/_/  _/    _/  _/_/_/      _/_/_/  _/
 _/    _/  _/_/      _/    _/  _/    _/  _/    _/  _/
_/    _/  _/        _/    _/  _/    _/  _/    _/  _/
 _/_/_/  _/          _/_/_/  _/_/_/      _/_/_/  _/
                            _/
                           _/
== Build complete
You can reach your site under: https://localhost:8443
==========
You are ready to get started!
==========

Visiting https://localhost:8443 gives this:

The README says you can log into https://localhost:8443/user/login with

Username: admin
Password: password

But that's not correct. Instead, you need to look at the output of build.sh -l for the correct, randomly-generated password.
Solr is available at http://localhost:8983/ as advertised, as is phpMyAdmin at http://localhost:8181
I ran the "Behat" tests and they all passed too (although there are just 3 scenarios of tests included).
Hope this helps!

@nutjob4life looks great! will assign next tasks from parent task: https://github.com/NASA-PDS/portal-wp/issues/106

@jordanpadams superb, thanks!
Okay to go ahead and close this one?

Without reviewing @nutjob4life's notes prior to trying myself, I had the same experience Sean had with standing up the site, and other than fixing the docker compose command, the site just ran on my machine....
== Drupal Reset Complete...
You can login with u:admin p:*****************************
== Memory Limit...
1024M
yarn install v1.22.22
[1/5] Validating package.json...
[2/5] Resolving packages...
[3/5] Fetching packages...
[4/5] Linking dependencies...
[5/5] Building fresh packages...
Done in 31.26s.

    _/_/_/    _/_/_/      _/_/_/  _/_/_/_/  _/      _/
   _/    _/  _/    _/  _/        _/        _/_/    _/
  _/_/_/    _/    _/    _/_/    _/_/_/    _/  _/  _/
 _/        _/    _/        _/  _/        _/    _/_/
_/        _/_/_/    _/_/_/    _/_/_/_/  _/      _/

== Build complete
You can reach your site under: https://localhost:8443
==========
You are ready to get started!
==========
anatha@MT-407379 Docker-Drupal-Demo %

Since we were able to follow the instructions on the demo site, I think we could close this ticket, what do you think @jordanpadams ?
The documentation does state not to use this code for a production site, so I think we'll need to have a discussion with Paul about their plans for integrating changes and if this repo will eventually contain a stable production ready codebase, but for now, it looks like it's enough to get us started

Testing completed. Done.


Change User Display When Creating/Modifying Repository

Hi,
I'm wondering if someone might know where to look and modify what is displayed on the Create/Modify repositories page for user delegation as contributor or administrator. I've looked in the database tables, but their must be some ASP work being down to retrieve user values. I hope to set the display to be Lastname, FirstName instead of username
Our facility uses two factor authentication, with the Active Directory usernames setup as their smartcard UPN number, not their legacy firstinitiallastname. Administrative account still use a readable username starting with ap., but normal users are set to UPN.
Thanks!

Same as #295


Restless Star with negative (-) active videos

Hi! The Restless Star site should be completed, but the summary page display negative active videos: "active":-9," since a few minutes. I got a "We're out of data" message briefly, but then it switched to another site. Is this a problem of the paused videos at Restless Star?

Sometimes the system loses count! I've corrected that now and marked the group as complete.


parent:: completion doesn't work

class SomeParent
{
    public function test(){}
    protected function testProtected(){}
    private function testPrivate(){}
}

class SomeChild extends SomeParent
{
    public function test()
    {
          parent::
    }
}

should return test and testProtected

Is there a chance to fix it?

@golaod I haven't started working on it yet. So you are welcome to send PR :)

please try #84


Add C-h as delete word hotkey

Closes #50

You offer to add option for choosing word delete hotkey? Like add some config file and set it's value there. In theory config file seems good, because it will be possible to add the feature to change any hotkey

No, I meant that Key::Ctrl('h') should behave like Key::Backspace and not Key::Ctrl('w')


[Feature Request] - Ability to customize the prompts

It would be great to have the ability to customize the prompts, or at least add lines to the initial description for extra guidance. For example, for the code suggestion prompt, users might want to add an extra line for telling the LLM to explicitly suggest type hints, possible unit tests, etc.

+1 The automatic reviews are already great but they would potentially be even better by adding a high-level description of the project the code change is being applied to along with other customizations like you mentioned.

Hi @georgeseifada, thanks for the feedback
Because we use a dedicated JSON format, our tools are customizable and adjustable.
In the configuration files you can control each tool. For example, pr-reviewer (/review):

This configuration enables you to turn on/off different fields.
The 'score' field, for example, was added in the PR: https://github.com/Codium-ai/pr-agent/pull/89, and you can see it was quite easy and straightforward to add.  Other fields can also be added easily
In addition,  regarding your suggestion for adding possible "extra instructions" for code suggestion - good idea :-)
So a possible format will be:
/imporve -extra_instruction="focus on the file ..."

been addressed in
https://github.com/Codium-ai/pr-agent/pull/159


New Paloma Landing Page

As a validator, developer, or investor of Paloma, I want to be able to land on a website where I can see the Paloma value propositions, use cases, and useful links to documents.
Follow Designs: https://www.figma.com/file/ClniQyKFqN1hSpWizV5O4A/Paloma?node-id=1676%3A25
1. Nav Bar:   (Still under the production of the submenus for the nav bar items)

[ ] Logo: https://drive.google.com/drive/u/1/folders/1OEA4oi87F06yySw0BAq5LZ_hloK0BFEH

[ ] Join our Community: We will redirect the user to the new Discord channel.
link: https://discord.gg/UmYfqbk4VZ

While hovering the button will change to: https://www.figma.com/file/ClniQyKFqN1hSpWizV5O4A/Paloma?node-id=1903%3A757

2. First Section: "The fastest Messenger Blockchain"

[x] Diagram here: https://drive.google.com/drive/u/1/folders/1OEA4oi87F06yySw0BAq5LZ_hloK0BFEH
[ ] Get Started -->: Link: https://palomachain.github.io/paloma-docs/guide/develop/quick-start/quick-start.html

While hovering the Get Started button will change to: https://www.figma.com/file/ClniQyKFqN1hSpWizV5O4A/Paloma?node-id=1903%3A1010

Section divider: "We go in all directions".

[ ] Footprints icon here:  https://drive.google.com/drive/u/1/folders/1OEA4oi87F06yySw0BAq5LZ_hloK0BFEH

Permissionless Cross Chain:

[x] Diagram 2: https://drive.google.com/drive/u/1/folders/1OEA4oi87F06yySw0BAq5LZ_hloK0BFEH

Value Propositions: "Send Messages Quickly & Safely"

[x] Icons: https://drive.google.com/drive/u/1/folders/1OEA4oi87F06yySw0BAq5LZ_hloK0BFEH
[ ] While hovering the rectangle behind all the value propositions will turn to a different color. Figma: https://www.figma.com/file/ClniQyKFqN1hSpWizV5O4A/Paloma?node-id=1903%3A1038

[ ] The pink text should be "Paloma Value Propositions"

[ ] How to build on paloma section doesn't match design and button to "view docs" button is not clickable

[ ] Shrink the paloma footprints. Following: https://www.figma.com/file/ClniQyKFqN1hSpWizV5O4A/Paloma?node-id=1967%3A117


Add reconnect button @ postges

When you run the query, an error occurs if the connection to the database was lost due to a server restart.
In this screen we need add reconnect button.
Step to reproduce:

open connection
restart db server
try execute query

postgres 9.6 \ dbeaver 5.1.2

I can't reproduce this (I think this happens only if shutdown operation longs for some time) but I'll add this error code in "connection lost" error code list.

Thank you!


Open search results in new tab

Prerequisites
affected version: 1.24.0
Would be great to have a button, which allows exporting search result from Tab A as the content of Tab B.

not directly but almost:
in chipmunk 3 you can export all search results in a file:


publish<EMAIL_ADDRESS>

Requested by: @marandaneto
Quick links:

View changes
View check runs

Assign the accepted label to this issue to approve the release.
Targets

[x] maven
[ ] github
[ ] registry

Targets marked with a checkbox have already been executed.  Administrators can manually tick a checkbox to force craft to skip it.

Merged tagged/github release/merged branch to main manually


Add .vscode to gitignore

Maybe you need add .vscode to gitignore

Ohh yes, you are right

Ok Google now >:D

Done! :dancers:


UDP Port for Go1 Low Level Control

For v.3.8.6 of the Go1 SDK, In the example for C++, the local port for UDP used is 8090, while for the Python example is 8080. Which one should I use? I am afraid it may cause connection problems in the future.
Additionally, I am experiencing a weird behavior. When I set a static IP in my external PC for connecting with Go1 to <IP_ADDRESS>, the UDP returns no message. However, when I switch the IP to <IP_ADDRESS>, and then revert back to <IP_ADDRESS>, the connection was established. I this a bug, or did I miss something here?
Note: I always use <IP_ADDRESS> for connecting to A1 with the same PC, and there is no such problem
Thanks in advance for your assistance!

Hello! For the first question, does it not work when using the default settings? The second question, according to the network framework of the Go1 robot, it is possible to set <IP_ADDRESS> for the external computer. Did you ensure that it takes effect after setting it?