[Bug]: GoCardless will implement strict ratelimiting

Verified issue does not already exist?

[X] I have searched and found no existing issue

What happened?
I got the following mail from GoCardless.
I think we should grab those headers from each response and keep count of where we're at, so that we can:

return a comprehensible error to the user when GoCardless limits are reached
filtering unecessary sync events by the user from the actual app,to make sure that "unnecessary" calls are not being made to the GoCardless API, because 4 syncs a day is not a lot...

Dear Customer,

We are reaching out to inform you of some important updates regarding our API that will enhance your experience and provide greater visibility into bank API rate limits.

Improvements in the API
=======================

As part of our effort to enhance transparency around API rate limits, we are implementing additional information in our API response - rate limit headers.

What are Rate Limit Headers?
============================

Rate limit headers are response HTTP header fields included in API responses that will inform you about your current usage and remaining API call limits. These headers will offer you clearer insight into your rate limits for all API requests.

The new headers will provide key details such as the total request limit, the number of requests remaining within a specified timeframe, and the time until the limits reset.
The introduction of new API rate limit headers will include the following response headers:

General Rate Limits (for all API requests):
===========================================

    HTTP_X_RATELIMIT_LIMIT: Indicates the maximum number of allowed requests within the defined time window.
    HTTP_X_RATELIMIT_REMAINING: Shows the number of remaining requests you can make in the current time window.
    HTTP_X_RATELIMIT_RESET: Provides the time remaining in the current window.

Account Success Request (for account details, transactions, and balances):

    HTTP_X_RATELIMIT_ACCOUNT_SUCCESS_LIMIT: Indicates the maximum number of allowed requests within the defined time window.
    HTTP_X_RATELIMIT_ACCOUNT_SUCCESS_REMAINING: Shows the number of remaining requests you can make in the current time window.
    HTTP_X_RATELIMIT_ACCOUNT_SUCCESS_RESET: Provides the time remaining in the current window.

How are these changes being made?
=================================

Starting from 19 August 2024, we will implement a limit of 10 API requests per day for each access scope (details, balances, transactions) on account id level as part of our ongoing efforts to enhance operational effectiveness and minimize potential issues for all our clients. This phased approach will allow us to optimize our services progressively.

We plan to establish a limit of four requests per day for each access scope in the next phase.

We will keep you informed and provide you with notice prior to the launch of the next phase, once we have finalized the timelines.

What does this mean for you?
============================

From 19 August 2024, there will be a 10 requests per day limit. However, we advise you to begin adjusting your applications to accommodate the upcoming four requests per day limit.

Please note that failed requests against details, balances, and transactions endpoints will not count against the bank API rate limits, but they will count towards the general[ Bank Account Data API rate limits](https://gocardless.cmail19.com/t/t-l-sljtyx-tyjdhjttdy-r/) Bank Account Data API rate limits.

As always, should you have any questions or require further assistance, please do not hesitate to reach out to our support team [here](https://gocardless.cmail19.com/t/t-l-sljtyx-tyjdhjttdy-y/).

For further details, please refer to our updated help article on bank rate limits [here](https://gocardless.cmail19.com/t/t-l-sljtyx-tyjdhjttdy-j/).

Thank you for your continued partnership.

Kind regards,

The GoCardless Bank Account Data team

What error did you receive?
No response
Where are you hosting Actual?
None
What browsers are you seeing the problem on?
No response
Operating System
None

There was a bit of conversation around this in the discord earlier if you're interested!
1 - return a comprehensible error to the user when GoCardless limits are reached
I believe that this is possible in some capacity at the moment, but only once the rate limit has been reached. The message would have to be something simple like "the rate limit has been reached, please wait for it to reset" as we can not access the headers to give information on when the time window resets through nordigen-node as it is currently implemented.
I've raised nordigen/nordigen-node#62 in hopes that something can be done about this.
2 - filtering unecessary sync events by the user from the actual app,to make sure that "unnecessary" calls are not being made to the GoCardless API
This is tough if I'm understanding what you're proposing. How can we define an unnecessary sync event?
The easier route to this (this could have been what you meant too, I could have misunderstood!) is to optimise the way that actual performs bank syncs. Currently, any sync will use 3 api calls, which means that when the 4/day limit is imposed we're limiting users to 1 sync per day.
Looking at the code we could optimise the way we fetch relatively easily, I'm looking at a change to do it at the moment but I'll be moving house tomorrow so it's all a little hectic here as you can probably imagine.

Closing this as we've made the sync as lean as possible and errors are shown in the UI when the rate limit is exceeded.


queries don't get executed result callback function not running

queries don't get executed result callback function not running
Expected Behavior
create a table if does not exist and view my data
Current Behavior
the result callback function returns nothing
Your Environment

react-native-sqlite-storage": "^5.0.0"

my code
     const createTable = async () => {            await db.transaction((tx) => {                    tx.executeSql("CREATE TABLE IF NOT EXISTS"                     +"chat_sms"                     +"(ID INTEGER PRIMARY KEY AUTOINCREMENT, name VARCHAR(255) NOT NULL);",                     [],                     ()=>{console.log("someth")});                 })       }

You need space between table name and fields, check the docs to enablePromise.


contant_case_test.exs missing s

It probably should be constant_cast_test.exs

@LostKobrakai yeap, thanks! Missed it.


use-recent-api-versions making incorrect recommendation

Bicep version
v0.17.1
Describe the bug
I am getting this bicep linting warning
Warning use-recent-api-versions: Use more recent API version for 'Microsoft.Insights/diagnosticSettings'.
'2021-05-01-preview' is 739 days old, should be no more than 730 days old, or the most recent.
Acceptable versions: 2016-09-01

Whilst it's true 2021-05-01-preview is old, i don't think an older version 2016-09-01 should be recommended.
To Reproduce
Declare this resource Microsoft.Insights/diagnosticSettings@2021-05-01-preview
Additional context
Could be related to #10345

@Gordonby Do you agree that https://github.com/Azure/bicep/issues/10345 would solve this issue for you?  Also, is it simple on/off switch to not prefer stable release enough, or does it need to be more finely tuned (please comment in https://github.com/Azure/bicep/issues/10345).  Thanks!

BTW, the current behavior is exactly the same as the ARM TTK.

@Gordonby Do you agree that https://github.com/Azure/bicep/issues/10345 would solve this issue for you?  Also, is it simple on/off switch to not prefer stable release enough, or does it need to be more finely tuned (please comment in https://github.com/Azure/bicep/issues/10345).  Thanks!

Yes, it would.
Simple switch works fine

Closing in favor of https://github.com/Azure/bicep/issues/10345.  Thanks!


openjdk: add support for powerpc64le

Description of changes
This commit adds entries to the Rosetta Stones in adoptopenjdk-bin/generate-sources.py and compilers/openjdk/8.nix, and runs adoptopenjdk-bin/generate-sources.py to regenerate adoptopenjdk-bin/sources.json.
With this commit, nix-build . -A jdk8_headless succeeds on powerpc64le.  Headless jdk is used as part of the build process for many packages so this opens up access to them.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin
[X] powerpc64le-linux

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

@ofborg eval


Max datetime is 2037 year

SELECT toDateTime('2037-01-01 00:00:00') return 2037-01-01 00:00:00, but
SELECT toDateTime('2038-01-01 00:00:00') return 0000-00-00 00:00:00
It's a bug?

No, it's not a bug. It's because ClickHouse uses 32-bit type to store DateType.
https://en.wikipedia.org/wiki/Year_2038_problem


pip install -e . fails

Thanks for your awesome work!
Hi, I get the following error when building:
    -- Generating done
    -- Build files have been written to: /home/larsh/development/pixel-perfect-sfm/build/temp.linux-x86_64-cpython-39
    [ 56%] Built target pixsfm
    [ 60%] Building CXX object pixsfm/CMakeFiles/pypixsfm.dir/bundle_adjustment/bindings.cc.o
    In file included from /home/larsh/development/pixel-perfect-sfm/pixsfm/bundle_adjustment/src/costmap_extractor.h:18,
                     from /home/larsh/development/pixel-perfect-sfm/pixsfm/bundle_adjustment/bindings.cc:14:
    /home/larsh/development/pixel-perfect-sfm/pixsfm/bundle_adjustment/src/reference_extractor.h:331:32: error: redefinition of default argument for ‘int N_NODES’
      331 | template <int N_NODES = Eigen::Dynamic>
          |                                ^~~~~~~
    /home/larsh/development/pixel-perfect-sfm/pixsfm/bundle_adjustment/src/reference_extractor.h:52:32: note: original definition appeared here
       52 | template <int N_NODES = Eigen::Dynamic>
          |                                ^~~~~~~
    make[2]: *** [pixsfm/CMakeFiles/pypixsfm.dir/build.make:132: pixsfm/CMakeFiles/pypixsfm.dir/bundle_adjustment/bindings.cc.o] Error 1
    make[1]: *** [CMakeFiles/Makefile2:376: pixsfm/CMakeFiles/pypixsfm.dir/all] Error 2
    make: *** [Makefile:136: all] Error 2
    Traceback (most recent call last):
      File "<string>", line 2, in <module>
      File "<pip-setuptools-caller>", line 34, in <module>
      File "/home/larsh/development/pixel-perfect-sfm/setup.py", line 112, in <module>
        setup(
      File "/home/larsh/micromamba/envs/pp_sfm/lib/python3.9/site-packages/setuptools/__init__.py", line 87, in setup
        return distutils.core.setup(**attrs)
      File "/home/larsh/micromamba/envs/pp_sfm/lib/python3.9/site-packages/setuptools/_distutils/core.py", line 185, in setup
        return run_commands(dist)
      File "/home/larsh/micromamba/envs/pp_sfm/lib/python3.9/site-packages/setuptools/_distutils/core.py", line 201, in run_commands
        dist.run_commands()
      File "/home/larsh/micromamba/envs/pp_sfm/lib/python3.9/site-packages/setuptools/_distutils/dist.py", line 968, in run_commands
        self.run_command(cmd)
      File "/home/larsh/micromamba/envs/pp_sfm/lib/python3.9/site-packages/setuptools/dist.py", line 1217, in run_command
        super().run_command(command)
      File "/home/larsh/micromamba/envs/pp_sfm/lib/python3.9/site-packages/setuptools/_distutils/dist.py", line 987, in run_command
        cmd_obj.run()
      File "/home/larsh/micromamba/envs/pp_sfm/lib/python3.9/site-packages/setuptools/command/develop.py", line 34, in run
        self.install_for_development()
      File "/home/larsh/micromamba/envs/pp_sfm/lib/python3.9/site-packages/setuptools/command/develop.py", line 114, in install_for_development
        self.run_command('build_ext')
      File "/home/larsh/micromamba/envs/pp_sfm/lib/python3.9/site-packages/setuptools/_distutils/cmd.py", line 319, in run_command
        self.distribution.run_command(command)
      File "/home/larsh/micromamba/envs/pp_sfm/lib/python3.9/site-packages/setuptools/dist.py", line 1217, in run_command
        super().run_command(command)
      File "/home/larsh/micromamba/envs/pp_sfm/lib/python3.9/site-packages/setuptools/_distutils/dist.py", line 987, in run_command
        cmd_obj.run()
      File "/home/larsh/development/pixel-perfect-sfm/setup.py", line 55, in run
        self.build_extension(ext)
      File "/home/larsh/development/pixel-perfect-sfm/setup.py", line 98, in build_extension
        subprocess.check_call(['cmake', '--build', '.'] + build_args,
      File "/home/larsh/micromamba/envs/pp_sfm/lib/python3.9/subprocess.py", line 373, in check_call
        raise CalledProcessError(retcode, cmd)
    subprocess.CalledProcessError: Command '['cmake', '--build', '.', '--config', 'Release', '--', '-j']' returned non-zero exit status 2.
    [end of output]

gcc -v
gcc version 11.3.0

Thanks for reporting this, it is indeed a bug. Just remove " = Eigen::Dynamic" in line 331 and the install should work again.
Please feel free to add a PR with this fix if it solved your problem.


Fastnlight: Blacklist bookcaliforniatour.com

Fastnlight requests the blacklist of the website bookcaliforniatour\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

bookcaliforniatour\.com has been seen in 6 true positives, 0 false positives, and 0 NAAs.

Rejected by Dragonrage in Charcoal HQ. Reason: 'FIRE let me down and didnt tell me there was an open PR :('


Use BufWriter when writing bloom filters and limit tests (#3318)

Which issue does this PR close?

Closes #3318
Rationale for this change

The tests were taking an exceedingly long time to run, this disables them except for a few select tests that explicitly test the bloom filter.
This also adds a BufWriter when writing bloom filters, it is almost certain that this can be optimised further, but this was a quick win that took i32_column_bloom_filter from taking 17 seconds to 2. This is still terrible, but not catastrophic :sweat_smile:
What changes are included in this PR?

Are there any user-facing changes?

Thank you for the quick turnaround


Already up-to-date.
npm WARN package.json<EMAIL_ADDRESS>No README data
npm WARN package.json<EMAIL_ADDRESS>homepage field must start with a protocol.
npm WARN deprecated<EMAIL_ADDRESS>CoffeeScript on NPM has moved to "coffeescript" (no hyphen)
./app.pl assets
perl6 htmlify.p6 --parallel=1
Initializing ...
===SORRY!===
QAST::Block with cuid 25 has not appeared
make: *** [htmlify] Error 1
CC $

That's a new one. You probably shouldn't use sudo, but I don't think that's
got anything to do with that.
What's the perl6 version?

Let me check that. It looks like an error in a version of a particular package.

Oops, closed by mistake.

CC $perl6 -v
This is Rakudo Star version 2018.01 built on MoarVM version 2018.01
implementing Perl 6.c.
CC $
I used the # sudo make  and # make at the same time

You mean you get a different error if you use sudo make than if you use
make?

On the other hand, you would be using two different versions of perl6 if
you are doing it via user or sudo.

Apparently the second problem is due to having an old version of LWP::Simple. The first one is probably also due to a similar problem. Let's focus on the make one, without sudo. Please try and do
zef install --deps-only .

in the main directory of perl6/docs. If that does not reinstall the existing modules, please try and find out the versions of them you are using and get back to us.

The versions of perl are the same, but maybe the module versions are not.
Re-installing LWP::Simple might fix that. Can you try the latest version of
that?

i trying

I use "LWP::Simple:ver<0.097>:auth" get the same results

Do you usually use those convenient chat tools?
It's too inconvenient to communicate through [github issues]

I'm jjmerelo in Telegram and Twitter, in case you want to use that. I am
also often in the #perl6 IRC channel.

Anyway. Baseline is, I can't reproduce your error, so it must be due to some module and/or Perl version. Perl 6 2018.01 should be OK, so it must be some module that has not been updated for some time. Since rakudobrew forgets all installed modules, if you upgrade to 2018.03 and reinstall the modules with zef it might fix the problem. If it does not, please get back to us again, since it might be some OSX specific thing, since #1828

Many chatting tools can't be used in many things in China

Tell me which one... Or just email, if you prefer that.

QQ is popular in China. I don't know if you can use it there.
my QQ Noumber is<PHONE_NUMBER>

I'll give it a try...

Can't do. The only platform it seems to be available for and that I use is
Android, and I really prefer not to do these things using my thumbs. I'm
going to try WeChat.

This tool has been developing for many years in China. It has been super powerful. If you are used to it, you can build a discussion group or group through this tool. The efficiency is very efficient

WeChat can also, but it has no such function, which is equivalent to a streamlined version.

You can add me in the form of WeChat scan two-dimensional code.

OK, done, but I can only use it from the phone. Not ideal...

After talking for a while, looks like it's a Pod::To::BigPage error, and more precisely the one posted in perl6/perl6-pod-to-bigpage#1 It's better if this issue is closed, and that one reopened with the new information.

I had the same crash with an earlier Rakudo, but upgrading to 2018.03 resolved the issue.
I've also just now had a build of 2017.11 rakudo with all the latest modules, and the same error occurred, so I doubt it's just some module that's problematic.
There's no Rakudo Star newer than 2018.01 yet (next one will be 2018.04, due in 4-5 weeks), but if you want you could try building compiler-only from source

@ccworld1000 can you please upgrade to 2018.03 and see if that fixes the issue?

@ccworld1000 did you try and use 2018.03? You can download and build it from source, for instance

There's no response fromp the OP, and a solution has been suggested, so I'm closing this issue. Please feel free to reopen it if you think it deserves more attention.

Well the issue was opened just 16 days ago, and if I got it right there was an issue with the latest rakudo star? We don't have a 2018.04 star yet (not even 2018.04 rakudo release actually), so this issue is probably not resolved yet. At the very least I'd much rather hear something from the OP.

Wouldn't the next Rakudo * be 2018.05?

No, it's Jan, Apr, Jul, and Oct.

Typically it's .01, .04, .07 & .10

Ah, OK. Great. Let's wait for the answer from the OP and/or the new version.

Hi, while 'make install' of Rakudo Star 2018.04, I noticed p6doc 'FAIL - proceeding with force install' or something like that. Rakudo Star installed correctly, and zef subsequntly installed cro and dependencies without a problem. So I thought I'd try to install and make a local copy of the docs, to test the above issue, and found that 'make html' from within the doc directory fails.
make_p6doc_html.txt
Is this the same problem that the OP reported?
bazzaar

I'm looking at it, and I think the problem has something to do with some node module, which is used for highlighting. There are some warnings related to coffeescript, which should have been fixed.
That is in the same ballpark as this particular issue, but it's a different one altogether. I would be very grateful if you would open a new issue, indicating all the particulars. In fact, if there's no response from the OP, I'm assuming this issue has been solved and will close it.

I assume this problem has been solved with the new version. If it's not, please reopen and add the new error message.


User auth - enforce rbac in all new EA related api's

Summary
Apply rbac on all external apps api's, enforce in rest handler's

app listing
helper api
app detail
resource tree
manifest and values

Proposal
How do you think this should be implemented?

Merged


[Fix] Update new SVG file and snapshots for Polygon

Description
Updated new polygon icon svg
Issues Resolved
Resolve #1068
Screenshots
After

Check List

[ ] New functionality includes testing.
[ ] New functionality has been documented.
[x] All tests pass

[x] yarn lint
[x] yarn test-unit

[x] Update CHANGELOG.md
[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

You may want to check similar icons with enclosed areas to see what the issue is.

@joshuarrrr, just updated the polygon svg. Thanks for the heads up.

Thanks @BSFishy! Based on these two sources: clip-rule & fill-rule. I think we can remove the clip-rule and remain the fill-rule.

@Willie-The-Lord Thanks so much for your patience and persistence here.I'm a little torn between approving this and merging as is (because I do believe it fixes https://github.com/opensearch-project/oui/issues/1068. But the pedantic side of me wonders why we're introducing the first

@joshuarrrr I completely agree. It seems like there is no clear instruction on how to defineviewbox, height and width when creating new icons. Should we involve the UX team in this discussion?

The thickness of the lines is more than other 16x16 icons. I feel the original was just scaled while it should have been redrawn.


Ceph volumes cannot be attached to the pods after 1.12

Deploying with the same bundle before/after switching to 1.12 by default.
Bundle: https://pastebin.ubuntu.com/p/TY4YW5myBh/
Error: https://pastebin.ubuntu.com/p/5rwM7hCfq2/
Prior to the version update there was no storage class created by default by kubernetes-master charm, as well as there was not option "default-storage" available (kubernetes 1.11 was used).
I was creating it manually and everything worked perfectly. Now I'm getting 2 cni-XXX storage classes, one of them marked as default. I'm creating Ceph pool, the volumes get created in it and marked as Bound in "get pvc" output, however they cannot be used by pods.
Also, unless I set "allow-privileged=true" to kubernetes-master config, the units cannot succeed with an error while installing addons: https://pastebin.ubuntu.com/p/M8V4FFgCkp/
If I tried to execute same command manually:
sudo /snap/cdk-addons/445/kubectl apply -f /root/snap/cdk-addons/445/addons --recursive -l cdk-addons=true --prune=true --force

the install command exits with "Not allowed by cluster policy"

Running into the same problem after following these instructions. K8s cluster is also version 1.12.

I've opened three PRs to fix the originally reported issue:
https://github.com/juju-solutions/cdk-addons/pull/59
https://github.com/juju-solutions/cdk-addons/pull/60
https://github.com/juju-solutions/cdk-addons/pull/61
The key bit from the error output was this:
2018-10-09 03:50:03.268166 7f75987a0d40  0 librados: client.kubernetes authentication error (1) Operation not permitted

The charms don't create a client.kubernetes user in our Ceph cluster. The template we pull from upstream was updated to specify it, and we left it in: https://github.com/ceph/ceph-csi/commit/b0e68a52e0e533523cd9bf7e2f06aa894ffeafdc#diff-592b6313f1ac4f9644734ca5bc1a0710R27
The PRs fix it by forcing it back to the admin user.

Please note that, while testing this fix, I encountered other issues with Ceph that are equally problematic. I recommend subscribing to them as well: https://github.com/juju-solutions/bundle-canonical-kubernetes/issues/668 https://github.com/juju-solutions/bundle-canonical-kubernetes/issues/669

Fixed by https://github.com/juju-solutions/cdk-addons/pull/59, which has just been released via cdk-addons on channel 1.12/stable. New deployments should work immediately, and existing deployments should update within the next 24 hours.

Also, unless I set "allow-privileged=true" to kubernetes-master config, the units cannot succeed with an error while installing addons


Failed to start LoRa App Server.

After version 0.5.0 I can not run Lora app server. I ask to help, what can be done.

Check the logs? Check the release notes about what has changed and what settings needs to be updated?

It does not start because of postgresql.
line: POSTGRES_DSN=postgres://loraserver:password@localhost/loraserver?sslmode=disable
If you change to drain: POSTGRES_DSN=postgres://loraserver:password@localhost/loraserver?sslmode=disabl
Then lora app server starts with errors.

I have postgresql v 9.6.2

Then lora app server starts with errors.

Which errors? ;-)

Thanks Brocaar, figured out.
Put the jwt-secret and lora app server started.


Fix jsonToReact(JSONArray jsonArray) when parsing boolean

Hi.
Sorry, my last patch contained the wrong variable name in jsonToReact().
This will fix it.
v2.1.7 wont compile, because I'm trying to access a variable named writableMap, but should instead use writableArray.
Best regards,
Christian.

Thanks for the quick fix, and it's included in latest 2.1.8 release.


feat(search): Don't close dialog on ctrl+click at quick search [Option 1]

Hey, I just made a Pull Request!
This draft is a proposal for https://github.com/backstage/backstage/issues/14903. A suggestion to improve user experience while using quick search. The other option is here.
After opening quick search, if you need to go though multiple results by clicking ctrl/cmd + click to open new tabs, the quick search dialogs closes.
With this it is kept open, so you can go through the other results without having to open the quick search again:

:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[x] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

I think option 2 is the way we're heading so I'm gonna close this and we can keep the conversation going there.


Implement difficulty calculation for homestead

Just looked over this.  Looks good.

I've added a TODO for the API to reconfigure start blocks; if you're ok with that I think this is ready for final review/merge

will merge when it is green.  Thanks


Automatically copy the config file from deepforest_config.yml to deepforest/data

When you push to pypi, we package the config file for users. Right now we are copying that over before pushing a new release. This is an accident waiting to happen. I added a test to make sure that it matches.
https://github.com/weecology/DeepForest/pull/530
But we still want to cp this during pypi push.

Just to make sure I understand this issue. We want to include /deepforest_config.yaml in the package. Right now we are doing that by manually syncing /deepforest_config.yaml with /deepforest/data/deepforest_config.yaml and packaging the alter file using include_package_data=True and listing the file in MANIFEST.in.  Is that right?

Yes, that is correct.
On Sun, Aug 11, 2024 at 9:01 AM Ethan White @.***>
wrote:

Just to make sure I understand this issue. We want to include
/deepforest_config.yaml in the package. Right now we are doing that by
manually syncing /deepforest_config.yaml with
/deepforest/data/deepforest_config.yaml and packaging the alter file
using include_package_data=True and listing the file in MANIFEST.in. Is
that right?
—
Reply to this email directly, view it on GitHub
https://github.com/weecology/DeepForest/issues/531#issuecomment-2282805407,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJHBLDORSELM5OSAFCPC4LZQ6DGVAVCNFSM6AAAAABMK3TCMKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOBSHAYDKNBQG4
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Ben Weinstein, Ph.D.
Research Scientist
University of Florida


zds: fix retrying a bad netns

Fixes https://github.com/istio/istio/issues/52858

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

/test all

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

test_ztunnel
2f4c91556b5ff8b14767af13ebd5a3f8ad93f375
link
true
/test test

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

In response to a cherrypick label: new pull request created: #1285


Add deprecation warnings

Introduce remaining deprecation warnings for features that are removed in 5.0.0.
This borrows heavily from how we deprecate features in Neat using Sass @warn: thoughtbot/neat@5c337cc
One key difference from Neat is that we only want to use this mixin for
deprecations, not all warnings. The reason is because with CSS, it's not always
a straightforward task to upgrade libraries or frameworks; particularly major
releases. So we offer a toggle to globally disable deprecation warnings, but for
other meaningful warnings we need to throw, we still output those.
People can disable deprecation warnings by setting a variable to false:
$disable-bourbon-deprecation-warnings: true;

One problem with this system is that we cannot use these deprecation mixins to
deprecate a function. This is because Sass doesn't allow you to include a mixin
in a function. For now, since we only have a few functions up for deprecation,
we can live with this and manually write warnings for those.
The plan is to merge this and release it as a 4.3 release to solely introduce
these warnings.

Whew that was a long one. Code looks good. I have a particular comment about the double negative variable $disable-bourbon-deprecation-warnings and perhaps turning into an explicit positive. Other than that 🍔

Thanks, @jasonramirez!

In case anyone else got confused, the pull request description says:
$disable-bourbon-deprecation-warnings: true;

But the actual code has:
$output-bourbon-deprecation-warnings: true;


Question: Does this plugin support 2-factor authentication?

If a user has facebook 2-factor authentication enabled, would this plugin support that?

No


Add toroidal mapping 3d -> 2d for "textures"

Add a method to translate any point on a torus to a 2d point (2 angles). The first is the angle to the R3::basis::X in the XY plane. The second is the rotation around the donut (find through dot ?)

Claude.ai answered that we should use:
u = (atan2(y, x) + π) / (2π)
v = (atan2(z, sqrt((sqrt(x² + y²) - R)² + z²) - R) + π) / (2π)

Claude.ai is incorrect.


nix: upgrade 2.0.4 -> 2.3.4

The current nix version does not have builtins like fromTOML
and in general lacks behind what we use in stable.
see https://github.com/nix-community/NUR/pull/217

cc'ing all the maintainers:
@domenkozar @garbas @matthewbauer @grahamc

@Mic92 Thank you!

This will break darwin, see previous attempts.

Context: https://github.com/travis-ci/travis-build/pull/1804#issuecomment-550502870

do you know of any project using Travis and doing OSX builds we can test reverting the multi-user part for darwin.

do you know of any project using Travis and doing OSX builds we can test reverting the multi-user part for darwin.

@garbas https://travis-ci.org/github/abathur/resholved/builds/693499850

@Mic92 @garbas @domenkozar I've got a project that tests against macOS if it helps get this PR (or #1833, or #1804?) un-stuck.

I am going to close this one, as I don't want to fix macOS support.
One might have a look at https://github.com/cachix/install-nix-action/blob/master/lib/install-nix.sh to get an idea on how to fix this.

do you know of any project using Travis and doing OSX builds we can test reverting the multi-user part for darwin.

@garbas We do, https://github.com/dapphub/dapptools/blob/master/.travis.yml

@asymmetric Do you have time to look at OSX support on travis? and see what github install-nix action does, that we should also do here?
I'm currently working on other things and I don't see time to fix this in a reasonable amount of time.

@garbas we're already working on this in #1833


Create tags for past versions of resume

For example:

ed1a520555a0528c95dd0e7bf98675aa5204bb0c is when I just graduated, with two pages.
949c83da3d6c2c65b72b4e7a6df624b9ad1b17f9 (at the time of writing) is with my first full-time job, and one page.

Created releases and tags following semantic versioning philosophy.


Recompile on changed locale files

Is there a way to get cargo leptos watch to recompile the lib (load_locales!) when files in the locales directory change?
I tried watch-additional-files = ["locales"] in Cargo.toml but cargo doesn't think that anything in the lib has changed so just links.

Yes this is a known issue: #86, and I don't have a solution yet to this problem.


Throium wont open

System Details

OS: Linux mint 21.3
Thorium Version M124.0.6367.218

Problem
This used to happen a while ago, but by simply restarting my pc it would perfectly work. but now whatever i do when i click on throium or try to run it , nothing happens
Screenshots

Additional Notes
Same thing with the SSE3 package

QT issue. fixed by adding ~/.profile
 export QT_QPA_PLATFORMTHEME=qt5ct
 ``` prob should include/mention this somewhere so people in the future know if something similar happens ig idk

@PhantomRex123 Tested, could reproduce, tested your fix, and it fixed it. Where did you find the fix?
I could add it to the FAQ

@PhantomRex123 Tested, could reproduce, tested your fix, and it fixed it. Where did you find the fix?
I could add it to the FAQ

So its kinda weird, the issue starting when i changed from gnome to kde, and by kde setting the qt5 theme it failed to work, now when i saw the issue searched it up on google and found a website which i cannot find anymore lol however found this similar post https://forum.manjaro.org/t/stable-update-2021-07-23-kernels-nvidia-mesa-wine-firefox-thunderbird-virtualbox/74937/28 and i think by setting it to rather than qt5, the reason why i said is that after setting it to qt5 my whole kde setup gets messed up
QT_QPA_PLATFORMTHEME=thorium-browser

now its weird cause when running this same exact configuration it seems that with specific qt themes the browser refuses to work

So i'll do a bit of testing here and there and try and update you on what works and what doesnt, cause im not very sure myself right now.

As for the images, it seems right now on my "gtk2" theme and "Windows" themes the browser doesnt work but the rest are working

after further testing ive concluded that its the theme, and by having that line QT_QPA_PLATFORMTHEME i guess it just sets the default theme or something along those lines, but i removed that and with the "Fusion" theme it works, so right now only windows and gtk2 dont work and i assume its because they arent official so might aswell include this so people know

even with this error thorium works - so im fairly confident its something with the themes

Setting export QT_QPA_PLATFORMTHEME=qt5ct in ~/.profile somehow did not work in my case, even though qt5ct was installed on this Debian derivative operating system.  However, qt6ct was also installed, and launching thorium-browser from a terminal did work, as follows  (not as root;  I think it is not recommended to run a browser as root):
env QT_QPA_PLATFORMTHEME=qt6ct thorium-browser


Atlas-drawing module extensibility?

Hoping someone on the SDK team can answer this one.
I've asked a few questions in regards to drawing functionality, both on stackoverflow and learn.microsoft.com.  The previous program manager (rbrundritt - thank you) has been kind enough to answer and informs me that the functions I require are not possible (without some hackery on my part) and not a priority for the team to extend the current library either.  I understand projects have priorities, resource limitations, etc.  However, my requirements don't go away with this knowledge.
I'm left with a quandary.  I must use Azure Maps as I must use the indoor map feature.  No other solution on the market supports what I need so I am committed to Azure Maps in that regard.  However, I also need slightly more enhanced drawing capability for polygons than is currently available.
I am looking for a user experience similar to what's available in MapBox drawing and extension modules when it comes to both drawing and editing a polygon.  Specifically the following:

A delete-vertex whilst in Edit Geometry mode. Mapbox drawing supports this with a vertex single click and click of trash icon.  This can be experienced at geojson.io.  Also, the user should be able to cancel edits and have the original shape unchanged.
A snap-to-vertex when creating a new polygon and editing an existing one.  For clarity of what I am looking for, the behaviour as shown in the mapbox extension mapbox-gl-draw-snap-mode will show that this is different to what the atlas-drawing snap capability is, meaning this happens whilst editing.
Ability to draw holes in existing polygons.  This isn't as much of a priority as the first 2 features but something that would be useful in some use cases.

So to my options. Assuming the team cannot supply the time or resource to extend functionality currently, can the following be answered please:

Are there plans on near horizon to make the drawing module open source, and if so how soon?  Not saying this solves my problem but I can at least make the changes I need in a fork.
If the answer to 1 is no, could this functionality be on the current SDK roadmap within the next 6 months?  6 months + implementation time possibly could still be something I can work to but only if it's a certainty.
If the answer to 2 is no, am I prohibited by any SDK licensing to writing my own equivalent of mapbox drawing that works with the underlying atlas.js, whether it be a new internal or OSS module?  A replacement for the current atlas-drawing.js essentially.  Not saying this is even pragmatic or possible but I need to rule out the licensing issue first.  I'd much rather be writing business functionality so this is not my preference by any stretch.  Just looking for a yes you can, or a no you cannot do that.
Any other options anyone can think of?

Thank you in advance.

Most of these where a part of the original thinking for this feature as future extensions, but that never happened and not sure if or when they might bubble back up in priority (I'll try and get some clarity internally). That said, I have been considering creating an open-source module that adds these extended capabilities and adding it to the list of open source modules for the web SDK: https://github.com/microsoft/Maps/blob/master/AzureMaps.md#open-web-sdk-modules
I still have my original specification document, here is a brief list of additional features I had in mind. I have additional details for how most of these would function in my spec but there's a lot of internal info in that doc I can't share, so would have to do a cleanup before sharing it:

Remove coordinates

On right click or long touch a drag handle displays a small popup that has a delete button in it. If the button is pressed, delete the coordinate. This popup should be hidden if the user mouse’s down or touches anything else on the page or exits the edit mode.
User tabs to the drag handle via keyboard, then presses “-“, remove the coordinate. Alternatively, if it is easier, show the popup with the delete option so the user can easily tab into it.

Edit Properties – A UI where additional metadata can be added to a shape. (This had mostly been built out, just not completed. Code is in the SDK today).
Copy/Paste – make it easy to duplicate a shape. Possible resource https://learnosm.org/en/hot-tips/copy-paste/
Draw/Edit – MultiPolygon, MultiPoint, MultiLinestring
Cut holes – Provide the ability to draw a polygon on top of another polygon but it become a hole of that polygon. Support this for circle and rectangle, but those would output a polygon. For this and line splitting we will need a library that can perform Boolean operations, this might be an option or this
Cut/Split shape – If the shape is a linestring, polygon, circle or rectangle, let the user draw a linestring across it which will then be used to split the shape into two. Circle and possibly rectangle would be converted into a polygon because of this.
Magic wand – Let the user click a point on the map (possibly an image overlay they have added) and do a flood fill based on color (with tolerance) on all surrounding pixels to generate a polygon. (i.e. building floorplan overlaid on the map. User clicks inside a room and we generate a polygon of that room. Possible have artifact removal tolerance to remove text).
Orthogonalize Polygon –If the user draws a polygon it might not have perfectly square corners, this would straighten it out. OpenStreetMaps does this for in their drawing tools for building footprints. Possible code1, possible code2. Also investigate potential of supporting this for lines.
Merge/Union/Combine of Shapes – Possibly limit to like types (points, lines, or polygons but not a mix). Disconnected polygons would become MultiPolygons, but if the two polygons overlap, they would simply become a new single polygon. Have option to select by either clicking on individual shapes or by drawing polygon and getting all shapes within.
Make Valid – Run some algorithms over a drawn shape to ensure that it is a valid spatial object (no kinks/overlapping edges/holes inside…) so that it can easily be imported into other spatial platforms, such as SQL server.
Measuring tools – Simply add measurement numbers to the drawing tools; lengths, radius, area. This ended up being easy enough to achieve with the drawing tools as is, so not needed out of the box. But if we had an extension module that is loaded in addition to the drawing tools module, this might be a nice easy thing to add out of the box.

Some other ideas that were being considered but hadn't been scoped out:

Undo/Redo – Nice to have but likely would add a lot of complexity. Use Z/R keyboard shortcuts like ESRI.
Draw/Edit – ellipse, curve/spline
Settings panel UI – There will be many options for the drawing tools such as rendering styles and snap grid size, in addition to being able to set these programmatically, also provide a UI where the user can modify these.
Scale/Resize shape – with/without respecting aspect ratio
Snap to geometries within the base map. We have road data, if the user wants to draw a polygon that has an edge that follows a road, can we make this easier to achieve.
Draw elevation – When we support extruded polygons we could support grabbing a polygon and stretching it vertically. Possibly offer 3D measuring tools like ESRI, however since we don’t have ground terrain this won’t be accurate, thus lower priority for the web control (Cesium might have a tool for this if customers need this scenario).
Mirror shape – create a copy of it that is a mirror version.
Geodesic Densifier – Add points between each set of coordinates that approximate a geodesic path.
Buffer shape – Apply a distance buffer to a shape. This can already be achieved using Turf.js, but we could add a UI for setting this.
Draw arrow – Create a polygon of an arrow and add to map. Rotate/drag already supported.
Flip shape – horizontal/vertical
K-mean clustering of data points.
3D drawing – Sphere?
Styles – colors, patterns, icons – have as P3 as this is configured at the layer level, not the data level necessarily.
Bring forward/back – Similar to power point and would change order within the data source. However, this would be limited to be limited to shapes of the same type due to layering system. (i.e. you could move one point below another point, but not below a polygon, unless the layer the polygon is in is already above the point layer).
Box Select – When in this mode, disable box zoom and let the user use shift+click+drag to draw/select shapes on the map.
Fast Rectangle drawing option – Consider adding a rectangle drawing mode where the user draws two lines for faster drawing of rotated rectangles. Mouse down/drag for first line, on mouse up the second line should move with the mouse perpendicular to the first drawn line.  When the user clicks the mouse a second time. The rectangle is completed.

One important thing is that accessibility support has been a key focus of these drawing tools (and the Web SDK in general). The Azure Maps Web SDK is one of the most, if not the most, accessible map control currently available which is important for a lot of businesses.

@rbrundritt thank you for the most detailed response.  You are obviously very knowledgeable in this area.
I hope you have some luck with the team.  If not, I for one would love to see an open source extension with those capabilities.  And of course I understand the push to keep alignment with accessibility regardless of what implementation manifests.
Would you see this hypothetical extension incorporating or replacing atlas-drawing?  I ask due to the encapsulation in the current package kinda being what limits me from extending something myself.
As for the features you described, I like them all, but the Make Valid is something I would also make use of beyond my original short list.  Snap to vertex helps to create non-overlapping polygons but would be nice to select a set of polygons and adjust nearby vertices accordingly, or even just to highlight/style those that are overlapping.  Validating overlaps in a polygon layer would be really useful to me in a business context.

Ideal it would extend the existing library, but I agree that unless a few bugs are fixed and a few small things aren't exposed, it would be hard to build something that can both extend and support all these scenarios. I'm still waiting to hear back from the Azure Maps team on their plans. Once we know, we can then come up with a plan of our own. Perhaps atlas.AdvancedDrawingTools will be a new community module.

@rbrundritt just checking any chance there's been some feedback from the team on this issue?

I think moving it open source would be best from my perspective, considering no published plan ahead for even the basic functionality I need pretty soon.

@rbrundritt is the SDK based/layered on maplibre by any chance, and would that also mean the underlying map instance is available?  If so I was wondering does that mean mapbox-gl-draw is something that could be consumed for drawing purposes?  I'm only thinking short term to get me past my immediate needs is all.

Yes, Azure Maps uses MapLibre under the hood for rendering. The native map instance is access (not documented) but can be accessed using the path map.map, where the first map is your Azure Maps instance. In theory the mapbox-gl-draw library could be used, although there may be some issues since the mapbox namespace isn't exposed in Azure Maps, so if this library tries to access some class, it might throw an error.

Thanks for that. I understand why some things may not work. Would have to trial and see.


Update to KoreBuild-dotnet

Blocked by dotnet/cli#710
@victorhurdugaci

Niiiice!

Is it blocked?

Yeah, .testPublish doesn't contain sqlite3.dll so net451 tests fail.

Added the issue to my tracking list here: https://github.com/aspnet/Universe/issues/324

@bricelam can you workaround this based on the suggestions here?


Validation fix

Category

[ x] Content fix

Docs Build status updates of commit fd6cb29:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/schema/localizeddisplayname-element-in-localizeddisplaynames-subscription-schema.md
:warning:Warning
View
Details

docs/schema/localizeddisplayname-element-in-localizeddisplaynames-subscription-schema.md

Line 2, Column 1: [Warning: ms-prod-and-service - See documentation] Only one of the following attributes can exist: 'ms.prod', 'ms.service'. Use ms.prod for on-premise products, or ms.service for cloud services.

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

Global effort to fix build validation errors
@VesaJuvonen
The Content & Learning team is fixing build validation errors on docs.microsoft.com for the rest of H2. This will eliminate potential accessibility, security, and usability issues. This PR includes only build validation fixes and does not change other content. Please review within five business days and merge or comment in the PR with any changes you'd like to see. Thank you!


Detecting and failing on old MSBuild versions

The dotnet/roslyn repo requires the MSBuild from at least VS2017 update 2 to compile. This the earliest version that has a number of SDK bug fixes that we depend on. When built with earlier versions of MSBuild we end up with some fairly cryptic error messages like
RuntimeIdentifier must be set for .NETFramework executables

This means customers with RTM builds of our product come up to our repos, build and get crpytic error messages. Feel like that makes our build process look broken / flaky. Trying to come up with a better experience here.
The ideal is that when you run msbuild on our solution you end up with one or more messages that say:
This project requires at least MSBuild from VS2017 Update 2.

That would be actionable by customers and save a lot of headache on our part. Think this would be really easy to do in a BeforeBuild Target / Task. But I can't figure out the best way to map an installation of MSBuild to say VS2017 update 2. Looking on my machine I can see the file version is 15.1.1012.6693. So I could just say if the version is greater than or equal to that then you're at least update 2. Feels a bit hacky though. Is there a better way of doing a version comparison on MSBuild binaries?

CC @tannergooding

We have $(MSBuildAssemblyVersion) . . . but it does what it says, and isn't the file version that actually changes. We should introduce (and document) a $(MSBuildVersion).
But that's just a nicer way to expose the information you're already looking at. We do tag our releases here with the VS version they correspond to, so you can manually correlate in that way.
You could possibly correlate with $(VisualStudioVersion), after https://github.com/Microsoft/msbuild/pull/2258 is fixed (in 15.5). Internally that's a headache across branches of course.
⚠ As we've discussed offline, there can be reasons beyond the MSBuild engine + common.targets version that you can depend on. Having a heuristic for MSBuild version will probably catch most things especially for non-Microsoft users, but to be fully accurate you'd need {MSBuild engine} + {dotnet sdk} + {any additional installed SDKs like VSSDK} + {compiler} + ... versions.

We should introduce (and document) a $(MSBuildVersion)

That would definitely help the scenarios that I'm interested in.

We do tag our releases here with the VS version they correspond to, so you can manually correlate in that way.

Perfect. Thanks!

You could possibly correlate with $(VisualStudioVersion),

That's a MSBuild variable that I generally avoid, possibly incorrectly. In my experience it's only set when building through Visual Studio while a lot of our builds are driven strictly through command line. Or maybe I've just gotten unlucky here and it's intended to be set more often.

Having a heuristic for MSBuild version will probably catch most things especially for non-Microsoft users, but to be fully accurate you'd need {MSBuild engine} + {dotnet sdk} + {any additional installed SDKs like VSSDK} + {compiler} + ... versions.

Yep. 100% agree.
Been asking other teams similar questions to this one today 😄

You could possibly correlate with $(VisualStudioVersion),

That's a MSBuild variable that I generally avoid, possibly incorrectly. In my experience it's only set when building through Visual Studio while a lot of our builds are driven strictly through command line. Or maybe I've just gotten unlucky here and it's intended to be set more often.

I also tend to avoid it so it's hard for me to say this is a bad or wrong policy. The linked #2258 will ensure that it gets set all the time (in future MSBuild versions), but it's not very granular--it'll be 15.0 for the lifetime of the VS2017, so it won't help achieve your stated goal.

Took a stab at implementing this for Roslyn and I think this represents roughly the work required:
https://github.com/dotnet/roslyn/pull/21218/commits/cfb19ce4616c5b573ab82ded92b3a8add3f0b81a
Recent conversations with the SDK team make me feel even stronger that this is a feature we should have in MSBuild. There are now at least three different functionality levels of MSBuild within 15.0:

15.0: brings SDK 1.0
15.1: brings SDK 1.1.
15.3: min-req for SDK 2.0

This means using 15.0 to compile SDK 1.1 or 2.0 ends in a lot of cryptic error messages. Hence any repo that moves to SDK 2.0 is basically asking to confuse users until they upgrade to a new MSBuild version. Feel like we're really doing a disservice to our users there.

@jaredpar is your requirement to check for the actual msbuild version or the loaded version of an SDK (e.g. Microsoft.NET.Sdk)? The latter could be completely independent of the host msbuild version depending on what the resolvers resolve to.
From what I've read, I think the plan is still to ship VS 15.3 with MSBuild 15.3 and SDK 1.1 but include an SDK resolver that detects installed versions of the 2.0 SDK - so unless someone installs the 2.0 SDK, checking for MSBuild 15.3 is pretty useless if you depend on 2.0 SDK features. The mono version of MSBuild uses the same resolver.
While the SDK introduced a UsingMicrosoftNETSdk property, it may make more sense to add a version property as well to allow for checks. (this may be especially useful for nuget packages containing props/targets)

@dasMulli

is your requirement to check for the actual msbuild version or the loaded version of an SDK (e.g. Microsoft.NET.Sdk)?

Eventually both. This particular thread though I'm focused on the MSBuild version checking. The SDK team, as you pointed out, already provides a way to somewhat check the version already.

so unless someone installs the 2.0 SDK, checking for MSBuild 15.3 is pretty useless if you depend on 2.0 SDK features.

It's actually very useful. If you depend on the 2.0 SDK then you also implicitly depend on MSBuild 15.3. Hence it makes a lot of sense in your "BeforeBuild" targets to ensure you're on a minimum of MSBuild 15.3. Otherwise why bother with anything else in the compilation? It will just fail with a cryptic error that no one will understand.


WIP: Bun support

Bun is a new JavaScript runtime with a native bundler, transpiler, task runner and npm client built-in. This PR aims to add support for it.
All tests except the following pass:

serves static files
doesn't fully buffer binary stream
sends multiple cookies

1 requires a static file server. Bun seems to have some support for it (with bun dev) but seemingly not as part of its HTTP server.
2 requires streaming support over web stream API without buffering. It seems that Bun does some buffering.
3 requires Set-Cookie related non-standard functionality in the Headers class. Our polyfill doesn't seem to work.
Also I don't know how to get the IP address issuing the request.

I added static file serving and X-Forwarded-For support. I'm gonna go ahead and merge it since it is now fairly usable (as much as Bun itself is usable):

I'm fairly certain Bun doesn't offer a way to determine the client IP but X-Forwarded-For mitigates it somehow when behind a proxy.
Set-Cookie is also an upstream issue and probably not that critical at this stage.
I don't know why streaming doesn't work, I'm gonna ask at Bun's Discord server. But I'm fairly certain the code on our side is correct (because it works on so many different platforms already) and it's also an upstream issue.


Brightness profile uncertainties

The brightness profile values need to have uncertainties associated with each point. Initial thought is to just assume poisson errors.

The spirit of this changed when I re-assigned it to stacked brightness profiles. Robust individual brightness profile uncertainties are something I will get back to later, and may well change in my stacked profiles, but for now the covariance matrix will do.


Added table exports to Externs

Fixes issue #17
Include table exports into the Externs instance

Ah yes, sorry about that

@jlgaffney thanks for this!


There is no readme

do you are have stupid, mayhaps?

i'm sure you can do it

i'm sure you can do it

ok

ok


Fixes 'spo field get' and 'spo field remove' commands. Closes #3480

Fixes spo field get and spo field remove commands. Closes #3480
Marked the fieldTitle option as deprecated and introduced a new option called title.

Thank you @nanddeepn we will review shortly 👍

ready to be merged during next PRs review 👍
thanks for your awesome work 🤩

merged manually. Thanks for your awesome work 👍


Added Hindi translations

I have added Hindi translations (hi) to the packages. Below are some screenshots.

Thank you!


still need data dir when using alias option?

I just pulled pubget and installed it with pip install -e . in the pubget folder
I ran pubget run -q "fMRI[Abstract] AND music[Title]" --labelbuddy --alias fmri_music
I got this error: pubget run: error: the following arguments are required: data_dir

On the poster, we use the alias option without specifying a data directory. Is this how it's supposed to work? Or do we need both? If so, maybe we should update the code on the poster.

sorry I missed this! yes we still need the data_dir, which is the root of all pubget data. the alias symlink and the query directory go inside the data_dir. As you know we fixed the poster. what we could consider is have a default for the data dir, such as the current working directory. It would certainly make some examples look a bit simpler, but I think in most cases users will want to specify a specific data directory. WDYT?


markdown-it dependency is insecure

At time of writing, this package depends on<EMAIL_ADDRESS>CVE-2022-21670 reports that versions of markdown-it prior to 12.3.2 are insecure, so it would be good to update the dependency in this package to a secure version.

The seems to just be a performance-impacting issue. Fairly busy as of right now, so don't think I can spend time fixing this any time soon. Happy to review and merge in a PR if you fork it and update markdown-it.

See #10 - I'm not sure how best to check the upgrade doesn't break anything...


Nat Network Support

This PR contains the functionality in #13 and #11, rebased onto master and updated to use the Mange().run functions.
I am using this functionality successfully with a patched version of the terraform provider, and a couple of other minor patches to this library (to be introduced in different PRs).

Hi @jpts, Since isage no longer works on #11 I have closed the PR. I have also asked patsevanton on #13 is this PR a more complete version. Lets just wait on their response, and if there is none, I will close #13 and merge this change.


Fix LocalDateTime test

The test tries to cast a date to it's current day - 1, but that sometimes causes the day to be zero (only on the first day of every month).
This test was not correct, and this commit fixes it.

@EarthCitizen, @reckter and @sksamuel FYI. Merging when green


bug(#37 queue and context fields were not set)

now set in init method

Would it be ok to set these in the constructor?

I believe so, merging and publishing this - let me know if something comes up!
@worldspawn
Great job 💯 👍 👍

These are set within .init(), since sometimes we don't have everything setup right away.
Is that not working? @worldspawn

I can't see where that happens - can you link me a line?

Oh my goodness, sorry about that - I was super tired yesterday!
The code I looked at was from what you added 😆 🤣
Merging this and publishing a new version, sorry about the mix up!!
You rock 🎸


Bungeecord network bug

Description
I setup a Bungeecord network and MySQL server and also setup MySQL in Luckperms (in the config of each server).
The database is working but I have one problem:
when I created a group with lpb creategroup name and then tried to use lp group in one of the spigot servers the group didn't appear in the list.
Also when I used lp group info name it worked.
Reproduction steps

Setup a Bungeecord network and MySQL server.
Setup MySQL in luckperms config in each server.
Create group in the bungeecord server with lpb create group name and try to use lp group in one of the spigot servers.
the group doesn't appear in the list of the lp group command.

Expected behavior
I expect that the group will be appear in the list
Environment details
I have three servers:

Server type/version:  This server is running Spigot version 1.16.4.
Luckperms version: v5.3.13.
Server type/version:  This server is running Spigot version 1.16.4.
Luckperms version: v5.3.13.
Server type/version:  This server is running Bungeecord version 1.8.x - 1.16.x.
LuckPerms version: v5.3.13.

Is it listed in /lp listgroups after running /lpb networksync?
What is sync-minutes set to in your LP config? Both in the server and proxy.

Is it listed in /lp listgroups after running /lpb networksync?
What is sync-minutes set to in your LP config? Both in the server and proxy.

Yes It is listed.
The sync-minutes set to -1.

Also this is listed without /lpb networksync.
it dosen't appear in the tab complete but still working.
My Problem is with the tab complete in commands that have groups list in tab complete.

This is because LP does not send update changes when creating a group (it sends the action logging but not the actual "hey this group has updated" message) and I noticed that running /lp listgroups does "fix" it since it reloads the groups list.
Not sure why Luck decided to do it this way? I want to know what his thoughts are.
For reference (no messaging service update and same applies for other group-changing events like the deletegroup command):
https://github.com/lucko/LuckPerms/blob/f8131224682d84922da8fcdb84f97e2e8393c42b/common/src/main/java/me/lucko/luckperms/common/commands/group/CreateGroup.java#L86-L110

This is because LP does not send update changes when creating a group (it sends the action logging but not the actual "hey this group has updated" message) and I noticed that running /lp listgroups does "fix" it since it reloads the group list.
Not sure why Luck decided to do it this way? I want to know what his thoughts are.
For reference (no messaging service update and same applies for other group-changing events like the deletegroup command):
https://github.com/lucko/LuckPerms/blob/f8131224682d84922da8fcdb84f97e2e8393c42b/common/src/main/java/me/lucko/luckperms/common/commands/group/CreateGroup.java#L86-L110

OK, Thanks.
I will create a plugin that will run lpb networksync command every second to fix that.

That's not ideal... if you're going to make a custom plugin anyways I suggest you use the LuckPerms API and listen to the GroupCreateEvent and GroupDeleteEvent and push updates through the messaging service when that happens.
You can find more info on how to use the LP API on these two pages:
Introduction: https://luckperms.net/wiki/Developer-API
Usage (overview): https://luckperms.net/wiki/Developer-API-Usage
Example plugin for reference: https://github.com/LuckPerms/api-cookbook
JavaDoc for more info: https://javadoc.io/doc/net.luckperms/api/latest/index.html
You can join our discord server if you have any more questions about the API :)

This is hardly a critical bug that renders the plugin unusable...

I will create a plugin that will run lpb networksync command every second to fix that.

Please don't 😅
Resolved by the commit above.


ForceUpdateTag property should be mentioned

The How to run custom script more than once with CLI section should be extended by adding reference to the ForceUpdateTag property that can be used to execute the script multiple times.
See https://docs.microsoft.com/en-us/dotnet/api/microsoft.azure.management.compute.models.virtualmachineextension.forceupdatetag?view=azure-dotnet

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

ID: 40aaccb8-19ba-168f-f5bf-65f1f5331541
Version Independent ID: 6cdfd166-633a-50a3-2b45-cde0e991e540
Content: Azure Custom Script Extension for Windows
Content Source: articles/virtual-machines/extensions/custom-script-windows.md
Service: virtual-machines-windows
GitHub Login: @danielsollondon
Microsoft Alias: danis

Thanks for the feedback! We are currently investigating and will update you shortly.

@hobpet I will assign to the content author to review and determine if we should include that information.
@danielsollondon can you take a look?

Document updated.  Please comment below if there are any issues.


Configurable file extension to support other files like .jsx

Using an option, we can have a default regex that can be overridden for matching filenames.
We'll replace
      ).sort().filter(path => path.endsWith('.js'));

with
.filter(path => /.jsx?$/.test(path))

Replaced the matcher with one that simply checks for a file extension. This can be overridden in the plugin config:
rails({ matcher: /\.jsx?/ })

Released in 1.0.3


Switch from ktlint -> ktfmt

It seems like ktlint keeps triping me up because it doesn't support running as part of the workflow (I make a small change, forget to fix some spacing then get a failure). I'd like to try out https://github.com/facebookincubator/ktfmt.
I know the maintainers and would love to give it a try

Hmm looks like there's no published artifact yet and it requires maven 🙃?

Yeah, I would check in a binary
On Wed, Feb 19, 2020, 6:05 PM Yang<EMAIL_ADDRESS>wrote:

Hmm looks like there's no published artifact yet and it requires maven 🙃?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/dropbox/Store/issues/116?email_source=notifications&email_token=AAMUYKLDDG5Z33KBFIY6FHTRDXQPLA5CNFSM4KX6B6UKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEMKOKGI#issuecomment-588571929,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMUYKK3SGPCIXJNGX2JJMTRDXQPLANCNFSM4KX6B6UA
.

ktfmt is not a full replacement, right? ktlint is a formatter but also a linter.

@ychescale9  @eyalgu  we started publishing artifacts to Maven Central: https://search.maven.org/artifact/com.facebook/ktfmt/0.12/jar
@tasomaniac  correct; ktfmt is a code formatter. It's analogous to google-java-format in the Java world.

https://github.com/diffplug/spotless/pull/569 is bringing ktfmt to Spotless. I see Store already uses Spotless, so it should be easy to try out ktfmt.

Yeah it is a good issue, require a discussion for it.


Warning when a player logs in with the same IGN as another player but different UUID.

Description

Recently came across a warning/error when a player logged in with the username "Imperial", if you don't know this IGN is actually tied to two different UUID's/Mojang accounts. This causes LuckPerms to throw a warning in console when they both login or when one logs in after the other. I'm unsure of what consequences this may have on luckperms
Reproduction steps

Get an account with the same IGN as another player.
Login both accounts to the server.

Expected behaviour

I'm not sure to be honest, Because of the consequences of running LP commands on these players (how does it determine which one to run it for if not using UUID's) perhaps LuckPerms should not allow these accounts to join the server?
Environment details

Server type/version: Latest Waterfall/Bungee
LuckPerms version: 4.4.31 (yikes i know its old)

https://gist.github.com/BrodyBeckwith/e55b9d0372cd5dad88eae23a9536052a

That old version is no longer supported.
So unless you can recreate that issue on a modern LP version it’s pretty much mute.

That old version is no longer supported.
So unless you can recreate that issue on a modern LP version it’s pretty much mute.

That old version is no longer supported.
So unless you can recreate that issue on a modern LP version it’s pretty much mute.

I get same warning on 5.1.86 on one of our backend servers. I assume this would hold true for even the current version of LuckPerms as well. I see no commits which would of done anything about this?

That old version is no longer supported.
So unless you can recreate that issue on a modern LP version it’s pretty much mute.

I get same warning on 5.1.86 on one of our backend servers. I assume this would hold true for even the current version of LuckPerms as well. I see no commits which would of done anything about this?

You can either:

ignore the warnings
prevent those players from joining your server

I'm reluctant to remove the warnings - because well, they are warning you about a valid issue lol
Plugins simply are not going to behave correctly when two players are able to join the server using the same username. LuckPerms should be mostly ok, but bear in mind if you use their usernames to make changes using commands, it's probably going to actually apply the change to whichever user logged in most recently.

You can either:

ignore the warnings
prevent those players from joining your server

I'm reluctant to remove the warnings - because well, they are warning you about a valid issue lol
Plugins simply are not going to behave correctly when two players are able to join the server using the same username. LuckPerms should be mostly ok, but bear in mind if you use their usernames to make changes using commands, it's probably going to actually apply the change to whichever user logged in most recently.


feat: experiment with a new setting widget builder

This will create more widgets per setting, but the goal is to more simply allow us to add various free text to settings that need them.
I don't plan on migrating all settings, but I migrated all the enum dropdowns so I removed that implementation.
It's not really a builder pattern but it tries to be similar to it


On step 6, the ellipsis do not appear in my Azure Portal, and the table is not being created

Hello
I have an issue whereby I set up ETW and the WADETWEventTable is not being created under the storage account (although the Perf and Log tables related to the diagnostics extension are).
So I went back and the obvious thing to do is check step 6 above, however, the  ellipsis does not appear so I cannot check if the destination table name.
Please advise

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

ID: 27d373d7-cd58-4a33-382c-469aa8f17432
Version Independent ID: c4e10ac5-6ab2-b609-9f6b-d48c9e00e1c0
Content: Collecting Event Tracing for Windows (ETW) Events for analysis Azure Monitor Logs - Azure Monitor
Content Source: articles/azure-monitor/agents/data-sources-event-tracing-windows.md
Service: azure-monitor
Sub-service: agents
GitHub Login: @guywi-ms
Microsoft Alias: guywild

@CXMelga
Thanks for your feedback! We will investigate and update as appropriate.

@CXMelga Thanks very much for your feedback!
@Lukeout the UI seems to have changed. Please advise.

An internal work item has been opened to address this issue: USER STORY 261684
#please-close


Add a clear button for number input

Fades in (or maybe pops in playfully) on the left when the user focuses on a field, should never be obscured, should fade in over the text when there's lots of content
After clearing (do a cool animation), show an undo button for a moment (3 seconds?)

On focus, automatically empty a field that's just 0.00


preLoginFlow fails

[x] I've searched the Issues
[x] I've read the basic concepts
[x] I'm using the latest version

preLoginFlow started to fail because of one of the requests in it, guessing it is msisdnHeaderBootstrap
Fails with error
POST /api/v1/accounts/msisdn_header_bootstrap/ - 401 Unauthorized; Please wait a few minutes before you try again

I thinks it's better to include try/catch block in the example here

you can remove unused methods from it.

you can remove unused methods from it.

Does this fix the login issues that are happening?

Try, preloginflow is optional.

I had the same issue. I find this is because I was login a multiple times. To reset the issue I connected my PC to my phone shared connection, rerun, login ok. Then switch to your wifi back.
Source: https://www.alphr.com/please-wait-a-few-minutes-before-you-try-again/

There was the same error when I threw it on the hosting, although when I started it locally there was no error and the bot was logged in, I removed this waiting.simulate.preLoginFlow() line from the Client file.js and everything worked, thanks.


Use this card with a simple bluetooth dongle instead of an ESP32 device

Hello :wave:
In case it can be of any use to others, I wanted to get this card working with a simple bluetooth dongle instead of having to buy a separate, ESP32 device.
I've created the following repo: https://github.com/maxime1992/linak-2-mqtt Everything is explained in the readme :+1:
(feel free to close that issue anytime :+1: just thought it may be useful for other people!)

Hello. I'll take a look how it can be adopted :)

Looks like you make you component work with my card. If so, and nothing else required from my side I can at lest add link to your project in README.md. Thanks 😄

Yes I'm indeed using your card (wouldn't have spammed you otherwise :angel:).
The only thing that's not working is the up and down arrow but it should be simple enough for someone else to do that binding and raise a PR if they feel like it. All I was interested in was the sit and up positions so these 2 are working for sure and the current height is displayed correctly as well.
Good idea for the readme :)


Cannot download OmniSharp behind proxy/firewall

From @TheBlueSky on July 18, 2016 4:37
Hello everyone,
I'm not sure if this is a VS Code or OmniSharp issue, so feel free to move it to the right repository if necessary.
When I open a .Net project, I get the following in Output window:
[INFO] Starting OmniSharp at 'c:\Users\xyz\Documents\My Web Sites\SomeProject'...
[INFO] Installing to C:\Users\xyz\.vscode\extensions\ms-vscode.csharp-1.2.2\.omnisharp
[INFO] Attempting to download omnisharp-1.9-beta11-win-x64-net451.zip...

And it gets stuck there. Nothing is downloaded and my project doesn't have any kind of Intellisense (obviously).
Maybe this is relevant, and maybe not, but at work, we are behind a firewall and a proxy server (from Fortinet) that does strange stuff. One of which is using its own certificate for all HTTPS requests, which I faced an issue with when using an application that require a valid  certificate signed by trusted CA, or an application that uses certificate pinning technique.

Open a .Net project :)

Copied from original issue: Microsoft/vscode#9427

Hi @TheBlueSky, have specified your proxy settings within VS Code?

No. The kind of firewall/proxy that we have in the company does not require specifying proxy settings. You come in the morning, login to the firewall/proxy, and that's all.
I started to face this issue only after v1.0 of .Net Core/ASP.Net Core (not sure what the corresponding OmniSharp or VS Code versions were). Haven't faced this issue during DNX days.
My memory isn't clear whether I tested it with ASP.Net Core RC2 or not; I just know it never worked after v1.0.

The extension just uses your default proxy settings unless you specified something different. I assume other operations work? For example, dotnet restore on a new .NET Core project at the command line?

Yes, dotnet restore works from command line, whether using Command Prompt or VS Code terminal. From the VS Code Command Palette however, it doesn't; I'm getting "OmniSharp server is not running."

Yes, the VS Code command palette uses OmniSharp to determine which projects to run dotnet restore on. I'd like to dig into the proxy issue deeper, but I think there's an additional problem that will keep this from working in the meantime. It appears that you're on 32-bit Windows, correct? Unfortunately, at the moment, the OmniSharp language server only supports 64-bit Windows.

At work, I'm using Windows 8.1 Pro 64-bit... at least this is what Windows reports :)

Great! In that case, try the following workaround to get things working:

Download and unzip "omnisharp-win-x64-net451.zip" from the latest OmniSharp release.
In Visual Studio Code, select File->Preferences-> User Settings.
In user.settings, add the following:

"csharp.omnisharp": "C:/Path/To/The/OmniSharp/You/Unzipped/OmniSharp.exe"

Restart Visual Studio Code.

Now, try opening a .NET Core project and ensure that things are working.

Thanks @DustinCampbell.
So, this worked, as a workaround. Do we know the root cause of the issue? If not, what can I help with?

Nope, I don't know the root cause, but it smells like a proxy issue. I'm glad you're up and running for now though.

We've done a lot of work to improve this in the latest release to respect http.proxy and http.proxyStrictSSL values. With https://github.com/OmniSharp/omnisharp-vscode/pull/958, I'm hopefully that our last issues with this should be addressed. Let me know if you experience this problem with the latest release (1.5.2).

@DustinCampbell, unfortunately, this didn't address the issue and OmniSharp is still unable to get the required dependencies/packages... it's the only thing that doesn't work in VS Code behind our firewall, I mean from the options I tried; e.g.; updating VS Code itself, updating its extensions, dotnet restore, and maybe couple of others.

Do you set the http.proxyStrictSSL setting in VS Code? We recently addressed another issue where we were properly passing the value of this setting if http.proxy wasn't set as well. This addressed a similar-looking issue: https://github.com/OmniSharp/omnisharp-vscode/issues/957. Would you willing to give a build of the extension with that fix a try?

Open Visual Studio Code and select View->Extensions to display the Extensions Pane.
In the Extensions pane, clear the little gear icon for the C# extension and select "Uninstall". Confirm Visual Studio Code's prompts and allow it to reload.
Download the latest beta .vsix here.
In the Extensions pane, click the [...] in the top-right corner and select "Install from VSIX...". Locate the .vsix you just download and click "Open".
Allow Visual Studio Code to reload and open your project.

Let me know if the downloads work for you now.

If that still doesn't address it, I wonder if we need a way to add additional root certs like in https://github.com/nodejs/node/issues/3742

If that doesn't address it, we should push hard for this: https://github.com/Microsoft/vscode/issues/6929. :smile:

@DustinCampbell, happy to announce that 1.5.3 fixed it. I updated the extension this morning and finally saw the long-awaited message Finished 😄
And by the way, I am supporting Microsoft/vscode#6929 as well :)

Glad to hear it! Fantastic!

i can confirm this problem still exists:

Updating C# dependencies...
Platform: win32, x86_64
Downloading package 'OmniSharp (.NET 4.6 / x64)' failed (error code '407')
Failed at stage: downloadPackages
407
Finished
my settings.json:

// Platzieren Sie Ihre Einstellungen in dieser Datei, um die Standardeinstellungen zu überschreiben.
{
"http.proxyStrictSSL": false,
"http.proxy": "http://companydomain\username:password@x.x.x.x:8080/",
"https.proxy": "http://companydomain\username:password@x.x.x.x:8080/",
"csharp.omnisharp": "D:/omnisharp/omnisharp-win-x64-net46/OmniSharp.exe"
}
i have also tried installing manually via vsix files (even different versions to see if the problem still consists...)
i am unsure if it works or not, as its still giving the error as shown above...
could someone tell me how id find out if omnisharp net 4.6.x integration works?
should there be a debugger showing up for .net 4.x in the list of debuggers if it works?
im a bit confused...
thanks for any kind of help with this...

This worked for me:
{
    "http.proxyStrictSSL": false,
    "http.proxy": "http://username:password@x.x.x.x:8080/"
}

This worked for me (in VS Code settings file):
{
    "http.proxyStrictSSL": false,
    "http.proxy": "http://username:password@x.x.x.x:8080/"
}

Unfortunately this doesn't work for me. Our proxy doesn't need username password so it's just:
"http.proxyStrictSSL": false,
"http.proxy": "https://proxy.companydomain.com:9090",
And as said above, this is the ONLY thing I can't access or download from within Visual Studio behind our corporate firewall.

Yep this is still an issue.
The environment variables used by vs code and everything else, seem to be ignored by this particular extension

Can you suggest a workaround?
I got archives listed in package.json for windows and unzipped them to their respective directories, but when I open folder with a .Net project the ms-vscode.csharp extension still wants to download the Omnisharp archive.
How can I signal to this extension that Omnisharp has been installed?

omnisharp should respect
"http.proxyAuthorization": "Basic base64encoded(user:password)"
in settings.json


[WIP] support non-destructive editing of content w/ legacy shortcodes

Pre-Flight checklist

[ ] Screenshots and design review for any changes that affect layout or styling

[ ] Desktop screenshots
[ ] Mobile width screenshots

[ ] Testing

[ ] Code is tested
[ ] Changes have been manually tested

What are the relevant tickets?
(Required)
What's this PR do?
This implements support for nondestructively editing markdown files which have 'legacy' shortcodes in them (these are shortcodes inserted into converted content by ocw-to-hugo to support a series of things which aren't supported in plain markdown).
How should this be manually tested?
Screenshots (if appropriate)

Codecov Report

Merging #1013 (54ae91c) into master (b42edfb) will increase coverage by 2.51%.
The diff coverage is 28.33%.

@@            Coverage Diff             @@
##           master    #1013      +/-   ##
==========================================
+ Coverage   91.05%   93.56%   +2.51%
==========================================
  Files         194       96      -98
  Lines        7420     2597    -4823
  Branches     1301      535     -766
==========================================
- Hits         6756     2430    -4326
+ Misses        582      167     -415
+ Partials       82        0      -82

Impacted Files
Coverage Δ

static/js/lib/ckeditor/plugins/ResourceEmbed.ts
92.18% <ø> (ø)

static/js/lib/ckeditor/plugins/LegacyShortcodes.ts
14.00% <14.00%> (ø)

static/js/lib/ckeditor/CKEditor.ts
100.00% <100.00%> (ø)

static/js/lib/ckeditor/plugins/Markdown.ts
100.00% <100.00%> (ø)

...tic/js/lib/ckeditor/plugins/TableMarkdownSyntax.ts
100.00% <100.00%> (ø)

static/js/lib/ckeditor/plugins/constants.ts
100.00% <100.00%> (ø)

websites/signals.py

content_sync/constants.py

websites/messages.py

content_sync/tasks.py

... and 95 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b42edfb...54ae91c. Read the comment docs.


Update docs to new frontend/UI

TLDR: Update datasets docs to the new syntax (markdown and mdx files) & frontend (as how it looks on hf.co/transformers)

Light mode
Dark mode

Checklist

[x] update datasets docs to new syntax (should call doc-builder convert) (this PR)
[x] discuss @property methods frontend https://github.com/huggingface/doc-builder/pull/87
[x] discuss inject_arrow_table_documentation (this PR) https://github.com/huggingface/datasets/pull/3690#discussion_r801847860
[x] update datasets docs path on moon-landing https://github.com/huggingface/moon-landing/pull/2089
[x] convert pyarrow docstring from Numpydoc style to groups style  https://github.com/huggingface/doc-builder/pull/89(https://stackoverflow.com/a/24385103/6558628)
[x] handle Raises section on frontend and doc-builder https://github.com/huggingface/doc-builder/pull/86
[x] check imgs path (this PR) (nothing to update here)
[x] doc exaples block has to follow format Examples:: https://github.com/huggingface/datasets/pull/3693
[x] fix this docstring (causing svelte compilation error)
[x] Delete sphinx related files
[x] Delete sphinx CI
[x]  Update docs config in setup.py
[x] add versions.yml in doc-build  https://github.com/huggingface/doc-build/pull/1
[x] add versions.yml in doc-build-dev https://github.com/huggingface/doc-build-dev/pull/1
[x] https://github.com/huggingface/moon-landing/pull/2089
[ ] create new github actions. (can probably be in a separate PR) (see the transformers equivalents below)

build_dev_documentation.yml
build_doc_test.yml
build_documentation.yml
delete_dev_documentation.yml
doctests.yml

Note to reviewers
The number of changed files is a lot (100+) because I've converted all .rst files to .mdx files & they are compiling fine on the svelte side (also, moved all the imgs to to doc-imgs repo). Moreover, you should just review them on preprod and see if the rendering look fine.
Therefore, I'd suggest to focus on the changed .py and CI files (github workflows, etc. you can use this filter here) during the review & ignore .mdx files. (if there's a bug in .mdx files, we can always handle it in a separate PR afterwards).

We can have the docstrings of the properties that are missing docstrings (from discussion here) here by using your new inject_arrow_table_documentation onthem as well ?

@sgugger & @lhoestq could you help me with what should the docs section in setup.py be changed to here ?
as a reference, here is a transformers setup.py docs section

For now, you can put an empty list. Once the doc-builder is in a PyPi package (with the bug we fixed on Datasets but still waiting on the standing PR with the code switch) we can put it there.

None of those dependencies are needed from this list?
    "docs": [
        "docutils==0.16.0",
        "recommonmark",
        "sphinx==3.1.2",
        "sphinx-markdown-tables",
        "sphinx-rtd-theme==0.4.3",
        "sphinxext-opengraph==0.4.1",
        "sphinx-copybutton",
        "fsspec<2021.9.0",
        "s3fs",
        "sphinx-panels",
        "sphinx-inline-tabs",
        "myst-parser",
        "Markdown!=3.3.5",
    ],

@lhoestq feel free to request reviews from other maintainers 😊

Thanks ! @mariosasko and @albertvillanova feel free to take a look :)
I can do a thorough review this afternoon

Cool thanks ! Feel free to merge master into this branch and run make style to fix the python code formatting

Love the colorful vibes here!

Could you update preprod please ? Or is there a preview somewhere I can check to make sure everything is ok ?

I'll let you know once preprod gets updated

One more tiny error that doesn't seem specific to Datasets (Transformers example here), but apostrophes and symbols aren't properly displayed in the right navbar:

In the latest commit https://github.com/huggingface/datasets/pull/3690/commits/20bddf28b22798c309e6eb1198a716f055889e1b, I tried to reflect changes from https://github.com/huggingface/transformers/pull/15903 , however, the gh workflow is not being triggered. @lhoestq do you know why it might be the case?

I removed this line to trigger the job
  pull_request:

but got this error
[Error: .github#L1](https://github.com/huggingface/datasets/commit/033fe623c556b9dbc964708b672ff9bb4896c906#annotation_2897984435)
a step cannot have both the `uses` and `run` keys

It seems to be working again, let me re-add the line I removed

@lhoestq if the CI passes, Im gonna merge this PR
please let me know if that sounds good


Setup improvement idea (unclear interactive CLI tool configuration)

When running mailing for the first time, the interactive tool asks the following question:
? Where should we put your emails? › ./emails/

This doesn't make it clear that the only allowed values here are ./emails and ./src/emails. This should either be documented or refactored to truly allow custom values. Answering this question with ./src/mailing will not work with the preview and there's currently no way to fix this.

Ah great catch, thank you. I think we should add a key to the package.json with some config to store the file location...
{
  ...
  "mailing": {
    emailsDir: "src/mailing"
  }
}

@psugihara That sounds good! Personally, I'd prefer a separate config file or an option to use one instead of package.json

yeah I like that too, mailing.config.json

fixed by #116


Why not use straight-x-fetch-all in straight-x-pull-all?

I really enjoy the parallelism brought by straight-x-fetch-all. But the parallelism does not exist in straight-x-pull-all.
Current implementation of straight-x-pull-all calls straight-pull-all and then straight-x-thaw-pinned-versions.
https://github.com/raxod502/straight.el/blob/af5437f2afd00936c883124d6d3098721c2d306c/straight-x.el#L197-L203
straight-pull-all consists of straight-fetch-all and straight-merge-all.
https://github.com/raxod502/straight.el/blob/af5437f2afd00936c883124d6d3098721c2d306c/straight.el#L6161-L6176
I am wondering is it possible to let straight-x-pull-all directly call straight-x-fetch-all and straight-merge-all so that the fetch part in straight-x-pull-all can enjoy the parallelism?
Thanks. :)

Pulling may require user intervention to resolve the state of user's repositories.
That would need to be redesigned to accommodate concurrency.

Is user intervention required in the fetch part?
If not, I think it's okay to replace only the fetch part to the parallel version and leave the merge part unchanged.

It's possible, but it's not as trivial as just calling straight-x-fetch-all prior to merging. The fetches happen asynchronously, so one would have to implement either a way to know when they had all finished, or a UI which allowed the rest of the process to take place asynchronously.


Aboutus Section Implemented

Fixes #477
Screenshots for the change:

@chashmeetsingh , @jogendra , @SIMRAN1  please review the PR

Here is the demo URL of the implementation
https://youtu.be/jicl0qjsb00

@jogendra  Please Review the PR again.

@jogendra

suggest format attributes like textColour and Size! or grey colour and size 17 is fine ?
and I formatted the text by giving proper spacing between the characters.

Is this size of logo and top margin is fine ?

I looked into the failing circle-ci linting and found that, it is failing due to large number of characters in line as line should be 150 characters or less

suggest format attributes like textColour and Size! or grey colour and size 17 is fine ?
and I formatted the text by giving proper spacing between the characters.

Colors, size and all is fine, I am pointing about the the way text written eg. there is no space after ., space before , did you get me?

Is this size of logo and top margin is fine ?

Make is little more small

I looked into the failing circle-ci linting and found that, it is failing due to large number of characters in line as line should be 150 characters or less

You have to solve that by using swift mutiline feature. Put the string between '''  ''' ex.
let loremIpsum = """
Lorem ipsum is the exapmle text
"""

@jogendra , making multiline  leads to unequal spacing between words , unaligned characters and looking weird .

@jogendra , done the changes suggested by you ,

circle-ci is failing due to issues in other files , will make a new PR for failing circle-ci linting.
please review the changes ☺

Changes made in https://github.com/fossasia/susi_iOS/pull/478#issuecomment-450471945 looks good. Do not make it multiline. Look like the logo is squeezed, please make sure logo is in the proper ratio

@jogendra , Here is the SS

can i squash the commits ?


Hinged doors should have an extra link at joints, so doors don't clip into walls

as mentioned in https://github.com/osrf/rmf_demos/pull/42

I tried out the few options and believe that moving the joint axis inwards by a fixed offset is definitely the most elegant solution, we will then have no need for extra links or walls, and we also won't need to update all the older .building.yaml
The offset is currently 3 times the door's thickness,

for completeness sake, here is a screenshot of how it looks like when its fully opened 90 degrees.


implement resize command for absolute dimensions

Just piggybacked on the existing resize_tiled() function here. resize width 300px sets width of container to 300px (if doing so is possible given constraints of layout).

How do I keep managing to push out-of-date commits? I should not be trusted with Git.

Thanks. At some point I'd like to consider extending these to support floating windows as well.


Forwarding and backwarding need more arguments

:octocat: cibot: Thank you for posting issue #398. The person in charge will reply soon.


Add system exit code for incorrect command line arguments(#1324)

fix for issue #1324

@gaugebot recheck the pull request.

@joshua-jin  Thanks for your pull request! Looks like this is your first contribution to Gauge. Before we can look at your pull request, you'll need to sign our Contributor License Agreement (CLA).
Please visit https://gauge-bot.herokuapp.com/cla/ to read and sign it.

@gaugebot recheck the pull request.

@joshua-jin Thanks for you contributions.


Revise help text for pool metadata source CLI option

Issue Number
ADP-482
Overview
Noticed while reviewing input-output-hk/cardano-launcher#90.

Fix variable names to reflect the new option
Add more detail to CLI help text

bors merge


remove case sensitivity from postgres/redshift dbname verification

This PR addresses 1800 and may also resolve  1914. It appears to fix the case sensitivity issue for database verification against both postgres and redshift.
Code Changes

modified the verify_database function in impl.py for the postgres adapter. This change appears to fix redshift without any additional code changes.

Test Changes
I made two changes to the unit tests:

modified test_postgres_adapter.py and added a test function called test_dbname_verification_is_case_insensitive. I also noticed that test_connection_fail_select appeared to be indented incorrectly so I moved it up by a level. Unit tests still pass so I'm guessing its nesting was a typo. If not I'll revert.
modified test_redshift_adapter.py and added a test function called test_dbname_verification_is_case_insensitive

Verification of fix
Revert impl.py and run tests. Unit tests will fail with an error matching the aforementioned issues.
Final thoughts
The tests I put together are clunkier than I'd like. I'm having to modify the adapter even though it's already created in setUp and is a protected property. To avoid refactoring all of the other tests, I ended up mirroring the steps performed in setUp in each test.
Long term it might make sense to break these into their own classes so that setting up the conditions for failure are easier.
Let me know if there's anything you want me to refactor, and thanks for reading. :)

@beckjake Thanks for the feedback. I made the requested change, rebased, and force pushed over my branch as I'm sure you've seen.
Let me know if you want me to make any other changes!

@JusLarsen This looks great! The Windows tests are passing here: https://dev.azure.com/fishtown-analytics/dbt/_build/results?buildId=1030 so I'm going to merge it. Thanks for your contribution!


3/31 PM Publish

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

File
Status
Preview URL
Details

docs-conceptual/azps-7.3.2/az-predictor.md
:white_check_mark:Succeeded
View (azps-7.3.2)

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


Sparse GP regression (Michalis Titsias style).

A new method, SGPR does sparse GP regression. The q(u) distribution is set optimally as per Titsias (2009) (see docstring for citation), and the method is limited to Gaussian likelihoods.
The method is tested in the same way as existing methods. In addition, I added test_method_equivalence.py, which ensures that our methods that should be similar are giving the same results (adapted from sanity_check.ipynb).

Current coverage is 95.38%

Branch #19 has no coverage reports uploaded yet.

No diff could be generated. No reports for master found.

Powered by Codecov. Updated on successful CI builds.