SwapRoutes tests fail on macOS

When running the tests of swap routes, I get an error:
pyvrp/search/tests/test_SwapRoutes.py ..Fatal Python error: Aborted

Current thread 0x0000000202b42080 (most recent call first):
  File "/Users/leonlan/Dropbox/PyVRP/pyvrp/search/tests/test_SwapRoutes.py", line 36 in test_apply
	...
Extension modules: numpy.core._multiarray_umath, numpy.core._multiarray_tests, numpy.linalg._umath_linalg, numpy.fft._pocketfft_internal, numpy.random._common, numpy.random.bit_generator, numpy.random._bounded_integers, numpy.random._mt19937, numpy.random.mtrand, numpy.random._philox, numpy.random._pcg64, numpy.random._sfc64, numpy.random._generator, numpy.linalg.lapack_lite, matplotlib._c_internal_utils, PIL._imaging, matplotlib._path, kiwisolver._cext, matplotlib._image, matplotlib.backends._macosx (total: 20)
[1]    84851 abort      poetry run pytest

Specifically, these tests fail:
https://github.com/PyVRP/PyVRP/blob/f246c30322ae4f059aca191eeffa0c225592faae/pyvrp/search/tests/test_SwapRoutes.py#L17-L20

I have a hard time getting the debugger to work. (Specifically, I can attach to Python process with lldb and step into the C++ calls, but I can't get source code to appear in lldb only machine instructions.)
Anyway, isolating the test and running it in this script.py:

script.py
from typing import List

import pytest
from numpy.testing import assert_, assert_allclose, assert_equal

from pyvrp import CostEvaluator, VehicleType
from pyvrp.search import SwapRoutes
from pyvrp.search._search import Node, Route
from pyvrp.tests.helpers import make_heterogeneous, read

L = [
    ([], [1]),  # first empty, second non-empty
    ([1], [2, 3]),  # both non-empty but unequal length
    ([2, 3], [1]),  # both non-empty but unequal length (flipped)
    ([2, 3], [1, 4]),  # both non-empty equal length
]

def test_apply(visits1: List[int], visits2: List[int]):
    """
    Tests that applying SwapRoutes to two different routes indeed exchanges
    the visits.
    """
    data = read("data/OkSmall.txt")

    route1 = Route(data, idx=0, vehicle_type=0)
    for loc in visits1:
        route1.append(Node(loc=loc))

    route2 = Route(data, idx=1, vehicle_type=0)
    for loc in visits2:
        route2.append(Node(loc=loc))

    route1.update()
    route2.update()

    # Before calling apply, route1 visits the clients in visits1, and route2
    # visits the clients in visits2.
    assert_equal(visits1, [node.client for node in route1])
    assert_equal(visits2, [node.client for node in route2])

    op = SwapRoutes(data)
    op.apply(route1, route2)

    # But after apply, the visits are now swapped.
    assert_equal(visits2, [node.client for node in route1])
    assert_equal(visits1, [node.client for node in route2])

for v1, v2 in L:
    test_apply(v1, v2)

results in the following error:
Python(95471,0x202b42080) malloc: Heap corruption detected, free list is damaged at 0x600002da3fe0
*** Incorrect guard value: 0
Python(95471,0x202b42080) malloc: *** set a breakpoint in malloc_error_break to debug
[1]    95471 abort      poetry run python pyvrp/script.py

However, the failure sometimes doesn't occur (like one out of 10 times).

Perhaps superfluous: I ran the above script ten times through the debugger (gdb) on my platform (Ubuntu). No issues. I don't know if I can be of much help debugging this issue.

I'm going to pause working on this because I just can't figure it out. I'll just continue working while commenting this test out.
Some things I have tried:

Benchmarking all GH and X instances for 1 second with only SwapRoutes does not lead to any problems.
I have added print statements in many places of the source code. But it is impossible to find the exact place where the fault occurs. From running the script above, it seems that the error mostly happens when doing something with route 2. Sometimes it's when constructing the route, sometimes it's when inserting the node, ...

Updated script.py with print statements
from typing import List
import pytest
from numpy.testing import assert_, assert_allclose, assert_equal
from pyvrp import CostEvaluator, VehicleType
from pyvrp.search._search import Node, Route
from pyvrp.tests.helpers import make_heterogeneous, read
def test_apply(visits1: List[int], visits2: List[int]):
"""
Tests that applying SwapRoutes to two different routes indeed exchanges
the visits.
"""
print("Py: New test.")
data = read("data/OkSmall.txt")
print("Py: Create route 1.")
route1 = Route(data, idx=0, vehicle_type=0)
for idx, loc in enumerate(visits1):
    print(f"Py: Start insert node: {idx}/{len(visits1)-1}")
    route1.append(Node(loc=loc))
    print(f"Py: End insert node: {idx}/{len(visits1)-1}")

print("Py: Create route 2.")
route2 = Route(data, idx=1, vehicle_type=0)
for idx, loc in enumerate(visits2):
    print(f"Py: Start insert node: {idx}/{len(visits2)-1}")
    route2.append(Node(loc=loc))
    print(f"Py: End insert node: {idx}/{len(visits2)-1}")

print("Py: Update route 1.")
route1.update()

print("Py: Update route 2.")
route2.update()

# Before calling apply, route1 visits the clients in visits1, and route2
# visits the clients in visits2.
assert_equal(visits1, [node.client for node in route1])
assert_equal(visits2, [node.client for node in route2])

from pyvrp.search import SwapRoutes

op = SwapRoutes(data)
op.apply(route1, route2)

print("Py: after apply.")

# But after apply, the visits are now swapped.
assert_equal(visits2, [node.client for node in route1])
assert_equal(visits1, [node.client for node in route2])

L = [
([], [1]),  # first empty, second non-empty
([1], [2, 3]),  # both non-empty but unequal length
([2, 3], [1]),  # both non-empty but unequal length (flipped)
([2, 3], [1, 4]),  # both non-empty equal length
]
for v1, v2 in L:
test_apply(v1, v2)

That's interesting, because the MacBook Pro 2017 is Intel-based. So this issue may be specific to macOS.

Hmm it's getting interesting. While working on #369 I uncommented the SwapRoutes tests, but now thetest_wrong_load_calculation_bug() in SwapStar now also fails.

After realizing that SwapStar also fails, I figured that it must have been some way earlier commit that caused this issue.
I've managed to isolate the specific commit that introduced the failing bug (see detail below), which is a12c4fb.

Finding the commit
Ticked boxes means that the test fails.

[x] f347884 Simplify cli (#357)
[ ] 2f9d7b9 Allow incomplete solutions in LS (#354)
[ ] 0535b04 Cast measure (#356)
[ ] b989713 Fix wrong load difference evaluation in SWAP* (#353)
[ ] ae00385 Test within route two opt (#351)
[ ] 597ec84 Add page with some guidelines and experiences implementing new VRP variants in PyVRP (#350)
[ ] ad461cc Test descriptions (#349)
[ ] 7f75c2e (alns) Improve rng (#347)
[x] 5724b0f Make returned matrix non-writeable (#346)
[x] a12c4fb Add Route destructor
[ ] PASS 7bef1f0 Documentation fixes (#342)
[ ] PASS 28f8aed Cast Matrix/np.ndarray (#338)
[ ] PASS 10ab8c2 Add a SWAP* test that checks in-place swaps (#340)
[ ] 8801e27 Update description of symbols we reserve for internal use
[ ] PASS 5d67fe0 Simplify cli (#339)
[ ] 0c86f94 Merge branch 'main' of github.com:PyVRP/PyVRP
[ ] 248cf97 Remove superfluous helper
[ ] 9eb94b2 Pickle support for Solution and Solution::Route (#337)
[ ] PASS c8cd92b Simplify SWAP* (#302)

Removing the destructor would mean any nodes contained in the route would still reference a by-then-nonexistent Route as their route() member. That's also not ideal.

My first guess was that maybe we don't have the appropriate call/return value policies in place, but looking at the bindings I do not immediately see any issues with those.

What might be the case is that we need to keep the route object alive when we add the node to it. I'm not entirely sure if that is currently set up correctly.

Purely speculating, but what might be the case is that we need to keep the route object alive when we add the node to it. I'm not entirely sure if that is currently set up correctly. I think that's something like py::keep_alive<2, 1>, and might need to be added to append and insert?

This works for me!

What's the issue with setting up a debugger exactly?

The main problem was that after attaching to the Python process I can't get to step through the C++ source code. Tried this with terminal, VSCode, CLion. In some cases it just doesn't show anything, in other cases I get assembly (after which I then followed this guide to get source code to appear, but no success).
There were also these steps to use CLion for debugging the C++ code. I followed both approaches in the guide, but neither worked because CLion doesn't hit/identify the breakpoints.

I use this extension in VS Code, but I'm not sure if that also helps in your case. I do think we need to figure out a way to get the debugger to work for you, because it's very helpful in figuring this sutff out.
In any case: TIL that Python on Mac has a really active garbage collector :-).

I use this extension in VS Code, but I'm not sure if that also helps in your case. I do think we need to figure out a way to get the debugger to work for you, because it's very helpful in figuring this sutff out.

Yeah agreed, I'll look into this more the coming weeks!


pull request for e2e test at 12/14/2016 19:15:11

:x: Validation status: errors
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Port useful code from other projects along with small refactorings

Fix named imports on mockAxios per @brandongregoryscott feedback in previous PR of React package
Break out files where types are bundled in same files (ie. PromiseFactory)
Port additional utilities/types/etc... from previous projects

Working on this currently.

Published in v0.0.5


Clean IntelliJ "out" dir

Since IntelliJ now uses the "out" dir to store build files, I am wondering if this plugin should handle cleaning that dir by appending logic to the default clean task.
clean {
    delete 'out'
}

The plugin has nothing to do with idea’s compilation process. You can use any other output directory in idea, or not use idea compilation at all, or you can even use any other ide.

Whops, sorry got this repo confused with the gradle idea plugin.


The split method throws out of bounds error

Describe the bug
The split method is receiving an expected value from col_convert_df = col_read_df.loc[(col_read_df['Sheet Name'] == sht) & (col_read_df['Column Name'] == col)]
which seems to not have error handling, therefore the remaining .values[0] is out of bounds
To Reproduce
Steps to reproduce the behavior:

Call converter.py , file name: test_version5_flapjack_compiler_sbol3_v005.xlsx

Expected behavior
Provide helpful logging to the user about how to fix the error themselves if it is something wrong with the spreadsheet
Screenshots
If applicable, add screenshots to help explain your problem.

Additional context
            if cell_val != '':
                # checks that the cell isn't blank
                col_convert_df = col_read_df.loc[(col_read_df['Sheet Name'] == sht) & (col_read_df['Column Name'] == col)]

                # split method
                try:
                    print("checkpoint0 row number: ", row_num)
                    print(col_convert_df['Split On'])
                    split_on = col_convert_df['Split On'].values[0]
                    #print("checkpoint1")
                    split_on = split_on.split('"')
                    #print("checkpoint2")
                    split_on = [x for x in split_on if x != '']
                    #print("checkpoint3")
                    split_on = '[' + "".join(split_on) + ']'
                    #print("checkpoint4")
                except:
                    print ("This is the split on value")
                    print(split_on)
                    print ("Error Sheet Occurred")
                    print(col_read_df['Sheet Name'].values[0])
                    print ("Error Column Occurred")
                    print(col_read_df['Column Name'].values[0])

Traceback (most recent call last):
File "C:\Users\saisa\Excel-to-SBOL\excel2sbol\excel2sbol\converter.py", line 7, in
e2s(file_path_in, file_path_out, sbol_version=2)
File "C:\Users\saisa\Excel-to-SBOL\venv\lib\site-packages\excel2sbol\converter.py", line 23, in converter
e2s.column_parse(to_convert, compiled_sheets, sht_convert_dict,
File "C:\Users\saisa\Excel-to-SBOL\venv\lib\site-packages\excel2sbol\compiler_test.py", line 270, in column_parse
cell_val = lk.up(col_convert_df, cell_val,
File "C:\Users\saisa\Excel-to-SBOL\venv\lib\site-packages\excel2sbol\lookup_compiler.py", line 8, in up
if col_def_dict['Tyto Lookup'][0] and not col_def_dict['Sheet Lookup'][0]:
IndexError: list index out of range
Recommendation
Add some safety measures such as "try/except/finally" when looping through map values

Adding @cjmyers for visibility

Can you attach the file you are using?

I am rather  confused by the way you have documented this. It seems almost like separate errors. The screen shot does not actually help much. What is the code you have added as additional context? Is that part of your debugging? What prints out for those values? Do you think it is an issue with splitting that is showing up in the lookup? Based on the bottom error I would have guessed the issue was with an empty Tyto value or something similar but am not sure if you have tracked it down further.

test_version5_flapjack_compiler_sbol3_v005.xlsx

I added a try except for the original error in the split method which was
occurring somewhere in the Assay column definitions/sheet, which allowed
the code to continue past the error with the split method until the Tyto
Lookup stopped it again.
I was trying to understand how the col_read_df [Sheet Name] and col_read_df
[Column Name] worked to see why the split method was not being properly
processed on my sheet and possible issues with Assay contributing to it.
On Mon, Mar 21, 2022, 1:21 PM JMante1 @.***> wrote:

I am rather confused by the way you have documented this. It seems almost
like separate errors. The screen shot does not actually help much. What is
the code you have added as additional context? Is that part of your
debugging? What prints out for those values? Do you think it is an issue
with splitting that is showing up in the lookup? Based on the bottom error
I would have guessed the issue was with an empty Tyto value or something
similar but am not sure if you have tracked it down further.
—
Reply to this email directly, view it on GitHub
https://github.com/SynBioDex/Excel-to-SBOL/issues/104#issuecomment-1074324373,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACQZPMLY35ZVKY7LMHDFLLLVBDD5FANCNFSM5RF2XSZQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

I think this might have been solved by one of the other updates as I can't see this error anymore


dynamic reweighting causes performance degradation in reproducing

Hi,
thanks for sharing the code! Great work!
I have a small question in reproducing your result.
I run the CDN-S model (res50, 3+3). It gave a result of about 31.5 or 31.2 (I run 2 times) after the first training stage (train the whole model with re gular loss). But after the second training stage (decoupled training) is finished, the performance downgrades to 31.0 and 30.4 for these 2 runs separately. For full mAP, rare mAP and non-rare mAP, this trick seems to be not helpful.
So I wonder what could goes wrong during my reproduction or what can be the reason. I will paste the commands and log below. Thanks. Nice day :3

@zhangaixi2008 It does not work for me either. The reweighting retraining leads to performance drop.
Details are as follows:

Using given script training on HICO-DET:
CDN S:

best in first 90 epoch: 31.71
fine-tune degrades to 30.96
CDN B:
best in first 90 epochs: 31.6
fine-tune degrades to 30.6

On CDN-S, try your script above:
fine-tune degrades to 30.4 (w/o reweight 31.71).
log.txt

Please help  me
.

@zhangaixi2008 It does not work for me either. The reweighting retraining leads to performance drop.
Details are as follows:

Using given script training on HICO-DET:

CDN S:

best in first 90 epoch: 31.71
fine-tune degrades to 30.96

CDN B:

best in first 90 epochs: 31.6
fine-tune degrades to 30.6

I'm using the above script to re-run cdn-s finetuning.

@haak0 please upload your model here, let me have a look.

@zhangaixi2008 Hi, some of my checkpoints are overwritten. I am re-running the experiments.

Hi, I re-do the experiments, and here's the log.
CDN small:

best in first 90 epoch: 30.99
fine-tune degrades to 30.3
Here's the script and log small.txt

CDN base:

best in first 90 epochs: 31.98
fine-tune degrades to 30.6
Here's the script and log  base.txt

I mean, you can give a link for your models of first 90 epochs, and I will check the fine-tune result based on your model.

Hi, I made a mistake in the previous readme for running the fine-tune process. Please use the script I provide above under this issue. As we claimed in the paper, we use a small learning rate to fine-tune the first model. Thus, we set lr as 5e-6 and lr_backbone as 5e-7 for bs=8, or lr as 1e-5 and lr_backbone as 1e-6 for bs=16. Please try again and let us see the results.
Sorry for our carelessness, we have already modified the readme.

@zhangaixi2008 hi, I reproduce the finetune result following your script, and the result is reasonable.
CDN-base:

first 90 epoch: 32.05
fine-tune: 32.12
All the result are evaluated using the python script.

BTW, what's the meaning of the "vis_tag" in hico_eval.py?

For CDN-base, you have already surpassed our reported (official matlab 31.78, python 31.86) in our paper. Good job^^
For 'vis_tag', you can see the evaluation script. In short, we filtered the already matched ground-truth hoi to calculate fp and tp during evaluation.

The issue about the "Re-weight module" seems to be solved. If any other issues, feel free to open a new issue.


Support EIP-155 in E2E test

We update dependencies to the latest versions to support EIP-155 replay-protected transactions.
PrivateKeyWalletSubprovider expects chainId to be provided as a constructor argument. If the value is not provided the provider assumes it deals with mainnet (chainId = 1).
After EIP-155 the chainId is included when calculating a signature, and transactions that are sent to the Ethereum endpoint with non-matching chainId are rejected with invalid sender.
The previous solution worked fine with Ethereum endpoint that had replay protection disabled, but once the protection is enabled we need to include the correct chainId in the transaction signature.

I invoked a test run for this branch to see if the fix works: https://github.com/keep-network/local-setup/runs/3789133012

I invoked a test run for this branch to see if the fix works: https://github.com/keep-network/local-setup/runs/3789133012

I see that the job has failed during installation of keep-ecdsa, which is strange, because your changes didn’t touch that area (and E2E tests on main are failing at later stage) :thinking_face:

Next try: https://github.com/keep-network/local-setup/actions/runs/1434707961

Next try: https://github.com/keep-network/local-setup/actions/runs/1434707961

Unfortunately it's still failing with this web3 is not defined issue...

Next try: https://github.com/keep-network/local-setup/actions/runs/1434707961

Unfortunately it's still failing with this web3 is not defined issue...

Of course, it does fail. I forgot to commit the change 😅 I'm sorry, one more try! 🤞🏻
https://github.com/keep-network/local-setup/actions/runs/1439292511

Of course, it does fail. I forgot to commit the change 😅 I'm sorry, one more try! 🤞🏻 https://github.com/keep-network/local-setup/actions/runs/1439292511

This time we've got this:
Starting deposit number [1]...

Test errored out with error:  TypeError: Cannot read property 'toBN' of undefined
    at createDeposit (file:///home/runner/work/local-setup/local-setup/e2e/e2e-test.js:265:16)

@nkuba, what's the status of this PR? Any chance for getting the tests fixed?


Update/upgrade docs

What would you like to be added:
Add docs about how to update and upgrade your operator under features docs section. Make sure to explain what are update and upgrade plans and how to use them through CLI.
Why is this needed:

we have some details in the runbook and we have operator specific docs (example: https://kudo.dev/docs/runbooks/kafka/upgrade.html)
We should promote update / upgrade in standard docs from a conceptual standpoint to augment the operator docs!


Request Headers Set is Not working in Web Connectivity Component (Web.java)

https://github.com/mit-cml/appinventor-sources/blob/master/appinventor/components/src/com/google/appinventor/components/runtime/Web.java
line no - 1196
-->>  connection.addRequestProperty(name, value);
this line are not set the headers Correctly
please replace this line with this
-->>  connection.setRequestProperty(name, value);
Thankx

hello, I want to work on this issue


⚠ Samsung has degraded performance

In f831295, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2659 ms

Resolved: Samsung performance has improved in b696523.


[SPARK-11537] [SQL] fix negative hours/minutes/seconds

Currently, if the Timestamp is before epoch (1970/01/01), the hours, minutes and seconds will be negative (also rounding up).

Merged build triggered.

Merged build started.

Test build #45142 has started for PR 9502 at commit a0bfae4.

Test build #45142 has finished for PR 9502 at commit a0bfae4.

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/45142/
Test PASSed.

Merged build triggered.

Merged build started.

Test build #45155 has started for PR 9502 at commit 8719a61.

Merged build triggered.

Merged build started.

Test build #45167 has started for PR 9502 at commit a02a5c1.

Test build #45155 has finished for PR 9502 at commit 8719a61.

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

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

Merged build finished. Test PASSed.

Merging this into master and 1.5, 1.6 branch.

Test build #45167 has finished for PR 9502 at commit a02a5c1.

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/45167/
Test PASSed.


Flash Bluetooth firmware

I used rpi-update to upgrade an outdated copy of Pi MusicBox to Jessie. But I was struggling to get the Bluetooth controller to work, until I found a magical incantation on some forum:
/usr/bin/hciattach /dev/ttyAMA0 bcm43xx 921600 noflow -

After running that, the Bluetooth controller worked fine. Is flashing the Bluetooth firmware something that rpi-update should do for me?

Hm, it looks like the pi-bluetooth package is supposed to do it:

Loads BCM43430A1 firmware on boot


OEL9: Cannot connect to legacy systems, getting error message "no matching host key type found. Their offer: ssh-rsa"

Trying to connect from an OEL9 system in FIPS mode to a legacy system (CentOS6), the following error message is displayed
Raw
$ ssh legacy-system
Unable to negotiate with IPAddr port 22: no matching host key type found. Their offer: ssh-rsa
Trying to connect from an OEL8 system works fine
This was described and apparently fixed in https://access.redhat.com/solutions/7056072

Oracle Linux customers, please file your issue at https://support.oracle.com
Thanks for filing an issue with Oracle Linux.
GitHub Issues is not an official support channel and we don't offer
product support here. If you're not yet an Oracle Linux customer,
consider signing up at https://linux.oracle.com.
Even if you're not a customer, if we can confirm that an issue is a
bug we will do our best to fix it and to update this issue
once it has been fixed. We don't guarantee a fix or feedback and
for now, we will close this issue. If you have Oracle Linux support,
please use support.oracle.com to report issues.

OL9 is compatible with RHEL9 so the solution offered above should work the same on OL9. If not, please let us know what part of that solution is working differently in OL9 compared to RHEL9.

Thank you for replying, hbergh!
I tried RHEL's "Solution 1 - regenerate keys on the legacy system", via generating an ECDSA key on the server, adding it to the sshd_config there, and restarting sshd.  I assume (don't have a RHEL9 system to test with) this works on RHEL9 given that they present it as a solution, but it didn't work on OEL9 (client) with CentOS6 (server).  The OEL9 client runs "OpenSSH_8.7p1, OpenSSL 3.0.7 1 Nov 2022" and the CentOS6 server runs "OpenSSH_5.3p1, OpenSSL 1.0.1e-fips 11 Feb 2013".  Thank you!

Did you also generate an ecdsa key pair and add the id_ecdsa.pub to .ssh/authorized_keys of the Centos6 user? What was the exact error message you got, and can you provide ssh -vvv output?

No, I didn't generate an ECDSA key pair on the OEL9 client; the error message complained about a "host key" so I didn't think that would be relevant. Am I wrong on that?  Normally I expect that when a server's host key changes, the SSH client prompts for whether to accept the new key.
Given that the server apparently continued to offer only "ssh-rsa", I took that to mean that it wasn't presenting the ECDSA host key, and assumed then that the server didn't handle ECDSA host keys.  I didn't think that the issue was the client's public key pair, I didn't think that should have required any change.  Again, maybe I was wrong?
The error message was the same as in the RHEL "solution":
(OEL9) $ ssh legacy-system
Unable to negotiate with IPAddr port 22: no matching host key type found. Their offer: ssh-rsa

Unfortunately I didn't save the verbose output (it all looked normal except for that error message), and I reverted the system (removed the ECDSA key) because it's a production server and that caused other problems for a user.  If need be I can probably redo the change to capture the "-vvv" output.
Thank you -- Ken


[firebase_messaging] iOS Push Notification not Received in Notification tray

iOS Push Notification not Received in Notification tray (when app is Not running or in Background) Received when app is launched.
Followed all the steps to integrate firebase_messaging 6.0.16 into our Flutter Application.
PFB Flutter Code
@override
  void initState() {
    super.initState();
    //for android
    mainpref = SPManager();
    // With this, we will be able to check if the permission is granted or not
    // when returning to the application
    WidgetsBinding.instance.addObserver(this);
    // With this, we will be able to check if the permission is granted or not
    // when returning to the application

    if (Platform.isIOS) {
      iOSPermission();
    }
    configLocalNotification();

    fcm.getToken().then((token) {
      FLog.debug(
          className: this.runtimeType.toString(), text: 'token : ' + token);
      mainpref.setNotificationToken(token);
    });

    fcm.configure(
      onMessage: (Map<String, dynamic> message) async {
        FLog.info(text: "onMessage: $message");
        showNotification(message);
      },
      onLaunch: (Map<String, dynamic> message) async {
        FLog.info(text: "onLaunch: $message");
        var tag =
            Platform.isIOS ? message['screen'] : message['data']['screen'];
        var data = Platform.isIOS ? message : message['data'];
        Utility().isLaunchedViaNotification = true;
        callPage(tag, data);
      },
      onResume: (Map<String, dynamic> message) async {
        FLog.info(text: "onResume: $message");
        var tag =
            Platform.isIOS ? message['screen'] : message['data']['screen'];
        var data = Platform.isIOS ? message : message['data'];
        callPage(tag, data);
      },
    );
  }

void iOSPermission() {
    //Needed by iOS only
    fcm.requestNotificationPermissions(const IosNotificationSettings(
        sound: true, badge: true, alert: true, provisional: false));
    fcm.onIosSettingsRegistered.listen((IosNotificationSettings settings) {
      mainpref.setalert(settings.alert);
      mainpref.setsound(settings.sound);
      mainpref.setbadge(settings.badge);
      FLog.info(text: "Settings registered: $settings");
    });
  }

iOS Code
import UIKit
import Flutter
import Firebase
import flutter_downloader

@UIApplicationMain

@objc class AppDelegate: FlutterAppDelegate {

  override func application(
    _ application: UIApplication,

    didFinishLaunchingWithOptions launchOptions: [UIApplication.LaunchOptionsKey: Any]?
  ) -> Bool {
    FirebaseApp.configure();
    GeneratedPluginRegistrant.register(with: self)
    if #available(iOS 10.0, *) {
      UNUserNotificationCenter.current().delegate = self as? UNUserNotificationCenterDelegate
    }

    FlutterDownloaderPlugin.setPluginRegistrantCallback { (registry) in
        if (!registry.hasPlugin("FlutterDownloaderPlugin")) {
            FlutterDownloaderPlugin.register(with: registry.registrar(forPlugin: "FlutterDownloaderPlugin")!)
        }
    }
    return super.application(application, didFinishLaunchingWithOptions: launchOptions)
  }

    override func application(_ application: UIApplication, handleEventsForBackgroundURLSession identifier: String, completionHandler: @escaping () -> Void) {
        completionHandler();
    }
}

To Reproduce
Steps to reproduce the behavior:

Sending the Push Notification using FCM Rest API. (https://fcm.googleapis.com/fcm/send)

Expected behavior
Notification should be received in notification tray when app is not running or app is in background.
Flutter doctor
[flutter] flutter doctor -v
[✓] Flutter (Channel beta, 1.19.0-4.3.pre, on Mac OS X 10.15.4 19E287, locale en-IN)
    • Flutter version 1.19.0-4.3.pre at /Users/iosdev/flutter sdk/flutter
    • Framework revision 8fe7655ed2 (12 days ago), 2020-07-01 14:31:18 -0700
    • Engine revision 9a28c3bcf4
    • Dart version 2.9.0 (build 2.9.0-14.1.beta)

[!] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at /Users/iosdev/Library/Android/sdk
    • Platform android-29, build-tools 28.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)
    ! Some Android licenses not accepted.  To resolve this, run: flutter doctor --android-licenses

[✓] Xcode - develop for iOS and macOS (Xcode 11.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.2.1, Build version 11B53
    • CocoaPods version 1.9.0.beta.2

[✓] Android Studio (version 3.6)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 45.1.1
    • Dart plugin version 192.7761
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)

[✓] VS Code (version 1.47.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.12.2

[✓] Connected device (2 available)
    • Air 2         • ***************************************************** • ios • iOS 13.5.1
    • iPhone 11 Pro • 9E3AE2F3-278E-4B2F-A466-4BF78F9B335E     • ios • com.apple.CoreSimulator.SimRuntime.iOS-13-2 (simulator)

! Doctor found issues in 1 category.
exit code 0

Please help me in this issue.
Thanks

I'm facing de same issue by now, Android works fine but iOS doesn't. There are various workarrounds of December and January on github with this same issue but nothing is working for me. I've configured everything that the docs are saying but I'm only receiving notifications with the app in foreground

I am facing the same problem, maybe you can try this comment #2854

Closing as duplicate of https://github.com/FirebaseExtended/flutterfire/issues/2854#issuecomment-656316262


Fix link to base script in howto docs

This pull request fixes a link to the _base script in the howto docs. Initially it was pointing to /themes/_base.sh, but at some point this script was moved to /_base.sh

Thank you


Feat:Improve the navbar of the homepage.

Describe the feature
Navbar's alignment seems improper as the screen size reduces.Also the hamburger icon on click does not displays anything.
Add ScreenShots

Record

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

@meetarora10 Assigned!


Add GitHub Super-Linter and editorconfig

I'm a big fan of linting, and GitHub Actions to automate checks. Given, how important and widely used this library is, I think it would be good to add these to ensure cleanliness of code and avoid bugs. Ideally we'd also automated testing the code, but linting is a start.
The GitHub Super-Linter is a GitHub Action which bundles many linters, implemented in many languages, in a Docker image, giving you access to all of them without needing to add any dependencies to the project. It also checks more than just code, but also Markdown, JSON, YAML...etc. For the actual JS/TS it will use the eslint config file already set up in this project.
I've also added an .editorconfig file to enforce certain things (spaces over tabs, no trailing spaces, file format, trailing newline at EOF...etc.) for those that use that (there's a VS Code plugin to automatically pick it up).
The good news is the code it very clean, with only a few small tweaks needed to get it to pass the linters.
With this PR, these standards would be enforced and automatically checked on each commit. It takes a couple of mins to run  and only runs on changed files (running for the whole code base still only takes 3 mins as this repo is small).
Let me know what you think.

In general I'm very supportive of adding these types of things. And I'd very much like to set up GitHub actions to run the tests, etc. automatically on PR submissions.
I think I'd prefer to wait until after v3 ships though, just to make sure none of these changes slow that down.

No bother. Want me to close this for now? Or happy to leave it open to remind us?
BTW a refactor/version upgrade is maybe the best time to have a linter 😉 And by default it won't be merge blocking (though we should add that once we're comfortable with it).
Will also hold off looking at CI tests for now (not sure how feasible this is on a headless server anyway), but definitely would be nice!

FYI @philipwalton I merged this with latest main so good to have a review whenever you get a chance.

Closing in favour of #288 which allows fixing to be run locally (and automated with Husky).


Has the disable hyperthreading script moved?

@velloduff, has the disable hyperthreading script moved?
When I try to run the WRF post-install script, I'm getting:
Running transaction
  Installing : perf-4.14.77-69.57.amzn1.x86_64                              1/3
  Installing : htop-1.0.1-2.3.amzn1.x86_64                                  2/3
  Installing : dstat-0.7.0-1.7.amzn1.noarch                                 3/3
  Verifying  : dstat-0.7.0-1.7.amzn1.noarch                                 1/3
  Verifying  : htop-1.0.1-2.3.amzn1.x86_64                                  2/3
  Verifying  : perf-4.14.77-69.57.amzn1.x86_64                              3/3

Installed:
  dstat.noarch 0:0.7.0-1.7.amzn1           htop.x86_64 0:1.0.1-2.3.amzn1
  perf.x86_64 0:4.14.77-69.57.amzn1

Complete!
--2018-11-14 18:28:03--  https://cfncluster-public-scripts.s3.amazonaws.com/disable_hyperthreading
Resolving cfncluster-public-scripts.s3.amazonaws.com (cfncluster-public-scripts.s3.amazonaws.com)... <IP_ADDRESS>
Connecting to cfncluster-public-scripts.s3.amazonaws.com (cfncluster-public-scripts.s3.amazonaws.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 403 Forbidden

We moved to ParallelCluster this week.  I will need to update these workshops, and I also need to find out what happens to the old buckets too.   I'll update when I have something.
You can use this for the disable_ht script:
function fix_boot_disable_ht() {

  echo "${0} Updating kernel line"

  if [[ -x /sbin/grubby ]] ; then
      /sbin/grubby --update-kernel=ALL --args=maxcpus=$total_cores
  fi

  if [ -e /etc/default/grub ]; then

      if grep -q maxcpus /etc/default/grub; then
          sed -i "s/maxcpus=[0-9]*/maxcpus=$total_cores/g" /etc/default/grub
      else
          sed -i "/^GRUB_CMDLINE_LINUX_DEFAULT=/ s/\"$/ maxcpus=$total_cores\"/" /etc/default/grub
          sed -i "/^GRUB_CMDLINE_LINUX=/ s/\"$/ maxcpus=$total_cores\"/" /etc/default/grub
      fi

      if [ -e /etc/grub2.cfg ]; then
          grub2-mkconfig > /etc/grub2.cfg
      fi

      if which update-grub; then
          update-grub
      fi
  fi

}

function disable_ht {
  echo "${0}: disabling HT"

  parent_cores=$(cat /sys/devices/system/cpu/cpu*/topology/thread_siblings_list | cut -d, -f1 | cut -d- -f1 | tr '-' '\n' | tr ',' '\n'| sort -un)

  # If there are no parents, HT is probably already disabled.
  if [ "$parent_cores" == "" ]; then
              parent_cores=$(cat /sys/devices/system/cpu/cpu*/topology/thread_siblings_list)
  fi

  total_cores=$(cat /sys/devices/system/cpu/cpu*/topology/thread_siblings_list | cut -d, -f1 | cut -d- -f1 | tr '-' '\n' | tr ',' '\n'| sort -un | wc -l)
  sibling_cores=$(cat /sys/devices/system/cpu/cpu*/topology/thread_siblings_list | cut -d, -f2- | cut -d- -f2- | tr '-' '\n' | tr ',' '\n'| sort -un)

  # Ensure enabled cores are enabled - starting at 1, cpu0 can't be changed
  for p in $parent_cores; do
      if [ $p -ne 0 ]; then
          echo 1 > /sys/devices/system/cpu/cpu${p}/online
      fi
  done

  if [ "$parent_cores" == "$sibling_cores" ]; then
      echo "Hyperthreading already disabled"
  else
      # Ensure disabled threads are actually disabled
      for s in $sibling_cores; do
          echo 0 > /sys/devices/system/cpu/cpu${s}/online
      done
  fi

  fix_boot_disable_ht

}

disable_ht

@veloduff thanks so much for the quick response and the script!
I have placed it here:
https://s3.amazonaws.com/esipfed/roms/disable_hyperthreading.sh
(and thanks for the heads up on https://github.com/aws/aws-parallelcluster, which I just had figured out after spinning up a cluster with cfncluster yesterday following the WRF workshop.  No worries though, I learned a lot)


Multiple Interface UDP Routing Problem

I've been using Fishnet to develop games/apps in Unity. We've been facing an issue for awhile. If there is multiple interfaces on the server system, mostly our servers have more then two running interfaces, server send UDP Packages on wrong route. So, client doesn't get packages.
I've check the code base and solve the problem. If we set the socket option DontRoute to true, it'll work. I found at NetManager.Socket.cs file and added this line.
Could you please investigate and let me know. If you want I can make PR.

Server Interfaces
WiFi: <IP_ADDRESS>
LAN : <IP_ADDRESS>
Client IP <IP_ADDRESS>
Server listens <IP_ADDRESS>. Client tries to connect <IP_ADDRESS>, Server send packages via <IP_ADDRESS>. I set DontRoute option, it solved the problem.
socket.SetSocketOption(SocketOptionLevel.Socket, SocketOptionName.DontRoute, true);

Library version: All version of Fishnet affected. I don't know exactly which version using in Fishnet.
Framework: Unity
OS: Linux Windows Macos

Fixed by #540


Support syntax highlight for pm- tabs

Example usage: https://github.com/unjs/undocs/blob/main/layers/core/app/components/global/Pm-x.vue
Relavant PR https://github.com/unjs/undocs/pull/41
/cc @farnabaz @benjamincanac if you have any ideas how to properly use UI+MDC to highlight with programmatic tabs that would be really appreciated if can help 🙏🏼

It seems for custom components they implemented kinda of the same way in nuxt/ui -> https://github.com/nuxt/ui/blob/dev/docs/composables/useShikiHighlighter.ts

That's not good :( I hope there was one standard shiki module that provided this functionality at least. /cc @antfu what do you think?

Ref: https://github.com/pi0/nuxt-shiki

Let's see if we can fix it once for all https://github.com/nuxt-modules/mdc/issues/148

I would also mention they use plugins and also use shiki re-hype etc. I don't if it's good to create ways to implement these too for them to fully support it. https://github.com/nuxt-modules/mdc/tree/main/src/runtime/highlighter

Also not mention another layer that antfu was working up could be releated but could be merged in with nuxt-shiki possibly -> https://github.com/antfu/nuxt-content-twoslash

Feel free to open issues in repo or make PRs to help on it :)

Just for my reference of the current tree:
@nuxt/mdc -> @nuxt/content -> @nuxt/ui.
@nuxt/mdc
Contains the main code for the parser and rehype.

nuxt-modules/mdc@main/src/runtime/highlighter

@nuxt/content
Nuxt content uses mdc to run the shiki code.

antfu/nuxt-content-twoslash

@nuxt/ui
Creates a useShikiHighlighter composable for there custom ui components to parse code blocks

<EMAIL_ADDRESS>

As I checked the repo, I'm suggesting not to run shiki separately. We can achieve with a simple remark plugin to convert pm-x into code-group at parse time. (Using latest mdc module)
// mdc.config.ts
import { defineConfig } from '@nuxtjs/mdc/config'
import { visit } from 'unist-util-visit'

export const packageManagers = [
  { name: 'npm', command: 'npm', install: 'i', run: 'run ', x: 'npx' },
  { name: 'yarn', command: 'yarn', install: 'add', run: '', x: 'yarn dlx' },
  { name: 'pnpm', command: 'pnpm', install: 'i', run: '', x: 'pnpx' },
  { name: 'bun', command: 'bun', install: 'i', run: 'run ', x: 'bunx' },
] as const

export default defineConfig({
  unified: {
    pre: (stream) => {
      stream.use(() => {
        return (tree) => visit(tree, (node) => node.name === 'pm-x', (node) => {
          node.name = 'code-group'
          node.children = packageManagers.map((pm) => {
            return {
              type: 'code',
              lang: 'bash',
              meta: `[${pm.command}]`,
              value: `${pm.x} ${node.attributes.command}`,
            }
          })
        })
      })
      return stream
    }
  }
})

This mean that pm-x component is only useful for typing and meta information and will be replaced by code-group
As for UI Docs, The use case is totally different, in UI Docs we needed to highlight dynamic code in runtime, based on user inputs. Shiki should run in client side. But here we need to highlight codes only at built time.

Thanks but i prefer a real simple way to a real simple problem not making it more complex this way!
Code group does not allows rendering single markdown component not even additional comment blocks inside it work (a limitation with UI pro) and MDC does not exposes shiki utils (that’s why made nuxt-shiki module to be able to reuse)

I don't suggest using ProseCode this way, but you can use it if you see remark plugin complex for you.
As for shiki utils, we can improve mdc module to simplify simplify usage.

(it seems mission impossible today to fix this in a proper way, i give up)


Increment version for  releases

Increment version for  releases

This won't be needed since #7312 will update the version in master to 2.0.0-preview.1


Noticed a problem

I was trying to run this program on my Python but it won't let me run until I intended a certain spot before the "board = [' ' for x in range(10)]" statement,  posting this if anybody else is having trouble running the file.
Example:
def main():
board = [' ' for x in range(10)]

def insertLetter(letter, pos):
board[pos] = letter
def spaceIsFree(pos):
return board[pos] == ' '

Sorry about the late reply, the program is running fine, I just checked, If
you created a new branch, just merge and pull. If your having trouble with
visual studio, run as administrator. We will be tearing the project apart
and creating classes to prep for creating a graphical interface as a
stretch goal.
On Wed, Apr 14, 2021 at 5:41 PM MurilloSeb @.***> wrote:

Closed #2
https://github.com/JamesTaylor-creator/Tic-Tac-Toe-COSC-1436-301/issues/2
.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/JamesTaylor-creator/Tic-Tac-Toe-COSC-1436-301/issues/2#event-4596155280,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACAKRF5GQK3VDWO6GFAODW3TIYKZDANCNFSM426I2VYA
.

There are also a few bugs we need to fix to complete the requirements
listed in our project.
On Fri, Apr 16, 2021 at 9:33 PM Thabiso J. Taylor @.***>
wrote:

Sorry about the late reply, the program is running fine, I just checked,
If you created a new branch, just merge and pull. If your having trouble
with visual studio, run as administrator. We will be tearing the project
apart and creating classes to prep for creating a graphical interface as a
stretch goal.
On Wed, Apr 14, 2021 at 5:41 PM MurilloSeb @.***>
wrote:

Closed #2
https://github.com/JamesTaylor-creator/Tic-Tac-Toe-COSC-1436-301/issues/2
.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/JamesTaylor-creator/Tic-Tac-Toe-COSC-1436-301/issues/2#event-4596155280,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACAKRF5GQK3VDWO6GFAODW3TIYKZDANCNFSM426I2VYA
.


Evict content indirectly referenced via symlinks from VFS

Before this change we evicted content from the VFS that was referenced directly via symlinks to avoid relying on cached data that might change without notification. However, content that is marked as directly accessed in the VFS might still be referenced via symlink if one of its parents is accessed via symlink. This PR makes sure that we also evict content referenced like this.
Fixes https://github.com/gradle/gradle/issues/26201
Reviewing cheatsheet
Before merging the PR, comments starting with

❌ ❓must be fixed
🤔 💅 should be fixed
💭 may be fixed
🎉 celebrate happy things

@bot-gradle test APT please.

I've triggered the following builds for you. Click here to see all build failures.

AllPerformanceTestsReadyForRelease build

@bot-gradle test APT please.

I've triggered the following builds for you. Click here to see all build failures.

AllPerformanceTestsReadyForRelease build

Looks like the simple approach leads to 5-10% regression in some important performance tests, since we are spending more time at the end of the build reading the file system:

Some things we could try:

parallelize the extra work
keep track of the state of the file system for all the parents we look up doing this would require watching those locations; something that would be a lot more expensive
move VFS eviction from "before build finished" to "between builds"

I think the we should attempt 3) next, as for real-world builds it would actually lead to slightly faster builds from the user's perspective.

@bot-gradle test QFL please.

I've triggered the following builds for you. Click here to see all build failures.

QuickFeedbackLinux build

@bot-gradle test QFL please.

I've triggered the following builds for you. Click here to see all build failures.

QuickFeedbackLinux build

@bot-gradle test this please.

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle test this please.

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle test this please.

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle test PT please.

I've triggered the following builds for you. Click here to see all build failures.

AllPerformanceTestsReadyForNightly build

@bot-gradle test QFL PT please.

I've triggered the following builds for you. Click here to see all build failures.

QuickFeedbackLinux build
AllPerformanceTestsReadyForNightly build

@bot-gradle run APT please.

Sorry I don't understand what you said.
Currently, the following commands are supported:
@bot-gradle test <BuildTrigger1> <BuildTrigger2> ... <BuildTriggerN> [without PTS]
Add without PTS to run the build with full test coverage.

A trigger is a special build for this PR on TeamCity, common triggers are:

SanityCheck/CompileAll/QuickFeedbackLinux/QuickFeedback/PullRequestFeedback/ReadyForNightly/ReadyForRelease
Shortcuts: SC/CA/QFL/QF/PRF/RFN/RFR
Specific builds:

BD: BuildDistributions/BuildDocs so that you can preview the generated docs/distribution.
PT: PerformanceTest, all performance tests for Ready For Nightly stage.
APT: AllPerformanceTest, all performance tests, including slow performance tests.
AST: AllSmokeTestsPullRequestFeedback
AFT: AllFunctionalTestsPullRequestFeedback
ASB: AllSpecificBuildsPullRequestFeedback
ACC: AllConfigCacheTestsPullRequestFeedback
ACT: AllCrossVersionTestsReadyForNightly
AFTN: AllFunctionalTestsReadyForNightly
ACTR: AllCrossVersionTestsReadyForRelease
AFTR: AllFunctionalTestsReadyForRelease

@bot-gradle squash

Squash the current pull request into a single commit. The squash message will come from the pull request body, with:

All Signed-off-by: lines kept.
All authors being Co-Authored-By:

@bot-gradle merge

Enqueue this PR into GitHub merge queue for merging.

GitHub will create a merge commit from your PR branch HEAD and the target branch
A GitHub Merge Queue Check Pass build will be triggered on the merge commit
When the build passes, the target branch will be fast-forwarded to this merge commit (i.e. merge the PR)

@bot-gradle squash and merge

Squash the current pull request into a single commit as described in squash command above,
then enqueue this PR into GitHub merge queue as described in merge command above.

@bot-gradle cherrypick to <branch>

Cherrypicks the current PR to another branch.
A new PR will be created and a build will triggered automatically if there is no conflict.

@bot-gradle cancel

cancel a running build in GitHub merge queue and remove the PR from the queue

@bot-gradle clean

clear the conversation history

@bot-gradle help

display this message

To run a command, simply submit a comment.  For detailed instructions see here.

@bot-gradle test APT please.

I've triggered the following builds for you. Click here to see all build failures.

AllPerformanceTestsReadyForRelease build

@bot-gradle test APT please.

I've triggered the following builds for you. Click here to see all build failures.

AllPerformanceTestsReadyForRelease build

@bot-gradle help

Currently, the following commands are supported:
@bot-gradle test <BuildTrigger1> <BuildTrigger2> ... <BuildTriggerN> [without PTS]
Add without PTS to run the build with full test coverage.

A trigger is a special build for this PR on TeamCity, common triggers are:

SanityCheck/CompileAll/QuickFeedbackLinux/QuickFeedback/PullRequestFeedback/ReadyForNightly/ReadyForRelease
Shortcuts: SC/CA/QFL/QF/PRF/RFN/RFR
Specific builds:

BD: BuildDistributions/BuildDocs so that you can preview the generated docs/distribution.
PT: PerformanceTest, all performance tests for Ready For Nightly stage.
APT: AllPerformanceTest, all performance tests, including slow performance tests.
AST: AllSmokeTestsPullRequestFeedback
AFT: AllFunctionalTestsPullRequestFeedback
ASB: AllSpecificBuildsPullRequestFeedback
ACC: AllConfigCacheTestsPullRequestFeedback
ACT: AllCrossVersionTestsReadyForNightly
AFTN: AllFunctionalTestsReadyForNightly
ACTR: AllCrossVersionTestsReadyForRelease
AFTR: AllFunctionalTestsReadyForRelease

@bot-gradle squash

Squash the current pull request into a single commit. The squash message will come from the pull request body, with:

All Signed-off-by: lines kept.
All authors being Co-Authored-By:

@bot-gradle merge

Enqueue this PR into GitHub merge queue for merging.

GitHub will create a merge commit from your PR branch HEAD and the target branch
A GitHub Merge Queue Check Pass build will be triggered on the merge commit
When the build passes, the target branch will be fast-forwarded to this merge commit (i.e. merge the PR)

@bot-gradle squash and merge

Squash the current pull request into a single commit as described in squash command above,
then enqueue this PR into GitHub merge queue as described in merge command above.

@bot-gradle cherrypick to <branch>

Cherrypicks the current PR to another branch.
A new PR will be created and a build will triggered automatically if there is no conflict.

@bot-gradle cancel

cancel a running build in GitHub merge queue and remove the PR from the queue

@bot-gradle clean

clear the conversation history

@bot-gradle help

display this message

To run a command, simply submit a comment.  For detailed instructions see here.

@bot-gradle test PT please.

I've triggered the following builds for you. Click here to see all build failures.

AllPerformanceTestsReadyForNightly build

@bot-gradle merge this please.

The merge queue build has started. Click here to see all failures if any.

The merge queue build has started. Click here to see all failures if any.

The merge queue build has failed. Click here to see all failures.

@bot-gradle merge this please.

The merge queue build has started. Click here to see all failures if any.

@bot-gradle cancel merge.

Sorry some internal error occurs, please contact the administrator @blindpirate

@bot-gradle cancel

@bot-gradle merge please.

The merge queue build has started. Click here to see all failures if any.


Final OSX cmake fixes

Please reconsider the CMake GLOB for source files.

I can do it for you by new pull request. Ok?


Fixed Odometry Publisher Angular Velocity Singularity in 3D

🦟 Bug fix
Fixes #2347
Summary
The angular velocity was calculated using euler angles which led to singularities (see #2347 for a more detailed description). In addition to the fix, I also removed the usage of the rolling mean accumulator for angular velocity. While I see its usefulness for outlier rejections, I still didn't really see why this would be necessary. If the maintainers want to keep this mean accumulator, I'll be happy to reimplement it. Also the commit changes seem a bit messy, I was using clang format with the Google C++ Style guide as configuration, which leaded to the changed formatting that you can see in the pull request.
Checklist

[x] Signed all commits for DCO
[x] codecheck passed (See contributing)
[x] All tests passed (See test coverage)

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

Hi @mjcarroll,
is there anything more that needs to be changed to merge this pull request ?

Hello @mjcarroll,
are there any news or requests from your side regarding this pull request ? Are there any plans to address this issue ?

I've merged https://github.com/tomcreutz/gz-sim/pull/1

Please consider adding a test case in https://github.com/gazebosim/gz-sim/blob/gz-sim8/test/integration/odometry_publisher.cc to prevent a regression in computing 3d angular velocity close to singularity. Thank you!

Added test case in 5b877ca

@osrf-jenkins run tests please

Looks like Windows CI is failing as M_PI cannot be found. I guess we should GZ_PI instead in test/integration/odometry_publisher.cc

Thank you very much for your efforts @azeey and @shameekganguly . Just returned back from holidays and wanted to take care of this in the following days.

Not sure why CI on Jammy failed. The test seems completely unrelated to this PR, so I'm rerunning it to see if it's a flake. The previous CI job was green https://build.osrfoundation.org/job/gz_sim-ci-pr_any-jammy-amd64/744/.


Flatlist nesting is not working properly Scroll

Info
React Native Environment Info:
System:
OS: Linux 4.15 Ubuntu 18.04.3 LTS (Bionic Beaver)
CPU: (8) x64 Intel(R) Core(TM) i7-4790 CPU @ 3.60GHz
Memory: 474.79 MB / 15.55 GB
Shell: 4.4.20 - /bin/bash
Binaries:
Node: 8.10.0 - /usr/bin/node
Yarn: 1.16.0 - /usr/local/bin/yarn
npm: 3.5.2 - /usr/bin/npm
npmPackages:
react: 16.8.3 => 16.8.3
react-native: 0.59.9 => 0.59.9
npmGlobalPackages:
react-native-cli: 2.0.1
Example Code
              <FlatList
                        data={this.state.grouplist}
                        style={styles.leftflatlist}
                        // contentContainerStyle={{height: 200}}
                            numColumns={4}
                            extraData={this.state}
                            horizontal={false}
                            renderItem={({item, index}) => {
                                let list = item;
                                let _index = index;
                                return (
                                        <FlatList
                                             listKey={_index}
                                            data={list}
                                            extraData={this.state}
					   horizontal={false}
					style={styles.scroll}
					numColumns={4}
					renderItem={({item, index}) => {
					return (
                                                    <TouchableHighlight
                                                    underlayColor="rgba(0,0,0,0)"
                                                    key={item.index}
                                                    onPress={this.editorGroup.bind(this, _index, index, item)}>
                                                        <Avatar
                                                        shake={this.state.rotateList[_index]}
                                                        icon={{uri: item.face_image}}
                                                        name={item.name}>
                                                    </Avatar>
                                                    </TouchableHighlight>
												)
	                      </FlatList>
            </FaltList>

Inner layer cannot be scrolled

I might be wrong but seems you don't actually need 2 FlatLists to produce the results you're looking for. Perhaps a SectionList would better fit your needs as it can separate its content in groups while using only 1 scrollable area. Nested FlatLists will mostly likely not work as you are expecting as one might end stealing the scrollEvent from the other. A good example for nested FlatLists is when their direction diverges (e.g the main FlatList using vertical scrolling while its children using horizontal scrolling).


[MINOR] Fix PostgresIntegrationSuite

What changes were proposed in this pull request?
The pr aims to  fix PostgresIntegrationSuite.
Why are the changes needed?
At present, the GA of the master branch has break. Let's restore it first.
https://github.com/yaooqinn/spark/actions/runs/8518017152/job/23329558427

Does this PR introduce any user-facing change?
No.
How was this patch tested?
Pass GA.
Was this patch authored or co-authored using generative AI tooling?
No.

cc @yaooqinn @HyukjinKwon @dongjoon-hyun @LuciferYang

Thanks @yaooqinn !

The diff here looks weird. So, I reverted SPARK-47684 and added your credits here https://github.com/apache/spark/pull/45810/commits

It's really weird,
After revert the PR https://github.com/apache/spark/pull/45810/files, the GA of the master has been successfully restored.


[Terraform] Fix config spacing

Per the Terraform Style Conventions (https://www.terraform.io/docs/language/syntax/style.html) indentation should be 2 spaces. The rest of the file follows this except this one section

Thanks @WalshyDev, going ahead and merging in.


[Bug]: KeyError: 'results' when calling moderation API

Describe the bug
When I run the Python Teams Chef Bot sample, I meet following error:
  File "C:\Users\xxx\git\teams-ai\python\packages\ai\teams\app.py", line 359, in on_turn
    await self._start_long_running_call(context, self._on_turn)
  File "C:\Users\xxx\git\teams-ai\python\packages\ai\teams\app.py", line 459, in _start_long_running_call
    return await func(context)
  File "C:\Users\xxx\git\teams-ai\python\packages\ai\teams\app.py", line 399, in _on_turn
    is_ok = await self._ai.chain(context, cast(StateT, state), self._options.ai.prompt)
  File "C:\Users\xxx\git\teams-ai\python\packages\ai\teams\ai\ai.py", line 186, in chain
    plan = await self._options.planner.generate_plan(
  File "C:\Users\xxx\git\teams-ai\python\packages\ai\teams\ai\models\openai\planner.py", line 99, in generate_plan
    plan = await self.review_prompt(turn_context, state, prompt_template)
  File "C:\Users\xxx\git\teams-ai\python\packages\ai\teams\ai\models\openai\planner.py", line 158, in review_prompt
    res = await self._client.create_moderation(
  File "C:\Users\xxx\git\teams-ai\python\packages\ai\teams\ai\models\openai\client\openai_client.py", line 67, in create_moderation
    data=CreateModerationResponse.from_dict(data),
  File "C:\Users\xxx\git\teams-ai\python\packages\ai\teams\ai\models\openai\client\schemas\moderation.py", line 32, in from_dict
    results: List[Any] = data["results"]
KeyError: 'results'

To Reproduce

Steps to reproduce the behavior:

Open python\samples\04.ai.a.teamsChefBot in this repo
Fill Azure Open AI key, endpoint, model deployment name to .env.local and .env.local.user in env folder
Press F5 to run the app
Send anything to the bot

Expected behavior
The bot can response the messages from user
Desktop(s):

OS: Windows
Browser: Edge
Version: 117.0.2045.41

for now we have disabled content moderation for azure open ai for the following reasons:

azure doesn't have moderations api, it has a content safety api and also performs content safety checks automatically on all prompts submitted to the model
due to the above, calling content safety would be duplicate work and not needed by default.


refactor IR for periodic columns

[x] Add the cycle length of the periodic column to the IdentifierType::PeriodicColumn enum variant
[x] update the insertion of periodic columns into the graph to include the cycle length
[x] change the computation of degrees so that nothing needs to be passed to the method

Closed by #49


Images are no longer copied to dest.

Version

<EMAIL_ADDRESS>-<EMAIL_ADDRESS>
Platform
Windows
What steps will reproduce the bug?
Put some images in a content folder next to the *.md File
change version of lume from<EMAIL_ADDRESS>to<EMAIL_ADDRESS>or<EMAIL_ADDRESS>What do you see instead?
Starting with Lume version 2.2.1 there are only the generated html files in dest, but no images anymore.
Is this a bug or intentional behavior? Can not find anything in the CHANGELOG for this.

How are you copying the files? Can I see your _config.ts file or a repository reproducing this bug?

add your _config.ts
site.copy("yourimagedir", "yourimagedir");

Check the path that the HTML is currently outputting after building, and change it to your image directory.

Hey @oscarotero this is my _config.ts file
import lume from "lume/mod.ts";
import base_path from "lume/plugins/base_path.ts";
import sass from "lume/plugins/sass.ts";
import picture from "lume/plugins/picture.ts";
import transformImages from "lume/plugins/transform_images.ts";

import esbuild from "lume/plugins/esbuild.ts";
import postcss from "lume/plugins/postcss.ts";
import { esbuildExternalFilter } from "./plugins/config.plugins.ts";

const theme = "clrCSS";

const site = lume({
    location: new URL('...'),
    src: "./src",
    dest: "./dist",
    components: {
        variable: "comp",
        cssFile: "/components.css",
        jsFile: "/components.js",
    }
});

//------------- Clarity styles - https://clarity.design/pages/developing ------------
const cssFiles = [
    { src<EMAIL_ADDRESS>name: "modern-normalize.css" },
    { src: 'https://unpkg.com/<EMAIL_ADDRESS>name: 'global.min.css' },
    { src: 'https://unpkg.com/<EMAIL_ADDRESS>name: 'theme.dark.min.css' },
    { src: 'https://unpkg.com/<EMAIL_ADDRESS>name: 'clr-ui.min.css' }
];

cssFiles.forEach(item => {
    site.remoteFile(item.name, item.src);
});
//----------------------------------

site.ignore("README.md", "CHANGELOG.md", "node_modules");

site.use(base_path());
site.use(sass());

site.use(postcss());
site.use(picture());
site.use(transformImages());

//  Some JS and CSS files for an external library.
site.copy("_static, ".");

site.use(esbuild({
    extensions: [".ts"],
    options: {
        bundle: true,
        format: "esm",
        minify: true,
        keepNames: true,
        ignoreAnnotations: true,
        platform: "browser",
        target: "esnext",
        treeShaking: true,
        external: ['https://esm.sh/v135/<EMAIL_ADDRESS>        entryPoints: ["./src/clr-cds.ts"],
    }
}));

site.hooks.addEsbuildPlugin(esbuildExternalFilter);

export default site;

{
  "tasks": {
    "lume": "echo \"import 'lume/cli.ts'\" | deno run -A -",
    "build": "deno task lume",
    "serve": "deno task lume -s",
  },
  "compilerOptions": {
    "types": [
      "lume/types.ts"
    ]
  },
  "imports": {
    "lume/"<EMAIL_ADDRESS>    "denoStd/"<EMAIL_ADDRESS>  }
}

@oscarotero thanks, I will try to check where the problem was and report it. With your project it works as expected.

@oscarotero I think I've figured out what the problem is. It was kind of a self-made issue because there was a _data.js file (no typings), and we used some properties of the page data, which changed.
There was a exported url function in a _data.js file to filter images for data-pagefind-meta.
The return was based on page.src.asset to return images or html files
if (page.src.asset) {
      return `${url}${page.src.ext}`
 } else
      return `${url.replace(/\/index$/, '/index.html')
}`;

I found this change https://github.com/lumeland/lume/blob/ca04a1612db69fa5302ce6fe19148b3ae699eed8/core/utils/page_url.ts#L61 to https://github.com/lumeland/lume/blob/66dc59af843ab3ae0ce3bb4cc662299a38798ea8/core/utils/page_url.ts#L61
https://github.com/lumeland/lume/commit/396aee98b4dfa78d4c51897d9f33268f719d6251
I tried switching _data.js to _data.ts and using types. Now I got the error from the typings saying that this property doesn't exist. So, everything seems to be working as expected.

Great!


Link to the QR code for testing Draft form

Similar to the App User's QR code issue is it possible to get the link to Draft Form testing QR Code image by API call, or include this QR Code into the response when posting a draft form using Creating a Draft Form.
Thank you for your great work!

I thought that

When a Draft Form is created, a Draft Token is also created for it, which can be found in Draft Form responses at draftToken. (link)

could be the solution but I don't see draftToken in the response when I post form in R:
Response I get:
Response [https://[server]/v1/projects/[pid]/forms]
  Date: 2020-06-07 14:36
  Status: 200
  Content-Type: application/json; charset=utf-8
  Size: 418 B

Hi @OdiljonQurbon!

I don't see draftToken in the response when I post form in R:

I think the Form endpoints don't return a Draft Token, but the Draft Form endpoint does: if you request /v1/projects/[projectId]/forms/[xmlFormId]/draft, the response should contain a Draft Token.

Hello @matthew-white !
My first call from R is following:
  pxlsxfile <- PATH_TO_XLSX_FILE
  un <- MYUSERNAME
  pw <- SUPERSECRETPASS

  server <- MYSERVER_URL
  pid <- 1

  url <- glue("{server}/v1/projects/{pid}/forms/?ignoreWarnings=false")

  ## POST form in xlsx format to Central
  myxlsxfile <- httr::upload_file(xlsxfile, type = "application/vnd.openxmlformats-officedocument.spreadsheetml.sheet")
  httr::POST(url,
            httr::add_headers(
              "X-XlsForm-FormId-Fallback" = 'simple'
            ),
            httr::authenticate(un, pw),
            body = myxlsxfile
  )

got the response:
Response [https://[server]/v1/projects/1/forms?ignoreWarnings=false]
  Date: 2020-06-07 20:39
  Status: 200
  Content-Type: application/json; charset=utf-8
  Size: 422 B

Then I tested below code to /draft endpoint as you suggested:
  pxlsxfile <- PATH_TO_XLSX_FILE
  un <- MYUSERNAME
  pw <- SUPERSECRETPASS

  server <- MYSERVER_URL
  pid <- 1

  url <- glue("{server}/v1/projects/{pid}/forms/simple/draft?ignoreWarnings=false")

  ## POST form in xlsx format to Central
  myxlsxfile <- httr::upload_file(xlsxfile, type = "application/vnd.openxmlformats-officedocument.spreadsheetml.sheet")
  httr::POST(url,
            httr::add_headers(
              "X-XlsForm-FormId-Fallback" = 'simple'
            ),
            httr::authenticate(un, pw),
            body = myxlsxfile
  )

Response [https://[server]/v1/projects/1/forms/simple/draft?ignoreWarnings=false]
  Date: 2020-06-07 20:41
  Status: 200
  Content-Type: application/json; charset=utf-8
  Size: 16 B

As documentation  indicates

As of Version 0.8, Forms will by default be created in Draft state, ...

So I cannot see any difference in the API call response to above two calls. Maybe I'm doing something wrong but don't know what exactly should I fix.

once the form is created you will want to GET the /draft details, not post to it. then you should find the draftToken.

Thanks @issa-tseng !

once the form is created you will want to GET the /draft details, ...

GET request brings draftToken. But I think using this draftToken with ODK Collect is not ready yet, am I right? I still need to use QR code to be able to send submissions.

Thanks @issa-tseng ! I'm getting draftToken using GET call to the server.

Having draftToken makes possible to generate QR Code. Thank you.


[Docs] Problems  on Waymo 3d Box

Branch
main branch  https://mmdetection3d.readthedocs.io/en/latest/
📚 The doc issue
Hello, I am confused on the waymo_infos_val.pkl information (bbox 3d). Why is the z-coordinate -24, did it go underground?

Suggest a potential alternative/fix
No response

waymo coordinate system is same as kitti dataset which is camera coordinate system, plz refer to this document https://mmdetection3d.readthedocs.io/en/latest/user_guides/coord_sys_tutorial.html. So if that is the case, i believe Z points to downwards which is in -ve. They suggests that it needs to be converted to Lidar coordinate system.


Roll engine fa9b5bd0e286..7dd62d6b3ca4 (3 commits)

https://github.com/flutter/engine/compare/fa9b5bd0e286...7dd62d6b3ca4
git log fa9b5bd0e286e91bb685b22adf346e7e19b944d2..7dd62d6b3ca4c7d00e4382491b5a692a5a4554b1 --no-merges --oneline
7dd62d6b3 Roll src/third_party/skia 1013ecfb3421..859f7108a5af (19 commits) (flutter/engine#9136)
e8aa12048 New Plugin API PR5: Integrates plugin lifecycle control with FlutterFragment. (flutter/engine#9083)
8b1199c4d Implemented Log proxy that only logs in BuildConfig.DEBUG (#25391). (flutter/engine#9122)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (cbracken@google.com), and stop
the roller if necessary.

Auto-roller completed checks. About to merge.


Regenerate with latest GIR

See https://github.com/gtk-rs/gir/issues/508
and https://github.com/gtk-rs/gir/issues/509

:+1:

CI is happy

Thanks!


Metabolomics workflow distribution outside PNNL

Metabolomics workflow is working at PNNL but need to figure out DB infrastructure that would allow it to be distributed outside of PNNL.

@ssarrafan, @kfagnan closing the issue based on today discussion during Tech Sync Meeting, we are working on the long term solution for the issue, and we agreed it should not be the priority for May sprint


How to use bread translate

Laravel Version: 5.4.*
Voyager Version: LAST
PHP Version: 7
Database Driver & Version: last too

Description:
I would like to know how im suppose to use bread to translate admin area, thanks

Hi @sparkosis,
Admin translations are not supported at the moment, we are working on that. See #760.

Hey,
Thank u for ur reply, then im waiting.
When does it coming ? approximately.
i can help to do french translation if needed

Hello, hopefully this week I'll work on that and will let you know.
Thanks for the help.

Good news then. I have to déploy voyager for my company
Télécharger Outlook pour Android
On Tue, Jun 6, 2017 at 8:08 PM +0200, "Bruno Torrinha"<EMAIL_ADDRESS>wrote:
Hello, hopefully this week I'll work on that and will let you know.
Thanks for the help.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


I can't save the solver options anymore.

Not sure which information you need for this to replicate. My setup is attached.
The save Button is disabled
settings (4).zip

Something else to check is to press f12 to open the browser developer tools window and check if there are any errors in the console tab.

No errors, in the developer tools console tab while opening the tab, trying to save. the button is just simply unclickable and looks faded.

Also, I've sent you an email on<EMAIL_ADDRESS>Let me know.

I can second the issue, both using imported settings from 0bc4ee6 as well as using clean settings.  It seems to be disabled because the Max Length setting has a default value of 0 - setting a non-zero value allows the Save button to function normally from then on.  The breaking change seems to be in 461b228: in app/js/controllers/main.js maxLength is initialized to 0, but in app/modals/options.html it has a minimum value of 1.
Simply changing the default value of maxLength to a positive number fixes it for new instances (eg. using defaults), but with pre-existing settings users still have to set the value manually.

Ah, damn, and there I thought I got the defaults right for existing users. Sorry about that, I'll fix it later today.

Deployed to beta site. It should automatically fix the saved settings when you reload.


Add staging & testing rootDomain support for all validating functions in at_onboarding_flutter package

Is your feature request related to a problem? Please describe.
I'm trying to create a custom login screen. And trying to authenticate with
OnboardingService.getInstance().authenticate('@bob🛠', jsonData: atKeysFileData);

I'm running om the local server(vip.ve.atsign.zone). The authentication was failing with ResponseStatus.serverNotReached response cause _checkAtSignServerStatus('@bob🛠') returns ServerStatus.unavailable. This is because, The implementation of the function has AppConstants.rootDomain which is root.atsign.org. This won't let me check my local server atsigns.
Describe the solution you'd like
In the RootEnvironment extension for domain, assign values before returning them. And in case, If users want to have a custom login screen, they don't use any RootEnvironment. In such a case, the functions that use the rootDomain must be passed as an optional parameter.
NOTE:
If this wasn't modified as mentioned, then testing on local servers won't be possible for custom login screens.

@yahu1031
For now, you can change the rootDomain in AppConstants from at_onboarding_flutter.
AppConstants.rootDomain = "vip.ve.atsign.zone"

Because we will have a new at_onboarding package (which refactors the UI and flow). So  I think no need update the current at_onboarding_flutter package for this issue.
Please let me know your thoughts. Thank you!

Yes, cool. So this can go under enhancement for that new code then.

Moving this to "upstream dependency" till the new refactored at_onboarding_flutter package is released.

@yahu1031
Please check the latest version of at_onboarding in trunk branch
We can pass rootDomain into AtOnboardingConfig when call the function AtOnboarding.onboard...

@sonle-geekyants Lemme check and update this

Closing as completed


Bug?!

So, i'm trying to play without names
Bug :

2nd Bug :

The black dot is the (cell) Me
@Megabyte918 is there anyfix?
I cant spawn without a name ;(

@ExobyteZ918 Did you use an unedited version of this "bug"?
Because otherwise if you did something to one of the files that could've made the bug. Try using an unedited version.

Provide steps to reproduce.

@AlexHGaming I'm using the latest version

Same in Minions too ;( @AlexHGaming @MastaCoder @Megabyte918

Try using an UNEDITED VERSION, as in you not making ANY changes to ANY file (serverport excluded)
Does this occur on another client?

@AlexHGaming Unedited same in other cilent

@Exobyte918 I will look into it.

@Exobyte918 I downloaded a fresh new server. It seems to work completely fine with 200 minions on.
Please verify you have no skin scripts enabled.


AIMP XspfBuilder added, some files formatted

I added AIMP .xspf file builder, bcs I need it ;)
You can add "Create a VLC/AIMP playlist with the audio files" to Readme.md.
It would also not be superfluous to get rid of the need to download ffmpeg and yt-dlp in the release, it is better to give the opportunity to download it separately from the releases page, and first search for the already installed in PATH.

Looks good to me.
I'm going to add a config file of some sort to make the user have the ability to customize what type of playlist file they want.


Fix: Eliminate possible deadlock related to memory allocation on global heap

Addresses https://github.com/microsoft/Detours/issues/70 by utilizing the HeapLock/HeapUnlock APIs as discussed in the issue.
Microsoft Reviewers: Open in CodeFlow

This doesn't solve the issue. Calling HeapLock while a suspended thread has acquired a heap lock by itself will deadlock. The actual resolution would be to create our own heap (using HeapCreate) and allocate everything in that, as this isn't subject to concurrency issue with any suspended thread.

Ah nevermind, I didn't notice this was in DetourTransactionBegin, before any thread could be suspended. This is good then.

Thanks for the feedback. I've updated the PR.

HeapLock/HeapUnlock is not a perfect solution, check out the related debate in this PR.

So, as I understand, the main problem with the HeapLock approach is that there is a race condition: after we have locked the heap in DetourTransactionBegin, there could be another thread which may unlock it before we have a chance to suspend that thread. This seems to be a bummer indeed...
It looks like then there's no other foolproof solution than using HeapCreate and custom allocation. So I'm closing this in favor of your PR. (I suggest though you clean it up a bit by squashing commits and split it into two parts: one that addresses the deadlock issue and one that introduces DetourUpdateAllOtherThreads. So it may have a better chance to get merged.)

@adams85
Thank you for your actions and suggestions, because the title of my PR is "Fix if the hook is not called at the start of the process, the hook may fail", and deadlock is part of the issue covered in this issue and DetourUpdateAllOtherThreads is the solution part of the way, so I don't think they are easily separable.


adds save and load single control

This change resolves #13132 by adding the option to save and load single control in gui editor

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/13812/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/13812/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/13812/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/13812/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/13812/merge
https://gui.babylonjs.com/?snapshot=refs/pull/13812/merge
https://nme.babylonjs.com/?snapshot=refs/pull/13812/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/13812/merge#BCU1XR#0

@RaananW formatter (npm run format:fix) has been run in the latest commit
commit: 51c29288db3a1970306d0f74f2378cd1c628a772


Add micro elements to the cards

There is no micro elements to the cards so i will add some interactive elements to it while hovering so please do assign it

There is no micro elements to the cards so i will add some interactive elements to it while hovering so please do assign it

show a preview of what you are trying to achieve

The cards present are so simple so i will add some micro elements like
hovering changing colors to make it neat amd creative
On Wed, Oct 23, 2024 at 4:55 PM KorryKatti @.***> wrote:

There is no micro elements to the cards so i will add some interactive
elements to it while hovering so please do assign it
show a preview of what you are trying to achieve
—
Reply to this email directly, view it on GitHub
https://github.com/KorryKatti/Mirage-web/issues/53#issuecomment-2431797884,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5MWOJ2VTP2JZZIT4LQY5PDZ46BTDAVCNFSM6AAAAABQOSDEMGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMZRG44TOOBYGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


Create Player/AI Data Class For Battles

This is the class that will be interfaced with by the networking library.
This class should include:

A single owner (Player/AI)
Relevant events
Troops (Equipment/Positions/Formation/Alive)

The goal for this class is to have all the necessary information centralized to synchronize battles.

Are we able to use the existing multiplayer framework?

@SeanMichaelRyan you could. I looked into it a bit, we would have to customize quite a few things. Not sure if doing our own optimization would be better.

Hi @garrettluskey , @SeanMichaelRyan,
This is probably a bit stupid but I'm not actually sure where to begin here. Is there a similar example of a class where we already do this to work from/look at?
Maybe I'm overcomplicating what needs to be done for this?
Thanks
Jack

@DrMorphDev there really isn't a good example for this one. The idea here is to create a data class. That data class will be used to synchronize battles. Let me know if you still have questions.


Mention the limitation that you can't convert evaluation datacenter to retail standard edition without reinstalling

It might be worthwhile to mention here that you can't convert datacenter editions to standard editions without reinstalling.  That limitation is documented elsewhere, but it came up as a question for a customer that wanted to flip their Server 2019 Datacenter Eval to Server 2019 Standard Retail.

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

ID: b1b19ef8-a273-c1c8-18ff-7d29664bb9d2
Version Independent ID: 8cf37a43-cff4-78de-0d66-2f482c022dc7
Content: Upgrade and Conversion Options for Windows Server 2016
Content Source: WindowsServerDocs/get-started/Supported-Upgrade-Paths.md
Product: windows-server
Technology: getting-started
GitHub Login: @jaimeo
Microsoft Alias: jaimeo

Author metadata is out of date. @JasonGerend

@JasonGerend

It has been quite a while since I have seen anything posted by @JasonGerend, I wonder if he is too occupied elsewhere, e.g. with the undocumented conversion to Azure.

I'm looking for an opportunity to create some impact; if you'd like I can submit a pull request to handle this.  It'll be my first time though; do I need to do anything special for i18n?

Everyone is welcome to contribute, be it via issue tickets or pull requests. Unless you want to use your own local flavor of Git tools on your own computer, you should be good to go with the web based tools to edit the page in question and open a new pull request with your suggested changes.
Feel free to ask for help too, we are here to help if we can.

Hi @ElizabethGreene,
Thanks for the feedback and your dedication to our documentation. I have reviewed you comment and believe this has already been address by the article updates on the 25th November 2021. The last bullet in the Upgrading licensed versions of Windows Server section now reflects your comment.
With regard to creating some impact, this is great to hear. We have a Microsoft Docs contributor guide that you can review.
To follow up on this issue, leave us a comment with an @mention.
Thanks,
Robin
#please-close


Couldn't complete step 8.

Step 8. "Navigate to the Schema page again and on the right hand side find listPictures(...):[PictureConnection] under the Query type and click the resolver to edit it. Update the REQUEST mapping template with the code in ./AppSync/Query.listPictures.request. Use "Return single item" as the response template. Press Save at the bottom.
This will use fine grained authorization controls to filter GraphQL responses so that users only see their data from DynamoDB."
Should I click attach?

Seem like selecting Data source name step is missing in description.

Hi @MikeVaz, thanks for your feedback. You are correct you should click the Attach button under Resolver column. Also you need to select PictureTable as Data source. I will modify it to make it more clear.
Thanks!

Readme updated. Thanks @MikeVaz


"Transaction has expired" Error on TestNet when creating accounts

Describe the bug
As reported by several developers, they often met "Transaction has expired" error when creating accounts on TestNet.
To Reproduce
The issue happens randomly, and there's no deterministic way of reproducing the issue.
Below are how developers met the issue when :

create new testnet accounts with wallet and error pops up;
create accounts with NEAR CLI.

Expected behavior
Creating accounts should be stable on testnet
Screenshots
Create account with NEAR CLI, and met the error below by developers:

Version (please complete the following information):

testnet

Additional context
This may also be related to the recent change to testnet account creation https://github.com/near/near-contract-helper/pull/447

@mikedotexe @vgrichina @volovyk-s What do you think?

@frol we have this error in our schema: https://github.com/near/near-api-js/blob/6e36ba2c3eb896cd72b5da8690d703cc5b70f693/lib/generated/rpc_error_schema.json#L610
So probably the origin of this error is not near-api-js.
I'm not sure if I fully understand why this error is thrown, like.. what is expired?

@volovyk-s it is a similar error to InvalidNonce. Let me show the SignedTransaction structure that gets submitted to the blockchain:
SignedTransaction {
    transaction: Transaction {
        actions: [
            Transfer(
                TransferAction {
                    deposit:<PHONE_NUMBER>000000000000000,
                },
            ),
        ],
        receiver_id: "qq.frol4.testnet",
        signer_id: "frol4.testnet",
        // public key must be registered as a signer's FullAccess key, and its corresponding private key is used to sign the hash of this transaction to produce the `signature` (see below)
        public_key: ed25519:3oN25HE9tf9YUGXQyXCFB2rEtpomijG8ghVV2JvrA4zJ,
        // nonce should monotonically increase with each transaction submitted for the given access key
        nonce: 19,
        // this is a hash of some recent block to ensure that the transaction is constructed for the specific chain thus avoiding replay attack when you have the same account ids on testnet and mainnet and someone just re-sends your transaction from testnet to mainnet
        block_hash: `27hfTxUF3wCgV5U16uvhrEQRZmbDHQkVh25Hb8tf1e7i`,
    },
    hash: `2DLtZEMeG1wp3wyxgXpFmrhfe1Xm7YN1VP8LNoNhC3FG`,
    size: 131,
    signature: ed25519:byRKymkv5KThHc9Mv1zjtyKf8N3CRigp5NRCm8hqqSWxZMfHEPGkCQV6fvCTR2xX98C8g8iv7Hic7Vc6WF6uX9H,
}

You get InvalidNonce when the nonce is less than the latest observed value on the blockchain (e.g. you send two transactions concurrently with the same nonce, only one will be applied and another will receive InvalidNonce)
You get InvalidChain when the block_hash is not part of the current chain.
You get Expired when the block_hash is of the block that has a height that is more than validity_period blocks away from the tip of the chain (on mainnet and testnet the transaction_validity_period is set to 86400)

Updates from discussion with @frol:

Expired error can be thrown from the outdated node (1 day +) @bowenwang1996 is this situation possible? Should we restrict validation from outdated nodes?
we can handle it the same way we are handling InvalidNonce, and that should be the fix for this issue.

Expired error can be thrown from the outdated node (1 day +) @bowenwang1996 is this situation possible? Should we restrict validation from outdated nodes?

Yes if near-api-js pulls the nonce from a node that is very out of sync. Maybe we should check that the node is healthy first (query the health endpoint).

@bowenwang1996 you cannot guarantee that after you do the health check your next RPC call will land on the same node, thus, we should just retry here (we could figure out the latest block from the status endpoint and checking that the node we reached out returns syncing: false, so we don't force user to re-sign the transactions with ledger unnecessary number of times)


addition of openSUSE

Proposed changes
I've provided documentation on how to install docker on openSUSE with some additional info for the index for docker on openSUSE arm

Hi @thaJeztah would you mind merging this to add openSUSE? The checks passed. However, I don't see the opensuse.md in the preview option. Not sure why it's just showing the index.md preview

Thanks for contributing!
Unfortunately, I don't think we can merge this PR. Docker currently does not provide packages for openSUSE. While there are packages of docker in openSUSE's package repositories, those packages are maintained by the SUSE distro maintainers and may be built with non-standard configuration or from different source code as the packages we provide, and for which we cannot provide support.
Adding instructions for installing the distro packages in our documentation would raise the incorrect expectation that Docker provides support for those, which would not be the case.
That said, we are in the process of reviewing our build-matrix; we did have packages for openSUSE and SUSE as part of Docker Enterprise, and some of the distros supported by Docker Enterprise may find their way back in our list of packages.
While those changes are not yet in place, we prefer users to follow the instructions under Other Linux distributions

Thanks for contributing!
Thank you @thaJeztah
That said, we are in the process of reviewing our build-matrix; we did have packages for openSUSE and SUSE as part of Docker Enterprise, and some of the distros supported by Docker Enterprise may find their way back in our list of packages.

Thanks for the info. I notice that Fedora is there with a 6 month release cycle, where openSUSE Leap is about a year cycle. Tumbleweed is a rolling release, so no sure how useful that would be for Docker, but perhaps it is for Kubernetes. SUSE packages it as well as openSUSE for Leap and Tumbleweed. https://software.opensuse.org/package/docker
Leap and SUSE align, but your right that perhaps maybe the binaries are not the same with SUSE. If It's sources, then they will be the same. Just an FYI, for openSUSE Leap 15.3, it looks like it will be source and binary compatible with SUSE, so maybe you could share that info in the review of the build-matrix.

I use the Docker version from the openSUSE repository. However, a official docker version would be a loooot better..

I use the Docker version from the openSUSE repository. However, a official docker version would be a loooot better..

However, a official docker version would be a loooot better..

The best way to get that prioritized, and get visibliity is to open a ticket in our public roadmap; https://github.com/docker/roadmap
Given that we currently don't have official packages for (open)SUSE, I'm closing this PR for now, but thanks again for contributing!


rename routes

server/routes/v1/api.js > server/routes/api/v1/index.js - more readable and logical

5f55d83d353655bf2b38ccb3f4412d5a2a9403d7 - fix


Is there a way to monitor this system i.e if it failed or has stopped running?

Is there a way to monitor this system i.e if it failed or has stopped running?

https://github.com/Yelp/elastalert/issues/3178

elastalert is no longer maintained


Add support to access project repositories via the Project API

Description:
$title.
We need to access the various project repositories to get the modules imported in each repository such as modules pulled from central, local repository and distribution repository. Currently we can only access the distribution repository.

Added a PR[1] by adding utils to get packages from each repo in DependencyUtils.
@nadeeshaan We can either use those utils, or you can do the same on the lang server side. WDYT?
[1] https://github.com/ballerina-platform/ballerina-lang/pull/33839

Added a PR[1] by adding utils to get packages from each repo in DependencyUtils. @nadeeshaan We can either use those utils, or you can do the same on the lang server side. WDYT?
[1] #33839

@pramodya1994 I think it is better to have the utils, since the other tools also can access the content via a common utility API.

@nadeeshaan As @pramodya1994 suggested you can access the packages in the central cache and local repo as follows:
BallerinaUserHome ballerinaUserHome = BallerinaUserHome.from(project.projectEnvironmentContext().environment());
// Get packages in remote repo
Map<String, List<String>> remoteRepoPackages = ballerinaUserHome.remotePackageRepository().getPackages();
// Get packages in local repo
Map<String, List<String>> localRepoPackages = ballerinaUserHome.localPackageRepository().getPackages();

There won't be any changes to these APIs in SL, and so it's safe to use these APIs directly. We can provide a util in the future if a need arises.

@azinneera thank you. We will have a look at this approach.

Closing since the solution suggested in https://github.com/ballerina-platform/ballerina-lang/issues/29555#issuecomment-979700615 is implemented.


cargo neon start failed

when I tried to start it locally according to the quick start, some wrong things occured,
the console output:
[root@localhost neon]# cargo neon start
Finished dev profile [unoptimized + debuginfo] target(s) in 1.22s
Running target/debug/neon_local start
command failed: Failed to spawn storage_broker subprocess
Caused by:
0: Could not spawn storage_broker, see console output and log files for details.
1: No such file or directory (os error 2)
Starting neon broker at <IP_ADDRESS>:50051
cause I'm a newer to neon postgres, I  have no idea to do next step, anyone can help

did you pull the submodule and build postgres?


Program not compatible with version 0x06140001 / No support for force distance graphs?

I know you've stated there are hardcoded dependencies for certain versions, so this issue is already known. I've tried running the software with check_version=False, in which case the read() operation succeeds and returns a <nanoscope.nanoscope.NanoscopeFile> object. When trying to do any processing however, it returns a NoneType object. I've checked the methods and there are no images in the list_images. Is this package only designed for image files then? I'm unsure if the problem is because of the version or because I'm processing curves not images.
Please let me know if you have any advice on correctly pulling the F-D data from the binary portion of the file!

The software was initially designed with only images in mind - so the issue is likely that there is no support for curves in the parsing code rather than the file version. As far as fulling the F-D data from the raw binary I'd have to check the documentation on how it's stored.


python311Packages.hachoir: 3.2.0 -> 3.3.0

Diff: https://github.com/vstinner/hachoir/compare/refs/tags/3.2.0...3.3.0
Changelog: https://github.com/vstinner/hachoir/blob/3.3.0/doc/changelog.rst
Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 276368 run on x86_64-linux 1

4 packages built:

hachoir (python311Packages.hachoir)
hachoir.dist (python311Packages.hachoir.dist)
python310Packages.hachoir
python310Packages.hachoir.dist


Migrate to singleton

Only one instance of VideoBackground is needed to display any number of video backgrounds on views. Therefore, we will be migrating to a singleton. Note that this will be a breaking change, and will probably occur at release 3.0.0

There is already a singleton option, and I guess we could leave init public and deprecate it.

@quanvo87 this will no longer be necessary after our decision to switch to a UIView extension moving forward, correct?

Correct

Oops, just read your message in the PR.. I'll leave this open until we decide how we want to move forward.


Wrong time is returned

When using the sentence remind me to wash clothes in 5 minutes it's returning the wrong date. But when using the same sentence using remind me to wash clothes after 5 minutes it's working perfectly. Please fix this thing.
Waiting for the fix soon

@typhoon11 you mint want to check with the underlying package used here https://github.com/wanasit/chrono