interesting topic

It's quite interesting if this approach could be used for matching the ICD codes based different language version or do sort of machine medical concepts translation based on the term representation.

Thanks for your interest in this work.
Although, I don't exactly understand what you meant. Are you talking about projecting words and medical concepts (e.g. ICD9 codes, medication codes, procedure codes) to the same latent space?
We are considering similar approaches for our future work, but not just words but also other modalities of medical data.

Just a few thoughts and they may not be possible in your case. What I meant was to match for example ICD-9 procedure codes to other version, different language or different medical classification codes (e.g. ICHI) by the description. It may require the semantic matching tecnologies I may wonder.

I see. Let me re-iterate your goal just to see if I am not mistaken.
Given two different coding schemes for the same medical concept, such as ICD-9 procedure codes and CPT procedure codes, you want to see which ICD-9 procedure code corresponds to which CPT procedure codes.
In that case, it would be easy if you have two datasets where one dataset uses, for example, ICD-9 diagnosis codes and ICD-9 procedure codes, and another dataset uses ICD-9 diagnosis codes and CPT procedure codes. Using the first dataset, you can project ICD-9 diagnosis codes and the ICD-9 procedure codes in the same latent space. Using the second dataset, you can project ICD-9 diagnosis codes and the CPT procedure in another same latent space. Then, you can select one diagnosis code and retrieve k-nearst procedure codes respectively from each latent space and compare the retrieved  procedure codes.
Of course, to use this approach, the two dataset need to consist of similar patients (if one dataset is of children and another of seniors, the distribution of medical codes won't match), similar patient size, similar ICD-9 diagnosis codes. But this approach does not require you to compare the descriptions of ICD-9 procedure codes and CPT procedure codes to see which corresponds to which, thus eliminating the need to use NLP techniques.

Thank you for the inspriation. However, the first approach may require the same/similar granularity of the code schema for matching precisely. I'm wondering if your second proposal will work by using learning representation to encode the medical concepts based on their semantic information. If I'm right, your med2vec could be alternative to the exsiting coding system like ICD-9,10 and it will include not only the classification meaning but also the linguistic information. A previous work I have done is to use NLP to create a mapping table between ICD-9 and ICHI.

My second proposal was actually more similar to your previous work (creating a mapping table between ICD-9 and ICHI).
But if you have a good way to embed the descriptions of medical codes (such as doc2vec, or any sentence embedding algorithm) then you can project, for example, ICD-9 procedure codes and CPT procedure codes to the same latent space using their descriptions.
This would enable you to find out which ICD-9 procedure codes are similar to which CPT procedure codes.
But this approach requires a pre-trained sentence embedding (or text embedding) model. Typically, embedding algorithms used in NLP (word2vec, doc2vec or other approaches) require a huge corpus to train on. But the descriptions for the medical codes are very limited. So it is unlikely to work if you train your embedding algorithm only with the code descriptions. You will need pre-train the embedding model using some large medical text, then apply the model to the code descriptions.

Hello Ed,
I play around with med2vec model on mimic-3 for a few days, and I try @paulcx 's thoughts: Merge codes that are encode under different coding scheme. (this is one application I can think of to evaluate the quality of the vector of medical concepts)
I split the DRUG Code into 2 dataset to mimic two different hospital's data, and keep ICD code the same. After I ran the med2vec model, the performance is very good: I got 80% recall@8.
However, one thing I notice is that: the good result is achieved by the visit-level cost. If I only use the code-level cost, the performance will lower to 5% for recall@8, if I only use visit-level cost, the performance will still be around 80%.
Another thing is that the visit-level cost is much higher than code-level cost. which is reasonable consider the sigmoid function on visit-level, but will that cause the impact that: the model will focus much more on visit-level?
Given the above two things, my question is: During your experiment, do you think visit-level is the key of the success of the medical vector and code-level is not that important? Or do I miss anything?
Thanks!

Hi Xianlong,
Generally, I wouldn't recommend running med2vec on MIMIC-III.
It is a very small dataset(about 45K patients in total), and there are probably only a couple thousand patients (or even less) that made at least three visits, because it's an ICU dataset. Therefore the visit-level softmax loss probably won't train well.
I uploaded process_mimic.py so that people can "try out" med2vec, not to obtain state-of-the-art performance with MIMIC-III.
And you even divided MIMIC-III into two parts, so the data size is even more problematic.
Now, to your findings:
When you say you got certain recall@8, I have so many questions to ask regarding your experiment setup. And when you say that the visit-level cost is much higher than code-level cost, I'd like to know how much. So if you have time, we can talk on Skype. I'm interested to learn your findings.
Please send me an email to<EMAIL_ADDRESS>BTW, you might be right about the balance between visit-level cost and code-level cost. Empirically, summing the two worked just fine. But if you can think of some clever way to balance the two and run experiments, it would be great to learn new findings.
Thanks,
Ed

@1230pitchanqw  Hi，I have almost same questions as @mp2893 asked. It would be nice if you can share your findings in details and we could talk about it.

Hi Edward,
I have run your code on my dataset and got the .npz files. I find it contains 6 numpy.array variables W_emb b_output b_hidden b_emb W_output W_hidden. But on the GitHub Repo I can’t find further description about these output variables. Can you give some detailed instruciton about the output?

@1230pitchanqw Thanks for your insights. I'm wondering if this paper 'Earth Mover’s Distance Minimization for Unsupervised Bilingual Lexicon Induction' would help somehow.


Please add this micro.blog social link

Minima didn't support micro.blog as a social link yet. Here is the icon for the assets/minima-social-icons.svg file:
<symbol id="microdotblog" fill-rule="evenodd" clip-rule="evenodd" stroke-linejoin="round" stroke-miterlimit="1.414">
        <path d="M13.8958184,12.3539692 C13.7934913,12.5766333 13.7214973,12.7460692 13.6798363,12.8622768 C13.534018,13.2690171 13.4876062,13.5959963 13.4737723,13.8031622 C13.4047037,14.8374843 13.8695637,15.452098 13.9792411,15.6152902 C14.143626,15.8598834 14.2103299,15.996999 14.1793527,16.0266369 C14.1187128,16.1331597 13.9329861,16.1331597 13.6221726,16.0266369 C13.1559524,15.8668527 11.8671503,15.3666667 11.3815104,14.6616443 C11.1859547,14.3777486 11.0620507,14.2565499 10.9838025,14.2130531 C10.0617698,14.5547858 9.05472096,14.7428571 8,14.7428571 C3.581722,14.7428571 0,11.4425562 0,7.37142857 C0,3.30030099 3.581722,0 8,0 C12.418278,0 16,3.30030099 16,7.37142857 C16,9.29250872 15.2024583,11.041949 13.8958184,12.3539692 Z M7.94285714,9.78095238 C9.51605596,10.7990263 10.3706771,11.2483272 10.5067204,11.1288551 C10.6427638,11.0093829 10.4795457,10.0577483 10.0170661,8.27395135 C11.4827084,7.15954158 12.1741114,6.48559023 12.0912751,6.2520973 C12.0084388,6.01860437 11.0529434,5.87976276 9.22478879,5.83557246 C8.61970046,4.10392098 8.19238991,3.23809524 7.94285714,3.23809524 C7.69332437,3.23809524 7.26601382,4.10392098 6.6609255,5.83557246 C4.82960347,5.88951121 3.87410803,6.02835282 3.7944392,6.2520973 C3.71477036,6.47584178 4.40617335,7.14979313 5.86864817,8.27395135 C5.36780472,9.99753359 5.20458662,10.9491682 5.37899385,11.1288551 C5.55340109,11.308542 6.40802218,10.8592411 7.94285714,9.78095238 Z"></path>
</symbol>

here is the line to add the actual entry in _includes/social.html:
  {%- if site.microdotblog_username -%}
<svg class="svg-icon"><use xlink:href="{{ '/assets/minima-social-icons.svg#microdotblog' | relative_url }}"></use></svg> {{ site.microdotblog_username| escape }}
{%- endif -%}

You can now just add an entry to your _config.yml:
microdotblog_username: manton

and get a link + icon to your micro.blog page.

See PR #267

First I've heard of it.  Looks interesting, though.  Is it especially popular?  According to Alexa, it's barely in the top 200,000 sites in just the US, barely the top million globally.

It was a successful kickstarter a while ago, and is rather skewed towards the Mac-using end of things.

Fixed in #267


Docs: built-in tokens list

Is there a complete well-documented list of built-in token types and their special properties? I looked into the documentation several times, but I can't seem to find anything.

Sorry, not available. It's better to dig source code.


AOT error

I have aot error:
/node_modules/angular2-modal/plugins/bootstrap/src/angular2-modal-bootstrap.ng-flat.ngfactory.ts:17:35: Namespace '''' has no exported member 'BootstrapModalModule'.
/node_modules/angular2-modal/plugins/bootstrap/src/angular2-modal-bootstrap.ng-flat.ngfactory.ts:20:21: Namespace '''' has no exported member 'Modal'.
/angular2-modal/plugins/bootstrap/src/angular2-modal-bootstrap.ng-flat.ngfactory.ts:293:20: Namespace ''*'' has no exported member 'BSMessageModalTitle'.
....
How can I fix it ?

I can't help here.
Please reopen a new issue with information about the version of the library, angular version and other information you might think will help understanding the problem.

I had also this issue. Upgrading to angular 4.2.6 solved it for me.


[FIX] limit graph to 20k commits

From then on, risks are it becomes slow on huge repos. You still can adapt the setting, if need be.
Fix #494

@divmain is this it?

See plans on #494


Grid names are hard to see in dark mode

[ ] Make the colour a bit brighter and match the “More booking info” colour if possible

Figure: Names in the grid are really hard to see

Duplicated. Refer to second task of https://github.com/SSWConsulting/SSW.SophieBot/issues/143#issuecomment-961748347


Update fs2-core to 3.1.1

Updates co.fs2:fs2-core from 2.5.9 to 3.1.1.
GitHub Release Notes - Changelog - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "co.fs2", artifactId = "fs2-core" } ]

labels: library-update, semver-major

Superseded by #397.


⚠ Bofrost.fr has degraded performance

In 1a36073, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1165 ms

Resolved: Bofrost.fr performance has improved in 86bccc4.


all image assets start with  './/' at the key of the manifest file

I've been using a version of your suggested webpack.config.js file you used in your flask-webpack video but for some reason this is the manifest file i'm getting ('.//' at each key and it won't add the images folder from the path, but the values are as expected):
{"assets":{
   ".//japan-tickets.png":"images/japan-tickets.db7c68e1892ff11bf2dff9256a8a3cd9.png",
   ".//japan-tickets-large.png":"images/japan-tickets-large.97a1e7f33489d30ee39b23e981d37b5e.png",
   ".//cc/mastercard.png":"images/cc/mastercard.373e4f1ac72b50605183e8216edde845.png",
   ".//favicon.png":"images/favicon.7a490944f315e24d179e27f3962f3545.png",
   ".//cherryblossom.jpg":"images/cherryblossom.116db0d040ecec60aae1b5051d4a48ca.jpg",
   ".//Wisteria-tunnel-japan-shutterstock.jpg":"images/Wisteria-tunnel-japan-shutterstock.41b33e7718dba673ba7c0d372ec1619f.jpg",
   ".//cc/jcb.png":"images/cc/jcb.58f43e5f1fb8c6a4e7e76a17e7824e29.png",
   ".//spinner.gif":"images/spinner.3930838bac528f00cb908784164c0a9f.gif",
   ".//cc/diners-club.png":"images/cc/diners-club.03afaaa2d75264e332dc28309b7410b9.png",
   ".//jt.png":"images/jt.747ad1791eec4550135c59788f60a245.png",
   ".//cc/visa.png":"images/cc/visa.26bcf191ee12e711aa540ba8d0c901b7.png",
   ".//cc/american-express.png":"images/cc/american-express.8a5ade08365dcc7e5fa39a946bb76ab8.png",
  ".//cc/discover.png":"images/cc/discover.f89468f36ba1a9080b3bb05b9587d470.png",
"app_js.js":"app_js.d8c4209cd30405e63bd3.js"},
"publicPath":"http://localhost:2992/assets/"}

And this is my config file:
var fs = require('fs');
var path = require('path');

// Webpack and third party plugins.
var webpack = require('webpack');
var ExtractTextPlugin = require('extract-text-webpack-plugin');
var ManifestRevisionPlugin = require('manifest-revision-webpack-plugin');
var HotModuleReplacementPlugin = require('manifest-revision-webpack-plugin');
var historyApiFallback = require('connect-history-api-fallback');

// Environment detection.
var node_env = process.env.NODE_ENV || 'development';

/*
 Configuration settings
 ------------------------------------------------------------------------------
 */

// Where is your project located relative to this config?
var context = path.join(__dirname, '..');

// Where are your source assets located?
var rootAssetPath = 'japantickets/static';
var contextRoot = path.join(context, rootAssetPath);

// Where will the files get built to?
var buildOutputPath = path.join(__dirname, '../build/public');

// How should certain asset types be configured?
var assets = {
    bootstrap:{
      path: 'bootstrap',
      filename: '[name].[hash].css'
    },
    fonts: {
        path: 'fonts',
        filename: '[path][name].[hash].[ext]'
    },
    images: {
        path: 'images',
        filename: '[path][name].[hash].[ext]'
    },
    scripts: {
        path: 'scripts',
        filename: '[name].[hash].js',
        chunkFilename: '[id].[hash].js'
    },
    styles: {
        path: 'styles',
        filename: '[name].[hash].css'
    }
};

// Which top level JS and CSS files should get output?
var chunks = {
    app_js: [
        path.join(contextRoot, assets.scripts.path, 'app_js.js')
    ]
};

// Where will assets get served in development mode? This depends on running
// the webpack dev server.
var publicPath = process.env.PUBLIC_PATH || 'http://localhost:2992/assets/';

/*
 Do not edit past this line unless you are tinkering with webpack.
 ------------------------------------------------------------------------------
 */

// Plugins that will load in all environments.
var plugins = [
    // http://webpack.github.io/docs/list-of-plugins.html#noerrorsplugin
    new webpack.NoEmitOnErrorsPlugin(),

    // https://github.com/webpack/extract-text-webpack-plugin
    new ExtractTextPlugin(assets.styles.filename),

    // https://github.com/nickjj/manifest-revision-webpack-plugin
    new ManifestRevisionPlugin(path.join(__dirname, '../build/manifest.json'), {
        rootAssetPath: path.join(rootAssetPath, assets.images.path),
        ignorePaths: ['/styles', '/scripts', '/fonts', '/bootstrap']
    }),

    // https://webpack.js.org/plugins/hot-module-replacement-plugin/
    new webpack.HotModuleReplacementPlugin()
];

// Development environment only plugins.
if (node_env !== 'development') {
    var developmentPlugins = [
        // http://webpack.github.io/docs/list-of-plugins.html#uglifyjsplugin
        new webpack.optimize.UglifyJsPlugin({compressor: {warnings: false}}),

        // http://webpack.github.io/docs/list-of-plugins.html#dedupeplugin
        new webpack.optimize.DedupePlugin(),

        // https://webpack.js.org/plugins/hot-module-replacement-plugin/
        new webpack.HotModuleReplacementPlugin()
    ];

    plugins.push(developmentPlugins[0])
}

module.exports = {
    context: path.join(__dirname, '../'),
    entry: chunks,
    output: {
        path: buildOutputPath,
        publicPath: publicPath,
        filename: assets.scripts.filename,
        chunkFilename: assets.scripts.chunkFilename
    },
    resolve: {
        // Allow requiring files without supplying the extension.
        extensions: ['*', '.js', '.css']
    },
    devtool: 'inline-source-map',
    devServer: {
      contentBase: path.join(__dirname, '../build/public'),
      historyApiFallback: true,
      watchOptions: { aggregateTimeout: 300, poll: 1000 },
      headers: {
        "Access-Control-Allow-Origin": "*",
        "Access-Control-Allow-Methods": "GET, POST, PUT, DELETE, PATCH, OPTIONS",
        "Access-Control-Allow-Headers": "X-Requested-With, content-type, Authorization"
      }
    },
    module: {
        loaders: [
            {   test: [/\.DS_Store$/,
                       /\.zip$/],
                loader: 'ignore-loader'
            },
            {
                test: /\.js$/, loader: 'babel-loader',
                exclude: /node_modules/
            },
            {
              test: /\.css$/,
              loader: 'css-loader'
            },
            {
              test: /\.*(sass|scss)$/,
              loaders: ['css-loader', 'sass-loader']
            },
            {
                test: /\.(jpe?g|png|gif|svg([\?]?.*))$/i,
                loaders: [
                    'file-loader?context=' + rootAssetPath + '&name=[path][name].[hash].[ext]',
                    'img-loader?bypassOnDebug&optimizationLevel=7&interlaced=false'
                ]
            },
            {   test: /\.woff(2)?(\?v=[0-9]\.[0-9]\.[0-9])?$/,
                loader: "url-loader?limit=10000&mimetype=application/font-woff"
            },
            {   test: /\.(ttf|otf|eot|svg)(\?v=[0-9]\.[0-9]\.[0-9])?$/,
                loader: "file-loader"
            }
        ]
    },
    plugins: plugins,
};

Then i'm trying to add an asset in the html with an asset key from the manifest:
<link rel="shortcut icon" type="image/x-icon" href="{{ asset_url_for('.//favicon.png') }}" />

This makes the asset_url_for tag from flask_webpack not work (script and style tags are working great) because of this if in the asset_url_for function (line 123 in flask_webpack/init.py):
if '//' in asset:
            return asset

I'm guessing there's some logic to that if (happy to know why), so i was trying to get the plugin to output the keys correctly but with no luck. any idea why i see this issue?
Thanks,
Sarai.

I was able to resolve my issue (not the bug itself though) by adding '/' to the rootAssetPath.
(I also gave the app's static folder instead of the static/images so it will include the images folder in the path, i guess this is also part of the bug, but adding the end '/' solved the issue in both cases)
Used to be something like 'path/to/assets', now 'path/to/assets/' and it got rid of one '/' in the manifest for some reason. Now a manifest entry looks like:
"./images/img.png":"images/img.db7c68e1892ff11bf2dff9256a8a3cd9.png",
still has the './' in the beginning, but at least now i can use the url_for tag with flask_webpack.
I hope this helps resolving this issue.

I have the same issue with ./ being at the beginning of image.


Document Extraction cognitive skill supporting Blob URL

Hi there,
I am exploring this Cognitive Skill to implement a Use Case from the company I'm working on.
The feature seems to fit our needs to do the document cracking from a Table Storage as a data source, where it will store the Blob URL of the file that needs to extract the content.
Running some tests, I had some challenges using the skill, and the document mentions using an object like this:
{
  "$type": "file",
  "data": "BASE64 encoded string of the file"
}

However, I realized the field data does not fit our case because it requires the blob's content, but this is the output that we're expecting from this skill.
Exploring further the skill capabilities, I could see if I use the object below, it will work as expected:
{
  "$type": "file",
  "url": "blob's URL with the public token"
}

Is it a capability of the feature? If so, I'd suggest adding it to the document because it's a pretty new feature, and it's not easy to find some resources to explore.
Thank you so much.

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

ID: 888362f6-45c3-2bb6-47b5-80c756ac77ad
Version Independent ID: 5ca9f578-ad8c-90a6-9ceb-a23b41427451
Content: Document Extraction cognitive skill - Azure Cognitive Search
Content Source: articles/search/cognitive-search-skill-document-extraction.md
Service: cognitive-search
GitHub Login: @Careyjmac
Microsoft Alias: chalton

@viniciustotti Thanks for the feedback. I have assigned the issue to the content author to evaluate and update the document as appropriate.

Hi there,
You are correct that the object definition using "url" instead of "data" will also work. I actually have a task on myself to do an entire separate document that better explains everything regarding file references in skillsets, but I am not sure when I am going to have time to do it. In the meantime, I'll try to make sure I set aside some time next week to update the doc to including mentioning the "url" option as you have noticed.

Thank you so much for your reply @Careyjmac.
I'm designing a feature into the company platform and I just want to make sure that we could rely on that implementation using the object:
{
  "$type": "file",
  "url": "blob's URL with the public token"
}

Yes, you can rely on that implementation. In fact, if you wanted to and if it was easier, we actually also support this:
{
  "$type": "file",
  "url": "blob's URL"
  "sasToken": "blob SAS token"
}

If that is easier for your use case to keep the URL and SAS token separate.

That is great!
Thanks @Careyjmac

@viniciustotti @Careyjmac   I'm going to close this issue.  It looks like the customer is unblocked and Carey has a work item to fill in the content gap.  @viniciustotti if you need more help with this particular issue, feel free to re-open.  Thanks everyone!!
#please-close


Sort exercises in track config

I normalized current difficulties to 2 (easy), 4 (medium), or 8 (hard) and then sorted the exercises by difficulty and name. That'll let students find exercises for each difficulty quicker.

Doesn't it make sense to keep the difficulty values as they are and to sort by them?
The user might not see the difference but earlier exercises in a certain difficulty level should still be easier than the later ones.

I think using prerequisites responsibly would convey that intent much clearer and take better advantage of how the website presents content. Do this easier string exercise before trying this other string exercise that might be harder. The site doesn’t make any claim that the display order is the recommended order so I don’t think we can do that here.
Sorting by raw difficulty helps if there are clear differences between the levels. The differences between an easy and medium exercise are usually  evident judging by the number of student attempts and solutions. The differences between difficulty 2 and 3 might be too small to be useful though. We also don’t know how the difficulty levels were originally assigned which would have been different people at different periods.

But if you want to keep the difficulties as-is and then sort the exercises, that's fine by me. I just have a strong opinion. :)

Didn't find the time to read and think about it yet, will respond the following days! Sorry for the delay.

No rush.

Perhaps consider upgrading the difficulty of a few exercises:
2 -> 4 - matching-brackets
2 -> 4 - simple-cipher
5 -> 8 - zebra-puzzle

@BNAndras Isn't a syllabus necessary to make use of the prerequesites?
In tracks that don't feature a learning mode I usually just go through the exercises by order. Certainly not everyone will do the same, but I think it's an obvious and simple approach. Just "go next".
Imo a good UX will accommodate different types of students / learning approaches.
@keiravillekode lgtm. In my experience the "easy" difficulty tends to include jumps in difficulty on some tracks bc the problem-specifications feature dead simple exercise and then still pretty basic exercises, but they certainly take more effort. Feels off to me sometimes. That's why I at least would like to try to order the exercises by ascending difficulty within the 3 broad difficulty categories.

@BNAndras Can you please include the script that you are using to order the exercises in the bin directory and ask contributors to execute that script when contributing a new exercise? :)

@BNAndras Isn't a syllabus necessary to make use of the prerequesites? In tracks that don't feature a learning mode I usually just go through the exercises by order. Certainly not everyone will do the same, but I think it's an obvious and simple approach. Just "go next". Imo a good UX will accommodate different types of students / learning approaches.

Oh, boy. That's egg on my face. You're 100% correct on the prereqs.
As far as the script goes, it's a jq snippet at https://forum.exercism.org/t/are-go-practice-exercises-ordered/4464/7. So yeah I can toss it in the bin folder. We can make a GHA that checks PRs to see if the exercises are ordered so that'll catch a new exercise being added in the wrong place.

Oh, boy. That's egg on my face. You're 100% correct on the prereqs.

All good, I looked it up in the docs because I wasn't sure myself :)

As far as the script goes, it's a jq snippet at https://forum.exercism.org/t/are-go-practice-exercises-ordered/4464/7. So yeah I can toss it in the bin folder. We can make a GHA that checks PRs to see if the exercises are ordered so that'll catch a new exercise being added in the wrong place.

Great idea! :)
It might be even possible to have a GHA that automatically adds a commit if the PR is not ordered, but not sure how much hassle that is to set up.


Import AddUninstallTarget.cmake from YCM v0.14.1 to fix compilation with ninja

From https://github.com/robotology/ycm/blob/v0.14.1/modules/AddUninstallTarget.cmake
The osqp-eigen PR https://github.com/robotology/osqp-eigen/pull/117 introduced a ninja-related regression as it ported from YCM the PR https://github.com/robotology/ycm/issues/169  without porting the related commit https://github.com/robotology/ycm/commit/f2c61b1b9f5b2a7935bf79d1d43a68a218225804 that is a fix for https://github.com/robotology/ycm/issues/240 . To avoid the problem, I just import the module from the latest YCM release.
Fix https://github.com/robotology/robotology-superbuild/issues/1153 .

Thank you @traversaro


conflicting formulas

seems formula have been added upstream: https://github.com/Homebrew/homebrew/commit/f8e10bbd42a9eeae801499fbfdb27e49b9b88c5e
> brew update; brew upgrade
Warning: Could not create link for jingweno/ccat/ccat, as it
conflicts with Homebrew/homebrew/ccat. You will need to use the
fully-qualified name when referring this formula, e.g.
  brew install jingweno/ccat/ccat
Updated Homebrew from 84dd2e2f to d956958c.
==> New Formulae
ccat
...

one solution is to remove this tap:
brew untap jingweno/ccat

That's right. It was accepted to upstream. I have updated the README.


Remove trailing 'L' off the string returns from hex [Python 2.x]

Hi,
Seems hex is being used to convert integer to hex string, when ethjsonrpc constructs the RPC requests (the argument value of eth_sendTransaction specifically).
The hex in Python 2.x will represent a long with a trailing 'L':
>>> hex(1L)
'0x1L'

>>> hex(12345678901234567890)
'0xab54a98ceb1f0ad2L'

'L' isn't legal for a hex string. The odd fact is that geth RPC often accepts such false format without complaining, but not always. Here is an error message from "Geth/v1.4.0-unstable/linux/go1.5.1":
{"code":-32602,"message":"Unable to parse number"}

Removing the tailing 'L' solved the error.
While still no idea what causes the difference (the error only happens on one of the geth instances we installed), the error is real. Worth mentioning the issue here in case others are having the same problem.
It also worth a simple fix like hex(x).rstrip("L") since the hex string feeds geth PRC could indeed contain the illegal character.

Thank you, fixed in https://github.com/ConsenSys/ethjsonrpc/commit/fe27f551aa943532e0a12740fb02ab998345dac5 .

Thank you 👍


Buffer depreciated

Buffer deprecated
i've noticed there is a depreciated warning from the use Buffer class from Authorization example

I would suggest replacing line 19  of the app.js file with the following code
'Authorization' : 'Basic ' + (new Buffer.from(client_id + ':' + client_secret, 'toBase64'))

Perhaps that might work?

Thanks mate that's worked for me


Script that switches between profiles

Fixes #294.
Adds a script that overwrites the user's settings file with another one based on a profile name extension.
Example:
$ cat ~/.sematic/settings.yaml
default:
  AWS_S3_BUCKET: XXX
  SEMATIC_API_ADDRESS: https://dev1.XXX
  KUBERNETES_NAMESPACE: XXX
  SEMATIC_API_KEY: XXX
$
$ bazel run //bazel:switch-settings -- local
INFO: Analyzed target //bazel:switch-settings (0 packages loaded, 0 targets configured).
INFO: Found 1 target...
Target //bazel:switch-settings up-to-date:
  bazel-bin/bazel/switch-settings
INFO: Elapsed time: 0.113s, Critical Path: 0.00s
INFO: 1 process: 1 internal.
INFO: Build completed successfully, 1 total action
INFO: Build completed successfully, 1 total action
Successfully switched to local!

$ cat ~/.sematic/settings.yaml
default:
  SEMATIC_API_ADDRESS: http://localhost:3000
  SEMATIC_API_KEY: XXX
$

@neutralino1:

How about before cp, we move the current file to $SETTINGS_FILE.backup just in case there was some important stuff in there?

Moved to ${SETTINGS_FILE}_bck so it doesn't overwrite a potential existing "backup" or "bck" profile.

Also, at the top of the script, you can set -x at the top of the script for more visibility. It will print the executed commands to the terminal.

+ SETTINGS_FILE=/Users/tudorscurtu/.sematic/settings.yaml
+ USAGE='\n
Overwrites your /Users/tudorscurtu/.sematic/settings.yaml with the contents of a file that is expected to be found at /Users/tudorscurtu/.sematic/settings.yaml.<env_name>\n
Usage: bazel run //bazel:switch-settings -- <env_name>
\n'
+ '[' 1 -ne 1 ']'
+ ENV_FILE=/Users/tudorscurtu/.sematic/settings.yaml.local
+ '[' '!' -f /Users/tudorscurtu/.sematic/settings.yaml.local ']'
+ cp /Users/tudorscurtu/.sematic/settings.yaml.local /Users/tudorscurtu/.sematic/settings.yaml
+ echo -e 'Successfully switched to local!\n'
Successfully switched to local!

Looks kinda messy. Given that the argument is that only the Sematic devs will be using this, I think it's fine to not have this verbose.

Finally, why put this is bazel/? How about tools/?

Done.

@neutralino1 Done. Done.


Sigv4 Blog Post PR

This PR adds a blog post for the sigv4 extension, and discusses why it was added, how to use it, and how it affects Collector configuration.

Overall LGTM, two thoughts:

You promise "why it was added" though that's not explained
The post leads with the SigV4 extension and mentions as an aside the change in the PRW (is it worth a separate post? should that combo be the center of attention?)

For the first point, I will reword it to be more clear with what the rest of the post explains, something like "what it is used for" instead. As for the second point, I don't think it is worth a separate post, but I will change the title of the post since the migration to using sigv4 + PRW is a large focus.


Merge the v1.1.1 branch

🗣 Description
This pull request merges the v1.1.1 branch with the following changes:

#9
#10
#11
#12
#13

💭 Motivation and context
The desired changes for v1.1.1 have been made and now need to be rolled out as a new release.

🧪 Testing
Automated tests pass. This branch was tested in this Actions run.

✅ Pre-approval checklist

[x] This PR has an informative and human-readable title.
[x] Changes are limited to a single goal - eschew scope creep!
[x] All relevant type-of-change labels have been added.
[x] I have read the CONTRIBUTING document.
[x] These code changes follow cisagov code standards.
[x] All new and existing tests pass.

✅ Pre-merge checklist

[ ] Finalize version.

✅ Post-merge checklist

[ ] Add a tag or create a release.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

4 of 20 (20.0%)  changed or added relevant lines in 3 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.1%) to 30.333%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/ymlmd/yml2md.py
0
2
0.0%

src/yml/normalize_yml.py
2
5
40.0%

src/mdyml/convert_cisagov.py
2
13
15.38%

Files with Coverage Reduction
New Missed Lines
%

src/mdyml/convert_cisagov.py
2
20.94%

Totals

Change from base Build<PHONE_NUMBER>:
0.1%

Covered Lines:
107

Relevant Lines:
352

💛  - Coveralls


Fix https://github.com/ballerina-platform/ballerina-lang/issues/7601

There, even though the literal value is the same, the context is different, it is not the context which it expects the "default package path" value. So it cannot be used there.


Efficient git traversal

See: https://github.com/Byron/gitoxide/discussions/617

closed by 2ff17dac1198dd328185e5d173a44ca4a75f800b


fix: Improve injectQuery path handling (fix #2422)

Fixes #2422.
This PR updates injectQuery to handle Windows paths in a saner manner.

As url.parse has been deprecated in Node, relative paths are now resolved using new URL(url, 'relative:///').
When a Windows drive letter is erroneously used as a protocol (e.g. c:), it instead resolves it using url.pathToFileURL and removes the first character (a forward slash) so that the following code in importAnalysis can properly handle it as an in-root URL:

https://github.com/vitejs/vite/blob/76f35352dbfc4e16cb65005e57e2757a97675bfd/packages/vite/src/node/plugins/importAnalysis.ts#L182-L192

Sorry, Please merge this soon. I can't continue my development process because of this error: (https://github.com/sveltejs/kit/issues/460)

Please merge.. waiting so long to try sveltekit.. Thanks

Yes !!!


FancyZone activated automatically without holding Shift when dragging window for the first time after logging via RDP after update

Microsoft PowerToys version
0.81.1
Installation method
GitHub
Running as admin
No
Area(s) with issue?
FancyZones
Steps to reproduce

Enable "Hold Shift key to activate areas when dragging windows" (I'm using Chinese version so translation may not be 100% correct)
Login the account via RDP, then drag any window (only hold left button), FancyZone is activated automaticaly.
After the first time, this won't happen anymore, it behaves normally.

Logfile exported:
PowerToysReport_2024-06-14-09-27-31.zip
Settings:

✔ Expected Behavior
FancyZone only triggered when holding Shift
❌ Actual Behavior
FancyZone is triggered without holding Shift although it is set to do so.
This was not the case before I update, but I couldn't recall the version before the update.
Other Software
No response

Hi I'm an AI powered bot that finds similar issues based off the issue title.
Please view the issues below to see if they solve your problem, and if the issue describes your problem please consider closing this one and thumbs upping the other issue to help us prioritize it. Thank you!
Open similar issues:

FancyZones Zones automatically activated when dragging a window (#30090),  similarity score: 0.78
Shift key invokes FancyZones when not moving the mouse (#29560),  similarity score: 0.76
Zone Behavior malfunction (#27615),  similarity score: 0.75

Closed similar issues:

FancyZones not working after computer restart (#13668),  similarity score: 0.76
[FancyZones] does not activate while holding down Shift key (#6036),  similarity score: 0.75

Note: You can give me feedback by thumbs upping or thumbs downing this comment.

I have checked these issues and none of them solve my problem

I also get this issue, though mine seems to happen randomly after waking from sleep or hibernation on my Windows 11 PC, it's like a phantom SHIFT key has been pressed. Pressing SHIFT again seems to toggle it back off though. I am using PowerToys 0.81.1 on Windows 11 24H2 (26100.994)

I am having the same issue. I agree it seems to happen mostly when I first use RDP, but might happen upon waking other times as well, but less predictably.


Suggestion: Place value boxes outside of cards

Hi,
I noticed that value boxes always get wrapped into cards (at least I couldn't find out how to change this default behavior in the UI editor). I think the resulting app would look better, if value boxes don't have an additional border around them.
Currently it looks like this:

Here's what I think would be nicer default behavior (from bslib::bslib_theme_preview()):

Best
Albert

Agreed. Currently, this is because only items that are gridlayout aware can go directly on the grid. That being said, there is a wrapper in gridlayout already that adds this functionality so it should be as simple as writing out grid_place(area="AREA", value_box(...)).

Another option will be once the api for the column layout stuff in bslib is stable, that will be added to the editor, so no gridlayout will be needed. Once the next release of the editor is out I can look at getting this figured out.

This change worked nicely for me. Thanks. One thing I noticed is that when I reopen the UI editor the value boxes are displayed as unknown UI at the bottom of the card. But in the app they are at the top of the card (above the chart). Is that intended behavior?


Error connecting in 2.7.23 and 2.7.24

The integration is no longer able to connect to the charging station. I can access the charging station on the local IP via the Alfen MyEve app.
Alfen Firmware 6.5.0-4217
Alfen Model: Eve Singe Pro-line
Home Assistant version 2024.1.4
Supervisor version: 2023.12.1
Alfen integration problem version: 2.7.23 and 2.7.24 after going back to 2.7.20 everything works again.
2024-01-20 07:54:22.948 ERROR (MainThread) [custom_components.alfen_wallbox] Unexpected error creating device <IP_ADDRESS> 'Timeout' object does not support the context manager protocol 2024-01-20 07:54:25.158 WARNING (MainThread) [homeassistant.helpers.service] Referenced entities number.alfen_eve_single_pro_line_power_connector_max_current are missing or not currently available 2024-01-20 07:54:30.163 WARNING (MainThread) [homeassistant.helpers.service] Referenced entities number.alfen_eve_single_pro_line_power_connector_max_current are missing or not currently available 2024-01-20 07:54:32.439 ERROR (MainThread) [custom_components.alfen_wallbox] Unexpected error creating device <IP_ADDRESS> 'Timeout' object does not support the context manager protocol

let's try this beta
https://github.com/leeyuentuen/alfen_wallbox/releases/tag/2.7.25

Seems to work fine! Thnx


Play/Pause with Single Tap on Web

Hi, I am trying out the web examples provided on https://pub.dev/packages/media_kit (i.e. Visit). On desktop browsers (Safari and Chrome), single tap does not seem to pause / play the video. However, I also tried it on my phone browser (iOS; Safari and Chrome) and the single tap function seems to work. I did not write any code in this case, as I am simply visiting the link provided on pub.dev. Is this behavior expected? If this is indeed expected, are there settings that can enable single-tap to play/pause in the Player or VideoController class?

I am also concerned about the same issue. Does anyone know the details?

check last version on pub


[LLM Runtime] GGUF Enhencement

Type of Change
This PR is the GGUF Enhencement for https://github.com/intel/neural-speed/pull/40.
New feature add

Support loading & inference official GGUF models from HF.
Support loading & inference official GGUF models from llama.cpp.

Description:
Verified GGUF models from HF:

TheBloke/Llama-2-7B-Chat-GGUF : q4_0 / q5_0 / q8_0
TheBloke/Mistral-7B-v0.1-GGUF : q4_0 / q5_0 / q8_0
TheBloke/CodeLlama-7B-GGUF : q4_0 / q5_0 / q8_0
TheBloke/CodeLlama-13B-GGUF: q4_0 / q5_0 / q8_0

Verified GGUF models from Llama.cpp:

meta-llama/Llama-2-7b-chat-hf : f32 / f16 / BTLA
tiiuae/falcon-7b : f32 / f16 / BTLA
tiiuae/falcon-40b : f32 / f16 / BTLA
mpt-7b : f32 / f16 / BTLA
mpt-30b : f32 / f16 / BTLA
bloomz-7b1 : f32 / f16 / BTLA

How to load the GGUF bin file in NeuralSpeed:
    prompt = "Once upon a time"
    tokenizer = AutoTokenizer.from_pretrained(args.model_path, trust_remote_code=True)
    inputs = tokenizer(prompt, return_tensors="pt").input_ids
    streamer = TextStreamer(tokenizer)

    model = Model()
    model.init_from_bin(args.model_name, gguf_path)
    outputs = model.generate(inputs, streamer=streamer, max_new_tokens=300, do_sample=True)

# Please check this script for more details and input parameters.
# python scripts/python_api_example_for_gguf.py --model_name falcon --model_path /home/model/falcon-7b -m /home/model/falcon-7b/ggml-model-f32.gguf

Note: These GGUF models from Llama.cpp can be accelerated by NeuralSpeed BestTLA.
How to accelerate GGUF by BTLA:

# quantization and re-run the python_api_example_for_gguf.py
./build/bin/quant_falcon --model_file /home/model/falcon-7b/ggml-model-f32.gguf --out_file ne-falcon-q4_j.bin --weight_dtype int4 --compute_dtype int8

How to load the GGUF bin file in intel-extension-for-transformers:
from transformers import AutoTokenizer, TextStreamer
from intel_extension_for_transformers.transformers import AutoModelForCausalLM, WeightOnlyQuantConfig

# Specify the GGUF repo on the Hugginface
model_name = "TheBloke/Llama-2-7B-Chat-GGUF"
# Download the the specific gguf model file from the above repo
model_file = "llama-2-7b-chat.Q4_0.gguf"
# make sure you are granted to access this model on the Huggingface.
tokenizer_name = "meta-llama/Llama-2-7b-chat-hf"

prompt = "Once upon a time"
tokenizer = AutoTokenizer.from_pretrained(tokenizer_name, trust_remote_code=True)
inputs = tokenizer(prompt, return_tensors="pt").input_ids
streamer = TextStreamer(tokenizer)
model = AutoModelForCausalLM.from_pretrained(model_name, model_file = model_file)
outputs = model.generate(inputs, streamer=streamer, max_new_tokens=300)

Expected Behavior & Potential Risk
N/A
How has this PR been tested?
CI tests.
Dependency Change?
N/A

https://inteltf-jenk.sh.intel.com/job/neural_speed_extension/28/


IOS  getIpAddress    Display under Network.

it look like this

But  under wifi it normal
And how  to get truIpAdress?

It's a promise, you need to use .then()
It will always return your local IP address, this is normal.
To get your external IP, try fetching http://whatsmyip.com or something similar


Add roll damage button to weapons

Invalidated by DMV's new "damage" buttons. Will be fixed in the next release.


PulseAudio: Remove get_latency() caching

Removes the caching on AudioDriverPulseAudio::get_latency() and slightly renames variable name palat to pa_lat to slightly improve readability.
Rationale
As mentioned over at https://github.com/godotengine/godot/issues/38215#issuecomment-753456581 the API call pa_stream_get_latency returns different values at different times. Being able to sample it only once per program execution is a severe handicap when doing any profiling or using it for things such as latency compensation in rhythm games, as it can be wildly out if sampled under some load conditions.
As a somewhat hyperbolic analogy - what if you could only measure frame time once per program execution? What impact would that have on debugging and optimising your game?
Personally, I was about ready to tear apart my initialization routines when I noticed that my output latency number went from ~50ms to ~30ms on runs where I had breakpoints in initialization. Had I not investigated the engine source code I would have wasted considerable time chasing a dead end because of this cached value under the impression that Pulse was giving me conservative settings under certain start conditions!
Knock-on effects
Most of my testing has been on the 3.2 branch (Birdulon/godot/PAlatency) and all I've done on master is verify that it compiles and prints sensible values in a minimal project as I can't port my actual game to master due to unimplemented mesh rendering etc. (made this branch entirely because the default PR text told me to!)
In 3.2 I've observed no degraded performance with the Debugger's Monitors showing Audio Output Latency continuously during gameplay, however I noticed small bits of lag when I also had a label calling it 60 times a second in addition to the monitor. The documentation might need a small note to reflect that AudioServer.get_output_latency() and by extension Performance.get_monitor(Performance.AUDIO_OUTPUT_LATENCY) are potentially costly calls that should be used sparingly.
In master none of the performance monitors seem to draw for me, but I confirmed behavior with a label changing in _process.
I stress again that it should be preferable for this function to return what it says it does, even at cost, even if approximate, rather than lie to the programmer for the sake of saving cycles.

The documentation might need a small note to reflect that AudioServer.get_output_latency() and by extension Performance.get_monitor(Performance.AUDIO_OUTPUT_LATENCY) are potentially costly calls that should be used sparingly.

Could you do that in this pull request (in the same commit)? See this page for instructions on updating the class reference.

The documentation might need a small note to reflect that AudioServer.get_output_latency() and by extension Performance.get_monitor(Performance.AUDIO_OUTPUT_LATENCY) are potentially costly calls that should be used sparingly.

Could you do that in this pull request (in the same commit)? See this page for instructions on updating the class reference.

Thanks for that, for some reason I'd assumed it would be a separate issue in the godot-docs repo! Added.

Thanks for that, for some reason I'd assumed it would be a separate issue in the godot-docs repo! Added.

Is there anything else that should be addressed?

Is there anything else that should be addressed?

The PR is fine but it needs to be reviewed by a contributor who knows about PulseAudio and/or general constraints on what can and can't be done in AudioServer (likely @reduz for the latter).

I like this change, but could we get some performance numbers? How long does this hold the global audio server lock for? Because people will 100% call this every frame and expect it to work. If it takes a millisecond or something I can live with that, but if it's holding the global audio server lock for 10 milliseconds that's not okay because it'll most likely cause buffer underruns.

I haven't run numbers on it, but I was able to play my rhythm game with the performance monitor charting it constantly on the 3.2 PR, without any noticeable audio issues.

Also do you have an idea of how much variation there is in the values returned by this function? If it's +/- 5% or something I'm not really convinced it's necessary to remove caching, but if there are latency spikes occasionally this might be a good change to merge.

As mentioned in the linked comment, it can be pretty severe. I've even seen it go over 100ms during initialisation, likely tied to whatever is happening on the frame it is called.

I don't really think that ripple is worth tracking perfectly, especially because I suspect it's caused by the time to the next mix being different each time get_latency is called. That said, the spike there could cause serious problems. What if the value that we cache in the audio driver is way higher than the average latency because we got unlucky and cached an abnormally high value? We'd just have to wait until the game restarts I guess? Not good.
One thing I'd like to ask you to change is to continue caching the result of the call how we were before, but to clear the cache every time audio is mixed. That way we can continue to track the overall latency trend and give fresh data back, while providing an upper bound on the number of times we do this per frame. Latency isn't likely to change a huge amount between mixes, since unless I'm misunderstanding what it measures, it can't change any faster than time passes.
Also: Make sure to only ever read from or write to the cached value while the global audio server lock is held. We have enough intermittent threading issues as it is, I don't want to introduce more.
Thank you for looking into this by the way! The audio drivers are not my area of expertise so I appreciate people taking the time to contribute patches back to them.

Any updates on this? We ran into this issue with just trying to replicate Ace Attorney blips which we expected to be extremely basic but is giving us hell. The reference video we're using for replicating that is https://www.youtube.com/watch?v=Min0hkwO43g
And in Godot the audio playback ended up being completely random and hard to figure out the culprit other than determining it must be audio buffer and inconsistent latency. We also encountered the issue of being unable to use the provided functions to introduce delay which further makes it impossible to develop any kind of sound-precision on even the basic audio presentation level.

@Crystalwarrior Please don't copy-paste comments across issues, as this makes the discussion more difficult to track for others.
I recommend testing this pull request locally and making it sure it actually fixes the issue for you.

Seems like this is generally positively received, so would be nice to finish it for 4.1 :)

I will note that caching and clearing the cache every mix is still something that I think could be an improvement here because it would be nice if users could call this several times back-to-back without consequences, instead of having to call it once and cache it themselves. Not sure how beneficial that would be and it's a fair amount of extra work. It should be fine to ship this without that.

Sorry I should have mentioned this earlier too, could you squash the commits together? The doc change is directly required by the removal of caching so it makes sense to so both in the same commit.

Squashed

scons target=editor tests=yes dev_build=yes scu_build=yes precision=double use_asan=yes use_ubsan=yes linker=gold succeeded on my machine, is there an intermittent build failure on CI?

It fails after the build when running a test project. That's a known intermittent issue indeed. I restarted the CI job.


Handle newtypes

This PR makes sure that purenix handles newtypes in a sane way.
With this PR, if you have the following PureScript file:
module SemiTest where

newtype Foobar = Foobar Int

mkFoobar :: Foobar
mkFoobar = Foobar 3

mkFoobar2 :: Int -> Foobar
mkFoobar2 = Foobar

useFoobar :: Int
useFoobar =
  case mkFoobar of
    Foobar 100 -> 200
    Foobar x -> x
    Foobar _ -> 300

it gets generated into:
let
  module = { };
  Foobar = x: x;
  mkFoobar2 = Foobar;
  mkFoobar = 3;
  useFoobar =
    let
      __pattern0 = __fail: if mkFoobar == 100 then 200 else __fail;
      __pattern1 = __fail:
        let
          x = mkFoobar;
        in
          x;
      __pattern2 = __fail: 300;
      __patternFail = builtins.throw "Pattern match failure in src/SemiTest.purs at 24:3 - 27:20";
    in
      __pattern0 (__pattern1 (__pattern2 __patternFail));
in
  {inherit Foobar mkFoobar mkFoobar2 useFoobar;}

A couple things to notice here:

Newtype constructors become identity functions.
Newtype applications are removed where possible.  You can see in mkFoobar we are able to remove the Foobar constructor, but in mkFoobar2 we are not.
Newtypes are basically ignored in pattern matches.

This is similar to what PureScript does when generating JS.  For the above PureScript, the following JS is generated:
var Foobar = function (x) {
    return x;
};
var mkFoobar2 = Foobar;
var mkFoobar = 3;
var useFoobar = (function () {
    if (mkFoobar === 100) {
        return 200;
    };
    return mkFoobar;
})();
module.exports = {
    Foobar: Foobar,
    mkFoobar: mkFoobar,
    mkFoobar2: mkFoobar2,
    useFoobar: useFoobar
};

I noticed this when playing around with typeclasses, which appear to be encoded as newtype wrappers around records of functions.
For the following PureScript code:
module SemiTest where

class Semigroupoid a where
  compose :: forall b c d. a c d -> a b c -> a b d
  something :: a Int Int

instance semigroupoidFn :: Semigroupoid (->) where
  compose f g x = f (g x)
  something i = i

blahblah :: Int
blahblah = (compose (\a -> a) (\a -> a)) 3

Before this PR, the following Nix code was generated:

With the change from this PR, the following Nix code is generated:
let
  module = { };
  Semigroupoid-Dict = x: x;
  something = dict:
    let
      __pattern0 = __fail:
        let
          v = dict;
        in
          v.something;
      __patternFail = builtins.throw "Pattern match failure in src/SemiTest.purs at 5:3 - 5:25";
    in
      __pattern0 __patternFail;
  semigroupoidFn =
    { compose = f: g: x: f (g x);
      something = i: i;
    };
  compose = dict:
    let
      __pattern0 = __fail:
        let
          v = dict;
        in
          v.compose;
      __patternFail = builtins.throw "Pattern match failure in src/SemiTest.purs at 4:3 - 4:51";
    in
      __pattern0 __patternFail;
  blahblah = compose semigroupoidFn (a: a) (a: a) 3;
in
  {inherit compose something blahblah semigroupoidFn;}

I noticed this when playing around with typeclasses, which appear to be encoded as newtype wrappers around records of functions.

Ohhhhhhhh yeah I remember seeing something like that, and then I forgot to investigate any further. I'm very glad that this solves that issue, along with just improving newtype handling.


Array example fails

Go to the array example storybook:
http://localhost:6007/?path=/story/node-graph-editor--array-inputs
Delete the edges. Try to re-add them. Only one will connect. Connecting another will delete the first one.

Can you take a look at my PR and let me know if you have any concerns?

Ah, yeah, looks good. I was looking in the wrong place.


[0.9.x] Add int8 command

Description
(Brief description on what this PR is about)
Checklist
Essentials

[ ] PR's title starts with a category (e.g. [BUGFIX], [MODEL], [TUTORIAL], [FEATURE], [DOC], etc)
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage
[ ] Code is well-documented

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

cc @dmlc/gluon-nlp-team

Job PR-1174/2 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-1174/2/index.html


Add MulticlassRecallResult

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

Project coverage is 100.00%. Comparing base (295e5e7) to head (d685226).
Report is 7 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #170   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           58        58
  Lines         1345      1353    +8
  Branches       143       143
=========================================
+ Hits          1345      1353    +8

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


Add clipboard permissions

Why it is needed?
iOS16 will ask permission to access clipboard - https://9to5mac.com/2022/06/06/ios-16-permission-copy-paste-between-apps/
Possible implementation
iOS only permissions which return truthy or falsy based on user input
Code sample
useEffect(() => {
    async function getClipboardPermission() {
      const result = await checkPermission(CLIPBOARD_PERMISSION)
      if (result === RESULTS.BLOCKED || result === RESULTS.UNAVAILABLE) {
        hasClipboardAccess(false)
      }
    }

    getClipboardPermission()
  }, [])

Closing this as clipboard permissions are not persistent and need to be requested each and every time.


Booster Packs Allow duplicates

The current booster pack generate allows duplicate non-foil cards in the the booster pack.

closed as of #13


[🐞] SPA reload randomly fails to update the values loaded by the routeLoader$

Which component is affected?
Qwik City (routing)
Describe the bug
I am making app that uses routeLoader$ for loading dynamic data.
I found the SPA reload sometimes fails to update the data like this.
https://github.com/user-attachments/assets/a88f5efb-625d-4b67-8c1f-c87978e3cc29
Note that the 3rd reload (around 0:07 at the video) trigger the call of the route loader and logged the count at server side but doesn't change the values in client side.
The next click updates the values but doesn't trigger the call.
I expected the each reload can update the value.
Reproduction
https://github.com/genki/qtest
Steps to reproduce
Please clone the repo above and
pnpm i
pnpm run dev

then open http://localhost:5173/
System Info
System:
    OS: macOS 14.5
    CPU: (8) arm64 Apple M2
    Memory: 117.77 MB / 24.00 GB
    Shell: 3.6.1 - /opt/homebrew/bin/fish
  Binaries:
    Node: 20.12.1 - /opt/homebrew/opt/node@20/bin/node
    Yarn: 1.22.19 - /opt/homebrew/bin/yarn
    npm: 10.5.0 - /opt/homebrew/opt/node@20/bin/npm
    pnpm: 9.6.0 - ~/Library/pnpm/pnpm
    bun: 1.1.21 - ~/.bun/bin/bun
  Browsers:
    Chrome: 127.0.6533.101
    Safari: 17.5
  npmPackages:
    @builder.io/qwik: ^1.8.0 => 1.8.0
    @builder.io/qwik-city: ^1.8.0 => 1.8.0
    typescript: 5.4.5 => 5.4.5
    undici: * => 6.19.5
    vite: ^5.2.10 => 5.3.5

Additional Information
Interestingly, this behaviour doesn't occur at the stackbliz.
https://stackblitz.com/edit/github-ahtxgw?file=src%2Froutes%2Findex.tsx

This issue is not only when the reload but for when SPA navigation too.

As the @auth/qwik using the routeLoader$ inside to load the session information, this issue causes a vulnerabilities.
If an user has done signOut and navigated to another page by SPA navigation, the old session data randomly remains. So another user can see the previous user's private data if they share the same machine.

I found that the problem is solved if I checked the "Disable cache" at the devtools.
It means the dynamic q-data.json is unexpectedly cached (besides, it happens randomly).
The reason of this issue may be around the loadClientData

<EMAIL_ADDRESS>"^1.9.0",
<EMAIL_ADDRESS>"^1.9.0",
This issue has arisen again.
q-data.json is unexpectedly cached
the problem is solved if checked the "Disable cache" at the devtools.

@lontten can you check the headers that the browser received for the q-data.json? And please specify your environment, SSG or not, anything that could impact the caching

access-control-allow-origin:
*
cache-control:
max-age=5, stale-while-revalidate=604800
content-type:
application/json; charset=utf-8
date:
Fri, 04 Oct 2024 12:58:18 GMT
server-timing:
useServerTimeLoader;dur=0.2779999999984284,useBook;dur=92.89895899999829

System:
OS: macOS 14.6.1
CPU: (8) arm64 Apple M1
Memory: 95.77 MB / 8.00 GB
Shell: 5.9 - /bin/zsh
Binaries:
Node: 20.17.0 - ~/.nvm/versions/node/v20.17.0/bin/node
npm: 10.8.2 - ~/.nvm/versions/node/v20.17.0/bin/npm
pnpm: 9.11.0 - ~/.nvm/versions/node/v20.17.0/bin/pnpm
bun: 1.1.29 - ~/.bun/bin/bun
Browsers:
Edge: 129.0.2792.79
Safari: 18.0
env:  bun run dev

https://github.com/user-attachments/assets/412656b4-8f72-42ac-8885-82c43b13ac56

@lontten cache-control: max-age=5 tells your browser to cache it...

@wmertens
why cache 5s
doc show
Route Loaders are executed on the server, after every navigation. This means that they are executed every time a user navigates to a page in an SPA or MPA, and they are executed even if the user is navigating to the same page.
they are executed every time.

I found the cache cofig is layout.tsx > onGet > cacheControl
web page show the cache
but, the routeLoader$  still be executed,The caching feature in routeLoader is meaningless and also results in error displays.

@lontten it's not useless, it depends on the use case. Sometimes you need caching, sometimes not

I mean, when  caching, the routeLoader$ still called the API and obtained the data, so it didn't reduce the pressure on the API server.

Ah that's because of stale-while-revalidate=604800. So the bug is that qwik should somehow detect this and use the new data when it comes in. Can you open an issue?

Qwik could fetch from cache only first and then with default settings. If the data changes it will update reactively


Blackpill flashing process completes, but device does not appear.

I have followed the very clear flashing guide for an STM32F401CCU6, on Windows 10, using v6.2.4  blackpill401.bin
The experience goes exactly as described in the guide, except, that after the final message "Transitioning to dfuMANIFEST state", no new device appears in the Device Manager\Sounds   After flashing I hear the USB device disconnection sound and 'STM32 BOOTLOADER' in the 'libusbK USB Devices' section disappears, but I do not hear a connection sound.  If I unplug or reconnect the device, I do not hear any sound notifications and devices do not change.
I have also tried v6.2.3 with the same result.

ID 0483:df11
Run-time device DFU version 011a
Claiming USB DFU Interface...
Setting Alternate Setting #0 ...
Determining device status: state = dfuERROR, status = 10
dfuERROR, clearing status
Determining device status: state = dfuIDLE, status = 0
dfuIDLE, continuing
DFU mode device DFU version 011a
Device returned transfer size 2048
DfuSe interface name: "Internal Flash  "
Downloading to address = 0x08000000, size = 239192
Download        [=========================] 100%       239192 bytes
Download done.
File downloaded successfully
Transitioning to dfuMANIFEST state
I have tried two different Blackpill devices, same model.
Any ideas?

Weird. I have the same model hooked up on my hardware test rig and everything works just fine. Would it be possible for you to try the same thing in Ubuntu?

Oh I figured it. You've hit the joy of having almost-the-same-but-then-not-quite variant of the STM32F4 MCU.
The board I have features F401CE. You have F401CC. The difference is the amount of RAM and flash - yours has 256kB of flash as opposed to 512kB on CE variant, so flash pages aren't correctly generated. The board goes to hard fault mode straight after flashing most likely. I'll add a new variant later today so you can flash that.

Ah, thank you that would be ace.
I was about to pull an old laptop out to install Ubuntu.  However out of curiosity, I tried it on an Mac Air M1, and got the same result.  Slightly later version of dfu-util, which does provide hints that all is not well.
Opening DFU capable USB device...
Device ID 0483:df11
Device DFU version 011a
Claiming USB DFU Interface...
Setting Alternate Interface #0 ...
Determining device status...
DFU state(10) = dfuERROR, status(10) = Device's firmware is corrupt. It cannot return to run-time (non-DFU) operations
Clearing status
Determining device status...
DFU state(2) = dfuIDLE, status(0) = No error condition is present
DFU mode device DFU version 011a
Device returned transfer size 2048
DfuSe interface name: "Internal Flash  "
Downloading element to address = 0x08000000, size = 239192
Erase           [=========================] 100%       239192 bytes
Erase    done.
Download        [=========================] 100%       239192 bytes
Download done.
File downloaded successfully
Submitting leave request...
Transitioning to dfuMANIFEST state
I had to install homebrew to get dfu-util, using this command.
/bin/bash -c "$(curl -fsSL https://raw.githubusercontent.com/Homebrew/install/HEAD/install.sh)"

dfu-util is mentioned in packages to install. ;)
I've attached test binary for F401CC. Can you try it? I don't have that board so I can't test it. Note that only 1 preset is supported on that board.
blackpill401cc.zip

dfu-util is mentioned in packages to install. ;)
Yes, I was just leaving the command for other M1 users to install homebrew :)

OK, well the new binary seems to work a treat.  Flashed, and restarted, appears as in Device Manager as described in the guide.
The offline OpenDeck UI detects the device!
Thank you, what wonderful support, and what an interesting project.  I spent many weeks trying to write my own with Arduino, I'm looking forward to converting the thing I made to work with OpenDeck.
I will get something attached to properly test later today.

Awesome. The CC variant will be part of the new release in a day or so, so you can grab future updates from GitHub or update through web tool.

Yep, I tested a simple analog pot and mapped it through to VCV Rack, works very well.  Thank you!!

Solved with:
https://github.com/shanteacontrols/OpenDeck/commit/c85396f1bd41f26abb84ec2b7a5f16964b6d629d
https://github.com/shanteacontrols/OpenDeck/commit/84b673b20a10bbf4b87676b4de15062ba15d840f

Added to the new release:
https://github.com/shanteacontrols/OpenDeck/releases/tag/v6.3.0


ONGL Expression can't support char '}' ecscape

in org.apache.ibatis.parsing.GenericTokenParser class
This class only support char '{' ecscaping
char '}' can't escape by using ''

You don't need to escape '}'.
There are test cases covering various patterns.
https://github.com/mybatis/mybatis-3/blob/master/src/test/java/org/apache/ibatis/parsing/GenericTokenParserTest.java
If you think there is a missing pattern, please show us an example.

When calling method with parameter of List type,The conflict will occur
such as expression:
"${utils.concat({"aa","bb","cc"})}"

${} is used in a SQL statement as a string replacement and it does not take OGNL expression.
It takes a parameter expression like ${id} or ${bean.prop}.
OGNL expression can be used in test attribute of if / when elements and value attribute of bind element, but the expression is not enclosed by ${} and GenericTokenParser is not used to parse it.
<if test="bean.prop != null">

<bind name="pattern" value="'%' + _parameter.getTitle() + '%'" />

The bind element allows you to assign the result of an OGNL expression to a variable and this might be what you are looking for.
https://mybatis.github.io/mybatis-3/dynamic-sql.html#bind
If it is not, please create a test case so that I can understand what you are trying to achieve.
Thank you,
Iwao

I might have been wrong about ${} does not take OGNL expression...I'll get back to you later.

I didn't know that, but the ${} can take OGNL expression and there should be a way to escape '}' character as you reported.
I have already committed the change.
Please try the latest snapshot and let me know if it works as you expect.
Thank you!
Iwao


Finish package validation

part of #307

[ ] check if verification code is required when files_analyzed = True and make check for it
[ ] write tests for package verification code validation
[ ] write tests for "no CONTAINS or CONTAINED_BY relationships if files_analyzed = False"

After some research, I haven't found evidence that the verification code is mandatory if files_analyzed = True.
There's an open issue on the topic here: https://github.com/spdx/spdx-spec/issues/802

fixed by #461

fixed by #461


LVS mismatch errors in SKY130 SRAM

I am encountering LVS mismatch issues while generating a single-port SRAM using the Sky130 technology.
Could the problem be related to num_spare_col and num_spare_row in our config? How can I address and resolve this issue?
Another issue I'm facing is that the spare_wen pin is not in the lib file.
Also, the power values in the lib file appear to be excessively large, around 10^9.
Can you provide guidance on how to address this?
My OpenRAM version is v1.2.45.
|==============================================================================|
|=========                      OpenRAM v1.2.42                       =========|
|=========                                                            =========|
|=========               VLSI Design and Automation Lab               =========|
|=========        Computer Science and Engineering Department         =========|
|=========            University of California Santa Cruz             =========|
|=========                                                            =========|
|=========          Usage help<EMAIL_ADDRESS>         =========|
|=========        Development help<EMAIL_ADDRESS>      =========|
|=========                See LICENSE for license info                =========|
|==============================================================================|
** Start: 12/18/2023 02:25:16
Technology: sky130
Total size: 128 bits
Word size: 8
Words: 16
Banks: 1
RW ports: 1
R-only ports: 0
W-only ports: 0
DRC/LVS/PEX is only run on the top-level design to save run-time (inline_lvsdrc=True to do inline checking).
Characterization is disabled (using analytical delay models) (analytical_delay=False to simulate).
Only generating nominal corner timing.
Words per row: None
Output files are:
/home/user/OpenRAM/result/sram_1rw_8_16_sky130/sram_1rw_8_16_sky130.lvs
/home/user/OpenRAM/result/sram_1rw_8_16_sky130/sram_1rw_8_16_sky130.sp
/home/user/OpenRAM/result/sram_1rw_8_16_sky130/sram_1rw_8_16_sky130.v
/home/user/OpenRAM/result/sram_1rw_8_16_sky130/sram_1rw_8_16_sky130.lib
/home/user/OpenRAM/result/sram_1rw_8_16_sky130/sram_1rw_8_16_sky130.py
/home/user/OpenRAM/result/sram_1rw_8_16_sky130/sram_1rw_8_16_sky130.html
/home/user/OpenRAM/result/sram_1rw_8_16_sky130/sram_1rw_8_16_sky130.log
/home/user/OpenRAM/result/sram_1rw_8_16_sky130/sram_1rw_8_16_sky130.lef
/home/user/OpenRAM/result/sram_1rw_8_16_sky130/sram_1rw_8_16_sky130.gds
** Submodules: 2.9 seconds
** Placement: 0.0 seconds
** Routing: 158.3 seconds
ERROR: file magic.py: line 385: sram_1rw_8_16_sky130       LVS mismatch (results in /tmp/openram_root_24133_temp/sram_1rw_8_16_sky130.lvs.report)
** Verification: 169.8 seconds
** SRAM creation: 331.1 seconds
SP: Writing to /home/user/OpenRAM/result/sram_1rw_8_16_sky130/sram_1rw_8_16_sky130.sp
** Spice writing: 0.1 seconds

@KEVIN09876 Can you share sram_1rw_8_16_sky130.lvs.report ?


fix(Button|Menu|List|Step): fix behaviour of onClick when is disabled

Fixes #1964.
Affects:

Button
ListItem
MenuItem
Step

We have disabled prop also in:

FormField - I don't touch because it doesn't have onClick handler and I'm not sure that it will be correct behaviour
TableCell/TableRow - I think we can update the behaviour there, but these components don't have onClick handler
Icon, Image, Loader, Reveal, Segment, Progress - These components don't have onClick handler and I'm not sure that we need to change there something
Checkbox, Dropdown, Rating - already handles disabled
Dimmer - disabled has there another context

Codecov Report

Merging #2006 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff           @@
##           master   #2006   +/-   ##
======================================
  Coverage    99.8%   99.8%
======================================
  Files         148     148
  Lines        2589    2589
======================================
  Hits         2584    2584
  Misses          5       5

Impacted Files
Coverage Δ

src/elements/Step/Step.js
100% <100%> (ø)
:arrow_up:

src/elements/List/ListItem.js
100% <100%> (ø)
:arrow_up:

src/collections/Menu/MenuItem.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Let's merge this as it is a great step forward.
We may need to double back and add onClick handlers for the others you've mentioned, but let's see what the community response is before investing the time.  Also, the CSS for many disabled classes includes pointer-events: none which effectively does the same thing.

Released in<EMAIL_ADDRESS>


Make Missed Call Line Background Color Dark

This Red Color Defined is not clear In VS Code Editor
Signed-off-by: FANG.Ge<EMAIL_ADDRESS>

Can't test it right now, can you add screenshots that show before and after, please? Preferably also with different themes.

i tried to do that, but speed limit makes image upload always fail.

@xiaoyun94: Did you tried to save the file as PNG? In any case it would be enough to only upload partial images (some lines), not the complete screen - and as PNG that should be below 100KB.

@xiaoyun94 Could you please rebase and possibly try to upload before/after images again?


when will the FusedChat be updated here?

Nice work! and I am wondering when you will upload the FusedChat dataset. Look forward to it. Thanks a lot!!

Asap
On Wed, 15 Sep 2021, 16:47 lytum, @.***> wrote:

Nice work! and I am wondering when you will upload the FusedChat dataset.
Look forward to it. Thanks a lot!!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tomyoung903/FusedChat/issues/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKQ3UYAONDIYGAKQXXCYVPDUCBMT3ANCNFSM5EB5QB2Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Nice work! and I am wondering when you will upload the FusedChat dataset. Look forward to it. Thanks a lot!!

Hi, the dataset is released now. Any questions can be asked here.

Thanks a lot. Could you share some infos about split dataset 'train/validation/test'?

An update to the repo is about to be released within about a week. The info
on partitions will be shared at the same time.
Cheers,
Tom Young
PhD Student
School of Computer Science and Engineering (SCSE)
Nanyang Technological University (NTU)
50 Nanyang Ave, Singapore 639798
tomyoung903.github.io
On Wed, Feb 16, 2022 at 6:19 PM lytum @.***> wrote:

Thanks a lot. Could you share some infos about split dataset
'train/validation/test'?
—
Reply to this email directly, view it on GitHub
https://github.com/tomyoung903/FusedChat/issues/1#issuecomment-1041331492,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKQ3UYEVN2AHEUE42CIUUHTU3N24ZANCNFSM5EB5QB2Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you commented.Message ID:
@.***>

An update to the repo is about to be released within about a week. The info on partitions will be shared at the same time. Cheers, Tom Young PhD Student School of Computer Science and Engineering (SCSE) Nanyang Technological University (NTU) 50 Nanyang Ave, Singapore 639798 tomyoung903.github.io
…
On Wed, Feb 16, 2022 at 6:19 PM lytum @.> wrote: Thanks a lot. Could you share some infos about split dataset 'train/validation/test'? — Reply to this email directly, view it on GitHub <#1 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKQ3UYEVN2AHEUE42CIUUHTU3N24ZANCNFSM5EB5QB2Q . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub. You are receiving this because you commented.Message ID: @.>

Thanks the reply form the author. May I ask when there will be updates here? Thanks a lot and really appreciate!


dotbim and stl

This is a very interesting project. I am very interested in how this can be taken forward in context of 3d model to simulations. Therefore my question is are there converters for dotbim to stl or something which can be used for meshing in fem solver? I can imagine this could be straightforward as both format stores meshes

Hey @raviapatel!
The simplest solution if you'd like to convert to .stl is to use https://3dviewer.net/ exporter. You can easily update .bim file and export .stl from there to use it for meshing in another FEM software.

Let me know what you think and the results of your tests :)

ok i see any solutions with python ??? which can be integerated directly in workflows?

I haven't created any .bim -> .stl Python scripts / converters, but I guess they should be relatively easy to implement. Check out this example with numpy-stl: https://micronote.tech/2020/12/Generating-STL-Models-with-Python/

Also you can check a trimesh library, that has some .stl capabilities: https://trimsh.org/trimesh.exchange.stl.html
There is some example of .bim -> trimesh, so maybe this one would be interesting for you: https://github.com/paireks/dotbimpy/blob/master/dotbimpy/other/DotbimToTrimeshScene.ipynb


Allow to take a link to line number in comparison of files

Gitea version (or commit ref): 7ea4bfc
Can you reproduce the bug at https://try.gitea.io:

[x] Yes
[ ] No
[ ] Not relevant

I am using gitea inside Docker from gitea/gitea. The relevant part of docker inspect gitea/gitea:latest output is:
{
    "Id": "sha256:f50274d2fddab78c52ef199c6d42b06af6defd3e5fcfc890d46618a1546a51c0",
    "RepoTags": [
        "gitea/gitea:latest"
    ],
    "RepoDigests": ["gitea/gitea@sha256:0367cde3c4e6041e0e63042f22e4787edd441e42976a64d698e8b9387f72dd75"],
    "Parent": "",
    "Comment": "",
    "Created": "2018-04-29T06:50:27.057796146Z"
    /** ... */
}

Description
When have a file open in gitea you can click the line number to get a direct link to that line, but if you have a comparison of two files (both in unified or split view) you can't even click the line number.
Is it possible to implement a link to line number in compare mode? I think it would be great if the link goes to line in the comparison itself, but if it points just to the line in each file at the right commit could be ok, thought.
Since we don't have yet the possibility of comment in changes of pull requests at least a link to file at a particular line could be very useful, as when you click in "Files changed".
Sreenshots
Here you can see the cursor change and click gives me to that line in that file in that particular commit:

When comparing two files you can't click line number of any file:

For instance, GitHub gives you a link to that line in the comparison: see here

bump

Is this possible yet?

Not yet right now. Contributions are welcome.


Automically adjust indentation

Is your feature request related to a problem? Please describe.
Can we automically adjust the indentation for Bend syntax, does not force to 2 spaces
Describe the solution you'd like
What happen now is Bend force us to use 2 spaces
fzn0x@DESKTOP-69TAUPC:/mnt/d/strassen$ bend run test.bend
Errors:
In test.bend :
Indentation error. Expected 2 spaces, got 4.
  2 |     return 2 + 3

fzn0x@DESKTOP-69TAUPC:/mnt/d/strassen$ bend run-c test.bend
Errors:
In test.bend :
Indentation error. Expected 2 spaces, got 4.
  2 |     return 2 + 3

fzn0x@DESKTOP-69TAUPC:/mnt/d/strassen$ bend run-c test.bend
Errors:
In test.bend :
Indentation error. Expected 2 spaces, got 1.
  2 |  return 2 + 3

Let adjust the indentation so it's not expecting 2 spaces
Describe alternatives you've considered
Adding Bend to Prettier? https://prettier.io/docs/en/plugins.html
Additional context

Yes, this is planned.


Constant reloading of the whole rule set every 30 seconds causing disruption

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you do?
I am using traefik in a docker environment (not swarm) using proper labels for the containers that need to be exposed. For TLS certificates I am using a combination of lets encrypt public certificates for certain service and static certificates for others.
All is working as expected except that the rule set is completely rebuilt every 30 seconds
What did you see instead?
Because of the constant rebuild, if a request comes at a bad moment, the default certificate is provided instead of the lets encrypt or static one. I also suspect that some requests are actually simply ignored / rejected (but this is speculative).
The end effect is obviously that my service is not reliable.
What version of Traefik are you using?

# The Traefik dashboard
api:
  dashboard: true
  debug: true
  insecure: true

log:
  level: DEBUG
  format: common

# Define an HTTPS entrypoints
entryPoints:
  web:
    address: ":80"
  secure:
    address: ":443"

providers:
  file:
    directory: /datastore/loadBalancer
    watch: true

  docker:
    endpoint: "unix:///var/run/docker.sock"
    watch: true
    exposedbydefault: false
    network: "thor_default"
    swarmMode: false
    swarmModeRefreshSeconds: 30

# Static TLS files
tls:
  certificates:
  - certFile: /datastore/certificates/ap.crt
    keyfile: /datastore/certificates/ap.key
  - certFile: /datastore/certificates/api.crt
    keyfile: /datastore/certificates/api.key
  - certFile: /datastore/certificates/charon.crt
    keyfile: /datastore/certificates/charon.key
  - certFile: /datastore/certificates/portunus.crt
    keyfile: /datastore/certificates/portunus.key
  options:
    default:
      minVersion: VersionTLS12

    mintls13:
      minVersion: VersionTLS13

certificatesResolvers:
  letsEncryptResolver:
    acme:
      email: xxxxxxxxxx
      storage: /datastore/loadBalancer/acme.json
      httpChallenge:
        # used during the challenge
        entryPoint: web

Add more configuration information here.
If applicable, please paste the log output in DEBUG level
Example of lines that appears every ~30 sec
time="2022-09-28T12:12:10Z" level=debug msg="Configuration received: {"http":{},"tcp":{},"udp":{},"tls":{"options":{"default":{"minVersion":"VersionTLS12","clientAuth":{},"alpnProtocols":["h2","http/1.1","acme-tls/1"]},"mintls13":{"minVersion":"VersionTLS13","clientAuth":{},"alpnProtocols":["h2","http/1.1","acme-tls/1"]}}}}" providerName=file
time="2022-09-28T12:15:14Z" level=debug msg="Adding certificate for domain(s) xxxxx"
time="2022-09-28T12:49:37Z" level=debug msg="Adding route for xxxxxx with TLS options default" entryPointName=secure

Found the problem, on my side. Sorry. I am closing it


ASR Bypass Network Placement

Hi Team,
The bypass network isn't clear on where it needs to be placed.
Should virtual network be placed within the Primary Network and ASR be setup in the Failover i.e UK South Primary and UK West failover. Or, should the bypass network be setup within the failover network along with ASR?
If the diagram could state Primary and Failover, that would be really helpful to future users.

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

ID: 603107b4-2419-0477-a677-c7c8c64073fe
Version Independent ID: 6266bf3d-682e-ac71-2d83-28ed47df8a60
Content: Enable replication for on-premises machines with private endpoints - Azure Site Recovery
Content Source: articles/site-recovery/hybrid-how-to-enable-replication-private-endpoints.md
Service: site-recovery
GitHub Login: @ankitaduttaMSFT
Microsoft Alias: ankitadutta

@Insight-UK-CurtisMilne Thanks for your comment! We will review and provide an update as appropriate.

Thanks @KrishnaG-MSFT  !
Please let me know if I can help. Happy to redesign the diagram and submit for your approval.

Hi there,
Has any progress been made on an updated diagram. It would also be useful to clarfiy what the recommmendation is for the placement of the bypass network with respect to the Azure Landing Zone model. It is recommended to place this 'bypass network'  in a a new spoke vNET Peered to the Hub or a subnet within the hub vNET?
Thanks

Hi @gerrynicol
It doesn't look like there has been an update so far.
However during my build, I placed the bypass network in a spoke network connecting into the Hub. My philosophy behind this is that the bypass network is only providing a service for ASR. Therefore, it would not require the endpoints to be within a central location such as firewall, bastion or hybrid connectivity.
Control routing from on-premise into hub and allow firewall to control route ASR traffic from hub to spoke.
I hope that helps in some way 😁

Hi @Insight-UK-CurtisMilne,
Thanks for taking the time out to reply here :). I agree that that is likely the best solution for the bypass network and was the design I was going to follow.

No problem at all @gerrynicol
Quick tip: if you are going to be using the endpoint setup for both the vault and the cache log in storage accounts, be sure to enter FQDN's for both the vault and storage container into your on-premise DNS forwarders. This caught me out 😅

Thanks @Insight-UK-CurtisMilne. Yes, afaiks, the document doesnt mention the DNS resolution required from on-prem either. Out of interest, what did you foward to? Did you use a private DNS resolver or use the firewall as a DNS proxy. Or something else?

For on-premise DNS, I used DNS on the on-premise DC. I made an entry for each private endpoint linking the private ip's to the associated fqdns in microsoft azure. From there, on-premise routing takes over as it knows the particular range hosted in azure. Traffic travels from the VMware VM > Config Server > Firewall > ExpressRoute > VWAN (Hub) > Spoke (via peer)

Adding @shubhamsangal-msft to help with this.

Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. The requested updates have not been made since the creation of this issue, and the timeline for resolution may vary based on resourcing, so we've created an internal work item to incorporate your suggestions. We are closing this issue for now, but feel free to comment here as necessary. #please-close


[expat] Ensure dllimport

For https://github.com/microsoft/vcpkg/issues/35878#issuecomment-2034823675.

CC @AlessandroScrem @frabera

I actually encounter the same error as before #37976 (comment), the patch proposed by @AlessandroScrem #37956 worked.

Double check that you test the expat from this PR, and that your project doesn't set definitions inappropriately. With this PR, the installed include/expat_external.h for dynamic linkage has (check it!):
#if ! defined(XML_STATIC) && ! defined(XMLIMPORT)
#  ifndef XML_BUILDING_EXPAT
/* using Expat from an application */

#    if defined(_WIN32) && ! defined(__BEOS__) && ! defined(__CYGWIN__)
#      define XMLIMPORT __declspec(dllimport)
#    endif

#  endif
#endif /* not defined XML_STATIC */

which translates to
#if ! 0 && ! 0
#  if 1
/* using Expat from an application */

#    if 1 && ! 0 && ! 0
#      define XMLIMPORT __declspec(dllimport)
#    endif

#  endif
#endif /* not defined XML_STATIC */

which translates to
#      define XMLIMPORT __declspec(dllimport)

which has the effect of
-#define XMLPARSEAPI(type) XMLIMPORT type XMLCALL
+#define XMLPARSEAPI(type) __declspec(dllimport) type XMLCALL

Thanks for the fix; /Za is just pain and it hurts every time people ask for the extra bugs mode but at least they can get something that works.


Minor edit to Quickstart docs, code block formatting

The markdown for the GetUsers() code block was a little awry with the code not appearing in a code block.
This change rectifies the issue.

Hi @naeemsarfraz, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

The problem code snippet from Protecting an API using Passwords

thanks!


AKRenderTapAKLazyTap3rdTry

Third time is the charm!  Sorry about all the churn here, I think this one will stick.
AKRenderTap is now initialized with it's render block, which simplified the class greatly.
AKLazyTap now has an AKRenderTap as an ivar rather than inheriting from it.

I will want to see this new stuff in action when I am back from my vacation!


NAS-132055 / 25.04 / Fix zfs get acl unit test

Context
Fix failing zfs unit test in ixdiagnose.

Jira URL: https://ixsystems.atlassian.net/browse/NAS-132055

Jira URL: https://ixsystems.atlassian.net/browse/NAS-132055

This PR has been merged and conversations have been locked.
If you would like to discuss more about this issue please use our forums or raise a Jira ticket.


Fix for the tree-view context-menu in Atom ~1.17.0

In a recent Atom update a new panel functionality was introduced. As part of this the tree-view was placed into one of these new panels. When that happened tree-view had some updates to it that broke pathing and the method to get to tree-view's serialize function has changed. This update fixes those.

If this needs to be backwards compatible with older version of Atom let me know I can fix that up.

Great thanks! My stable version of atom was upgraded automatically so I didn't get a chance to verify it. I'm going to bump the engine version in the package.json so atom won't allow updating the package if on a version lower than 1.17.


Dowload newst version if only browser name is given on windows

if swapping bits is needed and there is no version provided by the
config, use the newest one instead of the first element in the list

Great! I'll review this and merge this ASAP

Sorry for the delay here. Been really bussy!


Feature: Grid lines and snap to grid

This adds a new grid set of options that can be used to render grid lines underneath the graph.
The grid has main grid lines and inner grid lines. Snapping to the grid when dragging nodes can be turned on/off.
Grid options:

renderGridLines: Enable/disable grid lines
snapToGrid: Whether or not to snap to grid lines
gridWidth: Distance between each main grid line on the X axis
gridHeight: Distance between each main grid line on the Y axis
strokeWidth: How thick each main grid line is
strokeColor: Color of main grid lines
innerGridXDivisions: How many divisions to make between each major grid line on the X axis
innerGridYDivisions: How many divisions to make between each major grid line on the Y axis
innerGridStrokeWidth: How thick the inner grid lines are
innerGridStrokeColor: Color of the inner grid lines

Using these combined with listening to changes to zoom lets you do really cool stuff when using react-d3-graph, like changing thickness/number of grid lines with the zoom level!

https://user-images.githubusercontent.com/504259/122706725-65d81580-d20d-11eb-9bcd-f1de90c11ce6.mp4
https://user-images.githubusercontent.com/504259/123028912-0f95de80-d395-11eb-81a1-6e033b06bf78.mp4

Closing this since it seems like this project is no longer maintained 😢


fix: git config fixes

Description
Git config fixes
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Manually

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Test coverage results :test_tube:
:white_circle: Total coverage has not changed
// Code coverage diff between base branch:release and head branch: git-config-updates

Status
File
% Stmts
% Branch
% Funcs
% Lines

:red_circle:
app/client/src/reducers/uiReducers/gitSyncReducer.ts
11.32 (0)
0 (-100)
0 (0)
11.32 (0)

:red_circle:
app/client/src/selectors/gitSyncSelectors.tsx
71.43 (-0.11)
26.19 (0)
15.91 (-0.37)
62.65 (-0.31)

/ok-to-test sha=2cead4e


fix deprecated

Hi
Our group in LSST is exploring your VAE for deblending.
I found these error messages running the tutorial.
data_folder_path = pkg_resources.resource_filename('debvader', "data/")

field_img = np.load(data_folder_path + '/dc2_imgs/field/field_img.npy', mmap_mode = 'c') # DC2 field image
cutout_galaxy_img = np.load(data_folder_path + '/dc2_imgs/field/galaxies_from_field.npy', mmap_mode = 'c') # stamps from galaxies below a magnitude of 24.5 in the r-band
field_galaxies_distances_from_center = np.load(data_folder_path + '/dc2_imgs/field/ra_dec_coord.npy', allow_pickle=True) # Distances from center of the different galaxies from object catalog (in pixels). Mag cut at 24.5
all_field_galaxies_distances_from_center_truth = np.load(data_folder_path + '/dc2_imgs/field/gal_coordinates_complete_truth_catalog.npy', allow_pickle=True) # Distances from center of the field for the galaxies from the DC2 truth catalog (in pixels). Mag cut at 26

field_img_saving_iter = field_img.copy()

---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
Cell In[15], line 1
----> 1 data_folder_path = pkg_resources.resource_filename('debvader', "data/")
      3 field_img = np.load(data_folder_path + '/dc2_imgs/field/field_img.npy', mmap_mode = 'c') # DC2 field image
      4 cutout_galaxy_img = np.load(data_folder_path + '/dc2_imgs/field/galaxies_from_field.npy', mmap_mode = 'c') # stamps from galaxies below a magnitude of 24.5 in the r-band

AttributeError: module 'importlib.resources' has no attribute 'resource_filename'

-------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
Cell In[43], line 1
----> 1 field_img = np.load(data_folder_path+'/dc2_imgs/field/field_img_3.npy', mmap_mode = 'c') # DC2 field image
      2 all_field_galaxies_distances_from_center_truth = np.load(data_folder_path+'/dc2_imgs/field/gal_coordinates_complete_truth_catalog_3.npy', allow_pickle=True) # Distances from center of the field for the galaxies from the DC2 truth catalog (in pixels). Mag cut at 26
      3 field_center = np.load(data_folder_path+'/dc2_imgs/field/field_center_3.npy')

TypeError: unsupported operand type(s) for +: 'PosixPath' and 'str'

field_img = np.load(data_folder_path + '/dc2_imgs/field/field_img_2.npy', mmap_mode = 'c') # DC2 field image
all_field_galaxies_distances_from_center_truth = np.load(data_folder_path+'/dc2_imgs/field/gal_coordinates_complete_truth_catalog_2.npy', allow_pickle=True) # Distances from center of the field for the galaxies from the DC2 truth catalog (in pixels). Mag cut at 26
field_center = np.load(data_folder_path+'/dc2_imgs/field/field_center_2.npy')

IterativeDeblender = IterativeDeblendField(net, field_img, epistemic_uncertainty_estimation=False)
res_iterative_procedure = IterativeDeblender.iterative_deblending(galaxy_distances_to_center=None,
                                              optimise_positions=True,
                                              epistemic_criterion= 20,
                                              mse_criterion=0.5)
res_iterative_procedure_meta = IterativeDeblender.get_deblending_meta_data()

--------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
Cell In[49], line 1
----> 1 field_img = np.load(data_folder_path + '/dc2_imgs/field/field_img_2.npy', mmap_mode = 'c') # DC2 field image
      2 all_field_galaxies_distances_from_center_truth = np.load(data_folder_path+'/dc2_imgs/field/gal_coordinates_complete_truth_catalog_2.npy', allow_pickle=True) # Distances from center of the field for the galaxies from the DC2 truth catalog (in pixels). Mag cut at 26
      3 field_center = np.load(data_folder_path+'/dc2_imgs/field/field_center_2.npy')

TypeError: unsupported operand type(s) for +: 'PosixPath' and 'str'

These changes helped to run the whole notebook
Path Handling: Replaced pkg_resources.resource_filename with importlib.resources for Python 3.9+.
Fixed Type Errors: Used PosixPath syntax (/ instead of +) for path joining to prevent TypeError issues.
Compatibility: Reduced reliance on deprecated libraries.
Hope it helps

This conda config worked for tensorflow with GPU:
conda create -n debvader python=3.11

conda activate debvader

conda install jupyter

conda install -c anaconda ipykernel

pip install --upgrade typing_extensions

python -m ipykernel install --user --name=debvader

pip install -r requirements.txt

pip install --editable .

conda install -c conda-forge cudatoolkit=11.8 cudnn=8.8

pip install tensorrt

echo 'export LD_LIBRARY_PATH=$LD_LIBRARY_PATH:/home/parlange/anaconda3/envs/debvader/lib/python3.11/site-packages/tensorrt' > $CONDA_PREFIX/etc/conda/activate.d/env_vars.sh

tensorrt still pending
This made TF work with GPU
conda install -c conda-forge cudatoolkit=11.8 cudnn

python3 -c "import tensorflow as tf; print(tf.config.list_physical_devices('GPU'))"


use_wandb_tracking was not stored in most Trainers when it is False

In the __init__s of SemanticTransformerTrainer, CoarseTransformerTrainer and FineTransformerTrainer, the attribute self.use_wandb_tracking was only set when the parameter use_wandb_tracking was True. This caused the Trainers to throw an AttributeError upon training when not using WandB.
This is now fixed.

@LukasNel @orrp oh nice, thank you!


Shadowing loop variable with local variable triggers assert

I was writing a kernel containing a snippet like this:
for idx in range(0, end, step):
    idx = idx + offset

(if needed I can provide a full repro)
And I got this compilation error:
python: /data/home/lcw/xlf/triton/lib/Dialect/TritonGPU/Transforms/Pipeline.cpp:645: void {anonymous}::LoopPipeliner::emitPrologue(): Assertion `v && "Value not found in valueMapping"' failed.
Aborted (core dumped)

which comes from here:
https://github.com/openai/triton/blob/52c146f66b79b6079bcd28c55312fc6ea1852519/lib/Dialect/TritonGPU/Transforms/Pipeline.cpp#L643
After changing idx = ... to my_idx = ... the error disappeared.

Here you go: https://gist.github.com/lw/7c8b7f7df2fe44fca0218e942f4808ea

I got this when trying to reproduce the issue:
ModuleNotFoundError: No module named 'triton.ops.matmul'

BTW, I have installed triton from the source code.