Federation services, complete

@tvvignesh – some knowledge on hooking federation directly into stitching here if that's something you're looking to do.

@gmac Thanks. But since I have completely moved away from federation to stitching now, this may no longer be needed for me (I hope stitching works well for my usecase as I go along, but I will learn along the way anyways).
Btw, thanks a ton for this: https://github.com/gmac/schema-stitching-demos/tree/master/hot-schema-reloading
This was something I had been discussing with @yaacovCR and I was looking for a reference implementation I can use with stitching directives and that definitely helps a lot. I am going with this 🎉
Just have one doubt though. Let's say that the upstream SDL of one of the services is invalid (I don't know when/if this can happen since I can't put my mind around it), but in such cases would the gateway use the valid SDL / invalid SDL
Sorry for the lack of activity/response so far. I was working on some of the issues I was facing with Kubernetes and had not concentrated on this. Getting back on track now.

Federation SDL to valid SDL is really cool, and I love how you then construct the merge config automatically. Wondering if we should export this as helper perhaps within a separate library or within graphql-tools.... Should endpoint be final merge config as you have, or just directives that fit stitching directives format...

@tvvignesh — I do have ambitions to make a “Testing and Versioning” example, though it’ll be more involved and will have to put some thought into it. Long story short is that you commit schemas into a registry where they are tested together, and then have hot reload simply hit the registry for a precomposed version that increments. You can read about how Apollo does it here: https://www.apollographql.com/docs/federation/managed-federation/overview/. Personally, I find their approach to be a little sloppy in that they consider composition === integrity, which is not always the case in my experience (i.e: you can have a perfectly valid schema that breaks due to implementation). We still don’t do any kind of hot reloads at my company; we commit schemas into the gateway app’s repo where they have mocked service tests run against the composed schema, then redeploy. It’s slow, but the integrity is very high. We’ve talked recently about putting some effort into designing a system that maintains our integrity while speeding up versioning and deployment. This future example may be a rough prototype of it!
@yaacovCR — converting fed SDL to stitching SDL is interesting, but seems like an extra step in the ultimate goal of SDL to subschema config. Either way, my only hesitance to a formal implementation somewhere is the maintenance... one of the nice things of an example like this is that it shows you pretty directly how to do it without promising that it’ll work for your exact schema which inevitably always contains some bizarre outlier.

@gmac @yaacovCR I used https://github.com/gmac/schema-stitching-handbook/tree/master/hot-schema-reloading as the inspiration, made some changes and I can confirm that the schema hot reloading is working great when any of the subservices are changed
I also had a look at the new Video series about local/remote schemas, single record typemerging, array batching made by you and this really is fantastic. Clearly explained (Wish I had this a month back 😅)
Working on Type merging now with stitching-directives. Will update this thread.

Awesome!! I’ve also got a demo of hot reloading in the works using a GitHub repo as a schema registry to poll. Should have that demo released in the next week

@gmac Great! I use Gitlab though (but since it is working by loading a remote SDL anyways I can work with this for now).
Also, I had a doubt. I wanted to do array batched merging using stitching directives and did refer https://github.com/gmac/schema-stitching-handbook/tree/master/stitching-directives-sdl but did not find an example for that.
So I went through the tests @yaacovCR had written. Is this the one to be used for array batched type merging?
test('adds key and argsFromKeys functions with argsExpr argument using a fully qualified key', () => {
    const typeDefs = `
      ${stitchingDirectivesTypeDefs}
      scalar _Key
      type Query {
        _user(key: _Key): [User] @merge(argsExpr: "key: [[{ id: $key.id }]]")
      }
      type User @key(selectionSet: "{ id }") {
        id: ID
        name: String
      }
    `;

I am not sure, but does your example here: https://github.com/gmac/schema-stitching-handbook/blob/master/stitching-directives-sdl/services/products/schema.graphql use array batched type merging? Cause I see keyField which I guess is part of array batched merging. Am I right in my assumption?

I actually don’t know. I’m not very familiar with the SDL api yet, that’s
all @yaacovCR. I’ve got a draft PR open for distilling knowledge down into
a formal docs page. It has a ways to go. For me, I still don’t understand
how _Key works.
On Wed, Dec 9, 2020 at 5:58 PM Vignesh T.V<EMAIL_ADDRESS>wrote:

I am not sure, but does your example here:
https://github.com/gmac/schema-stitching-handbook/blob/master/stitching-directives-sdl/services/products/schema.graphql
use array batched type merging? Cause I see keyField which I guess is
part of array batched merging. Am I right in my assumption?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/gmac/schema-stitching-handbook/pull/6#issuecomment-742116207,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFRROC6VVDHVLVPUHXYYHDST76HXANCNFSM4UPSD3OA
.

@yaacovCR @gmac Got type merging working finally with stitching-directives 🎉🎉
Currently just went with batched merging. Will probably look at array batched merging later after I get some more clarity on it.
Thanks for all your help. It actually turned out to be quite simple 😀

Btw @merge has too many keys though: directive @merge(argsExpr: String, keyArg: String, keyField: String, key: [String!], additionalArgs: String) on FIELD_DEFINITION
keyArg,keyField,key - All of which are not explanatory to be honest. But anyways, I don't want to bother you guys again with this. So, I will leave it at this.

@tvvignesh – I just added notes to the in-process chapter about Registry and Versioning... https://github.com/gmac/schema-stitching-handbook/compare/versioning?expand=1
While the narration isn't done yet and some of the code will get cleaned up, it all works and the instructions are complete enough that they should walk you through it. That uses a GitHub repo as a schema registry to manage versioned releases that get hot-reloaded into the production gateway when commits are made.

There is no difference between batch merging and regular merging, it is introspected automatically from the return type of the resolver that you attach the merge directive to.. keyArg is when you want to specify which argument that you should send the key to for that resolver... keyField is for when you want to use only a simple key rather than an object and so you specify which field from the selection set will be that simple key, key is when you need to customize how to map the selection set to the argument via a custom key....
Basically if you have got it working with without batching, you should just be able to change the return type to a list and everything should work automatically

@yaacovCR Thanks for the clarification (Still trying to digest this part alone). Since the JS implementation uses argsFromKeys, keys for batch merging and args for regular merging, I was looking to understand if I have to change the directives as well similar to that. But from what you are saying, I guess it infers it automatically.
Currently something like @merge(keyField: "id") is working well for me since all my types have an ID field which is what I would be using to stitch together multiple types from multiple services. So, I guess I need not use the rest like keyArg or key

After doing this, I feel like I have more control with the schema on my end unlike federation where a lot of things were abstracted away from me. While that was simple, I feel like this is powerful to work with as we scale and the modular nature of stitching with graphql-tools would help in just using the parts as needed without any bloat. Liking it ❤ And with this handbook now in place along with the video series by @gmac new devs should feel right at home 👍
I would say that if the default naming of the arguments is reviewed again, I don't think there is any other problem with stitching.
Also, maybe the gateway can also become a lib in the future like @graphql-tools/gateway with built in support for stitching-directives, hot reloading, and so on which people can use if needed or build their own putting everything together.
And the registry can become @graphql-tools/registry with multiple backends like @graphql-tools/backend-github , @graphql-tools/backend-localstorage and so on in the future.
A lot of potential to expand this especially keeping this handbook as a reference implementation of everything.

@tvvignesh - awesome, glad that versioning demo was insightful. All the instructions worked okay? Also stay tuned, next up after that one will be testing schemas, so that you can be running CI on your registry using your own validations that you can be as stringent or as lax as you want to be with... versus federation where it just checks if your schemas compose together and green-lights the release assuming they do.


test(connector-fabric): consolidate 3 tests into deploy-lock-asset.test

This is making the test case harder to read but shaves off easily
10 to 15 minutes from one of our slowest CI jobs which can take up to an
hour to run when the GitHub runners are feeling lazy.
That above is my only justification for it. The test cases I'm consolidating
are relatively stable at this point (took us years to get here but now they
are passing with a high ratio and the false negatives have pretty much
disappeared).
We are downloading and launching the fabirc AIO ledger 10+ times which
is very resource intensive and this could help make a dent in it.

[x] Have git sign off at the end of commit message to avoid being marked red. You can add -s flag when using git commit command. You may refer to this link for more information.
[x] Follow the Commit Linting specification. You may refer to this link for more information.

Character Limit

[x] Pull Request Title and Commit Subject must not exceed 72 characters (including spaces and special characters).
[x] Commit Message per line must not exceed 80 characters (including spaces and special characters).

A Must Read for Beginners
For rebasing and squashing, here's a must read guide for beginners.

LGTM with small nitpick.
BTW When I was creating iroha2 tests I was thinking about using common ledger setup for all the tests, I even moved env setup to separate class for that reason (see https://github.com/hyperledger/cacti/tree/main/packages/cactus-plugin-ledger-connector-iroha2/src/test/typescript). In the end I did not do that (to adhere to general principle that tests should not depend on each other too much and because iroha starts really quickly), but maybe this could be considered for fabric connector where the ledger takes a looong time to start up.

@outSH Agreed, 100%. Now that I know that there's some support for the idea I'll consolidate a few more tests into one! Stay tuned for an upcoming PR :-)

@petermetz LGTM, thanks! Please remember to update the commit/PR title to include the new test name, apporving to speed this up :)

@outSH Done and done, thank you for catching that and also much appreciated on the pre-approval :-)


Hide result numbers

Hey,Thank you in advance for your good plugin
but I need to hide numbers in search result OL
How i can do it ?

@Mohamm6d I'm very sorry for never responding to you here. As my silence likely hinted, I stopped maintaining this project long ago. I'm now officially deprecating it to make that clear.
If you'd like to continue supporting a forked version, please reach out on Twitter (@kidjustino) to have it listed in the README.


Pool size calculations are missing closing angle brackets after second criteria

[Enter feedback here]
Under the "How do I choose the correct pool size" section there are two formulas provided to properly select the eDTU/vCore assignment of your elastic pools.  The calculations use angle brackets to contain criteria to compare.  In both calculations, the last criterion is missing the closing angle bracket (>).
For the DTU-based calc, this is the current line:
MAX(<Total number of DBs X average DTU utilization per DB>, <Number of concurrently peaking DBs X Peak DTU utilization per DB)
And it should be corrected to:
MAX(<Total number of DBs X average DTU utilization per DB>, <Number of concurrently peaking DBs X Peak DTU utilization per DB>) (Note the addition of a '>' character just prior to the closing parenthesis)
The exact same omission occurs in the vCore-based calc that follows on the page.

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

ID: 954fdc5f-1572-c6a5-eb19-22e890b1174f
Version Independent ID: b5776cca-f233-4c34-a649-9a61324a2edd
Content: Manage multiple databases with elastic pools - Azure SQL Database
Content Source: articles/azure-sql/database/elastic-pool-overview.md
Service: sql-database
Sub-service: elastic-pools
GitHub Login: @oslake
Microsoft Alias: moslake

@timkatje We are checking this and we will get back to you.

@timkatje We have raised Pull Request to make the change - https://github.com/MicrosoftDocs/azure-docs-pr/pull/125878
Once approved the changes will be visible in approx 24 hours.


RFP - diff-highlight

https://github.com/git/git/tree/master/contrib/diff-highlight
This is part of Git, but Git for Windows does not ship it. Not sure whether this can be packaged?
If this is packaged, it should depends on Git. Shall version be just dates or same as Git?

It's just a Perl script (by the looks of things) so don't see any reason it can't be packaged. I would suggest it keeps the same version as the version of Git that last updated it.

Ok, but then it would be difficult to automate. It is trivial to get latest updates of a file, but “finding a tag that contains specific commit” isn't trivial. The best way I can think of is to clone the repository and run local Git command. The GitHub website makes request to https://github.com/git/git/branch_commits/4551fbba141e0b2e4d16830f76f784e9c960bdf8 for this list, but it isn't documented, so I expect it to change without notice.

Ok, but then it would be difficult to automate

How often is it actually going to / does it change?

Not often; I guess once or twice a year. I'll update it manually.

Hi @FranklinYu , could you explain how you install it manually until now on your windows OS? Which files are needed, where should they been copied? Depends only on git (and maybe on Perl)? How to check easily to know if the installation of the script is successfull? Thanks in advance!

@chtof It should depend on Git only. I think it’s using the perl in the MSYS2 that came with Git.
As for what to package, I read the Makefile:
https://github.com/git/git/blob/master/contrib/diff-highlight/Makefile
I think we need to concatenate DiffHighlight.pm and diff-highlight.perl, then prepend the shebang.

This package exists, but need triage now.
Do you handle it ? @FranklinYu ?


Remove non_exhaustive from error structs

Currently we have a few public error structs the sole purpose of which is to hide error internals. As such they have private (pub(crate)) fields and cannot be constructed by lib users, they do not need the non_exhaustive attribute for this reason.

This is follow up to #44


<EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>

Is this project still maintained? If so, could you please update the peer dependency to latest react version to hide this warning during application build?

Hey, @renepardon. I'm currently not working on this project (see the status update), but critical issues and vulnerabilities should be fixed even in this mode.
Do you mind shipping that dependency version bump in a pull request? If there's any build/test failures I'll gladly help to resolve them. Thanks.

@kettanaito If @renepardon has not yet I would be happy to oblige

Hey, @ajschlosser. That would be great, I could use some help with this issue. Let me know if you need any help in preparing a pull request. Thank you!

This is till an issue
$ npm install react-advanced-form --save
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/react
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/react-advanced-form
npm ERR!   react-advanced-form@"*" from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!
npm ERR! See C:\Users\ashaheen\AppData\Local\npm-cache\eresolve-report.txt for a full report.

npm ERR!      C:\Users\ashaheen\AppData\Local\npm-cache\_logs\2021-03-25T15_35_11_599Z-debug.log


Feature/react router

Implemented react-router. This closes #5

Codecov Report

Merging #20 (a3c17fa) into main (8b5e4b1) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #20   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines            1         1
=========================================
  Hits             1         1

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 8b5e4b1...a3c17fa. Read the comment docs.


🐛 Issue with unsaved files

VS Code version
1.92.2
Extension version
2.3.0
Biome version
1.8.3
Operating system

[X] Windows
[ ] macOS
[X] Linux

Description
I sometimes abuse unsaved files in an open project as a notepad, when I do that, CSC seems to try to run on them which fails with the following message

Steps to reproduce

Open new file in a project that uses Biome
Don't save it
Write something to trigger Biome

Expected behavior
No errors
Does this issue occur when using the CLI directly?
No
Logs
Copying binary to temporary folder: file:///home/stan/.vscode-server/data/User/workspaceStorage/af22f4201bf263a53fb80c4d8a54ca94/biomejs.biome/biome
Executing Biome from: /home/stan/.vscode-server/data/User/workspaceStorage/af22f4201bf263a53fb80c4d8a54ca94/biomejs.biome/biome
[cli-stdout] data 43
[cli] exit 0
Connecting to "/home/stan/.cache/biome/biome-socket-1.8.3" ...
[Info  - 4:37:44 PM] Server initialized with PID: 207559
[Error - 5:17:41 PM] Request textDocument/codeAction failed.
  Message: failed to access range Range { start: Position { line: 7, character: 0 }, end: Position { line: 7, character: 0 } } in document untitled:Untitled-1 LineIndex { newlines: [0, 45, 46, 83, 84, 135, 136], line_wide_chars: {} }
  Code: -32603
failed to access range Range { start: Position { line: 7, character: 0 }, end: Position { line: 7, character: 0 } } in document untitled:Untitled-1 LineIndex { newlines: [0, 45, 46, 83, 84, 135, 136], line_wide_chars: {} }

Caused by:
    position Position { line: 7, character: 0 } is out of range

We recently released a nightly version, which takes a slightly different approach. Can you check if it solves your issue?


Shell environment resolution terminates with incorrect shell

Type: Bug
VSCode does not resolve the correct shell environment for use by the application, breaking plugins which rely on it.
The $PATH variable used by VSCode is:
/home/timo/.local/bin:/opt/Xilinx/vivado_student/Vivado/2017.2/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin:/home/timo/.local/bin:/usr/local/go/bin

This matches my $PATH as defined in ~/.profile, but not the one in ~/.bashrc.
I use zsh shell and os is Arch

It's hard to tell what exactly is going on in these types of issues. VS Code's environment (which I'm assuming is the first one?) will be either inherited from a terminal is launched from there (which we barely touch), if you launch from a dock we will attempt to source a "development environment" you would normally use in your terminal. That's done here:
https://github.com/microsoft/vscode/blob/8db902c64c2d6ce1dd19340336df0da35e3b802a/src/vs/platform/shell/node/shellEnv.ts#L102
My guess is that somehow we're not able to source the environment, and therefore the extension host is getting a version without having run .bashrc. And the terminal is working as we run a non-login shell there.
My recommendations are:

Investigate on your side, debugging into this VS Code function to see what's happening
Move any environment stuff to your profile (ie. what happens on login), and keep only things needed for an interactive session in your .bashrc (ie. what happens in a terminal)
Make .bashrc idempotent and run it from your .profile/.bash_profile (easy fix to workaround a lot of these types of problems)


Dropdown.js - Inconsistent option property naming.

Pretty simple
From https://github.com/Dogfalo/materialize/blob/master/js/dropdown.js#L11-L20
 var defaults = {
      inDuration: 300,
      outDuration: 225,
      constrain_width: true, // Constrains width of dropdown to the activator
      hover: false,
      gutter: 0, // Spacing from edge
      belowOrigin: false,
      alignment: 'left',
      stopPropagation: false
    };

All properties are camelCase except for constrain_width which is underscored.  Took a look in the code to find out why this option wasn't working for me intuitively.
The documentation DOES note this correctly but it should probably be swapped out to match all other options, and deprecated, or at least a secondary check added for convenience on the option.

Fixed in 66d37f0


Cannot find module when dynamically require images

Versions

nuxt: 2.14.6
node: v12.20.1

Reproduction
I don't know if it's a bug or a feature, but when I want to dynamically load images stored in assets I get the error "Cannot find module..." =)

As far as I understand, when I load an image, everything in the string is taken. To confirm this theory I created a project in assets which has 2 files: README.md and icon.png.

If you connect the icon.png by string then everything works correctly:

But if the icon.png is placed in a variable, then everything in assets is loaded, because there is a string value '~/assets/' in require and README.md connection error is displayed.

The problem is that even if I use the variant from the documentation:

it still loads all the .jpg images recursively located in '~/assets/img/'. But I have a lot of images in subfolder img which I don't need to load and waste time optimizing them. What's the best way to do this?

As far as I understand, when I load an image, everything in the string is taken. To confirm this theory I created a project in assets which has 2 files: README.md and icon.png.

This is correct. Webpack takes every possible file into account that could match the expression. This is the reason why you have to be a strict as possible (e.g. assets/img/some/subfolder/${image}.jpg instead of assets/img/${image}. Those are in your bundle but might not necessarily be loaded all together.
Further read: How to load dynamic images in Vue and Nuxt with ease


[MRG] stc.plot resorts to mpl when Mayavi not installed.

@agramfort wanted an alternative plotter for stc.plot. It is really slow, but useful for training purposes.

The result looks like this

Rotating it takes too long so this is purely for creating still images.

Neat.

You should be able to use the same colormap(s) that PySurfer uses.
You should probably set the axes aspect ratio to "equal" if there is such a thing for 3D view.
Can we disable interactive rotation? Better to prevent people from accidentally making their interpreter hang.

Codecov Report

Merging #4086 into master will decrease coverage by 0.01%.
The diff coverage is 68.13%.

@@            Coverage Diff            @@
##           master   #4086      +/-   ##
=========================================
- Coverage   86.12%   86.1%   -0.02%
=========================================
  Files         350     350
  Lines       63553   63627      +74
  Branches     9689    9702      +13
=========================================
+ Hits        54734   54786      +52
- Misses       6137    6157      +20
- Partials     2682    2684       +2

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 e5b0447...a60fb91. Read the comment docs.

This is ready for review.

I just tried
%run plot_read_stc.py
fig = plt.figure()
stc.plot(figure=fig, subject='sample')

and I get a big blue blob :'(
I am putting this PR back to WIP

Seems that _morph_buffer wasn't doing the same thing after all. Now the dependency on surfer is back again.

Seems that _morph_buffer wasn't doing the same thing after all.

Perhaps not identical inputs/outputs but the operation should be the same. Can you investigate instead of using surfer?

On Thu, Apr 27, 2017 at 3:48 PM, Eric Larson<EMAIL_ADDRESS>wrote:

Perhaps not identical inputs/outputs but the operation should be the same.
Can you investigate instead of using surfer?

+1

Spent considerable amount of time trying to debug why the vertices wouldn't fit only to realize later that I was trying to plot data morphed to average brain on sample surface.
The problem with _morph_buffer was that it couldn't handle vertice numbering that's not running numbers (like is the case with our sample data). This still needs cleaning and there might be a better way of doing this.

The problem with _morph_buffer was that it couldn't handle vertice numbering that's not running numbers (like is the case with our sample data). This still needs cleaning and there might be a better way of doing this.

In general the subset of vertices will not be consecutively ordered. fsaverage in an ico5 configuration just so happens to have vertices [np.arange(10242), np.arange(10242)], but in general it's not the case. Our morphing functions work fine with all source spaces, though. So I don't understand the statement about _morph_buffer (which IIRC is used under the hood by stc.morph) requiring consecutively numbered vertices.

It was handled in the morph_data function. Should be good now.

@jaeilepp can you share a full code snippet that produces a nice figure and share the generated image?
also you should not display the 3d axis, show the brains on black background so changing backend will produce figures as similar as possible.
thanks

Code snoppet:
import os.path as op

import mne
from mne.datasets import sample

data_path = sample.data_path()
subjects_dir = op.join(data_path, 'subjects')
fname = op.join(data_path, 'MEG', 'sample', 'sample_audvis-meg')

stc = mne.read_source_estimate(fname)
stc.plot('sample', hemi='rh', smoothing_steps=15, time_viewer=True,
         backend='matplotlib' , subjects_dir=subjects_dir, initial_time=0.085)

ok cool
now you need to blend the curvature colors with the activations to make it
look like mayavi
see for example this code:
https://github.com/agramfort/matlab_mesh_toolbox/blob/master/blend_data.m
the important line is this one:
https://github.com/agramfort/matlab_mesh_toolbox/blob/master/blend_data.m#L117

This is going to be really useful and nice. I am very happy to see this happening. We should also figure out if the figure space can be used more efficiently, I'd try to enlarge somehow the brain.

ok we're making progress. i am a bit disturbed by the alpha transparency of the surface. it should not be transparent.
also I am wondering if we could not display a low res surface when using matplotlib as it takes 45s to render on my machine. It's not really usable although it fixes a real problem.

We can add a spacing argument and use the source space surface decimation
code to get the subset of indices to display. Should be really quick to
compute.

It seems that decimate_surface messes up the order of the triangles making it impossible to plot the data on it. I don't know if the order can somehow be retained.

you would need to use the source space subsampling strategy.

project data to full resolution
make an ico5 surface to subsample the full resolution surface
show the low res surface

ok?

I finally got the spacing to work. I also enabled the rotating again, since it's much faster when decimated enough. Ready from my end.

it should not

if it passes on travis is ok for me.

How much coverage do we lose for omitting the all case?
Are those numbers for your system or Travis?

Just saw you said your system. Can you check Travis? I'm guessing it's more
like 30 or 60 seconds there, which isn't worth it to me. (That's 0.5-1% of
our allotted testing time for something that is both rare and not
computationally / implementation-wise all that complex.)

Sorry I mean 1-2%

Yeah 30s is probably close to the truth. Removing 'all' wouldn't decrease coverage very much. I'll remove it.

mpl still has no proper line edit, but the slider works quite well as a time_viewer IMO.

for me the interaction with the slide does not work. I can click on it or
press arrows but nothing happens
also I see now white spaces around the brain

I suppose this is another MAC backend problem. I can't figure why it wouldn't work.

it works for me with qt4 backend
can you make the slider less thick and make it respond to arrows?
one arrow press move to next samples and shift+arrow move by steps of 10ms?
thx

IIRC the modifier won't work on MACOS either. Do we recommend qt backend somewhere in the docs?

Yeah, the slider probably got carbage collected (I thought I had it stored). I don't see any edges around the brain, only the figure margins. I'll see if those can be removed. Also, for some reason, shift modifier wouldn't work for me. I used control modifier instead.

@jaeilepp please rebase and let us know when travis is happy

you're still moving by 100ms and not 10ms with ctrl+arrow. Maybe i was not clear before.
the arrow should just make you move by one sample in the stc.
besides +1 for MRG

you're still moving by 100ms and not 10ms with ctrl+arrow. Maybe i was not clear before.
the arrow should just make you move by one sample in the stc.

I don't get it. Arrow key moves you one sample (10ms on sample_audvis-meg) and ctrl+arrow moves you ten samples.

ah ok. I had in mind to step by 10ms (not 10 samples) with ctrl+arrow
this would be consistent with mne_analyze

@jaeilepp can you rebase?
also do we have now steps that are consistent with mne_analyze?
thx

Step size with ctrl in mne_analyze is 5. Now it is here too.

perfect ! thx heaps @jaeilepp
this is very very useful.


Refactor builder class

What have I done

Get the SpoonacularBuilder class out of the Builder class's hierarchy.
Delete the build_user method from the SpoonacularBuilder class.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Files with missing lines
Coverage Δ

webpage/modules/builder.py
95.56% <100.00%> (+1.09%)
:arrow_up:

... and 16 files with indirect coverage changes

🚨 Try these New Features:

Flaky Tests Detection - Detect and resolve failed and flaky tests


mib2dev.py: automatic values hit ConstraintsIntersection

While generating a data set, mib2dev.py, I got the error:
 *** Inconsistent value: <ConstraintsIntersection .... '<ConstraintsUnion object at 0x7fd9feb73438 consts <SingleValueConstraint object at 0x7fd9feb73400 consts 1, 2, 3>  .... # Scalar PROFLINE-STARG2-MIB::starg2SetClockLockTo (type LockToSourceModes)

Processing this MIB entry:
starg2SetClockLockTo OBJECT-TYPE
   SYNTAX          LockToSourceModes
   MAX-ACCESS      read-write
   STATUS          current
   DESCRIPTION     "Lock the internal clock to the time received from an external source.
                    1 = off
                    2 = sntp over mgmt
                    3 = ntp over mpe"
   ::= { starg2SetClock 2 }

Since the script suggest me the proper values, is it possible to add an option so that the generator just pick one of them (randomly) ?
Is the option:  [--automatic-values=<max-probes>]  is the one used for that ?
EDIT: After I tried 15 MIBs from different companies, 8 cannot be synthesized ... mainly because of
ConstraintsIntersection, or "invalid literal for int() with base 10" (generated value is a string 'kept Jaded...'
Thanks for this project,

is it possible to add an option so that the generator just pick one of them (randomly) ?

Depending on the underlying type of  LockToSourceModes, you may try --unsigned-range=<min,max> or --integer32-range=<min,max> option to set the range of all values of this type for the MIB you are simulating.
Once you have the initial .snmprec file, you could edit it by hand to adjust the values.
There is currently no option to limit the values of specific MIB objects, but it could be added.

The --XXX-range options can't be used since they affects all the value.
The best would be to add the option, since the possible values are present in the error message.
One problem is that we don't want to go interactive - an interim solution would be to add an option to: output the variable name (as MIB::var) that can't be generated on stdout and continue.
The idea here is to generate some data ASAP when we develop. an SNMP driver.

That seems doable. Let me see if I get a bit of time to work on this.

That seems doable. Let me see if I get a bit of time to work on this.

Well, not quite, apparently. Trouble is that whenever we fail on SNMP table index, we effectively fail on the whole table row... There is a good chance that, for some tightly constrained table indices, we are unlikely to ever pick the right value. That leads to skipping the whole table, what makes this tool next to being useless.
That makes me think that the better course of action would be to "reverse" the constraints meaning the ability to generate samples out of constraints.
That would be the reflective operation in the sense that what we currently have is that we feed a value to the constraints tree to see if the value fits the constraints. What we'd need, I think, is given the constraints tree we produce a value which would satisfy the constraints.

Indeed, that would the best solution and a more architecturally sound solution.
Thanks.
Still, the option to "not got interactive and just log the OID that cannot be resolved" would be an easy and quick way so that the script continue to run as long as possible.


angular cli not conform with angular documentation

Angular Cli projects are not conform with the angular documentation. For example
import {HTTP_PROVIDERS, Http} from 'angular2/http';

does not work

@doczoidberg , try this => import {HTTP_PROVIDERS, Http} from '@angular/http'
With the rc version, all namespace have change. https://github.com/angular/angular/blob/master/CHANGELOG.md

c:/temp/app1e/tmp/broccoli_type_script_compiler-input_base_path-rlvaekkB.tmp/0/src/main.ts (5, 30): Cannot find module '@angular/http'.

Maybe it's not present on your package.json. Can you check it ?

@doczoidberg If you are using the latest version of angular-cli you don't have the package installed by default. You need to run npm install @angular/http --save

I installed the package and added  "@angular/http": "^2.0.0-rc.1", to package.json
but still get the same exception

works now, thanks

You should also inject HTTP_PROVIDERS in main.ts
bootstrap(HELLOAppComponent, [HTTP_PROVIDERS]);


One Setup for Tilemap

The functionality is there.  I am going to commit this, however there will be many improvement tasks along the way with this that i will be entering.


Pass along data to action.open callback when returning a promise

When actions return a promise, It would be convenient if callbacks registered with action.open received the original payload. The main use case being for performing optimistic updates before hearing back from the server.
function updateFoo(data) {
  return ajax.patch('foo', data)
}

repo.push(updateFoo, { foo: 'bar'})

// ...elsewhere in a domain or effect:
register() {
  return {
    [updateFoo.open]: data => console.log(data) // undefined instead of { foo: 'bar'}
  }
}

Here's where the work needs to be done: https://github.com/vigetlabs/microcosm/blob/master/src/coroutine.js#L51-L60

This is in the current alpha, if you'd like to give it a spin:
yarn add<EMAIL_ADDRESS>

It's working great! A+


update codeowners

Fixes # .
Changes proposed in this PR:

add my account to codeowners

we are going to close this and merge #1824 which includes more new codeowners


Use instanceof instead of is in type guard function syntax

The type guard syntax introduces a new keyword is
#1007 Support user-defined type guard functions
#3262 Custom type guard function
(the latter was implemented by @tinganho and merged in master recently with https://github.com/Microsoft/TypeScript/commit/6e69a9e380ea793e4d13409afc25ed590462956f )
The syntax is quite elegant and clean, but can be further improved. It's generally a good rule to try and avoid introducing new keywords. Especially when a very similar one already exists in the language.
I suggest we use the existing instanceof instead. The meaning fits, and it's very familiar to a lay JS folk.
In simple words, to a person with JS background, but a limited TS one (1) leaves more to guess than (2):
function isApple(p1: any): p1 is Apple { }    // (1)   <-- man... let me remember what's the difference...

function isApple(p1: any): p1 instanceof Apple { }    // (2)   <-- hello, I know you!

I do agree that for the C# audience they both are fine. But similarly to bool/boolean swap of the early TS versions, preferring JS over C# is the right thing.

I'm not sure the is keyword is actually based on C#. That was not my interpretation.
I will say instanceOf has a prototype connotation, whereas is is intended to describe our structural type system. In that light, I actually think instanceOf could be misleading. The user might think that they should only use it when the function implementation does an instanceOf check.

Experience with re-using an existing value-position keyword as a type-position operator with a different meaning has not proven it to be a good approach.
People are continuously confused about the difference between type-position typeof and value-position typeof -- people try to write typeof T in a type position where T is a type parameter, or think that typeof returns something other than a string.


[Darwin] Restrict inference of the simulator environment

The *-simulator target triples have been used mostly consistently in build systems for
several years to indicate simulator targets... except for Swift's own CMake configuration
and build-script, which was still not doing it until recently. There are still .swiftinterface
files around that depend on the inference. Narrow inference of the "simulator" environment
to a change to the triple (vs. a separate predicate), and warn about it in the driver to
help other tools migrate.
This is as far as we can go toward completely removing the hack (rdar://problem/35810403),
because of the need to continue supporting existing .swiftinterface files.

@swift-ci please smoke test macOS

@swift-ci please test Linux


Android studio3.1.1

This change is not working I assume (never did before at least):

maven { url "https://maven.google.com" } => google()

Works locally of course, but the jitpack build will fail...

I tested it and it works properly
https://github.com/hannesa2/changelog-1/commit/64e1ad257c73c5ef0930a91e7c6d4ed6bd4c6b41

Good to know, then I will merge it. But I'll change the definition of the support lib version, as I use another format in all my projects. Will look like the one I use here:
https://github.com/MFlisar/DragSelectRecyclerView/blob/master/library/build.gradle


Decide where /catalog route should go.

Done when we decide where the /catalog route should go, and if it's a page we already have, go ahead and do it. If not, then make a new ticket for creating that page.
Currently /catalog goes here:

@afred: This will be handled by #139: We'll redirect to add the missing "access" facet, and then we'll get that list. Keep this around until that is merged just to be sure.

Yep, just checked and it's working fine.


Why not remove environment variables in auth.xauth and clients.ReaderClient?

Hello, I have created a fork from your master branch in which auth.py and clients.py remove mentions of required_from_env environment variable settings. Thus, my readability docs are changed in the following manner:
token_key, token_secret = xauth(username="a user's email",
      password="a user's password", consumer_key="client's consumer key",
      consumer_secret="client's consumer secret")
client = ReaderClient(token_key="a user's key", token_secret="a
      user's secret", consumer_key="client's consumer key",
      consumer_secret="client's consumer secret")

Why not incorporate these features in the master python-readability-api branch?

Hi @tanimislam, we need to be able to pass these variables via env for a variety of reasons but it's not a requirement. You can always choose to pass them explicitly.
token_key, token_secret = xauth('username', 'password')
client = ReaderClient(
    token_key=token_key,
    token_secret=token_secret,
    consumer_key='key',
    consumer_secret='secret')

this does not work for me:
    token_key, token_secret = xauth(READABILITY_USER, READABILITY_PASSWORD)
    rdd = ReaderClient(
        token_key=token_key,
        token_secret=token_secret,
        consumer_key=READABILITY_CONSUMER_KEY,
        consumer_secret=READABILITY_CONSUMER_SECRET)

TypeError: xauth() takes at most 1 argument (2 given)
When I try this:

    token_key, token_secret = xauth(username=READABILITY_USER, password=READABILITY_PASSWORD)
    rdd = ReaderClient(
        token_key=token_key,
        token_secret=token_secret,
        consumer_key=READABILITY_CONSUMER_KEY,
        consumer_secret=READABILITY_CONSUMER_SECRET)

  File "..../env/local/lib/python2.7/site-packages/readability/core.py", line 14, in required_from_env
    "Required argument '{}' not supplied and not found in environment variables".format(key))
ValueError: Required argument 'READABILITY_CONSUMER_KEY' not supplied and not found in environment variables

What am i doing wrong ?
$ pip freeze | grep readability-api
readability-api==1.0.1

@jpcaruana you are right, there's still a bug, this time in the xauth implementation. I have it fixed, will push here and to pypi in a bit.

maybe using a lambda when reading the arguments would work fine

The issue is that I'm calling the function inside the kwargs.get and supplying the function call as the default return value if the value is not present in the get. I had erroneously thought that the function call would short circuit if a value for the given key is present but the function call is always made for some reason. The fix is easy, move from kwargs.get(foo, some_function()) to kwargs.get(foo) or some_function() which gets us the correct short circuit functionality I had intended. It should be pushed soon.


[stable/kibana] Custom Pod Label Support

I think this is a feature request as I couldn't see the functionality in the code.
I would like the ability to set custom pod labels to the Kibana deployment.

Why are you need to  set custom labels on the Kibana pod?
If this feature need, i can send a PR to add this feature.

I usually add custom labels such as owner / team etc to all pods in the cluster for reporting and tracking purposes.

@amitlalani I send a PR to implement this feature, you can see it.
I don't have an environment to verify it. If you have any questions when use it, i hope you can tell me. Thanks.


Dynamically update lib version and publish all plugins

TL;DR
Please replace this text with a description of what this PR accomplishes.
Type

[ ] Bug Fix
[ ] Feature
[ ] Plugin

Are all requirements met?

[ ] Code completed
[ ] Smoke tested
[ ] Unit tests added
[ ] Code documentation added
[ ] Any pending items have an associated Issue

Complete description
How did you fix the bug, make the feature etc. Link to any design docs etc
Tracking Issue
https://github.com/lyft/flyte/issues/
Follow-up issue
NA
OR
https://github.com/lyft/flyte/issues/

Codecov Report

Merging #364 (a8da42b) into master (2b9b0d1) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #364   +/-   ##
=======================================
  Coverage   96.00%   96.00%
=======================================
  Files           2        2
  Lines          75       75
  Branches        8        8
=======================================
  Hits           72       72
  Misses          1        1
  Partials        2        2

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 2b9b0d1...a8da42b. Read the comment docs.


[QUESTION] Multi-select Audiences?

What is the context of your question

[ ] Installation
[ ] Learning Pathways Content
[ ] SharePoint Success Content
[x] Administration
[ ] Custom Content
[x] Viewer Web Part

What is your question
Is it possible to select multiple audiences per playlist? Right now I don't think there is or am unaware of a configuration option which would make this possible. If a change is made, it would have to be on both the Administration and Viewer Web Part side of things.
It seems logical that a playlist could be targeted towards, say, a Change Manager and a Content Editor.

I'm prepared to take the code and modify it myself so please let me know if I've missed something :).
Many thanks!

Hi @tandrasi ,
Thank you for submitting your query here. Unfortunately, there is no option available in the webpart to configure the Audience dropdown to MultiSelect. This has to be changed/configured in the code only and it requires changes in multiple other places which will impact the current functionality, so I will let @dcashpeterson to advise on this.
Hi @dcashpeterson, Could you please advise on this query? Thanks

@tandrasi Thanks for the interest in adding multiple audiences. This is an open source project and we would welcome your contribution. If you would like to submit a PR for this that would be great. A few things to keep in mind here.

This might require an update to the data model - if so that will require a major version update and with that an update path (see Update Service)
As you mentioned you would need to update both the Admin web part and the Content web part
This change would only apply to custom content as the Microsoft provided content only supports 1 audience.

Please reach out in this ticket if you have any questions.

@dcashpeterson hi, just an update, I had a quick look and it's relatively simple, but the client has opted for a different approach meaning that we no longer need multiple audiences (well, they declined the additional cost of development time) so I won't be able to complete this work. Even if it would take me a couple of hours, I'm stretched very thin in professional and personal life.


Fix typo : "of of" -> "of"

Hey there, thanks so much for contributing! The file you edited is the built page for the documentation, so you’ll need to edit the source code of the page first, then generate the built site as well. (We’re working on improving this, but for now it’s a bit of a pain.)

If you haven’t yet cloned the repo onto your computer and run npm install, do that now :wink:
Edit src/templates/index.html on your branch, making the correction
Run npm run doc from your terminal
Push back up to this branch and let me know you’ve updated it

Let me know if you need any help, and thanks again! 😀

Hey! Oops, sorry, I missed the built files explanation in the README...
I hope it's complete this time ! 😉
Thanks for your time and the detailed procedure 😄

🙌🏻! Thanks so much. I'm going to take another look at this on my computer later, but it should be good to go 😀

Hi ! Any update on this pull request ? 😉

Sorry Thomas! This PR looks mergeable but when I look at the changes tab, it looks like you force-pushed and something went wrong. Could you take a look into that? Thanks :)

Sorry, I tried to solve this problem but closed the PR by mistake. I preferred to create a clean new one here : #65. I hope it will be fine this time, and sorry for the mess here...

@Thms-Brnrd don't worry about it dude! everything's good and thanks for helping out :)


Any sample code for Graph Convolutional Network?

Is there any sample code for Graph Convolutional Network? Particularly, for classification.

You can see my code https://github.com/sufeidechabei/graphnets_gcn

Really appreciated.

Can you give me a star?


Nagios [Epic Task 4]: Align Hosts Found In Nagios But Not Ansible

Align Nagios & Ansible Inventory Files, Fix Hosts In Nagios But Not Ansible
Part Of #2445
2 Hosts Currently Have Discordance From Nagios/Ansible, this needs to be corrected in the Nagios/Ansible configurations to be consistent with the hostname, this should be checked across Jenkins/Nagios/Ansible
Nagios : test-marist-ubuntu1604-s390x-3 - <IP_ADDRESS>
Ansible : test-marist-ubuntu1804-s390x-1 - <IP_ADDRESS>
Hostname : test-marist-ubuntu1804-s390x-1
Nagios : test-packet-ubuntu1604-x64-1 - <IP_ADDRESS>
Ansible: test-equinix-ubuntu1604-x64-1 - <IP_ADDRESS>
Hostname : test-equinix-ubuntu1604-x64-1

Nagios configuration altered for test-marist-ubuntu1804-s390x-1 to align with hostname, ansible & jenkins.
Nagios & Jenkins configuration altered for test-equinix-ubuntu1604-x64-1 to align with hostname & ansible


Flutter daemon devtools.serve returns null for host and port

@gspencergoog commented on Jun 30, 2021, 7:54 PM UTC:
Steps to Reproduce
When I run a Flutter app in IntelliJ, with the "Open Flutter Inspector view on app launch" setting set to true (or if I just open it by clicking on the sidebar tab), it will put up a progress bar that says "Starting DevTools" for about 15-20 seconds, and then it will show the following:

I've tried following those instructions multiple times, restarting IntelliJ and clearing caches, restarting my app, and uninstalling and reinstalling the Flutter and Dart plugins (restarting in between), and it still won't start the inspector.
When I run with verbose output, it does print the following once the app starts:
[{"event":"app.started","params":{"appId":"cf719a58-20d4-4b61-b0bb-ed39d4cfd839"}}]
Activating Dart DevTools...
[{"event":"app.progress","params":{"appId":"cf719a58-20d4-4b61-b0bb-ed39d4cfd839","id":"2","progressId":null,"message":"Activating Dart DevTools...","finished":false}}]
[{"event":"app.progress","params":{"appId":"cf719a58-20d4-4b61-b0bb-ed39d4cfd839","id":"2","progressId":null,"finished":true}}]
[{"id":0,"result":{"value":"linux","type":"_extensionType","method":"ext.flutter.platformOverride"}}]
[{"id":1,"result":{"timeDilation":"1.0","type":"_extensionType","method":"ext.flutter.timeDilation"}}]
[{"id":2,"result":{"enabled":"false","type":"_extensionType","method":"ext.flutter.debugPaint"}}]
[{"id":3,"result":{"enabled":"false","type":"_extensionType","method":"ext.flutter.debugPaintBaselinesEnabled"}}]
[{"id":4,"result":{"enabled":"false","type":"_extensionType","method":"ext.flutter.repaintRainbow"}}]
[{"id":5,"result":{"enabled":"false","type":"_extensionType","method":"ext.flutter.showPerformanceOverlay"}}]
[{"id":6,"result":{"enabled":"true","type":"_extensionType","method":"ext.flutter.debugAllowBanner"}}]
[{"id":7,"result":{"enabled":"false","type":"_extensionType","method":"ext.flutter.inspector.structuredErrors"}}]
[{"id":8,"result":{"enabled":"false","type":"_extensionType","method":"ext.flutter.inspector.show"}}]
[{"id":9,"result":{"enabled":"false","type":"_extensionType","method":"ext.flutter.inspector.trackRebuildDirtyWidgets"}}]
[{"id":10,"result":{"enabled":"false","type":"_extensionType","method":"ext.flutter.inspector.trackRepaintWidgets"}}]
[{"id":11,"result":{"enabled":"true","type":"_extensionType","method":"ext.flutter.inspector.structuredErrors"}}]
[{"id":12,"result":{"enabled":"false","type":"_extensionType","method":"ext.flutter.inspector.trackRebuildDirtyWidgets"}}]
[{"id":13,"result":{"enabled":"false","type":"_extensionType","method":"ext.flutter.inspector.trackRepaintWidgets"}}]

Version info
helin24 from flutter/flutter-intellij#5611.

@gspencergoog sorry I didn't see this before. Do you still have this issue? And if so, would you mind summarizing what it is? :)

I haven't used IntelliJ since about then, so no, I don't have this issue anymore, and I don't really remember the details.

All right, I will optimistically close this issue. To anyone who might hit this, feel free to re-open.


Does app need to request permissions from a directory admin again after token is expired?

In the Protocol diagram if the directory admin has granted permissions and a token is acquired with the client credentials, such as client_secret.  After the token expires, will it be necessary to request for directory admin to grant permissions again or a new token can be acquired by using the same client credentials?
If not, will the permissions granted by the directory admin expire?

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

ID: d32f55ac-acbc-292b-d1b7-ffdd257b11e1
Version Independent ID: ae2a266c-051b-7307-93fd-74bc521c93bf
Content: OAuth 2.0 client credentials flow on the Microsoft identity platform
Content Source: articles/active-directory/develop/v2-oauth2-client-creds-grant-flow.md
Service: active-directory
Sub-service: develop
GitHub Login: @hpsin
Microsoft Alias: hirsin

#please-close


When will the new version be released？

Hi! In other issure, I see you reply that there is a new version coming soon, I am very interested in it, I want to ask when the new version will be released, or can I get a trial access? Thank you so much!

same question here!


Not reciving modifiers

I won a growth modifier but did not receive it. Same happened with a cosmetic modifier but I have no proof of that.

Hi @Jayparalysis! How did you run the Growth Modifier exactly please? e.g: Which crate is this from please?

Hi @Jayparalysis! How did you run the Growth Modifier exactly please? e.g: Which crate is this from please?
My bad! I got it from the vote crate


simplify FAQ

Proposed changes
Some of the info in the FAQ no longer makes sense when presented as documentation for Docker products.
Also assorted other clarifications

LGTM, with exception of one small comment on changing OS X to macOS.

@friism I'm going to merge this, I'm just going to fix some broken links I found first. Not anything that you did here, it looks like they were there already. I'll do this soon.

awesome, thanks @londoncalling

It would be great to have an operating system/container OS compatibility matrix for the question "does Docker run on...?" that shows that macOS and Linux can run Linux containers and Windows can run Windows and Linux containers. WDYT?

@dsheets I agree that would be nice, but I'm not sure it necessarily belongs in the FAQ (it seems more like an introductory-type item). If we add it here, it would just be because the FAQ used to have a long-winded explanation.

@dsheets @friism Here are some options for pages it might make sense to add such a matrix. What do you think:
Top-level Engine install page  - this might make the most sense, and then we can have a brief topic on the other pages linking back to this matrix
Docker for Mac install page
Docker for Windows install page - this already has this topic which could be revised to include or link to the matrix if it is elsewhere
Get started with Docker for Mac
Get started with Docker for Windows  - this already has this topic on switching between Windows and Linux containers, where we could link back to that matrix if that topic is elsewhere


Modal not working sometimes on click event

I am new here and wanted to bring a problem I'm having in developing an application.
I put the code here to explain what happens :
ASPX
Producto

		</asp:Panel>
	</ContentTemplate>
	<Triggers>
		<asp:AsyncPostBackTrigger ControlID="buMPDesripcion" EventName="Click" />
		<asp:AsyncPostBackTrigger ControlID="buMPCodigoAlterno" EventName="Click" />
	</Triggers>
</asp:UpdatePanel>

I build the modal code from the server and so far I have no problems where the problems start is the time to show .
It is that sometimes when I click the asp: LinkButton in the first sometimes I open the black background and I do not get modal and already for the second time if I get good data showing no problems.
But when I'm in the text box and enter me I should show modal and not what you do, just do it when I leave the crowded and sometimes enter when they closed me loose .
Yes please can you help me with this I would appreciate it .
Excuse my English .
Thanks in advance.

excuse the code is not show , how should I put it to show it

Personal support requests should be directed to StackOverflow or the Slack / IRC chat platforms.
For bugs with the framework, please post a JSBin or JSFiddle including the code to demonstrate the issue. In the future, code on GitHub should be wrapped with backticks (`) for inline code or 3 backticks for code blocks.

See https://github.com/twbs/bootstrap#community for more info on the community support forums mentioned.
Closing since we can't provide assistance with problems related to backend frameworks, such as ASP.


Issue with keybindings, lazy loading and session managers

Update (fewer steps to replicate)

The replication steps on the original post are quite cumbersome, you can replicate the same problem just by doing :LspRestart on any of the buffers in the vsplit view (without using auto-sessions at all).

Repeat the 3rd step on the original issue below: Add your custom keybinding to gr.
Open 2 different buffers in a vsplit view and check that your lsp servers are attached.
Execute :LspRestart on 1 of your buffers.
Now do :nmap gr on each buffer and check if they refer to your custom keymap (from step 1) or the default lsp keybindings from lsp-zero.

Original issue (more steps to replicate)

When restoring a saved session (with ANY session manager plugin), my custom keybindings act weird. At least 1 buffer have the correct keybinding while the others use the default keybindings provided by lsp-zero.
You can notice in the screenshot that I'm doing :nmap gr and for the buffer on the right it shows my custom keymap ✅ but for the buffer on the left it shows the default keymap ❌
The lsp servers work as expected, the issue is with the custom keymaps only.
Version
lsp-zero: dev-v3
lazy.nvim: latest
Steps to replicate

Add a session manager plugin to your config, for lazy:

.../plugins/auto-session.lua

Don't worry, the following spec is configured to never do any kind of auto-saving... everything is manual.
return {
	"rmagatti/auto-session",
	opts = {
		log_level = "error",
		auto_session_root_dir = vim.fn.stdpath("data") .. "/sessions/",
		auto_session_suppress_dirs = { os.getenv("HOME") },
		auto_session_enable_last_session = false,
		auto_session_enabled = false,
		auto_session_create_enabled = false,
		auto_save_enabled = false,
		auto_restore_enabled = false,
		auto_session_use_git_branch = false,
	},
	event = "VeryLazy"
}

Add these sessionoptions:

vim.o.sessionoptions = "buffers,curdir,folds,globals,tabpages,winpos,winsize"

Add your custom keybinding to gr.

.../plugins/lsp-zero.lua

For the following snippets I used these guides:
LSP: Creating new keybindings
References and guides: Lazy loading with lazy.nvim
Notice the lazy = false in the neovim/nvim-lspconfig spec. Either with true or false the problem persist.
return {
   ...
	{
		"VonHeikemen/lsp-zero.nvim",
		branch = "dev-v3",
		lazy = true,
		config = false,
         ...

	{
		"neovim/nvim-lspconfig",
		lazy = false,
		cmd = { "LspInfo", "LspInstall", "LspStart" },
		event = { "BufReadPre", "BufNewFile" },
                ...
		config = function()
			local lsp = require("lsp-zero")
			lsp.extend_lspconfig()

			lsp.on_attach(function(_, bufnr)
				-- Disable some keybindings
				-- See: https://github.com/VonHeikemen/lsp-zero.nvim/blob/dev-v3/doc/md/lsp.md#disable-keybindings
				lsp.default_keymaps({ buffer = bufnr, exclude = { "<F2>", "<F3>" } })

				-- Add new keybindings
				-- See: https://github.com/VonHeikemen/lsp-zero.nvim/blob/dev-v3/doc/md/lsp.md#creating-new-keybindings
				vim.keymap.set("n", "gr", function()
					-- require("telescope.builtin").lsp_references()
                                        vim.print("my custom keybinding!")
				end, { buffer = true })
                                ....
...

Now that your setup is complete, open 2 different buffers in a vsplit view and save a new session with :SessionSave (please make sure to close any tree plugin before saving the session... nerd-tree, neo-tree, etc)

Now quit and reopen vim :qa and then nvim in the same directory (the session in saved by directory). Your session will not be restored automatically, this is normal.

Manually restore your session with SessionRestore. You should see your same buffers in a vsplit view.

Now do :nmap gr on each buffer and check if they refer to your custom keymap or the default lsp keybindings.

Clean up
Clean the saved session with :Autosession delete
Expected result
I did several tests with different session managers and this problem is present in all of them (that is because all of those plugins use the builtin neovim session functionality). It would be amazing to have a solution in which I'm able to restore a session without losing my custom keymaps for lsp-zero.
References
These are my dotfiles

I ended up migrating to my own lsp setup and both of my problems dissapeared (session managers and :LspRestart). Thank you so much for this guide: You might not need lsp-zero and for this one: lsp-zero under the hood
Here are the commits of my migration in case anyone is interested.
I opted for doing the server setups in the lspconfig plugin instead of doing them in mason-lspconfig, maybe that was the culprit of the problem but I'm not really sure.

I could not reproduce the issue.
And I have no clue what could be the cause. default_keymaps() is not doing anything special.

I could not reproduce the issue.
And I have no clue what could be the cause. default_keymaps() is not doing anything special.

Thank you for your reply! I'm sorry you couldn't replicate the issue. I'm willing to debug this a little bit further. I will create a minimal git repo that you can clone y and test the issue there, would that be ok?

Okay. I manage to reproduce the issue with another config. I think the problem is that Neovim assigns your custom keybinding only to one buffer because you have {buffer = true}.
Try to use the variable bufnr when making your custom keybinding ({buffer = bufnr}). This should work.
vim.keymap.set("n", "gr", function()
  vim.print("my custom keybinding!")
end, { buffer = bufnr })

Okay. I manage to reproduce the issue with another config. I think the problem is that Neovim assigns your custom keybinding only to one buffer because you have {buffer = true}.
Try to use the variable bufnr when making your custom keybinding ({buffer = bufnr}). This should work.
vim.keymap.set("n", "gr", function()
  vim.print("my custom keybinding!")
end, { buffer = bufnr })

I totally missed that!! This solves the issue. Thank you so much and sorry about that :sad:


Add a link to this github area

Perhaps with the "Fork me on Github" ribbon you see on alot of repos.


Clarification on GMM sampling

Hello! Thank you for the amazing work! Really interesting approach using domain randomization for contrast/resolution agnostic brain segmentation !
I had a question about the GMM generative model. In the paper, it is mentioned in section 3.1.2 that an initial synthetic scan G is generated by sampling the GMM conditioned on the (deformed) label map. Now, in my (limited) understanding of GMMs, I know that they follow an expectation-maximization (EM) process to estimate the parameters of the model. I am unable to find any info about the EM algorithm (in the paper and code). Could you please provide some insights on how this GMM was trained? In fact, was it trained? I think I am missing something here.
Thank you!

Hi,
a GMM is not necessarily trained, it all depends if you already know how to assign the classes. The EM algorithm you mention is a possible way to create classes of Gaussians, but here the classes are already defined by the values in the label map, and the parameters (mean, variance) of these Gaussians are randomised
So the image G is formed exactly as exposed in equations 11 to 13:

Assuming our training label map has K classes, we first create K different Gaussians with random means and variance
for each voxel of G, we look at the corresponding label value k, and we sample an intensity from the Gaussian number k.

Note that the parameters of the K Gaussians change for each training example.
Hope this helps,
Benjamin


Problem rendering on Firefox

The page is not scaling correctly on Firefox:
when I have resize down the browser window, it does not render the first part of the page (the one with the planet)
see video below:
paginaegh.webm

Closing as it seem ok now.


Calling methods on invalid types

This is a follow up of the discussions in:

https://github.com/pydata-apis/dataframe-api/issues/6#issuecomment-639754182
#11 (question: pandas has parameters (bool_only, numeric_only) to let only apply the operation over columns of certain types only. Do we want it?)

See this example:
>>> df[['name', 'population']].mean()
population    2.729748e+07
dtype: float64

Even if the name column is selected, it is being ignored, since the mean of a string columns does not make sense. As opposed to raising an exception.
Many reductions implement a parameter to let control this behavior:
df[['name', 'population']].mean(numeric_only=False)
TypeError: could not convert string to float:

If we consider more methods to be applied directly over a dataframe, for example:
>>> df[['first_'name', 'last_name']].str.lower()

We may end up with a huge amount of string_only, bool_only, numeric_only parameters. All meaning something similar, but IMO adding a decent amount of complexity, and being difficult to keep the behavior consistent.
My preference would be to always raise, but being a software engineer I'm biased, and I guess many users may want this "magic".
So, I guess implementing an option, for example: pandas.options.mode.invalid_dtype {raise or skip} could make more sense.
The main problem with this approach is probably that it's not as easy to define the behavior for each operation:
(df.mean(numeric_only=True)
   .mean(numeric_only=False))

Personally, I don't see this as an issue. IMO, the behavior depends more on the user than on the operation. I'd say for production code, having to be explicit, and selecting the columns to operate with, makes more sense. While in a notebook, avoiding exceptions with this sort of "magic" seems to be more useful.
I guess for Series/1-column DataFrame (see #6) it always makes sense to raise an exception.
Thoughts?

FWIW in pandas we have deprecated all of the foo_only=None cases, with the future defaults all being False for these.


Send GQL API

Purpose

$subject

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

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

Project coverage is 28.99%. Comparing base (839dd54) to head (932ea6f).
Report is 26 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2231      +/-   ##
==========================================
+ Coverage   28.42%   28.99%   +0.56%
==========================================
  Files         124      118       -6
  Lines       14546    14161     -385
  Branches     2681     2681
==========================================
- Hits         4135     4106      -29
+ Misses      10274     9918     -356
  Partials      137      137

Flag
Coverage Δ

adapter
27.41% <ø> (-1.06%)
:arrow_down:

common-controller
?

config-deployer-service
25.99% <ø> (ø)

idp-domain-service
80.21% <ø> (ø)

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

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


GoControl WS15Z-1 Not Detected

I installed a GoControl WS15Z-1 Z-Wave on/off switch (not a dimmer) and it's not being detected by homebridge-isy-js. I tried making a scene for it but it won't respond to any on/off commands.Is there any way that support can be added?

Unfortunately Z-Wave devices need to be added one by one to the underlying isy-js library. They don't have a generic set of type definitions exposed through ISY which allows me to identify the device type.
You can add support yourself by adding it to the isy-js device list. Go into the homebridge-isy-js/node_modules/isy-js directory and edit the file isydevicetypes.json file. You need to add a new line which tells isy-js how to recognize the device as an on/off switch.
You should be able to take this line:
{ "type": "<IP_ADDRESS>", "address": "", "name": "ZW002_1 - On/Off Power Switch", "deviceType": "Light", "connectionType": "ZWave", "batteryOperated": false },
And replace the type entry and the name entry. The name entry should be the friendly name you want for the type. GoControl WS15Z-1 for example.
The type entry you can get from your isy. Go to chrome and visit http:///rest/nodes and find the node entry in the XML which belongs to the above device. Find the type field and use that value in your new entry.
So, if your entry looked like this:

ZW005_1
Deck Door Lock
4
42035
<IP_ADDRESS>
true
0
0
0
ZW005_1
1

4.64.3
297.2.0
111
5

Then you would set the new line in isydevicetypes.json to be:
{ "type": "<IP_ADDRESS>", "address": "", "name": "GoControl WS15Z-1", "deviceType": "Light", "connectionType": "ZWave", "batteryOperated": false },
(Yours of course will be different)
If your entry is correct then restarting homebridge should enable the device to show up.
Assuming that works and you can send me the line you added to isydevicetypes.json and I'll make sure it makes it into a future version of isy-js so you won't need to add it manually in the future. (And neither will others).

Great! I'll give it a try. If I can get it to work I will I'll provide you with the info. Thanks.

It works! Here's the line I added:
{ "type": "<IP_ADDRESS>", "address": "", "name": "GoControl WS15Z-1 On/Off Power Scene Switch", "deviceType": "Light", "connectionType": "ZWave", "batteryOperated": false }

Excellent news. I'll put this in my worklist for the next version of isy-js. Thanks for sending the information!


[MODFISTO-430] - Updated rollover error message

Purpose
Describe the purpose of the pull request.
Approach
How does this change fulfill the purpose?
TODOS and Open Questions

[ ] Use GitHub checklists. When solved, check the box and explain the answer.

Learning
Describe the research stage. Add links to blog posts, patterns, libraries or addons used to solve this problem.


Rendering of annotations in HistomicsTK front-end

This issue is about adding the functionality necessary for rendering annotations on top of images in the HistomicsTK front-end.
@manthey and @jbeezley. Can one of you please take this up?
You can use some dummy annotations of each type to build the functionality necessary to render them.

If @manthey hasn't started on this yet, I can take it on.

Fixed by PR #103.


Using this plugin on a page with mulitple maps generates Document update conflicts

When using this plugin on a page with more than one map (both maps have same bounds and tiles), on initial sync, it generates a lot of document update conflicts. This is because it is trying to put tiles from two sources at the same time (I think).
To get around it I use the pouchdb.upsert plugin and change the put code to following:
        this._db.putIfNotExists(tileUrl, doc).then(function(res) {
            // success, res is {rev: '1-xxx', updated: true}
        }).catch(function(err) {
            // error
        });

I am not sure if this sort of thing should be in the core library but wanted to raise this.

probably the put code modified link is not valid anymore.
Any chance to upload here at least the diff to test locally?


Assistants API

Hey, thank you for this project! It's truly amazing!
I was wondering if there is any plan to integrate the new OpenAI releases into the library. I'm especially interested in the AssistantsAPI at:
https://platform.openai.com/docs/api-reference/assistants
Let me know if anyone want to collaborate to make this happen. Thank you!

Hello there thank you for your concern feel free to propose a PR I would be glad.
This library must be improved in order to follow the latest updates, so any help is welcome!

I've made some modifications for the latest API. I hope they can help you@adamakio
openai.txt


changed logo size #912

fixed logo size #912

@policorn I have answered @Karljoones comment :-)

Hi @shivanshsoni, code looks good, but before merging, can we have a screenshot of the updated icon? :)

Merged, thanks @shivanshsoni !


feat(user-header): flex component distribution

:heavy_check_mark: Was validated

Changed the component and fixed the view when on a small screen device


Shared server code

So I've noticed there are 3 different servers, Koa/Production, Test, and Webpack-Dev. My question is, is there any shared server code besides what's in the config file? If I wanted to put in my own API that interfaces with a database, websockets, or other features, is there already a file I could put that that would then be accessible to all 3 different servers? Or would I have to create an entirely new module and import to all three separately?
Also, and this isn't quite as important, when I run the dev server, Wappalyzer says that the app is built with Express, is that because Webpack uses express to serve the files? I'm just curious.

To answer your last question: yes, Webpack's dev server uses Express to serve assets. From http://webpack.github.io/docs/webpack-dev-server.html:

The webpack-dev-server is a little node.js Express server, which uses the webpack-dev-middleware to serve a webpack bundle.

For the first question, this is where things get trickier. Given the current setup, you would most likely integrate it all in the Koa server and spin that up alongside the Webpack dev server, then have your client application point its API requests to the Koa server. There's no current connection between the webpack-dev-server and Koa; if you're looking to get rid of webpack-dev-server and have everything served from a single app server, there's some discussion around that here: https://github.com/davezuko/react-redux-starter-kit/issues/58 (specifically the last comment).

Fascinating. Alright I'll look issue #58 in greater depth and get back to you if I have anymore questions. You've been incredibly helpful. Thank you!

Closing this since the starter kit no longer comes with a server.


Issues - Search Entry Not Being Changed to Explore

Using version 1.04 (not installed on pebble.social) there seems an issue with the Search menu not being remapped to Explore and the # icon missing and no animated icon when clicked.

That's weird, can you show me what classes the "Search" tab on the sidebar has? Neither of the instances I am on have it so it's hard to tell what's wrong by myself

Not exactly sure what you need but applied 1.0.5 and took the following screenshot

Fixed with 457d0bc

Does not appear fixed in 1.0.6

Pull from latest, it's not fixed in 1.0.6 as I fixed it after publishing 1.0.6


error in analysis

hi,
while uploading count matrix and meta file i am getting this error.
can you help me?

Hi @harshagr21 ,
Please take a look at https://github.com/ventolab/CellphoneDB/issues/18 - could it be that your adata.obs.index has duplicated values? Hope this helps.
Best,
Robert.

Hi,
thanks for the reply yes it was the same problem, i fixed it.


Add support for FlatCallTracer

Describe the feature
ref #201 might require some helper for TracingConfig
blocked by #10989
we now have a new variant:
https://github.com/paradigmxyz/reth/blob/c42d5447a443e95ae92437e4d91f7dc06e253e70/crates/rpc/rpc/src/debug.rs#L392-L396
that behaves exactly as the parity tracer, so we can reuse that impl:
https://github.com/paradigmxyz/reth/blob/c42d5447a443e95ae92437e4d91f7dc06e253e70/crates/rpc/rpc/src/trace.rs#L341-L358
Additional context
No response

can take this

unblocked now


Split-direction preselection

Feature:
As a user coming from bspwm I would really like to see split-direction preselection support in Hyperland.
Currently Hyperland decides whether to split horizontally or vertically based on the window W/H ratio and after that chooses between top/bottom or left/right based on the pointer position.
I would like to see a dispatcher that allows manually setting the split direction for the next opened window.
When preserve_split is true any of the direction left, right, up and down can be preselected. Without preserve_split only left/right or only up/down can be preselected based on the W/H ration of the window.
Hyperland could visualize an active preselection by drawing a rectangle over the area where the new window will be placed.
An active preselection could be removed by either calling the dispatcher again with the same direction or by a special clear parameter to the dispatcher.
Scenario example:
The user config contains
preserve_split=true
bind=SUPER_CTRL,left,splitpresel,l
bind=SUPER_CTRL,right,splitpresel,r
bind=SUPER_CTRL,up,splitpresel,u
bind=SUPER_CTRL,down,splitpresel,d

The user opens a window (window1)
Window1 takes up the whole screen space.
The user opens another window (window2) while window1 is active and the pointer is in the left half of window1
Window1 takes up the right half of the screen. Window2 takes up the left half of the screen
The user presses Super + Ctrl + left while window2 is active.
Window1 takes up the right half of the screen. Window2 takes up the left half of the screen. A rectangle is overlayed above the left half of window2.
The user opens a new window (window3).
Window1 takes up the right half of the screen. Window2 takes up the 3rd quarter from the right of the screen.


Hold - Duplicate expense report displayed when employee submit expense

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.21-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4863120&group_by=cases:section_id&group_order=asc&group_id=309128
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause Internal Team
Action Performed:
Precondition:

Login as the owner of the workspace
Create a workspace
Invite the approver and employee
Navigate to more features
Enable "workflows"
On the "Workflow" editor - enable "Add Approvals"
Set the Approver account as the Approver

Steps:

As employee submit 2 expenses to the workspace chat
As Approver Hold one of the expenses and partially approve the report
As employee submit another expense to the workspace chat

Expected Result:
There should be only one new expense present on workspace chat, along with expense that was put on hold
Actual Result:
Duplicate expense report displayed when employee submit another expense while there is a expense with hold status
Workspace avatar turn to offline image
Workaround:
Unknown
Platforms:

[x] Android: Native
[x] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/4c96e2a5-3a89-4393-8858-32394214b60e

View all open jobs on GitHub

We think that this bug might be related to #wave-collect - Release 1

@sakluger  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

@Gonals can you check and confirm that this is a BE bug?

CC: @robertjchen for eyes.

We had a fix go out earlier today that stops the inaccurate Onyx response- can we please re-test to see if this is still reproducible?

Bug is still reproducible.
https://github.com/user-attachments/assets/40d573b6-e767-452c-af66-90b8432a9df4

@robertjchen what should our next step be on this one?

I'll take a closer look tomorrow!

After further investigation, I believe this to be an external issue.
The new expense is part of the same report the HELD expense is, so the backend returns this correctly. The issue is that the frontend initially adds is separately to the chat, instead of adding it to the right report.
cc @aimane-chnaif

So I spent some time investigating this issue, here's some info that I found:
After the approver partially approved the expense, ApproveMoneyRequest returned Onyx data to update iouReportID, which is the reportID of the current expense that the approver just approved.

Approving the full case is correct. However, with partial approval, that's wrong iouReportID. Why? Because that expense is marked as approved. Meanwhile, if that expense is approved partially, we should return the reportID of the remaining expense, aka the expense that's being held.
Next time, when the employee creates a new expense, we will get the report data from this iouReport here:
https://github.com/Expensify/App/blob/4af620b439636f714ff72a26320ed57f9d2faac3/src/libs/actions/IOU.ts#L2042
And shouldCreateNewMoneyRequestReport here will return true, because that iouReport was already approved:
https://github.com/Expensify/App/blob/4af620b439636f714ff72a26320ed57f9d2faac3/src/libs/actions/IOU.ts#L2045
So I think we can fix this one from our BE. If the Approver approves an expense partially, the iouReportID should be the expense that's being held.
https://github.com/user-attachments/assets/be6fa4e6-2713-4fb3-a1ca-d4af817e3a5e

@hungvu193 thanks for the thorough investigation, I really appreciate it!
@Gonals - I know you said that you thought this was an external issue, but what do you think of @hungvu193's investigation and theory that this is a BE issue?

Appreciate the in-depth analysis, this definitely looks like a BE issue- it appears we're returning an Onyx update for the wrong report after the ApproveMoneyRequest involving a partial approval

I still can reproduce it, make sure that the employee needs to  submit the expense (not the owner) and the approver need to hold one of these expense before partial approve. IF you can't reproduce, try to repeat the steps 2,3 times.
https://github.com/user-attachments/assets/ab3f59d7-44f9-44c9-b229-8e885aabb548

Hmmm. I still can't seem to reproduce it.
I gotta focus on some travel-related stuff for a day or two, but I'll come back to this and try again.

I'll get back to this as soon as I can!

@Gonals any updates? Have you been able to reproduce the issue?

I haven't been able to get to it yet. Travel is taking priority (and my life). I'll get back to it as soon as I have any bandwidth

Blocking some time for this tomorrow morning

Managed to reproduce this in Staging, but not in Dev, which makes testing a bit hard. I DO have a theory on how to fix it, so I'll give that route a shot and see what happens

Encouraging news!

Looks like the PR is merged and deployed! Hopefully it fixes the bug.

@sakluger This issue is not repro anymore, build - v9.0.41-2
https://github.com/user-attachments/assets/43b7ad2f-58b1-4178-96ea-7b91372f2964

Thank you! 🙇

Apparently still reproducible: https://expensify.slack.com/archives/C036QM0SLJK/p1727771195051329?thread_ts=1727720815.151679&cid=C036QM0SLJK

@Gonals any ideas what might still be causing this?

Dammit -.-
Well, I used my ideas in the previous fix. I'll try to reproduce in dev again (which I have not been able to so far). With that, I should be able to figure out what's up.

Ok. This is getting overwritten by a "newer" Onyx Update, which wrongly sets the iouReportID to the oldOne. I'm digging out the origin.

Still chasing this. It is NOT coming from this flow, but from a concurrent event, so it has become a way harder thing to chase.
I think I have it surrounded, though

Finally got a solution!
Buuuut... it has unearthed another (existing) bug. I'll create an issue for it.

New issue: https://github.com/Expensify/Auth/pull/12721
Fix for it already in review

Issue not reproducible during KI retests. (First week)

@Gonals is this fixed now? If so, did any of your PRs have C+ reviews that require payment?

Yep! Should be fixed! No C+ reviews


String property escaping incorrectly?

Version
2.4.2
Reproduction link
http://jsfiddle.net/yMv7y/3274/
Steps to reproduce
Pass some html-entity escaped string with an ampersand as a prop value to a control, try to display it.
E.g. "Example & stuff"
What is expected?
"Example & stuff"
What is actually happening?
"Example & stuff"

(Fiddle links to http://vuejs.org/js/vue.min.js instead of a specific version)

Because you are writing amp; two times
"Banking &**amp;** Financial Services"

I see it but I don't believe it; I showed a colleague this before I submitted as a bug, didn't see & ... checking to make sure some copy-pasta didn't happen, but I still have this effect happening in my project, so maybe there's double-escaping there...


🔼 Differences in calculated GW when pollen contamination is applied

Describe the Bug
There is a diffferent in GW and Ne possibly due to pollen contamination (PC). PC should increase Ne and drop GW. See old SPAR packages for rules. This relies on #1402

Expected Behaviour
When pollen contamination is present, we should see an increase in Ne and a drop in GW.
Acceptance criteria

Pollen contamination drops GW and increases Ne
Test cases and unit tests completed
Tested and merged

GW calculation without pollen (current implementation):
Part 1:
for (List<String> traitCode : traitList) {
  BigDecimal percent = calculateThreshold(allPtList, traitCode);
  if (percent >= 70) {
    BigDecimal genWorthValue = calculateValue(allPtList, traitCode);
  }
}

Part 2: (calculateValue function)
BigDecimal malePollenSum = reducePollenCount(traitsDto);
BigDecimal femaleConeSum = reduceConeCount(traitsDto);
BigDecimal sumGw = BigDecimal.ZERO;

for (OrchardParentTreeValsDto dto : traitsDto) {
  BigDecimal traitBreedingValue = getTraitValue(dto, trait);
  if (traitBreedingValue != null && traitBreedingValue != 0) {
    BigDecimal pi = calculatePi(dto.pollenCount(), dto.coneCount(), malePollenSum, femaleConeSum);
    BigDecimal gwValue = traitBreedingValue.multiply(pi);
    sumGw = sumGw.add(gwValue);
  }
}
return sumGw;

Rolled over from Sprint 35.

@SLDonnelly
Reviewing changes on this PR, one of the DB columns I flagged to review is Seedlot.nonOrchardPollenContaminationPercentage. This column doesn't exist originally in the legacy. So I want to make sure what to store here.
This column exists in the legacy for the SEEDLOT_PARENT_TREE.NON_ORCHARD_POLLEN_CONTAM_PCT table though.
Another confusing thought is that the legacy call is orchard_pollen_contamination_perc but we have it here as "non_orchard..."

@RMCampos I think that it's picking up another column on the seedlot table in oracle:
I'm not quite sure that the difference between the two is, I'll keep digging

@SLDonnelly I guess I got it.
On the Orchard table we have the user's input contaminant percentage. And after the calculation, the value will likely change. And then will be stored in the Seedlot table.
Thank you!


Title elements contain a concise, descriptive title

Title elements contain a concise, descriptive title
User Story
AS A marketing agency
I WANT a codebase that follows accessibility standards, where the codebase has Title elements contain a concise, descriptive title
SO THAT our own site is optimized for search engines
Acceptance Criteria
Image and icon elements contain accessible alt attributes
GIVEN the horiseon-web-application codebase
WHEN the codebase is reviewed
THEN Title elements contain a concise, descriptive title with in the codebase.

no changes further changes needed rebased to current main and issue has been addressed. closing


try to make it FUD

I have run an encryption using XOR cipher, and the scan detection is fixed, so when I scan it, it works, and it's fully undetectable (FUD). However, when I run the shellcode, it encounters a problem during execution. It says that powershell stager.exe is not FUD, and suspicious activity is detected. My question is: do you have any suggestions on how I can make it FUD, or do you have a solution?
Here is a picture

loool just crypt it 💀

Hm... r77 is FUD when using Windows Defender. For at least the past 6 months Windows Defender has done nothing to detect r77.
If you're using a different AV, then there might be an issue that allows detection. However I can't fix individual AV issues all the time. I only do implementation improvements to support AV evasion and Windows Defender in particular.


ravenclaw-10 added

Description 📜
It calculates the decimal number , binary number, hexadecimal number and octal number from each other.

Checklist ✅

[ x ] I follow Contributing Guidelines & Code of conduct of this project.
[ x ] I have performed a self-review of my own code or work.
[ x ] I have commented my code, particularly in hard-to-understand areas.
[ x ] My changes generates no new warnings.
[ x ] I'm GSSOC'22 contributor
[ ] Have you added a working video of your calculator ?

Screenshots / Gif (Optional) 📸

Okay sure.

Done.

Done

@mansi-garg-12 @Chayan-11 @AnuragParashar2000

Updated Readme.md. Please review it

Please ignore the GCD calculator commit

closing this pr!!
if you have any queries, reach out to me on discord :)


ci: Run shellcheck in CI linter job

What this PR does / why we need it:
Install and run shellcheck in the CI linter job. Lint all files matching '*.sh' in the tree.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

None

/lgtm


No longer maintained Termbox is causing a panic error

I'm getting this error with gopass create:
panic: termbox: error while reading terminfo data: termbox: unsupported terminal

goroutine 1 [running]:
github.com/gopasspw/gopass/pkg/cui.GetSelection(0xda1c60, 0xc0002a6480, 0xc938ce, 0x39, 0xc95c47, 0x41, 0xc0002ab0e0, 0x5, 0x5, 0x0, ...)
        /var/src/gopass/pkg/cui/cui.go:184 +0x58f
github.com/gopasspw/gopass/pkg/action/create.Create(0xda1c60, 0xc0002a6480, 0xc0002fa160, 0xd9ebe0, 0xc0002a6420, 0xad6e36, 0xc0002bb260)
        /var/src/gopass/pkg/action/create/create.go:57 +0x374
main.getCommands.func28(0xc0002fa160, 0x0, 0xc0002bb260)
        /var/src/gopass/commands.go:293 +0x86
github.com/urfave/cli.HandleAction(0xb4f840, 0xc000296340, 0xc0002fa160, 0xc0002fa160, 0x0)
        /var/go/pkg/mod/github.com/urfave/cli@v1.22.4/app.go:526 +0x11a
github.com/urfave/cli.Command.Run(0xc406ec, 0x6, 0x0, 0x0, 0xc000294440, 0x1, 0x1, 0xc7e2df, 0x1c, 0x0, ...)
        /var/go/pkg/mod/github.com/urfave/cli@v1.22.4/command.go:174 +0x57a
github.com/urfave/cli.(*App).Run(0xc0002d4000, 0xc0000ba000, 0x2, 0x2, 0x0, 0x0)
        /var/go/pkg/mod/github.com/urfave/cli@v1.22.4/app.go:279 +0x7c7
main.main()
        /var/src/gopass/main.go:67 +0x314

Which is what's reported in https://github.com/gopasspw/gopass/issues/1072 reported.
This issue is due to a termbox issue (https://github.com/nsf/termbox-go/issues/215) which I suggested to fix in https://github.com/nsf/termbox-go/pull/216 but @nsf said that termbox is no longer maintained and that tcell is a recommended replacement. Gopass uses termbox inderectly - via gocui which also acts as unmaintained according to it's PRs. Because of all that, I patched termbox in my own fork and patched gocui and gopass to use my own fork at https://github.com/doronbehar/gopass/tree/doronbehar-termbox and I experience this error no more.
Would you be willing to consider switching to my fork of termbox until a replacement for it will be written? I can send a PR straight from https://github.com/doronbehar/gopass/tree/doronbehar-termbox .
Regards.

That sounds good. Feel free to send a PR.


[spi-dev, doc] Inconsistency in new SPI-DEV ingress/egress buffer descriptions

New implementation od SPI-DEV have modified SRAM layout for FLASH/TPM buffer.
There is misliding inconsistency between *.hjson and documentations.
Image describe 4 sections in ingres buffer but hjson speak about 5 sections.
There is one 256B field in comment of ingres buffer that isn't show on image.
Image:  https://opentitan.org/book/hw/ip/spi_device/doc/spid_sram_layout.svg

spi_device.hjson Define ingres buffer as composition of sections:
 { skipto: "0x1000" }
    { window: {
        name: "egress_buffer",
        items: "SramEgressDepth",
        validbits: "32",
        byte-write: "false",
        unusual: "true"
        swaccess: "wo",
        desc: '''
          SPI internal egress buffer.

          The lower 2 kB is for Read content emulating eFlash.
          The next 1 kB is for the Mailbox buffer.
          Then the next 256 B is for the SFDP buffer.
          Finally, the buffer spaces end with a 64 B TPM Read FIFO.
          '''
      },
    },
    { window: {
        name: "ingress_buffer",
        items: "SramIngressDepth",
        validbits: "32",
        byte-write: "false",
        unusual: "true"
        swaccess: "ro",
        desc: '''
          SPI internal ingress buffer.

          The layout is as follows (starting from offset 0):
              - 256 B SFDP buffer
              - 32 B CmdFIFO
              - 32 B AddrFIFO
              - 256 B payload FIFO
              - 64 B TPM Write FIFO
          '''
      },

The hjson isn't right, and the image is. Needs a fix eventually.

Unfortunately reggen don't generate any layout or const for this buffers.

Unfortunately reggen don't generate any layout or const for this buffers.

That's just simply false. These are all generated via reggen:
// Sram eFlash read buffer offset (from egress buffer start). Word size is
// 32bit width.
#define SPI_DEVICE_PARAM_SRAM_READ_BUFFER_OFFSET 0

// Sram eFlash read buffer entries. Word size is 32bit width.
#define SPI_DEVICE_PARAM_SRAM_READ_BUFFER_DEPTH 512

// Sram mailbox buffer offset (from egress buffer start). Word size is 32bit
// width.
#define SPI_DEVICE_PARAM_SRAM_MAILBOX_OFFSET 512

// Sram mailbox entries. Word size is 32bit width.
#define SPI_DEVICE_PARAM_SRAM_MAILBOX_DEPTH 256

// Sram SFDP buffer offset (from egress buffer start). Word size is 32bit
// width.
#define SPI_DEVICE_PARAM_SRAM_SFDP_OFFSET 768

// Sram SFDP entries. Word size is 32bit width.
#define SPI_DEVICE_PARAM_SRAM_SFDP_DEPTH 64

// Sram TPM RdFIFO offset (from egress buffer start). Word size is 32bit
// width.
#define SPI_DEVICE_PARAM_SRAM_TPM_RD_FIFO_OFFSET 832

// Sram TPM RdFIFO entries. Word size is 32bit width.
#define SPI_DEVICE_PARAM_SRAM_TPM_RD_FIFO_DEPTH 16

// Sram payload FIFO offset (from ingress buffer start). Word size is 32bit
// width.
#define SPI_DEVICE_PARAM_SRAM_PAYLOAD_OFFSET 0

// Sram payload FIFO entries. Word size is 32bit width.
#define SPI_DEVICE_PARAM_SRAM_PAYLOAD_DEPTH 64

// Sram command FIFO offset (from ingress buffer start). Word size is 32bit
// width.
#define SPI_DEVICE_PARAM_SRAM_CMD_FIFO_OFFSET 64

// Sram command FIFO entries. Word size is 32bit width.
#define SPI_DEVICE_PARAM_SRAM_CMD_FIFO_DEPTH 16

// Sram address FIFO offset (from ingress buffer start). Word size is 32bit
// width.
#define SPI_DEVICE_PARAM_SRAM_ADDR_FIFO_OFFSET 80

// Sram address FIFO entries. Word size is 32bit width.
#define SPI_DEVICE_PARAM_SRAM_ADDR_FIFO_DEPTH 16

// Sram TPM Write FIFO offset (from ingress buffer start). Word size is 32bit
// width.
#define SPI_DEVICE_PARAM_SRAM_TPM_WR_FIFO_OFFSET 96

// Sram TPM Write FIFO entries. Word size is 32bit width.
#define SPI_DEVICE_PARAM_SRAM_TPM_WR_FIFO_DEPTH 16

That is interesting... i just regenerated registers map for TockOS and i don't see it.

That is interesting... i just regenerated registers map for TockOS and i don't see it.

I guess the Rust generator doesn't export the parameters.


Fix GraalVm error with Log manager Class

Fixed by adding the switch -H:IncludeResources=loggin.properties.


Mistral provider is hard-coded to https://api.mistral.ai/v1/chat/completions

There is no way for the Mistral provider to override the API URL. So you can't test against, for example, Mistral hosted on Azure. It would be ideal if this provider supported the apiHost, apiBaseUrl, and apiKeyEnvar configuration parameters like the OpenAI provider does.
If there's interest, I can work on a PR for this.

Would definitely merge a PR for this.  Feel free to take a stab at it!


Koios: Add support for new ScriptService

Add support for following apis in ScriptService for Koios backend

datum by datum hash (Both Json & Cbor)
PlutusScript or NativeScript by script hash
Script cbor for Plutus script and Json for Native Script

@edridudi fyi