index optimize procedure with update statistics  on memory optimized tables fails

Description of the issue
Date and time: 2024-09-19 09:30:44
Server: SQLServer\Instance1
Version: 16.0.4085.2
Edition: Enterprise Edition: Core-based Licensing (64-bit)
Platform: Windows
Procedure: [DbaTools].[dbo].[IndexOptimize]
Parameters: @Databases = 'DB', @FragmentationLow = NULL, @FragmentationMedium = NULL, @FragmentationHigh = NULL, @FragmentationLevel1 = 5, @FragmentationLevel2 = 30, @MinNumberOfPages = 1000, @MaxNumberOfPages = NULL, @SortInTempdb = 'N', @MaxDOP = NULL, @FillFactor = NULL, @PadIndex = NULL, @LOBCompaction = 'Y', @UpdateStatistics = 'ALL', @OnlyModifiedStatistics = 'Y', @StatisticsModificationLevel = NULL, @StatisticsSample = NULL, @StatisticsResample = 'N', @PartitionLevel = 'Y', @MSShippedObjects = 'N', @Indexes = NULL, @TimeLimit = 600, @Delay = NULL, @WaitAtLowPriorityMaxDuration = NULL, @WaitAtLowPriorityAbortAfterWait = NULL, @Resumable = 'N', @AvailabilityGroups = NULL, @LockTimeout = NULL, @LockMessageSeverity = 16, @StringDelimiter = ',', @DatabaseOrder = NULL, @DatabasesInParallel = 'N', @ExecuteAsUser = NULL, @LogToTable = 'N', @Execute = 'Y'
Version: 2022-12-03 17:23:44
Source: https://ola.hallengren.com
Date and time: 2024-09-19 09:30:44
Database: [DB]
State: ONLINE
Standby: No
Updateability: READ_WRITE
User access: MULTI_USER
Recovery model: SIMPLE
Is accessible: Yes
Msg 41317, Level 16, State 0, Line 1
A user transaction that accesses memory optimized tables or natively compiled modules cannot access more than one user database or databases model and msdb, and it cannot write to master.
Completion time: 2024-09-19T09:30:47.0307071+02:00
SQL Server version and edition
Execute SELECT @@VERSION
Version: 16.0.4085.2
Edition: Enterprise Edition: Core-based Licensing (64-bit)
Version of the script
--// Source:  https://ola.hallengren.com                                                        //--
--// License: https://ola.hallengren.com/license.html                                           //--
--// GitHub:  https://github.com/olahallengren/sql-server-maintenance-solution                  //--
--// Version: 2022-12-03 17:23:44                                                               //--
What command are you executing?
EXECUTE dbo.IndexOptimize
@Databases = 'SentryOne',
@FragmentationLow = NULL,
@FragmentationMedium = NULL,
@FragmentationHigh = NULL,
@UpdateStatistics = 'ALL',
@OnlyModifiedStatistics = 'Y',
@TimeLimit = 600
What output are you getting?
Msg 41317, Level 16, State 0, Line 1
A user transaction that accesses memory optimized tables or natively compiled modules cannot access more than one user database or databases model and msdb, and it cannot write to master.

Hi,
I have the same problem.
SQL Version 16.0.4131.2
IndexOptimize Version: 2022-12-03 17:23:44
IndexOptimize SP, CommandExecute SP are stored in the DB where I get the error msg.
Script call:
USE DB
GO
EXECUTE dbo.IndexOptimize
@Databases = 'DB',
@FragmentationLow = NULL ,
@FragmentationMedium = NULL ,
@FragmentationHigh = NULL ,
@UpdateStatistics = 'ALL' ,
@OnlyModifiedStatistics = N'Y' ,
@LogToTable = N'Y';
Result:
Msg 41317, Level 16, State 0, Line 3
A user transaction that accesses memory optimized tables or natively compiled modules cannot access more than one user database or databases model and msdb, and it cannot write to master.

Hi,
It's necessary to add in code of optimize: "AND tables.is_memory_optimized = 0"
      IF @PartitionLevelStatistics = 1
      BEGIN
        SET @CurrentCommand = @CurrentCommand + ' OUTER APPLY sys.dm_db_incremental_stats_properties(stats.object_id, stats.stats_id) dm_db_incremental_stats_properties'
      END

      SET @CurrentCommand = @CurrentCommand + ' WHERE objects.[type] IN(''U'',''V'')'
                                                + CASE WHEN @MSShippedObjects = 'N' THEN ' AND objects.is_ms_shipped = 0' ELSE '' END
                                                + ' AND NOT EXISTS(SELECT * FROM sys.indexes indexes WHERE indexes.[object_id] = stats.[object_id] AND indexes.index_id = stats.stats_id) AND tables.is_memory_optimized = 0'


Cannot use any voice model on a mac M3

Hi there,
I really like this repo and would lik to use it, but I can't. I have installed everything and I started the program with this sell command "sh ./run.sh" (i made the pip install torch and everything). When I am on the web page I cannot load voice models as you can see :

The model selector is blank even if I have downloaded all the assets available on the RVC repo that are indicated. Even if a hit the button "refresh models" the selector remain blank. When I hit this button I have this message "2024-01-10 16:31:35 | INFO | httpx | HTTP Request: POST http://localhost:7865/api/predict "HTTP/1.1 200 OK"" that is written in the terminal.
Also I am working in a proper conda environment python=3.10.
How can I use the repo with my apple m3 mac please :)

same issue with you

not sure if this could solve your issue.
But I had the similar problem once with another project.
I used Metal Api with pytorch then everything worked out fine

@haseebsultankhan I tried your method but it did not work :// so sad I can't use RVC 😪

Nobody have any solution?

same issue

Somebody knows who contact to solve this issue?

same issue


Bot c#: How can I store and load all conversations from iframe

I've inserted my Bot in xamarin app, and I'm using iframe for that. But when I start conversation with the bot it always starts new conversation. I want to keep the whole history in the chat. My question is where should I store the conversation and how can I load them when user opens the chat?

hi @msBot1111 - can you provide more detail about your issue? for example:
Environment

Operating System:

Windows
macOS
Linux

SDK Platform:

C#/.NET
Node.js

SDK version:
Development environment:

localhost
Azure Web App
Azure Bot Service App
other

If you are using the Node.js SDK:

Node.js version:
NPM version:

Issue
What the issue is, in broad strokes. Link to Stack Overflow post.
Example Code
The code snippet or complete bot example (preferred) that demonstrates the issue.
Steps to Reproduce
Please provide the shortest amount of steps to reproduce your issue.
Expected Behavior
What you expected to happen.
Actual Results
What actually happened. Please give examples and support it with screenshots, copied output or error messages.

I'm using Bot Framework c# for my Bot, and development environment is Azure Web App. I have also Xamarin Forms App and I've added my Bot into my app with iframe. When I open the chat with Bot it's working well, my Bot returns the answers. My problem is that I'm not keeping the conversations. Every time I reopen the chat it gives me new conversation. Is it possible to keep history of conversations per user and when I reopen the chat to load all conversations with that user as it is in skype channel.

@msBot1111 This may help you get started on the right path here https://blog.botframework.com/2017/05/05/Saving-Bot-Actions-On-Azure-Sql-Server/

https://blog.botframework.com/2017/05/05/Saving-Bot-Actions-On-Azure-Sql-Server/
I've stored the conversation as it is in the example, but now how can I load the conversation when the chat is opened?

@msBot1111 I do not have an exact code example but, in your messageController, in the HandleSystemMessage() method you could load it all up and display it instead of a welcome message like below
            else if (message.Type == ActivityTypes.ConversationUpdate)
            {
                // Handle conversation state changes, like members being added and removed
                // Use Activity.MembersAdded and Activity.MembersRemoved and Activity.Action for info
                // Not available in all channels
                IConversationUpdateActivity iConversationUpdated = message as IConversationUpdateActivity;
                if (iConversationUpdated != null)
                {
                    ConnectorClient connector = new ConnectorClient(new System.Uri(message.ServiceUrl));

                    foreach (var member in iConversationUpdated.MembersAdded ?? System.Array.Empty<ChannelAccount>())
                    {
                        // if the bot is added, then
                        if (member.Id == iConversationUpdated.Recipient.Id)
                        {
                            var reply = ((Activity) iConversationUpdated).CreateReply(
                                $"Hi! I'm a Spotify bot! Ask me about your favorite artist.");
                            await connector.Conversations.ReplyToActivityAsync(reply);
                        }
                    }
                }
            }


Explicitly include hubspot form each time it is used

Description
Tweaks Hubspot form code to be only included on pages it's actually used.
Additionally, it sets the async flag, and renders the form using the onload callback so it's not page load blocking.
Please verify this is behaving as expected locally as part of review.
Related Issue(s)
#401
Checklist

[x] I have read the contribution guidelines
[x] I have considered the performance impact of these changes
[-] Suitable unit/system level tests have been added and they pass
[-] Documentation has been updated

verified - working locally - thanks Pez


Feature Request: Support for Inline Meta Fields

Dataview's inline meta fields have become a popular method of inserting metadata within the content of a note. Unfortunately, it doesn't appear that these fields are recognized by the plugin. I think it would be useful for many users if support was added for this, particularly those who would like to be able to see a comprehensive table of meta fields at the top of their notes without having to copy relevant attributes found within the body to the frontmatter.

Hey @lolfert sounds like something to explore. My initial reservation is that it would likely be depending on dataview itself to ensure compatibility which might be a bit too much. I'll think about it though, thanks!


cdist is very slow if custom weight vector is supplied

Is your feature request related to a problem? Please describe.
Several distance metrics in cdist take an optional weight vector by which to scale the input vectors, e.g. sqeuclidean. The issue is that supplying such a vector causes significant performance degradation in terms of computational time.
The underlying bottleneck seems to be the result of the data validation done on the weight vector. The function _validate_vector in distance.py is called every time the cdist function is invoked. When cdist is used in an optimization problem with potentially many iterations, _validate_vector will be called myriads of times, essentially for no good reason.
The work-around I am currently using is to manually re-scale the input vectors and then supply them to cdist with the default None value for the weight vector. The performance increase is staggering.
Describe the solution you'd like
Wouldn't it be the Pythonic way to simply use the supplied weights vector as it is, and if malformatted raise an appropriate exception, so that the user makes sure the right data type is supplied? As it stands currently, validating data types and squashing to required dimensions is just too time consuming in the context of an optimization problem.
Describe alternatives you've considered
Manually re-scale the input vectors by the custom weight vector and use cdist with no optional arguments.

Actually most functions are doing data validation on the inputs. So it is true that in an optimization problem, there would possibly be a lot of unnecessary operations. A possible way to mitigate this would be to have a way to deactivate data validation. I don't see an obvious way to deal with this problem. We could think about some mechanism to globally deactivate all sort of validation, using a defined signature for all validators and mocking could do the trick even if I am not sure about the overhead.

Thank you for your answer @tupui. I wouldn't go as far as proposing a global switch to turn off all data validation. In this particular case, however, I do believe that not doing any data validation at all on the weights is a good default case.
I think that any exception arising from invalid weights (e.g. NaN/non-numeric values, etc.) will be easy to track down by the user.  If a user has made it that far to require custom weights, they must know what they're doing, consequently can validate the weights they're supplying.
I'd prefer this to having an extra switch that turns off weight validation, simply because it'll complicate the function signature. But that's really a matter of preference. As it stands, though, I do believe that cdist  is an unexpected bottleneck in scenarios where it's most needed.

Well I played a bit around and this is actually coming from the fact that applying the weights is an operation done on the metric function itself. So it is done for every points, not just once at the end. Hence the huge slowdown...
So here there is nothing about the validation. To mitigate this, we would need to move the weight part after the computation of the metric. This means separating the metric calculation from applying the weights. Doable, but the strategy being different for the metrics, multiple functions have to be created for that.

If you use sqeuclidean, there is no validator defined in _METRICS. It means that the function will be run again and get called point by point... I am actually not sure if this is wanted. I will dig more.
So right now, some extra operations due to the weighting is done per points. Hence the slow down.

Ok it's not about the validator sorry. It's due to _select_weighted_metric. sqeuclidean is changed here into test_sqeuclidean and thus the function is called on every points. Still not sure why this is happening.

When there is no C version, the test mode is used with the python version which is passed as a callable. For the test version, there is no vectorization and the function is called for every points.
Looking at the whole function, it could probably be refactored. First to not oblige to have a C version in order to act on all points at the same time, and then to separate the tests from here.
cc @rgommers

If you use sqeuclidean, ~there is no validator defined in _METRICS~. It means that the function will get called point by point... I am actually not sure if this is wanted. I will dig more.
So right now, some extra operations due to the weighting is done per points. Hence the slow down.
If you try seuclidean you will see absolutely no slow down as the validation is done once and computation is done for all points at once.

Unfortunately, I do need sqeuclidean, because it supports individual weights for each vector dimension. seuclidean does not.

The work-around I am currently using is to manually re-scale the input vectors and then supply them to cdist with the default None value for the weight vector. The performance increase is staggering.

Could you share some timings, or benchmarking code? Staggering can mean 20% or 20x, would be nice to have a bit more detail.

Looking at the whole function, it could probably be refactored. First to not oblige to have a C version in order to act on all points at the same time, and then to separate the tests from here.

spatial.distance is a bit of a mess indeed. Two large rewrites have been attempted over the years, but none of them was completed. There's a lot of performance to gain, as well as design consistency. I know @peterbell10 has some interest in working on this topic.

Could you share some timings, or benchmarking code? Staggering can mean 20% or 20x, would be nice to have a bit more detail.

Hi @rgommers Unfortunately, it'd be quite time-consuming for me to post a minimal working code example based off of my actual application. I have definitely measured at least one order of magnitude slow-down with vectors of moderate size, e.g. (20000 x 50).
I'm sure that any test with similarly sized vectors can show the performance difference with vs without custom weights.

spatial.distance is a bit of a mess indeed. Two large rewrites have been attempted over the years, but none of them was completed. There's a lot of performance to gain, as well as design consistency.

I'd like to help out the community, but don't have the bandwidth at the moment. Hopefully just reporting issues would help to prioritize internally.

Here is a straight-forward example:
In [1]: import numpy as np
   ...: from scipy.spatial.distance import cdist
   ...: a = np.random.rand(100, 10)
   ...: w = np.random.rand(10)
   ...: %timeit cdist(a, a, metric='euclidean')
   ...: %timeit cdist(a, a, w=w, metric='euclidean')
337 µs ± 3.77 µs per loop (mean ± std. dev. of 7 runs, 1000 loops each)
173 ms ± 1.45 ms per loop (mean ± std. dev. of 7 runs, 10 loops each)

We can see that introducing weights results in a 3 order of magnitude slow-down. Exactly because it falls-back to calling 'test_euclidean' individually for each x[i], y[j] pair in the distance matrix.
There is a similar performance cliff for pdist as well.
I have time to work on this and am interested in basically re-writing spatial.distance. Ideally, the distance metric should be implemented once in C++ for weighted and non-weighted, and there would be infrastructure to generalize that automatically to nd-arrays, cdist and pdist automatically.


docs: update docs, remove bcs-saas

Codecov Report

Merging #42 (364518e) into master (8379c1f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #42   +/-   ##
=======================================
  Coverage   52.88%   52.88%
=======================================
  Files          19       19
  Lines        1004     1004
=======================================
  Hits          531      531
  Misses        406      406
  Partials       67       67

Continue to review full report at Codecov.

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


feat: characterise independence in compactly-generated lattices

Also some loosely-related short lemmas

Split out from #7963

bors merge


Multi Instagrams don’t work.

Describe the issue
I installed instagram app for first and second accounts. But there installed apps share the data of login account.
Instructions to reproduce

Tap +.
Serect Instagram.ipa.
Install again Instagram.ipa for second account.
Serect “Keep both, don’t share data”.

What version of LiveContainer are you using?
2.1.2-release main
Other
No response

Unfortunately, keychain is not separated at the moment due to its complexity.


Fix Pillow dependency

Recent versions of Pillow break the tests, released versions 8.3.0 and 8.3.1 must be avoided.
a fix has been written for the Pillow library, but it is not released yet.
https://github.com/python-pillow/Pillow/commit/01e423da00c7039e447a477c21a7414a2ccae5e1

Thanks for this!


KeyImage Generation for Change Subaddress

Motivation
Updating bindings to fix KeyImage generation
In this PR

Update to native libs
Fix for test that now would cause native exception

@dolanbernard please link the corresponding PR for the android-bindings changes

Relevant PR for android bindings


Android AAR project

Instead of having the Android Playground app, split in an shared BabylonNative AAR library and a smaller app.
This change consists in adding a new Module project to the playground. Moving code into that new module: JNI layer, Java view and Wrapper, cmake to make the jni.so. Java namespace renamed to make it easier to integrate. And finally, add a dependency to the .aa in the app.
Min Android SDK version bumped to v25.
Running Android Playground App is the same: open with Android Studio then run app on device.
Produced .aar can be shared (like a npm, nuget, or local file ...) and integration will be way simpler: no need to clone BN, modify  app gradle with cmake, add .cpp, etc...
Integration/doc will be part of another PR.
fixes #1435
cc @okwasniewski

was this waiting on something that we can help with?

For #4, I think @okwasniewski had a plan to make BN a cocoapod package for iOS and visionOS, following the same pattern of adoption into BRN as this PR does for Android AAR.

I've open this issue to centralize questions/technical points : https://github.com/BabylonJS/BabylonNative/issues/1446

Hey @CedricGuillemet are you planning to integrate this into Babylon React Native?

Hey @CedricGuillemet are you planning to integrate this into Babylon React Native?

Yes, it's in my todo list : https://github.com/BabylonJS/BabylonNative/issues/1446
I'd like to do the same thing with pod too (see next steps)


Validation on init causing double render error.

The observer in the bs-form that validates the model on init causes a double render in some cases.  Perhaps it should be configurable?   Specifically if you render a form in a bs-modal.body and reference the changeset in the bs-model.header or footer.

Thanks for mentioning. Do you have some example code that reliably reproduces that issue?

Yes I have an application that is experiencing this issue.  I will try a recreate it with a twiddle.  I have removed the on init observer in my application as I believe it is redundant.  The attributes are validated on mutation and the whole model is validated before submit.

@arenoir would I be right in assuming you have bindings to changeset.isDirty?
This is a nasty one as changeset#validate will actually notifyPropertyChange(CHANGES) always, regardless of actual changes, which will fire further updates on isPristine and in turn isDirty. Hence, anything bound on changeset.isDirty will fire twice in the render cycle. I'm guessing using an observer in bs-form#_initValidation doesn't help either, but this is more likely ember-changeset's fault for preemptively calling notifyPropertyChange(CHANGES).

https://github.com/kaliber5/ember-bootstrap-changeset-validations/pull/13


separate cmnd, stat and custom topic values

I need to separate receiving commands and statuses from switches and relays. In tasmota, this is done with the switchtopic topic_to_listen command. Then the values from switches come in mqtt.0.cmnd.topic_to_listen.POWER1, without switching at the relay. I can control the relay through the topic mqtt. 0. cmnd.tasmota.POWER1. To obtain the current status of a topic using mqtt.0.stat.tasmota.POWER1. Is it possible to implement this in this adapter?

up

up


Use registry.default for login/logout

This changes cargo login and cargo logout to use the registry configured at registry.default as the registry instead of crates.io. For cargo login, this was an unintentional regression from #6466. The documentation has always stated that it will use the default registry.
This makes the command more in line with other registry-involving commands. There are still some inconsistencies.
These commands use the default if not specified:

cargo init
cargo new
cargo owner
cargo search
cargo yank
cargo publish uses the default, but will also look at the publish field Cargo.toml and use that if the registry is not specified.

These commands would always use crates.io if --registry is not specified:

cargo login
cargo logout
cargo install

I'm a bit uncertain how to proceed, since this is technically a breaking change particularly if someone has scripted it. I suspect that the number of users that use registry.default is very small, and those that script cargo login are even smaller, and thus the intersection is probably small or nonexistent. However, there is some risk here.

FYI If using the registry extension function is all that is needed, it looks like cargo add respects registry.default

I'm a bit uncertain how to proceed, since this is technically a breaking change particularly if someone has scripted it. I suspect that the number of users that use registry.default is very small, and those that script cargo login are even smaller, and thus the intersection is probably small or nonexistent. However, there is some risk here.

Technically, any behavior change from any big fix could be a breaking change.  Since this was the former login behavior, the documentation still says it works that way, and the use case seems small, I'm inclined to say let's merge this.
However, I think it would be good to do a straw poll on whether we should risk breaking some people with this bug fix.  Note that logout is not yet stablized (#11950) so only login users would be potentially broken.
@rfcbot fcp merge

Error messages that reference running cargo login need to be updated to take this into account. If crates-io isn't the default, then it will need to include --registry crates-io
https://github.com/rust-lang/cargo/blob/c429784fd5e456705b539ee7752bf6cfbe3f8d00/src/cargo/util/auth.rs#L361

Error messages that reference running cargo login need to be updated to take this into account.

Sure! I have pushed an update for that error message.

:pushpin: Commit a9e0b505d640bdb4b90952f4e69f169bdadafb16 has been approved by epage
It is now in the queue for this repository.

:sunny: Test successful - checks-actions
Approved by: epage
Pushing 96f8d6ced858a687db2ef53f44f531517ef5c6c2 to master...


(fix) exclude ATs from sonar cloud

Signed-off-by: Frank Li<EMAIL_ADDRESS>

PR description
Excludes ATs from sonar cloud runs
Fixed Issue(s)
fixes https://github.com/hyperledger/besu/issues/3142

Ok so sonar cloud on 8f02b9c flags code in the acceptance-tests folder that doesn't directly run tests as uncovered code.

Nothing flagged after adding the config option

Note to myself while I remember - we also need a to test that coverage
works for non AT files (don't think that'd be an issue but just in case)
On Sat, 11 Dec 2021 at 11:38 am, sonarcloud[bot] @.***>
wrote:

Kudos, SonarCloud Quality Gate passed!    [image: Quality Gate passed]
https://camo.githubusercontent.com/e4a90cf346b46d44c5652111d68603714bfb7693043d07a95992de3a30bfce04/68747470733a2f2f736f6e6172736f757263652e6769746875622e696f2f736f6e6172636c6f75642d6769746875622d7374617469632d7265736f75726365732f76322f636865636b732f5175616c6974794761746542616467652f7061737365642d313670782e706e67
[image: Bug]
https://sonarcloud.io/project/issues?id=hyperledger_besu&pullRequest=3149&resolved=false&types=BUG [image:
A]
https://sonarcloud.io/project/issues?id=hyperledger_besu&pullRequest=3149&resolved=false&types=BUG 0
Bugs
https://sonarcloud.io/project/issues?id=hyperledger_besu&pullRequest=3149&resolved=false&types=BUG
[image: Vulnerability]
https://sonarcloud.io/project/issues?id=hyperledger_besu&pullRequest=3149&resolved=false&types=VULNERABILITY [image:
A]
https://sonarcloud.io/project/issues?id=hyperledger_besu&pullRequest=3149&resolved=false&types=VULNERABILITY 0
Vulnerabilities
https://sonarcloud.io/project/issues?id=hyperledger_besu&pullRequest=3149&resolved=false&types=VULNERABILITY
[image: Security Hotspot]
https://sonarcloud.io/project/security_hotspots?id=hyperledger_besu&pullRequest=3149&resolved=false&types=SECURITY_HOTSPOT [image:
A]
https://sonarcloud.io/project/security_hotspots?id=hyperledger_besu&pullRequest=3149&resolved=false&types=SECURITY_HOTSPOT 0
Security Hotspots
https://sonarcloud.io/project/security_hotspots?id=hyperledger_besu&pullRequest=3149&resolved=false&types=SECURITY_HOTSPOT
[image: Code Smell]
https://sonarcloud.io/project/issues?id=hyperledger_besu&pullRequest=3149&resolved=false&types=CODE_SMELL [image:
A]
https://sonarcloud.io/project/issues?id=hyperledger_besu&pullRequest=3149&resolved=false&types=CODE_SMELL 0
Code Smells
https://sonarcloud.io/project/issues?id=hyperledger_besu&pullRequest=3149&resolved=false&types=CODE_SMELL
[image: No Coverage information]
https://sonarcloud.io/component_measures?id=hyperledger_besu&pullRequest=3149&metric=coverage&view=list
No Coverage information
[image: No Duplication information]
https://sonarcloud.io/component_measures?id=hyperledger_besu&pullRequest=3149&metric=duplicated_lines_density&view=list
No Duplication information
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/hyperledger/besu/pull/3149#issuecomment-991405721,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJMWR424OY4YNGIBCMRFTMDUQKTRDANCNFSM5JX4FHHA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Ok I've tested:

File under acceptance-tests with no tests, coverage tested (not good)
File under acceptance-tests with no tests, new config, coverage not tested (what we want)
File outside of AT, new config, coverage still tested (good)


Scenarios with ambigious solutions with respect to the spec?

This is the scenario: https://github.com/astral-sh/packse/blob/0.3.12/scenarios/prereleases.json#L320
It's listed as:
"expected": {
    "satisfiable": false,
    "explanation": "Since the user did not explicitly opt-in to a prerelease, it cannot be selected."
}

But there are many ways to interpret the spec, especially given the spec does not explicitly consider resolution. Currently pip resolves this scenario with: a==1.0.0 b==1.0.0 c==2.0.0b1.
What is packse's intent here? To set tests against the design principles of uv or against the spec?
Might it worth "expected" being a list of possible options when facing ambigious scenarios like this?

Yeah this is tough...
Ideally we'd test against the specification rather than uv's design principles. I'm not sure what we should do when the spec is ambiguous. Ideally the spec wouldn't be ambiguous so maybe we should propose changes.
I worry about allowing multiple expected outcomes, it sounds complicated but perhaps that's the right solution. We could also consider some sort of "feature flag" style annotation for scenarios outcome where, for example, we have two pre-release modes explicit and implicit and the scenario is tagged as such so it can be filtered out of implementations based on the mode they support.

I would be comfortable with a mode that meant something like "allow package to be marked prerelease by transitive dependency" and when False then pip wouldn't test against it.


Control panel disappeared in insider version

I used to have a stable version of jypyter and I had a control panel to see variables and other execution features but it looks like all of them disappeared in the insider version.

Hi @xsoheilalizadeh, the toolbar in insiders is actually located in the top right.

Thanks @claudiaregio, but it hasn't all the features like the variables table.

We are still working on feature parity with the new Native Notebook experience, variable explorer coming back soon :)

Any update on when the variable explorer will be coming back to Insiders? I still do not see it.

Any update on when the variable explorer will be coming back to Insiders? I still do not see it.

It is being worked on right now, hoping to ship in the next two releases!

It is being worked on right now, hoping to ship in the next two releases!


circular dependency

pyproject.toml
[build-system]
requires = ["poetry-core>=1.0.0a3"]
Poetry requires cleo and cleo requires poetry-core.  Poetry-core does not provide poetry_core. After installing poetry-core from wheel import poetry_core fails. How can cleo be installed from source?

Related https://github.com/python-poetry/poetry/issues/1975

Related https://github.com/python-poetry/poetry/issues/1975

This was a bug when poetry-core was refactored from poetry_core -> poetry.core in https://github.com/python-poetry/poetry-core/pull/3. It's since been fixed!


Change docs/4_extending.srt

Fix documents about extending functions.

Hi @SuperBo - I am not sure what exactly this is fixing. Is there an issue with the current documentation?
Function and AnalyticFunction can both be imported from the top level as specified in the existing documentation e.g. from pypika import Function.

@mikeengland : You can' t import them directly with latest version (0.42.1)

ImportError: cannot import name 'Functions' from 'pypika' (/Users/y.nguyen/Workspace/venv/data/lib/python3.8/site-packages/pypika/init.py)

My mistake, thanks for fixing!


Sandbox icon is missing and thumbnail is the wrong size

Dev Home version
No response
Windows build number
No response
Other software
No response
Steps to reproduce the bug
Have Windows Sandbox enabled
Navigate to environments page
Expected result
The thumbnail image should be the same size as the others and there should be an icon to the left of "Windows Sandbox"
Actual result
Looks like the thumbnail is at a different aspect ratio and there's no icon:

Included System Information
No response
Included Extensions Information
No response

@cinnamon-msft  Can you check if you have multiple versions of Dev Home installed like the Dev version and the Canary version? If so can you make sure the sandbox extension for the dev version is turned off. That might fix the icon missing issue. I do believe I've seen the icon showing up before.
About the thumbnail: @pehammer would either you or Christian be able to get us a new image for sandbox scaled appropriately just like we did for Hyper-V and Dev Box? Looks like we're using the light version of bloom here for sandbox

I don't have a Dev version on this machine, just Preview and Canary, and Preview doesn't have Sandbox yet

Sounds, good thanks Kayla just wanted to make sure it wasn't one of those multiple dev home version issues.  Would you also be able to add the sandbox log file and the dev home log file to this issue as well when you can thanks!

Here you go!
devhome20240606.zip

Thanks, looks like the issue is that the build constants are not present in the sandbox project e.g for Hyper-V:
we have this line in the csproj file. It declares STABLE_BUILD and CANARY_BUILD as constants for the project. The Sandbox uses these same constants in this file just like we do for HyperV here. However since those constants aren't declared in the sandbox csproj file, the ifdef's go into the else case, where it looks for this file:
ms-resource://microsoft.windows.devhome.dev/Files/Assets/windows-sandbox-icon.png
and you'll notice that the package its using is microsoft.windows.devhome.dev , which like you said above doesn't exist on your machine. So therefore an image isn't found.
This should be a simple fix though, we just need to declare those constants in the sandbox csproj file in here somewhere: https://github.com/microsoft/devhome/blob/9db92955056f1be932ee500c876b6f62ff626c3b/extensions/WindowsSandboxExtension/WindowsSandboxExtension.csproj

Sandbox extension has been removed, so this issue is no longer applicable.


wayvnc: 0.5.0 -> 0.6.1, enable checks

Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[x] riscv64-linux (cross)
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

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

3 packages built:

aml
neatvnc
wayvnc

Thanks @NickCao! :)
Do you actively use these packages and would you happen to be interested to replace me as the Nixpkgs maintainer of these packages? I haven't used them in quite a while and I'm looking for a new maintainer.

I found them handy for quick screen sharing. Happy to adopt them on the next update.


A crash occurs when the app starts

New installation, after opening the app to authorize root, it will crash back.
log:
https://paste.ubuntu.com/p/qN89g3Md7V/

Should i fixed it and create pull request ?

Duplicate of #3


Add user path to runtimes search

Proposed Changes

Use the runtimes default paths and the user path when searching for the runtimes

Types of Changes

Feature

Verification

Testing

Linked Issues

https://github.com/k3s-io/k3s/issues/10761

User-Facing Change

Runtimes detection will now use $PATH

Further Comments

I think this will be a good idea, since we will also detect runtimes in the entire user path

/trivy

/trivy

/trivy


Missing location for az network private-endpoint create

In the current example az network private-endpoint create creates the endpoint in the default region, which may be incorrectly set. When you query DATA_ENDPOINT_PRIVATE_IP, you use $REGISTRY_LOCATION explicitly.
I would suggest updating the docs by adding --location $REGISTRY_LOCATION to az network private-endpoint create.

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

ID: 8e01945a-5892-a416-716f-c499f110586c
Version Independent ID: e2aef346-266f-3e69-842d-58004084609b
Content: Set up private endpoint with private link - Azure Container Registry
Content Source: articles/container-registry/container-registry-private-link.md
Service: container-registry
GitHub Login: @dlepow
Microsoft Alias: danlep

Sorry, my bad, it was the registry location that messed up the process, not the endpoint location.


Mutated Issue: "Splice Region" always get picked as variant class

When having multiple consequences and if splice_region_variant is presented, annotator/GN always picks/adds "Splice Region" as the variant class, even if other consequences are "3'UTR", "5'UTR", "intron" and no protein change annotation. Importer doesn't filter "Splice Region" variant class and these variants eventually got imported as "MUTATED".
For example,
prad_p1000

cellline_ccle_broad

@ino and @n1zea144 suggested maybe we could use "HGVSc" as protein change for these cases?
For more examples and complete "mutated" variants stats, please check here:
https://docs.google.com/spreadsheets/d/1RFj8zzMPv4PCAs3sFM9ro3FP4vRg-Za-JI2AFlx6FHg/edit#gid=1944280727

Updated link
For more examples and complete "mutated" variants stats, please check here:
https://docs.google.com/spreadsheets/d/1WjAjwZdzkbs1eUwSC-QVCyhDf-GNRdWT/edit#gid=2065093988


Initializing carbon components on reactive DOM update

This issue is related to #441, #518, and a few others.
I'm trying to use Carbon Components with Vue.js, but I'm having trouble with components not being properly initialized. I tried running CarbonComponents.watch(), but nothing changed. Similarly, in the update cycle, to try to get a data table to work, I tried:
    let e = document.getElementsByClassName("bx--data-table-v2-container")[0];
    let a = CarbonComponents.DataTableV2.create(e);
    console.log(a);

However, the table created has 0 expandable rows and doesn't work at all, even though when I print out the table element (e), I can see that it was fully initialized with all rows an headers. What am I doing wrong? I also tried grabbing the element bx--data-table-v2, which didn't work (but I don't think that one is right).
Edit: I've also gotten e with document.querySelector('[data-table-v2]'), to get the container element, which didn't work

I should also include the error generated when I interact with my table:
Uncaught TypeError: Failed to execute 'insertBefore' on 'Node': parameter 1 is not of type 'Node'.
    at t._rowExpandToggle (carbon-components.min.js:2)
    at t.value (carbon-components.min.js:2)
    at t.value (carbon-components.min.js:1)
    at t._toggleState (carbon-components.min.js:2)
    at HTMLDivElement.<anonymous> (carbon-components.min.js:2)

Ah, I see that I just have to call refreshRows and it works. What really confused me is that create wasn't doing anything to update the instance (as far as I can tell), just grabbing a reference and returning it.
I'm going to leave this issue open for a bit because I'd really appreciate correction if I've misinterpreted something, but will close it in a few days if I hear nothing.

Hi @mtn great to see that you figured it out yourself! You are right that .create() is just a factory method to create a instance. .refreshRows() is a table-specific method, which has to be called upon any dynamic row addition/removal. Feel free to tell us if you need any further clarification/fix. Thanks!


update packages & refactor code

Just refactored some code & updated the packages.  will continue on this once my personal things get caught up.

Hey @afkcodes it's been a while since any progress has been made on any of the projects in the developersIndia GitHub org. We had a long discussion on what the future of any of the current & prospective projects in the org entails. So, please subscribe to any updates we share here or in our Discord server.
On that note, this repository needs to be cleaned up & refactored quite a lot, so let me know if you would still like to proceed ahead with the PR?

Yeah will do that, also I want to be an active contributor to the project, as this whole project goes dormant for a long time. This is the best way we can help beginners and others as well. I also checked reached out to folks but no response.

Yeah will do that, also I want to be an active contributor to the project, as this whole project goes dormant for a long time. This is the best way we can help beginners and others as well. I also checked reached out to folks but no response.

Understandable. We had a long chat about the future of all current & prospect projects in the developersIndia community. So, we'll trying to revive contributions to the project once again this time with serious intent.
That said, please feel free to contribute to the project(s) as you see fit. Also do check out the issue threads, we're looking for a designer who can share a first iteration of the official landing page for the community. If you know someone who's willing to contribute to the project, let them know about it.

Thanks for the update I know I am not a founding member but can I join you, folks, I think I can make a difference.
Regarding the PR i will update it by tomorrow.

in my opinion, we don't need a full-fledged design right now we can start taking references from any dribble design modify the way we want and in the way, once someone is willing to contribute as a designer we will revamp.

... but can I join you, folks, I think I can make a difference

Of course! Any sort of contributions to any of the projects in this community are welcome! Please feel free to join our Discord channel for further discussions & general chit-chat.

... we don't need a full-fledged design right now ... taking references from any dribble design modify the way we want ... contribute as a designer we will revamp

That's the plan for now as well, release & deploy a VERY rudimentary landing page for the first iteration. It should be enough to get the name of the community out there. So, all that's required for now is a minimal MVP with enough SEO for Google to rank the page well.
And thank you if you could help us with your Dribble projects! None of us are designers as such we need help from others who're proficient in web design. If you would like to further discuss about design ideas, feel to respond to #6 (discussion thread).

Got the point what I was asking is that I think we didn't discuss the website being live and running again on discord please correct me if I am wrong not being so active there for a while.

Pinged you on discord, please have a look lets make this big.

I made an informal announcement last week & discussed it with the founders only today afternoon. I'm organizing certain things related to the developersIndia GitHub org along with @Bhupesh-V for now. Once it's all done, I'll make a formal announcement for collaboration efforts in our Discord server. So, keep an eye-out for it & let us know if you've any questions.
As for the "core members", there's nothing of that sort for now. developersIndia is a community & everything related to it is community-led. We, as-in me, Bhupesh, Sid & the others are just here to provide a roadmap for the community with their ideas & projects.
I suggest discussing about it in the Discord channel or the subreddit or in a GitHub Discussion to keep this PR on-topic.

@afkcodes are you still interested in pursuing this PR? If so, let me know so that I can help you guide through what needs to remain & what doesn't.

Don't you've the perms to fork the project & send a PR? If so let me know because we're in the process of reorganizing the developersIndia GitHub Organization.
Else if you want, send in a new PR & link #13 to close it. Let me know when you do it, so that I can close this current PR in favor of the new one.

lets close this PR i will create a new one.

PR has been raised #23 in support of this. lets close this.


Could not create a rule using ElastAlert2 Server API

Hi,
Thanks for forking and maintaining ElastAlert2 server. I am exploring it to be used in my class room to demonstrate alerting capability and integration with Opensearch, SOAR and a few other tools.
Currently, I am testing the integration with ElastAlert2 server's API, however I have got problem when trying to create a rule with the API via Swagger UI.
I can get the existing rules but not able to create a new rule. The 1st picture below was captured from the response of the API call "POST /rules/:id". It showed the rule was created but when getting the rules, the new rule was not created.
Here is my request body:
{
"yaml":   "name: Brute Force Logins\ntype: frequency\nindex: logstash-tenant01-syslog*\n\nrealert:\n  minutes: 2\n\nnum_events: 2\ntimeframe:\n  hours: 1\nquery_key: user\n\nfilter:\n- query:\n    query_string: {query: 'status:"password check failed"'}\nhttp_post_url: http://<IP_ADDRESS>:3001/api/v1/hooks/webhook_e80145a0-b9ea-4c5a-bf8c-b8befab2a5fd\nhttp_post_all_values: true\n"
}
I also checked on the rules directory inside ElastAlert2 server, there wasn't any rule created. So, could someone help me how to achieve my requirement?
Thank you.
1st picture:

This 2nd picture below showed existing rules:

Hi!
Probably issue with permission to rules folder.
Please check this: https://github.com/Karql/elastalert2-server/issues/48#issuecomment-1061409257
I have to fix return http code on failure so leave this issue open.

Hi Karql,
Thank you for your response to my post.
I should give more detail that I run ElastAlert2 server on Kubernetes. I could manage to run the container with uid 1000 as you can see in below pictures:
1st picture:

2nd picture:

Before I did the change, I found an error regarding permission in Kubernetes logs but after the change the error disappeared, however I still could not create any rule via Swagger UI as I checked inside the rules directory no rule was created as shown in 2nd picture. The Swagger UI gave same response as shown in 3rd picture.

Hope to have your next response.
Thank you.

Unfortunately I have no experience running it in kubernetes 😢

Hi Karql,
Thanks again for your response.
I managed to resolve the issue after finding out that Kubernetes mapped the rules directory from my configmap as read-only file system. So, I just run the container without mapping it. I then tried creating a rule, this time the rule created.
May I ask if once a new rule created, how to make the rule active? Earlier, when rules directory was mapped by Kubernetes all rules inside rules directory were active automatically.
Thank you.

Rule after add should works immediately.
rules folder should be common for server and elastatalert2
This how is look like when you enter the container:

Check where rules are stored and if config.json and elastalert.yml poiting the same folder.

Hi Karql,
Thank you for your response.
As you said, once the rule is created it will be active immediately. My SOAR tool now already received alert from ElastAlert2 as shown below.

Thank again for your help.

I am glad that you managed it 😉

Thanks again Karql for your guidance.
If anyone is interested in or has issue deploying ElastAlert2 on Kubernetes, please let me know.
Thank you.

Hello Pjancha, would it be possible to discuss some issues I'm having with ElastAlert2 on Kubernetes? Thank you!


[KOGITO-4135] Improve SEO of DMN.new and BPMN.new

JIRA link: https://issues.redhat.com/browse/KOGITO-4135

@tomasdavidorg  @domhanak Hi guys! We have consistent failures of the Chrome and VS Code extension tests on this PR for some reason. Might be a good one to try and see what can be causing these failures. Can you please take a look?

@tiagobento Chrome test fails are caused by the incompatibility of Chrome driver and Chrome browser (This branch is not rebased with the master branch.) Desktop test fail on windows is caused by some instability of the github ci machine. Desktop fail on the Mac is probably something related with connection to selenium server... I will take a look on this one more.

VSCode on mac-os fails because the dependencies are not resolved

@tomasdavidorg @domhanak Ok! Thx! Merging this PR as test failures are unrelated.


Expose a type alias of the underlying JsonValue representation

Internally in Fleece this is JsonValue, but externally this is not always re-exported.
Only for Newtonsoft it is defined as JsonValue, other implementations needs to open their specific namespace (ie: open FSharp.Data) in order to have JsonValue available.
This is normally not a big issue, but it would be nice (and possible clearer) if Fleece expose a unique type alias that maps to the specific JsonValue (or JToken) of the underlying Json lib.
It could be called JsonValue or JValue.

Sounds like a good thing!


Title should not wrap but shrink if too long

When fixed, set fontSize back to 10vw

:tada: This issue has been resolved in version 1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Increment features

If I'm understanding this object structure correctly, the value for the key in the features array is the word frequency, correct? I noticed while testing this out that no matter what the word frequency for each token in a document was, the count in the features object for any given key was always 1. This should increment the value if present in the features object, or set it to 1 if this is the first instance of it.

Any update on this and/or when the next package release will go live?

hey sorry about the delay on this one, merging now will probably bump the version soon


Update polybar.pacscript

I was getting the following error with the current pacsript implementation:
[+] INFO: Extracting polybar-3.6.3.tar.gz
[+] INFO: Running functions
	[>] Running prepare
	[>] Running build
CMake Error: The source directory "/tmp/pacstall" does not appear to contain CMakeLists.txt.
Specify --help for usage, or press the help button on the CMake GUI.
make: *** No targets specified and no makefile found.  Stop.
	[!] ERROR: Could not build polybar properly
dpkg: warning: ignoring request to remove polybar which isn't installed
[+] INFO: Cleaning up

I fixed it and successfully installed the package with my proposed changes.
Please let me know if those changes are okay or need improvements.

Fix the title.


zif_ajson

to #39
constants, types moved to zif_ajson. All methods aliased from there as well.
@larshp @mbtools What do you think about it ? Imho looks a bit messy ... is there benefits to keep reader/writer ?

Benefit 1: you can pass instance of ajson to a method, which is intended to use reader or writer only, thus it's params are defined.

Benefit 2: kind of very clear which methods are changing and which are reading only

Cons 1: it is convenient to use 1 interface, this is where development process will natively gravitate. It will probably be an "official recommendation" too, so ... will anyone use reader/writer at all ?

Cons 2: type definition. Makes sense to do it in one place ? zif_ajson. Then reader will depend on zif_ajson and zif_ajson on reader (via alieases) e.g. #41 @jrodriguez-rc

Any opinions ? :)
P.S. would I separate the interfaces if I designed the whole class initially (and not reader first, and the writer like it happened). I guess, not. Hmmm ...

I think lets just try it out 😄

I think lets just try it out 😄

OK, will keep it till tomorrow and merge, unless no more comments.
But I feel a strong desire to remove reader/writer IFs in future

OK, will keep it till tomorrow and merge, unless no more comments.
But I feel a strong desire to remove reader/writer IFs in future

I'm using ajson now in a few places and it's ok with separate interfaces. However, I find that I have to ask myself often if a lo_json is ref to read or write interface or ref to class - three possibilities. Sure, I can encode the type in the variable names. But I think it would be better to have it be part of the object (a property like the freezing). While a reader-only makes sense, you might have to read a writer-object too. I vote for combining but don't mind if it stays as is.

I'm using ajson now in a few places and it's ok with separate interfaces. However, I find that I have to ask myself often if a lo_json is ref to read or write interface or ref to class - three possibilities. Sure, I can encode the type in the variable names. But I think it would be better to have it be part of the object (a property like the freezing). While a reader-only makes sense, you might have to read a writer-object too. I vote for combining but don't mind if it stays as is.

yeap ... ok ... so, currently there are tons of aliases allowing to call methods from object itself and from zif_ajson ...
let's merge then. I'm leaving the original IFs for a while. And I've put a warning into readme so that zif_ajson is kind of recommended way. Let's see how it goes. Will have to change some code in our products as well :)))
Thanks for comments !

yeap ... ok ... so, currently there are tons of aliases allowing to call methods from object itself and from zif_ajson ...
let's merge then. I'm leaving the original IFs for a while. And I've put a warning into readme so that zif_ajson is kind of recommended way. Let's see how it goes. Will have to change some code in our products as well :)))
Thanks for comments !


[splix/app-config] app services/config, icons, and fmt

also includes shutdown for geth and connector on electron app.quit()

Closing because my git is all messed up. Will reopen a new one pronto.


Brittney/crud front

Added CRUD front end, adjusted JPA annotations and removed some todo's in backend, added CORS config to backend (I had some CORS issues with my CRUD requests).

Update/Edit not working - will resubmit


blocked roll: Flutter host test tests fail to load zircon

https://bugs.fuchsia.dev/p/fuchsia/issues/detail?id=53668

Fixed in https://fuchsia-review.googlesource.com/c/topaz/+/396700


Databases not dropping properly - weakly-referenced object no longer exists

It appears that in the Database Drop of a Site Delete process, the code is using a weakly referenced MySQL object that the GC destroys before the operation is completed. The user and additional database are usually not dropped in this circumstance. See:
Executing ee site delete devtest2.wildwingstudios.com --no-prompt
Deleting Database, devtesv1wa29GVCL, user devtesO3MpaVJ1AC
Database devtesO3MpaVJ1AC not dropped
Deleted Database successfully.
Deleting Webroot, /var/www/devtest2.wildwingstudios.com
Deleted webroot successfully
Reload : nginx     [OK]
Deleted site devtest2.wildwingstudios.com

Errors
Exception ignored in: <bound method Cursor.__del__ of <pymysql.cursors.Cursor object at 0x7ff853c4d2e8>>
Traceback (most recent call last):
  File "/usr/local/lib/python3.4/dist-packages/PyMySQL3-0.4-py3.4.egg/pymysql/cursors.py", line 41, in __del__
  File "/usr/local/lib/python3.4/dist-packages/PyMySQL3-0.4-py3.4.egg/pymysql/cursors.py", line 47, in close
ReferenceError: weakly-referenced object no longer exists

Executing sudo service nginx reload
 * Reloading nginx configuration nginx
   ...done.

I'm hitting this same error on site creation - although not so consistently. I updated the PyMySQL package and it now seems to be OK, unless I run it from Pexpect - which is the primary way I run it.

Hi, I have got absolutely the same error message while using remote mysql server. Below is full error message:
ee site delete test2.com
Are you sure, you want to delete database [y/N]: y
Deleting Database, test2_com, user test2_com
Database test2_com not dropped
Exception ignored in: <bound method Cursor.del of <pymysql.cursors.Cursor object at 0x7f985b68c2b0>>
Traceback (most recent call last):
File "/usr/local/lib/python3.4/dist-packages/PyMySQL3-0.4-py3.4.egg/pymysql/cursors.py", line 41, in del
File "/usr/local/lib/python3.4/dist-packages/PyMySQL3-0.4-py3.4.egg/pymysql/cursors.py", line 47, in close
ReferenceError: weakly-referenced object no longer exists
Deleted Database successfully.
Are you sure, you want to delete webroot [y/N]: y
Deleting Webroot, /var/www/test2.com
Deleted webroot successfully
Reload : nginx     [OK]
Deleted site test2.com
In same time I can't reproduce it while using local mysql server. Is there any know solutions for this? EE does remove mysql database from the remote mysql server, but doesn't remove mysql user.

Sorry for a delayed reply. I hope your issue is fixed by now.
If you are still facing the issue, please use http://community.rtcamp.com/c/easyengine for further support.
We are cleaning up issue tracker and closing this issue because we won't be able to contribute to this discussion further.


Isotope peak finder charge detection

Isotope peak finder fails the charge detection
Isotope pattern is recognised correclty with (2 peaks):

But the charge is recognised as 2:

Parameters:
Isotope finder module
Feature lists:	E39c10_500ms_RC2_2_2284.d chrom  sm res adap expanded  res deiso
Chemical elements:	[Element(1812542745, S:C, AN:6), Element(719377338, S:O, AN:8)]
m/z tolerance:	0.003 m/z or 3.0 ppm
Maximum charge of isotope m/z:	2
Search in scans:	SINGLE MOST INTENSE

@robinschmid i think this is fixed or?


Setup MegaLinter

Automate and limit technical debt & security in each pull request + fixes on current sources :)

I made a real PR to the main repo (I didn't want to pollute your CI during the setup^^) , let's continue here :)
https://github.com/scolladon/sfdx-git-delta/pull/569


fix: tests: esp32: drivers: spi_loopback

Fixes SPI loopback tests for Espressif boards.
Tests were previously relying on SPI label properties defined on dts files which the following PRs removed:
#48023
#48652

@glaubermaroto take a look at https://github.com/zephyrproject-rtos/zephyr/pull/48222

@glaubermaroto can you test my PR https://github.com/zephyrproject-rtos/zephyr/pull/48222 on those boards.

@glaubermaroto can you test my PR #48222 on those boards.

@galak I get the following test log:
Running TESTSUITE test_spi
===================================================================
START - test_spi_loopback
I: SPI test on buffers TX/RX 0x3ffb0389/0x3ffb136e
    Assertion failed at WEST_TOPDIR/zephyr/tests/drivers/spi/spi_loopback/src/spi.c:546: test_spi_loopback: (!spi_is_ready(&spi_slow) is false)
Slow spi lookback device is not ready
 FAIL - test_spi_loopback in 0.21 seconds
===================================================================
TESTSUITE test_spi failed.
===================================================================
PROJECT EXECUTION FAILED

Maybe it is something from my side? Some new configuration I am missing/not aware of?

Maybe it is something from my side? Some new configuration I am missing/not aware of?

what board are you testing?

@glaubermaroto can you try:
diff --git a/tests/drivers/spi/spi_loopback/src/spi.c b/tests/drivers/spi/spi_loopback/src/spi.c
index 2c6501f7a5..1a74d9b22c 100644
--- a/tests/drivers/spi/spi_loopback/src/spi.c
+++ b/tests/drivers/spi/spi_loopback/src/spi.c
@@ -30,7 +30,7 @@ LOG_MODULE_REGISTER(main);

 struct spi_dt_spec spi_fast = SPI_DT_SPEC_GET(SPI_FAST_DEV, SPI_OP, 0);
-struct spi_dt_spec spi_slow = SPI_DT_SPEC_GET(SPI_FAST_DEV, SPI_OP, 0);
+struct spi_dt_spec spi_slow = SPI_DT_SPEC_GET(SPI_SLOW_DEV, SPI_OP, 0);

tests/drivers/spi/spi_loopback/src/sp

Maybe it is something from my side? Some new configuration I am missing/not aware of?

what board are you testing?

I am using an esp32 devkit, unfortunately, that modification didn't work for me.

I am using an esp32 devkit, unfortunately, that modification didn't work for me.

I assume you mean esp32c3_devkitm

@glaubermaroto try now.  I fixed a bug in the zassert_true(... spi_is_ready...) test logic.

@glaubermaroto try now. I fixed a bug in the zassert_true(... spi_is_ready...) test logic.

@galak Yes, now it works!
As the solution is on the way i am closing this PR ;)


Faking redisearch-py? (mocking)

We have a pretty complicated setup currently, and to speed up our tests we would like to mock redis + redisearch. Do you know of any preexisting library that does it? We looked into fakeredis, issue is that all of the FT.* commands fail. I've reached out to the maintainer to see if he'd be interested in hosting the project there if there is no current alternative.

@kingbuzzman - what are you trying to mock?
You can use FT.EXPLAIN to check that your FT.SEARCH and FT.AGGREGATE commands do not have any syntactic errors.

We're looking to make a fake redis server + redisearch to be able to run all our tests and most importantly run them in parallel and not have them interfere with one another.
Another valid option is to spawn a couple of redis server + redisearch module attach them to individual tests and run them in parallel, then turn them off when done.

You don't have to create a different server for each test, you can also create a different index for each test.


Implicit integer conversion is deprecated

I am getting the following warning:
  /usr/lib/python3/dist-packages/qtawesome/icon_browser.py:170: DeprecationWarning: an integer is required (got type float).  Implicit conversion to integers using __int__ is deprecated, and may be removed in a future version of Python.
    self.setGridSize(QtCore.QSize(tileWidth, tileWidth))

Adding the line tileWidth = int(tileWidth) after setting iconWidth on line 168 should fix this:
https://github.com/spyder-ide/qtawesome/blob/b3645859132d097af817455b7dd337c2c002a48c/qtawesome/icon_browser.py#L166-L170

Hi @juliangilbey thanks for the feedback! Let us know if you want to submit a PR to do the fix :)

Hi @juliangilbey thanks for the feedback! Let us know if you want to submit a PR to do the fix :)

@dalthviz Done!

@dalthviz Done!


Support enumeration type definitions

https://doc.rust-lang.org/stable/reference/items/enumerations.html

Added in  21b7bc.


disable nvidia package downloads?

When installing jax via pip, by default it includes all of the prerequisite nvidia packages it needs which totals about 3GB on extraction.
Downloading nvidia_cublas_cu12-<IP_ADDRESS>-py3-none-manylinux1_x86_64.whl (417.8 MB)
Downloading nvidia_cuda_cupti_cu12-12.2.142-py3-none-manylinux1_x86_64.whl (13.9 MB)
Downloading nvidia_cuda_nvcc_cu12-12.2.140-py3-none-manylinux1_x86_64.whl (21.3 MB)
Downloading nvidia_cuda_runtime_cu12-12.2.140-py3-none-manylinux1_x86_64.whl (845 kB)
Downloading nvidia_cudnn_cu12-<IP_ADDRESS>-py3-none-manylinux1_x86_64.whl (720.1 MB)
Downloading nvidia_cufft_cu12-<IP_ADDRESS>-py3-none-manylinux1_x86_64.whl (98.6 MB)
Downloading nvidia_cusolver_cu12-<IP_ADDRESS>-py3-none-manylinux1_x86_64.whl (124.9 MB)
Downloading nvidia_cusparse_cu12-<IP_ADDRESS>-py3-none-manylinux1_x86_64.whl (195.3 MB)
Downloading nvidia_cuda_nvrtc_cu12-12.2.140-py3-none-manylinux1_x86_64.whl (23.4 MB)
Downloading nvidia_nvjitlink_cu12-12.2.140-py3-none-manylinux1_x86_64.whl (20.2 MB)
I already have all of these packages installed on my system and there is no reason for them to be installed esp as they're all shared libraries. Would it be possible to make this an optional install - perhaps an option to disable all as well as for each of the individual packages in case someone doesn't have them all? The issue is compounded when 100's of users on the same system install multiple environments with this requirement. (And I do appreciate the complexity of nvidia's (in?)compatibility between packages).
Thanks.

Indeed, that's already supported: https://jax.readthedocs.io/en/latest/installation.html#pip-installation-gpu-cuda-installed-locally-harder
There's a link to this doc from the README (https://github.com/google/jax/blob/8f08ee9c201a1df65f8cadbb6658342342a77960/README.md?plain=1#L403).
We don't encourage that path, because it's much more complicated for users and you're much more likely to get it wrong (e.g., not install the right package versions). But it's there if you want it.


🎮 Game Request
Game logic and basic description:
Solitaire is a classic single-player card game where the objective is to move all cards to four foundations in ascending order (Ace to King) (52 cards) and separated by suit. The aim is to arrange the cards in some systematic order or, in a few cases, to pair them off to discard them.
Point down the features
Game points:
The game is won when all cards have been moved to the four foundation piles, with only a single suit in each pile, and all cards being played in ascending sequence. The game is over if there are no more moves that can be made.
Select program in which you are contributing
No response
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @nikki-05 !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

/assign

Hey @nikki-05 ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


Possibility to define Apollo's fetchPolicy per request by using useAsyncQuery is MISSING

Environment

Operating System: Darwin
Node Version:     v18.19.1
Nuxt Version:     3.12.2
CLI Version:      3.12.0
Nitro Version:    2.9.6
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      hooks, typescript, css, postcss, build, components, devServer, devtools, vite, app, modules, pinia, apollo, routeRules, delayHydration, eslint, runtimeConfig, experimental, sourcemap
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Describe the bug
The Nuxt Apollo module has the problem that you cannot specify an option for apollos fetchPolicy in the useAsyncQuery composable. There is the option cache (from Nuxts useAsyncData) as boolean, but this has been changed in this commit  so that either the fetchPolicy from the nuxt config is used (if cache option is false) or cache-first is always used (if cache option is true). So if cache-first is set for all queries in the nuxt config, there is no option to change the policy to no-cache for just one query. It would only work the other way around, i.e. nuxt config no cache and then activate the cache as 'cache-first' by setting cache = true for a specific request.
Additionally this is still limited. Apollo graphql provides 6 different possible fetchPolicies, which are not covered here.
Currently we only have the option in nuxt config to set one fetchPolicy which is true then for all request, but the possibility to decide per request which fetchPolicy to use is completely missing here!!!! this is a big, big issue.
Expected behaviour
Correct would be if the useAsyncQuery composable would provide the possibility to set fetchPolicy per request. So you can decide on request basis which of the 6 possible fetchPolicies of apollo you want to use. The fetchPolicy from the nuxt config should only be a default setting if nothing else got set on the composable usage itself, it should not be the one and only rule for all requests of a project.
Reproduction
No response
Additional context
This problem resulted in several issues and PRs in the past, but it never got fixed properly:
Examples:
https://github.com/nuxt-modules/apollo/issues/348
https://github.com/nuxt-modules/apollo/pull/625
https://github.com/nuxt-modules/apollo/issues/467
Logs
No response

@Diizzayy


Chrome extension for taking backup

There has been a suggestion to develop a Chrome Extension for taking backup. Not sure if this is really required but lets leave this here till final decision.

An extension that injects a button to download all codes on the specific "My Codes" page?

Sample:

Sounds like a plan. (CounterStrike reference, sorry)
Yeah, please send a PR for the same if you want to contribute. And please create separate directory for it. :-)

Roger That! I'll create the extension and send a PR in some time. Will probably host on the Chrome store as well.

@rhnvrm Go go go! I do have an account on Chrome Store, if required.


Add scroll to top button

Adds a scroll to top button to all the pages, it shows only if the user scroll past a decent screen size. It's put on the center top so that it's easily accessible.
Edit: Show only when the user is scrolling towards the top.

closes #3642

Updated so it's visible while scrolling up, also adjusted a bit of styling and animation. Let me know if you would prefer this over always visible. After implementing I tend to like this version better. Thoughts?
https://github.com/HyphaApp/hypha/pull/3667/commits/ee3c9859c533d96daa7c0d247aee4434d48565cc

@theskumar I'll be checking this out in the next hour or so. Looks like a great addition!

Latest version on test in a moment.


Resolution changes when ci runs on self-hosted windows 10 runner

Description
The resolution of the window notebook is as follows.

The actual resolution changes as soon as ci is turned, and the ui test fails.
Doesn't happen in previous runner versions
I couldn't find that version because it was a long time ago. Probably 2.307.1
(Should I write it like this? It's my first time, so I have a lot of shortcomings)
Platforms affected

[ ] Azure DevOps
[ ] GitHub Actions - Standard Runners
[ ] GitHub Actions - Larger Runners

Runner images affected

[ ] Ubuntu 20.04
[ ] Ubuntu 22.04
[ ] macOS 11
[ ] macOS 12
[ ] macOS 13
[ ] Windows Server 2019
[X] Windows Server 2022

Image version and build link
actions-runner-osx-x64-2.311.0
Is it regression?
2.307.1
Expected behavior
Runner resolution equal to window os resolution
Actual behavior
Smaller resolution when running on Runner
Repro steps

Install runner in window 10 notebook
Run UI test using Pytest, selenium in runner
Verify that the resolution runs the same as window 10 notebook

Hello @jiyeexxi.
This repository contains code for images used by GitHub-hosted runners and Microsoft-hosted agents.
I would suggest asking in https://github.com/actions/runner for self-hosted runner issues


Viva Learning supports .mp3 audio

[Enter feedback here]
Viva Learning supports .mp3 audio in Sharepoint Online content but the documentation only listed Audio (.m4a) as the only supported audio format. Kindly review.

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

ID: 3055d301-6a49-ab29-5fd0-2bdf1b4076d6
Version Independent ID: 3055d301-6a49-ab29-5fd0-2bdf1b4076d6
Content: Add SharePoint as a learning content source for Microsoft Viva Learning
Content Source: Viva/learning/configure-sharepoint-content-source.md
Service: viva
Sub-service: viva-learning
GitHub Login: @bhaswatic
Microsoft Alias: bhaswatic

Document has been updated


Three patches (needs testing)

[Hopefully] fixes #11, fixes #12, fixes #13 -
This code is untested, as I do not have an easy way to test these, so it should be deployed to a test server or after Doyenne West.
I hope that all my TypeScript is correct, I'm pretty rusty.

Testing Process: Does not fix #11


Enabling Comments and Reactions for the Plays

Description
Enabling the reactions and comments using Giscus for the plays.
Fixes # (issue) #67
Type of change
We need to include the Giscus integration with the GitHub repo secrets. The secrets to be read from the .env file.
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce.
Checklist:

[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Hello @atapas @nirmalkc
We have upgraded to React v18 as part of this issue: https://github.com/atapas/react-play/issues/87
We have tested the existing plays. All good. Please pull this change to your branch and keep it up-to-date. You have to do sanity of things to make sure everything works for your changes in your branch.
Happy Coding!


Review multiple comments

Issue #:
https://github.com/bcgov/met-public/issues/740
Description of changes:
Changes to review comments to allow for mutiple comment review
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the met-public license (Apache 2.0).

Codecov Report

Merging #808 (c7ec845) into main (023a807) will increase coverage by 6.01%.
The diff coverage is 53.65%.

@@            Coverage Diff             @@
##             main     #808      +/-   ##
==========================================
+ Coverage   67.17%   73.19%   +6.01%
==========================================
  Files          95      169      +74
  Lines        1892     4264    +2372
  Branches      268      268
==========================================
+ Hits         1271     3121    +1850
- Misses        588     1110     +522
  Partials       33       33

Flag
Coverage Δ

metweb
67.17% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

met-web/src/apiManager/endpoints/index.ts
100.00% <ø> (ø)

met-api/src/met_api/services/comment_service.py
50.81% <40.00%> (ø)

met-api/src/met_api/models/comment.py
54.41% <44.44%> (ø)

met-api/src/met_api/resources/comment.py
51.78% <55.55%> (ø)

met-api/src/met_api/models/submission.py
68.88% <100.00%> (ø)

met-api/src/met_api/schemas/comment.py
100.00% <100.00%> (ø)

met-api/src/met_api/services/submission_service.py
82.97% <100.00%> (ø)

met-api/src/met_api/services/engagement_service.py
55.05% <0.00%> (ø)

met-api/src/met_api/version.py
100.00% <0.00%> (ø)

met-api/src/met_api/models/contact.py
89.65% <0.00%> (ø)

... and 71 more


feat: upgrade to TypeScript 4.4

Description
https://devblogs.microsoft.com/typescript/announcing-typescript-4-4/

Reviewers your opinions on api-utils package? Is it a good solution? Or did is it too simple? @fekecrad @chorobin @bauer01

Yeah, I think this is fine. Now its unknown you should always check the type with a type guard, typeof, instanceof etc before trying to cast it to the type you want. Having utils for this is useful.
Although, should we also check for undefined if we're also checking for null?


adds subsites to cabled array

@DanielJMaher

@DanielJMaher it's a mismatch in the ref, just needed an update. all set!

@maka-io OBE by #752 ?

#hacktoberfest

@oceanzus This does have some elements that we will need, but it was never fully finished (lack of info from MIO).  We will have to revisit at some point, but not yet.

OBE, closing.


Prepare for arrow-rs 13.0.0 release

Which issue does this PR close?
re https://github.com/apache/arrow-rs/issues/1623
Rationale for this change
see subject
What changes are included in this PR?

Update version
Changelog. See rendered version here: https://github.com/alamb/arrow-rs/blob/alamb/prepare_for_13.0.0/CHANGELOG.md

Codecov Report

Merging #1624 (312075a) into master (ded4ce8) will increase coverage by 0.08%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1624      +/-   ##
==========================================
+ Coverage   82.94%   83.03%   +0.08%
==========================================
  Files         193      193
  Lines       55635    55472     -163
==========================================
- Hits        46148    46060      -88
+ Misses       9487     9412      -75

Impacted Files
Coverage Δ

arrow/src/array/equal/structure.rs
95.00% <0.00%> (-5.00%)
:arrow_down:

parquet_derive/src/parquet_field.rs
65.98% <0.00%> (-0.23%)
:arrow_down:

arrow/src/array/data.rs
83.05% <0.00%> (-0.20%)
:arrow_down:

arrow/src/ipc/reader.rs
89.37% <0.00%> (-0.07%)
:arrow_down:

arrow/src/array/array_union.rs
90.71% <0.00%> (ø)

arrow/src/array/equal/union.rs
100.00% <0.00%> (ø)

arrow/src/array/equal/boolean.rs
97.14% <0.00%> (ø)

arrow/src/array/equal/decimal.rs
100.00% <0.00%> (ø)

arrow/src/array/equal/primitive.rs
100.00% <0.00%> (ø)

arrow/src/array/equal/dictionary.rs
100.00% <0.00%> (ø)

... and 12 more

Continue to review full report at Codecov.

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

Waiting for https://github.com/apache/arrow-rs/pull/1621 and will update to get https://github.com/apache/arrow-rs/pull/1629

Merging this one in to create a release candidate


[Go Live][March 30]

Push to live

Docs Build status updates of commit 15273a1:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/concepts/device-capabilities/browser-device-permissions.md
:white_check_mark:Succeeded
View

msteams-platform/mstdd-landing.yml
:white_check_mark:Succeeded
View

msteams-platform/sbs-tab-device-permissions.yml
:white_check_mark:Succeeded
View

msteams-platform/tabs/how-to/authentication/auth-aad-sso.md
:white_check_mark:Succeeded
View

msteams-platform/whats-new.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Programming exercises: Add possibility to configure Docker run arguments on Jenkins

Checklist
General

[ ] I tested all changes and their related features with all corresponding user types on a test server.
[x] This is a small issue that I tested locally and was confirmed by another developer on a test server.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Server

[x] I followed the coding and design guidelines.
[x] I added multiple integration tests (Spring) related to the features (with a high test coverage).
[x] I implemented the changes with a good performance and prevented too many database calls.
[x] I documented the Java code using JavaDoc style.

Changes affecting Programming Exercises

[ ] I tested all changes and their related features with all corresponding user types on Test Server 1 (Atlassian Suite).
[x] I tested all changes and their related features with all corresponding user types on Test Server 2 (Jenkins and Gitlab).

Motivation and Context
Closes #6042.
See #5971 for the Bamboo implementation.
Description
Jenkins allows us to pass arguments to the Docker run command as well.
The YAML configuration format has been adapted a bit to be more flexible:

short arguments like -v host_path:/container_path are now possible as well
the same argument can be passed multiple times, e.g., --env SOME_VAR=1 --env OTHER_VAR=2

For Bamboo this change now means that the extra arguments are not passed as
--cpus=2
--memory=2g
--pids-limit=1000
--memory-swap=2g

but as
--cpus
"2"
--memory
"2g"
--pids-limit
"1000"
--memory-swap
"2g"

The two variants should be equivalent. So I assume this should not matter for the functionality.
I did not test it yet on Bamboo, since the test servers are not available at the moment.
Steps for Testing

Prerequisites:

1 Instructor

Create a new programming exercise.
The exercise details page you are on now contains a link to the build plan of the base and solution build plans below the task description.
Follow one of the links to Jenkins.
In the sidebar on the left select the ‘Configure’ entry.
Scroll down to the pipeline script definition (very bottom). The script should contain something like

agent {
    docker {
        image '[…]'
        label 'docker'
        args '--cpus 2 […]'
    }
}

Review Progress

Code Review

[ ] Review 1
[ ] Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Test Coverage

Class/File
Branch
Line

ProgrammingExerciseConfiguration.java
100% of changed code
100% of changed code

Codecov Report

Merging #6045 (e7353e2) into develop (367af2f) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##             develop    #6045   +/-   ##
==========================================
  Coverage      80.09%   80.09%
- Complexity     12684    12686    +2
==========================================
  Files           2193     2193
  Lines          86999    87002    +3
  Branches       12998    12998
==========================================
+ Hits           69684    69688    +4
+ Misses          9606     9604    -2
- Partials        7709     7710    +1

Impacted Files
Coverage Δ

...temis/config/ProgrammingLanguageConfiguration.java
98.21% <100.00%> (+0.06%)
:arrow_up:

...vice/connectors/bamboo/BambooBuildPlanService.java
83.25% <100.00%> (ø)

...ce/connectors/jenkins/JenkinsBuildPlanCreator.java
78.84% <100.00%> (+0.20%)
:arrow_up:

...e/scheduled/WeeklyEmailSummaryScheduleService.java
65.51% <0.00%> (-3.45%)
:arrow_down:

...idebar/conversation-selection-sidebar.component.ts
88.31% <0.00%> (-0.65%)
:arrow_down:

...va/de/tum/in/www1/artemis/service/FileService.java
80.58% <0.00%> (+0.73%)
:arrow_up:

Flag
Coverage Δ

client
76.12% <ø> (-0.01%)
:arrow_down:

server
84.54% <100.00%> (+<0.01%)
:arrow_up:

Continue to review full report at Codecov.

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

In Bamboo (ts1), it's still being passed like this. Jenkins server wasn't available for testing.

@grochlitzzz Thanks for testing. Are you sure the release for this branch got deployed? According to the Bamboo page I don’t think it was.

That's weird, but thanks for the remark. My internet connection was unstable, maybe it didn't deploy and I didn't realize. Let me try again.

:+1: manual test on local Jenkins/GitLab
Looks right in build job:
        agent {
            docker {
                image 'ls1tum/artemis-maven-template:java17-12'
                label 'docker'
                args '--cpus "2" --memory "2g" --memory-swap "2g" --pids-limit "1000"'
            }
        }

Gets passed called correctly in Build pipeline:
$ docker run -t -d -u 0:0 --cpus 2 --memory 2g --memory-swap 2g --pids-limit 1000 -w /var/jenkins_home/workspace/TESTCOURSETEST/TESTCOURSETEST-BASE --volumes-from 6eee6c2408e648f71d09f7ed2097745bcea1ea75f9205d4c0223fe22f3decc10 -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** ls1tum/artemis-maven-template:java17-12 cat

Changed the default parameters in the config (just cpus: 3) and it still works as intended:
        agent {
            docker {
                image 'ls1tum/artemis-maven-template:java17-12'
                label 'docker'
                args '--cpus "3"'

@b-fein: I'll also test Atlassian and changing the values if that's desired, as this can't be done on the testservers?

@4ludwig4 Thanks for testing!

I'll also test Atlassian and changing the values if that's desired, as this can't be done on the testservers?

I don’t think that’s necessary. If it’s not broken with the new default config like in the comment posted above yours, then it should work for changed values, too.


Jest ignores Errors thrown by async functions in some cases

I recently noticed that Jest seems to ignore errors thrown by async functions in certain cases.
I discovered this while writing some tests for a React component that calls an async function from componentDidMount. If the async function calls setState- and the component un-mounts before that happens- the resulting warning/error issued by React is silently swallowed.
Here is an annotated example of what I'm describing (but without React):
https://github.com/bvaughn/jest-uncaught-error-repro/blob/master/index.test.js
To repro this false positive:
git clone git@github.com:bvaughn/jest-uncaught-error-repro.git
cd ./jest-uncaught-error-repro
yarn install
yarn test

In my repro, throwing the error within an expect(...).not.toThrow() block "fixes" things so that the test fails, but that is not the case within the test that first uncovered this issue. It's also not a great solution, since errors aren't guaranteed to be thrown within such a block.
Am I missing or overlooking something?

Maybe related to #5311, #3251

It was pointed out to me that this:
await Promise.resolve(r => setTimeout(r, 200));

...was a bit bogus and should instead be this:
await new Promise(r => setTimeout(r, 200));

...which does indeed seem to fix my repro.


Cast double to half through float on clang to avoid linking to compiler-rt

https://github.com/tinygrad/tinygrad/pull/7365/files#r1830523405

Can you test for this?

Can you test for this?

fixed

Obviously not testing the rewrite! Test to confirm it doesn't link to compiler-rt.


Implement COR SEC   Return File Processing

Thanks @killianbrackey and @Sezzle for all the work in adding support.

Coverage decreased (-4.7%) to 91.24% when pulling 477e545ce8baeac39b07bff171bf4b0825a3329f on wadearnold:Implement-COR-SEC---Return-File-Processing into 81e7c97d7e5accfe1b5bbcea3e143a4124f6d9db on moov-io:master.

Coverage decreased (-4.9%) to 91.024% when pulling 0f42d771e1f38649a977e42bf49ee8d3ff033b1b on wadearnold:Implement-COR-SEC---Return-File-Processing into 81e7c97d7e5accfe1b5bbcea3e143a4124f6d9db on moov-io:master.

The deeper I jump into this rabbit hole the more I think we need a Addenda, AddendaReturn, AddendaCOR, AddendaIAT, etc like we have done with Batch. Just talking to myself publicly.

Coverage decreased (-4.4%) to 91.578% when pulling 9e1072f996188ac51d05f5c05baef71ae5f52cb7 on wadearnold:Implement-COR-SEC---Return-File-Processing into 81e7c97d7e5accfe1b5bbcea3e143a4124f6d9db on moov-io:master.

Coverage decreased (-3.3%) to 92.614% when pulling 6cb4ddec6c60e80bb33bc7bd65fe278119d8092d on wadearnold:Implement-COR-SEC---Return-File-Processing into 81e7c97d7e5accfe1b5bbcea3e143a4124f6d9db on moov-io:master.

Adding changes to master that support @zelle changes. I am working on a new branch that address the Addenda types such as NOC, IAT, etc that need to eventually be supported. @killianbrackey you may need to comment out the ReturnAddenda validator on the ReturnCodeDict so that this works for BatchCOR/NOC until I get the second phase of this refactor committed.


Display only white instead of color

I am using the RGB mode to get three values. The values do not represent colors, they actually represent a directed vector, I want to use the widget for Pan/Tilt/Zoom.
Is there a way to disable the part that draw the colors on the screen?
Thanks,
Terry

Hello,
The colors are drawn by redrawColor() function. To disable it, you could insert this code after jquery.wheelcolorpicker.js script which overrides the function to do nothing.
<script>
jQuery.fn.wheelColorPicker.ColorPicker.prototype.redrawSliders = function(force) { return this; };
</script>

Hope it helps.


Test Result Panel has some strange styling

Version: 1.86.0-insider
Commit: 72cb92fdc5d08101d65d59e4b1c046d397896dd2
Date: 2023-12-12T05:36:41.973Z
Browser: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Code-Insiders/1.86.0-insider Chrome/118.0.5993.159 Electron/27.1.3 Safari/537.36
It's the same issue as https://github.com/microsoft/vscode/issues/196521. Since that issue is locked and the related issue is closed, but I still can see this strange style in the test results panel. So, I have to create a new one.
Steps to Reproduce:
I haven't found a way to reproduce it using the extension sample. It's reproducible in the Java test runner extension. What the extension does is just simply append plain text to the output via appendOutput() api: https://github.com/microsoft/vscode-java-test/blob/22a11be0c6074c536e40b7b2e4c8fdb05c8803dd/src/runners/junitRunner/JUnitRunnerResultAnalyzer.ts#L55
And I found a pattern that if I click the Clear All Result button, then the text rendering will become normal afterward.
Here is a video:
https://github.com/microsoft/vscode/assets/6193897/e04ff2fd-a64f-4f49-9ea1-e5c9a40fa60f

Duplicates https://github.com/microsoft/vscode/issues/194156, reopened that issue


Not able to build in Windows 10

Hi all,
I couldn't able to build the hygieia-core project in Windows 10.
Steps to Reproduce:

Clone the repo
Open Command prompt/Power Shell
Navigate to the cloned repo
Run mvn clean install package

Expected:
Build to be successfull
Actual:

Im getting the same issue on mac

    <dependency>
        <groupId>javax.annotation</groupId>
        <artifactId>javax.annotation-api</artifactId>
        <version>1.3.2</version>
    </dependency>

    <dependency>
        <groupId>javax.xml.bind</groupId>
        <artifactId>jaxb-api</artifactId>
        <version>2.2.11</version>
    </dependency>
    <dependency>
        <groupId>com.sun.xml.bind</groupId>
        <artifactId>jaxb-core</artifactId>
        <version>2.2.11</version>
    </dependency>
    <dependency>
        <groupId>com.sun.xml.bind</groupId>
        <artifactId>jaxb-impl</artifactId>
        <version>2.2.11</version>
    </dependency>
    <dependency>
        <groupId>javax.activation</groupId>
        <artifactId>activation</artifactId>
        <version>1.1.1</version>
    </dependency>

Adding the following repositories fixed this issue for me

split packages causing issues with the module system? Try jdk8

Added Dependencies that fixed it

@D-M-Moriarty Is it #43 to fix the build issues with JDK11?

no on JDK10
On Tue, Feb 19, 2019 at 5:50 PM Ragha Vema<EMAIL_ADDRESS>wrote:

@D-M-Moriarty https://github.com/D-M-Moriarty Is it #43
https://github.com/Hygieia/hygieia-core/pull/43 to fix the build issues
with JDK11?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Hygieia/hygieia-core/issues/37#issuecomment-465238579,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARJ4fddYw-q0rb3rr8gyuOPFT4rT3vDPks5vPDlpgaJpZM4aX86y
.

I will merge it then to make it compatible with JDK10

Can you confirm if this is still an issue?

No it's not an issue for me anymore
On Fri, Apr 19, 2019, 18:14 Ragha Vema<EMAIL_ADDRESS>wrote:

Can you confirm if this is still an issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Hygieia/hygieia-core/issues/37#issuecomment-484960652,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEJHQ7OCMRENDIZXICLTNKDPRH4X7ANCNFSM4GS7Z2ZA
.

Thanks for confirming!


Update README.md with slightly updated GitHub Action example

A slightly updated GitHub Action example could really help first time add-on developers (like me) to unleash the full power of this awesome project!

I came across this today while trying to build my own addon with this action and these changes would indeed make the readme more helpful.


Training time for ImageNet and CIFAR on 4xV100 GPUs

Hi, there! May I ask how long would the training normally takes for training on the CIFAR10 and ImageNet? I'm using 4x 16GB V100 GPUs
I used the following settings

export OPENAI_LOGDIR="improved_diffusion"
MODEL_FLAGS="--image_size 64 --num_channels 192 --num_res_blocks 3 --learn_sigma True --class_cond True"
DIFFUSION_FLAGS="--diffusion_steps 4000 --noise_schedule cosine --rescale_learned_sigmas False --rescale_timesteps False"
TRAIN_FLAGS="--lr 3e-4 --batch_size 256 --microbatch 16"
NUM_GPUS=4
DATA_DIR="cifar_train/"
CUDA_VISIBLE_DEVICES=0,1,2,3 mpiexec -n $NUM_GPUS python scripts/image_train.py --data_dir $DATA_DIR $MODEL_FLAGS $DIFFUSION_FLAGS $TRAIN_FLAGS

and it outputs 3 lines

Logging to improved_diffusion/
creating model and diffusion...
creating data loader...
training...

and stays like this forever without new output.

Hello,
How did you setup your hyperparameter? or where can i find them?
I trying to setup mine to train the model
Thanks.

Hello,
I met the same problem, have you already solved it?
Thanks.


Support for experimentalFragmentVariables

Is there support for something similar to graphql-codegen's experimentalFragmentVariables to solve parameterized fragments by any chance?

Unfortunately not, and I don't think it's supported by our parser, but it looks promising!


[Coil] Placeholder plugin duplicates parent's padding

Please complete the following information:

Library Version v2.2.1
Affected Device(s) Google Pixel 3a API 33

Describe the Bug:
When I ran as follows, I found that Placeholder runs in a smaller size compared to the original image.
CoilImage(
    imageModel = { "https://picsum.photos/200" },
    component = rememberImageComponent {
        +PlaceholderPlugin.Loading(placeholder)
        +PlaceholderPlugin.Failure(placeholder)
    },
    imageOptions = ImageOptions(
        contentScale = ContentScale.Crop,
        alignment = Alignment.Center,
    ),
    modifier = Modifier
        .padding(16.dp)
        .size(100.dp)
        .clip(CircleShape),
)

Placeholder
Original Image

After some workarounds, I found out that Modifier.padding was the cause, and when I put the Box on the outside of CoilImage and padded it, it worked as expected.
Box(modifier = Modifier.padding(16.dp)) {
    CoilImage(
        ...
        modifier = Modifier
            .size(100.dp)
            .clip(CircleShape),
    )
}

Expected Behavior:
The placeholder image must have the same size as the original image.

@wisemuji Thank you for bringing this issue to my attention! Originally, the plan was to emit the modifiers into composable plugins, but it seems to be a little bit buggy. Rest assured, the fix will be included in the upcoming release (2.2.4). I will keep you informed as soon as the upcoming version is released.

@wisemuji The new version 2.2.4 has been released. Please build your project again with this new release, and make sure it works well as expected.

I'm happy to hear that. Cheers!


Update Fortune list to 2015

This PR updates the Fortune list to include the 2015 Fortune 1000.
It's not pretty, but it adds a script/scrape that works by pulling the inline JSON that's embedded in Fortune's WordPress.com hosted site. It parses that to get name and rank, then grabs each individual company in parallel to get the company's domain.
/cc @leereilly, @bkeepers

:heart_eyes_cat:


Bootstrap theme download URL is broken

https://bootswatch.com/flatly/bootstrap.min.css doesn't exist anymore and there is no proper redirect to the current version.
This is the new URL https://bootswatch.com/4/flatly/bootstrap.min.css

Opened a pull request here: https://github.com/gammons/todolist_frontend/pull/5


Upgrade Android Gradle to version 7.2

Upgrades Android Gradle build tools to the current stable version: 7.2 and the Android Gradle plug-in to the current stable version 7.0.2.
The upgrade was done using ./gradlew wrapper --gradle-version 7.2 --distribution-type bin. This ensures that the gradle wrapper (gradle-wrapper.jar) and the gradle wrapper scripts (gradlew and gradlew.bat) are upgraded too. Furthermore, using --distribution-type bin (the default) ensures that only the binaries are installed instead of the source and documentation, which has being done since #31919 for no good reason I could identify.
Includes updating the .gitattributes file to ensure that the Windows only gradlew.bat file's line endings are not changed by git from CRLF to LF.
Can be cherry-picked for 3.x.
The Android Gradle plug-in 7.0 is dependent on JDK 11, so #53609 is required for passing CI.

This version of the Android gradle plugin may deprecate the enableResourceOptimizations flag which would reintroduce #50623.
Could you check that it's not the case.

I've confirmed that upgrading the AGP to 7.0.2 does not reintroduce #50623.

WARNING:: The option setting 'android.enableResourceOptimizations=false' is deprecated.
The current default is 'true'.
It will be removed in version 7.0 of the Android Gradle plugin.

Despite the warning message stating that "It will be removed in version 7.0 of the Android Gradle plugin.", the deprecation "has been reverted to being softly enforced": https://issuetracker.google.com/issues/184666005#comment4. It's still something that will ultimately need to be addressed. I suggest we open a separate issue for it.

Thanks!


How to extract INS.DUP type REF and ALT sequence

Operating system
linux
I call SV with PBSV call ,and I get some INS.DUP,BND,Now I want to get ALT  DUP sequences ,but it does not appear in vcf file,
How  can I get this sequences?
chr1    339052  pbsv.INS.DUP.19 C          .       PASS    SVTYPE=DUP;END=339380;SVLEN=328 GT:AD:DP        0/1:6,5:11
chr1    339337  pbsv.INS.DUP.20 T          .       PASS    SVTYPE=DUP;END=339450;SVLEN=113 GT:AD:DP        0/1:5,5:10

This is not supported. Pinging @ctsa who is developing our pbsv2 successor


DOCS-4443 add link to 3.9 version

also moved advanced info on assigning a lambda to a DW variable to its own section at the end

tested link