Installing issues

I was following your instructions (which are kinda not clear), but app doesn't start after npm run dev
`jd@jd-XPS:~/next/next.js$ npm run dev

<EMAIL_ADDRESS>dev /home/jd/next/next.js
next

sh: 1: next: not found
npm ERR! Linux 4.4.0-47-generic
npm ERR! argv "/usr/bin/nodejs" "/usr/bin/npm" "run" "dev"
npm ERR! node v6.9.1
npm ERR! npm  v3.10.8
npm ERR! file sh
npm ERR! code ELIFECYCLE
npm ERR! errno ENOENT
npm ERR! syscall spawn
npm ERR<EMAIL_ADDRESS>dev: next
npm ERR! spawn ENOENT
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>dev script 'next'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the next package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     next
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs next
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls next
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     /home/jd/next/next.js/npm-debug.log
`
My actions:

Cloned the repo && cd
Run npm install next --save
Add in package.json "dev": "next"
Created file ./pages/index.js with mentioned in manual content
Then I run npm run dev

Do you have nextin your folder node-modules?
Which version of node and npm are you running?

Cloned the repo && cd

Do you mean you cloned the zeit/next.js repo ? If so, follow the instruction with an empty dir.

Ok, now it work, thank you (still, I don't know why I couldn't did it first time, it was saying "Can't install dependency as itself" or kind of)

I'm having the same issue with Node v7.8.0 and npm v4


Reflection: Update extra inhabitants logic for structs.

Structs now use extra inhabitants from whichever field has the most, not only the first field. rdar://problem/44239246

@swift-ci Please test

cc @milseman

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - 68e60dd51d645d767b3898da76f3e8978e46aeef

Build failed
Swift Test OS X Platform
Git Sha - 68e60dd51d645d767b3898da76f3e8978e46aeef

@swift-ci Please test Linux


List Argument Suggestions only provide suggestion for first Argument

As is stated on the title, when you create a suggestion for the List type, it only provides a suggestion for the first args of the type, everything else gets no suggestion. It could be an implementation issue, but I'm not entirely sure. Below is the implementation code I have created.
Code:
public class SettingParameter extends ArgumentResolver<CommandSender, List> {
  @Override
  protected ParseResult<List> parse(Invocation<CommandSender> invocation, Argument<List> context, String argument) {
    return ParseResult.success(Collections.singletonList(argument));
  }

  @Override
  public SuggestionResult suggest(Invocation<CommandSender> invocation, Argument<List> argument, SuggestionContext context) {

    if(argument.getKeyName().equalsIgnoreCase("world-setting")) {
      return SuggestionResult.of(PhantomWorlds.createTabs);
    }
    return SuggestionResult.of(new ArrayList<>());
  }
}

//command method parameters.
public void create(@Context CommandSender commandSender, @Arg("world name") final String name, @Arg("environment")World.Environment environment, @OptionalArg("world-setting") List<String> settings)

images:
First param:

Second Param:

You can create an argument resolver for string SettingParameter extends ArgumentResolver<CommandSender, String> and suggest that what do you want.
Then register it with the custom key:
.argument(String.class, ArgumentKey.of("world-setting"), new SettingParameter<>(suggester))
and use it in your code:
@Arg("world-setting") List<String> settings

But if you want to suggest only the first argument, maybe it is better to create argument resolver for one argument with your custom type e.g. SettingOption?


Class system

With a mission option, alternative loadouts system that enforces actual classes instead of people being able to play rambo (a side effect of the option to remove fatigue entirely...)

I'll have a look into that, but if I'll implement it, then only as optional parameter.
That's because I don't want to create real restrictions for groups which are capable of organizing themself in a senseful way.
For only 24/7 public server this could be more interesting, sure.


CSS path completion "./" only suggests "." files/folders

1.22 insiders on Mac. Testing #46537.
I expected all folders/files to show:

Likely, the filter string must be set to include the beginning of the edit.

Can this be related to #46638?


Wrong body padding-right on modal with fixed-top nav

This padding-right in body on modal active is wrong when using nav with .fixed-top
https://jsfiddle.net/rorc/nke1dvpz/3/

The same problem with .sticky-top navbar. Here might be something wrong with JavaScript when it is calculating the body width.


I am interested in using your game on my website. Could you add a license file like the MIT one in this pull request so I know how you want your software used?

Okaay I'll, and please stay in touch and let me know what you think about the game !


[Kernel] Add exception principles for Kernel to solidify the rules for our exception framework

Which Delta project/connector is this regarding?

[ ] Spark
[ ] Standalone
[ ] Flink
[ ] Kernel
[ ] Other (fill in here)

Description

How was this patch tested?

Does this PR introduce any user-facing changes?

I think we may need to think through how to handle the exceptions happening the lazy iterators. Should we pass some context to them when creating the iterators? This can be a separate task, need not block this PR.

I think this would be nice to have but might be a non-trivial design/code change (not sure how we'd incorporate this?) for a smaller gain in debuggability. I'm viewing most of these exception changes as best effort to improve our error messages... this seems more complex than we need right now.


Add KerbalGalaxy2 Remaster from SpaceDock

This pull request was automatically generated by SpaceDock on behalf of XxLedvin25xX, to add KerbalGalaxy2 Remaster to CKAN.
Please direct questions about this pull request to XxLedvin25xX.
Mod details:
name = /mod/2146/KerbalGalaxy2%20Remaster
author = XxLedvin25xX
abstract = This mod adds a galaxy, with solar systems orbiting a black hole, and wormhole near Jool that leads to the Gargantua system.
license = CC-BY-NC-SA
Homepage = https://forum.kerbalspaceprogram.com/index.php?/topic/184275-beta17x-kerbalgalaxy2-remaster/
description =
[BETA][1.7.x]KerbalGalaxy REMASTER

This mod is a BETA, can contain many errors, I recommend waiting for version 1.0, the BETA can damage your SAVE, be careful, YOU HAVE BEEN WARNED!

The Original Mod Made By Artyomka15.
Link To Orginal Mod "KerbalGalaxy2"
Remastered By: XxLedvin25xX and Olympic1
License: CC-BY-NC-SA

You Need Kopernicus, Module Manager and TextureReplacer
Download the Textures From HERE

1- Download Kopernicus, ModuleManager and TextureReplacer for your game version.
2- Download The "[BETA] (1.7.x) V0.8 KerbalGalaxyRemaster".
3- Download the "KGRemaster Textures".
4- Drop "GameData" in your folder "KerbalSpaceProgram" and replace all.
5- Enjoy!

This mod is only an update for the new versions I do not intend to earn credit, or money, or anything, only that the community enjoys.

This mod adds a galaxy, with solar systems orbiting a black hole, and wormhole near Jool that leads to the Gargantua system, this mod adds the following stars with their planets:

WhormHole in Jool to Gargantua

POLARIS

Green ocean planet PANDORA

Yellow gas giant RYLA

Icy moon HOTH

Moon CITE

Brown Dwarf NIM

Red dwarf ERAVATE

Planet SEHERA

Planet EDEN

Blue supergiant Oblok

Gas giant AELION

Dwarf planet PHOENIX

Dwarf planet SUNFLOWER

Neutron star Osiris

Planet Ida

Gas planet Isus

Moon Rocko

Moon Volco

Yellow-White star Abbadon

Planet Violetta

Planet Nix

Moon Icy

Planet Aphrodite

Gargantua system

Miller's planet

Mann's planet

Bugs and Reports

The KSC is misplaced on the first screen, but the rockets are still working the same and the takeoff zone is the same, the map inside the ship is not affected by this, but in the research center if it is affected.

ckan compat add 1.6

Install instructions:
3- Download the "KGRemaster Textures". Link http://bit.ly/KGRTex (This link is only a shortener and does not contain ads)

That link:

@Olympic1, you're listed as co-author. What's going on with this texture download?

#rebuild

Don't use the shortened link as that needs a key for some reason, use the link on SD or the thread. I don't know if we can let CKAN download from mega.nz though.

I don't know if we can let CKAN download from mega.nz though.

Yeah, doesn't look like it, at least not from that URL. I keep getting a rich page instead of a download, and  wget 'https://mega.nz/#!eHplXKzT!2PhEZFAI0B7TDbRUv3jXQ_VITX-W6A4vMouYVb8krmY' saves an index.html file instead of the zip file.
Is there any other way to access this file? (For example, put it in the same download on SpaceDock?)

Deleted from SpaceDock.


Location is missing for some Places

yup it was implemented later on and was an option for someone to add location if they want with in an image card.
if you want to add locations to the image cards ...i can assign it....
comment if interested

I have added for some of them, I have forwarded your repo to my friends and told them about the locations, they will do it by tomorrow or day after tomorrow...


We cannot verify this contract | unidentified contract error

Suddenly i started to getting this error and my nft's started listing like this

Also in polygon network, im not be able to approve my nfts. Transactions gettin stuck at pending.
What is happening i cant get it...

https://rinkeby.etherscan.io/address/0x57f1887a8bf19b14fc0df6fd9b2acc9af147ea85#code
^ I don't think that 0x57f1887a8bf19b14fc0df6fd9b2acc9af147ea85 on Rinkeby is your NFT contract.
Your OpenSea link is viewing that contract.
I think this 0xd0dd53b77bd01a9992a8b3d5ecde29c4a9a3fdf2 should be yours:
https://rinkeby.etherscan.io/address/0xd0dd53b77bd01a9992a8b3d5ecde29c4a9a3fdf2#code
https://testnets.opensea.io/assets/0xd0dd53b77bd01a9992a8b3d5ecde29c4a9a3fdf2/0

My main question is why my contracts not verified on metamask page? Why am i getting this:

Why is your cost<PHONE_NUMBER>0000000000 wei, which is 40 ETH?
Are you sure you want to charge 40 ETH for a NFT?

I will deploy my contract on Polygon network so i wanted to set my price as 40 MATIC

I see.
I'm able to deploy the contract and mint myself.

I'm able to mint and approve myself now too but in polygon main network we cant approve our nfts because all transactions are stuck on pending. I guess there is a error about polygon network? Am i right?

Does setApprovalForAll work on Rinkeby? If so, should work fine on Polygon.
Sometimes, it could be due to the default Metamask RPC (you may want to replace it with an Alchemy RPC).
Send me the etherscan link for the pending tx.

https://docs.alchemy.com/alchemy/guides/connecting-metamask-to-alchemy

Im getting this error when i tried to add alchemy... ):

Why my contract is not verifying? How can i fix that?

Clear / cancel your pending tx.

I verified my contract with hardhat verify and its verified on etherscan. I still have no idea why im getting this warning for all my contracts...
Also i have an another question can i mint an nft for someone with erc721a? I mean, i mint one nft for you and this nft's owner will be you but i will pay gas-fee for you.

You can direct mint to others, but the token IDs must be in sequential order.


Improve trades.get_any performance

The function has changed a lot and looks more complicated and has different arguments now, but it is very faster for big amounts of datas, so its possible to run it on rokko. Please tell me if I should change anything. I use it for the experiments, maybe we can think about a definitive version of it.

@FelixKleinpeter Can I review it? you already finished implementation?

I will finish to set it usable for deployment. At the moment I focused on increasing the speed. It will be done when you will come back from your holidays!

I fixed types and add a new function. The code is more complicated but the new function makes possible to run on rokko. Thanks to apply your review!

Last changes are done. Can I merge it?


not working on ios9 and camera plugin

camera plugin and this crop plugin not work correctly together on ios 9. the crop never start.
var cameraPopoverHandle = navigator.camera.getPicture(function(cameraImageURI){//ok

    plugins.crop.promise(cameraImageURI)

    .then(function(newPath){
        //success crop
    })
    .catch(function(err){
        //error crop
    });

},function(message){
    //error camera
},{
    quality:100,
    destinationType: Camera.DestinationType.NATIVE_URI,
    sourceType:Camera.PictureSourceType.CAMERA,
    correctOrientation: true
});

doesn't work on iOS

@jeduan Would be nice to have this fixed 😄

@ljubic no longer working on Cordova. PRs accepted! 😊

@jeduan  can you make it working in ios?

@dhrumil162 literally the comment above yours says I'm no longer working on this project.

For some time I also thought it does not working, my fault, the plugin is working perfectly.. maybe you guys need to debug on safari to see the logs.
here is my app
https://itunes.apple.com/app/firetask-classic/id1229505010
@jeduan thanks for sharing

plug-in is working in phone gap not in all cross platform apps
On May 7, 2017 7:50 PM, "Stewan"<EMAIL_ADDRESS>wrote:

For some time I also thought it does not working, my fault, the plugin is
working perfectly.. maybe you guys need to debug on safari to see the logs.
here is my app
https://itunes.apple.com/app/firetask-classic/id1229505010
@jeduan https://github.com/jeduan thanks for sharing
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jeduan/cordova-plugin-crop/issues/29#issuecomment-299709342,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQbpGQuHMNq5bsPdBpebrId3X1glLc9Aks5r3dMUgaJpZM4KXlMX
.

ok my app is on both apple store and google play
https://play.google.com/store/apps/details?id=io.firetask
https://itunes.apple.com/app/firetask-classic/id1229505010


bind with --mode instead of --sets-mode

Whoops! So, --sets-mode changes the current mode. We want to set the binding for the current mode instead. Totally different thing. Thank you!


Automated cherry pick of #3684: fix: 避免有些云不能创建default名称安全组

Cherry pick of #3684 on release/2.13.
#3684: fix: 避免有些云不能创建default名称安全组

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.13@1f51836). Click here to learn what that means.
The diff coverage is 0%.

@@              Coverage Diff               @@
##             release/2.13   #3685   +/-   ##
==============================================
  Coverage                ?   6.95%
==============================================
  Files                   ?     500
  Lines                   ?   83918
  Branches                ?       0
==============================================
  Hits                    ?    5838
  Misses                  ?   77584
  Partials                ?     496

Flag
Coverage Δ

#aFlag
6.95% <0%> (?)

Impacted Files
Coverage Δ

pkg/multicloud/aws/securitygroup.go
0% <0%> (ø)

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 1f51836...4bb9685. Read the comment docs.

/lgtm

/approve


Ensure compatibility with older modules

It's important we take some time to verify the integrity and compatibility of the current version of Nevermore against modules designed for prior versions of Nevermore, such as the one hosted by Quenty/NevermoreEngine.

What do you mean?

I think we're good on this one.


Add CJK fixed-pitch font remarks for xAvgCharWidth

When using CJK fixed-pitch font on Windows, double of xAvgCharWidth scaled value seems to be used for full-width glyph advance. In practice, the value should be a half-width glyph width, not an average of all non-zero width glyphs. So, we should add remarks regarding these points.
xAvgCharWidth description in OpenType 1.4+ specification says "The Average Character Width parameter specifies the arithmetic average of the escapement (width) of all non-zero width glyphs in the font". If a CJK fixed-pitch font follows the description, xAvgCharWidth value is nearly full-width glyph width because it has over 6,000 full-width ideograph glyphs and only 158 half-width glyphs. Using the font on Windows, it will show unnatural horizontal spaces between full-width characters, e.g. Hiragana and Ideograph.
MS Gothic is the typical Japanese fixed-pitch font. Though it's OS/2 table version is 3, xAvgCharWidth is half-width glyph width, i.e. 128 while UnitsPerEm is 256.

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

ID: 1ae9457c-0d3d-c137-588b-81d54a086f07
Version Independent ID: 658c6d22-b9d1-1007-9cdb-6898911c9c75
Content: OS/2 - OS/2 and Windows metrics table (OpenType 1.9) - Typography
Content Source: typographydocs/opentype/spec/os2.md
Product: typography
GitHub Login: @PeterCon
Microsoft Alias: alib

Some older fonts might have been updated with the OS/2 version changed but with the xAvgCharWidth left the same. Strictly speaking, that doesn't conform to the spec, though the font developers might have done that because they thought there was less compatibility risk, or maybe they overlooked it (e.g. maybe they updated the OS/2 version to take advantage of a newer field and forgot to check this).
In any case, applications should no be relying on xAvgCharWidth for getting glyph advance widths or calculating layout of lines of text.

See the OT 1.9.1 alpha for draft revisions addressing this issue.

I'm not aware of any issues using MS Gothic in GDI.

MS Gothic seems to follow the old calculation method for xAvgCharWidth in OS/2 table.
Have you tried a CJK fixed-pitch font using new (v3) calculation method for xAvgCharWidth in OS/2 table in GDI?
If you haven't, I suggest trying out VL-Gothic 20200720. It's avaiable from https://osdn.net/projects/vlgothic/downloads/73361/VL-Gothic-Regular.ttf/ .  As seen in the following screenshot, it shows unnatural spaces between Japanese letters in GDI:

At least, the specification should note this compatibility risk.

I suspect this might be an application-specific issue rather than something affecting GDI itself. I don't have access to a Windows machine this week to investigate but will do so next week.

This is fixed in OT 1.9.1. See the OT 1.9.1 beta for draft revisions addressing this issue.


Issue #713: Better placement for 'create a guide' inside a crop's page Resolved

-Added the "Make your own!" link at the top for each crop. Just below the "Guides for this crop" text.
-Added a conditional in the guides_results_list view to show the create a guide button only for crops with more than 10 guides.
I've tested the conditional and it works.
Let me know if any changes are required. Thanks. :)

Coverage remained the same at 99.358% when pulling 2f644e022840401ef28396c8a99a766d04ac2b19 on TanSA05:master into 73bb26e1b481310ac1b6cd95ab78c553da192331 on openfarmcc:master.

Coverage remained the same at 99.358% when pulling b32d75c9603513903166e4697a90048e235fe90a on TanSA05:master into 73bb26e1b481310ac1b6cd95ab78c553da192331 on openfarmcc:master.

Coverage remained the same at 99.358% when pulling b32d75c9603513903166e4697a90048e235fe90a on TanSA05:master into 73bb26e1b481310ac1b6cd95ab78c553da192331 on openfarmcc:master.

Thanks for this PR @TanSA05! Thanks also for listening to the Hound. It can come off as pretty picky, but it's a pretty easy way of keeping a joint coding style. I'll have a look at the changes now.

Coverage remained the same at 99.358% when pulling 25dcd10434b649593d670332eb6781a760dcb576 on TanSA05:master into 73bb26e1b481310ac1b6cd95ab78c553da192331 on openfarmcc:master.

Looks awesome! I've made one comment above, and then I think it's ready for merging :)

Coverage remained the same at 99.358% when pulling 45170d4f4a51f0e06bfc1109998eb4f2c956412d on TanSA05:master into 73bb26e1b481310ac1b6cd95ab78c553da192331 on openfarmcc:master.

Coverage remained the same at 99.358% when pulling 1256438c276cdeef82e75a1755214297eacb002f on TanSA05:master into 73bb26e1b481310ac1b6cd95ab78c553da192331 on openfarmcc:master.

Merged!
I've also just removed .vagrant from the files that are tracked by git. It's now in the .gitignore file, and you won't have to delete it. You should see the changes live on staging in a couple of minutes!

Just checked it out on staging! :+1: :+1:

Thanks for testing @roryaronson, it's now live on production as well!

Hey.
This fixes Issue #713. Close it, please?
Thanks! :D

Done! PS, if you add into your PR's title "Closes #713", then when the PR is merged, #713 will automatically close!


 Turning interactive mode on.
QXcbConnection: XCB error: 145 (Unknown), sequence: 171, resource id: 0, major code: 139 (Unknown), minor code: 20
device: cuda:0
then it stops ,and not return anythings

second

I added a dockerfile that solves this:
https://github.com/WenmuZhou/PAN.pytorch/pull/43 @Atlanticsea


SQL Server: Unable to resolve field, the connection was closed

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
This bug happens only on SQL Server with SqlConnection, to make Hot Chocolate unable to use at all. The error is about the dbcontext closed too early (sharing dbcontext) even using Pooled and DbContextFactory, creating new dbcontext every action and await on them. The problem happens 50/50 chance happen. Tried on SqlLite is fine, so only happen when use Sql Server.
Steps to reproduce

Clone https://github.com/ChilliCream/graphql-workshop

Got to Step 8, switch to use SqlServer instead of SqlLite

At Startup.cs

            services.AddDbContextFactory<ApplicationDbContext>(
                (s, b) =>
                {
                    var connection = new SqlConnection
                    {
                        ConnectionString = "Server=(localdb)\\mssqllocaldb;Database=ConferenceDB;MultipleActiveResultSets=True;"
                    };

                    b.UseSqlServer(connection);
                });

Launch BCP,  run the query with 2 or more resolvers, eg:

query {
  sessions {
    nodes {
      title
      attendees {
        firstName
      }
      speakers {
        name
        sessions {
          title
        }
      }
      track {
        name
      }
    }
  }
}

5 . Try to run couple times, you will see error
The connection was not closed. The connection's current state is open.
Relevant log output
"errors": [
    {
      "message": "Unexpected Execution Error",
      "locations": [
        {
          "line": 10,
          "column": 9
        }
      ],
      "path": [
        "sessions",
        "nodes",
        0,
        "speakers",
        0,
        "sessions"
      ],
      "extensions": {
        "message": "The connection was not closed. The connection's current state is open.",
        "stackTrace": "   at System.Data.ProviderBase.DbConnectionInternal.TryOpenConnection(DbConnection outerConnection, DbConnectionFactory connectionFactory, TaskCompletionSource`1 retry, DbConnectionOptions userOptions)\r\n   at System.Data.SqlClient.SqlConnection.TryOpen(TaskCompletionSource`1 retry)\r\n   at System.Data.SqlClient.SqlConnection.OpenAsync(CancellationToken cancellationToken)\r\n--- End of stack trace from previous location ---\r\n   at Microsoft.EntityFrameworkCore.Storage.RelationalConnection.OpenInternalAsync(Boolean errorsExpected, CancellationToken cancellationToken)\r\n   at Microsoft.EntityFrameworkCore.Storage.RelationalConnection.OpenInternalAsync(Boolean errorsExpected, CancellationToken cancellationToken)\r\n   at Microsoft.EntityFrameworkCore.Storage.RelationalConnection.OpenAsync(CancellationToken cancellationToken, Boolean errorsExpected)\r\n   at Microsoft.EntityFrameworkCore.Storage.RelationalCommand.ExecuteReaderAsync(RelationalCommandParameterObject parameterObject, CancellationToken cancellationToken)\r\n   at Microsoft.EntityFrameworkCore.Query.Internal.SingleQueryingEnumerable`1.AsyncEnumerator.InitializeReaderAsync(DbContext _, Boolean result, CancellationToken cancellationToken)\r\n   at Microsoft.EntityFrameworkCore.SqlServer.Storage.Internal.SqlServerExecutionStrategy.ExecuteAsync[TState,TResult](TState state, Func`4 operation, Func`4 verifySucceeded, CancellationToken cancellationToken)\r\n   at Microsoft.EntityFrameworkCore.Query.Internal.SingleQueryingEnumerable`1.AsyncEnumerator.MoveNextAsync()\r\n   at Microsoft.EntityFrameworkCore.EntityFrameworkQueryableExtensions.ToListAsync[TSource](IQueryable`1 source, CancellationToken cancellationToken)\r\n   at Microsoft.EntityFrameworkCore.EntityFrameworkQueryableExtensions.ToListAsync[TSource](IQueryable`1 source, CancellationToken cancellationToken)\r\n   at Microsoft.EntityFrameworkCore.EntityFrameworkQueryableExtensions.ToArrayAsync[TSource](IQueryable`1 source, CancellationToken cancellationToken)\r\n   at ConferencePlanner.GraphQL.Types.SpeakerType.SpeakerResolvers.GetSessionsAsync(Speaker speaker, ApplicationDbContext dbContext, SessionByIdDataLoader sessionById, CancellationToken cancellationToken) in C:\\Users\\alwo\\source\\repos\\GraphQL\\GQL-Workshop\\SQLDB.NET5\\GraphQL\\Types\\SpeakerType.cs:line 40\r\n   at HotChocolate.Resolvers.Expressions.ExpressionHelper.AwaitTaskHelper[T](Task`1 task)\r\n   at HotChocolate.Types.FieldMiddlewareCompiler.<>c__DisplayClass3_0.<<CreateResolverMiddleware>b__0>d.MoveNext()\r\n--- End of stack trace from previous location ---\r\n   at HotChocolate.Types.ScopedServiceObjectFieldDescriptorExtensions.<>c__DisplayClass0_1`1.<<UseScopedService>b__1>d.MoveNext()\r\n--- End of stack trace from previous location ---\r\n   at HotChocolate.Types.ScopedServiceObjectFieldDescriptorExtensions.<>c__DisplayClass0_1`1.<<UseScopedService>b__1>d.MoveNext()\r\n--- End of stack trace from previous location ---\r\n   at HotChocolate.Execution.Processing.ResolverTask.ExecuteResolverPipelineAsync(CancellationToken cancellationToken)\r\n   at HotChocolate.Execution.Processing.ResolverTask.TryExecuteAsync(CancellationToken cancellationToken)"
      }
    }
  ]

Additional Context?
I have created another sample to demonstrate this issue: https://github.com/alnw/HCContoso
Product
Hot Chocolate
Version
12.5

Why are you instantiating the SqlConnection yourself? This might not be the issue, but I have not seen the usage of SqlConnection outside of really barebone implementations using SqlCommand. Unless there is a specific reason you are doing this, I would suggest changing your code to this:
services.AddDbContextFactory<ApplicationDbContext>( (b) => b.UseSqlServer("Server=(localdb)\\mssqllocaldb;Database=ConferenceDB;MultipleActiveResultSets=True;"));

This is just speculation on my part since normally the SqlConnection has to also be cleaned up and I'm not sure how nice it plays with the DbContextFactory.

@tobias-tengler Woooo !! Thanks for your prompt response. You are correct!!!! Looks like once I use your suggestion, my demo project seems working. This is great news!
However in our production environment we are using that for access token, here is the sample code
var tokenProvider = new AzureServiceTokenProvider("....");
var connection = new SqlConnection
{
ConnectionString = "connection-string",
AccessToken = tokenProvider.GetAccessTokenAsync(".....").Result,
};
Do you know why this could fail when using HC?

I looked around a bit and found that generally the AccessToken will be set from within an interceptor, when connecting to the Database. I even found this example regarding what you are trying to do: https://docs.microsoft.com/en-us/ef/core/logging-events-diagnostics/interceptors#example-connection-interception-for-sql-azure-authentication-using-aad
With this you can continue using my code from above and also set the AccessToken, without having to manage the SqlConnection yourself. Just copy over the interceptor from the example, make your necessary changes and call .AddInterceptor(new AadAuthenticationInterceptor()) after the UseSqlServer(...).
I'm closing this issue now, since I suspect the behavior was caused by the pooled DbContext factory itself and not Hot Chocolate, so it's more of an EF Core issue. Feel free to report back whether what I proposed is working for you or what you ended up doing in the end though, as reference for other developers. :)

Thanks @tobias-tengler, yes the solution you proposed is promising. I think we should all avoid using SqlConnection as HC suggested using Pooled DbContext, this should be noted somewhere in HC. Thanks a lot for this issue, really appreciate it!


albums endpoint should have country/market param

Very simple but the albums(albums) endpoint should be albums(albums, country). According to Spotify Get Several Albums.

Just "album" is also missing a market/country param.

You probably refer to https://spotipy.readthedocs.io/en/2.19.0/#spotipy.client.Spotify.albums or https://spotipy.readthedocs.io/en/2.19.0/#spotipy.client.Spotify.album which looks for specific album(s) while the market parameter already exists on https://spotipy.readthedocs.io/en/2.19.0/#spotipy.client.Spotify.artist_albums, see country

Those are different endpoints...Artist_albums does not give any track information while spotify albums / album does, therefore have different responses / data. https://spotipy.readthedocs.io/en/2.19.0/#spotipy.client.Spotify.albums and https://spotipy.readthedocs.io/en/2.19.0/#spotipy.client.Spotify.album should have a "country" param.

Has this been added yet?


Focal Point not working, if not the full url path to the image

I have an vertical image and I set the focal point in the asset interface to the top, because the face is in the top section of the image.
I need to create a square thumbnail image. To generate the thumbnail I use https://example.com/api/cockpit/image and the direct output o=1.
The collection returns a relative path to the image like /storage/uploads/2018/10/04/image.jpg. So this must be the url to the thumb (of course... src url encoded):
https://example.com/api/cockpit/image?token=XXX&q=80&w=200&h=200&m=thumbnail&src=/storage/uploads/2018/10/04/image.jpg&o=1
But this url does not affected the focal point. If i add the full path of cockpit to the src, it works:
https://example.com/api/cockpit/image?token=XXX&q=80&w=200&h=200&m=thumbnail&src=https://example.com/storage/uploads/2018/10/04/image.jpg&o=1
It seems, that cockpit does not recognize that the image is an asset:
https://github.com/agentejo/cockpit/blob/2d79c26856799f179ef91d776165d1867e430b35/modules/Cockpit/bootstrap.php#L125
With relative path to the asset $fp is always center
https://github.com/agentejo/cockpit/blob/2d79c26856799f179ef91d776165d1867e430b35/modules/Cockpit/bootstrap.php#L227

if you're using assets, then please use the asset _id as src e.g:
https://example.com/api/cockpit/image?token=XXX&q=80&w=200&h=200&m=thumbnail&src=5a00e80002ac8doc1362529026&o=1

cockpit then also get the focal point settings

if you're using assets, then please use the asset _id as src e.g:
https://example.com/api/cockpit/image?token=XXX&q=80&w=200&h=200&m=thumbnail&src=5a00e80002ac8doc1362529026&o=1

cockpit then also get the focal point settings

That's great @aheinze , was not aware that fp would be handled by the thumbnail function automatically.
Updated the image styles addon so when generating styles for assets they will take the fp in consideration.
Think the asset field could be improved by providing a quick way to access the asset details (and set there the focal point), or at least provide some events that can handled:

otherwise, you need to click on replace and use the edit from the listing
what do you think?

@pauloamgomes yes, I think that would be a great improvement. I'll add it to my todo list

@aheinze I did something on PR  #921  that extracts the editing functionality from cp-assets into cp-asset, so it can be reused in the field. From my tests, it seems to work well, but kindly review and see if that make sense.


fix rubocop complains about initializer file name (use snake_case)

What's this PR do?

use snake_case for initializer file name.
surpress rubocop complains that use snake case for initializer file name.

Where should the reviewer start?
README.md
Any background context you want to provide?
config/initalizers/rack-attack.rb is recommended in README.md
What are the relevant tickets?
RuboCop complains about initializer file name · Issue #275 · kickstarter/rack-attack

ran into this too

@tjnet Sounds good, thanks

@grzuy thank you for you review! 😄


6.0 Feedback:  Protocols with long names overlap in protocol run history

Overview
1315X840
Current Behavior

Expected Behavior
No response
Steps To Reproduce
No response
Operating system
Windows
Robot setup or anything else?
No response

[ ] Need the design team support for spacing

Ask about adding padding to the protocol to Emily and Mel. If they agree with that as a solution, we can add it.
If not, we should close this since actually there is no overlap.

@koji - any update on this?

@nusrat813 This will be solved by #11054


Cannot update CEP: StorageError: invalid object

Likely not a serious error so the work here may be to hide this error to not raise unnecessary concerns:
level=error msg="Cannot update CEP" containerID=cilium-loc controller="sync-to-k8s-ciliumendpoint (45702)" endpointID=45702 error="Operation cannot be fulfilled on ciliumendpoints.cilium.io \"cilium-health-ip-172-0-52-103.us-west-2.compute.internal\": StorageError: invalid object, Code: 4, Key: /registry/cilium.io/ciliumendpoints/kube-system/cilium-health-ip-172-0-52-103.us-west-2.compute.internal, ResourceVersion: 0, AdditionalErrorMsg: Precondition failed: UID in precondition: 9118baef-8e28-11e8-a7dc-029cdb62bcae, UID in object meta: " ipv4=<IP_ADDRESS> ipv6="f00d::a02:100:0:b286" k8sPodName=":" policyRevision=0

@tgraf Did this resolve after a while? Did it happen with any other endpoints? (this one was a cilium-health EP)
I have, in any case, just disabled the CEP for the cilium-health endpoint in #4981

@raybejjani Yes, the error was temporary. I'm going through log files of testing clusters and looking for errors. We should try and only emit on real issues.

hmm... if it "never" happens with normal endpoints, I'd rather not suppress any errors. If we do see it on the non-health EPs then I'll start only emitting the error if it fails for some minutes... In the case of the CEPs, I'm worried that they may not be updated and that would be unclear during a debug.


Youku

I would like to see the source code and contribute

I forgot that branch existed! I just need someone with a Youku client id to test that it works.
@NotCJSBirdie @ImranBugCMG @rocklaam are you able to test the youku branch?
Something like:
<ReactPlayer
  url='https://v.youku.com/v_show/id_XNTg4NjExNzY3Ng==.html'
  config={{ clientId: '12345' }}
/>

@cookpete so far findings are that example document is not working for youku in this branch.
In example code after entering youku url in input field, it shows an error using the alert method of browser which reads: [Fail]The params of {vid,target,client_id} are necessary !
and in console I was getting following error: TypeError: Cannot read properties of null (reading 'getElementsByTagName')
If I get time I will try to debug this.

Thanks for trying @ImranBugCMG – If you are happy to privately share your client ID with me, I can try and debug it too. I’m at https://twitter.com/cookpete or email pete [at] cookpete [dot] com

Thanks for trying @ImranBugCMG – If you are happy to privately share your client ID with me, I can try and debug it too. I’m at https://twitter.com/cookpete or email pete [at] cookpete [dot] com

I have sent you an email

Did any progress get made on this? I will have an ID to test with if you need one, and will have a look myself to try and get it working.


Update to version 1

<!--
Thank you for your pull request.
Below are a few things we ask you kindly to self-check before getting a review. Remove checks that are not relevant.
-->
Checklist

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

Closes: #12 #13

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

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

requirements: host: poetry-core>=1.1.0a6 must contain a space between the name and the pin, i.e. poetry-core >=1.1.0a6

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


Jack cut feature

Example implementation of a merged JackRafterCut and the CutFeature. Including client code in L-Miter.

I'm making an effort to make this change as gradual as possible. Meaning the two systems live side-by-side. For that purpose I created the new JackRafterCut in a new package _fabrication as otherwise there are some circular import issues.

What type of change is this?

[ ] Bug fix in a backwards-compatible manner.
[x] New feature in a backwards-compatible manner.
[ ] Breaking change: bug fix or new feature that involve incompatible API changes.
[ ] Other (e.g. doc update, configuration, etc)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[ ] I added a line to the CHANGELOG.md file in the Unreleased section under the most fitting heading (e.g. Added, Changed, Removed).
[ ] I ran all tests on my computer and it's all green (i.e. invoke test).
[ ] I ran lint on my computer and there are no errors (i.e. invoke lint).
[ ] I added new functions/classes and made them available on a second-level import, e.g. compas_timber.datastructures.Beam.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] I have added necessary documentation (if appropriate)

In case there are two btlx processes that are correlated (i.e StepJoint, StepJointNotch) and share most most of the parameters should the process be implemented as:

two separate classes in the same module (_fabrication/step_joint) and generate the parameters twice if they both need to be applied in the case of a joint with two beams (one on main_beam and one cross_beam).

or

as one class containing all the combined parameters with a boolean for applying the step or the step_notch

(I attached the parameters for each one of the processes for the example of the StepJoint/StepJointNotch)

In case there are two btlx processes that are correlated (i.e StepJoint, StepJointNotch) and share most of the parameters, should the process be implemented as:

two separate classes in the same module (_fabrication/step_joint) and generate the parameters twice if they both need to be applied in the case of a joint with two beams (one on main_beam and one cross_beam).

or

as one class containing all the combined parameters with a boolean for applying the step or the step_notch

(I attached the parameters for each one of the processes for the example of the StepJoint/StepJointNotch the common parameters should have the same values if a joint between two beams is created)

here I'd implement both separately. could be in the same module, but definitely as two separate classes. If the parameters of one of them is a subset of the other's then you could use inheritance to avoid having duplication code.
Additionally, if it makes sense, we could make a factory method which creates matching pairs of StepJoint and StepJointNotch from the necessary collection of parameters.

@papachap I think I addressed all of your comments.
I think it is currently not necessary to partially replace the processes just for the L-Miter. I'd leave it like this for now and make the swap from joint factories to process/features once we have all processes properly implemented.
If it's all the same to you, I'd go ahead an merge this one. Just hit approve if it's all good, otherwise feel free to leave some more comments.


extraction of room acoustics

Could the next release include this particular feature? To paraphrase from the abstract " transfer of extracted room acoustics to new environments"

When exporting the model, an impulse response is saved as a .wav file, you can then use a convolution reverb to apply it to virtually any other signal !


[Enhancement] Add in-app wizards for new users.

I never though this would be necessary, but

Add a blinky wizard on how to add a blog from the manager to the queue after adding the first blog, or how to make it more obvious?
Add a startup tips like window on how to add multiple blogs, and how to edit multiple blogs at once.
Maybe add an import function similar to the export function?
Consider changing the behaviour of the "download reblogged post"-option to make it the default, but instead use a "download orignal post"-checkbox. Thus, people will usually download everything, except they tick this option.
Update the UrlValidator to allow inputs in the form of http://www.tumblr.com/blog/nature-e instead of only http://nature-e.tumblr.com/.

I wonder if wizards could be avoided with a simplification of the layout? Admittedly I wasn't sure what I was supposed to do when I first pulled down TumblThree, and took a bit of trial and error to nut it out.
What confuses me layout-wise is really the bar at the bottom. It is the home of everything, but the actions are largely contextual. I wasn't really sure what the crawl button was supposed to do, nor the difference between pause and stop.
If I was driving the visual direction of this, I would probably:

Try to contextualise everything visually. Easier said than done I guess...
Get rid of textual labels and instead rely on tooltips and layout.
Show less stuff by default.
Show more visual feedback.
Reduce the start/stop/pause/resume concept to just pause/resume. It's either downloading stuff from the queue, or it isn't. Only show the relevent action. EG Pause OR Resume.
Move the user account to be a 'sessional' thing rather than a 'settings' thing. Still persist it of course.
Restore previous state with respect to pause/resume, otherwise if new launch with no items just be in crawl mode already such that as soon as something is queued it 'just works'.

Something like this I guess - please forgive my terrible mock and (lack of) design skills:

Thanks a lot for your contribution! I'll merge the import to the master, and answer your other questions more thoroughly when I've the chance to do so. Or maybe we can just chat over skype or a similar software. I am just lacking the spare time now that it's before christmas, thus I'm not delaying your contribution because I'm not intereseted in, but just because I need more than 5 minutes to properly answer you.
Yes, I know, the UI isn't the best. In my experience, GUI stuff tends to take a lot of the code base (way to much for my taste), and it's soo tedious and laborious, that I try to avoid it in my hobby projects as much as possible. I'm sure I've already opened a "redesign the UI" ticket, but actually realizing it is way more than a few days work. Plus I've never refactored anything in the UI yet, hence the ui code isnt of the best quality. Since I am also not a designers guy, i.e. lacking all the arts skills, it always led me to fixing/improving application/business code, instead of UI code.
Hence, If you like to, you can change the UI as much as you want. If It gets more functional, usable, or obvious, I'm all in for it.


HTTP parsing broken in interop tests (for large messages)

Multiple Interop tests (Java vs PHP, C#, Node and Ruby) are currently flaky:
https://github.com/grpc/grpc/issues/9937 (java csharp)
https://github.com/grpc/grpc/issues/9762 (java csharp)
https://github.com/grpc/grpc/issues/9878 (java node)
https://github.com/grpc/grpc/issues/9865 (java ruby)
https://github.com/grpc/grpc/issues/9797 (php java)
I believe they all have the same rootcause, and it seems to be connected to HTTP parsing in C core.
What we know:

easy to reproduce (e.g. run java client against C# server locally)
it happens much more often for large messages (custom_metadata and large_unary  are using a large request).
it seems not to happen when insecure channel is used
we see E0228 14:18:02.422869<PHONE_NUMBER>37664 src/core/lib/http/parser.c:287: HTTP header max line length (4096) exceeded  (or lots of instance of E0228 14:18:02.422869<PHONE_NUMBER>37664 src/core/lib/http/parser.c:287: HTTP client max line length (4096) exceeded when run prior to  https://github.com/grpc/grpc/issues/9925)

Here's the log from C# server with GRPC_TRACE=http,http1,tcp,secure_endpoint,transport_security https://gist.github.com/anonymous/bb08ca5796e7268d812a2a5fbb262361
Discussion:
For some reason, C core seems to be trying to interpret payload (plenty of zeroes because of the interop test spec) as an  HTTP header:
https://github.com/grpc/grpc/blob/f2ac4bd4886c4f8122e7f5b10f28ed7a2cfaa0a8/src/core/lib/http/parser.c#L287
The addbyte method never runs for the actual headers (they are parsed by I0302 15:00:31.959544<PHONE_NUMBER>40928 src/core/ext/transport/chttp2/transport/parsing.c:477:). The bytes seen by addbyte are all zeros so grpc_http_parser_parse never sees "end of http header line" and errors out.
The thing I don't understand is why is grpc_http_parser_parse running to parse new slices at all, after we've already read the actual headers and we know the payload should be coming (maybe a data race?)
Disclaimer: I haven't verified that java actually sends well-formed http2 frames as a request, but given the failures are flakes, it seems more likely that there's a problem with http parsing on the C core side.

@ctiller, please assign to someone on C core team.

CC @murgatroid99 @stanley-cheung @ejona86  @apolcyn

This is a very important issue, thank you @jtattermusch for creating it. It's been keeping the interop tests broken for a while.

I'm running 'tools/run_tests/run_interop_tests.py -s csharp -l java
--use_docker' and not seeing the failure... any better way to reproduce?
On Mon, Mar 6, 2017 at 12:52 PM David G. Quintas<EMAIL_ADDRESS>wrote:

This is a very important issue, thank you @jtattermusch
https://github.com/jtattermusch for creating it. It's been keeping the
interop tests broken for a while.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/grpc/grpc/issues/9956#issuecomment-284528466, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AJpudRKHjfZfeBTGWWjMNb1-p5eQmB_dks5rjHIPgaJpZM4MREXI
.

It should happen fairly often.
Running manually:
grpc-java client:
./gradlew :grpc-interop-testing:installDist -P skipCodegen=true
./run-test-client.sh --use_tls=true --use_test_ca=true --test_case=custom_metadata --server_port=8070 --server_host=localhost --server_host_override=foo.test.google.fr

c# server
~github/grpc$ tools/run_tests/run_tests.py -l csharp -c dbg --build_only
~/github/grpc/src/csharp/Grpc.IntegrationTesting.Server/bin/Debug$ GRPC_TRACE=http,http1,tcp,secure_endpoint,transport_security GRPC_VERBOSITY=debug mono Grpc.IntegrationTesting.Server.exe --port=8070 --use_tls=true 2>error_output.txt

++ dirname tools/run_tests/helper_scripts/build_csharp.sh
+ cd tools/run_tests/helper_scripts/../../../src/csharp
+ xbuild /p:Configuration=Debug /p:NativeDependenciesConfigurationUnix=dbg Grpc.sln
tools/run_tests/helper_scripts/build_csharp.sh: line 36: xbuild: command not found

any solution that doesn't involve having to maintain a c# installation?

you can use https://github.com/grpc/grpc/blob/master/tools/gce/create_linux_performance_worker.sh to get yourself a VM with all the prerequisites installed.

Also, I double checked and I can reproduce the issue on my machine with tools/run_tests/run_interop_tests.py -l java -s csharp --use_docker very reliably (more than 50% of time).

Can reproduce now.
With GRPC_TRACE=connectivity_state:
D0307 18:47:33.240479<PHONE_NUMBER>96128
src/core/lib/transport/connectivity_state.c:184: SET: 0x7f1178010e08
server_transport: READY --> SHUTDOWN [close_transport] error=0x7f1178003710
{"created":"@1488912453.240422563","description":"Delayed close due to
in-progress
write","file":"src/core/ext/transport/chttp2/transport/chttp2_transport.c","file_line":517,"grpc_status":14,"referenced_errors":[{"created":"@1488912453.240421565","description":"Failed
parsing
HTTP/2","file":"src/core/ext/transport/chttp2/transport/chttp2_transport.c","file_line":1971,"referenced_errors":[{"created":"@1488912453.240369476","description":"frame
of size 4900 overflows incoming window of
0","file":"src/core/ext/transport/chttp2/transport/parsing.c","file_line":391}]}]}
Looks like a flow control disagreement... not sure as yet which side.
On Tue, Mar 7, 2017 at 9:44 AM Jan Tattermusch<EMAIL_ADDRESS>wrote:
Also, I double checked and I can reproduce the issue on my machine
with tools/run_tests/run_interop_tests.py
-l java -s csharp --use_docker very reliably (more than 50% of time).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/grpc/grpc/issues/9956#issuecomment-284799682, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AJpudXCZD3Gvb86rvdei3Db67YhkUDYWks5rjZdogaJpZM4MREXI
.

@ctiller tracked down the flow control issue due to SETTINGS updating INITIAL_WINDOW_SIZE. On further investigation it seemed Java was consuming additionally-given window before sending a SETTINGS ACK.
I can confirm it appears Java/Netty is doing the wrong thing. encoder.remoteSettings() is called before writeSettingsAck(). encoder.remoteSettings() calls flowController().initialWindowSize() which results in the flow controller writing pending bytes.
Two immediate changes should probably be made: 1) Send the settings ack immediately before processing the settings frame in the decoder. Any failure in processing would probably result in the connection being closed, so this doesn't seem dangerous. 2) Stop triggering writePendingBytes() in the flow controller and instead wait for the flush() like normal.
We should test to make sure, but we believe grpc-java v1.1 is impacted. That means it's unclear how C should cope with the bug.
We should really have some better way of dealing with this. In spending 2 seconds thinking about the problem, Craig and I are thinking of sending a trash number in the initial SETTINGS. The number could be randomly selected each release. That way, if we find a bug like this, we could workaround it.

We tested and were not able to reproduce the issue with v1.0 but were with v1.1. This agrees with when grpc-java upgraded to Netty 4.1.8 from 4.1.6 (as in v1.0.2; v1.0.1 had an earlier version of netty). We should still check whether v1.0.1 is impacted.

https://github.com/netty/netty/issues/6520


restoreModel Issue with multiple values allowed

Hello there.
I had an issue when I was trying to restore data in my angular-advanced-searchbox model.
My model was quite simple. It contained properties by default with multiple values like this example:
{mySearchParam:['value1', 'value2']}
But in this situation, the system failed with a circular trouble.
I fixed it by replacing code in the restoreModel function stored in the angular-advanced-searchbox-tpls.js file.
I replaced...
//    if (searchParam !== undefined)
//        $scope.addSearchParam(searchParam, value, false);
...by:
if (searchParam !== undefined){
if(angular.isArray(value)){
for(var key in value){
$scope.addSearchParam(searchParam, value[key], false);
}
}
else{
$scope.addSearchParam(searchParam, value, false);
}
}
It's probably not the best solution, but it works for me.
Maybe dnauck will be inspired by this idea ;)
Gagjules

thanks, I was just having this issue!


Check whether {filename} conflicts with standard library

I think this would be a useful rule, I found a few other rules that use {filename} (e.g. INP001) but as far as I can tell this one doesn't exist.
The rule would check whether {filename} is the same as any module in the standard library.
Creating a file like that can cause problems, e.g.:
   # bisect.py
   ...

   # main.py
   import random

with:

   Traceback (most recent call last):
     File ".../foo.py", line 1, in <module>
       import random
     File "/usr/lib/python3.12/random.py", line 62, in <module>
       from bisect import bisect as _bisect
   ImportError: cannot import name 'bisect' from 'bisect'

This rule would help prevent these types of errors.

A bit unrelated but Python 3.13 will include an improved error message of this: https://docs.python.org/3.13/whatsnew/3.13.html#improved-error-messages

There is A005 https://docs.astral.sh/ruff/rules/builtin-module-shadowing/ (which is currently in preview), does this rule show an error?

Yes, that's exactly what I was looking for! Sorry for not finding it before.


Release CI (AKS Streaming) is failing due to short-lived auth tokens

The release CI is failing because the az cli auth token acquired at the beginning of the pipeline run expires during execution. It seems to be valid for a short 10 minute duration only.
Error: failed to get token: expected an empty error but received: AzureCLICredential: ERROR: AADSTS700024: Client assertion is not within its valid time range. Current time: 2024-04-10T17:21:48.2805365Z, assertion valid from 2024-04-10T17:09:49.0000000Z, expiry time of assertion 2024-04-10T17:14:49.0000000Z. Review the documentation at https://docs.microsoft.com/azure/active-directory/develop/active-directory-certificate-credentials . Trace ID: 7379e1d7-8b5c-4385-b328-bd31e9c21e01 Correlation ID: b1bb41e5-b669-41d4-aad5-1fceaa3fd405 Timestamp: 2024-04-10 17:21:48Z Interactive authentication is needed. Please run: az login

Seems related to https://github.com/Azure/login/issues/432

https://github.com/Azure/azure-cli/issues/28708#issuecomment-2049014471 seems to have worked.


I'm getting an Exception when deobfuscating

Hi,
Now that I've managed to build and execute the solution, I'm getting a weird Exception when executing it.
The CrackMe.exe file is a .NET oriented file obfuscated with ConfuserEX1.0.

Ideas?

@doria90 I'm experiencing the same issue, did you manage to solve it?

Nope. Dropped it altogether and debugged it manually using dnSpy.

Anyone fix this?

@bobbyobrien44 What version of .NET framework are you using?


Add overloadParams annotation for Siddhi extensions

Description:
This is track $subject...

siddhi-execution-string (Done) :heavy_check_mark:

siddhi-execution-regex (Done) :heavy_check_mark:

siddhi-execution-math (Done)  :heavy_check_mark:

siddhi-execution-unique (Done) :heavy_check_mark:

siddhi-execution-reorder (Done) :heavy_check_mark:

siddhi-execution-json (Done) :heavy_check_mark:

siddhi-execution-map (Done) :heavy_check_mark:

siddhi-execution-unitconversion (Done) :heavy_check_mark:

siddhi-execution-time (Done) :heavy_check_mark:

siddhi-execution-streamingml (Done) :heavy_check_mark:

siddhi-execution-tensorflow (Done) :heavy_check_mark:

Completed the task for siddhi execution extensions.


Version 0.20.0

Fix EJS template (PR #270)
Add theme-color support to make the page surround in Chrome’s tab view on Android match the black GOV.UK header (PR #278)
Add text-decoration-skip: ink to all links on GOV.UK (PR #281)
Improve contrast of links when focused (PR #272)
Make header text colour black when focused (PR #274)

Don’t merge this until we’ve moved over to Travis (https://github.com/alphagov/govuk_template/pull/267).

#267 has been merged - we're now using Travis, so I'll merge this when green. Thanks @robinwhittleton!


NAV -> Kartverket

update scope prefix and information about delegated scopes to Kartverket

Summary by CodeRabbit

New Features

Updated documentation for the applications.skiperator.kartverket.no resource to clarify access requirements and naming conventions.
Enhanced descriptions for integration settings related to Maskinporten, ensuring accurate representation of organizational access.

Unfortunately this change won't work, as we're currently embedding NAV's API's from liberator. When generating CRD's (make generate) these changes would be overwritten.
We should probably fix it upstream.


Low pass filter pauses when filterStrength is at 1.0

Hello, this is more a curiosity, I noticed that when setting .filterStrength on a LowPass filter to 1.0 it loops the last two frames.
I'm wondering if there is a clean way to get only one frame to pause instead of looping two? I looked at how it's using a buffer but I can't seem to make just one frame pause.
I want to have multiple live camera inputs displayed with the ability to pause and unpause one or both at will.
Thanks for any advice and thank you for your time.

A 1.0 filter strength simply means that the rolling average doesn't actually average anything, it just uses the last frame's colors and doesn't care about any incoming frames. Therefore, the frozen image. 0.0 means it uses only the colors from the input frame, 0.5 uses half the color from the incoming frame and half the color from the previous aggregate, and 1.0 uses all the color from the aggregate with no contribution from the incoming frame.
I'm not sure that there is a great way to pull a single frame and pause it for later reuse in any of the stock operations. You might be able to rewrite something like the buffer to pull a single framebuffer and hold onto it for later use.


comment

Hi Jeff,  love your project, found it on Adrians site
I look as old as you,  been 20 years in electronics, and about the same in software.  Now doing windows based comms,  web servers, api servers etc.
if you need some help, yell]
<EMAIL_ADDRESS>

Thanks! My project is still too early in development to benefit from help. But I really appreciate the offer and will definitely send you a note in about 3 months when I'm at a more stable and more complete release. I am going to put up the imagenode repository in the next month, then the imagehub repository after that. The docs and design will be stable enough then to make collaboration easier.

Thanks again for your offer to help. I really appreciate it. I am closing this issue for now and will re-open it when I have the code further along. When I re-open the issue, you will be pinged by GitHub.

I'm prepping for my PyCon 2020 presentation. Pycon 2020 is moving to an online video & posted slide decks format due to the COVID-19 crisis. I'm going try using an "Open an Issue" format for Questions and Answers for that presentation, in this yin-yang-ranch repository. It should go live in a week or 2. if You could take a look and leave feedback either here, or in one of the new issues, that would be helpful. I'm closing this particular issue for now.


Use mainline Linux load tracking events names

Actually, the kernel does not have a sched_pelt_se event, it only has a sched_pelt_se tracepoint. That said, it's probably a better idea to give the same name to the trace point and associated event.
TODO: update tools/kmodules/ to emit the sched_pelt_se name rather than sched_load_se


[Feat] Extend Logging Capabilities

Why: Why you want the feature
Currently the tie2index.py python script have no debug, error,info output for an administrator to check if the script is working.
What: What you want to happen
Extend the logging capabilites of the python script. So that an administrator know in which state is the python script currently and is the script working.
Example:
http://dev.splunk.com/view/logging/SP-CAAAFCN
How: How it is implemented

[ ] Add logging splunk extension to python script

This is part of issue #10 : we log to the dcso_tie.log file when running within the Splunk environment, otherwise to sys.stderr.


DPL-638	Cardinal informatics: map which qc metrics belong to which pool

In the Cardinal project we have repeated samples and need to figure out which qc metrics belong to which pool
What would be the mlwh query to get the mapping between mlwarehouse.qc_result.id_pool_lims (something like 'SQPP-2744-H:D5') and a mlwarehouse.sample.name experiment_id - something like 'CRD_CMB13195925'?
query is:
SELECT * FROM (SELECT DISTINCT donors.id_sample_tmp,sample.name as experiment_id,original_study.name as cohort,donors.customer_measured_volume,donors.supplier_name as donor FROM mlwarehouse.iseq_flowcell
 JOIN mlwarehouse.sample ON iseq_flowcell.id_sample_tmp = sample.id_sample_tmp
 JOIN mlwarehouse.study ON iseq_flowcell.id_study_tmp = study.id_study_tmp
 JOIN mlwarehouse.iseq_product_metrics ON iseq_flowcell.id_iseq_flowcell_tmp = iseq_product_metrics.id_iseq_flowcell_tmp
 JOIN mlwarehouse.iseq_run on iseq_run.id_run = iseq_product_metrics.id_run
 JOIN mlwarehouse.iseq_run_lane_metrics on iseq_run_lane_metrics.id_run = iseq_run.id_run
 JOIN mlwarehouse.psd_sample_compounds_components pscc ON iseq_flowcell.id_sample_tmp = pscc.compound_id_sample_tmp
 JOIN mlwarehouse.sample as donors ON donors.id_sample_tmp = pscc.component_id_sample_tmp
 JOIN mlwarehouse.stock_resource ON donors.id_sample_tmp = stock_resource.id_sample_tmp
 JOIN mlwarehouse.study as original_study ON original_study.id_study_tmp = stock_resource.id_study_tmp
 WHERE sample.name IN ('CRD_CMB13195930','CRD_CMB13195926','CRD_CMB13195922','CRD_CMB13195927','CRD_CMB13195928','CRD_CMB13195921','CRD_CMB13195929','CRD_CMB13195924','CRD_CMB13195920','CRD_CMB13195925','CRD_CMB13195923','CRD_CMB13195918','CRD_CMB13195919')
 AND donors.supplier_name LIKE "%30007454087%") as A
 LEFT JOIN (SELECT id_sample_tmp,id_pool_lims,CONCAT(value,' ',units) as 'live_cell_count' from mlwarehouse.qc_result  WHERE qc_type = 'live_cell_count') as B on A.id_sample_tmp = B.id_sample_tmp;

The samples are repeated at 24h and 48h they have the same id_sample_tmp. but they end up in a different pools with a different ids.
from the info on the pool level i can find the id_sample_tmp = ‘8380396’ but this id maps to two different mlwarehouse.qc_result.id_pool_lims
so I need to find which one bellowngs to which by finding the additional join element instead of id_sample_tmp which would be mlwarehouse.qc_result.id_pool_lims
What is the bet way to join instead of A.id_sample_tmp = B.id_sample_tmp to avoid the replication of qc fields such as live_cell_count
Relates to RT770309 also discussion in Cardinal Slack channel
Contact: Matiss O (mo11)

Adding some notes:
From Cardinal Knowledge Transfer document slide 4, Compound samples - data model:

A compound sample has many component samples.
A component (=normal) sample can be part of several compound samples. e.g. if it's been down the pipeline more than once on different plates
A sample cannot be both a compound and a component sample.

In qc_results, id_sample_tmp is for components.
In iseq_flowcells, id_sample_tmp is for compounds.
psd_sample_compounds_components gives the mapping between components and compounds.

We do not have the mapping in mlwarehouse. Limber File exports give metrics per well though, which correspond to id_pool_lims. For example,
Plate SQPP-2744-H at https://limber.psd.sanger.ac.uk/limber_plates/78b39ca4-3410-11ed-94dc-fa163eea3084
See SQPP-2744-H:D5 (Export2.csv - September 14, 2022 12:24) at https://limber.psd.sanger.ac.uk/qc_files/c5439e34-341f-11ed-b805-fa163eac3af7
and
Plate SQPP-2829-L at https://limber.psd.sanger.ac.uk/limber_plates/3475d9fa-34ea-11ed-8920-fa163eac3af7
See SQPP-2829-L:F7 (Export_p3.csv - September 15, 2022 13:28) at https://limber.psd.sanger.ac.uk/qc_files/de8ef956-34f1-11ed-9f51-fa163eea3084

Notes: https://docs.google.com/document/d/1JvRFYc1ISB8nnXlrZaRfyqLSZSamOQme_8rY_bopc-s/edit?usp=sharing

We have created a new story: DPL-682 Provide a mapping between qc_results and compound_samples in mlwarehouse for Cardinal Project #3755


Request: ability to adjust maximum darkness level of scene when synced with world clock

Currently, if a scene is set to synchronize its darkness level with the Pf2e world clock and the darkness level of the scene is at 1, it's extremely hard for everyone (and especially people with vision issues) to see areas they've already explored on the scene. While I can, of course, detach the scene's darkness level from the world clock, that's not really a good solution, since I still want the scene to get lighter or darker depending on the time of day in the game -- I just don't ever want it to be darkness level 1.
I therefore propose that the Pf2e system allow GMs to set the maximum darkness level for their world, which would then apply to all scenes synchronized with the game clock. That way, I can set darkness level to, say, 0.7 and it will never get darker than 0.7 on a scene.

Would you mind providing some screenshots of what you're seeing that is undesirable?

Haven't forgotten about this! Will post screenshots tomorrow.

@weepingminotaur if you change the color of darkness to something lighter, then even at darkness 1 the already-explored areas are still visible.
The default is "#2d2d52", but if you set the darkness color for the scene to something like "#7777bb" then darkness 1 will actually be brighter than the default is at .7, while still being clearly dark.

@weepingminotaur if you change the color of darkness to something lighter, then even at darkness 1 the already-explored areas are still visible.
The default is "#2d2d52", but if you set the darkness color for the scene to something like "#7777bb" then darkness 1 will actually be brighter than the default is at .7, while still being clearly dark.

Thank you for this suggestion. Explored fog is so much more visible now. This issue can be closed.


Added additional skip project button

Current button layout appearance:

Works the same as the "project is not here" button and is viewable in the list of skipped projects.

Looks good, nice work Nicolas!


Homework 2(30.09)

Implemented Merge sort.
Generalized sorting.
In-place qsort already implemented.
Added fantomas formatter CI and locally.

Made so that input array in a sort function doesn't change.

Added tests for empty arrays


Organize packages

We are organizing packages for release the new modules

I dislike the .test package, but there's nothing much we can do for android - hence :+1:

@qmx I also dislike that, but we need a different packed name to run our tests :(


Add test suite.

Closes #8.
PR adds basic tests and "test runner" to the project. Design is pretty straightforward, I've added support for sshing any machine and for now it's configured to run against localhost as a "remote build machine", but if we'll find out that neither Travis nor CircleCI allow us run sshd, we'll setup separate machine and then we'll be able to encrypt ssh key for connection from Travis or CircleCI using their apis for encryption.
I don't have permissions to add CI like Travis or CircleCI fluck-fluck, so we'll figure out that later in a separate PR.
For now we can run test suite locally (for each PR): bash test.sh, this is how it looks like:

cc @dmitry-novikov PTAL (not blocking reviewer).

There is a medication for what you have, my friend.


Highlight groups in syntax.txt are not highlighted on Windows

Steps to reproduce

In cmd.exe do gvim --clean or vim --clean.
:help hl-ColorColumn
The highlight groups are not highlighted:

Expected behaviour
The highlight groups should be highlighted, like on Linux:

Version of Vim
8.2.4929 (from vim/vim-win32-installer/releases)
Environment
Windows 10
cmd.exe
Logs and stack traces
No response

If you run these Ex commands:
:help hl-ColorColumn
/^ColorColumn\t

You cursor should end up on the name of the ColorColumn group which is meant to be highlighted.
From this position, what is the output of these Ex commands:
:echo prop_type_list({'bufnr': bufnr('%')})->index('help-hl-ColorColumn')
:echo prop_list(line('.'))

This is what I get:
:echo prop_type_list({'bufnr': bufnr('%')})->index('help-hl-ColorColumn')
-1
:echo prop_type_list({'bufnr': bufnr('%')})
[]
:echo prop_list(line('.'))
[]

If you write this code in a script:
vim9script
help syntax
search("^ErrorMsg\t")
var word: string = 'ErrorMsg'
var buf: number = bufnr('%')
prop_type_add('help-hl-' .. word, {
    bufnr: buf,
    highlight: word,
    combine: false,
})
prop_add(line('.'), 1, {length: word->strlen(), type: 'help-hl-' .. word})

Then, start Vim with no config and source the previous script:
vim -Nu NONE -S /path/to/script

Is ErrorMsg highlighted as an error?

Also, for this to work, Vim needs to have been compiled with the +textprop feature.  Is +textprop present in the output of :version?

Is ErrorMsg highlighted as an error?

Yes:

Is +textprop present in the output of :version?

Yes:

@LemonBoy Yes, changing that line as you suggested fixed the problem.

I get this error:
"syntax.txt" [readonly][unix] 5824L, 231142B
Error detected while processing BufRead Autocommands for "C:/Program Files/Vim/vim82/doc/*.txt"..FileType Autocommands for "*"..Syntax Autocommands for "*"..function <SNR>8_SynSet[25]..script C:\Program Files\Vim\vim82\syntax\help.vim:
line  218:
E492: Not an editor command: {
line  219:
E1124: ":var" cannot be used in legacy Vim script
line  221:
E492: Not an editor command:     path = path->tr('/', '\')
line  223:
E121: Undefined variable: path
line  228:
E1128: } without {: }

OK, new try:
diff --git a/runtime/syntax/help.vim b/runtime/syntax/help.vim
index f5a50bfac..d6e526f6e 100644
--- a/runtime/syntax/help.vim
+++ b/runtime/syntax/help.vim
@@ -215,12 +215,17 @@ hi def link helpError		Error
 hi def link helpTodo		Todo
 hi def link helpURL		String

-if expand('%:p') == $VIMRUNTIME .. '/doc/syntax.txt'
+let s:path = $VIMRUNTIME .. '/doc/syntax.txt'
+if exists('+shellslash') && !&shellslash
+  let s:path = s:path->tr('/', '\')
+endif
+if expand('%:p') == s:path
   " highlight groups with their respective color
   import 'dist/vimhelp.vim'
   call vimhelp.HighlightGroups()
 endif

+
 let b:current_syntax = "help"

 let &cpo = s:cpo_save

Yeah, this worked on Windows. I think it should work on Linux too but I cannot test it at the moment.

Does this work:
if expand('%:p') =~ $VIMRUNTIME .. '[/\\]doc[/\\]syntax.txt'

This does not work on Windows. I think it has to do with backslashes.
Here's a sample session that could help figure out why it's not working.
:echo expand('%:p') =~ $VIMRUNTIME .. '[/\\]doc[/\\]syntax.txt'
0

:echo expand('%:p')
C:\Program Files\Vim\vim82\doc\syntax.txt

:echo $VIMRUNTIME
C:\Program Files\Vim\vim82

:echo '[/\\]doc[/\\]syntax.txt'
[/\\]doc[/\\]syntax.txt

:echo  $VIMRUNTIME .. '[/\\]doc[/\\]syntax.txt'
C:\Program Files\Vim\vim82[/\\]doc[/\\]syntax.txt

:echo 'C:\Program Files\Vim\vim82\doc\syntax.txt' =~ $VIMRUNTIME .. '[/\\]doc[/\\]syntax.txt'
0

:echo 'C:\Program Files\Vim\vim82\doc\syntax.txt' =~ 'C:\Program Files\Vim\vim82\doc\syntax.txt'
0

echo '\doc\syntax.txt'  =~ '[/\\]doc[/\\]syntax.txt'
1

:echo expand('%:p') =~ '[/\\]doc[/\\]syntax.txt'
1

FWIW, the last posted patch works on Windows:

Yeah, this worked on Windows.

And I've tested it on Linux where it works too.

Yeah, your patch does work. I like @brammool's one-liner though and I don't understand why it doesn't work.
Is below the expected behaviour?
:echo 'foo\bar' =~ 'foo\bar'
0

I would expect the output to be "1". These are single quoted string and :help literal-string says
This string is taken as it is.  No backslashes are removed or have a special
meaning.  The only exception is that two quotes stand for one quote.

Single quoted strings are useful for patterns, so that backslashes do not need
to be doubled.

Something is weird with those backslashes or I'm missing something.

Right, the backslashes in $VIMRUNTIME need to be escaped.
Would expand() use forward slashes while $VIMRUNTIME has backward slashes?

The latest runtime files update (3f32a5f1) partially fixes the issue with syntax/help.vim.
There is still a problem on Windows when shellslash is set. To reproduce:
vim --clean -c "set shellslash -c "help hl-ColorColumn"

And the highlight groups are not highlighted.
This is because expand('%:p') contains slashes but $VIMRUNTIME contains backslashes in this line:
https://github.com/vim/vim/blob/d881d1598467d88808bafd2fa86982ebbca7dcc1/runtime/syntax/help.vim#L218

Fixed in the latest runtime files update (30ab04e16e1e). Thanks!
Closing.


Make wget continue

Stop skipping over partially downloaded/interrupted files

I ran a quick test to try this out.   I killed the script in the middle of downloading hadoop.  When I ran fluo-dev download again, it continued downloading hadoop where it left off and the checksum was correct.  Of course it worked, but it was neat to see that wget option in action.   I had never used it before.


Invalid Return Signature on Call Java Void function from Rust

Hello, First of all, thank you very much for this 'library'.
I am trying to call a void function from rust, but the signature from method is wrong
This is the output
> Task :Main.main() FAILED
	Call from Rust getWorld method
Error while throwing Java exception: Java exception was thrown

Deprecated Gradle features were used in this build, making it incompatible with Gradle 8.0.

You can use '--warning-mode all' to show the individual deprecation warnings and determine if they come from your own scripts or plugins.

See https://docs.gradle.org/7.6.1/userguide/command_line_interface.html#sec:command_line_warnings
4 actionable tasks: 1 executed, 3 up-to-date
Exception in thread "main" java.lang.NoSuchMethodError: Linternal/Hello;.print()Ljava/lang/Object;
	at internal.Hello.world(Native Method)
	at main.Main.main(Main.java:12)

FAILURE: Build failed with an exception.

I think this problem is in the "Ljava/lang/Object" return.
Would it be related to this issue https://github.com/giovanniberti/robusta/issues/8 ? Maybe is time to release a new version.
main/Main.java
package main;

import internal.Hello;

public class Main {
    public static void main(String[] args) {

        System.load("/home/palomo/libs/libcore_serve.so");

        // Java
        Hello hello = new Hello();
        System.out.println(hello.world());

    }
}

internal/Hello.java
package internal;

public class Hello {

    public native String world();

    public String getWorld() {
        System.out.println("\tCall from Rust getWorld method");
        return "World";
    }
    public void print() {
        System.out.println("\tCall from Rust print method");
    }

}

lib.rs
extern crate core;

use robusta_jni::bridge;

#[bridge]
mod jni {
    use robusta_jni::convert::{Signature, TryIntoJavaValue, TryFromJavaValue};
    use robusta_jni::jni::JNIEnv;
    use robusta_jni::jni::objects::{AutoLocal, JValue};
    use robusta_jni::jni::errors::Result as JniResult;
    use robusta_jni::jni::errors::Error as JniError;

    #[derive(Signature, TryIntoJavaValue, TryFromJavaValue)]
    #[package(internal)]
    pub struct Hello<'env: 'borrow, 'borrow> {
        #[instance]
        raw: AutoLocal<'env, 'borrow>,
    }

    impl<'env: 'borrow, 'borrow> Hello<'env, 'borrow> {
        pub extern "jni" fn world(self, env: &JNIEnv) -> String {
            self.getWorld(env);
            self.print(env);
            return "Hello World".to_string();
        }

        pub extern "java" fn getWorld(&self, env: &JNIEnv) -> JniResult<String> {}

        pub extern "java" fn print(&self, env: &JNIEnv) -> JniResult<()> {}
    }

}

I solved this problem by using the git version instead of the crate version
Cargo.toml
[dependencies]
- robusta_jni = "0.2"
+ robusta_jni = { git = "https://github.com/giovanniberti/robusta.git" }

I think it is time to release a new version of this crate

Hi, thank you for the issue! I've just released version 0.2.1, so you can use that instead.


Flask-Migrations

To enable more agile development of the DB models we should start using Flask-Migrations or a similar tool.
It was discussed that it's ok to export and import the data once to create a clean migration history if that proves necessary.

This was closed by 158228dcf8e6fd9a2b0ab360c54a44d85ec6663f


Max entities per query?

Hi there, just curious. Is there a max number of entities limit on each datastore query?
Sorry for asking here, the docs is vague saying 1000 limit on Lookup, not sure if that's what I'm looking for.
Thank you.

I don't know of any limits. Our library doesn't impose any, so it should be set by the API, if there is one. You could try to ask this over on https://stackoverflow.com/questions/tagged/google-cloud-datastore.


Basic signup flow page

Hi Pedro,
This is the basic flow for the signup page.

Issue This PR Addresses

Type of Change

[ ] Bugfix: Change which fixes an issue
[ ] New Feature: Change which adds functionality
[ ] Documentation Update: Change which improves documentation
[ ] UI: Change which improves UI

Description

Checklist

[ ] Quality: This PR builds and passes our npm test and works locally
[ ] Tests: This PR includes thorough tests or an explanation of why it does not
[ ] Screenshots: This PR includes screenshots or GIFs of the changes made or an explanation of why it does not (if applicable)
[ ] Documentation: This PR includes updated/added documentation to user exposed functionality or configuration variables are added/changed or an explanation of why it does not(if applicable)

@PedroFonsecaDEV


Move the code to 2018 edition of Rust

CC @rust-embedded/cortex-a  in case someone feels bored and wants to do this exercise ;)

Closed by ad2d0d11828b3881793928a2f906ca4f506f9979


Extract symsrv.rs into a separate crate

As discussed in https://github.com/nico-abram/blondie/issues/2
I haven't modified anything in symsrv, just moved it out. After uploading symsrv to crates io you should be able to change the path dependency to a proper crates io one.
Cargo new defaulted to the 2021 edition, hope that's alright. It compiled without changes.

I've moved it to a new repo and published it on crates.io.


Conference Presentation Submission Forms

We decided to change the "Presentation Type" download options from 3 to 2. Need to change everything classified as "Presentation (2 or more speakers)" and "Presentation (1 speaker) to simply "Presentation". There should only be two options on this dropdown:

Poster
Presentation

File upload in Optional metadata not necessary, since the upload is provided above (though would be good to allow up to three file uploads). On the detail page (http://aashe-hub-dev.herokuapp.com/browse/presentation/9/A-Balancing-Act-St-Louis-Higher-Education-Sustainability-Consortium), uploaded files also show up in two places. All uploaded files should be in one place.

Fixed in 4f500597021141b11f9ef66201dfb27cfd4692fa. Closing as the second bullet point is addressed in #73.

Has the first bullet been addressed? I'm not seeing the change on the dev site.

Fixed in the code, not deployed yet.


Can now filter for database properties

Checklist

[x ] the individual contributor license agreement is signed

Description of change

Now allows users to filter the database connections to their liking.

@drewwills  I need this merged in to be able to create docker images. Otherwise the docker image cannot connect to a database.

Would the solution used in Jasig/uPortal/pull/853 work here?
cc @andrewstuart

@ChristianMurphy no it would not. That is for the old uPortal. We talked at the conference about changing that PR to be more inline with Apereo and not tied to Andrew's stuff. Also @drewwills wanted a gradle process to produce a docker image. I have most of that working right now locally.

We certainly can consider including this stuff as long as we have filters, but (in the not too distant future) we were hoping to get away from cooking settings into the artifacts at build time altogether.
We're also hoping to be 100% Gradle-based.

no it would not. That is for the old uPortal. We talked at the conference about changing that PR to be more inline with Apereo and not tied to Andrew's stuff.

Fair enough, I am not privy to that conversation, but more generic support for containerization does seem like a good direction to go.

Also @drewwills wanted a gradle process to produce a docker image. I have most of that working right now locally.

Right, that is my understanding as well.
Do you have any thoughts on how this would be Gradle-ized?
Would you be interested in helping Gradle-ize this and other tasks?

The thought ATM is that all deployment-specific configuration settings will be handled in a .properties file external to the uPortal webapp.  It would be similar to the overrides.properties approach (but may benefit from some revisions).  There may also be some Gradle-based tooling to help you create such a file, if there's value in that.

I'd recommend heavily towards just using a standard spring PropertySourcesPlaceholderConfigurer for externalizing configuration and getting rid of all possible build-time filtering.
The benefits with containerization are that you can include properties as YAML, foo.bar=baz-style, or even by setting environment variables like FOO_BAR=baz. It really ends up giving you the most flexibility with containerized solutions as to how you want to go about specifying your runtime configuration.
Moving towards a more standard Spring runtime-configurable application will also help enable cool solutions like Spring Cloud Configuration.
Anyway, just my two cents :-)

Most configs already go through the PropertySourcesPlaceholderConfigurer;  the ones that don't will need to be refactored to go through it... at least if they need to be adjusted per-deployment.

I can try my hand at making this use OS environmental variables and push up commits for that. As for turning the entire thing into a gradle based build, I'm not sure how much I can help, but I can get the ball rolling for the docker part.

I can try my hand at making this use OS environmental variables and push up commits for that.

:+1:

I'm not sure how much I can help

There are plenty of bite sized tasks, moving tasks that are currently in ant and maven over to gradle.
Where whatever time you can contribute helps move the effort forward. :smile:
Jasig/uPortal/pull/890 and Jasig/uPortal/pull/889 are two excellent examples. :star2:

I can get the ball rolling for the docker part.

@drewwills @andrewstuart when we are talking about removing filtering, are we talking about removing it completely? Like we no longer need filters/x.properties. All XML files will have the environmental variables directly instead of using .properties file (that will pull from OS environmental variables)? Or do we want to go with a half way step for right now? Like leave the filters/.properties files and have them link to the OS environment variables?

I would personally say completely eliminated. There is a slight issue with the web.xml and such files where they don't get processed by spring, but my experience has been that even those can be replaced by -D defining properties during startup. Docker's entrypoint facility makes it really easy to do that and even extend that to using environment variables with defaults. The downside of that approach is that it can potentially complicate regular tomcat usage by requiring properties to be set.
Also, re: encrypted properties, I know it's popular and looks good to users to allow encrypted properties, but my experience has been that it's really dangerous because you might be hiding really really bad passwords behind an encrypted layer that makes people forget that they're using admin:password in production. This exact thing just happened on a project and nobody noticed because the encrypted properties obscured the very poor passwords.
So I guess just a word of caution about making encrypted properties files a first-class uPortal feature. There are much more secure ways these days of providing secure access to secrets that can enforce security policies and rotation (e.g. Vault) that encrypted static files just can't provide.

If we completely eliminate properties then all portlets will have to be updated to use OS environmental variables as well. We would still have to allow portlets to filter for older versions of uPortal (4.3.x series). Documentation will need to be very clear about how that works for using a portlet in uPortal 5 verse uPortal 4.

We may want to use OS environment variables in some specific/important cases, but I think most of it --
at least in a basic install -- will be in an external (not in Tomcat/uPortal/) .properties file.

Just to clarify, if it is set up to use Spring properties and the @Value annotation, and if you use the default spring property config, or make sure you enable the environment provider, then you get both environment variables and properties files for free. Spring can/will examine both (plus a few other sources like --command.line.params=foo and potentially  -Ddefined.property=foo IIRC), and is smart enough to use the proper conventions when searching e.g. system environment variables.
So for example, given @Value("foo.bar") private String fooBar = ""; or in an XML config file, the placeholder ${foo.bar}, an application can set the value of fooBar by:

including a properties file with foo.bar=baz
setting the environment variable FOO_BAR=baz
passing the cli argument/param of --foo.bar=baz
(maybe) defining the system property foo.bar a la -Dfoo.bar=baz

The out-of-the-box Spring config for this is very flexible and it can also be configured to the project needs. So I would

@andrewstuart Is there a good documentation source for this system?  (I'd like to include it by reference.)

This may be the best: https://docs.spring.io/spring-boot/docs/current/reference/html/boot-features-external-config.html. It's technically for Spring Boot, but I believe that just means that the documented behavior is how Spring Boot configures the Spring PropertySource-related utilities. All the configuration behaviors should hopefully be attainable without requiring Spring Boot.

@drewwills @ChristianMurphy one of you said both and the other said just OS env vars. Can we come to an agreement on this?

@bpowell Sorry for the confusion, both.

Hey @bpowell maven has been completely removed from uPortal-start. (#21)
Since this PR is maven based, I'm going to close it out.
It will still be really great to have these be more configurable. :+1:
Hope to see a Spring based PR soon. :smiley:


[cdk8s-plus] Simplify cdk8s-plus API by removing the spec nesting level

Description of the feature or enhancement:
Instead of:
new kplus.Pod(this, 'Pod' {
  spec: {
    containers: ...
  }
})

Lets have this:
new kplus.Pod(this, 'Pod' {
  containers: ...
})

Use Case:
Better API ergonomics
Proposed Solution:

Other:

What about possibly collisions with the metadata property?

[ ] :wave: I may be able to implement this feature request
[ ] :warning: This feature might incur a breaking change

This is a :rocket: Feature Request

Resolved by https://github.com/awslabs/cdk8s/pull/347


Fix and pull in brotli encoding (aka Revert "Revert "Add support for brotli encoding"")

Reverts jfhbrook/node-ecstatic#230

Codecov Report

Merging #231 into master will decrease coverage by 0.45%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master     #231      +/-   ##
==========================================
- Coverage   75.86%   75.41%   -0.46%
==========================================
  Files           9        9
  Lines         518      541      +23
  Branches      117      125       +8
==========================================
+ Hits          393      408      +15
- Misses         45       46       +1
- Partials       80       87       +7

Impacted Files
Coverage Δ

lib/ecstatic/opts.js
77.96% <66.66%> (-0.3%)
:arrow_down:

lib/ecstatic.js
73.02% <66.66%> (-0.73%)
:arrow_down:

Continue to review full report at Codecov.

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


ci: foxy

This is based on feat/api-gateway as soss doesn't work in foxy.

merged into #121 as they need each other.


MySql tab, setup

Estimate (hours): 5
Uncertainty factor: 2

Estimate (hours): 5
Uncertainty factor: 2

Estimate (hours): 4
Uncertainty factor: 2

Estimate (hours): 4
Uncertainty factor: 2


Unable to start project

Issue and Steps to Reproduce
Clone the repository.
First, npm install.
Then, npm start
Versions
0.1.0 (current "master" branch)
Screenshots
Expected
The application is displayed.
Actual
Unable to launch project :
prototype.js:1 Uncaught Error: Cannot find module './locale'
at webpackMissingModule (prototype.js:1)
at Module../node_modules/moment/src/lib/moment/prototype.js (prototype.js:1)
at webpack_require (bootstrap:781)
at fn (bootstrap:149)
at Module../node_modules/moment/src/lib/moment/moment.js (moment.js:1)
at webpack_require (bootstrap:781)
at fn (bootstrap:149)
at Module../node_modules/moment/src/moment.js (moment.js:1)
at webpack_require (bootstrap:781)
at fn (bootstrap:149)
at Module../node_modules/react-datepicker/es/index.js (index.js:1)
at webpack_require (bootstrap:781)
at fn (bootstrap:149)
at<EMAIL_ADDRESS>(CustomDate.js:23)
at webpack_require (bootstrap:781)
at fn (bootstrap:149)
at<EMAIL_ADDRESS>(DateInput.js:27)
at webpack_require (bootstrap:781)
at fn (bootstrap:149)
at<EMAIL_ADDRESS>(index.js:7)
at webpack_require (bootstrap:781)
at fn (bootstrap:149)
at<EMAIL_ADDRESS>(index.js:22)
at webpack_require (bootstrap:781)
at fn (bootstrap:149)
at Module../src/layout/Header/Header.js (Header.container.js:9)
at webpack_require (bootstrap:781)
at fn (bootstrap:149)
at Module../src/layout/Header/index.js (index.js:1)
at webpack_require (bootstrap:781)
at fn (bootstrap:149)
at Module../src/layout/App/App.js (index.js:14)
at webpack_require (bootstrap:781)
at fn (bootstrap:149)
at Module../src/layout/App/index.js (index.js:1)
at webpack_require (bootstrap:781)
at fn (bootstrap:149)
at Module../src/index.js (reducer.js:16)
at webpack_require (bootstrap:781)
at fn (bootstrap:149)
at Object.0 (index.js:1)
at webpack_require (bootstrap:781)
at checkDeferredModules (bootstrap:45)
at Array.webpackJsonpCallback [as push] (bootstrap:32)
at main.chunk.js:1
Additional Details

Installed packages

https://github.com/moment/moment/issues/5472

thank for you issue, you should resolve by using the version 2.2.24 for moment.
https://github.com/moment/moment/issues/5472
can you make a pull request, please ?

You can find the solution here moment/moment#5472 a comment from @MohsinGhani.
I am using yarn and it solved the issue. ;)

Hello Guys!
I faced the same issue from last few hours and finally I have resolved it through changing moment version.
Reason:
moment release the new version 17 hours ago from 2.24.0 to 2.25.0
Solution:
Step 1: change the versions

"moment": "2.24.0",
"moment-timezone": "^0.5.28",

Step 2: If you are using yarn please add resolutions in your package.json file like this
"dependencies" {
"moment": "2.24.0",
"moment-timezone": "^0.5.28"
},
"resolutions": {
"moment": "2.24.0"
},
Thank You.

Yes we found this solution, Rémi is making a PR 🥇
But Thank you very much for your comment 👍

OK, I will force "moment" to 2.24.0 version.
It works with exactly with version.

See : https://github.com/AxaGuilDEv/react-starter/pull/13


Passes auth info from the URL to the underlying http(s) get() call

url.parse() returns an 'auth' property: https://nodejs.org/docs/latest/api/url.html#url_urlobject_auth
http(s) accepts an auth property: https://nodejs.org/api/http.html#http_http_request_options_callback
This allows you to use URLs like<EMAIL_ADDRESS> Previously the auth information would be parsed but not used in the actual call.

Thanks! Published as v2.8.0


removed info about spellcheck

Description of the change: removed info about spellcheck
Reason for the change: refer to https://github.com/sendgrid/docs/pull/4044#issuecomment-427507779

Hello @StrikerRUS,Thanks again for the PR! It's HACKTOBERFEST! We want to show our appreciation by sending                                              you some special Hacktoberfest swag. If you have not already, could you please fill out                                              this form so we can send it to you? Thanks!                                             Team SendGrid DX


feat(event): sign up notification on slack

Proposed changes
Sends a message on Slack when events open for signup
SLACK_BOT_TOKEN env-var must be added in prod for this to work
Issue number: closes #
Pull request checklist
Please check if your PR fulfills the following requirements:

[x] CHANGELOG.md has been updated. Guide
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been reviewed and added / updated if needed (for bug fixes / features)
[x] The fixtures have been updated if needed (for migrations)

Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

Hvilken kanal er annet?

@Eirsteir Ser at det var vanskelig å vite, endret navn på channel variabler til SLACK_BEDPRES_OG_KURS_CHANNEL_ID og SLACK_ARRANGEMENTER_CHANNEL_ID. Måtte også slette migrasjons-testen din for grupper/studie siden den failet.


Scala Import fails to add Scala SDK

IntelliJ IDEA 2019.2.2 (Community Edition)
Bazel Plugin: source build
Sync fails attaching Scala SDK to module. Can be reproduced since 5f03bde0f02f5d1693a4567dc129230420a30372
Repo to reproduce: https://github.com/liucijus/bazel-scala-example

cc @iirina this change caused all Scala workspaces to stop resolving code. Do you know what may be the root cause here?

cc @brendandouglas this is pretty serious as it causes all Scala workspaces to not work. In the mean time while we find a mitigation (either the plugin or rules_scala), shall we revert the 5f03bde0f02f5d1693a4567dc129230420a30372 and cherry pick the revert into the next release?

Narrowed it down to this diff:
 def get_java_provider(target):
     """Find a provider exposing java compilation/outputs data."""
-    if hasattr(target, "java"):
-        return target.java
+    if JavaInfo in target:
+        return target[JavaInfo]

Background of this change: https://github.com/bazelbuild/bazel/issues/7598

Thanks @jin for finding a fix!

@jin @iirina was this due to a bug in JavaInfo? If so any idea what's the bug?  Ideally we'd like scala to use the regular JavaInfo and not a custom attribute

My hunch is that the issue was caused by https://github.com/bazelbuild/bazel/issues/8916 which is now fixed but not released yet.

@liucijus I tried reproducing the issue by manually editing the intellij_info_impl.bzl file to the order pre the fix but couldn't reproduce it. Can you reproduce locally? I was trying to see if the fixed Bazel version indeed fixes it but after 2-3 hours of failing to reproduce I stopped.

I can't run this fix without upgrading rules_scala version. But if I upgrade problem does not reproduce with older versions of bazel as well. For example, with rules_scala_version = "b2273e7a90eac81132c9cdb8b2ca05fdbba74e46" problem does not reproduce with Bazel 0.29.0.

	File "/home/vaidas/.cache/bazel/_bazel_vaidas/f790999ae9f6a398f2727c584e8fa64d/external/io_bazel_rules_scala/scala/scala_import.bzl", line 32, in struct
		_create_provider(current_jars, transitive_runtime_j..., <5 more arguments>)
	File "/home/vaidas/.cache/bazel/_bazel_vaidas/f790999ae9f6a398f2727c584e8fa64d/external/io_bazel_rules_scala/scala/scala_import.bzl", line 73, in _create_provider
		java_common.create_provider(use_ijar = False, compile_time_jar...]), <3 more arguments>)
type 'java_common' has no method create_provider()

That's strange because I think this reproduced internally when we were on a relatively advanced version of rules_scala. What do you think we should do here?

With the bazel plugin 2<IP_ADDRESS>.2 and intellij ultimate 2019.2.4 I'm still seeing the Scala SDK get dropped whenever a sync happens. Any ideas?

This happens every time to me with
IntelliJ IDEA 2019.3.5
Bazel plugin 2<IP_ADDRESS>.0
Bazel 3.2.0
HigherKindness rules_scala_annex (@ b772564a20eee9271068cfba55147191385343bd) https://github.com/higherkindness/rules_scala

I'm running into the Scala SDK being dropped on sync with the following versions:
IntelliJ IDEA 2021.3.2 (Ultimate Edition) Build #IU-213.6777.52
Bazel Plugin 2<IP_ADDRESS>.1-api-version-213
Bazel 4.2.1
Scala Rules for Bazel (https://github.com/bazelbuild/rules_scala)  e7a948ad1948058a7a5ddfbd9d1629d6db839933
I have a prototype project where these versions work fine and don't drop the SDK setting on a sync, but in a much larger project where it is being dropped. Any suggestions on what else I could look into?


synced documentation not working

Documentation for terraform-modules etc isn't working anymore and needs to be fixed.

Solved by https://github.com/XenitAB/xenitab.github.io/pull/46