react-native link react-native-sentry does nothing

Platform:

[x ]  iOS
[ x]  Android

I have following issue:
Recently attempted to update my version of react-native-sentry in my app. I installed the latest version of react-native-sentry with yarn. Ran react-native unlink react-native-sentry followed by react-native link react-native sentry. Nothing is linked or added. Output looks like this
poffdeluxe:app (build-fixes)$ react-native link react-native-sentry
Scanning 808 folders for symlinks in /Users/poffdeluxe/Development/app/node_modules (7ms)
poffdeluxe:app (build-fixes)$

If I attempt to re-run react-native unlink react-native-sentry I get:
rnpm-install info Android module react-native-sentry is not installed
rnpm-install info iOS module react-native-sentry is not installed

According to docs, I should have files added and patched with the link but none of that is happening.
Using:
react: 16.0.0-alpha.12
react-native: 0.47.0
sentry-cli: 1.19.1
react-native-sentry: 0.22.0

Hey, thanks for reporting.
Can you please upgrade to react-native 0.47.2 this should fix this error.
This was an error in react-native itself.

Dang! Thought I was all up-to-date. I'll give this a shot soon. Thanks for the help and sorry for cluttering up your issues 😆

Worked! All good!


Editing suddenly fails: command 'rust-analyzer.onEnter' not found

I've been using rust-analyzer for a few days now, and in the middle of editing a document vscode gave me this error: command 'rust-analyzer.onEnter' not found.  I am no longer able to add newlines to the file.
According to rust-analyzer.exe --version I'm running version 6db2da4 which appears to be after some fixes related to the vim plugin.  However, I am not using that plugin.
Reloading the window fixed the problem.

Why did it happen while in the middle of editing a file?
Why did reloading the window fix the issue?

Rust analyzer probably crashed. Is there anything on the output panel for "Rust Analyzer Language Server"? Reloading the window restarts rust analyzer, thus fixing the issue.

Did you build rust-analyzer  from source and set the following in your settings.json file?
{
    "rust-analyzer.serverPath": "rust-analyzer"
}

Otherwise, the extension will use the prebuilt binary. To get the version of rust-analyzer for sure you should press Ctrl+Shift+P and input > Rust Analyzer: Show RA Version.
This is very odd that onEnter was deregistered in the middle of editing a document. I wish I could see the logs from Help > Toggle Developer Tools > Console when this happened.
If that happens again might I ask you to copy them here?

I've had it happen twice so far.  I'm using rust-analyzer built from source, which is how I got the version number 6db2da4.
I don't recall seeing the ability to see Rust Analyzer output when it was in that state, however I see the ability to see output now.  I'll post its output next time it happens, or I'll let you know if that terminal window disappears.

@xobs, might this be related to vscode-tree-sitter extension? It appears that it aborts the extension host on any unhandled promise rejections, that would cause onEnter not working...

It's entirely possible another extension is doing something like that, but no, I don't have vscode-tree-sitter installed.  I do have several others, and I'll start disabling those to see if that improves things.

@xobs are you still seeing this?
Node that we now disable onEnter by default.

@xobs are you still seeing this?
Node that we now disable onEnter by default.

I haven't seen this in a while, so I will close this as it is no longer reproducible. Thanks!

I haven't seen this in a while, so I will close this as it is no longer reproducible. Thanks!


409: Label name exists or conflicts - aborted error

The issue tracker is for reporting product deficiencies. How do I questions should be posted to the discussion forum at https://groups.google.com/group/got-your-back. When in doubt, start at the discussion forum and return here only when instructed to do so.
Please confirm the following:

I have upgraded to the latest GYB release from https://github.com/jay0lee/got-your-back/releases and I still have this issue.
I am typing the command as described in the GAM Wiki at https://github.com/jay0lee/got-your-back/wiki

Full steps to reproduce the issue:

Perform a back up of email address
Attempt to restore to new email address for archiving.  Email is shared with multiple backups of other email addresses.  Use '--label restore" to label back up under email name
Process for restore begins, then halts with "409: Label name exists or conflicts - aborted"

Expected outcome (what are you trying to do?):
I am trying to restore the backups of employee email domains to a single archive domain.
Actual outcome (what errors or bad behavior do you see instead?):
The process of back up suddenly halts, either half way or part way with this error even if the intended label is brand new.  This is happening with two different email addresses, and repeats regardless of a new name or label being assigned to the restored files

Hi, I'm seeing this issue too.
In one case it's a label with a whitespace at the end (possibly generated via IMAP), in another case it's a simple "sonstiges" (else in german).
Haven't reached the label limit, under 2000....
Anything i can do to help idnetifiying this?
Thanks Luca

I was able to find a work around for the problem.  I added initials to all
existing labels in the original email to prevent duplicate labels and
previous emails that were having issues I stripped of their old labels and
created a new overall label designation.  After these adjustments I had had
no problem restoring them to an archive regardless of how many files were
being backed up.
Thanks for touching base
Jessica Dayley
Admin Assistant, Canyon State Electric Co., Inc.
602-437-3088 |<EMAIL_ADDRESS>| www.canyonstateelectric.com
|
4640 E. McDowell Rd, Phoenix, AZ 85008
IMPORTANT: The contents of this email and any attachments are confidential.
They are intended for the named recipient(s) only. If you have received
this email by mistake, please notify the sender immediately and do not
disclose the contents to anyone or make copies thereof.
Get a signature like this: Click here!
http://ws-promos.appspot.com/r?rdata=eyJydXJsIjogImh0dHA6Ly93d3cud2lzZXN0YW1wLmNvbS9lbWFpbC1pbnN0YWxsP3dzX25jaWQ9NjcyMjk0MDA4JnV0bV9zb3VyY2U9ZXh0ZW5zaW9uJnV0bV9tZWRpdW09ZW1haWwmdXRtX2NhbXBhaWduPXByb21vXzU3MzI1Njg1NDg3Njk3OTIiLCAiZSI6ICI1NzMyNTY4NTQ4NzY5NzkyIn0=&u=582954008207327
On Wed, Mar 29, 2017 at 5:00 AM, lbenelli<EMAIL_ADDRESS>wrote:

Hi, I'm seeing this issue too.
In one case it's a label with a whitespace at the end (possibly generated
via IMAP), in another case it's a simple "sonstiges" (else in german).
Haven't reached the label limit, under 2000....
Anything i can do to help idnetifiying this?
Thanks Luca
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jay0lee/got-your-back/issues/124#issuecomment-290068399,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AYFEug-V9ID48k6rlnhBK1Nja3z-fBlZks5rqkf6gaJpZM4LpeHD
.


Added Support for MQTT

Added

mqtt.vue a component allows connecting to MQTT broker over Websockets.
Added paho-mqtt library.

Notes:

Used manualDisconnect property to check wether connection was closed intentionally. (onConnectionLost is called in all cases, connection disconnected intentionally or because of some error)

Travis tests have failed
Hey @rahulnpadalkar,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.
TravisBuddy Request Identifier: fd284290-63c6-11ea-88d5-a5122dc139bd

Hey @rahulnpadalkar,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: e19c0910-63c8-11ea-88d5-a5122dc139bd

I discovered a bug (insecure websocket connection is not established over HTTPS). I have fixed and tested it. So, should i put a new issue and do another pull request? Sorry i am new, so i don't know the exact process.

Update your repo and make another Pull request. That's it.


[INLONG-4755][Manager] Resolve swagger display LocalDateTime LocalTim…

…e format issues
Prepare a Pull Request
(Change the title refer to the following example)

Title Example: [INLONG-XYZ][Component] Title of the pull request

(The following XYZ should be replaced by the actual GitHub Issue number)

Fixes #XYZ

Motivation
Explain here the context, and why you're making that change. What is the problem you're trying to solve?
Modifications
Describe the modifications you've done.
Verifying this change
(Please pick either of the following options)

[ ] This change is a trivial rework/code cleanup without any test coverage.

[ ] This change is already covered by existing tests, such as:
(please describe tests)

[ ] This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
If a feature is not applicable for documentation, explain why?
If a feature is not documented yet in this PR, please create a follow-up issue for adding the documentation

If the back end interface returns LocalDate or LocalDateTime or LocalTime ,Using Swagger2 causes the page to fail to display ,The Swagger2 configuration supports the display of LocalDate, LocalDateTime and LocalTime


Update HWIMO-TEST mocha timeout to 10s.

This is necessary for some build machines where dependency
loading is too slow to stay under the 2s per test timeout.
@RackHD/corecommitters

Thanks @jimturnquist , This will be much better than changing the unit-test code.
:+1:

👍


What is the exact command to set the starting claim blocks?

I am trying to set the starting claim blocks of my server to 500. A day or two ago you referenced me to this: https://github.com/MinecraftPortCentral/GriefPrevention/wiki/Options , which shows me the permission, but where do I input the value for the permission? (so like could you for example show me the permission you would type to make starting claim blocks 500).

It is at bottom of options page.
/lp user/group <user|group> meta set <option> <value> [server] [world]


NIFI-13545 Improve BootstrapListener getClusterStatus to handle facto…

…ry being null

Summary
NIFI-13545
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[X] Apache NiFi Jira issue created

Pull Request Tracking

[X] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[X] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[X] Pull Request based on current revision of the main branch
[X] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[X] Build completed using mvn clean install -P contrib-check

[X] JDK 21

Licensing

[ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[ ] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[ ] Documentation formatting appears as expected in rendered files

Thanks @bbende changes make sense to me. +1 Will be happy to merge once Github Actions complete.


Header icons

Added

fontAwesome dependency
header with title and icons

only fullscreen icon/link works

Next Steps

add information page popup
add share dropdown
add tests

Can you please include a screenshot in the PR description? I'll check out the code locally to see how it renders, but still a good practice.


CANNOT IMPORT DO_MAIN

Traceback (most recent call last):
File "/usr/local/bin/ciscoreputation", line 7, in
from ciscoreputation import main
File "/usr/local/lib/python3.5/dist-packages/ciscoreputation/init.py", line 1, in
from ciscoreputation import do_main
ImportError: cannot import name 'do_main'

Traceback (most recent call last):
File "/usr/local/bin/ciscoreputation", line 7, in
from ciscoreputation import main
File "/usr/local/lib/python3.5/dist-packages/ciscoreputation/init.py", line 1, in
from ciscoreputation import do_main
ImportError: cannot import name 'do_main'

For whatever reason the virtualenv isn't picking up the package in site-packages.  If you go cd to lib/python3.5/ and:
ln -s site-packages/ciscoreputation/ciscoreputation.py ./
ln -s site-packages/ciscoreputation/about.py ./
There's then errors about "main" not being able to be imported in bin/ciscoreputation
I was able to clear up that error by changing main to do_main but the code still fails for me trying to query anything.


Enshrines in stone the CentCom-given right of mute people to say their last words

About the PR
Adds a comment to the checks keep the feature as is
fixes #19019
Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

I-I was m-merely pretending...

I agree, mute people shouldn't be able to death gasp.

Disabled mute deathgasp as suggested


Mechanical pipes not transparent

Hi all,
Recently played a bit with Mekanism and I noticed that the mechanical pipes are not transparent, so they are not showing the fluid that they're transporting. I looked it up but I can't find why this is happening.
I'm playing mekanism in the Modern Skyblock 3 modpack, but other people playing an earlier version of this modpack don't have this issue.
I'm using Mekansim 9.4.13 and Forge version <IP_ADDRESS>
Here's a screenshot of the machanical pipes

There's a setting for opaque rendering.


Email in return results is inconsistent when using auth-refresh when an expand option is used

If email visibility is not set for the record then /api/collections/users/auth-refresh will return the authenticated user for that token.  This record will contain the email.
However if /api/collections/users/auth-refresh?expand=users_sites(user).user is called then the top record will not contain the email.  But the expanded relation record for user will contain the email.
However, If I turn on email visibility for this user record then it will appear in both places (in top level record and in expand section.
So there is a bug relating to both the variables together which check email visibility and if expand is happening.  Also this might be a good time to check if you're being overly protective of email visibility, or if it could slip through with other API calls.

I've tested it locally and I'm not able to reproduce the issue.
The authenticated user should always have access to their email, no matter of the emailVisibility flag.
Or in other words, it is expected to see the email of the authenticated user in the auth-refresh call, as well as in the expand result (if the same user is listed).
If that's not what you are experiencing, please export your collections (Admin UI > Settings > Export collections).

So I just ran into this bug and am able to reproduce it with my set up. I have confirmed that removing the expand params from the auth refresh call will show email regardless of emailVisibility but once I set the expand params, the email disappears when emailVisibility is false.
Maybe this is something specific to the way permissions/filters are set up?

@jmar910 As mentioned above I wasn't able to reproduce the issue.
If you are not able to identify the cultprit please provide at least some steps to reproduce, for example more details about your collections structure (you can export your collections from the Admin UI > Settings > Export collections) and a code sample showing how you are sending the request.

@jmar910 Also note that if you use Next.js it could be cache related.
To rule out issue with other applications, please try to test with a static html in pb_public or by directly sending a request via curl, Postman, Insomnia, etc.

@jmar910 I'm not able to reproduce it. To test it on your own, please create the following pb_public/index.html file:
<!DOCTYPE html />
<html>
<head>
    <meta charset="utf-8" />
    <title>Expanded user email test</title>
</head>
<body>
    <script type="module">
        import PocketBase from<EMAIL_ADDRESS>
        const pb = new PocketBase("http://<IP_ADDRESS>:8090");

        const result1 = await<EMAIL_ADDRESS>"1234567890")
        console.log("user after auth:", result1.record.email)

        const result2 = await pb.collection("users").authRefresh({ expand: "roles" });
        console.log("user after refresh:", result2.record.email)
    </script>
</body>
</html>

Navigate to http://<IP_ADDRESS> in your browser and check the browser console.

pb_schema.json
Was able to reproduce using your example and this set of collections. My guess is it has something to do with our roles/permission/permissionSets in relation to API filters, but I still wouldn't expect the field to just disappear from the collection I do have access to.

Thank you for the json. I was able to reproduce it (it is a side effect from the fields resolver). Will submit a fix sometime later today.

Thank you! Not urgent as we have worked around it for my case, but glad we could get to the bottom of it. Totally appreciate your quick and helpful responses/examples.

I've updated the field resolver tests and pushed a small fix as part of PocketBase v0.21.2 release.
If you are still experiencing the issue, feel free to let me know and I'll try to investigate it in more details tomorrow.


Prepend syntax selectors with .syntax-- for Atom 1.13+ compat

Thanks @sknoslo

Hey @jmcalaway glad to help. I intended to mention this in my PR but I believe you will need to add atom >= 1.13 to package.json, as this change is not backwards compatible.

Good point! Thanks again


Need MSYS path to build caffe on Windows machine

msys dir must be added to PATH to make compilation of lib-openblas working.
MinGW installer does modify PATH variable by default.

script download-and-install-package/install.bat contains calls like sleep 2, but Windows has not this command (reported here https://github.com/ctuning/ck-env/issues/20)
makefile for lib-openblas calls for 'uname' and 'perl' which are not available by default and contained in msys

It seems, installation guide for window should note about it.

Actually, openblas's wiki (https://github.com/xianyi/OpenBLAS/wiki/Installation-Guide) tells about this requirement:
In order to build OpenBLAS on Windows you will need:
MSYS (a collection of Unix tools, including "make"). We recommend this distribution: http://sourceforge.net/projects/mingwbuilds/files/external-binary-packages/

I am trying to minimize dependencies. I fixed the problem with sleep, and as for the second part, I use mingw32-make.exe from MingW GCC so normally it should not require MSYS (I think perl is not strictly necessary). But we will test it soon on a Windows 10 Surface Pro which we should get soon...

I believe I fixed that. I added detection of Unix Utils on Windows and in fact they are available in the Git installation (if selecting option of Unix utils), or they will be also detected if downloaded and unzipped manually to a user space ... I close this ticket ...


chore: update ansi-regex

Description
Update ansi-regex which was flagged as vulnerable. See https://github.com/advisories/GHSA-93q8-gq69-wqmw
Tested
CI passes
How others should test
N/A
Related issues
N/A
Backwards compatibility
Yes

Hey @MuckT since NA is mentioned in how other should test section we are skipping this PR
CC: @jeanregisser
Thanks


Make CheckedFile support the *BSDs

Currently, CheckedFile.cpp feature checks only cover Apple, Windows
and Linux; on BSDs the build bombs out with "no supported OS platform
defined" (l.51).
The BSDs follow POSIX for file operations - which is almost like
Apple, only with the addition of sys/stat.h for the file modes,
so supporting Free/Open/NetBSD can be done with very little changes.
Alternatively, the "linux" branch could be changed from
_LARGEFILE64_SOURCE to _FILE_OFFSET_BITS=64 (switching from lseek64() to
lseek()), but there is no reason given for the current choice which
makes risk assessment harder. This patch chooses the path of least
change to other (linux) platforms.
Build tested on FreeBSD and as part of FreeCAD on FreeBSD (where I
noticed the problem for the first time), the other BSDs should be
fine according to their man pages.

Thanks Christophe!

could be changed from _LARGEFILE64_SOURCE to _FILE_OFFSET_BITS=64 (switching from lseek64() to lseek()), but there is no reason given for the current choice

Unfortunately that goes back to before I forked it, so I have no way of knowing.
This file (and specifically these includes/file functions) have been a source of problems several times. It could benefit from a review/rewrite.


Mobile layout

In portrait mode, on my galaxy nexus, the "got unread" goes away if there are thread marks.
In landscape mode, the threadmark button isn't aligned with the unread button properly

Screenshot?

Turns out XF hides the unread link on narrow viewports!
See page_nav.css:
@media (max-width:@maxResponsiveNarrowWidth)
{
  .Responsive .PageNav .unreadLink
  {
    display: none;
  }
}
</xen:if>

The landscape issue might(?) be unavoidable as I think the pagination links are meant to wrap down to the next line if there's not enough space to fit it on one line. Can you verify?
I think it's worse for mods/admins/thread starters as they have the Thread tools and Watch thread links on the same line as the pagination which reduces the available space.

The Thread starter also can have the same Thread Tools options. Playing around with Chrome's emulation stuff, the pagination links do wrap to the next line. But you need to make the screen utterly unusable small.
However, having a phone/tablet where the unread button was there, and then suddenly is not after a feature is unused in the thread is not in my mind desirable behaviour. If they never had the button, it would be a different story.

As a thread starter, I'm seeing the following at ~500px width, a reasonable viewport size:

The lack of top margin on the  Go to first unread button looks like a styling bug in XF to me.
When logged in as a non-thread starter (so only Watch Thread is visible) the styling is setup to ensure the pagination never wraps. Given this is the behaviour the majority of users see, I think we should ensure it stays that way when the threadmarks is there. I'll look into that now.
As pagination wrapping for mods/admins/thread starters is already happening in XF I think we should leave that as is.
I'm not sure what you mean in your second paragraph above. If you're referring to the Go to first unread button disappearing on smaller viewports then that's XF's styling which again, I don't think we should change.

Sorry,I was being very verbose in saying;

If they have an unread button with a given resolution, they should keep having one.

So you're saying the XF behaviour is the problem? This isn't an issue we need to address in this add-on?

For borderline resolutions, adding a threadmark button will make the unread button go away. I would prefer to avoid that.

I'm not sure at what resolutions this happens.
Can you try PR #34 and see if it fixes it?

Ok, testing now

Fixed by #34.


Remove covid banner from question page

We've decided the covid banner on the covid question isn't needed
as we have it on the TS homepage.
After changes

Checklist:

[x] Have you documented your changes in the pull request description?
[ ] Does the change present any security considerations?
[ ] Is any gem functionality overloaded? Eg: Devise controller methods being overloaded.
[ ] Has acceptance criteria been tested by a peer?
[ ] Has the CHANGELOG been updated? (If change is worth telling users about.)

 Remove banner for COVID from Covid question


Move all linux slaves to docker images

This updates all our linux slaves to be driven from docker images (defined in this repo). Configuration for the master was tweaked here and there for new paths and such, but otherwise all changes here are just creating the docker images. All-in-all, we've now got 4 docker images:

android - this is used for the auto-linux-64-x-android-t and nightly-dist-rustc-arm-android builders. This may be able to get merged with the linux image below in the future, but it would require figuring out how to isolate emulators. Right now two builders use this image and it only works because the slaves are limited to just one build.
dist - our CentOS 5 image with a "very old libc". This has tons of custom-built software, but it's used to produce cargo and rustc nightlies. This is not used for MUSL or cross nightlies.
linux - the standard Ubuntu 14.04 image we run all linux tests in. This is also used to build MUSL as it's installed on the side.
linux-cross - a 14.04 image to build all the cross-std architectures and such. This is not included in the normal linux image because Ubuntu's current situation means that you can't have a multilib (32 + 64 bit) gcc while also having all the cross compilers. Instead we just have a 64-bit gcc with all the cross compilers.

r? @brson
(rust_highfive has picked a reviewer for you, use r? to override)

May not want to merge just yet as the final tests are still running, but wanted to get some initial feedback

:+1: if the tests behave as expected

r+

Everything passed in dev!


Move signup state to server side

This is a moderately large PR that makes a significant change to the logic of the application in how signups' state is handled. Closes #33.

Signup state (in quota, in open quota, in queue) is now computed server-side. Previously the only place that computed this was the frontend.
The state is now stored in the database.

This makes some code less clean and less performant, since we need to recompute and dynamically update the state whenever stuff is touched. Especially, we lock the event's database rows for the recomputation - benchmarking TBD.
However, this has a major benefit: regardless of what affects the computed statuses of signups, we can now detect it in one place and e.g. send "you got in from the queue" emails.

Previously this was very hard, as every place that could affect this (deleting/expiring signups, changing/deleting quotas, ...) would need to delete events one by one and carefully update the next one from the queue.
In fact, it wasn't being done at all in multiple places.
Previously, if a place was missed, that change was gone. Now we can fix it with just a recompute.
The recomputation is idempotent, so extra recomputes don't hurt.

The code doing status computations is now much cleaner on both sides.

There are also some other changes here:

Users are now not marked as kicked out of events when quota sizes are reduced. This applies both to the open quota and normal quotas.

Deleting quotas still deletes signups in them. #41 will improve that with warnings.
After this PR there is no way to force accepted users out of quotas.

We should discuss if this is a desirable functionality, and what would be the expected behavior when reducing quota sizes. I think we should warn the editor (#41) and then kick users out of events again. It would also be nice to notify these users in some way.

This now causes an error:users A and B load editor
user B deletes question/quota Q
user A saves event with question/quota Q still there

This is much better than the old behavior, where it would silently re-create Q, and delete and re-create it on every further save until a page reload. The optimal solution would be real conflict handling, but that's a can of worms.
getEventDetails was cleaned up, along with its API which now returns more sensible results.
Some bugfixes.

I support the signup state refactoring logic written above, as I think that the event of adding or deleting one signup should be uncoupled from any side effects the action might cause for other signups. Though the side effects are inevitable, the responsibility of managing those side effects don't belong to the event.
If this solution ends up being performant enough, this should be good as it is.
I do wonder though... as it seems to me, the only REAL reason we need to store the status of the signup in the DB state is to be able to track signups that are moving from one status to another (in queue -> in quota). Otherwise storing the status is not needed - the current (and for most functions the only meaningful) state of the signup can be trivially calculated from the whole of the event's signup data.
Another way to implement the status change notification could be as follows:

At one moment, fetch the signups, calculate their statuses, and store those statuses.
At another moment (after an update or several), fetch the updated state of signups.
Calculate the statuses for updated signups, and compare those to the stored ones.
Send notifications to any users with signups that have changed status between the updates.
Store the calculated statuses and repeat.

As said, this might all be unnecessary if the current solution or optimizations of it are sufficient. Just writing it down as it comes to me...!
The smaller changes are also meaningful and cleanly executed. :+1:


Selected Text Not Deselected When Clicking on Empty Space

I'm having an issue with the text editor where selected text doesn't always get deselected when I tap on an empty space. Sometimes it stays selected even after multiple taps, and the behavior seems inconsistent.

Hello @fmhayirli 👋
As soon as you tap an area of the editor, the focus will shift from the selected text to the tapped area.
If you tap a zone that isn't focusable (e.g., outside of the editor), the text will remain selected and retain focus. This is the system's behavior.
You should check the size of your editor and the zone you are touching.


Enable bytechecked deserialization for the plugin

Describe the feature
We are currently doing unchecked deserialization (https://github.com/swc-project/swc/blob/main/crates/swc_common/src/plugin/serialized.rs#L124) for the plugin deserializations.
This was deliberated decision at the moment, however it exposes some of cons recently - as it implies, it is unsafe operation and can take down host randomly without having additional details. We could try checked instead to ensure host - plugin can exchange data as expected, and know it'll fail prior to actual serialization.
Unfortunately this is blocked by https://github.com/swc-project/swc/issues/5406, since this requires to enable bytecheck for each structs. We may carefully approach enabling bytecheck first to get there.
Babel plugin or link to the feature description
No response
Additional context
No response

https://github.com/swc-project/swc/pull/7280


Added python3 hello world example.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.danderson seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

@zzaarrgg if possible, can you please sign the CLA on this one ? Thanks !


image_zfs: Fix inverted logic

zpool list -Ho ${ZFS_POOL_NAME} exits with 0 if the name exists. This prevents overwriting the existing pool.
Fixes: ae641ed4ff326fb338aff781506e4b06ead65bfc
cc/ @allanjude

Good catch, thank you.
@evadot Can we get this merged quickly as a followup fix

@allanjude I have another fix (that avoids a segfault), I haven’t fully investigated the cause, but will sumbit a PR just in case. Thank you!


updating readme file

Updates
I updated the readme file. Please review and accept

Thoughts
I probably shouldn't but okay. 👍


Arabic Support

Could you add Arabic letter characters Support ?

Barcodes (as in: 1D) can only contain the characters specified in their specification. For example, EAN13 can only hold numbers, Code128 can contain numbers, US-ASCII letters and some special characters, etc.

If you are talking about 1D-Barcodes, look into their definition. I know of no 1D-barcode that can hold unicode (specifically arabic) letters.

http://stackoverflow.com/a/35427835
You might want to look into printing your data on a 2D barcode such as a QR-code.

it is working only with Format: GenericBarcode:

Generic Barcode is just that. A generic barcode that always generates the sequence 10101010...10101. It accepts everything but does not encode it.

If you have an barcode symbology that can encode Arabic characters, please link me to it and I'll see what I can do. But when searching for it I can't find any symbologies that support utf-8 so I think you are out of luck :unamused:

Have your question been answered? And can the issue be closed?


rotating the camera

Hi,
First of all, thank you for this wonderful addon.
I'm wondering if you have a solution for rotating the camera point of view inside the 3d scene?
I'm currently playing with 'setOrientation()' in the resize() method but it's a nightmare compositing the angles when rotating around more than one axis.
Any advice maybe ... ?
Antoine

Hello,
Try using ofNode. You should be able to set the dome camera rig as a child and transform the node.

thank you a lot. that's indeed the way to go :)


[ Gateway ] : Update project dependencies to the latest one

Issue Type

[x] bug report
[ ] feature request
Current Behavior

Currently, most of the packages and node modules are outdated and incompatible with the latest ones.
Eg. @testing-library/react need react: 18+
likewise, update node module and packages to the latest one and validate the application.
Expected Behaviour

Steps to Reproduce

Suggested Fix

Closing


Make sure all repos support add_metadata_columns setting

Originally discussed in slack https://meltano.slack.com/archives/C03E93T5XE2/p1684425396046939

This makes me want to have a section that lists the "standard" fields you get when it's on the SDK. But, separate that would be.


Update gpg to version 3.0.0

Gpg4win version 3.0.0 don't contain gpg2.exe

There isn't even a gpg.exe

gpg4win-3.0.0.exe
├── $TEMP
│   ├── gnupg-w32-2.2.1-20170919-bin.exe
│       ├── bin
│           ├── gpg.exe

Now it uses the gpg4win installer instead of extracting it. Is this working for you?

Yes, it's works. But now I have UAC promt. Maybe replace Gpg4win with clean gpg from gnupg.org

Or we could just add gnupg as a separate manifest (extracting this will work fine). 😄
And we also extract Gpg4win. Don't know if these two will work together (but it's probably just a PATH thing)

Gpg4win components:

gnupg - backend, cli
Kleopatra - gui for gnupg
GpgOL - outlook plugin
GpgEX - shell extension

Anyway we can't use shell integrations without installer because it requires adding registry keys.
Old version manifest for gpg4win used extraction executable file and bin section contains only gpg and gpg2(now is gpg.exe).
If we use only cli app, why we download gpg4win instead gnupg?

Focuses on developer tools. While it would be easy to install Skype with Scoop, this will probably never be in Scoop's main bucket (app repository). Scoop focuses on open-source, command-line developer tools.
Chocolatey Comparison

We are working in the Gpg issue in #1730

Just a friendly reminder that GnuPG is in extras: gnupg.json.

Any progress?

Added gpg4win.json to the extras bucket
Added gnupg.json as an gpg.json alias to main bucket
Switched gpg4win to gpg https://github.com/lukesampson/scoop/commit/9e191a107e80103033eb566ce28bfa1320d0a0da

Based on the above dialog, it appears this PR can be closed. Please reopen, if I’m misunderstanding. Thanks!


Dependencies for ledis branch

Going to merge dependencies into _third_party before the other branch rather than after. Ignore this.


Upgrade message shown during upgrade k3s cluster doesn't show all nodes if concurrency is more than 1

What kind of request is this (question/bug/enhancement/feature request):
Bug
Steps to reproduce (least amount of steps as possible):

Import k3s cluster (1 control, 6 workers) - version 1.16.3
Edit cluster and upgrade it to 1.16.7 with concurrency 3

Result:
While upgrading 3 worker nodes simultaneously, upgrade message shows only 1 node.

Expectation
While upgrading 3 worker nodes simultaneously, upgrade message should show all nodes.
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): master-2486-head- cf6d0c979
Installation option (single install/HA): Single

Tested on Rancher version master-2523-head- bd2eb0cf9
Upgrade message shows all the nodes which are getting upgraded.
Tested for concurrency 1 and concurrency 3.
Sample
This cluster is currently Upgrading.
Worker node [ip-xx-xx-xx-xx, ip-yy-yy-yy, ip-zz-zz-zz-zz] being upgraded.


Keywords are not reserved in the reference implementation

consume() disregards the type of the token and just inspects its text.

Pretty sure that was fixed a while ago. The brand new lexer very much reserves keywords.


@ implode() error preventing setup from completing

Any information on the error?

Yes, sorry - this is an old "fix" for installer quitting when display_errors and error_reporting are configured for debugging...
18 Jul 2020 21:41:05 Sam Poyigi<EMAIL_ADDRESS>
Any information on the error?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub[https://github.com/tastyigniter/setup/pull/2#issuecomment-660523868], or unsubscribe[https://github.com/notifications/unsubscribe-auth/AAEZT6VV3IHPHLCFZYCUJEDR4HUD3ANCNFSM4O7JM2JQ]. [https://github.com/notifications/beacon/AAEZT6XKXNQVVE4VZYURY73R4HUD3A5CNFSM4O7JM2J2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOE5PMWXA.gif]

Do you have a full stack trace to the error? Or something to help reproduce the error?

Closed due to no response.


cmd/go: reject module path beginning with slash

Re #28336, we should

Reject go mod init /absolute/path.
Refuse to load any go.mod that says module /absolute/path.

If we'd done either of those (we should do both), then the confusing build errors in #28336 would not have happened.

To me it seems that denying full path in mod or init it would be counterproductive for people willing (or have no choice but) to specify a full path because thats how they have their build system set up
I would not surprised to have expect this in a bootstrap script
go mod init /bigdisk/buildmount/$USER/$REPO/some/path

For reference, cmd/go/internal/module.CheckPath is the function go get uses to validate module paths. It performs a number of checks, not just for leading slashes.
In tests, I think we're using module paths that don't satisfy this check (e.g., the module path is a single word without a .). It's likely other people are taking advantage of this as well, both in tests and in monorepos that are never fetched with go get.

Trailing slashes should probably also be blocked. It works as long as you don't have sub packages. As soon as you try to import a sub package you get errors like
build github.com/satappscatapult/warmconsidering/: cannot find module for path github.com/satappscatapult/warmconsidering/message
which is mighty confusing
series of comands
go mod init github.com/satelliteapplicationscatapult/warmconsidering/
code .
# write some things and create a submodule
go build
build github.com/satappscatapult/warmconsidering/: cannot find module for path github.com/satappscatapult/warmconsidering/message

edit the go.mod file to remove the trailing slash from the build starts to work as expected.

This is now rejected with go: malformed module path "/example": empty path element in both cases


Android Studio //i am noob

Can it run with an API less than 28, for example API 25?

Yes, you can set the api_level in android_ndk_repository and android_sdk_repository in the WORKSPACE file. For example, to make the cpu demos work on a Google Nexus 5 with Android 6,  we need to set the api_level of android_ndk_repository to be 21. See more details in https://github.com/google/mediapipe/issues/178#issuecomment-547619564.


Fail to build repository image on Apple ARM-based chips (M1, M2)

Following the instruction in the README you can build the repository image by running
docker-compose -f environments/build-repository.yaml build

However this fails when running from an Apple ARM-based host, essentially because python is not found
14.71 ><EMAIL_ADDRESS>install /select-contract-form/node_modules/node-sass
14.71 > node scripts/install.js
14.71
14.92 Downloading binary from https://github.com/sass/node-sass/releases/download/v4.14.1/linux-arm64-83_binding.node
15.56 Cannot download "https://github.com/sass/node-sass/releases/download/v4.14.1/linux-arm64-83_binding.node":
[...]
16.01 ><EMAIL_ADDRESS>postinstall /select-contract-form/node_modules/node-sass
16.01 > node scripts/build.js
16.01
16.08 Building: /usr/local/bin/node /select-contract-form/node_modules/node-gyp/bin/node-gyp.js rebuild --verbose --libsass_ext= --libsass_cflags= --libsass_ldflags= --libsass_library=
16.12 gyp info it worked if it ends with ok
[...]
16.13 gyp verb check python checking for Python executable "python2" in the PATH
16.13 gyp verb `which` failed Error: not found: python2
[...]
16.13 gyp ERR! configure error
16.13 gyp ERR! stack Error: Can't find Python executable "python", you can set the PYTHON env variable.
[...]
16.13 gyp ERR! System Linux 6.4.16-linuxkit
16.13 gyp ERR! command "/usr/local/bin/node" "/select-contract-form/node_modules/node-gyp/bin/node-gyp.js" "rebuild" "--verbose" "--libsass_ext=" "--libsass_cflags=" "--libsass_ldflags=" "--libsass_library="
16.13 gyp ERR! cwd /select-contract-form/node_modules/node-sass
16.13 gyp ERR! node -v v14.21.3
16.13 gyp ERR! node-gyp -v v3.8.0
16.13 gyp ERR! not ok
16.13 Build failed with error code: 1
[...]

But this Python error is raised because it cannot find a Node native module for the sass package. From the excerpt above
14.92 Downloading binary from https://github.com/sass/node-sass/releases/download/v4.14.1/linux-arm64-83_binding.node
15.56 Cannot download "https://github.com/sass/node-sass/releases/download/v4.14.1/linux-arm64-83_binding.node":

it then tries to compile it, failing because Python is not installed.

I tried to fix the issue by changing the platform in environments/build-repository.yaml. This avoids to compile the Node native module, given it can be found in their respective GitHub releases[1]
  repository:
    platform: linux/amd64

However, after this change it now get stuck when running npm run build from the Dockerfile
 => [repository stage-0 11/11] RUN mkdir /redirects && cd select-contract-form && npm install && npm run build                                                 46.6s
 => => # > react-scripts build && cp -r build/* ../redirects
 => => # Creating an optimized production build...
 => => # Browserslist: caniuse-lite is outdated. Please run:
 => => # npx browserslist@latest --update-db
 => => # Why you should do it regularly:
 => => # https://github.com/browserslist/browserslist#browsers-data-updating

Not sure why this happens. react-scripts build seems to be quite opaque command, with no verbose option to see what's going on.

[1] https://github.com/sass/node-sass/releases/tag/v4.14.1

Another fail attempt to solve this issue was to try to actually install Python and compile the Node native module. Adding this to the beginning of the Dockerfile
RUN apk add python
# or
# RUN apk add python2

However Python 2 is not available FROM node:14-alpine
1.366 ERROR: unable to select packages:
1.385   python (no such package):
1.385     required by: world[python]

And installing Python 3 raises a syntax error
15.93 gyp ERR! stack Error: Command failed: /usr/bin/python -c import sys; print "%s.%s.%s" % sys.version_info[:3];
15.93 gyp ERR! stack   File "<string>", line 1
15.93 gyp ERR! stack     import sys; print "%s.%s.%s" % sys.version_info[:3];
15.93 gyp ERR! stack                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
15.93 gyp ERR! stack SyntaxError: Missing parentheses in call to 'print'. Did you mean print(...)?

Would updating the node version make the needed python version available? Maybe one of the node:18 version
I believe it wouldn't, Python2 was removed from Alpine.

Is the main sourcify branch hitting this issue also?

It doesn't look like. I have checked both these repos https://github.com/ethereum/sourcify and https://github.com/sourcifyeth/h5ai-nginx and found nothing related.

This was solved in the submodule dependency https://github.com/sourcifyeth/h5ai-nginx/commit/80938c49df26994a4bf69e87c626f6d667333657.
Essentially the key was to use node:14.14-alpine so that python2 make g++ can be installed to compile sass native module.


sqlite error when running ????? syncdb

when I run the sample it will not syncdb - also momadjango.org is likewise failing on registration
with an sqlite error
for the syncdb: I get the following:
Traceback (most recent call last):
File "manage.py", line 25, in
execute_from_command_line(sys.argv)
File "C:\Users\Marc\virtual_main\prj14_moma_django\lib\site-packages\django\core\management_init_.
py", line 443, in execute_from_command_line
utility.execute()
File "C:\Users\Marc\virtual_main\prj14_moma_django\lib\site-packages\django\core\management_init_.
py", line 382, in execute
self.fetch_command(subcommand).run_from_argv(self.argv)
File "C:\Users\Marc\virtual_main\prj14_moma_django\lib\site-packages\django\core\management\base.py",
line 196, in run_from_argv
self.execute(*args, **options.dict)
File "C:\Users\Marc\virtual_main\prj14_moma_django\lib\site-packages\django\core\management\base.py",
line 232, in execute
output = self.handle(*args, **options)
File "C:\Users\Marc\virtual_main\prj14_moma_django\lib\site-packages\django\core\management\base.py",
line 371, in handle
return self.handle_noargs(**options)
File "C:\Users\Marc\virtual_main\prj14_moma_django\lib\site-packages\django\core\management\commands
syncdb.py", line 57, in handle_noargs
cursor = connection.cursor()
File "C:\Users\Marc\virtual_main\prj14_moma_django\lib\site-packages\django\db\backends_init_.py",
line 317, in cursor
cursor = self.make_debug_cursor(self._cursor())
File "C:\Users\Marc\virtual_main\prj14_moma_django\lib\site-packages\django\db\backends\sqlite3\base.
py", line 288, in _cursor
self._sqlite_create_connection()
File "C:\Users\Marc\virtual_main\prj14_moma_django\lib\site-packages\django\db\backends\sqlite3\base.
py", line 278, in _sqlite_create_connection
self.connection = Database.connect(**kwargs)
sqlite3.OperationalError: unable to open database file

You are correct (sorry - I just noticed that post). I fixed the example web site and it is working now.
With regards to the sqlite - you should have a database file created under the moma_example directory. Do you have that file there?


Dark mode stopped working for empty cells

Class "isEmpty" isn't getting added to the <li>s, so those cells have the same color regardless of light or dark mode. Text elements properly switch between modes, just not the empty cells. EDIT: no class names are getting pushed. "today" is missing as well.

Yes, sorry for not adding that.

Same problem, just installed the plugin

In commit 649a7fd638937bace97dbd16196f1355b707fee8, file main.ts, the code
const box: Box = {
    classNames: [],
}

was replaced with
const box: Box = {}

So the following operations won't push the classes (today, hasData and isEmpty) into box.classNames as we expected, thus cause this issue and #63.
// main.ts
box.classNames?.push("today")
box.classNames?.push("hasData")
box.classNames?.push("isEmpty")


Button Click Triggers OnSubmit listener

Using "redux-form": "^6.0.5", clicking on any element of type button without explicitly declaring that button element to be type="button" triggers the form submission. The functionality should require the button to be explicitly declared as type="submit".

This was such a PITA to figure out, should be in the Docs as a red popup. OR make redux form require a submit button...

It don't think it should be mentioned in the documentation. If button doesn't have type attribute set it is going to be submit button by default. That is defined by specification and developers should be aware of that so this isn't redux-form problem.
https://www.w3.org/TR/html401/interact/forms.html#edef-button
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/button

Thanks for doing the job @niksajanjic :)


fix launch error for vehicle_launch

Signed-off-by: 1222-takeshi<EMAIL_ADDRESS>Fixed autoware_launch (psim/lsim) cannot be started at humble.

https://github.com/autowarefoundation/autoware.universe/pull/2171
This PR will close when the above PR is merged.

I think it's good to merge this a while later. It replaces a deprecated API.

The work on this PR has already been completed, but I believe there's no longer a need to merge it, so I will close it.
If any problems occur, I will reopen this.


Fix StringType access, remove unused imports

Fix for IPythonNote book using SVG rendering.
from rdkit import Chem
from rdkit.Chem.Draw import IPythonConsole
IPythonConsole.ipython_useSVG=True
This isn't tested anywhere outside the pidtest.py which I assume is tested with nose.

Unfortunately it's just plain not tested. This area is a gap in the test coverage. Given that it's not critical functionality, this hasn't ever really struck me as a massive problem and now that we will be moving to using the C++ code to produce SVG it seems even less urgent.
Ah, excuses, excuses...


Basketball Add Total, Avg and Projected Points to Player Object

Basketball Add Total, Avg and Projected Points to Player Object for #281
Updates stats map key for readability

Codecov Report

Merging #282 (1a05b3a) into master (f0818f5) will increase coverage by 0.09%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #282      +/-   ##
==========================================
+ Coverage   87.68%   87.78%   +0.09%
==========================================
  Files          46       47       +1
  Lines        1624     1637      +13
==========================================
+ Hits         1424     1437      +13
  Misses        200      200

Impacted Files
Coverage Δ

espn_api/basketball/box_player.py
96.29% <100.00%> (ø)

espn_api/basketball/box_score.py
89.65% <100.00%> (ø)

espn_api/basketball/constant.py
100.00% <100.00%> (ø)

espn_api/basketball/league.py
80.95% <100.00%> (ø)

espn_api/basketball/player.py
97.22% <100.00%> (+1.06%)
:arrow_up:

espn_api/basketball/team.py
97.87% <100.00%> (ø)

espn_api/requests/__init__.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Add Readme

#21 Pleas pull it

Add more contents,
Add Badges :+1:

Hey, is this issue still open to be worked on?

Yes.

Great. On it.

I have created a PR for this. See #28


Compact Players not respecting Compact Capabilities

There are two issues with buttons in Android being displayed based on configured capabilities.

https://github.com/doublesymmetry/react-native-track-player/issues/1701
https://github.com/doublesymmetry/react-native-track-player/issues/1698
https://discord.com/channels/567636850513018880/600714011318681610/1016786934888149082

Capabilities need to be re-evaluated and we should ensure that the proper controls show up corresponding to the options that are configured.
Note: we should be sure to test with both multi-track and single-track queues.

I think the problem occurs when having a single track in the queue. If you have at least 2 track at all times, the remote skip button appears correctly.

My app never has more than one track in the queue and always shows the previous track button. Can't get rid of it.

Currently investigating this. Fixed a few bugs regarding which buttons appear by default. Will continue to work on it.

I think the problem occurs when having a single track in the queue. If you have at least 2 track at all times, the remote skip button appears correctly.

@ghiculescualexandru This is default player behavior as it makes no sense for this button to exist when there is no other track to skip to.

My app never has more than one track in the queue and always shows the previous track button. Can't get rid of it.

@RamProg This will always show even if there is only a single track in the player, as skip to previous will then act as a "skip to beginning". This is default ExoPlayer behavior and not sure if it can be changed, will look into it.

Pleas see the discussion on https://github.com/doublesymmetry/react-native-track-player/issues/1708

@RamProg This is intended and will always show even if there is only a single track in the player, as skip to previous will then act as a "skip to beginning". As far as I'm aware the internal library we use doesn't allow that to change.

Then we don't have a way of preventing that button from showing up?
This was possible on previous versions. So it's not a bug, but a limitation of the library?

@RamProg yes, it's a limitation.

@mpivchev kotlinaudio 1.2.3 contains this fix right?

This is now fixed in main and will be released with 3.2 (and/or nightly, whichever comes first).


[Question]How to change GraphQL schema in runtime.

Hi.
I'm trying to use spring-graphgl in new project.
That project have  dynamic database that users can use to create new tables and columns.
Currently loading the schema defined in "schema.graphqls".
I would like to be able to change the GraphGL schema without rebooting.
I didn't have much information,I need some help, please..
Thanks.

This looks the same as #452. Short answer, you can transform the schema.

Thank you for the useful information!
I will try that.


Controller

Do I need to add something else to Controller?

Do I need to add something else to Controller?

Right now - no. We don't have Hunter entity, so We can't predict how it will behave (well, actually We can, but i don't like these mock and stub stuff)


:memo: Refactor the structure of the global hub agent

Summary
Related issue(s)
Fixes # https://issues.redhat.com/browse/ACM-14760
Tests

[ ] Unit/function tests have been added and incorporated into make unit-tests.
[x] Integration tests have been added and incorporated into make integration-test.
[ ] E2E tests have been added and incorporated into make e2e-test-all.
[ ] List other manual tests you have done.

/retest

/cc @clyang82 @ldpliu

Prow issue, retest it
INFO[2024-10-12T10:10:10Z] Ran for 0s
ERRO[2024-10-[12](https://prow.ci.openshift.org/view/gs/test-platform-results/pr-logs/pull/stolostron_multicluster-global-hub/1156/pull-ci-stolostron-multicluster-global-hub-main-test-integration/1845044250014650368#1:build-log.txt%3A12)T10:10:10Z] Some steps failed:
ERRO[2024-10-12T10:10:10Z]
  * could not initialize namespace: could not update namespace to add labels, TTLs and active annotations: Operation cannot be fulfilled on namespaces "ci-op-t6xf1z2c": the object has been modified; please apply your changes to the latest version and try again
INFO[2024-10-12T10:10:10Z] Reporting job state 'failed' with reason 'initializing_namespace'

/test test-integration

/test test-integration


How to log the response body/payload?

Is this possible?

I reviewed the source code, it should be easy to add, may I provide a PR for this?


dependency issue: org.cassandraunit:cassandra-unit:<IP_ADDRESS>

in gradle build, I have included: com.websudos:phantom-testing_2.10:1.4.0
It started complaining about:
Could not find org.cassandraunit:cassandra-unit:<IP_ADDRESS>
seems <IP_ADDRESS> is taken out form http://maven.websudos.co.uk/list/plugins-release/org/cassandraunit/cassandra-unit/ as its no more listed there.
please suggest.

Same problem occurs for me on a fresh clone of this repo and attempting to run sbt test.  In addition, if I change the dependency from cassandra-unit:<IP_ADDRESS> to cassandra-unit:<IP_ADDRESS>, then I hit another missing dependency: com.websudos:util-testing_2.11:0.5.0.  Changing that to 0.6.10 compiles successfully, but fails a bunch of unit tests.  I don't know if the failures are related to the changes or not.

@nathanalderson Thank you for reporting this. We are aware of the issue, it's to do with our custom Maven repository having a downtime. The issue has now been fixed and all the missing dependencies are being systematically re-deployed back to Maven to restore functionality.

@alexflav23, we are still facing the issue with cassandra-unit:<IP_ADDRESS>.

@kkishordn cassandra-unit is now re-published to our Maven repository and should be back and working in the phantom-testkit.

It looks like <IP_ADDRESS> is available from maven.websudos.co.uk, but phantom still depends on <IP_ADDRESS>, which is still unavailable.  Also, util-testing_2.11-0.5.0 is still not available.  Are we doing something wrong?

@nathanalderson We are working to publish a new cut of phantom with the 2 new dependencies and that should hopefully be available today.
In the mean time you can exclude cassandra-unit from the phantom-testkit dependency and manually add <IP_ADDRESS>. You can also bump the util version used to 0.6.11, which is available on our Maven repo.

@nathanalderson A Phantom 1.5.0-R2 has just been published to our Maven repo to address the issues with dependency resolution. Everything should be working fine now. Please let us know if anything goes wrong.

phantom-testing_2.11-1.5.0-R2.pom also declares the dependency
     <dependency>
        <groupId>org.cassandraunit</groupId>
        <artifactId>cassandra-unit</artifactId>
        <version><IP_ADDRESS></version>

can you check it again?


Data consistency docs

Fixes #19
Full preview URL of this page: https://turso-docs-git-fork-codingdoug-data-consistency-chiselstrike.vercel.app/reference/data-consistency

@honzasp @MarinPostma please check again - your prior concerns should be resolved.


This docker-compose file is completely broken

The .env which is being mounted does not exist.
Usage of $PWD doesn't make any sense since docker-compose can resolve relative paths.
An environment variable called $ROOT is being used and you expect a cert folder in it. In the nginx sub-folder you try to use letsencrypt. It would be smarter to not do https in your web-application but rather delegate it to something like jwilder/nginx-proxy with letsencrypt-companion
There is no need to map all exposed ports. I would recommend to only map port 80 of the web application. Within the docker-compose infrastructure all containers can access each other via network using the service-name from the docker-compose file.

Theres a bunch of stuff wrong with the docker-compose file like how the compose file is pulling a broken and outdated docker container image.  There's really no indication that the main branch of the app is stable it seems and the main developer doesn't seem to be in their own Gitter.im channel.  I PR'd a few adjustments recently.


Environment Variables are not pushed anymore

When switching from com.microsoft.azure:azure-webapp-maven-plugin:1.12.0 to com.microsoft.azure:azure-webapp-maven-plugin:1.13.0, the plugin does not set environment variables anymore.
It seems not to push both:

implicit environment variables (like DOCKER_CUSTOM_IMAGE_NAME)
explicit environment variables (as defined in appSettings)

@blacelle Thanks a lot for your report and sorry for the trouble, this is an known issue which has been fixed in the webapp-maven-plugin:1.14.0, could you please upgrade the plugin and try again?

@Flanker32 Thanks for the feedback. But it also appears in 1.14.0. I tried being more precise indicating the issue popped in 1.13.0 however I observed it initially but upgrading from 1.12 to 1.14.

@blacelle Thanks for your feedback. Could you please share me a demo project with 1.14.0? I tried to deploy a docker app with appsettings with maven plugin 1.14.0 and it seems works well.

@Flanker32 Any news on this? Thanks

@blacelle Sorry but I could not repro this issue, could you please try update to the latest maven plugin and try again? Or could you please share a demo configuration for your project?


maintain the cluster name label

Signed-off-by: Wei Liu<EMAIL_ADDRESS>

/cherry-pick release-2.4

/assign @ldpliu

refer to https://github.com/stolostron/backlog/issues/19229

/lgtm


failed to run custom build command for openssl v0.9.24

error: failed to run custom build command for openssl v0.9.24
Caused by:
process didn't exit successfully: /tmp/cargo-installS2uIYh/release/build/openssl-c4cb15e89bd781b7/build-script-build (exit code: 101)
--- stderr
thread 'main' panicked at 'Unable to detect OpenSSL version', /home/cheng/.cargo/registry/src/github.com-1ecc6299db9ec823/openssl-0.9.24/build.rs:16:14
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace
warning: build failed, waiting for other jobs to finish...
error: failed to compile semantic-rs v1.0.0, intermediate artifacts can be found at /tmp/cargo-installS2uIYh
Caused by:
build failed

libssl-dev is already the newest version (1.1.1-1ubuntu2.1~18.04.5).


Commands for convenient regex testing

create RegularExpressions cog
create !regexp search command

I'll take a more in depth look at the code soon, but one question I have now is whether executing arbitrary RegExp code is safe?
We used to have an issue on the bot where the RegExp used for code block detection resulted in the bot locking up due something called catastrophic backtracing. How can we prevent unsafe regular expressions from locking up the bot?

Found the conversation for when we discovered the issue in the bot, the solution was to adjust the regular expression to something safe but this still leaves the risk of abuse if people use dangerous regular expressions.

@jos-b Why not set a small time limit?

Right, but then we need to isolate the process which would require migrating a regex executor to another platform (which with the new UNIX commands in snekbox and possibility for more things in future is feasible) which would make this implementation obsolete.

Could they use snekbox somehow? Format the regex into a template snippet which executes it, then prints to stdout a JSON result. Then parse that in the bot, handling errors.

Example of catastrophic backtracing from StackOverflow

Snekbox could definitely be used, either by adding a new executor or by basically sending all the processing code as a regular eval to snekbox.

However at that point I think it becomes something where we need to discuss things among the core developers before we can think about implementation detail.

The regex module seems to handle such regular expressions without any issues. All examples of catastrophic regexes I could find work instantly.

The example from the StackOverflow link I posted above results in 100% CPU when tested locally:

I meant the third-party regex module:

A possible issue with regex is that it adds some extra regex features, so a regular expression might work with the command, but not in python with re, which might confuse people. A simple fix would be to parse the regex using re as well and see if it throws an error.

I've been investigating regex a bit more and have found it is still very much susceptible to DoS attacks and should not be used with arbitrary user input.
I'll attach an example:
>>> import regex
>>> regex.match("^(([a-z])+.)+[A-Z]([a-z])+$", "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa!")
... hang ...

I've been investigating regex a bit more and have found it is still very much susceptible to DoS attacks and should not be used with arbitrary user input.
I'll attach an example:
>>> import regex
>>> regex.match("^(([a-z])+.)+[A-Z]([a-z])+$", "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa!")
... hang ...

>>> regex.match("^(([a-z])+.)+[A-Z]([a-z])+$", "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa!", timeout=0.05)
>>> TimeoutError: regex timed out

Huh!

Good point, though, I'll add that as well.

Alternatively, we can use snekbox for execution, leaving DoS and other similar attacks out of the equation. Would require some effort to move the code there, though.

Now it looks like this:

TODO:

[ ] somehow show zero-width groups (right now they just don't show up)
[ ] support named groups (right now they show up as anonymous groups)

[ ] possibly move the matching/searching to snekbox


add MIRROR env to support custom nodejs dist mirror

Just like NVMW_NODEJS_ORG_MIRROR in nvmw.

+!

Duplicate of #97.


How to use the pre-trained StyleGAN2 for another architecture?

Hi I want to replace the pre-trained StyleGAN2 model ffhq.pkl with rosinality's StyleGAN2 Implementation, It seems that currently your implementation of StyleGAN2 is based on StyleGAN2-ada. These two implementations have different weights paramenters & architecture.  Besides changing the Generator network, Is there any  way for a convertion?  Thanks  for your great work!

@mazzzystar
Hi,
Unfortunately I don't think there is a simple way to do this other than changing all the generator references to use rosinality's implementation.
Perhaps you could alternatively write a script similar to this one: https://github.com/rosinality/stylegan2-pytorch/blob/master/convert_weight.py, that would perform the conversion for the weights file.

Many thanks!


"Houston, we have a problem! ..." - debugging options

Hiting "Save instance" gives: "Awesome! OK xxx hosts....*". Then when I open Icinga Multi Status I see "Houston, we have a problem! Something seems broken, sorry."
How can I debug this?

I guess "Houston, we have a problem! Something seems broken, sorry." means that something in your Icinga is in a Critical state.

Not really :) It means that plugin can't connect to Icinga. I don't remember how but I did eventually manage to run plugin but it doesn't meet my needs.


Added implementation of NMS_FT_180 i.e invalid valid of messagePerWeek

Can one of the admins verify this patch?

ok to test

ok to test


Align portfolio toolbar with title text

This will align the portfolio toolbar more accurately with the title text in the column left from it.

The portfolio content such as the toolbar is currently aligned with the top of the widget title on XL display and under the title on mobile. This is the same consistent layout used by almost every widget.

First off, thanks so much for developing the academic theme, it's been super useful, and the documentation has been amazingly helpful. As an inexperienced user, I was wondering if there is a way to center the project toolbar within any given page. Right now, I have a custom page called 'research', in which I built a page section based on the 'portfolio' widget and have all the projects in card view. However, both the toolbar and the cards appear left aligned, and I could not figure out how to center align them. Thanks for your help in advance. I am using v4.4 of the theme.


build(deps): pin ALL dependency versions in package.json files 2023-07-24

Background information on why doing this is crucial:
https://reproducible-builds.org/


Update posthog-product-analytics.json

Currently the WAU graph seems broken, instead we can use the group by week of the DAU to fix it.

Working out a way to test this now...

Closing as needed for the tests


feat(p2p): send goodbye messages on disconnecting to peers

Overview
Features:

Adds a goodbye message to reqresp such that peers can be sent a goodbye message along with a reason when they are disconnected from the peer set

Misc:

Did some refactoring of the reqresp code to make it more managable
Do avoid a circular ref when passing reqresp to the peer manager to allow it to send goodbyes, it was required
to split out the peer scoring component such that reqresp could still manage peer scores

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#10920  👈 (View in Graphite)
#10907
master

This stack of pull requests is managed by Graphite. Learn more about stacking.


fix: Switch InsecureSkipVerify to true

Checklist:

[ ] My organization is added to USERS.md.

I'm still getting
{"code":14,"message":"connection closed"}

Yeah, same. Looking into it

It works for me if I revert this:
        dialOpts := []grpc.DialOption{
                grpc.WithDefaultCallOptions(grpc.MaxCallRecvMsgSize(MaxGRPCMessageSize)),
-               grpc.WithInsecure(),
        }
+       if as.tlsConfig != nil {
+               dialOpts = append(dialOpts, grpc.WithTransportCredentials(credentials.NewTLS(as.tlsConfig)))
+       } else {
+               dialOpts = append(dialOpts, grpc.WithInsecure())
+       }
+

``

And I run this
./dist/argo server --secure --port 2345

I don't seem to need GODEBUG=x509ignoreCN=0.

@alexec I reverted #5550 and it works. I had done so in my local from an earlier test bud hadn't noticed. I just tested and it also works for me. We should be good to go.


[Package Issue]: Neovim.Neovim cannot be uninstalled

Please confirm these before moving forward

[X] I have searched for my issue and not found a work-in-progress/duplicate/resolved issue.
[X] I have not been informed if the issue is resolved in a preview version of the winget client.

Category of the issue
Installation issue.
Brief description of your issue
I have installed Neovim.Neovim with winget install neovim. When I try to uninstall it I get the following message:
No installed package found matching input criteria.

Steps to reproduce
Run winget install neovim and then winget uninstall --id Neovim.Neovim.
Actual behavior
winget outputs the following message:
No installed package found matching input criteria.

Expected behavior
Neovim.Neovim gets uninstalled.
Environment
Windows Package Manager v1.7.11261
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.22631.3672
System Architecture: X64
Package: Microsoft.DesktopAppInstaller v1.22.11261.0

Screenshots and Logs
WinGet-2024-05-31-16-32-43.295.log

Unfortunately, uninstallation isn't currently tested by the pipelines.

I have it listed as installed twice, although I only installed it once:
> winget list | findstr Neovim
Neovim                                  Neovim.Neovim                            0.9.5                 0.10.0    winget
Neovim                                  Neovim.Neovim                            0.10.0                          winget

I uninstalled the 0.9.5 manually from the menu Start > Add and remove programs.
After doing so the uninstallation through winget worked.
This is not the first app that I found installed twice on my system. What's going on with these double installations?


Extracted table wrong in tabula-java 1.0

Hey @melisabok,
I was wondering if you could take a look at this. I tried processing this PDF —mentioned in tabulapdf/tabula#656.
Lattice mode works just fine in 0.9.2, but it's completely unusable in 1.0.0. To reproduce:
java -jar target/tabula-1.0.0-SNAPSHOT-jar-with-dependencies.jar -p 1 -a 56.925,24.255,549.945,786.555 -r Publication_of_award_of_Bids_for_Transport_Sector__August_2016.pdf

Sure, I'll take a look.

From the debug tool, these are the characters' bounding boxes with 0.9.2:

…and with 1.0.0:

Might be worth taking a look at the height calculation of TextElements.

You were right, the height for some TextElements was too high. I fixed it following the recommendation of this issue.
The code that I added was in the LegacyPDFStreamEngine but for some reason I removed it, I don't remember why :(.
All the tests are passing, I had to update a json file because of height differences.
https://github.com/tabulapdf/tabula-java/pull/156

You rock. Thanks!


Migrate built-in node label list into Reference section

This is a cleanup request

Problem:
https://kubernetes.io/docs/concepts/scheduling-eviction/assign-pod-node/#built-in-node-labels is reference-type information that appears inside an already long concept page.
Proposed Solution:
Migrate https://kubernetes.io/docs/concepts/scheduling-eviction/assign-pod-node/#built-in-node-labels (the heading and the following section) to be within
https://kubernetes.io/docs/reference/
Page to Update:
https://kubernetes.io/docs/concepts/scheduling-eviction/assign-pod-node/#built-in-node-labels

/language en
/priority awaiting-more-evidence
/kind cleanup

/remove-lifecycle stale

Still seems useful to do
/remove-lifecycle stale

Also relevant: #26989

Done
/close


cctyper dies with missing files

Hi Jakob,
I was running cctyper (fresh conda install)  on a large fasta file (7.7GB) and it seems that it runs smoothly until minced step where it's trying to locate a missing file. Any idea on what might be the issue ?
Joseph.
cctyper -t 64 m64241e_210617_232502.hifi_reads.fasta cctyper.m64241e_210617_232502.hifi_reads.out
[2021-06-25 12:22:40] INFO: Running CRISPRCasTyper version 1.4.1
[2021-06-25 12:23:23] INFO: Predicting ORFs with prodigal
[2021-06-25 13:31:06] INFO: Running HMMER against Cas profiles
100%|█████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 693/693 [13:06:04<00:00, 68.06s/it]
[2021-06-26 07:21:47] INFO: Parsing HMMER output
[2021-06-26 07:21:48] INFO: Subtyping putative operons
[2021-06-26 07:28:48] INFO: Predicting CRISPR arrays with minced
Traceback (most recent call last):
  File "/projects/codon_0000/apps/miniconda3/envs/cctyper/bin/cctyper", line 86, in <module>
    crispr_obj.run_minced()
  File "/projects/codon_0000/apps/miniconda3/envs/cctyper/lib/python3.9/site-packages/cctyper/minced.py", line 79, in run_minced
    self.write_spacers()
  File "/projects/codon_0000/apps/miniconda3/envs/cctyper/lib/python3.9/site-packages/cctyper/minced.py", line 156, in write_spacers
    f = open(self.out+'spacers/{}.fa'.format(crisp.crispr), 'w')
FileNotFoundError: [Errno 2] No such file or directory: 'cctyper.m64241e_210617_232502.hifi_reads.out/spacers/m64241e_210617_232502/165939000/ccs_1.fa'

Hi Joseph
That's odd. How does your fasta headers look like?

They look like this, and that's probably these "/" that cause the issue, I'll try editing and run again :'(
>m64241e_210617_232502/20/ccs
>m64241e_210617_232502/6686011/ccs
>m64241e_210617_232502/14681679/ccs
>m64241e_210617_232502/22611689/ccs
>m64241e_210617_232502/30213142/ccs
>m64241e_210617_232502/37619981/ccs
>m64241e_210617_232502/45156585/ccs
>m64241e_210617_232502/52627283/ccs
>m64241e_210617_232502/60162622/ccs
>m64241e_210617_232502/67700160/ccs

Yeah, definitely the '/' that are messing things up.

I'm glad you like the tool! But I had some issues about minced
Traceback (most recent call last):
File "/project/hz/.conda/envs/AcaFinder/bin/cctyper", line 91, in
crispr_obj.run_minced()
File "/project/hz/.conda/envs/AcaFinder/lib/python3.8/site-packages/cctyper/minced.py", line 75, in run_minced
subprocess.run(['minced',
File "/project/hz/.conda/envs/AcaFinder/lib/python3.8/subprocess.py", line 493, in run
with Popen(*popenargs, **kwargs) as process:
File "/project/hz/.conda/envs/AcaFinder/lib/python3.8/subprocess.py", line 858, in init
self._execute_child(args, executable, preexec_fn, close_fds,
File "/project/hz/.conda/envs/AcaFinder/lib/python3.8/subprocess.py", line 1720, in _execute_child
raise child_exception_type(errno_num, err_msg, err_filename)
FileNotFoundError: [Errno 2] No such file or directory: 'minced'


flow-remove-types no longer correctly removes class properties with variance sigils

Good: everything before and including 2.119.1
$ yarn global add<EMAIL_ADDRESS>> /dev/null
$ flow-remove-types --version
v2.119.1%
$ flow-remove-types <<< '// @flow
class Foo {
  +bar: number;
}'
//
class Foo {

}

Bad: everything after 2.119.1
$ yarn global add<EMAIL_ADDRESS>> /dev/null
$ flow-remove-types --version
v2.120.1%
$ flow-remove-types <<< '// @flow
class Foo {
  +bar: number;
}'
//
class Foo {
  +bar        ;
}

Haven't tested the babel preset.

IIRC, I fixed a bug because the meaning of an uninitialized vs missing class property changed in the TC39 proposal since we implemented it. it's now supposed to leave the prop and just erase the type. but yeah, if it's leaving the variance sigil that's a bug. thanks!

That's this https://github.com/facebook/flow/commit/5f4b44e991d84cf08d44834629a9fbb09a4edbf8 then.


ISSUE with my abilities not the files

Heyo, Im a 16 y/r trying to learn more complex html. I have some experience with html but just basic. Nothing much other than making a website. I would really like some help with this and im trying to wrap my head around everything here. Any help would be awesome. If you would like to help me please add me on discord @ pineapplebegin#1989 Thanks

nigger this ain't html wtf u talkin about? :joy:

Git's issue tracker is not meant to post about your personal issues, but to discuss the issues in the repository's code.
Also I'm not giving any personal/private lessons


Upload big file but return  499 Client Closed Request

There is a question:
use postman/swagger to upload large files directly succeeded.
But when using the gateway Ocelot, it returns 499.
The gateway module is also configured with large file upload.
my NetCore is 3.1 and Ocelot is 16.0.1.
Why? How to deal with it,Tks!!!

Hi,
@zrf518 can you please explain how you have configured to accept large file upload & please us know hosting environment on which you faced this issue.
Issue https://github.com/ThreeMammals/Ocelot/issues/749 and I also tried that above 30MB file upload we get 404, so I raised PR for status code 413.

@zrf518 Are you still with Ocelot?

Outdated issue :exclamation:
@zrf518 You have to upgrade your solution to the latest Ocelot version 23.3.0, and try to reproduce the bug once again.
We believe this problem should not be in latest versions.
You have the right to reopen this issue in the future if you come with a PR ready.


NIFI-10499: Allow passing AWS region to AWS Processors via flow file attributes

Summary
NIFI-10499
I made this change in Anetac's Nifi fork months ago and we are using it in production.
Tracking
Issue Tracking

[x] Apache NiFi Jira issue created

Pull Request Tracking

[x] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-10499
[ x Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-10499

Pull Request Formatting

[x] Pull Request based on current revision of the support/nifi.1x branch
[x] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[x] Build completed using mvn clean install -P contrib-check

[x] JDK 21
[x] JDK 8

Licensing

[x] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[x] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[x] Documentation formatting appears as expected in rendered files

Reviewing...

Thanks for this proposed change @grishick, and thanks for the helpful feedback @turcsanyip.
Given that the existing implementation works with Parameter value references, and that additional changes would be required to handle FlowFile attributes, I am closing this current pull request against the support branch. Feel free to revisit this in a new pull request against the main branch that includes additional changes necessary for S3 processors.


Running with FreeSurfer v7

Hello,
I see in the recon_surf.sh pipeline that using FastSurfer with FreeSurfer v7 is not recommended. Is this still the case? If so, is the issue that FastSurfer was trained exclusively on FreeSurfer 6.0 data (as per issue #47) or is there something else?
What could be done to run FastSurfer with FreeSurfer v7?
Bests,
Lucas

Hi, yes, we noticed some reduced test-retest reliability values in FreeSurfer V7. Not sure this is still true for 7.2 (we haven't tested). This was tested and shown directly in vanilla FreeSurfer. FastSurfer inherits those problems if we switch to using the FreeSurfer 7.1 binaries for surface processing.
Not sure why you want to use FreeSurfer 7? If you want to test things you can either:

try what happens if you source FS7 instead of 6 and run the pipeline (could fail). This scenario would use FS7 binaries, but the pipeline (recon-surf) would be more similar to how things are done in FS6.
I started to adopt recon-surf to the 7.1 recon-all steps a while ago and can share that script for testing.

Anyway, short answer, FS 71 is not recommended, and 7.2 is not tested.

Hi, I'm new here and I have a similar question. And may I ask what's the difference between 6 and 7? From what I saw, 7 is a lot faster than 6.

For differences between FreeSurfer versions consult their webpages, e.g.
https://surfer.nmr.mgh.harvard.edu/fswiki/ReleaseNotes
and mailing list. We are currently exploring basing our surface pipeline on FS72 especially for high-res processing.

With #114 this has been added to the dev branch and is currently undergoing testing.

Finally we found the cause for the reduced test retest reliability. We now replace FreeSurfer's global alignment on the sphere with our own code (see #164 ). This has been tested in the dev branch. It was the last thing holding us back to upgrade our FreeSurfer dependency to 7.2 and release a new version of FastSurfer.


fix(select): change event emitted before data binding is updated

Fixes the select's change event emitting before the value binding has been updated, causing consumers that check the data-bound output to get a stale value.
Fixes #8739.

Can you advise when this fix will be deployed and what angular/material release number it will be in? I am still seeing this issue in version 6.1.0.


create erd filled with orders

Updated to now show correct tables
Fixed ordering issue
ensured db can connect via Docker and DBeaver

Looks good!


Fixes error after duplicating a node with no name.

This pull request changes "_validate_node_name" method in scene/main/node.cpp.
After duplicating a node with no name, duplicated node's name is set to "@node", because it can't be empty.
This pull request fixes #27319

Nodes without name and with a parent should be an invalid use case, so I think the bug is elsewhere.

@reduz, are you sure, that nodes with a parent and without name should be an invalid use case? I ask, because when I don't let to create nodes without a name, Godot doesn't start and there is a lot of errors.

When you add a node without a name, Godot will assign it one, this is why the use case for nodes without name added to the tree should never exist.

Superseded by #29246.


I found a way to achieve the effect of WHILE keyword

@gaurav294
just modify this file robot\running\steprunner.py, near line 33
edit this function to
    def run_steps(self, steps):
        errors = []

        cursor = 0  # the index to execute
        keywords = [ str(step).lower() for step in steps ]

        while cursor < len(steps):
            try:
                step = steps[cursor]
                if str(step).lower() == "reloop":
                    assert "exit for loop if" in keywords or "exit for loop" in keywords,\
                        "both for loop and exit condition are needed"
                    cursor = 0
                    continue
                cursor += 1
                self.run_step(step)
            except ExecutionPassed as exception:
                ......

Then the RELOOP "keyword" can jump to the first line immediately but it won't carry the next element of the range.
So we can use it like this
*** Test Cases ***
Example_Case
    ${j}    set variable  ${0}
    :for    ${n}    in range    ${1}
    \       ${j}    set variable  ${${j}+1}
    \       log many    ${j}
    \       exit for loop if      ${j}==${1000}
    \       reloop
    log many        done    ${j}

this means
j=0
while True:
    j=j+1
    print j
    if j==1000: break
print "done", j

but a bit ugly
any other good modifies?
#2721

This is interesting but not something that will be directly added to RF. Native WHILE loop support may be added after we got IF/ELSE and TRY/EXCEPT done.

Wondering if any progress was made on implementing IF/ELSE and TRY/EXCEPT ?
tnx


Annotation task cannot be finished for second annotator in extensive coverage

Steps to reproduce

create extensive coverage job for 2 annotators
As annotator 1 annotate the document and the job (job finishes successfully)
Open the job for annotator 2, annotate the document and finish the job

Actual result
"Saved successfully" message appears, but the annotation job for annotator 2 is not finished. Job status remains "in progress".

Expected result
After clicking "Finish labeling" , the annotation job is finishes and status of task changes to "Finished"

In case of annotation task can not be finished, we have to show error message in front-end

fixed


Update Shell /vars  command output

Purpose
Fixes #36319

Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[x] Added necessary tests

[x] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Are we gonna keep module-enum-decl under /dclns?

Are we gonna keep module-enum-decl under /dclns?

Yes,  module-enum-decl will be keep under /dclns


Urgent college project so hoping for your reply to be quick.

How can i add a personal event to my calendarview in background so the user can't change it & how do i add public hoildays to it.

Thank You in Advance.

How can i add a personal event to my calendarview in background so the user can't change it

you can disable paging and the top bar as well.

how do i add public hoildays to it.

You can use decorators for holidays.
Please check the documentation and sample for more information.


Broken link to Ajax Ng-table library

The script and css locations defined for ng-tables are broken, the rawgit links popup 404 error and the interface is unusable :
cdn.rawgit.com/esvit/ng-table/master/dist/ng-table.min.css
cdn.rawgit.com/esvit/ng-table/master/dist/ng-table.min.js
Please update the default template to point to cloudflare :
cdnjs.cloudflare.com/ajax/libs/ng-table/1.0.0/ng-table.min.css
cdnjs.cloudflare.com/ajax/libs/ng-table/1.0.0/ng-table.js

Please submit a pull request to update the default template to point to cloudflare. Thanks!

I was able to get the service management webapp finally configured to let me in when I ran into this issue :-(
Any status on an update to the overlay project?

This is fixed in 5.0.2

That may be but the version in this overlay is 5.0.1 and this is the repo that is referenced in the documentation.

And 5.0.2 is released the versions will change.

OK, good to know.  Thank you.


Integrate Localchain into App

Integrate localchain library
(should be a separate story in localchain repo) update library to support NFT types
Mint a sample NFT per user

Closing as already implemented.


SQL injection is possible

Using fmt.Sprintf from a user input data:
https://github.com/nuveo/prest/blob/master/adapters/postgres/postgres.go#L58
and then calling postgres.Query:
https://github.com/nuveo/prest/blob/master/controllers/databases.go#L25
is a very bad practice that allows to user make SQL injections.

@pixelbender  thanks for reporting. Do you want to send a pull request fixing the issue?

Hello guys, I'll take care of this problem.
Suggestions are welcome.

@pixelbender thanks for reporting this bug, we at @Nuveo wasn't thinking that initially because it would use the pREST in private network (no public access), we will put priority on this bug.


Test gardener/gardener master branch with Go 1.20 image

/kind enhancement
What this PR does / why we need it:
This PR updates test jobs for gardener/gardener to golang-test:1.20
Special notes for your reviewer:
/hold
Until https://github.com/gardener/gardener/pull/7553 is approved.

/approve
/lgtm

/lgtm

/hold cancel

/hold
https://github.com/gardener/gardener/pull/7553 is not approved yet. Merging it will cause test to fail for other PRs.

/unhold It is approved

Little latency in https://github.com/gardener/gardener/pull/7553 approved mail 😃 .


Complete the homepage UI of nodes

Not every node has an image and we want the image, if it exists, to show up below the content (text).
We need to have larger nodes, consequently, in fewer columns. This is necessary, especially because some node titles are long.
We need to add more features about each node:

nodeType
corrects (# of upvotes)
wrongs (# of upvotes)
contributors (much smaller than the node page, maybe overlapping MUI avatars),
tags
We may exclude institutions and citations (references) from this page.

We need to be able to filter by:

tags
contributors
institutions (of the contributors) ** This is especially important
updatedAt

We need to be able to sort by the following features both ascendingly and descendingly:

updatedAt
correct - wrongs (upvotes minus downvotes)

[IMPORTANT]: We need a URL for each Filter and Sort; e.g., we should be able to share all the contributed nodes by students/researchers from the University of Michigan in one page. However, we should not refresh the page every time they change the filter or sort, similar to how Google and other search engines do this. For this purpose, we can use URL rewrites or a method like what I used in the homepage that when you scroll throughout the page, it changes the URL without refreshing the page: https://1cademy.us/home

In homepage, please:

Remove the divider line below the title
When the node is not expanded, there is no reason to show the divider line below the node. That divider should only show up if the node is expanded.
Margins of all sections of each node should be aligned. Currently, the node type margin-left differs from the rest of the node.
Based on different screen widths, the number of node columns should differ. The page is not responsive now.
We need long-descriptive tooltips for the number of votes. Change the tooltip for upvote to "___ people found this node helpful and voted to prevent further changes. To vote, you should create an account." Change the tooltip for downvote to "___ people found this node unhelpful and voted to delete it. To vote, you should create an account."

In node pages, please:

When we load the page, CSS takes along time to load. That's why we see the big icons when the page gets loaded, then it adds CSS. We definitely need to fix this.
Move the node image, if it exists, below the node content. (This was correct in my previous version. I don't know why you changed it.)
Fix the width and height of the node image. Its width should cover all the space and its height should be determined proportionately. (This was correct in my previous version. I don't know why you changed it.)
When hovering on the node image, the tooltip should show up. (This was correct in my previous version. I don't know why you changed it.)
When clicking the node image, the full-screen image should show up. (This was correct in my previous version. I don't know why you changed it.)
Remove the divided line after the node image.
Margins of all sections of the node column should be aligned. Currently, the node type margin-left differs from the rest of the node.
When there are multiple tags or references, add divider lines between them.
The node title should be selectable, not clickable. The mouse should be a cursor not a pointer.
In the page footer, links should not be in orange.


Reference correct script in README

The publishing section of the README refers to a build-android script which seems to have changed to yarn build-skia-android at some point

Thanks, @cjhines - would you try to sign the CLA so that we can approve?

@cjhines - still getting the following error when running the CLA test:
Error: In order to merge this pull request, all contributors must sign [Shopify’s CLA](https://cla.shopify.com/).

- `cjhines<EMAIL_ADDRESS>Connect your email address with a GitHub account and [sign the CLA](https://cla.shopify.com/) to have your PR reviewed.


Rejecting all MLIDs associated w/ certain column values

After chatting w/ Emilie, I think we can reject all MLIDs associated with any rejection. Should be able to do so in one go after the rest of the column based site review steps.


Also test with deletions

Deletions/updates are common in real search applications -- can we compare the two engines with varying %tg of deletions?

Good idea, I think we can assign the same IDs for both engines and at the end of indexing, randomly deletes enough documents keyed by IDs


[Snyk] Upgrade @types/react from 17.0.45 to 18.0.8

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade @types/react from 17.0.45 to 18.0.8.

:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

Warning: This is a major version upgrade, and may be a breaking change.

The recommended version is 9 versions ahead of your current version.
The recommended version was released 22 days ago, on 2022-04-26.

Release notes

Package name: @types/react

18.0.8 - 2022-04-26

18.0.7 - 2022-04-25

18.0.6 - 2022-04-21

18.0.5 - 2022-04-14

18.0.4 - 2022-04-13

18.0.3 - 2022-04-12

18.0.2 - 2022-04-12

18.0.1 - 2022-04-09

18.0.0 - 2022-04-07

17.0.45 - 2022-05-06

    from @types/react GitHub release notes

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Codecov Report

Merging #35 (d5a4726) into master (819a325) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master       #35   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            2         2
  Lines           22        22
  Branches         3         3
=========================================
  Hits            22        22

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


Namespace option ignored in some cases

Currently the -n --namespace option seems to be ignored in the use-cases which I have.
My main use-case is that I want to be able to:
$ kubecfg diff dev/myproject.jsonnet
$ kubecfg update dev/myproject.jsonnet

to view changes, then update my dev env - which works fine as the jsonnet doesn't define namespace explicitly so it's using my own from the current context. But I'd then like to:
$ kubecfg diff ci/myproject.jsonnet -n ci-namespace

to verify the changes when using the ci jsonnet. But this just diffs against my own namespace (similar if I kubecfg update -n nonexistentnamespace dev/myproject.jsonnet it doesn't error but deploys using my namespace)
Currently the only way I can achieve this is by explicitly including the namespace metadata in the jsonnet, then call with:
$ kubecfg diff ci/myproject.jsonnet -V namespace=ci-namespace

or similar, but this then requires the same ext var for anyone deploying on dev.

Let's close this issue and move to #103, since I did an independent analysis of the cause there.

Reopening, since some of the details here are diff-specific (namespace in local objects) and separate to #103.

I reproduced in 0.5.0 but I cannot reproduce in 0.6.0; it might be (accidentally?) fixed.


PE-4253: add error handling on login page

Addresses common errors on login page

Closing as it won't be implemented now.


After the last update, an error with local database appeared

System information:

Windows 10 64 bits
DBeaver 6.1.5

Connection specification:

Database name and version: Mysql 5.6
Driver name:
Do you use tunnels or proxies (SSH, SOCKS, etc)?  No

Describe the problem you're observing:
After this new update (6.1.5) this message appears when I try to use my local database:
"The server time zone value 'Hora Padr�o de Buenos Aires' is unrecognized or represents more than one time zone. You must configure either the server or JDBC driver (via the serverTimezone configuration property) to use a more specifc time zone value if you want to utilize time zone support."
Steps to reproduce, if exist:
Just trying to connect my local database
Include any warning/errors/backtraces from the logs

Duplicate of #6553
This bug was fixed in 6.1.5 hotfix version.
You can just re-download 6.1.5.