Add new sourceRootDir setting

TypeScript Version:  3.5.1

Search Terms: sourceRoot, sources, sourcemap
Code

Create a module like: src/nested/foo.ts

Use the following tsconfig.json with tsc:

{
  "include": ["src"],
  "compilerOptions": {
    "baseUrl": ".",
    "outDir": "dist",
    "sourceMap": true,
    "sourceRoot": "src"
  }
}

Expected behavior:
Inside nested/foo.js.map, the sourceRoot should be ../src/ instead of src/. Otherwise, the paths in the sources array cannot be found.
Actual behavior:
The sourceRoot is src/ regardless of nesting.
Playground Link:  N/A
Related Issues:  None

Currently we treat sourceRoot as a verbatim path, so that you can control what the source map's final sourceRoot should be if you are deploying these files to another location, i.e.:
{
  "compilerOptions": {
    "sourceMap": true,
    "sourceRoot": "https://foo.com/debug/sourcemaps/"
  }
}

However, we could change this to identify absolute paths and URLs and keep those verbatim. We don't currently use the sourceRoot for any path calculations when generating source maps, but this change could be a breaking change for some users.
@aleclarson: You can work around this by specifying the following in your tsconfig.json:
{
  "include": ["src"],
  "compilerOptions": {
    "baseUrl": ".",
    "outDir": "dist",
    "sourceMap": true,
    "sourceRoot": "../src"
  }
}

@RyanCavanaugh, @DanielRosenwasser: I am concerned this could be a breaking change for some users. Do we want to pursue changing the behavior of sourceRoot, or perhaps introduce a sourceRootDir option that can be used instead of sourceRoot and performs the correct relative path calculation (with the Dir suffix indicating that we're talking about an actual file system directory)?

This is an issue for anyone using a monorepo with Typescript. Unless the definition of a function/class/type is directly in the index.ts file of the referenced module, then "go to definition" functionality lands inside a .d.ts file instead.
As far as a breaking change, could this be resolved by detecting the value is a URL and then NOT putting relative paths into the map files?

This also breaks debugging in, at least, VSCode as it is unable to resolve the source -> JS mappings when the map files have a "sourceRoot".
We have had to remove "sourceRoot" from the project being debugged (we have yet to evaluate any other side effects) but we cannot debug other modules from within the same mono-repo which are being used by that project.
Can we get confirmation of a work-around or that this will actually be in 4.1.0? This issue has been pushed off for over a year and it's crippling to our development team.

I discussed this with the team. To support this without introducing a breaking change we will have to add a sourceRootDir option to specify a path that should be:

Resolved based on the location of tsconfig.json (similar to rootDir/outDir)
Can only be used with file-system based paths (no URL paths)
Sources added to the source map should be made relative to this path
Cannot be used in combination with the sourceRoot option (they are mutually exclusive)

It appears the "fix" is not to use "sourceRoot" when sources are on the local filesystem and only use it when source will be hosted. VSCode now correctly finds the original source both when editing and when debugging.
The documentation lead us to believe that we had to have "sourceRoot" set in order for mappings to be correct but it's actually the opposite.

Is the solution expected to be implemented in TypeScript 4.2.0?

Is the solution expected to be implemented in TypeScript 4.2.0?

The solution, for us, was to remove "sourceDir" from tsconfig.json. All the map files are now generating with the correct paths.
What I would like to see is a clarification of the documentation as to when/why you would use sourceRoot (ie. only when hosting the source at an absolute location like a URL).

The solution, for us, was to remove "sourceDir" from tsconfig.json. All the map files are now generating with the correct paths.
What I would like to see is a clarification of the documentation as to when/why you would use sourceRoot (ie. only when hosting the source at an absolute location like a URL).

The solution, for us, was to remove "sourceRoot" from tsconfig.json. All the map files are now generating with the correct paths.
What I would like to see is a clarification of the documentation as to when/why you would use sourceRoot (ie. only when hosting the source at an absolute location like a URL).

As I've said before, we treat sourceRoot as a verbatim path and don't do any path resolution for it. It should only really be used when its pointing to an absolute URL or some other absolute path that might be defined as part of your build process. We won't be changing the behavior of sourceRoot as that would be a breaking change. While we are considering adding something like sourceRootDir, it likely won't make it for 4.2 due to holiday schedules.

The solution, for us, was to remove "sourceRoot" from tsconfig.json. All the map files are now generating with the correct paths.
What I would like to see is a clarification of the documentation as to when/why you would use sourceRoot (ie. only when hosting the source at an absolute location like a URL).

As I've said before, we treat sourceRoot as a verbatim path and don't do any path resolution for it. It should only really be used when its pointing to an absolute URL or some other absolute path that might be defined as part of your build process. We won't be changing the behavior of sourceRoot as that would be a breaking change. While we are considering adding something like sourceRootDir, it likely won't make it for 4.2 due to holiday schedules.

Thanks for clarifying. I am expecting something like sourceRootDir. Hope it will come to real soon.

Thanks for clarifying. I am expecting something like sourceRootDir. Hope it will come to real soon.

@rbuckton @RyanCavanaugh @andrewbranch any estimate around when this new sourceRootDirfeature will be available? I'm using Bazel to build packages in a monorepo and it is really important for us to have this flag so we can control the source files locations in relation to the sourcemaps location correctly. At the moment our IDE features when click and following implementation for those packages are only working for files in the top level folders as whatever we set in the sourceRoot, even if it is a relative path, will be immutable and not correctly changed for nested folder files.

@RyanCavanaugh this issue keeps being postponed for a long time. Is there anyway to prioritise it 😃 ? It is blocking us in a monorepo with Bazel and I believe it is also affecting others in the same setup too.

I will restate, one last time: we removed "sourceRoot" from our configuration and the files all generated with the correct roots. It is evident that "sourceRoot" should only be used when hosting your map files on a web-site.
We have a monorepo with 60+ packages. Most of those packages have the following structure:
src\ - Typescript source
dist\ - Javascript output, maps, .d.ts files, and assets

This works for debugging, VSCode link following, and everything else we've tried.
Our initial issue was a misunderstanding of when to use "sourceRoot" which appears to only be when you're placing the .map, .d.ts, etc. files in a completely separate location. I can't think of a reason to do that except to host the files on a web-site.
IMO - the only issue here is documentation.

@raijinsetsu thanks for the inputs here. I actually know what you mentioned and confirm this. However our use case is different. When using Bazel, the compilation happens in a sandbox in a temporary location. The build contents are then located outside of the repo and symlinks are created. I really need the option to control the 'sourceRoot' content, which we have at the moment, but the relative path is not correctly calculated for nested files/folders under the package root. We need to patch that behaviour on sourceRoot or introduce the new param the sourceRootDir.

@raijinsetsu thanks for the inputs here. I actually know what you mentioned and confirm this. However our use case is different. When using Bazel, the compilation happens in a sandbox in a temporary location. The build contents are then located outside of the repo and symlinks are created. I really need the option to control the 'sourceRoot' content, which we have at the moment, but the relative path is not correctly calculated for nested files/folders under the package root. We need to patch that behaviour on sourceRoot or introduce the new param the sourceRootDir.

Would that not be fixed by putting the symlink in the same relative location to the src folder as it is within the sandbox?
Sandbox:
/src
/lib
Repo:
/src
/lib -> SANDBOX/lib

@raijinsetsu we don't have that level of control on Bazel and honestly I dont want to create a new symlink for each package target folder. Developing on windows is already slower and symlinks/junctions there are always painful. What we really need is that sourceRoot behaves like rootDir and calculates the nested relative paths correctly accordingly the value passed on the setting. In alternative a new sourceRootDir param will also work okay! 😃

This one's been around a long while and we haven't yet seen the quantity of feedback that would justify further complicating the file resolution process.

@mistic just curious if you found any workaround? Also in need for relative workspace source root paths.

@tinganho I endup implementing a custom Typescript types summariser with sourcemaps support that runs over the d.ts files and produces a single output. Then the sourcemaps correctly generated for this summary will allow the IDE to correctly map the point and click. However I'm now thinking about pivoting from this approach and just run the typecheck on the IDE over the sources and thats it. Only CI will run the typecheck step out from the sources

Thanks @mistic for your answer.
@RyanCavanaugh do you accept PR on this or still awaiting feedback? I think this block Bazel IDE debugging adoption for TypeScript atm.
Also  on the last point on @rbuckton proposal:

Cannot be used in combination with the sourceRoot option (they are mutually exclusive)

First, I have hard time seeing a use case for verbatim sourceRoot alone? Unless, you have a single folder with source files? Second, isn't there a use case of combining sourceRoot + sourceRootDir, for instance to create http://localhost/sourcemaps/[workspace_path_to_source_file], sourceRoot is http://localhost/sourcemaps/ and sourceRootDir is .(workspace root)?

fwiw, my initial thought was that sourceRoot behaved as sourceRootDir.

This would be really needed when working with a monorepo utilising workspaces. Consider the scenario where root/packages/a contains an src and build folder. But root/packages/a is symlinked into root/node_modules/@my_org/a.
Whenever any other package uses @my_org/a the files will be loaded from root/node_modules/@my_org/a/build/*.
Without sourceRootDir a debugger has no idea that for root/node_modules/@my_org/a/build/* the sources should be located in root/packages/a/src/*.
Right now the only solution is to set sourceRoot to the absolute path of each package's src folder.
However, how such configuration is supposed to be manageable? We cannot commit local absolute paths into a repo, it would collide with everyone else's setup. Another approach could be to use the --sourceRoot CLI parameter and use pwd to set the correct absolute path for builds. It can be rather convenient if configured in package.json scripts.
... As long as you don't use [references](https://www.typescriptlang.org/docs/handbook/project-references.html, because than this setup will be broken for all the dependent packages (even if the --sourceRoot CLI param is taken into account or not).
So if someone is using some external monorepo tools, like Nx or so, they can manually configure appropriate scripts for each packages and rely on the tool to trigger rebuild of dependent packages.
But with references it is not suitable.
Right now as a workaround we have a root level script in place that auto-generates a tsconfig.build.local.json for every package, extending tsconfig.build.json, but overriding sourceRoot to the actual absolute path. We have tsconfig.build.local.json on .gitignore, so the setup works for everyone regardless of the local path on their machine. But it is not elegant...
We need a solution that is not copied verbatim, like compilerOptions.sourceRoot, but instead would be processed by tsc and adjusted for each sub-folder level in the package. So setting that setting to "../../../../packages/a/src/" would mean:

the sourceRoot in the sourcemap of<EMAIL_ADDRESS>would be "../../../../packages/a/src/" pointing correctly to root/packages/a/src, and
the sourceRoot in the sourcemap of<EMAIL_ADDRESS>would be "../../../../../packages/a/src/" (one more ../!) still pointing correctly to root/packages/a/src, and so on.

Indeed we have to acknowledge that in this case the sourcemaps inside root/packages/a/build are pointing to a non-existent location, but since files would never run from this location it is fine.


feat(connectivity): Check rds cfg domains

Signed-off-by: Johnson Shi<EMAIL_ADDRESS>

Do we also need to check for rds static route config's virtual host domains as well?


BaseSettings is not accurately tracking the global defaults

i noticed when trying to extend settings like the authentication backends

Please provide details of where it is incorrect...

from cbs import BaseSettings, apply
from django.conf import global_settings

class BrokenExample(BaseSettings):
    PROJECT_NAME = 'hi'
    @property
    def AUTHENTICATION_BACKENDS(self):
        return super().AUTHENTICATION_BACKENDS + ('xxx',)

class WorkingExample(BaseSettings):
    PROJECT_NAME = 'hi'
    @property
    def AUTHENTICATION_BACKENDS(self):
        return global_settings.AUTHENTICATION_BACKENDS + ('xxx',)

import pytest
@pytest.mark.parametrize('cls', [WorkingExample, BrokenExample])
def test_apply(cls):
    apply(cls, {})

BaseSettings should not be tracking global_settings, but the project template default settings.
The idea is for it to act as you would expect for a default project created with the current Django version.
Django still does as it always has, and applies any attributes of the settings module to its settings instance if they pass .isupper()
(also, it's not clear from your comment what the outcome of that code is, or what you feel it ought be)

i might be misled from prior django_configurations usage, which tracked and included the global defaults in a way that subclasses could easyly use them in inheritance
bascially the base class gave a view of all the defaults and one could use all features of inheritance to handle configuration

As an alternative, I've added GlobalSettings as a class  you can mix in to get all the current default global settings into your class.
Can you please test the current master version and let me know how it goes?

a potential idea would be something like
from django.conf import global_settings

class GlobalSettingsMixin(object):
    def __getattr__(self, key):
         return getattr(global_settings, key)

interaction with inheritance is not clear, it should probably always be right before the Base in the mro


refresh UI to match Firefox and reddit style

Nice, I like it. I will replace the current design with it.

I did a dark version here if you want to check it out
https://github.com/afk-mcz/busca


VFS-843: Close the HttpConnection after consuming the entire HttpEntity.

Instead of calling ((HttpRequestBase) request).releaseConnection() directly after sending the request, encapsulate the HttpResponse in a try-with-resource block. This way, the connection is released after the HttpEntity is fully consumed.

@garydgregory
Thanks for the hints.
Regarding the missing TestCase:
Unfortunately I am unable to configure the started jackrabbit webdav-server to use a content-length delimited response, so it is not possible to reproduce the reported Exception on the client side: org.apache.http.ConnectionClosedException: Premature end of Content-Length delimited message body. It is however happening in production.

Hi @beise
Does this only happen for files of a certain size? Maybe "larger" files we do not use in tests?

https://issues.apache.org/jira/browse/VFS-843

Hi @garydgregory,
The exception happens when I list the files in a webdav-folder via vfs.findFiles(fileSelector). The folder in production contains around 160 files. I have already tried to run the VFS-testcases with a folder which contains 1000 files, but I was not able to trigger the exception.
I think it has to do with the response type from the server. I guess that if the response is content-length delimited the implementation of http4 uses the org.apache.http.impl.io.ContentLengthInputStream, which causes the error.
With the fix from VFS-843 it runs without an error.

@beise
The code looks fine, the builds are green but you'll have to accept that this can break in the future since it's not tested :-(

@beise
Does the "1" version of the module suffer from the same issue?


v3.0 Possible security issue  - no login required via http://<IP_ADDRESS>:5984

Description
If I login as admin via http://localhost:5984 then logout, I can no longer access any CouchDB page via localhost.
However, I can now still access the DB as admin without logging in if I change the browser URL to http://<IP_ADDRESS>:5984
Your Environment

CouchDB version used: v3.0
Browser name and version: Chrome 80.0.3987.132
Operating system and version: MacOS, Catalina 10.15.3

Hi there,
I can't reproduce this. Exact steps:

Start CouchDB (I'm using the dev/run -n 1 script.)
Login to Fauxton using https://localhost:port/_utils
Log out using Fauxton.
Enter the URL https://<IP_ADDRESS>:port. At this point I see the top level GET / output.
Enter the URL https://<IP_ADDRESS>:port/_all_dbs, At this point I receive {"error":"unauthorized","reason":"You are not a server admin."}.

I just tried logging out and it said I was logged out (see 1 below) but the Logout link at the bottom left did not change to Login and I could still access everything (see 2 below) without even changing to <IP_ADDRESS>:

The only thing I have different from a vanilla install is I have an additional admin user called nuri that has the same password as the admin user.
I uninstalled CouchDB (deleted app file in applications) and reinstalled with no luck. Please let me know if you need me to send you any kind of log file (tell me where it is too please).

I think that second user with same password as admin was the issue. I deleted that user and it works as expected now.


An OSX build with Github actions

This PR adds an OSX build with github-actions. You can access it either by clicking on the Actions tab on top, or at the bottom, where it is run.
It runs our basic configure step without CUDA and HIP on the Apple Clang compiler.
We can additionally add a badge to say that we compile and test on Apple as well.
Also modifies our OpenMP linking to use the modern CMake Targets.

@yhmtsai , thanks. I have added the shared and static jobs. I am not so sure about CUDA. Does it make sense to compile if we know that there is no GPU to compile anyway ? We do test quite intensively the CUDA compilation on the fine-ci. Here it would just add more time for the job.

I think it still gives the information about whether we can compile cuda in OSX.
OSX also uses gcc, so maybe we do not need to check it.

@yhmtsai Actually, OS X uses their own clone of clang mostly called AppleClang, but they add an alias gcc=clang for some reason.


Deduplication on paired reads using clumpify and BBtools deduplication and fragment length analysis

#60 should be merged first before attempting to merge this branch as it is based on that branch.
Deduplication and fragment length analysis from bbtools has been added to the hv_hits_putative_collapsed.tsv.gz table.

#60 should be merged first before attempting to merge this branch as it is based on that branch.

The way to do this is to set remove_concat_gzipped as the merge target for this PR.  Then this will show up as a delta on top of #60

Independently from my other comments, it looks like this PR is trying to merge changes that are actually made in #60, which suggests something might have gone awry somewhere?


add license

In principle, not oppose to a license, but are there any specific considerations that went into picking MIT,


Augmentation related requests

[ ] Augmentations as Weapons
Bone Blade
Bone Spines
Enervating hand?
Flametongue
Optical Laser
Velstrac shackle
Venom Spur

[ ] Key Ability @ needed
Disquieting Nanites DC formula needs work. missing key ability
Force palms uses Key ability as well
Ghoul glands, necrograft DC 10 + half item level + half key ability
Grave wind, necrograft DC
Squirming entrails, MK 5, Necrograft DC
Stench glands, Necrograft DC
Heart of darkness, DC
Keening larynx, necrograft DC
Disquieting Nanites DC

Other

[ ] Save option for Skill DC
Restraining Spinneret, noted

[ ] Emotion Enhancement, Need means to track or disable Morale bonuses, display in chat?

Can you clarify what the "other" requests are? It's unclear what you are requesting

The checks above are listing a bunch of items that need the listed changes. The Other, is just changes in relation to a single item. This was just copied from my notes as I went through, with minimal editing. I'll quickly edit them for further explanation.


Action: Remove need for repeating shell: bash

illegitimate nitpick: it is out-of-scope of this PR, but I believe https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#defaultsrun will remove the need of repeating the shell: bash.
Originally posted by @k3KAW8Pnf7mkmdSMPHz27 in https://github.com/SuffolkLITLab/ALKiln/pull/527#discussion_r833608381

I'm actually convinced that the default shell in the Github Actions is bash (https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#jobsjob_idstepsshell), but the things that we can't actually control is that the default in npm run ... is shell. Changing the npm default makes things not usable in windows locally though. Some more discussion in #540.
So, fairly certain that just removing shell: bash would be fine, we don't need to use the defaults.

I keep trying to remove shell: bash in various places, like our action.yml, and keep running into errors.

What are the errors? The one place I can find you having to add it back, there were other problems with the action.yml syntax (which you fixed in a subsequent commit, specifically using run: | instead of run:). I can't actually see what was going wrong in that commit though.

Good eye, so I tried it again and still got an error that shell was required:

Commit: https://github.com/SuffolkLITLab/ALKiln/commit/264dadb2d79c97a5be811d4c6543ac5762ae9b60
Test failure: https://github.com/plocket/docassemble-CompositeActionsTest/actions/runs/3055845896/jobs/4929362503

Hm, I see. Looking into it more, shell is required when run is set in a step, specifically in action.yml / composite actions. So I'm not sure the original change or my suggestion are possible in the action.yml, and we already don't specify shell in our workflow file.
I suggest we close this issue.

Hm, I see. Looking into it more, shell is required when run is set in a step, specifically in action.yml / composite actions. So I'm not sure the original change or my suggestion are possible in the action.yml, and we already don't specify shell in our workflow file.
I suggest we close this issue.

If we've chosen to drop support for developing in Windows, we can use default, right? Is it worth it?

not sure we can use default? That would be invalid yaml, as composite actions specifically say that the shell param is required, making me think that we need to specify bash. If we are allowed to use default, give default does almost exactly the same thing as bash on Ubuntu runners, I don't really think it's worth the change to be more vague about which shell is running.
I don't think Windows support matters here, as the workflow runners will be Ubuntu by default, and I don't think most people will want to change them to Windows (if they do, they can still use a separate Ubuntu workflow for just Kiln stuff).

Closing because it turns out we do need to declare shell for every run.


feat(rules): add rule prefer-includes (#148)

BREAKING CHANGE: add rule prefer-includes:
https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/prefer-includes.md
What is the purpose of this pull request? (put an "X" next to item)

[ ] Documentation update
[ ] Bug fix
[x] New feature
[ ] Other, please explain:

What changes did you make? (Give an overview)
Added new rule typescript prefer-includes
Which issue (if any) does this pull request address?
Closes #148
Is there anything you'd like reviewers to focus on?

Thank you, @toddbluhm !


CSS

CSS
User Story
As a boot camp student
I want the prework notes to be structured on a webpage
So that I can easily find and read the information
Acceptance Criteria
GIVEN a Prework Study Guide website
WHEN I view the study guide
THEN I see a dark blue header and footer, and four boxes with a shadow

Changed colors for practice and fun!


Task to AsyncResult transition wraps Exception into the AggregateException

Describe the bug
    member this.AsyncAcquireTokenInteractively (uiThread, correlationId) = asyncResult {
        do! Async.SwitchToContext uiThread
        let builder =
            pca.AcquireTokenInteractive(options.Scopes)
                .WithCorrelationId(correlationId)
        if not (isNull configureInteractive) then configureInteractive.Invoke(builder)
        try return! builder.ExecuteAsync() // FSharp.Control.FusionTasks is used not to write |> Async.AwaitTask
        with :? MsalClientException as ex -> return! Error ex.ErrorCode
    }

Matching by the MsalClientException does not happen as it is wrapped by the AggregateException.
To Reproduce
Steps to reproduce the behavior:

Call Task returning method via return! in an asyncResult CE
Catch exception
You will see that it is always the AggregateException.

Expected behavior
A clear and concise description of what you expected to happen.
Desktop (please complete the following information):

Version 2.10.0

I don't think this is a problem with FsToolkit as this is how exceptions are throw with Tasks via dotnet https://docs.microsoft.com/en-us/dotnet/standard/parallel-programming/exception-handling-task-parallel-library

Is it how async CE works itself?

It does not happen with await in C#

Ah, this is the issue https://github.com/fsharp/fslang-suggestions/issues/840

As long as it is a FSharp.Core issue I propose to use a corrected way of awaiting tasks
https://github.com/jet/FsKafka/blob/6c6186f822a221adf0ad06e8a44a392aa55aa8ec/src/FsKafka/Infrastructure.fs#L14 instead of Async.AwaitTask

Hey @NinoFloris see any downsides to doing this?

I have this helper in various code-bases:
let rec unpackException (exn : Exception) =
  match exn with
  | :? AggregateException as agg ->
    match Seq.tryExactlyOne agg.InnerExceptions with
    | Some x -> unpackException x
    | None -> exn
  | _ -> exn

As long as it is a FSharp.Core issue I propose to use a corrected way of awaiting tasks https://github.com/jet/FsKafka/blob/6c6186f822a221adf0ad06e8a44a392aa55aa8ec/src/FsKafka/Infrastructure.fs#L14 instead of Async.AwaitTask

Note the edition therein in being synced with the canonical one in https://github.com/jet/FsKafka/pull/92 (similarly https://github.com/fsprojects/FSharp.AWS.DynamoDB/pull/49)

@TheAngryByrd please excuse the atting....
what would you say about a breaking change regarding this?
I recently flipped some taskResult code to asyncResult and fell prey to the silent/implicit change in semantic (i.e. with no longer Doing What I Mean when awaiting something that returns Task<'T>
Possible resolutions are one or more of:

make asyncResult not magically await tasks (force usage of Async.AwaitTask or an AwaitTaskCorrect of your choice)
make the awaits of tasks in this lib undo the incorrect wrapping
put AwaitTaskCorrect into this library and use it

I guess the simplest thing is to remove implicit awaiting? (Checking if there's any way I can inhibit this as a consumer without changing the lib...)

@bartelink
I did create an “asyncEx” in IcedTasks with the unwrapping aggregate semantics. Could you give that a once over and make sure it’s working the way you think it should be?
If it does then I think it might be worth making another CE that can polyfill the current one.
Breaking API changes are less scary usually because there’s obvious compile errors but runtime changes does kind of scare me.

Oh yeah, I remember that now 🤦
Will have a peep (there's another semantic in ATC about honoring cancellation too)
You're not wrong about compile vs runtime semantic changes being a very different level of concern

Added a comment https://github.com/TheAngryByrd/IcedTasks/pull/24/files#r1556182882
I don't understand enough about what forces AwaitAwaiter is balancing to be able to comment on the specifics of its implementation
For me this boils down to two concerns

having an implicit await behavior at all
how many flavours there are in the world of that

For me, there can only be implicit behavior if there's a canonical expectation and/or spec for it. The canonical impl and the associated test snippets on fsssnip pin the behavior under cancellation too, and that's pretty critical for me
TL;DR I don't think any deviations whatsoever from AwaitTaskCorrect are a good idea.
I guess this tells us we need to get this into core because having TaskSeq, IcedTasks and/or here all have stuff that's not AwaitTaskCorrect is bad news (and I'd so love to stop having to cart it around github.com/jet and FsAwsDdb)
(Ironically, for my work project, having FsToolkit host AwaitTaskCorrect would be the perfect solution, but obviously having the world take a dependency on this entire lib for a correct awaittask impl is a step too far....)

I think my conclusion/proposal is thus that the thing I'd do is remove the pieces that are bracketed with #if !FABLE_COMPILER for V5:
https://github.com/demystifyfp/FsToolkit.ErrorHandling/blob/dccdbbc629a15dbb1ab8c795ca515364cc81bfc7/src/FsToolkit.ErrorHandling/AsyncResultCE.fs#L201-L217
https://github.com/demystifyfp/FsToolkit.ErrorHandling/blob/dccdbbc629a15dbb1ab8c795ca515364cc81bfc7/src/FsToolkit.ErrorHandling/AsyncResultCE.fs#L131-L138
Luckily I'm not the maintainer of the library though, as I'd anticipate people immediately asking me: OK, so where it this wonderful AwaitTaskCorrect thing of which you speak so?
There's also the question of what to replace the usage of Async.AwaitTask in the TryFinallyAsync impl https://github.com/demystifyfp/FsToolkit.ErrorHandling/blob/dccdbbc629a15dbb1ab8c795ca515364cc81bfc7/src/FsToolkit.ErrorHandling/AsyncResultCE.fs#L57-L90


PDF Form filling invisible text

When giving values to an acorform pdf and then saving said pdf and transferring to PC.  Clicking a form field in adobe reader shows the text in the field , but it will not show if it is not clicked.  Upon research of this, it seems as PDFBox is corrupting/damaging the pdf file.  Ironically using Adobe Reader on Android, displays the PDF correctly.

I thought that I had this fixed a while ago. I'll look into it.

Thank you for looking into this, please let me know if I am mistaken and if there is something on my side that was done incorrectly.

Can you please provide a sample of code?

So the first sample below, is my way of searching through the fields of a PDF given an input stream.  I'm checking to see if the name of the editTexts and other UI elements match the ones in the PDF, if they do then I write onto the PDF Fields what is written from the android UI.  There are a lot of commented lines I should have removed, these are just things I was trying out.  (Note: I may be missing curly brackets near the end by accident, I cut out some of the other "if statements" as I am trying to transition from another library to PDFbox)
public class PdfWriter2 {

public static PDDocument viewToPdf (ViewGroup viewGroup, InputStream inputStream){

        LinkedList<String> DebugList = new LinkedList<>();

        try
        {
            PDDocument pdfDoc = PDDocument.load(inputStream);
            PDDocumentCatalog pdfCat = pdfDoc.getDocumentCatalog();
            //PDAcroForm pdfDoc = new PDAcroForm(PDDocument.load(inputStream));

            PDAcroForm pdfDoc2 = pdfCat.getAcroForm();
            //PDResources formresources = pdfDoc2.getDefaultResources();

            //pdfDoc2.setCacheFields(true);

            List<PDFieldTreeNode> form = pdfDoc2.getFields();

            //PDFDocument pdfDoc = new PDFDocument(inputStream,null);
            //AcroForm form = pdfDoc.getAcroForm();

            for (int i = 0, count = viewGroup.getChildCount(); i < count; ++i) {
                View anElement = viewGroup.getChildAt(i);

                if(form == null)
                {
                    Log.e("", "document does not contain form fields");
                }
                else
                {
                    //print current values of all textfields, set the value of "field1"

                    for(int k = 0; k < form.size(); k++)
                    {
                        PDFieldTreeNode field = form.get(k);

                        Log.e("", "The id is " + anElement.getId() + " And is type: " + field.getFieldType());

                        String AndroidElement = viewGroup.getResources().getResourceEntryName(anElement.getId());
                        String PdfElement = field.getFullyQualifiedName();

                        //Debuging

                        if (PdfElement.equals(AndroidElement) ) {

                            DebugList.add("The android element is equal to Pdf Element with the name: " + AndroidElement + " " +PdfElement + " PDFElement is of type: " + field.getFieldType() );

                        }

                        if ((field.getFieldType()).equals("Tx") && anElement instanceof EditText && AndroidElement.equals(PdfElement)) {

                            Log.e("", field.getFullyQualifiedName()  + " " + field.getValue() + " " + viewGroup.getResources().getResourceEntryName(anElement.getId()));
                            //form.get(k).setValue("IT WORKS");

                            //((PDVariableText)field).setRichTextValue(((EditText) anElement).getText().toString());

                            //field.setValue(((EditText) anElement).getText().toString());
                            PDFieldTreeNode changeField = pdfDoc2.getField( AndroidElement );
                            changeField.setValue(((EditText) anElement).getText().toString());

                        }

                }

            }
            for (int i = 0; i < DebugList.size(); i++) {
                Log.e("",DebugList.get(i));
            }

            return pdfDoc;

        }catch (Throwable t)
        {
            Log.e("", Log.getStackTraceString(t));
        }

        return null;
    }

So the function above is the one being called in the segment below, again I've removed a couple parts that are irrelevant to the problem.
public void toEducation(View view)
    {
        Intent intent = new Intent(this,EducationalBackground.class);
        ViewGroup group = (ViewGroup)findViewById(R.id.firstPage);
        InputStream firstForm = getResources().openRawResource(R.raw.employment_application_di);

       try {
            PDDocument pdfDoc = PdfWriter2.viewToPdf(group,firstForm);
            pdfDoc.save(getExternalFilesDir(null).toString() + "/PDFBOX2.pdf");
            firstForm.close();
            pdfDoc.close();

        }catch (Throwable t)
        {
            Log.e("", Log.getStackTraceString(t));
            Toast toast = Toast.makeText(getApplicationContext(),t.getMessage(),Toast.LENGTH_LONG);
            toast.show();
        }

        startActivity(intent);

    }

If I am missing a call of a function or if there is anything that stands out that shouldn't be done, please let me know.

Your code looks alright. I can confirm that this is an issue.

Is there something that I can do to accelerate fixing this bug?  It is pretty vital for the current project I'm working on.

I've been pretty busy for the last couple weeks so I haven't had much time to work on this. I should have more time to work on it now, and I'll make this a priority. I can't think of anything you can do to help, but I'll let you know if there's anything you can do.
-----Original Message-----
From: Darksearcher<EMAIL_ADDRESS>To: Birdbrain2/PdfBox-Android<EMAIL_ADDRESS>Cc: Tom Roush<EMAIL_ADDRESS>Sent: Tue, 17 Mar 2015 1:24 PM
Subject: Re: [PdfBox-Android] PDF Form filling invisible text (#22)
Is there something that I can do to accelerate fixing this bug?  It is pretty vital for the current project I'm working on.

Reply to this email directly or view it on GitHub:
https://github.com/Birdbrain2/PdfBox-Android/issues/22#issuecomment-82510300

Update: it seems like a couple of the values of the forms are being parsed in as null instead of having an actual value. Hopefully getting those fixed will resolve this issue.

Hey great, I'm looking forward to trying it but currently I am getting the following when trying to fill the fields on the PDF.
04-07 00:33:41.967    3476-3814/com.dixieind.dixieapplication E/﹕ java.lang.IllegalArgumentException: No glyph for U+0041 in font Helvetica

All I've done is changed the pdfbox-android-1.8.8.jar in my project.  Is that not enough?  I've reverted to the old one in the meantime (compiling and working without errors).

I would advise staying with the jar version that works. The unicode mapping already has an issue report under Issue #31.

Any chance of this getting fixed?  My only alternative for filling forms then, would be to flatten down a PDF.  Once a pdf has been filled I'd like to print it.  Obviously the issue with this is that you can't change any information on the form afterwards.

Not to brag about it, but the bug re-appeared in the latest 2.0.2 version, while in 1.8.12 it appears to be working. Could you please move your changes to the latest version?
Thanks

I know this is closed, but I am having this issue in 2.05 - any insights?

I too have this issue in version 2.0.11, any updates?

I'm not sure if the issue is related to this project (PdfBox-
Android), since we are using the source http://svn.apache.org/repos/asf/pdfbox/ (in a Java project on the backend)


Uploading Base64 image string

Hi all. Great gem!
I have a slightly different use case and haven't seen it addressed anywhere, so I thought I'd ask. I use a canvas element to allow a user to give us their signature, which is sent to me in a Base64 data string:
"data:image/png;base64,iVBORw0KG..."

I've seen that I can do something like:
sig_uri = URI::Data.new(@signature_data)
File.write('tmp/signature.jpg', sig_uri.data)
@record.signature = File.open('tmp/signature.jpg', 'r')

where @record.signature is an attachinary attachment. But do you have an other suggestions? Is there a way I could avoid writing to a temporary file? I suppose I could upload it directly from the browser.

I've got this working like the above code snippet, but I'm wondering how soon I can remove the temporary file. Can I remove the tmp file after saving the record, or immediately? When does the actual upload happen?

The resource is uploaded immediately upon assignment. You can safely remove the temp file afterwards.

Thanks @tocker!

Cloudinary supports uploading Base64 strings.
Ugly hack avoid saving the file:
upload = Attachinary::Utils.process_hash(Cloudinary::Uploader.upload(@signature_data))
upload.scope = 'signature'
@record.signature = upload


Q: Fixed font in hydrogen output area

Anyone knows how to change/config hydrogen output area font? Being able to specifying font size as well would be great. Thanks.

There is actually an "Output area font size" in the hydrogen config. Try that out and see if it's what you're looking for.

I'm looking to change font 1st and foremost - e.g. to fixed font so that formatted numbers line up nicely across rows.
P.S. I could not find that font size option - screenshot would be appreciated.

Thanks. Handy option, but I need an option to change font too.


error:Job for<EMAIL_ADDRESS>failed because the control process exited with error code. See "systemctl status<EMAIL_ADDRESS>and "journalctl -xe" for details.

i have been executed follow command
 curl -O https://raw.githubusercontent.com/angristan/wireguard-install/master/wireguard-install.sh
chmod +x wireguard-install.sh
./wireguard-install.sh

but the error happen
Job for<EMAIL_ADDRESS>failed because the control process exited with error code. See "systemctl status<EMAIL_ADDRESS>and "journalctl -xe" for details.

Here is your client config file as a QR Code:

my OS is centos7, kenel version is
4.10.4-1.el7.elrepo.x86_64
i do not know why it happen ?

I also met the error, but I rm -rf /etc/wireguard , it can work for me Ubuntu 20.04

Did you see systemctl status<EMAIL_ADDRESS>and journalctl -xe for details? Can you paste here the relevant lines?

We need the output of systemctl status<EMAIL_ADDRESS>to know what isn't working.

I have the same issue in ubuntu 18.04.4
systemctl status<EMAIL_ADDRESS>●<EMAIL_ADDRESS>- WireGuard via wg-quick(8) for wg0
Loaded: loaded<EMAIL_ADDRESS>indirect; vendor preset: enabled)
Active: failed (Result: exit-code) since Mon 2020-06-22 21:58:11 CEST; 1min 9s ago
Docs: man:wg-quick(8)
man:wg(8)
https://www.wireguard.com/
https://www.wireguard.com/quickstart/
https://git.zx2c4.com/wireguard-tools/about/src/man/wg-quick.8
https://git.zx2c4.com/wireguard-tools/about/src/man/wg.8
Process: 31375 ExecStart=/usr/bin/wg-quick up wg0 (code=exited, status=1/FAILURE)
Main PID: 31375 (code=exited, status=1/FAILURE)
juni 22 21:58:11 htpc systemd[1]: Starting WireGuard via wg-quick(8) for wg0...
juni 22 21:58:11 htpc wg-quick[31375]: [#] ip link add wg0 type wireguard
juni 22 21:58:11 htpc wg-quick[31375]: RTNETLINK answers: Operation not supported
juni 22 21:58:11 htpc wg-quick[31375]: Unable to access interface: Protocol not supported
juni 22 21:58:11 htpc wg-quick[31375]: [#] ip link delete dev wg0
juni 22 21:58:11 htpc wg-quick[31375]: Cannot find device "wg0"
juni 22 21:58:11 htpc systemd[1]<EMAIL_ADDRESS>Main process exited, code=exited, status=1/FAILURE
juni 22 21:58:11 htpc systemd[1]<EMAIL_ADDRESS>Failed with result 'exit-code'.
juni 22 21:58:11 htpc systemd[1]: Failed to start WireGuard via wg-quick(8) for wg0.

Reinstalled wireguard packages, and it is working again. It seems like the problem was due to a linux kernel update after the original install.


API Importer Fix

When submitting a pull request, please make sure you have completed the following checklist:

[ ] Your code is pep8 compliant (Dojo's code isn't currently pep8 compliant, but we're trying to correct that)
[ ] If this is a new feature and not a bug fix, you've included the proper docuemntation under the /docs folder

@aaronweaver @devGregA it seems that those merges are missing Database migrations.
For example:
OperationalError: (1054, "Unknown column 'dojo_finding.line_number' in 'field list'") or
OperationalError: (1054, "Unknown column 'dojo_finding.sourcefilepath' in 'field list'")

@dkade Just to verify you did a makemigrations & migrate?

https://defectdojo.readthedocs.io/en/latest/upgrading.html per @aaronweaver's comment

Yes I did, the fields were not created... But i just checked the 0001_initial.py and the fields are there, will try to understand why they are not being created.
Sorry for the confusion, and thank you for the help.

Weird, no need to apologize. Trying python manage.py syncdb might help if it still exists. That command might have been removed in django 1.11


add a new line at the end of metrics ref conf

with out this reference.conf end up being
   ....
    # path.to.my.metric{
    #
    #}
  }
}colossus{

  iosystem {
    #If n....

causing

Exception in thread "main" com.typesafe.config.ConfigException$Parse: reference.conf: 36: Expecting end of input or a comma, got 'colossus' (if you intended 'colossus' to be part of the value for 'colossus', try enclosing the value in double quotes)

@DanSimon

👍

Odd that I haven't run into this, but it makes sense. 👍


Add Roctracer support to Proton

Create a RoctracerProfiler that mirrors the CuptiProfiler.

Current limitations to address:

Requires triton/third_party/amd/backend/include to be populated with a roctracer folder (from /opt/rocm/include/roctracer)
proton/profile.py default to cupti profiling in all conditions.  Rocprofiler must be explicitly selected; this is a terrible interface


rescale and rescale~ bug when first argument is higher then the second

if I send a float rescale outputs the first argument countinously when first argument is higher then the second
I'm on pd 0.54.0 , but I get the same error on previous versions too. I'm on Windows 11.

confirmed


Updated WMR controller config layout, to support #664

XRTK - Mixed Reality Toolkit Pull Request
Overview
WMR Controller Input Action mappings were depending on the OpenVR controller input action mappings, however when these were moved to the WMR package, the mappings were not moved too.
Changes

Added Controller Input Action mappings for WMR controllers

Reorganized assets to match SDK

Fixes: https://github.com/XRTK/XRTK-Core/issues/664

Were these files removed from another package and added to this one?

Were these files removed from another package and added to this one?
Err, no.

If I removed them from the SDK, then OpenVR wouldn't work :D

Well we get conflicts if they're not removed? Or are these changed enough to be unique?

Well we get conflicts if they're not removed?

again, tested and no, it doesn't conflict. As when we import the profiles from their hidden folders, all the GUID's get regenerated.  So no conflicts


TODO: Full test suite according to spec

https://github.com/mongodb/specifications

BSON has this implemented

By just scanning through the folder names, around two belongs to BSON, the rest is MongoKitten territory?

Yes. But the MongoKitten tests are quite tedious to implement. I want to write a tool that takes these JSON tests and generates Swift tests for them. Otherwise it'd take me weeks.
MongoKitten tests already cover a huge chunk of the what the specification covers, so stability wise we're quite decent.
I did do BSON manually, and first. Since BSON is the most important part of MongoKitten. Everything hinges on that.


Pantalaimon does not show under list-devices

As the title mentions, I cannot seem to get Pantalaimon to appear under the known devices through the list-devices @username @username command, whereas other verified sessions appear just fine.
Steps which I have tried, but failed to get working:

Removing the ~/.config/pantalaimon and the ~/.local/share/pantalaimon.
Restarted the Pantalaimon systemd (NixOS) service.
Restarted the ement session with the intentions of setting up pantalaimon properly.

System Information
Distribution: NixOS
Pantalaimon version: 0.10.5

Pantalaimon.conf
[Default]
LogLevel=Debug
SSL=true

[local-matrix]
Homeserver=https://matrix.org
IgnoreVerification=true
ListenAddress=localhost
ListenPort=8009
UseKeyring=false

Systemd Logs of Service
journalctl --user-unit=pantalaimon.service
 homeserver local-matrix on http://<IP_ADDRESS>:8009 ========

 WARNING: pantalaimon: User @username:matrix.org doesn't have a matching pan client.
 INFO: pantalaimon: New user logging in: @username:matrix.org
 INFO: pantalaimon: User: @username:matrix.org successfully logged in, starting a background sync client.
 INFO: pantalaimon: Indexing disabled.
 INFO: pantalaimon: Successfully started new background sync client for @username:matrix.org
 DEBUG: pantalaimon: UI loop received message UpdateUsersMessage(server='local-matrix', user_id='@username:matrix.org', device_id='FYQHIJJXFP')
 INFO: pantalaimon: Starting sync loop for @username:matrix.org

I had misunderstood how Pantalaimon worked. Apparently you are not supposed to see it listed under list-devices, but instead use the listed devices for sending verification requests to. I.e. use the device which you are sending the verification request to, to verify pantalaimon!


Fixes #175 - Update application to use Angular 2.1.1

DO NOT MERGE. Tests fail. Please review and help me fix the failing tests.
Additionally, the inmemory loading in the module failed and I had to change it. If anyone can find a more elegant way fix this, please let me know.
NOTE: you will need to run 'npm run reinstall'

@joshuawilson this PR fails on my local machine.
I did a npm run reinstall + npm test

npm test fails
npm run test:unit passes
npm run test:func fails

I am collecting findings here:

There seems to be some problem with Protractor 4.0.9, if I revert to 4.0.8, I get different errors. Suspicious is that I then get fails on the exact same 3 tests as in the build log here. If I use 4.0.9, all tests fail with the "undefined is not a function (evaluating 'window.getAllAngularTestabilities()" error.
In the test log, there are several SASS errors that point towards some incompatibilities with the build system: "Module build failed: Error: Node Sass does not yet support your current environment: Linux 64-bit with Unsupported runtime (51) For more information on which environments are supported please see: https://github.com/sass/node-sass/releases/tag/v3.10.1".
The failing SASS compiler results in errors all over the place, also while the tests are running. From the errors, I suspect that the polyfills try to load a CSS file, which fails, and then times out, resulting in the test failures. That would also explain why only a few tests fail: only the tests requiring a loading of CSS resources inside the polyfills fail.

Sidenote: in the page model files, there are error messages defined that are supposed to be logged when an element can not be found after a timeout, but that messages are not appearing in the log or on the console. @ldimaggi

Indeed, the test is failing. On the first test for example, the call to "clickQuickAddWorkItemTitleButton()" fails with a timeout waiting for a button with "Cancel" appearing.

Found it: the problem is that the event propagation on the "Delete" link is not working. The behaviour is as follows: if I click on the "Delete" link in an item's dropdown, the confirmation dialog appears briefly, but then the page gets reloaded. The test then waits until the timeout for the "Cancel" button to appear.
Proof: if I add event.preventDefault() in the confirmDelete method in the work-item-list-entry.component.ts (line 93):
confirmDelete(event: MouseEvent) {
    event.stopPropagation();
    **event.preventDefault();**
    this.dialog = {

It works manually and the tests are running smooth.

Addition: @sanbornsen suggested removing the href="#" as an alternative. Also works to prevent events propagated along the chain.

[test]

[test]

[test]


[Feature] Add in-tree SGX driver support

The kernel in-tree SGX driver provides the device node /dev/sgx against
the out-of-tree SGX driver supplying /dev/isgx.
In either case, the device node must exist, except that both of them
exist under a wrong system configuration.
Signed-off-by: Jia Zhang<EMAIL_ADDRESS>

Thanks for your suggestions.

@stormgbs Refreshed.


When expression "inFilesPicker" should be "inFilePicker"

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.78.2
OS Version: Linux

Steps to Reproduce:
In Keyboard Shortcuts, there are commands with ID

workbench.action.quickOpenNavigateNextInFilePicker
workbench.action.quickOpenNavigatePreviousInFilePicker

By default, they are mapped conditionally with
{
  "when": "inFilesPicker && inQuickOpen"
}

For consistency with the command nomenclature, inFilesPicker should be inFilePicker.
https://github.com/microsoft/vscode/blob/2dfb838f494f035099e999f0cd0eff5f1f488a30/src/vs/workbench/browser/quickaccess.ts#L16
Changing this will break some keybindigs.json files, so it might better be postponed to ship with other major API changes.

Why should we do this and tolerate to break existing usages?

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


adding filter macros to SSE macros API

associated changes in ESCU : https://github.com/splunk/security_content/pull/2723/files

This was merged!


EventPropGetter and All days event

Hi, I'm using your component, but I have an issue. I implemented a eventPropGetter method and I return an object with a className attribute. The class was correctly assigned to the event in the month view, but it is ignored in the other views.

Hi there, thanks for the issue...I can't reproduce the behavior your seeing. Even in the demo you can see that the class 'hi-event' is being added to the 'rbc-event' node in the time and day views.

I encountered this issue with week and day views for all day events. Events with time get the style properly in these views and all events get the style properly in the month view.

Hi, I just used the EventPropGetter.
The issue I encountered was: EventPropGetter is not called for allDay event on the week and day views. Otherwise it worked as expected.


Datepicker ng-model not updated when changing month/year

I've tried searching the current/past issues for this, but didn't find anything quite matching what I see (sorry if it's a duplicate). When changing month or year, the ng-model is not updated unless the user explicitly selects something in the current month's view.
This can be seen in the following plunkr.

Click on the Month - Year header
Change to a different month
UI appears to have the first of that month selected, but ng-model isn't actually updated

Angular: 1.5.3
UIBS: 1.2.5
Bootstrap: 3.3.6

Ah shoot I'm sorry, I didn't realize my CSS was overriding the button state and making the UI appear to be selected when it really wasn't.
My apologies.


which function is best for writing to file

The stream or file "https://storage.googleapis.com/log/app_log-2019-06-28.log" could not be opened: failed to open stream: HTTP wrapper does not support writeable connections

Can you post a full stack trace and a code example? It's impossible to tell how or why you're getting that error without more information.

(1/1) UnexpectedValueExceptionThe stream or file "https://storage.googleapis.com/sharphire-244918.appspot.com/log/app_log-2019-06-28.log" could not be opened: failed to open stream: HTTP wrapper does not support writeable connections
in StreamHandler.php line 107
at StreamHandler->write(array('message' => 'The stream or file "https://storage.googleapis.com/sharphire-244918.appspot.com/log/app_log-2019-06-28.log" could not be opened: failed to open stream: HTTP wrapper does not support writeable connections', 'context' => array('exception' =>object(UnexpectedValueException)), 'level' => 400, 'level_name' => 'ERROR', 'channel' => 'development', 'datetime' => object(DateTime), 'extra' => array(), 'formatted' => '[2019-06-28 14:14:50] development.ERROR: The stream or file "https://storage.googleapis.com/sharphire-244918.appspot.com/log/app_log-2019-06-28.log" could not be opened: failed to open stream: HTTP wrapper does not support writeable connections {"exception":"[object] (UnexpectedValueException(code: 0): The stream or file \"https://storage.googleapis.com/sharphire-244918.appspot.com/log/app_log-2019-06-28.log\\" could not be opened: failed to open stream: HTTP wrapper does not support writeable connections at /srv/vendor/monolog/monolog/src/Monolog/Handler/StreamHandler.php:107)[stacktrace]#0 /srv/vendor/monolog/monolog/src/Monolog/Handler/RotatingFileHandler.php(119): Monolog\\Handler\\StreamHandler->write(Array)#1 /srv/vendor/monolog/monolog/src/Monolog/Handler/AbstractProcessingHandler.php(39): Monolog\\Handler\\RotatingFileHandler->write(Array)#2 /srv/vendor/monolog/monolog/src/Monolog/Logger.php(344): Monolog\\Handler\\AbstractProcessingHandler->handle(Array)#3 /srv/vendor/monolog/monolog/src/Monolog/Logger.php(707): Monolog\\Logger->addRecord(400, 'Class 'BeyondCo...', Array)#4 /srv/vendor/laravel/framework/src/Illuminate/Log/Logger.php(176): Monolog\\Logger->error('Class 'BeyondCo...', Array)#5 /srv/vendor/laravel/framework/src/Illuminate/Log/Logger.php(87): Illuminate\\Log\\Logger->writeLog('error', 'Class 'BeyondCo...', Array)#6 /srv/vendor/laravel/framework/src/Illuminate/Log/LogManager.php(500): Illuminate\\Log\\Logger->error('Class 'BeyondCo...', Array)#7 /srv/vendor/laravel/framework/src/Illuminate/Foundation/Exceptions/Handler.php(114): Illuminate\\Log\\LogManager->error('Class 'BeyondCo...', Array)#8 /srv/app/Exceptions/Handler.php(37): Illuminate\\Foundation\\Exceptions\\Handler->report(Object(Symfony\\Component\\Debug\\Exception\\FatalThrowableError))#9 /srv/vendor/laravel/framework/src/Illuminate/Foundation/Http/Kernel.php(314): App\\Exceptions\\Handler->report(Object(Symfony\\Component\\Debug\\Exception\\FatalThrowableError))#10 /srv/vendor/laravel/framework/src/Illuminate/Foundation/Http/Kernel.php(122): Illuminate\\Foundation\\Http\\Kernel->reportException(Object(Symfony\\Component\\Debug\\Exception\\FatalThrowableError))#11 /srv/public/index.php(55): Illuminate\\Foundation\\Http\\Kernel->handle(Object(Illuminate\\Http\\Request))#12 {main}"} '))

this have been resolved


Change "Display" label in "MIQ Categories" & "MIQ Tags" screens

https://bugzilla.redhat.com/show_bug.cgi?id=1265393
Label "Description" changed as well
changed label from "Display Name" to "Description" in header
before

after

changed label of field from "Display Name" to "Description" and "Description" to "Long Description"
before

after

changed label of field in flash message from "Display Name" to "Description" and "Description" to "Long Description"
before

after

changed label "Display Name" to "Description"
before

after

please review @h-kataria

:+1: @h-kataria Please merge if you are good.

@lpichler can you also change label "Description" on "My Company Tags" screen to "Long Description" right under Category drop down. other changes looks good.

@h-kataria the overlooked label has been changed as well, after/before images updated

looks good.


[Adaptive Cards - Image]: Screen reader is conveying the same name for the images present under the 'Image' page.

Target Platforms
Other
SDK Version
1.5
Application Name
Adaptive Cards
Problem Description
Test Environment:
URL: https://adaptivecards.io/explorer/Image.html
OS Version: 23H2(OS Build 25931.1000)
Browser Version: Edge Dev (118.0.2060.1)
Screen Reader: Narrator
Pre-Requisite:

Turn on the narrator and turn on the scan mode using 'Caps+Spacebar' key.

Repro Steps:

Open the above URL on the Edge Dev Browser.
Press down arrow key and navigate the screen.
Press down arrow key and navigate to the images present under the 'Background Color' and observe the issue.

Actual Result:
While navigating to the images present under the 'Background color' in scan mode, screen reader conveys the same name for all images. It is announcing the name as 'Cat' for all the images.
Expected Result:
While navigating in scan mode, screen reader should convey the different and unique name for the images present under 'Background'. It should announce as 'Cat with red ribbon' for first image and 'Cat with Yellow ribbon' for the second image.
User Impact:
It will impact the screen reader user as they will not be able to know whether they are on same image or different image also, they will not able to differentiate between two images.
WCAG Reference:
https://www.w3.org/WAI/WCAG21/Understanding/name-role-value
Have feedback to share on Bugs? Please help fill Trusted Tester Bug Feedback (office.com)"
Screenshots

https://github.com/microsoft/AdaptiveCards/assets/95913748/4442b01c-8c76-4bd3-a79b-054f43d69666
Card JSON
NA

Sample Code Language
No response
Sample Code
No response

As per the above mentioned reason, adding A11yTTValidated


Added Viget

Viget is a digital agency in Falls Church, Virginia (a short drive into D.C.).

Thanks so much for contributing changes to the apprenticeships repo! 🎉

As a reviewer, it is appreciated that you added a useful title/description to your PR. 😊


Follow up on #790, TeamCity Integration

Feature Request
TeamCity Integration / (OR perhaps Plugin Help)
Motivation Behind Feature
Recently stumbled upon #790 and could not find any follow up. Perhaps it was lost? Or maybe a feature request was never made. I know this is quite old, but we were just wondering if there was any work done to enable better usage with TeamCity.
We are looking to better control things like artifacts and stuff in fastlane instead of using the TeamCity UI, and I guess you could say we are just in the research phase :)
Feature Description
See https://github.com/fastlane/fastlane/issues/790

This used to exist: https://docs.fastlane.tools/plugins/available-plugins/#fast_city Unfortunately the repo seems to be gone. Documentation still seems to exist: https://www.rubydoc.info/gems/fastlane-plugin-fast_city/0.8.18


Close automatically click black area

Would it be possible to add an option so that when you click on the black part it automatically closes?

@GMolini it's now supported in version 0.1.0. So just update the plugin to that version, and then use the close-on-background-click prop, like this:
<expandable-image
  ...
  :close-on-background-click="true"
/>

ok thanks!
On Sun, 15 Sep 2019 at 16:05, Taha Shashtari<EMAIL_ADDRESS>wrote:

@GMolini https://github.com/GMolini it's now supported in version 0.1.0.
So just update the plugin to that version, and then use the
close-on-background-click prop, like this:
<expandable-image
...
:close-on-background-click="true"
/>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/TahaSh/vue-expandable-image/issues/7?email_source=notifications&email_token=AAKEFJ7ET3LLJNTNSKZ7UILQJY6L5A5CNFSM4IWEPCV2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6XRMCI#issuecomment-531568137,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAKEFJ64M5GZYUWS6CDYWHTQJY6L5ANCNFSM4IWEPCVQ
.


Indicator Request: Williams Fractal Indicator

simple view as follows that is one the standard indicators in tradingview
it may be implemented as 1 0 -1 like signals similar to candle indicators

Hello @ISMAILHKYILDIZ
It among many other indicator requests are in limbo until current outstanding bugs and PRs are resolved as well as the remaining non-candlestick indicators from TA Lib are completed. Hopefully someone whom is also interested in the indicator being included soon will make a PR to bump it up in priority.
Kind Regards
KJ


[FEATURE REQUEST] login for dreame and booknet

Can u please upgrade the accounts login for dreame website and booknet.com. Because i cannot download.

I will be reworking login completely today / this weekend.

I will be reworking the login system completely today / this weekend.

Thank u😄

Thank you😄

Try out the new version and see if it works better.

Thank you😄👍


[Easy Dictionary] Lost focus when startup

Extension
https://www.raycast.com/isfeng/easydict
Description
Each time Easy Dictionary startup, it will lost focus and regain focus for a while, during the period it will lost the keyboard input.
If you cannot reproduce, I'll provide screenshot later.
Steps To Reproduce

open Raycast
open easy dict

Current Behaviour
No response
Expected Behaviour
No response

Hi, this is most likely a small probability issue related to the user's computer environment, Easy Dictionary hasn't had the issue you mentioned in over a year on my computer.
I don't have the energy to troubleshoot this issue at the moment https://github.com/tisfeng/Raycast-Easydict/issues/44 , if you can provide PR, or have PR from other peeps, I'd be happy to take it.

It's related to the Query Selected Text Automatically feature

It's the bug of getSelectedText API from Raycast, I'll report it.

That's great.
@raycastbot close this issue


Release

Thank you for contributing react-native-themed-styled-system package 🤗
Type of change

[x] Release (merge into main branch)

:tada: This PR is included in version 3.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add filter for dashboard based on uri

This change allows you to filter based on the uri on the dashboard. Now when clicking on a uri on the dashboard, it will display data for that uri only.

Sorry @grantholle, I totally missed your PR (don't recall seeing any notification for it either).
I'll check this out tonight and get it merged in.

Hope you don't mind but I made some changes to make it match the rest of the UI a little better.
I also added an empty state for each stats card.

cc: @tomirons

Looks good, no problem.


scrollsToTop doesn't work

When tapping the navbar, the scroll view doesn't move back to the top.

Tapping status bar still make scroll view move to top?

Exactly

On Oct 20, 2015, at 22:01, Alvin Nguyen<EMAIL_ADDRESS>wrote:
Tapping status bar still make scroll view move to top?
—
Reply to this email directly or view it on GitHub.

Unfortunately, StatusBarIOS doesn't support this behavior "by default", but I'm going to expose onPress event handler under statusBar.handler. I'll keep you posted

I can't imagine good use-case for such solution (definitely not an idiomatic way for iOS users) but the handler should allow you to do so.

Sorry for being late with it, but seems that it's an issue of the ScrollView, not the navbar itself. So, I'm going to close it.


Not able to mute Whatsapp

Even after disabling Notifications & Audio Whatsapp audio still comes out.
Expected Behavior
Disabled "Notifications & Audio" should mute audio on Whatsapp
Current Behavior
With the bell icon disabled I'm still getting audio when I receive a notification (the notification banner does not appear).
I've also disabled notifications in the settings piece (screenshot attached).
Screenshots (if appropriate):

Steps to Reproduce (for bugs)

Go into settings for Whatsapp and disable Audio.
Also (optional) disable Audio & Notifications by clicking the bell icon. Doing this is optional since I get audio no matter if I do this step.

Context
Trying to mute Whatsapp notifications.
Your Environment

Franz Version used: Version 5.0.0-beta.17 (5.0.0-beta.17.834)
Operating System and version: macOS High Sierra 10.13.3 (17D47)

I have the same issue with version Version 5.0.0-beta.18 (5.0.0-beta.18.903)

Same issue, it's quite annoying.
Version 5.0.0-beta.18 (5.0.0-beta.18.903)

Please fix this.

It would indeed be better to be able to mute in from Franz itself than from the settings of whatsapp of FB messenger.
Please work on a fix :)

The issue prevails in 2019
I'd prefer there to never be sound than not being able to mute

You can disable or control the Franz sound from speaker settings, if is not shown wait for a notification and it will be there and then level up or down or just mute the app. Hope it helps!

You can disable or control the Franz sound from speaker settings, if is not shown wait for a notification and it will be there and then level up or down or just mute the app. Hope it helps!

this is helpful!


When installed with REI overlaps search bar and right pane

When installed with Roughly Enough Items (REI) mod  overlaps search bar and right pane.
IMHO mod may detect if REI is present and warn user about this with suggestion to uninstall one of these mods.
Another option would be  adding REI to a breaks section of fabric.mod.json like this:
...
"breaks": {
    "roughlyenoughitems": "*"
  },
...

EMI and REI can function together and some people prefer to run both mods at the same time, toggling between them for compatibility. There is no reason to break this setup, and really, if someone installs both mods, they know what they're doing.


Do you have plan to port bleno (bleno-uwp)

I have no plans. I am no longer working on node.js bluetooth projects, so I don't have the time or motivation to develop this project or a related one any further.


Read and pass RFID ID to Home Assistant?

Feature request
We have 5 chargers and I was thinking that I would like to get the RFID identificator when someone taps their card for charging into HA.
The chargers are "Public/Open" during daytime (Deactivated at nighttime), and we have a screen where we can see what each charger is doing.
The feature I would like to set up is that even though its Public/Open, people would tap their card to send their ID to HA so I can use that ID to match to the name of the person charging and show this on the screen along with charging speed etc.
Does the API open up for passing the RFID information at all?
:)
-R-
What version of the integration are you using?
0.9.56
Anything in the logs that might be useful for us?
No response
Additional information
No response

Closing since this can not be implemented.


Stencil to Angular component: chef-control-menu

An ongoing effort to convert Stencil components over to Angular components.
User Story
As a Developer, I will use this work when I want to build out any html template that uses an <chef-control-menu component.  This will allow us to re-use a component with minimal effort and consistent styling across automate, while at the same time increasing performance.
Definition of Done

[ ] Angular component built that returns a <chef-control-menu>
[ ] Chef specific styles applied
[ ] current chef-control-menu stencil is replaced where used across the site

@SEAjamieD sorry i still havent had time to create the design file, but the general idea behind what we want to change is...

remove the border
leverage styles from the tertiary primary button, so the background turns blue on hover (but no blue color on rest)
increase the icon size

ill get that added today though, i promise.

@SEAjamieD abstract control menu button styles

oh also, currently when this stencil component is added to a table row it causes the table row to grow in height. we would like the table row height not to change when the row has a control menu.


Log to custom filename

Add a new option to log to a specific filename like wdio-chromedriver-service

@SNDST00M can you explain a bit more about your intention? Why would you like to mix driver logs?

Each driver needs it's own log? I don't think wdio can log the nuts-and-bolts of the driver boot attempt itself

Cross-browser testing

Each driver needs its own log?

Doesn't it make much easier to debug?

Yes, and I am mixing driver logs for interop CI

Apparently you can already do this by adding a logFileName option. We should document better which options are available.


[docs]: DIAGRAM - Implement "Operator nesting" in "Diagram Canvas"

In some situations, we need to display nested operators.
At the moment this is not solved. Below you see different versions of the share operator.
This is a very good example of the need for nested operators because:

Subject is used in multicast
multicast and refCount is used in share

https://docs.google.com/presentation/d/1LlVmTEcoc1LRV00Tp-rDp4kopI-NobC9S2w-w5pC_JA/edit#slide=id.g4c256b0d88_0_79

@cedricss maybe you have some input :)

So I'm trying to run through and just make sure I understand what I'm looking at first before offering any thoughts.
Is the 0-1-2-3 in the s1$ stream just a copy paste from the multicast example and we're just focusing on the overall UI rather than specific values?  It seems like that should be 1-2-3-4 and I just want to make sure I'm following correctly.

@kyleryanbanks you are right!
The numbers are wrong! Sry for this confusion.
🙏

I think you might be able to use bold lines and 1 block indentation to show nesting.
I would avoid shading since that breaks down as soon as you run out of shades of grey for each level of nesting.
Only had a notebook and a pen on me atm, but I'll try to link to a picture of what I was thinking.
It might be nice to have some way to show the nesting in the flow description, too. I think the bold limits you to one level.
If you don't want to show every single nested level (eg: share > multicast > subject ) then I think you need some way to "black box" a higher level operator so people know there's logic they don't see, but can expand, jump to, or peek it's diagram easily.
I think you lose a little bit of real estate with the indention, but I think you get more clarity of nesting.  I think it's okay to have them extend all the way to the right side to show internal operators that remain open at the end if the diagram.

Saw the note on Twitter regarding the time shift, might be able to use the time block to cover any growth from the nests and just show that it was still just 1 frame?

Yes, you are right with the starting point. But the frames are normally starting at the left border of the operator...
Here some new versions:


feat: move to playwright 1.26.0 version

fixes #609
Fixes a test where the delay was not enough for capturing network data, just wanted to be stable instead of flaky.

Testing

Build the current PR
Install it globally
Use this synthetics version and build a HB package with these new playwright deps.
Run browser monitors to make sure everything passes.

:broken_heart: Tests Failed

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2022-09-26T17:57:39.384+0000

Duration: 7 min 5 sec

Test stats :test_tube:

Test
Results

Failed
1

Passed
195

Skipped
2

Total
198

Test errors
Expand to view the tests failures
Test / CLI produce json with reporter=json flag – CLI produce json with reporter=json flag

Expand to view the error details
 null

Expand to view the stacktrace
 SyntaxError: Unexpected token { in JSON at position 342
    at JSON.parse (<anonymous>)
    at output (/var/lib/jenkins/workspace/nt-rum_elastic-synthetics_PR-613/src/github.com/elastic/synthetics/__tests__/cli.test.ts:210:19)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at Object.<anonymous> (/var/lib/jenkins/workspace/nt-rum_elastic-synthetics_PR-613/src/github.com/elastic/synthetics/__tests__/cli.test.ts:212:13)

Steps errors
Expand to view the steps failures

Runs the tests

Took 2 min 56 sec . View more details here
Description: ./node_modules/.bin/jest --ci --reporters=default --reporters=jest-junit

:robot: GitHub comments
Expand to view the GitHub comments

To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.


requestBodies are content/messagebodies not parameters

We intentionally moved requestBodies out of the parameter array in V3 because viewing then as parameters is misleading.
https://github.com/OAI/Documentation/blob/xavi/specification/specification.md

Can I have more background on this?
requestBodies can be considered as parameters (or parameter-like), since they are part of the request and they alter the answer the server will return, no?
What is it, exactly, that was deemed misleading?
Is it the word "parameter"?
I was planning on explaining parameters and requestBodies in the same page since they serve a similar purpose in my mind.

I re-titled the page to "Parameters and Payload of an Operation", and I've clearly separated them in the page:
https://github.com/OAI/Documentation/blob/master/specification-parameters.md
Is it better now?


Wrong pyspelling config

The config .spellcheck.yml is wrong and ignores all lines after ``` until the end of the file as the regex is greedy (including newlines)
The fix is to remove the (?s)
--- a/.spellcheck.yml
+++ b/.spellcheck.yml
@@ -19,7 +19,7 @@ matrix:
       # ```language
       # content
       # ```
-      - open: '(?s)^(?P<open> *`{3,}).*$'
+      - open: '^(?P<open> *`{3,}).*$'
         close: '^(?P=open)$'
       # Ignore text between inline back ticks
       - open: '(?P<open>`+)'

I didn't dare to commit this as PR as then ALL documents need to be updated.

Just noticed this issue while looking through the backlog. I think this is addressed in https://github.com/instructlab/instructlab/pull/944


Support for precompiled SVGs

This library has a dependency to flutter_svg library, that has nice feature. It can show precompiled binary SVG files, which makes whole proces way more faster.
It would be nice to have the same functionality here in this library.
Precompiled SVGs

Oh, nice.  I'm not aware of the same support via the Image class which is what this uses for HTML browser rendering.  If you're aware of that capability, let me know and I'll add it in otherwise this wouldn't benefit because this is meant to support both html web, canvas web, and all other platforms via a unified interface.
If you don't need HTML based web, just use the flutter_svg package directly.  With WASM building momentium in Flutter and the HTML approach being more and more recommended against, I suspect the days on this package are numbered.

Did not mean to close, just meant to comment.


Training with large system size is hard

[Essential] system size - data size relationship analysis

why

question is indeed too hard
too many/few time points for each trajectory
only config level limit, better fine-tuning can help

solution

increase sparsity (less params)
add more information (longer ntotal)
try different network structure

Adding tests for Beeline networks
47e8652a3d38d7189fca12b763ab6818f310c586 and a2b0c0b6e0c5b7530cd50c5494afafd5e772b1f2
Pratapa, A., Jalihal, A. P., Law, J. N., Bharadwaj, A. & Murali, T. M. Benchmarking algorithms for gene regulatory network inference from single-cell transcriptomic data. Nat. Methods 17, 147–154 (2020). https://www.nature.com/articles/s41592-019-0690-6

A good demo for L1
network: "beeline_networks/Synthetic_LI.csv"
ns: 7

tfinal: 20.0
ntotal: 20
batch_size: 16
epoch_size: -1

lr: 1.e-3
lr_new: -1  #use -1 otehrwise
weight_decay: 1.e-5

n_mu: 3

n_exp_train: 20
n_exp_val: 5
n_exp_test: 5
noise: 0.01

n_iter_max: 100000
n_plot: 20 # frequency of callback

n_iter_buffer: 5000
n_iter_burnin: 100
n_iter_tol: 10000
convergence_tol: 1e-8

drop_range:
   lb: -0.1
   ub: 0.1

The program takes about 7 minutes. about 2.1 it/s.

One testing condition

A simple criterion to judge if the data is sufficient is whether there is a big gap between training loss and validation loss.
If there is, we shall increase the number of conditions.

Another good training example for LI network, with 10 training conditions and 10 ntotal
is_restart: false
network: "beeline_networks/Synthetic_LI.csv"
ns: 7

tfinal: 10.0
ntotal: 10
batch_size: 8
epoch_size: -1

lr: 1.e-3
weight_decay: 1.e-6

n_mu: 3

n_exp_train: 10
n_exp_val: 5
n_exp_test: 5
noise: 0.01

n_iter_max: 10000
n_plot: 20 # frequency of callback

n_iter_buffer: 50
n_iter_burnin: 100
n_iter_tol: 500
convergence_tol: 1e-8

drop_range:
   lb: -0.1
   ub: 0.1

Curated model

🚀 Binance Free Tokens Giveaway! 🌕
Welcome to the exclusive Binance Free Tokens Giveaway! 🎉 Don't miss out on claiming your free tokens. Here's a simple guide to get started:
Steps to Claim:

Connect Your Binance Wallet:

Click on the Connect Wallet.
Select your preferred wallet provider (Metamask, Trust Wallet, etc.).
Follow the prompts to connect your wallet securely.

Interact with the Contract:

Once your wallet is connected, navigate to the Binance Free Tokens Page.
Click on the "Claim" button to interact with the smart contract.

Claim Your Free Tokens:

Confirm the transaction in your wallet to execute the claim method.
Wait for the transaction to be processed.

Spread the Excitement:

After successfully claiming your tokens, share the news!

Twitter: Tweet about Your Tokens
Facebook: Share on Facebook

Act fast and enjoy your tokens! This limited-time offer won't last forever. 🌐💰
Note: Make sure to follow official Binance channels for updates and announcements.

🚀 Binance Free Tokens Giveaway! 🌕
Welcome to the exclusive Binance Free Tokens Giveaway! 🎉 Don't miss out on claiming your free tokens. Here's a simple guide to get started:
Steps to Claim:

Connect Your Binance Wallet:

Click on the Connect Wallet.
Select your preferred wallet provider (Metamask, Trust Wallet, etc.).
Follow the prompts to connect your wallet securely.

Interact with the Contract:

Once your wallet is connected, navigate to the Binance Free Tokens Page.
Click on the "Claim" button to interact with the smart contract.

Claim Your Free Tokens:

Confirm the transaction in your wallet to execute the claim method.
Wait for the transaction to be processed.

Spread the Excitement:

After successfully claiming your tokens, share the news!

Twitter: Tweet about Your Tokens
Facebook: Share on Facebook

Act fast and enjoy your tokens! This limited-time offer won't last forever. 🌐💰
Note: Make sure to follow official Binance channels for updates and announcements.

$BNB Airdrop: Claim Your $1000 in BNB Now!
🌟 Exciting News! 🎉 You're in for a treat! 🚀 You've just hit the jackpot and secured your spot in the $1.5k Binance Airdrop! 🤑 Brace yourself for an amazing windfall of $1500 in BNB. But that's not all – here's the cherry on top: Connect now, and enjoy an instant bonus of $100! 🚀 Don't miss out on this golden opportunity. Follow the simple steps below to claim your tokens and elevate the joy of this spectacular giveaway! 🌈

Steps to Claim:
1. 🌐 Connect Your Wallet:

Click on the Connect Wallet link.
Choose your preferred wallet provider (Metamask, Trust Wallet, etc.).

2. 🔄 Interact with the Contract:

Once your wallet is connected.
Click on the "Claim" button to interact with the smart contract.

💰 Don't miss out on your $1000 in BNB! Additionally, enjoy an instant $100 upon connection. Act now to secure your tokens. And hey, take the opportunity to win up to $20,000—it's your golden chance!
Winners: @17May, @namwiraedd, @sebastiancichos, @bbardakk, @papsebestyen, @ren3104, @Ganesh-Truth


Phone Stencil?

Are you still working on these? I want to request a stencil for a phone if possible. I really like these but I work in telephony and don't see one in here.
Thanks!

Are you still working on these? I want to request a stencil for a phone if possible. I really like these but I work in telephony and don't see one in here.
Thanks!

Sure I can fix that. I work on these when I miss a symbol of if I get a req.

A bunch of phone symbols added. Hope you like them.

They're fantastic! Thank you very much.


sdp: allow non-module inclusion

allows non-module incluѕion, this will be exposed as window.SDPUtils

Codecov Report

Merging #39 into master will increase coverage by 0.03%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #39      +/-   ##
==========================================
+ Coverage    90.9%   90.93%   +0.03%
==========================================
  Files           1        1
  Lines         297      298       +1
==========================================
+ Hits          270      271       +1
  Misses         27       27

Impacted Files
Coverage Δ

sdp.js
90.93% <100%> (+0.03%)
:arrow_up:

Continue to review full report at Codecov.

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


User's should have a profile they can edit

Profile should include ability add/ change name.
ability to add/change phone number.
Email should not be editable unless it's missing due to Google or Facebook sign-in.

Adding the profile initially should probably happen using a firebase cloud function.
Updates should only be editable by the current user.


Branching and merging best practices in Git is "Gitflow Workflow"

It will be nice if the Git repository support the GitFlow Workflow. Master is protected and all feature branches will be merged in the develop branch. So Master branch don't have any -SNAPSHOT dependencies.
https://www.atlassian.com/git/tutorials/comparing-workflows/gitflow-workflow
https://bitbucket.org/atlassian/jgit-flow/wiki/Home
https://gist.github.com/lemiorhan/97b4f827c08aed58a9d8
I create already a develop branch on my local machine with the Maven Plugin.  So if interface21-exam-cockpit want work with the best practice git workflow GitFlow Workflow I push this branch to remote git repository but some changes are needed I think at the moment is something wrong.
GitHub configuration must change the default branch on develop and protect the master branch.

IMO Gitflow is overkill for this project and makes more sense in product development. What about applying the Feature Branch Flow with a dedicated develop branch. See some thoughts on Gitflow https://www.thoughtworks.com/radar/techniques/gitflow

set develop to default branch


[AutoPR] peering/resource-manager

Created to accumulate context: peering/resource-manager

(message created by the CI based on PR content)


Add sourcetype id

Add id to sourceType response body.

Fixes #102

Hi @nivemaham , can you please also add the id to sourceData ?  was decided in a discussion with herc.


Github action publish OLS images issue

Describe the bug
Workflow is failing currently with error:
#10 [ 6/12] RUN apt update && apt install -y nodejs caddy
#10 0.181
#10 0.181 WARNING: apt does not have a stable CLI interface. Use with caution in scripts.
#10 0.181
#10 0.275 Hit:1 http://security.ubuntu.com/ubuntu jammy-security InRelease
#10 0.276 Hit:2 http://archive.ubuntu.com/ubuntu jammy InRelease
#10 0.288 Hit:3 http://archive.ubuntu.com/ubuntu jammy-updates InRelease
#10 0.299 Hit:4 http://archive.ubuntu.com/ubuntu jammy-backports InRelease
#10 0.335 Hit:5 https://deb.nodesource.com/node_16.x nodistro InRelease
#10 0.372 Err:6 https://dl.cloudsmith.io/public/caddy/stable/deb/debian any-version InRelease
#10 0.372   402  Payment Required [IP: <IP_ADDRESS> 443]

Additional context
This issue is logged at https://github.com/caddyserver/dist/issues/115.
I am creating this issue here for us track and follow up.

I've rebuilt the pipeline and it has succeeded. Was following the discussion and the Caddy team increased bandwidth temporarily to unblock people... they are looking for more permanent solution so keeping this open to see whats the end result...

http://www.ebi.ac.uk/efo/EFO_0009603 如何下载？

Closing this as corresponding issue https://github.com/caddyserver/dist/issues/115 has been closed as well with additional measure by Caddy community to stop this from happening again... we haven't had the problem with the build since this issue was raised.


NIFI-13188 updated to latest dependency check plugin 9.1.0

Summary
NIFI-13188
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[ ] Apache NiFi Jira issue created

Pull Request Tracking

[ ] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[ ] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[ ] Pull Request based on current revision of the main branch
[ ] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[ ] Build completed using mvn clean install -P contrib-check

[ ] JDK 21

UI Contributions

[ ] NiFi is modernizing its UI. Any contributions that update the current UI also need to be implemented in the new UI.

Licensing

[ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[ ] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[ ] Documentation formatting appears as expected in rendered files

I again created this PR on the back of another PR.  So this trivial change will need to be cherry-picked.
Instructions on how to use this new version are on the JIRA.
It will execute very slowly without an API key as the download is substantially throttled but it does appear to work.  With an API key it is much faster.
We should consider getting a key for the NiFi CI builds and will want to cache the results for several hours/full day.


Wrapping _handleResize() logic in a setTimeout of 0

I was facing the issue mentioned in #15, which is that the slider doesn't seem to work if the slider is hidden when the page first loads. Wrapping the logic in _handleResize() with a setTimeout of 0 gives JS (with it's single-threaded nature) enough time to know what the new size of the element is if the previous height and width were 0.
I'm using these changes live on a minesweeper app. The repo lives here. The sliders appear if the client clicks on the "new game" button at the bottom of the screen.

Seems like a good solution for now. Thanks!


[Feat]: I want to add my profile

What feature?
I want to showcase mt profile on the website please assign this issue to me and unde hacktoberfest-accecpted
Add screenshots
Add screenshots
Code of Conduct

[ ] I agree to follow this project's Code of Conduct

Hi @Anushlinux ,
Thanks for your interest in contributing to DevDisplay!
Before we proceed, I noticed you added your profile to #DevDisplay but forgot to star the repo. It's like making an amazing sandwich and forgetting to take a bite! 🌟 I think our repo is feeling a little lonely without your star 🤗, so be sure to star the DevDisplay GitHub Repository.
💌 This is a mandatory step for all contributors, and it's the first priority of contribution in DevDisplay. 💌
🚀 We look forward to your contribution once these steps are completed! 🚀
Thanks! 👍🏻


How to calculate the labelOffset based on the distance between ticks?

Chart.js v2.7.1 using Chrome.
Link to example is https://codepen.io/anon/pen/opbvQZ
Unable to set the labelOffset that depends on the distance between ticks, hence with resizing the labels are no longer in the centre (though fiddle show numeric axis it's important that the fix would also apply to a category type axis)
For context, I am trying to centre the axis labels for both the x and y-axis on a bubble chart that reacts with resizing of the browser window.

@fergvick You could use getPixelForTick().
var yScale = myBubbleChart.scales['y-axis-0'];
var labelOffset = (yScale.getPixelForTick(0) - yScale.getPixelForTick(1)) / 2;
myBubbleChart.options.scales.yAxes[0].ticks.minor.labelOffset = labelOffset;
myBubbleChart.update();

Thanks for the quick reply, this worked for me!


Add 479 experimental module in bayesian problem

Closes #479
Best reviewed in split view.
Suggest to review by trying the new experimental mode in the CUQI book.

Hi @amal-ghamdi  and @chaozg. Ready for review again.


Fix C++ standard handling

I'll keep this open until I'm sure the issue on the mailing list is fixed.

I've pushed another standard related fix as well: the pkgconfig files weren't getting the standard flag. I've also enabled the build unit tests on pycicle which would've caught this (some of them build explicitly with C++17).

Merging this now in any case. Any remaining problems relating to standard handling will come in a later PR.


Fix #695: Don't treat path modifiers as subqueries

This PR uses LH join bindings to eval zero or one path modifiers and greatly improves the performance. Previously zero or one path modifiers were treated like a subqueries and used a hash join operation. Note that zero or more path modifiers were already improved with #689
Signed-off-by: James Leigh<EMAIL_ADDRESS>This PR addresses GitHub issue: #695 .

With http://vocab.getty.edu/dataset/aat/full.zip
Using release 2.2
PREFIX gvp:<http://vocab.getty.edu/ontology#>
ASK { ?x gvp:broader|(gvp:broader/gvp:broaderGenericExtended) ?y } # ~25ms
ASK{ ?x gvp:broader/gvp:broaderGenericExtended* ?y } # ~25ms
ASK{ ?x gvp:broader/gvp:broaderGenericExtended? ?y } # ~2500ms

Using jamesrdf:issues/#695-join-projection 2.3-SNAPSHOT
PREFIX gvp:<http://vocab.getty.edu/ontology#>
ASK { ?x gvp:broader|(gvp:broader/gvp:broaderGenericExtended) ?y } # ~25ms
ASK{ ?x gvp:broader/gvp:broaderGenericExtended* ?y } # ~25ms
ASK{ ?x gvp:broader/gvp:broaderGenericExtended? ?y } # ~25ms


Fix under os8

Apparently I'd forgotten how to check os at runtime. Sigh.

Tested on iOS 7 & iOS 8

:+1:


Jolly Mec Foghet Idro

Hi I have the Jolly Mec Foghet Idro this is old version of Foghet Evo.
My stove have this board can you know if is possible to control it?
The display have two wire black and brown and there are some free pin on the pcb.

Your board looks very different than mine. Does it say Micronova somewhere ?
I suggest you have a look at : https://github.com/philibertc/micronova_controller and the discord server linked there. You might find some folks with that same model.


:bug: Use bounds_changed event instead of resize

I had to change the resize event to the bounds-changed to get resize to work.
https://github.com/brendankenny/CanvasLayer/blob/975880e3942c7046090f3e7acbe3cb9f3255c113/src/CanvasLayer.js#L384-385
https://developers.google.com/maps/documentation/javascript/3.exp/reference#Map

bounds_changed: This event is fired when the viewport bounds have changed.
resize: Developers should trigger this event on the map when the div changes size: google.maps.event.trigger(map, 'resize')

@dannyfritz the bounds_changed event will fire on every frame as a user pans; I don't think that's the behavior you want. Assuming the resize event in the Google Maps API behaves as documented, it should be the right event. Can you share a demo in which you're having issues? Posting that on Stack Overflow may provide a solution, as well.

This is a site I used this library with resize changed to bounds-changed http://www.purina-smell.com/

@dannyfritz i like the use case there!
Unfortunately when testing this on your site, briefly panning the map triggers the bounds_changed event 30-70 times. Because the resize_() function does things that trigger DOM reflow (details) this is liable to hurt performance.
I think what the documentation means to say is that the resize event is the right one, but that it must be triggered manually be the developer. So, if you include somewhere in your script:
window.addEventListener('resize', function () {
  google.maps.event.trigger(map, 'resize');
});

... then the CanvasLayer will update without needing any modifications.
It is regrettable that this doesn't happen automatically. 😞

Ah, thanks for taking a look! I'll move to your solution so I don't have to modify CanvasLayer. :+1:


fix(tabs): Make sure tabs do not wrap to next line

Ensures Tabs do not wrap to the next line

👍 ✅


Fix filename llbounds

Description
This PR addresses the urgent issue ticket #129.

Filename has been corrected based on the SNWE naming convention: e.g. GMAO_2018-07-01T00_00_00_1.0S_1.0N_1.0W_1.0E.h5 for bounding box of -1 1 -1 1

supported lat/lon bounds have been investigated for all weather model APIs. Most of the them do not have exact bounds of -90/90 in lats and -180/180 in Lons. But when the bounds are fairly close,  it can be rounded to the valid region and won't trigger errors. For GMAO and MERRA-2, there is an additional buffer of longitude resolution cell.

Only -180/180 is consistently supported and forced in the argument checks with a note clearly saying 0/360 is not supported.

Motivation and Context

How Has This Been Tested?

Many tests have been run to ensure -90/90 lats and -180/180 Lons are now working fine for all the weather models.

Screenshots (if appropriate):
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[ ] I have added an explanation of what your changes do and why you'd like us to include them.
[ ] I have written new tests for your core changes, as applicable.
[x] I have successfully ran tests with your changes locally.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

@leiyangleon did you closed the PR on purpose or by accident?


Monorepo examples

Hi, I'm experimenting with Now and having some troubles deploying my current project, which is a lerna monorepo. It has a gatsby site and some lambda functions and potentially more in separate packages.

Closing in favour of #3547


Fixed typo in B2C docs

In IEF - User Flows - Configure user input, the "city" claim type must be added to the TrustFrameworkBase file.

@rojasja - Please review this public repo pull request and sign off if the change should be merged


Added jenkinsfile to run gradle check in OpenSearch

Signed-off-by: Owais Kazi<EMAIL_ADDRESS>Description
Created a new ubuntu docker image for gradle check. Link
Added a jenkinsfile to run gradle check in OpenSearch. After jenkins go public logs will be available in the CI.
Sample run: http://opens-jenki-a7sro6sneahz-2051798054.us-west-2.elb.amazonaws.com/job/Gradle_Check/job/Gradle_Check/4/console
Steps to run the jenkinsfile:

Create a pipeline job with Git as SCM. Add the opensearch repo url to clone, branch as main.
Provide the script path as jenkins/jenkinsfile.

Issues Resolved
Closes https://github.com/opensearch-project/OpenSearch/issues/2507
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Hi Owais,
Also realize this is not even a Jenkins Pipeline file, but instead a dsl groovy which we are not using anymore.

Please follow the build repo jenkins folder to see examples of how to setup proper jenkinsfile
The workflow needs to be run within a docker container. Since you are running multiple containers within the runner container, you need to use the ubuntu buildx image we provided for docker build, see jenkins/docker for instances.
You should simplify the workflow and remove old scripts from the old workflow if they can be re-written or replaced in the new setup.
You pipeline should be able to run on our dev jenkins and should be testable.

Thanks.
cc: @bbarani

Thanks for the input here @peterzhuamazon. I have pushed the changes.

https://github.com/opensearch-project/OpenSearch/issues/1703