Validate PX4 SITL for ROS, add 3D sensor

@vilhjalmur89 Could you please validate the instructions here on Monday:
http://dev.px4.io/simulation-ros-interface.html
And report here if anything doesn't work?

It seems to work

Ok -can you move your workflow to it?

Closing, please reopen if still needed.


cmd/gofmt: inconsistent spacing on slice indices

What version of Go are you using (go version)? go version go1.4.2 linux/amd64
What operating system and processor architecture are you using? Ubuntu 14.04
What did you do? Run gofmt on below code examples
What did you expect to see? Consistent spacing of slice indices
These are two lines within the same function. They vary slightly, but the interesting part is the [1:len(key-1] which has inconsistent spacing. Adding spaces to the first one will result in them being removed when gofmt is run, and removing the spaces from the second will only have them added back.
If this is not a bug, please let me know what difference here is causing the spacing to change.
return "?" + key[1:len(key)-1]

key = key[1 : len(key)-1]

It's working as intended. There's a good explanation here.

We like regularity too, but we like readability more,
and the few breaks from regularity in gofmt are for
things that we felt, after over a year of writing
Go programs, enhanced readability enough to justify
breaking from the simpler rules.
In this specific case, 1<<x + y is much clearer as
to its meaning than 1 << x + y, because it emphasizes
the precedence.
Your example -  "x && rand.Int()%RAND_FACT" - is not
the whole story: you lifted a fragment.  The real expression
would have been something like
x && rand.Int()%RAND_FACT != 0

since a % cannot produce a bool and && requires one.
If the op were & instead of %, like in
x && rand.Int()&RAND_MASK != 0

then I appreciate the visual reinforcement that it's being
parsed the way I intend (i.e. it has a different meaning
here than it would in C).
Especially because operator precedence is a notorious
source of confusion in C and because Go changed
the precedences to simplify things, we think this is an
important way to help programmers adjust.

Hmm, I understand that there may be times where one formatting is better than another. But in this case the core piece of the line is exactly the same in both cases. Is there a more specific reasoning that causes this particular difference?

@tgrosinger You are correct - they should format the same:

The spacing changes when going from http://play.golang.org/p/ToSuqHfqWu to http://play.golang.org/p/rVyw-A9om1 - the difference is just the added parentheses around the slice expression. However, the slice expression already introduces a form of parentheses (square brackets), so inside there shouldn't be a change.

There are long-standing TODOs for exactly this issue in go/printer:776 (also line 769).

The fix is trivial, but in the interest of not introducing lots of changes over huge amounts of code, we haven't pushed it through, and I suggest leaving it as is for now.
That said, I've been working on a new version of gofmt (long overdue, hopefully after 1.5 I can concentrate on finishing it) which should solve many of the open gofmt issues. This may be a good time to address this one as well.

Good to know, thank you.


Faq implementation

Pull Request: [FEAT] - Create FAQ Endpoint #298
Description
Added functionality to create a new FAQ in the Help Center module. This feature allows administrators to add frequently asked questions with optional categories and tags.
Endpoints

POST api/v1/admin/faqs

Request Body
{
  "question": "string",
  "answer": "string",
  "category": "string",
  "tags": ["string"]
}

Response Format

Create FAQ (POST /faqs){
  "status_code": 201,
  "success": true,
  "data": {
    "id": "String",
    "question": "String",
    "answer": "String",
    "category": "String",
    "tags": ["String"],
    "createdBy": "String",
    "createdAt": "String"
  }
}

Manual Testing

Create a New FAQ

Action: Send a POST request to the POST /faqs endpoint with a valid CreateFaqDto payload.
Expected Result: You should receive a response with status code 201 and the created FAQ object.

Example Request:
curl -X POST http://localhost:3000/faqs \
-H "Content-Type: application/json" \
-d '{
  "question": "What is the return policy?",
  "answer": "Our return policy allows returns within 30 days of purchase.",
  "category": "Policies",
  "tags": ["return", "policy", "purchase"]
}'

Invalid Input Data

Action: Send a POST request to the POST /faqs endpoint with invalid data.
Expected Result: You should receive a response with status code 400 and an appropriate error message.

Checklist

[x] Implemented the endpoint to create a new FAQ.
[x] Added validation for required fields and data types.
[x] Verified that all optional fields are handled correctly.
[x] Ensured proper error handling and logging for invalid input.

Reference Issue
[FEAT] Create an endpoint for Admin to Create FAQs

@Heba-WebDev
@Homoakin619
@Cobidex
here is the PR for FAQ please review


group: do not render gap for empty help view

(*Group).View renders enabled keybindings in a footer view according to the current (Field).KeyBinds(). This footer view is offset vertically by at least one unconditional (Theme).FieldSeparator.
This is fine in most cases as the user will usually have key bindings returned by (Field).KeyBinds() for every Field – and indeed every package-provided Field returns something meaningful when they do.
However, if the user wishes to not render this footer view in a custom Field (by returning a nil or empty slice from (Field).KeyBinds() and using the KeyMap directly in Update), then the renderer can get confused about the intended output height.
This PR adds a condition on rendering the footer view and its (Theme).FieldSeparator gap to ensure they contain more than just white space when combined together.

Yikes, rebased against the wrong remote. Need to rebase again to squash all these changes.

Hey @ardnew, let me know if you're able to rebase / open another PR for this one!

Man, I'm having a lot of trouble fixing this headache. I may close this PR and open another one with a much cleaner patch. Is this okay with you? @maaslalani

Of course! @ardnew

Oh, just re-read your previous comment suggesting I open another PR as an option. I'm gonna take that route. :wink:


pull request for e2e test at 02/05/2017 06:15:07

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Initial cypress setup

Initial set up of https://github.com/ethyca/fides/issues/713
Code Changes

[x] Install cypress as a dependency
[x] Install cypress eslint as a dependency
[x] Configure Cypress for our app
[x] Simple test to show things working
[x] Configure CI

Steps to Confirm

[x] Run nox -s dev
[x] In clients/admin-ui run npm run cy:open to see the cypress UI do its thing.

Pre-Merge Checklist

[x] All CI Pipelines Succeeded
Documentation Updated:

[ ] documentation complete, or draft/outline provided (tag docs-team to complete/review on this branch)
[ ] documentation issue created (tag docs-team to complete issue separately)

[ ] Issue Requirements are Met
[ ] Relevant Follow-Up Issues Created
[x] Update CHANGELOG.md

Description Of Changes
Cypress can do full end to end tests to make sure that our frontend and backend are communicating properly. However, the full e2e would be a lot more to set up, as we would have to get the backend running and populated with sample data as well. Cypress recommends separating e2e from other tests, where e2e test really critical paths (i.e. login), and other than that, using mocks to manipulate data as needed. So this PR does not set up e2e, but does get Cypress running on the frontend such that we can mock out API calls and do a lot more robust frontend testing 🎉

I did a rebuild with nox -s dev and then tried npm run cy:run from the UI container. It gives me some missing dependency errors. I think we need to manually install these in our our docker image build to make the tests runnable locally -- I'm guessing you did that in your image while setting this up?
I suppose the github action works because it uses a cypress image?
root@eef9474c7203:/fides/clients/admin-ui# npm run cy:run

> cy:run
> cypress run

It looks like this is your first time using Cypress: 10.3.0

Your system is missing the dependency: Xvfb

Install Xvfb and run Cypress again.

Read our documentation on dependencies for more information:

https://on.cypress.io/required-dependencies

If you are using Docker, we provide containers with all required dependencies installed.

----------

Error: spawn Xvfb ENOENT

----------

Platform: linux-arm64 (Debian - 10.12)
Cypress Version: 10.3.0

I did a rebuild with nox -s dev and then tried npm run cy:run from the UI container.

Rereading your steps to confirm, running the tests from the host machine (mac) I think is the approach you took. That's good too, in which case I had to npx cypress install after the dev setup. I don't think there's a convenient way to script that, in which case it could be documented in the README.

I did a rebuild with nox -s dev and then tried npm run cy:run from the UI container.

Rereading your steps to confirm, running the tests from the host machine (mac) I think is the approach you took. That's good too, in which case I had to npx cypress install after the dev setup. I don't think there's a convenient way to script that, in which case it could be documented in the README.

Oh yup, I've been running from the host machine, and just running npm i whenever dependencies get updated (cypress should be in the package.json under dev dependencies). I'd rather not run cypress within the dockerfile since it probably needs all sorts of stuff (chrome, electron, etc) that would be sad for our image to have. I'll update the README though to be clearer about that

I'd rather not run cypress within the dockerfile since it probably needs all sorts of stuff (chrome, electron, etc)

I got the headless mode to work: https://github.com/ethyca/fides/compare/aking-713-cypress-setup...ssangervasi/713/cypress-setup
Still happy to run from the host machine though.

I'd rather not run cypress within the dockerfile since it probably needs all sorts of stuff (chrome, electron, etc)

I got the headless mode to work: aking-713-cypress-setup...ssangervasi/713/cypress-setup
Still happy to run from the host machine though.

Hey nice!! I've always run frontend stuff outside of Docker, so I'm not sure what is best practice here. I'm pretty sure @ThomasLaPiana wants to dockerize everything, but I suspect adding all those packages just for Cypress might make our image bloated? What are your thoughts?

I'd rather not run cypress within the dockerfile since it probably needs all sorts of stuff (chrome, electron, etc)

I got the headless mode to work: aking-713-cypress-setup...ssangervasi/713/cypress-setup
Still happy to run from the host machine though.

Hey nice!! I've always run frontend stuff outside of Docker, so I'm not sure what is best practice here. I'm pretty sure @ThomasLaPiana wants to dockerize everything, but I suspect adding all those packages just for Cypress might make our image bloated? What are your thoughts?

Going to go ahead and merge this, and if we want to docker-ify Cypress we can do it in another ticket. After thinking about it some more, it probably makes sense, if we do want to go forward with it, to just use the Cypress image as a totally separate image from our own Dockerfile (or if needed, extend their image)


skip files and folder with same name

, while scanning large library if there are two folders with same name it stops, it there an custom arguments to skip folder if already exist.

As for "skip files and folders with same name" - I'm not really sure what you're talking about. This project will not auto-update, or watch for changes on the drive folder. As such, whenever you use this script, it will intentionally iterate through each directory - and if the target directory already exists, the script will delete it.
Regarding "if there are two folders with same name it stops". Yes, if your drive has two folders/files with the same name together, the script will run into an error. This is intentional behaviour.
Your suggestion to add a custom argument to suppress this is possible. I'll probably implement this in a future update. Possibly a --suppress-errors flag or something similar. Thanks for the suggestion :)


[Az] Use stable apiversion for azure resource calls in registration script, fix Az token for AzureUSGovernment

@keystroke please review https://github.com/Azure/AzureStack-Tools/pull/611/commits/d1440ea6399d1fa018ec295d1ea93dfee37b9f85


How to add v4 printer driver in PrinterExtensionSample code?

How to add v4 printer files (.ini or .inf) in the example of PrinterExtensionSample code provided here? Anyone can help?
@barrygolden @jmrossy @wm1

Hello @mnomanmemon, did you happen to find the answer to this question? since I have the same doubt and I see you got no answer from those two "quite resonsive" forums.


Navbar - Create Locale Menu Dropdown

Requirements

Create a new vue dropdown component and integrate into the TopNav vue component
Using vue best practices, wire up logic for opening and closing the menu based on local vue state.
You can use English and Japanese as temporary, hard-coded options in the menu for this PR.
UI should be responsive for when page changes width and should show in the hamburger menu

consider https://vuejsexamples.com/tag/dropdown/ as ideas. No need to wire to any data yet


Example Scripts

Category
[ ] Bug
[X] Enhancement
Examples to add
Add more sample scripts for transformation

Publishing Pages Examples
Publishing Pages Examples with logging
Processing library of pages

All based on PnP PowerShell

@jansenbe we talked about adding example scripts - what is needed and I can contribute to those? Feel free to assign to me.

@pkbullock : as you're using PS scripts to perform transformation it would be good to share some of your more advanced scripts in https://github.com/SharePoint/sp-dev-modernization/tree/dev/Scripts/PageTransformation. Today there's a very basic example, but for example having a sample script that shows "real world" usage would be super useful for folks

Closing, thanks for your contribution @pkbullock


New content: Proposed Documentation on Headings and Elements Nesting in Rich Text Field

@ArafatAbdussalam please make sure you understand what you’re working on. What you did in #338 isn’t a good use of time for anyone – we can’t have documentation so heavily based on screenshots, and this point you’re reporting on is specifically about _ rich text fields_.

Alright @thibaudcolas thank you for the feedback


Russia 1992 generator issues

VictorWeikum
Too many socialists for Russia of 1992, where people were mostly Liberals or Liberal Socialists.
Russian armies have attrition at 1992 start, player always have to split them, which is annoying
Russian army placed at chineese province

Pops ideologies are generic, but will change the initial political composition so that Liberals have more power in Parliament.


Use C++11-ish way of getting numeric limits

INT32_MAX is not defined in some platforms.

@bhousel stumbled upon this one on the Qt CI.

Can we also make a new release? I will need to bump the version used by mapbox-gl-native.

Thanks for the catch!

Can we also make a new release?

Sure I'll do this now..


how can i run the index.html from a tomcat or python server ?

It gives me the path error for bower components.

There is not enough context to help and even if there would, support questions like those should be asked on StackOverflow or a similar support forum....


CLI Testing

Describe what should be investigated or refactored
Currently we have no tests to evaluate the CLI commands and flags.
Links to any relevant code
None
Additional context
It would be good to evaluate and implement Lula CLI testing to ensure a robust CLI experience for users and better methods for developers to evaluate how changes impact the system.

Current progress is retroactively adding tests in src/test/e2e as executed through the CLI commands themselves. Closing this with intent to file specific issues for areas that need to be cleaned up.


#internal/nitro/virtual/app-config is not defined in package

Hi there, same issue as https://github.com/nuxt/icon/issues/204 with this package. Fix seems to be here: https://github.com/nuxt/icon/pull/249

Not very familiar with nitro, but looking af the PR fixing nuxt icon, it seems like the imports in these files needs to be changed from nitropack/runtime to #imports. Sorry for not being able to push this further!

Just realized that nuxt-content was a dependency of this, so the issue could just come from them: https://github.com/nuxt/content/issues/2736 https://github.com/nuxt/content/issues/2736

I also have this problem as soon as adding @nuxtjs/sitemap as a nuxt module.
i went through every nitropack import and replaced with #imports ... and this didn't fix it.
So I went through every dependency of sitemap that has a nitropack import and replaced with #imports and this did fixed it.
Not sure "which" change helped or if all were required to fix my problem.
sitemap
robots
nuxt-site-config
devtools-kit
@nuxt/kit
What's the best way to prevent doing this?
bash makes this easy but where's the root cause of this?

@harlan-zw I'd really appreciate if you could take a look

Most likely related to the Site Config module trying to read config from the app config when it's not enabled for some reason.
If you have a reliable way to replicate it, it would help. I've never personally come across this.

Works on github codespaces, not on several Macos computer we tested

I use linux :/

I'm not able to replicate using Linux Mint.
v3 of the Nuxt Site Config module drops support for app config which may solve this, I'll have all of the modules updated to use this version shortly.


remove duplicate display

already applied in button component

Should fix https://github.com/Dogfalo/materialize/pull/4496

Waves is not only meant for buttons so removing this will probably have negative consequences when placed on other elements.

Waves is not only meant for buttons

not only or only?

Not only. You can use waves on images, nav links, and really anything you want. Its easy to tunnel vision on testing a fix for a specific bug but as a framework this needs to be flexible.

http://codepen.io/anon/pen/XMwWKN
Or do we need more tests?

You can use waves on images, nav links, and really anything you want.

Is this somewhere mentioned?
http://materializecss.com/waves.html

I still think we are misunderstanding each other. Waves can be placed on elements that are not buttons.

I still think we are misunderstanding each other. Waves can be placed on elements that are not buttons.

I understood this. Then we should change the paragraph in the docs to reflect this. This is what I wanted to tell you ;-)
Anyway, so far the proposed change does not break anything in the codepen. See https://github.com/Dogfalo/materialize/pull/4496#issuecomment-292034036

Well your codepen only tested buttons. I feel like the bug this is solving is minor, and the bugs it can cause could be more major. For now there is an easy 1 line css fix to the bug so I'd rather not remove the display globally until it is more thoroughly tested

Sure. Please let me know if you can provide some more tests / examples. I will try to find some. Would be great to have some more input on this cc @tomscholz @fega

Bump.


Update quill-jdbc, quill-jdbc-zio to 3.13.0

Updates

io.getquill:quill-jdbc
io.getquill:quill-jdbc-zio

from 3.10.0 to 3.13.0.
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.
Configure Scala Steward for your repository with a .scala-steward.conf file.
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 = "io.getquill" } ]

labels: library-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #263.


ADF Dataflow Pagination Examples are Outdated

Hi there,
https://learn.microsoft.com/en-us/azure/data-factory/connector-rest?tabs=data-factory#pagination-support
This part of the article states that you can do all of these things with pagination rules, however the UI has changed and it is difficult if not impossible to replicate the examples.
In particular, I've really wanted to be able to follow the following pattern:
"Next request’s query parameter = property value in current response body."
I cannot figure this out.
Best,
Ryan

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

ID: c9da1816-3f64-bd45-443f-5bdb48bacc24
Version Independent ID: e649f787-62da-fc97-2e9a-ba89765beed4
Content: Copy and transform data from and to a REST endpoint - Azure Data Factory & Azure Synapse
Content Source: articles/data-factory/connector-rest.md
Service: data-factory
Sub-service: data-movement
GitHub Login: @jianleishen
Microsoft Alias: makromer

@rruenroeng
Thanks for your feedback! We will investigate and update as appropriate.

@rruenroeng
I've delegated this to @jianleishen, a content author, to review and share their valuable insights.

Thanks @rruenroeng for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. The requested updates have not been made since the creation of this issue, and the timeline for resolution may vary based on resourcing, so we've created an internal work item to incorporate your suggestions. We are closing this issue for now, but feel free to comment here as necessary. #please-close

:/
On Fri, Jan 19, 2024, 14:53 prmerger-automator[bot] <
@.***> wrote:

Closed #112203 https://github.com/MicrosoftDocs/azure-docs/issues/112203
as completed.
—
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/112203#event-11542089803,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEA3YKE45RMAXDZRS5S6PWLYPLMN3AVCNFSM6AAAAAA2JYFDG6VHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJRGU2DEMBYHE4DAMY
.
You are receiving this because you were mentioned.Message ID:
@.***>


Feature: openURL using a proxy?

One of our next measurement campaigns will be in a region with a very small network bandwidth.
We are thinking about using a local proxy to cache all images requests from our servers.
Have you thought on adding a proxy parameter? Or should we do this on our library by a prepared request?

See PR #373 and the discussion.


LintError - Extend the schema with custom resolvers

Is there an existing issue for this?

[X] I have searched the existing issues

Which topic?
https://developer.adobe.com/graphql-mesh-gateway/gateway/extending-unified-schema/
What's wrong with the content?
We followed this document (https://developer.adobe.com/graphql-mesh-gateway/gateway/extending-unified-schema/) to create custom resolvers.
However, when we copy the meshConfig content into the mesh.json file and copy the additional-resolvers content into the additional-resolvers.js file. We then run the update mesh command
aio api-mesh:update mesh.json
Error occurred:
{"message":"LintError in File ./additional-resolvers.js: Empty block statement., requestId: b74d1fb5-dae4-444c-bfae-03bb773aa6bf","statusCode":400}  ›   Error: Unable to update the mesh. Please check the mesh configuration file and try again. If the error persists please contact support. RequestId: 430d3fac-32bd-4669-b0c1-d508d35c4cfa
It seems that the additional-resolvers.js file content is incorrect
What changes do you propose?
Additional-resolvers.js file content may need to be adjusted
Anything else that can help to cover this?

@haunguyenlxag I have just tested this file and it works correctly. My assumption, based on your description, is that you removed the file from the mesh. There should not be a separate additional-resolvers.js file. The file is wholly contained within the mesh file as demonstrated in the docs. Please try again using the following mesh without modifying it:
{
    "meshConfig": {
      "sources": [
        {
          "name": "Venia",
          "handler": {
            "graphql": {
              "endpoint": "https://venia.magento.com/graphql"
            }
          }
        },
        {
          "name": "DiscountsAPI",
          "handler": {
            "JsonSchema": {
              "baseUrl": "https://raw.githubusercontent.com/AdobeDocs/graphql-mesh-gateway/main/src/pages/_examples/discounts-api.json",
              "operations": [
                {
                  "type": "Query",
                  "field": "discounts",
                  "path": "/",
                  "method": "GET",
                  "responseSample": "https://raw.githubusercontent.com/AdobeDocs/graphql-mesh-gateway/main/src/pages/_examples/discounts-api.json"
                }
              ]
            }
          }
        }
      ],
      "additionalResolvers": [
        "./additional-resolvers.js"
      ],
      "files": [
        {
          "path": "./additional-resolvers.js",
          "content": "\r\nmodule.exports = {\r\n\tresolvers: {\r\n\t\tConfigurableProduct: {\r\n\t\t\tspecial_price: {\r\n\t\t\t\tselectionSet: \"{ name price_range { maximum_price { final_price { value } } } }\",\r\n\t\t\t\tresolve: (root, args, context, info) => {\r\n\t\t\t\t\tlet max = 0;\r\n\r\n\t\t\t\t\ttry {\r\n\t\t\t\t\t\tmax = root.price_range.maximum_price.final_price.value;\r\n\t\t\t\t\t} catch (e) {\r\n\t\t\t\t\t\t// ignore\r\n\t\t\t\t\t}\r\n\r\n\t\t\t\t\treturn context.DiscountsAPI.Query.discounts(\r\n\t\t\t\t\t\t{ root, args, context, info, selectionSet: \"{ name discount }\" }\r\n\t\t\t\t\t)\r\n\t\t\t\t\t\t.then((response) => {\r\n\t\t\t\t\t\t\tconst discountConfig = response.find((discount) => discount.name === root.name);\r\n\r\n\t\t\t\t\t\t\tif (discountConfig) {\r\n\t\t\t\t\t\t\t\treturn max * ((100 - discountConfig.discount) / 100);\r\n\t\t\t\t\t\t\t} else {\r\n\t\t\t\t\t\t\t\treturn max\r\n\t\t\t\t\t\t\t}\r\n\t\t\t\t\t\t})\r\n\t\t\t\t\t\t.catch(() => {\r\n\t\t\t\t\t\t\treturn null;\r\n\t\t\t\t\t\t});\r\n\t\t\t\t},\r\n\t\t\t},\r\n\t\t},\r\n\t},\r\n};\r\n"
        }
      ]
    }
  }

hi @jhadobe,
Thank you for your feedback,
After I delete the additional-resolvers.js file locally and copy the mesh content you sent into the mesh.json file. The error no longer appears, the mesh has been updated successfully.
However, we discovered that special_price doesn't seem to be set correctly, it's always null when I check 2 products "Roxana Cropped Sweater" and "Hanna Sweater".
I guess it hasn't run into additionalResolvers
You can see the picture below

However, we discovered that special_price doesn't seem to be set correctly, it's always null when I check 2 products "Roxana Cropped Sweater" and "Hanna Sweater".

Thanks, @haunguyenlxag. I was able to reproduce this locally. I am checking with the dev team and will get back to you when they have finished investigating.

@haunguyenlxag It looks like the path needs to be updated to the following:
{
  "meshConfig": {
    "sources": [
      {
        "name": "Venia",
        "handler": {
          "graphql": {
            "endpoint": "https://venia.magento.com/graphql"
          }
        }
      },
      {
        "name": "DiscountsAPI",
        "handler": {
          "JsonSchema": {
            "baseUrl": "https://raw.githubusercontent.com/AdobeDocs/graphql-mesh-gateway/main/src/pages/_examples",
            "operations": [
              {
                "type": "Query",
                "field": "discounts",
                "path": "/discounts-api.json",
                "method": "GET",
                "responseSample": "https://raw.githubusercontent.com/AdobeDocs/graphql-mesh-gateway/main/src/pages/_examples/discounts-api.json"
              }
            ]
          }
        }
      }
    ],
    "additionalResolvers": [
      "./additional-resolvers.js"
    ],
    "files": [
      {
        "path": "./additional-resolvers.js",
        "content": "\r\nmodule.exports = {\r\n\tresolvers: {\r\n\t\tConfigurableProduct: {\r\n\t\t\tspecial_price: {\r\n\t\t\t\tselectionSet: \"{ name price_range { maximum_price { final_price { value } } } }\",\r\n\t\t\t\tresolve: (root, args, context, info) => {\r\n\t\t\t\t\tlet max = 0;\r\n\r\n\t\t\t\t\ttry {\r\n\t\t\t\t\t\tmax = root.price_range.maximum_price.final_price.value;\r\n\t\t\t\t\t} catch (e) {\r\n\t\t\t\t\t\t// ignore\r\n\t\t\t\t\t}\r\n\r\n\t\t\t\t\treturn context.DiscountsAPI.Query.discounts(\r\n\t\t\t\t\t\t{ root, args, context, info, selectionSet: \"{ name discount }\" }\r\n\t\t\t\t\t)\r\n\t\t\t\t\t\t.then((response) => {\r\n\t\t\t\t\t\t\tconst discountConfig = response.find((discount) => discount.name === root.name);\r\n\r\n\t\t\t\t\t\t\tif (discountConfig) {\r\n\t\t\t\t\t\t\t\treturn max * ((100 - discountConfig.discount) / 100);\r\n\t\t\t\t\t\t\t} else {\r\n\t\t\t\t\t\t\t\treturn max\r\n\t\t\t\t\t\t\t}\r\n\t\t\t\t\t\t})\r\n\t\t\t\t\t\t.catch(() => {\r\n\t\t\t\t\t\t\treturn null;\r\n\t\t\t\t\t\t});\r\n\t\t\t\t},\r\n\t\t\t},\r\n\t\t},\r\n\t},\r\n};\r\n"
      }
    ]
  }
}

PR #196 updates the example in the documentation.

hi @jhadobe ,
Thank you, It worked fine for me when changing the path.
However, I'm still not clear where the word "ConfigurableProduct" is declared.
If I replace the word "ConfigurableProduct" with "SimpleProduct" to apply the discount to simple products. It doesn't work for simple product named "Night Out Collection", special_price is null.

hi @jhadobe ,
Thank you, It worked fine for me when changing the path. However, I'm still not clear where the word "ConfigurableProduct" is declared. If I replace the word "ConfigurableProduct" with "SimpleProduct" to apply the discount to simple products. It doesn't work for simple product named "Night Out Collection", special_price is null.

This is part of Adobe Commerce: https://developer.adobe.com/commerce/webapi/graphql/schema/products/interfaces/types/configurable/
also the item, you are referring to is labelled "unavailable for purchase" in Venia: https://venia.magento.com/night-out-collection.html

@haunguyenlxag the Night Out Collection (SKU: VA24) is a bundle product consisting of 6 simple products: The Augusta Trio (SKU: VA23) is a grouped product with 3 simple products. I'm not sure you'll be able to do all the queries you want, since both Night Out and Augusta are "Currently unavailable for purchase."


Update from hotfix-example-7

🚀 Opening a pull request for this branch

CLA Assistant Lite bot ✍ All contributors have signed the CLA


Fill in file contents when creating files from NvimTree

I think you may want to provide a sample skeleton just to show how to add and use one.
It will also allow to see that the plugin works out of the box without trying to make your own skeletons and failing.
I have this config:
require("esqueleto").setup(
    {
      -- Directory where templates are stored
      directory = "~/.config/nvim/skeletons/", --default template directory

      -- Patterns to match when creating new file
      -- Can be either (i) file names or (ii) extension globs.
      -- Exact file name match have priority
      patterns = { "README.md", "*.py", "*.hey", "*.hi", "LICENSE.md" }
    }
)

And I added these files:
> tree .
.
├── LICENSE.md
├── skeleton.hey
└── skeleton.my-test.hi

And I created .hey, .hi and LICENSE.md files via NvimTree and it didn't work.
But it works when I use :e LICENSE.md from command mode.

@Invertisment thanks for the issue, will check what's the problem with NvimTree specifically.
Was planning in adding a video demo of the plugin and a directory with community submitted skeletons, so that information should be up in the following days whenever I release version 0.0

Alright, thanks.
Also thanks for the plugin.

Also didn't you think about using this kind of config?
https://zactyh.medium.com/custom-vim-skeletons-file-templates-c6fe771012b3
Why the plugin if this does the trick?

The plugin in its current state is a minimum viable product, it basically does what is showcased in the blog with some additional functionality for multiple templates per file type and file matching (README.md files, for example)
I intend on adding a roadmap in the future in order to have a reference in the functionality I would like this plugin to have, maintaining it minimal and modular.
I'll be adding some video examples to showcase the extra functionality this week, including some example templates in order to jumpstart the use of the plugin.
Thanks for the suggestions and for the blog URL, found some ideas there I might add to the plugin.


Would it make sense to unify BayesianLinearRegressor and BasisFunctionRegressor?

Hi! Thank you for this package, it is really nice.
I was surprised to see feature mappings implemented as a separate wrapper object BasisFunctionRegressor since BayesianLinearRegressor is just BasisFunctionRegressor with ϕ = identity. Would it not make sense for BayesianLinearRegressor  to take an optional ϕ which defaults to identity instead?
There might be efficiency gains or other benefits from this implementation I am not seeing. It would be interesting to hear the motivation behind this implementation!

So we introduced the BasisFunctionRegressor in this PR.
I think I pretty much just stated that I wanted to have it implemented the way that it is without stating why. I think it's really just a matter of taste. I guess I personally just feel that keeps the BLR implementation really clean -- if you just want to do BLR you should not (in my view) have to worry about a feature mapping when you're reading the code.
There aren't any particular performance-related reasons to do it this way.
Is there a particular reason that you're keen to unify the two?

I was thinking it require less code and has a smaller API surface. Having a single struct exposed feels more intuitive to me. I see that the exact same idea was proposed and rejected in the linked PR though (I completely missed the discussion in the PR) which I of course respect.

That's fair. I think on balance I'd rather keep it as-is, for the reasons I've mentioned above if that's okay.
Although now looking at the basis-function implementation I think we've got a bug in posterior (it returns a BLR, rather than a BFR, which is a problem)... @rossviljoen did you never encounter this when we first implemented the feature?

Yeah that doesn't look correct.  I looks to me like solving it in the current setup requires a new type like FiniteBFR to dispatch on, but wouldn't that also require duplicating the interface functions for that type as well? What are your thoughts on solving it? (Of course if there was only one unified implementation we would circumvent the issue entirely ;) )


TimeBox groups colors conflicts with the a-priori group

Tried again new defaults, when not receiving color attribute fall back to hues for the brushes c7ed6f5498669d2c2c9d4f6ab7d64abe4654aaca

TODO: Make sure the grays match the luminosity


Git status is not displayed in explorer sidebar with soft link

Sometimes it looks like #70963
Reproduce:
git init demo
touch demo/newfile
mkdir somedir
ln -s ../demo somedir/link_to_demo
code somedir/link_to_demo

Expected Result:

Actual Result:

Root Cause:
git rev-parse --show-toplevel returns a real path which not works well with the path VSC holds.
Solution:
try to normalize it back to the soft link, if possible.

is there any updates to this PR ? this is a critical bug

Since this pull request has been submitted we have made some improvements related to symbolic links. Using the steps provided in this pull request, I am unable to reproduce the issue with the latest VS Code release. I will go ahead and close this pull request.


Boxplot is showing flat lines

Looked into this a bit.  It looks to be caused by fact that the line df = super(BoxPlotViz, self).get_df(query_obj) aggregates according to the group by. But this is too early because the rest of the BoxPlotViz get_df tries to calculate the quartiles on a single value.
The quick and dirty solution is to create a dataframe from the raw query, then you're able to produce a boxplot with variation.

What if I face this exact issue? Has this been fixed? I really can't understand what I'm doing wrong.

Still got the same issue. The boxplot is showing flatlines


[Question] Master can not be started

**For better global communication, Please describe it in English. If you feel the description in English is not clear, then you can append description in Chinese(just for Mandarin(CN)), thx! **
Describe the question
When I set the parameter registry.servers="lcoalhost:2181", Master can be started.
However it cann't when I set the romote server address.
And I find out that Apache Curator Framework sets ENABLE_CLIENT_SASL_KEY to be the default value  'true'.
So this is  a switch to disable SASL completely.
Which version of DolphinScheduler:
-[dev]
Additional context
related #5792
Requirement or improvement

Please describe about your requirements or improvement suggestions.

a.the zookeeper version is update to 3.5.9/3.6.3
b.add code:
1、CuratorFramework client = CuratorFrameworkFactory.builder()
2、       .zookeeperFactory(new DefaultZookeeperFactory())
3、        .connectString(CONNECT_ADDR)
4、        .connectionTimeoutMs(CONNECTION_TIMEOUT)
5、        .retryPolicy(retryPolicy)
6、        .build();
client.start();
boolean state = client.blockUntilConnected(600, MILLISECONDS)
//if delete line 2,state is false
System.out.println(state);//true
class DefaultZookeeperFactory implements ZookeeperFactory{
@Override
public ZooKeeper newZooKeeper(String s, int i, Watcher watcher, boolean b) throws Exception {
ZKClientConfig config = new ZKClientConfig();
config.setProperty(ZKClientConfig.ENABLE_CLIENT_SASL_KEY,"false");
return new ZooKeeper(s,i,watcher,b,config);
}
}

@CalvinKirs


add  vearch vectorstore

hey @zhanghexian, what's the change here?

No change,thanks!
---Original---
From: @.>
Date: Tue, Sep 12, 2023 05:58 AM
To: @.>;
Cc<EMAIL_ADDRESS>Subject: Re: [langchain-ai/langchain] add  vearch vectorstore (PR #10432)
hey @zhanghexian, what's the change here?
—
Reply to this email directly, view it on GitHub, or unsubscribe.


Merge native and UWP WebRTC libraries into one

We need to merge all 3dtoolkit patches into one unified library. This includes:

[ ] Unified Video Frame extensions for frame prediction
[ ] Merge UWP MF encoder and NvPipe encoder
[ ] Unify MediaEngine decoding implementations

MediaEngine implementation does not work correctly on Desktop UWP. More work is required on the MediaStreamSource implementation to add correct timestamps and duration. Since UWP Desktop client is out of scope at the moment, we will use the MediaEngine only for HoloLens clients.

@anderm I would like to make sure we're on the same page - it's out of scope of that task, but MediaEngine will still be adapted, so both HL and Desktop clients may be implemented in UWP architecture?

@machur MediaEngine will be available on UWP clients but initial tests show that the feed is not stable on desktop. As soon as this task is finished, we will add new issues to track MediaEngine performance on UWP desktop.

closed with #240

Guys, I see that you merged WebRTC dependencies, so transition to UWP architecture should be possible now. Does it also mean that current native server plugin is fully compatible with UWP? Are you planning to work on any further tasks regarding UWP implementation in the toolkit?


Lossy Tolerance

When diffing PNGs or other lossy items the API provides a tolerance parameter.  Sometimes canvas is lossy when we don't expect it to be.  Writing / reading imagedata with an alpha layer is lossy (see http://www.whatwg.org/specs/web-apps/current-work/multipage/the-canvas-element.html#pixel-manipulation).
Can we encapsulate this away from the user?
Or can our matchers intelligently suggest this is the issue when a test has failed?

Explore canvas' CanvasRenderingContext2d#patternQuality method, changing from good to best
https://github.com/LearnBoost/node-canvas


Refactor for ILPv4 and split codecs

Major refactor to:

Create custom annotation for consistent use of Immutables
Split out codec framework
Split out core lib codecs
Split out transport layer protocols (to be added in a future commit)
Support new packet types for ILPv4

Incorporates changes from:

PR #81 from @sappenin (Immutables for crypto-conditions)
PR #77 from @earizon (New ILPv4 packets)
PR #65 from @sappenin (Jackson serializers)

Working on addressing comments for @sappenin.
Also waiting on #89 from @earizon review before merging.

Codecov Report

Merging #85 into master will decrease coverage by 4.38%.
The diff coverage is 64.05%.

@@             Coverage Diff              @@
##             master      #85      +/-   ##
============================================
- Coverage     78.56%   74.18%   -4.39%
+ Complexity      367      306      -61
============================================
  Files            91       70      -21
  Lines          2622     1739     -883
  Branches        271      170     -101
============================================
- Hits           2060     1290     -770
+ Misses          425      370      -55
+ Partials        137       79      -58

Impacted Files
Coverage Δ
Complexity Δ

...rledger/encoding/asn/framework/CodecException.java
0% <ø> (ø)
0 <0> (?)

.../interledger/core/InterledgerRuntimeException.java
0% <ø> (ø)
0 <0> (?)

...ger/cryptoconditions/utils/UnsignedBigInteger.java
37.5% <0%> (-12.5%)
2 <1> (ø)

...n/codecs/AsnInterledgerFulfillPacketDataCodec.java
0% <0%> (ø)
0 <0> (?)

...org/interledger/core/InterledgerFulfillPacket.java
0% <0%> (ø)
0 <0> (?)

...r/encoding/asn/codecs/AsnPrintableStringCodec.java
0% <0%> (ø)
0 <0> (?)

...interledger/core/InterledgerProtocolException.java
0% <0%> (ø)
0 <0> (?)

...asn/codecs/AsnPrintableStringBasedObjectCodec.java
0% <0%> (ø)
0 <0> (?)

...n/codecs/AsnInterledgerPreparePacketDataCodec.java
100% <100%> (ø)
3 <3> (?)

...re/asn/codecs/AsnInterledgerRejectPacketCodec.java
100% <100%> (ø)
1 <1> (?)

... and 80 more

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 a5715ea...5fe87a0. Read the comment docs.


Card fix 1

http://manage.dimagi.com/default.asp?215785

I can't figure out why the reordering is happening, pretty deep in the SGLM code, though seems possibly related to this known bug: https://github.com/lucasr/twoway-view/issues/190
However the behavior we're seeing is different, screen rotation non-deterministically will trigger a reordering, then scrolling will always revert the menu to its original/correct ordering
This hacky workaround basically just invalidates the current spannable cache and makes a call to the grid manager's checkForGaps method:
https://android.googlesource.com/platform/frameworks/support/+/a910619/v7/recyclerview/src/android/support/v7/widget/StaggeredGridLayoutManager.java#216

The reordering issue on scroll might be related to this line: https://github.com/dimagi/commcare-odk/blob/master/app/src/org/commcare/dalvik/activities/HomeActivityUIController.java#L87
When I removed that line I didn't experience the reordering on scroll, only reordering when you pause and resume the activity

Actually, might be wrong on that one... It might also be fixed by adding uiController.refreshView to CommCareHomeActivity.onResume

@phillipm tried that among many other similar "fixes"

Think I found something: I modified ResizingTextView, adding a call to ResizingTextView.reset() at this line: https://github.com/dimagi/commcare-odk/blob/master/app/src/org/commcare/android/view/ResizingTextView.java#L72. Seems to have resolved the issue for me.

@phillipm here's what I'm seeing just implementing that change - is this the desired/acceptable output? Buttons still misaligned, logout button on right side cc @ctsims @amstone326 @biyeun

@phillipm just saw you opened https://github.com/dimagi/commcare-odk/pull/1046, will move this discussion over there and close this branch for now


chore: remove redundant // +optional for CRDs' fields

What this PR does / why we need it:
This PR removes redundant // +optional comment for fields in CRDs, check the docs. Moreover nothing is regenerated after this change thus this comment is ineffectual for our code.

I wonder if that actually doesn't provide some value.
Even projects like Gateway API use these https://github.com/search?q=repo%3Akubernetes-sigs%2Fgateway-api+%2Boptional&type=code 🤔

My rationale is that we had those // +optional only in some places in the repo and not in others. So nobody could rely on them

This is what I found on this topic:
https://github.com/kubernetes/community/blob/4709a93393954b40eebe3780687640b71bcefa98/contributors/devel/sig-architecture/api-conventions.md#optional-vs-required

Good find, interesting. I wonder how up-to-date it is (it's 4 years old). In kubeuilder for CRD validation there is only // +kubebuilder:validation:Optional. Furthermore we don't have // +optional in similar cases e.g.

KongConsumer
KongPlugin

and everything works.


fix: update field mapping for contributor lists create

Description

Updates the fields to match the data returned from the API when creating a list

What type of PR is this? (check all applicable)

[ ] 🍕 Feature
[x] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 🎨 Style
[ ] 🧑💻 Code Refactor
[ ] 🔥 Performance Improvements
[ ] ✅ Test
[ ] 🤖 Build
[ ] 🔁 CI
[ ] 📦 Chore (Release)
[ ] ⏩ Revert

Related Tickets & Documents

Mobile & Desktop Screenshots/Recordings

Added tests?

[ ] 👍 yes
[ ] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help

Added to documentation?

[ ] 📜 README.md
[ ] 📓 docs.opensauced.pizza
[ ] 🍕 dev.to/opensauced
[ ] 📕 storybook
[ ] 🙅 no documentation needed

[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?

This also needs the interface update https://github.com/open-sauced/app/blob/0e8f622b001fcd7d1fdd86fb7b52ee149b586d58/next-types.d.ts#L52

👍🏼
Closes https://github.com/open-sauced/app/issues/1743


Using the CDN

Hello
How you add js file in css link tag?
Best regards
Christoph

Hi Christoph,
You need to use script tags to include the CDN:
<script src="https://unpkg.com/vue-star-rating/dist/star-rating.min.js"></script>


OSError: We couldn't connect to 'https://huggingface.co' to load this file

执行
python tools/llama/generate.py
--text "要转换的文本"
--prompt-text "你的参考文本"
--prompt-tokens "fake.npy"
--checkpoint-path "checkpoints/text2semantic-400m-v0.2-4k.pth"
--num-samples 2
--compile
报错：
resolved_config_file = cached_file(
File "/data/workpace/fish-speech/.conda/lib/python3.10/site-packages/transformers/utils/hub.py", line 441, in cached_file
raise EnvironmentError(
OSError: We couldn't connect to 'https://huggingface.co' to load this file, couldn't find it in the cached files and it looks like fishaudio/speech-lm-v1 is not the path to a directory containing a file named config.json.
Checkout your internet connection or see how to run the library in offline mode at 'https://huggingface.co/docs/transformers/installation#offline-mode'.
尝试从huggingface下载config.json文件到本地，但是模型下面没有看到有config.json
请问如何解决呢？

可以下载 fishaudio/speech-lm-v1 并手动指定 --tokenizer 为本地文件夹.


Flatten Array - Allow tests to PASS for both nil and empty slices.

Update flatten_array_test.go so that test cases will pass for both nil and empty slices. Some students have had problems where the test FAILS reporting [] != []:
=== RUN   TestFlatten/empty
    flatten_array_test.go:12: Flatten([]) = [], want: []

=== RUN   TestFlatten/all_values_in_nested_list_are_null
    flatten_array_test.go:12: Flatten([<nil> [[[<nil>]]] <nil> <nil> [[<nil> <nil>] <nil>] <nil>]) = [], want: []

--- FAIL: TestFlatten (0.00s)
    --- FAIL: TestFlatten/flattens_a_nested_array (0.00s)
    --- FAIL: TestFlatten/all_values_in_nested_list_are_null (0.00s)

I propose that we implement the same fix used in Exercise grep : fix test case which incorrectly fails on nil slice, but expect empty slice (PR #2452)
Original
func TestFlatten(t *testing.T) {
    for _, tc := range testCases {
        t.Run(tc.description, func(t *testing.T) {
            if actual := Flatten(tc.input); !reflect.DeepEqual(actual, tc.expected) {
                t.Errorf("Flatten(%v) = %v, want: %v", tc.input, actual, tc.expected)
            }
        })
    }
}

Proposed
func TestFlatten(t *testing.T) {
    for _, tc := range testCases {
        t.Run(tc.description, func(t *testing.T) {
            actual := Flatten(tc.input)

            // We do not care whether the result is nil or an empty slice.
            if len(tc.expected) == 0 && len(actual) == 0 {
             return
            }

            if !reflect.DeepEqual(actual, tc.expected) {
                t.Errorf("Flatten(%v) = %v, want: %v", tc.input, actual, tc.expected)
            }
        })
    }
}

Alternatively we could change the error message to explain that zero length and nil are similar, but not the same.
I am happy to submit the PR based on the feedback to this issue.

I think in the context of this exercise, it makes sense to expect a result, that would actually render as [] when e.g. formatted as JSON (how to return [] from a JSON API is a common question from beginners). I also think it is a good practice for the student if the test fails to think about how to fix this. I totally agree there should be a better error message specifically for this case though. E.g. if it would way "got nil slice, expected empty slice", that would be enough for a student to research their way to a solution I think. Could you adjust the PR in this way (or create a new one)?

I am happy to adjust this further to make it clear. I took @junedev 's  idea "got nil slice, expected empty slice", but wanted to keep the same or similar formating to the existing error messages.
New
=== RUN   TestFlatten/empty
    flatten_array_test.go:13: Flatten([]) = [] (nil slice), want: [] (empty slice)

if actual := Flatten(tc.input); !reflect.DeepEqual(actual, tc.expected) {
     if actual == nil && tc.expected != nil && len(tc.expected) == 0 {
	  t.Errorf("Flatten(%v) = %v (nil slice), want: %v (empty slice)", tc.input, actual, tc.expected)
     } else {
	  t.Errorf("Flatten(%v) = %v, want: %v", tc.input, &actual, tc.expected)
     }
}

I have implemented
=== RUN   TestFlatten/empty
    flatten_array_test.go:13: Flatten([]) = [] (nil slice), want: [] (empty slice)
=== RUN   TestFlatten/flattens_a_nested_array
    flatten_array_test.go:13: Flatten([[[]]]) = [] (nil slice), want: [] (empty slice)
=== RUN   TestFlatten/all_values_in_nested_list_are_null
    flatten_array_test.go:13: Flatten([<nil> [[[<nil>]]] <nil> <nil> [[<nil> <nil>] <nil>] <nil>]) = [] (nil slice), want: [] (empty slice)

Your approach is good, but I wanted to put out an additional option for consideration.
If we add the # verb modifier, we can see the difference between a nil and empty slice without modifying  any of the surrounding logic.
original
t.Errorf("Flatten(%v) = %v, want: %v", tc.input, &actual, tc.expected)
with # modifier
t.Errorf("Flatten(%v) = %#v, want: %#v", tc.input, &actual, tc.expected)
=== RUN   TestFlatten/empty
    flatten_array_test.go:16: Flatten([]) = &[]interface {}(nil), want: []interface {}{}
=== RUN   TestFlatten/flattens_a_nested_array
    flatten_array_test.go:16: Flatten([[[]]]) = &[]interface {}(nil), want: []interface {}{}
=== RUN   TestFlatten/all_values_in_nested_list_are_null
    flatten_array_test.go:16: Flatten([<nil> [[[<nil>]]] <nil> <nil> [[<nil> <nil>] <nil>] <nil>]) = &[]interface {}(nil), want: []interface {}{}

I am happy to adjust this further to make it more clear. I took @junedev's idea "got nil slice, expected empty slice" but wanted to keep the same or similar formatting to the existing error messages.
New
=== RUN   TestFlatten/empty
    flatten_array_test.go:13: Flatten([]) = [] (nil slice), want: [] (empty slice)

Original
=== RUN   TestFlatten/empty
    flatten_array_test.go:13: Flatten([]) = [], want: []

I considered another apporach of adding the # verb modifier to differentiate the nil and empty slice, but I found the results less clear than the solution above.
t.Errorf("Flatten(%v) = %#v, want: %#v", tc.input, &actual, tc.expected)
=== RUN   TestFlatten/empty
    flatten_array_test.go:16: Flatten([]) = &[]interface {}(nil), want: []interface {}{}

I had the same thoughts around #v and came to the same conclusion as you.


Pin cuDNN via run_export

context: https://github.com/AnacondaRecipes/aggregate/issues/173#issuecomment-550072394, https://github.com/conda-forge/cupy-feedstock/pull/18#discussion_r354670446
cc: @jakirkham

wrong repo 😞


Enable Py3k compatibility (fixes #1527)

This branch is addressing compatibility with python3.
Note: don't pull yet, I'm using the PR to get automatic travis builds.

Travis should be happy, please review!

The travis test are still running with miniconda 2.7 for 3.6 tests.

Earlier, Travis was failing because of code not working with Python3. I guess there's an interpreter mixup somewhere. Thank you, will investigate.

@peristeri actually my memory was corrupted, I guess python3 was running on my computer only... and I hadn't tested everything. No need to review further, more work is needed on this PR...

Fix all the incompatibilities in this pull request would be too much work:

there is WIP GUI code that uses PyQt4 (python2 only) and I'd prefer not to touch it for now
there are tools using xlutils which is python2 only (and other similar situations)

So I propose to make sure that the code itself is python3-compatible, but won't actually have a python3 installation for now. This would be in a further PR. The corresponding issue #1527 won't be closed with this PR.


Add HTTP 8080 listen for conformance tests

A conformance test requires testing against a listener on port 8080. Our manually-deployed test Kong instance doesn't have this at present, so the test fails. Adding such a listen (or testing against an operator-deployed instance) should let the test pass, though this isn't confirmed.

This should become moot, upstream plans to remove it https://github.com/kubernetes-sigs/gateway-api/issues/2094

closing as superseded by #4563


Comparison with other tools

Is there an existing proposal similar to this?

[X] I have searched the existing proposals

What are you proposing?
It would be helpful when choosing a framework like this to compare with the existing ecosystem. It's always a gamble to choose a "new" framework so it helps to lay the ground clearly about how this compares with other existing competition - in this case, frameworks like Nest and Adonis come to mind
Is there any specific group of users that will benefit from this?
Anyone performing discovery on which tooling to use
What problems are you trying to solve?
I'm trying to make the pros and cons of this tool more discoverable
Do you have any references or examples that can illustrate your idea?
There are many comparisons of existing frameworks covering things like:

Features
Benchmarks
Integrations
Community
Etc..

What type of idea is this?
Improvement of existing idea: Similar idea exists but this is an improvement

Hi @tomatau thank you for opening the issue, your point is valid.
Right now we are looking for people to create content about ExpressoTS, either Youtube, Twitter, Dev.to, Medium. These advocates or content creators will significantly clarify the project goal as well as to expand its presence around the globe.
As framework creator, with limited time and resources available I need to focus energy in few things at the time. My current priorities are to deliver few other tool that are crucial for more complex backend scenarios.
In the next couple of months we will dedicate a huge portion of our time to make the framework well known and understood in terms of capabilities and how its compare with against their competitors.
Here a technical comparison we did months ago and we already outperform Nestjs. Adonisjs is a basically stale product that is probably will not continue to push the boundaries forward. The big real competitor in the Typescript space is Nestjs.
In terms of differences, we have a complete different philosophy and all the mechanics within the  tool are confirming and affirming the framework philosophy. We aim simplicity (and to make it simple, requires complex features). Simplicity for the developer, code their company next project in a framework is fully featured, has a natural and easy way to structure the project and so on.
Would be lovely to see people like yourself pushing ExpressoTS forward. You may end up in a page like this: https://expresso-ts.com/team
We will create another section/page for content creators to tag their content. :)


Serializers can incude related resources using dot-path notation

For now this is just a failing test demonstrating the issue.
Closes #708

@cibernox I just merged #705, feel free to move forward on this

@samselikoff Done.
The logic I've decided to follow is:

If the request contains an included param, that wins over any configuration, even if that included params is an empty string (that would be like asking to not include related resources to an endpoint that does include them by default)
If the request doesn't include it, use the serializer's default config.

Looks great! Sorry about the delay, and thanks again :+1:


[Enhancement] Create shell redirects

MSYS doesn't pick up batch files as executable, so you have to run them with cmd //c <prog>. This is pretty easy to work around with a shell script named <prog> (no extension):
#!/bin/sh
cmd //c <prog.cmd> "$@"

Alternately, you could just remove the redirects and add the executable-containing folders to the path directly. I think I'd prefer that, honestly.

Shimgen has fixed this.


HTMLSelectElement.prototype.remove not remove dom Element

The remove() method is used to remove an option from a drop-down list and IE requires option index.
Please review my pull Request!

OK, thanks for the PR but I don't really fancy it, here the reason:

you are not doing any feature detection, whatever IE needs here seems to be very obtrusive for every other browser too
you are attaching non standard methods to the prototype ... not needed and obtrusive. You can just use the dom4 scope to eventually trap that method and use call/apply without needing to attach it with the native prefix
you are checking the argument, not checking that this points to an instanceof Select, there are hidden potential errors in this patch
the test changes another one, you could just create a new one specific for this select problem, instead of changing one that was testing other nodes

Last, but not least, I'd like to understand with a basic HTML example which case, and also which IE will fail through remove
Thanks for improving this PR or for telling me more about the issue.
Best Regards

It is not IE specific problem. I posted this bug at 2013 https://www.w3.org/Bugs/Public/show_bug.cgi?id=20720
The problem will be somewhere here:
while( node.firstChild ) {
  node.firstChild.remove();
}

for the most developers this code will be clearing node from others nodes like node.innerHTML = '', but if there will be a select node, it will never stops - node.firstChild.remove() will not lead to node.firstChild removal... just unexpected behavior.

this has been fixed in 1.4.8 in a different way, obtaining same result ;-)
Thanks


Doesn't build on Arch (it used to)

Steps to reproduce
use an up-to-date Arch Linux to build the master branch
Compiler output
compiled with: CC=gcc ARCH=amd64 util/build
and: CC=clang ARCH=amd64 util/build
both gave the same error
9660srv.o: In function `iattach':
/home/rain/VMs/Harveydev/harvey/sys/src/cmd/9660srv/9660srv.c:212: undefined reference to `__stack_chk_fail'
9660srv.o: In function `iwalkup':
/home/rain/VMs/Harveydev/harvey/sys/src/cmd/9660srv/9660srv.c:249: undefined reference to `__stack_chk_fail'
9660srv.o: In function `iwalk':
/home/rain/VMs/Harveydev/harvey/sys/src/cmd/9660srv/9660srv.c:306: undefined reference to `__stack_chk_fail'
9660srv.o: In function `ireaddir':
/home/rain/VMs/Harveydev/harvey/sys/src/cmd/9660srv/9660srv.c:360: undefined reference to `__stack_chk_fail'
9660srv.o: In function `opendotdot':
/home/rain/VMs/Harveydev/harvey/sys/src/cmd/9660srv/9660srv.c:565: undefined reference to `__stack_chk_fail'
9660srv.o:/home/rain/VMs/Harveydev/harvey/sys/src/cmd/9660srv/9660srv.c:755: more undefined references to `__stack_chk_fail' follow
2017/09/08 00:02:43 exit status 1
  71.02s user 19.46s system 65% cpu 2:17.34 total

That symbol looks like is not in harvey's code. Please, could you post the linking line previous to error? Also the version of binutils and gcc. Thanks.

We have to find the switch to turn it off.

The switch to fix this is -fno-stack-protector
what clang will do with that is anybody's guess.

Solved in #645


[ENH, MAINT, BUG] Refactor mne.viz.Brain

Describe the new feature or enhancement
To me mne.viz.Brain is a misnomer when there are multiple views; matplotlib.pyplot.Axes isn't a single subplot or multiple subplots, it's just a single subplot axis, I think mne.viz.Brain should just be a single 3D rendered brain object/scene.
Describe your proposed implementation
The GUI code could all be pulled out into an mne.viz.BrainViewer or such that handles one or multiple mne.viz.Brains and synchronizes them.
Describe possible alternatives
If the sidebar and the toolbar work synchronously for multiple views, it seems to me that those could only make sense at level of the mne.viz.BrainViewer if this approach were to be taken so that would need to be moved up a level as well.
Additional context
I don't think this would be years or work, I think this would be a lot like abstracting SliceBrowser out of the IEEGLocatorGUI, a week or so.
Also, when I do stc.plot(...) the time course annoyingly pops up off the screen and I have to resize the window and then click and drag it up to see it, I would say this is a bug that would be fixed by this as well.

Please read my posts from before a bit more carefully -- this is not what I suggested. I think we want to keep the Brain API the same (multiple views, methods, etc.) and just change the guts behind it.

I read your posts carefully and I agree on moving to the QInteractor and using the abstraction, those both seem like great ideas. This is separate but related and given your comment, I think it's clear I disagree with you that this is not a problem, I think it really is not the best way to do things and should be fixed.

I would much rather keep the API of Brain more or less as it is, and instead have what you want to make "Brain" be called "BrainView" or something that implies "one single view of the brain". The code churn / deprecation for people is not worth what more or less amounts to a slightly clearer name at our end. I think our threshold for deprecation should be higher than this

Ok, I'll go ahead and close the issue. I still think this is a design issue and since mne.viz.Brain is tagged as experimental it would be a good time to fix it but I'm not going to press the issue. I think in the long run, it will be something that really simplifies mne.viz.Brain but it works okay mostly now so not the highest priority.


Activating some sidebar panels takes keyboard focus away from game applet

When activating some panels (configuration and hiscores that I've noticed, including using the hiscores lookup function ingame--maybe anything with a text input field?) it'll take keyboard focus away from the game, so fkeys and arrow keys won't work until you click back inside the rs applet which is kind of annoying. Typing while the focus is lost to the panel doesn't even seem to do anything so this seems unintended

At least for the hiscores panel, this is likely intentional since most players will want to type to search for a player when opening it.

Text input fields taking focus is intentional, for Hiscore and Config panels, but if you can't start typing right away to input to text fields there, it sounds like a bug.

Fixed in 46b56797418264b0b8427e7b5baed2f3976d17eb


System.NullReferenceException: RegexInterpreter.FindFirstChar

Hello there,
We are using this library, and we are getting a null exception in your library's code, as per my assumption, this is the part of code throwing an exception.
You can find more detailed stacktrace here.
Thanks.

Thanks for posting this, I'm looking into it.


Incorrect recognition with rotated images

The following tests the recognition of the phototest image which has been rotated 90, 180 and 270 degrees, with both --psm 3 and --psm 6. While the program does not crash, the recognition is incorrect in most of the rotated images.
ubuntu@tesseract-ocr:~/tesseract$ tesseract ./test/testing/phototest.tif - --tessdata-dir ../tessdata_fast --oem 1 --psm 3 -l eng -c page_separator=''
Page 1
This is a lot of 12 point text to test the
ocr code and see if it works on all types
of file format.
   The quick
brown dog jumped over the lazy fox.
ubuntu@tesseract-ocr:~/tesseract$ tesseract ./test/testing/phototest-rotated-R.png - --tessdata-dir ../tessdata_fast --oem 1 --psm 3 -l eng -c page_separator=''
This is a lot of 12 point text to test the
ocr code and see if it works on all types
of file format.
   The quick
brown dog jumped over the lazy fox.
ubuntu@tesseract-ocr:~/tesseract$ tesseract ./test/testing/phototest-rotated-L.png - --tessdata-dir ../tessdata_fast --oem 1 --psm 3 -l eng -c page_separator=''
“‘xoy AZe] By} JBAO peduunl Bop umosgq
yoinb ay *xo} Aze] au} JOA peduun{
Bop umosg yoInb ay “xo Aze] ay} JaA0
peduun{ Bop umoiq yoinb aus ‘xoy Aze]
24} J8A0 peduun{ Bop umosg yoinb ayy
“JEUOY SII} JO
sadfj ||— UO SyJOM }I JI 89S PUe Bpod 490
OU} 4S9} 0} }X9} Julod Z|, JO jo] © SI SIYL
ubuntu@tesseract-ocr:~/tesseract$ tesseract ./test/testing/phototest-rotated-180.png - --tessdata-dir ../tessdata_fast --oem 1 --psm 3 -l eng -c page_separator=''
“‘xoy AZe] By} JBAO peduunl Bop umosgq
yoinb ay *xo} Aze] au} JOA peduun{
Bop umosg yoInb ay “xo Aze] ay} JaA0
peduun{ Bop umoiq yoinb aus ‘xoy Aze]
24} J8A0 peduun{ Bop umosg yoinb ayy
“JEUOY SII} JO
sadfj ||— UO SyJOM }I JI 89S PUe Bpod 490
OU} 4S9} 0} }X9} Julod Z|, JO jo] © SI SIYL
ubuntu@tesseract-ocr:~/tesseract$ tesseract ./test/testing/phototest-rotated-180.png - --tessdata-dir ../tessdata_fast --oem 1 --psm 6  -l eng -c page_separator=''
“‘xoy AZe] By} JBAO peduunl Bop umosgq
yoinb ay *xo} Aze] au} JOA peduun{
Bop umosg yoInb ay “xo Aze] ay} JaA0
peduun{ Bop umoiq yoinb aus ‘xoy Aze]
24} J8A0 peduun{ Bop umosg yoinb ayy
“JEUOY SII} JO
sadfj ||— UO SyJOM }I JI 89S PUe Bpod 490
OU} 4S9} 0} }X9} Julod Z|, JO jo] © SI SIYL
ubuntu@tesseract-ocr:~/tesseract$ tesseract ./test/testing/phototest-rotated-L.png - --tessdata-dir ../tessdata_fast --oem 1 --psm 6  -l eng -c page_separator=''
o Dp
o o .
2s £88x x
Se Lecos
a= o£EE§s
Da $535 >
2. oa8ek
2° BSvie
% acs ea
<2 @ Be
25 ESssz
25 §2385
a Seezs
£2 Bale
oe SxF 8
as eat e8
NN ® case
~2 §358
54: 8 as S
5 ;50RG5
em egos
SEESERS 2
Qoevrs 8
s2s5oxog
ogeesege
255 = es
255 2RSES
E8sfass
FOoOoFS
ubuntu@tesseract-ocr:~/tesseract$ tesseract ./test/testing/phototest-rotated-R.png - --tessdata-dir ../tessdata_fast --oem 1 --psm 6  -l eng -c page_separator=''
x
TOD
Sre2as
o5 987357
aT =
Sgag2°85
2 aSeca®
ges ESBS
SoRataes
o>
EaRssRac
3sc22' 2
gases 85
x aS an
Bass =3
gisge 25
Ss2aes z2
nees§ ea
s £53 Soa
34233 a
Regeo g8
<2 ee >
sss a é
s5538 2
2605 oF
off ZZ
eke $i
®
n
ubuntu@tesseract-ocr:~/tesseract$

These images are used in the osd test and get the correct result.
[----------] 1 test from TessdataFastRotated90/OSDTest
[ RUN      ] TessdataFastRotated90/OSDTest.MatchOrientationDegrees/0
************ Orientation in degrees: 90, Orientation confidence: 9.30
Script: Latin, Script confidence: 8.06
[       OK ] TessdataFastRotated90/OSDTest.MatchOrientationDegrees/0 (415 ms)
[----------] 1 test from TessdataFastRotated90/OSDTest (415 ms total)
[----------] 1 test from TessdataFastRotated180/OSDTest
[ RUN      ] TessdataFastRotated180/OSDTest.MatchOrientationDegrees/0
************ Orientation in degrees: 180, Orientation confidence: 10.99
Script: Latin, Script confidence: 5.83
[       OK ] TessdataFastRotated180/OSDTest.MatchOrientationDegrees/0 (414 ms)
[----------] 1 test from TessdataFastRotated180/OSDTest (414 ms total)
[----------] 1 test from TessdataFastRotated270/OSDTest
[ RUN      ] TessdataFastRotated270/OSDTest.MatchOrientationDegrees/0
************ Orientation in degrees: 270, Orientation confidence: 9.62
Script: Latin, Script confidence: 3.03
[       OK ] TessdataFastRotated270/OSDTest.MatchOrientationDegrees/0 (412 ms)
[----------] 1 test from TessdataFastRotated270/OSDTest (412 ms total)
My suggestion would be that tesseract also use the osd check internally and rotate the image as needed.

If you anticipate that some of the images might be rotated, use psm 1.
Use psm 3 only for pages that you know that are unrotated.
'you' - not just you, but any user.

@amitdo Thank you. That works. Now need to check whether it will work with the image from Spanish UNLV test.
ubuntu@tesseract-ocr:~/tesseract$ tesseract ./test/testing/phototest-rotated-R.png - --tessdata-dir ../tessdata_fast --oem 1 --psm 1  -l eng -c page_separator=''
This is a lot of 12 point text to test the
ocr code and see if it works on all types
of file format.
   The quick
brown dog jumped over the lazy fox.
ubuntu@tesseract-ocr:~/tesseract$ tesseract ./test/testing/phototest-rotated-L.png - --tessdata-dir ../tessdata_fast --oem 1 --psm 1  -l eng -c page_separator=''
This is a lot of 12 point text to test the
ocr code and see if it works on all types
of file format.
   The quick
brown dog jumped over the lazy fox.
ubuntu@tesseract-ocr:~/tesseract$ tesseract ./test/testing/phototest-rotated-180.png - --tessdata-dir ../tessdata_fast --oem 1 --psm 1  -l eng -c page_separator=''
This is a lot of 12 point text to test the
ocr code and see if it works on all types
of file format.

Page segmentation modes:
  0    Orientation and script detection (OSD) only.
  1    Automatic page segmentation with OSD.
  2    Automatic page segmentation, but no OSD, or OCR.
  3    Fully automatic page segmentation, but no OSD. (Default)
  4    Assume a single column of text of variable sizes.
  5    Assume a single uniform block of vertically aligned text.
  6    Assume a single uniform block of text.


preexecute + standalone circuit breaker

Hi,
What purpose does preexecute serve? Testing circuit breaker in standalone mode the circuit seems to be opened and closed when expected without making any calls to preexecute (or recordSuccess).

When in a half-open state we don't want to allow too many concurrent executions. So we limit the circuit breaker to only allow as many executions as are configured in the success or failure threshold. Calling preExecute helps the CircuitBreaker track how many executions are in progress.

Closing for now. Please reopen if needed for further discussion.


Allow ignore directive on same line

Also some minor fixes tangentially related to ignore directives.
Fixes #447

Merging without review for inclusion in release.


Fix activesupport dependency for rails 5.1.1

Coverage remained the same at 90.934% when pulling f0930476b86b21803a9263521ccf32bcaeec8185 on m1neral:rails5.1.1 into bd07783f27acc6fa84ae95db28033eceac82284a on collectiveidea:master.


include:template: field

i"m hitting this issue, because i use merge requests pipelines.
https://docs.gitlab.com/13.8/ee/ci/yaml/#workflowrules-templates
a partial implementation could be that when the template value is one of the documented values of gitlab self, that the include is ignored because it has no impact on the joblist

Can you paste a minimal .gitlab-ci.yml that illustrates your needs? @mjrider
I'm sure i'll be able to implement include:template pretty fast, since they are just a preset of regular remote includes.

Something like this
---

include:
  - template: 'Workflows/MergeRequest-Pipelines.gitlab-ci.yml'

foobar:
  image: alpine:latest
  stage: test
  script:
    - echo "foobar"

https://github.com/firecow/gitlab-ci-local/pull/95 Closing this issue, since a pull request have been made
@mjrider


Materialize returns duplicated triples?

Materialize triples might be generating duplicate triples. Double check.

@marianormuro Is this issue is still valid or not?

We don't have it written but we agreed to keep duplicates
On Nov 13, 2015 6:31 AM, "Guohui Xiao"<EMAIL_ADDRESS>wrote:

@marianormuro https://github.com/marianormuro Is this issue is still
valid or not?
—
Reply to this email directly or view it on GitHub
https://github.com/ontop/ontop/issues/17#issuecomment-156403553.

Good. Then I close this issue.


cleanup screenshots, formatting and typos ahead of showtime

For Summit Session CO219 - Hands-on-Lab: Experience a New Horizon for Intra-Company Collaboration


Add LibreWolf

Implements #258.

I am just having a think about best to handle CODENAMES_SUPPORTED for the future. It should be possible to handle via the Debian and Ubuntu codenames only.


pdf aktreport generation besides generating correct month report also created data1.yml once

It happened only once so check next time

newer reproduced again


Final Review

Final Review

Items
Self
Evan Yang
Lily
Average

N@TM Att
1
1
1
---

N@TM Cap
1
1
1
---

Team Presentation to Teacher
2.5
2.5
2.5
---

Individual Runtime
5
4
5
---

Individual Code
4
4
4
---

Total
13.5/15
12.5/15
13.5/15
13.16 /15

Missing Create, JWT during n@tm

Score: 12.5/15, graded by: Evan Yang
Individual Attendance: 1
Captures:1
Team Presentation to Teacher: 2.5
Individual Score for Runtime: 4, has create, read, update, and, delete, runtime works on deployed
Individual Score for Code Review 4, has JPA, API, backend has create, read, update, delete, get, post for creating people
Team review ticket: has review ticket and runtime work


Can I use NBitcoin.Litecoin with your project

https://github.com/MetacoSA/NBitcoin.Litecoin
I tried it but transaction is always executed within Bitcoin Network not Litecoin

what code did you write.


Uncaught TypeError: Cannot read property 'bind' of undefined

I'm trying to require dropbox-js into my React project, however I am getting the following error:
Uncaught TypeError: Cannot read property 'bind' of undefined

it is relating to line 40 of dropbox.js (from the node_modules folder):
DbxEnvRequire = module.require.bind(module);

If I comment it out, I am able to get my app to run, however I'm not entirely sure what it is doing/why it is failing.
Any ideas? Happy to share any specifics you think will help.

Bear with me, I'm still quite new to javascript so I'm going to think this one through out loud....
So after doing a bit of digging in the coffeescript source, I see that this is part of an if statement that checks what environment the app is running. Basically, the end result declares two objects (or slight variants thereof):
  DbxEnvGlobal = global;
  module.exports = Dropbox;

As it is executing the node.js part of the if statement, I presume that it is meeting the following conditions:
if typeof global isnt 'undefined' and typeof module isnt 'undefined' and
    'exports' of module

Even though I'm not using node.js, I'd like to use commonjs modules so I can require dropbox-js where needed in the app. Is there a way I can fix this without modifying the lib manually?

Experiencing the same problem. Any update on this from the author?
Thanks.

Same issue here.
Yep, you cannot use the npm version "outside node" you can patch the module or simply use: https://github.com/lavelle/dropbox which shim it with bower.
Anyway even with bower dropbox end to be in the global

You may want to look at this PR https://github.com/dropbox/dropbox-js/pull/183 which is trying to fix it for Browserify.

I'd love to see this sorted for Webpack in addition to Browserify!

Any luck fixing this on Webpack?

@sashaperigo @ngalluzzo If you're using webpack, you can include the library from the CDN in your HTML template, and include 'Dropbox' in the externals config.
example.html
  <head>
    <meta charset="utf-8">
    <title>example</title>
    <script src="//cdnjs.cloudflare.com/ajax/libs/dropbox.js/0.10.2/dropbox.min.js"></script>
  </head>

webpack.config.js
module.exports = {

  externals: {
    'dropbox' : 'Dropbox'
  },

};

Then require the library when you need to use it.
app.js
var Dropbox = require("dropbox");
new Dropbox.Client({ key: "key" }).authenticate(...)

+1, same problem here with a react webpack application


Token filters

Hi Hannes,
I am currently in the process of training a new prior, I want this model to be able to generate compounds with [SH] but this, like my previous issue, is filtered out.
Are the _token_filters completely necessary also or can they be removed if one is training a new prior and wishes to generate this chemistry?
(Presumably the models vocabulary can be updated when one trains with this chemistry? - sorry if this question is naiive)
Kind regards,
Alex

The current implementation basically has the rules for the original Reinvent prior hard-coded.  It is possible to configure the standardization code but it is not exposed to the user.
What you can do is to switch off standardization when creating an empty model.  What this does is really just a scan for all possible tokens in the input SMILES and to populate the vocabulary accordingly.  At this point the vocabulary is fixed.  To ensure consistent chemistry  you would need to carefully preprocess the input SMILES.  This means that you take direct control of the standardization.
To actually train the model you run transfer learning (TL).  To switch off the standardizer you will need to edit reinvent/runmodes/TL/run_transfer_learning.py:
        if model_type == "Reinvent":
            actions = [conversions.randomize_smiles]
            logger.debug("Applying standardization and randomization of SMILES")
        elif model_type == "Mol2Mol":
            actions = [conversions.convert_to_standardized_smiles]
            logger.debug("Applying standardization of SMILES")

If you use a validation set in TL, remember to ensure consistents tokens too because you have fixed the set of tokens when creating the empty model.


Add unit tests for authentication of proxy

Pull Request Test Coverage Report for Build 7729

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.09%) to 55.178%

Totals

Change from base Build 7728:
0.09%

Covered Lines:
8728

Relevant Lines:
15818

💛  - Coveralls


Trignometric Values Calculator

Description
A seperate trignometric values calculator is required in this project
Screenshots
No response
Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[X] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[ ] I am a SWOC'24 contributor.

Code of Conduct

[X] I agree to follow this project's "Code of Conduct".

Sir please assign me this issue under iwoc @Rakesh9100

Trigonometric calculator is already present. Pls create new issues @Sakshamjain98

ok sir


this.seq.first and this.seq.last error on OEIS A006577

On the sequence https://oeis.org/A006577, nothing in ui2 works (none of the visualizers) since we are receiving
this.seq.first = Infinity and this.seq.last = -Infinity

Closing as duplicate of #407.


Dark theme

Prerequisites

[X] This is not a support issue or a question. For support, questions, or help, see support links.
[X] I performed a cursory search of the issue tracker on APK Tool GUI to avoid opening a duplicate issue
[X] This is not about an issue with APK Tool GUI

Describe the feature you'd like
Looking a way to implement dark theme natively, similar to Notepad++. I don't want to use 3rd party libs because it requires replacing all controls manually

Only title bar is natively dark using DarkNet lib, everything else I customized a lot by my own


RuntimeError: Could not infer dtype of NoneType

when I use deepspeed to train this model，eval function meet this problem.

评估数据集中有些ids是None


"unable to load preload script" error when changing contextIsolation to true and nodeIntegration to false

I git cloned and started https://github.com/herberttn/bytenode-webpack-plugin/tree/main/examples/electron-forge-typescript-webpack
I tried to change the contextIsolation  and the nodeIntegration settings respectively to true and false :
const createWindow = (): void => {
  // Create the browser window.
  const mainWindow = new BrowserWindow({
    height: 600,
    width: 800,
    webPreferences: {
      //contextIsolation: false,
      //nodeIntegration: true,
      //preload: MAIN_WINDOW_PRELOAD_WEBPACK_ENTRY,

      preload: MAIN_WINDOW_PRELOAD_WEBPACK_ENTRY,
      nodeIntegration: false,
      contextIsolation: true,
      nodeIntegrationInWorker: false,
      nodeIntegrationInSubFrames: false,
      webviewTag: true,
      webSecurity: true

    },
  });

  // and load the index.html of the app.
  void mainWindow.loadURL(MAIN_WINDOW_WEBPACK_ENTRY);

with preload.ts :
console.log('💃 hello from preload')

// https://www.electronjs.org/docs/latest/tutorial/ipc

const {
  contextBridge,
  ipcRenderer,
  shell,
  webFrame
} = require("electron")

export {}
declare global {
  interface Window {
    api: {
      send: (channel: string, ...arg: any) => void;
      receive: (channel: string, func: (event: any, ...arg: any) => void) => void;
      electronIpcSendTo: (window_id: string, channel: string, ...arg: any) => void;
      electronIpcSend: (channel: string, ...arg: any) => void;
      giveMeAStream: (eventId: string) => void;
      electronIpcOn: (channel: string, listener: (event: any, ...arg: any) => void) => void;
      electronIpcSendSync: (channel: string, ...arg: any) => void;
      electronIpcInvoke: (channel: string, ...arg: any) => void;
      electronIpcPostMessage: (channel: string, message: any, transfer?: MessagePort[]) => void;
      electronIpcOnce: (channel: string, listener: (event: any, ...arg: any) => void) => void;
      electronIpcRemoveListener:  (channel: string, listener: (event: any, ...arg: any) => void) => void;
      electronIpcRemoveAllListeners: (channel: string) => void;

      setFullscreen: (flag: any) => void;

    };

    attachEvent(event: string, listener: EventListener): boolean;
    detachEvent(event: string, listener: EventListener): void;

    // https://www.electronjs.org/docs/latest/tutorial/context-isolation#usage-with-typescript

    bwScrollAPI: {
      getScrollAmount: () => void;
    }
  }
}

contextBridge.exposeInMainWorld(
  "api", {

      electronIpcPostMessage: (channel: string, message: any, transfer?: MessagePort[]) => {
        ipcRenderer.postMessage(channel, message, transfer)
      },
      send: (channel: any, data: any) => {
          console.log("preload-send called: args: ", data);
          ipcRenderer.invoke(channel, data).catch(e => console.log(e))
      },
      receive: (channel: any, func: any) => {
        console.log("preload-receive called. args: ");
        ipcRenderer.on(channel, (event, ...args) => func(...args));
      },

      electronIpcSendTo: (window_id: number, channel: string, ...arg: any) => {
        ipcRenderer.sendTo(window_id, channel, arg);
      },

      electronIpcSend: (channel: string, ...arg: any) => {
        ipcRenderer.send(channel, arg);
      },

      giveMeAStream: (eventId: string) => {
        ipcRenderer.send('give-me-a-stream', eventId)
      },

      electronIpcSendSync: (channel: string, ...arg: any) => {
        return ipcRenderer.sendSync(channel, arg);
      },
      // https://www.electronjs.org/docs/latest/api/ipc-renderer#ipcrendererinvokechannel-args
      electronIpcInvoke: (channel: string, ...arg: any) => {
        return ipcRenderer.invoke(channel, ...arg)
      },
      electronIpcOn: (channel: string, listener: (event: any, ...arg: any) => void) => {
        ipcRenderer.on(channel, listener);
      },
      electronIpcOnce: (channel: string, listener: (event: any, ...arg: any) => void) => {
        ipcRenderer.once(channel, listener);
      },
      electronIpcRemoveListener:  (channel: string, listener: (event: any, ...arg: any) => void) => {
        ipcRenderer.removeListener(channel, listener);
      },
      electronIpcRemoveAllListeners: (channel: string) => {
        ipcRenderer.removeAllListeners(channel);
      },

      setFullscreen: (flag: string) => {
        ipcRenderer.invoke('setFullscreen', flag);
      },

  },
)

But, after this change, I get this error: unable to load preload script

what am I missing ? How to make it work?

Hi @jjeff !
In this repo: https://github.com/raphael10-collab/bytenode-contextIsolationTrue-electron-forge-typescript-webpack
you can see that I've set nodeIntegration to false, and contextIsolation to true
As you explained here: https://github.com/herberttn/bytenode-webpack-plugin/issues/29#issuecomment-1702995183 Bytenode always require Node to work.
If I want to keep for good electron safety practice nodeIntegration to false, and contextIsolation to true,  you suggest me to have all Bytenode compiled code in the preload scripts, because preload scripts have access to the renderer scope.
I've set up my preload.ts script as follows:
https://github.com/raphael10-collab/bytenode-contextIsolationTrue-electron-forge-typescript-webpack/blob/master/src/main/preload/preload.ts   :
console.log('💃 hello from preload')

// https://www.electronjs.org/docs/latest/tutorial/ipc

const {
  contextBridge,
  ipcRenderer,
  shell,
  webFrame
} = require("electron")

export {}
declare global {
  interface Window {
    //setupRenderer,
    api: {
      send: (channel: string, ...arg: any) => void;
      receive: (channel: string, func: (event: any, ...arg: any) => void) => void;
      // https://github.com/frederiksen/angular-electron-boilerplate/blob/master/src/preload/preload.ts
      // https://www.electronjs.org/docs/all#ipcrenderersendtowebcontentsid-channel-args
      electronIpcSendTo: (window_id: string, channel: string, ...arg: any) => void;
      electronIpcSend: (channel: string, ...arg: any) => void;
      giveMeAStream: (eventId: string) => void;
      electronIpcOn: (channel: string, listener: (event: any, ...arg: any) => void) => void;
      electronIpcSendSync: (channel: string, ...arg: any) => void;
      // https://www.electronjs.org/docs/latest/api/ipc-renderer#ipcrendererinvokechannel-args
      electronIpcInvoke: (channel: string, ...arg: any) => void;
      // https://www.electronjs.org/docs/latest/api/ipc-renderer#ipcrendererpostmessagechannel-message-transfer
      electronIpcPostMessage: (channel: string, message: any, transfer?: MessagePort[]) => void;
      electronIpcOnce: (channel: string, listener: (event: any, ...arg: any) => void) => void;
      electronIpcRemoveListener:  (channel: string, listener: (event: any, ...arg: any) => void) => void;
      electronIpcRemoveAllListeners: (channel: string) => void;

      setFullscreen: (flag: any) => void;

    };

    attachEvent(event: string, listener: EventListener): boolean;
    detachEvent(event: string, listener: EventListener): void;

    // https://www.electronjs.org/docs/latest/tutorial/context-isolation#usage-with-typescript

    bwScrollAPI: {
      getScrollAmount: () => void;
    }
  }
}

contextBridge.exposeInMainWorld(
  "api", {
      // https://www.electronjs.org/docs/latest/api/ipc-renderer#ipcrendererpostmessagechannel-message-transfer
      electronIpcPostMessage: (channel: string, message: any, transfer?: MessagePort[]) => {
        ipcRenderer.postMessage(channel, message, transfer)
      },
      send: (channel: any, data: any) => {
          console.log("preload-send called: args: ", data);
          ipcRenderer.invoke(channel, data).catch(e => console.log(e))
      },
      receive: (channel: any, func: any) => {
        console.log("preload-receive called. args: ");
        ipcRenderer.on(channel, (event, ...args) => func(...args));
      },
      // https://www.electronjs.org/docs/all#ipcrenderersendtowebcontentsid-channel-args
      electronIpcSendTo: (window_id: number, channel: string, ...arg: any) => {
        ipcRenderer.sendTo(window_id, channel, arg);
      },
      // https://github.com/frederiksen/angular-electron-boilerplate/blob/master/src/preload/preload.ts
      electronIpcSend: (channel: string, ...arg: any) => {
        ipcRenderer.send(channel, arg);
      },

      giveMeAStream: (eventId: string) => {
        ipcRenderer.send('give-me-a-stream', eventId)
      },

      electronIpcSendSync: (channel: string, ...arg: any) => {
        return ipcRenderer.sendSync(channel, arg);
      },
      // https://www.electronjs.org/docs/latest/api/ipc-renderer#ipcrendererinvokechannel-args
      electronIpcInvoke: (channel: string, ...arg: any) => {
        return ipcRenderer.invoke(channel, ...arg)
      },
      electronIpcOn: (channel: string, listener: (event: any, ...arg: any) => void) => {
        ipcRenderer.on(channel, listener);
      },
      electronIpcOnce: (channel: string, listener: (event: any, ...arg: any) => void) => {
        ipcRenderer.once(channel, listener);
      },
      electronIpcRemoveListener:  (channel: string, listener: (event: any, ...arg: any) => void) => {
        ipcRenderer.removeListener(channel, listener);
      },
      electronIpcRemoveAllListeners: (channel: string) => {
        ipcRenderer.removeAllListeners(channel);
      },

      setFullscreen: (flag: string) => {
        ipcRenderer.invoke('setFullscreen', flag);
      },

  },
)

What should I practically add to the above preload.ts code in order to have my all Bytonode compiled code within preload.ts ?

You might want to play with https://github.com/spaceagetv/electron-bytenode-example and try different configurations in a simplified environment. I have yet to implement Electron security best practices in that repo, but I think it can be done. In most of my applications, I'm not loading untrusted web pages into my renderers, so I'm okay without context isolation enabled.

What should I practically add to the above preload.ts code in order to have my all Bytonode compiled code within preload.ts ?

I don't know if there's anything you need to add to the preload script to compile it with Bytenode. You just need to configure webpack to compile preload scripts.
If you're asking if it's possible to move your renderer script code into your preload script so that it will be compiled, it's usually possible to do this. The main trick is to use the DOMContentLoaded event to keep your DOM-interacting code from running before the DOM has been loaded.

I've set nodeIntegration to false and contextIsolation to true
And
I've configured webpack to compile preload scripts, I've set the plugins in webpack config, specifying the compileForElectron flag, and I've tried to put inside the preload this example code: https://github.com/spaceagetv/electron-bytenode-example/blob/main/src/renderer/preload.ts.
But it does not work
I've updated the repo in github: https://github.com/raphael10-collab/bytenode-contextIsolationTrue-electron-forge-typescript-webpack.git to contain all these aforementioned trials
So, I guess there is something missing, in order to make bytenode work with the Electron security best practices in place,
with nodeIntegration to false and contextIsolation to true as fixed landmarks

Just my two cents; I've spent a few weeks this year (yes... full time 😅) trying to get bytenode to work with nodeIntegration off, but was unable to get it to work.
What did work was to make a fork of bytenode that strips the 'encoding' bits from the library, and only leave the decoding bits.
This is done in electron-vite as well, and in the end I migrated from webpack to vite so as to not manage all this myself.


Fix empty error message thrown upon 404 pages

This fixes an empty message box shown when search results return a 404.
When Anki encounters an uncaught custom exception, it uses the __str__ method to determine the error message. Since these exceptions didn’t have __str__ defined, the error message was empty. I also added a new exception superclass, AnkiForvoException to help debug this in the future.
Fixes #106

I can confirm I see the same empty error message. The fix looks good to me!


import simulator failed

Centos release 6.8
Python version 2.7.13
I installed cocotb via pip command
pip install cocotb
I have a error for import cocotb  such as
>>> import cocotb
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/my_env_p2713/lib/python2.7/site-packages/cocotb/__init__.py", line 41, in <module>
    import cocotb.handle
  File "/my_env_p2713/lib/python2.7/site-packages/cocotb/handle.py", line 45, in <module>
    import simulator
ImportError: No module named simulator

Do I miss any dependency?
Thank you

I have the exact same problem


报错了 404

--2024-06-13 16:50:00--  https://cdn0.spiritlhl.top/https://raw.githubusercontent.com/spiritLHLS/pve/main/extra_scripts/ndpresponder.service
Resolving cdn0.spiritlhl.top (cdn0.spiritlhl.top)... <IP_ADDRESS>, <IP_ADDRESS>
Connecting to cdn0.spiritlhl.top (cdn0.spiritlhl.top)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 404 Not Found
2024-06-13 16:50:02 ERROR 404: Not Found.
Failed to start ndpresponder.service: Unit ndpresponder.service is masked.
Failed to enable unit: Unit file /etc/systemd/system/ndpresponder.service is masked.
○ ndpresponder.service
Loaded: masked (Reason: Unit ndpresponder.service is masked.)
Active: inactive (dead)

哦哦哦，忘了仓库换地址这个没换，我改一下，等等

https://github.com/oneclickvirt/6in4/commit/facddaaf61ddd2fde42d90f46368c96bb833baee
已修复，你看看


after called adapter.notifyDataSetChanged(),when scroll RV, some items move to right position from wrong position

Hi,@davideas  as title, and have called adapter.setAnimationOnScrolling(false).setAnimationOnReverseScrolling(false);
can u help me!

Hello @54binge, this movement is not part of Adapter scrolling animation, especially if the 2 methods you mention are set false (by the way it is useless to call setAnimationOnReverseScrolling(false), because it is ignored by the false of setAnimationOnScrolling() as documented.
If you directly call notifyDataSetChanged() all animation are not performed at all, you should know this.
Is it grid or staggered?
Are you sure about the wrong position for these 2 items? which header they have?
You can also activate the logs and see what the Adapter is doing.

after notifyDataSetChanged() called and scrolling bottom up, the item displayed in wrong position, the position became correct after release the finger.

This is due to the layout that the RV is applying. As you see they don't change position but fill the space automatically, otherwise you have the empty space right there. This is a normal behavior, I've read this in the StaggeredLayout but probably it also occurs with GridLayout.


BUG about 'kb' obj or 'btnm' obj with new release v5.3

Hi,
I found a strange bug about 'kb' obj or 'btnm' obj ?
the code:
lv_res_t action(lv_obj_t *obj)
{
	lv_obj_t * kb1 = lv_kb_create(lv_scr_act(), NULL);
	lv_obj_align(kb1, NULL, LV_ALIGN_IN_BOTTOM_MID, 0, 0);
	lv_kb_set_ta(kb1, obj);
	return LV_RES_OK;
}
void lv_test_kb_1(void)
{

    lv_obj_t * ta = lv_ta_create(lv_scr_act(), NULL);
    lv_obj_align(ta, NULL, LV_ALIGN_IN_TOP_MID, 0, 30);
    lv_ta_set_action(ta,action);
    /* Default object*/
    lv_obj_t * kb1 = lv_kb_create(lv_scr_act(), NULL);
    lv_obj_align(kb1, NULL, LV_ALIGN_IN_BOTTOM_MID, 0, 0);
    lv_kb_set_ta(kb1, ta);
}

the picture:

you can see,the normal pictures are the first run of the program, then press the symbol 'X' to colse the 'kb',click the ‘ta’ obj and reload the 'kb'；
Then it was found that the ‘’#‘’ symbol and only the ‘’#‘’ symbol was not displayed, but there was no error in clicking on the button with the '#'

I found it was due to a problem with the recolor command.How to fix it？

@as564335sa It's probably because # is the character used for recoloring.
@kisvegabor Please comment.

I examined it and found the issue. The recolor feature was not initialized in lv_btnm_craete therefore it got random value (from memory garbage).
Fixed in master. It it working now?

Great! It is working now.


Fix type in documentation of logout tutorial.

Closing issues
closes #3676
Self Check before Merge
Please check all items below before review.

[x] Corresponding issues are created/updated or not needed
[x] Docs are updated/provided or not needed
[x] Examples are updated/provided or not needed
[x] TypeScript definitions are updated/provided or not needed
[x] Tests are updated/provided or not needed
[x] Changesets are provided or not needed

Thank you @dipbazz ⚡


Parsing error: No Babel config file detected

What version of this package are you using?
v1.4.0
What operating system, Node.js, and npm version?
macOS Catalina v10.15.7, node v14.15.0, nom v6.14.8, yarn 1.19.0.
What happened?
Getting Parsing error: No Babel config file detected in VSCode while running standard from command-line works ok.
As recommended in https://github.com/eslint/eslint/issues/13542#issuecomment-667437352 it is now recommended to use @babel/eslint-parser instead of babel-eslint in order to get fixes to things like TypeError: Cannot read property 'value' of null . My standardjs configuration is in package.json:
"standard": {
    "parser": "@babel/eslint-parser",
    "ignore": [
      "**/node_modules/**",
      "**/public/**",
      ....
    ],
    "envs": [
      "es6",
      "browser",
      "jest"
    ]
  }

and I have this in my VSCode settings:
"javascript.validate.enable": false,
"standard.usePackageJson": true,
"standard.autoFixOnSave": true,

Unfortunately with this config I am getting Parsing error: No Babel config file detected in each single file that I am open in the editor.
What did you expect to happen?
Expect linting to work in VSCode.
Are you willing to submit a pull request to fix this bug?
No

What version of standard are you using?
standard -v
npx standard -v

$ yarn standard --version
16.0.2

Closing as it is a duplicate of #100.
And there is actually a solution, reopen if you think it's a mistake, thanks!


chore: fix installation script for darwin with arm processor

On Darwin arm, yq installation fails.

Could you please rewrite your commit message as something like fix: allow darwin arm processor installation

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

v1.3.2
GitHub release

Your semantic-release bot :package::rocket: