Update should not sort keys when upsert is true

Versions
v0.8.0
What did you do?
See repro in https://github.com/FerretDB/FerretDB/pull/1735
What did you expect to see?
The order of keys unchanged.
What did you see instead?
That the keys were sorted.
Environment
- OS: Ubuntu
- Architecture: x86_64
- Version: 22.10
- Go: go1.19.3 linux/amd64
- Git: 2.37.2
- Deployment: Docker
- Deployment details: 20.10.16

TODO for preparation: check similar cases (e.g. update, not only upsert)

Replace test with upsert:
func TestReplace(t *testing.T) {
	setup.SkipForTigris(t)

	t.Parallel()
	ctx, collection := setup.Setup(t, shareddata.Scalars)

	expected := bson.D{{"_id", int32(1)}, {"c", int32(1)}, {"b", int32(2)}, {"a", int32(3)}}

	_, err := collection.ReplaceOne(ctx, bson.D{{"_id", 1}}, expected, options.Replace().SetUpsert(true))
	require.NoError(t, err)

	res := collection.FindOne(ctx, bson.D{{"_id", 1}})

	var actual bson.D
	require.NoError(t, res.Decode(&actual))

	assert.Equal(t, expected, actual)
}

To run mongo-test task you can temporary replace tls arguments and just connect to task run instance
 mongo-test:
     desc: "Run legacy MongoDB shell (`mongo`) with test script"
     cmds:
       - >
         docker compose run --rm legacy-mongo-shell
-        --tlsCertificateKeyFile /etc/certs/server.pem
-        --tlsCAFile /etc/certs/rootCA.pem
-        --tls
-        'mongodb://username:password@host.docker.internal:27017/?authMechanism=PLAIN'
+        'mongodb://host.docker.internal:27017/?heartbeatFrequencyMS=300000'
         /legacy-mongo-shell/test.js

``

https://github.com/FerretDB/FerretDB/pull/1946#discussion_r1104026810


RFC: Reduce Build Complexity of .Net Runtimes

Readable

@fg-j I think I would like to do this as a smaller chuck of work that would make the re-ordering easier to do in the future.

IMO it would be nice to combine all of these concerns into one RFC

I have spent a little time looking at the re-architecture and I feel like I would just need to tack an additional RFC on to the end of this RFC that is contingent on this RFC passing. I am happy to do that but I think that the purposed re-architecture that @fg-j is purposing becomes a trivial RFC that can reference this RFC and the SDK RFC once they both pass. Please let me know you would like me know what y'all are feeling and I will go with the popular vote.

+1 I'd like it all to happen in this RFC. Gives us an opportunity to evaluate if there are edge cases we're missing and how we could cover them.

2022-07-26 Working Group notes: waiting for approvals

@paketo-buildpacks/dotnet-core-maintainers Am I good to merge this and get cracking on implementation?

2022-08-02 WG Notes:
Ready to merge.


Release Service Bus Track 2 Library Preview 1 for Java

This is a parent issue tracking the work planned for Service Bus to release the first preview in March following the Java Azure SDK guidelines
Code complete date: March 23th, 2020
API design

[ ] Revisit the client structure i.e the various entry points in the package

Features

[ ] Send a message to Queue and Topic

[x] send and sendBatch
[x] schedule
[x] cancelScheduledMessage

[ ] Receive and settle message from Queue and Topic

[x] receive and auto-complete on each message
[x] peek and peekBatch
[x] abandon
[x] complete
[x] defer
[x] deadLetter
[x] renewMessageLock
[x] receiveDeferredMessage and receiveDeferredMessageBatch

[x] Support Azure Identity like we did in Event Hubs
[x] Support retry configurability on clients like we did in Event Hubs
[x] Support Web sockets & proxy like we did in Event Hubs

Out of scope

[ ] Receive and settle message from session enabled Queue/Topic

@hemanttanwar, @conniey, Can either of you paste the link to the released package in maven?


Anthropic model setup producing error

I've enabled Anthropic model usage and have it configured to use the latest Haiku model throughout. I've generated a fresh API key which I've set up, and have tried restarting Obsidian to allow changes to propagate. I'm getting what seems to be a generalized error for the tool. Can confirm that I'm able to successfully use OpenAI models without errors.

@lyonisaac will look into it thanks for reporting
our most tested models are:

gpt-4o
codegemma (open source)
groq (llama-70b)

i will test with haiku today


unknown error: row read wait bigger than connection timeout

More complicated queries result in a timeout:
engine.execute("SELECT o.symbol, coalesce(max(y.eod), '1900-01-06') - INTERVAL 5 DAY FROM optionable o LEFT OUTER JOIN yahoo y on y.symbol = o.symbol GROUP BY o.symbol")

result in:
Traceback (most recent call last):
  File ".venv/lib/python3.7/site-packages/mysql/connector/connection_cext.py", line 489, in cmd_query
    raw_as_string=raw_as_string)
_mysql_connector.MySQLInterfaceError: unknown error: row read wait bigger than connection timeout

@KIC apologies for the delayed response. This is a Dolt performance issue, rather than a Doltpy issue per se, though we could increase the timeout on the connector, or expose that configuration to the user in case they want to execute longer running queries.
Do you have any idea how long this took? Did you try running the query directly via dolt sql?


v0.60.0 Markdown code fences no longer highlighted using Pygments

I use pygments to highlight code fences. For example, I have this simple markdown code block:

```yaml
key: value
array:
- val1
- val2
```

With v0.59.0, I get this nicely syntax highlighted block via Pygments:

key: value
array:
- val1
- val2

After upgrading to v0.60.0 I get a lot more HTML generated, but the colors aren't really doing anything useful (for example, YAML field names are not highlighted):

key: value
array:
- val1
- val2

As for my config.yaml, I don't have any blackfriday settings or goldmark settings. I just have these:
pygmentsUseClassic: true # disable Chroma
pygmentsCodeFences: true
pygmentsCodeFencesGuessSyntax: true
pygmentsUseClasses: false
pygmentsStyle: perldoc

I just saw release notes:

We have removed the option to use Pygments as a highlighter. #4491

This is pretty sad because Pygments can actually highlight YAML files.
Chroma’s YAML highlighting is in a really poor shape. (https://github.com/alecthomas/chroma/issues/118)
I have a feeling like this will impact websites like Kubernetes.io, Istio.io and Knative.dev once they migrate.

I'm new to the party here, @ahmetb . What's the consensus on getting YAML highlighting in Hugo > 0.60? Or is this just no longer a possibility at all?

@chipzoller Actually Chroma has fixed their YAML highlighting issues. https://github.com/alecthomas/chroma/pull/307 However the fix is not in v0.7.0 release of Chroma, which is used by the newer version of Hugo.
I've locally recompiled Hugo from master, with the updated Chroma version, so it works like a champ. I think it'll eventually get caught up here. Maybe we can make Hugo import a non-tagged commit for Chroma, idk. Maintainers should decide.

Great info to know, @ahmetb. Can you show an example of the "after" highlighting abilities of 0.7.0 using a K8s manifest as an example? I'll be including many such snippets in my markdown in Hugo, so that'll be the litmus test for me as to if I want to invest in recompiling it myself.

It only highlights field names and quoted strings with color. It’s a lot better than no highlighting. It’s hard to notice the full support is missing.


Create PR-Review process

Introduction
After internal discussions on the 22nd of December, 2023, it was clear that we should have a semi-formalized process for creating and reviewing PRs. This pull request provides such a procedure.
Design goals:

Address the concern of existing members that new members are overriding their opinion
Address the concern of new members that existing members can not review pull requests in time due to their availability
Address the concern of members that pull requests are not documented properly
Address the scope of reviews and when they can be dismissed.

None goals:

Create a pull request pattern, this should be done on a project level.

The doc here seem a bit too strict. Imo, i think a better balance is:

PRs should sit for 24 hours since initial creation before they can be merged. If it is a critical issue such as a crash fix, approvals from two other maintainers that didn't make PR can override the time limit. This will be handled on a case-by-case basis. Use maintainer ping if needing approvals. There should be enough people that can hop on phone, read up on what is happening, and approve. 24 allows anyone in any time zone to have an opportunity to read PR and say their concerns or approve it. This is respectful of people and their free time.

Reviews left on PR that as not prefixed with "Passing remark" will initiate a 24 hour waiting period on PR to give time for PR maker to answer the comment/fix the remark. This gives time to the reviewer to actually reach the response/new work and say if their concerns are resolved or not. Generally, people are going to reply earlier than 24 hours but in the case it does exceed the limit, PR maker can ask another maintainer to read the original review and the responses/fixes to see if it did resolve issue. And if that maintainer approves PR, then PR can be merged despite original review not closed by original reviewer within 24 hours.

PR titles and descriptions should be clear and easy for us to look back on in months and know what the PR was for and why. Description should be at least 1 sentence minimum. Larger PRs would be expected to have larger descriptions detailing what is going on and why. This would be handled case-by-case basis. if one feels a PR description is too short for a gigantic PR that has a huge number of changes, they can leave a comment asking for PR maker to expand the description and that's it. No issue.

Breaking change PRs all must list what the breaking change is, why it is decided to be broken, and what the workaround is. Modders will be looking for the PR to know what the workaround is. They won't be combing the changelog nor discord. They are going to do a Blame in GitHub or a keyword search for the PR and will be reading the PR body for the workaround.

Lets change this to be a 32 hour slowmo based on a PR's last commit if the PR has the "Non-trivial" tag applied. This tag can be added by any maintainer based on their discretion of if a PR is trivial or not. Maintainers that make a PR can also apply the tag to their own PR. This lets trivial small PRs to be merged quickly for quick fixes/testing/etc without being bogged down


Wrong behaviour inside of string

After some of recent updates plugin works wrong inside of the string.
Works fine on new line or in the end of the line.
https://user-images.githubusercontent.com/51912173/131106671-0a83dbca-4ca3-464c-8f99-16d61c95965f.mp4

you should check setting ignored_next_char. it will skip if next char is a dot.


merge package from arlac77/npm-package-template

package.json

chore(package<EMAIL_ADDRESS>chore(scripts): cover@#overwrite c8 --temp-directory build/coverage ava && c8 report -r lcov --temp-directory build/coverage
chore(package): add nyc from template
chore(package): set $.ava.require='esm' as in template
chore(package): set $.ava.files='tests/-test.js,tests/-test.mjs' as in template
chore(package): set $.ava.extensions='js,mjs' as in template

Coverage decreased (-0.6%) to 47.348% when pulling 4896b9e0d59f27c2dbed8ee6d28e7433a11bacad on template-sync-1 into 215232cf6080dff82900f756e0cc97aa94333ba8 on master.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[ALLUXIO-2572] Fix return value when error

https://alluxio.atlassian.net/browse/ALLUXIO-2572

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/14400/
Test PASSed.

Merged build finished. Test PASSed.

@uronce-cc do you have more comments?


Make Control-A in the interactive client exit and put the cursor at the start of the line.

When in interactive mode, we could make Control-A exit, insert the current selection, and put the cursor at the start of the line. This would save our users one keystroke if they want to edit the current selection before executing it. (This could later be extended to other "horizontal" cursor movement keys, like left arrow to insert the current selection and put the cursor one character back from the end.)
I think this would need a change in command/client/search/interactive.rs to catch the Control A and return some extra information to command/client/search.rs for that. That would need to be encoded into the string that eprintln! sends to the shell.
I guess we could send an integer and a string to the shell, instead of just the string; the integer would be the cursor position. Happy to entertain other suggestions.
@ellie  suggests we should be able to update the shell plugins at the same time as updating atuin itself for all shells except for nushell.  One way to deal with that would be to add an option to the command line client that selects the format to return (plain string vs position + string); the current nushell plugin would then just not see the new output because it doesn't ask for it.

itself for all shells except for nushell

Why not nushell? I think that this would be very important for nushell bc its still in beta and is subject to change

NuShell is the only one we can't automatically change the shell bindings for with an atuin upgrade
Doesn't mean we shouldn't implement this for it, but something to bear in mind


Security evaluation and risk profile

Security evaluation and risk profile. And have a good discussion on the security versus usability of the firmware.
Not all risks will be mitigated, due to limited resources. This project needs to be balanced.

Added a fix to the UI, so there are no more clear text UI passwords visible. Thanks to @DutchessNicole for the simple UI fix.


On parent, override instead of resetting

When setting a new parent, the default behavior for now is to set reset the values of the child to the parent values. Instead, each field could be checked and a new list of overrides created depending on whether the data is the same as the parent's or not. A global button called "reset all" could be implemented to easily reset all values.
This would prevent loss of data when integrating the SOVariant in an existing workflow.

implemented in #10


feat: oauth integration

Implements #13.
Todo:

[ ] Add a sexy sign-up/sign-in button
[ ] Change to GitHub provider
[ ] Figure out how things interplay with Stripe
[ ] Add integration instructions

Preview deployment is still not working even though it's a non-forked branch, @kt3k. My understanding is that this should work.

@iuioiua This still looks from your fork (prefixed with your fork name)

BTW currently saaskit project in Deploy doesn't seem setting any env vars. Do you know who manages those?

Doh! I thought it wasn't a fork 😓

BTW currently saaskit project in Deploy doesn't seem setting any env vars. Do you know who manages those?

It might be Andy. But he hasn't told me of doing so. I'll ask him once he's back.

#16 is a prerequisite for this PR.

Closing in favour of #35.


Proposal: hide organizations for users are not part of

In my typical scenario, will be extremely useful if a user is not allowed to see all the organizations through the explore page, but only those he is part of.
May be needed, for example, add a "private" flag to the organization, just like repos already have.
Thank you!

Seems to be a duplicate of #714

@DblK yes. Let's continue there. @usef


Roman to Integer in Java

💥 Proposal
Given a roman number in string using characters ('I', 'V', 'X', 'L', 'C', 'D', 'M'), convert it to integer.
Language:Java
(A clear and concise description of what the proposal is.)
Have you read the Contributing Guidelines on Pull Requests?
Yes
(Write your answer here.)

/assign


Clicking "Select Top 1000 Rows" slow to respond

Issue Type: Bug
Reproduction:

Connect to a database (local MS Sql Server)
Open the Databases folder
Open a database
Open the Tables folder
Right click on a table, click "Select Top 1000"

Expected Behaviour:
A tab to open showing the query and results
Actual:
Nothing appears to happen. While writing this report, eventually multiple tabs opened showing the query but this was over 3 mins after clicking the "Select Top 1000" option.
The issue has been ongoing for several weeks. I have restarted and updates Azure Data Studio to the latest version but the bug persists.
I also use Micrsoft SQL Server Management Studio and do not experience any issue viewing table rows, so this appears to be related to Azure Data Studio, and not the database.
Azure Data Studio version: azuredatastudio 1.36.1 (e56e1d931bbb110fe8f24751423c4e184bb8f22d, 2022-04-21T21:25:10.399Z)
OS version: Windows_NT x64 10.0.22000
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-10610U CPU @ 1.80GHz (8 x 2304)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.61GB (1.06GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (1)

Extension
Author (truncated)
Version

profiler
Mic
0.11.0

Console and Application logs attached
ApplicationLogs.zip
console.log


Enable Micropython CI workflows

Currently all Micropython's CI workflows were disabled on https://github.com/lvgl/lv_micropython/commit/43ec34e2760f58299a4f780b602cb335b757476a
With some work, at least some of them can be enabled on lv_micropython.

It's probably not worth re-enabling the ports ones (I think ours effectively do the same thing), but the formatting and code size ones look useful.

It's probably not worth re-enabling the ports ones

My thought was to enable the port ones, specifically ports we don't support yet.
That way we would know it builds on other platforms, and be ready to the moment someone starts using lv_micropython with a new port.

CI for stm32/esp32/unix ports could also be useful for providing prebuilt firmware effortlessly, as was requested before.

@amirgon If you have a moment, can you take a look at the ESP-related commands in this Action, and let me know if I missed something? I think I've followed all the steps correctly but it's not finding some header files.
ESPIDFMOD-GEN build-GENERIC_SPIRAM/espidfmod/mp_espidf.c
In file included from ../../lib/lv_bindings/driver/esp32/espidf.h:101:
/home/runner/work/lv_micropython/lv_micropython/esp-idf/components/esp32/include/esp_clk.h:1:2: warning: #warning esp_clk.h has been replaced by esp32/clk.h, please include esp32/clk.h instead [-Wcpp]
 #warning esp_clk.h has been replaced by esp32/clk.h, please include esp32/clk.h instead
  ^~~~~~~
In file included from ../../lib/lv_bindings/driver/esp32/espidf.h:104:
/home/runner/work/lv_micropython/lv_micropython/esp-idf/components/driver/include/driver/gpio.h:17:10: fatal error: esp_err.h: No such file or directory
 #include "esp_err.h"
          ^~~~~~~~~~~

@embeddedt my thinking was to enable Micropython's CI instead of creating a custom one.
The advantage is that we would follow upstream more closely.  This is especially important now that ESP32 port on Micropython is migrating to CMake.
What do you think?

The problem is that MicroPython's CI is not building multiple boards (at least when I investigated it). It is just building enough to verify that compilation still succeeds in general. E.g. for STM32 it is only building pyboard variants.
Do you think it's better to modify their CI script to do that than to make one for our purposes?

Do you think it's better to modify their CI script to do that than to make one for our purposes?

I don't know about STM32, but for ESP32 the CI is maintained and modified according to build changes.
For example, when Micropython supports a different (or an additional) ESP-IDF version, this is reflected in CI changes.
See for example: https://github.com/micropython/micropython/commit/aa3d6b6aa5ec9fdf15b5c2d30a1c9f9c9064cd3c#diff-398f66844239bcfec34a0362acfee807d8d697c15490291b03bce4904182a45f
So my thinking was - instead of maintaining the CI script separately ourselves and follow build changes, let's reuse Micropython's CI.
Instead of modifying their scripts, I thought of trying to call these scripts and add additional actions (such as actions/upload-artifact@v2) where needed.
Creating prebuilt firmware would be nice and useful but I think it's a secondary goal.

@kisvegabor, @embeddedt I've enabled ESP32 workflow and I was wondering about providing prebuilt firmware

Do we really want to provide prebuilt firmware? Or leave it to the users as we do today? Would we be able to provide good support for novice users who use prebuilt firmware? One of the advantages of leaving the build step to the users is that today we almost always interact with users who have at least some technical understanding.
If we do want to provide prebuilt firmware, for which boards?
Technically how do we create a prebuilt firmware workflow? Where would we publish the pre-built firmware? What would trigger the workflow?

What do you think?

Technically how do we create a prebuilt firmware workflow? Where would we publish the pre-built firmware? What would trigger the workflow?

Take a look at this example:
https://github.com/lvgl/lv_port_stm32f746_disco/blob/master/.github/workflows/stm32_port.yml
We can then use https://nightly.link/ to provide direct links instead of needing to browse to the action artifacts.

Take a look at this example:

What triggers the workflow? Every push? that could create lots of artifacts. Or do we need some release schedule?
Also, I think that GitHub holds artifacts for no longer than 90 days, so I'm not sure about storing artifacts in GitHub.


parallelize SimTools.py

@cekees @mfarthin
A branch for searching out file i/o in SimTools.py to support parallelization

notice it merges into branch nsps_numerical_fluxes and not into master

Thanks! I'm going to go ahead and merge since I'm merging the other branches into master.


moved client installations to recipes in order to catch defined version

whitespace issues. whoops. i'll create a new pull request with fixes for those.


Add debug logs to entities that are discarded during the process execution.

Add debug lines for vms are discarded in the if conditions, to help understand and debug situations where this happend.
here are the conditions for vms https://github.com/newrelic/nri-vsphere/blob/e3c4a5ec1b6b35c96f53b6478766866de060602c/internal/process/vms.go#L43
I would also be valuable to add this logs on very type of entity that is discarded due to this kind of checks.

this issue has been closed with #76


after modify docs no need build project always

Describe the PR
e.g. after modify docs no need build project always.
Relation issue
e.g.
Additional context
when modify api comments need build project

@wksw lint check failed, please fix by executed gofmt  cmd.
/home/travis/gopath/src/github.com/swaggo/swag/gen/gen.go
^ - Repo contains improperly formatted go files

Codecov Report

Merging #167 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #167   +/-   ##
=======================================
  Coverage   89.04%   89.04%
=======================================
  Files           5        5
  Lines         913      913
=======================================
  Hits          813      813
  Misses         62       62
  Partials       38       38

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 45ceeeb...09c411e. Read the comment docs.

please create the new PR for it because toward master branch so far away.


to_numpy method of CompositeSpec does not work

Describe the bug
The to_numpy method of CompositeSpec results in an error. The offending line seems to be in torchrl/data/tensor_specs.py in the to_numpy method of CompositeTensorSpec. Specifically, in the dictionary comprehension, the _to_numpy method of each of the child specs is being called. It does not appear, however, that TensorSpecs have a _to_numpy method.
To Reproduce
import numpy as np

from torchrl.data import BoundedTensorSpec, CompositeSpec

from gymnasium import spaces

box1 = spaces.Box(low=np.zeros(2), high=np.ones(2))
box2 = spaces.Box(low=np.zeros(2), high=np.ones(2))
spec1 = BoundedTensorSpec(minimum=torch.zeros(2), maximum=torch.ones(2), dtype=torch.float32)
spec2 = BoundedTensorSpec(minimum=torch.zeros(2), maximum=torch.ones(2), dtype=torch.float32)
spec = CompositeSpec(x=spec1, y=spec2)

td = TensorDict({'x': torch.zeros(1, 2), 'y': torch.zeros(1, 2)}, batch_size=(1,))
spec.to_numpy(td)

Additional context
Add any other context about the problem here.
Reason and Possible fixes
Unless this breaks something that I'm not aware of, just changing _to_numpy() to numpy() resolves the issue.
Checklist

[ x  ] I have checked that there is no similar issue in the repo (required)
[  x ] I have read the documentation (required)
[  x ] I have provided a minimal working example to reproduce the bug (required)

Hi, I'll have a look at what's going on and see if I can fix it!


ROS encryption

Hi all,
I have a very general question about ROS and its messaging system.
Is there any chance to communicate between nodes, services and so on in a encrypted way that no external person/software can sniff on data which is problished by the ROS participants?
Regrads
matl

Please ask questions on http://answers.ros.org according to our support policy: http://wiki.ros.org/Support We try to keep the issue tracker reserved for bugs and feature developments.


Improvement for k8s.io/docs/reference/kubectl/cheatsheet/

Need examples of kubectl cp

/sig cli
/triage accepted
Page to update: https://k8s.io/cheatsheet

/retitle Mention “kubectl cp” in https://k8s.io/cheatsheet


Validate if payment() is null

Right now this results in a null-pointer error, which is useless.

Done, ready to push.

Fixed and pushed.


Bug fix in codegen helper: delete LOCAL statement

LOCAL statement was not deletected correctly
TODO: IR generation still fails with

@georgemitenkov : I fixed LOCAL node deletion but IR generation still fails with below error:
$ ./bin/nmodl ../test.mod --output tmp llvm --ir --vector-width 8 passes
...
VOID nrn_state_hh(INSTANCE_STRUCT *mech){
    INTEGER id
    INTEGER node_id
    DOUBLE v
    for(id = 0; id<mech->node_count-7; id = id+8) {
        DOUBLE yyy
        node_id = mech->node_index[id]
        v = mech->voltage[node_id]
        yyy = 12
    }
    INTEGER epilogue_node_id
    DOUBLE epilogue_v
    for(; id<mech->node_count; id = id+1) {
        DOUBLE yyy
        epilogue_node_id = mech->node_index[id]
        epilogue_v = mech->voltage[epilogue_node_id]
        yyy = 12
    }
}
terminate called after throwing an instance of 'std::runtime_error'
  what():  Error: incorrect IR has been generated!
Stored value type does not match pointer operand type!
  store double 1.200000e+01, <8 x double>* %yyy, align 8
 <8 x double>
Aborted

If you have quick fix, feel free to push to this branch.
fixes #594

@pramodk I think that we might want to avoid local allocations for now in kernels? (By that I mean doing a separate PR that fixes this bug and allocations all together - I think these issues are quite linked). My reasoning is the following:

Local allocations should be pulled out of the loops (as mentioned with the benchmark bug: I guess we can discuss this in more detail tomorrow during the sync)
Local allocations must also be renamed in the remainder of the loop: here we have yyy that is vector in the first loop and scalar in the second. Since the name conflicts, the second (scalar)alloca has a name yyy1 due to LLVM’ renaming. This means that storing to yyy is now storing to the vector and is wrong!

This can be avoided by:

Finding all local variables on the loop
Renaming remainder loop ones if there is an epilogue loop
Pulling the allocations outside

My personal view is that it’s better done on NMODL level, since we just move CodegenVarList blocks and do renaming on them too. What do you think?
Alternatively, we can just add this to this PR as you suggested😊

By that I mean doing a separate PR that fixes this bug and allocations all together - I think these issues are quite linked

Now I remember discussing this. Agree - we can split these issues.
We can discuss this tomorrow but my question would be - how this is handled in LLVM? I don't think LLVM pass pulls this outside? i.e. there must some pass to get these local variables optimised out?


typo: Correct "recieved" to "received" in MistralRequestValidator

Corrected a typo in the MistralRequestValidator class. The word "recieved" was misspelled and has been changed to "received" to ensure proper spelling in the comment "if we have an assistant message and we have not received all the function calls".

Thanks Charles for the fixes!
I would suggest combining the few typo-fix PRs into one, so that they are easier to review, as they don't require as much modularization as code change?
I am new to this repo, though, so open to comments from more active contributors.


Weird CORS issue with Lighthouse search

lbry.tv
Type hello in a normal pace.  Everything works fine (first error is expected):

Rinse and repeat:  backspace x3 + "llo".

Weird CORS issue (I'm at lbry.tv, and previous same query worked):

I'm guessing some spam-checking mechanism is kicking in, but I encountered this while really searching for something.  The steps above is just the fastest way to reproduce the same error.

this is a throttle from LH to prevent spam. The right solution would be https://github.com/lbryio/lbry-desktop/issues/6314
In this case, we can show a message that says "please try your search again in 1 minute"

A PR is coming up to do exactly as 6314.
However, I'm still frustrated at being penalized just for making typos.  Example:  I was searching for the binance dude, "CZ".  Since 2 characters are not allowed, plus I can't recall his exact name, I did some trial-and-error but was penalized by the third attempt.

I think once we get that Change in, we can relax the spam  rules. We'd have to see about the 2 word search too...

Close
Not seeing the lockup anymore, since we are only sending queries 1s after typing idle


r75

don't let this great module get outdated : )
Its been really helpful for making lines render consistantly across OSX and Windows

This looks awesome! Have you noticed the npm test demo is half size? I think it may just be due to the three-orbit-viewer (so unrelated to your PR).
I can review more carefully on Monday or Tuesday.

No need to rush with the review : )
I didn't notice the size difference ¯_(ツ)_/¯

I went through and cleaned this repo up to match some of my new JS practices (new linter, some different ways of building ThreeJS demos across versions, etc).
Note: This should still work as before, where you need a ShaderMaterial for the basic shader (not RawShaderMaterial).


Template not found during load of published map

Errors from the JavaScript console in Chrome:
common_vendor.js:152277 Deprecated include of L.Mixin.Events: this property will be removed in future releases, please inherit from L.Evented instead. Error
    at checkDeprecatedMixinEvents (https://nick.localhost/assets/1.0.0-assets.88/javascripts/common_vendor.js:152279:47)
    at Function.push../node_modules/leaflet/dist/leaflet-src.js.Class.extend (https://nick.localhost/assets/1.0.0-assets.88/javascripts/common_vendor.js:152207:3)
    at Object../node_modules/internal-carto.js/node_modules/torque.js/lib/torque/leaflet/canvas_layer.js (https://nick.localhost/assets/1.0.0-assets.88/javascripts/common_vendor.js:103823:25)
    at __webpack_require__ (https://nick.localhost/assets/1.0.0-assets.88/javascripts/builder_embed.js:79:30)
    at Object.<anonymous> (https://nick.localhost/assets/1.0.0-assets.88/javascripts/common_vendor.js:104260:1)
    at Object../node_modules/internal-carto.js/node_modules/torque.js/lib/torque/leaflet/torque.js (https://nick.localhost/assets/1.0.0-assets.88/javascripts/common_vendor.js:104789:30)
    at __webpack_require__ (https://nick.localhost/assets/1.0.0-assets.88/javascripts/builder_embed.js:79:30)
    at Object../node_modules/internal-carto.js/node_modules/torque.js/lib/torque/leaflet/index.js (https://nick.localhost/assets/1.0.0-assets.88/javascripts/common_vendor.js:104084:5)
    at __webpack_require__ (https://nick.localhost/assets/1.0.0-assets.88/javascripts/builder_embed.js:79:30)
    at Object../node_modules/internal-carto.js/node_modules/torque.js/lib/torque/index.js (https://nick.localhost/assets/1.0.0-assets.88/javascripts/common_vendor.js:103799:1)
checkDeprecatedMixinEvents @ common_vendor.js:152277
common_vendor.js:116617 carto.js 4.1.11-0
common_vendor.js:178203 Deprecation warning: value provided is not in a recognized RFC2822 or ISO format. moment construction falls back to js Date(), which is not reliable across all browsers and versions. Non RFC2822/ISO date formats are discouraged and will be removed in an upcoming major release. Please refer to http://momentjs.com/guides/#/warnings/js-date/ for more info.
Arguments:
[0] _isAMomentObject: true, _isUTC: false, _useUTC: false, _l: undefined, _i: 2019-05-15 19:19:30 UTC, _f: undefined, _strict: undefined, _locale: [object Object]
Error
    at Function.createFromInputFallback (https://nick.localhost/assets/1.0.0-assets.88/javascripts/common_vendor.js:178230:94)
    at configFromString (https://nick.localhost/assets/1.0.0-assets.88/javascripts/common_vendor.js:180088:11)
    at configFromInput (https://nick.localhost/assets/1.0.0-assets.88/javascripts/common_vendor.js:180457:9)
    at prepareConfig (https://nick.localhost/assets/1.0.0-assets.88/javascripts/common_vendor.js:180440:9)
    at createFromConfig (https://nick.localhost/assets/1.0.0-assets.88/javascripts/common_vendor.js:180407:40)
    at createLocalOrUTC (https://nick.localhost/assets/1.0.0-assets.88/javascripts/common_vendor.js:180494:12)
    at createLocal (https://nick.localhost/assets/1.0.0-assets.88/javascripts/common_vendor.js:180498:12)
    at hooks (https://nick.localhost/assets/1.0.0-assets.88/javascripts/common_vendor.js:177926:25)
    at child.render (https://nick.localhost/assets/1.0.0-assets.88/javascripts/common.js:50587:20)
    at child.render (https://nick.localhost/assets/1.0.0-assets.88/javascripts/common.js:50888:28)
warn @ common_vendor.js:178203
builder_embed.js:913 Dashboard has some errors: Error: Template 'tpl_d2870a2b_f042_447f_a9b3_748baadf1f63' of user 'nick' not found
    at Object.error (common.js:50164)
    at Object.error (common_vendor.js:131582)
    at common_vendor.js:119711
    at Array.forEach (<anonymous>)
    at Engine.<anonymous> (common_vendor.js:119710)
(anonymous) @ builder_embed.js:913
common_vendor.js:119712 Uncaught (in promise) WindshaftErrorcontext: undefinedmessage: "Template 'tpl_d2870a2b_f042_447f_a9b3_748baadf1f63' of user 'nick' not found"origin: "windshaft"subtype: undefinedtype: "unknown"_error: {type: "unknown", message: "Template 'tpl_d2870a2b_f042_447f_a9b3_748baadf1f63' of user 'nick' not found"}__proto__: Object
embed:1 This site does not have a valid SSL certificate! Without SSL, your site's and visitors' data is vulnerable to theft and tampering. Get a valid SSL certificate before releasing your website to the public.

This is fixed, by updating the tiler.internal section of the app_config.yml to hit the windshaft container directly, over HTTP. This is similar to a fix discussed in issue 22 on the sverhoeven repo:
https://github.com/sverhoeven/docker-cartodb/issues/22#issuecomment-492396542


Add notebook timing.

@ehrlinger,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Egg's Lap Mod

This Isn't Really A Issue, But Is It Possible To Add This Mod To The Port Menu?

already been requested before & if i recall correctly has been stated to not be happening. if the port is open-sourced one day, it would be pretty cool to see it added though

Thanks for letting me know.


Hitman Absolution: aim mode rendering broken.

The right mouse button aim mode rendering is affected by a screen wide glitch, making it nearly unusable.
From observation, the behavior of the glitch seems consistent with the depth buffer not being cleared between frames. For instance, walking towards objects makes them render correctly, while walking back makes everything go blank.
Software information
Hitman Absolution (Steam); happens even with lowest settings
System information

GPU: GTX 1060
Driver: 396.24.02
Wine version: wine-3.11 (Staging)
DXVK version: 0.61 and master

Apitrace file(s)
I tried to make apitrace about 10 times, but the furthest I could get before crashing was the main menu, and that was on one lucky attempt.
Log files

d3d11.log: HMA_d3d11.log.gz
dxgi.log: HMA_dxgi.log.gz

Can't debug without apitrace.

Any idea how to make it work? It seems it has a high chance of crashing on transitions between the intro cinematics or menu screens, no matter if I press skip or try to wait it out naturally. Apitrace mentions getting exception c0000005 (access violation it seems) in stderr.

It might be impossible to make it work, but that also means that it's going to be impossible for me to ever do something about this issue.

You could try making a D3D11 Renderdoc capture on Windows, it's not going to be the most useful thing in the world but I could at least see what the game is trying to do.

OK, so since intro videos are all in separate files, I removed or replaced all of them that delayed loading the level, and finally got it to run long enough to record a trace with the bug:
https://drive.google.com/file/d/1dIJ4g7HFxOGcu8uk4qaace2Q1kQktDvu/view
When playing back with DXVK the zoom in rendering bug occurs as I see it in actual play. The playback also has some weird flickering on the road etc, but that does not actually happen.

Thanks, will look into it.


Fix README.md

Invalid Links in README.md
The Discord Server link leads to an invalid invite
and Official Documentation leads to your subdomain which does not exist as of now
Please update it 😀

Currently they are placeholder. Will be pushing out docs and discord server sometime soon 👍

closing this issue since the pr is merged which shows a badge of coming soon


Am I the only one where both pyls and pylsp are not attaching?

nvim --version: v0.5.0-dev+1260-ga6504ec33
nvim-lsp version(commit hash): 0a921bf2be74d293c8a8d6dd70122c1d01151a30
What language server (If the problem is related to a specific language server): pyls/pylsp
Can you reproduce this behavior on other language server clients (vscode, languageclient-neovim, coc.nvim, etc.): No
Can you reproduce this behavior on other language servers offered in the nvim-lspconfig repo? (pyls -> pyright): Yeah
Is the problem isolated to a particular language server: Just the python ones
Operating system/version: Ubuntu 10.04.2 LTS (WSL)

How to reproduce the problem from neovim startup
Open any Python file with the following config
nvim_lsp["pylsp"].setup{}
Actual behaviour
:LspInfo returns that there are 0 clients attached to server (and active), but the command is detectable and exectable.
Expected behaviour
The client attached
Minimal init.vim or init.lua and code sample

local nvim_lsp = require'lspconfig'
nvim_lsp["pylsp"].setup{}

Health check

Checkhealth result

`INFO: pylsp: configuration checked`
LSP log

Log file

I looked through it, there didn't seem to be much useful. There were some error messages from `rpc.lua` line 457 that `pylsp` was missing some modules like `yapf` and other formatters, but I fail to see why that would matter.

Worth to know that when I manually run :LspStart pylsp it works.

I'm guessing this is a root-directory issue. Is your project in a git config or does it havea file that mathes root directories from CONFIG.md?  Did you try the debugging steps in the readme?

Hey, sorry for getting back so late, I'll check these out tonight. I'll also give the log.
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Michael Lingelbach @.>
Sent: Friday, May 14, 2021 3:50:48 PM
To: neovim/nvim-lspconfig @.>
Cc: Sammy Al Hashemi @.>; Author @.>
Subject: Re: [neovim/nvim-lspconfig] Am I the only one where both pyls and pylsp are not attaching? (#893)
EXTERNAL EMAIL:
I'm guessing this is a root-directory issue. Is your project in a git config or does it have a file that matches root directories from CONFIG.md? Did you try the debugging steps in the readme?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/neovim/nvim-lspconfig/issues/893#issuecomment-841466784, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AE6VULK5BEIXHEMODYXMSSLTNV5JRANCNFSM44UILHZQ.

Just to be clear, should your config not simply say:
local nvim_lsp = require'lspconfig'
nvim_lsp["pyls"].setup{}

ie. "pylsp" -> "pylsp".
Seems to me like it might just be a simple typo?
At least I don't see pylsp as a possible option in CONFIG.md

Sorry for the delay, here is the log
[ START ] 2021-05-11T00:32:15-0400 ] LSP logging initiated
[ START ] 2021-05-11T00:33:06-0400 ] LSP logging initiated
[ ERROR ] 2021-05-11T00:33:30-0400 ] /usr/local/share/nvim/runtime/lua/vim/lsp/rpc.lua:457 ]	"rpc"	"pylsp"	"stderr"	"2021-05-11 00:33:30,889 UTC - WARNING - pylsp.config.config - Failed to load pylsp entry point 'autopep8': No module named 'pycodestyle'\n"
[ ERROR ] 2021-05-11T00:33:30-0400 ] /usr/local/share/nvim/runtime/lua/vim/lsp/rpc.lua:457 ]	"rpc"	"pylsp"	"stderr"	"2021-05-11 00:33:30,894 UTC - WARNING - pylsp.config.config - Failed to load pylsp entry point 'mccabe': No module named 'mccabe'\n"
[ ERROR ] 2021-05-11T00:33:30-0400 ] /usr/local/share/nvim/runtime/lua/vim/lsp/rpc.lua:457 ]	"rpc"	"pylsp"	"stderr"	"2021-05-11 00:33:30,895 UTC - WARNING - pylsp.config.config - Failed to load pylsp entry point 'pycodestyle': No module named 'pycodestyle'\n"
[ ERROR ] 2021-05-11T00:33:30-0400 ] /usr/local/share/nvim/runtime/lua/vim/lsp/rpc.lua:457 ]	"rpc"	"pylsp"	"stderr"	"2021-05-11 00:33:30,895 UTC - WARNING - pylsp.config.config - Failed to load pylsp entry point 'pydocstyle': No module named 'pydocstyle'\n"
[ ERROR ] 2021-05-11T00:33:30-0400 ] /usr/local/share/nvim/runtime/lua/vim/lsp/rpc.lua:457 ]	"rpc"	"pylsp"	"stderr"	"2021-05-11 00:33:30,896 UTC - WARNING - pylsp.config.config - Failed to load pylsp entry point 'pyflakes': No module named 'pyflakes'\n"
[ ERROR ] 2021-05-11T00:33:30-0400 ] /usr/local/share/nvim/runtime/lua/vim/lsp/rpc.lua:457 ]	"rpc"	"pylsp"	"stderr"	"2021-05-11 00:33:30,896 UTC - WARNING - pylsp.config.config - Failed to load pylsp entry point 'pylint': No module named 'pylint'\n"
[ ERROR ] 2021-05-11T00:33:30-0400 ] /usr/local/share/nvim/runtime/lua/vim/lsp/rpc.lua:457 ]	"rpc"	"pylsp"	"stderr"	"2021-05-11 00:33:30,897 UTC - WARNING - pylsp.config.config - Failed to load pylsp entry point 'rope_completion': No module named 'rope'\n"
[ ERROR ] 2021-05-11T00:33:30-0400 ] /usr/local/share/nvim/runtime/lua/vim/lsp/rpc.lua:457 ]	"rpc"	"pylsp"	"stderr"	"2021-05-11 00:33:30,898 UTC - WARNING - pylsp.config.config - Failed to load pylsp entry point 'rope_rename': No module named 'rope'\n"
[ ERROR ] 2021-05-11T00:33:30-0400 ] /usr/local/share/nvim/runtime/lua/vim/lsp/rpc.lua:457 ]	"rpc"	"pylsp"	"stderr"	"2021-05-11 00:33:30,898 UTC - WARNING - pylsp.config.config - Failed to load pylsp entry point 'yapf': No module named 'yapf'\n"
[ START ] 2021-05-11T00:36:22-0400 ] LSP logging initiated
[ START ] 2021-05-11T00:38:30-0400 ] LSP logging initiated

```'
So obviously I'm missing stuff, but I used to use pyls fine and it just worked. New machine, new issues I guess. I don't recall having to install any other modules.

I verified everything is working as expected.

mkdir test && cd test && git init
touch main.py
conda create -n repro
conda activate repro
`pip install 'python-lsp-server[all]' (You need [all] for completion and error checking)
nvim main.py
Enjoy errors and completion

If you have additional problems, please ask on the discourse.

This issue has been mentioned on Neovim Discourse. There might be relevant details there:
https://neovim.discourse.group/t/i-have-been-unable-to-setup-neovim-lsp-for-python-nothing-seems-to-work/1141/1

So I figured it out. I had a python.vim file in my ftplugin directory. I tried removing each line but it seems like the presence of any command in that file will cause the lsp servers not to load on attach. Is this meant to happen? Removing the file or removing all the contents of the file causes the lsp servers to attach on open.


IPL, SPL, boot0, mask ROM...

explain that vendors/projects have different wording and start counting from different perspective in terms of program loaders / boot loaders

done in ff0428b2eb2f68bebd0b9738f49fe934e37744ad


Exported middleware causing error in latest version

Issue
Setup:

Deno Version: <1.9.0>
v8 Version: <9.1.269.5>
Typescript Version: <4.2.2>
Opine Version: <1.3.2>

Details
log.ts
export const logErrors = (error: any, req: Request, res: Response, next: NextFunction) => {
  console.error(error.stack);
  next(error);
};

app.ts
import { logErrors } from './log.ts';
...
app.use(logError);

error: TS2612 [ERROR]: Property 'resolve' will overwrite the base property in 'Deferred'. If this is intentional, add an initializer. Otherwise, add a 'declare' modifier or remove the redundant declaration.
public readonly resolve!: () => void;
~~~~~~~
at https://deno.land/x/evt@v1.9.12/tools/Deferred.ts:57:21
kindly help me out of it

Hi @rssaranece94 👋
I'm struggling to reproduce 😢  - I have created an app.ts
import { opine } from<EMAIL_ADDRESS>import { logErrors } from "./log.ts";

const app = opine();

app.use(() => {
  throw new Error("test-error");
});

app.use(logErrors);

app.listen(3000);

and a log.ts
import type { Request, Response, NextFunction } from<EMAIL_ADDRESS>
export const logErrors = (
  error: any,
  req: Request,
  res: Response,
  next: NextFunction,
) => {
  console.error(error.stack);
  next(error);
};

And can see the server is working as expected.
It is curious that you are getting an error thrown from the evt library as that was removed as a dependency from Opine in version 1.2.0 ( see https://github.com/asos-craigmorten/opine/blob/main/.github/CHANGELOG.md#120---06-03-2021 ). Please check that you are using the latest version of Opine 1.3.2 in all of your code, and if that isn't resolving the issue, perhaps try using the --reload flag when you next run the app to refetch dependencies in case something has happened to corrupt your Deno cache.
If none of this fixes the issue, please could you provide a fuller example of the exact code that is not working for you, or share a gist / repo etc? 😄

Thanks for the help. Sorry my bad I missed old version import in sub files. Updated those works well.

Ah, glad you were able to sort it 🎉 nice one 😄


proto: use full package path in go_option

The official docs for the go_option option specify
that the full go package import path should be used.
This, together with the paths=source_relative parameter, results
in the same output paths.

/assign @mikehelmick

/ok-to-test

/lgtm
/approve

@gurayAlsac @squee1945 as FYI. Shouldn't impact anything.


Random Probability Maker files added

Description 📜
This app takes user's daily tasks as input and randomly selects 1 task to do for the day
Fixes #1635
#1676

Type of change 📝
Please delete options that are not relevant. In order to tick the check box just put x inside them for example [x] like this

[x] Add a new project

Domain of Contribution 📊
Please delete options that are not relevant. In order to tick the check box just put x inside them for example [x] like this

[x] Project using HTML, CSS & JS

Checklist ✅
Please delete options that are not relevant. In order to tick the check box just put x inside them for example [x] like this

[x] I have performed a self-review of my own code or work.
[x] I have commented on my code, particularly in hard-to-understand areas.
[x] My changes generate no new warnings.
[x] I'm GSSoC'22 contributor

Screenshots / Gif (Recommended) 📸
https://user-images.githubusercontent.com/91609324/168741711-482c4b44-a79e-4604-86ea-07df9228e151.mp4

@Mrjoy832  bro i have closed my previous pr as i have uploaded wrong files in that
I have also mentioned the issue and closed pr's number also ...please do check :)

Why these are changed?

 Why these are changed?

Bro fix these


add: content for challenge - match ending string patterns

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 🎉


infrastructure services containers are not starting correctly on CEntOS hosts/VM

Rancher Version:
Components version:

Rancher v1.2.2
Cattle v0.174.13
User Interface v1.2.40
Rancher Compose v0.12.1

Docker Version:
Docker Version: 1.12.5
OS and where are the hosts located? (cloud, bare metal, etc):
two Vagrant VMs, CentOS Linux release 7.3.1611 (Core). all behind a proxy
Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB)
one node as rancher server with internal DB , another one as rancher agent
Environment Type: (Cattle/Kubernetes/Swarm/Mesos)
Cattle
Steps to Reproduce:

install and run one Rancher server
Add one node

Results:
ipsec, healthcheck and scheduler containers keeps restarting.
And I can't run any user stack I've tryied.
Expected:
All Stacks to be up and running, mainly infrastructure stack.
LOG: service healthcheck in healthcheck and LOG: service scheduler in scheduler:
ERROR 	service.activate.wait.exception 	Expected state running but got error: Timeout getting IP address
ERROR 	service.activate.exception 	Expected state running but got error: Timeout getting IP address

command I've used to start rancher server
docker run \
            -e NO_PROXY='localhost,.hacklab,<IP_ADDRESS>,<IP_ADDRESS>,102,<IP_ADDRESS>/24' \
            -e no_proxy='localhost,.hacklab,<IP_ADDRESS>,<IP_ADDRESS>,102,<IP_ADDRESS>/24' \
            -e HTTPS_PROXY=http://<IP_ADDRESS>:5865/ \
            -e HTTP_PROXY=http://<IP_ADDRESS>:5865/ \
            -d --restart=unless-stopped -p 80:8080 rancher/server

command I've used to start rancher node
docker run \
            -e NO_PROXY='localhost,.hacklab,<IP_ADDRESS>,<IP_ADDRESS>,102,<IP_ADDRESS>/24' \
            -e no_proxy='localhost,.hacklab,<IP_ADDRESS>,<IP_ADDRESS>,102,<IP_ADDRESS>/24' \
            -e HTTPS_PROXY=http://<IP_ADDRESS>:5865/ \
            -e HTTP_PROXY=http://<IP_ADDRESS>:5865/ \
            -d --privileged \
            -v /var/run/docker.sock:/var/run/docker.sock \
            -v /var/lib/rancher:/var/lib/rancher rancher/agent:v1.1.2 \
                 http://<IP_ADDRESS>/v1/scripts/881572FBF8F0B054CFC8:1482948000000:gFZuQwrdBnfLf0byJKEEM0QXdo

Could you paste some rancher/server and rancher/agent logging?

closing due to inactivity.


Error in cachekey when using GroupBy

Hi there!
When I use FromCache using follow EF instructions, Z throws a error.
Using

Z.EntityFramework.Plus.EFCore: 1.4.29
Microsoft.EntityFrameworkCore.SqlServer: 1.1.0

My code
var tags = ...; // Dynamic tags

query
    .Where(q => q.Active.Equals(1))
    .GroupBy(q => q.FederalState)
    .Select(q => q.Key)
    .Select(q => new FederalStateDto
    {
        Name = q
    })
    .CachedData(tags)
    .ToList();

Exception
System.ArgumentOutOfRangeException was unhandled by user code
  HResult=-2146233086
  Message=Index was out of range. Must be non-negative and less than the size of the collection.
Parameter name: index
  ParamName=index
  Source=System.Private.CoreLib
  StackTrace:
       at System.ThrowHelper.ThrowArgumentOutOfRange_IndexException()
       at Z.EntityFramework.Plus.InternalExtensions.CreateCommand(IQueryable source, RelationalQueryContext& queryContext)
       at Z.EntityFramework.Plus.QueryCacheManager.GetCacheKey(IQueryable query, String[] tags)
       at Z.EntityFramework.Plus.QueryCacheExtensions.FromCache[T](IQueryable`1 query, MemoryCacheEntryOptions options, String[] tags)

Hello @diegobrum ,
Thank you for reporting this issue.
I will check it when I will be back.
What exactly is doing the method CachedData?
Best Regards,
Jonathan

Sorry @JonathanMagnan
Where you see CachedData, read FromCache... I copy/past wrong code :/

Hello @diegobrum ,
The issue happens because the query is not longer really a query.
When you do
.Select(q => new FederalStateDto
    {
        Name = q
    })

We try to retrieve the SQL Command generated to create the cache key, and we cannot find the query anymore.
By example, the cache will probably work if you move it here:
query
    .Where(q => q.Active.Equals(1))
    .GroupBy(q => q.FederalState)
    .FromCache(tags)
    .Select(q => q.Key)
    .Select(q => new FederalStateDto
    {
        Name = q
    })
    .ToList();

There are some other options that could work which doesn't require our library to generate the SQL Command for the cache key entry:
QueryCacheManager.UseFirstTagAsCacheKey = true;
QueryCacheManager.UseTagsAsCacheKey = true;

Let me know if you can use one of the solution above to solve your issue.
Best Regards,
Jonathan

Hi @JonathanMagnan
I used first option and works well. Thanks :)
But, when you say "We try to retrieve the SQL Command generated to create the cache key, and we cannot find the query anymore", I don't know, I feel is incorrect, because I do this in a lot of parts of code (like bellow) and works well. I think the GroupBy clause is the bad guy in this story.
var tags = ... // dynamic tags

this._dbContext
    .City
    .Where(q => q.FederalState.Equals(federalState))
    .OrderBy(q => q.Name)
    .Select(q => q.Adapt<CityDto>())
    .FromCache(tags)
    .ToList();


After October 19 release copy and paste of grid results now include a tab after every record

Azure Data Studio Version:

Steps to Reproduce:

Run any query.
Click on record in grid.
Copy record.
Paste record anywhere else (Notepad, Azure Data Studio query editor, etc.)

You will see a tab after the record.

This should be fixed with #7516 so I'm going to close this - let us know if you still encounter this issue after the next stable version is released!


LightGBM Rank Model predict score is negative

When I do a rank task, I use the lightgbm ranker model, the loss function uses lambdarank, the data labels are 0 and 1, representing the correlation, when I do the prediction, the return result is negative, I don't quite understand why the score is negative. When we do binary classification, the score returned by the model can be understood as a probability, does anyone know what that means? thank you.

Hi @Man1978-scd, thanks for using LightGBM. Which API are you using with LightGBM, and how is the prediction done? I think in your case LightGBM is providing the raw prediction scores.
Also. not that the goal of lambdarank (or the underlying algorithm lambdamart) is not to predict the relevance scores of the documents in a query. Instead, the goal is to provide a ranking order of the documents as consistent as the ordering according to their relevance scores.
So, though the predicted scores are not direct regression to the original relevance scores, but their values do indicate the relevance. A higher score indicates a higher relevance of a document.
Thus, negative values are allowed, since the lambdarank cares only about the ranking, not the actual score value.

You may refer to this document for a more clear understanding of the LambdaMart algorithm.
https://www.microsoft.com/en-us/research/wp-content/uploads/2016/02/MSR-TR-2010-82.pdf


Setup the Math magician React app

Setup the app using create-react-app

Use the Gitflow workflow
Made the Development branch the reference branch
Add linter files and configuration
App runs in the browser without error

Microverse Readme Template added


Log commit hashes when creating commits

This is applicable to git gud commit and calls to create_tree(). This should be done inside the  create_tree and inside the handle_commit method.
This is useful because it gives the user information about the events that are happening to create the current tree.

@bthayer2365 how should the messages look like. Is it possible to get an example for each of git gud commit and create_tree()
For example this is what I have for git gud commit

I am yet to figure out the create_tree()

What you have there is good, thanks for working on this.
I pasted below what it normally looks like when committing.
[master f7778e8] Added 3
 1 file changed, 0 insertions(+), 0 deletions(-)
 create mode 100644 3

I think we can cut down the hash to 8 characters like git does normally and maybe change the message you have to be prettier.
So, something like
...
Simulating: git commit -m "3"
New Commit: d77413e8

Also, I took a look at your code, and unfortunately, for comparability reasons, we can't use f-strings 😢 but oh I do love them so much! f-strings were introduced in python 3.6, and some people only have 3.5 or lower, so to make it easier for everyone to install, we need to avoid using f-strings in favor of 'string'.format()


Not compatible with greiman/SdFat

Either implement a wrapper in the Logger library which would require the Logger to include SdFat, which is impractical. Maybe create a fork of SdFat and implement there similar to https://github.com/greiman/SdFat/issues/471.
Then create a PR...

Fixed in #8


Important URLs in documentation are not valid!!

Reading instructions from:
https://github.com/coreos/updateservicectl/blob/master/Documentation/example-container-update.md
I noticed that some of the links that are essential to get updateservicectl to work properly refer to the main document page and not the correct ones.
Examples:
example update repository
Zero Downtime Frontend Deploys with Vulcand
Thanks

example-container-update.md would still be a very useful page, also there is still a link to it in the main README file.

@stilliard Thanks for the feedback. The docs were incomplete & incorrect so they were probably more misleading than helpful. That's why we decided to remove them for now until we have time to revise them or come up with a more generalized approach.
Custom update solutions will require custom code to speak the Omaha Protocol. So I'd refer you to the protocol docs here: https://github.com/coreos/updateservicectl/blob/master/Documentation/protocol.md

Thanks for the reply @sym3tri, I'll look into the Omaha protocol a bit more.
But in my case, all my apps are up on docker hub so I guess the simplest method is to have a timer that runs daily, pulls down the latest versions of the images and restarts the services.
I was just looking for that document in case there were any suggestions around this method, or any reasons it wouldn't work well.


Skip np.float128 assertion in test_convert if type doesn't exist

It isn't available on all systems (e.g., 32-bit x86 machines).

Thanks.


sql: layers hashes, and rewards

Motivation
part of https://github.com/spacemeshos/go-spacemesh/issues/2918
Changes

move layers hashes (blocks hare and aggregated hash) to layers table
reward table
sql function add_uint64 to add rewards in sqlite (sqlite doesn't support uint64 natively)

bors merge

flake in TestBuilder_RestartSmeshing

bors merge

bors merge


Animation on svg icons just like on Font Awesome icons

Sorry, It seems this feature will not be shipped in future version of Waves, unless there is high demand from people that want this implemented. But, thanks for your suggestion.


Forensic gloves for detective

About the PR

Added forensic gloves for the detective. They will facilitate his work, as they do not leave fibers and fingerprints. They appear in the detective's cabinet.
Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl:

add: Added forensic gloves for the detective. They do not leave fibers and fingerprints.

Why not give this attribute to nitrile gloves?

Why not give this attribute to nitrile gloves?

These gloves are used when more reliable protection from chemical compounds is required. Nitrile gloves are not unique and they are not too difficult to get. Only Detective has forensic gloves, and to get them, you need to break into his office or kill him - this is the price of gloves that leave no trace at all. I don't see a single reason to give this property to nitrile gloves.


[catch2] Build error

Copy issue body from /home/nio/workspace/tech_test/vcpkg/installed/vcpkg/issue_body.md

network download error


Revert Makefile and update README

Following after this comment: https://github.com/bitjourney/docker-elasticsearch-ci/pull/4/files#r116897483

Revert Makefile to manually publish to Docker Hub
Update README

LGTM


Update openssl-src

OpenSSL has been bumped to version 1.1.1.t, if compiling with vendored OpenSSL to address RUSTSEC-2023-0006, RUSTSEC-2023-0007, RUSTSEC-2023-0009 and RUSTSEC-2023-0010.

bors merge


Fix table of contents: undetected adblock

In a previous PR, an user deleted "#how-to-use-the-undetected-adblock" header of the README.md file; screwing up the table of contents.
This new PR fixes this problem by removing the entire portion of undetected adblock as normal user does not need to know about it.

As  stated by the BlueWorld1, I will close this PR!


Polyline is drawn straight instead of proper driving directions route path from point A to B

I have 2 points A & B.
I am simply trying to draw a polyline, but it draw a straight line instead of proper driving directions route.
Here is my plunker:
http://plnkr.co/edit/Uz8Dz0zB4Ea8CJTe6wYM?p=preview
Can anyone identify the issue if any? because I am following the docs

@MartiniHenry1988 there is no support right now for the directions service. So I close this because we have already an Issue for that.

@SebastianM can we expect the directions support in near future ?


add strict string values for voucherType and voucherStatus

Thanks very much for the effort, but we already decided not to type in literals as it would force the user to either give the whole for example invoice object the related type or to add "as const" to every literally typed property. We decided it would be easier to use this way. Thank you!


Clicking upgrade notification closes Kap without saving

Platform: macOS 10.13.4
Kap Version: 2.1.2
Steps to reproduce

Start and finish a recording, keeping the window open and don't save yet
Click the notification to get the latest version of Kap
Watch your work disappear 😀

Current behaviour

Simply quits without saving

Expected behaviour

Maybe prompt me to save before closing

Workaround

Think before pressing "Upgrade" 😂

Howdy there! I don't mind taking a stab at this one!

We're not gonna have too many (or any? @skllcrn ) updates to 2.0, so I think this should be fixed in the upcoming 3.0 branch, thoughts @karaggeorge?

Would love your help on this @maxcell! Thinking it might be easier to do once #450 is merged to master and we have a Beta build? Related issue: #481.

@skllcrn That sounds good to me! I don't mind waiting especially if we are changing some big behavior to how the app works. In the meantime, I can get familiar with what we are doing in #450

@skllcrn It seems like the workflow might be slightly different now so it isn't possible to reproduce the error in the latest version. When going and performing a recording it seems as that the new window cropper cannot be reopened while recording (if I'm wrong, please do inform me how to get the bar back). Due to that, one has to stop the recording and when you do and try to get to preferences, it seems like the problem isn't there. However I think #481 could be good to have since there is no real feedback at the moment beyond just being disabled. I do think that the logic with having a notification appear is still valuable, but maybe the test could be changed to say "Latest version" or some other text.
@LinusU If you could check out the latest update and see if the problem still is occurring, please let me know!

I think I understand what you're getting at @maxcell. In the current version on master you can still manually check for updates using the 3-dot more menu in the toolbar of the main app window, and it still doesn't have any feedback which is why #481 is open.
The question @sindresorhus raised is if we want to be able to manually check for updates at all, which can be addressed in #481. However, wether the update in manual or automatic, we still need to make sure that no work is lost when clicking the update notification.
With the current master version you can have multiple files open at once, which means there can be one or more recordings that need to be exported to avoid losing any work when updating. So if you dismiss an update you need to be able to trigger it again once you've exported the work.

@skllcrn Apologies for asking so many more questions but just want to be sure I fully follow! 😄
I love what you are saying here, we should make sure that users can 1) once again try to check for updates after it has already been done but also 2) making sure that it doesn't interfere with things currently exporting. That context helps a lot more to understand how to approach solving this. Thank you so much 😄


Remove test boilerplate files

Remove boilerplate test files which would cause go test ./... to fail.
/assign @jakexks

Makes sense, I see that we have removed the hack/boilerplate/test in cert-manager too.
/lgtm


change: provision tls for all appinstance endpoints, including custom…

… cluster-domains
Signed-off-by: Thorsten Klein<EMAIL_ADDRESS>Ref #206

@iwilltry42  Might be unrelated but was testing out this PR with this Acornfile and the endpoint state never seems to leave the pending state

This is on a Digital Ocean Cluster Kubernetes 1.25.4-do.0 w/ NGINX Ingress Controller v1.2.1
jsilverio@joshuas-mbp acorn % ./bin/acorn app
NAME          IMAGE          HEALTHY   UP-TO-DATE   CREATED   ENDPOINTS                                   MESSAGE
proud-night   03d8411b2c53   6         6            37m ago   https:// => default:8080   OK

@jsilverio22 thanks for the insights!
Given your debug information it looks like the ingress object and the TLS secret have been created already.
The endpoint is pending probably because the ingress didn't get a load balancer IP yet.
Can you confirm this?
If that's the case, do you have a way to check the logs of the ingress controller to see if there's anything wrong?

Seems like this is the error being thrown
I1215 00:49:02.221646      10 store.go:425] "Ignoring ingress because of error while validating ingress class" ingress="proud-night-46cb4936-2cf/default" error="ingress does not contain a valid IngressClass"


[TD-12657](tools): taosdump fetch lengths

Codecov Report

Merging #83 (ead5a76) into develop (dd78bff) will increase coverage by 0.05%.
The diff coverage is n/a.

:exclamation: Current head ead5a76 differs from pull request most recent head 2977c24. Consider uploading reports for the commit 2977c24 to get more accurate results

@@             Coverage Diff             @@
##           develop      #83      +/-   ##
===========================================
+ Coverage    51.59%   51.64%   +0.05%
===========================================
  Files           10       10
  Lines         8170     8194      +24
  Branches       388      388
===========================================
+ Hits          4215     4232      +17
- Misses        3758     3764       +6
- Partials       197      198       +1

Impacted Files
Coverage Δ

TDengine/src/kit/taos-tools/src/benchQuery.c
65.87% <0.00%> (-0.17%)
:arrow_down:

TDengine/src/kit/taos-tools/src/benchUtil.c
59.28% <0.00%> (+0.11%)
:arrow_up:

TDengine/src/kit/taos-tools/src/taosdump.c
55.99% <0.00%> (+0.13%)
:arrow_up:

Continue to review full report at Codecov.

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


Only check imageUid for services of type service

@jgreat @EvanSnapp - great find! Let's squash this PR into a single commit. Here's a pretty clear/short write-up on how. I'll add notes to any lines that need changes before merge. Thanks!

@EvanSnapp two minor things: do a find replace to correct the misspelling "Enviorment". Style-wise, we use upper-case for the top level describe calls (which you've done) but lower case for everything after that including the should descriptions. The output ends up looking something like:
Module Under Test
  when things are good
    should be really good
  when things are not
    should catch fire, fall over and sink into the swamp

We'll rework the logging after this commit is merged.


No option to add Terms/Privacy links

As per googles developer policies I am trying to configure the plugin to display TERMS and Conditions/Privacy Policy links.
The plugin does not give any customization for such thing.
Possible for such feature to be implemented in next update?

The Google Trust & Safety team mentioned that I must provide the following:
Google Sign-in
•	Make sure the Google sign-in button on your project’s homepage  https://forum.hgnetwork.net/index.php follows the Google sign-in branding guidelines. https://developers.google.com/identity/branding-guidelines#top_of_page

@effone - maybe we should add this, because not only Google requires this :(
Any basic common accept terms/privacy page after registration for the first time would be enough?

Being a multi-login system it is not possible to add every Service's TOS below the plugin.
This plugin provides basic integration. Any user can modify the button using any of his own choice / supplied by the provider and the plugin will work fine as long as it provides the right gateway in url.
For example, this can be added anywhere in the template and will work for Google seamless:
Login with Google


Reflection Baking fails on nested class inside a DLL

I have a DLL with some basic models that uses the Factory and MemoryPool from Zenject.
    public class Player : IPlayer     {         public int Id { get; set; }         public class Pool : MemoryPool<Player> { }     }
When I run the reflection baking on the DLL, I get the following error:
Could not find actual type for type 'Player/Pool', skipping
If I declared the pool as a non-nested class, there is no error message.
I am not sure if I am doing something incorrectly or maybe it is simply not possible to make nested class in the DLL for reflection baking.

Sounds like this is a bug in reflection baking


kitex内存占用高

Describe the bug
A clear and concise description of what the bug is.
相比于其他rpc，kitex内存占用高（Intel(R) Xeon(R) Platinum 8255C CPU @ 2.50GHz，2g内存，CentOS Linux release 7.6.1810 (Core)）
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Kitex version:
Please provide the version of Kitex you are using.
Environment:
The output of go env.
Additional context
Add any other context about the problem here.

kitex 有内存池，内存占用会高一些，但不会高很多


Support custom page sizes specified in points in PPD files

I have a printer with some odd page sizes specified in points rather than inches or mm.
*OpenUI *PageSize/Media Size: PickOne
*OrderDependency: 10 AnySetup *PageSize
*DefaultPageSize: w81h252
*PageSize w81h252/Address - 1 1/8 x 3 1/2":			"<</PageSize[81 252]/ImagingBBox null>>setpagedevice"
*PageSize w101h252/Large Address - 1 4/10 x 3 1/2":		"<</PageSize[101 252]/ImagingBBox null>>setpagedevice"
*PageSize w54h144/Return Address - 3/4 x 2":			"<</PageSize[54 144]/ImagingBBox null>>setpagedevice"
*PageSize w167h288/Shipping Address - 2 5/16 x 4":		"<</PageSize[167 288]/ImagingBBox null>>setpagedevice"
*PageSize w162h540/Internet Postage 2-Part - 2 1/4 x 7 1/2":	"<</PageSize[162 540]/ImagingBBox null>>setpagedevice"
*PageSize w162h504/Internet Postage 3-Part - 2 1/4 x 7":	"<</PageSize[162 504]/ImagingBBox null>>setpagedevice"
*PageSize w41h248/File Folder - 9/16 x 3 7/16":			"<</PageSize[41 248]/ImagingBBox null>>setpagedevice"
*PageSize w41h144/Hanging Folder - 9/16 x 2":			"<</PageSize[41 144]/ImagingBBox null>>setpagedevice"
*PageSize w153h198/3.5" Disk - 2 1/8 x 2 3/4":			"<</PageSize[153 198]/ImagingBBox null>>setpagedevice"
*CloseUI: *PageSize

(full file is http://www.opensource.apple.com/source/cups/cups-26/ppd/dymo.ppd)
The custom page size regex in cups-connector is currently ([\d.]+)(?:mm|in)?x([\d.]+)(mm|in)?, which doesn't match strings like "w81h252".
https://github.com/google/cups-connector/blob/3d6844ab925dad5acaa6972419f11cbc4608a9ee/cups/translate-ppd.go#L716
Are you willing to add support for this and/or accept a pull request adding support?

I'm always willing to review and accept pull requests.