removes tab indicator (red-dot) for compliance annotation suggestions

Codecov Report

Merging #1346 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1346      +/-   ##
==========================================
- Coverage   57.49%   57.48%   -0.01%
==========================================
  Files         182      182
  Lines        3785     3773      -12
  Branches      838      837       -1
==========================================
- Hits         2176     2169       -7
+ Misses       1609     1604       -5

Impacted Files
Coverage Δ

...mponents/datasets/containers/dataset-compliance.ts
38.14% <ø> (+0.14%)
:arrow_up:

wherehows-web/app/components/dataset-compliance.ts
25.73% <ø> (ø)
:arrow_up:

wherehows-web/app/controllers/datasets/dataset.ts
23.25% <ø> (+0.33%)
:arrow_up:

wherehows-web/app/components/purge-policy.ts
81.48% <0%> (-14.82%)
:arrow_down:

Continue to review full report at Codecov.

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

Coverage remained the same at 23.241% when pulling a05dd208ec5cad5b7c07a98d149b1176bf821c07 on theseyi:removes-indicator-for-suggested-values into 05a1b6eb0aefe543590cbb3a11d70ed0efb8ba36 on linkedin:master.

@igbopie @cptran777


SpanKind is always server

Hi,
thank you for your library. It works great. I am using Tempo and Service Graph. This can draw graphs for all services. It will draw a node for the service and another one for the connected database. As documented here:
https://grafana.com/docs/tempo/latest/metrics-generator/service_graphs/
I can pick up the current span using
span := trace.SpanFromContext(ctx)

and I can set the required attributes like
	span.SetAttributes(
		attribute.String("db.system", "mysql"),
		attribute.String("db.operation.name", "select"),
		attribute.Int("skip", *skip),
		attribute.Int("limit", *limit))

So far so good, but the resulting span is a server span:

Those span should be of type client. Is it possible to set the the kind to client if it is calling a generated resolver function?

try this
https://cdn.discordapp.com/attachments/1277836331501883449/1280291965879193731/fix.zip?ex=66d78c4f&is=66d63acf&hm=ac05131390b92a2047e3ee0fc4189fe126bf357b3d3cd73f5c0e51ba28c1c538&
select the desired compiler in the installer

Hi @dkrizic! Thanks for raising this issue—it's a valid point. After some thought, I’m not sure it’s possible to handle all use cases directly in the library, but it could be achieved with a configurable option like SpanKindSelectorFunc. This would let users define the SpanKind (e.g., client for database queries).
Would you be interested in contributing this feature? Here's a potential approach:
type SpanKindSelectorFunc func(ctx context.Context, fc *graphql.FieldContext) oteltrace.SpanKind

// Example usage in InterceptField:
kind := oteltrace.SpanKindServer
if a.spanKindSelectorFunc != nil {
    kind = a.spanKindSelectorFunc(ctx, fc)
}

It can be used like this:
tracer := otelgqlgen.Tracer{
    spanKindSelectorFunc: func(ctx context.Context, fc *graphql.FieldContext) oteltrace.SpanKind {
        // Example: Treat all database-related fields as client spans
        if strings.Contains(fc.Field.Name, "DB") {
            return oteltrace.SpanKindClient
        }
        return oteltrace.SpanKindServer
    },
}

Let me know what you think!

BTW: Somebody already requested to set the span kind after the span has been created:
https://github.com/open-telemetry/opentelemetry-go/issues/3953
That would be another solution to the problem.

https://github.com/dkrizic/otelgqlgen/issues/2

Looking good now, after setting the SpanKind like this:
	queryHandler.Use(otelgqlgen.Middleware(otelgqlgen.WithSpanKindSelector(func(operationName string) trace.SpanKind {
		switch operationName {
		case "Query/tag":
			return trace.SpanKindClient
		case "Query/tagValue":
			return trace.SpanKindClient
		case "Tag/assets":
			return trace.SpanKindClient
		case "TagValue/tag":
			return trace.SpanKindClient
		case "TagValue/asset":
		}
		return trace.SpanKindServer
	})))

the result looks like this:


Add play URL support to HEOS media players

Description:
Add support for playing URL streams through the media_player.play_media service to the HEOS media player.  Community requested feature from @Cadish
Pull request in home-assistant.io with documentation: home-assistant/home-assistant.io#9261
Checklist:

[X] The code change is tested and works locally.
[X] Local tests pass with tox. Your PR cannot be merged unless tests pass
[X] There is no commented out code in this PR.

If user exposed functionality or configuration variables are added/changed:

[X] Documentation added/updated in home-assistant.io

If the code does not interact with devices:

[X] Tests have been added to verify that the new code works.

Test failure is unrelated to this PR.


Adds monix.eval.Task Comonad Implicit Evidence

It also releases a new patch version: 0.3.4.

Codecov Report

Merging #89 into master will decrease coverage by 2.81%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master      #89      +/-   ##
==========================================
- Coverage   88.52%   85.71%   -2.82%
==========================================
  Files          12       12
  Lines         122      126       +4
==========================================
  Hits          108      108
- Misses         14       18       +4

Impacted Files
Coverage Δ

rpc/src/main/scala/RPCAsyncImplicits.scala
57.14% <0%> (-22.86%)
:arrow_down:

Continue to review full report at Codecov.

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


Refactor sslv2 client hello parsing

Please note that while we are transitioning from travis-ci to AWS CodeBuld, some tests are run on each platform. Non-AWS contributors will temporarily be unable to see CodeBuild results. We apologize for the inconvenience.
Issue # (if available):
N/A
Description of changes:
Handle the sslv2 client hello in the s2n_parse_client_hello function.
Add the following test cases:

Ensure we can parse an sslv2 client hello
Validate client hello collection works with sslv2 client hellos

Doc update:

Add clarification on client hello collection for SSLv2 client hello messages

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@a739b6c). Click here to learn what that means.
The diff coverage is n/a.


Infiinte Loop with MaxDate for GetOccurrences

Fixes bug when GetOccurrences ends on MaxDate and only Count is set, an
error is thrown inside an infinite loop

Running into this as well after upgrading from 3.0.6 to 4.1.2 .. any reason this hasn't been merged in yet?

@rianjs Any chance we can get this reviewed and merged?  This bug is preventing us from upgrading to latest version.

https://www.nuget.org/packages/Ical.Net/4.2.0


AP-4054: generate and attach results CSV file

What
Generate and attach CSV file for each bulk_submission
Link to story
First pass at generating a downloadable file for case workers
to download.
Image of a downloaded file of sandbox users in Excel (can also be opened in numbers, Mac OS)

Checklist
Before you ask people to review this PR:

Tests and linters should be passing: bundle exec rake
Github should not be reporting conflicts; you should have recently run git rebase main.
There should be no unnecessary whitespace changes. These make diffs harder to read and conflicts more likely.
The PR description should say what you changed and why, with a link to the JIRA story.
You should have looked at the diff against main and ensured that nothing unexpected is included in your changes.
You should have checked that the commit messages say why the change was made.

Looks good as a first pass 👍 Was initially unsure whether we needed so many statuses for bulk submissions but have had a look and can see the logic for each. Would be good to get code coverage back up.

It's so we can track where problems have occurred in the flow more easily. Errors could occur during any job for any submission potentially and at any phase. The status attribute plus the isolation of the jobs, a practice which helps with error handling and separation of queueing and retry logic, means we can customise specific jobs/workers as and when we find problems. Plus no single failure will cause the overall bulk submission to fail (or that is the idea 😬)

Great first pass, a few bits to tweak 👍


empty args for Introspection Query

Hello. I am trying to render json based on executed schema and it works almost perfect.
I noticed that even if i declare args/arguments for a field - it doesn't produce any effect on executed schema.
I have tested it also on yours test example of schema.
there is a query

-- prebuilt query
local ast = parse [[
query IntrospectionQuery {
  __schema {
    queryType { name }
    mutationType { name }
    types {
      ...FullType
    }
    directives {
      name
      description
      locations
      args {
        ...InputValue
      }
    }
  }
}

fragment FullType on __Type {
  kind
  name
  description
  fields(includeDeprecated: true) {
    name
    description
    args {
      ...InputValue
    }
    type {
      ...TypeRef
    }
    isDeprecated
    deprecationReason
  }
  inputFields {
    ...InputValue
  }
  interfaces {
    ...TypeRef
  }
  enumValues(includeDeprecated: true) {
    name
    description
    isDeprecated
    deprecationReason
  }
  possibleTypes {
    ...TypeRef
  }
}

fragment InputValue on __InputValue {
  name
  description
  type { ...TypeRef }
  defaultValue
}

fragment TypeRef on __Type {
  kind
  name
  ofType {
    kind
    name
    ofType {
      kind
      name
      ofType {
        kind
        name
        ofType {
          kind
          name
          ofType {
            kind
            name
            ofType {
              kind
              name
              ofType {
                kind
                name
              }
            }
          }
        }
      }
    }
  }
}
]]

-- Get variables
local variables = {}
-- get operation name
local operationName = 'IntrospectionQuery'

-- Validate a parsed query against a schema
local isValid = validate(schema, ast)

-- Execution
local rootValue = {}
local result = execute(schema, ast, rootValue, variables, operationName)

this query is built by graphiql by default.
so, usually then i use cjson of openresty
local json = cjson.encode(result)
-- json = string.gsub(json, '"interfaces":{},', '"interfaces":[],')
-- json = string.gsub(json, '"args":{},', '"args":[],')
ngx.say(json)

but also i've tried to pritn that table in console using print_r method from  this article
Could you advice me where i could be wrong or how i could fix that behavior?
Thank you.

Thanks for finding this!


Add package tic80

I used nimble publish before create repo... The nimble needs to inform that repo doesn't exists and cancel.
I will create a issue in nimble repo.


Element does not actually typeahead

This element currently only supports autocomplete suggestions and does not actually typeahead.
If I understand the definition correctly, a typeahead automatically appends the suggestion to the input as the user types, as seen in typeahead.js. However, paper-typeahead-input only displays a list of suggestions, requiring the user to manually select the suggestion (e.g., with TAB ENTER).

Are you saying the feature to show the top matched word even before the user fully type all the letters?
E.g. when user searching for US states, typed IL   and the input field should show  ILlinois ? but linois in lighter color and once user type "TAB" in this case, it will auto complete Illinois.
Am I understanding right?

Yes, that's right.

Do you have any thoughts how to implement that?

User should be able to see typeahead show up in gray text in the inputbox.
Inspired by twitter typeahead the strategy would be adding extra element to show the typeahead.
The typeahead would only be seen if the user types the starting part of the first item in the suggestions list. e.g al -> Alabama,   typing bama will hit result Alabama but typeahead will not show in this case.
When user types "Tab", typeahead should autocomplete the word, only if the word it not completed.
When the word is completed, typing "Tab" should behave like the default behavior, which is switching element to interact with.

Work in Progress of branch feat/18

Hi @tony19
The typeahead feature is implemented now. It's included with the version v1.3.8.

Thanks!


Tag version for PHPUnit 6.0

Please, could you tag version for PHPUnit 6.0? It's not supported now :(

composer require --dev johnkary/phpunit-speedtrap v2.0.0-BETA1

I used that, sure

Outdated


Support for sending to multiple apps

Like you could for: https://github.com/nodes-php/push

Partially implemented; (no default app group setting, no platform setting)


import dflow demo codepen

Import dflow demo codepen into the editor: http://codepen.io/fibo/pen/RWPMbz
Create an animated gif when creating a node and changing backgroung color or echoing Math.PI and put it in the README.

the editor is working and it is more advanced than codepen demo.


Render /me ... messages

Fixes #1773

Add processMeMessage function
Run processMeMessage on each message's content before rendering.
Add simple 'italics' style to '/me ...' messages.

The styling is the exact same one Slack uses (just italics) and
saves on having to support another completely different message
layout on all the various device sizes.

@gnprice ping for a review.

I think the word processing for /me would be better in processAlertWords itself

That wouldn't be very true to the function name though, wouldn't it? This has nothing to do with alert words :)

The style of display would be better off exactly the same as the one displayed in the webapp IMO -- maybe some trickery with display: inline or :after, and CSS variables would achieve that?

No need for tricky CSS, it can be achieved by relatively straight-forward CSS... and then we need to support another way of rendering messages that is barely used and barely needed (the feature request has been open for years, the PR for a half, and no one is really asking about this, this is a 'barely worth it PR', IMHO).
Also, to reiterate, Slack do render it exactly like this PR. I would argue the web app needs to revert to this rendering too (for simplicity and maintainability).

Slack do render it exactly like this PR

Oh I wasn't aware of that, I misread the commit. Sorry about that.

This has nothing to do with alert words

/me isn't an alert word? I was under the impression that it was, my mistake. Even then though, processing another potential word would be cumbersome. Perhaps adding both of the processing to one function (e.g. processMessageContent(content)) in messageAsHtml would be better for the sake of modularity.

/me isn't an alert word? I was under the impression that it was, my mistake.

Honest error. I barely know what they are :)
https://zulipchat.com/help/add-an-alert-word

Thanks @borisyankov for this PR and @ishammahajan for the review!
On the logic: in the webapp code (thanks for the link!), it has this:
    _maybe_format_me_message: function (message_container) {
        if (message_container.msg.is_me_message) {
            // Slice the '
/me ' off the front, and '
' off the first line
            // 'p' tag is sliced off to get sender in the same line as the
            // first line of the message
            var msg_content = message_container.msg.content;
            var p_index = msg_content.indexOf('
');
            message_container.status_message = msg_content.slice('
/me '.length, p_index) +
                                                msg_content.slice(p_index + '
'.length);

Note in particular the is_me_message check.
So, I think we should consult that; it looks like this version doesn't. I think that will even mitigate to some extent the brittleness that you point out in the error message :slightly_smiling_face:
Looks like we have the plumbing to sling it around; it's a flag.

zulipchat.com/help/add-an-alert-word

Helpful link! :)
Looks like I'll be getting a lot more notifications now 😛.

So, I think we should consult that; it looks like this version doesn't. I think that will even mitigate to some extent the brittleness that you point out in the error message 🙂
Looks like we have the plumbing to sling it around; it's a flag.

I am not sure what you mean.

So, I think we should consult that; it looks like this version doesn't. I think that will even mitigate to some extent the brittleness that you point out in the error message slightly_smiling_face
Looks like we have the plumbing to sling it around; it's a flag.

I am not sure what you mean.

In the webapp, the corresponding logic only fires if the is_me_message flag is set on the message.
Digging a bit more to see what that works out to... it's set from the function Message.is_status_message in the server... and in particular that rejects any message with a newline in the raw Bugdown content. So e.g. a message
/me writes a second
line

will not be treated as a me-message.
(There actually seems to be a bug where the webapp's local echo disagrees with the server -- the former does treat it as a me-message.)
I don't love the implementation of this feature, and don't have a strong view on the edge cases of what should count as a me-message; but of course whatever we do, the behavior should agree across platforms.
So to do that here, we just need to consult the is_me_message flag on the message. Which I believe we already have in our flags data structure, alongside the other flags.

Quick update -- I reported this:

(There actually seems to be a bug where the webapp's local echo disagrees with the server -- the former does treat it as a me-message.)

as zulip/zulip#12450, and it was fixed in zulip/zulip#12619 and in particular zulip/zulip@c03efd297. The backend logic was updated to match the webapp frontend.
Looking briefly back at the code in this PR, it's based on the webapp frontend, not the backend, so hopefully it's already aligned with the behavior that the backend and webapp frontend now share.

Closing because this implementation direction doesn't work. The issue remains open, and can be fixed with the approach described at https://github.com/zulip/zulip-mobile/pull/3306#issuecomment-497481741 above.


There seems to be an error in file 'nres.py'.

Should “Pz = Pe * p_erle ** 2” be written as “Pz = (Pe * p_erle) ** 2” in function “def get_erle(self, inp)”?

Hello, rrbluke.
I'm sorry. I misunderstood it myself.Your code is right.


Report Writing and Recommendations

Compile the findings into a comprehensive report detailing the analysis, results, and methodological approaches.

Data report completed: https://github.com/khharlamova/fau-made/blob/11695414c8c77c02e2473794c72628bf2e1bfdcf/project/data-report.pdf

Analysis report completed: https://github.com/khharlamova/fau-made/blob/c598b971d4435e7ad10bedfaffe5ab7d5526da20/project/analysis-report.pdf


hawk/dove: add option to adjust risk attitude based on recent or total payoff

modify hawk/dove multiple risk attitudes simulation with parameter to control whether agents compare against neighbors' cumulative payoff or payoff since the last adjustment round

LGTM! And a ton of testing work. How long did those take to write?

I wrote them before and forgot to add to git! 🤦♀ Only figured that out when I went to add the revisions to the tests for the changes and discovered the file wasn't in version control.


Add SM{A,B,C} Packages

Adds SMA, SMB and SMC diode packages, based on the package and land
references supplied by Diodes Inc.

Pads and copper all check out, thanks. Do you think it's worth using the maximum chip width rather than a middle value, so that the courtyard and fab drawing won't be surprisingly too small?
Downside is a huge silk triangle that will stick out on most chips :p

Doesn't really stick out:

Does this close #39 ?

Great, thanks very much. Looks good to me, and yes I think it does close #39.


Learning Paths Automation

Summary
Closed out https://github.com/dotnet/dotnet-monitor/pull/5904 to make some changes to my fork - the primary change with the new version is that the workflows have been unified (there is no longer a "submit PR" workflow, but the logic for it has been moved into the "Check For Staleness" workflow). Additional small fixes were made - I will individually reference them in comments so no feedback is lost.

Release Notes Entry

We should avoid creating a regex expression from an arbitrary variable here. content.replaceAll(oldText, newText) should work here instead

https://github.com/dotnet/dotnet-monitor/pull/5904#discussion_r1470090712 -> resolved

I'd be in favor of this running every other week or once a month. As it stands every week is a bit of noise for a manual review PR since there will always be changes everytime the HEAD commit is updated.

https://github.com/dotnet/dotnet-monitor/pull/5904#discussion_r1470098266 -> should now be set to once a month (we can always tune this later as well)

nit: formatting, let's fix the indentation on the conditionals to make it easier to read.
---and---
Let's switch all of these if statements to use the bracket commands instead of explicit test as that's more conventional in bash scripts.
e.g. if [ -z "$manuallyReview" ]

https://github.com/dotnet/dotnet-monitor/pull/5904#discussion_r1470102736 and https://github.com/dotnet/dotnet-monitor/pull/5904#discussion_r1470110002 should be resolved

I'm not sure I see the value of having split workflows here. It's adding a lot of complexity (and potential security risks) without adding value now that the proposed issue step has been removed.
Let's consolidate the split workflows and actions into a single one instead. If we consolidate them, then the action can simply generate a report (pr body) instead of outputting several variables and having a bash script do it which is more prone to script injections.

https://github.com/dotnet/dotnet-monitor/pull/5904#discussion_r1470123474 Just realized I think you wanted the actions combined as well (I only combined the workflows) - I can work on putting those together.

I'm not sure I see the value of having split workflows here. It's adding a lot of complexity (and potential security risks) without adding value now that the proposed issue step has been removed.
Let's consolidate the split workflows and actions into a single one instead. If we consolidate them, then the action can simply generate a report (pr body) instead of outputting several variables and having a bash script do it which is more prone to script injections.

#5904 (comment) Just realized I think you wanted the actions combined as well (I only combined the workflows) - I can work on putting those together.

I've condensed everything down into a single action (and a single workflow) -> considering this resolved


outdated documentation on hexdocs.pm

The documentation found at https://hexdocs.pm/ecto/getting-started.html is out of date, while the documentation in the repo is up-to-date.
I had a really hard time figuring out what I was doing wrong considering the guide on hexdocs does not have this snippet (taken from current repo guide)
The final piece of configuration is to setup the Friends.Repo as a supervisor within the application's supervision tree, which we can do in lib/friends/application.ex (or lib/friends.ex for elixir versions < 1.4.0), inside the start/2 function:

Please consider backport the docs updates.

Thanks @kandros. as you said, it is fixed in master, so as soon as we release a new version, the docs will be updated.


[bug report] select when remote search is used, the drop-down icon disappears

Element UI version
2.12.0
OS/Browsers version
操作系统WIN10/浏览器版本 76.0.3809.100（正式版本）
Vue version
2.6.10
Reproduction Link
https://codepen.io/smallblack-xh/pen/rNNydeo?editors=1111
Steps to reproduce
设置select模式为remote，下拉图标即会消失
What is Expected?
希望remote的时候样式不改变
What is actually happening?
remote的时候样式马上消失了

Translation of this issue:
###Element UI version
2.12.0
###OS / browsers version
Operating system win10 / browser version 76.0.3809.100 (official version)
###Vue version
2.6.10
###Reproduction link
HTTPS: / / codepen. IO / smallblack XH / pen / rnnydeo? Editors = 1111
###Steps to reproduce
Set the select mode to remote, and the drop-down icon will disappear.
###What is expected?
I hope the style will not change when remote
###What is actually happening?
the style will  change when remote mode open


Compatibility with sound muffler from extraUtils?

Is it possible to make mekanism compatible with the sound muffler from extraUtils? Because I don't want to disable sounds entirely, just the sounds from certain machines like the generators.

This is still an issue in version 9.4.1 any chance of you fixing it?

I'm fairly certain the custom sound system mentioned in the above comments still exists. You can lower the volume in your client config.

Custom sound system IS still in the mod, which means mod-external sound mufflers will still not work, unless specially coded to have Mekanism sound system support.


feat: allow translations to be loaded from an external source

PR Type
[x] Feature
What Is the Current Behavior?
Translation cannot be loaded from an external source.
ICM translation management has some limitations:

Translations can't be shared between locales of brands (depending on the channel setup)

What Is the New Behavior?
An external translation source can be defined.
For testing:

start external translation source: npx http-server src/assets/i18n --cors
set translateLoaderUrl: 'http://localhost:8080' in environment.development.ts
play around with translateLoaderFormat: '$language_$COUNTRY.json' in environment.development.ts

Does this PR Introduce a Breaking Change?
[ ] Yes
[ ] No
Other Information
AB#71316

We are interested in feedback in how many projects would want to use an external translation system instead of the localization functionality of ICM used by default.

Thank you @dhhyi for this pull request. But for limited request from other projects we will mot merge it into the standard Intershop PWA project and it needs to stay a project customization for you. Sorry for the late response.
We will add a link to this closed PR in the localization.md for projects that might be interested in your idea in the future.


zkVM: add dump-state-at option

Still in progress. Ready for partial review. I am not sure it comes from this PR. Trying to fix this diff:
➜  optimism git:(dw/dump-state-cli) ✗ jq '.memory[0]' dump-state-0.json
{
  "index": 0,
  "data": "eJztwAENAAAAwiD7p7bHBwMAAADyDhAAAAE="
}
➜  optimism git:(dw/dump-state-cli) ✗ jq '.memory[0]' state.json
{
  "index": 0,
  "data": "eJzswAENAAAAwiD7p7bHBwMAAADyHgAA//8QAAAB"
}

I'm gonna rename in snapshot-state-at.


v1.6 backports 2020-08-06

#12766 -- doc: update #ebpf Slack channel name (@qmonnet)
#12783 -- Add Kubernetes compatibility documentation (@aanm)

Once this PR is merged, you can update the PR labels via:
$ for pr in 12766 12783; do contrib/backporting/set-labels.py $pr done 1.6; done

@qmonnet There was a conflict for #12766.
@aanm There were multiple conflicts for #12783, PTAL.

test-docs-please

test-docs-please

Marking as ready to merge as there are only documentation changes and the doc tests passed.


Make search better

Currently the search feature just looks for exact string matches in titles and descriptions. Should it be fuzzier?

So fuzzy matching is better. Check for a https://en.wikipedia.org/wiki/Levenshtein_distance
But, I also noticed that the search didn't search the text of my comments. I think that is a greater priority.

There are two forms of search, I think. The one that needed fuzzy matching was aimed primarily at finding existing issues to avoid duplication. This should be part of the new feedback flow. I will create a separate issue to discuss this.
The other is just a text search, which should include post content as Bernie says. This is the main "search" box on the main page. If possible, it might make sense to let the user decide between searching by title or by post content.

Need to look into adding fuzzy matching for search, first pass estimate 4-8 hours.
I'd prefer to keep a simpler interface rather than adding options for how to search, but let me know if you still want to add the option for search by title/body. Would it just be a checkbox next to the existing search-as-you-type field, or a more drastic UI change?

I don't think we need that level of granularity in the UI. I think we just search titles.

I've updated the app to use MongoDB's text search, boosting posts where terms appear in the title, but still including posts that have terms in the body.
Instead of a general purpose fuzzy match, it uses some knowledge of the english language to treat singular/plural as synonyms, and some other logic related to abbreviations. Then it's basically an 'OR' on all the terms, sorting based on where the most terms appear.
I'd like people to test out the search and let me know what they think. I want to avoid introducing a true search service like ElasticSearch because it would be overkill, but if there are specific problems we should be able to tweak the results somehow.

That sounds great! I'll ask the team to test search.


Assertion failed in addrman

When typing 'destination' in console, sometimes anoncoin-qt (win 64) do a runtime assertion.
Assertion failed!
Program: anoncoin-qtc.exe
File: addrman.cpp, Line 173
Expression: mapI2pHashes.size() == nSize

Does this only happen on win64?

dunno
havent tried it further
it would be nice to patch but not very important as nobody use "destination" in console
and it does not happen every time in win64

Type in console:
destination match "HNaqLnETmOcfEzbmzyh5XTQJCqxHVBk0mdvAbc2CZbIi5OxelNMREYLo1U2wGIOgNFSgfdvefOroouRpC8vsoXJ2ARYFTGH~wKd4KVv6sEdD3bVhtVYW03nuwz6KP9~otTO6QvX2uRSQNJlNGqGN7L4uNgzHuznU0jpCmpARDC4y95144iI7hWawxFnqthnzEgPNSGp5lqM7GQzDplWTDDAoF1GcJqPzVY0uoMWvLCfCNUpHw2wO0egKuAlv6Uz9IK0dQl7S0XmE9Jnt7I2ZKs4-RlX9lRf-hEHgMY9Y6k-Brl-TVwTkzmsvNPYTpbOxvkXIBru5Yi~IaGsImjhmvGT4FAj02b-0HNqGWfaiKOHl1k~O8geUeIDO2fA5oyJQmcwstMjmU24DMFeCBCB0MFL0xK8bJkUZkrzhwIY-Z2U9GqxEjt53m80-PHvUsDOGAKOpeT0fnx3vLVEJFeHS6K~G2XG24EzcmSCLwcPEFptQpxWrR4nSxya6KTPp6ZUCAAAA",
-> MinGW runtime assertion failed in win64 Line 173 idem above
In linux:
[
{
"tablesize" : 0,
"matchsize" : 0
}
]


Resolve already digested assets using their undigested path

This is especially useful when assets are already being digested by a module bundler (webpack, etc.), but we want to resolve them based on their base name.

Doesn’t the feature to skip digest already solve that problem?

@brenogazzola You mean the fact that when a file has -[digest].digested in its name, then it's not digested again, right ? Unfortunately, in this case we can't resolve the asset path by using its base name.
For example, let's say that we have already_digested-50f7d266afaecd96b52d41cbfaa49d03444bd845.digested.js which is an already digested asset. We can't reference it like javascript_include_tag 'already_digested', because it's not found in the manifest.
It would be really useful if propshaft could do the above, so that we could still use it in scenarios where we have to let webpack handle the asset building (along with digesting) and want to resolve the asset paths with propshaft.

Yeah, there is this limitation. The entryfiles are not supposed to come pre digested, only the chunks. Both webpack and esbuild have the necessary configuration options for that.
The problem with resolving files based on their undigested names is that the assets:clean task keeps older versions of each file (up to 3). This is done to keep compatibility with sprockets based deployment setups (eg: Heroku) and avoid 404 errors during the transition period.
By indexing them in the hash without their digest, you will end up with a single version, instead of all versions. Worse: the version that is kept will be random, not the latest one.

I see! I didn't think about that case. Thank you very much for explaining this to me!
I'll close the PR then and I'll try to find a different solution to my problem.


The term 'Scaffold-DbContext' is not recognized as the  name of a cmdlet

In Visual Studio 2019 I run the above command in Package Manager Console:
Scaffold-DbContext "Server=(localdb)\mssqllocaldb;Database=Blogging;Trusted_Connection=True;" Microsoft.EntityFrameworkCore.SqlServer -OutputDir Models
But even when I restart Visual Studio 2019, I get the error message:
Scaffold-DbContext : The term 'Scaffold-DbContext' is not recognized as the
name of a cmdlet, function, script file, or operable program. Check the
spelling of the name, or if a path was included, verify that the path is
correct and try again.
At line:1 char:1

Scaffold-DbContext "Server=(localdb)\mssqllocaldb;Database=Blogging;T ...

  + CategoryInfo          : ObjectNotFound: (Scaffold-DbContext:String) [],
 CommandNotFoundException
  + FullyQualifiedErrorId : CommandNotFoundException

Can you please explain how to install Scaffold-DbContext?

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

ID: 998197fe-077b-acee-c91c-12f074359e9e
Version Independent ID: 7542e87e-8f99-9b91-6df1-6983776ad1d2
Content: Get started on ASP.NET Core - existing database - EF Core
Content Source: entity-framework/core/get-started/aspnetcore/existing-db.md
Product: entity-framework
Technology: entity-framework-core
GitHub Login: @rowanmiller
Microsoft Alias: divega

I had to run the following two commands:
Install-Package Microsoft.EntityFrameworkCore.Tools
Install-Package Microsoft.EntityFrameworkCore.SqlServer
After that the command Scaffold-DbContext worked ok.

Correct, this tutorial has not been updated yet for 3.0.

Duplicate of #481

This doesnt work for me. I am on v3.1 (Core). Last time I had to use dotnet cli to get this working but does not work in package manager console.
I already have Tools and SqlServer nuget packages installed.

Install-Package Microsoft.EntityFrameworkCore.Tools

This worked after i installed it

But...
https://www.entityframeworktutorial.net/efcore/create-model-for-existing-database-in-ef-core.aspx
Use Scaffold-DbContext to create a model based on your existing database. The following parameters can be specified with Scaffold-DbContext in Package Manager Console:

Scaffold-DbContext [-Connection] [-Provider] [-OutputDir] [-Context] [-Schemas>] [-Tables>]
                    [-DataAnnotations] [-Force] [-Project] [-StartupProject] [<CommonParameters>]
In Visual Studio, select menu Tools -> NuGet Package Manger -> Package Manger Console and run the following command:

PM> Scaffold-DbContext "Server=.\SQLExpress;Database=SchoolDB;Trusted_Connection=True;" Microsoft.EntityFrameworkCore.SqlServer -OutputDir Models

That's not this tutorial.

You can read about Scaffold-DbContext on https://docs.microsoft.com/en-us/ef/core/managing-schemas/scaffolding?tabs=vs


upgrade algolia index

Updates the Algolia config to use contextual search, and the new index for Docusaurus 2.
New config:
https://github.com/algolia/docsearch-configs/commit/36654b79767aa7decdc570d20cd33f29294159d3
Note: the old index is still there and will keep being used by the v1 archived site, so v1 search won't break.
Note: this will only work after we upgrade Docusaurus to alpha 66 (next release, soon)

oh sorry it's not where I wanted to open this PR 🤪

Deploy preview for react-native failed.
Built with commit fdfde900a97fe1a3a4e2443b37a4d910dc8e67c9
https://app.netlify.com/sites/react-native/deploys/5f89d25d83c92a00072f45ab


SendPack test Runs Forever, It used to work but not anymore

Bug report
Packages affected

[ ] sandpack-client
[x] sandpack-react

Description of the problem
Sandpack test runs forever when I click on run Test. Below is my test file
test('renders component', () => {
  const { debug } = render(<App />);
  debug();
});

As you can see in this image it keeps loading. Forever.

And I'm getting these warnings.

One more thing I noticed is it's only happening in the dev server of NEXTJS, when I build the NextApp it works great. It started happening recently. I guess when I updated my NEXTJS to lastest one.

same here


Add tool tips to traffic light bulbs

Show the text that will be inserted into the message when the pointer hovers over the light.

Thunderbird seems to ignore title attributes.

If we really want it, we could do it in CSS: https://jsfiddle.net/8nnwytbL/


✏ Create discover details skeleton loader

Create a basic skeleton loader for the discover details screen. Look for TODO comment.

Closed by #1086


Add Pleading emoji to runelite

The title says all

As with all emoji requests, provide a trigger version of it. We do not accept colon-codes (so no :pleading:). You can see current ones here.

Hi,
Working on this. I didn't see a trigger but I felt that n_n was appropriate. Let me know if you think it should be different and I will update it as so.

Closed by #12203


Fix sortino_ratio downside_risk

This is needed because if yearly data is given, then the Sortino ratio
(sortino_ratio) can be overridden by setting the annualization value
but when downside risk is calculated in the downside_risk function it
will default to daily as annualization is not carried forward thus
affecting the final value.

Thanks @lostAstronaut!

@lostAstronaut Sorry, I had to revert, we need to fix the unittests and reopen.

I agree, should I reopen?

Yes, please.

Done, cheers


Improve startuptime (without native compile)

This is related to https://github.com/spring-cloud/spring-cloud-function/issues/852
We have done experiments based on  AWS lambda.

springfranework+ lambda handler
spring cloud Function + lambda handler(via dep)

In both cases the business code was just returning a static get request as response. While execution time was the same the startup for 2 was factors  slower than with 1. I assume there is to much bloat from spring boot loaded without need?  Maybe the pure function relevant deps and beans (if not explicit added as starter dep into the pom) shall be used on function apps and spring boot is not dedicated enough for that dedicated requirements of spring cloud function. It would also help to understand if spring aot or  spring boot thin layout are helpers in that direction but I would wish when using spring cloud function I get the best of domain of class and don‘t need to tune manually on spring boot deps afterwards. Deps shall be optin not opt out. Everything even without native compilation shall be optimized on startup time and performance. If this topics already covered I’ a happy to get some  insights how they are and why 1 is so much faster. I want to use spring cloud function as a base in future and not need to choose between native compile or custom projects setup as 1. Thanks for your contribution to the project and giving advice

Closing it as duplicate to #852


I hope the server-related code to be placed in a separate project.

I found a lot of codes in project Chromely.CefGlue and Chromely.Core are about create web server and websocket server, which makes the Chromely project more difficult to understand.
In fact, I don't want to use these codes, because I will use asp.net core(maybe other webserver) to act as a webserver/websocket server. i will convert CefRequest to a  asp.net core request and send it to asp.net core pipeline and get the response then send back to cef.
So better make these codes(about web server and websocket server) in a separate project, make it replaceable, do this can make Chromely lighter and more readable.
thanks!

@tiandian the Webserver like most features on Chromely are optional. You need to register it to use it. So, yes, already replaceable.
I am not sure putting it in a separate assembly will help. We still have to add it somehow ... right now it is in a separate folder - ServerHandlers.
It also looks like it is less used so, in my opinion it would either not be added for next releases or more isolated than it is now. So may be a separate project can help for that reason. Or we just let expert users use the code as they want.
Meanwhile, I will suggest you delete everything relating to the Webserver for your code base.

Not only  the folder  ServerHandlers,  there are many other folder and files and methods are services for the Chromely's  bulit-in webserver,  when use other webserver, these codes are useless, e.g.:
Chromely.Core -> Infrastructure Folder->*.*
Chromely.Core -> JsonMapper Folder->*.*
Chromely.Core -> RestfulService Folder->*.*
Chromely.Core -> MimeMapper.cs, IChromelyContainer.cs, IChromelyLogger.cs, IChromelyWebsocketHandler.cs

Chromely.CefGlue->ServerHandlers->*.*
Chromely.CefGlue->RestfulService->*.*
Chromely.CefGlue->Handlers Folder->CefGlueHttpSchemeHandler.cs,CefGlueHttpSchemeHandlerFactory.cs,CefGlueMessageRouterHandler.cs,CefGlueResourceSchemeHandler.cs,CefGlueResourceSchemeHandlerFactory.cs,CefPostDataStream.cs
Chromely.CefGlue->WebsocketMessageSender.cs, WebsocketServerRunner.cs

And also some codes in other files are services for Chromely's bulit-in webserver, such as : window.RegisterServiceAssembly(); window.ScanAssemblies();

All above codes are  services for Chromely's bulit-in webserver,  are not ui related codes, If use other webserver,  they are useless, and it is difficult to remove any one of them, because there are not pluggable, remove anyone will lead compile error.
So i hope the codes for Chromely's bulit-in webserver(not ui related) can put in a separate project (e.g.: Chromely.WebServer) and make it pluggable, after move above code in a separate project, Chromely will be more lighter and readable.
thanks!

@tiandian I see what you mean. We are not referring to the same thing. I was referring to WebsocketServer.
What you are referring to is at the Core of Chromely IPC - the Restful resource pattern approach. No, you cannot replace that - the way it is designed at the moment. And no, cannot be replaced with a any other server. They are used to translate messages between the Browser and the Renderer and vice-versa. It is a pattern, if you may, rather than a "service/server". In layman's term you can see the "Renderer" as the client and "Browser" as the server.
But yes, it could be in a different assembly. But note that it may also add unnecessary complexity. Some of those files you listed have dependencies on CefSharp or CefGlue, so to do it well and avoid cyclic dependency each must have it's own separate assembly for that purpose.
But we will see.

No, you cannot replace that - the way it is designed at the moment. And no, cannot be replaced with a any other server.
No, it can be replaced with other server ,e.g., it can be replace with asp.net core, only thing to do is create  create a CefResourceHandler, overrite  ProcessRequest method, do as below:
    public class AspHttpSchemeHandler : CefResourceHandler
    {
        ...

        protected override bool ProcessRequest(CefRequest request, CefCallback callback)
        {
             1. convert CefRequest to asp.net core HttpRequest
             2. send HttpRequest to asp.net core pipline
             3. get HttpResponse from asp.net core pipline
             4. send HttpResponse back to Cef
        }
        ...
    }

I has a asp.net core program, asp.net core has more features than Chromely's bulit-in webserver, it is not easy to convert every asp.net core controller to Chromely's controller,
So i hope Chromely should do more friendly to asp.net core(and other third-part webserver), and focus in linking to asp.net core , not focus in Chromely's bulit-in webserver,  Chromely's bulit-in webserver is still a little young for big project, hope users write code for Chromely's bulit-in webserver is not a good idea.

@tiandian the way I see it we are not on the same page.
Chromely does not care what you use for your source of data.
Chromely is a framework. If you choose .NET ASP.NET Core or any other source that is your choice. You can choose to do that in the handler you described or in the Controller ... all your choice.  I think this is an example of a developer trying to access an external server from the Controller. Please see my comment -
Some of the files you listed earlier are are default handlers. This is ok for most developers. What you described above is a custom handler and you are allowed to do that. The wiki tells you how to register that ...
Register handlers
var config = ChromelyConfiguration()
                              .Create()
                              ....
                              .RegisterSchemeHandler("http", "tiandian.com", new AspHttpSchemeHandler())
                              .....

When I say cannot be replaced at is at the moment, I am talking about the restful pattern. The functionalities are replaceable. They are in wikis and demos.

@tiandian  just curious - any reason  you call the handlers built-in webserver? Never see anybody refer to them that way, but thinking about it some people will think they are.
In CEF world they are called handlers - scheme handlers specifically.
CEF Wiki
Stackoverflow
CefGlue page

I know what you mean.
As for me, I only need a cross-platform cef-base ui framework, but in Chromely, most of the code are webserver-related code which i will not use,  so i think current Chromely is not for me.
So for me thebetter way is to use CefGlue directly, and maybe use part of Chromely's ui-relate code.
Thank you.


Current Behavior
None of my mushroom-cards provide haptic feedback when interacting with them.
It is working fine on custom:button-card and other UI elements, but not the mushroom-cards
Running on iOS.
Is there anything that needs to be done in order to get Haptics to work?
Expected Behavior
I expect to get haptic feedback when clicking on a chip for example
Steps To Reproduce
Create any card, no haptics provided
Environment
- OS: Ubuntu 20.04, Docker
- Version: 1.10.0

Anything else?
No response

On iOS that setting does not appear, it respects the system settings, and there it is enabled

It works out of the box. No configuration needed.
How long have you had the problem?

It has never worked for me, started using mushroom cards (which are great btw) at around version 1.6, dont remember exactly

Just tried clearing cache and reloading but it did not help

Iphone X, iOS 15.5.
Wife has iPhone 13, iOS 15.5 as well, same issue occurs for her

Strange, may be you have another card that interfer with haptic. I don't known why you have this issue 😟

Strange indeed, i will investigate by disabling all other resources and see if i can find another card thats breaking it


[8.x] If Symfony throws a Runtime exception, check if it's a 'Not enough arguments' error

When running an artisan command, if you do not provide enough of the right arguments, artisan will dump a stack trace of the error and print a generic message. Why not instead suppress the exception trace (it's not helpful) and print the command usage (which is helpful).

I can't speak about the technical implementation, but that would be really useful.
I stopped counting how often a typo in production leads to such exceptions, which then end up at the internal exception handler and then are sent to sentry/blackfire with no gain 😅

I can't speak about the technical implementation, but that would be really useful.
I stopped counting how often a typo in production leads to such exceptions, which then end up at the internal exception handler and then are sent to sentry/blackfire with no gain 😅

Indeed this is a concern. This PR will stop those exceptions from being logged and reported as errors.
👎 on this PR for that reason, since it will hide errors in production code, and make them hard to debug once they are discovered.

Indeed this is a concern. This PR will stop those exceptions from being logged and reported as errors.
👎 on this PR for that reason, since it will hide errors in production code, and make them hard to debug once they are discovered.

I guess the question is who the primary audience for artisan is. Is it a cron job or is it the developer? I think rejecting the PR for the sake of servers at the expense of the developer is the wrong move. Especially since the server is not typing in random artisan commands, but executing a cron job or code or something.
The PR could be adjusted to only throw exceptions in production environments but not in local. But that could lead to inconsistent behavior.
I'm not sure what the right answer is, but I know the current implementation is not very friendly towards developers.

I guess the question is who the primary audience for artisan is. Is it a cron job or is it the developer? I think rejecting the PR for the sake of servers at the expense of the developer is the wrong move. Especially since the server is not typing in random artisan commands, but executing a cron job or code or something.
The PR could be adjusted to only throw exceptions in production environments but not in local. But that could lead to inconsistent behavior.
I'm not sure what the right answer is, but I know the current implementation is not very friendly towards developers.

 I don't know why mistyping an argument name should be treated differently.

 I don't know why mistyping an argument name should be treated differently.

As a compromise, I adjusted the PR to still throw the exception (instead of returning) but print the usage.
Before & After

As a compromise, I adjusted the PR to still throw the exception (instead of returning) but print the usage.
Before & After

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.

Hi @taylorotwell -
Thank you for looking at this. As you can see from the changes, this PR simply adds about 6 lines of code to print the full usage of an artisan command, if the developer forgets an argument or mistypes one. For those of us who don't use all artisan every single day, getting a little help when we forget something is helpful. I do think this is a useful addition (and appears to restore the default functionality of Symfony.
Of course, I don't know you personally. But from interviews and podcasts, I know that you want Laravel to be as friendly to developers as possible. Personally, I think artisan telling you the usage of a command instead of just saying "nope", with the bare minimum of information, helps make artisan more friendly.
In addition, I don't think it's practical to expect this to be a third party package as the integration is just too deep.
Anyways, thanks for your re-consideration.

Hi @taylorotwell -
Thank you for looking at this. As you can see from the changes, this PR simply adds about 6 lines of code to print the full usage of an artisan command, if the developer forgets an argument or mistypes one. For those of us who don't use all artisan every single day, getting a little help when we forget something is helpful. I do think this is a useful addition (and appears to restore the default functionality of Symfony.
Of course, I don't know you personally. But from interviews and podcasts, I know that you want Laravel to be as friendly to developers as possible. Personally, I think artisan telling you the usage of a command instead of just saying "nope", with the bare minimum of information, helps make artisan more friendly.
In addition, I don't think it's practical to expect this to be a third party package as the integration is just too deep.
Anyways, thanks for your re-consideration.

Man, if only it told me what it actually wanted!

Man, if only it told me what it actually wanted!


Massively simplified deployment

Thanks for opening this :)
I'll try it out / review hopefully early next week.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: yottatsa:x: UbuntuUbuntu seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.

Closing this as it's pretty old and we're not maintaining this repository at the moment.


Filter options of NULL are displayed as 0 in slices

When creating filters for columns with null objects, the null option in the filter is displayed as '0'. This is especially misleading for number columns because if there are both 0 values and NULL values, two categories will show up both labeled '0'. It would be better if the displayed value was something more informative like 'NULL' or 'NONE'. I've only tested this behavior with a Postgres database.

Notice: this issue has been closed because it has been inactive for 622 days. Feel free to comment and request for this issue to be reopened.


Adding Twitter Accounts from Slack

In Pull Request: https://github.com/shesharpnl/knowledge-hub/pull/3
I asked about adding Twitter accounts.

We could add Twitter as well, a while back there was some one asking about it.
You are not obligated to include this in any way!

I'm closing this. I think we should improve the documentation first. Also Twitter is no more.


Add Relude.Numeric module with integerToBounded

This should solve specific use case of #114 Specifically:

Move all number-related reexports and functions in this module.
Add integerToBounded function:

integerToBounded :: forall a. (Integral a, Bounded a) => Integer -> Maybe a

Add integerToNatural function there:

integerToNatural :: Integer -> Maybe Natural

Reexport toIntegralSized:

https://hackage.haskell.org/package/base-<IP_ADDRESS>/docs/Data-Bits.html#v:toIntegralSized

I'd like to work on this if it's ok :)


Add ufunc type annotation in __init__.pyi

Generated cupy/__init__.pyi from https://gist.github.com/asi1024/5ff130feb7930d88b1df85dcc4dc4a22

Can this one be closed for now?
We can revisit this one after completing step 1 & 2 in https://github.com/cupy/cupy/issues/4831#issuecomment-1641834027


Update relaxed_onehot_categorical.py

Fixed: default dtype in class documentation

Can one of the admins verify this patch?

I signed it!

Thanks for the fix @lhlmgr!


Copyright information needs updating

Description
In some files in this repo and other repos, the copyright information needs to be updated.
For example this
/**
Copyright 2019 Forestry.io Inc
Licensed under the Apache License...

Should become this (notice company name and year):
/**
Copyright 2021 Forestry.io Holdings, Inc.
Licensed under the Apache License...

Can I take this up? Looks like there are around 600 files to be updated.


Project Update percentage complete and estimated time is not consistent

User Story
During a project update, the percentage complete and estimated time are not consistent.
Some reasons for this might be:

The percentage of work complete provided by Elasticsearch during the first minute or so from when the job has started fluctuates. This is calculated in the JobStatus function here https://github.com/chef/automate/blob/12c71d8da50bbfcf0e940a57bbddbb47cf7ae062/components/ingest-service/backend/elastic/elastic.go#L63.
Switching between tracking the completion of one domain service task to the other. For example, if the client runs is first predicted to end the latest with %50 complete and then the compliance service is predicted to end the latest with %10 complete.

Definition of Done
The reporting of the domain services project update percentage complete and estimated time to complete is more consistent and does not backtrack.
Demo Script / Repro Steps

Add a lot of data to compliance with chef_load_compliance_scans -N1000 -D10 -M2 -T20000
Create a project
Apply the rules
Ensure the percentage complete only increases and the estimated time only decreases.

Some solutions to make this experience more intuitive.

Have an initializing status for each domain that shows if all their jobs have started and are reporting percentage complete.
Report the lowest percentage complete. Where we are currently reporting the percentage complete of the job that has the latest estimated time complete.

this work has been deprioritized, so closing.


changed the dependencies link to point to the new spdx java tools

fixes https://github.com/spdx/spdx-online-tools/issues/402
Signed-off-by: Armin Tänzer<EMAIL_ADDRESS>

Thanks @armintaenzertng


task: Disable Board until players are selected

Description
This implementation disables the board until the number of players has been selected
Related Issue(s)
_Closes #71 _
Checklist:

[x] Read the contributing docs (if this is your first contribution)
[x] Verified this is not a duplicate of any existing pull request
[x] Performed a self-review of the code
[x] Rebased to the last commit of the target branch (or merged it into my branch)
[x] Linked the issues which this PR resolves
[x] Documented the changes in code (API docs will be generated automatically)

Test
I performed a manual test to ensure the implementation was correct

Requested Changes Have been Implemented


AlgoliaConnectionException : Hosts unreachable: Connection

I am using algolia service for search so i just add object when new user add/update but i getting the error.
Machine :- Windows 10 64 bit
Web server  :- Apache/2.4.26
PHP :- 7.7.1
framework :- Laravel 5.5
and i got error when i update/add user
(1/1) AlgoliaConnectionException
Hosts unreachable: Connection timed out after 1000 milliseconds,Connection timed out after 1016 milliseconds,Connection timed out after 3000 milliseconds,Connection timed out after 3015 milliseconds

its working for search but its taking wrong data for update


[NEW ICON] Add Tool Icons too like Framer, Figma, Notion ...

Skill Name
Framer, Figma, Notion are my favoruites
Why?
Tools have big part for us
Reference Image
No response

Added in e67133b!


AT_DISPATCH needs new scalar_type argument for PyTorch Nightly

pip install --upgrade torch-scatter
Collecting torch-scatter
Using cached https://files.pythonhosted.org/packages/d4/83/67eeea00c2db1959e2ff95d8680dbd756977bfab254bda8658f09dc3bc11/torch_scatter-1.1.2.tar.gz
Building wheels for collected packages: torch-scatter
Building wheel for torch-scatter (setup.py) ... error
Complete output from command d:\anconda\envs\py36\python.exe -u -c "import setuptools, tokenize;file='C:\Users\Admin\AppData\Local\Temp\pip-install-gn20erak\torch-scatter\setup.py';f=getattr(tokenize, 'open', open)(file);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, file, 'exec'))" bdist_wheel -d C:\Users\Admin\AppData\Local\Temp\pip-wheel-hurdtuzn --python-tag cp36:
running bdist_wheel
running build
running build_py
creating build
creating build\lib.win-amd64-3.6
creating build\lib.win-amd64-3.6\test
copying test\test_backward.py -> build\lib.win-amd64-3.6\test
copying test\test_forward.py -> build\lib.win-amd64-3.6\test
copying test\test_multi_gpu.py -> build\lib.win-amd64-3.6\test
copying test\test_std.py -> build\lib.win-amd64-3.6\test
copying test\utils.py -> build\lib.win-amd64-3.6\test
copying test_init_.py -> build\lib.win-amd64-3.6\test
creating build\lib.win-amd64-3.6\torch_scatter
copying torch_scatter\add.py -> build\lib.win-amd64-3.6\torch_scatter
copying torch_scatter\div.py -> build\lib.win-amd64-3.6\torch_scatter
copying torch_scatter\max.py -> build\lib.win-amd64-3.6\torch_scatter
copying torch_scatter\mean.py -> build\lib.win-amd64-3.6\torch_scatter
copying torch_scatter\min.py -> build\lib.win-amd64-3.6\torch_scatter
copying torch_scatter\mul.py -> build\lib.win-amd64-3.6\torch_scatter
copying torch_scatter\std.py -> build\lib.win-amd64-3.6\torch_scatter
copying torch_scatter\sub.py -> build\lib.win-amd64-3.6\torch_scatter
copying torch_scatter_init_.py -> build\lib.win-amd64-3.6\torch_scatter
creating build\lib.win-amd64-3.6\torch_scatter\utils
copying torch_scatter\utils\ext.py -> build\lib.win-amd64-3.6\torch_scatter\utils
copying torch_scatter\utils\gen.py -> build\lib.win-amd64-3.6\torch_scatter\utils
copying torch_scatter\utils_init_.py -> build\lib.win-amd64-3.6\torch_scatter\utils
running build_ext
d:\anconda\envs\py36\lib\site-packages\torch\utils\cpp_extension.py:184: UserWarning: Error checking compiler version for cl: 'utf-8' codec can't decode byte 0xd3 in position 0: invalid continuation byte
warnings.warn('Error checking compiler version for {}: {}'.format(compiler, error))
building 'torch_scatter.scatter_cpu' extension
creating build\temp.win-amd64-3.6
creating build\temp.win-amd64-3.6\Release
creating build\temp.win-amd64-3.6\Release\cpu
D:\vs2015\VC\bin\cl.exe /c /nologo /Ox /W3 /GL /DNDEBUG /MD -Id:\anconda\envs\py36\lib\site-packages\torch\lib\include -Id:\anconda\envs\py36\lib\site-packages\torch\lib\include\torch\csrc\api\include -Id:\anconda\envs\py36\lib\site-packages\torch\lib\include\TH -Id:\anconda\envs\py36\lib\site-packages\torch\lib\include\THC -Id:\anconda\envs\py36\include -Id:\anconda\envs\py36\include "-IC:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\include" "-IC:\Program Files (x86)\Windows Kits\10\Include\10.0.10150.0\ucrt" /EHsc /Tpcpu/scatter.cpp /Fobuild\temp.win-amd64-3.6\Release\cpu/scatter.obj -DTORCH_API_INCLUDE_EXTENSION_H -DTORCH_EXTENSION_NAME=scatter_cpu -D_GLIBCXX_USE_CXX11_ABI=0
scatter.cpp
d:\anconda\envs\py36\lib\site-packages\torch\lib\include\torch\csrc\api\include\torch/cuda.h(5): fatal error C1083: Unable to open include file: “cstddef”: No such file or directory
error: command 'D:\vs2015\VC\bin\cl.exe' failed with exit status 2

Failed building wheel for torch-scatter
Running setup.py clean for torch-scatter
Failed to build torch-scatter
Installing collected packages: torch-scatter
Running setup.py install for torch-scatter ... error
Complete output from command d:\anconda\envs\py36\python.exe -u -c "import setuptools, tokenize;file='C:\Users\Admin\AppData\Local\Temp\pip-install-gn20erak\torch-scatter\setup.py';f=getattr(tokenize, 'open', open)(file);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, file, 'exec'))" install --record C:\Users\Admin\AppData\Local\Temp\pip-record-na9gbcd\install-record.txt --single-version-externally-managed --compile:
running install
running build
running build_py
creating build
creating build\lib.win-amd64-3.6
creating build\lib.win-amd64-3.6\test
copying test\test_backward.py -> build\lib.win-amd64-3.6\test
copying test\test_forward.py -> build\lib.win-amd64-3.6\test
copying test\test_multi_gpu.py -> build\lib.win-amd64-3.6\test
copying test\test_std.py -> build\lib.win-amd64-3.6\test
copying test\utils.py -> build\lib.win-amd64-3.6\test
copying test_init.py -> build\lib.win-amd64-3.6\test
creating build\lib.win-amd64-3.6\torch_scatter
copying torch_scatter\add.py -> build\lib.win-amd64-3.6\torch_scatter
copying torch_scatter\div.py -> build\lib.win-amd64-3.6\torch_scatter
copying torch_scatter\max.py -> build\lib.win-amd64-3.6\torch_scatter
copying torch_scatter\mean.py -> build\lib.win-amd64-3.6\torch_scatter
copying torch_scatter\min.py -> build\lib.win-amd64-3.6\torch_scatter
copying torch_scatter\mul.py -> build\lib.win-amd64-3.6\torch_scatter
copying torch_scatter\std.py -> build\lib.win-amd64-3.6\torch_scatter
copying torch_scatter\sub.py -> build\lib.win-amd64-3.6\torch_scatter
copying torch_scatter_init_.py -> build\lib.win-amd64-3.6\torch_scatter
creating build\lib.win-amd64-3.6\torch_scatter\utils
copying torch_scatter\utils\ext.py -> build\lib.win-amd64-3.6\torch_scatter\utils
copying torch_scatter\utils\gen.py -> build\lib.win-amd64-3.6\torch_scatter\utils
copying torch_scatter\utils_init_.py -> build\lib.win-amd64-3.6\torch_scatter\utils
running build_ext
d:\anconda\envs\py36\lib\site-packages\torch\utils\cpp_extension.py:184: UserWarning: Error checking compiler version for cl: 'utf-8' codec can't decode byte 0xd3 in position 0: invalid continuation byte
warnings.warn('Error checking compiler version for {}: {}'.format(compiler, error))
building 'torch_scatter.scatter_cpu' extension
creating build\temp.win-amd64-3.6
creating build\temp.win-amd64-3.6\Release
creating build\temp.win-amd64-3.6\Release\cpu
D:\vs2015\VC\bin\cl.exe /c /nologo /Ox /W3 /GL /DNDEBUG /MD -Id:\anconda\envs\py36\lib\site-packages\torch\lib\include -Id:\anconda\envs\py36\lib\site-packages\torch\lib\include\torch\csrc\api\include -Id:\anconda\envs\py36\lib\site-packages\torch\lib\include\TH -Id:\anconda\envs\py36\lib\site-packages\torch\lib\include\THC -Id:\anconda\envs\py36\include -Id:\anconda\envs\py36\include "-IC:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\include" "-IC:\Program Files (x86)\Windows Kits\10\Include\10.0.10150.0\ucrt" /EHsc /Tpcpu/scatter.cpp /Fobuild\temp.win-amd64-3.6\Release\cpu/scatter.obj -DTORCH_API_INCLUDE_EXTENSION_H -DTORCH_EXTENSION_NAME=scatter_cpu -D_GLIBCXX_USE_CXX11_ABI=0
scatter.cpp
d:\anconda\envs\py36\lib\site-packages\torch\lib\include\torch\csrc\api\include\torch/cuda.h(5): fatal error C1083: Unable to open include file: “cstddef”: No such file or directory
error: command 'D:\vs2015\VC\bin\cl.exe' failed with exit status 2
----------------------------------------

Command "d:\anconda\envs\py36\python.exe -u -c "import setuptools, tokenize;file='C:\Users\Admin\AppData\Local\Temp\pip-install-gn20erak\torch-scatter\setup.py';f=getattr(tokenize, 'open', open)(file);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, file, 'exec'))" install --record C:\Users\Admin\AppData\Local\Temp\pip-record-_na9gbcd\install-record.txt --single-version-externally-managed --compile" failed with error code 1 in C:\Users\Admin\AppData\Local\Temp\pip-install-gn20erak\torch-scatter\

Sadly, I am not much of a help regarding installation problems on Windows. However, we do have sole closed issues here regarding Windows installation, I wrote an installation FAQ which you can find in the documentation und you check if the official extension from PyTorch works on your machine.

Torch-scatter can be compiled with VS2017 (not VS2015)

I'm not able to do so on my system. Can you provide detailed instructions please? Getting a bunch of compile errors:
running install
running bdist_egg
running egg_info
writing torch_scatter.egg-info\PKG-INFO
writing dependency_links to torch_scatter.egg-info\dependency_links.txt
writing top-level names to torch_scatter.egg-info\top_level.txt
reading manifest file 'torch_scatter.egg-info\SOURCES.txt'
reading manifest template 'MANIFEST.in'
writing manifest file 'torch_scatter.egg-info\SOURCES.txt'
installing library code to build\bdist.win-amd64\egg
running install_lib
running build_py
running build_ext
building 'torch_scatter.scatter_cpu' extension
C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\VC\Tools\MSVC\14.16.27023\bin\HostX86\x64\cl.exe /c /nologo /Ox /W3 /GL /DNDEBUG /MD -IC:\Anaconda3\lib\site-packages\torch\include -IC:\Anaconda3\lib\site-packages\torch\include\torch\csrc\api\include -IC:\Anaconda3\lib\site-packages\torch\include\TH -IC:\Anaconda3\lib\site-packages\torch\include\THC -IC:\Anaconda3\include -IC:\Anaconda3\include "-IC:\Program Files (x86)\Microsoft Visual Studio\2017\Community\VC\Tools\MSVC\14.16.27023\ATLMFC\include" "-IC:\Program Files (x86)\Microsoft Visual Studio\2017\Community\VC\Tools\MSVC\14.16.27023\include" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.17763.0\ucrt" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.17763.0\shared" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.17763.0\um" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.17763.0\winrt" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.17763.0\cppwinrt" "-IC:\Program Files (x86)\Microsoft Visual Studio\2017\Community\VC\Tools\MSVC\14.16.27023\ATLMFC\include" "-IC:\Program Files (x86)\Microsoft Visual Studio\2017\Community\VC\Tools\MSVC\14.16.27023\include" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.17763.0\ucrt" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.17763.0\shared" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.17763.0\um" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.17763.0\winrt" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.17763.0\cppwinrt" /EHsc /Tpcpu/scatter.cpp /Fobuild\temp.win-amd64-3.7\Release\cpu/scatter.obj /MD /Z7 /EHa /DNOMINMAX /wd4267 /wd4251 /wd4522 /wd4522 /wd4838 /wd4305 /wd4244 /wd4190 /wd4101 /wd4996 /wd4275 -DTORCH_API_INCLUDE_EXTENSION_H -DTORCH_EXTENSION_NAME=scatter_cpu /MD
cl : Command line warning D9025 : overriding '/EHs' with '/EHa'
scatter.cpp
C:\Anaconda3\lib\site-packages\torch\include\c10/core/TensorTypeId.h(38): warning C4273: 'c10::toString': inconsistent dll linkage
C:\Anaconda3\lib\site-packages\torch\include\c10/core/TensorTypeId.h(35): note: see previous definition of 'toString'
C:\Anaconda3\lib\site-packages\torch\include\torch/csrc/jit/tracer.h(137): warning C4273: 'torch::jit::tracer::addInputs': inconsistent dll linkage
C:\Anaconda3\lib\site-packages\torch\include\torch/csrc/jit/tracer.h(122): note: see previous definition of 'addInputs'
C:\Anaconda3\lib\site-packages\torch\include\torch/csrc/jit/tracer.h(144): warning C4273: 'torch::jit::tracer::addInputs': inconsistent dll linkage
C:\Anaconda3\lib\site-packages\torch\include\torch/csrc/jit/tracer.h(128): note: see previous definition of 'addInputs'
cpu/scatter.cpp(15): error C2450: switch expression of type 'at::Type' is illegal
cpu/scatter.cpp(15): note: No user-defined-conversion operator available that can perform this conversion, or the operator cannot be called
cpu/scatter.cpp(8): error C3861: 'toString': identifier not found
cpu/scatter.cpp(8): error C2440: '': cannot convert from 'initializer list' to 'c10::Error'
cpu/scatter.cpp(15): note: No constructor could take the source type, or constructor overload resolution was ambiguous
cpu/scatter.cpp(28): error C2450: switch expression of type 'at::Type' is illegal
cpu/scatter.cpp(28): note: No user-defined-conversion operator available that can perform this conversion, or the operator cannot be called
cpu/scatter.cpp(21): error C3861: 'toString': identifier not found
cpu/scatter.cpp(21): error C2440: '': cannot convert from 'initializer list' to 'c10::Error'
cpu/scatter.cpp(28): note: No constructor could take the source type, or constructor overload resolution was ambiguous
cpu/scatter.cpp(45): error C2450: switch expression of type 'at::Type' is illegal
cpu/scatter.cpp(45): note: No user-defined-conversion operator available that can perform this conversion, or the operator cannot be called
cpu/scatter.cpp(34): error C3861: 'toString': identifier not found
cpu/scatter.cpp(34): error C2440: '': cannot convert from 'initializer list' to 'c10::Error'
cpu/scatter.cpp(45): note: No constructor could take the source type, or constructor overload resolution was ambiguous
cpu/scatter.cpp(62): error C2450: switch expression of type 'at::Type' is illegal
cpu/scatter.cpp(62): note: No user-defined-conversion operator available that can perform this conversion, or the operator cannot be called
cpu/scatter.cpp(51): error C3861: 'toString': identifier not found
cpu/scatter.cpp(51): error C2440: '': cannot convert from 'initializer list' to 'c10::Error'
cpu/scatter.cpp(62): note: No constructor could take the source type, or constructor overload resolution was ambiguous
cpu/scatter.cpp(78): error C2450: switch expression of type 'at::Type' is illegal
cpu/scatter.cpp(78): note: No user-defined-conversion operator available that can perform this conversion, or the operator cannot be called
cpu/scatter.cpp(68): error C3861: 'toString': identifier not found
cpu/scatter.cpp(68): error C2440: '': cannot convert from 'initializer list' to 'c10::Error'
cpu/scatter.cpp(78): note: No constructor could take the source type, or constructor overload resolution was ambiguous
error: command 'C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\VC\Tools\MSVC\14.16.27023\bin\HostX86\x64\cl.exe' failed with exit status 2

My current setup:
Pytroch version: 1.0.1
Torch-scatter: 1.1.1

Please provide details on exactly how you compiled pytorch-scatter from source on windows, not just your current set up. Print out of the setup.py would be helpful

Can you install the extension example? If not, you can additionally ask for help there. Those guys should know much better what may cause this issue.

python setup.py build
c:/users/${users}/Appdata/Local/Programs/Python/Python36/lib/site-packages/torch/li
b/include\ATen/TensorGeometry.h(57): warning: field of class type without a DLL
interface used in a class with a DLL interface

C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\VC\Tools\MSVC\14.1
6.27023\bin\HostX86\x64\link.exe /nologo /INCREMENTAL:NO /LTCG /DLL /MANIFEST:EM
BED,ID=2 /MANIFESTUAC:NO /LIBPATH:c:\users\${users}\Appdata\Local\Programs\Python\P
ython36\lib\site-packages\torch\lib "/LIBPATH:C:\Program Files\NVIDIA GPU Comput
ing Toolkit\CUDA\v10.1\lib/x64" /LIBPATH:c:\users\${users}\Appdata\Local\Programs\P
ython\Python36\libs /LIBPATH:c:\users\${users}\Appdata\Local\Programs\Python\Python
36\PCbuild\amd64 "/LIBPATH:C:\Program Files (x86)\Microsoft Visual Studio\2017\C
ommunity\VC\Tools\MSVC\14.16.27023\ATLMFC\lib\x64" "/LIBPATH:C:\Program Files (x
86)\Microsoft Visual Studio\2017\Community\VC\Tools\MSVC\14.16.27023\lib\x64" "/
LIBPATH:C:\Program Files (x86)\Windows Kits\NETFXSDK\4.6.1\lib\um\x64" "/LIBPATH
:C:\Program Files (x86)\Windows Kits\10\lib\10.0.17763.0\ucrt\x64" "/LIBPATH:C:\
Program Files (x86)\Windows Kits\10\lib\10.0.17763.0\um\x64" cudart.lib c10.lib
caffe2.lib torch.lib torch_python.lib caffe2_gpu.lib _C.lib /EXPORT:PyInit_scatt
er_cuda build\temp.win-amd64-3.6\Release\cuda/scatter.obj build\temp.win-amd64-3
.6\Release\cuda/scatter_kernel.obj /OUT:build\lib.win-amd64-3.6\torch_scatter\sc
atter_cuda.cp36-win_amd64.pyd /IMPLIB:build\temp.win-amd64-3.6\Release\cuda\scat
ter_cuda.cp36-win_amd64.lib
   Creating library build\temp.win-amd64-3.6\Release\cuda\scatter_cuda.cp36-win_
amd64.lib and object build\temp.win-amd64-3.6\Release\cuda\scatter_cuda.cp36-win
_amd64.exp
Generating code
Finished generating code

@rusty1s,
yes I'm able to build the cpp extension. I had to remove the following line from torch.h, but otherwise it worked
#warning
"Including torch/torch.h for C++ extensions is deprecated. Please include torch/extension.h"
This is because the warning syntax for MSVC compilers is different. However I get different errors when I compile scatter_cpu
cpu/scatter.cpp(15): error C2450: switch expression of type 'at::Type' is illegal
cpu/scatter.cpp(15): note: No user-defined-conversion operator available that can perform this conversion, or the operator cannot be called
cpu/scatter.cpp(8): error C3861: 'toString': identifier not found
cpu/scatter.cpp(8): error C2440: '': cannot convert from 'initializer list' to 'c10::Error'
cpu/scatter.cpp(15): note: No constructor could take the source type, or constructor overload resolution was ambiguous
cpu/scatter.cpp(28): error C2450: switch expression of type 'at::Type' is illegal
cpu/scatter.cpp(28): note: No user-defined-conversion operator available that can perform this conversion, or the operator cannot be called
etc.. I have no idea how it is working for neon5d

@rusty1s,
can you please try building on windows to see if it is working for you? I'm a bit hesitant to post on the cppextension repo for an build issue with this repo.

I do not have a Windows machine, but im 100% sure that installation works on Windows. This seems to be a weird bug with AT_DISPATCH_ALL_TYPES. Can you try to replace the calls in cpu/scatter.cpp with either
AT_DISPATCH_ALL_TYPES(x.dtype(), "xxx", [&] {

or
AT_DISPATCH_ALL_TYPES(x.scalar_type(), "xxx", [&] {

like here and see id this fixes the issue?

Edit: This seems to be the issue. The argument type recently changed in PyTorch master (see, e.g., here).

AT_DISPATCH_ALL_TYPES(x.scalar_type(), "xxx", [&] {
worked, thanks.
AT_DISPATCH_ALL_TYPES(x.dtype(), "xxx", [&] {
this one didn't.
The other two libs - cluster and sparse also need the same change. Best to update the code in master?

Cool! I will check if this change works on PyTorch stable and update in master.

Does not work on PyTorch 1.0.0. I need to wait before changing this in master. I leave this issue open as a reminder.

Torch-scatter can be compiled with VS2017 (not VS2015)

thanks,I will try it.

Fixed in master.

@ankur6ue @rusty1s  hi guys can you pls share more details on the solution i am trying to install torch scatter 1.1.1 with cuda9.0 pytorch 1.0 but still facing the same errors on top of tat i made few modifications after google and now the error is
"+ 0x14 bytes(s)
0x00007FFED626D721 (0x0000000000000000 0x0000000000000000 0x0000000000000000 0x0000000000000000), RtlUserThreadStart() + 0x21 bytes(s)
nvcc error   : 'cicc' died with status 0xC0000005 (ACCESS_VIOLATION)
error: command 'C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v9.0\bin\nvcc.exe' failed with exit status 5
"
SSupport will be highly appreciated

Is there any more information given by the installation procedure to help us to track down this error?

I have the setup of
pytorch 1.01
windows 10
cuda 9
gpu:geforce 1060
VS:2019
i am not sure y it is searching for c10 which is cuda10

c10 does not mean cuda10, it is a subpackage from PyTorch. My guess is that it is coming from a wrong MSVC version. In addition, we did not support Windows back then (although I know that it was possible to install the packages).
Overall, installation on Windows is a lot smoother now, and I advise you to upgrade your PyTorch version if possible.

@rusty1s thanks for the response , i am not able to uprade my pytorch version since for cuda9.0 the supporting versions are either 1.01 or 1.1.0 both of which i have tried by building torch scatter version pytorch_scatter-1.0.3 to pytorch_scatter-1.1.0 , is there any wheels available for cuda9.0 version

Your system-wide CUDA does not have to match, since PyTorch will install a cudatoolkit by default, i.e. in {miniconda_path}/lib (there should exist some files such as libcusparse.so). You can then simply link your system to this CUDA installation via
export LD_LIBRARY_PATH={miniconda_path}/lib:$LD_LIBRARY_PATH

@rusty1s  thanks for the response the issue is solved for me


We are receiving "[Error: No valid replicaset instance servers found]"  when node js program tries to connect to mongo cluster.

Hi All,
We are receiving :
[Error: No valid replicaset instance servers found]  while connecting from node js .
Details:
Version of node js: 0.10.25
Version of mongoose: 3.8.5
Recent version of mongo db
Any help is appreciated.

What does your connection code look like? And can you actually connect to your replica set from the mongodb shell?

Hi ,
The connection code is :
mongoose.connect(util.format("%s/%s_%s",config.db.mongoUri, S(config.db.tenantId).replaceAll('.','_'), config.db.name),function(err){
Where mongoUri : mongodb://hostname1:port,mongodb://hostname2:port,mongodb://hostname2:port

I have tried supplying options with replicaset , but with no use.
I have tried not using "mongodb:" in the second and third hostnames, but no use.

Yes, I can able to connect to replicaset from shell.
Thanks in advance.

So point number 2 above is one of the issues, the URI format is:
"mongodb://host1:port1,host2:port2,host3:port3"
See http://docs.mongodb.org/manual/reference/connection-string/
Also, try specifying replicaSet option as the node driver docs show: http://mongodb.github.io/node-mongodb-native/driver-articles/mongoclient.html
This stack overflow also has some helpful advice: http://stackoverflow.com/questions/25082295/mongodb-db-open-returns-replicaset-error-but-no-error-in-mongodb-log-files

Thanks a lot, vkarpov.


Code Compilation and Formatting Check Workflows

This includes

Checking for compilation errors on Windows, MacOS, and Ubuntu for the whole project via cargo check. Example of results found here.
Ensuring that all code adheres to a standard format via cargo fmt --check. Example of results found here.

Currently, the CI running cargo test is having some issues with the examples provided in the documentation. Should these examples be fixed, or ignored?

Currently, the CI running cargo test is having some issues with the examples provided in the documentation. Should these examples be fixed, or ignored?

I'm not sure how I feel about clippy since it makes some code quality suggestions I'm not sure I agree with. Also it would take a bit of time to configure it how I would like. I don't think we should be running any tests since this library doesn't really have any tests worth running.
In one of my other projects I do run a light cargo format check. I could see that being helpful here:
https://github.com/StarArawn/kayak_ui/blob/main/.github/workflows/rust.yml

Okay, that sounds reasonable. Based on your feedback, I replaced the call to run tests to just check the code for all targets, and made the code quality workflow run the light cargo format check from your project. I also made the dependencies and workflow triggers consistent from the project to this CI.
How does this look for code quality, and for the CI?

Hey @StarArawn, congratulations for getting through the recent rewrite! I was wondering if you'd still be interested in this feature.

We are still interested in this :) I'm going to push up a couple changes here with @bzm3r.

I couldn't get pushing to your branch working, so I continued this in #248. Thanks!


Add managed instrumented executors

Problem:

For threads we have a global counter. Would prefer a counter per executor service (Use a global pool and local thread number similar to DefaultThreadFactory). Can move threadFactory logic to its own class. Would've extended threadFactory but its a private static class.
nameFormat seems very unintuitive. It's not clear to users that they have to specify one. Also its usage across classes vary. (ScheduledExecutorServiceBuilder doesn't have that requirement).
People who aren't familiar with the ThreadPoolExecutor may run into issues like https://github.com/dropwizard/dropwizard/issues/2553 when using the ExecutorServiceBuilder.

Solution:

Use logic used in DefaultThreadFactory with the ability to name threads
Avoid nameFormat since the NamedThreadFactory deals with this
Create helper methods to managed/instrumented executor services from the Executors class

Result:

User passes a prefix and the threadFactory will add pool/thread number.
Whatever the user passes is what is used in the metric name. The methods getNameWithoutFormat in this https://github.com/dropwizard/dropwizard/pull/3142 will return a string appended to itself, so it may have to be reverted. @pkwarren - Any feedback/concerns?
Includes managed and instrumented executor service factory methods. ThreadPoolExecutor Javadocs urges users to use the Executors factory methods that preconfigure settings for the most common usage scenarios. Also the InstrumentedExecutorService has additional metrics like idle/duration timers which InstrumentedThreadFactory (used in ExecutorServiceBuilder) doesn't have

Don't think anything here can be considered as breaking changes. Can update release notes/docs/tests if you think this is a reasonable approach.
Event if a subset of these changes are acceptable, let me know and I'll make appropriate changes.

In the previous commit I was mainly hoping to restore the behavior from DropWizard 1.x (which used Guava's ThreadFactoryBuilder and the nameFormat required a format string). This was changed when Guava dependencies were removed and led to incorrect names on the metrics for the executor.
To be fully compatible with what is out there, it feels like we should keep (and deprecate) all usage of nameFormat (including ExecutorServiceBuilder constructors) and add a new .withMetricPrefix(String) method to ExecutorServiceBuilder that allows explicit naming of metrics.
Then if nameFormat is not set, we skip wrapping the thread factory (relying on caller to use NamedThreadFactory as you've proposed) and use metricPrefix to name metrics.

I went through the history and totally understand the rationale behind your commit. Let me give this some thought and make some changes locally based on your feedback. Will post back here shortly

Spent some time trying to implement it such that it is backward compatible, but had a few questions. My questions are based on what I expect to be the behavior for end users (Correct me if I'm wrong)

Thread name and metric name share prefix, unless user provides their own threadFactory (That's the current behavior and a good way to correlate logs and metrics)

Achieve this by sharing metricPrefix between threadFactory and metrics

metric/thread name is always unique

Using a default value with global counter or user provided prefix

Questions -

If users leverage the deprecated constructors and calls withMetricPrefix(), what takes precedence for naming the metrics/threadFactory, nameFormat or metricPrefix?

Are we fine with making metricPrefix a constructor argument instead of a builder method. Reasons why -

It's harder to share same metricPrefix with threadFactory and metrics, since threadFactory is created in the constructors whereas metricPrefix is set later on in builder method. Can add null check and move threadFactory creation to the build() method, but would like to avoid that.
We always require metricPrefix. Can add a null check and provide a default metricPrefix, but would like to avoid that.

If it's in the constructor, I can create a default metricPrefix (if not provided), share metricPrefix with thread/metric name and avoid null checks. But if we move it to a constructor and I deprecate nameFormat and add namePrefix, you'll have two constructors that look very similar which is weird.
It feels like it may be confusing to the end user if we try to support both nameFormat and namePrefix (this is subjective). Although this PR changes thread/metric name, end users can still get the same thread/metric name they require by passing appropriate parameters (Can document this in release notes?). Code will be a lot cleaner too. Let me know your thoughts and I can go ahead and make the changes
PS - To keep this PR focussed on nameFormat, I can remove the executor factory methods into it's own PR if needed. Let me know

@pkwarren  - Just checking if you had any feedback/concerns and if/how I should proceed.
Tagging other contributors as well - @joschi @jplock


Quick and dirty sorting for infra

Resolves #978
Trentin asked for surfacing the approved proposals to the top. https://t.me/c/2066716446/6/829
This is a temporary solution since we will replace the queryAPI and thus also change how the feed works.
I split the feed component bc it only affects the feed infrastructure-committee and not the other portals.
This PR also fixes an error flash for events-commitee hasModerator is not a function.
I added to fetch 20 proposals on initial load so it actually shows approved proposals on top. See picture:

@race-of-sloths include

@Tguntenaar Thank you for your contribution! Your pull request is now a part of the Race of Sloths!

Current status: waiting for scoring
We're waiting for maintainer to score this pull request with @race-of-sloths score [0,1,2,3,5,8,13] command. Alternatively, autoscoring [1,2] will be applied for this pull request
What is the Race of Sloths

Race of Sloths is a friendly competition where you can participate in challenges and compete with other open-source contributors within your normal workflow
For contributors:

Tag @race-of-sloths inside your pull requests
Wait for the maintainer to review and score your pull request
Check out your position in the Leaderboard
Keep weekly and monthly streaks to reach higher positions
Boast your contributions with a dynamic picture of your Profile

For maintainers:

Score pull requests that participate in the Race of Sloths
Engage contributors with fair scoring and fast responses so they keep their streaks
Promote the Race to the point where the Race starts promoting you
Grow the community of your contributors

Feel free to check our website for additional details!

Bot commands

For contributors

Include a PR: @race-of-sloths include to enter the Race with your PR

For maintainers:

Invite contributor @race-of-sloths invite to invite the contributor to participate in a race or include it, if it's already a runner.
Assign points: @race-of-sloths score [1/2/3/5/8/13] to award points based on your assessment.
Reject this PR: @race-of-sloths exclude to send this PR back to the drawing board.
Exclude repo: @race-of-sloths pause to stop bot activity in this repo until @race-of-sloths unpause command is called

@race-of-sloths score 3


Error when activating darkstudio on Mac

Hi,
I am trying to setup darkstudio on Mac, but when trying to use
darkstudio::activate()
I receive the following error message
Error in dirname(path) : a character vector argument expected
Is this an issue with elevated privileges as you mentioned in the installation steps?
Cheers,
Andy

I have the same issue on the m1 MacBook

Duplicate of #21


Add QA session timings

Display the two session times on each poster page (in UTC + local timezone, as with ICLR)
Add iCal download links for each session

Solves https://github.com/acl-org/acl-2020-virtual-conference/issues/74
Tests will fail until https://github.com/acl-org/acl-2020-virtual-conference-sitedata/pull/7 is merged

Resulting poster page:

Also added indication for +/- 1 day on localised time - if this isn't desired then it's fully contained within the last commit


[FEATURE] huggingface chat

I think it would be more practical that at the moment of generating the prompt that is going to be passed to the huggingface endpoints they follow their format, I say this mostly because I have been trying to use a huggingface endpoint for that but it doesn't work and gives me strange results.
I only mention this because I was checking your code and I didn't see anywhere that it follows the typical format of
<s>[INST]previous question[/INST] answer</s>
<s>[INST]<<SYS>>system prompt<</SYS>>> question[/INST]answer</s>

This is only specific to Llama2 format, and there are lots of other models on HF. Hence its better for user to specify by themselv on the agent system prompt


Ssl support added process to send response to client when app is ready

I added in a callback process as I call it.  The callback process stores the method and outputstream used for sending a response to the client and then notifies server that of what data the client sent over but doesn't expect a response immediately from the server.  The server can then call sendCallBackResponse(Response response) when it is ready to send a response to the client.

This pull request is just too big, take care not to format the class because now we can not detect what your real change was. In the mean time the SSL support was merged into the master, so if you want to publish you change please create a new pull request for the master branch. If possible with a unit test.


iphone 5s (iphone 9,1) not supported ,iphone 9,2 supported

why

I can't wait for all of these to be closed.

i have 5s 6.2 its not supported atm

https://pbs.twimg.com/media/C3VM72FWQAAad7P.jpg
https://pbs.twimg.com/media/C3VM74oW8AAstR2.jpg
#tihmstar
iphone 6,2

uhm
new commit?
i cant compile the ipa , no way to run a fonctional virtual machine grr

yes 10.1.1

#64

https://www.dropbox.com/s/abf08lkkgz6sc3m/yalu102_alpha by sonyccyong.ipa?dl=0

Why do you share the fcking ipas like this?he said no and no means no!

ohh thanks !

Please

@hcl2k Erase that IPA, it has malware, it can bootloop your device

That's IPA well working on my iPhone 5s, Thanks.

@apaukraine what version of iOS are you running on the iP5s? 10.2?

Yes

I have saved blobs, can try on another versions

@apaukraine weird, the latest commit still doesn't work on my iPhone 5S

@timonlone, You have iPhone 9,1 or 9,2

isn't iPhone9,1 an iPhone 7? (see below). Then why say "That's IPA well working on my iPhone 5s"
iPhone9,1	iPhone 7 (A1660/A1779/A1780)
iPhone9,2	iPhone 7 Plus (A1661/A1785/A1786)
iPhone9,3	iPhone 7 (A1778)
iPhone9,4	iPhone 7 Plus (A1784)

please use #186

this issue is CLOSED

@timonlone I install IPA from dropbox


[Fix #2731] Customise method-creating methods.

This addresses the issue raised in #2731 where methods (often from outside of Ruby itself) may be called within the context of a module that in turn define methods. The Lint/UselessAccessModifier wasn’t taking these into account.
Much like the existing approach taken with blocks/context-creating methods, there’s now a configuration option MethodCreatingMethods for the cop. This setting is an array of methods which, when called, are known to create other methods in the module’s current access context.
# Lint/UselessAccessModifier:
#   MethodCreatingMethods:
#     - delegate
require 'active_support/core_ext/module/delegation'
class Foo
  # this is not redundant because `delegate` defines methods
  private

  delegate :method_a, to: :method_b
end

I realise the option MethodCreatingMethods is not the most obvious of names, so I welcome suggestions on alternatives! Also, if there's anything I've missed (this is my first time mucking about with Rubocop's internals) do let me know.

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Used the same coding conventions as the rest of the project.
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] All tests are passing.
[x] The new code doesn't generate RuboCop offenses.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[x] Updated cop documentation with rake generate_cops_documentation (required only when you've added a new cop or changed the configuration/documentation of an existing cop).

maybe MethodCreatingMethods instead of ClassMacros?

This is a nice addition @pat!
Maybe you can also add a whitelist for RuboCop's own configuration file, for def_node_matcher et. al. as part of this PR? 😊

Thanks for the feedback folks :)
@backus: Just to be clear: did you mean to suggest I rename MethodCreatingMethods to ClassMacros? (I'm currently reading your suggestion as the other way around.) ClassMacros is certainly shorter, but I feel it doesn't quite represent the purpose of the setting - essentially, a list of method calls which create method definitions. Class-level macros, on the other hand, might create methods, but might do other things instead/as well.
@Drenmi: I think I understand your suggestion (but again: I'm new to the Rubocop implementation): are you suggesting I add this setting to .rubocop.yml in the root of the project? I can see how that theoretically makes sense, but are there specific situations where the Lint/UselessAccessModifier cop is finding false positives in the Rubocop source due to this not being possible previously? My current approach to such things - which maybe is not the Right Way? - is to add configuration settings as needed, rather than trying to cover potential situations.

I'm new to the Rubocop implementation): are you suggesting I add this setting to .rubocop.yml in the root of the project?

No worries. We can add it in later. 😀 The def_node_matcher macro works in class bodies, but if it's the only thing after the private modifier, RuboCop will complain.

Guess this will do for now! Thanks! 👍

Thanks for merging :)


Error message if the ToDo EditText is empty.

Error message now appears if Todo EditText is empty after the FAB is clicked.
Updated Strings XML with a new String.
todo_error appears if the ToDo EditText is empty after the FAB is clicked.

👍


Cocoapods podfile required a higher minimum deployment target

Hi !
First, thanks for your plugin it really works fine on android.
I use the 3.0.0 version :)
But i can't run my app in IOS...
Here is the error message :
`Resolving dependencies of Podfile
[!] CocoaPods could not find compatible versions for pod flutter_facebook_login:
In Podfile:
flutter_facebook_login (from .symlinks/plugins/flutter_facebook_login/ios)
Specs satisfying the flutter_facebook_login (from .symlinks/plugins/flutter_facebook_login/ios) dependency were found, but they required a higher minimum deployment target.`
Could you help me please ?

Also, with your 2.1.0 version i don't have the problem...

You can try 194

@tuanitpro ...
Well...
It works fine for me. I do what you told in the 197, i also resolve my flutter doctor issues.
Then it works well.
Thanks a lot man ! Really


mgmt, add convenience code for DiskEncryptionSet

Description
Fix https://github.com/Azure/azure-sdk-for-java/issues/27966
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-resourcemanager-compute

Need this PR in to fix CI failure.

LGTM.