Perf: Improve perf of SqlDateTimeToDateTime

While reviewing https://github.com/dotnet/SqlClient/pull/853 I did some digging into the related code and profiling. The function used to convert sql date times to .net datetimes came up as hot so I tinkered with it a bit. The most important part is to avoid creating a datetime just to get a constant value from it, hardcode it instead.
Gist of the benchmark code is here: https://gist.github.com/Wraith2/d0d220f07e640c17daeb9db2d446aaa4
results:
BenchmarkDotNet=v0.12.1, OS=Windows 10.0.19042
Intel Core i9-9900K CPU 3.60GHz (Coffee Lake), 1 CPU, 16 logical and 8 physical cores
.NET Core SDK=5.0.200-preview.21079.7
  [Host]     : .NET Core 3.1.12 (CoreCLR 4.700.21.6504, CoreFX 4.700.21.6905), X64 RyuJIT
  DefaultJob : .NET Core 3.1.12 (CoreCLR 4.700.21.6504, CoreFX 4.700.21.6905), X64 RyuJIT

Method
Mean
Error
StdDev
Ratio

Original
271.50 ms
4.619 ms
4.321 ms
1.00

New
82.45 ms
0.232 ms
0.206 ms
0.30

/cc @bjornbouetsmith who this may interest

Seems thr's a test failure you may want to take a look:
System.OverflowException : SqlDateTime overflow. Must be between 1/1/1753 12:00:00 AM and 12/31/9999 11:59:59 PM.
...
   at Microsoft.Data.SqlClient.ManualTesting.Tests.DateTimeTest.ReaderParameterTest()
... ManualTests\SQL\DateTimeTest\DateTimeTest.cs:line 188

Nice catch. Didn't see it among the other CI failures.
Reworked slightly to deal with the bounds and I've added a uint range check trick that wasn't worth using before (regressed it by 4%) but now does seem worthwhile. The range checks work by knowing that negative integers cast to extremely large positive uints so if we do zero based checks we only have to check 0 to max and not a min check.

Method
Mean
Error
StdDev
Ratio

Original
258.05 ms
2.267 ms
2.010 ms
1.00

New
85.68 ms
0.589 ms
0.551 ms
0.33


Inserting json

Hi!
Is there an api call for inserting a json to another json? For example I have a json for header, I want to put that into a json for body. Another scenario is, there is a json that can be inserted even at the middle part of the body. Is that even possible? Thank you!

Hi, is this still an issue for you? What type of JSON are you looking on inserting?

Hi! yes, the JSON that I will insert is also an editable.

You can update the editor state with editor.trigger.editable.add(yourEditableObject), but you can't merge two editables at the moment in a native way. One possibility to merge them, would be to combine the cells properties of both, so something like:
const editable1 = { /* ... */ }
const editable2 = { /* ... */ }

const mergedEditable = {
  ...editable1,
  cells: [ ...editable1.cells, ...editable2.cells ]
}

Thanks @arekkas!

Hi @arekkas, one more thing, is there a way to insert or push an object in the children of a layout plugin? Maybe calling it inside that layout component? Thanks!

Hm, it is of course supported within the editor itself, but not really exposed via an API. One possibility would be to dispatch one of the CELL_INSERT_{ABOVE|BELOW|LEFT_OF|...) actions using editor.store.dispatch(), but you would have to have some knowledge of the internals such as the cell id.


Computed fields can't be added to nested objects

This would enable a computed to belong to the same namespace(i.e object) of a reactive property, for better organization
https://codesandbox.io/s/valtio-simple-counter-forked-z9mzx?file=/src/App.js

Interesting. That might be a nice feature addition (and this would solve "unsupported externals" issue). I'm afraid the code becomes 2-3 times bigger and complicated, though.
One small issue I noticed is, if we change doubled to get, it behaves totally wrongly.
const state = proxyWithComputed(
  { math: { number: 1 } },
  {
    math: {
      get(snap) {
        return snap.math.number * 2;
      }
    }
  }
);

On second thought, merging two math objects seems extremely hard. externals might be easier...
Let's see if some new ideas would click me...

So, the question is how should we handle the get shorthand.

Any idea for this? especially to make it typescript friendly.

The example you added should error because it's trying to overwrite a reactive field, so it's just a matter of updating the docs with this caveat:
const state = proxyWithComputed(
  { math: { number: 1 } },
  {
    math: { // maybe verify this with eslint-plugin-valtio, cause it's trying to overwrite the reactive property math<Object>
      get(snap) {
        return snap.math.number * 2;
      }
    }
  }
);

I mean, in the context of computed fields, get and set are reserved

I mean, in the context of computed fields, get and set are reserved

If you are ok with it, then fine. That example is actually not an error, it's a valid get.
The remaining concern is typescript...

I thought I made it (incl. typescript) but it doesn't work...
https://github.com/pmndrs/valtio/tree/feat/nested-computed
Not sure why this happens... (undefined. the order issue again.)

BTW when a computed field has something wrong the state is still silently created with a subset of fields, it should throw an error

Added a issue to eslint-plugin-valtio to check on computed fields overwriting reactive fields, but that might change a little bit when we support the nested behavior https://github.com/pmndrs/eslint-plugin-valtio/issues/5

I didn't have time since the first trial. It would be really nice if that impl worked.
overwrite checking should be easy. typing would be hard. we could try it separately before supporting nesting.

Typing can be added later

Typing can be added later

Well, it's controversial. In my recent TS projects, I avoid designing APIs that are not type friendly.
That said, detecting overwrites is just nice to have, and I'd just give up if it's not possible.

Theoretically Typescript should never get in the way of a feature


Export Issue : Not All fields are populated in the MCAS Identity Security Posture

Hello Team,
We Sync multiple on-premises Active Directory Domain to Azure. When Navigating portal.cloudappsecurity.com --> Investigate --> Identity Security Posture and trying to export report e.g. Unsecure Kerberos delegation , Domain Column is not been exported.
I tried to look up several ways to export this MCAS API, Powershell but all in vain.
Can you please provide any guidance in how I can go and achieve this.
Kindly note that we need this data badly to improve our Organization's overall Security Posture.
Refer attached images (Exported report and MCAS UI)
Thank You

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

ID: 06ac1c39-fce7-aa71-b56b-385c2b72ca63
Version Independent ID: 6813852c-5cbc-9843-fd43-53972d120ae5
Content: Microsoft Defender for Identity's identity security posture assessments
Content Source: ATPDocs/isp-overview.md
Service: microsoft-defender-for-identity
GitHub Login: @shsagir
Microsoft Alias: shsagir

@arvinder-dhar We researched your issue and determined it's a bug. An internal ticket has been raised.
We're closing this as an open issue for now, but feel free to continue to comment here as necessary. We appreciate your contribution to our community.


Subdomains return incorrect URL (http vs https)

Example code:
tunnel = new localtunnel(this.port, {
  subdomain: 'mysubdomain'
}, function (err, sshtunnel) {
  console.log(sshtunnel.url)
})

I expected this to return https://mysubdomain.localtunnel.me, but it actually returns http://mysubdomain.localtunnel.me... notice it's missing the s in the protocol.

I am seeing this too.  Is there any explanation around this issue?

Why was this closed?


Sharepoint Search Query Tool Pagination of results

Sharepoint Search Query Tool v2.10.0
Results in the tool are limited to 500 regardless of row limit or rows per page entered. Pagination doesnt seen to work.
Is this a limitation of the REST query or is the tool not paginating the results correctly?
Case Scenario: I have almost 10000 sites to crawl. I'm trying to return a list of sites that an account has access to through the Everyone permission (instructions are on the Microsoft Support site here)
The query:
https://example.sharepoint.com/sites/Example_Site_SPO/_api/search/query?querytext='path%3Ahttps%3A%2F%2Fexample.sharepoint.com'&enableinterleaving=false&rowsperpage=500&rowlimit=500&clienttype='ContentSearchRegular'
My query returns
Elapsed Time (ms): 1355
Primary Query Results:
	Total Rows: 74631
	Total Rows Including Duplicates: 97912

But the Raw tab contains only an xml or json object with 500 entries and the Primary Results lists only 500
I can modify the Start position to skip the first 500 and collect the next 500, but for 10000 results this is not ideal.

See https://learn.microsoft.com/en-us/sharepoint/dev/general-development/pagination-for-large-result-sets on how to use search to page large result sets. Or see https://www.techmikael.com/2023/08/how-to-paginate-large-results-sets-for.html to do the same using Graph API.

Im aware of how to use startrow to page through large data sets, as I
stated in my last comment.  The issue i have is I need to manually edit the
startrow value in the Query Tool and re-run the query every time and for
thousands of sites this means manually changing the value dozens of times.
With Graph API I can script the use of the @odata.nextLink returned in the
results and loop the requests to collect them all, but this doesn't work in
this tool.
What would be nice would be a Next Page button in the tool to page through
the results automatically instead of needing to manually set the startrow
value every time i need the next page of results.
On Thu, 10 Aug 2023, 19:59 Mikael Svenson, @.***> wrote:

See
https://learn.microsoft.com/en-us/sharepoint/dev/general-development/pagination-for-large-result-sets
on how to use search to page large result sets. Or see
https://www.techmikael.com/2023/08/how-to-paginate-large-results-sets-for.html
to do the same using Graph API.
—
Reply to this email directly, view it on GitHub
https://github.com/pnp/PnP-Tools/issues/333#issuecomment-1673748861, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AENCHN5EKSL5DJYLYPG7TD3XUUVRZANCNFSM6AAAAAA3LYTSWY
.
You are receiving this because you authored the thread.Message ID:
@.***>

Happy to accept a PR for this and makes sense :)


Fixed #12561: providing links to the .cum* in .expanding

[x] closes #12561
[ ] tests added / passed
[ ] passes git diff upstream/master | flake8 --diff
[x] whatsnew entry

#12651
commit for the .cum* in .expanding

Should close #12651 (not #12561).
I think it is better to link each other via docstring as described in #12561, or use section links ( ..sealso:: ). It's unclear what the updated part tries to explain.

sectional links provided.

pls reopen if you like to update according to comments


Unable to get ollama serve working

I have installed Ollama and the Ollama python client on Ubuntu. I am unable to interact with it using the Ollama Python client.
$ ollama list
NAME            ID              SIZE    MODIFIED
llama2:70b      e7f6c06ffef4    38 GB   2 hours ago
llama2:latest   78e26419b446    3.8 GB  4 days ago

$ OLLAMA_HOST=<IP_ADDRESS>:7656 ollama serve

When I try to interact with ollama in Python, I get a ResponseError. Thank you in advance.
import ollama
response = ollama.chat(model='llama2', messages=[
  {
    'role': 'user',
    'content': 'Why is the sky blue?',
  },
])
print(response['message']['content'])

@harsham05 if you want to connect to ollama on a custom port, you'll have to create a custom client to do this (this is in the ollama-python readme ).
An example:
from ollama import Client
client = Client(host='http://<IP_ADDRESS>:7656')
response = client.chat(model='llama2', messages=[
  {
    'role': 'user',
    'content': 'Why is the sky blue?',
  },
])

I'm going to go ahead and close the issue, but feel free to keep commenting.


Provide full control over bokeh grid styles

Provides control over grid styling in the bokeh backend.

[x] Addresses https://github.com/ioam/holoviews/issues/1903 and should fix https://github.com/ioam/holoviews/issues/1967
[x] Adds unit tests
[x] Adds documentation entry

Ready for review.


fixing vm az check #235

Description
fixing VMs Availability Zone check
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: #235
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[x] Created/updated tests
[x] Unit tests passing

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

Project coverage is 46.79%. Comparing base (d82d313) to head (c6f14f9).
Report is 26 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #236      +/-   ##
==========================================
- Coverage   48.44%   46.79%   -1.66%
==========================================
  Files          72       78       +6
  Lines        3625     4020     +395
==========================================
+ Hits         1756     1881     +125
- Misses       1854     2123     +269
- Partials       15       16       +1

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


GPS feature

Hi,
Thanks for updating ypur plugin to consider the rotation of the phone. I have one more question, is it possible to take the photo with GPS coordinates embeded in the photo or is it something that had to be done after the photo is taken and embed the GPS location using EXIF?
Thanks

Hi,
I will take a look at this and see whether it's possible to integrate it directly to the capturing process or if you have to add it manually to the image. I'll keep you posted.

Hi,
Any update on the GPS feature. I have been trying on my side with no luck. Also wondering do you have something similar to this for Android Phones?
Thanks


Trying to remove Administrator role from console leads to session timeout

Describe the issue:
https://github.com/wso2/product-is/assets/35717390/5a1d677c-313b-4785-8268-f97aa6484aeb
Better if we can restrict the role delete if that role is not allowed to be deleted. "Administrator" role.
It looks when trying to delete the role, the token is revoked. It should be happen in normal cases but in this case, the revocation shouldn't happen as the role was not deleted. Looks there is an issue in the order of execution.

In a pack built on 02/02/2024 couldn't see the delete option:


Changing repository urls from HTTP to HTTPS for security reasons

This change was necessarily created because I am using the SuperVanish dependency for one of my resources and I am also using the latest version of Apache Maven (available as of 3.8.2) that already blocks HTTP repository urls. Some repository urls had to be changed as the author probably switched to another platform. This HTTP urls block multiple resources, mostly my own, as I do not use the mirror option to prevent HTTP repositories from the SuperVanish dependency.
The Vault repository had to be changed from the old to the Jitpack as it is no longer available (at least for me).
I also changed the version for the affected dependencies to use the latest one. There will most likely be no broken compatibility between and when using resources.
Closes #60
Edit: also fixes https://github.com/LeonMangler/SuperVanish/pull/60#issuecomment-898624452

Thank you, this finally works!


[Bicep] - Support using an existing ANF

Closing as this is already possible thru the mount options in the configuration file


can you release a version?

Hello~ community maintainer:
the community has not released a version for a long time, can you release a version?

Hey, https://github.com/unknwon/goconfig/releases/tag/v1.0.0 has been tagged!


🐞 TFRobot [Bug]: failed to run deployments if couldn't find enough zos4 nodes

What happened?
trying to run deployment on dev net with 5 nodes requested, the deployer didn't filter for zos3 nodes if no zos4 nodes were found and if fails immediately
I think the deployer should check for zos3 nodes if couldn't find enough zos4 nodes
which network/s did you face the problem on?
Dev
Relevant log output
10:14AM INF done validating configuration file
10:14AM INF Running deployment Node group=group_a
10:14AM INF Filter nodes Node group=group_a
10:14AM DBG error="could not find enough nodes with specified options" opts="free_mru:<PHONE_NUMBER>0, free_sru:<PHONE_NUMBER>0, total_cru: 2, region: europe, available_for: 4653, healthy: true, features: network-light, status: up"
10:14AM DBG failed to deploy error="could not find enough nodes with specified options" Node group=group_a
10:14AM INF Retrying to deploy Deployment trial=2 Node group=group_a
10:14AM INF Filter nodes Node group=group_a
10:14AM DBG error="could not find enough nodes with specified options" opts="status: up, free_mru:<PHONE_NUMBER>0, free_sru:<PHONE_NUMBER>0, total_cru: 2, region: europe, available_for: 4653, healthy: true, features: network-light"
10:14AM DBG failed to deploy error="could not find enough nodes with specified options" Node group=group_a
10:14AM INF Retrying to deploy Deployment trial=3 Node group=group_a
10:14AM INF Filter nodes Node group=group_a
10:14AM DBG error="could not find enough nodes with specified options" opts="region: europe, available_for: 4653, healthy: true, features: network-light, status: up, free_mru:<PHONE_NUMBER>0, free_sru:<PHONE_NUMBER>0, total_cru: 2"
10:14AM DBG failed to deploy error="could not find enough nodes with specified options" Node group=group_a
10:14AM INF Retrying to deploy Deployment trial=4 Node group=group_a
10:14AM INF Filter nodes Node group=group_a
10:14AM DBG error="could not find enough nodes with specified options" opts="region: europe, available_for: 4653, healthy: true, features: network-light, status: up, free_mru:<PHONE_NUMBER>0, free_sru:<PHONE_NUMBER>0, total_cru: 2"
10:14AM DBG failed to deploy error="could not find enough nodes with specified options" Node group=group_a
10:14AM INF Retrying to deploy Deployment trial=5 Node group=group_a
10:14AM INF Filter nodes Node group=group_a
10:14AM DBG error="could not find enough nodes with specified options" opts="available_for: 4653, healthy: true, features: network-light, status: up, free_mru:<PHONE_NUMBER>0, free_sru:<PHONE_NUMBER>0, total_cru: 2, region: europe"
10:14AM DBG failed to deploy error="could not find enough nodes with specified options" Node group=group_a
10:14AM INF Retrying to deploy Deployment trial=6 Node group=group_a
10:14AM INF Filter nodes Node group=group_a
10:14AM DBG error="could not find enough nodes with specified options" opts="free_sru:<PHONE_NUMBER>0, total_cru: 2, region: europe, available_for: 4653, healthy: true, features: network-light, status: up, free_mru:<PHONE_NUMBER>0"
10:14AM DBG failed to deploy error="could not find enough nodes with specified options" Node group=group_a
10:14AM INF canceling contracts project name=vm/group_a
10:14AM DBG checking url=https://graphql.dev.grid.tf/graphql
10:14AM DBG checking url=https://graphql.dev.grid.tf/graphql
10:14AM DBG checking url=https://graphql.dev.grid.tf/graphql
10:14AM DBG checking url=https://graphql.dev.grid.tf/graphql
10:14AM INF project is canceled project name=vm/group_a
ok: {}

10:14AM INF Deployment took 3.351210766s

updated tfrobot filtration to use zos3 node if it failed to find zos4 node with requested resources

before the updated tried to run deployments for a configuration file and got this errors

after the update tried to run deployments for the same configuration and the deployer was able to recover from the errors


[Sprint  22 / PD-50] - [Feature] Add implementation GRPC

Jira Link:
https://ichiro-its.atlassian.net/browse/PD-50?atlOrigin=eyJpIjoiYzAxZWU5MzYxMGQwNDQ0ZWE3MjY2ZDQxMDg2MzdhYmEiLCJwIjoiaiJ9
Description
I have added a gRPC feature to read and save config. The gRPC server runs on port :5050 and consists of 2 RPCs, GetConfig and SaveConfig. I have tried this using Postman, and it works successfully. Due to issues in the previous branch and unsuccessful attempts to rebase with the master several times, so i created a new branch.
Type of Change

[ ] Bugfix
[ ] Enhancement
[x] New feature
[ ] Breaking change (fix or feature that would cause the existing functionality to not work as expected)

How Has This Been Tested?

[ ] New unit tests added.
[x] Manual tested.

Checklist:

[x] Using Branch Name Convention

feature/JIRA-ID-SHORT-DESCRIPTION if has a JIRA ticket
enhancement/SHORT-DESCRIPTION if has/has no JIRA ticket and contain enhancement
hotfix/SHORT-DESCRIPTION if the change doesn't need to be tested (urgent)

[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] I have made the documentation for the corresponding changes.

also, is it possible for you to refactor into 1 file containing 1 class for 1 service? Seems too many classes inside ConfigGrpc. If you think this will be too much changes, can create new task for this

o okay mbak, I'll will try to refractor  after the akushon is finished and also refractor akushon too

o okay mbak, I'll will try to refractor after the akushon is finished and also refractor akushon too

you can fix the code format first, for this refactor can be in separated task

Anyway @marfanr, since your changes is 800++, i expect more details on description of changes that you were made.


Update mesh_list.go

feat(cmd/cli): Added requested features
Mesh list command now display controller pods and joined namespaces.
Resolves issue #1841
Signed-off-by: mudit singh<EMAIL_ADDRESS>
resolve #1841

Affected area:

New Functionality      [ ]
Documentation          [ ]
Install                [ ]
Control Plane          [ ]
CLI Tool               [X]
Certificate Management [ ]
Networking             [ ]
Metrics                [ ]
SMI Policy             [ ]
Security               [ ]
Tests                  [ ]
CI System              [ ]
Performance            [ ]
Other                  [ ]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project? If so, did you notify the maintainers and provide attribution?
No

Codecov Report

Merging #2362 (e41bb64) into main (1c00c37) will decrease coverage by 0.14%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #2362      +/-   ##
==========================================
- Coverage   58.55%   58.40%   -0.15%
==========================================
  Files         149      149
  Lines        6660     6677      +17
==========================================
  Hits         3900     3900
- Misses       2746     2763      +17
  Partials       14       14

Impacted Files
Coverage Δ

cmd/cli/mesh_list.go
25.45% <0.00%> (-11.39%)
: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 1c00c37...e41bb64. Read the comment docs.

Codecov Report

Merging #2362 (e41bb64) into main (1c00c37) will decrease coverage by 0.14%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #2362      +/-   ##
==========================================
- Coverage   58.55%   58.40%   -0.15%
==========================================
  Files         149      149
  Lines        6660     6677      +17
==========================================
  Hits         3900     3900
- Misses       2746     2763      +17
  Partials       14       14

Impacted Files
Coverage Δ

cmd/cli/mesh_list.go
25.45% <0.00%> (-11.39%)
: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 1c00c37...e41bb64. Read the comment docs.

Thank you @mudit-01 ! This is a great addition. Would you mind adding some unit tests here?

Sure will add.

Thank you @mudit-01 ! This is a great addition. Would you mind adding some unit tests here?

Sure will add.

Thank you @mudit-01 ! This is a great addition. Would you mind adding some unit tests here?

@michelleN will it be ok if I remove the functions(getJoinedNamespaces and getControllerPods) as they only perform small tasks?

Thank you @mudit-01 ! This is a great addition. Would you mind adding some unit tests here?

@michelleN will it be ok if I remove the functions(getJoinedNamespaces and getControllerPods) as they only perform small tasks?

Thank you @mudit-01 ! This is a great addition. Would you mind adding some unit tests here?

Added unit test

Thank you @mudit-01 ! This is a great addition. Would you mind adding some unit tests here?

Added unit test

Approve

Thanks @Aisuko for the review :)

@michelleN its ready for review

@michelleN thanks for the review is there any other change requested as the merging is blocked?

@mudit-01 - Could you squash your commits for us so we can merge this in easily?

Added the hold label until commits get squashed so maintainers don't accidentally merge.  cc/ @mudit-01 @shashankram

@mudit-01 - Could you squash your commits for us so we can merge this in easily?

Hi @michelleN I am not sure of how to squash my commits isnt it done while merging?

Hi, @mudit-01 - while a repo's maintainers can squash commits for you if it's configured for that, at this time, this repo is set up so that the person submitting the PR needs do the squash. You may find https://stackoverflow.com/questions/14534397/squash-all-my-commits-into-one-for-github-pull-request useful - basically you want to turn all your commits on this PR into one commit and force push it to your fork on this patch branch. Please let us know if you would like more details. Thanks!

Thanks @bridgetkromhout I have squashed the commits as per link, but now its the DCO error maybe I have missed "<>" any workaround to it?

@mudit-01 Great! You're almost there - now you need to sign that one commit, like described here: https://github.com/openservicemesh/osm/pull/2362/checks?check_run_id=1955479159
Thanks!

Thanks @bridgetkromhout for the help to resolve the DCO, is it ok now?

@shashankram @michelleN Sorry I lost track and messed up the PR while performing squash, I should delete this PR and raise a new one.

@mudit-01 - Let me know if you need any help with rebasing or figuring any git stuff out. I'm happy to jump on a zoom with you. feel free to ping me on the cncf slack if ya like.

@mudit-01 almost there, you still need to squash the commits in https://github.com/openservicemesh/osm/pull/2362/commits locally and force push.

The linter github checks are failing. @mudit-01 could you format the file as per the errors displayed in https://github.com/openservicemesh/osm/pull/2362/files

@mudit-01, needs a squash again! Don't forget to properly format your commit by including the signature in the commit body and not the commit title, as is the case now. Sample commit msg:
Author: Joe Smith<EMAIL_ADDRESS>Date:   Thu Feb 2 11:41:15 2018 -0800

cmd/cli: update mesh list command

Updates mesh list command to display controller pods
and joined namespaces.

Signed-off-by: Joe Smith<EMAIL_ADDRESS>

Hey @mudit-01, please refer to https://github.com/openservicemesh/osm/blob/main/CONTRIBUTING.md#pull-request-workflow if you haven't already. This has the steps listed on creating a pull request.

Hi @shashankram I have created the PR as given in the document I am experiencing issue while squashing commits

Hi @shashankram I have created the PR as given in the document I am experiencing issue while squashing commits

I don't think you are squashing the commits correctly. You should not be using git pull or git merge while rebasing and squashing, which appears to be done in your case.
I have created a PR on your behalf (you are still the author of the commit): https://github.com/openservicemesh/osm/pull/2596
We can close this one.

Hi @shashankram I have created the PR as given in the document I am experiencing issue while squashing commits

I don't think you are squashing the commits correctly. You should not be using git pull or git merge while rebasing and squashing, which appears to be done in your case.
I have created a PR on your behalf (you are still the author of the commit): #2596
We can close this one.

Thank you very much @shashankram from next PR I will definitely take care of these things.

Closing this in favor of https://github.com/openservicemesh/osm/pull/2596


improvement: remove custom logo and use Unique Official favicon

To kickstart the project, a custom logo "fork" was added.

Replace the chart icons with: https://www.unique.ch/hubfs/unique_assets/favicon/glyph_caumasee.svg
Also replace them in the index.yaml on the gh-pages branch
Remove the custom svg

Fixed in #19


plotting tools for ptype data

Python script that has several plotting functions including uncertainty, probability, ptype, winds, dewpt, sp, and temp. Used primarily for generating useful visualization for a single valid time or a time lagged ensemble.

Thank you for making the changes. The only request I have is to move plotting_tools.py to the ptype-physical/ptype/ folder and change the imports accordingly (from ptype.plotting_tools import ...) as this is our typical code structure. Everything else looks good to me.

Thanks for updating the structure. Looks good to me, merging.


[Feature Request] Add additional filters to "get_transactions"

I'd like to see the ability to search by Merchant and Amount.
I'm trying to write a script to categorize my amazon transactions based on my downloaded amazon history.  Finding a transaction by date, merchant and amount should allow me to identify most transactions programmatically.

I cloned the repo and played around with the code looking at the chrome dev tools (first time using the dev tools and playing around with apis), i figured out how to find transactions by amount.  There are 2 values, that i assume are floats, that can be used as search filters.
absAmountGte & absAmountLte
I assigned them as floats and it works.  I've never done a pull request before or else i'd submit one.
under the async def get_transactions() block at line 1412 I added the following 2 lines
        AmountGt: Optional[float] = None,
        AmountLt: Optional[float] = None,

then there are a series of if statements under the comment
"# If bool filters are not defined (i.e. None), then it should not apply the filter"
so at line 1546 I added
        if AmountGt is not None:
            variables["filters"]["absAmountGte"] = AmountGt
        if AmountLt is not None:
            variables["filters"]["absAmountLte"] = AmountLt

Merchants appears to just be a list of Merchant Id's with an api value of "merchants", but i haven't added the code for it yet.


Include XHR request time

Do you want to request a feature or report a bug?
feature
What is the expected behavior?
Would love it if the XHR breadcrumb could include the request time, e.g. time from when the request starts to when the Promise resolves. We're having some "slow loading" problems with some users, and having this available in Sentry would make things much more easy to debug. Seems to be easy to include the additional data in the fetchData property, but I'm not sure if there needs to be additional changes to the server in order for it to be presented correctly.

Seems to be easy to include the additional data in the fetchData property, but I'm not sure if there needs to be additional changes to the server in order for it to be presented correctly.

@genne – I can see how this feels like an easy thing to add, but there's a lot of small feature suggestions like this that over time will serve to bulk up the library past it's relatively small 8kb footprint today.
If you're interested in this data, I'd recommend looking into window.performance. You can use dataCallback to basically dump HTTP information into extra_data, which can give you a window of loading times for a given captured exception/message. Note though that there are data limits on extra_data, so I don't recommend just dumping the entire object in there – you should probably reduce it to the bits you want.

@benvinegar perhaps just add in an optional callback at different stages of instrumentation?

@ThisIsMissEm – I'd have to do both a pre and post callback for that to work. That's pretty complicated, and again, more code.

hmm.. I might have a crack at this one day; I'm thinking just:
if (typeof config.preFetchDataHook === 'function') config.preFetchDataHook(/* args */)

Which I'm sure isn't too much. Maybe this is me being optimistic though.

This is a serious issue, if for example you have some jQuery ajax that due to some network issue or temporary problem takes minutes or never complete, an error is never ever sent through Sentry!

This can be achieved through custom transport in new SDK


.mat-form-field CSS blocks custom ones.

Bug:
.mat-form-field overwrites custom css.
What is the expected behavior?
custom-component.css should overwrite the style sheets that come with Angular Material.
What is the current behavior?
The style sheet that comes with Angular Material is blocking custom-component.css.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
"@angular/material": "^2.0.0-beta.10"
Is there anything else we should know?

Increase your specificity

Closing.  As mentioned you will need to increase your specificity, unfortunately we are unable to get a ton less specific than one class name without being too broad.


webpack-dev-server version problem

Do you want to request a feature or report a bug?
bug

What is the current behavior?
Initialize the project with vue scaffolding,webpack-dev-server2.9.1version 'Syntax error',webpack-dev-server2.7.1version  normal operation

What is the expected behavior?
webpack-dev-server2.9.1version also can normal operation
Please mention other relevant information such as the browser version, Node.js version, webpack version and Operating System.
ie9+
nodejs 8.4.0
windows 7 64 bit

Closing due to inactivity. Please test with latest webpack and webpack-dev-server version and feel free to reopen if still regressions. Thanks!


I413 correct cache clearing documentation

Issue 413
Correct documentation that says where to clear the EC cache.

This PR was closed due to an error on which initial base to use.


Roll Flutter Engine from 3460f349b01d to 332c0a2f207c (1 revision)

https://github.com/flutter/engine/compare/3460f349b01d...332c0a2f207c
2022-12-29<EMAIL_ADDRESS>Roll Skia from c776239198f7 to 13435162b783 (1 revision) (flutter/engine#38561)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Remote control event not working in the sample app.

In the MediaSample android app the remote control events for controlling the playback state are not working. Clicking the play/pause button on the lock screen and in the notification area are not working. I am testing on Galaxy s6 (api 23)

I have a hacky fix for this but not sure if you are going to like it. Basically what is happening is the SessionManager that is passed from the client is null when remote event is triggered. The session manager is set on the AudioPlayerBase but the service has no access to it when notification is received.
With this change in $/XamarinMediaManager-master/MediaManager/Plugin.MediaManager.Android/MediaPlayerService/MediaServiceBase.cs
I am able to handle remote events.
public void HandleIntent(Intent intent)
{
if (intent?.Action == null || SessionManager == null)
{
var c = (MediaManagerImplementation)CrossMediaManager.Current;
SessionManager = c.MediaSessionManager;
}
SessionManager?.HandleAction(intent?.Action);
}


Enable multiple configuration for multi-trajectory

Hi, I just started to enable multi-trajectory on cartographer_ros. This is some portions for that. Could you review it?

I did not look at the code yet, I think we should get the base branch in.
But one of the things I wonder immediately is how the user will start a new trajectory with different options. So how do you envision .lua files and ROS services to change for this?

Yes, I should get PR of cartographer 248 done first.
I am working on making a "start_trajectory" ros service to start a new trajectory with different options and subscribers for new input topics.

Let me know when this is ready for another look.

I think this PR could be closed because https://github.com/googlecartographer/cartographer_ros/pull/329 covered this.
Regarding "start_trajectory" service, I will send another PR.


Surface standalone

Made surfaces no longer owned by the shells.
Thus is to fix #175
TODO

[ ] segfault when closing compositor when there are surfaces

Superceded by #184


Event shown is not the nearest upcoming

Describe the bug
The widget does not show the next upcoming event, but rather the one that is furthest away.
For example: my next upcoming calendar event is tomorrow at 1pm. After that I have another event tomorrow at 5pm. The widget shows the 5pm event, instead of the 1pm event.
Expected behavior
I would expect the widget to show my next event (1pm in my example), instead of the one furthest away.
Smartphone (please complete the following information):

Device: Samsung Note 10+
OS Version: Android 10
App Version: v2.0.14 (106)

Lastest beta version should have resolved this issue.


algo va mal

Does this occur consistently?
Repro steps:

Action: azureFunctions.deploy
Error type: NoResouceFoundError
Error Message: No matching resources found.
Version: 0.23.0
OS: linux
OS Release: 4.15.0-112-generic
Product: Visual Studio Code
Product Version: 1.48.0
Language: es

Call Stack
AzureAccountTreeItemWithProjects.<anonymous> extension.bundle.js:1:575977
fulfilled extension.bundle.js:1:569926

@carlosnp please add detailed repro steps


docs: update Flask-Babel links

SUMMARY

This PR updates Flask-Babel links in the documentation site because the old links do not work now.
This PR also updates the Babel link in the contributing guide to make it consistent with the documentation site.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

Build the documentation site locally and verify that the new links work as expected.
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

Thanks for the updates, @haocheng6! We always love it when people help touch up the documentation. If you have further interest in this effort, ping us here or on Slack!


Georeferencing English text

These two labels shouldn't be capitalised like this, and the second is poor English:

In Which Column Are Your Region Names Stored?

--> Which column contains region names?

If Your Column Isn't Country Data, Where's It?

--> Which column contains country names (if applicable)?
or
--> If those regions aren't countries, which country are they in?
(At very least change "Where's it?" to "Where is it?" - the former is very un-idiomatic)

Select column or type it

--> Select column or type country name

Heh, I tried to find an explanation of why "Where's it" is not good English, then this happened: http://english.stackexchange.com/questions/281461/why-is-wheres-it-grammatically-incorrect

Thx for reporting! Will fix this eventually
cc @CartoDB/copywriters

I know we recently edited the Georeference /  Lon/Lat Columns text, I'm trying to find that issue for you as a reference...  Please edit the Georeference / Admin Regions  text as follows:
original: "Select the column that has Administrative Regions
Georeference your data by country’s, state's, province's or municipality’s"
edited: "Select the column that has Administrative Regions
Georeference your data by country, state, province or municipality"
original: "1. In Which Column Are Your Region Names Stored? Select column"
edited: " Select Your Region Name, Select column"
original: 2. If Your Column Isn't Country Data, Where's It? Select column or type it"
edited: "Select Your Country Data, Select column or type it"

fixed on new designs! Thanks!


Add rel-mask-icon link tag

[ ] head tag
[ ] SVG icon

https://developer.apple.com/library/content/documentation/AppleApplications/Reference/SafariWebContent/pinnedTabs/pinnedTabs.html

Pairs well with #164


Drop the "size" optimize mode

As I understand, David studied the construction of compilers, that is why he create a library with two possible implementations:

statically generated functions ("speed" optimizing mode)
interpreted bytecode ("size" optimizing mode)

Frankly speaking, size of the generated parsers in the "size" mode don't differ much from these parsers, generated in the "speed" mode, but their parsing speed is worse.
Generating in the "size" mode would be valuable if the generator had generated only bytecode instructions; but it is also generating the interpreter, which for small grammars don't differ much in the size from the grammars, generated in the "speed" mode.
Judging that, there is no reasons to keep the "size" mode, so I suggest to drop it. I think, that nobody uses that mode.

Presumably, the option would still be legal, but ignored (at least for now)?
(leaving open the possibility we come up with a completely different optimization later)

I think it would be good to fix the whitespace issues and other things like that before we make a decision here.  After that the decision may look similar or may look different
I think Mingun is probably right, but re-adding it afterwards would be brutal

I think it would be good to fix the whitespace issues and other things like that before we make a decision here. After that the decision may look similar or may look different

SGTM.  Depends on #65 then, and perhaps #68.


buildguide: changed require version of gcc

I tested gcc version 4.5~gcc 6, waifu2x-convterter-cpp needs at least version 6 of gcc (c++17 is not fully supported in gcc 4 ~ gcc 5)
check #69 & #70

@YukihoAA can you add:
if(CMAKE_CXX_COMPILER_ID STREQUAL "GNU")
  if(CMAKE_CXX_COMPILER_VERSION VERSION_LESS "6.0")
    message(FATAL_ERROR "This software requires GCC6 or higher")
  endif()
endif()

@DeadSix27 I added new commit.

@YukihoAA I saw, thanks :)


Skip Project piece list request

The repository view displays the objects locked by project-generated piece list transport request

... and these types of requests don't lock the objects. Therefore, they should not be considered by AG.
LGTM


Encoding for "Windows-1252" not working in .NET Core

In a ASP.NET Core application the encoding for windows was not found, the line below failed:
public static readonly Encoding AnsiEncoding = Encoding.GetEncoding("Windows-1252");
I downloaded the master branch, in which this was fixed according to issue history. However, I did not get it working correctly. But if I did this it worked just fine:

Add nuget packet System.Text.Encoding.CodePages
Change the line above to:

public static readonly Encoding AnsiEncoding = CodePagesEncodingProvider.Instance.GetEncoding(1252);
If not issue or if I did misbehave close issue as you see fit :)

From this link, it's not an issue


Installation Issue - requirement python-algorithms-3x

Trying to install (windows 11, Python 3.11)
pip install data-structures3x==1.4.6
getting an error:
ERROR: Could not find a version that satisfies the requirement python-algorithms-3x (from data-structures3x) (from versions: none)
ERROR: No matching distribution found for python-algorithms-3x
Any idea who to resolve?
which package is the dependency on (tried several)
Thank you

The issue is arising from the fact that the package depends on python-algorithms-3x.
However on Pypi, python-algorithms-3x is published under the name superalgo.
I believe this is an unintentional mistake on the part of @timthedev07-buddy; but this prevents any installation of their packages since they are all dependencies of each other.
A fix would be if the package name for python-algorithms-3x is corrected by @timthedev07-buddy in all dependencies.
Another fix is to download the packages you need from Pypi, extract the tar.gz and use the files you need.


Fix open dialog default behavior on Linux

Open dialog now defaults to the same directory as the open file, or the user's home directory if there is no file active (such as when viewing the Settings page). This is the convention for gedit and other text editors on Linux. Because of the render/browser process split, this requires async messages to get the path over. The alternative is passing this data when calling application:open, but this requires more extensive changes.
Note: this change is only for linux but it could easily be expanded to other platforms assuming the dialog module works the same way.
Fixes at least #6762

fyi - I just built this now on debian 8 and your fix works.  I didn't realize your wording "user's home directory if there is no file active" - what should happen is that the current project directory opens if there is one.  The home directory should be the backup to that.  We also need to consider osx.  I'm looking into this now on a branch - given the debugging tips you gave.  I looked up the browser/renderer processes and can move forward with that reference now.


4.0.0: cannot build documentation

Something is wrong (I'm using sphinx 7.3.7)
+ /usr/bin/sphinx-build -n -T -b man doc/source build/sphinx/man
Running Sphinx v7.3.7
making output directory... done
[autosummary] generating autosummary for: api.rst, changelog.rst, cli.rst, definitions.rst, developer.rst, documentation.rst, examples.rst, features.rst, heterogeneous.rst, index.rst, install.rst, license.rst, migration.rst, morph_check.rst, morph_stats.rst, quickstart.rst, spherical_coordinates.rst, tutorial.rst, validation.rst

Traceback (most recent call last):
  File "/usr/lib/python3.10/site-packages/sphinx/ext/autosummary/generate.py", line 503, in generate_autosummary_docs
    name, obj, parent, modname = import_by_name(entry.name)
  File "/usr/lib/python3.10/site-packages/sphinx/ext/autosummary/__init__.py", line 658, in import_by_name
    raise ImportExceptionGroup('no module named %s' % ' or '.join(tried), exceptions)
sphinx.ext.autosummary.ImportExceptionGroup: no module named neurom.check.morphology_checks

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/lib/python3.10/site-packages/sphinx/events.py", line 97, in emit
    results.append(listener.handler(self.app, *args))
  File "/usr/lib/python3.10/site-packages/sphinx/ext/autosummary/__init__.py", line 818, in process_generate_options
    generate_autosummary_docs(genfiles, suffix=suffix, base_path=app.srcdir,
  File "/usr/lib/python3.10/site-packages/sphinx/ext/autosummary/generate.py", line 508, in generate_autosummary_docs
    name, obj, parent, modname = import_ivar_by_name(entry.name)
  File "/usr/lib/python3.10/site-packages/sphinx/ext/autosummary/__init__.py", line 715, in import_ivar_by_name
    real_name, obj, parent, modname = import_by_name(name, prefixes)
  File "/usr/lib/python3.10/site-packages/sphinx/ext/autosummary/__init__.py", line 658, in import_by_name
    raise ImportExceptionGroup('no module named %s' % ' or '.join(tried), exceptions)
sphinx.ext.autosummary.ImportExceptionGroup: no module named neurom.check

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/lib/python3.10/site-packages/sphinx/cmd/build.py", line 332, in build_main
    app = Sphinx(args.sourcedir, args.confdir, args.outputdir,
  File "/usr/lib/python3.10/site-packages/sphinx/application.py", line 268, in __init__
    self._init_builder()
  File "/usr/lib/python3.10/site-packages/sphinx/application.py", line 339, in _init_builder
    self.events.emit('builder-inited')
  File "/usr/lib/python3.10/site-packages/sphinx/events.py", line 108, in emit
    raise ExtensionError(
sphinx.errors.ExtensionError: Handler <function process_generate_options at 0x7f50ee4c5c60> for event 'builder-inited' threw an exception (exception: no module named neurom.check)

Extension error (sphinx.ext.autosummary):
Handler <function process_generate_options at 0x7f50ee4c5c60> for event 'builder-inited' threw an exception (exception: no module named neurom.check)

Ah, it's the --nitpicky, -n        nit-picky mode: warn about all missing references option that gives the errors.

Yep .. those warnings are affecting html output. Some parts are highlighted but there is no links.
I'm not affected by those issue because I'm interested only roff output AKA man page but I'm reporting such things as parto f the packaging process 😋

I think these should be fixed by:
https://github.com/BlueBrain/NeuroM/pull/1124
Perhaps you can test @kloczek?

Any luck with this @kloczek?

I think https://github.com/BlueBrain/NeuroM/pull/1124 covers the problems, so I'm going to close this.
If you find more errors, please open a new issue.


Export-MsIdAzureMfaReport

As described, the Export-MsIdAzureMfaReport "exports the list of users that have signed into the Azure portal, Azure CLI, or Azure PowerShell over the last 30 days by querying the sign-in logs. In Microsoft Entra ID Free tenants, sign-in log retention is limited to seven days."
Since Microsoft will soon require MFA for the Entra admin center, Intune admin center, such as Windows 365 Cloud PC, could you please update the report to include activities from these services as well?
Thank you!

Due to the way App IDs etc work the list of users you see includes portals that depend on the Azure Portal.
So when you see Azure Portal ticked it includes Entra, Intune etc as well.


File Does Not Launch on Manjaro KDE

After integrating with appimage launcher, the appimage does not launch. Running Manjaro with KDE Plasma as the DE. If you need more info just tell me

I had the same problem with Fedora 34. After debugging, I realized that some dependencies were missing.
Try again with the latest build.

It works now!


deno svelte kit

Describe the problem
svelte kit is great project, but building it using node runtime make it difficule to maintain: security pb and modularity system ( node module)
Describe the proposed solution
svelte kit is great project, but why not use deno instead of node ? this may be a new project but it will not be from scrach.
existing adapter can be updated or rewriting to work.
there are many benefit of using deno :

good dev experience ( no node_module)
easy debug, most of deno module are ts
no loosing all existing work in deno svelte since we use the kanguage
using existing hosting plus deno deploy
and more

Alternatives considered
No response
Importance
would make my life easier
Additional Information
No response

This looks something like a deno adapter like this [adapter-deno](https://github.com/pluvial/svelte-adapter-deno) can fix.  Can you try this out?

SvelteKit depends on a number of other projects (notably Vite) that only work in Node. You can use an adapter to produce a build that can run on Deno in production. but it's not even remotely practical for the development experience to also use Deno.

hey could we re-visit this now that deno supports npm and vite?
I've been playing around trying to get deno to run my svelte-kit starter template but coming up with errors atm.
$ deno task dev
Task dev deno run -A --node-modules-dir npm:vite

  VITE v3.2.4  ready in 1057 ms

  ➜  Local:   http://localhost:5173/
  ➜  Network: use --host to expose
TypeError: types.isProxy is not a function
    at Object.record<ByteString, ByteString> (file:///home/aaron/Documents/threlte-BVH/node_modules/.deno/undici@5.12.0/node_modules/undici/lib/fetch/webidl.js:267:16)
    at Object.webidl.converters.HeadersInit (file:///home/aaron/Documents/threlte-BVH/node_modules/.deno/undici@5.12.0/node_modules/undici/lib/fetch/headers.js:493:63)
    at new Headers (file:///home/aaron/Documents/threlte-BVH/node_modules/.deno/undici@5.12.0/node_modules/undici/lib/fetch/headers.js:169:32)
    at ServerResponse.<instance_members_initializer> (https://deno.land/std@0.165.0/node/http.ts:244:14)
    at new ServerResponse (https://deno.land/std@0.165.0/node/http.ts:267:5)
    at https://deno.land/std@0.165.0/node/http.ts:579:23
    at new Promise (<anonymous>)
    at handler (https://deno.land/std@0.165.0/node/http.ts:578:16)
    at Object.serve (deno:ext/flash/01_http.js:559:24)
    at async Promise.all (index 0)

or maybe someone can point me in the right direction?

hey could we re-visit this now that deno supports npm and vite?
I've been playing around trying to get deno to run my svelte-kit starter template but coming up with errors atm.
$ deno task dev
Task dev deno run -A --node-modules-dir npm:vite

  VITE v3.2.4  ready in 1057 ms

  ➜  Local:   http://localhost:5173/
  ➜  Network: use --host to expose
TypeError: types.isProxy is not a function
    at Object.record<ByteString, ByteString> (file:///home/aaron/Documents/threlte-BVH/node_modules/.deno/undici@5.12.0/node_modules/undici/lib/fetch/webidl.js:267:16)
    at Object.webidl.converters.HeadersInit (file:///home/aaron/Documents/threlte-BVH/node_modules/.deno/undici@5.12.0/node_modules/undici/lib/fetch/headers.js:493:63)
    at new Headers (file:///home/aaron/Documents/threlte-BVH/node_modules/.deno/undici@5.12.0/node_modules/undici/lib/fetch/headers.js:169:32)
    at ServerResponse.<instance_members_initializer> (https://deno.land/std@0.165.0/node/http.ts:244:14)
    at new ServerResponse (https://deno.land/std@0.165.0/node/http.ts:267:5)
    at https://deno.land/std@0.165.0/node/http.ts:579:23
    at new Promise (<anonymous>)
    at handler (https://deno.land/std@0.165.0/node/http.ts:578:16)
    at Object.serve (deno:ext/flash/01_http.js:559:24)
    at async Promise.all (index 0)

or maybe someone can point me in the right direction?

you can discus about this in deno discor,  you can get help there

cheers


Use the same "Jupyter is not installed" message everywhere

For https://github.com/microsoft/vscode-python/issues/16102
This PR does 2 things:

jupyterNotInstalledPrompt -> showJupyterNotInstalledPrompt in  src/client/jupyter/jupyterNotInstalledNotificationHelper.ts
remove installJupyterExtension from src/client/jupyter/jupyterExtensionDependencyManager.ts

Insiders failure not related to this PR, merging it.


[AutoPR datafactory/resource-manager] Fix ssis execution credential

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/5037

Closing PRs that are older than 1 month.


Allow group config from function

I have a system that automatically creates an array that can be accepted as groups but I get a fatal error involving return when I try to require classes in the default config file.

It is because the wrong headers are sent.

Nevermind. I fixed it.


NTUEEPLUS-40. Footer makes width of page bigger

What is this PR for?
Fix the Footer's size and background colors.
What type of PR is it?
[ Improvement |
Todos
None
What is the Jira issue?
https://ntueeplus.atlassian.net/browse/NTUEEPLUS-40
Screenshots (if appropriate)

@featherchen  @vwvwMM


Add new javasrc2cpg type model and JarTypeReader

This is the first PR in the project to move javasrc2cpg type solving away from the sometimes broken javaparser symbol solver. This introduces a new type model and the JarTypeReader which extracts the necessary information from JARs. None of this information is currently used, but PRs for that will follow.

I need some help understanding the reason for not using this work already...do the frontend tests pass? if yes, then why not switch directly? if not, then what are the open issues?

I need some help understanding the reason for not using this work already...do the frontend tests pass? if yes, then why not switch directly? if not, then what are the open issues?

This is the first part of a more research-y exploration of java types and the only purpose of this PR is to add the component that can read the type information we need from JARs. It is not yet the full type solver (and at this point it's still unclear what exactly that will look like). The plan is to create separate PRs for the various components that will eventually make up part of this, since that makes the components easier to review, creates a more structured breakdown and timeline of what is included in the solver and avoids the extra cost of maintaining a separate branch that's ever growing.
So I guess the reason for not using this comes down to the scope of the PR vs the scope of the project as a whole. When we get to the point where the frontend gives the same results with the new types as with javasrc, then the scope of the project has been fulfilled. For now, however, the scope of this PR is just the JAR type reader which is working at least as far as shown by the unit tests included here.


Detect --commit-range automatically

Different CI systems have different ways of providing us
this information - some easily & some not. We can add support
for the major providers here so we have some way to automatically
detect it.
This adds support for detecting commit-range value in github
actions.

Codecov Report

Merging #32 into master will decrease coverage by 1.9%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master      #32      +/-   ##
==========================================
- Coverage   28.76%   26.85%   -1.91%
==========================================
  Files           5        6       +1
  Lines         226      242      +16
==========================================
  Hits           65       65
- Misses        161      177      +16

Impacted Files
Coverage Δ

hubploy/__main__.py
0% <0%> (ø)
:arrow_up:

hubploy/commitrange.py
0% <0%> (ø)

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 6f684ad...79ad7de. Read the comment docs.


Code Timer

We need a timer that monitors the time it takes the user to solve the challenge.
We mainly have two options:

Implement it in the front end, but there is a risk that the user will mess with it.
Implement it in the back end, the timer would run as a background task.

Any changes can be discussed below.

Perhaps implement on the front end but also have the background task on the backend. We can check periodically that the timers match up and sync them if not. Maybe through the connection and sync events as I don't think they will be out of sync very often and these events don't fire too much.

Perhaps implement on the front end but also have the background task on the backend. We can check periodically that the timers match up and sync them if not. Maybe through the connection and sync events as I don't think they will be out of sync very often and these events don't fire too much.

Why though? Wouldn't one timer be enough?

Perhaps implement on the front end but also have the background task on the backend. We can check periodically that the timers match up and sync them if not. Maybe through the connection and sync events as I don't think they will be out of sync very often and these events don't fire too much.

Why though? Wouldn't one timer be enough?

It would, yeah. Thought it would be nice to display to the frontend though.

Perhaps implement on the front end but also have the background task on the backend. We can check periodically that the timers match up and sync them if not. Maybe through the connection and sync events as I don't think they will be out of sync very often and these events don't fire too much.

Why though? Wouldn't one timer be enough?

It would, yeah. Thought it would be nice to display to the frontend though.

Oh right, I didn't think about that. Also because having an event just for the timer might be too much.

Exactly, the server and client will be talking a lot with sending possibly hundreds of requests per min


PTRENG-1055 - add support for creating users

[x] All tests passed. If this feature is not already covered by the tests, I added new tests.
[x] This pull request is on the dev branch.
[x] I used gofmt for formatting the code before submitting the pull request.

Changes made based on other PR comments

Changes made based on other PR comments

@chb0github,
Since https://github.com/jfrog/jfrog-client-go/pull/271 already adds user management support, this PR can probably be closed.

@chb0github,
Since https://github.com/jfrog/jfrog-client-go/pull/271 already adds user management support, this PR can probably be closed.


Deprecate Input Helper component

Description
The Sage Input Helper was intended for use in a feature that was partially implemented, but ultimately shelved. We can safely remove this component and its references within the Input Group as there is no (known) future use.

Duplicate Issue: https://github.com/Kajabi/sage-lib/issues/196


Made embedding_dim configurable based on vocab

I also changed embedding_size to embedding_dim, which is why this PR touched so many files.  The main difference is in text_trainer.py.  And while I was at it, I updated a couple of tests to add a new convenience method I added a few PRs ago, and that revealed some problems in _set_max_lengths_from_model, which I also fixed.
And I removed a test that I didn't think was necessary anymore, because we don't really use the multiple choice memory network - I didn't want to do the work necessary to make the test pass with this new set up.

And I removed a test that I didn't think was necessary anymore, because we don't really use the multiple choice memory network - I didn't want to do the work necessary to make the test pass with this new set up.

So the multiple choice memory  network is broken, or just the test?

The memory network code is at the moment incompatible with the words and characters tokenizer, if you use a bag of words encoder, because of encoding dimension stuff.  It still works with other tokenizers, as the tests I left in show.  I figured it wasn't worth the time to make that old code compatible with words and characters tokenizers.

The memory network code is at the moment incompatible with the words and characters tokenizer, if you use a bag of words encoder, because of encoding dimension stuff.

ok, that should be mentioned somewhere appropriate...not sure where though

I also just realized I forgot to update all of the example experiments to use the new parameter specification.  I'll do that in another PR, in which I'm also updated bidaf_squad.json to use more sane parameters.


Add support for summary list component

@jonheslop found that the summary list component wasn't working; this PR adds a transformation to njk_to_j2 to fix it; it also adds tests (although these will currently be skipped because we're testing against govuk-frontend 2.4.0.

Woops I merged this by accident :sweat_smile:


Bump ubuntu version

Ubuntu 20 is going to lose support this year. Additionally, it comes with Python 3.8, which is no longer supported.

I guess this line is no longer required:
https://github.com/anza-xyz/platform-tools/blob/3837a8df7967e65c9ee3dc7a83b76293362e0836/build.sh#L120
You can allow it to fail like this cp -R rust/build/${HOST_TRIPLE}/llvm/lib/python* deploy/llvm/lib/ || true

I guess this line is no longer required:
https://github.com/anza-xyz/platform-tools/blob/3837a8df7967e65c9ee3dc7a83b76293362e0836/build.sh#L120
You can allow it to fail like this cp -R rust/build/${HOST_TRIPLE}/llvm/lib/python* deploy/llvm/lib/ || true

It is required. Python wasn't built due to a missing component.

The Python folder is necessary to run the Solana debugger. We need python3-dev and swig on Ubuntu for it to be created. It works on MacOS (see the CI), but on Linux the folder silently isn't constructed.


[Idea] Give the ability to zoom for large amounts of data

Hi,
I currently use chartkick.js to parse a lot of data with a scatterplot, to plot messages over time. Is there anyway to zoom in so you can see where the specific points are, instead of looking at one big blob of points?

Hey @DrySoldier, sorry for the delay. There isn't a way to zoom as far as I know. You'll probably want to use a charting library directly.


Support for Model Versioning

Describe the feature
This extension does not seem to support dbt models that have been versioned (e.g., <model_file_name>_v2.sql with model config properties).
Tests, parent/child models, and documentation that used to appear for a non-versioned model no longer appear once that model has been versioned. Do note that all of this does work from dbt itself, so the issue appears isolated to just this plugin.
Describe alternatives you've considered
No response
Who will benefit?
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Any update on this issue? It's starting to become a blocker in our versioning and migration work.

@smitsrr We plan to pick this up later this month. I will keep you all posted.

@smitsrr We estimated that implementing this requires a multi-week effort since we need to introduce a new identifier type for model nodes across the codebase.  We don't have the bandwidth to take this up in the next month or so. I will keep you posted if things change in the future.

hi, is there an update on this enhancement?

Both Lineage and documentation editor will not work for Versioned Models, i'd also greatly appreciate a fix

Would greatly appreciate this feature, as currently using versioned models makes the lineage functionality of DBT Power User un-usable 🙏

Hello @cashel @wahlrkr @MalteNiederstadt @chetchavat @smitsrr , in the latest release, we have added model version support, which is a significant change impacting almost all functionalities of the extension. We would greatly appreciate it if all of you could try it out and let us know if everything is working as expected. Thanks!

Lineage works great at first glance, i'll keep testing and report any issues.
Thanks for your efforts, appreciate it greatly.


Run cargo clippy when building the Pokémon Service

We're only running cargo test in CI:
https://github.com/awslabs/smithy-rs/blob/fd94858db80d4ea03cbff47f223171172e85faae/tools/ci-build/scripts/check-server-e2e-test#L10
https://github.com/awslabs/smithy-rs/blob/fd94858db80d4ea03cbff47f223171172e85faae/tools/ci-build/scripts/check-server-python-e2e-test#L9

Are you wanting the rust-runtime Makefiles to be edited to include clippy as a target? Or would you rather have the cargo clippy command injected in the respective ci-action/scripts? Do you want to use --fix? I you want the action scripts to be edited, I noticed the Python one copyies some dynamic libraries around during build -- does this need to happen for cargo clippy to work correctly? (For the Python script there is just a call to make test and no reference to be able to use clippy outside of the Makefile definition, so I wasn't sure.

I guess we should create a test target in aws-smithy-http-server/examples/Makefile just as we have in aws-smithy-http-server-python/examples/Makefile, and then both CI scripts (1, 2) invoke make test, for consistency.

Are you wanting the rust-runtime Makefiles to be edited to include clippy as a target?

Yes. The CI scripts then just invoke make clippy.

Do you want to use --fix?

No.

If you want the action scripts to be edited, I noticed the Python one copies some dynamic libraries around during build -- does this need to happen for cargo clippy to work correctly?

Should not be required, no.

Just out of curiosity, what's the benefit here? Is this to just convey "we use clippy and you should too, see how it can be included in this sample project"?

Yeah, mostly. We want to provide a good example service and running clippy in CI is a good practice to follow idioms and such.

Closed by https://github.com/awslabs/smithy-rs/pull/1742.


Invalid parameter number with LIKE mode

When I change to LIKE mode I'm getting an error showing that there are too many parameters.
SQLSTATE[HY093]: Invalid parameter number: parameter was not defined
"select count(*) as aggregate from recurring_profiles where account_id = :account_id AND (id LIKE :_search0 OR account_id LIKE :_search1 OR title LIKE :_search2) and recurring_profiles.deleted_at is null",
 [
 1,
"%liabiliti%",
"%liabiliti%",
"%liabiliti%",
"%liabiliti%"
 ],

It is happening when I search multiple models in the same request


Average of different prediction horizons as a metric?

Hello Authors,
Could you please clarify the usage of the average of different prediction horizons as a benchmarking metric? Why was it used, and how to justify the validity of this?
I am doing a similar project and trying to report values at different horizons. My model is not getting values close to those reported in SOTA (top 5) models like yours.  Could you please help with the intuition on reporting the average rather than individual horizons?
Thanks
Santosh

IIRC that's a convention inherited by Informer and the followup works to it that have come out since this repo's initial release and before it's more recent versions. The accuracy of individual timesteps into the future can be arbitrary and hard to interpret. 1 step predictions are too easy, but distant predictions can be very difficult given a fixed length context window which may be too short. In highly periodic domains some distant horizons can also be easy (such as 24 hours ahead in a dataset with clear daily periodicity like weather forecasting). So reporting every horizon metric takes a lot of explaining, large tables, and can be misleading. Averaging gives a better sense of the model's performance over the entire duration we care about.
At a few points during this project I hacked together logging metrics for accuracy at each individual timestep as a sanity-check. In my experience you can expect a roughly linearly increasing error as you predict further into the future.
As far as replicating the results on these datasets in your own project, double check that you aren't counting missing datapoints in the metrics. This can make a huge difference and is something a lot of the literature (and early versions of this codebase) get wrong.

I agree with Jake, averaging over the whole prediction horizon makes sense in order to compare single numbers as a metric.
It is a pity though that different benchmarks use different metrics.
For example, check here for PEMS-Bay:
https://paperswithcode.com/sota/traffic-prediction-on-pems-bay
They report RMSE (I guess this is averaged over the whole horizon)
and MAE @ 12 step (this is for a single prediction 12 steps into the future)
It would be good to have more standardized metrics.
In the paper there is no RMSE for PEMS-Bay. There is MAE, MSE and MAPE, but unfortunately PapersWithCode does not report those.
This is not a question, just a comment, sorry for the spam! 😁

Yeah the traffic datasets / literature is the main example where reporting multiple horizons is the default. The longest horizons are 12 timesteps so this can be feasible. Once you get longer than that it stops making sense to report arbitrary intervals in tables in my opinion. It would be interesting if the convention for reporting forecasting results was a plot of error over forecast duration for each dataset. That wasn't necessary at the time (2021) but I think this is probably what I would do if I were to redo this project today...


Old Wallpaper URL no longer reachable

What's the problem?
The old wallpaper address is not accessible (error 404 - not found).
On line 848 of the "install.ps1" script, I replaced the old address with "https://github.com/mandiant/flare-vm/blob/main/Images/flarevm-background.png".
Update required.
Thanks ;-)
Steps to Reproduce

Environment
VM Win10x64 with VMWare.
Additional Information

The url to download the image is https://raw.githubusercontent.com/mandiant/flare-vm/main/Images/flarevm-background.png, the url you are trying to use is the GH page that displays that image and that is not what you want. That is what is used in the latest version of the installer.
As mentioned in the bug issue template, please use the latest version of the installer and do not report bugs for previous versions.


Custom ESPEventLoop

Hello!
Can someone please help me the create a custom ESPEventLoop? If the constructor of ESPEventLoop is called without a parameter, then the default event loop gets created. Should I store this custom ESPEventLoop as a shared_ptr withing my class?
Thank you in advance!

@Teodor12 The constructor of ESPEventLoop always takes an std::shared_ptr<ESPEventAPI>. I.e., you need to create a shared_ptr with ESPEventAPICustom and pass it to the constructor. E.g.:
esp_event_loop_args_t loop_args = {};
loop_args.queue_size = 5;
loop_args.task_name = "test";
loop_args.task_priority = 3;
loop_args.task_stack_size = 0x1000;
loop_args.task_core_id = 0;
ESPEventLoop loop2(std::make_shared<ESPEventAPICustom>(loop_args));

Note that the api object might be needed later if you set loop_args.task_name to nullptr, which means you manually need to call api->run(...) later to process the events.


Only use .vue files in karma and ignore the rest of files

Currently, if you want to do unit test and we use external files, such as html, css, pug, sass, etc ... we need to load the loaders by hand in ' webpack.test.conf.js`. Vue-loader already has all loaders needed, so if we only use the .vue files from our components, vue-loader will be in charge of processing everything (templates and styles included)
Thanks :smile:

So if I understand this correctly, this will only require .js and .vue files for coverage?


Release/v0.2.0

Contains the following features and updates;
Improve accessibility #8 by @adamsilverstein
Update chars.json key/value data #6 by @jeffpaul
Feature: character search/filter #5 by @adamsilverstein
Feature: enable overridable character map #3 by @adamsilverstein
Thank you both or your contributions! :)
There were a few minor conflicts @adamsilverstein between the search and the refactor of Chars / characters if you'd like to check I've not missed one that'd be cool. I had to re-implement your readme changes too. I'll merge this is in and release the new version to npm asap

Super, thanks! I'll take a look and test this out.

In https://github.com/Dayjo/react-character-map/pull/10/commits/5e61c32412ef5fb07346f78f2c62a7105c2b5693 I fixed an issue where the tab changig broke :) When I moved the construction method to componentDidMount, I didn't realize its structure included the active tab and that tab's characters. After this commit, the structure is recalculated whenever the tab changes (as expected).

@Dayjo I noticed when installing the 0.1.10 from npm, the dist folder was missing.
The issue is npm is ignoring the dist folder bevause its in .gitignore. To fix you can either remove dist from the .gitignore and commit the dist folder, or you can create an empty .npmignore file to override using the git ignore file - see https://docs.npmjs.com/misc/developers#keeping-files-out-of-your-package
One advantage of committing the dist folder is that users can check out the repo and use as is without building. One disadvantage is the continual merge configs the dist files create - because they tend to be single line files, any change will result in a conflict. For this reason I suggest adding an empty .npmignore file before publishing.

@adamsilverstein Thanks for the review. I have merged in your fix.
Just to clarify your suggestion on ignore files. I've committed the dist directory and added it to .gitattributes to attempt to prevent conflicts. The existing .npmignore file should be ok.


TypeScript development environment setup

@ncthbrt
I checked out the branch refactor-typescript, on which you did some refactoring.
I found you changed many things in nact-core, but the others that depend on nact-core haven't been changed properly according to it.
Now it is failing to compile.
I wonder whether you used tsc compiler on your works.
If needed, I want to share with you something about TypeScript development environment and how to run tsc compiler.
And what kind of IDE do you use on development?
I'm using VSCode with some TS extensions, such as TSLint, TS Prettier.
It's good to use some kind of linter and code prettier (formatter).
Thank you.

Hey. Sorry the branch is still in progress, as I was using it to experiment
to understand how Typescript might be used to type RouteThink. This
experimentation led to me being quite aggressive in refactoring the
codebase, so aware that it may have been breaking other parts of the code
base. I may have also made some mistakes due to a beginner's understanding
of how Typescript's structural typing works; for instance, so of the type
constraints may be incorrect/poorly expressed.
I'm also using VSCode for development.
On Mon, Jun 3, 2019 at 12:58 PM Hyeonsoo David Lee<EMAIL_ADDRESS>wrote:

@ncthbrt https://github.com/ncthbrt
I checked out the branch refactor-typescript, on which you did some
refactoring.
I found you changed many things in nact-core, but the others that depend
on nact-core haven't been changed properly according to it.
Now it is failing to compile.
I wonder whether you used tsc compiler on your works.
If needed, I want to share with you something about TypeScript development
environment and how to run tsc compiler.
And what kind of IDE do you use on development?
I'm using VSCode with some TS extensions, such as TSLint, TS Prettier.
It's good to use some kind of linter and code prettier (formatter).
Thank you.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ncthbrt/nact/issues/61?email_source=notifications&email_token=ACCJEWJX53BL5CS4O2UHB23PYT2NZA5CNFSM4HSG4VYKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4GXIDPUA,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACCJEWJ4Y6RX3BIKAGTJOXTPYT2NZANCNFSM4HSG4VYA
.

@civilizeddev I added typescript bindings to a beta release of nact. Hopefully they should fulfill your needs. Would love to get some feedback on them.


the memory usage of zero3 is larger than zero1

When I run the step1_supervised_finetuning script, I find that the memory usage of zero3 is larger than that of zero1, which seems unreasonable. Is there any other optimization here?

@blldd could you provide more details, like the training scripts, GPU numbers etc?

Close the issue since there is no followup. Please reopen it if necessary


Unable to generate C# types for GitHub's webhook schema

I'm attempting to generate C# types for GitHub's webhook schema. There is no official schema, but they have been generated here, and I am specifically attempting to use this file.
Using the following code:
var schema = await JsonSchema.FromUrlAsync("https://unpkg.com/@octokit/webhooks-schemas@4.2.1/schema.json");
var settings = new CSharpGeneratorSettings
{
    Namespace = "Octokit.Webhooks"
};
var generator = new CSharpGenerator(schema, settings);
var file = generator.GenerateFile()

I get this code, which is a subset of the types in the JSON schema definition.
The JSON schema definition is using draft 7, and I have seen #574, so I assume it might be something to do with that?

I think the main problem is that oneOf inheritance is not yet supported - only OpenAPI's allOf, which works in the "other" direction - i.e. with all of you specify the schemas which inherit from "this" schema, with allOf you define the base schema.

Thanks for the explanation. Should I follow #13 for updates?


Recursive @apply issue in watch mode (Angular)

What version of @tailwindcss/jit are you using?
v0.1.17
What version of Node.js are you using?
v12.18.2
What build tool (or framework if it abstracts the build tool) are you using?
<EMAIL_ADDRESS><EMAIL_ADDRESS>What browser are you using?
Chrome
What operating system are you using?
macOS
Reproduction repository
https://github.com/ivanchenhz/angularTailwindJit
Modify the following code like change the tw-text-gray-100 to tw-text-pink-60 won't take effect in watch mode ( re-start the dev sever works )
@layer components {
  .text-primary {
    @apply tw-text-gray-100;
  }

  .tw-dark .text-primary {
    @apply tw-text-gray-05;
  }

  .s-text-primary {
    @apply text-primary;
  }
}

Seems it not related to the recursive @layer, it's still not working even after change the code to following, close it with further research.
@layer components {
  .s-text-primary {
    @apply tw-text-blue-30;
  }

  .tw-dark .s-text-primary {
    @apply tw-text-gray-05;
  }
}


Refactor write process and allow index insert.

In this PR, we propose the following change:
1. temporarily remove region pre-split login
2. support index insert(when meet duplicate, an exception will be thrown)
The reason to make the first change is that the assumption of region pre-split has changed. We need to rethink how to do region pre-split.

/run-all-tests

/run-all-tests

/run-all-tests

/run-all-tests

/run-all-tests

/run-all-tests

/run-all-tests


Removed: Unnecessary font overridden of titles for category navbar(#488)

Related Issues

Closes #488
Short Description and Why It's Useful

Removing the font override will make it consistent.
Screenshots of Visual Changes before/after (If There Are Any)

Before

After

IMPORTANT NOTICE - Remember to update CHANGELOG.md with description of your change
Contribution and Currently Important Rules Acceptance

[x] I read and followed contribution rules

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

Thanks @dt2patel suggesting this improvement

We have removed the font override here and now the headings matches with the Figma.

Nothing to merge after a rebase

Nothing to merge after a rebase


Package URL in README is old

Hi @muenzpraeger ,
The package URL in README.md seems to be old. The latest package url is the following.
https://login.salesforce.com/packaging/installPackage.apexp?p0=04t0Y000000smT0
Would you please update the link?

This repo got deprecated over a year ago. Check out the README headline which points to the successor => https://github.com/muenzpraeger/salesforce-einstein-platform-apex


renderQueqe for the shader

Hello! How can I set renderQueqe to Transparent+10 for the shader?
Because when the camera inside the fog we can see the water (3000 renderQueqe), for example
Water is rendering "before" the fog, even if it far away
I changed "beforeTransparents" to true in VolumetricFogFeature.cs but nothing changed

There has been a setting added to the Render Feature that allows you to select if you want to render before or after the transparent objects pass. If you set the option to not draw before transparents, it might properly occlude your water shader, although it won't interact properly with the fog and will be completely covered. Only materials with a render queue 2500 or less actually draw to the camera depth texture, so the fog volume can't determine whether or not it is marching in front or behind transparent objects.
There are also a ton of people who have encountered the same problem, and I found a forum post that implemented a possible solution using the Render Objects feature. If you'd like to give it a try and see if it helps, here is the link: Transparent Shader Problem
This is part of a more general problem with transparents and the depth texture/buffer due to the way they work. There are solutions, such as rendering a second pass with only occluder objects drawing directly to the depth texture. I will look into implementing this in the future using a custom layer mask for transparent occluders instead of delegating the work to Render Objects.


[BUG] - <Encoding in Windows isn't considered well. We are still users>

Description
None of the encoding strategies work. Windows Users are not welcome.
I have tried all kinds of utf=8 encoding. Adding to .py files and setting to default environment.
Please consider many users are using win1x.
Traditional RAG works good, but GRAPHRAG isn't working right. No outputs are allowed to generate after indexing. Sad
FOR ALL PDFS i UPLOAD
Indexing [1/1]: semRegularized.pdf
=> Converting semRegularized.pdf to text
=> Converted semRegularized.pdf to text
=> [semRegularized.pdf] Processed 44 chunks
=> Finished indexing semRegularized.pdf
Error: 'gbk' codec can't encode character '\xa9' in position 127: illegal multibyte sequence
FOR ALL TEXTs I upload
D:\anaconda3\envs\kotaemon\lib\site-packages\numpy\core\fromnumeric.py:59: FutureWarning: 'DataFrame.swapaxes' is deprecated and will be removed in a future version. Please use 'DataFrame.transpose' instead.
return bound(*args, **kwds)
Traceback (most recent call last):
File "D:\anaconda3\envs\kotaemon\lib\runpy.py", line 196, in _run_module_as_main
return _run_code(code, main_globals, None,
File "D:\anaconda3\envs\kotaemon\lib\runpy.py", line 86, in run_code
exec(code, run_globals)
File "D:\anaconda3\envs\kotaemon\lib\site-packages\graphrag\index_main.py", line 104, in
index_cli(
File "D:\anaconda3\envs\kotaemon\lib\site-packages\graphrag\index\cli.py", line 178, in index_cli
progress_reporter.stop()
File "D:\anaconda3\envs\kotaemon\lib\site-packages\graphrag\index\progress\rich.py", line 119, in stop
self._live.stop()
File "D:\anaconda3\envs\kotaemon\lib\site-packages\rich\live.py", line 147, in stop
with self.console:
File "D:\anaconda3\envs\kotaemon\lib\site-packages\rich\console.py", line 864, in exit
self._exit_buffer()
File "D:\anaconda3\envs\kotaemon\lib\site-packages\rich\console.py", line 822, in _exit_buffer
self._check_buffer()
File "D:\anaconda3\envs\kotaemon\lib\site-packages\rich\console.py", line 2024, in _check_buffer
self._write_buffer()
File "D:\anaconda3\envs\kotaemon\lib\site-packages\rich\console.py", line 2060, in _write_buffer
legacy_windows_render(buffer, LegacyWindowsTerm(self.file))
File "D:\anaconda3\envs\kotaemon\lib\site-packages\rich_windows_renderer.py", line 19, in legacy_windows_render
term.write_text(text)
File "D:\anaconda3\envs\kotaemon\lib\site-packages\rich_win32_console.py", line 403, in write_text
self.write(text)
UnicodeEncodeError: 'gbk' codec can't encode character '\u280b' in position 0: illegal multibyte sequence
Reproduction steps
1. Go to '...'
2. Click on '....'
3. Scroll down to '....'
4. See error

Screenshots
![DESCRIPTION](LINK.png)

Logs
D:\anaconda3\envs\kotaemon\lib\site-packages\numpy\core\fromnumeric.py:59: FutureWarning: 'DataFrame.swapaxes' is deprecated and will be removed in a future version. Please use 'DataFrame.transpose' instead.
  return bound(*args, **kwds)
Traceback (most recent call last):
  File "D:\anaconda3\envs\kotaemon\lib\runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "D:\anaconda3\envs\kotaemon\lib\runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "D:\anaconda3\envs\kotaemon\lib\site-packages\graphrag\index\__main__.py", line 104, in <module>
    index_cli(
  File "D:\anaconda3\envs\kotaemon\lib\site-packages\graphrag\index\cli.py", line 178, in index_cli
    progress_reporter.stop()
  File "D:\anaconda3\envs\kotaemon\lib\site-packages\graphrag\index\progress\rich.py", line 119, in stop
    self._live.stop()
  File "D:\anaconda3\envs\kotaemon\lib\site-packages\rich\live.py", line 147, in stop
    with self.console:
  File "D:\anaconda3\envs\kotaemon\lib\site-packages\rich\console.py", line 864, in __exit__
    self._exit_buffer()
  File "D:\anaconda3\envs\kotaemon\lib\site-packages\rich\console.py", line 822, in _exit_buffer
    self._check_buffer()
  File "D:\anaconda3\envs\kotaemon\lib\site-packages\rich\console.py", line 2024, in _check_buffer
    self._write_buffer()
  File "D:\anaconda3\envs\kotaemon\lib\site-packages\rich\console.py", line 2060, in _write_buffer
    legacy_windows_render(buffer, LegacyWindowsTerm(self.file))
  File "D:\anaconda3\envs\kotaemon\lib\site-packages\rich\_windows_renderer.py", line 19, in legacy_windows_render
    term.write_text(text)
  File "D:\anaconda3\envs\kotaemon\lib\site-packages\rich\_win32_console.py", line 403, in write_text
    self.write(text)
UnicodeEncodeError: 'gbk' codec can't encode character '\u280b' in position 0: illegal multibyte sequence
use_quick_index_mode False
reader_mode default
Using reader <kotaemon.loaders.pdf_loader.PDFThumbnailReader object at 0x000001E2A1E84130>
Page numbers: 22
Got 22 page thumbnails
Adding documents to doc store
indexing step took 6.375478029251099

Browsers
Chrome
OS
Windows
Additional information
No response

I have the same problem

#373 Is the same bug, check out the comments in this Bug, There is a issue with the This is actually a issue from the llama_index SimpleDirectoryReader Class


Force adding LANG environment for reliable output

Summary
whois command has inconsistent parameters between distros.
This resulted an inconsistent output on .co.jp and .jp.
To resolve this issue, LANG parameter on env was explicitly added to make sure that Japanese NICs always return responses in Japanese.
It is also good to note that LANG parameter is exclusively used for Japanese NICs at the current stage. 😞
Reference
Unix Official Guideline:
https://www.unix.com/man-page/redhat/1/whois/
ENVIRONMENT
LANG
When querying whois.nic.ad.jp and whois.jprs.jp English text is requested unless the LANG or LC_MESSAGES environment variables specify a Japanese locale.

On Debian:
https://manpages.debian.org/testing/whois/whois.1.en.html
ENVIRONMENT
LANG
When querying whois.nic.ad.jp and whois.jprs.jp English text is requested unless the LANG or LC_MESSAGES environment variables specify a Japanese locale.

On FreeBSD: https://www.freebsd.org/cgi/man.cgi?query=whois
There wasn't LANG variable taken into account. It turns out that Japanese output was always printed as a result.

This resolves the question I commented in https://github.com/DannyCork/python-whois/pull/124


arch: arm: stm32f2: remove core zephyr header inclusions from soc.h

The stm32f2 version of soc.h misses the changes done in commit
aee97be ("arch: arm: soc: remove core zephyr header inclusions
from soc.h").
Signed-off-by: Istvan Bisz<EMAIL_ADDRESS>

Codecov Report

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

@@           Coverage Diff           @@
##           master    #8774   +/-   ##
=======================================
  Coverage   52.36%   52.36%
=======================================
  Files         195      195
  Lines       24713    24713
  Branches     5136     5136
=======================================
  Hits        12940    12940
  Misses       9699     9699
  Partials     2074     2074

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 db1075e...db1d240. Read the comment docs.


feat: add mixin of commands on ctx

Summary

#14  is built into this pr (please check that out first)
changed the setMyCommands function to be able to merge SetMyCommandsParams from two or more different sources, based on the fact that toSingleScopeArgs group them by language. Merge function its basically a reducer that concat lang.commands arrays of the same language and returns an array of setMyCommandsParams. It does not break scoping since it's performed after the toSingleScopeArgs function. If an empty array is passed, it does not throw, if only  one Commands instance is passed it treats it ok.

the name of the result variable might be better to change it to acc_Array or something in those lines

add info jsdoc info about lang's iso codes (an enum would be ideal 👍 )

PD: I closed the other PR since it was a little messy on the branch managment end, this one is ok.
PD2: I can refactor this into a MixedCommands class that extends the Commands Class if that suits more the style.
Another way would be to add this as a method to the Commands Class, but I don't know good of an idea would be to mix them before toSingleScopeArgs in a method like commands.mergeWith(otherCommands) because it would effectively duplicate commands registration if someone use it on let say commands A and B, and register both into bot.use,  registering all at once on A (B was merged into A), and then re-register B commands. Even in that case they would simply overwrite themselves I think so, no?
eg:

snippet from the gif (localization in the admin command... 🥴)
// localized start command //
userCommands.command("end", "end", async (c) => {
  await c.setMyCommands(userCommands)
}).localize("es", "fin", "finaliza")
  .addToScope(
    { type: "all_private_chats" },
    async (ctx) => {
      ctx.reply(`finalizado`)
      await ctx.setMyCommands(userCommands)
    },
  );
adminCommands.command("admin", "i am the admin", async (c) => {
  await c.setMyCommands(userCommands, adminCommands);
}).localize("es", "start", "el admin soy yo");

If the merge commands function is not to clear on what it's doing, just let me know, I can refactor it and can add it as a commands method

Removed pollution from commit history

Test added for merging

Is anything blocking this to be merge?

GitHub won't let me merge for some reason and I haven't had the time to do a local merge.
Could you try do rebase this branch on the main branch of the official repo and do a push --force-with-lease, please?

Now, should be good. Merging main into this branch resulted in duplicate commits


Challenge 1 extension

Challenge 1 migrated to an extension following #234
Closes #235

Lgtm! Added some small changes you need to do

Thanks!! Changes done!


Accessibility: Add a link to skip the twitter blocks

The twitter content on the page forces a keyboard user to tab through every single link inside before making it to the bottom and moving on with the content of the page. Adding a skip link before this block allows users to continue with the content.

Hey, im a student in an open source class. I'm looking to contribute to open source projects. Is it okay if I work on this?

Please do!

Can i get some tips on where to start?

I want to resolve this issue. I will start working on this from later today

from which pages this issues are coming

There is one reference here: https://reactjs.org/community/support.html

I feel this should be pretty easy to do. Is it as simple as going to content/community/support.md
and adding a link before the twitter block that references an element right after the twitter block?

I would think so!
On Fri, Apr 13, 2018 at 10:02 PM nestor-c<EMAIL_ADDRESS>wrote:

I feel this should be pretty easy to do. Is it as simple as going to
content/community/support.md
and adding a link before the twitter block that references an element
right after the twitter block?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/reactjs/reactjs.org/issues/445#issuecomment-381301476,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABgtaV_l9R5Aqj_ihFoXqPpPyMFzhZWiks5toXTZgaJpZM4RHwIT
.

--
Jen Luker
<EMAIL_ADDRESS>Twitter: @knitcodemonkey<EMAIL_ADDRESS>

@knitcodemonkey Pull request submitted.

@knitcodemonkey, I was looking through the old issues here and found this one.
Since the current docs doesn't have the Twitter section anymore, can you close the issue?
@nestor-c, the same applies for your PR.

Sounds good!


Fix: Incorrect use of partial in TweedieDistribution._rowwise_gradient_hessian

This PR addresses issue #888 by fixing the incorrect use of partial in the TweedieDistribution._rowwise_gradient_hessian function.
Checklist

[x] Added a CHANGELOG.rst entry

Hm, it might be a bit more involved than that. Is it possible that inv_gaussian_log_rowwise_gradient_hessian is incorrect? Shouldn't it be
@@ -276,12 +276,12 @@ def inv_gaussian_log_rowwise_gradient_hessian(
         inv_mu = 1 / mu[i]
         inv_mu2 = inv_mu ** 2

-        gradient_rows_out[i] = 2 * weights[i] * (inv_mu - y[i] * inv_mu2)
-        hessian_rows_out[i] = 2 * weights[i] * (inv_mu - 2 * y[i] * inv_mu2)
+        gradient_rows_out[i] = weights[i] * (y[i] * inv_mu2 - inv_mu)
+        hessian_rows_out[i] = weights[i] * (2 * y[i] * inv_mu2 - inv_mu)

instead?


Add cursor pointer property to images in today specials section

Issue #40
I Added a cursor pointer property to images in today's specials section

@Prajwal0225 have some conflict in index.html please resolve it .


Added sample configuration for reference

Added a sample docker-compose.yml file configuration for reference and easy understanding.

[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.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Thank you for opening this pull request.
This is a standard message notifying you that we’ve reviewed your pull request and have decided not to merge it. We would welcome future pull requests from you.
Thank you and happy coding. ✈


Refactor: Remove duplicates from ASTNode

Having duplicates in a union type does not affect the type's functionality, so we can safely remove them.

Hmm, looks like the PR template was not applied by GitHub here?


Meta-annotate @Story with @ExtendWith(StoryExtension.class)

If @Story is always declared at the class level and cannot be used without the StoryExtension, it is then a best practice to meta-annotate @Story with @ExtendWith(StoryExtension.class) to simplify configuration for your users.

Cool, that makes sense. Thanks for pointing it out. Updated "@Story"

👍