Empty workflow list when navigating back to Workflows view

What happened:
When coming back to the workflow list from within a given workflow's view, the list is empty, an no error is to be found in logs and/or requests.
What you expected to happen:
The list should display existing workflows
How to reproduce it:
Navigate to a specific workflow's view, then go back to the workflow list view using the breadcrumb, for example.
If it is, as I think, related to #90, you should first follow the latter's instructions on how to repro.
Anything else we need to know?:
This is probably linked to #90, as I could only reproduce it when it occurs

I think this was a side effect of the exception raised in #90 . If not, reopen this issue with details.


Documentation on ReadTheDocs is missing all the module and method reference information

The table service query_entities method is defined here:
https://github.com/Azure/azure-storage-python/blob/09e9f186740407672777d6cb6646c33a2273e1a8/azure/storage/table/tableservice.py#L385
However, it does not show up in the ReadTheDocs documentation, either when doing a search:
http://azure-storage.readthedocs.org/en/latest/search.html?q=query_entities&check_keywords=yes&area=default
Or when looking at the page about the tables:
http://azure-storage.readthedocs.org/en/latest/table.html
Or when looking at the module page in the index:
http://azure-storage.readthedocs.org/en/latest/ref/azure.storage.table.tableservice.html
In fact, it seems like all the module pages, which are presumably the official API reference for what each method does and what each argument means, are blank. Presumably this isn't intentional.

Hmmm, taking a look now. Looks like a publishing process issue. We're in the process of moving our docs from a custom format to the ReStructured text supported best by Sphinx, so I'm guessing something got messed up there.

ReadTheDocs is still having build issues for us, so we've temporarily (or however long it takes to resolve these issues) moved to a GitHub.io page.

@emgerner-msft - Can you elaborate on the build issues you are seeing? It would be nice to get the RTD in a similar state to azure-sdk-for-python


Unable to write or read using webclient or postman

I am trying to read from w3c using webclient and postman and I can authorize but when I try to read, write,etc a value:
Example with postman
GET http://localhost:8090/vss/api/v1/
{
"action": "get",
"requestId": "1234",
"path": "Vehicle.OBD.EngineSpeed"
}
It says:

ERROR: HTML status = 404
{
"action": "get",
"error": {
"message": "I can not find Requested resource do not exist in my db",
"number": 404,
"reason": "Path not found"
},
"requestId": "1079400840",
"timestamp":<PHONE_NUMBER>
}

The vss_rel_2.0.json is correct and testclient works.
This problem was an issue closed here https://github.com/eclipse/kuksa.invehicle/pull/113
but it was not resolve.

Hi, I am not "fluent" in the REST interface, and need some time to replciate, however, just in case the 404 is misleading: Did you authorize first? The REST API also needs a valid  JWT token before it works.
testclient uses websocket, so it could be that it works, while you did something else with POSTMAN.  Did you also try with the REST test thingy https://github.com/eclipse/kuksa.val/tree/master/test/web-client  ? (and if the testclient works you might look what it does using Wireshark, and compare it what you are doing with postman)
maybe @BojanKV is still alive and remembers something

Hi, yes, I have not problems with testclient using websocket, I can authorize, read and write with no problem.
Using  web-client and postman (I use postman to do different proofs configuring w3c-visserver with a different IP to access to it from external)
With POSTMAN and web-client I can only authorize.
Good idea, I am going to use wireshark to compare.

For future reference, GET  needs to be on e.g. signals/Vehicle.Speed  with the experimental REST support,


Node.js  Community Committee Meeting 2021-04-01

Time
UTC Thu 01-Apr-2021 16:00 (04:00 PM):

Timezone
Date/Time

US / Pacific
Thu 01-Apr-2021 09:00 (09:00 AM)

US / Mountain
Thu 01-Apr-2021 10:00 (10:00 AM)

US / Central
Thu 01-Apr-2021 11:00 (11:00 AM)

US / Eastern
Thu 01-Apr-2021 12:00 (12:00 PM)

EU / Western
Thu 01-Apr-2021 17:00 (05:00 PM)

EU / Central
Thu 01-Apr-2021 18:00 (06:00 PM)

EU / Eastern
Thu 01-Apr-2021 19:00 (07:00 PM)

Moscow
Thu 01-Apr-2021 19:00 (07:00 PM)

Chennai
Thu 01-Apr-2021 21:30 (09:30 PM)

Hangzhou
Fri 02-Apr-2021 00:00 (12:00 AM)

Tokyo
Fri 02-Apr-2021 01:00 (01:00 AM)

Sydney
Fri 02-Apr-2021 03:00 (03:00 AM)

Or in your local time:

https://www.timeanddate.com/worldclock/fixedtime.html?msg=Node.js+Foundation+Community Committee+Meeting+2021-04-01&iso=20210401T1600
or https://www.wolframalpha.com/input/?i=04PM+UTC%2C+Apr+01%2C+2021+in+local+time

Links

Minutes Google Doc: https://docs.google.com/document/d/1tVE7B-GUUAxYzAaeZq6S1_E5vCvZCtZrn5M0iJul7Yw/edit

Agenda
Extracted from cc-agenda labelled issues and pull requests from the nodejs org prior to the meeting.
nodejs/admin

Create new github-bot personal token for nodejs/node-v8 automation #587

nodejs/community-committee

When do we do the election of CPC Voting member from CommComm? #670

Invited

CommComm Members: @nodejs/community-committee

Observers/Guests
Feel free to follow along on the YouTube live stream, or attend meeting as a guest
by calling in to Zoom, using the links below. If you will be attending as a guest,
please comment on this issue to let us know you'll be joining.
Notes
The agenda comes from issues labelled with cc-agenda across all of the repositories in the nodejs org. Please label any additional issues that should be on the agenda before the meeting starts.
Joining the meeting

Link for participants: https://zoom.us/j/95198277339
For those who just want to watch: https://www.youtube.com/channel/UCQPYJluYC_sn_Qz_XE-YbTQ

Invitees
Please use the following emoji reactions in this post to indicate your
availability.

:+1: - Attending
:-1: - Not attending
:confused: - Not sure yet

I'm not going to be able to make the meeting this week.


As a content editor, I want to embed images in pages that don't span the full width of the page so I can create more varied or readable page layouts.

Describe the solution you'd like
Wagtail evidently has a native solution for applying special styles to images embedded in rich text: "image formats". As the docs say:

Images embedded in Rich Text fields can’t be controlled by the template developer as easily. There are no image objects to work with, so the {% image %} template tag can’t be used. Instead, editors can choose from one of a number of image “Formats” at the point of inserting images into their text.
Wagtail comes with three pre-defined image formats, but more can be defined in Python by the developer.

Image formats do two things:

Apply a configured image transformation, e.g. width-800
Apply a class so the image can be styled with css, e.g. left

The three predefined formats are Full width, Left-aligned, and Right-aligned, which sound very congruent with what we want.
Describe alternatives you've considered
If the Wagtail-native solution doesn't work, we can look at the PPA backend implementation, which creates custom block types for half-width images.
Additional context
This functionality was first implemented in PPA as "side-by-side images"; for the relevant styles see:
https://github.com/Princeton-CDH/ppa-django/blob/9324b103f56dcd59a4507f4479a2e0772df45404/srcmedia/scss/_mixins.scss#L170-L220

@cvansant I'm hoping to get to this issue and adapt the related work we did in PPA - you can see an example page that uses it in Gissoo's essay on the design.
Image display options on PPA are left (half width), right (half width) and full width (default, what we have now).
Content width is narrower on CDH web than on PPA, so I want to try making the left/right images stick out from the content a bit on desktop. I was also thinking that a extra wide (not sure what to call this) option might be nice (I already figured out the styles when I did the code block).
I think these will be nice to have generally, but would also be something we could use if we want to create some gallery pages somewhere on the site to better showcase CDH work.
Please let me know if you think that set of options sounds like enough (content width, left, right, extra wide) and please suggest a better name for that last one if you have one.


  It appears that this page is probably outdated based on the link below.
https://docs.microsoft.com/en-us/dynamics365-release-plan/2020wave2/finance-operations/dynamics365-finance/financials-embedded-powerbi-enhanced-renamed

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

ID: 84992dbc-779a-2e46-de21-556a6031c561
Version Independent ID: 04d7cf2c-f55e-8765-d3ed-44502e259bc2
Content: Financial Insights - Finance & Operations | Dynamics 365
Content Source: articles/fin-ops-core/dev-itpro/analytics/financial-insights.md
Service: dynamics-ax-platform
Product: ****
Technology: ****
GitHub Login: @kweekley
Microsoft Alias: kweekley

@apierre2000, thanks for the feedback.
@kweekley and @roschlom, please review.

Bug 555833 logged to track this issue.

I have edited the Financial Insights document and updated it to reflect the current Financial Analysis dashboard.
https://docs.microsoft.com/en-us/dynamics365/fin-ops-core/dev-itpro/analytics/financial-analysis


Do not split Swedish text on hyphen

The tokenizer for Swedish shouldn't split on hyphen. Many Swedish words, mostly nouns, contains hyphen, e.g. "it-bolaget", "30-åring".

Thanks for pointing out the issue, and for correcting it! Will merge this right away.


Nested array with incomplete objects leads to type being lost

If an array contains a empty or partial object, the type inferred is lost. Example:
require("generate-schema").json(
{array:[
    {"a2":[{something:""}]},
    {"a2":[]},
]})

The object inside a2 array should have the "something" field. But the result is now
{"a2":{"type":"array","items":{}}}

vs
require("generate-schema").json(
{array:[
    {"a2":[{something:""}]}
]})

which correctly infers
{"a2":{"type":"array","items":{"type":"object","properties":{"something":{"type":"string"}}}}}

The current code handles non nested array nicely though.
require("generate-schema").json(
{array:[
    {"a2":[{something:""}]},
    {}

]})

generates
{"a2":{"type":"array","items":{"type":"object","properties":{"something":{"type":"string"}}}

Great catch, added fix for this, and a fix for different types in the nested array. This also handles mixed objects & types properly as well now

Released in v2.4.0


Injector: Act on multiple components before they are built

Issue #331 dealt with acting on a single component during the configuration phase, but sometimes, like when you need to set the Dbms password, you need to access multiple components at the same time (in this case PasswordComponent and ProjectComponent). There must therefore be some interface for doing both.
This could look like this:
Injector.builder()
    .put(PasswordComponentImpl.class)
    .put(ProjectComponentImpl.class)
    .beforeResolved(
            PasswordComponent.class,
            ProjectComponent.class, (passComp, projComp) -> {
        projComp.getProject().dbmses().forEach(dbms -> {
            passComp.setPassword(dbms, "GLOBAL PASSWORD");
        });
    })
    .build();

A potential issue with the example above is that it doesn't clarify which state the second parameter is in when the action is executed. Must it execute before both of them? Does it execute one time for each or one time in total?

This is required to solve #330...

Maybe something like this:
injectorBuilder.executeBefore(initialized(PasswordComponent.class)
        .action((passComp) -> {
            // Do something with one component
        })
    );

injectorBuilder.executeBefore(initialized(PasswordComponent.class)
        .withStateInitialized(ProjectComponent.class)
        .action((passComp, projComp) -> {
            // Do something with two components
        })
    );

injectorBuilder.executeBefore(initialized(PasswordComponent.class)
        .withStateInitialized(ProjectComponent.class)
        .withStateResolved(ThirdComponent.class)
        .action((passComp, projComp, thirdComp) -> {
            // Do something with three components
        })
    );


Cut new release

npm install -g web-component-tester fails because of https://github.com/Polymer/web-component-tester/commit/0adf97770d8f15cd2e29fd6d6967d765d7ad6ba8. That was fixed after the latest 4.1.0 release. Can we cut a new release?
@rictic @azakus

Published v4.2.0 with this fix and new httpbin functionality.


fix: make fullscreen mode trigger automatically

Thanks for the contribution! I should note that it would be better if formatting changes were done on a separate commit, so the code is easier to review. It's not a big deal though, just a general recommendation.


coprocessor/dag/expr: implement SubDurationAndDuration

Signed-off-by: caohe<EMAIL_ADDRESS>
What have you changed? (mandatory)

Implement builtin function: SubDurationAndDuration in coprocessor.

What are the type of the changes? (mandatory)

New feature (non-breaking change which adds functionality)

How has this PR been tested? (mandatory)

Unit test

Does this PR affect documentation (docs) update? (mandatory)

No

Does this PR affect tidb-ansible update? (mandatory)

No

Refer to a related PR or issue link (optional)

#3275

@zz-jason @rleungx  PTAL


Make _tmain() return int instead of void.

Some samples won't compile since _tmain() is declared void.
Signed-off-by: Ted Lyngmo<EMAIL_ADDRESS>

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: TedLyngmo sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

@TedLyngmo : Thanks for your contribution! The author(s) have been notified to review your proposed change.


gsoler2 Acrolinx quality push: Install-MultiPoint-services.md

#assign: @robinharwood, @Xelu86
@robinharwood @Xelu86
Can you review the proposed changes?
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Some minor changes, merging now.


[Issue] azuretools.azd: ##[error]EPERM: operation not permitted, symlink '...\azd-install\azd-windows-amd64.exe' -> '...\azd-install\azd.exe'

- [ ] Make sure you've installed the latest version using instructions in the wiki <-- Does not apply
Output from azd version
NA... trying to install it
Describe the bug
After installing the azuretools.azd extension, when running setup-azd@0 on Self-Hosted Agent, it results in an error:
##[error]EPERM: operation not permitted, symlink '...\azd-install\azd-windows-amd64.exe' -> '...\azd-install\azd.exe'

To Reproduce
Run an Azure DevOps Pipeline with setup-azd@0 task
Expected behavior
No error :)
Environment
Information on your environment:
* Language name and version NA
* IDE and version : [e.g. Visual Studio 16.3] NA
Additional context
Thank you for all your efforts out there.  They are appreciated. 🙏

@hemarina can you triage this?

@vhvb1989 could it be some permissions issue on the self-hosted agent?
@Mike-E-angelo do you run it as admin?

Hi @rajeshkamal5050 I did look into this and it's running under NETWORK SERVICE.  I tried giving this folder Full Control to this account specifically and propagating this down to all children but the problem still persists.  I am open to any further suggestions on how to address this.

Hi @rajeshkamal5050, I can reproduce the same problem when I run setup-azd@0 on Self-Hosted Agent:

When I run setup-azd@0 on the windows-latest vmImage in the Microsoft-Hosted Agent, this issue does not occur.

Thank you for the ping @hemarina,  Am I to understand correctly that setup-azd@0 is not compatible with Windows Self-hosted agents?  And are you saying that nothing can be done about this from the extension side?  This is surprising if so.

Feel free to re-open it if there's any questions or concerns

Additionally @hemarina I do not have the option to reopen this issue.  Shall I create a new one?

You need to update configuration settings on self-hosted agent to grant Azure DevOps user the access to perform written permission

Can you please explain why the extension cannot do this?  This seems to be the kind of thing an install (extension) performs.  At the very least, this seems to be something that should be captured in documentation somewhere and I have not seen anything about this.
Thank you for any clarification that you can provide.

I also want to be sure of the experience here.
The user:

Installs Install azd extension on Windows self-hosted agent, expecting this to make all necessary installation and configurations to use setup-azd@0 task on the target server
Attempts to use setup-azd@0 task
Encounters an obscure permission error without any guidance in the message on how to resolve it
Does search for error, with a little luck will find this issue
Learns that the extension is not necessary at all and will instead need to copy/paste an MSI command
Uninstalls Install azd as it is not even needed

Two possible resolutions:

(not preferred) From a basic outset, it looks and feels like Windows is not a supported platform for the extension and this should be noted (even restricted?) from the install page.  Otherwise, this will lead to 1-star reviews once future users experience the above.
(preferred and expected) The Install azd extension finds a way to assign necessary permissions to the Log On identity to use the setup-azd@0 task

Thanks for your feedback @Mike-E-angelo

The Install azd extension finds a way to assign necessary permissions to the Log On identity to use the setup-azd@0 task

This is not possible by OS design.  Applications can't auto-elevate its access permissions and more important, they shouldn't.
What you are facing is beyond Azure DevOps tasks. It is 100% related to how the self-host is configured to dispatch pipelines.  Only the self-host owner should be able to set what OS user handles the pipelines and what permissions that user has.
A local system account is used to run your pipeline in the self-host machine. You need to know what that account limits are and set it up.
The account might not only fail for the install azd task, but for any application trying to write files in disk.
The install azd taks is getting an OS - access error when trying to write a file to this path:

You can try to set up the account in the self-host to be able to write new files in that path by running a pipeline with a pwsh task to write and move files.
I think that, since the path where the account is running was not created by that account (but with another account), there is no access allowed.  Make the NT AUTHORITY\SYSTEM account an owner of the C:/agent path

Thank you for your reply @vhvb1989 it is appreciated. 🙏

Applications can't auto-elevate its access permissions and more important, they shouldn't

I am nodding along with this but install applications do run with elevated permissions to perform the necessary functions required for a successful installation of an application correct?  Isn't this what happens when the Install azd extension runs?

What you are facing is beyond Azure DevOps tasks.

Just so I confirm my understanding here, it's not just me but anyone who attempts to use Install azd extension on Windows self-hosted agents, correct?  As it was confirmed by your team the concern is that any future users who attempt to use this will encounter this as well, leading to a poor experience.

A local system account is used to run your pipeline in the self-host machine.

For clarity (and so I understand 😊) in my case it is the NT Authority\Network Service and not the NT Authority\System account:

Make the NT AUTHORITY\SYSTEM account an owner of the C:/agent path

This was one of the first actions I tried (with NT Authority\Network Service) but it did not address the problem.  There may be other considerations here, however.
As I have executed the MSI command I am unblocked on this issue.  I suppose my concern now is for future users of your product who may run into the same issue.  Perhaps I worry too much about your product and should let this go. 😁

@Mike-E-angelo , according to this: https://stackoverflow.com/questions/66361808/what-permissions-are-needed-for-the-azure-devops-deployment-group-agent:

If you run it under other accounts/roles, it will fail with unexpected errors

You might want to look at this.

You might want to look at this

Whoa @vhvb1989 thank you for sharing, and I have to say that is really surprising to me. 😅  I installed 3 different agents on 3 different machines using default settings and they are all using Network Service. 🤔  This is in the past year as well.  I do not recall getting fancy here and went with the default settings for all 3 installations.

Ahhhh I closed too soon @vhvb1989 apologies. 🤦♂  While the Network Service might explain my situation, it does not explain how @hemarina was able to reproduce this with (what I assume is) a clean/new install.
Can you confirm that System was used for your reproduction @hemarina?

OK one less mystery to worry about @hemarina 😅  Thank you for the clarification 🙏


User should be able to sell token for collateral in balancer pool

Users must be able to sell one of their tokens to the balancer pool and receive collateral back. This calculation is too complex to be done in the EVM so a typescript function is needed. This applies to the AMMFactory sell and exitPool soldiity methods.
(AMMFactory here is unrelated to the old AMMFactory. They just do the same sort of thing but very differently.)
This ticket also includes tests for adding, removing, then adding liquidity. As well as buy-sell-buy.

I think we should start with ONLY handling a 3-pool!


Fix generating empty classes

Closes #116

What:
Remove empty classes.

Why:
#116

How:
Rules that don't have any declarations are removed. This includes rules that have composes since they are removed earlier. If there are no rules in css or styled, the createEmotionStyledRules and createEmotionRules functions are removed.

Checklist:

[ ] Documentation N/A
[x] Tests
[x] Code complete

Codecov Report

Merging #118 into master will increase coverage by 0.09%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #118      +/-   ##
==========================================
+ Coverage   88.34%   88.44%   +0.09%
==========================================
  Files          16       16
  Lines         592      597       +5
  Branches      128      131       +3
==========================================
+ Hits          523      528       +5
  Misses         55       55
  Partials       14       14

Impacted Files
Coverage Δ

src/babel.js
100% <100%> (ø)
:arrow_up:

src/parser.js
100% <100%> (ø)
: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 cd2f3dd...700b404. Read the comment docs.


refactor: 💡 remove two methods

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


docker executable not available to run docker build and docker push

docker executable not available to run docker build and docker push.

cc @michaelneale @ndeloof @ydubreuil

until you run Docker in Docker this doesn't make more sense
and if you don't, and just bind mount /var/lib/docker.sock, then it's safer to also bind mount /usr/bin/docker to ensure you run the correct docker cli version

agree with @ndeloof. Docker-in-Docker is not a recommended solution for anyone and not supported. Bind mounting in both the docker socket and binary is recommended way and requires nothing else installed.

@michaelneale @ndeloof How can a job running in a container (cloud slave provided by jenkins-kubernetes, jenkins-mesos, jenkins-amazon-ecs) can create an image and push it to a registry?
I am not asking to do a docker run from this jenkins build.

Both docker build and push commands do run on daemon, not client.
Considering a kibernetes / mesos hosted container as a plain classic
Jenkins slave is a misconception
Le 18 nov. 2015 4:39 PM, "Cyrille Le Clerc"<EMAIL_ADDRESS>a
écrit :

@michaelneale https://github.com/michaelneale @ndeloof
https://github.com/ndeloof How can a job running in a container (cloud
slave provided by jenkins-kubernetes, jenkins-mesos, jenkins-amazon-ecs)
can create an image and push it to a registry?
I am not asking to do a docker run from this jenkins build.
—
Reply to this email directly or view it on GitHub
https://github.com/cloudbees/java-build-tools-dockerfile/issues/14#issuecomment-157752074
.

+1
I have a similar issue. I am running Jenkins on Kubernetese. I normally use just start Jenkins on RAW docker. However I ave switched to Rancher/Kubernetese. I have a Jenkins slave running using the kubernetese plugin. I get the following issue.
docker build -t hub.docker.com/testing/testing_quickstart:29 --pull=true --file=docker/ /home/jenkins/workspace/01-DEV/01-testing/03-Packaging
ERROR: Cannot run program "docker" (in directory "/home/jenkins/workspace/01-DEV/01-testing/03-Packaging"): error=2, No such file or directory
I have put "-v $(which docker):/usr/bin/docker" into "Arguments to pass to the command" and mapped /var/run/docker.sock under the volume configuration.
Any help appeciated

@t5unamie -v $(which docker):/usr/bin/docker won't work as $(which docker) is resolved on client machine, not docker host. You need to assume docker is installed on standard /usr/bin/docker location on host or get this information from administrator.

Same issue. I have installed Jenkins on Kubernetes. I am trying to build a docker image and push it to the registry from the Jenkins build but it throws Cannot run program "docker" (in directory "/home/jenkins/workspace/myjob"): error=2, No such file or directory error.

+1 Please, include docker binaries in this image.
I have mounted /usr/bin/docker and /var/run/docker.sock for my kubernetes nodes, and get this error:
docker: error while loading shared libraries: libltdl.so.7: cannot open shared object file: No such file or directory
If the binaries are included, this will not happen any more.

@vinixavier what's your use case to use the docker executable? for people who want to build docker images from within Docker containers, there are today several solutions such as Google's Jib and Google Kaniko

@cyrille-leclerc i'm using Jib + Amazon ECR Jenkins Plugin for authentication with docker.withRegistry() function. This way use docker login command.

understood, thanks.
Would the following binaries do the job? https://download.docker.com/linux/static/stable/x86_64/

Yes. It is enough! 👍

@cyrille-leclerc How did you finally solve this problem ? Did you extend image by adding docker binaries ?


Comment popup

Add functionality for comment popup

Create a module file for a modal popup
Add structure and styling
Add JS functionality

Hey, @netman5  you did a good job, I liked the structure and style, there is just a little issue in the form as you see in the screenshot, I suggest giving it a display flex column to solve it.

Thank you @TimmyChan99, honestly don't know why is like that, below is the picture of how it looks in my local machine

Not sure what to do now.


Fix issue with multiple types of biometrics on Android

Fixed mytonwalletorg/mytonwallet-dev#2799

It is probably necessary to move the getAllowedAuthenticators function in its entirety and insert its result, as shown in these links.

canAuthenticate https://github.com/SelfLender/react-native-biometrics/blob/master/android/src/main/java/com/rnbiometrics/ReactNativeBiometrics.java#L57
setAllowedAuthenticators https://github.com/SelfLender/react-native-biometrics/blob/master/android/src/main/java/com/rnbiometrics/ReactNativeBiometrics.java#L191

The prompt is set in the Activity in our plugin (separate file).
(sorry for my auto translate)

We are fixing an issue where biometrics do not work correctly on a specific Android device (when both face and fingerprint are requested simultaneously). We know that other applications do not have this problem, so I assumed that this code would fix the issue, most likely because the face is not classified as a STRONG type.


Fix issue with shallow navigation

Project organization

Closes https://github.com/liteflow-labs/starter-kit/issues/168
Related to the new filter's UI

Description
The filters introduce new navigation with the shallow option that basically changes the URL without having to have a full query on the server side. This is great as it is then faster for the user and has a lot less cloud function consumption.
That said, because of that, a lot of pages that were designed with SSR first approach do not work anymore, and for example, in #168, the pagination was not up to date, and was still getting the initial page from the SSR query.
I moved all the retrieving of pagination and sort (filter was already done) on the client side, this way, all issues are solved, and we can now correctly get the params from SSR or the shallow redirect.
Note the last commit, cce46ea88721beb76ed0387d7d3f0045e38c08c4, is not required and can be reverted if it's too much, it's just to ensure that all the codebase is similar and that we will not have an issue in the future
How to test
The best way is to go on the explore, navigate to the next page, and select a filter. Previously this would not redirect to page 1 or no page, now it should

Nice catch, I believe we had this issue for quite some time but that's now fixed b792541


Add Mk 47

Can you please compile and upload somewhere(or make a release in your fork on github) a GFChipCalc  .jar standalone file with your commit included for ordinal players to use? Since original GFChipCalc repo seems abandoned for a long time.

I need AuthCode.java file. I can extract it from a released GFChipCalc.jar but distributing it may cause some issue.

Chipset system is removed from game


Add Clipboard interface, readText, writeText

Description of the change
See title :smile:

Checklist:

[ ] Added the change to the changelog's "Unreleased" section with a reference to this PR (e.g. "- Made a change (#0000)")
[ ] Linked any existing issues or proposals that this pull request should close
[ ] Updated or added relevant documentation
[ ] Added a test for the contribution (if applicable)

Can you post the link to the spec?

Oh sure, sorry.

Here's the modification to the Navigator interface to add clipboard: https://w3c.github.io/clipboard-apis/#navigator-interface
Here's the Clipboard interface itself: https://w3c.github.io/clipboard-apis/#clipboard-interface

I perhaps also should have mentioned this is a partial implementation of the API, I need writeText but don't have time to figure out all the ClipboardItem stuff so just included the plain text operations. Taking the lazy approach where implementation is forced by need :wink:

Can you add a changelog entry?


Document Debian Based Distros

Document how to set up  Debian based distros like Debian 8 Jessie or Ubuntu 14.04 Trusty.
There is support for it:
https://github.com/pr1ntf/iohyve/blob/dev/iohyve#L260

Merged #15


NPE at new MyDataTypePrimaryKeyIndex(consumer)

I have a MyDataTypePrimaryKeyIndex which is generated from MyDataType.
When the index is created it throws NullPointerException from new MyDataTypePrimaryKeyIndex(consumer) because
consumer.getStateEngine().getSchema("MyDataType")) returns null.
Is there any way to correctly initialise the state engine that it doesn't throw NPE?
Probably a more understandable exception should be thrown instead.

Seems like this could be related to https://github.com/Netflix/hollow/issues/140


Support VCF files as input

Current input file format is not commonly supported and requires custom treatment. Ideally, we would support using the common variant file format VCF. Potential routes:

Include a script to convert from VCF to CALDER-formatted input file.
Convert input/internal-use file format to VCF.

Hello @mmyers1,
Any update with this great step? It would be awesome if we can infer the HATCHET output and integrate into CALDER input.


WoW Cataclysm - CreateUnsecuredObjectPool is nil

Any chance for classic support? CreateUnsecuredObjectPool doesn't exist (and is in 2 files), but CreateObjectPool works. Any chance on adding some game client version detection to support this?

Ugh, it's a sensitive topic.
I don't play classic at all, so I avoid adding classic support to my addons. Toasts are the only exception because initially it's pretty easy to implement for vanilla, tbc, and kinda wotlk, but, ngl, even that's gotten pretty annoying lately because the state of classic API is a complete mess, updating that addon for cata was a royal pita.
For this reason I want to stick to just retail for the rest of my addons.


python310Packages.teslajsonpy: 2.4.2 -> 2.4.3

Description of changes
https://github.com/zabuldon/teslajsonpy/releases/tag/v2.4.3

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

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

2 packages built:

python310Packages.teslajsonpy
python39Packages.teslajsonpy


Update stream file doc, change mlp-width default to 10

This is a very straightforward PR. I added a little more description about the metadata available for stream files in the CLI. I also am changing the default --mlp-width to be 10 rather than inferred from the metadata width.
This resolves #141  and #142

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

Comparison is base (ccc58c0) 78.92% compared to head (0f3046d) 78.83%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #144      +/-   ##
==========================================
- Coverage   78.92%   78.83%   -0.09%
==========================================
  Files          52       52
  Lines        2344     2344
==========================================
- Hits         1850     1848       -2
- Misses        494      496       +2

Flag
Coverage Δ

unittests
78.83% <ø> (-0.09%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Dyson Cloud never worked, using Dyson Local only now

Hello together,
In the past, I never got the dyson cloud integration working and still had this issue until today (also solutions above did not help). Since yesterday I invested several hours again, but I did not make it to get dyson cloud integration working. Finally I found a solution (hopefully stable) which I would like to share.
Existing Problems

Constant dyson cloud errors in the logs and dyson appeared as "not loaded". see https://github.com/shenxn/ha-dyson/issues/171#issuecomment-1369998737
Maybe other people from DE, AU or US the same issue like me:  libdyson.exceptions.DysonInvalidAuth when trying to use the python3 get_devices.py (see link). And after months and years (why not earlier ??? ) I found a suitable solution / workaround to get credentials for dyson local (see option 2 below).

Solution: Dyson Local only !

Finally I decided to completely switch to dyson local only and kicked out dyson cloud from my configuration at all ! (it's better local than relying on communication with the cloud!)
And the good news, I can use the dyson integration in parallel with the Dyson Android mobile app (what did not work for me before).

What did I did:

I deleted both integrations (dyson cloud and dyson local) from HA again.
I checked in the /config folder of HA, that both directories (for dyson cloud and dyson local) were gone.
Restarted HA.
Installed dyson local again (using HACS).
In HA > integrations: Started dyson local configuration by adding it to the integrations list.
As "Setup Method" I selected "Setup manually"
6.1. Serial*: Enter the serial number of your device (e.g. K3G-EU-RA....)
6.2. Credential*: Get the credential (see below) and enter here.
6.3. Device Type: Select your device type (in my case "Pure Cool Formaldehyde")
Press send.

Finally dyson local got successfully configured and all required data / services are available in HA! That's it!
How to get credentials?
Usually, there are two options (@Kakise, @shenxn: Wouldn't it make sense to document option 2 in the official documentation?):

Using python3 get_devices.py see https://github.com/shenxn/ha-dyson#setup-manually (never worked for me)
Using Android app to catch credentials locally as described by @proatwork. < the only solution that worked for me!

Using an Android phone
You can probably do this with an iOS device, but I haven't tested (https://stackoverflow.com/a/8512455)

Download the Packet Capture app from Google Playstore (https://play.google.com/store/apps/details?id=app.greyshirts.sslcapture&hl=en&gl=US)
Open Dyson Link and leave it running in the background
Run the Packet Capture  app, allow it to create a VPN if you're prompted
Hit the play button with the 1 and choose the Dyson Link app
Switch to the Dyson Link app, it will probably do a quick refresh of the welcome page
Switch back to Packet Capture and stop capturing
Look through the captured packets for something like this:

Hope other people can benefit from it.
Cheers

Thanks @erd82 following your Android tutorial, I got my fans connected to Home Assistant!

Is there any way to do this without an Android device? 🤔

Is there any way to do this without an Android device? 🤔

I guess you need to check for a suitable network sniffer, which is able to catch the respective network package.
For Windows: maybe Wireshark?
For Apple: Is there any experienced Apple user who can recommend something? Maybe something like Intercepter-NG? https://alternativeto.net/software/wireshark/?platform=iphone
@Community: Feel free to share your experience here.

Sadly the App Store seems to be missing a chunk of packet sniffers so I'm out of luck

I had managed to successfully get a network capture with the following app. Unfortunately the SSL capture feature is behind a subscription but it’s not too bad.
https://apps.apple.com/app/id1610958307

Thanks @erd82 , I managed to get this working using an app called PCAPdroid (Packet Capture is not compatible with recent versions of Android). But when I enter the captured details, I get the error "Failed to find the device using discovery". Any ideas?

Thanks for this @erd82. Just a note to say that Packet Capture doesn't appear to work with recent versions of Android, but I was able to get this working with an app called PCAPdroid. I'm still struggling to connect to my HP07, as described here, but I don't believe that's an issue with the captured credentials.


Feat/login_providers

Add new login providers.
Wallet-services version v2 updated

Should we also remove the loginParams from the request method? Since it also uses the wallet services?

Closing this in favor of #72


unable to run benchmarks

running benchmarks with ASV currently does not work due to a missing configuration file

thanks. See also PR gh-71

This should be fixed by Matti's PR #71. Please re-open if you're still unable to run the ASV-based benchmarks.


Add Brian's information to AboutUs

Description
This PR addresses issue #18 by removing the dummy information from the "About Us" section.
Changes

Removed placeholder text and replaced it with updated content in the "AboutUs" section.

Issue
Closes #18
Notes
Please review and let me know if any further modifications are needed.

LGTM!


Add deployment tolerations

This PR is to add deployment tolerations

Looks good - can you bump the version so the tests pass?

I bump chart version to v3.0.12

Thanks!


should eql? be needed for proxies?

I'm having crashes with supervisor pools, specifically when terminating,
module Celluloid
	module Supervision
		class Container
			# Manages a fixed-size pool of actors
			# Delegates work (i.e. methods) and supervises actors
			# Don't use this class directly. Instead use MyKlass.pool
			class Pool
				# Spawn a new worker for every crashed one
				def __crash_handler__(actor, reason)
					p "Oh no! #{actor.inspect} has died because of a #{reason.class}"

					@busy.delete actor
					@idle.delete actor
					@actors.delete actor
					return unless reason
					@idle << __spawn_actor__
					signal :respawn_complete
				end
			end
		end
	end
end

crashes because the actor is dead and calling eql? on the actor crashes the crash handler.
I was wondering if it makes sense to add eql? to the list of needed methods, e.g.
# Base class of Celluloid proxies
class Celluloid::Proxy::Abstract < BasicObject
  # Used for reflecting on proxy objects themselves
  def __class__
    ::Celluloid::Proxy::Abstract
  end

  # Needed for storing proxies in data structures
  needed = [:object_id, :__id__, :hash, :private_methods, :eql?] - instance_methods
  if needed.any?
    include ::Kernel.dup.module_eval {
      undef_method(*(instance_methods - needed))
      self
    }
    # rubinius bug?  These methods disappear when we include hacked kernel
    define_method :==, ::BasicObject.instance_method(:==) unless instance_methods.include?(:==)
    alias_method(:equal?, :==) unless instance_methods.include?(:equal?)
  end
end

At least this appears to fix the problem but I'm not 100% I understand what is going on here.

@tarcieri @digitalextremist can you give some feedback on this.

Seems like a reasonable request

How has this not been a problem for other people?

It probably has...

IS this the correct way to fix this issue? Like I just took a random stab, after searching for :hash and finding something here. I added eql? with only a vague understanding of what was going on, and it seemed to fix the problem.

To be honest I really don't like the way the proxies are implemented at all and wish there were a more first-class feature in Ruby for building proxy objects
I think overriding :eql? to require actor invocations is nuts, so my gut feeling is yes. Equality is something we should be able to test more simply.
However, this may mean proxies to the same actor do not compare equally, so perhaps the proxies need a more specific definition of :eql? that can compare the actor they're proxying to without making actor invocations.

Can't eql? be object and type equality of the actor it is looking at? Shouldn't that technically be the same as Hash?

That's what I was saying, yes...

Then why is this a problem?

However, this may mean proxies to the same actor do not compare equally, so perhaps the proxies need a more specific definition of :eql? that can compare the actor they're proxying to without making actor invocations.

Implementing eql? could be done using just Actor#type and Actor#object_id right?

Sorry, I guess I don't really understand how the current implementation works. I'd like to see the Proxy implement #hash and #eql? so that they don't need to invoke the actor at all. How can I do this?

I would suggest confirming class and @mailbox are the same


KE-23593 upgrade jackson

What changes were proposed in this pull request?
upgrade jackson to 2.10.5
How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)
Please review http://spark.apache.org/contributing.html before opening a pull request.

retest this please

retest this please


Add GetCurrentProcessToken

Suggestion
The original winapi has a method for getting the process token .GetCurrentProcessToken()
But in this api for rust, I have to do something like:
let current_process_handle: HANDLE = unsafe { GetCurrentProcess() };
    let token_handle: *mut HANDLE = ptr::null_mut();
    unsafe {
     OpenProcessToken(current_process_handle, TOKEN_ADJUST_PRIVILEGES | TOKEN_QUERY, token_handle).unwrap()
      }

And also .GetCurrentProcess() from the example above always return invalid handle, -1 value. So I can't get current process token. (I know about .GetCurrentProcessHandle())

My suggestion is to implement .GetCurrentProcessToken() in this api, if it doesn't already exist and I'm just dumb

GetCurrentProcessToken isn't a function exported by Windows DLLs. It's just an inline function that always returns -4. From the C headers:
HANDLE GetCurrentProcessToken() {
    return (HANDLE)(LONG_PTR) -4;
}

Sorry, I made a mistake and it looks like this function is not needed, but GetCurrentProcess is a similar function that always returns (HANDLE)-1, so...


docs: add example and testcase for variable definition

I just stumbled across this and think the failing test is originating in the duplicate top level __definitions__ key. Since this is not valid yaml the parser fails to execute both __definitions__ so the variable test is missing. The simple workaround is to wrap other __definitions__ in something like ?for _ in [] so there are no multiple keys anymore. The README mentions the following:

It can be used anywhere in the YAML, also repeatedly and inside of ifs or loops.

We should probably explicitly mention the duplicate key issue there or it might be quite confusing for new users otherwise unless you have another solution in mind @johanneskoester?

I just stumbled across this and think the failing test is originating in the duplicate top level __definitions__ key. Since this is not valid yaml the parser fails to execute both __definitions__ so the variable test is missing. The simple workaround is to wrap other __definitions__ in something like ?for _ in [] so there are no multiple keys anymore. The README mentions the following:

It can be used anywhere in the YAML, also repeatedly and inside of ifs or loops.

We should probably explicitly mention the duplicate key issue there or it might be quite confusing for new users otherwise unless you have another solution in mind @johanneskoester?

Thanks! Indeed this is the issue. The empty for loop is quite a hack. I don't think this should be a suggested solution. Rather I'd like to have a sentence in the readme that clarifies that definitions may occur at multiple places, but not more than once on the same level.


0.6.1

Changes:

Update version to 0.6.1

https://github.com/roy-ht/editdistance/tree/v0.6.1

Linter check found the following problems:
The following problems have been found:
WARNING: /tmp/abs_d7_xqevn6w/clone/recipe/meta.yaml:47: missing_license_url: The recipe is missing a license_url
WARNING: /tmp/abs_d7_xqevn6w/clone/recipe/meta.yaml:47: missing_doc_source_url: The recipe is missing a doc_source_url
Warnings were found

Linter check found the following problems:
The following problems have been found:
WARNING: /tmp/abs_1b5q9c5qao/clone/recipe/meta.yaml:53: missing_license_url: The recipe is missing a license_url
WARNING: /tmp/abs_1b5q9c5qao/clone/recipe/meta.yaml:53: missing_doc_source_url: The recipe is missing a doc_source_url
Warnings were found

Linter check found the following problems:
The following problems have been found:
WARNING: /tmp/abs_fbk7iw71bd/clone/recipe/meta.yaml:54: missing_license_url: The recipe is missing a license_url
WARNING: /tmp/abs_fbk7iw71bd/clone/recipe/meta.yaml:54: missing_doc_source_url: The recipe is missing a doc_source_url
Warnings were found

Linter check found the following problems:
The following problems have been found:
WARNING: /tmp/abs_9eetv26mw0/clone/recipe/meta.yaml:54: missing_license_url: The recipe is missing a license_url
WARNING: /tmp/abs_9eetv26mw0/clone/recipe/meta.yaml:54: missing_doc_source_url: The recipe is missing a doc_source_url
Warnings were found


Make neo4j module support Gradle caching

From Gradle docs:

Some tasks, like Copy or Jar, usually do not make sense to make cacheable because Gradle is only copying files from one location to another.
But we need the cache, since else we can't cache the test?

Yeah, I read that as well and came to the conclusion that this is a case where it makes sense to break the rules.


Unable to sign in: ERR_TUNNEL_CONNECTION_FAILED

{
"message": ""ERR_TUNNEL_CONNECTION_FAILED""
}

What OS are you on?

win10
storage explorer version <IP_ADDRESS>
Not a proxy

Have you tried to use Device code flow or MSAL to sign-in?  If yes, were you able to sign-in?  Also, have you tried to use Name/Key to connect to any of your storage accounts?
Is this only happening to this particular machine or all machines you've tried?  Can your co-workers sign in?  Thanks.

Closing due to lack of response. Please open a new issue if you are still having problems.

Click the login page to flash back.

the page will close.


Popkey

Pretty new GIF sharing platform, similar to Giphy. https://api.popkey.co

I like this. What would you prefer, having a command specifically for popkey or having it random select between giphy and popkey?


helm chart - typo in cronjob template

change promscal -> promscale on line 60
Error: UPGRADE FAILED: template: promscale/templates/cronjob-maintenance.yaml:60:25: executing "promscale/templates/cronjob-maintenance.yaml" at <include "promscal.fullname" .>: error calling include: template: no template "promscal.fullname" associated with template "gotpl"

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for the fix @wileyj
But the maintenance job is by default disabled from two releases as TimescaleDB does by using UDJ (user-defined jobs), Any specific reason why are you explicitly enabling the maintenance job?

Thanks for the fix @wileyj
But the maintenance job is by default disabled from two releases as TimescaleDB does by using UDJ (user-defined jobs), Any specific reason why are you explicitly enabling the maintenance job?

Yes - using timescale-cloud, there is apparently a bug in the postgres extension 2.4.1 and the suggestion from support was to enable the maintenance jobs in the meantime.
Changing false to true (i did notice the comment about it not being needed https://github.com/timescale/promscale/blob/master/helm-chart/values.yaml#L96-L98) produced the error above.

Plus, it never hurts to fix a typo! This one just happened to help me get past some storage usage issues using timescale-cloud.

bors r+

@wileyj Thank you!


Can't divide by 0

Describe the bug
It seems like you can't divide by 0. This needs to be fixed ASAP as possible.
Steps To Reproduce

Click on any number
Click on division symbol
????????
Profit

Expected behavior
The result of dividing a number by 0.
Device and Application Information (please complete the following information):

OS Build: Windows 10.

Additional context

Please fix this fast, I need to divide by 0.

This is your friendly Microsoft Issue Bot. I've seen this issue come in and have gone to tell a human about it.

This is why they are bot open sourcing Windows 😂

I am not able to replicate the issue.

The proper output when dividing by zero should be:
E̹̝̖ͅr̷̷̗̰̲͉͍̝ŕ͚͈ò͓̺̼̫̖r̲͚͞,̴̯͓̤̕ ̰͔̭̬̮̱̲d͔̠̰̙̦͍͍͔͜͞ͅi̶̳̪̯͎̠̩͉v̷̗͓͇̤̰i̵̧̖̞̗̠̯͜ͅd̲͎̯͍͉̖̙͜e̶̮̝͙͇ ̷̬̘͚̥̹͖͕̘b̡̠͖̝͚̹y̨̬͕̣̫̭ͅ ̢̨̠̗̜̬̰͞ź̲̬̦̤̥̗̗̥̻͢͞è̛̲̪͕̦̠r͍̭̫̣̘̥̕͟o̷҉̥̜̟͔͎̯͕̯ ̜̫̲͓̰̠͞i̷͚̪͕s͙̘͔̀͠ ̵͎͓̟̦͞n̷̪̲̞̩͇̜̤ͅo͉̪͡t̪͜ ͔̝v͍̝͖a̼̤̗͈l̴̮̰̫͎̞͙̫͖̕į̘̭ḏ̨̘̬.҉̳̰̗̙̦̦̬̯

According to IEEE 754 should return ±infinity.
If the divisor is zero and the dividend is a finite nonzero number, then the
division by zero exception shall be signaled. The result, when no trap
occurs, shall be a correctly signed INFINITY

It'd be nice if everyone didn't have to fill the repo with troll issues and replies, some of us were hoping subscribing to the repo wouldn't equate to this garbage.

Oh come on.
@ScottieD is right tho, works on my end too.

Ha, thanks for the report! You had @MicrosoftIssueBot confused for a bit there...


Unable to use different separators in numbered list

I can't get the numbering to look how I want:
#set enum(numbering: "1.a)")

I expected this to give "1." and "a)".

I'm guessing this wouldn't be so trivial to fix, as this behavior could be legitimate - for example, this would make perfect sense for a numbering of (1.a).
Perhaps this could be given some sort of toggle (to use each individual suffix instead of just the last one)?

I made a quick patch in https://github.com/PgBiel/typst/tree/numbering-fix to demonstrate this. (Note that this is just a prototype, not supposed to turn into a PR or anything at the moment.)
It would replace (1.a.b) with (1., then (a., then (b), which I think the normal user wouldn't expect.
Of course, all of this wouldn't matter if we all simply used functions for numbering - but I guess the usage of a string in numbering is mostly a heuristic anyway.

Regardless of where this goes, here's a solution to your particular case:
#set enum(full: true, numbering: (..nums) => {
  let midmarker = "."
  let endmarker = ")"
  let nums_len = nums.pos().len()
  for (i, num) in nums.pos().enumerate() {
    let marker = if nums_len != 1 and i == nums_len - 1 { endmarker } else { midmarker }
    [#num#marker]
  }
})

+ a
+ a
  + b
  + b
    + c
  + c
+ d
+ e

@PgBiel I've run into this too and I think it'll be a common issue. What if we replaced the string with an array? That seems to be the fundamental flaw in the current approach: we can't differentiate between intermediate separators and styling. If numbering accepted an array instead, we could add styling around each level of the list. Thoughts? cc @laurmaedje

Regarding my workaround, here's an updated version with letters instead of numbers at the second level (and beyond):
#set enum(full: true, numbering: (..nums) => {
  let midmarker = "."
  let endmarker = ")"
  let nums_len = nums.pos().len()
  for (i, num) in nums.pos().enumerate() {
    let marker = if nums_len != 1 and i == nums_len - 1 { endmarker } else { midmarker }
    let num-rep = numbering(if i == 0 { "1" } else { "a" }, num)
    [#num-rep#marker]
  }
})

+ a
+ a
  + b
  + b
    + c
  + c
+ d
+ e

Or, to only show the last level on each line:
#set enum(full: true, numbering: (..nums) => {
  let midmarker = "."
  let endmarker = ")"
  let nums_len = nums.pos().len()
  let (i, num) = nums.pos().enumerate().last()
  let marker = if nums_len != 1 and i == nums_len - 1 { endmarker } else { midmarker }
  let num-rep = numbering(if i == 0 { "1" } else { "a" }, num)
  [#num-rep#marker]
})

+ a
+ a
  + b
  + b
    + c
  + c
+ d
+ e


Imposible to get it working with Slack on ios

Howdy!
As the time of writing Slack does not support non-http urls.

According to documentation, redirect url for ios is #{yourappname}://oauth.
This makes it impossible to get react-native-oauth working with slack on ios.
I suppose we could support universal links instead of deep links on ios.
I'd like to hear your thoughts about it.
For now, workaround would be to redirect to our server script which redirects to the app through a deep link and passes parameters.

This is the update:


Add impl fmt::{UpperHex,LowerHex} for {Uuid,Hyphenated,Simple}

Not sure if this is useful, just for API completeness.

Thanks!


Use Hygiene in ast_transform

r? @JmPotato

Hm, this should go the other way around, we should not expose Hygiene from hir in the first place....
This needs some design, I guess the bottom line is that we probably want to remove hir path resoling from Semantics altogether...


Add license file

Fixes #13.  Duplicate of the JupyterLab license without the extra rider for the third party file.

Thanks @blink1073!


Degree Symbols

We recently updated a number Attributes  as the website was displaying a masculine ordinal indicator instead of a degree symbol.
I  checked with you both and agreed that it was Ok to make the change.
However, we have had a helpdek query in today regarding https://www.datadictionary.nhs.uk/data_elements/degrees_of_flexion_range__primary_knee_replacement_.html?hl=degrees%2Cflexion%2Crange%2Cprimary%2Cknee%2Creplacement where the degree symbol looks odd:

Note that this looks OK in V3: https://v3.datadictionary.nhs.uk/data_dictionary/attributes/d/dec/degrees_of_flexion_range_for_primary_knee_replacement_de.asp?shownav=1
On checking other items, there is a difference in what is displayed:
https://www.datadictionary.nhs.uk/attributes/tibia-hindfoot_alignment_code_for_primary_ankle_replacement.html
https://www.datadictionary.nhs.uk/attributes/ankle_dorsiflexion_code_for_primary_ankle_replacement.html
https://www.datadictionary.nhs.uk/attributes/ankle_plantarflexion_code_for_primary_ankle_replacement.html
https://www.datadictionary.nhs.uk/attributes/degrees_of_fixed_flexion_deformity_for_primary_knee_replacement.html
Is this something for @jamesrwelch or @likin1 to fix?
Can we fix it for the December 2022 release?

How are you entering those symbols?  The codes are not coming through cleanly in the XML - they'd be better if they were coming through as °.

As an aside, the copyright symbol is having the same issues in the onto server bit of the XML, not that it's used.

@likin1 - you haven't mentioned this. Is it something you can fix?

@AngelaFaulding yes this should be fixed in december release.

Thanks @likin1 - I will check when I have the dita output.

These are now OK in the December release.


Duplicate Property Assignment for position in style.css

Description:
I have discovered an issue in the style.css file where the position property is assigned multiple times for the .layout-container class. This can lead to unexpected behavior in the layout and should be addressed to improve the code quality.

Issue Details:

File: style.css
Container: .layout-container (on line 910)
Duplicate Property: position

Screenshot:

Expected Behavior:
Each CSS property should only be assigned once per container to avoid conflicts. The second assignment of position will override the first, leading to potential layout issues and confusion.

Proposed Solution:

Remove the duplicate position property. Determine the intended value (either relative or sticky) and keep only that assignment in the code.

Additional Information:
Fixing this issue will enhance the maintainability of the code and ensure that the styling behaves as expected.

@ANSHIKA-26  I would love to work on this issue, please assign it to me

@ANSHIKA-26 can you please add the labels like gssoc-ext, hacktoberfest, hacktobderfest-accepted too


feat: adding a basic wallet creation scheme

Closes #272
Note:

I can't use this scheme in the integration tests in miden-tx/tests/test-miden-wallet.rs because in those tests I need accounts with a nonce > 1
I will use the AuthScheme later, when the Falcon key implementation is ready
After https://github.com/0xPolygonMiden/miden-vm/issues/1098 is closed, we can move the MASM code into a file

I will use the AuthScheme later, when the Falcon key implementation is ready

We already have this - so, would be good to do this even now.

So this is how the wallet code would look like. Unclear is yet where to put the public_key? Should I store it in slot 1?
    let auth_scheme_procedure = match auth_scheme {
        AuthScheme::RpoFalcon512 { pub_key } => { "basic::auth_tx_rpo_falcon512" },
    };

    let account_code_src: &str = format!(
    "
    use.miden::wallets::basic->basic_wallet
    use.miden::eoa::basic

    export.basic_wallet::receive_asset
    export.basic_wallet::send_asset
    export.{auth_scheme_procedure}

    ").as_str();

So this is how the wallet code would look like.

Yes - that's exactly how I thought about it.

Unclear is yet where to put the public_key? Should I store it in slot 1?

I'd put it into slot 0 (I think that's what we assume in miden::eoa::basic::auth_tx_rpo_falcon512)

I'd put it into slot 0 (I think that's what we assume in miden::eoa::basic::auth_tx_rpo_falcon512)

I think we can make this a more general convention: authentication data for an account is stored in slot 0. This could work for any single-owner scheme, but also for multi-owner schemes (slot 0 could contain a root of a tree with all owner keys).

So this is how the wallet code would look like.

Yes - that's exactly how I thought about it.

The last question here is, what do we want the user to create this wallet account with? If the user would provide a seed with which we can create a valid AccountId, the user already needs to have the code_root and the storage_root.
But code and storage is not "known" to the user when calling create_wallet. So the user could provide a init_seed and we grind a valid seed. But the init_seed would be random bytes anyway, so we can also just create everything in create_wallet itself.
Any thoughts?

The last question here is, what do we want the user to create this wallet account with? If the user would provide a seed with which we can create a valid AccountId, the user already needs to have the code_root and the storage_root.
But code and storage is not "known" to the user when calling create_wallet. So the user could provide a init_seed and we grind a valid seed. But the init_seed would be random bytes anyway, so we can also just create everything in create_wallet itself.
Any thoughts?

Ah - good point! Yes, let's change the interface. Maybe something like this would work better:
pub fn create_basic_wallet(
    auth_scheme: AuthScheme,
) -> Result<(Account, Word), AccountError>

The returned tuple would contain both account and the seed needed to create the account.


HHH-11205 - Fix nationalized support when using char data type.

https://hibernate.atlassian.net/browse/HHH-11205

Applied upstream.


Pretrained parameters

Hi, thanks for your excellent work and sharing codes. I have a small question. If I apply packed ensemble, the pre-trained parameters of the ResNet on the ImageNet can't be used. Is it right?

Hi, thank you for your interest! Indeed, the pretrained parameters of the ResNet on the ImageNet can't be used as such. Therefore, we plan to release as soon as possible our trained ResNet parameters on ImageNet. We will keep you posted on this matter.

Hi!
Great work! I was wondering, on Hugging Face you have the resnet18 version of packed ensembles trained on CIFAR100, but is there also a version trained on IMAGENET besides the resnet50 version?
Bests!

Hi @Vujas-Eteph, sorry for the delay.
I've started working once again on the Hugging Face models. In the following days, I'll try to put several checkpoints for Deep Ensembles and the Packed-Ensembles checkpoints on ImageNet.
Happy Holidays!

Hi @o-laurent, no worries about the delay!
It's great hearing that you'll add more checkpoints for the Packed-Ensembles (ImageNet variant) :)
Happy Holidays to you too! ❄


SetAuthCookie(ticket) doesn't create any ticket

Hi!
After a successfully login I try to set the CAS cookie using DotNetCasClient.CasAuthentication.SetAuthCookie(t);
But it seems it doesn't create any cookie.
Could it be the framework 4.7.2?
Thank you
S

ASP.NET WebForms, Web Pages or MVC?  Which version of those are you using also?  Can you also provide a snippet of your web.config, without anything sensitive, for the  and  sections.  Are you using the latest NuGet package in your project?
Do you get an errors?

Also, which ticket storage implementation are you using?   The default is in-memory, but there are a few other providers we offer.

Did your code work in the same project targeting a prior version of the .NET framework? e.g. 4.5?  I.e. was this code ever working at one point and magically one day it stopped?

Closing due to lack of response from the OP.


Pretty JSON

Check for existing issues

[X] Completed

Describe the feature
Allow to prettify hairy JSON payloads.
If applicable, add mockups / screenshots to help present your vision of the feature
No response

@krstp would you mind expanding on how this is different from formatting a JSON buffer?

Well, JSON can be minified or simply not formatted correctly. What I mean I would like to operate on sectional portions of the code, for example I am writing a Markdown file and bit of it carries a JSON code within the block, such as:
 ```json
 {}
 ``` .

and I would like to be able to select desired portion of the code and invoke JSON prettifier just to tidy up selected portion of the code. That's it. I hope it explains.


tests/_site/templates/layout.html omits {% block content_wrapper %}

I have recently been working on improving OSCAR_HIDDEN_FEATURES so that it supports the "wishlists" and "alerts" keys.  One facet of my effort was to hide the "Wish Lists" link on /accounts/profile when the "wishlists" feature is hidden.
My test plan for this particular modification is very simple:

Render /accounts/profile and verify that a link to /accounts/wishlists is present.
Use a context manager to temporarily add "wishlists" to OSCAR_HIDDEN_FEATURES, render /accounts/profile, and verify that a link to /accounts/wishlists is not present.

The following test case implements this plan:
from django.core.urlresolvers import reverse

from oscar.test.testcases import WebTestCase

class TestProfilePage(WebTestCase):
    is_anonymous = False

    def test_wishlists_link_exists(self):
        account_page = self.get(reverse('customer:profile-view'))

        html = account_page.content.decode('utf-8')
        self.assertTrue(reverse('customer:wishlists-list') in html)

    def test_wishlists_link_does_not_exist(self):
        with self.settings(OSCAR_HIDDEN_FEATURES=['wishlists']):
            account_page = self.get(reverse('customer:profile-view'))

        html = account_page.content.decode('utf-8')
        self.assertFalse(reverse('customer:wishlists-list') in html)

Unfortunately, TestProfilePage.test_wishlists_link_exists fails.
I have investigated the source of this failure and determined that it arises from the differences between src/oscar/templates/oscar/layout.html and tests/_site/templates/layout.html.  The former defines {% block content_wrapper %}, while the latter does not.  This is significant because templates/oscar/customer/baseaccountpage.html extends templates/oscar/layout_2col.html and renders the left navigation (where I expect the link to appear) into {% block column_left %}.  {% block column_left %} is defined by layout_2col.html within {% block content_wrapper %}.
Thus, because tests/_site/templates/layout.html does not define {% block content_wrapper %}, {% block column_left %} is not rendered, and TestProfilePage.test_wishlists_link_exists fails.
I suspect that tests/_site/templates/layout.html has been simplified to avoid rendering unnecessary HTML during tests.  This issue documents a case where rendering the full layout.html could contribute to improved test coverage.

During the latest review I had a closer look on feature disable functionality and found out that wishlists URL is not hidden yet on the account page, when feature is disabled, simply because it's not wrapped with {% iffeature "wishlists" %}{% endiffeature %} tag.
@zbmott you were right regarding the test template - not having {% content_wrapper %} block prevented from proper testing of account navigation, so I added it in PR #2522.


Create Question IDs for JSON

In order to create advanced features for the assessment tool, question IDs are required.
Need to update the survey.json file with proper QIDs

Might not need IDs for all questions but rather use the valueName to point to Mitigations Commitments.
Basically, when someone states that they will do X measure in the mitigation questions section, it should display the commitments taken in the Results page.
However, introducing Question IDs would be a way to ensure other types of functionalities in the future.


"center" should mean Centroid (Center of Mass for shapes with uniform density)

Currently the polyhedron.center property appears to be defined as an average of the vertices. However, this does not always coincide with the shape's center of mass: https://bell0bytes.eu/centroid-convex/
https://github.com/glotzerlab/coxeter/blob/0b02755cb0cf9550f1ed3a94b5c9eeed21d25830/coxeter/shapes/polyhedron.py#L474-L477
I recommend the following:

Clarify and define relevant properties (center = center of mass = centroid = barycenter) [1]
The current behavior of center could be retained under another name, but the proper name for this is unclear to me

[1] I note that Wikipedia says the center of mass, centroid, and barycenter are synonymous for shapes of uniform density:

While in geometry the word barycenter is a synonym for centroid, in astrophysics and astronomy, the barycenter is the center of mass of two or more bodies that orbit each other. In physics, the center of mass is the arithmetic mean of all points weighted by the local density or specific weight. If a physical object has uniform density, its center of mass is the same as the centroid of its shape.

Here are my thoughts

The reason for the Wikipedia statement is that the centroid is the average of all points in the figure, where points does not mean vertices but literally all points. That would mean performing an integral and then dividing by the area, which is the calculation I was referring to in the discussion today when I said that for a constant density the two should coincide. I conveniently forgot that averaging over all points is in no way equal to averaging over the vertices.
I don't think there is any issue with redefining the current behavior of center, although we should probably verify that before deciding whether to change it. In any case I would prefer to define a new property com or center_of_mass and move the existing definition of center to some more appropriate name. The final thing to do would be to decide which one center should be an alias for; I would also lean towards center of mass, but I think that should be assessed more carefully.
Properties like moments of inertia etc should still be calculated relative to the origin of the coordinate system. We have discussed this before, and there are various reasons to prefer that over implicitly using the center of mass. If a user wants properties computed relative to the center of mass, they should set the com to the origin.

If we correctly calculate the center of mass, it should look similar (as an integral) to the inertia tensor computation. The inertia tensor currently isn't implemented for spheroshapes, meaning that we likely don't have a simple method to compute the center of mass, either. Here is a potential reference for computing these quantities for spheropolygons: https://journals.aps.org/pre/abstract/10.1103/PhysRevE.82.056713


How to use Redlock in cluster

Hi @antirez,
At present, many companies use redis cluster to cache data or do other things. If these companies want to use the RedLock algorithm to implement distributed locks, they must create additional and independent redis nodes just for the Redlock. Because the Redlock algorithm assumes the nodes is independent. As quoted below.

In the distributed version of the algorithm we assume we have N Redis masters. Those nodes are totally independent, so we don’t use replication or any other implicit coordination system.

It is a waste to do that and it's better for us to reuse the cluster. This link provides more information about this problem.
Do you have any ideas about it?
Thanks.

I ran into this problem too.
redisson used a funny method to compute the slot by key.In ClusterConnectionManager.java File,
`
public int calcSlot(String key) {
    if (key == null) {
        return 0;
    } else {
        int start = key.indexOf(123);
        int result;
        if (start != -1) {
            result = key.indexOf(125);
            key = key.substring(start + 1, result);
        }

        result = CRC16.crc16(key.getBytes()) % 16384;
        this.log.debug("slot {} for {}", result, key);
        return result;
    }
}

key.indexOf(123) to find character '{' and key.indexOf(125) to find '}'。So we had better not to use {} in the key string. But when we use Redlock in cluster, we can use the code below to ensure the lock is distributed on different redis node(3 master node for example):
@Autowired
RedissonClient clusterClient;

private RedissonRedLock getRedLock(String key){
    RLock lock1 = clusterClient.getLock(key.concat("{lock1}"));
    RLock lock2 = clusterClient.getLock(key.concat("{lock2}"));
    RLock lock3 = clusterClient.getLock(key.concat("{lock3}"));

    return new RedissonRedLock(lock1, lock2, lock3);
}

`
Simple use key.concat("{1}") also will work.Personly, I don't think is a good idea that redisson make a trick in slot compute algorim, but it just happen to help.


chore: upgrade kafka  addon to v1.0 API, refactor scripts and add ut

Codecov Report
Attention: Patch coverage is 0% with 59 lines in your changes missing coverage. Please review.

Project coverage is 0.00%. Comparing base (8ed36fe) to head (002fd95).

Files with missing lines
Patch %
Lines

...kafka/scripts-ut-spec/kafka_exporter_setup_spec.sh
0.00%
59 Missing :warning:

Additional details and impacted files
@@          Coverage Diff          @@
##            main   #1114   +/-   ##
=====================================
  Coverage   0.00%   0.00%
=====================================
  Files         34      35    +1
  Lines       4352    4411   +59
=====================================
- Misses      4352    4411   +59

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


Fold alg into signer property of Issuer

Reviewing this codebase. Really like the concept of an Issuer interface.
Right now it looks like Issuer has three properties did, signer, and alg?. Here alg seems a bit out of place. If this could somehow be folded into the signer property it would make the interface a lot cleaner.
Right now Signer is defined as:
type Signer = (data: string) => Promise<EcdsaSignature | string>

Could it be changed to something like this?
interface Signer {
  (data: string) => Promise<EcdsaSignature | string>;
  alg?: string;
}

The goal of the Issuer interface was to provide a common envelope for these 3 properties that need to match to be able to verify the resulting JWT through did-jwt.
did-jwt resolves the iss field to collect a list of public keys that should be valid signers of a JWT. The signature produced by the signer needs to match one of those keys as well as the algorithm of the header. If we are to change the interface it would introduce a new level of indirection without any real benefit.
BUT, this issue raises the question if the Issuer interface is even needed, or if it should be defined here.
It is essentially a subset of the properties of the JWTOptions interface used in did-jwt so it makes a bit more sense to define the Issuer interface in did-jwt and use it here, or even remove it completely and no longer create confusion.

I like the general idea of the Issuer interface, and I was intrigued by the fact that EthrDID actually implements it. Made me think that it could work as a general interface for interacting with DIDs. Maybe you would consider that out of scope though?
Totally makes sense for this to live in did-jwt.

ethr-did is doing a "drive-by implementation" just because the types match 😅
haha :)

As a developer I'd like to not have to care about the internal of some issuer that I'm using. For example:
const user1: Issuer = new EthrDID(...)

const user2: Issuer = new ThreeId(...)

const jwt1 = await createJWT(payload, user1)

const jwt2 = await createJWT(payload, user2)

Here EthrDID will use ES256K-R while 3ID uses ES256K. It seems like a leaky abstraction if I as a dev need to know this.

Right. I totally agree that basics could be simpler but ultimately it's a key that signs, not a DID.
A DID may express multiple keys in their DID doc.
Also, in the case of ES256K vs -R it's the same key that can produce both signatures so there should be a way to override that.

Right, but all of that can be handled from within the EthrDID, or ThreeId libraries.


computed opinion hash doesn't match signed

very strange, it failed to decode on the first try and succeeded on the second. there should be non-determinism the decoding process

It has the same root cause as https://github.com/spacemeshos/go-spacemesh/issues/3749
Block is not added to the state, when in fact it is expected to be added

Root cause is the same as for https://github.com/spacemeshos/go-spacemesh/issues/3749.
There is a lag between block was generated and added to the tortoise, due to the limited resources that i configured for testing network.
Solution for this particular issue is to add block metadata from the ballot itself when it was seen the first time.


Please, extend skyremote to SkyGlass

Hi,
I notice that this palette works really good with Sky Glass, but some commands are missing.

Volume control (there is no other way to change volume than the sky remote)
Volume mute
External source change (HDMI 1, HDMD2, HDMI3, DVB)

It would be wonderful (I don't know it is possible!) if different command were available for "power_on" and "power_off"...
Here below my remote. On Glass don't exist the pure "red", "green", "yellow"  and "blue" keys, and also "record".
I'm obviously available for testing.
Thank you!

I don't have a chance to check if it works (I'm not so expert), but it seems that there is a way to know if the box is on or off, then send "power_off" or "power_on" command instead the blind toggle "power".
Maybe it works. GIT: dalhundal/sky-q.


Centered item in navbar.

I am trying to figure out how to center an item in the navbar. I would essentially like to be able to have two navs, one left and one right. Then have an element (logo or CSS styled div) in the middle of the nav. It would be nice to have something similar to .pull-right, a .pull-center or something to assist with this. I have been trying to override the css and write it in myself, but for the life of me can't get it to work right.

late into 2015 this still seems to be one of the most popular searches for bootstrap navbars, and there are a myriad of half working solutions out there.  has anything been accepted by this community as an unofficial standard for centering items in a navbar?

@calben Try asking on StackOverflow and perhaps keep an eye on #17524. Thanks!


MSID: 91512 Version: 2 DOI: 10.1101/2<IP_ADDRESS>3933

MSID: 91512
Version: 2
Preprint DOI: https://doi.org/10.1101/2<IP_ADDRESS>3933
Step 1. Awaiting reviews
Editorial to post reviews via hypothesis
Useful links:

DocMap: https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=91512
New model tracking: https://docs.google.com/spreadsheets/d/1_fHaoOy7hjyocptKtVJRijeNpUY4hBS7Ck_aVmx6ZJk/
Reviews on sciety: https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>3933

For trouble shooting (e.g. no Docmaps available):

DocMap issue addressing: https://miro.com/app/board/uXjVNCwK6EI=/
Explore DataHub DocMaps API: https://lookerstudio.google.com/reporting/4c2f0368-babb-4beb-b5b3-497e7e7b0f08/page/ejphD
Unmatched submissions and preprints: https://lookerstudio.google.com/u/0/reporting/9f86204f-3bf7-477c-9b18-5c5ef141bf69/page/p_gxi57ha93c
Unmatched manuscripts spreadsheet: https://docs.google.com/spreadsheets/d/15QcK8w-ssB7109RQEDtFpJPZ0J5HTGxoHa_2TtpMBbg/edit#gid=1336081641

Step 2. Preview reviewed preprint
Production QC content ahead of publication
Instructions:

QC preview: https://prod--epp.elifesciences.org/previews/91512v2
Update ticket with any problems (add blocked label)
When QC OK, add QC OK label to ticket and add publication date and time to https://docs.google.com/spreadsheets/d/1amAlKvdLcaDp5W8Z8g77NmkwbMF5n_u89ArSqPMO8jg
Move card to next column
(At end of the day post link in #enhanced-preprint and ask for PDF to be generated)

Useful links:

Preprint DOI :  https://doi.org/10.1101/2<IP_ADDRESS>3933
Confirm reviews returned by EPP: https://prod--epp.elifesciences.org/api/reviewed-preprints/91512/v2/reviews
To update the MECA path in the docmap: https://docs.google.com/spreadsheets/d/1mctCQuNFBjSn97Lihy7_vBO6z7-N-oqyLv4clyi6zHg

Step 3: Awaiting search reindex
This step adds the reviewed preprint to the homepage: https://elifesciences.org
The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.
Useful links:

Jenkins pipeline to reindex search can be triggered sooner or monitored here: https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 4: Published! PDF requested
Waiting for PDF to be generated
Useful links:

PDF tracking: https://docs.google.com/spreadsheets/d/106_XeDjmuBae7gexOTNzg60lapeqjl2aRn9DzupGyS8/

Step 5: Introduce PDF to data folder and git repo
Upload PDF to relevent folder in git repo https://github.com/elifesciences/enhanced-preprints-data/
Step 6: Done!

[ ] Kettle is on!

elocations missing from references.


How could I set a corner radius to a Polygon mask image view?

I'm using an AnimatableImageView and I set the border width to 15 points, how could I make the corners rounded?

@felixsolorzano currently, the Polygon doesn't corner radius yet.

@felixsolorzano we don't have that feature either yet 😭


Adding a rule to a table row throws an OverflowException

Adding a rule to a column in a row of a table throws an OverflowException during render.
var table = new Table();
table.AddColumn("header");
table.AddRow(new Rule("test"));
console.Write(table); // throws during render

It seems to happen in CalculateColumnWidths:

And the rule is generating an int.MaxValue, which is then making the linq Sum() failing:

Oops, never mind, I found why. It was because I was using an offscreen rendering! (so I setup the max width to int.MaxValue)
Completely my fault. Closing as it is expected. Sorry for the noise.


Update project files to utilize .Net 7

During recent activity with Azure Learn Sandbox involving this repo - I was getting such issue:
`$ dotnet run
You must install or update .NET to run this application.
App: /home/xyz/mslearn-communicate-with-storage-queues/start/bin/Debug/net6/StorageQueueApp
Architecture: x64
Framework: 'Microsoft.NETCore.App', version '6.0.0' (x64)
.NET location: /usr/share/dotnet
The following frameworks were found:
7.0.5 at [/usr/share/dotnet/shared/Microsoft.NETCore.App]
Learn about framework resolution:
https://aka.ms/dotnet/app-launch-failed
To install missing framework, download:
https://aka.ms/dotnet-core-applaunch?framework=Microsoft.NETCore.App&framework_version=6.0.0&arch=x64&rid=mariner.2.0-x64`
and after update of such to .Net 7:
<TargetFramework>net7</TargetFramework>
sample was able to run properly.
Thus proposing bump up to .Net 7 in this PR.

@microsoft-github-policy-service agree

@tpetchel - could you review this PR?


Cannot run the sample-ar-model-viewer-java

I downloaded code from the Github code page, opened the master branch as AndroidStudio project, builded it and tried to run on AndroidStudioEmulator. It throwed the next error:
java.lang.RuntimeException: Unable to start activity ComponentInfo{com.google.ar.sceneform.samples.gltf/com.google.ar.sceneform.samples.gltf.MainActivity}: java.lang.IllegalStateException: Couldn't create Engine

AndroidStudio version:  2021.1.1 Patch 2
AndroidStudioEmulator version: Android 10.0 API 29 x86
Graddle: bundled 211.7628.21.2111.8193401
OS: Windows 10 x64
RAM: 16Gb
VM: enabled
sceneform version: Just downloaded from GitHub
JDK: 1.8
Java: 1.8
ARGooglePlayServices: 1.30.0_x86_for_emulator
Full log: log.txt

What can I do?

Hi!
It looks like OpenGL ES 3 isn't supported by the emulator. Can it be the case on your computer?

https://sceneview.github.io/sceneform-android/emulator

Thank you very much! Setting the advanced render settings to Desktop native GL/Renderer maximum helped.


chore: remove bogus hint in make javascript

+1. If we start getting emfile errors then we'll need to revisit - but I've never seen emfile errors when running make javascript.

Yeah, that was from before we deleted the databases we created in each test.


Slow MP4 video's  are being recorded when low network conditions with both RTMP streaming and recording .mp4 video at same time.

When we RTMP stream and record local mp4 video's at same time . In Low network conditions frames are dropped by mediamuxer when
RTMP pipline is slow due to low network conditions.can u please suggest any solutions for this fix.

Hello,
MediaMuxer never drop frames, only the stream client drop frames if it is necessary. If you implemented the BitrateApdater then you are reducing the bitrate to avoid drop frames and this affect to the local video quality because reduce the bitrate to both, record and stream.

Thanks @pedroSG94 ,Even  we removed BitrateApdater for Video MediaCodec but enabled for stream client.we are absorbing low quality video's in  low network conditions Please help us to make "Video recording and stream client independent of each other".can they process video frames parallel not sequential.

Hello,
I added a feature to support record and stream with different quality. This is available with StreamBase and Camera2Base.
This way you can use BitrateAdapter without affect to record and keep constant the video quuality. The only difference is that this feature require use a different resolution in stream and local record. Example:
genericStream.prepareVideo(width, height, vBitrate, rotation = rotation, recordWidth = recordWidth, recordHeight = recordHeight, recordBitrate = recordBitrate)

Requirements to work:

Stream and record resolution must be different.
Stream and record resolution must use the same aspect ratio
Only Available with Camera2Base and StreamBase classes

This feature is not released yet but you can use this gradle meanwhile:
  implementation 'com.github.pedroSG94.RootEncoder:library:b51a9aa081'

Thanks @pedroSG94  for your time. Is there any feature that  support's  segmented(15minutes) video file while recording.


is there a way to use this on my own network without having to port ford if so please tell me

I have set up the build on how i want, i tested it on my own pc and it works so i uninstalled it from my pc, the next day i try use it on another pc i got (windows laptop and it has no anti virus installed) and it gets installed in the write places but when i go to see if there are any connections there are none.

If you're using it on your LAN, get your local IPV4 address and input it into Quasar's host

Where do I find that?

Is the PC on your local network? Can the PC ping your local IP address?

Yeah I think I've found it and fixed it I'll have to try lafter thow doing stuff thanks :)


Searching While Awaiting Geolocation Feature

As I a user viewing the application for the first time I should have some visual notification that I have either denied geolocation, or that my geolocation is currently in process. Along with this visual notification I should also be allowed to use the search effectively. Currently if you have blocked geolocation or in the process of getting your current position the search is broken and is throwing errors due to userPosition not being set in the home route state.

@someyoungideas great summary of the issue - I'll assign to you if you want to work on this issue.
Here's a useful reference doc for navigation.geolocation: https://developer.mozilla.org/en-US/docs/Web/API/Geolocation/Using_geolocation

@jkwening I would be glad to take this issue. I may have some questions along the way. Would you like me to just post them in here and mention you as they come up, would there be a more preferred place?

@someyoungideas assigned to you. Yeah, just post in here and mention me as they come up. Thanks!


app/config.yml is reset after after upgrade

As reported by @mickaelandrieu:

Every time a shop is upgraded, the file app/config.yml is reset and their bundle configuration is lost. We need to document where people can keep their settings without risks.

Isn't the option "Keep the theme files" helping ?


Add parallel deployment to the new cluster

What does this pull request do?
Adds parallel deployment to the new cluster and a load balancer between them.
This is controlled by "blue" and "green" weights in the values-{env}.yaml

🔵 "blue": goes to the current cluster
🟢 "green": goes to the new cluster

Background work

Had to upgrade the Ingress resource to apiVersion: networking.k8s.io/v1 (f369a29dcc8f9328f0c727fd8f882a1743dcdaa7)
Manually created a hmpps-interventions-dev-deploy CircleCI context to override the KUBE_* environment variables for the deployment.
Manually copied all unprovisioned secrets over to the new cluster that blocked boot.

What is the intent behind these changes?
We need to migrate over to the new cluster this year.

 ❗Need to migrate dev/research env to EKS cluster in Cloud Platform

Testing by deploying to both environments with

[ ] To the old:
helm upgrade --debug --install --reset-values --history-max 10 --namespace=hmpps-interventions-dev --set ingress.migration.colour=blue --set image.tag=2021-11-29.23934.4a8704d hmpps-interventions-service hmpps-interventions-service --values=values-dev.yaml

[x] To the new:
helm upgrade --debug --install --reset-values --history-max 10 --namespace=hmpps-interventions-dev --set ingress.migration.colour=green --set image.tag=2021-11-29.23934.4a8704d hmpps-interventions-service hmpps-interventions-service --values=values-dev.yaml

Well, apparently the v1 Ingress doesn't work on the old cluster


Cannot block indent after selecting text from end to beginning

Description of the problem or steps to reproduce
If I select text by clicking and dragging right/down, I can then press tab to indent the entire block, or shift+tab to unindent the entire block.
If, however, I select text by clicking at the end of the text I want to select, and dragging up/left, pressing tab and shift-tab does nothing.
Specifications
Commit hash: f7560c3
OS: Archlinux, Ubuntu 16.04
Terminal: Xfce4-terminal, Gnome-terminal

Thank you for the speedy fix! Micro keeps getting better and better :)


update syn + quote

I'm not sure on lots of this, but the tests pass. I had multiple versions of syn in one of my projects because of diesel. Figured I'd just update it. It was more involved than I expected.

General workflow question: do you prefer "fixup" commits and then squash before merging, or should I just amend this commit?

Separate commits during code review. I will often request a squash after everything looks good

Thanks for working on this. I have to go make dinner, but I will finish reviewing tomorrow

@killercup feel free to merge if you think it's good to go
On Mon, Dec 5, 2016 at 5:37 AM Pascal Hertleif<EMAIL_ADDRESS>wrote:

@killercup approved this pull request.
Great! I'm fine with merging this as-is, but also left a few more inline
comments with really tiny stuff.
@sgrif https://github.com/sgrif wanted to review this as well. I don't
know if he finished dinner already, though. Apparently, there was a huge
amount of cheese involved, so I don't want to rush him ;)
In diesel_codegen/src/schema_inference.rs
https://github.com/diesel-rs/diesel/pull/504#pullrequestreview-11349379:

let connection = establish_connection(database_

url).unwrap();

let data = get_table_data(&connection, table_name).unwrap();
let primary_keys = get_primary_keys(&connection, table_name).unwrap()

let connection = establish_connection(&database_url).unwrap();
let data = get_table_data(&connection, &table_name).unwrap();
let primary_keys = get_primary_keys(&connection, &table_name).unwrap()

Just for a fun exercise: I'm pretty sure the &s you added here are no
longer necessary – the bindings are already references. (Deref coercion
allows rustc to accept &&&foo where &foo was wanted.)
In diesel_codegen/src/embed_migrations.rs
https://github.com/diesel-rs/diesel/pull/504#pullrequestreview-11349379:

@@ -15,7 +15,7 @@ pub fn derive_embed_migrations(input: syn::MacroInput) -> quote::Tokens {
}

 let options = get_options_from_input(&input.attrs, bug);

let migrations_path_opt = options.map(|o| get_option(&o, "migrations_path", bug));

let migrations_path_opt = options.as_ref().map(|o| get_option(&o, "migrations_path", bug));

Nice. I think you can just give it o instead of &o here (same as other
comment).
In diesel_codegen/src/ast_builder.rs
https://github.com/diesel-rs/diesel/pull/504#pullrequestreview-11349379:

@@ -11,6 +11,6 @@ pub fn ty_path(path: Path) -> Ty {
pub fn path_ident(ident: Ident) -> Path {
Path {
global: false,

   segments: vec![PathSegment::ident(ident)],

   segments: vec![PathSegment::from(ident)],

}

FYI this whole function can be replaced with ident.into()
In diesel_codegen/src/schema_inference.rs
https://github.com/diesel-rs/diesel/pull/504#pullrequestreview-11349379:

@@ -62,7 +62,7 @@ fn column_def_tokens(
let path_segments = column_type.path
.into_iter()
.map(syn::Ident::new)

   .map(syn::PathSegment::ident)

   .map(syn::PathSegment::from)

I added a bunch of From impls to syn a while back. I don't think we need
the .map(syn::Ident::new) here anymore.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/diesel-rs/diesel/pull/504#pullrequestreview-11349379,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABdWK0kN4kOiW0-cCs-il-1lNycE0PyTks5rE-ldgaJpZM4LBGAM
.

@nerdrew, can you also have a look at the conflict and the CI failure? If you don't have time, that's okay, I can do it tomorrow.
Travis says:
error: unused variable: `lifetimes`
  --> /home/travis/build/diesel-rs/diesel/diesel_codegen/src/identifiable.rs:10:9
   |
10 |     let lifetimes = model.generics.lifetimes;
   |         ^^^^^^^^^
   |

which surprises me because

it's not a bug in cargo/rustc (yay, CI works again!),
this line is not in identifiable.rs:10 but in insertable.rs:21 (maybe proc macros messing up the source of the error),
lifetimes is passed to the _Insertable! as (#(#lifetimes),*),
and it compiles fine locally with my nightly from 2016-09-28.

You can find the code in its current form here. Maybe adding #[allow(unused_assignments)] (or similar, with a comment) as a temporary solution is fine.

I think the reason it's unused is that this branch was made before we had lifetimes there, but the file merged fine, so the "usage" is using the wrong syntax and therefore not actually being used.

@sgrif ya, those "used variable" errors are caused by quote changes. The errors are real. Fixing now.

Thanks!


exp/ingest: Bucket stream not restarted in case of PROTOCOL_ERROR

What version are you using?
release-horizon-v0.24.0 branch.
What did you do?
Trying to ingest state.
What did you expect to see?
Successful retry of bucket stream.
What did you see instead?
When XdrStream.ReadOne in SingleLedgerStateReader.readBucketEntry returns an error like:
net/http.http2StreamError {StreamID: 75, Code: http2ErrCodeProtocol (1), Cause: error nil}

calling XdrStream.CloseWithoutValidation returns the same error and the code that should attempt to retry streaming is not run and the pipeline returns:
Pipeline processing error: Processor RootProcessor errored: Error while reading from buckets: Error on XDR record 99999 of hash '113d2be8882161621a76
39089b360544401c8afb34721fcfefdf699bbfc40be3': Error closing stream: stream error: stream ID 75; PROTOCOL_ERROR

It looks like for PROTOCOL_ERROR errors we may need to ignore errors returned by CloseWithoutValidation. Possibly something like: https://github.com/snapcore/snapd/pull/7231 (found this linked https://github.com/golang/go/issues/29125).

Fixed in #1984.