How to set url for enterprise github?

My github org repo is like github.company.net/repo/
but this action is rendering the url like github.com/repo/

+1

I changed the env variables but still same

The latest version  v1.9.0 doesn't contain the changes that fix that, did you tried to point it to main ? Also , could you share some logs from the action ?

It doesn't seem to work to me:
env:
  GITHUB_SERVER_URL: ${{github.server_url}}

      - uses: trstringer/manual-approval@dd1555b1d17966cb58500676d79a181c53bb6c12

And I still get the URL from github.com.

Hello @aserrallerios
From the GH Actions docs

The default environment variables that GitHub sets are available to every step in a workflow.
Because default environment variables are set by GitHub and not defined in a workflow, they are not accessible through the env context

It is not possible to override a default environment variable (source item #2), it depends where your runners are running and how you configured the environment variables on the runner.  For example if I run this action using act, and pass those values on .env file: GITHUB_SERVER_URL=www.not-github.com GITHUB_API_URL=api.not-github.com that will do the trick.
The fix that was done on #90 was to format the message with the value of the GITHUB_SERVER_URL and not have it as a static string (https://github.com/%s) nor as customized value

Hey @augustomelo sorry for the delay.
I'm not sure I understand your message, so I removed the explicit GITHUB_SERVER_URL update and validated that my runners were using the right value for the variable.
Run echo $GITHUB_SERVER_URL
https://github.some-domain.com/

But it turns out that the approval still has the github.com domain.
Workflow is pending manual review.
URL: https://github.com/xxxxxx

So I might be missing something super obvious here.

Hello @aserrallerios, no worries 😄
Looking at the code it creates the github client by using the values that are set on $GITHUB_SERVER_URL and $GITHUB_API_URL ((code)[https://github.com/trstringer/manual-approval/blob/dd1555b1d17966cb58500676d79a181c53bb6c12/main.go#L106-L107]) and from the docs those values cannot be override by the actions, so I am not sure what is happening on your end.
Also it is important to notice that I am currently not using this action, so it can be that my fix didn't fix anything at all.

Weird, I'm going to ask the our platform team. May I ask if you're using an alternative action to this one?

No I am not using an alternative action


[BYON] Implement GET /notebook REST endpoint.

Implement the /notebook input for GET  to retrieve the list of custom note book images.  This should return a list of custom note book images according to the REST API specification.

Solved via https://github.com/opendatahub-io/odh-dashboard/pull/156, please close.

fixed.


Provider doesn't support the replay_protection value when creating a cloudflare_ipsec_tunnel

Confirmation

[X] My issue isn't already found on the issue tracker.
[X] I have replicated my issue using the latest version of the provider and it is still present.

Terraform and Cloudflare provider version
Terraform v1.5.4
on windows_amd64
+ provider registry.terraform.io/cloudflare/cloudflare v4.29.0

Affected resource(s)

cloudflare_ipsec_tunnel

Terraform configuration files
resource "cloudflare_ipsec_tunnel" "example" {
  account_id           = var.cloudflare_account_id
  name                 = "example_tunnel"
  customer_endpoint    = "<IP_ADDRESS>"
  cloudflare_endpoint  = "<insert anycast ip>"
  interface_address    = "<IP_ADDRESS>/31"
  health_check_enabled = true
  health_check_target  = "<IP_ADDRESS>"
  health_check_type    = "request"
  psk                  = "<insert psk>"
  allow_null_cipher    = false
  # No option to set the replay_protection value. Always defaults to false
}

Link to debug output
n/a
Panic output
No response
Expected output
Expected ability to be able to configure replay_protection
Actual output
Unable to configure replay_protection
Steps to reproduce

Create a basic cloudflare_ipsec_tunnel resource
Attempt to set a replay_protection value
Check documentation, find no mapping for that value
Review provider code, find no code to support that mapping

Additional factoids
The link below is to the Cloudflare terraform provider documentation. It appears that the provider doesn't support the ability to enable replay protection.
cloudflare_ipsec_tunnel | Resources | cloudflare/cloudflare | Terraform | Terraform Registry
You can see within the codebase itself, within the read function, it doesn't reference the replay_protection value
terraform-provider-cloudflare/internal/sdkv2provider/resource_cloudflare_ipsec_tunnel.go at master · cloudflare/terraform-provider-cloudflare (github.com)
You can see within the List IPsec Tunnels API call however that Cloudflare does expose it
Cloudflare API Documentation
References
No response

Appears to be related to an issue with the cloudflare-go module not supporting this attribute. PR has been submitted to add this functionality:
https://github.com/cloudflare/cloudflare-go/pull/1710


cros_sdk Traceback error

I also can not find where the shim or anything is located

It would also be appreciated to see the entire log of the script, not just where the error occurs.

Ngl, had a lot of issues with this, at first I had 20.04 LTS running on a VM, with sudo access, and still didn't work, was getting the same issues as you, then I decided to dual boot my main pc with 20.04 Beta, WITH sudo access then it worked fine... honestly its really touchy and its a hit or miss.

Yeah, this script is pretty temperamental in its current state. I tested this mostly on VMs, but then again, it all depends on other external factors. I can't really change that, because it literally just runs the steps illustrated in these guides:

https://chromium.googlesource.com/chromiumos/platform/factory/+/HEAD/setup/RMA_SHIM.md
https://chromium.googlesource.com/chromiumos/docs/+/HEAD/developer_guide.md


ADD: getBrowser()

Added getBrowser() function to allow setting of timeout:
$adapter = new BuzzAdapter('your_access_token');
$adapter->getBrowser()->getClient()->setTimeout(30);

👎 Don't think it's a good idea to expose that for mutating. Better for you to set that on construction.


Standard Array is all 0's

Choose the "Standard Array" generation method creates an array of all 0's. A character created this way will have stats of all 0, as well as modifiers of all 0.

Method referenced incorrect variable when passing values to assignScores.
public void genStandardArray()
{
    int[] standardArray = {15,14,13,12,10,8};
    this.assignScores(genScores);
}

genScores is used in the other generations methods and has been changed to standardArray
public void genStandardArray()
{
    int[] standardArray = {15,14,13,12,10,8};
     this.assignScores(standardArray);
}

An additional provision for an ability score of 0 or lower has also been added to genAbilityModifiers, assigning a value of -6 to the mod. This coincides with the maximum mod of 6 for an ability score of 22 or higher.


How to control code points

I intend to use this tool to generate a font with only country flags from Twitter Emoji. My website already supports Unicode emoji flags, however, Windows does not support so I intend to create a fallback font for unfortunate Windows users. I couldn't find a way to control the generated font to use the code points for flags instead of just starting from 1. Is this feature available?

@ranisalt I'm not sure if you've seen the code-point-related options? I think the project's README is outdated.
$ icon-font-generator -h | grep -F -- --codepoint
  --codepoint      The starting character code to count up from (Optional)
  --codepoints     Path to explicit character code mapping JSON file (Optional)

@ranisalt I'm not sure if you've seen the code-point-related options. (I think the project's README is outdated.)
$ icon-font-generator -h | grep -F -- --codepoint
  --codepoint      The starting character code to count up from (Optional)
  --codepoints     Path to explicit character code mapping JSON file (Optional)

@ranisalt
Did you achieve any success with this issue? I would also like to be able to control the code points...


Address boundary handling issues in non-blocking raw text reader, and its blocking wrapper

Issue #, if available: n/a
Description of changes:
Previous implementation of the non-blocking reader, and its blocking wrapper, had two bugs when handling incomplete errors, and continuing after an incomplete error.
The non-blocking reader did not take into account the step_out nesting when unsetting the need_continue. When stepping out of a container that contained a container that had not yet been read, this would result in the reader stepping out beyond the target depth.
The blocking reader did not properly mark the stream as complete when stepping out of a container. If a step_out caused the last remaining data from the stream to be read (completing the stream) root level values at the end of the stream could be ambiguous and continue to cause incomplete data errors, until the stream had been marked complete.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Added a unit test which currently validates the step_out nesting fix when continuing from an incomplete. I do not think it validates the end of stream on step_out anymore (It has been modified since I found the issue). It will take me a bit to repro, in order to get the data, and read sizes, to line up so it exhausts on step_out.
I'll make sure to include that in the format agnostic PR if no one minds its absence here.

tyvm! Added commit for clippy issues.


Require hook for ts-node doesn't work

<EMAIL_ADDRESS> devtool -r ts-node/register server.ts

Fails with error:
...\node_modules\ts-node\register.js
...\node_modules\ts-node\dist\index.js
...\node_modules\source-map-support\source-map-support.js
An error occurred while trying to read the map file at ...\node_modules\source-map-support\foo.js.map
Error: ENOENT: no such file or directory, open '...\node_modules\source-map-support\foo.js.map'

Some research:
    var hasComment = convertSourceMap.commentRegex.test(script);
    var hasMapFile = convertSourceMap.mapFileCommentRegex.test(script);
    // if we have a map pointing to a file, inline it as base64
    if (!hasComment && hasMapFile) {
      var sm = convertSourceMap.fromMapFileSource(original, sourceFileDir);  // <--- ERROR

mapFileCommentRegex =  /(?:\/\/[@#][ \t]+sourceMappingURL=([^\s'"]+?)[ \t]*$)|(?:\/\*[@#][ \t]+sourceMappingURL=([^\*]+?)[ \t]*(?:\*\/){1}[ \t]*$)/mg is incorrect.
It detect regexp in the middle contents of the file (I suppose it should match only last line).
It detects comment sourceMappingURL=foo.js.map from source-map-support/source-map-support.js

Mention mainterners of source-map-support // @evanw @julien-f @LinusU

Same issue here. @unlight did you find a workaround?

I don't know what exactly trips up devtool, but convert-source-map just contains a valid JS comment.
If devtool gets confused with JS comments then it needs to be fixed here instead of trying to fix the symptom cases that causes it to trip.


Android user agent

I've gone through the documentation and i can't seem to find how to configure a custom user agent for android.
This feature exist for ios. Is there any work around for android?

You have full access to the WebView object itself, so you can retrieve that from the TurbolinksSession and configure it as needed.
For example, if your'e using the default session:
WebView webView = TurbolinksSession.getDefault().getWebView()
Once you have that, you can set the user agent as you normally would in Android.


Fixes garbled logs when using % log tag

[approve ci docs]

Cherry-picked to 9.0.x

Lets get a PR against 8.1.x for this, to make sure we don't garble releases.


View > Background option runs into "change…" button in Boards macro for Confluence Cloud

Preflight Checklist

[x] I agree to follow the Code of Conduct that this project adheres to.
[x] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:
When I expand the Format panel in the draw.io Board macro, the text of the "Background" checkbox always runs into the "Change" button on the right. I can adjust the width of the panel, but once I collapse the panel and expand it again, it's the same scenario again.
Expected behavior
Text and Button don't overlap.
Screenshots

I don't see this, what browser?

Chrome, latest version.

Looking into it. If, say, you needed a screenshot or video for now, use Firefox 8-).

Fixed in 15.3.2


Add support for multiple hosts #212

Added support for multiple hosts (select any project with .ftpconfig and click connect).
Added disconnect option in context-menu sidebar.

Will look over this pr in the next few days

Linking #212

@zxwef Your amends are now in 0.9.0. You will need to enable multiple hosts in the package preferences to use it.

I have checked multiple hosts in the preferences. What do I need to put in the .ftpconfig file to include multiple hosts?


Changed ValueMutex to new FuriMutex

I'm not 100% sure if it's correct, but it's done just like in the example I found.
It would close #6.

Hi 👋,
Thanks for your help, and sorry for the delay.
Bes tregards


Enhanced fuzzing code

Coverage remained the same at 97.907% when pulling 0b3575ffdcea21f86303397391cc48ae0f3c9c24 on dev into 603e99ea37887f4c4d1b3ba80d9ca69290663ecd on master.


NuGet add too many frameworks

If you just download, unzip and open this repo in Unity 2019.4.1 NuGet will download the packages but include frameworks that are not required for Unity in the project.

This results in compile errors:
PrecompiledAssemblyException: Multiple precompiled assemblies with the same name Microsoft.MixedReality.SceneUnderstanding.DotNet.dll included or the current platform. Only one assembly with the same name is allowed per platform.
Repro steps

Download the repo via a zip file
Unzip
Open the project in unity 2019.4.1
Observe compile errors

Workaround

Create a new project
Copy assets folder from the repo into the new project
NuGet will correctly include only the required frameworks in the project.

Addtional information
I haven't had chance to debug the exact cause of this, but it likely something odd related to the projects settings
I think this issue about the instructions missing crucial steps is probably caused this problem. I think if this problem were solved, then the instructions would be adequate.

It seems that Nuget For Unity didn't kick in and selected the right libs.
The SU nuget has libs for different targets. Say, if you wanted to use SU in a native C# app using Dotnet, you would need a different lib. Nuget for Unity is there to handle this and avoid your project form having both assemblies in your project at the same time, Since Unity historically has had a problem with same name assemblies.
Can you help me understand the issue?

Did you open the project using an up to date version of Unity Hub?
Does the same thing happen to you with a 'git clone' copy of the project.
Does the issue resolve itself if you close and open and project again?

I followed your repro steps, but i don't have the issues on my side. I'll investigate further.

Just to follow up. I just tried changing the path to "C:\Users\Edd\Downloads\MRSU\MRSU" and it worked. So definitely a path length issue with NuGetForUnity, not really your problem, but I have had a few complications now around excessive path lengths in the other Mixed Reality libraries. Perhaps consider less verbose project/library naming :)


nextcord.errors.ApplicationInvokeError: Command raised an exception: IndexError: list index out of range

Summary
look
Reproduction Steps
@bot.slash_command(dm_permission=False)
async def play(inter: nextcord.Interaction, query: str):
    if not inter.guild.voice_client:
        player = await inter.user.voice.channel.connect(cls=mafic.Player)
    else:
        player = inter.guild.voice_client

    tracks = await player.fetch_tracks(query)

    if not tracks:
        return await inter.send("No tracks found.")

    track = tracks[0]

    await player.play(track)

    await inter.send(f"Playing {track.title}.")

Minimal Reproducible Code
tracks = await player.fetch_tracks(query)

    if not tracks:
        return await inter.send("No tracks found.")

    track = tracks[0]

    await player.play(track)

Expected Results
to play music
Actual Results
it gave me a error
Lavalink Information
Version: 3.7.5
Build time: 06.03.2023 12:02:05 UTC
Branch HEAD
Commit: c00286d
Commit time: 06.03.2023 11:59:46 UTC
JVM: 17.0.6
Lavaplayer 1.4.0-original
System Information

Python v3.10.11-final
mafic v2.4.2
aiohttp v3.8.3
nextcord v2.4.0-final
system info: Windows 10 10.0.22621

Checklist

[X] I have searched the open issues for duplicates.
[X] I have shown the entire traceback, if possible.

Additional Context
nope

I have shown the entire traceback, if possible.
You have not provided the traceback.

wtf is a traceback


[firebase_messaging] Another exception was thrown: NoSuchMethodError: The method 'toRawHandle' was called on null

Firebase messaging not working in background for android.
it shows this error, Another exception was thrown: NoSuchMethodError: The method 'toRawHandle' was called on null.

write it in your main.dart, below is my code snippet
`Future myBackgroundMessageHandler(Map<String, dynamic> message) async{
print(message);
}
main() async {
final FirebaseMessaging _firebaseMessaging = FirebaseMessaging();
_firebaseMessaging.configure(
onMessage: (Map<String, dynamic> message) async {
print("onMessage: $message");
},
onBackgroundMessage: myBackgroundMessageHandler,
onLaunch: (Map<String, dynamic> message) async {
  print("onLaunch: $message");

},
onResume: (Map<String, dynamic> message) async {
  print("onResume: $message");

},

);
runApp(App());
});`

In order to attach onBackgroundMessage, the handler function needs to be a top level method as kept by @mudi256. This is also described in optionally-handle-background-messages

i am getting data messages even app is background..
also i can show notification by using local_notification plugin. I just want to ask how can i navigate a page by using data message..
For example if data message contains 'You have a new message', i want to navigate messagesPage when user clicks the notification..
onBackgroundMessage should be static or top-level function but if we make it static we can not get context :(
Please help

@emrealtunbilek : As per receiving-message, notifications messages are delivered to the app or system tray depending upon whether the app is in foreground or background respectively.
Data messages, on the other hand, are to process the data on the client side when the app is in background. For such messages there is no provision to launch the app.
So, if you want to process the data along with displaying some message withing the app, leverage notification message. Do take care of following:

Set click_action: FLUTTER_NOTIFICATION_CLICK so that notification can be delivered to system tray when app is in background.
Configure onResume and onLaunch callbacks to handle message when user taps the notification.  Also add onMessage that will be called when app is in foreground.
Pass additional data with notification message you need to process on client side.

@emrealtunbilek : As per receiving-message, notifications messages are delivered to the app or system tray depending upon whether the app is in foreground or background respectively.
Data messages, on the other hand, are to process the data on the client side when the app is in background. For such messages there is no provision to launch the app.
So, if you want to process the data along with displaying some message withing the app, leverage notification message. Do take care of following:

Set click_action: FLUTTER_NOTIFICATION_CLICK so that notification can be delivered to system tray when app is in background.
Configure onResume and onLaunch callbacks to handle message when user taps the notification.  Also add onMessage that will be called when app is in foreground.
Pass additional data with notification message you need to process on client side.

Thanks for reply.. But can you tell me how can i use local_notification package for custom notication, not auto notification
I want to show user profile picture in notification , if i use notification messages, system created auto notification in system tray and i can not edit them as far as i know..
Because of that i am using data messages...
Is there a way to do that with notification messages not data messages ?

Hey all 👋
As part of our roadmap (#2582) we've just shipped a complete rework of the firebase_messaging plugin that aims to solve this and many other issues.
If you can, please try out the dev release (see the migration guide for upgrading and for changes) and if you have any feedback then join in the discussion here.
Given the scope of the rework I'm going to go ahead and close this issue in favor of trying out the latest plugin.
Thanks everyone 🤓


[BEAM-14164] [Website] change styles

[BEAM-14164]

change padding in css
change class in index.html from playground to playground-section

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @AydarZaynutdinov

R: @pcoet

retest this please


Problems since Update for PSA Vehilce SoC

After update from V0.101 to 0.102.1 evcc can‘t get information for my Peugeot e2008 Form the PSA-API:
vehicle soc: Get "https://api.groupe-psa.com/connectedcar/v4/user/vehicles/xxx/status?client_id=xxx\": Post "https://idpcvs.peugeot.com/am/oauth2/access_token\": net/http: request canceled (Client.Timeout exceeded while awaiting headers) (Client.Timeout exceeded while awaiting headers)
Till V0.101 it works without problems.
Is Theresa anything to change in the config?
Thanks and best regards,
Sven

Problem with the PSA api.

have the same issue since 0.102 now on 0.104 still the same
ehicle soc: Get "https://api.groupe-psa.com/connectedcar/v4/user/vehicles/1566c497a5655746153316861546b6f324f5449774f446b6a497a63304d32466a4d446b344e47517a4f4745334d6d5a6b5a544d334f4455334e325534597a466a4f5745324f444d784d4455785a44646c4e44566d4f4459325a6a566a5a5451774d474d314f474d334f5451354e574d3d/status?client_id=1eebc2d5-5df3-459b-a624-20abfcf82530\": oauth2: cannot fetch token: 400 Bad Request\nResponse: {"error":"invalid_grant","error_description":"grant is invalid"}

Different error, new topic.


[Feat]: ui/vulnerabilities - increase information density

Is your feature request related to a problem? Please describe.
the page listing the vulns only shows about 5 at a time on my screen. I have had to review lists of hundreds of cves, and the more that fit on the screen the better

Describe the solution you'd like
ideally a compact display with one line of text per vuln for maximum eyestrain
Describe alternatives you've considered
No response
Additional context
No response

We need to come up with a new UX. Also need to think of the implications for mobile/small displays.

since this is a big change, https://github.com/project-zot/zot/issues/1999 is a workaround for reviewing larger lists, until the UI work is done

@mikemccracken this has been done in https://github.com/project-zot/zui/pull/409, please let us know your feedback.

@mikemccracken we have another update, merged in https://github.com/project-zot/zui/pull/419 / https://github.com/project-zot/zot/pull/2233

I am happy with the current one line layout. it's enough to look at a reasonable sized list, and if there's a huge number then I can use the csv export.


Button disable

The .btn-disabled style did not set background colors, so would require unsetting the other .btn- styles to appear disabled. This fixes that:

:+1:


fix fix bug in agg on constant column

Proposed changes
For performance reason, we want to remove constant column from groupingExprs.
For example: select 'abc', sum(a) from t group by 'abc' is equivalent to select 1, sum(a) from t
We can remove constant column abc from groupingExprs.
But there is an exception:
sql1: select 'abc' from t group by 'abc'
sql2: select 'abc' from t
sql1 is not equivalent to sql2.
We need to keep some constant column if all items in groupingExprs are constant columns.
Consider sql3 select a from (select "abc" as a, 'def' as b) T group by b, a;
if the column is in select list, this column should not be removed.
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[ ] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[ ] No
[ ] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[ ] No Need

Does it need to update dependencies:

[ ] Yes
[ ] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[ ] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 35.25 seconds
load time: 480 seconds
storage size:<PHONE_NUMBER>1 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230206083620_clickbench_pr_90790.html

@starocean999 PTAL. This pr is related to ur previous fix.


Thread Safe Problem

`bool LGBMPredictor::Predict(
const float* matrix,
int nrow,
int ncol,
std::vector* preds)
{
// Comm::LogErr("[LGBMPredictor][debug] nrow:%lu, ncol:%lu", nrow, ncol);
Predictor predictor(boosting_model_.get(),
                config_.num_iteration_predict,
                config_.predict_raw_score,
                config_.predict_leaf_index,
                config_.predict_contrib,
                config_.pred_early_stop,
                config_.pred_early_stop_freq,
                config_.pred_early_stop_margin);

auto pred_fun = predictor.GetPredictFunction();

for(int i = 0; i < nrow; ++i)
{
    std::vector<std::pair<int, double>> one_row;
    auto start = matrix + ncol * i;
    for(int j = 0; j < ncol; ++j)
    {
        one_row.push_back(j, static_cast<double>(*(start + j)));
    }
    pred_fun(one_row, &((*preds)[i]));
}
return true;

}`
I am going to write predict code like above, without any thread lock. This funtion is supposed to called in multithreading, is that thread safe?

where do you parallel? parallel call this function?

where do you parallel? parallel call this function?

yes, I have 25 threads to call this function.

Further, i read the c_api code. To my understanding, the lock in Booster::Predict is just used to maintain the parameter like ### double* out_result, because I could not find any member variable would be modify in boosting_ and predictor during multithreading. Therefore I think it is thread safe naturally, even without a thread lock.

if you can ensure  the argument preds is not written to the same index in different threads, it is thread-safe.

if you can ensure the argument preds is not written to the same index in different threads, it is thread-safe.

that's nice, thank you very much

there is lock in predict now, for the boosting is modified here https://github.com/microsoft/LightGBM/blob/f01b2aca13e787e2945eba7c2bd85e31c65923f8/src/boosting/gbdt.h#L334-L345
However, I think the num_iteration_for_pred_  could be moved to predictor, then we can remove this lock.

there is lock in predict now, for the boosting is modified here
https://github.com/microsoft/LightGBM/blob/f01b2aca13e787e2945eba7c2bd85e31c65923f8/src/boosting/gbdt.h#L334-L345
However, I think the num_iteration_for_pred_ could be moved to predictor, then we can remove this lock.

yes, that will be


Need to detect and delete illegal configurations from storage

At the moment, parsing from config strings, such as reading from storage, does not check the parameter values. They ought to be checked (numeric values are numeric, bounded-range values within ranges, choice values have a valid choice), and action taken (likely raise exception).
This needs tools to clean out improper configurations from storage, too.
This situation arises when the bounds of a parameter have been tightened.

Addressed in #59


use constants in updater

Closing as housekeeping, see https://github.com/jevakallio/redux-offline/pull/15#issuecomment-290047175

Closing as housekeeping, see https://github.com/jevakallio/redux-offline/pull/15#issuecomment-290047175


refactor: reduce build time by using different way to extend multiple…

… interfaces

Codecov Report

Merging #37 (c7bd620) into main (46d7226) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #37   +/-   ##
=======================================
  Coverage   84.76%   84.76%
=======================================
  Files         223      223
  Lines       14740    14740
  Branches     2448     2448
=======================================
  Hits        12495    12495
  Misses       2215     2215
  Partials       30       30

Impacted Files
Coverage Δ

...rrow-valued/src/arrow/arrow-valued-graph-hashed.ts
100.00% <ø> (ø)

...rrow-valued/src/arrow/arrow-valued-graph-sorted.ts
100.00% <ø> (ø)

...graph-arrow-valued/src/arrow/arrow-valued-graph.ts
50.00% <ø> (ø)

...h-edge-valued/src/edge/edge-valued-graph-hashed.ts
100.00% <ø> (ø)

...h-edge-valued/src/edge/edge-valued-graph-sorted.ts
100.00% <ø> (ø)

...es/graph-edge-valued/src/edge/edge-valued-graph.ts
50.00% <ø> (ø)

...ckages/graph/src/graph/arrow/arrow-graph-hashed.ts
100.00% <ø> (ø)

...ckages/graph/src/graph/arrow/arrow-graph-sorted.ts
100.00% <ø> (ø)

packages/graph/src/graph/arrow/arrow-graph.ts
50.00% <ø> (ø)

packages/graph/src/graph/edge/edge-graph-hashed.ts
100.00% <ø> (ø)

... and 17 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 46d7226...c7bd620. Read the comment docs.


Update revealjs to its latest version (5.0.4)

Main new feature is scroll view, which is the new default for mobile view.
I have tested this new version with my own slides and haven't seen any regression/bug.

The new feature in Reveal v5 is 'scroll view' but how do users enable or disable 'scroll view' with this PR, it's unclear?

The new feature in Reveal v5 is 'scroll view' but how do users enable or disable 'scroll view' with this PR, it's unclear?

Sorry, Geo. Users just need to set view: scroll under reveal_options. It's enabled by default on mobile.


Add JSN-SR04T Distance Measuring Ultrasonic Waterproof Sensor Module

Add JSN-SR04T Distance Measuring Ultrasonic Waterproof Sensor Module, Range Sensor

https://github.com/hornmich/JSN-SR04T

https://github.com/jumejume1/Arduino/blob/master/JSN-SR04T/JSN-SR04T.ino

https://github.com/insomartin/JSN-SR04T-2.0-ultrasonic-sensor/blob/master/JSNSR40.ino

https://github.com/kd8bxp/jsn-sr04t-2.0-car-sensor

https://github.com/pantaluna/esp32_jsnsr04t_using_lib/tree/master/components/mjd_jsnsr04t

EspHome
https://esphome.io/components/sensor/jsn_sr04t.html

Done
https://github.com/bareboat-necessities/bbn-m5atomS3-lite/tree/main/bbn_m5atomS3_lite_JSN-SR04T


Commits time does not match up

On the Dashboard when I commit something it says that it was committed 1 min ago.
When I go into the commits it says 18 hours ago... I am in Australia, all the times in gogs seem to be in PST.  The dashboard screen seems to take my local time zone into account, the +18 is the -8 to +10 timezone difference.
I don't know if there is a way to change the time-zone (please let me know)
Regardless, if it can take local timezone into account, it should show all timezones as the browsers timezone should it not?
Also when I hover over the 1 min ago it still states that it was 18 hours ago, but PST (so now but a diff time zone).

Hi, can you take screenshots of commits time (in hover) and actual time of commit (in command line)?

Here is the screenshot of the web page...

Will have to check @ home for the command line as I am @ work.

@i00 , thanks your info!
But I think you have misunderstanding between time of activity and time of commitment, they are obviously different, and no guarantee to be the same.

The time was the same.. It was pushed when it was committed.

I need you to hover on the time to get exact time of commit:

And exact time from command line such as output of git log -1.
Relative time adds no value for debugging.

Close due to lack of feedback.

The time zone when I hover it is in PST ... my computer is set to EST (Australia Eastern time, Brisbane), also I use the Windows version of Git.
The commits in the screenshots were committed and pushed at the same time ... the screenshots were taken at the same time ... therefore the X Hours ago should be the same???

I will check the other things that you stated when I get back to work on Monday.

Thanks for following up!

The commits in the screenshots were committed and pushed at the same time

I need to know exact (absolute) time, not same time, and such relative time.

Hrm... strange...

Both my client and my server are set up as AEST??

Yes, and please do:

And exact time from command line such as output of git log -1.

ping @i00

commit 6e6ca41128b04289c3e17e2294f9aaa9bc998bdc
Author: Kris Bennett<EMAIL_ADDRESS>Date:   Sun Feb 7 22:55:17 2016 +1000
Comment follows...

@i00 is anywhere I can get this repository history?

ping @i00

Sorry it is private.. Any other way?

OK, I rechecked your screenshot, I think commit time are showing exactly right.
Only the timezone is different.

Close due to lack of feedback.


Readme files should support windows users and not only mac

WHY
We want to expand our audience and a lot of developers are not on a mac or linux computer.
HOW
We need someone to run this full system on a windows computer and make a specific windows.md file that we reference.

Closing. Feel free to re-open

Here is a rough script to build using Windows. Few things still outstanding, but mostly there.
$SCRIPT_DIR = $PSScriptRoot
$PLUGIN_SRC_DIR = "$SCRIPT_DIR\..\plugins"
$PLUGIN_OUTPUT_DIR = "$SCRIPT_DIR\..\bin\plugins"

$PLUGINS = @()
if ( -not $PLUGIN ) {
    #Write-Information "Building all plugins"
    Write-Host "Building all plugins"
    $PLUGINS = $(Get-ChildItem -Path $PLUGIN_SRC_DIR -Directory -Depth 0)
} else {
    Write-Information "Building the following plugins: $PLUGIN"
    foreach ($p in $($PLUGIN.Split(','))){
        $PLUGINS+="$PLUGIN_SRC_DIR\$p"
    }
}

if (Test-Path -Path $PLUGIN_OUTPUT_DIR) {
    Remove-Item -Path $PLUGIN_OUTPUT_DIR -Recurse -Force -ErrorAction Continue
}

$JOBS = @()
foreach ($PLUG in $PLUGINS) {
    $NAME = (Get-Item -Path $PLUG).BaseName
    Write-Host "Building plugin $NAME to bin/plugins/$NAME/$NAME.so with args: $*"

    $j = Start-Job -ScriptBlock { go build -o "$($args[0])\$($args[1])\$($args[1]).so" } -WorkingDirectory $PLUGIN_SRC_DIR\$NAME -ArgumentList $PLUGIN_OUTPUT_DIR, $NAME
    $JOBS += $j

    # get the number of processing units
    # we limit the number plugin builds to that.
    $totalLogicalCores = ((Get-CimInstance –ClassName Win32_Processor).NumberOfLogicalProcessors | Measure-Object -Sum).Sum

    # do a quick sleep to try prevent waiting for the jobs below
    Start-Sleep -Seconds 1
    if ($($JOBS | Where-Object { $_.State -eq "Running" }).Count -eq $totalLogicalCores) {
        Wait-Job -Job $JOBS
    }
}

if ($($JOBS | Where-Object { $_.State -eq "Running" }).Count -gt 0){
    # we wait for any jobs that might still be running
    Wait-Job -Job $($JOBS | Where-Object { $_.State -eq "Running" })
}

Amazing, I thought Golang Plugin was only available on the *nix platform, was it working properly on Windows?

Yes that is the big thing here, it "builds", but it's pointless at present because plugin support is not available for Windows and probably won't see it anytime soon. The above is not for debugging. Big reason why I kept it closed.


plz rewrite in Go

it's so much more practical vmvc

I opened an issue with the upstream library, once that's resolved I'll work on that.


[#49] Added support for Ubuntu 20.04 and MySQL

I've seen success with these changes, but someone else needs to try running this on their machine before we merge it.

Gonna give this a whirl

I'm still working out some kinks, but feel free to give it a shot.


Can't select the curve handles when the whole object is scaled

The editor still detect the handle at its non scaled position, making it impossible to use

Actually, this bug also appear on the regular Godot gizmo as well so it's probably not my fault

Fixed upstream


get_hex doesn’t work outside of setup unless reload

The get_hex doesn’t get the correct color unless used directly inside of setup.
Does not work
Will work if nvim config is reloaded while running.
local bg = get_hex(“Normal”, “bg”)
require(“cokeline”).setup({
  components = {
    { text = “|”, bg = bg }
    …
  }

Does work
require(“cokeline”).setup({
  components = {
    { text = “|”, bg =  get_hex(“Normal”, “bg”) }
    …
  }

The problem with the second approach is that it’s not DRY. I know I can use default_hl for bg but there are highlights that I want to use repeatedly without having to call get_hex with same parameters multiple times.

I remember this happening a while back and can't repro now, so I believe this has been fixed. Feel free to reopen if this is still happening.


Logo and background. #100

Added the logo and background I even added the desktop icon, feel free to change it to the other as i didn't know which one to use. I also fixed the credits screen and the README.
I used my version of importing files into the menu. It makes it easier to import images, rather than including files that have nothing to do with it.
If you need any more information... feel free to ask.
P.S. You need the logo to be 512 x 512px              (I did a quick resize, hence why it is blurry)

Can one of the admins please verify this patch?

GooeyHub is a bot. If you're not on the whitelist it asks for admin verification. In either case, it runs a final test before the pull request is merged. @Cervator will notice this eventually, just give them time. See #107 for an example :D.

I noticed this earlier but have been struggling a bit on how to respond to it and the other comments just posted on other issues. There's a bit of ... awkward history in place here, especially when comparing this PR to some discussion yesterday. Probably best if I get a hold of @crazywolf132 in private sometime - you know where to find me.
The biggest issue with this PR right now is that it doesn't actually compile. I figure code was copy pasted from the custom fork Path to Mani without adjustments. There are a fair few other issues too.


dialogic_default_action isn't respected by Centered Choices box

The problem
Describe the bug
dialogic_default_action isn't respected by Centered Choices box. So when I click on a button that I've assigned to this, it doesn't work.
To Reproduce
Steps to reproduce the behavior:

Assign a different button to dialogic_default_action in the Input Map. I used CTRL.
Make a timeline with centered choices.
Try to press ctrl to choose one.
It only works with the default buttons.

Expected behavior
It works with the new button.
Screenshots
If applicable, add screenshots to help explain your problem.
System (please complete the following information):

OS: Fedora Linux 40 KDE Spin
Godot Version: 4.3
Dialogic Version: 2.0 Alpha 15

Solutions
Workaround
You could probably send a space input action when ctrl is pressed?
Possible fixes
Not sure.

Choices are buttons and that means they use the ui_accept action not the advance action so this is expected. However I think we might be able to add a setting to also use the input action additionally to the ui_accept action.

When I press advance it disables choosing anything for some reason.

This is slightly off topic, but could you help me? I have a giant button over the whole screen that sends a dialogic default action event in the code. The choices lock up when I press it on the choice.
Code:
var dialog_advance_action_value: bool = true
func _on_button_button_down() -> void:
	print("Pressing dialogic_default_action...")
	# This one doesn't work. Use other way, as documented in action_press
	# docs. Input.action_press("dialogic_default_action")
	var action_dialog_proceed = InputEventAction.new()
	action_dialog_proceed.action = "dialogic_default_action"
	action_dialog_proceed.pressed = !dialog_advance_action_value
	Input.parse_input_event(action_dialog_proceed)

I assume that it's because I'm not freeing the dialog_advance_action_value by doing a key up? But I did it this way because the advance button gets broken otherwise.

What do you mean it freezes up the choices?

They don't work anymore. But I do see it's on my end. Sorry!

Should I close this issue since you can just rebind ui_accept?

@mak448a I think we can close this. You can now tell dialogic to use the default action for choices (in addition to the ui_accept action):
Dialogic.Choices.use_input_action = true

This is currently false by default and not exposed to the settings menu. If you feel like it should be, then let me know.

Note: this setting only uses non-mouse based events from the input_action, because the advance action often includes left mouse button, and we don't want a click anywhere to select the focused choice.

Could you add this to the docs? Thanks!


How to list all jobs for a compute target in AzureML SDK v2?

I'm trying to list all jobs for a compute target in AzureML SDK v2 but I can't see how to do it. In SDK v1 the code would look like this:
from azureml.core import Workspace, ComputeTarget, Run
workspace = Workspace.from_config()
compute = ComputeTarget(workspace, "name")
runs = Run.list_by_compute(compute=compute)

If it's not possible, will support be added to reach parity with v1?

Hi @letmaik , thanks for your contact! We'll investigate asap!

currently its not supported in SDK V2

adding a backlog item.


Xcode version doesn't match one being used

I was looking through logs, and I noticed that the UI specifically calls out "Xcode 14.0", but Xcode 14.1 was actually being used.

This is minor, but I thought I'd report it just in case.

Thanks Matt, I noticed that as well just yesterday :) Will address this when switching to Xcode 14.2, which is overdue as well.


Added Light Theme

New Feature: Added Header element with Name Heading and Theme Toggler button.
#4

LGTM!!🎉
Great Work @sonylomo

@sonylomo can you also add a lighter background colour for the light theme, it remains dark regardless the theme :)


Debugger not working

What steps will reproduce the problem?

In Spyder 5.1.5, set a breakpoint and run some code. The debugger launches but there is nowhere to enter commands. It just hangs there.

Hey @jtmbeta, thanks for reporting. This error is already fixed in our latest version (5.2.2).
Since it's still not available in Anaconda, you have two options:

Use our macOS installer, which you can find here.

Create a new environment with conda-forge packages. For that, please close Spyder, open a terminal and run the following commands there:
 conda create -n spyder-cf -c conda-forge spyder
 conda activate spyder-cf
 spyder

After that, you can connect Spyder to your base environment by following these instructions:
http://docs.spyder-ide.org/current/faq.html#using-existing-environment

Thanks!
Joel

On 22 Feb 2022, at 16:13, Carlos Cordoba @.***> wrote:
Hey @jtmbeta, thanks for reporting. This error is already fixed in our latest version (5.2.2).
Since it's still not available in Anaconda, you have two options:
• Use our macOS installer, which you can find here.
• Create a new environment with conda-forge packages. For that, please close Spyder, open a terminal and run the following commands there:
conda create -n spyder-cf -c conda-forge spyder
conda activate spyder-cf
spyder
After that, you can connect Spyder to your base environment by following these instructions:
http://docs.spyder-ide.org/current/faq.html#using-existing-environment
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.


System.IO.IOException: DeviceError on Linux

We are using

     Linux 4.12.4-1 Arch

     Libnserial.so.1.1

     SerialPortStream <IP_ADDRESS>

     3 COM ports (9600, 38400, 57600 baud)

     .NET Core 2.1

We open the ports once.
Then a short time for 8 min all is running well.
But then we get following errors at SerialPortStream.ReadExisting()/.Write():
System.IO.IOException: Device Error
at RJCP.IO.Ports.SerialPortStream.ReadCheckDeviceError(Boolean immediate) in C:\EASE\MetaBridge_Core\Brabender.Common.Hardware5.SerialPortStream\SerialPortStream.cs:line 630
at RJCP.IO.Ports.SerialPortStream.ReadExisting() in C:\EASE\MetaBridge_Core\Brabender.Common.Hardware5.SerialPortStream\SerialPortStream.cs:line 932
at Brabender.Common.Hardware5.BrabSerialPort.SerialPort.ReadExisting() in C:\EASE\MetaBridge_Core\Brabender.Common.Hardware5.BrabSerialPort\SerialPort.cs:line 502
or
System.IO.IOException: Device Error
at RJCP.IO.Ports.SerialPortStream.WriteCheckDeviceError() in C:\EASE\MetaBridge_Core\Brabender.Common.Hardware5.SerialPortStream\SerialPortStream.cs:line 1101
at RJCP.IO.Ports.SerialPortStream.WriteCheck(Byte[] buffer, Int32 offset, Int32 count) in C:\EASE\MetaBridge_Core\Brabender.Common.Hardware5.SerialPortStream\SerialPortStream.cs:line 1091
at RJCP.IO.Ports.SerialPortStream.Write(Byte[] buffer, Int32 offset, Int32 count) in C:\EASE\MetaBridge_Core\Brabender.Common.Hardware5.SerialPortStream\SerialPortStream.cs:line 1139
at Brabender.Common.Hardware5.BrabSerialPort.SerialPort.Write(Byte[] buffer, Int32 offset, Int32 count) in C:\EASE\MetaBridge_Core\Brabender.Common.Hardware5.BrabSerialPort\SerialPort.cs:line 397
When we enable tracing then the problem disappears.
So we have debug the problem by a remote session:

the device error indicates that the I/O thread has stopped due to an I/O error earlier. To debug this, you need to capture traces (preferably verbose, if not, then info) with the information as provided in the wiki. See Tracing. To get to the root cause, it may be necessary to recreate also with "strace" on your system, which shows the results of all system calls.
When providing information about the libnserial, did you compile it yourself, or use a .deb package I've already provided? If you compiled it yourself, the commit ID is useful.

As mentioned, we have already enabled tracing.
We do this in code as following because .NET Core doesn’t supports app.config anymore.

But if we run our application with enabled tracing we don’t get the error anymore.
So we have debugged the problem as already shown.
In case of the problem we get an error result at the call “m_Dll.serial_read(m_Handle, (IntPtr)bo, length);”
The error code is 5 and the message is “Read-End-Of-File reached”.
Update:
When we enable tracing and starting remote debugging then the problem appears too.
So we think it is a timing Problem.
1120c_RJCP.log

Hello, are you able to track down if this is a device issue? To confirm, an strace is needed at the time of failure.

The strace will take some time because our Linux expert is currently not present.
We use multithreaded access to SerialStreamPort, but to different ports.
The accesses to each port are already locked at high level.
By trial, I have locked each SerialStreamPort call with a static object, without success.
In the log there is always an “abortwatiforevent” before the error appears:
IO.Ports.SerialPortStream Verbose: 0 : /dev/ttyBrab_Bb9ur5o6jsg10_0: ReadWriteThread: serial_waitforevent(RJCP.IO.Ports.Native.Unix.SafeSerialHandle, ReadEvent) == ReadEvent
IO.Ports.SerialPortStream Verbose: 0 : /dev/ttyBrab_Bb9ur5o6jsg10_0: ReadWriteThread: serial_abortwaitforevent(RJCP.IO.Ports.Native.Unix.SafeSerialHandle) = 0
IO.Ports.SerialPortStream Error: 0 : /dev/ttyBrab_Bb9ur5o6jsg10_0: ReadWriteThread: Error reading data; errno=5; description=Read End-Of-File reached
I think the abortwaitforevent comes from a Write call to port while the port receives characters.
Do you think this can be a problem?

Not sure how to continue, but at this time I believe it is a driver error, as it isn't expected to get a read of zero. If you're able to get more information, please reopen this ticket and I'll look at it again.


Space between the words Google and Drive

@cakesoft-shivani Similar issue needs to be fixed on "Google Drive Backup" screen.

-Verified this issue on dev app v1.7.5(299)


Last prompt in install.py doesn't default to "Y" as indicated

After using Enter for the defaults all the way through the install script, the install exits with the following message:
Ready to apply settings and start downloading? [Y/n]:
Install cancelled. No changes were made. Have a nice day! :-)

Adding  or "Y" to the end of line 95 of install.py should fix it.
https://github.com/InfoSecInnovations/concierge/blob/3c4a299e9cdf24dfd4be0d118f82080666d4d764/install.py#L95

While hitting Enter will get you to the last question, we do want folks to have to take a positive Y to do the install. I.e., this is I think a good thing?
Would you be more open to the "must hit Y" if we worded that final prompt a bit differently?

As a user, it cancelled without error, and I had to read the code to figure out why.
The [Y/n] at the end follows a common convention that the caps option would be the default, which I think is what had me confused.
So, either make them both should be capital, or add language as you suggested.
Following flows that folks are more likely used to, you could do something similar to SSH when contacting a new system.

Note that someone that enters y instead of Y to that prompt will cancel their install, as well, since line 97 doesn't use .upper(). It will jump straight to line 123. They'd probably be more confused than I was.
https://github.com/InfoSecInnovations/concierge/blob/3c4a299e9cdf24dfd4be0d118f82080666d4d764/install.py#L97-L125

Excellent feedback. Thanks so much. Will test with the "ssh" style option first.

I think that's clearer. (Personally, I don't see the need to default to "no" for the final opt-in, but it's at least clear what's happing now.)

OK sounds like we got what you were after. Please re-open if you think this needs further work.
Cheers!


Cannot change locale in monaco-typescript

monaco-editor version: 0.23.0
Browser: Microsoft Edge 89.0.774.45
OS: Windows 10 (20H2)
Playground code that reproduces the issue:
I want to set the locale of monaco typescript to zh-cn.
However, using the code as follow, I can only see the editor showing in Chinese. The diagnostic messages of typescript are still English.
require.config({
  'vs/nls': {
    availableLanguages: {
      '*': 'zh-cn'
    }
  }
});

The value of 'monaco.languages.typescript.compileroptions.locale' in Console is 'zh-cn'.
I tried to put the folder zh-cn with the file diagnositcMessages.generated.json under the path 'vs/language/typescript', but it doesn't work as well.
Does it mean I did the wrong steps or this function is not available for monaco editor now.
Thanks a lot for developing this project!

找到解决方法了吗？

https://github.com/microsoft/monaco-editor/blob/c5586a25cfd069349b6242a3d140223cc3d9110c/browser-amd-localized/index.html#L1-L30

For anyone else having this issue, pending official support by Monaco I've made this package which adds TS/JS diagnostic message translation https://www.npmjs.com/package/monaco-editor-typescript-locales


SSCS-11619 corrects PageId of Liberty_To_Apply

JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/SSCS-11619
Change description
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ x ] No

increment the version number please.


Move the icons into their own folder

Ewww, that pollutes the root of the project so much!!

Ahahah thanks, good urls don’t change, but I should definitely fix that, or someone else! :D #hacktoberfest

closed by #20 🎉


What is the proper way to encrypt table id?

Hi,
Im using uuid for my primary keys but i want is to encode uuid before returning it to user.
for example:
{
  id: "a3ae9976-94b4-46ce-afdd-76a665ee7316"
}

what i want to do is to make something like this:
let list = model.fetchAll().map((item) => {
  return {id: encode(item.id)};
});

which will become
{
  id: "45IOgC2FDfr5bkbS"
}

Best Regards,
Jason Villalon

If you are ok with always converting id you can use the parse method.

Hi,
Thank you for reply.
i already tried to convert it on parse. sorry i didnt mention it. thing is that my schema is something like this:
id: uuid
parent_id: uuid
submodules() {
  return this.hasMany(Modules);
}

and when i fetch the records using
.fetchAll({withRelated: ['submodules', 'submodules.submodules']

i receive error that the encrypted id is not a valid uuid since it is encoded

looks like i cannot format ids on parse when there is related hasMany in model.
is there a way to decode the id before passing them when fetching related records?
thanks in advance

You could useparse with format
parse(attrs) {
  attrs.id = econde(attrs.id); // _.update(attrs, 'id', encode);
  return attrs;
}
format(attrs) {
  attrs.id = decode(attrs.id); // _.update(attrs, 'id', decode);
  return attrs;
}

Another alternative is using the Visibility plugin:
Model.extend({
  tableName: 'users',
  hidden: ['id'],
  parse(attrs) {
    attrs.enconded_id = encode(attrs.id);
  },
  submodules() {
    return this.hasMany(Modules, 'parent_id', 'id');
  }
})

That way you create relationships using id but the users only see enconded_id.

Hi!
Thank you for your reply. the combination of your 2 suggestions solved my problem.
what happen is all primary keys which is "id" is added as hidden using visiblity plugin
and all the foreign keys are manually encoded and decoded in parse and format methods.
encoding and decoding id (primary key) in parse and format will cause of unpopulated related records.
and the reason why i cannot add the foreign keys in hidden is that it will not being populated when fetching withRelated.
All in all. thank you very much. closing this issue now.
Best Regards,
Jason Villalon


Wall Snap Points

I know I brought this up on Discord but figured I'd bring it up here as well just for documentation, apologies if unnecessary.

If possible, add either by default (or a toggle in the config) snap points to the corners of mineshaft walls to help with more precise building.

can't realistically be done here


Remove unnecessary regex

This fixes #24
As part of #23, I added some regexes that I thought were required to make escaped brackets (\{) and "special" brackets (eg %r{), but it turns out that none of that was necessary. All that was necessary was changing sweatifySpan to use the regexes created above, rather than the arrays.

Tested locally, works as expected. Commit should be merged and swackets package updated.


OTAR user query: investigate missing random harmonised files from the FTP

Jeremy S. identified three studies that used to host harmonised sumstats and do not anymore.
They are all part of big users' submissions that have had their multiple studies published and unpublished various times -perhaps the links with harmonised files for these individual studies were lost somewhere in the process?
GCST90002229 - from https://www.ebi.ac.uk/gwas/deposition/submission/5f0450988a8b2700011b9412
GCST90007250 - from https://www.ebi.ac.uk/gwas/deposition/submission/5f4e7796fa551c0001c9cec3
GCST90000291 - from
https://www.ebi.ac.uk/gwas/deposition/submission/5ee157e8427baf000130055e

thanks @jdhayhurst, I sent this ticket to @Jeremy37 as reference too


subscriber process event asyncly

process msg asynchronously to improve performance.

I merged it but have to think about it if this is an improvement or not, it increases allocations number
plus if the number of subscribers is high it actually slows down
before
BenchmarkPublish-4                   	 5214352	       226 ns/op	       0 B/op	       0 allocs/op
BenchmarkSubscribe-4                 	  632084	      3319 ns/op	     730 B/op	       5 allocs/op
Benchmark1Subscriber1Topic-4         	 5545311	       225 ns/op	       0 B/op	       0 allocs/op
Benchmark1Subscriber100Topics-4      	 2261838	       808 ns/op	       0 B/op	       0 allocs/op
Benchmark100Subscribers1Topic-4      	   61615	     19539 ns/op	       0 B/op	       0 allocs/op
Benchmark100Subscribers100Topics-4   	   60561	     34031 ns/op	       0 B/op	       0 allocs/op

after
BenchmarkPublish-4                   	  632365	      1935 ns/op	       1 B/op	       0 allocs/op
BenchmarkSubscribe-4                 	 1000000	      3950 ns/op	     726 B/op	       4 allocs/op
Benchmark1Subscriber1Topic-4         	  628796	      1835 ns/op	      11 B/op	       0 allocs/op
Benchmark1Subscriber100Topics-4      	  621848	      1902 ns/op	       0 B/op	       0 allocs/op
Benchmark100Subscribers1Topic-4      	    6956	    238003 ns/op	   12639 B/op	      28 allocs/op
Benchmark100Subscribers100Topics-4   	    6709	    298301 ns/op	       0 B/op	       0 allocs/op


[Improvements] Error handling in html, gtk, gorotunes and eyr file

Changes:

Error handling in html, gtk, gorotunes and eyr file
Removed unnecessary return nil
Renamed variable name of ProgramState  to ps
Doubt:
For Gtk-window//set-size  there are two times arg1 is used. It require arg2 instead of arg1?

Also getting some messages while build:
May be it is related to something GTK UI configuration environment. Not related to my changes.
go build -tags "b_gtk"
pkg-config --cflags  -- gio-2.0 glib-2.0 gobject-2.0 gio-2.0 glib-2.0 gobject-2.0 gio-2.0
Package gio-2.0 was not found in the pkg-config search path.
Perhaps you should add the directory containing gio-2.0.pc' to the PKG_CONFIG_PATH environment variable No package 'gio-2.0' found Package glib-2.0 was not found in the pkg-config search path. Perhaps you should add the directory containing glib-2.0.pc'
to the PKG_CONFIG_PATH environment variable
No package 'glib-2.0' found
Package gobject-2.0 was not found in the pkg-config search path.
Perhaps you should add the directory containing gobject-2.0.pc' to the PKG_CONFIG_PATH environment variable No package 'gobject-2.0' found Package gio-2.0 was not found in the pkg-config search path. Perhaps you should add the directory containing gio-2.0.pc'
to the PKG_CONFIG_PATH environment variable
No package 'gio-2.0' found
Package glib-2.0 was not found in the pkg-config search path.
Perhaps you should add the directory containing glib-2.0.pc' to the PKG_CONFIG_PATH environment variable No package 'glib-2.0' found Package gobject-2.0 was not found in the pkg-config search path. Perhaps you should add the directory containing gobject-2.0.pc'
to the PKG_CONFIG_PATH environment variable
No package 'gobject-2.0' found
Package gio-2.0 was not found in the pkg-config search path.
Perhaps you should add the directory containing `gio-2.0.pc'
to the PKG_CONFIG_PATH environment variable
No package 'gio-2.0' found
pkg-config: exit status 1

Hi, yes that with use of arg1 two times is a bug. Nice find! It's been few years since I've worked with GTK module for Rye ... probably the dependencies need to be updated. You can check what are the current versions of them and try changing the imports or I can?


Add ChatTask for multi-turn and follow-up generation

Description
This PR adds the ChatTask to easily define and prepare a task to generate follow up messages or even to use the multi-turn information to generate a response based on the previous context.
The ChatTask produces a Chat following the pre-defined formatting, but this should be discussed before merging, as an alignment between Prompt and Chat is much needed, as well as a proper identification of the different purposes to serve i.e. Chat and Instruct, and what's the best approach to cover those while still providing some formatting functions.
Closes #157
Example
import os
from uuid import uuid4

import argilla as rg
from datasets import Dataset
from distilabel.llm.openai import OpenAILLM
from distilabel.pipeline import Pipeline
from distilabel.tasks.chat import ChatTask

if __name__ == "__main__":
    dataset = Dataset.from_dict(
        {
            "messages": [
                [
                    {
                        "role": "system",
                        "content": "You are a helpful assistant that always answers like a pirate.",
                    },
                    {
                        "role": "user",
                        "content": "What's the most terrifying creature in the sea?",
                    },
                ]
            ],
        }
    )

    generator = OpenAILLM(
        model="gpt-4",
        task=ChatTask(input_format="openai", output_format="openai"),
        openai_api_key=os.getenv("OPENAI_API_KEY"),
        max_new_tokens=1024,
        prompt_format=None,
    )

    pipeline = Pipeline(generator=generator)
    dataset = pipeline.generate(
        dataset=dataset,
        shuffle_before_labelling=False,
        num_generations=2,
        skip_dry_run=True,
        display_progress_bar=True,
    )  # type: ignore

    rg_dataset = dataset.to_argilla()

    rg.init(
        api_url=os.getenv("ARGILLA_API_URL"),
        api_key=os.getenv("ARGILLA_API_KEY"),
    )

    rg_dataset.push_to_argilla(name=f"chat-{uuid4()}", workspace="admin")

Ideally, we want to generate data with follow-up questions.


Fix issue with IsDirty property

This fix a bug with IsDirty property in ValidatableModelBase

Looking at the line of code changed, it seems like a good bug flx.
One thing I would like to comment on is the definition of IsDirty when using validation with Entity Framework. An EF entity is dirty if it is added or deleted from the context, even though none of its properties is dirty. For example, a deletion may be invalid even though the unmodified entity itself is valid. What is happening, in effect, is the state of the entity is (or should be) part of the validation process.

This would have fixed it if I had not fixed it myself just now.


Add duck-facts extension

Description
Duck Facts is a Raycast extension that displays interesting facts about ducks. This extension generates and displays random facts about ducks using AI.
Repo: https://github.com/validatedev/duck-facts
Screencast
https://github.com/raycast/extensions/assets/29679552/8202dbb6-8243-4f9e-9d2d-199b8a4d6850
Checklist

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

Yaaaaaay thanks 🦆🎉 @pernielsentikaer


Deprecated Atom APIs will be removed June 1st

Atom will no longer load this package after June 1st without changes. There is only one change required on the latest version of this package:
All you need to do is merge https://github.com/bigfive/atom-sublime-select/pull/50 and publish a new version.
Visit https://gist.github.com/benogle/6d09e295c84b717ef9b4 and search for your package name to see up-to-date deprecations.
See https://github.com/atom/atom/issues/6867 for more info. Thanks!

Thanks so much for fixing this. The message in 1.2.3 was the most common deprecation message people were bumping into.


Windows printing wrong help message on incorrect command

Report from user:
"running into an issue with CCM on windows where it prints the general usage string with any incomplete command, rather than the usage for that specific command"

To clarify, CCM prints the usage string for any command, whether complete or incomplete. It seems to only occur in the command prompt and never in powershell. The only solution that worked was rebooting the Windows machine.


search.exclude is being ignored

Issue Type: Bug
The setting search.exclude is being ignored. I think the (partly default) globs configured are correct, so searching for text in files will happily search through everything, including the excluded files, and report a lot of "noise" to me that is of no use to me.
It could be that I'm doing something wrong, but I can't imagine what that would be. The search feature is pretty basic - not a whole lot that can go wrong in terms of usage.
It seems I found some sort of bug, if I'm not mistaken.
Here is a screenshot to prove what I mean:

Please note that this problem also occurs with all extensions disabled.
VS Code version: Code 1.40.2 (f359dd69833dd8800b54d458f6d37ab7c78df520, 2019-11-25T14:54:45.096Z)
OS version: Windows_NT x64 10.0.18363

System Info

Item
Value

CPUs
Intel(R) Core(TM) i9-8950HK CPU @ 2.90GHz (12 x 2904)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: enabledrasterization: enabledskia_renderer: disabled_offsurface_control: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.74GB (13.98GB free)

Process Argv

Screen Reader
no

VM
67%

Extensions (22)

Extension
Author (truncated)
Version

better-comments
aar
2.0.5

overtype
ada
0.2.0

Bookmarks
ale
10.6.0

color-info
bie
0.5.1

mustache
daw
1.1.1

vscode-eslint
dba
1.9.1

EditorConfig
Edi
0.14.2

vscode-great-icons
emm
2.1.47

file-icons
fil
1.0.19

duplicate
gee
1.0.1

docthis
joe
0.7.1

rainbow-csv
mec
1.3.1

vscode-duplicate
mrm
1.2.1

vscode-docker
ms-
0.9.0

remote-wsl
ms-
0.40.3

vscode-versionlens
pfl
0.24.0

vscode-template-literal-editor
pli
0.9.0

vscode-sort-json
ric
1.18.0

code-settings-sync
Sha
3.4.3

stylelint
shi
0.51.0

mdx
sil
0.1.0

vscode-svg-previewer
vit
0.5.1

(1 theme extensions excluded)

You have explicitly disabled the exclude:

To enable it again, use the "⚙minus" button at the end of exclude input in extended search panel:

Well, that's awkward 😣
Take it like this: that little button controls a lot of noise being outputted, or not. It's also really hard to see the state of that button, and it's not possible to know what it's trying to tell without hovering the mouse cursor on it.
I honestly didn't notice that text either "ignored files are disabled". It's not there in my mind, I just immediately focus on the results.

Sure, I don't blame you, it's easy to overlook such detail. At the same time it's hard to design such panel to be both non-distracting and verbose enough to clearly communicate everything it does.
(See: I've messed up instructions to get to this button myself: I've thought it is under left ">" toggle, but in fact is under right "..." toggle.)
I assume the default state of the "⚙➖ use Exclude Settings [...]" toggle button is "On" and user is expected to remember when and why he disables it and what it does. You are right that notice about it's effect above search results is really quite sunken…

Sorry @thany, thanks @myfonj :)


Be more strict with types

Make sure getMaxAge() always returns null or int. Also trigger deprecations if wrong types are passed to SetCookie
This will fix #2888

We should probably turn off the authoritative phpdoc types setting in phpstan (or whatever it's called).

Yeah... But I rather just add this to the baseline.
Note how I prepare the user and code for version 8.

Ok, sure. Re-gen the baseline then.

Baseline is updated and the CI is green.


add the autostarting .desktop file to the linux client, and update packaging with a feature gate

@cjb
This is all hacked into the Debian script, so I still need to factor out some shared Linux build code.

:thumbsup: after delete keybase.desktop

pushed as 29795fd5aa3e976c65ecebc8e13d7d268352a853


fix JDK7 compatibility

Now mockk is unusable with JDK7 'cause Integer.toUnsignedString() is private in it. This PR is an attempt to fix this incompatibility.

Codecov Report

Merging #42 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #42   +/-   ##
=========================================
  Coverage     78.95%   78.95%
  Complexity      527      527
=========================================
  Files            62       62
  Lines          1421     1421
  Branches        237      237
=========================================
  Hits           1122     1122
  Misses          196      196
  Partials        103      103

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 d0f0a14...fb8527f. Read the comment docs.

Thanks!

It seems like there are a lot more compatibility problems than a thought at first. :disappointed:
For example: ther's a class called JvmAutoHinter, which uses forEach method of map. JVM's Map has such method since JDK8 and Kotlin collections also has such method. The problem is: for some reason JVM's implementation is being used => lambda implements interface java.util.function.BiConsumer which is of course unavailable in JDK7. I guess it's a Kotlin's compiler plugin problem, cause target bytcode for your project should be 1.6.


Inconsistent URL Encoding in Generated Services

Description
In the services generated by the openapi-ts library, there appears to be an inconsistency in how URLs are encoded. Some URLs are returned URL encoded while others are not. This inconsistency can lead to issues when these URLs are used in requests.
Example
In the generated code, we have services like:
return __request(OpenAPI, {
    method: 'GET',
    url: '%2Fats%2Fjobs',
});

This should be:
return __request(OpenAPI, {
    method: 'GET',
    url: '/ats/jobs',
});

OpenAPI specification (optional)
https://api.kombo.dev/openapi.json
Configuration
npx @hey-api/openapi-ts -i https://api.kombo.dev/openapi.json -o generated

System information (optional)
macos, node v20.11.0, @hey-api/openapi-ts 0.43.0,

@k9ordon This is fixed in v0.43.1, please open a new issue if you spot any other problems!

@mrlubos It works perfectly with version 0.43.2. Thanks for the quick fix! 🥳


Update "date-fns" dependency to the latest version

Is your feature request related to a problem? Please describe.
I use the react-datepicker in one big multilingual project. We've found that the current version of the react-datepicker (3.1.3) does not have Hindi and Kazakh languages. That is probably because you use an old version of the "date-fns" library as a devDependency (2.0.1). The latest version (2.14.0) of "date-fns" library has those two locales.
Describe the solution you'd like
Update please the "date-fns" library to the version 2.14.0
Let me know if it's feasible.
Thank you in advance,
Artem

Feel free to submit a PR for this, happy to review it.

Can't push to a feature branch => permissions denied.

You'll need to fork the repository first before pushing to a branch. Once you've made your changes, you can open a PR to this repo.

Btw, this lib isn' a devDependency but a peerDependency. This means, you can specify the wanted version yourself as long as it's above "2.0.1".

Thanks, I see it's already here: https://github.com/Hacker0x01/react-datepicker/pull/2271


Remove redundant jarvis.jl file

There were two jarvis.jl files in the jarvis_march directory, this pull request simply removes the redundant file.

Hey @Antetokounpo, thanks for the change! Since you are a first-time contributor, would you mind putting your name at the bottom of the CONTRIBUTORS.md file? If you would prefer not to do that, it's totally fine!

Yeah sorry, I didn't think I had to since it was a really small contribution. I added it.


Update README.md with Passkey details

fixes #88

I checked the spec and found that the BLE passkey is exactly 6 digits.
Error occurred for more than 7 digits.
(https://github.com/espressif/esp-nimble/blob/49d60705f6e085e670b6d425387e75054bf5dffe/nimble/host/src/ble_sm.c#L2792)

Interesting.. most things I looked up said 6-9.. good to know.
I can add that to my PR later. Do you want me to add anything to the methods or any asserts?
-Sam

From: taks @.>
Sent: Friday, February 16, 2024 7:30:15 AM
To: taks/esp32-nimble @.>
Cc: sam @.>; Author @.>
Subject: Re: [taks/esp32-nimble] Update README.md with Passkey details (PR #89)
I checked the spec and found that the BLE passkey is exactly 6 digits.
Error occurred for more than 7 digits.
(https://github.com/espressif/esp-nimble/blob/49d60705f6e085e670b6d425387e75054bf5dffe/nimble/host/src/ble_sm.c#L2792)
—
Reply to this email directly, view it on GitHubhttps://github.com/taks/esp32-nimble/pull/89#issuecomment-1948484974, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACDGQ5YZ7H2FHPCVVBDP7CTYT5UPPAVCNFSM6AAAAABDL6B2UCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNBYGQ4DIOJXGQ.
You are receiving this because you authored the thread.Message ID: @.***>

It would be appreciated if you could add this to the rustdoc.

Do you want me to add anything to the methods or any asserts?

Good idea. I think there should be a debug assert that the passkey must be between 000000 and 999999 (inclusive). Doesn't have to be part of this PR.

Feel free to add anything (if you're asking me) I just haven't had more time to finish it up.
-Sam

From: Rajas Paranjpe @.>
Sent: Sunday, February 25, 2024 7:00:12 PM
To: taks/esp32-nimble @.>
Cc: sam @.>; Author @.>
Subject: Re: [taks/esp32-nimble] Update README.md with Passkey details (PR #89)
Do you want me to add anything to the methods or any asserts?
Good idea. I think there should be a debug assert that the passkey must be between 000000 and 999999 (inclusive). Doesn't have to be part of this PR.
—
Reply to this email directly, view it on GitHubhttps://github.com/taks/esp32-nimble/pull/89#issuecomment-1963190436, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACDGQ52T4F3UMRXT7WRK4HDYVPUCZAVCNFSM6AAAAABDL6B2UCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRTGE4TANBTGY.
You are receiving this because you authored the thread.Message ID: @.***>

Took a stab...open to comments

Good comments I'll make some revisions.. should be exactly 6
-Sam

From: Rajas Paranjpe @.>
Sent: Monday, February 26, 2024 8:17:56 PM
To: taks/esp32-nimble @.>
Cc: sam @.>; Author @.>
Subject: Re: [taks/esp32-nimble] Update README.md with Passkey details (PR #89)
@ChocolateLoverRaj requested changes on this pull request.

In src/ble_security.rshttps://github.com/taks/esp32-nimble/pull/89#discussion_r1503581257:

/// * The passkey returned should be between 000000..=999999 inclusive

/// and will always be exactly 6 digits, i.e. if 1234u32 is returned
/// the correct passkey will be '001234' and should be presented to the user padded to 6 digits
/// i.e. format!("{:0>6}",pkey)

I think this can just say "The passkey will always be 6 digits. Setting the passkey to 1234 will actually make the passkey "001234""

In README.mdhttps://github.com/taks/esp32-nimble/pull/89#discussion_r1503581421:

.set_io_cap(SecurityIOCap::NoInputNoOutput) // You can choose any IO capability
.resolve_rpa(); // Crucial for managing iOS's dynamic Bluetooth addresses

- `.set_auth(AuthReq::Bond)` sets up bonding, crucial for storing security keys that enable future automatic reconnections.
- `.resolve_rpa()`: This function is essential for adapting to the changing Bluetooth addresses used by iOS devices, a feature known as Resolvable Private Address (RPA). It's vital for maintaining reliable and seamless connections with iOS devices, ensuring that your ESP32 device can recognize and reconnect to an iOS device even when its Bluetooth address changes.
+- BLE Passkeys are a minimum of 6 digits by spec, so if you set a passkey of '1234' it is actually '001234' so to properly display the code to a user you must pad the left i.e. `format!("{:0>6}",pkey)`

Is it minimum 6 digits or exactly 6 digits? It's kind of confusing.

—
Reply to this email directly, view it on GitHub<https://github.com/taks/esp32-nimble/pull/89#pullrequestreview-1902440097>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/ACDGQ537RTX655AHVZ76AZTYVVF6JAVCNFSM6AAAAABDL6B2UCVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTSMBSGQ2DAMBZG4>.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

Thanks for the change.

Thank you, the library has been a big help to my project so least I can do is contribute a little
-Sam

From: taks @.>
Sent: Wednesday, February 28, 2024 12:08:04 AM
To: taks/esp32-nimble @.>
Cc: sam @.>; Author @.>
Subject: Re: [taks/esp32-nimble] Update README.md with Passkey details (PR #89)
Thanks for the change.
—
Reply to this email directly, view it on GitHubhttps://github.com/taks/esp32-nimble/pull/89#issuecomment-1968359755, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACDGQ5ZBP4LHMB3VMILKLGDYV3JVJAVCNFSM6AAAAABDL6B2UCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRYGM2TSNZVGU.
You are receiving this because you authored the thread.Message ID: @.***>


branching diagrams link broken

404 on this link: https://github.com/dslp/dslp/blob/master/branching/branching-patterns.pdf

Fixed in #9


Add genertion of tests results in GitHub Action Summary

This PR adds a step to generate a test result that will be added to the GitHub Action Summary of the test stages

pytest generates XML reports as well, so we should see if we can add the summaries to conda-python-tests.yaml also.

https://github.com/rapidsai/rmm/blob/09208b4a9a086db8f2b90984522114bd6deb8e5c/ci/test_python.sh#L39

pytest generates XML reports as well, so we should see if we can add the summaries to conda-python-tests.yaml also.
* https://github.com/rapidsai/rmm/blob/09208b4a9a086db8f2b90984522114bd6deb8e5c/ci/test_python.sh#L39

Yes I will, I'm just testing with CPP for now.


Jetbrains IDE keeps rendering tooltips

Hyprland Version
Hyprland, built from branch HEAD at commit bfb4d66c8144a36f2c52530f10ae4bac2ac03b09  (compositor: spawn environment setup with keybind manager (3722)).
Bug or Regression?
Regression
Description
Jetbrains IDE keeps rendering tooltip. The bug I have is identical to #2245 . It is working in v0.33.1.
I bisected and found ba2af6f86d26fbf4b51aa37bf901e0118f46bbe3 to be faulty.
Reverting the commit on top of bfb4d66c8144a36f2c52530f10ae4bac2ac03b09 removes the bug.
How to reproduce
Open a JetBrains IDE and leave mouse idle on left bar buttons. The tooltip will flicker, opening and closing repeatedly.
There is a video in the older ticket #2245 .
Crash reports, logs, images, videos
hyprland.log
Reading the socket while hovering a button (for 2sec) that should display a tooltip : socket2.log
My config

Thanks to https://github.com/hyprwm/Hyprland/issues/2315#issuecomment-1555896701 it solved my issue with version v0.34.0 !
The workaround is to add windowrulev2 = noinitialfocus,class:^jetbrains-(?!toolbox),floating:1  to your config.


Do not override replicated_tensor_sharding attribute on construction

Since the replication factor can change at runtime, the choice of enabling RTS should be dynamic
What does this PR do?

Fixes # (issue)
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


[Suggestion] Portable git for version control

Right now, for majority of people using MeatKit, they will not have git installed on their machine and so they will have absolutely 0 version control. Version control is very important, especially with the legacy of Wolfie's project being destroyed. It would be very beneficiary if a version of portable git was included in the repo, along with a tool to manage versioning from inside Unity (I can program this).

also, maybe issue labels?

I agree that version control and backups are very important, and I feel like everyone should be doing one or both. But this to me just seems like a "You can lead a horse to water but you can't make them drink" situation.
The options are already there and it's not difficult to use them. Those who don't now likely still won't after, and those who didn't and learned why they should have hopefully went the extra couple steps to ensure it won't happen again. For this reason I'm not interested in having a feature like this in the main MeatKit repo.
If you just want an integrated Git(Hub) client for Unity, then that also already exists: https://unity.github.com/, otherwise if that's not quite what you're after I would recommend making your tool as a package that can be imported into any MeatKit project for those who wish to use it.

I agree with your points, but what I mean is having portable git in the project, along side a tool in meatkit that tracks it, like a button that says "backup project"


Support PostgreSQL 10

In PostgreSQL 10 environment, I saw the following error.
Failure/Error: SeedFu.seed

ActiveRecord::StatementInvalid:
  PG::UndefinedColumn: ERROR:  column "increment_by" does not exist

In version 10, it's introduced pg_sequence catalog that contains information about sequences. Each sequences no longer have increment_by and min_value.

This PR works fine for me 😍
@aserafin
Are there any chance to merge this one?

Reference: https://www.postgresql.org/docs/10/static/release-10.html

Move sequences' metadata fields into a new pg_sequence system catalog (Peter Eisentraut)
A sequence relation now stores only the fields that can be modified by nextval(), that is last_value, log_cnt, and is_called. Other sequence properties, such as the starting value and increment, are kept in a corresponding row of the pg_sequence catalog. ALTER SEQUENCE updates are now fully transactional, implying that the sequence is locked until commit. The nextval() and setval() functions remain nontransactional.
The main incompatibility introduced by this change is that selecting from a sequence relation now returns only the three fields named above. To obtain the sequence's other properties, applications must look into pg_sequence. The new system view pg_sequences can also be used for this purpose; it provides column names that are more compatible with existing code.
The output of psql's \d command for a sequence has been redesigned, too.

@mbleigh any chance you can merge this, it is blocking us from upgrading to pg 10 at Discourse

I've merged and published 2.3.7 to rubygems 👍


Dynamic page meta tags

Dynamic Title change has been implemented.
Dynamic meta tags is not yet working.


Restoring a PVC/PV using the default storageClass

Ark v0.5.1
k8s v1.7.10
Currently we rely on the default storageClass to dynamically provision our EBS volumes.  Ark will happily snapshot them and even restore the PV and PVC, but because it's a default storageClass there's no volumeID specified, so the PVC happily creates a new PV and ignores the previous one.
My question is, if we set volumeReclaimPolicy: Retain and restore from an ark backup, is it expected that we will need to manually modify all PVCs to point at the restored volumes?  Are there any plans to handle this situation automatically in ark?

Can you paste sample yaml for a PV and PVC?

Sure, here's a basic example without the reclaim policy set:
kind: PersistentVolumeClaim
metadata:
  annotations:
    pv.kubernetes.io/bind-completed: "yes"
    pv.kubernetes.io/bound-by-controller: "yes"
    volume.beta.kubernetes.io/storage-provisioner: kubernetes.io/aws-ebs
  creationTimestamp: 2017-11-29T20:23:32Z
  labels:
    app: dex-postgresql
    chart: postgresql-0.8.3
    heritage: Tiller
    release: dex
  name: dex-postgresql
  namespace: security
  resourceVersion: "4197"
  selfLink: /api/v1/namespaces/security/persistentvolumeclaims/dex-postgresql
  uid: 2b6fa8c6-d543-11e7-ad59-02df93e00a3a
spec:
  accessModes:
  - ReadWriteOnce
  resources:
    requests:
      storage: 8Gi
  storageClassName: gp2
  volumeName: pvc-2b6fa8c6-d543-11e7-ad59-02df93e00a3a
status:
  accessModes:
  - ReadWriteOnce
  capacity:
    storage: 8Gi
  phase: Bound

kind: PersistentVolume
metadata:
  annotations:
    kubernetes.io/createdby: aws-ebs-dynamic-provisioner
    pv.kubernetes.io/bound-by-controller: "yes"
    pv.kubernetes.io/provisioned-by: kubernetes.io/aws-ebs
  creationTimestamp: 2017-11-29T20:23:33Z
  labels:
    failure-domain.beta.kubernetes.io/region: us-east-1
    failure-domain.beta.kubernetes.io/zone: us-east-1b
  name: pvc-2b6fa8c6-d543-11e7-ad59-02df93e00a3a
  resourceVersion: "4195"
  selfLink: /api/v1/persistentvolumes/pvc-2b6fa8c6-d543-11e7-ad59-02df93e00a3a
  uid: 2bcaef10-d543-11e7-ad59-02df93e00a3a
spec:
  accessModes:
  - ReadWriteOnce
  awsElasticBlockStore:
    fsType: ext4
    volumeID: aws://us-east-1b/vol-009dd3e2df1be41c0
  capacity:
    storage: 8Gi
  claimRef:
    apiVersion: v1
    kind: PersistentVolumeClaim
    name: dex-postgresql
    namespace: security
    resourceVersion: "4153"
    uid: 2b6fa8c6-d543-11e7-ad59-02df93e00a3a
  persistentVolumeReclaimPolicy: Delete
  storageClassName: gp2
status:
  phase: Bound

So for testing, I delete the cluster this volume is in (which deletes the PV because of the reclaimPolicy), the backup has a valid snapshot.  If I redeploy the cluster and install ark, then do a restore, the EBS volume gets recreated from the snapshot, but the PVC is using a new different PV.

Can you also share the status portion from ark backup get <backup name>?

but because it's a default storageClass there's no volumeID specified, so the PVC happily creates a new PV and ignores the previous one.

Where are you seeing a lack of a volumeID?

Because we're using the default storageClass we don't specify the volume ID in the PVC.
  expiration: 2017-11-30T20:33:21Z
  phase: Completed
  validationErrors: null
  version: 1
  volumeBackups:
    pvc-2b6fa8c6-d543-11e7-ad59-02df93e00a3a:
      availabilityZone: us-east-1b
      snapshotID: snap-0618ccfa847dbe65f
      type: gp2

Because we're using the default storageClass we don't specify the volume ID in the PVC.

That's normal/expected and Ark should work just fine with that.

Could you please check your restore's logs and paste anything related to pvc-2b6fa8c6-d543-11e7-ad59-02df93e00a3a (and a few lines of context before/after too)?

I suppose we should work around this by using
    matchLabels:
      app: ...
      release: ...

In our PVCs then?  It was just unexpected behavior to me that it recreated the EBS volume, but then didn't use it.

You may be running into a bug, but the expected behavior is that it will create a new EBS volume from the EBS snapshot, and associate it with the PV/PVC.

You shouldn't need to do anything with matchLabels, no. Let's keep debugging 😄

ok, doing another delete/restore to capture logs (sorry, didn't keep the old ones around due to the low TTL)

ok, it worked that time...  the difference being that this is on k8s 1.8.4 now (testing upgrades right now).
I was wondering if maybe this happens if the pod gets scheduled to a different AZ than the original PV was created in?  We have 3 AZs configured.
I'll try to recreate with 1.7 if you think that might be relevant.

I was wondering if maybe this happens if the pod gets scheduled to a different AZ than the original PV was created in? We have 3 AZs configured.

It could be something along these lines. We'll examine the code and see if anything stands out.

  We have all our cluster configs in the same kubeconfig, and I believe I was actually backing up and restoring a different cluster than I was expecting because I forgot to set the current-context.
related: https://github.com/heptio/ark/issues/207


Add the Symphony Elements "button" tag

Fixed in https://github.com/symphonyoss/messageml-utils/commit/3d6ab0913ec218c61a22d0c2ba7f9211f05247b0


Enable an I/O (FieldSet) caching

Locally as well as on cluster, user may have - next to the original storage location of the fieldset files - another storage location for cached, high-speed file access (e.g. a solid-state drive). The idea, logically, would be to automatically use this high-speed storage location as cache to pre-buffer fieldset data and thus increase performance.
The initial motivation came from the OceanParcels internal, brand-new LORENZ cluster, which has high-speed I/O facilities for this purpuse. That said, the caching class is ideally made so that people can setup this caching mode also for their cluster, or even for a local laptop with multiple SSD's.

@erikvansebille Summary of the PR:
Purpose: buffer (or: cache) Field files on  e.g. SSD's locally and on clusters -> use the SSD's on LORENZ nodes
Parent branch: master (3.x and onwards) -> needs to merge in changes of zarr if need be
Status: merge-ready from momentary master head and can be merged to master -> changes are rather minor; tests available; tutorial notebook available; most business is going on in filecache.py
Documentation status: test is available; tutorial notebook is available; discussed the process and algorithm with @lvankampenhout in detail in June/July
advice: please merge into master after tests finish

Closing this PR for now, as will not integrate into master anytime soon


Cannot run Arm64 version on Mac OS with M1/.net 7

Followed instructions to install and run the ML.NET Tooling on MacOS on an M1 MacBook Pro with .net 7 installed.
dotnet tool install -g mlnet-osx-arm64
Tools directory '/Users/randy.magruder/.dotnet/tools' is not currently on the PATH environment variable.
If you are using zsh, you can add it to your profile by running the following command:
cat << \EOF >> ~/.zprofile
Add .NET Core SDK tools
export PATH="$PATH:/Users/randy.magruder/.dotnet/tools"
EOF
And run zsh -l to make it available for current session.
You can only add it to the current session by running the following command:
export PATH="$PATH:/Users/XXXXXXXXXXX/.dotnet/tools"
You can invoke the tool using the following command: mlnet
Tool 'mlnet-osx-arm64' (version '16.14.3') was successfully installed.
~ % ~/.dotnet/tools/mlnet
Unhandled exception. System.BadImageFormatException: Could not load file or assembly '/Users/XXXXXXXXXXXX/.dotnet/tools/.store/mlnet-osx-arm64/16.14.3/mlnet-osx-arm64/16.14.3/tools/net6.0/any/mlnet-osx-arm64.dll'. An attempt was made to load a program with an incorrect format.
File name: '/Users/XXXXXXXXXXXX/.dotnet/tools/.store/mlnet-osx-arm64/16.14.3/mlnet-osx-arm64/16.14.3/tools/net6.0/any/mlnet-osx-arm64.dll'
zsh: abort      ~/.dotnet/tools/mlnet

Hi @rmagruder I have some trouble reproducing the issue you have on M1/.net6. My assumption is you don't have dotnet 6.* installed on your system?

Tutorial merged.

I will try again!


fix address references for ts filler

Minor broken address references fixed.

🟡 Heimdall Review Status

Requirement
Status
More Info

Reviews

🟡
0/1

Denominator calculation

Show calculation

1 if user is bot
0

1 if user is external
0

From .codeflow.yml

1

Additional review requirements

Show calculation

Max
0

0

From CODEOWNERS

0

Global minimum
0

Max

1

1

1 if commit is unverified
0

Sum
1


A SHOT IN THE DARK

I've been studying coding for a little minute now and I know it's a shot in the dark but I would like for you to be my mentor. I know you probably get asked this often but I am really eager to learn and already have been studying on my own right now just JavaScript but I'm eager to learn whatever. So if it doesn't take up to much of your time could you PLEASE CONSIDER IT p.s. how bad would it be to have a protege

Find me on LinkedIn and we can continue the conversation there. GitHub (specifically on repositories) isn't a place for these kinds of discussions.


Full replays from Olympics.com seems to be broken

Checklist

[X] I'm reporting a broken site
[X] I've verified that I'm running yt-dlp version 2022.02.04. (update instructions)
[X] I've checked that all provided URLs are alive and playable in a browser
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[ ] I've read about sharing account credentials and I'm willing to share it if required

Region
Sweden
Description
Trying to download full replays from Beijing 2022 doesn't work, but it works without problem when downloading Tokyo 2020 replays. The URLs are different compared to earlier events, the URLs that don't work are https://olympics.com/en/original-series/episode/ when the previous URLs that work are https://olympics.com/en/video/
Verbose log
[debug] Command-line config: ['-vU', '-F', 'https://olympics.com/en/original-series/episode/mixed-relay-biathlon-beijing-2022-replays']
[debug] Encodings: locale cp1252, fs utf-8, out utf-8, err utf-8, pref cp1252
[debug] yt-dlp version 2022.02.04 [c1653e9] (win_exe)
[debug] Python version 3.8.10 (CPython 64bit) - Windows-10-10.0.22000-SP0
[debug] exe versions: ffmpeg git-2020-02-03-1c15111, ffprobe git-2020-02-03-1c15111
[debug] Optional libraries: Cryptodome, mutagen, sqlite, websockets
[debug] Proxy map: {}
Latest version: 2022.02.04, Current version: 2022.02.04
yt-dlp is up to date (2022.02.04)
[debug] [generic] Extracting URL: https://olympics.com/en/original-series/episode/mixed-relay-biathlon-beijing-2022-replays
[generic] mixed-relay-biathlon-beijing-2022-replays: Requesting header
WARNING: [generic] Falling back on generic information extractor.
[generic] mixed-relay-biathlon-beijing-2022-replays: Downloading webpage
[generic] mixed-relay-biathlon-beijing-2022-replays: Extracting information
[debug] Looking for video embeds
[debug] Identified a JSON LD
[generic] mixed-relay-biathlon-beijing-2022-replays: Downloading m3u8 information
ERROR: [generic] Failed to download m3u8 information: HTTP Error 403: Forbidden (caused by <HTTPError 403: 'Forbidden'>); please report this issue on  https://github.com/yt-dlp/yt-dlp , filling out the "Broken site" issue template properly. Confirm you are on the latest version using -U (caused by <HTTPError 403: 'Forbidden'>); please report this issue on  https://github.com/yt-dlp/yt-dlp , filling out the "Broken site" issue template properly. Confirm you are on the latest version using -U
  File "yt_dlp\extractor\common.py", line 730, in _request_webpage
  File "yt_dlp\YoutubeDL.py", line 3558, in urlopen
  File "urllib\request.py", line 531, in open
  File "urllib\request.py", line 640, in http_response
  File "urllib\request.py", line 569, in error
  File "urllib\request.py", line 502, in _call_chain
  File "urllib\request.py", line 649, in http_error_default

@Ashish0804 @u-spec-png

Looks like a situation where the video exists at both the original and this address, as they both call the same api endpoint with the same parameters.  @Zalbu can you get to the replay from https://olympics.com/en/video/mixed-relay-biathlon-beijing-2022-replays ?
As a short-term fix, until someone who knows what they are doing touches it you can change line 12 in extractors/olympics.py
from:
_VALID_URL = r'https?://(?:www.)?olympics.com(?:/tokyo-2020)?/[a-z]{2}/(?:replay|video)/(?P[^/#&?]+)'
to:
_VALID_URL = r'https?://(?:www.)?olympics.com(?:/tokyo-2020)?/[a-z]{2}/(?:replay|video|original-series/episode)/(?P[^/#&?]+)'

Looks like a situation where the video exists at both the original and this address, as they both call the same api endpoint with the same parameters. @Zalbu can you get to the replay from https://olympics.com/en/video/mixed-relay-biathlon-beijing-2022-replays ?

Yup, both accessing it in the browser and downloading it with ytp-dl works like normal, how strange. And now I get sent to the correct https://olympics.com/en/video/ URL as well when accessing the video from the Olympics website.
Shouldn't be a difficult fix to patch then if only another supported URL needs to be added.

Looks like replays from Paris 2024 Olympics are also broken, for example with https://olympics.com/fr/paris-2024/videos/jpn-fra-phase-de-groupe-h-basketball-jeux-olympiques-de-paris-2024
[generic] Extracting URL: https://olympics.com/fr/paris-2024/videos/jpn-fra-phase-de-groupe-h-basketball-jeux-olympiques-de-paris-2024
[generic] jpn-fra-phase-de-groupe-h-basketball-jeux-olympiques-de-paris-2024: Downloading webpage
WARNING: [generic] Falling back on generic information extractor
[generic] jpn-fra-phase-de-groupe-h-basketball-jeux-olympiques-de-paris-2024: Extracting information
ERROR: Unsupported URL: https://olympics.com/fr/paris-2024/videos/jpn-fra-phase-de-groupe-h-basketball-jeux-olympiques-de-paris-2024

Fixed in 2f1ddfe12a2c174bc777264c5c8ffe7ca0922d94


annotating inheriting classes may creates errors

Due to it's way of working if the parent class did not yet try to give annotations, the when a daughter class have one of it's attribute assigned it may result in it having the wrong type

todo, for milestone v1.2.3, by default, if a class has no annotation for an attribute, the attribute won't be typechecked.


How to set rssButton to False

rssButton is defined as always true, how do I set it to false?
	{{ partial "nav.html" (dict "context" . "rssButton" true "contentType" $contentType) }}
I am new to the Hugo ecosystem. Thanks for helping helping such a novice question :)

Currently, Dream Plus doesn't have such a feature which allows one to disable the RSS button. If you want to disable it, just replace the boolean value next to rssButton with false wherever the nav.html partial has been called.
That's the quick local fix. Do you want this to be implemented as a feature in the theme?

I would be able to do it manually, but definitely a worthwhile feature to consider implementing.

Ok, I will implement this feature in the later versions then.