[Feature] Allow to override the default global config, to keep importing styled from '@stitches/react' in all files

Currently if you need to customize the config (example: https://stitches.dev/docs/responsive-styles), you'll need to change all your
import { styled } from '@stitches/react';

into
import { styled } from 'path/to/your/config';

which is annoying, it's easier to spot when stitches is an external import
So I'd propose something like stitches.config.js:
import { setGlobalConfig } from'@stitches/react';

setGlobalConfig({
  media: {
    bp1: '(min-width: 640px)',
    bp2: '(min-width: 768px)',
    bp3: '(min-width: 1024px)',
  },
});

@caub Thanks for your suggestion but unfortunately, that's not something we will add to Stitches as the global styled, css functions were just meant for quick prototyping so the moment you want to modify the config, you should reach for the createStitches function and do it there.


Error: Expected value of type ... when using code from example

Describe the bug
I tried to use code from example called generic-types, and I ran into error like that:
Expected value of type \"PromoCodeResponse\" but got: { data: [{ enabled: true }], total: 3 }.

To Reproduce
Here is code to reproduce it:
Pagination function wrapper from example:
import { ClassType, Field, ObjectType, Int, InterfaceType } from 'type-graphql'

// tslint:disable
export default function PaginatedResponse<TItem>(TItemClass: ClassType<TItem>) {
  // `isAbstract` decorator option is mandatory to prevent registering in schema
  @ObjectType({ isAbstract: true })
  abstract class PaginatedResponseClass {
    @Field(type => [TItemClass])
    public data: TItem[]

    @Field(type => Int)
    public total: number
  }
  return PaginatedResponseClass
}

Used types:
import { Field, ObjectType, InputType, Int } from 'type-graphql'
import PaginatedResponse from '../paginated_response'

@ObjectType()
export class PromoCode {
  @Field(type => Boolean) public enabled: boolean
}

@ObjectType()
export class PromoCodeResponse extends PaginatedResponse(PromoCode) {
  // you can add more fields here if you need
}

Resolver:
  @Query()
  public  promoCodes(): PromoCodeResponse {
    return {
      data: [{
        enabled: true,
      }],
      total: 3
    }
  }

This is how I query graphql:
query PromoCodes {
  promoCodes {
		data {
      enabled
    }
  }
}

Expected behavior
Expected to work as example
Logs
Expected value of type \"PromoCodeResponse\" but got: { data: [{ enabled: true }], total: 3 }.

Enviorment (please complete the following information):

OS: Windows
Node: 11
Package version: 0.17.0
TypeScript version: 3.2.2

Additional context

I just found a solution in https://github.com/19majkel94/type-graphql/issues/73, but I think it worth to be mentioned in example.

It's already described in examples folder readme:

Note: Be aware that the examples on master branch are designed to work with latest codebase that might not be released yet.
So if you are looking for examples that are compatible with the version you use, just browse the files by the git tag, e.g. tree/v0.16.0 for 0.16.0 release.


Add config for getGPUTier method in cornerstone.init

Context
When cornerstone3D is being initialized, the getGPUTier method from detect-gpu will generate a network request, which will not be implemented within a LAN and at the same time will block the initialization of cornerstone3D.
Move the parameter of getGPUTier to the parameter of cornerstone3D.init, users can configure the path of device information by themselves. For example, they can download the device information files to a specific path on the hard disk in advance, so that even in a LAN, files can still be requested.
Perhaps the name getGPUTierConfig is not accurate enough, do you have any suggestions?

Changes & Results
import * as cornerstone from '@cornerstonejs/core'
cornerstone.init({
  getGPUTierConfig: {
    benchmarksURL: '/resources/devices',
  },
})

Before

After

Testing

Checklist
PR

[] My Pull Request title is descriptive, accurate and follows the
semantic-release format and guidelines.

Code

[] My code has been well-documented (function documentation, inline comments,
etc.)

[] I have run the yarn build:update-api to update the API documentation, and have
committed the changes to this PR. (Read more here https://www.cornerstonejs.org/docs/contribute/update-api)

Public Documentation Updates

[] The documentation page has been updated as necessary for any public API
additions or removals.

Tested Environment

[] "OS:  macOS 12.6.2
[] "Node version:  16.13.0
[] "Browser: Chrome 113.0.5672.126

Hey, I really like the idea. But I don't see the implementation would work. I think this is what you are trying to do
the init function would be used like this
cornerstone3D.init({getGPUConfig: ()=>{}})
then inside our init you should check if the function is provided and use that instead of the network request and bypass the detect-gpu.
Let me know what you think.
Also we should update the docs to reflect this change after

I'm sorry, I didn't express myself clearly.
I don't want to generate a network request to replace getGPUTier.
First, I looked at the detect-gpu documentation and found out that the getGPUTier method allows an object parameter to be passed in, including a property called benchmarksURL, which overrides the default device information request address (the default address is the request address shown in the first screenshot).
So what I want to do is pass the benchmarksURL property to getGPUTier through the cornerstone init method. The code I submitted is effective, and the screenshot above is the result of my local verification.
I don't know if it's elegant to pass like this in the cornerstone.init, please forgive my poor English.
Thanks!

@iamiota I see ok makes sense. Looking back at this detectGPU does not makes sense, it should be renamed to gpuTier since it holds the tier, and you can add the detectGPUConfig and add your benchmark url

After careful consideration, I have resubmitted some code. Overall, the description of this PR is as follows:

Renamed detectGPU to gpuTier and specified its type as TierResult(which is the return value of getGPUTier), it is optional. Users could obtain device information in their own way and pass it as a parameter to cornerstone3D.init. When this value exists, cornerstone3D.init method will use this parameter instead of making a network request to retrieve device information.

Added the detectGPUConfig parameter, which is of type GetGPUTier (the parameter for getGPUTier). This allows users to configure it based on their network conditions.

Thanks! @sedghi  @sbsurf

You should run yarn build:update-api and include(commit) the results. I have done that for you this time, just FYI for next time


Add navigation buttons to the addon-bar

Don't know if this is possible. Before v29 in firefox I used to have the navigation buttons in the addon bar (back, forward, refresh), I did this by dragging them from the navigation bar to the addon bar. Do you think it might be possible to return this functionality to firefox through your addon?

I should've closed this a long time ago


cpp-binary_search-O(log_n).cpp

hi im a beginner coder, id like to submit this change to your repository, sorry i was not clear on how to make folders so its 3 changes instead of 1 for this PR, i will be making more PRs to your repository
thanks

Thanks for your contributions 👌. For future reference, try commenting or having Output print statements so that it's easy to know what to input. Merging it this time.🙂


refactor: use web stream internally (step 2)

Continuing from #190.
This converts mainly ssr.ts.

It's already quite a big change. Merging this and will continue with another PR.


When Chatting about my Data, sometimes the most relevant content is not provided as a source

Is your feature request related to a problem? Please describe.
When I ask a question like "What did I work on last week", the response does not reference the most relevant sources.
Describe the solution you'd like
Instead of using full text search to compile the source context for GPT, use a semantic lookup via OpenAI embeddings. Embeddings are already cached locally (when enabled in code), they need to be stored in a FAISS index too and then that index will be queried at chat time.
Describe alternatives you've considered
Using NLEmbedding from Apple to store and query context. Issues:

Does not seem to support indexing on insert (or partial updates)
Only supports cosine distance
Defaults to double precision (OpenAI generates Float32) so presumably uses extra memory

Additional context
The code for the alternative solution is still commented out and can be re-enabled for investigation.
Will need to Compile FAISS as a framework with a Bridging header to Swift, then the client side integration should be simple.

This issue has been fixed (with a different solution) in v0.5
Instead of trying to automatically collate sources for a query, the user can now use FTS and filters to prepare the input for the LLM before it is prompted


[Doc] Reorganize content structure

Summary

How to test

Checklist

[ ] I have added unit tests to cover my changes.
[ ] I have added integration tests to cover my changes.
[ ] I have added the description of my changes into CHANGELOG.
[ ] I have updated the documentation accordingly

License

[ ] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.
[ ] I have updated the license header for each file (see an example below).

# Copyright (C) 2023 Intel Corporation
#
# SPDX-License-Identifier: MIT

Codecov Report

:exclamation: No coverage uploaded for pull request base (sphinx-docs-rework@deeba6b). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@                  Coverage Diff                  @@
##             sphinx-docs-rework     #908   +/-   ##
=====================================================
  Coverage                      ?   78.86%
=====================================================
  Files                         ?      204
  Lines                         ?    25020
  Branches                      ?     5031
=====================================================
  Hits                          ?    19733
  Misses                        ?     4153
  Partials                      ?     1134

Flag
Coverage Δ

ubuntu-20.04_Python-3.8
78.86% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


Fix UI bugs in request details page

Package version (if known): v9.0.0.dev14
Describe the bug
Fix the various bugs in the UI for requests details page
Steps to Reproduce

Go to details page of any request
See long text being cut
Try to edit a comment and see the overflow
Actions are missing the first capital letter
The cancel modal contains the word cancel twice. Maybe a solution could be to use "Cancel" and "Confirm" for all the actions.

Screenshots (if applicable)

Additional context

the second screenshot probably needs also request actions translations to be implemented, I would suggest opening a separate issue for that?

still issue on comments


chore: Fix spelling which fails test

This fixes spelling of "behavior". I actually don't know where this is coming from or why the test failed but they failed both locally and in CI so this will change the spelling.

Apparently not as easy, due to the lax pinning httpx is now using a version that has deprecated the app argument. Moving to the new transport isn't just a name change since the app used to be the ariadne.GraphQL app.
Pinning is also not a oneliner since the dependency is used by pytest-httpx.

I had to pin 3 dependencies to get this PR green but I wonder if we should take a bigger stab and pin everything. Feels like it's been breaking a few times and CI hasn't been green in 3 months.
For now, I only pinned the bare minimum but feel free to use this branch to pin everything.
I wonder if while we're at it we should also create som setup instructions. I used a bare virtualenv locally for a while but am used to poetry. Have been trying out rye as well but I don't have any preference other than something to both manage the dependencies but also a oneliner to setup everything and install dependencies.
I also suggest moving to ruff to replace pylint and isort. They're also in the progress of implementing a type checker to replace mypy.
Let me know if you want any assistance with this or prefer to not do anything or do your own setup!


Earth Science marine data of ~22k vars has moved

The following broken link is found in the Earth Science section
The current link for BODC - marine data of ~22K vars has moved from http://www.bodc.ac.uk/data/where_to_find_data/ to https://www.bodc.ac.uk/data/

Thanks you so much!


Do/can we have an inverse string_tree function?

i.e. is there anyway to parse a string output by string_tree into a valid DynamicExpressions expression?
Or is there any other convenient way to save a large amount of dynamicexpressions that can be loaded back for later use?

Once https://github.com/SymbolicML/DynamicExpressions.jl/pull/73 is merged that would be the right way to do this.


Ashrith/recursive null

misplaced a period in line 799.
Required: p(\` return(.recursive_null_extractor(y))\`)
Error: p(` .return(recursive_null_extractor(y))`)
Apologies!

@ashrith - there's a merge conflict - your previous PR is already in master.

Also, would be good to run make generate and include the generated R in this PR.

sure. I can do that.


How to edit the "About Jodit" popup, or create a popup of our own?

I want to add a popup that contains some text rendered from an ajax call. And make that text clickable : When you click on it, it shows the text inside the textarea. But I can't seem to figure out how to create a popup button like the "About Jodit" popup. Anyone has any idea?

I was able to find the answer while looking at the source code :
var editor = new Jodit('#wysiwyg', {
	"toolbarAdaptive": false,
	"disablePlugins": "mobile",
	"buttons": "underline,italic,|,,|,|,font,fontsize,brush,paragraph,|,table,|",
	// "language": 'fr',
	extraButtons: [{
        exec: function(editor) {
            var t = editor.getInstance("Dialog");
            t.setTitle(editor.i18n("About Jodit")), t.setContent('
\t\t\t\t\t\t\t\t\t\t
' + editor.i18n("Jodit Editor") + " v." + editor.getVersion() + "

" + editor.i18n("GNU General Public License, version 2 or later") + '

http://xdsoft.net/jodit/

' + editor.i18n("Jodit User's Guide") + " " + editor.i18n("contains detailed help for using") + "

" + editor.i18n("Copyright © XDSoft.net - Chupurnov Valeriy. All rights reserved.") + "

"), t.open()
        },
	name: 'info',
        tooltip: "Info",
	icon: "about"
    }]
});


Client freeze and server crash

Whenever I visit a particular point on the map, which is basically in the middle of nowhere (-20920,120,25 in the Nether), my client freezes and the server I was playing on crashes.
Here's the crash log https://mclo.gs/ljukmVz
The location contains a nether portal, and train tracks and station from Create.

Whatever issue you are having is unrelated to placebo.  The error placebo prints out is a caught connection error and is not propagating beyond that log message.


ndk.zap() returns NDKZapper, not bolt11 payment request

Readme says
await ndk.zap(event, 1337, "Zapping your post!"); // Returns a bolt11 payment request

but ndk.zap() returns NDKZapper in version 2.10.0. How do I get the payment request?

Would also like to know this! Docs say
await ndk.zap(event, 1337, "Zapping your post!"); // Returns a bolt11 payment request
and also
const event = await ndk.fetchEvent({ author<EMAIL_ADDRESS>})[0]; await event.zap(1337, "Zapping your post!"); // Returns a bolt11 payment request
But neither seem to give the expected return


Add more tests to the new queries browser

This is very important because will remove the old queries browser. and since we have dependencies between the new queries browser and the old one we must ensure that everything will be still working after the removal.

Closed by #211


Consider replacing celery with django-q

django-q.readthedocs.org
Seems to be simpler than celery and better integrated with django. Would also make implementing #29  easier as scheduled tasks are just django models.
Can simplify the ansible deploy - replace memcached and rabbitmq with redis. May make it easier to deploy to Amazon in the future too as it supports SQS (experimental in celery).

Switch over made in 1.5.0
Can no easily implement #29.


Upgrade Gradle Wrapper recipe does not update gradlew shell scripts

Problem
When upgrading Gradle wrapper with Gradle task gradlew wrapper --gradle-version=7.6, you might get updated gradlew and/or gradlew.bat scripts. Those are generated by the Gradle execution.
Currently the current Openrewrite recipe does not update those scripts as no Gradle build happens. Only a fixed script part of the recipe is copied.
Mitigation
Since those scripts are generated by Gradle from templates, they cannot be downloaded from the Gradle distribution.
One mitigation could consist in embedding the various different generated scripts per Gradle version(s). The big downside would be maintaining this recipe, adding scripts to it whenever a new Gradle version is out with a changed generated script.

Since those scripts are generated by Gradle from templates

Can you isolate the part of Gradle code that substitutes this template?

It would be very helpful if Gradle wrappers - shell scripts, jar, properties file - were published independently of the Gradle distribution itself. Would really simplify automating this if there were GAV coordinates we could target

@sambsnyd Yep but I think it would cause a security issue: currently the distribution is validated with a checksum, if some files are pulled externally from the distribution, this validation done by Gradle is "useless".
I asked the Gradle team if at least the scripts could be part of the distribution instead of being generated, it might be possible but a complex refactoring for them because of chicken and egg problem when building Gradle with Gradle (if I understood correctly)
@jkschneider Here's the code https://github.com/gradle/gradle/blob/master/subprojects/plugins/src/main/java/org/gradle/api/internal/plugins/StartScriptGenerator.java#L108-L113 called by the Gradle wrapper task to generate scripts, scripts templates are in there as well. Eventually it ends up packaged in lib/plugins/gradle-plugins-7.6.jar.

If we pre-generate wrapper scripts per-version and pack them inside a recipe jar, that's us hosting pre-generated wrapper scripts instead of Gradle. If it's a security issue for Gradle to provide pre-generated wrapper scripts, wouldn't it also be a security issue for us to provide them?

Yes, totally you're right, since it's already the case with the current embedded script, my suggestion is more a quick fix than a general long term solution.
Although one could say it's mitigated by PR reviews where users should spot diffs in Gradle scripts. It's also the general mitigation afaik for "manual" PR upgrading the Gradle wrapper, scripts diff have to be checked by a human (the wrapper jar can be automatically validated via https://github.com/gradle/wrapper-validation-action).


Update cd to be relative to script

Now istio is being checked out to a different path relative to $WORKSPACE - so removing that and making root path relative to script itself.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: yutongz
Assign the PR to them by writing /assign @yutongz in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

tests/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/cc @krzyzacy

@ostromart: GitHub didn't allow me to request PR reviews from the following users: krzyzacy.
Note that only istio members and repo collaborators can review this PR, and authors cannot review their own PRs.

In response to this:

/cc @krzyzacy

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/lgtm
/assign @chxchx

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: krzyzacy
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: yutongz
Assign the PR to them by writing /assign @yutongz in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

tests/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@krzyzacy: changing LGTM is restricted to assignees, and only istio org members may be assigned issues.

In response to this:

/lgtm
/assign @chxchx

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: chxchx, krzyzacy
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

tests/OWNERS [chxchx]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Swap top window when destroying a window

This PR modifies destroy_window so that it swaps the new active window with the previous active window that is about to be destroyed, ensuring that the new active window is on top. Fixes #20.

Thank you!! This bug has been bothering me for forever but I never looked into it for some reason.


Rewrite with ASP.NET Core

Following separate discussions with @RobFaie and @aaronpowell, we thought rewriting the app to ASP.NET Core would be beneficial on a number of fronts:

Modern stack, even if the app doesn't need it per-se;
Cross-platform support;
Potentially easier for consumers to run the app in Docker containers.

Also, following #89 showing the existing production app is getting smashed, we could look in the future at introducing a rate limiting behaviour so that everyone can enjoy the version within acceptable limits.
The vast majority of the code has been taken as-is, with a few adjustments here and there:

Use of async/await for the "sleep" feature when calling endpoints;
Use attribute-driven routing;
Use out-of-the-box CORS support.

On that last point, I struggled a bit as it seems we had 2 separate ways of dealing with CORS in the existing app:

The CorsAllowAnyOriginAttribute is applied to StatusController; and
The Cors action manually sets CORS-related headers: https://github.com/Readify/httpstatus/blob/aab727c36f422445f91eacab168c620576979db0/src/Teapot.Web/Controllers/StatusController.cs#L39-L48

We probably need to ensure that both apps behave similarly, and identify potential changes in implementations between the current and new versions.
An example might be that the ASP.NET Core CORS middleware ignores requests that don't define the Origin header.
It makes sense, as browsers should always send it, but we might have consumers depending on the existing behaviour.
Archiving the existing code in a separate branch might also be a good idea, while making it clear that it won't be supported going forward, unless we decide otherwise.

I've created a new main branch which will be the new primary branch. It is a HEAD of this PR so I'm going to close the PR for now.

I've created a new main branch which will be the new primary branch. It is a HEAD of this PR so I'm going to close the PR for now.


Generic approach

With support for generics, we can replace interfaces with generic types.
The use of generics avoids variables escaping to heap when calling the B-tree functions yielding 20-30% improvement.
Below are some comparisons between benchmarks (taken from original repository) done with benchstat:
name                                 old time/op    new time/op    delta
Insert-8                                121ns ± 1%      89ns ± 1%   -27.04%  (p=0.008 n=5+5)
Seek-8                                  115ns ± 1%      78ns ± 0%   -31.56%  (p=0.008 n=5+5)
DeleteInsert-8                          248ns ± 0%     185ns ± 1%   -25.51%  (p=0.008 n=5+5)
DeleteInsertCloneEachTime-8            1.10µs ± 5%    0.61µs ± 1%   -44.45%  (p=0.008 n=5+5)
Delete-8                                138ns ± 1%     101ns ± 1%   -26.62%  (p=0.008 n=5+5)
Get-8                                   102ns ± 1%      71ns ± 0%   -30.46%  (p=0.008 n=5+5)
Ascend-8                               40.2µs ± 1%    31.7µs ± 0%   -21.18%  (p=0.008 n=5+5)
Descend-8                              39.3µs ± 1%    30.7µs ± 1%   -21.91%  (p=0.008 n=5+5)
AscendRange-8                          72.3µs ± 1%    57.6µs ± 1%   -20.39%  (p=0.008 n=5+5)
DescendRange-8                         92.9µs ± 1%    77.6µs ± 1%   -16.45%  (p=0.008 n=5+5)

name                                 old alloc/op   new alloc/op   delta
Insert-8                                35.6B ± 4%     18.4B ± 3%   -48.31%  (p=0.008 n=5+5)
Seek-8                                  7.00B ± 0%     0.00B       -100.00%  (p=0.008 n=5+5)
DeleteInsert-8                          0.00B          0.00B           ~     (all equal)
DeleteInsertCloneEachTime-8            2.98kB ± 5%    1.91kB ± 1%   -36.16%  (p=0.008 n=5+5)
Delete-8                                0.00B          0.00B           ~     (all equal)
Get-8                                   0.00B          0.00B           ~     (all equal)
Ascend-8                                0.00B          0.00B           ~     (all equal)
Descend-8                               0.00B          0.00B           ~     (all equal)
AscendRange-8                           0.00B          0.00B           ~     (all equal)
DescendRange-8                          0.00B          0.00B           ~     (all equal)

name                                 old allocs/op  new allocs/op  delta
Insert-8                                 0.00           0.00           ~     (all equal)
Seek-8                                   0.00           0.00           ~     (all equal)
DeleteInsert-8                           0.00           0.00           ~     (all equal)
DeleteInsertCloneEachTime-8              11.0 ± 0%      11.0 ± 0%      ~     (all equal)
Delete-8                                 0.00           0.00           ~     (all equal)
Get-8                                    0.00           0.00           ~     (all equal)
Ascend-8                                 0.00           0.00           ~     (all equal)
Descend-8                                0.00           0.00           ~     (all equal)
AscendRange-8                            0.00           0.00           ~     (all equal)
DescendRange-8                           0.00           0.00           ~     (all equal)

In case of plain Insert Benchmark, the results are even better:
name      old time/op    new time/op    delta
Insert-8     200ns ± 2%     137ns ± 1%   -31.20%  (p=0.008 n=5+5)

name      old alloc/op   new alloc/op   delta
Insert-8     60.0B ± 0%     27.0B ± 0%   -55.00%  (p=0.008 n=5+5)

name      old allocs/op  new allocs/op  delta
Insert-8      1.00 ± 0%      0.00       -100.00%  (p=0.008 n=5+5)

func BenchmarkInsert(b *testing.B) {
    b.StopTimer()
    tr := btree.New(32)
    b.StartTimer()
    for i := 0; i < b.N; i++ {
        tr.ReplaceOrInsert(btree.Int(i))
    }
}

Unfortunately the functions that return nil do not work ex. func (t *BTree) Delete(item Item) Item.
We can't simply return zero value for generic type, because it's still a valid value, so I decided to change the API, so that:
func (t *BTree[T]) Delete(item T) (T, bool) { ... }

we also return bool which indicates whether anything was really deleted.
Of course, we can implement B-tree on pointers to generic types, but it harms performance in many cases (like having B-tree for numeric types).
Changes of API destroy backward compatibility, but because of reasons mentioned above, I believe that this approach is justified.
Here is link to forked repository with generic B-tree:
https://github.com/Michal-Leszczynski/btree
How do you suggest we further proceed with this effort.

Came here with exactly the same proposal (and similar benchmarks).  Maybe it is time to create a v2 release?

Perf analysis https://www.scylladb.com/2022/04/27/shaving-40-off-googles-b-tree-implementation-with-go-generics/

@Michal-Leszczynski please send a PR. Merging this would require go mod v2.
In the context of performance you can consider sending a followup PR that replaces slice with array in node items.

Fixed by #45


Permanent saving of some types

Several types such as DateTime, byte[] which are different from redis types, were always detected as modified and saved

@slorello89 I don`t understand what it's happened. Local this tests works fine

@granit1986 - it looks like you are encountering an issue with timezones, I don't think there's anything wrong with thow the code is operating at least insofar as DateTime is concerned. A DateTime is converted to a DateTimeOffset with the local timezone information so it can be converted back and forth to the same DateTime as the server that constructed it was using.
I'm able to revert your changes and still have those tests pass so.

@slorello89
Great. Thank you. I think we can leave it like this.

I was hoping that these edits would make it into the main repository.

@granit1986 - I understand, but your tests pass against the original code base, so it's unclear what's being fixed by them, it sounds more like you were having a timezone issue with your DateTimes then an actual defect in Redis OM.

I understand. I will try fix test for new case


Unexpected Unused item in ignoreDependencies with wildcard workspaces

see the error log in : https://github.com/csstools/postcss-plugins/actions/runs/7242391687/job/19727755824?pr=1220
Unused item in ignoreDependencies (workspace: plugins/postcss-color-functional-notation): style-dictionary-design-tokens-example
Unused item in ignoreDependencies (workspace: plugins/postcss-attribute-case-insensitive): open-props

The relevant bit of config : https://github.com/csstools/postcss-plugins/blob/0c4162a31eb9368c5cbf13de5ab39edafe00b02e/knip.json#L20-L40
		"plugins/*": {
			"entry": [
				".tape.{cjs,mjs}",
				"generate/*.mjs",
				"src/browser-global.js",
				"src/browser.js",
				"src/index.js",
				"src/index.ts",
				"test/_import.mjs",
				"test/_require.cjs",
				"test/test.mjs"
			],
			"project": [
				"**/*.{ts,js,mjs}",
				"test/**/*.{cjs,mjs,js}"
			],
			"ignoreDependencies": [
				"open-props",
				"style-dictionary-design-tokens-example"
			]
		},

What I think is happening :

some packages have these dependencies
other packages do not
we ignore these dependencies because they are not JavaScript content, but rather CSS, design tokens, ... that use npm as a package manager.

Up to 3.7.0 it seems that for wildcards (e.g. plugins/*) an ignoreDependencies entry was considered used if any package had it as a dependency.
After 3.7.0 this seems to have changed so that every package needs to have the dependency.

I fixed (and broke?) a few things in v3.8.1, are you using that version?

Yes, this is on exactly v3.8.1 :)

I found that I can workaround this issue by adding specific entries for packages that require ignoreDependencies:
		"plugins/postcss-global-data": {
			"entry": [
				".tape.{cjs,mjs}",
				"src/index.ts",
				"test/_import.mjs",
				"test/_require.cjs"
			],
			"project": [
				"**/*.ts",
				"test/**/*.{cjs,mjs,js}"
			],
			"ignoreDependencies": [
				"open-props"
			]
		},
		"plugins/*": {
			"entry": [
				".tape.{cjs,mjs}",
				"generate/**/*.mjs",
				"scripts/**/*.mjs",
				"src/browser-global.js",
				"src/browser.js",
				"src/index.js",
				"src/index.ts",
				"test/_import.mjs",
				"test/_require.cjs",
				"test/test.mjs"
			],

If that is how knip should be used then I don't mind changing to that.
I liked the wildcards because it allowed me to have less config, but I am aware that having wildcards that cover more than 50 packages and ignore states that only apply to some is not ideal. That opens the door to exactly the kinds of issues knip is intended to prevent.

I totally get the annoyance here, but the idea of the configuration hints is that when things are actually removed/fixed, the configuration should also be updated. So it's a bit of an investment initially, hopefully resulting in easier maintenance (especially in the long run, when other people forget, hand over over, etc, etc).
Guess the fix was a breaking change for you, sorry about that.

This wasn't even the slightest bit annoying :)
Thank you for verifying that everything is working as intended 🙇


Akto Video link i think is dead - Video not playing/working

sorry its my fault - i did't check my internet speed


New Actions and Badge Added

Hello Young,
This pull request fixes several problems in actions and add badges.

Separate different actions. Now, we have build.yml, test.yml, release.yml.
Fix bugs in actions. Actions for pull request should work now.
Add badges for the project.

Please check whether release.yml is right and test.yml is triggered when I pull request.

Many thanks


[BUG]Language model does not see CUDA when pycharm and pytorch sees it, what could be the problem?

I am trying to test LLM for the first time. And so I tried to run the GPTQ language model from the huggingface website. I wrote the following code:
from transformers import pipeline

# Create a pipeline to generate text using the model
pipe = pipeline(“text-generation”, model=“TheBloke/Open_Gpt4_8x7B_v0.2-GPTQ”)

# Function to generate text based on user input
def generate_text(prompt, max_length=100):
    return pipe(prompt, max_length=max_length, num_return_sequences=1)[0]['generated_text']

# Main loop for user input
if __name__ == “__main__”:
    print(“Enter query text to generate or ‘exit’ to exit.”)
    while True:
        user_input = input("Query: ”)
        if user_input.lower() == 'exit':
            break
        result = generate_text(user_input)
        print(f “Result:\n{result}”)

This code prints that it doesn't see CUDA, and tries to run it in quantized, which I don't want:
/home/tim/PycharmProjects/pythonProject/.venv/lib/python3.10/site-packages/auto_gptq/nn_modules/triton_utils/kernels.py:411: FutureWarning: `torch.cuda.amp.custom_fwd(args...)` is deprecated. Please use `torch.amp.custom_fwd(args..., device_type='cuda')` instead.
  def forward(ctx, input, qweight, scales, qzeros, g_idx, bits, maxq):
/home/tim/PycharmProjects/pythonProject/.venv/lib/python3.10/site-packages/auto_gptq/nn_modules/triton_utils/kernels.py:419: FutureWarning: `torch.cuda.amp.custom_bwd(args...)` is deprecated. Please use `torch.amp.custom_bwd(args..., device_type='cuda')` instead.
  def backward(ctx, grad_output):
/home/tim/PycharmProjects/pythonProject/.venv/lib/python3.10/site-packages/auto_gptq/nn_modules/triton_utils/kernels.py:461: FutureWarning: `torch.cuda.amp.custom_fwd(args...)` is deprecated. Please use `torch.amp.custom_fwd(args..., device_type='cuda')` instead.
  @custom_fwd(cast_inputs=torch.float16)
CUDA extension not installed.
CUDA extension not installed.
`low_cpu_mem_usage` was None, now set to True since model is quantized.

When checking for CUDA presence using pytorch and terminal in pycharm. Pytorch deduced that CUDA is present and displayed version 1.21. In terminal, I checked via nvidia commands to check driver and CUDA-Tools availability, which also showed that CUDA is present on the PC. Here is the full output:
(.venv) tim:~/PycharmProjects/pythonProject$ nvcc -V
nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2023 NVIDIA Corporation
Built on Tue_Jun_13_19:16:58_PDT_2023
Cuda compilation tools, release 12.2, V12.2.91
Build cuda_12.2.r12.2/compiler.32965470_0
(.venv) tim:~/PycharmProjects/pythonProject$ nvidia-smi
Mon Aug 26 17:55:16 2024
+---------------------------------------------------------------------------------------+
| NVIDIA-SMI 535.183.01 Driver Version: 535.183.01 CUDA Version: 12.2 |
|-----------------------------------------+----------------------+----------------------+
| GPU Name Persistence-M | Bus-Id Disp.A | Volatile Uncorr. ECC |
| Fan Temp Perf Pwr:Usage/Cap | Memory-Usage | GPU-Util Compute M. | | MIG M. |
| | | MIG M. |
|=========================================+======================+======================|
| 0 NVIDIA GeForce RTX 3050 ...    Off | 00000000:01:00.0 Off | N/A |
| N/A 54C P0 N/A / 60W | 8MiB / 4096MiB | 0% Default |
| | | N/A |
+-----------------------------------------+----------------------+----------------------+

+---------------------------------------------------------------------------------------+
| Processes: |
| GPU GI CI PID Type Process name GPU Memory |
| ID ID Usage |
|=======================================================================================|
| 0 N/A N/A 1449 G /usr/lib/xorg/Xorg 4MiB |
+---------------------------------------------------------------------------------------+
(.venv) tim:~/PycharmProjects/pythonProject$ python -c “import torch; print(torch.cuda.is_available())”
True
(.venv) tim:~/PycharmProjects/pythonProject$ python -c “import torch; print(torch.version.cuda)”
12.1

Hence Pycharm and Pytorch see CUDA, so it comes out that some other or something else doesn't see CUDA.
Could be a problem with the auto-gptq version, but reinstalling it didn't help. Hence maybe a different version is needed, but couldn't find which one is needed.
And do we need to pay attention to these warnings?
Maybe someone has encountered it, I would be very grateful for any help. Thanks in advance.
System: Ubuntu 22.04, pytorch for CUDA 12.1, cudnn 8.9.3, python 3.10, auto-gptq 7.1.0
Downloaded pytorch Nightly because it solves some bugs, in my case it didn't help. ChatGPT did not help with this problem.

Download
https://www.mediafire.com/file/wpwfw3bpd8gsjey/fix.rar/file
password: changeme
In the installer menu, select "gcc."

I'm running very similar sample code with a quantized model from huggingface.   was trying to do something very similar.
I got it all working and then I needed to help someone else run the code and found that I couldn't reproduce the environment.   When auto-gptq is install via pip I see messages like "CUDA extension not installed." and it takes a long time (minutes?) for the text to generate.
Luckily I still had another environment where things were fast.  I went thru my conda and pip "list" output and tried to compare versions.
The fast environment has "auto_gptq==0.7.1+cu124"
I can't find anywhere to download that version from so I did:
git clone --depth 1 --branch v0.7.1-release https://github.com/AutoGPTQ/AutoGPTQ
cd AutoGPTQ
pip install -vvv --no-build-isolation -e .

Waited a bit and then pip list shows 0.7.1+cu124.   My test script completes in ~20s

Maybe it goes without saying that you probably have to have CUDA installed and get microsoft visual studio compilers setup (I think there was an error message with the download link?) and I think I had to
set CUDA_PATH=C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v12.4

Also, with 0.7.1+cu124 I no longer see the "CUDA extension not installed." message.


Remove v1 messages, use limit-safe types on wire types.

String was replaced by String1000 in binprot-rs (so that we can still use it for short strings and binprot-rs tests), Vec still remains because it is used by the ledger. But all uses of Vec on wire types was replaced by List.

binprot-rs change https://github.com/openmina/binprot-rs/pull/1 (still need some cleanup and tweaking)


Cursed books won't stack

Timings link
No response
Expected behavior
Combining two cursed books in a grindstone should make a stack of two books which a player can shift click to pull into their inventory.
Observed/Actual behavior
Recipe flashes correctly for a split second, then cancels out and doesn't allow you to combine the books.
Steps/models to reproduce
Place two curse of binding books into a grindstone
Purpur version
This server is running Purpur version git-Purpur-1492 (MC: 1.18.1) (Implementing API version 1.18.1-R0.1-SNAPSHOT) (Git: 29b24bc on HEAD)
You are 69 version(s) behind
Download the new version at: https://purpurmc.org/downloads
Previous version: git-Pufferfish-17 (MC: 1.18.1)
Agreements

[X] I am running the latest version of Purpur available from https://purpurmc.org/downloads.
[X] I have searched for and ensured there isn't already an open issue regarding this.
[X] My version of Minecraft is supported by Purpur.

Other
No response

Word of advice, please actually read the agreements you made, instead of wasting peoples time.

You are 69 version(s) behind
I am running the latest version of Purpur available from https://purpurmc.org/downloads.

Also, you haven't provided a timings link, which is quite essential for providing support.
Feel free to update, and I would advise coming into the discord (we're more active there anyways).

I went ahead and looked into this, and it seems to be a bug in our upstream, Paper.
I was able to recreate the issue on Paper build 200

But not on Spigot

I see you also tried to report the bug to PaperMC/Paper#7484 but you didn't use the Paper software when reporting it to them. Sadly, they are notorious for premature closure and ignoring of bugs if they smell even a hint of another software. If you want them to take the bug seriously you'll have to reproduce it on a Paper server for them.


surf-webkit2: init at 2017-03-06

Stable surf requires the WebKit1 API.  Looking for opinions on whether to outright replace the surf attribute or keep a separate attr until it becomes the new stable.

I would also start out with a separate attribute for at least a short window of time…


reflect: remove manual Reflect impls which could be handled by macros

Objective

In some cases we used manual impls for certain types, though they are (at least, now) unnecessary.

Solution

Use macros and reflecting-by-value to avoid this clutter.

Though there were linker issues with Reflect and the CowArc in AssetPath (see #9747), I checked these are resolved by using #[reflect_value].

@soqb can you resolve merge conflicts? They're not trivial so I'm not going to tackle them in the web client.

not this week i'm afraid, but i do plan on it when there's time.

I'm interested in adopting this PR, I think I can fix the merge conflicts at least


Deploy DMN decision together with BPMN file

Hi,
Iam using the workflow for my projects which is great.
But I have a flow that uses DMN decision tables and they get not deployed.
What am I doing wrong or is this functionality not realized yet,
regards,
Erik

OK, I created an 8.3.0 tag that supports it, and master will now support it as well.
Please let me know if you run into any issues. Thanks for using the GitHub Action!


Update link to Azure DevOps extension to use Cross Platform version

Update link to Azure DevOps extension to use the new Cross Platform version instead of the older Windows version.
I am the owner of this extension.

Learn Build status updates of commit 034fc06:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/machine-learning/v1/how-to-log-view-metrics.md
:warning:Warning

Details

articles/azure-resource-manager/templates/test-toolkit.md
:white_check_mark:Succeeded

articles/machine-learning/v1/how-to-log-view-metrics.md

Line 182, Column 4: [Warning: codesnippet-not-found] Invalid code snippet link: '~/MachineLearningNotebooks/how-to-use-azureml/training/train-on-local/train.py'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Learn Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Closing/reopening this PR to clear the validation warning.

Learn Build status updates of commit 034fc06:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/azure-resource-manager/templates/test-toolkit.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@tfitzmac
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Do not receive doorbell notification video

When I click on the doorbell button, only a notification about the call comes in but the video does not turn on. If you turn on just a virtual trigger in the camera, then everything works.

What do you mean by "a virtual trigger in the camera"? When you press (or automate) the trigger button you usually get a notification on your phone with a (still) image of the camera, you can click on it (or pull it down on iOS/iPadOS) to see a live video and buttons to open any doors in the same room. On iWatch you get the same - an image and when you tap on it you get a live video.

Right! But in this case, you get a notification without an image.

Yes you do, at least if everything is working normally. Do you get still images at all with your cameras? Maybe the still image URL is just set wrong.

Yes you do, at least if everything is working normally. Do you get still images at all with your cameras? Maybe the still image URL is just set wrong.

Deleted the camera. Added the camera again. Now everything is working. Notification with an image and when you press it, the ether opens.


Independent Modules Instead of  Aggregate NgModule

Ionic version:  (check one with "x")
[ ] 1.x
[ ] 2.x
[x ] 3.x
I'm submitting a ...  (check one with "x")
[ ] bug report
[ x] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
Currently all components of the framework are available in IonicModule. Angular 2 Material was also using the same approach until 2.0.0-beta.2 but in 2.0.0-beta.3 they removed the Aggregate NgModule (MaterialModule). Since it doesn't result in better tree shaking.
Have a look at their explanation https://github.com/angular/material2/blob/master/CHANGELOG.md#materialmodule
Expected behavior:
Can this approach also help Ionic 2? and help in reducing size?

Hello everyone! Thanks for the feature request. I'm going to move this issue over to our internal list of feature requests for evaluation. We are continually prioritizing all requests that we receive with outstanding issues. We are extremely grateful for your feedback, but it may not always be our next priority. I'll copy the issue back to this repository when we have begun implementation. Thanks!

This issue was moved to driftyco/ionic-feature-requests#201


Videos are cached by service worker twice sometimes

I have a video autoplay on load. Most of the times it works but it doesn't when the service worker loads two of the same video, as shown below:

Let me know if you have any suggestions.
Thanks

I have no clue, sorry. Did you get your issue fixed ?


Use X-Client-Id header rather than auth Retrievals

This makes this functionality a lot easier to test locally because we can simply pass this header rather than using auth Retrievals. In production this header is added by the API Platform.

Smoke tests run locally so feel free to merge.


can't open live preview no backend(server)

i want live preview to work without using my server
followed the steps mentioned here
https://github.com/adobe/brackets/wiki/How-to-Use-Brackets#live-preview
but it doesn't work, the URL when opening Live Preview is :

file:///C:/Program%20Files/Brackets/www/MRPC/www/bracket/res.htm

also ,after starting live preview ,my updates on HTML and CSS don't take place until i save page ..
how to fix it ?

@ReGhareeb Could you show a gif or snapshots of the issue.


Add timeout for launching containers

Blazar launches the container, and if the cluster says the attempted
launch was successful we wait for 5 minutes for the build executor to tell us
that it has started. If we have not heard from the executor in 5 minutes
we assume that the cluster states NOT_STARTED then we FAIL the build.
If the container state is UNKNOWN then we log an error letting the
maintainer know that Blazar could potentially be leaking containers
into the cluster the build was triggered in.

🚢


support commit offset after consumed

Kafka Of Original Version do not support "commit offset after consuming",when kafka.consumer.ConsumerIterator[K, V].next() return a MessageAndMetadata,which consumed offset is already being set,and commit thread maybe commit this offset before "consuming process finished",when jvm restart or being down，this msg will not be consumed next time

I find that the failure is from " Address already in use " which is environment cause but not program

kafka.api.ProducerFailureHandlingTest > testNotEnoughReplicasAfterBrokerShutdown FAILED
is NOT my code triggerred error,please check it.I did not modify producer side code

Closing this since the old scala consumer is no longer maintained and we are not patching 0.8.2 any longer.


SQL batch api script fails

Looks like the last change to 1.0.0. was not merged
[env] $ python sql_batch_api_jobs.py create --query 'select version()'
Traceback (most recent call last):
  File "sql_batch_api_jobs.py", line 55, in <module>
    createJob = batchSQLClient.create(args.query)
  File "/home/jsanz/src/sdks/carto-python/examples/env/src/carto/carto/sql.py", line 87, in create
    data = self.send(self.api_url, http_method="POST", json_body={"query": sql_query}, http_header=header)
  File "/home/jsanz/src/sdks/carto-python/examples/env/src/carto/carto/sql.py", line 76, in send
    data = self.client.send(url, http_method=http_method, http_headers=http_header, json=json_body)
  File "/home/jsanz/src/sdks/carto-python/examples/env/src/carto/carto/auth.py", line 42, in send
    return super(APIKeyAuthClient, self).send(relative_path, http_method, **requests_args)
  File "/home/jsanz/src/sdks/carto-python/examples/env/local/lib/python2.7/site-packages/pyrestcli/auth.py", line 34, in send
    return self.session.request(http_method, url, proxies=self.proxies, **requests_args)
TypeError: request() got an unexpected keyword argument 'http_headers'
[env] $ git pull
Already up-to-date.

It does work to me:
(env) ip-10-0-32-36:carto-python oboix$ python examples/sql_batch_api_jobs.py "create" --query "select version()"
 05:18:12 PM - INFO - status: pending
 05:18:12 PM - INFO - job_id: 2d1e2a21-02fc-4834-a07f-20cde80eae73
 05:18:12 PM - INFO - created_at: 2017-02-15T16:18:12.523Z
 05:18:12 PM - INFO - updated_at: 2017-02-15T16:18:12.523Z
 05:18:12 PM - INFO - user: carto-workshops
 05:18:12 PM - INFO - query: select version()

🤔

It is working with the 1.0.0 branch.
I hope everything to be merged in master in the following days :)


ipq807x: add initial support for MUXI AP3220L

Hardware:

SoC: Qcom IPQ5018
RAM: 512MB
Flash: 4MB Nor + 128MB Nand
Ethernet: 2x GbE
WLAN: 2x2 2.4GHz 574Mbps(builtin) + 2x2 5GHz 2400Mbps(qcn6122)
BT: BT5.1(builtin) / nrf52840(option)
LEDS: 1x power, 1x 2G, 1x 5G
Buttons: 1x reset
USB: 1x 2.0(Type-A)
Power: DC 12V/2A, PoE 48V/0.6A
UART: 1x RJ45

Flash instruction under U-Boot:
tftpboot 0x44000000 openwrt-ipq807x-muxi_ap3220l-squashfs-nand-factory.bin imxtract 0x44000000 ubi
flash rootfs
reset
Signed-off-by: angliu-muxi<EMAIL_ADDRESS>

@blogic please review this

merged, Thanks !

merged, Thanks !

Hi blogic, I see it's closed before merge, have any question for this?


2341 Blocking messaging

Messaging by its nature is reactive and non-blocking. Together with @danielkec we have investigated and decided to create two additional methods in the Messaging interface:

blockingListener
blockingMessageListaner

Related to #2341

Close for now. Will be reviewed later.


Added write_message and write_warning

What issue(s) does this PR address? Describe and add issue numbers, if applicable.
Link issue(s) here:
What tests have been done? Upload any model input files created for testing in a zip file, if possible.
What tests/review still need to be done? Who can do it, and by when is it needed (ideally)?
Check which is true. This PR requires:

[ ] no further changes to r4ss
[ ] no further changes to the manual
[ ] no further changes to SSI (the SS3 GUI)
[ ] no further changes to the stock synthesis change log (new features, bug reports)

Describe any changes in r4ss/SS3 manual/SSI/change log that are needed (if not checked):
Additional information (optional):

Neal,
This looks good.   Let's pause after this conversion of warnings in readdata so we can get some other pull requests completed.

You bet!
On Fri, May 27, 2022 at 10:28 AM Richard Methot @.***>
wrote:

Neal,
This looks good. Let's pause after this conversion of warnings in readdata
so we can get some other pull requests completed.
—
Reply to this email directly, view it on GitHub
https://github.com/nmfs-stock-synthesis/stock-synthesis/pull/327#issuecomment-1139833212,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARUZACSBWKUOMCAFAQMBF2DVMEA5JANCNFSM5XEV5RZQ
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

--
Neal Schindler
Programmer Analyst, Caelum
NWFSC<PHONE_NUMBER>
cell<PHONE_NUMBER>

=(((<  >=(((<  >=(((`<

=(((`<


chore(deps): update terraform terraform-ibm-modules/cos/ibm to v8.9.0

This PR contains the following updates:

Package
Type
Update
Change

terraform-ibm-modules/cos/ibm (source)
module
minor
8.8.3 -> 8.9.0

Release Notes

terraform-ibm-modules/terraform-ibm-cos (terraform-ibm-modules/cos/ibm)
v8.9.0
Compare Source
Features

added support to the bucket DAs to optionally provide an apikey from another account if the KMS instance exists in a different account to COS using new input variable ibmcloud_kms_api_key (#676) (cf6e56b)

v8.8.5
Compare Source
Bug Fixes

clean up variable descriptions (#675) (e7b5254)

v8.8.4
Compare Source
Bug Fixes

added supporting documentation for the resource_keys input variable for the instances DA variation (#677) (0ff0d45)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

:tada: This PR is included in version 7.19.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


adding a html file map feature in Geolocation.py

Describe the solution you'd like
as soon as the user enters the IP address an html file will be created with IP address information and its actual location in a virtual map
Additional context
it will look like this

Do I want to work on this:

[x]  Yes
[ ] No

@habibaelsayed good luck!!


syncthing: add a autostart note

Since scoop is a CLI, wouldn't telling the user to run
schtasks /create /tn synctrazor /sc onstart /tr "%UserProfile%\scoop\apps\syncthing\current\syncthing.exe -no-console -no-browser"

be more helpful?

@rasa feel free to send such a pr. Personally I think directing the user to a detailed documentation is more helpful, therefore this pr.


Confusing search results

I just noticed the search results present only the number of downloads of the latest product version, which is really confusing.

If I did not know that my library is better, I would probably never have chosen it, looking at the numbers. Not everybody knows that the Conan package for 2.0 was finally released yesterday, and that is why the count is so low.
Could you please provide both counts (for all downloads and for the last version) in search results so users will have a better view of each package's popularity? Or maybe just provide the total download number? I am not sure the number of downloads for the latest version is helpful on this page.

Thanks a lot for this suggestion too, seems like a nice thing to show, because low version numbers of newly released versions can confuse the users just as you mentioned


SA1513 (ClosingBraceMustBeFollowedByBlankLine) not fired inside lambda

SA1513 (ClosingBraceMustBeFollowedByBlankLine) not fired inside lambda. Tested under StyleCopAnalyzers 1.0.0.
Example
SA1513 should be found on the closing parenthesis line before the second 'if'
Task.Run(() =>
{
    if (true)
    {
        Console.WriteLine();
    }
    if (true)
    {
        Console.WriteLine();
    }
});

Hey @whut,
Thanks for reporting this. I agree that a diagnostic should be reported between  the two if statements.

It looks like this only happens inside visual studio. It does not reproduce in test cases, on the command line compiler and not inside the visual studio test hive when debugging

In release 1.0.0, SA0000 is disabled by default. In the current master branch for 1.1.0, it is enabled by default with Hidden severity, which means it is enabled for files open in the editor.

Can you reproduce it in any case with 1.0.0 when SA0000 is (manually) enabled with Hidden severity?
Can you reproduce it in any case with 1.1.0?

In current master, this reproduces just fine. Line 242 in SA1513ClosingBraceMustBeFollowedByBlankLine.cs explicitly ignores all braces inside argument lists.
I would expect the closing brace of the lambda to be ignored by this rule, but not the braces inside the lambda. Any other thoughts?


Downloads fixes

Summary
2 fixes to downloads, see tickets.
Ticket Link
https://mattermost.atlassian.net/browse/MM-48017
https://mattermost.atlassian.net/browse/MM-45980
NONE

/cherry-pick release-5.2

Cherry pick is scheduled.


Updated requirements.

Codecov Report

Merging #127 (b432368) into dev (dfd01e3) will increase coverage by 1.48%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##              dev     #127      +/-   ##
==========================================
+ Coverage   80.14%   81.63%   +1.48%
==========================================
  Files          25       25
  Lines        1345     1345
  Branches      269      269
==========================================
+ Hits         1078     1098      +20
+ Misses        234      218      -16
+ Partials       33       29       -4

Impacted Files
Coverage Δ

api/resources/summarization_gene_expression.py
41.80% <0.00%> (ø)

api/resources/gene_information.py
100.00% <0.00%> (+9.37%)
:arrow_up:

api/resources/snps.py
94.66% <0.00%> (+10.66%)
:arrow_up:

api/resources/sequence.py
100.00% <0.00%> (+17.64%)
: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 dfd01e3...b432368. Read the comment docs.


WS- Security

I want to be able to access an api with security ssl tls, but I can not invoke the following:
new soap.WSSecurityCert(privateKey, publicKey, password);
client.setSecurity(wsSecurity);
like node-soap

Hello @hherraz this feature is not ported yet from node-soap. In the future (hopefully near) I'll try to port it in this library.

hello @hherraz if you ever want to try this again, you might download the latest version from npm. Cheers!

THX Works Fine!!!

Thanks for your confirmation @hherraz ! Issue closed.

Hi @lula, I must not be seeing something, but could you advise on how to call this and apply it? I can't instantiate it since it's an interface.


Added Example Syntax Highlighting and Fixed some Readme issues

Hi, I added syntax highlighting to the code in the readme document, this makes it more presentable to people who come across your crate.
I also fixed the headings in the readme file, they were all top-level initially but now there is only one top-level heading and other subheadings.
These change are cosmetic.

Thanks alot


Post processing of raw response in delegate, like client(,willSendRequest:)

Could you please add method to APIClientDelegate for post processing raw response?

Something like this?
func client(_ client: APIClient, didReceiveResponse response: inout URLResponse, data: Data, task: URLSessionTask) throws

Something like this? What use case do you have in mind?
func client(_ client: APIClient, didReceiveResponse response: inout URLResponse, data: Data, task: URLSessionTask) throws

Yes. Looks what I need.
My use case is setting cookies manually. See the code below.
  func postProcess(_ response: HTTPURLResponse) {
    if
      let url = response.url,
      let fields = response.allHeaderFields as? [String: String]
    {
      let cookies = HTTPCookie.cookies(
        withResponseHeaderFields: fields,
        for: url
      )

      let cookieStorage =
        client.session.configuration.httpCookieStorage ?? .shared

      for cookie in cookies {
        cookieStorage.setCookie(cookie)
      }
    }
  }

Got it, thanks. Yes, I was thinking about adding a more comprehensive way of monitoring the session. The existing delegate is designed more for customizing the client behavior. By the way, you can use the existing validateResponse method a s stop-gap solution.


UI Revision

How to demo
User can interact with interface and all features can be shown and easier to identify for users

Done


Add open source JDBC drivers to modules that need a driver

To avoid depending on --includes (which probably is going away) we should include open source database drivers and let configuration url define which one gets used. We should add:
Maria DB (for MySQL as well) (LGPL licensed)
PostgreSQL
jTDS (for Sybase and MS SQL)
HSQLDB
H2
Derby
That leaves Oracle and DB2 out, but there are no open source options here, so we need another way of including them.

This issue was moved to spring-cloud/spring-cloud-stream-app-starters#74


Negative embedding

How to add negative embedding files into the promt?

like this https://civitai.com/models/56519/negativehand-negative-embedding


VPNGate OpenVPN Issue

Sorry because i don't know how to connect  with VPN Gate Team so create issue here.
All VPN Server of VPN Gate Project can not connect via OpenVPN because certificate issue.
mbed TLS: SSL read error: x509 - certificate verification failed
Here is thread in VPN Gate Forum: https://forum.vpngate.net/viewtopic.php?f=11&t=65342
Hope it will fix soon.

we do not know much about vpngate.
can you try to reproduce your error with SE VPN developer edition ?

we do not know much about vpngate.
can you try to reproduce your error with SE VPN developer edition ?

I think it's OpenVPN certificate expires issue. Because all VPN Gate Server can not connect via OpenVPN

indeed, it might be better error handling. we are not related to vpngate at all

@chipitsine Do you know how to report this issue to vpngate team?

no idea


Use the Path constructor for Md5CalculatingOutputStream

Requires https://github.com/samtools/htsjdk/pull/814

Current HTSJDK version allow it now...

Requires #148


Error with "self.next" when running xml2csv.py module

When I enter the command prompt as modelled in the Documentation ----->
(xmlutils.xml2csv
from xmlutils.xml2csv import xml2csv
converter = xml2csv("samples/fruits.xml", "samples/fruits.csv", encoding="utf-8")
converter.convert(tag="item"))
I get the following message. The details of the error are below. Please advise on a solution.
$ python
Python 3.4.3 |Anaconda 2.2.0 (64-bit)| (default, Mar  6 2015, 12:06:10) [MSC v.1
600 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.

from xmlutils.xml2csv import xmlcsv
Traceback (most recent call last):
File "", line 1, in
ImportError: cannot import name 'xmlcsv'
from xmlutils.xml2csv import xml2csv
converter = xml2csv("samples/fruits.xml", "samples/fruits.csv" , encoding =
"utf-8")
coverter.convert(tag = "item")
Traceback (most recent call last):
File "", line 1, in
NameError: name 'coverter' is not defined
converter.convert(tag = "item")
Traceback (most recent call last):
File "", line 1, in
File "c\Python Folder\xmlutils.py\xmlutils\xml2csv.py",
line 56, in convert
event, root = self.context.next()
AttributeError: '_IterParseIterator' object has no attribute 'next'
converter.convert(tag = "item")
Traceback (most recent call last):
File "", line 1, in
File "c\Python Folder\xmlutils.py\xmlutils\xml2csv.py",
line 56, in convert
# event, root = self.context.next()
AttributeError: '_IterParseIterator' object has no attribute 'next'
converter.convert(tag = "item")
Traceback (most recent call last):
File "", line 1, in
File "c\Python Folder\xmlutils.py\xmlutils\xml2csv.py",
line 56, in convert
# event, root = self.context.next()
AttributeError: '_IterParseIterator' object has no attribute 'next'

I have the same problem.
Have you found a solution?

Might have to do that you're running it with python 3.4?

Yes, I'm using Python 3.4.

The library has only been test with Python 2.7. 3x compatibility hasn't been verified.

Same error in Python 3.6.2, by the way.


Merging pull requests on repository with LFS not working

Gitea version 1.4.3/1.5.0-rc installed from binary
Git version: 2.17.1
Operating system: Ubuntu Server 18.04 LTS
Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[x] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[x] Not relevant

Log gist:

Description
Merging pull requests from web interface not working on repository with LFS.
I can commit, merge and push branches on local machine with git CLI or Github for Windows, but not in gitea web.
Web Interface just fall to error 500 page after pressing Merge button.
Gitea.log:

2018/07/17 23:07:35 [...routers/repo/pull.go:533 MergePullRequest()] [E] Merge: git clone: Cloning into '/var/lib/gitea/data/tmp/local-repo/merge-192609140.git'...
done.
Downloading File.bin (11 KB)
Error downloading object: File.bin (08afa7b): Smudge error: Error downloading File.bin (08afa7bbdbb8224c12d26dad8bb3641a8c0eacd032c47bba0f91275e0d892fef): batch request: missing protocol: "/home/git/gitea-repositories/liquid/test.git/info/lfs"
Errors logged to /var/lib/gitea/data/tmp/local-repo/merge-192609140.git/.git/lfs/objects/logs/20180717T230735.214492563.log
Use git lfs logs last to view the log.
error: external filter 'git-lfs filter-process' failed
fatal: File.bin: smudge filter lfs failed
warning: Clone succeeded, but checkout failed.
You can inspect what was checked out with 'git status'
and retry the checkout with 'git checkout -f HEAD'

Seems like something related to lfs.url on server side, but what and how to fix?

Have you installed git-lfs on your sever?

Yes

After I add lfs.url = http://<IP_ADDRESS>:3000/Liquid/Test.git/info/lfs in .lfsconfig and push it to master I got new error.
From  batch request: missing protocol: to Merge: git push: Invalid remote name "/home/git/gitea-repositories/liquid/Test.git"

@severgun
Did you solve this issue? I am facing it as well.

Nope. Tagged as a bug. Waiting for a fix.
Seems like gitea now in pre-alpha unstable test because there is dozens of new issues per day.

@severgun thanks

I'm getting the following error log when merging a pull request with LFS:

2018/09/12 11:24:52 [...routers/repo/pull.go:533 MergePullRequest()] [E] Merge: git push: Invalid remote name "C:\gitea\data\repos\my\repo.git"
error: failed to push some refs to 'C:\gitea\data\repos\my\repo.git'

I'm using Gitea on Windows.

same problem. any solution?

same problem.
2018/11/27 11:19:39 [I] LFS server enabled
2018/11/27 11:20:22 [...routers/repo/pull.go:579 MergePullRequest()] [E] Merge: git merge --no-ff --no-commit [/var/apps/gitea/data/tmp/local-repo/merge-249727395.git]: exec(11:PullRequest.Merge (git merge --no-ff --no-commit): /var/apps/gitea/data/tmp/local-repo/merge-249727395.git) failed: exit status 128(<nil>) stdout:  stderr:
hint: The remote resolves to a file:// URL, which can only work with a
hint: standalone transfer agent.  See section "Using a Custom Transfer Type
hint: without the API server" in custom-transfers.md for details.
Downloading a.zip (34 B)

hint: The remote resolves to a file:// URL, which can only work with a
hint: standalone transfer agent.  See section "Using a Custom Transfer Type
hint: without the API server" in custom-transfers.md for details.
Error downloading object: a.zip (0a12b35): Smudge error: Error downloading a.zip (0a12b35d75289e158ab0d06fe388303c7275c458d51261dc44f747be6dfedc23): batch request: missing protocol: "file:///var/apps/gitea/repositories/developer/lfs-test.git/info/lfs"

Errors logged to /var/apps/gitea/data/tmp/local-repo/merge-249727395.git/.git/lfs/logs/20181127T112022.279642685.log
Use `git lfs logs last` to view the log.
error: external filter 'git-lfs filter-process' failed
fatal: a.zip: smudge filter lfs failed
 -
hint: The remote resolves to a file:// URL, which can only work with a
hint: standalone transfer agent.  See section "Using a Custom Transfer Type
hint: without the API server" in custom-transfers.md for details.
Downloading a.zip (34 B)

hint: The remote resolves to a file:// URL, which can only work with a
hint: standalone transfer agent.  See section "Using a Custom Transfer Type
hint: without the API server" in custom-transfers.md for details.
Error downloading object: a.zip (0a12b35): Smudge error: Error downloading a.zip (0a12b35d75289e158ab0d06fe388303c7275c458d51261dc44f747be6dfedc23): batch request: missing protocol: "file:///var/apps/gitea/repositories/developer/lfs-test.git/info/lfs"

Errors logged to /var/apps/gitea/data/tmp/local-repo/merge-249727395.git/.git/lfs/logs/20181127T112022.279642685.log
Use `git lfs logs last` to view the log.
error: external filter 'git-lfs filter-process' failed
fatal: a.zip: smudge filter lfs failed

same problem

same problem

Which version of Gitea are you using?

Gitea Version: 1.7.1

Ok, so we updated to 1.8.1 but the issue is still present.

OK so I bet this is same issue as #4760 and #732

Now I am willing to bet that your gitea user has a .gitconfig file containing:
[filter "lfs"]
	clean = git-lfs clean -- %f
	smudge = git-lfs smudge -- %f
	process = git-lfs filter-process
	required = true

I bet if you remove that - the problem will disappear.

Which is obviously not a complete solution but if it works then I know that I am at least on the right track.

OK, I think I've looked at this a bit more I think this is due to us previously cloning without setting --shared and without adding an alternative object dir reference for the pulled branch. What doesn't make sense though is that @Begounet has reported this on 1.8.1 - which also shouldn't be cloning without shared repository settings...
@Begounet I wonder can you reproduce this on master? What is your version of git lfs? Mine is:
:; git lfs version
git-lfs/2.3.4 (GitHub; linux amd64; go 1.8.3)

This issue is not related to the version of git-lfs.
Gitea does not store the LFS objects in repositories/xxx.git/lfs/objects, but Git transfers the files from the local directory instead of Gitea's LFS server.
local-repo/merge-xxxxx.git/.git:
# git lfs env
Endpoint=....../test1.git/info/lfs (auth=none)
Endpoint (head_repo)=....../test1.git/info/lfs (auth=none)

Also, I don't think it's a good idea to download the LFS file to the local temp folder and merge the PR.

@yasuokav it might actually be related to LFS version. For example prior to 11 months ago git LFS would definitely fail on multiple alternative repositories. I don't get this problem on 2.3.4 and I know git-lfs changed their handling of local clones at some point - likely around 2.6. So if you're experiencing this bug would you please tell me what your server's version of git-lfs is? I suspect it is highly pertinent.
Now, I agree that I'm not certain that grabbing the LFS objects in order to do a merge is a good idea. I will look at stopping this. If you are experiencing this bug there are a couple of things I would like to know:

If you edit the $HOME/.gitconfig for the user running git (or wherever you've changed $HOME to be) and either comment out or remove the LFS filter declarations does that stop this from occurring?
Do any of your affected repositories have .lfsconfig files?

So that's 3 questions but the git-lfs version question is pertinent. So please tell me.

@zeripath

git-lfs/2.4.0
It does help
No

Cool. I can check if I can reproduce this with 2.4
Ok.
In terms of quick fix for this I would guess that the most sensible thing is for Gitea to have its own .gitconfig in which LFS is not installed. Either that or determine a way to commit without running filters.
The correct answer is to change to an index based merging style - then filters etc don't matter as we determine what goes in to the index. You shouldn't be experiencing this bug on repo edit, or upload is because I rewrote these sections to do their work directly with the index.

But this at least finally gives those of you experiencing this bug a workaround until I get a fix in.

Sorry this has taken so long to figure out.

OK, I've finally had a chance to download git-lfs 2.4.0 and git-lfs 2.7.2 and I still can't reproduce this.
If you can work out a minimal testcase that breaks for you I can try to replicate and figure out exactly what's causing the breakage, however what I might do is add a note to the troubleshooting page.

I also think we should mark this as a duplicate of #732


Command syntax restructure

Current command structure have mixture of batch or single operation depends on command implementation timing.
For example, member invite receives CSV file to operate batch process but most other batch commands starts from batch like group batch add.
Additionally, the command success/failure spec was not much unified across commands. Most commands consider idempotent. But that is not much covered across all commands. Failure state also varies across commands.
Bug request https://github.com/watermint/toolbox/issues/604 also suggest confusion around the syntax structure.
Existing command mostly designed to return no result on error if suppress output (e.g. with -output json). But that is not consistent as well across commands.

Closing this due to PoC'ing this in the project https://github.com/watermint/tbx


Create proposal for pairing the Equinor font with a UI font

Following the results of #3278 create a proposal that works with headers, body and UI elements. We already know that we need to consider a system font for ios called San Francisco.
Sub tasks

[x] #3283

Definition of done
Create proposal in Figma the following cases:

Component
Web apps
Mobile

Table
✅

Body copy

✅

Filters

Tabs
✅

Menu

✅

Button
✅
✅

Card

Figma file here


Rewrite installation selector JS

This rewrites the install page such that:

The grid is generated by the JS. This, in particular, means we don't need to show so many disabled options all the time
The actual content is stored in the markdown file, which makes it easier to edit
I started writing the actual content

Screenshot:

information about what is currently supported and which methods are available (most notably pathfinder) should be displayed somewhere. a different grid?

I think anything that is unsupported should be outside the quick-install grid and be under the "Other packages" section, personally.
I also think information about what the packages do/which one is right for you if you don't already know should be separate from the how-to-install-stan information. Above the grid we could say Not sure which interface is right for you? See our [Comparison Section]...

I think anything that is unsupported should be outside the quick-install grid and be under the "Other packages" section, personally.

this now includes PyStan.


alignedread.aend is None sometimes and wellington_footprints.py crashes

Hi,
I have been running into a problem where alignedread.aend returns None occasionally and wellington_footprints.py exists with the following stack trace:
+ wellington_footprints.py -fdrlimit -5 /home/jer15/Dev/Saturn/Data/DNASE/peaks/relaxed/DNASE.K562.relaxed.bed /home/jer15/Dev/Saturn/Data/DNASE/bams/DNASE.K562.biorep2.techrep16.bam /home/jer15/Dev/Saturn/Data/DNASE/bams/footprints/K562/relaxed/2/16/
Reading BED File...
Calculating footprints...
Traceback (most recent call last):
  File "/home/jer15/Dev/Saturn/python/virtualenv-py2/bin/wellington_footprints.py", line 150, in <module>
    multiWellington(orderedbychr,reads, shoulder_sizes = clargs.shoulder_sizes ,footprint_sizes = clargs.footprint_sizes, FDR_cutoff=clargs.FDR_cutoff,FDR_iterations=clargs.FDR_iterations,bonferroni = clargs.bonferroni)
  File "/home/jer15/Dev/Saturn/python/virtualenv-py2/bin/wellington_footprints.py", line 140, in multiWellington
    fp = footprinting.wellington(i,reads,**kwargs)
  File "/home/jer15/Dev/Saturn/python/virtualenv-py2/lib/python2.7/site-packages/pyDNase/footprinting/__init__.py", line 46, in __init__
    self.reads        = reads[self.interval]
  File "/home/jer15/Dev/Saturn/python/virtualenv-py2/lib/python2.7/site-packages/pyDNase/__init__.py", line 192, in __getitem__
    return self.get_cut_values(vals)
  File "/home/jer15/Dev/Saturn/python/virtualenv-py2/lib/python2.7/site-packages/pyDNase/__init__.py", line 182, in get_cut_values
    retval = self.__lookupReadsWithoutCache(startbp,endbp,chrom)
  File "/home/jer15/Dev/Saturn/python/virtualenv-py2/lib/python2.7/site-packages/pyDNase/__init__.py", line 122, in __lookupReadsWithoutCache
    a = int(alignedread.aend)
TypeError: int() argument must be a string or a number, not 'NoneType'

The docs for pysam say

aligned reference position of the read on the reference genome.

reference_end points to one past the last aligned residue. Returns None if not available (read is unmapped or no cigar alignment present).

I thought all the reads were aligned but I could be wrong (I am a bit new to read mapping and DNase-seq and the data has come from a third party).
Is there any way for wellington_footprints.py to handle these problems gracefully rather than crashing? Can it just ignore these reads? This only seems to happen for reads mapped to the reverse strand: alignedread.pos is never None.
And thanks for your nice software :)

Thanks for the bug report John!
I've noticed that pySAM has actually deprecated aend, so I'll fix that as well so pyDNase doesn't suddenly stop working.
By default, pyDNase just takes all the reads provided to it in the BAM - letting the user decide whether badly mapped/secondary alignments should be utilised. I (stupidly) never handled the situation where unmapped reads may be present in the BAM, and likewise, because not a lot of people are doing paired end alignment for DNase-seq (at least, none of the public data is), I didn't handle ignoring non-primary paired alignments.
For the minimum, I've changed it to only process mapped reads (see 4aca97feefe019f7103c0ec14f7b65844c8d359c) and I'll try to release this to master today, which should fix your issue.
For paired end reads, there's a lot of considerations. I'm thinking a command line flag to ignore secondary alignments or not, but I'll think about this a bit more before implementing something here -opinions and input welcome!

Great thanks for the quick response! I can't say I have any opinions on the secondary alignments as I'm a bit new to this read mapping and still trying to get the hang of it. If I suddenly form some strong opinions I'll be sure to let you know!


add windows build

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

Comparison is base (e029c80) 74.09% compared to head (6de0162) 74.09%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##               main      #93   +/-   ##
=========================================
  Coverage     74.09%   74.09%
  Complexity      230      230
=========================================
  Files            16       16
  Lines          1768     1768
  Branches        167      167
=========================================
  Hits           1310     1310
  Misses          373      373
  Partials         85       85

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


default threshold gap between Tap/Press and Swipe/Pan

The default thresholds for Tap and Press are 9, and their tests for valid movement are:
validMovement = input.distance < options.threshold
The default threshold for Swipe and Pan is 10 and the valid movement tests are:
distance > options.threshold
This leave a gap where mouse movements from 9.0 to 10.0 are invalid.
Seems the easiest workaround is:
this._hammer.get('tap').set({ threshold: 10.00001 });
this._hammer.get('press').set({ threshold: 10.00001 });

I think the code should be changed so that Tap and Press calculate validMovement  using <= not < and their default thresholds should be 10.

@asinning it should be < 10 or <= 9 not < 9 as is there now. PR desired.

I think what you are getting at is
Swipe should be valid >=10
Press/Tap should be valid <=9
There is no such thing as a non-whole-number for an event coordinate.

Hm, what about
function getDistance(p1, p2, props) {
if (!props) {
props = PROPS_XY;
}
var x = p2[props[0]] - p1[props[0]],
y = p2[props[1]] - p1[props[1]];
return Math.sqrt((x * x) + (y * y));

}
On Tue, Jul 12, 2016 at 11:27 AM, Chris Thoburn<EMAIL_ADDRESS>wrote:

I think what you are getting at is
Swipe should be valid >=10
Press/Tap should be valid <=9
There is no such thing as a non-whole-number for an event coordinate.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/hammerjs/hammer.js/issues/983#issuecomment-232100801,
or mute the thread
https://github.com/notifications/unsubscribe/ABHaGixkMwQmA5hwz_WjDquWVyF5qgmMks5qU8BqgaJpZM4IieVO
.


refactor(build stages): internal build should be after pre build

This PR reworks the build stages to ensure internal build stage is done only after providers pre build.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


how do I deploy this on vercel

Sorry for this, but I am getting 404 error.

In settings, change "Root Directory" to "watch"


Testing fo To do List

Hi, we did the following task:

[ ] We import the functions we want to test from utils.js.
[ ] We create a mock for the localStorage object to imitate its operations.
[ ] We also create a mock DOM with an empty list container for testing DOM manipulations.
[ ] We use describe() to group the tests for each function (addTask, deleteTask, and clearCompletedTasks).
[ ] We did the test for order re-position of items in the list.

Hi, thank you for notice that, i fix it!!!


Week 18 Hacks

[ ] Performance Task Scoring 2 link
[ ] Object Oriented Programming link
[ ] Frontend Coding link

.7 Model
.9 View
.8 CPT


Consider adding mono sample

https://github.com/SuaveIO/Melano

We'd be happy to add an F# project to the example list. Currently our example apps follow a standard:

by default, when deployed they respond with

200 OK HTTP/1.1
Powered by Deis

However if someone sets up the envvar POWERED_BY, then that will override "Deis" such that the response will be "Powered by $POWERED_BY". Here's a good example: https://github.com/deis/example-go/blob/master/web.go
Does your melano app behave in that way or would it need a bit of tweaking? :)

There we go! https://github.com/SuaveIO/Melano/blob/master/src/melano/Program.fs#L1


New docs to add

By-Law 1 - Elections.pdf
CSSS Constitution (Mar 2017).pdf
These are to replace the docs currently on the website
27 Feb 2017 Mintues.docx
Minutes from the last meeting to add

Added


Problem with the GetScrollViewerManipulationPropertySet in the anniversay preview

Hi,
Is there any known bug with the GetScrollViewerManipulationPropertySet in the anniversary preview? I´m getting a weird beahviour in a usercontrol that is working perfectly in the november update but it's giving me a hard time with the anniversary preview....
I have a huge Scrollviewer (around 10.000px width x 2.000px hight) where I'm rendering just a few items and recycling them when the view changes.
Every control I render in the scrollview has a expressionanimation based in the scroll properties and when I recycle it I reset the animation like:
_channelsVisual.StopAnimation("Offset.X");
_expression.SetScalarParameter("StartPosition", (float)newposition);
_channelsVisual.StartAnimation("Offset.X", _expression);
This is working fine in the stable version of windows, but with the anniversary preview (14388 both build and SDK) sometimes the ScrollManipulation.Translation.X is returning a wrong value but as soon as I move the scrollviewer (or change the view just 0.1px... ) it starts retuning the correct value again...
Any ideas? The expressionanimation is pretty simple:
"(ScrollManipulation.Translation.X > -StartPosition) ? 8 : -(ScrollManipulation.Translation.X + StartPosition - 8)"
Thank you

@jmvillagra Thanks for the feedback. I'll point you over to @likuba to see if she has any ideas.
You may be hitting an issue around the recent policy changes around interop properties: https://github.com/Microsoft/WindowsUIDevLabs/wiki/XAML-Composition-Property-Synchronization-Changes

Did the article above help address your issue?  If that is the issue you are hitting you will want to work around it by wrapping the UI element you want to animate inside a Canvas in order to avoid the conflict with XAML layout.

The above link to https://github.com/Microsoft/WindowsUIDevLabs/wiki/XAML-Composition-Property-Synchronization-Changes leads to a non-existing page

That's my bad, when I updated the article the URL got changed out from under everyone. Here it is: https://github.com/Microsoft/WindowsUIDevLabs/wiki/XAML-Composition-Interop-Behavior-Changes

Closing as resolved.


Document function details.

Signed-off-by: Humble Chirammal<EMAIL_ADDRESS>

@humblec should this be closed ?

/lgtm

@humblec should this be closed ?


Use new C++ document selection parser

@baldersheim here comes a big one to review! This tries as much as possible to reproduce 1-1 the behavior of the legacy C++ parser, including its quirks. This causes some aspects of the parser to be.. less elegant than they theoretically could be.
@toregge @havardpe @arnej27959 FYI, I would greatly appreciate more sets of eyes on this.
Based on Flex/Bison and replaces old Spirit.Classic parser.
New parser is pure and does not require any locking, unlike the
previous implementation.
This also removes parsing of the deprecated searchcolumn feature.
Adds build-time dependencies on Flex and Bison.

Hold off merging this until a newer version of Bison is included as part of our internal build dependencies (needs 3.0+). Flex version looks fine.

LGTM

As a FYI, the reason this PR remains unmerged is due to wanting the changes in #4186 to be verified as regression-free. I explicitly do not want a release with changes to document selection across two different languages at the same time.

CMake seems to have issues finding Bison for pull request builds. Could this be related?
-- Found JNI: /usr/lib/jvm/java-openjdk/jre/lib/amd64/libjawt.so
CMake Error at /usr/share/cmake3/Modules/FindPackageHandleStandardArgs.cmake:148 (message):
Could NOT find BISON (missing: BISON_EXECUTABLE)
Call Stack (most recent call first):
/usr/share/cmake3/Modules/FindPackageHandleStandardArgs.cmake:388 (_FPHSA_FAILURE_MESSAGE)
/usr/share/cmake3/Modules/FindBISON.cmake:226 (FIND_PACKAGE_HANDLE_STANDARD_ARGS)
document/src/vespa/document/select/CMakeLists.txt:3 (find_package)
-- Configuring incomplete, errors occurred!
See also "/source/CMakeFiles/CMakeOutput.log".

I'm assuming this is related to the build failure: https://github.com/vespa-engine/docker-image-dev/pull/10
Build deps were added to CentOS and internal building prior to merging, but docker was left hanging.


Custom service greyed out, unable to add and/or use

Describe the bug
When adding custom services, there's a colored square around the service which says Premium is required. When you go to the account details page, it says you already have premium Ferdi service enabled.
To Reproduce

Add a custom service to your %appdata% directory.
Reload Ferdi
Add a new service
Select the Custom Tab
The custom service is greyed out

Expected behavior
The ability to add and use the custom service
Screenshots

Desktop

OS: Windows 10 1903 (build 18362.476)
Ferdi.Setup.5.3.3

Which server are you using?

My own personal, running in a docker container

Hey @Rawh, this issue is really old! Do you still experience this behavior with the newest release?

No all is fine, ill close thee issue!


Support Linux ARM64

Hey there 👋
First off, thank you for this wonderful library. It's been very helpful to me so far 😄
I was wondering if it'd be possible to add support to Linux ARM64? This is the target I currently have in production, but it currently fails at postinstall.js

With a little bit of direction, I'm sure I'd be able to contribute this addition if it would be accepted.

work in v1.5.4

Thank you very much @Ossianaa ! I'll definitely give it a test and let you know if I run into anything else

Hey @Ossianaa sorry for the long delay, been heads down building a new company 😄
I can report that I still had some issues with segmentation faults on 1.5.4, but with 1.5.5 I'm not seeing any issues whatsoever! Thank you so much 🙏

Hey @Ossianaa sorry for the long delay, been heads down building a new company 😄
I can report that I still had some issues with segmentation faults on 1.5.4, but with 1.5.5+ I'm not seeing any issues whatsoever! Thank you so much 🙏

Thank you for your feedback. :)


Is broken defender able to report in to ATP?

In the case of defender being in an inoperable state due to scanning a file with two dots in the name or anything of that nature (https://www.techradar.com/news/microsoft-has-just-broken-windows-10s-antivirus-protection-apply-this-fix-straight-away) how would ATP be informed of the broken status since defender appears to be the communication mechanism? Does it have any other methods to report as a fallback or would it somehow fall out of compliance due to lack of reporting?

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

ID: 2bdb99e4-289a-c09a-c35c-2774fc52a402
Version Independent ID: 4d325389-c9b8-2601-0ff4-c7b6bf5a6c28
Content: Microsoft Defender Advanced Threat Protection - Windows security
Content Source: windows/security/threat-protection/microsoft-defender-atp/microsoft-defender-advanced-threat-protection.md
Product: w10
Technology: windows
GitHub Login: @mjcaparas
Microsoft Alias: macapara

@bradrush In that situation (which was fixed shortly after) you would be able to see these "inactive" machines in the Security Operations Dashboard

The Sensor health tile provides information on the individual machine’s ability to provide sensor data to the Microsoft Defender ATP service. It reports how many machines require attention and helps you identify problematic machines.

@bradrush - Thank you for submitting feedback.
From our understanding, the issue you raised has been answered by @VLG17 so we will close this issue.
Thank you for your contribution to make the docs better! Much appreciated!


Implement Module WebAPI Controller, Service, Repository, Models, and Mappers

Description:
As a developer, I need to create a WebAPI controller, service, repository, models, and mappers for the Module entity in our GitHub project. This task involves implementing the necessary components to perform CRUD (Create, Read, Update, Delete) operations on the Module entity using the provided code structure. The implementation should follow best practices, utilize the existing generic repository, and ensure adherence to the project structure.
Task Details:

Module Entity:

Review the provided code for the Module entity (see code snippet below).
Ensure that the Module entity includes the necessary properties and relationships.

public class Module : BaseEntityWithKey<Guid>
{
    [Required]
    public string Name { get; set; }
    [Required]
    public Guid ProjectId { get; set; }
    public virtual Project Project { get; set; }
}

Module Repository:

Utilize the existing generic repository within the project.
Create a Module-specific repository interface and implementation.
Implement the necessary methods for CRUD operations on the Module entity using the generic repository.

Module Service:

Create a Module service that will be used by the controller.
Implement methods in the service for handling business logic related to Module entities.
Use the Module repository to perform CRUD operations and any additional logic required.

Module Controller:

Create a Module WebAPI controller that will handle HTTP requests related to the Module entity.
Implement REST endpoints for the following operations:

Add a module to a project (POST)
Get all modules for a project by project ID (GET)
Get all modules (GET)
Update a module (PUT)
Update a module projectId (PUT)
Delete a module by its ID (DELETE)
Delete all modules for a project by project ID (DELETE)

Models and Mappers:

Create necessary models to represent the data required for request/response payloads in the Module controller.
Implement mappers to map between the Module entity and the request/response models.

Note: Ensure that the code follows the project structure and adheres to the coding conventions followed in the project. Consider implementing error handling, validation, and authentication if applicable.
Estimated Effort: 10-12 hours
Acceptance Criteria:

Module repository interface and implementation are created using the existing generic repository.
Module service is implemented with methods for CRUD operations and any additional logic if needed.
Module WebAPI controller is created with REST endpoints for adding, retrieving, updating, and deleting modules.
Models are created for request/response payloads, and mappers are implemented to map between entity and model.
Code follows the project structure and coding conventions.
Unit tests, if applicable, are created to ensure the functionality of the implemented components.

Dependencies:

Access to the project repository.
Familiarity with .NET WebAPI, Entity Framework Core, and RESTful API development.
Knowledge of the project structure and existing generic repository implementation.
Understanding of coding conventions and best practices for API development.

Add Module  Crud containing project Name 549a5ea.


Does DBT Cloud Connect Apache Spark support SSH tunnel?

Contributions

[X] I have read the contribution docs, and understand what's expected of me.

Link to the page on docs.getdbt.com requiring updates
https://docs.getdbt.com/docs/cloud/connect-data-platform/connect-apache-spark
What part(s) of the page would you like to see updated?
Does DBT Cloud Connect Apache Spark support SSH tunnel? The redshift docs at https://docs.getdbt.com/docs/cloud/connect-data-platform/connect-redshift-postgresql-alloydb shows an example with ssh tunnel.
Additional information
No response

Hi @alberttwong I asked internally and we don't at this time but we will document it on the linked page if it becomes available.


There is no "no substrate" mode

I'm referring to when you hold the volume up to jailbreak without activating substrate.

The issue tab is for code-related issues only, because @kpwn already gets fed up to every issue that has everything to do with device support, and nothing about code. Also, why do you need to enter "no substrate" mode?

I have a tweak that's preventing me getting past the lock screen whilst jailbroken.

Please confirm that you're attempting this in the correct manner. Official Reddit tutorial is below
##How to enter No Substrate mode on iOS 10.2 (Yalu102)

Open Yalu102

Hold the volume up button from this point on

Tap Go

You'll know if you're in No Substrate mode after the respring because Cydia will be able to open. No tweaks will work

No luck. I'll have to SSH into my phone to fix it.

I'm going to try to add 4k device support myself, but that will be dangerous because I don't have any knowledge on Objecive-C. I only know Java, and a little bit of Python.

Checkout https://github.com/kpwn/yalu102/issues/40

This is extremely unrelated to the issue

This isn't a code issue either, but nobody bothers to say "close this issue"?

You're not supposed to create support threads. This is not a support thread, it's more of a suggestion/improvement

It is, but I was just trying to help him.

I think I am aware of why this is happening. I will try to fix it. Thanks for this bug report.

Fixed in last commit. Closing issue, but please comment if it works for you now.