Created TwillModel trait and TwillModelInterface

Description
This PR creates a trait and interface for the twill model functionality. Why? To allow easier integration for more complex systems. For example, you may be using models provided by another third party package, if you wanted to manage these using Twill you'd have to copy/replicate all that functionality within your new models. That's because we can only inherit from one class, in this case either the Twill model or the third party model.
To rememedy this we can take advantage of a trait and interface. That should ensure we can easily integrate Twill when we encounter scenarios like above.
For backwards compatability I've kept everything wrapped within the base model (A17\Twill\Models\Model). So previous implementations should still work and those looking to take advantage of third party models can implement the interface and use the trait.
More on this
Do I think this is a good idea? Honestly I'm not 100% on it, I'm hoping to get some feedback on this and see where it goes, maybe this is the type of thing that should be left for 3.x
Do I think it's useful? Yes, for the most part. I've implemented this into a work project, and why that almost made me feel this being in the core codebase was unnecessary, I ran into static analysis issues since the module repository (A17\Twill\Repositories\ModuleRepository) type hints at the base twill model. On this, just the interface if nothing else could also help here.
Let me know what you think 👌 (I've ommited some doc block changes from this PR to avoid drowning out the actual code changes for now)

Hi @OhKannaDuh,
This looks good! I like the idea of introducing interfaces for core Twill concepts in 3.x and beyond. With better use of the Laravel service container, perhaps this could be a good way to provide more extensibility for Twill.
I also agree that dockblocks and typehints can be improved overall.
I'm not 100% convinced about the TwillModel trait but it's nice to have this as a proof of concept. I'll play around with this on my end. Hopefully this PR can be a great place for further discussion. :)

Yes I'd absolutely love to see something like that implemented in the future.
Is there a roadmap for 3.0?

Hey @OhKannaDuh,
I will be merging this pr into another working branch of mine. I have removed the trait and kept the abstract model implementing the interface.
I will further extend where needed in the other branch.
Thanks a lot for your contribution!


A proposal for a valid research question for the literature search - Loss function

For the RQ of SLR of keypoint detection, I already told you to have separate sub-questions for dataset, task, etc.
There is an additional one which is required which is  loss function?
Loss function is more important as it will give you and answer to what uncertainty estimation can be used.
So the research question set can be :
Strucutred literature survey of keypoint detection:
RQ1.1 : What are the different papers in keypoint detection
RQ1.2 : Which are the different tasks  are solved using keypoint detection
RQ1.3 : Which datasets are available for each keypoint detection task?
RQ1.4 : Which loss functions are used for keypoint detection task ?
This can be answered in a table like this

Paper
Task
dataset
loss function
any other important metric

[1]
T1, T2, T3]
[D1, D2, D3]
[L1, L2, L3]
[.....]

The table then can also be converted to a graph for better understanding . Like here
https://towardsdatascience.com/create-a-graph-database-in-neo4j-using-python-4172d40f89c4
Or the graph can be the mind - map

Thanks for the input. I have added the sub-questions in the proposal.
R_D_PROPOSAL_Uncertainty_Estimation.pdf


[Automation] Generate SDK based on TypeSpec 0.20.0 DEV

[Automation] Generate SDK based on TypeSpec 0.20.0 DEV

API change check
API changes are not detected in this pull request.

You can ignore the error in healthinsights, as this lib is not synced before (we are waiting for their GA PR).
However, the api-version should not changed? (as the commit ID stays same)


corrupted jest-report.html with some test names

🐛 Bug Report
jest-report.html is corrupted and cannot be displayed in the browser if some test includes on its name the $'character sequence.
To Reproduce
Steps to reproduce the behavior:
1 .jest simple.test.js (see code below)
2. open jest-report.html in the broswer
-> The following error is thrown:
Uncaught SyntaxError: JSON.parse: bad character in string literal at line 1 column 1259 of the JSON data
    <anonymous> file:///home/alostale/ws/projects/react/openbravo/test-reports/jest-report.html:118
    <anonymous> file:///home/alostale/ws/projects/react/openbravo/test-reports/jest-report.html:118
    o file:///home/alostale/ws/projects/react/openbravo/test-reports/jest-report.html:61
    <anonymous> file:///home/alostale/ws/projects/react/openbravo/test-reports/jest-report.html:118
    o file:///home/alostale/ws/projects/react/openbravo/test-reports/jest-report.html:61
    <anonymous> file:///home/alostale/ws/projects/react/openbravo/test-reports/jest-report.html:61
    <anonymous> file:///home/alostale/ws/projects/react/openbravo/test-reports/jest-report.html:61

simple.test.js
it("Cannot generate report with '1.00 $'", () => expect(true).toBe(true));

Expected behavior
jest-report.html should be correctly rendered in the browser.

Link to repl or repo (highly encouraged)

<EMAIL_ADDRESS>envinfo

  System:
    OS: Linux 5.4 Ubuntu 18.04.5 LTS (Bionic Beaver)
    CPU: (8) x64 Intel(R) Core(TM) i5-8265U CPU @ 1.60GHz
  Binaries:
    Node: 13.14.0 - /usr/bin/node
    npm: 6.14.4 - /usr/bin/npm
  npmPackages:
    jest: 26.6.0 => 26.6.0

What's jest-report.html? I don't think I've ever heard of this file

What's jest-report.html? I don't think I've ever heard of this file

I realized this is not core jest but jest-html-reporters.
I reported the issue there, so closing this one.
@SimenB, thanks for your quick reaction.

I realized this is not core jest but jest-html-reporters.
I reported the issue there, so closing this one.
@SimenB, thanks for your quick reaction.


Cannot execute the substraction op with the broadcast mechanism

import tensorflow as tf
import numpy as np
x = tf.constant(np.random.random((500, 6)))
y = x - x[:,0]

This code raises the issue of not supporting [500,6] - [500] in the sub OP.
I suppose that it should compute correctly, using the broadcast automatically?
Is it a bug?
version: tf2.0

Could reproduce the issue with TF version 2.0. Please find the Gist here.
Thanks!

Same problem in div op.
Please have a check on it, too.

@jvishnuvardhan any progress here?

could reproduce the issue with TF version 2.2.0-dev20200319. Please, find the gist here. Thanks!

@TomorrowIsAnOtherDay I updated part of your code. You could use the following to access just a column and subtract it from the whole tensor.
import numpy as np
x = tf.constant(np.random.random((500, 6)))
indices = [0]
y = x - tf.gather(x, indices, axis=1)

Please close the issue if this was resolved for you. Thanks!

 I just want to write the code in a pythonic way, but the solution rolls me back into tf1.0.

This is consistent with the broadcast semantics of numpy:
>>> x=np.random.random((500, 6))
>>> x-x[:,0]
ValueError: operands could not be broadcast together with shapes (500,6) (500,)


ip instead url site

I sometimes have this kind of garbage, maybe it's just me.
elk - 1.0.10
PHP - 7.1.19

And again I didn't get notifications about issues... :-\
hmm... The interesting thing is that at least for 1 of the URLs, the domain is properly displayed.
It seems like at a certain point, some code replaces the correct url of the theme with the wrong one.
Do you have addons installed?

recently the error has disappeared. I think that at php the flag of global replacement sometimes falls off.

installed mods:
Bookmarks
Fancy Box 4 ElkArte
FlarumStyle
HTML BBCode
HTML5 Video Audio BBC
Recent Topics
Spoiler Modern

I know this is an old topic, but I have seen this on one of my servers, and its only (1) site of (5) that ever does the FQDN -> IP address swap ... which then causes the site to display as a mess since the style sheets are missing.
The "culprit" here seems to be function loadThemeUrls() which uses $_SERVER['HTTP_HOST'] and $_SERVER['SERVER_NAME'] for checks, neither of which are really that trustworthy.
TBH I'm not sure why that function is there, maybe it was intended if you mis-entered theme details in the admin panel?  The www redirect is understandable but the rest?
Anyway I think one of those $_SERVER values gets loaded with the IP, this is not an ElkArte issue but something on the server or the server site control panel.  I've only seen this on ispconfig and then only on the first site defined in that panel.  Never seen it on the other sites.
In that function you can add after 		$detected_url .= empty($_SERVER['HTTP_HOST']) ? $_SERVER['SERVER_NAME'] . (empty($_SERVER['SERVER_PORT']) || $_SERVER['SERVER_PORT'] == '80' ? '' : ':' . $_SERVER['SERVER_PORT']) : $_SERVER['HTTP_HOST'];
the following:
		if (strpos($detected_url, 'mydomain.com') === false)
			$detected_url = 'https://mydoman.com';

which simply prevents the IP address from propagating past that point.  Yes leave off the www,  otherwise site based www redirection will not work.  and of-course mydomain.com is your real site name.
perhaps updating that so it uses the boardurl as a check ....

Also it may work to change
			if ($do_fix === true || preg_match('~^http[s]?://(?:[\d\.:]+|\[[\d:]+\](?::\d+)?)(?:$|/)~', $detected_url) === 1)
to be
			if ($do_fix === true) to avoid "fixing" things should the detected url be an IP address and the board URL not being one


Ocr

Simple test for binarization done

Current coverage is 72.76%

Merging #22 into master will increase coverage by +0.50% as of 295e136

@@            master     #22   diff @@
======================================
  Files           15      16     +1
  Stmts          238     246     +8
  Branches        52      52
  Methods          0       0
======================================
+ Hit            172     179     +7
  Partial         20      20
- Missed          46      47     +1

Review entire Coverage Diff as of 295e136

Powered by Codecov. Updated on successful CI builds.


MD - Data Issues & Questions

Describe the bug
It appears data sent by ImageMover is getting into Maryland's feed without CLIAs. Currently Friendship Charter Schools are not using SimpleReport based on my investigation and further confirmation from Emma and Adam (there are no K-12 facilities using SimpleReport in MD). I remembered getting a message from Tyler (ImageMover) inquiring if they can start the process to send Friendship Charter School data via ReportStream for DC.

In addition to the questions above, can someone let me know if there is a way for us to not receive results that do not have CLIAs? We need to make sure there are no message failures. This is coming from Friendship Public Schools

Additional Questions:

Is there any way to know what facilities are included under Report Stream (not just ones we have signed up for SR)?

For OTC data, is there any way to know where this data is coming from? I cannot tell what company/test kit is being used. There is really not much information besides knowing it is a PCR or antigen test.

Impact
Maryland is getting message failure due to missing CLIAs.
To Reproduce
Steps to reproduce the behavior: need to ask Maryland for additional details, if needed.
Expected behavior
Messages sent to Maryland should not cause message failures.
Screenshots
If applicable, add screenshots to help explain your problem.
Logs
If applicable, please attach logs to help describe your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Please see email from Kristy Lunquest in the ReportStream inbox. The email came in originally on 10/1/21 and additional questions came in on 10/4/21. I responded on 10/4/21 with a clarification question. Will work with SimpleReport to respond to the rest of their questions.

Responses from the SimpleReport team for SR related questions:

Can only rapid antigen results be sent via SR? One of our users did not see a place for PCR tests.

SimpleReport will add devices for PCR tests; if it's not in the list, advise facilities to please contact<EMAIL_ADDRESS>to get it added.

We began receiving COVID rapid antigen results from Park Pediatrics earlier this month. However, I am only seeing results from 9/3 on our end, and Cecilia has told me she also reported on 9/4 and 9/10. Were these results received and sent to MDH?

In app, I'm seeing 0 results for 9/3, 2 for 9/4, 3 for 9/5, 11 for 9/7, 9 for 9/8, 2 for 9/9 and 1 for 9/10.

Linking issue to resend Park Pediatrics issue #2562.

@jbiskie can you let me know when the the fix for the MSH-4 is in production? I should let Maryland know or we need to figure out how to temporarily filter out the ones for Friendship Schools. I have a feeling it is causing problems for them.

Issue was addressed with MSH-4 fix.

@jorg3lopez have we resent the files to MD. We need to confirm with them on any lingering items.

Anshul checked with MD (kristy) on 11/12. Awaiting feedback from state on next steps and if any resubmissions are needed.

Per MD on 11/22:
"I wanted the following removed due to CLIA issues, but also because there are no filler order numbers.
Family & Nursing Care Select LLC
Friendship Public Schools"

MD needs us to filter out Family & Nursing Care Select LLC and Friendship Public Schools because they have no accession numbers.

Thanks @anshulkumar-usds. As you probably know both the facilities mentioned by MD are not SR users. I double checked this evening.

Hey @MikeC-A6 requesting your help here. MD is requesting we filter out two facilities from their data feed because these facilities are missing access numbers. Here is their request:
"Is it possible to filter out Friendship Public Schools and Family & Nursing Care Select? We cannot process these as they have no accession number and continue to fail in our system."
I don't see any obvious items missing in COVID metadata table.

MD confirmed issue is resolved.


Update to v0.3.20 from cloudyr drat repo

The latest cran version (0.3.12) is pretty outdated. Lots of bugs were
fixed since then in newer package versions as documented in the closed
github issues. The newer package versions are available via the cloudyr
drat repo as described in the readme of the package itself.

Checklist

[x] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

This is for the "dev" branch and "dev" conda-forge label.


Respect timeout when no cancellation token passed to OpenAmqpObject

CancellationToken should be used when opening links, as the link timeout is already set in the link settings which is what the service will use.
For opening AMQP session/connection, we use a timeout.

API change check
API changes are not detected in this pull request.

/azp run net - servicebus - tests

@JoshLove-msft Quick question about a behavior prior to this fix (SDK 7.7). Would having say an idle queue with many receiver (1k+) where client timeouts before the service, could lead to hitting the concurrent connections limits and trigger requests throttling ?

@JoshLove-msft Quick question about a behavior prior to this fix (SDK 7.7). Would having say an idle queue with many receiver (1k+) where client timeouts before the service, could lead to hitting the concurrent connections limits and trigger requests throttling ?

This would only impact the operation of establishing the connection with the service. So long as you are still disposing the ServiceBusClient, it shouldn't cause excess connections.


Submodule updated with shorter tests

At this time, I created a new submodule. Now, this submodule is pointing at the branch "pom-tests" at the air-water-vv repository. Moreover, the air-water-vv tests that will be run by Travis every time we make a pull request in proteus repository are shorter. I reduced the duration of the simulation in order to try to addapt to Travis limit time.
This change is provisional, since the submodule has to point in the master branch of the air-water-vv tests in the end. In the branch "pom-tests" in the air-water-vv repository I have removed the .travis.yaml file, since the air-water-vv tests will be run in the proteus submodule togheter with proteus.
I was discussing about this with @adimako, but it is not completely sure.
So, we have to decide what is the best way to test all these cases.

BTW, you may want to take a look a this: https://help.github.com/articles/why-are-my-commits-linked-to-the-wrong-user/. For some reason your commits aren't getting linked up properly.

I'm not sure why two of the tests are failing, but it looks like an import path issue. In order to test this you may need to do make distclean; make install and then run the test command as it is down in .travis.yaml

I think I know why these two tests are failing. I am going to try one more time. If not, I will try what you are suggesting.
Thank you Chris.

One thing we could do to speed up the builds is use the travis build matrix support. Right now we run the air-water-vv tests after the main proteus tests:
script:
- py.test --boxed -v linux2/lib/python2.7/site-packages/proteus-$(python -c "import proteus; print proteus.__version__")-py2.7-linux-x86_64.egg/proteus/tests --cov=linux2/lib/python2.7/site-packages/proteus-$(python -c "import proteus; print proteus.__version__")-py2.7-linux-x86_64.egg/proteus
- py.test --boxed -v air-water-vv/Tests/```

Instead, we could use the travis build matrix support, which allows you to define multiple "env's" that run in parallel. To be gin with we could just separate the main proteus and air-water-vv tests, but it would probably be useful to separate the basic proteus regression tests, which run quickly, from the longer tests that run full models. The full test matrix support is documented here:

https://docs.travis-ci.com/user/customizing-the-build/#Build-Matrix

And specific ideas on how to use it to speed up the test suite are here:

https://docs.travis-ci.com/user/speeding-up-the-build/

Another thing we can do to speed up the tests is to cache the .hashdist directory. I think we could do something like this in .travis.yaml:
cache:
  directories:
  - $HOME/.hashdist


WIP : Only space owner can create iterations.

Partial fix for https://github.com/almighty/almighty-core/issues/1301
Added a constraint in the controller to check logged in user from the request.
Tests/modified & added.
Question:
What should be done for existing iterations on production which are not created by space owner?
IMHO, it should be left as is.

Codecov Report

Merging #1304 into master will increase coverage by 0.04%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1304      +/-   ##
==========================================
+ Coverage   60.08%   60.13%   +0.04%
==========================================
  Files         119      119
  Lines       10704    10707       +3
==========================================
+ Hits         6432     6439       +7
+ Misses       3617     3616       -1
+ Partials      655      652       -3

Impacted Files
Coverage Δ

controller/space_iterations.go
78.46% <100%> (+2.65%)
:arrow_up:

controller/workitem.go
80.17% <0%> (+0.66%)
:arrow_up:

Continue to review full report at Codecov.

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

Question:
What should be done for existing iterations on production which are not created by space owner?
IMHO, it should be left as is.

Yes, we don't have to do anything with the existing iterations IMO.


Open popup in new window on mobile

I am trying to open youtube videos in a new window (when on mobile) using this script. Is there currently an option for that? Right now it opens them in the same window.

Yes iam  using private video on website.. On desktop its play fine. without any password.. but on mobile autoplay not working.. Need HELP!!


[patch] Runnable flexibility and a new output signal

Now we always use the same callback function, but each step of the Runnable.run cycle flexibly lets child classes define kwargs for each step. This lets us stop passing the callback function as an argument and just always use Runnable._finish_run.
In Node, the new kwargs mean we can move firing the signals over to the _run_finally method, and take advantage of this to add a new failed output signal.
There's a decent amount going on under the hood here, but for users there is no incompatible changes, just the new signal.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>1
Details

0 of 0   changed or added relevant lines in 0 files are covered.
39 unchanged lines in 4 files lost coverage.
Overall coverage decreased (-0.007%) to 91.247%

Files with Coverage Reduction
New Missed Lines
%

io.py
3
94.21%

mixin/run.py
10
90.2%

node.py
11
90.3%

nodes/standard.py
15
90.96%

Totals

Change from base Build<PHONE_NUMBER>5:
-0.007%

Covered Lines:
2992

Relevant Lines:
3279

💛  - Coveralls


fix: add reusable useHydrateField

This enables custom hydration and possibly rehydration via an underlying options flag.

I don't believe this belongs in the standard library

The hook encapsulates the implementation detail, that the fieldAtom has 2 atoms namely value and _initialAtom.
So that way, the hook is a glue between the jotai hook & the internals of fieldAtom.
Given that its already used in the useFIV hook, or put differently I've only extracted it, it makes sense to me that its the part of the core.
It does not extend any functionality, beyond the fact that the options flag is now not hidden. hmmm

It absolutely, positively does not necessitate its own abstraction in any way, shape, or form. I will not budge.
https://github.com/form-atoms/form-atoms/issues/68#issuecomment-1937821799

Interesting I would not call this abstraction at all. I think I've only refactored the code - extracted a reusable procedure.
I think of the useHydrateAtoms from jotai as a generic util which can work for aaaany custom thing you need.
But then as you have a fieldAtom, when you apply its shape into the generic util, you get a concrete one useHydrateField which specifically says, that ok, we are hydrating with the jotai util, but this one we need this exact structure.
Maybe we are of different schools :D


w550io + nrf51(pca10028) : Network initialization succeeds until first restart of host after it has been freshly flashed(programmed)

Dear all,
i would like to share my new observation.......
The network initialization always succeeds in the case of a recently programmed host(nrf51). But when i restart the host, network initialization fails.
But if i reprogram(flash) the host with the same .hex file, network initialization succeds.......what could be the reason

The following image should give you an idea of the scenario.

The following image should give an idea about the HW connections

I recommend you post the content on the forum. forum.wiznet.io
It will be difficult to hear the answer to your application here.
Thanks.


Helm release deployed but Not able to connect to database

Hi,
We have a running Helm release for mediawiki and we are able to connect to it successfully
I recently deployed a helm release for media wiki using the same helm charts as below:-
suv@Suvankars-MacBook-Pro[7:18:34]:~/thermeon/gke-staging-envs/namespaces/suv/releases$ kubectl get helmreleases
NAME            RELEASE                      PHASE       STATUS     MESSAGE                                                                          AGE
elasticsearch   elasticsearch-helpwiki-suv   Succeeded   deployed   Release was successful for Helm release 'elasticsearch-helpwiki-suv' in 'suv'.   6d22h
help-wiki       help-wiki-suv                Succeeded   deployed   Release was successful for Helm release 'help-wiki-suv' in 'suv'.                6d22h

suv@Suvankars-MacBook-Pro[7:19:09]:~/thermeon/gke-staging-envs/namespaces/suv/releases$ kubectl get all
NAME                                               READY   STATUS    RESTARTS   AGE
pod/elasticsearch-helpwiki-suv-6f7579c77c-vktc6    1/1     Running   0          37h
pod/help-wiki-suv-thermeon-wiki-64d995978f-kbxjg   3/3     Running   0          37h

NAME                                  TYPE       CLUSTER-IP      EXTERNAL-IP   PORT(S)          AGE
service/elasticsearch-helpwiki-suv    NodePort   <IP_ADDRESS>    <none>        9200:32063/TCP   6d22h
service/help-wiki-suv-thermeon-wiki   NodePort   <IP_ADDRESS>   <none>        80:31159/TCP     6d22h

NAME                                          READY   UP-TO-DATE   AVAILABLE   AGE
deployment.apps/elasticsearch-helpwiki-suv    1/1     1            1           6d22h
deployment.apps/help-wiki-suv-thermeon-wiki   1/1     1            1           6d22h

NAME                                                     DESIRED   CURRENT   READY   AGE
replicaset.apps/elasticsearch-helpwiki-suv-6f7579c77c    1         1         1       6d22h
replicaset.apps/elasticsearch-helpwiki-suv-7d5b46bbc6    0         0         0       6d22h
replicaset.apps/elasticsearch-helpwiki-suv-7d89995844    0         0         0       6d22h
replicaset.apps/help-wiki-suv-thermeon-wiki-64d995978f   1         1         1       6d22h
replicaset.apps/help-wiki-suv-thermeon-wiki-86bd4ddfbf   0         0         0       3d22h
suv@Suvankars-MacBook-Pro[7:19:20]:~/thermeon/gke-staging-envs/namespaces/suv/releases$
suv@Suvankars-MacBook-Pro[7:19:20]:~/thermeon/gke-staging-envs/namespaces/suv/releases$
suv@Suvankars-MacBook-Pro[7:19:20]:~/thermeon/gke-staging-envs/namespaces/suv/releases$
suv@Suvankars-MacBook-Pro[7:19:47]:~/thermeon/gke-staging-envs/namespaces/suv/releases$ kubectl get sealedsecrets
NAME                AGE
database-password   24h
ldap-password       24h
wiki-secret-key     24h

help-wiki - https://gist.github.com/suvsap/6f5933d270dbfdbd0afcb1a83bc879e3
elasticsearch- https://gist.github.com/suvsap/8b74d52ccd0fe495153560632f9f5f5b
When i try to check the url as below it gives me error

Can you kindly help how to trouble shoot this issue?

The production url working fine as below

I have used the same helm chart but just created the helm release in dev environment. Not sure how to resolve this


Not able to open Lapce on Windows 11

Lapce Version
I don't know the version as I wasn't able to open it. But it is the latest version I downloaded from the website just now.
System information
Windows 11 Ghost Spector
Describe the bug
I installed it and opened it then I got the following 3 errors.

Also please tell me can I run HTML file on localhost using Lapse? If not then add this feature please.

Duplicate of https://github.com/lapce/lapce/issues/19


Doesn't work on High Sierra

Hello,
after the upgrade to High Sierra, looks like Amphetamine is not working anymore. No errors, session can be started properly, but my MacBook will sleep anyway.
Even tried to reinstall to check if it was lacking some kind of permissions.
Can you confirm - disconfirm this?
Thanks

I am having trouble with Amphetamine on a system with High Sierra. I was using it to keep the system awake while encoding/burning a DVD, and simultaneously copy a 30GB file from an external SSD to a system drive. All activity slowed away down, and I stopped the copying and tried to eject the disk. After waiting several minutes I yanked out the USB plug, and the error message was that Amphetamine was still using the Photos Library (which I was trying to copy). I restarted a few times and same issue. The encoding never finished, either (10+ hours!). I quit Amphetamine and the copy worked fine.

Not working on SIerra 10.12.6

Still not working on High Sierra 10.13

I'm also experiencing this. Amphetamine shows active in the task bar, but computer sleeps regardless.

Hi All,
I had this issue on one of my Macs, which is running High Sierra, and found that resetting the SMC on that Mac resolved the issue. In my experience, I found that the problematic Mac was not respecting the PreventUserIdleDisplaySleep  power assertion that Amphetamine puts in place to keep the Mac awake. I found that other "keep-awake" utilities (Caffeine, built-in caffeinate command line utility, KeepingYouAwake, etc.) also did not work. These other apps work in a similar way to Amphetamine - by applying a PreventUserIdleDisplaySleep assertion. After resetting the SMC on my Mac, all of these apps started working again as expected.
Apple has a knowledge article on how to reset your SMC. Make sure you follow the instructions for your specific Mac: https://support.apple.com/en-us/HT201295.
And if you're unfamiliar with what the SMC does on your Mac, here is an informative article: https://support.apple.com/en-us/HT201295.

@x74353 , on which mac os u 've tested that ?

@ArtemBernatskyy , 10.13 (17A405) and 10.13.1 (17B48).

After resetting SMC  caffeinate  still isn't working (

@ArtemBernatskyy , if the built-in caffeinate command no longer works, then this is a bug within macOS or SMC firmware itself. Note that in order to actually keep your Mac's display awake, you need to run the command with the d parameter: caffeinate -d
I initially filed a bug report with Apple back when I was able to replicate the issue, but have since withdrawn it after the SMC reset fixed the issue for me. There seems to be mixed results with the SMC reset fixing the issue. Some people report the issue is resolved by an SMC reset, others say it persists or returns. See this thread for the KeepingYouAwake app:
https://github.com/newmarcel/KeepingYouAwake/issues/92
You can file a bug report with Apple about the issue, but my suspicion is that your SMC or macOS install is still corrupted. You might try testing from a known-good installation of macOS. Example: install macOS on an external drive, boot to the external drive, and run the caffeinate -d command. What are your results?

Sorry to say that but caffeinate -d still isn't working (

@ArtemBernatskyy , you'll need to contact Apple if the caffeinate command is no longer working on your Mac.

what should happen if i use "caffeinate -d" in terminal? i am not sure, if amphetime works now

Should it work with closed lid ? Or it should only prevent sleep when lid is open...

@oasis3000 - you can read the man page for the caffeinate utility to understand its function. Type man caffeinate into a Terminal window. Alternatively, read about it here:
https://mac-how-to.gadgethacks.com/how-to/take-control-your-macs-sleep-functions-with-these-commands-0168109/
@ArtemBernatskyy - Macs will only stay awake in clamshell/closed lid mode when very specific requirements are met. If you are not meeting these requirements, expect your Mac to sleep:
https://support.apple.com/en-us/HT201834

For me with High Sierra it is kinda broken, too. While it seems that it actually keeps the Mac awake, the screensaver function is broken. I can't get it to trigger the screensaver after a set time. Any thoughts on this one? I reset the SMC and also reset the prefs.

Since update to High Sierra my graphics was broken: artefacts and flickering screen up to a total black screen. I have tried and searched anything (yes also pram and smc reset) without success. After all the reason was Amphetamine. When killing this app my Mac runs as before. So I have to warn using Amphetamine together with High Sierra (10.13.3).

@10w73 This is a known issue. Screensaver activation is not working at the moment. Sorry a fix is taking so long.
@CarCode I fail to understand how Amphetamine could possibly cause the issue you've described. Does running the caffeinate -d command in Terminal also cause graphics distortion? Remember - correlation does not imply causation.

Me too, I don't understand why Amphetamine results in such a bad behavior. But after it is killed and completely erased with all its files my Mac is OK again. No artefacts and screen flicker anymore. I do use Caffeine now without problems.

Further investigation shows that Amphetamine switches off the high graphic use in the power settings. Normal graphic use is not sufficient anymore for High Sierra at least for my machine. This bug of Amphetamine should be fixed.

caffeinate -d works in my case, but Amphetamine doesn't.
I have checked the logs and I can see hundred of warnings like this
attempt to set <private> for key in <private> in non-persistent preferences domain CFPrefsPlistSource<0x6040000e0d80> (Domain: com.if.Amphetamine, User: kCFPreferencesCurrentUser, ByHost: No, Container: (null), Contents Need Refresh: Yes)

Also each time I open Amphetamine the welcome window at launch, even if I uncheck the "Show this window at launch", It seems the preference is not saved

@fernandomora Glad you were able to get Amphetamine working. If your preferences are not saving correctly, you will need to try deleting them using the following Terminal command (quit Amphetamine before running this):
defaults delete com.if.Amphetamine


Use asm unshadowed and upgrade to 9.4

Big heads up:

I did not run many tests and I even saw a failure. Not sure if I had something setup wrong or what.
The test I added here passes even when I had 9.2 in use so I know it's not working. That said, when pulling from maven local the snapshot version I no longer see the error on my project so I know this fixes it (but could cause other problems).

I will look into running the full suite (and even the quick suite) soon, but this is just something quick and dirty so show the proof of concept.
For what it is worth, here is my project where the fix is shown in use. It's not really anything to look at but if you'd like to clone it and try you can. The buildHealth task works with the snapshot build here but not with 1.14.1.
See #798

Steps

[ ] Add a test that fails with Java 19 toolchain and 9.2 shaded asm.
[ ] (Maybe) Catch the exception and try to give a better error message. Something like "Got a class version error, asm may need to be updated from version X.Y to work with newer versions of Java, [link to asm changelog]"
[ ] Actually use new version of asm 9.4 and publish shaded jar 😄 (will need assistance publishing when we get here)
[ ] Use new version of shaded jar and verify the test from step 1 passes
[ ] Publish new plugin version

https://github.com/autonomousapps/dependency-analysis-android-gradle-plugin/pull/813

I got a test failure finally so went ahead and rebased. Sure enough, it works now! Were you interested in the error message thing or the testing against more versions of gradle/java?

@autonomousapps no worries, take your time. Enjoy your Thanksgiving (if you're from the states). Glancing back over this now the change you made should fix the problem found in the issue. At the moment this is just a unit test to make sure it doesn't come back. Essentially your other PR bumping to 9.4 should fix everything, this is just the extra assurance.

Wow you were not wrong with the "Comment out all but the last for speed" comment 😅
I've updated the tests since there have been some breaking changes in the test fixtures support API.
I also updated the spec matrix to reference more versions of Gradle. I'm currently running it only against Gradle 7.6.
I tried to run it against Gradle 8.3 and 8.5, but go some errors. I don't have the bandwidth now to address them, but if you feel like picking this up again I will try to move faster.
I will merge this when green (running against 7.6).

I don't have the bandwidth now to address [Gradle 8.3 and 8.5 errors], but if you feel like picking this up again I will try to move faster.

Don't beat yourself up at all, we appreciate your work. ❤ I might take a peek, but no promises. At the moment I am more interested in getting Java 20 and Java 21 to work (I assume they don't, just responding because I saw this in my email) than getting more Gradle versions to work.


Add diag_fisher functionality to allow the Hessian to be computed "on the fly"

Instead of storing the entire Hessian matrix, diag_fisher=True calculates Hessian rows only when they must be used (i.e. in the relevant step of coordinate descent). It accomplishes this by storing the current state of the 1-D array hessian_rows (which appears in the middle of the sandwich product). This necessitates a greater number of matrix operations (many of them redundant), but also saves memory.
To-Do:

state.hessian does not need to be instantiated if diag_fisher=True; this is not yet accounted for in the current code.
Make sure we've addressed all entry points where the argument diag_fisher can be set by the user.
How do we implement memory views (and possibly nogil) in _cd_fast.pyx?

Please reopen when it's ready for review.


Some form of control over z-order of a layer when adding it

Perhaps this is related to issue 73, but I think it would be helpful to have two different types of tiled maps.
I'm using standard base maps from CloudMade, MapQuest, etc and I'm overlaying a custom tiled map of bike trails on top. Switching "base maps" puts my custom overlay map on the bottom and the base map on top, which is far from ideal.

Thank you!!

Should the zIndex or .setZIndex(x) in 2017?


How to model error states?

I am trying to use this library to model a calculator.  Some commands put the calculator into an error state.  For example trying to press the square root button when the current value is a negative number.  I don't see how I can model this with automat.
I have a state called awaiting_number_input which is used to receive number inputs (i.e. digits 0-9 or decimal point) and then I have an input called apply_unary_operator which accepts an operator class as a parameter.   I then setup an output called _apply_unary_operator which actually uses the class to calculate the result of the operator.  This allows me to have a generic calculator state machine which understands unary operators and binary operators can be applied but does not require specific states or inputs for every operator.
The problem is that the Square Root operator should throw the machine into an error state if the current value is negative.  So I want to be able to define transitions from various states where it is valid to apply a unary operator from to the "unary_operator_applied" state.  These transitions can use the _apply_unary_operator output and I want the output to be able to divert the transition to the "error" state if applying the operator fails.
Is this possible?

It depends what you mean by “error state”. It sounds like just another state to me!
We could use a bit more clarity around what automat does when you feed it an input with no defined transition though.

Imagine a state machine modeling a calculator. Number Entry(some number) ->
Operator(divide) -> Number Entry(0) -> Equals.
Normally I would be going to a state where I can enter another number or
another operator, but this operation failed so I need to divert to an error
state.
I guess I could externalize the logic so I have a wrapper that wraps the
calculator FSM and intercepts all the user inputs and maintain state there.
Maybe my approach which is to keep the internal calculator state (I.e the
numbers and operators entered) in the calculator FSM is not right.
On Mon 11 Nov 2019, 23:40 Glyph<EMAIL_ADDRESS>wrote:

It depends what you mean by “error state”. It sounds like just another
state to me!
We could use a bit more clarity around what automat does when you feed it
an input with no defined transition though.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/glyph/automat/issues/116?email_source=notifications&email_token=AABRLOSQDY2XFWHXHWX2NN3QTHUPJA5CNFSM4I4N2FCKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDYQVBY#issuecomment-552667783,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABRLOQ6TFZIAYQXFCNSIMTQTHUPJANCNFSM4I4N2FCA
.

I'm going to close this just because I don't think it's clear enough as stated what the work involved in closing it would be. Please feel free to reopen with a more precise definition of 'error state'.


non-ARM Default image for post-install-hook in Helm Chart

Description
Observed Behavior:
Fail to deploy on ARM64 clusters.
Its because bitnami/kubectl image does not support ARM arch.
Expected Behavior:
It just works.
Reproduction Steps (Please include YAML):
Try to deploy to ARM64 node / cluster.
Versions:

Chart Version: 0.37.1 / 1.0.0
Kubernetes Version (kubectl version): 1.30.3

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

I think default image for this hook should support both AMD64 and ARM64 architectures.

https://github.com/aws/karpenter-provider-aws/issues/6761 was covered already, and it's in the notes.
That being said I agree with you, because the default should just work on arm64 as well as amd64, which given AWS is pushing graviton more feels like a given.

HI @Vrtak-CZ,
As a workaround you can update your values.postInstallHook.image values in your helm values.yaml file to point to a compatible image with kubectl. ARM compatible version.

+1

Thanks @jigisha620 & @danielloader. I understand that this has been documented and it's "easy" to workaround. But whole karpenter and it's helm chart is compatible with ARM64. But post-install-hook with default config is not. From my point of view this is a bug.
Btw this bug is present in 0.37.1 (i know its "same" as 1.0.0) but upgrade from 0.37.0 to 0.37.1 does not work without any intervertion as "patch" vertsion bump suggests...

We plan to address this issue by using an image that is compatible with both AMD64 and Arm64.

@njtran why is a Bitnami image being used here? It's 293 MB compared to the official kubectl image (registry.k8s.io/kubectl:v1.30.4) which is 55 MB.

Closing this issue as the post install hooks have been removed https://github.com/aws/karpenter-provider-aws/pull/6827


Update the "release notes" and remove references to old ZIP filenames

There are at least 2 additional updates since January 2021:

Can someone please update the page itself to give the updates since January 2021??
It also does not make sense to refer to the filename if it is updated behind-the-scenes without additional context. Either update the page with release notes and file version... or just don't say anything about the version number at all.

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

ID: 6624d44c-f9f6-02f8-2c67-0f0637fed62e
Version Independent ID: 9346a8ff-bab5-be2c-9dc1-d546fda1efa9
Content: Azure icons - Azure Architecture Center
Content Source: docs/icons/index.md
Service: architecture-center
GitHub Login: @EdPrice-MSFT
Microsoft Alias: architectures

Thank you for your feedback! We have assigned this issue to the author to review further and take the next course of action.

Thank you for reporting this, @eponerine.
We've made several updates to the Azure Icons recently, and there are hundreds of new icons added.
The update is available for download here: https://learn.microsoft.com/en-us/azure/architecture/icons/
The release notes are also updated and will be published shortly.
Thanks!
#please-close


Question: Why isn't Eq implemented for Ipld data type?

I see that PartialEq is implemented (via derivation), however, I don't see any annotation in the source for why Eq cannot be implemented. Does this have something to do with Null?

The reason are probably the floats. Though as the IPLD forbids non-numeric floating point numbers (such as NaN), it should be possible to implement it manually.

I'm taking this to mean a PR will be accepted that implements it manually.

A problem I haven't thought about before seeing the PR is: Currently we don't enforce that there are no non-numeric numbers on the data model level. So how do we deal if someone is using a non-numeric value? Do we we just document it properly, that then equality might not work as expected? Should we put debug inserts in there? Any other ideas?
Thoughts from other would be appreciates (cc @Stebalien).

Is there a way to prevent the export of the individual enum constructors and use smart constructors that forbid the use of NaN and Infinity? That's the way I'd do it in Haskell. This would prevent manual construction and enforces that Iplds can only be constructed through its blessed APIs

So it appears Rust does not support private constructors for enums. I will adjust the PR to newtype f64 into a value that ensures finiteness.

I'd simply define Ipld::Float(NAN) == Ipld::Float(NAN), etc. We're not asking "are these numbers equal", we're asking "are these two IPLD nodes equivalent".
Really, equivalence should be defined as a == b <-> canonical_encoding(a) == canonical_encoding(b).

(which assumes NaN canonization, but we should be doing that anyways)

Hm. Of course, we currently don't accept those in DAG-CBOR (per spec) then accept them in the implementations... It would be convenient to just accept them and be done with it.

I just discovered that f64 got total_ord in Rust 1.62. What if we just use that? As per IPLD Data Model spec, there shouldn't be any non-number values, but if it happens, we just deal with as total_ord is defined.


Not able to use it in non-WSDL mode without hack

I had to bypass the SoapService constructor and create the client myself to avoid the thrown exception.
Instead, I just have $this->createClient(); to initialise the client.
But following this 'hack', I could use the service OK.
It would be good if laravel-soap supported non-WSDL mode by default.

I've fixed this in version 0.3.*.


Show dialog "settings will take effect upon next sign in" if Tauri client is authenticated when changing settings

Can be a source of user confusion we saw with Apple/Android as well

Customers are all on the prod endpoints, so this is mostly for the log filter, right?
The IPC service can't reload that at runtime right now but there is a way in tracing, I'll see if it works


[FEAT] add device control

Add "allow-device-control" in user.json (False)
Toggle switch in webUI
On 1st toggle user needs to agree from commandline running terminal window
Then will verify,
Verify process:

Browser request for token
Server generate 2 token:
i. One to use as url query so that browser can go to (probably use uuid.uuid4() library to generate a random token)
ii. Other in the get request in above link (similarly)
Server send 1st token and generate a site with it and put the 2nd there
Browser request <IP_ADDRESS>:port/?verifyDevice=token
Make the 4 request url Allow all origin header response in server
If Server gets request (that means same device, since <IP_ADDRESS> can only be used from same device), sends 2nd token, browser gets that
Browser sends the second token to server (same as step 4). Server approves both in browser and server and saves (that token) in user.json  and localstorage. Since its in localstorage, unless malicious intent, it won't be shared
Also the request message like "raise volume" will be revarified (using step 4) with the token.

It won't hurt to make some extra requests rather than letting someone control your pc

This is also done in every login or verify user

@RaSan147,
Hope you are doing well.
Can I work on this issue, if so, assign it to me and give contribution guidelines as there are no any contribution guidelines?

Just make sure to add soms good comments and file a PR
Modify the app_server.py to add accepting more url and query strings. Use pyrobox.py to check server documentation (readable enough)
Try to make the verification process in a different file if possible.
Thanks for your eagerness, don't worry if anything doesn't feel right to me, ill modify it.

@UzairURK Well what it needs most (and why i am postponing this) is because it needs the vast possibilities of what can be and should be controlled with chat or voice.
The server - Web-UI update will take me 2min to complete.
So I'd be glad if you simply make an text command input and do something based tools in a file (cli program will be enough and 1 input per command should be enough, I'll try to come up with similar inputs)
I'll merge it with the project ASAP

Thanks,
I'll keep updating you.

some examples:

increase/decrease volume (will change master volume of device)
pause music will pause media (follow media key, like the keys on laptop or new keyboards with Fn)
take Screenshot
paste (will paste clip board)
etc

Thanks, I'll keep updating you.

Thanksss, also added CONTRIBUTION section in the readme

some examples:

increase/decrease volume (will change master volume of device)
pause music will pause media (follow media key, like the keys on laptop or new keyboards with Fn)
take Screenshot
paste (will paste clip board)
etc

Alright

@RaSan147
have you used any Machine Learning Models? and I have some new ideas to be implemented in this?

@RaSan147 have you used any Machine Learning Models? and I have some new ideas to be implemented in this?

Not yet, thats on the end of the bucket list, because creating more and more personalized (not robotic) responses and its taking time (and my university admission running)
Once i have enough enough IO to charecterize the AI, I'll move on.
Please do share your idea. I'm all open😺

Also the other issue on having ML is intent management and data collecting.
Training ML to reply on "open google" -> "ok" and actually open "google" in the middle by storing the data, thats where I'm struggling.
Training IO is easy but the middle work is something I don't know😥

great! my study is still running! The idea was to respond by Video with somehow real gestures if a person talks to her, AI should respond like Asuna is talking, the tone of AI should vary based on the location of the user and there must be a caption option if necessary for someone. hope you get it. Otherwise, we will talk on video chat in a free slot.

great! my study is still running! The idea was to respond by Video with somehow real gestures if a person talks to her, AI should respond like Asuna is talking, the tone of AI should vary based on the location of the user and there must be a caption option if necessary for someone. hope you get it. Otherwise, we will talk on video chat in a free slot.

I'm also struggling with voice output, as for you know voice training and output requires intense hardware and i tried on my gtx 1660, a simple sentence took 1min, so without MS speech sdk online, it may not be possible to make well customized voice. Since i don't earn and its open source, planning to use any free alternative
Also gestures will be added once i implement voice. Currently I'm more focused on server scaling and IO data generation 😵

@RaSan147
I have run RUN_ME.py server. should I run other python files to get startd?

alright

@RaSan147 I have run RUN_ME.py server. should I run other python files to get startd?

Did you install all the requirements 1st?
If yes then
Runme should give you your IP and server port
ie http://<IP_ADDRESS>:45454
Copy and open it in browser. Viola you have Asuna in your PC

As for the caption, once i add voice, I'll move chat in front screen with transparency, so above we'll see Asuna, botton we can type and see msg

And sorry in advanced, it uses regular expressions to catch input so it only knows around 30-40 types inputs
If you use my demo server, i can check the unknown inputs and add them to handler

lastly, can you just tell me which files I have to work on for this issue as it saves my time?

@UzairURK just make a separate file with input and action + print output. I'll merge it then

Alright
On Fri, 24 Mar 2023, 03:38 RaSan, @.***> wrote:

@UzairURK https://github.com/UzairURK just make a separate file with
input and action + print output. I'll merge it then
—
Reply to this email directly, view it on GitHub
https://github.com/RaSan147/VoiceAI-Asuna/issues/6#issuecomment-1482004934,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATYQJ6MWA4MJNXGEKEYZIF3W5TGGFANCNFSM6AAAAAAUQUK3UQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

nice

@UzairURK any update bro

@RaSan147
bro I have created two files and in which I create the two token but it fails the verification due to device Id, can you tell me that device Id should be random token or fetch the actual device id from device?

@UzairURK you don't need to make the verification tool
Just make a file that will simply take CLI command and do that work. Then I can easily integrate it.
Take your time, Ramadan Mubarak 😇
Maybe soon I'll make a branch with the verification activated so that you can do your part easily.

@RaSan147
just to implement the feature which takes CLI command and does the work as we do by built-in commands on the window or just give me an example of a CMD command and do what kind of work?

some examples:

increase/decrease volume (will change master volume of device)
pause music will pause media (follow media key, like the keys on laptop or new keyboards with Fn)
take Screenshot
paste (will paste clip board)
etc

ie:
import os

from tools import change_volume
# can be found in https://github.com/AndreMiras/pycaw/blob/develop/examples/audio_endpoint_volume_example.py

cmd = input()
if cmd == "raise volume":
    change_volume(20) # raise by 20%
elif cmd == "lower volume":
    change_volume(-20)
elif cmd== "open download folder":
    # check os and use file manager accordingly
    os.system("explorer.exe %USERPROFILE%/Downloads")
elif .....

Something like this, as many as possible

@RaSan147 bro I have created two files and in which I create the two token but it fails the verification due to device Id, can you tell me that device Id should be random token or fetch the actual device id from device?

I'd be more than happy to explain everything to you on the process, but the issue is the server "pyrobox" i made and using now has very little documentation, so if you try to update the server code yourself, it'd cause more work (not questioning your skill or something) afterwards.
And if you planned to write in Django, I'll need to rewrite it again, so not much help.
I'm sorry. If you need I'll write the code for you (extremely busy now)😥

some examples:

increase/decrease volume (will change master volume of device)
pause music will pause media (follow media key, like the keys on laptop or new keyboards with Fn)
take Screenshot
paste (will paste clip board)
etc

ie:
import os

from tools import change_volume
# can be found in https://github.com/AndreMiras/pycaw/blob/develop/examples/audio_endpoint_volume_example.py

cmd = input()
if cmd == "raise volume":
    change_volume(20) # raise by 20%
elif cmd == "lower volume":
    change_volume(-20)
elif cmd== "open download folder":
    # check os and use file manager accordingly
    os.system("explorer.exe %USERPROFILE%/Downloads")
elif .....

Something like this, as many as possible

I have created this type of code, in which you command something It will perform an action, I was just conforming that is this complete task. You have told in beginning to create two tokens can you just explain it? I will write code on my own.
So, let me confrom two thing I have to do 1) CLI command and 2) two tokens generate and verify the device. Is it right?
Thanks

Yeah, if you want to colab, the complete task is the 1) CLI command
I already have made modules and functions for 2) 2 token generate and verify. So there's nothing to do in that part.

Yeah, if you want to colab, the complete task is the 1) CLI command
I already have made modules and functions for 2) 2 token generate and verify. So there's nothing to do in that part.

Alright.


Fix Google dialect match regexp

Update tests to verify is_match for docstring dialects

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (f034f6a) 96.39% compared to head (cf9e53f) 96.39%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #471   +/-   ##
=======================================
  Coverage   96.39%   96.39%
=======================================
  Files          69       69
  Lines        6487     6487
=======================================
  Hits         6253     6253
  Misses        234      234

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Thank You @tyzhnenko


Trying to scan barcodes with android phone(chrome)

I'm trying to run any of the examples (i.e. simple_jquery.html) on a mobile device: Sony xperia m4 aqua (Android 5.0 - Chrome 47.0.2526.83)
QRCode scannig is working fine, but I have problem with barcodes.
It gives the following js error:
js/DecoderWorker.js:333 Uncaught TypeError: Cannot read property '0' of undefined
js/DecoderWorker.js:470 Uncaught TypeError: Cannot read property '0' of undefined
On desktop it's working fine and no errors appear in console.

Are you over SSL ?

Yes, I am. But I think that's not the problem actually.
I got it work with v2.0.5

I'm sorry,
but I'm not able to reproduce the bug
I tried on some android machine. ZTE Blade v5, Teclast Tablet, etc.

Regards


Repo sync for protected CLA branch

The pull request is created from main637950585800478009sync_temp to main to fix git push error for protected CLA branch

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


Subshift Broken

Running subshift shift operation works via npm start but not in the released binary

If the user installs subshift globally ie.
npm i -g subshift

then it works - however, this partially defeats the purpose of having a binary, since, ideally, subshift would be included in the binary.
Need to find a better solution

A better solution would be to use a standalone cli tool rather than subshift, which relies upon an existing install of Node.js underneath. For my personal use case however, this is sufficient. Closing as I will not fix this.


Celluloid::TimeoutError: receive timeout exceeded

Running a job inside a loop. The loop fires async jobs. Some jobs returns:
Celluloid::TimeoutError: receive timeout exceeded
/Users/blanchma/.rvm/gems/jruby-1.7.13@applicationservice/gems/celluloid-0.16.0/lib/celluloid/mailbox.rb:80:in receive' /Users/blanchma/.rvm/gems/jruby-1.7.13@applicationservice/gems/celluloid-0.16.0/lib/celluloid/calls.rb:97:in wait'
org/jruby/RubyKernel.java:1501:in loop' /Users/blanchma/.rvm/gems/jruby-1.7.13@applicationservice/gems/celluloid-0.16.0/lib/celluloid/calls.rb:96:in wait'
/Users/blanchma/.rvm/gems/jruby-1.7.13@applicationservice/gems/celluloid-0.16.0/lib/celluloid.rb:116:in suspend' /Users/blanchma/.rvm/gems/jruby-1.7.13@applicationservice/gems/celluloid-0.16.0/lib/celluloid/calls.rb:88:in response'
/Users/blanchma/.rvm/gems/jruby-1.7.13@applicationservice/gems/celluloid-0.16.0/lib/celluloid/calls.rb:92:in value' /Users/blanchma/.rvm/gems/jruby-1.7.13@applicationservice/gems/celluloid-0.16.0/lib/celluloid/proxies/sync_proxy.rb:33:in method_missing'
/Users/blanchma/.rvm/gems/jruby-1.7.13@applicationservice/gems/celluloid-0.16.0/lib/celluloid/proxies/cell_proxy.rb:17:in _send_' /Users/blanchma/.rvm/gems/jruby-1.7.13@applicationservice/gems/celluloid-0.16.0/lib/celluloid.rb:169:in new'
Environment:
jruby-1.7.13
celluloid (0.16.0)

@blanchma can you please post the repro you're talking about, so I can help here?

@digitalextremist Unfortunately I don't work on this project anymore, so you can discard my comment.

@basgys-cw can you unsubscribe from the issue so I can just leave the issue intact, and people can see that you're no longer participating by virtue of your comment to that effect?

I am now having this issue in jruby 1.7.19. Will address in 0.17.0

Duplicate of #464


Unexpected token import

Hi,
When I try to use your library with SystemJS by importing it like so:
import MaterialDatePicker from 'material-datetime-picker';

or
import MaterialDateTimePicker from 'material-datetime-picker';

I got this error:
Error: (SystemJS) SyntaxError: Unexpected token import

Do you have any idea about where this could come from? Because after looking at the code, I think the README is pointing to the wrong component.
Thanks for your time,
Raphaël
Environment
Browser

Chrome 51 (Windows, Mac)

Library

v1.0.1 (via jspm => npm)

You'll probably need the systemjs babel plugin. The library is written in ES6, so if you don't have a transpiler then things wont work correctly!

My code is also written in ES6 and I already use Babel as a transpiler. The core of my project is using mithriljs. I will try to reproduce this error in a fresh environment with JsFiddle for you to see.
Thank you for your time.

Yes, that will be useful, thanks! i don't know why your transpiler is moaning about import statements. Are you using them elsewhere in your project? I think you need the [es2015] babel preset for imports, perhaps that is what is missing.
I will add it to the project as well.

I am also experiencing this issue and I have the [es2015] babel preset. Is there any solutions to this?

@simonq-nvc Is the rest of your project ES6? Do other import statements complain? What version of babel are you using?

same here. other imports do work though.
babel packages:
    "babel-preset-stage-2": "^6.11.0",
    "babel-plugin-transform-object-rest-spread": "^6.8.0",
    "babel-core": "^6.7.4",
    "babel-loader": "^6.2.4",
    "babel-preset-es2015": "^6.6.0",
    "babel-preset-react": "^6.5.0",

@orwldn can you share your .babelrc? I think by default babel will not compile anything in node_modules.

it only includes transform-object-rest-spread

+1

I have tried to fix this but it seems that browserify/babel makes it really confusing the build standalone modules. I will continue trying today.

@Jivings That's why I don't think use import in a npm module is a good idea

@brickyang Agreed. There's should be a compiled dist version available.

Inside the create-react-app environment, it falls into the same error. Any recommendations?

+1

I suppose if there were used require ensure instead of imports at least webpack could build it.
Is there is currently any way to launch it?

The version on master (since merging #99) can be imported by Webpack (including create-react-app) and SystemJS/JSPM.
I'll cut a 2.0.0 release in the next few days. Thanks for your patience!

Can you try 2.0.0-pre?
The build issues should be resolved there.

Fixed in 2.0.0.


Double sass in path in map file

Original post
Hi Guys,
Could you help me a bit to understand why compiler generating wrong source path in map file?
I have nested folder with scss and different one for css. CSS files are generated fine and in proper place, but my map file contains wrong path inside.
Part of wrong map file style.css.map:
{"version":3,"sources":["../sass/sass/hold/style.scss","../sass/hold/style.scss","../sass/sass/bootstrap/mixins/_vendor-prefixes.scss","../sass/sass/bootstrap/_variables.scss"],"names":[],"mappings":.........}

I want to avoid this double /sass/sass/ in first path. Could you help me a bit?
Here is my structure:

My settings:
    "liveSassCompile.settings.formats": [
        // This is the default.
        {
            "format": "expanded",
            "extensionName": ".css",

            // null for all three -> denotes the same path as the SASS file
            "savePath": "themes/hold/css/",
            "savePathSegmentKeys": null,
            "savePathReplaceSegmentsWith": null
        }

Tasks for 5.2 release

[x] Find cause
[x] Apply fix

How very strange! I ran my own test and was able to reproduce it. It seems to be an issue with autoprefixer so I have opened an issue there - postcss/autoprefixer#1437

Thanks for collaboration on it! I saw respond from Andrey (ai):

The problem is that if you take from option and add sassMap.source to it, you will not get path to the Sass source.
It leads to sources duplication.
You need to fix paths, but I can’t help you where exactly is a mistake.

Do you find time to look into it? ;)

I will fix it so it won't include the duplicate/incorrect SASS path. However, it will always include a reference to the CSS file itself (in your case style.css) which, according to this comment, they will not be fixing/addressing.

Okay, so what needs to be done manually?

However, it will always include a reference to the CSS file itself (in your case style.css)

What that mean for me?
Thanks for your time and help!

Okay, so what needs to be done manually?

Nothing, I'll push a new release soon. I have already applied the fix to the ahead branch ready to publish. If you wanted to remove the unnecessary binding then you'd have to build a script that removes them once a .css.map file is saved

What that mean for me?

It just means that without autoprefixer a map file will be interpreted as below (the column on the right is where the browser would take you if you wanted to see the source of the item on the left)

And with autoprefixer applied it will instead be interpreted like this

What this basically means is that the new map file will be unnecessarily bigger to reflect the unmapped lines that the SASS' map doesn't include (as the browser will never interpret/care about them).
I hope this all makes sence

Yes, I am totally new to Sass things and doing them casually, so I spot this issue totally not expected ;)
Thanks for your deep analysis and fix! I appreciate it!

Hi, i just wanted to ask when the update is pushed?
I really would like this issue to be fixed 😬
Thanks for the great work.

Hey @DvdRom87,
Sorry for the delay. It has been a bit of a crazy couple of months - moving, new baby, work, uni studies - which has pushed these things to the back burner.
I was also waiting on #151 to confirm that the fix worked, this way I can push it in this release. But the change should fix the top-level problem and more work may be needed to address their other concerns, so I'll push an official release tonight.


Support bazelisk

Closes https://github.com/bazelbuild/bazel-watcher/issues/339
TODO:

[ ] Add tests

Thanks for the quick review @achew22 , those are great points. Updated.


Fix Pythagorean method

There is currently a method called pythagorean that lives in /lib/pythagorean.js.
It is incomplete and needs to be fixed!
Acceptance Criteria:

Running yarn test pythagorean results in tests passing.

Please include the skeleton of a new method + an accompanying test for someone else to work on at the time of creating a pull request!

I will start working on this.

Awesome- Thanks @h93xV2 !


docs: Provide information about Request & Response-Logging

Hello,
please please please provide some prominent (I would suggest in the main README/Homepage) Information about how to enable/disable automatic wire Request/Response Logging.
Any developer needs this sooner or later. And I was not able to find any information and I am still searching, how this can be done (declaratively via config, not via code)?
Thanks very much.

Will do - you can use https://cabinjs.com btw - there is an example there.
https://cabinjs.com/#/?id=automatic-request-logging


Unable to compile in MacOS

I had to make the following changes to get msp_lib to link properly:
https://github.com/MrPrezident/MSPhysics/commit/0e6c4c458ae9765fc6e3561b15ac72a0be89d418
However, even after those changes, the extension is loading, but it is not working. I'm seeing these errors in the Ruby Console when trying to run a simulation:
Entity at index 0 was not added to simulation:
TypeError:
Entity #<Sketchup::ComponentInstance:0x007f96fd813968> doesn't have any valid sub-collisions, making it an invalid compound collision!
/Users/nathan/Library/Application Support/SketchUp 2018/SketchUp/Plugins/MSPhysics/collision.rb:76:in `create'
/Users/nathan/Library/Application Support/SketchUp 2018/SketchUp/Plugins/MSPhysics/body.rb:155:in `initialize'
/Users/nathan/Library/Application Support/SketchUp 2018/SketchUp/Plugins/MSPhysics/simulation.rb:792:in `new'
/Users/nathan/Library/Application Support/SketchUp 2018/SketchUp/Plugins/MSPhysics/simulation.rb:792:in `add_group'
/Users/nathan/Library/Application Support/SketchUp 2018/SketchUp/Plugins/MSPhysics/simulation.rb:3207:in `block in activate'
/Users/nathan/Library/Application Support/SketchUp 2018/SketchUp/Plugins/MSPhysics/simulation.rb:3193:in `each'
/Users/nathan/Library/Application Support/SketchUp 2018/SketchUp/Plugins/MSPhysics/simulation.rb:3193:in `activate'
/Users/nathan/Library/Application Support/SketchUp 2018/SketchUp/Plugins/MSPhysics/simulation.rb:30:in `select_tool'
/Users/nathan/Library/Application Support/SketchUp 2018/SketchUp/Plugins/MSPhysics/simulation.rb:30:in `start'
/Users/nathan/Library/Application Support/SketchUp 2018/SketchUp/Plugins/MSPhysics/main.rb:809:in `block in <top (required)>'
SketchUp:1:in `call'
Error: #<NameError: uninitialized constant AMS::ClothWorks>
/Users/nathan/Library/Application Support/SketchUp 2018/SketchUp/Plugins/MSPhysics/control_panel.rb:129:in `open'
/Users/nathan/Library/Application Support/SketchUp 2018/SketchUp/Plugins/MSPhysics/simulation.rb:3318:in `activate'
/Users/nathan/Library/Application Support/SketchUp 2018/SketchUp/Plugins/MSPhysics/simulation.rb:30:in `select_tool'
/Users/nathan/Library/Application Support/SketchUp 2018/SketchUp/Plugins/MSPhysics/simulation.rb:30:in `start'
/Users/nathan/Library/Application Support/SketchUp 2018/SketchUp/Plugins/MSPhysics/main.rb:809:in `block in <top (required)>'
SketchUp:1:in `call'

I have fixed the Mac OS X solution but I see that resetting simulation leads to a crash. I think the issue is related to multi-threading, which is on TODO, but I'm not sure since I explicitly forced newton to use one thread at the moment.
Also, the first error you see, is normal and is handled by MSPhysics. It occurs whenever a group or a component added to simulation is flat. All other errors are typos and are I am fixing them.


🚀 Feature:

🔖 Feature description
I want to pass Azure Ad login user information to backstage template.
🎤 Context
I want to execute a custom action using an Ad user.
✌ Possible Implementation
No response
👀 Have you spent some time to check if this feature request has been raised before?

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

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
None

Hey :wave: So I think I understand the issue here, might need some more context.
But if you have Azure sign-in configured in Backstage you can use the RepoUrlPicker to get a token for the user.
https://backstage.io/docs/features/software-templates/writing-templates/#using-the-users-oauth-token


Works around encoding bug in zipkin-ruby by leniently parsing

In zipkin-ruby (zipkin-tracer up to 0.32.3), the "sa" annotation was
incorrectly encoded as "1" not true. This likely was due to the fact
that thrift encodes boolean as 1. This caused the remote endpoint to be
mistaken for a local endpoint as the parser thought "sa" was a normal
string tag with the local endpoint attached. The impact was
mis-rendering and also service graph aggregation would likely be
incorrect in some cases as well.
Definitely, zipkin-ruby should update as even if we leniently parse,
clones will not likely be as tolerant and this could hurt things.
However, it is best to be lenient as it is cheap and low impact. There
is unlikely any valid tag named "sa" "ca" or "sa" used for a purpose
besides the remote endpoint.
Fixes #2414

I tried this and it worked. Note that since conversion happens at ingress, it will only help with new spans.
Normal:

Lens:

Fixed by https://github.com/openzipkin/zipkin-ruby/pull/139


fix(label): RTL fix for label margin

Short description of what this resolves:
RTL fix (scss changes) for label margin for wp, ios and android platform
Changes proposed in this pull request:

Add label margin prop for RTL (md, ios)
Fix label margin for RTL (md, ios, wp)

Ionic Version: 2.x / 3.x
Fixes: #11211

Looks good to me. @manucorporat ?

Is there (or will there be) a PR for the input as well? This PR messes up the alignment for floating labels since the input's margin is incorrect. Screenshot before this PR:

After this PR:

@brandyscarney There is something weird here.
Clearly, the margin left is ($item-md-padding-right / 2), but you screenshot shows 0.
Also, no, there is no current input rtl PR, all in good time :)

@AmitMY This rule is taking precedence so it is getting margin-left: 0

@brandyscarney I'm doing it now, looks like one pr should be for input and another one for select and I think search-bar must change too ...

@sijav Take a look at this.
If it will get merged, then we can do the same thing for margin, border, and position, and to fix all of the components together, once. We are still finalising the syntax and changes, but I think this (or some version of it) will be merged soon.
I'm letting you know, just so you don't do extra work that will be rejected if this goes in. I will keep you updated

@AmitMY Great job! Thanks for letting me know, but we still need some support from typescript (searchbar for ios or sliding item component and so on)

@brandyscarney This can be closed

Closing this PR in favor of #11342


This utility no longer works... It is constantly saturated with requests

curl wttr.in/boston
Sorry, we are running out of queries to the weather service at the moment.
Here is the weather report for the default city (just to show you what it looks like).
We will get new queries as soon as possible.
You can follow https://twitter.com/igor_chubin for the updates.

https://github.com/chubin/wttr.in/issues/522

Can reproduce, https://vnc-gnome.9pfs.repl.co/vnc_lite.html

Fixed


Support &. with Ruby 2.3

I encountered syntax error when executed :w with syntastic but my syntax is all is well because I executed ruby -c target.rb and it returned Syntax OK
The cause of the error is obj&.foo.
&. is feature of ruby2.3 and it was released. https://bugs.ruby-lang.org/issues/11537
My environment is
vim: VIM - Vi IMproved 7.4
syntastic: 3.7.0
ruby: ruby 2.3.0p0 (2015-12-25 revision 53290) [x86_64-darwin14]

Please open your test file, set g:syntastic_debug to 3, run the checker, then run :mes, and look for makeprg in the output.  Run that command in a terminal, then run it from system() inside Vim.  See also the output of :echo syntastic#util#system('echo "$PATH"').

I see. :smile: Thanks.


How to get day selected (long) ?

calendarDay.getDate().toEpochDay()

that method is always return value < 0 ??

Is mothod calendarDay.getDate().getTime() has been removed ?

Yeah, and I've just jumped down to version 1.6.
I hope that you will add method getTime in the future. Thank you

calendarDay.getDate() is now returning a LocalDate as of https://github.com/prolificinteractive/material-calendarview/releases/tag/2.0.0.
Please refer to the class documentation instead https://docs.oracle.com/javase/8/docs/api/java/time/LocalDate.html, it should have what you need.
There is no plan on adding new API to the CalendarDay atm.


Google and Bing scraper's "end early" for big numbers of images

I watch in Nightmare and it seems they don't even finish scrolling (when run for 1000 items) and just terminate.
I want to get 1000 pictures with them!!!
Yahoo works fine etc.

Revamped everything. If this is still an issue please check. I'm closing it.


There are no syntax prompts in webstorm anymore

Version
@nuxtjs/tailwindcss:
nuxt:
Reproduction Link

Steps to reproduce
What is Expected?
What is actually happening?

I don't often use nuxt, but I have installed the "@ nuxtjs/tailwindcss" module, and the tailwind style can take effect. However, my webstorm doesn't have any code prompts. Is it because my project doesn't have tailwind.config.ts?

Sure enough, after I added the tailwind configuration file, the code prompt appeared. Which of the tailwind configuration items, tailwindcss.comnfig.ts and nuxt.config.ts, has higher priority, or will they be merged?

tailwind.config.ts has higher priority, but it would be ideal to point it to .nuxt/tailwind.config.cjs.


Mail preview for a specific order document contains wrong data

Description:
The mail preview for a specific order document (like an invoice for some specific order does not always contain data that does not belong to an order.
This is because the data used to fill in the mail template differs when previewing the mail and actually sending the mail:

Preview: The data is retrieved from the MailTemplateType which contains productive, uncensored data from some last mail that was actually sent. See [https://issues.shopware.com/issues/NEXT-15475|https://issues.shopware.com/issues/NEXT-15475|smart-link]  and [https://github.com/shopware/shopware/blob/4c862c05494a612462ba6f0198db49ffcc98fc59/src/Administration/Resources/app/administration/src/module/sw-order/component/sw-order-send-document-modal/index.js#L131-L133|https://github.com/shopware/shopware/blob/4c862c05494a612462ba6f0198db49ffcc98fc59/src/Administration/Resources/app/administration/src/module/sw-order/component/sw-order-send-document-modal/index.js#L131-L133|smart-link]
Actually sending: The data is freshly constructed with the given order and sales channel in the order detail page. See [https://github.com/shopware/shopware/blob/4c862c05494a612462ba6f0198db49ffcc98fc59/src/Administration/Resources/app/administration/src/module/sw-order/component/sw-order-send-document-modal/index.js#L157-L160|https://github.com/shopware/shopware/blob/4c862c05494a612462ba6f0198db49ffcc98fc59/src/Administration/Resources/app/administration/src/module/sw-order/component/sw-order-send-document-modal/index.js#L157-L160|smart-link]

**Environment: **
One of:

Local Dev Environment with SW<IP_ADDRESS>
Productive Shops with SW6.5.x

Steps to reproduce:
Send some emails on your system (e.g. via Flows)
Open the detail page of some order of which documents where not yet sent (only to ensure that you will get the wrong data in the preview)
Generate a document (e.g. an invoice) if none exist yet
Start sending the document via the "Send document" button in the document list
Look at the difference between the preview in the modal and the actual email that is sent
**Expected result: **
The preview in the modal of the order detail page uses the same data as the actually sent email.
**Actual result: **

[public] Shopware automated response: This issue is linked to the internal issue: NEXT-39696
[created from NEXT-39696, comment 513961]


Handle LookupError when detecting encoding

For reasons that are a slight mystery to me, chardet can return
encodings that Python doesn't actually support.
On the assumption that if we can't decode it we're probably not
interested in considering it as text (especially given that chardet
will sometimes detect an encoding that you can't actually decode as
when its confidence is < 1), this handles this commit by
considering it to not be decodable.

@DRMacIver thanks for catching this, much appreciated.


ARROW-16267: [Java] Adding support to compile Java code with JDK 18

Please @raulcd / @assignUser your support with this review, thank you in advance.

Does GitHub offer a way to just specify "latest" so that we can automatically get 19, 20, ..., etc when they release?

@lidavidm This is a docker issue, as we use some of these for the jobs and they don't have a 3.8.5-openjdk-latest tag. So there is no easy way to keep this up to date automatically. openjdk 20 is slated for release in 9/2022 so maybe we just open a jira with fix-version 10.0.0?


Abilty to correctly inform cargo of file dependnencies in build.rs

Description
See changelog.
Fixes
Issue Number: None (just went for it)
Checklist

[x] Update CHANGELOG.md

I believe test failure here is #141 and resolve by changes in #140 not going to address on this MR.

Going ahead and merging, will plan on fixing tests with continued work in #140


"Run" button isn't debounced

Steps to replicate:

Create a long running script with lots of nested loops
Click "run"
See no indication it's running, so click "run" multiple more times

Expected behavior:

It generates either the most recent "run", cancelling previous artwork in progress OR
It ignores new button presses until the original art finishes generating

Actual behavior:

It runs over and over without a way to cancel, eventually freezing the tab

Ah, to clarify this issue, I'm hitting the "shift+enter Run" button that generates art, not runs the machine. I have some art with a lot of recursion that ends up freezing the tab.

I see. It's not really possible to halt the process without the difficulties that come from running it in a web worker. You can reopen this but I'm not sure we'll address it soon.

If this is a "closed as won't fix" I think that's fine! Just getting this documented as a known "thing" is probably fine for now.

Are we closing this as fixed or as won't fix?

This is addressed by moving evaluation to a web worker which will be merged to main shortly.

Addressed with #341


Added support for the beta version of Spotify Desktop to the SpotifyLocalAPIClass

Basically, enables a developer to target the official release or the beta release via an optional parameter in SpotifyLocalAPIClass's constructor. Based on that setting, the library will target the spotify.exe/spotifywebhelper.exe official executables or the spotifybeta.exe/spotifybetawebhelper.exe beta executables. You can install the beta from here. Note that it installs side by side with the official release.

Nice! :smiley:
I just have one question about the constructor
Greetings
Johnny

Yeah, just answered your question. There was a reason I started off with an overloaded ctor but that ended up not being necessary.
By the way, so far, from my testing, spotifybetawebhelper.exe is far more reliable. :)
From: Jonas Dellinger<EMAIL_ADDRESS>Sent: Sunday, February 22, 2015 11:49 AM
To: JohnnyCrazy/SpotifyAPI-NET
Cc: Gus Perez
Subject: Re: [SpotifyAPI-NET] Added support for the beta version of Spotify Desktop to the SpotifyLocalAPIClass (#14)
Nice!   https://assets-cdn.github.com/images/icons/emoji/unicode/1f603.png
I just have one question about the constructor
Greetings
Johnny
—
Reply to this email directly or view it on GitHub https://github.com/JohnnyCrazy/SpotifyAPI-NET/pull/14#issuecomment-75453744 .  https://github.com/notifications/beacon/ABwUJHHyolUg5QcP5P1womLB7IIufDLZks5nuioagaJpZM4Dj-N9.gif

Nice, That would be really nice.
I will test the new Beta this week and maybe we can make this API more stable. :+1:

Okay, should be good to go now. :)

Done :+1:

Awesome. Thanks. So nice to find the beta's so far infinitely more reliable. :)


Crash on adding new moody after delete

Here is scenario: After adding new new mood, I try to delete it and then add new one. It crash at:
public func saveOrRollback() -> Bool {
    do {
        try save()
        return true
    } catch {
        rollback()
        return false
    }
}

I do some debugging by lldb with po $arg1 and it say: Constraint violation

@oldtrafford91 Good catch – this happens because of a uniqueness constraint conflict on a Country or Continent entity. Those entities have a uniqueness constraint defined (using the numericISO3166Code attribute. However, this uniqueness constraint should also include the markedForDeletionDate attribute to allow deleted and non-deleted instances of a country/continent to co-exist. I'll push an update to the code, let's see if that fixes it.

@oldtrafford91 Good catch – this happens because of a uniqueness constraint conflict on a Country or Continent entity. Those entities have a uniqueness constraint defined (using the numericISO3166Code attribute. However, this uniqueness constraint should also include the markedForDeletionDate attribute to allow deleted and non-deleted instances of a country/continent to co-exist. I'll push an update to the code, let's see if that fixes it.

@oldtrafford91 unfortunately this doesn't behave the way I expected it to... will think a bit more about what's a good solution.

@oldtrafford91 unfortunately this doesn't behave the way I expected it to... will think a bit more about what's a good solution.


I never see the 2-factor verification page

Hi
I'm not sure if I'm missing something, but no matter what I do, I never see the 2-factor verification page.  I just get logged in straight to the dashboard.

I have setup the config file as per the example.
verifyBackEnd is true
I have completed the setup process with Google Authenticator on the settings page.
I see the green "Your account is secured" text on the settings page.

If I log out and log back in, I go straight to the dashboard.  No 2-factor verification.
If I delete the user's record from the twofactorauthentication_user database table and try logging in again, I still go straight to the dashboard.  No 2-factor authentication.
The user's record in the twofactorauthentication_user database table does not get recreated until I return to the plugin's settings page.
I have tried with both an admin account and a non-admin account.
I am testing locally at the moment, which I assume is OK?  I can't think of anything else to try.  Am I missing something?
Thanks!
Craft version: <IP_ADDRESS> (also tested with 3.6.6)
Plugin version: 2.8.1

I've now also tried disabling dev mode, which didn't make any difference.

@petehjfd thanks! The verification is stored in the session. It looks like session is kept alive even after logout. I'll release a fix asap.

Thanks @roelvanhintum, that would be great!

@petehjfd before releasing, does the latest commit fix this for you? In composer this would be dev-craft-3.1 i think.
I now store the login timestamp and compare this to the lastest login timestamp.

Yes, that seems to have done the trick!

Glad to hear! Thanks for testing! Released in 2.9.0