Development: Make tutorial groups compatible with Postgres and reduce number of new change logs

Checklist
General

[x] I tested all changes and their related features with all corresponding user types on a test server.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Server

[x] I followed the coding and design guidelines.
[x] I added multiple integration tests (Spring) related to the features (with a high test coverage).
[x] I added pre-authorization annotations according to the guidelines and checked the course groups for all new REST Calls (security).
[x] I implemented the changes with a good performance and prevented too many database calls.
[x] I documented the Java code using JavaDoc style.

Motivation and Context

Fixed several problems in relation to the change to postgres and one bug I found in course resources
Note:

Description

Removed use of immutable set in course resource as it prevented update of the course object as a duplicated element error was thrown.
Changed from date type to string for several fields as there was a conversion problem when inserting string dates in "date" columns on Postgres. Change is no problem as the format is checked in the application to prevent poorly formatted date string insertion. Note: We can not use LocalDate as the type in the entity because it would be automatically converted to UTC when doing the insert which is not what we want here.
Same for time fields.

Steps for Testing

Create Tutorial Groups and Sessions -> No Bugs
Otherwise the integration tests should be enough

Review Progress
Code Review

[ ] Review 1
[ ] Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Exam Mode Test

[ ] Test

Test Coverage

Screenshots

Codecov Report

Merging #6001 (fb0a1bb) into develop (469b434) will increase coverage by 0.00%.
The diff coverage is 100.00%.

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

Additional details and impacted files
@@            Coverage Diff             @@
##             develop    #6001   +/-   ##
==========================================
  Coverage      79.90%   79.91%
- Complexity     12376    12378    +2
==========================================
  Files           2125     2125
  Lines          84487    84490    +3
  Branches       12680    12680
==========================================
+ Hits           67511    67516    +5
+ Misses          9439     9438    -1
+ Partials        7537     7536    -1

Impacted Files
Coverage Δ

...1/artemis/domain/tutorialgroups/TutorialGroup.java
83.72% <ø> (ø)

...s/domain/tutorialgroups/TutorialGroupSchedule.java
90.62% <ø> (ø)

...is/domain/tutorialgroups/TutorialGroupSession.java
85.71% <ø> (ø)

...e/tum/in/www1/artemis/web/rest/CourseResource.java
88.58% <100.00%> (ø)

...ctors/jenkins/jobs/JenkinsJobPermissionsUtils.java
95.52% <0.00%> (+3.33%)
:arrow_up:

Flag
Coverage Δ

client
75.77% <ø> (ø)

server
84.50% <100.00%> (+<0.01%)
:arrow_up:

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 469b434...6ce573e. Read the comment docs.


Show Wear on Applied Stickers in Market/Inventory

Last part is a duplicate of https://github.com/csfloat/extension/issues/137


matching translations to paragraphs of edition

@aditiagunawan — please describe the structural problem you mentioned yesterday. Is it a problem of how to encode paragraphs of translation, or is a question of display, or both?

@arlogriffiths - My question is how to Identifying correspondence between  and . If we look at the Svayambhu example in EGC 9.1.3,
 is associated with xml:id of a
 on CritEd (so we have
), not xml:id of  in CritEd (so not
. I see the HTML display for the translation button only has one per
. Will this not be a problem for chapters consisting of more than one
?

I made this example in the svayambhu, since I didn't know which part it was matching, i choose the higher one. For now, the display is based on the higher level, since I don't any translation to work with

Aditia has given a little bit of contents to his file DHARMA_CritEdSiksaKandangKaresian_transEng01. He can give more if that will be useful for testing purposes, but he will need your guidance on what to do with xml:id="SKK-01-01" etc.

@aditiagunawan
I have update the main @xml:idof your edition to skk. The ids should be provided in the main edition, I have added them a few days ago to allow the display of the document outline on the side.  Translation only used the attribute @corresp to align the paragraphs given in the translation with the ids given in the edition, and don't use ids to identify its component. To fill them, add an hashtag and then the id of the part, you are translating.
I have rewritten the attributes of the first paragraph in DHARMA_CritEdSiksaKandangKaresian_transEng01 to show you how it works. You can also use the section related to translation in the encoding guide: §9.1, the content are closed to the EGD, except for the parallelism between the files.
Let me know if you have question or facing issues.

@ajaniak
Thank you. I try to apply the same outline for the next three paragraphs. Please check my file and see if that is the correct way to correspond the translation to the main edition. I see that the transformation is work with the first paragraph but not (or should wait for moment?) with the next paragraphs.
Thank you.

@ajaniak It works and I like it a lot! Thank you Axelle.


container cap in older versions

Would y'all be okay bumping the container cap in 3.7 and 3.9 since those are the most used versions right now?

@Gl4di4torRr it doesn't really fulfill the characteristics of something we'd normally backport, but I guess i don't have a strong objection.  Note that it would also need to be backported to 3.10 since the PR merged into our 3.11(master) branch.  3.10 is in codefreeze right now, so it would need to be done after 3.10 GAs.

@gabemontero i removed all the jenkins 1 changes. i accidently closed #641 and just pushed the update and opened #642

no worries @Gl4di4torRr

@gabemontero everything is merged. yay! thank you for all your help yesterday updating the packages.

yw @Gl4di4torRr and thanks again for contributing


fix(proto): handle index less zero in low cardinality

Codecov Report

Merging #124 (280aef9) into main (fe50213) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##             main     #124   +/-   ##
=======================================
  Coverage   76.83%   76.83%
=======================================
  Files         120      120
  Lines        5676     5676
=======================================
  Hits         4361     4361
  Misses       1110     1110
  Partials      205      205

Impacted Files
Coverage Δ

proto/col_low_cardinality.go
66.31% <0.00%> (ø)

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 fe50213...280aef9. Read the comment docs.


Error: unsupported instruction `mov' when running "make all"

Full Error Message
$ make all
 BUILD: arch/x86/boot.S
arch/x86/boot.S: Assembler messages:
arch/x86/boot.S:78: Error: unsupported instruction `mov'
arch/x86/boot.S:80: Error: unsupported instruction `mov'
arch/x86/boot.S:84: Error: unsupported instruction `mov'
arch/x86/boot.S:86: Error: unsupported instruction `mov'
arch/x86/boot.S:88: Error: unsupported instruction `mov'
arch/x86/boot.S:94: Error: invalid instruction suffix for `push'
make: *** [makefile:24: build/obj/arch/x86/boot.S.o] Error 1

gcc,binutils Version
$ i686-elf-gcc -dumpmachine
i686-elf
$ i686-elf-gcc --version
i686-elf-gcc (GCC) 12.2.0
Copyright (C) 2022 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
$ i686-elf-as --version
GNU assembler (GNU Binutils) 2.39
Copyright (C) 2022 Free Software Foundation, Inc.
This program is free software; you may redistribute it under the terms of
the GNU General Public License version 3 or later.
This program has absolutely no warranty.
This assembler was configured for a target of `i686-elf'.

System Info
OS: Arch Linux x86_64
Host: HP EliteBook 840 G3
Kernel: 6.0.1-arch2-1
Uptime: 8 hours, 9 mins
Packages: 1123 (pacman)
Shell: zsh 5.9
Resolution: 1920x1080
DE: Plasma 5.26.0
WM: KWin
Terminal: yakuake
CPU: Intel i5-6200U (4) @ 2.800GHz
GPU: Intel Skylake GT2 [HD Graphics 520]
Memory: 4298MiB / 15883MiB

我猜测是binutils-2.39不再支持类似的指令了？请问我需要改用低版本的gcc和binutils吗？

Commonly speaking, this behaviour is usually caused by trying to compile a 32-bits asm code using 64-bits gcc. However, this surely contradicts your configuration.
My rough guess is there could be some alteration in the compiling process of gcc v12.2, perhaps, some switches/flags need to be flipped...? which will be investigated further.
And... yes, it is worth to try downgrade your toolchain. (Mine is gcc v11.2, as v2.37)
Should the issue remains, please let me know! Meanwhile, I will start investigating this issue and get you an update if any progress.

Ok... I have setup a toolchain as per your config. And unfortunately, I could not reproduce your issue.
However, there is an theoretically useless fix: Append a "-m32" flag to the $CFLAGS located in config/make-cc and see if the problem remains. This will force gdb to use 32bits ABI to compile your code.
Although, I believe it is unlikely to work as your gdb is already configured to i686 (hence "theoretically useless"), but it is worth to give a shot.
If that does not work as well. Then, you can post the boot.S source file so I can make sure that we are on the same page (i.e. you are compile the latest commit).

Commonly speaking, this behaviour is usually caused by trying to compile a 32-bits asm code using 64-bits gcc. However, this surely contradicts your configuration.
My rough guess is there could be some alteration in the compiling process of gcc v12.2, perhaps, some switches/flags need to be flipped...? which will be investigated further.
And... yes, it is worth to try downgrade your toolchain. (Mine is gcc v11.2, as v2.37)
Should the issue remains, please let me know! Meanwhile, I will start investigating this issue and get you an update if any progress.

Acctually after reinstalling gcc and binutils like the gcc-build.sh you provided, it just works.
I think all these errors are because I have gcc and binutils installed in two different directories, while they should be in the same place.**My fault!**😂


Fix unresponsive "forgot" link

Details

Pressing "Forgot" wasn't doing anything, this PR will fix that.
Fixed Issues

$ https://github.com/Expensify/App/issues/8924
Tests | QA steps

Navigate to staging.new.expensify.com
Enter a email in the email field > Click "Continue"
Click on "Forgot?"

[x] Verify that no errors appear in the JS console

PR Review Checklist

Contributor (PR Author) Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for Staging and/or Production testing in the QA steps section
[ ] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] iOS / native
[x] Android / native
[x] iOS / Safari
[x] Android / Chrome
[x] MacOS / Chrome
[x] MacOS / Desktop

[x] I verified there are no console errors (if there’s a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained “why” the code was doing something instead of only explaining “what” the code was doing.
[x] I verified any copy / text shown in the product was added in all src/languages/* files
[ ] I verified any copy / text that was added to the app is correct English and approved by marketing by tagging the marketing team on the original GH to get the correct copy.
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named “index.js”. All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] Any functional components have the displayName property
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose and it is

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn’t already exist
[ ] The style can’t be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.

PR Reviewer Checklist

[x] I verified the correct issue is linked in the ### Fixed Issues section above
[x] I verified testing steps are clear and they cover the changes made in this PR

[x] I verified the steps for local testing are in the Tests section
[x] I verified the steps for Staging and/or Production testing are in the QA steps section
[x] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[x] I checked that screenshots or videos are included for tests on all platforms
[x] I verified tests pass on all platforms & I tested again on:

[x] iOS / native
[x] Android / native
[x] iOS / Safari
[x] Android / Chrome
[x] MacOS / Chrome
[x] MacOS / Desktop

[x] I verified there are no console errors (if there’s a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I verified proper code patterns were followed (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained “why” the code was doing something instead of only explaining “what” the code was doing.
[x] I verified any copy / text shown in the product was added in all src/languages/* files
[x] I verified any copy / text that was added to the app is correct English and approved by marketing by tagging the marketing team on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named “index.js”. All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I verified that this PR follows the guidelines as stated in the Review Guidelines
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] Any functional components have the displayName property
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose and it is broken down into smaller components in order to separate concerns and functions

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn’t already exist
[ ] The style can’t be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.

Screenshots

Web

https://user-images.githubusercontent.com/83179295/167833311-f726fe7d-bdbc-4c32-bda5-bf2ee5a82682.mov
Mobile Web
https://user-images.githubusercontent.com/83179295/167835121-515d92a8-c4f9-4a53-ab9b-eae3937f2112.mov
Desktop
https://user-images.githubusercontent.com/83179295/167835026-93b5ddf0-3051-4dfa-9454-77a68f92d4a2.mov
iOS

https://user-images.githubusercontent.com/83179295/167835535-8b4735a2-8882-434d-a38d-43232d1ed9da.mov
Android

https://user-images.githubusercontent.com/83179295/167835547-6c23eb75-6063-41dc-bcda-366a4823cfc3.mov

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

Manually cherry-picking as I forgot to add the label

🚀 Cherry-picked to staging by @roryabraham in version: 1.1.57-13 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by @chiragsalian in version: 1.1.57-17 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


UNR-543 Fix default map loading

Contributions: We are not currently taking public contributions - see our contributions policy. However, we are accepting issues and we do want your feedback.

Description
Fixing a bug which caused the PIE clients booted from the editor to always use the GameDefaultMap and ignore the currently open man. This did not match Unreal's default behaviour and made map switching a pain. This could also lead to strange behaviour where the server loads one map and the clients another.
Tests
Tested by changing the GameDefaultMap / ServerDefaultMap and ensuring that the currently open map is used for both the server and the clients.
What automated tests are included in this PR?
Documentation
There is a known issue which can be removed.
Primary reviewers
@m-samiec @girayimprobable

@m-samiec Will handle that by altering Sahil's docs PR.


add label selectors to service attributes

When streaming the service entries (converted from services),
it is useful to have these values. Also, we will use this in
EDS code refactoring or even inside the registry code itself
when we need to access a service's selector labels instead of
looking up the k8s service/service entry.

@rshriram: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
8be5f1ced59b33c0a6418e099c2077f68e1c0e0f
link
/test gencheck_istio

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

@rshriram: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
8be5f1ced59b33c0a6418e099c2077f68e1c0e0f
link
/test gencheck_istio

lint_istio
8be5f1ced59b33c0a6418e099c2077f68e1c0e0f
link
/test lint_istio

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

@rshriram: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
8be5f1ced59b33c0a6418e099c2077f68e1c0e0f
link
/test gencheck_istio

lint_istio
8be5f1ced59b33c0a6418e099c2077f68e1c0e0f
link
/test lint_istio

unit-tests_istio
8be5f1ced59b33c0a6418e099c2077f68e1c0e0f
link
/test unit-tests_istio

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

I think in the follow up we can get rid of the logic about getting svc label selector from k8s svc cache.

That was the goal. I wanted to keep the PRs separate

/cherrypick release-1.6

@rshriram: #23997 failed to apply on top of branch "release-1.6":
error: Failed to merge in the changes.
Using index info to reconstruct a base tree...
M	pilot/pkg/model/service.go
M	pilot/pkg/serviceregistry/kube/controller/controller.go
A	pilot/pkg/serviceregistry/serviceentry/conversion.go
A	pilot/pkg/serviceregistry/serviceentry/conversion_test.go
Falling back to patching base and 3-way merge...
Auto-merging pilot/pkg/serviceregistry/kube/controller/controller.go
CONFLICT (content): Merge conflict in pilot/pkg/serviceregistry/kube/controller/controller.go
Auto-merging pilot/pkg/serviceregistry/external/conversion_test.go
Auto-merging pilot/pkg/serviceregistry/external/conversion.go
CONFLICT (content): Merge conflict in pilot/pkg/serviceregistry/external/conversion.go
Auto-merging pilot/pkg/model/service.go
Patch failed at 0001 add label selectors to service attributes

In response to this:

/cherrypick release-1.6

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@rshriram: new issue created for failed cherrypick: #24356

In response to this:

/cherrypick release-1.6

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


Not working on Firefox

Hi.
I have been testing my app in Chrome and Your parallax worked fine. But when I switched to Firefox, I got this error

TypeError: Argument 2 of EventTarget.addEventListener is not an object.

Also, there's link with solution from Firefox here

Fixed in #4


Unable to enable/disable extension per VS Code workspace

Before submitting your bug report

[X] I believe this is a bug. I'll try to join the Continue Discord for questions
[X] I'm not able to find an open issue that reports the same bug
[X] I've seen the troubleshooting guide on the Continue Docs

Relevant environment info
- OS: Linux
- Continue version: >= v0.8.61
- IDE version: VS Code 1.96.0

Description
It seems like you can't enable or disable the extension per-workspace anymore. This used to work with v0.8.60 and earlier, but not in v0.8.61  and later. It's also not working on the current pre-release v0.9.245 (pre-release).
To reproduce
Right click on the extension in the extension list and try to Enable (Workspace) or Disable (Workspace):

The options should be enabled and clickable.
Log output
No response

I experience the same on:
- OS: MacOS
- Continue version: v0.8.66
- IDE version: VS Code 1.96.0


DNS domain search

/etc/resolv.conf can specify search section that is used for domains without TLD.
For example on openshift, there is service foobar. If you try dig foobar you won't get an answer.
The proper host is foobar.something.svc.cluster.local. That domain is defined in the search section of /etc/resolv.conf.

This is crazy mad. :+1:


Investigate supporting SmartyStreets verification for International addresses

When our address verification tool was created, SmartyStreets did not support International addresses. Now that it is supported, we should review our code to determine if we can modify it to no longer strip out non-US addresses, and if so, update our code to properly support.

**lurch: add

Tracking W-021244


Add full slicing capabilities to the RMatrix object

At the moment the matrix object supports basic indexing operators $auto \  value = M_{i, j}$ and $M_{i, j} = value$ and so on, it also needs to support full slicing such as $M[i..j, k..l]$ and $M[i..j, k..l] = value$ and $M_{A}[i..j, k..l] = M_{B}$ and so forth. Also, revisit view operations as well as other design and implementation aspects of the RMatrix object.

There are a few things to think about for this implementation:

Value vs reference semantics such as views on a matrix. Is it useful to be able to pass around (immutable?) matrix slice references instead of directly subsetting matrices?
Using expression templates or something similar?

Done


Open cpus_per_task option for SLURM training

🚀 Feature
It would be great to open the cpus_per_task option for SLURM training to the hydra config. Right now it is inferred from the number of processes per node i.e. the number of GPUs per node with a baseline value of 8 CPU per process. See: https://github.com/facebookresearch/vissl/blob/master/vissl/utils/distributed_launcher.py#L259
Motivation & Examples
On the cluster I am using, we have 3 CPU per GPU in our octo-gpu nodes and 10 CPU per GPU in our quadri-gpu nodes. We need to be able to setup this value accordingly.

Adding a CPU_PER_PROC with 8 as default value should be enough I think. What is your opinion on this?

Adding a CPU_PER_PROC with 8 as default value should be enough I think. What is your opinion on this?

The alternative would be to use the value inside config.DATA.NUM_DATALOADER_WORKERS since it parameterise the number of threads that will be popped to deal with the construction of batches.
I am not too sure which is best: it might not be that NUM_DATALOADER_WORKERS is the best amount of threads (maybe we just need one half), but on the other side, asking for 8 CPUs to use only 4 NUM_DATALOADER_WORKERS is probably a waste.
@OlivierDehaene, @prigoyal what do you think?

Indeed a nice feature request. I do think it makes sense for us to open the parity with the SLURM features in VISSL. Regarding #data workers and #cpus, this is something I think users should configure themselves and we should offer full flexibility.
@OlivierDehaene , @QuentinDuval , please feel free to grab the issue if you are interested in doing this. :)

The alternative would be to use the value inside config.DATA.NUM_DATALOADER_WORKERS since it parameterise the number of threads that will be popped to deal with the construction of batches.

I think it is better to have an independent parameter and raise an error or a warning if NUM_DATALOADER_WORKERs > CPU_PER_PROC. The reason I would prefer this implementation is because on our cluster, if we ask for more than 3 or 10 CPUs, SLURM will allocates more than the required number of nodes.

Also, it would be great to add an option to pass SLURM_ ADDITIONAL_PARAMETERS to submitit. In my case I needed to add {"account": ****}.
Overall the changes would be:
defaults.yaml
CONFIG:
...
    SLURM:
        CPU_PER_PROC: 8
        ADDITIONAL_PARAMETERS: {}
        ...

vissl/utils/distributed_launcher.py
executor.update_parameters(
        name=cfg.SLURM.NAME,
        slurm_comment=cfg.SLURM.COMMENT,
        slurm_partition=cfg.SLURM.PARTITION,
        slurm_constraint=cfg.SLURM.CONSTRAINT,
        timeout_min=cfg.SLURM.TIME_HOURS * 60,
        nodes=cfg.DISTRIBUTED.NUM_NODES,
        cpus_per_task=cfg.CPU_PER_PROC * cfg.DISTRIBUTED.NUM_PROC_PER_NODE, #modified
        tasks_per_node=1,
        gpus_per_node=cfg.DISTRIBUTED.NUM_PROC_PER_NODE,
        mem_gb=cfg.SLURM.MEM_GB,
        slurm_additional_parameters=cfg.ADDITIONAL_PARAMETERS #added
    )

And then I can launch with:
./dev/launch_slurm.sh \
config=pretrain/barlow_twins/barlow_twins_4node_resnet.yaml \
config.SLURM.CPU_PER_PROC=10 \
config.SLURM.ADDITIONAL_PARAMETERS.ACCOUNT=****

looks great to me. @OlivierDehaene , do you want to open a PR with your proposed changes? :)

@OlivierDehaene , just an update: we are looking into this task ourselves and will keep posted on this issue of the progress.

@prigoyal, Great! Sorry, I don't have time to work on VISSL at the moment.

Hi @OlivierDehaene, I am new to the vissl team. I was able to get to your request here: https://github.com/facebookresearch/vissl/pull/282/files. Please look at the testing steps of the PR on how to use and let me know if you have any issues.
Thanks for raising this issue and feel free to reopen if you have any more questions!


Error when running preview on Windows

Describe the bug
I am using Windows. The following error occurred during the tutorial https://flowershow.app/docs/publish-tutorial.
> npx flowershow@latest preview
⚠ This may not work as expected. You're trying to run Flowershow CLI on Windows, which is not thoroughly tested. Please submit an issue if you encounter any problems: https://github.com/flowershow/flowershow/issues

><EMAIL_ADDRESS>predev
> npm run generate

><EMAIL_ADDRESS>generate
> contentlayer build && cross-env NODE_OPTIONS="--experimental-json-modules --experimental-modules" node ./scripts/search.mjs

Warning: Contentlayer might not work as expected on Windows
Error: Found 1 problems in 1 documents.

 └── Encountered unexpected errors while processing of 1 documents. This is possibly a bug in Contentlayer. Please open an issue.

     • "index.md": UnexpectedMDXError: Error: Build failed with 1 error:
     error: Cannot find tsconfig file "undefined/tsconfig.mdx.json"

To Reproduce
Steps to reproduce the behavior:

Create parent directory

Create new vault (Obsidian GUI)

Vault name is my-digital-garden
Location is some-parent-dir

Create folder assets (Obsidian GUI)

Set assets as attachment folder (Obsidian GUI)

Create new index.md note (Obsidian GUI)

Run cd some-parent-directory

Run npx flowershow@latest install

? Create Flowershow project in current directory? Yes
? Path to the folder with your markdown files my-digital-garden
? Select a folder with your assets (attachments) assets

Run npx flowershow@latest preview

The error described above occurs.

Versions

OS: Windows 10
Node: v18.14.2
Npm: 9.5.0
PowerShell: 7.3.2

INVALID. WONTFIX. we are no longer using contentlayer so this isn't relevant anymore.


Wrong fieldname in script?

@MarkPrinsen48 commented on Thu May 23 2019
Shouldn't "StrongAuthenticationRequirements" be "StrongAuthenticationMethods"?

Documentdetails
⚠ Dit gedeelte niet bewerken. Het is vereist om problemen te koppelen tussen docs.microsoft.com en GitHub.

ID: 4cb129d2-9594-bf0c-5946-982f0ed0d888
Version Independent ID: 6522b038-6853-3552-55c8-f174cf56573d
Content: Azure multi-factor Authentication gebruikersstatussen - Azure Active Directory
Content Source: articles/active-directory/authentication/howto-mfa-userstates.md
Service: active-directory
Sub-service: authentication
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

Hi team,
I moving this here since the contributor is making a suggestion on the code
best

Hi @CeciAc and @MarkPrinsen48
The script disables MFA by setting StrongAuthenticationRequirments to an empty array, and it removes the StrongAuthenticationMethods.
So I believe it should be correct. CC @MicrosoftGuyJFlo

Adding to this -
The part that says 'To disable MFA, use this script:' has this code below:
Get-MsolUser -UserPrincipalName<EMAIL_ADDRESS>| Set-MsolUser -StrongAuthenticationRequirements @()
This should be -StrongAuthentationMethods . I've run a test, StrongAuthenticationRequirements can be blank with StrongAuthenticationMethods set. Blanking StrongAuthenticationRequirements does nothing, while blanking StrongAuthenticationMethods disables MFA.


bug: Key press transfers to the next Input when setFocus in used inside onKey

Note: I'm not sure if this is a bug or expected behaviour.
Describe the bug
Say you have inputs A and B. If you use B:setFocus() inside of A:onKey, pressing a key inside of A will output it to B.
To Reproduce

Run the following basalt snippet:

local A = main_frame:addInput("A")
	:setPosition(1, 1)
	:setSize(20, 1)
	:show()

local B = main_frame:addInput("B")
	:setPosition(1, 2)
	:setSize(20, 1)
	:show()

A:onKey(function(self, event, key)
	B:setFocus()
end)

Click on Input A, and press 'a'.

The output 'a' will be produced to B.
Expected behavior
Again... I'm now sure if this is a bug, or expected behaviour. However, I expected the 'a' to appear in the first input, and then the focus be set to B.
Screenshots
https://user-images.githubusercontent.com/43142209/194728378-e883cf94-82d6-4962-ac02-7611494d73df.mp4
Checklist
[x] I am running the latest version.
Tick the box if you are running the latest version!

Hey! This isn't a bug. :setFocus sets the object immediatley as the focused object. The reason why your custom onKey event handler happens before the inputbox's key event handler is because of the following code:
A:onKey(function(self, event, key)
   if(key==keys.a)then
      return false
   end
end)

this code would make it impossible to write "a" into the inputbox. A possibility would be to create a delayed task:
A:onKey(basalt.schedule(function(self, event, key)
  sleep(0.1)
  B:setFocus()
end)

This code would wait 0.1s before changing the focus.
https://basalt.madefor.cc/#/objects/Basalt/schedule

I'd also be fine with the key not being recorded by any of them. My major problem is that the key 'passes over' to B.
Is there a way to avoid that?

A:onKey(function(self, event, key)
  B:setFocus()
  return false
end)

This would make it impossible to write into the input "A" - i am not sure if that's what you want.

Not exactly. What I want is to capture a specific key, say ., and move to the next input after ..
However, using this code:
A:onKey(function(self, event, key)
	if key == 46 then
		B:setFocus()
	end
end)

ends up with the dot in B. Again, the problem is not that the output does or doesn't go to A. The problem is that the output goes to B:
https://user-images.githubusercontent.com/43142209/194730095-41655867-ce9d-4d46-9d22-4daf4a5f2da2.mp4
The output I expected was "This is a sentence." in A, and nothing in B. If the code had a return false, I would expect the output to be "This is a sentence" in A, and nothing in B.
The sleep solution feels... suboptimal. I feel like the output should just not be recasted to B (i.e. instead of outputting to the currently focused object, it should output to the target object of the keypress). More specifically:

If return == false, then no output should be produced in either.
If return ~= false, then the output should be produced to A, not to B.

Hello, thank you very much. I understand now why this is happening. Basalt 1.6.3 has the fix for it - i seperated the char events from the key events, this menas you would need to use a :onChar() event instead of :onKey().
If you are curious on how the events work:
I am using the cc:tweaked event "char" for changing the input. The problem here was the key event always happens before the char event, and i decided one time to call both, the key and the char event when someone uses :onKey(f) which means in your case, you click the ".", the key loop runs, you make the b-input to be the focused one, then the char event loop runs and it writes the . into the b-input (because its focused).

Just checked. Everything is working as intended now!
Thank you for the quick fix, and thank you for the compliment! I'm using basalt for a personal project, so expect to hear from me again! Cheers


Ad Auth - Can't login with groups assigned to clusters that contain special characters.

Issue:
Add AD Auth with a group that contains special characters.
Set site access to "Allow members of Clusters, Projects, plus Authorized Users and Organizations"
Assign a group that contains special characters.
Attempt to login as a user that has special characters in their name from the group.
Notes:
https://github.com/rancherlabs/the-ranch/wiki/Authentication-Test-Setup-configuration
Used test data AD with TLS enabled
Group: test(group&)abc
User: test(san)&abc
Results:
Unable to login.

This also occurs if the group is assigned to a project.

Verified fix
Master 4/13/2018


[keyvault] Change over to using core-tracing

Change keyvault to use the core-tracing package's version of createSpan. There's an upcoming update to the opentelemetry package in the repo that would require some updates to everyone's custom createSpan implementation.
Rather than do that I'm just centralizing it all before the package update so future updates should be more minimal.
For the most parts the updates just remove an unneeded RequestOptionsBase conversion. So for the most part the code is simpler. I've added in some TODO comments to point out areas where I'm unsure what we should do so please check them out.
This PR is built on top of a "pending" PR so you'll want to just pay attention to commits from here onward: https://github.com/Azure/azure-sdk-for-js/commit/3aaf2247d55af223e28a2913c63fd96b64a9ad30

Fixed merge conflicts. There was some options blending, that's been updated to still use the createSpan() methodology that I had before.

/azp run js - keyvault-admin - tests

/azp run js - keyvault-certificates - tests

/azp run js - keyvault-keys - tests

/azp run js - keyvault-secrets - tests

/azp run js - keyvault-secrets - tests

/azp run js - keyvault-keys - tests

/azp run js - keyvault-certificates - tests

/azp run js - keyvault-admin - tests

Looks good, but since it's a long PR, let's wait for Maor's review before this gets merged. Perhaps the wait can also help fix the CI issue!

The test failure above is a failure to setup the HSM resource, not related to code changes.

I kicked off another build of KV Admin. The Managed HSM is still in preview but more recent builds have deployed it successfully so let's see how that goes...


Support LEFT(), RIGHT()

I wasn't able to use PDO::sqliteCreateFunction() for LEFT/RIGHT because these are SQL keywords, so it would be great to use the methodRewrite functionality in this package to wrap these keywords in backticks and then implement the function extensions in StringExtended
Incoming MR 🙂

@spam-n-eggs is this project abandoned? i can just maintain a separate fork if that's easier for you. i know open source projects are cool at first and then are a pain to maintain, esp if you're not using Laravel or PHP anymore 😆

Hi Dave,
The project's not abandoned, I just haven't had a ton of time to maintain it.  I'm going to (eventually) move it to GitLab CI and run it in our FOSS group over there.  You just got me at a bad time - my parents just drove across the country over the past couple days, and as luck would have it, the power supply for my Dominion Solutions laptop died at the same time.  I'll take a look at your PR this morning.
Mark Joseph Horninger,
Principal Engineer, Dominion Solutions LLC
+1<PHONE_NUMBER>
@.***
https://dominion.solutions
---- On Wed, 12 Apr 2023 16:51:31 -0400 Dave Rogers @.***> wrote ---
https://github.com/spam-n-eggs is this project abandoned? i can just maintain a separate fork if that's easier for you. i know open source projects are cool at first and then are a pain to maintain, esp if you're not using Laravel or PHP anymore 😆
—
Reply to this email directly, https://github.com/dominion-solutions/laravel-mysqlite/issues/104#issuecomment-1505921242, or https://github.com/notifications/unsubscribe-auth/ADERZCBACP6JDGYKTGWIWGDXA4IVHANCNFSM6AAAAAAWWDAG7M.
You are receiving this because you were mentioned.


Fix 302 validation error first

I'm getting this validation error because of DB names, that contain caracters such "$" or "(". Properties are well refactorized when I first run scaffold-DbContext, but this error prevents from changing rules.json file.
Will you fix it? Thanks a thillion!

What DBMS are you using?  I'm surprised that you can use those characters in the column names.  I can adjust the validation to accommodate the actual allowed characters set.

Updated validation to support full ASCII character set.
Update to v1.2.21 and latest VS extension

DBMS is Azure SQL. Astounding, isn't it? Not my DB, not my headaches...
Thank you very much!

Relaxed the naming validation further.  Now, anything is allowed in the DB Name.  That certainly isn't considered best practice though.
Latest build should work.

Tested and working fine. thanks!
I absolutely agree to not being this naming best practice, since it's an awful practice. But I should say that it's what you get whenever you work with Bussiness Central extensions by Microsoft, where, due to its policy naming, field captions stand for field names. In my discharge, I say my duty here is to extend a third part extension from outside (you can say something of hacking but only with data). In this regard, I'd like to ask if there is any issue when defining custom FKs when you only have read permission (third part grant me only read permission). I bet it's my fault in setting the FKs in JSON file, but I cannot find the problem and I wonder if that could be, since I make valid FKs through code.
Again, thanks a lot.

Generally, if you have a custom foreign key defined in the EDMX then all the necessary configurations will be added to the json file.  At the root of the model, the FK itself must be defined as shown:

Then the constraint reference should be made from the custom navigation:

As long as the FK is configured properly on compatible key fields it should work.  The scaffolding output (log) may indicate any issue that was encountered.


StreamPipeWriter.Complete should throw if there's unflushed data

StreamPipeWriter wraps a stream and pretends to be a pipe, but it buffers everything in memory. On a normal Pipe if you call Complete then the data would be available to the consumer, but for StreamPipeWriter the data is all still sitting in the buffers.
Throw an InvalidOperationException if FlushAsync is not called to flush the buffered data before Complete is called.
@jkotalik

Who is responsible for calling Complete on the StreamPipeWriter? Is Complete required to be called before Dispose?


Refacto gql queries to use variables instead of string

As in the doc of gql : https://gql.readthedocs.io/en/stable/usage/variables.html
We can use variables with $ instead of doing string.format()
See gql_create_graph_pipeline in gql_template for example

Trello card: [#53] Refacto gql queries to use variables instead of string

Trello card: [#53] Refacto gql queries to use variables instead of string


sokol_gfx_imgui: crash in _sg_imgui_draw_uniforms_panel

in _sg_imgui_draw_uniforms_panel when pip== NULL, it crashes on the line:
_sg_shader_t* shd = _sg_lookup_shader(&ctx->sg->pools, pip->shader_id.id);

True, without pipeline or shader it's not possible to display anything useful anyway, so I'm returning early now in that case.


[BUG]

Description
In dev mode, when there's no data loaded, we currently have an alert box on the home page. However, it's not appearing at the  location, which is at the top of the page under the main navbar. This issue is about relocating the alert box to the top.
Screenshots

Do you want to work on this issue?
Yes
Additional information
No response

Thank you for raising the issue, I have assigned it to you 👍


[QUESTION] Would containerClassName be more useful applied to a div vs. a span element?

Thanks for this simple library! Very quick to implement and saved me a ton of time :).
I'm trying to style my video embed player and finding that the <span> element means that I can't set widths like I want to using the containerClassName prop. Is there a reason that the component is wrapped in a <span> vs. being wrapped in a <div> element? Happy to submit a PR to make the change if that makes sense and is helpful.

Initially we did not offer a containerClassName prop and wrapped the youtube player with a span. Because of the default behaviour of the span it is possible to just apply styles to the player directly and not have to worry about the span being there.
containerClassName was mostly added for those rare cases where one does have to style the span element to achieve a certain behaviour.
Since the span is meant as a non-obtrusive wrapper I'd like to keep it that way. If you want to give the player a width you should apply that directly to the player. If you really have to give the width to the span apply display: inline-block...
PS
the reason for having a wrapper in the first place is so that react can have an element to remove during unmounting. Without a wrapper the modifications by the youtube iframe api make React loose track of the dom element.


Allow customizing the arguments passed to the verify backend

The verification backend now can receive arguments passed as
an array in the user configuration.
Currently the configuration seems to be loaded only once, and
the onDidChangeConfiguration event handler seems not to be
appropriately hot-reloading the settings.
Current workaround: reload the extension by reloading the window.
This addresses #41

It seems that sonnar-scanner command is not available on Travis and it fails.

Thanks @ssaavedra!


v2: try to fix some constantly failing tests

Because it is a bit annoying having to rerun tests all the time, this PR is looking into if we can make it so that we rerun tests less in future.
On another PR I kept getting failures like this one: https://github.com/bluesky/ophyd/actions/runs/4605633309/jobs/8137912252
In the last collaboration meeting it was suggested to move this smoke test to the bottom of the file and see if that changes anything. Either way it would be good to do some investigation here into why the tests are so slow. Perhaps we need to increase the timeout if nothing else can be done?

it's like playing whack-a-mole...
https://github.com/bluesky/ophyd/actions/runs/4667812788/jobs/8264070337


init should be implemented by the macro

Raised due to comment: https://github.com/fortanix/rust-mbedtls/pull/128#discussion_r539563306
Need to update init/new macro so that it can do more advanced logic as needed in ctr_drbg / context / config new functions

Close this since this is already done


Click function

Why the click function don't works with this toggle? I need to use a function like this :
 $('checkbox').click(function(){
var valor = $(this).attr('id');
});
how can i use this?

Use the API to listen for the change event...
$('.mycheckboxes').change(function() {
  var valor = $(this).attr('id');
  //you can check its checked state with $(this).prop('checked');
})

But i need the click event, is there any way ?

It will have the same effect.  Clicking a checkbox will change it (unless you e.preventDefault() it from functioning that way) so you are listening for that change event to fire.    What exactly is the issue you have that demands you use .click() ?  I'm trying to think of a case when the change() wouldn't work and I can't.

I want to confirm action before actually toggle the checkbox. Here is a similar case with good explaination.
And because I ask server, not user, I think that toggling checkbox back (if unconfirmed) will look bad.
 $("#myDiv").find('input').click(function(event){
    var $this = $(this);
    event.preventDefault();
    $.post(node.data.changeStateUrl,
        {'id':node.key, 'newState':$this.prop('checked')},
        function(){ $this.prop('checked', !$this.prop('checked')).change(); }
    );
});

any update on this?

You can change status back without triggering change event : #135


Still search is not working well

Search is a vital option for your software sir. I think you should fix it first.

Fix for search has already been committed and I will release a new version very soon which will include a proper search function.

@ashik685  Download latest version of the app. It includes a fix for search option.

search is still not working fine , i think we need to load all courses ?

@vikasnewports What issue are you facing with course search? And no you don't need to load all courses.

Now need another option , remember me option. So that we don't need to login always

Search is working fine :+1:


motion sensor not sending home assistant discovery packets

Hey there - thanks a billion for this work. Allows me to stop using my janky python library I wrote years ago on top of @whaleygeek's library.
Just set this up, and I've got it working nicely, and I can see everything I'd expect in the raw energenie topic; but for some reason there doesn't seem to be any discovery info sent to the homeassistant topic for the Energenie motion sensor I have. Its in the raw topic, but not discovery.
Anything I should be looking out for?

Nevermind, seems I was just too impatient and hadn't triggered the sensor enough for it to have a discovery packet sent yet. Has come up now after about 30 min of it running.

Discovery messages are sent every 10 minutes during normal running. The motion sensor only sends out a packet if it detects/clears motion, so 20 mins is usually long enough to wait for it to appear.  Glad you got it sorted. :)

Thanks again :)


Add published validation

Checklist

[x] Appropriate branch selected (all PRs must first be merged to dev before they can be merged to master)
[x] Any modified or new methods or classes have helpful JavaDoc and code is thoroughly commented
[x] The description lists all applicable issues this PR seeks to resolve
[x] The description lists any configuration setting(s) that differ from the default settings
[x] All tests and CI builds passing

Description
This PR adds the publishedValidationSummary field to the FeedSource model. Also, the feedVersionId and namespace fields are added to the FeedValidationResultSummary class. These things help with rendering the refactored FeedSourceTable (issue: https://github.com/conveyal/datatools-ui/issues/422) (pr: https://github.com/conveyal/datatools-ui/pull/429).

I can't approve my own PR, but I'd say let's merge this after addressing my one comment on the changes.

Codecov Report

Merging #195 into dev will decrease coverage by 0.06%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##              dev    #195      +/-   ##
=========================================
- Coverage     6.2%   6.13%   -0.07%
  Complexity    115     115
=========================================
  Files         133     133
  Lines        6962    7039      +77
  Branches      913     928      +15
=========================================
  Hits          432     432
- Misses       6494    6571      +77
  Partials       36      36

Impacted Files
Coverage Δ
Complexity Δ

...ls/manager/models/FeedValidationResultSummary.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

.../conveyal/datatools/manager/models/FeedSource.java
5.91% <0%> (-0.24%)
3 <0> (ø)

.../conveyal/datatools/manager/models/Deployment.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...conveyal/datatools/manager/models/FeedVersion.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...s/editor/controllers/api/EditorControllerImpl.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

...tools/editor/controllers/api/EditorController.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

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 853ddb8...4d2d0e4. Read the comment docs.

:tada: This PR is included in version 3.4.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Update Heroku configuration

Hello,
I was looking for having my own CAS server for developing a PHP CAS library and I found out that Heroku was the perfect place to set this up.
Unfortunately, the demo server provided by Apereo do not have the Proxy authentication enabled.
So, I forked this project, updated the branch heroku-caswebapp and did some modifications so I can have the Proxy authentication enabled on my server: https://psrcas-server.herokuapp.com/cas/login
I also did some tidying up in the repo and I found out that it was much easier to edit a configuration file than editing a list of parameters in the Procfile file.
This PR:

[x] Update the Procfile file, mandatory for Heroku
[x] Use a cas.properties file instead of using inline parameters in Procfile.
[x] Does not have tests

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 3 committers have signed the CLA.:white_check_mark: mmoayyed:x: Misagh Moayyed:x: drupolMisagh Moayyed seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


how to upload an *.apk but wont rename to *.zip

flagrow/upload extension version: <0.6.0>
flarum version: 0.1.0-beta.7
upload adapter causing issues: local
mimetype configurations: (application/(apk|exe))

When i upload an fuckmyself.apk, it rename to fuckmyself.zip.
How can i configer it that when i upload an fuckmyself.apk and download it still named fuckmyself.apk.

Duplicate of https://github.com/flagrow/upload/issues/99


Fixed typo in date-and-timeofday-support.md

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

File
Status
Preview URL
Details

Odata-docs/webapi/date-and-timeofday-support.md
:white_check_mark:Succeeded
View

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


fix: run po for longtask only once

Thank you for this huge fix!

@SuperOleg39 if you have a minute, would you mind have a PR with tests as well, please.

if you have a minute, would you mind have a PR with tests as well, please.

Wiil try! This case looks like more for some integration in-browser test, but maybe units will be enought, need to check

@Zizzamia ready https://github.com/Zizzamia/perfume.js/pull/217


Could not build wheels for pysqlite3

I am trying to install pysqlite3 on my Macbook Pro M2 Pro Apple Silicon Chip Machine. However everytime I try to install pysqlite3 using
pip3 install pysqlite3
I get the Following Error:
note: This error originates from a subprocess, and is likely not a problem with pip.   ERROR: Failed building wheel for pysqlite3   Running setup.py clean for pysqlite3 Failed to build pysqlite3 ERROR: Could not build wheels for pysqlite3, which is required to install pyproject.toml-based projects
I am able to Install the Dependency on my older Macbook a Intel Based Chip but installing this dependency on the M2 Pro Chip Machine gives this above issue.
What Steps I can follow to Solve this Issue ?

same issue here, no solution yet :(

I am trying to install pysqlite3 on my Macbook Pro M2 Pro Apple Silicon Chip Machine. However everytime I try to install pysqlite3 using
pip3 install pysqlite3
I get the Following Error:
note: This error originates from a subprocess, and is likely not a problem with pip. ERROR: Failed building wheel for pysqlite3 Running setup.py clean for pysqlite3 Failed to build pysqlite3 ERROR: Could not build wheels for pysqlite3, which is required to install pyproject.toml-based projects
I am able to Install the Dependency on my older Macbook a Intel Based Chip but installing this dependency on the M2 Pro Chip Machine gives this above issue.
What Steps I can follow to Solve this Issue ?

i have the same issue, no idea why. but interestingly, i can just use sqlite anyway, even when pipenv failed to lock because of that error, very weird imo.

Same issue here and also not working afterwards.
Collecting pysqlite3
  Using cached pysqlite3-0.5.4.tar.gz (40 kB)
  Installing build dependencies ... done
  Getting requirements to build wheel ... done
  Preparing metadata (pyproject.toml) ... done
Building wheels for collected packages: pysqlite3
  Building wheel for pysqlite3 (pyproject.toml) ... error
  error: subprocess-exited-with-error

  × Building wheel for pysqlite3 (pyproject.toml) did not run successfully.
  │ exit code: 1
  ╰─> [60 lines of output]
      running bdist_wheel
      running build
      running build_py
      creating build/lib.macosx-14.0-arm64-cpython-312/pysqlite3
      copying pysqlite3/__init__.py -> build/lib.macosx-14.0-arm64-cpython-312/pysqlite3
      copying pysqlite3/dbapi2.py -> build/lib.macosx-14.0-arm64-cpython-312/pysqlite3
      running build_ext
      Builds a C extension linking against libsqlite3 library
      building 'pysqlite3._sqlite3' extension
      creating build/temp.macosx-14.0-arm64-cpython-312/src
      clang -fno-strict-overflow -Wsign-compare -Wunreachable-code -fno-common -dynamic -DNDEBUG -g -O3 -Wall -isysroot /Library/Developer/CommandLineTools/SDKs/MacOSX14.sdk -Qunused-arguments -DMODULE_NAME=\"pysqlite3.dbapi2\" -I/usr/include -I/Users/ql/venvs/foursquare-swarm-ical/include<EMAIL_ADDRESS>-c src/blob.c -o build/temp.macosx-14.0-arm64-cpython-312/src/blob.o
      clang -fno-strict-overflow -Wsign-compare -Wunreachable-code -fno-common -dynamic -DNDEBUG -g -O3 -Wall -isysroot /Library/Developer/CommandLineTools/SDKs/MacOSX14.sdk -Qunused-arguments -DMODULE_NAME=\"pysqlite3.dbapi2\" -I/usr/include -I/Users/ql/venvs/foursquare-swarm-ical/include<EMAIL_ADDRESS>-c src/cache.c -o build/temp.macosx-14.0-arm64-cpython-312/src/cache.o
      clang -fno-strict-overflow -Wsign-compare -Wunreachable-code -fno-common -dynamic -DNDEBUG -g -O3 -Wall -isysroot /Library/Developer/CommandLineTools/SDKs/MacOSX14.sdk -Qunused-arguments -DMODULE_NAME=\"pysqlite3.dbapi2\" -I/usr/include -I/Users/ql/venvs/foursquare-swarm-ical/include<EMAIL_ADDRESS>-c src/connection.c -o build/temp.macosx-14.0-arm64-cpython-312/src/connection.o
      src/connection.c:1395:10: error: call to undeclared function 'sqlite3_enable_load_extension'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
       1395 |     rc = sqlite3_enable_load_extension(self->db, onoff);
            |          ^
      src/connection.c:1395:10: note: did you mean 'pysqlite_enable_load_extension'?
      src/connection.c:1382:18: note: 'pysqlite_enable_load_extension' declared here
       1382 | static PyObject* pysqlite_enable_load_extension(pysqlite_Connection* self, PyObject* args)
            |                  ^
       1383 | {
       1384 |     int rc;
       1385 |     int onoff;
       1386 |
       1387 |     if (!pysqlite_check_thread(self) || !pysqlite_check_connection(self)) {
       1388 |         return NULL;
       1389 |     }
       1390 |
       1391 |     if (!PyArg_ParseTuple(args, "i", &onoff)) {
       1392 |         return NULL;
       1393 |     }
       1394 |
       1395 |     rc = sqlite3_enable_load_extension(self->db, onoff);
            |          ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
            |          pysqlite_enable_load_extension
      src/connection.c:1419:10: error: call to undeclared function 'sqlite3_load_extension'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
       1419 |     rc = sqlite3_load_extension(self->db, extension_name, 0, &errmsg);
            |          ^
      src/connection.c:1419:10: note: did you mean 'pysqlite_load_extension'?
      src/connection.c:1405:18: note: 'pysqlite_load_extension' declared here
       1405 | static PyObject* pysqlite_load_extension(pysqlite_Connection* self, PyObject* args)
            |                  ^
       1406 | {
       1407 |     int rc;
       1408 |     char* extension_name;
       1409 |     char* errmsg;
       1410 |
       1411 |     if (!pysqlite_check_thread(self) || !pysqlite_check_connection(self)) {
       1412 |         return NULL;
       1413 |     }
       1414 |
       1415 |     if (!PyArg_ParseTuple(args, "s", &extension_name)) {
       1416 |         return NULL;
       1417 |     }
       1418 |
       1419 |     rc = sqlite3_load_extension(self->db, extension_name, 0, &errmsg);
            |          ~~~~~~~~~~~~~~~~~~~~~~
            |          pysqlite_load_extension
      2 errors generated.
      error: command '/usr/bin/clang' failed with exit code 1
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
  ERROR: Failed building wheel for pysqlite3
Failed to build pysqlite3


add a new branch

www.shjgu.edu.cn

[ ] ？

@L049 Please provide us with:

the school official website URL
the school street address, including city and country
an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course
a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.

@L049 You might miss my previous comment. Please react.

www.shjgu.edu.cn （ offical website URL）
2859 Shuichan Road, Baoshan District, Shanghai, China {post code:201901}(address)
Home page of Mechanical and Electrical Department of Shanghai JiGuang Polytechnic College
Picture Class Schedule of Grade 21 in the Second Semester of Mechanical and Electrical Department in 2021-2022 Academic Year
Class Schedule of Grade 21 computer network technology class 1 curriculum
The official email portal is located directly above the school's homepage, and the image I have provided illustrates this entrance. This is the official email home page.

@L049 Thank you for your patience while waiting for a review and processing your request. Pull request merged.


Browser Version

Hi,
It looks like a very useful plugin.
I  want to try this in a standard Javascript in browser, without typescript, import, export.
Your browser version is in Typescript.
Can you provide the standard JS to work in browser and code snippet?
Thanks,
Muthu

Not sure I understand what you mean. This example is in JavaScript: https://github.com/alonrbar/easy-template-x#browser-example.
If you are trying to run it as-is, you are right it won't work because async/await is not supported on top level code. You need to wrap the code with an async function, like this:
async function main(){
  const response = ...
  ...
}
main()

Does that answer your question?

Thanks for the quick response.
Since i couldn't find the dist/compiled JS, i refer from here - https://www.jsdelivr.com/package/npm/easy-template-x
In my page, i have reference to<EMAIL_ADDRESS>and my js file with your browser-example snippet.
Since few other references are missing (not bundled) i get below error

Note: I use ES6 Javascript version (not typescript). If you could give me a link to refer your plugin (with all references) that would be really great.
Thank you.

Hi Alonrbar,
Sorry if i still confusing you.
I am just wondering if there is compiled version (distributed file) that i can refer in my HTML form to work in browser.
Thanks.

Hi,
You cannot consume easy-template-x by referencing directly it in a script tag. You need to use a JavaScript bundler like Webpack or Parcel.
I also encourage you to checkout the live demo which demonstrates how to do what you want. You can follow the instructions I wrote here to run it locally (it relies on Parcel for bundling and running a dev server).

Duplicate of: https://github.com/alonrbar/easy-template-x/issues/47

Thanks Alonrbar.


SASS Support

Next.js allows importing .scss and .sass files, details here: https://nextjs.org/docs/basic-features/built-in-css-support#sass-support


Fix leftElement to work with any element and not just icon names

Fix for issue #45 .

I agree, check last commit.


Current model has no Policy/Compliance/Governance module

One of the core components of any platform is ultimately answering: how does any/all of this fit into an organization's overall security posture. Especially these days with everyone marching, in some way shape or form, towards ZTA, I think having the built in tooling to help enable that and even better accelerate it, is extremely important.
As the current model stands it does not incorporate any tooling around this. I believe there should be some thinking put into what should be added in. From the CNCF side OPA and Kyverno are in play. Both have their merits and shortcomings, however, I think it would benefit the project and the overall movement if there is some discussion on the topic. Being able to say things like: you get NIST 800-53 out of the box, or PCI, or insert any of the industry standards we all have to one way or another find a way to adhere to, is going to be just as important as being able to spin up a platform I can build on top of.

closing and moving over to https://github.com/cnoe-io/idpbuilder


Got error when click on a component which created before vue-router installed.

Hey, can you boil down the error and provide a repro using this template?
Thanks!

@posva Can't provide one with codepen or some other online environment. Because, they all add vendor scripts before user code.
I write a example:
<!DOCTYPE html>

    <meta charset="utf-8">
    <title>testvue</title>

Open it as a html file in your browser, then open dev-tools, and click on the BeforeRouterInstalled instance.

It's should be an issue of vue-router instead.

 router: new VueRouter([]), should be  router: new VueRouter({ routes: [] }),
An HTML file is absolutely fine 🙂

router: new VueRouter([]), should be router: new VueRouter({ routes: [] }),
Oh, it's my mistake. But it doesn't interrelate the issue.

It's still `need-repro". Is there any problem to reproduce this?

Sorry, I just didn't have the time to check 🙂
It's actually not a bug on the devtools but rather something that happens in vue-router. You can trigger it without the devtools with a created hook in the first instance:
created () {
    this.$route // boom
}

edit: It may not happen all the time because of a race condition probably
But at the same time, you're using Vue twice in your code but only installing vue-router after initializing a part of your application... So, I don't really know if we want to support it. Eitherways, you can open an issue on vue-router, you can put this in a fiddle:
<!DOCTYPE html>

    <meta charset="utf-8">
    <title>testvue</title>

Sorry, I just didn't have the time to check :slightly_smiling_face:

It's OK!

It's actually not a bug on the devtools but rather something that happens in vue-router.

Same thought. I will submit an issue on vue-router later.
Thanks!


Add support for the conversion between kind and OpenYurt cluster

Ⅰ. Describe what this PR does
1.Only one node in the cluster is installed with openyurt, and then any node in the cluster can be converted.
2.Add support for the conversion between kind and OpenYurt cluster.
Ⅱ. Does this pull request fix one issue?
fixes #230.
Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.
Ⅳ. Describe how to verify it
make test
make build
cp _output/bin/yurtctl /bin/
yurtctl convert
yurtctl revert
Ⅴ. Special notes for reviews

/lgtm


Fix PhaseStatus that gets displayed for Presto tasks

TL;DR
When running a Presto task, the status of each of the 5 queries is not displayed properly.

Type

[X] Bug Fix
[ ] Feature
[ ] Plugin

Are all requirements met?

[X] Code completed
[ ] Smoke tested
[X] Unit tests added
[ ] Code documentation added
[ ] Any pending items have an associated Issue

Complete description
Due to how the state machine the Presto plugin uses goes back and forth between various states, I keep track of the of both the current and previous states and use the previous state for the status that gets displayed to users.
Tracking Issue
https://github.com/lyft/flyte/issues/237

So what's the end result of this?  The text for each log link still moves through the phases right? Like the first one will appear queued, then running, then succeeded, and then the next one will show up queued. etc.
If we're storing the individual query IDs, and they're always run serially, and you only launch the next one if the current one succeeds, then we can also just always assume that all the previous ones are in a succeeded state right?

@wild-endeavor Yeah that seems like a reasonable assumption to make. The part where it sets the previous status for the last time (ie. succeeded) is here.

The problem that was happening is that after each query, we set the phase to "queued" (so that it can keep advancing through all 5 queries). However, because this is set to "queued" it keeps that phase through to when it gets logged and that's why it shows up that way in the UI


”Argetoaia”, ”Băranu”, ”Berbeșu” are listed in the drop-down menu

Description:
”Argetoaia”, ”Băranu”, ”Berbeșu” the locations appear in the drop-down menu next to the comune and the county witch is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Argetoaia"

Expected results
The location appears in the drop-down menu as "Sat Argetoaia, DOLJ (Argetoaia)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Băranu"

Expected results
The location appears in the drop-down menu as "Sat Băranu, DOLJ (Argetoaia)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Write in the search bar "Berbeșu"

Expected results
The location appears in the drop-down menu as "Sat Berbeșu, DOLJ (Argetoaia)".


[CHORE] updated reconciliation readme

CHORE
Objective:
Update reconciliation readme

Skimmed through, dint find anything out of the ordinary. So good to me.


[Addition]: spoofed clusters of pisces miners in Philippines (owner 14a26AUV92h5jcMYQqmjkt7WS7zg3P3DC5erJ45n1a6QLMX2EB3)

Hotspot b58 Addresses
11kW7rnq8oBTTMum42Df47ritrbYuP8ytZEXbABU5xUomZu43fY
1122bm5cNgtzCiE174VhXvE7gvUUs8MRykSdxaqEatG2cyDwUWK7
1123nqXS68rDcTPH97EGHe1tvz7ZEes1ZqEN2yfY2zj31stNe2E9
112tx87Lbe9dpxLrYj7jihQFsKdYSAAxVAacKG3UuXa8af7zLGF5
112GpLX1oMbUectNzQuY21pYSo5LD5jqM9c8Aap8YEEwpmiVrQJF
112YA5yGgG7QshFKGF8oKQAAzWLPpuh9U963h2jrntgQunhPLjBv
11ZW3TmCqG9YE4oyZ69v826LokHzLJ5V3yHfuyCfo7XbTvY2zxw
112WQ4PXggeZyqvf9HsisnRfs31vuJAPGWC9HNLRYFi3ywneoMiz
112EECXvmcizNauWYkZRYsnQknHPKQoWWEFrGWNFXGTWhFYAjhwJ
11CDkmCMYWVr3wUxKikMF7uaJ7TEWVGs3kEksZHtxntzC3foFNp
Discord Handle
screwdriver#9187
Reason(s)
spoofed cluster miners. fake asserted location.

We have taken into account this addition request in the next iteration of the denylist. Some or all of the Hotspots listed may be added to the denylist. This is an automated comment due to the volume of addition and removal requests. A future tool is in the works to provide more insight into the analysis approach made by the Helium team and community members that maintain this list.


v0.3.1: Cannot use import statement outside a module

Hi, I'm trying out this module and the latest version seems to crash with the following error.
The component is rendered with no errors using<EMAIL_ADDRESS>so I think it has something to do with 0.3.1.
$ yarn run dev

...

[vite] Error when evaluating SSR module /node_modules/svelte-lexical/src/core/plugins/collaboration/collaboration.ts: failed to import "y-websocket"

...

Internal server error: Cannot use import statement outside a module
      at internalCompileFunction (node:internal/vm:73:18)
      at wrapSafe (node:internal/modules/cjs/loader:1176:20)
      at Module._compile (node:internal/modules/cjs/loader:1218:27)
      at Module._extensions..js (node:internal/modules/cjs/loader:1308:10)
      at Module.load (node:internal/modules/cjs/loader:1117:32)
      at Module._load (node:internal/modules/cjs/loader:958:12)
      at ModuleWrap.<anonymous> (node:internal/modules/esm/translators:169:29)
      at ModuleJob.run (node:internal/modules/esm/module_job:194:25)
/*****/node_modules/y-websocket/src/y-websocket.js:7
import * as Y from 'yjs' // eslint-disable-line
^^^^^^

SyntaxError: Cannot use import statement outside a module
    at internalCompileFunction (node:internal/vm:73:18)
    at wrapSafe (node:internal/modules/cjs/loader:1176:20)
    at Module._compile (node:internal/modules/cjs/loader:1218:27)
    at Module._extensions..js (node:internal/modules/cjs/loader:1308:10)
    at Module.load (node:internal/modules/cjs/loader:1117:32)
    at Module._load (node:internal/modules/cjs/loader:958:12)
    at ModuleWrap.<anonymous> (node:internal/modules/esm/translators:169:29)
    at ModuleJob.run (node:internal/modules/esm/module_job:194:25)

Hi @yheuhtozr ,
I am not facing this issue in playground and sveltekit demo projects. Probably the issue caused by a combination of sveltekit and CollaborationPlugin.
Could you confirm if you are using CollaborationPlugin in a sveltekit project?

Thank you. I'm using SvelteKit but I don't think I explicitly turns on CollaborationPlugin. My current import is:
import {
	Composer,
	ContentEditable,
	PlainTextPlugin,
	HistoryPlugin,
	PlaceHolder
} from 'svelte-lexical';

I don't have a plan to use CollaborationPlugin anyway, so it'd be fine as well if there is a way to turn it off manually.

Hi @yheuhtozr ,
Seems like the issue is with JS module import. Do you have "type": "module" in your package.json file?

Yes I have. I think it came by default. Does it cause problem?

Hi @yheuhtozr ,
I updated all the packages to the latest version. Now, I am able to reproduce the issue. It means that some change in one of the dependencies probably vite is causing it.
It only happens in dev, release build is not impacted which further suggests that the issue is related to some change in behaviour in vite.

Found the root cause... y-websocket package doesn't have "type": "module" in package.json. Vite doesn't like that with dev build.
For now, I have moved out y-websocket code into playground instead of the core library to resolve the issue.

Hi @yheuhtozr ,
I have fixed the issue and released a new package, version 0.3.2.
Could you please try with 0.3.2 and confirm?

@umaranis Now it just works fine! Thank you for the quick update.


seperating each part of problem1, you can run the file directly under…

seperating the problem files,meanwhile combining different method of 1-4 and improving 1-5

part2result


"Skipped" is not a valid outcome value and does not match karma-trx-reporter's outcome

Currently, this results processor outputs a "Skipped" outcome for any tests which are marked to be skipped. However, this does not match what karma-trx-reporter outputs or what Visual Studio's api lists (see outcome property of Add test results to a test run section). Both list the outcome should be NotExecuted for skipped tests.
Can we switch the outcome of skipped tests to be NotExecuted instead of Skipped to align with the above? I can submit a PR for this if it sounds good.

Fixed, released as 0.0.7.

@no23reason no worries, thanks for fixing this! I missed the last comment saying the change was good.


Some bug in generate process

Thank you for all your contribution! However, when I reached the fifth part, I encountered the following error:
Traceback (most recent call last):
  File "/root/anaconda3/envs/mdm/lib/python3.8/runpy.py", line 194, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/root/anaconda3/envs/mdm/lib/python3.8/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "/root/hemu/HOI-Diff/sample/local_generate_obj.py", line 379, in <module>
    main()
  File "/root/hemu/HOI-Diff/sample/local_generate_obj.py", line 176, in main
    sample = sample_fn(
  File "/root/hemu/HOI-Diff/diffusion/gaussian_diffusion.py", line 730, in p_sample_loop
    for i, sample in enumerate(self.p_sample_loop_progressive(
  File "/root/hemu/HOI-Diff/diffusion/gaussian_diffusion.py", line 817, in p_sample_loop_progressive
    out = sample_fn(
  File "/root/hemu/HOI-Diff/diffusion/gaussian_diffusion.py", line 625, in p_sample
    out["mean"] = self.condition_mean(
  File "/root/hemu/HOI-Diff/diffusion/gaussian_diffusion.py", line 1797, in condition_mean
    return super().condition_mean(self._wrap_model(cond_fn), *args, **kwargs)
  File "/root/hemu/HOI-Diff/diffusion/gaussian_diffusion.py", line 462, in condition_mean
    loss, grad, loss_list = cond_fn(x,
  File "/root/hemu/HOI-Diff/diffusion/gaussian_diffusion.py", line 1885, in __call__
    return self.model(x, new_ts, **kwargs)
  File "/root/hemu/HOI-Diff/sample/condition.py", line 54, in __call__
    loss, grad, loss_list = self.gradients(x, t, self.afford_sample, y['obj_points'], y['obj_normals'])
  File "/root/hemu/HOI-Diff/sample/condition.py", line 155, in gradients
    grad = torch.autograd.grad([loss_sum], [x])[0]
  File "/root/anaconda3/envs/mdm/lib/python3.8/site-packages/torch/autograd/__init__.py", line 303, in grad
    return Variable._execution_engine.run_backward(  # Calls into the C++ engine to run the backward pass
RuntimeError: one of the variables needed for gradient computation has been modified by an inplace operation: [torch.cuda.FloatTensor [196, 512, 3]] is at version 10; expected version 9 instead. Hint: the backtrace further above shows the operation that failed to compute its gradient. The variable in question was changed in there or anywhere later. Good luck!

could you please help to handle this bug in the generate process?

Hello, I have encountered the same problem just now. Have you solved it out?

Hi, did you use "with torch.autograd.set_detect_anomaly(True):" to check where the error occurs?

Haven't. Maybe the problem comes from pytorch version? My torch version is 1.11.0

Haven't. Maybe the problem comes from pytorch version? My torch version is 1.11.0

Most likely! You can test the version of torch 1.7.1+cu110.

Thank you ! It works!!


Strict and explicit config resolution logic

This was one of the most confusing thing about MPR.
given an invalid path in a projects config property:
// package.json
projects: [
  'project1',
  'project2',
  'some_non_existing_thing',
]

current version of Jest would just either ignore this path completely and remove it from the project list, or, even worse, resolve back to the exact same package.json and try to run a project with all -test.js files from all the projects using the default configuration.
This PR:

guards agains having two projects resolve to the same config file
guards agains trying to specify a non-existing file or directory as a config
rewrites resolution logic and ads a bunch of tests

the only part that is lost is the ability to specify projects by a glob
projects: ['examples/*']

but i think this isn't a good use of globs and we don't even use it for our examples either. Anything that needs dynamic project list can use a .js based config and run glob explicitly.
If we still want to keep glob functionality, we should at least enforce that every glob resolves into at least one match.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@e8ed43e). Click here to learn what that means.
The diff coverage is 42.1%.

@@            Coverage Diff            @@
##             master    #4083   +/-   ##
=========================================
  Coverage          ?   60.65%
=========================================
  Files             ?      197
  Lines             ?     6781
  Branches          ?        6
=========================================
  Hits              ?     4113
  Misses            ?     2665
  Partials          ?        3

Impacted Files
Coverage Δ

...st-config/src/read_config_file_and_set_root_dir.js
0% <0%> (ø)

packages/jest-config/src/normalize.js
79.12% <0%> (ø)

packages/jest-config/src/index.js
0% <0%> (ø)

packages/jest-config/src/constants.js
100% <100%> (ø)

packages/jest-config/src/resolve_config_path.js
95.65% <95.65%> (ø)

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 e8ed43e...1cc00a2. Read the comment docs.

Please bring back the glob functionality, that should definitely be part of the feature and I don't want to break it. I'm ok with the added validation otherwise.


Fixing link to tutorial for Django web apps.

Found this while looking for something else in the docs.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/python/azure-sdk-for-python.md
:white_check_mark:Succeeded

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.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/python/azure-sdk-for-python.md
:white_check_mark:Succeeded

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.


Remove six

Feature Request
Describe the solution you'd like
Since we've removed support for older python versions we can now also remove six.

I hope we don't need any extra test cases as well as doc improvements, what do you think @jonathan-s


Fixes for running CMSSW12 geometry NTuples and fixing the efficiency

As per title. As shown in our last meeting, the efficiency is largely restored, being comparable with the old geometry for all samples:
New branch (new geometry):
muonGun
muonGun_highE
PU200
Current master (old geometry):
muonGun
muonGun_highE
PU200

Timing (comparable with current master):

LGTM!
I think we can deal with the simhit momentum functions later once we have the tracking ntuple code updated


Added DockerfileLightCaddy & Fixed the Caddyfile

I am still relatively new to Caddy as well, but my setup was following the setup I had in the wiki for single domains, since this container will only be running one domain. Have you tested the handle_path directive extensively? I know I had some issues with initial testing before reaching the end product [here[(https://github.com/PhlexPlexico/G5V/wiki/Webserver-Setup-With-Caddy) with having to strip prefixes, or you would get 404's if someone went to https://your.get5.instance/api/matches/ (notice the ending /, if that doesn't get removed it 404s). But if you can confirm all the traversing (going to the API, loading app, pressing back button doesn't produce a 404), then this should be good to merge in :)

Oh! Speaking of, and looking back at the branch you wish to merge into - I now remember the problem that I ran into with Caddy! Since Caddy requires variables for reverse proxy, nostname, and the API endpoint at build time, there is no possible way for this to be created into a package like the NGINX version! That's the one main issue I had when looking into this. However, we can still merge it in, but just know that it won't be at all possible to have this in a docker-compose file due to this issue!
Thanks!

Updated the readme to fix the change issue and the Caddyfile to reflect your changes, just delete the dockerfilefullcaddy and we should be good to merge in!

you got it :)


state & stateAbbr do not exist.

The Readme says that functions exist to fake states and state abbreviations. However, the Src for Generator.php does not actually include these options and I'm not able to use them in my tests.
Is there a plan to include these options?

These are locale-specific formatters - the exist e.g. in en_US.


POC for cloudflare and open-next

This is a POC for using cloudflare and aws together to deploy open-next
This is very messy at the moment as i just copied a bunch of stuff
What's included
A cloudflare worker is used as a router to either serve static file or to redirect to either the imageOptimizer function in aws or the external middleware which is also on a cloudflare worker.
The middleware cloudflare worker then execute all the necessary routing (including next.config headers, redirects and rewrites) and the middleware.
It also creates all the necessary functions on lambda (image optimization, default server and all the other server functions)
It also creates an R2 bucket for the static assets.
Requirements
A middleware.ts file is necessary.
It also requires some specific stuff in open-next.config.ts :
import type {OpenNextConfig} from "open-next/types/open-next.js"

const config = {
  default: {
  },
  // This middleware part is necessary to make it work
  middleware: {
    external: true,
    override: {
      wrapper: "cloudflare",
      converter: "edge"
    }
  },
} satisfies OpenNextConfig

export default config

What's not implemented or broken
There is no automatic upload to R2, this needs to be done manually.
Somehow every fetch to the cloudflare api is broken for me (i've probably done something wrong, i have a connect timeout for every call), so in order to test i had to break the WorkersUrl, i then manually enabled the url in cloudflare
Edge function are not deployed, api route (from app router) should already work if implemented in the construct
No support for wasm and assets, it should work, but we need to add this to the construct.

Replaced by #588


Add module name/description tags to pom files

The following POM files are missing their name and description tags:

data-platform-services
data-platform-services-api
data-platform-services-server
ranger-connector
security-officer-connectors
security-officer-services
security-officer-services-api
security-officer-tag-connector
virtualization-services-api

The snippet below is an example of these tags
    <name>OMRS REST API Connector</name>
    <description>
        Uses the OMRS REST API to call an open metadata compliant repository.
    </description>

i think this can be closed?

For virtualization-services-api, yes. :)

My error - I misread - whilst some changes are done, there are more to do.

Hi @danielaotelea could you please have a quick fix on the pom so this issue can be closed before 1.2 release?

All of them have been fixed by adding name and description. So I will close this issue.


Plans to sync with Gatsby?

This looks really cool.  The challenge for gatsby/TS users is that Gatsby's schema isn't aligned.  We have to do overrides like https://gist.github.com/bsgreenb/05202d74c7ef079970b536fad0fc5c88
So don't think this tool would help us?  Any recommendations?

Hi @bsgreenb,
that's an interesting question, I haven't thought about it yet.
How did you generate the types mentioned? '
Also, I wonder what's missing from the types you provided?
best
marco

Hey @marcolink
I use graphql codegen, import the generated types, then modify them.
What's missing is ability to distinguish required fields from not, so I need to manually un-null them.

@bsgreenb
unfortunately, I don't see how this could play together. If I understand it right, the types you generate are in the context of GraphQL, with all GraphQL specific fields and so on. Currently, this library only creates plain types for your content models, without any additional information.


python_virtualenv_constants: upgrade virtualenv to 16.1.0

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[ ] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[ ] Have you successfully run brew style with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

@BrewTestBot test this please

That does not trigger a rebuild on Azure Pipelines 🙃

@lembacon I've triggered one manually.


Corrected "minecraft:on_friendly_anger" description

The trigger has the same description as minecrft:on_death. This commit fixes that by taking its description from this page (https://docs.microsoft.com/en-us/minecraft/creator/reference/content/entityreference/examples/entitytriggers/minecrafttrigger_on_friendly_anger).

Fixes https://github.com/MicrosoftDocs/minecraft-creator/issues/345.

Thanks for submitting! Merging this now.

Thanks for submitting! Merging this now.


Fraud Proofs Tracking Issue

Per the spec, Celestia will support two types of native fraud proofs, Bad encoding and State. This will require quite a few changes across more than one repo, and this issue is meant to track those satellite issues required to properly implement fraud proofs.
BadEncodingFraudProof

[x] #461

StateFraudProof we will focus on state fraud proofs later

[x] https://github.com/celestiaorg/celestia-specs/pull/190
[ ] #462
[x] #481
[ ] #480

MessageInclusionFraudProof

[ ] https://github.com/celestiaorg/celestia-core/issues/519

PR #190 was created to add state fraud proofs to spec.

ref #132

I think we can close this here. BadEncodingFraudProofs will be implemented in node/the DA layer only and state fraud proofs are not happening before the mainnet milestone.


Remove version: CrystalDewWorld.CrystalDiskInfo.AoiEdition version 9.4.1

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

Close with reason: Duplicate;


Unable to authenticate users as members of allowed groups

When I try to log in with my LDAP credentials, I see the error message: "Incorrect Login. User is not member of allowed groups". However, my user is definitely within the groups I have defined in the LDAP. I believe it is connecting fine to the LDAP, but it doesn't know how to map the LDAP groups to the Azkaban roles. Do we need a separate config file for that? How do I map 'azkaban_admin' to grant users within it the 'ADMIN' permissions?
Here's what I have the properties set as (sensitive info scrubbed):
user.manager.class=net.researchgate.azkaban.LdapUserManager
user.manager.ldap.host=xx.xx.xx.xx
user.manager.ldap.port=xxxxx
user.manager.ldap.useSsl=false
user.manager.ldap.userBase=ou=Users,dc=zzzz,dc=com
user.manager.ldap.userIdProperty=cn
user.manager.ldap.emailProperty=mail
user.manager.ldap.bindAccount=cn=DirectoryManager,dc=zzzz,dc=com
user.manager.ldap.bindPassword=qqqq
user.manager.ldap.allowedGroups=azkaban_admin,azkaban_readonly
user.manager.ldap.groupSearchBase=ou=Azkaban_Groups,dc=zzzz,dc=com
user.manager.ldap.embeddedGroups=true

Unfortunately LDAP servers seem to behave sometimes a bit differently when it comes to groups (see also https://github.com/researchgate/azkaban-ldap-usermanager/pull/2).
Could you post an example for results from your ldap server doing manual user and group lookups?

Here's the ldif dump containing my azkaban groups from running an ldap search from the cli. Let me know if this helps.

## COMMAND RUN: ldapsearch -x -h [HOST] -p [PORT] -b "dc=zzzz,dc=com" -v
# extended LDIF
#
# LDAPv3
# base <dc=zzzz,dc=com> with scope subtree
# filter: (objectclass=*)
# requesting: ALL
#
# zzzz.com
dn: dc=zzzz,dc=com
objectClass: organization
objectClass: dcObject
dc: zzzz
o: org
# Users, zzzz.com
dn: ou=Users,dc=zzzz,dc=com
ou: Users
objectClass: organizationalUnit
# Azkaban_Groups, zzzz.com
dn: ou=Azkaban_Groups,dc=zzzz,dc=com
ou: Azkaban
ou: Azkaban_Groups
objectClass: organizationalUnit
# azkaban_admin, Azkaban_Groups, zzzz.com
dn: cn=azkaban_admin,ou=Azkaban_Groups,dc=zzzz,dc=com
uniqueMember: cn=[USER],ou=Users,dc=zzzz,dc=com
uniqueMember: cn=[USER],ou=Users,dc=zzzz,dc=com
uniqueMember: cn=[USER],ou=Users,dc=zzzz,dc=com
cn: azkaban_admin
objectClass: top
objectClass: groupOfUniqueNames
# azkaban_readonly, Azkaban_Groups, zzzz.com
dn: cn=azkaban_readonly,ou=Azkaban_Groups,dc=zzzz,dc=com
uniqueMember: cn=[USER],ou=Users,dc=zzzz,dc=com
cn: azkaban_readonly
objectClass: top
objectClass: groupOfUniqueNames
# [USER], Users, zzzz.com
dn: cn=[USER],ou=Users,dc=zzzz,dc=com
sn: [USER]
cn: [USER]
objectClass: top
objectClass: person
objectClass: organizationalPerson
userPassword:: [PASSWORD_HASH]

I have the same issue but 'user.manager.ldap.embeddedGroups=true' is not relevant to, I removed the value from user.manager.ldap.allowedGroups and just left behind 'user.manager.ldap.allowedGroups=' and I can now login.

Removing the embeddedGroups value and setting allowedGroups to empty allowed me to successfully authenticate with my ldap credentials, but there is no role management with this approach. Every user that is within Azkaban_Groups can login and is granted ADMIN access, however I wish to have some with only read only access (sub group azkaban_readonly). Have you been able to map different ldap groups to different roles within Azkaban?

yeah, I hear you.
No on the mapping but I am not fussed as I only have admin users. What I did want to do was deny users from being able to login. Right now everyone in login.

If I see this correctly from the information you provided the differences are:

The field where the members of a group are stored is uniqueMember instead of memberuid
This fields contains the users DN instead of UID

@devangorder can you check if this branch: https://github.com/researchgate/azkaban-ldap-usermanager/tree/issue-3-use-dn-for-allowed-groups
works for you.
You will need to set:
user.manager.ldap.groupMemberField=uniqueMember
user.manager.ldap.useDnForGroupMembership=true
If it does not you could try editing LdapUserManagerTest with your information and trying to run it agains your LDAP server and have a look if the group check starting at LdapUserManager::138 works correctly for your LDAP server.
It is kind of hard to test for me, because the servers I have accessible behave differently.

well it's the difference I suppose between 'groupofnames' and a 'posixgroup'. I believe groupofnames is more the 'active directory' way of doing things where posixgroup is the unix way.
so in 'groupofnames' members are denoted by the 'member' attribute which are full DN's to the user.
and in a 'posixgroup' members are denoted by the 'memberuid' attribute which are just the UID's of the users in the group.

@jurgenweber @devangorder the issues around the allowedGroups have been addressed in v1.2.0
At the moment the project supports the two following LDAP group constructs:

posixGroup:
this is default construct which is used when you have "user.manager.ldap.allowedGroups" defined but are not using "user.manager.ldap.embeddedGroups"

groupOfNames:
This is supported only with an active overlay which provides a memberOf attribute for each user (e.g. should be the default within ActiveDirectory). In order to use this feature you need have "user.manager.ldap.allowedGroups" defined and need to set "user.manager.ldap.embeddedGroups=true"

For further reference see also https://www.rainingpackets.com/ldap-posixgroup-groupofnames
The "groupOfUniqueNames" objectClass is not supported yet.
I will close this particular issue for now since the underlying issues with non-matching allowedGroups has been addressed. If you should not be able model your group setup with the mentioned supported constructs, please open an additional ticket for enhancement to support also the groupOfUniqueNames objectclass.