add option to set name of the template literal tag to amplify codegen

Is your feature request related to a problem? Please describe.
I wrote the GraphQL code because the generated queries, mutations, and subscriptions have the limitation that the depth cannot be changed one by one.
To generate the variables and response types for that code, I added src/**/*.ts and src/**/*.tsx to the includes in .graphqlconfig.yml.
...
    schemaPath: amplify/backend/api/<API_NAME>/build/schema.graphql
    includes:
      - src/**/*.ts
      - src/**/*.tsx
    excludes:
      - ./amplify/**
    extensions:
...

And I run graphql codegen, I get the following error:
$ amplify codegen
✔ Generated GraphQL operations successfully and saved at src/graphql
✖ Could not parse graphql operations in src/components/ButtonLink.tsx
  Failed on : appearance-none inline-block px-10 py-3 rounded-full text-center whitespace-no-wrap

This is because template string literals that are not GraphQL code are recognized as GraphQL code.
Describe the solution you'd like
To avoid this, I need the tagName option to set a tag or magic comment to distinguish between template string literals in GraphQL code and others.
This option exists as a tagName argument in the generate function of graphql-types-generator, but has been forced to '' by amplify-codegen.
I need this option in a config file such as .graphqlconfig.yml.
Please let me know if there is any other way I can type variables and responses in my GraphQL code.

I found how to generate type of  queries, mutations, and subscriptions I wrote in https://github.com/aws-amplify/amplify-cli/issues/2420#issuecomment-537844610 .

You can add this query in src/graphl directory of your project in a separate file - say custom-queries.ts and then run amplify codegen again to regenerate the API.service.ts having the queries present in your custom-queries.ts file.

But I think this should be mentioned in the documentation.

for me it's important to define custom template literal tag. this should be possible to place queries near my components and not in one place.


ReferenceError: Can't find variable: angular

Hi,
when run this application through this error ReferenceError: Can't find variable: angular.
can anyone give the solution ?

Hi,
Did you follow the full install procedure, including the  "npm install" step ?
Sebastien

Hi,
i followed steps to installation but through this error,
D:\Projects\Examples\mean-master>npm install
npm WARN package.json<EMAIL_ADDRESS>No license field.

<EMAIL_ADDRESS>postinstall D:\Projects\Examples\mean-master
bower install --config.interactive=false

bower ENOGIT        git is not installed or not in the PATH
npm ERR! Windows_NT 6.1.7601
npm ERR! argv "C:\Program Files\nodejs\\node.exe" "C:\Program Files\nodejs
\node_modules\npm\bin\npm-cli.js" "install"
npm ERR! node v0.12.7
npm ERR! npm  v2.11.3
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>postinstall: bower install --config.interactive=false
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script 'bower install --config.i
nteractive=false'.
npm ERR! This is most likely a problem with the meanjs package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     bower install --config.interactive=false
npm ERR! You can get their info via:
npm ERR!     npm owner ls meanjs
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     D:\Projects\Examples\mean-master\npm-debug.log

Do you have bower installed correctly on Windows? It requires an extra step than mac/unix users
https://www.npmjs.com/package/bower#windows-users

The error clearly states that either git is not installed or is not in the
path. Can you confirm if you installed git?
On Thursday, July 16, 2015, Cody B. Daig<EMAIL_ADDRESS>wrote:

Do you have bower installed correctly on Windows? It requires an extra
step than mac/unix users
https://www.npmjs.com/package/bower#windows-users
—
Reply to this email directly or view it on GitHub
https://github.com/meanjs/mean/issues/650#issuecomment-121826167.

Hi,
Solved this issue, thank you very much..


Example for ICON Name Service

[x] Get address
[x] Get multihash
[x] Get resolver

Added in #17


Chrome login failed - Test run fails due to login redirection

Hello!,
After making my required changes and executing the test case, chrome launches and it connects to CRM, it enters the specified credentials, then chrome redirects to my organization page where in it expects the credentials to be specified again.  The code isn't able to handle that as it expects CRM to be launched by then.  I am able to get over this during debug as I can manually key it in to proceed.  This should be something you have encountered?

Hi,
The login procedure assumes single signon.
If you are connecting out side your domain, you need to connect through a VPN for ADFS to work.
Hope this helps
Regards,
Eduard

I believe you are running into an issue that I ran into right away when I was creating my automation test suite. Our previous QA engineer created a new class file called "LoginHelper" and this seemed to help a lot. Try this code and see if this works:
//create a new class under Microsoft.Dynamics365.UIAutomation.Sample.Web named "LoginHelper.cs".
using Microsoft.Dynamics365.UIAutomation.Api;
using Microsoft.Dynamics365.UIAutomation.Browser;
using OpenQA.Selenium;
namespace Microsoft.Dynamics365.UIAutomation.Sample.Web
{
public class LoginHelper
{
public static void ADFSLoginAction(LoginRedirectEventArgs obj)
{
IWebElement iframeElement;
IWebDriver Cdriver = obj.Driver;
Cdriver.FindElement(By.Id("passwordInput")).SendKeys(obj.Password.ToUnsecureString());
        //Cdriver.ClickWhenAvailable(By.Id("submitButton"), new TimeSpan(0, 0, 6));
        Cdriver.FindElement(By.Id("submitButton")).Submit();
        Cdriver.FindElement(By.Id("idSIButton9")).Submit();
        Cdriver.WaitForPageToLoad();
        try
        { //Workaround for the email not sent warning dialog in the QA environement.
            iframeElement = Cdriver.FindElement(By.Id("InlineDialog_Iframe"));
        }//close try
        catch (NoSuchElementException) { return; } //If there is no warning dialog we are done
        //TODO: Log presence of warning dialog
        //Since we have an Iframe switch to it
        Cdriver.SwitchTo().Frame(iframeElement);

        Cdriver.FindElement(By.Id("crmDialogFooter"));
        Cdriver.FindElement(By.XPath("//*[@id=\"crmDialogFooter\"]/div[2]"));
        //Due to issues with the framework click is not handled correctly and throws a no such element error
        // Thus the try catch wrapper.
        try { Cdriver.FindElement(By.XPath("//*[@id=\"butBegin\"]")).Click(); }
        catch (NoSuchElementException)
        { //Log that we closed a dialog window.
        }//close catch for Click action

    }//close Internal ADFS login Action

}//close login helper class

}
Now, in your TestMethods, you're going to have this default code intended for SSO, looking like this: xrmBrowser.LoginPage.Login(_xrmUri, _username, _password);
Edit that code and use this instead:
xrmBrowser.LoginPage.Login(_xrmUri, _username, _password, LoginHelper.ADFSLoginAction);
xrmbrowser.GuidedHelp.CloseGuidedHelp();

Hi,
The login procedure assumes single signon.
If you are connecting out side your domain, you need to connect through a VPN for ADFS to work.
Hope this helps
Regards,
Eduard

Hi Eduard - I am connecting from MS n/w itself.  Its just I am connecting with my v-id.  I don't think that should case the redirection and its something most of us would experience.
Regards,
Arvind

@ArvindBhat83 - As @sammcquistan suggested, and from your description of events, you'll need to utilize the alternate login method using a login helper redirect method. The intent behind this is to deviate from the standard O365 login experience and allow you to handle custom ADFS pages.
Depending on the element IDs for your custom forms login experience, you may need to make small tweaks to the suggested methods. Please let us know if you need any further assistance on this.
Regards,
Tyler


Restore Gradle source directory names in Kotlin DSL tooling API models

Follows up https://github.com/gradle/gradle/pull/30438
Fixes https://github.com/gradle/gradle-private/issues/4470
See https://ge.gradle.org/scans/tests?search.timeZoneId=Asia%2FShanghai&tests.container=org.gradle.kotlin.dsl.tooling.builders.r54.KotlinBuildScriptModelCrossVersionSpec for the caused failures

Reviewing cheatsheet
Before merging the PR, comments starting with

❌ ❓must be fixed
🤔 💅 should be fixed
💭 may be fixed
🎉 celebrate happy things

@bot-gradle test ACT

I've triggered the following builds for you. Click here to see all build failures.

All Cross-Version Tests for Ready for Release (Trigger)  (Ready for Release)


Added code-compile-run feature

Added code-compile-run feature using JDoodle API
Allowed languages are C, C++, Python, Java, and PHP
Made required changes on index.html, index.css, and js/home.hs
Thoroughly check the code and maintained the code standard
Did unit testing of my feature and all test were passed
No breaking changes

@BiswajitSahoo-tech there seems to be a conflict with the PR so please resolve it first. Also provide some screenshots of the changes that you made.

@lovishprabhakar Please check it now
Thanks for guiding

Nice work @BiswajitSahoo-tech!


update catalog drawer design

📝 Description
update catalog drawer design
🎥 Demo

please review @yaacov

What do you think if we split the component creating QuickCreateForm and LoadingFooter components?


Button issue in lumina-screenshot

Within MainUI.ui  there is a 'gap' before the 'Snap' button.
I tried to reorder them to fit the typical workflow within applications.
ie. new always being at the far left of a toolbar, next being open (not applicable in this application), then save functions, then all the edit functions, and quit always being at the far upper right.
Since that is the typical known order of things in virtual every program I tried to mirror that same order in the screenshot utility.
I tried to re-order lines  157 to 161 from:
   <addaction name="actionSave"/>
   <addaction name="actionquicksave"/>
   <addaction name="actionEdit"/>
   <addaction name="actionNew"/>
   <addaction name="actionQuit"/>

to

   <addaction name="actionNew"/>
   <addaction name="actionSave"/>
   <addaction name="actionquicksave"/>
   <addaction name="actionEdit"/>
   <addaction name="actionQuit"/>

However no matter where I place "actionNew" in the order, there is always a dead gap in front of it.  I've search through the source of everything else and can't seem to find the reason for this gap.  Ideally the gap could be between Edit and Quit... but I cant seem to make that work.  Is there something else causing this gap behavior?

 If you are going to re-arrange the toolbar actions you will probably need to move that spacer as well.


Used latest RemoteSecret's grafana dashboard

Include changes https://github.com/redhat-appstudio/remote-secret/pull/325

/lgtm


Unexpected token with Uglify

I'm getting the following error when trying to build for production (No problem with building for dev)
ERROR in static/js/vendor.d98368c92c87b257a0a0.js from UglifyJs
Unexpected token operator «=», expected punc «,» [./node_modules/timeslotjs/timeslot.js:6,0][static/js/vendor.d98368c92c87b257a0a0.js:16760,39]

The timeslotjs library is basicly using a default parameter in a function, which is a ES6 feature that UglifyJS2 doesn't support.
I'm not sure how to best solve this.
One way to solve it is to update the template to use uglify-es, instead of uglify-js.
Another solution is to update the template/webpack config to have babel transpile ./node_modules/timeslotjs/timeslot.js.
The problem that I see is that if the file/library isn't transpiled, the end-users browser would have to support ES6, no? Thus it might not be optimal to use uglify-es instead of uglify-js.
On the other hand, I'm not sure if it's "correct" to have babel transpile something from node_modules?
Is there another way to do it, maybe during build, the module is copied to assets?
Thoughts?

I use the EventEmitter module and I face a similar issue:
ERROR in static/js/vendor.11a3c006c10e183eb4a0.js from UglifyJs
Unexpected token: name (EventEmitter) [./~/EventEmitter/src/index.js:16,0][static/js/vendor.11a3c006c10e183eb4a0.js:53528,6]

I guess it's because of this line - the module uses:
export default class EventEmitter

If you are using an npm package written in ES6, it fails because:

Babel skips JS files under "node_modules" folder.
UglifyJS cannot handle ES6.

Possible solutions:

You can force babel to not skip "node_modules".
https://github.com/vuejs-templates/webpack/blob/master/template/build/webpack.base.conf.js#L51
  {
    test: /\.js$/,
    loader: 'babel-loader',
    include: [resolve('src'), resolve('test'), resolve('node_modules')]
  },

Maybe you are importing the source code of the package, which is in ES6. You could checkout its docs or readme, to see if the package provides an ES5 bundle.

I don't understand why this issue is closed? I have the same error with 2 other packages.
@myst729 1 solution doesn't work. I got tons of this kind of messages
⠸ building for production...[BABEL] Note: The code generator has deoptimised the styling of "/home/nman/Documents/WORK/EXAMPLES/vue-examples/node_modules/lodash/lodash.js" as it exceeds the max of "500KB".

And then I wait 5 minutes and cancel building. It's just freezes.

If I specify exact folder, like resolve('node_modules/vue-particles') then I still have the error
ERROR in static/js/vendor.7a706748dbe52517ba42.js from UglifyJs
Unexpected token: punc ()) [./~/menu-aim/src/index.js:1,0][static/js/vendor.7a706748dbe52517ba42.js:42492,2]

I don't understand why this issue is closed?

Because @myst729  explained how to solve the issue.
The only general solution in this template would be to have babel compile all code from /node_modules by default, which would make even small builds incredibly slow.


Skirmish feat description massively overflows the description box

I can shorten it a bit and leave the long description in the help.

My latest pull request should fix the issue.


Mirai only responding with

Mirai has only been responding to messages with "" and I can't figure out why

?

Mirai is only responding with  whenever I try to ask her something

This is not Mirai :)

how so?

Mirai is only responding with "html" but with <> like a script does

She's responding only with html not <>

Mirai is responding only with html

Did you ask her to be your web browser?

html

^

@BrooklynAQ Same with me dude! I don't understand how is this even possible!

I think she might be responding with  can anyone confirm?

Yes

Me too thanks

im sure thats how she usually respond

No she NEVER responded like that before
On Wed, Feb 15, 2017 at 2:33 PM, OrigamiSS<EMAIL_ADDRESS>wrote:

im sure thats how she usually respond
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Kwoth/NadekoBot/issues/1063#issuecomment-280130301,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AYmqArzZtiuzh1YePQu7FQjRiRaZDOoqks5rc2EagaJpZM4MB93K
.

She hadn't responded like that previously.

html


add support for enumerated field types

I have included your lib for my project at work, had to do few workarounds but most of the library worked for me.
It would be nice  if your library could handle enumerated field types as well. The changes and tests are included in my PR , I put the check for enumerated after checking for list type in query.py for the reason I don't know yet how to test for List[Enum] and I am not sure support for this type of field  is ever needed. Feel free to change suggest in my PR.

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.34 :tada:

Comparison is base (39870b8) 91.54% compared to head (f74b602) 91.89%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##           master       #2      +/-   ##
==========================================
+ Coverage   91.54%   91.89%   +0.34%
==========================================
  Files           4        4
  Lines          71       74       +3
==========================================
+ Hits           65       68       +3
  Misses          6        6

Impacted Files
Coverage Δ

gqlrequests/query.py
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

I'll take a look at this as soon as I have time! Thanks for the PR request 😄

Looks great! Thanks for adding the test as well.

Hey @venediktov, I've revisited this project and completely changed a lot of syntax. It should now support all the basic types of int, float, str and bool as well as enums and lists of any of the types (e.g. List[Enum]). If you're still using this package in some project, I would love to hear if the current implementation of enum still satisfies your usage 😃

Hi @BeatsuDev , I am gonna check with my python team and get back to you, thank you!

@venediktov it recently came to my attention that composing dynamic GraphQL queries already exists as a part of the gql python package. https://gql.readthedocs.io/en/latest/advanced/dsl_module.html
I just want to give you a heads up that if you're looking for a more stable solution, I suggest using that module instead. If you find this package useful, then I would recommend pinning the exact version of this package to avoid potentially breaking changes.
I haven't quite used this package enough or received enough issues to see if the current syntax and conceptual modelling of the solution holds up well in all types of GraphQL projects, so I don't feel comfortable bumping it up to a minor or major version yet (at which point I would start thinking of backwards compatibility).

@BeatsuDev sorry for not getting back to you earlier. So majority of our code uses gql , however it did not  not support DSL style coding or there was something else as I could not use  python data classes that would produce valid graphQL request format?
I could be wrong or misleading you, this is as far as I remember :-)

@venediktov I think you're right. A quick search didn't reveal any way of easily converting dataclasses to gql queries with gql's DSL at least... I'm glad this repository actually has a use-case then! 😃
The latest update of gqlrequests only attempts to improve the readability of the syntax and doesn't add much more features or fix any bugs - it just changes a lot. If you and your team do decide to upgrade to the latest version, it might be good to know that if you want to convert dataclasses to query strings you will first have to use the from_pydantic method to create a QueryBuilder (example below). Also, making a dataclass inherit from QueryBuilder now fails (also shown below).
All this is to say that if the library is working fine for your team as it is now, then I would suggest pinning the version to the one you are currently using. If your team upgrades, then here's the new syntax:
from dataclasses import dataclass
import gqlrequests

@dataclass
class Example:
    test: int

ExampleBuilder = gqlrequests.from_pydantic(Example)
print(ExampleBuilder().build())

This on the other hand DOES NOT WORK:
# DOES NOT WORK
@dataclass
class Example(gqlrequests.QueryBuilder):
    test: int

It seems like this PR was added after v0.0.7, but then I started rewriting a whole bunch and released it for v0.0.8... Meaning that supporting enums isn't in any version release before the rewrites starting v0.0.8 afaik... That's my rookie mistake as this should have definitely been released before I started rewriting everything...
If you need enum support, you might be required to update to the newer version of this package. If the new syntax requires too much work to implement in your code base, an alternative would be to fork the repo and reset back to this commit: bc123d95a05366e065f30eea3aba369916dbbbf3 and then build the package from source and it include it manually in the project. Sorry for the mistake...


Can't create gam project. ERROR: 403: Permission iam.serviceAccountKeys.create is required to perform this operation on service account

Confirming the following:

I have upgraded to the latest GAM release from https://git.io/gamreleases and I still have this issue - YES
I am typing the command as described in the GAM Wiki at https://github.com/jay0lee/gam/wiki - YES

Full steps to reproduce the issue:

Ensure GAM is installed and up-to-date
Attempt to set up a Google API project for GAM

Expected outcome (what are you trying to do?):

Successful GAM project creation

Actual outcome (what errors or bad behavior do you see instead?):
GAM is now installed. Are you ready to set up a Google API project for GAM? (yes or no) yes
Please enter your Google Workspace admin email address: [REDACTED]

Go to the following link in your browser:

	https://gam-shortn.appspot.com/[REDACTED]

Enter verification code: [REDACTED]
Creating project "GAM Project"...
Checking project status...

Creating Service Account
 Generating new private key...
 Extracting public certificate...
 Done generating private key and public certificate.
 Uploading new public certificate to Google...

ERROR: 403: Permission iam.serviceAccountKeys.create is required to perform this operation on service account projects/-/serviceAccounts/111146252721742176654. - 403
Project creation failed. Trying again. Say N to skip project creation.

This appears to be impacting others, as seen at https://groups.google.com/g/google-apps-manager/c/kKfd7lVIVaQ/m/3XvD5am2CwAJ

Seems to be a timing issue, the key upload succeeds if you try again a few seconds later. No idea why Google servers suddenly need more time before they honor the permissions but I'll get a patched build out soon to fix this.

This is now fixed in GAM 6.03.


[SPARK-8861][SQL]Add accumulators to SparkPlan operations

Prototype implementations of metrics accumulator update included in a LeafNode (LocalTableScan) and a UnaryNode (basicOperators#Project). Will extend to other SparkPlan ops after initial review.
Notes for reviewers:

Accumulator updates are currently achieved by prematurely calling actions, which is undesirable. Perhaps it may be better to instrument everything except for LeafNodes?

Merged build triggered.

Merged build started.

Test build #38050 has started for   PR 7590 at commit 1a4c9de.

Test build #38050 has finished for   PR 7590 at commit 1a4c9de.

This patch fails to build.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class Accumulator[T](

Merged build finished. Test FAILed.

Merged build triggered.

Merged build started.

Merged build triggered.

Merged build started.

Test build #38095 has started for   PR 7590 at commit 9629ce0.

Merged build finished. Test FAILed.

Test build #38095 has finished for   PR 7590 at commit 9629ce0.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class Accumulator[T](

Merged build finished. Test FAILed.

Merged build triggered.

Merged build started.

Test build #38099 has started for   PR 7590 at commit 1849fe3.

Test build #38099 has finished for   PR 7590 at commit 1849fe3.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Merged build triggered.

Merged build started.

Test build #38120 has started for   PR 7590 at commit 1d5fb86.

Test build #38120 has finished for   PR 7590 at commit 1d5fb86.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Merged build triggered.

Merged build started.

Test build #38732 has started for   PR 7590 at commit 8b9a36c.

@andrewor14 Test was failing because cacheTable was creating SparkPlans and I'm not sure that my fix is appropriate. I know you're working on something larger involving this instrumentation so do you mind checking to make sure this PR is useful? If not, I can close.

Merged build triggered.

Merged build started.

Test build #38745 has started for   PR 7590 at commit d5b070e.

Test build #38745 has finished for   PR 7590 at commit d5b070e.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test build #38732 has finished for   PR 7590 at commit 8b9a36c.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class Abs(child: Expression) extends UnaryExpression with ExpectsInputTypes

Merged build finished. Test FAILed.

Merged build triggered.

Merged build started.

Test build #38764 has started for   PR 7590 at commit a5455b1.

Test build #38764 has finished for   PR 7590 at commit a5455b1.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

@feynmanliang I think the changes here are useful. We do want to thread the memory statistics to the SQL operators as well. However, my parallel changes may conflict with this patch.

@andrewor14 Ok, I have some other things to do for QA so I will hold off on
this until you merge.
On Tue, Jul 28, 2015 at 3:58 PM andrewor14<EMAIL_ADDRESS>wrote:

@feynmanliang https://github.com/feynmanliang I think the changes here
are useful. We do want to thread the memory statistics to the SQL operators
as well. However, my parallel changes may conflict with this patch.
—
Reply to this email directly or view it on GitHub
https://github.com/apache/spark/pull/7590#issuecomment-125776599.

@feynmanliang Thanks for your PR. However, I resolved SPARK-8861 in #7774 in a similar way. Could you close this one?


Dependencies are not checked out automatically

Wiki is outdated:

All external libraries needed are checked out automatically with the main API library using Subversion's externals facility.

But since we are now using git instead of Subversion how to checked out external libraries automatically?

I think you can just git clone --recursive https://github.com/google/google-api-objectivec-client

@riannucci thanks. The doc should be updated anyway.

Yeah, it should :D I'm not a member of this project though (just a fellow interested personage) :)

Did a quick update on the wiki pages, hopefully I got the reference.
fyi - you can actually clone github wikis and send back pull requests for update the docs.


RPC reduce tests code complexity

Pull Request Test Coverage Report for Build 440431029

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

Totals

Change from base Build 440148062:
0.06%

Covered Lines:
2487

Relevant Lines:
7026

💛  - Coveralls


chain operator name

I guess that the chain operator name is pretty well-established, both here and outside the project. Iʼm thinking of possibility to alias it to flatMap, so it would align slightly better with JS API for arrays etc.
See also  similar issue with discussion elsewhere.
(Iʼm more used to of instead from for bind operator but thatʼs not a case with JS API.)

Hey @konradekk. Thank you for the issue.
I think, I could add an alias too for the chain method to Either and Maybe monads (but not in the Monad interface, because it will break compatibility).


results sorted by ADG with days on binance

didn't notice some of the configs were WE 0.3 and some WE 1.0
👎

This is nice though! Thank you! The recursive ADGs seem to be outperforming clock by a bit, but clock also handles uPNL better.


remove reference to outdated flag in SPM

https://bugs.swift.org/browse/SR-3407
"The default behaviour for swift build is to build without tests so the flag is not needed"

@ddunbar ?

The proposals in swift-evolution are part of the historical record, not the current documentation. I don't think we want to try and keep them up-to-date with how the command behaves today versus what it did when the proposal was authored.


Add nice to have question to new supplier flow

Part of this story on Pivotal https://www.pivotaltracker.com/story/show/129841633.
Tests will not pass until

pull request on the frameworks repo is merged.

This PR updates the supplier frontend to deal with the new nice-to-have-requirements question
The nice-to-have-requirements question uses the progressive disclosure pattern to collect the suppliers evidence. This requires an updated version of the govuk frontend toolkit.
The new version of the content loader fixes a bug with the unpacking of brief_response data for dynamic lists questions.
Finally, a small update to the update_brief_response view to only replay the form data from a POST request if the request fails validation. Previously the form date was being merged with the service data already pulled from the api, and this was causing issues with data being displayed when it shouldn't have been.
Also tests.

Going through the new flow while applying to a brief with no nice-to-have questions I noticed the page for this is still shown. We shouldn't show this question in this case.

@tombye Yep, that was a great shout. We've added a couple of bits to skip an optional question if it doesn't need answering, and to return 404 if someone tries to access said question directly.

Looks good to me 👍


gscrabble: init at  unstable-2019-03-11

Motivation for this change
i had to restore gnome3.gnome-icon-theme-symbolic , some review about it needed.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Fits CONTRIBUTING.md.

@jtojnar : i think all is good now ;)

Hmm, I am getting
/nix/store/sck8x99a3lsjlxwns4sgy1zaf8ll31hc-gvfs-1.38.1/lib/gio/modules/libgioremote-volume-monitor.so: undefined symbol: g_mount_operation_get_is_tcrypt_system_volume
Failed to load module: /nix/store/sck8x99a3lsjlxwns4sgy1zaf8ll31hc-gvfs-1.38.1/lib/gio/modules/libgioremote-volume-monitor.so

** (gst-plugin-scanner:9640): CRITICAL **: 12:18:04.914: pygobject initialization failed

This still fails to run for me:
Traceback (most recent call last):
  File "/nix/store/92wzfvip0pvkhzi44029hybm7vjmwxdh-gscrabble-unstable-2019-03-11/bin/..gscrabble-wrapped-wrapped", line 15, in <module>
    import gscrabble
  File "/nix/store/92wzfvip0pvkhzi44029hybm7vjmwxdh-gscrabble-unstable-2019-03-11/share/GScrabble/modules/gscrabble.py", line 11, in <module>
    gi.require_version('Gst', '1.0')
  File "/nix/store/bncy9d114kp08ln751f89qvgb674j6qf-python3.7-pygobject-3.30.4/lib/python3.7/site-packages/gi/__init__.py", line 129, in require_version
    raise ValueError('Namespace %s not available' % namespace)
ValueError: Namespace Gst not available

It seems the GI overrides from ${python3.pkgs.gst-python}/lib/python3.7/site-packages/gi/overrides are not being picked up.
I however don't know how the gio override magic works :-)

i've environment.variables.GIO_EXTRA_MODULES = [ "${pkgs.gvfs}/lib/gio/modules" ]; # lets PCManFM discover gvfs modules from https://nixos.wiki/wiki/Samba (that failed as all of this documentation on the wiki ) but perharps it covers me to see such bug. I should have a closer look at this.

i've git rebase and fix things, hope it's good now.

Got it to run without erroring out, but the language selection seems to be buggy.
Any idea what fixed the Namespace Gst not available error now?
What about the version number? Readme in the upstream repo mentions 0.1.2, but there seem to be neither tags nor release tarballs. Could you ask upstream to make a release (maybe after the language selection is fixed)?

the language selection seems to be buggy.

Yes you need to fix in the preference, it wont work from start screen.

Any idea what fixed the Namespace Gst not available error now?

Can't be sure since i didn't have this problem with my old git tree.

What about the version number? Readme in the upstream repo mentions 0.1.2, but there seem to be neither tags nor release tarballs. Could you ask upstream to make a release (maybe after the language selection is fixed)?

If there is no tagged i take the latest that fix things for us ( https://github.com/RaaH/gscrabble/issues/3 )
and since i already 2 PR pending on this, i'll wait finish french translation and more worried bug without ask for a release.

Well, I assume you'll maintain the package here and bump once the french translations or bugs are fixed, and take care asking upstream about a release ;-)

Started hitting the same issue as @jtojnar when trying to open a PDF with Zathura, which used to work just before the last channel update about 2 weeks ago.
/nix/store/cvzabqj2wiymkkrdw0l0mk6pw9rkdx4v-gvfs-1.40.1/lib/gio/modules/libgioremote-volume-monitor.so: undefined symbol: g_mount_operation_get_is_tcrypt_system_volume
Failed to load module: /nix/store/cvzabqj2wiymkkrdw0l0mk6pw9rkdx4v-gvfs-1.40.1/lib/gio/modules/libgioremote-volume-monitor.so

@FlorianFranzen looks like some kind of impurity. Please open a new issue.


Milestone 8: Add devise

[x] Remove current_user method in ApplicationController, devise will provide us with one.
[x] Install and setup devise.
[x] Can register a new user.
[x] User logs in with a combination of email and password.
[x] Hashed password should be stored in the database.
[x] Ask for confirmation of email.
[x] Can reset password.

Status: Invalid 🔴
Hi @Christianib003 🙋♀,
While you made a great effort in this project, I cannot proceed to review your code.
Invalid Code Review Request
You have submitted a project using the wrong link on the dashboard. You proposed using the add authorization rules link instead of the add devise link. Kindly use the correct link to submit your project.
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers 🥂  and Happy coding!!! 👯
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me @KDlamini  in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.


Implement importing a list of files

Allows <Import Project="import1.props;import2.props" /> to work.  This also updates Exists so that this will work:
<Import Project="import1.props;import2.props"
   Condition="Exists('import1.props;import2.props')" />

It also works with properties:
<Project>

  <PropertyGroup>
    <Imports>import1.props</Imports>
    <Imports>$(Imports);import2.props</Imports>
  </PropertyGroup>

  <Import Project="$(Imports)" Condition=" '$(Imports)' != '' And Exists($(Imports))" />
</Project>

It does not work with items because items are not available when evaluating <Import /> elements.
Closes #890

Looks good! We'll hold until Monday or Tuesday, this shouldn't go in the first preview of Update 3 and we haven't branched yet.


change on type of keyof in TS2.9 breaks some typings

From TS2.9 on, the type of keyof (unless the compiler option keyofStringsOnly is set) is now number | string | symbol (now AKA as PropertyKey) instead of just strings.
This made the following errors pop up:
build/0/compileTypescript/0  node_modules/@material-ui/core/index.d.ts:12:3 - error TS2344: Type '"classes" | Removals' does not satisfy the constraint
'string'.
build/0/compileTypescript/0    Type 'Removals' is not assignable to type 'string'.
build/0/compileTypescript/0      Type 'keyof C' is not assignable to type 'string'.
build/0/compileTypescript/0        Type 'string | number | symbol' is not assignable to type 'string'.
build/0/compileTypescript/0          Type 'number' is not assignable to type 'string'.
build/0/compileTypescript/0
build/0/compileTypescript/0  12   'classes' | Removals
build/0/compileTypescript/0       ~~~~~~~~~~~~~~~~~~~~
build/0/compileTypescript/0
build/0/compileTypescript/0
build/0/compileTypescript/0  node_modules/@material-ui/core/index.d.ts:57:77 - error TS2344: Type 'keyof U' does not satisfy the constraint 'string'.
build/0/compileTypescript/0    Type 'string | number | symbol' is not assignable to type 'string'.
build/0/compileTypescript/0      Type 'number' is not assignable to type 'string'.
build/0/compileTypescript/0
build/0/compileTypescript/0  57 export type Overwrite<T, U> = (U extends ConsistentWith<U, T> ? T : Omit<T, keyof U>) & U;
build/0/compileTypescript/0                                                                                ~~~~~~~

If you want to keep compatibility with old TS versions you can use this trick from the release notes:
If you intend on using --keyofStringsOnly and migrating off, instead of PropertyKey, you can create a type alias on keyof any, which is equivalent to string | number | symbol under normal circumstances, but becomes string when --keyofStringsOnly is set.

type KeyofBase = keyof any;


Update to RepoToolset 1.0.0-beta-62705-01

The new toolset includes fixes required by Orchestrated and Source builds.

OSX failure is due to network failure

@nguerrera @dsplaisted @rainersigwald Please review

@nguerrera I left the removal of Xliff package as is for now. I though I already fixed it in the toolset but turns out I didn't. Next update will be able to remove that.

@rainersigwald Could you please merge? We need the fix for orchestrated build.

What in orchestrated build is consuming our master branch? I thought nothing was (I am pleasantly surprised that something is).

I'm not sure about the branch. Perhaps this needs to be merged to other branches as well?

Right now our 15.7 branch is in the past, not using repotoolset. Now when we merge to it it'll get this. That's on our to-do list.


pkg_resources-style namespace package issue

Hello.
I have one issue related to namespace package. The project I am working on [0] is using pkg_resources-style namespace packages in a correct way described by documentation [1].
the process is:

During a build of a first subpackage as RPM package setuptools skip installation of <namespace-package>/__init__.py with message Skipping installation of /builddir/build/BUILDROOT/python-moksha-common-1.2.5-1.fc27.x86_64/usr/lib/python3.6/site-packages/moksha/__init__.py (namespace package)
Then I install newly created RPM from the step one.
When I try to build a second subpackage as RPM package (which has the first one as a dependency) build fails because the second subpackage needs to import the first one installed from RPM, but the first one doesn't contain init.py file and that is the reason why the namespace logic fails and the other sub-packages is not importable.

In docs [1] there is:

Every distribution that uses the namespace package must include an identical __init__.py. If any distribution does not, it will cause the namespace logic to fail and the other sub-packages will not be importable. Any additional code in __init__.py will be inaccessible.

The question is: Why setuptools skip installation of pkg_resources-style __init__.py file in namespace package when it must be included in every part of a namespace package?
My current workaround is to add missing files to RPM package manually and then everything works. I also tried to delete __init__.py files from the second subpackage before running tests with python setup.py test (where import of the first subpackage is needed) but then it fails with error in moksha.hub setup command: Distribution contains no modules or packages for namespace package 'moksha'.
I thought that the problem comes from Python 3.6 (bug report with more details and examples [2]) but it seems that Python 3.6 is only more strict in requiring __init__.py files everywhere as described in docs.
Maybe I just misunderstood something.
Thank you for help and have a nice day.
[0] https://github.com/mokshaproject/moksha
[1] https://packaging.python.org/guides/packaging-namespace-packages/#pkg-resources-style-namespace-packages
[2] http://bugs.python.org/issue29144

I have encountered this same issue with a debian package on Python 3 missing the namespace __init__.py files causing a ModuleNotFoundError with the plugins. Manually copying the __init__.py from source to relevant directories fixes the issue so I guess will have to do that in the debian package.

I may found part of the problem, where setuptools is no longer using nspkg.pth files: https://github.com/pypa/setuptools/issues/805
However the removal of __init__.py files predates that change so perhaps when not using nspkg.pth the namespace __init__.py files should not be skipped?
I tracked down the original commit with the reason for skipping __init__.py: https://github.com/pypa/setuptools/commit/92a61578a4b7d09ad6a6f9524163e67b62782e18
Just to add that I have tried in the past to move our application to native namespace but since we use eggs for plugins we need to use pkg_resources namespace so this is an annoying issue.

Thanks @cas for summary and renewed interest in this issue. Sorry for not having responded to this issue sooner.

Just to add that I have tried in the past to move our application to native namespace but since we use eggs for plugins we need to use pkg_resources namespace so this is an annoying issue.

I would consider eggs deprecated. The PyPA has a distinct interest in removing all support for eggs. Is there a reason you couldn't use wheels (even zip wheels) for your plugins? Would you consider filing a ticket with pypa/packaging-problems and mention me, describing your use-case and what feature of eggs you rely upon?

setuptools is no longer using nspkg.pth

This is not quite true. Consider, for example:
draft $ python -m venv env
draft $ env/bin/pip install -U setuptools
Collecting setuptools
  Using cached https://files.pythonhosted.org/packages/e7/16/da8cb8046149d50940c6110310983abb359bbb8cbc3539e6bef95c29428a/setuptools-40.6.2-py2.py3-none-any.whl
Installing collected packages: setuptools
  Found existing installation: setuptools 39.0.1
    Uninstalling setuptools-39.0.1:
      Successfully uninstalled setuptools-39.0.1
Successfully installed setuptools-40.6.2
You are using pip version 10.0.1, however version 18.1 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.
draft $ env/bin/pip install jaraco.functools --no-deps
Collecting jaraco.functools
  Using cached https://files.pythonhosted.org/packages/c2/f3/ce1bd8d5434227d26c2c2e07a9f89229a6b81122437452871d7e67f0ec76/jaraco.functools-1.20-py2.py3-none-any.whl
Installing collected packages: jaraco.functools
Successfully installed jaraco.functools-1.20
You are using pip version 10.0.1, however version 18.1 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.
draft $ cat env/lib/python3.7/site-packages/jaraco.functools-1.20-py3.6-nspkg.pth
import sys, types, os;has_mfs = sys.version_info > (3, 5);p = os.path.join(sys._getframe(1).f_locals['sitedir'], *('jaraco',));importlib = has_mfs and __import__('importlib.util');has_mfs and __import__('importlib.machinery');m = has_mfs and sys.modules.setdefault('jaraco', importlib.util.module_from_spec(importlib.machinery.PathFinder.find_spec('jaraco', [os.path.dirname(p)])));m = m or sys.modules.setdefault('jaraco', types.ModuleType('jaraco'));mp = (m or []) and m.__dict__.setdefault('__path__',[]);(p not in mp) and mp.append(p)
draft $ env/bin/python
Python 3.7.1 (v3.7.1:260ec2c36a, Oct 20 2018, 03:13:28)
[Clang 6.0 (clang-600.0.57)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import sys
>>> sys.modules['jaraco']
<module 'jaraco' (namespace)>

The work in #805 ended up creating nspkg.pth files that perform the same logic as PEP-420 namespace packages, using the same API calls (importlib.util.module_from_spec) on Python 3.5 and later. Still, as you can see, setuptools produces nspkg.pth files when creating wheels and pip installs them.
As you've noticed, the omission of of __init__.py for namespace packages goes back a very long time, to 0.6a11, so I dare say this behavior is by design. The declare_namespace call in __init__.py (and the similar call in pkgutil) was meant as a compatibility shim until namespace packages had proper support (which originally was meant to be pkg_resources in stdlib, but later evolved into PEP 420 as implemented in Python 3.3 and improved in Python 3.5).
What's surprising to me is that you shouldn't need the -nspkg.pth files on Python 3.5 or later at all (continuing from prior example):
draft $ env/bin/pip install jaraco.functools
Requirement already satisfied: jaraco.functools in ./env/lib/python3.7/site-packages (1.20)
Collecting more-itertools (from jaraco.functools)
  Using cached https://files.pythonhosted.org/packages/79/b1/eace304ef66bd7d3d8b2f78cc374b73ca03bc53664d78151e9df3b3996cc/more_itertools-4.3.0-py3-none-any.whl
Collecting six<2.0.0,>=1.0.0 (from more-itertools->jaraco.functools)
  Using cached https://files.pythonhosted.org/packages/67/4b/141a581104b1f6397bfa78ac9d43d8ad29a7ca43ea90a2d863fe3056e86a/six-1.11.0-py2.py3-none-any.whl
Installing collected packages: six, more-itertools
Successfully installed more-itertools-4.3.0 six-1.11.0
You are using pip version 10.0.1, however version 18.1 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.
draft $ rm env/lib/python3.7/site-packages/jaraco.functools-1.20-py3.6-nspkg.pth
draft $ ls env/lib/python3.7/site-packages/jaraco
__pycache__  functools.py
draft $ env/bin/python
Python 3.7.1 (v3.7.1:260ec2c36a, Oct 20 2018, 03:13:28)
[Clang 6.0 (clang-600.0.57)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import jaraco.functools
>>>

As you can see, even after removing the -nspkg.pth file, the namespace package imports just fine, even though there's no __init__.py.
You should be able to replicate this behavior very simply:
draft $ mkdir foo
draft $ python -q
>>> import foo
>>> foo.__spec__
ModuleSpec(name='foo', loader=<_frozen_importlib_external._NamespaceLoader object at 0x10c8a94a8>, submodule_search_locations=_NamespacePath(['/Users/jaraco/draft/foo']))

No -nspkg.pth file and no __init__.py. That's how it's meant to work.
The issue becomes if you install not to a one of the known site-packages directories on a Python 3.4 or earlier, the nspkg.pth file won't be loaded and the package won't load properly (on 3.3 or 3.4, it might work or there may be other issues).
So I'm unable to replicate the issue in the original report.
I suspect there's another factor. Maybe you have a mixed environment, where some of the paths have the namespace package with __init__.py and others do not. If that's the case, I'll need some assistance recreating the environment in which you're encountering the issue.
Would you do something for me? Would you work to distill the issue to as minimal a case as you can muster? If you're familiar with Docker, an easy way to share the replication is to build a Dockerfile that replicates the steps, especially if it's unique to a particular Linux environment.

@jaraco Thanks for the comprehensive reply!
I have created an issue to describe the plugins egg usage: https://github.com/pypa/packaging-problems/issues/212
I will attempt to distil the problem down but for reference the precise issue arose from .deb package install on Ubuntu Bionic/Cosmic with Deluge develop PPA. The workaround currently applied to the deb package is to manually copy over deluge/__init__.py and deluge/plugins/__init__.py. So to replicate my reported issue would require deleting those two __init__.py files in /usr/lib/python3/dist-package/deluge/

I went back and checked the debian package and realised that the nspkg.pth file was not being copied from source to the install. This never seemed to be an issue on Python 2 however apparently it does on Python 3, likely due to pkg_resources vs native namespace.
Therefore the proper packaging fix was not to restore the __init__.py files but ensure the nspkg.pth file exists since we are using pkg_resources namespace.


Rails hot reloading in development environment causes error in Dry::Container::Error

I have class in my project
class PhoneToken::Generate < Dry::Types::Struct
  ...
end

It works fine until I edit file containing this code and do a request. Rails tries to reload edited file which leads to this error:
Dry::Container::Error - There is already an item registered with the key "phone_token.generate"

The same behavior can be reproduced using reload! in rails console.

There's dry-data-rails made for dry-data, so it's a matter of renaming things there and pushing a new release. Please report an issue there. I'm gonna close this one as dry-types will never have any rails-specific code, so it's an external concern.

@solnic thank you for the link! I did not know that solution already exist.

I renamed things and renamed the gem to dry-types-rails (version 0.2.0).

@jeromegn great, thanks for the info!


Old broken tests

Some older test files, e.g. test_vector_api.py, are essentially deprecated and fail nosetests. If they are truly deprecated, we should delete all of them.

This was fixed by #82.


Docs improviments (Storm integration)

Description
After making tests and fixies the integration between rgb-node and storm-node, I decided to add information about storm integration works.
Related Issues

https://github.com/LNP-WG/lnp-node/pull/59
https://github.com/RGB-WG/rgb-node/pull/203
https://github.com/RGB-WG/rgb-node/pull/202
https://github.com/Storm-WG/storm-node/pull/2

Do we need to get all PRs listed in the description to be merged before merging this one?

Yes, I could transfer the consignment file only with all the fixes.

Pls check my review comment

Sure.


Use compartment_id to create the instance principal policy

@hyder , per our email conversation...
To create the policy we could use the compartment_id instead of compartment_name:
Allow dynamic-group base-bastion-instance-principal to manage all-resources in compartment id <ocid>
If we take this route, then we can remove compartment_name from the list of input variables. Please consider this suggestion.

merged in #5


Strange Adafruit TFT FeatherWing - 3.5" 480x320 issue

Hi
I'm using the Adafruit 3.5"  TFT Featherwing with an Adafruit Huzzah32 board and having an issue.
The display appears to be working fine using this example code in that it it shows the write/list/chart example screens. However the touch operation is not working, and every 3s roughly the screen switches to the next of the three tabs in the example cycling around them. I am using the configuration for the TFT featherwing in menuconfig but this appears to also select settings for the touch controller without a way to change the CS so I'm not sure if it is this.
I'm building on Windows with ESP-idf 4.0.1 and everything compiles and runs without error but not what I'd expect on screen.

The error comes from here:
https://github.com/lvgl/lvgl_esp32_drivers/blob/6b1e219a8711bba4439e06b738662be4f6d92ca1/lvgl_touch/stmpe610.c#L74-L78
0x811 is the STMPE_CHIP_ID reset value, maybe the spi configuration isn't properly done.

@C47D right I tried removing SPI_DEVICE_HALFDUPLEX and it hasn't changed things. I've also tried a 10k pulldown to ground on MISO as I couldn't find a 4.7k. So far neither has made a difference.
I do have a salae analyzer somewhere, I'll try to dig it out.
The display is working though and they're both on the same SPI bus so the comms should be working, has to be config on the touch device I think

@C47D right I tried removing SPI_DEVICE_HALFDUPLEX and it hasn't changed things. I've also tried a 10k pulldown to ground on MISO as I couldn't find a 4.7k. So far neither has made a difference.
I do have a salae analyzer somewhere, I'll try to dig it out.
The display is working though and they're both on the same SPI bus so the comms should be working, has to be config on the touch device I think

I suggested removing the SPI_DEVICE_HALFDUPLEX flag because the communication is full duplex.

I do have a salae analyzer somewhere, I'll try to dig it out.

Sure, thanks, please send the capture so I can take a look at it.

The display is working though and they're both on the same SPI bus so the comms should be working, has to be config on the touch device I think

Sharing the SPI gave me problems before, but I haven't had time to check the code, my pc was very slow and I gave up, we might be missing something silly on the configuration.

I suggested removing the SPI_DEVICE_HALFDUPLEX flag because the communication is full duplex.

I do have a salae analyzer somewhere, I'll try to dig it out.

Sure, thanks, please send the capture so I can take a look at it.

The display is working though and they're both on the same SPI bus so the comms should be working, has to be config on the touch device I think

Sharing the SPI gave me problems before, but I haven't had time to check the code, my pc was very slow and I gave up, we might be missing something silly on the configuration.

Hi,
I came also around this problem. I'm using Adafruit HUZZAH32 with Adafruit 3.5" Featherwing with ESP-IDF v4.4. With the Arduino component from Espressif it's working fully (Display, Touch, SD card). But not straight forward with lv_port_esp32 / lvgl_esp32_drivers. I would like to get rid of the Arduino component by switching to these projects.
Is it still possible to get help here?
Unfortunately, I don't have a logic analyzer. But I could give help by doing specific tests or having an interactive remote test session or even donate the hardware to somebody who would have the knowledge to deep-dive into problem solving.
Thanks a lot


[VS Code] Joining collaboration session: The specified value isn’t a valid Live Share URL. Please check the link provided by the host and try again.

Error:
Joining collaboration session: The specified value isn’t a valid Live Share URL. Please check the link provided by the host and try again.
Steps to Reproduce:

Version Data

extensionName
VSLS

extensionVersion
1.0.18

protocolVersion
2.2

applicationName
VSCode

applicationVersion
1.32.3

platformName
Linux

platformVersion
4.15.0-46-generic

a

@Emmanuelgg Probably by design. You'll get this message if you paste something other than a valid link in the join dialog.


Exposing client Staleness property in AppUpdateInfo

It would be nice if the plugin exposes the Client Staleness value so that the host app can decide to show felxible update / immediate update .
I know there is a priority for the same , but i feel this value can also be used in case the users are using an older version of the app.

Hi @Rahul0106231,
thank your for your request!
I assume you mean clientVersionStalenessDays.
Great idea!


Do an apt-get upgrade after the apt-get update

Without an apt-get upgrade after apt-get update Java doesn't install itself anymore.
I get this error:
    amazon-ebs: Some packages could not be installed. This may mean that you have
    amazon-ebs:
    amazon-ebs: requested an impossible situation or if you are using the unstable
    amazon-ebs: distribution that some required packages have not yet been created
    amazon-ebs: or been moved out of Incoming.
    amazon-ebs: The following information may help to resolve the situation:
    amazon-ebs:
    amazon-ebs: The following packages have unmet dependencies:
    amazon-ebs: oracle-java8-installer : Depends: java-common (>= 0.24) but it is not installable
    amazon-ebs: Recommends: gsfonts-x11 but it is not installable
    amazon-ebs: E: Unable to correct problems, you have held broken packages.

LGTM


Not responding to level horizon command

Current master (20160208) does not respond to level horizon command from QGC
Only verified on VTOL vehicle
All other sensor calibrations work correctly
QGC does not show a progress bar and no info messages on the matter are being sent making it appear as if the command was never received. However, the QGC version used to test has successfully ran this command on earlier versions
/cc @AndreasAntener

@DonLakeFlyer any idea?

Debugged through QGC side. Calibration message is being sent by QGC but no response from firmware.

@tumbili can this be related to your recent commit?
https://github.com/PX4/Firmware/commit/afb01e6d9aab2e7feb32125948d55d0066565a9e

@bugobliterator bisecting this issue points to any of your following commits:
https://github.com/PX4/Firmware/commit/c9ac15f0ddb54483194e4c8b14e3f246338c9f5a
https://github.com/PX4/Firmware/commit/f811777789ed9901e7f94d3a1d9f4adfbdd8beb4
https://github.com/PX4/Firmware/commit/78b8deda1592f16785d585d456bc2d294767f253
Can you review?

@sanderux sure I will have a look

Fixed in https://github.com/PX4/Firmware/commit/081c40c6fa2045560f80b0c0707f400eb31b7ff4


Does TTTAttributedLabel support paragraph head indent?

I'm having a hard time figuring out if TTTAttributedLabel supports all the styling of NSParagraphStyle including headIndent. From what I can tell, it doesn't appear to. If I were to implement this, where would I start?

Yes, see the dedicated firstLineIndent property. You may also enjoy #494 and #540.

Thanks @jhersh, I should of been more clear. From what I can see the firstLineIndent applies to the whole text rather than sub paragraphs within an attributed string. I'm currently using DTCoreText to format html, but using TTAttributedLabel for the label. DTCoreText create sub paragraphs with different indentation depending on the html (like an ordered list for example), but TTAttributedLabel doesn't seem to fully respect the indents from what I ca tell.

That's correct. Anything specific to add here, or perhaps we might close this in favor of #494?

I'll move over there. :)


SyntaxError: Unexpected token o

I get the error of
                                            SyntaxError: Unexpected token o
    at Object.parse (native)
    at Z:\casu\system\node_modules\scribe-js\lib\logWriter.js:81:45
    at FSReqWrap.readFileAfterClose [as oncomplete] (fs.js:380:3)
sdfsdfsd

from commandline i run
node server.js

which is
//setup logging
var scribe      = require('scribe-js')({createDefaultConsole : false}); //loads Scribe
var console     = scribe.console({logWriter : {rootPath : ".\\logs\\Server"}}); //set log dir app
//setup global vars
global.scribe   = scribe;
require('./app/runtest'); //run app

runtest.js is simply:
setTimeout(function() {
    console.log('sdfsdfsd')
    process.exit();
}, 500);

If i put process.exit() outside, everything works
Any operation\method\async that I run, it throws the error, but data will still complete
It was working fine last week (on different files, havent tested them again because they are in production.  I upgraded my dev of node to 4.1.1 and removed\installed clean scribe and scribe-js
when looking at logWriter.js:81, I added a console.log(data) and the output is not in correct JSON format
{"dates":{"1441944000000":["logs\\Server\\2015\\Sep\\11_sep_15.log.json","logs\\Server\\2015\\Sep\\11_sep_15.error.json"],"1442030400000":["logs\\Server\\2015\\Sep\\12_sep_15.log.json","logs\\Server\\2015\\Sep\\12_sep_15.error.json"],"1442203200000":["logs\\Server\\2015\\Sep\\14_sep_15.log.json","logs\\Server\\2015\\Sep\\14_sep_15.error.json"],"1442808000000":["logs\\Server\\2015\\Sep\\21_sep_15.log.json","logs\\Server\\2015\\Sep\\21_sep_15.error.json"],"1442894400000":["logs\\Server\\2015\\Sep\\22_sep_15.log.json","logs\\Server\\2015\\Sep\\22_sep_15.error.json"],"1442980800000":["logs\\Server\\2015\\Sep\\23_sep_15.log.json"]}}ogs\\Server\\2015\\Sep\\23_sep_15.error.json"]}}

Notice the last line, it has an extra:
}ogs\\Server\\2015\\Sep\\23_sep_15.error.json"]}}

I rewrote that file form scratch and didnt get an error, but now working on another file and getting that same error
doit('http:\\www.something.com\getthisfile.zip','c:\\helloworld.zip')

function doit(url,localfile) {
    async.waterfall([
        function(callback) { //download file
            //### if i do console.log here, this line outputs ##Before error
            if (url) {
                var file = fs.createWriteStream(localpath + localfile);
                http.get({
                    url: url,
                    binary: true
                }, function (err, res){
                    if(err) {
                        console.error(err);
                        console.log('Error in Download')
                        callback(null)
                    }
                    else {
                        file.write(res.body);
                        file.end(function () {
                         //### if i do console.log here, this line outputs ##After error
                          callback(null)
                        })
                    }
                });
            }
            else {
                console.log('Bad URL')
                callback(null)
                //### if i do console.log here, this line outputs ##After error
            }
           //### if i do console.log here, this line outputs ##Before error
        },

        function (callback){
           //### if i do console.log here, this line outputs ##After error
            //some processing

       }], function (error, results) { //done
           //####never gets here
    });
}

I don't know the cause of this, but the fix is to blow away your logs directory. Per what you indicated, it's likely because something was written incorrectly. It's known that calling fs.write multiple times on the same file without waiting for the callback is unsafe; I assume the scribe writes are non-blocking, so that may be the issue, that multiple calls to log aren't properly serialized.

Hello,
I'm facing the same problem, if the server crash for any reason the file is corrupted and the only solution that i have found is to dlete the logs folder. Is there a fix or roundabout for this?
Thanks in advance

Dev branch has a new build using a mongodb backing. perhaps that might be of use


Add support for all platforms riff support

Includes additional support for MacOS arm, Linux arm and Linux x86_64.

Just noticed the hash is wrong for at least Intel Linux. Will double check them all and update the PR tonight.

Just noticed the hash is wrong for at least Intel Linux. Will double check them all and update the PR tonight.

Done!


Albania players are wrong (same as Swiss players)

The list of Albania is actually the list of Switzerland.

Thanks. Fixed now.


Excise support for Boost Wave

We don't need it anymore, we use clang library preprocessor on all
configurations now.

LGTM!

This is your kind of patch, @fpsunflower !


Update build_spec.yaml

AUTH TOKEN is breaking the build .

removed token from build spec.

merged build spec


API Response: Add count of number of is_* attributes

When a post receives feedback from multiple users, it'd be helpful to know how many of each type (is_fp, is_tp, and is_naa) were received.
For example, post https://metasmoke.erwaysoftware.com/post/44634 received 2 is_tp and 1 is_naa and has a response object that looks like this:
{   'has_more': False,
    'items': [   {   u'body': <snipped for brevity>,
                     u'created_at': u'2016-10-25T09:50:45.000Z',
                     u'downvote_count': None,
                     u'id': 44634,
                     u'is_fp': False,
                     u'is_tp': True,
                     u'link': u'//superuser.com/a/1138708',
                     u'post_creation_date': None,
                     u'score': None,
                     u'site_id': 3,
                     u'stack_exchange_user_id': 37707,
                     u'title': u'What are all the "about:" pages, in Internet Explorer?',
                     u'updated_at': u'2016-10-25T10:11:20.000Z',
                     u'upvote_count': None,
                     u'user_link': u'//superuser.com/u/656344',
                     u'user_reputation': 1,
                     u'username': u'user656344',
                     u'why': u'Post - Contains 10 unique characters'}],
    'page': 1,
    'total': 1}

If there could be three new attributes showing the count of each, that'd be helpful. Example:
count_tp: 2,
count_fp: 0,
count_naa: 1,

An alternative to adding three new attributes, is changing the existing (and new) is_* attributes from a boolean to an integer.

Gonna take this one. Shouldn't be much more than moving an existing route to use a jbuilder view and a join. Possibly a day or two before I can work on it, though.

Gonna have to drop this one again - unexpected things came up, and I'm gone for a few days to sort them out. Will get round to it when I get back, if @Undo1 doesn't want it.

Currently FDSC does this by making a second request to the /api/post/:id/feedback route, then counting the number of feedbacks of each type. You could use that approach in the meantime until Art gets a chance to fix it.
Although, it would be much more efficient (both time and network wise) to include this in the original request.

@ArtOfCode- any updates on the status of this?


Write unit tests for configurations

Write unit test for created configuration class
Write unit test for classes that extend DataProvider
Write unit test for classes that extend ServerIdentifier
Write unit test for manager classes

Solved in #3.


[HUDI-1357] Added a check to ensure no records are lost during updates.

What is the purpose of the pull request
Add a check to ensure there is no data loss when updating HUDI dataset
Brief change log

Added a new HoodieWriteConfig setting to enable data loss checks
Added a new stat to HoodieWriteStat which tracks the number of records written to older version of the data file
When data loss check is enabled:

Before HoodieMergeHandle is closed, it reads the last version of the data file (if present) and find the number of records written in that and compares against the current number of records written

Verify this pull request
This change added tests and can be verified as follows:
Added a unit test to ./hudi-client/hudi-spark-client/src/test/java/org/apache/hudi/client/TestHoodieClientOnCopyOnWriteStorage.java
Can be verified by running that unit test as follows:
mvn test -pl hudi-client/hudi-spark-client -Dtest=TestHoodieClientOnCopyOnWriteStorage
Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Codecov Report

Merging #2216 (da2ff45) into master (6310a23) will decrease coverage by 7.46%.
The diff coverage is 90.00%.

@@             Coverage Diff              @@
##             master    #2216      +/-   ##
============================================
- Coverage     53.67%   46.21%   -7.47%
+ Complexity     2849     2100     -749
============================================
  Files           359      282      -77
  Lines         16565    12628    -3937
  Branches       1782     1236     -546
============================================
- Hits           8892     5836    -3056
+ Misses         6916     6363     -553
+ Partials        757      429     -328

Flag
Coverage Δ
Complexity Δ

hudicli
38.37% <ø> (ø)
0.00 <ø> (ø)

hudiclient
100.00% <ø> (ø)
0.00 <ø> (ø)

hudicommon
55.31% <90.00%> (+0.62%)
0.00 <4.00> (ø)

hudihadoopmr
?
?

hudispark
?
?

huditimelineservice
?
?

hudiutilities
10.39% <ø> (-59.70%)
0.00 <ø> (ø)

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


Learning DE Roadmap

A FAQ in the community is a structured roadmap for learning Data Engineering and it's about time we start addressing it. We currently have a getting started guide but it's not detailed enough and was meant to be improved on anyways.
It can be a complex question to answer but we can simplify it by adding a few constraints. Since the majority of folks asking are those who are new to DE or trying to transition we should focus on skills for junior/entry level and mid level roles. While there aren't many jr roles at the moment it can still be useful to make the distinction for foundational skills. To make it as general as possible, I believe we should exclude tools/requirements that only apply to FAANG-like companies since they are more niche and oftentimes FAANG companies have developed their own internal tooling to solve their unique problems. Finally, the focus should be on core concepts instead of tooling. While we can include specific tools, we should try to avoid directly recommending specific tools and instead point learners to pages that have lists of the current popular tools to keep this resource as evergreen as possible (example: workflow orchestration popular tools).
While I don't believe a diagram is a requirement, I do think it could be helpful if we can get it to render nicely in mermaid because we can then make it interactive and link to other notes in the wiki like we do with other diagrams. The canvas feature for Obsidian publish is not yet supported so we would probably use a mermaid flowchart for now.
Existing popular roadmap shared in the community:

Data Engineer Roadmap 2021 by datastacktv

Visually appealing
I like how they distinguished nice to have skills
Comments under major topics explaining why it's necessary is helpful
Just found that there's a text version as well: https://github.com/datastacktv/data-engineer-roadmap/blob/master/text/roadmap.md
Overall I really like it but it could probably be broken up by role or skill level to make it more manageable
Looks like the company is now defunct so the roadmap hasn't been updated (maybe we can adopt the repo?)

For V1, please share any thoughts/ideas/constructive criticism on the structure and core concepts. I'll start a new branch after Christmas and start something we can work from.

https://awesomedataengineering.com/

Here are couple of ideas:

Anything that is not a core data concept should be after core concepts. Ex. Anything related to testing should be way after SQL. You should not be writing integration or even unit tests if you cant build a working pipeline.
Core concepts can be defined as: Concepts that can be found most of the junior/mid level data engineers day-to-day tasks

SQL
Programming Language
Storage (S3 / Harddrive)
OLAP / OLTP Database
Orchestrator (I would even put this as three, it is core of batch processing which is most of the industry)
Basic data modelling (data types, partitions, maybe primary, foreign, unique keys and indices)
Git
So no Streaming, NoSQL, MapReduce, Hadoop etc.

After core concepts, more advanced and less used one can be given such as:

Distributed Computing, Hadoop, MapReduce
Spark
NoSQL (Just to understand more dist. computing and to pull / push data)
Docker and basic infrastructure and networking
Advanced data modelling (Snowflake, Star schema, One Big Table)
Streaming (I believe this is way too advanced for beginners)
Anything that I missed or more related to general software engineering (Tests, CI/CD etc.)
Cloud and advanced infrastructure (maybe Kubernetes)

Like anything else, there's basic and advanced streaming, it's not a niche skill nor is it gated to senior engineers. IMO It's a huge disservice for a junior level not to have basic familiarity with where the industry is heading.

I just created a page to start playing around with. I figured we'd start with entry level -> mid level roles since that's what most people are looking for. I'll add more to this later but feel free to make edits.

I just created a page to start playing around with. I figured we'd start with entry level -> mid level roles since that's what most people are looking for. I'll add more to this later but feel free to make edits. https://github.com/data-engineering-community/data-engineering-wiki/tree/49-roadmap

In my opinion databases should be part of the computer science
"computer resources" would probably just be called "Operating Systems"
So computer science fundamentals would be something like:
Operating Systems -> Networks -> Databases or ->Databases->Networks
But generally not sure if the whole thing is not just reinventing the wheel at the end.


'aws:assumeRole' is not optional

What happened?
Either the Docs or the cli is wrong, one says assumeRole is required, the other that it is optional.
Given that a role is already provided to be assumed, I guess the cli is wrong.
Example
Using this example from
https://www.pulumi.com/registry/packages/aws/installation-configuration/
config:
  aws:assumeRoleWithWebIdentity:
    roleArn: arn:aws:iam::<ACCOUNT_ID>:role/<ROLE_NAME>
    # Define either webIdentityToken or webIdentityTokenFile
    webIdentityToken: <your-web-identity-token>
    webIdentityTokenFile: webidentitytokenfile.txt

I get the error:
 $ pulumi stack select "${STACK}"

Previewing update (test):
 +  pulumi:pulumi:Stack baseline-test create
@ previewing update.....
 +  pulumi:pulumi:Stack baseline-test create error: Missing required configuration variable 'aws:assumeRole'

Eventhough assumeRole is said to be Optional
Output of pulumi about
$ pulumi about
CLI
Version      3.113.3
Go Version   go1.22.2
Go Compiler  gc
Host
OS       debian
Version  11.9
Arch     x86_64
Backend
Name           pulumi.com
URL            https://app.pulumi.com/
User           Unknown
Organizations
Token type     personal
Pulumi locates its logs in /tmp by default
warning: Failed to read project: no Pulumi.yaml project file found (searching upwards from /builds/idg3/migration/Content). If you have not created a project yet, use `pulumi new` to do so: no project file found
warning: Failed to get information about the current stack: no Pulumi.yaml project file found (searching upwards from /builds/idg3/migration/Content). If you have not created a project yet, use `pulumi new` to do so: no project file found

Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

@Askill this is a strange one! I'm not aware of anywhere that we are requiring aws:assumeRole to be set. Couple of questions.

What is the version of the AWS provider you are using?
Can you give us a minimal example that we can use to reproduce?
Where are you storing your config? Do you have any configuration that is depending explicitly on aws:assumeRole? For example, something like this would also cause that error const role = new pulumi.Config("aws").require("assumeRole");

That could be the case, we got it working after we removed the usage of some custom modules one of our other teams build
const portfolioAccess = new GrantAccessToServiceCatalogPortfolio(   "twa-building-blocks-access",   { portfolioParameter: TwaPortfolioIdParameter } ); // needs to be done only once per account
GrantAccessToServiceCatalogPortfolio is a custom function, I haven't had the chance to check what exactly the function does, but I assume your guess was correct.
Feel free to close the issue, as it seems to be specific to us, however: I do think the documentation should not call the parameter outright "optional"

And thank you for you very quick response! :D


Learning Path Exercise Failed

Does this occur consistently?  Yes
Repro steps:

Follow the steps documented at https://docs.microsoft.com/en-us/learn/modules/automatic-update-of-a-webapp-using-azure-functions-and-signalr/7-exercise-host-a-static-website-using-a-storage-account
2.Do all steps up to and including "Deploy the web application to Azure Storage". This is the point where the failure occurs.

Action: azureStorage.deployStaticWebsite
Error type: Error
Error Message: AzCopy Transfer: "Failed".
Version: 0.11.0
OS: linux
OS Release: 4.4.0-19041-Microsoft
Product: Visual Studio Code
Product Version: 1.52.1
Language: en

Call Stack
extension.bundle.js:37:15387
extension.bundle.js:37:13040
n extension.bundle.js:37:12726
handleJob extension.bundle.js:37:14031
Object.<anonymous> extension.bundle.js:37:15435
fulfilled extension.bundle.js:37:12794

@robidas are you using WSL or Remote SSH? Or is linux your host OS?

@robidas are you using WSL or Remote SSH? Or is linux your host OS?

WSL
Ubuntu 20.04.1 LTS (GNU/Linux 4.4.0-19041-Microsoft x86_64)
running on a Win10 pro guest vm, which in-turn is running on a win 10 pro hyper-v host

From: Will Lorey<EMAIL_ADDRESS>Sent: January 15, 2021 3:24 PM
To: microsoft/vscode-azurestorage<EMAIL_ADDRESS>Cc: Robi (Rob) Das<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [microsoft/vscode-azurestorage] Learning Path Exercise Failed (#917)
@robidashttps://github.com/robidas are you using WSL or Remote SSH? Or is linux your host OS?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/vscode-azurestorage/issues/917#issuecomment-761229602, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AELMSXMR4ZCPEYNO5WRD3BLS2C6CLANCNFSM4WBTX3IA.

WSL
Ubuntu 20.04.1 LTS (GNU/Linux 4.4.0-19041-Microsoft x86_64)
running on a Win10 pro guest vm, which in-turn is running on a win 10 pro hyper-v host

From: Will Lorey<EMAIL_ADDRESS>Sent: January 15, 2021 3:24 PM
To: microsoft/vscode-azurestorage<EMAIL_ADDRESS>Cc: Robi (Rob) Das<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [microsoft/vscode-azurestorage] Learning Path Exercise Failed (#917)
@robidashttps://github.com/robidas are you using WSL or Remote SSH? Or is linux your host OS?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/vscode-azurestorage/issues/917#issuecomment-761229602, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AELMSXMR4ZCPEYNO5WRD3BLS2C6CLANCNFSM4WBTX3IA.

@robidas You can fix this by installing libsecret-1-dev in WSL:
sudo apt update
sudo apt install libsecret-1-dev

It looks like the packages installed by default in Ubuntu 20.04 on Windows are pretty minimal. This package appears to be included in standard Ubuntu.

@robidas You can fix this by installing libsecret-1-dev in WSL:
sudo apt update
sudo apt install libsecret-1-dev

It looks like the packages installed by default in Ubuntu 20.04 on Windows are pretty minimal. This package appears to be included in standard Ubuntu.

This issue has become stale and is at risk of being closed. The community has 60 days to upvote the issue. If it receives 5 upvotes we will keep it open and take another look. If not, we will close it. To learn more about how we handle issues, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this issue has received less than 5 community upvotes and we closed it. Still a big Thank You to you for taking the time to create it! To learn more about how we handle issues, please see our documentation.
Happy Coding!


Looking forward to get involved and contribute to the community 🎉

First Name
Mahar
Last Name
Zeeshan
Discord Username (if applicable)
zeeshanmahar#9894
Let us understand your motive.
I have seen your post about SidHacks on LinkedIn.By joining this community want to learn new things also take part in upcoming projects and hackathon and learn more about open source. I will contribute to my community by arrange session for them where they can get to know about new trend in technologies and ask their questions.

Hey @zeeshanmahar007 👋🏻
Thanks for requesting an invite! Welcome to SidHacks Community 🎉
Before going ahead, I would want you to complete the below To-Do list.
Your To-Do List ✔

Accept the invitation sent to you via mail.
Change the appearance of this organization to "Public" from the people list here.
Join our Discord Server here.

Accept the Rules from #📓〡rules channel.
Introduce yourself to our community in 👋〡introduction channel.

Feel free to share your excitement on socials with #SidHacksCommunity. You can also tag me @siddharth_hacks.

Close this issue once you complete all the above points.

Looking forward to interacting with you in our community! Welcome once again! 🤓


Remove settings.xml so it doesn't interfere with builds

Because settings.xml defines a bunch of repos that aren't updated, we should just remove it before the build starts

The error we're currently seeing that this is attempting to fix is:
[WARNING] Failure to transfer commons-logging:commons-logging/maven-metadata.xml from https://repository.apache.org/snapshots/ was cached in the local repository, resolution will not be reattempted until the update interval of apache-snapshots has elapsed or updates are forced. Original error: Could not transfer metadata commons-logging:commons-logging/maven-metadata.xml from/to apache-snapshots (https://repository.apache.org/snapshots/): Connect to repository.apache.org:443 [repository.apache.org/<IP_ADDRESS>] failed: Connection timed out

This doesn't seem to happen on developers machines.