fix FileNotFound Error for local runs with uri_files and AzureMLDataset

Small modifications to the _load function so that uri_file are correctly downloaded for local runs. See issue #68

@fdroessler will it fix the problem mentioned in #68 ?

@marrrcin Yes I was able to reproduce this, not sure why it worked once before for me but in a new setup I can confirm this fixes the issue. @Gabriel2409 can you adjust the comment copied over so it does not refer only to the case in the uri_folder dataset but in general to the case that if the path points to a file we need to take the parent.

@fdroessler,
I suppose you are talking about these comments:   # if the filepath is to a file in the folder dataset we need the parent folder and # we take the relative within the Azure dataset to avoid downloadingall files in a folder dataset. I can just remove the word folder if needed.
However, even if the solution I proposed works as a workaround, I would like to mention that it leads to download all of the files in the same folder as the uri_file dataset, which may not be what we want as a uri_file dataset is supposed to be only one file.
Indeed, we still loop over the parent: see for fpath in fs.ls(path_on_azure)
Maybe a better solution would be to call fs.download directly on path_on_azure for uri_file datasets, like this
    def _load(self) -> Any:
        if self._download:
            try:
                azureml_ds = self._get_azureml_dataset()
            except ResourceNotFoundError:
                raise VersionNotFoundError(
                    f"Did not find version {self.resolve_load_version()} for {self}"
                )
            fs = AzureMachineLearningFileSystem(azureml_ds.path)
            if azureml_ds.type == "uri_file":
                # relative (to storage account root) path of the file dataset on azure
                path_on_azure = fs._infer_storage_options(azureml_ds.path)[-1]
                logger.info(f"Downloading {path_on_azure} for local execution")
                # using APPEND will keep the local file if exists
                # as versions are unique this will prevent unnecessary file download
                fs.download(path_on_azure, self.download_path, overwrite="APPEND")
            elif azureml_ds.type == "uri_folder":
                # relative (to storage account root) path of the folder dataset on azure
                dataset_root_on_azure = fs._infer_storage_options(azureml_ds.path)[-1]
                # relative (to storage account root) path of the dataset in the folder on azure
                path_on_azure = str(
                    Path(dataset_root_on_azure)
                    / self._dataset_config[self._filepath_arg]
                )
                # if the filepath is to a file in the folder dataset we need the parent folder
                if fs.isfile(path_on_azure):
                    path_on_azure = str(Path(path_on_azure).parent)
                # we take the relative within the Azure dataset to avoid downloading
                # all files in a folder dataset.
                for fpath in fs.ls(path_on_azure):
                    logger.info(f"Downloading {fpath} for local execution")
                    # using APPEND will keep the local file if exists
                    # as versions are unique this will prevent unnecessary file download
                    fs.download(fpath, self.download_path, overwrite="APPEND")
            else:
                raise ValueError("Unsupported AzureMLDataset type")
        return self._construct_dataset().load()

This leads to a bit of code duplication but at least you only download the dataset and not other files. And this has the benefit of completely separating uri_file and uri_folder usecases.
Note that even though this solution seems to work in my case, it leads to failed test cases. I think if this alternate solution is chosen, part of the mock classes and functions will need to be rewritten.
What do you think?

You're right, I remember thinking of the case of a file sitting in a folder full of files and that leading to huge download load. So I don't think this is a viable option.
But how about this, we check in both cases (folder/file dataset) if the final path is a file. If yes we download it straight away, no need for fs.ls() of determining the parent we already have everything we need. Only in cases of partitioned folder datasets or when we want to download a whole folder we go through individual file download case using fs.ls().
            fs = AzureMachineLearningFileSystem(azureml_ds.path)
            if azureml_ds.type == "uri_file":
                # relative (to storage account root) path of the file dataset on azure
                path_on_azure = fs._infer_storage_options(azureml_ds.path)[-1]
            elif azureml_ds.type == "uri_folder":
                # relative (to storage account root) path of the folder dataset on azure
                dataset_root_on_azure = fs._infer_storage_options(azureml_ds.path)[-1]
                # relative (to storage account root) path of the dataset in the folder on azure
                path_on_azure = str(
                    Path(dataset_root_on_azure)
                    / self._dataset_config[self._filepath_arg]
                )
            else:
                raise ValueError("Unsupported AzureMLDataset type")
            # if the filepath is to a file in the folder dataset we need the parent folder
            if fs.isfile(path_on_azure):
                fs.download(path_on_azure, self.download_path, overwrite="APPEND")
            else:
                # we take the relative within the Azure dataset to avoid downloading
                # all files in a folder dataset.
                for fpath in fs.ls(path_on_azure):
                    logger.info(f"Downloading {fpath} for local execution")
                    # using APPEND will keep the local file if exists
                    # as versions are unique this will prevent unnecessary file download
                    fs.download(fpath, self.download_path, overwrite="APPEND")

@fdroessler I modified the code following your suggestion. Note that I had to pass the path as a string because the tests were failing if not (even though the actual download when testing it with real azure uri_file datasets in local runs succeeded).
The reason comes from fsspec:
def expand_path(self, path, recursive=False, maxdepth=None, **kwargs):
	...
	# No check for Path objects, only strings
    if isinstance(path, str):
        out = self.expand_path([path], recursive, maxdepth)
    else:
        out = set()
        # Leads to PosixPath object is not iterable error during tests
		# which is fortunate because at least the error is not silent
		path = [self._strip_protocol(p) for p in path]

Please note that, as mentionned above, I also removed lines from the _infer_storage_options of the AzureMLFileSystemMock class. To be honest, I don't know why they were present in the first place as the same operations were performed inside the _load function but I may be missing something here.
Please tell me if you think further modifications are needed.

lgtm, I think the lines in test were left overs before I got the fs.is_file working.

Looks OK, @Gabriel2409 please update CHANGELOG.md.

Hi @marrrcin I updated the changelog. If you are ok with the PR, you can also close the associated issue when merging.
Thank you :)

Thanks for your contribution @Gabriel2409


Cannot see the entire interface with browser set to 100%

On both my widescreen monitors, both in Chrome and Edge, I have to reduce my zoom to see the entire interface.

Not a big deal - just FYI

Thanks for reporting - the fixed heights and sub containers need improvement.. indeed zooming out is the current solution.

Recent improvements (in v2.1.0) should alleviate this.. but not necessarily fully resolve - so still keeping this issue open.


Server hangs in shutting down

Bug Report
What did you do?
Trying to restart prometheus server.
What did you expect to see?
Clean shutdown / start.
What did you see instead? Under which circumstances?
Hangs in shutting down after msg="Scrape manager stopped":

Environment

System information:

$ uname -srm
Linux 3.10.0-862.14.4.el7.x86_64 x86_64

Prometheus version:

prometheus, version 2.4.3 (branch: HEAD, revision: 167a4b4e73a8eca8df648d2d2043e21bdb9a7449)
  build user:       root@1e42b46043e9
  build date:       20181004-08:42:02
  go version:       go1.11.1

This could be fixed by https://github.com/prometheus/prometheus/pull/4922. You could try v2.6.0-rc.0 and see if it helps.

Cool. Thanks.

@spjspjspj Did #4922 fix the problem?


Add support for abort notification

Successful PR Checklist:

[ ] Tests

[ ] (not applicable?)

[x] Documentation

[ ] (not applicable?)

PR label(s):

[x] https://github.com/procrastinate-org/procrastinate/labels/PR type%3A breaking 💥
[x] https://github.com/procrastinate-org/procrastinate/labels/PR type%3A feature ⭐
[ ] https://github.com/procrastinate-org/procrastinate/labels/PR type%3A bugfix 🕵
[ ] https://github.com/procrastinate-org/procrastinate/labels/PR type%3A miscellaneous 👾
[ ] https://github.com/procrastinate-org/procrastinate/labels/PR type%3A dependencies 🤖
[ ] https://github.com/procrastinate-org/procrastinate/labels/PR type%3A documentation 📚

Context
So far, the only use of LISTEN/NOTIFY has been to let the worker know that a new job is ready to be processed.
This extends listen_notify to accept different types of notification through using the payload:

job_inserted: the existing notification
abort_job_requested: when a running job is requested to be aborted.

Instead of making explicit calls to the database every time a job calls should_abort, the worker uses the same mechanism than for detecting new jobs, a combination of Postgres notification and polling.
Other changes
The edge case of retrying a job to be aborted is handled at the database level instead of the worker.
Next steps
Once this change lands in v3, it will help with cancellation during the shutdown event and #1084

Excellent. I will have a look at it tomorrow 🙂. I think it's a good idea to handle more stuff at the database level (as long as we can still test it).

Excellent. I will have a look at it tomorrow 🙂. I think it's a good idea to handle more stuff at the database level (as long as we can still test it).

It also removes some race condition with the previous approach.
Glad you had an acceptance test written against it already or I will have surely missed it.

Excellent. I will have a look at it tomorrow 🙂. I think it's a good idea to handle more stuff at the database level (as long as we can still test it).

It also removes some race condition with the previous approach.
Glad you had an acceptance test written against it already or I will have surely missed it.

Cool, let me know when it's ready for review (still a draft currently and some connector excluded from tests/acceptance/test_async.py). But looks pretty good at first glance.

So, after updating those docstrings, I have nothing to add. You contributed a great feature here. Thanks a lot 🙂.
@ewjoachim Do you have something to add?

https://github.com/procrastinate-org/procrastinate/pull/1179
PR should be merged quickly. Sorry for the mess (it's all GitHub's fault :D )

#1179 PR should be merged quickly. Sorry for the mess (it's all GitHub's fault :D )

No worries. I can rebase once it lands to v3 branch

v3 is up to date :)

Rebased and squashed
All 🟢  🥳

And Merged !


WORK IN PROGRESS - Saves more important output on predictiontest models

What
records training and evaluation error on the predictiontest models

The training error is the mean error over the training sample
The test (or generalization) error is the expected prediction error over an independent test sample:

Why
This information can help us know if we're under-fitting or over-fitting the data.
Feedback
Feedback welcome.

Closing this for  now.


Exception raised in ColormapDialog

With recent updates of the ColormapDialog, I get this error from time to time:
Exception ignored in: <bound method ColormapDialog._itemAboutToFinalize of <silx.gui.dialog.ColormapDialog.ColormapDialog object at 0x7f12b196d438>>
Traceback (most recent call last):
  File "/users/tvincent/src/silx/build/lib.linux-x86_64-3.7/silx/gui/dialog/ColormapDialog.py", line 1201, in _itemAboutToFinalize
    self.setItem(None)
  File "/users/tvincent/src/silx/build/lib.linux-x86_64-3.7/silx/gui/dialog/ColormapDialog.py", line 1148, in setItem
    self._invalidateData()
  File "/users/tvincent/src/silx/build/lib.linux-x86_64-3.7/silx/gui/dialog/ColormapDialog.py", line 957, in _invalidateData
    if self.isVisible():
RuntimeError: wrapped C/C++ object of type ColormapDialog has been deleted

PyQt or PySide?
Maybe we can check together.
Looks like the dialog receive event after it was closed?

Hmmm... Which is the case most of the time for _itemAboutToFinalize, makes sense.


[bug_fix] bug fix zoneout lstmcell layer

This patch enable zoneout even though the zoneout rate is smaller than epsilon

Signed-off-by: hyeonseok lee<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #1921. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.


WIKISEARCH argument for # of results returned

Lots of potential here... but I only want WIKISEARCH to return 1 result, so I can have many searches going on a single sheet (one per row) and only receive the top result.
Currently, if I try to duplicate the formula in each row, I get #REF Error: Array result was not expanded because it would overwrite data in C3. The search is broken by any data in the cells below it. Ignoring this error and writing the cells anyway would be an easier fix in theory, but I'm not sure if that's possible.
Is that possible?
Thanks!

Also, it would be helpful to have the URL of the search result appear in the next column, so we can visit the page.

Nevermind, I was able to do this with Google App Script, then I'll use your app for the rest of the functionality.

Thanks for letting me know. You can also use the INDEX() function directly in Sheets.


[feat]: added parameter validation check within 'ensureFileExixts' function

In a situation where external developers misuse the function, a well defined error would be helpful!

Regarding the filePath check in ensureFileExists, I have a few thoughts

Given the nature of our library, error handling might be more valuable for input files (pages or app directory) rather than output files
The main purpose of ensureFileExists seems to be creating files when they don't exist, as the name suggests
TypeScript already handles filePath as a required parameter

Suggestion: How about moving the error handling focus to input file validation instead of the current location?


Path as an argument to LibPQ

Maybe:
Allow optional second argument to LibPQ() that passes the path record to override LibPQPath

I needed that myself so implemented using Lines.FromBinary(#"Parameter Name"), and just used the parameter to store local path to text file.

Thanks for your feedback!
Unfortunately I don't quite get what you mean: did you encounter a use case where you've needed to override module path for a single LibPQ call? Why was it not ok to add that path to the workbook's LibPQPath record with a higher priority? I'm trying to understand your use case better to be able to design a convenient API that would help. I have not encountered such case myself yet.

Hello again!
First what I have done.
My LibPQPath query in Power BI looks like this:
let
    LibPQPath = [
        Local = Lines.FromBinary(File.Contents(#"LibPQPathLocal Location")),

        Web = {
            "https://raw.githubusercontent.com/sio/LibPQ/master/Modules/",
            "https://raw.githubusercontent.com/tycho01/pquery/master/"
        }
]
in
    LibPQPath

Then in #"LibPQPathLocal Location" (a parameter) I assign the local path to a text file:
C:\........\ProjectName\LibPQPathLocal.txt
And finally LibPQPathLocal.txt looks like this:
C:\........\ProjectName\LibPQ-master\Modules
C:\........\ProjectName\LibPQ-master\Tests
C:\........\ProjectName
C:\........\ProjectName\Database
C:\........\ProjectName\Demand

It just felt better to me something that is project specific to be abstracted out of your project's code, and following an easy given format imported from a project file.
That way you can:

edit the file in the editor of your choice
don't have to change the .pbix/.pbit every time you add a new path, with commit's changed files being more explicit about what it is that changed
have version control of the path strings independently and review more easily if needed (don't have to open .pbix/.pbit file to review)
possibly add auto-descovery functionality with any way it can be achieved, I myself don't currently know how exactly this would be done but I can imagine several ways and may implement it later myself
if you share a project with someone (maybe a client) and you use LibPQ to manage your queries outside Power BI and then import them with LibPQ(), the paths must be exactly the same on each computer, which can produce problems - using a parameter for where the project's paths file is located, I believe you have a cleaner/quicker solution for the recipient to enter just one parameter when opening the file and everything works as expected (in this situation I believe you want your paths to be constructed dynamically using the parameter as root and joining the following path from file line by line - haven't implemented myself yet).

I don't fully understand the relevance concerning Power BI Service at the moment as I have not yet set-up a development environement there, since I am building my first project with these tools and am still in the Data Modeling phase of it. If/when I come upon more relevant information I will share here.
Thank you so much again for your wonderful project.

Now I understand what you've meant :) It's a valid concern even though it's not what this issue was about.
I was considering allowing to override the values provided by LibPQPath for some calls, e.g. if you'd want LibPQ("CoolModule") to return one thing in one report and another thing in another one - both in the same workbook. I have not encountered a use case for this idea, so I've just left it at that.
Now about your case. LibPQPath is decoupled from the main LibPQ code exactly to allow hacks and customizations like yours. You can fetch and/or calculate the LibPQPath value in any way you think is best for your project. File imports are a tricky matter though, and while Formula.Firewall errors are not a big deal for you, they might be for some other users - that's why I think the default behavior should not be changed.
Looks like your case and other possibilities of customizing LibPQPath should be documented. I'll try to do that some time :) Thank you for your feedback!

I truly agree both about the customization it allows and the default behavior having to be as is.
Do you think automatic discovery of sub-folders of mentioned paths in LibPQPath, for by-default supporting importing modules from project's sub-folders, would be possible, worthwhile, or of interest? Is it possible with M or are there limitations? If there are, what about R and Python scripting? Only asking for your thoughts since you will have more experience with this kind of thing than me, not an implementation, with much respect for your time 😛

M provides enough tools to implement file discovery in subfolders. By parsing the output of Folder.Files and Folder.Contents you could add all subfolders of a given folder to LibPQPath if you want to go that way. If by project subfolders you meant paths relative to the location of workbook, then it's not as simple. You would need to invoke some external mechanism to calculate the path to current workbook. See example here.
I've briefly considered the tree-like structure for modules when I first started with LibPQ, but I've decided not to go that way. It would require making too much assumptions and would result in too much implicit behavior that the user would need to read the docs to understand.
For example, resolving the name collisions. As it is now, if you have two folders is LibPQPath and you have modules with the same name in both of them, it is very simple to understand that the folder that comes first will be used. It would not be as explicit if those were automatically discovered subfolders of an item in LibPQPath. What sort order would we use? Alphanumeric? In which locale? Or maybe we would require user to explicitly say which subfolder to use? But that would not be much different from what we have now, except we would need to define the separator character - again implicitly and counterintuitive for some users.
PS: While I was writing this reply I've discovered that LibPQ already supports import from subfolders :-D You can try it with LibPQ("Subdir/Module"). This behavior is entirely accidental and should not be relied upon though.


MapLibre Main page shows Crimea as Russia

MapLibre Mainpage shows Crimea as Russia (https://maplibre.com/),
despite it being still officially a Ukrainian territory and despite the fact
that the MapLibre logo contains the Ukrainian flag as a symbol of Support.
This creates a very poor PR image for the whole Project.

Thanks for reporting @jelenjelinek. See #9


let dev server port  configurable  DEFAULT_PORT

Pre-flight checklist

[X] I have read the contribution documentation for this project.
[X] I agree to follow the code of conduct that this project uses.

Problem description
const d = debug('electron-forge:plugin:webpack');
const DEFAULT_PORT = 3000;
const DEFAULT_LOGGER_PORT = 9000;
in my project  i need  the port 3000  but   that will be  conflict
Proposed solution
let dev server port  configurable
Alternatives considered
show what port we can't use
Additional information
No response

this is already configurable: https://js.electronforge.io/plugin/webpack/interfaces/webpackpluginconfig#port


Tutorials of MCTS or Retro* for single retro synthesis model 【Help wanted】

I have noted that you provided the use of AndOr_BreadthFirstSearch in 1- end to end retrosynthesis with a toy model.ipynb and reproduced PaRoutes in 2a- advanced algorithms on PaRoutes.ipynb.
Could you provide an example of using Retro* or MCTS for template-free models like Root-Aligned? Many Thanks.

I have noted that you provided the use of AndOr_BreadthFirstSearch in 1- end to end retrosynthesis with a toy model.ipynb and reproduced PaRoutes in 2a- advanced algorithms on PaRoutes.ipynb.
Could you provide an example of using Retro* or MCTS for template-free models like Root-Aligned? Many Thanks.

Hello @zw-SIMM. You can put one of the template free models into the syntheseus wrapper class here: https://github.com/microsoft/syntheseus/blob/main/syntheseus/reaction_prediction/utils/syntheseus_wrapper.py
However, as mentioned in #58 we are in the process of making this simpler, and in the upcoming v0.4.0 release you will be able to directly put the template-free models into the algorithms without a wrapper class. I hope this release will be available in 1 week or so.

Thanks again for your nice reply! Great work!! Really Learn a lot!!!

We're still working towards the simplification mention by Austin, but in the meantime, this tutorial shows explicitly how to use the wrapper class. Hope this helps and let us know if you run into any issues.

We're still working towards the simplification mention by Austin, but in the meantime, this tutorial shows explicitly how to use the wrapper class. Hope this helps and let us know if you run into any issues.

Clear! Run successfully!


MetalDock Setup -- ORCA (Linux)

In the MetalDock setup instructions for ORCA, it says to export the full path to the binary as follows:
export ASE_ORCA_COMMAND='/full/path/to/orca/orca PREFIX.inp > PREFIX.out'
However, the ORCA instructions indicate the following exports:
export PATH="/home/rjrich/zware/orca:$PATH"
export LD_LIBRARY_PATH="/home/rjrich/zware/orca:$LD_LIBRARY_PATH"
I have the above two exports in my ".bashrc" file.
For parallel execution, the ORCA instructions recommend calling orca with the full path:
$PATH/orca $jobname.inp
Is the same thing accomplished by adding the line you have suggested to my ".bashrc" file?
export ASE_ORCA_COMMAND='/full/path/to/orca/orca PREFIX.inp > PREFIX.out'
I currently have it set up this way in ".bashrc":
ORCA 5.04
export PATH="/home/rjrich/zware/orca:$PATH"
export LD_LIBRARY_PATH="/home/rjrich/zware/orca:$LD_LIBRARY_PATH"
export ASE_ORCA_COMMAND='/home/rjrich/zware/orca/orca PREFIX.inp > PREFIX.out'
However, when I try to run MetalDock with more than 1 cpu process, ORCA crashes. On the other hand, it runs with ncpu = 1. So, it seems that something is wrong with my parallel setup for ORCA.
Thanks.
-- RJR

Fixed this issue via email correspondence.


Correct format in list environments

@marotpam

Hi @lussn! Thanks for contributing 😄 Unfortunately this fix is already fixed here, sorry for solving more things than expected in that PR 😢 But feel free to go through the list of issues if you want to make stripe_planner great again!


MVC companion to aspnet/Routing#238

This is a companion change to make the RouteValueDictionary type more
promient. Using the concrete type here allows to avoid allocations in a
lot of common scenarios.

See https://github.com/aspnet/Routing/pull/238/ for details

:shipit:


USB bootable creation

Summary of the new feature/enhancement

add two new abilities to Windows to allow users to easily create bootable USB media
Proposed technical implementation details (optional)

context menu on .iso image files to "create USB media"
context menu on USB media to "Write image to drive"

Both options would open the same program window, with 1) above pre-populating the selection field with the iso image; 2) above requiring manual selection of the image to write to the drive.
Options could include "make bootable", "(u)EFI or BIOS", "MBR or GPT" (if the drive is applicable) and possibly a "Create DOS bootable" to bring back this functionality - this is often needed for server admins (and even PC enthusiasts) to create BIOS & Firmware update media, and would be very helpful. This would use MS-DOS 6.22, or users could specify any other DOS which they have legally obtained.  Additionally, a "Create Linux bootable" would also be useful, and could even tie-in with WSL, and could be used to create an Ubuntu LTS network install media (which would download and write Ubuntu's mini.iso image to the disc in a one-click operation)  For that matter, adding in the ability to download/write Hyper-V Server 2016/19 could be useful as well.

Is the goal here to get pre-setup VMs?  Or create bootable media?  Rufus to me always seemed to be the gold standard.
for easy VM access, have you seen the Quick Create in Windows 10?

The goal is to have the same resources as "Burn image to disc" but for USB drives, such as to create the full file system needed for USB drives to boot.
If I have an .iso image, say Windows 10 installer or Ubuntu installer, I right click on that image file, and choose "create USB bootable"
Yes, it would perform similar function to rufus.  It would also be "built-in" to Windows, with all of the benefits of being integrated into the UI.
At one time, "Burn image to disc" was seen as doing the same thing as a plethora of other programs as well.  For simple burning .iso images to CD/DVD/BD, using the Windows built-in tool is more than adequate and much more efficient for the majority of use cases outside of disc authoring.
I'd just like to see something as simple, with the same level of verification given to the disc burning software.
I'm not sure if I can be any more clear than that without specific questions - but I can try, if there is still uncertainty!   Thank you!  👍

Reminds me of this: https://www.youtube.com/watch?v=BrcBKpE9aQg Booting Windows from an SD CARD

Booting an OS and booting an OS installer from removable media are two different things.  Windows ToS does not allow standard Windows 10 installs to be bootable from removable media, must use the feature built into Pro and Enterprise.  I'm not asking to bypass that.  I'm simply asking for an "official" method of writing any installer image to removable media without requiring third party tools. Since PowerToys is under the Microsoft Github account, it's more official and less thirdparty than anything else.

@AeSix  Check this out!!! https://github.com/ventoy/Ventoy


Localize an optimistic error that was only in English

@nkuoch please review
HOLD ON COPY CONFIRMATION: https://github.com/Expensify/Expensify/issues/218743#issuecomment-1221652943
Details
Fixed the local failure data error message to be localized instead of just in English.
Fixed Issues
None, just a comment from @iwiznia here: https://github.com/Expensify/App/pull/10269#discussion_r949330820
Tests

Added in a throw in the BeginSignIn API command
When I attempted to log in, made sure the error message showed correctly (see screenshot below)

QA Steps

None

Screenshots

@nkuoch updated spanish and removed HOLD

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by @nkuoch in version: 1.1.93-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


Invite all members of a dojo to a dojo event

One of the features of the previous system was the ability to invite ninjas to an event..
Its something quite a few parents relied on (I get emails since the switch to zen asking when we are restarting, or why they didn't get an event).
It is possible to invite a single member to an event, but the ability to invite all members of a dojo to an event is not present. (Leaving the dojo should stop mails).
Can this feature be added?

You can invite all members of a Dojo to an event as a ticketing admin, did you want the ability to do this a a Dojo member?

When was this added? I guess I need to see release notes, didn't know that
it had become available (explains why it's off the backlog)
On 7 Oct 2015 14:23, "Ursula Clarke Everett"<EMAIL_ADDRESS>wrote:

You can invite all members of a Dojo to an event as a ticketing admin, did
you want the ability to do this a a Dojo member?
—
Reply to this email directly or view it on GitHub
https://github.com/CoderDojo/community-platform/issues/721#issuecomment-146195219
.

It was added as part of the new events system, requested here https://github.com/CoderDojo/community-platform/issues/663
Here is the pull request for that feature https://github.com/CoderDojo/cp-zen-platform/pull/691 which released two days ago.

Hi @willknott - can you take a look at this issue to remove current event invite feature and replace with an invite all instead? Want to know Champions thoughts on this in terms of usability - https://github.com/CoderDojo/community-platform/issues/764

Thank the God of your choosing that the feature is added.
Instead of getting multiple mails a day about when the next session is, the
parents / ninjas are signing themselves up.
One  catch is, if you set up multiple events (and invitations), they sign
up for one, and show for another. (work around... Only one invite at a
time)
On 20 Oct 2015 12:10, "rosalanghammer"<EMAIL_ADDRESS>wrote:

Hi @willknott https://github.com/willknott - can you take a look at
this issue to remove current event invite feature and replace with an
invite all instead? Want to know Champions thoughts on this in terms of
usability - #764
https://github.com/CoderDojo/community-platform/issues/764
—
Reply to this email directly or view it on GitHub
https://github.com/CoderDojo/community-platform/issues/721#issuecomment-149529917
.

Hi @willknott can you add comments to #764 with regards to what the best invite would be :)

Closing this - we can look at implementing https://github.com/CoderDojo/community-platform/issues/764 if we want to change the invite functionality, further discussion can go there.


Add .NET 4.6 Support

I am using .NET 4.6, and I would like to use this library to check for and install it. I am a complete C/C++ dummy, so I can't do much there. I did try a few weeks ago to get it working, but didn't really get anywhere.

See https://github.com/ProjectHuman/NsisDotNetChecker/pull/20

That's exactly what I did. Strange that it didn't work at that time, but it works now? Might have been a Windows 8.1 issue. :confounded:


Added EH,GG,YT,IM,JE countries dial code

Hi there,
I fork this repository again, and create new pull request.

Can you tell me why you want to include these areas? Do we even have flags for them?
Apparently Western Sahara is a disputed territory, so I don't want to include that. Guernsey, Isle of Man and Jersey are all subsideries of England, and have the same intl dial code, so I don't see the point in adding them. Remember this is all extra weight in the plugin and it's really important to keep this as small as possible.

Can you tell me why you want to include these areas? Do we even have flags for them?

Yes, we have.
According to the [ISO 3166-2|https://en.wikipedia.org/wiki/ISO_3166-2], they are officially assigned their own country codes.

Ok I have added a section to data.js detailing the list of regions that I have omitted and a reason why.
If you would like to challenge any of these reasons, please state clearly why you think it is worth adding to the plugin, and give links to examples of other country lists which also include that region.

You know what, I've changed my mind. If people from these regions identify that way, then we should support that.

Oh it seems that you have deleted your repository now, so I can't just re-open this pull request - I'm sorry about that. Is it easy to get it back?

New PR coming, https://github.com/Bluefieldscom/intl-tel-input/pull/295


Installation problem

Getting the following error when run luarocks install fbpython
error: ‘class thpp::Storage’ has no member named ‘getIOBuf’

I get the same error:
Scanning dependencies of target storage_test
[ 88%] Building CXX object test/CMakeFiles/storage_test.dir/StorageTest.cpp.o
/tmp/luarocks_fbpython-0.1-2-2477/fblualib/thpp/thpp/test/StorageTest.cpp: In member function ‘virtual void thpp::test::Storage_CustomAllocator_Test::TestBody()’:
/tmp/luarocks_fbpython-0.1-2-2477/fblualib/thpp/thpp/test/StorageTest.cpp:64:24: error: ‘class thpp::Storage<float>’ has no member named ‘getIOBuf’
     auto buf = storage.getIOBuf();
                        ^
/tmp/luarocks_fbpython-0.1-2-2477/fblualib/thpp/thpp/test/StorageTest.cpp:78:24: error: ‘class thpp::Storage<float>’ has no member named ‘getIOBuf’
     auto buf = storage.getIOBuf();
                        ^
make[2]: *** [test/CMakeFiles/storage_test.dir/StorageTest.cpp.o] Error 1
make[1]: *** [test/CMakeFiles/storage_test.dir/all] Error 2
make: *** [all] Error 2
-- The C compiler identification is GNU 4.8.4
-- The CXX compiler identification is GNU 4.8.4
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Found Glog: /usr/include
-- Found Torch7 in /home/alexander/torch/install
-- REQUIRED_ARGS (missing:  THPP_INCLUDE_DIR THPP_LIBRARIES)
-- Boost version: 1.54.0
-- Found the following Boost libraries:
--   thread
-- Found PythonInterp: /usr/bin/python2.7 (found suitable version "2.7.6", minimum required is "2.7")
-- Found PythonLibs: /usr/lib/x86_64-linux-gnu/libpython2.7.so (found suitable version "2.7.6", minimum required is "2.7.6")
-- Found PythonInterp: /usr/bin/python2.7 (found version "2.7.6")
-- Found NumPy: version "1.11.1" /usr/local/lib/python2.7/dist-packages/numpy/core/include
CMake Error: The following variables are used in this project, but they are set to NOTFOUND.
Please set them or make sure they are set and tested correctly in the CMake files:
THPP_INCLUDE_DIR
   used as include directory in directory /tmp/luarocks_fbpython-0.1-2-2477/fblualib/fblualib/python
THPP_LIBRARY
    linked by target "lua_module" in directory /tmp/luarocks_fbpython-0.1-2-2477/fblualib/fblualib/python

@BAILOOL I was able to install this package by compiling THPP separately on the d358a52 commit (sept 27/16).

@outgrabe thank you. I tried the way you suggested but I get the following error now:

@outgrabe You made my day! How the heck did you find out that? :)

@BAILOOL at a quick look, that looks like a linker error with gtest. Maybe you don't have the right file?
@desertnaut Glad it worked for someone else. THPP was the breaking point of the installation, so I wanted to install it separately to figure out the problem. It had a commit a few days before the issues started popping up (and none of the other repos did), so I rolled back to before it. :)

@BAILOOL I tried your suggestion and I was able to install THPP with success but when I tried to install fbpython with luarock install fbpython I got the same error. Did I miss something ?

@outgrabe Thank you so much. After a lot of effort, finally your comment solve my problem.

@outgrabe's solution worked for me.

@outgrabe what do you mean by :"compiling THPP separately on the d358a52 commit "
Can you explain how I can do that? what is THPP?

@monabiyan Get THPP at https://github.com/facebook/thpp, then
git checkout d358a52
and compile that (follow the instructions, I don't remember how exactly). Then hopefully this project will also compile.


decoder registered with ReplaceTypeDecoder not invoked

Hi, first of all, httpin is great! Thanks for your hard work!
Im trying to use ReplaceTypeDecoder to override the time.Time decoding like such (v0.10.1):
// register custom decoder for time.Time
func init() {
	httpin.ReplaceTypeDecoder(reflect.TypeOf(time.Now()), httpin.ValueTypeDecoderFunc(decodeDartTimestamp))
}

// custom time.Time decoder
func decodeDartTimestamp(value string) (interface{}, error) {
	return time.Parse(time.RFC3339, value)
}

// example of query input
type UpcomingReturnsRequest struct {
	After      time.Time `in:"query=after"`
}

// example of body input
type PrioritizedInput struct {
	httpin.JSONBody
	PrioritizedAt    time.Time `json:"prioritized_at"`
}

Unfortunately, my custom decode never gets executed (verified with log as well as breakpoint. What am I doing wrong?

Hi @rlimberger, thanks for the report. I have tried your code on the Go playgroud.
For the query input example, the custom decoder did get executed, you can check it here: https://go.dev/play/p/eDUe--DgFpu
For the body input example, since httpin uses the Go builtin json package encoding/json by default to decode the whole request body directly, the custom time decoder won't be used.

Closed since this works as intended.


Integrate wfm functionability into wfi

General improvements to lower resolution images, 720p now has rather reliable but can still make a mistake here and there.

Better user feedback on what is happening whens something goes wrong.

Better settings screen, allow the user to toggle more features previously only exclusive to the settings.json

Snap it will now list ducat to plat efficiency

some optimization (Reliably under 200mb of ram now! almost halved!)

a credit page for the developers

better performance for auto with warframe not running.
And for the big changes:

Warframe market web socket integration. Online/in game/offline status automatically handled

Having wfi open will set you to online in wfm.
Having warframe open along side will set you ingame (note, you'll have to press the activation key if you run warframe after wfi)
Having not clicked/typed anything for 15 minutes will set you to offline.

List it, a new and faster way of listing items on warframe.market.
When you run a mission wfi will store the results and the selected reward, till you activate it. Activate it either by having auto detected an end of mission or manually press the button on the main window. A window will appear with the rewards that appeared on screen allowing you to select something else if the auto detection failed below the top 5 listings for the item will show allowing you to make an up-to date listing where you will neither get underpaid or never have it sell at all!

Search it, A way to quickly create a listing out of thin air
use the new hotkey tilde, `, by default. Or by the search icon in the main window.
Running search it will spawn a single textbox in the center of your screen to allow you to type in a prime item (note: Only prime items are supported as of now! This is currently not subject to change) it will then spawn a List it screen with the prime item

Pls resolve merge conflict.

Why not in github? I couldn't get that last bit to properly change over without using github >.>


ability to set displayPriority on Annotations

You've done an incredible work here, thanks for making the component available to everyone!
I was wondering if the displayPriority property could be added to Annotations?

Disregard - I need something like zIndex and displayPriority is not that

Thanks for the suggestion! It looks like the annotations are ordered by MapKit JS in the way they are added to the map. It is true that it’s difficult in some cases to set this precisely using mapkit-react (for instance if you want to place a new annotation between two that were already displayed beforehand).

Thanks! Yes, I've noticed that. Additionally adding more annotations on the map seems to be a lot slower than just adding an initial set of annotations. It is especially noticeable on mobile. Is there anything I could potentially do to get around that?


Bot: expecting 'pokestop' in edits file but getting 'stop'

I submit the creation POI types as 'pokestop' and 'gym', but in the edits json it is 'stop' and 'gym'. The Bot currently expects both to be 'pokestop' and 'gym' and thus states 'JSON parsing failed' when receiving an edits file.
This should probably be adapted in the script, to be more consistent.

done in 7392a36


request: Add distance block for checking if sprites are near each other

I spent a few minutes looking and surprisingly could not find any block that provide the distance formula / Pythagorean theorem to check how far away sprites are from each other easily, analogous to how scratch provides distanceTo.
I would expect this to be included on a platform designed to teaching children programming through 2d games. Sure, I could dedicate an entire lesson to teaching how the pythagorean works, or manually check the differences betwen player.x - other.x < threshold and .... But for some children that is just too complicated to teach at the moment or too many steps to keep in their head at once, and serves as a conceptual roadblock in lessons, when they otherwise would be able to follow the rest of the gameplay logic just fine.
I would rather not define my own function for this. It is reasonable for it to be builtin.

@spaceunifyfifty
You can use the extension https://github.com/jwunderl/arcade-sprite-util for now.


[Packaging] Reduce size of final app package

don't dump everything in.

Ref #1296!

Sorry, nevermind. This is app packaging. :P

I plan to look into this in the next few days. @bbowman let me know if you've done anything toward this yet.


NYSE Calendar differences prior to 1990

I see that the earliest starting date for the NYSE (XNYS) calendar is 1990-01-01 and from that date onwards the trading_calendars and pandas_market_calendars implementation are the same. But prior to that date there are many changes in the holiday calendar that pcal has that trading_calendars does not. If you are OK, I would suggest bringing over the pre-1990 holidays.
These holidays were added by @pldrouin in https://github.com/rsheftel/pandas_market_calendars/pull/30. He was nice enough to upload a csv of all the holidays in the /tests/data/nyse_all_full_day_holidays_since_1928.csv and the results can be verified here: http://s3.amazonaws.com/armstrongeconomics-wp/2013/07/NYSE-Closings.pdf
Example:
trading_calendars:
from .us_holidays import (
    USNewYearsDay,
    USMartinLutherKingJrAfter1998,
    USMemorialDay,
    USIndependenceDay,
    Christmas,
    MonTuesThursBeforeIndependenceDay,
    FridayAfterIndependenceDayExcept2013and2019,
    USBlackFridayBefore1993,
    USBlackFridayInOrAfter1993,
    September11Closings,
    HurricaneSandyClosings,
    HurricaneGloriaClosing,
    NewYorkBlackout,
    PresidentialElectionDays,
    USNationalDaysofMourning,
    ChristmasEveBefore1993,
    ChristmasEveInOrAfter1993,
)

pandas_market_calendars
def regular_holidays(self):
       return AbstractHolidayCalendar(rules=[
           USNewYearsDay,
           USMartinLutherKingJrAfter1998,
           USLincolnsBirthDayBefore1954,
           USWashingtonsBirthDayBefore1964,
           USWashingtonsBirthDay1964to1970,
           USPresidentsDay,
           GoodFriday,
           USMemorialDayBefore1964,
           USMemorialDay1964to1969,
           USMemorialDay,
           USIndependenceDayBefore1954,
           USIndependenceDay,
           USLaborDay,
           USThanksgivingDayBefore1939,
           USThanksgivingDay1939to1941,
           USThanksgivingDay,
           USElectionDay1848to1967,
           USElectionDay1968to1980,
           USVeteransDay1934to1953,
           USColumbusDayBefore1954,
           ChristmasBefore1954,
           Christmas,
       ])

@jmccorriston - this makes sense to me. Thoughts?

Yeah, I have not done the Research pre-1990, and we don't depend on that for Q, so let's pull it in!


[dart fix] --code enable unconfigured lints

The current dart fix --code support works by filtering diagnostics which means that it can only be used to apply fixes for lints that are enabled via analysis options.   In some cases it might be useful to allow fix to enable lints that are not explicitly enumerated in options (see e.g., https://github.com/dart-lang/sdk/issues/47219#issuecomment-1216293923).
/fyi @mit-mit @bwilkerson @davidmorgan @jacob314

(Follow-on issue from https://github.com/dart-lang/sdk/issues/47219 which tracks the main dart --apply --code feature.)

Another option that was just suggested on a chat was for a dart fix --add --code mode that would both update the analysis options file to add the lint and fix the violations.


Comms n widgets

Rough implementation of comms and widgets support for the context.
Includes a test notebook with a number of test cases for expected behavior.
Example implementation of a widget manager is https://github.com/blois/portable-widget-manager

Ah, missed the test.ipynb notebook on my first pass. The waiting on next output test is currently failing. See screenshot attached.

I did my best to dig in, and it looks like the output area code here uses the createRenderedMimetype, which returns the output without waiting on the results for the output.renderModel. I'm not positive on the call flow to get to this function (having a bit of trouble getting console logs out of it), but this seems to be the source of this bug.
Is this the right function to be looking at? If not, where does the renderModel function end up getting called? I can dig in from there!

There's also a minor bug in the validate function in the test that ensures the context object includes the correct data. Namely, it's not checking that the expected keys are actually present in the real value (it just checks that the present keys are In the expected). You can see the fix in the commit here: https://github.com/jupyterlab/richoutput-js/pull/7/commits/47ed40c9f96e02359da8a5fc0cd189070de5d2e3#diff-470867b8d89ae95d93ab6a54fae70e6ac7c9eba172b54e9a187d3a3365cec89aR221


Modularization code and add unit test to Quiz Game

Type of change

[ ] Added new project
[ ] Bug fix
[x] New features
[x] Enhanced documentation

Changes proposed in this pull request

Project name : Quiz Game

Short description :  Modularization code. I created 2 modules gamesystem.py and gameplay.py so that we can maintenance efficiency. In addition, I created the unit test to make test easy.  There were no docstrings, so added docstrings as well.

Checklist

[x] The code is Python 3 compatible
[x] I have added relevant comments in my file to make code understandable
[x] I have added my project in a folder which includes all the necessary files

I changed the project directory name to import module. Is that okay?


feat: introduce StatefulSet analyser.

Closes #
📑 Description
This PR adds a StatefulSet analyser which only checks if the service which is specified in ServiceName, exists.
✅ Checks

[ ] My pull request adheres to the code style of this project
[ ] My code requires changes to the documentation
[ ] I have updated the documentation as required
[ ] All the tests have passed

ℹ Additional Information

Great job! I suggest simply adding a test for a scenario that goes well. However, this point is not blocking for the Pull Request merge.

hey @matthisholleville, I added the "happy path" test.


Add visualization

Add a visualization tool to get an overview of the GBFS feed

I changed the homepage to have 2 different pages, validation /validator and visualization /visualization.

:information_source: https://gbfs-validator.netlify.app/?url=XXX will be correctly redirected to /validator

Features

Summary of stations / vehicles counts
Show a dot for each freefloating vehicles

color per form factor
popup to get all details (vehicule_type_id / plan_id are expended)

Show a dot for each stations

color according to the number of vehicles inside
popup to get all details (merge of station_informations and station_logs)

Show polygons of geofencing zones
popup with details

Missing

Better colors for geofencing zones according to rules
Pricing tables / Simulator

I created a new function to only return content's feed (faster than also validate).
I tested with multiple feeds / versions (not 3.0) but some issues can be still there.
Invalid feed can show nothing or break the page.
:warning: I used my own Mapbox key (https://github.com/MobilityData/gbfs-validator/pull/83/files#diff-c5580fd8f5f02724fac95dbe8fa01c1ecd4efee526378be9f0b96b1d84704c99R40)
MobilityData can create an account a use their own key (Should be totally free for the number of visitors, but a credit card may be required).
We can switch to another tiles' server.

Hi @davidgamez, on my last commit I added a new function that exposes the openapi schema of the api.
It can be imported into Postman to get the available endpoints with parameters.

Hi @davidgamez, on my last commit I added a new function that exposes the openapi schema of the api. It can be imported into Postman to get the available endpoints with parameters.

Thanks!

I rebased, if everybody is ok, can we merge ?

I rebased, if everybody is ok, can we merge ?

Merging now!


CMake support for elf2cfetbl usage

The current CMake support structure does not include support
for the old style CFE Tables -- that is, where the table is
embodied in a source file within the application, which is
compiled with the application headers (as well as headers
from CFE, OSAL, and so on), and which is then converted from
the ELF Object File format into a TBL file.
Asking each project that wants to convert to CMake to independently
work out how to get the files compiled then converted is asking for
a significant duplication of effort -- as this turns out to be
something more than a couple easy lines of Make scripting. It is more
reasonable for us to work it out once, and provide a solution that
projects can use (or adapt) as desired.

Imported from trac issue 48.  Created by glimes on 2015-05-18T16:48:09, last modified: 2019-03-05T14:57:55

Trac comment by jphickey on 2015-05-19 15:24:01:
I have pushed commit [changeset:42d808f] with my proposed solution to this
This integrates "elf2cfetbl" into the CMake rules for an app.

A new CMake function called {{{add_cfe_tables()}}} to define table files associated with an app.  This can be included in the app-specific CMakeLists.txt file that is distributed with the application code.
The function will first search in "tables" subdirectories of the the mission definitions and at the top level for an identically-named file, and use that as an override if one exists.
If those do not exist, then the (default) table source included with the app is used.
The table source file is compiled using the target compiler to produce an intermediate "o" file, using the same set of CFLAGS and include paths as the rest of the code for the app.
"elf2cfetbl" is called to convert the "o" file into a tbl file.
The tbl file is copied to the target tree during install

Trac comment by sstrege on 2015-07-24 17:18:29:
Approve

Trac comment by glimes on 2015-07-27 12:55:48:
Approve.

Trac comment by sduran on 2015-07-27 17:51:04:
approve.

Trac comment by glimes on 2015-09-08 12:56:00:
... somehow was not closed after the 07-28 integration was complete.


Updating code on node.js server causes VSC to complain that this is not latest version - no changes made server-side or elswhere

Does this occur consistently?
Repro steps:

Action: writeFile
Error type: Error
Error Message: ETag does not represent the latest state of the file "hostingstart.html". Download the file from Azure to get the latest version.
Version: 0.19.0
OS: win32
OS Release: 10.0.17763
Product: Visual Studio Code
Product Version: 1.50.0
Language: en

Call Stack
AppServiceFileSystem.<anonymous> extension.bundle.js:309:491914
rejected extension.bundle.js:309:490241
processTicksAndRejections task_queues.js:94:5

Have you tried redownloading the file and trying to make your edits again?

Hello Nathan,
Thank you for your reply. My expectation was once I was connected to Azure
and able to read code from the default Azure App Server instance, I would
then be able to update the code and submit it back to the app server. I did
try redownloading, restarting VSC and even rebooting my workstation. Seeing
that none of this worked, I figured maybe the only way to update was to
establish a CI/CD pipeline. I'm not too sure how far I am on or off the
ranch.
I'm open to screen share to demonstrate if this helps you.
Thanks again,
Matthew
On Tue, Oct 13, 2020 at 1:35 PM Nathan<EMAIL_ADDRESS>wrote:

Have you tried redownloading the file and trying to make your edits again?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode-azureappservice/issues/1803#issuecomment-707993899,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARFASHCN5INOOJKWCXRTRIDSKS22XANCNFSM4SOFE23Q
.

This is definitely expected to work especially if you're just editing the default hostingstart.html file.  A successful attempt will look like this:

Is there any possibility that you can use a different web app on a different subscription?  I'm wondering if that Microsoft Partner Network has some properties that are blocking the update.


Allow overriding the  scheduler_host_manager

Deployments with Ironic need a custom host manager. Allow overriding it.
Don't add PciPassthroughFilter explicitly to the active filters.
Some releases of Nova have a bug using this filter with the
ironic scheduler_host_manager.
nova.scheduler.filters.all_filters should include this if it's in the
default filters (as it is in metadata/service/control/cluster.yml),
and if it is removed from scheduler_default_filters in
the pillar, the new scheduler_custom_filters will allow users to
manually add it.

Any update? @pupapaik please update review.


"Failed to minify code" (create-react-app)

I'm trying to use this library together with resin.io's electron + react template, which uses create-react-app.
When I run the build script it fails telling me that this library (specifically dist/Reader.js) can't be minified and directs me to this link, which suggests filing an issue on the dependency in question, so here I am.
Native node modules + create-react-app + electron is asking for trouble, I know, but maybe you have some suggestions?

Hi @haakonmt,
thank you for posting your issue.
I personally use nfc-pcsc inside an Electron app which is built using Webpack and it works great, but it requires some config (in some cases):

if you use Webpack (create-react-app uses it), you need to add nfc-pcsc to externals section in webpack.config.js (it is because it cannot be loaded and built with Webpack as it loads Node native modules)
{
  // ... your Webpack object
  externals: ['nfc-pcsc']
}

be aware of Node Native modules usage in Electron, see FAQ Can I use this library in my Electron app? in README (it might work out of box, but in most cases you need to rebuild modules)

I hope you will be able to make it work with these info. 🙂
However, if you get stuck, you can create repo with your code (it can be just a minimal example), where the issue occurs and I'll take a look at it (clone, install, run).
Hope it helps.

Thank you for the fast response!
Adding the library to webpack's externals seems like a perfectly good solution, but I forgot to add that I would try to avoid ejecting if possible.
Nevertheless, I found a solution! By using window.require to import the nfc module (const { NFC } = window.require('nfc-pcsc');), I was able to make webpack ignore it altogether, while still making it work in a node environment (in my case electron).

@haakonmt How did you managed to get this running with React without eject ?
Where did you required the files ?

@Ionut-Milas Wherever you want! For instance in src/nfc.js:
const { NFC } = window.require('nfc-pcsc');

const nfc = new NFC();

nfc.on('reader', reader => {
});

Then at the top of src/index.js: import './nfc';

Thanks a lot !


[core] Migrate to from utility-hooks to react-use

See https://github.com/streamich/react-use

@umidbekkarimov utility-hooks are not fully compatible with react-use (.e.g usePureMemo). Should we create usePureMemo for locally?

@doniyor2109 I will take care of this task, work on your roadmap items :) or migrate your tests to cypress :)


[Publishing][Python] Blog for Kafka to BigQuery Template

Write a blog for Kafka to BigQuery dataproc template.

Blog is posted: https://medium.com/google-cloud/dataproc-serverless-template-to-migrate-your-data-from-kafka-to-bigquery-9b635a66b9dc

Blog is posted: https://medium.com/google-cloud/dataproc-serverless-template-to-migrate-your-data-from-kafka-to-bigquery-9b635a66b9dc


Ignore agent state "running" for scheduling

The agent state "running" is not a reliable indicator.  Sometimes,
especially when an assignment was done very fast, the agent may fail to
reset its status to online again.

Coverage decreased (-0.59%) to 65.05% when pulling 941893d7a6b36e94824e7472744e62a84fbcbdec on guidow:ignore_running_status into b186a96ecc8da74e872a7a02667f82cbc12530a9 on pyfarm:master.

Coverage decreased (-0.59%) to 65.05% when pulling 941893d7a6b36e94824e7472744e62a84fbcbdec on guidow:ignore_running_status into b186a96ecc8da74e872a7a02667f82cbc12530a9 on pyfarm:master.

Coverage decreased (-0.59%) to 65.05% when pulling 941893d7a6b36e94824e7472744e62a84fbcbdec on guidow:ignore_running_status into b186a96ecc8da74e872a7a02667f82cbc12530a9 on pyfarm:master.

Looks good, tests are failing due to the sqlalchemy problem.  Merging since this is a bug fix.


[Bug Report]: FileLoadException on ConsumerWorker execution is silently ignored

Prerequisites

[X] I have searched issues to ensure it has not already been reported

Description
If the ConsumerWorker background task fails to start, it is silently ignored without any exception or logs to the KafkaFlow users.
In my particular case, the background task fails with TypeLoadException due to a missing binding redirect, leaving the background task in a faulted state - it will appear to the users as if the application is not consuming, and no middlewares will be executed.
I ended up resolving the root cause by adding a binding redirect, but I think that KafkaFlow should let the user know something has failed in order to assist troubleshooting.
Steps to reproduce

Create a ASP.NET Framework web application (in my case, using .NET Framework 4.6.2)
Add and configure KafkaFlow packages (I'm using version 3.0.10). Add any middlewares you want.
Add Microsoft.Bcl.AsyncInterfaces version 8.0.0 to your web application.
Ensure no binding redirect is added to app.config/web.config.
Debug the application until reaching the ConsumerWorker.StartAsync(...).

KafkaFlow is expecting Microsoft.Bcl.AsyncInterfaces version 6.0.0, and as such will throw a TypeLoadException due to failed assembly binding.
Expected behavior
I expected that KafkaFlow would report an error through an exception or through an error log.
Actual behavior
Silently fails the background task on ConsumerWorker. After inspection on debug, the following exception is observed:
Could not load file or assembly 'Microsoft.Bcl.AsyncInterfaces, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51' or one of its dependencies. The located assembly's manifest definition does not match the assembly reference. (Exception from HRESULT: 0x80131040)
KafkaFlow version
3.0.10

Adding screenshot.

Let me also suggest resolving this with a task continuation that verifies if this task has failed, and takes proper action after (e.g. emit an error log and setting the consumer state appropriately).


Need a link to instructions for resetting just 1 connection

I can't find instructions anywhere for how to reset one of our multiple connections (currently has status = unknown).  I can only find how to reset the whole gateway, which I don't want to do while any traffic is being passed.

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

ID: 1fe1809b-6ed1-ff3e-bd4f-ee80095a8efc
Version Independent ID: d03a9904-4fca-0df6-1a0b-11b6ebeb1784
Content: Add multiple VPN gateway Site-to-Site connections to a VNet: Azure Portal: Resource Manager
Content Source: articles/vpn-gateway/vpn-gateway-howto-multi-site-to-site-resource-manager-portal.md
Service: vpn-gateway
GitHub Login: @cherylmc
Microsoft Alias: cherylmc

@KarlaK-fl, Thanks for your feedback. We are looking into this query and will update you as soon as possible.

@KarlaK-fl , Appreciate your patience. As far as i know, there is only one possibility and that was resetting the whole gateway where all connection will be rebooted. Generally it would take a minute to get rebooted and sometimes up to 30 to 45 minutes depending on the gateway instances. How ever let me confirm with the team regarding this issue and i will get back to you.

@KarlaK-fl , I have confirmed with the team and unfortunately  there is no possibility of resetting a single connection in the gateway. Instead of that you can disconnect the connection and try to reconnect it which is better than resetting the whole gateway.
We are closing this issue for now. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.

How do I disconnect and reconnect the connection without deleting it completely?
Thanks,
Karla Keeney
Director of Technology
Resource Consulting Group
<EMAIL_ADDRESS>407-422-0252

From: suvasara<EMAIL_ADDRESS>Sent: Friday, October 4, 2019 5:54:33 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Karla Keeney<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Need a link to instructions for resetting just 1 connection (#39750)
@KarlaK-flhttps://github.com/KarlaK-fl , I have confirmed with the team and unfortunately there is no possibility of resetting a single connection in the gateway. Instead of that you can disconnect the connection and try to reconnect it which is better than resetting the whole gateway.
We are closing this issue for now. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/39750?email_source=notifications&email_token=AGWCFTHWJUTO5OCEILTFPNLQM63RTA5CNFSM4I3T7EDKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAM7MUY#issuecomment-538572371, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AGWCFTCN36P6MKVRVJ3WEI3QM63RTANCNFSM4I3T7EDA.


Roll Engine from 829d03a03466 to dfb68886ff45 (1 revision)

https://github.com/flutter/engine/compare/829d03a03466...dfb68886ff45
2021-04-06<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from txu1APnex... to dl1JUKVuj... (flutter/engine#25448)
Also rolling transitive DEPS:
fuchsia/sdk/core/linux-amd64 from txu1APnexLlQ to dl1JUKVujkyS
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md

Commit queue failed; closing this roll.


SWAP - this pseudo can be done through 3 XORs

A SWAP can be done through 3 XORs without allocating a temporary register or using stack:
X := X XOR Y
Y := Y XOR X
X := X XOR Y

What's the benefit of this over just doing PSH A;MOV B, A;POP B?

no stack operation. For a hardware viewpoint, no memory access and so no latency.

But isn't this a software project? If this gets implemented in hardware, this might be worth it, but I don't really see the use of implementing this in software if it would only benefit (potential) hardware. This is just my point of view though.

First, I inform there is a smarter way used by all CPUs having no SWAP instruction.
Second, while I'm no saying you should do, it is a very easy change with no backdraw on compatibility in machine code (the old one will still work)
So I don't understand why you are arguing here

I've seen the xor swap trick before, and I think it's pretty cool. I probably would have written it this way too actually, but I didn't have an xor operation then. I'm happy to change it.
@redfast00 while it is indeed primarily a software project, I've started looking into implementing 16bitjs in verilog so it can be programmed onto an fpga (inspired by some discussions with @hlide). I think we probably need to make a couple of changes (and likely compromises) to make that happen, but I think it would be awesome to do.

That seems really cool! If you do it, will you write a blog post about it? I wonder if you will convert the JS code to Verilog or if you will rewrite the whole thing.

I think it's safe to say it will be a rewrite - it's such a different kind of programming that the only thing I can really port over is the instruction set (which is fine). If it works then yes for sure I will be writing a post about it. I'm working on another at the moment which is about writing a programming language from scratch (the Lel project on my github) so that's a little ahead in my priority queue, but I'm itching to get started now!

Or you could first design your cpu with logisim; http://www.cburch.com/logisim/ and https://github.com/lawrancej/logisim. I remember there is a MIPS implementation done with logisim and you can execute step by step and vizualize the result.

@hlide that looks like an amazing program, I'm definitely going to take a look into that.

Implemented in https://github.com/francisrstokes/16bitjs/pull/51


Bundle with ambient dependencies

When running typings bundle -o dist/ --ambient in my project that has cx.google.core as a dependency I get.
Attempted to compile "cx.google.core" as a dependency, but it contains some ambient module declarations ("cx/google/core.d.ts").
The cx.google.core module is installed via npm and it's index.d.ts (defined in package.json) is build with the same command.
Digging around the source for typings/core I found this https://github.com/typings/core/blob/477f029a8686cdf856844dd7d70e315d2ebe1f7a/src/lib/compile.ts#L284. It seems to set ambient to false when resolving dependencies, this is probably by design. But I cant figure out in what what to change the bundling of either of my projects get it to work.
During the build step on cx.google.core I run the bundle and the rename the file to index.d.ts. The file looks like this:
// Generated by typings
// Source: src/cx/google/core.d.ts
declare module 'cx/google/core' {
  class Client {
    ...
  }

  class Auth {
    ...
  }
}

Is there a way to get around this?

It is by design. You don't want to wrap ambient modules within your bundling module. Or else the user will get duplicate typings.
Users need to install other ambient modules themselves

Closing with @unional's answer. I'd be open to careful consideration or perhaps a flag, but it's definitely intentional. Ambient dependencies are "global", you don't want to bundle that up into your definition. The bundle feature should mostly be used for testing and is made to make allow TypeScript to test the definition by making the external modules into ambient declarations.

I will post this here as I think it might be related.
I've rewritten my modules to have there own definitions and don't use typings bundle anymore.
The two ambient modules are cx.google.core and cx.google.drive. Where cx.google.drive depends on cx.google.core.
In cx.google.drive::package.json cx.google.core is declared as a dependency.
When trying to add these in my project I get the same error as before.
# In my project
$ npm install --save cx.google.drive

# works fine
$ typings install npm:cx.google.core --save --ambient

# throws error "Attempted to compile "cx.google.core" as a dependency, but it contains some ambient module declarations ("cx/google/core.d.ts")."
$ typings install npm:cx.google.drive --save --ambient

What's really odd here is that cx/google/core.d.ts is no where defined in the NPM package.
It was defined in a previous version of cx.google.core. And it's defined as ./src/cx/google/core.d.ts in the GitHub repo but is moved and renamed as index.d.ts when publishing to NPM.
So I guess my question is how could one have a ambient module be dependent on, via package.json, another ambient module and still be to install both definitions?

@fredrikbonander Please share some code, it'd really help to debug the issue. However, the error you are talking about no longer exists if you update too. However, it sounds like you may have an issue in the format of the definition.

@blakeembrey I just updated typings to the latest version and now it works as expected. Thanks for your help.


Fix test_make_parent_mount_private

This fixes #471
The checks have just been broken out to their individual pieces rather than testing the entire comparison of the MountArgs so we can test for / or /tmp set as target, the reasoning for this is outlined in the issue.

Codecov Report

Merging #472 (f1c662b) into main (8cc892a) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #472      +/-   ##
==========================================
- Coverage   60.46%   60.46%   -0.01%
==========================================
  Files          82       82
  Lines       12150    12148       -2
==========================================
- Hits         7347     7345       -2
  Misses       4803     4803


[ISSUE] Documentation for creating catalog and adding members

Description
I am trying to create a group with an azure service principal and add that service principal as a member of the group. I had to dig through the documentation to find the iam.ComplexValue() syntax to pass into the members parameter of a.groups.create(). I am able to pass in my own credentials into the members variables and it works as expected but I cannot figure out how to pass in the service principal credentials. I have tried passing in the client_id and the databricks acccount name into service_principal_name but neither add the service principal into this group. Am I missing something? Or is there a better way to do this?
Reproduction
a.groups.create(
display_name = <group_name>,
id=None,
members=[
iam.ComplexValue(display=service_principal_name, primary=None, type=None,value=spID),
iam.ComplexValue(display='', primary=None, type=None, value='<account_ID')
]
)
I am not sure if this is the place to post this but could not find anywhere else.

Thank you for filing this report @cpc-ruffin! We do need to improve the documentation for how to use this API, to be honest. I'll work with the upstream IAM team to see if they can help improve the documentation for this API. We also have https://github.com/databricks/databricks-sdk-py/issues/400 which may help you better navigate the docs and find the data structures you're looking for.

Thank you for filing this report @cpc-ruffin! We do need to improve the documentation for how to use this API, to be honest. I'll work with the upstream IAM team to see if they can help improve the documentation for this API. We also have #400 which may help you better navigate the docs and find the data structures you're looking for.

Thanks for the response @mgyucht. The method above did correctly add the service principal to the group but it did not have the correct permissions I was assigning it. We did some further testing with service principals/groups/permissions (with/without the sdk) and we believe Databricks has an internal issue with service principals and groups. There seems to be no functionality issues with the adding service principals to a group through the sdk (I don't think).
On the other hand, I do agree that the documentation needs to be improved. I shouldn't have to dive into the repo to find the syntax on how to add members into a newly created group. This feature is not the only time I ran into documentation issues with the sdk, but I am glad you guys are working on it. Other than that, the sdk as been super useful and I appreciate the work you guys are doing!
(Also, small UI error also with the below page. You cannot scroll when the navigation goes below your viewable screen. https://databricks-sdk-py.readthedocs.io/en/latest/workspace/index.html )

Thanks for the tip on the docsite navigation. We're tracking this here: https://github.com/databricks/databricks-sdk-py/issues/415, which follows an upstream issue in our Sphinx documentation generator: https://github.com/sphinx-doc/alabaster/issues/212.


upgrade to moab 5.4.0

Description

Type of change

[ ] Bug fix~~
[ ] New feature~~
[ ] Documentation update~~
[ ] Other~~

How Has This Been Tested?

Checklist:

[ ] I have followed the style guidelines of this project.~~
[ ] I have performed a self-review of my own code.~~
[ ] I have commented my code where applicable.~~
[ ] I have updated the release notes.~~
[ ] I have made corresponding changes to the documentation.~~
[ ] I have added debugging support to my changes.~~
[ ] I have added tests that prove my fix is effective or that my feature works.~~
[ ] I have confirmed new and existing unit tests pass locally with my changes.~~
[ ] I have added new baselines for any new tests to the repo.~~
[ ] I have NOT made any changes to protocol or public interfaces in an RC branch.~~
[ ] I have assigned reviewers (see VisIt's PR procedures for more information).~~

@brugger1 what do you think of allowing an update to MOAB version on the RC? I can't recall but if --moab is off by default in build_visit, I think its harmless except for people wanting MOAB plugin.

I believe --moab is part of the optional list. I do build it on some platforms. I disable it on any platforms where the build fails. Is it a change that breaks compatibility? I believe I upgraded to 5.3.1 with the last 3.3.1 release so updating a build_visit library on the RC isn't without precedent. If the current plugin builds and the tests for moab pass then I believe it is fine to update it on the 3.3RC.


Allowing selection text on multiple paragraph with flutter_markdown

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce

Create a MarkdownBody widget
Add a text with multiple paragraph
Put the selectable parameter to true
Try to select all the paragraph at once.

Expected results
All the text would be selectable together.
Actual results
Only one paragraph can be selected.
Code sample
Code sample
class TestMarkdownPage extends StatelessWidget {

  const TestMarkdownPage({super.key});

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        automaticallyImplyLeading: true,
      ),
      body: const Center(
        child: MarkdownBody(
          data: 'En cours de développement.\n\nMerci',
          selectable: false,
        ),
      ),
    );
  }
}

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.13.5, on macOS 14.0 23A344 darwin-x64, locale fr-DE)
    • Flutter version 3.13.5 on channel stable at /Users/fghjkl/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 12fccda598 (9 weeks ago), 2023-09-19 13:56:11 -0700
    • Engine revision bd986c5ed2
    • Dart version 3.1.2
    • DevTools version 2.25.0

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/fghjkl/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.0.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15A507
    • CocoaPods version 1.13.0

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)

[✓] IntelliJ IDEA Community Edition (version 2023.2)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.84.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension can be installed from:
      🔨 https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Connected device (4 available)
    • iPhone (mobile)            • 00008030-000278182285802E            • ios            • iOS 17.0.1 21A340
    • iPhone 15 Pro Max (mobile) • 599195C9-152C-4B26-88CB-234014EC7424 • ios            •
      com.apple.CoreSimulator.SimRuntime.iOS-17-0 (simulator)
    • macOS (desktop)            • macos                                • darwin-x64     • macOS 14.0 23A344 darwin-x64
    • Chrome (web)               • chrome                               • web-javascript • Google Chrome 119.0.6045.159

[✓] Network resources
    • All expected network resources are available.

Closing this as a duplicate of https://github.com/flutter/flutter/issues/99819.
Kindly subscribe to the original issue for updates.
Thank you


Remove useless lines

Removed two lines saying "more to be added" since we are about to finalise the DG.

Codecov Report

Merging #290 (64d0ee8) into master (6847c0a) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #290   +/-   ##
=========================================
  Coverage     73.64%   73.64%
  Complexity      760      760
=========================================
  Files           105      105
  Lines          2436     2436
  Branches        325      325
=========================================
  Hits           1794     1794
  Misses          519      519
  Partials        123      123

Continue to review full report at Codecov.

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


Better host support

With this change you can now set the image to use other docker registries, not just dockerhub.
I.e. now using image: gchr.io/me/addon will work.

@agners @robbiet480 ?

This splits repository/image like so:
repository=gchr.io/me
image=addon

LGTM


Implement Settings view

At this point, it would be nice to expose the Settings tab where we can customize the behavior of the editor environment.
Let's add a new popup with controls that allow to change some of the Editor state / app global state

I added something in #54


Deps for PHP 7.2 unavailable

The package deps-7.2-vc15-x64.7z and deps-7.2-vc15-x86.7z is not available on http://windows.php.net/downloads/php-sdk/.
I need this for build automatically PHP's extensions for Windows with appveyor.

Thanks for the report. You can use the phpsdk_deps tool as in readme, to fetch the dependencies. Perhaps it would also make sense to extend the tool to keep providing the full dependencies package.
Thanks.

@macintoshplus the matter is now cleared out. The 7z packages for 5.6, 7.0 and 7.1 are still provided. For 7.2 please used the new SDK tools to fetch the dependencies.
Thanks.


User session count limiter with dedicated page

Hello I need to implement dedicated empty error page when I will get specific error from keycloak. I have set inside keycloak a dedicated flow with User session count limiter

I would like to have a possibility to show this error page only for this specific error.
Now keycloak context contains pageId: 'error.tfl' with correct summary property in message, but also there is something inside auth:

In current state I would like to have something to distinguish (probably by authenticator inside auth > authenticationSelections > authenticationExecution> authenticator) but this is not inside keycloak context.
This is my current state

Is there a way to achieve this distinguish?

Hello @smietanka,
Well first of, the KcContext type definition is only an insight of what's usually in the kcContext at runtime. It's in no way exhaustive, we only define the properties that are actually used in the default pages.
Now you have to way to go about what you'd like to do.
First is to just don't care about type safety and use (kcContext as any).auth... for your test.
But it you want to go the clean route you'd extend the type definition of the KcContext for the error.ftl page.
src/login/kcContext.ts
import { createGetKcContext } from "keycloakify/login";

export type KcContextExtension =
	| {
		pageId: "error.ftl";
		auth?: {
			authenticationSelections?: {
				authenticationExecution?: {
					authenticator?: string;
				}
			}[]
		}
	};

export const { getKcContext } = createGetKcContext<KcContextExtension>({
	mockData: [
	],
	mockProperties: {
		MY_ENV_VARIABLE: "Mocked value"
	}
});

export const { kcContext } = getKcContext({
	// Uncomment to test the login page for development.
	//mockPageId: "login.ftl",
});

export type KcContext = NonNullable<ReturnType<typeof getKcContext>["kcContext"]>;

With that TypeScript will let you access the properties you need:
src/login/KcApp.tsx
import "./KcApp.css";
import { lazy, Suspense } from "react";
import Fallback, { type PageProps } from "keycloakify/login";
import type { KcContext } from "./kcContext";
import { useI18n } from "./i18n";
import Template from "./Template";

const Error = lazy(() => import("keycloakify/login/pages/Error"));

const classes = {
} satisfies PageProps["classes"];

export default function KcApp(props: { kcContext: KcContext; }) {

    const { kcContext } = props;

    const i18n = useI18n({ kcContext });

    if (i18n === null) {
        return null;
    }

    return (
        <Suspense>
            {(() => {
                switch (kcContext.pageId) {
                    case "error.ftl":
                        return kcContext.auth?.authenticationSelections?.[0]?.authenticationExecution?.authenticator === "user-session-limits" ?

My custom error page
 :
                            <Error {...{ kcContext, i18n, classes }} Template={Template} doUseDefaultCss={true} />;
                    default: return <Fallback {...{ kcContext, i18n, classes }} Template={Template} doUseDefaultCss={true} />;
                }
            })()}
        </Suspense>
    );
}

This is good as it let you easyly create a test case for storybook:
src/login/pages/Error.stories.tsx
import { Meta, StoryObj } from '@storybook/react';
import { createPageStory } from "../createPageStory";

const { PageStory } = createPageStory({
    pageId: "error.ftl"
});

const meta = {
    title: "login/Error",
    component: PageStory,
} satisfies Meta<typeof PageStory>;

export default meta;
type Story = StoryObj<typeof meta>;

export const Default: Story = {
    render: () => <PageStory />,
};

export const ExceededNumberOfConcurrentSessionView: Story = {
    render: () => <PageStory kcContext={{
        auth: {
            authenticationSelections: [
                {
                    authenticationExecution: {
                        authenticator: "user-session-limits"
                    }
                }
            ]
        }
    }} />
};


Fine-tune BERT on CORD-2019 dataset

Fine-tune BERT (or word embeddings?) on CORD-2019 dataset published on Kaggle:
[CORD-19 is a resource of over 29,000 scholarly articles, including over 13,000 with full text, about COVID-19, SARS-CoV-2, and related coronaviruses.]
The dataset has 2GB. I guess the domain is quite different from the FAQ, as the dataset is made up of scientific papers, but could still be valuable to introduce some substantial vocabulary related to the virus.

I will look in to this!

Finetuning bert-base-cased to CORD-19 (30k docs, 6million sentences). Estimating it will take about 8-9 hours if it doesn't crash. You can follow the progress here (https://public-mlflow.deepset.ai/#/experiments/2/runs/f7d06c1f44f04119a63dda93c4ae71fe)
GPUs: 4 x v100
batch_size: 128
epochs: 1
learning rate: 2e-5

Model is published via hugging face's model hub and get be loaded in FARM, haystack or transformers via deepset/covid_bert_base (https://public-mlflow.deepset.ai/#/experiments/55/runs/57074b3298e443b89adc5c0bed8590e4).
The embeddings of this model perform a bit better for question similarity perform than plain BERT, but are (as expected) still a bit worse than sentence-bert. However, it might be useful as a basis for other tasks on CORD-19 data (e.g. extractive QA, NER ...).


Next major release

Change default option value

setup_method: nothing

install support specific version

ignore_note: enable

Change internal function

automatic install stub
need preflight running(generate cache) and run as follow commandmypy --install-types

default on option

Start to work


Fix anchoring of delegate regular expressions

Before, e.g. a lookahead like (?=x|a) would be wrong. The delegate RE
for the x|a part inside was compiled as ^x|a. This means only the
x was anchored, but not a.
Fix it by adding a non-capturing group so that the delegate is
^(?:x|a), which results in correct anchoring in this case (and
possibly others).

Thanks for the fixes, appreciated!


Cant receive forgot password email

Subject: Cant receive forgot password email
Description:
I would like to retrieve my password for the account<EMAIL_ADDRESS>but once I click submit, nothing happen in my mail box even after 30 min.
I tried multiple times...
Thanks
Results:
I would like to retrieve my password for the account<EMAIL_ADDRESS>but once I click submit, nothing happen in my mail box even after 30 min.
I tried multiple times...
Thanks
Backend version: develop-4fbb818d
Frontend version: develop-4fbb818d
User Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) SamsungBrowser/25.0 Chrome/<IP_ADDRESS> Safari/537.36
Page: https://couchers.org/password-reset
User:

#3520


chore: Add a11y checks for app layout toolbar

Description

Related links, issue #, if available: n/a
How has this been tested?

Review checklist
The following items are to be evaluated by the author(s) and the reviewer(s).
Correctness

Changes include appropriate documentation updates.
Changes are backward-compatible if not indicated, see CONTRIBUTING.md.
Changes do not include unsupported browser features, see CONTRIBUTING.md.
Changes were manually tested for accessibility, see accessibility guidelines.

Security

If the code handles URLs: all URLs are validated through the checkSafeUrl function.

Testing

Changes are covered with new/existing unit tests?
Changes are covered with new/existing integration tests?

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Moved into https://github.com/cloudscape-design/components/pull/3063


FAI-4901: Hasura End-to-End Testing

Description
This PR fixes 2 bugs that present in similar ways but due to different codes paths have different solutions
SaaS Bug
In #862, we added a step to batch processing to separate objects into groups by the keys present in the objects.  We then constructed on_conflict.update_columns arrays based on the keys of each group.  This was done to fix a bug that nulled out fields of existing rows that were present in the update_columns array but not present in the object.
The (flawed) procedure for doing this was roughly:

for a batch separate into groups by keys of objects
add FKs to objects from prior upserts
using first object of a group as prototype, construct update_columns for the group.

This procedure was flawed because the 2nd step could alter keys of a subset of group.  Said another way, not all objects in the are guaranteed to have the same set of FKs added.  In the specific case we encountered, the first object used for computing update_columns did not have data for a particular FK.  A subsequent object did and since the update_columns didn't include the FK column, that object's FK was not upsert'ed into the database.
The fix is to reorder steps 1 & 2 so we add all possible data to the objects prior to grouping by keys.
CE Bug
WIP
Type of change

[X] Bug fix
[ ] New feature
[ ] Breaking change

Can you please update the description with what is the actual bug fix.

Can you please update the description with what is the actual bug fix.

Done.  Still working on CE validation but please take a look at the description and tell me if it makes sense.