build: expected targetOrResolver to be a function or class, but got '...'

Hi,
I'm trying to inject my middleware function as per your example, https://github.com/talyssonoc/node-api-boilerplate/blob/adce807fdd36ac86a44965d0327ec06b9e803225/src/interfaces/http/user/UsersController.js
But I get this error message": "build: expected targetOrResolver to be a function or class, but got. ....",
This is how I registered it
// Operations
container.register({
  getAllEmplois: asClass(GetAllEmplois),
  getEmploi: asClass(GetEmploi),
});

And in my controller:
get router() {
    const router = Router();

    router.get('/', inject('getAllEmplois'), this.all);
    router.get('/:id', inject('getEmploi'), this.one);

    return router;
  }

Am I missing something?
Thank you

That’s the old signature, please check the docs for inject usage

Ok, but if I do it this way:
router.get('/', inject(({ getAllEmplois }) => this.all));

all(req, res, next) {
    const { getAllEmplois } = req;
    const { SUCCESS, ERROR } = getAllEmplois.outputs;
...

getAllEmplois doesn't get resolved

Inject no longer sets anything on req. Use the factory function to return an inline handler and capture the dependency.


Missing tests for invalid_constructor_name diagnostic

There are no tests in the analyzer code base for the CompileTimeErrorCode.INVALID_CONSTRUCTOR_NAME diagnostic code.
There might be tests for it in the language or co19 tests, but it would still be good to have tests for it in the analyzer code base (or to remove it if it can't actually be generated).

https://dart-review.googlesource.com/c/sdk/+/226160


Option for putting generated file at the same level as target folder

Could it be possible to have the generated file like this?
models\
  course.dart\
  user.dart\
models.dart\ <- The generated file which exports models/course.dart and models/user.dart

This structure can make the import paths look better.
import '../models.dart';
// or this if I only want the course model
import '../models/course.dart'

Btw this extension is really helpful to organise the files, especially in a big project. Nice work!

Hi @drinkmorecola! Thanks for opening an issue 👋🏼!
There's the option GDBF: Current with subfolders. I have now seen that it is not documented in the readme!
Could you try it and see if it helps?
Thanks for the support!

I'm closing the issue now as I'm not receiving any response! The option I have proposed it should produce the output you are looking for.
If there's any issue, reopen the issue!

Sorry for late reply. That result is a bit different from what I’m expecting. The generated file is in the target folder (E.g. the path of generated file in the example is ../models/models.dart), but I want the path to be ../models.dart, which means the models.dart is out of the models folder. They should be at the same level.

Just something that I do not understand from your example: course.dart and user.dart are files or folders? The / at the end makes it confusing as normally folders are ended with the dash.

🙈they are files. Here’s the correct example.
models/
  course.dart
  user.dart
models.dart

So models.dart is outside the models/ folder?

Yes that’s what I mean.

Does the models folder have a parent component?
If so, you can do:

In your settings, turn to true the dartBarrelFileGenerator.promptName. This will display an input when you are generating a barrel file.
Right click the parent folder and generate for the current with subfolders.
An input will show and ask you for a name. Enter the name you want (models in this case).

It should generate a barrel file outside the models/ folder with the models.dart name.
Would that help you?

If there are more than one folder, for example
models/
  course.dart
  user.dart
services/
  service.dart

The generated models.dart will export files under services  too, which is unexpected.

Yes, it will. I'm planning on also adding a feature that would allow you to exclude folders. As of now this is not possible.
I'm sorry but what you are asking for it is really complex to do, and I'm currently not with enough time to do it. I also have the feeling that this is a really ad-hoc case, so I would need to find a solution for many specific cases, therefore implementing complexity in the code. 🙏🏼
Nonetheless, I would not recommend having barrel files outside the folder from they should be exported, as this breaks with the idea of why they exist.
I will close the issue but I will keep the idea in mind. Maybe if I have time enough I will try to find a solution...

Really appreciate your patience!


Update the code owners for BOM and other SDKs

/check-enforcer override

Thank you for the review.


Made sure the .config.bundle is initialized correctly and thus a corr…

…ect URL is generated when a catalog item  is added from the composer

LGTM - merging


Cannot restore geometry in showEvent

First of all let me say that this might or might not be a bug, let me explain.
I used to save the geometry of my MainWindow in hideEvent and restore it (setGeometry) in showEvent with FramelessHelper and that used to work flawlessly.
When using qwindowkit however, restoring the geometry in showEvent doesnt work anymore. I have to set it again, after mainWindow->show(); is called.
Before with FramelessHelper:
class MainWindow
{
	void hideEvent(QHideEvent* event)
	{
		// save geometry
	}

	void showEvent(QShowEvent* event)
	{
		// restore geometry
	}
};

MainWindow* main = new MainWindow();
main->show();

Now with qwindowkit:
class MainWindow
{
	void hideEvent(QHideEvent* event)
	{
		// save geometry
	}

	void showEvent(QShowEvent* event)
	{
		// restore geometry - DOESNT WORK
	}
};

MainWindow* main = new MainWindow();
main->show();
main->setGeometry(geo);  // HAVE TO DO IT HERE NOW

OK, let me investigate.

OK, let me investigate.

Restoring geometry was already tricky in the original Frameless, I remember we discussed a couple bugs before you got it right

Previously we would center the window before show, automatically. So you can't move the window in showEvent. Now this behavior is changed. QWK won't move the window automatically now, you may try latest code to see whether your issue is solved or not.

yeah its fixedm thanks
closing


[BUG]: 最新版本 1.1.28 安装不可用

开发环境：nuxt3.6.5
问题：

目前解决办法：回退至 1.1.27 正常

npm 上发布的版本缺少dist目录

I am having same problem.

I had the same issue and found out that the 1.1.28 is missing a dist folder with the built out dependencies causing the Failed to resolve entry for package "vue-codemirror6". Downgraded to 1.1.27 for now until it is fixed!

Had the same problem...please fix thank you

Sorry for the inconvenience.
I've updated it now.


Google Apps Scripts (backend)

You can use the free tier of google apps scripts to fully power a jwt rest api using google sheets. You can also connect to a databasing using a JDBC connection. It provides POST and GET support. This example should use google sheets as a data storage
Google sheets size limits per https://support.google.com/drive/answer/37603?hl=en. You can only have 5 million cells per sheet (not per document). So if you had 2 columns you could have 2.5 million rows. There are no limits to writing and reading from google sheets using their direct api, external api calls do count against the limit. There is also a cache available.
It probably should be started off from https://github.com/labnol/apps-script-starter
The quota limits are here. https://developers.google.com/apps-script/guides/services/quotas
It would work fine on mobile, but b/c of cors will need to use a proxy such as php proxy or host a nodejs one through https://glitch.com/
If done right shouldn't need to use a google cloud api key
End goal

using rest api with POST and GET methods from google scripts
google sheet as a data store
jwt authorization
match realworld app spec
cors with a php local proxy

may have to do a format such as url?method=user to match spec with url?method= being the base url

use free google scripts tier

Here's my source code WIP https://github.com/lastlink/realworld-apps-script/tree/restapi.

I went through all of the steps written in the readme file. when I tried to deploy it throw an error.  please help me to figure out it.
D:\Appscript\realworld-apps-script-githubRepo\realworld-apps-script-master>npm run deploy

<EMAIL_ADDRESS>deploy D:\Appscript\realworld-apps-script-githubRepo\realworld-apps-script-master
npm run build && npm run upload

<EMAIL_ADDRESS>build D:\Appscript\realworld-apps-script-githubRepo\realworld-apps-script-master
webpack

Hash: 46e7353f68dc3a5e49af
Version: webpack 4.44.2
Time: 25029ms
Built at: 28/12/2020 3:24:48 pm
Asset       Size  Chunks             Chunk Names
AES.js     13 KiB          [emitted]
Cipher.js   1.41 KiB          [emitted]
DES.js   20.9 KiB          [emitted]
Rabbit.js   13.2 KiB          [emitted]
TripleDES.js   20.9 KiB          [emitted]
appsscript.json  505 bytes          [emitted]
code-0.0.3.js   27.6 KiB       0  [emitted]  main
Entrypoint main = code-0.0.3.js
[0] ./src/index.js 114 bytes {0} [built]
[1] (webpack)/buildin/global.js 472 bytes {0} [built]
[2] ./src/server/webapp.js 6.76 KiB {0} [built]
[3] ./src/server/mock.js 794 bytes {0} [built]
[4] ./src/server/config.js 146 bytes {0} [built]
[5] ./src/server/articles/index.js 2.1 KiB {0} [built] [1 error]
[6] ./src/server/utils.js 1.21 KiB {0} [built] [1 warning]
[7] ./src/server/auth/index.js 8.66 KiB {0} [built] [1 error]
WARNING in ./src/server/utils.js
Module Warning (from ./node_modules/eslint-loader/dist/cjs.js):
D:\Appscript\realworld-apps-script-githubRepo\realworld-apps-script-master\src\server\utils.js
23:20  warning  Unexpected unnamed function  func-names
26:22  warning  Unexpected unnamed function  func-names
41:24  warning  Unexpected unnamed function  func-names
✖ 3 problems (0 errors, 3 warnings)
@ ./src/server/articles/index.js 2:0-39 16:14-25
@ ./src/server/webapp.js
@ ./src/index.js
ERROR in ./src/server/articles/index.js
Module Error (from ./node_modules/eslint-loader/dist/cjs.js):
D:\Appscript\realworld-apps-script-githubRepo\realworld-apps-script-master\src\server\articles\index.js
1:20  error  Unable to resolve path to module '../config'  import/no-unresolved
1:20  error  Missing file extension for "../config"        import/extensions
✖ 2 problems (2 errors, 0 warnings)
@ ./src/server/webapp.js 2:0-40 63:17-31 78:15-29 105:15-29 112:15-29 119:15-29 127:15-29 135:15-29 141:15-29 179:17-31 183:17-31 198:15-29 204:15-29
@ ./src/index.js
ERROR in ./src/server/auth/index.js
Module Error (from ./node_modules/eslint-loader/dist/cjs.js):
D:\Appscript\realworld-apps-script-githubRepo\realworld-apps-script-master\src\server\auth\index.js
2:20  error  Unable to resolve path to module '../config'  import/no-unresolved
2:20  error  Missing file extension for "../config"        import/extensions
✖ 2 problems (2 errors, 0 warnings)
@ ./src/server/webapp.js 3:0-33 46:28-39 51:17-28 57:17-28 86:47-58 94:17-28 192:15-26
@ ./src/index.js
npm ERR! code ELIFECYCLE
npm ERR! errno 2
npm ERR<EMAIL_ADDRESS>build: webpack
npm ERR! Exit status 2
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      C:\Users\Hp\AppData\Roaming\npm-cache_logs\2020-12-28T09_54_49_121Z-debug.log
npm ERR! code ELIFECYCLE
npm ERR! errno 2
npm ERR<EMAIL_ADDRESS>deploy: npm run build && npm run upload
npm ERR! Exit status 2
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>deploy script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      C:\Users\Hp\AppData\Roaming\npm-cache_logs\2020-12-28T09_54_49_352Z-debug.log

Hello, we moved the addition of new implementations to this website : https://codebase.show/projects/realworld
Please submit your implementation there.

@nitingupta-123 I would need more details on what you did.
Generally you need to run npm install, copy the config file, and run npx clasp login before you can deploy. It seems like you just skipped these steps.


[Question] Log executed query without parameters

Hello. I would like to log and monitor queries executed by our platform.
I'm going to use CustomHandlers, which seems to have almost all information which i need.
I can get executed query from request.Content, but i wonder - is there any way to get that query without parameters values?
(We can't log full query with values due to privacy concern).

@bchong95 any suggestions?

Another thing is that there are a lot useful information like (operationName, adressResolved, whole DocumentServiceRequest, which are internal and i can't them without using reflection. There is possibility that you could make them public? :)

You can parse the Content, since it's a Json Object with queryText and paramaters property. log only the queryText.

Yes, but the content for queries generated from linq or passed as pure string has query in whole version - without parameters. I don't want to force usage of SDK in my whole platform to API .WithParameters(). And there is always chance that someone put pure string query anyway.

It sounds like you want to emit an obfuscated query. In which case I recommend parsing the query with this parser:
https://github.com/Azure/azure-cosmos-dotnet-v3/blob/master/Microsoft.Azure.Cosmos/src/Query/Core/Parser/SqlQueryParser.cs
and use this obfuscator:
https://github.com/Azure/azure-cosmos-dotnet-v3/blob/master/Microsoft.Azure.Cosmos/src/SqlObjects/Visitors/SqlObjectObfuscator.cs
You are going to need access to internals to get this to work.

Please reopen the issue if there are further questions.


[tests] add tests for Border Routing Counters

This PR adds tests for verifying the functionality of the D-BUS API GetBorderRoutingCounters.

The CI should pass after https://github.com/openthread/ot-br-posix/pull/1602 is merged.


A question of sinc filter

Hello author, I have a question. SR is a process of denoising and deblurring so that we can add blur and noise to HR image. But the ringing and overshoot artifacts usually appear on the SR result image（HR, not LR）, the method of adding sinc filter on LR is really effective??

@yulei1234
We want to learn a mapping: LR (with ringing and overshoot artifacts) ----> SR (want to be perfect w/o ringing artifacts). So, we want to simulate LR containing ringing artifacts.   For the synthesis, you can first apply since filter followed by downsampling, or you can downsampling followed by since filter. Both are reasonable. For example, images from cell phones usually have ringing artifacts. If we want to upsample those images, the ringing artifacts are in the LR images.


DNM test: avoid line wrap on fedora console

What this PR does / why we need it:
Apparently, stty columns 500 is not effective on fedora console.
So for fedora console only, setterm -linewrap off is used.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Fixes bug introduced in PR https://github.com/kubevirt/kubevirt/pull/3882
Release note:

Some general question:

These new commands seem to be lower level ones which stty was supposed to wrap, no?
Can these be used instead of the stty?
I noticed some sources mention that using stty requires an exec bash for the changes to take affect.

The new commands seem too mystical to me:

Why unset COLUMNS?
What tput rman does exactly?

Regardless, you can simplify a bit the snd/expect:
unset COLUMNS; setterm -linewrap off && tput rmam\n

Currently on this PR is just experiments to find something that is actually working
On fedora the line is sometimes still wrapped.

Currently on this PR is just experiments to find something that is actually working
On fedora the line is sometimes still wrapped.

Ohh. Can you please marked it as draft then?

i think that WIP would block merge automatically, not sure if DNM as well btw

/assign @jean-edouard

/approve

Fixes: #3964


Update README.md and Dockerfile.stac pip install issue

this updates the readme so the code box can be copied and run in the terminal without error due to the prompt symbol included in the code box $
it also addresses the issues listed in https://github.com/developmentseed/eoAPI/issues/49

🙏 thanks @rbavery
Tests are failing for other reason so I'll merge this PR and try to fix it in another PR


Bid Viewability module documentation

Refer PrebidJS PR: https://github.com/prebid/Prebid.js/pull/6206
Also added an API fix in documentation of Instream tracking module

Thank you @bretg !
Waiting for code PR to merge.

Hello @bretg ,
The code PR is merged , can you please merge this PR too?

@bretg, gentle reminder to merge the PR

Hello @bretg, gentle reminder to merge the PR

@pm-harshad-mane - we don't merge docs PRs until after the code is released -- not after it's merged. Because it's live on the site immediately once merged.
For the record, there's a website rotation and the person on-duty should be merging PRs that are ready. It's not my week, but I'll merge this for you.


ruby-modules: check tzinfo version

Motivation for this change
💎 💎 💎
tzinfo 0.3.xx does not contain the file
lib/tzinfo/zoneinfo_data_source.rb
it is only included in the 1.x branch.
closes #24080
Things done

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

[x] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Looks good to me!


tag sas

Hi Anna @annatisch and Rakshith @rakshith91
I will merge this pr to stg73 for now, you can comment on this pr if you have any concern! https://github.com/Azure/azure-sdk-for-python/pull/12272


Remove darwinFlags in rust module

Remove darwinFlags to fix linking error when compiling no_std crate. Closing #1355
Only tested with Developer Frameworks already installed in the system.
I am not sure if this change will break those who only rely on Developer Frameworks in Nixpkgs and don't have Developer Frameworks installed outside Nix.

What if you need to compile against a framework library on macos?

@domenkozar, I tested this out in a fresh VM.
Nix already adds the -iframework CFLAG for us.
The -L framework rust flag doesn't seem necessary. I tested a bunch of random OSS projects that link to frameworks — they compile without issues. Is this around from the pre-1.0 era when we didn't have setup hooks?


Revert to cramtools 2.1

I am using the cram compression option in bcbio for archival of the alignment. Using cramtools 2.1 this goes smoothly; however, bcbio now seems to install cramtools 3.0-b47, which fails with Unknown read or base category: i error when running the bcbio builtin conversion. This cramtools 3 seems to have other issues as well, for instance here https://github.com/enasequence/cramtools/issues/48 where it forces one to retain read names even if it is specified to omit them. The cramtools people are working on it, but the issue is still open. I therefore suggest to revert to the working cramtools 2.1 version that is in homebrew until all issues are fixed.

Sven-Eric;
Thanks for reporting this and sorry about the issues. Do you think it's worth trying out the latest cramtools 3.0 before reverting? I'm a bit nervous about moving backwards with CRAMs continued evolution and there have been a few changes since b47, and this could also be due to bcbio being out of date. I could bump this to the latest and update bcbio CRAM support first to see if that resolves it. Would that work for you?

Sure, let's try that first. I'm a little doubtful since cramtools development doesn't seem to be overly active, but of course we should try to go with the current major release. I'll can test tomorrow.

Sven-Eric;
Thanks much. I bumped to the latest version of cramtools if you update. Fingers crossed. I also did some cleanup of bcbio and where we use cramtools. If this is still an issue we could try switching over to samtools view for BAM to CRAM conversion. The only downside is that I don't believe samtools has a way to do quality score binning so we'd lose that functionality to help reduce file sizes. Thanks for looking at this.

So the new cramtools version seems to require Java 8 (Java error is unsupported major.minor version 52.0). Bcbio comes with Java 7. I'll stick with cramtools 2x for now but I'm fine with monkey patching that in myself if need be.

I'd recommend going with samtools view, though - it will be better supported in the future and also the quality binning will be added later on by the devs, from what I heard.

Sven-Eric;
Sorry about the continued problems. I agree, I'd rather have stable CRAM output we can trust rather than quality binning for size reduction. Hopefully that'll come to samtools soon as well and we can add it in. I pushed this fix so the latest development should hopefully work cleanly on your example. Let me know if you run into anything else with this.

Great; there may be a chance to add binning back in right now by piping from bamUtil squeeze into samtools. That would also allow replacing read names with integers to go down to 20% size or so. But that also depends on the additional effort for maintaining that new binary, of course. See here: http://genome.sph.umich.edu/wiki/BamUtil:_squeeze At least it is actively maintained: https://github.com/statgen/bamUtil


Bower dependencies

Hi,
Is it necessary to add "bootstrap" as "dependencies" inside bower.json?
I'm using "bootstrap-sass", but because I have also added yamm3 inside bower I get also "bootstrap" (less version) included inside my bower_components folder. It just adds extra weight on bower install which is done inside CI (jenkins) after each build.
Also it is confusing to see "bootstrap" and "bootstrap-sass", because if you are new in the project you are not sure which is in use (unless search, but then you are confused that why that "bootstrap" is installed)
Also when you think about how yamm3 is used (at least from my perspective)

You choose your CSS framework what you start using in your project i.e. Bootstrap (less or sass or even plain css)
You then search additional components around it like yamm3, but in this point you already have installed your framework -> it is not needed to add "bootstrap" as dependency for yamm3.

Br,
Markku

This has been brought up here before. Until the issue is fixed, you can add the following to your .bowerrc
 {
  "directory": "bower_components",
  "ignoredDependencies": [
    "bootstrap"
  ]
}

Removed bootstrap from bower. I'll release v1.3 with latest changes. Sorry for the sooo lazy answer


The link on github is not working

@muhammederdinc Thank you for reporting this.
This entire repo will be archived soon as a bunch of plugins are moving to a mono repo : https://github.com/csstools/postcss-plugins
Did you have any specific questions or issues to which you did not find the answer in the readme? Then we can update where needed.

There was no question that I could not find the answer to in readme, I wanted to report when the link did not work :)


Add Shutter Glasses

Looks like we are using two different prettier configs. I just pushed my .prettierrc to master for future work.

thanks!
On Mon, Jul 29, 2019 at 9:35 PM bryan donovan<EMAIL_ADDRESS>wrote:

Looks like we are using two different prettier configs. I just pushed my
.prettierrc to master for future work.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bpdo/yogapose/pull/14?email_source=notifications&email_token=AAK5U6RK6CVMYNJDXF2OQG3QB6LFNA5CNFSM4IHVYZAKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3CPHEA#issuecomment-516223888,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAK5U6W2U7YA3SYDYM4TPQTQB6LFNANCNFSM4IHVYZAA
.


Code Quality: Generative resource management and refactoring

Resolved / Related Issues
To prevent extra work, all changes to the Files codebase must link to an approved issue marked as Ready to build. Please insert the issue number following the hashtag with the issue number that this Pull Request resolves.

From #15504

Steps used to test these changes
Stability is a top priority for Files and all changes are required to go through testing before being merged into the repo. Please include a list of steps that you used to test this PR.

Generated Strings properties from keys in primary Resources file
Compatible import from .resw and .json file

Is this still on-development?
If you are ok, I’d like to review.

Could you duplicate ResourceHelpers in Strings.MarkupExtension.cs as partial class of Strings?
Sample impl:
[MarkupExtensionReturnType(ReturnType = typeof(string))]
public partial class Strings : MarkupExtension
{
    public string Name { get; set; }

    protected override object ProvideValue()
        => GetType().GetProperty(Name)?.GetValue(this);
}

Sample usage (not for now):
<TextBlock Text="{helpers:Strings Name=Allow}" />

<TextBlock Text="{helpers:Strings Name=Allow}" />

I would use instead of Name=, Key=
  <TextBlock Text="{helpers:Strings Key=Allow}" />

[MarkupExtensionReturnType(ReturnType = typeof(string))]
public partial class Strings : MarkupExtension
{
    public string Name { get; set; }

    protected override object ProvideValue()
        => GetType().GetProperty(Name)?.GetValue(this);
}

This way it would be usable, you would just have to introduce a rule into the generator that key names e.g. 'Key' or 'Name' cannot be used because they are used internally. Evaluating with an analyzer would be good.

Make use of SyntaxKind for C# keywords

[x] Finished

Make use of Constants class for raw strings in generated file such as <auto-generated/> and license header.

[x] Finished

Now need testing

Thank you


IR Benchmarking

Pull Request
What does this PR do?
Adds replication steps for IR Benchmarking paper, "Moving Beyond Downstream Task Accuracy for Information Retrieval Benchmarking"
Description
Adds replication steps for IR Benchmarking paper, "Moving Beyond Downstream Task Accuracy for Information Retrieval Benchmarking"
Request Review
Requesting review from: Avi Sil, Martin Franz, and Keshav Santhanam
Versioning

[ ] Have you updated the VERSION?
[X] Or does this PR not change the primeqa package or was not into master?

Releases
After Merging
If merging into master and VERSION was updated, after this PR is merged:

[ ] Create a release from the master with version equal to VERSION
[X] Not merging to master or VERSION not updated

Checklist
Review the following and mark as completed:

[X] Tag an issue or issues this PR addresses.
[X] Added description of changes proposed.
[X] Review requested as appropriate.
[X] Version bumped as appropriate.
[X] New classes, methods, and functions documented.
[X] Documentation for modified code is updated.
[X] Built documentation to confirm it renders as expected (see here).
[X] Code cleaned up and commented out code removed.
[X] Tests added to ensure all functionalities tested at >= 60% unit test coverage (see here).
[X] Code cleaned up and commented out code removed.
[X] Release created as needed after merging.

Left a couple comments to remove comments :)

Thank you for checking the code! I removed the comments that were left


Populate returnURL on Forbidden mapping for cookie auth

This way a forbidden page could prompt for a new login and redirect back to the previous resource, in much the same way a login/unauthorized page does now.

Adding scenario - if a user needs extra auth for sensitive operations you could, for example, check for a 2FA claim, get forbidden and then prompt for a login, redirecting back once complete.

dc6e916bd465628c3e6441925cdb16239a0b82b9


feat: select release and sub release run IDs

This PR include minor aesthetic changes to facilitate the selection of the run ID that we want to explore or use to compare.
The list of available metrics was getting quite long, so I thought that selecting the major release ID first and then the specific run for that release will make it easier.
This is how it looks like, all tabs have been adapted.

Instead of having to go through the full list of

The code look OK, however something seemingly off. When I deployed on my local machine, the exploration function didn't work for pre-ETL runs. It does work on the current production one.
I got this error message:
Error executing job with overrides: []
Traceback (most recent call last):
  File "/Users/dsuveges/repositories/ot-release-metrics/app.py", line 72, in main
    data.query('variable == "associationsIndirectTotalCount"')[
IndexError: index 0 is out of bounds for axis 0 with size 0

Set the environment variable HYDRA_FULL_ERROR=1 for a complete stack trace.
Error executing job with overrides: []
Traceback (most recent call last):
  File "/Users/dsuveges/repositories/ot-release-metrics/app.py", line 72, in main
    data.query('variable == "associationsIndirectTotalCount"')[
IndexError: index 0 is out of bounds for axis 0 with size 0

The UI only shows the evidence count:

The production shows this on an other pre-ETL dataset:

The selected_run variable will contain the release, not the actual run. That's why it always assumes it is a post-ETL run, because the condition if "pre" not in selected_run is always true.
I added some log:
[2024-02-15 12:01:24,880][__main__][INFO] - Selected run: ['24.03']
[2024-02-15 12:01:24,882][__main__][INFO] - We are in the post ETL run space: ['24.03']

An other problem I found is that the current implementation won't allow to compare two ETL run of the same release, however I think it is of interest.

@DSuveges Yes, the specific run for a a release is an array that contains all IDs relevant to the releases array.


Add a note about a VPN adapter

Add a note about a VPN adapter as follows:
Note A VPN adapter should set *NdisDeviceType to 0. If it set *NdisDeviceType to NDIS_DEVICE_TYPE_ENDPOINT (1), a UWP is prevented from sending a packet to the VPN adapter.

Hi @aktsuda, thanks for the PR. I made some minor editorial polishes, but otherwise the change looks good so I'll get this merged for you. It should be live later today, Redmond time. Thanks!


1440 x 2560 screen issue

When running in Portrait on an LG G3 the menu expands to around 150% of the screen width when DRAWERTYPE_IMAGE is set and only about 3% when no image is set.
It works without issue in Landscape.

It is a currently bug with xxxhdpi devices.
It will be fixed with the next release

With the last commit the fix is setted, please into the dev project.


No matching distribution found for casparser-isin

=> ERROR [stage-0  7/13] RUN pip3 install -U casparser-isin                                                          1.8s

Facing this issue while trying to install it in docker container

[stage-0  7/13] RUN pip3 install -U casparser-isin:
#10 1.432 Collecting casparser-isin
#10 1.658   Could not find a version that satisfies the requirement casparser-isin (from versions: )
#10 1.660 No matching distribution found for casparser-isin

The modules has a dependency on python versions >=3.8 . Are you using an older version, by any chance?

Hi @codereverser ,
Yes, when I RUN apt-get install -y python3 python3-pip then docker is installing Python 3.7.

There is no issue with the package apart from it do you have any idea how to install python 3.8?
I searched by didn't find any working solution for it 😔

Which base image are you using for your container?
Here's a stripped-down version of a working Dockerfile that I'm using
FROM python:3.8-slim
RUN python3 -m venv /venv
RUN /venv/bin/pip install -U pip wheel setuptools
RUN /venv/bin/pip install -U casparser-isin
CMD /venv/bin/casparser-isin --version

Actually, I am using FROM php:7.4-fpm and need to use it within the PHP container.

Oh. that is tricky. You could try manually installing  python3.8 by including the build instructions from https://github.com/docker-library/python/blob/master/3.8/buster/slim/Dockerfile as both python:3.8-slim and php:7.4-fpm use the same base image.
Also here's an experimental Dockerfile that may do the same trick.. (not tested well; may have unintended bugs)
FROM php:7.4-fpm-buster
# Copy files from python-3.8 image
COPY --from=python:3.8-slim-buster /usr/local /usr/local
COPY --from=python:3.8-slim-buster /usr/lib   /usr/lib
COPY --from=python:3.8-slim-buster /lib64 /lib64
COPY --from=python:3.8-slim-buster /lib /lib
RUN ldconfig
# Setup virtual env
RUN python3 -m venv /venv
RUN /venv/bin/pip install -U pip wheel setuptools
RUN /venv/bin/pip install -U casparser-isin
CMD /venv/bin/casparser-isin --version

Hi @codereverser
Thanks for the help! BTW it didn't work for me :-).
I am closing this issue as I think it might be this is case.

also, I loved your both packages (casparser & casparser-isin), and also if possible please keep them updated.
And also can you please provide support for the parsing of the CAMS portfolio (MF & Stocks)?

Hey no worries! I tried to port both to python3.6 but they (casparser esp) extensively use the walrus operator for regex match and validation. Might not be worth the effort with more systems adopting python3.8 as base version.
re; the MF & Stocks portfolio, as far as I am aware CAMS provide only MF portfolio . Do you mean CSDL/NSDL statements? If not, can you please provide a link how to generate the statement from CAMS?

Ohh, Yes my bad. I mean CDSL Consolidated Account Statement (CAS)

re; the MF & Stocks portfolio, as far as I am aware CAMS provides only MF portfolio. Do you mean CSDL/NSDL statements? If not, can you please provide a link to how to generate the statement from CAMS?

Are you planning for the same?

Yeah . that feature is in the pipeline. Don't have enough samples with variations to start the task.


fixed github workflows; minor changes to satisfy pylint & flake8

18.04 workflow runners are deprecated, switched to 20.04.
I've also tried to resolve the errors that pylint and flake8 threw. Please check, and feel free to add/change anything as needed.
Additionally fixed some typos in communities/as1764.txt.

found some more errors... closeing for now. sorry for the trouble, will update in another pull..


Update README.md

Adding simple numbers to make the install documentation easier

Hi @franciosi
Thanks for your contribution!
However, I don't see how adding numbers in the readme makes difference in understanding it better..
I encourage you to add a contribution in the code :)

Hi @franciosi
Thanks for your contribution!
However, I don't see how adding numbers in the readme makes difference in understanding it better..
I encourage you to add a contribution in the code :)

Hi @danielsagi
OK, it was just a simple improve. This is how normal we do with documentation, like https://www.freebsd.org/doc/handbook/. About code, yes, I plan to do it, as soon as I have more time. Very much involved with FOSS documentation and translations for now.

Hi @danielsagi
OK, it was just a simple improve. This is how normal we do with documentation, like https://www.freebsd.org/doc/handbook/. About code, yes, I plan to do it, as soon as I have more time. Very much involved with FOSS documentation and translations for now.


provide prerequisites setup example for ubuntu

make setup a bit quicker
Test
validated on a fresh ubuntu22.04 instance

FYI: Currently only Ubuntu 18.04 and 20.04 are officially supported. Using anything else isn't recommended for production use so you're on your own if something's going wrong. 20.04 is recommended.

FYI: Currently only Ubuntu 18.04 and 20.04 are officially supported

ty, switched to 20.04

@Fraser999 , be aware that for some reason, authorship was not kept here by the merge.


Add tooltip to TN tertiary screenshot

Updates the TN tertiary (hospitalization) screenshot to include the tooltip for the latest day.
The url field in YAML is the url to the tableau embed in the current url.
Here's what the new screenshot looks like:

@schwartzadev that's great! 🎉 One small request: could you remove the URL field? For now we're trying to keep all the URLs in the spreadsheet, and @muamichali has added that URL there. Once we do the big screenshot config switcheroo, the URLs will all be moved over to yaml.

Done! See cf09631


fixed typo in service

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/java-quickstart-maven-webapps.md
:white_check_mark:Succeeded
View (azure-java-stable)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Installation stalled in apt-key adv --keyserver keys.gnupg.net --recv-keys EAD883AF

Hello,
I had been trying using dokku-alt and the installation worked without problems. I launched a new server today and the installation stops in apt-key adv --keyserver keys.gnupg.net --recv-keys EAD883AF I don't know how to debug this. Maybe the key server is down?
apt-key adv --keyserver keys.gnupg.net --recv-keys EAD883AF
Executing: gpg --ignore-time-conflict --no-options --no-default-keyring --homedir /tmp/tmp.yT2Brqr0lW --no-auto-check-trustdb --trust-model always --keyring /etc/apt/trusted.gpg --primary-keyring /etc/apt/trusted.gpg --keyserver keys.gnupg.net --recv-keys EAD883AF
gpg: requesting key EAD883AF from hkp server keys.gnupg.net
gpg: keyserver timed out
gpg: keyserver receive failed: keyserver error

My machine has a normal connection and I don't know what else to try related to the problem.
Thanks!

Never mind, this was a firewall issue on my side. Sorry!


[Device]: Smart Light & Motion Sensor

Manufacturer
_TZE200_3towulqd
Model
TS0601
Battery Type
CR2450
Battery Quantity
1

This has been added, thanks.


Replace pull-right (broken) with pull-xs-right

twbs/bootstrap#18340 removed .pull-left and .pull-right
since they're redundant with .pull-xs-left and .pull-xs-right.
This solves the styling bug explained in
pydanny/cookiecutter-django#452.

Thanks!


after list generated, collapses once urls added

Could not repro, maybe newer gradio fixed:

tried several times:


92 replace exchange rate and current tokenworth with current gains

#92

i dont see a test for true,true - false ,false

Added


Main screen improvements

Screens are here
https://zpl.io/2pMmPPj
https://zpl.io/VqWLnGl
https://zpl.io/2v06g0Q
https://zpl.io/aNzDkzk
I suggest we show only 5 last transactions on the main screen, for more transactions user can use separate screen. Transaction search, payment proof and filters (when we will implement that) will be on transactions screen as well.

checked


Revert "[HUDI-4741] hotfix to avoid partial failover cause restored s…

…ubtask timeout (#6796)"
This reverts commit e222693d87d48416670ca14c6f7fd69307432786.
Change Logs
revert c
Impact
Reverts before we do more tests and prove the correctness.
Risk level (write none, low medium or high below)
none
Documentation Update
Describe any necessary documentation update if there is any new feature, config, or user-facing change

The config description must be updated if new configs are added or the default value of the configs are changed
Any new feature or user-facing change requires updating the Hudi website. Please create a Jira ticket, attach the
ticket number here and follow the instruction to make
changes to the website.

Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

+1


using product of events and preconditions for decision making

I agree, Robert pointed out that it should not give only the newest event of a certain content a chance, in his example: imagine two preconditions with +40 vs. +4000 interval. it should also be able to match the one that was longer ago if the happening of the event was remembered.


Test base update

Codecov Report

Merging #15 (1c4252c) into main (94debeb) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #15   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines          106       106
  Branches        10        10
=========================================
  Hits           106       106

Continue to review full report at Codecov.

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


Changes to All Clubs page

The all clubs page corrupted when I minimized the screen so I adjusted the display (The club cards were overflowing when you add more than 3 clubs, not properly going onto the next line)
All the club info was hardcoded into the AllClubsPage.js file, but this PR puts all the hardcoded info into an array inside AllClubsArray.js and made it so that we read off that array and display it on the screen (I think this is just better bc once the backend is set up, we’ll prob be receiving an array of clubs from them)
In AllClubsPgae.js, I called the component  instead of hardcoding every club info

#4


Not working on w10 pro 19042.906

Hello, it is not working for me for this version. please can help me.

same here!

same here!

https://github.com/stascorp/rdpwrap/issues/1360#issuecomment-814676181

Hi! I have the same picture with all green panels, but Its not working more than 1 RDP connection. If I try to reboot - RDP Windows Service run not correctly and RDP not working. Please help

same here!It look like quite dififcult

Hi! I have the same picture with all green panels, but Its not working more than 1 RDP connection. If I try to reboot - RDP Windows Service run not correctly and RDP not working. Please help

Possible problems with Windows. Run sfc / scannow

Duplicate of https://github.com/stascorp/rdpwrap/issues/1317


Mr   categorised as a person with the name  Mr

A not so nice example, but the text "Mr Clinton raped" is being classified as male name with the name of "clinton raped".
This is the body in question :
"Just a week after his acquittal in the Senate impeachment trial, Bill Clinton faces a fresh scandal about his past sexual conduct after a former campaign worker alleged that he raped her 21 years ago. Juanita Broaddrick, who now runs a nursing-home business in Arkansas, told the Wall Street Journal that Mr Clinton raped her in the Camelot Hotel in Little Rock in 1978, when he was the state attorney-general. The newspaper reports that Mr Clinton persuaded Mrs Broaddrick to have coffee with him in her hotel room during a conference of nursing home administrators in 1978.  When it was over, Mrs Broaddrick claims, Mr Clinton told her that she should not worry because he was sterile due to a bout of childhood mumps. Mrs Broaddrick is not directly quoted in the Journal's account of the alleged rape but in response to Mr Clinton's sterility claim she said: "As though that was the thing on my mind - I wasn't thinking about pregnancy or about anything. I felt paralysed and was starting to cry." She added: "This is the part that always stays in my mind - the way he put on his sunglasses. Then he looked at me and said 'You better put some ice on that'. Then he left." Norma Rogers, a friend of Mrs Broaddrick, told the Journal that she found the alleged victim in a state of shock. In reported speech, she is said to have alleged that Mrs Broaddrick's lips were discoloured and swollen to twice their normal size and the crotch of her tights was torn. "She just stayed on the bed and kept repeating 'I can't believe what happened'," Mrs Rogers said. Mrs Broaddrick's story has been the subject of rumours for years, and was widely circulated in 1992 when Mr Clinton was running for president. Until now she has refused to speak about the incident, and denied the story in an affidavit to lawyers for Paula Jones, who brought a sexual harassment case against Mr Clinton in 1994. Mrs Broaddrick's story surfaced in an American supermarket tabloid magazine in January, along with subsequently disproved claims that Mr Clinton had fathered the son of a black prostitute in Little Rock. On January 20 Mrs Broaddrick finally gave NBC television what is said to be her first media interview confirming the original rape allegation. Though scheduled for airing on January 29, the NBC interview has not yet been broadcast. NBC has described it as a "work in progress" - leading to rumours that the White House had put pressure on the network to withhold the story. The allegations come at a time when many of those involved in the past year's presidential crisis - including Monica Lewinsky, Linda Tripp and Gennifer Flowers - are telling their stories in books and media interviews. Mrs Broaddrick has also claimed that in 1991 she was called out of a meeting to discover Mr Clinton waiting for her. She said that he told her he wished to apologise and asked what he could do to make things up to her. Mrs Broaddrick said he could do nothing, and walked away. Shortly afterwards, Mr Clinton announced his presidential bid."
Calling nlp(body).people().data() results in 2 errors, "clinton raped" and "clinton's sterility"

ah, nice one. Yeah, it looks like the Mr honourific is being way too greedy. Hopefully two cases of the same bug. Will check it out today

hey Jonathan, got this squashed in v9.0.0:
let r = nlp('Mr. clinton raped and Mr. Clinton\'s sterility');
console.log(r.people().out('array'));
// [ 'mr clinton', 'mr clinton\'s' ]

you may also be pleased that it includes some pretty big performance improvements - something like ~40% faster.
Please let me know if you see anything else that's funky.
cheers


POP3 Support

I´m working on POP3 support for using it in gmail instead of forwarding mails. I got a unsecure solution. Google does not support self signed certs :(
I would like separate ssl configuration for pop3 and imap, is this possible in dovecot?
My fork with a pop3 branch is here:
https://github.com/dragonito/dockermail/tree/pop3d
If its runnig good i´ll add an pull request for merging. Perhaps you got some good ideas for resolving the "google" problem. Turning of ssl for both pop3 and imap is spooky.

I have never used pop3 for anything, so I'm afraid I can't really help you with that.

I discarded this feature. Instead I forward the mails like in issue https://github.com/lava/dockermail/issues/5

Kinda stuck with the same problem. Setting ssl to "yes" instead of "required" or "no" kinda does the trick. I added a entrypoint file that reads the env variables and sets ssl and disable_plaintext_auth to your likings. But for that you would have to change the makefile to include your env vars.
Maybe we should look at the improvments from #9 and backport the docker-compose stuff to have a nicer solution to this.


LinuxAcademy-DL is broken since today

The script worked fine for me yesterday but since today, i get the following error when trying to download any course:

Requesting download of https://linuxacademy.com/cp/modules/view/id/314
Sleeping for 15 seconds..
Attempting to login..
Login successful...
Getting lesson links...
Starting download...
[generic] 314: Requesting header
[redirect] Following redirect to https://linuxacademy.com/containers/training/lesson/course/docker-deep-dive-part-1/name/about-this-course
[generic] about-this-course: Requesting header
WARNING: Forcing on generic information extractor.
[generic] about-this-course: Downloading webpage
[generic] about-this-course: Extracting information
ERROR: Unsupported URL: https://linuxacademy.com/containers/training/lesson/course/docker-deep-dive-part-1/name/about-this-course
ERROR: Unsupported URL: https://linuxacademy.com/containers/training/lesson/course/docker-deep-dive-part-1/name/about-this-course
Downloading failed. Perhaps, the cookies have expired.

You can see that now the that first download url is always https://linuxacademy.com/containers/training/lesson/course/course-name/name/about-this-course.

The problem came from a bad cookie file.


Potential DDoS agent

Attacker sends a BindRequest to Pions and gets saved as a validPair.
Attacker never replies to Pion's BindRequest messages (so it will stay a validPair not selectedPair).
Attacker/Pions performs DTLS handshake.
Attacker sends a BindRequest with a higher priority and a spoofed return address (the target), which is also saved as a validPair.
Pions now proceeds to send raw SRTP/data to the target address.
Attacker can send RTCP/keep-alive information to keep the connection open.

This allows an attacker to DDoS a target without sending the data itself.
The fix involves requiring a SuccessResponse from a CandidatePair before writing to it. This may go against what the ICE RFC requires. It might actually not be possible because of NAT stuff, which is potentially a huge issue.

Hey @kixelated
I don't think this is possible anymore, BindRequests are filtered via MessageIntegrity (so if the attacker doesn't know the pwd we are safe)

Nope, not safe. The attacker in this case will set up a PeerConnection using it's own credentials. It can set up ICE/DTLS/SRTP everything. Except it will suddenly claim to have a different IP address by forging the UDP return address header. This tricks the server to take the attacker's legitimate PeerConnection and redirect it elsewhere.
For example, if Pions gets a STUN binding request that claims to be sent from google.com:80, it will mark it as a validPair and potentially use it as the result from getBestPair. Pions would then proceed to shove SRTP/DTLS traffic at Google which won't know what to do with it. The PeerConnection can be kept alive by the attacker because they can still send RTCP/STUN packets from their valid but lower priority real IP address.

if Pions gets a STUN binding request that claims to be sent from google.com:80
How? Binding requests are discarded unless you know the uPwd

Talked to @kixelated on Slack I understand the issue now.

We have to check TransactionIDs and we MUST require success responses from peers. Otherwise a candidate could give us invalid prflx candidates (and we would start sending traffic to 3rd parties unexpectedly)


Broken examples on LiteDb homepage

Seems to be a problem with the sample code on the homepage. For instance, the crossRef example code does not compile in a stock .Net 4.6 environment. I converted the "new List()" calls to "List" but ran into further problems.
The line "customers.Insert(john);" breaks with "cannot convert from Customer to LiteDB.BsonDocument.  The same occurs on the lines inserting products and orders.
Am I missing a ref or something? Should I ignore the homepage and go straight to the docs?
Thanks, otherwise looks like a great product that I'm eager to jump into using!

Hi! With the objective of organizing our issues, we are closing old unsolved issues. Please check the latest version of LiteDB and open a new issue if your problem/question/suggestion still applies. Thanks!


CRUD-9-Enums-Issues

Have tested, but was not able to compile successfully due to error in seed file. But is working on controller. Apparently it is creating controller files as we are calling seed file at last.

Once PR #28 is merged, merge main into this branch and check if it works fine


Add creator stage name to SolutionInfo

In #162 , @v4hn suggested the option of executing the solution sequentially to execute custom functions. We tried using this in an application, but when stepping through the solution's subtrajectories, we are somewhat blind as the solution does not seem to contain any semantic cues by itself*. If it does, a student of mine and I could not find them. This is a complex piece of software.
This PR extends the subtrajectory message by the name of the stage that created it so that the user can tell which part of the task they are at and execute arbitrary code (e.g. gripper open/close, set I/O, change display, call service...) when desired. Naming stages appropriately seems to be the easiest way to implement what @v4hn called "the trivial way".
Adding code hooks may be the proper way, but as mentioned in #162 there is no time frame on this feature, and this workaround is simple, lightweight and works in Python.
This does not need to be merged, it can just be a reference for whoever wants to execute solutions this way.
*edit: Yes, you could get the stage_id from the SolutionInfo of each trajectory and search through the TaskDescription to retrieve the name, but it is awkward.

*edit: Yes, you could get the stage_id from the SolutionInfo of each trajectory and search through the TaskDescription to retrieve the name, but it is awkward.

Indeed, but what's wrong with extracting the stage_id once from the description once and using the id to find the correct solution?
The id is much more appropriate for your use-case anyway, as there is no strict unique-name assumption among the stages.
Alternatively, assuming you have the Task object in the same process you can also extract the id directly:
auto state = make_unique<Stage>();
auto* stage_ptr { &stage };
task.add(move(state))
task.init();
uint32_t stage_id = task.introspection().stageId(stage_ptr);

Should we add a simpler interface for that last line?

If I understand correctly, you would have to store the task description along with the solution to know which stage_id to look up, which to me is just another opportunity to mix up numbers. A cleartext description in the solution to describe the purpose of each trajectory seems a lot more intuitive.
Also, I imagine that users will call action servers that return the solution for a task (Pick/Place/PickPlace). Even if the task description was returned alongside the solution, searching through the message manually is a hassle. By contrast, stepping through the trajectories and checking each stage name is a breeze.

Should we add a simpler interface for that last line?

I am not sure I even understood the introspection pattern, so I don't think I can answer that completely. But I'll note that we call action servers and execute the solutions in Python, so I don't think it will make a difference to us.

If I understand correctly, you would have to store the task description along with the solution to know which stage_id to look up,

Yes if you want to infer the correct stage remotely from the description.
But no, if you directly get the correct id from the task - probably using the new method in fae4520 as I outlined above.

which to me is just another opportunity to mix up numbers. A cleartext description in the solution to describe the purpose of each trajectory seems a lot less error-prone and more intuitive.

As long as you do not use the same stage description twice that's an option.
From my perspective I think both approaches are equally error-prone, it's just a bit easier to debug discrepancies with string. :-)
For a better solution, what do you think about adding something like Property[] execution_properties to the SolutionInfo.
My motivation is to allow user-defined flags like "enable_suction", "switch_display_mode", "change_safety_parameter" to be specified with any SubTrajectory. That way, you can have a sane execution management that simply acts according to these flags instead of hard-coding stage names and at the same time you could have a Propagator stage that sets these properties in your Task model already.

if you directly get the correct id from the task

That is assuming that you have access to the task, which you would not if you get the solution from a move_group capability as described in #112.

From my perspective I think both approaches are equally error-prone, it's just a bit easier to debug discrepancies with string. :-)
For a better solution, what do you think about adding something like Property[] execution_properties to the SolutionInfo.
My motivation is to allow user-defined flags like "enable_suction", "switch_display_mode", "change_safety_parameter" to be specified with any SubTrajectory. That way, you can have a sane execution management that simply acts according to these flags instead of hard-coding stage names and at the same time you could have a Propagator stage that sets these properties in your Task model already.

I like it, although the name execution_properties makes me expect modifying parameters or things I should not touch as a user. custom_flags or similar would probably be clearer.
That said, even though using the stage name to transmit flags/signals like that is definitely hacky, I think the stage name is also helpful for debugging and displaying, so including it in the solution makes sense to me.

But for any kind of interleaved-execution or custom hooks, you would need [access to the Task] anyway. The only other way to support similar functionality is by additional information in the solution message, as we discuss right now.

I assumed that one should be able to execute the solution without knowing the task description, just like one can follow a manual without knowing all of the specifications and constraints that were needed to create the manual. I consider MTC a problem solving engine, so I would not expect to need the Task object (part of the solver) to execute the solution, and it seems reasonable to add information to the solution message. That's just my interpretation though!

The concept of properties is very general in MTC and you already modify the properties of stages to setup your problem. We could also go for Property[] external or external_properties, I would just like to avoid ambiguities with the Properties that are used to generate a specific solution (we do not export those), although maybe we should.

I'm not sure which properties are used where, and which properties are used to generate a "specific" solution rather than the returned message.

Properties can be boolean flags, but providing arbitrary data, e.g. to specify the pressure for each element in a suction array, is supported here, so "flags" is no good name.

Then maybe custom_properties? I know that technically everything can be customized, but the point is to convey that it's a setting the user should configure. When I read through MTC, I found it difficult to know what is essential to the framework (= "better not touch"), and what can/should be changed when making a new task. I would hope a name like this makes it easier to grok.

But then we would add redundant information to the messages and it also makes sense to avoid that. The information is now cleanly split between task description, statistical data and actual solutions and we decided not to use the stage name as linking key. If you want to debug MTC plans, you should save the description/statistics topics anyway or debug from within the C++ code in which you run the Task.

True, it would be somewhat redundant, which is why I had doubts about merging this. I guess it's a question of how self-contained you want the solution to be, and what users should be able to do with the solution message alone.
For completeness: a use case outside of debugging is to display the stage name to show what the robot is currently doing.

@v4hn I've tried something very similar to what you suggested above. You suggested this:
auto state = make_unique<Stage>();
auto* stage_ptr { &stage };
task.add(move(state))
task.init();
uint32_t stage_id = task.introspection().stageId(stage_ptr);

I tried this:
moveit::task_constructor::Stage* open_fingers_stage_;
...
// Note, I use moveit::task_constructor::stages::MoveTo rather than Stage
auto stage = std::make_unique<MoveTo>(
  PICK_STAGE_MAP.at(PickStageEnum::OPEN_FINGERS),
  sampling_planner);
stage->setGroup(finger_group_);
stage->setGoal(arm_prefix_ + "fingers_open");
open_fingers_stage_ = &stage;
task_.add(std::move(stage));

It doesn't compile because of the type difference.
 error: cannot convert ‘std::unique_ptr<moveit::task_constructor::stages::MoveTo, std::default_delete<moveit::task_constructor::stages::MoveTo> >*’ to ‘moveit::task_constructor::Stage*’ in assignment
  130 |     open_fingers_stage_ = &stage;

I believe the MoveTo type is correct, and I need the Stage* type to be able to retrieve the ID. So... I do not see a way around this.

Obviously, stage is a unique_ptr. To get a raw address, you need to use stage.get()!

Thank you Robert!
I'm getting close but there is one more issue:
uint32_t stage_id = task_.introspection().stageId(open_fingers_stage_);
error: ‘uint32_t moveit::task_constructor::Introspection::stageId(const moveit::task_constructor::Stage*)’ is private within this context

Would you accept a PR to make stageId() public?

The const version of stageId() is public. Please don't hijack PRs / issues with unrelated questions, but open a separate thread instead.

Alright. I would not say the discussion is off-topic because it is related to accomplishing the purpose of this PR. It would be useful to anybody reading this PR.
Also, you said:

Obviously, stage is a unique_ptr

What isn't so obvious is that MoveTo inherits from Stage. If I read the header file, MoveTo inherits from PropagatingEitherWay. I guess there are several layers of inheritance, which can make it difficult to find things.

Closing in favour of #194


GEODE-9191: PR clear could miss clearing bucket which lost primary

Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[ ] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[ ] Is your initial contribution a single, squashed commit?

[ ] Does gradlew build run cleanly?

[ ] Have you written or updated unit tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, check Concourse for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

Is it possible to break this PR down to multiple PRs. The changes seem overwhelming. Let's separate functionality change from refactoring (renaming etc.)

Is it possible to break this PR down to multiple PRs. The changes seem overwhelming. Let's separate functionality change from refactoring (renaming etc.)

The reason the PR is so huge is that the GEODE-7665 branch was rebased on develop after this PR was created. The PR needs to be rebased on the GEODE-7665 branch before it can be reviewed. @gesterzhou


Don't fail shutdown-does-not-crash test if merely ungraceful

Fixes https://github.com/aspnet/AspNetCore-Internal/issues/2101

@aspnet-hello triage https://dev.azure.com/dnceng/public/_build/results?buildId=136722

This comment was made automatically. If there is a problem contact ryanbrandenburg.
I've triaged the above build.

@anurse this is good to go.

2.2 branch isn't open yet, we should wait for the branding change.

Branch is now open, please merge ASAP.

Yes, please do 😉


Monitor can't deleted

⚠ Please verify that this bug 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

Description
After delete monitor and refresh page I have this page :

And monitor can't deleted
👟 Reproduction steps
Delete monitor and refresh page
👀 Expected behavior
Monitor must be removed
😓 Actual Behavior
Monitor not be removed
🐻 Uptime-Kuma Version
1.21.2
💻 Operating System and Arch
Ubuntu 22.04
🌐 Browser
Brave 1.49.132
🐋 Docker Version
No response
🟩 NodeJS Version
14.20.1
📝 Relevant log output
No response

Please see #1661

Good evening,
Thank you, it's ok while waiting longer.
Sorry to bother you
THANKS


Stash commands and status

We're missing the stash list on the status page, as well as stash commands.
Start with save / pop / apply.
Some of these commands open up a prompt, like zz prompts you to name the stash.
zp will also prompt which stash to pop (default to 0, or the marked one).

I think keeping the reference to Repository immutable is not a requirement, but makes the code easier to reason about. If it needs to be a Rc<RefCell> (I think?), it's not a huge deal.
Had a peek over at Gitoxide and it seems there's no support for it, else it could have been a good segue into starting to use that.
I found the git cli function here.
It seems to be referencing refs/stash, perhaps there's a way to list these with more nitty-gritty git2 commands?

@golden-expiriensu I played around with it a bit, stole some code from the log fn. It lists a lot more than just what git stash would, not sure why. But the stash entries are indeed found in there.
#[test]
fn stash_list() {
    let repo = Repository::open(".").unwrap();
    let mut revwalk = repo.revwalk().unwrap();
    revwalk.push_ref("refs/stash").unwrap();
    dbg!(revwalk
        .flat_map(|maybe_oid| maybe_oid.and_then(|oid| repo.find_commit(oid)))
        .map(|commit| commit.summary().unwrap().to_string())
        .collect::<Vec<_>>());

    todo!();
}

Again, if git_stash_foreach proves to be really simple to use, we could compromise and get rid of the immutable reference to Repository!

I believe I found another way, under the hood git uses reflog to find changes of ref/stash file, so I think git reflog stash is a solution


Error in proc.go: ‘state’ declared and not used

Hi,
I am trying to run a multi cluster with etcd and when I try to run goreman using "go get github.com/mattn/goreman", I get the following error:
$ go get github.com/mattn/goreman
github.com/mattn/goreman
./proc.go:89:2: error: ‘state’ declared and not used
state := true
^
Any idea what could be causing this?

Yes, you are right. But I can't reproduce it. I will fix in soon. But what version of go do you use?

I am using go1.4.2.
$ go version
go version go1.4.2 gccgo (Ubuntu 5.1~rc1-0ubuntu1) 5.0.1 20150414 (prerelease) [gcc-5-branch revision 222102]

Ah, gccgo.

Yeah, sorry, I guess I forgot to mention that!

Thanks for your answer.

HI, I am running goreman as an example to run multi cluster etcd example. When I run "goreman start", I am getting this:
07:43:41 etcd3 | 2015/07/2 07:43:41 rafthttp: request sent was ignored (cluster ID mismatch: remote[a8266ecf031671f3]=7e27652122e8b2ae, local=e9c7614f68f35fb2)
07:43:41 etcd3 | 2015/07/2 07:43:41 rafthttp: request sent was ignored (cluster ID mismatch: remote[a8266ecf031671f3]=7e27652122e8b2ae, local=e9c7614f68f35fb2)
07:43:41 etcd2 | 2015/07/2 07:43:41 raft: 6e3bd23ae5f1eae0 is starting a new election at term 2631
07:43:41 etcd2 | 2015/07/2 07:43:41 raft: 6e3bd23ae5f1eae0 became candidate at term 2632
07:43:41 etcd2 | 2015/07/2 07:43:41 raft: 6e3bd23ae5f1eae0 received vote from 6e3bd23ae5f1eae0 at term 2632
07:43:41 etcd2 | 2015/07/2 07:43:41 raft: 6e3bd23ae5f1eae0 [logterm: 1, index: 3] sent vote request to 924e2e83e93f2560 at term 2632
07:43:41 etcd2 | 2015/07/2 07:43:41 raft: 6e3bd23ae5f1eae0 [logterm: 1, index: 3] sent vote request to a8266ecf031671f3 at term 2632
07:43:41 etcd2 | 2015/07/2 07:43:41 rafthttp: request sent was ignored (cluster ID mismatch: remote[a8266ecf031671f3]=7e27652122e8b2ae, local=e9c7614f68f35fb2)
07:43:41 etcd2 | 2015/07/2 07:43:41 rafthttp: request sent was ignored (cluster ID mismatch: remote[a8266ecf031671f3]=7e27652122e8b2ae, local=e9c7614f68f35fb2)
07:43:41 etcd2 | 2015/07/2 07:43:41 rafthttp: request sent was ignored (cluster ID mismatch: remote[a8266ecf031671f3]=7e27652122e8b2ae, local=e9c7614f68f35fb2)
07:43:41 etcd3 | 2015/07/2 07:43:41 rafthttp: request sent was ignored (cluster ID mismatch: remote[a8266ecf031671f3]=7e27652122e8b2ae, local=e9c7614f68f35fb2)
07:43:41 etcd3 | 2015/07/2 07:43:41 rafthttp: request sent was ignored (cluster ID mismatch: remote[a8266ecf031671f3]=7e27652122e8b2ae, local=e9c7614f68f35fb2)
07:43:42 etcd2 | 2015/07/2 07:43:42 rafthttp: request sent was ignored (cluster ID mismatch: remote[a8266ecf031671f3]=7e27652122e8b2ae, local=e9c7614f68f35fb2)
07:43:42 etcd2 | 2015/07/2 07:43:42 rafthttp: request sent was ignored (cluster ID mismatch: remote[a8266ecf031671f3]=7e27652122e8b2ae, local=e9c7614f68f35fb2)
07:43:42 etcd3 | 2015/07/2 07:43:42 rafthttp: request sent was ignored (cluster ID mismatch: remote[a8266ecf031671f3]=7e27652122e8b2ae, local=e9c7614f68f35fb2)
07:43:42 etcd3 | 2015/07/2 07:43:42 rafthttp: request sent was ignored (cluster ID mismatch: remote[a8266ecf031671f3]=7e27652122e8b2ae, local=e9c7614f68f35fb2)
07:43:42 etcd2 | 2015/07/2 07:43:42 rafthttp: request sent was ignored (cluster ID mismatch: remote[a8266ecf031671f3]=7e27652122e8b2ae, local=e9c7614f68f35fb2)
07:43:42 etcd2 | 2015/07/2 07:43:42 rafthttp: request sent was ignored (cluster ID mismatch: remote[a8266ecf031671f3]=7e27652122e8b2ae, local=e9c7614f68f35fb2)
07:43:42 etcd3 | 2015/07/2 07:43:42 rafthttp: request sent was ignored (cluster ID mismatch: remote[a8266ecf031671f3]=7e27652122e8b2ae, local=e9c7614f68f35fb2)
07:43:42 etcd3 | 2015/07/2 07:43:42 rafthttp: request sent was ignored (cluster ID mismatch: remote[a8266ecf031671f3]=7e27652122e8b2ae, local=e9c7614f68f35fb2)
07:43:42 etcd2 | 2015/07/2 07:43:42 rafthttp: request sent was ignored (cluster ID mismatch: remote[a8266ecf031671f3]=7e27652122e8b2ae, local=e9c7614f68f35fb2)
07:43:42 etcd2 | 2015/07/2 07:43:42 rafthttp: request sent was ignored (cluster ID mismatch: remote[a8266ecf031671f3]=7e27652122e8b2ae, local=e9c7614f68f35fb2)
07:43:42 etcd3 | 2015/07/2 07:43:42 rafthttp: request sent was ignored (cluster ID mismatch: remote[a8266ecf031671f3]=7e27652122e8b2ae, local=e9c7614f68f35fb2)
07:43:42 etcd3 | 2015/07/2 07:43:42 rafthttp: request sent was ignored (cluster ID mismatch: remote[a8266ecf031671f3]=7e27652122e8b2ae, local=e9c7614f68f35fb2)
07:43:42 etcd2 | 2015/07/2 07:43:42 rafthttp: request sent was ignored (cluster ID mismatch: remote[a8266ecf031671f3]=7e27652122e8b2ae, local=e9c7614f68f35fb2)
07:43:42 etcd2 | 2015/07/2 07:43:42 rafthttp: request sent was ignored (cluster ID mismatch: remote[a8266ecf031671f3]=7e27652122e8b2ae, local=e9c7614f68f35fb2)
07:43:42 etcd3 | 2015/07/2 07:43:42 rafthttp: request sent was ignored (cluster ID mismatch: remote[a8266ecf031671f3]=7e27652122e8b2ae, local=e9c7614f68f35fb2)
07:43:42 etcd3 | 2015/07/2 07:43:42 rafthttp: request sent was ignored (cluster ID mismatch: remote[a8266ecf031671f3]=7e27652122e8b2ae, local=e9c7614f68f35fb2)
07:43:42 etcd2 | 2015/07/2 07:43:42 rafthttp: request sent was ignored (cluster ID mismatch: remote[a8266ecf031671f3]=7e27652122e8b2ae, local=e9c7614f68f35fb2)
07:43:42 etcd2 | 2015/07/2 07:43:42 rafthttp: request sent was ignored (cluster ID mismatch: remote[a8266ecf031671f3]=7e27652122e8b2ae, local=e9c7614f68f35fb2)
07:43:42 etcd3 | 2015/07/2 07:43:42 rafthttp: request sent was ignored (cluster ID mismatch: remote[a8266ecf031671f3]=7e27652122e8b2ae, local=e9c7614f68f35fb2)
07:43:42 etcd3 | 2015/07/2 07:43:42 rafthttp: request sent was ignored (cluster ID mismatch: remote[a8266ecf031671f3]=7e27652122e8b2ae, local=e9c7614f68f35fb2)
Any idea what exactly is wrong here?

Could you please show me your Procfile?

cat Procfile
Use goreman to run go get github.com/mattn/goreman
etcd1: /usr/bin/etcd -name infra1 -listen-client-urls http://localhost:4001 -advertise-client-urls http://localhost:4001 -listen-peer-urls http://localhost:7001 -initial-advertise-peer-urls http://localhost:7001 -initial-cluster-token etcd-cluster-1 -initial-cluster 'infra1=http://localhost:7001,infra2=http://localhost:7002,infra3=http://localhost:7003' -initial-cluster-state new
etcd2: /usr/bin/etcd -name infra2 -listen-client-urls http://localhost:4002 -advertise-client-urls http://localhost:4002 -listen-peer-urls http://localhost:7002 -initial-advertise-peer-urls http://localhost:7002 -initial-cluster-token etcd-cluster-1 -initial-cluster 'infra1=http://localhost:7001,infra2=http://localhost:7002,infra3=http://localhost:7003' -initial-cluster-state new
etcd3: /usr/bin/etcd -name infra3 -listen-client-urls http://localhost:4003 -advertise-client-urls http://localhost:4003 -listen-peer-urls http://localhost:7003 -initial-advertise-peer-urls http://localhost:7003 -initial-cluster-token etcd-cluster-1 -initial-cluster 'infra1=http://localhost:7001,infra2=http://localhost:7002,infra3=http://localhost:7003' -initial-cluster-state new
proxy: /usr/bin/etcd -name proxy1 -proxy=on -listen-client-urls http://localhost:8080 -initial-cluster 'infra1=http://localhost:7001,infra2=http://localhost:7002,infra3=http://localhost:7003'
I am using e one provided by etcd as it.

Hmm, I can't reproduce.

@mattn you are using gccgo or go compiler?

Since gccgo is not provided on windows, I use go compiler always.

hmmm, could it be a gccgo issue then?

No, this is an issue of code of goreman.
See https://groups.google.com/forum/#!topic/golang-dev/QaAXYhC22G0

Well, the issue about etcd, may be not an issue of gccgo.

Hmmm, I don't know any other player in the game! It's either etcd or gccgo!


Dependencies Policy

Fixes #876 Task-12 Security Slam
Description
Currently, Dependencies Policy test is failing due to this. Although i am not familiar what will be the policy-url for this.
Example - https://github.com/argoproj/argo-cd/blob/master/SECURITY-INSIGHTS.yml#L127-L128
// @jonathan-innis would be good if you can mention about what should i add in policy-url
How was this change tested?
https://clomonitor.io/projects/cncf/karpenter#karpenter_security
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage remained the same at 79.925%

Files with Coverage Reduction
New Missed Lines
%

pkg/controllers/provisioning/scheduling/nodeclaim.go
2
88.17%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
7656

Relevant Lines:
9579

💛  - Coveralls

any update here?

/ok-to-test

Created the issue related to this here -> #915


Explore Refactoring the headers on each transform function to pre-stage the set up computations.

Pre-stage the computations at the start of each transform to eliminate the repetitious set up for each frame. Doesn't seem it would make much of a performance difference, but it would make multi-threading of the transforms easier.

Done.


Subaru: add flag checking helpers

using &'s everywhere will be easy to mess up. for example
(SubaruFlags.HYBRID | SubaruFlags.GLOBAL_GEN2) & self.CP.flags will return true for global gen2 and hybrids
but in the future if we wanted  "gen2 hybrid", and did this:
GEN2_HYBRID = SubaruFlags.HYBRID | SubaruFlags.GLOBAL_GEN2
SubaruFlags.GEN2_HYBRID & self.CP.flags will return true for non-hybrid gen2 or hybrid non-gen2s
we instead have to do
(SubaruFlags.GEN2_HYBRID & self.CP.flags) == SubaruFlags.GEN2_HYBRID, which is very verbose and confusing
instead, I'm adding
SubaruFlags.HYBRID.check(CP.flags) -> is car hybrid
(SubaruFlags.HYBRID | SubaruFlags.GEN2).any(CP.flags -> is car hybrid or gen2
and
(SubaruFlags.HYBRID | SubaruFlags.GEN2).check(CP.flags) will fail, since SubaruFlags.HYBRID | SubaruFlags.GEN2 is not part of the SubaruFlags enum

it just needs to be clear where you are do any any check or an all check at a glance, without having to know exactly how bit anding works
ret.dashcamOnly = bool(ret.flags & (SubaruFlags.PREGLOBAL | SubaruFlags.LKAS_ANGLE | SubaruFlags.HYBRID))

maybe we just ban flag oring so that you have to explicitly check each
(ret.flags & SubaruFlags.PREGLOBAL) or (...)

I think .all is confusing when you only have 1 flag to check though
SubaruFlags.GLOBAL_GEN2.all(self.CP.flags)

I don't see why there needs to be two functions for this. The or is clear

this is the case I'm worried about that might make it unclear
GEN2_HYBRID = GEN2 | HYBRID
then try checking
# is the car a GEN2 hybrid
if GEN2_HYBRID & self.CP.flags:

I'll think about this more, but I think as we add more and more flags it will be important to make it easily understandable


chore(DIST-978): Yarn cache

:tada: This PR is included in version 0.36.1 :tada:
The release is available on:

npm package (@@release-0.x dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


This only works for Google Authenticator with my GMail account.

These scripts only produce the correct OTPs for my Google GMail account. I've tried to use them for Google Authenticator with my other non-Google accounts, but for some reason the OTP codes won't work. Why would that be?

Did you program your yubi key with the personalization tool to use the secret key from the other service? You must provide the hex formatted secret, not the base 32 ecoded one (that's what the --convert-secret option is for).
I've never tested it with another service, but it should work in theory if the service is using TOTP in the same was as google.

Yes, I used --convert-secret. I followed exactly the same procedure that worked for setting up 2FA for my GMail account.
I'm thinking that other services would require yubi-goog to communicate with a server or something?


Delay enter if key_enter is already detected as key pressed

Related issues
Closes #625.
Summary
Delays the enter key if key_enter is already set in key_check() beforehand. Otherwise, those presses of the enter key won't be delayed.
Playtest script
local Event = Elona.require("Event")
local Item = Elona.require("Item")
local Chara = Elona.require("Chara")

Event.register(Event.EventKind.ScriptLoaded, function() Item.create(Chara.player().position, 313, 1) end)

I checked this bug and it was resolved with this change. Thank you.


circuit.library.fourier_checking needs testing

The object FourierChecking (in qiskit/circuit/library/fourier_checking.py) needs testing.  No line is currently coveraged.

Hi I would like to work on this issue, please may I be assigned?

Assigned! Let me know if you need any help.

Thanks, I'll get on it now!

Testing it now, what should be the expected behavior for the FourierChecking object?

Hi, I'm sorry but as this is my first issue please could I get some help with this?

Sorry I'm taking so long, have had a lot of stuff going on, Testing the FourierChecking object now

Hi!
Would it be fine for me to pick this issue up if it is not being worked on at the moment? I am working on a similar issue (#4349) at the moment, so I think I would have enough background to pick this one up.

Hi again!
I looked into this briefly and came up with the following:
@ddt
class TestFourierCheckingLibrary(QiskitTestCase):
    """Test the Fourier Checking circuit."""

    def assertFourierCheckingIsCorrect(self, f, g, fc_circuit):
        """Assert that the Fourier Checking circuit produces the correct matrix."""

        simulated = Operator(fc_circuit)

        num_qubits = int(np.log2(len(f)))

        # create Hadamard matrix, re-use approach from TestMCMT
        h_i = 1 / np.sqrt(2) * np.array([[1, 1], [1, -1]])
        h_tot = np.array([1])
        for _ in range(num_qubits):
            h_tot = np.kron(h_tot, h_i)

        f_mat = np.diag(f)
        g_mat = np.diag(g)

        expected = np.linalg.multi_dot([h_tot, g_mat, h_tot, f_mat, h_tot])
        expected = Operator(expected)

        self.assertTrue(expected.equiv(simulated))

    @data(
        ([1, -1, -1, -1], [1, 1, -1, -1])
    )
    def test_fourier_checking(self, f, g):
        """Test if the Fourier Checking circuit produces the correct matrix."""
        fc_circuit = FourierChecking(f, g)
        self.assertFourierCheckingIsCorrect(f, g, fc_circuit)

    @data(
        ([1, -1, -1, -1], [1, 1, -1])
    )
    def test_invalid_input_raises(self, f, g):
        """Test that invalid input truth tables raise an error."""
        with self.assertRaises(CircuitError):
            FourierChecking(f, g)

Would something like this be good for this test? It is a simple check based on calculating the circuit matrix multiplying Hadamard and diagonal truth table matrices. I assumed this straightforward check should be fine. Additionally, I thought it would make sense to add a check that an error is raised with an invalid input.
If you would think this looks good, I would be more than glad to make a pull request with the test. If someone else is working on this issue maybe this piece of code could be helpful for others.
If this code does not really make sense, feel free to ignore this comment 🙂
Thank you!

Thanks for the contribution @mantcep, please open a PR for this! 👍

Thank you for the feedback, @Cryoris! I have created the PR and already got a review on that 🙂


[Typo/Mistranslation] 07-design-report-in-power-bi-desktop-enhanced

On Practice 5, task 1 step 2, the column and table names need to stay in English so it will align with the lab environment. It is very confusing for the learners to see the column/table names in Korean on the instruction and then have to match it with the corresponding English names on the lab environment.
Current:
범주별 수량
제품 세부 정보
To be:
Category별 Quantity
Product Details

Hello contributor,
Thank you for your contribution! We are processing your suggestions and we will update you shortly.
Kind regards,
Microsoft Worldwide Learning Team

Locfile at PHO - Project HO disabled - must follow up with the PM

Discussed with the PM - update made to the older HB locfiles and this will seed the TM when handedback


How to keyboard?

If i keyboard: true, ALL galleries on my page will cycle at the same time on button press. Is there a parameter to cycle ONLY the focused one?

Hi @MM3y3r , for more info I highly recommend you should check Swiper API documentation instead.


Wrong code color syntax highlighting due to docstring

Environment data

Reposting from https://github.com/microsoft/vscode/issues/165030
VS Code version: Code 1.72.2 (d045a5eda657f4d7b676dedbfa7aab8207f8a075, 2022-10-12T22:15:18.074Z)
OS version: Windows_NT x64 10.0.19043
Modes:
Sandboxed: No
Python 3.7,4 - PyPI
Code Snippet

Write this piece of code in .py or ipynb file,
def foo(x):
    """foo
    """
    return (
        x
        * 42  # why wrong colour? treated as comment by syntax higlight
    )

def bar():
    """bar
    """
    return 42

Repro Steps

Write and see

Expected behavior
Not highlighted as comment
Actual behavior
Highlighted as comment

Screenshoot here:
https://github.com/microsoft/vscode/issues/165030#issuecomment-1296495718

@yasirroni this seems to work just fine for me

What does it look like for you? I'm not reproing your screenshot. THis is with the Monokai Dark theme.
This is what it looks like with default dark:

What's your pylance extension version? These screen shots were taken with 2022.10.40

I'm sorry for not closing this issue. It is based on better comment. See: https://github.com/microsoft/vscode/issues/165030#issuecomment-1304847665


[NEW ALGORITHM] FCFS CPU Scheduling Algorithm

Issue will be closed if:

You mention more than one algorithm. You can create a separate issue for each algorithm once the current one is completed.
You propose an algorithm that is already present or has been mentioned in a previous issue.
You create a new issue without completing your previous issue.

Note: These actions will be taken seriously. Failure to follow the guidelines may result in the immediate closure of your issue.

Name:
FIRST COME FIRST SERVE CPU SCHEDULING ALGORITHM
About:
The First-Come, First-Served (FCFS) algorithm is a simple, non-preemptive scheduling method used in operating systems. It processes tasks in the exact order they arrive, without priority. Once a task starts executing, it runs until completion.The C program implements First-Come, First-Served (FCFS) scheduling for processes. It defines a Process structure to store details like burst time, arrival time, start time, and completion time. The program sorts processes based on their arrival times using qsort(), ensuring they are executed in the correct order. After scheduling, it calculates turnaround time and waiting time for each process, printing the results in a formatted output.
Labels:
new algorithm, gssoc-ext, hacktoberfest, level1

Assignees:

[X] Contributor in GSSoC-ext
[X] Want to work on it

assigned


select.min.js via npm repository not working

Downloading via npm is not working (via bower works normally)
Link to minimally-working plunker that reproduces the issue:

"angular": "^1.5.9",
"ui-select": "^0.19.6"

You're not loading the correct npm module... look at your paths.


embark run fails

After installing embark and running the demo as described in the readme the embark run command fails with the following output:
path.js:1144
          cwd = process.cwd();
                        ^

Error: ENOENT: no such file or directory, uv_cwd
    at Error (native)
    at Object.resolve (path.js:1144:25)
    at Function.Module._resolveLookupPaths (module.js:361:17)
    at Function.Module._resolveFilename (module.js:431:31)
    at Function.Module._load (module.js:388:25)
    at Module.require (module.js:468:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/usr/local/lib/node_modules/embark-framework/bin/embark:9:14)
    at Module._compile (module.js:541:32)
    at Object.Module._extensions..js (module.js:550:10)

what version of node are you running?

v6.3.0

Actually I was running an old version (4.7 I think) but updated to see if this would fix the problem.

what about geth? what version are you using?
On Fri, Jul 8, 2016 at 10:38 AM GuutBoy<EMAIL_ADDRESS>wrote:

Actually I was running an old version (4.7 I think) but updated to see if
this would fix the problem.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/iurimatias/embark-framework/issues/167#issuecomment-231376979,
or mute the thread
https://github.com/notifications/unsubscribe/AAKyUFlEdSQLhxwKtrx6yvsdfJkx3HBuks5qTmDugaJpZM4JHylJ
.

I am not at my machine right now so I cannot check, but it's the latest from brew as far as I recall.
Den 9. jul. 2016 kl. 01.44 skrev Iuri Matias <notifications<EMAIL_ADDRESS>what about geth? what version are you using?
On Fri, Jul 8, 2016 at 10:38 AM GuutBoy <notifications<EMAIL_ADDRESS>wrote:

Actually I was running an old version (4.7 I think) but updated to see if
this would fix the problem.

You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/iurimatias/embark-framework/issues/167#issuecomment-231376979,
or mute the thread
https://github.com/notifications/unsubscribe/AAKyUFlEdSQLhxwKtrx6yvsdfJkx3HBuks5qTmDugaJpZM4JHylJ
.

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/iurimatias/embark-framework/issues/167#issuecomment-231497889, or mute the threadhttps://github.com/notifications/unsubscribe/AGmJPvOPFObdLwjYADDLY6n_zQVeN3QVks5qTuDFgaJpZM4JHylJ.

@GuutBoy did you ever get this solved?

No, I stopped trying a while ago.


wall model in Modelica.Fluid.Examples.HeatExchanger.BaseClasses.BasicHX

Reported by ulf.nordstrom on 15 Nov 2013 13:40 UTC
Hi,
One of our users reported
"
The "wall" model Modelica.Fluid.Examples.HeatExchanger.BaseClasses.BasicHX references the average of the pipes cross sectional areas for the heat transfer area area_h. Shouldn't it reference the pipe parameter area_h_1 and area_h_2?
"
Could you have a look at it.
Best regards,
Ulf

Migrated-From: https://trac.modelica.org/Modelica/ticket/1352

Modified by beutlich on 21 May 2014 20:41 UTC

Comment by otter on 26 Jun 2014 15:45 UTC
I assume the proposal to change in Modelica.Fluid.Examples.HeatExchanger.BaseClasses.BasicHX:
  BaseClasses.WallConstProps wall(
    ...
    area_h=(crossArea_1 + crossArea_2)/2)

to
  BaseClasses.WallConstProps wall(
    ...
    area_h=area_h)

I think you are right, but someone from the Fluid group should verify

Comment by hubertus on 26 Jun 2014 18:07 UTC
Martin, I think your assumption is right. With the current settings heat transfer is about 2 orders of magnitude too low.
BTW: the example ha a number of other problems. This error would have been detected easily, if customary variables for heat exchangers like the heatcapacity flow would have been included.
The model shows how you can build a heat exchanger from the base classes in Modelica.Fluid, but from the point of view of usability it is atrocious, seriously very bad. I am also not sure what the example is supposed to show.
I leave it to active members of the fluid group what to do about this, but it seems to me rather a "how not to use object orientation" counter-example.

Comment by otter on 27 Jun 2014 07:36 UTC
Fixed in 1347a2db16a852a7a5f583239c3be93145374df7 (trunk) and in 3614de4d4bc8c5145a688de4854246c1d4cefe89 (maintenance)


Update sbt-example to 9.0.0

Updates com.thoughtworks.example:sbt-example from 6.0.1 to 9.0.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.thoughtworks.example", artifactId = "sbt-example" } ]

labels: sbt-plugin-update, early-semver-major, semver-spec-major

Superseded by #77.


Store id token along with access token

This is so that the id token can be used for authentication purpose with downstream APIs via JWT.

LGTM


fix(api): Use async APIs in GraphQL IAM integration tests and remove cleanUp() from AWSAPIPlugin.reset()

Issue #, if available: https://github.com/aws-amplify/amplify-swift/issues/2120
Description of changes: Use async APIs in GraphQL IAM integration tests and remove cleanUp() from AWSAPIPlugin.reset()
Check points: (check or cross out if not relevant)

[x] Added new tests to cover change, if needed
[x] Build succeeds with all target using Swift Package Manager
[x] All unit tests pass
[x] All integration tests pass
[x] Security oriented best practices and standards are followed (e.g. using input sanitization, principle of least privilege, etc)
- [ ] Documentation update for the change if required
[x] PR title conforms to conventional commit style
- [ ] If breaking change, documentation/changelog update with migration instructions

DataStore checkpoints (check when completed)

[ ] Ran AWSDataStorePluginIntegrationTests
[ ] Ran AWSDataStorePluginV2Tests
[ ] Ran AWSDataStorePluginMultiAuthTests
[ ] Ran AWSDataStorePluginCPKTests
[ ] Ran AWSDataStorePluginAuthCognitoTests
[ ] Ran AWSDataStorePluginAuthIAMTests

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

The changes mentioned by CRT team will involve updating Swift dependency to new version of aws-sdk-swift. Currently, moving this to draft.

Removing the call to optional method cleanUp() in APIPlugin reset() method. Confirmed by CRT team at https://github.com/awslabs/aws-crt-swift/issues/134 that it is expected to hang if there is a reference to CRT resource still held.


Documentation update

Hi there,
I'm new to amphp/parallel. Please correct me if wrong, but it seems as if the README needs to be updated.
The example below will throw an error:
<?php

require __DIR__  . '/../../vendor/autoload.php';

use Amp\Future;
use Amp\Parallel\Worker;
use function Amp\async;

use ChessServer\FetchTask;

$urls = [
    'https://secure.php.net',
    'https://amphp.org',
    'https://github.com',
];

$executions = [];
foreach ($urls as $url) {
    // FetchTask is just an example, you'll have to implement
    // the Task interface for your task.
    $executions[$url] = Worker\submit(new FetchTask($url));
}

// Each submission returns an Execution instance to allow two-way
// communication with a task. Here we're only interested in the
// task result, so we use the Future from Execution::getFuture()
$responses = Future\await(array_map(
    fn (Worker\Execution $e) => $e->getFuture(),
    $executions,
));

foreach ($responses as $url => $response) {
    \printf("Read %d bytes from %s\n", \strlen($response), $url);
}

PHP Fatal error:  Uncaught Error: Call to undefined function Amp\Parallel\Worker\submit()

Also an error is thrown if the following example is run:
<?php

require __DIR__  . '/../../vendor/autoload.php';

use Amp\Parallel\Worker;
use ChessServer\FetchTask;

$worker = Worker\createWorker();
$task = new FetchTask('https://amphp.org');

$execution = $worker->submit($task);

$data = $execution->await();

PHP Fatal error:  Uncaught Error: Call to undefined function Amp\Parallel\Worker\createWorker()

Thanks for the help and keep up the great work!

Hi @programarivm!
These two examples are working for me. Be sure that your path to autoload.php is correct so that the functions.php file where these functions are defined can be loaded.

Sorry @trowski, it seems as if the documentation is not detailed enough. Or perhaps I am missing something? Let me rephrase the question.
I am getting the following error when trying to run the Hello World example.
php cli/amphp/hello-world.php
PHP Fatal error:  Uncaught Error: Call to undefined function Amp\async() in /home/standard/projects/chesslablab/chess-server/cli/amphp/hello-world.php:9
Stack trace:
#0 {main}
  thrown in /home/standard/projects/chesslablab/chess-server/cli/amphp/hello-world.php on line 9

👉 What do I need to do specifically for the "Hello world" example to work as expected?
PHP Fatal error:  Uncaught Error: Call to undefined function Amp\async()

🙏 Thank you for the help.

Yes, that's the thing. I am just following the instructions in the docs.

Hi there,
👍 I can confirm that the docs examples work for me in a fresh install using version ^3.0 for amphp/amp so you may want to close this issue.
mkdir foo
standard@standard:~/projects/chesslablab$ cd foo
standard@standard:~/projects/chesslablab/foo$ composer require amphp/amp
./composer.json has been created
Running composer update amphp/amp
Loading composer repositories with package information
Updating dependencies
Lock file operations: 2 installs, 0 updates, 0 removals
  - Locking amphp/amp (v3.0.2)
  - Locking revolt/event-loop (v1.0.6)
Writing lock file
Installing dependencies from lock file (including require-dev)
Package operations: 2 installs, 0 updates, 0 removals
  - Installing revolt/event-loop (v1.0.6): Extracting archive
  - Installing amphp/amp (v3.0.2): Extracting archive
A connection timeout was encountered. If you intend to run Composer without connecting to the internet, run the command again prefixed with COMPOSER_DISABLE_NETWORK=1 to make Composer run in offline mode.
Generating autoload files
1 package you are using is looking for funding.
Use the `composer fund` command to find out more!
No security vulnerability advisories found.
Using version ^3.0 for amphp/amp
standard@standard:~/projects/chesslablab/foo$ php hello-world.php
Let's start: Hello World from the future!

However, if running the exact same command in the chesslablab/chess-server repository, it is version ^2.6 what will be installed instead of ^3.0.
composer require amphp/amp
./composer.json has been updated
Running composer update amphp/amp
Loading composer repositories with package information
Updating dependencies
Nothing to modify in lock file
Writing lock file
Installing dependencies from lock file (including require-dev)
Package operations: 0 installs, 0 updates, 1 removal
  - Removing revolt/event-loop (v1.0.6)
Generating autoload files
65 packages you are using are looking for funding.
Use the `composer fund` command to find out more!
No security vulnerability advisories found.
Using version ^2.6 for amphp/amp

Thus, if specifying the package version the following problems will occur:
composer require amphp/amp:3.0.2
./composer.json has been updated
Running composer update amphp/amp
Loading composer repositories with package information
Updating dependencies
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Root composer.json requires amphp/amp 3.0.2 (exact version match: 3.0.2 or <IP_ADDRESS>), found amphp/amp[v3.0.2] but these were not loaded, likely because it conflicts with another require.
  Problem 2
    - amphp/parallel v1.4.3 requires amphp/amp ^2 -> found amphp/amp[v2.0.0, ..., v2.6.4] but it conflicts with your root composer.json require (3.0.2).
    - chesslablab/php-chess 1.4.66 requires rubix/ml ^2.4 -> satisfiable by rubix/ml[2.5.1].
    - rubix/ml 2.5.1 requires amphp/parallel ^1.3 -> satisfiable by amphp/parallel[v1.4.3].
    - chesslablab/php-chess is locked to version 1.4.66 and an update of this package was not requested.

Use the option --with-all-dependencies (-W) to allow upgrades, downgrades and removals for packages currently locked to specific versions.

Installation failed, reverting ./composer.json and ./composer.lock to their original content.

This is because rubix/ml is locked to version 2.5.1.
See https://github.com/RubixML/ML/issues/305

@programarivm Good call, maybe we should add a version constraint to the install command?

Well I'm not too sure about this one at the moment. These errors shouldn't be too common unless there is a significant difference between versions ^2.6 and ^3.0 I believe.
If there are out there any libraries using previous versions you may want to just add an informative message.

This is the documentation for version 3. Please note that there are significant differences between versions 2 and 3, and you may encounter dependency issues if your codebase is using any Composer package depending on amphp/amp 2. If that is the case, try to specify the newest version on installation. composer require amphp/amp:3.0.2