Unable to create custom search parameter for Careplan medicaldevice

Description
I tried to create the following custom search parameter for CarePlan but server returns bad request error.
{
   "resourceType": "SearchParameter",
   "id": "careplan-medicaldevice",
   "url": "http://nictiz.nl/fhir/SearchParameter/careplan-medicaldevice",
   "version": "1.0.0",
   "name": "careplan-medicaldevice",
   "status": "active",
   "publisher": "Nictiz",
   "contact": `[`
       {
           "name": "Nictiz",
           "telecom": [
               {
                   "system": "email",
                   "value"<EMAIL_ADDRESS>                   "use": "work"
               }
           ]
       }
   ],
   "purpose": "To search on a DeviceUseStatement resource, referenced from an extension in CarePlan. Main purpose is to use this search parameter for including DeviceUseStating resouces in the result for a search on CarePlan.",
   "code": "medicaldevice",
   "base": [
       "CarePlan"
   ],
   "type": "reference",
   "description": "To alow a search (_include) on the materials used in nursing procedures, such as bandages.",
   "expression": "CarePlan.activiy.detail.extension('http://nictiz.nl/fhir/StructureDefinition/extension-medicaldevice')"
 }

FHIR Version
Stu3
Data provider
CosmosDB
Expected behavior
Should be able to create the custom search parameter with the url 'http://nictiz.nl/fhir/SearchParameter/careplan-medicaldevice'
Actual behavior
Fhir server returns the following error message
{
    "resourceType": "OperationOutcome",
    "id": "1b3598f0-f3af-40b7-8739-41b121106d80",
    "issue": [
        {
            "severity": "error",
            "code": "not-supported",
            "diagnostics": "The search parameter with definition URL 'http://nictiz.nl/fhir/SearchParameter/careplan-medicaldevice' is not supported."
        },
        {
            "severity": "error",
            "code": "exception",
            "diagnostics": "An error occurred updating the custom search parameter.  The issue must be resolved and the update resubmitted to be applied."
        }
    ]
}

AB#101795

The expression needs to be a type that can converted from the Model Type to a Search type, so it can be indexed. In the example above the expression ends with type Extension.
If you take http://hl7.org/fhir/us/core/STU3.1.1/SearchParameter-us-core-race.html for example, it specifies Patient.extension.where(url = 'http://hl7.org/fhir/us/core/StructureDefinition/us-core-race').extension.value.code which would be able to be converted to a token search type.
If the extension data was similar to:
"extension":  [
                    {
                        "url": "http://nictiz.nl/fhir/StructureDefinition/extension-medicaldevice",
                        "valueReference": {
                            "reference": "DeviceUseStatement/zib-wound-medicaldevice-01",
                            "display": "Verband"
                        }
                    }
                ]

The parser would need something like the following to detect support correctly:
CarePlan.activiy.detail.extension('http://nictiz.nl/fhir/StructureDefinition/extension-medicaldevice').value

Thanks for the update!
I modified the expression as you mentioned but still I get the same error. It complains about the value of SearchParameter.url    property.
{
    "resourceType": "OperationOutcome",
    "id": "f84282eb-ad9e-4618-bffd-d993b7a93bb9",
    "issue": [
        {
            "severity": "error",
            "code": "not-supported",
            "diagnostics": "The search parameter with definition URL 'http://nictiz.nl/fhir/SearchParameter/careplan-medicaldevice' is not supported."
        },
        {
            "severity": "error",
            "code": "exception",
            "diagnostics": "An error occurred updating the custom search parameter.  The issue must be resolved and the update resubmitted to be applied."
        }
    ]
}

Can you send the updated search parameter you posted?

Here it is
{
                "resourceType": "SearchParameter",
                "id": "careplan-medicaldevice",
                "url": "http://nictiz.nl/fhir/SearchParameter/careplan-medicaldevice",
                "version": "1.0.0",
                "name": "careplan-medicaldevice",
                "status": "active",
                "publisher": "Nictiz",
                "contact": [
                    {
                        "name": "Nictiz",
                        "telecom": [
                            {
                                "system": "email",
                                "value"<EMAIL_ADDRESS>                                "use": "work"
                            }
                        ]
                    }
                ],
                "purpose": "To search on a DeviceUseStatement resource, referenced from an extension in CarePlan. Main purpose is to use this search parameter for including DeviceUseStating resouces in the result for a search on CarePlan.",
                "code": "medicaldevice",
                "base": [
                    "CarePlan"
                ],
                "type": "reference",
                "description": "To alow a search (_include) on the materials used in nursing procedures, such as bandages.",
                "expression": "CarePlan.activiy.detail.extension('http://nictiz.nl/fhir/StructureDefinition/extension-medicaldevice').value"
            }

Let me recall my previous statement...
Main reason why it's not working:
CarePlan.activiy is misspelled it should be CarePlan.activity

Thanks I have corrected the spelling as mentioned but I get a different error now. I have provided value for all the mandatory properties, I'm not sure what I'm missing still.
{
                "resourceType": "SearchParameter",
                "id": "careplan-medicaldevice",
                "url": "http://nictiz.nl/fhir/SearchParameter/careplan-medicaldevice",
                "version": "1.0.0",
                "name": "careplan-medicaldevice",
                "status": "active",
                "publisher": "Nictiz",
                "contact": [
                    {
                        "name": "Nictiz",
                        "telecom": [
                            {
                                "system": "email",
                                "value"<EMAIL_ADDRESS>                                "use": "work"
                            }
                        ]
                    }
                ],
                "purpose": "To search on a DeviceUseStatement resource, referenced from an extension in CarePlan. Main purpose is to use this search parameter for including DeviceUseStating resouces in the result for a search on CarePlan.",
                "code": "medicaldevice",
                "base": [
                    "CarePlan"
                ],
                "type": "reference",
                "description": "To alow a search (_include) on the materials used in nursing procedures, such as bandages.",
                "expression": "CarePlan.activity.detail.extension('http://nictiz.nl/fhir/StructureDefinition/extension-medicaldevice').value"
            }

{
    "resourceType": "OperationOutcome",
    "id": "a7be835d-3d8e-4e99-974f-daee7060ba99",
    "issue": [
        {
            "severity": "error",
            "code": "exception",
            "diagnostics": "An error occurred updating the custom search parameter.  The issue must be resolved and the update resubmitted to be applied."
        },
        {
            "severity": "error",
            "code": "exception",
            "diagnostics": "Value cannot be null. (Parameter 'uriString')"
        }
    ]
}

@CaitlinV39

@rotodd Would you mind taking a look at this one?

Hi, Is there any update on this issue?

AB#87251

@sasireha - are you still facing the issue? This is a good reference page : https://learn.microsoft.com/en-us/azure/healthcare-apis/fhir/how-to-do-custom-search.
Please let us know if you have additional questions. If we have no response - this issue will be closed (given it was created more than year ago)

Closing the issue due to no response


Delivery challans

[x] Crud
[x] Preferences
[x] Show delivery 'challan number' in creation screen.
[x] Add default challan types like supply of liquid gas, job work,supply on approval and others.
Notes: options changes in detailed view is not based on status. it works based on functions.
[x] In Delivered/returned status: 1.Remove edit, send mail option. 2. partial return option at the time of multiple item/quantity.
[x] Show list of invoices created from delivery challan.
[x] Need checking like sales order conversion quantity not more than invoiced(also need to consider quantity returned)
[x] More columns in listing.
[x] Path to preferences,manage custom fields.
[x] invoice details under open status delivery challans is showing due date as invalid

Completed


App Branding

This PR addresses

#13

Various assets are updated to give Liquid app it's own identity.
Changelist:

Updated App logo

and it's color on App Bar(Light Theme only)

Update Drawer footer image
Applied theme changes to splash screen on Android
Apply Breez theme on system navigation bar

There seems to be an inconsistency between the base and round/foreground images.

Round icon is used for app icon on launcher, foreground image is used on splash screen whenever possible.

@dangeross I've simplified resources, splash & launcher values are separated properly with no overlaps.


Code alignment is disorted when transaction block is placed inside try-catch

 Issue by erandidesilva
Thursday Oct 19, 2017 at 12:15 GMT
Originally opened as https://github.com/ballerina-lang/composer/issues/4008

Please see the attached image. The code generation is disorted when transaction is placed inside try-catch

This issue is not reproducible in 0.981.2-SNAPSHOT so closing this issue.


Added Expired and PendingOpen to ExchangeAPIOrderResult

This PR contains the following breaking changes to the ExchangeAPIOrderResult enum:

PendingOpen is an important state that many exchanges have, which is separate from Open
Renamed Pending state to Open
Renamed Error state to Rejected, since this is how most exchanges refer to it
Added Expired state, which is separate from Rejected

also contains the following changes

added stricter checks to FTX OnGetOrderDetailsAsync() and OnPlaceOrderAsync isPostOnly
improved FTX ToExchangeAPIOrderResult(), renamed to FTXExtensions to reduce possibility of collision in the future

Great stuff ;)


Style Guide

Design System: Design system created By google to help teams build high-quality digital experience.

Get an overview as well as resources for setting up your environment, implementing Material Components, and theming them to match your brand.

Content added [Style Guide]Added Material UI style guide. #9


Use TypedArrays instead of Nodejs' Buffer

Closes #4
This lets this parser be used from more environments, namely browsers. #4 has more context.
https://sindresorhus.com/blog/goodbye-nodejs-buffer has some context on why we should move on from NodeJS buffers.
Probably also closes #3, because I updated the instructions as a side effect and didn't see that PR.

Once you remove the change in the builder file, I'll merge this. Thanks again!


Idle scripts not fires when state changing

Perform all steps below and tick them with [x]

[x] Check the related part of the Documentation
[x] Update openHASP to the latest version
[x] Reproduce the issue and describe all steps

Describe the bug

#531 adds idle scripts.
but scripts not fires when idle state changing.
#[00:29:31.895][65524/75644 13][56344/56344  0] MQTT: Disconnected
#[00:29:36.557][65524/77228 15][56344/56344  0] HASP: First touch Disabled
**should run idle_off.cmd**
#[00:29:36.572][65524/75644 13][56344/56344  0] MQTT: Not connected ??? backlight => {"state":"on","brightness":3}
#[00:29:36.584][65524/77228 15][56344/56344  0] MQTT: Not connected ??? idle => off
**should run idle_short.cmd**
#[00:29:42.037][65524/77064 14][56344/56344  0] MQTT: Not connected ??? idle => short
#[00:29:43.906][65524/77228 15][56344/56344  0] MQTT: Transport error
#[00:29:43.917][65524/75644 13][56344/56344  0] MQTT: Disconnected

But run idle short command through Telnet/Serial will fire the script.
#idle short
[00:29:19.249][65524/77064 14][56344/56344  0] MSGR: idle=short
#idle shor[00:29:19.272][65524/75480 13][56344/56344  0] MSGR: Loading L:/idle_short.cmd
#idle shor[00:29:19.302][61428/70976 13][56344/56344  0] MSGR: backlight={"state": false, "brightness": 3}
#idle shor[00:29:19.313][59380/69392 14][56344/56344  0] HASP: First touch Enabled
#idle shor[00:29:19.328][61428/70976 13][56344/56344  0] MQTT: Not connected ??? backlight => {"state":"off","brightness":3}
#idle shor[00:29:19.341][59380/69392 14][56344/56344  0] MSGR: moodlight={"state": true, "color":"green"}
#idle shor[00:29:19.359][61428/70976 13][56344/56344  0] MQTT: Not connected ??? moodlight => {"state":"on","brightness":255,"color":"#008000","r":0,"g":128,"b":0}
#idle shor[00:29:19.371][59380/74408 20][56344/56344  0] MSGR: Loaded L:/idle_short.cmd
#idle shor[00:29:19.383][65524/77064 14][56344/56344  0] MQTT: Not connected ??? idle => short
#[00:29:19.862][65524/77228 15][56344/56344  0] MQTT: Transport error
#[00:29:19.873][65524/75644 13][56344/56344  0] MQTT: Disconnected

To Reproduce

Create L:/idle_short.cmd and etc.
backlight {"state": false, "brightness": 3}

Set Display Settings - Short Idle.
Telnet or Serial to the plate for logging.
Expected behavior

should run L:/idle_short.cmd automatically.
Screenshots or video

This issue should be resolved in RC9. Feel free to test and report back.


204 return on testChunks

After commit
The README says to return a HTTP code 204 for the testChunks GET requests when the chunk is not available on the server yet.
However, commit 8242cc862a uses 204 as a success code in the test request.
Which status code should now be used for indicating a missing chunk on the server?
Probably the README needs to be updated too.

+1

I guess technically the two could co-exist -- but you're absolutely right that its a very bad idea to do so. Bad judgement on my part and rolled back. Thanks you for bring this up.


How do miniCaffe used in MTCNN?

mtcnn need net_input_blobs_ and net_output_blobs_ but minicaffe not include, how change to mtcnn?

use net->blob_by_name() function to get data pointer from network internal blobs.
You can refer to the exampes.

use net->blob_by_name() function to get data pointer from network internal blobs.

I'm also interested in trying MTCNN with mini-caffe, but I still don't understand.
How can I replace it? using blob_by_name("input")?
also, input_blobs() return a vector of blobs, but blob_by_name returns a shared_ptr.
How should I use it?
Thanks!

I have used minicaffe implemented MTCNN : https://github.com/QiYueFeiXue/MTCNN/tree/run_minicaffe

I have solved this problem, look at this https://github.com/QiYueFeiXue/MTCNN/tree/run_minicaffe


CSS classes are missing when Rails runs in produciton

I'm looking for a guidance how to run svelte-rails in production.
My issue is when I precompile assets: RAILS_ENV=production bundle exec rake assets:precompile and run Rails in production mode: RAILS_SERVE_STATIC_FILES=yes RAILS_ENV=production rails s, Svelte components are loosing their classes that are defined in <style>...</style> section.
This issue is not happening if Rails runs in development. I'm using a preinstalled Hello.svelte component as a test.
Not sure if it gives a clue, but if I run RAILS_ENV=development ./bin/webpack & restart Rails production, the issue goes away.

Did you embed stylesheet/javascript packs in application?
    <%= javascript_packs_with_chunks_tag 'application' %>
    <%= stylesheet_packs_with_chunks_tag 'application' %>

Thanks! I think you're right, not sure how I missed it. Now, I'm getting a Rails error that css file is missing in manifest.json, but it must be a Rails specific issue. I'm trying to figure out how to resolve it. It seems to be some recent changes in Rails, i.e. https://github.com/rails/webpacker/issues/2605

I'll try again with a new Rails6 application and write down steps I followed, it's going to be easier to figure out what's missing.

Interesting enough a new Rails setup works as expected. I'll close this ticket out, must be a local configuration issue then. The only thing that could be worth mentioning in README is pack_tag, but I'll leave it up to you. Thanks for your help.


Support primitive types when resolving setter method

This is a bugfix for my previous Beans changes (#3) as I did not test setting primitive types on bean-like objects. Without this patch, boxed primitives returned by ConfigValue#unwrapped() are not assignable via reflection because the method lookup for setFoo(Integer) will not find setFoo(int). Other libraries, such as ClassUtils in Apache's commons-lang, have an extensive matching approach to find primitive methods given a boxed type.
To work around the problem and avoid unnecessary library dependencies, the method is looked up by name and arity rather than by name and argument type. The first method found will be invoked "blindly" (the JVM will unbox the argument as necessary). Note that when multiple setters are defined for a given property, which is selected may be non-deterministic. If the invocation fails, a ConfigException.WrongType is thrown. This approach should work for most beans, including my intended use with HikariCPDataSource, which I expect will have a single setter for a each property.
This patch also includes a check which throws ConfigException.BadPath if a setter with the requisite name is not found.
The unit tests have been updated to verify assignment of the following property types:

java.lang.Boolean/scala.Boolean
java.lang.Double/scala.Double
java.lang.Integer/scala.Int
java.util.List
java.lang.Long/scala.Long
java.lang.String

Commit has been revised to use a Java 6 and 7 compatible test for method arity as Method#getParameterCount was added in Java 8.

Thank you for your contribution!


FLEX-5084: Filters events on description.

SonarQube analysis reported 1 issue

 1 major

Watch the comments in this conversation to review them.


Limit max amount

blocked by https://github.com/AlpacaDB/backlight/pull/74 , because this PR is branched from it.
Actual diff is only this commit. https://github.com/AlpacaDB/backlight/pull/75/commits/29f1163dd7826f87755cc1f68acccd455f5a36fb

This PR does not break any api. I will merge it soon. Please add comments if you find any problems.


fix(v2): fix link to logo in footer

Motivation
If the target URL contains a slash at the end, the logo will not be displayed,
For example, the target URL is http://localhost:3000/blog/, then the logo URL will be http://localhost:3000/blog/img/logo.svg, not http://localhost:3000/img/logo.svg.
Have you read the Contributing Guidelines on pull requests?
Yes.
Test Plan
Related PRs

@endiliey done :heavy_check_mark:

Nice find, thanks for the fix @lex111!


_content/doc/play: fix loop bounds to match function comments

// pi launches n goroutines to compute an
// approximation of pi.

To match comments, launch n go routines instead of n+1.

@googlebot I signed it!

@googlebot I fixed it.


:bug: Make can configuration exception safe

Tested by building TOT of CanOpener with a 5.0.1 version of the library. Can confirm that this works.


Use --cmd instead of -c to set default filetype for vim

You are overriding the filetype set by the user using the -c option. This makes
it hard to set a filetype in the vimrc.

We set the filetype because we want files named PULLREQ_EDITMSG to get the same treatment as git commit messages.
How does setting the filetype hurt your workflow?

The problem was that you could not override the setting with a vimrc anymore, I changed my pull request and now it keeps the filetype setting but it allows overriding because of the --cmd option
This is from vim --help:
   --cmd <command>	Execute <command> before loading any vimrc file
   -c <command>		Execute <command> after loading the first file

Is there a reason why this is not merged? I changed te pull request to use another method to allow overriding the filetype, which keeps the default set by you.

Nice; sorry that I haven't seen your update back then. The workaround looks good. Let me try it myself locally and then I'll merge.


Sub domain routing?

What is the feature you are proposing?
Any plan to add sub domain routing?
For example -

api.example.com to go apiRoutes()
docs.example.com to go docsRoutes()

Here is an example in nestjs
https://docs.nestjs.com/controllers#sub-domain-routing

Hi, @vickyRathee!
In Hono, the getPath option and the route method can be used to define routing using the "Host" header as follows
import { Hono } from "hono"
import { getPath } from "hono/utils/url";

const app = new Hono({
  getPath: (req) => {
    const path = getPath(req)
    const host = req.headers.get('Host')
    // validate the value of the host header, if necessary.
    return `/${host}${path === '/' ? '' : path}`
  },
})

const app1 = new Hono()
app1.get('/', (c) => c.text('app1'))
app.route('app1.dev', app1)

const app2 = new Hono()
app2.get('/', (c) => c.text('app2'))
app.route('app2.dev', app2)

export default app

% curl -H 'Host: app1.dev' http://localhost:3000/
app1

thanks, I will use it this way.

This can be fixed. Thanks @usualoma

@yusukebe
This way the subdomain is added on each route using openapi middleware, any suggestion to remove that from the documentation?

@vickyRathee
I think there is no good way. The following is a workaround, but it's dirty:
app.get('<IP_ADDRESS>:8787/newDoc', async (c) => {
  const res = await fetch('http://<IP_ADDRESS>:8787/doc')
  const text = await res.text()
  const newText = text.replaceAll('<IP_ADDRESS>:8787/', '/')
  return c.json(JSON.parse(newText))
})


Update For Spree 4.1

Removed the coffee script requirement.
Fixed deface override missing admin config link.
Added dropship role functionality.
Improved discounts table in view to be more descriptive.

Thank you @MatthewKennedy 👍


support to exclude specific tags or edges when migrate space

Codecov Report
Patch and project coverage have no change.

Comparison is base (9bafb76) 31.33% compared to head (e8d5edf) 31.33%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff            @@
##             master      #93   +/-   ##
=========================================
  Coverage     31.33%   31.33%
  Complexity       78       78
=========================================
  Files            30       30
  Lines          1519     1519
  Branches        239      239
=========================================
  Hits            476      476
  Misses          973      973
  Partials         70       70

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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

please help to review this pr again, thanks @wey-gu


Error on npm run wasm

When I used the latest master branch, and run npm run wasm, I am getting the below error
Compiling wasm source for 3d-cube
docker run --rm -v C:\Users\Phanindra\Downloads\angular-wasm-master\angular-wasm-master:/src emscripten/emsdk emcc -Os src/app/wasm/3d-cube/soil/libSOIL.bc src/app/wasm/3d-cube/3d-cube.c -o src/assets/wasm/3d-cube.js -s LEGACY_GL_EMULATION=1 -Isrc/app/wasm/3d-cube/soil -s EXTRA_EXPORTED_RUNTIME_METHODS="['ccall']" -s FORCE_FILESYSTEM=1 -s MODULARIZE=1 -s EXPORT_NAME="Cube3dModule"
'docker' is not recognized as an internal or external command,
operable program or batch file.
child_process.js:674
throw err;

Hello. You need docker to build the WebAssembly examples.

Hello, I have docker installed and i still have this issue appearing. Any idea what could be the cause of it?
Thank you

After rechecking it tells me this : docker: invalid reference format.

I have no idea, to be honest. Could it be this https://stackoverflow.com/questions/45682010/docker-invalid-reference-format ?


dart2js: Eliminate warning for instance/static method clash

This is the dart2js specific issue for #31201, which has the details.

Closing: dart2js (version 2.1.0-dev.2.0) does not any longer emit the error that it should stop emitting (presumably because it is using the common front end), so this issue is now obsolete.
Here's the example that I used to check it:
class A {
  static int get foo => 42;
}

class B {
  set foo(int i) { print(i); }
}

main() {
  B().foo = A.foo;
}


Composer add dev tool Rector

Purpose
Rector help migrate zf1-future  to higher PHP version more efficient, consistent, safely
Near future help fix some common bug like:

Dynamic Properties are deprecated on PHP 8.2
Deprecated Passing null to parameter error on PHP 8.1

Work done

Added Rector to composer.json dev section
Setting rule CompleteDynamicPropertiesRector help fix issue 'PHP 8.2: Dynamic Properties are deprecated'

What next
After this PR merged, i will use ./bin/rector tests to fix 'PHP 8.2: Dynamic Properties are deprecated' tests source tests/Zend
Next, ./bin/rector library to fix PHP 8.2: Dynamic Properties are deprecated core source library/Zend
How to use
On the contributor's computer
composer install
./bin/rector tests
./bin/rector library

For contributor
Future, when need migrate zf-future to higher PHP version or fix some common bug please add

Add minimal rector rule into here ./rector.php
Run ./bin/rector
Make sure unit test run success on php 7.4 -> 8.x

docker run -e CI:true -u "$(id -u):$(id -g)" -w "$(pwd)" -v "$(pwd):$(pwd)" php:8.2-rc-cli-alpine -d memory_limit=-1 ./bin/phpunit -v --bootstrap tests/TestHelper.php tests/Zend/AllTests.php

docker run -e CI:true  -u "$(id -u):$(id -g)" -w "$(pwd)" -v "$(pwd):$(pwd)" php:8.1-cli-alpine  -d memory_limit=-1 ./bin/phpunit -v --bootstrap tests/TestHelper.php tests/Zend/AllTests.php

docker run -e CI:true  -u "$(id -u):$(id -g)" -w "$(pwd)" -v "$(pwd):$(pwd)" php:8.0-cli-alpine -d memory_limit=-1 ./bin/phpunit -v --bootstrap tests/TestHelper.php tests/Zend/AllTests.php

docker run  -e CI:true -u "$(id -u):$(id -g)" -w "$(pwd)" -v "$(pwd):$(pwd)" php:7.4-cli-alpine -d memory_limit=-1 ./bin/phpunit -v --bootstrap tests/TestHelper.php tests/Zend/AllTests.php

Reference:

https://github.com/rectorphp/rector
https://github.com/rectorphp/rector/blob/main/docs/rector_rules_overview.md#completedynamicpropertiesrector

Huh, seems github has lost counting who is new. or it doesn't count merged PR's as contribution?

Huh, seems github has lost counting who is new. or it doesn't count merged PR's as contribution?

Look like a bug's github or may be feature's github :D


fixed a possible typo

i think it was meant to be latest python 3 technologies

Thanks for this PR.

use the imperative form in commit messages "fixed" -> "fix".
don't use words like "probably", "possible", etc. If there's no typo, the patch won't be merged.
please, read the DCO and re-submit.


Parquet source doesn't respect time_unit parameter

related to #776
Parquet files are now loading, but they do not respect the time_unit parameter.  Example:
messages = await kd.sources.Parquet.create(
    "messages.parquet",
    time_column = "ts",
    key_column = "channel",
    time_unit = "s"
)
messages.preview(5)

_time
_key
thread_ts
ts
channel
user
text

0
1970-01-01 00:00:01.690783200
General
NaN
1690783200.000000
General
fb3da5bf
Good morning team! I hope everyone had a great...

1
1970-01-01 00:00:01.690783201
Project
NaN
1690783201.000000
Project
bb9d2b01
Good morning everyone! I had a relaxing weeken...

2
1970-01-01 00:00:01.690783202
General
NaN
1690783202.000000
General
03cc4325
Morning all! My weekend was good. I'm also loo...

3
1970-01-01 00:00:01.690783203
General
NaN
1690783203.000000
General
3e44cfa1
Hey everyone! Had a good weekend too. Ready to...

4
1970-01-01 00:00:01.690783204
General
NaN
1690783204.000000
General
ea27bbff
Morning team! Hope you all had a great weekend...

When loading the same file with jsonl, 2023-07-31 06:00:00 is the parsed _time for the first row

the test data for the above exists here: https://github.com/kaskada-ai/beep-gpt/blob/esp/generate_slack/slack-generation/messages.parquet


Add script path to -v argument info.

Checklist

[x] I'm reporting a feature request
[x] I've verified that I'm running youtube-dl version 2021.06.06
[x] I've searched the bugtracker for similar feature requests including closed ones

Description
If I want to get more info about youtube-dl I run youtube-dl -v which prints me all kinds of useful info. Would it be possible to add code there that prints out youtube-dl path on disk? For example  I installed it with  pip install --upgrade youtube_dl but I have hard time figuring out where the youtube-dl is located because the real life seems to be different from internet tutorials.
solution would be to print __file__ if it runs from script or sys.executable if it is compiled exe.

Might be useful for some users, I don't want to reveal my exact path to the public when I make issue reports and I don't want this.

A backtrace includes this information anyway. For instance, in #29720 we can see that the program is installed in /usr/local/bin/youtube-dl/youtube_dl/.
Perhaps --version and --verbose together could show the installation details.

Might be useful for some users, I don't want to reveal my exact path to the public when I make issue reports and I don't want this.

If it is there you can always remove it yourself but the opposite is not possible - I cant add there something I dont know.
For example, it is common that log files of many softwares contain exact paths. I often use text editor's built in replace all feature before posting them.


fix compilation errors in unikernel examples

With these minor adjustments, all of the unikernel examples compile
without errors.  Built against Mirage's "unix" target using OCaml
4.06.0.

Thanks for your PR, @mneilsen - highly appreciated.  I develop this library elsewhere, and had these issues already fixed in a separate branch, which I now pushed to GitHub #3.  This adds as well TLSA and SSHFP resource records.  Since my PR includes your changes, I'll close yours.

I also added https://github.com/roburio/udns/commit/a5bbd1eead94f4f7ffe935ac24e0444675f74c07 which automatically tests all the example unikernels to avoid breaking the examples in the future.


Add examples for using variables with Script Path

I have an inline script that references pipeline variables as follows:
call az group create --location uksouth --name $(resourcegroup)
call az appservice plan create --name $(appserviceplan) --resource-group $(resourcegroup) --sku B1
As soon as I switch to using a Script Path and using a .bat file the variables are not found. I suspect there is something simple that I am missing but have been unable to find the answer. Some examples of how to pass pipeline variables to the script file would be really useful.

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

ID: 8de3304e-98f9-bac0-3ccc-c65f68a89e87
Version Independent ID: 8fb2f6a9-545d-dff1-ff86-974694a85054
Content: Azure CLI task - Azure Pipelines
Content Source: docs/pipelines/tasks/deploy/azure-cli.md
Product: devops
Technology: devops-cicd
GitHub Login: @RoopeshNair
Microsoft Alias: ronai

The syntax for using these environment variables depends on the scripting language. Are you referring to them as in Batch script format?

Batch script: %VARIABLE_NAME%
PowerShell script: $env:VARIABLE_NAME
Bash script: $VARIABLE_NAME

That is great it was a simple syntax issue, I should have been using the batch script syntax. I think it would be worth linking to the documentation that describes the environment variable syntax for each scripting language.
Blog posts tend to use one of these without explicitly calling out the type of script being used. Is there a general page that contains this information? If not I would be happy to help create one.

Here it is: https://docs.microsoft.com/en-us/azure/devops/pipelines/process/variables?view=azure-devops&tabs=yaml%2Cbatch#set-variables-in-pipeline

Brilliant - thanks! :-)


Test voting endpoint just to make sure the civic API works in prod

Summary of changes

Asana Ticket: TICKET_NAME

General checks

[ ] Are the changes organized into self-contained commits with descriptive and well-formatted commit messages? This is a good practice that can facilitate easier reviews.
[ ] Testing. Do the changes include relevant passing updates to tests? This includes updating screenshots. Preferably tests are run locally to verify that there are no test failures created by these changes, before opening a PR.
[ ] Tech debt. Have you checked for tech debt you can address in the area you're working in? This can be a good time to address small issues, or create Asana tickets for larger issues.

New UI, or substantial UI changes

[ ] Cross-browser compatibility is less of an issue now that we're no longer supporting IE, but changes still need to work as expected in Safari, Chrome, and Firefox.
[ ] Are interactive elements accessible? This includes at minimum having relevant keyboard interactions and visible focus, but can also include verification with screen reader testing.
[ ] Other accessibility checks such as sufficient color constrast, or whether the layout holds up at 200% zoom level.

New endpoints, or non-trivial changes to current endpoints

[ ] Have we load-tested any new pages or internal API endpoints that will receive significant traffic? See load testing docs
[ ] If this change involves routes, does it work correctly with pertinent "unusual" routes such as the combined Green Line, Silver Line, Foxboro commuter rail, and single-direction bus routes like the 170?

This was only ever for testing purposes, and it's served said purposes. Thank you, and goodbye.


packspec vs specification_version

The field descriptions use packspec as the keyword in the schema. The examples use specification_version.
My 2 cents is I dislike specification_version as not being intent specific enough. I prefer packspec but would prefer something even more descriptive. packspec_schema_version is one suggestion.

packspec_version?

My 2 cents is I dislike specification_version as not being intent specific enough. I prefer packspec but would prefer something even more descriptive.

Examples were not updated, the schema is actually correct here, it supposed to be just packspec.
I personally like just packspec more. On top of declaring the actual specification version, the presence of the field itself can serve the purpose of "magic bytes", signifying that this is supposed to be a packspec file.


Feature Request / Discussion: Ballista Support

First of all thanks for this awesome Project!
I was wondering about the maintainers feeling about ballista as a distributed engine based on DataFusion.
Would switching to Ballista from "pure" DataFusion something that could be a direction of RoAPI in the Future?

yeah, i am open to PRs :)


emacsPackages.lsp-bridge: 20240516.145 -> 20240520.1548

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for emacsPackages.lsp-bridge is: A blazingly fast LSP client for Emacs
meta.homepage for emacsPackages.lsp-bridge is: https://github.com/manateelazycat/lsp-bridge
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 20240520.1548 with grep in /nix/store/rhb7d59f092l4xxk03bm37l61dsrsdr0-emacs-lsp-bridge-20240520.1548
found 20240520.1548 in filename of file in /nix/store/rhb7d59f092l4xxk03bm37l61dsrsdr0-emacs-lsp-bridge-20240520.1548

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

4 package rebuild(s)

First fifty rebuilds by attrpath

emacsPackages.acm
emacsPackages.acm-terminal
emacsPackages.lsp-bridge

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/rhb7d59f092l4xxk03bm37l61dsrsdr0-emacs-lsp-bridge-20240520.1548 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A emacsPackages.lsp-bridge https://github.com/r-ryantm/nixpkgs/archive/7906653799a3645d94439426db3af9eca91d9bf0.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/7906653799a3645d94439426db3af9eca91d9bf0#emacsPackages.lsp-bridge

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/rhb7d59f092l4xxk03bm37l61dsrsdr0-emacs-lsp-bridge-20240520.1548
ls -la /nix/store/rhb7d59f092l4xxk03bm37l61dsrsdr0-emacs-lsp-bridge-20240520.1548/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

3 packages built:

emacsPackages.acm
emacsPackages.acm-terminal
emacsPackages.lsp-bridge

Maintainer pings
cc @fxttr @kira-bruneau for testing.

Result of nixpkgs-review pr 313348 run on x86_64-linux 1

3 packages built:

emacsPackages.acm
emacsPackages.acm-terminal
emacsPackages.lsp-bridge


script can't input 2fa code properly

Looks like the script can't go to the beginning of the 2fa code field by pressing home anymore (tested manually, pressing home does nothing), so I made a workaround just by making it press left 10 times instead of home (I know half the keypresses are redundant, but just to be sure), which works fine so I thought i should make the issue known.

Still working just fine here. I wonder if it's a user agent issue somehow?
Try with Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:79.0) Gecko/20100101 Firefox/79.0


Performance: Typing is painting entire row

I was doing some benchmarking of the pre-release 0.2.19 build, and saw there is still one issue with the typing performance:

(You can reproduce this by using the 'Rendering' -> 'Paint Flashing' tool in the dev tools).
As we type, the entire row gets repainted - this is not ideal and is a cause for unnecessary latency in the typing operation.
It looks like there are two contributing factors:

The <KeyboardInput> we use to manage IME is causing paint, even though its opacity is 0. It looks like promoting it to a layer (transform: translateZ(0px)) solves this. It's strange that it's painting with opacity 0, though.
The canvas seems to be redrawing the entire string of tokens, instead of just the last one (as would be expected). Need to investigate further.

In addition, as part of this issue, we should investigate a proactive strategy to prevent this from regressing build-over-build. The webContents actually exposes a paint event: https://github.com/electron/electron/blob/master/docs/api/web-contents.md
A reasonable test case might be to validate that the paint rectangle does not exceed a certain area after typing.

As part of the performance test, we should also validate that we are adding elements continuously - this will help us catch issues like #1146 more quickly

Is this still an unsolved problem? Just curious

@bryphe where would be a place to look into this? Which file(s) use the KeyboardInput, or do you think the issue is mostly in the KeyboardInput itself?

@Bretley - actually, I don't think it makes too much sense to work on this now. With @Cryza 's super-fast WebGL work (#2120, #2198, #2199, #2207) , this isn't really necessary - I think in lieu of this, it's better to just invest in the WebGL renderer. I'll close this out, but let me know if you have questions or ideas.


[FLINK-25199][network] Make sure StreamEdges are unique

Previously, if there was a node that was self-unioned with itself,
it was creating a situation with two identical StreamEdges. Both
with the same partitioning, from the same source node to the same
target node.
This was causing issues when constructing output collectors and
picking the correct RecordWriters, as StreamTask was not able to
uniquely identify given StreamEdge and was assigning the same
RecordWriter to both of the edges. As a result all stream elements
were sent twice through the same RecordWriter. It was actually pretty
harmless apart of calculating the combined watermark downstream,
since all watermarks were always comming just from one single
edge/inputgate, and the unused edges were always stuck with
min watermark.
As a solution we are making sure that StreamEdges are unique
by introducing a uniqueId field, incremented for every pair
of StreamEdges connecting the same node.
Verifying this change
This change adds a new checkState for early detection of similar problems and a new ITCase.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

Thanks for the reviews.

I agree with @matriv the uniqueId could be an int in StreamEdge, but it's really minor.

I would prefer it to be long, so that it could be used in any way in the future, including a global incremental counter. But indeed, that’s a minor issue, and since both of you feel the same way, I've changed it to int.


SSL error on "Downloading detection model "

Bug

I cannot run converter.convert('...') due to failing to downloading the detection model. Specifically, I get the following exception:
URLError: <urlopen error [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1000)>

Steps to reproduce

Attempt to convert document with clean install
Docling version

(venv3.12) pipelines0$> docling --version
Docling version: 2.5.2
Docling Core version: 2.3.2
Docling IBM Models version: 2.0.3
Docling Parse version: 2.0.3
Python version

Python 3.12.6

@JensGM which OS do you use? It looks to me that you miss a local certifcate (see here).

@JensGM which OS do you use? It looks to me that you miss a local certifcate (see here).

Thanks, this was the issue. It was macos


Support mobile urls

Current version of ng-youtube-embed doesn't play well with mobile urls as video src
For example:
'https://www.youtube.com/watch?v=E813VYySueM' => works as expected
'https://m.youtube.com/watch?v=E813VYySueM' => broken

Thanks for reporting this issue, it has been fixed in v1.7.10


Update M05-Unit 6 Create a front door for a highly available web appl…

…ication using the Azure portal.md
Module: 00
Lab/Demo: 00
Fixes # .
Changes proposed in this pull request:

Submitted by cjpluta on behalf of MOC Labs Review


MSBuild integration doesn't work with

I've tried including this into WPF-Math, which uses multiple <TargetFrameworks>.
It turns out we have to put the .props and .targets files into buildCrossTargeting for that to work. See https://github.com/dotnet/msbuild/issues/1860 for details.

See also https://github.com/NuGet/Home/issues/4098: it looks like it is now called buildMultiTargeting?


New version of qpdf produces a different version of PDF than previous versions of qpdf.

Input PDF is %PDF-1.6
When using qpdf version 10.1.0 from debian bullseye:
qpdf --deterministic-id --recompress-flate --compression-level=9 --empty --pages "in.pdf" 1-z -- "out.pdf"

produces a PDF that is %PDF-1.3
When using appimage qpdf version 11.4.0:
qpdf --deterministic-id --recompress-flate --compression-level=9 --empty --pages "in.pdf" 1-z -- "out.pdf"

produces a PDF that is %PDF-1.6
The rest of the file is bit-for-bit identical (except for a different /ID in the files).
Why is PDF 1.6 generated, instead of the lowest compatible version?

What version is in.pdf? A bug was fixed prior to the release of 10.6.0 that took the highest version from all the input files when copying pages. Prior to 10.6, the version was copied from the initial input file, which in the case of --empty, is 1.3. S most likely in.pdf has version 1.6.

qpdf can't determine the lowest compatible version. It trusts the file headers.


README minor update

Description
Few minor changes
Checklist/ToDo's

[x ] Added documentation?
[ /] Added release note entry? (  docs/release-notes.md )
[x ] Tested?

Thank you for your contribution.


DNSConfig support on windows pods seems to not work?

Expected Behavior
custom dnsConfig options should work for windows containers.  this may or may not be a calico bug, depending on where we assign the requirement of container DNS.  Since the CNI.spec actually defines dns and there are issues hinting at this, we might
Current Behavior
On EKS with calico, dns records cant be injected through the dnsConfig field, as is done in normal linux containers.  I suspect this is simply a no-op currently in calico.  Interesting i found https://github.com/projectcalico/cni-plugin/issues/67, which seems to imply that at some point, dns for calico via the CNI provider... might be "a thing"?  But there wasnt sufficient motivation to implement it.
Maybe, @caseydavenport ... now with windows support, the time has arrived :) ?
Excuse me if i might be missing some context though, as things might have changed alot since 2016 since that original issue was filed.
In any case, here is what ipconfig sais inside a calico EKS windows pod, which has a custom<IP_ADDRESS> rule injected (note, the rule is not in there...),.   Also, if i checked the windows/system32/drivers/etc/hosts file, i also dont see this file.
Im not sure yet, wether the kubelet, or cni, should be responsible for DNSConfig inside windows nodes , so , maybe the answer to that question will really tell us wether this is a bug, or just a pheneomenon that the kubelet itself should be able to manage (for example, maybe it needs to make containerd/docker dns aware via some other mechanism).
      : No
   Autoconfiguration Enabled . . . .  : Disabled
   Connection-specific DNS Suffix Search List :
                                       dns-7718.svc.cluster.local
                                       svc.cluster.local
                                       cluster.local

Possible Solution

Have calico write data to /Windows/System32/drivers/etc/hosts ?
Have calico read in the CNI settings for DNS that ( i think ) might be plumbed in through the kubelet (not sure) ?
Have calico REJECT creation of pods which have DnsConfig requests at the CNI ADD level,  that way we 'fail closed'

Steps to Reproduce (for bugs)

setup a cluster on EKS with calico according to https://aws.amazon.com/blogs/containers/open-source-calico-for-windows-containers-on-amazon-eks/
Run the e2e test package  (i.e. for windows, i do wget https://storage.googleapis.com/jayunit100/content/e2e.test.exe, followed by  C:\Users\jayun\Dropbox\SYNC\executables\content\e2e.test.exe --provider=skeleton --kubeconfig=$HOME/.kube/config --ginkgo.focus="should support configurable pod DNS server"  --ginkgo.skip="Testpattern|Slow|plugin|SecurityContext" --dump-logs-on-failure=false --node-os-distro=windows
NOTE you must run the specific e2e.test.exe file above because there was a bug in upstream on EKS e2es, which is fixed there, so that the containers schedule properly :).  This should merge to k8s masterbranch soon.

Your Environment

Calico version 3.1.15
Kubernetes version 1.20

Feels wrong that this is handled by kubelet on Linux but gets passed down to the CNI plugin on Windows.  What business do we have modifying files inside the container?

Feels wrong that this is handled by kubelet on Linux but gets passed down to the CNI plugin on Windows.  What business do we have modifying files inside the container?

So, Evidently DNS is  part of the spec for CNI !

So, Evidently DNS is  part of the spec for CNI !

Can you link to the spec for dnsConfig?  Can't seem to find it.

Can you link to the spec for dnsConfig?  Can't seem to find it.

https://github.com/containernetworking/cni/blob/master/SPEC.md#parameters

https://github.com/containernetworking/cni/blob/master/SPEC.md#parameters

I think the issue is Calico CNI plugin uses hcsshim v1 API to program network endpoints. v1 API exposes DNSServerList and DNSSuffix as the only options for DNS config.
https://github.com/microsoft/hcsshim/blob/master/internal/hns/hnsendpoint.go#L21
Calico CNI plugin should switch to v2 API so all fields of DNSConfig can be programmed to network endpoints.
https://github.com/microsoft/hcsshim/blob/fd21b8d1922c7fb8b4a50c76d048fe1a69b7e7dc/hcn/hcnnetwork.go#L45
I would say it is bug.

I think the issue is Calico CNI plugin uses hcsshim v1 API to program network endpoints. v1 API exposes DNSServerList and DNSSuffix as the only options for DNS config.
https://github.com/microsoft/hcsshim/blob/master/internal/hns/hnsendpoint.go#L21
Calico CNI plugin should switch to v2 API so all fields of DNSConfig can be programmed to network endpoints.
https://github.com/microsoft/hcsshim/blob/fd21b8d1922c7fb8b4a50c76d048fe1a69b7e7dc/hcn/hcnnetwork.go#L45
I would say it is bug.

ahh ok ! thanks

ahh ok ! thanks

FWIW,  this is what the records should look lik.e..
      : No
   Autoconfiguration Enabled . . . .  : Disabled
   Connection-specific DNS Suffix Search List :
                                       resolv.conf.local

FWIW,  this is what the records should look lik.e..
      : No
   Autoconfiguration Enabled . . . .  : Disabled
   Connection-specific DNS Suffix Search List :
                                       resolv.conf.local

i just finished updating https://github.com/kubernetes/kubernetes/pull/97987 to expose this directly , just fyi might be useful as verifier

i just finished updating https://github.com/kubernetes/kubernetes/pull/97987 to expose this directly , just fyi might be useful as verifier

Thanks @jayunit100 for more details. My current understanding is there are two issues.

Calico CNI does not respect dnsConfig settings from a pod Spec. This should not be hard to fix. Once it is done, NameServers and Search list of dnsConfig can be supported. See definitions below.
Calico CNI uses hcsshim v1 API which has no way to program dns Options. It is not an easy task moving from v1 to v2. We would need plan ahead for this.

I suggest we fix the first one with Calico v3.18 release. This would support most use cases and allow k8s Windows e2e to pass. @caseydavenport @fasaxc
type PodDNSConfig struct {
	// A list of DNS name server IP addresses.
	// This will be appended to the base nameservers generated from DNSPolicy.
	// Duplicated nameservers will be removed.
	// +optional
	Nameservers []string `json:"nameservers,omitempty" protobuf:"bytes,1,rep,name=nameservers"`
	// A list of DNS search domains for host-name lookup.
	// This will be appended to the base search paths generated from DNSPolicy.
	// Duplicated search paths will be removed.
	// +optional
	Searches []string `json:"searches,omitempty" protobuf:"bytes,2,rep,name=searches"`
	// A list of DNS resolver options.
	// This will be merged with the base options generated from DNSPolicy.
	// Duplicated entries will be removed. Resolution options given in Options
	// will override those that appear in the base DNSPolicy.
	// +optional
	Options []PodDNSConfigOption `json:"options,omitempty" protobuf:"bytes,3,rep,name=options"`
}

Thanks @jayunit100 for more details. My current understanding is there are two issues.

Calico CNI does not respect dnsConfig settings from a pod Spec. This should not be hard to fix. Once it is done, NameServers and Search list of dnsConfig can be supported. See definitions below.
Calico CNI uses hcsshim v1 API which has no way to program dns Options. It is not an easy task moving from v1 to v2. We would need plan ahead for this.

I suggest we fix the first one with Calico v3.18 release. This would support most use cases and allow k8s Windows e2e to pass. @caseydavenport @fasaxc
type PodDNSConfig struct {
	// A list of DNS name server IP addresses.
	// This will be appended to the base nameservers generated from DNSPolicy.
	// Duplicated nameservers will be removed.
	// +optional
	Nameservers []string `json:"nameservers,omitempty" protobuf:"bytes,1,rep,name=nameservers"`
	// A list of DNS search domains for host-name lookup.
	// This will be appended to the base search paths generated from DNSPolicy.
	// Duplicated search paths will be removed.
	// +optional
	Searches []string `json:"searches,omitempty" protobuf:"bytes,2,rep,name=searches"`
	// A list of DNS resolver options.
	// This will be merged with the base options generated from DNSPolicy.
	// Duplicated entries will be removed. Resolution options given in Options
	// will override those that appear in the base DNSPolicy.
	// +optional
	Options []PodDNSConfigOption `json:"options,omitempty" protobuf:"bytes,3,rep,name=options"`
}

@jayunit100 Are you running your cluster on EKS? If that is the case, are you using AWS CNI on Windows nodes?

@jayunit100 Are you running your cluster on EKS? If that is the case, are you using AWS CNI on Windows nodes?

in my case, the eks cluster is using calico for the cni...

in my case, the eks cluster is using calico for the cni...

I have same behaviour in case of AWS CNI, basically dnsConfig are ignored for windows pod. Appreciate for any suggestions.


MINOR: [Python] Adding .hypothesis directory to gitignore

I'm not sure what changed but I've lately been seeing a .hypothesis directory get generated in my python dir.  I'm not entirely sure what sequence of steps causes it to reproduce either.

@westonpace Feel free to merge when CI passes!


Cool idea, if anyone is down to help?

I want to make the backtrack chams cover all ticks, not just the last one for example, masterloosers x88 backtrack chams or spirthacks backtrack chams.
https://cdn.discordapp.com/attachments/629365117930438667/638325082917175297/unknown.png

I've been trying too but with no luck :( hopefully somebody can help us out

supported

this is where backtrack chams are made:
https://github.com/danielkrupinski/Osiris/blob/master/Osiris/Hacks/Chams.cpp#L156-L166
all you need to do is create a loop and replace "record->back()" with the current record. Pretty easy if you know c++

In backtrack change:
hooks.modelRender.callOriginal<void, void*, void*, const ModelRenderInfo&, matrix3x4*>(21, ctx, state, info, record->at(x).matrix);
to:
						for (int x = 0; x < record->size(); x++) { 							hooks.modelRender.callOriginal<void, void*, void*, const ModelRenderInfo&, matrix3x4*>(21, ctx, state, info, record->at(x).matrix); 						}

@JDeu you should would be amazing

lmao, it's actually ez af

I have 0 experience with coding whatsoever

@JDeu your method works great for the masterlooser backtrack. Hopefully Daniel will fix the backtrack chams overlaying the enemy in the near future :)

Made a pull request #730
Think some of the formatting is messed and the option should be stored in Chams really but I'm lazy.


Option for disabling sanitization

Hello, I'm creating events on a different page and showing events on another calendar page, so all data is being checked for xss attacks already. I want to put onclick event to my custom detail popup but dompurify removes all events. A property is needed for disabling dompurify for good. Thx.

@serkanyazicioglu
I understand. but Are you putting your events to DOM elements to attributes directly?
<button class="button" onclick="{some script}">I'm button</button>

Then I strongly recommend using addEventListner instead. Here's why. It'll be a workaround for the current situation as well.
Besides that, I'll consider opt-out DOMPurify with an option.

I'm just passing data to schedules with body. So detail popups are still being created via tui calendar and html is being checked via DomPurify. Thank you for your hard work🙏

@serkanyazicioglu
Could you give me an example of data?
I assume that you're passing data as HTML strings to the calendar and that HTML strings contain the onclick attribute. This can be sanitized by DOMPurify because It is vulnerable(on DOMPurify's side).
And I'd like to know how did you implement your custom detail popup.
Did you disable the useDetailPopup option and assigned an event creating your own popup when clicking any schedules? or Did you just customize the style of the default detail popup?
Of course, the latter one will be sanitized by DOMPurify.

Sorry for late response. useDetailPopup is enabled on my case. I'm providing html to body for having a custom detail popup experience. disableClick and disableDblClick are both true because players can't use this calendar for creating events on this screen. Calendar is just readonly on my case. On detail popup there's only a button which has a click event and navigates to event detail.

@serkanyazicioglu
To clarify, I assume that your server sends the list of schedules like this.
// One of your schedule
{
  id: "schedule-id",
  title: "Meeting 1",
  // ...
  body: `<button onclick="goToEventDetail()">Go to the event detail</button>`
  // ...
}

If so, that's the case you're putting an event to the DOM directly. Which is not a good practice for me.
https://github.com/nhn/tui.calendar/issues/1004#issuecomment-1020636532

But I've been thinking about your request and I can modify the scope of sanitization. then your implementation won't be a problem anymore. Please understand that I cannot guarantee the release date.

Yep exactly like this.
No worries about the time, I just use the latest version before sanitization changes. Thank you...


Add empty portable-atomic-util crate

This adds an empty crate named portable-atomic-util.
For now, I'm considering putting the features proposed in #1 (#8) and #37 into this crate, but they may be merged into the main portable-atomic crate in the future.

FYI @notgull


Docker container cannot access EdgeTPU

Description
Hello, I am now doing project related to use EdgeTPU usb to making the inference of car flow and velocity, since I would like to test some code locally on my laptop (M1 sys mac), I feel like it's a good try with docker, however, when I set up the env like the tutorial on offical doc, the comtainer inside cannot access my TPU device, so I wonder it is the problem exisist on my laptop, since I also try the procedure of start a docker container on Ubntu and it just work fine.
Click to expand!
Issue Type
Support
Operating System
Mac OS
Coral Device
Dev Board, USB Accelerator, Enviro Board, Coral Camera
Other Devices
No response
Programming Language
Python 3.8
Relevant Log Output
No response

Please try the follow steps:
mkdir usb
cd usb
touch Dockerfile

Place the below content on Dockerfile
FROM debian:sid-slim
WORKDIR /home
ENV HOME /home
RUN cd ~
RUN apt-get -y update
RUN apt-get -y dist-upgrade
RUN DEBIAN_FRONTEND=noninteractive TZ=Etc/UTC apt-get -y install tzdata
RUN apt-get install -y git nano python3.9 python3.9-venv python3-pip wget usbutils curl unzip sudo python3.9-dev pkg-config udev

RUN echo "deb https://packages.cloud.google.com/apt coral-edgetpu-stable main" \
| tee /etc/apt/sources.list.d/coral-edgetpu.list
RUN curl https://packages.cloud.google.com/apt/doc/apt-key.gpg | apt-key add -
RUN apt-get update

RUN sudo apt-get install libedgetpu1-std

RUN mkdir coral
RUN git clone https://github.com/google-coral/pycoral.git coral/pycoral

RUN bash coral/pycoral/examples/install_requirements.sh classify_image.py

RUN python3.9 -m pip install --extra-index-url https://google-coral.github.io/py-repo/ pycoral~=2.0

RUN python3.9 -m pip install pyftdi pyserial mendel-development-tool

# docker build -t "coral-usb" .
# docker run -it --device=/dev/bus/usb:/dev/bus/usb coral-usb
# cd coral/pycoral
# python3.9 examples/classify_image.py   --model test_data/mobilenet_v2_1.0_224_inat_bird_quant_edgetpu.tflite    --labels test_data/inat_bird_labels.txt   --input test_data/parrot.jpg

docker build -t "coral-usb" .
docker run -it --device=/dev/bus/usb:/dev/bus/usb coral-usb
cd coral/pycoral
python3.9 examples/classify_image.py   --model test_data/mobilenet_v2_1.0_224_inat_bird_quant_edgetpu.tflite    --labels test_data/inat_bird_labels.txt   --input test_data/parrot.jpg

Here are the logs for  reference: https://github.com/hjonnala/snippets/blob/main/USB/docker_logs.txt


Incorrect Navigation

In the 'Configure customer-managed keys for an existing account' section, there is a slight navigation problem,
point 2:
instead of
On the Settings blade for the storage account, select Encryption. By default, key management is set to Microsoft-managed keys, as shown in the following image.
it should be
On the Under Security + networking, select Encryption. By default, key management is set to Microsoft-managed keys, as shown in the following image.

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

ID: 2834e9eb-aeda-730f-8b87-1d2cf9a5acc1
Version Independent ID: 2d9e4ded-b188-e8d2-2036-a0dfbd4d417c
Content: Configure cross-tenant customer-managed keys for an existing storage account - Azure Storage
Content Source: articles/storage/common/customer-managed-keys-configure-cross-tenant-existing-account.md
Service: storage
Sub-service: common
GitHub Login: @tamram
Microsoft Alias: tamram

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

@MayurSapaliga
Thanks for reporting this! We have created a PR for this issue and the changes should go live soon.


how to use "group by"  in influxdb

here is the case : I use influxdb to record the accesslog of  nginx ,I want to seek out the frequence of every different ip in a minute .how do I work the "influxdb sql" out ? I will appreciate it if you give me some tips?

@testzlx,
Because this isn't a feature request or bug, I'm closing this in favor of you submitting a support ticket. You can do that by emailing<EMAIL_ADDRESS>and we'll be happy to help you out.


Ospo noworkflow

Hi @carlosmalt,
I just noticed that date and lastmod date weren't the actual ones. This was causing my blog post not to be properly indexed among the other posts. Hope this fix the problem.
Best,
Jesse

Please use a fresh branch (i.e., after resyncing your fork with the origin) when creating a pull request. Otherwise your old commits will undo any updates others (including me) have contributed on top of your contributions. I'm closing this PR and cherry-picked 47ea10c and cdea703 in PR #250.


dynamicadaptiveparameters

for issue 17:  add methods which allow the values to be changed and also redo the necessary adaptive fitting steps

ok to review it.

Hopefully these methods still work OK and give sensible results?  (eg you could check by putting in print statements to check the values of self.bandwidth)

alpha, bandwidth do get updated when this function "kde.set_adaptive_parameter(newalp, newbw)" is called but printing kde.bandwidth returns just updated global bandwidth but not an array.

I fixed the docstring and comment formats but it would be good if you remember to use numpy style comments / docstrings.


Hi, I want to know what does notepad2-mod doing with network?

I was uesed notepad2-mod for many years, but I found it ues my network to do something today, and I don't know what does it doing, is there anybody can help me?

@XhmikosR
sorry, why closed?


Compatiblity with django2.0

I think within the new version of django-plans we should bring compatibility with dj2.0 and drop support for python2.x

Fixed in https://github.com/cypreess/django-plans/pull/67


Fix propTypes in NavBar

Fix warnings related to type mismatch.

Oops, i created a duplicate request.
@leoz can you change titleStyle PropTypes with this?
titleStyle: PropTypes.oneOfType([PropTypes.array, PropTypes.object])
so we can use an array for style without warning.

#688

Hi @sarovin , I guess our patches could be applied one on top of another as they fix different properties.

Introduced by #706.


Dropdown shows disabledHint when items is empty

See flutter/flutter#18770 for the original PR.
Some changes in this patch:

Removed the behavior that showed this when onChanged == null. Most
of dropdown's unit tests left onChanged as null and this broke them.
Can put it back in and update them if that was just brittleness in the
tests and we're not actively trying to preserve the previous behavior
in that case.
Added the items != null assert back in. This functionality works
just as well on an empty list as on null, and generally it's easier to
reason about objects when they're guaranteed to be non-null.
Fix a bug where disabledHint wasn't being given enough padding,
since items was empty.
Minor style changes.

Manually setting the CLA label to "yes" since the original author contributed their commit to this project in the previous PR.

We should also disable if onChanged is null, as with other widgets.

I agree with @Hixie also please allow null again as it was.

Btw why creating a new PR?

OK, I mentioned it in the PR comment but I'll break and fix the existing tests then @Hixie. They were passing in onChanged = null and then manipulating the dropdown, I thought it was likely deliberate behavior we wanted to keep backwards compatible instead of just the test missing params.
@escamoteur I made this a new one because of how Github ties PRs to branches. As far as I know the only way for flutter/flutter#18770 to be updated is for escamoteur/flutter/improved_dropdownbutton to have additional commits pushed to it. I can push these commits to your fork instead if you'd prefer that to a separate PR?

I you push them on my PR I see when my PR is merged :-)

OK, I'll move these commits into escamoteur/flutter/improved_dropdownbutton instead. :)


Spelling mistakes in Russian translation

Just installed ScreenToGif and found one spelling mistake. It would be foolish to make a pull requester for one, so I fixed everything I found. ✌

Thank you :)
How do you wish to be credit as author of this update? With our GitHub nickname or another name?

Yep. Qwitriy would be fine


Documenting AppCompat events in EDP

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

@DaniHalfin and @brianlic-msft - can one of you please take a look at these proposed additions to the diagnostic data content? This adds additional events. Thanks!

@DaniHalfin and @brianlic-msft - We would like to follow up on this. Can you please assist to check, approve and merge the content for this PR?
Thank you.

@mypil - we took the discussion on this offline. Going to hold on this PR for now.

Merged internally.


Use PHY Coded as Primary PHY for advertisement

I use softdevice 140 on nrf52840. I want to use Coded PHY as Primary PHY during Advertisement.
I configured it and used it like this:
        let config = peripheral::Config{
            primary_phy: Phy::Coded,
            secondary_phy: Phy::Coded,
...

let conn = unwrap!(peripheral::advertise_connectable(sd, adv, &config).await);

but I get this error when flushing to the device:
WARN  sd_ble_gap_adv_set_configure err InvalidParam
└─ <mod path> @ <file>:0
WARN  sd_ble_gap_adv_stop: BleInvalidAdvHandle
└─ <mod path> @ <file>:0
ERROR panicked at 'unwrap failed: peripheral :: advertise_connectable(sd, adv, & config).await'
error: `Raw(InvalidParam)`

According to the documentation Coded PHY should be supported as Primary PHY for advertisement.
I tried to dig into the code at https://github.com/embassy-rs/nrf-softdevice/blob/master/nrf-softdevice-s140/src/bindings.rs#L6073 but it just seem to be some wrapped assembly? I don't understand how this error is thrown or how it can be fixed.
Is there some knowledge around this?

coded must use extended advertising. this works for me:

let config = peripheral::Config {
    primary_phy: Phy::Coded,
    secondary_phy: Phy::Coded,
    tx_power: TxPower::Plus8dBm,
    interval: 1600, // 1s
    timeout: None,
    max_events: Some(1),
    filter_policy: peripheral::FilterPolicy::Any,
};
let data = peripheral::ConnectableAdvertisement::ExtendedNonscannableUndirected {
    adv_data: &adv_data,
    set_id: 0,
};

match peripheral::advertise_connectable(sd, data, config).await { ... }


Discussion: What's the right way to add timeouts for testing?

My students' will sometimes accidentally code infinite loops. As written, the entire test suite will time out when I test such code, which isn't as helpful as it could be.

What's the right way to flexibly provide good testing of students' code that may infinite loop? Here's what I'm currently using:

(require racket/engine)

(define timeout-symbol (gensym 'timeout))

(define-syntax-rule (run-w/timeout e)
  (let ([w/timeout (engine (λ (f) e))])
    (if (engine-run *TIMEOUT-MS* w/timeout)
        (engine-result w/timeout)
        '(timeout-symbol "test timed out"))))

I'm currently needing to wrap this around almost all my tests, which seems gross and wrong. In principle I'll want to timeout on almost any tests, b/c they could accidentally loop in any program they write.  It's a shame to have to set one single time-out.
@wilbowma , are you currently using https://gist.github.com/wilbowma/79330280f474ecc456916787028206cc ? Matthias suggests<EMAIL_ADDRESS>on
So what's the right API and a good standard implementation to support it?

This seems like something that should be a part of or an extension to rackunit, rather than hand-rolled each time. I assume there's an obvious reason rackunit doesn't include some solution for this already: what is that reason?

https://github.com/rymaju/gradescope-racket/commit/53c2eaf0638b9f1acaf0f29ccd6f1e33947f2041
This seems to work? Not sure if a more experienced Racketeer could chime in on whether this is robust enough. Of course this should be optional as well -- maybe through passing in some optional arguments / a parameter?

Unfortunately, it seems like the best way to do this would be to time out each individual test with something like racket/engine, I would have loved to run everything async, but that only works if you don't have any before or after setup/teardown.

Fwiw, @rymaju, I definitely like your proposal!

closing via #24


Detecting Bounding Boxes along with class labels realtime

I am using TF Lite Android Image Classifier App provided with this project with GPU support for Mobilenetv1. I want to know how I can detect the bounding boxes of inferred/detected objects along with the class labels?
Can anyone please tell me what modifications I have to make?

You may want to start from the object detection example first https://github.com/tensorflow/examples/tree/master/lite/examples/object_detection/android

@freedomtan Thanks a lot for the reply. It (https://github.com/tensorflow/examples/tree/master/lite/examples/object_detection/android) is working for the input size 224 when I am changing the resized bitmap size from 224 to 416 or 608 I am getting the following error
04-30 22:00:51.810 26345-26363/org.tensorflow.lite.examples.detection E/AndroidRuntime: FATAL EXCEPTION: inference
Process: org.tensorflow.lite.examples.detection, PID: 26345
java.lang.ArrayIndexOutOfBoundsException
at android.graphics.Bitmap.checkPixelsAccess(Bitmap.java:1718)
at android.graphics.Bitmap.getPixels(Bitmap.java:1659)
at org.tensorflow.lite.examples.detection.tflite.TFLiteObjectDetectionAPIModel.recognizeImage(TFLiteObjectDetectionAPIModel.java:156)
at org.tensorflow.lite.examples.detection.DetectorActivity$2.run(DetectorActivity.java:215)
at android.os.Handler.handleCallback(Handler.java:790)
at android.os.Handler.dispatchMessage(Handler.java:99)
at android.os.Looper.loop(Looper.java:164)
at android.os.HandlerThread.run(HandlerThread.java:65)
can anyone please tell me how to resolve this issue?

Please provide details about what platform you are using (operating system, architecture). Also include your TensorFlow version. Also, did you compile from source or install a binary?
Make sure you also include the exact command if possible to produce the output included in your test case. If you are unclear what to include see the issue template displayed in the Github new issue template.
We ask for this in the issue submission template, because it is really difficult to help without that information. Thanks!

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


Pasting from clipboard strips line endings

When pasting text from terminal, text I'm copying does not contain correct line endings - I simply get single line of content. I tried with WSL, Git Bash and PowerShell.
I have found plenty of issues related to copy-pasting - either closed as resolved (but this issue is still present) or not related to missing line ends (rather adding more line ends).
Environment

Platform ServicePack Version      VersionString
-------- ----------- -------      -------------
Win32NT             10.0.18363.0 Microsoft Windows NT 10.0.18363.0

Windows Terminal (Preview)
Version: 0.7.3291.0

Steps to reproduce

Having profiles:

        {
            "guid": "{00000000-0000-0000-0000-000000000001}",
            "name" : "Bash",
            "commandline" : "\"%PROGRAMFILES%\\git\\usr\\bin\\bash.exe\" -i -l",
            "hidden": false,
            "acrylicOpacity": 0.8,
            "fontFace": "Fira Code",
            "fontSize": 10,
            "padding": "4, 4, 4, 4",
            "useAcrylic": true,
            "closeOnExit" : true,
            "icon" : "%PROGRAMFILES%\\Git\\mingw64\\share\\git\\git-for-windows.ico",
            "startingDirectory" : "%USERPROFILE%"
        },
        {
            "guid": "{2c4de342-38b7-51cf-b940-2309a097f518}",
            "hidden": false,
            "name": "Ubuntu",
            "source": "Windows.Terminal.Wsl",
            "acrylicOpacity": 0.8,
            "fontFace": "Fira Code",
            "fontSize": 10,
            "padding": "4, 4, 4, 4",
            "useAcrylic": true
        },
        {
            "guid": "{61c54bbd-c2c6-5271-96e7-009a87ff44bf}",
            "name": "Windows PowerShell",
            "commandline": "powershell.exe",
            "hidden": false,
            "fontFace": "Fira Code",
            "fontSize": 10,
            "padding": "4, 4, 4, 4"
        },

... and keybindings:

    "keybindings": [
        { "command": "copy", "keys": ["ctrl+insert"] },
        { "command": "paste", "keys": ["shift+insert"] }
    ]

Open either WSL, Git Bash or PS terminal
Run:
Git Bash/WSL: curl -v -X GET https://www.example.com/
PS: (curl -v http://www.example.com/).RawContent
Select output, copy to clipboard (Ctrl+Insert)
Open notepad.exe / code / sublime / ... (text editor of your choice)
Paste from clipboard (Shift+Insert)

Expected behavior
Text copied from clipboard is well formatted, line endings respected:
curl -v -X GET https://www.example.com/
Note: Unnecessary use of -X or --request, GET is already inferred.
*
Trying <IP_ADDRESS>...
* Connected to www.example.com (<IP_ADDRESS>) port 443 (#0)
* found 148 certificates in /etc/ssl/certs/ca-certificates.crt
* found 592 certificates in /etc/ssl/certs
* ...
> GET / HTTP/1.1
> Host: www.example.com
> User-Agent: curl/7.47.0
> yada yada yada request headers
>
< HTTP/1.1 200 OK
< yada yada yada response headers
<
<!doctype html>
...

Actual behavior
Single line pasted:
curl -v -X GET https://www.example.com/                                                                                Note: Unnecessary use of -X or --request, GET is already inferred.                                                                                *   Trying <IP_ADDRESS>...                                                                                > GET / HTTP/1.1                                                                                > Host: www.example.com                                                                                > User-Agent: curl/7.47.0                                                                                > Accept: */*                                                                                >                                                                                < HTTP/1.1 200 OK                                                                                <!doctype html>.

Duplicate of #3367

@mkitzan I'm not sure whether that ticket refers to same issue:

It is about copying from terminal and pasting to another application (e.g. notepad)
not related to MSYS, I can reproduce it in Git Bash, WSL and PowerShell

Doesn't look like a pasting problem, because the paste will behave the same regardless of where you paste the text. It's a WT copying problem. When you copy multiple lines from WT, the text is copied as a single line (padded by spaces) instead of copying the line break. You can check out the hexdump of this behavior here.

@mkitzan oh, I see. I found other issues (and followed links) - seems this is quite recurring theme. If we only used same words for describing same thing 🤞 -> closing this one.


RuntimeWarning for channels with different filter settings on import

Recently (as of #12441), MNE started to raise warnings when importing EDF/BDF files containing channels with different filter settings (see here):
RuntimeWarning: Channels contain different highpass filters. Highest filter setting will be stored.
RuntimeWarning: Channels contain different lowpass filters. Lowest filter setting will be stored.

There are three issues with this behavior:

All channels are used to check for identical filter settings. This is not a good idea, since different channel types are likely to have different filter settings. In particular, STIM channels are also considered in this comparison, so BDF files always trigger the warning (STIM channels contain empty filter strings).
I think the warning message (and possibly also the resulting behavior) is incorrect. IMO, if channels (of the same type) contain different highpass filters, the lowest filter setting should be stored. Similarly, for different lowpass filters, the highest setting should be stored. These correspond to the weakest thresholds, and this is how it's done in read_raw_brainvision() (see here and here). Interestingly, read_raw_nsx() also seems to do the wrong thing (see here).
In my opinion, raising a RuntimeWarning is too much. Even if the check is only performed within (certain) channel types, I would only log to info.

It looks like those warnings have been in place for 9 years now. Rather than change their behavior to info now, I'd rather fix (1) and (2). To suppress the warnings (in the meantime or later) you could for example also read with verbose="error".

They were in place for so long without being noticed most likely. I've never encountered a BrainVision file that triggered the warning, and the other format is very niche.
I still think different filter settings do not warrant RuntimeWarnings. Why do you think this is so important that an info message is not sufficient? In other words, do you have a reason other than the age?

And for EDF/BDF these warnings are new, so if you don't want to change the warnings for the other formats, then let's either do it just for EDF/BDF or revert that change entirely.

They were in place for so long without being noticed most likely.

I seem to recall there being other PRs that were similar, and IIRC there have been some tests in place for them. But I'd need to look.

Why do you think this is so important that an info message is not sufficient? In other words, do you have a reason other than the age?

Well one thing to keep in mind is that age is at least some justification -- it suggests that behaviors have been around a while with expectations and experiences suggesting that they're somewhere between tolerable and useful. But beyond that, in general when we detect something is wrong/inconsistent/un-representable-in-MNE for a file that could lead to some bad behavior, emitting a warning is a reasonable thing to do, as it's much more visible than info. In the case of wrong/incompatible filter settings (e.g., you think your lowpass for all channels in X when really it's Y for some of them) we tend to emit warnings as you point out above. I think this PR from a few years ago that adds a warning about filtering has a similar idea behind it as well:
https://github.com/mne-tools/mne-python/pull/8584/files#diff-1d508c4fe7f39a5d09e8cd44e2996647850b3398a3813e37e9e97654581b689cR477
So to me the right thing to do here is improve the specificity of the warning as in (1) and (2).

I think there's a big difference between incompatible/wrong filter settings vs. just different filter settings. What exactly are the consequences of not knowing that a common cutoff frequency has been set that is compatible with all filter settings? The signals are not affected at all, it's just some meta info field that MNE is setting. And people would still be able to see it in info, which is the default logging level.
But okay, since it is not likely that I'll convince you, someone should take care of (1) and (2). I don't have the time, so if no one can do it soonish I'd still revert the EDF/BDF change as an intermediate fix. Right now, this warning is triggered for all BDF files.

Yeah if that's the case then that's not good. I'm suggesting that we should make it useful rather than eliminate it in all cases though.

This sounds reasonable to me!!

I was trying to argue for a quick hotfix (possibly to be released in 1.7.1), because I currently don't have the time to look into these two issues. I can only revert the warning for EDF/BDF, but if you would like to have a proper fix (i.e. make the warning actually useful and also do the right thing), then someone else would have to implement it in the near future. Again, this warning is triggered for every BDF file no matter what.

I can reproduce it, it is an interaction with the exclude parameter. I'll try to figure out what's wrong.

Since the warning has been fixed, there are two remaining issues:

Should we check for identical filter setting within channel types? Currently, all channels (except STIM channels) are used. I'd say YAGNI for now (because we could assume the majority of EDF/BDF files contain just one channel type).
What happens if the channels are associated with different filter settings? The behavior of read_raw_brainvision() is inconsistent with read_raw_{e|b}df() and read_raw_nsx(). Whereas the former uses the lowest (highest) cutoff frequency for different highpass (lowpass) filters, the latter use the highest (lowest) cutoff frequencies for different highpass (lowpass) filters. What is the correct approach here?

Should we check for identical filter setting within channel types?

I think we might for other readers but no strong feeling

What happens if the channels are associated with different filter settings?

I would go with whichever one we had the most extensive discussions about. Not sure offhand which one that is, will require some GitHub archaeology


pika-init bin is outdated on npm

I tried to install the Typescript template, but it didn't work due to missing files. The version of the binary is 1.1.2. The --help command also is different than on the readme:
❯ pika-init --help
Usage: pika-init --template [template] <project-name>

Options:
  -V, --version              output the version number
  -t, --template [template]  template choice (optional)
  -h, --help                 output usage information

Examples:
  $ pika-init --template app-preact my-new-app
  $ pika-init my-new-app

    If you have any problems, do not hesitate to file an issue:
      https://github.com/pikapkg/init/issues/new

After cloning and installing it locally, the bin looks and fails the same way.

@ricardomatias can you please elaborate? How does it fail?
I believe the version currently on npm does in fact have a "typo" in its config.json, its fixed in master on here though (c6575553f133d469943a3a969e47b3def6bcb0b6).
the typescript example just built fine for me however.

@ndom91 Same here, the typescript template does not exist in npm<EMAIL_ADDRESS>

@FredKSchott when you have some time, could you please push the current master to npm? Thanks!

Done!


Please release 0.6.1

I'd be most grateful if a new release 0.6.1 (or 0.7?) could be made to crates.io as we'd like to publish a crate ourselves that requires 28e50672800f6f456cf8be53825001592d5a4eb3 and therefore we'd need a version of glow on crates.io that has the API.
Thanks in advance! :)

0.6.1 is published now: https://crates.io/crates/glow/0.6.1


[bug]it seem not explort uv2  from model .

it seem not explort uv2  from model .
according  the docs, it's can be show more than one channels options ,
but my model show nothing .

I'm sure My model have 2 channels.May be it's a bug.

Hi @sonygod,
Are both UVs imported and used in Unity ?
Would if be possible to share the sketchfab URL so that I can check what has been exported and why you don't have the switch option ?
Thanks,

yes ,both uv use in unity.
https://skfb.ly/69TOt

sorry,this model have no uv2 ,but there are some other issues.
the maintexture is not upload .
here is the originally  result

but the gltf is not contain the maintexture of each meshrender

https://github.com/sonygod/unityAssets/blob/master/Assets/testmodel.unitypackage
here is source package.

https://skfb.ly/69Vvv
this model I'm sure have 2 uv channels

hi,it's seem uv0,and uv1 was revert each other.
https://skfb.ly/69VEw
here is uv1

uv0

and in  unity is use uv1 instead uv0,
what's wrong about that ?

Hi,
Sketchfab is handling UVs a such way that the index is often not relevant (i.e the order in the geometry is often different from the order in the dropdown on Sketchfab) , we are aware of this and it could be improved.
Most of the time, everything should be set correctly so that you don't need to go in the 3D editor to fix settings like that.
I downloaded your sample and will take a look, because this should be a seamless workflow.
Thanks for the samples and details!

thank you for your attention,
if you have time,please consider support our custom shader  "Standard2" was include in sample package.


build(dependencies): 🧱 update crates

Description
Update dependencies:
Updating cc v1.0.104 -> v1.0.106
Updating clap v4.5.8 -> v4.5.9
Updating clap_builder v4.5.8 -> v4.5.9
Updating rustls v0.23.10 -> v0.23.11
Updating rustls-webpki v0.102.4 -> v0.102.5
Updating serde v1.0.203 -> v1.0.204
Updating serde_derive v1.0.203 -> v1.0.204
Updating syn v2.0.68 -> v2.0.70
Updating tinyvec v1.6.1 -> v1.8.0
Updating toml_edit v0.22.14 -> v0.22.15
Updating windows-targets v0.52.5 -> v0.52.6
Updating windows_aarch64_gnullvm v0.52.5 -> v0.52.6
Updating windows_aarch64_msvc v0.52.5 -> v0.52.6
Updating windows_i686_gnu v0.52.5 -> v0.52.6
Updating windows_i686_gnullvm v0.52.5 -> v0.52.6
Updating windows_i686_msvc v0.52.5 -> v0.52.6
Updating windows_x86_64_gnu v0.52.5 -> v0.52.6
Updating windows_x86_64_gnullvm v0.52.5 -> v0.52.6
Updating windows_x86_64_msvc v0.52.5 -> v0.52.6
Updating zerocopy v0.7.34 -> v0.7.35
Updating zerocopy-derive v0.7.34 -> v0.7.35

Type of change

[X] Dependency update

How Has This Been Tested?

[X] cargo test run with all tests passing

Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[X] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes
[X] Any dependent changes have been merged and published in downstream modules

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 36.55%. Comparing base (d89dcf1) to head (f5ea266).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #100   +/-   ##
=======================================
  Coverage   36.55%   36.55%
=======================================
  Files           4        4
  Lines        1160     1160
=======================================
  Hits          424      424
  Misses        736      736

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


MM - 35835 show cc info in purchase modal

Summary
NOTE: temporarily base is MM-35435-... branch. since this work is a continuation of that one.
This PR adds the Credit card information on the first load of the purchase modal. It shows also an edit button which when clicked will display the payment form.
Ticket Link
https://mattermost.atlassian.net/browse/MM-35835
Screenshots

When clicking "Edit" button:

Release Note
NONE

@pablovelezvidal as discussed today, we need to add which is the current plan they are in when they already have a payment method. Will you include this in this PR?

yes, I will update this PR asap with those changes.

The current plan tag looks a little low maybe - is it aligned with that Starter plan?

The current plan tag looks a little low maybe - is it aligned with that Starter plan?

yes, I just pull the badge 5 pixels up and looks more aligned. Thoughts @annelieseklein ?

looks good - thanks pablo!

Also the title should be 16px away from the first plan


Add e2e tests using gRPC and REST

Changes
Add e2e test for ListRecords, ListResults, GetRecord, GetResult, DeleteRecord, DeleteResult, Authorization, Authentication and Impersonation using both gRPC and REST.
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you review them:

[ ] Has Docs included if any changes are user facing
[ ] Has Tests included if any functionality added or changed
[x] Tested your changes locally (if this is a code change)
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including functionality, content, code)
[x] Has a kind label. You can add a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[x] Release notes block below has been updated with any user-facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
[x] Release notes contain the string "action required" if the change requires additional action from users switching to the new release

Release Notes
NONE

/kind misc

/test all

/assign @adambkaplan

/approve
Marking conditional approval so other contributors can add lgtm. The impersonation RBAC needs to be moved to our test overlay/kustomization, and I have a question about the tls hostname override. Otherwise looks good.

@tektoncd/results-collaborators this PR is ready for an LGTM review.

cc @tektoncd/results-maintainers

@khrm @enarha This should be in the release, as features from this release and previous release are cover in these e2e tests.

/lgtm