chore: Hide dock icon on MacOS and Windows

Now the dock icon is hidden on MacOS and Windows.
Closes: #39

Dear friend!
It's not working on Mac.
Please, take a good look (again) 💜


Contact Page

Add a good Contact Page.

 Best one will be merged!

Claim

Go ahead!! Best one will be merged!

Go ahead!! Best one will be merged!

claim

hey @Rakesh9100 , please assign me this task as this is open now, I can make a good contact page

Claim
Issue #1
Completed, added contact page


new nlpinsights image version

Signed-off-by: David Ranum<EMAIL_ADDRESS>

New version of nlp-insights has been delivered.  This PR will update the dockerhub version in the nlp-insights chart.


Null Reference Exception in RestoreWallets

McmSave.cs 194
if (charData.WalletData != null) {
var character = Character.CharacterList.FirstOrDefault(c => charData.CharacterInfo.GetIdentifier() == c.Info.GetIdentifier());
Above code needs to be guarded against a nullreference if Character does not return a valid List

Noted

Still occurs nearly ever round change for me. (New campaign with MCM installed mid-campaign once it was updated to work with the latest version)
 [SV LUA ERROR] System.NullReferenceException: Object reference not set to an ins tance of an object.    at MultiplayerCrewManager.McmSave.<>c__DisplayClass42_0.<RestoreCharactersWal lets>b__0(Character c)    at System.Linq.Enumerable.TryGetFirst[TSource](IEnumerable1 source, Func2 p redicate, Boolean& found)    at MultiplayerCrewManager.McmSave.RestoreCharactersWallets()    at MultiplayerCrewManager.McmMod.<InitMethodHooks>b__39_9(Object instance, Pa rameterTable ptable)    at Patch_mcm_crewmanager_initround_55093b3e8dac4838b3448d5dfe369dfc_Barotraum a_CrewManager-Void_InitRound__.HarmonyPostfix(MethodBase __originalMethod, Objec t __instance)

I will note I started the campaign with 2 PC's, but generally that 2nd PC isn't logged in.
I feel like there is some slight difference between PC's and NPC's, a NPC's aren't technically supposed to have wallets/get wages/etc?

@Akronyhm sure i'll give it a shot.
Maybe throw some console logging into the null checks to log what it was accessing at the time?

No exceptions in my 1st test and money was properly restored to last wallet balance (edited in what I lost lol), Would have tested further but.. died, and need to get some work done. lol. Will test further when I play some more today/tomorrow.

Seems to have fixed the wallet issue in my limited testing (a few rounds)


additional instructions for cleaning up

added the notes in 02 notebook to provide instructions for cleaning up with running 04 notebook.

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Approved


Saving and reading config to and from the generated readmes

Is your feature request related to a problem? Please describe.
When users want to modify their readme already created with GitHub Profilinator, they need to start from scratch as there is no config option.
Describe the solution you'd like
At markdown generation, the config can be added to it as comments then it can be loaded directly by providing the link to the existing readmes generated via GitHub Profilinator.
Describe alternatives you've considered
An alternative would be allowing users to download the config but again they will have to manage storage, add it to the readme as comment looks like a better solution.
Additional context
none

Agreed with not going with config file down solution and about the readme, user have to share the raw file link in order us to get text instead of html and find the config area and parse it.
Another solution can use adding login with github and then save the config using backend may be firebase if not our own.

@smkamranqadri That's another good solution but I feel it increases the complexity of the solution by involving a backend. The config as a comment solution is simple to implement and manage. What do you think? Do you see any drawbacks to this solution?

A part from user to give us raw file link, nothing I can think of.

I've added TODOs for this issue. See if you can take them up.


Re-instated (previously deleted account) on bsky can't get the bridge working

I think I’ve seen this issue before. This account on bluesky deleted his account a few months ago.
https://bsky.app/profile/bassesbasta.bsky.social
Which meant the name was available, however, he created a new account with the same name a few days ago, and also re-followed the bridge on bsky recently. But even though he re-followed the bridgy account on bsky, his posts doesn’t get bridged to the fediverse. Is there something you can do to help? :-)

Hi! Yeah we don't officially support this yet, but we should. That's tracked in #1293. For now, I've fixed this one manually.


Remove unhelpful stack traces in yaml validate by default

Description
Currently when you use the yaml validator, it generates a stack trace that would not be helpful to an end user. This PR removes that stack trace by default.
As an example,
Before this PR
java -jar dockstore-client/target/dockstore-client-1.14.0-SNAPSHOT-shaded.jar yaml validate --path dockstore-client/src/test/resources/YamlVerifyTestDirectory/incorrectly-named-parameter-in-yml
dockstore-client/src/test/resources/YamlVerifyTestDirectory/incorrectly-named-parameter-in-yml/.dockstore.yml is a valid yaml file
15:48:39.882 [main] ERROR io.dockstore.common.yaml.DockstoreYamlHelper - Error reading .dockstore.yml: Cannot create property=tools for JavaBean=io.dockstore.common.yaml.DockstoreYaml12@7e9da981
 in 'string', line 1, column 1:
    version: 1.2
    ^
Cannot create property=publih for JavaBean=io.dockstore.common.yaml.YamlTool@40368a46
 in 'string', line 3, column 6:
      -  subclass: WDL
         ^
Unable to find property 'publih' on class: io.dockstore.common.yaml.YamlTool
 in 'string', line 10, column 14:
         publih: true
                 ^

 in 'string', line 3, column 3:
      -  subclass: WDL
      ^

org.yaml.snakeyaml.constructor.ConstructorException: Cannot create property=tools for JavaBean=io.dockstore.common.yaml.DockstoreYaml12@7e9da981
 in 'string', line 1, column 1:
    version: 1.2
    ^
Cannot create property=publih for JavaBean=io.dockstore.common.yaml.YamlTool@40368a46
 in 'string', line 3, column 6:
      -  subclass: WDL
         ^
Unable to find property 'publih' on class: io.dockstore.common.yaml.YamlTool
 in 'string', line 10, column 14:
         publih: true
                 ^

 in 'string', line 3, column 3:
      -  subclass: WDL
      ^

        at org.yaml.snakeyaml.constructor.Constructor$ConstructMapping.constructJavaBean2ndStep(Constructor.java:321)
        at org.yaml.snakeyaml.constructor.Constructor$ConstructMapping.construct(Constructor.java:207)
        at org.yaml.snakeyaml.constructor.Constructor$ConstructYamlObject.construct(Constructor.java:358)
        at org.yaml.snakeyaml.constructor.BaseConstructor.constructObjectNoCheck(BaseConstructor.java:270)
        at org.yaml.snakeyaml.constructor.BaseConstructor.constructObject(BaseConstructor.java:253)
        at org.yaml.snakeyaml.constructor.BaseConstructor.constructDocument(BaseConstructor.java:207)
        at org.yaml.snakeyaml.constructor.BaseConstructor.getSingleData(BaseConstructor.java:191)
        at org.yaml.snakeyaml.Yaml.loadFromReader(Yaml.java:477)
        at org.yaml.snakeyaml.Yaml.load(Yaml.java:406)
        at io.dockstore.common.yaml.DockstoreYamlHelper.readContent(DockstoreYamlHelper.java:223)
        at io.dockstore.common.yaml.DockstoreYamlHelper.checkForUnknownProperty(DockstoreYamlHelper.java:254)
        at io.dockstore.common.yaml.DockstoreYamlHelper$Version$3.validateDockstoreYamlProperties(DockstoreYamlHelper.java:86)
        at io.dockstore.common.yaml.DockstoreYamlHelper.validateDockstoreYamlProperties(DockstoreYamlHelper.java:240)
        at io.dockstore.client.cli.YamlVerifyUtility.dockstoreValidate(YamlVerifyUtility.java:167)
        at io.dockstore.client.cli.YamlClient.handleCommand(YamlClient.java:73)
        at io.dockstore.client.cli.Client.run(Client.java:734)
        at io.dockstore.client.cli.Client.main(Client.java:633)
Caused by: org.yaml.snakeyaml.constructor.ConstructorException: Cannot create property=publih for JavaBean=io.dockstore.common.yaml.YamlTool@40368a46
 in 'string', line 3, column 6:
      -  subclass: WDL
         ^
Unable to find property 'publih' on class: io.dockstore.common.yaml.YamlTool
 in 'string', line 10, column 14:
         publih: true
                 ^

        at org.yaml.snakeyaml.constructor.Constructor$ConstructMapping.constructJavaBean2ndStep(Constructor.java:321)
        at org.yaml.snakeyaml.constructor.Constructor$ConstructMapping.construct(Constructor.java:207)
        at org.yaml.snakeyaml.constructor.BaseConstructor.constructObjectNoCheck(BaseConstructor.java:270)
        at org.yaml.snakeyaml.constructor.BaseConstructor.constructObject(BaseConstructor.java:253)
        at org.yaml.snakeyaml.constructor.BaseConstructor.constructSequenceStep2(BaseConstructor.java:469)
        at org.yaml.snakeyaml.constructor.BaseConstructor.constructSequence(BaseConstructor.java:435)
        at org.yaml.snakeyaml.constructor.Constructor$ConstructSequence.construct(Constructor.java:560)
        at org.yaml.snakeyaml.constructor.BaseConstructor.constructObjectNoCheck(BaseConstructor.java:270)
        at org.yaml.snakeyaml.constructor.BaseConstructor.constructObject(BaseConstructor.java:253)
        at org.yaml.snakeyaml.constructor.Constructor$ConstructMapping.newInstance(Constructor.java:333)
        at org.yaml.snakeyaml.constructor.Constructor$ConstructMapping.constructJavaBean2ndStep(Constructor.java:298)
        ... 16 common frames omitted
Caused by: org.yaml.snakeyaml.error.YAMLException: Unable to find property 'publih' on class: io.dockstore.common.yaml.YamlTool
        at org.yaml.snakeyaml.introspector.PropertyUtils.getProperty(PropertyUtils.java:155)
        at org.yaml.snakeyaml.introspector.PropertyUtils.getProperty(PropertyUtils.java:145)
        at org.yaml.snakeyaml.constructor.Constructor$ConstructMapping.getProperty(Constructor.java:337)
        at org.yaml.snakeyaml.constructor.Constructor$ConstructMapping.constructJavaBean2ndStep(Constructor.java:261)
        ... 26 common frames omitted
Your .dockstore.yml is invalid:
dockstore-client/src/test/resources/YamlVerifyTestDire

After this PR
java -jar dockstore-client/target/dockstore-client-1.14.0-SNAPSHOT-shaded.jar yaml validate --path dockstore-client/src/test/resources/YamlVerifyTestDirectory/incorrectly-named-parameter-in-yml
dockstore-client/src/test/resources/YamlVerifyTestDirectory/incorrectly-named-parameter-in-yml/.dockstore.yml is a valid yaml file
Your .dockstore.yml is invalid:
dockstore-client/src/test/resources/YamlVerifyTestDirectory/incorrectly-named-parameter-in-yml/.dockstore.yml has the following errors:
Unknown property: 'publih'. Did you mean: 'publish'?

This was caused by lines such as this one,
https://github.com/dockstore/dockstore/blob/00cccd2a1afc47d51c06a877df003577305afa62/dockstore-common/src/main/java/io/dockstore/common/yaml/DockstoreYamlHelper.java#L266
Which log errors and those errors were showing up in the output (and by default all errors will show up in the output of the CLI).
To correct this issue, I have checked to see if either the debug or info flag has been provided, and if they haven't I have turned off all log messages.
Review Instructions
To review this ticket, you should

[ ] Verify that if the info or debug flags are provided, the full stack traces are shown
[ ] Verify that helpful messages are still provided even though all logs have been turned off.

Issue
https://github.com/dockstore/dockstore/issues/5186
DOCK-2268
Security
If there are any concerns that require extra attention from the security team, highlight them here.
Please make sure that you've checked the following before submitting your pull request. Thanks!

[x] Check that you pass the basic style checks and unit tests by running ./mvnw clean install
[x] If this PR is for a user-facing feature, create and link a documentation ticket for this feature (usually in the same milestone as the linked issue). Style points if you create a documentation PR directly and link that instead.

I do not have any reason to believe that this PR would cause bitbucket tests to fail. So, I will leave it as is.

While this change is good, the PR title nor linked issue seem applicable. Could you please either fold into your other PR, or update the title/description of this PR?

I have updated the title and description.


Fixed not-unique salt

Fixed a bug that the salt might not be unique per application.
The salt was generated with the first 4 letters of the application key. Since more than 2 years, the key is prepended with base64:, which means the salt for nearly every application might be "base" and the class name.
Increased performance
While env() reads the environment file, config() can access the apps config-variable and therefore is much faster. Therefore, the env-helper should never be used in application code, only to resolve config in the first place.
Allow easy overwrite for the salt generation method
Because of the changed behaviour, the salt for a lot of implementations would change. Therefore, we should provide an easy method to revert the salt-generation method. This is done with the implementation of makeHashedIdSalt which can be overridden in the trait-using model.

Thanks for pointing that out.


Implement n_kv_heads argument for LLaMA-2 Transformer to enable 70B model

PR https://github.com/tinygrad/tinygrad/pull/1284 adds LLaMA-2 support, but the 70B model is disabled at this moment:
https://github.com/tinygrad/tinygrad/blob/12dd09ad5406af090cdb5d6ac95a0277bd60a99b/examples/llama.py#L185-L189
To enable it, we need to implement n_kv_heads for LLaMA-2.
Upstream implementation: https://github.com/facebookresearch/llama/blob/master/llama/model.py

I'd like to take this up if no one's on it yet


Prefix exception message with Lint failed or Analysis failed

In order to correctly classify errors in Gradle Enterise, exception messages should be prefixed with any of the messages listed below:

Analysis failed
Check failed
Compilation failed
Code generation failed
Generation failed
Lint failed
Processing failed
Test failed
Tests failed
Testing failed
Verification failed

See docs for more information:
https://docs.gradle.com/enterprise/failure-classification/#verification_failures
Observed Behavior
The current exception message Build failed with 2 weighted issues.
Happy to contribute this change.

If only the detekt org had access to a Gradle Enterprise instance to test this with ;)
This change makes sense. Just for my own info, does the GE logic rely on the message bring in an exception?

That's correct. See the failure classification guide for more information. https://docs.gradle.com/enterprise/failure-classification/#verification_failures

Do you know if Gradle enterprise is planning to create new interfaces for these verification failures? Using prefix string matches does not feel robust enough to minimize false positives and false negatives.

This is the first false positive I have discovered in my 1.5 years at Gradle. There is no plan @chao2zhang.
Feel free to browse our open source instances of Gradle Enterprise to see if there are any other occurrences of this happening in order to consider prioritizing this feature: https://gradle.com/enterprise-customers/oss-projects/


lnd: adds NOT_FOUND status code for LookupInvoice

Fixes #5510
Pull Request Checklist

[x] All changes are Go version 1.15 compliant
[ ] Your PR passes all CI checks. If a check cannot be passed for a justifiable reason, that reason must be stated in the commit message and PR description.
[x] If this is your first time contributing, we recommend you read the Code Contribution Guidelines

[x] The code being submitted is commented according to Code Documentation and Commenting
[x] Commits have a logical structure according to Ideal Git Commit Structure

[ ] For new code: Code is accompanied by tests which exercise both the positive and negative (error paths) conditions (if applicable)
[ ] For bug fixes: If possible, code is accompanied by new tests which trigger the bug being fixed to prevent regressions
[ ] Any new logging statements use an appropriate subsystem and logging level
[ ] For code and documentation: lines are wrapped at 80 characters (the tab character should be counted as 8 characters, not 4, as some IDEs do per default)
[x] A description of your changes should be added to running the release notes for the milestone your change will land in.

Needs a rebase.

You should do a rebase instead of merging to preserve a clean git history.


GCE launcher - test DIND

Failed using alpine... closer with ubuntu.    able to launcher vms but unable to connect to the docker process (blocking the extraction of the dcos installer)

Note: This issue has been migrated to https://jira.mesosphere.com/browse/MARATHON-1844. For more information see https://groups.google.com/forum/#!topic/marathon-framework/khtvf-ifnp8.


Defining Credit Card format as 16 digits is incorrect

This sentence is misleading because Credit Card Numbers are not always 16 digits.
"16 digits which can be formatted or unformatted (dddddddddddddddd) and must pass the Luhn test."
This may make people think DLP cannot detect credit card numbers that are not 16 digits long, however I validated with testing 14 and 15 digit card numbers those scenarios are covered as well.

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

ID: 0c16b548-1fc9-2765-3e8a-7559be41fd9d
Version Independent ID: 7fe5f4ba-e53d-d86d-4ab9-64e872e33069
Content: What the sensitive information types look for - Microsoft 365 Compliance
Content Source: microsoft-365/compliance/what-the-sensitive-information-types-look-for.md
Service: o365-seccomp
GitHub Login: @chrfox
Microsoft Alias: chrfox

@AveryBaker Thank you for your feedback. We have made the changes to the doc via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/1300
Thanks
Sri

Hi @AveryBaker
Thanks for your comment, it has been addressed in the document and published


netlrts-linux-x86_64-pxshm-smp build fails

Original author: Jim Phillips (@jcphill)
Original issue: https://charm.cs.illinois.edu/redmine/issues/638

../bin/charmc  -optimize  -o machine.o  -DFOR_CPLUS=1 machine.c
In file included from machine-common-core.c:523,
from machine.c:224:
machine-pxshm.c:387:24: error: macro "CmiMemoryReadFence" passed 2 arguments, but takes just 0
In file included from machine-common-core.c:523,
from machine.c:224:
machine-pxshm.c: In function 'CmiSendMessagePxshm':
machine-pxshm.c:387: error: 'CmiMemoryReadFence' undeclared (first use in this function)
machine-pxshm.c:387: error: (Each undeclared identifier is reported only once
machine-pxshm.c:387: error: for each function it appears in.)
machine-pxshm.c:388:25: error: macro "CmiMemoryWriteFence" passed 2 arguments, but takes just 0
machine-pxshm.c:388: error: 'CmiMemoryWriteFence' undeclared (first use in this function)
machine-pxshm.c:437:26: error: macro "CmiMemoryReadFence" passed 2 arguments, but takes just 0
machine-pxshm.c:438:27: error: macro "CmiMemoryWriteFence" passed 2 arguments, but takes just 0
machine-pxshm.c:759:27: error: macro "CmiMemoryReadFence" passed 2 arguments, but takes just 0
machine-pxshm.c: In function 'emptyAllRecvBufs':
machine-pxshm.c:759: error: 'CmiMemoryReadFence' undeclared (first use in this function)
machine-pxshm.c:760:28: error: macro "CmiMemoryWriteFence" passed 2 arguments, but takes just 0
machine-pxshm.c:760: error: 'CmiMemoryWriteFence' undeclared (first use in this function)
machine-pxshm.c:776:28: error: macro "CmiMemoryReadFence" passed 2 arguments, but takes just 0
machine-pxshm.c:777:29: error: macro "CmiMemoryWriteFence" passed 2 arguments, but takes just 0
machine-pxshm.c:812:26: error: macro "CmiMemoryReadFence" passed 2 arguments, but takes just 0
machine-pxshm.c: In function 'flushAllSendQs':
machine-pxshm.c:812: error: 'CmiMemoryReadFence' undeclared (first use in this function)
machine-pxshm.c:813:27: error: macro "CmiMemoryWriteFence" passed 2 arguments, but takes just 0
machine-pxshm.c:813: error: 'CmiMemoryWriteFence' undeclared (first use in this function)
machine-pxshm.c:825:27: error: macro "CmiMemoryReadFence" passed 2 arguments, but takes just 0
machine-pxshm.c:826:28: error: macro "CmiMemoryWriteFence" passed 2 arguments, but takes just 0
Fatal Error by charmc in directory /ssd/jim/charm-6.6.1/netlrts-linux-x86_64-pxshm-smp/tmp
Command gcc -m64 -DCMK_GFORTRAN -D_REENTRANT -I../bin/../include -D__CHARMC__=1 -DFOR_CPLUS=1 -O2 -fno-stack-protector -c machine.c -o machine.o returned error code 1

Original author: Jim Phillips (@jcphill)
Original date: 2014-12-22 23:21:29

net-linux-x86_64-pxshm-smp build succeeds.

Original author: Jim Phillips (@jcphill)
Original date: 2014-12-22 23:26:11

But crashes:
[13] Assertion "dstRank >=0 && dstRank != pxshmContext->noderank" failed in file machine-pxshm.c line 300.
------------- Processor 13 Exiting: Called CmiAbort ------------
Reason:
LDB: Central LB being created...
[13] Stack Traceback:
  [13:0] CmiAbort+0x3e  [0xbafece]
  [13:1] CmiSendMessagePxshm+0x30c  [0xbb09bc]
  [13:2] DeliverOutgoingMessage+0x2b5  [0xbb3055]
  [13:3] CmiGeneralSend+0x42  [0xbb40b2]
  [13:4] _Z15_skipCldEnqueueiP8envelopei+0xbb  [0xabb43b]
  [13:5]   [0xabb708]
  [13:6] CkSendMsgBranch+0x1b  [0xabc58b]
  [13:7] _ZN8ProxyMgr13registerProxyEi+0xb8  [0x9c0b38]
  [13:8] _ZN10ProxyPatchC1Ei+0x4d  [0x9d127d]
  [13:9] _ZN8ProxyMgr13createProxiesEv+0x26a  [0x9bd6da]
  [13:10] _ZN4Node7startupEv+0x7ac  [0x97508c]
  [13:11] CkDeliverMessageFree+0x44  [0xabbe94]
  [13:12]   [0xabc116]
  [13:13]   [0xabc3b5]
  [13:14] _Z15_processHandlerPvP11CkCoreState+0x7ff  [0xabe75f]
  [13:15] CsdScheduleForever+0x78  [0xbb8678]
  [13:16] CsdScheduler+0x2d  [0xbb877d]
  [13:17]   [0xbb1e74]
  [13:18]   [0xbb1f43]
  [13:19]   [0x301e4079d1]
  [13:20] clone+0x6d  [0x301e0e8b6d]


Program_options not necessary

Correct me if I'm wrong, but it seems that boost::program_options is not necessary to build anything using mqtt_cpp, it seems that it is only required when building examples? I'm trying to build deb package for mqtt_cpp from tag  v11.1.0 for our internal use and I ran into an issue where mqtt_cpp requires boost::program_options - we have custom build of boost-1.71.1 and boost::program_options library is not included in this build.
I tried doing this:
diff --git a/CMakeLists.txt b/CMakeLists.txt
index e30f2a7..72f2c1a 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -123,10 +123,10 @@ FIND_PACKAGE (Threads REQUIRED)

 IF (MQTT_USE_LOG)
     MESSAGE (STATUS "Logging enabled")
-    FIND_PACKAGE (Boost 1.67.0 REQUIRED COMPONENTS system date_time log filesystem thread program_options)
+    FIND_PACKAGE (Boost 1.67.0 REQUIRED COMPONENTS system date_time log filesystem thread)
 ELSE ()
     MESSAGE (STATUS "Logging disabled")
-    FIND_PACKAGE (Boost 1.67.0 REQUIRED COMPONENTS system date_time program_options)
+    FIND_PACKAGE (Boost 1.67.0 REQUIRED COMPONENTS system date_time)
 ENDIF ()

And it seems that everything works. Is it possible to apply this change in future releases?

boost::program_options is not necessary to build anything using mqtt_cpp, it seems that it is only required when building examples

this is correct.

the cmakelist should really be gating the program options library search behind whether the examples are being built.

Thank you all! @ineffective could you try #859 ?

@redboltz Hi, thank you for fast reaction and sorry for late response (in comparison). It seems that yes, everything works without problems. I'm closing this issue. Thank you!


Documentation of qiskit.optimization.INFINITY looks strange

What is the expected enhancement?
INFINITY is a constant to represent the infinite number.
The current description seems to be strange.

Convert a string or number to a floating point number, if possible.

https://qiskit.org/documentation/stubs/qiskit.optimization.INFINITY.html#infinity

That link is dead, I suppose this issue could be closed?

The docs were moved and it looks like redirect that was in place has been discontinued. It can be seen here now
https://qiskit.org/ecosystem/optimization/apidocs/qiskit_optimization.html


potter/ fix Azure destination test

Cherry pick Azure Destination fix from Roman's PR.

make title of pr a conventional commit message (the pr title will be the merge commit title by merge queue)

can you give a more clear description of what was wrong and how this fixes: both in changelog and pr description

code added to lint improvement instead.


Group timed add

https://github.com/PEXPlugins/PermissionsEx/blob/STABLE-1.23.4/src/main/java/ru/tehkode/permissions/PermissionUser.java
When adding a user to a group for a certain time, it only adds time to current time, it doesn't take in account if the user has previous time.
Example:
29-10-17 00.00: pex user x group add y * 68400 <-- (1 day)
29-10-17 12.00: pex user x group add y * 68400 <-- (1 day)
30-10-17 12:00: The user x will be removed from group y
I added (in total) 2 days for the user, so his rank would end the 31-10-17, but due to current system, his rank will end 12hours before (because pex doesn't sum new group time to (if) previous group time).

up

Unfortunately PermissionsEx is currently unmaintained and no further updates are planned. Because of this, all pull requests are being closed. If you are looking for an actively maintained permissions plugin, take a look at LuckPerms. It is compatible with major server platforms and can import data from PEX.


Refactor builder

Pull Request doc
Purpose
Refactor the building phase of a scenario since we will in the near future use grid model in the form of a PyPSA Network object. Closes #632.
What the code is doing
Allow to build a scenario from a PyPSA Network object
Testing
Existing unit tests. Especially, the powersimdata.scenario.tests.test_create
Where to look
The powersimdata.scenario.create module
Usage Example/Visuals
N/A
Time estimate
15 min

This is a test comment. Can someone reply so I can see if I get email notifications now?


Resize app.html .ion-modal-view in web layout

The app.html information modal ( .ion-modal-view ) is too small in the web layout, the modal needs to present the info better.  Either:

wider and devided into two collapsable collumns
or

a longer modal

@Ozcino Can you give more details on this issue as its quite diffuclt to know exactly what you mean. Thanks!

Will close this issue until we have some more info why its necessary.


NotSabs won't Activate No matter what?

Please Help. No matter what I do, notSabs will NOT activate. I've changes the package names numerous times. Unloaded and reloaded the app too. I get to the part where I paste in my knox key and hit the activate license button. I keep getting "License activation failed." I'm using my current license #(128bit) Not the KLM. I'm also using the latest copy of Android Studio and all seems OK..... Samsung S8 running Oreo 8.0(not rooted)

Hi everyone, I believe the issue is the bogus keys being generated by Samsung's key generator.

Thanks burgerbse.    Its in the Samsung EDU  SKD......Thanks.


Feature Request: Add -tag:v option for hvc1

New apple software (ios 11, high sierra, tvos11) requires the fourcc codec id to be "hvc1" and not "hev1" to playback HEVC/H265 natively. Is it possible to add an option for this for encoding and copying if the codec matches hevc/h265/x265?
As of now, this command only works with the latest ffmpeg snapshot for output to .mp4 and .m4v.
An example is here for the option:
ffmpeg -i sample.wmv -c:v libx265 -preset medium -crf 23 -tag:v hvc1 -c:a libfdk_aac -b:a 128k test.mp4
Thanks

Done

Thanks! how do I toggle this option for conversion?

It's added automatically for anything converted to that codec
Sent from my iPhone

On Aug 6, 2017, at 06:31, automator0<EMAIL_ADDRESS>wrote:
Thanks! how do I toggle this option for conversion?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.

Hello, does this also work if the video is copied? meaning it is already hevc but just adding the tag? i've test it already and the tag doesn't seem to work.
Thanks


Ammendment to the compact list mode

Since the reason for using the compact list mode is to be able to see more info, would it be possible for the "drag and drop" msg to be removed and the right side shifted up to see more of the description

Not sure I really want to change the right side of the screen. I'd rather keep the current alignment and text as is regardless of list mode.

The only reason I suggested it was because when the gui is set to auto/4, there's not much space for the description box, and getting rid of the msg, would increase the boxes height somewhere between 33% and 50% which is quite a bit. And I feel like this space issue is why people wanted a compact mode to begin with (to be able to see more mods because of how big the full size can be with high gui scales).
But yeah if you're against it, you can close this

I think for now, I'm gonna say no to this.


👌 IMPROVE: Lhs menubar persistent

This PR adds a function call if a sidebar status has been set in localStorage. If the sidebar is open, it will stay open across pages and sessions (persistent).
Fixes #131
A variable check has been added {% if persistent_sidebar is defined and persistent_sidebar is sameas true %}. If true the sidebar will be persistent. Config entry being: persistent_sidebar: true

Need to add a config option to toggle the persistent sidebar.

Tested with variable but doesn't seem to be working. I think the Jinja logic looks ok. Have I missed something obvious @AakashGfude ?

@DrDrij needed to add the variable in theme.conf.j2 and use theme_persistent_sidebar instead of persistent_sidebar as sphinx attaches theme_ for each HTML theme option variable.
Using it inside:
sphinx:
  config:
    html_theme_options:
      persistent_sidebar: true

Thanks @AakashGfude. I think this is ready — however the new variable needs to be added to lecture source config. What would be the best way to proceed/test build here @mmcky?

hey @DrDrij we open a PR on lecture-python.myst and mark it as "[TEST] " and update the environment.yml to replace quantecon-book-theme=0.2.x with
git+https://github.com/quantecon/quantecon-book-theme@lhs-menubar-sticky

@DrDrij I linked a test case above for you to look at for the python.quantecon.org site.
@AakashGfude can you document (if it isn't already) how to regenerate the fixtures for @DrDrij  -- I am guessing that is what is causing the ci issues?

@mmcky the ci issue is caused by codecov which I mentioned yesterday. Getting back to it now,


feat(extras): enhance incremane

bind C+u to clean renaming words

i think , this is not a better way


mac electron build can fail #3634

I have added removing osxdmg directory.  Ant moving task worked incorrectly with this directory.

kick jenkins

kick jenkins

kick jenkins

kick jenkins

i still get this error if i make it twice in a row:
Done running the applescript...
Fixing permissions...
Done fixing permissions.
Blessing started
Blessing finished
Unmounting disk image...
"disk2" unmounted.
"disk2" ejected.
Compressing disk image...
Preparing imaging engine…
Reading Protective Master Boot Record (MBR : 0)…

hdiutil: convert failed - File exists
:makeInstaller FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':makeInstaller'.
> Process 'command './create-dmg/create-dmg'' finished with non-zero exit value 1

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

BUILD FAILED


Cannot use with electron-builder when asar: false

I'm trying to use this module for the Agregore Browser with the latest version of Electron and I'm trying to use ESM.
Although Electron doesn't support ESM natively, it's possible to use it if one disables ASAR support and imports files from the filesystem.
However, for some reason electron-builder has been getting me the following error when I try to build a binary without ASAR (on Liunux). You can find more detailed build logs here: https://github.com/AgregoreWeb/agregore-browser/actions/runs/4009313691/jobs/6884515962#step:5:99
 • asar usage is disabled — this is strongly not recommended  solution=enable asar and use asarUnpack to unpack files that must be externally available
  ⨯ ENOENT: no such file or directory, symlink '/home/mauve/programming/agregore-browser/node_modules/go-ipfs/go-ipfs/ipfs' -> '/home/mauve/programming/agregore-browser/release/linux-unpacked/resources/app/node_modules/go-ipfs/bin/ipfs'  failedTask=build stackTrace=Error: ENOENT: no such file or directory, symlink '/home/mauve/programming/agregore-browser/node_modules/go-ipfs/go-ipfs/ipfs' -> '/home/mauve/programming/agregore-browser/release/linux-unpacked/resources/app/node_modules/go-ipfs/bin/ipfs'
    at processImmediate (node:internal/timers:471:21)
From previous event:
    at Object.copyAppFiles (/home/mauve/programming/agregore-browser/node_modules/app-builder-lib/src/util/appFileCopier.ts:85:27)

Sometimes not using ASAR is important to making apps work and this is currently prevent that option from being pursued.
Are there any thoughts on what can be done about this or what I can do to debug the issue further?
I'll post any fixes I find here as I go.

I ended up having asar:true but extracting pretty much all the files at runtime to get it to work. 🙃
https://github.com/electron/electron/issues/21457#issuecomment-1404327157


alpha versioning looks problematic with crates.io

I was trying to make a crate depend on the version 0.4.0-alpha.1 that's on crates.io, but if I specify the dependency as ndarray = "0.4" I get the cargo error unable to get packages from source, and if I try to put the exact version number cargo errs out telling me it can't parse the cargo.toml.
However this works fine when specifying the dependency via git.
So maybe alpha versions should not be uploaded to crates.io as it seems they can"t be used from there.

I've managed to use it, exact version "0.4.0-alpha.1". Does that work?

I guess alpha versioning is unconventional. I want to have the new code out there before we make a new point release.

I've tested a bit more, the exact version "0.4.0-alpha.1" works with a nightly setup, but not on beta or stable.

version-wise, It's expected that "0.4" does not match the alpha version. 0.4.0 is a greater (later) version than any 0.4.0-alpha release.  I guess this is something that's fixed in cargo nightly.

Will fix using the other issue #73


No more letters and language selection after last update

There are no letters visible after the update, and there is no input. Also the diacritics selection in the setup menu is empty.
The cut and paste menu still works.

Smartphone (please complete the following information):

Device: fairphone 4
OS: android 12

I am seeing the same issue on a Pixel 5. I'm using the beta stream from the play store.

for me is the same samsung s7 with oreo

same here,
on one plus 7tpro and microg lineage android 12

CF: #312
Fixed on playstore. Not yet on F-droid

Why isn't the release in Github releases section?

Don't know. I'm also working on a github actions which should smooth releases (creating the changelog/release on github, pushing to google playstore/fdroid)

@urmomseksy

Why isn't the release in Github releases section?

After checking F-droid doc, the release is automated. It's based on tags.
There's no new tag for the bug fix, therefor no new version on F-droid is pushed

So, if it's solved why is not the corrected version in f-droid?

As I wrote in the previous comment, @flide has to create a new tag/release note for F-droid to pick it up.
I'm sorry I cannot do more atm

ok, thanks

I am able to type again. I have however lost the ability to type the number zero.
Screenshots for the two symbols pages on the default english layout:

I'm fairy certain there was a zero option before.

Thx. I'll have a look as soon as I'm back home. I'll probably create a PR in a few hours 🙂

@mtivi
PR created: #314

Duplicate for #312


RealSense SDK 2.x installer does not include realsense2Config.cmake

RealSense SDK 2.x installer (Intel.RealSense.SDK.exe) does not include realsense2Config.cmake.
Why does not includ config file in installer?

Required Info

Camera Model
SR300 / D400

Operating System & Version
Windows

SDK Version
All Release of RealSense SDK 2.x

Thanks,

Where are the prop pages for the librealsense 2.10.0?  Also, do I have to build the libraries from the source code to compile and run the examples?

[Realsense Customer Engineering Team Comment]
@ecarlsruh
Please find v2.10.0 in https://github.com/IntelRealSense/librealsense/releases
If you use windows system, you can install Intel.RealSense.SDK.exe to run the examples.

[Realsense Customer Engineering Team Comment]
@UnaNancyOwen  Would like to know if you still need help on this question.

Was this actually resolved @dorodnic ? I can't seem to find this file https://github.com/zivsha/librealsense/blob/getting_started_example/CMake/realsense2Config.cmake.in (without the .in suffix of course) after having installed the SDK on Windows.
Visual Studio 2019 has good out-of-the-box support for CMake now which makes it even more interesting then before as a cross-platform build tool.


Update to use the latest Roslyn version and add support for C# 7

Two main things to do:

Reference the newer version of Roslyn
Evaluate all new language features to see what works and what doesn't.

Even for things that work, need to add tests for new syntax features to make sure it never breaks.

Related issue w/ tuple syntax in generics: https://github.com/aspnet/Razor/issues/1070

Evaluate all new language features to see what works and what doesn't.

Went through the features and they all parse correctly (yay!).
The work for this issue is then:

Reference the newer version of Roslyn
Add tests for C# 7 features

FYI checking this in is blocked right now on us having the right versions for our tooling to consume.

1696e2aebf07da475ce71178a448071058068078

Do we need to add <LangVersion>latest</LangVersion> or is this the default?

Yes, you need to set LangVersion. MVC uses your projects' settings for compilation

@Eilon @NTaylorMullen @ryanbrandenburg Just a heads-up: we plan to release C# 7.1 along with Visual Studio 2017 version 15.3 (and C# 7.2 with version 15.5). My understanding is that ASP.NET Core and Razor don't need official updates for users to adopt those new versions of the compiler. I have instructions from Damian which I collected here.
What is your official adoption plan? Do you want me to ping you once nuget packages get published? Right now we have preview packages available.
Also, just to confirm, since I saw a discussion of LangVersion "latest" above: "latest" means latest minor version (7.0, 7.1, 7.2, 8.0, etc), whereas "default" means latest major (7.0, 8.0, etc). Most Visual Studio templates use "default" and we let people opt-into "latest". That avoids problems if different people on a team use different versions of Visual Studio 2017. What language version does ASP.NET Core project teamplates use? I'd recommend either specific version that gets ramped up, or "default".

What is your official adoption plan? Do you want me to ping you once nuget packages get published? Right now we have preview packages available.

There was a recent email thread about this - I've added you to that.
We ship a dependency on Roslyn which we use for runtime compilation. We will need to adopt your candidate (non-build number stamped) packages when they are available from myget, and sim-ship with you when we go to nuget.org. It sounds like the right folks are looped in on your side.

Regarding the templates, I believe ours are default. It seems weird that we'd want to break from the norm, but I'll start a discussion on our side about setting latest.
For Razor, it's our desire that runtime compilation should behave similarly to the rest of your project, so we want to get that information from the project compilation settings, and not hardcode anything.

Sounds good. Just to clarify, I think "default" is good. I don't recommend changing to "latest".


Modal is not displayed on Android

Describe the bug
Android emulator device does not show the modal at all.
To Reproduce
Steps to reproduce the behavior:

Generate<EMAIL_ADDRESS>project
Add react-native-magic-modal + peer deps<EMAIL_ADDRESS>and<EMAIL_ADDRESS>Configure portal etc
Show view with set width and height and set white BG to make sure to see some kind of modal
See it on iOS Simulator/real device
Do not see it on the Android device

Expected behavior
See the modal on the Android device
Screen capture
test on iOS
test on android.webm
Smartphone (please complete the following information):

Device: Medium Table Emulator
OS: Android
Version 35

Additional context
I tried figuring out and it seems from the logs that everything renders, just does not appear on the screen.
Changing zIndex to 1 on the surrounding View in index.cjs seems to fix the issue, but that's not a solution, just a hint where to look
zindex 1 android.webm

I've created a branch with the same dependencies you mentioned, but I was not able to repro the issue:
https://github.com/user-attachments/assets/1c86a3b4-a701-41b3-832d-d19e79efb8de
Branch: chore/reproduce-issue
Can you try it on my branch? I would appreciate a branch that I can clone so that I can see this in action before following up with a fix.

@GSTJ, unable to repro on your branch as well, please try to reproduce on a bare app setup with<EMAIL_ADDRESS>I also think it's due to some changes in the react-native, mainly the way that they default View props on Android, I think.
I was preparing an example for you in the chore/reproduce-issue branch, but since I am not familiar with pnpm and your project in general, it's taking way too much time.

@GSTJ figured out pnpm, going to push to the same branch soon

@GSTJ notifying that was not able to repro on bare with the specified react-native version, trying to figure out the deps in the project I am working on.

Thanks!
Might be where your magic portal is positioned somehow, try moving it to render in the front of everything.
iOS has a full window overlay that brings it to the front always, Android needs the portal to be correctly positioned.

I love how I just figured it out and got back to tell you that my portal was before NavigationContainer :)
All works now, whoops 😓


Cask request: boot2docker-status

http://boot2docker-status.nickgartmann.com/

When making a request, please make it clear in the issue’s title.

didn't know. got it.
On Mon, Mar 16, 2015 at 6:45 PM, Vítor Galvão<EMAIL_ADDRESS>wrote:

When making a request, please make it clear in the issue’s title.
—
Reply to this email directly or view it on GitHub
https://github.com/caskroom/homebrew-cask/issues/10071#issuecomment-81835826
.

--
andreineculau.com http://www.andreineculau.com


WIP Moving to ruby 2 2 3

We can probably close this now, as I 've pulled in the application building and container all together in #805  which results to a usable image.


Update @aws/chat-client-ui-types to 0.1.0

Problem
chat-client-ui-types is outdated
Solution
Bump version

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

Changes look good to me but does the commit follow the conventional commit standard?


fix: fix links to docs

also fix the single instance that made me notice that all of our links are
broken.

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

v3.0.32
GitHub release

Your semantic-release bot :package::rocket:


Conversion failing on XMC4100

I am running svd2rust on an XMC4100.svd file and it is panicing with the following

'C:\Users\lucas.cargo\registry\src\github.com-1ecc
6299db9ec823\svd-parser-0.5.2\src\lib.rs:576 parse::u32(t)', src\libcore\opti
on.rs:839:4

If I followed it correctly it is for a check on 'resetMask'.
I ran the SVD against an XML validator for the SVD schema and it stated everything was ok.
Any suggestions on what i could test to track down the issue?

Would you mind upload the .svd file or create a minimal example that reproduces this bug?

Sorry about that. Should have done that earlier
XMC4100.zip

@lucasbrendel could you please retry this with the current master branch? It looks like some related issues were cleaned up.
Thanks!

@jamesmunns yeah I will take a look at it this weekend. Thanks for the update

@lucasbrendel any update?

@jamesmunns Sorry for the long wait but yes, using master with svd-parser 0.6.0 fixes my issues. The rust file will at least be generated. I haven't checked anything past that yet. The latest release of svd2rust using svd-parser 0.5.2 does not work though, just an FYI


fix(lib): add translatable as part of the plugin payload

What?
Library
Added new property to the plugin.data object called translatable. This is a field setting that can be adjusted by the content editor and came in as a feature request from a customer.
Sandbox
In order to make the local development easier, sandbox was extended with a checkbox 'Translatable' where the user can manipulate the new property accordingly.
Documentation
You can review the Documentation updated here.
Open Tasks

[x] Sandbox Changes
[x] Updating Documentation

Why?
JIRA: EXT-2225

I need to check it, however I haven't changed anything that would affect the templates, maybe it has something to do with the origin feature where messaging only works for two specific domains?

I need to check it, however I haven't changed anything that would affect the templates, maybe it has something to do with the origin feature where messaging only works for two specific domains?

Our test helper is using the sandbox origin, though 🤔
https://github.com/storyblok/field-plugin/blob/d576fe405b854c0f0401d1531cbd4bd13cd514e4/packages/field-plugin/helpers/test/src/index.ts#L12
Let me know if you need help!

You are absolutely right @eunjae-lee. I think I have found the cause of this :) looking at it now

@eunjae-lee I made an addition to @storyblok/field-plugin/test.
Fixing this issue, made me think that maybe some validations are missing in regards to releases, currentReleaseId  and  interfaceLang because inside PR #344 the tests passed without any changes to @storyblok/field-plugin/test. What do you think?

@eunjae-lee I made an addition to @storyblok/field-plugin/test.
Fixing this issue, made me think that maybe some validations are missing in regards to releases, currentReleaseId and interfaceLang because inside PR #345 the tests passed without any changes to @storyblok/field-plugin/test. What do you think?

Ahhh... okay, so Storyfront already has been sending translatable in the payload. We made the change on the SDK side, but actually it was not feature-parity with the container implementation of the test helper.
Oh then, this leads to the fact that we still have two different implementation of container (the test helper and the sandbox), and when we extract the implementation and reuse it, we could write proper tests for better validation.
In the meantime, do you have any idea how to improve this at the current stage? (but feel free to skip if not)

I agree with you @eunjae-lee it would be much clearer if the container and tests shared this functionality, I will create a ticket for it (if we do not already have one). I think there is not much more we can do at this point, just to keep it in the back of our minds when reviewing such changes.


Error when quering relationship existence (Model::has()) while using multiple connections

Laravel Version: 8.4.0
PHP Version: 7.4.1
Database Driver & Version: MySQL 8.0

Description:
Querying relationship existence using has() method result to an error if the relationship references a model with a different connection than the parent.
Steps To Reproduce:
Assume the following setup:
config/database.php
'connections' => [
    'main_connection' => [
        ...
        'host' => 'localhost',
        'database' => 'main',
        ...
    ],
    'personnel_connection' => [
        ...
        'host' => 'localhost',
        'database' => 'personnel',
        ...
    ],
]

App\Models\Unit
class Foo extends Model
{
    protected $connection = 'main_connection';
    ...
}

App\Models\Bar
use App\Models\Unit;

class Employee extends Model
{
    protected $connection = 'personnel_connection';
    ...

    public function foo(): \Illuminate\Database\Eloquent\Relations\BelongsTo
    {
        return $this->belongsTo(Unit::class);
    }
    ...
}

MySQL "main" database
+-------+
| units |
+-------+
| id    |
| name  |
| ...   |
+-------+

MySQL "personnel" database
+-----------+
| employees |
+-----------+
| id        |
| name      |
| unit_id   |
| ...       |
+-----------+

When querying the relationship existence, the following error occurs:
Employee::has('foo')->get();
// Illuminate/Database/QueryException with message 'SQLSTATE[42S02]: Base table or view not found: 1146 Table 'personnel.units' doesn't exist (SQL: select * from `employees` where exists (select * from `units` where `employees`.`unit_id` = `units`.`id`))'

However if I do:
Employee::whereHas('unit', fn ($query) => $query->from('main.units'))->get();
// OK!

The error occurs only when using the has method. For instance, eager-loading the relationship works as expected:
Employee::with('unit')->get();
// OK!

Using has method to query relationship existence when the two models using the same connection works as expected too

I really wish people wouldn't split relationships across databases 😬

Laravel doesn't provide built-in support for querying relationships across database. Generally I just think it's a bad idea and don't even want to encourage people to build applications like that. It seems you already found a work around.

I took an even deeper dive into that multi connection relationship problem and have come to the conclusion that it's indeed not support but even that it is absolutely impossible to ever do this. Take the currently generated query from below:
select * from `employees` where exists (select * from `units` where `employees`.`unit_id` = `units`.`id`)

This fails because "units" lives on a different connection. As you cannot connect two tables from a different connection in a single query there's no way that something like has can ever work in a "join" fashion. To solve this you'd have to gather identifiers up front and compare them to a table on a different connection but that could lead to very expensive queries. We "could" look into just checking if the identifier was "null" in the parent connection query:
select * from `employees` where `employees`.`unit_id` is not null

But that's a totally different query and doesn't gives the assurance that the relation effectively exists in the other connection. There's also no re-assurance from foreign keys as these don't exists between connections. The fact that with works is simply because a separate query is run in isolation for the eager load.
I'll send in a PR to document this constraint of using multiple connections in combination with has.

Docs PR: https://github.com/laravel/docs/pull/6604

I took an even deeper dive into this and have come to the conclusion that it's indeed not support but even that it is absolutely impossible to ever do this. Take the currently generated query from below:
select * from `employees` where exists (select * from `units` where `employees`.`unit_id` = `units`.`id`)

This fails because "units" lives on a different connection. As you cannot connect two tables from a different connection in a single query there's no way that something like has can ever work in a "join" fashion. To solve this you'd have to gather identifiers up front and compare them to a table on a different connection but that could lead to very expensive queries. We "could" look into just checking if the identifier was "null" in the parent connection query:
select * from `employees` where `employees`.`unit_id` is not null

But that's a totally different query and doesn't gives the assurance that the relation effectively exists in the other connection. There's also no re-assurance from foreign keys as these don't exists between connections. The fact that with works is simply because a separate query is run in isolation for the eager load.
I'll send in a PR to document this constraint of using multiple connections in combination with has.

Thank you for pointing this out!

Hi @driesvints as you said you are going to add it to the docs, I just want to give you an example.
One can create two different connections Laravel-wise with same credentials but just different databases for convenience.
So when performing a query that should be from two different connections, the RDBMS can actually query the two different schemas/databases as the connection credentials on the running query builder are the same as the other one which will become implicit.
I know MySQL and SQL Server support it (I guess Postgres would also support, but I never used this feature with it). For example let's say in your example the units table lives under a coporative database, and the employees table lives under a sales database, both on the same RDBMS server, as the different configured Laravel connections are actually the same when you consider the credentials, this query is possible:
select *
from `sales`.`employees` -- database.table notation
where exists (
    select *
    from `corporative`.`units`  -- database.table notation
    where `employees`.`unit_id` = `units`.`id`
)

SQL Server can also run that query even if the databases are on different servers as long there is a linked server configured between them and the user credentials is allowed to access that linked server.
In other words the query is run from a single PDO connection, but if that connection's credentials have access to both databases it will work.
The workaround cited by @iraklisg might be working due to that being the case, and as the table name is forced by calling ->from(...) the end query is valid and can be run. Just for reference this is the workaround I am referring to:
Employee::whereHas('unit', fn ($query) => $query->from('main.units'))->get();

I agree splitting relations between databases is not the ideal scenario, and I didn't built the app I saw that being used but joined a project where that was already place.
But I also think there are some situations where the developer is building an app around an existing database and that situation can happen. Which as the reason the project I referred above needed to use that setup, it was built to replace a in-house reporting solution built previously in plain PHP. The many databases schemas were from different CRMs and other systems it needed to generate reports from.
As you said you are adding it to the docs I think it is more accurate to say this setup is considered to be an edge case and support across all query builder features for such setup is not guaranteed and might not work as expected.
Hope this helps.

Hey @rodrigopedra. Thanks for the thorough reply as usual! The problem I see with this is that this will still fail if the connection credentials are different. And from what I can tell from the internals they are currently totally unequipped to handle multiple connections in where statements for these sorts of queries.

Hi @driesvints , thanks for reading all up as usual, I am too wordy and need to work on that =)
The setup I was trying to illustrate is when you have multiple databases in the same server, which I've seen in several projects. I agree is not the ideal one, but it happens often.
As a eloquent model has a dedicated $connection property, but doesn't have a dedicated $database property, when dealing with tables from different databases, but in the same server, it is common that people creates different connection configurations in their ./config/database.php file to the same server, with same configurations, changing only the 'database' key. So using models that live in different databases becomes easier.
Think in the multi-tenant using different databases per-tenant (I know it is a controversial example). It is common in those setup to have a shared database with tenant and user login data, and maybe also some shared lookup tables, for example a roleswhen role is not tenant-configurable. And then we have one database per tenant.
Most probably the developer will have two database connection configurations in their ./config/database.php, so they can keep the shared database untouched, and just switch the tenant specific database name once per request/entry point.
Then each model will have their $connection property to their corresponding Laravel connection, which uses the same database credentials (I've seen usage of different credentials per tenant, but in such setup all tenant had read access to the tables in the shared database).
MySQL, for example, is capable of running a SQL statement joining tables from these two different databases, similar to the one in my last response. Or for example some SQL that joins the users table from the "shared" database to any tenant-specific table.
Of course when such SQL statement is executed, it is executed just from one PDO connection, PHP doesn't parse the SQL statement to check if all referenced tables exist in that connection. But as the credentials used on that PDO connection can read from both the shared and the tenant-specific database, MySQL (as an example, other RDBMS are able too) also doesn't care if that statement is referencing tables that live in different database within the same server as long the credentials used for running that statement has access grants to each database and the statement uses proper notation.
Again, I know this is not an ideal scenario. Ideally an app should join tables from the same database for opitmal performance. But my intention was to highlight that such situation is not impossible, and most of the time Eloquent handles it fine, as it just builds the query and delegates execution to MySQL, for example, as long the proper notation on tables' names is used, and the connection credentials are capable of running such SQL statement.
In my opinion it would be better if the comment added to docs told something in the lines of: "Hey building queries for dealing with tables from different connections is considered an edge case and might not give you the results you expect when using the query builder".
Saying its "impossible" is what I was suggesting to avoid. For example the workaround proposed in the first post works fine as it "forces" the database.table_name notation to the query builder on run time.
Sorry if my first response lead to any confusion, I need to work on being more concise. I hope this is clearer. And thanks again for your time.

@rodrigopedra maybe it's best that you attempt a PR to the docs yourself?

Hi @driesvints , I can give a try, but I am not confident on my English skills. I apologize if I made this seems harder than it should be, wasn’t my intention.
I am fine with your current PR to the docs. This is an edge case anyway, I was just thinking of changing the word “impossible” to something else, but not sure how to better phrase it. But I might be being picky on this one.
I am not at my office right now, and it should take some time until I get back. I can give it a try when I am back.
Thanks again for your time and patience.

Okay, I've changed it to "not possible at this time" instead :thu

Thanks @driesvints


feat: add lifecycle rules support to S3 resources in Terraform

#13  👈
main

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

Join @MickVanDuijn and the rest of your teammates on  Graphite


Feature/4497 technical support functions

Description
This PR provides a base for user uploaded (error) logs. It also enables sharing of these logs to friends and family via the share menu. Note you have to enable this particular logging first. Unlike development builds, this log does not work without permission.
Link to Jira

EXPOSUREAPP-4497
EXPOSUREAPP-5782

Screenshots

App-Informations:

ViewController

Start logging was tapped:

Upload was tapped -> Consent Screen:

Save locally was tapped -> Export Screen:

History Screen:

Moving the pb.swift files, is not a good Idea IMHO, now we have the files in two different places.

Moving the pb.swift files, is not a good Idea IMHO, now we have the files in two different places.

 The moving was an attempt to get some structure into them and maybe this caused this even bigger mess. I'll check it again.

Would have been nicer to split this monster up into smaller PRs, like we usually do with our Story PRs.
 It’s that story branch plus some new stuff from Carsten :)


Join Discord Button

I think our Discord Server is still quite unknown. You'll only find it if someone tells you or you go to Menu > Links Overview > Discord
I'd like to have a prominent button for joining Discord.

It would be really good if there was a way to put such a button immediately on the new matchmaker tab. I feel like putting it in an area where you need to click 3 times to get it won't be as efficient for filling it with casual users.

This is my suggestion:

It's pretty much implemented, I just need to find out how to truly detach the process from the client.

Let's do this I like it

It would be really good if there was a way to put such a button immediately on the new matchmaker tab. I feel like putting it in an area where you need to click 3 times to get it won't be as efficient for filling it with casual users.

Waiting for things that not yet exist is not really a thing I like to do

This is my suggestion:

It's pretty much implemented, I just need to find out how to truly detach the process from the client.

Why don't you just open it in the browser? Cause not everybody has Discord installed.

Why don't you just open it in the browser? Cause not everybody has Discord installed. Browser finds out if it is installed.

Because I have higher ambitions than that ;) it sucks when it needlessly opens the browser. Of course, I also covered the case where Discord isn't installed.


Doesn't fetch data about playlist.

When I paste a link to the playlist most of times it doesn't fetch data about it, so I have to download every video inside the playlist one by one. a link from the screenshot: https://youtube.com/playlist?list=OLAK5uy_nrO7jmxxFUnFAneYpErd7k9pJ9bFhaCQI

I've released an update that should fix this issue

Now it fetches data however after clicking download button it doesn't download the content

Sorry for this, it should be fixed now in 1.9.6

everything works just fine! No reason to excuse.


gRPC Support

Hello. :wave: Looking to try using bender for gRPC services. Are there any plans to add executors for gRPC any time soon?

Nothing planned on our end, but we’d be happy to accept a contribution that adds gRPC support.


journals: named offsets within journals

As an operator / application developer, I want to gazette to track & manage "checkpoints" of journals which I can refer to by a name of my choosing (rather than offset).

Closing until / unless there's a clearer motivating use-case.


DOCS - Example Task by command

I tried to follow the example in the url https://rocketry.readthedocs.io/en/stable/tutorial/advanced.html to execute a task by command, but, following the example app.task('daily', command ='echo "Hello world"'), an error occurs. To work, it was necessary to include the parameter shell=True. app.task('daily', shell=True, command='echo "Hello world"')

Hmm, ye. I think there is some room to improve the command task. Furthermore, the CommandTask does not support asyncio even though it could (there are subprocess functionalities in asyncio).
Which OS are you using perhaps? I actually think that needs shell as "echo" is not a program itself but a shell command. But not a shell/terminal expert.


Propagate error from erroneous VM deployment

When creating virtual machines the CloudStack API call may fail but a VM is still created. In this case, we populate VM data on the CloudStackMachine object so the end user can manually clean up. However, we don't return the underlying error that results in the error being swallowed.

/run-e2e

@chrisdoherty4 a jenkins job has been kicked to run test with following paramaters:

kubernetes version: 1.27.2
CloudStack version:
hypervisor: kvm
template: ubuntu-2004-kube
Kubernetes upgrade from: 1.26.5 to 1.27.2

Setting up environment failed

@chrisdoherty4 is this ready for testing?
/run-e2e

@rohityadavcloud a jenkins job has been kicked to run test with following paramaters:

kubernetes version: 1.27.2
CloudStack version:
hypervisor: kvm
template: ubuntu-2004-kube
Kubernetes upgrade from: 1.26.5 to 1.27.2

Setting up environment failed


Suggestion: Add Tides of Chaos second paragraph to the ability for Wild Magic Sorcerer

You've recreated the Wild Magic Sorcerer from the players handbook pretty much word for word BUT you kind of left out the thing that really made wild magic sorcerers fun and usable as a subclass.
The second paragraph of the "Tides of Chaos" ability is this:
Any time before you regain the use of this feature, the DM can have you roll on the Wild Magic Surge table immediately after you cast a sorcerer spell of 1st level or higher. You then regain the use of this feature.
Without this feature, the wild magic surge just happens too seldomly (only 1 out of 20 times when casting a non-cantrip) for the subclass to really register at all. If by using the Tides of Chaos ability means that your next non-cantrip spell will automatically cause a wild magic surge AND give you the Tides of Chaos feature back before a long rest, THAT's what really makes the wild magic sorcerer usable.
I never met a DM in pen and paper D&D that didn't allow for this effect, because otherwise the wild magic sorcerer is extremely tame and not a beneficial subclass to pick.
Thanks!

Thanks for suggestions. This sub is still in Alpha and there will be a lot of QA before we release to ensure it adapts well to a computer game.

Cool! It would be interesting to see how you've interpreted the "Wild Magic Surge" table. Obviously some stuff on there would just not be adaptable or relevant to Solasta, like your height changing a few inches or you getting older/younger, etc.

1 – Chaos - Trigger a random Wild Surge effect (except this one) at the start of each of your turns for the next minute.
2 – Fireball - You cast a 3rd level Fireball centered on self.
3 – Healing - You regain 2d10 hit points.
4 – Burning - Each creature within 20 feet of you (including you) catches on fire.
5 – Teleport - You can teleport up to 60 feet to an unoccupied space of your choice that you can see as a free action before your turn ends.
6 – Frightened - You become frightened until the end of your next turn.
7 – Invisibility - You become invisible for the next minute. The invisibility ends if you attack or cast a spell.
8 – Random Poison - A random creature within 60 feet of you becomes poisoned for 1 hour.
9 – Spell Slot - You regain your lowest-level expended spell slot.
10 – Maximize Spell - Maximize the damage of the next damaging spell you cast within the next minute.
11 – Random Flight - A random creature within 60 feet of you can fly for a minute.
12 – Grease - You cast Grease centered on self.
13 – Mirror Image - You cast Mirror Image.
14 – Wild Healing - Each creature within 30 feet of you (other than you) takes 1d10 necrotic damage. You regain hit points equal to the sum of the necrotic damage dealt.
15 – Mass Invisibility - Each creature within 30 feet of you becomes invisible for the next minute. The invisibility ends on a creature when it attacks or casts a spell.
16 – Action Surge - You can take one additional action immediately.
17 – Piercing Vulnerability - You and all creatures within 30 feet of you gain vulnerability to piercing damage for the next minute.
18 – Resistances - You gain resistance to all damage for the next minute.
19 – Lightning Strike - Up to three creatures you choose within 30 feet of you take 4d10 lightning damage as a free action before your turn ends.
20 – Refill Sorcery - You regain all expended sorcery points.

I’m not doing 100 of them 😂

Ah, cool! Well, in truth, there's only 50 possible ones on the Wild Surge table! And maybe 1/3 of those are not applicable to the game.

Tides of chaos if implemented as is will be a pain to play with lots of reactions pop up.
What if Tides auto reroll any 1s on attacks, checks, and savings?
Whenever it happens, wild surge is triggered?

That sounds like a pretty good idea. I do wonder though if the wild magic surge will still happen too rarely for it to be much of a factor.  Maybe also have Wild Magic Surges happen on a roll of 1-2 on a d20 for any time you cast a non-cantrip spell.
I can see your point about all the reaction pop-ups that would ask you if you want to use Tides of Chaos and that being annoying, but the Tides of Chaos feature is meant to give the Wild Magic sorcerer an actual advantage at doing stuff like attacks and saves.
What if when they roll a 1-2 on an attack, check, or save, they then automatically are successful on that roll but it causes a wild magic surge? Then they are getting some benefit to the ability while still getting the randomness of the wild magic surge.
Without their being some benefit..... then it's kind of like "why play a Wild Magic sorcerer? Just to watch random stuff occasionally happen that may or may not be a benefit to you?"

The simple truth is a Wild Magic sorcerer is gonna have to deal with a lot of reaction pop-ups anyway for their 6th level "Bend Luck" feature.

Bend luck isn't that problematic as it only triggers if abs(delta) <= 4. I like the idea of 1-2s, we were discussing that already internally. Also like the idea of auto succeed to get something back.


Bug: can't create new working notes cards

Broke in 7a7e789e03ee928f38f40d71d9374b1e9d19eb8f,  which predicted it would break something 😬


QuickSort in python

Good job!!

Kindly star the repo!!

I had already starred it

do I need to do anything else?

No, you are good to go!

why it's showing like this


Change analyzer dependency to 0.28.1+8.

This brings in fixes which are required to support lints with the new
task model.

@pq

LGTM!


Add mailalias_core to dependencies

mailalias_core is not bundled with Puppet 6 so needs to be listed as a
dependency.
The first release only claimed puppet 6 support, but since 1.0.5
puppet 4 and 5 are also supported.

Thanks for doing that. I've just added a few comments.

Looks like this was done in https://github.com/camptocamp/puppet-postfix/commit/5babfd818a001725345951a9f8a782ee54ccb6f4


Windows support is broken

Inquirer v3.x simply hangs on Windows when it is supposed to prompt user for a variable value.
This is fixed in newer versions of inquirer.
#18 includes fix for this problem.

Fixed in v1.1.1. Thanks for your contributions!


Android API level update

We need to update the API level on Android due to Google requirements.

Closed by: https://github.com/HathorNetwork/hathor-wallet-mobile/pull/320


[FEATURE] Add capability to suppress SHOW VIEWS, SHOW TABLES queries in Spark base connector

Is your feature request related to a problem? Please describe.
When listing tables in the connector we’re seeing that in addition to the ODBC SQLTables call, Tableau is issuing a ‘SHOW TABLES IN’ and a ‘SHOW VIEWS IN’ query. These two queries are redundant, and with our setup these perform a lot worse than SQLTables, so we’d like to suppress them. When querying a database with ~2000 tables, SHOW VIEWS can take up to a minute to complete. It seems that these queries are called from the Spark base class of the connector.
Describe the solution you'd like
We would like to be able to set a capability flag to suppress those queries
About You
Name: Christian Stuart
Company: Databricks

Thanks Christian!  I filed enhancement: 1168693

Hey Christian, I have added a capability called CAP_SUPPRESS_ENUMERATE_TABLES_VIA_SQL in our internal code, will be ready to use in our next release, 2020.1. feel free to reach out to me if you have any issues regarding this, thanks!


What type of update are you requesting?
A new version of an existing package
Current Package Identifier
Autodesk.AutodeskAccess
Package Version
<IP_ADDRESS>
Please describe the changes you would like to see
Almost every time I ever try and update this package (different versions), it is broken and refuses to update. The package hash never matches.
I assume that Autodesk is replacing their installer after the update is added, thereby breaking winget checks. That is a terrible practice - a single file with the same name should existing with a single hash. The only other option I see is the person maintaining this update is not updating the hash when a version is updated. All other packages in winget update fine.
Either way:
Found Autodesk Access [Autodesk.AutodeskAccess] Version <IP_ADDRESS>
This application is licensed to you by its owner.
Microsoft is not responsible for, nor does it grant any licenses to, third-party packages.
Downloading https://emsfs.autodesk.com/utility/access/1/installer/latest/AdAccess-installer.exe
██████████████████████████████   514 MB /  514 MB
Installer hash does not match; this cannot be overridden when running as admin

I guess, looking at the URL, since there is no version number in the URL, the more likely option is they've simply updated the installer again.
Would be good for AutoDesk to version installer upload filenames.

The installer was automatically updated to <IP_ADDRESS> by the auto bot on 10/22/24. The hash was set then.
However, the current EXE download has a different hash, digitally signed on Sunday, November 3, 2024 10:11:39 PM.
Looks like AD are releasing the same version multiple times!. Should have had a version bump.

[!NOTE]

Will be resolved by #193429

https://github.com/microsoft/winget-pkgs/issues/163098

Note

Will be resolved by Automatic update of Autodesk.AutodeskAccess <IP_ADDRESS> #193429

Thank you.


[bug] font-family stack not referencing correct font

The font-family stacks are referencing non-spaced versions of the fonts.
"RedHatDisplay"', '"RedHatText"', "RedHatMono"'
As discussed in standup there is a difference between what PatternFly and RHDS would reference.
Should this reference be the following instead?
"Red Hat Display"', '"Red Hat Text"', "Red Hat Mono"'
https://github.com/RedHat-UX/red-hat-design-tokens/blob/80f172e7479507d7fa64bebf831259976de42d27/tokens/font/family.yml#L12-L48

@zeroedin Is it okay to use both? I pulled this from redhat.com.
font-family: RedHatDisplay,"Red Hat Display",Overpass,Overpass,"Helvetica Neue",Arial,sans-serif;

I had an issue with rh-footer not respecting Hebrew fonts. This was the solution I prepared:
--rh-font-family-heading: RedHatDisplay "Red Hat Display", "Noto Sans Hebrew", Overpass, Helvetica, Arial, sans-serif;
--rh-font-family-body-text: RedHatText, "Red Hat Text", "Noto Sans Hebrew", Overpass, Helvetica, Arial, sans-serif;
--rh-font-family-code: RedHatMono, "Red Hat Mono", "Courier New", Courier, monospace;

Since Noto Sans Hebrew only has Hebrew glyphs it should just transparently slot into the font stack

It might be worth our while to pile all those specific glyphsets on to the design system, so as to provide reasonable defaults for any situation. The downside to that approach would be extra bytes over the wire. We could alternatively recommend that users with specific i18n needs set the --rh-font-family-* variables themselves.

Could we not use the [lang] attribute in our base CSS to pass in the proper Noto Sans font stack?
[lang="he"] {
    --rh-font-family-heading: RedHatDisplay "Red Hat Display", "Noto Sans Hebrew", Overpass, Helvetica, Arial, sans-serif;
    --rh-font-family-body-text: RedHatText, "Red Hat Text", "Noto Sans Hebrew", Overpass, Helvetica, Arial, sans-serif;
}


chore: use forc 0.61.2

Checklist

[x] I have added necessary labels.
[x] I have done my best to ensure that my PR adheres to the Fuel Labs Code Review Standards.
[x] I have requested a review from the relevant team or maintainers.

we had a new release so I updated the version


Can't return an ElementFinder from an async function

The ElementFinder may have a .then member. If it does, functions passed to the .then member will be passed a value of type any. The current type annotations for this member do not accurately represent this and cause errors when using async/await.
The errors can be seen by pasting the following code into a TypeScript file:
import { browser, by, element, ElementFinder, ExpectedConditions } from "protractor";

// The ElementFinder we return here is just an ordinary element
// ERROR: [ts] An async function or method must have a valid awaitable return type.
async function getElementWhenVisible(): Promise<ElementFinder> {
    let myElement = element(by.css(".foo"));
    await browser.wait(ExpectedConditions.visibilityOf(myElement));
    // ERROR: [ts] Return expression in async function does not have a valid callable 'then' member.
    return myElement;
}

// elementWithActionResult is an ElementFinder whose parent ElementArrayFinder
// has action results and therefore has a then function. This type of usage does not
// have any issues and compiles fine.
function getActionResultOldSchool(elementWithActionResult: ElementFinder): PromiseLike<string> {
    return elementWithActionResult.then(someValue => {
        return someValue;
    });
}

// elementWithActionResult is an ElementFinder whose parent ElementArrayFinder
// has action results and therefore has a then function
async function getActionResultAsync(elementWithActionResult: ElementFinder): Promise<string> {
    // ERROR: [ts] Operand for await does not have a valid callable 'then' member.
    return await elementWithActionResult as any;
}```

Thanks for investigating. Historically, we removed the then function on elements because it prevented bad use cases where there was nothing to resolve to. This might be a valid reason for returning it though - or at least messing with the typing.

Just to add some clarification for anyone reading this:
https://github.com/angular/protractor/blob/master/lib/element.ts#L808 shows that the then function is optional (may or may not be there, and hence should be declared as then?). However, https://github.com/angular/protractor/blob/master/lib/element.ts#L795 declares it as always being there.
Every async function undergoes a transpilation process (whether at transpilation time or at runtime on supported engines) so that it returns a promise. The return value of the function is what is used to resolve that promise.
If a "standard" ElementFinder (one that does not have a then function on it) is returned from an async function, then the promise returned by the async function will be resolved with a reference to that ElementFinder. See getElementWhenVisible() in the example above which currently fails to compile.
However, if the promise implicitly returned by the async function is resolved with a thenable (i.e. we returned an ElementFinder which had a then function) then the promise returned by the async function will be resolved with the value which would be given to a callback passed to the thenable (which in this case is ElementFinder.then, representing the action result of the parent ElementArrayFinder). An example of this is in getActionResultAsync() in the example above.
I've submitted a PR with what I believe is the correct typing for the then function:
https://github.com/angular/protractor/pull/3835
I submitted this PR on the basis that it would be incorrect to leave the then function declared as always being there (i.e. not suffixing with ?) because this doesn't reflect the runtime reality.
I should note that it may be possible to have a subclass of ElementFinder which has the then function declared as always being there (possibly even a generic class which types the return value of then), and removing then altogether from the normal ElementFinder, but this would definitely involve changing a quite a bit more code.

@juliemr  I think this issue can be closed as the PR was merged :)

@massimocode Your PR #3835 did not fix the issue. Typescript still complains "An async function or method must have a valid awaitable return type."

@awaisraad the PR did fix it at the time making the the then function optional. Since then in typescript 3.9 they've added stricter checks to optional properties. This is the reason why you're getting this issue now.

@Fuun347 thank you for clarifying.

Is there any known workaround for this issue in Typescript 3.9?

@blidblid yes the workaround is to update the types for ElementFinder in element.d.ts.
Remove
then?: (fn: (value: any) => any | wdpromise.IThenable<any>, errorFn?: (error: any) => any) => wdpromise.Promise<any>;
and it will be all okay.

@blidblid yes the workaround is to update the types for ElementFinder in element.d.ts.
Remove
then?: (fn: (value: any) => any | wdpromise.IThenable<any>, errorFn?: (error: any) => any) => wdpromise.Promise<any>;
and it will be all okay.

sorry but that's not a workaround at all. How Am I supposed to remove that in a CI job where the dependencies are installed on a remote server then immediately built? Very dissapointed to see that this post is 2 years old and there is still no real fix.
This is causing real issues for us updating to angular 10.

@Roaders for now I cannot see a way to remove from protractor as I believe it would be very breaking. The only real workaround I see is not to return ElementFinder or ElementArrayFinder. Also just to clarify, this bug reported in 2016 is not related to the current issues. The current issue is caused by Typescript 3.9 (Changelog). In 3.9 they have added stricter checks to optional properties, which now catches the optional then.
I cannot really also classify this as a bug as it's intended behavior. You should not be able to return a thenable and ElementFinder and ElementArrayFinder where indeed thenables.


Broken on master

Using julia master and DataFrames master.
using DataFrames

ERROR: LoadError: LoadError: error in method definition: function StatsBase.describe must be explicitly imported to be extended
 in include at ./boot.jl:254
 in include_from_node1 at ./loading.jl:133
 in anonymous at no file:110
 in include at ./boot.jl:254
 in include_from_node1 at ./loading.jl:133
 in reload_path at ./loading.jl:157
 in _require at ./loading.jl:69
 in require at ./loading.jl:52
while loading /Users/malmaud/.julia/v0.4/DataFrames/src/abstractdataframe/abstractdataframe.jl, in expression starting on line 66
while loading /Users/malmaud/.julia/v0.4/DataFrames/src/DataFrames.jl, in expression starting on line 79

Looks like you just need to replace describe with StatsBase.describe when defining it. But I don't understand why it has started to fail now...

+1 Also getting this on 0.4
Version 0.4.0-dev+5624 (2015-06-26 20:21 UTC)
Commit 2b07246* (0 days old master)
x86_64-unknown-linux-gnu


Feature/#32 add refreshpaths method

Coverage increased (+0.003%) to 99.262% when pulling d497c7d524fe2d653dd7c705c68df177fb73a8da on feature/#32-add-refreshpaths-method into 5ed2ac8d4ec77623f769ee40784cf46d9cee288d on master.

Released as 0.9.0. I also added resource.$refresh for symmetry.


Refactor: split overly large _base.py file

What was a single _base.py is now

_crs.py (pyproj CRS wrapper)
_geom.py ("Shapely Geometry with CRS" class and helpers)
_geobox.py (GeoBox class and related utilities)

This had to be done carefully to preserve git history across the split, hence two commits (step 1|2)
Checking history is preserved on all 3 new files:

https://github.com/opendatacube/odc-geo/blame/refactor/odc/geo/_geobox.py
https://github.com/opendatacube/odc-geo/blame/refactor/odc/geo/_geom.py
https://github.com/opendatacube/odc-geo/blame/refactor/odc/geo/_crs.py

Note that there is a weak intentional cyclic dependency between _crs.py and
_geom.py. Mostly _geom needs CRS, that's a fundamental dependency. But CRS
also uses Geometry classes for some convenience methods, namely to report "valid
area" for a given CRS object.

Right now almost everything is exposed from one flat namespace, hence the _. But we should think about having better hierarchy than just flat namespace.


Fixing subSkel behavior to be ViUR2 compliant

the subskel named "*" was always integrated in the past
it was possible to create subskels without the "key" bone
IMHO "creationdate" and "changedate" should also be taken always

Ok I removed "creationdate" and "changedate" again, rest remains. Please use "squash & merge" to minimize number of commits.


fileHandler saves written files to tmp folder

Codecov Report

Merging #128 (17141b9) into main (ab487f7) will decrease coverage by 7.44%.
The diff coverage is 30.00%.

@@            Coverage Diff             @@
##             main     #128      +/-   ##
==========================================
- Coverage   85.22%   77.77%   -7.45%
==========================================
  Files          10       10
  Lines         176      198      +22
  Branches       16       17       +1
==========================================
+ Hits          150      154       +4
- Misses         24       41      +17
- Partials        2        3       +1

Impacted Files
Coverage Δ

backend/controllers/admissions.js
68.83% <ø> (-12.13%)
:arrow_down:

backend/services/fileHandler.js
30.00% <30.00%> (-3.34%)
: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 ab487f7...17141b9. Read the comment docs.


Awesome Books project

Hello!
please find below some interesting javascript features we just set up in this project.
kindly have a look and let us know if there's anything that needs improvements.
General requirements

Make sure that there are no linter errors. 💯
Make sure that you used correct Github Flow. 💯
Make sure that you documented your work in a professional way. 💯

Faithfully yours

thank you dear @Omar-Muhamad .
we've updated the content and fixed the error. 🙏 👏
best regards


Fixes Issue #191

Fixes Issue #191

The root of this bug is where the expansion of \e to the
binary escape character happens.  In the case before this PR
it's being done by printf.

FreeBSD does not have \e in its printf implementation.
Linux does.

The fix I am suggesting is to get the shell to do the expansion
instead.  By changing the string definitions from using double
quotes to dollar single-quote format - the shell will do the
expansion.

I'm unfamiliar with the $'..' syntax. And it seems to be undefined in POSIX sh (according to the CI linter)

I just noticed that.  I need to work out why that is and if that's the case - how does POSIX sh deal with esc?
(I mean, I could just use ^v, but that's binary in a shell script which we try to avoid).

To add insult to injury, their recommended fix won't work on FreeBSD.  I'll look for other alternatives before I consider direct insertion.

Would \033 work to replace \e here?

Maybe - I was just booting my FreeBSD box to look at the printf man page.
On the bright side, $' ' is on its way into POSIX by the looks of it.
But it's been in a committee for 11 years.
https://www.austingroupbugs.net/view.php?id=249

Looks promising - I'll test it on FreeBSD, Linux - and if successful, push it.

Looks good - re-doing the PR


Missing git tag for version 2.3.4

There are git tags for many other versions, just not for 2.3.4. Which might be nice for posterity.

forgot about that, thanks

Just FYI: I think that 2.3.4 as released on npm was actually 514de88, not 9e13ea9.
The diff is a single line to the README.md, but I noticed it and just wanted to post it. I don't expect any action.
npm version 2.3.4 doesn't contain  commit 8a5fb48fd4c10ac28b3c4a6feb08d52657a8958a which is a change to the README.md:
commit 8a5fb48fd4c10ac28b3c4a6feb08d52657a8958a
Author: Cristian Garcilazo<EMAIL_ADDRESS>Date:   Fri Mar 8 15:33:57 2019 -0300

    Add Data Providers to readme

diff --git a/README.md b/README.md
index 255170a..9bf185b 100644
--- a/README.md
+++ b/README.md
@@ -39,6 +39,7 @@ Also check https://cli.vuejs.org/guide/build-targets.html#library

 - [Draxed](https://www.draxed.com/?utm_source=github&utm_medium=web&utm_campaign=vue-grid-layout)
 - [cryptotiles](https://www.cryptotiles.io/?utm_source=github&utm_medium=web&utm_campaign=vue-grid-layout)
+- [Data Providers](https://www.dataproviders.io/?utm_source=github&utm_medium=web&utm_campaign=vue-grid-layout)

 *Know of others? Create a PR to let me know!*


User avatar if one exists, if not use Initials

I know that I have 2 options for user image in the config file:
'avatar_image_database_field'   => 'avatar',
Will show the avatar in my DB
'avatar_image_database_field'   => '',
Will show the username initials and a color background.
Is there a way to defualt to the Initials if the avatar field is null? That is how my app is set up so the forum would be nice to see like that as well. Just from a UI standpoint, it would be a nice option for anyones app since as of now there is just no image if the user doesnt set one.

you need to override

home.blade.php

discussion.blade.php

and add your logic there you can see how to override these php files on the main documentation page under OVERRIDING VIEWS  heading, you can search for chatter_avatar class in these two php files this class is assigned to the div that contains user avatar


Deprecate old create endpoints

At this point every client should be migrated to the new v2 CREATE2 endpoints, so old endpoints should be deprecated to prevent people creating old v0.1.0 Safes. The endpoints to be deprecated are:

POST /v1/safes/
POST /v1/safes/estimate/
PUT /v1/safes/{address}/funded/

What do you think @tschubotz @rmeissner @DmitryBespalov?

do you have any stats on usage?

@rmeissner That endpoint has created:

1 Safe in November (Dima's testing one)
No Safes in October
1 in September
6 in August
1 in July
54 in June
115 in May

I would say there's no risk on deprecating the endpoint in next version

I would say I don't really know if the people who created safes with old endpoints have upgraded to the new app version. Maybe they still have the safe but did not actually use it and it sits on their phone.
I would be careful with our users on that side. I'd propose to send out a push notification and or write a blog post that we are deprecating old versions and people are encouraged to update to ta new version.

@DmitryBespalov This change would just affect people creating new safes, not for doing txs with old safes. I think it has been enough time to update the apps to the new versions. In my opinion we should just deprecate the endpoints and not make a big deal of that


Add optimizer overrides to explorer

TT-explorer now supports editing output tensor layout of ops from the UI.
Changes made:

Create editable fields for op output layout instead of readonly attributes.
Keep perf trace as state for every executed model on the backend
Clean up location parsing logic
Fix a few bugs in override parsing / handling

Closes #1178

@vprajapati-tt I did make some changes to how location is parsed and used. We might be missing functionality here to cover all the usecases discussed. Let's revisit this after the holidays.


fix: session storage bugs

Summary | Résumé
No issue for this pull request.
Two main bugs fixed.

When a user logged out and logged back in with a separate account they were able to continue editing a form which they built using their other account.
The resume edit button would always show up on the myforms page even if form creation has not been started since the layout was set to use the Template component which is wrapped by the TemplateStoreProvider that would initialize the sessionStorage.

Test instructions | Instructions pour tester la modification

Log in and go to the myforms page. Confirm that only the "create a form" button shows up on the page.
Click this button and start drafting a form. Confirm the creation of the sessionStorage key
Now log out and confirm that the sessionStorage has been cleared

Unresolved questions / Out of scope | Questions non résolues ou hors sujet
N/a
Pull Request Checklist
Please complete the following items in the checklist before you request a review:

[x] Have you completely tested the functionality of change introduced in this PR? Is the PR solving the problem it's meant to solve within the scope of the related issue?
[ ] The PR does not introduce any new issues such as failed tests, console warnings or new bugs.
[ ] If this PR adds a package have you ensured its licensed correctly and does not add additional security issues?
[ ] Is the code clean, readable and maintainable? Is it easy to understand and comprehend.
[ ] Does your code have adequate comprehensible comments? Do new functions have docstrings?
[ ] Have you modified the change log and updated any relevant documentation?
[ ] Is there adequate test coverage? Both unit tests and end-to-end tests where applicable?
[ ] If your PR is touching any UI is it accessible? Have you tested it with a screen reader? Have you tested it with automated testing tools such as axe?

@Moro-Code A pull request from @timarney (https://github.com/cds-snc/platform-forms-client/pull/1437) was merged this morning to fix bug (2). Am I correct @timarney ?

@Moro-Code A pull request from @timarney (#1437) was merged this morning to fix bug (2). Am I correct @timarney ?

I pulled develop to see if it was fixed and it was there so not sure but let me check

@Moro-Code A pull request from @timarney (#1437) was merged this morning to fix bug (2). Am I correct @timarney ?

I pulled develop to see if it was fixed and it was there so not sure but let me check

@craigzour yes I pulled his changes down and it didn't fix it. The Template component is wrapped with the TemplateStoreProvider which initializes the sessionStorage and so it will never be empty

@Moro-Code A pull request from @timarney (#1437) was merged this morning to fix bug (2). Am I correct @timarney ?

I pulled develop to see if it was fixed and it was there so not sure but let me check

@craigzour yes I pulled his changes down and it didn't fix it. The Template component is wrapped with the TemplateStoreProvider which initializes the sessionStorage and so it will never be empty

@Moro-Code  Just tested it once again on the develop branch. Maybe you are talking about a different issue then?
https://user-images.githubusercontent.com/1763961/207908067-4ed0bdcd-12c7-48e4-a51f-9fb048b2e075.mov

@Moro-Code A pull request from @timarney (#1437) was merged this morning to fix bug (2). Am I correct @timarney ?

I pulled develop to see if it was fixed and it was there so not sure but let me check

@craigzour yes I pulled his changes down and it didn't fix it. The Template component is wrapped with the TemplateStoreProvider which initializes the sessionStorage and so it will never be empty

@Moro-Code Just tested it once again on the develop branch. Maybe you are talking about a different issue then?
Screen.Recording.2022-12-15.at.10.57.50.AM.mov

Weird let me take a look again. Thanks for pointing this out @craigzour

RE: #1437 it's a different scenario --- that one looks to see if we actually have a session that should be resumed ... current check is do we have a form with a title.
The issue I was finding was we could end up with a session i.e. initialize was called to reset the form (in turn resetting session to empty) in which case there's really nothing to resume.

@Moro-Code A pull request from @timarney (#1437) was merged this morning to fix bug (2). Am I correct @timarney ?

I pulled develop to see if it was fixed and it was there so not sure but let me check

@craigzour yes I pulled his changes down and it didn't fix it. The Template component is wrapped with the TemplateStoreProvider which initializes the sessionStorage and so it will never be empty

@Moro-Code Just tested it once again on the develop branch. Maybe you are talking about a different issue then?
Screen.Recording.2022-12-15.at.10.57.50.AM.mov

Weird let me take a look again. Thanks for pointing this out @craigzour

@craigzour seems like it has indeed been fixed. I will revert my changes for that aspect then


⚠ ClamAV has degraded performance

In 822ba09, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 490 ms

Resolved: ClamAV performance has improved in e298676.


api: remove cc_generic_services dependency.

We can dynamically discover service/method descriptors in Envoy, so we
don't need to generate any C++ stubs for this. This simplifies the
Google import and removes an unnecessary build output.

@lizan

@mrice32

LGTM


Implement wire cutting as a two-qubit instruction

[x] wait for #153 to merge
[ ] add new Move instruction to Sphinx API References
[ ] write a tutorial notebook

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

30 of 30 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.1%) to 88.258%

Totals

Change from base Build<PHONE_NUMBER>:
0.1%

Covered Lines:
2082

Relevant Lines:
2359

💛  - Coveralls

We'll need to remove the "limitation" from the cutting explanatory when this goes in.
https://qiskit-extensions.github.io/circuit-knitting-toolbox/circuit_cutting/explanation/index.html

This looks great.
One question: I understand that the Move instruction decomposes to a swap and a reset when "decomposed" by Qiskit. What I don't understand is when in our workflow that SWAP happens or when we add the SingleQubitQPDGates containing the wire cut QPDBasis. Going to keep studying it but maybe you can say something about how this happens in our current workflow

After looking at the tutorial, I think #326 is what I was missing. Looks good, modulo the a small comment or two