Add a callback to loggers called after removing a checkpoint

Edit:
Editing this to add slightly more context as this issue went slightly stale and this issue really handicaps the usability of WandbLogger.
Description & Motivation
When using ModelCheckpoint(save_top_k=3) and WandbLogger(log_model="all") you end up with only top 3 models saved locally, but multiple models uploaded to wandb, which feels like a buggish behavior.
This is because currently loggers are notified after a checkpoints is saved, but are NOT notified after a checkpoint is removed. This could be solved, by adding a callback so that a logger knows, when a checkpoint is removed.
Described behavior is obviously undesirable because it creates tons of unnecessary checkpoints stored in a cloud.
Using log_model=True is not really a solution to this problem, because it defers uploading model until the training is complete and works ONLY when the training succeeds, so no checkpoints if your training crashes at any point.
Pitch
Add a callback to loggers that is called after a checkpoint is removed to allow for removal of uploaded artifacts by a logger.
This callback would presumably require a path to deleted checkpoint to be passed as an argument, but it seems like a reasonable aporoach.
Then modify WandbLogger to remove deprecated uploaded artifacts from the could.
Then log_model="all" mode could be replaced with log_model="immediate", and possibly log_model=True with log_model="end", as that would be more appropriate naming, and more in-line with the description in the docs.
Alternatives
WandbLogger could scan for local checkpoints and remove non-existent ones from the cloud without the need to actually add another callback method. This however seems like a slightly hacky solution.
Additional context
I'd be willing to write a PR for this if given an approval that is makes sense.
cc @borda @tchaton @justusschock @awaelchli @morganmcg1 @borisdayma @scottire @parambharat
Example
Given code:
import torch
import lightning
import lightning.pytorch.callbacks
import lightning.pytorch.loggers
import wandb

class MockLightningModule(lightning.LightningModule):
    def __init__(self) -> None:
        super().__init__()
        self.layer = torch.nn.Linear(1, 1)
        self.scores_in_each_epoch = [
            10, 9, 8, 7, 6, 5, 4, 3, 2, 1
        ]

    def training_step(self, batch):
        return self.layer(batch)

    def validation_step(self, batch, batch_index):
        self.log("mock_score", self.scores_in_each_epoch[self.current_epoch])

    def configure_optimizers(self):
        return torch.optim.Adam(self.parameters())

class MockDataModule(lightning.LightningDataModule):
    def train_dataloader(self):
        return torch.utils.data.DataLoader(torch.range(0, 10))

    def val_dataloader(self):
        return torch.utils.data.DataLoader(torch.range(0, 10))

def run_mock_wandb_training(should_crash: bool, log_model):
    dm = MockDataModule()
    model = MockLightningModule()

    logger = lightning.pytorch.loggers.WandbLogger(project='mock_project', log_model=log_model, name=f"{should_crash=}-{log_model=}")
    callback = lightning.pytorch.callbacks.ModelCheckpoint(save_top_k=3, monitor="mock_score", mode="min")

    if should_crash:
        max_epochs = 11
    else:
        max_epochs = 10

    trainer = lightning.Trainer(max_epochs=max_epochs, callbacks=callback, logger=logger)
    try:
        trainer.fit(model, datamodule=dm)
    except:
        # this training is supoorsed to fail, so it's okay
        pass
    logger._experiment.finish()

run_mock_wandb_training(should_crash=False, log_model=True)
run_mock_wandb_training(should_crash=False, log_model="all")
run_mock_wandb_training(should_crash=True, log_model=True)
run_mock_wandb_training(should_crash=True, log_model="all")

You get:
3 checkpoints

10 checkpoints:

0 checkpoints:

10 checkpoints:

I'd be willing to try and add this, provided that someone else approves that this would be a good addition.

Writing here since this got no response yet.
@carmocca (I see that you are set to be autotagged to model checkpoint and logger issues), what is the course of action here?
Should I do something with this issue that I perhaps missed or should this issue simply wait for triage?

@awaelchli what do you think about this case? :rabbit:

Adding the hook makes sense to me. I take it that it would be similar to the existing after_save_checkpoint hook for the loggers. Changing the meaning of log_model is probably not possible due to backward compatibility.

Somewhat yes, but when calling after_save_checkpoint you do not specify directly which checkpoint was saved, (ModelCheckpoint has attribute last_model_path). With after_remove_checkpoint I think that you'd have to specify which checkpoint was removed directly.
Also I see that NeptuneLogger does in fact delete checkpoints that are no longer in top_k and it does that in after_save_checkpoint. It might be possible to add a similar solution to WandbLogger. I am not yet sure though.

@janpawlowskiof Then I think we should definitely investigate that first. Good call. Either way, with or without a new hook, I unfortunately couldn't find an easy way from the wandb api docs to delete a single file (or even check if one exists with a given name). It would require a bit more digging, if anyone wants to help.

I think this would fix https://github.com/Lightning-AI/lightning/issues/17147

Nice. This went super stale, because I was focused on my Master Thesis, which I finished writing literally yesterday (yay!). I will probably submit a draft any day now, but I need to do some more testing.

I just asked the guys at wandb for some input here on how to delete an Artifact that might be uploading without waiting for it to finish uploading. If this is not possible, we'd have to figure out some workaround.
Although even if you tried to wait() for an artifact for example in _scan_and_log_checkpoints and delete it (which you can do), something goes wrong down the line and I get:
wandb: ERROR Error while calling W&B API: base artifact QXJ0aWZhY3Q6NDk5OTE4NTEx is no longer committed (DELETED), rebase required (<Response [409]>)

I hope that fixing this should be a matter of locating what tries to reference the deleted Artifact.
@schmidt-ai feel free to give any suggestions and ideas that you might have.

Is there any update on this?

Unfortunately, in the end I didn't locate what caused mentioned error, and I had to move on due to lack of free time, sorry :c
There was some weird update two weeks ago on the issue/question I posted on linked wandb's github, but I doubt any progress has happened on their side.
My guess is that this feature is very much doable with the current wandb API, but I never tracked down this error.


Remove yapf requirement to support python > 3.6

This PR fixes https://github.com/dnanhkhoa/nb_black/issues/40. Based on @IsaGrue's fork https://github.com/IsaGrue/nb_black

Looks like no-one is going to get to this. @augustobottelli it might be a shout to simply make a new repository with a new name and upload to pypi.

Still an issue


navigatedEvent in SharePoint Framework Extension is triggering on same page reload

Category

[x] Bug

Describe the bug

navigatedEvent in application customizer is triggering on same page reload.

Steps to reproduce

Create navigatedEvent in onInit method
Reload same page
Navigated event will be triggered

Expected behavior

Action created for navigatedEvent should trigger only when navigated to another page from current page.

@user546 do you mean you hit f5 to refresh the page? Or you click on navigation item?

Closing for lack of an update.  Feel free to open a new issue if you encounter a problem.  Thanks!


Add support for Web Modeler public API

Here is the API spec: https://modeler.cloud.camunda.io/swagger-ui/index.html
Here is a PoC CLI client for it: https://github.com/camunda-community-hub/web-modeler-cli

Thanks for sharing @pihme 💯 That's a great idea!


Deploy our OCI artifact to a private registry

What does this PR do?
This PR starts deploying our OCI artifacts to an internal registry.

Motivation

Additional Notes

This is probably a bit too datadog-agent centric and should be reworked to have a base job that handles multiple flavors, but we'll see about that when we get there
Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


protonvpn-gui: add glib-networking dependency

Description of changes
This dependency is needed for the initial login captcha. Without it, it is impossible to log in.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

I added an overlay in my config to get it working: https://github.com/gvolpe/nix-config/commit/c78e985876db4dd7214bfa872c83631f5e60fc27

Result of nixpkgs-review pr 179730 run on x86_64-linux 1

1 package built:

protonvpn-gui

Should this get backported to 22.05 along with #172939 ?


feature: Add "Next" and "Previous" Buttons for Module Navigation

Description:
We need to implement "Next" and "Previous" buttons below the video to allow users to easily navigate between modules and courses.
Scope:

Add two buttons, "Next" and "Previous," below the video in the course module page.
The buttons should allow users to:

Move to the next module if available.
If there are no more modules in the current course, navigate to the next course.
Move to the previous module if available.
If at the first module, navigate to the previous course’s last module.
If no further modules or courses are available, disable the respective button.

Acceptance Criteria:

"Next" button should navigate to the next module in the current course.
If the last module of the course is reached, the "Next" button should navigate to the first module of the next course.
"Previous" button should navigate to the previous module in the current course.
If at the first module of the course, the "Previous" button should navigate to the last module of the previous course.
Both buttons should be disabled when there are no further modules or courses.
Buttons should be visible below the video player on the module page.

UI/UX Considerations:

Place buttons directly below the video player.
Ensure the button design is consistent with the overall UI theme.
Add hover and disabled states to indicate functionality.

Hey i was also looking into it. Completely agree with you @MeerUzairWasHere because the problem is routes are not linear in nature. would like to know more about your approach.
And i also seem to remember previously this feature was available so i will dig into the commits once.

I already told my approach above. If any repo maintainer agrees to add prev and next urls. Then I'll look into it.

Don't think we can easily get the next and prev urls. We need to calculate with the tree that we have of course content


Allow erb extension for template files

Is your feature request related to a problem? Please describe.
If I want to override the default template, for example lib/generators/rspec/model/templates/model_spec.rb, I need to create a file lib/templates/rspec/model/model_spec.rb. In this file, there is erb syntax. Because of that, some checks (rubocop...) do not pass.
Describe the solution you'd like
When searching for templates, check for files with rb or erb extensions.
Describe alternatives you've considered
Currently I need to exclude the templates from the checks (pre-commit hooks...).
Additional context
Example: let's just say I want to remove require 'rails_helper' from the template. I copy-paste the content from lib/generators/rspec/model/templates/model_spec.rb into lib/templates/rspec/model/model_spec.rb then I do the changes. I run rubocop::

:wave: The generators are based off Rails generators, which basically take a path to look for and I'm not sure how well that would supporting looing for multiple files, and then how priority would work for that. So I'm going to close this as "not planned".
However you should be able to easily add Exclude: lib/templates/rspec/**/*.rb to your rubocop config either for all rules or individual rules which would solve your problem.


Enhancement: support  chatglm3

What features would you like to see added?
support  llm chatglm3，and function  calling
More details
no more detail
Which components are impacted by your request?
Endpoints, Plugins
Pictures
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

There may be a huggingFace integration soon but is not a priority. You could get any local large language model working with LiteLLM + VLLM or some other tool: https://docs.litellm.ai/docs/providers/vllm


TimeNorm for Italian

Hello,
As requested, I created a pull request for adapting TimeNorm for Italian.
Cheers,
Paramita

Sorry about the long delay. I've finally included your Italian time normalizer additions. I didn't use the pull request directly, since there were some edits in there that shouldn't be incorporated (e.g., to pom.xml and README.md), and because I wanted to improve how additional language tokenizers like yours are included. But I believe I've incorporated all of your code in one form or another in the commits 844c7d6, d53a99a, f99bf2b, and 889a934.


feat: added a vesting.md documentaion

Added a vesting.md markdown file that explains the vesting component for protocols looking to use Konoha.

Imagine that you are a protocol that wants to implement Vesting of their token for their team and they open documentation. Will they understand what Konoha can offer and what it cannot?

you fixed the few concere issues I raised but didn't address the bigger picture.

once ready for review, please re-request review

Again, you only addressed the concrete minor issues I raised but not the bigger picture. Closing


Enable kernel verification

This completes the kernel update code. Last piece will be to update the post-install script action so the script won't clobber the kernel update_engine already installed.

Nits. LGTM pending @marineam 's decisions.


Added tests for Site.find_for_request

This tests the Site.find_for_request function with a few different types of requests to make sure it works under various conditions.
This is my first time writing a test. Please tell me if you notice a way it can be organized better or more effective. Thanks!

Merged in 7296a1dd7198ab60747572cde154fd927facebf0 and split up in 0dc8fdb3b3442377cb0e0ed6843efd53404a035d
Thanks!


Use the compatible release clause in requirements.txt

Coverage increased (+0.1%) to 85.23% when pulling 5c62917d7cdb2caec8b5084a1c65ec8a4b3ed590 on compatible_release into 65331b50661d2f12c9463f99c055ab447fe5a207 on master.

Coverage increased (+0.1%) to 85.23% when pulling 5c62917d7cdb2caec8b5084a1c65ec8a4b3ed590 on compatible_release into 65331b50661d2f12c9463f99c055ab447fe5a207 on master.


Including SVGs in Markdown results in

I have the following page bundle:
content
├── en
└──── basics/
        ├── index.md
        ├── picture.svg
        └── picture.png

Inside index.md I have the following:
# Page title

Lorem ipsum here is a picture:

![Picture text.](picture.png).

This works perfectly fine, and the site builds normally. However, if I want to use the .svg version of this image, then I get the following error:
ERROR 2022/04/28 12:33:18 "/home/io/Code/lotus-docs/layouts/_default/_markup/render-image.html:2:18": execute of template failed: template: _default/_markup/render-image.html:2:18: executing "_default/_markup/render-image.html" at <$image.Resize>: error calling Resize: *resources.genericResource is not an image

If I switch the image reference back to the .png version, then the site builds normally again.
Is there a special way to reference SVGs in Doks?

same here

Is there a special way to reference SVGs in Doks?

No, SVG (and WebP) are not supported yet. Will certainly be added!


Default to shallowClone

This will not be done, it's great to be able to keep a cache of the latest build, and with the normal mode, it's easier to do.

which is faster?

@thelinuxlich depends your need, personally I don't use shallowClone, I use a local directory as workspace, so I don't have to re-install node_modules at each deployment.


Update README.md

What kind of change does this PR introduce? (check one with "x")

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ x] Other... Please describe: readme change

Does this PR introduce a breaking change? (check one with "x")

[ ] Yes
[x ] No

Other information:
AoT is very interesting because of performance on bootstrap of webapp

 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.

but before add this I have problem when running my app using --aot LOL it stop at 20% of building I don't know why this happends and how you proceed


[priority] Detect hostname of DSN

If Sentry is used, detect the hostname of the DSN. Are they sending to sentry.io or some other domain? This can help distinguish self-serve vs. SaaS customers.

addressed in https://github.com/sentry-demos/detective-scentry/pull/17

Completed


Zoom button

Monil will provide mock up of zoom in and out buttons.
We observed some folks struggling to use the zoom wheel during some testing, buttons might help.

Makes sense. Supposedly easy to do with the library that I'm using.


Delete temporary self-signed certs in SSLEngineTest-based tests

Motivation:
Lots of usages of SelfSignedCertificates were not deleting the certs at
the end of the test. This includes setupHandlers() which is used by
extending classes. Although these files will be deleted at JVM exit and
deleting them early does not free the JVM from trying to delete them at
shutdown, it's good practice to delete eagerly and since users sometimes
use tests as a form of documentation, it'd be good for them to see the
explicit deletes.
Modifications:
Add missing delete() calls to ½ of the SelfSignedCertificates-using
tests.
Result:
Tests that more clearly communicates which resources are created and
may accumulate without early delete.
CC @normanmaurer

Cherry-picked into 4.1 (2fbce6d470862b8896a44cddf1a061b3b738d206) and 4.0 (62a207b5e268ebe483cbcaca0142510ec3b28870)

@ejona86 thanks buddy!


Components in Maven Central are assumed to have extension of jar

Describe the bug
The results for<EMAIL_ADDRESS>result in a component not found.
To Reproduce
1: Go to https://central.sonatype.com/artifact/com.fpliu.ndk.pkg.prefab.android.21/curl/7.82.0/overview
2: Click on browser plugin icon to display results.
3: Component not found is displayed.
Expected behavior
For non-jar results, such as occurred with this component (it is an aar filetype), return a "non-supported" result.

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

Chrome Web Store
v2.10.5
GitHub release

Your semantic-release bot :package::rocket:


testcafe sample to be added

Is your feature request related to a problem? Please describe.
feature request - we are using testcafe for web automation. not found example of axe-pipelines on how to use for testcafer
https://github.com/DevExpress/testcafe
Describe the desired outcome
testcafe axe a11y automation sample to be provided

Hello @hasans30, thank you for your interest in Accessibility Insights and this feature request. You should be able to use the axe-core integration to testcafe and follow documentation available in one of the samples to integrate automated accessibility checks in engineering pipeline. Even when the samples are specific to Selenium integration to axe-core, the concepts are applicable and extendable to any UI test frameworks like testcafe, Protractor, Cypress, etc.
Please let us know if you run into issues. We will keep this request open to assess the community interest in a separate sample for testcafe.

Moved to close as was resolved with recommendation by Dec. 2019 comment.


Add-Migration InitialCreate creates error "Unable to create an object of type 'RazorPagesMovieContext'."

When entering the command "Add-Migration InitialCreate" an error is displayed "Unable to create an object of type 'RazorPagesMovieContext'. For the different patterns supported at design time, see https://go.microsoft.com/fwlink/?linkid=851728" and no migration file is created. Using .net 5.0 Core instructions.
VS Version Info:
Microsoft Visual Studio Enterprise 2019
Version 16.8.4
VisualStudio.16.Release/16.8.4+30907.101

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

ID: 6719f08e-3bd7-dc1a-71df-f2ef9fbca9d8
Version Independent ID: 7096fdb3-612e-9e00-bd0b-8ea4886a09ce
Content: Part 2, add a model
Content Source: aspnetcore/tutorials/razor-pages/model.md
Product: aspnet-core
Technology: aspnetcore-tutorials
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

The tutorial works. I'm not sure what step you missed. You might try asking on StackOverflow,.

I was able to identify the issue. We use this tutorial to help our students get introduced to Razor pages, and I do not remember this issue appearing in the past several versions we have used. However the additional details are below.
The problem occurs until you modify the startup.cs as later indicated. See the image below. No step was missed.
<EMAIL_ADDRESS>Once you modify the startup.cs file with the code, the message then changes to the one warned about in the tutorial. See image below.
<EMAIL_ADDRESS>Rick Smith
Email: rictersmith<EMAIL_ADDRESS>From: Rick Anderson<EMAIL_ADDRESS>Sent: Saturday, February 06, 2021 8:49 PM
To: dotnet/AspNetCore.Docs<EMAIL_ADDRESS>Cc: Rick Smith<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [dotnet/AspNetCore.Docs] Add-Migration InitialCreate creates error "Unable to create an object of type 'RazorPagesMovieContext'." (#21462)
The tutorial works. I'm not sure what step you missed. You might try asking on StackOverflow,.

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdotnet%2FAspNetCore.Docs%2Fissues%2F21462%23issuecomment-774596555&data=04|01||44077c7cd6ee494a399308d8cb23ae4e|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637482701391716971|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=N7trVs4%2FnCiypluZ%2F34UhTZSdtY3XN3FetdSFho%2B52M%3D&reserved=0, or unsubscribehttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAFI2AFWYTBLYQOOKZSWONGLS5YLTTANCNFSM4XHA6VJQ&data=04|01||44077c7cd6ee494a399308d8cb23ae4e|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637482701391726936|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=D7Jk8WW0%2BOKzanvr0ZkxhJUupBkWext9Rr3saPNWOKQ%3D&reserved=0.

In case the images did not come through, I am trying to upload them here.

Sounds like you figured out the problem.

Yes, thank you. I still think it would be beneficial for learners to adjust the instructions so that the startup.cs file is modified before instructed to run the DB commands. I've annotated our teaching notes so that we can help students realize the error will resolve itself in the next step. We encourage them that when errors come up during these tutorials to stop and go back to see what was missed. In this case, nothing is missed, and a change in the order in the instructions, I think would help.
Rick Smith
From: Rick Anderson<EMAIL_ADDRESS>Sent: Sunday, February 07, 2021 10:37 AM
To: dotnet/AspNetCore.Docs<EMAIL_ADDRESS>Cc: Rick Smith<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [dotnet/AspNetCore.Docs] Add-Migration InitialCreate creates error "Unable to create an object of type 'RazorPagesMovieContext'." (#21462)
Sounds like you figured out the problem.

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdotnet%2FAspNetCore.Docs%2Fissues%2F21462%23issuecomment-774729161&data=04|01||91471984b29143da224608d8cb9769de|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637483198461591888|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=ZsYzRzYXn2nqx0O3kkNxMUq%2FJD37f2YCe55t95w3uR8%3D&reserved=0, or unsubscribehttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAFI2AFVVEP4Q7Y5VWZ7GJN3S53MWFANCNFSM4XHA6VJQ&data=04|01||91471984b29143da224608d8cb9769de|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637483198461601846|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=3pRwpHMgadHpFiT20uhuRsGzI%2BI8AOkjq7MxmyXYYwQ%3D&reserved=0.

The tutorial works as it is. Can you reproduce this behavior? What startup needs to be changed for it to work? We have over a million reports of success and this is the first failure that you describe.

Rick,
Thanks for getting back to me. As a reminder, this is using .Net Core v5.0 with the latest version of visual Studio. It is totally possible that this issue was introduced by .Net 5 or an update to Visual Studio that has since broken the flow of the tutorial since I last ran through it with our students.
I recorded a zoom video that can be seen here. You can see that I can reproduce the error, and it should help validate that I did not miss any steps in the tutorial.
https://byui.zoom.us/rec/share/IYgcLh9o_XDTGGmtwB0597a2UYhg8V5F36demJJfPPLSshhjo8xyCUfCGBj7fFQP.8xp1yYPc7P6GetaV?startTime=1612749931000
Rick Smith
From: Rick Anderson<EMAIL_ADDRESS>Sent: Sunday, February 07, 2021 10:56 AM
To: dotnet/AspNetCore.Docs<EMAIL_ADDRESS>Cc: Rick Smith<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [dotnet/AspNetCore.Docs] Add-Migration InitialCreate creates error "Unable to create an object of type 'RazorPagesMovieContext'." (#21462)
The tutorial works as it is. Can you reproduce this behavior? What startup needs to be changed for it to work? We have over a million reports of success and this is the first failure that you describe.

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdotnet%2FAspNetCore.Docs%2Fissues%2F21462%23issuecomment-774733187&data=04|01||7c58981335db40e5c17508d8cb9a1051|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637483209837736986|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=npTP9MznqvMvfqzXgZ%2FjXeCqTZmhd%2FxcjKTNMkipTxk%3D&reserved=0, or unsubscribehttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAFI2AFTRNYE2NM3Y5VJ2Y5LS53O5NANCNFSM4XHA6VJQ&data=04|01||7c58981335db40e5c17508d8cb9a1051|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637483209837736986|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=4iA0BhX6VW00FXKtHaR3aEZYtFu4%2BXgi5y79kQimtXY%3D&reserved=0.

What exactly is the step that's missing?

Rick,
The suggestion I made in the case, and in the video, was to adjust "when" you run the two commands:
Add-Migration InitialCreate
Update-Database
If you first modify the startup code with this, Add-Migration will not error out. Otherwise it refuses to run and the process is stalled. Users are left to guess at what they should do next or simply try the next steps to see if it fixes the issue or not:
public void ConfigureServices(IServiceCollection services)
{
services.AddRazorPages();
services.AddDbContext(options => options.UseSqlServer(Configuration.GetConnectionString("RazorPagesMovieContext")));
}
Again, I have no idea why this is now happening with Visual Studio 2019 and .net Core 5.0. We have been leveraging this tutorial since at least 2.2 and every version since. This is the first time we are trying the steps using .net 5.0, which is when we became aware of the issue.
Maybe the issue isn't with the instructions but with the Visual Studio or .net core 5.0 code updates, which might justify having you submit a bug fix to the other teams. I don't know myself, and so maybe the suggestion to modify when the Add-Migration is merely a work around to a different problem that needs to be solved so that the modification to the startup.cs file isn't required before running the Add-Migration InitialCreate command. I'm hoping your expertise on what "should" be happening, might help pinpoint root cause for the error appearing.
I apologize if what I have been sending is not helpful in showing exactly where the issue is. I'm just that one in a million guy trying to make it better for the next million. If I didn't care, I would not have submitted any feedback at all.
Rick Smith
From: Rick Anderson<EMAIL_ADDRESS>Sent: Sunday, February 07, 2021 8:27 PM
To: dotnet/AspNetCore.Docs<EMAIL_ADDRESS>Cc: Rick Smith<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [dotnet/AspNetCore.Docs] Add-Migration InitialCreate creates error "Unable to create an object of type 'RazorPagesMovieContext'." (#21462)
What exactly is the step that's missing?

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdotnet%2FAspNetCore.Docs%2Fissues%2F21462%23issuecomment-774857271&data=04|01||e9b3c3b60c4148cb548a08d8cbe9d092|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637483552373771097|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=PEERLEYLb48J9aQh4HhzEtb91pRgvdkqUPrkTMjkbJ8%3D&reserved=0, or unsubscribehttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAFI2AFWZP2VB6MHLDXO77OLS55R2FANCNFSM4XHA6VJQ&data=04|01||e9b3c3b60c4148cb548a08d8cbe9d092|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637483552373771097|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=QRoR1jXUrEwmUL5UU5jeA15joFH8FZnmRKdRXcX%2Bzc8%3D&reserved=0.


.MapFallbackToPage() in RazorPages-Project fails with "UnsupportedApiVersion"

2.1+ allows Razor Pages to pass through

Sorry to say, but adding
services.AddApiVersioning(options =>
{
  options.DefaultApiVersion = new ApiVersion(1, 0);
  options.AssumeDefaultVersionWhenUnspecified = true;
});

and
app.UseEndpoints(endpoints =>
{
  endpoints.MapRazorPages();
  endpoints.MapFallbackToPage("/Index");
});

to a NET5.0 Razor Pages project created from the default template also throws the UnsupportedApiVersion error:
{"error":{"code":"UnsupportedApiVersion","message":"The HTTP resource that matches the request URI 'http://localhost:5000/' is not supported.","innerError":null}}
Referenced ApiExplorer is 5.0.0-preview.1
Originally posted by @DNF-SaS in https://github.com/microsoft/aspnet-api-versioning/issues/189#issuecomment-713680503

Are you saying this only happens if you use MapFallbackToPage? API Versioning does not consider endpoints that do not use [ApiController]  conventions. Do you have a repro?

Are you saying this only happens if you use MapFallbackToPage? API Versioning does not consider endpoints that do not use [ApiController]  conventions. Do you have a repro?

Yes.
And yes, very simple:

Create new project from template "ASP.NET Core Web Application" using C# - select "ASP.NET Core Web App" in the "Create a new ASP.NET Core web application"-page of the wizard
Run the application - it will start and show the index-Page (big 'Welcome' Headline)
Stop the application.
Add NuGet-Reference to Microsoft.AspNetCore.Mvc.Versioning.ApiExplorer (5.0.0-preview.1)
Open Startup.cs
Add services.AddApiVersioning(); to ConfigureServices()-Method (after Line 26)
Add endpoints.MapFallbackToPage("/Index"); to Configure()-Method (after Line 50)
Start application again and voilà: the ApiVersionUnspecified-Error shows

This is, in fact, a bug.
It turns out this is a regression due to IDynamicEndpointMetadata. API Versioning previously didn't care about this at all, but OData makes extensive use of it. It turns out that other policies such as mapping a fallback also uses this. I need to dig in some more to figure out how to disambiguate dynamic OData routes or whether there is an alternative.
Thanks for trying out the preview and reporting this. It will almost certainly affect the current release candidate too.


[Snyk] Security upgrade mongoose from 4.2.4 to 5.13.20

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `npm` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

package.json
package-lock.json

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Breaking Change
Exploit Maturity

798/1000  Why? Proof of Concept exploit, Recently disclosed, Has a fix available, CVSS 8.1
Prototype Pollution SNYK-JS-MONGOOSE-5777721
Yes
Proof of Concept

(*) Note that the real score may have changed since the PR was raised.

Commit messages

Package name: mongoose
    The new version differs by 250 commits.

0f3997a chore: release 5.13.20
f1efabf fix: avoid prototype pollution on init
98e0762 chore: release 5.13.19
7e36d21 chore: release 5.13.18
6759c60 undo accidental changes and actually pin @ types/json-schema
4ed4a89 chore: pin version of @ types/json-schema because of install issues on node v4 and v6
9a9536d Merge pull request #13535 from lorand-horvath/patch-12
26424d5 5.x - bump mongodb driver to 3.7.4
4b8b0a9 add versionNumber to 5.x
1bc07ec chore: release 5.13.17
3f827b3 Merge branch '5.x' of github.com:Automattic/mongoose into 5.x
eeabe5f chore: run CI tests on ubuntu 20.04 because 18.04 no longer supported
14464d1 Merge pull request #13195 from raj-goguardian/gh-13192
7e888e4 fix(update): handle $and & $or in array filters.
5dd0a4e Merge pull request #13138 from rdeavila94/gh-13136
c8191da Update model.indexes.test.js
7364264 Update model.indexes.test.js
77b9d99 Updated the isIndexEqual function to take into account non-text indexes when checking compound indexes that include both text and non-text indexes
9dd82be Merge pull request #13132 from rdeavila94/gh-12654
d0e149b Merge pull request #12737 from Automattic/vkarpov15/gh-12654
e76c41c chore: release 5.13.16
cdab11e chore: remove Node 5 and 7 from CI because GitHub actions is bugging out with them
e33a8be fix(types): add missing typedefs for bulkSave() to 5.x
896cd76 Merge pull request #12692 from hasezoey/backportLinkUpdate

See the full diff

Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Prototype Pollution

Checkmarx One – Scan Summary & Details – 9e096945-cc3a-4565-a7d9-f0cd54589c71
New Issues

Severity
Issue
Source File / Package
Checkmarx Insight

CVE-2022-25883
Npm-semver-6.3.0
Vulnerable Package

CVE-2022-25883
Npm-semver-5.7.1
Vulnerable Package

CVE-2022-25883
Npm-semver-5.7.0
Vulnerable Package

CVE-2022-25883
Npm-semver-5.5.0
Vulnerable Package

CVE-2022-25883
Npm-semver-1.1.4
Vulnerable Package

CVE-2023-26136
Npm-tough-cookie-2.4.3
Vulnerable Package

CVE-2023-26115
Npm-word-wrap-1.2.3
Vulnerable Package

Fixed Issues

Severity
Issue
Source File / Package
Checkmarx Insight

CVE-2017-20165
Npm-debug-2.2.0
Vulnerable Package

CVE-2018-16490
Npm-mpath-0.1.1
Vulnerable Package

CVE-2019-17426
Npm-mongoose-4.2.4
Vulnerable Package

CVE-2020-13110
Npm-kerberos-0.0.24
Vulnerable Package

CVE-2020-7610
Npm-bson-0.4.23
Vulnerable Package

CVE-2021-23438
Npm-mpath-0.1.1
Vulnerable Package

CVE-2022-2564
Npm-mongoose-4.2.4
Vulnerable Package

Cx3ac14f8c-0652
Npm-mongoose-4.2.4
Vulnerable Package

Cxba0aa4f8-fd76
Npm-mongoose-4.2.4
Vulnerable Package

Cxd6c215a2-86bd
Npm-mongodb-2.0.46
Vulnerable Package

CVE-2019-2391
Npm-bson-0.4.23
Vulnerable Package

CVE-2020-35149
Npm-mquery-1.6.3
Vulnerable Package

Cxc8ffd605-ddff
Npm-mquery-1.6.3
Vulnerable Package


ectra user activation failed

hello i tried to use the code for extra user code you gave 99999930?, but didint work, what do i do?

Hi. Was qbactivator running when you attempted to enter the activation code? If you exit the activator prematurely, the activation may not be successful. Also ensure that you have an internet connection available for successful activation.

hello,thnaks bro , it worked after running keeping the activator running

Well done. You're welcome.


feat(vmix): add support for starting and stopping VB.NET scripts

This PR is being opened on behalf of TV 2 Norway.

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

Feature

What is the current behavior? (You can also link to an open issue here)

vMix scripts cannot be started or stopped with TSR.

What is the new behavior (if this is a feature change)?

vMix scripts can be invoked and cancelled now:
export const input: TSRInput = {
	timeline: [
		literal<TSRTimelineObj<TimelineContentVMixScript>>({
			id: 'script0',
			enable: {
				start: Date.now(),
				duration: 20 * 1000,
			},
			layer: 'vmixLayer0',
			content: {
				deviceType: DeviceType.VMIX,
				type: TimelineContentTypeVMix.SCRIPT,
				name: 'myscript',
			},
		}),
	],
}

Other information:

Tested via quick-tsr by writing a short VB.NET script that just has two console writes with a sleep between them:
Console.WriteLine("I sleep...")
Sleep(5000)
Console.WriteLine("I wake!!!")

Codecov Report
Base: 31.18% // Head: 31.12% // Decreases project coverage by -0.07% :warning:

Coverage data is based on head (9f2d4ee) compared to base (4235237).
Patch coverage: 15.15% of modified lines in pull request are covered.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@              Coverage Diff              @@
##           release49     #244      +/-   ##
=============================================
- Coverage      31.18%   31.12%   -0.07%
=============================================
  Files             67       67
  Lines           8555     8588      +33
  Branches        2176     2183       +7
=============================================
+ Hits            2668     2673       +5
- Misses          5365     5390      +25
- Partials         522      525       +3

Impacted Files
Coverage Δ

...state-resolver/src/integrations/vmix/connection.ts
40.53% <0.00%> (-1.69%)
:arrow_down:

...line-state-resolver/src/integrations/vmix/index.ts
0.00% <0.00%> (ø)

packages/timeline-state-resolver-types/src/vmix.ts
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Add example of using fhir views with data pipes

Description of what I changed

Add example of using FHIR views with data pipes
E2E test

TESTED:
Please replace this with a description of how you tested your PR beyond the
automated e2e/unit tests.
Checklist: I completed these to help reviewers :)

[x] I have read and will follow the review process.

[x] I am familiar with Google Style Guides for the language I have coded in.
No? Please take some time and review Java and Python style guides.

[x] My IDE is configured to follow the Google code styles.
No? Unsure? -> configure your IDE.

[N/A] I have added tests to cover my changes. (If you refactored existing code that was well tested you do not have to add tests)

[x] I ran mvn clean package right before creating this pull request and added all formatting changes to my commit.

[x] All new and existing tests passed.

[x] My pull request is based on the latest changes of the master branch.
No? Unsure? -> execute command git pull --rebase upstream master

Codecov Report

Merging #715 (a041fa3) into master (a57f13b) will increase coverage by 4.05%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master     #715      +/-   ##
============================================
+ Coverage     34.04%   38.10%   +4.05%
+ Complexity      291      251      -40
============================================
  Files            71       51      -20
  Lines          3818     2661    -1157
  Branches        437      311     -126
============================================
- Hits           1300     1014     -286
+ Misses         2397     1527     -870
+ Partials        121      120       -1

Impacted Files
Coverage Δ

...h/src/main/java/org/openmrs/analytics/FhirEtl.java
0.00% <0.00%> (ø)

...in/java/org/openmrs/analytics/PipelineManager.java
13.74% <0.00%> (-1.09%)
:arrow_down:

... and 29 files with indirect coverage changes


Update fog to 1.36.0 for DigitalOcean v2 API support

Use latest released version of fog Gem to support DigitalOcean V2 API.

this requires some doc updates to... standalone readme.  otherwise :+1: lgtm

It'll also require an update to omnibus-coopr... I will file that one, first then docs


Support to execute RedArmory by Scheduler

Ideas to achieve
Command line interface for Task scheduler

Prepare command line interface and function to register/remove RedArmory task to scheduler

Command line interface for Windows Service

Someone user may dislike Windows Service

I decided to adopt the command line interface idea.

Add task schedule from GUI.
#16 is for this task.

Close by f5f26c431f8514e9cae75d3fb49e2afbf6d3f026


Version 1.2.0 System sounds no longer works

System sounds volume knob (fallback setting) on focused app volume plugin does not work in version 1.2.0.
Tested version 1.1.0 and confirmed it works in that version.
Note: the "reset" functionality on a long press seems to work, but it does not control the volume, mute, or display the correct information for system sounds specifically. It seems to work perfectly for every other app I have tried.

That is interesting. Its the setting I use for my setup, and i haven't seen any issues with it.
If you turn the knob, does it change the volume, but just not display the right number, or does it do nothing?
do you use anything like voicemeeter, or multiple output devices, or anything else that may be unique about your setup?
There's a possible defect with the way it gets the system sound session. It iterates through all output devices to find the system sounds. I wrote that code when working on the voicemeeter fixes, and maybe there's a way for there to be a system sound session for multiple devices.
Though i thought when i was testing that i could only get a system sound session to be associated to a single device.

unable to reproduce - closing due to inactivity


cursor: account for both width and height when picking nearest cursor

Cursors don't have square form in general, so we should match
both width and height when picking nearest images.

Looks good to me, thanks!
Has this been tested on a non-square cursor theme?

Has this been tested on a non-square cursor theme?

Yes, on this one https://www.gnome-look.org/p/999983/ . It has pretty weird sizes.


Mrc-4720 Backend For all reports page + misc

This PR contains the BE work needed for the all reports page. Note: some FE stuff has been added just to ensure the BE things work as expected. Thus may be best to not review FE and review it once i make new ticket for all FE changes.
Below are things done:

Schema index add
add endpoint + rest for all reports
FE: add Home page to test BE changes (this will be updated in next PR so can probs skip over)
Added more demo data for testing purposes

Oh, of course the github worflows aren't running because they're configured to only run for PRs into main - just ran into this one myself.. Can temporarily update to include PRs to mrc-4716


[DOC] Refine documentation of runtime feature detection

Description
This refines the documentation of the Python API.
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[x] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[x] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[x] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

@aaronmarkham

@mxnet-label-bot add[Doc, pr-awaiting-review]

One other thing - I couldn't find how to navigate to this API. I had to search for it.
It would make sense to have it listed here:
http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-14238/2/api/python/index.html#tools

@aaronmarkham There's a link in the page you showed: "Run-Time Feature detection / Library Info
Run-Time Feature detection / Library info
"

@aaronmarkham is this PR good to go?


ets_ros2: 0.1.1-2 in 'crystal/distribution.yaml' [bloom]

Increasing version of package(s) in repository ets_ros2 to 0.1.1-2:

upstream repository: https://github.com/brunodmt/ets_ros2.git
release repository: https://github.com/brunodmt/ets_ros2-release.git
distro file: crystal/distribution.yaml
bloom version: 0.8.0
previous version for package: null

ets_cpp_client
* Add some more parameters
  Specifically acceleration, engine_running, world_placement (x,y,z,
  heading,pitch,roll)
* Add trailer_connected value
* Add RPM value
* Add missing CMakeLists files
* Initial commit
* Contributors: Bruno Demartino, Hernán Gonzalez

ets_msgs
* Add some more parameters
  Specifically acceleration, engine_running, world_placement (x,y,z,
  heading,pitch,roll)
* Add trailer_connected value
* Add RPM value
* Add missing CMakeLists files
* Initial commit
* Contributors: Bruno Demartino, Hernán Gonzalez

ets_plugin
* Add some more parameters
  Specifically acceleration, engine_running, world_placement (x,y,z,
  heading,pitch,roll)
* Add trailer_connected value
* Add RPM value
* Fixed unused-parameter warnings
* Add missing CMakeLists files
* Initial commit
* Contributors: Bruno Demartino, Hernán Gonzalez

Closing and reopening to retrigger Travis.


Extraction of scalar pattern matching has regressed

Extraction results in following errors using latest Lean libraries and latest Charon:
error: ././././Verification/Extracted.lean:206:4: unsupported pattern in syntax match
  (-2)#i8

Desynchronized Charon/Aeneas + Lean's library issue.


Add Debian/Ubuntu t64 packages.

Debian is making some changes to support 64-bit time_t on some 32-bit platforms (most importantly ARM).  As part of this, a number of library packages' names have changed to add a t64 suffix on all platforms, including ones like amd64 that always used a 64-bit time_t.
Ubuntu has incorporated these changes in 24.04 LTS (Noble Numbat), their latest release, so their users will also be affected.
We've already seen Firefox crash reports where stack unwinding failed because these scripts didn't know to scrape the new packages.  I've edited them to add the t64 versions; I found the affected packages with this small shell script:
awk '/packages=/,/^"/{if(NF>1){print$1}}' script.sh | while read pkg
do apt-cache show ${pkg}t64 >/dev/null 2>&1 && echo UPDATE $pkg; done

I'm not sure if this is the right approach, or if it would be better to make the script automatically check for t64 versions of every package.  Also, this is untested, and I don't know what the right way is to test changes to these scripts.

I'm not sure if this is the right approach, or if it would be better to make the script automatically check for t64 versions of every package.

I think we can do that, lemme try.

Also, this is untested, and I don't know what the right way is to test changes to these scripts.

I'll run them locally, you can also test them by updating this and running one of the system symbols scraping tasks with ./mach try fuzzy --full.


Adds fullscreen config and keybinding feature (Fixes #38)

This should probably be added to https://github.com/jwilm/alacritty/pull/1570.
The issues discussed in that PR probably apply for this one too.
I'd recommend sending a PR to his version of this change for the keybinding feature, then the remaining issues can be resolved by whoever wants to tackle them.


add additional call number parser with regex feature

Hi,
we have lots of non-standart call number types at the bonn university library (please see attachment). None of alma call number parser works for us. Therefore we would like to process these special formats with a regex call number parser.
For example,
call number: Z4'68/151(7)+1 (left one) => spine label (right one)

Regex for this example:
.replace(/^Z(\d)'(\d{2})/(\d{3})((\d))+(\d)$/, "Z $1° $2 ---- $3 ($4) ^$5");
Is it possible to split call number parts into lines and use the parts for adding as field on template?
call_number_part_1: Z 4°
call_number_part_2: 68
call_number_part_3: / or ----- (a solid line)
call_number_part_4: 151
call_number_part_5: (7)
call_number_part_6: ^1
Best regards,
Berker
Etiketten.pdf

Hi,
Could you give me any feedback, if it is possible?
Best
Berker

@berkerY, this enhancement is still under consideration.

@berkerY, in the latest version of the SpineOMatic Cloud App, we expose the first 10 individual parts of the parsed call number for inclusion in a label template. Please check that out to see if it helps you in any capacity.

@mgobat the following fields are required for printing labels:
holding_data.call_number
item_data.call_no
item_data.alternative_call_number
Unfortunately, the call number parts in the new version doesn't help us, because the call number must be split using regex.

@berkerY, any thoughts on my comment above? Thanks.

@mgobat Sorry for the late feedback, but i'm trying to build a one regex for all our call number. By the way, i would you ask you, if it's possible to use multiple regex on a call number.

@berkerY, the proposed solution is for one regex applied to the call number per template definition.
You could have multiple templates supporting different regex, which would require you to know which call number types you are scanning so the correct template could be selected.

First attempt at this enhancement available in the v1.3.6-beta version. To try the beta, click the "i" (information) icon on the SpineOMatic tile in the "Cloud App Center" in Alma, then click the "Try v1.3.6-beta" link.

Available in v1.3.6.


Feature Request: Long Shot But... Random Results from Notes

Going to throw this out there in case it's possible.
Ability to randomly roll on a folder so that instead of having to maintain a note for each item and a table that contains all the items so you can roll on it... just have the ability to return a random note from a folder. With the ability to return a result that includes the ability to display properties from that note.
Example:
Item Name - Cost - Weight - etc
Ability to define a Qty of results would be cool too.
Example:
Create list of 10x items where rarity = common

This would be really cool Maybe you can roll a "folder" for a "note" with "properties" or "tags" as filters for the roll?


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target/pull/482 is merged into live branch.
Auto archive content to https://github.com/VSC-Service-Account/docs-archive-test-target.git

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

File
Status
Preview URL
Details

:x:Error

Details

[Error-GitBranchDeletedOrForcePushed]

Cannot sync git repo to specified commit because branch Release_Archive_master_2020-05-25-13-46-10 has been deleted or has been force pushed: fatal: couldn't find remote ref refs/heads/Release_Archive_master_2020-05-25-13-46-10

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


image update, but no new release

⚠ Please verify that this question has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

📝 Describe your problem
some tags, like 1, latest and debian (and 1.23.11?) were updated yesterday: https://hub.docker.com/r/louislam/uptime-kuma/tags, but I can't find any changelogs. What was changed?/Why were the tags updated/why 1.23.11? I'm a bit confused
📝 Error Message(s) or Log
No response
🐻 Uptime-Kuma Version
1.23.13
💻 Operating System and Arch
unrelated
🌐 Browser
unrelated
🖥 Deployment Environment

Runtime: Docker

@louislam ?

Ops, I pressed the wrong button yesterday, there is no update. But I need to fix the affected tags.

latest, 1, 1-debian and debian tags should be fixed.
Unfortunately, The 1.23.11 tag was overridden incorrectly, I have no idea how to recover it. I hope it won't affect many people as it is an older version tag.


LLVM converts integer types to floats

Hey!
I'm making a compiler which will use LLVM as its backend. But while I was doing some tests, I saw that LLVM, for some reason, was transforming any integer types (i8, i16, i32 or i64) to floats (more precisely, ppc_fp128). But some other types like double work and are not converted to floats (since doubles can hold decimals already).
What I don't understand is that I'm not using any decimal values, and it is still converting them into floats, which actually breaks my code.
Here is a sample demo which does this behavior:
unsafe
{
    LLVMModuleRef module = LLVM.ModuleCreateWithName((sbyte*)0);
    LLVMBuilderRef builder = LLVM.CreateBuilder();

    const string targetStr = "x86_64-unknown-none";

    LLVM.InitializeX86TargetInfo();
    LLVM.InitializeX86Target();
    LLVM.InitializeX86TargetMC();
    LLVM.InitializeX86AsmParser();
    LLVM.InitializeX86AsmPrinter();

    module.Target = targetStr;

    var target = LLVMTargetRef.GetTargetFromTriple(targetStr);
    var machine = target.CreateTargetMachine(targetStr, "generic", "", LLVMCodeGenOptLevel.LLVMCodeGenLevelNone,
        LLVMRelocMode.LLVMRelocStatic, LLVMCodeModel.LLVMCodeModelKernel);

    LLVM.SetModuleDataLayout(module, machine.CreateTargetDataLayout());

    var function = module.AddFunction("main", LLVM.FunctionType(LLVM.VoidType(), null, 0, 0));
    var stack = new Stack<LLVMValueRef>();

    LLVM.SetLinkage(function, LLVMLinkage.LLVMExternalLinkage);
    builder.PositionAtEnd(function.AppendBasicBlock("entry"));

    // Simulating what the stack would look like before Add
    stack.Push(LLVM.ConstReal(LLVM.Int32Type(), 8));
    stack.Push(LLVM.ConstReal(LLVM.Int32Type(), 4));

    var value2 = stack.Pop();
    var value1 = stack.Pop();

    stack.Push(builder.BuildAdd(value1, value2));

    fixed (LLVMOpaqueType** ptr = new[] { LLVM.Int32Type() })
    {
        var tmp = module.AddFunction("tmp", LLVM.FunctionType(LLVM.VoidType(), ptr, 1, 0));
        LLVM.SetLinkage(tmp, LLVMLinkage.LLVMExternalLinkage);
        builder.BuildCall(tmp, new[] { stack.Pop() /* This should be the result of the previous Add instruction */ });
    }

    builder.BuildRetVoid();

    LLVM.DumpModule(module);
}

Here is the LLVM IR output:
target datalayout = "e-m:e-p270:32:32-p271:32:32-p272:64:64-i64:64-f80:128-n8:16:32:64-S128"
target triple = "x86_64-unknown-none"

define void @main() {
entry:
  call void @tmp(ppc_fp128 add (ppc_fp128 0xM40200000000000000000000000000000, ppc_fp128 0xM40100000000000000000000000000000))
  ret void
}

declare void @tmp(i32 %0)

I'm not quite sure if this is a bug, or if I'm using LLVM(Sharp) wrong here. Any help would be greatly appreciated!

Could you clarify which version of LLVMSharp you're using?
For reference, this is incorrect usage and likely and I'm actually surprised its producing IR at all:

LLVMConstReal just calls ConstantFP::get(RealTy, N): https://llvm.org/doxygen/IR_2Core_8cpp_source.html#l01396
This in turn ends up calling FV.convert(Ty->getScalarType()->getFltSemantics(), APFloat::rmNearestTiesToEven, &ignored): https://llvm.org/doxygen/Constants_8cpp_source.html#l00926
getFltSemantics switches on the type id and should be raising llvm_unreachable("Invalid floating type"): https://llvm.org/doxygen/Type_8cpp_source.html#l00067

LLVMSharp is ultimately fairly simple bindings over LLVM. As such, it is very lowlevel and only really has the safety that libLLVM has built into itself, but should in general work exactly like it does in C.
I'd like to eventually build a higher level safe wrapper on top (much like I did for ClangSharp) and there is some semblance on the start of that with things like: https://github.com/dotnet/LLVMSharp/blob/main/sources/LLVMSharp/LLVMContext.cs, but its not nearly there or close to complete yet.

Could you clarify which version of LLVMSharp you're using?

I'm using LLVMSharp 14.0.0-beta1 from NuGet.

For reference, this is incorrect usage and likely and I'm actually surprised its producing IR at all:

I'm fairly new to LLVM in general, so I just guessed how to use it. And documentation doesn't really help in my case, as I'm not actually making an AST or anything, I'm transforming the bytecode to LLVM IR.
So I guess it's not how you create constant values. So yeah, it doesn't generate any error or anything. For reference (which may or may not help), here is what I get when running llc --version, I get this on the first line:
Ubuntu LLVM version 14.0.0

Thanks! That all lines up then. I'm unsure why its not erroring, as it really should be.
I'll leave this issue open to track adding some diagnostics on the LLVMSharp side to help catch some of these issues. The raw LLVM.RealConst wouldn't expose them, but I can expose it via LLVMValueRef.CreateConstReal which is a "safer" (but still lowlevel) wrapper over the raw bindings.


Initial state for a SwitchAction

How can I set the initial state for a SwitchAction before it was inflated? Only writing down a needed value for a needed key to SharedPrefs? Is it possible to do it dynamically?

@Gaket You can copy the SwitchAction class and modify the constructor or add a method by yourself.


Fail to replicate the experiment

Hi there,
I attempted to replicate the experiments using a40-GPUs, but the results are worse than those reported in the paper. I followed the same parameter settings as your code, except I used gradient accumulation to achieve an equivalent batch size.
I have a few questions:

In your code "run_main.py," both the training and testing pipelines are included, and you use accelerate with multiple processes. I noticed that the test loader is also split into multiple parts for each process, yet only the test results of the main process are outputted. Does this mean that you only evaluate your model on a portion of the test dataset?
How many training epochs are typically needed? The validation loss appears to keep increasing during training, causing early stopping to terminate the training around 10 epochs, which is the value of the default patience.
Was the experiment conducted with only one fixed random seed (==2021)? It seems that the random seed has a significant influence on the results. For instance, with different random seeds, my replicated results on ETTh1_96 yield MSE values of 0.392, 0.395, and 0.385 respectively (compared to the reported value in the paper of 0.362).

Are there any implementation details that I might have missed? Thank you in advance for your reply!

Hi there, I attempted to replicate the experiments using a40-GPUs, but the results are worse than those reported in the paper. I followed the same parameter settings as your code, except I used gradient accumulation to achieve an equivalent batch size.
I have a few questions:

In your code "run_main.py," both the training and testing pipelines are included, and you use accelerate with multiple processes. I noticed that the test loader is also split into multiple parts for each process, yet only the test results of the main process are outputted. Does this mean that you only evaluate your model on a portion of the test dataset?

Hi, we use the huggingface Accelerator framework for parallel processing. The accelerator framework ensures that the output is complete. Our train, valid, and test data loaders have all been processed through accelerator.prepare. The final output of the test results represents the entire dataset.

How many training epochs are typically needed? The validation loss appears to keep increasing during training, causing early stopping to terminate the training around 10 epochs, which is the value of the default patience.

Please refer to the relevant content in the paper for specific training configurations. There may be differences in configurations between different datasets.

Was the experiment conducted with only one fixed random seed (==2021)? It seems that the random seed has a significant influence on the results. For instance, with different random seeds, my replicated results on ETTh1_96 yield MSE values of 0.392, 0.395, and 0.385 respectively (compared to the reported value in the paper of 0.362).

For deep learning, the random seed does have a significant impact on the results. Additionally, using the DeepSpeed framework for mixed-precision training does introduce some level of randomness in the results. Variations in GPU models, CUDA versions, framework versions, etc., can also lead to fluctuations in the prediction results. It is recommended to train using 8 A100.
Thank you for your attention and support for our work. If you have further questions about the experimental details, please feel free to communicate with us via email.

Thanks for your reply, but there is still something confusing me.

Hi, we use the huggingface Accelerator framework for parallel processing. The accelerator framework ensures that the output is complete. Our train, valid, and test data loaders have all been processed through accelerator.prepare. The final output of the test results represents the entire dataset.

The accelerate.prepare will split the dataloader into multiple parts for each process, but you use the accelerate.print to output the test results, which will only print if it's the main process. Therefore, in the "run_main.py" file, the "Test Loss: " is the evaluation result on a portion of the test dataset, unless you use accelerate.reduce to integrate all the results of different processes. You mentioned that  "The final output of the test results represents the entire dataset.", could you clarify if this means there is additional code for evaluating the whole test dataset, or if the output of "run_main.py" indeed represents the results of the entire test dataset? If it's the latter, then I can't agree.

Please refer to the relevant content in the paper for specific training configurations. There may be differences in configurations between different datasets.

The paper provides the epoch number settings for different datasets, such as ETTh1 with 50 epochs. However, I'm wondering if this is a fixed value or just an upper limit where early stopping should be considered.

For deep learning, the random seed does have a significant impact on the results. Additionally, using the DeepSpeed framework for mixed-precision training does introduce some level of randomness in the results. Variations in GPU models, CUDA versions, framework versions, etc., can also lead to fluctuations in the prediction results. It is recommended to train using 8 A100.

So the results in paper is run with only one fixed random seed (==2021) one time, rather than the average of multiple runs with different random seeds?
Looking forward for your reply!

Thanks for your reply, but there is still something confusing me.

Hi, we use the huggingface Accelerator framework for parallel processing. The accelerator framework ensures that the output is complete. Our train, valid, and test data loaders have all been processed through accelerator.prepare. The final output of the test results represents the entire dataset.

The accelerate.prepare will split the dataloader into multiple parts for each process, but you use the accelerate.print to output the test results, which will only print if it's the main process. Therefore, in the "run_main.py" file, the "Test Loss: " is the evaluation result on a portion of the test dataset, unless you use accelerate.reduce to integrate all the results of different processes. You mentioned that "The final output of the test results represents the entire dataset.", could you clarify if this means there is additional code for evaluating the whole test dataset, or if the output of "run_main.py" indeed represents the results of the entire test dataset? If it's the latter, then I can't agree.

Thank you for the reminder. We overlooked incorporating this into our evaluation workflow when organizing & publishing the code. We will revise the corresponding files and include comments to prevent any misunderstandings.

Please refer to the relevant content in the paper for specific training configurations. There may be differences in configurations between different datasets.

The paper provides the epoch number settings for different datasets, such as ETTh1 with 50 epochs. However, I'm wondering if this is a fixed value or just an upper limit where early stopping should be considered.

For deep learning, the random seed does have a significant impact on the results. Additionally, using the DeepSpeed framework for mixed-precision training does introduce some level of randomness in the results. Variations in GPU models, CUDA versions, framework versions, etc., can also lead to fluctuations in the prediction results. It is recommended to train using 8 A100.

So the results in paper is run with only one fixed random seed (==2021) one time, rather than the average of multiple runs with different random seeds? Looking forward for your reply!

The accelerate.prepare will split the dataloader into multiple parts for each process, but you use the accelerate.print to output the test results, which will only print if it's the main process. Therefore, in the "run_main.py" file, the "Test Loss: " is the evaluation result on a portion of the test dataset, unless you use accelerate.reduce to integrate all the results of different processes. You mentioned that "The final output of the test results represents the entire dataset.", could you clarify if this means there is additional code for evaluating the whole test dataset, or if the output of "run_main.py" indeed represents the results of the entire test dataset? If it's the latter, then I can't agree.

Thank you for the reminder. We overlooked incorporating this into our evaluation workflow when organizing & publishing the code. We will revise the corresponding files and include comments to prevent any misunderstandings.

The paper provides the epoch number settings for different datasets, such as ETTh1 with 50 epochs. However, I'm wondering if this is a fixed value or just an upper limit where early stopping should be considered.

The epoch we currently set is the maximum number of runs, combined with corresponding learning rate adjustments, scheduler, and early stopping strategies to achieve the best results.

So the results in paper is run with only one fixed random seed (==2021) one time, rather than the average of multiple runs with different random seeds?

This open version was built upon Time Series Library as we acknowledged, where the random seed was 2021 by default for demo purposes. Different seeds can introduce variability in results, and we reported the averaged results.
We appreciate your interest in our research and hope this is helpful to you.

Thanks for your reply, it is helpful.

Hello!
Congrats on this work!
I too am not able to replicate your results. Would it be possible for you to share the hyperparameters(basically the seed) that yielded the best results in your experiments? Are they the ones mentioned in the scripts directory?
I understand the hardware setup may be the reason to this discrepancy. I was wondering if you have conducted any experiments using fewer A80 GPUs to observe how the results vary based on the hardware configuration.
I would greatly appreciate any insights or suggestions you could provide to help me better understand and replicate your findings.
Thank you in advance for any reply!

I also tested around 3.9 MSE on ETTh1, I wonder if the reviewer ever reproduce it.

I also tested around 3.9 MSE on ETTh1, I wonder if the reviewer ever reproduce it.

Thank you for your interest in our work. For current deep learning, considering the differences in Python, Torch, and CUDA versions, as well as differences in running devices, and especially after using advanced operations such as mixed precision and gradient clipping in deepspeed, ensuring consistent results across different environments, devices, and settings is indeed challenging. We are also working hard to overcome this issue. However, as mentioned earlier, this involves differences in open-source frameworks and various engineering issues. In our new work, we are also thinking hard about how to solve this problem. If we make progress in this area, we will let everyone know promptly. Thank you again for your constructive suggestions.


TTS through forked-daapd plays on all speakers.

As stated in this topic, when you send a TTS to the server entity it tries to play on all speakers instead of only the turned on ones.

This is an issue with the HA component and not with this library. I'm in the process of updating the component now, so we can discuss how to improve this functionality. Please post your suggestions on the HA forum topic you linked.


[SLM] Add support for gptj architecture

Due to the issue #1669 , this PR supports gptj model architecture.

It's very strange, if I want to meet the requirements of isort check, then I can't satisfy the requirements of black.

cc @MasterJH5574 @junrushao https://black.readthedocs.io/en/stable/guides/using_black_with_other_tools.html
Perhaps we should remove isort from the lint and focus on black for now

@MasterJH5574 Hi, please take a look at my CI errors.
error: <unknown>:0:0: local memory (66560) exceeds limit (65536) in function 'fused_NT_matmul4_add5_kernel'
What can I do to correct the code? Thanks!

Hi @tlopex! Sorry for being late. Regarding your issue error: <unknown>:0:0: local memory (66560) exceeds limit (65536) in function 'fused_NT_matmul4_add5_kernel', could you help me check the following list?

are you compiling the model on CUDA?
could you help add --debug-dump dist/gptj/debug to the compile command mlc_chat compile ... and compile again? I want check what the fused_NT_matmul4_add5_kernel function looks like in the file dist/gptj/debug/debug-phase4.py.
we had a patch here https://github.com/mlc-ai/relax/commit/2c1ce3ab467f9367c14afd9579ed1388aaae0b90 which can perhaps address this issue if you are running on CUDA and the function fused_NT_matmul4_add5_kernel is a GeMM kernel.

If my hypothesis is incorrect, let's work together to find out the reason.


docs: fix example in passing-props.md

I think the example given for passing in id through props is incorrect. I needed to assign the return value of defineProps and then reference that in order to get the right behavior.
Vue 3.4.29, Vue-Router 4.3.3

The example appears to be correct to me. It shouldn't be necessary to use props.id in the template. This isn't specifically a Vue Router thing, accessing props like this is part of Vue core.
Perhaps you already have something called id in the same scope that's clashing with the prop?

Thanks for the pr but it’s what @skirtles-code said

Ah, okay, thanks for the hint. I double-checked and using {{ id }} without other qualification in the template works fine, but referring to it anywhere in code requires a reference to props. Sorry for the trouble.


Doesn't provide line number of error

Don't see line number specified by the validator, would it be possible to add?

When the JSON is parsed the line number information is lost. The errors do have a context which returns where in the JSON structure the error occurred. It would be possible to reconstruct the line number by looking up the line number of that JSON structure yourself, but I'm not sure of the added value by including that by default in this library itself.

There's been some talk with the JSON Schema spec of suggesting (although likely not mandating) output and error formats, and we'd almost certainly go with a JSON Pointer for indicating where in the document the error occurred.  Possibly some sort of JSON Pointer over the in-memory structure rather than documents, so given
{
    "type": "object",
    "properties": {
        "foo": {"$ref": "#"},
        "bar": {"type": "integer"}
    }
}

with instance
{"foo": {"foo": {"bar": "whut"}}}
you'd have things like /properties/foo/$ref/properties/foo/$ref/properties/bar to show you that you found an error three levels down a recursive schema instead of just /properties/bar which only tells you where in the schema file the error happened.
This is a lot more portable and robust to things like minification than line numbers.

I just recently started using this, and I'd love more information in the error messages.


PATH include wrong $JAVA_HOME/bin when using jdkVersionOption: '1.11'

Question, Bug, or Feature? Type: Bug
Enter Task Name: MavenV3
Environment

Server - Azure Pipelines https://dev.azure.com/quarkus-ci/quarkus/_build/results?buildId=964
Agent - Hosted or Private: ?

Issue Description
see https://github.com/quarkusio/quarkus/issues/2012 for full background, but the TL;DR is that when you specify jdkVersionOption: '1.11', then you would expect that Runtime.exec("java") from some test gives you Java 11 - but it does not, you get a Java 8... :sob:
You should prepend $JAVA_HOME/bin to PATH, but AFTER you set that JAVA_HOME in function of the jdkVersionOption - do you see what I mean? (I could not find where the script which does this kind of thing is here.)

Hey @vorburger, are you saying that your maven tests are failing after you build using Java 11 because the tests use Java 8?

maven tests are failing after you build using Java 11 because the tests use Java 8?

No, a test launched an external java process, without specifying a directory, so it uses whatever java is first found on PATH. And that seems to always be a Java 8, even when you want when you specify jdkVersionOption: '1.11'.
It's not really strictly related to tests. You just need to set PATH correctly, as described above.
Does this make sense?

Hey @vorburger, this is as designed. The Maven task only sets up the java version for the maven process itself, not for the entire pipeline.
If you want to set the java version for an entire pipeline, then setting up the PATH as you described is a valid solution.


support custom mirror that does not use redirects

We experienced that the youtube-dl default mirror has not been reachable some time ago. By using our own S3 mirror we can make our system independent of the default mirror.
Modeling the re-direct with S3 is hard, I altered the existing download code to also support a 200 status code instead of a 302 status code, which is interpreted as the first request sending the file.

@przemyslawpluta Would it be possible for you to check out this pr?

@przemyslawpluta Would it be possible for you to check out this pr?

Hey @n1ru4l
This has been changed at<EMAIL_ADDRESS>and now the version is fetched directly from GitHub releases, so the redirect handling is no more necessary
context: https://github.com/przemyslawpluta/node-youtube-dl/pull/350  🙂

@Kikobeats That still does not allow setting up a custom mirror. We host our youtube-dl file on an S3 bucket YOUTUBE_DL_DOWNLOAD_HOST "https://xxxxxxxxxxx.s3.xxxxxxx.amazonaws.com/2021.02.10-youtube-dl".

I'm not following; it should enough providing YOUTUBE_DL_DOWNLOAD_HOST
https://github.com/przemyslawpluta/node-youtube-dl/blob/master/lib/downloader.js#L10
what am I missing?


Resolve the majority of pedantic clippy warnings

Resolve most warnings shown by cargo clippy -- -W clippy::pedantic:

clippy::single_match_else
clippy::if_not_else
clippy::must_use_candidate
clippy::module_name_repetitions

This leaves only the Panics/Errors documentation warnings as discussed in #28

Hey, thanks for getting the ball rolling here. I merged this as https://github.com/fspoettel/advent-of-code-rust/commit/d7af3dca9e52f9868b346d4058fe68248fb8950b after fixing some merge conflicts and marked you as co-author.


[DOCS] Add link to PanFLUte repo into Corona-tracker

⚠ IMPORTANT: Please fill out this template to give us as much information as possible to consider/implement this update.
Summary

Add link into Corona-tracker README that points to the PanFLUte repo.
Add it towards the top of the README in a prominent location with a clear description of what it is.
see:
https://github.com/COVID-19-electronic-health-system/PanFLUte
Motivation
**Why are we doing this? **
We want to let people visiting CoronaTracker that we have an open-source hardware based spirometer they can assemble.
**What use cases does it support? **
More tech to measure the extent of COVID-19 over time
What is the expected outcome?
People will be able to see the hardware repo and decide if building it would help their cause, whatever it may be.
Possible Alternatives

None, its a low cost solution.
Additional Context

see:
https://github.com/COVID-19-electronic-health-system/PanFLUte

@akilhylton Check this issue out.


Remove unused arguments from the methods get_bib_record and get_bib_r…

closes #914

Coverage remained the same at 76.954% when pulling 13dac99541d4e4ae25e1f4a5fdfad4a40f1c598e on 914-unused-arguments into b0d93837ea2eb682b000d4c2ba1551fbe515991e on alma.


Only use relative filepaths to session files

Requires this PR: https://github.com/vais-ral/CILViewer/pull/133

Ensures that the full path to the session folder is never saved. The app always works in the session folder, and uses relative paths to the files in the session folder. The only case where a full file path would be saved is if the setting to copy the image files to the session folder has not been selected (so the file lives outside of the session folder).

This means a session folder could be moved to another computer, and the app would still be able to load it. (not possible in current version)

In the case where a session has been loaded but an image file external to the folder has been deleted / cannot be loaded for some reason, an error dialogue appears which allows the user to load an image from another location. This allows the session to still be loaded rather than it crashing as it does currently.

This closes: https://github.com/TomographicImaging/iDVC/issues/13

I tried the app and get the following error
(dvc) PS C:\Users\ofn77899> idvc
QWindowsWindow::setGeometry: Unable to set geometry 2125x1987+0+0 (frame: 2125x1987+0+0) on QWidgetWindow/"VisualisationWidgetClassWindow" on "\\.\DISPLAY1". Resulting geometry: 2125x1987+1691+960 (frame: 2125x1987+1691+960) margins: 0, 0, 0, 0 minimum size: 54x54 MINMAXINFO maxSize=0,0 maxpos=0,0 mintrack=54,54 maxtrack=0,0)
[['C:/Users/ofn77899/Data/dvc/frame_000_f.npy'], []]
[['C:/Users/ofn77899/Data/dvc/frame_000_f.npy'], ['C:/Users/ofn77899/Data/dvc/frame_010_f.npy']]
The in file is:  ['C:/Users/ofn77899/Data/dvc/frame_000_f.npy']
The current working directory is:  C:\Users\ofn77899\DVC_Sessions\tmp_ai2jbla
Started worker
Spacing  (1.3088298169756785, 1.3088298169756785, 1.0)
Length of header:  96
array shape<PHONE_NUMBER>
target<PHONE_NUMBER>
Traceback (most recent call last):
  File "C:\Apps\miniconda3\envs\dvc\lib\site-packages\ccpi\viewer\utils\io.py", line 125, in <lambda>
    lambda: finish_fn(*finish_fn_args, **finish_fn_kwargs))
TypeError: save_image_info() got an unexpected keyword argument 'output_dir'

which I guess is due to https://github.com/vais-ral/CILViewer/pull/133


Fetch all customers

I'm trying to fetch all customers ,
but it returns only 10 instead of all
How to get all without out any limit
        $api = new Api($keyId, $secret);
        $relativeUrl = 'customers';
        $customers = $api->request->request('GET', $relativeUrl, []);

Environment:
PHP 5.6
composer 1.5.2
Laravel 5.4

Any suggestions or help ?

Ok got the solution
giving customers?count=100 returns upto 100 items


Upgrade to work with pandoc 3.x

The library requires pandoc 2.x, but the most recent version is 3.x
See also:
https://github.com/chrizel/things-to-org/issues/1

@metbril Thanks for the note. Can we easily upgrade to 3.x or are there some breaking changes?

Sorry. I wouldn't know. I temporarily downgraded to 2.x to use the downstream software and never tried with 3.x

where do we require pandoc 2.x?

See https://github.com/chrizel/things-to-org/issues/1 for a use case

I've examined the code in detail and found that the downstream script uses pandoc to convert from markdown to org. This has nothing to do with your library, as you stated. Sorry for the inconvenience. The library is and remains very useful though.

@metbril 👍
For context: the things-to-org script reads task["notes"] via things.py and then uses the pandoc Python library for further processing.
I'm glad you pointed out the things-to-org library, I added it to the "Used By" section of the README.


Validate Algorithm Step (6) has an error

https://w3c-ccg.github.io/vc-status-list-2021/#validate-algorithm

Let credentialIndex be the value of the statusListIndex property of the StatusList2021Credential.

However, StatusList2021Credential has no such property. Instead, it should be a reference to the StatusList2021Entry.

cc: @msporny


Not clear enough definitions make impression that LOH and 2nd Generation are the same things.

First of all, thank you for the article, it's quite informative.
Nevertheless, I find definitions like "Large objects are always allocated in generation 2." not clear enough. I think that the statements from the Fundamentals of garbage collection | Microsoft Docs article are more appropriate.

Newly allocated objects form a new generation of objects and are implicitly generation 0 collections. However, if they are large objects, they go on the large object heap (LOH), which is sometimes referred to as generation 3. Generation 3 is a physical generation that's logically collected as part of generation 2.

Objects on the large object heap (which is sometimes referred to as generation 3) are also collected in generation 2.

Could you please align these articles and make the definitions clearer.
Thanks!

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

ID: 819bcbc3-b2f7-b1eb-2e45-0f19c8da797c
Version Independent ID: ed840139-0331-77b1-5645-0cac6c16edcf
Content: Large object heap (LOH) on Windows
Content Source: docs/standard/garbage-collection/large-object-heap.md
Product: dotnet-fundamentals
GitHub Login: @gewarren
Microsoft Alias: gewarren

@rustamhajiyev Thank you for the feedback!

Thanks a lot!


Page turn/flip no longer working

Swiping the image past the threshold no longer causes the image to smoothly transition to the next.
Instead, it gets stuck, then the next image suddenly jerks into place.
Doesn't seem like a performance issue, as the performance monitor doesn't show any dips in FPS beyond the expected from touch overhead.
Using 2.2.26. It used to work fine, but uncertain what version that was.

Some problem!

I've replaced the library with this one instead:
https://github.com/antonKalinin/react-native-image-view
Works much better.


update to new pallets commit

@clangenb if this commit is fine for you, please also do the merge right away


Autoplay ?

How to autoplay ?

Any idea?

Simple as this:
const playback = L.trackplayback(...);
playback.start();


ci: add ubuntu noble numbat (24.04) to the CI matrix

i know this needs documenting and communicating and I do plan to do that but, this kind of change first needs to go into https://github.com/canonical/se-tooling-ci-common and then sync into here.

this is now resolved.


Breakpoints are all inactive on a asp.net core app created from the template

VS 15.7, vanilla ASP.NET Core app:

It's started working after terminating the debug session and restarting it again:

So, the scenario is:

create a project
hit F5 without opening any file
breakpoints are inactive
terminate the debug session
hit F5 again (having the controller file open)
debugging works

Spoke with @vasily-kirichenko and it seems to have been after dirtying the editor by executing the unused opens code fix in a debug session

Yes, sorry for this.


Exit gracefully if the Base OS is unknown

See #117 for more info.
2018-11-07 12:31:41,261 - DEBUG - tern - Starting...
2018-11-07 12:31:41,268 - DEBUG - container - Running command: docker ps
2018-11-07 12:31:41,344 - DEBUG - report - Setting up...
2018-11-07 12:31:45,410 - DEBUG - container - Running command: docker images docker.io/hello-world:latest
2018-11-07 12:31:45,478 - DEBUG - container - Running command: docker save docker.io/hello-world:latest
[sudo] password for nisha:
2018-11-07 12:31:51,953 - DEBUG - rootfs - Running command: sudo mount -o bind temp/f22789aafb439644ab76948424a0c1b64f328cecde82c95edad3813aaadbe077/contents temp/mergedir
2018-11-07 12:31:52,019 - WARNING - command_lib - No listing for key . Consider adding this listing to command_lib/base.yml.

2018-11-07 12:31:52,020 - WARNING - report - No shell listing in command library. Using default shell
Traceback (most recent call last):
  File "./tern", line 91, in <module>
    main(args)
  File "./tern", line 47, in main
    report.execute_docker_image(args)
  File "/home/nisha/tern3.7/tern/report/report.py", line 393, in execute_docker_image
    analyze_docker_image(full_image)
  File "/home/nisha/tern3.7/tern/report/report.py", line 176, in analyze_docker_image
    binary, default_shell=constants.shell)
KeyError: 'binary'

Docker's hello-world container image contains only one file which can be executed in a shell. It doesn't exist as a binary in command_lib/base.yml so it is a good image to test this bug against.

Error occurs because of a formatting syntax error. Nevertheless, there are some changes to reporting.


fix: binaryExpr type checks

fix the number and string checks to be bidirectional.
Fixes #685

Closing in favour of #715


Do something better than silently disabling tests when PyYAML is not found

On #7449 , @GMNGeoffrey  commented:

I really don't like silently disabling tests (and yes, warnings in the long list of warnings that no one reads counts as silently). This sort of thing is how you discover that some tests were never getting run.

What can we do that would be better?  Provide PyYAML in third_party/ ? Just require PyYAML when building any tests at all (and then, offer an option to opt out of the tests that need it and still be able to build the rest of tests, just not as default behavior?)

Sending to backlog due to the date of the issue - please reprioritize if needed.

Only one test still uses PyYAML after https://github.com/iree-org/iree/pull/18262, and I've just left that test disabled on CI. Closing this.


Fix Odin language server ols for windows hosts

Describe your changes
The installation of the Odin language server (ols) on win_x64 hosts is failing due to misconfiguration. Likely due to ols changing its packaging format.

The win_x64 release was incorrectly pointing to a .exe file. Changed to the correct .zip as can be verified here
Removed the unsupported win_x86 target

Checklist before requesting a review

[x] I have successfully tested installation of the package.
[x] I have successfully tested the package after installation.

The target should probably be updated for MacOS and Linux packages too. I'm currently getting a wget error on NixOS with the latest nightly build.

Duplicate of #6456

Duplicate of #6456

Apologies. When I tried looking for any existing issues I didn't come across this one. Clearly my searching ability needs approving. I think the original PR, which also resolves the issues with the other platforms, seems like the way to go.
Happy for this one to be closed.


[Go SDK] Don't depend on the go 1.21 standard lib yet

Beam is currently using go 1.20, but was accidentally depending on the go 1.21 standard libary versions of "maps" and "slices". This changes them to be the compatibility packages.
While beam advocates using the latest release (go 1.21) of Go when using it (which avoids problems when compiling those targets), the occasionally subtle dependency semantics allowed for that mismatch, leading to the compiler error. (IIRC this is not a problem past 1.21, which has better tooling awareness, but won't cause regressions for packages authored before those semantics...)
Going to cherry pick this into the 2.54.0 release to restore the compatibility with go 1.20.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

R: @zechenj18


subcommand indexing or "fzf.vim & fzf-lua difference"

Hello. I am a long time user of fzf.vim plug-in. Recently, I discovered this plug-in and decided to try.
I use "\" to find the correct command. It is much faster for me.
map  :FzfLua commands
fzf.vim introduced all commands on the "top" level, so I can easily find "BTags" or BLines", for example.
fzf-lua implements all commands as the FzfLua arguments, thus they are not listed in the total list of commands.
I do like fzf-lua decision, after all it is much easier to see all the commands it handles, but since my typing skills are not great, it takes me much longer to get to the right command.
Is it possible to list all second level commands in the "commands" list? I mean without forcing me to create aliases?
Thank you.

Try :FzfLua builtin, you’ll get a list of all the commands inside fzf so you can fuzzy search them.
If that doesn’t meet your needs you can always create copies of the commands with any name you want using :command!.

cnoreabbrev can also be an alternative.

Thank you very much. I understand those options. I hoped to find solution that will not force me to define "shortcuts". fzf-lua is not the only plug-in that use such technique.

Thank you very much. I understand those options. I hoped to find solution that will not force me to define "shortcuts". fzf-lua is not the only plug-in that use such technique.

We’ll it’s either you define it or I have to define it for everyone which then becomes another configuration option, given that most are ok with the current approach I think we can live with it :)

Once again - thank you.


Exception during saving model

Version of Archi

com.archimatetool.editor.buildid=202004150922
com.archimatetool.editor.versionid=4.7.0
Archi Plug-ins

Operating System

W10 Pro Version 1903
Expected Behaviour

Saving model
Actual Behaviour

Exception
Steps to Reproduce the Behaviour

Not known
Archi-Exception-log.txt

The model has somehow got corrupted. This can happen under some circumstances with coArchi.

I think I know the reason of the loop.
I created a grouping that encapsulate a link to a view and associated the grouping with a flow between to application component. In the encapsulated view, I detailed the flow (that is a complex interface between two systems). After putting elements in tat view I decided to put them inside the grouping. That reates a recursive loop I think.

I rolled back to last commit (only one hour lost).
I will try reproducing the error in a dummy model.

It could be caused by this https://github.com/archimatetool/archi-modelrepository-plugin/issues/136

I don't think so the view is a complete new view.
I think a created a circular reference in a grouping by using it like that :

created two application components in a new view
created a flow between the two components
added a grouping in the model
putted a link to a new other view inside the grouping
putted a relationship between the grouping and the flow
in the new other view : reused the two components and added more detailded components and behavior (the flow between the two components comes automatically)
putted the grouping in the model
resized the grouping and putted the elements of the view into (BANG), including the flow associated with itself (I'm able to create a circular relationship)

I tried to follow these steps but some are not clear to me. Could you simplify them?

I'll close this as I think it really is a result of archimatetool/archi-modelrepository-plugin#136


Make numpy and scipy depend on openblas

This configures numpy and scipy to use the openblas framework for their linear algebra operations, which seems preferable since it reduces the number and complexity of dependencies (a single library for BLAS and LAPACK). In addition openblas seems to be a lighter-weight dependency (in terms of build time, not sure about size) than ATLAS. In addition, on OSX, the numpy setup.py (and presumably scipy) will impurely check for the existence of the Accelerate framework, and if found, will pass it as an argument to clang, which will fail. It will not do this if there's a site.cfg telling it where to find openblas. This keeps things pure, and also makes it cross-platform.

@domenkozar @spwhitt @ttuegel

We already had that once and had to revert. I'll look into the reasons why.

History of numpy and openblas is here https://github.com/NixOS/nixpkgs/pull/6381 and here https://github.com/NixOS/nixpkgs/issues/6772.

Hmm, well that's unfortunate. I'm curious whether this is fixed with numpy 1.9.2. I'm building it right now and we'll see.

Ugh, OK well I'm getting a segfault in the numpy tests when building with openblas. Yikes. Googling around I don't see anyone else getting this issue but I might investigate further... I guess for now sticking with atlas is the best.
I suppose one thing that I could do is attempt to write a custom test function so that we can confirm that it works, as currently numpy has doCheck = false.

For the record, here's the output I'm getting with openblas:
>>> import numpy
>>> numpy.test('full', verbose=10)
Running unit tests for numpy
NumPy version 1.9.2
NumPy is installed in /nix/store/mp2y8m2sdr4fvrl850wsq1kxlf87bk53-python2.7-numpy-1.9.2/lib/python2.7/site-packages/numpy
Python version 2.7.10 (default, Jan 01 1970, 00:00:01) [GCC 4.2.1 Compatible Clang 3.6.0 (tags/RELEASE_360/final)]
nose version 1.3.4
nose.config: INFO: Ignoring files matching ['^\\.', '^_', '^setup\\.py$']
nose.config: INFO: Excluding tests matching ['f2py_ext', 'f2py_f90_ext', 'gen_ext', 'pyrex_ext', 'swig_ext']
test_complex (test_abc.ABC) ... ok
test_floats (test_abc.ABC) ... ok
test_int (test_abc.ABC) ... ok
test_uint (test_abc.ABC) ... ok
test_api.test_array_array ... ok
test_api.test_fastCopyAndTranspose ... ok
test_api.test_array_astype ... ok
test_api.test_copyto_fromscalar ... ok
test_api.test_copyto ... ok
test_api.test_copyto_permut ... ok
test_api.test_copy_order ... ok
test_api.test_contiguous_flags ... ok
test_api.test_broadcast_arrays ... ok
Basic test of array2string. ... ok
Test custom format function for each element in array. ... ok
This should only apply to 0-D arrays. See #1218. ... ok
test_arrayprint.TestArrayRepr.test_nan_inf ... ok
test_str (test_arrayprint.TestComplexArray) ... ok
test_arrayprint.TestPrintOptions.test_basic ... ok
test_arrayprint.TestPrintOptions.test_formatter ... ok
test_arrayprint.TestPrintOptions.test_formatter_reset ... ok
test_arrayprint.test_unicode_object_array ... ok
Ticket 844. ... ok
test_blasdot.test_blasdot_used ... ok
test_blasdot.test_dot_2args ... [1]    22018 segmentation fault  result/bin/np_test

Closing this in favor of #8045

Segfault is due to OpenBLAS using INTERFACE64=1, which is incompatible with reference BLAS and LAPACK. A fix is on the way (later today, probably).