[BUG] Sync-PnPSharePointUserProfilesFromAzureActiveDirectory fails with custom Azure AD property

Reporting an Issue or Missing Feature
I am executing the following command to sync one custom property from Azure AD profile to Sharepoint profile :
Sync-PnPSharePointUserProfilesFromAzureActiveDirectory -UserProfilePropertyMapping @{"Mut-IpPhone"="extension_6a0e2ef7f60e4959a5b417230e8a1875_ipPhone"} -Folder "Documents partages/SynchroSpAzureAd"  -Users $users -Wait -Verbose
The command fails.
Expected behavior
The json configuration file is generated and the sync job is launched
Actual behavior
This command fails with the following message :
`COMMENTAIRES : Using provided user collection containing 1 user
COMMENTAIRES : Creating mapping file, uploading it to SharePoint Online to folder 'Documents partages/SynchroSpAzureAd' and executing sync job
Sync-PnPSharePointUserProfilesFromAzureActiveDirectory : Failed to create sync job. Ensure you're providing users to sync and that the mapping is correct.
Au caractère Ligne:1 : 1

Sync-PnPSharePointUserProfilesFromAzureActiveDirectory -UserProfilePr ...

  + CategoryInfo          : WriteError: (:) [Sync-PnPSharePo...ActiveDirectory], PSInvalidOperationException
  + FullyQualifiedErrorId : EXCEPTION,PnP.PowerShell.Commands.UserProfiles.SyncSharePointUserProfilesFromAzureActiveDirectory

`
My investigations
The connection and rights are OK.
The Mut-IpPhone Sharepoint field exists
The same command with DisplayName Azure AD Field as source works perfectly
The extension_6a0e2ef7f60e4959a5b417230e8a1875_ipPhone AzureAd field exists and was retrieved using the following command :
Get-AzureADUser -ObjectId<EMAIL_ADDRESS>select -ExpandProperty ExtensionProperty
An exception is thrown by the command at this point : https://github.com/pnp/powershell/blob/10d6ba6e5ae07c4ac6dd796d80eff7124f173619/src/Commands/UserProfiles/SyncSharePointUserProfilesFromAzureActiveDirectory.cs#L99
The error happens in the SyncFromAzureActiveDirectory function of the SharePointUserProfileSync class.
I am sure I go beyond this point : https://github.com/pnp/powershell/blob/e816dc17e4f9da8b5752281b55a9e155db2ecec8/src/Commands/Utilities/SharePointUserProfileSync.cs#L42
I suspect this command to retrieve a null value, but I am not able to debug this code : https://github.com/pnp/powershell/blob/e816dc17e4f9da8b5752281b55a9e155db2ecec8/src/Commands/Utilities/SharePointUserProfileSync.cs#L53
Steps to reproduce behavior
Create a custom field in Azure AD user profile
Create a custom field in Sharepoint user profile
Launch the command with appropriate parameters to sync those 2 fields
What is the version of the Cmdlet module you are running?
1.11.0
Which operating system/environment are you running PnP PowerShell on?

[x ] Windows
[ ] Linux
[ ] MacOS
[ ] Azure Cloud Shell
[ ] Azure Functions
[ ] Other : please specify

@JMTeamway - Can you please update to the latest release 2.1.x however you will need PS 7.2 or later to benefit from all the bug fixes and improvements, and let us know.
PS5 support has been dropped in the new release.

Closing as no response.

Hi,
I got the same problem even after upgrading to PnP PS 2.1.1 and PS 7.3.4.
I enabled the trace but it's empty without anything.
Could you please investigate ?
Thank you


doc: add licence

As suggested in #1, I've added a basic MIT licence file & a shields.io badge for the licence in the readme

:tada: This PR is included in version 0.1.3 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Cow: Watch Drop_Notcoin_Robot(?# t.me)

Cow requests the watch of the watch_keyword Drop_Notcoin_Robot(?# t\.me). See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Drop_Notcoin_Robot(?# t\.me) has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by VLAZ in Charcoal HQ


Maven source download fails from intellij

Strangely, source download for 3.2.1-RELEASE version fail in intellij (whereas other spring and spring-cloud libraries don't suffer from this problem in my environment)

Strangely, artefacts from https://repo1.maven.org/maven2/org/springframework/cloud/spring-cloud-open-service-broker/3.1.2.RELEASE/ listed from https://mvnrepository.com/artifact/org.springframework.cloud/spring-cloud-open-service-broker/3.1.2.RELEASE have invalid size and seem empty, including the jar

I must have a configuration issue somewhere, otherwise project wouldn't build

This is the first I've been learned of this issue. I have made some significant gradle updates and changes in the 3.2.0-SNAPSHOT with how the jars are packaged and published, so I'm curious if it suffers from the same issue.


Pointwise POW Port Count fix.

POW operation requires 3 ports.
Currently cudnnFinalize fails with cudnn_status: CUDNN_STATUS_BAD_PARAM regardless whether you set B descriptor or not. This change fixes OperationBuilder and allows a successful execution.

Hi @Riottomsk ,
Thanks for pointing out the issue.
Over the past few months we have been refactoring our code internally. And hence your MR cannot be merged directly.
We have incorporated your suggestion in our latest release. (You can find mention of this PR in the release notes.)
We hope you like our new 1.0 API and examples.
Thanks


deprecation of developer.walmartapis.com

This site is being deprecated and will be removed on March 31, 2017. Please use the enhanced Developer Portal API site at: https://developer.walmart.com.

Over at developer.walmartapis.com, the document is pointing to a much more primitive version of the "enhanced Developer Portal API" at developer.walmart.com. The new site contains a much more basic set of UI and only the Feed APIs are listed there with things like items related APIs missing (https://developer.walmartapis.com/#overview). Do you think these APIs are also being deprecated or is it just the documentation site?

The new documentation is this one https://developer.walmart.com/#/apicenter/marketPlace

thanks @kushan85


Spec Version

Update text at http://docs.biocomputeobject.org/top-level/
2.0.1 BCO version "spec_version"
The version of the BCO specification used to define the BCO. It is recomended that this value be a permalink as defined in the w3id.org/biocompute repository.
"spec_version": "https://w3id.org/ieee/ieee-2791-schema/"

b9eb576

b9eb576


With 'watch' option enabled ALL chunks are removed after each new compilation

I wanted to try this new watch option, but it doesn't act as I expected.
When using webpack --watch with this option enabled ALL of the previously created chunks are removed after new compilation.
I am using CommonsChunkPlugin in order to apply long-term cache support. So when I use webpack --watch not all of my chunks are updated, but only those, which were affected by the last changes.
So I expected for clean-webpack-plugin to remove only obsolete chunks.
This would be very nice, cause I will be able to use webpack watch mode along with karma watch mode.
The problem is that I use [name].[chunkhash].js pattern for my app.js chunk (and others). So at the moment it is not very efficiently to use karma watch mode along with webpack watch mode, karma will include all my chunks (old and new), because it uses app*.js pattern for files to include in it's config.
I suppose this issue is related to this PR

OK, I actually think that this issue has nothing to do with this plugin. I think the feature I describe should be implemented in a separate plugin.

I've bumped into this exact issue. It would be ideal if watch: true could only clean the files that were effected by the change.

@simonsmith for this purpose you can use my plugin https://github.com/GProst/webpack-clean-obsolete-chunks

@GProst I literally just found that! It works perfectly. Thanks!


Feature/#21663

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add migration to clean up old verification state

Description
Clean up the redux store after the verification changes from this PR: https://github.com/celo-org/wallet/pull/377/
Other changes
N/A
Tested
Manually and with a unit test
How others should test
Upgrade for an older version (1.14.x or older) to the latest one and make sure that the verification status is not lost and there are no other unexpected things related to verification.
Related issues

Fixes #116

Backwards compatibility
N/A

Did both those things but I don't really understand why we have different versions in test/schemas.ts. Wouldn't it be easier to read if we had a full schema with all the properties instead of having it be divided by version? This version doesn't match the migration version either, so I feel like I'm missing something.

Would be easier to read, but the reason why it's divided by version is so we can use them to test migrations.
I think we should go back to have the test schemas version in sync as a good practice:
https://github.com/celo-org/wallet/blob/0db5533c13f4b8e787497c4333cb805fa3711e40/packages/mobile/test/schemas.ts#L368-L369

All right, I haven't been using the schemas from that file for migration tests but I can see why we would want that. In that case, I renamed the latest one to v13Schema, added a storeVersion field to it and a test that checks that it's in sync with the store. Hopefully this helps us to keep them in sync :) someone could still technically set the storeVersion field of the v13Schema to 14, but that should be flagged in a review, otherwise we could move the test schema file to an object similar to the migrations one.

hi @gnardini we have verified this issue on latest Android internal build V1.15.0 & Testflight build V1.15.0(63) and observe the followings:

[ ] User is able to see completed verification flow when user upgraded to V1.14-V1.15 if user already verified phone number in V1.14

[ ] User is able to see all data correctly cUSD/CELO

[ ] First & Second code fields are shown blank when user ungraded to V1.14-V1.15 given below are the steps to repro issue:

Install V1.14 build
Paste only first & second fields Keep third code as blank then skip the verification
Make sure user should able to see first & second codes when resume on V1.14 build
Upgrade it to V1.15 & go to verification flow
Observe First & second fields are shown blank.
Attachment
Please let us know we need to open new issue for this.
Thanks!

The described behavior is fine, thanks for checking!

https://valora.testquality.com requirement R32. Do not remove or edit.


Cropped image is not showing before page reload in bs modal. It saves first opened image does not destroy the old image and not showing the cropped on.

I have used fengyuanchen/cropper v0.9.2 for image crop and rotation in bootstrap modal. It's working fine for both crop and rotation but problem is that it saves the first image in modal even after crop, suppose I have opened an image in modal, crop it, save it(from the server) and dismiss the modal. Then if I open the image again it is showing the old image not the cropped one. But if I reload the page then it is showing cropped one. Below is some code that I have used.
$(document).on('click', '.original_image_show', function() { //click on the image
var data_original = $(this).find('img').attr('data-original');/ //get the src to to crop
// #originalImageShow is the modal div's id.
$("#originalImageShow").find(".bootstrap-modal-cropper img").attr('src', data_original); //set the src of bootstrap modal from clicked image's src.
var $image = $('.bootstrap-modal-cropper > img');
$('#originalImageShow').on('shown.bs.modal', function() {
//-----cropping code here-----
}).on('hidden.bs.modal', function() {
$image.cropper('destroy');//I have used this code to destroy the modal.
});

muradhassan , do u save img like in DEMO example? how u pass the data to php file for saving ??

Hi alefbetta, thank you for your reply. I have solved the problem. It was for caching of image. When I click the image first, then browser cached the image and next it shows the old image from cache not the cropped one.
var data_original = $(this).find('img').attr('data-original');  //get the src to to crop
var data_original = data_original+ '?'+Math.random()*Math.random();
// #originalImageShow is the modal div's id.
$("#originalImageShow").find(".bootstrap-modal-cropper img").attr('src', data_original); //set the src of bootstrap modal from clicked image's src.
I did not save image like DEMO example, I have used ThumbHandler php library, but modified the class because existing class has no rotation option.

muradhassan thanks for reply.
The problem is that i cannot send data to php file using DEMO example.  i get parseerror.
It would be useful if cropper Creator just update the DEMO example with PHP file for saving the cropped image. Then we will see a proper sample of how it should be...

Hi alefbetta,
You can try this:
      aspectRatio: 16 / 9,
      preview: '.img-preview',
      crop: function (data) {
       cropImgData = data; // save returned data to cropImgData.
       console.log(cropImgData);   cropImgData is the returned data from cropper

}
Now you can use cropImgData in any event to pass the data to your server side file.
suppose use this
$(document).on('click', '#saveCroppedBtn', function(){
var url ='your_ajax_page_url.php';
$.ajax({
type: "POST",
url: url,
data: cropImgData,
success: function(res) {
//do something.
}
});

muradhassan you are right and i went the same way, i used this
$(document).on('click', '#saveCroppedBtn',
function(){
var url ='crop.php';
$.ajax({
type: "POST",
url: url,
data: here is my data,
success: function(res) {
console.log(cropImgData);
}
});
});
But data must be sent in a proper way to crop.php file ...  so that'a the problem.. i need to gather all information encode it with jason and send to CROP.php file like  it goes in AVATAR Example

This is not a issue. Please understand the example! Don't just copy the code...


update method receives stale props

Problem
Scenario: I have a button component that receives a loading bool for display and an onclick callback
The Login page uses the button and keeps track of the loading state, passing it to the button
When the outer Component is called back on click, it passes the correctly updated props (logged) to the button, which does receive update() but with the initial props instead of the updated ones (logged).
Steps To Reproduce
Steps to reproduce the behavior:

Start the attached demo project
wasm-demo.zip

Run wasm-pack build && webpack

serve the dist/index.html file using your preferred app (e.g. right click -> open in browser in CLion/Intellij)

Open console, click button and notice the stale button state

Expected behavior
The button should receive the new props.
Screenshots

Environment:

Yew version 0.13.2 features = ["web_sys"]
Rust version:
cargo --version
cargo 1.41.0 (626f0f40e 2019-12-03)
// wasm-pack 0.9.1
Target if relevant [e.g. wasm32-unknown-emscripten]
web-sys = "0.3.36"
OS: Linux Mint; 4.15.0-54-generic
Browser chrome
79.0.3945.130 (Official Build) (64-bit)

Questionnaire

[x] I'm interested in fixing this myself but don't know where to start
[ ] I would like to fix and I have a solution
[ ] I don't have time to fix this right now, but maybe later

Update: In src/virtual_dom/vcomp.rs VComp::new gets called with the current props, while the closure that captured the old props is called a few lines and moments later.

@ZainlessBrombie hey 👋
I dug into this and was able to find the issue. Your button is nested like this: StyledComponent > CenterBox > Card > Button. While your Button::change method was properly updating the button props, the other three are not. If you implement change in StyledComponent by recreating vcomp on new props, and overwrite props in CenterBox and Box, everything should work as expected.
This is a very easy mistake to make so I'd like to update the component trait like I've described in  https://github.com/yewstack/yew/issues/830 and then have a proper default implementation for change that overrides props. This should help avoid this common footgun. Sorry you wasted time on this, but thanks for helping us realize how important it is to improve this API!

Oh dear. Ok yes I understand, it's because the children are kept internally and are thus stale.
And explains why I have read through literally the whole virtual_dom part of the library and could only determine that everything was sound :smile:
While I am a little frustrated I didn't realize this myself, thank you very much for taking care of this issue :)

Don't sweat it, it's too unintuitive IMO! As soon as someone picks up #830 things will be much better because Yew will hold onto props internally


SwitchRexpression Pattern Matching is not breaking

The 3rd part of this is way past the printWidth but doesn't break.
var member = expression switch
{
    MemberExpression { Expression: Expression target, Member: MemberInfo propertyOrField } => new Member(
        expression,
        propertyOrField,
        target
    ),
    MethodCallExpression { Method: var instanceMethod, Object: Expression target } => new Member(
        expression,
        instanceMethod,
        target
    ),
    MethodCallExpression { Method: var extensionMethod, Arguments: { Count: > 0 } arguments } when extensionMethod.Has<ExtensionAttribute>() => new Member(
        expression,
        extensionMethod,
        arguments[0]
    ),
    _ => default
};

I believe the problem is more with the IsPattern, see
            if (
                !(node
                    is PrefixUnaryExpressionSyntax { Operand: ParenthesizedExpressionSyntax { Expression: IsPatternExpressionSyntax { Pattern: DeclarationPatternSyntax, } isPattern, }, } notExpression)
            ) {
                return;
            }

This will hopefully be resolved in #154

The original issue was resolved with #382


Constantly seeing: A connection to http://<IP_ADDRESS>:54688/ was leaked

Hello, I regularly see these warning logs from OkHttpClient:
A connection to http://<IP_ADDRESS>:54688/ was leaked. Did you forget to close a response body?
Is this a known issue or am I doing something wrong?

Same here...

same.

same to me,
when use ExoPlayer

same to me, when use AndroidSystemPlayer


Warning in Docker builds

Module not found: Error: Can't resolve 'utf-8-validate' in '/home/node/app/node_modules/ws/lib'
Whyyyyy?

Module not found: Error: Can't resolve 'bufferutil' in '/home/node/app/node_modules/ws/lib'


Support correspondence mutator for collections and json.

Do we have other types where that'd be useful?

Done. Actually json already implemented mutable collections so did not have to change anything there :)


Cancer example for the docs

Used the cancer data example from the examples directory and to it I added a short discussion on the choice of lenses.

Thank you! I'll give it a thorough review early next week.  I'd like to add this in a more prominent way to the docs site.

Cool!

hey @karinsasaki, I finally got a chance to look through the notebook, and it's great! I didn't realize there was all the code demonstrating great use of the plotly vis stuff.
The PR #172 better incorporates lots of what you wrote.
I was thinking, would it make sense to take a bunch of the plotly examples and put them in the Plotly-demo notebook, down at the empty 'Customization' section?  Then we can keep the Cancer-demo notebook to be very focused on working through different lenses.


⚠ Secret Website 4 has degraded performance

In beab987, Secret Website 4 ($SECRET_WEBSITE_4) experienced degraded performance:

HTTP code: 200
Response time: 5067 ms

Resolved: Secret Website 4 performance has improved in 9ba71ce.


Confusing short shortcut

Actually (you know), the short is 2 byte integer [−32767, 32767] so maybe it is good to "decrease" its size. I understand that maybe you would like to emulate a half of int (similar to C-like byte,short,int,long) but having short as 2 byte integer and int for other integers (4 and 8) avoids the confusing.

I agree. What name do you suggest for 4 byte integers? int is already taken by Dart for 8 byte integers ☹

As I understand you need new names for distinction? So here it is the solution:
typedef byte = int;
typedef short = int;
typedef int32 = int;
typedef int64 = int;
typedef long = int;

Numbers are not recognised by syntax highlighting 😭

Shit happens :)
typedef byte = int;     // 1 byte
typedef short = int;    // 2 bytes
typedef shortint = int; // 4 bytes
typedef longint = int;  // 8 bytes
typedef long = int;     // 8 bytes

closing this for now because I haven't seen other users disliking the naming and I don't really see a great alternative


Cannot load filer-metadata to different filer when IP differs

Describe the bug
When exporting metadata using fs.meta.save, it gets saved to a local file in the same directory.
It chooses a random filer to save from. This can be problematic, when using multiple and they havent synced (yet or never due to errors).
When importing the metadata using fs.meta.load <file>, it wants to import the metadata into the same filer it was saved from (identified by its IP).
This is a bit unintuitive and doesnt work at all when using containers and dynamically assigned IPs.
System Setup
weed version
version 30GB 3.53 2c4c2f0994604510631920f4d0d9ee817ec29224 linux amd64
Expected behavior
fs.meta.save should optionally allow setting which filer should be chosen.
fs.meta.load <file> should optionally allow setting which filer should be chosen. If none was chosen, choose a random one (they should then sync to-eachothers anyways)
Additional Context
I am not sure if that maybe has a special reason?

I have checked the code and apparently there is no IP read.. maybe i was just too fast when re-reading :)
That or mixed with theother problem i had, the filers were so out of sync and it was weird..


[bug] Heruistic of deciding whether something is component is impacted by satisfies

Describe the bug
Hello 👋
I'm not 100% sure whether this is a bug and how to treat that. There're some issues with heurisitc that decides whether something is component or not. I'm using this plugin within my Remix app and stuff like Remix loader and action which are functions exported from route modules are sometimes flagged as components.
I've tried to dive into what exactly causes that and one thing that caught me off guard is the fact that me using satisfies impact this somehow?
Screenshots using the built-in debug config:
While typing these functions with satisfies they're somehow inferred as components?

When they're typed exactly the same, but I'm not using satisfies they aren't identified as components:

This causes problems with some issues e.g. "prefer readonly props". Thanks in advance for looking into it and for the time spent on this plugin 😃
Reproduction
Here's a repository with the debug rules setup - https://github.com/pawelblaszczyk5/react-eslint-x-repro
app/root.tsx file has these exports typed with satisfies
app/routes/_index.tsx don't use satisfies
Expected behavior
These functions ideally wouldn't be marked as components, using/not using satisfies shouldn't impact the way they're identified under the hood?
Platform and versions
eslint: 9.13.0,
typescript-eslint: 8.11.0,
@eslint-react/eslint-plugin: 1.15.0

The rest is not applicable here but I can provide anything more needed.

Stack trace
n/a

Additional context
n/a

Thank you for providing the reproduction. You are correct; these functions should not be marked as components. This is indeed a bug. It appears that when accessing the parent node to retrieve the function's variable name and checking its capitalization, we did not account for cases where it is wrapped by TypeScript's as and satisfies expressions, which affected the component detection.

Thanks a lot!


Error getting Rcall to start for the first time

I tried this code, which is the first example given in the Readme.md file.
load('TestData.mat')
Rpush('dat',dat,'grp',grp)
Rrun('fit <- lmFit(dat,grp)')
fit = Rpull('fit');
Rclear

I have R 4.2.1 and MATLAB R2022b on Windows
the result is

Rcall: An R interface for MATLAB.
Copyright (C) 2022 Janine Egert and Clemens Kreutz.
THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES ARE DISCLAIMED; for details see "LICENSE".
Redistribution and use in source and binary forms, with or without modification, are permitted provided certain conditions; for details see "LICENSE".
Error using Rinit
Check if the Rpath "the" is correct and if the R path is set in the
environmental variables. '"the"' is not recognized as an internal or external
command,
operable program or batch file.

Hi Mike,
thanks for asking. This means your R path is unknown, or somehow set to "the". The R path is set in the Rinit function, see line 63 OPENR.Rexe = system('where R.exe'). Can you get a hint where your R path is set to "the" by checking the OPENR.Rexe variable in line 63 ff. ?
Best,
Janine

Wow! Fast reply Janine :)  Thank you.
I ask for the environment variable Rpath in MATLAB and it seems to be empty.  Tried a couple of capitalisation variations too
>> getenv('Rpath')

ans =

  0×0 empty char array

>> getenv('RPATH')

ans =

  0×0 empty char array

When is this environment variable supposed to be set?  Should I have done it manually or should the R installer have done it?
Explicitly giving the path to my R, works fine.
rpath = "C:\Program Files\R\R-4.2.1\bin\x64\R.exe"
Rinit('limma',rpath)
Rcall: An R interface for MATLAB.
Copyright (C) 2022 Janine Egert and Clemens Kreutz.
THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES ARE DISCLAIMED; for details see "LICENSE".
Redistribution and use in source and binary forms, with or without modification, are permitted provided certain conditions; for details see "LICENSE".

Rversion =

    'R version 4.2.1 (2022-06-23 ucrt) -- "Funny-Looking Kid"'

This made me think that limma had been loaded but
load('TestData.mat',rpath)
Rpush('dat',dat,'grp',grp)
Rrun('fit <- lmFit(dat,grp)')
fit = Rpull('fit');
Rclear

Warning: Variable 'C:\Program Files\R\R-4.2.1\bin\x64\R.exe' not found.
Warning: Ignoring the version specified. The version flag is not required when
using the '-append' flag.
> In Rpush (line 97)
Warning: Ignoring the version specified. The version flag is not required when
using the '-append' flag.
> In Rpush (line 97)
Error using Rexec
Error in Rrun.R : Installation of package limma was not successfull. Try
package installation in R beforehand. If your package has been installed, check
if the R version and the R libraries are set in the system environmental
variables.  there is no package called 'BiocManager'

Error in Rpull (line 23)
Rexec

I tried to install limma in R but
> install.packages('limma')
Warning in install.packages("limma") :
  'lib = "C:/Program Files/R/R-4.2.1/library"' is not writable
--- Please select a CRAN mirror for use in this session ---
Warning message:
package ‘limma’ is not available for this version of R

A version of this package for your version of R might be available elsewhere,
see the ideas at
https://cran.r-project.org/doc/manuals/r-patched/R-admin.html#Installing-packages

Fair enough that limma didn't work since I can't seem to install it in R but this is the first example seen in the Readme file.
Maybe a simpler first example might be helpful for newbies like me?

Line 63 of Rinit.m for me is commented out
%         if ~strcmp(OPENR.Rexe(1),'"')

and is unchanged from when I downloaded it from File Exchange

Even though R is installed on my machine, I get
system('where R.exe');
INFO: Could not find files for the given pattern(s).

Hi Mike,
true my line 63 has been your line 69 the one in your last comment (system('where R.exe'). That is, unfortunately, the output of your Windows console, so R can not be found by your Windows console. There are two options, either you add the Rpath to your system environment variables, so Rcall is able to automatically find your R installation folder, or you specify the R path yourself as an input variable like you did. This is also safer to keep up with your R version, etc. Actually, I captured this error in the code by identifying the word 'information', however this seems to be a German translation for 'info' (not quite, but sometimes computers do different things than brains). So I will update it. And I also added a linear model example without limma.
Thanks for the hint!
Best,
Janine

I understand now where "the" came from.
Because R isn't found by the where command:
OPENR.Rexe = system('where R.exe')

gives

'INFO: Could not find files for the given pattern(s).
     '

Lines 71 and 72 of the current File Exchange version are
OPENR.Rexe = sort(strsplit(OPENR.Rexe));
OPENR.Rexe = OPENR.Rexe{end};

which leaves us with
OPENR.Rexe

ans =

    'the'

Thanks Janine.  I'm getting there :)
Now I'm trying your linear model example.  This is my full code
rpath = "C:\Program Files\R\R-4.2.1\bin\x64\R.exe";
Rinit('',rpath);

load('hospital')
tbl = table(hospital.Sex,hospital.BloodPressure(:,1),hospital.Age,hospital.Smoker, hospital.Weight, ...
    'VariableNames',{'Sex','BloodPressure','Age','Smoker','Weight'});
Rpush('tbl',tbl)

%% Matlab
fit = fitlm(tbl,'BloodPressure~Age+Smoker+Weight');
cm = fit.Coefficients.Estimate

%% R
Rrun('fit <- lm(BloodPressure~Age+Smoker+Weight, tbl)')       		%# fit linear model
Rrun('cr <- fit$coefficients')								        %# empirical Bayes statistics
%Rrun('p <- fit$p.value')

% Evaluate R commands and get variables
cr = Rpull('cr')

There is no mention of limma here but it is still stuck on my failed attempt earlier:
Error using Rexec
Error in Rrun.R : Installation of package limma was not successfull. Try package
installation in R beforehand. If your package has been installed, check if the R
version and the R libraries are set in the system environmental variables.
there is no package called 'BiocManager'

Error in Rpull (line 23)
Rexec

Error in r_example (line 19)
cr = Rpull('cr')

Somehow it has stored the fact that I tried to install limma and keeps trying to do so.

Hi Mike,
thanks, these lines were updated in github. Now a new File Exchange version exists too.
Oh you prompted your new status.. try Rclear, to clear all existing Rcall files/variables..

Rclear worked!  Rebooting the machine didn't :)
However, now I have
Error using Rexec
Error in Rrun.R : all elements of a list must be named

Error in Rpull (line 23)
Rexec

Error in r_example (line 19)
cr = Rpull('cr')

Hi Mike,
this is a problem in the R code, you can run the Rrun.R file in R to see in which line there is a problem. I will update my R version to see if that causes the problem. In R 4.1.2 it still works. We will see.

I installed the older version. It's not working in my R 4.1.2.  Same error.
Which version of MATLAB are you using?
I edited the Rrun.R file to expose the error and it was this:
> source("Rrun.R")
Error in attach(data_Rpush) : all elements of a list must be named

I tried Matlab R2020b and R2021b and R-4.1.2 and R-4.2.1 and did not get any error..

Is the R.matlab package working? How does your data_Rpush look like? Is it correctly read in by readMat? Should be a list of 2 with OPENR and tbl.


AzCopy always creates directory and zero length file with same name as directory

Which version of the AzCopy was used? 10.20.1
Which platform are you using? (ex: Windows, Mac, Linux)
Linux
Linux version 5.15.0-1042-azure (buildd@lcy02-amd64-005) (gcc (Ubuntu 9.4.0-1ubuntu1~20.04.1) 9.4.0, GNU ld (GNU Binutils for Ubuntu) 2.34) #49~20.04.1-Ubuntu SMP Wed Jul 12 12:44: 56 UTC 2023
What command did you run?
azcopy cp "/mnt/repos/test_logs/*" https://myaccount.dfs.core.windows.net/packagetests/test_logs/ --log-level DEBUG
and
azcopy cp --recursive=true /mnt/repos/test_logs/ https://myaccount.dfs.core.windows.net/packagetests/
What problem was encountered?
When the files are uploaded a zero-length file is created with the same name as the directory. This means, when you download to Windows, this does not work because it's an invalid state.

How can we reproduce the problem in the simplest way?

Try to copy a directory from Linux to a storage account using this command:
azcopy cp --recursive=true /mnt/repos/test_logs/ https://myaccount.dfs.core.windows.net/packagetests/
Inspect the storage account and there will be both a directory created and a file with the same name as the directory.

Have you found a mitigation/solution?
You can avoid this by not copying directories.

Hi, include "--include-directory-stub=false" in your command.


Return PENDING status for transactions without receipts

When you first submit a transaction but before it has been mined, the
transaction receipt comes back as undefined. This commit changes the behavior of
this code to a) not throw an error in this case, b) return "PENDING" for those
transactions.

@pcardune – it's useful to be able to distinguish pending transactions. However, pre-Byzantium JSON-RPC didn't return the status field at all.

It also returns either :
root : DATA 32 bytes of post-transaction stateroot (pre Byzantium)
status: QUANTITY either 1 (success) or 0 (failure)

https://github.com/ethereum/wiki/wiki/JSON-RPC#eth_gettransactionreceipt
I think we should cover all cases:

status informed (SUCCESS or FAILURE)
receipt is null (PENDING)
status is empty (happens for pre-Byzantium, in which case the status field would be null)

Would that work for you?

That works for me. updated!


[BUG] Drawing a pattern with a path effect is not clipping to the points

Description
I am drawing a path of points with an inner hatch pattern. However, the pattern is not getting clipped to the interior of the points. Note ( I have also tried g.ClipPath(gp), but it did not affect the results). Below is the path drawn with the inner path and later with the outline of the points in the path. The green hatch is the path effect. (Notice that the hatch is getting drawn beyond the path points.)

Code
        Dim hatchMatrix As SKMatrix = SKMatrix.CreateScale(10, 10)
        Dim hatchWidth As Single = 20

        ' create the path (diagonal hatch) with the center at 0,0
        Dim hatchPath = New SKPath()
        hatchPath.MoveTo(0, hatchWidth)
        hatchPath.LineTo(hatchWidth, 0)
        hatchPath.LineTo(0, -hatchWidth)
        hatchPath.LineTo(-hatchWidth, 0)
        hatchPath.LineTo(0, hatchWidth)

        Using hatchPaint As New SKPaint
            hatchPaint.PathEffect = SKPathEffect.Create2DPath(hatchMatrix, hatchPath)
            hatchPaint.Color = SKColors.LimeGreen
            hatchPaint.Style = SKPaintStyle.Stroke
            hatchPaint.StrokeWidth = 1

            g.DrawPath(gp, hatchPaint)
        End Using

Expected Behavior
I would expect that the hatch is getting clipped to the polygon itself, not the outer extents of the polygon.
Actual Behavior
The hatch pattern is getting clipped to the extents of the polygon.
Basic Information
net6.0-windows10.0.19041.0
SkiaSharp version 2.88.2

I used your code to generate the same effect and with ClipPath before DrawPath I achieved the desired effect:
g.Save() ' this is to be able to restore clipping path
    g.ClipPath(gp)
g.DrawPath(gp, hatchPaint)
g.Restore()

Before clip:

After clip applied:


Add YandexGPT LLM model

Introducing an ability to work with the YandexGPT language model.

Looks good, thank you!


Need HTTPS authentication functionality

If I run compy with mitm support and supply username and password, on my remote computer browser and phone, whenever I try to access a site, a dialog pops up asking for username and password and then errors out. It say ERR_UNEXPECTED_PROXY_AUTH.
Note: authentication only works with http sites and not https

Nvm, found a solution here (tested with mitm support) https://github.com/ggramaize/compy/commit/b1fdc4716eb8ed2c63556372b65680fdf8dbec3d


Inline mobile and desktop conditional

Dear Team,
can we have better inline condition  like
<mj-text font-size="{sm: "20px", "lg": "25px"}">Good job team!</mj-text>

or
<mj-text sm-font-size="20px" lg-font-size="25px">Good job team!</mj-text>

Thanks team

Hi !
We don't want to do this as it would be expected to work on any tag, we prefer to let user declare their media query so they can have full control of responsivness instead of hiding it behind a non customizable attribute


Add support for flash.rs on bigger chips

The current code to erase flash is only using the mer field.
On bigger chips, there is two memory banks, so the mer field is split in two for the two memory banks, mer1 and mer2.
This PR add checks to configure either mer or mer[12] for erase functions.

See https://github.com/stm32-rs/stm32f7xx-hal/pull/167#issuecomment-1024871906.

I confirm that this example works on my Nucleo-F767ZI


"editor.suggestSelection" fix #42795

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


send_poi no longer working after captcha requirement

Describe the issue
I'm able to successfully use the CLI to get the status of my vehicle using the captcha-token. However I'm no longer able to call send_poi. I was able to use the CLI with this vehicle before the captcha requirement.
You can see the 200 response in debug so it's not the account or auth
% bimmerconnected --debug sendpoi --captcha-token $CAPTCHA $BMW_USER_NAME $BMW_PASSWORD $BMW_REGION $BMW_VIN $LAT $LON
DEBUG:bimmer_connected.account:Getting vehicle list
DEBUG:bimmer_connected.account:Getting vehicle list
DEBUG:httpx._client:HTTP Request: POST https://cocoapi.bmwgroup.us/eadrax-vcs/v5/vehicle-list "HTTP/1.1 200 OK"
DEBUG:httpx._client:HTTP Request: GET https://cocoapi.bmwgroup.us/eadrax-vcs/v5/vehicle-data/profile "HTTP/1.1 200 OK"
DEBUG:httpx._client:HTTP Request: POST https://cocoapi.bmwgroup.us/eadrax-vcs/v5/vehicle-list "HTTP/1.1 200 OK"
DEBUG:httpx._client:HTTP Request: POST https://cocoapi.bmwgroup.us/eadrax-vcs/v5/vehicle-list "HTTP/1.1 200 OK"
DEBUG:bimmer_connected.vehicle.vehicle:Getting vehicle list
DEBUG:httpx._client:HTTP Request: GET https://cocoapi.bmwgroup.us/eadrax-vcs/v4/vehicles/state?apptimezone=0&appDateTime=1732638728000 "HTTP/1.1 200 OK"
DEBUG:httpx._client:HTTP Request: POST https://cocoapi.bmwgroup.us/eadrax-dcs/v2/user/b2409aba-1056-4e5b-88b7-e77e182a4933/send-to-car "HTTP/1.1 400 Bad Request"
ERROR:bimmer_connected.api.client:MyBMWAPIError due to HTTPStatusError: Bad Request -
MyBMWAPIError: HTTPStatusError: Bad Request -
Expected behavior
send_poi should send the coordinates to my BMW iDrive as a message allowing me to set the destination
Which Home Assistant version are you using?
command is running in terminal for testing
What was the last working version of Home Assistant Core?
pre-captcha was working
What is your region?
North America
MyBMW website

[X] I can still successfully login to the BMW MyBMW website and the car status is available there.
[X] I have MyBMW enabled for my vehicle.

Number of cars

[ ] I have 2 or more cars linked to the MyBMW account.
[ ] I have a Mini vehicle linked to my account.
[ ] I have a Toyota Supra vehicle linked to my account.

Output of bimmer_connected fingerprint
{
    "capabilities": {
        "a4aType": "BLUETOOTH",
        "alarmSystem": false,
        "climateFunction": "VENTILATION",
        "climateNow": true,
        "climateTimerTrigger": "DEPARTURE_TIMER",
        "digitalKey": {
            "bookedServicePackage": "NONE",
            "isDigitalKeyFirstSupported": false,
            "state": "NOT_AVAILABLE",
            "vehicleSoftwareUpgradeRequired": false
        },
        "horn": true,
        "isBatteryPreconditioningSupported": false,
        "isBmwChargingSupported": false,
        "isCarSharingSupported": false,
        "isChargeNowForBusinessSupported": false,
        "isChargingHistorySupported": false,
        "isChargingHospitalityEnabled": false,
        "isChargingLoudnessEnabled": false,
        "isChargingPlanSupported": false,
        "isChargingPowerLimitEnabled": false,
        "isChargingSettingsEnabled": false,
        "isChargingTargetSocEnabled": false,
        "isClimateTimerSupported": true,
        "isClimateTimerWeeklyActive": false,
        "isCustomerEsimSupported": false,
        "isDCSContractManagementSupported": false,
        "isDataPrivacyEnabled": false,
        "isEasyChargeEnabled": false,
        "isEvGoChargingSupported": false,
        "isLocationBasedChargingSettingsSupported": false,
        "isMiniChargingSupported": false,
        "isNonLscFeatureEnabled": false,
        "isOptimizedChargingSupported": false,
        "isPersonalPictureUploadSupported": false,
        "isPlugAndChargeSupported": false,
        "isRemoteEngineStartSupported": false,
        "isRemoteHistoryDeletionSupported": false,
        "isRemoteHistorySupported": true,
        "isRemoteParkingEes25Active": false,
        "isRemoteParkingSupported": false,
        "isRemoteServicesActivationRequired": false,
        "isRemoteServicesBookingRequired": false,
        "isScanAndChargeSupported": false,
        "isSustainabilityAccumulatedViewEnabled": false,
        "isSustainabilitySupported": false,
        "isThirdPartyAppStoreSupported": false,
        "isWifiHotspotServiceSupported": false,
        "lastStateCallState": "ACTIVATED",
        "lights": true,
        "locationBasedCommerceFeatures": {
            "fueling": false,
            "parking": false,
            "reservations": false
        },
        "lock": true,
        "personalPictureUpload": {
            "aspectRatio": "2:1",
            "state": "NOT_AVAILABLE"
        },
        "remote360": true,
        "remoteChargingCommands": {},
        "remoteServices": {
            "batteryPreconditioningControl": {
                "executionApi": "RS_API_V5",
                "id": "batteryPreconditioningControl",
                "state": "NOT_AVAILABLE"
            },
            "centralLockControl": {
                "executionApi": "RS_API_V5",
                "id": "centralLockControl",
                "state": "NOT_AVAILABLE"
            },
            "departureTimerControl": {
                "executionApi": "RS_API_V5",
                "id": "departureTimerControl",
                "state": "NOT_AVAILABLE"
            },
            "doorLock": {
                "executionApi": "RS_API_V5",
                "id": "doorLock",
                "state": "ACTIVATED"
            },
            "doorUnlock": {
                "executionApi": "RS_API_V5",
                "id": "doorUnlock",
                "state": "ACTIVATED"
            },
            "hornBlow": {
                "executionApi": "RS_API_V5",
                "id": "hornBlow",
                "state": "ACTIVATED"
            },
            "inCarCamera": {
                "executionApi": "RS_360_V4",
                "id": "inCarCamera",
                "state": "NOT_AVAILABLE"
            },
            "inCarCameraDwa": {
                "executionApi": "RS_360_V4",
                "id": "inCarCameraDwa",
                "state": "NOT_AVAILABLE"
            },
            "interiorPreconditioningControl": {
                "executionApi": "RS_API_V5",
                "id": "interiorPreconditioningControl",
                "state": "NOT_AVAILABLE"
            },
            "lightFlash": {
                "executionApi": "RS_API_V5",
                "id": "lightFlash",
                "state": "ACTIVATED"
            },
            "remote360": {
                "executionApi": "RS_360_V4",
                "functions": {
                    "encryptionMethod": [
                        "ENCRYPTION_METHOD_A"
                    ]
                },
                "id": "remote360",
                "state": "ACTIVATED"
            },
            "surroundViewRecorder": {
                "executionApi": "RS_360_V4",
                "id": "surroundViewRecorder",
                "state": "NOT_AVAILABLE"
            },
            "telematicsWakeup": {
                "executionApi": "RS_API_V5",
                "id": "telematicsWakeup",
                "state": "NOT_AVAILABLE"
            },
            "wakeup": {
                "executionApi": "RS_API_V5",
                "id": "wakeup",
                "state": "NOT_AVAILABLE"
            },
            "windowControl": {
                "executionApi": "RS_API_V5",
                "id": "windowControl",
                "state": "NOT_AVAILABLE"
            }
        },
        "sendPoi": true,
        "specialThemeSupport": [],
        "thirdPartyAppStore": {
            "state": "NOT_AVAILABLE"
        },
        "unlock": true,
        "vehicleFinder": true,
        "vehicleStateSource": "LAST_STATE_CALL"
    },
    "state": {
        "checkControlMessages": [
            {
                "severity": "LOW",
                "type": "ENGINE_OIL"
            }
        ],
        "climateTimers": [
            {
                "departureTime": {
                    "hour": 7,
                    "minute": 0
                },
                "isWeeklyTimer": false,
                "timerAction": "DEACTIVATE",
                "timerWeekDays": []
            },
            {
                "departureTime": {
                    "hour": 7,
                    "minute": 0
                },
                "isWeeklyTimer": true,
                "timerAction": "DEACTIVATE",
                "timerWeekDays": [
                    "MONDAY"
                ]
            },
            {
                "departureTime": {
                    "hour": 7,
                    "minute": 0
                },
                "isWeeklyTimer": true,
                "timerAction": "DEACTIVATE",
                "timerWeekDays": [
                    "MONDAY"
                ]
            }
        ],
        "combustionFuelLevel": {
            "range": 219,
            "remainingFuelLiters": 22
        },
        "currentMileage": 111573,
        "departurePlan": {},
        "doorsState": {
            "combinedSecurityState": "UNLOCKED",
            "combinedState": "CLOSED",
            "hood": "CLOSED",
            "leftFront": "CLOSED",
            "leftRear": "CLOSED",
            "rightFront": "CLOSED",
            "rightRear": "CLOSED",
            "trunk": "CLOSED"
        },
        "driverPreferences": {
            "lscPrivacyMode": "OFF"
        },
        "isLeftSteering": true,
        "isLscSupported": true,
        "lastFetched": "2024-11-26T17:21:03.888Z",
        "lastUpdatedAt": "2024-11-26T13:23:57Z",
        "location": {
            "address": {
                "formatted": "some_formatted_address"
            },
            "coordinates": {
                "latitude": 12.3456,
                "longitude": 34.5678
            },
            "heading": 123
        },
        "range": 219,
        "requiredServices": [],
        "roofState": {
            "roofState": "CLOSED",
            "roofStateType": "SUN_ROOF"
        },
        "securityOverviewMode": null,
        "vehicleSoftwareVersion": {
            "iStep": {
                "iStep": 540,
                "month": 11,
                "seriesCluster": "S15A",
                "year": 19
            },
            "puStep": {
                "month": 11,
                "year": 19
            }
        },
        "windowsState": {
            "combinedState": "CLOSED",
            "leftFront": "CLOSED",
            "leftRear": "CLOSED",
            "rightFront": "CLOSED",
            "rightRear": "CLOSED"
        }
    }
}

Anything in the logs that might be useful for us?
DEBUG:bimmer_connected.account:Getting vehicle list
DEBUG:bimmer_connected.account:Getting vehicle list
DEBUG:httpx._client:HTTP Request: POST https://cocoapi.bmwgroup.us/eadrax-vcs/v5/vehicle-list "HTTP/1.1 200 OK"
DEBUG:httpx._client:HTTP Request: GET https://cocoapi.bmwgroup.us/eadrax-vcs/v5/vehicle-data/profile "HTTP/1.1 200 OK"
DEBUG:httpx._client:HTTP Request: POST https://cocoapi.bmwgroup.us/eadrax-vcs/v5/vehicle-list "HTTP/1.1 200 OK"
DEBUG:httpx._client:HTTP Request: POST https://cocoapi.bmwgroup.us/eadrax-vcs/v5/vehicle-list "HTTP/1.1 200 OK"
DEBUG:bimmer_connected.vehicle.vehicle:Getting vehicle list
DEBUG:httpx._client:HTTP Request: GET https://cocoapi.bmwgroup.us/eadrax-vcs/v4/vehicles/state?apptimezone=0&appDateTime=1732638728000 "HTTP/1.1 200 OK"
DEBUG:httpx._client:HTTP Request: POST https://cocoapi.bmwgroup.us/eadrax-dcs/v2/user/b2409aba-1056-4e5b-88b7-e77e182a4933/send-to-car "HTTP/1.1 400 Bad Request"
ERROR:bimmer_connected.api.client:MyBMWAPIError due to HTTPStatusError: Bad Request -
MyBMWAPIError: HTTPStatusError: Bad Request -

Additional information
No response

Can't close this until they release the fixed library and I update the plugin to use it.


Beta 14 - Caching issue?

I am using beta 14 on Ubuntu 14.04, and I have the following issues:

every time I open franz, my office 365 session is logged out and I have to log back in again
every time one of my slack channels is out of focus (so not everytime I re-start franz), it seems to be logging out and for whatever reason it takes forever to reload, making all my slack channels unusable.

This is the output of my console (please notice the possible memory leak):
rollup-core_required_libs.8e73dad….min.js:93 Warning: a promise was created in a handler but was not returned from it
    at M (https://a.slack-edge.com/bv1-1/rollup-core_required_libs.8e73dada0d381d3f8d4b.min.js:93:17087)
    at O.r._warn (https://a.slack-edge.com/bv1-1/rollup-core_required_libs.8e73dada0d381d3f8d4b.min.js:93:13380)
    at Object.I [as checkForgottenReturns] (https://a.slack-edge.com/bv1-1/rollup-core_required_libs.8e73dada0d381d3f8d4b.min.js:93:16899)
    at O._settlePromiseFromHandler (https://a.slack-edge.com/bv1-1/rollup-core_required_libs.8e73dada0d381d3f8d4b.min.js:93:50842)
From previous event:
    at O.P [as _captureStackTrace] (https://a.slack-edge.com/bv1-1/rollup-core_required_libs.8e73dada0d381d3f8d4b.min.js:93:16419)
    at O._resolveFromExecutor (https://a.slack-edge.com/bv1-1/rollup-core_required_libs.8e73dada0d381d3f8d4b.min.js:93:50178)
    at new O (https://a.slack-edge.com/bv1-1/rollup-core_required_libs.8e73dada0d381d3f8d4b.min.js:93:43920)
    at /opt/Franz/resources/app.asar/node_modules/electron-spellchecker/lib/spell-check-handler.js:73:99
    at SpellCheckHandler.switchLanguage (/opt/Franz/resources/app.asar/node_modules/electron-spellchecker/lib/spell-check-handler.js:465:7)
    at Spellchecker.enable (/opt/Franz/resources/app.asar/webview/spellchecker.js:22:25)
    at EventEmitter._electron.ipcRenderer.on (/opt/Franz/resources/app.asar/webview/plugin.js:39:20)
    at emitTwo (events.js:106:13)
    at EventEmitter.emit (events.js:194:7)
v @ rollup-core_required_libs.8e73dad….min.js:93
rollup-core_required_libs.8e73dad….min.js:93 Uncaught Error: cannot enable cancellation after promises are in use
    at Function.r.config (rollup-core_required_libs.8e73dad….min.js:93)
    at e (rollup-core_required_ts.1dbcd33….min.js:1)
    at Object.e [as boot] (rollup-core_required_ts.1dbcd33….min.js:1)
    at messages:2279
rollup-core_required_libs.8e73dad….min.js:201 SoundManager 2: No Flash detected. Trying HTML5-only mode.
rollup-core_required_libs.8e73dad….min.js:201 SoundManager V2.97a.20150601 (HTML5-only mode)
rollup-core_required_libs.8e73dad….min.js:201 SoundManager 2 HTML5 support: mp3 = true, mp4 = true, ogg = true, opus = true, wav = true
rollup-core_required_libs.8e73dad….min.js:201 SoundManager 2: Ready. ✓
webview.js:5 getTeamIcon 0
webview.js:10 Uncaught TypeError: Failed to execute 'getComputedStyle' on 'Window': parameter 1 is not of type 'Element'.
    at getTeamIcon (webview.js:10)
    at setTimeout (webview.js:38)
internal/process/warning.js:21 (node:3071) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 electron-remote-event-webcontents-5-context-menu-5 listeners added. Use emitter.setMaxListeners() to increase limit
process.on @ internal/process/warning.js:21

Update: actually, slack channels are NEVER loaded. This does not happen in beta 13, though.

Similar to #369 (since my slack channels did not work, I had to downgrade to beta 13 before receiving the notivication about #369 :sunglasses: ). Closing.


Semantics: "Header" should be "Heading" in rich.html

I'd like to point out minor semantics issues I see in rich.html.
Currently the example refers to h1–h6 as “headers”, when really they should be called “headings”.
This perpetuates incorrect semantics when referring to these tags. “Headers” are generally known to be layout blocks that contain header information, while “Headings” are defined as block-level text titles. While people casually use these two terms interchangeably, they really should be better defined to avoid confusion.

Thanks for the feedback. Would be happy to accept a pull request for this if you have time.

I can take on this @tylercraft , but to be clear does it mean changing, e.g header-one  to heading-one ?

Any reference to headers should be changed to headings. It's how the W3C refers to them as well
https://www.w3.org/TR/html-markup/h1.html

Semantically this should change, but it will be too big of an issue to migrate. Therefore this is not something we will do so I am closing it out.


representation of port using a path

Hi,
is it possible to convert localhost:8000 to some path like localhost/rfdoc or something like this? if yes which files needs to be adapted?
because I don't want the port number to be visible, it would be nice if it is represented using a path.
Thanks and regards
Divakar R

Hi @divakar5, this would probably require having a public IP, and domain, and it is outside of package possibilities.


Passing 0 to pollInterval should mean "don't poll"

The vue-apollo query docs' descriptions of pollInterval simply links to the Apollo query docs, which say that a value of 0 means that no polling takes place. But when 0, or any invalid value such as -1 or undefined, is passed to the pollInterval prop, vue-apollo polls the query as fast as possible. The startPolling() function behaves slightly better, as it errors out on falsey values.

I'm not doing something special regarding polling, just passing the option along to apollo-client.


[feature] add filtering capability

re: #3 (point 5 and filtering) - i see that there's a search field and i think that's a good thing to have, but what i'm referring to is a filter system that auto-filters articles - so in other words the user creates a filer in the options (filter option) "headline contains" (string) "breaking" and then that article is mirrored in a special 'folder' for filtered items - for instance in Feedbro, that folder is labeled "starred items"
the purpose is make sure one doesn't miss something they are particularly interested in, so the article is copied to the special folder and the user can immediately see, upon clicking that folder (or category or whatever you want to call it) that something important has landed there rather than having to sift through potentially hundreds of articles - so this is just sort of an automated version of the current 'search' function, but indeed it can be quite handy

Okay, I see. This is indeed nice to have feature, though it's not high priority

As I got back to this issue it seems it's not actually filtering as creating multiple filters and listing only items meeting certain filters criteria but throwing all interesting items into one, common bag? Do you think I can use Pinned status for that? It would be fairly easy to do, just may require moving Pinned to the top of the list for better access or making it configurable (references #106)

i think that'd be fine - move pinned to the top and use it for filtered feed items
the filtering in Feedbro was/is rather granular as i recall and i don't think all that is needed - perhaps just a single, multi-line text input control where one can enter words/phrases separated by a delimiter (comma, semicolon)
then there's the question of whether you want to provide options to search the title, body, or title and body for the key word(s) - in my case i think searching only titles would be sufficient
and then a question of what results to return - so if i enter 'cat' in the filter control, do we return feed titles with 'cat', 'cats', 'caterpillar', etc., and do we perhaps have wildcard () so only the whole word 'cat' is returned unless the filter key word is 'cat' or 'cat' which returns 'cats' and 'scatter', etc.
as for case-sensitivity, i think ignore case
while a wildcard could be omitted for simplicity, and just return only exact matches, i think it would be pretty beneficial to have - i'm thinking of use cases like 'COVID-19' which may simply appear in a title as 'COVID' or 'Covid' - if no wildcard, then one would have to list all forms of the word in the filter box, or just 'covid*' with a wildcard

very off-topic, but for those interested...
[Coronavirus (COVID-19, SARS-CoV-2) news and resources, updated daily](https://12bytes.org/articles/health/covid-19-coronavirus-information-resources)

I think I'll just make it work like search does (so if title, author and if you want contents contains specified string it's in, no whole words constrains), at least for first version

Implemented as of 2.21.0

Implemented as of 2.21.0

hi zakius - thanks for implementing this and, again, thanks a ton for your work on the extension - it's come a long way since you forked it

hi zakius - thanks for implementing this and, again, thanks a ton for your work on the extension - it's come a long way since you forked it


jsonMapper is mentioned as a param but not used

https://github.com/spdx/tools-java/blob/8846a186888ea55e07dc892b276a0537d2f26a21/src/main/java/org/spdx/tools/schema/OwlToJsonSchema.java#L101
For the method toArrayPropertySchema(OntClass ontClass, int min) of https://github.com/spdx/tools-java/tree/master/src/main/java/org/spdx/tools/schema/OwlToJsonSchema.java
https://github.com/spdx/tools-java/blob/8846a186888ea55e07dc892b276a0537d2f26a21/src/main/java/org/spdx/tools/schema/OwlToJsonSchema.java#L105
jsonMapper is mentioned as a param but it is not used as a parameter in the function

Looks like the JavaDoc param should just be removed - @ek08 feel free to submit a PR

Resolved with PR #11


Reduce for loops by 2-3 bytes each in many cases

This issue was imported from Closure Compiler's previous home at http://closure-compiler.googlecode.com
The original discussion is archived at:
http://blickly.github.io/closure-compiler-issues/#992

Quoting @acleung:

It's puzzling sometimes.

I have tried on a few occasions to do something like this. It has minor improvement on unzipped size and in almost all cases hurts gzipped size.
I wish I can explain to you why.....


Incompatible with embroider

We're starting to test our Ember projects against the embroider build system at https://github.com/embroider-build/embroider, and we're currently unable to build.
Are there a known set of remedies this project needs, to support embroider builds?
The specific issue we're hitting relates to ember-component-css's pod-styles file.
Build Error (OneShot)

/var/folders/kj/m25gsmy53134qzh_6w2jmjq40000gn/T/broccoli-136603YXK5PprOZui/out-07-funnel/addon.scss:1:2: Error: File to import not found or unreadable: pod-styles.
        on line 1 of ../../../../var/folders/kj/m25gsmy53134qzh_6w2jmjq40000gn/T/broccoli-136603YXK5PprOZui/out-07-funnel/addon.scss
>> @import "pod-styles";

   ^

        at SassCompiler

The particulars of our current setup are:

pods folder structure
Ember 3.12 LTS, soon moving to Ember 3.16 LTS
a mix of PostCSS and SASS, moving to PostCSS
an Ember pods PostCSS app that depends on an Ember pods SASS addon

This issue intends to start an Embroider-specific conversation around this particular build issue, as I may have time to contribute some fixes to get this project working.

I'm seeing this in Octane with Ember 3.22 - did you get this working without embroider?

At present, our setup is working OK on a mix of Ember 3.12 and 3.16 (which has Octane stuff).
I'll update this issue if I hit the same problem without Embroider.

@BryanCrotaz @timiyay - We are using Ember 3.24.0 with Octane and it works okay. But encountered this issue when trying to build with Embroider.
BTW, we are using LESS

sorry. I haven't been working on ember for a while, and this project has gone a little derelict. There are some fundamental assumptions with the design that do not work with octane, not embroider. There is a rewrite of this under https://github.com/webark/ember-cli-styles that was going to funnel into this (this is the rewrite of #333 broken up into sub packages) This is about 85% done. there are some differences outlined in #333 . If someone wanted to hack out a temp solution using the current architecture, i'll merge it through.

@webark - does that mean that we should migrate from ember-component-css to ember-cli-styles? Is it production ready? What are the changes compared to ember-component-css? Thanks for working on that!

@boris-petrov not production ready yet probably. it's been almost a year since i have worked on it. I haven't ran it through embroider yet, but the principal of how it works is sound in theory. The big difference is that you are importing directly the namespace during build time, rather then having it be interpreted at runtime. This was the change that was needed for it to be more "future proof". If you wanted to help vet it, and iterate on some of the issues that come up, i could assist. it's much simpler in its premise, so the theory is it would be easier for people to assist. I just moved to the devops side, and the company i work for switched to react (😢) so I haven't had the time, and personal life has been filled up so haven't had time to work on it.


feat(stage-extensions): New abstraction for non-core stages and tables

Description
Progress toward

#4054

Changes made

Adds a mechanism to add new stages and tables with minimal footprint on existing crates.

When reth is started, a config can be used to switch on "non-core" tables/stages. When reth goes to make all its "core" tables/stages, an additional codepath checks for "non-core" and applies those as well.
Composed of:

New methods to the RethNodeCommandConfig trait. One for adding a custom stage, one for adding additional tables.
Modifications to the table-creation macros to permit re-use for non-core tables.

Includes the name of the tables enum and the count of its variants.
Grouping of the existing tables! methods under a TableMetadata trait so that non-core tables can be passed generically via that trait.

// examples/additional-stage/src/table.rs
tables!(NonCoreTable, 1, [(AddressAppearances, TableType::Table)]);

Discussion
Presented for conceptual feedback. In the main tracking issue, it was raised that some new abstractions are likely required to enable additional indices. This PR is an attempt at such an abstraction.
The implementation is incomplete, and the intention is to see if the idea is a good direction before proceeding more. Perhaps there are obvious flaws or preferred alternative directions?
Apologies for touching so many files: many are just involved on the way to get the number of optional new tables to the set_max_dbs(max_tables) function.

Started to play around with the RethNodeCommandConfig trait to see where it could be used. Removed the stage-extensions crate and instead created a new ./examples stage-extensions binary. Still a work in progress.

@onbjerg if you have some time - could you please give a vibecheck on the overall design here?

@onbjerg if you have some time - could you please give a vibecheck on the overall design here?

I've just updated the summary describing the design

Thanks for taking a stab at this! I think there are a few unresolved things with this approach

This approach will not allow you to add custom stages in between 2 default ones, which might be desireable in some cases. This might not be too big of a deal at first
This approach only allows you to extend historical sync - the live sync portion ("blockchain tree") will not be extended at all, meaning a custom stage that indexes e.g. address interactions will only work as long as the pipeline is used - as soon as reth is syncing the tip, nothing new would be indexed

Do you have some ideas for #2?

This approach only allows you to extend historical sync - the live sync portion ("blockchain tree") will not be extended at all, meaning a custom stage that indexes e.g. address interactions will only work as long as the pipeline is used - as soon as reth is syncing the tip, nothing new would be indexed

Ok that is interesting.
From the docs:
Then, the Pipeline::run function is called, which starts the pipeline, executing all of the stages continuously in an infinite loop. This process syncs the chain, keeping everything up to date with the chain tip.

I took this to mean that the stages restarted, looping as new blocks arrive:
## initial sync
A (genesis to tip) -> B (genesis to tip) -> C (genesis to tip) ...
## maintenance
- Block `n`: A -> B -> C ...,
- Block `n + 1`: A -> B -> C ...,
- Block `n + 2`: A -> B -> C ...,

I'll look more closely to understand better how it actually works

Do you have some ideas for point 2?

cefec0a20121ed43b3185271b61e07a09666967b
Here would be one approach I could see:

Trigger extra stages at the end of the blockchain-tree AppendableChain::validate_and_execute()
Get data from stages and represent as a user-defined struct
Tack that struct inside PostState
Call a user-defined db write function on the data when PostState calls db write.

Tack that struct inside PostState

Hit an impasse with this approach as introducing generics in that struct was infeasible.
Have played with a few other approaches. Nothing compelling identified yet

Closing this as stale, we'll re-incorporate these ideas as we venture into custom stages with Reth CLI Extensions and the Declarative Node Builder API that @mattsse is spearheading.

Appreciate you takign the tiem on these @perama-v, it's effort that we'll take into account seriously as I think this is a good idea


Review "Disable all Notifications" feature in Horreum

A recent performance regression was detected in a project and Horreum correctly identified the regression but did not raise an alert. After review, it was discovered that all notifications had been disabled for that particular test;

This feature exists to temporarily prevent large numbers of notifications from being sent when a user is uploading data to Horreum in bulk.
In order to use this feature a user needs to;

Manually disable notifications for a test
perform the bulk insertion of data
re-enable notifications for the test

This feature, as it is currently implemented, allows for 2 failure modes;

Users may forget to enable notifications after a bulk upload of data
General users may not be aware of the notification subscription mechanism within Horreum (i.e. they can un-subscribe from notifications), and may disable notifications thinking it is disabling for just them. A single user can permanently disable notifications for all users for a particular test

It might be more robust to add an additional parameter to the upload API to disable notifications for that particular upload. Removing the "All notifications are disabled" toggle from the UI would help us prevent this situation from re-occurring. In addition, the feature (of disabling notifications for a dataset) would be documented in the API

@johnaohara Should we also detect bulk uploads and force users to explicitly disable or enable notifications if e.g. more than 10 runs are uploaded to the test in less than 10 minutes?

For the record, an alternative would be to add an expiration (e.g. 24 hours), username and optional explanation when disabling the notifications (not stating any preference). After this time notifications would be automatically re-enabled.

Is it bulk uploads that are the issue, or bulk notifications?
we could;

Rate limit uploads - preventing uploads if a particular user is sending more than a specified threshold
Rate limit notifications - prevent notifications from being sent if the system detects so many per minute being sent and notify the user that a temporary ban has been put in place unless they explicitly opt-in to sending notifications

If we implement a rate limit per use basis, do we need to ensure that each test running in an automated env would need its own upload credentials instead of sharing creds across different tests

For the record, an alternative would be to add an expiration (e.g. 24 hours), username and optional explanation when disabling the notifications (not stating any preference). After this time notifications would be automatically re-enabled.

There is a risk here that if a regression occurs within the expiration time window then we will still miss a genuine notification.  This probability of this risk occurring increases if the frequency of "real" test results is equal to or less than the expiration timeout

I wouldn't limit uploads, that never has been an issue and would be rather annoyance when the threshold is hit accidentally.
I was already thinking about delaying & aggregating notifications - when there's a regression in one dataset that triggers multiple variables you get one email for every variable. We could delay notifications e.g. for 1 minute after last added event so that we accumulate these and can create a big bulk email for all of them (per test).
Do you think a weekly overview would be useful, too?

For the record, change detection notifications are aggregated now and sent for whole dataset in single email.


HoloLens 2 features not working with OpenXR

Describe the bug
We're having trouble with spatial mapping on a build for HoloLens 2 using OpenXR.
With MRTK 2.5.4, we can't get spatial mapping to work, even after cloning and modifying the MRTK profiles as shown on this page. Other features like hand tracking do work.
With MRTK 2.6, spatial mapping still doesn't work for us, and we also can't get hand tracking to work.
If we don't use OpenXR, it all works for us, but our project requires we use OpenXR.
We're not sure if this is a bug or just additional configuration needed when using OpenXR. We've tried following the latest docs.
We have sample projects that we can share.
To reproduce
Steps to reproduce the behavior:

Have an app using the XR Plugin Management system, the OpenXR Plugin and the Mixed Reality OpenXR plugin in a Unity app, with MRTK 2.5.4 and spatial mapping enabled.
Create a build with the Surface Magnetism sample scene.
Deploy to HoloLens 2.
The panels do not stick to the surfaces, and the spatial mapping mesh is not visible.

Expected behavior
The panels stick to the surfaces, and the spatial mapping mesh is visible (like it is when using WMR instead of OpenXR).
Screenshots
(None for now)
My setup

Unity 2020.2.3f1
MRTK 2.5.4 (also 2.6)
HoloLens 2 with OS version 10.0.19041.1136

Target platform

HoloLens 2

Additional context
I can send sample projects showing the problem. It all works with WMR (no OpenXR).

I opened https://github.com/microsoft/MixedRealityToolkit-Unity/issues/9491 to track the spatial mapping implementation


Code organization

I think we need to start cleaning up luigi.*. What do you think about something like
luigi.tasks – contains all built-in tasks, like the hadoop mapreduce, scalding, hive, etc
luigi.targets – all filfesystem stuff
luigi.task_history – foursquare's and mortardata's task history implementations
luigi.task_storage – future implementation of different storage engines for the scheduler
etc

I like the batteries included approach as well. Sub-modules based on functionality could help organize better. But I have also run into libraries where there were many sub-modules and it wasn't obvious where a particular class belonged.
I would suggest that if we create sub-modules, we limit it to just a few such that the destinations of a class is rather obvious.

I like the batteries included approach. Let's keep adding well-tested stuff to luigi.contrib. If we ever hit say 40-50 submodules then we can move it to other repos

Sounds good plan to me as well. Can this be closed?

ok just like 1999 more issues to close now :)


feature: allow custom canvas size

Current canvas is hardcoded with fixed size (500, 500). This PR will allow custom size to be passed.

improves type stability.

Sorry Initially I thought this was aspectratio PR and changed title without checking code and repo. My apologies.

Ok. Let me signal go for this. We can comeback for this later. But comments are still welcome.


outputsize='full' in get_daily_adjusted() traceback

Hello, when i use the optional argument outputsize='full' in the get_daily_adjusted() function i get:
TypeError: init() got an unexpected keyword argument 'outputsize'
code snipped I use:
import alpha_vantage
from alpha_vantage.timeseries import TimeSeries
ts = TimeSeries(key='my API key', outputsize='full')
data = ts.get_daily_adjusted('VOO')

what am I doing wrong?
another question: Is there a way to retrieve a defined timeframe of data (like for example the last year)?

Hi there Steffen,
The problem is that the outputsize parameter hast to be used in the api call, not in the class initialization.
import alpha_vantage
from alpha_vantage.timeseries import TimeSeries
ts = TimeSeries(key='YOUR_API_KEY')
data = ts.get_daily_adjusted('VOO', outputsize='full')

with regard to your second question, I would suggest using the pandas format supported by the module, pandas is quite popular in python and finance and allows you to slice your data nicely.
Best Regards
Romel Torres


Better to have code formatting

Actually, there was code formatting a couple of versions ago, but I removed it because it didn't work good enough. I'll surely get back to it.

Just update to v0.5.0, and don't forget to switch on the "Text Editor / Formatting / Format On Type" setting.

The on-type formatting feature was implemented, so I'm closing this issue, though I've also opened #99 about the code blocks formatting.


adding flag to disable service worker

fixes #371

@developit  @lukeed it worked for me bothways with and without camel casing

btw made suggested changes, is it ready to merge yet?


[BUG] .json changes not working properly

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.
[X] I have thoroughly looked through the available Wiki articles and could not find a solution to my problem.
[X] I am using the latest version of Bloxstrap.
[ ] I did not answer truthfully to all the above checkboxes.

What problem did you encounter?
I managed to override the Montserrat Extra Bold weight using the .json file in the bloxstrap files, but there's a problem, the thickest the font would go to is the Bold weight, it's not calling the black weight nor is the proper weight being applied.

As you can see, despite file modifications and the fonts corresponding in the code, the font weights are still messed up.
Bloxstrap Log
N/A

how is this our problem


Use recommended 'inputs' context in github workflows

Use recommended 'inputs' context in github workflows to fix the if condition check.

/skip-all


Please help, I want to pass parameters to superset， how can i do

##I need to modify the data value of  the ”Upload a CSV“,such as
df_empty = g.lytmp
            database.db_engine_spec.df_to_sql(
                database,
                csv_table,
                df_empty,
                to_sql_kwargs={
                    "chunksize": 1000,
                    "if_exists": form.if_exists.data,
                    "index": form.index.data,
                    "index_label": form.index_label.data,
                },
            )

[drag & drop image(s) here!]
and  I use the global variable g to save the incoming value， such as
def create_app(data) -> Flask:
    app = SupersetApp(__name__)
    with app.app_context():
        g.lytmp = data
    try:
        # Allow user to override our config completely
        config_module = os.environ.get("SUPERSET_CONFIG", "superset.config")
        app.config.from_object(config_module)

        app_initializer = app.config.get("APP_INITIALIZER", SupersetAppInitializer)(app)
        app_initializer.init_app()

        return app

    # Make sure that bootstrap errors ALWAYS get logged
    except Exception as ex:
        logger.exception("Failed to create app")
        raise ex

[And when i run it, its wrong, Error message: '_AppCtxGlobals' object has no attribute 'g.lytmp', what should i do ,please help me]
Design input
[describe any input/collaboration you'd like from designers, and
tag accordingly. For design review, add the
label design:review. If this includes a design proposal,
include the label design:suggest]

Closing this as stale since it's been silent for so long, and we're trying to steer toward a more actionable Issues backlog. If people are still encountering this in current versions (currently 3.x) please open a new Issue or a PR to address the problem.


add css for textarea mapfile.content

related #3198

As discussed in issue this change will not word-wrap anymore long lines when editing map-files

I've tested this and applied it to all textarea elements, works well, so let's finally merge it.

Merged, thanks!


Migrate to Jakarta EE 9

Migrate to Jakarta EE 9 with the current available stabile wildfly Release.

Switched to jakarta ee 9 and java 17


add intro to Foundry

介绍了 Foundry 的使用

Merge it to task 71 would be better I think. rename task 71 as Foundry.


Stages and statuses

Improve the stages and statuses that applications can be assigned upon their journey from 'registration' to 'handover'.
See this google doc for more information
To do
We will approach this work in three parts
1. Stages and statuses

[x] Update the existing stages and statuses screens to support the new stages and statuses
[x] Centralise the logic
[x] Add custom page size
[ ] Update reports which include stage & status - e.g. diversity report

2. Staged applications / registration
We recently enabled exercises to have 'staged applications' where partial information could be requested up-front and at each stage of processing. This work needs iterating on to ensure initial 'registration' applications are not treated as having 'applied'

[ ] Change the status when an initial application is made to REGISTERED instead of APPLIED (only when exercise configured as staged applications)

[ ] Enable registrations to take part in QTs and have an associated status/outcome recorded
[ ] Update reports which include 'Applied' so they correctly reflect 'Registered' as well as 'Applied'

3. Back-fill data / support current and new stages & statuses
We will look to update all past exercises and applications to have the 'new' stages and statuses. If this is not possible or too scary (!) then we will look to support current and new stages & statuses (via the exercise '_versionNumber' field)

[ ] Assess the scope/scale of back-filling data
Then either:
[ ] Back-fill data
or:
[ ] Introduce a new exercise version number 3 so we can support current and new stages & statuses

@warrensearle  - What is the update with this work? Why is it blocked?


how i can user

there is no documentation

README has been added back. If you have any questions, please feel free to ask in Stack Overflow.


quick fix

Avoid error when config.adminIndex is not equal to 'device-manager'

The package-lock.json file should be updated also

done


Ambiguous description of pt2ts.py

The description of pt2ts.py in the utils README could be interpreted that the model will be saved in the same directory as the pt2ts.py file, but it is instead saved in the directory that pt2ts.py is called from.
Two potential solutions are:

Update the README to make the distinction clear
Update pt2ts.py (in quite a few places, including the benchmark repo) to get its location, and prepend this to the save path, and clarify this in the README

I would lean towards the latter, so the location is more consistent.
For example, pt2ts.py might be called from the model directory, a build directory, or the directory up containing (such as from a script like run_benchmarks.sh), each of which currently would save the model in a different location.

Reading it again, perhaps the description is clear enough at the moment, so the question is maybe more whether (by default) pinning the location to the same directory as pt2ts.py would be preferable?

I agree, if running the script from in build/ for some reason to regenerate the model we should expect it to be output in the same place.
We can probably do this with a relatively small change (and hidden from the user) using the path package.
IIRC there is a command (or maybe two) to get the parent of the script that is currently running.

I'm revisiting this, and under the "principle of least surprise" I think the file appearing wherever you are locally calling pt2ts.py from is actually the more logical option.
Any thoughts @ElliottKasoar?
I'm trying to clear out some simple issues so can update the docs to be clearer on this if you think that is OK?

In retrospect, yes I'd say the user's current working directory is probably the most logical place, so I'm happy with the current functionality remaining the same.
Ideally it might be nice to have the output filename as an optional argument, which the comments suggest may already be possible, but I think sys.argv is only used at the end for the check that the model file was created, which is confusing.
I might be missing something though, and if not, it's probably worth a separate issue.

Ah, this is a good spot @ElliottKasoar Thanks.
Looks like @jwallwork23 added the option to pass a filename in, but as you say only used it at the end.
In this case I agree that #171 should use that input everywhere for the filename if provided (with the default being as it currently operates).
I'll make those updates when I can and tag you again for a review if that's OK?


Generic style rule is breaking automatic updating of iframe height

If I have this style in head:
iframe {
  height: 100%; /* or any other value */
}

Then the resizing of the iframe does not work, since the rule always overrides the height attribute. I suggest this plugin is also/instead updating iframe.style.height, which is harder to override accidentally.

We're not likely to make this change right now. If you need to style iframes this way, it may make sense to update your selector to exclude it or add a secondary rule that will override it for Pym embeds, such as [data-pym-loader] iframe { height: auto }.

Ok, it makes sense. It was from the consideration of using Pym in a 3rd party solution, which embeds an iframe into an existing website. And because we can't control exactly what CSS rules people write, the 3rd party code is quite vulnerable to such clashes. We managed to work around it though.


Add ERC20 Name to Cadence Fungible token on User's storage iterator script

Instructions
Issue To Be Solved
Currently this convenience script to iterate over the user's storage gives all fungible tokens. The tokens bridged from evm have a long hard to understand name.
https://github.com/onflow/flow-evm-bridge/blob/a50fdd62ca272153e8a5f83683ecaee7c08225e8/cadence/scripts/tokens/get_all_vault_info_from_storage.cdc#L58
(Optional): Suggest A Solution
Would it be possible to add "ERC20" name?
Flow Port displays a contract name which the bridge creates, makes sense to display the evm ERC20 name.

closing, looks like the erc-20 name is available.


Update index.js

Lines 39, 43, 47: added a : symbol for slightly more understandable logs.

Hi, please don’t spam with poor PR’s


Bugfix default values to timedelta

Description:
We support now timedelta for SCAN_INTERVAL this PR set all values to timedelta from int.

🐬  !

Thanks! Sorry I missed that.


wrong expression result

It gives wrong expression results for trigonometry operations. for example the value of  tan(45) is 1 but it gives 1.6 something. please check.
Thanks.

By default, trigonometric functions are interpreting their parameters to be in radians, not degrees.
If you want the parameter to be interpreted as degrees, there are a couple of ways you can do this:

Add the degree sign to the parameter: tan(45°)

Wrap the parameter in the dtor function (which is what the ° operator does, essentially): tan(dtor(45))

Alter the angleMeasurementMode of your DDMathEvaluator. This will change the interpretation of parameters for all future evaluations using that evaluator object.
 [DDMathEvaluator sharedMathEvaluator].angleMeasurementMode = DDAngleMeasurementModeDegrees;
 NSLog(@"%@", [@"tan(45)" numberByEvaluatingString]);

ok thanks... :)

hello,
Is it possible to get correct result without brackets..
for example: I want to find the value of  tan(45°) , can I get result by calling tan45° ?

No, not without significant alterations to the fundamental workings of the math parser.


GUACAMOLE-1418: Add support of SQL Server JDBC plugin in Docker Image

Implement SQL Server support in Docker Image.
According to JIRA issue, need to check if it's legally allowed (saw other ASF software are using it so I guess it could be)
Have a good review :)

saw other ASF software are using it so I guess it could be

Can you provide these examples?

Here for example: https://github.com/apache/skywalking-java/blob/e6447d517558031f49cc4450467e41b644320071/apm-sniffer/apm-sdk-plugin/mssql-jdbc-plugin/pom.xml (pom dependency)

@jbpaux In the case you mentioned, the pom.xml file is actually not pulling in the download - it lists it as a dependency, but the <scope>provided</scope> tag on that dependency means that Maven will proceed building without actually pulling in those files, and will rely on them being provided by the JDK/JRE at runtime. See the provided description in the following section:
https://maven.apache.org/guides/introduction/introduction-to-dependency-mechanism.html#Dependency_Scope

Ok, no pb, here then : https://github.com/apache/juddi/blob/5988d077548d54da86c982295e48df3e771ae53e/juddi-tomcat/pom.xml#L88 no provided I think (I'm not java expert so maybe I guess wrong 😄 )

The JDBC driver provided by Microsoft for SQL Server is MIT-licensed and should be fine to bundle in the Docker image: https://github.com/microsoft/mssql-jdbc

Any feedback about this PR @mike-jumper @necouchman ? 😅

Hi, do you plan to merge it or it requires some updates  @necouchman  ?


test php 8.x support

rolling 7.4 currently on LTS systems / ddev

Deprecated:  Return type of DirFilter::getChildren() should either be compatible with RecursiveFilterIterator::getChildren(): ?RecursiveFilterIterator, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in /var/www/html/system/backend/php/lib/HAXCMS.php on line 1708

Deprecated:  Return type of DirFilter::accept() should either be compatible with FilterIterator::accept(): bool, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in /var/www/html/system/backend/php/lib/HAXCMS.php on line 1704

https://github.com/php/php-src/blob/e4e2541c1a90daf708ab1ee4d44f2ab2883ce84b/UPGRADING#L515-L520
this is missing so sitemap fails hard

https://markbakeruk.net/2020/11/17/under-the-radar-a-backwards-compatible-break-for-splfixedarray-in-php-8/ has clues as well

https://spencermortensen.com/articles/php-error-handling/


FluentBootstrap RadioFor() does not check the radio button if the field value is equal to the radio value, contrast to Razor RadioButtonFor() behavior.

The controller

The view

The result

Also, I would like to give you an applause for @Html.Bootstrap().SelectFor() accepting options passed as IEnumerable of KeyValuePair <string,string>. Do you think you can do the same for Radio? (Accepts IEnumerable of string and/or KeyValuePair<string,string> and generates the radio list appropriately)
Thank you in advance and keep up the good work!

Thanks - another very good catch. It looks like neither the radio or select are picking up the selected state. It'll probably be next week at the earliest that I get a chance to dig into this, but I'll hopefully be able to get this and the similar select issue as well as your previous one sorted out soon. Once they're all wrapped up I'll push a new release to NuGet.
Keep them coming!

This is now resolved in the develop branch. I've also captured the request for better extensions for making sequences of form elements as issue #34


Add globalPrefix before performing sourceRegex match

This would allow me to decide if I want the global prefix in my source or my metric name. Before the only option is to prefix the metric name.
Side note: might be nice to trim leading and trailing periods from both.

@bmarini Looks good to me, merged!


feat: flag for supporting core-js in babel

Types of changes

[ ] Bug fix (a non-breaking change which fixes an issue)
[x] New feature (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description

With current change, user can user functional presets to set core-js version:
yarn add --dev core-js@3 @babel/runtime-corejs2 --save

export default {
  build: {
    babel: {
      presets({ isServer }) {
        const targets = isServer ? { node: '10' } : { ie: '11' }
        return [
          [
            require.resolve('@nuxt/babel-preset-app'),
            {
              targets,
              corejs: { version: 3 }
            }
          ]
        ]
      }
    }
  }
}

But it may be not very simple, so we can discuss to see if you have any idea @nuxt/core-team .
I have a rough idea which may be not very elegant as well, current build.babel may be kind of complex and unhelpful, but for not breaking anything, how do you think add build.babel.nuxtPreset or sth else:
export default {
  build: {
    babel: {
      nuxtPreset: {
        corejs: { version: 2 },
        // or any option of @nuxt/babel-preset-app
        // or client/server specific
        serverTargets: {},
        clientTargets: {}
      }
    }
  }
}

@clarkdo What if we extract nuxtPreset option to something like build.preset and leaving build.babel to directly being passed to babel-loader?

@clarkdo What if we extract nuxtPreset option to something like build.preset and leaving build.babel to directly being passed to babel-loader?

Yes, it's also an idea what I'm thinking.

Codecov Report

Merging #5411 into dev will decrease coverage by 0.03%.
The diff coverage is 87.5%.

@@            Coverage Diff             @@
##              dev    #5411      +/-   ##
==========================================
- Coverage   96.01%   95.97%   -0.04%
==========================================
  Files          74       74
  Lines        2532     2536       +4
  Branches      641      643       +2
==========================================
+ Hits         2431     2434       +3
- Misses         85       86       +1
  Partials       16       16

Impacted Files
Coverage Δ

packages/babel-preset-app/src/index.js
96.77% <87.5%> (-3.23%)
:arrow_down:

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 3b85dd9...a55c541. Read the comment docs.

@clarkdo As default is 2, should not we add @babel/runtime-corejs2 to @nuxt/babel-preset-app dependencies?

@clarkdo As default is 2, should not we add @babel/runtime-corejs2 to @nuxt/babel-preset-app dependencies?

Currently default config is @babel/runtime since useBuiltIns is always usage

But docs says:
yarn add --dev core-js@3 @babel/runtime-corejs3
 # or
 yarn add --dev core-js@2 @babel/runtime-corejs2

Should we also change to something like this?
yarn add --dev core-js@3 @core-js-compat
 # or
 yarn add --dev core-js@2 @babel/preset-env

core-js-compat is directly dependent by @babel/preset-env, so I think we don't need to let user install it.
For @babel/runtime-corejs3 and @babel/runtime-corejs3, it's for the users who are not using useBuiltIns: 'usage'


Custom auth with tests

Description of changes:
Adds authenticationFlowType key to the awsconfiguration file for ios and android only.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

I have verified that :

awsconfiguration.json is updated with authenticationFlowType as CUSTOM_AUTH if user enables custom auth and configures the required lambda triggers in the cli flow by running amplify add auth
awsconfiguration.json is updated with authenticationFlowType as USER_SRP_AUTH if user disables custom auth and consequently removes lambda triggers by running amplify update auth


Get-Team update (0.9.5) doesn't return correct Teams

In the new release (0.9.5), we have noticed that Get-Team returns teams that a user is not a member or owner of.
I assumed this might be because the command now runs as an administrator showing all Teams. However, random teams are still missing from the list (even if we are a member or owner of those Teams).
Downgrading to version 0.9.3 fixes the issue.

@kenwith is this anything you can advise on?

@CloudDesignBox  Thank you for this feedback
We apricate you pointing out this potential bug. Let me get this information over to the PowerShell team for investigation.
The team will get this prioritized into a sprint plan. Thank you for reporting and making the docs better. Much appreciated.
I made a note to request the team to update this when the work is complete.

@AndreaBarr Thank you!

@islubin - FYI on potential bug in latest module.
@CloudDesignBox - If you wouldn't mind please open a service ticket in your tenant. They can group these together with other service tickets so that @islubin team can get this figured out.
@AndreaBarr - do you have that link on how to open a service ticket handy?

@kenwith I've added this as a service ticket in the tenant. We have noticed this in a couple of our customer tenancies so far but i think it is related to the PowerShell module not the tenant. Happy to provide further information via email if required.

@CloudDesignBox  I will wait for the support ticket to come through! The ability to get all teams in an organization didn't exist in the 0.9.3 release, so when you say that it works in 0.9.3 what functionality are you referring to?

@CloudDesignBox and @kenwith Here is the link I have for opening a ticket.
Here is a quick video on opening a ticket: https://www.youtube.com/watch?v=puuOy0sEg1Y

@AndreaBarr Thanks I logged this yesterday but not had a response yet.

@CloudDesignBox  Okay great. I am going to close this issue for now. Please let us know what the resolution is when you get it.

This has now been replicated to MS support. I'll update you when i know more....

Latest update from MS Support:
It seems to be an issue with lot of tenants as I am also able to replicate this on my end with the same behavior.
We have raised this with the concerned teams through internal ticket and waiting to hear back from them.
I will update you soon on the same.

It looks like this issue has been resolved. I am closing this issue at this time. Please re-open this issue if it is not resolved.

This has not been resolved and is now happening in version 1.9.6. I'm still in talks with Microsoft to resolve this.

Hi @CloudDesignBox , thank you for your time and feedback on this.
Do you have an update from support? Thanks!

@dariomws306 They have recently got back to me but i am no longer using PowerShell - we are now using the Graph API which responds as expected.
I'm still keen to get to the bottom of it but we no longer have access to the tenancies to replicate this for the Microsoft Team. Are you experiencing the same issue?

@CloudDesignBox thank you!
The way to manage this kind of issues is to open a support ticket, and you did it.
I have tried from my side and I can get all the teams running Get-Team alone, but I have tried with -User and I get some Teams that the user is not member of.
I think product group is still working on this cmdlet because you can see this in the article:
"Note: This cmdlet is currently in Beta."
"Note: we are still backfilling existing teams to get returned by this command."

I'm happy to close this as we are no longer using the PowerShell commandlets. However it would be useful to clearly state that there will be different behaviour with different releases of Teams PowerShell.


Add 'reboot_device' and 'get_uplinks_loss_and latency'

Hi, I work in support. I had been working on a personal project to reboot devices and figured I'd add this to the Meraki repo.

Also added a function for get_uplinks_loss_and_latency (https://dashboard.meraki.com/api_docs#return-the-uplink-loss-and-latency-for-every-mx-in-the-organization-from-at-latest-2-minutes-ago)