cone object detection pipeline in photonvision

create an object detection pipeline in photon vision and tune it for yellow cones
save it to settings of photon vision in raspberry pi
export setting and save in new directory inside of the root

closed with this merge https://github.com/DoF-6413/chargedUp/pull/35


[3317] - Add V2 API user notifications create endpoint

Context
In order to allow users to opt in and opt out of receiving course update notifications we need to add some new end points to V2.
Changes proposed in this pull request

Endpoint added for creating a user notification - POST /api/v2/users/:user_id/notifications

Accepted parameters:

user_id (str)
course_create (bool)
course_update (bool)

Guidance to review
Checklist

[x] Make sure all information from the Trello card is in here
[x] Attach to Trello card
[x] Rebased master
[x] Cleaned commit history
[x] Tested by running locally

 [S] API - Create Notifications endpoint

Didn't we decide we were not going to update in the create and that we were going to create one Notification per accredited body rather than one per provider that has the accredited body as its accredited body?

Didn't we decide we were not going to update in the create and that we were going to create one Notification per accredited body rather than one per provider that has the accredited body as its accredited body?

I'm not sure I understand, this change will create a user_notification row for every accredited body that the current user is associated with. At the moment we're doing this as a master switch, so when the endpoint gets hit, it turns on all possible notifications for that user!


Camera laptop not recognized by Skype with flatpack package

Describe the bug

Skype currently do not detect my laptop camera, though it detect my laptop mic !
To Reproduce

Steps to reproduce the behavior:
login to Skype account & try to launch camera. You will see "no device detected". The camera icon is inactive. This also happening if you try to test camera. Please look at my mouse pointer within attached screenshot bellow.
Expected behavior

It should detect camera.
Screenshots

Desktop:

OS: [Fedora Linux X64 bit, version 30, Cinnamon edition]
Ferdi Version [e.g. 5.4.3 X64 bit]

Server Used: [I'm using Ferdi without Ferdi account]

Debug information: https://debug.getferdi.com/...
Additional context

I'm installing Ferdi as a flatpak package from FlatHub.

Same problem.
Desktop

OS : LinuxMint 19 Cinnamon

Ferdi 5.4.3

Server Used: [I'm using Ferdi without Ferdi account]

I'm installing Ferdi as a flatpak package from FlatHub.

Same with WhatsApp

@lhw could it be related to https://docs.flatpak.org/en/latest/sandbox-permissions.html#device-access?

Yes I already noticed this too. Will fix it asap

Had time to test it. Works with the version that will be up in a few minutes to hours. Depending on how fast the build servers are.

@lhw Thank you! 🙏

@lhw
Thank you very much!
Only one thing: does this fix will make camera available on other services like WhatsApp & ICQ ?
In Skype there is already icon for video call. But with WhatsApp & ICQ there is no such icon ?

What I noticed is that some webapps require the chrome built-in "allow access to webcam" to function. I tried with Discord and it just auto accepted it in the background. But I am not sure how WhatsApp handles this. Maybe @kytwb knows more about this. The camera is now generally available to all applications in Ferdi so that is not the blocker.

@Nokia808 are you missing the video call icons in the latest flatpak version?

@kytwb
@lhw
Hi all. I just received new update few minutes ago.

It is very perfect regarding Skype, so this issue is fixed really.

but other services like: WhatsApp, Telegram, & ICQ, the only change is that before this fix I was unable to use camera to take a picture by it & send this picture as attachment through the service like WhatsApp, but after this update now I can use my laptop camera through WhatsApp JUST to take picture then send it as attachment. BUT:

I can not make audio / video calls through WhatsApp, Telegram, ICQ. The question is that: is this a bug or the matter is just that the web applications of these services do not support audio / video calls & they infact support audio / video message as attachment ?? When I use Telegram official application as flatpak, there is icon for audio calls, but when I open Telegram from Ferdi there are only option (icon) to send audio messages (recording) & other for sending a media (as attachment) audio or video file (not recorded from within Telegram).
If this (point 3) is a bug, then inform me to open a new bug.

@Nokia808 awesome! Regarding the rest, I think it's the latter; I also can't do video calls in the web version of WhatsApp. Regarding Telegram, I suspect the same as the current web app used is very limited; we are thinking of changing it for a web app with more features via https://github.com/getferdi/recipes/issues/56. Not sure for ICQ.

I've a similar issue with snap version (Ferdi 5.6.3) although I give the package a full permissions as shown below:
$ snap connections ferdi
Interface                 Plug                   Slot                             Notes
audio-playback            ferdi:audio-playback   :audio-playback                  -
browser-support           ferdi:browser-support  :browser-support                 -
content[gnome-3-28-1804]  ferdi:gnome-3-28-1804  gnome-3-28-1804:gnome-3-28-1804  -
content[gtk-3-themes]     ferdi:gtk-3-themes     gtk-common-themes:gtk-3-themes   -
content[icon-themes]      ferdi:icon-themes      gtk-common-themes:icon-themes    -
content[sound-themes]     ferdi:sound-themes     gtk-common-themes:sound-themes   -
desktop                   ferdi:desktop          :desktop                         -
desktop-legacy            ferdi:desktop-legacy   :desktop-legacy                  -
gsettings                 ferdi:gsettings        :gsettings                       -
home                      ferdi:home             :home                            -
network                   ferdi:network          :network                         -
opengl                    ferdi:opengl           :opengl                          -
pulseaudio                ferdi:pulseaudio       :pulseaudio                      manual
unity7                    ferdi:unity7           :unity7                          -
wayland                   ferdi:wayland          :wayland                         -
x11                       ferdi:x11              :x11                             -
$

It seems this issue occurs in snap disrto while it doesn't occur in appimage distro


Handle Trusted Advisor entitlement issue gracefully

Handle Trusted Advisor entitlement issue gracefully. Resolves https://github.com/toniblyx/prowler/issues/414
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@toniblyx I've raised a new pull request for Prowler

Thanks!


Add BANFrance Geocoder

Hi there,
A Geocoder service from the  adresse.data.gouv.fr
It does not require any Api_Key nore registration.
cheers...
sheub

Hi @KostyaEsmukov,
Thanks for the review, I did my best to make the corrections on time. Hope it's better now,
cheers,
Sheub


Add SpaceRanger processing to pipeline

This PR adds a SpaceRanger subworkflow to the pipeline, as well as several other additions, including container directives and labels for most modules as well as separation of single cell parts from the spatial parts. The pipeline can work with either SpaceRanger input (raw FASTQs, images and probesets) or the previous default with SpaceRanger output. More testing is needed to make sure that the SpaceRanger subworkflow works with real data and capture any potential errors.

@nf-core-bot fix linting

How far are we with this @fasterius ?

I'm a bit new to these linting things, so I'm probably missing something, but the errors I'm seeing are either (1) some error in the linters themselves (they doesn't seem to run, but I might be wrong), or (2) Wrong amount of left-padding spaces(want multiple of 4) in EditorConfig. I've successfully run the linters locally and added changes accordingly, but this didn't affect anything in terms of checks online. I'm guessing I'm missing something - any ideas, @apeltzer ?

Likely the pipeline code is out of sync with latest linting / nf-core template 2.6 -- we should merge this one: https://github.com/nf-core/spatialtranscriptomics/pull/21 to spaceranger (changed the base branch now to see where the changes are). I might help on that, but would need to see when I find time for it :-(

Okay, I'll get started on that! (I've been away on parental leave, hence the lack of activity lately)

Linting finally works! Would it be possible to get a review @cavenel so that we might merge it into dev?

The last missing test is somethign else, will merge this to dev now. You may have to merge in the latest template to get a running lint again on dev though.


Update typing rewrites for typing_extensions 4.6.0+

As discussed in https://github.com/asottile/pyupgrade/issues/842.
typing_extensions 4.6.0 backported many bugfixes and enhancements from newer Python versions:

Several bugfixes for Literal that were fixed in 3.10 but no earlier
Pickleable newtypes from 3.10
NamedTuples and TypedDicts with correct __orig_bases__ attributes from 3.12
Much more performant runtime-checkable protocols from 3.12
A better repr for Unpack, from 3.12
A new parameter for dataclass_transform, from 3.12

Thanks!


CB-4366 Nginx should allow underscores in header names

Nginx by default prevents underscores in header names.
The HTTP spec allows underscores in headers. This change
configures Nginx to allow underscores in headers. It needs
to be configured at the top level since there are multiple
Nginx server config blocks.
http://nginx.org/en/docs/http/ngx_http_core_module.html#underscores_in_headers
Some additional background on underscores in headers can
be found here:
https://stackoverflow.com/questions/22856136/why-http-servers-forbid-underscores-in-http-header-names
Signed-off-by: Kevin Risden<EMAIL_ADDRESS>

This went into master so the first set of images will come with rc-2.16


Remove appcast from money

Link goes to 404

Thank you for the contribution. It was merged directly as c1b8c0dca73183881eb40a9fb416167f373bed73 to keep commit history cleaner. Your contribution is still credited to you.


Add in a new View for the club user that will allow them to apply for an event.

DoD: The user from the club should be able to click on a tab to then open the view that will allow them to submit to the admin their event for approval/rejection. They should have the same categories as the Event Response page since this will be going from here to there.

Added in a new view that will allow the moderator (club event planner) to now apply for an event.


TypeError: expected str, bytes or os.PathLike object, not NoneType

MacOS 12.7.6
Python 3.12
How to fix this?
python has already been granted location services

Hi there, this is a compatibility issue listed in the project README.
For more information, please refer to issue #19.


my bot doesn't answer distest bot, but me

Hi there,
When I try to use distest, i get the message, my bot is present, but not online.  In my discord app I see it online and it is answering my commands. But my bot does not answer the distest bot.  Did I miss something?
Thanks,
Martin

I figured it out. My solution was to inherit the discord.py bot:
class MyBot(commands.Bot):
    async def process_commands(self, message):

        """
        By default the bot does not forward messages to commands if
        these come from a bot.

        Here we override this behaviour and give the app a config switch
        to make it possible to test it by a bot.
        """
        if not Config().answer_bots:
            return super.process_commands(message)

        ctx = await self.get_context(message)
        await self.invoke(ctx)

I figured it out. My solution was to inherit the discord.py bot:
class MyBot(commands.Bot):
    async def process_commands(self, message):

        """
        By default the bot does not forward messages to commands if
        these come from a bot.

        Here we override this behaviour and give the app a config switch
        to make it possible to test it by a bot.
        """
        if not Config().answer_bots:
            return super.process_commands(message)

        ctx = await self.get_context(message)
        await self.invoke(ctx)

Cool! I'll be making a patcher function that does this for you soon, it's in the works right now

Cool! I'll be making a patcher function that does this for you soon, it's in the works right now


App crashes when no devices are marked as active.

After an inactivity period, devices will no longer be marked as active. When trying to act upon a device (adding a track to the player queue for example), a 404 will be returned and the error isn't handled properly and the app crashes.
Catching the error when adding to the queue with the spotify API will solve this.
Should relay the error to the client so the end user can take action to rectify. Do this because as far as I know there is no way to automatically get an active device.

#17


Test if kernels are signed before setting dev_boot_signed_only=1

This has been discussed extensively in #765.

The verification has been modified a little: We check all bootable kernels (instead of hardcoding KERN-A and KERN-B). This should play better with Chrubuntu.
We start the kernel verification process in parallel with bootstrapping.
After boostrapping is done, we check the status, and enable dev_boot_signed_only if required.
We inform the user of the modification in a trap, so it's displayed in all cases.
Warnings regarding USB/legacy booting are only displayed on success (it's less critical to display them).

I also provide a test, and the function is put in installer/functions, before ###, so it's not included in host|chroot-bin files.

Any news on when or if this will be merged? When just changing the kernel boot command line we also need to set dev_boot_signed_only=0, however we don't really need to set other boot options (usb and/or legacy).
Current workaround for me seems to set other boot options, to at least prevent  crouton installer setting dev_boot_signed_only=1.

Haven't heard anyone hitting this in a long time.  @divx118, do you still see value in such a patch?  Are we not seeing any bug reports just because your scripts/instructions anticipate and avoid the issue?

@dnschneid I indeed avoid the issue in my script https://github.com/divx118/crouton-packages/blob/master/change-kernel-flags#L221 So it isn't really an urgent problem for me. IIRC It takes some time to check if the kernels signed or not.
IMHO in the long run this should be fixed somehow, because it can leave you with an non bootable system when you just set the flag and don't check the kernels. People are not obligated to use my script, they may as well change kernel flags and don't notice that crouton will  set dev_boot_signed_only.

Going to close this for now.  We can re-open if this somehow resurges as an issue for people.


How to train on custom datasets?

can some one point out how we can train on custom datasets ?

The model accepts as input (i) a vector of floating point values corresponding to continuous features and (ii) a set of indices corresponding to categorical features, and produces as an output a probability of the associated event. If your dataset can be distilled into this input and output you can try interfacing it with DLRM. As an example, you can review the existing interface.

lets say we want to build recommendation system of CTR on E- commerce , where we have 3 users (u1,u2,u3) and 4 items (i1,i2,i3,i4) and say these were the transactions
u1 clicked i1
u1 did not click i2
u2 clicked i3
u3 clicked i4
u3 did not click i2 even after looking at the item
can you explain how do we encode for  X , LS_o and LS_i , T

You will have to find a way to transform your data into the model input. You will find a lot of scientific literature about this topic in KDD. That's probably the best place to look for an answer to this type of question.


the java framework discovery classes can be changed to stateless and static services

currently the java framework discovery classes (MicronautFramework,GrpcFramework,SpringBootFramework,AbstractJaxrsFramework)
are members of JavaLanguageService, they need a lateInit because they require JavaPsiFacade search methods, but indexes are not ready when JavaLanguageService is constructed.
a late Init is confusing and error prone.
these classes can be changed to static services without any state or initialization. when they require an annotation or class then search for it in place when required.  the search facilities are very fast and anyway we do all discovery in the background.
this will avoid the need to keep references to PSI elements in these classes which is discouraged by jetbrains.
and also avoid the lateInit methods.
see for example JavaSpanDiscoveryUtils and JavaCodeObjectDiscovery are static method and stateless, everything needed for the discovery is acquires in place when needed. so there is no need for any late init processes or keeping references to PSI elements.
if there is a need for long initialization of any of these classes then it should be a project service that is called first time when first needed to initialize itself in smart mode when indexes are ready. or it can be instantiated in a startup activity in a ReadAction in smart mode.
Minimizing read access periods:
we need to minimize the periods that we hold a read access, in JavaEndpointNavigationProvider its not easy because of the way the framework classes are designed, it will require too much duplicate code to implement it for each framework discovery class.
it will probably be easier to change the design in a way that will make it easier to reduce read access time.
they need to be more like JavaSpanNavigationProvider.

@shaykeren
the impact of this issue is if we get a ProcessCanceledException the endpoint links will not work.

will resolve #1265


Apptentive build broken in Cocoapods 1.0.0-beta6

Not sure if this is an Apptentive SDK issue or a Cocoapods issue but wanted to give you heads up so you can investigate.  Tried upgrading to Cocoapods 1.0.0-beta6 today and everything worked fine except for the trying to copy the ApptentiveResources.bundle.  I got an error saying that the bundle could not be found.  I noticed in the apptentive-ios-ApptentiveResources target that the Per-configuration Build Products Path ended with .../Build/Products/Debug-iphonesimulator/apptentive-ios/apptentive-ios.  Note the two apptentive-ios dirs.  There should just be one.

Same issue any solution?

We'll be including a fix in our next release.

Update: CocoaPods 1.0.0 beta 8 includes a fix for this issue. Please update CocoaPods and give it another try.


Calculate retirement capacity from start of period in multi-period

Add as the Min_Retired_Cap_MW (plus Energy/Charge_Cap) columns. Need to know the minimum required capacity to retire within each period based on planned retirements and age-based.

Should be included in 0.6.2


EACCES: permission denied, open '/Users/{user}/.amplify/logs/amplify-cli-2022-10-17.log'

Before opening, please confirm:

[X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
16.13.1
Amplify CLI Version
10.2.3
What operating system are you using?
Mac OS 12.6
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
No
Amplify Categories
hosting
Amplify Commands
configure
Describe the bug
Get the following error: EACCES: permission denied, open '/Users/{user"/.amplify/logs/amplify-cli-2022-10-17.log'
It will not connect to AWS through terminal. Amplify is installed and functional, but cannot create access to aws for some reason.
Expected behavior
I expected it to work.
Reproduction steps
amplify configure
GraphQL schema(s)

# Put schemas below this line

Project Identifier
No response
Log output

# Put your logs below this line

Additional information
No response

Hey @borisgotz :wave: thanks for raising this! Out of curiosity how was the CLI installed? npm i -g @aws-amplify/cli?

without sudo i got the following:
[image: Screen Shot 2022-10-17 at 12.55.46 PM.png]
Most of the solutions online have been to just use sudo
On Mon, Oct 17, 2022 at 10:30 AM josef @.***> wrote:

Hey @borisgotz https://github.com/borisgotz do you also experience this
when installing the CLI without sudo? Typically it is not recommended to
install packages from npm with sudo. If you are experiencing a permissions
issue installing global npm packages without sudo please refer to this
npm doc
https://docs.npmjs.com/resolving-eacces-permissions-errors-when-installing-packages-globally
—
Reply to this email directly, view it on GitHub
https://github.com/aws-amplify/amplify-cli/issues/11193#issuecomment-1281058200,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHSOWTIDKAI64WP4IWVWNX3WDVWJBANCNFSM6AAAAAARHAMKTU
.
You are receiving this because you were mentioned.Message ID:
@.***>

I installed node using nvm, and now the errors are gone! Thanks!
On Mon, Oct 17, 2022 at 12:56 PM Boris Gotz @.***> wrote:

without sudo i got the following:
[image: Screen Shot 2022-10-17 at 12.55.46 PM.png]
Most of the solutions online have been to just use sudo
On Mon, Oct 17, 2022 at 10:30 AM josef @.***> wrote:

Hey @borisgotz https://github.com/borisgotz do you also experience
this when installing the CLI without sudo? Typically it is not
recommended to install packages from npm with sudo. If you are experiencing
a permissions issue installing global npm packages without sudo please
refer to this npm doc
https://docs.npmjs.com/resolving-eacces-permissions-errors-when-installing-packages-globally
—
Reply to this email directly, view it on GitHub
https://github.com/aws-amplify/amplify-cli/issues/11193#issuecomment-1281058200,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHSOWTIDKAI64WP4IWVWNX3WDVWJBANCNFSM6AAAAAARHAMKTU
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hey @borisgotz that screenshot did not come across from the email reply, but I'm glad to hear you're back up and running! Do you have a link to the tutorial you were referring to? Perhaps we can get this updated 🙂

I am also running into this issue when I do amplify pull. But adding sudo solves the issue. However, I think adding sudo changed my xcode project.pbxproj file permission, which led to locking XCode project, which looks like this. The solution in the stock overflow post works, but have to do it everytime after sudo amplify pull, which feels pretty redundant.

Closing due to inactivity. @ZeeLivermorium if you are still experiencing this issue please file a separate issue


Backport raise errors to 1.4 Stable

The problem occurs when the identifier service is unreachable and we are passing the :raise_errors option set to false.
The fetch_token method throws NoMethodError while trying to perform the following code:
response.parsed.merge(access_token_opts)
This happens because the response.parsed is nil.
Backported from #460
Closes #459

Hello @pboling, sorry for the 9 months delay, but completely forgotten about the promised backport to 1.4 Stable, if you have any spare time to look into this it would be greatly appreciated 🙏

Pull Request Test Coverage Report for Build 1052

0 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
6 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-1.8%) to 95.879%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/oauth2/client.rb
0
1
0.0%

Files with Coverage Reduction
New Missed Lines
%

lib/oauth2/client.rb
1
92.08%

lib/oauth2/strategy/assertion.rb
1
77.78%

lib/oauth2/access_token.rb
4
90.0%

Totals

Change from base Build 1044:
-1.8%

Covered Lines:
349

Relevant Lines:
364

💛  - Coveralls

@joaolrpaulo Thank you!!!

Ping @joaolrpaulo ! Did you see my comment ^

Ping @joaolrpaulo ! Did you see my comment ^

Hey @pboling unfornatelly no! 😞
I will take care of that really soon

Hey @pboling unfornatelly no! 😞
I will take care of that really soon

I am closing this PR since after further investigation this has already been fixed in #518, the original problem of calling the method merge in a String variable was fixed.
Sorry for the long delays here!
And really thanks for you time @pboling!


Unable to build librtlpower.so for arm/Raspberry Pi

I am trying to build spektrum under PiAaware 4.0 (5.4.51-v7+ armv7l).  But librtlpower won't build properly.
Source is your fork of rtl-sdr.  Two separate problems arise while following https://osmocom.org/projects/rtl-sdr/wiki.

The autoreconf method fails to compile librtlpower at all.

make[2]: Entering directory '/home/pi/rtl-sdr/src'
  CCLD     rtl_power
/usr/bin/ld: rtl_power.o: in function `scanner':
/home/pi/rtl-sdr/src/rtl_power.c:245: undefined reference to `scan_tune'
/usr/bin/ld: rtl_power.o: in function `init_misc':
/home/pi/rtl-sdr/src/rtl_power.c:282: undefined reference to `rectangle'
/usr/bin/ld: rtl_power.o: in function `main':
/home/pi/rtl-sdr/src/rtl_power.c:340: undefined reference to `rectangle'
/usr/bin/ld: /home/pi/rtl-sdr/src/rtl_power.c:340: undefined reference to `hamming'
/usr/bin/ld: /home/pi/rtl-sdr/src/rtl_power.c:340: undefined reference to `blackman'
/usr/bin/ld: /home/pi/rtl-sdr/src/rtl_power.c:340: undefined reference to `blackman_harris'
/usr/bin/ld: /home/pi/rtl-sdr/src/rtl_power.c:340: undefined reference to `hann_poisson'
/usr/bin/ld: /home/pi/rtl-sdr/src/rtl_power.c:340: undefined reference to `youssef'
/usr/bin/ld: /home/pi/rtl-sdr/src/rtl_power.c:340: undefined reference to `kaiser'
/usr/bin/ld: /home/pi/rtl-sdr/src/rtl_power.c:340: undefined reference to `bartlett'
/usr/bin/ld: /home/pi/rtl-sdr/src/rtl_power.c:410: undefined reference to `frequency_range'
/usr/bin/ld: /home/pi/rtl-sdr/src/rtl_power.c:491: undefined reference to `generate_sine_tables'
/usr/bin/ld: /home/pi/rtl-sdr/src/rtl_power.c:319: undefined reference to `frequency_range'
collect2: error: ld returned 1 exit status

The cmake method completes build without error.  But after make install, only librtlsdr.so is installed into /usr/local/lib/.  Even though a file librtlpower.so is produced in build/src/ directory, processing ends in segmentation fault if that file is substituted as shared library.  Apparently build/src/ librtlpower.so is a PIE executable.

pi@piaware:~/rtl-sdr $ file build/src/librtlpower.so.0.5git
build/src/librtlpower.so.0.5git: ELF 32-bit LSB pie executable, ARM, EABI5 version 1 (SYSV), dynamically linked, BuildID[sha1]=20a4989056d206fe71c990d2de5208d604a0bc42, not stripped

Let's track everything pi related on one place - i think there will be more problems on the way to get this running on pi
https://github.com/pavels/spektrum/issues/39


Question about Project Settings and pyrightconfig.json

Hi!
Is it possible to use project specific settings in conjunction with pyrightconfig.json settings?
My setup is a bit convoluted:
I have a sub-project within the folder structure of my main project.
The sub-project folder structure starts from the same location as the main project as I have dependencies through out the main project that I want to be able to work with in the sub-project.
This means the pyrightconfig.json in the root of the main project defines the pyright settings for the sub-project.
To work around this, I am trying to add extraPaths to the project settings but the imports in sub-project are not being resolved unless I add them to the main project's pyrightconfig.json.
My structure is like so:

Root

pyrightconfig.json
MainProjectRoot

SubProjectRoot

Project Settings

Main Project pyrightconfig.json:
{
    "pythonPlatform": "Windows",
    "pythonVersion": "3.7",
    "extraPaths": [
        "<MainProjectRoot>",
        "libs/_all_versions/Lib/site-packages",
        "libs/37/Lib/site-packages"
    ],
    "ignore": ["./*.pyi"],
    "stubPath": "./../../../.projects/.stubs",
    "executionEnvironments": [
        {
            "root": "<MainProjectRoot>"
        },
        {
            "root": "<MainProjectRoot>/<SubProjectRoot>",
            "pythonPlatform": "Windows",
            "pythonVersion": "3.9"
        }
    ]
}

And my project settings are:
{
"settings": {
      "LSP": {
         "LSP-pyright": {
            "settings": {
                "python.analysis.extraPaths": [
                   "<absolutePathToSubProjectDependencyDirectory>"
                ]
            }
         }
      }
   }
}

Any advice would be appreciated!

I'm not sure if you are confused about this or not but the location of the project file (which I believe you are calling "project settings" which is not entirely precise) is not really relevant. The settings in project file apply to folders defined by the folders array in that file. The location of that file doesn't matter.
That said, I don't think I fully understand your issue. Maybe it would be more clear to me if you would specify what do folders in project file point to.

yours sounds more like a job of pyright rather than lsp-pyright.

Many servers choose to limit lookup of configuration files to the root folder of the opened project. And there are often good reasons for that. For example check https://github.com/microsoft/pyright/issues/1756

Yes, non-root config files are ultimately something for pyright to support, if they can be persuaded to do so, but I doubt it. Some linters like eslint do it by default, many others don't.
Having per-directory configs is a very useful pattern for a lot of reasons, it's a shame some tools decide to not support it.

This can work if configuration options apply only to the currently checked file. If some of the options are global in nature and apply to the whole project then it becomes undefined which configuration applies when. Or you end up with options that don't do anything and it's not obvious. It's just hard to support and in many cases doesn't make sense, depending on the architecture of the server.

If some of the options are global in nature and apply to the whole project then it becomes undefined which configuration applies when

In case of multiple files, the nearest (e.g. more specific) config file could outweight the others. Of course, care needs to be taken in how the options are merged together (e.g. in some cases it makes sense to combine, in others to replace), but I think this is a solveable problem in general.
For my use case specifically, it's not even multiple config files. I have projects in a subfolder whose parent folder is added to the Sublime's sidebar. I don't use workspaces or any of Sublime's config files at all as I want to be editor-agnostic. I'd like either LSP-pyright or pyright to find the closest config file. Currently this is only possible if I drag each individual project folder into ST, but that is cumbersome if you have hundrets of projects you switch between often.

My situation is a bit more unique I think, where I have a project within a project.
The sub-project can be a standalone package or a sub-package of the main project. It also has dependencies on packages in the main project.
So when I open the main project I want pyright to treat the sub-project as a package of the main project, which it does.
But when I open the sub-project I want it to open as if it is a top level project, but as they both share a root path the sub-project opens with the pyright config of the main project. I wanted to override this behaviour in the sub-projects sublime settings but it appears to have no effect.
Sorry - quite hard to explain 😞

Oh and nested pyrightconfigs would be great if supported - but obviously that is for pyright to sort not you guys

Since this is something that can't be addressed by this package, there is no reason to keep it open.
As for your setup, I feel that it's unnecessarily complicated. You could probably have those two projects completely separate and then add both as individual folders into the sidebar / project. And then to make Python resolve paths correctly, you could extend python.analysis.extraPaths in either of those or in project settings.


ControllerActivator should be able to use controllers registered as

services

Added WithControllersFromServiceProvider that replaces the default
controller activator with a service based one.
Move activation to DefaultControllerFactory
Modify [Activate] behavior so that it no longer activates services. Use
[FromService] attribute to hydrate services

Fixes #1707

Updated.

Updated.

:watch:

:watch:

Updated.

:shipit:

:shipit:


I know this will be considered a blog and not posted but I just wanted to wish everyone a safe and happy holiday. T...


Add Laravel Valet Extension

Description
Add "Laravel Valet" extension that you can use to search for local Valet sites and open the site in the browser, terminal, or finder.
Requirements
Laravel Valet must be installed and configured for the extension to work properly.
Screencast

Checklist

[X] I read the extension guidelines
[X] I read the documentation about publishing
[X] I ran npm run build and tested this distribution build in Raycast
[X] I checked that files in the assets folder are used by the extension itself
[X] I checked that assets used by the README are placed outside of the metadata folder

Hi @pernielsentikaer,
Thank you for your contribution, these are nice changes! 👏
I've tested the changes and everything works fine! 😁


Feature/cloudfront access log adapter

This is looking good! Something I remembered - CloudFront access log files start with two header rows:
#Version: 1.0
#Fields: date time x-edge-location sc-bytes c-ip cs-method cs(Host) cs-uri-stem sc-status cs(Referer) cs(User-Agent) cs-uri-query cs(Cookie) x-edge-result-type x-edge-request-id x-host-header cs-protocol cs-bytes time-taken

It would be good if these were swallowed (=> None) rather than turned into bad rows...

Good point! I'll implement this in the loader rather than the adapter (since adapters have to return NonEmptyLists of raw events)

Thanks Fred...

Also note that the exact contents of line 2 will be somewhat different across the different access log formats.

Looks great Fred! Beyond the comment about an upper-case f, can you do two things:

Create a ticket for the Hadoop test you added
Remove any CHANGELOG changes to prevent merge conflicts when I pull into the release branch

Once those 3 things are done, I will merge into pygmy-parrot...

I moved the commits onto the latest version of release/r61-pygmy-parrot and now the CHANGELOG is unchanged.

Nice - thanks Fred!


Long messages are arbitrarily cut

Dear all,
This is a great cordova plugin. It has reliably worked for me in most situations.
However, if I have a long response message the response is cut arbitrarily and another onReceive Event is triggered with the rest of the message... (very long messages are even cut into multiple pieces). This wouldn't be a problem if there would be some kind of mechanism to check what should get together.
If it is unclear what I mean, I can provide an example where a IRC server is called and the users of a channel are queried. Effectively this is a long string. The response (depending on the size) is cut into various pieces. There is however no indication on how to glue these together other than information inside the message itself, which is very error prone...

I'm seeing this problem too, did you find any solution?
This seems to happen whenever the receive buffer gets full.  You can make it trigger more easily by decreasing the bufferSize parameter in src/android/ChromeSocketsTcp.java.


Robyn error while running OutputModels <- code - Error in task 7 failed - "arguments imply differing number of rows: 7, 9"

Project Robyn
Describe issue
This error keep showing while I try to test Robyn in a small sample of dataframe, I'm using the latest dev version.
Provide reproducible example
Step2: Creating First Inputs
InputCollect <- robyn_inputs(
dt_input = subset_data,
dt_holidays = dt_prophet_holidays,
date_var = "DATE", # date format must be "2020-01-01"
dep_var = "revenue", # there should be only one dependent variable
dep_var_type = "revenue", # "revenue" (ROI) or "conversion" (CPA)
'prophet_vars = c("trend", "season", "holiday"), # "trend","season", "weekday" & "holiday"'
'prophet_country = "BR", # input one country. dt_prophet_holidays includes 59 countries by default'
'context_vars = c("competitor_sales_B", "events"), # e.g. competitors, discount, unemployment etc'
paid_media_spends = c("Bing", "Tik_Tok", "Facebook"), # mandatory input
paid_media_vars = c("Bing", "Tik_Tok", "Facebook"), # mandatory.
'paid_media_vars must have same order as paid_media_spends. Use media exposure metrics like'
' impressions, GRP etc. If not applicable, use spend instead.'
' organic_vars = "newsletter", # marketing activity without media spend'
' factor_vars = c("events"), # force variables in context_vars or organic_vars to be categorical'
window_start = "2023-06-05",
window_end = "2023-06-11",
adstock = "geometric" # geometric, weibull_cdf or weibull_pdf.
)
print(InputCollect)
OutputModels <- robyn_run(
InputCollect = InputCollect, # feed in all model specification
cores = NULL, # NULL defaults to (max available - 1)
iterations = 2000, # 2000 recommended for the dummy dataset with no calibration
trials = 5, # 5 recommended for the dummy dataset
ts_validation = TRUE, # 3-way-split time series for NRMSE validation.
add_penalty_factor = FALSE # Experimental feature. Use with caution.
)
print(OutputModels)
Error:
Input data has 7 days in total: 2023-06-05 to 2023-06-11
Initial model is built on rolling window of 7 day: 2023-06-05 to 2023-06-11
Time-series validation with train_size range of 50%-80% of the data...
Using geometric adstocking with 11 hyperparameters (11 to iterate + 0 fixed) on 1 core (Windows fallback)

Starting 5 trials with 2000 iterations each using TwoPointsDE nevergrad algorithm...
Running trial 1 of 5
|                                                                                |   0%Timing stopped at: 0.71 0.09 0.81
Error in { :
task 7 failed - "arguments imply differing number of rows: 7, 9"

Environment & Robyn version
Robyn Version: >‘<IP_ADDRESS>00’
R version 4.2.0
dummy_data.csv

I've never seen Model fitting with only 7 days. Please use the full dates and retry


E_TOMANY_RULES

I'm literally going to start writing services in python so you all leave me alone ;p
Almost everything about standard is a barrier to entry for people both learning JS and existing experienced devs who have their own style.   It's also a lazy replacement for good code-review practices and knowledge sharing that used to happen back in the days when code was a craft.
Linters IMO should find bugs like undefined functions, unused imports etc.

And in many iflix repos refuse to let the developer push code unless they comply.. the kicker? the pre-compiler strips those spaces out again.
Honestly, nobody in the company gives a @#%^ about that space, it doesn't suddenly make you unable to comprehend the code, but it does add friction when someone gets it wrong and they have to correct it. To what end? really?
I agree we should have some standards but they should be verbal not hard-enforced, and bad programming practice should be spotted in code-review and then explained to the person so they become a better developer through learning, not blindly bashing their head against an angry linter
until it passes.. and then having no clue why they should have done it that way at all.
I realise I'm in the minority here, so I dont expect anything to change, but I'd ask that you all consider exactly when linting is a good idea and when it is not a good idea. Given we pre-compile it's (mostly) only having an impact on engineers, and the impact IMO is:

Less awareness of why some techniques are good / bad.
Less discussion of programming practice (craft / tacit knowledge transfer) through prescriptive rather than consultative rules.
Slows down engineers when they are being creative and experimenting.

I'm literally going to start writing services in python so you all leave me alone ;p

Surely you meant s/python/go ;)
Still, I'm 50/50 with the sentiment of the above comment though. I think code consistency across an Organization is worthwhile, both in style and quality. Linting solves one of those challenges, a good engineering culture solves the other.

I was in the same boat as you, it took me ages to stop typing ; because of muscle memory, and to remember to add spaces. Now i'm 6 months into using it for everything (even my personal) stuff and I wouldn't go back.
The reason we do this is not to make it easier for the compiler, but to make it easier for humans to come into a massive codebase (comprised of 100+ repositories) and have them understand it easier because they're all written the same way.
The goal is simple: one way of writing syntax.
It's not that big of a stretch. Python has pep8. Go has fmt. Javascript has standard(?)
Whether or not we want to include semicolons or not, or enforce spaces after function names is a debatable issue that not everyone is going to agree on, but instead of bikeshedding for months on end trying to come up with a conclusion that everyone is happy with, sometimes we just have to make a call.

Ok so we did that with Standard, so by your argument, forking standard is a return to the bike-shedding you wanted to avoid in the first place?

see: https://github.com/iflix-letsplay/standard/issues/1#issuecomment-254110536

See my reply to that comment josh just linked :)
@pomke potentially yes. But I've allocated a set timeframe (1m) and that is nearly up, so the bikeshedding won't last forever :p

So lets just not do it at all, stick with Standard as it is, even a month sets a precedent for change.


why the gfnet was deleted

why the gfnet was deleted：Global Filter Networks for Image Classification

Compared to latest models like VAN, its result is not that good anymore. And the main reason is fft operation is not conversion friendly to another framework like onnx (based on my own testing).


Proposal: Add a 'Create Account' on login page to lower hurdles for new Users

Currently a user must already have an account on a pod.
New users may not know what to do when they downloads the app.
What's about making a relative simple 'Create Account' button on sign in page?
This may simply forward to a (random, precoded?) open pod somewhere? No complicated 'Find a pod' procedure, but just forwarding to a set of predefined pod/users/sign_up  - URLs?
Bonus:
(Later) this list may be curated externally by reading a file (from Git page?)

I guess we could do, but it might be better to delay this until after the API is in a stable diaspora* release.
The three options I see are linking:

https://diaspora.podupti.me/go.php?software=diaspora, redirects to a random pod with open sign ups, might be quite confusing
https://diaspora.podupti.me/, renders a list of diaspora pods, still quite confusing, "what's this?"
https://wiki.diasporafoundation.org/Choosing_a_pod, annoying wall of text but maybe the best option, as in your scenario a user is likely unaware of what diaspora exactly is and how it works.

Like the first link in combination to an explanation.
Like also the Idea to delay this a bit after release.
Just a simple link-text (in tint color) and a (?) - Info -Sign that links to a well chosen explanation site.

Triggered by what?
I suggest to have a shirt direct link (see my screenshot) to a random pod and the modal could appear after tapping the (?) button.
This might keep the path short but also has more informations.

It would appear after clicking on a sign up link on the sign in page.
Linking to a random pod on the dominant link and only explaining things on a tiny small link that maybe 5% of the users that would benefit from the explanation is pretty dark UX IMO. It's actually quite important to consciously decide where you want to sign up ("just give me a random one" being a total legit decision when made consciously), I don't feel comfortable making it for the user. Imagine a mail client linking to a random mail provider in their onboarding flow.

Like this. Full agreement.


Greenify the Travis build

I know that @jamescway was already working on this, but here's an issue to keep track of it!

Looks like @adriancole has fixed a good chunk of things thus far, woohoo!

It has been green for a while now! :smile:


Switch to poru

you are making milarato 2 right so erela.js no longer maintained so change to poru it also like erela.js but it get more update like its the next erela.js(poru) i suggest to switch to poru https://www.npmjs.com/package/poru

I think you don't know that there is no reason to switch, if erela.js has supportive of 100% lavalink's feature(s).
All I am doing with this FORK is making it "better" for my needs.
which i would have needed to do with "poru" as well.

Also poru is not as efficient as my erela.js version.


Should not contain child node_modules

Directoy of lua-types v2.10.1

This will cause conflicts with the @moddota/dota-lua-types

When I download v2.10.1, it doesn't contain any folders besides core and special, no node_modules.
have you tried force-reinstalling, or do you still have a npm link or yarn link up?

@GlassBricks I've been using yarn install, just now I deleted the node_modules directory and reinstalled using npm install and got the same result, which seems to be submodules automatically installed by yarn and npm.
I think it maybe a version problem. I use typescript-to-lua v1.0.1, but lua-types still dependent on "typescript-to-lua": "^0.39.0".

@RobinCodeX I was testing some stuff and updated the version myself like you noticed. I have just release a new version, please give it a try and test if this works.

@Perryvw I tested it and it's fine


Add idle_timeout units in comments.

Can you also add this to the Rust documentation please? https://github.com/cloudflare/quiche/blob/master/src/lib.rs#L653

@ghedo added, thanks


Needs to Support Cookies

During the login post you have a commented out line "credentials: "same-origin"". (https://github.com/ni/niauth-js/blob/master/index.js#L189)
This means that the additional services aren't authenticated.
You obviously reference on the first GET that this causes other issues with prior logins which need resolving but I don't see any other way of using this library with additional web service calls

Looks like the login 200 response returns the logged in user in the body. I get a response:
Logged in as: admin
I think the correct function would be:

Allow the cookies.
If you get a 200 to the parse out the username.
If it matches. do nothing more.
If it doesnt then logout and re-login as the new user.

An API call to check this on application load would also be fantastic. How does this fit with your use case?

Yep, that hadn't been fully baked. Commit 7b777f28baac should resolve most of what you're asking for. updateFromSession is the "check on application load" function you describe.
The only thing that's missing is the automatic "logout-and-re-login" case; the current implementation of the "login" function is starting to get a bit unwieldy, so I'd like to have a crack at splitting that up first before I add yet another callback-indentation level to deal with automatic logout-and-back-in.


adding replication alerting

Checklist

[x] The PR has a meaningful title. It will be used to auto generate the
changelog.
The PR has a meaningful description that sums up the change. It will be
linked in the changelog.
[x] PR contains a single logical change (to build a better changelog).
[x] Categorize the PR by adding one of the labels:
bug, enhancement, documentation, change, breaking, dependency
as they show up in the changelog.
[x] Link this PR to related issues or PRs.

@Kidswiss I tried to add some general tips how to debug, but since I never fix that in prod it's hard to guess what could be usefull, there are such docs: https://bootvar.com/useful-patroni-commands/ , but I don't see sense in putting it into our docs. wdyt? have You got anything better?

@wejdross I would explicitly put in the patroni reinit for the replication alerts, because that's how you fix a broken replication.
Also for the pod replicas I would add a few commands how to check the statefulset for events.
It doesn't have to be much just a few lines.


Add Schema.org or Croissant metadata to header of Dataset view page

Currently the JSF Dataset page has schema.org info embedded in the header, which in the future may be replaced with Croissant.  The SPA version of the page has to replicate this.
Here is what it looks like in the JSF Header:
<script type="application/ld+json">{"@context":"http://schema.org","@type":"Dataset","@id":"https://doi.org/10.5072/FK2/SCYB0O","identifier":"https://doi.org/10.5072/FK2/SCYB0O","name":"Testing embargo","creator":[{"@type":"Person","givenName":"Guillermo","familyName":"Portas","name":"Portas, Guillermo"}],"author":[{"@type":"Person","givenName":"Guillermo","familyName":"Portas","name":"Portas, Guillermo"}],"datePublished":"2024-03-14","dateModified":"2024-03-14","version":"1","description":"test","keywords":["Business and Management"],"license":"http://creativecommons.org/publicdomain/zero/1.0","includedInDataCatalog":{"@type":"DataCatalog","name":"Root","url":"https://beta.dataverse.org"},"publisher":{"@type":"Organization","name":"Root"},"provider":{"@type":"Organization","name":"Root"},"distribution":[{"@type":"DataDownload","name":"dataverse_files (2).zip","encodingFormat":"application/zip","contentSize":4540,"contentUrl":"https://beta.dataverse.org/api/access/datafile/26133"},{"@type":"DataDownload","name":"FilesIT.java","encodingFormat":"text/x-java-source","contentSize":154657,"contentUrl":"https://beta.dataverse.org/api/access/datafile/26132"}]}

The Dataverse API for getting this uses the exporter, for Schema.org:
https://beta.dataverse.org/api/datasets/export?exporter=schema.org&persistentId=doi:10.5072/FK2/SCYB0O
And for Croissant format:
https://beta.dataverse.org/api/datasets/export?exporter=croissant&persistentId=doi:10.5072/FK2/SCYB0O

One concern we have is what to do when the schema.org or croissant files get large, such as 7 MB for a dataset with 25k files. These issues are related:

https://github.com/mlcommons/croissant/issues/646
https://github.com/IQSS/dataverse/issues/10542

Also, in JSF we show the schema.org version unless the croissant jar file is present:

https://github.com/IQSS/dataverse/pull/10382

I wrote some docs about this in an (open) pull request:

https://dataverse-guide--10533.org.readthedocs.build/en/10533/admin/discoverability.html#schema-org-head
https://github.com/IQSS/dataverse/pull/10533

For a quick proof of concept, it would be ideal to do a simple insert of the expected script (hardcoded & type="application/ld+json") in question into the head of the single index.html that handles the SPA.
Simply from the home (Collection page), in a useEffect that runs only once, so we can simulate how it would really be the insertion of this script inside the head once the SPA Javascript is loaded and thus confirm through Search Google Rich Results if the script is being detected or not.
As a second approach, if we know that the script is detected, we should detect the persistentId in question through the url of the page of a Dataset, fetch the endpoint mentioned with the persitentId and insert the result in a script type “application/ld+json” in the header of the html.
And when the user navigates away of the page, in the return of the useEffect that will be executed when this component/page is unmounted, delete the script in question. (This only if we are not in a mobile device, this could be detected in a very simple way at the moment through the screen width.)

beta.dataverse.org has been updated with a robots.txt to allow all, so now https:/beta.dataverse.org is being crawled successfully, but individual dataset pages are not being indexed by Google. See this page for the Rich Results test: https://search.google.com/test/rich-results/result?id=XS1bhHFD7CEtXP5vHMIxog.  Putting it back in This Sprint for further investigation, since it's a lower priority for Q2.

Moving it to the backlog due to a problem with the server configuration for the SPA redirection.
Currently when entering directly to a SPA url other than the main /spa/ it is returning the index.html document but with a 404.
This is because of web.xml located on frontend repo under deployments/payara/ is handling urls that dont belong to an actual file or folder as an error page and returning index.html with a 404 Not Found page status, making it not crawlable.
  <error-page>
    <error-code>404</error-code>
    <location>/index.html</location>
  </error-page>

This problem must be solved in order to return to this issue.

2024/07/10

Removing the On Hold status and moving back to SPA classification


SPARK-33888 Restored scale metadata for ARRAY type (Postgres)

This satisfies PostgresDialect's requirement for scale in Numeric arrays (as it was before SPARK-33888 removed the metadata)
What changes were proposed in this pull request?
Ensuring that a numeric scale component is provided to Postgres for Array type metadata processing.
Why are the changes needed?
The initial changes in SPARK-33888 restrict the "scale" column metadata to NUMERIC and DECIMAL types, but PostgresDialect was also using "scale" for Array types.
Does this PR introduce any user-facing change?
No. (This restores master to the same [Postgres] behavior as it had on 3 Jan 2021)
How was this patch tested?
Manually tested by loading a Postgres table that specified a numeric array column.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

ok to test

ok to test

Thanks for your contribution, @skestle ! Could you add tests in PostgresIntegrationSuite?

Thanks for your contribution, @skestle ! Could you add tests in PostgresIntegrationSuite?

Also, could you file new jira for this issue and describe what's an issue there? I think we need it for better issue traceability.

Also, could you file new jira for this issue and describe what's an issue there? I think we need it for better issue traceability.

Test build #134242 has started for PR 31252 at commit 9fe9769.

Test build #134242 has started for PR 31252 at commit 9fe9769.

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

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

Test build #134242 has finished for PR 31252 at commit 9fe9769.

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

Test build #134242 has finished for PR 31252 at commit 9fe9769.

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

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

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

Thankyou @sarutak for following up with #31262 when I didn't have the time ;)

Thankyou @sarutak for following up with #31262 when I didn't have the time ;)


Camelize props

Can I camalize props? Something like this
= react_component('HelloMessage', props: {name: 'John'}, {camelize_props: true})

@devality that sort of functionality is best handled either at the JBuilder level, like
https://github.com/rails/jbuilder

You can set this globally with the class method key_format (from inside your environment.rb for example):

Jbuilder.key_format camelize: :lower
You can also create a utility method like this:
def camelize_keys(hash)
  hash.deep_transform_keys! { \|key\| key.to_s.camelize(:lower) }
end

Hey @justin808 how could we use the Jbuilder partials with the react_component function?

Hey @justin808 how could we use the Jbuilder partials with the react_component function?

Hi @jesster2k10, I'm not sure what you're asking for. Prop serialization is best handled outside of the react_component method (ruby methods). You can create your own helper method that wraps react_component.

Hi @jesster2k10, I'm not sure what you're asking for. Prop serialization is best handled outside of the react_component method (ruby methods). You can create your own helper method that wraps react_component.


gcov: Support for the most streamlined profile of LLVM-embedded-toolchain-for-Arm

Summary
1. Excerpted from: https://github.com/ARM-software/LLVM-embedded-toolchain-for-Arm/blob/main/samples/src/cpp-baremetal-semihosting-prof/proflib.c
2. Since llvm profile supports more than just gcov, and some features have not yet been explored, two clang gcov implementations are supported after this patch
3. Using this lib only supports the gcov compilation options of "-fprofile-instr-generate -fcoverage-mapping"
4. This file is heavily dependent on the compiler clang version, and is currently aligned with ci, supporting 17.0.1 and below. 18 and above are not supported by this library due to different internal implementations of the compiler

Impact
！！！This file is heavily dependent on the compiler clang version, and is currently aligned with ci, supporting 17.0.1 and below. 18 and above are not supported by this library due to different internal implementations of the compiler
Testing
Using this lib only supports the gcov compilation options of "-fprofile-instr-generate -fcoverage-mapping", and __llvm_profile_dump output the results

This patch comes from https://github.com/ARM-software/LLVM-embedded-toolchain-for-Arm/blob/main/samples/src/cpp-baremetal-semihosting-prof/proflib.c. Its implementation is more concise than rt.profile and focuses on gcov functions. Can you consider merging it? If you consider merging it, I will update this patch in the next step.

Do we have any research on GCC and CLANG gcov implementations? I am curious why we chose to use CLANG gcov implementations not GCC, (especially PR #14472 introduced a lot of clang runtime library implementations )

Do we have any research on GCC and CLANG gcov implementations? I am curious why we chose to use CLANG gcov implementations not GCC, (especially PR #14472 introduced a lot of clang runtime library implementations )

the profile implementation is totally different from clang and gcc, so you can't use gcov library directly with clang.

Do we have any research on GCC and CLANG gcov implementations? I am curious why we chose to use CLANG gcov implementations not GCC, (especially PR #14472 introduced a lot of clang runtime library implementations )

the profile implementation is totally different from clang and gcc, so you can't use gcov library directly with clang.

so why choose CLANG not GCC ?

Do we have any research on GCC and CLANG gcov implementations? I am curious why we chose to use CLANG gcov implementations not GCC, (especially PR #14472 introduced a lot of clang runtime library implementations )

the profile implementation is totally different from clang and gcc, so you can't use gcov library directly with clang.

so why choose CLANG not GCC ?

Which we should select is decided by compiler: clang variant must use runtime profile, gcc variant must use gcov.

The result is as shown below：

The result is as shown below：

Sorry I have to cancel the CI Check, there's a problem with CI Test:

https://github.com/apache/nuttx/pull/14667#issuecomment-2461197521

Sorry, I accidentally closed this PR

rebase the patch @W-M-R

depends on https://github.com/apache/nuttx-apps/pull/2842

depends on https://github.com/apache/nuttx-apps/pull/2842


Collect connection address in connectivity test

We now have connectivity test functions for dialer wrappers, rather than just for resolvers.
This also adds config.WrapPacketDialer

This needs to be updated. See https://github.com/Jigsaw-Code/outline-sdk/discussions/209#discussioncomment-9216652


Change Repository name and Update README.md

Repository name should be MLlib, not MLLib.

Is the actual repository name going to be changed to match?

I am leaving the name as is for now.. and will try to fix again next week when I have more time.


Add a way to "blacklist" MACs

In addition to #27, once we introduce the concept of unregistered MACs, it would be nice to have a way to blacklist a MAC so that it neither belongs to a person nor is unregistered. This way we'd not have the number inflated by Raspberry Pi's and the like.

Tips for anyone who will be eager to implement this in the future:
I added entry with preferable configuration env variable for this scenario at #24.
Take a look at storage.UpdateStatuses function. You chould pass models.Config structure through context with the help of the router and use it inside this function to filter out MAC addresses.


gridApi.edit.on.afterCellEdit fires a second time after initial row edit.

After the first row edit, if i wait nothing happens (as it should) if i click on the grid then scroll down (scroll wheel up) in OSX, then the afterCellEdit fires.
First Edit - I'm editing Name on first Row (from Alex to Dayan)

All should be done but if I scroll down, it fires again.

Most likely what is happening is that you have a particular cell focused, you scroll down, which triggers a lose focus (endCellEdit) on that cell.  It also selects another cell, which if you have edit on focus, will start  editing that cell.  When you scroll again, it loses focus on the second cell, and selects another cell.
The if (newValue !== oldValue) { check should avoid any logic running in this situation, so it shouldn't break anything.
The best fix for this would be for us to discard the focused cell upon scroll (and not select another cell).  Unfortunately we haven't been successful in doing this without having bad side effects in the code - we got problems when you selected a cell and then a scroll was triggered (perhaps by the cell being partly off the page, and then typing into it, although there were other scenarios).
What problem does this cause for you?

Paul,
The biggest issue for me is the fact that my newValue still remains as Dayan so the if statement I have in place is allowing another Postback via the saveFieldEdit  in the screenshot examples.
Right now I just made this hack to get around the problem:
if (newValue !== oldValue && $scope.recentEditedValue !== newValue)
In the above, $scope.recentEditiedValue is updated after every successful Edit with the new value, so when the afterCellEdit kicks in, it checks and sees that recentEditedValue is equal to newValue  then just returns false.
Thanks,
Dayan

Could you provide a plunker for that?  It sounds like you're saying that the second time it fires it is showing a different newValue and oldValue, and so it runs even though it shouldn't?
We did have a problem like that in earlier versions, but if you're on rc18 or rc19 you shouldn't be getting that issue.

No, the second time it fires shows the same as the first time it fires, which is why it gets through. If the newValue that was previously entered is indeed equal to the old value (the current value in the grid) then yes, the if statement would stop that from calling the afterCellEdit function. But what happens is the following.
EmployeeName: Dayan
I change this to Jenny
oldValue: Dayan
newValue Jenny
Data gets Posted back, all is good.. Now I scroll and look what happens to the old and new values.
EmployeeName: Jenny
Since it was recently changed from Dayan...
oldValue: Dayan
newValue: Jenny
They don't change.

@PaulL1 do you want to try to reproduce this one?

I believe @swalters recent changes have fixed this.  Refer this plunker: http://plnkr.co/edit/XdRX2aLEHSRMgiONRDpj?p=preview - I put a console.log in the after cell edit, and tried various flavours of editing and scrolling.  It only ever wrote the row id once.

Closing, believe this is fixed.


docs: Add Atsign-Foundation NoPorts to the Hall of Fame

Summary
We're using generators for our Dart and Python release artifacts e.g.
https://github.com/atsign-foundation/noports/releases/tag/v5.2.1-rc1
https://github.com/atsign-foundation/noports/releases/tag/p0.4.9
Testing Process
N/A - docs update
Checklist

[x] Review the contributing guidelines
[ ] Add a reference to related issues in the PR description.
[x] Update documentation if applicable.
[ ] Add unit tests if applicable.
[ ] Add changes to the CHANGELOG if applicable.

Welcome! You mentioned in #3617 to make this a multiline. I'd accept that. But just FYI, the "rich diff" view on github also makes the change clear. https://github.com/slsa-framework/slsa-github-generator/pull/3616/files?short_path=b335630#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5

@ramonpetgrave64 rather than a new PR are you OK with me pushing the multiline version for #3617 to this branch, in which case I'll convert this to draft for now and set it back when ready.
Also any thoughts on ordering: alphabetic or leave things how they are?

Let's keep the order, but I'm okay with either the multiline or single line. Up to you.

Looks good, but one more thing. You'll have to signoff on your commits. see the details on the DCO check.

@ramonpetgrave64 doh! Signoff now added to that last commit.


Remove synonym "lower grade glioma" from MONDO_0005499 brain glioma

(This is a request to remove, rather than add, a synonym, but there isn't a specific issue template.)
Stemming from https://github.com/EBISPOT/efo/issues/685
MONDO_0005499 brain glioma
Has (exact?) synonym "lower grade glioma", coming from DOID. But MONDO_0021637 'low grade glioma' already exists and the two are not the same, so you may want to remove the synonym.
Also, I noticed that MONDO_0005499 brain glioma has an "Editor note: TODO report incorrect EFO assigment of NCIT:C71732". I think I fixed that previously while working on another term, see https://github.com/EBISPOT/efo/issues/340#issuecomment-456006897. If that's what the Editor note referred to, could you delete that comment please?
Also, do let us know if there are other similar TODOs for EFO buried in Mondo comments somewhere :-) Thank you!

I see comments mentioning EFO on two classes:
'palatal neoplasm'
'mucoepidermoid tumor'

@nicolevasilevsky

I see comments mentioning EFO on two classes:
'palatal neoplasm'
'mucoepidermoid tumor'
Thank you. For the former, the action item is for EFO first (FYI I opened https://github.com/EBISPOT/efo/issues/710). For the latter, the action item seems to be for Mondo first. Best wishes,
Paola


Checkmark is not applied on select

Please verify the version of auro-combobox you have installed
2.1.4
Please describe the bug

Go to the DocSite
Interact with the "Example with checkmark for selected option"
Type "Apple"
Click on "Apple" from dropdown
Note that checkmark is not applied

https://github.com/user-attachments/assets/9dc10fad-b9ba-4be0-af3f-ae35a1188cc3
Expected behavior
The checkmark should be applied when selected to indicate the option has been chosen.
What browsers are you seeing the problem on?
No response
Additional context
No response
Exit criteria
No response

This is a duplicate of #184


Wrong permissions in Linux archive

The following programs are not marked as executable

bin/antares-8.0-solver
xpansion-launcher/antares-xpansion-launcher

Correction done for antares-8.0-solver but antares-xpansion-launcher is still not marked as executable

corrected for antares-xpansion-launcher in v0.1.0-rc tag


sourcePath is not correct

add dist to the sourcePath parameter in LightboxAsset.php
public $sourcePath = '@bower/lightbox2/dist/';

Fixed by https://github.com/hiqdev/yii2-asset-lightbox2/commit/f082ba006e625456f169b6f3b7f76f23dc5c8703
Thank you

Bumped https://github.com/hiqdev/yii2-asset-lightbox2/releases/tag/<IP_ADDRESS> release


Appstudio update test-component-pac-cgqi

Pipelines as Code configuration proposal

Pipelines as Code CI/test-component-pac-cgqi-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
10 seconds
init

✅ Succeeded
22 seconds
clone-repository

✅ Succeeded
2 minutes
build-container

✅ Succeeded
40 seconds
sanity-inspect-image

✅ Succeeded
39 seconds
deprecated-base-image-check

✅ Succeeded
4 minutes
clair-scan

✅ Succeeded
1 minute
clamav-scan

✅ Succeeded
31 seconds
sbom-json-check

✅ Succeeded
32 seconds
sanity-label-check

✅ Succeeded
31 seconds
sanity-optional-label-check

✅ Succeeded
6 seconds
show-summary

Pipelines as Code CI/test-component-pac-cgqi-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
7 seconds
init

✅ Succeeded
18 seconds
clone-repository

✅ Succeeded
33 seconds
build-container

✅ Succeeded
13 seconds
deprecated-base-image-check

✅ Succeeded
9 seconds
sanity-inspect-image

✅ Succeeded
12 seconds
clair-scan

✅ Succeeded
50 seconds
clamav-scan

✅ Succeeded
10 seconds
sbom-json-check

✅ Succeeded
14 seconds
sanity-optional-label-check

✅ Succeeded
13 seconds
sanity-label-check

✅ Succeeded
12 seconds
show-summary


Updates rails version to <IP_ADDRESS>

CVE-2015-7576
CVE-2015-7577
CVE-2015-7578
CVE-2015-7579
CVE-2015-7581
CVE-2016-0751
CVE-2016-0752
CVE-2016-0753
http://weblog.rubyonrails.org/2016/1/25/Rails-5-0-0-beta1-1-4-2-5-1-4-1-14-1-3-2-22-1-and-rails-html-sanitizer-1-0-3-have-been-released/

I just did https://github.com/alphagov/business-support-finder/compare/bump_rails, then realised you beat me. Merging this instead. (Business Support API and its Finder are together in my brain, and splitting them between two teams has always struck me as weird.)


[MAPS_API_KEY] Use distinct keys from GCP projects corresponding to the  environment

Set the maps api key as environment variable instead of hard coded.

Should we also use secret manager for staging and prod? I don't think we should put keys in github (I've received warnings about other keys I've owned in the past).

Put all api key in secret manager now


Add an initial document for our release process

[x] 📘 I've reviewed the OpenZeppelin Contributor Guidelines
[x] ✅ I've added tests where applicable to test my new functionality.
[x] 📖 I've made sure that my contracts are well-documented.
[x] 🎨 I've run the JS/Solidity linters and fixed any issues (npm run lint:all:fix).

Thanks @elopio! I edited the guide to add more detail and explanation. Would you mind reviewing those?


Can't get Gen command working

Thanks for the great work.
I'm new to neovim, I use nvchad.
I installed gen.nvim with the minimal configuration
~/.config/nvim/lua/custom/plugins/plugins.lua
_______
  ...
  {
      "David-Kunz/gen.nvim",
  },
...

. After ollama serve (version is 0.1.22), I opened  a text file, and found the Gen command not working. Here's a screenshot.

Turns out I need to add event="VeryLazy"
...
{
      "David-Kunz/gen.nvim",
       event = "VeryLazy",
       opts = {
          model = "mistral", -- The default model to use.
...


cannot load such file -- chef_compat - chef_client 12.7.2

Compiling Cookbooks...
================================================================================
Recipe Compile Error in /var/chef/cache/cookbooks/compat_resource/libraries/autoload.rb
LoadError
cannot load such file -- chef_compat
Cookbook Trace:
/var/chef/cache/cookbooks/compat_resource/libraries/autoload.rb:12:in `<top (required)>'
Relevant File Content:
/var/chef/cache/cookbooks/compat_resource/libraries/autoload.rb:
5:  begin
6:    compat_resource_gem = Gem::Specification.find_by_name("compat_resource")
7:  rescue Gem::LoadError
8:  end
9:
10:  if compat_resource_gem
11:    # The gem is installed.
12>>   require 'chef_compat'
Worked fine after chef-client downgrade to 12.4.0. Also works on 12.5.1
The dependency is brought by iptables

iptables

compat_resource

<EMAIL_ADDRESS><EMAIL_ADDRESS>

It's probably a good idea to just bring in the latest compat_resource
(12.9.0), using a downgraded compat resource on a later chef is a generally
bad idea.
On Fri, Mar 11, 2016, 12:05 PM Amod Kumar Pandey<EMAIL_ADDRESS>wrote:

<EMAIL_ADDRESS><EMAIL_ADDRESS>—
Reply to this email directly or view it on GitHub
https://github.com/chef-cookbooks/compat_resource/issues/47#issuecomment-195525834
.

this occurs for me on compat_resource 12.9.1


当父容器上存在mouseover等事件时，modal显示后也会触发mouseover事件，有没有办法阻止呢？

当父容器上存在mouseover等事件时，modal显示后也会触发mouseover事件，有没有办法阻止呢？

what do you mean by mouseover on parent? are you trying to prevent close the modal when clicked outside or something like this?
Describe your scenario and we can help you.

     ......
     document.querySelector('.app-root')}>

The alert event is also triggered when the mouse is moved into Modal

Thanks, @StefanYohansson, for jump in.
@ChenGuanglin0924 can you make a codepen or codesandbox with a reproducible example?

https://codesandbox.io/s/wy1pp713k5 @diasbruno

https://codesandbox.io/s/wy1pp713k5 @diasbruno

@ChenGuanglin0924 in this case, since the modal is inside the
 with the event listeners, you need to stop the propagation on those handlers, so they don't go through the modal.

React-Modal doesn't provide the way to stop the propagation, but there is one way, for example, add a div outside the Modal, and stop the propagation inside the div, but I don't think this method is very good, it will leave a empty div in the dom. @diasbruno

......

{e.stopPropagation()}}>
<Modal parentSelector={() => document.querySelector('.app-root')}></Modal>

Perhaps, for a cleaner solution, there are 2 options in this case:

Make the div and the modal siblings:

This way the modal is not on the hierarchy and doesn't receive the event.

Control using isOpen property:

I would recommend to always use this property and never keep it on a local state. It gives better control.
You can use this state to just make the event listener a noop.

MouseEvent(flag) {
  // note that it's a method to get the current value. I don't thing it's safe.
  // better to leave the parent component to say when the modal should be opened.
  if (this.refs.modal.isOpen()) {
    return;
  }
  ...
 }
}

@ChenGuanglin0924 Is the comment above helped or have you solved this issue?

@ChenGuanglin0924 I'll close this issue for now. Please feel free to reopen if this issue still persist. Thanks.


minimal actually returns a Range

The docs for minmax say they return Lists. But they actually return Ranges:
> (1,2,3).minmax.^name
Range

> <a b c>.minmax.^name
Range

I think the documented behavior is the one I'd prefer and expect (a list of two elements), but the tests in S32-list/minmax.t show them returning Ranges.

A workaround is to immediately call .bounds on the result


ImportError: cannot import name 'SortedDict'

Hello Django-tinymce developers, I have just Upgraded Django 1.8.1 and I got this problem when I try to build my web application. Hope to see your reply soon, and sorry for my bad English
File "C:\Python34\lib\site-packages\tinymce\widgets.py", line 17, in
from django.utils.datastructures import SortedDict
ImportError: cannot import name 'SortedDict'

you need to use collections.OrderedDict
so your import is like this in tinymce/widgets.py
replace
from django.utils.datastructures import SortedDict
by
from collections import OrderedDict as SortedDict
in the same time it solve the Django warning : SortedDict deprecated an Removed in Django 1.9
read :  https://code.djangoproject.com/wiki/SortedDict


Refs #29548 -- Mentioned MariaDB in database system checks.

Preparation for #12459.

@carltongibson Thanks :+1:

Thanks!


Kindly spare some time to assign. Before raising any pull requests.

Kindly spare some time to assign. Before raising any pull requests.
Originally posted by @Sulagna-Dutta-Roy in https://github.com/Sulagna-Dutta-Roy/GGExtensions/issues/217#issuecomment-2114283954

Closing this issue because already raised before. #216


Installation of krakend-integration fails in Go 1.18 or higher

As per the instructions in https://www.krakend.io/docs/developer/integration-tests/, running go install<EMAIL_ADDRESS>should add the binary to my PATH.
However, the package download fails in Go version 1.18 or higher:
 » go version
go version go1.19.3 darwin/amd64
 » go install<EMAIL_ADDRESS>go<EMAIL_ADDRESS>(in github.com/krakendio/krakend-ce/v2@v2.1.2):
        The go.mod file for the module providing named packages contains one or
        more replace directives. It must not contain directives that would cause
        it to be interpreted differently than if it were the main module.

As a temporary workaround, running tests in Go 1.17 does the trick, but it would be nice to have version parity between tests and running environments of krakend-ce.

Hi @jguerreiro-sqsp
Thank you for reporting this issue.
We are looking into removing the go mod replace directives for the next version.
As a workaround, if you clone the git repository and run the go install from the krakend-integration directory should do the trick.

Duplicates #470


Can I intercept navigation events?

I am trying to see if I can use this library to implement OAUTH in my dotnet application. In order to accomplish this, I need to be able to intercept the redirect responses from the server, because I need to compare them against the OAUTH callback value.
Is there a way to register for events for when the url changes in the browser? Basically the "Navigation Starting" event from WebView2 (https://learn.microsoft.com/en-us/microsoft-edge/webview2/concepts/navigation-events)
I did notice a WebMessageReceivedHandler available, but (at least on windows) I can't get it to fire to understand what it does.
(EDIT: I now understand this event appears to fire only when something clicked, so it won't help me).

We do not currently handle those kind of events (yet). These events would need to be added to the list of events we already do handle, which someone with time and interest could probably accomplish. It is not going to be on our immediate TODO list though.

You can do this, but not using the built in c#/js interop.  You need to build a proper webapi and then self host it in the Photino app.

If you want to handle the navigation events on the client, that can be done in JavaScript. It's built into many SPA frameworks. Or in pure JavaScript, take a look at https://developer.mozilla.org/en-US/docs/Web/API/Navigation/navigate_event#examples. If you want to handle it on the back end, then @rtmann is correct, you can handle it in the web server. ASP.NET for example, which is small, fast and already included in the deployment. the only place you can't handle it right now is in the WebView control where it raises an event sent back to your .NET app. We may be able to add that at some point.


init storage engine before ngxs starts

Hello,
I am trying to use the async-storage-plugin together with cordova-sqlite-storage.
The Storage Engine needs to get initialized (open DB etc.)
Where is the right place to do this? AsyncStoragePlugin always wants to access before the DB is available.
I was trying to init the DB with angular APP_INITIALIZER but this gets triggered after Ngxs init routines.

I also tried to use ngxsOnInit hook. but when using AsyncStoragePlugin ngxsOnInit never gets called. But it gets called when i disable AsyncStoragePlugin

Maybe too late, but for Ionic 4 with Ionic storage, I tried initializing in the angular APP_INITIALIZER (with blocking to wait until ionic storage was ready) and then loaded ngxs in a child module (e.g. core.module.ts), and all of my states were in feature modules, and that's worked for me on iOS, Android, and web.


Show progress with tqdm in extract frames from video

Call tqdm when processing all frames of a video file in order to give users an estimate of execution time, similar to whats done in deepface.

Added on commit https://github.com/rafribeiro/forensicface/commit/7161a0fb710de47388a474215c79ee633ef8e844


How do we test SQLite

Package version (if known): 0.1.0
Describe the bug
Currently, it seems unclear how we should test SQLite
Expected behavior
I would expect the following to work:
env DB=sqlite ./run-tests.sh

This is now working by not setting DB. This is because DB does not need to be booted up.


Adding your repo to awesome-tinkerpop

Hello, would you like to get your repo to tinkerpop curated list awesome-tinkerpop?
Please add an adequate short description to your repo and let me know here. I would highly appreciate if you send a pull request.

Yes, sure. I will do it next week. I am out of town...

On a related note, we could also add this to the the TinkerPop home page in the provider index. It looks like you meet our guidelines for doing so:
http://tinkerpop.apache.org/policy.html
Would you like to see it added?

Sure, I was going to find out the process to get the library listed on the apache Tinkerpop project.. Your help will be very welcome if you can do it..
Thanks

I've started a thread on the dev list recommending that we add it:
<EMAIL_ADDRESS>i will add it once that's settled.

I just added it to awesome-tinkerpop. Check it here

Thanks

Note that the provider listing on the TinkerPop home page is also updated. I think that a lot of people have been looking for this functionality. This is a very nice contribution to the TinkerPop and Neo4j communities.
@rjbaucells could you offer any indication as to how well gremlin performs over Bolt? have you made any comparison to the embedded version we have at TinkerPop?

Thanks.
I have not done a comparison with the embedded version since the application I am using the library for uses a remote server. The library performs well and I have not seen any issues so far related to performance. It is good to notice that the BOLT protocol support for Java data types is limited, making the gremlin test suites fail in some cases (BOLT does not support Integer, only Long; casts within the test suite to Integer will fail). see https://neo4j.com/docs/developer-manual/current/drivers/#driver-types for supported data types.

Hmm - you set the Features properly to not support Integer:
https://github.com/SteelBridgeLabs/neo4j-gremlin-bolt/blob/master/src/main/java/com/steelbridgelabs/oss/neo4j/structure/Neo4JGraphFeatures.java#L219
Do you have an example of such a failing test that you could point me at?

Yes, the features are set with the actual support. I will run the tests again and put some examples, there are explicit casts in the test suite that make the tests fail.

This is one of the examples while running:
@Test
@FeatureRequirementSet(FeatureRequirementSet.Package.SIMPLE)
public void shouldGenerateSameGraph() throws Exception {
}

gremlin-test-3.2.3-sources.jar!/org/apache/tinkerpop/gremlin/algorithm/generator/CommunityGeneratorTest.java line 116

java.lang.ClassCastException: java.lang.Long cannot be cast to java.lang.Integer
at org.apache.tinkerpop.gremlin.algorithm.generator.CommunityGeneratorTest$DifferentDistributionsTest.lambda$verticesByOid$4(CommunityGeneratorTest.java:153)
at java.util.TimSort.countRunAndMakeAscending(TimSort.java:355)
at java.util.TimSort.sort(TimSort.java:234)
at java.util.Arrays.sort(Arrays.java:1512)
at java.util.ArrayList.sort(ArrayList.java:1454)
at java.util.Collections.sort(Collections.java:175)
at org.apache.tinkerpop.gremlin.algorithm.generator.CommunityGeneratorTest$DifferentDistributionsTest.verticesByOid(CommunityGeneratorTest.java:152)
at org.apache.tinkerpop.gremlin.algorithm.generator.CommunityGeneratorTest$DifferentDistributionsTest.communityGeneratorTest(CommunityGeneratorTest.java:170)
at org.apache.tinkerpop.gremlin.algorithm.generator.CommunityGeneratorTest$DifferentDistributionsTest.shouldGenerateSameGraph(CommunityGeneratorTest.java:121)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47)
at org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
at org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:26)
at org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27)
at org.junit.rules.TestWatcher$1.evaluate(TestWatcher.java:55)
at org.junit.rules.RunRules.evaluate(RunRules.java:20)
at org.junit.runners.ParentRunner.runLeaf(ParentRunner.java:325)
at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:78)
at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:57)
at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
at org.junit.runners.Suite.runChild(Suite.java:128)
at org.junit.runners.Suite.runChild(Suite.java:27)
at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
at org.junit.runners.Suite.runChild(Suite.java:128)
at org.junit.runners.Suite.runChild(Suite.java:27)
at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
at org.junit.runners.Suite.runChild(Suite.java:128)
at org.apache.tinkerpop.gremlin.AbstractGremlinSuite.runChild(AbstractGremlinSuite.java:212)
at org.apache.tinkerpop.gremlin.AbstractGremlinSuite.runChild(AbstractGremlinSuite.java:50)
at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
at org.junit.runner.JUnitCore.run(JUnitCore.java:137)
at com.intellij.junit4.JUnit4IdeaTestRunner.startRunnerWithArgs(JUnit4IdeaTestRunner.java:78)
at com.intellij.rt.execution.junit.JUnitStarter.prepareStreamsAndStart(JUnitStarter.java:212)
at com.intellij.rt.execution.junit.JUnitStarter.main(JUnitStarter.java:68)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at com.intellij.rt.execution.application.AppMain.main(AppMain.java:140)

i see - that's an example of a test with a bad FeatureRequirement annotation (or a bad implementation depending on how you look at it).  The test could either have a String for the "oid" value or the test should have a FeatureRequirement for FEATURE_INTEGER_VALUES.
would you like to submit a pull request to correct that (and others like it)? That way your test suite will pass in more legit fashion for next release. if you aren't so inclined to do a pull request, i can get it fixed (and others if you tell me what they are). wdyt?

@spmallette I do not have time now to create a pull request, I can do it in about a month or so... In general I think it would be better to be able to specify a default data type for all tests that are not type-specific in the test suites, a value provider or something that let the implementor choose a type that is fully supported by the implementation. I know that Integer should be common enough, but in this case is not supported...
As a starting point, all tests defined in org.apache.tinkerpop.gremlin.algorithm.generator.CommunityGeneratorTest.java fail due to the explicit Integer cast verticesByOid() (line 153). The same pattern is implemented in other parts of the test suite (org.apache.tinkerpop.gremlin.algorithm.generator.DistributionGeneratorTest.java, line 145)
The ProcessStandardSuite.class is in better shape, it is following the Feature definition but too many tests are skipped due to dependency on non-supported types in the BOLT protocol.

I do not have time now to create a pull request, I can do it in about a month or so.

ok - not a problem - i will fix the "generator" tests myself then.

I know that Integer should be common enough, but in this case is not supported...

we've never seen a problem with such basic data types like Integer. this seems like a problem specific to Bolt. making the tests more type agnostic would be a fat undertaking i think.

@spmallette I forgot I had removed the datatype verification at the time of setting a property value to a Vertex/Edge. After I put it back again the org.apache.tinkerpop.gremlin.algorithm.generator.CommunityGeneratorTest tests fail with the exception (java.lang.IllegalArgumentException: Property value [0] is of type class java.lang.Integer is not supported). The errors are thrown when trying to assign an Integer value to a vertex property (afterLoadGraphWith(), line 146)

yeah - i guess we never realized an int was being used in that test. it should have had:
@FeatureRequirement(featureClass = Graph.Features.VertexPropertyFeatures.class, feature = Graph.Features.VertexPropertyFeatures.FEATURE_INTEGER_VALUES)

i'm adding that now to those tests, so it should be fixed in the future for you.

Same case on:
org.apache.tinkerpop.gremlin.algorithm.generator.DistributionGeneratorTest, afterLoadGraphWith() line 181
org.apache.tinkerpop.gremlin.structure.EdgeTest, shouldNotHaveAConcurrentModificationExceptionWhenIteratingAndRemovingAddingEdges() line 283, 286
Thanks,
RJB

ok - fixed those. should be available on 3.1.6 and forward.

Thanks, I will try it as soon as it is available

Hi @spmallette,
I modified the travis build script to execute the Tinkerpop integration tests to validate the changes you did on the up-coming 3.1.6 release but I am still getting datatype related errors while running the tests. You can see the details on the build log:
https://travis-ci.org/SteelBridgeLabs/neo4j-gremlin-bolt#L1193
https://travis-ci.org/SteelBridgeLabs/neo4j-gremlin-bolt#L1261
https://travis-ci.org/SteelBridgeLabs/neo4j-gremlin-bolt#L1316

not sure if it matters but those links you provided to the build logs didn't seem to point to much when i clicked on them. in any case, i did scroll through and find the errors. it didn't appear as if i'd published a fresh snapshot after i made my changes to the code on 10/26. i just did that now so hopefully if you restart your travis job, you'll get a good build at this point.
https://repository.apache.org/content/groups/snapshots/org/apache/tinkerpop/gremlin-test/3.1.6-SNAPSHOT/

Thanks @spmallette, I ran the test suite again and I only have one error now:
org.apache.tinkerpop.gremlin.structure.PropertyTest.shouldAllowRemovalFromEdgeWhenAlreadyRemoved() (line 108) is adding an Integer value without testing support for it...
Is there any plans for a release with these changes?
Current tinkerpop version is 3.2.3, are the test fixes going to be applied to a version after it (they are only on 3.1.6-SNAPSHOT)?
Thanks for your help...

dah - we're missing a Integer feature definition on shouldAllowRemovalFromEdgeWhenAlreadyRemoved. I can add that and get a new SNAPSHOT published. Both of my systems are busy in long builds right now, but as soon as one is free i'll do that and post back here.

Is there any plans for a release with these changes?

There hasn't been any discussion in the community about release plans at this point. We do have a good body of changes in place right now, so perhaps it's time to do one.

Current tinkerpop version is 3.2.3, are the test fixes going to be applied to a version after it (they are only on 3.1.6-SNAPSHOT)?

The fixes have been made on the 3.1.x line but they are merged forward to 3.2.x and 3.3.x, So, they should be on 3.16, 3.2.4 and 3.3.0 when those releases are made.

just deployed a fresh SNAPSHOT of 3.1.6. hopefully that solves all your issues now. i also deployed 3.2.4-SNAPSHOT and 3.3.0-SNAPSHOT, in case you want to start testing against those. note that we likely won't get to an official release of 3.3.0 for a while - it's still in fairly early stages.  the community expects to do a few milestone releases on that line of code as it will likely contain breaking changes to contend with and for folks to get used to. if you don't already follow the dev mailing list, you might want to consider subscribing:
<EMAIL_ADDRESS>all technical discussion as well as release related talk happen there. as a "provider" you might find the discussions we have there useful/interesting.

Good news, the test suite passed without errors:
Tests run: 1627, Failures: 0, Errors: 0, Skipped: 1039
I test it using 3.2.4-SNAPSHOT. As you can see there are a some tests that are been skipped due to data type support missing in the BOLT protocol....
Thanks @spmallette for your help... I will be waiting on a release on your side to do the same here.
RJB

I'm not sure if this is useful to you, but I just implemented this for 3.2.4 (still under review as a pull request):
https://github.com/apache/tinkerpop/pull/521
If there are expensive resources that could be released as a result of the way that bolt works, this could be a way for you to allow users to release those resources.

Yes, that will help once OLAP API is implemented on this project. So far we have implemented the OLTP gremlin API... (see #46 for details)

Closing this issue after testing the library using the Apache Tinkerpop 3.2.4 release


Add generate proof

Current dependencies on/for this PR:

main

PR #11   👈

This stack of pull requests is managed by Graphite.


Multi-select validation error is inconsistent with others

Describe the bug
Several question types allow admins to set bounds for the answer:

ID - min/max length
Checkbox - min/max selections
Number - min/max value
Text - min/max length
Enumerator - min/max values (pending in #7136)

For all of these, the error messages if you enter an invalid value are something like:

Must contain at least {0} characters
Must contain at most {0} characters

However, for the checkbox question, the upper bound error message is "Please select fewer than {0}", where {0} is the upper bound + 1. This is weird technically, and could be confusing to applicants who have seen the other error message and might mis-interpret this one.
To Reproduce

Create a program with a checkbox question with 2 choices and a "maximum choices allowed" of 1
Apply to the program, select both checkbox options, and click next

Expected behavior
The error message should say "Please select at most 1" rather than "Please select fewer than 2".
Screenshots

I am also going to use this bug as a test case for changing localized strings, to validate the documentation at https://github.com/civiform/civiform/wiki/Internationalization-(i18n)#changing-existing-strings

This is blocked on translations for "validation.tooManySelections.v2".

Translations are ready: #7918


[RDBMS] az postgres flexible-server migration: Use Cloud supplied URL's rather than hardcoded management URL's

Description
Earlier, The management URLs were hardcoded because of which the CLI was unusable in the national clouds.
With this change, the CLI will work for all clouds when the user selects the cloud.
Testing Guide
Tested manually in the local environment.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

RDBMS

Just wanted to make sure if this one is tested in the mooncake region?

Just wanted to make sure if this one is tested in the mooncake region?

@DaeunYim , the FMS based migrations are not available in Mooncake region right now. Since we have removed the hardcoded management URL's, should it work out of the box?


Organization plugins

Just a reaction to seeing the organization secrets PR, a related issue is organization-level plugins i.e. to add default notification plugins.
We've run straight into this issue using drone currently where we have a shared mailing list which should receive build notifications, but a large number of repositories where we'd need to manually specify email servers/settings (or script it, but the maintenance overhead is substantial).
This would be an extra field against an organization which pointed to a git repository which contained the shared .drone.yml file for the organization. A refinement of the concept might allow for a folder structure to customize this per repository.

this has come up in the past and we've declined to implement this feature. It comes with quite a bit of complexity and edge cases, and we felt the complexity was justified.


Results fetched but error dialogue prompt says otherwise

Search for dehydrogenase leads to an error prompt. However, the underlying API endpoint does retrieve formatted results.

Fixed by using the first element of the ko_number array.

I believe this is the same as #333.