Decouple attribute alias from type definition

This PR is created because of this thread in the forums:
https://discourse.rom-rb.org/t/ability-to-alias-an-attribute-but-keeping-type-inferred/289
Basically, it decouples an attribute alias from its type, so it will be
possible to define an alias but still make use of schema inference.
This is still a WIP, but I'm looking for some feedback in order not to do extra work.
Some considerations I'd like to have your thoughts:

The DSL is still not adapted. I haven't tested it, but I guess it won't work
defining an attribute without specifying its type, because the main purpose
of this PR is to be able to do a simple:
attribute :foo, alias: :bar

The option to define an alias is called alias:. This is because as: can't
be freely used because it is already a method name and dry-initializer
creates methods from option names. We could use method renaming feature in
dry-initializer for that, but then we have to juggle with the renaming in
the rest of code, like in aliased and meta methods, and it is a bit ugly.
The handicap is that in other places of the API as: is used for similar
purposes, like in association aliasing.

alias method has been removed because it is defined out of the box by
dry-initializer.

The attribute ast will now always return alias: nil when an alias is not
defined. This is different from before, where alias key was not present in
those situations. In my opinion, it makes more sense to always return all the keys.

The test for returning aliased attribute identified by canonincal name has
been removed, because I think that, being now an option and not part of the
type, it makes no sense.

I have redefined ROM::SQL::Function#name method in the repository
integration tests. This should be changed in rom-sql project. How would yo
go with it?

I guess that it would also make sense to move wrapped from the meta to an
option. Would you like to change it also in this PR?

What do you think about primary_key meta? Should we consider it part of the
type (meta) or also an option? Even if it related with the value (the value
being unique), it is so only considering the rest of values. It is like
dynamic typing. Anyway, it would be more convenient to have it as option, I
guess.

Thanks!

The test for returning aliased attribute identified by canonincal name has
been removed, because I think that, being now an option and not part of the
type, it makes no sense.

Could you elaborate a bit more here?

The test for returning aliased attribute identified by canonincal name has
been removed, because I think that, being now an option and not part of the
type, it makes no sense.

Could you elaborate a bit more here?

Well, I guess this test existed in order to be sure than aliasing an attribute didn't change the name in the meta of the type. But now they are two different concepts. One thing is the type of an attribute and another thing are their options (like alias:). So, I think it makes no sense to test that the change in the type doesn't affect the options... why should it? I think that keeping the test just reflects the development history (the fact that the alias was part of the type before). Furthermore, rename seems very well tested in rename_spec file.
But maybe I'm missing something :slightly_smiling_face:

Well, I guess this test existed in order to be sure than aliasing an attribute didn't change the name in the meta of the type.

This test exists because it checks we can still access attributes using canonical names, and rename_spec doesn't cover this scenario, so it seems like we should not remove the test.

Makes sense. I readded the test, fixed rubocop issues and forced the push rebasing from master.
There is still more to review, however. Besides the points I mentioned in the first comment, we should also decide about the #inspect output for Attribute now that it has more options.

Besides the points I mentioned in the first comment, we should also decide about the #inspect output for Attribute now that it has more options.

We could do:
def inspect
  %(#<#{self.class}[#{type.name}] #{meta.merge(options).map { |k, v| "#{k}=#{v.inspect}" }.join(' ')}>)
end

I added a commit for each change. We can squash them later if needed.

Besides the points I mentioned in the first comment, we should also decide about the #inspect output for Attribute now that it has more options.

We could do:
def inspect
  %(#<#{self.class}[#{type.name}] #{meta.merge(options).map { |k, v| "#{k}=#{v.inspect}" }.join(' ')}>)
end

As with attribute ast, inspect will always include alias: key even when the value is nil. Personally I prefer being explicit in these situations, but, anyway, I guess we must be consistent with the behaviour for meta keys (if there is still any at the end of the work in this PR).

Please open a PR in rom-sql which changes this, preferably with tests. You could temporary target this branch in the rom-sql's Gemfile, once it's merged we can go back to depending on master.

Before I needed to push here the update to the DSL schema. I added some comments to the code to make it clear some intentions.

I'm stuck with the last change needed in order to have the new feature available.
We need the following to work:
schema(infer: true) do
  attribute :first_name, alias: :name
end

Attribute inferrer checks whether an attribute has been already defined by the user comparing names. However, :name is an option in meta, which is something related with the type, and with this new DSL feature we would be defining the attribute options but not the type. Looks like we should also move :name to be an attribute option?

Looks like we should also move :name to be an attribute option?

Yes, this makes perfect sense. My understanding is that first and foremost we're moving attribute-related options to the new options hash, instead of type meta. So :name should be moved there too.

https://github.com/rom-rb/rom/blob/a9a9f2d1676e2a48a1ec94c8b85e97c9c162793d/core/lib/rom/schema.rb#L140-L148
Do you think that backward compatibility issues with plugins could arise? Changing :name to be an option would make the attribute information extracted from the DSL to be always a Hash.

Changing :name to be an option would make the attribute information extracted from the DSL to be always a Hash.

Not sure if I follow, wdym by the attribute information extracted from the DSL?

@waiting-for-dev thanks again for working on this. I rebased it and cleaned up commits. I'll merge once CI is green.

Hey @solnic I was going to tackle CI offenses rigjt now, but it seems you just merged it at the same exact time?

@waiting-for-dev feel free to push such fixes directly to master

Ok @solnic , thanks for your feedback and your support with this PR.


THis is the issue i m unable to install angular cli

Bug Report or Feature Request (mark with an x)
- [ ] bug report -> please search issues before submitting
- [ ] feature request

Command (mark with an x)
- [ ] new
- [ ] build
- [ ] serve
- [ ] test
- [ ] e2e
- [ ] generate
- [ ] add
- [ ] update
- [ ] lint
- [ ] xi18n
- [ ] run
- [ ] config
- [ ] help
- [ ] version
- [ ] doc

Versions

Repro steps

The log given by the failure

Desired functionality

Mention any other details that might be useful

kindely get solution to this error
i installed node and then installing angular
npm install -g @angular/cli
after that if it is installed correclty or not checking purpose i m using ng -v command it is unable to display the angular cli  picture

Duplicate of #12861


Fix scary warnings on mac and windows (pay the people for code signing certs)

Feature details

Currently, when launching NiceNode, users are warned against running it and have to make a few extra steps.
Code Signing requires $99 for macOS and ~few hundred $ for Windows.
See electron-build's code-signing docs: https://www.electron.build/code-signing.html

Windows is done. However, the warning will continue for some amount of time or until a certain number of installs as detailed here: https://stackoverflow.com/a/66582477


Suggestion 1: Change wording of error message.

Change wording in CLI argument error - "'maxPasswordLen' must be greater than 'minPasswordLen'" (to: must be equal or greater than);

fixed in 0.9.1


🚀 Feature: Rename to description-to-co-authors

Bug Report Checklist

[X] I have tried restarting my IDE and the issue persists.
[X] I have pulled the latest main branch of the repository.
[X] I have searched for related issues and found none that matched my issue.

Overview
Coming over from https://github.com/JoshuaKGoldberg/all-contributors-for-repository/issues/252: I'd like to use this package more generally than just on commits. It'll support issue bodies & the like too.
Additional Info
💖

Fixed by 0817424.


Fixes known vulnerability CVE-2017-18077

Known vulnerability found CVE-2017-18077 (Moderate severity)
index.js in brace-expansion before 1.1.7 is vulnerable to Regular Expression Denial of Service (ReDoS) attacks, as demonstrated by an expand argument containing many comma characters.
update suggested:
brace-expansion ~> 1.1.7

currently a transitive dependency of:
<EMAIL_ADDRESS>~><EMAIL_ADDRESS>~><EMAIL_ADDRESS>~><EMAIL_ADDRESS>

I'm totally aware of prior deprecation notice as of #809 though this might deserve some attention @gotwarlost @davglass
cheers.

Coverage remained the same at 97.523% when pulling ccbb6196de3a7160acc2f76a6f5be4aa9a04104f on ivoputzer:patch-1 into bc84c315271a5dd4d39bcefc5925cfb61a3d174a on gotwarlost:master.


esp - No rule to make target person_detect_model_data.cc

@tensorflow/micro
System information

Host OS Platform and Distribution (e.g., Linux Ubuntu 16.04):

macOS Catalina - 10.15.5 (19F101)

TensorFlow installed from (source or binary):

source cloned from git - have tried r2.2, r2.3 and master

Tensorflow version (commit SHA if source):

f295633406569f9a6ee71467a9bb34ef1cc6852b

Target platform (e.g. Arm Mbed OS, Arduino Nano 33 etc.):

ESP32
Describe the problem
Trying to generate all projects fails
Please provide the exact sequence of commands/steps when you ran into the problem
$make -f tensorflow/lite/micro/tools/make/Makefile TARGET=esp generate_projects
tensorflow/lite/micro/tools/make/Makefile:305: warning: overriding recipe for target 'tensorflow/lite/micro/tools/make/downloads/ruy'
tensorflow/lite/micro/tools/make/Makefile:305: warning: ignoring old recipe for target 'tensorflow/lite/micro/tools/make/downloads/ruy'
tensorflow/lite/micro/tools/make/Makefile:305: warning: overriding recipe for target 'tensorflow/lite/micro/tools/make/downloads/person_model_grayscale'
tensorflow/lite/micro/tools/make/Makefile:305: warning: ignoring old recipe for target 'tensorflow/lite/micro/tools/make/downloads/person_model_grayscale'
tensorflow/lite/micro/tools/make/Makefile:305: warning: overriding recipe for target 'tensorflow/lite/micro/tools/make/downloads/person_model_int8'
tensorflow/lite/micro/tools/make/Makefile:305: warning: ignoring old recipe for target 'tensorflow/lite/micro/tools/make/downloads/person_model_int8'
make: *** No rule to make target 'tensorflow/lite/micro/tools/make/gen/esp_xtensa-esp32/prj/person_detection/make/tensorflow/lite/micro/examples/person_detection/person_model_grayscale/person_detect_model_data.cc', needed by 'generate_person_detection_make_project'.  Stop.

ok - I think this is just me not understanding how this should work - running:
make -f tensorflow/lite/micro/tools/make/Makefile TARGET=esp  generate_person_detection_esp_project

Seems to work - am I right that each project needs to be generated individually?

That's correct. See example usage below;
https://github.com/tensorflow/tensorflow/blob/a4725be4a74c9acae623231e80d5a8002a54e94a/tensorflow/lite/micro/tools/make/Makefile#L10
Thanks!


MODE-2570 Trims the modeshape binary distribution by removing the demos, test and test-source artifacts.

It also updates the JavaDoc plugin number and configuration to run with JDK 8.

Looks good.


steps for install and running FALCON_unzip

Hello everyone:
I wonder could somebody please write a short summary of steps for installing and running FALCON. Many thanks.

solved.
I'd be glad to help if somebody needs such steps.

Hi, could you please sent the steps to me? I have some problem running Falcon_unzip. My e-mail is<EMAIL_ADDRESS>Thanks a lot.

Hi, wsuplantpathology
Could you please send the steps for installing and running Falcon_unzip to me as well.
My email is<EMAIL_ADDRESS>

Hi @chad388 @yilunhuangyue :
I realize the install steps may be different on different OS platforms, here is general steps:

make sure you have all required executables installed, following https://github.com/PacificBiosciences/FALCON_unzip/wiki
I use this install pipline
http://pb-falcon.readthedocs.io/en/latest/quick_start.html

The following is my specific steps in my system:
export LD_LIBRARY_PATH=/your/library/path/ # e.g.: blasr
export PERL5LIB=/your/perl/lib # e.g.: some perl libraries may be need
cd FALCON-integrate/
git checkout master
make init
source env.sh
make config-edit-user
make -j all
export PYTHONUSERBASE=/your/path/to/FALCON-integrate/fc_env
cd FALCON_unzip/
python setup.py install --prefix=/your/path/to/FALCON-integrate/fc_env
export PYTHONUSERBASE=/your/path/to/FALCON-integrate/fc_env
export PATH=$PATH:{PYTHONUSERBASE}/bin:/your/path/to/graphviz-2.40.1/bin:/your/path/t0/SDL2-2.0.5/bin:/your/path/to/samtools-0.1.19:/your/path/to/pitchfork/bin:/data/chen/software/GenomicConsensus-smrtanalysis-3.0.2/bin:$PATH # you need install all these tools first
bash --init-file /data/chen/software/pitchfork/setup-env.sh # I don't know why, but with this step 'blasr' will run successfully
fc_unzip.py fc_unzip.cfg

One important tip:
!!! change
!!! /data/chen/software/FALCON_unzip/falcon_unzip/unzip.py Line 90
Original: {samtools} sort tmp_aln.bam -o {ctg_id}_sorted.bam
After: {samtools} sort tmp_aln.bam -o {ctg_id}_sorted.bam > {ctg_id}_sorted.bam
!!! same change /data/chen/software/FALCON_unzip/build/lib/falcon_unzip/unzip.py
##again, I don't know why, without these changes my run failed in blasr step, but after I changed this, blasr step was succeed.

Good luck

Original: {samtools} sort tmp_aln.bam -o {ctg_id}_sorted.bam
After: {samtools} sort tmp_aln.bam -o {ctg_id}_sorted.bam > {ctg_id}_sorted.bam

That's really weird. -o means "dump to that file". What version of samtools are you using?

Thank you wsuplantpathology for the details.
This is very helpful!

@chad388 or @wsuplantpathology, What version of samtools are you using?

Hi @pb-cdunn @chad388
I was using samtools 0.1.19. Probably it's the issue. I didn't check it with new version (e.g.: 1.3.1).


Add new admin reports

Overview
Adds new administrative reports for:

Percent of facilities claimed (monthly)
Duplicates merged per month
Facilities marked as closed per month
Names and addresses promoted per month

Some reports have proven difficult or inefficient to create using pure SQL, so the names and addresses promoted per month report introduces a new method of running reports which uses a custom Python function on a per-report basis.
Connects #1346
Demo

Testing Instructions

Run ./scripts/server
Claim a facility
Merge two facilities
Mark a facility as closed
Promote a facility address
Confirm that each report's logic appears sound
Confirm that the facility actions you have taken show in the reports
Confirm that the downloaded CSV files are correct

Checklist

[x] DROP any commits that change Jenkinsfile
[x] fixup! commits have been squashed
[x] CI passes after rebase
[x] CHANGELOG.md updated with summary of features or fixes, following Keep a Changelog guidelines

@jwalgran It seems like the regular SQL reports are working well on staging, but the non-SQL report is returning a 504 error. Do you have any thoughts on how to proceed?

I reverted the "Test on staging" commit to prevent deployments while I test a release.

It seems like the regular SQL reports are working well on staging, but the non-SQL report is returning a 504 error. Do you have any thoughts on how to proceed?

As an optimization, we could experiment with running a single values_list query against the FacilityHistory table that sorts by facility ID, then history_date. This should avoid multiple trips to the database and avoids instantiating model instances for each row returned. If that query is fast enough and we can iterate over it fast enough, we may be able to produce a result before the request times out.

@jwalgran I've updated the query using values_list as you suggested, with significant time improvements (in development at least). Running it previously locally took ~2-4 seconds, and now takes ~0.00-0.02 seconds, which is in line with our other reports.

Jenkinsfile commits have been dropped and the sort order has been updated. Thanks for all of your work and troubleshooting of the new Python report with me!


hello word

What does the program do ?

console.log with styling
In what programming language it is written?
javascript

Thank you for participating  in hacktoberfest ❣ if you liked working on this repo you can share it with friends and star this project for helping others


[Monorepo Builder] Avoid duplicating entries in composer.json's repositories

As requested in https://github.com/symplify/symplify/pull/2712#discussion_r549346977.
Currently, if composer.json already has an entry for the local package packages/package-one, like this:
{
    "name": "example/package-three",
    "repositories": [
        {
            "type": "path",
            "url": "../../packages/package-one",
            "options": {
                "symlink": true
            }
        }
    ]
}

after running monorepo-builder localize-composer-paths it would duplicate the entry, like this:
{
    "name": "example/package-three",
    "repositories": [
        {
            "type": "path",
            "url": "../../packages/package-one",
            "options": {
                "symlink": true
            }
        },
        {
            "type": "path",
            "url": "../../packages/package-one",
            "options": {
                "symlink": false
            }
        }
    ]
}

In this PR, it checks if the entry exists, and if so it already uses it:
{
    "name": "example/package-three",
    "repositories": [
        {
            "type": "path",
            "url": "../../packages/package-one",
            "options": {
                "symlink": false
            }
        }
    ]
}

Thank you :+1:


Exchange Online, on or off?

You should clarify if it requires the customer to migrate local Exchange to Exchange online for it to work flawlessly, and what effect it has when a customer have an on-premise Exchange.
The text here is not true, as Exchange is assigned automatically when the end user is assigned a Teams Exploratory license. This leads to confusion, af the new mailbox in Office 365 will not automatically work.
The Teams Exploratory experience comes with an Exchange Online license but it won't be assigned to the user until the admin assigns it. If the user doesn't have an Exchange license already and the admin has yet to assign the Exchange Online license, the user won't be able to schedule meetings in Teams and may be missing other Teams functionality.

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

ID: f897ae9b-852a-b617-7de0-d76dbd9f1820
Version Independent ID: 161b1845-53fd-8912-a5f9-737d3bcdaadc
Content: Manage the Microsoft Teams Exploratory experience - Microsoft Teams
Content Source: Teams/teams-exploratory.md
Service: msteams
GitHub Login: @LolaJacobsen
Microsoft Alias: lolaj

@flcev Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

Hi @flcev, thank you for your feedback.
About Exchange on-premises, move/migration recommendations are the same:
https://docs.microsoft.com/exchange/exchange-hybrid
If you want a full Teams service for this user, a mailbox is required. You can move the mailbox to EXO or if you don't:
https://docs.microsoft.com/en-us/microsoftteams/exchange-teams-interact
About:
"The Teams Exploratory experience comes with an Exchange Online license but it won't be assigned to the user until the admin assigns it. If the user doesn't have an Exchange license already and the admin has yet to assign the Exchange Online license, the user won't be able to schedule meetings in Teams and may be missing other Teams functionality."
Tested it and was true in my lab:

Hi Dario,
I just added a user from admin portal and the Exchange Online is ticked automatically:
I’ll just make a template without the checkmark in Exchange Online.
<EMAIL_ADDRESS>Med venlig hilsen - Kind regards - Mit freundlichen Grüssen
Esben Vidstid
Kunderådgiver
<EMAIL_ADDRESS>FLC Danmark ApS
Brendstrupgårdsvej 21G
DK8200 Aarhus N
www.flc.dkhttp://www.flc.dk
Tlf
Helpdesk
Mobil
Direkte
Email
+45 70 25 09 55
+45 70 20 92 96
+45 28 19 81 84
+45 36 98 35 74
ev<EMAIL_ADDRESS>Fra: Dario Woitasen<EMAIL_ADDRESS>Sendt: Tuesday, 24 March 2020 20.04
Til: MicrosoftDocs/OfficeDocs-SkypeForBusiness<EMAIL_ADDRESS>Cc: Esben Vidstid<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Emne: Re: [MicrosoftDocs/OfficeDocs-SkypeForBusiness] Exchange Online, on or off? (#4254)
Hi @flcevhttps://url12.mailanyone.net/v1/?m=1jGops-0004p4-4R&i=57e1b682&c=d4m524yBy7Bc1t83OreC3Uzow8GnvLhspBYWwV1lQBMShRoBlKhOut2sNSWJ7EnOSl2Yakm4YPl5WnZ7-MwIag1BNlV_JyoTthtRWCGFZWfFuddFBYY0PtwYQ_zrzgzFx0NrLfILI7H3tPHS_X32IXM21eEmgszvgow7GQu7AS4UGJYaZ3FnDKuU_PK1TaZJ8c_ibWhxY99_38Wn5KaFb1yLjpIBqmeqRn4qfzMi6Vg, thank you for your feedback.
About Exchange on-premises, move/migration recommendations are the same:
https://docs.microsoft.com/exchange/exchange-hybridhttps://url12.mailanyone.net/v1/?m=1jGops-0004p4-4R&i=57e1b682&c=5cX6l-Oj1zFXTBQ_ecWbuJamjt_ji4KCwcjOFMkBScyeEURMsEk_-5riugbJDXj0UvLJxYlxPVkgCIqBtDkN76tcuMUCFCLQI6qo63vbrvfhkFtfBckJD2VQlDTdxHkdA6n9ZFgQ_dKsThiR8IdCAqMzYJO8af2sN_FjTAws_fXqZwvQaMSazw50DpCKryXP9F831aJZzi87V-nas6v3pT9FHpeOgPATl2wTdFgSpWTvmYm708uWNHldmysXzDJH
If you want a full Teams service for this user, a mailbox is required. You can move the mailbox to EXO or if you don't:
https://docs.microsoft.com/en-us/microsoftteams/exchange-teams-interacthttps://url12.mailanyone.net/v1/?m=1jGops-0004p4-4R&i=57e1b682&c=1WITkrD5jVISmrwbhDZqREHg8TwlzmDTEizgd65CN03OqoviaqPr1Rc0t0t0tCeK3xmRpXUSnAn505Sxdx-enp-6jec_lPeUTbzthLV4P5KB8CU4H-6RAmuIwqqvRxwN_jpRlMA804WoxR3ithS8VA6Ppj2r7dUP6u2QFeUNCD6odHqEJd4U46fg62JMDpWzVRTVmnPFFDEcXdsmz2zfzx93JxsmFuJJT63CYSVZ_eMY5MliORP9A7iK4msQITlbhxePOvG91sWgDkrH8CWwSLUPODUT0oqdjoQFzeuv4nc
About:
"The Teams Exploratory experience comes with an Exchange Online license but it won't be assigned to the user until the admin assigns it. If the user doesn't have an Exchange license already and the admin has yet to assign the Exchange Online license, the user won't be able to schedule meetings in Teams and may be missing other Teams functionality."
Tested it and was true in my lab:
[image]https://url12.mailanyone.net/v1/?m=1jGops-0004p4-4R&i=57e1b682&c=d8V_Ou3HBf0e4sn6brl4E-n8dS8c8ZGIzjF1MWyqLDaNr6Ot9NqrydTOutuR8pQO0F_88Fcuy2jL3eCDNHmtod0RFzOzdTTAfjgbl4-zKjBswKgtlReMIhjyd7p5LQVz43FDMbgz61A2Rg4jCuxB9qyyk0R8fbv3koSIaqY8NfIT72kp_bPLSSLX323vZQEGiVaEffU3Shs3kZSc3Nkw5S-F1F9fwPAE-ODA-4aYYMS0ddL4KogQAhmopeN2kMihdGDe87ZcFQ9RBoiS0Q3HUc8gqRT7rvNVajyvyD3NXy4Ljwgx0IZiEtw5N2XZk-lv
[image]https://url12.mailanyone.net/v1/?m=1jGops-0004p4-4R&i=57e1b682&c=SuclV99I4iCQofv6PgEnpb6_Aq8U2dHnVyX7NvfR6CooYS1x19RG6HInumsnjewSKFcp3xHKf-JHkU6xcK9eO7cA2g0GaByZFISP65AjxexvgSGjtJDZcTxxMWkzgqfJm3RMfIxaWcYfKxcXp04NpTxD2bJ3oLPwNrXO3ySGL18_AHlDvuTjHPNVs9ZWMU3l09d8oUHlKxvyrVZvfntzuYrXDzTYSecz-jWG9UMw2doiKiJ87RBD_dabY5ErCr-15jZhK7wDLkocfJ29QLy4BZ55fsS3w44nD93WH21VnTxFHMt58sn8ief-nDJ1Vhad
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://url12.mailanyone.net/v1/?m=1jGops-0004p4-4R&i=57e1b682&c=DZJV6ESUJg2slgxU2lx09p76_ifQvO7xc-xu6_CtDM2PfBNiHm3g9LEM6Zg_iNH9HWGrAtm_BStp1Ja_t0vPl7vYdd8AW71fCfeae8xl1A2vs6jKUb4xhW6T6yP8AdTSa-pUx6MmmSnrzEbD0OmOpZwJrBnIFGOk5_YG9RGgxhk607VawWkpDb4DosG9BDBrphTHw8NNO2L0nlsrk3od3PsTs7yXLs_TBX7qjsvmtPMdAoR1lHs_5gyTXmU-WxZxZRpbw-VTnF2ObP3mTfouPO7hGp4bqdDa41Mxv9wq0-5NGjYXp9HamUAeesAzYG79, or unsubscribehttps://url12.mailanyone.net/v1/?m=1jGops-0004p4-4R&i=57e1b682&c=fr-zZZprnMCqZlBnpvrM9OFrncaxi46YU_IyAT-IuEZUW3Od-d3HD9JkK9-0yNGQyuKCVzFhMnY32vh1hiSk3MayGIvrpMoXnHW2S5GejYd5yzsloHW0kpI7pQBRxcfyhS9DrkV1-OVGBUd0Glx0dt5pH2Doz4t4Q4Z-lGo_9bumMc8l2gjYc9DKNHC_PnGx_Ig-tZoT911BNLutljTvSmyhKhig29St6ZpwYfgigxVlRi-E29ER2AfqoGbjjRiqQUq-V1XP8xU6pH7EbpwH0yq_jUv5Kv8oaqJfsTVZIMDLPzE3ZsKQXD0_vFYL-vQ7.

How I can help you? Docs.microsoft.com feedback is available to improve documentation, I can help if you think there is something to update/fix in the article.
Thanks!

is this the same as than the 6 months E1 trial with teams just with less licences? I have been trying to get it for 5 days but this one went straight on

Hi @Biobobski, It could be similar but not the same and they have different purposes.

@flcev Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


Deployment seems cached after updating name in canary.yaml

Had a weird issue where I created a new deployment, and then updated the name field in the canary.yaml. After this every time I did a canary release flagger would roll out podinfo.demo and demo.demo.
It seemed like the name: podinfo was cached and still being triggered along with name: demo? They would each rollout at the same time but with individual config, podinfo.demo had a stepWeight: 20 but demo.demo had stepWeight: 2.
The change that caused the behaviour to start: https://github.com/guyfedwards/istio-demo/commit/397850d2c1d9834639a6b42ad7ef68907ccfa193

I had definitely deleted the namespace, not sure about the deployment itself. I will try and reproduce at a later date 👌


configure ci

ci should run tests and doctests

maybe it would also be nice to run all code snippets from readme and check that they do not fail.


v-model using value from v-for (v-model and v-for on the same element)

Are v-model bindings using values from a v-for possible? I couldn't find any examples showing how to do this. I'm trying to populate a large list of predefined values using this system. Here's an example of what I'm trying to do:
<input v-for="props in fields" v-model="users.primary.{{ props.key }}" type="text" placeholder="{{ props.placeholder }}"/>
<input v-for="props in fields" v-model="users.secondary.{{ props.key }}" type="text" placeholder="{{ props.placeholder }}"/>
<script>
var app = new Vue({
    data: {
        fields: {
            {"key": "username", "placeholder": "Username"},
            {"key": "address", "placeholder": "Address"},
            {"key": "email", "placeholder": "Email"},
            {"key": "name", "placeholder": "Name"},
            {"key": "pass", "placeholder": "Password"},
            {"key": "pass_repeat", "placeholder": "Repeat Password"},
            {"key": "city", "placeholder": "City"},
            {"key": "zip", "placeholder": "Zip"},
            {"key": "phone", "placeholder": "Phone"}
        }
    }
});
</script>

The "placeholder" part of this works, but the two way data binding on the v-model isn't working.

I moved this to http://forum.vuejs.org/topic/1317/v-model-using-value-from-v-for-v-model-and-v-for-on-the-same-element

The link to that page doesn't work anymore.


ng serve does not work with Internet Explorer 11

🐞 Bug report
Command (mark with an x)

- [ ] new
- [ ] build
- [x] serve
- [ ] test
- [ ] e2e
- [ ] generate
- [ ] add
- [ ] update
- [ ] lint
- [ ] xi18n
- [ ] run
- [ ] config
- [ ] help
- [ ] version
- [ ] doc

Is this a regression?

 Yes, the previous version in which this bug was not present was: 7.x
Description
 A clear and concise description of the problem...
Internet Explorer 11 does not load site when using ng serve.
Only errors in the dev console.
🔬 Minimal Reproduction

ng new ienotworking
ng serve
or
ng serve --prod
🔥 Exception or Error

SCRIPT1002: Syntax error
polyfills.js (2891,5)
SCRIPT1002: Syntax error
vendor.js (156,1)
SCRIPT1002: Syntax error
main.js (87,24)

🌍 Your Environment

Angular CLI: 8.0.0-rc.4
Node: 10.15.3
OS: win32 x64
Angular: 8.0.0-rc.4
... animations, cli, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... platform-server, router, service-worker

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.800.0-rc.4
@angular-devkit/build-angular     0.800.0-rc.4
@angular-devkit/build-optimizer   0.800.0-rc.4
@angular-devkit/build-webpack     0.800.0-rc.4
@angular-devkit/core              8.0.0-rc.4
@angular-devkit/schematics        8.0.0-rc.4
@angular/pwa                      0.12.4
@ngtools/webpack                  8.0.0-rc.4
@schematics/angular               7.2.4
@schematics/update                0.800.0-rc.4
rxjs                              6.4.0
typescript                        3.4.5
webpack                           4.30.0
Anything else relevant?

IE11 only.

Hi, thanks for reporting this. However this is now by design in version 8.
By default in version 8 we enable differential loading for ng build. However for ng test and ng serve we only generate a single ES2015 build which cannot run in IE 11.
There are a couple of options that you can do if you want to have ES5 code during serve.
1. Disable differential loading completely, (Not recommended)
You can turn differential loading off by changing the target from es2015 to es5 in your tsconfig.json
2. Have multiple configuration for serve.
Create a new tsconfig tsconfig.es5.json next to tsconfig.app.json with the below contents
{
 "extends": "./tsconfig.app.json",
 "compilerOptions": {
     "target": "es5"
  }
}

In your angular.json add two new configuration section under the build and serve  target to provide a new tsconfig.
"build": {
  "builder": "@angular-devkit/build-angular:browser",
  "options": {
      ...
  },
  "configurations": {
    "production": {
        ...
    },
    "es5": {
      "tsConfig": "./tsconfig.es5.json"
    }
  }
},
"serve": {
  "builder": "@angular-devkit/build-angular:dev-server",
  "options": {
      ...
  },
  "configurations": {
    "production": {
     ...
    },
    "es5": {
      "browserTarget": "app:build:es5"
    }
  }
},

You can then run the serve with this configuration using the below command.
ng serve --config es5

While that is a "workaround" for now, home come the CLI can't do differential loading in serve mode also, so the application at least behaves the same for both prod and non-prod?

Workaround not working for me, I run ng serve --config es5 and the CLI returns Unknown option: '--config' and Unknown option: 'es5'

Workaround not working for me when running ng serve --config es5. The CLI returns Unknown option: '--config' and Unknown option: 'es5'
I ran ng s -c=es5 which worked for me

Try ng serve --configuration es5, I think the previous mentioned solution was not working cause config does not exist as an option or alias OOTB

Coming from #14656 here 😄, I have one last question:
Why differential loading has only been implemented for ng build ?
Why can't we have an angularCompiler option or an angular.json flag to simply activate it no matter what in serve/test/e2e. It could even have a default value to turn it off (ie the new default behaviour)

From what I can see of the new differential build, the process is effectively done twice. i.e. once for es5 and once for es6. I imagine it would significantly slow down the dev server reload time if differential loading was enabled by default. My project is already pretty slow so I if this was added to the dev server
I would vote for it be opt-in. Personally, I think the ng serve -c es5 workaround is fine, but it would be great if it was documented on the main angular.io site.

I'm getting the following error when trying the workaround above:
Schema validation failed with the following errors:
  Data path "" should NOT have additional properties(tsConfig).

and VSCode underlines tsConfig

@ciesielskico You are mixing up the serve and build sections. Take a closer look at the example above.

The manual steps required to update the multiple files bothered me and I didn't want to repeat it 20+ times this year for all our new projects so I created a npm Dev dependency to automate the workaround config updates.  You can find it here : https://github.com/ChrisMeeusen/ng-ie-serve.  Cheers.

@ChrisMeeusen
thanks for that lib. Just one note: why do you use fs.readFileSync to read .json files and then parsing them instead of using require? From the node.js docs
LOAD_AS_FILE(X)
1. If X is a file, load X as JavaScript text.  STOP
2. If X.js is a file, load X.js as JavaScript text.  STOP
3. If X.json is a file, parse X.json to a JavaScript Object.  STOP
4. If X.node is a file, load X.node as binary addon.  STOP

@destus90
Hey thanks for the feedback.  Honestly I didn't use require because I didn't know that it had those capabilities.
However, after trying to use require I get the following runtime error:
Error: Cannot find module 'package.json'
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:587:15)
    at Function.Module._load (internal/modules/cjs/loader.js:513:25)
    at Module.require (internal/modules/cjs/loader.js:643:17)
    at require (internal/modules/cjs/helpers.js:22:18)
    at Object.<anonymous> (C:\github\ng-ie-serve\src\package-json-utils.js:2:19)
    at Module._compile (internal/modules/cjs/loader.js:707:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:718:10)
    at Module.load (internal/modules/cjs/loader.js:605:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:544:12)
    at Function.Module._load (internal/modules/cjs/loader.js:536:3)

I pushed my attempt at using require to a branch called require-json-read if your interested at seeing / fixing what I'm doing wrong.
Thanks!

Also situation with 'ng test' on IE11/PhantomJS is totally diferent it seems, as even after setting target to 'es5' in tsconfig it doesn't work correctly, due to missing polyfils, as said in my issue: #14691

Is it possible to document this better please? seems a good angular.io place which is woefully misleading.
For a start polyfills for es6 are now part of the angular build but I had to find that out by reading many articles from google.
If I remember the generation of the es5 polyfill file was automatic in beta - now we have some extra recommended steps which are undocumented.

@alan-agius4  can we think about the possibility of adding an option in the cli menus to ask if you want  IE support for ng serve AND auto generate tsconfig.es5.json if the dev answer Yup.
For me it makes it boring and it's waste! to do that each time we create a project and also you forced [BY DESIGN] the creation of this -> https://github.com/ChrisMeeusen/ng-ie-serve
Hope you do something about it, Angular Team.

From what I can see of the new differential build, the process is effectively done twice. i.e. once for es5 and once for es6. I imagine it would significantly slow down the dev server reload time if differential loading was enabled by default. My project is already pretty slow so I if this was added to the dev server
I would vote for it be opt-in. Personally, I think the ng serve -c es5 workaround is fine, but it would be great if it was documented on the main angular.io site.

Why not just add this to the angular.json file and then a user could run it or not

Hi, thanks for reporting this. However this is now by design in version 8.
By default in version 8 we enable differential loading for ng build. However for ng test and ng serve we only generate a single ES2015 build which cannot run in IE 11.
There are a couple of options that you can do if you want to have ES5 code during serve.
1. Disable differential loading completely, (Not recommended)
You can turn differential loading off by changing the target from es2015 to es5 in your tsconfig.json
2. Have multiple configurations for serve.
Create a new tsconfig tsconfig-es5.app.json next to tsconfig.app.json with the below contents
{
 "extends": "./tsconfig.app.json",
 "compilerOptions": {
     "target": "es5"
  }
}

In your angular.json add two new configuration section under the build and serve target to provide a new tsconfig.
"build": {
  "builder": "@angular-devkit/build-angular:browser",
  "options": {
      ...
  },
  "configurations": {
    "production": {
        ...
    },
    "es5": {
      "tsConfig": "./tsconfig-es5.app.json"
    }
  }
},
"serve": {
  "builder": "@angular-devkit/build-angular:dev-server",
  "options": {
      ...
  },
  "configurations": {
    "production": {
     ...
    },
    "es5": {
      "browserTarget": "app:build:es5"
    }
  }
},

You can then run the serve with this configuration using the below command.
ng serve --configuration es5

Hi, thanks for reporting this. However this is now by design in version 8.
By default in version 8 we enable differential loading for ng build. However for ng test and ng serve we only generate a single ES2015 build which cannot run in IE 11.
There are a couple of options that you can do if you want to have ES5 code during serve.
1. Disable differential loading completely, (Not recommended)
You can turn differential loading off by changing the target from es2015 to es5 in your tsconfig.json
2. Have multiple configurations for serve.
Create a new tsconfig tsconfig-es5.app.json next to tsconfig.app.json with the below contents
{
 "extends": "./tsconfig.app.json",
 "compilerOptions": {
     "target": "es5"
  }
}

In your angular.json add two new configuration section under the build and serve target to provide a new tsconfig.
"build": {
  "builder": "@angular-devkit/build-angular:browser",
  "options": {
      ...
  },
  "configurations": {
    "production": {
        ...
    },
    "es5": {
      "tsConfig": "./tsconfig-es5.app.json"
    }
  }
},
"serve": {
  "builder": "@angular-devkit/build-angular:dev-server",
  "options": {
      ...
  },
  "configurations": {
    "production": {
     ...
    },
    "es5": {
      "browserTarget": "app:build:es5"
    }
  }
},

You can then run the serve with this configuration using the below command.
ng serve --configuration es5

workaround is not working for me. I am getting below error
SCRIPT1046: Multiple definitions of a property not allowed in strict mode
File: vendor.js, Line: 4066, Column: 25

@sandipsrane, did you enable ie mode in the browserlist file IE 9-11?

@ricardosaracino I did after your response but same result..

@sandipsrane, have you tried just using this :
https://github.com/ChrisMeeusen/ng-ie-serve
It should just make all the changes for you.  Install it, run it, and move on.

@ChrisMeeusen giving me this error

@sandipsrane Sorry there was an issue with the package and bin command setup.  Please install version 1.0.1 and kick the tires.
Thanks for letting me know about the issue.

@ChrisMeeusen I am still getting the same error after installing version 1.0.1

@sandipsrane created an issue here as to not pollute this thread.  I'm working on a fix for this.
https://github.com/ChrisMeeusen/ng-ie-serve/issues/2

Options mentioned above are working now. It was not working because Ivy was enabled in tsconfig.app.json. After making it false it started working.
"angularCompilerOptions": {
"enableIvy": true
}

    "es5": {
      "tsConfig": "./tsconfig-es5.app.json"
    }

shouldn't it read src/tsconfig-es5.app.json instead?
tsconfig.app.json lives under src/ and therefore I created tsconfig-es5.app.json there, next to it.
and I think it is worth pointing out that in
{
  "browserTarget": "app:build:es5"
}

"app" should be replaced by the actual app name (actual key value used under "project" key)

It is not working

It is not working

@chintandalwadiwk
If IE11 throws object doesn't support..., it is often because IE is running in compatibility mode (eg 5). What is the number shown on the top right (just above the word "Target" (at the top right corner of your screen capture)?

Hello,
We had the issue too while migrating as we have to support IE10-IE11. I have followed the instructions here and read other thread and that helped fixing the script issues and supporting IE for local dev testing (ng serve) but there is still a issue with some issues in rendering. Could be related to flex / break-points calculations . We use Material with Bootstrap grid for layout (only grid, not full Bootstrap) and nothing really special aside of that.
Is there a way to find what polyfill where removed that could affect css and such?
I will try to compare the runtime of our older 7.x build where everything worked just fine (without anything extra in the polyfill file) but I would gladly take any hint.

If this can be usefull to someone else, my coworker was able to figure it out.
It was something missing interacting with bootstrap that he fixed with something like this:
@import './variables';
.row {
    width: calc( 100% + (#{$gutter} * 2));
}

Some more testing required but it seem to be fine now.

It is not working

Did you resolve this issue? Same have. IE-11, and I checked what not compatibility mode. I got "Object doesn't support property or method 'values'" error in infinity cycle.


Segmentation fault when running habitat-viewer and SHIFT + Left Click for semantic annotations on 4K display

Habitat-Sim version
0.3.0
🐛 Bug
When running the HM3D semantics example in habitat-viewer, I get a Segmentation fault (core dumped) error when SHIFT+ Left Click an object to print it's semantic class. This only happens when I am connected to my 4K display. It doesn't happen when I connect a 1080p monitor or if I remote in to my desktop from my laptop (1080p) over xRDP. As far as I can tell, all other mouse clicks in the habitat-viewer app are handled successfully on the 4K display. I believe it has something to do with the DPI of the display because if I launch the application on the 1080p monitor and then switch to the 4K monitor it still works.
Steps to Reproduce
Steps to reproduce the behavior:
habitat-viewer --dataset './data/scene_datasets/hm3d/hm3d_annotated_basis.scene_dataset_config.json' TEEsavR23oF

Connect to a 4K display
Run the above command to launch the habitat-viewer application with a scene with semantic annotations
SHIFT + Left Click an object

Expected behavior
The semantic class of the clicked object to be printed out below the rest of the UI overlay
System Info

OS (e.g., Linux): Ubuntu 22.04
Python version: 3.9.18
GPU models and configuration: NVIDIA GeForce RTX 4090 (Driver Version: 535.154.05)
CUDA version: 12.2

@frank-gallagher thanks for reporting this bug.
We are aware of some issues with viewer application logic on high-DPI displays. For example https://github.com/facebookresearch/habitat-sim/pull/2321.


[5.2] Checking value in isJsonCastable function

Situation
We need to store some json data in pivot table. And for that we decide to use Eloquent attribute casting functionality, for more easily setting and getting data.
For that we must create custom pivot model and set protected $casting property.

expected:
attribute setted in $casting property as json or array will be automaticaly casted in array by accesing that property on our custom pivot model and any setted array  or JsonSerializable object will be json encoded for storing in database.
actually:
when pivot model loading, values in castable properties will be double encoded and therefore can not be correctly decoded as we expect.
#10533 and partially #8972

how to reproduce:

You can clone prepared by me app for demonstrating this issue. https://github.com/evsign/laravelpivotissue.git/
Or follow instructions:

create new laravel project
php artisan make:migration create_students_table --create=students
php artisan make:migration create_subjects_table --create=subjects
php artisan make:migration create_student_subject_table --create=student_subject
php artisan make:model Student
php artisan make:model Subject
php artisan make:model StudentSubjectPivot

Migrations
students:
    $table->increments('id');
    $table->string('name');
    $table->string('email');
    $table->timestamps();
subjects:
    $table->increments('id');
    $table->string('name');
    $table->string('description');
    $table->timestamps();
student_subject:
    $table->unsignedInteger('student_id');
    $table->unsignedInteger('subject_id');
    $table->text('marks'); //or $table->json('marks');
    $table->timestamps();

Student.php model
<?php

namespace App;

use Illuminate\Database\Eloquent\Model;

class Student extends Model
{
    protected $fillable = [
    	'name',
        'email'
    ];

    public function subjects()
    {
        return $this->belongsToMany('App\Subject')->withPivot('marks');
    }

    public function newPivot(Model $parent, array $attributes, $table, $exists)
    {
        if ($parent instanceof Subject) {
            return new StudentSubjectPivot($parent, $attributes, $table, $exists);
        }

        return parent::newPivot($parent, $attributes, $table, $exists);
    }
}

Subject.php model
<?php

namespace App;

use Illuminate\Database\Eloquent\Model;

class Subject extends Model
{
	protected $fillable = [
		'name',
		'description'
	];

	public function students()
	{
		return $this->belongsToMany('App\Student')->withPivot('marks');
	}

	public function newPivot(Model $parent, array $attributes, $table, $exists)
    {
        if ($parent instanceof Student) {
            return new StudentSubjectPivot($parent, $attributes, $table, $exists);
        }

        return parent::newPivot($parent, $attributes, $table, $exists);
    }
}

StudentSubjectPivot.php model
<?php

namespace App;

use Illuminate\Database\Eloquent\Relations\Pivot;
use Illuminate\Database\Eloquent\Model;

class StudentSubjectPivot extends Pivot
{
	protected $casts = [
		'marks' => 'array'
	];

}

Then fill database with test data including pivot marks column with json
$student = App\Student::create(['name' => 'evsign', 'email' => 'evsign@live.ru']);
$subject = App\Subject::create(['name' => 'mathematics', 'description' => 'language of the universe']);
$student->subjects()->attach([$subject->id => [ 'marks' => '[{"date":"2009-12-29 03:01:18","mark":"D"},{"date":"2009-12-29 03:00:40","mark":"F"},{"date":"2009-12-29 03:01:00","mark":"B"},{"date":"2009-12-29 03:00:33","mark":"C"},{"date":"2009-12-29 03:01:09","mark":"A"},{"date":"2009-12-29 03:00:27","mark":"A"},{"date":"2009-12-29 03:00:37","mark":"A"},{"date":"2009-12-29 03:00:25","mark":"F"}]']]);
dd(App\Student::find($student->id)->subjects()->first()->pivot->marks); // will be string instead of array

For fix this issue
Use accessors and mutators with checking values
or
I suggest to send and check values for array or object in isJsonCastable function in Illuminate\Database\Eloquen\Model.php.  :relaxed:

Did you uncover the root reason why the values are double encoded? What is causing that in the Laravel core?

@taylorotwell Yes, when pivot relation setting, pivot model filling with fill function and all attributes sets by calling setAttribute function inside wich situated all mutating and casting logic. Whereupon all values retrieved from database passing in this function and will be double setMutated/casted. In another words, values retrieved from database already casted/setMutated and in situation of array casting we can just check value for array or object.
Also i try to suggest, maybe casting/setMutating logic must be in moment of saving model to database and reverse logic in setAttribute function.

I see the problem but this isn't really the correct fix. The problem lies in the Pivot model constructor where we call ->fill()... however, changing this now would introduce another breaking change if someone is manually creating a new Pivot instance and they expect the values they pass to be mutated.

Any change would probably have to go on 5.3 since it would be breaking.


Delb plugins

here's my progress on the delb plugins that covers ExistClient.from_url so far.
if you have time to review and even more time, you could look into an exceptions hierarchy. (i had removed the unused snakesist exception, but now added new ones.)

thx for the extensive work on this. i'll try to do a review today.
is it ok by you if i update the travis config here to reflect the current setup?

is it ok by you if i update the travis config here to reflect the current setup?

no idea what you're aiming at, but okay. the required delb-reference-plugins should be fetchable by now. either the cheeseshop is laggy or there's an issue w/ poetry.
i'm already working on the removal of now obsolete code parts. if you are faster w/ the review, just merge it and i'll open another pr.

sorry, should have been more explicit about what i meant: with the current travis config there's a now obsolete step where an exist docker container is being set up, which then, if i see this right, gets in the way of the current testing setup. i was meaning to remove that step.
i can wait with the review, if that makes things simpler.

ah, thanks i hadn't noticed that.
i don't know what works best for you a smaller or a bigger review. bets may be commit-by-commit.

a bigger review should be fine, i think.

okay, i'm done with it. add commits as you like and i'll have a look at them.

i'm only aware the the following could / should be done:

establishing an exception hierarchy and emitting them where appropriate
having a look at the docs

currently i feel to close to the gegenstand to do both properly.
when you feel that things are settled here, we should coordinate the releases of new delb & snakesist versions.

i will try to do those things. it might take a bit, as i have no prior experience with creating a proper exception hierarchy, but this is a good opportunity for me to catch up on the topic.

i suggest to keep it simple & extensible, because it should serve actual problems encountered in actual usage and who knows what these response.raise_from_status() may yield.

i suggest to keep it simple & extensible, because it should serve actual problems encountered in actual usage and who knows what these response.raise_from_status() may yield.

so this is what i managed to come up with... and i'm almost sure it's quite a mess, but that's ok for a first iteration, i guess. i'll have a fresh look at it next week. if you have the time, any feedback would be super appreciated.
i'll review the docs after the exceptions topic is resolved.

i will not add any further commits here unles you aks me to.


Add AssemblyAIAudioToTextService.cs

Add AssemblyAIAudioToTextService
This PR is in draft because it relies on my locally compiled .NET SDK for AAI which hasn't been released yet.
Todo:

Add sample for using AssemblyAIAudioToTextService

AssemblyAI will be built into Semantic Kernel as a connector: https://github.com/microsoft/semantic-kernel/issues/5392


2 issues with Archive pricing

You need to change the COGS acronym - that isn't a common enough term to have in a header, I think you could use a more normal/friendly work, like just "cost" for example.
Secondly, I was hoping this page would help me understand what the billing process would be. In the overview it talks about setting up an Azure subscription but this Pricing page doesn't mention Azure Storage costs at all. Will a customer be billed through their nomral Azure invoice or something else?

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

ID: e0bb494d-2f9f-a6a8-d2a3-4f55b6b052db
Version Independent ID: e0bb494d-2f9f-a6a8-d2a3-4f55b6b052db
Content: Pricing model for Microsoft 365 Archive (Preview) - Microsoft Syntex
Content Source: microsoft-365/syntex/archive/archive-pricing.md
Service: microsoft-syntex
GitHub Login: @chuckedmonson
Microsoft Alias: chucked

@thommck -- Hi, and thanks for your feedback. I'll follow up with some folks and get back. Thanks again.

To make it easier for you to submit feedback on articles on learn.microsoft.com, we're transitioning our feedback system from GitHub Issues to a new experience.
As part of the transition, this GitHub Issue will be moved to a private repository. We're moving Issues to another repository so we can continue working on Issues that were open at the time of the transition. When this Issue is moved, you'll no longer be able to access it.
If you want to provide additional information before this Issue is moved, please update this Issue before December 15th, 2023.
With the new experience, you no longer need to sign in to GitHub to enter and submit your feedback. Instead, you can choose directly on each article's page whether the article was helpful. Then you can then choose one or more reasons for your feedback and optionally provide additional context before you select Submit.
Here's what the new experience looks like.
Note: The new experience is being rolled out across learn.microsoft.com in phases. If you don't see the new experience on an article, please check back later.
First, select whether the article was helpful:

Then, choose at least one reason for your feedback and optionally provide additional details about your feedback:

Article was helpful
Article was unhelpful

Finally, select Submit and you're done!


Decode CF bounds to coords

CF conventions define Cell Boundaries and specify how to encode the presence of cell boundary variables in dataset attributes.

To represent cells we add the attribute bounds to the appropriate coordinate variable(s). The value of bounds is the name of the variable that contains the vertices of the cell boundaries.

For example consider this dataset: http://esgf-data.ucar.edu/thredds/dodsC/esg_dataroot/CMIP6/CMIP/NCAR/CESM2/historical/r10i1p1f1/Amon/tas/gn/v20190313/tas_Amon_CESM2_historical_r10i1p1f1_gn_200001-201412.nc
url = 'http://esgf-data.ucar.edu/thredds/dodsC/esg_dataroot/CMIP6/CMIP/NCAR/CESM2/historical/r10i1p1f1/Amon/tas/gn/v20190313/tas_Amon_CESM2_historical_r10i1p1f1_gn_200001-201412.nc'
ds = xr.open_dataset(url)
ds

gives
<xarray.Dataset>
Dimensions:    (lat: 192, lon: 288, nbnd: 2, time: 180)
Coordinates:
  * lat        (lat) float64 -90.0 -89.06 -88.12 -87.17 ... 88.12 89.06 90.0
  * lon        (lon) float64 0.0 1.25 2.5 3.75 5.0 ... 355.0 356.2 357.5 358.8
  * time       (time) object 2000-01-15 12:00:00 ... 2014-12-15 12:00:00
Dimensions without coordinates: nbnd
Data variables:
    time_bnds  (time, nbnd) object ...
    lat_bnds   (lat, nbnd) float64 ...
    lon_bnds   (lon, nbnd) float64 ...
    tas        (time, lat, lon) float32 ...

Despite the presence of the bounds attributes
>>> print(ds.time.bounds, ds.lat.bounds, ds.lon.bounds)
time_bnds lat_bnds lon_bnds

The variables time_bnds, lat_bnds, and lon_bnds are not decoded as coordinates but as data variables. I believe that this is not in accordance with CF conventions.
Instead, we should decode all bounds variables to coordinates.
I cannot think of a single use case where one would want to treat these variables as data variables rather than coordinates. It would be easy to implement, but it is a breaking change.
Not that this is just a proposal to move bounds variables to the coords part of the dataset. It does not address the more difficult / complex question of how to actually use the bounds for indexing or plotting operations (see e.g. #1475, #1613), although it could be a first step in that direction.
Full ncdump of dataset

xarray.Dataset {
dimensions:
	lat = 192 ;
	lon = 288 ;
	nbnd = 2 ;
	time = 180 ;

variables:
	float64 lat(lat) ;
		lat:axis = Y ;
		lat:bounds = lat_bnds ;
		lat:standard_name = latitude ;
		lat:title = Latitude ;
		lat:type = double ;
		lat:units = degrees_north ;
		lat:valid_max = 90.0 ;
		lat:valid_min = -90.0 ;
		lat:_ChunkSizes = 192 ;
	float64 lon(lon) ;
		lon:axis = X ;
		lon:bounds = lon_bnds ;
		lon:standard_name = longitude ;
		lon:title = Longitude ;
		lon:type = double ;
		lon:units = degrees_east ;
		lon:valid_max = 360.0 ;
		lon:valid_min = 0.0 ;
		lon:_ChunkSizes = 288 ;
	object time(time) ;
		time:axis = T ;
		time:bounds = time_bnds ;
		time:standard_name = time ;
		time:title = time ;
		time:type = double ;
		time:_ChunkSizes = 512 ;
	object time_bnds(time, nbnd) ;
		time_bnds:_ChunkSizes = [1 2] ;
	float64 lat_bnds(lat, nbnd) ;
		lat_bnds:units = degrees_north ;
		lat_bnds:_ChunkSizes = [192   2] ;
	float64 lon_bnds(lon, nbnd) ;
		lon_bnds:units = degrees_east ;
		lon_bnds:_ChunkSizes = [288   2] ;
	float32 tas(time, lat, lon) ;
		tas:cell_measures = area: areacella ;
		tas:cell_methods = area: time: mean ;
		tas:comment = near-surface (usually, 2 meter) air temperature ;
		tas:description = near-surface (usually, 2 meter) air temperature ;
		tas:frequency = mon ;
		tas:id = tas ;
		tas:long_name = Near-Surface Air Temperature ;
		tas:mipTable = Amon ;
		tas:out_name = tas ;
		tas:prov = Amon ((isd.003)) ;
		tas:realm = atmos ;
		tas:standard_name = air_temperature ;
		tas:time = time ;
		tas:time_label = time-mean ;
		tas:time_title = Temporal mean ;
		tas:title = Near-Surface Air Temperature ;
		tas:type = real ;
		tas:units = K ;
		tas:variable_id = tas ;
		tas:_ChunkSizes = [  1 192 288] ;

// global attributes:
	:Conventions = CF-1.7 CMIP-6.2 ;
... [truncated]

Output of xr.show_versions()

INSTALLED VERSIONS
------------------
commit: None
python: 3.6.7 | packaged by conda-forge | (default, Jul  2 2019, 02:07:37)
[GCC 4.2.1 Compatible Clang 4.0.1 (tags/RELEASE_401/final)]
python-bits: 64
OS: Darwin
OS-release: 16.7.0
machine: x86_64
processor: i386
byteorder: little
LC_ALL: None
LANG: en_US.UTF-8
LOCALE: en_US.UTF-8
libhdf5: 1.10.5
libnetcdf: 4.6.2
xarray: 0.14.0+19.gba48fbcd
pandas: 0.25.1
numpy: 1.17.2
scipy: 1.3.1
netCDF4: <IP_ADDRESS>
pydap: None
h5netcdf: 0.7.4
h5py: 2.10.0
Nio: None
zarr: 2.3.2
cftime: <IP_ADDRESS>
nc_time_axis: 1.2.0
PseudoNetCDF: None
rasterio: None
cfgrib: <IP_ADDRESS>
iris: None
bottleneck: 1.2.1
dask: 2.4.0
distributed: 2.4.0
matplotlib: 3.1.1
cartopy: 0.17.0
seaborn: 0.9.0
numbagg: None
setuptools: 41.2.0
pip: 19.2.3
conda: None
pytest: 5.1.2
IPython: 7.8.0
sphinx: 1.6.5

Yes, we should move forward with https://github.com/pydata/xarray/pull/2844

Ak I had no idea there was even an open PR for this! Sorry!

@rabernat it would be helpful if you could comment there (#2844) on where we store the coordinates and bounds attributes.

bounds and grid_mapping?

Closed by #2844


Allow array to be used as a function name again

Rationale
https://github.com/sqlparser-rs/sqlparser-rs/pull/429 accidentally introduced a regression that found while testing downstream in DataFusion : https://github.com/apache/arrow-datafusion/pull/1956
In particular, sql parser used to parse queries like this:
SELECT array(c1, cast(c2 as varchar)) FROM test

As a call to a function named array. However, after https://github.com/sqlparser-rs/sqlparser-rs/pull/429 it generates a parser error: SQL(ParserError("Expected [, found: ("))',
Changes

Restore the behavior of parsing array(..) as a function call.
Tests for same

Note array[] is still parsed as an array expression.
More details here https://github.com/sqlparser-rs/sqlparser-rs/pull/429#issuecomment-1062041430
cc @monadbobo

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

13 of 13 (100.0%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.009%) to 90.45%

Files with Coverage Reduction
New Missed Lines
%

src/parser.rs
1
83.78%

Totals

Change from base Build<PHONE_NUMBER>:
-0.009%

Covered Lines:
7274

Relevant Lines:
8042

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

13 of 13 (100.0%)  changed or added relevant lines in 2 files are covered.
20 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.01%) to 90.449%

Files with Coverage Reduction
New Missed Lines
%

src/parser.rs
1
83.77%

tests/sqlparser_common.rs
19
99.02%

Totals

Change from base Build<PHONE_NUMBER>:
-0.01%

Covered Lines:
7273

Relevant Lines:
8041

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

13 of 13 (100.0%)  changed or added relevant lines in 2 files are covered.
20 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.01%) to 90.449%

Files with Coverage Reduction
New Missed Lines
%

src/parser.rs
1
83.77%

tests/sqlparser_common.rs
19
99.02%

Totals

Change from base Build<PHONE_NUMBER>:
-0.01%

Covered Lines:
7273

Relevant Lines:
8041

💛  - Coveralls


Paypal Sandbox server down

The last couple of days we've being getting this error from our ci server.
 JSON::ParserError:
      756: unexpected token at '<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN">

      <title>502 Proxy Error</title>

Proxy Error

The proxy server received an invalid
      response from an upstream server.

      The proxy server could not handle the request POST /AdaptivePayments/CancelPreapproval.

      Reason: Error reading from remote server

I believe that this is caused by  Request#post not being able to handle this 502 error from PayPal
https://github.com/tc/paypal_adaptive/blob/master/lib/paypal_adaptive/request.rb#L95
Would you accept a pull request to have this method throw a Net:HTTP exception just for this case?

It might be a better idea if you can:
check the http response code
if 200, try the json parse
else throw an exception with the http code.
On Sat, Mar 3, 2012 at 8:09 PM, Javier Alvarez
<EMAIL_ADDRESS>wrote:

The last couple of days we've being getting this error from our ci server.
 JSON::ParserError:
     756: unexpected token at '<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN">

     <title>502 Proxy Error</title>

Proxy Error

The proxy server received an invalid
     response from an upstream server.

     The proxy server could not handle the request POST /AdaptivePayments/CancelPreapproval.

     Reason: Error reading from remote server

I believe that this is caused by  Request#post not being able to handle this 502 error from PayPal
https://github.com/tc/paypal_adaptive/blob/master/lib/paypal_adaptive/request.rb#L95
Would you accept a pull request to have this method throw a Net:HTTP exception just for this case?

Reply to this email directly or view it on GitHub:
https://github.com/tc/paypal_adaptive/issues/45

--
Tommy Chheng


[BugFix] Avoid NPE when using coord.getQueryProfile

What type of PR is this：

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

After executing a SQL, there are some writing profile operations using coord.getQueryProfile().
However, coord.queryProfile is init in coord.prepareProfile, which causes sometimes it hasn't been inited when the SQL is finished to execute. For example, if the preparation failes or query queue management returns error, coord.prepareProfile will never be invoked.
        if (coord != null) {
            coord.getQueryProfile().getCounterTotalTime().setValue(TimeUtils.getEstimatedTime(beginTimeInNanoSecond));
            coord.endProfile();
            coord.mergeIsomorphicProfiles();
            profile.addChild(coord.getQueryProfile());
            coord = null;
        }

Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto backported to target branch

[x] 2.5
[x] 2.4
[x] 2.3
[x] 2.2

[FE PR Coverage Check]
:disappointed: fail : 0 / 5 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/qe/StmtExecutor.java
0
5
00.00%
[274, 275, 276, 277, 278]

run starrocks_admit_test

@Mergifyio backport branch-2.5

@Mergifyio backport branch-2.4

@Mergifyio backport branch-2.3

@Mergifyio backport branch-2.2


Fit & finish - UI Copy | Bot templates

Fit & finish - UI Copy | Templates
White cells = existing copy
Yellow/blue cells = new copy
Orange = P0
Link to 'live' copy on ' Bot Templates ' tab:
https://microsoft.sharepoint.com/:x:/r/teams/ConversationalAI785/_layouts/15/Doc.aspx?sourcedoc={6B1FACBD-B5FE-4AD2-A35B-995635C1696B}&file=glossary.xlsx&action=edit&mobileredirect=true&wdPreviousSession=36539654-a42b-4b87-be1b-de96507b4422&wdOrigin=TEAMS-ELECTRON.teams.undefined&cid=676cf454-61e0-4182-bd27-39044bb51cea

Shiproom - approving as today is fit and finish day.  must be merged by EOD today


Update readme.md

I missed this when deploying.

Thank you!

Seems I broke the list numbering @frankmarasco. It goes 1-5 then restarts at 1 now.

No worries I didn't notice that either thanks for letting me know.
Sent from my phone excuse my thumbs
On Wed, Mar 23, 2016 at 2:11 PM -0700, "Patrik Björklund" <notifications<EMAIL_ADDRESS>wrote:
Seems I broke the list numbering @frankmarascohttps://github.com/frankmarasco. It goes 1-5 then restarts at 1 now.

You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHubhttps://github.com/OfficeDev/PnP-Tools/pull/38#issuecomment-200544957


bug: TypeScript errors with noImplicitAny:true tsconfig

Type: bug
Ionic Version: 2.x
Platform: all
I want to be pedantic about typing in my Ionic app, so I added "noImplicitAny":true to my tsconfig.json.  Works great, except there seems to be some typing issues with the Swiper widget.  I get this when trying to compile my TypeScript:
ERROR in [default] /Users/lehresman/Projects/gazelle-mobile/node_modules/ionic-framework/components/slides/swiper-widget.d.ts:8:2
'update', which lacks return-type annotation, implicitly has an 'any' return type.

ERROR in [default] /Users/lehresman/Projects/gazelle-mobile/node_modules/ionic-framework/components/slides/swiper-widget.d.ts:9:2
'slideNext', which lacks return-type annotation, implicitly has an 'any' return type.

ERROR in [default] /Users/lehresman/Projects/gazelle-mobile/node_modules/ionic-framework/components/slides/swiper-widget.d.ts:10:2
'slidePrev', which lacks return-type annotation, implicitly has an 'any' return type.

When I edit swiper-widget.d.ts and add :any to all the function declarations, it works fine.  But I suspect that's not the actual solution and figured I should report it.
I'm running 2.0.0-beta.0.

This issue was moved to driftyco/ionic-cli#742

Thanks for the issue! We actually have quite a few untyped files still, and resolving the ones with noImplicitAny will be the next step in fully adding types to the framework.

@tlancina @jthoms1 should we close this?

According to @danbucholtz this has been resolved, closing.


[Market Standalone Page] - Favorites

The star next to the market should be filled with colour here like above so it's clear that it is selected and you can click on it again to remove.

@achilleas-kal Thanks for the feedback - we will be redesigning the trading UI at some point.
Will keep the color mild (i.e. grey / white) for now as the trading screen is already condensed with lots of information.

Sure @terencetang9 wasn't referring to the colour specifically, normally grey would mean it's not selected and white that it is (at least to me) and given that BAYC/WETH PERP is selected here I expected to see it "white" and not grey.


linux-aarch64 support

I am trying to install   sqlite-vec via pixi on a linux-aarch64 platform and it fails
x failed to solve the pypi requirements of 'notebooks' 'linux-aarch64'
|-> failed to resolve pypi dependencies
`-> Because sqlite-vec==0.1.3 has no wheels with a matching platform tag
and only sqlite-vec<=0.1.3 is available, we can conclude that sqlite-
vec>=0.1.3 cannot be used.
And because you require sqlite-vec>=0.1.3, we can conclude that your
requirements are unsatisfiable.
  hint: Pre-releases are available for sqlite-vec in the requested
  range (e.g., 0.1.4a2), but pre-releases weren't enabled (try:
  `--prerelease=allow`)

Issue #110 seems to be related too. I've personally been running into issues trying to install the package on an macbook air m1 inside of docker (arm image).
9.963   Unable to find installation candidates for sqlite-vec (0.1.3)
9.964
9.964   at /usr/local/lib/python3.11/site-packages/poetry/installation/chooser.py:74 in choose_for
9.970        70│
9.970        71│             links.append(link)
9.970        72│
9.970        73│         if not links:
9.970     →  74│             raise RuntimeError(f"Unable to find installation candidates for {package}")
9.970        75│
9.970        76│         # Get the best link
9.970        77│         chosen = max(links, key=lambda link: self._sort_key(package, link))
9.970        78│
9.970
9.970 Cannot install sqlite-vec.

I wonder if the cosmopolitan builds could be used for aarch64. It looks like the biggest limitation to arm builds is that github doesn't support arm runners generally at the moment.

Forgot to mention: as of v0.1.6 there are official Linux arm/aarch64 builds of sqlite-vec available.
Here's a direct link if you'd like to test: https://github.com/asg017/sqlite-vec/releases/download/v0.1.6/sqlite-vec-0.1.6-loadable-linux-aarch64.tar.gz
Please let me know if you have any issues with it, otherwise I'll close this issue for now.

thanks @asg017 for the update!
I'm trying to pull it in my Ruby project but it does not find it:
Fetching gem metadata from https://rubygems.org/.........
Could not find gems matching 'sqlite-vec' valid for all resolution platforms (arm64-darwin, x86_64-darwin, x86_64-linux, x86_64-linux-gnu, x86_64-linux-musl,
aarch64-linux, arm64-darwin-24) in rubygems repository https://rubygems.org/ or installed locally.

The source contains the following gems matching 'sqlite-vec':

... cut

  * sqlite-vec-0.1.6-arm64-darwin
  * sqlite-vec-0.1.6-arm64-linux
  * sqlite-vec-0.1.6-x86_64-darwin
  * sqlite-vec-0.1.6-x86_64-linux
  * sqlite-vec-0.1.6-x86_64-mingw32

It seems bundle expects to find aarch64-linux instead of arm64-linux.

Forgot to mention: as of v0.1.6 there are official Linux arm/aarch64 builds of sqlite-vec available.
Here's a direct link if you'd like to test: https://github.com/asg017/sqlite-vec/releases/download/v0.1.6/sqlite-vec-0.1.6-loadable-linux-aarch64.tar.gz
Please let me know if you have any issues with it, otherwise I'll close this issue for now.

if you do file vec0.so on that package ,
it returns
vec0.so: ELF 32-bit LSB shared object, ARM, EABI5 version 1 (SYSV), dynamically linked, BuildID[sha1]=126100873b8858013bd396b329eccfa35f27acf5, not stripped


New version: Google.CloudSDK version 493.0.0

Checklist for Pull Requests

[X] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[X] Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

What does this mean?


Error in installation

Hello :)
I am trying to implement MultiK on my dataset of 10k cells, however I am running into an issue of installation. All my packages and R is up to date. Below is the error I run into:
Error in utils::download.file(url, path, method = method, quiet = quiet,  :
download from 'https://api.github.com/repos/siyao-liu/MultiK/tarball/HEAD' failed
How should I resolve this?

@Lakshya3141 Hi! I'm not familiar with the error message you are getting. Maybe you can you try downloading the package from the github and install it locally from your computer?


Notebooks: Cannot "Save As..." notebook

Steps:

open a notebook
select File > Save As...

=> 🐛 nothing happens, I am not being asked for a path.

Looks like notebookEditorInput is missing to implement the saveAs method.

/duplicate of https://github.com/microsoft/vscode/issues/96482 . Thanks for reporting, it might require some API change, we will look into this in next Iteration.


Keep the indentation of JSON file

When the package.json is formatted, the indent with 2 spaces become a indent with 4 spaces.
It's caused by the json_encode() function.

Added by ef9e93183527d319a85b38d2b2e7f34b421e8720.


Validation status is not reset after form reset

https://github.com/morewings/react-forge-ui/assets/1158959/826eeb52-d510-4179-8ea0-cf54490e5c87
https://github.com/orgs/react-hook-form/discussions/2993

:tada: This issue has been resolved in version 0.11.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


393 - Place 'clear filter' labels at the top of the Data Choices menu

This PR also animates the 'clear filter' labels to fly from their associated UI components to their place among the other 'clear filter' labels. The aim here is to make it unmistakable that the orange labels emerge as a result of choosing filters, and that they move to the top of the Data Choices menu.
Addresses #393

@ptgott - that sounds pretty slick! I will be able to review tonight.

@ptgott
The animation definitely helps to clearly designate that the filters are a result of the user action. I'd be curious to hear what others think of this UI behavior - @ostermanj ? @ableo57 ? @thischrisoliver ? Here's a screenshot for easier viewing:

I'm also doing a review with a couple bugs I found plus some usability tweak suggestions

@NealHumphrey Thanks for your review! Here's a checklist summary of your comments. I'll start pushing commits as I work through them.

[ ] Make the gray 'clear all' pillbox be completely clickable with no 'x'.

[ ] Put an anchor div at the fly-to destination and have the 'label' be the actual pillbox. Or otherwise, avoid duplication during flying.

[ ] Animate the height transition of the #clear-pillbox-holder to avoid the sense of the data selection UI jumping.

[ ] Fix bug: When applying multiple filters, sometimes the filter doesn't place the correct pillbox, or removes the incorrect pillbox.

[ ] Fix bug: When adding/removing pillboxes, sometimes the code adds multiple copies of an active pillbox (not the one currently clicking). This is likely due ot re-adding pillboxes above the last one after removing some in the middle/end. Check the data binding between pillboxes and activeFilterIds.

[ ] Change the 'merge' call around line 74 to the statement, 'var allPills = newPills.merge(oldPills);'

[ ] Fix bug: When clicking 'clear filters', the function to recalculate filteredData 'is run like 25 times'. This might be down to the change in 'isTouched' and the code that calls it.

@ptgott this looks great! It looks like all the issues are resolved.
I did a couple small edits of animating the exit of the pillboxes with a fade-out to clean things up a bit. I tried to get the animate height transition to work as noted in the last bullet point but had some tricky issues - the method I mentioned works (setting max-height using d3, using a css transition on max-height to do the animation) but it was buggy to get the proper max-height to set using d3 because sometimes the pillboxes wrap and sometimes they are on new lines depending on their height. The fadeout helps with the jerkiness on the back end, so I say leave this alone for now and we can decide if we want to do it as a final cleanup step later down the line.


Response modifier

Proposed changes:

...

Status (please check what you already did):

[ ] added some tests for the functionality
[ ] updated the documentation
[ ] updated the changelog (please check changelog for instructions)
[ ] reformat files using black (please check Readme for instructions)

@amn41 is this still relevant?

this was just a bit of fun, which @tttthomasssss was going to play with at some point. Thomas, do you still think its interesting or shall we just close this?

Yes, lets close it for now.


Can't set buttons on pushed NavigationController

Function setRightButtons works only in TabBar components. If I try to push to new controller, buttons don't appear.
componentDidMount() {
    const Controllers = require('react-native-controllers');
    const navigationController = Controllers.NavigationControllerIOS("create_expense");
    navigationController.setRightButtons([{
        icon: require('image!save'),
        onPress: function() {
            // code
        }
    }]);
}

ok, here's what going on:
The native implementation of setRightButtons is to set the buttons on the navigation bar's topViewController.
I assume that you're calling setRightButtons immediately after the push and that the push has animation. What probably happens is that topViewController gets set natively to the new screen only after the animation is over. This means that you're actually setting the buttons on the screen that called the push, not on the screen that gets pushed.
The best practice in this case is not to use setRightButtons at all. To work around the race conditions in this case, we have the rightButtons parameter on navigationController.push(), see the readme here.
Only use setRightButtons if you need to change the buttons in runtime. For example, if you have a screen that has a disabled button that can turn enabled after the user does something, this would be the way to do it.
Theoretically we can fix the native implementation and cause setRightButtons to behave correctly in your case by saving a property with the currently animating view controller, but I don't think that's necessary. I think setting the button on the push is the correct way to go here.
If the thing that worries you is separation of concerns and having the button logic on the pushed screen, I recommend that you switch to react-native-navigation. This wrapper library is much simpler to use and actually handles the separation of concerns better by having buttons defined as a static variable on the pushed component. Take a look, it's awesome. Internally in Wix, we don't use react-native-controllers directly, only through the wrapper.


[BUG] cortextool alertmanager get does not output valid YAML

Steps to reproduce:
$ cortextool alertmanager get | yq .
Error: bad file '-': yaml: control characters are not allowed

Figured it out. the command outputs a "Template:" declaration which is empty, causing YAML syntax error. Will open a PR soon.
# example bad YAML
global:
  victorops_api_key: XXXXXXXXXXXXXXXX
  smtp_from<EMAIL_ADDRESS>  smtp_smarthost: smtprelay:2525
  smtp_require_tls: false

route:
  group_by: ["..."]
  group_wait: 30s
  group_interval: 5m
  repeat_interval: 3h
  receiver: my-receiver

  routes:
    ...
Templates:

This line needs to be executed if and only if there are more than 0 templates:
https://github.com/grafana/cortex-tools/blob/c88f1fa749cb1446a32b34c3217af4181c3687cc/pkg/printer/printer.go#L63

use --disable-color:
$ cortextool alertmanager get --disable-color | yq .
  # They are grouped by product and environment rather than cluster
  # and alertname.
  routes:
    - receiver: 'teams-webhook-test2'
      group_by: [product, environment]
      matchers:
        - team="frontend"
Templates:


[Discussion][Feature] Support for large message chunked transfer

Apache RocketMQ, like all messaging systems, imposes a size limit on each message sent to a broker. This prevents the load per message from exceeding the maxMessageSize set in the broker, which defaults to 4 MB.
However, in application scenarios such as image processing and audio processing, many users need RocketMQ clients to send large messages to brokers. You can achieve this by adjusting maxMessageSize. However, this approach can cause many problems. For example, if a client publishes a 100 MB message, and the broker allows storing the message to disk, the broker will spend a lot of IO resources processing the message. This will affect broker flushing performance and cause a backlog of messages.
Therefore, when RocketMQ sends large messages, in addition to increasing the value of maxMessageSize, we can also support the 'message chunking' function.
Through message chunking, producers can split large messages into multiple chunks according to maxMessageSize, and send each chunk to the broker as a normal message. The consumer will then assemble these chunked messages back into the original message.
For example:

Therefore, it's time to discuss whether 'message chunking' should be supported or not.

IMO, large message  is mostly not needed, In most cases prefer to upload to SSO first and send the data address through MQ.The benefit of this is that the MQ can send messages more efficiently and the MQ can store less large objects, which is mostly used for storage such as SSO.

IMO, large message is mostly not needed, In most cases prefer to upload to SSO first and send the data address through MQ.The benefit of this is that the MQ can send messages more efficiently and the MQ can store less large objects, which is mostly used for storage such as SSO.

I don't think it's a good solution to store the 'message itself' with the help of an external system:

In some scenarios (not audio and video), the size of the message body has exceeded 'maxMessageSize', and we cannot separate the message and store it in other systems. Although such scenarios are rare, it does not mean that there are no such scenarios.
The message system itself is a storage engine, and we don't want to use an additional external system to store messages.


SubmodelProvider provided idShortPath incorrect

According to the API docs, the path for invoking an operation is
/shells/{aasId}/aas/submodels/{submodelIdShort}/submodel/submodelElements/{idShortPathToOperation}/invoke
or for the async-variant
/shells/{aasId}/aas/submodels/{submodelIdShort}/submodel/submodelElements/{idShortPathToOperation}/invoke?async=true
In the oven example imagine it instanced like
/shells/eclipse.basyx.aas.oven/aas/submodels/Oven/submodel/submodelElements/activateOven/invoke
Expected: Here, the idShortPath should be activateOven.
Problem: The org.eclipse.basyx.submodel.restapi.SubmodelProvider class forwards incorrect idShortPaths of the submodel element to the enclosed ISubmodelAPI. In the example, instead of activateOven, it forwards /activateOven/invoke, not having removed the trailing /invoke.
In the async invoke case, it forwards /activateOven/, having removed the trailing invoke but not the trailing slash.
For other methods, those don't have a trailing path segment to be removed, but more precisely, for all methods, imho the leading slash should also be removed for consistency with the URL pattern, so it should be activateOven instead of /activateOven.
See https://github.com/eclipse-basyx/basyx-java-sdk/blob/754b1c11e9961822b6dad28b4837103a9cde745e/src/main/java/org/eclipse/basyx/submodel/restapi/SubmodelProvider.java#L253
It can be seen that in the async-branch, there is an instruction to remove the trailing part, which is missing in the else-branch for non-async invocations.

Many thanks for pointing this out. We will fix this soon.


Any solution for iPhone.

First of all, thank you @AlexxIT for your work. It works perfectly on my PC.
And I can only view it locally In iPhone.
But on the iPhone I can't find a way to do it, or hls, or anything. Any solution in sight.

I don't understand the question

@AlexxIT I can see the cameras locally, but I can't see them outdoors with 4G.
My installation is Ha in Proxmox.

#378

Thank you, I will consult both to see if I can get anything.
Thanks for your time.

Latest integration version should support HLS technology. It's awful, but should works on iPhones.

Latest integration version should support HLS technology. It's awful, but should works on iPhones.

I added to my card:
mode: hls
And I got no positive result. It didn't even play locally.
No mode: played in local RTC.

It doesn't work on bad cameras.

It doesn't work on bad cameras.

VStarcam. I'm sure that's it.

You can try to setup ffmpeg source for your camera in the go2rtc config.


parameter auto complete with required-properties is shown in reverse order than the custom type or object

main.bicep
param DeploymentHub DeploymentHubType

type DeploymentHubType = {
  @description('The object to define the Hub Deployment')
  prefix: string
  orgName: string
  appName: string
}

produces

@brwilkinson It is not reversed. It is alphabetical. I have started discussion about this a couple of days ago #10883

Oh great, thank you.. I'll close this as dupe.

Oh actually it's slightly different. It was actually review of your issue, that was the prompt to open this one... Anyway, I will review and update notes accordingly.

The sort is alphabetical in params files, too. I changed a property name to alter the alphabetical ordering (orgName => xorgName) and saw that the snippet was updated to reflect an alpha sort:

Thanks @jeskew @slavizh I will merge this info into #10883 and close this out as duplicate.


settings-ui: Fix tab-switcher misalignment for iPad size devices.

Fixes: #14913.
Testing: manually
I avoided the change the name of Subscription | All streams to Mine | All as mentioned in the issue

Merged as a5f0379e0f0957905907756d585b87476e993877, using a cleaner way of doing the media query that we should probably do more of.
I think there may be a better responsive arrangement, BTW; perhaps we should just drop from 2-column to 1-column a bit earlier (so that there's never the state with just the + on a line by itself).  What do you think?

Yeah that will the better way
For mobile size devices, we do drop 2 columns to 1
For small size iPad, we can definitely do the same


TypeConverter.ResolveActualType returns null

The TypeConverter.ResolveActualType method assumes that a generic type is always a nullable value type. We have several tables with some columns containing JSON data (objects, arrays). As of now, we can't declare properties of type IList<Type> because it fails with NRE.

I don't see where the NRE is coming from, can you give me an example?

When the type is typeof(IList<string>), the line 172 in TypeConverter.cs will set actualType to null. It causes NRE somewhere the returned value is used.

Which method on the session are you calling?

session.InsertAsync(instance)... here is the stack trace:
   at MicroLite.Mapping.ILGeneratorExtensions.EmitUnboxOrCast(ILGenerator ilGenerator, Type type)
   at MicroLite.Mapping.DelegateFactory.CreateInstanceFactory(IObjectInfo objectInfo)
   at MicroLite.Mapping.PocoObjectInfo..ctor(Type forType, TableInfo tableInfo)
   at MicroLite.Mapping.ConventionMappingConvention.CreateObjectInfo(Type forType)
   at MicroLite.Mapping.ObjectInfo.For(Type forType)
   at MicroLite.Core.AsyncSession.<InsertAsync>d__13.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.GetResult()

What is 'instance'?

public class UserRow
{
    public long Id { get; set; }
    public long AccountId { get; set; }
    public string FullName { get; set; }
    public string DisplayName { get; set; }
    public string Email { get; set; }
    public IList<ContactDetail> ContactDetails { get; set; }
    public DateTime DateCreated { get; set; }
    public bool IsBlocked { get; set; }
}

The reason this isn't working is that MicroLite doesn't support collection properties as it doesn't do lazy loading, the idea with MicroLite is to use the include API to retrieve associated data so that you control exactly what data is loaded and when it is loaded.

Well, it does. It supports collection properties. You only need to create your own ITypeConverter. When I change the property to be of non-generic collection type, ti works pretty well.
public ContactDetails ContactDetails { get; set; }

The whole collection is mapped to one column containing JSON data.

OK, what I meant was it doesn't natively support collections, you didn't mention you were using a custom ITypeConverter.

Ok :) And I think it should work the same way with generic collection types. But it doesn't because of the NRE.

Yes, the code you mentioned is there to allow MicroLite to work with nullable struct properties (e.g. public int? Numner { get; set; } but it's also matching IList<string> in your case because it is a generic type however the call to Nullable.GetUnderlyingType(type) is where the null comes from because the IList<T> isn't a Nullable<T>.
Do you want to have a go at fixing it and sending a pull request?

I want, but I haven't a chance yet to figure out what is wrong with StyleCop. https://github.com/TrevorPilley/MicroLite/issues/395

No worries, I haven't upgraded to VS 2015 yet so when I do I'll see if theres something odd going on with StyleCop

I've just pushed a beta of MicroLite 6.2.5 to nuget - let me know if that fixes your problem and I'll push it as a full release.

Thank you, it's fixed.

Excellent, re-publish 6.2.5 as a non beta package later


Process list and Insights pages need some explanatory text

Dev Home version
No response
Windows build number
No response
Other software
No response
Steps to reproduce the bug
This is arguably a feature request, but still bug-sized. When the user goes to the Process List page, there's no indication that selecting an item from the list will make PI target that app. This is particularly a problem if they're already targeting a different app, but say they just click in the list as part of scrolling.
There's also no description of the filter dropdown.
Expected result
Tooltips have their own problems, so ideally, we should just have a short description at the top of the list, something like "Select a process from the list to direct Project Ironsides to start gathering data about that process. You can type in a process name at the top to filter the list. By default, the list excludes certain common processes. You can also select items in the filter drop-down to include any of these.".
Actual result
No description.
Included System Information
No response
Included Extensions Information
No response

Duped to 3054


[MPWG Release] Design Card for Social Media, Blog announcement and Website

At the moment, microprofile.io still mentions MP 4.0. This needs to be updated to point to MP 4.1.

[x] On the top card, update MicroProfile 4.0 to 4.1
[x] On the top card, More Information points to https://github.com/eclipse/microprofile/releases/tag/4.1
[x] On the 2nd card, Update MicroProfile 4.0 to 4.1. Health 3.0 will be updated to Health 3.1 while the others are unchanged.
[x] Move the MicroProfile 4.1 presentation from this location  to the latest Release Prsentation  and then move MP 4.0 deck to the archive dir or 2020 release folder

I have updated slide 7 (MicroProfile 4.1 Released!) in the presentation to include the lifted compatibility requirement for CCR. Can you please verify if this text is correct?

Thank you @ederks85 ! Looks good!

This one depends on #547 #548

@Emily-Jiang @ederks85 @aeiras
Should these updates be done now? or are we waiting for something?

The updates mentioned in this issue can be done now.
Additionally:
#547 requires a change in the Duke logo used in the presentation
#548 has been reviewed. You only need to accept the suggestions

MP Presentation has been relocated (checkbox 4 in the issue summary). Here is the 4.1 presentation

@Emily-Jiang @aeiras @edbratt @ederks85
What we are missing to get this released today:

update block image "3.1 health"  -  DONE  (Not sure we can add "Jakarta Annotation" today- also it will make a odd structure the blocks )
Update drive with 4.1 pres (in latest pres folder)
4.1 Short descriptions for home page
Tweet message for the release. (image I can do)
Blog on WP

All that is left is moving the 4.1 presentation to the correct "latest version folder" on the drive and move the 4.0 presentation to the old version folder.

It appears the 4.1 presentation is in the latest version folder here and it appears the 4.0 presentation is in the 2020 Releases folder here
It appears the Tweet with the designed social card about the announcement has been published here:
And that the Microprofile.io homepage has been updated and the blog announcement published.

Debbie,
Good call out Debbie. There is an ongoing discussion where I explained to Emily that we won't be moving Releases content to the EF infra drive.
The work was completed and stays under the MP Community drive that will continue to provide and welcomed contributors from all walks in our ecosystem and not those from the Steering Cmtee only.
This is nothing new on the process, https://drive.google.com/drive/u/0/folders/1pyUT50_taXQe-8qKAzBZcwJDb5s6TedR.  I will delete the other content to avoid duplications. Thanks for reminder.
--
The only thing missing is the finalizing of the word document review to inserted into the wiki. This ticket and others related to tasks done can be closed. I will go to it now and help on that as well. Ñ)


Added default color correction profile

Added new 'default' color correction profile (and associated tests).  These changes add the profile to the machine, but do not automatically associate it with any led devices. It can be overridden by the user in their machine config.
This addresses issue #370.
You can decide whether or not to pull this into 0.30 (even though it is a new feature, although very limited in scope) or wait for 0.31.

Coverage decreased (-0.04%) to 89.207% when pulling 81b3d9998bee637ed59f5007fa069b2364be1d5e on default-color-correction-profile into 030996a0373a1111352eb623ebfaf7060c2a3cf1 on dev.

Coverage decreased (-0.07%) to 89.18% when pulling f029e20cb14cfa3973d4133367f87f2ec7f29d0a on default-color-correction-profile into 030996a0373a1111352eb623ebfaf7060c2a3cf1 on dev.

I recommend including this in 0.30 as there is some important new validation for LED color correction profiles (exception is now thrown on a bad color correction profile value whereas before the profile was simply skipped and a warning was logged).


Compare option is not available in locale other than english.From wishlist also add to compare option is not available.

Bug report
Title
Compare option is not available in locale other than english.
Issue Description
Compare option is not available in locale other than english.
Preconditions
1. framework Version.-master

Steps to reproduce
1. step1- Change locale to any other than english
2. step2- Check compare section in header

Expected result
Compare option should be available
Actual result

@jyoti-webkul
make sure the compare feature should be available for all locale in the configuration else it will not show in Interface.


Unable to reset filter; stuck on only seeing installed platforms.

Describe the bug
A clear and concise description of what the bug is.
The UI doesn't allow me to disable the filter that only shows me installed XCode platforms. It seems that the filter icons have disappeared, and I can't get it to revert to its original state through settings or moving the application to trash and reopening the original downloaded file from the zip folder.
At this point, I've installed v1.10, so I'm still able to use the functionality of XCodes that I was looking for. I appreciate any help to restore the UI to the default state on the latest version!
To Reproduce
Steps to reproduce the behavior:
I'm not exactly sure; I got started with this program, messed around with the filters a bit, and ended up stuck on only being able to access my currently installed XCode versions.
Expected behavior
A clear and concise description of what you expected to happen.
I expect for icons to have appeared allowing me to disable the filter. If there is a way to revert back to the original view, it is not clear to me how I am supposed to do so.
Screenshots
If applicable, add screenshots to help explain your problem.

Version

OS: Sonoma 14.4.1
Xcodes: v2.1.2

I believe MacOS will shove those in the >>  sometimes when it thinks it can't draw it on the nav bar

You are correct @MattKiazyk.
They now appear in that "overflow" menu for me.

I made the same mistake, and the options in the overflow menu are greyed out and can not be unselected.

As a hack if you are stuck like I was: defaults write com.xcodesorg.xcodesapp xcodeListCategory "" will fix you.


[frontend] Revamp data providers to project services

introduce provider services for:

graph table
vcs root combo box
branch layout writer

Minor comments, almost ready to approve :+1:


Create Project Structure.md

Closes  #133
I have added the Project Structure.md a seperate file . This guides contributors on where specific files and folders are located and what they contain.
Note: Please make sure to add necessary labels

Ok I will make changes

@ombhojane
I have made changes once check
tell me if any further changes needed