Regarding mail system

Hi Rohit ,
thanks for making this library . i have a question like do this library has the customization option in mail feature as well .like if i want to trigger a mail for errors is that feature there?

Hi Rikesh, the emails are sent for critical logs.
Can you explain why you want the feature to configure the mail sending for error?
You can achieve the same task if you can call the critical instead of the error while using the library.
If you want to send emails, simply use critical-
try:
    # statement which raises an exception/error
    z = 1/0
except:
    my_logger_obj.critical('Mail will be sent for this log message')

Help me understand the issue better.
I'm open to adding new features to the library, provided they have a generic use case.

Thanks Rohit I was trying the same way by using critical but there is some with SMTP will attach the error logs later
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Rohit Lal @.>
Sent: Thursday, August 25, 2022 11:01:51 PM
To: rohitlal125555/autopylogger @.>
Cc: Chhetri, Rikesh - @.>; Author @.>
Subject: Re: [rohitlal125555/autopylogger] Regarding mail system (Issue #1)
Hi Rikesh, the emails are sent for critical logs.
Can you explain why you want the feature to configure the mail sending for error?
You can achieve the same task if you can call the critical instead of the error while using the library.
If you want to send emails, simply use critical-
try:
# statement which raises an exception/error
z = 1/0
except:
my_logger_obj.critical('Mail will be sent for this log message')
Help me understand the issue better.
I'm open to adding new features to the library, provided they have a generic use case.
—
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https://github.com/rohitlal125555/autopylogger/issues/1*issuecomment-1227568776__;Iw!!N11eV2iwtfs!sKsUs8suVQJ_JWVPooKUrXZ49NZ-F26gBiBlysopHF5ft4uxCKpg9ViK2xH8Y-EOJXK8IoMeYNtOrPXWnhfBt_U$, or unsubscribehttps://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AZ5CMLBTQS2RBAP4RWNFTY3V26UYPANCNFSM57N645OQ__;!!N11eV2iwtfs!sKsUs8suVQJ_JWVPooKUrXZ49NZ-F26gBiBlysopHF5ft4uxCKpg9ViK2xH8Y-EOJXK8IoMeYNtOrPXWkW-zwcQ$.
You are receiving this because you authored the thread.Message ID: @.***>

The contents of this email and any attachments are confidential and intended only for the person or entity to whom it is addressed. If you are not the intended recipient, any use, review, distribution, reproduction or any action taken in reliance upon this message is strictly prohibited. If you received this message in error, please immediately notify the sender and permanently delete all copies of the email and any attachments.
Click here for translations of this disclaimer.https://secure.wiley.com/email-disclaimers

but there is some with SMTP will attach the error logs later

I'm not sure what you meant by that ?
On Thu, Aug 25, 2022 at 11:10 PM ChhetriRikesh @.***>
wrote:

Thanks Rohit I was trying the same way by using critical but there is some
with SMTP will attach the error logs later
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Rohit Lal @.>
Sent: Thursday, August 25, 2022 11:01:51 PM
To: rohitlal125555/autopylogger @.>
Cc: Chhetri, Rikesh - @.>; Author @.>
Subject: Re: [rohitlal125555/autopylogger] Regarding mail system (Issue #1)
Hi Rikesh, the emails are sent for critical logs.
Can you explain why you want the feature to configure the mail sending for
error?
You can achieve the same task if you can call the critical instead of the
error while using the library.
If you want to send emails, simply use critical-
try:
statement which raises an exception/error
z = 1/0
except:
my_logger_obj.critical('Mail will be sent for this log message')
Help me understand the issue better.
I'm open to adding new features to the library, provided they have a
generic use case.
—
Reply to this email directly, view it on GitHub<
https://urldefense.com/v3/__https://github.com/rohitlal125555/autopylogger/issues/1*issuecomment-1227568776__;Iw!!N11eV2iwtfs!sKsUs8suVQJ_JWVPooKUrXZ49NZ-F26gBiBlysopHF5ft4uxCKpg9ViK2xH8Y-EOJXK8IoMeYNtOrPXWnhfBt_U$>,
or unsubscribe<
https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AZ5CMLBTQS2RBAP4RWNFTY3V26UYPANCNFSM57N645OQ__;!!N11eV2iwtfs!sKsUs8suVQJ_JWVPooKUrXZ49NZ-F26gBiBlysopHF5ft4uxCKpg9ViK2xH8Y-EOJXK8IoMeYNtOrPXWkW-zwcQ$

.
You are receiving this because you authored the thread.Message ID:
@.***>

The contents of this email and any attachments are confidential and
intended only for the person or entity to whom it is addressed. If you are
not the intended recipient, any use, review, distribution, reproduction or
any action taken in reliance upon this message is strictly prohibited. If
you received this message in error, please immediately notify the sender
and permanently delete all copies of the email and any attachments.
Click here for translations of this disclaimer.<
https://secure.wiley.com/email-disclaimers>

—
Reply to this email directly, view it on GitHub
https://github.com/rohitlal125555/autopylogger/issues/1#issuecomment-1227576913,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF4DCWGEE2SXUUXL7QQTF5DV26VYTANCNFSM57N645OQ
.
You are receiving this because you commented.Message ID:
@.***>

Actually  the program is unable to send the mail
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Rohit Lal @.>
Sent: Thursday, August 25, 2022 11:17:27 PM
To: rohitlal125555/autopylogger @.>
Cc: Chhetri, Rikesh - @.>; Mention @.>
Subject: Re: [rohitlal125555/autopylogger] Regarding mail system (Issue #1)

but there is some with SMTP will attach the error logs later

I'm not sure what you meant by that ?
On Thu, Aug 25, 2022 at 11:10 PM ChhetriRikesh @.***>
wrote:

Thanks Rohit I was trying the same way by using critical but there is some
with SMTP will attach the error logs later
Get Outlook for Androidhttps://aka.ms/AAb9ysghttps://urldefense.com/v3/__https://aka.ms/AAb9ysg*3E__;JQ!!N11eV2iwtfs!ogHyVGb9gnRX_DGgK004OQOkLir6Q5Js_FRxAJcaja5jgOuZ7T3DR_mAcVfyzVi_La4n9cbFquf1n_-cIVUVuMk$

From: Rohit Lal @.>
Sent: Thursday, August 25, 2022 11:01:51 PM
To: rohitlal125555/autopylogger @.>
Cc: Chhetri, Rikesh - @.>; Author @.>
Subject: Re: [rohitlal125555/autopylogger] Regarding mail system (Issue #1)
Hi Rikesh, the emails are sent for critical logs.
Can you explain why you want the feature to configure the mail sending for
error?
You can achieve the same task if you can call the critical instead of the
error while using the library.
If you want to send emails, simply use critical-
try:
statement which raises an exception/error
z = 1/0
except:
my_logger_obj.critical('Mail will be sent for this log message')
Help me understand the issue better.
I'm open to adding new features to the library, provided they have a
generic use case.
—
Reply to this email directly, view it on GitHub<
https://urldefense.com/v3/__https://github.com/rohitlal125555/autopylogger/issues/1*issuecomment-1227568776__;Iw!!N11eV2iwtfs!sKsUs8suVQJ_JWVPooKUrXZ49NZ-F26gBiBlysopHF5ft4uxCKpg9ViK2xH8Y-EOJXK8IoMeYNtOrPXWnhfBt_U$>,
or unsubscribe<
https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AZ5CMLBTQS2RBAP4RWNFTY3V26UYPANCNFSM57N645OQ__;!!N11eV2iwtfs!sKsUs8suVQJ_JWVPooKUrXZ49NZ-F26gBiBlysopHF5ft4uxCKpg9ViK2xH8Y-EOJXK8IoMeYNtOrPXWkW-zwcQ$

.
You are receiving this because you authored the thread.Message ID:
@.***>

The contents of this email and any attachments are confidential and
intended only for the person or entity to whom it is addressed. If you are
not the intended recipient, any use, review, distribution, reproduction or
any action taken in reliance upon this message is strictly prohibited. If
you received this message in error, please immediately notify the sender
and permanently delete all copies of the email and any attachments.
Click here for translations of this disclaimer.<
https://secure.wiley.com/email-disclaimers>

—
Reply to this email directly, view it on GitHub
https://github.com/rohitlal125555/autopylogger/issues/1#issuecomment-1227576913https://urldefense.com/v3/__https://github.com/rohitlal125555/autopylogger/issues/1*issuecomment-1227576913*3E__;IyU!!N11eV2iwtfs!ogHyVGb9gnRX_DGgK004OQOkLir6Q5Js_FRxAJcaja5jgOuZ7T3DR_mAcVfyzVi_La4n9cbFquf1n_-cn7hUnl4$,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF4DCWGEE2SXUUXL7QQTF5DV26VYTANCNFSM57N645OQhttps://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AF4DCWGEE2SXUUXL7QQTF5DV26VYTANCNFSM57N645OQ*3E__;JQ!!N11eV2iwtfs!ogHyVGb9gnRX_DGgK004OQOkLir6Q5Js_FRxAJcaja5jgOuZ7T3DR_mAcVfyzVi_La4n9cbFquf1n_-cAIkVlrc$
.
You are receiving this because you commented.Message ID:
@.***>

—
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https://github.com/rohitlal125555/autopylogger/issues/1*issuecomment-1227582868__;Iw!!N11eV2iwtfs!ogHyVGb9gnRX_DGgK004OQOkLir6Q5Js_FRxAJcaja5jgOuZ7T3DR_mAcVfyzVi_La4n9cbFquf1n_-crrId9GA$, or unsubscribehttps://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AZ5CMLH2E2VA4ARTSSWXF33V26WS7ANCNFSM57N645OQ__;!!N11eV2iwtfs!ogHyVGb9gnRX_DGgK004OQOkLir6Q5Js_FRxAJcaja5jgOuZ7T3DR_mAcVfyzVi_La4n9cbFquf1n_-cX2IhoEg$.
You are receiving this because you were mentioned.Message ID: @.***>

The contents of this email and any attachments are confidential and intended only for the person or entity to whom it is addressed. If you are not the intended recipient, any use, review, distribution, reproduction or any action taken in reliance upon this message is strictly prohibited. If you received this message in error, please immediately notify the sender and permanently delete all copies of the email and any attachments.
Click here for translations of this disclaimer.https://secure.wiley.com/email-disclaimers

This could possibly be due to the improper mail configuration.
Share with me the code snippet that you are using and also the error stack trace that you're getting.


Statinamic is now Phenomic

https://phenomic.io/
For information about the change https://github.com/MoOx/statinamic/issues/306

Thanks!


The tooltip of controls present on 'Iris the Bot' dialog is not hover able.

Is it an issue related to Adaptive Cards?
No.
What is the PWD impact?
If the tooltip of controls present on 'Iris the Bot' dialog is not hover able and when large pointers have been selected via platform settings or assistive technology, the pointer can obscure a significant area of the additional content. This will impact the users who utilize screen reader feedback on mouse interactions.
What browsers and screen readers do this issue affect?
Windows: Chrome with NVDA
Are there any code-based customization done to Web Chat?
No, I am using Web Chat without any customizations except "styleOptions".
What version of Web Chat are you using?
Latest production
Which area does this issue affect?
Others or unrelated
What is the public URL for the website?
Request Contributors - POC (sharepoint.com)
How to reproduce the issue?
Step 1: Open the above URL in New Edge/Chrome browsers with valid credentials.
Step 2: Invoke the 'Ask iris now' button which is present under the questions section. Iris the bot dialog will appear. Navigate to the hamburger menu and hover the mouse. Tooltip will appear.
Step 3: Verify that the Tooltip of controls present on 'Iris the Bot' dialog is hover able or not.
What do you expect?
Tooltip of controls present on 'Iris the Bot' dialog should be hover able.
What actually happened?
The tooltip of controls present on 'Iris the Bot' dialog is not hover able.
Do you have any screenshots or recordings to repro the issue?

Did you find any DOM elements that might have caused the issue?
No response
MAS reference
https://aka.ms/MAS1.4.13
WCAG reference
https://www.w3.org/WAI/WCAG21/Understanding/content-on-hover-or-focus.html
WAI-ARIA reference
No response
Adaptive Card JSON
No response
Additional context
No response

#A11yMAS;#A11ySev3;#MAS1.4.13;#HCL;#Accessibility;#FeatureBotframeworkwebchat-Apr23;

Bug copied from MCAPS: ISSUE 1767689

As per the above comment, this issue is not under bot framework team. Hence closed the bug and updated to partner team (WWL).


Fix parameter scale mapping for condition table parameters

Codecov Report

Merging #102 into develop will decrease coverage by 0.1%.
The diff coverage is 50%.

@@             Coverage Diff             @@
##           develop     #102      +/-   ##
===========================================
- Coverage    59.92%   59.81%   -0.11%
===========================================
  Files            7        7
  Lines          756      759       +3
  Branches       167      167
===========================================
+ Hits           453      454       +1
- Misses         275      277       +2
  Partials        28       28

Impacted Files
Coverage Δ

petab/core.py
79.47% <50%> (-0.41%)
:arrow_down:

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 25aea56...b059eab. Read the comment docs.

@yannikschaelte : I was just wondering: was the plan to have the non-overriden condition-specific parameter as IDs or as value in the mapping table? This change assumes the former.

@yannikschaelte : I was just wondering: was the plan to have the non-overriden condition-specific parameter as IDs or as value in the mapping table? This change assumes the former.

What would a non-overriden parameter be? Even if a simulation parameter is constantly overriden by a fixed (optimization) parameter, I would still have the parameter here because it might get optimized. But I guess that's not what you meant.

Sorry, I meant parameter which are overridden by a numeric value. Currently these values are not filled in.

So that would correspond to condition-specific constant parameters, as amici calls them. It might make sense to include them also here, but that then touches the same topic as yesterday whether we want to treat amici constant parameters and fixed (i.e. could but won't compute derivatives) parameters differently. Eventually, we will anyway need to revise how the amici simulation parameter vector is created ...
So, I would advocate including all simulation parameters in the mapping table, and then possibly checking later before amici compilation which we can/want to have as constant parameters.

So, I would advocate including all simulation parameters in the mapping table, and then possibly checking later before amici compilation which we can/want to have as constant parameters.

By including you mean filling in numeric values? Or keep as is?

essentially, we have this line: https://github.com/ICB-DCM/pyPESTO/blob/master/pypesto/objective/petab_import.py#L177, which will fail anyway now

i mean filling in numeric values

Just talking about PEtab for now :)

yes, but how we create the table is affected by what we want to extract how later

Rather the other way around. Okay, this needs to be addressed elsewhere then.


[SPARK-26137][CORE] Use Java system property "file.separator" inste…

… of hard coded "/" in DependencyUtils
What changes were proposed in this pull request?
Use Java system property "file.separator" instead of hard coded "/" in DependencyUtils.
How was this patch tested?
Manual test

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Thank you for submitting a PR to fix hard coded character. Is this only one that we have to fix regarding this hard coded character?

Would it be possible to update the PR description based on the template?

ok to test

Test build #99140 has started for PR 23102 at commit 09cb210.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/5247/
Test PASSed.

Hi, @markpavey .
According to the JIRA description, you tested your PR on the Windows. As @kiszk advised, could you add more specific description about what you tested on Windows manually? That would be helpful for the others.

Just for double checking, are the changes really needed? resolveGlobPaths calls resolveGlobPath -> toUri -> normalizePath in which Windows specific \\ is replaced by /: https://github.com/apache/hadoop-common/blob/42a61a4fbc88303913c4681f0d40ffcc737e70b5/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/Path.java#L242-L250
We need a test (or a reproducible example) for this at least.

Test build #99140 has finished for PR 23102 at commit 09cb210.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99140/
Test PASSed.

@markpavey, can you write a test? I can run the test on Windows via AppVeyor.

Test build #99190 has started for PR 23102 at commit 49ce5a8.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/5287/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/5288/
Test PASSed.

Test build #99190 has finished for PR 23102 at commit 49ce5a8.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99190/
Test PASSed.

@MaxGekk This PR may change a separator for userJar that has \ on Windows. resolveGlobPaths is not applied to userJar.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/5363/
Test PASSed.

Test build #99280 has started for PR 23102 at commit bfabd4f.

Test build #99280 has finished for PR 23102 at commit bfabd4f.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99280/
Test FAILed.

Test build #99281 has started for PR 23102 at commit 4ab68f0.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/5364/
Test PASSed.

Test build #99281 has finished for PR 23102 at commit 4ab68f0.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99281/
Test PASSed.

Test build #99319 has started for PR 23102 at commit 46a6dd4.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/5401/
Test PASSed.

Test build #99319 has finished for PR 23102 at commit 46a6dd4.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99319/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/5411/
Test PASSed.

Test build #99333 has started for PR 23102 at commit e1e97aa.

Merged to master/2.4/2.3


Expanding relation with Pocketbase version 1.13.1 leads to error?

I am trying to expand a relation, however I am getting the following error.
  File "test.py", line 65, in test_func
    records = pb.collection("product_details").get_full_list(
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/homebrew/lib/python3.11/site-packages/pocketbase/services/utils/crud_service.py", line 17, in get_full_list
    return self._get_full_list(self.base_crud_path(), batch, query_params)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/homebrew/lib/python3.11/site-packages/pocketbase/services/utils/base_crud_service.py", line 31, in _get_full_list
    return request(result, 1)
           ^^^^^^^^^^^^^^^^^^
  File "/opt/homebrew/lib/python3.11/site-packages/pocketbase/services/utils/base_crud_service.py", line 23, in request
    list = self._get_list(base_path, page, batch_size, query_params)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/homebrew/lib/python3.11/site-packages/pocketbase/services/utils/base_crud_service.py", line 44, in _get_list
    items.append(self.decode(item))
                 ^^^^^^^^^^^^^^^^^
  File "/opt/homebrew/lib/python3.11/site-packages/pocketbase/services/record_service.py", line 50, in decode
    return Record(data)
           ^^^^^^^^^^^^
  File "/opt/homebrew/lib/python3.11/site-packages/pocketbase/models/utils/base_model.py", line 16, in __init__
    self.load(data)
  File "/opt/homebrew/lib/python3.11/site-packages/pocketbase/models/record.py", line 18, in load
    self.load_expanded()
  File "/opt/homebrew/lib/python3.11/site-packages/pocketbase/models/record.py", line 26, in load_expanded
    self.expand[key] = self.parse_expanded(value)
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/homebrew/lib/python3.11/site-packages/pocketbase/models/record.py", line 22, in parse_expanded
    return cls(data)
           ^^^^^^^^^
  File "/opt/homebrew/lib/python3.11/site-packages/pocketbase/models/utils/base_model.py", line 16, in __init__
    self.load(data)
  File "/opt/homebrew/lib/python3.11/site-packages/pocketbase/models/record.py", line 13, in load
    super().load(data)
  File "/opt/homebrew/lib/python3.11/site-packages/pocketbase/models/utils/base_model.py", line 30, in load
    self.id = data.pop("id", "")
              ^^^^^^^^^^^^^^^^^^
TypeError: pop expected at most 1 argument, got 2

After further testing with the source_code, I noticed that even though data is typed as dict, I am getting a list.
Here is the output I am printing inside the load function:
[{'collectionId': '32zt00urx7zeou4', 'collectionName': 'testcollection', 'created': '2023-03-19 18:26:45.604Z', 'id': 'rm74nish4xa4inm', 'image_url': 'https://www.example.com', 'updated': '2023-03-19 18:26:45.604Z'}, {'collectionId': '32zt00urx7zeou4', 'collectionName': 'testcollection', 'created': '2023-03-19 19:43:03.957Z', 'id': 'ju4srx55a5v7cmj', 'image_url': 'https://www.example.com', 'updated': '2023-03-19 19:43:03.957Z'}]

Here is my query to the pocketbase function get_full_list()
query_params={"expand": "image"}

Sorry but i cannot reproduce your issue with the current master branch of this repo. Are you sure that your record field "image" is present in the database as a relational field?
The test case
    def test_get_record_expand_full_list(self, client: PocketBase, state):
        rel = client.collection(state.coll.id).get_full_list(
            query_params={"expand": "rel"}
        )
        i = 0
        for r in rel:
            while r != None:
                if hasattr(r, "expand"):
                    if "rel" in r.expand:
                        i += 1
                        r = r.expand["rel"]
                        continue
                r = None
        assert i == 10

works well on a example database that has 10 records each with a relational field "rel".
I just tested this to work both with a pocketbase v0.13.1 as well as the current pocketbase v0.16.4.
note even a recursive expand with depth six query_params={"expand": "rel.rel.rel.rel.rel.rel"} well on my side

I'm experiencing a similar problem, but it only happens when I try to expand a multiple-choice relationship. It just works fine for single relationship fields.


fix/test: inverse relationship issues

Related to: #4979
What is the purpose of the change
This is a known issue of #4979 but it seems that I've run into it independently while debugging https://github.com/osmosis-labs/osmosis/issues/4925.
This PR resolves the issue by rounding in CalculatePriceToTick. The test vectors got resolved by this change.
Brief Changelog

Separate TickToPrice and TickToSqrtPrice functions where TickToSqrtPrice calls TickToPrice
Add a lot of test vectors to reproduce the inverse relationship problem
resolve it by rounding instead of truncating

Testing and Verifying
This change added tests
Documentation and Release Note

Does this pull request introduce a new feature or user-facing behavior changes? no
Is a relevant changelog entry added to the Unreleased section in CHANGELOG.md? no
How is the feature or change documented? not applicable

Merging to unblock further PRs. Will be continuing to investigate this behavior further as well


[vim] Use cmd.exe in Windows

When running Vim on Windows,

use cmd.exe as launcher and shell
enable shellslash within s:cmd()
don't escape ' ' and '\' for paths

Thanks. Let me review the commit. By the way, we don't use l: prefix for local variables. Can you remove the prefixes for consistency?

I haven't tried running Vim plugin for fzf on Windows, and never expected it to work correctly. Can you share your overall experience?

With this PR, it works on file paths if without spaces or not network drives (it breaks on C:\ ).
It works fine if running fzf, via the Vim plugin, within the home directory (because of ~ expansion).
:FZF redraws the entire screen for cmd.exe to run fzf and it behaves as expected when selecting files/directories (even with spaces) to open.
To run terminal Vim, I use ConEmu with Powershell (to minimize file path issues).
I used cmd.exe in the PR because Powershell has long startup time.

For now, in this PR, FZF and FZF! commands behave the same on terminal Vim in Windows because I did not set the geometry for cmd.exe similar to the xterm_launcher.

Thanks for the explanation. Merged, thanks!

Thanks for the explanation. Merged, thanks!


can't be too strict with Browser#html spec

Since different browsers will return different page sources for the same page, this spec should be less strict.

I think this is mostly handled.


[feature request] Make vue-autosuggest fully compatible with BEM

It's great that vue-autosuggest follow the BEM convention, but it would be even greater if it followed the BEM convention fully ;)

autosuggest__results_item-${index} should be autosuggest__results-item--${index}
autosuggest__results_item-highlighted should be autosuggest__results-item--highlighted
autosuggest__results_item shoud be autosuggest__results-item
autosuggest__results_title should be - autosuggest__results-title``
autosuggest__results_title_${this.section.name} should beautosuggest__results-title--${this.section.name}
autosuggest__input-open should be autosuggest__input--open
<ul> element doesn't have a class. Let's give it a autosuggest__results-list class.

I'm going to be honest with you that I have never actually worked with BEM before so I'm actually quite surprised it's so close to BEM convention. Close, but no cigar.
This looks like a good PR for outsiders who don't necessarily need to understand the codebase. PR'd against the 2.0 branch if you were up for contributing. If you're interested, a PR with BEM standards, as well as prefix customization would be welcome!

@darrenjennings
I'll create a PR soon.

Can I play devil's advocate here? I wonder if the missing feature just to be able to override two things:

the class prefix
each individual class used in the library

By providing these two features and not changing the existing class names, consumers will have the ability to classes that follow BEM convention or any other convention they choose, while not introducing a breaking change for existing users.
I understand that this change would be in a major release and breaking changes are acceptable, but I still think minimizing them when possible is ideal. If the believe is the majority of consumers want the BEM classes, then the change makes sense, but if only a few do, this is a breaking a change that only really benefits a few so that they won't have to provide their classes via configuration.

@scottadamsmith thanks for the feedback, always appreciated.
Class prefix customization will be fixed by #97. Originally, I did not want to add any classnames on any element, but the "rubber to the road" truth was that I needed them for query selecting and accessibility. I believe this to still be an acceptable architecture, but ideally we need to solve use cases like users coming from bootstrap, or tailwind. I don't like prop explosion so I'm less likely to go with solutions like #90.

If the belief is that the majority of consumers want BEM classes, then the change makes sense. But if only a few do, this is a breaking a change that only really benefits a few so that they won't have to provide their classes via configuration.

This lib is small enough that I'm willing to "break" more things, so that new users can benefit, while alleviating the punishment on existing users with a nice changelog/migration guide. My goal is to break as many features as possible in 2.0 in an attempt to make it more intuitive for new users, while (hopefully) increasing the longevity of vue-autosuggest.
Going forward, I think we should accept BEM defaults, allow to customize via prefix and append new classnames via v-bind props, similar to how the <input class works: https://github.com/darrenjennings/vue-autosuggest/blob/b825be24211450da3700898ade30701ecdd3b491/src/Autosuggest.vue#L8

Closed by 2.0.0
Was an oversight now that I'm reviewing, I did not add ability for class to be added <ul>. Would accept a PR but closing this ticket since it relates to BEM, not new classes.


Reduce package size

Do you want to request a feature or report a bug?
Chore.
What is the current behavior?
Contain .github template.
What is the expected behavior?
Don't contain .github template.
If this is a feature request, what is motivation or use case for changing the behavior?
Reduce package size
Please mention other relevant information such as your webpack version, Node.js version and Operating System.
Not required.

@evilebottnawi - This will be taken care of in #70 which has a properly defined files property in the package.json

@d3viant0ne thanks!


Switch all to using annotation for all kernel examples

Please switch all examples to use the annotations instead of just a bare NSC container with an ENV variable patches in.

@edwarnicke It is possible for all kernel examples.
Could you say more about how we can change env to annotation for memif, vfio examples?

kernel examples are sufficient :)

OK, now it is clear, thanks!

Improvement in logging required:
https://github.com/networkservicemesh/integration-tests/issues/694

closed with https://github.com/networkservicemesh/deployments-k8s/pull/3663


How to force reload of tile's ImageData

I  have DZI data in both RGB and multiple greyscale channels.  I want to make a viewer that allows
user selectively show different channel at  a time or composition of multiple.  I have tried the layering
of multiple DZI with  addTiledImage. That did not work because  I could not alter each layer independently except their opacity. Now, after consulting issue #389, I have worked out a solution with
a tile-drawing handler.  I created a new ImageData space, copied the the original into it without
one of the channel data and successfully altered the display by calling putImageData. Now if the user
want the channel back, I need to be able to force a reload of the ImageData from tiles because the ImageData in the context has been altered permanently by the putImageData call.

Found the answer,  viewer.world.resetItems()

Just so you know, I just published a filtering plugin which might help you: https://github.com/NIST-ISG/OpenSeadragonFiltering
It already handles the restoration of the original ImageData.

Note that it requires OSD 2.1 which is not yet released. So you have to checkout the master branch for it to work.

thanks @avandecreme, I will check it out.

I have checked out your code. It is indeed very helpful. One of the requirement of our filter is
to blend data from multiple tiledImages into a mergedImageData. For example, if we load several
grey scale DZI data, we want to be able to colormap one of them to Red(colorization filter) and
then other to Blue(colorization filter) and blend them into a merged result(blending filter).
How to access other matching tiledImage tile with the current event's is something I need. Is it
possible?

I support this feature, I'll need it sooner or later (fluorescence microscopy scans)

For such a use case, I believe it would be easier with virutal images #119
However it may be possible using the filtering plugin if the filters accepted a tile as argument:
function(tile, context, callback)
The filters could then fetch the other channels tiles and blend them together.

Another possibility is e.g. if your viewer has the tilesource: 'file.dzi', you convert it to 'file.dzi?' + new Date().getTime();
and replace the old tiledImage with the new one. Then OSD is forced to redraw the tiles.


Fix MapComparator util

Summary of the changes / Why this improves CrateDB
Fix comparison for nested maps (objects), i.e.:
m1: {a={b={123}} // 123 Short
m2: {a={b={123}} // 123 Long

compareMaps(m1, m2) must return 0 (true)
Found via:
./gradlew :server:test --tests "io.crate.integrationtests.TransportSQLActionTest.test_types_with_storage_can_be_inserted_and_queried" -Dtests.seed=CBA296E46FC0EC -Dtests.locale=en-AE -Dtests.timezone=Asia/Muscat

Inserted value = {x={x=9145}} where 9145 is Short
Response value = x={x=9145}} where 9145 is Long
Checklist

[x] Added an entry in CHANGES.txt for user facing changes
[x] Updated documentation & sql_features table for user facing changes
[x] Touched code is covered by tests
[x] CLA is signed
[x] This does not contain breaking changes, or if it does:

It is released within a major release
It is recorded in CHANGES.txt
It was marked as deprecated in an earlier release if possible
You've thought about the consequences and other components are adapted
(E.g. AdminUI)

/cc @mfussenegger

LGTM - if we have a reproducible scenario (user facing) - can we add a change entry describing it?

I cannot think of a case.


[bug]: UTFiles does not exist

Provide environment information
System:
    OS: Linux 6.2 KDE neon 5.27 5.27
    CPU: (8) x64 AMD Ryzen 5 3350G with Radeon Vega Graphics
    Memory: 6.65 GB / 13.57 GB
    Container: Yes
    Shell: 5.1.16 - /bin/bash
  Binaries:
    Node: 21.5.0 - ~/.nvm/versions/node/v21.5.0/bin/node
    npm: 10.2.4 - ~/.nvm/versions/node/v21.5.0/bin/npm
  npmPackages:
    @uploadthing/react: 6.2.2 => 6.2.2
    typescript: ^5.3.3 => 5.3.3
    uploadthing: 6.3.3 => 6.3.3

Describe the bug
based off of the documentation i should be able to add customId to uploaded file using UTFiles like this
    .middleware(({ req, files }) => {
      // Check some condition based on the incoming requrest
      req;
      //^?
      // if (!req.headers.get("x-some-header")) {
      //   throw new Error("x-some-header is required");
      // }

      // (Optional) Label your files with a custom identifier
      const filesWithMyIds = files.map((file, idx) => ({
        ...file,
        customId: `${idx}-${randomUUID()}`,
      }));

      // Return some metadata to be stored with the file
      return { foo: "bar" as const, [UTFiles]: filesWithMyIds };
    })

but when trying to import UTFiles i get an error
//Module '"uploadthing/next"' has no exported member 'UTFiles'.(2305)
import { createUploadthing, UTFiles } from "uploadthing/next";

this even happens in provided minimal example

when going inside node_modules/uploadthing/dist/next.d.ts those are the only exports

Link to reproduction
https://stackblitz.com/github/pingdotgg/uploadthing/tree/main/examples/minimal-appdir?file=src%2Fserver%2Fuploadthing.ts
To reproduce
no additional steps required
Additional information
No response
👨👧👦 Contributing

[ ] 🙋♂ Yes, I'd be down to file a PR fixing this bug!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

in fact dist directory does not seem to include anything with "UTFiles"

sorry it's not released yet - will be next update

I've added a version tag in the docs about it too. Thanks for bringing it up, we strive to always put the version things are added but forgot that here...

I've added a version tag in the docs about it too. Thanks for bringing it up, we strive to always put the version things are added but forgot that here...

Hey! am I missing something? the docs say v6.4, I'm using 6.4.1 and I'm still getting "Attempted import error: 'UTFiles' is not exported from 'uploadthing/server' (imported as 'UTFiles')"
Assuming is not released yet, there's currently no say to attach metadata to files, right?
I'm having the need to split my files into "projects", and was planning to use customId to do so.
Appreciate the efforts beforehand, pretty cool/useful project! 👍

whoops looks like it's exported from everything but the server entrypoint xd
you can import it from any other framework entrypoint, it's the same thing realyl...
import { UTFiles } from "uploadthing/next";

will also make sure to export it from the server entrypoint

whoops looks like it's exported from everything but the server entrypoint xd
you can import it from any other framework entrypoint, it's all the same thing just re-exported
import { UTFiles } from "uploadthing/next";

will also make sure to export it from the server entrypoint

Hey! thanks, that's basically what I was doing (although I was importing it from internals) but I'm still not able to edit the file's customId though, is this working already?
As shown in the image, looks like I'm editing the metadata, but is not affecting the file's customId?

I tried manually returning a customId just to see what happens, and I managed to see the metadata in my onUploadComplete, but the file is still not affected:

try again - seems like last deploy we changed some stuff and missed the customId for certain uploads...

try again now - seems like last deploy we changed some stuff and missed the customId for certain uploads...

It works now, thank you so much!
I may have misunderstood the use though, I thought I could upload different images with the same customId (as customId was a metadata, and not the primary key), looks like is not the case?
I'm getting an error saying my customId already exists.
Assuming is working correctly, I'm probably better off storing slugs/keys or something in my own DB, and relationships with users, as I wouldn't be able to filter files by metadata in uploadthing anyways, right?
I was planning on using customId to assign files to specific projects.

That's right, customIds are unique to an app. People wanted it since they wanted their own ids and also allows you to use it sort of like a KV-store since you'll know the id before uploading as opposed to the fileKeys which are generated by us out of your control.
It looks like you'll want to store the file metadata in your db with the relation to each project.

That's right, customIds are unique to an app. People wanted it since they wanted their own ids and also allows you to use it sort of like a KV-store since you'll know the id before uploading as opposed to the fileKeys which are generated by us out of your control.
It looks like you'll want to store the file metadata in your db with the relation to each project.

Cool! I just checked upload thing's doc today and in the roadmap looks like you have folders, perhaps that'll help too but you're right, I'll probably stick to DB's relationships now, thanks! 🙏


Butterflies performance issues

Hello, I recently added this mod to my server. One of my friends lives pretty far away from me, and her internet connection isn't the best, so I ran some tests and I found this:

Now, I'm not an expert with minecraft performance, neither in Java, even tho I'm a JavaScript developer. Seeing this error, it's obvious that there is something wrong with butterflies, so I tried turning them off in the config.
The config presents this line: spawn.butterflies=true, which is apparently a Boolean, so I just changed it to false, but the butterflies are still here. I also tried checking the source code but well, not being a Java developer I couldn't figure out much.
Is there actually anything wrong with butterflies? And can it be related to internet connection issues?

I'll take a look at the butterfly code. How exactly does the config file look?

This is exactly what it looks like now. Also, the file name is "clutterconfig.properties" tho. Is it correct?

And that's the file on the server?

And that's the file on the server?

Yes, exactly. Sorry for the late answer

Did you also restart the server?

Did you also restart the server?

Yeah exactly

Weird that they're still spawning with the config set to false, pretty sure I checked that.
Also, I've changed the way butterflies pathfind to (hopefully) increase performance. This will be changed in the next update.
Lastly, what did you use to get those performance numbers?

Weird that they're still spawning with the config set to false, pretty sure I checked that. Also, I've changed the way butterflies pathfind to (hopefully) increase performance. This will be changed in the next update. Lastly, what did you use to get those performance numbers?

Nice! I used the mod Spark to get all the results


cmake error in master branch when compiling from source

I've also tried compiling in master branch but there seems to be a problem with the optional dependency eigenpy (which it seems to find but marks as not found?):
 cmake ..
-- Package version (ROS package.xml): 2.5.6
-- Since urdfdom >= 1.0.0, the default C++ standard is C++11. The project is then compiled with C++11 standard.
-- Found Boost: /usr/include (found version "1.69.0") found components: filesystem serialization system
-- The Python bindings of Pinocchio will be compiled along the main library. If you want to disable this feature, please set the option BUILD_PYTHON_INTERFACE to OFF.
-- PythonLibraryDirs: /home/user/.conda/envs/pinocchio/lib
-- PythonLibVersionString: 3.8.8
-- Python site lib: lib/python3.8/site-packages
-- Found Boost: /usr/include (found version "1.69.0") found components: python38
-- Boost_PYTHON_LIBRARY: /usr/lib64/libboost_python38.so
-- eigenpy FOUND. eigenpy at /home/user/.conda/envs/pinocchio/lib/libeigenpy.so
CMake Error at cmake/package-config.cmake:84 (find_package):
  Found package configuration file:

    /home/user/.conda/envs/pinocchio/lib/cmake/eigenpy/eigenpyConfig.cmake

  but it set eigenpy_FOUND to FALSE so package "eigenpy" is considered to be
  NOT FOUND.  Reason given by package:

  eigenpy could not be found because dependency Eigen3 could not be found.

Call Stack (most recent call first):
  CMakeLists.txt:136 (ADD_PROJECT_DEPENDENCY)

-- Configuring incomplete, errors occurred!
See also "/home/user/src/github/pinocchio-fork/build/CMakeFiles/CMakeOutput.log".

I don't know how relevant it is, but I'm trying to compile in Fedora 32 inside a conda environment.

Your environnement is not clean at all. Please start from a clean distribution.

Just for future googlers arriving here: The problem was caused by having run cmake .. in the devel branch and trying to execute it again in master.
I don't understand why cmake would retain state but a simple rm -rf build/ and mkdir build && cd build && cmake .. fixed it.

You have a CMakeCache.txt which does the saving.

Yes, though in my experience, running cmake multiple times hasn't been a problem and should regenerate the build system just fine.
Surely there might be complex build systems that are the exceptions but it sounds to me that eigenpy is being cached wrong, as it was found.


fails to open on OSX

Every time I click a .apk, Android tool opens with error.
when i click"reopen" it works.
Attached are two screenshots.
1st image  shows the error i et.
2nd image shows it opening when i click "reopen"

Looks like a bug. Until this bug is cleared, you can install apks by dragging the file directly onto the device in Android Tool


About depth_distribution_similarity_loss

Hi! Thank you for the good work!
I am trying to train the network on DTU using your scripts and I have doubts about depth_distribution_similarity_loss.
pixel_wise_loss starts around 1.0 and decreases, dds_loss does not decrease and ranges from 140 to 80. At the same time, the predictions are still good
I looked at the code, you pass the depth values to kl-divergence, as far as I know kl-divergence usually expects the probability of distribution as inputs. And your article says that p and q are depth distributions
Can you explain why this is so?
P.S. I use batch_size = 1. I hope that's not the problem
def depth_distribution_similarity_loss(depth, depth_gt, mask, depth_min, depth_max):
    depth_norm = depth * 128 / (depth_max - depth_min)[:,None,None]
    depth_gt_norm = depth_gt * 128 / (depth_max - depth_min)[:,None,None]

    M_bins = 48
    kl_min = torch.min(torch.min(depth_gt), depth.mean()-3.*depth.std())
    kl_max = torch.max(torch.max(depth_gt), depth.mean()+3.*depth.std())
    bins = torch.linspace(kl_min, kl_max, steps=M_bins)

    kl_divs = []
    for i in range(len(bins) - 1):
        bin_mask = (depth_gt >= bins[i]) & (depth_gt < bins[i+1])
        merged_mask = mask & bin_mask

        if merged_mask.sum() > 0:
            p = depth_norm[merged_mask] # is depth values, not probs
            q = depth_gt_norm[merged_mask]
            kl_div = torch.nn.functional.kl_div(torch.log(p)-torch.log(q), p, reduction='batchmean')
            kl_div = torch.log(kl_div)
            kl_divs.append(kl_div)

    dds_loss = sum(kl_divs)
    return dds_loss

Hi, thanks for your interest in our work.
Firstly, to be honest, batch_size=1 always meets some "ghost" problem, so please check it first.
As for your problem, as described in Sec. 3.3 in our paper, we found that the data distribution satisfies the Mixed-Gaussian Model. So we pretend to calculate the similarity of two depth distributions (estimated depth & depth GT). That's the motivation of the code you referenced in our implementation.
As for

pixel_wise_loss starts around 1.0 and decreases, dds_loss does not decrease and ranges from 140 to 80. At the same time, the predictions are still good

We need to admit that the data of small scenes such as DTU do not fully meet the perfect Gaussian model, so different training iterations can not accurately describe the similarity of curves, resulting in loss shaking in the training of this loss.
So a better way to characterize the depth of the whole scene (v.s. pixel-wise loss measure) will be one of the directions of future development, I think.
We are really sorry for confusing you and hope the reply helps.

@doubleZ0108 hi, I also have some doubts about the code of dds_loss.

In Section 3.3 of the paper, it is mentioned that a GMM with K = 1 and K = 2 is used to fit the full-scene depth distribution. However, in the code, only one Gaussian component is utilized. Does this imply that using one Gaussian component to calculate the depth value range for representing the distribution of each scene is sufficient?

After obtaining the depth interval of 1 (μi - 3σi, μi + 3σi), why do you use 48 smaller bins to calculate KL divergence between the predicted depth and ground truth depth for each bin, rather than calculating the entire scene at once?

When normalizing the depth, you use the formula depth * 128 / (depth_max - depth_min). Why not use (depth - depth_min) * 128 / (depth_max - depth_min), which is a more commonly used normalizing method?

4)I have used the script to train the network without making any parameter adjustments (batch size = 4 with 2 GPUs).

I found the dds_loss curve in each training stage didn't exhibits a decrease trend. I'm curious whether this will lead to improved reconstruction results? (And the pw_loss of stage 0 has also increased as training continues)
These questions might be a bit extensive, but they've been causing me confusion for some time. I would greatly appreciate it if you could provide some clarifications. Thank you in advance!

@xingchen2022 Yes, dds_loss is not perfect. The mixed Gaussian model analyzed in Section 3.3 is our ultimate vision, but dds_loss is a trade-off. It does not perfectly decrease in all training iterations, but I believe that the loss part of characterizing the curve can still guide the network to some extent (just like the momentum strategy). As for the setting of 48, it is because we use binning for calculation, which involves continuous and discrete conversions.

@xingchen2022 Yes, dds_loss is not perfect. The mixed Gaussian model analyzed in Section 3.3 is our ultimate vision, but dds_loss is a trade-off. It does not perfectly decrease in all training iterations, but I believe that the loss part of characterizing the curve can still guide the network to some extent (just like the momentum strategy). As for the setting of 48, it is because we use binning for calculation, which involves continuous and discrete conversions.

Thank you very much, It's valuable information!  And could you tell me your insights regarding the depth normalizing code? (Q3: you use the formula depth * 128 / (depth_max - depth_min). Why not use (depth - depth_min) * 128 / (depth_max - depth_min), which is a more commonly used normalizing method?)

@xingchen2022 Sorry for the missing. Yeah, the way you said is correct and I have tried during my experiments before. However, I found that the training effect was not as good as it is now. I think that normalization is just a strategy for data scaling and does not have strict mathematical constraints, so -depth_min is not necessary in my opinion.

@xingchen2022 Sorry for the missing. Yeah, the way you said is correct and I have tried during my experiments before. However, I found that the training effect was not as good as it is now. I think that normalization is just a strategy for data scaling and does not have strict mathematical constraints, so -depth_min is not necessary in my opinion.

Got it! Better result is all we want, hhh
Thanks again, and wish you have a good rest tonight ❤

@doubleZ0108 Thank you for your answers
I think the issue may be closed


Invalid value of operation id

isb-cgc-pipelines list jobs

JOBID    PIPELINE     OPERATION ID    TAG                                     STATUS    CREATE TIME    PREEMPTIONS
1        mypipeline    None            1d50232e-3489-4f3b-a3c6-3f182c10fe6c    ERROR     None           0

gcloud compute operations describe 1d50232e-3489-4f3b-a3c6-3f182c10fe6c

Invalid value '1d50232e-3489-4f3b-a3c6-3f182c10fe6c'. Values must match the following regular expression: 'a-z?'

the error reported by isb-cgc-pipelines in this case occurred before it was actually submitted as a GGP task -- note the "None" in the "OPERATION ID" column (the 1d50...fe6c is the isb-cgc-pipelines "TAG")
try looking under /var/log/supervisor/ on the machine where you are running isb-cgc-pipelines for information in the log files that can be found there


BUG: After adding image to clusterfile, the program is blocked

Sealos Version
main
How to reproduce the bug?
basic Clusterfile
apiVersion: apps.sealos.io/v1beta1
kind: Cluster
metadata:
  name: default
spec:
  image:
  - hub.sealos.cn/labring/kubernetes:v1.25.6
  - hub.sealos.cn/labring/helm:v3.11.0
  - hub.sealos.cn/labring/flannel:v0.21.4

apply
sealos apply -f Clusterfile
add new app hub.sealos.cn/labring/nginx:v1.23.3 to Clusterfile
apiVersion: apps.sealos.io/v1beta1
kind: Cluster
metadata:
  name: default
spec:
  image:
  - hub.sealos.cn/labring/kubernetes:v1.25.6
  - hub.sealos.cn/labring/helm:v3.11.0
  - hub.sealos.cn/labring/flannel:v0.21.4
  - hub.sealos.cn/labring/nginx:v1.23.3

apply again, program is blocked
What is the expected behavior?
nginx installation complete.
What do you see instead?
program is blocked.
2023-04-27T15:16:12 info Executing ConfirmOverrideApps Pipeline in InstallProcessor Stopped here.
root@sealos-test:/test/sealos-test# sealos apply --debug -f Clusterfile
2023-04-27T15:16:12 debug loadClusterFile loadRenderValues: map[]
2023-04-27T15:16:12 debug rendered Clusterfile: apiVersion: apps.sealos.io/v1beta1
kind: Cluster
metadata:
  name: default
spec:
  image:
  - hub.sealos.cn/labring/kubernetes:v1.25.6
  - hub.sealos.cn/labring/helm:v3.11.0
  - hub.sealos.cn/labring/flannel:v0.21.4
  - hub.sealos.cn/labring/nginx:v1.23.3

2023-04-27T15:16:12 debug loadClusterFile loadRenderValues: map[]
2023-04-27T15:16:12 debug rendered Clusterfile: apiVersion: apps.sealos.io/v1beta1
kind: Cluster
metadata:
  creationTimestamp: "2023-04-27T07:14:15Z"
  name: default
spec:
  hosts:
  - ips:
    - <IP_ADDRESS>
    roles:
    - master
  image:
  - hub.sealos.cn/labring/kubernetes:v1.25.6
  - hub.sealos.cn/labring/helm:v3.11.0
  - hub.sealos.cn/labring/flannel:v0.21.4
  ssh: {}
status:
  commandCondition:
  - images:
    - hub.sealos.cn/labring/kubernetes:v1.25.6
    - hub.sealos.cn/labring/helm:v3.11.0
    - hub.sealos.cn/labring/flannel:v0.21.4
    lastHeartbeatTime: "2023-04-27T07:14:15Z"
    message: Applied to cluster successfully
    reason: Apply Command
    status: "True"
    type: ApplyCommandSuccess
  conditions:
  - lastHeartbeatTime: "2023-04-27T07:14:15Z"
    message: Applied to cluster successfully
    reason: Ready
    status: "True"
    type: ApplyClusterSuccess
  mounts:
  - env:
      criData: /var/lib/containerd
      defaultVIP: <IP_ADDRESS>
      disableApparmor: "false"
      registryConfig: /etc/registry
      registryData: /var/lib/registry
      registryDomain: sealos.hub
      registryPassword: passw0rd
      registryPort: "5000"
      registryUsername: admin
      sandboxImage: pause:3.8
    imageName: hub.sealos.cn/labring/kubernetes:v1.25.6
    labels:
      auth: auth.sh
      check: check.sh $registryData
      clean: clean.sh && bash clean-cri.sh $criData
      clean-registry: clean-registry.sh $registryData $registryConfig
      image: ghcr.io/labring/lvscare:v4.1.6
      init: init-cri.sh && bash init.sh
      init-registry: init-registry.sh $registryData $registryConfig
      io.buildah.version: 1.28.1
      org.opencontainers.image.description: kubernetes-v1.25.6 container image
      org.opencontainers.image.licenses: MIT
      org.opencontainers.image.source: https://github.com/labring-actions/cache
      sealos.io.type: rootfs
      sealos.io.version: v1beta1
      version: v1.25.6
      vip: $defaultVIP
    mountPoint: /var/lib/containers/storage/overlay/f446971c79475e35a495f509e4de5a60f8ff8ef9519978908930b550c50f7f56/merged
    name: default-rn5f5jsy
    type: rootfs
  - cmd:
    - cp -a opt/helm /usr/bin/
    imageName: hub.sealos.cn/labring/helm:v3.11.0
    labels:
      io.buildah.version: 1.28.1
    mountPoint: /var/lib/containers/storage/overlay/00da0d4e3488b71fe76f5fcdee72c38b574237769459ad5e4a3efd60a8d51a5b/merged
    name: default-h9cd45t3
    type: application
  - cmd:
    - bash entrypoint.sh
    imageName: hub.sealos.cn/labring/flannel:v0.21.4
    labels:
      io.buildah.version: 1.28.1
    mountPoint: /var/lib/containers/storage/overlay/3ef9b905bf3366a2730150e865047ece49e05e1b4d224c25f0e237214ffd0991/merged
    name: default-aezf5xvq
    type: application
  phase: ClusterSuccess

2023-04-27T15:16:12 debug run new images: [hub.sealos.cn/labring/nginx:v1.23.3]
2023-04-27T15:16:12 info start to install app in this cluster
2023-04-27T15:16:12 info Executing SyncStatusAndCheck Pipeline in InstallProcessor
2023-04-27T15:16:12 debug parse reference a5fd294ad99fd9e83bd60a5efabd05d7a342ca91eda012ef7561257b047f71af with transport containers-storage
2023-04-27T15:16:12 debug parse reference 34018b130618f340f685ea8a30dec40a74b846a5a80cfd905666b837dcf3b50a with transport containers-storage
2023-04-27T15:16:12 debug parse reference 88e8e96d5c07d1490039da94109818db43a111b43db3e18ab95aa9d2978f6f01 with transport containers-storage
2023-04-27T15:16:12 debug sync cluster status is: apiVersion: apps.sealos.io/v1beta1
kind: Cluster
metadata:
  creationTimestamp: null
  name: default
spec:
  hosts:
  - ips:
    - <IP_ADDRESS>
    roles:
    - master
  image:
  - hub.sealos.cn/labring/kubernetes:v1.25.6
  - hub.sealos.cn/labring/helm:v3.11.0
  - hub.sealos.cn/labring/flannel:v0.21.4
  - hub.sealos.cn/labring/nginx:v1.23.3
  ssh: {}
status:
  mounts:
  - env:
      criData: /var/lib/containerd
      defaultVIP: <IP_ADDRESS>
      disableApparmor: "false"
      registryConfig: /etc/registry
      registryData: /var/lib/registry
      registryDomain: sealos.hub
      registryPassword: passw0rd
      registryPort: "5000"
      registryUsername: admin
      sandboxImage: pause:3.8
    imageName: hub.sealos.cn/labring/kubernetes:v1.25.6
    labels:
      auth: auth.sh
      check: check.sh $registryData
      clean: clean.sh && bash clean-cri.sh $criData
      clean-registry: clean-registry.sh $registryData $registryConfig
      image: ghcr.io/labring/lvscare:v4.1.6
      init: init-cri.sh && bash init.sh
      init-registry: init-registry.sh $registryData $registryConfig
      io.buildah.version: 1.28.1
      org.opencontainers.image.description: kubernetes-v1.25.6 container image
      org.opencontainers.image.licenses: MIT
      org.opencontainers.image.source: https://github.com/labring-actions/cache
      sealos.io.type: rootfs
      sealos.io.version: v1beta1
      version: v1.25.6
      vip: $defaultVIP
    mountPoint: /var/lib/containers/storage/overlay/f446971c79475e35a495f509e4de5a60f8ff8ef9519978908930b550c50f7f56/merged
    name: default-rn5f5jsy
    type: rootfs
  - cmd:
    - cp -a opt/helm /usr/bin/
    imageName: hub.sealos.cn/labring/helm:v3.11.0
    labels:
      io.buildah.version: 1.28.1
    mountPoint: /var/lib/containers/storage/overlay/00da0d4e3488b71fe76f5fcdee72c38b574237769459ad5e4a3efd60a8d51a5b/merged
    name: default-h9cd45t3
    type: application
  - cmd:
    - bash entrypoint.sh
    imageName: hub.sealos.cn/labring/flannel:v0.21.4
    labels:
      io.buildah.version: 1.28.1
    mountPoint: /var/lib/containers/storage/overlay/3ef9b905bf3366a2730150e865047ece49e05e1b4d224c25f0e237214ffd0991/merged
    name: default-aezf5xvq
    type: application
  phase: ClusterInProcess

2023-04-27T15:16:12 info Executing ConfirmOverrideApps Pipeline in InstallProcessor

Operating environment
- Sealos version:
- Docker version:
- Kubernetes version:
- Operating system:
- Runtime environment:
- Cluster size:
- Additional information:

Additional information
https://github.com/labring/sealos/blob/d528d6be713b9b9cf92169e5822d354d29fffb9d/pkg/apply/processor/install.go#L103
seems to be here.

#2993
caused by this pr.
  - cmd:
    - cp -a opt/helm /usr/bin/
    imageName: hub.sealos.cn/labring/helm:v3.11.0
    labels:
      io.buildah.version: 1.28.1
    mountPoint: /var/lib/containers/storage/overlay/00da0d4e3488b71fe76f5fcdee72c38b574237769459ad5e4a3efd60a8d51a5b/merged
    name: default-h9cd45t3
    type: application
  - cmd:
    - bash entrypoint.sh
    imageName: hub.sealos.cn/labring/flannel:v0.21.4
    labels:
      io.buildah.version: 1.28.1
    mountPoint: /var/lib/containers/storage/overlay/3ef9b905bf3366a2730150e865047ece49e05e1b4d224c25f0e237214ffd0991/merged
    name: default-aezf5xvq
    type: application
  phase: ClusterInProcess

2023-04-27T15:16:12 info Executing ConfirmOverrideApps Pipeline in InstallProcessor
>> invalid input, please enter 'y', 'yes', 'n', or 'no'
2023-04-27T16:00:03 debug write cluster file to local storage: /root/.sealos/default/Clusterfile
Error: ^C

Is there an interactive prompt for the first apply? I recreate the process, and both apply have interactive prompts: Warning: Using an even number of master nodes is a risky operation and can lead to reduced high availability and potential resource wastage. " + 		"It is strongly recommended to use an odd number of master nodes for optimal cluster stability. " + 		"Are you sure you want to proceed?

Hi, @LZiHaN
Can you use sealos rmi labring/nginx:v1.23.3 to remove the image and try to apply again?

Hi, @LZiHaN
Can you use sealos rmi labring/nginx:v1.23.3 to remove image and apply again? I tried many times, but there was no blocking.

Hi, @LZiHaN Can you use sealos rmi labring/nginx:v1.23.3 to remove image and apply again? I tried many times, but there was no blocking.

Maybe you need to re-execute the above steps after sealos reset.

sealos reset & sealos rmi -f images
sealos apply -f Clusterfile
add hub.sealos.cn/labring/nginx:v1.23.3 to Clusterfile & sealos apply -f Clusterfile

first apply:
second apply:
I'm sorry I can't reproduce the problem. But another problem was found, prompting too many times.

no interactive prompt  this happens when I use.

sealos reset & sealos rmi -f images
sealos apply -f Clusterfile
add hub.sealos.cn/labring/nginx:v1.23.3 to Clusterfile & sealos apply -f Clusterfile

first apply:
second apply:
I'm sorry I can't reproduce the problem. But another problem was found, prompting too many times.

maybe you can fix your known problems, let me troubleshoot and modify this bug.

fixed #3020


Added a stack trace to error handling.

At the moment error handling is a bit opaque...
e.g.
 Morearty: render error. Continuing.
Morearty.js:292 Error details: undefined is not a function

This pull requests adds the stack trace to the error logging.
e.g.
 Morearty: render error. Continuing.
Morearty.js:292 Error details: undefined is not a function TypeError: undefined is not a function
    at eval (webpack:///./views/planting/New.jsx?:84:109)
    at eval (webpack:///../~/immutable/dist/immutable.js?:1139:46)
    at eval (webpack:///../~/immutable/dist/immutable.js?:3173:43)
    at List.__iterate (webpack:///../~/immutable/dist/immutable.js?:2688:13)
    at OrderedMap.__iterate (webpack:///../~/immutable/dist/immutable.js?:3172:25)
    at KeyedIterable.mappedSequence.__iterateUncached (webpack:///../~/immutable/dist/immutable.js?:1138:23)
    at seqIterate (webpack:///../~/immutable/dist/immutable.js?:601:16)
    at KeyedIterable.Seq.__iterate (webpack:///../~/immutable/dist/immutable.js?:261:14)
    at KeyedIterable.mixin.forEach (webpack:///../~/immutable/dist/immutable.js?:4256:19)
    at eval (webpack:///../~/immutable/dist/immutable.js?:3125:16)Morearty.js:292 catchingRenderErrorsMorearty.js:301 render

Thanks, @andru. This is something we wanted to do as well.


Repo not found or no access token provided

Hello,
After clicking on the JitPack link: "https://jitpack.io/#christophesmet/android_maskable_layout"
I'm getting this text:
"Repo not found or no access token provided".
Also in my project, it says: "Error:(125, 13) Failed to resolve: com.github.christophesmet:android_maskable_layout:-SNAPSHOT".

+1

Jitpack updated their artifact providing.
They give timeouts and appeared to be down for a while.
Try this:
repositories {
maven { url 'https://jitpack.io' }
}
dependencies {
compile 'com.github.christophesmet:android_maskable_layout:v1.0.0'
}
Tried and tested it, this should work :)

I've tried and got no luck. gradle console output:

Could not resolve com.github.christophesmet:android_maskable_layout:v1.0.0.
Could not get resource 'https://jitpack.io/com/github/christophesmet/android_maskable_layout/v1.0.0/android_maskable_layout-v1.0.0.pom'.
Could not GET 'https://jitpack.io/com/github/christophesmet/android_maskable_layout/v1.0.0/android_maskable_layout-v1.0.0.pom'.
Connect to <IP_ADDRESS>:64873 [/<IP_ADDRESS>] failed: Connection refused (Connection refused)
Any idea @christophesmet ?

Seems like an issue on your end. The dependency is available online, so it has something to do with your internet connection.
Closed: Jitpack has been updated, connection refused on clients end.


Issue: Export fails immediately

Hi Leon,
I'm running WordPress in a Docker container locally.
PHP version 8.0.10
Wordpress: latest
upload_max_filesize set to 500M
max_execution_time set to 36000
Downloaded WP2Static source from GitHub and built OK
Downloaded WP2Static add on and build OK
Wordpress installed and runs on localhost:8000 (default theme at this point)
install WP2Static and WP2Static S3 addon via Add New PlugIn and uploads the zip files - both activated
Add S3 credentials and bin settings to the addon.
Click on Run/Generate Static site
Error returns immediately error code 500 - message about increasing max_execution_time
Looking at the network tab on Chrome developer tools, there's no other errors except for the admin-ajax.php call with Form Data:
action: wp2static_run
Any ideas?
Many thanks

Hi @SChubb,
This is a common issue when using certain Docker configurations.
WP2Static attempts to crawl your site using the WP defined Site URL.
In a Docker setup, where Docker is exposing a port to the host, such as 8000 in your case, but the container's webserver serves at 80, WP2Static runs on the container and is attempting to crawl http://localhost:8000 (the WP site URL), but there is no webserver running on the container at 8000, only 80.
Editing the config per Docker guest to serve at the same port as is exposed to the host will fix this, but may be impractical, depending on your workflow.
My Lokl project makes this easier, allowing to quickly run multiple local WP sites optimised for static publishing on your local computer.
Please explore adjusting your current Docker setup or trying Lokl and letting me know if you find a suitable solution.
I'll leave this open to try and provide better error messaging in this case.

Hi @leonstafford,
Thanks for the response - couple of questions if that's OK?
So, to test things are working properly, I have WordPress running locally (not in a Docker container) via the WAMP stack - with a basic theme running. Site is available under localhost/mysite and can browse to it happily enough.
I've added in the WP2Static and WP2Static-S3-addon plugins and configured the S3 plugin with credentials that I've verified have upload rights to the named S3 bucket.
There's a couple of blank fields in the settings S3 settings - (profile, path prefix and all Cloudfront settings) - I'm assuming region would be in the form of eu-west-1? Sorry but the link to the docs https://wp2static.com/addons/s3/ is a 404.
Now, running WP2Static generates the following log:
2021-09-10 10:12:53: S3 Addon deploying 2021-09-10 10:12:53: Starting deployment 2021-09-10 10:12:53: Post-processing completed 2021-09-10 10:12:53: Finished processing crawled site. 2021-09-10 10:12:16: Processing crawled site. 2021-09-10 10:12:16: Starting post-processing 2021-09-10 10:12:16: Crawling completed 2021-09-10 10:12:16: Crawling complete. 4 crawled, 2354 skipped (cached). 2021-09-10 10:11:57: 404 for URL /robots.txt 2021-09-10 10:11:53: 404 for URL /favicon.ico 2021-09-10 10:11:38: Using CrawlCache. 2021-09-10 10:11:38: Starting to crawl detected URLs. 2021-09-10 10:11:38: Starting crawling
When it hits the S3 deploying line, it throws the error about error code 500 - message about increasing max_execution_time - even though when looking at the Diagnostics page, it confirms PHP execution time is unlimited.
Any ideas?

Hi @SChubb,
Couple of things come to mind:

it shows most files not crawled due to caching. If there are files in the wp-content/uploads/wp2static-processed-site dir after a failure, you can be sure it got that far at least. If nothing in there, try WP2Static > Caches > Delete all caches, to confirm if site is actually being crawled.
the check max_execution_time message isn't great, it's suggested as a common issue on some hosting, but it can be misleading when an unrelated issue. We'll improve that
WAMP for this plugin is much less reliable than any Linux/UNIX/BSD based hosting. I can't recall if WAMP uses Windows filepaths or UNIX filepaths (C:\dir vs /dir). If it uses Windows ones, I'd try a different local hosting environment that does use UNIX file paths
if you can get the processed site to generate OK, then you could try the ZIP deployment addon and use the AWS CLI tools to deploy the generated static site. That's often helpful, just to get your first deploy done all the way through, then can come back and figure out any issues with deploying to S3 directly from the plugin

Not sure if the above will sort things out for you, please let me know if they don't and we'll try some other things.
Cheers,
Leon

Hi @SChubb,
This is a common issue when using certain Docker configurations.
WP2Static attempts to crawl your site using the WP defined Site URL.
In a Docker setup, where Docker is exposing a port to the host, such as 8000 in your case, but the container's webserver serves at 80, WP2Static runs on the container and is attempting to crawl http://localhost:8000 (the WP site URL), but there is no webserver running on the container at 8000, only 80.
Editing the config per Docker guest to serve at the same port as is exposed to the host will fix this, but may be impractical, depending on your workflow.
My Lokl project makes this easier, allowing to quickly run multiple local WP sites optimised for static publishing on your local computer.
Please explore adjusting your current Docker setup or trying Lokl and letting me know if you find a suitable solution.
I'll leave this open to try and provide better error messaging in this case.

Is there a way to expose the correct port via docker-composer yaml file?


vertical centring

This might be a mistake in my css or html.
When I first open the modal window it is not centered vertically - if I resize the window it then centres correctly.
I am not adding anything to the init - just   $('#mydiv').plainModal('open')
Any ideas?
Thanks for nice simple plug in!

Hi cretace, thank you for your post.
So, I have questions:

Which browser (and version) does that problem occur in?
Could you show me your code? (JS, HTML and CSS)
Does that problem occur also in my DEMO page and that browser?

No of course it doesn't appear on your own page!
It's the same in FF and Chrome for me
But really I just wondered if you had any idea what might cause it.

Thanks for info.
So, could you show me your code?
If you can't, you can use such as http://jsfiddle.net/ to show sample code.

Ok It seems to have settled down now
I was loading text via an ajax call  after opening the modal box - I've now moved the open modal to end of the ajax  (doh!)
Thanks

Ok, I see. :smile:
Now, I understand your case.
I think that your code changed the size of modal window by adding contents.
The information about offset option below will help you.
http://anseki.github.io/jquery-plainmodal/#offset
The modal window is positioned considering that size at a necessary timing. Therefore if you changed that size after that was positioned, you must fix the position. plainModal provides the way to do that via offset option.
Anyway, this issue was solved? and it's closed?


Export world or extract

Would be neat to be able to export it somehow so it can be used for notetaking etc.

Go to the configuration page for your world. The URL looks like http://www.yourworldoftext.com/accounts/configure/~andrew/example.

Replace "configure" with "printable", like http://www.yourworldoftext.com/accounts/printable/~andrew/example

This only works for smaller worlds for now...

CC @chauffer

Thanks so much! Awesome thing :)

You're welcome! :)


Project Autocomplete Dropdown Does Not Allow for Project Selection

💻 Environment

Platform: macOS
OS Version: 10.15
Toggl Version: 7.4.1023
🐞 Actual behavior
When typing in a project name from the "What Are You Doing" area, a dropdown appears where suggested projects are listed. Projects on this list are not able to be selected by a cursor. Occasionally, quitting and restarting the app helps.
💯 Expected behavior
Projects in the dropdown should be selectable.
🔨 Steps to reproduce

Open app
Start typing in a description and/or project name in the "What Are You Doing" window
Attempt to click on a project name to select it

📦 Additional info
User Report
(above report includes log files)

I would close this ticket since it's already fixed in https://github.com/toggl/toggldesktop/pull/3419.
We're aiming to release the BETA on next Monday 👍

@NghiaTranUIT Can you ping me in Slack if/when ready so I can inform the user?

Sure, will notify you when we release the DEV today and the BETA for next Monday. It would include the fix 👍


Implemented Ternary Search in Java

Have you read the Contributing Guidelines on Pull Requests?
Yes
Description
Implemented Ternary Search in Java
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
#1333

@atarax665  Please review. Thank you.

@atarax665 @iamrajiv Updated Readme also. Please merge this pull request.


Write a prettier plugin that formats admonition blocks as markdown

Problem

Prettier ignores admonition blocks in documentation because it doesn't know they are Markdown. This forces documentation writers to manually wrap these blocks and reviewers to request changes to accommodate this.
Description

Write a plugin for prettier that identifies admonition blocks and passes them through the Markdown formatter.

Duplicate of https://github.com/WordPress/openverse/issues/1986


spring-openid - How to get clientId and clientSecret?

This is regarding the "spring-openid" I don't see you provided README.md or steps to execute this particular project. Also there is no details, how to obtain the google.clientId and google.clientSecret.
I see querydsl and some other project are not running properly. Did you tested these projects on Eclipse or STS?

That's a good point - I'll add it to the TODO list - should be sorted out soon.
As for QueryDSL - I see you already opened a separate issue - that's good - I'll follow up there.
Cheers,
Eugen.

Quick update on this - there is now a readme in the module - hope that sorts things out.
Cheers,
Eugen.


Add stdin support to rich.json

Type of changes

[ ] Bug fix
[x] New feature
[ ] Documentation / docstrings
[ ] Tests
[ ] Other

Checklist

[x] I've run the latest black with default args on new code.
[ ] I've updated CHANGELOG.md and CONTRIBUTORS.md where appropriate.
[ ] I've added tests for new code.
[x] I accept that @willmcgugan may be pedantic in the code review.

Description
Add support for rich.json to read input from stdin (eg. pipes) when no file name is provided.

Unsure if I could add a test for this.
Does this really warrant adding myself to contributors.md?

The other commands python -m syntax, and python -m markdown use a different convention, where if the path is - then it reads from stdin. Probably best to use that for consistency sake.
I haven't added tests for these "main" blocks. I consider them a demo more than anything else, not the core lib. That might change in the future, but for now its fine not to test this.
Yeah, please add yourself to contributors.md!

Yeah I should've noticed the mention of '-' in the help string of the PATH argument. Thanks!

Fantastic. One last thing, would you mind adding a new Unreleased entry at the top of changelog.md?

Done ✨

LGTM. Thanks Tushar.


Adds support for upgrading versions from 4.7.5 to 5.x thus connecting…

Adds support for upgrading versions to the 4.7.5->5 path

JIRA Ticket:Resolves: https://jira.lyrasis.org/browse/FCREPO-2984

Other Relevant Links (Mailing list discussion, related pull requests, etc.)

What does this Pull Request do?
Users can now migrate versions from 4.7.5 to 5.1.1 and by implication 6.
What's new?

Support for versions 4.7.5->5.x
Enhanced integration test

How should this be tested?
From the fcrepo-migration-utils:
mkdir -p ~/tmp/fcrepo-5.1.1-with-versions-export
# upgrade 4.7.5 to 5.1.1
java -Dfcrepo.log=DEBUG -jar target/fcrepo-upgrade-utils-6.0.0-SNAPSHOT.jar -i src/test/resources/4.7.5-export  -s 4.7.5 -t 5+ -o ~/tmp/fcrepo-5.1.1-with-versions-export -u http://localhost:8080/rest
# upgrade 5.1.1 to 6.
mkdir  -p ~/tmp/fcrepo-6.0.0-ocfl-with-versions/data
java -Dfcrepo.log=DEBUG -jar target/fcrepo-upgrade-utils-6.0.0-SNAPSHOT.jar -i /Users/danielbernstein/tmp/fcrepo-5.1.1-with-versions-export-3  -o ~/tmp/fcrepo-6.0.0-ocfl-with-versions/data  -u  http://localhost:8080/rest -s 5+ -t 6+

# run the latest  fcrepo main:
mvn -Dfcrepo.home=~/tmp/fcrepo-6.0.0-ocfl-with-versions jetty:run -pl fcrepo-webapp

Browse the application and verify that  you can see all the resources.
Note for binaries and binary metadata you'll need to access Fedora via the command line:
curl -I http://localhost:8080/rest/container1/tesbinary/fcr:metadata/fcr:versions
curl -I http://localhost:8080/rest/container1/tesbinary/fcr:versions

Interested parties
Tag (@ mention) interested parties or, if unsure, @fcrepo4/committers

I am unable to test this since F4 versions appear to no longer be exported. See: https://github.com/fcrepo-exts/fcrepo-import-export/issues/155
My test process is:

Start F4
Create F4 objects/binaries with versions
Run import/export of F4
Run upgrade/utils on export -> F5
Run upgrade/utils on export -> F6
Start F6 over export

The third step above does not produce version exports... which is one of the key points of testing for this PR.
@dbernstein : did you employ a different process for creating exported F4 versions?

@awoods : did you use fcrepo-import-export-0.3.0 to create your fcrepo 4.7.5 export?

Hi @dbernstein ,
No, I did not. Thanks for the link to the migration wiki. I have now moved it to be a child of: https://wiki.lyrasis.org/display/FF/Fedora+Upgrade+Notes

@awoods I was just reading through your comment and one thing struck me. You ran:
java -jar fcrepo-upgrade-utils-6.0.0-SNAPSHOT.jar -i ../f4-export/ ../f5-upgrade -s 4.7.5 -t 5+ -u http://localhost:8080/rest
And saw exceptions like:
ERROR 13:35:34.267 (MigrateResourceTask) Failed to process ResourceInfo{parentId='info:fedora', fullId='info:fedora/a', nameEncoded='a', outerDirectory=../f5-upgrade/rest, innerDirectory=../f5-upgrade/rest/a, type=CONTAINER} java.time.format.DateTimeParseException: Text 'v1' could not be parsed at index 0
???
I thought MigrateResourceTask was just used when migrating to F6, but your command is for migrating to F5.

It is a good question, @pwinckles . I have not looked into where MigrateResourceTask is getting pulled in here... or why in the F5 context.

@awoods :  I'll make sure that the output directory is being created automatically in 5->6

I ran a tweaked version of @awoods's script against F5, did an export of F5, and then migrated that export to F6 without any problems.
I then did a cursory comparison of the exported F5 content against the upgraded F4-F5 content, and it appears that versions are not being handled correctly. For example:
# Upgraded content
$ ls f5-upgrade/rest/objects/raven/pages
cover  cover.ttl  cover.ttl.headers  page0.ttl  page0.ttl.headers  page1.ttl  page1.ttl.headers

# F5 native content
$ ls f5-export/rest/objects/raven/pages
cover  cover.ttl  page0  page0.ttl  page1  page1.ttl

$ ls f5-export/rest/objects/raven/pages/page0
fcr%3Aversions.ttl

Note the missing page0/fcr%3Aversions.ttl.

In trying to upgrade a repository from 5->6 that was deployed to Tomcat (i.e. /fcrepo/rest/), I get the following error:
java.io.UncheckedIOException: java.nio.file.NoSuchFileException: ../f5-upgrade/rest.ttl
	at org.fcrepo.upgrade.utils.f6.RdfUtil.parseRdf(RdfUtil.java:78)
	at org.fcrepo.upgrade.utils.f6.ResourceMigrator.readRdf(ResourceMigrator.java:500)
	at org.fcrepo.upgrade.utils.f6.ResourceMigrator.migrateContainer(ResourceMigrator.java:162)
	at org.fcrepo.upgrade.utils.f6.ResourceMigrator.migrate(ResourceMigrator.java:126)
	at org.fcrepo.upgrade.utils.f6.MigrateResourceTask.run(MigrateResourceTask.java:59)

..due to the assumption: https://github.com/fcrepo-exts/fcrepo-upgrade-utils/blob/4.7-5.x/src/main/java/org/fcrepo/upgrade/utils/F5ToF6UpgradeManager.java#L55
This issue should be worked in a separate JIRA/pull-request.

External content appears to work. It would be ideal if someone with an F4 with ACLs could test that ACLs are correctly migrated.

@awoods :  ACLs are not covered in this PR. They will not work.

In trying to upgrade a repository from 5->6 that was deployed to Tomcat (i.e. /fcrepo/rest/), I get the following error:
java.io.UncheckedIOException: java.nio.file.NoSuchFileException: ../f5-upgrade/rest.ttl
	at org.fcrepo.upgrade.utils.f6.RdfUtil.parseRdf(RdfUtil.java:78)
	at org.fcrepo.upgrade.utils.f6.ResourceMigrator.readRdf(ResourceMigrator.java:500)
	at org.fcrepo.upgrade.utils.f6.ResourceMigrator.migrateContainer(ResourceMigrator.java:162)
	at org.fcrepo.upgrade.utils.f6.ResourceMigrator.migrate(ResourceMigrator.java:126)
	at org.fcrepo.upgrade.utils.f6.MigrateResourceTask.run(MigrateResourceTask.java:59)

..due to the assumption: https://github.com/fcrepo-exts/fcrepo-upgrade-utils/blob/4.7-5.x/src/main/java/org/fcrepo/upgrade/utils/F5ToF6UpgradeManager.java#L55
This issue should be worked in a separate JIRA/pull-request.

https://jira.lyrasis.org/browse/FCREPO-3522

External content appears to work. It would be ideal if someone with an F4 with ACLs could test that ACLs are correctly migrated.

https://jira.lyrasis.org/browse/FCREPO-2985


[material-ui][FormControlLabel] Fix applying of slotprops

closes https://github.com/mui/material-ui/issues/42277

[x] I have followed (at least) the PR section of the contributing guide.

I don't think argos change is related to this PR, reason being neither FormControlLabel or Typography components are used to render chart

Hey @sai6855, thanks for working on this
Does https://github.com/mui/material-ui/pull/42358 fix it?

Does https://github.com/mui/material-ui/pull/42358 fix it?

yes, it did. tried your fix in this commit https://github.com/mui/material-ui/pull/42303/commits/eecb0a259240509e027a44f035c84f1fb4d78358 and newly added test is passing, also checked manually whether styles are getting applied properly.


RBTree - fix potential nullptr dereference

This fixes a potential nullptr dereference. The node color comparison handles that by treating a nullptr as a BLACK node. Most of the code was converted to use that, but this line wasn't. Not sure why we haven't seen this in production, but it was breaking things for me while testing TxnBox. As long as I fixed it there, I might as well get it fixed here.

Yes, this should go in 9.0.

Cherry-picked to v9.0.x branch.