SharePoint/ Spfx

Thank you for reporting an issue or suggesting an enhancement. We appreciate your feedback - to help the team to understand your needs, please complete the below template to ensure we have the necessary details to assist you.
(DELETE THIS PARAGRAPH AFTER READING)

Category

[x] Question
[ ] Typo
[ ] Bug
[ ] Additional article idea
[x] Example checked item (delete this line)

For the above list, an empty checkbox is [ ] as in [SPACE]. A checked checkbox is [x] with no space between the brackets. Use the PREVIEW tab at the top right to preview the rendering before submitting your issue.
If you are planning to share a new feature request (enhancement / suggestion), please use SP Dev UserVoice at http://aka.ms/sp-dev-uservoice. (DELETE THIS PARAGRAPH AFTER READING)
(DELETE THIS PARAGRAPH AFTER READING)

Expected or Desired Behavior

If you are reporting a bug, please describe the expected behavior. If you are suggesting an enhancement please describe thoroughly the enhancement, how it can be achieved, and expected benefit.
(DELETE THIS PARAGRAPH AFTER READING)

Observed Behavior

If you are reporting a bug, please describe the behavior you expected to occur when performing the action. If you are making a suggestion, you can delete this section.
(DELETE THIS PARAGRAPH AFTER READING)

Steps to Reproduce

If you are reporting a bug please describe the steps to reproduce the bug in sufficient detail to allow testing. Only way to fix things properly, is to have sufficient details to reproduce it. If you are making a suggestion, you can delete this section.
(DELETE THIS PARAGRAPH AFTER READING)

Submission Guidelines

All suggestions or bugs are welcome, please let us know what's on your mind.
If you are reporting an issue around any of the documents or articles, please ensure that you have clear > reference on the specific file or URL, which should be fixed.
If you have technical questions about the framework, we’ll be monitoring #spfx, #spfx-webparts, and > #spfx-tooling on (SharePoint StackExchange)[http://sharepoint.stackexchange.com/]. You can also > alternatively submit your question to (SharePoint Developer group)> [https://network.office.com/t5/SharePoint-Developer/bd-p/SharePointDev] at Office Network.
Remember to include sufficient details and context.
If you have multiple suggestions or bugs please submit them in separate bugs so we can track resolution.

(DELETE THIS PARAGRAPH AFTER READING)

Thanks for your contribution! Sharing is caring.

Hi All,
I'm facing this error in Visual studio code ...cannot find module './helloworldwebpart.module.scss'
please do needful

I'm sorry there is not enough information provided to help you. Please update your original issue with the template that was provided.
Also please provide

What version of SPFx you are using
What version of SharePoint your SPFx solution is targeting
What version of Node you are using.

Naively speaking, there is your solution is unable to find your sass module. Either it's missing, or isn't bundling correctly. Please update your ticket with the requested information.

Closing this as there's no way to act on this, nor does the OP contain any of the minimally required detail to help. Please feel free to open a new issue with relevant information.


Fixed view event from clicking on map bug

Fixed view event from clicking on map bug
also idk if this went through in a previous pr, but I finished the views and adapters on view event and filter event

Code Review:
Good job Naoroj, this looks great, but I have a few comments.
Coding style looks good, but please try to add more documentation when making changes. I don't really see any comments or any JavaDocs.
However, I think the correctness is good, as the code works as intended without introducing any bugs.
Another issue is that I don't see any tests or test coverage. In the future, please include these.
You did a good job adhering to Clean Architecture and SOLID, and describing what you fixed in the pull request title.


Some small improvements to SiestaUI proposed in #131 and in #155

added an optional image transform to RemoteImageView applied on both placeholder and downloaded image see #131
added a reference to the retry button of ResourceStatusOverlay (for translation) see #155

Looks good. Sorry for the slow follow-up — and congratulations! This is comment number 1000!
Fixes #131. Fixes #155.


Netlify Extension: update icon

Description
New Icon for Netlify
Screencast

Checklist

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

@peduarte Very snazzy! I see icon.png was removed, however that flat logo is still in use as the icon for a few of the Action menus. Maybe keep that png around?

@jasonbarry good shout, I overlooked this. Normally extension icons are only used for that! Will add it back. Thanks for the heads up!

@jasonbarry all done, to avoid confusion, I called the extension icon extension-icon which is a pattern we try to follow anyway, and the other one netlify-icon. Cheers!

Cool, just waiting for final confirmation from Nichlas.

Updated icon by our designer


[Feature Request]: Support Diagnostics.Metrics instead of prometheus-net

Is your request related to a problem you have?
N/A
Describe the solution you'd like
What about supporting Diagnostics.Metrics instead of prometheus-net? Using Metrics we are able to consume from OpenTelemetry for example
Are you able to help bring it to life and contribute with a Pull Request?
Yes
Additional context
No response

Hi @alefcarlos,
exporting metrics using OpenTelemetry is already in our backlog and will be included soon :)

Hi, do you already know when it will be implemented?


Secp256k1: Encoding special case for identity

The uncompressed SEC-1 v2 encoding specifies 0x00 as encoding for the identity. The publicKeyFromEncoded() and PublicKey::toEncoded() functions to not implement this special case.

Fixed via https://github.com/pmerkleplant/crysol/commit/fe723e3bdab8731fecf753aef48610d38a141383.


[Question]: Update the base docker image packages, especially azure-cli

Describe your question
Hello team,
Not a bug and not a feature, but an ask. Could someone please update the packages inside the ubuntu docker image used for the agent? It's woefully out of date and while one might argue that most of the outdated packages are irrelevant, there are some which are notable, for example azure-cli, which is still on some 2.0 version from 2019. Things such as federated logins do not work on this version and it is very cumbersome to have to manually go in and update all packages after updating the image tags to newer agent versions.
With kind regards!
Versions
All docker versions of the agent
Environment type (Please select at least one enviroment where you face this issue)

[ ] Self-Hosted
[ ] Microsoft Hosted
[ ] VMSS Pool
[x] Container

Azure DevOps Server type
dev.azure.com (formerly visualstudio.com)
Operation system
No response
Version controll system
No response
Azure DevOps Server Version (if applicable)
No response

@bufnitza, could you specify which Ubuntu image you are referring to? I assume it's when attempting to run the Azure Pipelines Agent in the docker container, correct?

@ivanduplenskikh I'm referring to the images built by Microsoft and pushed to mcr.microsoft.com/azure-pipelines/vsts-agent
After opening this ticket, I also bumped into the article you shared and after discussing with the team, we'll probably switch to our locally built agents, as per those instructions. The above still remains valid, but I'm not sure what the support policy is for those images on the MCR.


Support Purescript

Ended up here from https://github.com/purescript/purescript/issues/2289. Your plugin seems like a nice idea for me :-) So I'm opening this feature request.

I've tried to add support locally since it should have been a trivial copy of the Haskell/Elm implementation but it seems to enter in conflict with the well known https://github.com/raichoo/purescript-vim.

I've discovered there's a new fix for such cases of wishing to apply conceal over syntax groups through :call matchadd('Conceal', '\', 10, 99, {'conceal': 'λ'}), for example, Maybe it should be adopted in the implementation of the functionality of this plugin.

FYI, this is how I'm applying the lambda conceal on my .vimrc now:
set conceallevel=2
au FileType purescript set concealcursor=vin
au WinEnter,BufEnter,BufRead,FileType,Colorscheme *
    \ if exists('w:lambda_conceal')                                                  |
    \     call matchdelete(w:lambda_conceal)                                         |
    \     unlet w:lambda_conceal                                                     |
    \ endif                                                                          |
    \ if &ft == 'purescript'                                                         |
    \     let w:lambda_conceal = matchadd('Conceal', '\\', 10, -1, {'conceal': 'λ'}) |
    \     hi! link Conceal Operator                                                  |
    \ endif


[SPARK-31777][ML][PySpark] Add user-specified fold column to CrossValidator

What changes were proposed in this pull request?

This patch adds user-specified fold column support to CrossValidator. User can assign fold numbers to dataset instead of letting Spark do random splits.
Why are the changes needed?

This gives CrossValidator users more flexibility in splitting folds.
Does this PR introduce any user-facing change?

Yes, a new foldCol param is added to CrossValidator. User can use it to specify custom fold splitting.
How was this patch tested?

Added unit tests.

Test build #123426 has started for PR 28704 at commit baec279.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/28043/
Test PASSed.

Test build #123426 has finished for PR 28704 at commit baec279.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/123426/
Test PASSed.

cc @mengxr @srowen

For Python side, I think to add it in a follow-up.

I'm not super against handling Python separately, but it should be a small change, and logically belongs here. I know we've separated them in the past, and I know we usually get around to adding it, but, I'd lightly prefer doing it all in one go.

Ok, sounds good to me. I'm adding a new commit for above comments. Once you think Scala side is good, I will add python change.

Test build #123442 has started for PR 28704 at commit f34ab0d.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/28059/
Test PASSed.

Test build #123444 has started for PR 28704 at commit 786abaa.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/28061/
Test PASSed.

Test build #123442 has finished for PR 28704 at commit f34ab0d.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/123442/
Test FAILed.

Test build #123444 has finished for PR 28704 at commit 786abaa.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/123444/
Test PASSed.

Test build #123463 has started for PR 28704 at commit 244befc.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/28083/
Test PASSed.

Test build #123463 has finished for PR 28704 at commit 244befc.

This patch fails Python style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/123463/
Test FAILed.

Python change was added.

Test build #123464 has started for PR 28704 at commit 9c8658b.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/28084/
Test PASSed.

Test build #123464 has finished for PR 28704 at commit 9c8658b.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/123464/
Test FAILed.

Merged build finished. Test FAILed.

retest this please

Test build #123470 has started for PR 28704 at commit 9c8658b.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/28091/
Test PASSed.

Test build #123470 has finished for PR 28704 at commit 9c8658b.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/123470/
Test FAILed.

retest this please

Test build #123489 has started for PR 28704 at commit 9c8658b.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/28112/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/123489/
Test PASSed.

@srowen @huaxingao any more comments for this change that I need to address?

@mengxr you opened the JIRA - any comments? Looks reasonable to me.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/28336/
Test PASSed.

Added fold number check both in Scala and Python.

Test build #123712 has started for PR 28704 at commit aa7c8d0.

Test build #123712 has finished for PR 28704 at commit aa7c8d0.

This patch fails Python style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/123712/
Test FAILed.

Test build #123713 has started for PR 28704 at commit 218db10.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/28337/
Test PASSed.

Test build #123713 has finished for PR 28704 at commit 218db10.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/123713/
Test PASSed.

@srowen @huaxingao @zhengruifeng thanks for previous review. Do you have more comments for this change?

LGTM

Thanks! I will merge this in one day if no other opinions.

Merging to master!


Optimize oref0-pump-loop, and bug fixes

More refactoring to optimize oref0-pump-loop to run faster, especially on the Pi Zero W.
Also contains some bug fixes, including:

Fix some incorrect "skipping bolus wizard entry with carbs and no insulin" messages where the bolus wizard and corresponding bolus record are logged with timestamps that differ by 1 second.  The fix in commit 2eee14b6e44493e9f0442f6b33e980480290a325 is to look for records up to 1s apart.
Fix the calculation of bwCarbs: bolus wizard carbs weren't being counted as either bwCarbs or nsCarbs, so oref0 was not properly disabling SMB for A52 risk reduction.  Fixing that also required fixing the variable reference to profile.A52_risk_enable.

Added some bug fixes: see updated description above.

Is this branch ready to test?  www/app.pyc should not be added to version control.

pump loop won't resolve errors and keeps mmtuning,  log excerpt
Preflight OK. Profile less than 60m old and valid. RefreshValueError: need more than 0 values to unpack

module.js:340
    throw err;
          ^
Error: Cannot find module '/root/myopenaps/monitor/pumphistory-merged.json'
    at Function.Module._resolveFilename (module.js:338:15)
    at Function.Module._load (module.js:280:25)
    at Module.require (module.js:364:17)
    at require (module.js:380:17)
    at Object.<anonymous> (/root/src/oref0/bin/oref0-calculate-iob.js:43:18)
    at Module._compile (module.js:456:26)
    at Object.Module._extensions..js (module.js:474:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Function.Module.runMain (module.js:497:10)

Couldn't calculate IOB
oref0-pump-loop failed. pump_loop_completed more than 15m old; waiting for 40s silence before mmtuning
Radio ok. Listening: .No interfering pump comms detected from other rigs (this is a good thing!)

@PieterGit I just committed 134458cffc74664a5d829ebf4d5ccd172a64275e which might help with your pumphistory-merged.json issue.

Does anyone else want to test anything specific on this before I merge it to dev?  If not, I think it's about ready.


Return same refresh_token unless provided new one

In some cases (e.g. google) the server does not return a refresh token. Since oauth2 has one struct that represents both refresh_token and access_token, it makes sense to return the same refresh token unless overridden by the response.

Coverage decreased (-1.02%) to 91.713% when pulling 44d1f0cb5ec94e83bf5e72de4004131634db34b9 on tsubery:master into 75c397e2291f557ddaf5744904b6ef4fe510afb2 on scrogson:master.

Coverage decreased (-1.02%) to 91.713% when pulling 44d1f0cb5ec94e83bf5e72de4004131634db34b9 on tsubery:master into 75c397e2291f557ddaf5744904b6ef4fe510afb2 on scrogson:master.

👍

Thank you!


Ability to add comments when editing/appending in a spreadsheet

Would be nice to use the same sheets api to add comments by doing something like this
google.sheets.spreadsheets.values.append({
  resource: {
    values: [
      [
        { value: 'row cell text', comment: 'comment on cell' }
      ]
    ]
  }
})

where the values are changed from using a string input to an object input. This could also allow for manipulating other properties of the cell like color, etc.
I know this is a feature request for the google sheets api and also might be a pipe dream but I think this ux would open up a lot of possibilities

Here is the form where you can request a new feature in the Google Sheets API.


[build] detox: Command failed, exit code: 65

Hi,
I am trying to follow Readme.md and build application to iOS.
After yarn install I am getting following error message:
[4/4] 📃  Building fresh packages...
[-/4] ⡀ waiting...
[2/4] ⡀ @sentry/cli
[3/4] ⡀ detox
error /Users/vn/go/src/github.com/privacybydesign/irma_mobile/node_modules/detox: Command failed.
Exit code: 65
Command: scripts/postinstall.sh
Arguments:
Directory: /Users/vn/go/src/github.com/privacybydesign/irma_mobile/node_modules/detox
Output:

irma_mobile [master] % node -v
v8.12.0

irma_mobile [master] % yarn -v
1.10.1

irma_mobile [master] % git rev-parse HEAD
4ce95722b16ff8fa4f0bab5762f043fda804a14a

Could you please suggest something for fixing it?
Looks like detox is not yet built, when it trying to use it.

If I execute
irma_mobile [master] % yarn add detox
....
warning "detox" is already in "devDependencies". Please remove existing entry first before adding it to "dependencies".
✨  Done in 101.31s.

and then ... yarn install, everything fine. But still unclear why if it in devDependencies, I need it install separately.

A pretty old version of detox is in master currently, so I can imagine there are some build problems on your system. As detox is only used during testing (and not much in master now), the building of the application should work regardless. I'll try to bump detox soon, and will look if that error pops up again. Thanks for reporting!


Incorrect parsing of items

hey,
every time I open the app in the last couple of days I always get this view:

It's the latest release from the apple app store.
The view of HN Jobs is always the same no matter what topic (New, Top, Ask) I select.
Am I doing something wrong? 8-)

Probably HN changed the website again and the scrapper doesn't like it. You'll have to wait for an updated release on the App Store.
However, from what i got from the latest commits in libHN this should be solved already and quicker to fix next time (i.e. no App Store submission needed).

New update will be submitted today. Thanks for being patient!

Update arrived today  :tada: ! Thank you very much! :+1:

No problem, glad you like it!


File Previewing, missing file formats (Version 1.26.1)

File previewing is a great feature.
But File Previewing does not work for:

csv
file names without extensions

October 2022 (Version 1.26.1)

We fixed a bug to enable previewing files without extensions by adding value "" to the allowed file extension list. We will also enable the preview action if the blob/file has a few well-known text content type (e.g. text/csv).


GPT2LMScorer batching optimization

Following the split of #4, I propose here to optimize the way GPT2LMScorer evaluated multiples sentences by offering the possibility to input the sentences by batches.
I also add several tests specific to this new batching feature in test_gpt2.
I notice that, it is changing the score value, but really at the margin. For instance :
scorer = GPT2LMScorer("gpt2")
sentences = [
    "I am good",
    "I have a big amount of money.",
    "This is the best day of my life.",
    "Let us finish this quickly!"
]
scorer.batch_size = 1
scores_batch_1 = scorer.sentence_score(sentences, reduce='gmean')
print(scores_batch_1)

scorer.batch_size = 4
scores_batch_4 = scorer.sentence_score(sentences, reduce='gmean')
print(scores_batch_4)

Will give :
[0.0023175945971161127, 0.027517564594745636, 0.06669076532125473, 0.00807127170264721]
[0.002317607868462801, 0.02751740626990795, 0.06669124215841293, 0.00807106401771307]

Hi Simone,
Is is still something you are considering to add?

Yes, I'm definitely interested, it's just that I didn't have time yet to review the changes (I'm working on my thesis as well 😬).

@dldk-gael I refactored the in internal API to make this PR easier.
I also added more tests so that we can be more confident that everything works as expected.
If you have time, can you update this PR to make it working with the new API?
I tried to re-implement the batching optimization using the new API and the transformers batch_encoding_plus function, but it's not working yet (it gives the wrong probabilities).
If you want you can try to fix the code below instead.
Thanks!
from typing import *  # pylint: disable=wildcard-import,unused-wildcard-import

import torch
from transformers import AutoTokenizer, AutoConfig, GPT2LMHeadModel
from transformers.tokenization_utils import BatchEncoding

from .abc.transformers import TransformersLMScorer

class GPT2LMScorer(TransformersLMScorer):
    # @overrides
    def _build(self, model_name: str, options: Dict[str, Any]) -> None:
        super()._build(model_name, options)

        # pylint: disable=attribute-defined-outside-init
        self.tokenizer = AutoTokenizer.from_pretrained(
            model_name, use_fast=True, add_special_tokens=False
        )
        # Add the pad token to GPT2 dictionary.
        self.tokenizer.add_special_tokens({'additional_special_tokens': ["<|pad|>"]})
        self.tokenizer.pad_token = "<|pad|>"

        config = AutoConfig.from_pretrained(model_name)
        self.model = GPT2LMHeadModel.from_pretrained(model_name)
        # We need to resize the embedding layer because we added the pad token.
        self.model.resize_token_embeddings(len(self.tokenizer))
        self.model.eval()
        if "device" in options:
            self.model.to(options["device"])

    def _add_special_tokens(self, text: str) -> str:
        return self.tokenizer.bos_token + text + self.tokenizer.eos_token

    # @overrides
    def _tokens_log_prob_for_batch(
        self, text: List[str]
    ) -> List[Tuple[torch.FloatTensor, torch.LongTensor, List[str]]]:
        outputs:List[Tuple[torch.FloatTensor, torch.LongTensor, List[str]]] = []
        if len(text) == 0:
            return outputs

        # TODO: Handle overflowing elements for long sentences
        text = list(map(self._add_special_tokens, text))
        encoding: BatchEncoding = self.tokenizer.batch_encode_plus(
            text, return_tensors="pt",
        )
        with torch.no_grad():
            ids = encoding["input_ids"].to(self.model.device)
            nopad_mask = ids != self.tokenizer.pad_token_id
            logits: torch.Tensor = self.model(ids)[0]

        for sent_index in range(len(text)):
            sent_nopad_mask = nopad_mask[sent_index]
            # len(tokens) = len(text[sent_index]) + 1
            sent_tokens = encoding.tokens(sent_index)[1:]
            # sent_ids.shape = [len(text[sent_index]) + 1]
            sent_ids = ids[sent_index, sent_nopad_mask][1:]
            # logits.shape = [len(text[sent_index]) + 1, vocab_size]
            sent_logits = logits[sent_index, sent_nopad_mask][:-1, :]

            # ids_scores.shape = [seq_len + 1]
            sent_ids_scores = sent_logits.gather(1, sent_ids.unsqueeze(1)).squeeze(1)
            # log_prob.shape = [seq_len + 1]
            sent_log_probs = sent_ids_scores - sent_logits.logsumexp(1)

            sent_log_probs = cast(torch.FloatTensor, sent_log_probs)
            sent_ids = cast(torch.LongTensor, sent_ids)

            output = (sent_log_probs, sent_ids, sent_tokens)
            outputs.append(output)

        return outputs

    # @overrides
    @classmethod
    def _supported_model_names(cls) -> Iterable[str]:
        return GPT2LMHeadModel.pretrained_model_archive_map.keys()

Hi Simone,
Ok, I will update the PR to match the new API today.
I think I found the problem in the code you proposed: you have to remove the score associated with the padding_token_id (in the vocabulary dim) before the logsumexp. I obtain the right score when doing that. So I will simply use your code with the correction in the PR

you have to remove the score associated with the padding_token_id (in the vocabulary dim) before the logsumexp.

Right! Thanks for the catch!

So I will simply use your code with the correction in the PR
Perfect!

Can you can force push the upstream/master into this branch? (make sure to backup the changes).
Your branch has too many commits.
If you prefere to keep your branch, you can create a new branch from the current master, add the changes again and open a new PR instead.
Also the additional tests you add are not necessary, you can remove them.
They are already implemented here.

Ok I go for second option.


Add get_subscription_details command to client

A useful RPC for the client would be get_subscription_details, to check the number of remaining slots and subscription expiry time, as well as to compare it with its local view.

Going to give this a whirl

Closed via #251

Closed via #251


Breaking on permissions

Hello, I can't seem to get passed the permissions step when trying to install your generator
> Setting global permissions to 755 / 644
find: missing argument to `-exec'
events.js:141
      throw er; // Unhandled 'error' event
      ^

Error: Command failed: find . -type f -exec chmod 644 {} \;
find: missing argument to `-exec'

    at checkExecSyncError (child_process.js:464:13)
    at Object.execSync (child_process.js:504:13)
    at module.exports.yo.generators.Base.extend.install (C:\Users\Joe\Sites\test\generator-craftinstall\app\index.js:301:23)
    at C:\Users\Joe\Sites\test\generator-craftinstall\node_modules\yeoman-generator\lib\base.js:429:16
    at processImmediate [as _immediateCallback] (timers.js:383:17)

Any ideas what could be wrong? I'm developing on Win 64, using git bash, if it matters. Thank you!

If you're working locally on a Windows machine, the generator's attempt to set permissions will cause this error. This is due to the app/index.js executing Linux-based permission settings (of course, you won't need to set permissions locally on Windows).
To get around the error, you may simply comment out the lines 297-310 with the comment header "Set permissions on the entire Craft install tree" in the app/index.js file.
Afterward, run 'yo craftinstall' command again and it should complete the generator.


Allow watching a file that didn't exist before

If a watch request fails because the file didn't exist, an OSError is
correctly raised, but the alias remains in the list of requests.
Therefore it isn't possible to catch the exception, and retry the watch
request once the file has been created.
This patch adds the watch request to the request queue after attempting
to call _setup_watch, to ensure it doesn't get left in the queue if
the setup fails.

I think the same should be done in the unwatch() method.
If a watched file is removed, it's not possible to unwatch it because the kernel will raise an OSError. In that case, the request is left around and you can't watch() the file again later.

What I described seems to be a similar issue to the one mentioned in https://github.com/rbarrois/aionotify/issues/2

@rbarrois hi, any chance this could be merged?

Any updates?

Is this project dead? no action for years.

I copy original module and apply all PR changes for my copy. All works fine, but this really sad when i see no progress all this time


vendor gems so app can be used without internet access

I've vendored dependencies so that the app works without internet access.

Also changed the app and specs to expect a 410 GONE on delete, for maximum RESTification.


Why are some clear functions missing?

There are clear functions for clearSelect() and clearWhere(), but why is there no clear functions for orderBy, groupBy, etc.?
Is this something that is planned for the future perhaps?

Because they have not been implemented https://github.com/tgriesser/knex/pull/1912


No 'Cloud' in HOS repo name (SOC-11184)

When trying to deploy a hosdevelcloud8 I see a failure in the step
looking for the repo signing key because it is searching for a key
in directories with 'Cloud' in the name.
However the HOS repo name doesn't have 'Cloud' in it, and hasn't
for a long time if memory serves, since it was renamed during the
final stages of the HOS 8 release.
Instead we should search for "OpenStack" which is common to both
the SOC and HOS repo directory names.

Testing HOS8 devel ==> SOC9 staging in: https://ci.suse.de/blue/organizations/jenkins/openstack-ardana-fergal/detail/openstack-ardana-fergal/36/pipeline/
Testing SOC8 devel ==> SOC8 staging ==> SOC9 staging in https://ci.suse.de/blue/organizations/jenkins/openstack-ardana-fergal/detail/openstack-ardana-fergal/37/pipeline

The HOS8 devel to SOC9 staging job completed successfully, which was the goal of this patch.
The SOC8 devel ==> staging ==> SOC9 staging failed during the SOC8 staging update, but that was after the update changes in this patch were exercised.
As such all 3 code changes have been validated as operating correctly.


API Changes Step 1: Formatting Changes Only

This PR ONLY includes formatting and space changes for multiple files.

Deleting due to unneeded changes.


Feature request: Note range variations: interval, chord, scale

Was wondering if it's possible to constrain the Note range variations with options for interval (e.g., note range 3" chooses only between the original note and its minor 3rd, not all 3 semitones), chord, and/or scale. That would make the note variations a bit more musical.

If you select a scale, note range will pick notes from that. If you want a more musical result you can create a custom scale containing only certain notes, for example root, 3rd, 4th, 5th, leaving out the dissonances like 2nd and 7th.

If you select a scale, note range will pick notes from that. If you want a more musical result you can create a custom scale containing only certain notes, for example root, 3rd, 4th, 5th, leaving out the dissonances like 2nd and 7th.

If you select a scale, note range will pick notes from that. If you want a more musical result you can create a custom scale containing only certain notes, for example root, 3rd, 4th, 5th, leaving out the dissonances like 2nd and 7th.

Good tip with the user scale, thanks! for

If you select a scale, note range will pick notes from that. If you want a more musical result you can create a custom scale containing only certain notes, for example root, 3rd, 4th, 5th, leaving out the dissonances like 2nd and 7th.

Good tip with the user scale, thanks! for

The idea always was to use this feature together with scales. There are two many additional parameters that would be needed to get more control of that.


guarantee that requests are cached and returned using the cache

in relation to #38
requests are now not timeing out anymore, however if the cache would work as intended, second request should be faster.

resolved in a7f3a6f by using redis as cache backend


Enable arm and ppc architectures on Linux

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

As expected, cross-compilation is not working. For the platform (i.e. Linux) in which emulation is available, I switched to emulation, while I had to disable arm64/macOS as it only works in cross-compilation (see issue https://github.com/conda-forge/qpoases-feedstock/issues/3). fyi @fabinsch

@conda-forge-admin, please rerender

Apparently this was good to go but I forgot to merge it.

@conda-forge-admin please rerender


OCP 4.11 install

The pipelines operator was not coming up in my OCP 4.11 install. I had to deinstall the operator, and install it from the OperatorHub. Then, relaunching the install.sh worked.
Did not have time to investigate really the cause, but I thought I'd share the issue.

fixed the issue, coming in PR.

@plewyllie thanks for raising the issue and for solving this in a PR! 👍


Deleting unused samples, fixing tests

@mamaso will merge soon... this fixes the tests that were broken with the last cleanup

Awesome, thanks @fabiocav !


send messages to unsubscribed streams.

Fixes: #21433
Screenshots and screen captures:

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Nice! Playing with the PR, I realized that the "Subscribe" button actually doesn't work properly. (This is the case in main, not new in this PR.) While it subscribes you to the stream, it's still not possible to send the message until you close and re-open the compose box.
I think ideally, rather than a subscribe button, we should have a "Subscribe and send" button, which would do both actions. So the two buttons (in order) would be:

"Subscribe and send"
"Send without subscribing"

@adambirds would you be up for implementing this as part of this PR?

@alya yes, I will expand this PR (I'm assuming you meant me and not @adambirds ;-)

@alya done and done - turned out to be an easy fix given the 'machinery' for send_anyway.

(I'm assuming you meant me and not @adambirds ;-)

Oops, my bad with the auto-complete!

@alya good call re alignment hereya go, also gave them a standard length:

for superwide monitors:

@timabbott all UI & eng changes done, commits squashed - re-review?  thx.

The above screenshots look good to me!
@timabbott ready for another review from you.

This likely needs to be reworked on top of the much nicer new compose_banner library.


Split up auth test

@jkwatson Good suggestion about helper methods. We lose a little locality, but I think this is definitely an improvement. WDYT?

:+1: :trophy:

@jkwatson Cool, this is in master now, so you might want to rebase your auth branch, if youre still in the middle of it.


[FALSE-POSITIVE] *.coacb.com

Domains or links
Please list any domains and links listed here which you believe are a false positive.
*.coacb.com
More Information
How did you discover your web site or domain was listed here?
2. Incorrectly marked as Phishing on Phishtank or OpenPhish?
Listed on virustotal:

Phishing Database
MalwareURL

You need to fill the form correctly, otherwise we do not know what we are dealing with, Please open in new issue and fill out the form in full.


Fix URL Rewriting causing HTTP/404

.htaccess responsible for url rewriting was ignored by missing "AllowOverride all"
This solves https://github.com/harvard-itsecurity/docker-misp/issues/5

This request needs to be committed @ventz. Normal deployment with default values results in errors without this extra line.
Also, changing credentials through build.sh causes conflicts with deployment as SQL passwords were not changed properly.
Thank you @bschildendorfer  for fix.

It looks like somehow this fell out of commit 50d2a42bbc801c56d5550820def01e3fab4239a0

Fixed and pushed out:
https://github.com/harvard-itsecurity/docker-misp/commit/59f63b7d6c06527a1c8881616e44995077d8a013
Thanks for catching this!


"Couldn't open file lang/english.txt!" error.

I'm trying to run a *.lua file using LuaConsole on Windows by doing luaw main.lua but only thing I get every time is :
Couldn't open file lang/english.txt! Failed to load lang file!

This is because lang/english.txt can't be found. It needs to be within the installation directory. Perhaps cwd.
You should use the development branch, as some oddities are repaired.
git clone --branch development https://github.com/tilkinsc/LuaConsole

LuaConsole.zip
Here is a prebuilt for windows. It should be directory independent now as long as lang/* is next to the folder. I included luajit as the primary, and lua-5.1.5, lua-5.2.4, lua-5.3.6, and lua-5.4.4


Fix GetTasksForQueueAsync

Speed up NmtQueue test with parallel corpora
#513

This change is

Codecov Report
Attention: Patch coverage is 0% with 3 lines in your changes missing coverage. Please review.

Project coverage is 56.95%. Comparing base (e2f1f25) to head (5d57c9f).

Files with missing lines
Patch %
Lines

...c/Serval.Machine.Shared/Services/ClearMLService.cs
0.00%
3 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #534      +/-   ##
==========================================
- Coverage   56.95%   56.95%   -0.01%
==========================================
  Files         299      299
  Lines       15734    15735       +1
  Branches     2173     2173
==========================================
  Hits         8962     8962
- Misses       6122     6123       +1
  Partials      650      650

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

src/Serval/test/Serval.E2ETests/ServalApiTests.cs line 218 at r1 (raw file):
Previously, ddaspit (Damien Daspit) wrote…
Please use an explict type if the type is specified elsewhere on the line.

Done.


Fixed non working StdClient not able to fetch from UDP upstreams

Refactored dns messages towards immutability
Added DNS server stub to ease testing of StdClient

Resolve https://github.com/noglitchyo/dealdoh-client/issues/1

Codecov Report

Merging #5 into master will decrease coverage by 6.89%.
The diff coverage is 86.29%.

@@             Coverage Diff              @@
##             master       #5      +/-   ##
============================================
- Coverage     96.51%   89.61%   -6.90%
- Complexity       66       79      +13
============================================
  Files             8        9       +1
  Lines           258      289      +31
============================================
+ Hits            249      259      +10
- Misses            9       30      +21

Impacted Files
Coverage Δ
Complexity Δ

src/Client/DohClient.php
100.00% <ø> (ø)
4.00 <0.00> (ø)

src/Client/GoogleDnsClient.php
100.00% <ø> (ø)
9.00 <0.00> (ø)

src/Factory/DohHttpMessageFactory.php
100.00% <ø> (ø)
7.00 <0.00> (ø)

src/Service/DnsPoolResolver.php
94.59% <ø> (ø)
14.00 <0.00> (ø)

src/Client/StdClient.php
72.22% <70.00%> (-27.78%)
16.00 <10.00> (+11.00)
:arrow_down:

src/Factory/Dns/MessageFactory.php
87.87% <96.15%> (-9.49%)
14.00 <8.00> (+1.00)
:arrow_down:

src/DohProxy.php
88.09% <100.00%> (ø)
10.00 <0.00> (ø)

src/Helper/MessageHelper.php
100.00% <100.00%> (ø)
1.00 <1.00> (?)

src/Mapper/GoogleDns/MessageMapper.php
100.00% <100.00%> (ø)
4.00 <2.00> (ø)

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 1e74ddf...7da2ad8. Read the comment docs.


Problem with operator< when binding std::map using VS 2017

The following code fails to compile with Visual Studio 2017 and sol 2.20.4 for reasons I don't understand. It complains about not being able to find a suitable operator<, which IMHO it should only need for the key type, which is std::string and therefore should have an appropriate operator<.
#include "sol.hpp"
#include <map>
#include <any>
#include <string>

int main() {
    typedef std::map<std::string, std::any> Map;

    sol::state lua;

    lua.new_usertype<Map>("Map",
        sol::constructors<Map()>(),
        "putString", [](Map &m, std::string const &k, std::string const &v){
            m.insert_or_assign(k, v);
        }
    );
}

BTW, I'm trying to come up with a smart way of handling std::any. I believe that std::any ought to have a special place in a binding of C++ to a dynamically typed language like Lua. Yet I'm finding it particularly hard to cope with. Any tip is welcome!

The way to handle std::any is to not touch std::any. It can be anything. The only way to know what it is is to poke at it, repeatedly, until you find out what type it is.
How many type checks am I supposed to do against it before I stop and just pass it through sol2 untouched? Do I check for int? What about short? long? long double? const char*? string_view? std::string? std::vector, and for what type-
You can see this rabbit hole doesn't have a bottom.
std::any is an extremely poor type, useful only for transporting through things like ABI or API boundaries.

How many type checks am I supposed to do against it before I stop and just pass it through sol2 untouched?

I think it would be best to only support an essential set, i.e. the types that Lua supports natively. You then offer the possibility for a user to plug in code that handles the other cases.
For example, I wouldn't recommend to support anything in a std::any that doesn't own what it refers to, so const char* would be down to the user to cope with, as well as std::string_view. This leaves std::string to translate to/from a Lua string as the only sensible default.
The myriad of integer types is probably the major headache, when it comes to unpacking a std::any, but at least the set is bounded.
But I admit I haven't thoroughly thought this through.

Ah, and one more thing:

The only way to know what it is is to poke at it, repeatedly, until you find out what type it is.

That's not strictly true. There's type(), which can be used to obtain type information for the value held inside. Maybe there's a way to use this in a smart way, instead of going through a long list of any_cast<whatever>() calls.

That returns std::type_info. sol does not use RTTI, so that throws out that approach already.

(Also, the last time I had a static {blah} like I would have a static std::unordered_map. somebody opened an issue against it saying it was non-reentrant as far as threading went, so that also throws that approach out too...)

FYI, gcc has similar complaints. It seems that operator== and/or operator< are being requested for the key/value pair, instead of the key alone. I haven't deciphered the error novel yet, to find where exactly it goes wrong.

A tidbit of potentially interesting information regarding this std::any binding business:
https://www.boost.org/doc/libs/1_68_0/libs/hana/doc/html/index.html#tutorial-quickstart-any

This code is compiling again on the latest sol2 (not sol3).
I can't seem to reproduce the problem anymore.

Tried on a few more platforms, could not reproduce.

Interesting. I have updated my Visual Studio Community 2017 to the current version 15.8.9, and the sol header to 2.20.5, and the problem persists. Which compilers have you tested with?


logging is always enabled

Integration Method: CocoaPods
Xcode Version: 8.1
Library Version: 3.0.6
Platform: iOS
Device: Simulator / iPhone 6s
Language: Objective-C
Description:
It seems that logging is enabled, in our every biuld scheme.
I think that there is a cause : https://github.com/mixpanel/mixpanel-iphone/pull/607
Expected Behavior:
logging will turn on with Build Settings to MIXPANEL_DEBUG=1

@masawo thanks for letting us know. We'll put out a fix with the next release but for now [[Mixpanel sharedInstance] setEnableLogging:NO] should turn it off.


Unecessary

https://github.com/django-import-export/django-import-export/blob/b367e106794ef3a029f933702644b4f81453a0be/import_export/admin.py#L415-L446
The purpose of this code snippet is just to get the queryset with the same filters and ordering of the django admin changelist.
However, in the django's ModelAdmin class has a method that returns a ChangeList instance: ModelAdmin.get_changelist_instance (self, request).
This entire code snippet could be replaced by:
return self.get_changelist_instance (request) .get_queryset (request)
With the advantage of avoiding copy-and-paste

Hi @sesostris! Is this a question, comment, request for changes?

This is a request for change. If approved I can submit a pull request.

Can you change the title? Its still unclear to me what the change requested is.


Invalid get index of type StringName on base Dictionary when key is an empty StringName

Godot version
4.0.3 stable
System information
Kubuntu 22.04
Issue description
I have a dictionary with string keys, and one of the keys is "" (empty string). To get values from dictionary by the key, the StringName parameter is used. So all works fine unless the key is an empty StringName.
Context: dictionary stores some data about input actions, that are type of StringName (and certain Input and InputEvent methods expects StringName arguments), that's why this type is prefered rather than String. And one of the dictionary elements is kinda nil-element, used to eliminate if _dict.has(key) checks.
Steps to reproduce

Make *.gd script
Add a Dictionary variable, e.g. var _dict:={}
Add some value with empty key _dict[&""]=someval or _dict[""]=someval
Try to get this value like print(_dict[&""])

Minimal reproduction project
Example declaration:
class_name Something extends Node

var _dict:={&"":"--emptyness--",&"a":"A"}

func get_val_by_strname(key:StringName):
   return _dict[key]

func get_val_by_strname_converted(key:StringName):
   return _dict[key as String]

func get_val_by_str(key:String):
   return _dict[key]

Example usage:
print(Something.get_val_by_str("A"))  #Will print "A".
print(Something.get_val_by_strname_converted(&"A"))  #Will print "A".
print(Something.get_val_by_strname(&"A"))  #Will print "A".

print(Something.get_val_by_str(""))  #Will print "--emptyness--".
print(Something.get_val_by_strname_converted(&""))  #Will print "--emptyness--".
print(Something.get_val_by_strname(&""))  #Will throw an error "Invalid get index of type StringName on base Dictionary".

This seems to be due to that this lookup uses the internal getptr function which doesn't do the conversion from StringName to String, will investigate further

I just encountered this too (v4.0.3.stable.official [5222a99f5]). Here's a shorter reproduction:
var dict := {}
dict[&""] = 0
dict[&""] += 1


Simplify code for adding failures

PR checklist

[ ] Addresses an existing issue: #0000
[ ] New feature, bugfix, or enhancement

[ ] Includes tests

[ ] Documentation update
[X] Code improvement

What changes did you make?
This greatly simplifies how to add new rule failures, using just a few helper functions.
Only 2 rules remain that directly call createFailure; these two keep their own list of failures and do complex things with them.  All other rules were just creating a failure and immediately adding it, so those steps can be combined.

@andy-hanson could you please enable Circle CI for your fork? I'd be happy to merge this after it is verified by CI.

How do I do that?

@andy-hanson go to https://circleci.com/gh/andy-hanson/tslint and look for the button to enable builds for the project

OK, enabled it.

Created #1867 for that. It requires some non-trivial changes but I think it's doable.
Making it private will cause pretty much every custom rule to have compile errors, but in most cases it will be easy to translate to addFailureAtNode. Just noting that people could use some heads-up.

good point


Pass context information to Function Call Thread

Expected Behavior
When using Function Calling with OpenAI and other models - the function gets invoked in a new thread.
The function being called in my application requires context, so it can persist the response data to the correct user / session etc.
My application uses Spring Security - however, I anticipate there could be other requirements to pass context to this thread.
It would be ideal to be able to store the context before the thread is called and then retrieve the context e.g. using:
SecurityContext context = SecurityContextHolder.getContext();
and
SecurityContextHolder.setContext( context );
Perhaps you could register listeners / functions that can be called to pass this information.
Current Behavior
The calling thread does not have any context about the session / user that invoked the call to the AI model.
Context
The only workaround is to request that the model pass an identified to the function via the prompt.

I've opened a similar issue 3 weeks ago:
https://github.com/spring-projects/spring-ai/issues/864
A possible workahound for it is passing an function "instance" in each interaction.
Then you can pass some user/session in constructor.

Thanks you highlighting this use case @dolukhanov, @miltonhit
Can you please elaborate on why the function information should be kept in Thread context rather, for example, being returned with the chat response metadata as suggested here?

Hi!  Can you see if https://github.com/spring-projects/spring-ai/pull/1458 fixes this issue for you.   Here are some minimal docs - https://docs.spring.io/spring-ai/reference/api/chat/functions/openai-chat-functions.html#_tool_context_support   see the test here as well https://github.com/spring-projects/spring-ai/blob/eb2deba9cb8d0bc22fb354ae3af59335eb4d740d/spring-ai-spring-boot-autoconfigure/src/test/java/org/springframework/ai/autoconfigure/openai/tool/FunctionCallbackWithPlainFunctionBeanIT.java#L60
please reopen if you think something is not addressed.


CLI:  ClickOnce signing workaround should be removed

@clairernovotny says this workaround should be removed.

With the tool operating locally, it should accept a directory as an argument and handle all the required files instead of needing them packaged up.


build(deps): bump terraform from 1.4.4 to 1.4.5

https://github.com/hashicorp/terraform/releases/tag/v1.4.5
https://releases.hashicorp.com/terraform/1.4.5/

Thanks a lot for sharing this @HorizonNet !


Open Metrics Link in new tab

users/tester prefer that the metrics open in a new tab when the click the metrics link from the toolbar menu

The link now opens into a new tab.


Annotate zerver/views/webhooks/bitbucket.py.

Automated message from Dropbox CLA bot
@rwbarton, it looks like you've already signed the Dropbox CLA. Thanks!

Coverage increased (+0.07%) to 64.637% when pulling 41fbc16f819f75d9272cd411362c2f997a1ba4ca on rwbarton:annotate into 8411b2e574ee40fbce87332f05a14d46a5995f34 on zulip:master.

Merged, thanks @rwbarton !


Subscription naming convention

Hi,  the company I work for use connectivity, management, and identity as a naming convention for their platform landing zone subscriptions.
All application landing zone subscriptions use something like fshare-devl.
Using the tool, how can I use longhand in some cases and shorthand in others?
I used the  Projects/Apps/Services component configuration for the connectivity sub with a shorthand name 'con', but when I attempt to generate a subscription name it forces the use of the shorthand name.
Is there a way, when adding values, that you can specify a flag to use shorthand or longhand?
Many thanks

Hi @furrm, the tool assumes for consistency and simplicity that short names will be used everywhere.  We will put the feature request in our backlog and prioritize it based on all feedback we receive.  Thanks for the feedback!


[webapp] add local context scope for az connect

Description

Add webapp local context scope for az connect
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

webapp

Could you share some context?

This is a draft PR which won't be merged.

Add a description

done

Why remove cupertino and add connect? You want to rename cupertino to connect? What is connect?
az cupertino -h
az: 'cupertino' is not in the 'az' command group. See 'az --help'. If the command is from an extension, please make sure the corresponding extension is installed. To learn more about extensions, please visit https://docs.microsoft.com/en-us/cli/azure/azure-cli-extensions-overview
az connect -h
az: 'connect' is not in the 'az' command group. See 'az --help'. If the command is from an extension, please make sure the corresponding extension is installed. To learn more about extensions, please visit https://docs.microsoft.com/en-us/cli/azure/azure-cli-extensions-overview

/azp run

/azp run

/azp run

Close this PR as it is only for testing purpose.


[TF2.0alpha] Eagerness/Distribute Strategies performance issues

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): 16.04/18.04
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): 2.0.0-dev20190327
Python version: 3.6
CUDA/cuDNN version: 10.0
GPU model and memory: TITAN X 12 GB

Describe the current behavior
I have some code, where to the best of my knowledge not everything can be wrapped with the tf.function. Thus, in the function calling strategy.experimental_run I removed the @tf.function wrapper. However, this tremendously decreases the overall performance and falls way behind the 'normal' eager execution.
For the toy example in this issue, these are the performance differences:
Computing for 1000-steps, no strategy: Time: 7.228003740310669 sec
Computing for 1000-steps, strategy with tf-function: Time: 2.6209354400634766
Computing for 1000-steps, strategy with no tf-function: Time: 63.60745906829834

Describe the expected behavior
I would expect that without tf.function at least the usual performance of eagerness is achieved.
Code to reproduce the issue
import time

import tensorflow as tf

class DebugModel(tf.keras.Model):
  def __init__(self):
    super(DebugModel, self).__init__()
    self.dense = tf.keras.layers.Dense(20, activation='relu', input_shape=(5,))

  def __call__(self, input, *args, **kwargs):
    x = self.dense(input)
    return x

def train_step(input):
  with tf.GradientTape() as tape:
    out = my_model(input)
    my_loss = -tf.reduce_mean(out)
  gradients = tape.gradient(my_loss, my_model.trainable_variables)
  optimizer.apply_gradients(zip(gradients, my_model.trainable_variables))
  return my_loss

def distributed_train():
  return strategy.experimental_run(train_step, train_iterator)

@tf.function
def distributed_train_tf_function():
  return strategy.experimental_run(train_step, train_iterator)

if __name__ == '__main__':
  dataset = tf.data.Dataset.from_tensor_slices(
    (tf.random.uniform([3000, 16, 5])))
  N_steps = 1000

  # Plain computation
  my_model = DebugModel()
  optimizer = tf.keras.optimizers.Adam(1e-2)
  _s = time.time()
  i = 0
  for data in dataset:
    if i >= N_steps:
      break
    train_step(data)
    i += 1
  print('Computing for {}-steps, no strategy: Time: {} sec'.format(
      N_steps, time.time() - _s))

  # Mirrored strategy with @tf.function
  dataset = tf.data.Dataset.from_tensor_slices(
    (tf.random.uniform([3000, 16, 5])))
  strategy = tf.distribute.MirroredStrategy()

  with strategy.scope():
    train_iterator = strategy.make_dataset_iterator(dataset)
    train_iterator.initialize()
    optimizer = tf.keras.optimizers.Adam(1e-2)
    my_model = DebugModel()

    _s = time.time()
    for s in range(N_steps):
      distributed_train_tf_function()
    print('Computing for {}-steps, strategy with tf-function: Time: {}'.format(
      N_steps, time.time() - _s))

  # Mirrored strategy without @tf.function decorator
  dataset = tf.data.Dataset.from_tensor_slices(
    (tf.random.uniform([3000, 16, 5])))
  strategy = tf.distribute.MirroredStrategy()
  with strategy.scope():
    train_iterator = strategy.make_dataset_iterator(dataset)
    train_iterator.initialize()
    optimizer = tf.keras.optimizers.Adam(1e-2)
    my_model = DebugModel()

    _s = time.time()
    for s in range(N_steps):
      distributed_train()
    print('Computing for {}-steps, strategy with no tf-function: Time: {}'.format(
      N_steps,
      time.time() - _s))

@goldiegadde @dynamicwebpaige Would be nice to see some progress on this issue

Assigning to @alextp for triage.
I might be wrong, but at a glance it might be a caching issue. You might want to wrap train_step into a tf.function.

@alextp can you please guide me, I want to work on this issue.

@shashvatshahi1998 the first step is to try to reproduce the issue, and then try to follow @mdanatg 's advice to wrap train_step in tf.function and see whether the issue goes away.
If it doesn't, running a profiler like python's cprofiler on the code to find out what is happening and triaging that to a separate issue is the next step.

Wrapping train_step in tf.function does not work. It raises the following error: RuntimeError: `merge_call` called while defining a new graph. [...].

@guptapriya is this a known issue internally?

There are couple of things mentioned here, let me clarify:

Performance without tf.function when using distribution strategy is terrible - this is known issue and we do not recommend doing this right now, and it is not something we are prioritizing yet either. The recommended approach is to wrap experimental_run inside a tf.function. Fixing this requires a significant amount of work inside MirroredStrategy.

Wrapping just the train_step into a tf.function: The error seen there is also known, and I am working on fixing that. However, that will not address the performance issue from #1. For anyone who can wrap their train_step in a tf.function, they should be able to wrap strategy.experimental_run inside the tf.function as well - AND get the performance benefit. So there is not a great reason yet to wrap just the train_step into a tf.function but not strategy.run. So while I am working on fixing that issue, it will still not help with performance.

Hope this helps. Please let me know if there are more questions.

@guptapriya Regarding 1.), this is rather confusing to me. My main problem causing this issue was my dependance on eager execution during the forward/backward pass.
However, wrapping my experimental_run with tf.function straps me from the eager execution. The new TF policy should not be eager-execution is the default except you want to use multiple GPUs for training, should it?

@meyerjo Ideally eager execution is available and performant for all use cases on day 1, but we have limited resources and have to prioritize what gets done first and then what gets fast first. I agree with @guptapriya's priorities outlined above as being the best to move the existing ecosystem forward, and I'm sure that once performance is good for the tf.function use cases we'll start to improve it for pure-eager multi-gpu.

It has been 14 days with no activity and the awaiting response label was assigned. Is this still an issue?

It has been 29 days with no activity and the awaiting response label was assigned. Is this still an issue?

It has been 14 days with no activity and the awaiting response label was assigned. Is this still an issue?

It has been 29 days with no activity and the awaiting response label was assigned. Is this still an issue?

It has been 44 days with no activity and the awaiting response label was assigned. Is this still an issue?

We are closing this issue for now due to lack of activity. Please comment if this is still an issue for you. Thanks!


#699 Fix Bugsnag being initialized during tests

Fixes #699
Checklist

[x] Documented
[x] Unit tested
[x] release_notes.md updated

Codecov Report

Merging #807 into master will decrease coverage by 0.01%.
The diff coverage is 66.66%.

@@             Coverage Diff              @@
##             master     #807      +/-   ##
============================================
- Coverage     79.09%   79.07%   -0.02%
+ Complexity      703      702       -1
============================================
  Files           138      138
  Lines          2932     2930       -2
  Branches        421      420       -1
============================================
- Hits           2319     2317       -2
  Misses          343      343
  Partials        270      270


float8 training: add static scaling

Summary:
This is useful for things such as:

activation_with_bounded_range -> linear (can set static scale to activation range)
bounding weight scales to known quantities if the modeling user can guarantee their magnitude throughout training

We don't have signal yet that this is useful for production things, but it would be good to land this to enable easy experimentation.
Test Plan:
Unit and integration tests pass:
./test/test_everything.sh
// note that there is a failure in `test_fsdp2.py` which is present on main

Use float8 profiling script to see GPU kernel time go down as we enable static scaling on a toy model:
https://gist.github.com/vkuzo/b2cf46f7cccb691125566873859ca39d
Reviewers:
Subscribers:
Tasks:
Tags:

cc @sayakpaul  this PR will be very relevant for LoRA training

Thanks for pinging me here :)
LMK how/if I can amend my script to benefit from this.


Allow jaeger-lib to be used with tally >= 2.1, < 4

This allows downstreams to use either tally v2.1 or the latest version (v3)

Coverage remained the same at 100.0% when pulling 97c10b5817718aeb2822e0523f872e36bb8c3ab1 on prashantv:tally_version into ffca3143c929e9b97325e6bebcc380e8c6e5fa0d on uber:master.


What do I need to post on a topic?

I have a server working
This is the start method
    public void Start()
    {
        _mqttServer = new MqttFactory().CreateMqttServer(options =>
        {
            options.ConnectionValidator = c =>
            {
                if (c.ClientId.Length != 12)
                {
                    return MqttConnectReturnCode.ConnectionRefusedIdentifierRejected;
                }

                if (c.Username != appConfiguration.MqttConfig.UserName)
                {
                    return MqttConnectReturnCode.ConnectionRefusedBadUsernameOrPassword;
                }

                if (c.Password != appConfiguration.MqttConfig.Password)
                {
                    return MqttConnectReturnCode.ConnectionRefusedBadUsernameOrPassword;
                }

                return MqttConnectReturnCode.ConnectionAccepted;
            };

        });
        _mqttServer.StartAsync();

        _mqttServer.ClientDisconnected += MqttServerOnClientDisconnected;
        _mqttServer.ClientConnected += MqttServerOnClientConnected;
        Logger.Info("MqttService started");
    }

and i have a public method
    public void SendMessage(string text, string clientId)
    {
            var message = new MqttApplicationMessageBuilder()
                .WithTopic(clientId)
                .WithPayload(Encoding.ASCII.GetBytes(text))
                .WithExactlyOnceQoS()
                .WithRetainFlag()
                .Build();
            _mqttServer.Publish(new[] {message});
    }

and i configured a client in MQTTlens, i can connect and suscribe to the same topic as the id (for example<PHONE_NUMBER>12) but if i send a message from the server using the method "Send message" i do not see it in the client.
What is missing?
Thank you very much

you have to subscribe to the topic you want to receive msgs for. in your case<PHONE_NUMBER>12

The device is suscribed to the topic if y send a message from another device (MQTTlens) y see it but if y send messages from the server i don't see anything

hmm do you subscribe with qos  exacly once? if you subscribe with lower qos the message will not be send.

I changed it but i do not see any change

and other strange issue _mqttServer.GetConnectedClients() returns always 0

can you please add logstatements to your clientvalidation callback just to verify the clients are accepted?

The clients accept the connection, i am debugging it. And i found where is the issue. i commented this line of code
_mqttServer.ClientDisconnected += MqttServerOnClientDisconnected
and it's works fine..
I emptied the ClientDisconnected function but it does not work
here is my ClientDisconnected function
private void MqttServerOnClientConnected(object sender, MqttClientConnectedEventArgs mqttClientConnectedEventArgs)
{
}

"hmm do you subscribe with qos exacly once? if you subscribe with lower qos the message will not be send." this is the solution. Now in MQTTlens  i deleted the suscription i created again and it's works.
Thank you very much for your help and apologies for the inconveniencie


Fix 500 out of range bug when completing last challenge

Can someone QA this PR. Verify that completing the last challenge redirects you to the map with a info flash message informing you of half way marker.


Error during backup/restore: Temporary Gmail IMAP error, try again later.

I get this error all the time whenever I try to backup my messages. I have not fiddled with the IMAP server settings, and IMAP access is enabled in my Gmail account. I managed to backup about 70 out of 500+ SMS a while back over several tries, but it has not worked since. None of my MMS were backed up.

bump

Hi! I too get this issue when ever I try to backup through WiFi! On the same WiFi, SMS Backup+ works initially without any problem. Then, after a some time, in my case a few days, this error starts popping up. I need to switch to my mobile carrier to ensure backup.

Same here.  Been unable to back anything up since the 17th.

Same here. I managed to get it working by disabling the Wifi connection.

@drbubba @jsleroy @pamfu @drkvogel Android version / phone ?

Android 2.2.1 / HTC wildfire

Android 2.3 / HTC Desire
Now, Android 4.0.1 / HTC Desire

Same here, disabling wifi worked for me also.
Android 4.0.2 / Galaxy Nexus

This is still a problem. While on wifi I get the error, when not, it seems to work.
Running SMS Backup+ v1.5.10 on a Oneplus One. Had the same problem on my previous Galaxy 2 and 3. Does not seem to be a problem on my Nexus 6P.

@adrian-vg Can you try again with the latest beta?
See https://github.com/jberkel/sms-backup-plus/#beta-testing

Hi,
Seems to be working.
Thanks!
Is it still that token-problem mentioned earlier, or something else?
--
//Sorin
From: Jan Berkel<EMAIL_ADDRESS>Sent: den 29 mars 2018 10:13
To: jberkel/sms-backup-plus<EMAIL_ADDRESS>Cc: Sorin Srbu<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [jberkel/sms-backup-plus] Error during backup/restore: Temporary Gmail IMAP error, try again later. (#174)
@adrian-vg https://github.com/adrian-vg  Can you try again with the latest beta?
See https://github.com/jberkel/sms-backup-plus/#beta-testing
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/jberkel/sms-backup-plus/issues/174#issuecomment-377158445 , or mute the thread https://github.com/notifications/unsubscribe-auth/AP-ksKFL91fjO2y0lXdyZeJyEqs5tcTEks5tjJedgaJpZM4ACEtZ .  https://github.com/notifications/beacon/AP-ksGykRyfQ8yiP71oBcCMmq_jOyVjpks5tjJedgaJpZM4ACEtZ.gif

It’s strange though that only the last four texts are backed up. The other 2000+ texts and almost 200 calls are not.
--
//Sorin
From: Sorin Srbu
Sent: den 29 mars 2018 10:37
To: 'jberkel/sms-backup-plus'<EMAIL_ADDRESS>Subject: RE: [jberkel/sms-backup-plus] Error during backup/restore: Temporary Gmail IMAP error, try again later. (#174)
Hi,
Seems to be working.
Thanks!
Is it still that token-problem mentioned earlier, or something else?
--
//Sorin
From: Jan Berkel<EMAIL_ADDRESS><EMAIL_ADDRESS>>
Sent: den 29 mars 2018 10:13
To: jberkel/sms-backup-plus<EMAIL_ADDRESS><EMAIL_ADDRESS>>
Cc: Sorin Srbu<EMAIL_ADDRESS><EMAIL_ADDRESS>>; Mention<EMAIL_ADDRESS><EMAIL_ADDRESS>>
Subject: Re: [jberkel/sms-backup-plus] Error during backup/restore: Temporary Gmail IMAP error, try again later. (#174)
@adrian-vg https://github.com/adrian-vg  Can you try again with the latest beta?
See https://github.com/jberkel/sms-backup-plus/#beta-testing
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/jberkel/sms-backup-plus/issues/174#issuecomment-377158445 , or mute the thread https://github.com/notifications/unsubscribe-auth/AP-ksKFL91fjO2y0lXdyZeJyEqs5tcTEks5tjJedgaJpZM4ACEtZ .  https://github.com/notifications/beacon/AP-ksGykRyfQ8yiP71oBcCMmq_jOyVjpks5tjJedgaJpZM4ACEtZ.gif

@adrian-vg not sure why what would be. maybe reset the backup state? (Select "Reset" from the menu)

Will try, thanks.


Support Java virtual threads

Hi
As you may know in newer Java versions they added support for "virtual threads". Is it possible to add an option for enabling virtual threads to be used instead of hardware threads? (for those who are using newer Java versions)
Possibly via a thread supplier Function or something.

I use Executors::newWorkStealingThreadPool in order to parallelize, rather than Thread objects themselves, so changing it to a virtual threading interface shouldn't be too difficult, as I just have to change what ExecutorService I create.

Nice,
For using virtual threads Executors.newVirtualThreadPerTaskExecutor() could be used.

I just tested the difference between the two, and there was no measurable improvement, so I don't think it's worth breaking compatibility for this.

so I don't think it's worth breaking compatibility for this.

No compatibility breaking would happen I think
https://github.com/thebombzen/jxlatte/blob/21f5ab01dfa1dd2d760f464b2972540ec80322c8/java/com/thebombzen/jxlatte/util/FlowHelper.java#L45-L47
Just need to add a new constructor
    public FlowHelper(Supplier<ExecutorService> supplier) {
        threadPool = supplier.get();
    }

So users can be able to provide their custom suppliers.

See: 53dca9853d94fbddd2fcf10b521d5b1a37a850ec


FLIR Lepton 3.5 with PyLepton in Jetson Nano

Hi everyone,
I currently run Pylepton with Lepton Breakout Board V2 with FLIR Lepton 3.5 in Jetson Nano. I used branch Lepton3 and got the error "Garbage frame number resetting ..." continuously.
Is there anyone have run Pylepton in Jetson Nano can give me recommendation about it ? The Pylepton have worked perfectly on my rasberry Pi4 .
Thank you so much.

Hello.
Is there a solution to this issue?

Working with Jetson Nano and Lepton 3.5.
Used the "lepton3-dev" branch of this repo in Raspberry Pi 4 and was able to succesfully capture images.
But the same repo did not work in Jetson Nano (got the same error "Garbage frame number resetting ..." continuously.)
I tried using another C++ based repo for interfacting Lepton 3.5 with Jetson Nano _https://github.com/Myzhar/Lepton3_Jetson_
The above repo is successfully able to stream thermal data in Jetson Nano. But, the accuracy of the radiometric data obtained using this repo does not reflect real-time temperature values (throughout the range of Lepton 3.5, both in High & Low gain mode)
_But, pylepton offers better radiometric accuracy throught the range of the sensor (Tested using Rpi4).**
So, need to get pylepton working in Jetson Nano_
My observation:
The error message "Garbage frame number resetting ..."  is due to SPI communication failure.
SPI has been enabled and working seamlessly in Jetson Nano (because the C++ library is able to successfully stream data)
So, the SPI configuration made in pylepton does not seem to match for Jetson Nano
If anyone has come across the solution, kindly suggest.
Thanks in advance!!

Hello.
Is there a solution to this issue?

Hello.
Is there a solution to this issue?

Hello.
I changed spi buffer size to 65536, and also changed spidev message limit to 8.
But it didn't work and had same question =>OSError: [Errno 90] Message too long

Hello.
Is there a solution to this issue?

I give up that runing Pylepton with Lepton Breakout Board V2 with FLIR Lepton 3.5 in Jetson Nano.
So I use PureThermal 2 I/O Board with FLIR Lepton 3.5 in Jetson Nano.

Hi was it available for Lepton 3.5 on PureThermal 2 Board


Anchor identification when emits onTargetFound

In image-target/three.js, line 161, I identified that any information is send to onTargetFound.
With multiple-targets, I´m interesting to identifiy which target emitted this event. How can I do this? Suggestions...

onTargetFound is bind to specific anchor already

Ok. How can access It?

anchor.onTargetFound = () => {
}

OK... I understood your point. However, I´m working with multi targets, dinamically loaded from a database.
I have this piece of code, to adding an anchor for each target. When the target is detected, then show a button (in thruth is an hyperlink) in the page. Each target must change the value of the parameter from the hyperlink.
for (var i = 0; .....) {
    ...
    const glb = await loadGLTF(<dinamically from a database>);
    const glbScene = glb.scene;
     ...
   const anchor = mindarThree.addAnchor(i);
  anchor.someAttribute = take the value from somewhere
  anchor.group.add(glbScene);

    anchor.onTargetFound = () => {
      buttonAR.href = buttonAR._href + "?id=" + <I wanna get someAttribute from the current anchor>
      buttonAR.style.display = 'block';
    }
    ....

}
Can you see? Anchor is dinamically created in multitarget problem.

Yes,  buttonAR.href = buttonAR._href + "?id=" + anchor.someAttribute should work.

I tried now, and it gave the error message "Uncaught ReferenceError: anchor is not defined"

I changed three.js with this code
if (this.anchors[i].onTargetFound) {
this.anchors[i].onTargetFound(this.anchors[i]);
}
It worked for me.

That's weird. I really don't think you need to do that.  But whatever works for you.

However, I would like to use your suggestion. After compile the code with my modification and use mindar-image-three.js, seems the model is shivering.

Everything is ok. I tried your suggestion.


[State Sync] Remove anyhow and use structured errors

Motivation
This PR cleans up error handling in the state sync code base. Specifically, it: (i) removes the use of anyhow errors and replaces them with the structured error types; and (ii) makes the code more consistent regarding failure cases and error messages (e.g., we remove the use of anyhow::bail and anyhow::ensure, and standardize the error outputs). The PR offers a number of commits going through each of the files in state sync.
Note:

This change doesn't affect backward compatibility. Error types on messages passed between components within each node have changed (e.g., see client.rs), but these messages are not passed between nodes, so we shouldn't need to worry about breaking changes.

Have you read the Contributing Guidelines on pull requests?
Yes.
Test Plan
All tests pass.
Related PRs
This relates to: https://github.com/diem/diem/issues/6795

Thanks @gregnazario!
/land


feat(1447): Add Notification Component

By creating this pull request you agree to the terms in CONTRIBUTING.md.
https://github.com/Infineon/.github/blob/master/CONTRIBUTING.md
--- DO NOT DELETE ANYTHING ABOVE THIS LINE ---
CONTRIBUTING.md also tells you what to expect in the PR process.
Description
Added the new notification component
Related Issue
#1447
Context

📦 Published PR as canary version: 25.7.0--canary.1493.df8290acfb7499847726d41bc3c061a1087be613.0

:sparkles: Test out this PR locally via:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

@pfafffabian-ifx

text is not centered

when icon is not set, its wrapper still occupies space

when text is long and it wraps, the icon should not stay centered in the middle, but be at the top corner like shown in
the design.

when Link button wraps below the text, there is more space between it and the text in the design, but almost no space in
the code component.

e2e test fails


Customize Text colour and Button size - Typeform user feedback (Sidetab)

I am implementing Typeform user feedback (Sidetab) in React using @typeform/embed-react library as following:
//In React I use the bellow code
<Sidetab
id={formId}
hidden={{
user_id : me.id,
name: me.first_name,
}}
ref={sidetabRef}
medium="demo-test"
buttonText="Send us feedback"
buttonColor="#ffa366"
/>
All looks good so far, the only issue is that I have not found a way to change text colour in Button or the button size. It seems there is no fields/props in Sidetab component to change button text colour/font and button size.

Thank you for your feedback. Indeed, these settings are not supported at the moment.
We are taking this as a feature request.

:tada: This issue has been resolved in version @typeform/embed-v4.3.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


fix(checkstyle): fix some checkstyle errors

There is some checktyle errors that prevents the project to build locally :

star imports
some lines exceeding 120 characters.

Checkstyle problem as been fixed.


training code

I'm very interested in your paper and I feel it's very innovative.Could you release your training code ?

Thank you for your interest. I and my colleagues are currently busy working on some new projects and cannot spare time to clean up the code. Thus we do not have recent plan to release the full code.


Question about the requirement of needing a separate user per device

Hi! First of all, I don't know how I missed this addon/integration until now! Stellar work!
I did look if someone had done something similar when I got my dahua doorbell but nothing came up in 2020. I fact I did try running asterisk myself and did some testing but I abandoned it for lack of time and SIP knowledge in general.
Now I'm going to try this. For now I'm trying from device to device before risking changing the SIP server on my dahua VTO/VTH duo. I did manage to make an audio call browser to browser already!
One thing that has raised questions is the need for different users for every device. The reason being that most people will use HA in their browser, their mobile app and sometimes a wall mounted tablet, and often creating different users is rather inconvenient since much of your UI might be user-customized. And when I call someone, I'd expect them to receive that call on any/all their devices.
I wonder what the reason for that limitation is, and if that could be configured at the card level or at the integration level.
Thanks!

Hi @cibernox,
The different users is so you can call to others for if you have wall mounted tablets like a intercom system.
For phone and desktop for the same person you don't need to make multiple users if you don't want to call between them.
And If you don't want to use persons at all, you can use the backup extension and only set custom extensions in the card.
Did that answer your questions?

Closing for now. Please reopen if you still got questions, or ask in the discord server.


Unable to access the AAS data

Hi, I am facing the following issues

Trying to access the following AAS data using the id aasServer/shells/www.company.com/ids/aas/8003_4110_1102_9521** even though I encode the assId that is in the URL format  aasServer/shells/www.company.com%2Fids%2Faas%2F8003_4110_1102_9521 I am getting error

The submodel information could not be retrieved. Getting the following error

I am unable to access the following url with the below AASID(text in bold)
/aasServer/shells/www.company.com/ids/aas/8003_4110_1102_9521
Following is the submodel URL that's not working as expected but a different AAS id data
http://svc-ezd-basyx-server-ocp050149.apps.titan.hco000139.mars.cloud.zf-world.com/aasServer/shells/AssetAdministrationShell---6003FF71/aas/submodels/Prüfstandszustand

Would it be possible to describe your usage of BaSyx SDK more into detail?  You are missing the "/submodel" suffix in the URL.

We are using the docker version of the Basyx SDK and as you mentioned I have already tried the API with submodel but still the issue exist
URL : /aasServer/shells/AssetAdministrationShell---6003FF71/aas/submodels/Prüfstandszustand/submodel
Error Msg:
{"success":false,"isException":true,"messages":[{"messageType":6,"code":"404","text":"ResourceNotFoundException: Could not resolve Submodel with idShort Pr%C3%BCfstandszustand for AAS {idType\u003dCustom, id\u003dAssetAdministrationShell---6003FF71}"}]}

Thanks for providing these details. Would it be possible to provide an executable example reproducing your issue? I will then gladly take a look

Is this still of relevance for you? If no, I will close this issue in a few days.


[cluster-test] Reduce get_pod_node_and_ip to 120 seconds

We had insanely large retry limit on get_pod_node_and_ip, this cause fail slow and creates backlog when something is wrong.
This allows to fail faster

@bors-libra delegate+

:v: @andll can now approve this pull request

@bors-libra r=changyonghu

:pushpin: Commit bc5fa36 has been approved by changyonghu

@bors-libra r=changyonghu p=100
Boosting priority as this might help draining land queue

:bulb: This pull request was already approved, no need to approve it again.

There's another pull request that is currently being tested, blocking this pull request: #4550

:pushpin: Commit bc5fa36 has been approved by changyonghu

:hourglass: Testing commit bc5fa3604dee7ff2997f132c0e8a34838f7f4324 with merge 67f313e99ba990a818683bd61638d6e906795adc...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: changyonghu
Pushing 67f313e99ba990a818683bd61638d6e906795adc to master...


[prior] Create THIRD-PARTY-NOTICES still not correct

What lacking there is not important
you have to understand what this file should store..
you   gets example in corona-warm-server
Make it the same!!
https://github.com/corona-warn-app/cwa-server/blob/master/THIRD-PARTY-NOTICES

https://github.com/idrice24/School-Management-Software/blob/master/ThirdPartyNotices.md


Crash in addon DragAndDrop: TypeError: e.target.className.includes is not a function

nextjs 10.0.5 with next-translate 1.0.6.
react-big-calendar 0.33.2.
loading svg through webpack @svgr/webpack ^5.5.0.
crash line https://github.com/jquense/react-big-calendar/blob/master/src/addons/dragAndDrop/EventWrapper.js#L45
the error happens if the event have button with svg icon, if the mouse event will fire from svg element in EventWrapper occur an error "e.target.className.includes is not a function" because svg element className gives SVGAnimatedString object instead of className string.

I'm also experiencing this... it's happening when i click an ant design icon in a popover I created - which are svg

Until a proper fix can be found, I was able to get it not throwing the error by setting the svg element (in my case an antdesign icon) with pointerEvents:"none" css - if you need the icon to be clickable wrap in a div and set an onClick event.

a PR is welcome - as mentioned pointer-events: none is a good workaround


Proxy ip in nextcloud.log

I use your nextcloud docker file and boring-nginx. If i look in boring-nginx log i see the real ip of my different client.
In nextcloud logs i see the ip of boring-nginx, not the real ip. The bruteforce protection of nextcloud is useless in this case. I can not use fail2ban to.
proxy_params is include in my conf file
Can i change something ??

What if you adds the following to config.php :
'trusted_proxies' => array('IP'),
'forwarded_for_headers' => array('HTTP_X_FORWARDED_FOR'),

Change IP with your docker bridge IP address. You can find it with :
docker exec -ti nextcloud ip route

docker exec -ti nextcloud ip route give me :

default via <IP_ADDRESS> dev eth0.
If i had this in config.php :
'trusted_proxies' => array('<IP_ADDRESS>'),
'forwarded_for_headers' => array('HTTP_X_FORWARDED_FOR'),
The problem is the same, but if i had the boring-nginx ip , the real client ip are shows.
When the container is restarted or recreated, its IP address change.
Is there any solution to fix a container ip adress without creationg a custom networks ??

Yes sorry, I'm using nginx on my host so yes, if you use boring-nginx, it's more easy : you can add a link to your nextcloud container, so you just have to add : array('boring-nginx').

Yes but when i add links in nextcloud to boring-nginx i had this error
ERROR: Circular dependency between nginx and nextcloud

We need to create a custom network for this. Config.php need ip adress .
With fixed ipv4 for boring-nginx all work fine.
Thanks for help ;)


Can't build the precompiled library

I must be missing something:
cd ~/Arduino/libraries/
git clone https://github.com/micro-ROS/micro_ros_arduino.git
cd micro_ros_arduino
docker run -it --rm -v $(pwd):/project --env MICROROS_LIBRARY_FOLDER=extras microros/micro_ros_static_library_builder:galactic

Error:

Not a single firmware compiles.
Please advise.

Sorry, this has been a bug included in an update of one of the micro-ROS layers. It is solved here: https://github.com/micro-ROS/rmw_microxrcedds/pull/221.
Please retry.


Does it support SFU?

https://github.com/zevarito/Licode-ErizoClientIOS#features says that this library supports MCU & P2P modes but SFU is not mentioned.

That line try to say it works either in P2P mode (client-2-client) or (client-server-client). MCU was the term adopted in early days of Licode to name that server component. Currently I think they refer as both MCU and SFU.
Probably the README should be updated to something less confusing.
Some interesting links:
https://github.com/lynckia/licode/issues/518
http://discourse.lynckia.com/t/licode-is-an-sfu-mcu/422
https://webrtcglossary.com/sfu/
https://webrtcglossary.com/mcu/


Revert "Update README.md"

The API call on fdroid shield line shows error
might wanna re-edit that one mate

permission to take files and apk


added incident status and adjusted summary format

Codecov Report

Merging #136 into master will decrease coverage by 0.21%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #136      +/-   ##
==========================================
- Coverage   61.54%   61.33%   -0.22%
==========================================
  Files          95       95
  Lines        1407     1412       +5
==========================================
  Hits          866      866
- Misses        541      546       +5

Impacted Files
Coverage Δ

src/components/Incident/IncidentSummary.js
0% <0%> (ø)
: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 9c8b526...ffd2536. Read the comment docs.

Coverage decreased (-0.3%) to 56.837% when pulling ffd2536826ed89ce9ee23d7eeec72f51b84b47a1 on yingzhao-1:yizha/addIncidentStatusInIncidentSummary into 9c8b526ef79aa7ad2f9b01c7a768da354bae5cf4 on Azure:master.


Release v0.7.0 beta.1

[Feature] Add builtin $Any for AnyType.
[Fix] Fix typo DocumenIoType to DocumentIoType.

Codecov Report

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

@@           Coverage Diff           @@
##           master      #52   +/-   ##
=======================================
  Coverage   65.86%   65.86%
=======================================
  Files          55       55
  Lines        1708     1708
  Branches      355      355
=======================================
  Hits         1125     1125
  Misses        439      439
  Partials      144      144

Impacted Files
Coverage Δ

src/lib/types/document.ts
55.81% <ø> (ø)
: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 4315f33...d509207. Read the comment docs.


Pull updates from upstream to sync with release version 0.6.0

LGTM - sorry this fell off my radar.


Goto definitions of the concurrency related keywords not working

Description:
$subject happens to the ballerina keywords such as

worker
future
thread
fork

Error log
[Error - 4:27:16 PM] Operation 'text/definition' failed! {uri: '/Users/buddhi/Documents/MyWork/Ballerina/bbe/concurrency/src/worker-interaction/worker_interaction.bal', line: 8, col: 8, error: 'Couldn't find a valid identifier token at cursor!'}
java.lang.IllegalStateException: Couldn't find a valid identifier token at cursor!
	at org.ballerinalang.langserver.util.references.ReferencesUtil.findCursorTokenAndCompileModules(ReferencesUtil.java:175)
	at org.ballerinalang.langserver.util.definition.DefinitionUtil.getDefinition(DefinitionUtil.java:70)
	at org.ballerinalang.langserver.BallerinaTextDocumentService.lambda$definition$6(BallerinaTextDocumentService.java:309)
	at java.util.concurrent.CompletableFuture$AsyncSupply.run(CompletableFuture.java:1590)
	at java.util.concurrent.CompletableFuture$AsyncSupply.exec(CompletableFuture.java:1582)
	at java.util.concurrent.ForkJoinTask.doExec(ForkJoinTask.java:289)
	at java.util.concurrent.ForkJoinPool$WorkQueue.runTask(ForkJoinPool.java:1056)
	at java.util.concurrent.ForkJoinPool.runWorker(ForkJoinPool.java:1692)
	at java.util.concurrent.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:157)

Steps to reproduce:
Try this BBE
import ballerina/io;

// The in-scope variables can be accessed by the workers inside the `fork` block.
public function main() {
    // These variables can be accessed by the forked workers.
    int i = 100;
    string s = "WSO2";
    map<string> m = {"name": "Bert", "city": "New York", "postcode": "10001"};

    string name = <string>m["name"];
    string city = <string>m["city"];
    string postcode = <string>m["postcode"];

    io:println("[value type variables] before fork: " +
                   "value of integer variable is [", i, "] ",
                   "value of string variable is [", s, "]");

    io:println("[reference type variables] before fork: value " +
      "of name is [", name , "] value of city is [", city, "] value of " +
      "postcode is [", postcode, "]");

    // Declares the `fork` block.
    fork {
        worker W1 {
            // Changes the value of the integer variable `i` within the worker `W1`.
            i = 23;
            // Changes the value of the `map` variable `m` within the worker `W1`.
            m["name"] = "Moose";

            fork {
                worker W3 {
                    // Changes the value of the `map` variable `m` within
                    // the worker `W3`.
                    string street = "Wall Street";
                    m["street"] = street;

                    // Changes the value of the integer variable `i` within
                    // the worker `W3`.
                    i = i + 100;
                }
            }

            // Waits for the worker `W3` to finish.
            wait W3;
        }

        worker W2 {
            // Changes the value of the `string` variable `s` within the worker `W2`.
            s = "Ballerina";
            // Changes the value of the `map` variable `m` within the worker `W2`.
            m["city"] = "Manhattan";
        }
    }

    // Waits for both workers `W1` and `W2` to finish.
    _ = wait {W1, W2};

    // Prints the values after the `fork` block to check if the values of the
    // variables have changed.

    // The value-type variables have changed since the original variables are passed in.
    io:println("[value type variables] after fork: " +
               "value of integer variable is [", i, "] ",
               "value of string variable is [", s, "]");

    // The internal content of the reference-type variables have got updated
    // since they are passed in as a reference to the workers.
    name = <string>m["name"];
    city = <string>m["city"];
    // Gets the value of the new field added to the `map` variable `m` inside the worker `W3`.
    string street = <string>m["street"];
    io:println("[reference type variables] after fork: " +
               "value of name is [", name,
               "] value of city is [", city, "] value of street is [", street,
               "] value of postcode is [", postcode, "]");
}

For the keywords, it is not possible to provide the go-to definition support. This is the expected behaviour and the exception here is gracefully handled. This particular exception is logged only when we enable debug logs and trace logs for development purposes. Hence closing this issue.


How to Edit Polygon (bitmap) on top of a Google Map but still allow to move the map with swipe gestures?

Hi,
I want to use this library in a Xamarin.Forms project.
I'm trying to put a bit map in a polygon shape (like your example: Show Non-Affine Matrix)
but this way I'm losing the swipe gestures on the map itself.
Is it possible to achieve both (maybe with absolut layout somehow)?

When a release the finger when finished to move one of the vertices, is there a way to convert the screen location to a longitude, latitude coordinate?

I am not too sure about the maps. I would ask on the Xamarin.Forms repo. I also think they have/should have the ability to make the pins / polygons on the actual map using the map framework. The effort to draw polygons over a map is not going to be worth it - also there will always be some lag and offsets. Most mapping controls have the ability to do fairly complex things.


[Go] add menu sample

This is based closely on the TypeScript code.
The s05 tests don't work yet. They get an error
LookupGeneratorAction: no generator action named "google-vertexai"/"gemini-1.0-pro-vision"
Using the gemini-1.0-pro action (without "-vision") doesn't work,
saying that it doesn't support the input type.

I don't understand how to update this for the changes on the main branch, so closing this pull request in favor of #300.

Pull and rebase as you would in gerrit, then git push -f.
I think that might mess with comments on defunct commits, but there weren't any in this case.

In Gerrit the pull and rebase is handled for me via git codereview sync.  Would that work on a non-Gerrit repo?

I don't know. I've never used that command (pure fear). I do it manually.
One concern is that in gerrit, each CL is one commit, so git codereview sync probably always rebases just one commit.
If your PR consists of multiple commits, you have to rebase them all onto main.
For example, if there are three commits, the command, when you are on the PR branch, is
git rebase HEAD~3 --onto main


Fix CentralPackageVersionOverrideEnabled

Fix references to EnablePackageVersionOverride as the correct property name CentralPackageVersionOverrideEnabled
Fixes https://github.com/NuGet/Home/issues/12445

This PR will fix https://github.com/NuGet/docs.microsoft.com-nuget/issues/2878 as well.


Refactoring of NLP process class

To do:

[x] Research existing classifications / taxonomies
[x] Collect additional ideas
[ ] Refactor process class

--> Google Doc for collecting ideas

@matthias-samwald I have drafted a suggestion for refactoring the 'Natural language processing' process class here.