Azure DevOps formatting?

I'd prefer not to get flamed out here by asking this, but I've looked around and it doesn't seem like anyone else has asked...  This tool is awesome, but I was wondering if there's any plans in play for making it compliant with Azure DevOps wiki formats?  I noticed when writing an example file that the MD format must be a little different between GitHub and DevOps, because certain things like the use of single and triple hashes don't work, and it didn't build a Table of Contents using the [[TOC]] command.
Is there any plan for how it can be updated to use the Azure DevOps spec as well?  If not, what could be done to help that along?
There isn't a lot of options for editing wiki formats in VS, which is where I'm coding anyway.  I'd love to be able to use this product.  Thank you!
[Edit: For reference, that TOC command was [ [ _ _ T O C _ _ ] ]

You can create a TOC by right-clicking and click "Generate TOC".
As for the AzDO markdown spec, I'd be happy to support any flavor of markdown possible. However, the extension uses the Markdig NuGet package for parsing markdown, so it can only support what Markdig supports. If you add a feature request on Markdig's github repo, then that's probably the fastest way to get the support added to this extension.


Budget Manager Extension

Hello 👋🏻 , as a GSSOC'22 Contributor i would like to give my contribution to this repo by adding Budget Manager Extension

@aka9ksha it would be great if u can explain this issue a little bit more

What this Extension Does?
This extension accepts the spending of user and displays the spending ,users can also set the limit of the spending and get notified when reached the limit .It also has option to reset the amount as well as the limit.
Tech Stack
HTML ,CSS AND JS.

@aka9ksha so will it be fetching the expenses we made online on various platforms?

No, the user have write the amount spended on its own.

oh okay!!
yes you can sure work on that
@aka9ksha assigning to you!!
all the best!!🚀💛


Added support for newest Microsoft.ML.Tokenizer.

Actually, if you do not specify a specific tokenizer for azure openai you will use a default implementation of GPT3 Token. The aim of this PR is using the newest Microsoft.Ml.Tokenizer, that support also count() to calculate token count without creating the array of tokens.
I've done modification only for AzureopenAI, but if the PR is ok I'll add also to the Standard OpenAi configuration.
Actually I had to modify the config file, because with Azure we have deployment name that does not identify correctly the deployment. I've added ModelName property that allow the user to specify the model used (maybe it could be possible to determine automatically the format with an api call).
This allows to create the correct tokenizer based on model and use a faster tokenizer. (Gpt-4o has a different tokenizer).

Is it possible, but the tokenizer you are referring to are one class for each tokenizer. The idea behind my PR is that we can have a single class that get constructed with the model like new ("gpt-4"), new ("text-embedding-ada-002") so you can avoid choosing C# type for the model, but instead you can simply use the very same c# class choosing the model on costructor.
I've create tiktoken specification file for cohere api, so it would probably be the only tokenizer class you use in the entire km :).

In the current code, the developer injects the tokenizer when using the builder, so having one class per model makes it easier, allowing to inject just the type if needed.
if we have a single class, how do we detect the model name? as you mentioned, Azure OpenAI works by deployment model, making this more complicated, but even with OpenAI, and particularly with refined models, autodetecting seems too hard, with code to maintain over time.
I think a configuration approach might be easier, where the developer chooses the tokenizer (either by name or by class). Thoughts?

by the way, I'd like to clean up the tokenizers logic and the list of tokenizers, having a consistent approach. One challenge to address is that KM can be used with other models, e.g. LLama, Phi, Anthropic, etc so we need a design that can be adapted to different models, and different tokenizers

This is the reason, IMHO, why having only one class that can simply choose the model by a string can be the solution. When you use a LLM or an embedding model there is only one tokenization model you can use, so we need to be able to automatically diagnose the type of model then choose the right tokenizer to be 100% accurate.
For azure chat model you can call a completion api, and in the response you got the model you are using, so it is perfectly possible to automatically determine the type of the model. Then having the model as string, you can easily create an instance of the tokenizer given the name.

For cohere models, I've managed to convert the original format to a file that can be opened by microsoft.ml.tokenizer.tiktoken class https://github.com/alkampfergit/ai-playground/blob/develop/src/python/langchainVarious/Tokenization/dotnetcohere.dib, so again we can still use the very same class for cohere models.
Since almost any LLM use the BPE algorithm or a variation of it, we only need to create the vocabulary file in the right format to be loaded by Microsoft Tokenizer and the game is done :).
I'll be OOF for 2 days, I hope to have small time in the weekend to extend the pull request with model autodetection (at least for openai and azure openai) so we can reeturn on this PR discussion next week.

Hi @dluc
I've radically changed the pull request, actually with this new version you pay the price of a two simple call to embedding and chat completion but this will allow you to get EXACT model and then you can use the new and fastest tokenizer from Microsoft.ML namespace.
I've left the configuration that allows for the user to specify the tokenizer but I really think that it should be deprecated.

I think this deserves a design conversation before moving forward, because we have already 2 set of tokenizers and the caching logic can lead to new issues and additional costs.

Agree we can open a discussion on this subject :) I also think that this is. a core part of the library that needs a good design.

I've refactored the code a bit, removing the old tokenizer. See https://github.com/microsoft/kernel-memory/pull/696
I think the feature here, if you want to pursue it, is about autodetecting which tokenizer to use.
The feature should consider refined models, and it should be designed to minimize cost and errors.
Rather than the application runtime, I would extend "dotnet run setup" to autodetect the model and storing the information in the config file. E.g. extending OpenAI and Azure OpenAI config classes to include the tokenizer details.
Autodetection should be optional, allowing the user to choose manually.
When using OpenAI and default models, autodetection doesn't need an API call, because the list of models is well known.
When using OpenAI/Azure OpenAI refined models, I don't know if it is possible to detect the original model name.

I think the feature here, if you want to pursue it, is about autodetecting which tokenizer to use. The feature should consider refined models, and it should be designed to minimize cost and errors.

Agree.

Rather than the application runtime, I would extend "dotnet run setup" to autodetect the model and storing the information in the config file. E.g. extending OpenAI and Azure OpenAI config classes to include the tokenizer details. Autodetection should be optional, allowing the user to choose manually.

It could be an interesting feature.

When using OpenAI and default models, autodetection doesn't need an API call, because the list of models is well known.

Agree!

When using OpenAI/Azure OpenAI refined models, I don't know if it is possible to detect the original model name.

I've done a quick check and without a call it seems not to be possible, but I'll do a check.

Closing for now, we can reopen it or develop the feature separately.


[CI] Default environment jobs to Ubuntu 22.04 instead of Ubuntu 20.04

Describe the enhancement requested
Ubuntu 24.04 is about to be available.
I think it's about time we move our default .env to UBUNTU=22.04.
This task is to cover that work.
Component(s)
Continuous Integration

Issue resolved by pull request 44151
https://github.com/apache/arrow/pull/44151


Possible bug inet_builder_get_text_orientation_options (or is this intentionally?)

Problem Description
The function et_builder_get_text_orientation_options takes two arguments: excluded options and included options. The implementation checks for the excluded options only:
Steps To Reproduce

Screenshot and/or gif

QA Checklist
Reporter

[ ] Helpful title used

Helpful title format: Scope(s) :: One Sentence To Describe The Issue
Example: VB :: Floating Toolbar :: Jumps To Top Of Page

[x] Detailed description
[ ] Steps to reproduce included
[ ] Support Ticket URL included
[ ] Image and/or gif

Developer

[ ] Steps to reproduce proven solved
[ ] Test included and coverage acceptable and all Tests passing?
[ ] PR description has layman's description of problem and solution and "Areas Affected" listed/explained?
[ ] PR branch includes latest commits from master branch (at the time of checking)?
[ ] Estimated points on issue
[ ] Proper code documentation and conventions?
[ ] Proper RTL appearance
[ ] i18n used if applicable
[ ] Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
[ ] Database saved/updated? Data sanitized and validated?
[ ] Variable output properly escaped and validated?

Peer Review

[ ] PR branch includes latest commits from master branch (at the time of checking)?
[ ] Steps to reproduce proven solved
[ ] Affected areas are tested and ok
[ ] Proper code documentation and conventions
[ ] Proper RTL appearance
[ ] i18n used if applicable
[ ] Test(s) included and coverage acceptable
[ ] Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
[ ] Database saved/updated? Data sanitized and validated?
[ ] Variable output properly escaped and validated?

Quality Assurance

[ ] All tests passed on CircleCI.
[ ] Steps to reproduce passed, the fix works.
[ ] No errors in console or output
[ ] Affected areas are tested and ok
[ ] The fix does not change a customer's website in any surprising or unintended way

[ ] If Divi's design is adjusted, settings are migrated gracefully and invisibly

Design Review

[ ] UI is to spec
[ ] UX is to spec
[ ] Design is to spec
[ ] No surprising visual changes to a customer's website will be experienced after updating

[ ] Any/all visual changes are migrated gracefully

Dev Review

[ ] Test(s) included and coverage acceptable
[ ] Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
[ ] Database saved/updated? Data sanitized and validated?
[ ] Variable output properly escaped and validated?
[ ] i18n used if applicable
[ ] Implementation acceptable, DRY, Scaleable, future safe
[ ] Best practices used, cleverness and grokage, ES6 use, etc.

@xxtesaxx I've checked the issue and you are correct, this was not intentional and should be fixed. We're fixing and testing it, thanks for noticing the issue 👍


didSelectItemAtIndexPath doesn't get called

    KKGridView *_gridView;
    _gridView = [[KKGridView alloc] initWithFrame:CGRectZero];
    _gridView.dataSource = self;
    _gridView.cellSize = CGSizeMake(145.f, 100.f);
    _gridView.cellPadding = CGSizeMake(10.f, 10.f);
    _gridView.scrollsToTop = YES;
    _gridView.autoresizingMask = UIViewAutoresizingFlexibleWidth | UIViewAutoresizingFlexibleHeight;
    _gridView.frame = CGRectMake(0, 0, self.view.bounds.size.width, self.view.bounds.size.height);
    _gridView.backgroundColor = [UIColor clearColor];
    [self.view addSubview:_gridView];

anything I am doing wrong?

Add  "_gridView.gridDelegate = self;"
"didSelectItemAtIndexPath" is called via the delegate not the datasource.

awesome. now it works. Many thanks.


Add CRITIC, an early work on general reasoning and tool-using for autonomous self-improve

CRITIC includes tools like search engines, code interpreters, and text APIs.

Thanks for the contribution!


feat: press to show the client object

https://github.com/neovim/nvim-lspconfig/assets/33713262/2c3e624a-33f2-41c6-9604-0eb88db186b4
This PR adds a keymap that prints the LSP client object of the client under the cursor in the LspInfo window. I find that peeking under the hood of a running LSP client helps when debugging LSP issues. For new users especially, as doing this manually requires pulling together a handful of concepts (inspecting a table, printing to new buffer, etc).

The output is printed to a new buffer for easy search/yank/diff/etc.
While the output is not valid lua syntax, it's close enough that calling set syntax=lua does make it easier to do a quick eye scan.
I have no strong opinion on what key to assign this to. I currently have it set to K.
K doesn't need to be pressed on a specific column, anywhere on the same line will do.

thanks for pr. but lspconfig just a config collection. we don't want add more in this repo. and there also have check health interface. anyway thanks agian.

Now that you mention it, I don't think I've ever used :checkhealth vim.lsp before. My go to has always been :LspInfo. I agree that, ideally, we should decide on using only one and appreciate that that is what you're trying to do here. This is anecdotal, but from browsing the /r/neovim subreddit I think most users rely on :LspInfo at this time?
While I understand that you don't want to increase the maintenance burden of this repo, I'd argue that this PR reduces it. It invites curious new users to peek into the LSP client config. It is those types of users who then become likely to identify issues and contribute bug fixes and improvements to the configs hosted in this repo, reducing the overall maintenance effort required by any single maintainer.
It's also worth mentioning that this has been requested at least once (#2412) before this year, by a plugin author no less (as opposed to a newer user). It would also close #2758.
That was only my opinion on the matter. And while I am disappointed, I respect that you might have different views because you see things differently (and you interact with users a lot more than I do).

That was only my opinion on the matter. And while I am disappointed

first I absolutely understand how you feel. I had the same experience before.
but i think it can be very simply to do like :echo v:lua.vim.lsp.get_clients() | put=execute('messages') just wrap in a command.
cc @justinmk


Can't run pls

I'm installing pls at the second time and it gives me a error

Apparently it can't find the "pkg_resources" module and don't start

What is your OS?
Apparently you didn't install the python-setuptools package on your distro


New atrioventricular valve terms

Add new terms:
enlarged atrioventricular valve
DEF: an increase in the total area occupied by either one or both of the atrioventricular valves
SYN: enlarged atrioventricular valves
SYN: enlarged AV valve
SYN: enlarged AV valves
SYN: increased atrioventricular valve size
SYN: increased AV valve size
Child terms: enlarged mitral valve MP:0010618, enlarged tricuspid valve MP:0010627
atrioventricular valve hyperplasia
DEF: increase in the number of normal cells in normal arrangement in either one or both of the atrioventricular valves, typically resulting in increased size
SYN: AV valve hyperplasia
SYN: hyperplastic atrioventricular valve
SYN: hyperplastic atrioventricular valves
SYN: hyperplastic AV valve
SYN: hyperplastic AV valves
Child terms: mitral valve hyperplasia MP:0031524, tricuspid valve hyperplasia MP:0031558
small atrioventricular valve
DEF: a decrease in the total area occupied by either one or both of the atrioventricular valves
SYN: small atrioventricular valves
SYN: small AV valve
SYN: small AV valves
SYN: decreased atrioventricular valve size
SYN: decreased AV valve size
SYN: reduced atrioventricular valve size
SYN: reduced AV valve size
Child terms: small mitral valve MP:0031522, small tricuspid valve MP:0031557
atrioventricular valve hypoplasia (child of small atrioventricular valve)
DEF: decrease in the number of normal cells in normal arrangement in either one or both of the atrioventricular valves, typically resulting in decreased size
SYN: AV valve hypoplasia
SYN: hypoplastic atrioventricular valve
SYN: hypoplastic atrioventricular valves
SYN: hypoplastic AV valve
SYN: hypoplastic AV valves
Child terms: mitral valve hypoplasia MP:0031523, tricuspid valve hypoplasia MP:0011680
thick atrioventricular valve
DEF: an increase in the ratio of an atrioventricular valve wall thickness to the atrioventricular septum thickness
SYN: thick atrioventricular valves
SYN: thick AV valve
SYN: thick AV valves
Child terms: thick mitral valve MP:0010620, thick tricuspid valve MP:0010629
atrioventricular valve stenosis
DEF: abnormal narrowing of either one or both of the atrioventricular valves
SYN: AV valve stenosis
SYN: stenotic atrioventricular valve
SYN: stenotic atrioventricular valves
SYN: stenotic AV valve
SYN: stenotic AV valves
Child terms: mitral valve stenosis MP:0006122, tricuspid valve stenosis MP:0006124

Added new terms:
enlarged atrioventricular valve MP:0031570
atrioventricular valve hyperplasia MP:0031571
small atrioventricular valve MP:0031572
atrioventricular valve hypoplasia MP:0031573
thick atrioventricular valve MP:0031574
atrioventricular valve stenosis MP:0031575


Is this plugin is up to date?

Hello, I purchased Leap Motion controller, I want to create a simple project using this, but I don't know any C++ coding and using the only Blueprint, is it possible to develop its base on your UE4-LeapMotion blueprint plugin? Is it updated to 4.19.2?
Thank you.

Seems not really.


wrong method name used in database_helpers

Should be drop_all_tables() but instead calling drop_tables().
Fixes https://github.com/hammerlab/pyensembl/issues/65

Yikes! Didn't see this. Merging.


Trouble importing

https://stackoverflow.com/questions/70817415/having-trouble-importing-a-library-cannot-set-properties-of-undefined-setting
I'm trying to import this library to my project, but I'm having trouble doing so. It's true that I'm not doing it like it is said to be done on the git demo (doing the js directly in the html file), but my guess is that it should still work.
I have something like this:
import * as NOISE from "../libs/perlin.js";
//...
var y = Math.round(noise.perlin2(x, z)); //like they do in the git example

But I'm getting an error regarding the library itself:
Cannot set properties of undefined (setting 'noise')
    at noise.js:18
    at noise.js:310

(function(global){
  var module = global.noise = {}; /*18*/
  //...
  module.perlin2 = function(x, y) {
  //...
})(this); /*310*/

What am I doing wrong?

Did you already find a solution for this problem?

I have made a simple wrapper for this library with a class in typescript, feel free to use it:
https://github.com/alanko0511/noisejs-typescript

@alanko0511, you are a life saver! 🙏

@alanko0511 you should post this to npm! there's not many ts-compatible perlin noise packages out there


[framework] Adds publisher module

Finalizes the experiment myself and @amnn started; and proposes a publisher module.
The goal of this module is to enable recognition of the Publisher authority. Potentially it could be used for:

Setting rules / registering a type in a system, such as metadata, AFTER the module was published
Authorizing admin actions (effectively replacing AdminCap which is often added manually)

@amnn @sblackshear on the topic: do you think we should rename it into PublisherCap instead of Publisher?

do you think we should rename it into PublisherCap instead of Publisher?

I think technically it's a witness (i.e. it witnessed the package being published), rather than a capability.

@amnn @sblackshear
I've addressed your comments above. Few changes:

adds std::address library with a length() function
adds address getter + module name getter in std::type_name
does a little cleanup in our sui::bcs library to switch to sui::address instead of using const for ADDRESS length


[Query Action] Variable assignment should not be needed when query completes normally

Description:
Atm, the static type of query action is considered to be error? which is not correct. This inturn requires the user to assign the result to a variable even when the return type is actually ().
public type MetaData record{|
    string absPath;
|};

function foo() returns string[] {
    string[] files = [];
    MetaData[] listResult = [];
    from var md in listResult // variable assignment is required
        do {
            files.push(md.absPath);
        };
    return files;
}

Steps to reproduce:
Affected Versions:
SL GA
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

Reopening as the fix was reverted with https://github.com/ballerina-platform/ballerina-lang/pull/37117 due to  https://github.com/ballerina-platform/ballerina-lang/issues/37310


feat: Don't insert imports outside of cfg attributed items

Closes #6939

Tests for this live in the auto-import assist for now as I'll have to change up the insert_use tests a bit first. Will get to that tomorrow.
bors r+


Fix user count not being updated before the first 30 second tick

Just like @melonmanchan would fix it

:+1:


When using Linking with sms on ios it fail

Description
To open sms application with text attached i use this code block:
 const separator = Platform.OS === 'ios' ? '&' : '?';
    const url = `sms:${phone}${separator}body=${text}`;
    console.log("url",url)
    Linking.openURL(url);

when i remove the body part its work fine.
using this code i get error:
Possible Unhandled Promise Rejection (id: 0):
Error: Unable to open URL: file:///private/var/containers/Bundle/Application/E18703D3-1FB9-49B0-BC6E-DA8C4FEFF458/xxx.app/sms:00000000%3Fbody=%D7%A9%D7%9C%D7%95%D7%9D%20%D7%94%D7%9B%20%D7%A0%D7%94%D7%A2,%0A
React Native Version
0.71.4
Output of npx react-native info
System:
OS: Windows 10 10.0.19044
CPU: (8) x64 Intel(R) Core(TM) i7-4790 CPU @ 3.60GHz
Memory: 17.19 GB / 31.91 GB
Binaries:
Node: 16.15.1 - C:\Program Files\nodejs\node.EXE
Yarn: 1.22.19 - ~\AppData\Roaming\npm\yarn.CMD
npm: 8.18.0 - C:\Program Files\nodejs\npm.CMD
Watchman: Not Found
SDKs:
Android SDK:
API Levels: 23, 28, 29, 30, 31, 32, 33
Build Tools: 28.0.3, 29.0.2, 29.0.3, 30.0.2, 30.0.3, 31.0.0, 33.0.0
System Images: android-30 | Google TV Intel x86 Atom, android-33 | Google APIs Intel x86 Atom_64, android-33 | Google Play Intel x86 Atom_64
Android NDK: Not Found
Windows SDK: Not Found
IDEs:
Android Studio: AI-213.7172.25.2113.9123335
Visual Studio: Not Found
Languages:
Java: <IP_ADDRESS> - C:\Program Files\Common Files\Oracle\Java\javapath\javac.EXE
npmPackages:
@react-native-community/cli: Not Found
react: 18.2.0 => 18.2.0
react-native: 0.71.4 => 0.71.4
react-native-windows: Not Found
npmGlobalPackages:
react-native: Not Found
Steps to reproduce

 const separator = Platform.OS === 'ios' ? '&' : '?';
    const url = `sms:${phone}${separator}body=${text}`;
    console.log("url",url)
    Linking.openURL(url);

call this code block and got the error
Snack, code example, screenshot, or link to a repository

 const separator = Platform.OS === 'ios' ? '&' : '?';
    const url = `sms:${phone}${separator}body=${text}`;
    console.log("url",url)
    Linking.openURL(url);

Face it every time!!!
Please help.
Jeff - Thanks for bringing it up

We seem to have the same problem but with a mailto: link.

We seem to have the same problem but with a mailto: link.

yes i can tell i have also the problem with mailto when adding text to it (only on iphone)

@react-native-bot ?

we are having exactly the same issue

Same problem here as well. I'm finding that if you wrap your link in encodeURI, that allows you to still include the body.

i can say this is working good, thanks

appreciate this is a different issue but seems there's some expertise on this thread that may know the answer:
have you guys been able to send an SMS to multiple users on iOS?
if i pass multiple phone numbers, in correct format, to the openURL function, only the first number provided appears in the 'To:' window on the external SMS screen. any advice much appreciated !

Looks like the issue is that the Linking.openURL call isn't allowing spaces anymore as of RN 0.71, but thankfully like @timhooker said up above, wrapping it all in encodeURI() makes it work again.


Respect cleanup table's "group" column in PCA vis

@subdavis, I made the requested change; please take a look and let me know what you think.

Should be fixed now.


Yarn

New Pull Request Checklist
Please mark the points which you did / accept.

[x] I read the Step Development Guideline
[x] I have a test for my Step, which can be run with bitrise run test (in the step's repository)
[x] I did run bitrise run audit-this-step (in the step's repository - note, if you don't have this workflow in your bitrise.yml, you can copy it from the step template.)
[x] I read and accept the Abandoned Step policy

Thanks it looks awesome!
Just one thing, could you please upgrade your Bitrise CLI to 1.4.2 and re-share 0.0.2 again? One PR should only include one version of the step.

In the meantime, @godrei summoned for a review ;)

@viktorbenei sure thing. Coming up


Windows/Privacy: repair line 14 broken in PR #4298

Description:
Pull Request #4298 forgot to resolve a broken line and an extra parenthesis between line 14 and line 15.
This resulted in no check marks for the text in line 14.
This PR resolves the left-over issue as noted by Dani Halfin in https://github.com/MicrosoftDocs/windows-itpro-docs/pull/4298#pullrequestreview-257597984.
Proposed change:
Retract the empty line with check marks back to line 14 where it belongs, removing the redundant parenthesis.

@DaniHalfin : This is the resolution you were looking for in https://github.com/MicrosoftDocs/windows-itpro-docs/pull/4298#pullrequestreview-257597984 .

cc @jvsam


repository /assets .. not found

Hi,
i had this error when i wanted to try this project

Help please !!

@mes1moha great question.
This is an old project (2014) and a lot of the code probably won't work anymore. However, the error you're seeing it because gulp wasn't ran.
I looked at this code and seems like updating will take a while.

@karbassi  thank you ;)


Question: How to add graticule grids to map via Mapfish Print V2 or Mapfish Print V3

Hello,
Sorry, I had to open it as a bug but I wanted to ask this question in the group but for some reason, I cannot access the group https://groups.google.com/access-error?continue=https://groups.google.com/g/mapfish-print-users., Currently, I'm using Geoserver 2.1.8 (installed via Geonode) printing module which uses Mapfish Print V2.  But I am unable to find any resource or reference on how to add graticule grids to maps with Mapfish Print V2.  Any direction or reference on how to add this to the map would be helpful.
Thank you in advance
Regards,
Pratyush

Hello, @Pratyush1991
Have you found any solution for this question? Your issue is just the same I'm trying to solve right now.
Please, let me know what the way you took.
Thanks in advance.


Cerialize deserializes empty strings as null (no user-visible bug)

...at least it does so sometimes (when the fields are tagged with @autoserializeAs(String). As far as I'm aware, this doesn't result in user-visible bugs in DSpace so far, but it's something we've run into using DSpace-CRIS that turned out to be somewhat difficult to debug.
The problem is this code in cerialize.js:
function deserializeString(value) {
    if (Array.isArray(value)) {
        return value.map(function (element) {
            return element && element.toString() || null;
        });
    }
    else {
        return value && value.toString() || null;
    }
}

If value is '', the return value will be null rather than the empty string that was passed in.
As far as I can tell, this only happens for fields decorated with @autoserializeAs(String), not those of type string which are merely decorated with @autoserialize. Changing the latter to the former (I believe this should not change behavior) allows the bug to become observable.
Again, this is not a user-visible bug right now, but it is surprising and arguably incorrect behavior of an underlying library which results in JavaScript objects that violate their TypeScript type descriptions, and it is possible to run into this when extending DSpace.
If people agree this is a cerialize bug, I'll report it there, of course.

@philipprumpf : I think this already has a bug ticket in cerialize here: https://github.com/weichx/cerialize/issues/58
Unfortunately, it doesn't appear that cerialize is being maintained.  So, it's possible we'd either need to find a way around this in our codebase, or we'd need to see if we can find a different library to replace it.

Oh, thank you! I'd missed that, somehow (or thought it was about the "null" -> null thing). I guess we can't do anything about it for now.


Add access_token to URL

I have a model invoice with a property pdfUrl that points to the PDF version of the invoice.
import DS from 'ember-data';

export default DS.Model.extend({
  // ...
  pdfUrl: DS.attr('string')
});

The URL is presented as a link to the user (target="_blank").
Because the link is handled by the browser and not Ember, the Authorization: Bearer xxx is not included in the headers.
Since emper-simple-auth also allows the access_token to be included via query parameter, I think there has to be a way to get the token and add it to the URL.
Something like this:
  pdfUrlWithAccessToken: function() {
    var token = ?;
    return this.get('pdfUrl') + '?access_token=' + token;
  }.property('pdfUrl')

How would you approach this?

You could inject the session into the model but I wouldn't suggest that. The better solution is probably to generate the URL in the controller or component where the session is available anyway. Also that's the better separation of concerns.

Ok, I now generate the URL in the controller (controllers/invoices/create.js):
import Ember from 'ember';

export default Ember.Controller.extend({
  pdfUrl: function() {
    return this.get('model.pdfUrl') +
           '?access_token=' + this.get('session.access_token');
  }.property('model.pdfUrl', 'session.access_token'),
});

But how would you do it in controllers/invoices/index.js? (i.e. for a list of invoices)
And how would I inject the session into the model? How is DI (http://emberjs.com/guides/understanding-ember/dependency-injection-and-service-lookup/) done in ember-cli?

The best solution would probably be to use a component for displaying each component and inject the token into the URL there. Also that way you would reuse the same logic for single and collection routes.
Injecting the session into the model would work like this:
container.injection('model:invoice', 'session', 'simple-auth-session:main');

but I wouldn't suggest doing that.


Thank you!

Hi, Jake!
Just want to say you thanks for open-sourcing this code and also writing articles on how you made the Big Web Quiz.
I wonder, how did you manage to write such a complex app without having some unit/e2e/integration tests? To be honest it looks to me as something unbelievable.

Glad the code is useful! It was hacked together pretty quickly.

how did you manage to write such a complex app without having some unit/e2e/integration tests? To be honest it looks to me as something unbelievable.

Urrrrr we just kinda crossed our fingers and hoped. Not something I'd recommend doing! But the app isn't all that complicated, there are only a few states.


Raw query returning a bigint for an int field

Bug description
When using prisma.$queryRaw to query data, columns of the Int.Unsigned type in MySQL will be converted to and returned as the Bigint type.
How to reproduce
Table Structure & Example Data:
CREATE TABLE `folder` (
  `id` int unsigned NOT NULL AUTO_INCREMENT,
  `name` varchar(64) NOT NULL,
  `pid` int unsigned NOT NULL DEFAULT 0,
  PRIMARY KEY (`id`),
  KEY `idx_pid` (`pid`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4 COLLATE=utf8mb4_0900_bin;

INSERT INTO `folder` (`id`, `name`, `pid`) VALUES
(1, 'folder-1', 0),
(2, 'folder-2', 0),
(3, 'folder-3', 0),
(4, 'folder-1-1', 1),
(5, 'folder-1-2', 1),
(6, 'folder-2-1', 2),
(7, 'folder-2-2', 2),
(8, 'folder-3-1', 3),
(9, 'folder-3-2', 3);

Execute the following query:
const list = await prisma.$queryRaw`
  WITH RECURSIVE folder_cte AS (
    SELECT id, name, pid
      FROM folder
      WHERE id IN (5, 7)
    UNION ALL
      SELECT f.id, f.name, f.pid
        FROM folder f
        INNER JOIN folder_cte fc ON f.id = fc.pid
  )
  SELECT DISTINCT * FROM folder_cte ORDER BY id ASC;
`

Returns rows:
[
  { "id": 1n, "name": "folder-1", "pid": 0n },
  { "id": 2n, "name": "folder-2", "pid": 0n },
  { "id": 5n, "name": "folder-1-2", "pid": 1n },
  { "id": 7n, "name": "folder-2-2", "pid": 2n },
]

Expected behavior
Int.Unsigned is 4-byte (32-bit). According to the documentation, it should return a Number type, not a BigInt.
Expected returns rows:
[
  { "id": 1, "name": "folder-1", "pid": 0 },
  { "id": 2, "name": "folder-2", "pid": 0 },
  { "id": 5, "name": "folder-1-2", "pid": 1 },
  { "id": 7, "name": "folder-2-2", "pid": 2 },
]

Prisma information
generator client {
  provider      = "prisma-client-js"
  binaryTargets = ["native", "rhel-openssl-1.0.x"]
}

datasource db {
  provider = "mysql"
  url      = env("DATABASE_URL")
}

model Folder {
  id        Int      @id @default(autoincrement()) @db.UnsignedInt
  name      String   @db.VarChar(64)
  pid       Int      @default(0) @db.UnsignedInt

  @@index([pid], map: "idx_pid")
  @@map("folder")
}

const prisma = new PrismaClient({
  datasourceUrl: `MY_DATASOURCE_URL`,
  log: ['warn', 'error'],
})

Environment & setup

OS: macOS
Database<EMAIL_ADDRESS>Node.js version: v16.19.0

Prisma Version
prisma                  : 5.13.0
@prisma/client          : 5.13.0
Computed binaryTarget   : darwin
Operating System        : darwin
Architecture            : x64
Node.js                 : v16.19.0
Query Engine (Node-API) : libquery-engine b9a39a7ee606c28e3455d0fd60e78c3ba82b1a2b (at ../../node_modules/.pnpm/@prisma+engines<EMAIL_ADDRESS>Schema Engine           : schema-engine-cli b9a39a7ee606c28e3455d0fd60e78c3ba82b1a2b (at ../../node_modules/.pnpm/@prisma+engines@5.13.0/node_modules/@prisma/engines/schema-engine-darwin)
Schema Wasm             : @prisma/prisma-schema-wasm 5.13.0-23.b9a39a7ee606c28e3455d0fd60e78c3ba82b1a2b
Default Engines Hash    : b9a39a7ee606c28e3455d0fd60e78c3ba82b1a2b
Studio                  : 0.500.0

Hey @iGarym, I took a quick look and was able to confirm the following data with the query and schema you provided!
[
  { id: 1n, name: 'folder-1', pid: 0n }
  { id: 2n, name: 'folder-2', pid: 0n }
  { id: 5n, name: 'folder-1-2', pid: 1n }
  { id: 7n, name: 'folder-2-2', pid: 2n }
]

So I took a quick look at the MySQL documentation and the problem here comes from it being an unsigned 32-bit value rather than a signed 32-bit value which gives it an upper range target of<PHONE_NUMBER> which is beyond the scope of js' 32-bit number. So this isn't a bug per se, it really should be a bigint.
I definitely see where the confusion comes from, however, we should probable have another addition in the table you mentioned that clarifies where unsigned integers land.

@Druue It is up to the developer to decide whether to return bigint, not prisma, which is directly defined and cannot be modified. prisma can provide an option to convert unsigned numbers from $queryRaw to bigint. In many cases, especially some complex queries, often use $queryRaw, this time the structure is more complex, but also manually to convert the Number is very troublesome. At the same time, the field defined in the.prisma file is not bigint, why find out that you want to arbitrarily change to bigint? .prisma file is defined in bigint field, find out is bigint no problem, can define is not bigint query is bigint, this is very bad. Because the results of these queries are bigint, the JSON serialization will also report errors, and the transmission to other programs will be converted every time, which is very difficult to use. This is definitely a bug, not the correct expected behavior.

Hey @iGarym, I took a quick look and was able to confirm the following data with the query and schema you provided!
[
  { id: 1n, name: 'folder-1', pid: 0n }
  { id: 2n, name: 'folder-2', pid: 0n }
  { id: 5n, name: 'folder-1-2', pid: 1n }
  { id: 7n, name: 'folder-2-2', pid: 2n }
]

So I took a quick look at the MySQL documentation and the problem here comes from it being an unsigned 32-bit value rather than a signed 32-bit value which gives it an upper range target of<PHONE_NUMBER> which is beyond the scope of js' 32-bit number. So this isn't a bug per se, it really should be a bigint.
I definitely see where the confusion comes from, however, we should probable have another addition in the table you mentioned that clarifies where unsigned integers land.

hi @Druue , thx a lot for you guys work on  this amazing project. I run into the same issue, and it turns out the maximum integer in javascript is Number.MAX_SAFE_INTEGER, which is much larger than<PHONE_NUMBER>. So I don't get it why prisma parse the unsigned int as BigInt. Can you give some more detailed info on this, thx in advance.


trying out playwright

pretty handy: https://github.com/TBD54566975/ssi-admin-ui/actions/runs/4361000632 - you can even see it retrying, and being flaky etc

@kirahsapong I think this one is the one! Nice suggestion on using playwright, it seems really good.


feat: radio component

@zedaan Cool thanks! There's one conflict in index.ts btw

@zedaan Cool thanks! There's one conflict in index.ts btw

@zedaan Yep, I've fixed it, let me push it

Also, I'm wondering how to use the radio in a group? Radio's are typically in a group but not sure how to do this with the component.
And design has a component Radio.Group https://ant.design/components/radio/#components-radio-demo-radiogroup
Maybe we can have a component called RadioGroup too.

Also, I'm wondering how to use the radio in a group? Radio's are typically in a group but not sure how to do this with the component.
And design has a component Radio.Group https://ant.design/components/radio/#components-radio-demo-radiogroup
Maybe we can have a component called RadioGroup too.

Yep, we have a Component for that as well. will add it after finishing the changes


Refactor Link.to_device family

Separates the structures and the algorithms of {Link,Chain,ChainList,...} × {to_device,to_cpu,to_gpu,to_intel64,to_chainerx,from_chainerx}.

Done:

Renamed to DeviceResidentMixin (from DeviceObjectVisitableMixin).
Overriding to_device() causes an error.
Overriding to_cpu/to_gpu/to_intel64 causes a warning, but they're still called (even if to_device was originally called).

Remaining:

Where to put DeviceResidentMixin and visitor implementations? _link.py is rather temporary.
I'm going to remove _GetSoleDeviceVisitor from this PR. It's implemented only for demonstration.

Hmm, there's an import cycle:
utils/__init__.py -> utils/walker_alias.py -> _link.py -> utils/__init__.py

Fixed.

As for the name of _link.py, how about just naming it _device_resident.py?

Sounds good. Renamed to device_resident.py (without underscore).

Removed _GetSoleDeviceVisitor.
Now ready for review.

Renamed DeviceResidentsVisitor.visit_visitable to visit_device_resident.

I found that it would be dangerous to mark the conventional to_cpu/gpu overriding as deprecated at the same time of introducing a new way (to_device). In my understanding, to_device interface is not mature yet and the desired API would be changed a bit in the future, so it's safer to wait for it maturing and then mark the old ways deprecated.

Removed deprecation.

Thanks, and sorry for the late response. LGTM.  Could you resolve the conflicts?

Resolved conflict.

Jenkins, test this please.

Jenkins CI test (for commit 53e28855e118241be6c344a70953a18df2ed1216, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

Jenkins, test this please.

Could you resolve flake8 errors?

Oops, fixed.

Jenkins, test this please.

Jenkins CI test (for commit f71a610c7ab4aedf8ca48b0712535a651d0876bb, target branch master) succeeded!

LGTM!


Basic restore saved analysis from db

Initial test to save/restore analysis of builtins module

https://github.com/microsoft/python-language-server/pull/1224


Fixed basic text formatting due to missing property

Missing "text-decoration": "none" in the ## Formatting multi-value fields section.
Without this, the text is underlined and does not match the example image that is displayed.
Category

[x] Content fix

Related issues
What's in this Pull Request?
Added missing property "text-decoration": "none" to match the sample image of the JSON output.

Docs Build status updates of commit 45dbea4:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: RuningBuildFailed] Some unexpected errors happened when running build, please open a ticket in https://aka.ms/SiteHelp and include the error report for our team to troubleshoot

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

closing/reopening to trigger rebuild

Docs Build status updates of commit 45dbea4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/declarative-customization/column-formatting.md
:white_check_mark:Succeeded
View

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

Docs Build status updates of commit ae11840:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/declarative-customization/column-formatting.md
:white_check_mark:Succeeded
View

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


image_transfomer sample yields "inconsistent global dictionary versions used" error

Following the image_transformer sample code verbatim, I get the following issue:
curl -v -H "Host: ${SERVICE_HOSTNAME}" -d @./cifar-input.json http://$CLUSTER_IP/v1/mod els/$MODEL_NAME:predict
* Expire in 0 ms for 6 (transfer 0x5585cdd69f50)
*   Trying [IP]..
* TCP_NODELAY set
* Expire in 200 ms for 4 (transfer 0x5585cdd69f50)
* Connected to [IP] ([IP]) port 80 (#0)
> POST /v1/models/transformer-cifar10:predict HTTP/1.1
> Host: transformer-cifar10.default.mtge.io
> User-Agent: curl/7.64.0
> Accept: */*
> Content-Length: 3422
> Content-Type: application/x-www-form-urlencoded
> Expect: 100-continue
>
* Expire in 1000 ms for 0 (transfer 0x5585cdd69f50)
* Done waiting for 100-continue
* We are completely uploaded and fine
< HTTP/1.1 500 Internal Server Error
< content-type: text/html; charset=UTF-8
< date: Fri, 17 Jan 2020 21:53:18 GMT
< server: istio-envoy
< x-envoy-upstream-service-time: 514
< Content-Length: 239
< Connection: keep-alive
<
* Connection #0 to host [IP]left intact
<title>500: INTERNAL:inconsistent global dictionary versions used: mixer knows 221 words, caller knows 222</title>500: INTERNAL:inconsistent global dictionary versions used: mixer knows 221 words, caller knows 222

The source of the issue seems to be in  Istio, here: https://github.com/istio/istio/blob/master/mixer/pkg/api/grpcServer.go#L87
Environment:
Istio Version: 1.2.10-gke.0
Knative Version: 0.11.0
KFServing Version: 0.2.2
Kubeflow version: (Not used)
Kubernetes version: v1.15.7-gke.2 (Google Kubernetes Engine)

not kfserving issue
/close


Android update

Pull in some necessary declarations from bionic, exclude some CRuntime_Glibc definitions, add ARM support extracted from my work with ldc, and replace all uses of Android with CRuntime_Bionic or linux.

Ping, no problems with the auto-tester for three weeks.  I think this and #1384, which has been sitting for more than five months, can be merged.

I'd like to get this and a couple other PRs merged into the ldc branches.  Should I do that or would you prefer to do it?

Regarding function implementations, I guess you're talking about __tls_get_addr?  The two function prototypes are slightly different, because X86 uses an extra underscore and I think the dmd patch dereferences the passed-in pointer, so only the last three lines are the same.  Rather than extract those out, I just cut-n-pasted it.


WIP Support for Leaflet Popup

The content of the Popup can be any DOMWidget.
In the example below, I used a bqplot figure.

cc @galleon

I also realized that displayed events must be propagated for leaflet layers like for DOM Widgets, which I am now doing.

@SylvainCorlay tried it this morning it was working fine with a Text widget. Wanted to try it again and it is not working anymore. I get this error:
Error: Couldn't create a view for model id '72fbd0…s-widgets/extension.js?v=20160524134002:32043:30

@SylvainCorlay Popups would be very useful for my current project but I can't figure out how they are used.  Can you share the code for your example above.


Update how-to-monitor-model-performance.md

Edit the yaml sample
Add component output & schema table

@Bozhong68

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off

#sign-off


Invalid combinations for Teams presence status

Hi, I've just downloaded the 4.7.0b1 where I'm trying to update my Teams presence status.
The feature works however some combinations can't be called by the service.
Eg. Availability: Available + Activity: In Call/In a Conference Call/Away/Presenting
Or Availability: Busy + Activity: Available/Away/Presenting
I found that Microsoft also lists the variations on their page: https://learn.microsoft.com/en-us/graph/api/presence-setpresence?view=graph-rest-1.0&tabs=http
Is there a possibility to only show possible variations when selecting Availability ?
Also, what I found is that Do Not Disturb and Away can't be set as Do Not Disturb/Away only.
Do Not Disturb can only be set to Presenting activity as per the Microsoft documentation.
When going through the setPresence documentation, it seems that might not work and setuserpreferredpresence might be required instead.
https://learn.microsoft.com/en-us/graph/api/presence-setuserpreferredpresence?view=graph-rest-1.0&tabs=http
Away, while the service is being called correctly does not change it to Away. Not sure if this is API limitation or an issue with the feature itself.
Logger: homeassistant.components.websocket_api.http.connection
Source: custom_components/o365/classes/teamssensor.py:88
Integration: Home Assistant WebSocket API ([documentation](https://www.home-assistant.io/integrations/websocket_api), [issues](https://github.com/home-assistant/core/issues?q=is%3Aissue+is%3Aopen+label%3A%22integration%3A+websocket_api%22))
First occurred: 11:32:26 (8 occurrences)
Last logged: 11:37:57

[140625181447616] Error handling message: Unknown error (unknown_error) NightMean from <IP_ADDRESS> (Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36)
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/components/websocket_api/decorators.py", line 26, in _handle_async_response
    await func(hass, connection, msg)
  File "/usr/src/homeassistant/homeassistant/components/websocket_api/commands.py", line 795, in handle_execute_script
    script_result = await script_obj.async_run(
                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 1600, in async_run
    return await asyncio.shield(run.async_run())
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 435, in async_run
    await self._async_step(log_exceptions=False)
  File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 487, in _async_step
    self._handle_exception(
  File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 512, in _handle_exception
    raise exception
  File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 485, in _async_step
    await getattr(self, handler)()
  File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 723, in _async_call_service_step
    response_data = await self._async_run_long_action(
                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 685, in _async_run_long_action
    return long_task.result()
           ^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/core.py", line 2279, in async_call
    response_data = await coro
                    ^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/core.py", line 2316, in _execute_service
    return await target(service_call)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 892, in entity_service_call
    single_response = await _handle_entity_call(
                      ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 962, in _handle_entity_call
    result = await task
             ^^^^^^^^^^
  File "/usr/local/lib/python3.12/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/o365/classes/teamssensor.py", line 88, in update_user_status
    status = self.teams.set_my_presence(
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/O365/teams.py", line 777, in set_my_presence
    response = self.con.post(url, data=data)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/O365/connection.py", line 886, in post
    return self.oauth_request(url, 'post', data=data, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/O365/connection.py", line 864, in oauth_request
    return self._internal_request(self.session, url, method, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/O365/connection.py", line 826, in _internal_request
    raise HTTPError('{} | Error Message: {}'.format(e.args[0], error_message), response=response) from None
requests.exceptions.HTTPError: 400 Client Error: Bad Request for url: https://graph.microsoft.com/v1.0/me/presence/setPresence | Error Message: The provided availability/activity is invalid.

Related to #197

Understood, thank you. Might be worth adding that to the documentation for reference.
Any reason why the setuserpreferredpresence was not used instead? That seems to support changing to DND/Away and so on. With setPresence it does not override the Teams status set by the user in the app/web
https://learn.microsoft.com/en-us/graph/api/presence-setuserpreferredpresence?view=graph-rest-1.0&tabs=http

I'll add an extra service, to do preferred status. Set Status is to give Teams a clue based on that application as to what the status should be. Set Preferred Status is equivalent to doing it in Teams itself.
I have to correct an error in the underlying library to add the extra service.

Perfect, much appreciated.
Thank you!

New service should be available in beta 2 - https://github.com/RogerSelwyn/O365-HomeAssistant/releases/tag/v4.7.0b2


ignore null or empty application name for property source

Description
A "/null/" context will be queried when the spring application is not configured. Which should not be queried at all.

Codecov Report

Merging #378 into azure-config will increase coverage by 0.13%.
The diff coverage is 100%.

@@                Coverage Diff                 @@
##             azure-config     #378      +/-   ##
==================================================
+ Coverage           51.25%   51.39%   +0.13%
- Complexity            480      482       +2
==================================================
  Files                 139      139
  Lines                2187     2189       +2
  Branches              126      127       +1
==================================================
+ Hits                 1121     1125       +4
+ Misses                991      990       -1
+ Partials               75       74       -1

Impacted Files
Coverage Δ
Complexity Δ

...cloud/config/AzureConfigPropertySourceLocator.java
88.88% <100%> (+5.16%)
13 <0> (+2)
: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 90de4d1...a47b23f. Read the comment docs.


add file for SARI,CAS

@susan-Rao First stage of verification passed, official domain is sari.cas.cn as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision. Proof of domain ownership: http://english.sari.cas.cn/

@susan-Rao Thank you for your patience while waiting for a review and processing your request. Pull request merged.


Added "Shared Hosting with cPanel" to "Running Your App" [ci skip]

Description
I wrote instructions to use CodeIgniter on Shared Hosting with cPanel, but it can also be applied on Shared Hosting in general.
Checklist:

[X] Securely signed commits
[X] User guide updated
[X] Conforms to style guide

I'm also torn on this one. I do think many people have an issue with the Primary Domain on CPanel, but there are other ways that work that will keep the files together, like mentioned in this article. There's probably a few thousand other articles out there like it since all of the other frameworks use a similar setup.
I think including a basic "server setup" page might be preferable that gives bare bones setup instructions for Apache, NGinx and on a tricky hosting platform like CPanel. But I'd like use to find the simplest way to host on a primary domain on CPanel that we can.

Hi, thanks for the feedback. At my Shared hosting I can't connect to the server with SSH, and haven't figured out any other ways to change it. It's a bummer that you can't change the document root from the UI like you can with addon (and sub-) domains.
@MGatner What if you want to use your primary domain you bought? Not everyone have one domain to waste. (Don't want to delete my primary domain to test it out myself, as it will wipe my current website.)
I wrote this part as it have come up a few times in the forum and people don't know what to do. I have seen people make .htaccess and redirect into /public/ folder just so their website worked. So that's why this where born. This aren't that hard to figure out if you know what you are doing, but as this is hard enough, I think the SSH part will be scary as hell.
We had a similar topic (if I remember correctly) in CI3 manual, on how to move index.php into public_html instead. But maybe this suites better for the "Learn" portal on the new website?

We still have docs on moving the main index.php file. But you bring up a good point - this would be a great thing for the new Learn section on the upcoming website. Let's leave if for that.


Browserify

I find this library useful in the browser as well.
I'm using browserify to build it and everything seems to work.  I used mochify to execute the test suite and it's all green.
The "mime-types" name is still available on bower

Thank you! Sorry for my ignorance on bower ( :) ), but if there a way we can not check in machine-generated files (like the browser/ directory) to GitHub? How do I publish a module to bower? Also, if you get a change, please take a look into the failing Travis CI tests :)

I've removed the machine-generated files. To generate them you need recent version of node (I think >=0.10.0), then
  npm install -g gulp
  npm run browserify

I've actually never published to bower but I think
bower register mime-types git://github.com/jshttp/mime-types.git and that's it

I'm not sure how bower could build the files if you don't want them in the repo. I'll check that tomorrow.

Cool :) So if we really have to have them in our repo, I suppose, but it would at least be good to know if it's possible not. The main reason that we can forget to rebuild them on every commit by accident.
If we do have to include them, since the source becomes part of our repo, we would need to know all the copyrights and full text licenses for all the included source (it looks like third-party code gets checked in) for legal reasons.
As for optionalDependencies, you cannot add gulp and friends there, as that is for end users, so suddenly installing this module from npm will try to install gulp and stuff on everyone's machine.

So it's not possible to not have them in the repo, since bower only stores the the git url and there isn't a way to a build with bower. I've seen that bluebird keeps the releases that contain the built files outside of the master branch. Other repos don't bother. Others like superagent consider the end-user has to browserify himself.
For the copyright stuff I don't really understand, I don't think third party code will be included in the final source code (unless generated code is considered as such?) I'm not sure I understand what you meant... Would it be ok now that I've remove the gulp dependencies?
I've remove the optionalDependencies. But I had to do a npm install of browserify and uglifyjs inside in npm scripts because I cannot add them to the dev dependencies otherwise travis CI tests will fail for node < 0.10. It's not very nice but I don't see any other way to do it.

I've seen that bluebird keeps the releases that contain the built files outside of the master branch.

That sounds reasonable. How can we achieve this :)? Any special thing you have to do to publish to bower so it knows to look in a certain branch?

For the copyright stuff I don't really understand, I don't think third party code will be included in the final source code (unless generated code is considered as such?) I'm not sure I understand what you meant...

So you used to have the file https://github.com/stropitek/mime-types/blob/master/dist/mime-types.js checked into this changeset. If you look at it, it contains source code from outside this repository. The first bit shows it's "Copyright Joyent, Inc. and other Node contributors." and include the license. There are other parts of code that is in there that is not from this repo and doesn't include the full license text, so that's ultimately the issue with us checking that file into this repo, as it constitutes "redistribution" and I don't know if I'm allowed to redistribute the code without seeing the license.

Oh yes I missed that. Browserify includes some code from this file because he has detected require('path') in your code. Since path has a dependency on process a second part is added that fakes process. I suppose it's this second part that your refer as to "other parts of code". This second part is actually not needed so it can be removed by calling browserify with the --detect-globals options set to false.

That sounds reasonable. How can we achieve this :)? Any special thing you have to do to publish to bower so it knows to look in a certain branch?

For example after publishing to npm you would:

create a local branch for example v2.1.2
Set version in bower.json to 2.1.2
Build browser files with npm run build
Commit and create a new tag that you name 2.1.2
Push the tag to github (no need to push the branch)

The bower package needs to be registered once but doesn't need to be published. To use your library bower users will need to specify the name of the package and optionally a version. Bower will then fetch the appropriate tag from github and copy what is needed.

Hm... well, we are only tagging the commits that are the exact one that was published to npm, so if bower goes of git tags, then that won't work, as our tags would be on master, rather than a second branch. It seems to be that bower is way too closely tied to the layout and management of a git repo. This means that, really, the bower mime-types module should actually be a seperate repo.

You could always publish to npm the tagged version with the browserified files. That's actually what bluebird does.

But the problem with that is we let your dependency's version float, but then the browserified copy, since it will contain the dependencies embedded, will of course not float, as it will be fixed as publish time. This means that installing this module you'll end up with a different MIME database depending on if you are using the browserified version or not (and also means we'd have to publish for every dependency update).

But even then, I don't see how publishing the browserified files will help with bower?

So I was reading about bower and talking about it with a coworker and there is no requirement at all for us to store any kind of browserified file here, as bower does not carer at all and you can always use browserify on your installed bower_components after the fact if you need to do that.

We either have to figure out how to get this module's dependencies on bower and then put this module on bower without bundling dependencies, or a bower version of this module needs to be a separate repo, since bower only works off the literal contents of the repos and our tags only point to the exact commits that are what was published to npm.
Can you help me determine how bower with with dependencies?

Hi @stropitek , any help with my last question?

I cannot see how to make it work using bower dependencies. Mime-db does not need to be built and could be added as is to bower, and mime-types could have it as a dependency. But how should the browserified version of mime-types know how to load that dependency?
The idea of a separate repo that bundles everything together seems easier and more maintainable to me, as long as it is a bot that updates the project every time it (or one of it's dependencies) is bumped. Since this repo does not have complex nested dependencies, that seems manageable...
Maybe there are some pitfalls I cannot see with that approach since I've never seen it used in any other project. If we go with that I also don't know how the repo should be versioned since it would need to be bumped more often than the original mime-types repo.
PS: I will probably we slow to respond until I get my computer replaced at home :smile:

Just install this module using npm and in a file you end up running through your browserify command, require('mime-types') somewhere and browserify will automatically bundle this module and it's dependency, without adding any additional dependencies to this repository.


ROCANA-6476 Windows process metrics

Add Windows process metrics support. There are some caveats:

It's not clear how to get the process name (as displayed in the task manager), versus the binary name. This reports this base name of the first module, which will be the executable filename in most cases.
There doesn't seem to be a canonical way to get the list of arguments for a process.
The mapping of Windows memory to Linux memory concepts doesn't seem to be exact.
Long-term we need to build a shim or something for Windows syscalls so there can be better unit tests. Right now we just check that the calls don't crash.

Did you consider using Process32First / Process32Next? I think this is the higher level API, but I'm not sure it provides everything we need/want.

It would have been nice to use the Process32First and Process32Next APIs, the existing SIGAR API is kind of restrictive in that it returns a list of pids, and then supports random accesses by pid.
We could rework the SIGAR API if there was a significant benefit from Process32, AFAIK they both provide the same functionality.

Re: Process32, the PROCESSENTRY32 struct does include szExeFile, which takes some of the guesswork out of determining the exe name by way of the first module (I'm not sure if that technique is portable).

There doesn't seem to be a canonical way to get the list of arguments for a process.

Just as a note, its possible to get the command line arguments via WMI along with (I think) the rest of the process information. Might be something to think about for the future, especially because a list of a dozen svchost.exe processes isn't super useful unless you can also see the command line arguments.
+1 LGTM

Yeah I agree with everything you've said. On Raymond Chen's blog post, its worth noting that unix has the same problem - a process can overwrite the memory containing its command line arguments.
WMI+COM is ugly for sure, but at least it means not spelunking around various levels of winapi calls and dealing with portability issues (e.g. PSAPI v1 and v2). Happy to revisit/sync on this when you're ready.


add option to pass existing MongoClient instance to MongoObserver

The point of this is to allow one to avoid creating redundant instances of MongoClient.
The pymongo docs briefly mention here that this should be avoided.

Coverage decreased (-0.05%) to 83.478% when pulling 38bbb7f23dacdde654b13f59f43500a8178f3e4f on rueberger:master into 5bbdf20bb2725b4d1e2f49af66ef4b55447d2a26 on IDSIA:master.

Create this client once for each process, and reuse it for all operations. It is a common mistake to create a new client for each request, which is very inefficient.

Hey, I guess you are referring to this line, which I interpret as an efficiency problem if you create lots of clients. So I'm fine with adding the capacity to pass in a client, but I'm not sure how critical it is to avoid multiple clients in the context of sacred.
One thing before I merge though: Could you add a check to make sure url and client are not being passed in together?

No, I think this is ok. Thanks.


koboldcpp: 1.78 -> 1.79.1

Changelog: https://github.com/LostRuins/koboldcpp/releases/tag/v1.79.1
Diff: https://github.com/LostRuins/koboldcpp/compare/v1.78...v1.79.1
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[x] x86_64-darwin
[x] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 361463 --extra-nixpkgs-config '{ allowAliases = false; allowBroken = false; allowUnfree = true; checkMeta = true; cudaSupport = true; cudaCapabilities = [ "7.5" ]; }'

x86_64-linux

:white_check_mark: 1 package built:

koboldcpp


Feature: pythreejs controls

@mgeier: I gave you credits for the 2nd commits.
Replaces #273 by allowing a pythreejs Controls. Example usage:
import pythreejs
import ipyvolume as ipv
import numpy as np
fig = ipv.figure()
ipv.plot_trisurf([0, 0, 3., 3.], [0, 4., 0, 4.], 2,
           triangles=[[0, 2, 3], [0, 3, 1]])
ipv.show()

control = pythreejs.OrbitControls(controlling=fig.camera)
fig.controls = control
control.autoRotate = True
# the controls does not update itself, but if we toggle this setting, ipyvolume will update the controls
# fig.render_continuous = True

Coverage increased (+0.0009%) to 66.223% when pulling e129a6b8caeb4f08f0059af5b5d754cdf6bb43aa on feat_pythreejs_controls into fad27f945f83ead65bd06675aa4c7f08c8979269 on master.

Do I understand this correctly:
This allows users to explicitly specify (properly functioning) pythreejs.OrbitControls, but it doesn't change anything in the default behavior?
If yes, this is not at all what I intended to do in #273.
I wanted the default to be reasonable.
And in the long run, I wanted to try to make "orbit" controls the default, because I don't like this "trackball" thing.
Being able to explicitly specify OrbitControls is a great feature, but something quite different.

This allows users to explicitly specify (properly functioning) pythreejs.OrbitControls, but it doesn't change anything in the default behavior?

yes.
Commented noted, I think this gives users more flexibility, without changing the current behavior, which I think is fine.


Header: Add guards for building with clang

Clang (NDK) fails when building the inline assembly
asm("DIVQ %[d]"
: "=a" (q)
: [d] "r" (d), "a" (UINT64_C(0)), "d" (u_hi));
with this error:
include/fxdiv.h:258:13: error: invalid output size for constraint '=a'
: "=a" (q)
Fix this by guarding against clang
Note:
Unfortunately I believe clang defines GNUC which makes this necessary.
Here is a reference that I found useful to know what macros are defined by each of the compilers:
http://nadeausoftware.com/articles/2012/10/c_c_tip_how_detect_compiler_name_and_version_using_compiler_predefined_macros

@kmonian Thank you. However, I can't replicate the original problem with compilation with Clang (neither with Android NDK r13b, nor with Apple Clang on Mac, nor with Clang 3.8 on Ubuntu). Which Clang version did you use?

I was using clang from NDK r12b.

@Maratyszcza, to clarify this further, the build works fine when build FXDiv itself, but fails when build NNPack (using clang included in ndk12b). Please let me know if you need any other info from me.

For some more context, this fails when compiling convolution-inference.c in the NNPack library.

I tried NDK r12b, but didn't see the problem either. In all cases I build NNPACK without any problems. Do you use ndk-build to build NNPACK?

I am using BUCK to build this, but I don't think that should make a difference. Are there any special flags you are passing in when building this?

@kmonian This looks like a bug in BUCK. Maybe it defines __x86_64__ when compiling for 32-bit x86?

FXdiv by itself doesn't require or expect any special command-line arguments for compilation.

@Maratyszcza, I got a chance to check this and you were right. x86_64 was being defined (although not due to a BUCK bug). Removing that fixes this issue. Thanks for your help on this!


Updated release note for April release

Docs Build status updates of commit 6469b90:
:clock10: Full build: the average full build time is 12 min(s) 54 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 6469b90:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/docs-conceptual/exchange-online/exchange-online-powershell-v2/exchange-online-powershell-v2.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit fff8706:
:clock10: Full build: the average full build time is 12 min(s) 55 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit fff8706:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/docs-conceptual/exchange-online/exchange-online-powershell-v2/exchange-online-powershell-v2.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 430aad3:
:clock10: Full build: the average full build time is 12 min(s) 58 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 430aad3:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/docs-conceptual/exchange-online/exchange-online-powershell-v2/exchange-online-powershell-v2.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 0d177e6:
:clock10: Full build: the average full build time is 12 min(s) 59 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 0d177e6:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/docs-conceptual/exchange-online/exchange-online-powershell-v2/exchange-online-powershell-v2.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 918972d:
:clock10: Full build: the average full build time is 13 min(s) 1 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 918972d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/docs-conceptual/exchange-online/exchange-online-powershell-v2/exchange-online-powershell-v2.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


chore: upgrade to protobuf.js 7.2.5

Requested in https://github.com/signalfx/splunk-otel-js/issues/768
Should fix a few CVEs

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

Comparison is base (f299105) 86.22% compared to head (0c07a27) 86.22%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #859   +/-   ##
=======================================
  Coverage   86.22%   86.22%
=======================================
  Files          19       19
  Lines         668      668
  Branches      153      153
=======================================
  Hits          576      576
  Misses         92       92

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


DenoflareResponse.statusText not implemented

It's impossible to use Response.statusText as it's not implemented:
https://github.com/skymethod/denoflare/blob/75ac0717ec34f1ed500c3605ef068394646a7486/common/denoflare_response.ts#L75C2-L81C6

I think it's safe to return an empty string as per https://developer.mozilla.org/en-US/docs/Web/API/Response/statusText

I can plumb it through rpc like the others - curious if you were using it explicitly or just enumerating through all response properties

I used it for error logging, like this:
https://github.com/worker-tools/router/blob/5e5a96b33178567c535f99644522e562f1a55947/index.ts#L142C1-L149C4

Ok, made it available as of a284d0f9730c38b662acda6fe4692ac22f7c176d


Test Project does compile (Delphi)

The Test project does not build, Unit "HlpIHash" is missing (used by ClpIDigest, Delphi 11).
Any Help ?

Hello,
It does compile just fine.
You need to include all necessary directories in your search path. Either project or Delphi Library Path.
Regards,
Ertan
Sent from my iPhone

On 2 Nov 2022, at 07:56, Laurent MEYER @.***> wrote:

The Test project does not build, Unit "HlpIHash" is missing (used by ClpIDigest).
Any Help ?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.

Ertan,
Would you provide a simple example of signing and verifying with ED25519 :
A Key (private) and a payload as input. If it works I am willing to pay…
Thank you,
Please let me know.
Laurent
From: ertankucukoglu @.>
Sent: Wednesday, November 2, 2022 6:06 AM
To: Xor-el/CryptoLib4Pascal @.>
Cc: Laurent MEYER @.>; Author @.>
Subject: Re: [Xor-el/CryptoLib4Pascal] Test Project does compile (Delphi) (Issue #34)
Hello,
It does compile just fine.
You need to include all necessary directories in your search path. Either project or Delphi Library Path.
Regards,
Ertan
Sent from my iPhone

On 2 Nov 2022, at 07:56, Laurent MEYER @.***> wrote:

The Test project does not build, Unit "HlpIHash" is missing (used by ClpIDigest).
Any Help ?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.

—
Reply to this email directly, view it on GitHub https://github.com/Xor-el/CryptoLib4Pascal/issues/34#issuecomment-1299576611 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AHYQMSSW23MERX3G5VPL6BLWGHZB7ANCNFSM6AAAAAARUWBDM4 .
You are receiving this because you authored the thread.Message ID: @.***>

Hello,
I'm sure library can do that. Unfortunately I don't know how.
However, you might want to contact with Xor-el as he can help you out.
Regards,
Ertan
Sent from my iPhone

On 2 Nov 2022, at 08:14, Laurent MEYER @.***> wrote:

Ertan,
Would you provide a simple example of signing and verifying with ED25519 :
A Key (private) and a payload as input. If it works I am willing to pay…
Thank you,
Please let me know.
Laurent
From: ertankucukoglu @.>
Sent: Wednesday, November 2, 2022 6:06 AM
To: Xor-el/CryptoLib4Pascal @.>
Cc: Laurent MEYER @.>; Author @.>
Subject: Re: [Xor-el/CryptoLib4Pascal] Test Project does compile (Delphi) (Issue #34)
Hello,
It does compile just fine.
You need to include all necessary directories in your search path. Either project or Delphi Library Path.
Regards,
Ertan
Sent from my iPhone

On 2 Nov 2022, at 07:56, Laurent MEYER @.***> wrote:

The Test project does not build, Unit "HlpIHash" is missing (used by ClpIDigest).
Any Help ?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.

—
Reply to this email directly, view it on GitHub https://github.com/Xor-el/CryptoLib4Pascal/issues/34#issuecomment-1299576611 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AHYQMSSW23MERX3G5VPL6BLWGHZB7ANCNFSM6AAAAAARUWBDM4 .
You are receiving this because you authored the thread.Message ID: @.***>
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.

Hi,
Any idea to how contact him in private ?
Lo.

Hi,
He should see these messages soon and get in touch with you. Just wait a
day or two.
Regards,
Ertan
Laurent MEYER @.***>, 2 Kas 2022 Çar, 08:34 tarihinde
şunu yazdı:

Hi,
Any idea to how contact him in private ?
Lo.
—
Reply to this email directly, view it on GitHub
https://github.com/Xor-el/CryptoLib4Pascal/issues/34#issuecomment-1299598699,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADQ6UEFAGT5YRR57RFFUCI3WGH4PPANCNFSM6AAAAAARUWBDM4
.
You are receiving this because you commented.Message ID:
@.***>

Hello @bnzbnz maybe you can provide sample key (public and private) and payload you wish to operate on.
You can also do same via mail as I have responded to you there too.

Here you are !
Private:
MC4CAQAwBQYDK2VwBCIEIBL22qIQTR8IhjsW85lc31qzZLDyNtKQClHqiNbRddEy
Public:
MCowBQYDK2VwAyEAqNtjQAGYGyVuhZoADbDu32eKsMk/0IY96RNWIVHhWpk=
Payload:
aaaa
Thank you so much.
From: Ugochukwu Mmaduekwe @.>
Sent: Wednesday, November 2, 2022 9:56 AM
To: Xor-el/CryptoLib4Pascal @.>
Cc: Laurent MEYER @.>; Mention @.>
Subject: Re: [Xor-el/CryptoLib4Pascal] Test Project does compile (Delphi) (Issue #34)
Hello @bnzbnz https://github.com/bnzbnz  maybe you can provide sample key (public and private) and payload you wish to operate on.
You can also do same via mail as I have responded to you there too.
—
Reply to this email directly, view it on GitHub https://github.com/Xor-el/CryptoLib4Pascal/issues/34#issuecomment-1299875730 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AHYQMSXJBULLKL2W22CK2PDWGIUA5ANCNFSM6AAAAAARUWBDM4 .
You are receiving this because you were mentioned.Message ID: @.***>

It’s also what I saw but
they come directly from ebay, created by eBayR.API.KeyManagement.createSigningKey with a signing  cipher = ‘ED25519’ (Edwards Curve).
From: Ugochukwu Mmaduekwe @.>
Sent: Wednesday, November 2, 2022 10:21 AM
To: Xor-el/CryptoLib4Pascal @.>
Cc: Laurent MEYER @.>; Mention @.>
Subject: Re: [Xor-el/CryptoLib4Pascal] Test Project does compile (Delphi) (Issue #34)
@bnzbnz https://github.com/bnzbnz  thanks.
taking a quick look at the keys you shared, are you sure they are ED25519 RAW Keys?
ED25519 Private Raw Keys are 32 bytes in length, yours when I decode the base64 string as hex is not 32 bytes in length.
How did you generate your keys?
—
Reply to this email directly, view it on GitHub https://github.com/Xor-el/CryptoLib4Pascal/issues/34#issuecomment-1299922982 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AHYQMSWWIKBISJDS5UFQIBLWGIW63ANCNFSM6AAAAAARUWBDM4 .
You are receiving this because you were mentioned.Message ID: @.***>


Let the user choose the resolution of images he/she sends

Newer cameras can produce images up to 20MB large.
Let the user choose the resolution of images he/she sends on mobile networks, with a minimum, to ensure the image is legible.

@teolemon I would like to work on this issue

Ya, I think it's a good idea. I would love to work on this.

I will like to work on it.


Adding in content from nether-deploy-analytics-ingest, but without the web job

Adding in content from nether-deploy-analytics-ingest, but without the web job.

[ ] Tick here to confirm that you have run RunCodeFormatter.ps1

Description:
Added in content from the nether-deploy-analytics-ingest, but not the components related to the web job (coming later).
Test:
Deploy the nether-deploy-quickstart.json template as a Custom Template through the Azure Portal and the event hub and storage account resources should be created for analytics use.
Make sure that all above information is provided before submitting your pull request)

@gohlivor,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.
It will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot

@gohlivor, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, Microsoft Pull Request Bot

After reviewing the proposed changes, I'll close this PR without merging the changes. We'd be happy to review another PR again that is more in line with the changes we want to make. Thanks for your support.


Incorrect compatibility statement for .Net 4,8

This is not true..

Windows Versions
https://github.com/Microsoft/dotnet-framework-early-access
The .NET Framework Build 3745 can be installed on the following Windows versions.
Client
Windows 10, version 1809
Windows 10, version 1803
Windows 10, version 1709
Windows 10, version 1703
Windows 10, version 1607
Windows 8.1
Windows 7 SP1

Server
Windows Server 2019
Windows Server, version 1803
Windows Server, version 1709
Windows Server 2016
Windows Server 2012
Windows Server 2012 R2
Windows Server 2008 R2 SP1

I'll remove, or just say for Windows Server 2008 R2/ 7 SP 1 onwards. I think I wrote it for the first 4.8 build.


Updated README with proposed process, members, and other info

This PR updates the README based on #6 , and also adds other information that seems appropriate for our first landing page.

As mentioned here, I would appreciate it if we would stimulate the use of this repository for collaboration on drafts too.
Perhaps you could consider adding a link to a separate document in which we describe how to use the GitHub interface, for those who are inclined to do so.

@jfjlaros : I think we would collaborate in the repo on drafts, but do so in branches that are merged as PRs. Does that satisfy your request? If not, can you please sketch out an alternative proposal?

Does that proposal satisfy your request?

As long as we document it. I am afraid that novice users will get lost very quickly otherwise.
I also presume that we want the repository to have some form of structure, maybe we can use
PR templates to guide contributors.

I merged this PR, with comments from several people and one approval, in the belief that it was a concrete step forward from the existing documentation. There were good suggestions in the comments and these should be taken up as separate PRs.
We should also agree on policies and norms for PR evaluation and merges.


Add "shared-mime-info"

This is especially for freedesktop.org.xml, which is needed for a recent change to the mimemagic Ruby Gem (due to licensing incompatibility).
See https://github.com/docker-library/ruby/issues/344 for more details (https://github.com/mimemagicrb/mimemagic/issues/97 -> https://github.com/mimemagicrb/mimemagic/commit/cd65290a28e58974ff0c5fa1d3f2ce858035f177).

Given this is already part of Ruby's non-slim variants, I'm more on the fence here.  However, looking at https://github.com/docker-library/buildpack-deps/runs/2157573634?check_suite_focus=true#step:8:19 vs https://github.com/docker-library/buildpack-deps/runs/2194879926?check_suite_focus=true#step:8:9, we must already have all the dependencies of this package, because there's literally no discernible difference in the final image size (803MB vs 803MB), so it's a pretty harmless add from what I can see.

Ahhh, the reason there's no size change is because this is already included.  Should've started by checking that. :smile:


Ability to set the active sheet index upon export

Adding that function:
public function setActiveSheetIndex($index)
{
    $this->sheet = $this->excel->setActiveSheetIndex($index);

    return $this;
}

makes it possible to do:
return Excel::create('filename', function ($excel) {
    // Creating one sheet
    $excel->sheet(...);
    // Creating a second sheet
    $excel->sheet(...);
    // Selecting the first sheet so when you open the Excel file the first sheet is selected by default instead of the last one generated
    $excel->setActiveSheetIndex(0);
})->download('xlsx');

I've tried to use:
return Excel::create('filename', function ($excel) {
    // Creating one sheet
    $excel->sheet(...);
    // Creating a second sheet
    $excel->sheet(...);
    $excel->getExcel()->setActiveSheetIndex(0);
})->download('xlsx');

but this doesn't work.
It only sets the property _activeSheetIndex properly on the PHPExcel object within the excel attribute on the LaravelExcelWriter object, which here is $excel.
The new function does both at the same time.
I couldn't find a way to achieve this out of the box. Is there one already?

Thanks man! 🎉
Is there a tag/release I can use for that?

Will tag it after I checked the other PR.

RAD
On 30 November 2016 at 13:36:22, Patrick Brouwers<EMAIL_ADDRESS>wrote:
Will tag it after I checked the other PR.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Maatwebsite/Laravel-Excel/pull/987#issuecomment-263874714,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAozA7gwdFcwnyJuouoxh9XaYDCQh9QVks5rDXvVgaJpZM4Kxg4z
.


React 16: Stateless function components cannot be given refs

I'm using the latest dnd v2.5.4 and still getting a console error for calls to DragLayer, which is set up in my app as a stateless function component. This is very similar to #896, which concerned DragDropContext
Log:
warning.js:33 Warning: Stateless function components cannot be given refs. Attempts to access this ref will fail.

Check the render method of `DragLayer(SitePageCustomDragLayer)`.
    in SitePageCustomDragLayer (created by DragLayer(SitePageCustomDragLayer))
    in DragLayer(SitePageCustomDragLayer) (created by SitePageContainer)
    in div (created by PanelBody)
    in PanelBody (created by SitePageContainer)

I too am still seeing the Stateless function components cannot be given refs. warning when using the following helper in my tests:
const wrapInTestContext = (WrappedComponent) => {
  const TestContextWrapper = props => (
    <Provider store={store}>
      <StaticRouter context={context}>
        <WrappedComponent {...props} />
      </StaticRouter>
    </Provider>
  );

  return DragDropContext(TestBackend)(TestContextWrapper);
};

Defining TestContextWrapper as a class removes the warning.

https://github.com/acdlite/recompose/blob/master/docs/API.md#toclass works great as workaround.

@bkniffler could you please give an example ?

You may have to rewrite it to a class.

https://hackernoon.com/react-stateless-functional-components-nine-wins-you-might-have-overlooked-997b0d933dbc

I have the same warning with DragDropContext()
Warning: Stateless function components cannot be given refs. Attempts to access this ref will fail.

Check the render method of `DragDropContext(WorkspaceList)`.
    in WorkspaceList (created by DragDropContext(WorkspaceList))
    in DragDropContext(WorkspaceList) (created by provideMountCallbacks(DragDropContext(WorkspaceList)))
    in provideMountCallbacks(DragDropContext(WorkspaceList)) (created by Connect(provideMountCallbacks(DragDropContext(WorkspaceList))))
    in Connect(provideMountCallbacks(DragDropContext(WorkspaceList))) (created by IndexContainer)
    in IndexContainer (created by Connect(IndexContainer))
    in Connect(IndexContainer) (created by RouterContext)
    in div (created by App)
    in LoadingContent (created by App)
    in div (created by App)
    in LocaleProvider (created by App)
    in App (created by Connect(App))
    in Connect(App) (created by Connect(Connect(App)))
    in Connect(Connect(App)) (created by RouterContext)
    in RouterContext (created by Router)
    in Router
    in Provider

Guess I have to change WorkspaceList to class...

@DanielRuf sure, and need to mention that if you don't need to setup state or bind callbacks you don't have to include constructor

I know but I've added it to prepare next versions which will use some config props.
It's also just there for reference, it does not hurt.

I'm having this issue when using redux, which wraps another wrapper:
DragDropContext(HTML5Backend)(connect(mapStateToProps, mapDispatchToProps, null, { withRef: true })(withRouter(FiltersSideBar)));
No matter who wraps who I get this error.
Any ideas?

I still get this error, how to fix it without changing the entire codebase and replacing stateless components with classes?

Still, I hope sometime there will be a fix for this issue, but for now using recompose toClass seems like the easiest way (and easiest to get rid of as soon as the issue is resolved). Borrowing @tough-griff example:
import { toClass } from 'recompose';

const wrapInTestContext = (WrappedComponent) => {
  const TestContextWrapper = toClass(props => (
    <Provider store={store}>
      <StaticRouter context={context}>
        <WrappedComponent {...props} />
      </StaticRouter>
    </Provider>
  ));

  return DragDropContext(TestBackend)(TestContextWrapper);
};

This is intended. You should change it to a class.

You can wrap your functional component with a div and give a ref to that div

What about wrapping it in a React.Fragment component?
It should work.

#1280 in the recent 7.4.0 made the issue appear for us with DragDropContext
We had DragDropContext decorating a function component, it worked fine so far, and now spits a "Function components cannot be given refs" error.

The warnings have been removed in a patch version. You can use React.forwardRef to get actual ref instances.


[BugFix] Check invalid window argument

What type of PR is this：

[x] bug
[ ] feature
[ ] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

Fixes https://github.com/StarRocks/starrocks/issues/5935
Problem Summary(Required) ：

Report Error message for nagative value of window.

[FE PR Coverage check]
:disappointed: fail : 6 / 8 (75.00%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/sql/analyzer/FunctionAnalyzer.java
6
8
75.00%

[FE PR Coverage check]
:heart_eyes: pass : 7 / 8 (87.50%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/sql/analyzer/FunctionAnalyzer.java
7
8
87.50%

[FE PR Coverage check]
:disappointed: fail : 6 / 8 (75.00%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/sql/analyzer/FunctionAnalyzer.java
6
8
75.00%

[FE PR Coverage check]
:heart_eyes: pass : 7 / 8 (87.50%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/sql/analyzer/FunctionAnalyzer.java
7
8
87.50%

[FE PR Coverage check]
:heart_eyes: pass : 7 / 8 (87.50%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/sql/analyzer/FunctionAnalyzer.java
7
8
87.50%

run starrocks_fe_unittest

[FE PR Coverage check]
:heart_eyes: pass : 8 / 8 (100.00%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/sql/analyzer/FunctionAnalyzer.java
8
8
100.00%

@Mergifyio backport branch-2.3

run starrocks_admit_test

run starrocks_admit_test

run starrocks_fe_unittest

run starrocks_admit_test

run starrocks_admit_test

run starrocks_fe_unittest

run starrocks_admit_test

run starrocks_admit_test

[FE PR Coverage Check]
:heart_eyes: pass : 8 / 8 (100.00%)
file detail

path
covered line
new line
coverage
not covered line detail

:large_blue_circle:
com/starrocks/sql/analyzer/FunctionAnalyzer.java
8
8
100.00%
[]

run starrocks_admit_test


display secondary structure

The end user would be able to see a graphical display of computed secondary structure (helices, beta strands, coils) for the reference sequence graphically above the MSA block. (or in a location parallel to the sequence logo)

codemill$1