Scope 2 Emissions change when manually selecting Scope 1 Processes

...if I select the first upstream process to be Scope 1, the Scope 2 share in the cake diagram increases?!
Selected parameters:

Motorcycle, bicycle, and parts; at manufacturer
750$
either 10% or 1% cut-off

Test case:
BEFORE adding Cardboard; at manufacturer to Scope 1:

Scope
%
absolute [GCC]

1
95.4
356.97

2
0.78
2.93

3
3.86
14.44

Nodes dataframe:
panel_lca_class_instance.df_graph_traversal_nodes
    UID  Scope 1?                                               Name  Cumulative     Direct  Depth  activity_datapackage_id  producer_unique_id           Branch
0     0      True    Motorcycle, bicycle, and parts; at manufacturer  374.357764   2.937854      1                      317                 NaN              NaN
1     1     False                         Cardboard; at manufacturer   23.625013  11.637081      2                       90                 1.0           [0, 1]
2     2     False                                    Truck transport   12.683286  10.792749      2                      301                 2.0           [0, 2]
3     3     False                          Aluminum; at manufacturer   29.173609  14.174594      2                      243                 3.0           [0, 3]
4     4     False                           Electricity; at consumer   15.087840  14.446327      2                       53                 4.0           [0, 4]
5     5     False                      Rubber tires; at manufacturer   12.721585   1.239501      2                      135                 5.0           [0, 5]
6     6     False            Nonferrous metal casts; at manufacturer    5.755789   1.083044      2                      219                 6.0           [0, 6]
7     7     False                                    Wholesale trade    6.110259   0.203769      2                      242                 7.0           [0, 7]
8     8     False                  Company and enterprise management    4.552558   0.199157      2                      375                 8.0           [0, 8]
9     9     False   Other primary nonferrous metals; at manufacturer   12.890891   4.066379      2                      347                 9.0           [0, 9]
10   10     False                 Aluminum products; at manufacturer   77.984801  10.092095      2                      271                10.0          [0, 10]
11   11     False  Other secondary nonferrous metal products; at ...    3.755852   0.709191      2                      120                11.0          [0, 11]
12   12     False              Cardboard containers; at manufacturer    4.459448   0.279432      2                      340                12.0          [0, 12]
13   13     False  Primary iron, steel, and ferroalloy products; ...  124.807762  67.328070      2                      374                13.0          [0, 13]
14   14     False                                    Truck transport    3.862767   3.286993      3                      301                14.0      [0, 13, 14]
15   15     False                                      Coal; at mine   15.850877  13.471448      3                       19                15.0      [0, 13, 15]
16   16     False                           Electricity; at consumer    9.571202   9.164249      3                       53                16.0      [0, 13, 16]
17   17     False                           Natural gas; at consumer    4.550983   3.676436      3                      180                17.0      [0, 13, 17]
18   18     False                          Aluminum; at manufacturer   48.849939  23.734742      3                      243                18.0      [0, 10, 18]
19   19     False                           Electricity; at consumer    4.437676   4.248993      3                       53                19.0      [0, 10, 19]
20   20     False                           Electricity; at consumer   14.579621  13.959717      4                       53                20.0  [0, 10, 18, 20]
21   21     False                           Electricity; at consumer    8.707077   8.336865      3                       53                 NaN              NaN

AFTER adding Cardboard; at manufacturer to Scope 1:


Update dependencies for 32-bit MinGW CI run.

ODBC packages were updated, so version mismatch was introduced between libodbc1 and libodbc2. Now we pin versions for both of them.

Thanks for looking into this.

Thanks for looking into this.

I've updated the dependencies in the past when they updated the packages... I'm still waiting for them to fix the multi-arch dependency resolving in apt.


Fix npm publishing workflow

https://github.com/kibertoad/better-fd-slicer/actions/runs/7725563295
I assume we need to configure a token

Yes ahhaah I think we can rebase and remove those commits/releases/tags.


Inherit annotation type from base class that defines new default value

Consider this code:
import attr

@attr.s(auto_attribs=True, kw_only=True)
class A:
    foo: str

@attr.s(auto_attribs=True, kw_only=True)
class B(A):
    foo = "bar"

b = B()

When compiled, this fails with:
TypeError: __init__() missing 1 required keyword-only argument: 'foo'

Yes, in this case, it would not be trouble to re-annotate "foo" as "foo: str" in the child class as well - but our classes use heavy/multi-level annotations like:
@attr.s(auto_attribs=True, kw_only=True)
class A:
    foo: List[Union[Need, Job, Sequence]]
    bar: List[Union[Job, Sequence]]
    baz: Union[bool, str, int, List[int]]
    .
    .
    .

We have dozens of such required parameters - it would be a great hassle to keep re-annotating them in every child class that provides a new default value. It would also be almost impossible to keep track of all re-annotations, as the number of child classes is growing every day. The whole idea of using attrs was to avoid having to re-annotate the arguments in every child class. Is it possible to achieve with the current framework? If not, can this be implemented?

I don't think this can be implemented in a way that doesn't introduce a load of edge cases and surprising behavior.
I suspect this is more another case for #637.


Discord broke the theme again

Friends list, user list when mentioning someone, vscode user area, compact mode, username background, and syntax highlighting

can i know what discord channel do you use (stable/canary/ptb)? also please provide fullscreen screenshot

I use discord flatpak so that should be the stable branch, also somehow it's fixed? Everything's themed properly again

thats weird, might be the theme/discord not loaded properly.


systematic issue

I have a systematic crash when i try to add a module in the pipeline config. I join the screen when i try and the different files about this crash

vkdt-bt-7690.txt

heya,
how did you build vkdt? inside the bin/ directory or top level? did you make install? which binary are you calling? /usr/local/bin/vkdt ? or inside the git directory bin/ ./vkdt? this dialog looks like it fails to find the module directory, which should be in the git repo as bin/modules or installed as /usr/local/lib/vkdt/modules .

I don't built vkdt by myself; i use the opensuse repository. I am under ubuntu 22.04 and i run by a simple command line 'vkdt'.

okay thanks will see how i can test this package and what might be happening.

thanks

should be fixed now, new obs packages are on the way. thanks for reporting!

that's fine !


Inspector should only embed images from whitelisted domains

As noted in https://github.com/OpenHistoricalMap/issues/issues/581#issuecomment-1679783531, the inspector automatically embeds any URL in an image:0, image:1, image:2, etc. tag as an image. #583 would at least check whether it’s an image before displaying it, but we don’t have any idea what the URL points to. The inspector should only embed the image if it comes from a domain on some project-wide whitelist.
Problem
In general, it’s quite risky for us to hotlink an image from an unknown source, especially since we present it as a signature part of the website content, rather than as part of something most laypeople would recognize as a user-contributed post.
Durability
This way links directly to an image hosted on Google Photos. Except for an earlier version that linked to the share page, we wouldn’t know whose Google Photo account it’s on or whether it’ll remain there long-term. For all we’d know, the mapper got the photo from somewhere else on the Internet, and the Google Photo account owner has no way of knowing that deleting their photo would break OHM.
Images on personal photo hosting services are not the only images at risk of breakage, but more public-facing services can be archived by the Wayback Machine or kept in sync with OHM through other means: https://github.com/OpenHistoricalMap/issues/issues/581#issuecomment-1681131541. By explicitly listing the domains that the inspector does support, we have a clearer strategy for keeping track of any widespread changes, such as imgur culling old images.
Privacy
The page for the Segedunum relation (the example given in https://github.com/OpenHistoricalMap/issues/issues/581#issuecomment-1679886563) embeds an image from this URL shortened by Bitly.[^bitly] If a user views this page in OHM, Bitly automatically sets a tracking cookie in the user’s browser, even if the user never clicks on it. It also sets referrer-policy to unsafe-url, which potentially undermines HTTPS security.

The OHM website makes no mention of Bitly’s cookie policy or links to their opt-out screen, which I assume would be required in some jurisdictions. Perhaps the feature could be tagged with image:1:cookie_policy or somesuch, but I don’t imagine that mappers would be very interested in doing this kind of accounting for an image just to have it show up in the inspector.
URL shorteners aren’t the only websites that set HTTP cookies on images – so does Wikimedia Commons. However, Bitly’s business model is to use these cookies for tracking purposes. I suspect it would be a lot easier for an OHM privacy policy to link out to a fixed set of third-party privacy policies than to have to automatically discover the privacy policy for any site linked from image:#.
Security
Many websites in OSM’s website tag have gone away, replaced by squatters who run the gamut from benign to malicious. Tracking these changes has been a challenge and an unsolved problem for OSM. Most of the occurrences of image:# in OHM are recent enough that they haven’t had time to rot yet, but it’s only a matter of time.
Even now, a number of image:# occurrences refer to HTTP URLs instead of HTTPS URLs. Google Chrome refuses to load images from HTTP in an HTTPS webpage.
Proposal
Maintain a JSON file in some repository under the OpenHistoricalMap organization. The JSON file would include an array of whitelisted domains.[^blacklist] In the inspector, fetch this file (caching if necessary) and match an image:# URL against the whitelist before showing it.
Document the process for OHM contributors to get another domain listed in this file, including the criteria for getting whitelisted. If necessary, publish the repository to GitHub Pages on a subdomain of openhistoricalmap.org to keep the file lightweight and independent of the usual deployment process.
Alternatives
@jeffreyameyer suggested in https://github.com/OpenHistoricalMap/issues/issues/581#issuecomment-1679886563 that the inspector should continue to show the tagged image in general, unless the domain is on a blacklist. This would give mappers instant gratification when including images from lesser-known websites. Unfortunately, I don’t think this approach would address the durability, privacy, and security concerns, and it would create maintenance overhead of a worse nature than the proposal above: if we don’t get around to approving a pull request for a whitelist, the user may be discouraged until we do respond. But if we don’t get around to approving a PR on a blacklist, OHM potentially faces a reputational issue in the meantime.
[^bitly]: This is a kind of rubegoldbergian image reference. Besides the URL shortening service, the rehydrated URL also goes through the Internet Archive and Cloudfront on its way to an AWS s3 bucket. At only 231 characters in length, the rehydrated URL would’ve satisfied OHM’s 255-character tag value length limit. Furthermore, the resulting image is a mere 600-pixel-wide thumbnail, whereas the full image at a much shorter URL is 2,000 pixels wide, allowing the user to see many more details than in the thumbnail.
[^blacklist]: A JSON file would allow us to potentially add a complementary blacklist of URL patterns in the future, in case there’s something from a trusted domain that should be tagged but would be problematic to show in the inspector for some reason.

I whipped up a quick script for generating an image gallery of all of the images tagged on OHM, which makes it easy to visually review images for mistagging and other problems. Unfortunately, JSBin appears to have banned my IP address as a result of writing or running this script:
var oql = "[out:json][timeout:25];\
nwr[~\"^image:[0-9]+$\"~\".*\"];\
out center;";
var query = "https://overpass-api.openhistoricalmap.org/api/interpreter?data=" + encodeURIComponent(oql);
fetch(query).then(function (response) {
  return response.json();
}).then(function (data) {
  var rows = data.elements.map(function (element) {
    var imageTags = Object.entries(element.tags).filter(function (tag) {
      return tag[0].search(/^image:\d+$/) === 0;
    }).map(function (tag) {
      return "";
    });
    var url = "https://www.openhistoricalmap.org/" + element.type + "/" + element.id;
    var label = element.type[0] + element.id;
    var link = "" + label + "";
    return "
" + link + "	" + (element.tags.name || "") + "	" + imageTags.join(" ") + "
";
  });

  var table = "

Element	Name	Images

\n" + rows.join("\n") + "\n

";

  document.body.innerHTML = table;
});

Here’s a preview of that page. Unfortunately I was only able to capture the first few screenfuls, but you can see that there are a lot of broken images, accounting for the vast majority of the non-Commons-hosted images.


check-env-file npm script assumes Python 2 is installed

We need to mention that Python 2 must be installed in the setup guide. Also, I had Python 3, so I got an error saying raw_input is not defined. I don't know Python, but on searching I found out that the Python 3 equivalent is something different - input() and raw_input() doesn't work in Python 3.
We must find a solution or work around for this, or I would say - just use NodeJs to do this checking, because Python 2 and Python 3 have so many differences and there's even a command called 2to3 to convert Python 2 source code to Python 3 source code. And people use different versions of Python and some may not use Python at all, so this will be an extra dependency to install.
Also surprisingly, when I got it work with Python 2, the script was asking me to enter values that are already present in the .env file. I think the script always asks for all the values ? That's how it looks, I think it has some bugs
The log for my raw_input not defined error :
karuppiah@karuppiah ~/F/Mail-for-Good> npm run dev;

><EMAIL_ADDRESS>dev /home/karuppiah/FOSS/Mail-for-Good
> npm run check-env-file && DEV_SEND_RATE=200 NODE_ENV=development node server/index.js

><EMAIL_ADDRESS>check-env-file /home/karuppiah/FOSS/Mail-for-Good
> python ./tools/setup/initial_setup.py

    |==========================================================================================================|
    | This is a short guide for setting up M4G. We'll check a few things to ensure you have things configured. |
    | We will also ask you for key information, and then set things up.                                        |
    |                                                                                |
    | If you have already configured your .env file - you've missed one or two settings. We'll add these now.  |
    |==========================================================================================================|

What is your Google consumer key (client id) and secret key (client secret)?

You can find these here https://console.cloud.google.com/apis/credentials
And read our guide here https://github.com/freeCodeCamp/mail-for-good/wiki/How-to-configure-your-Google-API-Keys

Traceback (most recent call last):
  File "./tools/setup/initial_setup.py", line 133, in <module>
    main()
  File "./tools/setup/initial_setup.py", line 38, in main
    GOOGLE_CONSUMER_KEY = raw_input('GOOGLE_CONSUMER_KEY = ')
NameError: name 'raw_input' is not defined
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>check-env-file: `python ./tools/setup/initial_setup.py`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>check-env-file script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!  Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>dev script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/karuppiah/.npm/_logs/2017-07-16T04_10_42_298Z-debug.log

cc @AndrewGHC

Thanks @karuppiah7890.
Python 2 should always be available on any unix-based machine. I will look into replacing raw_input with something cross-compatible with both 2 and 3, it's been a while since I last used python.
I threw it out pretty quickly to ensure users had the right setup installed. I don't want to invest time in making it much more complex, the point is just to create a .env file if one doesn't exist.

@AndrewGHC  Okay.. and I think the script doesn't check the existence of the .env file properly. I get asked to type the environment variable values even if my .env is always present. I don't why this if condition
if os.path.exists('../../.env'):
    return

is not working properly

I've removed the forced python check from the app - I think I was a bit hasty throwing it in there. Can come back to this later.


Make Tests for the Param Objects (ImpactHours, Hatch, DandelionVoting)

Make test functions for the classes and their functions. Unit tests as similar to the commons simulator testing setup. Possibly integration tests as well.

@LinuxIsCool @marthendalnunes   guysss.. Finally tests are running okay but would like some feedback on how I can write some more rigorous tests..


Open file dialog doesn't show and tab strings are not visible

Thanks for the cool project! I am trying to get it to run on Archlinux with wayland, but am having some issues. The file open dialog doesn't show unless I modify the Cargo.toml and replace:
rfd = "0.14.1"

with
rfd = { version = "0.14.1", default-features = false, features = ["gtk3"] }

The other issue is when I load one of the example tabs I am getting the following output

Last thing is I had to install the pipewire-alsa package to get it to start, but that's platform dependentan
Other than that it looks pretty nice for 0.1.0 release!

Thanks for the investigation!
Are you seeing the following log line on the vanilla binary?
WARN ruxguitar::ui::application] Failed to read GP file: dialog window closed without selecting a file

Yes, I am seeing that exact warning.

For the display issue, maybe you could try disabling the antialiasing in https://github.com/agourlay/ruxguitar/blob/master/src/ui/application.rs#L113

Changing the antialiasing to:
        .antialiasing(false)

fixes the strings display... Thanks!

Lucky shot :)
I will setup a flag in the CLI to disable antialiasing on demand then.

Regarding the file picker issue, is it possible for you to install the XDG Desktop portal?

So installing the xdg-desktop-portal-gtk fixed the file picker.

Fixed in 0.1.1 which:

introduced no-antialiasing options
FAQ with the XDG desktop portal requirement on Linux


Variable expansion outside queries

Describe your changes
Added variable expansion support for the string type arguments in the following:

%sqlcmd columns
%sqlcmd explore
%sqlcmd profile
%sqlcmd snippets
%sqlcmd tables
%sqlcmd test
%sql
%sqlplot

Integer/ float arguments of histogram plot like bins, binwidth, etc are not supported.
Issue number
Closes #699
Checklist before requesting a review

[ ] Performed a self-review of my code
[ ] Formatted my code with pkgmt format
[ ] Added tests (when necessary).
[ ] Added docstring documentation and update the changelog (when needed)

:books: Documentation preview :books:: https://jupysql--941.org.readthedocs.build/en/941/

I haven't added histogram specific arguments like bins, binwidth etc, and comparator arguments for testing like greater, greater_or_equal etc. Do we need to add these as well? @edublancas

@neelasha23 I think expanding the parameters when we read the arguments string would be better so we automatically add expansion to them. Otherwise, we'll have to remember to add this feature to every argument we add. If you look at the SQL magic implementation, there should be a part where we obtain the raw argument string e.g. --a something --b another; we could expand the values there

Addressed the comment @edublancas

@bryannho please review this, pay special attention to this comment

@edublancas @neelasha23 Do we want to allow users to specify argument flags using variables in this way? I assume this wasn't the original intent. It could definitely be helpful but might also open up more issues in the future.

I think for this PR we should only handle arguments and not flags. I have changed the approach. Previously I had tried a couple of approaches:

Regex based to extract --arg {{value}} patterns and then rendering. This works but I'm not very confident of this approach since false edge cases/ missed cases might come up.
Allow the argparse parser to parse the input as is and then substitute the args. So the argparse.Namespace would contain values like save: {{snippet}} and then {{snippet}} would be rendered by the actual snippet name. I feel this approach is less error prone and more deterministic, as now we are rendering only the argument values specifically, although it requires an additional step of iterating over the namespace. So I have reverted to this approach.

Here are the changes:
Regarding the error, now it will display error if we try to use the snippet:

Allow the argparse parser to parse the input as is and then substitute the args. So the argparse.Namespace would contain values like save: {{snippet}} and then {{snippet}} would be rendered by the actual snippet name. I feel this approach is less error prone and more deterministic, as now we are rendering only the argument values specifically, although it requires an additional step of iterating over the namespace. So I have reverted to this approach.

I'm confused. this approach sounds like a much more robust alternative than adding code to each of the SQL commands, and you mentioned "I have reverted to this approach",
but then I looked at the code and there are a few snippets that look like this:
   if is_rendering_required(" ".join(others)):
        expand_args(args, user_ns)

the objective is to have a single place for expanding the {{variables}} and I thought we had agreed to do that. but it seems like we still have an implementation that requires us adding the same snippet on each command?

the objective is to have a single place for expanding the {{variables}} and I thought we had agreed to do that. but it seems like we still have an implementation that requires us adding the same snippet on each command?

What i understood from your previous comment, you mentioned that we we might forget to add every-time we add a new argument so we need a approach where we dont need to bother abut adding the arguments to a list. But in the current approach we dont need to maintain any list of that sort it automatically detects the "{{value}}" pattern inside argparse and replaces them.
But we still need to call this util function at the different commands file right? like sql, sqlplot. and the different versions of sqlcmd. Also, sqlcmd is parsed differently than than the other two.
Now if we add any new command like sqlcmd something or sqlsomething only then we need to add this block of code otherwise not. Does this make sense?
@edublancas

@neelasha23 oh, you're right. I thought we could add a single piece of logic that would work for all commands (%%sql, %sqlcmd , etc). But then I realized that they are all implemented in different places as they are magic objects, so your approach makes sense.
just fix the conflicts, the integration tests and this should be good to go!

conflicts and tests are fixed now @edublancas


Toggle Voice Chat

Due Date
Jan 8th
Priority
Medium
Description
Make it so user can toggle voice chat in lobby and in game
Associated Requirement
N/A

Completed in commit: 9a7b0f801c3c7d9c85da4a6431dd5fcd924aef6c


[FEATURE] Set UID and GID of opensearch user and group to 10000

Is your feature request related to a problem?
Checkov recommends that the UID of containers run with 10000 or more - Ref: Ensure containers run with a high UID to avoid host conflict
What solution would you like?
Change the UID (and GID to make is consistent) defined in the Dockerfile to 10000
What alternatives have you considered?
Do you have any additional context?

Hi, @nilushancosta , with the concern that changing the UID and GID to other may break many, and this issue has been opened for quite some time now, I am closing it for now, please feel free to re-open it for any further discussion. Thank you !


Simiki.org site is currently inaccessible

The Simiki.org site is currently inaccessible
  You have reached a domain that is pending ICANN verification.

Thanks for your report. That's a strange problem which happened some days ago, I guess it's caused by domain whoisguard protection, and I will fix it ASAP.

The site is ok now.


Persist /etc/docker directory

Starting in docker 1.10, a persistent configuration file can be stored
(/etc/docker/daemon.json).  This change makes sure that we store
/etc/docker on the mounted filesystem so it will survive reboots.

ping @tianon @SvenDowideit could you please review this? We'd like to have this in 1.10 if possible

Do you think it would be better for docker to start with --config-file=/var/lib/boot2docker/daemon.json instead?

closes https://github.com/docker/docker/issues/19454

@cpuguy83 I would really prefer to stick with the standard path, since other tools/automation are likely to be built around that default path assumption.

Nice and simple, and makes sure hard-coded assumptions continue to work properly. :+1:
LGTM

Thanks all!

Thanks @tianon!


Update go libs

required for https://github.com/clastix/cluster-api-control-plane-provider-kamaji/pull/78
go 1.20 --> 1.22
kamaji v0.4.0 --> v0.5.0
cluster-api v1.4.0-beta.2.0 --> v1.6.4
kubernetes v0.29.0 --> v0.30.0
controller-gen v0.12.0 --> v0.14.0

I think we need to update also the Go version of the underlying GH actions.

I know the problem with controller-gen and Go 1.22, we just need to bump it to v0.14.0

This change is a bit intrusive 😄
Please, let's generate manifests with make release and let's remove the gocognit ignore, you can double check with make lint.

I had old controller-gen at my PC, now diff should be fixed

aha, you're right


Fix warning in block declaration from UICountingLabel

This PR updates UICountingLabel dependency to the latest version to remove the warning of block declaration. More info here.

[x] Update CocoaPods to 1.3.1
[x] Bump target to iOS 7.0 to be able to use latest UICountingLabel
[x] Update UICountingLabel to v1.4.1
[x] Hide 'Animations' label on Scatter Chart demo

@kevinzhow Thanks for the amazing lib, can you please review and merge if looks good, happy to address any comments.

Hi @kevinzhow, sorry for pinging you again but do you have an idea of when you could review this?

@kevinzhow All conflicts resolved and CI is green, please 👀

Looks good to me! @kevinzhow let's 🚀 it 🙏


Hide original comments

Hide original comments:

/lgtm

@pigletfly It seems still some wrong about the numerical order of the titles after my first commit:

So, I have to put all comments in a comment block, you can check it now: https://deploy-preview-20420--kubernetes-io-master-staging.netlify.app/zh/docs/concepts/extend-kubernetes/extend-cluster/

/lgtm

/assign @tengqm


Added authentication support for AQMP broker url

Updated AQMP url to include username, password and port params from the configuration:

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

Reference issue raised
https://github.com/RasaHQ/rasa/issues/8042

Thanks for submitting a pull request 🚀 @koaning will take a look at it as soon as possible ✨

can some one verify it ? or provide an ETA ?

Ah, it seems that I was notified of this during my holiday. The tagger shouldn't have tagged me in this case. I'll poke folks on slack to have a look right away. This is part of the codebase that I'm not familiar with so I prefer to have somebody else review.

@alwx Are we good now?

@nakshathru approved and ran all the tests.

Ok, now we have the code quality issue (https://github.com/RasaHQ/rasa/pull/8057/checks?check_run_id=3369311492) — you can resolve it by running make formatter and committing the changes.

@nakshathru just let me know if you still want to continue working on this and fix the remaining code style issues. Otherwise, I would step in and update the code style.

@alwx  Thanks for the reminder. I tried make formatter but it didn't fix anything. Later I found a max length lint error.
So updated the code and  make lint is now passing.
Please take a look!!!

@nakshathru thanks for your contribution!


Redirect traffic from supplementary catalog page on library website

https://library.princeton.edu/supplementary-catalog currently submits a search form directly to imagecat.
Instead, we'd like to redirect this url the imagecat url.
acceptance criteria:

[ ] copy the more extensive site description over to the new imagecat site front page
[ ] redirect in place

Options:

Instead of a search box, put a link to the new site
could we redirect this url to the new site at the load balancer level

Questions:

How important is the text? (copied below)

The Supplementary Catalog contains items cataloged before 1980. Records are arranged alphabetically with authors, titles, and subjects interfiled. This catalog has been largely replaced by the online Catalog.
For items in Chinese, Japanese, or Korean cataloged before 1983, consult the card catalogs in the East Asian Library.
For sound recordings cataloged before 1989, consult the card catalog in the Mendel Library. Although most of Princeton University Library's book and journal holdings are represented in the Main and Supplementary Catalogs, other forms of materials may not be included there. Consult Library staff if you do not find special format materials in the main or supplemental catalogs.
Formats such as sound recordings, maps, prints, posters, technical reports, theses (other than Princeton Ph.D. theses), journal articles, and pamphlets are generally cataloged locally. If a search of the Main or Supplementary Catalogs yields unsatisfactory results, you may need to consult a local or special catalog to find records representing the materials you seek. Please consult a reference librarian at any reference desk in Firestone Library or in a subject branch library.

@kevinreiss this page appears to be on the drupal -- is there anyone we need to consult or get approval from on this plan?

There may be an issue with a redirect -- once the site is migrated to OIT it would likely be lost.
Backup plan is to change the search box to a "go to the site" button.

Do we want to move "Enter an author's last name or the first words of a title (omit initial articles like "a", "an" and "the") or a Library of Congress subject heading." to be helptext above our search box?

Another option is have the search box submit to /search?search_term=[term]

We updated the search box submission.


Make a script for npm build-dev

Would it be possible to make a npm build-dev that will bundle the files as a dev bundle without the hash? Currently, I'm just modifying the build.js in the node_modules to point to the dev config file. But this is naturally overwritten when I do npm install.
Thanks.

Is this the same as https://github.com/facebookincubator/create-react-app/issues/1070?

Yes, pretty much. I would like the dev bundle files so I can work with integrating them in SharePoint.
Thanks


How to send order address to backend

Currently the ordering process will only work if the user is logged in, otherwise the backend will not receive the address(es).
We need to develop a way to get the full data for use in sending emails, github issues, etc.
One solution is to use the front end email field to query the back end and see if it exists. If it does, then use that user information. If they change the address, then update the user entry in the database. If the email does not exist in the database, then create a new user.
So basically - findOrUpdate - if find is true, then update, else create.

Also need to add error handling to form submission.

Reopening since we need to do this with both consumer and payer.


⚠ Turnos 24x7 has degraded performance

In 5a5148b, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 1138 ms

Resolved: Turnos 24x7 performance has improved in 8951395.


Render various common data structures to Tables

I don't know if there was a way to do this already built in, but I end up working with lots of data whether it's in a IDataReader, IEnumerable<T> or IEnumerable<dynamic> (from Dapper which I use frequently, I had to do some acrobatics to reflect on the dynamic IEnumerable but it's working out well).  If this didn't exist I didn't know if something like this would be useful.  I'm using this for a hobby project so I haven't thought it through API wise or refactored anything but below is a brief example of how I'm using it (apologies if the inlined SQL makes anyone cringe, it's for illustration):
In this example Terminal is just an IAnsiConsole with the additional rendering functions bolted on.  I wanted to write extension methods to AnsiConsole but the static class prohibited that (and I couldn't use the partial class pattern being outside of the Spectre.Console assembly.
using var conn = await Program.Db.GetOpenDbConnection();
var results = await conn.QueryAsync(@"SELECT table_name AS 'Table',
                                        ROUND(((data_length + index_length) / 1024 / 1024), 2) AS 'Size (in MB)',
                                        table_rows as 'Estimated Rows'
                                        FROM information_schema.TABLES
                                        WHERE table_schema = 'courier'
                                        ORDER BY(data_length + index_length) DESC");

// Can specify an optional lambda to operate on each row of whatever the data source is to format it.
Terminal.Write(results, "LightSkyBlue3_1", (row) =>
{
    row[0] = $"[bold LightSkyBlue1]{row[0]}[/]";
    row[2] = $"{row[2].FormatIfNumber(0)}";
});

I didn't know if anyone else would use this kind of functionality or find it desirable but I thought I'd ask.  Love the project also, thank you for sharing it!

I think some sort of extension would be pretty useful, but I don't think needs to be aware of those types. My first thought was implementing an interface, and wrapping your IDataReader.

@blakepell There will not be any built-in knowledge of IDataReader or similar, but I will add some convenience methods to render POCO-objects as tables.
Using AnsiConsole directly is mostly for convenience and small apps. I recommend that you pass an IAnsiConsole around for more complex applications. This way, it's easier to unit test the functionality of your application. You can get the IAnsiConsole for AnsiConsole via AnsiConsole.Console property.
Here's a quick experiment I did for adding rows from a collection of POCO objects:
/// <summary>
/// Adds multiple rows to the table.
/// </summary>
/// <typeparam name="T">The item type.</typeparam>
/// <param name="table">The table to add the row to.</param>
/// <param name="values">The values to create rows from.</param>
/// <param name="columnFunc">A collection of functions that gets a column value.</param>
/// <returns>The same instance so that multiple calls can be chained.</returns>
public static Table AddRows<T>(this Table table, IEnumerable<T> values, params Func<T, object>[] columnFunc)
{
    if (table is null)
    {
        throw new ArgumentNullException(nameof(table));
    }

    if (values is null)
    {
        throw new ArgumentNullException(nameof(values));
    }

    foreach (var value in values)
    {
        var columns = new List<IRenderable>();
        foreach (var converter in columnFunc)
        {
            var column = converter(value);
            if (column == null)
            {
                columns.Add(Text.Empty);
            }
            else if (column is IRenderable renderable)
            {
                columns.Add(renderable);
            }
            else
            {
                var text = column?.ToString() ?? string.Empty;
                if (string.IsNullOrWhiteSpace(text))
                {
                    columns.Add(Text.Empty);
                }
                else
                {
                    columns.Add(new Markup(text));
                }
            }
        }

        table.AddRow(columns.ToArray());
    }

    return table;
}

Usage:
var table = new Table()
    .AddRows(
        movies,
        m => $"[red]{m.Name}[/]",
        m => $"[green]{m.BoxOffice:C}[/]",
        m =>
        {
            if (m.BoxOffice ><PHONE_NUMBER>)
            {
                return new Panel("[green]😱[/]").BorderStyle("yellow");
            }

            return "[red]Lame[/]";
        });

@patriksvensson @blakepell, I wrote a library for Dumping any .NET object in a structured, colorful way. It uses Tables from Spectre.Console.
It is called Dumpify
https://github.com/MoaidHathot/Dumpify/
You can dump to Console, Debug, Trace or any custom target you want.
Some examples:
var moaid = new Person { FirstName = "Moaid", LastName = "Hathot" };
var haneeni = new Person { FirstName = "Haneeni", LastName = "Shibli" };

moaid.Spouse = haneeni;
haneeni.Spouse = moaid;

moaid.Dump();

@MoaidHathot Beautiful!


Proposal: Custom rate limiting rules

Preliminary: I know that rate limiting has been proposed before (in #387), but due to the lack of a description, I would like to submit a justified proposal herewith.
Description
I propose the option to implement basic rate-limiting to specific API endpoints. Most, if not all apps, would benefit from rate-limiting user and record creation. For this proposal, I want to create an imaginary app that is like Instagram.
Attack vector
Servers / apps will, sooner or later, be attacked.

A common attack vector is to spam the creation of posts (like pictures on our imaginary Instagram clone). At the current state, PocketBase (in its bare form) would allow the creation of thousands of posts per second. This is obviously nothing that we would want in our app.
Another attack vector is dumping the whole database. An attacker could query all posts and increase the "page" in a loop. This would cause intense load on the servers memory and the attacker would have every post locally (which might be millions on big apps).
The probably most dangerous attack would be to create thousands of users. Those are hard to delete, as it's not tracked which IP created the user. A simple script is currently capable to create users in a loop. Such an attack could hardly be prevented in PocketBases current form.

Current solution
The current solution requires the use of a reverse-proxy. This would be feasible, but the programmer would have to create individual rules for each route (which are currently not marked as stable, so the rules might need to be updated frequently). With larger apps this is hardly possible to keep synchronous and would require the usage of lots of wildcards (posts creation might have a different rate-limit than post-view for example). Also, some people do not have a reverse-proxy or are not capable to set one up.
Proposed solution
I propose that an option is added to each collection, how rate-limiting should be handled. This option could be embedded in the existing API rules. A simple implementation could look like this:

Note
Text in square brackets could be an input field as dropdown or raw input

Allow [user/IP/IP||user] [number] [creations/updates/views/lists] per [number] [second/minute/hour]

Note
The limiting metric IP||User should count request on the same IP or the same user, to prevent users to use proxy lists.
When anonymous users have access to the collection endpoint, it should fallback to IP||session_id

As a quick mockup, this could look like:

Maybe this could be integrated in the API Rule syntax somehow, but I'll leave that up to you :)
This could also be extended, so that unauthorized users, or users which fit a requirement (like a role), have different rate-limits.

Please let me know what you think, in my opinion this would be a very useful security feature!

While I agree that this would be a nice to have feature, for now it remains "unplanned" because there are way too many other issues with higher priority that represents a blocker to some use cases (multi-match and @request.data.* rel filter enhancements, private/protected files, single and combined field indexes, raw sql or collection views for aggregations, etc.).
I'm not even sure that I'll accept a PR if someone decide to work on this. I want to keep PocketBase as simple as possible.
While for the past month I'm lucky enough to be able to work full-time on PocketBase, I'm well aware that this will not last forever and I have to think a little more on the eventual maintenance burden that will come with every new feature (not to mention that I'm already slightly behind the original planned schedule with my other open source project - Presentator #183).
Maybe the better approach in the long run will be to prioritize the Admin UI extendability (#898) to allow users to create plugins that eventually may get integrated/embedded in the core.
I'll try to reorganize and update the roadmap after the upcoming few releases.

If you want to use the default echo rate-limiter middleware you can register it to the e.Router instance in the OnBeforeServe event hook:
app.OnBeforeServe().Add(func(e *core.ServeEvent) error {
	e.Use(middleware.RateLimiterWithConfig(yourConfig))

	return nil
})

You can specify to which routes to apply by using the Skipper field of the middleware config.
We can't register middlewares inside other event hooks because they are triggered more than once. Additionally the echo Use and Pre middleware registration functions are not concurrent safe and may cause some issues if a middleware is registered after the server start (I haven't really tested it).
In any case, as mentioned previously for me this personally is a very low priority.
I understand the eventual benefits of it but most users in normal situations don't really need it and in some cases using an in-memory rate-limiter may actually cause more harm than good since it will increase the load on the server and I don't want to rush implementation.

I made my own custom middleware now, and I think that should be enough for now. If anyone is interested, here is the implementation:
Middleware
// RateLimitCollectionOperations is a middleware that rate limits HTTP requests on collection operations.
// Possible operations:
// - list
// - view
// - create
// - update
// - delete
func RateLimitCollectionOperations(collection string, operations []string, limiter echo.MiddlewareFunc) echo.MiddlewareFunc {
	return func(next echo.HandlerFunc) echo.HandlerFunc {
		return func(c echo.Context) error {
			contains := func(s string, list []string) bool {
				for _, v := range list {
					if v == s {
						return true
					}
				}
				return false
			}

			collectionEndpoint := fmt.Sprintf("/api/collections/%s/records", collection)

			if contains("list", operations) && c.Request().URL.Path == collectionEndpoint && c.Request().Method == http.MethodGet {
				return limiter(next)(c)
			} else if contains("view", operations) && strings.HasPrefix(c.Request().URL.Path, collectionEndpoint) && c.Request().Method == http.MethodGet {
				return limiter(next)(c)
			} else if contains("create", operations) && strings.HasPrefix(c.Request().URL.Path, collectionEndpoint) && c.Request().Method == http.MethodPost {
				return limiter(next)(c)
			} else if contains("update", operations) && strings.HasPrefix(c.Request().URL.Path, collectionEndpoint) && c.Request().Method == http.MethodPut {
				return limiter(next)(c)
			} else if contains("delete", operations) && strings.HasPrefix(c.Request().URL.Path, collectionEndpoint) && c.Request().Method == http.MethodDelete {
				return limiter(next)(c)
			}

			return next(c)
		}
	}
}

Usage
func main() {
	app := pocketbase.New()

	limiter := middleware.RateLimiter(middleware.NewRateLimiterMemoryStore(2))
	app.OnBeforeServe().Add(func(e *core.ServeEvent) error {
		// Rate limit list and view operations on the "posts" collection
		e.Router.Use(RateLimitCollectionOperations("posts", []string{"list", "view"}, limiter))
		return nil
	})

	app.Start()
}

Warnng
I have not tested every case, but for now it seems to work fine. I'll update the solution when I find a bug.
Also, the code could be refactored, but I wanted to keep it a single function for sharing. Feel free to extract the contains method, etc...

We need a login abuse limiter as well

We need a login abuse limiter as well

I think the most common approach for login and registration abuse prevention is implementing a captcha. There are ones that can be configured to only prompt the user, if suspicious activities are detected. Even with basic rate limiting, logins could still be brute forced with proxy lists.
For basic rate limiting registrations, you can use the middleware I posted earlier (with a create action on the users collection). With some adaptions, this could also be used for login rate limiting, but like I said, this is usually not enough for those "critical public endpoints".

Hi

This would be a great addition to PocketBase! Max requests per op/user per action would be amazing!

While this feature is needed in production, I'm not sure it belongs to Pocketbase itself.
A reverse proxy, such as NGINX, is better suited for geo / rate limiting.
https://www.nginx.com/blog/rate-limiting-nginx/
https://gist.github.com/ipmb/472da2a9071dd87e24d3

@adam-ah Many highly scalable applications run behind a reverse proxy, of course. But PocketBase is an all-in-one solution. Therefore I think it should not be necessary to put a reverse-proxy in between. Also, some routes need a different rate-limit. When setting up a reverse-proxy you would have to know each API route and limit it individually, which is quite a time-consuming task. Especially when apps get new features, you would have to evaluate each time if the reverse-proxy config needs to be adjusted.

I think the right question to ask is "Is a rate limiter needed?" If the answer is "yes," then in the spirit of PocketBase's tagline, "Open Source backend for your next SaaS and Mobile app in 1 file," then rate limiting should be included in PocketBase.
Personally, I think rate limiting is an important feature, as all services exposed to the internet of any interest at all will suffer some attack. It is certainly as important as some of the nice-to-have features already slated to be implemented.

To add to alternatives, you can also apply a rate limit on a dns level. Cloudflare makes it easy to configure a WAF

You cannot apply "rate limit on a dns level", whatever that means, @TheRedSpy15
Cloudflare is effectively a reverse proxy, and this approach has been discussed before.

You cannot apply "rate limit on a dns level", whatever that means, @TheRedSpy15 Cloudflare is effectively a reverse proxy, and this approach has been discussed before.

DNS level referring to using your domain provider.

It needs some tests but a builtin rate limiter is implemented as part of the ongoing refactoring and will be available with the next v0.23.0 release (there are no hard ETAs but I expect to finalize at least an RC of the new Go APIs within a month or two at worst).
The planned rate limiter implementation is part of a new "Security" PocketBase Settings section.
The implementation differs a little from the proposal and it is not tightly coupled to the collections in order to allow managing rate limits for custom user defined routes in a unified manner. This is currently done using "tags".
The general idea is that users will have an option to define several "tagged" rate limit
configs (for example "users:create" -> 0.5 r/s) which will be then picked by the apis.RateLimit(optTags) middleware. In case the middleware has multiple tags listed we retrieve the rate config only for the first found tagged entry (left to right).
Note that we don't have to pass a concrete PocketBase core.App instance when registering the apis.RateLimit() middleware because with the new routing (and hooks) all handlers have access to e.App field (where e is the event handler argument).
The builtin endpoints have the RateLimit middleware registered by default with a predefined tag set (this will be documented additionally).
For example, the create action for the "users" collection will have something like apis.RateLimit("users:create", "*:create") (it is a little more complicated in reality because the collection name can change during runtime).
All apis.RateLimit registration will have the "*" tag available by default as a final fallback in case a global tag config is defined, or in other words:
apis.RateLimit("users:create", "*:create") is the same as apis.RateLimit("users:create", "*:create", "*")
For now the RateLimit middleware internally uses the std time/rate "Token bucket" implementation stored in plain map.
Based on my local tests it performs fine and I wasn't able to cause lock contention once replaced the regular sync.Mutex with sync.RWMutex (it is slower for new writes but allows multiple readers which is better suited for our case).
For simplicity the unique client identifier initially will be only the client IP. For the cases where PocketBase is deployed behind a reverse proxy there is an option in the settings for users to explicit mark X-Forward-For or other custom header(s) like CF-Connecting-IP as "trusted" (I haven't explored all cases yet but eventually we may also need an option to specify a list of trusted proxy IPs).
On every rate limiter setting change we reset all collected rates data.
In case the rate limiter is disabled from the settings the apis.RateLimit() middleware does nothing.

It needs some tests but a builtin rate limiter is implemented as part of the ongoing refactoring and will be available with the next v0.23.0 release (there are no hard ETAs but I expect to finalize at least an RC of the new Go APIs within a month or two at worst).它需要一些测试，但内置的速率限制器是作为正在进行的重构的一部分实现的，并将在下一个 v0.23.0 版本中提供（没有硬性的 ETA，但我希望至少在一个新的 Go API 中最终确定一个 RC）最坏的情况是一两个月）。
The planned rate limiter implementation is part of a new "Security" PocketBase Settings section.计划的速率限制器实施是新的“安全”PocketBase 设置部分的一部分。
The implementation differs a little from the proposal and it is not tightly coupled to the collections in order to allow managing rate limits for custom user defined routes in a unified manner. This is currently done using "tags".该实现与提案略有不同，并且它没有与集合紧密耦合，以便允许以统一的方式管理自定义用户定义的路由的速率限制。目前这是使用“标签”完成的。
The general idea is that users will have an option to define several "tagged" rate limit总体思路是用户可以选择定义几个“标记”速率限制 configs (for example "users:create" -> 0.5 r/s) which will be then picked by the apis.RateLimit(optTags) middleware. In case the middleware has multiple tags listed we retrieve the rate config only for the first found tagged entry (left to right).配置（例如“users:create”-> 0.5 r/s），然后由apis.RateLimit(optTags)中间件选择。如果中间件列出了多个标签，我们仅检索第一个找到的标记条目（从左到右）的速率配置。
Note that we don't have to pass a concrete PocketBase core.App instance when registering the apis.RateLimit() middleware because with the new routing (and hooks) all handlers have access to e.App field (where e is the event handler argument).请注意，在注册apis.RateLimit()中间件时，我们不必传递具体的 PocketBase core.App实例，因为使用新的路由（和挂钩），所有处理程序都可以访问e.App字段（其中e是事件处理程序）争论）。
The builtin endpoints have the RateLimit middleware registered by default with a predefined tag set (this will be documented additionally).内置端点默认使用预定义标签集注册RateLimit中间件（这将另外记录）。
For example, the create action for the "users" collection will have something like apis.RateLimit("users:create", "*:create") (it is a little more complicated in reality because the collection name can change during runtime).例如，“users”集合的创建操作将具有类似的内容 apis.RateLimit("users:create", "*:create") （实际上有点复杂，因为集合名称可以在运行时更改）。
All apis.RateLimit registration will have the "*" tag available by default as a final fallback in case a global tag config is defined, or in other words:默认情况下，所有apis.RateLimit注册都将具有"*"标记，作为定义全局标记配置时的最终后备，或者换句话说： apis.RateLimit("users:create", "*:create") is the same as apis.RateLimit("users:create", "*:create", "*") apis.RateLimit("users:create", "*:create") 是一样的 apis.RateLimit("users:create", "*:create", "*")
For now the RateLimit middleware internally uses the std time/rate "Token bucket" implementation stored in plain map.目前， RateLimit中间件在内部使用存储在普通映射中的标准time/rate “令牌桶”实现。
Based on my local tests it performs fine and I wasn't able to cause lock contention once replaced the regular sync.Mutex with sync.RWMutex (it is slower for new writes but allows multiple readers which is better suited for our case).根据我的本地测试，它表现良好，并且一旦用sync.RWMutex替换常规sync.Mutex ，我就无法导致锁争用（新写入速度较慢，但允许多个读取器，这更适合我们的情况）。
For simplicity the unique client identifier initially will be only the client IP. For the cases where PocketBase is deployed behind a reverse proxy there is an option in the settings for users to explicit mark X-Forward-For or other custom header(s) like CF-Connecting-IP as "trusted" (I haven't explored all cases yet but eventually we may also need an option to specify a list of trusted proxy IPs).为简单起见，唯一的客户端标识符最初只是客户端 IP。对于 PocketBase 部署在反向代理后面的情况，设置中有一个选项可供用户显式标记X-Forward-For或其他自定义标头（如CF-Connecting-IP为“可信”（我没有探索了所有情况，但最终我们可能还需要一个选项来指定受信任代理 IP 列表）。
On every rate limiter setting change we reset all collected rates data.每次速率限制器设置更改时，我们都会重置所有收集的速率数据。
In case the rate limiter is disabled from the settings the apis.RateLimit() middleware does nothing.如果从设置中禁用速率限制器，则apis.RateLimit()中间件不会执行任何操作。

v0.23.0-rc13 PrereleaserateLimit in middlewares_rate_limit.go seems to be private and cannot be called directly. Will it be made public in the future?

@cddqssc The rate limiter implementation has changed since the https://github.com/pocketbase/pocketbase/issues/1232#issuecomment-2272763235 announcement.
The rate limiter now is globally registered and it is controlled from the Dashboard > Settings > Applications > Rate limiting.


Contents of models.stubphp are incomplete

Describe the bug
Contents of models.stubphp are incomplete. I get errors about missing model class properties. E.g. id. This was not an issue in 1.4.0 version. I don't get this issue with _ide_helper_models.php generated by barryvdh/laravel-ide-helper.
Impacted Versions
1.4.1 and 1.4.2

Does laravel ide helper have any errors or anything? Are you able to share the code?

I have same problem in fresh Laravel Jetstream project, created with:
laravel new laravel-project --jet --teams --stack=inertia
Installed Psalm, this plugin and ide-helper. PhpStorm is able to find User::$id from _ide_helper_models.php but Psalm is not:

In case of @azurinspire this is caused by the same bug as in https://github.com/psalm/psalm-plugin-laravel/issues/146
A workaround is to replace:
$table->id();

within migration with:
$table->bigIncrements('id');

This creates same column.
Afterwards looking at models.stubphp one can find that stub was correctly generated
namespace App\Models{
/**
 * App\Models\User
 *
 * @property int $id
 *

I think this should be fixed now -- please reopen with code to reproduce if you are still experiencing a problem.


MAINT: set zip_safe to False in setup.py

Closes https://github.com/numpy/numpy-stubs/issues/36.
See the mypy docs:
https://mypy.readthedocs.io/en/latest/installed_packages.html#making-pep-561-compatible-packages
for more information.

Since this is a small change, I will merge it tomorrow unless someone complains.

Merged, thanks Josh


DSD-753 - Add validation for equal headers and data column

Fixes JIRA ticket DSD-753
This PR does the following:

Add validation for equal headers and data column to the Table component for unequal column lenths

How has this been tested?

Accessibility concerns or updates

Checklist:

[ ] I have updated the Storybook documentation accordingly.
[ ] I have added relevant accessibility documentation for this pull request.
[ ] All new and existing tests passed.

Front End Review:

[ ] View the example in Storybook

Tugboat has finished building the preview for this pull request!
Link:

https://pr994-dbhumcc20hwkixeqs9m3qkc7uure1res.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/62787a5d0f15004d8e88b213

Tugboat has finished building the preview for this pull request!
Link:

https://pr994-dbhumcc20hwkixeqs9m3qkc7uure1res.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/62787a5d0f15004d8e88b213

Tugboat has finished building the preview for this pull request!
Link:

https://pr994-dbhumcc20hwkixeqs9m3qkc7uure1res.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/62787a5d0f15004d8e88b213


Adding two permanent positions for Contract

Need to have

Permanent -- funded by University core funding
Permanent -- as long as funding is available

done


swalke : add commit stats

rubrics

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

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

Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 81.631%

Totals

Change from base Build<PHONE_NUMBER>6:
0.0%

Covered Lines:
951

Relevant Lines:
1165

💛  - Coveralls


use "exists" for environment vars instead of checking for non-zero

value
support first and last optimization via env vars

Google » souper #449 SUCCESS
This pull request looks good
(what's this?)

Raimondas can you take a quick look?

LGTM, please rebase to master before merging.

Google » souper #456 SUCCESS
This pull request looks good
(what's this?)


Trying to implement in a Golf Clubs community site, not working in Laravel 5.3  :\

Can you describe your issue more concrete, please? Right now it's looks like: Something isn't working. And it could be anything... Your domain, server hardware, server software, database, Laravel itself or  Messenger package...

closing for now, will reopen if needed after more details from @djamesfar are given


Handle importlib.metadata missing for Python 3.7

#1073  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @dongreenberg and the rest of your teammates on  Graphite


improve travis build - fix #45, #46

fix issues #45, #46
PR #47 is outdated by this PR

I can merge this first to master, but I won't be able to do release.
If I need to do release, I'll need to downgrade the gradle version to 5.x


Link configuration "supported" options to behaviour

These are currently ignored, other than as provided to the client via the discovery response. The client can still use unsupported options in requests and have them processed. For example

responseTypesSupported should be checked when processing an authorization request
algorithmsSupported should be checked in id token creation, user info responses, request object (when implemented) and  client auth signing. It may be sufficient to check some of them when registering the client, since the client's specific algorithms are stored with its data.

Both these and clientAuthMethodsSupported  should be checked when registering the client.

See also:
http://tools.ietf.org/html/rfc7591#section-2.1
and
http://openid.net/specs/openid-connect-registration-1_0.html#ClientMetadata
for information on validating the requested grant types based on supported response types.


Extend SEIRModel

[x] Remove ForwardModel base class
[x] Let SEIRModel inherit from pints.ForwardModel
[x] Init assigns default output_names, n_output, n_parameters
[x] implement n_outputs and n_parameters according to pints.ForwardModel
[x] implement set_outputs which can be used to select any subset of outputs (S, E, I, R or n_incidences)
[x] implement output_names which returns the output names (selected outputs)
[x] Refactor simulate to return set outputs only and remove flag return_incidences

#68 Completed


I can't write accented letters in Spanish

Basic information

Your Linux distribution and its version: ...

Your desktop environment: ...

Nuvola Runtime version: ...

The web app you use and its version: ...

Describe the bug

...
Steps to reproduce the behavior

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior

...
Debugging log

Attach the debug log here: ...
Screenshots

Attach screenshots here: ...
Additional context

...

I don't know why I saved it and went back to the original template


Why 'New-AzHDInsightCluster' returns a useless message

az feedback auto-generates most of the information requested below, as of CLI version 2.0.62

Describe the bug

I am following this guide. When I submit the request, I don't get a helpful message, and it only returns a cryptic message. How am I even supposed to debug this to understand what is wrong? What's the status code? and if you can even see the status code, what's the meaning? Is the code just wrong?
New-AzHDInsightCluster: Operation returned an invalid status code 'BadRequest'

https://docs.microsoft.com/en-us/powershell/module/az.hdinsight/new-azhdinsightcluster?view=azps-5.2.0#syntax
To Reproduce

Just run the code. Some details need to be modified.
# Metastore configs
$username = 'admin'
$password = 'PASSWORD12345678'
$secure_password = ConvertTo-SecureString $password -AsPlainText -Force
$hive_metastore_creds = New-Object System.Management.Automation.PSCredential($username, $secure_password)

$yarn_config = @{}
$yarn_config.Add("yarn.nodemanager.resource.cpu-vcores", "12")
$yarn_config.Add("yarn.scheduler.maximum-allocation-vcores", "12")

$hivesite_config = @{}
$hivesite_config.Add("hive.auto.convert.join.noconditionaltask.size","1145044992")
$hivesite_config.Add("hive.direct.sql.max.elements.values.clause","300")
$hivesite_config.Add("hive.direct.sql.max.elements.in.clause","300")

$config = New-AzHDInsightClusterConfig  `
    -ClusterType "interactivehive" `
    -HeadNodeSize "Standard_D13_v2" `
    -WorkerNodeSize "Standard_D14_v2" `
    -StorageAccountResourceId "storage_resource_id" `
    -StorageAccountKey "accountkey" `
    | Add-AzHDInsightMetastore `
        -SqlAzureServerName "sql_server_name.azurehdinsight.net" `
        -DatabaseName "sql_database_name" `
        -Credential $hive_metastore_creds `
        -MetastoreType HiveMetastore `
    | Add-AzHDInsightConfigValue `
        -Yarn $yarn_config `
        -HiveSite $hivesite_config

# SSH credentials configs
$username = 'sshuser'
$password = 'PASSWORD12345678'
$secure_password = ConvertTo-SecureString $password -AsPlainText -Force
$ssh_creds = New-Object System.Management.Automation.PSCredential($username, $secure_password)

# HTTP credentials configs
$username = 'admin'
$password = 'PASSWORD12345678'
$secure_password = ConvertTo-SecureString $password -AsPlainText -Force
$http_creds = New-Object System.Management.Automation.PSCredential($username, $secure_password)

New-AzHDInsightCluster `
    -Location "eastus" `
    -ResourceGroupName "autobenchkcheung" `
    -Version "4.0" `
    -ClusterName "kctest2383838" `
    -SshCredential $ssh_creds `
    -HttpCredential $http_creds `
    -ClusterSizeInNodes "2" `
    -Config $config

Expected behavior

Environment summary

Azure Cloud Shell
Additional context

Please open this issue in azure-powershell repo.

Please open this issue in azure-powershell repo.


samples/bluetooth/mesh-demo unable to send vendor button message

Running the zephyr bluetooth mesh sample from master branch on a BBC micro:bit, I get the following error when I press the button.
Seems the vendor button message is not being sent.
Any help would be appreciated!

*** Booting Zephyr OS build zephyr-v2.4.0-685-g7e64fba9ed84  ***
Initializing...
Unicast address: 0x0001
Bluetooth initialized
Mesh initialized
Loading stored settings
Using stored settings
Subscribing to heartbeat messages
button_send_pressed()
Unable to send Vendor Button message
Button message sent with OpCode 0x00c005f1

I have been tinkering with this a bit.
In prj_bbc_microbit.conf, if I set CONFIG_BT_SETTINGS=n instead of y, the micro:bit provisions successfully.
However, then I start getting -12 errors when binding, publishing and subscribing to models, which seems to mean that I am out of memory.
#define ENOMEM          12              /* Cannot allocate memory */
So, seems like past versions of Zephyr were a little smaller and fit nicely on the limited memory of a micro:bit, but these later versions are too big? Should I be assuming that??
Any guidance would be greatly appreciated! Thanks in advance!

Rolled back to Zephyr 1.14.
The example isn't fully working property, but it runs and in part, does what the demo intends.

Rolling back 1.14 LTS is the best option as there are things in play here:

The rewrite of BLE subsystem struggles with the m:bit memory constraints
The nRF51 series in being phased out
A new Micro:bit has just been released based on the nRF52833 (Nov 2020)

I expect in a matter of weeks there will support in zephyr for the latest Micro:bit

Thanks for your comment!


how do i use this?

Ive downloaded everything but I dont know how to run it, sorry if its a stupid question.

Install python, jupyter notebook and all other required python libraries. And open this with ipython/jupyter notebook.

Thanks! Didn't noticed this issue. The required module for https://github.com/tjwei/2048-NN/blob/master/my2048-rl-theano-n-tuple-Copy7.ipynb are theano, lasagne, selenium. plus, you also need to install the firefox browser.


Snapshots not available anymore

With the update of network parameters, V1 of go-tezos is not working anymore. However, as far as I could see, there is no documentation for V2 (at least I can't find one) and I can't find a way to query Snapshots in V2:
type *gotezos.GoTezos has no field or method SnapShot
Is there a way to get the assiociatedBlock of a cycle in V2? I used to do it like this and am looking for a working alternative in V2.
snapshot, err := gt.SnapShot.Get(cycle)
	if err != nil {
		fmt.Println(err)
	}
  percentageBlock := snapshot.AssociatedBlock

If this is not implemented in V2 right now, could we update V1 to support the new network parameters for now? Or maybe there is a better way to query this: I'm looking for a way to determine the percentage of total stake that is coming from the baker itself. Currently, I query the Snapshot, get the block_height of that cycle and query the balances.

Hi @fkbenjamin. go-tezos is still a currently evolving project, but the function you're looking for is gt.Cycle, which will give you the blockhash needed. Because it's evolving there may be breaking changes before the official v2.0.0 tag is used. Currently I am tagging things as alpha. I would recommend using V2 over V1, because even it's alpha state it is much safer and tested.
/*
Cycle represents the cycle RPC.

RPC:
	../blocks/<block_id>/context/raw/json/cycle/<cycle_number> (GET)
*/
type Cycle struct {
	RandomSeed   string `json:"random_seed"`
	RollSnapshot int    `json:"roll_snapshot"`
	BlockHash    string `json:"-"`
}

Here's the documentation you need: https://godoc.org/github.com/goat-systems/go-tezos

@fkbenjamin also I think we may be working on similar things, here is my payout script: https://github.com/goat-systems/tzpay
Specifically the delegation calculations happen here: https://github.com/goat-systems/tzpay/blob/master/internal/payout/payouts.go

Hey @DefinitelyNotAGoat ,
I already saw your tzpay script. This issue was not related to our payout software (working great for over a year now).
I rather wanted to use this to track earnings of bakers on a per-block basis (for auditing & reporting). I just realized that I probably can just copy that part over from our payout tool. I'll have a look at it again once I find the time and maybe submit a PR if I'm missing some functionality.

@fkbenjamin let me know if you need any help! Be happy to take a look at it with you!


Support /whois like command

The web and the desktop client will display some information about a participant's github details (name, email, etc).  I am not sure if its done via gitter or directly through github, but having the same displayed in /whois via IRC would be nice.

Pushing that user into the People list in the right sidebar and opening the user popover when the /whois command might be sufficient.


Some sticker sets not downloading correctly

All stickers in the set download as empty files (not blank but rather an image file without an image). Funnily enough all anime related. Some region locked some not.
what I've tried that failed. 569; 595; 564; 2462; 683
working ones: 2999; 3000; 3001; 2998; 2746; 2822; 2710; 925; 1917
they don't seem to follow the odd counting by 2's as the other issue. at first i thought it might be to do with how small the id's were but, 925 worked and 2462 failed.

Thanks for reporting, LINE store changes the scheme of the stickers' URL sometimes, and if that happens, I have to change some codes to cope with those changes.
From what I found so far, the issue seems to be:
This link will work
https://stickershop.line-scdn.net/stickershop/v1/sticker/4153/android/sticker.png
But this link wouldn't (I used this scheme in my code)
<EMAIL_ADDRESS>


[All] Add browserslist to fix babel transpilation

Description
Adds @babel/runtime-corejs3 to polyfil and transpile modern features to match our chosen browser matrix.
Fixes #549

This PR may have introduced an unintended side effect... corejs imports are left behind in the ESM bundles, pretty much turning corejs into a peer dependency, but I feel like that wasn't intended?
See here; I'm getting the same error in my project

@itsMapleLeaf thanks for that <3

It's "not" not expected shall we say haha 😀
We're working through adding polyfilling but obviously we've messed some stuff up.
Thanks for bearing with us…

@itsMapleLeaf We've removed the core-js polyfills. The latest release should work error-free for you now but let us know if not.

Awesome, the latest update seems to work without issues. Thanks for the quick update. ❤


Fix close deadlock

There was a race condition, where upon closing a conn and all its child
streams, a (potential?) issue in the underlying transport library caused
the 'Close' call to hang, leading to deadlocks in the peerstream code.
This changes the lock to only be held during the closed'ness check, and
should protect go-peerstream users against issues (misunderstandings?)
in the underlying transport libs.

I tried writing a test for this, but the race was very difficult to reproduce (reproduced once in 700 5 second runs). The ipfs dht tests do a better job of reproducing the issue (1/20 or so).

@whyrusleeping good find

alright, that should address the problem, without any changes needed in go-ipfs.

ok thanks
so many locks!


LoadProcess documentation and small improvements

Contains
removed NOP loadProcesses, docced implementors of StepBasedLoadProcess + the class itself, nothing implements directly LoadProcess anymore
How to test
Test nothing broke
Outstanding before merging
None of these is blocker, so might as well be left for another PR to make this one easily testable and reviewable

[ ] javadoc for lots of loadProcesses still missing
[ ] other loadProcesses might still be NOP, yet present

Hooray Jenkins reported success with all tests good!

I'm getting the NPE @Cervator mentioned even when only the Core module is active. I also get it when I run a headless server from IntelliJ.
This was when I had Core, MTE and WildAnimalsGenome active, but I got the same error in all cases.
java.lang.NullPointerException
    at org.terasology.logic.behavior.nui.BehaviorNodeFactory.addToCategory(BehaviorNodeFactory.java:183)
    at org.terasology.logic.behavior.nui.BehaviorNodeFactory.refreshPrefabs(BehaviorNodeFactory.java:177)
    at org.terasology.logic.behavior.nui.BehaviorNodeFactory.refreshLibrary(BehaviorNodeFactory.java:124)
    at org.terasology.logic.behavior.nui.BehaviorNodeFactory.postBegin(BehaviorNodeFactory.java:89)
    at org.terasology.engine.modes.loadProcesses.PostBeginSystems.step(PostBeginSystems.java:47)
    at org.terasology.engine.modes.StateLoading.update(StateLoading.java:234)
    at org.terasology.engine.TerasologyEngine.tick(TerasologyEngine.java:458)
    at org.terasology.engine.TerasologyEngine.mainLoop(TerasologyEngine.java:421)
    at org.terasology.engine.TerasologyEngine.run(TerasologyEngine.java:397)
    at org.terasology.engine.Terasology.main(Terasology.java:156)

@eviltak Sorry, I've kinda grown into ignoring all Github pings lately :sweat_smile: I'll try to take a look at it now.

Alright, I figured out LoadPrefabs is still needed, due to some lazy loading functionality on some of the functions way down the abstraction tree in AssetType.

Hooray Jenkins reported success with all tests good!


coap: finally get rid of hardcoded packet sizes at sending side.

We're abusing tinycbor's API, but at least we get to incrementally bump
a CoAP packet's buffer size (that should not happen frequently, since
the growth is exponential and it starts with a guessed typical OIC/cbor
payload size).
Signed-off-by: Gustavo Lima Chaves<EMAIL_ADDRESS>
@otaviobp: help with testing still welcome :)

lgtm and it didn't break any oic tests


False Positive on limits-dp-java-client-1.4.jar

False positive on library limits-dp-java-client-1.4.jar - reported as cpe:2.3:a:sun:sdk:1.4.0:::::::*
As per description , vulnerability is present in Double.parseDouble method in Java Runtime Environment (JRE) in Oracle Java SE and Java for Business 6 Update 23 and earlier, 5.0 Update 27 and earlier, and 1.4.2_29 and earlier

Without a packageUrl for this library we cannot fix it and suppression would be up to you, I cannot find where this library originates from.


Both methods for HOTP or TOTP code generation generates the same result

Is it an expected behavior that both methods for OTP generation keep generating the same value even when we set a time interval?

Time parameter is in ms, interval parameter is in seconds. You're probably passing in the wrong value for time or just not waiting long enough, because code stays the same for 30 seconds in your case. Try to pass in DateTime.now().millisecondsSinceEpoch and set interval to 1 second.

30 seconds = 30000 milliseconds
1 second = 1000 milliseconds

Well, if you pass 0, they will both return the same number. for HOTP tokens, you have to increment the value yourself every time, otherwise it will just keep generating the same number. the "time" and "count" parameters for TOTP and HOTP respectively are the seeds to the random number generator. If you pass in the same seed, you get the same result.
The only difference between TOTP and HOTP is that you use Time as the seed for TOTP, and I divide time by the interval (which defaults to 30 seconds). this is so that we get the same code for the whole 30 second slot.
If you set the Interval to 1 for TOTP and pass time into both functions, they will always be the same.
if you pass a time of 3 to TOTP and a count of 3 to HOTP (aka, the same seed), you should get different results, because the TOTP is divided by 1000 (to convert from milliseconds to seconds), then divided by the interval (default is 30), then floor(), which results in 0. while the HOTP will stay 3. which should give you 2 different codes.
30000 (30 seconds) for TOTP with default interval and 1 for HOTP will also give the same result.
since all the options are optional and have sane defaults, you can just do
int otpGenerated() {
    return OTP.generateTOTPCode("aRandOMSECRET", DateTime.now().millisecondsSinceEpoch)
}

To generate a new OTP every 30 seconds. If you need to generate 30 seconds before, and 30 seconds after for safety or server-side checking. you can just do DateTime.now().millisecondsSinceEpoch-30000 and DateTime.now().millisecondsSinceEpoch+30000 accordingly in your code if you ever need that.
But since this looks like a Flutter app, and is client side, you only need the current code.
I hope this helps.

Havent heard a response in a while, I closing this. If its still an issue, please reopen.


:hammer: Add _helpers.tpl and NOTES.txt to Helm chart and refactor labels

This PR includes:

standard labels using a _helpers.tpl  file to create those labels.
removed all the creationTimestamp inputs.
Included a NOTES.txt file to help users after the installing.
Service Account Name also generated by _helpers.tpl  file.

Also please be aware of the recently added 12-ngingx-config.yaml file (https://github.com/kubeshark/kubeshark/pull/1392)

Nice! I'll include the labels on that config than.

Codecov Report
Patch and project coverage have no change.

Comparison is base (65bda4e) 7.25% compared to head (6ea4d8f) 7.25%.

:exclamation: Current head 6ea4d8f differs from pull request most recent head 224f5c8. Consider uploading reports for the commit 224f5c8 to get more accurate results

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@          Coverage Diff           @@
##           master   #1406   +/-   ##
======================================
  Coverage    7.25%   7.25%
======================================
  Files          47      47
  Lines        2962    2962
======================================
  Hits          215     215
  Misses       2737    2737
  Partials       10      10

Files Changed
Coverage Δ

config/configStructs/tapConfig.go
0.00% <ø> (ø)

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


A text search that produces an exact match should go to that (label) page and not do full-text search

Have the user do Lucene queries for true full-text? Or have a mode toggle?

This query for an exact node label match should done in the search box, and not part of the search results page. We want to direct traffic away from the search results page.


Updated UI

Some UI improvements suggested by Jarno Van Driel.

Email from Jarno:

Thad Guidry pointed http://arborjs.org/halfviz/ out to me (http://arborjs.org/introduction), and since then I've been spending roughly 16 hours turning http://linter.structured-data.org/?url=http:%2F%2Fwww.cosmeticsurg.net%2Fblog%2F2015%2F01%2F23%2Fbreast-lift-scar-options%2F into a graph.

Yup, that’s a force-directed graph. And, you’re right that it can rapidly get out of hand. Probably need some way to collapse information down within some radius of the starting node.

Now if you copy the content of the attached files into http://arborjs.org/halfviz/ you'll that without zooming the browser out multiple time s (CTRL -), getting a proper view is impossible due to the amount of structured data. Now the reason why I'm using this page for the example is that only when one start to add extensive structured data does it become clear what additional type of functionality one might need.

Of biggest importance to me is to have a tool that helps me cut down on time I need to spend on analyzing HML as if it's Y2K all over again and let's me study a visual graph that I can zoom in/out and pan with but that also highlights certain types/properties and lets me explode nodes either by default or by double clicking/tapping them (got that idea from Allegrograph).

Yup, that’s the purpose of the tool. I think we can get most of the way there, and leave room for improvement. We’ll definitely need a way to get from the RDF to something that halfviz can handle, which I’ll need to look into.


Remove en-US locale from links

Learn Build status updates of commit 8b96ce2:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: CannotMergeCommit] Cannot merge commit 8b96ce253d62f3b26c1d71b9334df4e3b5a542a9 in branch patch-12 of repository https://github.com/riverar/windows-uwp into branch docs (commit d4ee8b23691334ec752f6302cdfde14486bbcba1). Please follow this documentation: https://help.github.com/articles/resolving-a-merge-conflict-using-the-command-line/ to use git.exe to resolve you content conflicts locally and then push to remote.

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 Learn 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 or suggestions 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 learn.microsoft.com contributor guidesPost your question in the Learn support channel


Can't insert using collection2

When using aldeed:collection2 i got this error: "When the modifier option is true, validation object must have at least one operator" when i try to add object.
Events = new Mongo.Collection("events");

var EventSchema = new SimpleSchema({
  "name": {
    type: String,
    label: "Name"
  },
  "description": {
    type: String,
    label: "Description"
  }
});

Events.attachSchema(EventSchema);

$scope.events = $meteor.collection(Events, false);
      $scope.events.save(newEvent)
      .then(function() {
        $state.go('events');
      }, function(err) {
        if (err && err.message)
          console.log('err', err.message);
      });

This is a angular-meteor problem or mine ?
Thanks for help

Can you please show how does 'newEvent' object looks like? Maybe it has to do something with the data you are trying to save.

The save syntax will be depracated in favor of the new helpers syntax in Angular Meteor 1.3.
Please try to migrate and tell me if it worked for you


Sync from main

Advantage 360 Pro PR template
What's changed:
Why has this change been implemented:
What (if any) actions must a user take after this change:

Please don't open PRs into upstream for personal keymap changes

Please don't open PRs into upstream for personal keymap changes

I have no idea how I created this pr accidentally, is there some trigger happening here or something incorrect in the upgrade documentation? I seem to be not the only one.


Include revision metadata in northbound objects to allow for atomic operations

The backend client KVPairs include revision data which allows atomic operations on objects to be performed.  The revision data is not passed up to the northbound resource definitions and so it is currently not possible to perform atomic operations using the northbound API.

Matt - I think you were at least thinking about this from your work in libcalico.

This is not required for GA, but will likely be required further out.

@heschlie this might be a good ramp-up candidate for you on libcalico-go.
@robbrockbank is it OK is @heschlie takes this? Is this somewhere in your plan?

@gunjan5 you did a little bit of work on this - is this closed? Is there more to do here?
Looks like maybe only some of the objects have revision data passed through?

@caseydavenport yeah it's only exposed for WEP at the moment. @robbrockbank has a branch for work on completely restructuring libcalico-go which will have this included (maybe not by the same means)

Will add these as needed by a particular feature.


GH2463: Support Context for items function in DoesForEach

Fixes #2463

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

Do i have to do anything?

@farodin91 your changes have been merged, thanks for your contribution 👍


Change way ids are being parsed

Closes #82
I've ignored moving actual object validation to binding, as there's #83 for that.

Shame direct parsing to uuid.UUID doesn't work, but oh well.


chore(vite-node-miniflare): simplify examples

Picking up trivial changes from https://github.com/hi-ogawa/vite-plugins/pull/594

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#633  👈
#632
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @hi-ogawa and the rest of your teammates on  Graphite

Merge activity

Aug 28, 11:09 PM EDT: @hi-ogawa started a stack merge that includes this pull request via Graphite.


Remove version: BlenderFoundation.Blender.LTS version 2.93.5

No longer part of LTS
Microsoft Reviewers: Open in CodeFlow

URL: https://download.blender.org/release/Blender2.93/blender-2.93.5-windows-x64.msi
OK
(Automated message - build 887)


CIV-7897 Update Event Roles

JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/CIV-7897
Change description
Update Event Roles
Testing branch and instruction
Test events with each type of user
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[x] No

same changes required in ga ccd plus the high level data setup


add homework 2

.\homework2\task_one.py:12:80: E501 line too long (93 > 79 characters)
I do not know how to make this line even shorter. I have already shortened the names of variables as much as possible.

Поправить flake8 ошибку и можно мёрджить.


Controller : Duplicate NodeId ? (CON-972)

I have a Matter Controller with a Matter Wifi-Node and a Matter Thread-Node commissioned to its fabric. Now I want to subscribe to a attributeo of the wifi and the thread node. At first I had the problem, that  when I subscribed to one of the nodes they lost the connection (kicked out of the fabric).
After an esp-matter update this seems not to happen anymore.
Now I can subscribe to one of the nodes - but not both. I`am not completely shure but it seems that both devices (including the controller) has got the same nodeId ?! (0x27A).
If I delete the wifi node to which i was able to subscribe to, i can subscribe to the thread node, which did not work previously.
For all devices I created a custom partition with the mfg_tool.
What did I forget ? Is it rellay correct to use the qrcode string (without MT:) generated by mfg_tool as nodeId for commissioning and subscribing (as the documentation suggestes https://docs.espressif.com/projects/esp-matter/en/latest/esp32/developing.html#pairing-commands)  ?

I already solved the problem on my own. I obviously have to assign the id myself when pairing. Then it will work.


Chef and division

problem : https://www.codechef.com/JAN21C/problems/DIVTHREE
I can do this in C++ kindly assign it to me .

I can do this in C

@TechSpiritSS Complete all previous works :)

@Nivedita967  i would like to contribute in python

@Nivedita967 I would like to contribute in java

@TechSpiritSS Complete all previous works :)

Done ; )

isn't this same as #156 ?

Yes, this is the same question but from different link

Yes, you are right. This issue is already open and happily @kanak22 wants to contribute in C++ and that issue is also open for C++ :)
@kanak22 please comment on the issue #156

I commented 😊


Serial Routing with Teams channels as Agent

The documentation doesn't specify how Serial Routing works when the Agent is a teams channel.
Is there a way to set the order of the member of the group?
What criteria does it use to ring users?

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

ID: b755d9ae-13ac-019f-f5f4-a95b1eee2989
Version Independent ID: f7afd8ad-e6e0-6c79-aed1-4ee87d48ba12
Content: Create a call queue in Microsoft Teams - Microsoft Teams
Content Source: Teams/create-a-phone-system-call-queue.md
Service: msteams
GitHub Login: @CarolynRowe
Microsoft Alias: crowe

@javiernegro thank you for reaching out and helping improve Microsoft Docs. I've found some tech support articles that might be helpful. Please let me know. https://techcommunity.microsoft.com/t5/skype-for-business-preview/call-queue-serial-routing-agent-order/m-p/126117 and https://docs.microsoft.com/answers/questions/433192/order-agent-list-for-34serial-routing34-using-34vo.html

Yes, I saw those. My comment  was to clarify and improve current docs.
Thanks!


Removed deprecated auth methods and functionality

@katowulf - Since we are going to do a major version bump, we can get rid of some old cruft. This removes all deprecated methods and functionality in our auth stuff. Here is a list of changes:

The previously deprecated ability to pass in credentials to the user management methods of $firebaseAuth as individual arguments has been removed in favor of a single credentials argument. We now throw a helpful error message.
The previously deprecated $sendPasswordResetEmail() method has been removed; use the equivalent $resetPassword() method instead.

Coverage increased (+0.15%) to 92.85% when pulling 8470e776edb42ab5bbb882df3179b65dd752b504 on jw-auth-creds into b48e0366b44dd94b9eecdaa60b723bf7191620f5 on master.

One minor suggestion. Feel free to ignore.

@katowulf - Good call on the messages. I took your suggestion. Back to you!

Coverage increased (+0.15%) to 92.85% when pulling b88d2737137f8cb4073126325ec99226c15bb404 on jw-auth-creds into b48e0366b44dd94b9eecdaa60b723bf7191620f5 on master.


minor changes

Description of the change: minor changes to .then() argument to make it consistent with other code style.
Reason for the change:
Link to original source:

Closes #

Please check out our open issues for changes that we’re looking for in this repo! We’re adding new ones from easy to difficult every day.


Paginated offer get method

Added paginated offer get method, still needs some fine tuning, @gzili  please inform me, what kind of return would you like to have if page index or page size are too small or to big.

Closing this, as it was coded on an incorrect base branch and #61 is a follow up that has all the issues fixed.


Renderer=Harfbuzz still loads the font without HarfBuzz

Description
A sentence or two describing the issue.
Check/indicate

[ ] Relevant for XeTeX
[x] Relevant for LuaTeX
[x] Issue tracker has been searched for similar issues?
[ ] Links to <tex.stackexchange.com> discussion if appropriate

Minimal example demonstrating the issue
\documentclass{article}
\usepackage{fontspec}
\setmainfont{Source Han Sans SC}[Renderer=Harfbuzz]
\begin{document}
你好
\end{document}

Further details
The above will still load the font without HarfBuzz when checking for font features, which 1) might give wrong results when HarfBuzz and default font loader disagree on supported font features and 2) more importantly, can freeze one’s system when loading huge fonts like this where the point of using HarfBuzz in this case would be to avoid such freeze (the default font loader consumes enormous amounts of memory the first time it tries to load a huge font like this).

This should be fixed by #397.

Thanks Marcel, much appreciated!


Add a pop up dialog box for subscribing to newsletter done ! #541 ✨✨

Hello, PR
Add new feature - #541
Add a pop up dialog box for subscribing to newsletter done properly -

📄 Type of Change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update (adds or updates related documentation)

✅ Checklist

[ ] I am a participant of GSSoC-ext.
[ ] I have followed the contribution guidelines of this project.
[ ] I have made this change from my own.
[ ] I have taken help from some online resources.
[ ] My code follows the style guidelines of this project.
[ ] I have performed a self-review of my own code.
[ ] I have added documentation to explain my changes.

Mandatory Tasks

[ ] Make sure you have self-reviewed the code. A decent size PR without self-review might be rejected.

🤝 GSSoC Participation

[ ] This PR is submitted under the GSSoC program.
[ ] I have taken prior approval for this feature/fix.

@YadavAkhileshh done, please merge my PR with level 3, If you check other projects, they are also giving level 3 for this..

You can check my code..
Thank you 🙏🏻


ANCHOR-231: Fix indicative quote fails validation

PR Checklist
PR Structure

[ ] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[ ] This PR avoids mixing refactoring changes with feature changes (split into two PRs
otherwise).
[ ] This PR's title starts with name of package that is most changed in the PR, ex.
paymentservice.stellar, or all or doc if the changes are broad or impact many
packages.

Thoroughness

[ ] This PR adds tests for the most critical parts of the new functionality or fixes.

What
Indicative quote fails validation in previous versions
Why
Bug fix
Known limitations
N/A

Something went wrong with PR preview build please check

Reference Server Preview is available here:https://anchor-ref-pr810.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr810.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr810.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr810.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr810.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr810.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr810.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr810.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr810.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr810.previews.kube001.services.stellar-ops.com/