Increase connection timeout for imports

TL;DR
Set the response timeout from 1s to 30s, because 1s is much too optimistic.
Why
From time to time I have the problem luck of being with a rather modest internet connection (~376Kb/s) and I noticed that even running something simple like:
let not = https://ipfs.io/ipfs/QmcTbCdS21pCxXysTzEiucDuwwLWbLUWNSKwkJVfwpy2zK/Prelude/Bool/not
in  not True

via dhall fails with ConnectionTimeout.  I saw that the connection response timeout is set to 1 second, which is much too short for a bad internet connection.
How
To address this, I raised the time limit from 1 second to 30 seconds, because that is also the time that is used in the default managerResponseTimeout (https://hackage.haskell.org/package/http-client-<IP_ADDRESS>/docs/src/Network-HTTP-Client-Types.html#ManagerSettings).
With that setting I can finally use dhall even when I don't have a fast internet connection available. :D

Yeah, great idea!  Sorry about picking such a strict default initially


Update sp-executesql-transact-sql.md

There are way more than 12 statements generated. You get one statement for each year and month combination for each language where the first 3 characters of the month name are different.
Personally, I think you would be better off using a table naming scheme that is not locale dependent like 'Sales_' +
cast(datepart(yy, @PrmOrderDate) as char(4)) +
'_' +
cast(datepart(mm, @PrmOrderDate) as char(2))
To see the table names that you get now, run the following ...
DECLARE @PrmOrderDate DATETIME;
SET LANGUAGE N'us_english';
SET @PrmOrderDate = '01/31/2021'; -- set date here because the date literal format also depends on LANGUAGE SELECT SUBSTRING(DATENAME(mm, @PrmOrderDate), 1, 3) + CAST(DATEPART(yy, @PrmOrderDate) AS CHAR(4)) + 'Sales'; SET LANGUAGE N'Español';
SELECT SUBSTRING(DATENAME(mm, @PrmOrderDate), 1, 3) + CAST(DATEPART(yy, @PrmOrderDate) AS CHAR(4)) + 'Sales'; SET LANGUAGE N'ελληνικά';
SELECT SUBSTRING(DATENAME(mm, @PrmOrderDate), 1, 3) + CAST(DATEPART(yy, @PrmOrderDate) AS CHAR(4)) + 'Sales';

@markingmyname
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

After discussion with the product group, we are revising that paragraph further in an internal PR, which will be published in the next day or so. Thank you for your feedback.


remove glusterfs references from the docs

we have removed glusterfs in-tree driver in this release (v1.26) via https://github.com/kubernetes/kubernetes/pull/112749 .
This PR remove the references from the docs.

Cc @tengqm @sftim @xing-yang

/lgtm

Thanks @humblec
I'd like to include glusterFS somewhere in the volumes page, so that if people search the docs, they find a match. A single sentence explaining that Kubernetes v1.25 was the last minor release with in-tree support is enough.

@sftim indeed thats a good thought. I have added a note in volumes guide about the same.

Please don't update any of the pages with auto-generated: true in their front matter. We'll cover those separately.

sure

One more thing. Sorry.

nw @sftim :+1: , I committed those changes. ptal. thanks.

Looks like this is almost ready, @humblec this needs a rebase & another checkin by @sftim if you find the time :)

sure @leonardpahlke . let me do one more rebase :) ..

K8s v1.26 is scheduled for less than a week, so we need to get this done as soon as possible! pray cc @krol3

@leonardpahlke @krol3 the PR has been rebased. we should be good to go . thanks !

@humblec Looks like the gate is not happy with the new commit? Could you pls try it again?

@humblec Looks like the gate is not happy with the new commit? Could you pls try it again?

@tengqm I just rebased the PR and fixed the conflicts. do we have pointer on what exactly causing the failure?

@humblec It smells like a transient netlify build timeout issue.
You may want to recommit this PR to trigger a new gate build.

@humblec It smells like a transient netlify build timeout issue. You may want to recommit this PR to trigger a new gate build.

Cool. I have retriggered the tests @tengqm .. Thanks :+1:

/lgtm

/label tide/merge-method-squash

/approve


Point of Order button

Question:
How should Openslides behave if a Users is allowed to put point of orders for other people on the LoS and is on the LoS with a point of order
Reproduction until issue occurs:

login as an admin
navigate to settings > LoS
activate 'Enable point of orders for others' and 'Enable point of order'
navigate to a LoS
press point of order and ok
you put yourself on the LoS with a Point of order
now the button changed to 'Withdraw Point of order'
And you have no possibility to add another User with a Point of order

What is wanted?

should a new button appear next to 'withdraw [...]' which is the same a the point of order button now?
should the withdraw button be changed (in what way) to have the same amount of buttons but more options?
should the status quo be kept?

obsolete with https://github.com/OpenSlides/openslides-client/pull/3348


added PySide6 shiboken6 support - Pt 2

Now passing new testing framework!

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.2 out of 3 committers have signed the CLA.:white_check_mark: masqu3rad3:white_check_mark: zoshua:x: joshochoajoshochoa seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Great news, thanks. So I think the reason tests don't run here is because of that CLA. It looks like you are both @zoshua and @joshochoa except @joshochoa is not a GitHub user.
Would it be possible to squash your commits, such that they are all committed under the same user?
From there, what's missing is tests actually running for PySide6. Currently, we only test whether the changes pass for Qt 4 and 5. For Qt 6, I expect we'll need to update the Dockerfile to include PySide6.

This piqued my interest, and re-affirmed why I avoid merge commits like the plague in feature branches, they make updating the history of a feature branch branch harder. The merging in of the newer commits in mottosso:master made simple squashing difficult.
I tried a few things but I had the most success with generating a patch file based on this. It will squash all commits into a single new commit on top of the latest commit in mottosso:master.
I recommend making these changes on your working copies master branch so you don't need to create a new pull request.

Make a backup. I normally create a branch or tag called temp. When done I know to remove it. If something goes wrong, I use git reset --hard ... to reset the broken branch back to the original value.
Stash any un-committed changes you may have in your working copy.
 Currently 2c0f4dc596204670f4597b841821cff8a171c3a2, I'll refer to it as <qt_py_hash>.
With your zoshua:master branch checked out generate a patch file with using the commit hash": git diff <qt_py_hash> HEAD -- > updates.dif. This shows the sum of all changes between your branch and Qt.py, ie what we see in the changes tab of this MR.
Now we want to reset your working copies zoshua:master branch to the current state of the Qt.py master branch. git reset --hard <qt_py_hash> again using the qt_py_hash. At this point your branch is exactly the same as mottosso:master.
If you want you can run the git apply --stat updates.diff and git apply --check updates.diff mentioned in the stack overflow article, but they are just checking if the patch will apply.
I couldn't get the git am command to work due to a empty ident name, but that seems to be actually committing the changes, you can just run git apply updates.diff. This apples your code changes as working copy changes.
Commit the changes as normal using the correct username.
At this point your zoshua:master should show the changes you want to make as a single commit. You will need to force push the changes to update this Pull Request's code.
Once you are happy with all of this you can remove your backup branch/tag in your working copy.

Warning:

As this involves rebasing and force pushing, I'll add the obligatory warning that you should (almost)never rewrite the history of a master branch other people are using. As zoshua:master is your fork and I'm guessing you are the only user of I'm not worried, but at this point if mottosso were to do stuff like this to mottosso:master it would annoy every other user was working on a fork of it. For future merge requests I would recommend making a feature branch not directly editing zoshua:master.

@MHendricks Thank you so much for this write up!
I totally goofed on not working from a feature branch in the first place.
I will run through the process asap and let you know if I hit any snags.

Closing this one out as we moved the submission to PR 394
Thanks to everyone for the contribution and guidance.


last search pattern concealed after invoking matchadd() to conceal any regex

vim --version: 8.0 patches up to 1173
Operating system: Linux, Ubuntu 16.04
Terminal name: xfce4-terminal
$TERM: xterm-256color

Consider this minimal /tmp/vimrc:
let @/ = '\cconceal'
set hlsearch
setlocal concealcursor=nc conceallevel=1
call matchadd('Conceal', '.$', 1, -1, {'conceal': '_'})

After starting Vim like this:
vim -Nu /tmp/vimrc /tmp/vimrc

The last character on every line is concealed, like expected, but the word conceal is also concealed. I don't understand why, as the regex passed to matchadd() is .$. It should only affect the last character on every line, not the last search pattern.
I found this issue ( highlighting conflicts when using conceal #229 ) which is similar but not exactly the same. The issue #229 seems to be about a conflict between different syntax highlighting mechanisms ('hlsearch', 'cursorline', concealed characters).
But this issue is about the last search pattern which shouldn't be concealed at all.
It only happens when 'hlsearch' is enabled.
The only solution which I found for the moment is to lower the priority of the match to 0, so that its priority  is not greater than the one created by 'hlsearch':
    call matchadd('Conceal', '.$', 0, -1, {'conceal': '_'})

Lowering the priority of the match seems a good enough solution for the moment. So, I close the issue.

Another hack work around is to use the regular expressions of the 2nd parameter passed to matchadd to have it assert: "highlight only the first instance in the string" and then make a secondary matchadd to assert "highlight only the last instance".
Then the first match color won't be obscured by the second because they are different instances of a match.  If your regex fu is strong, there are wildcards that can enumerate the first, second, third, fourth, and nth instances of your match using a for loop and making matchadds for each, the drawback is this boggs down the snappiness of the refresh after some 30 to 60 instances.


Graphical artifacts when scrolling on windows

when I scroll, the texts become all messed up

this only happens with v0.7.0
previous versions are fine, so I suspect it has something to do with the smooth scrolling.

Are you running with the multigrid enabled?

no, I'm running it without enabling multigrid.
when I run it with --multiGrid I get #483

Can you record a gif (I find https://www.screentogif.com/ to be very easy) of both multigrid and non multigrid? That might help me debug what is going on. Is this on a high dpi monitor? What about when you run from current main? https://github.com/Kethku/neovide/actions/runs/746999264
It seems like the background surfaces are using the wrong resolution and are being scaled up.

okey here is a screen record

also i noticed that the blurred floating windows are not working properly.

Collapsing under https://github.com/Kethku/neovide/issues/537


In pipeline, accept a JSON file as a data source

Currently accepts CSV. JSON data sources are useful too.

We'll use ijson

Already in https://github.com/frictionlessdata/tabulator-py will get here when we replace the custom iterator with Tabulator.


Making Form for All About Programming

@its-puneet You'll be able to share all the changes you require by adding up a comment in this issue and I'll be adding them up in the next commits.

Will reopen this issue if some changes are done in the form


pythonPackages.compiledb: init at 0.10.1 + dependencies

Motivation for this change
I need compiledb to run some proprietary software
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @

@costrouc thanks for the review :)

Nix review passes. Would still like to see that package names alphabetized in python-packages.nix.
Result of nix-review pr 76654 1

9 package were built:

python27Packages.bashlex
python27Packages.compiledb
python27Packages.shutilwhich
python37Packages.bashlex
python37Packages.compiledb
python37Packages.shutilwhich
python38Packages.bashlex
python38Packages.compiledb
python38Packages.shutilwhich

Neat, thanks for the PR!
Kicking the tires a bit, noticed something that seems undesired:
$ /nix/store/f0191y6jz8zqcx119yvy9xs2by9nd8xj-python3.7-compiledb-0.10.1/bin/compiledb --help
Unable to create '/nix/store/3isgyj3nyp7zjsp4i6np1q1fxqyrsw49-python3.7-bashlex-0.14/lib/python3.7/site-packages/bashlex/parsetab.py'
[Errno 30] Read-only file system: '/nix/store/3isgyj3nyp7zjsp4i6np1q1fxqyrsw49-python3.7-bashlex-0.14/lib/python3.7/site-packages/bashlex/parsetab.py'

Is this a problem, can it be fixed?

@dtzWill it's not a huge problem but there's definitely room for improvement here. I'll fix it

@dtzWill the warning shouldn't be there anymore, thanks for the report :)

Would still like to see that package names alphabetized in python-packages.nix. Otherwise LGTM.

Done, thanks!

@GrahamcOfBorg build python27Packages.bashlex python27Packages.compiledb python27Packages.shutilwhich python37Packages.bashlex python37Packages.compiledb python37Packages.shutilwhich python38Packages.bashlex python38Packages.compiledb python38Packages.shutilwhich


GlobalAttention doesn't use bias passed as input

GlobalAttention infer the bias from q_mask. the bias passed from input is not used.

Hi. This argument was intentionally not used and has been removed in 16bab4d331ef035d5cbbf5f54d2dc48b523307da so as to not be misleading.


fix: broken pod-install command

Description
This PR comes as a fix to broken pod update boost command. This is coming from issues related JFrog. I did the workaround suggested by the community while we're waiting for this to get solved in RN.

PR Checklist

[x] I have tested my changes on iOS and Android.
[x] I hid my changes behind a feature flag, or they don't need one.
[ ] I have included screenshots or videos, or I have not changed the UI.
[ ] I have added tests, or my changes don't require any.
[ ] I added an app state migration, or my changes do not require one.
[ ] I have documented any follow-up work that this PR will require, or it does not require any.
[ ] I have added a changelog entry below, or my changes do not require one.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

Patch boost.podspec - mounir

Need help with something? Have a look at our docs, or get in touch with us.

This PR contains the following changes:

Dev changes (Patch boost.podspec - mounir)

  Generated by :no_entry_sign: dangerJS against b5dc0ecb7cb7b8f6b3c52611eecc58e3a74bc3e0


Numba 0.46.0 depends on pkg_resources but does not declare a setuptools dependency

Results in this CI failure on a system without any setuptools installation:
https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=85208
Stack trace is:
    import numba
  File "/home/conda/feedstock_root/build_artifacts/clifford_1572349688184/.../lib/python3.7/site-packages/numba/__init__.py", line 30, in <module>
    from .decorators import (autojit, cfunc, generated_jit, jit, njit, stencil,
  File "/home/conda/feedstock_root/build_artifacts/clifford_1572349688184/.../lib/python3.7/site-packages/numba/decorators.py", line 14, in <module>
    from .targets import registry
  File "/home/conda/feedstock_root/build_artifacts/clifford_1572349688184/.../lib/python3.7/site-packages/numba/targets/registry.py", line 5, in <module>
    from . import cpu
  File "/home/conda/feedstock_root/build_artifacts/clifford_1572349688184/.../lib/python3.7/site-packages/numba/targets/cpu.py", line 20, in <module>
    import numba.entrypoints
  File "/home/conda/feedstock_root/build_artifacts/clifford_1572349688184/.../lib/python3.7/site-packages/numba/entrypoints.py", line 3, in <module>
    from pkg_resources import iter_entry_points
ModuleNotFoundError: No module named 'pkg_resources'

cc @hugohadfield

Good catch.  We should fix this in our planned patch release.

Note this is already fixed in numba 0.46.0_1 on conda-forge

Also have this problem, for install used pip
python main.py
Traceback (most recent call last):
  File "E:\servers\programming\pyton\test\main.py", line 1, in <module>
    from numba import njit
  File "C:\Users\perch\AppData\Local\Programs\Python\Python39\lib\site-packages\numba\__init__.py", line 39, in <module>
    from numba.core.decorators import (cfunc, generated_jit, jit, njit, stencil,
  File "C:\Users\perch\AppData\Local\Programs\Python\Python39\lib\site-packages\numba\core\decorators.py", line 12, in <module>
    from numba.stencils.stencil import stencil
  File "C:\Users\perch\AppData\Local\Programs\Python\Python39\lib\site-packages\numba\stencils\stencil.py", line 11, in <module>
    from numba.core import types, typing, utils, ir, config, ir_utils, registry
  File "C:\Users\perch\AppData\Local\Programs\Python\Python39\lib\site-packages\numba\core\registry.py", line 4, in <module>
    from numba.core import utils, typing, dispatcher, cpu
  File "C:\Users\perch\AppData\Local\Programs\Python\Python39\lib\site-packages\numba\core\dispatcher.py", line 16, in <module>
    from numba.core import utils, types, errors, typing, serialize, config, compiler, sigutils
  File "C:\Users\perch\AppData\Local\Programs\Python\Python39\lib\site-packages\numba\core\compiler.py", line 6, in <module>
    from numba.core import (utils, errors, typing, interpreter, bytecode, postproc,
  File "C:\Users\perch\AppData\Local\Programs\Python\Python39\lib\site-packages\numba\core\cpu.py", line 16, in <module>
    import numba.core.entrypoints
  File "C:\Users\perch\AppData\Local\Programs\Python\Python39\lib\site-packages\numba\core\entrypoints.py", line 4, in <module>
    from pkg_resources import iter_entry_points
ModuleNotFoundError: No module named 'pkg_resources'```

@PerchunPak thank you for adding this to the Numba issue tracker. Since you are commenting on an older issue here, it is likely that you have encountered a new issue that shows the same symptoms of the older issue. My recommendation to resolve this would be to:
a) Open a new issue
b) State which operating system and version you are working on
c) Submit a full trace-back of the installation process (including the command your typed and any and all output)
This should help us get to the bottom of your issue. Thank you kindly in advance for your cooperation.


Adding School Email Address to list of domains

s.hakmistelbach.txt

The domain students AND teachers use at our school for „official“ emails is      s.hakmistelbach.ac.at
This email-adress is used e.g. for Office 365 school-accounts.
Our official website url ist hakmistelbach.ac.at   WITHOUT the „s.“

@elisabettina Your request was answered by email, thank you!

@Aryan-401 Thank you for changes to #8521. I am closing #8521 as a duplicate to this request. However, we still need some more info from you. Please provide us with:

the school official website URL, if different from the domain you are submitting
the school physical address (which city and country the school is located in)
an URL of a page on the official website where an IT-related long-term (>1 year) course is offered by the school
an URL of a page or another proof (.pdf or a screenshot are OK) showing that the school recognizes the domain which you are submitting as an official email domain for the students.

Thank you.

Hi Philip,
here are the requested informations:
www.hakmistelbach.ac.at http://www.hakmistelbach.ac.at
Brennerweg 8
2130 Mistelbach
Austria
http://www.hakmistelbach.ac.at/ausbildung/hak_digbiz.asp            (5 years curriculum for HAK Digital Business = IT curriculum)
Screenshot of Microsoft 365 Admin Center where you can find some students and me with email-domain   s.hakmistelbach.ac.at
Thanks an best regards,
Bettina Haushofer
Von: Philip Torchinsky<EMAIL_ADDRESS>Gesendet: Montag, 22. Juni 2020 13:49
An: JetBrains/swot<EMAIL_ADDRESS>Cc: elisabettina<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS><EMAIL_ADDRESS>Betreff: Re: [JetBrains/swot] Adding School Email Address to list of domains (#8531)
@Aryan-401 https://github.com/Aryan-401  Thank you for changes to #8521 https://github.com/JetBrains/swot/pull/8521 . I am closing #8521 https://github.com/JetBrains/swot/pull/8521  as a duplicate to this request. However, we still need some more info from you. Please provide us with:

the school official website URL, if different from the domain you are submitting
the school physical address (which city and country the school is located in)
an URL of a page on the official website where an IT-related long-term (>1 year) course is offered by the school
an URL of a page or another proof (.pdf or a screenshot are OK) showing that the school recognizes the domain which you are submitting as an official email domain for the students.

Thank you.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/JetBrains/swot/pull/8531#issuecomment-647466663 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AEGGCPWOLTLCFR3LEV7GEALRX5AJTANCNFSM4OD2U24Q .  https://github.com/notifications/beacon/AEGGCPTHKUSF7X73M3IKMCTRX5AJTA5CNFSM4OD2U242YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOE2LY5JY.gif

Oh hey, Philip I just saw this mail after the closed case so sorry for the
reply on that email.
Anyway, here the details you requested:
School Website:
http://www.themillenniumschools.com/tmsnoida41/
School Physical Address:
D-108, Aghapur, Sector 41,
Noida, Uttar Pradesh
110032
URL of IT curriculum: Hey I couldn't find one of the School website, but
our school is a recognized CBSE School and provides all subjects which the
board (CBSE) allows. One of which is Informatics Practices (CBSE Subject
Code:#065) and have provided its Curriculum -
http://cbseacademic.nic.in/web_material/CurriculumMain21/SrSecondary/Informatics_Practices_Sr.Sec_2020-21.pdf
(This is only for Grade 11 and 12)
Proof of School recognition of Email domain:
Have attached two images, one is the Proof that me and my peers also use
the domain, and the second screenshot showing that our Senior Coordinator
uses the same domain.
Note: All the names without a title below them are students, also PGT
stands for Post Graduate Teacher.
I hope this helps!
-Aryan

@Aryan-401 Thank you for the details! Could you please attach the images directly via GitHub web interface, because GitHub strips it out if you send them by e-mail? Thank you!

--- Copied text from earlier e-mail ---
Oh hey, Philip I just saw this mail after the closed case so sorry for the reply on that email.
Anyway, here the details you requested:
School Website:
http://www.themillenniumschools.com/tmsnoida41/
School Physical Address:
D-108, Aghapur, Sector 41,
Noida, Uttar Pradesh, India
110032
URL of IT curriculum: Hey I couldn't find one of the School website, but our school is a recognized CBSE School and provides all subjects which the board (CBSE) allows. One of which is Informatics Practices (CBSE Subject Code:#065) and have provided its Curriculum -  http://cbseacademic.nic.in/web_material/CurriculumMain21/SrSecondary/Informatics_Practices_Sr.Sec_2020-21.pdf (This is only for Grade 11 and 12)
Proof of School recognition of Email domain:
Have attached two images, one is the Proof that me and my peers also use the domain, and the second screenshot showing that our Senior Coordinator uses the same domain.
Note: All the names without a title below them are students, also PGT stands for Post Graduate Teacher.
Hope this works for you
-Aryan

Hey Philip,
Just wondering about the progress since no emails during thr lat 24 hours.
Has there been any issue?

Aryan Garg

On Mon, Jun 22, 2020 at 7:41 PM Philip Torchinsky<EMAIL_ADDRESS>wrote:

@Aryan-401 https://github.com/Aryan-401 Thank you for the details!
Could you please attach the images directly via GitHub web interface,
because GitHub strips it out if you send them by e-mail? Thank you!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/JetBrains/swot/pull/8531#issuecomment-647544252, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AN6DXD4NAYLE3747UD6YAT3RX5Q7JANCNFSM4OD2U24Q
.

@Aryan-401 Thank you for the clarification. Pull request merged.
FTR: additional proof of the domain ownership and authenticity of the proofs above is at http://www.themillenniumschools.com/tmsnoida41/news/consilium-mun-2017


Grammatical issue

This sentence seems to be missing a portion around "to for the"
" You can update data in the current table from the history table to for the "oops" scenario. "

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

ID: a6bd9289-f5fc-c084-fac8-92024f35f9c5
Version Independent ID: 47bc2199-497a-3421-b0f9-ca9afa67e0a3
Content: Modifying Data in a System-Versioned Temporal Table - SQL Server
Content Source: docs/relational-databases/tables/modifying-data-in-a-system-versioned-temporal-table.md
Product: sql
Technology: table-view-index
GitHub Login: @CarlRabeler
Microsoft Alias: carlrab

@jons-aura -- thank you for your feedback. I am looking into this article.

@jons-aura @WilliamAntonRohm I concur - please fix grammatical error

@CarlRabeler -- Carl, please review private PR 15237.


API data gone?

It seems all the data is deleted from the API? When I go to "http://pokeapi.co/api/v2/pokemon/" it says:
{"count":0,"next":null,"previous":null,"results":[]}
EDIT: Seems they're updating the database and I quote: "Give it a few hours". :)

@waseland should be back in a bit. sorry :(


Add trusting a CA bundle to net-istio

We need to add
https://github.com/knative/serving/issues/14609 for net-istio as well.
Related to https://github.com/knative/serving/issues/15276

/assign

After some discussions it seems that this is unnecessary. Trust-bundles are only relevant for system-internal-tls, which makes not a lot of sense for net-istio in the first place. Istio has this built in using the istio-proxies while forming a mesh. So to summarize:

net-istio supports cluster-local-domain-tls which makes it capable of an ingress gateway with TLS mode: simple
net-istio supports external-domain-tls (already available for a long time)
net-istio will not support system-internal-tls as users can just enable mesh integration to have connections between Ingress-Gateway <> Activator <> and Queue Proxies encrypted

net-istio will not support system-internal-tls as users can just enable mesh integration to have connections between Ingress-Gateway <> Activator <> and Queue Proxies encrypted

Should we rollback the feature added in https://github.com/knative-extensions/net-istio/pull/1085 ?

Should we rollback the feature added in https://github.com/knative-extensions/net-istio/pull/1085 ?

Yeah I think that makes sense. I'll do a PR for that.


[libpqxx] update to 7.2.1

Fix #14908
Update libpqxx to the latest version 7.2.1
Note: No feature need to test

Thanks for your contribution!


Check box to toggle persistent link is not usable

The check box that allows you to switch your share between persistent and non-persistent only shows for a brief moment during the share flow (after clicking "Share", before the link is ready) and it cannot actually be used. It doesn't repro 100% of the time, but it was happening for me more often than not:

The checkbox is intentionally omitted the first time you share a given project while you're logged in (notice the wording being update existing share link), and only shows up when sharing a project for a second + time. I don't really remember why that decision was made / would personally prefer if it gave an option for a non-persistent link even if it defaulted to checked, though

Ah, yup. That's it. Confused me quite a bit :) Personally I think we should just show it, but maybe change the wording if needed.

This is per design. However will watch on this to see if there is any more reports.


Issue if Drawer has mdl-layout--small-screen-only or large-screen-only…

Issue if Drawer has mdl-layout--small-screen-only or large-screen-only. Then the hamburger Icon needs to be hidden on small/large screen.  Fixed this by adding same class on drawer to slider-Icon(hamburger).
Created a fiddle here. You can see it just by clicking the slider icon.
http://jsfiddle.net/u9L12t1r/
This way the media query automatically hides the Icon
If not then Slider Icon is visible. but empty overlay appears when clicked as drawer has mdl-layout--small-screen-only class.
Thanks

I signed it!

LGTM. Thanks for your work, @gs-akhan.


switching to absolute positions in the current buffer

Hi Clement,
First off: This is a very naive implementation. I don't expect you to accept this pull request as is.
Rather, it's a starting point for me to learn about fstar-mode.el and to make some changes to it corresponding to some upcoming changes in F*.
Basically, F* now reports errors of the form  source-range: (Error) "some error message" (see source-range)
The first source-range is a range (typically in the current overlay) that contains some error. The second source-range is another range somewhere else in the program (perhaps in the current overlay, elsewhere in the current buffer, or in another file altogether) that reveals more information about the current error.
To support this, I started by changing fstar-mode.el to make use of absolute position information within the current buffer, rather than translating things relative to the current overlay using
fstar-subp-adjust-line-numbers.
I'd be delighted to hear your thoughts about how best to indicate the related, second source-range reported in the message to the user. For example, currently, if the user navigates to the highlighted error location, we show the associated error message at the bottom. Perhaps at that point, if presses some key combination, we could jump to related source-range reported in the message? Any tips for how to implement something like that (or some other scheme that you think may be more appropriate) would be very helpful too.
Many thanks!
-Nik

Hey Nik,
Sorry for the delay!  This looks pretty good to me, actually.  I'll leave comments in the diff.
Some questions regarding displaying related information:

How large do you expect the second range to be? A few lines at most? Or could it be more?
Will there ever be more than one error at a time?  More than one per line?

Your idea sounds good (and easy to implement, too :). Here are another few options that we might consider in addition to jumping:

Displaying an inline pop-up showing the related span in question.
Showing a small button which, once activated, would jump to the related error (and another button to take you back)

A concrete example of such an error-location pair would be useful, too :)

All done :) Is there a reason for the F* subprocess to take line and column numbers, instead of just absolute offsets in the file?
Another thing that might be worth considering: using absolute offset makes it tricky to allow edition of comments or spaces without invalidating the corresponding region of the file.  Not sure whether this matters, though (one could imagine caching on the F* side that would make reprocessing the unchanged code very fast).


#146687391 Stabilise base application

Why is this important?
This will ensure that the application can be run seamlessly when a team member clones the repo.

I'm sure this doesn't look anything like the PR Naming Convention you were given. Please address.

I have made changes based on what I read here

Did you see the description?

I just followed the description now. Is this okay?


MM-28551  mmctl local mode API migration (phase 4)

Documentation for:

https://mattermost.atlassian.net/browse/MM-28551, which includes a number of related tasks (MM-30017 through MM-30025 inclusive)

Updated:

Added new mmctl webhook section (including top-of-page TOC link)
Added new child webhook commands

Hello @cwarnermm,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

Preparing to merge v5.30 docs into master - can open a new PR for any further edits.


eth_getCompilers is removed from recent ethereum

https://github.com/ConsenSys/mythril/blob/8d8f1bf6cd5dac9c5c3c311a214950c68f86224e/mythril/rpc/base_client.py#L428-L434
Exception: {'jsonrpc': '2.0', 'id': 1, 'error': {'code': -32601, 'message': 'The method eth_getCompilers does not exist/is not available'}}

eth_getCompilers is not available for neither of ipc nor rpc, and also the compileXxx methods are gone
You can see more here: https://github.com/ethereum/go-ethereum/issues/3793.
I will create a PR for this :)

I should close #91, since this one is associated with a PR


[Dev] Allow users to assign issues with a comment on the issue

Describe the enhancement requested
With the migration to GitHub users don't have permissions to assign issues to themselves. This was possible with JIRA and we should allow and encourage users to work on issues. The current dev_pr workflow will auto-assign if a PR is opened and the issue did not have any assignee but it is good to do this preemptively to tell the rest of the community that someone is working on the issue.
Other projects like pandas or Ibis allow this to be done via a comment like take or /take on the issue itself.
We can update the comment bot to do that.
Component(s)
Developer Tools

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


Correction for raw affiliation Dr Annie Pullen-Sansfaçon is Associate Professor at the Université de Montréal, Canada. Her research interests include social work ethics and professional identity as well as groupwork and social action. Dave Ward is Professor of Social and Community Studies at De Montfort University, Leicester, England. His research interests include groupwork, social exclusion, and probation and criminal justice.

Correction needed for raw affiliation Dr Annie Pullen-Sansfaçon is Associate Professor at the Université de Montréal, Canada. Her research interests include social work ethics and professional identity as well as groupwork and social action. Dave Ward is Professor of Social and Community Studies at De Montfort University, Leicester, England. His research interests include groupwork, social exclusion, and probation and criminal justice.
raw_affiliation_name: Dr Annie Pullen-Sansfaçon is Associate Professor at the Université de Montréal, Canada. Her research interests include social work ethics and professional identity as well as groupwork and social action. Dave Ward is Professor of Social and Community Studies at De Montfort University, Leicester, England. His research interests include groupwork, social exclusion, and probation and criminal justice.
new_rors: 0161xgx34;0312pnr83
previous_rors: 0161xgx34
works_examples: W2158211136
contact: c2ec2203b76b09312fdab42079a63c40:6a499c @ dmu.ac.uk

This issue was accepted and ingested by the OpenAlex team on 2024-10-10. The new affiliations should be visible within the next 7 days.


push to main

phase 1 of website launch is done, which includes a one-pager site.
Additional contents will be added in coming months.

@pawelru @cicdguy
I am merging revise to main for the first launch of the website (i.e. one-pager, with sub-pages to be created). Not sure if the deploy bot would now launch from main, as previously it was set up to auto refresh https://insightsengineering.github.io/nest/ whenever revise gets updated.

@ProjectsOfMLee - Do you want to update the pipelines so that the above takes place?

No problem.


Initial Project Setup

Adds initial Docker components, Protobuf files, and OpenAPI spec for server

This project should also have LICENSE, CONTRIBUTING.md at the root and README.md content on how to bring up a dev environment.

LICENSE, README, and CONTRIBUTING added. Other comments addressed.


Feature Request - Mental Health Bar

I propose implementing a mental health bar. This bar takes damage when you overcast, regens at a steady rate (which can't be sped up like vanilla health can), and it kills you if it hits 0.
A mental health bar would make "spam-overcasting 19-damage spells" unsustainable even if you have a way to cheese your vanilla health, while still allowing you to overcast in moderation.

Absolutely no gameifying mental health in this mod. But this is an issue we need to fix ... we'll worry about it after the merge, I guess.

I think Math raises some good points, but just to put in a dissenting opinion I think with some fixes the current overcasting mechanic is good, flavorful, very different from other magic mods (which give you a resourceless power bar), and also lowers the stakes of hexcasting.
Overcasting as it stands (with reasonable limits and with correct costing, see #77) opens some fun optimization problems for what we might call "blood" mages. It's possible to cast a spell cheap enough that you can then cast a regen that recovers your results. This is simply way cooler than relying on passive recharge rates. Doing this correctly means a handful of free magic.  I think that is cool and fun.
And in the grand scheme of things, no spell you can cast within 19.5 cost is actually that amazing or impactful compared to spells beyond there. It will still be preferable to use media in nearly every case.

The problem with casting from HP is that you need to implement all sorts of random workarounds to solve issues like:

Invincibility-frames making it possible to Thoth a 19 HP spell with no issues whatsoever.
Other mods giving the player a ridiculous number of extra hearts or damage-reduction or insta-healing.
Highly abusable vanilla+mod interactions to regain health absurdly quickly. (For example, having a mod apply Regeneration I for 0:05 seconds, over and over again every tick, effectively giving you half a heart every tick.)

Now, obviously these are solvable problems, but solutions wouldn't be perfect since you're effectively trusting other mods to not fundamentally screw with vanilla HP in any meaningful way. That isn't a safe assumption to make, especially in the large modpacks many players will find Hex Casting in. That's why I'm petitioning for a non-HP-based solution here, despite the points KirinDave brings up.
It'd still be possible to have a system where Doing this correctly means a handful of free magic, some hitstun and weird particles. Doing it wrong means you die. without that system having to rely on vanilla HP. All you need to do is have it rely on some other bar or resource separate from vanilla HP.

Math did get the last laugh here given the current Bloodweave stuff giving people hundreds of dust for free.


Please add a license file

I'd like to add this to awesome-zsh-plugins, but it doesn't have a license file.
Please add a license file - if you don't have one in mind,  I recommend using choosealicense.com to help choose an appropriate one.
Thanks!

Dreams can come true! Thanks!

Thanks for being so fast to add a license


Update IC Candid Files

Motivation
The Internet Computer APIs have updated.
Changes

Updated the interface description files for the Internet Computer.
Updated the javascript bindings for the latest IC interface.

Tests

[ ] Please check the API updates for any breaking changes that affect our code.

The fail in this PR makes sense.
One of the types is used in the tests and it's missing new fields added. In this case, we would need to update manually.


Install of outdated externals in build hook breaks builds

[ ] main
[ ] 4-3-stable

On ubuntu24. This is likely due to the outdated externals not being available in the repo, so trying to build it with the dev environment won't work.
Lines:
https://github.com/irods/irods_capability_indexing/blob/6ea172ce0f33c310831d108d251aafa71906c75b/irods_consortium_continuous_integration_build_hook.py#L18-L20
To get them to work locally, I replaced these with
        'irods-externals-boost-libcxx1.81.0-1',
        'rods-externals-clang-runtime13.0.1-0',
        'irods-externals-clang13.0.1-0'

Does this need to be changed on a per-OS basis? Or should this work for all of them?

The same externals packages should be used to build all packages for all platforms for a particular version. Therefore, the externals should not differ from OS to OS. We should update them to use the same externals as those being used by other plugins and the iRODS server (where applicable). For any externals which are specific to this plugin, we should find the latest one available and use that, if possible.

@FifthPotato - Please checkbox and close if complete. Thanks


Time's recipes don't have default time

corrected with before_save on commit 9e63935:
class Recipe < ActiveRecord::Base
    before_save :set_default_time

    # some stuff here

    private

	# set default time on t_baking, t_cooling, t_cooking, t_rest if not already set
	def set_default_time
		zero_time = Time.new 2000, 1 ,1,0,0,0
		self.t_baking 	= zero_time unless self.t_baking.present?
		self.t_cooling 	= zero_time unless self.t_cooling.present?
		self.t_cooking 	= zero_time unless self.t_cooking.present?
		self.t_rest 	= zero_time unless self.t_rest.present?
	end

end


PhpEnvironment\Request::detectBaseUrl() - E_WARNING: strpos(): Empty needle

I think those checks on following lines https://github.com/zendframework/zend-http/blob/master/src/PhpEnvironment/Request.php#L498 should use empty() instead of !== null because in some cases it triggers warning on a line:
https://github.com/zendframework/zend-http/blob/master/src/PhpEnvironment/Request.php#L522

Your diagnosis is off. The conditionals in line 498 are correct; what needs to happen is for line 522 to check for a non-empty $baseUrl before the strpos():
if (! empty($baseUrl) && 0 === strpos($requestUri, $baseUrl)) {

I'll work on a test for this shortly.

I've converted this issue with a pull request, which adds a test for the condition (it failed prior to the patch), and a patch to ensure the condition does not occur.


PYODBC Messages Attribute

Hi Everyone,
I am facing an issue while trying to execute sql script through pyodbc. I want to display the messages and for that I found a newly released feature of pyodbc known as .messages. When I try to run the python code in notebook, it shows me this error " pyodbc.cursor has no attribute messages". I have checked the version of pyodbc and it is up to date. I wanted to know if someone else is facing the same issue and what is the solution for it as it is a newly released feature and might not be working.Thanks.

Tell us about your environment. An ODBC trace would also be useful to see what's going on.


Store timestamps as one type but return as Unix timestamp

My DB has created_at and updated_at timestamps on all table created using table.timestamps(true, true) but my mobile app's local database sync needs the timestamps as Unix timestamps with millisecond precision. How do I go about doing that?

raw('round(extract(epoch from created_at) * 1000)').as('created_at') in my .select whenever I pull the table values works


(Blog) Time Zone Format/Display

It seems that the blog (at least) subsystem defaults to US mm/dd/yyyy format. Time itself looks local, although I can't tell until we change our clocks in Europe as I'm in GMT for another ten days or so, so they're the same thing.
The preview then shows time in US long format (again, subject to TZ caveat above).
See image, with editor date on left and preview date on right:

I'd suggest that, if this isn't configurable on a per-user level to local time/format, the first of these should at least be an unambiguous format so we can immediately see the difference between 5th March and 3rd May. While you can see it in the preview, it's not ideal when it's the editor date that you'd set.

There doesn't seem to be a way to set this per user.  In staging I updated the format to    with time in 24 hour notation.  I did confirm that it respects your local timezone.  I'll leave this open until things get merged back into main so we can confirm this will work for folks.

I'm wondering if we should change the blog post date format (both card and full post) to use the same DD MMM YYYY format. We don't need to (and shouldn't) use US date style just because that's the one I'm used to.

While it would be possible to use the Intl api https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Intl I don't think it's a good idea. It would only make sense if our content would also be translated.


improved readme and config files

@gokhanuluderya-msft,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


instant.io unable to download torrent seeded by webtorrent-hybrid

Hi Feross
my app is going places but I ran into two issues. One is related to this package. I use it on an digital-ocean ubuntu node express server  to seed some files. Which works fine when I want to download the torrent via the webtorrent package in an electron environment. But when the only seed is the webtorrent-hybrid ubuntu server and I drop the magnetURI into instant.io nothing is found. Once I drop the same magnet into my electron app, it finds the torrent, downloads, seeds and THEN the instant.io app finds the torrent and downloads it. I assume the problem is that the server doesn't provide webRTC connections and instant.io accepts only those, which are provided once my electron app seeds.
I have downloaded xvfb and installed it via apt-get onto my server and do the whole node routine when the server launches.
var Xvfb = require('xvfb')
var xvfb = new Xvfb()
xvfb.startSync()
var express = require('express')
var WebTorrent = require('webtorrent-hybrid');

client.seed(file.localDir, function onSeed(torrent){
    db.update({_id: file._id}, {$set: {magnetURI: torrent.magnetURI}}, {}, function(err, updates) {
      if(err){console.log(err); throw new Error}
      console.log(updates)
    })
 })

This sounds like a firewall issue. Are all the ports on your server open? Can you paste the output of sudo iptables -L here?
There's no guarantee that peers can connect with WebRTC, but you can increase the probably by using a TURN server and passing the details into new WebTorrent({ tracker: { rtcConfig: { /* config here */ }}})


PIL is deprecated and unsupported in in Django 1.8 -> switch to Pillow

PIL support should have been dropped in Django 1.8 as stated here https://docs.djangoproject.com/en/1.8/releases/1.8/#features-removed-in-1-8
We should switch to Pillow as well
References https://github.com/feincms/feincms/issues/574

Thanks for the report. Pillow is imported using from PIL import Image as well, or does Pillow have a different top-level package name too?
The dependency in setup.py also references Pillow, not PIL: https://github.com/feincms/feincms/blob/master/setup.py#L43

Sorry, my bad. It clearly depends on Pillow, not PIL, while trying to reproduce some other bug I just ran into a "No module named 'PIL'" Exception that threw me off and thought it would be better to catch the exception and mention Pillow.
But this can actually only happen if running from source without having run setup.py on that environment, so it's not an issue

Thanks!


Throughput values are not in the correct unit

This document indicates that the throughput limits are in MiB/s. This does not match up with the portal or the ARM schema resulting in the values being incorrect.
A P40 disk created in the portal shows a maximum throughput of 250MB/s, which translates to approximately 238 MiB/s (view on the confiuguration tab of the disk):
ACTUAL PERFORMANCE
IOPS limit	7500
Throughput limit (MB/s)	250
The schema for the ARM resource is also in MB/s and not MiB/s:
https://docs.microsoft.com/en-us/azure/templates/microsoft.compute/disks#diskproperties-object
The schema unmistakably defines the unit as MB and not MiB:
The bandwidth allowed for this disk; only settable for UltraSSD disks. MBps means millions of bytes per second - MB here uses the ISO notation, of powers of 10.
Unless both the portal and the ARM schema are incorrect, the unit of measure for disk throughput should be MB/s not MiB/s.

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

ID: 2cd6eb6c-745c-b3eb-8ea8-7fb3be24b607
Version Independent ID: 93637a78-fc66-6c5d-7845-a4fad0045c7d
Content: Select a disk type for Azure IaaS Windows VMs - managed disks - Azure Windows Virtual Machines
Content Source: articles/virtual-machines/windows/disks-types.md
Service: virtual-machines
Sub-service: disks
GitHub Login: @roygara
Microsoft Alias: rogarana

Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

@tdgray Thanks for reaching out, I've gone ahead and contacted the product team for clarification regarding this.
My current understanding is that the correct unit is MiB but, they'll confirm. Not sure why the portal is displaying MB (I verified that it is), even for standard disk types) but we'll look into it.
Regarding that schema, those are either ultra disk or shared disk specific entries. Both of which are using MB instead of MiB.

@tdgray Following up, the product team has gotten back to me and the correct unit is MB. I've gone ahead and merged changes in for this, they will be live tomorrow.
Accordingly, I'll close this out.
Thanks for bringing this to our attention and helping to improve our docs, we appreciate it! :)
#please-close


Fix StorageHelper.get() returning null after clearing

Return the defaultValue

Mark default value as required

Signed-off-by: Sebastian Malton<EMAIL_ADDRESS>This fixes master (which is currently failing CI)

StorageHelper.clear() is probably redundant since there is reset() method which is doing what you trying to fix.

So should we remove that too? I think we should still have this since a user can do this.set(null) manually.

So should we remove that too? I think we should still have this since a user can do this.set(null) manually.

I would add a guard in set() for defaultValue on null/undefined setting attempt.

@ixrock instead or as well?

@Nokel81 instead.


Provide ToText and FromText instances for Hash "Genesis"

Issue Number
#357
This PR enables us to parse user-provided genesis hash values from the CLI.
Overview
This PR:

[x] provides ToText and FromText instances for the Hash "Genesis" type.
[x] removes duplication among ToText and FromText instances for the family of Hash t types.

Comments

Would be good to have corresponding roundtrips for this, a bit of negative tests too.

Added!


Added context manager/monad-like wrapper

I've added a wrapping class to minimize downtime of CANape from the user during program execution while using the API. Documentation probably needs to be better fleshed out, I think a couple variables could be better named and tests should probably be added. I'm committing before leaving work for the day since I know I'm not going to have the time to finish every single thing.
This does not fix, but does alleviate #4 Canape Buttons Blocked.
Final note is I had no idea what a pre-commit hook was until now, I did all the manual checks you mentioned, I don't know if that does different, I don't understand how I'm even supposed to work with that. I figure earlier feedback would be better than fully finishing every single detail and than having to change everything.
Thanks for consideration/time

I've finished the main edit pending review.
All but one test case was skipped, I have not
viewed them yet. I'm about to switch tasks
so I'm asking this before I look. Pytest told
me it was skipping because couldn't find canape
or something. Would you consider it good if
I added a secondary suite of mocked tests
so that any of the code can actually run with
full coverage without needing to worry about
the external environment? I have done this on
one of our own projects and don't have a problem
doing it here but it does add a lot and it will be sitting
there to add to maintenance overhead. I'm asking
because I'm not likely to permanently be working
on this project so I don't want to add work that
I'm never going to touch myself.

I like the idea of connecting to CANape and immediately releasing it. But i cannot get it to work with more recent versions.
Even with a pure C++ implementation, if i try to connect to an open CANape instance, i get AEC_WRITE_CMD: Error writing data to memory mapped file.

@zariiii9003
I'll test it deliberately by hand Monday, I thought that I had connected to a running instance of CANape 15.
The reason that I haven't updated those docs is because they're a bunch of warning about methods not being
found in the dll so I wasn't sure if it absolutely was supported or not.

I don't know about 15, but i had no success with 20 and 21.
Here's the C++ code, that i used:
#include <windows.h>   // for WINAPI in canapapi.h
#include <iostream>
#include "CANapAPI.h"

int main(int argc, char const *argv[])
{
    TAsap3Hdl gHdl = NULL;  // Your Asap3 Handle
    TModulHdl gModule;      // Your Module/Device Handle
    unsigned short gTaskId; // a global task id

    gHdl = NULL;
    // Asap3Init Parameters
    unsigned long responseTimeout = 0;
    char *workingDir = NULL;
    unsigned long fifoSize = 2000;
    BOOL debugMode = FALSE;

    BOOL res = Asap3Init(&gHdl, responseTimeout, workingDir, fifoSize, debugMode);
    if (!res) {
        unsigned short errCode = Asap3GetLastError(gHdl);
        char* errMsg = NULL;
        Asap3ErrorText(gHdl, errCode, &errMsg);
        std::cout << errCode << " " << errMsg << std::endl;
    }
    return 0;
}

I read through the official docs, to me they definitely implied connecting to an already open instance, but yeah; behavior is the same. I was able to open in modal_mode (undefined in the official documentation for CANape 15), and only some of the buttons are hidden in the toolbar, I expect scripting and buttons to remain usable. I haven't exhaustively tested everything but CANape 15 does seem to be supported insofar as the generic forms of the functions found in the api aren't defined.
This library is usable and I will be going forward using this for our purposes, at least for now. This PR is definitely less useful without the potential to connect to an open instance. It might still be useful for someone but that's more specific and I'm not sure it's good to have in the library, especially considering how simple the code is. Should I just close this PR as the feature isn't useful enough to warrant the bloat or should I flesh this out and try getting this pushed?

Correction
Python setting interactive mode to true allows use of all buttons again in the toolbar.
This PR retains it's usefulness more with slight modification to instead pass through the
connection without managing the close/open status and might be desirable to group
actions together without really effecting the end user so bad. I believe it would better
be to modify the PR to do that, but at your disgression it still may be better to not add
the PR.


Add tax form support for cross-host expenses

Issue reported in https://opencollective.freshdesk.com/a/tickets/51876
Part of https://github.com/opencollective/opencollective/issues/5064
When looking for tax form statuses, we group entries by Expense.FromCollectiveId. With the introduction of tax forms across hosts, the payee can be a hosted initiative if the collective has an active fiscal host (fromCollective.HostCollectiveId) we should use it instead. We should also think about the UX and wording, as the person who needs to fill in this tax form is one of the host admins - not the collective admin.
Current workaround: manually mark tax form as received in the database. This is not ideal, and we should revert these entries once the fix will be live. Also, we should manually mark tax forms as received for the hosts we control (OSC, OCF) which should already resolve many situations.

We should manually mark tax forms as received for the hosts we control (OSC, OCF) which should already resolve many situations.

Has this part been done @Betree ?
Season of Docs 2022 will launch soon and will involve expenses across non-1st party hosts so we probably need to get this sorted.

We should manually mark tax forms as received for the hosts we control (OSC, OCF) which should already resolve many situations.

Has this part been done @Betree ?
Season of Docs 2022 will launch soon and will involve expenses across non-1st party hosts so we probably need to get this sorted.

Yes, tax forms have been marked as received for OSC (in 2021) and OCF (in 2020). Tax forms now being valid for 3 years, we should be goood.

When a Collective not under OCF (an IC or under another host like NumFOCUS) submits an expense to Season of Docs under OSC, how will this be handled?
For an IC, it seems straightforward: the tax form notification should go to the admins and OSC needs to collect a tax form in the name of the legal entity being paid. However, we end up with a lot of 'hosts' who are actually supposed to be ICs but have set themselves up as a host with no Collectives instead. So we can't rely on the IC distinction fully.
For a host, such a NumFOCUS, we need to think it through, because the expense should be treated as coming from the Collective in terms of where the money is going, but the tax form needs to come from the host, and the notification should be sent to the host admin. It's also important that the host is only prompted to submit it once, even though multiple Collectives under their umbrella might submit expenses.

Got a similar case in https://opencollective.freshdesk.com/a/tickets/62872
Feedback from @alanna:

The tax form in such a case is the HOST’s tax form. So it’s the host admins that need to be asked to fill it out. A Collective admin is not authorized to sign such a form on behalf of the host. If it’s an IC then we can safely assume the admin can provide it. If it’s a hosted Collective they cannot. So to generalise the flow, we should trigger the tax form to be signed by the host admin, and have the tax form received status applied to all that Host’s Collectives so it doesn’t become a duplicate request. We should go in and turn this requirement off between first party host’s though, because we can manage that internally (for expenses between OSC/OCF/OCEU). For third party hosts, OSC and OCF still needs their tax form so it should be treated like an organization submitting an expense.

Todo:

[ ] modify getTaxFormsRequiredForExpenses to check
[ ] modify getTaxFormsRequiredForAccounts to check
[ ] modify text of tax form banner
[ ] modify wording in email

I closed this bit quickly. API support is basically done but we still need to tackle the frontend/docs parts items from https://github.com/opencollective/opencollective/issues/5068#issuecomment-1073703106

Regarding modify text of tax form banner, we have multiple strings concerned by this:

On the expense status tag: We can't pay until we receive your tax info. Check your inbox for an email from HelloWorks. Need help? Contact<EMAIL_ADDRESS>This one doesn't need an update IMO.
On the expense page, after submission:

We need your tax information before we can pay you. You will receive an email with a link to fill out a form. If you have not received the email within 24 hours, check your spam, then contact . Questions? See help docs about taxes.

I think we can implement the following logic there:
const getTaxFormMessage = (loggedInUser, expense) => {
  if (expense.payee.host) {
    if (loggedInUser.isAdmin(expense.payee.host)) {
      return `As ${expense.payee.name}'s host, we need your tax form before we can pay this expense. (...)`;
    } else if (loggedInUser.isAdmin(expense.payee)) {
      return `We need your fiscal host, ${expense.payee.host.name}, to submit a tax form before this expense can be paid. They should have received an email about that, but feel free to contact them to give them more info`;
    }
  } else {
    if (loggedInUser.isAdmin(expense.payee)) {
      return '...'; // Current message (id="expenseNeedsTaxFormMessage.msg")
    } else {
      return `We need ${expense.payee.name} to submit their tax information before this expense can be paid`;
    }
  }
}


feat: data source registry

Description
Basic code to build datasource registry for one click binding.
Fixes #21506
Type of change

New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Jest

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

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

QA activity:

[ ] Test plan has been approved by relevant developers
[ ] Test plan has been peer reviewed by QA
[ ] Cypress test cases have been added and approved by either SDET or manual QA
[ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
[ ] Added Test Plan Approved label after reveiwing all Cypress test

/ok-to-test sha=f55574b951349a27441842145261a4603b1eab36

/ok-to-test sha=68012789af1b849fe850e66749e56c18d7eb9113

/ok-to-test sha=4b03e9c3cdb685cbdb9eef03a1ef61474d50006e


format issue

Hi, I use a format of dd/MM/yyyy. But when the user types the date directly into the input field it uses MM/dd/yyyy. I suppose the input validator is not taking the format into account when parsing...

Also having this problem.

m2

+1

+1

Same issue and it's very problematic :/
If anyone have a solution to skirt this issue ?
I tried to re-format the input value when a user change the input by adding a ng-blur fonction :
this.dateEnd = moment(this.dateEnd, 'DD/MM/YYYY').format('MM/DD/YYYY');

This work fine, but when a user just leave without change the input, the date is invalid...

Yeah, its very problematic for my app too

+1

@qmorel I do a similar thing and do the same when the controller loads.

+1

Having the same issue. I will try to submit a PR.

This issue was fixed by #251. It should be closed.

Broken again on version 2.0.3.
Version 2.0.1 works fine (but the datepicker modal is covering the input where I type the date, I had to adjust the CSS manually).

There are newer versions, but they were not released on bower or npm. You should install them from a git commit hash.


[vcpkg/cmake] Add pkg-config to path for CMake?

Cannot really tell if this is something desirable or not. I just see a know that a few ports use find_package(PkgConfig OPTIONAL). So I think this should be discussed.

will add how you can use it
https://github.com/Kitware/CMake/blob/master/Modules/UsePkgConfig.cmake
## Config
exec_program(pkg-config
              ARGS --cflags glib-2.0
              OUTPUT_VARIABLE GLIB_CFLAGS)
exec_program(pkg-config
              ARGS --libs glib-2.0
              OUTPUT_VARIABLE GLIB_LDFLAGS)

It seems like ports should use vcpkg_add_to_path if they want to do this.


Update to ES6 and refactor

Opening this issue to let you know that I had forked your project and I'm working on a massive refactor to change everything to ES6 and to remove some issues with variables reassignments that I found while was reading.

#2 did a pull request for those fixes

Thank you :)
Merged #2


Clarification on "Admin privileges on the Dynamics 365 instance where you are installing Marketing"

The user account used to install Marketing will need to
a) have a CRM license assigned
b) will need to have a security role (System Administrator) granted in the CRM org that Marketing is going to be installed into

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

ID: 01ffaa42-ad97-2a18-596b-b9f1249163bd
Version Independent ID: 6dbdc5b7-fbf5-dd22-112a-af92a208d094
Content: Set up Dynamics 365 Marketing
Content Source: ce/marketing/purchase-setup.md
Service: dynamics-365-marketing
GitHub Login: @kamaybac
Microsoft Alias: kamaybac

Hi @CarstenMS , thank you for your feedback. I took a look at the topic, and I think the details you are looking for are already covered in the box at the top of the Prerequisite  section. Here, we say that users must have "admin privileges", rather than name a specific security role, because role privileges and their names may be customized. But I'll add mention of the name of the default admin security role here, as you suggest. The need to have a CRM license is considered a known issue, so this is listed in its own bullet here, together with a link to a workaround.

Hi Karl,
the current documentation (as agreed by my escalation engineer Pedro) is unclear and caused case<PHONE_NUMBER>00019
where the user was Global Admin on the tenant (which a customer might well consider having "admin privileges")
What is explicitly needed is a System Administator role in CRM (which was present)
AND
a CRM license for the user
so to prevent other customers from running into this, please add the suggested clarification.
Clarification on "Admin privileges on the Dynamics 365 instance where you are installing Marketing"https://docs.microsoft.com/en-us/dynamics365/marketing/purchase-setup
The user account used to install Marketing will need to
a) have a CRM license assigned
b) will need to have a security role (System Administrator) granted in the CRM org that Marketing is going to be installed into
Regards,
Carsten Behle
Support Engineer
Dynamics 365
Customer Services & Support
+49 (89) 3176-3075
carsten.behle<EMAIL_ADDRESS>09:00-18:00h CET=GMT+1
[Microsoft]
If you have feedback about my work, please let either me or my managers Joao.Rodrigues<EMAIL_ADDRESS>or Jesse.Lewis<EMAIL_ADDRESS>know.
From: Karl Maybach<EMAIL_ADDRESS>Sent: Freitag, 10. Januar 2020 11:03
To: MicrosoftDocs/dynamics-365-customer-engagement<EMAIL_ADDRESS>Cc: Carsten Behle<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/dynamics-365-customer-engagement] Clarification on "Admin privileges on the Dynamics 365 instance where you are installing Marketing" (#1536)
Hi @CarstenMShttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FCarstenMS&data=02|01|Carsten.Behle%40microsoft.com|14fc9f91377a4994acbb08d795b43ba2|72f988bf86f141af91ab2d7cd011db47|1|0|637142473644806127&sdata=jZSjdm8Afp0MPxNGqqMOi2WE3Vjtjf8nrSC%2FrVlrEHE%3D&reserved=0 , thank you for your feedback. I took a look at the topic, and I think the details you are looking for are already covered in the box at the top of the Prerequisite https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs.microsoft.com%2Fen-us%2Fdynamics365%2Fmarketing%2Fpurchase-setup%23prerequisites-and-requirements&data=02|01|Carsten.Behle%40microsoft.com|14fc9f91377a4994acbb08d795b43ba2|72f988bf86f141af91ab2d7cd011db47|1|0|637142473644806127&sdata=IRZp6%2BiryqtJXna5hiy24cBbcDE0Hg89ogyu%2BwXgcA4%3D&reserved=0 section. Here, we say that users must have "admin privileges", rather than name a specific security role, because role privileges and their names may be customized. But I'll add mention of the name of the default admin security role here, as you suggest. The need to have a CRM license is considered a known issue, so this is listed in its own bullet here, together with a link to a workaround.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fdynamics-365-customer-engagement%2Fissues%2F1536%3Femail_source%3Dnotifications%26email_token%3DANMPDCPIK2T6UH2JNB5CMLTQ5BBUFA5CNFSM4KE3RTBKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEITLCAA%23issuecomment-572961024&data=02|01|Carsten.Behle%40microsoft.com|14fc9f91377a4994acbb08d795b43ba2|72f988bf86f141af91ab2d7cd011db47|1|0|637142473644816136&sdata=KB%2FMvm22qDTwJBzqEEphSTFTqjYrV%2F6LnuQrVsKFQpM%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FANMPDCOIVJF2Z4QAVHFK2F3Q5BBUFANCNFSM4KE3RTBA&data=02|01|Carsten.Behle%40microsoft.com|14fc9f91377a4994acbb08d795b43ba2|72f988bf86f141af91ab2d7cd011db47|1|0|637142473644826121&sdata=X0bHIParf97iM1AQbo7LHsVGIugmNVGEEmTMRTB9crA%3D&reserved=0.

Hi Carsten,
Ah, I see. Yes, the listed requirement is referring to being an admin on the tenant (not in CRM), so Global Admin does indeed meet this requirement. The issue of even needing a CRM license is considered a bug, as described now, but you are right that we don't mention here anything about the CRM security role required for the CRM user account. Your clarification helps, thanks. I'm revising the doc now.
. Karl
From: CarstenMS<EMAIL_ADDRESS>Sent: Friday, January 10, 2020 11:08 AM
To: MicrosoftDocs/dynamics-365-customer-engagement<EMAIL_ADDRESS>Cc: Karl Maybach<EMAIL_ADDRESS>Assign<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/dynamics-365-customer-engagement] Clarification on "Admin privileges on the Dynamics 365 instance where you are installing Marketing" (#1536)
Hi Karl,
the current documentation (as agreed by my escalation engineer Pedro) is unclear and caused case<PHONE_NUMBER>00019
where the user was Global Admin on the tenant (which a customer might well consider having "admin privileges")
What is explicitly needed is a System Administator role in CRM (which was present)
AND
a CRM license for the user
so to prevent other customers from running into this, please add the suggested clarification.
Clarification on "Admin privileges on the Dynamics 365 instance where you are installing Marketing"https://docs.microsoft.com/en-us/dynamics365/marketing/purchase-setup
The user account used to install Marketing will need to
a) have a CRM license assigned
b) will need to have a security role (System Administrator) granted in the CRM org that Marketing is going to be installed into
Regards,
Carsten Behle
Support Engineer
Dynamics 365
Customer Services & Support
+49 (89) 3176-3075
carsten.behle<EMAIL_ADDRESS>09:00-18:00h CET=GMT+1
[Microsoft]
If you have feedback about my work, please let either me or my managers Joao.Rodrigues<EMAIL_ADDRESS>or Jesse.Lewis<EMAIL_ADDRESS>know.
From: Karl Maybach<EMAIL_ADDRESS>Sent: Freitag, 10. Januar 2020 11:03
To: MicrosoftDocs/dynamics-365-customer-engagement<EMAIL_ADDRESS>Cc: Carsten Behle<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/dynamics-365-customer-engagement] Clarification on "Admin privileges on the Dynamics 365 instance where you are installing Marketing" (#1536)
Hi @CarstenMShttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FCarstenMS&data=02|01|Carsten.Behle%40microsoft.com|14fc9f91377a4994acbb08d795b43ba2|72f988bf86f141af91ab2d7cd011db47|1|0|637142473644806127&sdata=jZSjdm8Afp0MPxNGqqMOi2WE3Vjtjf8nrSC%2FrVlrEHE%3D&reserved=0 , thank you for your feedback. I took a look at the topic, and I think the details you are looking for are already covered in the box at the top of the Prerequisite https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs.microsoft.com%2Fen-us%2Fdynamics365%2Fmarketing%2Fpurchase-setup%23prerequisites-and-requirements&data=02|01|Carsten.Behle%40microsoft.com|14fc9f91377a4994acbb08d795b43ba2|72f988bf86f141af91ab2d7cd011db47|1|0|637142473644806127&sdata=IRZp6%2BiryqtJXna5hiy24cBbcDE0Hg89ogyu%2BwXgcA4%3D&reserved=0 section. Here, we say that users must have "admin privileges", rather than name a specific security role, because role privileges and their names may be customized. But I'll add mention of the name of the default admin security role here, as you suggest. The need to have a CRM license is considered a known issue, so this is listed in its own bullet here, together with a link to a workaround.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fdynamics-365-customer-engagement%2Fissues%2F1536%3Femail_source%3Dnotifications%26email_token%3DANMPDCPIK2T6UH2JNB5CMLTQ5BBUFA5CNFSM4KE3RTBKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEITLCAA%23issuecomment-572961024&data=02|01|Carsten.Behle%40microsoft.com|14fc9f91377a4994acbb08d795b43ba2|72f988bf86f141af91ab2d7cd011db47|1|0|637142473644816136&sdata=KB%2FMvm22qDTwJBzqEEphSTFTqjYrV%2F6LnuQrVsKFQpM%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FANMPDCOIVJF2Z4QAVHFK2F3Q5BBUFANCNFSM4KE3RTBA&data=02|01|Carsten.Behle%40microsoft.com|14fc9f91377a4994acbb08d795b43ba2|72f988bf86f141af91ab2d7cd011db47|1|0|637142473644826121&sdata=X0bHIParf97iM1AQbo7LHsVGIugmNVGEEmTMRTB9crA%3D&reserved=0.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fdynamics-365-customer-engagement%2Fissues%2F1536%3Femail_source%3Dnotifications%26email_token%3DAGJXB4MWGJ3VYPFHPATMTJTQ5BCHTA5CNFSM4KE3RTBKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEITMNTY%23issuecomment-572966607&data=02|01|Karl.Maybach%40microsoft.com|f6c046aba7894e6536ba08d795b4f5a6|72f988bf86f141af91ab2d7cd011db47|1|0|637142476769916520&sdata=LE9%2FfwGwN%2FytvklMKbgrvtJgmubYapAAFnRCkDNDC3A%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAGJXB4JV656B7AZRBOO3WALQ5BCHTANCNFSM4KE3RTBA&data=02|01|Karl.Maybach%40microsoft.com|f6c046aba7894e6536ba08d795b4f5a6|72f988bf86f141af91ab2d7cd011db47|1|0|637142476769926123&sdata=Ikd7Y6TiIX4IB7qQXJsBtwmlxK0Jzaz4jQ%2BAHu%2Ftn6M%3D&reserved=0.

PS: sorry, I did not go through the link provided - this in a different place explains that a CRM license is needed, but the "workaround" per my feelings should not be there
as probably no customer will convert a trial CRM instance into prod when his initial objective was to add Marketing to an existing CRM org
and Marketing trials can NOT be converted to prod ones at all (as link Dynamics 365 Marketing trial limits and restrictionshttps://go.microsoft.com/fwlink/p/?linkid=864735. above confirms on
and you can only create a trial with Marketing already included, not add Marketing to a trial
so the current partially wrong information on
https://docs.microsoft.com/en-us/dynamics365/marketing/readme#configuration-installation-and-first-run-experience
"As a temporary workaround, you can set up a trial Dynamics 365 instance (which will work for 30 days) and then install Marketing. You'll still need to convert your trial into a production instance before the 30 days are up, but this way you can start working in Marketing right away. More information: Try Power Apps and model-driven apps in Dynamics 365 appshttps://docs.microsoft.com/power-platform/admin/try-powerapps-dynamics-365."
should be removed. If you really think this section adds value and there are admins without CRM license available, it should be
As a temporary workaround if you don't have a CRM license, you can set up a trial Dynamics 365 instance with Marketing (which will work for 30 days)
From: Carsten Behle
Sent: Freitag, 10. Januar 2020 11:08
To: MicrosoftDocs/dynamics-365-customer-engagement<EMAIL_ADDRESS>MicrosoftDocs/dynamics-365-customer-engagement<EMAIL_ADDRESS>Cc: Mention<EMAIL_ADDRESS>Pedro Pires<EMAIL_ADDRESS>Subject: RE: [MicrosoftDocs/dynamics-365-customer-engagement] Clarification on "Admin privileges on the Dynamics 365 instance where you are installing Marketing" (#1536)
Hi Karl,
the current documentation (as agreed by my escalation engineer Pedro) is unclear and caused case<PHONE_NUMBER>00019
where the user was Global Admin on the tenant (which a customer might well consider having "admin privileges")
What is explicitly needed is a System Administator role in CRM (which was present)
AND
a CRM license for the user
so to prevent other customers from running into this, please add the suggested clarification.
Clarification on "Admin privileges on the Dynamics 365 instance where you are installing Marketing"https://docs.microsoft.com/en-us/dynamics365/marketing/purchase-setup
The user account used to install Marketing will need to
a) have a CRM license assigned
b) will need to have a security role (System Administrator) granted in the CRM org that Marketing is going to be installed into
Regards,
Carsten Behle
Support Engineer
Dynamics 365
Customer Services & Support
+49 (89) 3176-3075
carsten.behle<EMAIL_ADDRESS>09:00-18:00h CET=GMT+1
[Microsoft]
If you have feedback about my work, please let either me or my managers Joao.Rodrigues<EMAIL_ADDRESS>or Jesse.Lewis<EMAIL_ADDRESS>know.
From: Karl Maybach<EMAIL_ADDRESS>Sent: Freitag, 10. Januar 2020 11:03
To: MicrosoftDocs/dynamics-365-customer-engagement<EMAIL_ADDRESS>Cc: Carsten Behle<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/dynamics-365-customer-engagement] Clarification on "Admin privileges on the Dynamics 365 instance where you are installing Marketing" (#1536)
Hi @CarstenMShttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FCarstenMS&data=02|01|Carsten.Behle%40microsoft.com|14fc9f91377a4994acbb08d795b43ba2|72f988bf86f141af91ab2d7cd011db47|1|0|637142473644806127&sdata=jZSjdm8Afp0MPxNGqqMOi2WE3Vjtjf8nrSC%2FrVlrEHE%3D&reserved=0 , thank you for your feedback. I took a look at the topic, and I think the details you are looking for are already covered in the box at the top of the Prerequisite https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs.microsoft.com%2Fen-us%2Fdynamics365%2Fmarketing%2Fpurchase-setup%23prerequisites-and-requirements&data=02|01|Carsten.Behle%40microsoft.com|14fc9f91377a4994acbb08d795b43ba2|72f988bf86f141af91ab2d7cd011db47|1|0|637142473644806127&sdata=IRZp6%2BiryqtJXna5hiy24cBbcDE0Hg89ogyu%2BwXgcA4%3D&reserved=0 section. Here, we say that users must have "admin privileges", rather than name a specific security role, because role privileges and their names may be customized. But I'll add mention of the name of the default admin security role here, as you suggest. The need to have a CRM license is considered a known issue, so this is listed in its own bullet here, together with a link to a workaround.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fdynamics-365-customer-engagement%2Fissues%2F1536%3Femail_source%3Dnotifications%26email_token%3DANMPDCPIK2T6UH2JNB5CMLTQ5BBUFA5CNFSM4KE3RTBKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEITLCAA%23issuecomment-572961024&data=02|01|Carsten.Behle%40microsoft.com|14fc9f91377a4994acbb08d795b43ba2|72f988bf86f141af91ab2d7cd011db47|1|0|637142473644816136&sdata=KB%2FMvm22qDTwJBzqEEphSTFTqjYrV%2F6LnuQrVsKFQpM%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FANMPDCOIVJF2Z4QAVHFK2F3Q5BBUFANCNFSM4KE3RTBA&data=02|01|Carsten.Behle%40microsoft.com|14fc9f91377a4994acbb08d795b43ba2|72f988bf86f141af91ab2d7cd011db47|1|0|637142473644826121&sdata=X0bHIParf97iM1AQbo7LHsVGIugmNVGEEmTMRTB9crA%3D&reserved=0.


Add giftCardSettings query

Add GiftCardSettings type
Add giftCardSettings query

type GiftCardSettings {
  giftCardExpiryPeriod: Int
  expiryType: GiftCardSettingsExpiryTypeEnum!
  expiryPeriod: TimePeriod
}

enum GiftCardSettingsExpiryTypeEnum {
  NEVER_EXPIRE
  EXPIRY_PERIOD
}

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-3876 Add giftCardSettings query


[Vue warn]: A plugin must either be a function or an object with an "install" function.

Environment

Operating System: Darwin
Node Version:     v16.15.1
Nuxt Version:     3.0.0-rc.4
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      css, tailwindcss, modules, colorMode, pwa, buildModules, storyblok, build
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

Reproduction
https://github.com/jamaluddinrumi/lemahabang.v3
Problems
I import @formkit/auto-animate through plugin as below
/plugins/autoAnimate.js

import { autoAnimate } from "@formkit/auto-animate";

export default defineNuxtPlugin((nuxtApp) => {
  nuxtApp.vueApp.use(autoAnimate);
});

i got warn

i also tried to include @formkit/auto-animate in build.transpile but it's still the same
/nuxt.config.ts

...
build: {
    transpile: [
      "swiper",
      "@headlessui/vue",
      "@heroicons/vue",
      "@formkit/auto-animate",
      "@fortawesome/vue-fontawesome",
      "@fortawesome/fontawesome-svg-core",
      "@fortawesome/free-solid-svg-icons",
      "@fortawesome/free-brands-svg-icons",
    ],
...

Here's a working plugin for ya: https://github.com/jamaluddinrumi/lemahabang.v3/pull/1


Get Metadata Only from a remote image

Question about an existing feature
What are you trying to achieve?
Is it possible to use Sharp to get only metadata about images? I could not find anything about this concept in your documentation. But, I have an operation where I am storing data about images in a s3 bucket for use in other operations.  I don't actually need to download these images I just need the metadata real quick. I considered using S3s API or something like https://github.com/nodeca/probe-image-size but I prefer to use an active package like yours that is really simple.
The metadata provides what I need but I would like to save on the bandwidth if I can.
When you searched for similar issues, what did you find that might be related?

Did you see #3656?

No, I will give that a try today, thanks.

Based on the #3656 I think this does what I want and metadata is available. I don't that this is robust enough to handle all images I might encounter but for now this is at least workable for testing. Thanks for the help.
const fetchImageChunk = async (
  url: string,
  start: number,
  end: number
): Promise<Buffer> => {
  try {
    const response = await fetch(url, {
      headers: { Range: `bytes=${start}-${end}` },
    });
    if (response.ok) {
      return Buffer.from(await response.arrayBuffer());
    } else {
      throw new Error(`Failed to fetch image. Status code: ${response.status}`);
    }
  } catch (error) {
    throw new Error(`Error fetching image chunk: ${error}`);
  }
};

const buffer = await fetchImageChunk(url, 0, 1000); // Max size for peeking
const metadata = await sharp(buffer).metadata();


Not compatible with react-native 75

As of ReactNative 75 it looks as though they've removed the JSIModule* types for Android:
https://reactnative.dev/blog/2024/08/12/release-0.75#android-jsimodule-has-been-deleted
It's a fairly new release so I'm not expecting this to already be caught up, but, are there any plans to use TurboModules?

As of ReactNative 75 it looks as though they've removed the JSIModule* types for Android:
https://reactnative.dev/blog/2024/08/12/release-0.75#android-jsimodule-has-been-deleted
It's a fairly new release so I'm not expecting this to already be caught up, but, are there any plans to use TurboModules?

Thanks for the reminder. I'm well aware of JSIModule was set to deprecated and finally removed. The tricky part is I will have to make a Turbo Native Module to support RN 75+. Definitely one in my todos. But don't expect to see it anytime soon. Sorry :(


change pakage name (chisel -> chisel3)

@donggyukim , please see pr #21. I think it would be best if we could remove internal dependencies and avoid having to rename the package (i.e., only depend on the exported Chisel API).

I didn't know the existence of #21. Closing this request.


How To Add Shaders On main Menu

Describe your problem here.
How Do I Add Shaders On The Main Menu Like Dave And Bambi 3.0 When Its On Expunged Mode
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No response

https://github.com/MoldyGH/VsDave

copy the code in VsDave’s mainmenustate.hx that adds the shader code
and then paste it to the psych mainmenustate.hx
easy peasy


ENH: read and parse ORT files

This PR is a better method the loading of the ORT dataset into json dictionaries compared to the overall commit state in #16.

Parses multiple datasets in an ORT file into separate (complete) json dictionaries.
Amends the validation to work on those separate dictionaries.

I'll test and merge this update with the fromdict branch and let you know how it works.

@aglavic, it wasn't quite clear to me what the _staggered_repr was for on your branch. Is it for debugging?

@andyfaff it is for creating a more readable representation when used from command line. I replaced the class repr method for more complex objects with it to split the data on multiple lines.


Add new Reservoir node

Add a new reservoir node with the following features:

[ ] Evaporation and rainfall parameters (typically defined as L (e.g. mm))
[ ] Bathymetry (i.e. area and level) data to construct parameters that determine current area and level from volume (see also #179 )

[ ] Have evaporation and rainfall flow calculated from above.
[ ] Optionally allow rainfall to always use the maximum area of the reservoir (i.e. all rainfall runs off into the reservoir regardless of current volume)

[ ] Add optional output slots

[ ] Spill
[ ] Compensation
[ ] Losses (Output)

Agreed to not include abstraction or hydropower nodes initially.

#256


Insufficient To Be Able To Actually Implement

In the past this page also included a sample of how to implement the webhook and decode the events etc. Since the Azure Event Grid library changed this page no longer has any samples including the required C# implementation. This should be restored and updated to the latest Azure Event Grid library for web hooks with full example.

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

ID: 8ded37e1-ba50-6f17-2c99-519d2a563a56
Version Independent ID: b8bf5ffd-518d-58b3-6b36-bfdac2e57391
Content: Reacting to Azure Media Services events
Content Source: articles/media-services/latest/monitoring/reacting-to-media-services-events.md
Service: media-services
GitHub Login: @IngridAtMicrosoft
Microsoft Alias: inhenkel

Thanks for the feedback and bringing this to our notice . At this time we are reviewing the feedback and will update the document as appropriate .

@jhancock4d Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

@jhancock4d  - This article is conceptual and not a tutorial.
I think you are asking for a dedicated tutorial page or How-To page for using Event Grid with the latest library.
I just recently refreshed all of our .NET samples to the latest Event Grid SDK library and I would recommend looking at this first
https://github.com/Azure-Samples/media-services-v3-dotnet
Right now, the event grid and .NET SDK team keeps changing things so rapidly, that I kind of prefer to leave the sample on github as the source of truth.  I heard that there is yet another SDK change coming in the near future.   Let me confirm that with the team and make sure that whatever sample article I write up next will be a bit more stable.
I'll likely put it in How-To or Tutorials and just link to it from this article.

#please-close


Another repo that falls under joint ownership

The conversation was started a little while back about needing a Collaboration/Admin repo for the TSC and CommComm jointly administering the travel fund. There's now another repo I think falls under this category, too https://github.com/nodejs/getting-started.
I indicated this also in the TSC issue regarding the prior conversation about our collaborative committee work.
The need here is likely to have a meeting or create a doc with groups from both committees to get the ball rolling on this work.

Getting Started is in the Admin repo now 💥
Going to close this - feel free to re-open if further discussion is needed.


Modified to use dataset and pipelinedata

I modified the pipeline to be more "AMLish":

Added instructions to Getting Started to register a dataset (or set variables for the pipeline to do it for you)
Used input in training step to pass dataset object
Added pipelinedata object to pass trained model as output from train step and input to register step

I'm getting a "Bad Gateway" error in the smoke test after the model service is deployed. I don't think this is due to my changes - it looks like an AML service error, but we should probably hold off on merging until we troublehoot it.


Bump puppeteer and node libraries for M1 dev-time support

The latest version of puppeteer is required in order for it find  and use the chromium arm64 build. The latest puppeteer version in turn requires the latest webpack and css-optimizer versions. See example changes for Explorer here.

Pull out these into a separate branch/PR for review by the team. Explicit versions should be pinned in packages.json, rather than using latest.
README docs should be updated to include this M1-specific dev env setup

Acceptance Criteria:

All engineers using an M1 can run frontend tests successfully

@atolopko-czi and maybe @tihuan We don't use puppeteer in Data Portal. None of the packages that were upgraded in Explorer are in Data Portal. Is the primary goal here just to verify that our frontend tests in Data Portal are compatible with M1/M2 machines?

@atolopko-czi and maybe @tihuan We don't use puppeteer in Data Portal. None of the packages that were upgraded in Explorer are in Data Portal. Is the primary goal here just to verify that our frontend tests in Data Portal are compatible with M1/M2 machines?

The issue is just for Explorer, so if we can run our Explorer FE tests locally on M1/M2 hosts now, then this is fully addressed. Thanks!

Hmm yeah I'm able to run Puppeteer tests with my M2 machine, although tests are failing because the local app is dysfunctional. So perhaps that's a separate issue to track lol

@tihuan Just to clarify, you're talking about being able to run the Explorer Puppeteer tests on your M2 machine?

that's correct!

Awesome - going to close out this ticket then.


Train loss = nan after few steps

Hi,
I know this issue has been resolved in commit https://github.com/qqwweee/keras-yolo3/commit/b7cbd24d1b201d2d3e10d33fd79d3fd60d21a858. I am trying to train the model on custom data set that consists of single channel images. I modified utils.py to handle single channel images. The loss decreases for a few steps but after that it becomes nan and never comes back. Is anyone else still facing this problem?

check if you get inf before the nan somwhere. For me the issue was that one of the box somewhere had a dim of zero in one of its axis.  which result with log(0)

Here is my solution
Hope it would help
https://github.com/qqwweee/keras-yolo3/issues/171#issuecomment-522287613


⚠ Bofrost.lu has degraded performance

In 8128123, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1241 ms

Resolved: Bofrost.lu performance has improved in 467a8b7 after 1 hour, 12 minutes.