0.3.4 not working on arm

Describe the bug
A clear and concise description of what the bug is.
confusing image tagging
image tag 0.3.4 only support arch amd64
and now there is an v0.3.4 with multiarch support
To Reproduce
on arm docker run plndr/kube-vip:0.3.4
standard_init_linux.go:228: exec user process caused: exec format error
Expected behavior
A clear and concise description of what you expected to happen.
docker run plndr/kube-vip:0.3.4 on arm should work like version 0.3.0 before.

also kube-vip manifest is not working as it use 0.3.4

Image tagging is fixed from 0.3.5 onwards, apologies for the confusion!


⚠ Samsung has degraded performance

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

HTTP code: 200
Response time: 2355 ms

Resolved: Samsung performance has improved in ddfd536.


Enhancement (drafted) - extend graph to allow latex equations in popups

https://chat.openai.com/share/ab2b5332-30c5-4900-b607-be4c89da3f60

<script>
window.MathJax = {
  tex: {
    inlineMath: [['$', '$'], ['\\(', '\\)']]
  }
};
</script>
<script async src="https://cdnjs.cloudflare.com/ajax/libs/mathjax/3.2.0/es5/tex-chtml.min.js"></script>

Also, in the renderMathJax function, use MathJax.typesetPromise() instead of MathJax.Hub.Queue(). Here's an updated version of the function:
function renderMathJax(content) {
  MathJax.typesetPromise([content]).catch(function (err) {
    console.log(err.message);
  });
}

did you get chatgpt to spit out anything ? I've asked it before - give me in latex format - can you share picture? It would be awesome to get a blow out of dissecting the maths that each paper is built on.

Use Bard, because is train with the latest data and it is free, whereas in chatgpt you have to pay to use chatgpt4


PP-12468 Bump pay-js-commons update base client import

Update import of the base client to use the new format in pay-js-commons.
The client has updated from axios-base-client to https-base-client.
Update code and all tests.

@iqbalgds can this draft be closed?

@iqbalgds If you do want to keep this you will need to rebase against master since we changed which github PR checks are required and also renamed them for consistency

No longer required - closing.


Delete DofIndex, symbol and is fix! from dofs

Closes #57 and #40

Codecov Report

Merging #58 (d4190c4) into main (4500944) will decrease coverage by 0.21%.
The diff coverage is 34.88%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main      #58      +/-   ##
==========================================
- Coverage   26.04%   25.84%   -0.21%
==========================================
  Files          28       28
  Lines         883      859      -24
==========================================
- Hits          230      222       -8
+ Misses        653      637      -16

Impacted Files
Coverage Δ

src/ONSAS.jl
100.00% <ø> (ø)

src/Utils.jl
66.66% <ø> (ø)

src/algorithms/NR.jl
8.33% <0.00%> (ø)

src/core/ONSAS_solve.jl
0.00% <ø> (ø)

src/core/update_structure_state.jl
0.00% <0.00%> (ø)

src/interface/StructuralSolvers.jl
14.28% <ø> (ø)

src/interface/BoundaryConditions.jl
33.33% <37.50%> (ø)

src/interface/Meshes.jl
86.20% <66.66%> (ø)

src/interface/StructuralModel.jl
87.27% <66.66%> (ø)

src/interface/Elements.jl
65.00% <72.72%> (+5.50%)
:arrow_up:

... and 1 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


🤖 Update one or more manifest files in the .ci directory

Includes:

https://github.com/JuliaRegistries/RegistryCI.jl/pull/495
https://github.com/JuliaRegistries/General/pull/78464


Microsoft.ML sentiment-analysis results issue with different versions of Microsoft.ML

Seems to be an issue with this sample
The sample works with Nuget Package Microsoft.ML 0.3.0
When running 0.4.0 you get different results.
That is, both results in the console are positive for 0.4.0. The first result should be negative if it is supposed to be the same as 0.3.0, which I'd assume it would.

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

ID: c8e22825-c4ee-5a36-f911-8ad456970ecc
Version Independent ID: ad0bf640-7dfc-1184-077c-358bb14277a7
Content: Use ML.NET in a sentiment analysis binary classification scenario
Content Source: docs/machine-learning/tutorials/sentiment-analysis.md
Product: dotnet-ml
GitHub Login: @JRAlexander
Microsoft Alias: johalex

Same issue, get both as positive even if I use exact wording from sample, i.e. 'RUDE'
Also noted that if use 2.0 in csproj file get lowere percentages than if use 2.1.
Tried bringing in the csproj, project.cs and SentimentData.cs files from samples on GitHub but got the same, thats how I saw the test results differ based on .net core version

Indeed, output with 0.3.0 version:
PredictionModel quality metrics evaluation
------------------------------------------
Accuracy: 66.67%
Auc: 94.44%
F1Score: 75.00%

Sentiment Predictions
---------------------
Sentiment: Please refrain from adding nonsense to Wikipedia. | Prediction: Negative
Sentiment: He is the best, and the article should say that. | Prediction: Positive

Output with 0.4.0 version:
PredictionModel quality metrics evaluation
------------------------------------------
Accuracy: 61.11%
Auc: 85.19%
F1Score: 72.00%

Sentiment Predictions
---------------------
Sentiment: Please refrain from adding nonsense to Wikipedia. | Prediction: Positive
Sentiment: He is the best, and the article should say that. | Prediction: Positive

Note, evaluation metrics values have also changed.
It's not clear from the release notes what might have caused the observed change.
The used trainer is FastTreeBinaryClassifier. Its history shows mostly doc changes with some API update changes.
One more guess is that this commit might have introduced different behavior.
@OliaG do you know who might explain the observed difference in the tutorial output?

@GalOshri, any insight on this?

This is likely due to the datasets being very small so random fluctuations can cause large differences in metrics (the 5% difference in accuracy just means that 1 out of the 18 examples in the test data flipped).
The other factor is that the tree parameters ({ NumLeaves = 5, NumTrees = 5, MinDocumentsInLeafs = 2 }) are very small which limits the performance of the trees. When I switch to default parameters, the AUC matches between 0.3 and 0.4. The accuracy is still different but only corresponds to one example's prediction being flipped.
I tried the same pipeline on a much larger dataset and the AUC/accuracy differed by around 0.3% (which likely is still due to the relatively small tree parameters and current inability to set a random seed).

@GalOshri the code uses TextFeaturizer that seems to have been influenced by dotnet/machinelearning#548 (am I correct here?). So, two sets of numerical features produced by 0.3 and 0.4 versions might be different. Thus, trained models differ. Is my understanding correct?

Good point. It seems like the implementation changed which likely affects the predictions. @zeahmed, can you please verify this?

Due the change in https://github.com/dotnet/machinelearning/pull/548, one of the prediction changed.
Previously, it was 12/18 = 0.666666
No, it is 11/18 = 0.61111
The dataset is so small to make any conclusion. However, @justinormont may help us on this.
@justinormont, Do we have benchmarks for it? Can you confirm if the change in TextFeaturizer has adversely effected the benchmarks?

@zeahmed is this file considered as benchmarks? That file was changed as part of dotnet/machinelearning#548 as well.

@pkulikov, no these are just tests. Benchmarks are different.

I get more accurate results with the previous version (0.3.0) ? nothing else changed in my solution (sentiment analysis)

I tried running the solution today, it didn't work initially for me with given hyperparameters, but when I changed them to NumLeaves = 50, NumTrees = 50, MinDocumentsInLeafs = 20 , it gave me correct results.

Thanks, @mangeshw. @GalOshri - should I update the tutorial accordingly?

Running the sample code provided on GitHub on ML.net 0.5.0 causes predictions to be positive regardless of the sentiment text being provided. I've even used negative texts from the training set, still returns a positive prediction.
However, adding data from the training set to the testing set causes the accuracy to go up in the metrics output, so my theory is there's something wrong within the Predict method only.

Looks like it's just the output displaying "Positive" for negative sentiments and vice versa..
https://github.com/dotnet/samples/pull/329

Microsoft.ML 0.6.0
PredictionModel quality metrics evaluation
Accuracy: 94.44%
Auc: 98.77%
F1Score: 94.74%
Sentiment Predictions
Sentiment: Please refrain from adding nonsense to Wikipedia. | Prediction: Positive
Sentiment: He is the best, and the article should say that. | Prediction: Negative

The last code snippet has error:
foreach (var item in sentimentsAndPredictions)
{
Console.WriteLine($"Sentiment: {item.sentiment.SentimentText} | Prediction: {(item.prediction.Sentiment ? "Negative" : "Positive")}");
}
Console.WriteLine();
Should be
foreach (var item in sentimentsAndPredictions)
{
Console.WriteLine($"Sentiment: {item.sentiment.SentimentText} | Prediction: {(item.prediction.Sentiment ? "Positive" : "Negative")}");
}
Console.WriteLine();
PredictionModel quality metrics evaluation
Accuracy: 94.44%
Auc: 98.77%
F1Score: 94.74%

@bolajiniy
Try taking a look at the training set, it should be pretty obvious that 0 = positive and 1 = negative.
Please reference https://github.com/dotnet/samples/pull/329
Let me know if you still believe it to be an error.

Perhaps we should increase the size of the sample. The dataset is currently sized for a unit test; I don't think its small size is valuable for example code. Perhaps we should increase the size of the sample for use in the example. If the same dataset is used in unit tests, we should ensure the runtime of the unit tests are not adversely impacted. The only point of the unit tests are to see if something changed; they are not expected to make useful models.
Similar issues were mentioned in the main repo: https://github.com/dotnet/machinelearning/issues/708#issuecomment-425706329:

Try running on the full-sized dataset: https://aka.ms/tlc-resources/benchmarks/WikiDetoxAnnotated160kRows.tsv
The wikipedia-detox-250-line-data.tsv dataset is a 250 row sample of the original 160k rows. Training on this small of a sample won't create a useful model.

If someone wants to try a run on the full dataset, this would give the upper bound for how well a specific pipeline (transforms+learner) will perform on the dataset. Running on any small sample of the dataset will do worse (plus noise). We could find an optimal trade-off between sample size (which determines runtime) and output accuracy. This would let us pick a more optimal size dataset for the example.

tried the full dataset on v0.6.0
Accuracy: 50.00 %
Auc: 50.00 %
F1Score: NaN
Sentiment Predictions
Sentiment: Please refrain from adding nonsense to Wikipedia. | Prediction: Positive
Sentiment: He is the best, and the article should say that. | Prediction: Positive

Hi Team, I am use the sample dataset with 0.6.0 version
Output:
Warning: Format error at (83,3)-(83,4011): Illegal quoting
Processed 251 rows with 0 bad values and 1 format errors
Warning: Format error at (83,3)-(83,4011): Illegal quoting
Processed 251 rows with 0 bad values and 1 format errors
Warning: Format error at (83,3)-(83,4011): Illegal quoting
Processed 251 rows with 0 bad values and 1 format errors
Not adding a normalizer.
Making per-feature arrays
Changing data from row-wise to column-wise
Bad value at line 1 in column Label
Warning: Format error at (83,3)-(83,4011): Illegal quoting
Processed 251 rows with 1 bad values and 1 format errors
Processed 250 instances
Binning and forming Feature objects
Reserved memory for tree learner: 12771792 bytes
Starting to train ...
Not training a calibrator because it is not needed.
Bad value at line 1 in column Label
Processed 19 rows with 1 bad values and 0 format errors
PredictionModel quality metrics evaluation
Accuracy: 94.44%
Auc: 98.77%
F1Score: 94.74%
Sentiment Predictions
Sentiment: Please refrain from adding nonsense to Wikipedia. | Prediction: Positive
Sentiment: He is the best, and the article should say that. | Prediction: Negative
Sentiment: Hi There, Thanks for writing to us! Please can you explain bit more on your ask? | Prediction: Negative
Sentiment: Hi this is ridiculous | Prediction: Positive
Code:
Console.WriteLine($"Sentiment: {item.sentiment.SentimentText} | Prediction: {(item.prediction.Sentiment ? "Positive" : "Nagetive")}");

Let me know if i am doing any thing wrong?

I'm getting this with v0.7.0:
=============== Evaluating Model accuracy with Test data===============
Model quality metrics evaluation
--------------------------------
Accuracy: 94.44%
Auc: 98.77%
F1Score: 94.74%
=============== End of model evaluation ===============
=============== Prediction Test of loaded model with a multiple samples ===============
Sentiment: This is a very rude movie | Prediction: Toxic | Probability: 0.5297049
Sentiment: He is the best, and the article should say that. | Prediction: Toxic | Probability: 0.9918675
=============== End of predictions ===============

Changing "He is the best, and the article should say that." to "Very good" makes it not toxic. Seems the training data is not big enough.

I'd much prefer it if the sample was updated so that it would work as described. As it is, it gives the impression that something's wrong with ML or the user didn't follow the example through correctly.
Anyway, I wrote a quick gist that transforms the full Wikipedia toxicity dataset (see https://meta.wikimedia.org/wiki/Research:Detox/Data_Release) to the file format expected by the example and the numbers seem to make more sense as far as the Toxic detection is concerned. Hope it turns out to be useful to someone else.

Just for the record, the output with 0.8.0:
=============== Prediction Test of loaded model with a multiple samples ===============
Sentiment: This is a very rude movie | Prediction: Toxic | Probability: 0,5297049
Sentiment: Please refrain from adding nonsense to Wikipedia. | Prediction: Not Toxic | Probability: 0,1813062
Sentiment: He is the best, and the article should say that. | Prediction: Toxic | Probability: 0,9918675
=============== End of predictions ===============


Add "Schema link" for expanded Message's Key and Content

Describe the bug
Add "Schema link" for expanded Message's Key and Content
Set up
https://www.kafka-ui.provectus.io/
Steps to Reproduce

Steps to reproduce the behavior:

Navigate to Topics
Open the Topic
Expand the Message

Expected behavior
Should be available schema link for Key and Content
Screenshots

Good idea!
Since the UI already have the schema-version loaded for displaying the content, it should be easy to display some schema meta.
I would propose to not use the "Schema link" term, but instead use something like:
"Schema: subject version: 1"

Displaying the subject name will save many clicks.
Since the schema registry UI does not currently have a URL to the version, we would have to state the version in plain text.


Update every 30 sesonds

As discussed here: https://github.com/bramstroker/homeassistant-powercalc/discussions/878
It seems. that currently, even with scan_interval: 00:01:00 the power sensors are getting updated every 30 seconds. Checked the LUT ones and the fixed ones. Only the daily_fixed respect the default of 1800 seconds.
You can see this good in the energy_sensors.

I have found the issue. Essentially the power sensors were always updating every 30 seconds regardless of the scan_interval setting. It was even worse because when you set scan_interval was set to 1 minute it would result in duplicate state changes each minute.
I am also thinking about deprecating the scan_interval option to a more logical name.
For example:

update_frequency
refresh_interval
force_update_frequency

Or something like that. What do you think?

Yes probably. Closing this issue.


Add LinkCommonParams to initial LinkDefinition struct

This is meant to fix #1509

Still to fix the whole thing we need to catch and ignore the yaml type error.
I've also extended the tests to cover links with vars as well as with the new extended link format.
After #1475 is merged, we need to uncomment the checks for mac the addresses.

merged into #1475


function-quote-no-quoted-strings-inside auto fix can corrupt code

Auto fix for function-quote-no-quoted-strings-inside changed this code:
        @if $template {
            $template: $template quote($area-row);
        }
        @else {
            $template: quote($area-row);
        }

To this code:
        @if $template {
            $template: $area-row;
        } @else {
            $template: $area-row;
        }

which corrupts the code of course.
The actual use case here is building up values for grid-template-areas. So $template later contains "rowa" "rowb" "rowc"....

Sounds like a bug!


Fix#415

Hey Mike,
I don't know if you're aware of #415 : the OP describes the problem pretty well. To summarize, when the MinimumScaleMode#FIT was introduced, the logic to calculate the constrained X/Y scroll position needed to be adapted. Well, not exactly. In fact, in most cases there is no issue. But when a developer extends the TileView and overrides (as he's allowed to) the getScrollMinX and getScrollLimitX to return something different than 0, then the whole TileView is shifted at minimum scale.
Investigating on this, i found that it was not the only issue. Scrolling slowly at the boundaries of the TileView would make the scroll position change from eg -100 to 0 without any transition (which is a weird behavior). This happens when the scale is equal to mMinimumScaleY or mMinimumScaleX, depending on the shape of the tileset.
So i worked on the logic of getConstrainedScrollX and getConstrainedScrollY to work also when non zero return values are defined for getScrollMinX, getScrollLimitX, getScrollMinY, getScrollLimitY.
I came up with this implementation, which i tested and so did the author of #415.

+1 can you bump version, update readme and merge?  i'll pull and upload to jcenter asap

@peterLaurence i posted this in another PR a while back, ICYMI:

@peterLaurence ~are you following this? can you confirm all's well with the repo at this point?~ also, i'll be ready to get started on v3 in about 3 weeks, when summer term starts. first thing i'd like to do is the base component - a two-dimensional scroller. ZoomPanLayout will extend that as it's own component, for things like zoomable images or grids. My hope is that each component can exist separately on top of each other, until we finally get to pluggable things like paths, markers, hotspots, etc. anything in particular you'd like to achieve? we should probably move this out of issues, or at least out of this issue - how do you want to coordinate? wiki, or feature issues, or something else?

I don't know how i missed your message above.. sorry for that.
Ok for your strategy to decouple first. To coordinate, i find the "Projects" tab excellent. You define a new project, say "v3", then you can add columns e.g ("To Do", "In Progress", "Done"). Each columns can receive cards describing the task. Each card can be converted into an issue, and moved to other columns. This gives an overview of the project, while still using issues for details.
As for things i'd like to achieve : have a mode that loads tiles so that the user don't even see the loading. But something efficient (like don't just extend the viewport). I don't know if its feasible. We will see :)

Sounds great.  I created a project, will start to fill it in tonight.  I'll also publish the latest to bintray tonight.  Also, I like your idea - we might not be able to get it perfect, but I get we can come close.  I'll ping you later.  Thanks @peterLaurence

Ok great! I will look into that tomorrow when I get time.
The timing is perfect for me, I have finished some personal dev work so I have some spare time.

sounds great
also, 2.2.7 is on jcenter
thanks for the fix


toast font did not change

hi
i have two app and use Calligraphy on both
one of them is working
but another one did not change font on toast
i think there is a problem on style
how i can fix it ?

Last time I checked unless you are using a custom toast view you theres no way you could set xml for that view so it would never know to change it.
If you use a custom view and start your toast from your activity context, it should work.


openpower-pels: Create guard using libguard

Replace CreateWithEntityPath D-Bus method with guard library calls for creating guard entries, as CreateWithEntityPath is not an approved dbus method.
Tested and the guard record is created with the corresponding PEL id
before injecting the error
root@p10bmc:~# guard -l
No unresolved records to display

After injecting error, the guard is created using the PEL ID
root@p10bmc:~# guard -l
ID         | ERROR      | Type            | Path
0x00000001 | 0x5000592b | unrecoverable   | physical:sys-0/node-0/proc-0/eq-1/fc-0/core-0
root@p10bmc:~# peltool -l
{
    "0x5000592B": {
        "SRC":                  "BD13E510",
        "Message":              "Error Signature: 0x20DA0020 0x00000001 0x4D740407",
        "PLID":                 "0x5000592B",
        "CreatorID":            "BMC",
        "Subsystem":            "Processor Unit (CPU)",
        "Commit Time":          "10/17/2024 09:54:22",
        "Sev":                  "Unrecoverable Error",
        "CompID":               "bmc hw diags"
    }
}

Change-Id: I7531bce403206beaa119aea0a621e6b47d28ffd0

the upstream rev bump also hit something that probably needs a fix at https://gerrit.openbmc.org/c/openbmc/openbmc/+/75514

the upstream rev bump also hit something that probably needs a fix at https://gerrit.openbmc.org/c/openbmc/openbmc/+/75514

The upstream commit will get fixed by  https://gerrit.openbmc.org/c/openbmc/openbmc/+/75526
For the ghe, I have created a bump request to get the guard changes in this commit to be bumped. Once that is in, this should be successful. Thank you 0cf8f2bc5663893065c95f8aaf4b9d463085b08a.


Variable considered unused code although used in heredoc string returned by function

PHPMD version: 2.12.0
PHP Version: 8.1.8
Installation type: composer
Operating System / Distribution & Version: Alpine Linux v3.16.1

Current Behavior
When a variable is used in a heredoc string that is directly returned by a function PHPMD considers the variable unused code.
Expected Behavior
PHPMD detects that the variable is used in the heredoc string that gets returned.
Steps To Reproduce:
Example index.php:
<?php
function foobar() {
    $foo = 'foo';
    return <<<END
${foo}bar
END;
}
var_dump(foobar());

Result of running php vendor/bin/phpmd index.php text unusedcode:
index.php:3  UnusedLocalVariable  Avoid unused local variables such as '$foo'.

Checks before submitting

[x] Be sure that there isn't already an issue about this. See: Issues list
[x] Be sure that there isn't already a pull request about this. See: Pull requests
[x] I have added every step to reproduce the bug.
[x] If possible I added relevant code examples.
[x] This issue is about 1 bug and nothing more.
[x] The issue has a descriptive title. For example:  "JSON rendering failed on Windows for filenames with space".

Just for the record, this code will raise a deprecation notice in PHP 8.2:
Deprecated: Using ${var} in strings is deprecated, use {$var} instead

Using {$foo}bar instead of ${foo}bar will both help PHPMD to understand it and make your code PHP 8.2 ready.


wrong definition of dec_embed_input in test_decoding_layer for dlnd_language_translation.ipynb

original definition is:
dec_embed_input = tf.placeholder(tf.float32, [batch_size, 22, embedding_size])
it should be:
dec_embed_input = tf.placeholder(tf.int32, [batch_size, sequence_length])

finally it's proved that it's not a issue


How do i remove versioning from font files

Laravel Mix Version: 4.0.14
Node Version: v6.11.3
NPM Version: 3.10.10

Description:
I want to remove the versioning from the font files, because we have a client who can't see the glyphs and FontAwesome recommends to not verion the font files (See this issue: https://github.com/FortAwesome/Font-Awesome/issues/3286)

Mix file:
I only used the copyDirectory because it was suggested as a fix for the versioning but my font files still get versioned.
const mix = require('laravel-mix');

/*
 |--------------------------------------------------------------------------
 | Mix Asset Management
 |--------------------------------------------------------------------------
 |
 | Mix provides a clean, fluent API for defining some Webpack build steps
 | for your Laravel application. By default, we are compiling the Sass
 | file for the application as well as bundling up all the JS files.
 |
 */

mix.webpackConfig(webpack => {
    return {
        plugins: [
            new webpack.ProvidePlugin({
                $: 'jquery',
                jQuery: 'jquery',
                'window.jQuery': 'jquery',
            })
        ]
    };
});

mix.copyDirectory('resources/fonts/fontawesome-pro', 'public/fonts');
mix.copyDirectory('resources/images', 'public/images');

mix.js('resources/js/app.js', 'public/js');

mix.sass('resources/sass/app.scss', 'public/css');

I'm grateful for any pointers as i know this is not a real "issue". I tried hard to search but my google-fu failed me and maybe someone more experienced knows where i have to look.

I know this is very old, but did you ever get this fixed?

Try to use the "processCssUrls"" option. I have used it in a different project to prevent relative paths being changed to absolute paths and my result file doesnt seem to have versioning.
mix.options({ 	processCssUrls: false })

If you have absolute URLs you can turn off URL processing altogether with
.options({ // Do not process URLs anymore
  processCssUrls: false
});

But that's not my case, so I investigated futher and by looking into the source code of Laravel Mix I have found the code that manages fonts, which is in src/builder/webpack-rules.js, line 50, where the comment says // Add support for loading fonts.
Laravel Mix has a method called webpackConfig where you can specify rules to be merged with default ones. I have rewritten the code replacing Config.fileLoaderDirs.fonts with its value which is simply 'fonts' and removing the hash part. It worked and Google Lighthouse is not complaining anymore.
In my Mix file I have added:
.webpackConfig({
  // object will be merged with Laravel Mix config, so matching rules will be overwritten
  module: {
    rules: [{
      // overwriting file-loader rule for fonts in order to remove the hash (so we can pre-load without downloading it twice)
      test: /(\.(woff2?|ttf|eot|otf)$|font.*\.svg$)/,
      loaders: [{
        loader: 'file-loader',
        options: {
          name: (path) => {
            if (!/node_modules|bower_components/.test(path)) {
              return 'fonts/[name].[ext]';
            }
            const pathUpdated = path.replace(/\\/g, '/').replace(/((.*(node_modules|bower_components))|fonts|font|assets)\//g, '');
            return `fonts/vendor/${pathUpdated}`;
          },
        },
      }],
    }],
  },
})

to the webpack method.

My worked version
Mix.listen('configReady', function(config) {
    const rules = config.module.rules;
    for (let rule of rules) {
        if (rule.test.toString() == /(\.(woff2?|ttf|eot|otf)$|font.*\.svg$)/.toString()) {
            rule.options.name = (link) => {
                if (!/node_modules|bower_components/.test(link)) {
                    return path.join(mix.config.fileLoaderDirs.fonts, '/[name].[ext]');
                }
                const pathUpdated = link.replace(/\\/g, '/').replace(/((.*(node_modules|bower_components))|fonts|font|assets)\//g, '');
                return `fonts/vendor/${pathUpdated}`;
            }
            break;
        }
    }
});

i had the same problem. this fixed it:
 Mix.version([
        "public/fonts/myFont.woff"
    ]);

reding the docs actually helped.


Allow customizing request model serialization options

When using the library with Pydantic v2 models for API clients, there's no way to customize how request models are serialized before being sent. This is particularly problematic with optional fields that default to None - they are included in the request JSON even when they weren't explicitly set.
The serialization happens here: https://github.com/pmateusz/meatie/blob/7043da4612ba7fd6b9042463faf999cc8d47d208/src/meatie/internal/adapter/pydantic_v2.py#L37
For example, with this client code:
class MyClient(Client):
    @endpoint("/checkouts", private, method="POST")
    async def _create_checkout(self, request: Annotated[APIRequest, api_ref("body")]) -> APIResponse: ...

And this request model with Pydantic v2:
class APIRequest(BaseModel):
    model_config = ConfigDict(ser_json_exclude_unset=True)  # This is ignored
    data: CheckoutRequest

request = APIRequest(
    data=CheckoutRequest(
        type="checkouts",
        attributes=CheckoutRequestAttributes(
            custom_price=50000

        )
    )
)

The JSON sent still includes all the None fields despite the model config:
{
    "data": {
        "type": "checkouts",
        "attributes": {
            "custom_price": 50000,
            "product_options": null,
            "checkout_options": null,
            "checkout_data": null,
            "preview": null,
            "test_mode": null,
            "expires_at": null
        }
    }
}

Desired Behavior
We should be able to configure how request models are serialized, particularly the ability to exclude None values using something like Pydantic's exclude_unset=True or exclude_none=True.
The expected request JSON should only include set fields:
{
    "data": {
        "type": "checkouts",
        "attributes": {
            "custom_price": 50000
        }
    }
}

Possible Solution
A solution would be to add serialization options to the api_ref annotation:
@endpoint("/checkouts", private, method="POST")
async def _create_checkout(
    self,
    request: Annotated[APIRequest, api_ref("body", exclude_unset=True)]
) -> APIResponse: ...

Context
This is particularly important when working with APIs that have different behavior for explicitly set null values versus omitted fields. Many APIs treat "field": null differently from the field being absent in the request.
For example, the LemonSqueezy API uses the absence of fields to indicate default values should be used, while explicit null values might override defaults.
Thank you in any case for this truly amazing library, it is very enjoyable to work with.

FWIW, this was addressed in #70, and I added an example in https://github.com/pmateusz/meatie/issues/70#issuecomment-2424202508
Having said that, directly passing options would certainly be an improvement, as well.

The unwrap solution from #70 does not work in my case because it is not ran on body fields. To have it work i had to avoid typing my field as anything subclassing Pydantic's BaseModel and use fmt instead of unwrap (even tough it was inputted as a pydantic model). Avoiding BadeModel was needed because else it would try to dump it twice, from my fmt and internally. This is breaks type safety and uses parameters in unintended ways.

Specifically, I get this warning:
  Expected `APIRequest` but got `dict` with value `{'data': {'type': 'checko...ts', 'id': '593812'}}}}}` - serialized value may not be as expected
  return self.serializer.to_python(

Hi @Paillat-dev,
Thank you for the detailed feature request and including code examples!
I implemented a solution for the improvement you suggested in the PR: https://github.com/pmateusz/meatie/pull/79
The improvement allows using the fmt option to customize the serialization of the request body. The fmt option accepts a callback that given the request body, should produce either a str or bytes. The result is sent using aiohttp as data (https://docs.aiohttp.org/en/stable/client_reference.html#aiohttp.request).
This way we avoid the second serialization of a JSON payload into a JSON string.
I am proposing this solution because it is the most flexible I found. It gives the user full control over how the request body is serialized.

The improvement is available in v0.1.15

@pmateusz Thank you so much !
The fix is perfect, exactly what was needed, now it doesn't try to validate again my already formatted json, this is perfect.
Amazing project, thanks again for your work


Possible to change Gem payout?

As the title says, is there a way to change how many gems are rewarded to the player per duel? I get roughly 100 per game which is a little slow to me. I've been searching through the files for this but haven't found anything yet.

Found it. Sorry for the post. It was in Settings.json I skimmed past it by mistake


test_in_udp: Reduce execution time of message_length_limit test

Which issue(s) this PR fixes:
N/A
What this PR does / why we need it:
Although it's improved in #4676, it still takes 5 seconds on windows.
It can be reduced more by sending data that doesn't exceed the limit.
  test: message_length_limit:                           .: (1.104855)

Docs Changes:
N/A
Release Note:
N/A

A remaining failed test will be handled in another issue or PR:
1) Failure: test: multiple values are set to RUBYOPT(TestFluentdCommand::configured to run 2 workers):
  log doesn't match.


Only Ruby 2.2.2 installs, all other Ruby versions install fail looking for "-lyaml"

Very odd.
Brand new openSUSE 13.2.
Successfully used rbenv to install  ruby 2.2.2 and that mostly works except for an error installing rails (but that will be discussed elsewhere). So, have been attempting to install any other Ruby version.
Same error returned for others, the following is an example. I've also run with the verbose flag, but I don't see any more useful info, the error lines are exactly the same with no further detail. For some odd reason, I would expect that the "ld" binary should be looking for "yaml" but instead is looking for "-lyaml" and of course no file with that name exists.
TIA.
> rbenv install 2.1.0
Downloading yaml-0.1.6.tar.gz...
-> http://dqw8nmjcqpjn7.cloudfront.net/7da6971b4bd08a986dd2a61353bc422362bd0edcc67d7ebaac68c95f74182749
Installing yaml-0.1.6...
Installed yaml-0.1.6 to /home/user/.rbenv/versions/2.1.0

Downloading ruby-2.1.0.tar.gz...
-> http://dqw8nmjcqpjn7.cloudfront.net/3538ec1f6af96ed9deb04e0965274528162726cc9ba3625dcf23648df872d09d
Installing ruby-2.1.0...

BUILD FAILED (openSUSE project 13.2 using ruby-build 20150519-13-g702b410)

Inspect or clean up the working tree at /tmp/ruby-build.20150629205027.40381
Results logged to /tmp/ruby-build.20150629205027.40381.log

Last 10 log lines:
/usr/lib64/gcc/x86_64-suse-linux/4.8/../../../../x86_64-suse-linux/bin/ld: cannot find -lyaml
collect2: error: ld returned 1 exit status
Makefile:452: recipe for target '../../.ext/x86_64-linux/psych.so' failed
make[2]: *** [../../.ext/x86_64-linux/psych.so] Error 1
make[2]: Leaving directory '/tmp/ruby-build.20150629205027.40381/ruby-2.1.0/ext/psych'
exts.mk:190: recipe for target 'ext/psych/all' failed
make[1]: *** [ext/psych/all] Error 2
make[1]: Leaving directory '/tmp/ruby-build.20150629205027.40381/ruby-2.1.0'
uncommon.mk:180: recipe for target 'build-ext' failed
make: *** [build-ext] Error 2

Hi,
I went through the same issue...
And I sorted all out by re-installing all the necessaries packages :
zypper in ruby rubygem-bundler nodejs git    gcc make libopenssl-devel sqlite3-devel'

Can you try to install these packages, and re-try the rbenv ruby installation ?

It definitely sounds like you have to install libyaml on your system, @putztzu. Try "libyaml" or "yaml-devel"

If you still have build problems, please report at https://github.com/sstephenson/ruby-build/issues instead since rbenv itself is not responsible for installing Rubies, just switching them.


feat: add randomInt and randomNumber functions

[!TIP]
The owner of this PR can publish a preview release by commenting /publish in this PR. Afterwards, anyone can try it out by running pnpm add radashi@pr<PR_NUMBER>.

Summary

Rename random to randomInt
Add randomFloat, randomSecureFloat, and randomSecureInt

See the linked discussion for more details.
Related issue, if any:

https://github.com/orgs/radashi-org/discussions/78
For any code change,

[ ] Related documentation has been updated, if needed
[ ] Related tests have been added or updated, if needed
[ ] Related benchmarks have been added or updated, if needed

Does this PR introduce a breaking change?

Yes

Todo

[ ] Remove secure versions per @Minhir's advice
[ ] Keep random around for backwards compatibility.
[ ] Document randomFloat and randomInt (alias randomInt to random)

TODO

[ ] Revert randomNumber to randomFloat


spree::Admin::PaymentsController:Class (NoMethodError) on bundle exec install

I'm getting this error when trying to install the 2-3-stable branch running bundle exec rails g spree_store_credits:install
.rvm/gems/ruby-2.0.0-p247/bundler/gems/spree_store_credit_payment_method-aecee00bf815/app/controllers/spree/admin/payments_controller_decorator.rb:35:in `<top (required)>': private method `prepend' called for Spree::Admin::PaymentsController:Class (NoMethodError)

I see where the error is generated. I just don't know why its getting it.
Any help would be greatly appreciated.

This extension requires >= ruby 2.1.0 currently

Ahh. Thank you.


Added bubbleTintColor property

Enables users to set custom tint colors for images in bubbles.
Works with both easyButton = YES, and NO.
I updated the example to show how it works, but feel free to remove it if it shouldn't be there.

Hi Anton, thank you for the contribution! I will review the pull soon and merge.


Extend Action store to handle Page/Section/Component level actions

Currently we only execute actions declared at component level, matched by ID.
Do we need to extend this for sections/pages too?
Or do we just use button components to achieve the same thing?
Do we need "pre" actions as well as "post" actions?
I can't think of an example offhand, not needed for drop down population for example, that's part of the component itself, expecting a list passed in.

I'm not sure that this is actionable (no pun intended).
If a section has a "completed" button then we could apply the action on that, possibly need a SectionNavButton component, so that we don't have to make code changes when swapping sections around.  This would presumably validate all components in the section, then find the next section from journey.json and jump to the first question.
Page much the same, PageNavButtons, would have to be aligned to using the ProgressBar links when going back too, e.g. clear all data on page 3 if you go back to page 2, should that be a requirement.


Can't run tests if the sails server has not been started once before

When i'm tryin to execute tests for the first time (I open my IDE and run npm test), i have an error :
<testsuite name="Mocha Tests" tests="0" failures="2" errors="2" skipped="-2" timestamp="Mon, 11 May 2015 12:47:21 GMT" time="2.005">
<testcase classname="" name=""before all" hook" time="2"><failure><![CDATA[timeout of 2000ms exceeded. Ensure the done() callback is being called in this test.
Error: timeout of 2000ms exceeded. Ensure the done() callback is being called in this test.]]></failure></testcase>
<testcase classname="" name=""after all" hook" time="0"><failure><![CDATA[Cannot call method 'lower' of undefined
TypeError: Cannot call method 'lower' of undefined
    at Context.<anonymous> (test/controllers/UserController.spec.js:78:11)]]></failure></testcase>
</testsuite>

For each test written, a sails server is lifted with bootstrap.spec.jsfile and killed with done callback.
 But i have to start the server one time before. Why?
And if I set a remote database, i doesn't work. It only work in local with a local database
In local, it's not a problem but on Travis-CI, I can't start the server with sails lift and kill this process with CTRL + C and after execute tests then the build failed with the error above.
Sails : 0.11

I think you need to post your code. This isn't an issue I've encountered before.

I rename the issue, because the real problem is linked to the remote database connection. If I execute tests in a local MySQL server, even for the first time. It works fine but not on a remote server with the same version of MySQL (5.5.42).
The error message is the same.
Here is concerned files of my project :
connections.js
module.exports.connections = {
    developmentMysqlDatabase: {
        adapter:  'sails-mysql',
        host:     'localhost',
        port:     8889,
        user:     'user',
        password: 'password',
        database: 'database'
    },

    testingMysqlDatabase: {
        adapter:  'sails-mysql',
        host:     'remoteserver.com',
        port:     3306,
        user:     'user',
        password: 'password',
        database: 'database'
    }
};

Bootstrap.spec.js
var Sails = require('sails');
var sails;

before(function(done) {
    Sails.lift({}, function(err, server) {
        sails = server;
        if (err) {
            return done(err);
        }
        done(err, sails);
    });
});

after(function(done) {
    sails.lower(done);
});

I noticed that when I start the server with a remote database connection. Tables are not created into. I think these 2 problems are linked.

The problem was caused by short timeouts in mocha configuration (2000ms), with higher timeouts, it works fine.


[zabbix-community/zabbix] Fresh installation with 6.1.1 no DB schema

I'm struggling with version 6.1.1 to create a fresh installation of zabbix. Not sure if this is fixed with https://github.com/zabbix-community/helm-zabbix/issues/122 as it did not work for me
The zabbix-server "init-wait-for-database-schema" pod fails with
2024-12-03 14:04:23.380Z init-wait-for-database-schema ** DB Version not found yet, assuming DB schema is not yet created
2024-12-03 14:04:23.380Z init-wait-for-database-schema *** Waiting 5 seconds...

I do see the postgres server pod and can connect to it
zabbix@zabbix-zabbix-server-56889bb84f-kd5rl:/var/lib/zabbix$ psql -h zabbix-postgresql
Password for user zabbix:
psql (16.4 (Ubuntu 16.4-0ubuntu<IP_ADDRESS>), server 16.5 (Debian 16.5-1.pgdg120+1))
Type "help" for help.

zabbix=#
zabbix=# \dt
Did not find any relations.
zabbix=# \dn
      List of schemas
  Name  |       Owner
--------+-------------------
 public | pg_database_owner
(1 row)

zabbix=# \dt
Did not find any relations.

I'm using ArgoCD to deploy the helm chart with minimal configuration
values: |
        zabbixServerHA:
          enabled: false

Did I miss something?

Hi! I am rather sure it's not directly related to how the Helm Chart works and how it initializes the database and the schema, as this has been tested quite extensively. Could you give me a bit more context where this part exactly comes from?
values: |
        zabbixServerHA:
          enabled: false

Is this part of your ArgoCD "application"?
My suspect is that argoCD somehow ignores that part of values which should disable native Zabbix server HA, falling back to the default which is enabled: true and somehow struggles with the post-install job which then would be necessary to initialize the database. I have seen similar situations of ArgoCD ignoring jobs that are marked as post-install jobs when using a Helm Chart as a Subchart to another one.
By the way, the setting to disable native server HA is zabbixServer.zabbixServerHAenabled: false, so in case your notation as you did is correct, it should rather be
values: |
  zabbixServer:
    zabbixServerHA:
      enabled: false

Thanks a lot, I just missed the zabbixServer: part. Didn't look properly!
Now everything works


AppID mismatch

Hey there, I've been experiencing a problem with accessing the correct environment through the Amplify CLI, this is because of an "AppID mismatch" when trying to access or create anything. I have tried to reconfigure the CLI, invited new accounts through the Admin UI panel, but for some reason the CLI is connecting to an app ID that doesn't exist. In turn, it will not allow me to sign in to the account it's attempting to connect to, so I can't stop it.
I've tried accessing the Amplify UI through Chrome, Edge and Brave, all up-to-date, all on a Windows 10 Surface Pro. I believe I need to change the directory in the CLI to the app ID I'm working on, but I have no idea how to fix this. Can someone help please?
I've attached a screenshot as well.

Thanks for any help.
Kindest regards,
Adam

Tried this way so it is hooked up to the wrong ID

If anyone else does run into an issue like this, I just wiped everything in the amplify folder and my cloud, ran amplify init in the CLI and it works


.NET 6 Isolated: An error occurred while loading required library hostpolicy.dll

I am trying out Azure Functions 4.0 with .NET 6 Isolated (Out-of-process) and I am getting the following error.
Failed to load the dll from [...\AppData\Local\AzureFunctionsTools\Releases\4.10.1\cli_x64\hostpolicy.dll], HRESULT: 0x800700C1
An error occurred while loading required library hostpolicy.dll from [...\AppData\Local\AzureFunctionsTools\Releases\4.10.1\cli_x64\]

I am using Visual Studio 2022 / Version 17.1.0 Preview 1.0.
I have tried with .NET 5 Isolated (Out-of-process), it's a similar error.
However Azure Functions 4.0 with .NET 6, In-Process works just fine.

I was experiencing the same problem that you're describing, but managed to solve it by editing my system environment variables. My environment had a PATH set up with C:\Program Files (x86)\dotnet\ appearing before C:\Program Files\dotnet.  When I moved the 64-bit dotnet folder up so it appeared before the 32-bit version, then debugging started working again.
I arrived at this solution after attempting to create a new function app project using dotnet new on the command prompt.  I found that the dotnet.exe that the terminal was picking up was not one associated with the SDK.  Once I adjusted the path I was able to run dotnet new and also able to start debugging from Visual Studio.

@NimbleKelly, Thanks for the input. I just tried,
dotnet new

It doesn't even work.
Checked environment variables,

And yes, it's out of order. Moved the 64-bit dotnet before 32-bit version, and all is good now.
Thanks again.

We've seen this in a few other places as well -- I think something with the ordering of the .NET SDK installation put things into a bad state. I'm glad you have figured this out and hopefully it helps others!

For anyone else coming here -- this is noted in the .NET 6 known issues page now: https://github.com/dotnet/core/blob/main/release-notes/6.0/known-issues.md#2-upgrade-of-visual-studio-or-net-sdk-from-earlier-builds-can-result-in-a-bad-path-configuration-on-windows


feature idea: locate

First, thanks for the useful extension!
One thing that I like about the Alfred launcher is the ability not just to open a file directly (with ') but to open its location in the file explorer (with find). In the ulauncher-file-search extension, this could be done with a keyword such as fl. (Even better would be if ulauncher provided a secondary action for file results–eg, alt+1 to open, alt+shift+1 to locate–but I don't think the API is there.)
Use cases:

I'm looking for a sibling file whose name I don't remember
I want to perform a file operation on a file or its siblings
I want to navigate somewhere starting from a file's location

+1 for locate !, fd is too slow in ulauncher


Training on a local GPU

Hello! Thank you for your work on this project.
I'm trying to train the ant environment locally. But it's very slow as you can see, so I'm assuming it's on CPU (GPU is rtx 2070). Is it necessary to specify to use GPU like for the colab? Or something else? Grateful for advice.
(brax)  karl@omen:~$ learn ant.py
I0314 23:08:48.799718<PHONE_NUMBER>16768 metrics.py:41] Hyperparameters: {'log_frequency': 10, 'num_envs': 4, 'total_env_steps': 50000000}
I0314 23:08:48.804218<PHONE_NUMBER>16768 xla_bridge.py:247] Unable to initialize backend 'tpu_driver': NOT_FOUND: Unable to find driver in registry given worker:
I0314 23:08:48.904914<PHONE_NUMBER>16768 xla_bridge.py:247] Unable to initialize backend 'tpu': INVALID_ARGUMENT: TpuPlatform is not available.
I0314 23:08:48.905025<PHONE_NUMBER>16768 ppo.py:219] Device count: 1, process count: 1 (id 0), local device count: 1, devices to be used count: 1
I0314 23:09:01.497590<PHONE_NUMBER>16768 ppo.py:427] starting iteration 0 12.69543743133545
I0314 23:09:07.033340<PHONE_NUMBER>16768 ppo.py:459] {'eval/episode_reward': DeviceArray(-84.00571, dtype=float32), 'eval/episode_reward_contact_cost': DeviceArray(1.6300672, dtype=float32), 'eval/episode_reward_ctrl_cost': DeviceArray(1052.7314, dtype=float32), 'eval/episode_reward_forward': DeviceArray(-3.2535172, dtype=float32), 'eval/episode_reward_survive': DeviceArray(973.6094, dtype=float32), 'eval/completed_episodes': DeviceArray(128., dtype=float32), 'eval/avg_episode_length': DeviceArray(973.6094, dtype=float32), 'speed/sps': 0, 'speed/eval_sps': 23246.63754177282, 'speed/training_walltime': 0, 'speed/eval_walltime': 5.506169557571411, 'speed/timestamp': 0}
I0314 23:09:07.040651<PHONE_NUMBER>16768 metrics.py:50] [0] eval/avg_episode_length=973.609375, eval/completed_episodes=128.0, eval/episode_reward=-84.00570678710938, eval/episode_reward_contact_cost=1.630067229270935, eval/episode_reward_ctrl_cost=1052.7314453125, eval/episode_reward_forward=-3.2535171508789062, eval/episode_reward_survive=973.609375, speed/eval_sps=23246.637542, speed/eval_walltime=5.506170, speed/sps=0, speed/timestamp=0, speed/training_walltime=0
I0314 23:44:17.747002<PHONE_NUMBER>16768 ppo.py:427] starting iteration 1 2128.9448471069336
I0314 23:44:19.419119<PHONE_NUMBER>16768 ppo.py:459] {'eval/episode_reward': DeviceArray(-1885.5591, dtype=float32), 'eval/episode_reward_contact_cost': DeviceArray(0.771517, dtype=float32), 'eval/episode_reward_ctrl_cost': DeviceArray(2502.2344, dtype=float32), 'eval/episode_reward_forward': DeviceArray(-17.747227, dtype=float32), 'eval/episode_reward_survive': DeviceArray(635.1933, dtype=float32), 'losses/entropy_loss': DeviceArray(0.30761924, dtype=float32), 'losses/policy_loss': DeviceArray(7.857656, dtype=float32), 'losses/total_loss': DeviceArray(5405., dtype=float32), 'losses/v_loss': DeviceArray(5396.834, dtype=float32), 'eval/completed_episodes': DeviceArray(150., dtype=float32), 'eval/avg_episode_length': DeviceArray(635.04663, dtype=float32), 'speed/sps': DeviceArray(2368.8562, dtype=float32), 'speed/eval_sps': 76881.63453282246, 'speed/training_walltime': 2110.698221921921, 'speed/eval_walltime': 7.171062469482422, 'speed/timestamp': 2110.698221921921}
I0314 23:44:19.421806<PHONE_NUMBER>16768 metrics.py:50] [4999920] eval/avg_episode_length=635.046630859375, eval/completed_episodes=150.0, eval/episode_reward=-1885.55908203125, eval/episode_reward_contact_cost=0.7715169787406921, eval/episode_reward_ctrl_cost=2502.234375, eval/episode_reward_forward=-17.74722671508789, eval/episode_reward_survive=635.1932983398438, losses/entropy_loss=0.30761924386024475, losses/policy_loss=7.857656002044678, losses/total_loss=5405.0, losses/v_loss=5396.833984375, speed/eval_sps=76881.634533, speed/eval_walltime=7.171062, speed/sps=2368.856201171875, speed/timestamp=2110.698222, speed/training_walltime=2110.698222

Try increasing the num_envs from 4 to 2048 or so (I use 2048 on a 2080 Ti). I was as confused as you when I first tried it. Not sure why the default is only 4 envs.

It worked. Now it's as fast as advertised. Thank you for your help.


Fix issues with new lines not rendering, when multiline blocks are used

Bug/issue #, if applicable: 103087953
Summary
Fixes an issue where empty new lines are not rendered properly.

Dependencies
NA
Testing
codesample.json.zip
Using the example JSON above, try to copy the text into your editor.

Assert there is a single new line when you paste
Assert this works across browsers
Assert it does not regress

Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

[x] Added tests
[x] Ran npm test, and it succeeded
[x] Updated documentation if necessary

@swift-ci test


ftplugin(gdb): add Matchit support

thanks. Can you please add a line like this one please: https://github.com/vim/vim/blob/9a91d2b72c20f213bbf77f27b7edd01e0e43d5e0/runtime/syntax/go.vim#L9

Fixed.


Allow comma-separated ids

cryptocurrency/info supports an array of ids which is currently blocked. Can you please allow it? It saves so many api credits

Okay, I'll add it in the coming days. Previously this did not work with coinmarketcap api requests.


search by drop box

for the moment containing only 'title'. -> change to text or add fields.

AND:
reset -> clear


Consume config files from secrets for metadata agent

Depends-On: openstack-k8s-operators/dataplane-operator/pull/393

This PR includes the patch from https://github.com/openstack-k8s-operators/edpm-ansible/pull/324 that makes the container load config files as a config-dir. We should probably first land the PR I linked to, then rebase and merge this PR here.

Addressed comments:

removed unused changed_when, check_mode keywords;
use stat module to verify config file presence.

podified-multinode-edpm-deployment-crc job's failure seems like can be related to that patch as it failed in the "osp.edpm.edpm_ovn : discover secrets in /var/lib/openstack/configs/ovn" step. See: https://logserver.rdoproject.org/25/325/e5acb98c9c8231faf135d1916e9d76629adc8945/github-check/podified-multinode-edpm-deployment-crc/493951d/controller/ci-framework-data/artifacts/must-gather.local.7092492607379349338/quay-io-openstack-k8s-operators-openstack-must-gather-sha256-070a98749455b097c66a1a25ff827a3c0ee7385f1f0c8bf54f5b15438501f2bd/namespaces/openstack/pods/dataplane-deployment-ovn-openstack-edpm-xgrg4/logs/dataplane-deployment-ovn-openstack-edpm-xgrg4.log

podified-multinode-edpm-deployment-crc job's failure seems like can be related to that patch as it failed in the "osp.edpm.edpm_ovn : discover secrets in /var/lib/openstack/configs/ovn" step. See: https://logserver.rdoproject.org/25/325/e5acb98c9c8231faf135d1916e9d76629adc8945/github-check/podified-multinode-edpm-deployment-crc/493951d/controller/ci-framework-data/artifacts/must-gather.local.7092492607379349338/quay-io-openstack-k8s-operators-openstack-must-gather-sha256-070a98749455b097c66a1a25ff827a3c0ee7385f1f0c8bf54f5b15438501f2bd/namespaces/openstack/pods/dataplane-deployment-ovn-openstack-edpm-xgrg4/logs/dataplane-deployment-ovn-openstack-edpm-xgrg4.log

yeap become: false should clear this

Changes:

become: false for the task delegated to localhost
load 01-rootwrap.conf instead of /etc/neutron/rootwrap.conf

recheck

recheck

/approve

/lgtm


Add fpgabist to get checked for coding style

First commit. Adding this to check fpgabist for coding style.

@buie1 I also have some fixes for DMA code under review at: https://github.com/OPAE/opae-sdk/pull/228/commits/05081abace3cd593ffccd5dac27025f40fa037d3
We should make sure these get ported over to the AFU repo.

@southerngs Thanks for pointing this out Gabriel. Once you merge you changes I can handle the merge conflicts in this branch.  I wont want to merge this until I get the bist_afu app in main as well so I can knock out all the CI issues within the fpgabist too in one swoop.

@r-rojo @OPAE/ci
Finally completed all the fixes to get the the fpgabist tool passing the coding style tests. Ready to get this merged so future changes can all conform to this standard.

@OPAE/ci @nakulkorde @rpSTTD
Can we get this approved by you guys to get this merged in.


Allow Bypass of AWS Alias Check

Users want the ability to skip the AWS Alias check for a number of good reasons. This implements that request but in a way that keeps the safeguards in place.
Upstream References

https://github.com/rebuy-de/aws-nuke/issues/971
https://github.com/rebuy-de/aws-nuke/pull/983
https://github.com/rebuy-de/aws-nuke/issues/997

:tada: This issue has been resolved in version 3.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Rendering bugs with high physics fps

With physics fps set to ~200 and 60 fps lock some rendering problems appear. For example, screen flashing.
Tested on Linux, latest Godot build from github.

@zerornull  : what do you call "screen flashing" ?
Also, if your CPU is not strong enough, 200fps for the physics might be too much computations per seconds, and might lead to strange bottleneck issue that will affect the rendering frame rate.
Could you provide a minimal scene project showing this issue and tell us what is your CPU and GPU ?

@zerornull You shouldn't have problems like that if your objects are moving at sane speeds compared to their size. In most commercial games, physics run at 30-60 fps. You only get issues like that if you have tiny objects moving very quickly, in which case you should use sweeping collision detection.

The situation is slightly better with use_vsync instead of target_fps parameter, but sprites flickering still exists.

@abitofbyte Can you share GPU specs with us? It sounds like it might be the result of a poor GPU or driver. Also, can you make an example project exhibiting the issue so we can confirm and/or debug it?

@bojidar-bg Here are my GPU specs: http://ark.intel.com/products/75027/Intel-Core-i5-4200H-Processor-3M-Cache-up-to-3_40-GHz . I've tested everything on two different devices, the results were same. Also issue appears only in fullscreen mode. Here's an example project: https://www.dropbox.com/s/yg43fdrpamo5njp/Flickering Test.zip?dl=0

Ok, closing this one then..


Policy: IPCheck validate that port is not used.

This commit fix the issue when port is added into the forwarder-for
header and IPCheck will not match correctly.
Fix THREESCALE-5258
Signed-off-by: Eloy Coto<EMAIL_ADDRESS>

Looks great to me!


Allow additional fields to be transmitted to clients in config

Perhaps federation_name belongs in a additional JSON field like this. Perhaps not all federations want a name?


Always -y on drush commands causes unexpected behavior when using drush in container

What happened (or feature request):
We added the default "-y" to drush via a global drush settings file in the nginx container. The primary reason was that otherwise ddev exec <anything>  can hang if drush asks for input, and we didn't figure out a way around that.
However, there are drush commands that you definitely want to see the output from before you do anything. For example, drush upc, drush sql-drop, etc. You absolutely want to see what's about to happen before it happens. Most of these (like drush upc) would normally be executed interactively ssh-ed into the container rather than via drush exec.
IMO we should either:

Figure out how to handle interactive commands with drush exec or
Make the -y happen only when drush is being run via ddev exec

What you expected to happen:
How to reproduce this:
Version: Please include the output of ddev version, docker version and the project's .ddev/config.yaml.
Anything else do we need to know:
Related source links or issues:

Would this be a simple matter of including a regex when using ddev exec to match for drush and confirm a -y is present and (if not) add it?

Another super-simple approach (suggested I think by @beeradb ) would be for ddev exec to set an environment variable and for us to set the -y when that's present. This has bit me a few times, even in the demo today - I expect to get to confirm a drush command (but accept that I can't inside exec).

At a surface level, I'm onboard with an environment variable because that exposes to the end-user that this is occurring. My only caveat to this is that a user disabling this would experience some issues with drush not working if the command being run needed an interactive prompt. This could also be problematic if people stuff drush commands in their pre/post hooks and then try to share this config among team members but they have discrepancies on their environment variables.
The regex example to append a -y outside the container might be the best way to normalize this for consistency as well as prevent a user from getting stuck with an interactive prompt. We'd still probably want to warn or send a notice to the end-user telling them that we're doing this in case.


[rtextures] ImageDrawTextEx is not scaling down resulting image

[x] I tested it on latest raylib version from master branch
[x] I checked there is no similar issue already reported
[x] My code has no errors or misuse of raylib

Issue description
Glyphs are not scaled down when rendering text using ImageDrawTextEx
Environment
OS: Arch Linux
OpenGL: 4.6 (Core Profile) Mesa 22.1.7
GPU: Intel(R) HD Graphics 4600
Issue Screenshot

Gray area has the same size as returned by MeasureTextEx on same font with same size and spacing
Code Example
#include <raylib.h>

int main(void) {
  InitWindow(320, 240, "LoadFont");
  SetTargetFPS(15);

  // Changing it to LoadFontEx("./unscii-16-full.ttf", 16, 0, 256) fixes the issue for that font size only.
  Font font = LoadFont("./unscii-16-full.ttf");
  Image img = GenImageColor(320, 240, BLACK);
  Vector2 box = MeasureTextEx(font, "Hello, world!", 16, 0);
  ImageDrawRectangle(&img, 4, 4, box.x, box.y, GRAY);
  ImageDrawTextEx(&img, font, "Hello, world!", (Vector2){ 4, 4 }, 16, 0, WHITE);
  Texture2D tex = LoadTextureFromImage(img);
  UnloadImage(img);

  while (!WindowShouldClose()) {
    BeginDrawing();
    DrawTexture(tex, 0, 0, WHITE);
    EndDrawing();
  }

  UnloadFont(font);
}

Font used: unscii-16-full.ttf (homepage)
Details
Problem lies in the ImageTextEx function (it uses base font size instead of the one we specified):
https://github.com/raysan5/raylib/blob/11fd883ee443d3ea478e750d0116611d72d5708d/src/rtextures.c#L1231
Also, we're scaling font only up, but not down:
https://github.com/raysan5/raylib/blob/11fd883ee443d3ea478e750d0116611d72d5708d/src/rtextures.c#L1270-L1279
I think it'd be better to change condition on L1270 to fontSize != imSize.y
Even better, relying on image height to scale font up/down is not a good idea 'cause we have multiline support:
https://github.com/raysan5/raylib/blob/11fd883ee443d3ea478e750d0116611d72d5708d/src/rtextures.c#L1247-L1253
I think what we can do is count number of lines, multiply it by the font size and scale down/up in case it doesn't match.

Scaling when height returned by MeasureTextEx differs from ours works too. Gonna open a PR now


Update downloading of minishift binary to handle tar.gz subdirectory change

With commit https://github.com/minishift/minishift/commit/868b6759329ceb1baa0613e29835b5d69d6aaee7 , all content will be under a subdirectory.
Update Makefie post 1.7.0 release of minishift.

Resolved via https://github.com/minishift/minishift-b2d-iso/pull/72


Labelled section for Grasshopper

Make a component for labelled sections in Grasshopper

Continue from the C# classes from #50

You can assign this to me if you like :) I could do this for the webinar

Wouldn't it be cool to also read the results in the sections to grasshopper? Maybe you could have a look on implementing Labelled sections, Internal forces and Labelled sections, Resultants as well?

That would be great, could do that as a next step. Maybe depending on how much we want to work with results reading before we get the new results reader from Budapest.

@johannariad Would you like me to have a look at this?
I am kind of fresh about the topic and I might be able to help you

Please @Marco-Pellegrino if you have time that would be great. I haven't had a chance to look at it yet but I think it would be useful for lots of things 🙏 Perhaps "result points" should also be in the component (or have its own), it is kind of a similar topic


Mask of 19

If you use a mask of 19, it will show this in the input:
1_
If you enter a number such as 8:
18
The model will be updated to 8
If you enter the number 1:
11
The model never gets updated, although it should return a 1. You can see this on the demo page by using the mask of 19 and trying to enter a 1 or a 2. Also it seems that any letter/number you use with the number followed by it doesn't work. So 19 won't work for a mask, but 91 will. Same thing with aA not working, also Aa seems to glitch out and always puts in aa. If I should be using a different masking solution please let me know.
Other thing of note, if you use the keyboard to delete back and replace the last character with a 1 it won't work. BUT if you highlight the last character and change it to a 1 it will work as expected and my model will be 1.
I'm using chrome.

Thanks for pointing out this issue. It appears to be a slightly complicated issue dealing with some core functionality. Hopefully, I will be able to get a fix in place in the next day or two.


Revise differences in Transformer.functionToForm implementations

Revise differences in Transformer.functionToForm implementations:

(old implementation) cz.cvut.spipes.transform.TransformerImpl.functionToForm in s-pipes-forms module in cvut-kbss/s-pipes
(new implementation) og_spipes.service.FormService.OwnTransformer.functionToForm in cvut-kbss/s-pipes-editor

1. Change which statement objects of the function are transformed to sub questions.

old implementation selects objects as follows:

?function spin:constraint ?constraint .
?constraint spl:predicate ?object .

new implementation selects objects as follows:

?function sp:varName ?object .

Should we consider both selection patterns or the old one is obsolete?

We use the spin:constraint selection pattern in at least one script, see script aircraft-maintenance-planning-system/aircraft-maintenance-planning-model/data/data-lib.ttl

2. Changes in sub query instance initialization.

old implementation creates and initializes question with:

Question q = createQuestion(st.getObject().asResource());
...
private Question createQuestion(Resource resource) {
        Question q = new Question();
        initializeQuestionUri(q);
        q.setLabel(resource.getURI());
        ... // extracting description from rdfs:label and rdfs:commnt statements.
        q.setDescription(...)
        return q;
}

This results

new implementation creates and initializes question with this code :

Question q = new Question();
initializeQuestionUri(q);
q.setLabel(st.getObject().toString());

In both implementations st.getObject is the object as defined in comment 1. Change which statement objects of the function are transformed to sub questions.
Merge Action
Use old implementation.
Considerations:

old and new implementation uses a different method to set question label. The new implementation does not assume the object node is a resource.

Is it possible that the selected object is a literal?

old implementation sets description of generated question. The new implementation does not.

ad 1)
This is definitely valid way to define function parameter:
?function spin:constraint ?constraint .
?constraint spl:predicate ?object .

This is not a valid way to define the parameter of a function:
?function sp:varName ?object .

To me it is important to keep old way of doing this. In case you keep new way i do not care, but in the new way you should assume that the ?object is literal.

@blcham
More context regarding 1)
[2] defines two type of questions generated from triples relevant to this discussion, i.e. StatementQuestion and SchemaQuestion. The SchemaQuestions are created only when there is a defined parameter for the function/template for which there is no statement.
Here is an example with a module. I could not find example for a function but it should be the same for functions. Consider the module kbss-module:rdf4j-update defined with two parameters p-rdf4j-server-url and
p-rdf4j-repository-name. The following example shows statements of a module configuration of type kbss-module:rdf4j-update. The configuration specifies a value for the parameter p-rdf4j-server-url but not for p-rdf4j-repository-name.
:remove-old-statistics
  a kbss-module:rdf4j-update ;
  sm:next :mark-relevant-questions ;
  km-rdf4j:p-rdf4j-server-url "http://localhost:7200/repositories/some-repo-id" ;
  # p-rdf4j-repository-name  - not specified

According to [2] there should be a StatementQuestion generated for the p-rdf4j-server-url parameter and a SchemaQuestions generated for the km-rdf4j:p-rdf4j-repository-name parameter.
Additionally, according to [1] it should be possible to specify any spin expression to a value of a parameter, that includes constant, variable and function call. Here is an example snippet from [3] where the pipeline uses variables as values for the parameters of kbss-module:rdf4j-update module:
:bind-rdf4j-server-url
  a sml:BindWithConstant ;
  sm:next :bind-repository-name ;
  sm:outputVariable "rdf4jServerUrl" ;
  sml:value "http://localhost:8080/rdf4j-server" ;
.

:bind-repository-name
  a sml:BindWithConstant ;
  sm:next :remove-old-statistics ;
  sm:outputVariable "rdf4jRepositoryName" ;
  sml:value "record-manager-app" ;
.

:remove-old-statistics
  a kbss-module:rdf4j-update ;
  sm:next :mark-relevant-questions ;
  # binds variable expression to the 'p-rdf4j-server-url' parameter
  km-rdf4j:p-rdf4j-server-url [ sp:varName "rdf4jServerUrl" ;] ;
  # binds variable expression to the 'p-rdf4j-repository-name' parameter
  km-rdf4j:p-rdf4j-repository-name [ sp:varName "rdf4jRepositoryName" ; ] ;
...

Conclusion
Both selection patterns should be considered in the merged implementation of Transformer.functionToForm:

old implementation selects objects - for the generation of SchemaQuestions

?function spin:constraint ?constraint .
?constraint spl:predicate ?object .

new implementation selects objects - for the generation of StatementQuestions with a variable as the answer

?function sp:varName ?object .

Questions
Neither implementations will generate StatementQuestion for parameters with constant and function call expression values.

Should we implement generation of StatementQuestion for parameters with constant expression value ?
Should we implement generation of StatementQuestion for parameters with function call expression value ?

[1] - spin documentation
[2] - Figure 26: Module configuration form generation in 2018-Doroshenko-Yan-thesis.pdf
[3] - statistics-extraction.sms.ttl.gz

The script2form schema from [2]:

I still do not understand what following means:
?function sp:varName ?object .

Why do you have ?function  variable?


Warn on missing builder

We are going to try to create a builder if one is missing, so let's warn rather than report it as an error.

Fixes https://github.com/basecamp/kamal/issues/503


Fishshell autocompletion

Is it possible to add auto-completion for fishshell?

I tried to enable argcomplete in fish following https://pypi.org/project/argcomplete/, but the actual functionality is half-broken, as CLI is trying to execute the command first, and results in an error. I will look into it.

@jiasli support for fish was apparently implemented in argcomplete v1.10.0, but I am not sure how extensively it has been tested. If there are still problems with it, it might be good to re-open an issue on the subject on their github.

@jemag, I understand your concern, but currently we are working on other high priority tasks and fish is not officially supported by Azure CLI. Please consider using Bash instead. Thanks for your understanding.

Please consider reopening the issue.

@jiasli Any updates on the issue? Pretty much everything (apart from azure) has fish completion:

kubectl completion fish
helm completion fish
kind completion fish
...

az actually supported fish completion since 2019, but it's never been documented. az uses argcomplete internally which in turn has supported fish completion since 2019.  az simply inherited that feature. See https://github.com/kislyuk/argcomplete/pull/260 for details
I've implemented the completion for fish in https://github.com/fish-shell/fish-shell/pull/8141. It's available in fish 3.4.0 or greater

Creating ~/.config/fish/completions/az.fish should work.
function __fish_az_complete
    set -x _ARGCOMPLETE 1
    set -x _ARGCOMPLETE_DFS \t
    set -x _ARGCOMPLETE_IFS \n
    set -x _ARGCOMPLETE_SUPPRESS_SPACE 1
    set -x _ARGCOMPLETE_SHELL fish
    set -x COMP_LINE (commandline -p)
    set -x COMP_POINT (string length (commandline -cp))
    set -x COMP_TYPE
    if set -q _ARC_DEBUG
        az 8>&1 9>&2 1>&9 2>&1
    else
        az 8>&1 9>&2 1>/dev/null 2>&1
    end
end
complete --command az -f -a '(__fish_az_complete)'

This is functionally identical to my implementation in upstream fish: https://github.com/fish-shell/fish-shell/blob/master/share/completions/az.fish and https://github.com/fish-shell/fish-shell/blob/master/share/functions/__fish_argcomplete_complete.fish (also see my previous comment)
Just update your fish and you're good to go, no need to reimplement the wheel

These scripts should be identical as I use register-python-argcomplete --shell fish az to generate it.
I post it here to provide a solution for older version fish. If there is no further question, this issue could be closed later.


Timeout message not get redeliver in TopicsConsumer when use message listener

Expected behavior
Timeout message get redeliver in TopicsConsumer when use message listener
Actual behavior
Timeout message not get redeliver in TopicsConsumer when use message listener
Steps to reproduce
Here is an simple UT
   @Test(timeOut = 30000)
    public void testMultiTopicsMessageListener() throws Exception {
        String key = "MultiTopicsMessageListenerTest";
        final String subscriptionName = "my-ex-subscription-" + key;
        final String messagePredicate = "my-message-" + key + "-";
        final int totalMessages = 6;

        // set latch larger than totalMessages, so timeout message get resend
        CountDownLatch latch = new CountDownLatch(totalMessages * 3);

        final String topicName1 = "persistent://prop/use/ns-abc/topic-1-" + key;
        List<String> topicNames = Lists.newArrayList(topicName1);

        admin.tenants().createTenant("prop", new TenantInfo());
        admin.topics().createPartitionedTopic(topicName1, 2);

        // 1. producer connect
        Producer<byte[]> producer1 = pulsarClient.newProducer().topic(topicName1)
            .enableBatching(false)
            .messageRoutingMode(MessageRoutingMode.SinglePartition)
            .create();

        // 2. Create consumer, set not ack in message listener, so time-out message will resend
        Consumer<byte[]> consumer = pulsarClient.newConsumer()
            .topics(topicNames)
            .subscriptionName(subscriptionName)
            .subscriptionType(SubscriptionType.Shared)
            .ackTimeout(1000, TimeUnit.MILLISECONDS)
            .receiverQueueSize(4)
            .messageListener((c1, msg) -> {
                Assert.assertNotNull(msg, "Message cannot be null");
                String receivedMessage = new String(msg.getData());
                latch.countDown();

                log.info("Received message [{}] in the listener, latch: {}",
                    receivedMessage, latch.getCount());
                // since not acked, it should retry another time
                //c1.acknowledgeAsync(msg);
            })
            .subscribe();
        assertTrue(consumer instanceof MultiTopicsConsumerImpl);

        MultiTopicsConsumerImpl topicsConsumer = (MultiTopicsConsumerImpl) consumer;

        // 3. producer publish messages
        for (int i = 0; i < totalMessages; i++) {
            producer1.send((messagePredicate + "producer1-" + i).getBytes());
        }

        // verify should not time out.
        latch.await();

        consumer.close();
    }

Penghui meet this issue in their test.
This issue may be caused by the un-acked messages tracking in both individual ConsumerImpl and MultiTopicsConsumerImpl.
In MultiTopicsConsumerImpl, we are managing and tracking all un-acked message.  But when using MessageListener, there will be some issues.
This is because MessageListener.received() is only called 1 time when the message is delivered from underline individual ConsumerImpl.
The calling stack is like this:
MultiTopicsConsumerImpl.startReceivingMessages() // try to receive messages from all individual ConsumerImpl
\
MultiTopicsConsumerImpl.receiveMessageFromConsumer(ConsumerImpl consumer)
// call individual ConsumerImpl.receiveAsync() to import messages. And call MessageListener.received() in receiveAsync() call back.
\
MultiTopicsConsumerImpl.messageReceived()  // called in ConsumerImpl.receiveAsync() callback.
\
listener.received

While in ConsumerImpl.messageProcessed(), it not tracking messages for MultiTopicsConsumerImpl.
if (partitionIndex != -1) {
    // we should no longer track this message, TopicsConsumer will take care from now onwards
    unAckedMessageTracker.remove(id);    < === we are not tracking messages
} else {
    unAckedMessageTracker.add(id);
}

So for a timed-out message, it will not be redelivered in ConsumerImpl, and as above calling stack, MessageListener.received()  will not get called.

seems there was something wrong in above comments. will debug and post a new one.

The main reason for this error is that the message listener for TopicsConsumer was also set for individual sub-consumerImpl.
In above calling stack, when time-out, TopicsConsumer.UnAckedMessageTracker will let sub-consumerImpl to redeliver messages.

This is fixed by #2583


Expand pint requirements to include more versions

Description:

Type of change:

[ ] Bug fix.
[ ] New feature.
[ ] Documentation update.
[ ] Testing.

Checklist for the reviewer:

This checklist should be used as a help for the reviewer.

[ ] Is the change limited to one issue?
[ ] Does this PR close the issue?
[ ] Is the code easy to read and understand?
[ ] Do all new feature have an accompanying new test?
[ ] Has the documentation been updated as necessary?

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (ad90cc1) 75.08% compared to head (8af90b1) 75.08%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##           master     #150   +/-   ##
=======================================
  Coverage   75.08%   75.08%
=======================================
  Files          17       17
  Lines        1405     1405
=======================================
  Hits         1055     1055
  Misses        350      350

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Missing parameters

There are a few internal Parameters missing in the iDM integration.
B2 Durchflußsensor Heizung
B71 Heißgastemperatur 1
B86 Kondensationsdruck 1
These are all quite important parameters, always good to have a view on and see wrong developments. Could you add them to the integration?
Originally posted by @AndyNew2 in https://github.com/kodebach/hacs-idm-heatpump/issues/34#issuecomment-1770913585

My documentation for Modbus from IDM doesn't show list these parameters. @AndyNew2 If you have other documentation please tell me what it says for these parameters.

Actually you are right. I can't find them in my docu neiter. Crazy! These are key parameters very important to keep an eye on it, and especially this ones are not available on the interfaces. The same on KNX. I really don't understand why people like iDM, they crazy things.
I will try to contact iDM service. I case you do not know, the difference from Vorlauftemperatur and Heißgastemperatur should not be above 30°C. The higher the differenz the less efficient the pump is.
Furthermore it shall not reach 100°C, this is checked of course by the heatpump itself, however it is always good to know how close you got.
For the "Durchflußsensor" I know not all heatpumps have it, but it is a further critical parameter. It shall never drop below 6 l/min. Also a parameter you want to know how close you come to that...

Ah according the issue, this needs to wait, until supported by iDM. There is nothing to read of course, which had not been made available. The only other chance, if iDM will not help is to analyse the Windows program iDM provides. This one has all paramters included also the ones I mentioned. This had been the reason, I expected them on ModbusTCP as well. I thought this windows program uses the same interface. This is unfortunately not the case.

I just imagine another option without iDM support. But this is a little bit effort. We could use the web interface to iDM heatpump. The mentioned values are available via HTTP on http://<local_IP_heatpump>/#/settings/3,2,1,/details
The returned page holds the desired values...

I don't think the Windows App uses ModbusTCP. (1) AFAIK it only works after you log in, which indicates there using something that supports authentication, but more importantly (2) It works without enabling Modbus on the heat pump.
Regarding the HTTP idea: I don't want to mix that into this integration. (1) I have no interest in parsing HTML that isn't meant for automated reading (2) AFAIK the local heatpump interface page only works with Javascript enable, which makes the whole thing almost impossible anyway. (3) Even the more machine readable endpoints used by the interface (*) can't easily be accessed, because they need non-trivial authentication.
If you do find an HTTP endpoint, that can be accessed easily, I recommend trying the standard integration for simple HTTP requests.
(*) They still return HTML data with a JSON envelope, so not exactly machine readable.

Agree, and it would be a lot of work as well. Just noticed the Windows iDM App don't have these values anyway. Quite surprising, but this path seems not being a help.
The HTTP request returns just the frame and not the desired data inside. This path is difficult either. This is the result of this structured pages. Deeper anylsis needed to find actually the desired values. In any case this could be done via HTTP request function available as stated.
So I propose to keep this request around a month open to see if iDM responds. The chance is low they do, typically they ignore such requests and I know why they do so.
So it is up to you, you can close it straight away as well.

iDM answered as expected. You can't talk anymore to the people in Austria, they are all inpossible to contact. They always route you straight away to local service providers or sellers. So the answer was of course nonsense and the reality is no, they will not update the ModbusTCP implementation. iDM gots really kind of arrogant, since there is no way to propose and contact the core team, even with a kind of 2nd level support. They are untouchable...

ok in that case, let's just close this issue


Modify "Download" button midway down FCI homepage to "Request Access" button

Let's change this button on the FCI homepage from a "Download" button to a "Request Access" button and link here:
https://www.factcheckinsights.org/apply
This will make it easier to find the "Request Access" button for people on mobile.

Can we make sure to get this done tomorrow morning for the people who might be going to the site from their stickers or from any mention Bill gets to make in his remarks? I know I'm being annoying but this is mostly important for the duration of the conference. :D


Hacktoberfest readme update

Result of spammy PRs.
This PR should be reverted once the restriction is lifted.
Ref: https://hacktoberfest.digitalocean.com/hacktoberfest-update

Thank you, @jimangel!
/lgtm
/approve


the purpose of using identity operation

Hi @davidsandberg ,
The identify operations are used several times in your code,
for example, in facenet_train_classifier.py:
image_batch = tf.identity(image_batch, 'image_batch')
image_batch = tf.identity(image_batch, 'input')
label_batch = tf.identity(label_batch, 'label_batch')

I don't understand why the identity operations are needed.
The program goes well(in terms of executability) without those three lines.
This stackoverflow webpage discuss some cases of using identity, but seems irrelevant to the case here.

Hi @ugtony,
The reason for using it in this case is to be able to name the tensors that can later be used to feed data at that point, if you for example would like to bypass the input pipeline.
Then you can do something like
tf.get_default_graph().get_tensor_by_name("image_batch:0")

I see, thanks.


Correct username in 5.7 SU test

Overall Review of Changes:
The correct username in Amazon Linux 2 is ec2-user - Cannot see any usage of a Vagrant user in either the hardening repo or audit repo.
How has this been tested?:
We have run it against a EC2 instance that has had the Amazon2-CIS repo hardening steps completed.

Sorry! We should have read the Contributing guidelines. We will raise new PRs (keeping these two changes as separate PRs as originally intended) with GPG signed commits.

Sorry! We should have read the Contributing guidelines. We will raise new PRs (keeping these two changes as separate PRs as originally intended) with GPG signed commits.

Not a problem at all, we really appreciate feedback and contributions and keen to ensure people get credit for the contributions.
Thank you again


Don't allow deleting optimistic invited room member

Details

We don't want to allow the user to delete the optimistic invited room member.
Fixed Issues

$ https://github.com/Expensify/App/issues/48577
PROPOSAL: https://github.com/Expensify/App/issues/48577#issuecomment-2329585298
Tests

Same as QA Steps

[x] Verify that no errors appear in the JS console

Offline tests

Same as QA Steps
QA Steps

Open a room chat
Press on the header > Members > Invite member
Go offline
Invite a random user
Verify you can't remove the user
Go online
Remove the user
Verify the user won't reappear on the member list
Go back to the chat
Verify the added and removed system message shows the user login instead of Hidden

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

https://github.com/user-attachments/assets/5a7dd547-b134-4d5a-a059-56e1268982d2

Android: mWeb Chrome

https://github.com/user-attachments/assets/4dc48ed1-30d7-4dd0-9e8a-ec222077a2cf

iOS: Native

https://github.com/user-attachments/assets/811512e5-7be8-4847-b691-557be74dee53

iOS: mWeb Safari

https://github.com/user-attachments/assets/2c457ac0-1179-411b-978a-d6309c23b70c

MacOS: Chrome / Safari

https://github.com/user-attachments/assets/60b02fc5-1091-4cf0-bf30-41facda3f57f
https://github.com/user-attachments/assets/276136ee-e70b-4740-9fd8-d433abf3021b

MacOS: Desktop

https://github.com/user-attachments/assets/6fad2fb6-0a72-4f44-ae8f-9b2c91e1a5d0

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/arosiclair in version: 9.0.36-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/grgia in version: 9.0.36-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Print stylesheet and general style cleanup

Coverage increased (+0.15%) to 71.58% when pulling 6252c9d57ccca5a753475ad9cc7adfdeacf9b2ac on print-stylesheet into 413a813015702d91ee2ebf5fa02e2e9812aec3be on master.


Handle INT96 more gracefully in parquet-avro

The parquet-avro library does not support INT96 columns (PARQUET-323), and any attempt to process a file containing such a column results in:

throw new IllegalArgumentException("INT96 not implemented and is deprecated");

INT96 is still used in many legacy datasets, and so it would be useful to be able to process Parquet files containing these records, even if the INT96 values themselves aren't rendered.
The same functionality has already been re-added into parquet-pig (PARQUET-1133).
Reporter: Ben Watson
Note: This issue was originally created as PARQUET-1870. Please see the migration documentation for further details.

Ben Watson:
Thanks for the useful information Gabor. I'm not sure I want to add as much code as there is in that UUID PR - that's quite a lot of code for an unsupported, deprecated field.
I will take a look at other libraries, but I like parquet-avro because it lets me share code across my Avro and Parquet implementations, and deals with the JSON conversion nicely. I've had trouble finding an implementation that doesn't rely heavily on Hadoop Path objects or Spark, both of which cause a lot of problems within my plugin.
I have enabled INT96 support in my plugin by compiling parquet-avro myself with the one change I mentioned above, and I'm happy to stick to that solution going forwards. Thanks again.

Ben Watson:
For backstory, I maintain an Avro and Parquet Viewer IntelliJ plugin that allows Avro and Parquet files to be displayed visually, and a repeated complaint is that it's not possible to view files containing INT96 columns.
I have been able to solve this by replacing AvroSchemaConverter#308:

public Schema convertINT96(PrimitiveTypeName primitiveTypeName) {
  throw new IllegalArgumentException("INT96 not implemented and is deprecated");
}

with

public Schema convertINT96(PrimitiveTypeName primitiveTypeName) {
  return Schema.create(Schema.Type.BYTES);
}

This results in gibberish being printed, but at least the files are displayed.
I'm happy to raise a PR for this, but first want to check that this is an acceptable solution and that no one else has any better ideas.

Gabor Szadovszky / @gszadovszky:
We've already had a couple of discussions related to INT96. The main problem is that INT96 is not only deprecated but never specified properly. We don't want to encourage anyone to use it so we don't want to implement any additional support for this type.
Meanwhile, I understand your problem. One way to workaround this issue is to not use parquet-avro but another binding (e.g. parquet-thrift, parquet-protobuf etc.) that handles INT96.  Though, I am not sure which one would handle it properly.
If you want to fix this in parquet-avro, it would be better to hide this new behavior behind a flag. (You may check the related parts of this PR about introducing a new config.)


Proxy Authentication -

Hi,
in our company landscape it is necessary to use a proxy to access the internet, with authentication. When providing thoses information within the settings, I get a error when the report tries to display the popup asking for user credentials. The error is similar to : CALL SCREEN( not modal)... Sorry for not providing the full message text, because it is in German language.
This happens in method lcl_password_dialog->popup() when CALL SELECTION-SCREEN ... gets executed.
I should have mentioned that this happens, when I try to clone a repository and choose the button to select a specific branch.
Any chance for a quick fix?
Michael

also see https://github.com/larshp/abapGit/issues/658 and https://github.com/larshp/abapGit/issues/1010
Workaround is to create the online repository, and then use "Branch" -> "Change" to switch branches.

Maybe you schould show the proxy authentication popup right after saving the settings, presumbly the appropriate checkbox is active?
This way you can avoid poping the window up in background.

There is a refactoring task open for the password popups, https://github.com/larshp/abapGit/issues/1331

duplicate issue and workaround, closing


fix: let 'Change Recognizer' redirect to dialog root

Description
Closes #4652
History: https://github.com/microsoft/BotFramework-Composer/pull/3704/files#r518492203

Task Item

Screenshots

Coverage remained the same at 55.756% when pulling 66e05a70a6415aa02b89029fd2d9004f18d7b464 on yeze322:fix/4652 into de5adb1f79eae671df194803355a92e6879a98de on microsoft:main.


[2414] Enable multiple missing information items with the same label texts

Context
Followup to https://github.com/DFE-Digital/register-trainee-teachers/pull/1250
For situations with multiple missing information items, this PR ensures that the links reference the correct section.

Changes proposed in this pull request

Add a suffix to each attribute anchor where we have multiple items, eg, in a degrees section.

Guidance to review

 [S] Missing information UI (Degrees section)