Extending declared class results in "attempting to index a nil value"

declare class Test {};

class TestChild extends Test {

}

TestChild = TestChild or {};
TestChild.__index = TestChild;
TestChild.prototype = TestChild.prototype or {};
TestChild.prototype.__index = TestChild.prototype;
TestChild.prototype.constructor = TestChild;
TestChild.____super = Test;
setmetatable(TestChild, TestChild.____super);

-- Test ist nil therefor TestChild.____super is nil too
-- So TestChild.____super.prototype tries to index nil
setmetatable(TestChild.prototype, TestChild.____super.prototype);

TestChild.new = function(...)
    local self = setmetatable({}, TestChild.prototype);
    self:____constructor(...);
    return self;
end;

https://typescripttolua.github.io/play.html#src=declare class Test {}%3B

class TestChild extends Test {

}

That looks like an expected behavior to me. declare modifier just means that the following declaration should already exist at the runtime.
Considering that there is no way for normal lua to declare TSTL-compatible classes I don't see any real usage for declare class. The only place where it would appear would be compiled libraries, where it should work fine.

Yeah, even if we added the check, the class wouldn't work at all as expected. We could provide some kind of alternate behavior if prototype doesn't exist, but I'm not totally sure what would be the best thing to do in that case.

Closing this for now as it seems intended behavior.


fix: update dependencies to the latest possible

Test run: https://github.com/splunk/wfe-test-runner-action/actions/runs/9002116784/job/24729738045?pr=18#step:9:1

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


Set discard replies config in KafkaSource

KafkaSource shouldn't handle responses from sink services
Part of #312
Related to https://github.com/knative-sandbox/eventing-kafka-broker/pull/1307

Proposed Changes

Set discard replies config in KafkaSource

Release Note

None

Docs
None

/retest


Modal Documentation - .modal-dialog & .fade interaction

Hello, this is my first issue submission. I have attempted to follow the posted guidelines but if I have made an error please let me know so I may remedy that issue in the future.
The short version
There should be documentation outlining the fact that if a modal is created with the .fade class it is also required to have a .modal-dialog class.
What is occuring
If a modal is created with the .fade class and not the .modal-dialog class the modal will show but it's _isTransitioning property will never be reset.
https://github.com/twbs/bootstrap/blob/eb81c39f2c36d88d4ebc062a2fa179980827e8be/js/src/modal.js#L263-L271
As is shown here a listener for transitionEnd is attached to the child element of the modal with the class .modal-dialog. However, since .modal-dialog is null the listener is not attached and the transitionComplete method is never called. This results in a modal that cannot be hidden as it is considered to be transitioning at all times.
The issue
It makes sense to enforce the addition of the .modal-dialog class. A check could be implemented to skip the transitionEnd listener if the dialog element is null but that could lead to issues where users could re-trigger the modal before it has visually finished animating, which is what the entire _isTransitioning property seems to be preventing in the first place.
This information should be exposed in the documentation so developers looking to co-opt the javascript modal functionality without the styling are aware of this requirement.
Suggested solution
Refactor the Remove Animation section to be an Animation section that details the fact that .modal-dialog is required when using the .fade class because it is used to listen for a transition and then contains a second paragraph containing the existing remove animation content.

.modal-dialog is a required class and structural element for the component. No where does it imply in the docs that this class is optional as far as I know. Why assume it's not required when it's in the component's code examples?

Thank you for the response.
I do believe that is a fair point, and if the desired purpose for the component is to be used wholesale without edit my point is moot.
However I believe that for those who are looking to simply use the functionality of the component and handle the styling themselves that it could be stated more explicitly.
There are a large number of optional pieces to the modal component, and they are stated as such.
However it is not explicitly stated anywhere what is required. The fact that the modal can function properly without .modal-dialog as long as .fade is not present is what I believe warrants the need for it to be explicitly stated.
Again if the purpose of the component is to be copied wholesale I do not believe this request to be valid any longer, however I do not see the harm in explicitly stating the required  structure for the proper function of the component if the use of this component is to be more of a baseline of functionality to be expanded upon.

PRs to improve the docs are welcome.


actions: bump node 20.14.0

Applied mkpr on .github/workflows/*.yml
sed s/20.13.1/20.14.0/

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 99.219%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
109

Relevant Lines:
109

💛  - Coveralls


Scroll Bar issues in ie 11, works good in other browser and older IE.

Scroll bar appears perfectly well in IE 10, chrome, IE 8, where as it does not appear well in IE 11.
I have added the image for your reference,
I have the following setting,
ControlSettings ReportSettings = new ControlSettings();
ReportSettings.AsyncRendering = true;
ReportSettings.Width = new Unit("100%");
ReportSettings.Height = new Unit("440px");
ReportSettings.SizeToReportContent = false;
@Html.MvcReportViewerFluent(Model.ReportName).ControlSettings(ReportSettings).Attributes(new { @class = "form-control", style = "width : 100% ; height: 480px ;  overflow : hidden; border: none" })

This might be a fix: there is a css attribute in the reportviewer div called overflow:auto, remove that and add overflow-x:visible can fix the problem. you don't have to add overflow-x:visible to fix it. just remove the auto.

Thanks Lfwinster this is the only fix that is actually working for me.

I am able to resolve this by adding the inline style
_:-ms-fullscreen, :root #VisibleReportContentreportViewer_ctl09 {
position: absolute;
display: inline-block !important;
}

I only registered for say: VIJAYASHANKAR: YOU ARE AWESOME! Thanks for your solution.

@VIJAYASHANKAR  where should I add this style?

Please go through this blog https://www.c-sharpcorner.com/blogs/how-to-display-vertical-scroll-bar-for-asp-net-report-viewermvcreportviewer-in-ie11

VIJAYASHANKAR,
The above link is not working. Can you please help with where to add the code you mentioned.

The MVCReportViewer control has used to render the report in Asp.net MVC Apps, during validation found the report appears with the vertical scroll bar in IE 10, chrome, IE 8, whereas it does not appear well in IE 11. And most of the sites they have mentioned it's a known issue, however, this can be resolved by just adding a CSS to your report container as like below
_:-ms-fullscreen, :root #VisibleReportContentreportViewer_ctl09 {
position: absolute;
display: inline-block !important;
}
Ensure the above line has to added on the page where you are referring the report viewer controller in the page style block


Meanwhile items without titles aren't terribly useful

Problem: Meanwhile often contains things with descriptions like:

alex posted a note.

This isn't very useful, especially when the description is about a reply, e.g.:

alex posted a comment in reply to a note.

It says almost nothing interesting. If it didn't have links, it'd be a useless part of the interface and we'd probably get rid of it.
Instead of simply saying "a note," I propose we add a part of the note text into the description. For example:

alex posted a note ("Hello, world!")

The additional content would be wrapped in a <q> tag so it was semantically correct, and italicized via style=. Also, we'd truncate longer notes to something reasonable, so that e.g.:

Lorem ipsum dolor sit amet.

would become:

alex posted a note ("Lorem ipsum dolor...")

Thoughts?

Yes. Absolutely. And exactly like that.

Do you mean adding that to the web interface representation of the minor feeds, or to the actual activity descriptions?
I ask because anyone creating a client can make their client show nicer stuff, extracting from the object itself, but currently the webUI doesn't do such a thing. That's why I often ask people to title their posts, since that makes the minor feeds on the webUI, or other "limited" clients, much nicer for everyone. So affecting only the webUI's display would probably be enough.


Settings --> LDP labels do not translate when a non-American English locale is chosen

This issue has been moved from https://issues.folio.org/browse/LDPQURY-49
Overview: The LDP settings names are not translatable when a different language is chosen.
Steps to Reproduce:

Log into some FOLIO environment as a user with appropriate permissions to Settings
Go to Settings --> Developer and change the session locale to a locale other than American English

Expected Results: All settings names appear in the new locale name.
Actual Results: The names aren't correct. See attached screenshot.

Fixed in https://github.com/library-data-platform/ui-ldp/commit/e1488d7ee39694396beb3dd8ebc4ba93ad618d7f


usb: Add missing semicolon to comply es6

Change-Id: I94fdcbea6957d9f2c7eb5e28d8364363be6a3520
Signed-off-by: Philippe Coval<EMAIL_ADDRESS>

Is this project still active ?
Feedback welcome for top contributors:
@sandeepmistry @qingyuanz @leviathan88 @steinerj @urish @Loghorn

Merged downstream:
https://github.com/abandonware/node-bluetooth-hci-socket

Released in:
<EMAIL_ADDRESS>


Roundabouts are rendered with grey filling

Bug description
What is happening?
Enclosed ways 198571047 and 198571048 are rendered with grey filling
What should be happening?
Maybe render without filling, currently it implies being an area. Jeff asked to file report.
Repro Steps
Steps to reproduce the behavior:

Go to #map=17/59.42303/24.78610
Set year between 1970-1990
See error

Screenshots & URL
URL link is listed in Repro section's step 1.

Note: some ways weren't rendered yet when screenshot was taken.

Thanks for this! Because these are closed rings, our cartographic styling treats them as "transport areas." I am going to filter these areas out of the styling in the next round of updates.

Forthcoming styling:

are those for dirt roads, etc.?

This would be for any closed transit line that does not have an area=yes tag. And that is a tag that we will want exposed in the tiles to be able to symbolize actual transit areas.

From https://github.com/OpenHistoricalMap/issues/issues/205#issuecomment-826027674, area=yes was added in the Vtiles -  Staging.

This is fixed, clossing here!


[UI] Added filter config | CU-#85ztdxf0d

Notes for Reviewers
This PR adds UI for  #7964

Signed commits

[x] Yes, I signed my commits.

Task linked: CU-85ztdxf0d [Meshery UI] Filters - Grid View: Back of Card: display the name / filename and the configuration string.

@vaibhav135 will you investigate this failing Adapter status test? https://github.com/meshery/meshery/actions/runs/5493139166/jobs/10011161907#step:5:190

Be aware of the merge conflict.

Merge conflict and failing tests.

@leecalcote Yes I am looking into tests, seems like the page is being reset after the visit.

@leecalcote I've updated the tests. They are working in my local but failing in ci/cd

Let’s see how the tests turn out…

Integration test fix ref


Linked List Questions of Striver's SDE Sheet in Java

Reverse Nodes in k-Group
Intersection of Two Linked Lists
Linked List Cycle II

Kindly assign now will add in Java @aritra-tech

Kindly change the level to 3 since there are 3 ques with hard one included. @aritra-tech

Bro it should be level 3 if a hard ques is included. @aritra-tech


cy.wrap() makes methods loose reference

Current behavior
As describe in this issue: https://github.com/cypress-io/cypress/issues/14508
cy.wrap will make objects loose track of their methods and will return undefined
Desired behavior
I'm not extremely familiar with Cypress (or any Javascript),
but i believe that i should be able call methods inside objects as in both examples
Test code to reproduce
const bar = () => {
    // this represents a method
};

// This object will throw undefined when trying to use its methods
let ObjectListError = {
    entry = { propertyStr: "foo", propertyInt: 1, propertyFunc: bar}
}

// This object will work properly
let ObjectList = {
    entry = { propertyStr: "foo", propertyInt: 1, propertyFunc: () => {
            // this represents a method
        }}
}

When('Wraping objects', () => {
    cy.wrap(ObjectList.entry).as('ObjectList')
    cy.wrap(ObjectListError.entry).as('ObjectListError')
})

Then('Using properties from objects', () => {
    cy.get('@ObjectList').then($object=>{
        cy.log($object);
        // This will show that propertyFunc is a function
        // { propertyStr: "foo", propertyInt: 1, propertyFunc: function(){ }}
        // $object.propertyFunc() will work as intended
    })

    cy.get('@ObjectListError').then($object=>{
        cy.log($object);
        // This will show that propertyFunc as undefined
        // { propertyStr: "foo", propertyInt: 1, propertyFunc: undefined}
        // $object.propertyFunc() will throw an error
    })
})

scrennshot (true code ofuscated for privacy):

Versions
cypress: 6.1.0
-From what i've read in the changelog, this "bug" is still present in 6.2.0 and 6.2.1
cypress-cucumber-preprocessor: ^4.0.0
<EMAIL_ADDRESS>

I'm not able to reproduce this. You need to provide me with a cloneable repository which successfully illustrates the issue.

I'm not able to reproduce this. You need to provide me with a cloneable repository which successfully illustrates the issue.


[GSoC] RadioButton & RadioGroup Component

As part of my proposal, I have added a RadioButton component with tests. Next goal is to work on RadioGroup component.
Output of tests:

Can one of the admins verify this patch?

@Hardik5247  Please rebase your branch onto ucr branch.


Bring back Current visitors count on the Realtime page?

Describe the feature or enhancement
Found this helpful in the previous version where we could see the current visitors count on "Realtime" screen as well, can we bring that back?

Can take a look at the code changes that might be required, but want to confirm with you folks once

You can simply look at the Vistors count on the metrics bar. The reason we got rid of it was because it was confusing to people what that value actually meant.


Custom Tailwindcss theme with Extension.js

Howdy folks;
I have been developing with Extension JS and loving it. My only issue has been getting it to recognize the custom color theme I have (see below).  I've tested this with a vanilla JS project and a NextJS project and this works for both of those.
popup/base.css
@tailwind base;
@tailwind components;
@tailwind utilities;

tailwind.config.ts
import type { Config } from 'tailwindcss';
const {
  default: flattenColorPalette,
} = require('tailwindcss/lib/util/flattenColorPalette');

const config: Config = {
  content: [
    './**/*.{html,js,ts,jsx,tsx,mdx}',
  ],
  theme: {
    extend: {
      colors: {
        primary: {
          '50': 'hsl(180, 5%, 96%)',
          '100': 'hsl(200, 6%, 90%)',
          '200': 'hsl(204, 5%, 82%)',
          '300': 'hsl(203, 5%, 69%)',
          '400': 'hsl(205, 5%, 53%)',
          '500': 'hsl(203, 5%, 47%)',
          '600': 'hsl(204, 5%, 36%)',
          '700': 'hsl(210, 5%, 31%)',
          '800': 'hsl(210, 4%, 27%)',
          '900': 'hsl(204, 4%, 24%)',
          '950': 'hsl(210, 5%, 15%)',
        },
        secondary: {
          50: "#FEF9EC",
          100: "#FCF1D4",
          200: "#F9E4A9",
          300: "#F6D883",
          400: "#F3CA58",
          500: "#F0BC2D",
          600: "#D6A10F",
          700: "#A27A0C",
          800: "#694F07",
          900: "#342704",
          950: "#1D1502"
        },
      },
      fontFamily: {
        ibm_plex: 'var(--font-ibm-plex-sans)',
      }
    },
  },
  plugins: [addVariablesForColors],
};

function addVariablesForColors({ addBase, theme }: any) {
  let allColors = flattenColorPalette(theme('colors'));
  let newVars = Object.fromEntries(
    Object.entries(allColors).map(([key, val]) => [`--${key}`, val])
  );

  addBase({
    ':root': newVars,
  });
}

export default config;

Hi~,
I created two extensions using React-TypeScript and Vue-TypeScript respectively, and the custom colors are correctly recognized in VSCode. I wonder if you could provide me with more information?
https://github.com/cezaraugusto/extension.js/assets/10126623/5a2e11f6-f64c-4ccf-8ce1-10c5c34d0a38
https://github.com/cezaraugusto/extension.js/assets/10126623/18110a33-bbc0-4a0c-8c9d-3fa1034b816d

hi @mitchgillin, thanks for reporting. Could you provide us some sample repo so we can try to reproduce the issue?

Will do! On Jun 6, 2024, at 18:51, Cezar Augusto @.***> wrote:
hi @mitchgillin, thanks for reporting. Could you provide us some sample repo so we can try to reproduce the issue?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

@cezaraugusto, First of all thank you so much for kicking off and maintaining this project; it has been awesome to work with 😄
Here is the repo; apologies if it is something simple I am overlooking. https://github.com/mitchgillin/custom-tailwind-extension-js. I bootstrapped it using --template=react-typescript.

@cezaraugusto, First of all thank you so much for kicking off and maintaining this project; it has been awesome to work with 😄首先，非常感谢您启动和维护这个项目;与您合作非常棒。
Here is the repo; apologies if it is something simple I am overlooking. mitchgillin/custom-tailwind-extension-js. I bootstrapped it using --template=react-typescript.这里是回购;如果它是简单的东西，我忽略了道歉。mitchgillin/custom-tailwind-extension-js.我使用--template=react-typescript引导它。

I'm sorry, I think the repo you provided might not have any issues because I can easily identify them in VSCode. It's possible that I didn't fully understand your question.

Hey @OSpoon , you're understanding it perfectly. We are getting different outputs however, so it may be a problem on my end 😞. This is what I am seeing:

@mitchgillin hey there! we did a bunch of updates recently, might giving it another go using the latest release?
# The @latest part is important, npx applies some heavy caching
npx extensionc create my-extension --template=react-typescript

Thanks!

going to close this one as solved given the amount of patches we added after this issue creation. @mitchgillin feel free to check https://extension.js.org/docs/getting-started/templates where we list all the publicly supported templates and please let me know if the Tailwind option is still buggy for you. thanks!


S4581378-Siamese network for ISIC 2020 Kaggle Dataset

The following pull request contains an implementation for a siamese network to classify melanoma for the 256x256 ISIC 2020 melanoma project. the project contains a readme for usage, relevant links and all findings.

This is an initial inspection, no action is required at this point
Recognition Problem : total : 20
Solves problem: The solution is well-suited to the problem (5).

The model performs reasonably very well overall: Testing accuracy: 83.76%, testing sensitivity: 75.86% , and testing Specificity: 83.93%
The t-SNE and PCA plots suggest partial feature separation
Contrastive loss is utilized, which is suitable for the problem.
Class imbalance is addressed through data augmentation and resampling, ensuring better representation of minority cases

Implementation functions as intended (3)
Good design: Well-designed (1)
Commenting: Clear and sufficient comments throughout the code. (1)
Difficulty: Hard (10)
Note:
-The t-SNE and PCA plots (both at epoch 5 and the test set) show some clustering but not distinct separation between benign and malignant classes. In particular, there seems to be significant overlap between the two classes, suggesting that the embeddings are not yet fully separable.

The AUROC and accuracy curves for training and validation show some improvement, but the validation AUROC fluctuates and even declines over epochs, which might indicate overfitting or insufficient feature discrimination.
The loss curves suggest some progress in learning, but the presence of overfitting past epoch 5 and the fluctuating validation metrics imply that the model may not be generalizing well, a likely result of insufficient feature separation.
The classification report also indicates that the model struggles with low precision for the minority class (malignant), which suggests that it produces a high number of false positives.

Marking

Good/OK/Fair Practice (Design/Commenting, TF/Torch Usage)
Good design and implementation.
Spacing and comments.
No Header blocks. -1
Recognition Problem
Good solution to problem.
Driver Script present.
File structure present.
Good Usage & Demo & Visualisation & Data usage.
Module present.
Commenting present.
No Data leakage found. Has data splits.
Difficulty : Hard.  Siamese network (Hard Difficulty)
Commit Log
Good Meaningful commit messages.
Good Progressive commits.
Documentation
Readme :Good.
Model/technical explanation :Good.
Description and Comments :Good.
Markdown used and PDF submitted. PDF Checked.
Pull Request
Successful Pull Request (Working Algorithm Delivered on Time in Correct Branch). On time.
No Feedback required.
Request Description is good. 1
TOTAL-1

Marked as per the due date and changes after which aren't necessarily allowed to contribute to grade for fairness.
Subject to approval from Shakes

Good work!


Unable to uninstall Microsoft Rewards Automator

The issue pertains to a Contact Us email from John, who discovered Uninstall is unavailable. (resolved)
(Waiting for README.md Update)

v.8.0.0 RESOLVED


Bend malfunctions when two lists are declared

Describe the bug
When I have an unused list in addition to a used list, the program doesn't terminate correctly sometimes:
To Reproduce
Steps to reproduce the behavior:

Running command bend run-c bug.bend
With code

def map(fn, list):
  fold list:
    case List/Cons:
      return List/Cons(fn(list.head), list.tail)
    case List/Nil:
      return []

def tt(x):
  return x*2

def main():
  l = [5,6,7,8]
  k = [1,2,3,4]
  return map(tt,l)

Error...

bend run-c bug.bend
Result: (map tt [5, 6, 7, 8])

Expected behavior
bend run bug.bend
Result: [10, 12, 14, 16]

Desktop (please complete the following information):

OS:   Apple M1
CPU: Apple M1
GPU: ?
Cuda Version ?

Additional context
This code works fine:
def map(fn, list):
  fold list:
    case List/Cons:
      return List/Cons(fn(list.head), list.tail)
    case List/Nil:
      return []

def tt(x):
  return x*2

def main():
  l = [5,6,7,8]
  # k = [1,2,3,4]
  return map(tt,l)

I can't reproduce it but i swear i had the same issue even when not using run-c

When you add the second list it's causing the map(tt, [5,6,7,8]) part to be converted into a lazy reference which is not evaluated.
However we can't simply expand everything since infinite terms (recursive) will loop forever.
So it's a hard problem and I haven't found the right amount of reference expanding that needs to be done to meet user expectation.
For now maybe I'll add one layer of expansion to main and it should solve most of the issues people encounter

How can I make sure I don't encounter the problem? I still don't know what I did to cause this.
Currently it feels like it's really easy to induce this behaviour.

I've implemented this, but the solution (expanding terminal references in main, not extracting combinators from main), but it's giving me unexpected behaviour changes in some programs, so I'm investigating that first before merging the fix.

Hi i just want to expand the original scope of this issue,
this its not just a problem with the lists its a lazy eval problem when a var is defined but not used
For example the code
def simplefunc(a,b):
  return a + b
def main():
  a = 1
  b = 5
  c = 7
  return simplefunc(a,b)

should return Result: 6 but instead returns Result: (simplefunc 1 5)
A possible (but maybe not easy of implementing) fix should be have a register of the number of apparitions of each symbol in the symboltable, in this example:
a | 2
b | 2
c | 1

As c does only have 1 apparition in the function scope we know its just being defined and is not being used anywhere else

@FranciscoJavierPizarro as I mentioned, I already have a fix for this, but it sometimes triggers a bug in the HVM, so I'm waiting for it to be fixed first.
It'll take care of all these simple cases where main is basically just a call to another function (and some others as well).


Add Dockerfile

This PR contains the Dockerfile that enables you to execute the required make command in a container. It has for both Mono and .Net Core.
For Mono:
To build the container, run the following command:
docker build -f Dockerfile-mono .
To see whether the .exe file works, after you built the docker container, run the following command:
docker run <Docker-ID> bin/bash -c "make mono && mono output/runme.exe"
For .Net Core:
To build the container, run the following command:
docker build -f Dockerfile-dotnet .
To see whether the .exe file works, after you built the docker container, run the following command:
docker run <Docker-ID> bin/bash -c "make dotnet && dotnet output/secrethub.dll"
Note: The PR is dependent on #7

This PR will be put on hold and continued once we have the process of building the NuGet package in place.

I'm closing this PR, we will Dockerize the build process and the testing after we move the C# specific code to a separate repo.


Network benchmarking

The idea of adding a network benchmark came up in the chat. The reason is that validators have a required minimum network speed of 500 Mbit/s.
It seems that multiple hosters promise one thing and then deliver another. Adding a network benchmark will help with sieving out subpar hardware quicker.
How?
Some infrastructure would be required for this. There are three ways that I can think of (feel free to extend :laughing:):

A. Provide own infrastructure: some GCP servers in multiple locations with fast internet.
B. Rent a service that already offers A.
C. (Ab)use existing infrastructure, like GDrive or free services that provide A.

cc @koute

Adding some networking benchmark sounds nice first. However, we would need to ensure that we don't ddos someone. And 1000 other things that we need to ensure.
IMO this should be solved by recommending some external tool in the validator setup to verify their download/upload speed.

Yeah, this is tricky. As Basti said, there are many potential problems here that need to be handled to make sure this is reliable and doesn't DoS any third-party.
The way I see it, if we want to do this and have the benchmark be fully automatic and do it at least semi-reliably then I don't really see any other way than having our own infrastructure that we control.
Basically, something like this (just writing out anything which comes to my mind):

Have multiple servers in multiple geographical regions in multiple different datacenters.
When benchmarking make sure we pick a server which has enough spare bandwidth. (Other people currently benchmarking shouldn't affect your results.)
When benchmarking make sure to not pick a server in the same datacenter. (e.g. if someone's running their node in the same GCP datacenter they'll have blazingly fast connection to anyone else who does, but this might not be true on a global scale when looking at the whole swarm of nodes, and that is what we care about)
When benchmarking make sure we pick multiple servers across the globe. (Only a single server might have a temporary issue with its connection which might not be indicative of the user's network connection.)
Deal with possible inherent bandwidth limitations of some of the servers. (e.g. if we'll have a server somewhere in Siberia where the connectivity is poor to essentially everyone else this shouldn't tank everyone's benchmarks, because the problem here is not the user's connection, but the server's; so we'd have to potentially benchmark the servers themselves and use that to weight their contribution)

All in all, a lot of potential problems, probably way too many. Trying to implement this would be a fun technical challenge, but I'm not entirely convinced it'd be worth it; perhaps like Basti said it'd be better to just YOLO it and delegate it to an external tool and expect it to be just a very rough guideline? I'm not sure.

Trying to implement this would be a fun technical challenge, but I'm not entirely convinced it'd be worth it; perhaps like Basti said it'd be better to just YOLO it and delegate it to an external tool and expect it to be just a very rough guideline? I'm not sure.

Yea also from a cost-benefit perspective this is nothing that we have to do ourselves.
Then I will look for some external tooling that already offers this and recommend it.
Leaving the issue open until this is done.


[Code Improvement] Kotlin Delegates are not null safe and contain deprecated OnLifecycleEvent.

Describe the bug
Currant kotlin delegates are not null safe and also accessing methods using getMethod("bind") is not very good practice, even though there will be no problem using these. There is also deprecated claas
"@OnLifecycleEvent"
This annotation required the usage of code generation or reflection, which should be avoided,
alternative  "DefaultLifecycleObserver" can be used. We can also eliminate typecasting.
Get Method

Not null safe (can be fixed but lot of boilerplate code)

Deprecated OnLifecycleEvent

Soultion
I have added a link for the gist , which can be used to eliminate all the above issues. I have recently migrated most of the classes from the kotlin extension and findViewById to ViewBinding.I would love to implement this kotlin delegate with those classes also there is not much refactoring we have to replace only one line in every fragment or activity
Even though the current solution is ok and works fine, it might create issues in the future, with updated delegates code will become much cleaner and soild.
I need approval from the internal team to change these delegates.
Gist:https://gist.github.com/AndroidPoet/907a7c39c6312ab4d6db6b2581d8a0a1

How to use
Fragment

Activity

DialogFragment

Hi, @aitorvs please check these once and Let me know your thoughts on this, as we discussed about memory leaks, those are  bit untraceable but with this delegate, we can make sure that there is no memory leak as for as view binding is concerned.

Hey @AndroidPoet. Thanks for opening this issue and the time spent caring about the code base.
Unless there's something fundamentally wrong with the current code and/or we find and are able to consistently repro a bug eg. memory leak in the viewbinding delegate logic, I'd personally not be very supportive of changing it just because.


AddPushNotifications()

In this document you refer to adding .AddPushNotifications()
In Visual Studio this is highlighted with the following text "Use App Service Push instead."
Most of your documentation regarding the correct way to integrate push notifications is confusing and out of date.
You need to sort this out, I have been going round in circles trying to implement this feature!

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

ID: fd41b897-29ee-837b-99bd-2be5c5331e34
Version Independent ID: aeebb0e3-1698-b430-1912-7564f6ed99e4
Content: How to work with the .NET backend server SDK for Mobile Apps
Content Source: articles/app-service-mobile/app-service-mobile-dotnet-backend-how-to-use-server-sdk.md
Service: app-service-mobile
GitHub Login: @conceptdev
Microsoft Alias: crdun

Hi @ianpowell2017, as our product strategy evolves we have got some outdated docs that need work; these mobile docs are on the backlog to update.
Depending on your use case, App Center Push might be appropriate for your app?
Thanks for the feedback. We hope to have these docs updated soon.

Your outdated documentation is a serious setback.  I totally agree with @ianpowell2017!

Thanks both for your feedback. As the SDK suggests, and mentioned above, App Center or Notification Hubs might be a better choice for your app. We're working to make the options clearer in the products & documentation.
#please-close


ci: build and push docker image

Build a docker image tagged with the commit hash (eg: stratumn/filetmpop: 32fe9d75) after each successful build on master.
We could chose to push after each successful build on all branches. We'd just have to add:
all_branches: true

to the on: conditions
What do you think?

I think it's a good idea. Otherwise it's only pull requests and master?
I think the Makefile will also tag and push the image as latest. Do you think that's OK?

I feel like if we chose to build & push only on master then it's ok to push as latest.
If we build & push on all branches then we should probably only push with the commit hash as a tag

It's finally working!
What do we decide then @stephan83?
My guts tell me: only master with latest.

Note that pull request builds skip the deployment step altogether.


KAFKA-7895: fix Suppress changelog restore (#6536)

Several issues have come to light since the 2.2.0 release:
upon restore, suppress incorrectly set the record metadata using the changelog record, instead of preserving the original metadata
restoring a tombstone incorrectly didn't update the buffer size and min-timestamp
Cherry-picked from #6536 / 6538e9e
Reviewers: Guozhang Wang<EMAIL_ADDRESS>Matthias J. Sax<EMAIL_ADDRESS> Bruno Cadonna<EMAIL_ADDRESS> Bill Bejeck<EMAIL_ADDRESS>Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@bbejeck Here's the cherry-pick to the 2.1 branch. Thanks for your help!

Java 8 failed with
kafka.server.RequestQuotaTest.testResponseThrottleTimeWhenBothFetchAndRequestQuotasViolated
kafka.tools.MirrorMakerIntegrationTest.testCommaSeparatedRegex

Java 11 passed
retest this please

Reported flaky test for java 11:
https://issues.apache.org/jira/browse/KAFKA-8280
Retest this, please.

Merged #6616 into 2.1

Thanks, @bbejeck !


RHIDP-3389: Install and Run RHDH on OSD on GCP

IMPORTANT: Do Not Merge - To be merged by Docs Team Only
Version(s):
1.3
Issue:
RHIDP-3389
Link to docs preview:
Find the link within the PR
Reviews:

[x] SME: @subhashkhileri
[x] QE: @subhashkhileri
[x] Docs review: @themr0c
[ ] Additional review: @mention assignee (by writer)

Additional information:

I’ve updated the structure in the PR and also created a Jira to apply the same structure to the other titles: RHIDP-4165, so we don’t overlook these changes.

/cherry-pick release-1.3


Automatically create Jira tickets for dependabot upgrades

This will automatically create a Jira ticket for all dependabot PRs, link the PR to the ticket and then add the Jira URL as a comment on the PR itself.
Example Jira ticket: https://issues.redhat.com/browse/ISPN-15414
I'll create a different workflow for the backport branches, as we'll need to update an existing Jira instead of creating a new one.

Merged, thanks


Issue #93: user-defined sampling function

This is an update on an earlier pull request (https://github.com/SheffieldML/GPyOpt/pull/102) accounting for the refactorization in the recent release. A keyword argument has been added called user_def_dist which is a user-defined function that generates variable samples to initialize the optimization of the acquisition function. This is motivated primarily by the risk of random sampling failing to find a feasible point for challenging constraints in a reasonable amount of iterations.
I am also investigating whether including the optimizer fmin_cobyla from scipy might be a practical automatic option. If it ends up being helpful, I will submit it in a new pull request.

Codecov Report

Merging #123 into master will decrease coverage by <.01%.
The diff coverage is 91.48%.

@@            Coverage Diff             @@
##           master     #123      +/-   ##
==========================================
- Coverage   57.89%   57.88%   -0.01%
==========================================
  Files          83       83
  Lines        5083     5070      -13
==========================================
- Hits         2943     2935       -8
+ Misses       2140     2135       -5

Impacted Files
Coverage Δ

GPyOpt/optimization/anchor_points_generator.py
96.36% <100%> (+0.06%)
:arrow_up:

GPyOpt/methods/bayesian_optimization.py
84.26% <100%> (+0.17%)
:arrow_up:

GPyOpt/optimization/acquisition_optimizer.py
95% <66.66%> (+0.08%)
:arrow_up:

GPyOpt/util/general.py
62% <90.9%> (+0.07%)
:arrow_up:

GPyOpt/util/stats.py
9.81% <0%> (-0.9%)
:arrow_down:

Continue to review full report at Codecov.

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

Closing as we have declared end of maintenance.


Grant Milestone: Completion of Basics and Apps milestones on Android

I will complete tasks in the Basics and Apps milestones that have not been taken by the @crc-32 yet:
Milestone 1:

[X] Sync phone time to the watch (https://github.com/pebble-dev/mobile-app/issues/16)
[x] Calendar sync (https://github.com/pebble-dev/mobile-app/issues/26)
[x] Music Control (https://github.com/pebble-dev/mobile-app/issues/25)

Milestone 2:

[x] App installation fro pbw file (https://github.com/pebble-dev/mobile-app/issues/33)

[x] Store app locker on the phone (https://github.com/pebble-dev/mobile-app/issues/34 - This task is prerequisite for  both me and @crc-32. We have agreed that whoever needs it first, implements it)

[x] Complete PebbleKit Android implementation (https://github.com/pebble-dev/mobile-app/issues/37)

[x] Add support for Sports and Golf apps (https://github.com/pebble-dev/mobile-app/issues/38)

[x] Complete developer connection implementation (https://github.com/pebble-dev/mobile-app/issues/47)

✖ Add Data Logging support (https://github.com/pebble-dev/mobile-app/issues/39) WON'T DO

Limitations

Everything here applies to Android only. While I will try to implement as much features as possible in flutter, which is shared between Android and iOS, iOS developer will still need to connect all of those features with the iOS.
This does not include proper UI, since designs are still being done (https://github.com/pebble-dev/mobile-app/issues/43) and to be frank, I need to get some experience with Flutter UI first. As the part of this grant, I will add some rudimentary UI to the app that will allow use of the features, but it will not look or feel good. Proper "fancy" UI can be implemented after the fact.

:tada: This grant is approved!  Rebble Alliance will pay $1500 on completion of milestone 1, and $1500 on completion of milestone 2.  This is super exciting -- thanks!

This is absolutely amazing.  I agree that milestone 1 is complete!

This is absolutely amazing.  I agree that milestone 1 is complete!

@jwise I don't think I will be doing the data logging. It is very complicated, there is little documentation and from what I can see, no apps use it anymore. Can you deduct that from my grant, please?

Excellent, excellent work.  When I originally estimated Milestone 2, it looks like I included some PBW install work, but it looks like we assumed you might not be doing the app locker storage work -- but you did end up doing that after all.  So I'm happy to call it even at the originally agreed $1,500.  I agree that Milestone 2 is complete!

cc: @ishotjr !


Full pipeline

To fix the docs build, go to line 40, where there is the section that starts with @docs, and add the relevant function names to the newline-delimited list of names.

To fix the docs build, go to line 40, where there is the section that starts with @docs, and add the relevant function names to the newline-delimited list of names.

@DilumAluthge Not sure if this is an automatic comment. See #60


No way to enter decimals when sending funds

Maybe only for Europeans?

Vlad: could you include a test case for this too. The screen's too fragile

@colourfreak can you help to do a screenshot of the Settings.app > General > Language & Region screen, as well as the in-app Settings > Advanced > Change Language screen when you can reproduce this?
I can't reproduce the Send screen in your video which shows 0.3628 as the balance ("." as the decimal separator), but with the keyboard showing "," as the decimal separator.
This screen in Settings.app :

@colourfreak thanks. How about the in-app version?


Fix tagged template start to include tag

Regression of https://github.com/swc-project/swc/pull/677/files

Sorry, I was trying to test the changes earlier, but I had to do a bit of a refactoring because TokenAndSpan no longer implemented Spanned (no big deal, I ended up refactoring everything to use SpanData, which is probably more efficient).
Anyway, everything looks good now after this PR!

I've published a new version. Thanks for the pr!

Thanks!


Possible to copy texture/model of existing item?

Thermal Foundation includes a non-block geode item (thermalfoundation:geode) that I'd like to create variants of for the modpack I'm developing. They don't need to look different, they just need to process differently in my modular machines.
Looking through the documentation, I wasn't sure...is there a quick way to create new items and have them copy the textures/models from an existing item? My first instinct after reading the document would be to try something like this:
#loader contenttweaker

import crafttweaker.item.IItemDefinition;
import crafttweaker.item.IItemStack;
import mods.contenttweaker.Item;
import mods.contenttweaker.IResourceLocationSupplier;
import mods.contenttweaker.ResourceLocation;

var FireyGeode = VanillaFactory.createItem("Firey_Geode");
FireyGeode.displayName = "Firey Geode";
FireyGeode.itemTextureLocationSupplier = <thermalfoundation:geode>.itemTextureLocationSupplier;

...but my gut is usually wrong about these things. I suppose I could find and copy the texture over into the resource folder, but I just wanted to see if there was a simpler way to achieve the same effect.

You can try to modify the created firey_geode.json model file to point to the same texture location as TF.


BUG:Navbar does not reset scroll position when navigating between pages

Is there an existing issue for this?

[X] I have searched the existing issues

What went down?
When navigating between pages using the navbar, the scroll position from the previous page is retained. For example, if a user scrolls to the bottom of one page and then navigates to a different page, the new page also opens at the bottom, rather than resetting to the top.
Record

[X] I agree to follow this project's Code of Conduct
[ ] I'm a gssoc-24-extd contributor
[X] I want to work on this issue
[X] I check the issue doesn't exist

@RamakrushnaBiswal please assign this issue to me. I would like to fix this

Duplicate as #138


Bug V4:  Matrix view not loading

Describe the bug
after installing V4 with BRAT, the matrix view no longer loads and it shows this error

To Reproduce
install the plugin with brat, use the breadcrubs: show matrix view command
Expected behavior
showing the matrix view
Screenshots

Desktop (please complete the following information):

OS:Arch linux
Breadcrumbs version: 4.2.35 Beta
Obsidian version: 1.7.4

This is also true for Treeview with v4.2.35 beta

Updating to 1.7.6 resolved this problem for me.

its still there for me even with 1.7.6

was anyone able to do anything about this? its the only reason im not updating to obsidian 1.7.7

I paid attention a bit bc of your question. I noticed that now it still happens on open but it seems to resolve itself within 30-60s after indexing is done


Wordpress plugins version

Could you advise which version of plugins are you using with WordPress?

Of course, here are the current requirements from composer.json:
{
  "require": {
    "php": ">=7.3",
    "composer/installers": "^1.10.0",
    "oscarotero/env": "^2.1.0",
    "roots/bedrock-autoloader": "^1.0.4",
    "roots/roots-branding": "dev-master",
    "roots/wordpress": "5.6.1",
    "roots/wp-config": "1.0.0",
    "roots/wp-password-bcrypt": "1.0.0",
    "vlucas/phpdotenv": "^5.3.0",
    "wp-graphql/wp-graphql": "^1.1.3",
    "wp-graphql/wp-graphql-jwt-authentication": "^0.4.1",
    "wp-graphql/wp-graphql-woocommerce": "^0.7.0",
    "wp-premium/woocommerce-table-rate-shipping": "3.0.29",
    "wpackagist-plugin/autodescription": "^4.1.3",
    "wpackagist-plugin/backwpup": "^3.8.0",
    "wpackagist-plugin/jetpack": "^9.4",
    "wpackagist-plugin/woo-payment-gateway": "^3.2.18",
    "wpackagist-plugin/woocommerce": "^5.0.0",
    "wpackagist-plugin/woocommerce-pdf-invoices-packing-slips": "^2.8.2",
    "wpackagist-plugin/wp-mail-smtp": "^2.6.0",
    "wpackagist-plugin/wp-sentry-integration": "^4.1.1",
    "wpackagist-plugin/wp-smushit": "^3.8.2"
  }
}

The WordPress setup is based on Bedrock and I have a headless theme to make some customizations like adapting CORS headers and adding SEO data to the GraphQL API.

Oh, this is very nice approach. I will try Bedrock with my installation. For now I was using the standard WP installation with Woocommerce. CORS are handled by wp-graphql-cors and SEO working with wp-graphql-yoast-seo (with little changes with queries)


Make running commands in the terminal better

Platforms other than linux are now supported.
Also use systemEncoding to decode the processes stdout and stderr instead of assuming the bytes are valid.

Looks reasonable, could you please add some tests and ensure CI passes? Then we are 100% sure it is great!


Separate model got the error: cannot convert value of type ‘(user_Id: Int, user_Email: String, user_FirstName: String….)

Hi all, I face with the error 'cannot convert value of type ‘(user_Id: Int, user_Email: String, user_FirstName: String….)' when try to separate my model.  Thanks so much.

Looks like your last monadic operator is <^> (map) instead of <*> (apply). This needs to be fixed.

@gfontenot I change the variable 'role' to 'let user_Role: String' and replace (<^> json <| "role") to (<*> json <| "role") as your advice but still got the same error.

Oh, I see. You aren't currying your initializer either. You need to either write a curried static method, or you need to use curry function to curry UserModel.init. For example, see Curry.framework

@gfontenot not as you guess, I had Curry framework in pod file. And can parse data successful with another model. Like that:
struct CountryState{
let stateId: Int
let stateName: String
let stateAbbrev: String
}
extension CountryState: Decodable {
static func decode(j: JSON) -> Decoded {
return curry(CountryState.init)
<^> j <| "id"
<> j <| "name"
<> j <| "abbrev" // Use ? for parsing optional values
}
}
BUT when I try to parse UserModel with many variable. I face with the error as above.

The original code snippet you posted above doesn't use curry. Was that a typo?

@gfontenot ooh, my god.  You save my life.Thanks so much

No problem! Glad I could help. I'm going to close this, but feel free to continue asking questions.


Allow user to add an element to a beam line by dragging it onto the beam line preview window.

Many users, when first encountering the Elegant Bunch Transport tab, try to create beam lines by dragging it onto the preview window. Accommodate this by adding the element to the end of the current working beam line.
In the future, this may be improved by detecting where the user dropped the element in the picture and adding the element to the corresponding place in the beam line, but this is a difficult problem (especially with curved beam lines).

Fixed by commit 8afb066a3eba1911895dda2b58cf9ee06cd31cc6.


select box not inline in other fields

I've copied the code in the documentation and it always show like this. Is there any fix on this or related issue ?

    <input id="city" type="text" class="validate">
    City

    <input id="state" type="text" class="validate">
    State

    <input id="zip_code" type="text" class="validate">
    Zip Code

    Materialize Select
    <select>
      <option value="" disabled selected>Choose your country</option>
      <option value="1">Country # 1</option>
      <option value="1">Country # 1</option>
      <option value="1">Country # 1</option>
      <option value="1">Country # 1</option>
    </select>

Its because there is a conflict between what a select replacement is doing and the use of the .input-field class on the div containing the replacement. The CSS doesn't know how to properly handle the label.
I have provided some SASS that will line them up correctly for you, but you need to replace the .input-field on the div with the select with .select-field.
.select-field {
	@extend .input-field;

	label {
		@extend label.active;
	}

	.select-wrapper {
		margin-top: 0;
	}
}

We meant for alignment with selects to be without labels. If you remove the label, everything should align. If you want labels as well you need to have some custom styles


[IOTDB-1420] Fix compaction ttl bug

If the merge selects the file to be compacted, the isMerging field in TsFileResource should be set to true, so that the tsfile will not be deleted by ttl.
A large try-finally should be added outside the CompactionUtils.merge function, if a file is opened in the middle Exception, the file that has been successfully opened before should be closed in the finally block

Coverage decreased (-0.003%) to 67.724% when pulling d32da9507229e66c9c261bcded3279a213c7e763 on fix_compaction_ttl into 8c518ae213238a9aa68b2a18b011b35cd87f2d59 on master.


Fixed :meth:~Cylinder.add_bases opengl compatibility

Overview: What does this pull request change?

Motivation and Explanation: Why and how do your changes improve the library?

Fixes #1995
Line3D was failing due to opengl compatibility issues in base_classes method as the underlying opengl bases classes do not have fill_color and fill_opacity. Added some fixes for this and removed the deprecated u_min, u_max, v_min and v_max fields.
Links to added or changed documentation pages

Further Information and Comments

Reviewer Checklist

[ ] The PR title is descriptive enough for the changelog, and the PR is labeled correctly
[ ] If applicable: newly added non-private functions and classes have a docstring including a short summary and a PARAMETERS section
[ ] If applicable: newly added functions and classes are tested

This looks pretty reasonable to me, but maybe it would be better to split off the deprecation removal into a different PR for the sake of a clearer changelog?

This looks pretty reasonable to me, but maybe it would be better to split off the deprecation removal into a different PR for the sake of a clearer changelog?

Yea sure I can do that!


Should calls to pydantic dataclasses with invalid signature raise a ValidationError or TypeError?

I noticed that now, instead of raising a TypeError when you call dataclasses with an invalid signature, we raise a ValidationError.
Are we happy with this? In v1 it raises a TypeError:
import pydantic

@pydantic.dataclasses.dataclass
class X:
    x: int

X()

# V1 error:
# TypeError: X.__init__() missing 1 required positional argument: 'x'

# V2 error:
# pydantic_core._pydantic_core.ValidationError: 1 validation error for X
# x
#   Field required [type=missing, input_value=ArgsKwargs(()), input_type=ArgsKwargs]

I think it's much better now.

Main downside is the risk that someone had set up error handling for invalid calls. But I'm fine with this if you are. Feel free to just close the issue.

@samuelcolvin any reason we can't do something like:
from typing import Any

from pydantic import ValidationError

class ValidationTypeError(ValidationError):
    pass

def func(*args: Any, **kwargs: Any) -> Any:
    # some validation is happening
    if len(args) < 2:
        raise ValidationTypeError(
            'not enough args',
            errors=[],
        )

try:
    func(1)
except TypeError:
    print('okay')

try:
    func(1)
except ValidationError:
    print('okay')

Since ValidationError currently inherits form ValueError it might be tricky (as in, require several macros) to make this work in Rust. It's doable but I'm not going to implement it unless we need / really want to.


about seg_mask generation

Hi, could you please share your files in csv format? i.e. '/store/datasets/UAV/csv.csv' or '/store/datasets/UAV/val.csv' or sharing some code elaborating how the files in csv  format are being created? Thanks.

Hi, I just added my csv files as well as the scripts I used to convert the dataset format to the csv format to: https://polymtlca0-my.sharepoint.com/:f:/g/personal/hughes_perreault_polymtl_ca/EhqSkfDIJ-JBh9_YhCrPQrEBocvfP6BIucODKdcNjZzlcA?e=xqRuEv
The csv files are quite simple, the format is: path, x0, y0, x1, y1, label. In the scripts you can also find the sequences I used for val / train split.

Thanks for your reply!

I used the file "generate.py" to generate a optical flow image, and the file "flow_to_seg_bbox.py" to make segmentation labels. However, the label is not as credible as in the paper. Did you use other processes to generate optical flow images?

mask_generation.txt
Here is my code.

Here are example of segmentation label and its original image

I found where the code was wrong, sorry to disturb


[RELEASE] Release v2.3.0

Release the next version of opensearch-java with new features and bug fixes.
Requested in:

https://github.com/opensearch-project/opensearch-java/issues/369#issuecomment-1448724631
https://github.com/opensearch-project/opensearch-java/issues/376#issuecomment-1446413708
https://github.com/opensearch-project/opensearch-java/issues/282#issuecomment-1437699098

Is this release scheduled? Hoping to avoid switching to HLRC due to #282

Is this release scheduled? Hoping to avoid switching to HLRC due to #282

@rmertzpfpt Going to release this today :)

Is this release scheduled? Hoping to avoid switching to HLRC due to #282

@rmertzpfpt Going to release this today :)

Triggered the release workflow, waiting for approvals on #405

Got approval for the release #406
The jenkins workflow is failing: https://build.ci.opensearch.org/job/opensearch-java-maven-sign-and-release/26/console

We are working to fix the issue that is causing the workflow to fail. Once fixed, the artifact should be released.

Version 2.3.0 is released https://github.com/opensearch-project/opensearch-java/releases/tag/v2.3.0


WFCORE-981 - RemoteDomainConnectionService uses out of date HostInfo

Going to just add as a commit to WFCORE-310, closing


Started engine does nothing (MMT 3)

Copied engine from previous Ubuntu install:
./mmt start -e engdut
Starting MMT engine 'engdut'... OK
Loading models... OK
The MMT engine 'engdut' is ready.
You can try the API with:
curl "http://localhost:8045/translate?q=world&source=en&target=it&context=computer" | python -mjson.tool
loek@Doraemon:~/mmt8$ ./mmt translate -e engdut "hello world"
ERROR Illegal State: MMT engine 'engdut' is not running.
Start it with "./mmt start"
You can check the status of your engines with "./mmt status"

I have no idea of what you're referring to anymore. Please open one "question" issue at time, if you try to do everything at the same time, I cannot help you. Try to solve one problem at time.

This is MMT 3 in a different directory, with engines from a previous back-up that worked up to this morning.

This seems to be caused by multiple installations of numpy, when following the standard installation procedure.
pip uninstall numpy (repeat until nothing is left)
pip install numpy
solves this.


Fix images not being shown properly

@nenonix @JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks.

@lizap - PR has been copy edited and is ready for final review.
Thank you.


Generate Yup ValidationSchema from Graphql Endpoint

It would be convenient if there would be a way to generate yup validation schemas. You would need to have some kind of way to be able to extend the generated schemas, but it could save a lot of work

What would they be used for? What would they be generated from? What would they validate and not validate?
Sent with GitHawk

They would be used to do client-side validation of the data my users are sending, for example in combination with formik. They would be generated from the graphql endpoint or maybe the prisma schema, so if I have:
model Batch {
  id           String        @default(cuid()) @id
  createdAt    DateTime      @default(now())
  date         DateTime?
  manufacturer Manufacturer?
  batchNumber  String
  product      Product
}

it would generate a yup schema along the lines of:
Yup.object().shape({
date : Yup.date(),
manufacturer: ManufcaturerSchema,
batchNumber: Yup.string().required(),
product: ProductSchema.required()

Yeah the the GraphQL schema would be the source of truth for something like this.
Why not a TypeScript client? There is no actual validation taking place if deriving from GraphQL schema or Prisma schema. Just types. The difference is a string type vs a regex pattern. The latter is validation.
Generating clients from the GraphQL schema is something you can achieve with various tools already. If we want the clients to have real validation included to, then we should first figure out how a API author annotations GraphQL type fields with validation.


Update faq.yml

Numerazione FAQ e Update Tag HTML

Description

As already mentioned in #ISSUE_NUMBER, this PR tackles:

...
...
...

In particular, the ...
Checklist

[ ] I followed the indications in the CONTRIBUTING
[ ] The documentation related to the proposed change has been updated accordingly (also comments in code).
[ ] Have you written new tests for your core changes, as applicable?
[ ] Have you successfully ran tests with your changes locally?
[ ] Ready for review! :rocket:

Fixes

Fixes #

@bfabio @denismarini dopo revisione HD è possibile fare deploy?


SSL certificate problem

i'm pretty new to using this framework so excuse my noobness,
i'm attempting to retrieve registration ID's saved in firebase database and then push notifications to those devices.
require 'vendor/autoload.php';
$firebase = Firebase::fromServiceAccount(
    __DIR__.'/google-service-account.json',
    'https://c9029.firebaseio.com'
);

$database = $firebase->getDatabase();
$reference = $database->getReference('/tokens/');
$value = $reference->getValue();
echo ("id is$value");
//$title = $_POST["title"];
//$body = $_POST["body"];
$registrationIds = array($RID);

this is the code i am using to fetch those values.
in the database it is structured as /tokens/UniqueOrRandomID/token:value
however the following error is returned
Fatal error: Uncaught GuzzleHttp\Exception\RequestException: cURL error 60: SSL certificate problem: unable to get local issuer certificate (see http://curl.haxx.se/libcurl/c/libcurl-errors.html) in C:\xampp\htdocs\fcm\vendor\guzzlehttp\guzzle\src\Handler\CurlFactory.php:187 Stack trace: #0 C:\xampp\htdocs\fcm\vendor\guzzlehttp\guzzle\src\Handler\CurlFactory.php(150): GuzzleHttp\Handler\CurlFactory::createRejection(Object(GuzzleHttp\Handler\EasyHandle), Array) #1 C:\xampp\htdocs\fcm\vendor\guzzlehttp\guzzle\src\Handler\CurlFactory.php(103): GuzzleHttp\Handler\CurlFactory::finishError(Object(GuzzleHttp\Handler\CurlHandler), Object(GuzzleHttp\Handler\EasyHandle), Object(GuzzleHttp\Handler\CurlFactory)) #2 C:\xampp\htdocs\fcm\vendor\guzzlehttp\guzzle\src\Handler\CurlHandler.php(43): GuzzleHttp\Handler\CurlFactory::finish(Object(GuzzleHttp\Handler\CurlHandler), Object(GuzzleHttp\Handler\EasyHandle), Object(GuzzleHttp\Handler\CurlFactory)) #3 C:\xampp\htdocs\fcm\vendor\guzzlehttp\guzzle\src\Handler\Proxy.php(28): GuzzleHttp\Ha in C:\xampp\htdocs\fcm\vendor\kreait\firebase-php\src\Firebase\Exception\ApiException.php on line 28
i am running this using xampp locally
hope someone can help, thanks

Please have a look at http://firebase-php.readthedocs.io/en/latest/troubleshooting.html, perhaps the steps described there can help with the problem.
SSL errors usually hint to a problem of the system the application is running on, so maybe it could also help if you upgrade your xampp environment (if there is an update available).

adding the certificate file definitely worked :D
oddly no notifications are pushed

The PHP SDK currently only supports working with the realtime database - I don't know how notifications work, but there's nothing built-in here.

i found another php script online that takes care of notifications
<?php
require 'vendor/autoload.php';
$firebase = Firebase::fromServiceAccount(
    __DIR__.'/google-service-account.json',
    'https://riara-c9029.firebaseio.com'
);

$database = $firebase->getDatabase();
$reference = $database->getReference('/tokens/');
$value = $reference->getValue();

echo ("id is$value");
//$title = $_POST["title"];
//$body = $_POST["body"];
$registrationIds = $RID;
//was $registrationIds = arrays($RID);

       $message = array
       (
        'message'   => '123',
        'title'     => '123',
        'subtitle'  =>  '123',
        'tickerText'    =>  '123',
        'vibrate'   => 1,
        'sound'     => 1,
        'largeIcon' => 'large_icon',
        'smallIcon' => 'small_icon'
       );

       $url = 'https://fcm.googleapis.com/fcm/send';
       $fields = array(
         'registration_ids' => $registrationIds,
         'notification' => $message
        );
       $headers = array(
        'Authorization:key = AAAARG56bIg:APA91bFUOwbmV8PeqnJiOBw62TCHIpHjqYbxyXC9z9kWdnrV5xEwTzS5UbRVzQwYaRW3KoB77FV4mjFpBRX390jl23tcFWmqodgsTY6wBL29gB_ykMXcSUhdsnj8VYsH8PmlYe3N7icLKBE1MwcojEeDdgZgyg1kFw',
        'Content-Type: application/json'
        );
       $ch = curl_init();
       curl_setopt($ch, CURLOPT_URL, $url);
       curl_setopt($ch, CURLOPT_POST, true);
       curl_setopt($ch, CURLOPT_HTTPHEADER, $headers);
       curl_setopt($ch, CURLOPT_RETURNTRANSFER, true);
       curl_setopt ($ch, CURLOPT_SSL_VERIFYHOST, 0);
       curl_setopt($ch, CURLOPT_SSL_VERIFYPEER, false);
       curl_setopt($ch, CURLOPT_POSTFIELDS, json_encode($fields));
       $result = curl_exec($ch);
       if ($result === FALSE) {
           die('Curl failed: ' . curl_error($ch));
       }
       curl_close($ch);

       echo $result;

Uuh, thanks! This might come in handy for a future feature addition! <3

welcome hope i get it to work


set_spring_arm_spring_length does not change the spring length

Issue description
This might be a problem with the Godot docs, but https://github.com/ramokz/phantom-camera/blob/07e0b9893385fea75bae7b5bf0b7cd014319356e/addons/phantom_camera/scripts/phantom_camera/phantom_camera_3D.gd#L977 does not work.
It needs to be
_follow_spring_arm_node.spring_length = value

Steps to reproduce
Here's a simple mouse wheel zoom to add to any test project
@export_group("Zoom", "zoom_")
@export_subgroup("Speed", "zoom_speed_")
@export var zoom_speed_mouse_wheel: float = 2.0 ** (1.0 / 10.0)
@export var zoom_speed_joy: float = 8.0
@export_subgroup("Clamp", "zoom_clamp_")
@export var zoom_clamp_min: float = 0.1
@export var zoom_clamp_max: float = 10.0
func _unhandled_input(event: InputEvent):
	if event is InputEventMouseButton:
		match event.button_index:
			MOUSE_BUTTON_WHEEL_UP:
				var new_zoom = clamp( controller.follow_distance / zoom_speed_mouse_wheel, zoom_clamp_min, zoom_clamp_max)
				controller.follow_distance = new_zoom  # does nothing
				controller.set_spring_arm_spring_length( new_zoom ) # does nothing
				controller._follow_spring_arm_node.spring_length = new_zoom # actually works
			MOUSE_BUTTON_WHEEL_DOWN:
				var new_zoom = clamp( controller.follow_distance * zoom_speed_mouse_wheel, zoom_clamp_min, zoom_clamp_max)
				controller.follow_distance = new_zoom  # does nothing
				controller.set_spring_arm_spring_length( new_zoom )  # does nothing
				controller._follow_spring_arm_node.spring_length = new_zoom # actually works

(Optional) Minimal reproduction project
No response

Tired it out using the example code below in the _unhandled_input() function of player_controller_third_person.gd in the 3DFollowThirdPersonExampleScene scene and _follow_spring_arm_node.set_length(value) works fine for me.
if event is InputEventMouseButton:
  match event.button_index:
    MOUSE_BUTTON_WHEEL_UP:
       _player_pcam.set_spring_arm_spring_length(
           _player_pcam.get_spring_arm_spring_length() + 0.1
       )
    MOUSE_BUTTON_WHEEL_DOWN:
      _player_pcam.set_spring_arm_spring_length(
        _player_pcam.get_spring_arm_spring_length() - 0.1
      )

The syntax is a bit inconsistent in Godot's docs, but the setter method for SpringArm3D is correct.
Video example
https://github.com/ramokz/phantom-camera/assets/5159399/7c19db6b-15ab-4705-b92e-9d15976e1d03

The following code fails to zoom the camera:
func _unhandled_input(event: InputEvent):
	if event is InputEventMouseButton:
		match event.button_index:
			MOUSE_BUTTON_WHEEL_UP:
				var follow_before = pcam.follow_distance
				var spring_before = pcam._follow_spring_arm_node.spring_length
				pcam.set_spring_arm_spring_length( pcam.get_spring_arm_spring_length() - 0.1 )
				var follow_after = pcam.follow_distance
				var spring_after = pcam._follow_spring_arm_node.spring_length
				print ( "MOUSE_BUTTON_WHEEL_UP | follow_distance %s -> %s | spring_length %s -> %s"%[follow_before,follow_after,spring_before,spring_after] )
			MOUSE_BUTTON_WHEEL_DOWN:
				var follow_before = pcam.follow_distance
				var spring_before = pcam._follow_spring_arm_node.spring_length
				pcam.set_spring_arm_spring_length( pcam.get_spring_arm_spring_length() + 0.1  )
				var follow_after = pcam.follow_distance
				var spring_after = pcam._follow_spring_arm_node.spring_length
				print ( "MOUSE_BUTTON_WHEEL_DOWN | follow_distance %s -> %s | spring_length %s -> %s"%[follow_before,follow_after,spring_before,spring_after] )

MOUSE_BUTTON_WHEEL_DOWN | follow_distance 5 -> 5.1 | spring_length 5 -> 5
MOUSE_BUTTON_WHEEL_DOWN | follow_distance 5.1 -> 5.2 | spring_length 5 -> 5
MOUSE_BUTTON_WHEEL_DOWN | follow_distance 5.2 -> 5.3 | spring_length 5 -> 5
MOUSE_BUTTON_WHEEL_DOWN | follow_distance 5.3 -> 5.4 | spring_length 5 -> 5
MOUSE_BUTTON_WHEEL_DOWN | follow_distance 5.4 -> 5.5 | spring_length 5 -> 5
MOUSE_BUTTON_WHEEL_DOWN | follow_distance 5.5 -> 5.6 | spring_length 5 -> 5
MOUSE_BUTTON_WHEEL_DOWN | follow_distance 5.6 -> 5.7 | spring_length 5 -> 5
MOUSE_BUTTON_WHEEL_DOWN | follow_distance 5.7 -> 5.8 | spring_length 5 -> 5
MOUSE_BUTTON_WHEEL_DOWN | follow_distance 5.8 -> 5.9 | spring_length 5 -> 5
MOUSE_BUTTON_WHEEL_DOWN | follow_distance 5.9 -> 6 | spring_length 5 -> 5
MOUSE_BUTTON_WHEEL_UP | follow_distance 6 -> 5.9 | spring_length 5 -> 5
MOUSE_BUTTON_WHEEL_UP | follow_distance 5.9 -> 5.8 | spring_length 5 -> 5
MOUSE_BUTTON_WHEEL_UP | follow_distance 5.8 -> 5.7 | spring_length 5 -> 5
MOUSE_BUTTON_WHEEL_UP | follow_distance 5.7 -> 5.6 | spring_length 5 -> 5
MOUSE_BUTTON_WHEEL_UP | follow_distance 5.6 -> 5.5 | spring_length 5 -> 5
MOUSE_BUTTON_WHEEL_UP | follow_distance 5.5 -> 5.4 | spring_length 5 -> 5
MOUSE_BUTTON_WHEEL_UP | follow_distance 5.4 -> 5.3 | spring_length 5 -> 5
MOUSE_BUTTON_WHEEL_UP | follow_distance 5.3 -> 5.2 | spring_length 5 -> 5

Okay. I updated PhantomCamera to the latest and it fixed the problem. Looks like my test project was running a version before https://github.com/ramokz/phantom-camera/commit/140b4a74ab64447dcc1d2144a833ed73e69dea6f

By the way @ramokz, although this "works", is there a reason why this is its own separate function and not a part of the follow_distance property?

It is due to some properties are declared in a PhantomCameraProperties.gd, where they will require a Properties. in front of the property name to be assignable directly. Which means it became a bit of a guesswork for a user as to whether if individual properties needed that or not. In some cases, they didn't even always get assigned in non-addon scripts, while working fine in addon scripts, for no discernible reason. So for the sake of consistency and to reduce the amount of bugs/questions, it has been a setter/getter only approach.
Properties will, however, be assignable directly in 0.7 due to the refactor for how they're set up.


Feat/fe

Features

add getBasisPrice function
add _bondRecords map

Codecov Report
Patch coverage has no change and project coverage change: +1.49 :tada:

Comparison is base (cd07aa6) 94.84% compared to head (e7e61a9) 96.33%.

Additional details and impacted files
@@                  Coverage Diff                   @@
##           feat/deploy-script      #16      +/-   ##
======================================================
+ Coverage               94.84%   96.33%   +1.49%
======================================================
  Files                       6        6
  Lines                     349      355       +6
  Branches                   58       58
======================================================
+ Hits                      331      342      +11
+ Misses                     10        5       -5
  Partials                    8        8

Impacted Files
Coverage Δ

contracts/MangoBondPool.sol
96.62% <0.00%> (+3.56%)
:arrow_up:

... and 2 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


'RuntimeLibrary': value 'MDd_DynamicDebug' doesn't match value 'MTd_StaticDebug'

Describe the bug
I have installed OpenTelemetry using VCPKG. When I build my project via Visual Studio 2022 I am getting the following error:
error LNK2038: mismatch detected for 'RuntimeLibrary': value 'MDd_DynamicDebug' doesn't match value 'MTd_StaticDebug' in ConfigFileParser.obj I've attached the full error logs.
I had installed opentelemetry-cpp:x64-windows which was causing the errors above. I then installed opentelemetry-cpp:x64-windows-static-md  because it appears that the mismatch happens due to the compiler build options but I'm still getting the same errors when I buiild the project from Visual Studio 2022.
Environment

OS: Windows
Compiler: Microsoft Visual C++ compiler

To Reproduce
I am trying to integrate OpenTelemetry to our tool, Log Monitor, https://github.com/microsoft/windows-container-tools/tree/main/LogMonitor/src/LogMonitor
I used the following commands to install OpenTelemetry on my machine via VCPKG
1. vcpkg install opentelemetry-cpp
2. vcpkg install opentelemetry-cpp:x64-windows
3. vcpkg install opentelemetry-cpp:x64-windows-static-md

Expected behavior
Is there something that I'm missing to make Visual Studio pick opentelemetry-cpp:x64-windows-static-md  instead of opentelemetry-cpp:x64-windows so that I can build my project or what do I need to do to address these errors?
Failure logs
otellinkerror.error.txt

Please install the latest build tools in the Visual Studio Installer.

@bobsira Upstream added the use of vcpkg to export dll builds, which conflicts with https://github.com/microsoft/vcpkg/blob/25f613278886a8055aded1bffb14daba98c39d6c/ports/opentelemetry-cpp/portfile.cmake#L2 If the upstream supports dynamic builds, this should be removed.
https://github.com/open-telemetry/opentelemetry-cpp/blob/4daca390e9ca672420a9fad05fb70cac993089d6/CMakeLists.txt#L272

I'm leaving this comment here just incase it might help someone:
To sort the dynamic vs static linking error for me,
I installed the x64-windows-static triplet using  vcpkg install opentelemetry-cpp:x64-windows-static
//and add below to our .vcxproj under
x86-windows-static
x64-windows-static
And had this working.

@FrankXie05 AFAIU your comments were targeted at the library linkage, but the issue is about the CRT linkage.
@bobsira The vcpkg triplet's CRT linkage must match you project's CRT linkage. You started with x64-windows which uses dynamic CRT linkage, as does x64-windows-static-md. If you want to use static CRT linkage, you must use x64-windows-static.
But I don't use MSVS and can't tell you why it didn't work well automatically, assuming proper integration.
https://learn.microsoft.com/en-us/vcpkg/users/buildsystems/msbuild-integration


[BUG] Fix experts implementation

Align with https://github.com/huggingface/transformers/blob/main/src/transformers/models/mixtral/modeling_mixtral.py#L587

#30


removed auto-updater

Removed unused/out-dated auto-updater.
Signed-off-by: French Ben<EMAIL_ADDRESS>

LGTM


xml2rfc does not accept "The Netherlands" as a country name

Describe the issue
Specifying <country>The Netherlands</country> will cause xml2rfc to warn with the following message and not write the country name in the output:
"Warning: Country lookup failed for <country>The Netherlands</country>.
Use --country-help to show recognized country names and codes
 Created file rfc9102.html"

I figure this isn't fixable given that the library used for addresses is not being updated. The workaround is to use "Netherlands" instead.
Code of Conduct

[X] I agree to follow the IETF's Code of Conduct

Note that the wavering discussions on various lists agreed that we probably want to keep the country as the only semantic-markup "postal" element.  So this needs to be fixed.  If there is no library support, Likely, xml2rfc will just accrete a number of special case translations...

The problem is that the addresses are currently fed into an abandonware library for formatting, and it has an internal table that maps country names into address formats. With the change we need to make, <country> becomes a free text field.
I suppose the RPC could keep their own list of allowed country names but I don't see why they would, since you would expect people to know what country they live in.

"Free text field" isn't quite as semantic, but OK.
This just means that the work with normalizing this field is with the people who want to make use of it, not once at publication time.
(I'd rather just have a two-character TLD, but that has other interesting issues.)

@jrlevine The RPC does check country names against the English short names found in the ISO country codes. We have a small list of exceptions. For instance, we prefer "The Netherlands" over "Netherlands (the)".

Interesting. Excerp (since 8650)t:
 248 China
   2 FI
   4 FR
  92 Finland
 205 France
   6 NL
   2 NZ
  40 Netherlands
  17 New Zealand
   2 SE
  94 Sweden
  18 US
  37 USA
   2 United States
1019 United States of America
   8 中国

I don’t suppose we’ve had any authors from The Bahamas, The Gambia, or The Sudan.
Anyway I’d rather delegate the decision to sensible editors than the abandonware library.


Support Gradle config cache in detekt's build

A few improvements to eventually allow the Gradle configuration cache to be used when building detekt.
There are changes required in some Gradle plugins used by the detekt build before this will work:

Kotlin 1.4.30 or higher
Gradle 7.0 (or 6.8.4 which should include a backported fix for an issue)
https://github.com/Kotlin/dokka/issues/1217 to be fixed
https://github.com/Kotlin/binary-compatibility-validator/pull/50 to be merged & released

It's just to fix issues in the build for now. As I mentioned there are third party Gradle plugin updates that are required first, but once they've made fixes and detekt updates those dependencies it should be OK to enable.

@3flex should we have an issue to track the things that we need to do?


Add support for number_format().

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

Project coverage is 97.37%. Comparing base (de4011a) to head (bab6bd6).

Additional details and impacted files
@@             Coverage Diff              @@
##               main      #72      +/-   ##
============================================
+ Coverage     97.33%   97.37%   +0.04%
- Complexity       99      101       +2
============================================
  Files            11       12       +1
  Lines           225      229       +4
============================================
+ Hits            219      223       +4
  Misses            6        6

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


Is it possible to programm a new Layout like NEO 2.0?

I buyed a Kinesis Advantage2. But i use normal NEO 2.0 https://www.neo-layout.org/
is it possible to programm complete NEO 2.0 to Advantage?
Already programm al letters but not the other sighn... Dont know how to do this...

Hi @Tachiguro , I am not able to speak German so I can not help you. But maybe someone else can. Please submit a pull request with the neo layout to this repository. Then if someone knows how to help they can add to your layout.

neo looks like a software installation...  so this may be very hard to achieve on a kinesis.


(node:9136) DeprecationWarning: open() is deprecated in mongoose >= 4.11.0

Forked the project, performed:
npm cache clean
npm install -g npm@latest
npm install
node .
MongoDB shell version v4.2.8
MacOs Mojave
Getting this error when sending an SMS to the number defined on the .env file:
(node:9136) DeprecationWarning: open() is deprecated in mongoose >= 4.11.0, use openUri() instead, or set the useMongoClient option if using connect() or createConnection(). See http://mongoosejs.com/docs/4.x/docs/connections.html#use-mongo-client
(Use node --trace-deprecation ... to show where the warning was created)
Express server listening on *:3000
::ffff:<IP_ADDRESS> - - [15/Jul/2020:15:07:28 +0000] "POST / HTTP/1.1" 404 154 "-" "TwilioProxy/1.1"

disregard - updated the subscriber.js from the updated tutorial and it fixed the issue
https://www.twilio.com/docs/sms/tutorials/marketing-notifications-node-express


Verify addresses UX flow improvements

Overview
The current ledger swapper verify addresses flow is not clear for a user who has not successfully gone through this flow in the past.
All that's displayed is "verify BTC" with no other information. The user is able to click the button without the ledger app open on the device, which does nothing. Until the user figures out they need to open the correct app on the device for that asset, they are stuck.

References and additional details
Current flow
https://github.com/shapeshift/web/assets/125113430/839a2c96-8d42-4d67-b204-4c81b1789459
Acceptance Criteria

App steps the user through verify each address via some sort of stepper
App will disable the button until the correct app is open on the device (we do this for account management so its doable)
App will actually tell the user what they need to do

Need By Date
No response
Screenshots/Mockups
No response
Estimated effort
No response

@woodenfurniture thinking we could leverage the same ACK we would use on trade here.
If user clicks verify [asset], we will pop up the ACK and once we've detected the app is connected, close the ACK and verify the address.
Here is the reference again:

We need to double check that we can verify the app is open on the device and make sure the toasts fire; that seems to be part of the issue too.

Same solution here - https://github.com/shapeshift/web/issues/7339#issuecomment-2237699632

@0xApotheosis to shepherd through


Imperative Backend Bugfix (Init Conditional Problem)

Description
There was an issue with initializing the temporary variable holding the result of a conditional:
      using cond_8_t =
          typename std::common_type<decltype(tupl_7), decltypetupl_7)>::type;
      cond_8_t cond_8 = 0.;
      ^^^^^^^^^^^^^^^^^^^^^
      can not initialize tuple with double

Instead of initializing with 0., the behavior was changed to use uniformed initialization with {}. In the above example this would read
cond_2_t cond_2 = {};

I decided I like your version better


Showing "種類を下スマート カード ログオン] * * 名前。" (extra asterisks appears)

https://docs.microsoft.com/ja-jp/windows/security/identity-protection/hello-for-business/hello-hybrid-aadj-sso-cert#prepare-active-directory-certificate-authority

ドキュメントの詳細
⚠ このセクションを編集しないでください。 docs.microsoft.com で必須です ➟ GitHub の問題のリンク。

ID: 9f709fd4-0b5d-b238-4060-222d550cd538
Version Independent ID: fb3f29da-2236-c47d-6bea-8efbcec8d7d3
Content: AADJ オンプレミス シングル サインオンのシングル サインオンの証明書の使用
Content Source: windows/security/identity-protection/hello-for-business/hello-hybrid-aadj-sso-cert.md
Product: w10
Technology: windows
GitHub Login: @mapalko
Microsoft Alias: mapalko

@officedocsbot assign @e0i
@officedocsbot assign @dongmicro
@officedocsbot add label localization

@dongmicro Would you follow-up with this localization issue?
Similar issues:
#3159  #3394 #3452 #3451

@officedocsbot assign @MollyMalone

@officedocsbot unassign @MollyMalone
@officedocsbot assign @TinaMcN

I haven't quite figured out why the asterisks appear as they do in the Japanese version, but looking at the English (en-us) version of the page, it seems they need to be there as part of the explanatory text. I think this is exactly the same type of issue as in #3451 & #3452 , but I need to check it again and verify this before I can confirm that this is the case.
Excerpt from the en-us page version:

Verify the onPremisesDistinguishedName attribute is synchronized
The easiest way to verify the onPremisesDistingushedNamne attribute is synchronized is to use Azure AD Graph Explorer.

Open a web browser and navigate to https://graphexplorer.azurewebsites.net/

Click Login and provide Azure credentials

In the Azure AD Graph Explorer URL, type https://graph.windows.net/myorganization/users/[userid], where **[userid] is the user principal name of user in Azure Active Directory.  Click Go

In the returned results, review the JSON data for the onPremisesDistinguishedName attribute.  Ensure the attribute has a value and the value is accurate for the given user.

I have commented more in the English issue ticket #4589 (Missing step to set the Subject Alternative Name in Create a SCEP Certificate Profile section?), asking if the asterisks are needed at all or if they can be removed. This topic is unfortunately not within my experience or work area, so I have to rely on feedback from users who know this material to some extent already. Will post back here again as soon as there is a confirmed answer, one way or the other.

@officedocsbot unassign @dongmicro

@TinaMcN : As you can see, my PR #4626 to correct the bold marking in the page section https://docs.microsoft.com/en-us/windows/security/identity-protection/hello-for-business/hello-hybrid-aadj-sso-cert#create-and-assign-a-simple-certificate-enrollment-protocol-scep-certificate-profile has been merged into the source for the en-us version of the page. I can only suggest that you follow the development after the docs.microsoft.com page is updated to see when the next automated translation script procedure runs, to see if it solves this particular issue (and maybe one of the other related issues).
I was only able to find 1 more rogue asterisk ( * ) in the en-us version of this particular page (not sure if any issue is opened for that one on the ja-jp page), so I will keep looking around to see if I can locate any other pages where this particular type of issue happens because of a mix of HTML and MarkDown code inadvertently placed wrongly, causing asterisks to be reported from the Japanese pages.

Manually verified to be fixed.

Thanks for checking @e0i Glad to see this one is fixed.


No Data displayed

Describe the bug
I've setup up everything as in the get Started guide:
No error is thrown (not in browser console and not in vscode)
The grid is shown on the page but contains no data even i can see that the json data is passed to the view from the controller
To Reproduce
Controller:
[HttpPost]
        public JsonResult GetUsers(DataRequest request) {
            DataResult<AppUser> result = _userManager.Users.ToDataResult(request);
            return Json(result);
        }

View:
@(Html.JS().Datatable<AppUser>()
            .Name("PersonGrid")
            .Columns(cols =>
            {
                cols.Field(a=>a.Id).Visible(true);
            })
            .URL(Url.Action("GetUsers"), "POST")
            .ServerSide(true)
            .Render()
        )

Expected behavior
as there is no error thrown i would expect that the data is displayed.
Screenshots

Desktop (please complete the following information):

OS: MacOs Ventura
Browser chrome
Version [106.0.5249.119 ]

Additional context
Add any other context about the problem here.

Hi @madcoda9000
Are your properties Pascal-Case and data is Camel-Case I think,
Can you please check your serialization settings?

hmmm, that is courious. Because the object (from what i use the rpoperties) is inherited from IdentityUser (from identity Framework). So there is no chance for me change or rename the properties.
The json data passed back is generated by ToDataResult. So i don't know what i could change here...

Hi @madcoda9000 ,
It is not necessary to use Pascal-Case but it is an obligation that JSON and entity properties must be the same. The identity uses Pascal-Case I think.
Look at this please and don't hesitate to return back:
https://stackoverflow.com/questions/58392039/how-to-set-json-serializer-settings-in-asp-net-core-3

Hello @ekondur ,
many thanks for your help. I've set the options in program.cs like that now:
builder.Services.AddControllersWithViews().AddJsonOptions(options =>
{
    options.JsonSerializerOptions.PropertyNamingPolicy = null;
});

And now the returned json property names the same as the identity properties and the data is displayed in the datatable.
I just wanted to mention, that if i use vanilla "datatable js" it is only working without setting the serializer options. That means for the vanialla datatables js it is neccessary that all properties starts lowercase (which is documented in their project). So it seems that it is a standard that the json serializer returns properties starting lowercase. But with vanilla datatables js i am not able to use the html helpers, which is sad because i've written a lot :-)
Anyway. it is working now and I really appreciate your help.
Wish you All the best
Sascha

Hi, @madcoda9000
Thank you for your kind comments and useful suggestion.
Can you please try == instead of ===
Regards.

Hello @ekondur ,
thanks for your reply. That did not work.
This:

And this is the output:

Best Regards
Sascha

Hi @ekondur ,
me again (sorry for that). In the meantime i did a little research by myself.
I did the following:

remove the nuget packages to  DataTableJs and DataTableJs.Data from my Project
cloned your Repository locally
made the following changes to JsHelper.cs

build the DataTable projects
added Assembly References to my locally builded Datatable dll's

aaaand surpriiiiise (it's working now)

Best Regards
Sascha

Thank you so much @madcoda9000 ,
I will check the problem and release a new nuget version if we need.
Regards.

nothing to thank. I have to thank for this piece of software. Keep up that work!

Hi @madcoda9000
You can use it from now on,
.URL(Url.Action("GetDataResult"), "POST", camelCase: true)

Set the camelCase property to true.
It is released with version 3.4.1


A environment bug

I find a bug when I implement from torch_geometric import voxel_grid.

which version do you install?

I install requirement.txt,and found this bug(torch_geometric=1.7.0)

Can you try to reinstall with the alternative cammands?
pip install torch_sparse==0.6.12
pip install torch_points3d==1.3.0
pip install tensorboard timm termcolor tensorboardX

I try it,but find a new bug.Maybe numpy version is bug?

I haven't met that before. May I ask which version of numpy do you use?

@xsf777  excuse me, i also I also encountered the problem that the version of numpy didn't match .Have you solved it?

@xsf777 @wuzheaa Excuse me, thank you for your kindness.！Can you tell me if the issue of the numpy version being wrong has been resolved? And what is the solution?

sorry, I don't remeber have I solved this problem. It's too long.

路*/客
@.***

------------------ 原始邮件 ------------------
发件人: @.>;
发送时间: 2023年11月11日(星期六) 上午10:57
收件人: @.>;
抄送: @.>; @.>;
主题: Re: [dvlab-research/Stratified-Transformer] A environment bug (Issue #36)
@xsf777 @wuzheaa Excuse me, thank you for your kindness.！Can you tell me if the issue of the numpy version being wrong has been resolved? And what is the solution?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>


Updated README.md

This PR updates the README to be less confusing.
The old README said:

you need to add it to components.js

This, however, is not what contributors have to do. New languages have to be added to components.json and the project has to be rebuild using gulp.

@mAAdhaTTah Are we going to merge this?


Update aws-java-sdk-kinesis, ... to 1.11.749

Updates

com.amazonaws:aws-java-sdk-kinesis
com.amazonaws:aws-java-sdk-s3
com.amazonaws:aws-java-sdk-sqs

from 1.11.745 to 1.11.749.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.amazonaws" } ]

labels: library-update, semver-patch

Codecov Report

Merging #235 into master will not change coverage by %.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #235   +/-   ##
=======================================
  Coverage   47.68%   47.68%
=======================================
  Files          24       24
  Lines         281      281
  Branches        1        1
=======================================
  Hits          134      134
  Misses        147      147

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 caefbcb...27cc719. Read the comment docs.


use aliases to make searching and filtering easier

should they be in their own table?
right now, filtering on sort name is only supported for artists and areas

This depends on the API providing this to us.
artist lookup returns one sort-name
The rest do not return any

We need to use the query API, not lookup in order to get aliases. We can ensure we get exactly one result by passing the mbid as the query.
e.g. https://musicbrainz.org/ws/2/event/?query=eid%3A10b5e28b-95f5-482e-9524-c51be37f943d
{
	"created": "2024-12-28T01:06:17.300Z",
	"count": 1,
	"offset": 0,
	"events": [{
		"id": "10b5e28b-95f5-482e-9524-c51be37f943d",
		"type": "Convention/Expo",
		"score": 100,
		"name": "コミックマーケット65",
		"life-span": {
			"begin": "2003-12-28",
			"end": "2003-12-30"
		},
		"aliases": [{
			"sort-name": "Comic Market 65",
			"name": "Comic Market 65",
			"locale": "en",
			"type": null,
			"primary": true,
			"begin-date": null,
			"end-date": null
		}, {
			"sort-name": "Comiket 65",
			"name": "Comiket 65",
			"locale": "en",
			"type": null,
			"primary": null,
			"begin-date": null,
			"end-date": null
		}, {
			"sort-name": "コミックマーケット65",
			"name": "コミックマーケット65",
			"locale": "ja",
			"type": null,
			"primary": true,
			"begin-date": null,
			"end-date": null
		}, {
			"sort-name": "C65",
			"name": "C65",
			"locale": null,
			"type": null,
			"primary": null,
			"begin-date": null,
			"end-date": null
		}],
		"tags": [{
			"count": 1,
			"name": "doujin"
		}, {
			"count": 1,
			"name": "c65"
		}],
		"relations": [{
			"type": "held at",
			"type-id": "e2c6f697-07dc-38b1-be0b-83d740165532",
			"direction": "backward",
			"place": {
				"id": "a2076891-8d06-42f0-b44e-033f59e7cad4",
				"name": "東京国際展示場"
			}
		}]
	}]
}


Bug bash to find samples for Service Bus that need to use the latest package

Update from @ramya-rao-a
Review the Python samples for Service Bus in the below locations to find the ones that do not use the latest version of the Service Bus package. Please note that some of these maybe using the preview version of the latest Service Bus package, these need to be identified as well. Update this issue with the list of samples that need updating.

Service Bus Python samples repos under Azure-samples org
Service Bus Python  samples under docs.microsoft.com/samples

Additionally, ensure the samples that live next to the source code for Service Bus in this repo shows up in Service Bus Python samples under docs.microsoft.com/samples. This is done by setting up the right metadata. Ensure that these samples are not outdated.

https://microsoft.sharepoint.com/teams/AzureDeveloperExperience/_layouts/15/Doc.aspx?sourcedoc={41b0af1e-8da3-4d88-b296-9b616c3e7b6b}&action=edit&wd=target(Docs bug bash.one|687ba2d4-5f1a-4990-8625-ba7ad5df6dae%2FPython docs bug bash|35edca88-7068-43c2-b36c-1b5d77ed8bbb%2F)&wdorigin=703
Here are the findings

the samples read me has --pre flag in the pip install instructions.
-All the samples look good

The note on preview from samples readme is being removed in https://github.com/Azure/azure-sdk-for-python/pull/15881
Thanks @rakshith91!


Bring over BatchFixAllProvider tests from Roslyn

The CustomBatchFixAllProvider added in #1122 is nearly identical to the BatchFixAllProvider in Roslyn. The code which tests BatchFixAllProvider in Roslyn could be included in this project as well, providing improved coverage for any modifications we make to this class over time.

From the sound of it, the tests we use for fix all providers in this project might actually be more extensive than the reference tests. :no_mouth:


Samwelkanda/update python json serialization

Fixes bugs from end to end tests
Fixes installation issues by adding dependencies to project toml file

@samwelkanda as far as I can tell a lot of the content here overlaps with #1289, could we either deduplicate on the different PR or group the PRs in one?


Fix repo url decoding

What this PR does / why we need it:
Fixes #6048
Special notes for your reviewer:
To test better - I added mocks, and a memory based filesystem so that nothing gets written to the actual file system
If applicable:

[ ] this PR contains documentation
[x] this PR contains unit tests
[ ] this PR has been tested for backwards compatibility

Once this is merged, I will try to port the same logic into release-2.14 and dev-v3 branches

pkg/urlutil/urlutil_test.go:1::warning: file is not gofmted with -s (gofmt)

@karuppiah7890 generally speaking we only cut patch releases for the latest (in this case, 2.14.x). We don’t usually cut patch releases for older revisions. We also don’t cherry-pick commits into the release branch until we’re ready to cut a patch. Given this is a low severity issue that didn’t previously work, we probably won’t cut a patch release with this fix. I don’t believe this was a regression in older behaviour, just an enhancement that fixes odd behaviour.
Would you mind re-opening this against master instead? We can cherry-pick this in should we decide to do a 2.14 patch release. Thanks!

Sure, I'll raise it against master. Also, it's weird that the fmt error occurred, in local it occured and I fixed it in one of the commits if you see the list of commits. Anyways, let me raise with master and see if it's still a thing

Closing this as I'm going to raise a PR against master^

The new PR is here - #6060