VS Code Plugin Compatibility issue with Glimmer Templates Syntax for VS Code

OS: macOS Monterey 12.5.1
Editor: VSCode 1.71.0
Plugins:

Glint v0.9.1
Glimmer Templates Syntax for VS Code v1.0.7

Context
I have an ember-cli-typescript project that is consuming a glint enabled addon.
Expected
When trying to use the addon, I'm looking for completion suggestions from the Glint plugin (ex: <my-component @[...expect completion suggestions here]></my-component>).
What's happening
Instead I'm getting suggestions from history and other plugins but nothing from Glint.
Disabling the Glimmer Templates Syntax plugin allows glint to return completion suggestions.
Related
I also opened an issue in the repo of Glimmer Templates Syntax for VS Code: issue

Thanks for reporting this. It's extremely unclear to me why this would be happening, but I appreciate the report!

After doing a little poking, I strongly suspect this is because of a bad interaction between the language server supplied by vsc-ember-syntax and the Glint language server. I don't know whose "fault" that is yet, but it would definitely be preferable from my POV if vsc-ember-syntax focused on syntax highlighting and had a separate package to supply the other HBS functionality it supplies (which is fairly minimal; it no doubt made sense to include initially!).

I can confirm that Glint is still receiving the completion request and responding correctly to it. My best guess is that Glimmer Templates Syntax for VS Code is doing something to cause those suggestions to be ignored or preempted in the UI.
It may be that it's changing the token structure somehow such that it doesn't the editor doesn't think any of our results match the prefix there? Regardless, I don't think there's anything we're likely to be able to do from this end.

[Trace - 6:39:57 PM] Sending request 'textDocument/completion - (10)'.
Params: {
    "textDocument": {
        "uri": "file:///Users/dfreeman/code/glint/test-packages/ts-ember-app/app/components/ember-component.hbs"
    },
    "position": {
        "line": 9,
        "character": 7
    },
    "context": {
        "triggerKind": 1
    }
}

[Trace - 6:39:57 PM] Received response 'textDocument/completion - (10)' in 6ms.
Result: [
    {
        "label": "hasDefault",
        "kind": 5,
        "data": {
            "uri": "file:///Users/dfreeman/code/glint/test-packages/ts-ember-app/app/components/ember-component.hbs",
            "transformedFileName": "/Users/dfreeman/code/glint/test-packages/ts-ember-app/app/components/ember-component.ts",
            "transformedOffset": 1562
        }
    },
    {
        "label": "optional",
        "kind": 5,
        "data": {
            "uri": "file:///Users/dfreeman/code/glint/test-packages/ts-ember-app/app/components/ember-component.hbs",
            "transformedFileName": "/Users/dfreeman/code/glint/test-packages/ts-ember-app/app/components/ember-component.ts",
            "transformedOffset": 1562
        }
    },
    {
        "label": "required",
        "kind": 5,
        "data": {
            "uri": "file:///Users/dfreeman/code/glint/test-packages/ts-ember-app/app/components/ember-component.hbs",
            "transformedFileName": "/Users/dfreeman/code/glint/test-packages/ts-ember-app/app/components/ember-component.ts",
            "transformedOffset": 1562
        }
    }
]

Really strange, especially for hbs case, may be related to this, logic, it's calling https://github.com/lifeart/typescript-hbs-plugin and it's getting completions from here https://github.com/lifeart/typescript-hbs-plugin/blob/main/src/html-template-language-service.ts#L83

seems like vscode issue, because it's should collect all completions from extensions, instead of picking..

At least in the general case, I know Code does the right thing, because when you have the built-in TS tooling and Glint both active in the same project, you'll see completions from both of them intermingled.


Add AlmaLinux 8 and 9 to DistributionInfo.json

Creating a draft PR to include AlmaLinux 8 and 9 packages in DistributionInfo.json.
Packages have been uploaded to the Store.
TODO:
Amd64PackageUrl and Arm64PackageUrl need to be populated by publicwsldistros... URL
PackageFamilyName needs to be populated
CC @craigloewen-msft

what is holding this up?

@sirredbeard I appreciate the suspense, but really, why couldn't this be merged? it's just a json declaration pointing to the Windows Store app....
I'd like to avoid manual steps when setting up my AlmaLinux 9 WSL2 distro.

@namtab00 I understand your frustration. I apologize if my response came across as flippant. I promise there will more details I can share on this very shortly.


Fix add jindocache engine for jindodata 5.0 beta release

Ⅰ. Describe what this PR does
Ⅱ. Does this pull request fix one issue?

fixes #XXXX
Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

@frankleaf please fix the unit test first. thanks.

/test fluid-e2e

/test fluid-e2e

/retest

/test fluid-e2e


allow use with gpgme

The current version of gpgme 1.7 has a bunch of new features specifically to work with GnuPG 2.1. See for example the new keygen functions.
I would like to use these features but currently hombrew does not allow installing gnupg21 alongside gpgme.

Do they both brew link side-by-side without problem?

Yeah I think so, no trouble so far.

@jeroenooms I'm not sure how you got this to work given gpgme depends on gnupg2 and there's a conflicts_with that too...

I did something like this:
brew install gpgme
brew remove gpg-agent dirmngr gnupg2
brew install --force gnupg21

Note that recent versions of Debian/Ubuntu have no fully switched to GnuPG 2.1 in combination with GPGME 1.7 as well, if that is any precedent.

@jeroenooms That is good precedent, thanks. I think we'll pass on this PR as-is as it requires too many workarounds but the best next step would be to submit one to upgrade to 2.1 in homebrew/core. Thanks!

Do you recommend to move gnupg21 to homebrew-core, or simply bump gnupg2 to version 2.1?

Let's try the latter.


Unit tests are needed

Looking for some extra help to cover this library with some xunit tests.

:tada: This issue has been resolved in version 1.1.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Custom commands for Debug, Run, and Plot

Dear All,
Is there any way to customise the commands for Debug, Run, and Plot? I think that it would be great to have some sort of customisation. For example, I use specific plotting functions according to the data that I want to inspect. I would like to have the option to use them from the variable explorer drop down menu.
Versions and main components

Spyder Version: 2.3.9
Python Version: 2.7.11+
Operating system: Ubuntu 16.04LTS

Regards,
Felipe

There is no way to do this at the moment, but if this is important to you, you can come up with a pull request to fix it :-)

Hi Carlos,
Thank you. I will try to come out with something.
Thanks,
Felipe


push notifications

Is it possible to get messages through push notifications (using .net)?
We have a project where someone implemented a telegram interface using this lib, but he does polling to get the latest messages.

See https://core.telegram.org/tdlib/getting-started, especially section about updates handling and updateNewMessage.

I went through it, but this doesn't help me to know if the updateNewMessage event is triggered through a message received by a push notification, or is the result of polling inside the lib.
The app in question uses Telegram to pass trading signals so latency is very important which is why I am curious about the internal implementation.

There is no such thing as "polling" in Telegram. TCP connections are bidirectional, so most updates are "pushed" by the server if client has an active network connection.

ok, thanks; I have to understand what he did; he used TLSharp and has somewhere a polling interval of 300ms. I have no idea what he did exactly, but I'll investigate.


Data quality issue with CVE-2020-27388

The CVE ID
https://osv.dev/vulnerability/CVE-2020-27388
Describe the data quality issue observed
The CVE says it affects versions 1.5 to 1.7.10. OSV says all versions released since 1.5 (including the latest, 1.9.2) are affected. osv-scanner reports this vulnerability when commit 2efcb9e (1.9.2) is checked out, even though it was fixed in 04495e8.
Suggested changes to record
Set "Fixed" commit ID to 04495e8b17ffeedb695ef5a64389f6bee15d57fb.
Additional context
GHSA-pwgg-r6fq-mf94 is listed as an alias. That seems to have the correct Introduced and Fixed version numbers - although only the 1.7.x versions are actually listed under "Affected versions".

This is another instance of a formerly converting CVE that is no longer converting and will be deleted entirely when https://github.com/google/osv.dev/pull/2030 is complete.

"Deleted entirely" was the incorrect expectation to set, my apologies, this is now marked as withdrawn.


web: don't return federation errors over HTTP

We are writing federation responses streaming. So after
the first byte we wrote, the status header is fixed. We cannot
return an HTTP error for intermediate error but should just abort
and log instead.
Related to #2237

We should add a metric for this too

👍


Clean upgrade path for Medplum App

Move infra.init into Medplum CLI - then this is ready for bug bash.
This one is high priority for great write up and comms.


add new properties

https://issues.cask.co/browse/CDAP-11531
New Properties
hbase.version.resolution.strategy
hive.version.resolution.strategy
Modified description
security.authorization.cache.ttl.secs

👍 lgtm


Improve the compile_guide.md file, indicating the contents of the board.cfg configuration file needs to be modified

In GitLab by @liyanq528 on Dec 12, 2017, 10:10

In GitLab by @yhua123 on Nov 8, 2019, 07:43
in install_guide.md
http://<IP_ADDRESS>:10080/inspursoft/board/src/dev/docs/installation_guide.md

In GitLab by @sokril on Nov 8, 2019, 08:16
closed


Week 5 Review Ticket

Assignment and Link
Expectations Met
Hacks

"Notes" _page (Contains Javascript notes)
Creating a new "_pages" screen with a submenu which links to a Notebooks post using Javascript kernels
N/A (None mentioned)

Javascript Notes
Using Javascript kernels in Jupyter Notebooks post with output, customized HTML table formed based on Javascript data
N/A (None mentioned in "Javascript Tutorial")

Customized Python Server Screenshot
Adds custom elements to my home page through HTML and CSS
N/A

"Work Watch" Information
Defines team roles, outlines program purpose, acts as one of the two final project screens, changes the stub page, screenshot of second project screen for general viewers
Roles and schedule diagram hack mentioned in "Agile Methodology"

SCRUM Board
N/A
SCRUM Board hack mentioned in "Agile Methodology"

2.7+
Great job with the team and individual personalization, as well as going above and beyond on javascript

Group "The Watchers".  Nice diagram on Scrum.  Shared an Issue on weekly plan.   Made a "Work Watch" slide which had lots of ideas about taking breaks.  Had many elements on features.   Timer is a big focus.  No real visual on what the timer looks like but lots of ideas.
Drew - did not really have a submenu.  Simply links.  Did a lot of commenting in JavaScript, small adaptations.   Customized cards on Flask.
Lots of thought on team work.
2.7

2.7, very clear with the group work, lots of good planning. Fulfilled all the requirements as well.


adjust dotenvx help to display [options] following positional [command(s)]

touch .env.a
dotenvx set -f .env.a key value

Observed
error: missing required argument 'KEY'
Expected
.env.a shall contain an entry key=value
Additional
oddly enough this works (no space between -f and .env.a)
dotenvx set -f.env.a key value

dotenvx version:1.21.0

this is a parsing ambiguity when there are both positional arguments KEY value and flags -f. you have a few options but best practice is place flags at the end.
dotenvx set key value -f .env.a

option 2: full flag name with =
dotenvx set --env-file=.env key value

option 3: instruct end of options with --
dotenvx set -f .env -- key value

option 4: no space (your example)
dotenvx set -f.env key value

i considered having all instructions use -- but I didn't like the verbosity just for this rare edge case. i think most users just place their flags at the end when they experience this.
thanks for bringing up but will leave as is. closing this.

Two thougts:
Upate help
dotenvx set -h currently states:
Usage: dotenvx set [options] <KEY> <value>

Which does not hint to your mentioned solutions
Promot the -f flag
Seems to me, that the -f flag could "global", i.e. dotenvx -f <file(s)> set/get/decrypt/... instead of being a flag to each command. Most, if not all dotenvx commands support this flag.

https://github.com/dotenvx/dotenvx/pull/429
will go out as next release. great attention to detail. thank you!


Proper rendering of credentials

Feature description
As a user of the module we want proper render of credentials, without using vault provider
Why is it needed?
To make it easier to use the module.
Suggestion(s)/solution(s) [Optional]
Done in Presto module: https://github.com/fredrikhgrelland/terraform-nomad-presto/blob/master/conf/nomad/presto.hcl#L327
Definition of done

Rendering credentials in .hcl file

Checklist (after created issue)

[x] Added label(s)
[x] Added to project
[x] Added to milestone

@dangernil Wrong feature description -> As a user of the module we want proper render of credentials, without using vault provider
Take a look at #22


added dev only env variable for cypress testing and cypress command for login tests

Description:
Created a an environment variable that checks for production or dev environments for exposing testnet store credentials for cypress.
Created cypress command tool for logins
This PR modifies ... in order to support ...
vite.config.js in order to support dev env testing
router.ts in order to support dev env testing
cypress in order to support automated login command
Related issue(s):
Fixes #
Notes for reviewer:
Now we can run tests within the dev environment without having to worry about production leaks! Q('.'Q)
Checklist
-[x] Documented (Code comments, README, etc.)
-[x] Tested (unit, integration, etc.)

scooped


CoE Upgrades

Hi I am just wondering since the CoE team is constantly working to improve the CoE starter kit and always making updates, is there a place where we can find a list of all the updates that have been made so we can just import the specific components that have been updated or do we have to import the solutions from scratch every time there is an update? Is there a way to get upgraded as simply as possible? Thanks in advance!

Hi there,
Somebody knows how can i set the environment variables again? Thanks in advance
Sorry if this is not the right way to ask.
Bryan

Hi there. Sorry we never commented here to let you know, but we added a work item on our side to make understanding the content in new updates easier.

Hello, please see the release notes section for CoE Releases in the future.
Thanks for helping make CoE great!


Add list admins and remove admins because it does not make sence to always go to DB to do it

Add 3 slash commands for:

adminadd
adminremove
adminlist


Fix bug: Incorrect horizontal padding of code block

When windows width varies, the horizontal padding of the code block doesn't look right.
Align code block with other paragraphs

Thanks!


turning off

Would you be able to  add turn off features as well?
Cheers

Unless Microsoft has released more information about their API, no :-(
I do tend to look periodically to see if someone has figured it out, but for now o think we’re stuck.


Improve error message for getDigestFromBuildOutput

ERROR: getDigestFromBuildOutput: Could not find image digest in docker output is not useful.
In this case, we should print more of the build output so the user can include that, or even have a chance. (in this case, it looks like it was related to exhausting space on docker VM)

From Nick:

it's not even clear to me that it's buildkit related. it really needs repro steps

coincidentally, someone else exhausted their space in the docker VM yesterday in the #tilt channel and the error reported as
ERROR: ImageBuild: failed to create temp dir: mkdir /var/lib/docker/tmp/buildkit-mount737751980: no space left on device

@chuckha just ran in to the error message and it was similarly hard to debug in snapshots.
https://alerts.tilt.dev/snapshot/AeKRrNULf1HhuihId34=

OK, I was able to get a repro case for this.

Reset D4M with the smallest possible disk space
enter the VM with the command
docker run -it --privileged --pid=host debian nsenter -t 1 -m -u -n -i sh
create files under /var/lib until you run out of disk space
dd if=/dev/zero of=/var/lib/file.txt count=300000 bs=1024
On the host machine repo, create some large 100MB file in the docker context
Run tilt up

Often I needed to play around a bit to get just the right amount of disk space so that I got the errors reliably without crashing the kubernetes api server.
One thing that I found helped is to create a large file (O(many GB)) and a small file (O(a few hundred MB)), then delete the small file when i needed to reset kubernetes (rather than resetting the whole VM)
There's no actual error message coming from Docker. The response Reader is empty. It looks like it's failing while trying to process the tarball. It tries to stream the tarball to disk somewhere, fails, and just falls over without sending an intelligible error.

OK, this is definitely a docker bug. Here's a repro case outside of Tilt:

Create a large tarball suitable as a docker context, > 100MB
enter the VM with the command
docker run -it --privileged --pid=host debian nsenter -t 1 -m -u -n -i sh
create files under /var/lib until you run out of disk space
dd if=/dev/zero of=/var/lib/file.txt count=300000 bs=1024
Run

DOCKER_BUILDKIT=1 docker build - < archive.tar

Expected result:
Docker should fail with an error message
Actual result:
Docker hangs indefinitely with no reasonable error message. Note that this works correctly with the old build engine:
$  docker build - < ../archive.tar
Error response from daemon: mkdir /var/lib/docker/tmp/docker-builder312255349: no space left on device

Next steps:

File this upstream against buildkit
Try to detect this case in Tilt, and fallback to the legacy build engine

filed upstream as https://github.com/moby/buildkit/issues/1138

I'm also getting the Error: notify.Add("/home/awander/go/src/github.com/chuckha/capi-dev/cluster-api"): watcher.Add("/home/awander/go/src/github.com/chuckha/capi-dev/cluster-api/cmd"): no space left on device
I'm using @chuckha repo as well.
I'm on Ubuntu 18.04 and have over 300GB free disk space. The error occurs after about 5-6 retries of the tilt up command. I tried most of the suggestions here: https://forums.docker.com/t/docker-no-space-left-on-device/69205
The only fix I have is to restart my machine :(

@Arvinderpal have you tried deleting docker images/running pods (with -v) and pruning the docker system?

@Arvinderpal I think that's a different problem. I created issue #2079 for it.

@Arvinderpal can you paste the output of this command?
$ cat /etc/sysctl.conf | grep "fs.inotify"

@chuckha I will try it next time it happens.
@jazzdan I will share the output once I see the problem again.

The upstream Docker bug is fixed, but won't be out until the next Docker release. I have some ideas on how to detect this is happening.
Ya, the watcher error is unrelated, let's move discussion on that over to the other issue.

added a better error message for this


The timezone of timestamp doesn't match local one

I am trying to use this mixin to create timestamp for model, but The timezone of date doesn't match my local one.
Here is the simplified loopback app to reproduce the error:
https://github.com/jannyHou/loopback-sandbox/tree/simplify-app

npm install
open explorer
use POST/ Carts to create an instance of cart model, leave createdAt and updatedAt blank
use GET/ Carts to check the created instance, the timezone doesn't match local one.

My local timezone is UTC-05:00, but the createdAt and updatedAt I got are always 4 hours ahead.
Thank you in advance if anyone could take a look:)

Interesting, the code uses new Date() https://github.com/clarkbw/loopback-ds-timestamp-mixin/blob/master/es6/time-stamp.js#L26
Which can be used as a local or UTC date object, likely depends on the database juggler at that point.  I can dig around a little bit to see.

From a quick look I'd say https://github.com/strongloop/loopback-datasource-juggler passes this off to the connector to handle.  From the code it looks like the mysql connector is passing on the timezone information to the mysql module.  But that's all just a red herring.
Digging into the mysql module as pointed out in strongloop/loopback#2143 it looks like this is just how mysql works.  See https://github.com/felixge/node-mysql/issues/892#issuecomment-62322848 for more background.
The dateStrings solution seems to be the correct way to get the TZ you want without a lot of headache.  All the dates inside mysql are of the correct type and timestamp so you can still query and sort on them.  However when they return they won't be in the timezone you desire until you mess with them.


a little confused about the location the classes generated

I just found that the inner .class generated by enjarify is not in its original dir? is it possible to just generate it in the default dir? or could you please give me some explaination so that I could do it by myself since I am also using apktool(It works like that.)for example
here is the enjarify result
Mode                LastWriteTime         Length Name
----                -------------         ------ ----
d-----         2017/7/5     11:10                air
d-----         2017/7/5     11:10                atast
-a----         2017/7/5     11:10            452 AppEntry$1.class

where the following is the result of apktool in the dir atast
Mode                LastWriteTime         Length Name
----                -------------         ------ ----
-a----         2017/7/5     11:10           1366 AppEntry$1.smali
-a----         2017/7/5     11:10           1124 AppEntry$2.smali
-a----         2017/7/5     11:10           1094 AppEntry$3.smali
-a----         2017/7/5     11:10           2318 AppEntry$4.smali
-a----         2017/7/5     11:10         110804 AppEntry.smali
-a----         2017/7/5     11:10            668 GetVersionCode.smali
-a----         2017/7/5     11:10            509 R$attr.smali
-a----         2017/7/5     11:10            586 R$drawable.smali
-a----         2017/7/5     11:10            694 R$raw.smali
-a----         2017/7/5     11:10            758 R$string.smali
-a----         2017/7/5     11:10            590 R$style.smali
-a----         2017/7/5     11:10            549 R.smali

thanks a lot.

Could you explain what you mean please? Enjarify outputs a jar file. Also, can you provide an apk that demonstrates the behavior you are talking about?

I am so sorry for that, I just found that the classes in the dir
Mode                LastWriteTime         Length Name
----                -------------         ------ ----
d-----         2017/7/5     11:10                air
d-----         2017/7/5     11:10                atast
-a----         2017/7/5     11:10            452 AppEntry$1.class

may be were generated by the tools I used. I am now working on it to solve the problem.
And by the way, will this project add some doc about the code so that others can read the code easily or even make some contribution.
Thanks a lot.

will this project add some doc about the code so that others can read the code easily or even make some contribution.

What kind of documentation do you want?
At a high level, dex parsing code is in parsedex.py, dalvik.py, and dalvikformats.py. Everything classfile related is in jvm/ and typeinference/. Type inference is in typeinference.py, scalartypes.py and arraytypes.py. The bytecode optimization passes are in the optimization/ folder. The core of the bytecode translation logic is in writeir.py.
Also, note that the current type inference system is a hack which doesn't handle a few obscure cases and which is incompatible with Java 8, so if I had infinite time and motivation, I'd like to rewrite it. But it's not worth the effort, especially when I'm not sure how many people are actually using Enjarify.

Thanks a lot. Maybe these are already enough. And I think those people who are doing research about Android reverse engineering may indeed need Enjarify rather than dex2jar which is not better compared with the former as you said.
By using Enjarify and jadx, people can view the java code. JEB also can do similar things, but most people can not afford the cost.
So I think there still exists a lots of people who need it.


Deploying model.bim 1400 - 1520 to Power BI Premium (per User) gives error

Describe the bug
I want to deploy a model.bim that came from SQL Server 2017 (compatibility level 1400).
I modified the model.bim file, changing the compatibility level info from 1400 to 1520.
(Just like in this blog post:  https://data-marc.com/2020/06/16/migrate-analysis-services-models-to-power-bi-using-tabular-editor/).
When I deploy to Power BI online, after a while I get this error:

Error occured during deployment
Internal server error from PowerBI service.
Technical Details:
RootActivityId: e48b246d-2ee2-4595-968a-f16a03e4fa5c
Date (UTC): 3/4/2021 8:09:41 PM

To Reproduce
Steps to reproduce the behavior: The exact steps from:
https://data-marc.com/2020/06/16/migrate-analysis-services-models-to-power-bi-using-tabular-editor/
Expected behavior
I would expect a succesful deployment
Observed behavior
Error:

Error occured during deployment
Internal server error from PowerBI service.
Technical Details:
RootActivityId: e48b246d-2ee2-4595-968a-f16a03e4fa5c
Date (UTC): 3/4/2021 8:09:41 PM

Application specifics

Tabular Editor version: 2.14.2
Semantic Engine used: ?
Model Compatibility Level: from 1400 to 1520

Additional context
Thanks!

Instead of deploying directly using Tabular Editor, could you try to export the TMSL from the Deployment Wizard, and then execute the TMSL manually against your PBI Premium workspace, through SQL Server Management Studio? That should give you a more detailed error message.
Other than that, it's difficult to answer why this doesn't work in your case. A few things to check, since there are still a couple of AS modelling features which are not yet supported by Power BI Premium:

Does your model use DetailRowsExpressions?
Does your model use Legacy or Power Query data sources?
If you create a simple Model.bim file with just one or two tables, and follow the same blog post for migrating to PBI, does it work then?

Hi Daniel,
Thanks for your quick answer.
My model does not use DetailRowsExpressions.
I use a Legacy datasource.
I will check if a simple model.bim file works.
I tried it later with the Power BI ALM toolkit. That worked.
I will also see what message the script gives.
I am also willing to try if TE3 has the same issue 😊
Best regards,
Edgar Walther
ITsmart
www.itsmart.nlhttp://www.itsmart.nl/
<EMAIL_ADDRESS><EMAIL_ADDRESS>https://nl.linkedin.com/in/edgarwalther<EMAIL_ADDRESS>https://twitter.com/edgarwalther<EMAIL_ADDRESS>http://paper.li/edgarwalther/1421271985
From: Daniel Otykier<EMAIL_ADDRESS>Sent: vrijdag 5 maart 2021 09:08
To: otykier/TabularEditor<EMAIL_ADDRESS>Cc: Edgar Walther (ITsmart<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [otykier/TabularEditor] Deploying model.bim 1400 - 1520 to Power BI Premium (per User) gives error (#790)
Instead of deploying directly using Tabular Editor, could you try to export the TMSL from the Deployment Wizard, and then execute the TMSL manually against your PBI Premium workspace, through SQL Server Management Studio? That should give you a more detailed error message.
Other than that, it's difficult to answer why this doesn't work in your case. A few things to check, since there are still a couple of AS modelling features which are not yet supported by Power BI Premium:

Does your model use DetailRowsExpressions?
Does your model use Legacy or Power Query data sources?
If you create a simple Model.bim file with just one or two tables, and follow the same blog post for migrating to PBI, does it work then?

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/otykier/TabularEditor/issues/790#issuecomment-791240380, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AD47ZZ4D6DJGMOJXD5FJXTLTCCGNPANCNFSM4YT5FKCA.

For what it's worth: I've gotten the same error message when deploying to workspaces where "Large dataset storage format" has been set to default. When changing back to "Small..." the deployment works.
This has also been the case with PBI Desktop, so maybe it is an MS problem and not a TE issue at all?

Could you please give it another try with TE 2.15.0 which has the most recently updated TOM libraries?

Using TE 2.15.1 deploying to a PPU workspace with "Large dataset storage format":

Internal server error.

Got the exact same error message when I tried deploying to a brand new "Large..." workspace on Premium Capacity as well. This is a level 1540 model by the way.

@eybert is this only a problem when deploying to workspaces that use the "Large dataset storage format"?

I created two new workspaces just now, one on Premium Capacity, one on PPU.
Compatability: 1540.
Worked fine on Premium, but failed on PPU:

Power BI actually showed this familiar popup:

But the model wasn't there.

Created a new, tiny one table model with 100 rows.
Latest version of PBI Desktop, 1540 compatability.
Published successfully to both a P1 and a PPU workspace.
Opened the model from External Tools -> TE.
Tried publishing to a new database name in both workspaces. Failed in both instances with this message:

Interesting! The error message indicates that something is attempting to change the Culture property on the model during deployment. What is the value of this property when you get the error?

What happens if you save the model as a .bim file using Tabular Editor, then load that .bim file in Tabular Editor, modify the Culture property to "en-US" and then do a deployment?

Culture was: nb-NO - Norwegian, Bokmål (Norway)
Changed to: en-US - English (United States)
Workspace:
"ZZZ P1 small": Success
"ZZZ PPU small": Success

Thanks for testing this. Will inform the product team, that PPU deployment through XMLA endpoint fails when culture is nb-NO (possibly others too).

Hi Daniel,
That worked!
Many thanks for your help.
My offer to test TE3 still stands. I am currently involved in several Tabular Projects, so, we’ll be thoroughly testing the new product.
Cheers,
Edgar Walther
ITsmart
www.itsmart.nlhttp://www.itsmart.nl/
@.***D72021.7F329530]
@.***D72021.7F329530] https://nl.linkedin.com/in/edgarwalther @.***D72021.7F329530] https://twitter.com/edgarwalther @.***D72021.7F329530] http://paper.li/edgarwalther/1421271985
From: Daniel Otykier @.>
Sent: vrijdag 19 maart 2021 15:00
To: otykier/TabularEditor @.>
Cc: Edgar Walther (ITsmart) @.>; Author @.>
Subject: Re: [otykier/TabularEditor] Deploying model.bim 1400 - 1520 to Power BI Premium (per User) gives error (#790)
Could you please give it another try with TE 2.15.0https://github.com/otykier/TabularEditor/releases/tag/2.15.0 which has the most recently updated TOM libraries?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/otykier/TabularEditor/issues/790#issuecomment-802854251, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AD47ZZ6PILSZ53NJLDTXNXDTENKHLANCNFSM4YT5FKCA.

Hi,
Hit the same error with culture = en-AU - English (Australia)
May be a race condition as deploying multiple times from Tabular Editor 2.15.1 eventually succeeds.
Power BI Desktop always succeeds (same model as above, TE opened from External Tools or Folders).

@gavingc the team confirmed that there is a bug here, and are looking into it. PBI Desktop uses a different technique to publish the dataset (it basically uploads the entire .pbix file to the service), where as Tabular Editor uses the XMLA endpoint. So the workaround for now is to change the model culture to en-US before deploying.

@gavingc the team confirmed that there is a bug here, and are looking into it. PBI Desktop uses a different technique to publish the dataset (it basically uploads the entire .pbix file to the service), where as Tabular Editor uses the XMLA endpoint. So the workaround for now is to change the model culture to en-US before deploying.

The problem is that you cannot change the culture of an existing Power BI model. Or you have to recreate the model from scratch with a different Culture 😢


fix(ULYKOFYK-2): adds REQUEST_FILENAME to 942101, moves to PL2

This PR:

Adds REQUEST_FILENAME to 942101: libinjection is usually not matching strings with a leading path (being interpreted as a not unary arithmetical operator), but quotes and comment blocks lead to correctly evaluating some SQLi crafted inside the URL (e.g. /foo/24'union all select/bar).
Moves 942101 to PL2: although single quotes can be common in languages like French and Italian, after it, a valid SQL statement is required in order to match the rule. PL2 seems to be a good fit: it already assumes consciousness of possible false positives, but it is low enough to provide security in most real-life scenarios that are looking for it.
As a side note, other libinjection SQLi checks are made at PL1 (rule 942100).
Adds related tests.

Looks good now, thank you!


updating my forked repo

Another post added

@HosseinAbedi I can't merge, it says that the patch contains conflicts, my guess is that you forgot to pull from upstream. Try this:
git remote add upstream git@github.com:woforo/woforo.github.io.git
git pull upstream master
git push

Lemme know how it goes.


Include additional types from libc

Copied a few extra definitions from the libc crate.

Thanks, @Amanieu.
@homunkulus r+

:pushpin: Commit f9ab9f9 has been approved by japaric

:hourglass: Testing commit f9ab9f9fe7b600f7eb5259d48137b0b337876fdc with merge 4d8b2525ae6adcd5a2cd09ec575234f0f65a1a80...

:sunny: Test successful - status-travis
Approved by: japaric
Pushing 4d8b2525ae6adcd5a2cd09ec575234f0f65a1a80 to master...


No module named 'membrain_seg'

I cloned the latest git repo
commit 6582ac799f87dd45be6f2135de039cec8e5e005d (HEAD -> main, origin/main, origin/HEAD)
Merge: 5cb9deb 7c9ddea
Author: Valentin Maurer<EMAIL_ADDRESS>Date:   Thu Apr 25 21:51:23 2024 +0200

    Merge pull request #5 from Luc1100/main

    Option to save the downsampled point cloud

Then, I did
conda create --name colabseg python==3.8 pip

conda activate colabseg
pip install .

python -m ipykernel install --user --name=colabseg
jupyter notebook colabseg_demo_notebook.ipynb
However, I encounter

When I did jupyter notebook colabseg.ipynb
no error.


delayed gunshot task never cancelled in Sniper mode

Sniper mode registers a delayed task in its constructor. This delayed task reschedules itself forever. The task is never cancelled because delayed_cancel() is called with the wrong name.
This bug is not very apparent because delayed tasks are not running when the mode is not present in the ModeQueue.
The fix is to register the delayed handler in mode_started() and cancel the handler with the right name in mode_stopped().

You can experiment with this test program but be aware it does not simulate the ticking of delayed tasks the same way the run loop and the ModeQueue does.
test.zip


Rename software error exception to software check exception

The name agreed to by Priv spec is "Software check exception"
https://github.com/riscv/riscv-isa-manual/pull/1105#issuecomment-1712362372


[Swagger] Fixed host and missing path, and new UI

Ref:

https://github.com/epinio/epinio/issues/1360

This PR fixes the missing path /api/v1 and adds some metadata to our swagger.json file (auth, version and so on).
It also adds an handler to the epinio server on the /api/swagger.json path that will serve the specification.
This will be used by the new UI (https://github.com/swagger-api/swagger-ui) during the make swagger-serve and adds the possibility to "try out" the endpoints.

Authentication
It's now possible to authenticate with the Basic Auth (still missing the token auth):

In the following screenshot an example of the output

From this I've seen that we have some "mismatch" or strange things to fix.
The output of the values is the helm template, we are missing the errors declaration, and some catalog services APIs are under the "Services" section.

Codecov Report

Merging #1594 (e9d1257) into main (3ba5e29) will decrease coverage by 23.40%.
The diff coverage is 8.00%.

@@             Coverage Diff             @@
##             main    #1594       +/-   ##
===========================================
- Coverage   62.65%   39.24%   -23.41%
===========================================
  Files         167      167
  Lines       12321    12347       +26
===========================================
- Hits         7720     4846     -2874
- Misses       3625     6783     +3158
+ Partials      976      718      -258

Flag
Coverage Δ

acceptance-api
?

acceptance-apps
37.94% <8.00%> (-0.58%)
:arrow_down:

acceptance-cli
?

unittests
8.62% <ø> (ø)

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

Impacted Files
Coverage Δ

internal/cli/server/server.go
48.52% <8.00%> (-14.48%)
:arrow_down:

helpers/bytes/bytes.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/api/core/v1/models/sort.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/api/core/v1/models/namespace.go
0.00% <0.00%> (-100.00%)
:arrow_down:

internal/api/v1/application/match.go
0.00% <0.00%> (-82.36%)
:arrow_down:

internal/api/v1/configuration/match.go
0.00% <0.00%> (-82.36%)
:arrow_down:

internal/api/v1/configurationbinding/delete.go
0.00% <0.00%> (-82.36%)
:arrow_down:

internal/api/v1/appchart/match.go
0.00% <0.00%> (-80.65%)
:arrow_down:

internal/api/v1/namespace/match.go
0.00% <0.00%> (-80.65%)
:arrow_down:

internal/api/v1/service/match.go
0.00% <0.00%> (-75.68%)
:arrow_down:

... and 95 more

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 3ba5e29...e9d1257. Read the comment docs.


Blog post for how Pulumi can be used to automate member account creation in AWS Organizations

🛑 This blog post requires the sources from the yet-to-be merged https://github.com/pulumi/examples/pull/1330.
I am not a huge fan of the meta image for this post but it's the best I could come up with right now. So please let me know if you have any other ideas for it. I'd be happy to update the file as long as it's within my drawio skills 😄

@cnunciato @aaronkao We should slot this into our content cadence schedule.
@praneetloke Not that you need it (if Christian says it's good then it's good), but I'm getting ready to review for another set of eyes.

@cnunciato @scottslowe thank you for spending the time to review this. I am looking at the comments right now. I'll follow-up when it's ready for another review.

@cnunciato @scottslowe I've resolved all of the suggestions. Can you please have another look? There are a couple of conversation threads that I didn't resolve just so it's easier to see the areas where a follow-up is required.
I've also got updates to the example PR as well and getting ready to push those up too.

@cnunciato @scottslowe just following-up on this to see if there is anything else I might need to do to move this along? I'll update the post's date in the front-matter based on when this can be merged. Also not sure if you had a chance to the last comment from me about the GIF.

@cnunciato @scottslowe just following-up on this to see if there is anything else I might need to do to move this along? I'll update the post's date in the front-matter based on when this can be merged. Also not sure if you had a chance to the last comment from me about the GIF.

Hi @praneetloke! I was literally reading/reviewing the PR again when I saw an e-mail notification about your comment. I just finished reviewing and I think it's good to go (I want @cnunciato to weigh in as well). For now, we will hold off on merging to publish so that we can put it into our schedule for publishing posts, but no further action (except perhaps adjusting the date) is needed from your end. Thank you for this great post!

Tentatively scheduled for Feb 6-10.

@scottslowe wonderful! I've set the date to 2/6 as per @cnunciato's comment. Let me know if you would like me to adjust it again as we get closer to 2/6.


Istiod consumes a huge amount of memory with Discovery Selectors specified

Bug Description
I'm using Istio to manage a dozen of k8s clusters, with thousands of pods and service in these clusters.  For each cluster, only one or two namespce need to be discovered by the Istiod so I set Discovery Selectors to select these namespace.
I expect that the memory usage would match the pods and services of the selected namespace, however, it turns out the istiod memory increases with the scale of clusters. In my test environment, istio eats 10 G memory after 10 clusters have been added. From the flame graph of istio heap, most of the memory is cosumed by pods cache.

Version
istioctl version
client version: 1.12.9
control plane version: 1.12.6

➜  ~ kubectl version --short
Client Version: v1.22.5
Server Version: v1.16.3-tke.15
WARNING: version difference between client (1.22) and server (1.16) exceeds the supported minor version skew of +/-1

Additional Information
No response

The cause of this issue ist that the kube serviceregistry watches all the pods and uses the discovery selectors at the client side.
After applying the correct LabelSelector when createing the SharedInformerFactory, the memory downsized from 10 G to 1 G.

It's tricky to do service-side selecting since discovery selectors can be modified dynamically, so we would have to recreate the informers...

Actually, once a discovery selector has been set, it don't need to be changed at runtime. Normally we can just modify the label of a namespace to add or remove it from a mesh.

Discovery selector is matching namespace labels. You can't query k8s for "All pods in namespace matching X label". Actually you can;t even query for anything other than "all namespaces" or "one namespace" without opening a bunch of separate watches

How many pods total to get to 10gb ram btw?
One thing we could consider doing is make a StripUnusedFields for pods

Discovery selector is matching namespace labels. You can't query k8s for "All pods in namespace matching X label". Actually you can;t even query for anything other than "all namespaces" or "one namespace" without opening a bunch of separate watches

In my case, only the pods in the selected namespace need to be insert a sidecar, so I can use label selectors to do server side filtering on pods instead of namespaces.  If the Discovery Selector mechanism can also be applied to sidecar injection webhook, that would solve the problem.

StripUnusedFields

@howardjohn how can I strip the unsed files?

Right, discoverySelector isonly applied to namespace，not service pod， etc. So definitely can not be set to informer options.
Wonder how much before, want to have a compare

how can I strip the unsed files?

You would have to find out all pod fields that istio is using, I have tried that before, please see https://github.com/dddddai/istio/commit/e36cc5de293abe512d157fefb27420b7d65d91a3
Could you test it out and see if it helps? If it does I can file a PR, I'm not 100% sure I didn't miss any fields though


Model insert calling twice in GraphQL

I'm trying to use Objection.js in GraphQL (Apollo) and I have a mutation like this:
import User from '../models/user'

export default {
  Mutation: {
    createUser: (parent, args) => User.query().insert(args)
  }
}

When I run this mutation the insert is being called twice (the mutation itself is called only once, I even put a console.log there and the only thing that is called twice is the insert).
I enabled the debug: true and that is the return on the console:
{ method: 'insert',
  options: {},
  timeout: false,
  cancelOnTimeout: false,
  bindings: [ '5d7ee514-cb73-4ef5-b880-96053a551726', 'John Doe' ],
  __knexQueryUid: 'fe07743f-4e04-458a-b533-19635f8fc7b8',
  sql:
   'insert into "users" ("id", "name") values (?, ?) returning "id"',
  returning: 'id' }
{ method: 'insert',
  options: {},
  timeout: false,
  cancelOnTimeout: false,
  bindings: [ '5d7ee514-cb73-4ef5-b880-96053a551726', 'John Doe' ],
  __knexQueryUid: '7fdc03f8-6f58-4673-ba6b-b20c88af43ab',
  sql:
   'insert into "users" ("id", "name") values (?, ?) returning "id"',
  returning: 'id' }

And the return of that is:
"errors": [
    {
      "message": "insert into \"users\" (\"id\", \"name\") values ($1, $2) returning \"id\" - duplicate key value violates unique constraint \"users_pkey\"",
      "locations": [
        {
          "line": 2,
          "column": 3
        }
      ],

What is happening?

It was my mistake, the mutation should be created with the async keyword.

I encountered the same issue. I am trying to understand why this is happening. Returning a QueryBuilder instance from a resolver (which is a promise itself, a bluebird-promise to be specific) should just work fine, right? It shouldn't be necessary to add an async keyword (to return Promise) if await is not used in the function body. @koskimas may I ask for your insight here, please?

You aren't really giving me anything to work with. How could I possibly help here? Yes, no need to mark a function async if you return a promise from it.

Just a quess: something stores the return value from that mutation, which is actually a query builder, not a promise. QueryBuilder is thenable, not a promise, so calling then twice on that query builder will insert twice.
const builder = Person.query().insert(stuff)
// Starts a query
builder.then()
// Starts another query.
builder.then()
// This only starts one query, because `.then()` returns an actual promise.
builder.then().then()

Something storing a promise and calling .then twice on the same object seems like a poor design that isn't compatible with thenables.
As a remedy you can do one of the following:

Explicitly call execute() for the builder. execute() returns a real promise.
Convert the builder to a real promise Promise.resolve(Person.query().insert(stuff)
Mark the function async, which will wrap the return value into a native promise.

Thank you @koskimas that helps a lot. I know there is not much info here, a reproduction scenario would be definitely helpful. Anyway, the issue is outside the objection's scope. But with your hints I may suspect something fishy is going on in the graphql server itself. I will dig into that.

while trying to shoot the issue blindly from different angles, I noticed that adding then at the end: createUser: (parent, args) => User.query().insert(args).then(r => r) prevents the query to be executed twice, but that's probably logical, since the graphql resolves an already then-ed query 🤔

Yeah, QueryBuilder.then executes the query and returns a promise. As I mentioned earlier, query builders are not promises. They are objects that have a then function that returns a promise.

@koskimas I've found the culprit in graphql apollo server...
https://github.com/apollographql/apollo-server/blob/master/packages/graphql-extensions/src/index.ts#L273
https://github.com/apollographql/apollo-server/blob/master/packages/graphql-extensions/src/index.ts#L297
there result.then is called but it's not returned (where result is a QueryBuilder instance returned from a graphql resolver function)...

Just for the reference: my colleague has reported an issue on apollo-server. https://github.com/apollographql/apollo-server/issues/2501

:+1: The answer from the apollo-server people can easily be we don't support this kind of objects or that objection is doing it wrong. Those are both valid opinions. Thenable isn't really a well defined thing. I don't think there's a specification anywhere that says what should happen if then of a thenable is called multiple times.
https://promisesaplus.com/
I think the right solution is to just use async in the function definition. The performance drop from adding another promise wrap is a fraction of a microsecond per function call.

@koskimas they replied, just FYI ;) https://github.com/apollographql/apollo-server/issues/2501#issuecomment-509404448

btw, this is what we did in the meantime, as a workaround:
import { GraphQLField, GraphQLObjectType, GraphQLSchema } from 'graphql';
import { QueryBuilder } from 'objection';

const wrapResolver = (field: GraphQLField<any, any>): void => {
  if (typeof field.resolve !== 'function') {
    return;
  }

  const fieldResolver = field.resolve;
  field.resolve = (source, args, context, info) => {
    const res = fieldResolver(source, args, context, info);
    if (res instanceof QueryBuilder) {
      return res.execute();
    }
    return res;
  };
};

/**
 * Workaround for executing a query twice if an instance of objection.js' QueryBuilder is returned from a resolver.
 * A QueryBuilder is executed when .then() is called. An arbitrary execution is caused by "whenResultIsFinished"
 * function in "graphql-extensions" (by apollo - introduced in apollo-server 2.x) - handling "willResolveField" hook.
 *
 * The solution would be to either always remember about executing the query already in a resolver, or mark all resolvers
 * returning a QueryBuilder with "async" - which would return a Promise instead of a QueryBuilder itself, but
 * it would be prone to human errors. Better ensure it by wrapping all resolvers, at least until apollo guys fix the issue, if ever...
 *
 * https://www.wrike.com/open.htm?id=332253507
 * https://github.com/Vincit/objection.js/issues/1143
 * https://github.com/apollographql/apollo-server/issues/2501
 */
const wrapResolversReturningObjectionQueryBuilders = (schema: GraphQLSchema): void => {
  const typeMap = schema.getTypeMap();
  Object.keys(typeMap).forEach(typeName => {
    const type = typeMap[typeName];

    // it's not necessary to wrap mutations in our case because they are wrapped already with wrapTransacting
    // - effectively causing any querybuilder returned from a mutation resolver to be executed there (with await)
    if (typeName !== 'RootMutation' && !typeName.startsWith('__') && type instanceof GraphQLObjectType) {
      const fields = type.getFields();
      Object.keys(fields).forEach(fieldName => {
        wrapResolver(fields[fieldName]);
      });
    }
  });
};

export default wrapResolversReturningObjectionQueryBuilders;

this is to be used on a GraphQLSchema instance:
wrapResolversReturningObjectionQueryBuilders(schema);


Gutter is broken

Installed git-blame package and run ctrl + B then the gutter display is broken:

atom-text-editor .gutter[gutter-name="com.alexcorre.git-blame"] {
  width: 350px;
}

This style fixes it for me. The width that it gets originally (210px) is not wide enough to display names, it only shows the first two letters.

hi @adamreisnz you should be able to click and drag the gutter to make it larger. The width you drag it to should be remembered the next time you show the gutter.
Do you see the resize cursor when mousing over the right side of the gutter?

@alexcorre that is naturally the first thing I tried, but to no avail.
Resize cursor doesn't show up and I can't resize it.
Might have to do with the new panels UI in the latest Atom dev versions.


Redundant undefined emitted for optional properties

I'm not sure whether this should be considered a bug or feature request, so I hope this format is ok.

For every optional parameter, this library emits redundant undefined type, along with ?:
Source:
method(options?: { name?: string; timeout?: number });

Output:
method(options?: undefined | { name?: undefined | string; timeout?: undefined | number });

This looks odd. Why not produce the TypeScript native type? It would look more readable, without the extra undefined that makes documentation look too verbose, and less readable.

TypeScript v3.8.3
TypeDoc v0.16.11
NodeJS v13.8.0

I can see this was once suggested, in #1063, but then was closed without resolution.

PR welcome if you want to give this a shot. This should just be a theme change - the type should still be represented internally as a union type since that is what it is.

Version 0.20 (currently typedoc@beta) removes undefined from optional properties


Upgrade NodeJS from 12 to 16

Fixes Issue

Closes #36
Changes proposed

[x] Update from Node.JS 12 to Node.JS 16

Check List (Check all the applicable boxes)

[X] My code follows the code style of this project.
[X] My change requires changes to the documentation.
[X] I have updated the documentation accordingly.
[X] All new and existing tests passed.
[X] This PR does not contain plagiarized content.
[X] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

@eddiejaoude could you please review this PR? Thanks in advance!


fix: sqlite3 library deadlock

Check if this PR fulfills these requirements:

[x] My changes respect the Kodi add-on development rules
[x] I have read the CONTRIBUTING document.
[x] I made sure there wasn't another one Pull Requests opened for the same update/change
[x] I have successfully tested my changes locally

Types of changes

[ ] New feature (non-breaking change which adds functionality)
[ ] Feature change (non-breaking change which change behaviour of an existing functionality)
[ ] Improvement (non-breaking change which improve functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description
Bug fix for sqlite3 library deadlock described here: https://github.com/CastagnaIT/plugin.video.netflix/issues/425
This still ensures that there is only 1 active thread with sqlite3 connection (which https://github.com/CastagnaIT/plugin.video.netflix/commit/3392a7e8c579a251dfd91aec530a62bd41777110 introduced) and it now correctly handles reentrant handle_connection decorators.

Works for me!

so this can be a solution, but this reintroduce another problem,
cause a higher possibility that the database will break
multiple threads can read from db without problems, but if multiple threads write at the same time it will break with losing all user data.
this can be solved by change Lock() with RLock()
and reintroduce is_connected as reverted in: https://github.com/CastagnaIT/plugin.video.netflix/commit/3392a7e8c579a251dfd91aec530a62bd41777110#diff-82b0de5d29577af7f3dcaf3ae2d8008d

@CastagnaIT actually, it doesn't. Mutex is still here and first thread that tries to connect to database will lock it so only this one will be able to read/write to that db.
RLock would work just as well here and I considered it but just didn't really like it.

but your mutex is inside if not args[0].is_connected:
so in this way when the first connection is establish, any other thread would go through without wait the previous one

removed misread

@CastagnaIT is_connected is a thread local variable in my approach. So any other thread will see it set to False until it successfully grabs a mutex and connects to it.

misinterpreted the use of local so good


Multiple zum-waypoints on same page, one with offset="100%"

I have 2 waypoints set within 1 angular app.
The first fires as expected when that element hits the top of the viewport.
The second has an offset of 100%, but never seems to fire, whether that element hits the bottom of the viewport or the top.
Thoughts?

@cbaxter713 Thanks for the report.
I've made a few attempts to reproduce, but it seems to work for me.  Can you provide a reproduction?

Thanks for replying.  I figured out that my second waypoint was setup outside of the angular app declaration in the DOM.  Got it fixed.
Thanks!


ProjectsV2?

I'm seeing some weirdness in re: ProjectsV2. Specifically, this command:
gh api graphql -f query='
  query{
  node(id: "PVT_kwDOAU_qk84AHJ4X") {
    ... on ProjectV2 {
      fields(first: 20) {
        nodes {
          ... on ProjectV2Field {
            id
            name
          }
          ... on ProjectV2IterationField {
            id
            name
            configuration {
              iterations {
                startDate
                id
              }
            }
          }
          ... on ProjectV2SingleSelectField {
            id
            name
            options {
              id
              name
            }
          }
        }
      }
    }
  }
}'

Works fine against this repo (https://github.com/orgs/filecoin-project/projects/65/views/1).
However, the same thing in the API:
gh_token = os.getenv("GITHUB_TOKEN")
api = GhApi(owner="filecoin-project", token=gh_token)

p = api.projects.list_columns(node="PVT_kwDOAU_qk84AHJ4X", per_page=20, page=0)
print(f"{p.title} - {p.id}")

Returns not found. Querying issues normally works. Could this be a v2 issue?

All endpoints of ghapi are base on REST API V3.
ProjectV2 only exist in GraphQL from what I see in GitHub documentation.

oh rats - so i have to do it all by hand then? sigh. thanks


The mousehover for selecting the Verticle line should cover a wider area.  Currently, it is too precise for the user to select and drag the Verticle line

The mousehover for selecting the Vertical line should cover a wider area.  Currently, it is too precise for the user to select and drag the Verticle line
ScottPlot Version:5.0.33
Making it a wider area will be very useful for the user to select and drag the Vertical line easily.

Hi @kareem469,
To give the cursor a wider "hit box", increase the number 10 to a larger value
https://github.com/ScottPlot/ScottPlot/blob/d003f9a1f19aae487cb03a25becc2d54efd0a2e4/src/ScottPlot5/ScottPlot5 Demos/ScottPlot5 WinForms Demo/Demos/DraggableAxisLines.cs#L86


Consider using git2 instead of executing git process

It would be nice to adapt same approach https://github.com/cstorey/git-build-version uses - reading git metadata using git2 library. This would make building in restricted environment easier, e.g. docker images for cross do not contain git binary.

Good idea.  I'll get this worked in here shortly.

Great!
btw I hope you don't mind, for now I "borrowed" some of your code to create a pull request over there :)

Don't mind at all.  Beauty of open source.


Would you accept a PR that adds a GoTestEngine?

While I know it's not precisely an extendible test engine like TapTestEngine, it does only work in the MultiTestEngine context.. I'm happy to send a PR if you're cool with it.
final class GoTestEngine extends ArcanistUnitTestEngine {
public function run() {
$command = $this->getConfigurationManager()->getConfigFromAnySource('unit.engine.go.command');
$future = new ExecFuture($command);
do {
  list($stdout, $stderr) = $future->read();
  echo $stdout;
  echo $stderr;
  sleep(0.5);
} while (!$future->isReady());

list($error, $stdout, $stderr) = $future->resolve();

$parser = new ArcanistGoTestResultParser();
return $parser->parseTestResults("", $stdout);

}
}

Sure thing! Sorry for the delay, I am not being able to review your PRs, I will take a look at this soon.


fix(trx): fix transactional method return value & improve types

So this change is just about returning what was returned from transactional callback, preserving the type?
Sounds good, I just dont like this being called a fix :) lets call it refactoring

@B4nan how you wish. changed


[update_checkout] Add success message to script

I think the script for getting sources for Swift should provide not only a failing message but also a success message.

LGTM

@swift-ci smoke test and merge

Thank you!


[Enhance] updating Footer with subscribe section ,

Description: To improve user engagement and build our mailing list, we need to enhance the footer of our website by adding a subscription section. This section will allow users to easily subscribe to our newsletter for updates, news, and promotions.

Description: To improve user engagement and build our mailing list, we need to enhance the footer of our website by adding a subscription section. This section will allow users to easily subscribe to our newsletter for updates, news, and promotions.

Not required at present


(re)use text freely?

Can everything in this repo be used freely? As far as I can judge nothing is said about this. I would like to (re)use text for my project. Can I do that freely?

Ironically, I didn't set a license to that repository… it's now fixed! Thanks for reporting the issue and feel free to reuse it 🤗

Thanks you, I will at least make a reference from my project to this repo.


requests to  https://raw.githubusercontent.com/DotNetAnalyzers/StyleCopAnalyzers/master/StyleCop.Analyzers/StyleCop.Analyzers/Settings/stylecop.schema.json from VS

I have a lot of request to https://raw.githubusercontent.com/DotNetAnalyzers/StyleCopAnalyzers/master/StyleCop.Analyzers/StyleCop.Analyzers/Settings/stylecop.schema.json withing VS,
is there a way to configure this? I dont see the point of making that many request

❓ How many is "a lot"? These files are cached by Visual Studio on each machine to reduce the number of lookups.
You can copy the schema to a location within your source repository, and update your stylecop.json file to use a relative URI pointing to that instead.

about 100 requests a second
I will try that, thanks

@JuanZamudioGBM Which IDE are you using? Do you know the source of these lookups? These requests should only occur when the stylecop.json file is opened for editing, and should only occur once per user per machine.

VS2017 15.3 (released yesterday)
When I Open the file the requests don't stop until I close the file, I'm using .net core 2 and the version of the analyzer is 1.0.2


Me macro updates

-ME Tidal report macro
-ME Tidal compares cases macro
-Similar to ME Wave report and compares cases macros

@cpaulgilman I did come across that but I think I only ran each macro once. I will investigate.


Deployment w/ SSL?

It would be awesome to have some guidance in this repo's README on how to setup a Stout-deployed static site to be served up as "https" by default from CloudFront--since this is an AWS-centric tool anyways ;)

I added a note to the readme, thanks! https://github.com/EagerIO/Stout/commit/3f693f16756a585b4ac62d75c6948fa13508e787


Fix the windows build

Add a bin target to satisfy cmake windows requirement, bump the version to cmake 3.
The cmake version for Linux is per vfx-platform 2018, the version for Windows is the latest, otherwise VS2019 can't be supported.

Codecov Report

Merging #669 into master will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #669      +/-   ##
==========================================
- Coverage   81.69%   81.67%   -0.03%
==========================================
  Files          72       72
  Lines        2731     2728       -3
==========================================
- Hits         2231     2228       -3
  Misses        500      500

Flag
Coverage Δ

#py27
81.67% <ø> (ø)

#py36
?

#py37
?

Impacted Files
Coverage Δ

...ntimelineio/opentimelineio-bindings/otio_utils.cpp
87.05% <0.00%> (-0.16%)
:arrow_down:

...elineio/opentimelineio-bindings/otio_anyVector.cpp
100.00% <0.00%> (ø)

...eio/opentimelineio-bindings/otio_anyDictionary.cpp
100.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


Cluster: Remove panic in UpgradeMembersWithoutRole

Would be interesting to see the list of members and nodes when there's no id found, can that be added to debug level logging?

Good idea, will add.

@MathieuBordere done


Add adam.sr

Adds adam.sr to 512.club.  Cheers guys!

This PR is:

[x] Adding a new domain
[ ] Updating existing domain size
[ ] Changing domain name
[ ] Removing existing domain from list
[ ] Website code changes (512kb.club site)
[ ] Other not listed

[x] I used the uncompressed size of the site
[x] I have included a link to the GTMetrix report
[x] The domain is in the correct alphabetical order
[x] This site is not a ultra lightweight site
[x] The following information is filled identical to the data file

I confirm that I have read the FAQ section, particularly the two red items around minimal pages and inappropriate content and I attest that this site is neither of these things.

[x] Check to confirm

- domain: adam.sr
  url: https://adam.sr/
  size: 109
  last_checked: 2022-11-22

GTMetrix Report: https://gtmetrix.com/reports/adam.sr/PA6k77n6/

Thanks a lot @garritfra 😊!

Cheers!🙂


mixed fraction to improper fraction vice versa

Enter calculator name -
mixed fraction to improper fraction vice versa
Describe calculator functionality
converting mixed fraction to improper fraction and vice versa
Under which category you wish to place it -

[ ] Computer Science
[x ] Mathematics
[ ] Physics
[ ] Chemistry
[ ] Biology
[ ] Finance
[ ] Scientific
[ ] Health
[ ] Sports
[ ] Let project owner decide
[ ] Suggest a new category - (Answer here)

kindly assign this issue to me

Hey @munfaridzahera,
Assigned.
You may contribute your code under the folder path -
calc_for_everything\calculators\src\mathematical\mixedToImproperAndImproperToMixedFraction
Issue completion deadline - 12th Oct 8:00:00 am IST
NOTE - In case the issue completion deadline is not met, the issue will be assigned to a new participant


Epoch 285 catalyst rewards mismatch (bug)

There is a mismatch between cardano-db-sync catalyst rewards and cardano-cli (ledger). The db sync states 2 catalyst rewards which appeared in epoch 285. However, the sooner one of them is invisible to cardano-cli. A few examples:
https://cardanoscan.io/stakekey/05c6aa84de1fd5d5d15fa2cc0f30a5c804f52fb19725525aab54a839?tab=instantaneousrewards
Here you can see the two catalyst rewards which appeared 5 days ago. AFAIK cardanoscan simply sums up the rewards minus withdrawals. The reward sum with the two catalyst rewards for this address is 2649355 Lovelaces. According to cardano-cli, "rewardAccountBalance": 2629950. The difference is 19405 Lovelace, which corresponds to the sooner catalyst reward of the two.
Same here - https://cardanoscan.io/stakekey/stake1uyzuqsnynkxkqs848y8mgylywwezrgcrmd6xmqspx4kfk7gqtlzjv?tab=instantaneousrewards
Reward sum: 103826843
Cardano-cli reward: 103654676
Diff: 172167 = equal to the sooner of the two rewards
The same goes for basically any address you look up.
Db sync version: 9.0.0
This caused Adalite users not being to withdraw rewards until we started filtering out the first of epoch 285's rewards.

I think I have a fix for #696. @xdzurman do you have an specific address with a reward amount for epoch 285 that I can test?

two catalyst rewards showed up for all participants, but the first of the two did not make it through to node and cli, but it appeared in db-sync,

The node uses the same code as db-sync, so how could a reward payment make through to db-sync but not the node ?

I think I have a fix for #696 (still under test). @xdzurman do you have an specific address with a reward amount for epoch 285 that I can test?

Let's say stake1u9schrkl2ed8daympxqxd38z72kx00y69ky8udd7g4v53kcp5kkyg. The cardano-cli reward amount was
cardano-cli query stake-address-info --mainnet --address stake1u9schrkl2ed8daympxqxd38z72kx00y69ky8udd7g4v53kcp5kkyg
[
    {
        "address": "stake1u9schrkl2ed8daympxqxd38z72kx00y69ky8udd7g4v53kcp5kkyg",
        "rewardAccountBalance": 10750235,
        "delegation": "pool1j6490a90qefvsqxdurf99z9ld50rtthaz9ms6c8xh78s52veuq2"
    }
]

But if you calculate the rewards from db-sync, the outstanding reward balance is 10793839. The difference is 43604, which corresponds to the first of the two catalyst rewards for epoch 285.

two catalyst rewards showed up for all participants, but the first of the two did not make it through to node and cli, but it appeared in db-sync,

The node uses the same code as db-sync, so how could a reward payment make through to db-sync but not the node ?

Quoting Samuel Leathers:
okay, so @jared.corduan had a late night debugging and we see what the issue is. There's actually multiple, most are my fault, one was a misunderstanding in how MIR's worked.
To start off, first mistake I made was copy of wrong rewards distributed. My general QA for testing the json file is I check number of lines in it, and then post it to the channel for others to review. No one caught that the total rewards amount was wrong. In the future, I have a script now that sums the rewards and outputs a total value being distributed.
Second was I executed the script with wrong pot. This was because of a MIR we did from reserves to account for a bug in rewards calculation prior to allegra. I used the same script, and didn't switch it back from reserves -> treasury.
Third is where things get really hairy. I made the false assumption multiple MIR's would accumulate, but in fact they replace (but only prior to alonzo, they accumulate post-alonzo). @jared.corduan figured this out last night comparing the output of his stake address tool to rewards total balance. So essentially, everyone's been paid the second MIR but not the first.
I'll be formalizing this into a proper issue, but lots of lessons learned on this one.
If we'd like to have a call to discuss it, we can bring @jared.corduan in to explain technically what's happening if this isn't enough detail.

@xdzurman Unfortunately, your cardano-cli query returns an rewardAccountBalance which of course changes over time.
db-sync on the other hand stores credits to the reward account in the reward table and debits in the withdrawal table.
On current master (after merging of the reward validation) I get:
select reward.* from reward inner join stake_address on reward.addr_id = stake_address.id
    where stake_address.view = 'stake1u9schrkl2ed8daympxqxd38z72kx00y69ky8udd7g4v53kcp5kkyg'
    order by earned_epoch ;
    id    | addr_id |   type   | amount  | pool_id | earned_epoch | spendable_epoch
----------+---------+----------+---------+---------+--------------+-----------------
 13040354 | 1895674 | member   |  353659 |    7699 |          273 |             275
 13651004 | 1895674 | member   |  238593 |    7699 |          274 |             276
 14271526 | 1895674 | member   |  940700 |    7699 |          275 |             277
 14900823 | 1895674 | member   | 1352813 |    7699 |          276 |             278
 15547990 | 1895674 | treasury |   39108 |         |          277 |             279
 15547989 | 1895674 | member   |  493762 |    7699 |          277 |             279
 16210475 | 1895674 | member   | 1371629 |    7699 |          278 |             280
 16867777 | 1895674 | member   | 1236043 |    7699 |          279 |             281
 17533273 | 1895674 | member   | 1004487 |    7699 |          280 |             282
 18204762 | 1895674 | member   | 1112734 |    7699 |          281 |             283
 18883060 | 1895674 | member   |  895610 |    7699 |          282 |             284
 19567131 | 1895674 | member   |  851842 |    7699 |          283 |             285
 20268768 | 1895674 | member   |  845867 |    7699 |          284 |             286
 20268769 | 1895674 | reserves |   13388 |         |          284 |             286
 20986850 | 1895674 | member   |  973891 |    7699 |          285 |             287
 21694133 | 1895674 | member   |  225910 |    7699 |          286 |             288
 23036583 | 1895674 | member   |  518683 |    7699 |          287 |             289
(17 rows)

and
select withdrawal.* from withdrawal inner join stake_address on withdrawal.addr_id = stake_address.id
    where stake_address.view = 'stake1u9schrkl2ed8daympxqxd38z72kx00y69ky8udd7g4v53kcp5kkyg';
 id | addr_id | amount | tx_id | redeemer_id
----+---------+--------+-------+-------------
(0 rows)

Does that seem correct?

Yes, this seems correct. Until now, we queried ITN and catalyst rewards from reserve and treasury tables, but this solves it. If you check rewards for the same address from the reserve table, you can see two entries:
cardano=# select * from reserve where addr_id =1610222;
  id   | addr_id | cert_index | amount |  tx_id
-------+---------+------------+--------+----------
 54814 | 1610222 |          0 |  19405 | 10923521
 86899 | 1610222 |          0 |   5958 | 10991467

one of which should not be there. That was the whole issue.

I think I have a fix for #696 (still under test). @xdzurman do you have an specific address with a reward amount for epoch 285 that I can test?

Also, can you check rewards for stake1ux025zswj96df8esu78wqdhvfc56yg9yzztwhj4hgkp0rqswd7rl7? In db sync 11, all rewards from before epoch 257 are ignored.
Rewards from db sync 9.0.0:
    id    | addr_id | amount | epoch_no | pool_id | block_id
----------+---------+--------+----------+---------+----------
  1523654 |  303094 |   6724 |      237 |     454 |  5150301
  1620412 |  303094 |   3403 |      238 |    4040 |  5171559
  1721179 |  303094 |   3492 |      239 |    4040 |  5192987
  1825341 |  303094 |   2644 |      240 |    4040 |  5214386
  1936927 |  303094 |   8057 |      241 |    4040 |  5235804
  2055509 |  303094 |   7383 |      242 |    4040 |  5257390
  2183876 |  303094 |   6234 |      243 |    4040 |  5278881
  2321976 |  303094 |   7963 |      244 |    4040 |  5300366
  2468914 |  303094 |   4472 |      245 |    4040 |  5321693
  2619851 |  303094 |   5521 |      246 |    4040 |  5343201
  2783288 |  303094 |   7723 |      247 |    9788 |  5364557
  2960581 |  303094 |   5974 |      248 |    9788 |  5385933
  3159241 |  303094 |   6625 |      249 |    9788 |  5406924
  3378432 |  303094 |   4698 |      250 |    9788 |  5428248
  3621517 |  303094 |   7799 |      251 |    9788 |  5449333
  3885633 |  303094 |     20 |      252 |     454 |  5470255
  4165967 |  303094 |     26 |      253 |    6678 |  5490968
  4461010 |  303094 |     32 |      254 |    6678 |  5512319
  5814262 |  303094 |   3618 |      257 |    7717 |  5576454
  6570265 |  303094 |   1866 |      259 |     454 |  5619113
  6966660 |  303094 |   1264 |      260 |     454 |  5640553
  7376790 |  303094 |    853 |      261 |     454 |  5662037
  7798795 |  303094 |   1477 |      262 |     454 |  5683430
  8232488 |  303094 |    781 |      263 |     454 |  5704839
  8677097 |  303094 |   1033 |      264 |    8956 |  5726575
 14997015 |  303094 |   1521 |      271 |     454 |  5878959
  9594796 |  303094 |   1151 |      265 |    8956 |  5748284
 10066694 |  303094 |    948 |      266 |    8956 |  5770278
 10557469 |  303094 |    627 |      267 |     454 |  5791997
 15586281 |  303094 |      9 |      272 |     454 |  5900696
 11602292 |  303094 |    706 |      268 |     454 |  5813820
 12139677 |  303094 |    719 |      269 |     454 |  5835493
 13257765 |  303094 |    615 |      270 |     454 |  5857193
 16795160 |  303094 |     11 |      273 |     454 |  5922535
 17406737 |  303094 |     10 |      274 |     454 |  5944326
 18655144 |  303094 |     12 |      275 |     454 |  5965898
 19919270 |  303094 |     10 |      276 |     454 |  5987504
 20559745 |  303094 |     13 |      277 |     454 |  6009190
 21210116 |  303094 |     12 |      278 |     454 |  6030813
 21869264 |  303094 |      8 |      279 |     454 |  6052318
 22534301 |  303094 |     10 |      280 |     454 |  6074054
 23206785 |  303094 |      9 |      281 |     454 |  6095819
 23885506 |  303094 |     12 |      282 |     454 |  6117580
 24568320 |  303094 |      8 |      283 |     454 |  6139413
 25259671 |  303094 |      8 |      284 |     454 |  6160945
 25958140 |  303094 |      6 |      285 |     454 |  6182770
 26667651 |  303094 |     13 |      286 |     454 |  6204160
 28118640 |  303094 |     10 |      287 |     454 |  6225656

Rewards from db sync 11.0.0
    id    | addr_id |  type  | amount | pool_id | earned_epoch | spendable_epoch
----------+---------+--------+--------+---------+--------------+-----------------
  5512286 |  312446 | member |   3618 |    7717 |          257 |             259
  6269795 |  312446 | member |   1866 |     454 |          259 |             261
  6666441 |  312446 | member |   1264 |     454 |          260 |             262
  7075874 |  312446 | member |    853 |     454 |          261 |             263
  7498675 |  312446 | member |   1477 |     454 |          262 |             264
  7932392 |  312446 | member |    781 |     454 |          263 |             265
  8377630 |  312446 | member |   1033 |    8956 |          264 |             266
  8835356 |  312446 | member |   1151 |    8956 |          265 |             267
  9307180 |  312446 | member |    948 |    8956 |          266 |             268
  9798989 |  312446 | member |    627 |     454 |          267 |             269
 10316273 |  312446 | member |    706 |     454 |          268 |             270
 10853716 |  312446 | member |    719 |     454 |          269 |             271
 11409671 |  312446 | member |    615 |     454 |          270 |             272
 11984714 |  312446 | member |   1521 |     454 |          271 |             273
 12575182 |  312446 | member |      9 |     454 |          272 |             274
 13177526 |  312446 | member |     11 |     454 |          273 |             275
 13790404 |  312446 | member |     10 |     454 |          274 |             276
 14412845 |  312446 | member |     12 |     454 |          275 |             277
 15043802 |  312446 | member |     10 |     454 |          276 |             278
 15698747 |  312446 | member |     13 |     454 |          277 |             279
 16357527 |  312446 | member |     12 |     454 |          278 |             280
 17016870 |  312446 | member |      8 |     454 |          279 |             281
 17683530 |  312446 | member |     10 |     454 |          280 |             282
 18356209 |  312446 | member |      9 |     454 |          281 |             283
 19035558 |  312446 | member |     12 |     454 |          282 |             284
 19719657 |  312446 | member |      8 |     454 |          283 |             285
 20430683 |  312446 | member |      8 |     454 |          284 |             286
 21142437 |  312446 | member |      6 |     454 |          285 |             287
 21851842 |  312446 | member |     13 |     454 |          286 |             288
 22575761 |  312446 | member |     10 |     454 |          287 |             289

Could the fix be patch released until the hard fork?

@xdzurman how do you know db-sync-9 was correct?

That is the version everyone rolled back to after 10.0.0 introduced rewards issues.

Running master against testnet, there are couple of these as well and one needs to disable panicAbort to be able to fully sync from scratch
[db-sync-node:Warning:77] [2021-09-08 21:35:31.63 UTC] validateEpochRewards: rewards earned in epoch 84 expected total of<PHONE_NUMBER>7 ADA but got<PHONE_NUMBER>7 ADA
[db-sync-node:Warning:77] [2021-09-08 21:48:27.42 UTC] validateEpochRewards: rewards earned in epoch 107 expected total of<PHONE_NUMBER>38 ADA but got<PHONE_NUMBER>38 ADA

@xdzurman

That is the version everyone rolled back to after 10.0.0 introduced rewards issues. It has been deployed to production for a long time and stood the test of time (except a minor tweak regarding epoch 285's catalyst rewards).

It may well have stood the test of time, but it may still be wrong.

Running master against testnet, there are couple of these as well and one needs to disable panicAbort to be able to fully sync from scratch
[db-sync-node:Warning:77] [2021-09-08 21:35:31.63 UTC] validateEpochRewards: rewards earned in epoch 84 expected total of<PHONE_NUMBER>7 ADA but got<PHONE_NUMBER>7 ADA
[db-sync-node:Warning:77] [2021-09-08 21:48:27.42 UTC] validateEpochRewards: rewards earned in epoch 107 expected total of<PHONE_NUMBER>38 ADA but got<PHONE_NUMBER>38 ADA

That is an indication of a bug.

Running master against testnet, there are couple of these as well and one needs to disable panicAbort to be able to fully sync from scratch

If the panicAbout is triggered, that is an indication of a bug. That is why it is a panicAbort and not just a info level log message.

Created #805

That is the version everyone rolled back to after 10.0.0 introduced rewards issues. It has been deployed to production for a long time and stood the test of time (except a minor tweak regarding epoch 285's catalyst rewards).

Yes, they may have rolled back to 9.0.0 thinking it was correct, but I am 100% sure it was not correct.

@xdzurman
select hash_raw from stake_address where view = 'stake1ux025zswj96df8esu78wqdhvfc56yg9yzztwhj4hgkp0rqswd7rl7' ;
                           hash_raw
--------------------------------------------------------------
 \xe19eaa0a0e9174d49f30e78ee036ec4e29a220a41096ebcab74582f182
(1 row)

and then using the stake history tool in the node respository:
./stake-credential-history -c configuration/cardano/mainnet-config.json -s state-node-mainnet/node.socket \
       -t e19eaa0a0e9174d49f30e78ee036ec4e29a220a41096ebcab74582f182 --check-point-size 100000

shows that yes indeed, rewards for this address should start in epoch 238.
I think this is basically a duplicate of #769 .

That is the version everyone rolled back to after 10.0.0 introduced rewards issues. It has been deployed to production for a long time and stood the test of time (except a minor tweak regarding epoch 285's catalyst rewards).

Yes, they may have rolled back to 9.0.0 thinking it was correct, but I am 100% sure it was not and has never been correct.

I do not think that is the case. I mean, at least it was much more "correct" (like a fraction of a percent might have been incorrect). Otherwise we would see a lot more failed reward withdrawal transactions in Yoroi, as we would construct the transaction with incorrect reward in place and get rejected by the ledger. With 9.0.0, I really hardly recall any of it.
And as far as I know this has been the case with Adalite too, as they have upgraded from 9.0.0.

Same here. Users of adalite were always able to withdraw their funds while we were using 9.0.0 (we compute the reward amounts as a sum of all types of rewards minus withdrawals). This always worked, except the ones that retired their pools and got the deposit back, for which I created an issue more that 9 months ago. This issue (catalyst's epoch 285 double rewards) was the second major issue, but apart from that version 9.0.0 was correct (based on the Adalite's historical user logs and reported issues).
However, when switching to db synx 10.0.0, a huge portion of users had wrong rewards. These issues are still unresolved in 11.0.0, which forces us to show warning banners to our users that their reward history is most probably wrong. We also had to resort to ignore the reward sum from db sync and query it via ogmios so that the users can at least withdraw their funds. Ever since having to move from db sync 9.0.0, we are impatiently pleading for fixes to the major issues, so that we don't have to compromise the truthfulness of the data we serve to our users.

Hopefully this is fixed on https://github.com/input-output-hk/cardano-db-sync/releases/tag/13.0.0. Feel free to reopen if the issue persists.
Most of the work for rewards is not done in leger through ledger events.


Move statefulsyncer, storage, and utils to rosetta-sdk-go

Related PR: https://github.com/coinbase/rosetta-sdk-go/pull/99

Pull Request Test Coverage Report for Build 3887

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+4.07%) to 75.631%

Totals

Change from base Build 3877:
4.07%

Covered Lines:
599

Relevant Lines:
792

💛  - Coveralls


Release

Codecov Report

Merging #130 (53ddf89) into main (2d32d22) will increase coverage by 0.11%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #130      +/-   ##
==========================================
+ Coverage   88.60%   88.72%   +0.11%
==========================================
  Files          82       82
  Lines        1316     1330      +14
  Branches      122      130       +8
==========================================
+ Hits         1166     1180      +14
  Misses        148      148
  Partials        2        2

Impacted Files
Coverage Δ

src/api/storage/v0/storage.ts
97.72% <100.00%> (+0.25%)
:arrow_up:

src/resources/api.ts
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


How to set maximum video seconds?

Hello,
very great library it is. Can you please tell me how to set maximum seconds for recording video? I want to set maximum 20 seconds limit for video recording.

can you find the solution ? I also want to set the maximum duration


[BUG] segmentation fault (core dumped)

Software Versions:

Operating System:
Python version:
BusyLight version:

General Type of Problem

[ ] Installation
[ ] Command-line
[ ] Integrating into another project
[ ] Web API
[ ] other

Describe the Problem
A description of what the bug is.
Expected Behavior
What you expected to happen.
Error Output
If there was an error displayed, paste it in here.

fat fingered something. sorry.  closing this one.


Not receiving pokedollars from collecting any resource

Sponge Version:
Forge Version:
Total Economy Version:
Description of issue
Not receiving pokedollars from collecting any resource
Console errors
[TE] Failed to reload sets and/or jobs!
Steps to reproduce
mine any of the total economy blocks
Related Configuration Files
https://gist.github.com/Phynn2/d69afa21402dda8b344d8959077ba175

fixed, there was a missing "


fix: CI check go dependencies

Why the changes were made
Fixes https://github.com/openshift/oadp-operator/issues/1369
How the changes were made
Created make command that will check if go mod tidy and go mod verify were run.
How to test the changes made
Test new command, running make check-go-dependencies or make test

/retest

/hold

/remove-lifecycle stale

/unhold

/retest

/recheck

/retest


classNames

I see that react-modal used to have the ability to specify classes for styles but that was removed in favor of inline styles.
While some might argue that inline styles are better for react applications, not everyone has that option (including myself).
It would be nice if react-modal wasn't opinionated about this and supported both giving classNames and inline styles.

:+1:

+1 please bring back classNames. Its a pain to do custom animations with inline-styles that can't be overwritten with css.

:+1:

For those that are interested I've started a separate project react-modal2 which is less opinionated, accessible, stateless (dumb component), and is built via a reusable collection of modules. It also allows you to use classNames instead of inline styles.

:+1:

Seems like very little effort to support both. 👍

:+1:

Looks like #100 would fix this, but it hasn't been reviewed/merged yet.

v1.0.0 is now released with supported added back for classNames


LayoutReader and InteractionView interrupt the animation transaction chain

As a developer integrating Pillarbox in my user interface I would like to be able to use UIViewControllerRepresentable-based views like LayoutReader and InteractionView as I would use a simple ZStack. But if I currently have animations in my layout then those might not behave as expected depending on where they are applied.
Acceptance criteria 1
Animations must be correctly transferred through InteractionView (and LayoutReader). This means the following layout:
struct RectangleView: View {
    @State private var isLarge = false

    var body: some View {
        InteractionView {
            Color.yellow
            Rectangle()
                .frame(width: 100, height: 100)
                .scaleEffect(isLarge ? 2 : 1)
        }
        .animation(.easeInOut, value: isLarge)
        .onTapGesture {
            isLarge.toggle()
        }
    }
}

must be animated identically to the same code obtained by replacing InteractionView with a simple ZStack.
Acceptance criteria 2
Animations must be correctly transferred through InteractionView (and LayoutReader) and have correct limited scope according to where they should apply. This means the following layout:
struct RectangleView: View {
    @State private var isLarge = true

    var body: some View {
        ZStack {
            Color.yellow
            Rectangle()
                .frame(width: 100, height: 100)
                .scaleEffect(isLarge ? 2 : 1)
        }
        .animation(.easeInOut, value: isLarge)
        .onTapGesture {
            isLarge.toggle()
        }
    }
}

struct ContentView: View {
    @State private var isHalved = false

    var body: some View {
        VStack {
            if isHalved {
                Color.clear
            }
            RectangleView()
            Button(action: toggle) {
                Text("Toggle")
            }
        }
    }

    private func toggle() {
        isHalved.toggle()
    }
}

must be animated identically to the same code obtained by replacing InteractionView with a simple ZStack:

Tap on the square first. The square must smoothly shrink and expand with an animation.
Tap on the Toggle button. The transition must be abrupt as no animation modifier is applied.

Investigation strategy
The two separate acceptance criteria above can be combined into a single problem more general than Pillarbox view implementations.
Given a view hosting a UIHostingController which itself hosts a SwiftUI view:
struct HostedView<Content>: UIViewControllerRepresentable where Content: View {
    @Binding private var content: () -> Content

    init(@ViewBuilder content: @escaping () -> Content) {
        _content = .constant(content)
    }

    func makeUIViewController(context: Context) -> UIHostingController<Content> {
        .init(rootView: content())
    }

    func updateUIViewController(_ uiViewController: UIHostingController<Content>, context: Context) {
        uiViewController.rootView = content()
    }
}

the following layout:
struct RectangleView: View {
    @State private var isLarge = true

    var body: some View {
        HostedView {
            Color.yellow
            Rectangle()
                .frame(width: 100, height: 100)
                .scaleEffect(isLarge ? 2 : 1)
        }
        .animation(.easeInOut, value: isLarge)
        .onTapGesture {
            isLarge.toggle()
        }
    }
}

struct ContentView: View {
    @State private var isHalved = false

    var body: some View {
        VStack {
            if isHalved {
                Color.clear
            }
            RectangleView()
            Button(action: toggle) {
                Text("Toggle")
            }
        }
    }

    private func toggle() {
        isHalved.toggle()
    }
}

must animated identically if we replace HostedView with a simple ZStack:

Tap on the square first. The square must smoothly shrink and expand with an animation.
Tap on the Toggle button. The transition must be abrupt as no animation modifier is applied.

Current visible effects
In the demo custom player we can observe that the slider and volume button are not animated when toggling the controls. This is due to the use of LayoutReader in our layout which, because of this issue, does not allow animation transactions to cross the associated view boundaries correctly. Simply removing the LayoutReader in the layout (or replacing it with a ZStack) namely solves the issue (at the expense of full screen detection, though).
Tasks
Several mitigation strategies can be investigated:

[x] ❌ Use UIViewRepresentable instead of UIViewControllerRepresentable.
[x] ❌ Find a way to transfer the transaction to the inner wrapped view.
[x] ❌ Transfer the observed animation value to the inner wrapped view.
[x] ✅ Revisit implementation strategy to avoid SwiftUI - UIKit - SwiftUI containment, possibly with overlays, background modifiers, ZStack to stack content and detector views, etc.

We should also:

[x] Report the above issue to Apple if there is no way to fix it without actually having it implemented in the SwiftUI layout engine itself.

We can try to transfer the transaction to the inner wrapped view as follows:
struct HostedWrapperView<Content>: View where Content: View {
    let transaction: Transaction
    @Binding private var content: () -> Content

    init(transaction: Transaction, @ViewBuilder content: @escaping () -> Content) {
        self.transaction = transaction
        _content = .constant(content)
    }

    var body: some View {
        ZStack {
            content()
        }
        .transaction { transaction in
            transaction = self.transaction
        }
    }
}

struct HostedView<Content>: UIViewControllerRepresentable where Content: View {
    @Binding private var content: () -> Content

    init(@ViewBuilder content: @escaping () -> Content) {
        _content = .constant(content)
    }

    func makeUIViewController(context: Context) -> UIHostingController<HostedWrapperView<Content>> {
        .init(rootView: HostedWrapperView(transaction: context.transaction) {
            content()
        })
    }

    func updateUIViewController(_ uiViewController: UIHostingController<HostedWrapperView<Content>>, context: Context) {
        uiViewController.rootView = HostedWrapperView(transaction: context.transaction) {
            content()
        }
    }
}

The issue is that the transaction is not scoped, therefore the second acceptance criteria fails as halving the rectangle view space without animation will still apply the animation that has been attached before.

The problem affects the SwiftUI - UIKit - SwiftUI containment chain and how transactions can be propagated. Using UIViewRepresentable instead of UIViewControllerRepresentable therefore does not fix the issue.

LayoutReader can be updated to avoid the containment issue entirely.

There is a conflict between the route picker view and tap gestures we encountered before. We should report it to Apple, see #458.

SwiftUI animations do not flow down correctly across UIViewControllerRepresentable / UIHostingController boundaries reported to Apple as FB12663721.


refactor(project landing page): update metadata typings (DSP-1393)

resolves DSP-1393

@waychal I'm sorry, I was on the wrong PR and updated from main.

@waychal I'm sorry, I was on the wrong PR and updated from main.

Could you revert that pleae?

@kilchenmann No problem. I will check it. I normally rebase the main branch but merge is also fine.

@kilchenmann No problem. I will check it. I normally rebase the main branch but merge is also fine.

I'll fix it, if it's okay. Just a missing comma in app.module.ts and the build should run again

@mpro7 done in 73bfb09