Using TokenCredential trait in KeyVaultClient

resolve #5
Before updating examples, please check if you like the way I've updated the vault client.
If so I'll updated examples, but should it use the DeafaultCrendentials or the ClientSecretCredential maybe.

Can you merge in latest from master to turn CI green?

Can you merge in latest from master to turn CI green?

I think CI also depends on test, but I need to know what is the plan. If DeafaultCrendentials should be used or the ClientSecretCredential and preserve the dependency on the ENV as it is right now.

Looks good. I have some questions, but nothing that should prevent merging.

@rylev if you have issues with the above and have suggestion how to fix/improve them, could you please create a ticket ?
If I have the time I may take a look at it. I wanted to change no logic only perform as much modification as required to allow managed identity and other type of authentication to work smoothly with the keyvault. You might also consider adding the removal of unwrap from the token secret usages and let refresh_token return it. Thus the number of unwrap greatly decreases in the crate.

@gzp-crey sorry, those comments weren't really meant for you since they aren't directly about your changes. I just wanted to mark them so I wouldn't figure them. I can make issues for them.


Build queue for --watch

Use a queue for build requests in --watch mode so that duplicate builds can be removed, and optionally use a solution to build multiple projects to avoid dependencies being built simultaneously and causing file locking races.
Example config (albeit the simplest approach):
name: tye-build-fix
solution: tye-build-fix.sln
services:
- name: frontend
  project: frontend/frontend.csproj
- name: backend
  project: backend/backend.csproj
# both projects depend on SharedLibrary/SharedLibrary.csproj

note-1: I'm new to C# but experienced in other languages. It's very possible that I am not doing things in a canonical C# way.
note-2: to work around a bug in msbuild when using the -target commandline arg, I had to upgrade msbuild past version 16.10 (I'm using <IP_ADDRESS>601 specifically).
Resolves #580
Resolves #1187

 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.:x: james-allan-lloyd sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

@james-allan-lloyd Sorry for it taking so long to be able to get to this.  I hope you don't take offense to the changes I've made; feel free to suggest others.  I'd appreciate if you could take the latest code for a spin and see if it continues to work in the way you expect.

@ravipal This is another one to review, when you have a chance.  (Comment added since the PR's been open a while and the original notification would've been long ago.)

@james-allan-lloyd Sorry for it taking so long to be able to get to this. I hope you don't take offense to the changes I've made; feel free to suggest others. I'd appreciate if you could take the latest code for a spin and see if it continues to work in the way you expect.

No worries, I'll get back to you soon on it.

Hi guys, I only just now got to checking out the changes and they work for me (4 or so projects depending on a 2 shared libraries).
One small caveat; when using the solution build, it doesn't like nested targets (I'm using dotnet msbuild version 17.0.0+c9eb9dd64). This appears to be the default now when doing dotnet sln add... I worked around it by commenting out the  GlobalSection(NestedProjects) section in the solution file, but maybe there's a way to get the nested name from the file?
To be clear, my day to day work project looks a bit like this:

api-services

serviceA

serviceA.csproj

serviceB

serviceB.csproj

tye.yaml
MySolution.sln

I need to invoke msbuild like this to build a targeted project:
dotnet msbuild -target:api-services\serviceA


Incorrect Naming in Documentation

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report
[ ] Performance issue
[ ] Feature request
[x] Documentation issue or request
[ ] Other... Please describe:

Library version

Library version: 0.2.3

Current behavior

Documentation in the msal-angular library suggests that the public APIs are named in a snake_case styling, such as login_popup()
Expected behavior

The documentation should indicate the correct name of the APIs, which are camelCase, such as loginPopup()
Minimal reproduction of the problem with instructions

Observe the documentation here
Observe the actual method names, as can be found here

@finalgriever Thank you for pointing out the discrepancy. We have updated the Readme with the method names.


Update dependencies and frame-metadata feature flags

After a cargo update we will need to change the feature flags of frame-metadata, since they removed the old ones.
see https://github.com/paritytech/frame-metadata/pull/62
Thanks @OverOrion for the ping in https://github.com/integritee-network/worker/issues/1388

I believe we can use current and unstable as feature flags, because we already support v15.


feat: enable v2_meta flag

Just like remix-run/indie-stack#203, remix-run/blues-stack/pull/163 & remix-run/grunge-stack/pull/126
CC/ @dev-xo

Thanks for all these PRs, @MichaelDeBoey! I've written a message for you in the first PR


Rotate does not allow user to specify origin

The rotate transform for SVG takes two optional arguments that describe the point about which the element is to be rotated. Because the library does not expose these two arguments, the default of (0, 0) is always used. If I want to rotate an element about its center, I have to provide a transform value to compensate the rotation.
IMHO the ideal interface would look something like this:
rotate : Angle -> Maybe (Float, Float) -> Property

I ran into the same problem. It applies to all transforms. There is something in SVG like "transform-origin" with which you can set the origin of the transformation. See: https://developer.mozilla.org/nl/docs/Web/CSS/transform-origin. However, this attribute is not available in elm-style-animation or in elm-lang/svg.
Would be very helpful if this can be fixed.

And the rubber duck effect (https://en.wikipedia.org/wiki/Rubber_duck_debugging) kicked-in immediately after writing my comment. I found a solution:
rect (Animation.render style ++ [ Html.Attributes.style [ ( "transform-origin", "50% 50%" ) ] ] ++ [ x (ts (px + 10)), y (ts (py + 10)), width "100", height "100", rx "5" ]) []
So, I added the "transform-origin" attribute on my svg rect that I am animating and it works perfectly!

I'll take a look at this later today.  It's interesting to see the SVG rotate and the CSS rotate differ a bit regarding syntax around origin.
Fyi, you can also set transform-origin in your animation with Animation.exactly, you just can't animate it.

@berry good workaround! I was literally adding a second transformation each time to compensate for the rotation about (0,0); your solution is MUCH cleaner.

transformOrigin is now directly supported in 3.5.1!


Tabs with empty children throws a javascript exception

When I set the children to a Tabs object to either None or [], I get a javascript exception:
"undefined is not an object (evaluating 'S[j].props')"
That happens when I write something line:
dmc.Tabs(     id='some-id',     children=[] )
or
dmc.Tabs(     id='some-id',     children=None )
For others running into this problem, my workaround when I shouldn't have any tabs is to create a Tab() and assign it a class_name where the style is 'display: none'.
So in my python code:
dmc.Tabs(     id='some-id',     children=dmc.Tab(label='', class_name='tab-hidden'))  # Must have at least 1 tab or throws error
And in my css file:
.tab-hidden {     display: none; }

Fixed in the latest master.


fix: now the community is spelled right

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

GitHub release
v1.3.9

Your semantic-release bot :package::rocket:


Support for sample

Is your feature request related to a problem? Please describe.
I want to get a sample of data to visualize using matplotlib, however, because I have too many records I want to get a good sample to visualize the data distribution.
Describe the solution you'd like
I want a function sample(..) that can sample a dataframe and return a dataframe with much smaller dataframe.
Describe alternatives you've considered

I can convert from Daft to Pandas or other dataframe, but that would not be scalable if the data is too large.
I can use where, but that would require me to implement a solution that may not be correct.

Spark: https://spark.apache.org/docs/3.1.2/api/python/reference/api/pyspark.sql.DataFrame.sample.html
Pandas: https://pandas.pydata.org/docs/reference/api/pandas.DataFrame.sample.html

Hi @ammarar,
Thanks for making an issue for this! Would you be okay the Spark API for this? sample(withReplacement=None, fraction=None, seed=None). If so, it should be fairly straight forward and performant since we can sample each partition individually. Let me know if that works!

@samster25 Yeah the Spark API looks good to me. Thanks!


Unpin requirements

These should be set to minimum values instead of exact values in requirements.txt to avoid conflicts with other libraries:
courts-db==0.9.7
lxml==4.6.2
reporters-db==2.0.5
six==1.15.0

Oh, also I assume six isn't required anymore.

Oh, also I assume six isn't required anymore.

lxml would make me a bit nervous, but yeah, agree about the rest, though I'd wan to double check that we didn't use six anywhere.

lxml would make me a bit nervous, but yeah, agree about the rest, though I'd wan to double check that we didn't use six anywhere.

Oh, I wonder how requirements.txt relates to pyproject.toml's
[tool.poetry.dependencies]
python = "^3.7"
reporters-db = "^2.0.5"
courts-db = "^0.9.7"
lxml = "^4.6.2"

[tool.poetry.dev-dependencies]
...

I don't understand poetry (hah), but I'm guessing that list should be used to specify the dependencies for the pypi package, and requirements.txt should disappear?

lxml would make me a bit nervous

You should definitely pin it to a precise version in whatever projects you're including eyecite from. But eyecite itself shouldn't pin it beyond setting a minimum version -- if libraries all pinned lxml, you'd never be able to install two libraries that depended on lxml at the same time.

Oh, I wonder how requirements.txt relates to pyproject.toml's
[tool.poetry.dependencies]
python = "^3.7"
reporters-db = "^2.0.5"
courts-db = "^0.9.7"
lxml = "^4.6.2"

[tool.poetry.dev-dependencies]
...

I don't understand poetry (hah), but I'm guessing that list should be used to specify the dependencies for the pypi package, and requirements.txt should disappear?

lxml would make me a bit nervous

You should definitely pin it to a precise version in whatever projects you're including eyecite from. But eyecite itself shouldn't pin it beyond setting a minimum version -- if libraries all pinned lxml, you'd never be able to install two libraries that depended on lxml at the same time.

OK, yep, that all sounds right. I thought we were in the CL repo at first.
Um, yes, we should decide on either requirements.txt (pypi) or Poetry. Doing both is definitely wrong. We've been trying poetry over in CL and it's been pretty nice. I'd say we should go for it here too. I think that just means tweaking our auto-deploy to use it and maybe tweaking our docs to explain how devs should use it?

OK, yep, that all sounds right. I thought we were in the CL repo at first.
Um, yes, we should decide on either requirements.txt (pypi) or Poetry. Doing both is definitely wrong. We've been trying poetry over in CL and it's been pretty nice. I'd say we should go for it here too. I think that just means tweaking our auto-deploy to use it and maybe tweaking our docs to explain how devs should use it?

Sounds good to me! I haven't used poetry but have heard good things about it, so I'm happy with whatever you think makes sense.

Sounds good to me! I haven't used poetry but have heard good things about it, so I'm happy with whatever you think makes sense.

Do you think you'd want to tackle switching this over to Poetry? I just looked at the pypi.yml file for deployments and it's a bit tricky. I suspect it'd be better for @flooie to add this to his list (though it's pretty long right now).

Do you think you'd want to tackle switching this over to Poetry? I just looked at the pypi.yml file for deployments and it's a bit tricky. I suspect it'd be better for @flooie to add this to his list (though it's pretty long right now).


Fixes use of Duration methods (Issue #25)

Fixes issues that may arise when using app bundles and some Duration methods: https://issuetracker.google.com/issues/205866514


Import Error

How to use this library in NextJS project?
I got error below when trying to use it:
import { machineId } from 'machini';

export default function Home() {
  useEffect(() => {

    const getMachini = async () => {
      const machini = await machineId();
      console.log("machini", machini);
    };

    getMachini();
  }, []);

Import trace for requested module:
./app/page.tsx
 GET / 500 in 2143ms
 ⨯ ./node_modules/machini/dist/index.js:1:58
Module not found: Can't resolve 'child_process'

Hello, Thank you for using machini.
This module is used on the server, not the client. It requires a backend environment where the NodeJS API is available.
Therefore, it is difficult to get the device ID of the client user; you will need to use another method, such as UUID, to achieve this.
Thank you.


Cle/5082 hl7 elr accept

This PR finishes out the acceptance criteria for #5082, ensuring that there is feedback for non ORU_R01 messages.
Test Steps:

Run the unit tests

Changes

parsed out the message type and checked to verify it is ORU_R01

Checklist
Testing

[x] Tested locally?
[x] Ran ./prime test or ./gradlew testSmoke against local Docker ReportStream container?
[x] Added tests?

Pull reviewers stats
Stats for the last 30 days:

User
Total reviews
Time to review
Total comments

jeremy-page
22
1h 43m
0

carlosfelix2
14
16h 36m
69

MauriceReeves-usds
14
7h 40m
2

kevinhaube
13
1h 49m
26

clediggins-usds
9
3h 33m
15

oslynn
7
24m
2

TomNUSDS
6
6m
13

brick-green
6
1h 32m
0

ahay-agile6
5
1h 28m
2

jorg3lopez
5
19h 14m
0

bgantick
4
1h 49m
1

jbiskie
4
40m
0

doug-s-nava
3
2h 6m
28

TurkeyFried
2
23m
0

mreifman
2
17m
1

sethdarragile6
2
1h 50m
3

JosiahSiegel
1
3m
0

RickHawesUSDS
1
6m
2

jimduff-usds
1
12h 26m
0

snesm
1
39m
0

Tests worked fine and the code makes sense. Is there a way to manually test this?


Added some packages

https://github.com/spiritix/lada-cache does not enough stars as per the contribution guidelines. Send me the other two repos in separate PRs. I can accept those.


Rendering and export are broken when a save file contains a "selection" element with non-zero size

See demo video (sorry for the FPS, had to convert to GIF):

After bisecting to 5256096d76be4685d3a6f06e2be4926d0b0a9def I found out that the culprit is this context.translate( call in this renderElement function. It applies the translation to the canvas when the caller's loop over elements encounters a "selection" element, but does not revert the translation back after drawing the selection, so all the elements in the element array preceding the non-zero-size "selection" element remain on their positions, and the following ones get translated.
export function renderElement(
  element: ExcalidrawElement,
  rc: RoughCanvas,
  context: CanvasRenderingContext2D,
  renderOptimizations: boolean,
  sceneState: SceneState,
) {
  const generator = rc.generator;
  switch (element.type) {
    case "selection": {
      context.translate(
        element.x + sceneState.scrollX,
        element.y + sceneState.scrollY,
      );
      const fillStyle = context.fillStyle;
      context.fillStyle = "rgba(0, 0, 255, 0.10)";
      context.fillRect(0, 0, element.width, element.height);
      context.fillStyle = fillStyle;
      break;
    }

Then I found such a "selection" element in the save file I was loading which had one "selection" element with "height" of 1 (not zero):
  {
      ...
      "type": "selection",
      "x": 219,
      "y": 642,
      "width": 0,
      "height": 1,
      ...
    },

Looks like that code triggered because this other piece of code did not filter out such a "selection" element when the file was restored via restore in restore.ts:
export function isInvisiblySmallElement(element: ExcalidrawElement): boolean {
  if (element.type === "arrow" || element.type === "line") {
    return element.points.length < 2;
  }
  return element.width === 0 && element.height === 0;
}

A separate question is how such a non-zero-size "selection" element got into the save file (and then to the localStorage), and what is the purpose of this translation when drawing a selection.
"Broken" save file (contrived to highlight the issue, don't mind repeated "seed" and almost-repeated "id"s):
{
  "type": "excalidraw",
  "version": 1,
  "source": "https://excalidraw.com",
  "elements": [
    {
      "id": "6S5FSXXsnaIrNVIUavJb0",
      "type": "rectangle",
      "x": -600,
      "y": -600,
      "width": 200,
      "height": 600,
      "strokeColor": "#e60000",
      "backgroundColor": "#e60000",
      "fillStyle": "hachure",
      "strokeWidth": 1,
      "roughness": 1,
      "opacity": 100,
      "seed": 549830127,
      "points": []
    },
    {
      "id": "zfthBG-S3gUqjbJ26R8If",
      "type": "selection",
      "x": 200,
      "y": 500,
      "width": 0,
      "height": 1,
      "strokeColor": "#495057aa",
      "backgroundColor": "#087f5b",
      "fillStyle": "solid",
      "strokeWidth": 2,
      "roughness": 1,
      "opacity": 100,
      "seed": 595091438,
      "points": []
    },
    {
      "id": "6S5FSXXsnaIrNVIUavJb1",
      "type": "rectangle",
      "x": -500,
      "y": -500,
      "width": 300,
      "height": 400,
      "strokeColor": "#e67700",
      "backgroundColor": "#e67700",
      "fillStyle": "hachure",
      "strokeWidth": 1,
      "roughness": 1,
      "opacity": 100,
      "seed": 549830127,
      "points": []
    },
    {
      "id": "6S5FSXXsnaIrNVIUavJb2",
      "type": "rectangle",
      "x": -100,
      "y": -200,
      "width": 400,
      "height": 200,
      "strokeColor": "#e67700",
      "backgroundColor": "#e67700",
      "fillStyle": "hachure",
      "strokeWidth": 1,
      "roughness": 1,
      "opacity": 100,
      "seed": 549830127,
      "points": []
    },
    {
      "id": "6S5FSXXsnaIrNVIUavJb3",
      "type": "rectangle",
      "x": -50,
      "y": -150,
      "width": 400,
      "height": 200,
      "strokeColor": "#e67700",
      "backgroundColor": "#e67700",
      "fillStyle": "hachure",
      "strokeWidth": 1,
      "roughness": 1,
      "opacity": 100,
      "seed": 549830127,
      "points": []
    }
  ],
  "appState": {
    "viewBackgroundColor": "#ffffff"
  }
}

CC @vjeux

cc @dwelle

Thanks so much for the detailed investigation!!
We had a bunch of issues with selection not being terminated correctly, for example if you alt tab while selecting. Then you get an empty one in the state that would never go away.
We're no longer storing selection as part of the elements list so that no longer an issue but we should filter out the selection object from saved files that were affected.

@sompylasar thanks for the report. While this has now been fixed, it's not yet released since we've moved to a different release strategy (not directly from master). Will let you know once this lands.


Updating target version to  one with brotli encoding supported

The following pipelines have been queued for testing: You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow


Modal accessibility

Describe the bug
When a modal opens on the page, the focus does not shift to the modal and the user has to tab through the elements behind the modal before being able to interact with the modal. This could make for a confusing experience for folks using screen readers.
Also, the modal fails the accessibility test here with the message that not all content is contained within landmarks (https://dequeuniversity.com/rules/axe/4.6/region?application=axeAPI)
To Reproduce
Steps to reproduce the behavior:

Go to the block edit view
Use the tab button to navigate the page
Hit "Enter" when you have the "Delete screen" button selected
Use the tab button to continue navigating the page
You will have to tab through all elements behind the modal before getting to the modal

Expected behavior
When a user opens the modal and continues navigating the page via manual tabbing the elements on the modal should be the next ones selected.
Screenshots
You can see that "Edit visibility condition" is selected when I hit "tab" after opening the modal

Desktop (please complete the following information):

OS: iOS
Browser Firefox
Version 109.0.1

Additional context
Add any other context about the problem here.

Need to investigate if there are applicant-face modals.  If it's only admin, we probably don't need to do it, but if there are applicant-facing ones, we should do this soon.

Need to investigate if there are applicant-facing modals. If it's only admin, we probably don't need to do it, but if there are applicant-facing ones, we should do this soon.
Edit: May be some with the eligibility and address correction features. @gwendolyngoetz @seanmalbert @dkatzz can you take a look and see if this is an issue with modals for those features?

There aren't any modals for general eligibility, but I believe there are modals for the address correction feature. @seanmalbert and @gwendolyngoetz to confirm.

This is also an issue with the "Create account or sign in" modal, which is applicant-facing.


Implement user activity tracking season

Duck season

Flaky Test
Failure Summary
Logs

[chromium] should display error message for invalid login correctly.
The expected HTTP status code 401 was not received, instead a function was returned.
Logs ↗

[chromium] should login with credentials correctly correctly.
The login request to the server failed with a 404 Not Found error.
Logs ↗

[chromium] should return list of users correctly.
The API call to get the list of users failed with a 404 Not Found error.
Logs ↗

Test_ConflictError
failure_message: Failedfailure_text: === RUN   Test_ConflictError    api_test.go:60: Conflict error occurred--- FAIL: Test_ConflictError (0.00s)
Logs ↗

View Full Report ↗ ⋅ Docs


How to use ssh access the URL which localtunnel return ?

I have succeeded to install localtunnel and get url. But how to use ssh access the URL which localtunnel return ?   I try to use this command: ssh<EMAIL_ADDRESS>I get the error: Permission denied (publickey). How do fix this error.

You cannot. This is an http/s only tunnel for exposing websites to clients for demos, webhooks for testing, etc. This is not a one-size-fits-all tunnel.


Add support for static args in shard_map

For example marked using None in the in_spec.

We fixed this in #22049!


Automatically upgrading managed clusters to latest stable

Background: https://airtable.com/tblD3oZPLJgGhCmch/viwchkdQkRUw6hZ3z/reciYK8RX6LlaLeLz?blocks=hide
PM: @kannanlakshmi

This was done soon after the 19.1 release.


Documented conky integration

Solves #219 explaining a basic configuration of conky.
Any feedback is appreciated as this is my first PR here. I tried my best to follow guidelines but mistakes happen, specially the first time.

Looks good to me. No obvious red flags or issues TBH.

Oh, commit messages should take the form of Add conky configuration example, not Added conky configuration example. Noticed that one too late. Not a big deal though, but it's the usual convention.


I hope the custom style of the List component can customize the item's padding

This function solves the problem (这个功能解决的问题)
Currently, the List component cannot customize item padding, and can only be resolved through deep or CSS important methods
目前List组件无法自定义item的padding，只能通过deep或者css important方式解决
Expected API (期望的 API)
const listThemeOverrides: ListThemeOverrides = {
    itemPadding: '0',
};

This issue does not have any recent activity. If you are still experiencing similar problems, open a new error, including a minimal copy of the problem


Issue-189 (chore) : Implemented MakeFile

Signed-off-by: Abhijay Jain<EMAIL_ADDRESS>Description
added support for Makefile and did related docs changes as well in the contributing sections of the readme.
Fixes #189
Type of change
code and docs
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)
[x] This change requires a documentation update

How Has This Been Tested?
tested locally on a windows machine
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] I have checked my code and corrected any misspellings
[x] I have tagged the reviewers in a comment below incase my pull request is ready for a review
[x] I have signed the commit message to agree to Developer Certificate of Origin (DCO) (to certify that you wrote or otherwise have the right to submit your contribution to the project.) by adding "--signoff" to my git commit command.

I studied these updates. LGTM. I didn't test the Makefile however. We can do that once it merges.
Maybe while we're waiting for comments from additional reviewers, you can add show-args to the PHONY target.

sure, will do that
Thanks for the suggestion :)

I studied these updates. LGTM. I didn't test the Makefile however. We can do that once it merges.
Maybe while we're waiting for comments from additional reviewers, you can add show-args to the PHONY target.

I added the default target to PHONY which is pointing to show-args, so In that case, I don't think we need to add show-args to PHONY, not sure about it.
@johnwalicki can you please guide me on this ?

Ok - that works for me.

@Abhijay007 Have you resolved all of these issues by fixing or determining that it won't be fixed?  Or are you still working on this?

@Abhijay007 Have you resolved all of these issues by fixing or determining that it won't be fixed? Or are you still working on this?

@joewxboy I did all the changes requested by @Rene-Ch1, I re-request a review from her on this PR so that she will point out if something is leftover :)

@Rene-Ch1 Anything left to work on or is this ready to merge?


Pylint and RTD update patch, and other fixes

Okay, fixed the issue and submitted additional PRs for those repos that got approved.  Any that you haven't touched should also have the appropriate commit for this.  Thanks for catching it!

Got it, I'll check tonight an either change it or re-request the review if the diff is in fact acting up!

Yup, it was still there, just removed it!


Kobold (PC2) have wrong default languages

Kobold player ancestry should have Sakvroth, not Draconic, as initial language (along with Common).
They should also have Draconic, Empyrean and Fey as additional languages.

Sorry - already fixed in 6.2.1


tools/ci/testlist: Switch half build to clang

Summary
Impact
Improve the toolchain coverage
Testing
Pass CI

Good idea @xiaoxiang781216 !!! Now it has better coverage!

depend on https://github.com/apache/incubator-nuttx/pull/7399


ConnectionString Hive ODBC Driver

Hello,
I put in place a automation test for Big Data with like database Hadoop Hive.
My connection string is :
DRIVER=Hortonworks Hive ODBC Driver; Host=xx.xx.xx.xxxx; Port=xxxx; HiveServerType=2; AuthMech=2; ThriftTransport=Binary;Schema=hive_database; UID=; PWD=;
When i run the test, i get the result :

System.AggregateException : One or more errors occurred.
----> NBi.Core.ConnectionException : ERROR [IM002] [Microsoft][ODBC Driver Manager] Data source name not found and no default driver specified
Someone has already encountered this kind of problem?
Best regards,
Steve.

I doubt you have this issue with this specific connection string. Your connection string is dedicated to the HortonWorks driver but your exception is raised by the Microsoft driver. Not possible.

My résults tests :
NBi.NUnit.Runtime.TestSuite.TA.xxxxxxx:
System.AggregateException : One or more errors occurred.
----> NBi.Core.ConnectionException : ERROR [IM002] [Microsoft][ODBC Driver Manager] Data source name not found and no default driver specified
The connection string used was '          DRIVER=Hortonworks Hive ODBC Driver; Host=xx.xx.xx.xxxx; Port=xxxx; HiveServerType=2; AuthMech=3;ThriftTransport=Binary;          Schema=******; UID=*************; PWD=;'
at System.Threading.Tasks.Task.FastWaitAll(Task[] tasks)
at System.Threading.Tasks.Parallel.Invoke(ParallelOptions parallelOptions, Action[] actions)
at NBi.NUnit.ResultSetComparison.BaseResultSetComparisonConstraint.ProcessParallel(IResultSetService actual)
at NBi.NUnit.ResultSetComparison.BaseResultSetComparisonConstraint.Process(IResultSetService actual)
at NBi.NUnit.ResultSetComparison.BaseResultSetComparisonConstraint.Matches(Object actual)
at NBi.NUnit.Runtime.TestSuite.AssertTestCase(Object systemUnderTest, Constraint constraint, String stackTrace)
at NBi.NUnit.Runtime.TestSuite.ExecuteTestCases(TestXml test)
--ConnectionException
at NBi.Core.Query.QueryOdbcEngine.Execute(Single& elapsedSec)
at NBi.Core.Query.QueryOdbcEngine.Execute()
at NBi.Core.ResultSet.Resolver.QueryResultSetResolver.Load(IDbCommand command)
at NBi.Core.ResultSet.Resolver.QueryResultSetResolver.Execute()
at NBi.Core.ResultSet.ResultSetService.Execute()
at NBi.NUnit.ResultSetComparison.BaseResultSetComparisonConstraint.b__21_1()
at System.Threading.Tasks.Task.Execute()


Finish Implementing InBuiltPredicates

net.minestom.vanilla.datapack.loot.function.InBuiltPredicates
Search for "TODO" in this file. Implement all of them.

Uncomment the wip loot table tests and fix any breaking ones once finished.


Clean up some of the snapshot-related infra from the platform account

There are bits hanging around that can't do anything because snapshots run in the catalogue account.

I've cleaned up some IAM roles, a Lambda, CloudWatch logs, and leftover SQS queues and SNS topics.


New function - create collection if not existing

What do you think about adding a new method that would create the collection like drafted here?
Note this is just a draft top open the discussion.

I wonder what might be the potential use-case? Are you working on a project that requires this method?

In my opinion it is better to make a separate package dependent on it.

If you want I can make it.


unable to purchase (or get the library working)

I created a DonationHandler which contains this code https://gist.github.com/mrgreaper/9747b72118f1e800953ee00b8af0007e
The xml for it just has one button.
Line 19: Log.d("IAP","READY") is never triggered leading me to believe the library is not activating correctly
Line 50: Log.d("IAP","loaded")  does trigger
Line 52:Log.d("iap","CLICK") when the button is pressed
been trying to get IAP working all day (and most of the night) so I may be missing something obvious?
also does .price work with your library to get the price of an IAP ? (from https://developer.android.com/google/play/billing/billing_library_overview#kotlin)

Ok turns out the code does not work on an emulator...it does however work great on a physical device. Thank you for the library!  (wonder how many times yesterday I had the code working but the emulator said no? O.o)


[no-unused-state] Add ability to handle arrow function class method

For https://github.com/yannickcr/eslint-plugin-react/issues/1363, so that state property usage by object destructuring assignment in a class method defined with an arrow function gets recorded. Examples:
class ArrowFunctionClassMethodTest extends React.Component {
  constructor() {
    this.state = { foo: 0 };
  }

  doSomething = () => {
    const { state: { foo } } = this;

    return foo;
  }

  render() {
    return <SomeComponent />;
  }
}

should not be warned.
Let me know if there is anything I can do to improve this PR. Also, just to confirm, this rule doesn't care about whether the class method is used or not, right?

@ljharb
This list will only be added to if the property name gets assigned to a different variable name.
For instance,
class App extends React.Component {
        state = { foo: 0 };

        doSomething = () => {
          const { state: { foo: bar }} = this;

          return bar;
        }

        render() {
          return <SomeComponent />;
        }
      }

will add 'bar' to the list. I have added one more test regarding this.
I also refactored how the code deals with classInfo.aliases according to your feedback. We still need to clear the set after each method definition though because each alias added is local to the class method. Let me know if this looks better.

@ljharb
I have restored the implementation back to using classInfo.aliases = new Set() which is a part of the original code. However, we still need to do classInfo.aliases = new Set() for every class method defintion because the aliases in classInfo.aliases are local to each class method. Let me know if that's ok.

@ljharb
Thanks for the review! After this is merged, I can take a look at documenting no-unused-state as I saw there is an issue open for it.

One test for the no-typos rule is failing in the builds. Not sure why. Don't think it has anything to do with node versions.

@ljharb
I think this test is failing on master. Any clue? Maybe this test should have errors in the first place?

Thanks @jackyho112
This is because of the new ESLint release. They changed some internal process and a case that we didn't handle before will now work.
I can update the case to be valid, but it would only work with the latest ESLint. With older cases it will still fail. Even more: We had to write some work-around to make sure ESLint did not crash but I can't remove that code unless we update the peerDependency for ESLint.
I also commented here: https://github.com/yannickcr/eslint-plugin-react/pull/1375#issuecomment-329899585
But I think I can start a new PR and we can start/continue a discussion there.

Created a PR to move that test: https://github.com/yannickcr/eslint-plugin-react/pull/1437

@ljharb, @EvHaus, @yannickcr, @lencioni
Decided to adhere to the original implementation because it is the safest approach, it is easier to review and if there is a new way of defining a class method, the rule won't break immediately.
Now, this PR just merely adds the support for a class method defined by an arrow function based on how the original code intends to and does nothing else. I hope that is ok and should at least make this PR more mergable.

@EvHaus, @ljharb
Thanks for the approvals! 😄


hello-world: explain UnsupportedOperationException in HINT.md file [Fix #350]

Fixes issue #350.

Reviewer Resources:
Track Policies

@FridaTveit great tweaks, thanks! LGTM! :shipit:


spring boot debugging cleanup and fix wfs missing param

@metacosm let me know what you think.

thank you for all the feedback @Ladicek and @metacosm
Updated per your feedback.
Let me know what you think.


Invalid binding prefix for model with single property

I'm trying to bind model with single property:
public class TimeZoneModel
{
	public double? TimeZone { get; set; }
}

public class MyController : Controller
{
		public async Task<IActionResult> Get(TimeZoneModel model)
		{
				//...
		}
}

Property TimeZone can't be bound because binding name is model.TimeZone. If I add second property to the model then model binding name is TimeZone and binding is ok.
public class TimeZoneModel
{
	public double? TimeZone { get; set; }
	public string SomeOtherProperty { get; set; }
}

@anfomin what is submitted and what does "can't be bound" mean exactly?
Note the model binding system will attempt to bind "model.TimeZone" and "model.SomeOtherProperty" then fall back to "TimeZone" and "SomeOtherProperty" if the first attempt does not succeed.

I submit value for TimeZone, for example:
GET /mycontroller/get?TimeZone=10

If there is only one model property then MVC tries to bind only model.TimeZone but does not fall back to TimeZone without prefix. If you need I can share sample.

Tested on Beta5, works as expected.
GET /home/get?TimeZone=10

public class HomeController : Controller
{
    public async Task<IActionResult> Get(TimeZoneModel model)
    {
        // model.TimeZone = "10.0"
    }
}

Maybe your URL is bad?
GET /mycontroller/get?TimeZone=10

Shouldn't it be?
GET /my/get?TimeZone=10

@anfomin If you could share your code that would be great.
@dougbu Could you please try to reproduce this issue?

@danroth27 I'm sorry, I can share sample only tomorrow. I'll post a link...

I have investigated that there is an error with our custom model binder. MVC bind this model correctly.


Go: rework mock generation

The use of gomocks in our codebase currently has the following issues:

Generating mocks is not hermetic - it uses whatever version of mockgen is in the user's path.
There is no CI check to ensure that the generated mocks are consistent with the source.
There is no way to automatically catch generated mock files that remain after their source has been removed.
There is no documentation on what setup is required before tools/gomocks will actually work.

Proposed fix:

Use bazel to build the mockgen binary for the version of github.com/golang/mock we're using, and then use that in tools/gomocks.
Move from scion.sh mocks to make mocks. That way it can depend on setting up the required env (including compiling the mockgen binary).
CI needs to regenerate mocks and error out if they differ from the ones in git.
CI needs to error out if there are mock files with no source.

This can be done by either:

have a whitelist of mock_X dirs that should exist, and throw an error if any other mock_X dirs exist.
nuke all mock_X dirs before regenerating mocks.

Building mockgen with bazel fails because our version of rules_go pulls in a broken version of org_golang_x_tools (it predates https://github.com/golang/tools/commit/202502a5a9245830b5ec7b877e26b67760da8e67). The minimum version of rules_go to fix this seems to be 0.19.0.

Building mockgen with bazel fails with the updated rules_go because... it pulls in a newer version of org_golang_x_tools which has the same problem as the old version, just in a different file: https://github.com/golang/tools/commit/eec4c98bf5a6a9765c9de84cf61e447130fe8e54#diff-1910375731a87c0dea421568450908cc is the fix.


Fix issues with HTTPoison 1.4

Fixes #53

@werbitzky I'm merging this one as this seems to be a pretty big issue. I'll let you push it to hex.pm if it looks good to you.


Allow set notation for one to many relationship

Hey mate,
Is it possible to enable set notation for one to many relationships? It doesn't appear to be working. Here are my relations and the relate fill output:
Relate::oneToMany(
    'entry:{pens,markers,accessories}.refill',
    'refills.products'
);

Relate::oneToMany(
    'reviews.product',
    'entry:{pens,markers,accessories,refills}.reviews'
);

Output:
❯ php please relate:fill
Updating relationships. This may take a while if you have a lot of entries.

Updating relationship: (1-*) [refills.products {pens,markers,accessories}.refill]...
Updating relationship: (1-*) [refills.products {pens,markers,accessories}.refill]... done!
Updating relationship: (*-1) [reviews.product {pens,markers,accessories,refills}.reviews]...
Updating relationship: (*-1) [reviews.product {pens,markers,accessories,refills}.reviews]... done!

This is now available as a new feature in v2.1.0 (S4) 🙂
Set notation has been added to all relationship types (oneToOne, oneToMany, manyToOne, manyToMany)

@JohnathonKoster Thanks, but it still doesn't seem to be working after the update. Relate list is listing one relationships with the set notation and not listing all the relationships.

@mikemartin  I'll get you a new version tagged here soon
🤦♂🤦♂I didn't merge the branch.

@mikemartin  Just tagged 2.1.1 with the actual changes. Sorry about that 🐸

Awesome. Thank you!


clarify when prefetches don't prefetch

the malformed cases were missing, and also the cases should be symmetrical between it mode / cap mode which wasn't clear before
fixes https://github.com/riscv/riscv-cheri/issues/312

Alexandra/Evangelos has also pointed out that SHADD do not have the same malformed/reserved conditions as CADD

Alexandra/Evangelos has also pointed out that SHADD do not have the same malformed/reserved conditions as CADD

that should have another issue filed please

@tariqkurd-repo : I expanded the information to make it clearer instead of using cross-references.


Typo "Postgres extensions"→"PostgreSQL extensions"

https://docs.microsoft.com/en-us/azure/postgresql/concepts-extensions
https://github.com/MicrosoftDocs/azure-docs/blob/main/articles/postgresql/concepts-extensions.md
#PingMSFTDocs

@hyoshioka0128 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Looks good; thanks @hyoshioka0128.


geocoding is undefined

when i use library in Angular 8, this happens:
core.js:9110 ERROR TypeError: Cannot read property 'Geocoding' of undefined
apparently there would be a correct method to work in angular, what I did was invoke the library from the scripts defined in angular.json.

The same problem.
`import { Component, OnInit } from '@angular/core';
import { ActivatedRoute } from '@angular/router';
import { SharedService } from '../shared/shared.service';
import { IUserLocation } from '../shared/shared.interface';
import * as L from 'leaflet';
import * as esri from 'esri-leaflet';
import * as esriGeo from 'esri-leaflet-geocoder';
@Component({
selector: 'app-app-where-to-see',
templateUrl: './app-where-to-see.component.html',
styleUrls: ['./app-where-to-see.component.css']
})
export class AppWhereToSeeComponent implements OnInit {
userLocation: IUserLocation;
constructor(
private sharedService: SharedService,
public activatedRoute: ActivatedRoute
) { }
ngOnInit() {
const lMap = L.map('map', {
maxZoom: 18,
minZoom: 0
}).setView([0, 0], 15);
esri.basemapLayer('Topographic').addTo(lMap);

this.sharedService.currentLocation().subscribe((userLocation: IUserLocation) => {
  this.userLocation = userLocation;
  lMap.setView([this.userLocation.lat, this.userLocation.lng], 12);

  L.esri.Geocoding.reverseGeocode()
    .latlng([this.userLocation.lat, this.userLocation.lng])
    .run((error, result, response) => {
        console.log(result.address);
    });
});

}
}`
core.js: 7187 ERROR TypeError: Cannot read property 'Geocoding' of undefined

Also having this problem:
import * as L from 'leaflet'
import * as esri from 'esri-leaflet'
import * as esri_geo from 'esri-leaflet-geocoder'

var map = new L.Map('map', {maxZoom:21})

var searchControl = L.esri_geo.Geocoding.geosearch().addTo(map)

This throws the following error Cannot read property 'Geocoding' of undefined

// this means everything in the plugin is only going to be available via its own namespace
import * as esriGeo from 'esri-leaflet-geocoder';

// not L.esri.Geocoding.reverseGeocode()
esriGeo.Geocoding.reverseGeocode()

// not L.esri_geo.Geocoding.geosearch().addTo(map)
esri_geo.Geocoding.geosearch().addTo(map)

sigh Duh...
Thanks for reminding me how imports work

You are correct, I was forgetting how imports work, however I still get errors.
When using esri_geo.Geocoding.geosearch().addTo(map)
I now instead get Cannot read property 'geosearch' of undefined.

reading this and https://github.com/Esri/esri-leaflet-geocoder/issues/100, it appears that including .Geocoding is unnecessary too.

That works!
Thanks!

@C0rporeus @KroniK907  Is the original issue resolved here?

@gavinr Yes. Would be nice to add some documentation for how to use it when setting it up as a node module in a JS project.

@Gavinr For me this issue is resolved, although including how to use this when implemented in a JS framework as a node module would be nice, since how to do that is not clear from the existing documentation.

We always welcome pull requests, particularly for useful documentation updates, @C0rporeus @KroniK907, @studio-salamander.  Perhaps we can start out with a README update?

@jgravois thanks brother is works on vuejs, .Geocoding is unnecessary

@aacassandra  .... hi brother can u please help me with this in vuejs ...

I'm marking this as "FAQ" and will close it for general reference, but please note my comment just above.
https://github.com/Esri/esri-leaflet-geocoder/issues/219#issuecomment-567561335
We really do welcome README suggestions and improvements (pull requests are welcome!) from the community since you all more frequently see this library used in different environments and frameworks. Thanks.

esri_geo.geosearch().addTo(map

This is the Correct answer

esri_geo.geosearch().addTo(map

This is the Correct answer

esri-leaflet-geocoder getting error


HOLD https://claim-tax-benefits.azurewebsites.net/login/securityQuestion2

On this page, for the error message, please replace "select" with "choose"

Closing this as the security questions have been nerfed.


US-58 | Implement zero downtime re-indexing

Implemented zero downtime re-indexing using two actual ES indexes which are used in turns, and an index alias that points to the index that contains the latest complete data. When an import finishes, the alias is swapped to point to the newly finished index.
Although this seemed like a pretty simple thing to do, the implementation actually got stupidly complex. On a positive note, the whole thing is handled inside the base importer, so importers using it should not need to care about that.

Codecov Report

Merging #44 (3800451) into develop (d28a1a6) will decrease coverage by 2.59%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop      #44      +/-   ##
===========================================
- Coverage    58.87%   56.27%   -2.60%
===========================================
  Files           19       19
  Lines          620      661      +41
  Branches        83       90       +7
===========================================
+ Hits           365      372       +7
- Misses         252      286      +34
  Partials         3        3

Impacted Files
Coverage Δ

ingest/importers/base.py
26.37% <0.00%> (-9.05%)
:arrow_down:

ingest/importers/location.py
57.69% <0.00%> (+0.31%)
:arrow_up:

ingest/importers/event.py
60.34% <0.00%> (+1.02%)
:arrow_up:

Continue to review full report at Codecov.

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


How to make a visual with a "cutout" that uses a shape's GetAlphaMask() result?

Hello, I'm relatively new to the visual layer so please excuse any newbie-ness.  I'm trying to make a large solid color Visual that would block the current view and then have a shape in the center of it that is "cut out" of it.  The shape comes from a XAML Path.GetAlphaMask() call.  Then the idea would be to animate that cutout shape in a zoom in fashion to reveal the full screen behind it.
It seems like this would need to be done with some combination of effects, a backdrop brush, etc. but I'm having trouble making headway on this.  I know the CompositionProToolkit might support this, but I'm trying to sort out how to do it without any complex libraries attached.
Any help is appreciated!

Fundamentally you would achieve this using a CompositeEffect, but in a different way than we've demoed before. If you look at the effect description for a CompositeEffect, you'll notice that there is a Mode property. In your case, you'll want to set this to CanvasComposite.DestinationOut. What this will do is use the mask to remove non transparent pixels from the destination (in your case it would be a ColorSourceEffect). From there you should be able to animate the visual as you please!
Here's a reference for the effect description you would use:
var effectDescription = new CompositeEffect
{
    Mode = Microsoft.Graphics.Canvas.CanvasComposite.DestinationOut,
    Sources =
    {
        new ColorSourceEffect { Color = Colors.Blue },
        new CompositionEffectSourceParameter("mask")
    }
};

var effectFactory = compositor.CreateEffectFactory(effectDescription);
var effectBrush = effectFactory.CreateBrush();
effectBrush.SetSourceParameter("mask", alphaMask);

Hi @robmikh, thanks that gets me most of the way there.  I made a SpriteVisual and assigned the effectBrush (from your logic) to its Brush property.  I set the size of the SpriteVisual to the size of the logo, center it in the window, and see the cutout fine.
The problem now is that I need to "bleed" the edges of the sprite using my background color (Blue in your code above) all the way to the edges of the window.
Any suggestions on how I get the window area surrounding this sprite visual to be filled with Blue and also animate such that it's not interfering with the cutout scale zoom?

For that you'll want to chain yet another effect!
var alphaMask = MyControl.GetAlphaMask();
var myControlSize = new Vector2((float)MyControl.ActualWidth, (float)MyControl.ActualHeight);
var pageSize = new Vector2((float)ActualWidth, (float)ActualHeight);

var effectDescription = new CompositeEffect
{
    Mode = Microsoft.Graphics.Canvas.CanvasComposite.DestinationOut,
    Sources =
    {
        new ColorSourceEffect { Color = Colors.Blue },
        new Transform2DEffect
        {
            Source = new CompositionEffectSourceParameter("mask"),
            // You can think of this matrix as setting the CenterPoint of mask to
            // be in the center of the mask (the same size as the control it came
            // from), setting the Offset to be the middle of the page, and applying
            // a scale of 0.5f.
            TransformMatrix = Matrix3x2.CreateTranslation(myControlSize / -2.0f) *
                              Matrix3x2.CreateTranslation(pageSize / 2.0f) *
                              Matrix3x2.CreateScale(0.5f) *
                              Matrix3x2.CreateTranslation(myControlSize / 2.0f)
        }
    }
};

var effectFactory = compositor.CreateEffectFactory(effectDescription);
var effectBrush = effectFactory.CreateBrush();
effectBrush.SetSourceParameter("mask", alphaMask);

var visual = compositor.CreateSpriteVisual();

visual.Size = pageSize;
visual.Brush = effectBrush;

ElementCompositionPreview.SetElementChildVisual(this, visual);

// Hide the control
var controlVisual = ElementCompositionPreview.GetElementVisual(MyControl);
controlVisual.Opacity = 0;

The effect in question is the Transform2D effect, which allows you to apply a 3x2 matrix to an effect source. In this case we'll apply it to the alpha mask. Set the transform in such a way where the mask would be where you want it, and then make sure the visual is set to take up the entire screen (or even larger if you're going to be shrinking and growing the visual, that way you'd never see the edges). You should be able to animate the visual any way you want after that.
The trickiest part of this would be getting the matrix just right. In this example it's kind of static, but remember that you can declare certain effect properties as modifiable when creating the CompositionEffectFactory. That way you can apply animations and expressions to the TransformMatrix property of the effect brush. Setting up the matrix math in an expression that pulls values from a property set, and then changing the property set when the page changes size, is one use for this.
Let me know if you have any further questions!

Thanks for all the detail @robmikh, that works well.


Documentation about meta column order.

Closes https://github.com/dask/dask/issues/2789

+1 from me.  Merging shortly if there are no further comments

Thanks @TomAugspurger !


[EN] Feedback for: /web/tools/chrome-devtools/?hl=en

This STUPID BOX keeps popping up every time I type something and it is driving me nuts, I want to disable it PERMANENTLY. If you can show me how to do that I would greatly appreciate it.

Hi,
I'm not sure what pop up box you're referring, and without more details, I am going to close this. Please consider changing your username before posting about this or other errors again, as it's use of inappropriate language and slurs is unnecessary.


Add facemp.edu.br

@yanndere First stage of verification passed, official domain is facemp.edu.br as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.


getting 404 resource not found error

using azure openai as openai_base_url?
i am trying to do it and getting this error :
response: undefined, failed to send message to openai: Error: OpenAI error 404: {"error":{"code":"404","message": "Resource not found"}}, backtrace: Error: OpenAI error 404: {"error":{"code":"404","message": "Resource not found"}}
at Promise.then.promptTokens (/actions-runner/_work/_actions/fluxninja/openai-pr-reviewer/latest/dist/index.js:3310:29)
at processTicksAndRejections (node:internal/process/task_queues:96:5)
and this is how my workflow yaml looks like :
name: Code Review

permissions:
contents: read
pull-requests: write

on:
pull_request:
pull_request_review_comment:
types: [ created ]

concurrency:
group:
${{ github.repository }}-${{ github.event.number || github.head_ref ||
github.sha }}-${{ github.workflow }}-${{ github.event_name ==
'pull_request_review_comment' && 'pr_comment' || 'pr' }}
cancel-in-progress: ${{ github.event_name != 'pull_request_review_comment' }}

jobs:
review:
runs-on: self-hosted
environment: new-env
steps:
- uses: fluxninja/openai-pr-reviewer@latest
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
OPENAI_API_KEY: ${{ secrets.OPENAI_API_KEY }}
with:
debug: false
openai_base_url: https://"your endpoint deployment name".openai.azure.com/
openai_light_model: gpt-35-turbo
openai_heavy_model: gpt-35-turbo

Model name looks incorrect. Try gpt-3.5-turbo

Tried same error

@harjotgill i got the issue now , basically we are using the chatgpt node package but to use azure/openai as a openai base URL we will need to add support for azure/openai node package as they both have different api's altogether

Azure OpenAi support can be added with this I guess : https://github.com/transitive-bullshit/chatgpt-api/issues/352#issuecomment-1506801907

If you folks fix it, please contribute back! Cheers!

Yes it works , i will push it with generalising the feature with paramas
https://github.com/darshilshahquantive/ai-pr-reviewer

@darshilshalquantive Did you change anything typescript wise, or just index.js to get it working? This thing has been a huge pain in my butt these last couple weeks, so any help would be appreciated.

index.ts fix will work if you want to use azure 3.5 turbo with 16 k context you have to edit limit.ts else it will use this much tokrn limit by default https://github.com/darshilshahquantive/ai-pr-reviewer/blob/10783d111ba827426dc67c38e809283ad18877ca/src/limits.ts#L21


Elements  jumping when resizing from corner

Bug Description
This is a follow-up to #917. Potentially fixed by #2255(?)
Issue with setting min size of shape then dragging corner to resize.
Expected Behaviour
No jump should occur
Steps to Reproduce

Add any shape
Enter a value of 1 in the width input field then unlock aspect ratio
Click drag bottom control of element to resize. You'll see a jump and the height of the element is now 2873

Screenshots

Additional Context

Plugin Version:
Operating System:
Browser:

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

Implementation Brief

Closing for now, can't reproduce


[FIX,VM] Fix get_outputs on the vm with a single output

The VM uses an ADT for multiple outputs and an NDArray for a single output. The single output case was not being handled.
@tmoreau89

@areusch Can you review?

thanks @tkonolige , the PR is now merged!


Lease: refactor lease APIs to take client side response

hide grpc response from user by return client side response.
use listenableToCompletableFuture() instead of FutureConverter.toCompletableFuture() for convert listenable future.
minor refactoring on util and watch

lgtm


LED sensor Problem

Describe the Bug
Everytime I was trying to add an LED sensor to my robot and edit the LED shape, webots would crash and quit.
Steps to Reproduce

Go to 'Robot'
Click on 'children'
Add an 'LED'
Click on 'LED's children'
See error

Expected behavior
Hope I can add the LED sensor.
Screenshots

System

Operating System: Windows 10
Graphics Card: NVIDIA GeForce GTX 1060  3GB

Additional context

I cannot reproduce this crash. Could you tell me:

which version of Webots are you using?
which world file are you using?
is the simulation running or paused?
is your robot running some specific controller?
If you are using your own world file, please upload it.

Since we can't reproduce the problem and have no feedback I will close this issue. Do not hesitate to give more precisions and we will re-open it.


When can support Weblogic

When can support Weblogic

We have no plans yet.

We currently use large-scale Weblogic, hope to see you as soon as possible to provide support.

@linzhiqiang0514
Weblogic is supported


Doc update needed

You no longer have to approve email addresses on mailboxes for new User's that sync from AAD.  The only time you have to approve the emails now is if the email address is changed in Dynamics 365 CE.  Can we get this documentation updated to reflect that?  This helps lessen our need for a global admin and helps Enterprise customer's who read this documentation.

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

ID: 404a72e5-a513-82d9-91ac-f2b3dccfacf4
Version Independent ID: 72717494-eebe-7a26-90fa-b36986c663de
Content: Connect Dynamics 365 for Customer Engagement apps (online) to Exchange Online
Content Source: ce/admin/connect-exchange-online.md
Service: crm-online
GitHub Login: @jimholtz
Microsoft Alias: jimholtz

@sdieken
Topic updated with: You only need to approve the email address for a user when their email address has changed in Dynamics 365 for Customer Engagement.
Thanks for bringing this to our attention, Shawn.


[DOR-80-81] shortcut to the summary if all answers completed

Seems to need at least one piece of vessel info (when optional) to be 'complete' -
I went through a journey selected
No for priority,
(Optional) vessel details blank
Got to CYA went back to priority and selected yes & rightfully onto which priority page but needed at least one piece of vessel info to get directly back to CYA page (didn't seem to matter which piece though)
Routing for mandatory page:
ie. Went through export NEW journey (optional vessel details left blank)
Clicked change to request type and set to C1601 - got straight back to confirmation page & vessel details bypassed
Upon clicking change for the vessel details it brings straight to landing page
^^ Same for C1602 & Route hold
IF you input one piece of info for vessel doing the same as the above will bring you back through the journey BUT it will crash on the vessel-info-required page (Sorry there is a problem with this service)
Some odd behaviour with the change links in some cases too (sometimes just bringing straight back to trader services landing page) - haven't quite gotten to the bottom of why on this one yet (sometimes they work fine)


Downgrade Go version to 1.21

This pull request downgrade the Go version to 1.21.

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

Project coverage is 92.54%. Comparing base (d96f709) to head (84cec7f).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #108   +/-   ##
=======================================
  Coverage   92.54%   92.54%
=======================================
  Files          26       26
  Lines         992      992
=======================================
  Hits          918      918
  Misses         53       53
  Partials       21       21

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


Image from file not loading fast enough on first time

I have image from file i want to use as background.
I am using relative layout,  and i strech the image  all over the screen (the first child) and later i add some labels and entries to the relative layout.
The first time I loads it (using the cacheImage) a black screen appears with the labels and the entries presented and after around 1 second the image appears.
The second time I loads the screen the image is presented immediately - I guess the image is already in the cache.
How do I load the image to the cache in the first time ?

The first time your image is retrieved from disk and added to memory cache.
The time you see is the time to load from disk or to decode your image, it is independant from the library.
If you want to preload the image into memory cache you can use:
ImageService.Instance.LoadUrl(urlToImage).Preload();

However I doubt it will really help your scenario.


fix(devimint): run_ui does not block

The intention was for devimint run-ui to spawn stuff and block waiting for Ctrl+C. This got lost somewhere during refactorings of devimint and just now discovered.
I think the clean_on_exit is more versatile if it does not wait itself, and require logic that it wraps to block if that's the intention.

Successfully created backport PR for releases/v0.4:

#5621


SQLite DB

Hi, is it possible to use SQLite3 database on this version? I tried creating a demo sqlite.db but it doesn't work. Even a simple "connection" test doesn't work or maybe I forgot something?
Trying to check SQLite version.
$dbh = new PDO('sqlite::memory:'); print_r("SQLite version " .  $dbh->query('select sqlite_version()')->fetch()[0]); $dbh = null;
Fatal error: Uncaught Error: Call to a member function querySingle() on null in /var/task/user/api/index.php:27 Stack trace: #0 {main} thrown in /var/task/user/api/index.php on line 27
Thanks!

Sure it is. I have made a little demo for you.
https://github.com/juicyfx/vercel-examples/commit/13717d7964c5e357312b37c1ff2ed5865acc9a6a

/api/index.php

<?php

$db = new SQLite3('/tmp/db.sqlite', SQLITE3_OPEN_CREATE | SQLITE3_OPEN_READWRITE);

$db->query('CREATE TABLE IF NOT EXISTS "visits" (
    "id" INTEGER PRIMARY KEY AUTOINCREMENT NOT NULL,
    "url" VARCHAR,
    "time" DATETIME
)');

$statement = $db->prepare('INSERT INTO "visits" ("url", "time") VALUES (:url, :time)');
$statement->bindValue(':url', ($_SERVER['HTTP_X_FORWARDED_PROTO'] ?? 'https') . '://' . $_SERVER['HTTP_HOST'] . $_SERVER['REQUEST_URI']);
$statement->bindValue(':time', date('Y-m-d H:i:s'));
$statement->execute();

$visits = $db->querySingle('SELECT COUNT(id) FROM "visits"');

echo("User visits: $visits");

$db->close();

now.json

{
  "functions": {
    "api/index.php": {
      "runtime"<EMAIL_ADDRESS>    }
  },
  "routes": [
    { "src": "/(.*)",  "dest": "/api/index.php" }
  ]
}

Example URL here https://php-sqlite.vercel.app/

@f3l1x Sorry for bringing up an old thread, but what is the purpose of supporting SQLite? On every cold start, the database will be deleted or reset to its initial value...
Maybe there's an interesting use case I'm not seeing?


Error with qemu-guest-agent 9.0.1

I've finally got the script to work with the qemu guest agent 9.0.1.pkg. However, when i try to start the qemu-guest-agent, I get an error:
critical: error opening channel '/dev/vtcon/org.qemu.quest_agent.0' :  No such file or directory
critical: failed to create guest agent channel
critical: failed to initialize guest agent channel
I do not have a /dev/vtcon directory

Same problem here. Any solution?

now need to update to 9.0.2

I have an error when I tried to use install.sh on TrueNAS-13.0-U6.2
I install manually package with : pkg install qemu-guest-agent
virtio_console seems to be correctly loaded.
But I have this error message too:
critical: error opening channel '/dev/vtcon/org.qemu.guest_agent.0'
/dev/vton does not exist.

I found an issue with manual installation and after change processor type to kvm64.


Update dependecies from roadie

Hey, I just made a Pull Request!
This PR contains updates to the latest version of Roadie plugins which are made to follow upstream package updates.
:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[ ] All your commits have a Signed-off-by line in the message. (more info)

Hey @Irma12 is it also possible to update the @backstage/cli that is being used in these packages too? It looks like that's the cause of quite a lot things being duplicated for different versions in the yarn.lock

Hey @Irma12 is it also possible to update the @backstage/cli that is being used in these packages too? It looks like that's the cause of quite a lot things being duplicated for different versions in the yarn.lock

@benjdlambert Definitely yes, let me update the PR :)

@benjdlambert Dependencies are updated and I don't see duplicate packages anymore, hope this is better :)

Hey @Irma12 -  so I noticed that there were some other deps out of date, which is our fault, so I spent some time updating those too so we don't bring in duplicates.
https://github.com/backstage/backstage/pull/6510
https://github.com/backstage/backstage/pull/6500
I think you can pull from master and do a yarn install and the yarn.lock i think should only have the duplicated msw dependency which is less of a concern - that's next weeks job 😅


[sema] Add support for asserting on the first error that occurs.

This enables one to easily drop down into the debugger in such cases. This is
enabled by the option swift-diagnostics-assert-on-error and can only happen in
an asserts build.
rdar://34222540

@swift-ci smoke test and merge


Add a 'Deselect All' button for languages in Page Access Statistics

Motivation

With a lot of languages it becomes tedious to select only one or a few languages, as you need to deselect every language manually.
Proposed Solution

A button to deselect all languages at once. So that afterwards you can select only the languages you want selected.
(See design)
Alternatives

Keep it as it is.
User Story

As a manager I want to quickly filter the languages displayed in the page access statistics so that I can be more efficient.
Additional Context

Design Requirements

Design proposal (v3):
https://www.figma.com/file/6U7R7Xj4wL7sbjxKRmOG9D/CMS-Project?node-id=1179-830
Related Issues

Discussed in https://github.com/digitalfabrik/integreat-cms/issues/3133

@nikolahoff I created this based on the discussion in #3133 and was wondering if we also need a 'Select All' button?

@juliankehne it's the same checkbox:) when unchecked - no languages are selected, when checked - all languages are selected.


remove zipping

Make sure to resolve the conflicts first and then re-request a review from me. Thanks

I think you are looking at a different pr?


opening new file within Harpoon, error

Hi all,
After following ThePrimeagen's tutorial on installing harpoon im left with the following error when opening a file within harpoon.
E5108: Error executing lua: ...nvim/site/pack/packer/start/harpoon/lua/harpoon/ma rk.lua:124: You can't add harpoon to the harpoon stack traceback:         [C]: in function 'error'         ...nvim/site/pack/packer/start/harpoon/lua/harpoon/mark.lua:124: in funct ion 'filter_filetype'         ...nvim/site/pack/packer/start/harpoon/lua/harpoon/mark.lua:207: in funct ion <...nvim/site/pack/packer/start/harpoon/lua/harpoon/mark.lua:206>
Thank you in advance.

all good i had it open with i was trying to add a file to it..  problem solved.


Publish gradle plugin

Hi, it would be nice if gradle plugin was available from https://plugins.gradle.org
See https://plugins.gradle.org/docs/submit

Closed by #1790 ?
https://plugins.gradle.org/plugin/com.apollographql.apollo

Not yet, sorry. You won't find 1.2.2 on the gradle portal yet. This was reverted in https://github.com/apollographql/apollo-android/pull/1813, I'll enable it again in 1.3.x.


[BUG] Publishing Page transformation creates blank images webpart with no link to the images

Problem Area
Publishing Page Transformation: Error during the use of page transformation from .Net
An error occurred in ReturnCrossSiteRelativePath function at System.IO.Path.CheckInvalidPathChars(String path, Boolean checkAdditional)
Expected or Desired Behavior
Create images webpart with link to the correct images
Observed Behavior
it creates the webpart but is blank
Steps to Reproduce
Using latest version 1.0.2012 (Dec 8 2020)
using custom layout mapping which seems to work for text quite well, but not for images

Moving this issue to the PnP Framework repo as that's where we now host the modernization code.

Moving this issue to the PnP Framework repo as that's where we now host the modernization code.


New pictures for galaxy intro 101

In  topics/introduction/images/101_11.png  the first dataset should be 1: Exons.

Thanks @cmonjeau!


BUG: _find_simplex_directed brute force

Fixes gh-21279

In the event that a point is judged to be outside of a simplex in _find_simplex_directed, for now it appears to be more robust to fall back to a brute force check on the matter.

[skip circle] [skip cirrus]
Things that could be better here:

Instead of adding an .npy file, search for a smaller reproducing set of generators
Do some "whiteboard analysis" to find a faster solution than falling back to brute force (though correct > fast for now, I suppose)
Could test directly in spatial via the public find_simplex(), though adding a test that exactly mimics the ticket reproducer is perhaps not so terrible.

(Not a spatial expert, at all).
To make sure I understand what's going on here, is this correct:

the failing case has the query point to be one of the data points exactly on the convex hull;
_find_simplex thinks it's outside not inside;
there is even so tolerance for deciding if a query point is inside or outside, but in this dataset, the tolerance is not enough even through we're querying the point which should have a zero distance in exact arthmetics;
on main, the algorithm bails out for points outside of the convex hull;
with this fix, the algorithm switches from some fancy algorithm to a brute-force search instead?

Then of course the questions are

what's the big-O cost: Suppose there are M points inside and N points outside, what's the computational complexity change here?
would it be very wrong to check that the query point is exactly one of the data points?

Let me put a WIP label on it and do a bit more analysis.


InvalidOperationException - Debug IServiceCollection.Configure method

I'm attempting to create strongly typed configuration classes.  If I keep them really simple they work, but as soon as I try to do any real work I get an error when I attempt to resolve an instance of IOptions<ConfigurationClassHere>.  It contains an InvalidOperationException error that I have no access to.  It doesn't even throw it, so my program doesn't understand the configuration failed until I attempt to do something later.
How do I debug this?  Why didn't the initialization of the configuration throw the exception?  Why is DI ignoring this glaring issue?  Why doesn't DI include all the error information (stack trace)?

@Fosol can you share some code that demonstrates this? I don't think we've seen this before.

I've uploaded a basic web project on GitHub with an example of the issue.
Just place a break point in the HomeController.Index method.  You'll see the issue with the constructor parameter 'configuration'.
GitHub - https://github.com/Fosol/Example.StronglyTypeConfiguration
I'm sure I'm doing something wrong, but I would have hoped I'd get some kind of exception information that would help me debug.

Looks like configuration doesn't like arrays.  When I attempt to access the configuration later I get a DI error on them.  Have a debugged this to the root?
Does this mean strongly typed configurations don't support arrays?  Is there a way to support them?

Looks like a work around is to use List instead of string[].
I think this means DI needs to be updated so that it is able to create an instance of arrays.

@haok this sounds like a ConfigurationBinder limitation. Can you confirm?

Probably, I'll look at this next week, most likely a bug

@HaoK Hey, I found that arrays are not supported by the binder at all. I already gave it a try to implement that (see PR), let me know what you think

3125ec94c571a9f8063050ebcaab7fcf26950996


Remove glide and transition to go mod

Attempt to update circleci config - but this is untested

@MartinLeedotOrg Thanks for updating the stale package management way!


Fix layout shift on "What is Lit" page

Context
Resolves issue: #372
By adding width and height attributes, resolves layout shift.
Added width and height attributes to the logo in:

navigation
what is lit page
home page

Screenshots
Lighthouse performance for currently deployed docs page:

This change brings Cumulative Layout Shift down to 0. Ran a lighthouse test locally, against node production environment.

/gcbrun


Error @ UI.php#L123

https://crash.pmmp.io/view/9153904
https://github.com/GabBiswajit/SkyblockGB/blob/3f1503ad9fb743d4b0cbf309c5f4507edf8583b2/src/Biswajit/Skyblock/menu/UI.php#L97-L123

Please use IDE (PHPStorm) or static analysis tool (PHPStan) to catch all errors before calling it a "working plugin".

Don't have pc using phone. Btw it's now fix


Ability to add user information

Feature Description
For example, V Rising is 'family sharing enabled'. However, upon launching the game, it detects game license not found. There are lots of reason why https://www.reddit.com/r/vrising/comments/usbr5j/comment/iabtzxm/. I hope there is a feature to flag it as not shareable, so people don't get caught off guard. When the system detects a change, it should reflect the latest pulled information (overriding the user manually inputed information).

Tell developers to disable family sharing on Steam properly.


Wrong week and day is shown

For ex: today is Thursday of first week, and the site shows, that today is wednessday of second week
This semester started on 5, September with first week on schedule
Media included:

@Oleks-Y looks like backend issue: https://schedule.kpi.ua/api/time/current always returns currentWeek: 2

Seems to be fixed now


Fix onSelect close delay

onSelect close delay should be driven by item ripple duration: the dropdown should start disappear just after the item ripple effect ends instead of using the animationDuration as a delay.
Currently if user use rippleDuration=0, the behavior is that when pressing a dropdown item, there is a 225ms delay for no reason and no user feedback at all which is confusing

I created a "getItemProps" function because I need to get the real item ripple duration to compute the delay (as it can be provided globally or per item using props extractor)

Thanks for PR! I've improved delay calculation in 0.10.0 release, now ripple and fade out animations should run in parallel.


Pink ribbon animation

Describe the project you want to add with tech stack

This project contains a pink ribbon animation.Can you assign this task to me>
Expected behavior

Screenshots (optional)

Additional context (optional )

@MohitGupta121 ,
@TusharKesarwani Can you assign this task to me?

@MohitGupta121 ,@TusharKesarwani Can you Please assign this task to me?As Gssoc contribution period is going to end..this issues can level me up ..Please assign this issue to me...Please.


Unable to resolve path to module to any local file

So im getting an unable to resolve paths on files like
import Divider, { Props as DividerProps } from './divider';
this shouldn't be happening, what do i need to change?
eslint config
{
    "extends": ["plugin:import/recommended"],
    "parserOptions": {
      "sourceType": "module",
      "ecmaVersion": "latest",
      "ecmaFeatures": {
        "jsx": true,
        "modules": true
      }
    },
    "rules": {
      "import/no-anonymous-default-export": 0,
      "import/order": [
        "error",
        {
          "groups": [
            "builtin",
            "external",
            "internal",
            "parent",
            "sibling",
            "index",
            "object"
          ],
          "newlines-between": "always"
        }
      ]
    }
  }

It seems you're using TypeScript (Props as DividerProps), then you should use https://github.com/import-js/eslint-import-resolver-typescript

same isssssu

https://antfu.me/posts/why-reproductions-are-required


Skip placeholder default op

Skips the PlaceholderWithDefault op (Issue #82)

@jlebensold Please review if this fixes the issue.

Thanks @aseemw, your fix make it work for me

If it's skipped, does it work as intended?


Drawing label inside of node

I wanted to try implementing a custom NodeShape to render the node labels inside a node rect. I came up with this as a starting point:
#[derive(Clone, Debug)]
struct DepNodeShape {
    pub pos: Pos2,

    pub selected: bool,
    pub dragged: bool,

    pub name: String,
}

impl<N: Clone> From<NodeProps<N>> for DepNodeShape {
    fn from(node_props: NodeProps<N>) -> Self {
        let pos = node_props.location;
        let selected = node_props.selected;
        let dragged = node_props.dragged;
        let name = node_props.label.to_string();

        Self {
            pos,
            selected,
            dragged,
            name,
        }
    }
}

impl<N: Clone, E: Clone, Ty: EdgeType, Ix: IndexType> DisplayNode<N, E, Ty, Ix> for DepNodeShape {
    fn is_inside(&self, pos: Pos2) -> bool {
        todo!()
    }

    fn closest_boundary_point(&self, dir: Vec2) -> Pos2 {
        todo!()
    }

    fn shapes(&mut self, ctx: &DrawContext) -> Vec<Shape> {
        let mut shapes = Vec::with_capacity(2);

        let style = if self.selected {
            ctx.ctx.style().visuals.widgets.active
        } else {
            ctx.ctx.style().visuals.widgets.inactive
        };

        let color = style.fg_stroke.color;

        // First, the text with the dep name
        let galley = ctx.ctx.fonts(|f| {
            f.layout_no_wrap(
                self.name.clone(),
                egui::FontId {
                    size: 5.0,
                    family: FontFamily::Monospace,
                },
                color,
            )
        });
        let name_pos = Pos2::new(
            ctx.meta.canvas_to_screen_pos(self.pos).x - (galley.size().x / 2.),
            ctx.meta.canvas_to_screen_pos(self.pos).y - (galley.size().y / 2.),
        );
        let name_shape = TextShape::new(name_pos, galley, color);
        let box_rect = name_shape.visual_bounding_rect().expand(3.0);
        shapes.push(name_shape.into());

        // Now, the surrounding shape
        let box_shape = RectShape::stroke(
            box_rect,
            Rounding {
                nw: 0.2,
                ne: 0.2,
                sw: 0.2,
                se: 0.2,
            },
            Stroke::new(1.0, color),
        );
        shapes.push(box_shape.into());

        shapes
    }

    fn update(&mut self, state: &NodeProps<N>) {
        self.pos = state.location;
        self.pos = state.location;
        self.selected = state.selected;
        self.dragged = state.dragged;
        self.name = state.label.to_string();
    }
}

Once I got to implement is_inside, closest_boundary_point, and update, I couldn't figure out how to continue.
The issue is that the size of the surrounding Rect is dynamic on the length of the label, and I can only compute the TextShape when ctx: &DrawContext is available, which isn't the case in those other methods.
Is there a way to do this?

You could store the text size in the DepNodeShape inside shapes method. As it has &mut self reference.
I will add an example with flex nodes to demonstrate it.

You could store the text size in the DepNodeShape inside shapes method. As it has &mut self reference.

But wouldn't that mean that if closest_boundary_point or is_inside are called before the first call to shapes their results would be incorrect?

But wouldn't that mean that if closest_boundary_point or is_inside are called before the first call to shapes their results would be incorrect?

Yep, but only for 1 frame, which is negligible.
Please check the new example.

Closing this one. If you have any more questions, welcome in other tickets 😃

I was able to build on your example and get the result I wanted, thank you so much for taking the time!
This is a really cool lib :)


RequiredAt documentation improvement

It would be a great help to clueless newbies such as myself if the ...At function documentation showed an example.
We are not as adept in reading and understanding the type signatures of functions, and it is easy to miss the difference between Required (String -> ...) and RequiredAt (List String -> ...) especially since they are displayed in different layouts.
Perhaps something like this:
import Json.Decode exposing (Decoder, string)
import Json.Decode.Pipeline exposing (decode, required, requiredAt)

type alias Joke =
    { resultType : String
    , joke : String
    }

jokeDecoder : Decoder Joke
jokeDecoder =
    decode Joke
        |> required "type" string
        |> requiredAt [ "value", "joke" ] string

result : Result String Joke
result =
  Json.Decode.decodeString
    jokeDecoder
    """
{
   "type":"success",
   "value":{
      "id":496,
      "joke":"Chuck Norris doesnt wear a watch, HE decides what time it is.",
      "categories":[
         "nerdy"
      ]
   }
}
	"""

-- Ok { resultType = "success", joke = "Chuck Norris doesnt wear a watch, HE decides what time it is." }

A couple of more complex examples, especially of things like decoding into a more structured type alias, would be also be extremely helpful.

Also, a link to the http://noredink.github.io/json-to-elm/ tool would be incredibly helpful, since it can be used to generate examples tailored to the reader's exact needs.


I encapsulated it too

I encapsulated it too using same tutorial by Sara Soueidan ; P

https://github.com/yandongCoder/circular-menu

Nice, looks good. Great you made it framework independent and even without jQuery.


send photos

I am using windows os. I can send messages but I can not send photos

I solved the problem. I could send it when I change the aspect ratio of the photo


#10794: Use correct REG_TLB for BH and uplift tlb configs to pull from _implementation in UMD rather than DEVICE_DATA

Ticket
https://github.com/tenstorrent/tt-metal/issues/10794
Problem description
TLBs were getting configured using values from DEVICE_DATA struct in UMD. UMD has duplicated information in this struct and in <arch_name>_implementation.hpp header.
DEVICE_DATA for BH had incorrect values and the idea is to eventually remove DEVICE_DATA in UMD (as per @joelsmithTT) so we shouldn't be using it in Metal.
I will try to make a change in UMD to remove DEVICE_DATA
What's changed
Remove dependence on DEVICE_DATA and use values from <arch_name>_implementation.hpp
Checklist

[ ] Post commit CI passes
[ ] Model regression CI testing passes

Looks reasonable to me.  I hope in the medium term we can fix UMD so that this sort of code is not required at the Metal runtime layer.  The TLB index to use for register access (or any access for that matter) could/should be a UMD implementation detail.  That code like this exists and that you are having to fix it is a symptom of a design problem with the UMD.


[Question] pytest not found?

Inside virtual env
pytest
zsh: command not found: pytest
expected
pytest
========================================================================================================================== test session starts ===========================================================================================================================
platform darwin -- Python 3.9.9, pytest-7.1.2, pluggy-1.0.0
rootdir: $HOME/Documents/sample-django-vscode
collected 0 items
which python
$HOME/.asdf/shims/python
pip list
pytest is present
python --version
3.9.9

For us to help us best, please follow our GitHub issue template. There is not a lot of actionable things here.
If it's a question, use the recommended channels, it should not be a GitHub issue. If it's a bug, please fill in the reproduction steps & context about your environment, Stackoverflow has a guide on how to ask a good question


procs: 0.9.20 -> 0.10.0

Motivation for this change
https://github.com/dalance/procs/releases/tag/v0.10.0
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 85626 1

1 package built:
- procs


[Documentation] E-mart site

Describe the Feature you want to end
I would like to add readme file for e-mart site

Expected behavior

Screenshots (optional)

Additional context (optional )

@MohitGupta121 Kindly, assign this to me.


fix: :bug: correct some words

Dear @kamranabdicse ,
Thanks for the text correction, because there is a conflict , I will make these changes on my own and close this pull request.


[Task]: Fix the failing performance tests on the golden app

Is there an existing issue for this?

[X] I have searched the existing issues

SubTasks
Performance tests are failing because of this issue #11385.
Explore possible two solutions

Use postgres instead of mysql
Create the data source via puppeteer automation instead of depending on the credentials in export file.

@Rishabh-Rathod , There is a new modal that is added after import to update the data source credentials.
Can you handle it as a part of app import.

As we now need to migrate MySQL dump to Postgres. I believe my earlier estimate was not accounting for this. Also having low expertise in this area might result in more time, hence changing the estimate here.
If a backend dev can pick up this migration part, maybe we could close it faster.
cc: @somangshu

Low on backend bandwidth right now @Rishabh-Rathod, Ill still talk to other EMs about this!


Reduce the validity period of the bootstrapped kubelet to match controller-manager

Enhancement
Overview
Right now kubelet certificate has a validity of 8760hrs:
https://github.com/poseidon/terraform-render-bootstrap/blob/c3b1f23b5daa77c8ff96a2294924aa138a1433a4/tls-k8s.tf#L198-L205
And this does not match with kube-controller-manager flag, which renews the cert in every 72hrs:
https://github.com/poseidon/terraform-render-bootstrap/blob/c3b1f23b5daa77c8ff96a2294924aa138a1433a4/resources/static-manifests/kube-controller-manager.yaml#L28
Motivation
The extra time seems unnecessary. Can someone who has access to the old certificate which is valid do nasty things on the cluster, masquerading as kubelet user?

Oops wrong repo.

Ah realised that I can't create an issue on that repo.

I think this is only for CA and doesn’t affect the actual certs, closing this my bad.

It's just dead / unused state actually. That certificate is not distributed or output. I'll be sure it's cleaned up before next release.
Thanks


Use stderr instead of logrus for CLI error messages

This PR will remove the use of logrus when printing errors and warning
to stderr while running the CLI, for a couple of reasons:

When there's an error the user might see something like this (from build):

INFO[0000] The command '"/bin/sh -c echoo A"' returned a non-zero code: 127

The "INFO[0000]" part isn't user friendly and is more designed for log files
than normal cmd output.

when the output goes to a non-tty, like during our testing it gets even
worse, it looks like:

time="2015-03-26T01:27:38-04:00" level=info msg="some cool error"

which means our tests aren't seeing what the end user really sees. So
if we have tests that check the output we're not testing the same output
as a user. Bad.

I thought about creating a logrus formatter that just printed the
message w/o the "INFO[0000]" part but decided against it for a couple
of reasons:
3a) its a lot of extra work (with no benefit) just to print to stderr (or cli.err)
3b) at some point we may want to have error messages that are
prepended with the exe name, for example:

   docker: ps requires at least one argument

if we use a logrus formatter then (w/o a lot of extra work) all output
lines would be prepended with "docker:" and for multi-line error messages
I don't think we want that.

we were inconsistent about when we would use logrus vs cli.err.
For example, we print to cli.err while looping over args for stop
(https://github.com/docker/docker/blob/master/api/client/stop.go#L30) but then use
logrus for other errors (like at the end of docker/docker.go).

There was only one spot (in docker/docker.go) that I sent to logrus
and I did that because it was a daemon-side error msg and I thought
that should go to whatever logfile we have defined, important for service
debugging.
Signed-off-by: Doug Davis<EMAIL_ADDRESS>

LGTM

@duglin can you rebase please?
LGTM

@crosbymichael rebased

LGTM


Using ng-redux without ES6 modules

I am consuming ng-redux in a project that does not use any modules. The challenge that I ran into with this approach is with the config block. If you have any suggestions for this, I would greatly appreciate it!
angular.module('counter', [ngRedux])
  .config(($ngReduxProvider) => {
    $ngReduxProvider.createStoreWith(rootReducer, [thunk], [DevTools.instrument()]);
  });

What would be the best way to create reducers here? Should they be $providers so they can be injected in the config block?
If ngRedux provided a factory instead of a provider it could be setup in a run block and everything (reducers, actionCreators, etc.) can be defined as factory.
I understand that using ES6 modules is great and brings lots of benefits. I don't have that setup for my angular project though.

Are you using any bundler? In the app I work on, reducers are plain JS and are included using require('/path/to/some/reducer.js'). No ES6 modules, but we are using Browserify/CommonJS.

No, I don't have any bundlers at the moment. I am relying solely on the minimal load order that angular provides (with config blocks and providers, runs and factories). I haven't had any need for a bundler since I am completely inside the angular world.

Much like what @jbalboni said.
Using a bundler is the best solution here in my opinion.
Most of redux related things make the assumption that you are using one.
For example middlewares like redux-thunk, redux-devtools and many other things are exposed as module.
If you have a very specific reason for not having a bundler, you could still pass a dummy function to createStoreWith and then use replaceReducer after the config phase to use the reducers you defined as factory, that should work (never tried, so maybe I'm forgetting something), but you ought to really not want a bundler before going down that road.
I won't be adding support for reducers as service (like what was done for middlewares) because in my opinion it facilitate two very bad things:

using dependency injection in your reducer (you should keep them pure).
accessing your reducers from anywhere, which you should never have to do.

Thank you for this nice explanation. It makes sense to just use a bundler like you suggested.

using dependency injection in your reducer (you should keep them pure).

I don't think using injection implies impure reducer (certainly not always). I wanted to inject a service with pure methods to be used by a reducer - using pure function from pure function would result in a pure function.


Adding technology demonstration guide example #21

This PR was originally created for the pymapdl repo but I'm moving it here as the tech demos have been moved to another repo.
Link to initial PR.
In this PR, I would like to add a new example, taken from Chapter 21 of the Mechanical APDL Technology Showcase Manual. This examples shows how to use PyMAPDL to import an existing FE model and to perform a nonlinear buckling and postbuckling analysis using nonlinear stabilization. The problem uses a stiffened cylinder subjected to uniform external pressure to show how to find the nonlinear buckling loads, achieve convergence at the post-buckling stage, and interpret the results.

Hi @pmaroneh I just saw that the code-style checks where failing due to a missing "ignored" word (i.e. filname). It should run the workflow now.

Hi @RobPasMue, thanks a lot for correcting this.
I'm wondering how the examples from the tech demos are now run to generate the html files? My model takes a very long time to run (few hours) so I'm not sure how it will be rendered.
Also I tried to rerun it locally and faced (again) some issues that I thought I had resolved, see this discussion for context. I will try to take into account suggestions from @germa89 today.

Hi again @RobPasMue , sorry, another question, related to the one above. This examples requires a .cdb file to be read. Where should I place this .cdb file? Thanks

Hi again again @RobPasMue , I can't figure out what the code style check is failing at now, would you mind helping me out? Many thanks.

Hi @pmaroneh! Sorry, I was focused on other topics =)

Regarding the .cdb file, I think you can add it in the same folder as your techdemo. That should be enough =)
Code style stage is failing because of the following error:

--- a/doc/source/technology_showcase_examples/techdemo-21/21-example-technology-showcase-buckling.py
+++ b/doc/source/technology_showcase_examples/techdemo-21/21-example-technology-showcase-buckling.py
@@ -2[70](https://github.com/pyansys/pymapdl-techdemos/actions/runs/3591741195/jobs/6046635959#step:5:71),7 +270,7 @@ mapdl.nsubst(100, 10000, 10)
 mapdl.rescontrol("define", "all", 1)
 mapdl.outres("all", "all")
 mapdl.ncnv(2)  # Do not terminate the program execution if the solution diverges
-mapdl.ignore_errors = True # in order for PyMAPDL to not raise an error
+mapdl.ignore_errors = True  # in order for PyMAPDL to not raise an error
 output = mapdl.solve()
 print(output)
 mapdl.finish()

You just need to add a space in True # so that it looks like True  #. Another option is for you to run pre-commit run and commit the suggestions that pre-commit provides you :)

I'm taking steps to fix some issues here @pmaroneh. Wait for my input please.

I'm taking steps to fix some issues here @pmaroneh. Wait for my input please.

Hi @germa89 do you have updates on this?

I'm taking steps to fix some issues here @pmaroneh. Wait for my input please.

Hi @germa89 do you have updates on this?

I'm actually working on this.

I added the bits needed for the techdemos folder to run in here:
https://github.com/pyansys/pymapdl-examples/pull/21/commits/61aff024066485a8bbcf34df47f55d3dc795fecf
Can @clatapie check this when you have some time?

@pmaroneh @clatapie
I do remember to have said to Camille we will only accepts RST files for the repos. However, because this example is already finished and we do not have any guidance on how to do the RST files, let's gonna skip this one.
Also, the sphinx cache should detect that there is no changes in the python file and hence, it should not rerun it (assuming the sphinx cache is in place).
So yeah @clatapie , let's merge this one when ready. Apologies for keep changing the directives!


Change compilation targets

Remove net451 as a compilation target
Update tests and samples to netcoreapp2.0
Removed unused sample

@HaoK - are the tests supposed to work on AppV?
 System.Data.SqlClient.SqlException : A network-related or instance-specific error occurred while establishing a connection to SQL Server. The server was not found or was not accessible. Verify that the instance name is correct and that SQL Server is configured to allow remote connections. (provider: TCP Provider, error: 25 - Connection string is not valid)

The SqlClient exception was due to https://github.com/dotnet/corefx/issues/17166. Reverting to netcoreapp1.1 for now

@divega @bricelam @Eilon No LocalDb support seems pretty bad. Seems like this may block moving EF unless we add workarounds like using SQL Express for everything.
dotnet/corefx#17166 (Managed SNI on windows doesn't allow LocalDB connectivity)

@ajcvickers I added you to a thread about this.

Merged. We'll continue tracking the netcoreapp2.0 conversion via https://github.com/aspnet/Coherence/issues/166

Did they remove the native SNI version from the package? It's odd to me that that wouldn't resolve for the closure of packages on Windows anymore.

@bricelam sorry I don't know the reason (the email thread doesn't explain it). But we're getting this on the right radars to get to a resolution ASAP.


Add integration test with combinations of models

Add a parameterized integration test with all combinations of balancer, classifier, extractor, querier. Probably good enough to use sam:random with --init_seed=42, --n_included=5, --n_excluded=5, start with --seed, and stp:n with n=90 or something.
Note:

not every combination will work
required dependencies are different for some
lstm-base and lstm-pool dont work correctly yet

#19
#20

use each model's default parameterization for the time being

4ad02492ce0f6de3ce1ca0ab06aec80929b05129 calculates the 35 combinations of fex and cls and has a test that runs it with bal:double and qry:max, skipping over the model combinations that are not valid for whatever reason. Closing this issue.