Composer: Problems configuring a Composer app when services have routes with the same name

Conflicting path error doesn't provide any context
Error: Path "/" already exists
    at composeOpenApi (/Users/simonplend/dev/github/platformatic/sandboxes/runtime-test/node_modules/@platformatic/composer/lib/openapi-composer.js:20:15)
    at composeOpenAPI (/Users/simonplend/dev/github/platformatic/sandboxes/runtime-test/node_modules/@platformatic/composer/lib/openapi.js:56:33)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

Doesn't explicitly tell you this is a Composer error (needs to be inferred by reading the stack trace)
Doesn't tell you which service the path is already registered for or which service it just attempted to register the path for
Doesn't offer any guidance on how to resolve the conflict e.g. add openapi.prefix settings for each service

Composer service won't start even if two services with the same routes are configured to use prefixes
Crashes with error:
[
  {
    "instancePath": "/paths",
    "schemaPath": "#/additionalProperties",
    "keyword": "additionalProperties",
    "params": {
      "additionalProperty": "tangy/"
    },
    "message": "must NOT have additional properties"
  }
]
Error: 'specification' parameter must contain a valid version 2.0 or 3.0.x or 3.1.x specification
    at Parser.parse (file:///Users/simonplend/dev/github/platformatic/sandboxes/runtime-test/node_modules/fastify-openapi-glue/lib/Parser.js?ts=707084131378375:49:10)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async plugin (file:///Users/simonplend/dev/github/platformatic/sandboxes/runtime-test/node_modules/fastify-openapi-glue/index.js?ts=707084131378375:118:17)

This needs a better error message


Update meta tags.

This PR consists of quality of life updates and fixes to meta tags.

Though this does not really do anything since they're not server-side rendered ...


Database Support for forms and various registrations

Description
Added an image in assets to show DB Schema
Related Issues

Closes #705

Type of PR

[ ] Bug fix
[x] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
[Attach any relevant screenshots or videos demonstrating the changes]

Checklist

[x] I have gone through the contributing guide
[x] I have updated my branch and synced it with project main branch before making this PR
[x] I have performed a self-review of my code
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[x] I have commented my code, particularly in hard-to-understand areas.

@Pruthaa17 User can have some more additional fields like age. Refer what we taking in appointment form for it and appointment table and user table can have 1-1 relationship

Next to Hospital to labs, clinics 1-many

@Pruthaa17 It now makes sense!
Thanks for this!
Would you also add support for backend using PostgreSQL?

@varshith257 I will create a new pull request ,  by mistake i deleted the branch

@Pruthaa17 It now makes sense! Thanks for this! Would you also add support for backend using PostgreSQL?

Yes I'll

Why was this deleted? Make a new PR

Why was this deleted? Make a new PR

#724


Don't save unhashed password on profile form.

What's this PR do?
Fixes a bug with the user profile form saving incorrect data to the user's account.
How should this be reviewed?
Does it only save the things we want it to?
Any background context you want to provide?
🍃
Relevant tickets
Fixes #6444.
Checklist

[ ] Documentation added for new features/changed endpoints.

🚢


d_conv&expand

Hi, author,
How will d_conv and expand these two paramet  How will d_conv and expand these two parameters affect the performance of Mamba?ers affect the performance of Mamba?

d_conv=3 or d_conv=4 didn't seem to make a difference in quality.
expand will definitely change the number of parameters, you'd want to control the total number of parameters (e.g. by increasing depth or d_model). expand=2 seems to work well and easy to match Transformer's number of params (2 Mamba layers having around the same number of params as MHA + MLP).

tks!

thanks for the awsome work @tridao would you mind explain (I suggest probably in the readme as well will be helpful for later reference) what expand and d_state and d_conv actually means to the model?

expand changes the model's width inside the block, this is described in Sec 3.4 of the original Mamba paper (and you can read the Block's code)
d_state is the state expansion factor, you'll have to read the papers for details
d_conv is the convolution kernel width (the causal conv1d inside the block before the main SSM layer)


How to generate server_x_test.txt and server_y_test.txt for own dataset?

Hello, I am trying to apply ClusterFL to my HAR dataset. I am confused about how to create "server_x_test.txt" and "server_y_test.txt" for the global model. Based on the source code, this data is used for calculating KL distance. However, given that we run the model in the FL setting, should all the data be kept on the client side? How do I generate those data for the server?

Hi Felix! Sorry for the late reply. The data "server_x_test.txt" is just located on the server to evaluate the clustering relationship of nodes using their model output on the data. In our experiments, we just randomly select a small amount of data samples from each node to make the global data unbiased. Note that we exclude the selected data from the local training and testing set of nodes. We have also tried some synthetic data that is randomly generated (with the same dimension as the real data samples), which is also shown to be able to be effective. You can try both on your data.
Hope this addresses your problems. Thanks!


[editorial] The specification for just, just_error, and just_stopped can be folded together

See how the just* factories are defined in the reference implementation.

This can be closed with #439


fix(chat): fix chat input height (Issue #1150)

Description:
fix chat input height
Issues:

Issue #1150

UI changes

Checklist:

[x] the pull request name complies with Conventional Commits
[x] the pull request name starts with fix(<scope>):, feat(<scope>):, feature(<scope>):, chore(<scope>):, hotfix(<scope>): or e2e(<scope>):. If contains breaking changes then the pull request name must start with fix(<scope>)!:, feat(<scope>)!:, feature(<scope>)!:, chore(<scope>)!:, hotfix(<scope>)!: or e2e(<scope>)!: where <scope> is name of affected project: chat, chat-e2e, overlay, shared, sandbox-overlay, etc.
[x] the pull request name ends with (Issue #<TICKET_ID>) (comma-separated list of issues)
[x] I confirm that do not share any confidential information like API keys or any other secrets and private URLs

/deploy-review


[eroprofile] Add album downloader

Before submitting a pull request make sure you have:

[x] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense

What is the purpose of your pull request?

[x] New extractor

Description of your pull request and other information
This adds an album extractor to adult website EroProfile.
I don't think it needs to handle authentication, as each video should handle logging in on its own.

I believe I have fixed it, @pukkandan.
There is one odd thing with this: the output message order. Is this okay?
[debug] [EroProfile:album] Extracting URL: https://www.eroprofile.com/m/videos/album/....
[EroProfile:album] ...: Downloading playlist page 1
[download] Downloading playlist: ...
[EroProfile:album] ...: Downloading playlist page 2
[EroProfile:album] ...: Downloading playlist page 3

@pukkandan it seems that the yield refactor caused it to skip over pages. Hopefully I will have a second PR to fix this soon.


Handle multiple web sockets at once

This doesn't pass the build and has a merge conflict.
Why do you want to handle multiple WebSockets?

Hi. The user may have a URL open in more than one tab or window, either deliberately or accidentally. Without handling multiple sockets, the module will only update one of those tabs or windows – which not only limits the potential use cases but is confusing if the active tab or window is hidden from view.

The behavior you described is already handled by reload. As long as every webpage includes the reload script tag. The client side will connect to the socket the server has created in every client instance. I just tested this now again in one of my projects with two different tabs in two different windows

Hi @alallier! I’ve rebased this against the current master, and added a proper explanation in the commit message.
The short answer to your question is that indeed multiple windows have always been able to be signalled by stopping the server. The .reload() method, on the other hand, was only sending a message to the most-recently connected one.
PS.

Ahh okay I understand now! Thanks for clarifying. Will review soon. Sorry for the time delay in general on this.

Pasting commit message from 9b7fd0eac503e0cba4b46a596195a5395d5d0c26 for future clarification

Sometimes I (accidentally or deliberately) have more than one browser
window or tab open to my app, and in that case I would like all of
them to refresh when I change the code.
The old behaviour is that just one client, the last to connect to the
websocket server, would refresh when the reloadServer.reload() method
is called.
Note that this only changes the behaviour when the reloadServer.reload()
function is used: it would always behave correctly if the clients are signalled
by stopping the server.
This PR changes (an undocumented property of) the public API, in that the
.connection object is replaced by a .connections object. HOWEVER, this
change will not break any existing code, for the simple reason that previously
the .connection object was always undefined! The code was returning it before
it had been assigned: it was assigned in the handler for the wss "connection" event.

My pull request totally solves this in a more practically manor by using global socket messages. Please review and I think you will agree my pull request supersedes : https://github.com/jprichardson/reload/pull/62

Sorry this took so long, but thank you!

Reviewed the code. Why wouldn't you all just use the built in clients array to broadcast to all websocket clients???
  var wss = new WebSocketServer({ server: httpServer })

  var reloader = function(){
    wss.clients.forEach(function each(client){
      if (client.readyState === WebSocket.OPEN){
        client.send('reload')
      }
    })
  }

Seems soooooo much simplier and does not require event listening on sockets close.

Thanks, @AckerApple! I hadn’t spotted that WebSocketServer maintains an array of connected clients by default.
Out of interest, can you reproduce a situation where a client is in the array but doesn’t have an OPEN readyState? Or is that just a belt-and-braces check?

@robinhouston, you're welcome.
This documentation, for ws, should help give you the info you seek: https://www.npmjs.com/package/ws#broadcast-example
@robinhouston, I forked this reload library and made a large pull request for the betterment of this package but it was ultimately denied. I'm a little bitter, little rubbed the wrong way. I have published my fork on npm here : https://www.npmjs.com/package/ack-reload
Such as life. Try ack-reload. Tell your friends, I can't keep "trolling" this package.

Thanks @AckerApple for pointing this out. We can re-factor to use the native array.


Create README.md

Why:

What's being changed:

Check off the following:

[x] All of the tests are passing.
[x] I have reviewed my changes in staging. (look for the deploy-to-heroku link in your pull request, then click View deployment)
[x] For content changes, I have reviewed the localization checklist
[x] For content changes, I have reviewed the Content style guide for GitHub Docs.

วัน'เดอร์' เขียว'ววววว

<EMAIL_ADDRESS>

0645018601-patch-1

0645018601-patch-1


Add events to CacheResponseSubscriber

Please describe the feature you would like to see implemented.
Currently it is quity hacky (override the set cookie on the response event) to add extra 'layers' to the context cookie.
Adding 2 events in CacheResponseSubscriber would make this job a lot easier.

buildCacheHash
getSystemStates

Adding these events will make it possible for a plugin to hook into the cache key generation easily. This is needed for example when a plugin uses some custom code to alter the storefront for certain customers.
Currently, if these customers have the same active Rules, currency, ... as other customers, the cache will be shared between them. Adding these events can fix this for these plugins.

Sounds good. Do you want to work on that?


fix(@angular/cli): Angular Compiler CLI v5 requires TypeScript >= 2.4.2

May be wait a little bit more? angular/angular#19966 is going to allow TypeScript 2.5 compilation.

@sarunint That PR is not sufficient. The Compiler team will let me know when they fully support 2.5.

I see. Thanks a lot for information.


Add new document reusult1

Testing Check Open Api version by using DocumentTests.css

Testing Check Open Api version by using DocumentTests.css

How about adding a new test instead of changing it?


Rename \pinclude et al

In order to beautify a bit the commands for including etc, introduce new names for the different include commands:
\incl "blah.ly"
\inclOnce "blah.ly"
\condIncl ##f "blah.ly"
\condInclOnce ##f "blah.ly"

The old commands pinclude etc should continue working. They should be deprecated (produce warnings) as of v1.5.0, and removed as of v2.0.0.

See also #49, which is related.


conttable: yLabel property in response is the wrong type

The yLabel property of the response is a string[][]. It should just be a string[]. The labels are repeated. This appears to be an issue for both 2x2 and RxC.
E.g.,
{
	"yLabel": [
		[
			"Bury",
			"Bush/field/ground/stream/open sewer (includes dumping anywhere in environment outside compound)",
			"Do nothing",
			"Other",
			"Scatter in yard",
			"Toilet, latrine",
			"Unknown",
			"Ventilated improved pit with water seal"
		],
		[
			"Bury",
			"Bush/field/ground/stream/open sewer (includes dumping anywhere in environment outside compound)",
			"Do nothing",
			"Other",
			"Scatter in yard",
			"Toilet, latrine",
			"Unknown",
			"Ventilated improved pit with water seal"
		],
		[
			"Bury",
			"Bush/field/ground/stream/open sewer (includes dumping anywhere in environment outside compound)",
			"Do nothing",
			"Other",
			"Scatter in yard",
			"Toilet, latrine",
			"Unknown",
			"Ventilated improved pit with water seal"
		],
		[
			"Bury",
			"Bush/field/ground/stream/open sewer (includes dumping anywhere in environment outside compound)",
			"Do nothing",
			"Other",
			"Scatter in yard",
			"Toilet, latrine",
			"Unknown",
			"Ventilated improved pit with water seal"
		],
		[
			"Bury",
			"Bush/field/ground/stream/open sewer (includes dumping anywhere in environment outside compound)",
			"Do nothing",
			"Other",
			"Scatter in yard",
			"Toilet, latrine",
			"Unknown",
			"Ventilated improved pit with water seal"
		],
		[
			"Bury",
			"Bush/field/ground/stream/open sewer (includes dumping anywhere in environment outside compound)",
			"Do nothing",
			"Other",
			"Scatter in yard",
			"Toilet, latrine",
			"Unknown",
			"Ventilated improved pit with water seal"
		],
		[
			"Bury",
			"Bush/field/ground/stream/open sewer (includes dumping anywhere in environment outside compound)",
			"Do nothing",
			"Other",
			"Scatter in yard",
			"Toilet, latrine",
			"Unknown",
			"Ventilated improved pit with water seal"
		],
		[
			"Bury",
			"Bush/field/ground/stream/open sewer (includes dumping anywhere in environment outside compound)",
			"Do nothing",
			"Other",
			"Scatter in yard",
			"Toilet, latrine",
			"Unknown",
			"Ventilated improved pit with water seal"
		]
	]
}

so i think i did this intentionally, in case one of the x values doesnt have all y values. i dont think im returning explicit 0 values anywhere else, so i didnt want to assume it here.

In other words, an element of value may have fewer entries than the others, and we should use the yLabel field to determine which counts go with which values?
This doesn't seem great to me. I'd like to propose an alternative data structure:
type MosaicData = {
  [Xlabel: string]: {
    [Ylabel: string]: number;
  }
}

If we used that, then this:
{
  "data": [
    {
      "xLabel": [
        "No",
        "Yes"
      ],
      "yLabel": [
        [
          "No",
          "Yes"
        ],
        [
          "No",
          "Yes"
        ]
      ],
      "value": [
        [
          18405,
          1092
        ],
        [
          3070,
          0
        ]
      ]
    }
  ]
}

would become this:
{
  "data": {
    "No": {
      "No": 18405,
      "Yes": 1092
    },
    "Yes": {
      "No": 3070,
      "Yes": 0
    }
  }
}

This latter data structure is more concise, but also a little more clear (to me, at least) in how the labels and values are associated. Thoughts?

Note, we can use the variable metadata's vocabulary property to iterate on the complete set of values.

I agree its not ideal.. it was a bit of a compromise bc the way it's currently structured aligns well w the structure of the underlying R object and so can be converted to json more or less automatically. I'll think on it some, see how I can either rearrange the R object or flag this case somehow to print it in some custom way maybe.

If it's a pain to do this in R, then we can leave it as-is. It would be pretty easy to convert it to my suggested structure in Javascript. Really, I just don't want to pass around the current data structure, since I think it's a potential source of bugs. The conversion can be done in Javascript as the data is retrieved from the backend.

if its easy for you, thats probably the way to go. if it becomes a problem again, we can always revisit.

did we decide what were doing? should this be closed?

let's just close this and keep things as they are. it's working now.


impl plugin for bevy_replicon

i think bevy_replicon_renet_dtls is too long. any other names ??

i found this is not needed.
just refactoring our plugins will work.


Bug: React Drawer input checkbox  throws warning

When using the (react) Drawer the following warning is thrown.
You provided a checked prop to a form field without an onChange handler. This will render a read-only field. If the field should be mutable use defaultChecked. Otherwise, set either onChange or readOnly.

Hi @awolkers,
Thanks for finding this! The warning message should go away after #169 is merged. Until then, feel free to ignore it!


Capability StorageImageWriteWithoutFormat is required to write to storage image

I have got an error when try to write to storage image:
I/VALIDATION: UNASSIGNED-CoreValidation-Shader-InconsistentSpirv(ERROR / SPEC): msgNum: 0 - SPIR-V module not valid: Capability StorageImageWriteWithoutFormat is required to write to storage image
      OpImageWrite %263 %93 %220

        Objects: 1
            [0] 0, type: 0, name: NULL

#[spirv(block)]
pub struct PointsBuffer {
    points: [UVec2; 100],
}

#[spirv(compute(threads(1)))]
pub fn main_cs(
    #[spirv(global_invocation_id)]
    id: Input<UVec3>,
    #[spirv(descriptor_set = 0, binding = 0)]
    points_buffer: StorageBuffer<PointsBuffer>,
    #[spirv(descriptor_set = 0, binding = 1)]
    image: UniformConstant<StorageImage2d>,
) {
    let position = id.xy();
    for i in 0..100usize {
        let p0 = &points_buffer.points[i];
        let p1 = &points_buffer.points[i + 1];
        if p0.x == position.x && p1.y == position.y {
            unsafe {
                image.write(position, vec2(1.0,  0.0));
            };
        }
    }
}

System Info

Rust: 1.52.0-nightly
OS: Android
Spirv: 1.0
Vulkan: 1.0,

Thank you for your issue! The minimum version SPIR-V and Vulkan that we currently support is 1.3+ and 1.1+ respectively. You could try adding unsafe { asm!("OpCapability StorageImageWriteWithoutFormat"); } to the top of your function body and see if that works.

Question for us: We have a test for this that doesn't include StorageImageWriteWithoutFormat. However, the latest spec clearly states

The Image Format must not be Unknown, unless the StorageImageWriteWithoutFormat Capability was declared.

Why does that test pass? It shouldn't.

Thank you for your issue! The minimum version of SPIR-V and Vulkan that we currently support is 1.3+ and 1.1+ respectively. You could try adding unsafe { asm!("OpCapability StorageImageWriteWithoutFormat"); } to the top of your function body and see if that works.

It works, Thanks.

I can reproduce this locally. Validation fails for Vulkan environments not SPIR-V environments. I'll get to adding environments since that will let add a test to make sure this behaves correctly.

I believe this test has been implemented, and so this has been fixed.


Very very Web weird issue

Steps to reproduce
When connected via Websocket, break the connection (in my case restarting the WS server), minimize the web browser with the "connecting" loading spinner showing and wait for WS reconnection.
Expected results
To have the same outcome as if the web browser was in front (not minimized).
Actual results
A custom error due to some of the code seems to run normally (the WS callbacks) but other code seems to be postponed until the web browser came to the front of the screen.
Code sample
See here
Screenshots or Video
https://streamable.com/ftmvtw (forward to 2:25 to see the error)
Logs
Error: asManagedData() -- Online web devices count (Versioned) -- Error: Trying to set new value while app state is not
websocket connected. Is Websocket connecting
dart-sdk/lib/internal/js_dev_runtime/private/ddc_runtime/errors.dart 297:3                    throw
packages/busslina_flutter_web_lib/default/extensions/side_effects.rearch.extension.dart 33:13  checkScope
packages/busslina_flutter_web_lib/default/extensions/side_effects.rearch.extension.dart 53:7   setNewValue
packages/busslina_flutter_web_lib/default/app/app.default.dart 71:44
packages/busslina_flutter_web_lib/default/websocket/websocket.dart 274:9                       processMessage
dart-sdk/lib/_internal/js_dev_runtime/patch/async_patch.dart 84:54                             runBody
dart-sdk/lib/_internal/js_dev_runtime/patch/async_patch.dart 127:5                             _async
packages/busslina_flutter_web_lib/default/websocket/websocket.dart 231:22                      processMessage
packages/busslina_flutter_web_lib/default/websocket/websocket.dart 295:7                       processRawMessage
dart-sdk/lib/async/zone.dart 1594:9                                                            runUnaryGuarded
dart-sdk/lib/async/stream_impl.dart 339:5                                                      [_sendData]
dart-sdk/lib/async/stream_impl.dart 271:7                                                      [_add]
dart-sdk/lib/async/stream_controller.dart 784:5                                                [_sendData]
dart-sdk/lib/async/stream_controller.dart 658:7                                                [_add]
dart-sdk/lib/async/stream_controller.dart 606:5                                                add
dart-sdk/lib/async/zone.dart 1594:9                                                            runUnaryGuarded
dart-sdk/lib/async/stream_impl.dart 339:5                                                      [_sendData]
dart-sdk/lib/async/stream_impl.dart 271:7                                                      [_add]
dart-sdk/lib/async/stream_controller.dart 784:5                                                [_sendData]
dart-sdk/lib/async/stream_controller.dart 658:7                                                [_add]
dart-sdk/lib/async/stream_controller.dart 606:5                                                add
dart-sdk/lib/async/stream_controller.dart 876:5                                                add
packages/stream_channel/src/guarantee_channel.dart 125:5                                       add
packages/web_socket_channel/html.dart 145:23                                                   [_innerListen]
dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/operations.dart 550:37               _checkAndCall
dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/operations.dart 555:39               dcall
packages/web/src/helpers/events/streams.dart 165:63
dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/operations.dart 550:37               _checkAndCall
dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/operations.dart 555:39               dcall
Error: asManagedData() -- Visits count (Versioned) -- Error: Trying to set new value while app state is not websocket
connected. Is Websocket connecting
dart-sdk/lib/internal/js_dev_runtime/private/ddc_runtime/errors.dart 297:3                    throw
packages/busslina_flutter_web_lib/default/extensions/side_effects.rearch.extension.dart 33:13  checkScope
packages/busslina_flutter_web_lib/default/extensions/side_effects.rearch.extension.dart 53:7   setNewValue
packages/busslina_flutter_web_lib/default/app/app.default.dart 83:34
packages/busslina_flutter_web_lib/default/websocket/websocket.dart 274:9                       processMessage
dart-sdk/lib/_internal/js_dev_runtime/patch/async_patch.dart 84:54                             runBody
dart-sdk/lib/_internal/js_dev_runtime/patch/async_patch.dart 127:5                             _async
packages/busslina_flutter_web_lib/default/websocket/websocket.dart 231:22                      processMessage
packages/busslina_flutter_web_lib/default/websocket/websocket.dart 295:7                       processRawMessage
dart-sdk/lib/async/zone.dart 1594:9                                                            runUnaryGuarded
dart-sdk/lib/async/stream_impl.dart 339:5                                                      [_sendData]
dart-sdk/lib/async/stream_impl.dart 271:7                                                      [_add]
dart-sdk/lib/async/stream_controller.dart 784:5                                                [_sendData]
dart-sdk/lib/async/stream_controller.dart 658:7                                                [_add]
dart-sdk/lib/async/stream_controller.dart 606:5                                                add
dart-sdk/lib/async/zone.dart 1594:9                                                            runUnaryGuarded
dart-sdk/lib/async/stream_impl.dart 339:5                                                      [_sendData]
dart-sdk/lib/async/stream_impl.dart 271:7                                                      [_add]
dart-sdk/lib/async/stream_controller.dart 784:5                                                [_sendData]
dart-sdk/lib/async/stream_controller.dart 658:7                                                [_add]
dart-sdk/lib/async/stream_controller.dart 606:5                                                add
dart-sdk/lib/async/stream_controller.dart 876:5                                                add
packages/stream_channel/src/guarantee_channel.dart 125:5                                       add
packages/web_socket_channel/html.dart 145:23                                                   [_innerListen]
dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/operations.dart 550:37               _checkAndCall
dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/operations.dart 555:39               dcall
packages/web/src/helpers/events/streams.dart 165:63
dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/operations.dart 550:37               _checkAndCall
dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/operations.dart 555:39               dcall
Error: asManagedData() -- Unique visits count (Versioned) -- Error: Trying to set new value while app state is not
websocket connected. Is Websocket connecting
dart-sdk/lib/internal/js_dev_runtime/private/ddc_runtime/errors.dart 297:3                    throw
packages/busslina_flutter_web_lib/default/extensions/side_effects.rearch.extension.dart 33:13  checkScope
packages/busslina_flutter_web_lib/default/extensions/side_effects.rearch.extension.dart 53:7   setNewValue
packages/busslina_flutter_web_lib/default/app/app.default.dart 95:40
packages/busslina_flutter_web_lib/default/websocket/websocket.dart 274:9                       processMessage
dart-sdk/lib/_internal/js_dev_runtime/patch/async_patch.dart 84:54                             runBody
dart-sdk/lib/_internal/js_dev_runtime/patch/async_patch.dart 127:5                             _async
packages/busslina_flutter_web_lib/default/websocket/websocket.dart 231:22                      processMessage
packages/busslina_flutter_web_lib/default/websocket/websocket.dart 295:7                       processRawMessage
dart-sdk/lib/async/zone.dart 1594:9                                                            runUnaryGuarded
dart-sdk/lib/async/stream_impl.dart 339:5                                                      [_sendData]
dart-sdk/lib/async/stream_impl.dart 271:7                                                      [_add]
dart-sdk/lib/async/stream_controller.dart 784:5                                                [_sendData]
dart-sdk/lib/async/stream_controller.dart 658:7                                                [_add]
dart-sdk/lib/async/stream_controller.dart 606:5                                                add
dart-sdk/lib/async/zone.dart 1594:9                                                            runUnaryGuarded
dart-sdk/lib/async/stream_impl.dart 339:5                                                      [_sendData]
dart-sdk/lib/async/stream_impl.dart 271:7                                                      [_add]
dart-sdk/lib/async/stream_controller.dart 784:5                                                [_sendData]
dart-sdk/lib/async/stream_controller.dart 658:7                                                [_add]
dart-sdk/lib/async/stream_controller.dart 606:5                                                add
dart-sdk/lib/async/stream_controller.dart 876:5                                                add
packages/stream_channel/src/guarantee_channel.dart 125:5                                       add
packages/web_socket_channel/html.dart 145:23                                                   [_innerListen]
dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/operations.dart 550:37               _checkAndCall
dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/operations.dart 555:39               dcall
packages/web/src/helpers/events/streams.dart 165:63
dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/operations.dart 550:37               _checkAndCall
dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/operations.dart 555:39               dcall
Application finished.
Flutter Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.19.6, on Microsoft Windows [Versi¢n 10.0.19045.4291], locale es-ES)
[✓] Windows Version (Installed version of Windows is version 10 or higher)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
[✓] Chrome - develop for the web
[✓] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.4.4)
[✓] Android Studio (version 2022.1)
[✓] IntelliJ IDEA Community Edition (version 2021.1)
[✓] VS Code (version 1.89.1)
[✓] Connected device (3 available)
[✓] Network resources
• No issues found!

For now, a workaround that is "working" is just to log an error instead of throwing it. But I don't like it because as the app grows it could have bad consequences.

Also it's weird that only happens this issue if minimezing the browser with the loading screen on. If I minimize it before breaking the WS connection it doesn't happen this issue.

My bad 97%


Setup-cfg-fmt doesn't respect newline convention of source file nor read/write it with the correct encoding

Setup-cfg-fmt does not respect the EOL of the setup.cfg, and instead converts it to a platform-dependent default. I'm not aware of a case where that would be the desirable behavior, whereas many/most projects standardize on one newline convention (typically \n) for all source files, which this breaks. Furthermore, if using an EoL fixer with pre-commit, it will catch this but aside from extra noise in the output, if it isn't sequenced after this hook in the config file, it will cause the next run to fail too.
Fixing this is very simple—just check the current EoL character when reading in the file, and pass that explicitly on writing it. So at the top of format_file(), on the line after read()ing the file, you'd just add newline = f.newlines, and then pass newline=newline in the call to open() when writing it,
In addition, neither open() call specifies the correct encoding to use, which means the platform and environment-dependent locale encoding will be assumed. As setup.cfg is required to be UTF-8, this means loading will fail on Windows if any non-ASCII characters are present (common in e.g. authors' names), or on any platform that doesn't have UTF-8 mode enabled, and even if loading were to succeed, the file would be written in the incorrect encoding. Furthermore, not specifying an encoding when opening in text mode will be an optional warning in Python 3.10, with a DeprecationWarning planned to follow in later versions. To fix this, simply explicitly pass encoding="utf-8" to both calls.
Happy to submit a PR to fix both of these if you'd like. Thanks!

adding encoding='utf-8' seems fine, as for picking a line ending I'd lean towards always using \n -- I don't think your suggestion of newline = f.newlines works, that value is always None and determining mixed newlines or the proper ending seems out of scope and/or difficult (and/or better handled by a tool that actually is intended to solve that problem)

I don't think your suggestion of newline = f.newlines works, that value is always None

That attribute is lazy and you have to do a .read() first, or it will be None as you observed, which is why my suggestion added it it after the existing read().

determining mixed newlines or the proper ending seems out of scope and/or difficult (and/or better handled by a tool that actually is intended to solve that problem)

Good point on the mixed newlines; since that's a relatively uncommon edge case I hadn't considered it. However, in that case (where per the docs, f.newlines will return a tuple of line endings). if newlines is not a str it can simply fall back to \n as you suggest.
Other than cases of mixed EOLs where the file must be conformed to something, I'd think arbitrarily changing line ending, generally a per-project default handled by other hooks and orthogonal to the standards and conventions used in setup.cfg, would be out of scope here.

as for picking a line ending I'd lean towards always using \n

While hardcoding it would work for my particular application, it won't solve this problem for any projects that use another line ending, and potentially create a new one for those that use the platform default. So I'd suggest respecting the existing line ending for the common/simple case, and only conforming if its non-trivial.


Definition of the Weaver configuration at both a local and a global level

See discussion below for an updated alternative.
See https://github.com/open-telemetry/weaver/pull/231 for details on the implementation.
Proposal
Add support for a hierarchical definition of the Weaver configuration. The proposal consists of loading the configuration files in the following order:

the defaults/weaver_config directory embedded in the Weaver binary.
the $HOME/.weaver/weaver.yaml directory,
the parent directory (i.e., the directory containing all targets),
and finally the target directory,

A local definition will override any corresponding definition in the parent directory (shared across all targets), which itself can override the corresponding entry in the home directory (defined per user), and finally, the Weaver application binary (defined by Weaver authors). To summarize, this hierarchical structure will allow for configuration at the target level, across all targets, at the user level, and even reuse of configurations defined within the Weaver application binary.

This proposal is introduced, among other things, to support #211.

Why
There has been a past need to define configurations or definitions common to multiple languages, multiple teams, etc. This is the main reason for implementing a mechanism that allows for a hierarchical and override mechanism within Weaver’s configuration system.
The current organization (i.e., templates//) is more of a default organization that seemed logical at the time. The initial idea was to easily list the available targets for users and offer a hierarchical organization to share certain configuration elements (e.g., acronyms or whitespace control shared between multiple targets).
Alternative
If the need to list all the available targets is not important for the community, we could use a different approach based on two parameters:

--templates to point to the directory containing the templates to apply.
--config to list one or several weaver.yaml files that will be applied in the specified order. By default, this parameter could retrieve the config files based on the list of paths mentioned in the proposal above (minus the parent directory, which will no longer be valid in this alternative).

Discussion
After some reflection, I propose the following approach, which I believe will be more aligned with how Weaver is currently used in the context of code generation. Typically, Weaver is used on a GitHub repository dedicated to a specific language. In this context, the concept of target is no longer relevant since there will generally be only one target in this repository (this might not be true for entreprises using Weaver).
To recap, the current approach relies on two parameters: --templates and --target. --templates points to the root directory containing the targets for the semconv registries, or in the future, the targets for schemas (app telemetry schema). When this parameter is not set, it defaults to templates/. The --target parameter is mandatory and targets the templates to be resolved with the following logic:

When the command weaver registry generate --target rust is called, the templates that will be generated are loaded from templates/registry/rust.
When the command weaver schema generate --target rust is called, the templates that will be generated are loaded from templates/schema/rust.

Indeed, this approach, when applied to a repository dedicated to creating a client SDK for a single language, is somewhat overkill. I agree it's a bad UX.
New Proposal
This new proposal can be summarized as follows:

--templates now points directly to the directory containing the templates to be generated. By default, this directory will be templates/.
--target is no longer used and becomes optional.
--config is a new optional parameter (n-ary) that allows listing the weaver.yaml files to load before applying the template generation. Multiple configuration files can be specified, and their order is important because the override mechanism will be applied in the order of the files' appearance (i.e., file n overrides file n-1, etc.). When the --config parameter is not specified, it will automatically be set as follows:

the defaults/weaver_config directory embedded in the Weaver binary.
the $HOME/.weaver/weaver.yaml directory,
all weaver.yaml files defined from the root to the effective path specified by the --templates parameter. This approach is used, for example, by the cargo command to load and resolve Cargo.toml files.

In terms of UX, the authors of a repository will generally not need to specify anything if they place their templates directly in the templates/ directory. If they wish to implement variations for document or code generation, they can still define their own template organization and use the --templates parameter. If they do not want the default Weaver configuration resolution mechanism, they can then use the --config parameter.

Note: Although there are probably very few Weaver users at the moment, I propose the following transition, which remains compatible with the existing setup for a yet-to-be-defined period. The new approach is only applied when the --target parameter is not specified. The --config parameter can be applied to both approaches. I also propose adding a "deprecated" note indicating that the target concept will be removed in the near future (exact timeline to be determined).

So I'm reading through this and haven't had time to digest, so apologies if this is half formed.
Some quick thoughts.

I think in the long run, weaver should be providing "codegen packages".  That is bundled config, for many languages, that users can use and leverage/reference.  Possibly with release cadence and such similar to a docker image or cargo package.  Effectively we need way to create codegen and bump it, and I don't see weaver adopted past otel core without having a good default for codgen/docgen.  I do not think this codegen/docgen we provide by default will be semconv library, but something more targeted at non-otel libraries.
The current proposal w/ config is confusing in one sense:  Config defines the tempalte to use in a JQ expression.   I think that config belongs near the templates.  I think merging everythign together in a unified model is ok, but think about the use case for each location of config and maybe tune/optimise these.


Transition to sox

Using sox to grab the signal allows basic tracking through the CLI of peaked audio as well as (in combination with buffering) appears to solve dropped sample problem on slower mac

Is there any reason I miss for not using $(...) instead of `...`?

That is just a legacy of using Carsten Bleum's rather old Pashua calling function- I think it can certainly be updated!

Thank you! I guess it makes the code more readable, therefore easier maintainable in the future.

LGTM


Support faulty/slow/interrupted pod networking

One class of network chaos to support is for pod networking (vs host/node networking).  Scenarios might be:

high latency networking environments
-networks that are expected to periodically disconnect (moving vehicles)
poor quality networks (packet corruption, high tx retries, etc)

One possible way for Kraken to support these scenarios is via integration of Litmus experiments:
Pod network latency
Pod network loss
Pod network corruption
This issue might need to be broken out into multiple issues.

This is now supported in Krkn thanks to @yogananth-subramanian at both pod and node level: https://github.com/redhat-chaos/krkn/blob/main/docs/pod_network_scenarios.md and https://github.com/redhat-chaos/krkn/blob/main/docs/network_chaos.md. Thanks.


Drag Event

I'm doing a project where I'm trying to integrate this into a Prism (https://compositewpf.codeplex.com/) app.  I love the drag feature on the tabs, but managing state with the Prism Region API is being problematic when a tab is dragged from one tab control into another (either new or existing).  It would be really helpful if the control would emit an event so that I could handle the state change properly.
The things I would be interested in are:

The content that is being moved
The source TabablzControl
The destination TabablzControl

Thanks

Ideally an implementation of IInterTabClient client should give you a hook.  Perhaps in your case you want to implement IManualInterTabClient.  This would allow you to manually remove from one source and add to another.  This is the preferred way to jump in and manage your source collections manually.
There are also several events on the items themselves:

DragablzItem.DragStarted
DragablzItem.PreviewDragDelta
DragablzItem.DragDelta
DragablzItem.DragCompleted

You might want to use EventManager.RegisterClassHandler to create a hook.
Also, have you seen this test project? https://github.com/tstephansen/PrismExample
See where you can get by implementing IManualInterTabClient and exposing it on the InterTabController.  Then if you are still stuck if you can post a little test project on GitHub I'll see if I can help you get it running.

Is there an example of the ManualInterTabClient?  I tried looking at the examples to see if there was a sample implementation, because just implementing IManualInterTabClient, I started getting the exception below as soon as the tab was dragged out of the original Tab Control.
System.NullReferenceException: Object reference not set to an instance of an object.
   at Dragablz.DragablzItemsControl.InstigateDrag(Object item, Action`1 continuation) in c:\projects\dragablz\Dragablz\DragablzItemsControl.cs:line 185
   at Dragablz.TabablzControl.ReceiveDrag(InterTabTransfer interTabTransfer) in c:\projects\dragablz\Dragablz\TabablzControl.cs:line 970
   at Dragablz.TabablzControl.MonitorBreach(DragablzDragDeltaEventArgs e) in c:\projects\dragablz\Dragablz\TabablzControl.cs:line 939
   at Dragablz.TabablzControl.ItemDragDelta(Object sender, DragablzDragDeltaEventArgs e) in c:\projects\dragablz\Dragablz\TabablzControl.cs:line 859
   --- End of inner exception stack trace ---
   at System.RuntimeMethodHandle.InvokeMethod(Object target, Object[] arguments, Signature sig, Boolean constructor)
   at System.Reflection.RuntimeMethodInfo.UnsafeInvokeInternal(Object obj, Object[] parameters, Object[] arguments)
   at System.Delegate.DynamicInvokeImpl(Object[] args)
   at System.Windows.RoutedEventArgs.InvokeEventHandler(Delegate genericHandler, Object genericTarget)
   at System.Windows.RoutedEventArgs.InvokeHandler(Delegate handler, Object target)
   at System.Windows.RoutedEventHandlerInfo.InvokeHandler(Object target, RoutedEventArgs routedEventArgs)
   at System.Windows.EventRoute.InvokeHandlersImpl(Object source, RoutedEventArgs args, Boolean reRaised)
   at System.Windows.UIElement.RaiseEventImpl(DependencyObject sender, RoutedEventArgs args)
   at System.Windows.UIElement.RaiseEvent(RoutedEventArgs e)
   at Dragablz.DragablzItem.ThumbOnDragDelta(Object sender, DragDeltaEventArgs dragDeltaEventArgs) in c:\projects\dragablz\Dragablz\DragablzItem.cs:line 493
   at System.Windows.Controls.Primitives.DragDeltaEventArgs.InvokeEventHandler(Delegate genericHandler, Object genericTarget)
   at System.Windows.RoutedEventArgs.InvokeHandler(Delegate handler, Object target)
   at System.Windows.RoutedEventHandlerInfo.InvokeHandler(Object target, RoutedEventArgs routedEventArgs)
   at System.Windows.EventRoute.InvokeHandlersImpl(Object source, RoutedEventArgs args, Boolean reRaised)
   at System.Windows.UIElement.RaiseEventImpl(DependencyObject sender, RoutedEventArgs args)
   at System.Windows.UIElement.RaiseEvent(RoutedEventArgs e)
   at System.Windows.Controls.Primitives.Thumb.OnMouseMove(MouseEventArgs e)
   at System.Windows.UIElement.OnMouseMoveThunk(Object sender, MouseEventArgs e)
   at System.Windows.Input.MouseEventArgs.InvokeEventHandler(Delegate genericHandler, Object genericTarget)
   at System.Windows.RoutedEventArgs.InvokeHandler(Delegate handler, Object target)
   at System.Windows.RoutedEventHandlerInfo.InvokeHandler(Object target, RoutedEventArgs routedEventArgs)
   at System.Windows.EventRoute.InvokeHandlersImpl(Object source, RoutedEventArgs args, Boolean reRaised)
   at System.Windows.UIElement.RaiseEventImpl(DependencyObject sender, RoutedEventArgs args)
   at System.Windows.UIElement.RaiseTrustedEvent(RoutedEventArgs args)
   at System.Windows.Input.InputManager.ProcessStagingArea()
   at System.Windows.Input.InputManager.ProcessInput(InputEventArgs input)
   at System.Windows.Input.InputProviderSite.ReportInput(InputReport inputReport)
   at System.Windows.Interop.HwndMouseInputProvider.ReportInput(IntPtr hwnd, InputMode mode, Int32 timestamp, RawMouseActions actions, Int32 x, Int32 y, Int32 wheel)
   at System.Windows.Interop.HwndMouseInputProvider.FilterMessage(IntPtr hwnd, WindowMessage msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
   at System.Windows.Interop.HwndSource.InputFilterMessage(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
   at MS.Win32.HwndWrapper.WndProc(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
   at MS.Win32.HwndSubclass.DispatcherCallbackOperation(Object o)
   at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)
   at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)
   --- End of inner exception stack trace ---

OK...can you send me a project?   it's the quickest way for me to fix the bug and see where you're at.

I created a sample project at https://github.com/svantreeck/DragablzPrism that I think I got working.  I was originally using a Dictionary to map the region items to their owning TabablzControl, but now I'm keeping track of a list of open tab controls and looping through them.  It's less efficient than I wanted it to be, but in reality, at most I see probably a dozen or so items open, so O(1) vs O(N) isn't a big deal for finding the proper tab control.


Postgres json fields shown as [object Object] in csv and xls exports

As per subject. The field shows correctly in the UI.

@dop251 which version of SQLPad are you running? I just merged a fix for this 3 hours ago in #976 .
The fix should be in the latest dockerhub image if pulled within the last 2 hours or so.

I pulled it today but obviously prior to the fix, can confirm it's working now. Thanks!


Update README.md

:x: Package verification failed, please review the Appveyor Logs and the provided Artifacts before requesting a human reviewer to take a look.


Update .npmignore — Fixes #24

Also ignore .babelrc. Fixes #24

Thanks for the PR!

I was wondering: Now that it's merged, will a 0.7.1 release be released on NPM? That version (with this PR included) will fix react-native compatibility :-)

I plan to push both 0.7.1 and 0.8.0-beta.2, not sure though if i'll manage to do it today.

Thanks!


Implemented AddProblem and Login Fixes

There are some problems I think we may face in the future with current code:

For Login, how would we know the user's role if we can't instantiate the User class since its abstract
I've been passing the userID intent around from login->addProblem should we make a singleton for the logged in user?

Pictures

Point 1) We should be able to get the JSON and look at the user-type there to figure out which type of object to instantiate as; maybe we'll put this into a helper class since I've done this for my RegisterUserTask as well, we'll probably use it in lots of places.
Point 2) Shouldn't be too big of a problem but we can discuss this approach


handle unexpected exception on success callback of translog upload

Description

Translog upload success callback can throw unexpected exceptions.

To be able to log the exceptions occurring in onSuccess callbacks, we wrap those exception as FileTransferException

Related Issues
https://github.com/opensearch-project/OpenSearch/issues/12544
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Failing checks are inspected and point to the corresponding known issue(s) (See: Troubleshooting Failing Builds)
[x] Commits are signed per the DCO using --signoff
[x] Commit changes are listed out in CHANGELOG.md file (See: Changelog)
[ ] Public documentation issue/PR created

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Changes LGTM. Can we please add UTs for the change?

Like @sachinpkale  mentioned, please add unit tests.


Format json should send content-type application/json

Currently we don't send any content-type.

Fixed via https://github.com/StefMa/AppVersions/commit/f2c790f6f69b4932ad172c93e7e090dacefb4e8a


layers: Validate ONE_TIME_SUBMIT for secondary cmd buffers

Previously were only validating *_ONE_TIME_SUBMIT_BIT for primary cmd buffers.
After clarification on spec, moving the check so it will also hit secondary
command buffers.
Fixes #341

+1

cherry-picked and pushed to master


Audit and commit node_modules before public release

Not yet, as it will slow down development.

Closing due to complications implementing this, it makes a complete mess of the development process.
Instead we should update dependencies rarely, check everything that's being changed in the dependency tree and audit them.


Stellargraph on time-varying graphs

I'm interested in applying this to time-varying graph data,. Is it possible? I want this model to learn time-varying graph data and challenge the node classification task.

Thanks for the comment. We are looking at potential ways to use time-varying information and hope to incorporate this functionality into the library in the future.
Do you have any algorithms or use-cases in mind?
Regards,
Andrew

StellarGraph now incorporates several algorithms for graphs that have time information:

graphs with times associated with edges: https://stellargraph.readthedocs.io/en/stable/demos/link-prediction/ctdne-link-prediction.html
graphs with time series as features in each node: https://stellargraph.readthedocs.io/en/stable/demos/time-series/index.html

It's been several months since any update on this issue, so with the above and without more specifics, I'm going to call this resolved. Please let us know if you have any issues for further thoughts. If not, we'll close this in a week or so (but we can always reopen after too).


Update README.md

Description
There was a typing mistake in the Readme file.  The full form of GSSOC was Global Summer Of Code instead of the correct full form i.e. GirlScript Summer Of Code !!
This PR fixes #
This PR fixes this issue and corrects the mistake in the Readme File !!
Notes for Reviewers
Thanks For Assigning me this task!
It was so great to contribute in this project!
Hope this pull request will be accepted !+

Thank you @jayshivare7


Money glitch

*Detailed description of the issue:

Have hunter job
kill a mob just before you get teleported by nether portal
you obtain the kill reward several times.

its hard to time but one player did it twice and got 80k
you gotta really time it, when you teleport, its trial and error but its big money exploit once done
Jobs version:

Server Type (Spigot/Paper/etc):
paper
Server Version (using /ver):
this server is running Paper version git-Paper-134 (MC: 1.18.1) (Implementing API version 1.18.1-R0.1-SNAPSHOT) (Git: 55a1756)
Relevant plugins (Delete if this isn't needed):
multiverse netherportals, essentials, vault

No response? :(

Jobs issues payment on mobs death event, if there are multiple payments issued when you teleport then it might be minecraft bug and not Jobs in general. But at the end of day we need consistent way to reproduce this to determine what causes this and why. Would start by having only Jobs plugin and check if issue persists or not.

Is it fixed or?


deps: Drop expo-cli

I believe we don't ever use the expo CLI.
The expo-cli package is apparently in maintenance mode. It doesn't and won't support Node 18, the current LTS:
https://github.com/expo/expo-cli/issues/4583
https://github.com/expo/expo-cli/pull/4281
That effectively gives it an EOL of 2023-09, the EOL date of Node 16, which is less than a year away.
There's a successor CLI in the works:
https://blog.expo.dev/the-new-expo-cli-f4250d8e3421
But since we don't actually use this CLI in the first place, we can wait to pick that up until we find a need for it.
As a bonus, dropping this sheds about 90MB from our node_modules tree.

Cool, thanks for the correction.

I think it's probably fine to remove it, and if Expo's upgrade guides continue to ask us to use it, we can install it globally instead of in our project's devDependencies.

Not quite, I think: the new CLI specifically wants us to use the CLI from project dependencies, not a user-global install. Which seems for the best.
Rereading that blog post, though, I realize that it's no longer a separate package -- it's shipped via the expo package. So there's nothing to add; it'll already be there.
(That's the case starting with Expo SDK 46, anyway. So for the remaining upgrade to there from 45, I guess we might need to install the old CLI either globally or locally, and then uninstall it for good.)
I'll revise the commit message and merge.


[windows][cws]wkint-491] Add helper functions for normalizing drive p…

…aths.
Creates a map of device names to drive paths; is intended for consumption by future pr.
This will supercede #25298
Describe how to test/QA your changes
Testing will be covered in superceding PR.

/merge


⚠ Economy API Endpoint has degraded performance

In d6899c9, Economy API Endpoint (https://economy.roblox.com) experienced degraded performance:

HTTP code: 503
Response time: 9383 ms

Resolved: Economy API Endpoint performance has improved in 9e63cec.


feat(worker): allow passing options to worker constructors

Description
This PR adds support for the options parameter of worker constructors, notably allowing { type: "classic" } (fixes https://github.com/vitejs/vite/issues/2550).
Additional context
There is https://github.com/vitejs/vite/pull/2578, but it's closed at https://github.com/vitejs/vite/pull/2578#issuecomment-809109102. This PR workarounds that problem by prepending importScript(...) instead of import ... to classic worker scripts.
What is the purpose of this pull request?

[ ] Bug fix
[x] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the Commit Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[x] Ideally, include relevant tests that fail without this PR but pass with it.

I am using monaco editor via vitejs. The ts_worker is loaded as module worker, importScript fails when i use customWorkerPath.

This PR helps. Any chance to merge this?

Dingtalk's chromium is 72, Worker module type surport 80,  It Is Necessary!

Closing as it is supported by different syntax. https://github.com/vitejs/vite/issues/2550#issuecomment-1079618078
https://vitejs.dev/guide/features.html#import-with-constructors

#2550(comment)
It can be used in the development environment, but when 'NPM run build', my new URL is a typescript file?
new URL("./worker.ts", import.meta.url)
// after npm run build
const ng=new URL("./worker.ts",self.location);Yr=new Worker(ng,{type:"classic"});

Does this mean that I have to compile my web worker file in advance in this way? Or is there something wrong with the way I use it?
I still think this method can better solve the problem

import MyClassicWorker from './ classic-worker? worker'

const worker = new MyClassicWorker({ type: 'classic' })

@0x30 I feel your usage is not correct. Please create an Q&A discussion with reproduction.

Yes, you are right. You can see discussions#8525.

But I don't think the document tells us that new workers (New URLs) are not allowed to be separated.Based on previous experience, I think it is OK to transfer URL as a variable to the worker. Obviously, there may be some problems. Maybe the document can focus on this.
Yes, you can also see from the discussion that I did these operations when solving the problem of [dev mode classic worker],Although the problem of worker classic under dev has been modified pull/7099, you can see myexample. This problem still exists in vite 2.9.11. Or what? Did I use the wrong way again?

@0x30

Please create an issue or a PR about improving document.
I think it is #8470. You cannot use import in classic worker.


Adding a guidance questionnaire for visitors

As a GSSOC' 23 contributor, I would like to add a questionnaire that will ask new visitors about their interests, strengths and weaknesses anonymously. In the end it will display available content that can help the visitor.  The visitor can then sign-up and access the content as a user.

I want to work on this issue as a contributor.
Pleasure Assign so that I can show my skills under GSSoC'23.

@AbhaBarge are you working on it ?? What are the updates ?


Slow Detecting of QR Codes

Scanner is pretty slow in detecting QR codes. It needs the best environment like the best lighting conditions etc. Same time package which hasn't been updated since 3 years ago(flutter_barcode_scanner) is detecting rapidly. Please fix this issue.

The flutter_barcode_scanner seems to be using the old Vision library, see https://github.com/AmolGangadhare/flutter_barcode_scanner/blob/master/android/build.gradle#L41
However, the note at the top of the docs for that library states:
https://developers.google.com/android/reference/com/google/android/gms/vision/barcode/package-summary
"The Mobile Vision API is deprecated and no longer maintained. It is now a part of ML Kit which includes all new on-device ML capabilities."
Since mobile_scanner already uses MLKit, any performance issues relating to the detection of barcodes can only be fixed by upgrading MLKit (if MLKit has fixes for such issues)


BTC-PERT missing from devnet

Hi, I was going though the marketing making tutorial, and encountered an error at the last step (step 11):
$ mango-explorer marketmaker --name "BTC-PERP Marketmaker" --market BTC-PERP --oracle-provider pyth --confidenceinterval-position-size-ratio 0.1 --minimumcharge-ratio 0 --confidenceinterval-level 2 --confidenceinterval-level 4 --existing-order-tolerance 0.0001 --pulse-interval 30 --order-type POST_ONLY --biasquoteonposition-bias 0.00003 --log-level DEBUG --cluster-name devnet --dry-run

... ...

2022-08-29 23:21:44 🐛 Reconnecting Closing WebSocket for wss://mango.devnet.rpcpool.com
Traceback (most recent call last):
  File "/app/bin/marketmaker", line 221, in <module>
    market = mango.market(context, args.market)
  File "/app/mango/porcelain.py", line 93, in market
    raise Exception(f"Could not find market {symbol}")
Exception: Could not find market BTC-PERP

Upon further examination, I realized that the reason is that PERP-BTC doesn't existing in devnet, but exists in mainnet and testnet, see below:
$ poetry run bin/show-market --name mm --cluster-name testnet --market BTC-PERP
... ...

« PerpMarket BTC-PERP 6GTMXmpBdUDAojfjocxxn2nD7C5CiN9i4RLqcC3BWEG7 [BXhdkETgbHrr5QmVBT1xbz3JrMM28u5djbVtmTUfmFTH]
    « PerpMarketDetails Version.V1 [6GTMXmpBdUDAojfjocxxn2nD7C5CiN9i4RLqcC3BWEG7]

... ...

$ poetry run bin/show-market --name mm --cluster-name devnet --market BTC-PERP
... ...
Traceback (most recent call last):
  File "/home/stevel/workspace/crypto/solana/mango-explorer/bin/show-market", line 19, in <module>
    market = mango.market(context, args.market)
  File "/home/stevel/workspace/crypto/solana/mango-explorer/mango/porcelain.py", line 93, in market
    raise Exception(f"Could not find market {symbol}")
Exception: Could not find market BTC-PERP

If PERP-BTC is indeed missing in devnet, perhaps one of the following two changes should be made?

Replace devnet with testnet in the tutorial, or
Replace PERP-BTC with something else in the tutorial.

If you guys decide to go one way or another, am happy to submit a PR to save you guys some work...

PR submitted: https://github.com/blockworks-foundation/mango-explorer/pull/69

Fixed in #69


Add helpbox

Summary
Add a help box directing new students to the UCI Catalogue for help with class picking.

Test Plan

Verify that HelpBox looks good on desktop and mobile
No typos
Link to UCI Catalogue

Issues
Closes #352
Future Followup
After enrollment, we could use the helpbox for other tips like highlighting AntAlmanac features (#303)

Maybe include a close button that remembers it's closed with localStorage, duplicate the info in the about page, and display a snackbar telling the user to look in the about page if they need to see it again.

Me looking at the light mode screenshot

Maybe include a close button that remembers it's closed with localStorage, duplicate the info in the about page, and display a snackbar telling the user to look in the about page if they need to see it again.

I thought about this but I wasn't sure if it was worth the effort, since we'll probably take it down once SPOP ends. Would you like me to do it?

Me looking at the light mode screenshot

updated screenshot to darkmode 😌

Maybe include a close button that remembers it's closed with localStorage, duplicate the info in the about page, and display a snackbar telling the user to look in the about page if they need to see it again.

I thought about this but I wasn't sure if it was worth the effort, since we'll probably take it down once SPOP ends. Would you like me to do it though?

In that case, let's not add that functionality. Once we close this let's create an issue to comment it out once SPOP is done.


Gallery: Unhandled message type results in crash

Attempting to access /apps/groups/groups/~fabled-faster/storage/channels/heap/~fabled-faster/texts-9123 results in the following:
Error: Unhandled message type: {"code":"Intro:\n \nAm, F, C...\nF, Em7, C, Em7, F\nEm7, C, Dm, F, G\n \nVerse:\n \nAm\nSomeday\n   C          Em7   F\nI know we'll meet again\n    Em7  C       Em7     F\nWithout knowing exactly when\n    G      C\nAnd until then\n     Dm           F       G\nTake care of yourself my friend\n \n                      Am\nIf you tell it to my heart\n                  F\nwhere would you start?\n           G\nWith that truth we shared\n        Am         F        G\nor the lies that drove us apart?\n \n                      Am\nIf you tell it to my heart\n                  F\nwhere would you start?\n           G\nWith that truth we shared\n        Am         F        C\nor the lies that drove us apart?\n \n                      Am\nIf you tell it to my heart\n                  F\nwhere would you start?\n           G\nWith that truth we shared\n        Am         F        G\nor the lies that drove us apart?\n \nOutro:\n \nC\nHea-a-a-art\n              G\nTell it to my heart\n \nAm, F, G, C (ad infinitum)"}
    at kn (http://localhost:8080/apps/groups/assets/index.8af1b904.js:45:71331)
    at Kl (http://localhost:8080/apps/groups/assets/react-beautiful-dnd.3bfe057f.js:19:52884)
    at mf (http://localhost:8080/apps/groups/assets/react-beautiful-dnd.3bfe057f.js:23:8723)
    at pf (http://localhost:8080/apps/groups/assets/react-beautiful-dnd.3bfe057f.js:23:947)
    at yg (http://localhost:8080/apps/groups/assets/react-beautiful-dnd.3bfe057f.js:23:875)
    at rn (http://localhost:8080/apps/groups/assets/react-beautiful-dnd.3bfe057f.js:23:730)
    at sl (http://localhost:8080/apps/groups/assets/react-beautiful-dnd.3bfe057f.js:21:10884)
    at http://localhost:8080/apps/groups/assets/react-beautiful-dnd.3bfe057f.js:19:38716
    at e.unstable_runWithPriority (http://localhost:8080/apps/groups/assets/react-beautiful-dnd.3bfe057f.js:8:3811)
    at _r (http://localhost:8080/apps/groups/assets/react-beautiful-dnd.3bfe057f.js:19:38494)

@urcades Relevant to your interests

Fixed by #2318, should be out Monday morning.


[OSPK8-422] generate ctlplane Ansible inventory

This patch creates an Ansible inventory of the cltplane
IPs within the openstackclient pod. The intent is for this
to be used for RH node registration instead of relying
on the ConfigGenerator/TripleO inventory to be generated
which doesn't actually exist at the time we need it with
the new workflow.

@stuggi I pondered this approach a bit more yesterday and I now believe it is the wrong way forward. Instead of "manually" executing RHSM installation it would work much more nicely to just use the automated t-h-t templates for RHSM registration: https://access.redhat.com/documentation/en-us/red_hat_openstack_platform/16.2/html-single/advanced_overcloud_customization/index#deploying-the-overcloud-with-the-rhsm-composable-service-portal Thoughs? I will rework a new patch for dev-tools which can leverage Ollie's latest heat environment patch to implement this approach.

@stuggi I pondered this approach a bit more yesterday and I now believe it is the wrong way forward. Instead of "manually" executing RHSM installation it would work much more nicely to just use the automated t-h-t templates for RHSM registration: https://access.redhat.com/documentation/en-us/red_hat_openstack_platform/16.2/html-single/advanced_overcloud_customization/index#deploying-the-overcloud-with-the-rhsm-composable-service-portal Thoughs? I will rework a new patch for dev-tools which can leverage Ollie's latest heat environment patch to implement this approach.

@dprince this won't work without a change to tht, or we switch to the overcloud full image for the vmset/bmset nodes. Before the rhsm service runs, the common roles get triggered, which runs the tripleo-bootstrap role [1]. This includes installing the required ooo packages, which then won't install the required packages and later steps fail, e.g. missing network-scripts package. This will work if we'd reuse the overcloud full image which already has all the packages available.
[1] https://github.com/openstack/tripleo-heat-templates/blob/stable/train/common/deploy-steps.j2#L477


[Feature Request] Add day of week picker for recurring posts.

The current workflow allows recurring posts every X days from the current day. If creating the schedule on a Friday, all other posts will be in reference to Friday, for example.
A better workflow would be to have a Day picker along with the "every X days", or even easier would be an option "Every Tuesday and Thursday", for example. The community I run has a few daily posts that only get posted 2 days a week, so this would be helpful.

This use case is already supported:

Ha!
The greyed out box made me think I couldn't even click it... Thanks!


python311Packages.deezer-python: 5.9.0 -> 5.12.0

Diff: https://github.com/browniebroke/deezer-python/compare/refs/tags/v5.9.0...v5.12.0
Changelog: https://github.com/browniebroke/deezer-python/releases/tag/v5.12.0
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

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

4 packages built:

python310Packages.deezer-python
python310Packages.deezer-python.dist
python311Packages.deezer-python
python311Packages.deezer-python.dist


Add assertions

Added assertions to the following classes:

GuiSettings
LogsCenter
LogicManager
FindCommand
ModuleCodeContainsKeywordsPredicate

Resolves #91

Codecov Report

Merging #92 into master will decrease coverage by 0.59%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master      #92      +/-   ##
============================================
- Coverage     74.42%   73.83%   -0.60%
- Complexity      424      428       +4
============================================
  Files            75       75
  Lines          1251     1261      +10
  Branches        125      130       +5
============================================
  Hits            931      931
  Misses          273      273
- Partials         47       57      +10

Impacted Files
Coverage Δ
Complexity Δ

...n/java/seedu/address/commons/core/GuiSettings.java
62.06% <0.00%> (-7.17%)
6.00 <0.00> (+1.00)
:arrow_down:

...in/java/seedu/address/commons/core/LogsCenter.java
76.31% <0.00%> (-2.07%)
10.00 <0.00> (ø)

...rc/main/java/seedu/address/logic/LogicManager.java
68.18% <0.00%> (-6.82%)
4.00 <0.00> (+1.00)
:arrow_down:

...java/seedu/address/logic/commands/FindCommand.java
81.81% <0.00%> (-18.19%)
7.00 <0.00> (+1.00)
:arrow_down:

...el/module/ModuleCodeContainsKeywordsPredicate.java
77.77% <0.00%> (-22.23%)
8.00 <0.00> (+1.00)
:arrow_down:

Continue to review full report at Codecov.

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


Can the reading of the resuts be speed ut with burst reading?

I am making a power logger to log the power consumption for a esp32 BME280 sensor that's is active less than 350ms thanks to ESPNOW to a SD card and are with your great library getting really good measurement speeds since it can set the ADC to 9-bit unlike other libraries. I get a full sampling every 1700us (shunt, busvoltage, current and voltage) and have seen that I get varied values between the measurements so its not just the same INA219 reading read twice.
I noticed that each "get_value" request is taking way more (200-220us each, 620 for three) than the 84us the 9-bit conversion in the datasheet states and started to think it about and remembered that I read that Sparkfun BME280 library (in pull request 50) is using burst reading according to bme280 datasheet section 4 and reads all registers at one time and then divides them up. Would this be possible to do here as well and will it speed up the reading and minimise the risk that two "get" is from different samplings? I noticed that this is way over my head and I do not have the knowledge myself to do it but thought it might be something you know how to do and was interested in?
Just an idea... that might be totally stupid but thought it was worth asking you.
Best regards, Albin

Hi @Albin76 , sorry for the late response. A burst read should be possible since the registers are in sequential order. I will try that but it will still need some time.

I measured now (today) Attiny85, getCurrent_mA() only. 6 measure (and 6 serial.print) is 280ms, SAMPLE_MODE_16 equal with BIT_MODE_12, so the TinyWire is slow, maybe.

Hi @BCsabaEngine , can you share your complete code? Then I could see more details. e.g. if you are you using the continuous or triggered mode, which baud rate you use for Serial.print and so on.
And what is the clock speed of your ATtiny85? And which board package do you use? E.g. ATtinyCore from spence Konde or attiny from David A. Mellis?

Hi @BCsabaEngine ,
it is the Serial.print() which is slow. I have tested the following small sketch:
void setup() {
  Serial.begin(9600);
  delay(1000);
  unsigned long startTime = millis();
  unsigned long durationTime = 0;
  for(int i=0; i<100; i++){
    Serial.println("Hello World, here I am!");
  }
  durationTime = millis()-startTime;
  Serial.println(durationTime);
}

void loop() {   }

The result is 2646 milliseconds which means 26.46 milliseconds per Serial.println(). 6 Serial.println() of this length plus the expected 51 ms for 6 measurements at Sample_Mode_16 would take ~210 milliseconds.
If I reduce the string to be printed to "Hello Word" it's still 1483 milliseconds for 100 Serial.println().
With a baud rate of 115200 and "Hello World" 100 Serial.println() take only 129 milliseconds, which is 1.29 milliseconds per println(),
So, if you need to print all results then I would recommend to keep the output as short as possible and the baud rate as high as possible.
My ATttiny85 was set to 8 MHz clock rate and I have applied the package from Spence Konde.

Hi @wollewald !
Thank you for fast response. I use this code to measure 6 INA219 (at final with different I2C address) current, now emulate with same I2C address.
#include <SoftwareSerial.h>
#include <TinyWireM.h>

#include "INA219_WE.h"

#define TX    3
#define RX    2
#define LED    1
#define I2C_ADDRESS 0x40

SoftwareSerial Serial(RX, TX);
INA219_WE ina219 = INA219_WE(I2C_ADDRESS);

void setup() {
  pinMode(LED, OUTPUT);

  Serial.begin(9600);

  TinyWireM.begin();

  if (!ina219.init()) {
    Serial.println(F("INA219 NOT connected"));
    while (1) {}
  }
  else {
    Serial.println(F("INA219 connected"));
  }

  ina219.setMeasureMode(CONTINUOUS);
  ina219.setADCMode(SAMPLE_MODE_16);
  ina219.setBusRange(BRNG_16);
  ina219.setPGain(PG_40);
}

uint32_t lastOutput = 0;
void loop() {
  uint8_t isany = false;
  for (int i = 0; i < 6; i++) {

    int16_t mA = (int16_t)ina219.getCurrent_mA();

    if (mA < 0)
      mA = mA * -1;

    if (mA != 0) {
      isany = true;
      digitalWrite(LED, HIGH);
    }

    Serial.print(mA);
    Serial.print(F(" "));
  }

  Serial.print(millis() - lastOutput); // shows 250-320 msec
  Serial.println();
  lastOutput = millis();

  if (!isany)
    digitalWrite(LED, LOW);

  delay(1); // to operate LED
}

I use David A. Mellis soultion with 8Mhz (clock runs good, I measured) and serial with 9200baud. It can be a bottleneck, I will try the 115200 baud rate (now use 9200 for safety, because attiny85 talks with an atmega328. Both can use 115200, but it should be safe for me) and reduce the serial communication. I will disable RX side of serial (stopListening).
I have researched some improvement:

do now use abs(ina219.getCurrent_mA()), because abs() is a macro only and it occurs calculate exporession two times, use abs with local variable only.
use delay() to make the LED works

Thank you for your work, I will reply when I will get measure info.
ps: this is a DCC model train occupancy detector (is a train on the track?), when I measure AC current. Not a real AC (sinus wave), it just change the polarity very fast to send digital info. This is the reason of abs() use.

Problem solved. The RX input of SoftwareSerial add 250ms delay to my code. If I used stopListening, 6x measure is about 20ms. I have refactored code: output is a cstring, eliminate String object, etc.
I must use delay to slow down sketch to 10 iteration / sec. With 9600 baud. stopListening is the must.
#if not defined (__AVR_ATtiny85__)
#error "Unknown processor, can run on ATtiny85"
#endif

#include <SoftwareSerial.h>
#include <TinyWireM.h>

#include "INA219_WE.h"

#define TX            3
#define RX            99 //supress listening, earlier 2
#define LED           1
#define I2C_ADDRESS   0x40

SoftwareSerial Serial(RX, TX);
INA219_WE ina219 = INA219_WE(I2C_ADDRESS);

void setup() {
  pinMode(LED, OUTPUT);

  Serial.begin(9600);
  Serial.stopListening();

  TinyWireM.begin();

  if (!ina219.init()) {
    Serial.println(F("INA219 NOT connected"));
    while (1) {}
  }
  else {
    Serial.println(F("INA219 connected"));
  }

  ina219.setMeasureMode(CONTINUOUS);
  ina219.setADCMode(SAMPLE_MODE_16);
  ina219.setBusRange(BRNG_16);
  ina219.setPGain(PG_40);
}

char output [6 * (4 + 1)] = "";
uint32_t lastOutput = 0;
void loop() {
  uint8_t isany = false;

  output[0] = '\0';
  for (int i = 0; i < 6; i++) {

    int16_t mA = (int16_t)ina219.getCurrent_mA();
    mA = abs(mA);

    if (mA != 0) {
      isany = true;
      digitalWrite(LED, HIGH);
    }

    char value[4 + 1];
    sprintf(value, "%02X", mA);
    strcat (output, value);

    delay(10);
  }

  Serial.println(output);

  if (!isany)
    digitalWrite(LED, LOW);

  delay(10); // to operate LED
}

Thank you for your support!

Great that you found the bottleneck! Good luck for your project.


[Enhancement] Visualization of the flow

I want to be able to visualize the flow with a selected number of specifications

The steps in the given set of specifications are the nodes.
Two consecutive steps are connected with an edge with the arrow pointing towards the latter step.
I want an ability to manually select the specifications from a given test suite.
I want an ability to give a filter criteria to select the specifications to plot the information.

Closing this as Gauge will not be implementing this feature.


Image class has its edge pixel data distorted in a similiar way that "texture_repeat" changes higher level textures with no way to disable it

Godot version
4.0.3.stable
System information
Windows 10 - v4.0.3.stable - Vulkan (Forward+)
Issue description
When using Image.get_region(Rect2i) to return a certain region of an Image, and using that image data to construct a HeightMapShape3D collision shape, there is a glitch with the borders equivalent to setting"texture_repeat" to true on a StandardMaterial3D. However, there is no method or parameter to change on the Image class to disable this "texture repeat" artifact.
In the screenshot below, you can see that along chunk borders, the seam vertices of the collision shape are elevated far above the surrounding heightmap. This is because, in the Image data itself, there are problematic lighter colored pixels on the image border, resulting in the verts being placed higher than they should and breaking otherwise seamless chunks. This same issue occurred when I was creating the heightmap mesh vertex shader (visual shader). However, in that case, I was able to disable "texture_repeat" on the TextureParameter2D to fix the problem.

I don't know why "texture_repeat" is enabled by default for pixel data in the Image class (if that is actually what's happening) but I have not found a method to disable it. I need the pure pixel data, which because the Image is taken as a region of another Image, I know for certain is seamless. I've tried resizing, cropping, etc. and found no way to get rid of those incorrect edge pixels.
Steps to reproduce

var new_image = Image.get_region(Rect2i)

Constructing a new image from scratch or from texture will have these "bugged" edge pixels inside its Image.get_data() by default, whether it is constructed through Image.get_region() or Image.new()
Minimal reproduction project
The project below is an example I threw together to try and visually demonstrate the issue.
Plane1 has the original Texture2D image as an albedo. (you can toggle texture_repeat to see the seam disappear)
Plane2 has an ImageTexture as albedo. You can disable texture_repeat on Plane2 as well, but that is through an option on StandardMaterial3D. There is no way to do it with the raw Image data created from get_region(), which is where the issue lies.
ImageGetRect_BugExample.zip

Can you actually verify that this is from the image function and not something else? Because there's really not anything in the code that does this from what I can see

Your MRP doesn't actually describe the issue you are describing, turning repeat off does make the effect go away, please add an MRP showing the effect you are talking about, concerning the get_data

Your MRP doesn't actually describe the issue you are describing, turning repeat off does make the effect go away, please add an MRP showing the effect you are talking about, concerning the get_data, and so we can see that this isn't caused by some other issue or error in setting it up

I will try and reproduce MRP with the collision shape generation code I'm currently using. In the meantime I'll share the collision shape generation function.

How are you identifying this edge issue in the data?

I've encountered the same problem when making the heightmap vertex shader. I set the albedo texture to the same texture used to create the vertex offsets, (both used the same Image resource). Visually describing the resulting heightmap mesh; peaks were white and pits were black. Ratio is 1 pixel = 1 meter. Pixels and vertices lined up as expected. The unwanted artifacts around the edges were white, and peaked unnaturally just as the shader should have deformed them based off pixel color.

Does this go away for you if you disable mipmaps in import?
You should be able to use get_mipmap_offset(1) to get the end of the raw data (I haven't tested that it gives the right offset, but it should I believe), or just use clear_mipmaps, or just not generate them in the first place

I import the original texture as an Image. There are no import options, however I make sure to clear_mipmaps() anyways in the code before accessing get_data()
Collision Generation Function:
func _generate_collision():

	#>  chunk_size = 128    (128x128 = Size of Chunk Heightmap Image
	#>  map_size = 1024     (1024x1024 = Size of Base Heightmap Image)

	ref_terrain_mesh.hide()
	var heightmap_img : Image = chunk_heightmap_img

	heightmap_img.decompress()
	heightmap_img.normal_map_to_xy()
	heightmap_img.convert(Image.FORMAT_RF)
	heightmap_img.clear_mipmaps()
	heightmap_img.resize(chunk_size+1, chunk_size+1)
	heightmap_img.clear_mipmaps()

	var data = heightmap_img.get_data().to_float32_array()
	for i in range(0, data.size()):
		data[i] *= heightmap_scale
		#data[i] -= heightmap_scale * 0.00001

	var ncs_hm := HeightMapShape3D.new()

	ncs_hm.map_depth = heightmap_img.get_height() #chunk_size+1
	ncs_hm.map_width = heightmap_img.get_width() #chunk_size+1
	ncs_hm.map_data = data

	ref_collision_shape.shape = ncs_hm

Indeed it seems I did misunderstand how the methods worked. This issue is due to a miscalculation on my end, and this issue will be closed.


Support parsing standalone pdbs which is useful for snupkg

By doing some null checks.
Example when viewing details for pdb inside of snupkg:
Before:

After:

Snupkgs should still have the dll's it though? The main issue is that we try to convert Windows PDB's to Portable so we can do the rest uniformly. Windows PDB's need to have the PE file passed in too.
This also works for .symbols.nupkg files, so while technically snupkg does not support windows pdb's, .symbols.nupkg does.

When I use
<IncludeSymbols>true</IncludeSymbols>
<SymbolPackageFormat>snupkg</SymbolPackageFormat>

only the PDB is inside the .snupkg

Yes, and I haven't changed the path for Windows PDBs. So now it is possible to get more info about portable PDBs without the corresponding DLL.
https://github.com/NuGetPackageExplorer/NuGetPackageExplorer/blob/f486e94454d0e79ff408d65afcf4325ab32b5709/Core/AssemblyMetadata/AssemblyDebugParser.cs#L25-L28
Before a NRE would be thrown for portable PDBs (and catched).


Am I using it wrong?

Hi!
I'm trying to get into object detection. I've tried to use your SSD7 model on Udacity dataset to detect traffic lights. I've managed to sometimes to spot some traffic lights, but nothing more. It wasn't doing well, so I tried more simple example.
Now I'm trying to use it for one of the simplest tasks - recognizing digits.
I'm generating artificial dataset using this notebook.
I'm training in this notebook using your ssd7 model. But as you can see in the pic in the bottom of the notebook, it is not doing well. I don't understand why it not working at all.
Can you please point me what am I doing wrong?
Thanks, Alexander.

Hey Alexander, the problem is likely in the scaling factors you chose. You set all scaling factors to 1.0, which is a non-sensical anchor box size for the size of the objects you are trying to detect. All anchor boxes will be as large as the training images themselves. This results in a bad matching process between anchor boxes and ground truth boxes: For any given ground truth box, no anchor box will meet the required IoU similarity with that ground truth box since all anchor boxes are way too large, which leads to the anchor box with the highest IoU similarity to be matched to that ground truth box. Since those IoU similarities with the huge anchor boxes don't have much meaning, the matching process becomes meaningless and a bad target tensor will be created.
Try the following:
Set all scaling factors to a value that leads to anchor boxes whose sizes correspond to the sizes of the objects you are trying to detect. Set the aspect ratios accordingly, too. The sizes and shapes of the anchor boxes should be similar to the sizes and shapes of the objects you are trying to detect. In your case, since all objects you're trying to detect have the same shape and size, try setting all scaling factors and aspect ratios to the same values. Also, it is not immediately clear whether there are any advantages or disadvantages to using multiple predictor layers versus using only one predictor layers if all detectable objects are the same size.
Let me know if this solves the problem.

Thank you for the reply!
I chose these scaling factors because I thought it would be easier to learn. I've used only 1 font and 1 font size, so I basically wanted it to overfit very easy (but I've never seen loss of 0).
I tried these:
1.)
scales = [0.7, 0.8, 0.9, 1.0, 1.2]
aspect_ratios = [0.5, 1.0, 2.0]
nothing changed
2.)
scales = None
aspect_ratios = [1.0]
threw an error
Also, aspect_ratios of [1.0, 1.0] throws an error (something with tensor shapes).
So, what's wrong?

Blind trial and error won't help for setting the scales, you have to compute meaningful values based on your dataset.  The important point here is that the scaling factors need to represent what fraction of the total image size the detectable objects are. Your ground truth boxes around the digits are only 20% by 10% of the image size, so scaling factors of 0.7 and larger are still way too large. Choosing the scaling factors and (somewhat less importantly, but still importantly) the aspect ratios carefully has strong implications on the matching process.
About the error when passing scales = None: I've (hopefully) just fixed this.

Thanks for clarifying!
I fetched updates for your .py files and now it throws a "numpy" error.

ValueError: Error when checking model target: the list of Numpy arrays that you are passing to your model is not the size the model expected. Expected to see 1 array(s), but instead got the following list of 128 arrays.
128 is batch_size.

So, another batch of blind trials and errors:
1.
scales = None
aspect_ratios = [0.5]
batch_size = 128
here
2.
scales = None
aspect_ratios = [0.5]
batch_size = 1
here
3.
scales = [0.14, 0.14, 0.14, 0.14, 0.14]
aspect_ratios = [0.5]
batch_size = 128
here
4.
scales = [0.12, 0.13, 0.14, 0.15, 0.16]
aspect_ratios = [0.45, 0.5, 0.55]
batch_size = 128
here
I haven't figured yet why where should be different predictor level, it would take some time.

I will only get a chance to try to reproduce your error on Monday, but I think I know where the ValueError comes from.
Could you please try the following: In the arguments of val_dataset.generate() in the Jupyter notebook, set train=True. That should get rid of the error. Please let me know if that works.
Also, another thing: Note that scales and min_scale/max_scale are meant to be used either/or. Whenever you set scales = None, then min_scale and max_scale will be used to compute the scale values. So if you set scales = None like in your first two examples, but at the same time you leave min_scale = 1.0 and max_scale = 1.0, then all scaling factors will be 1.0 and you will get the same useless results as in your initial trials. So if you want to set scales = None, then you have to set min_scale and max_scale to some values around 0.14 in order for the model to work as you intend.

Thank you so much!
It is finally working! ;-)
After some trials & erros I figured that I used "centroids" instead of "minmax" when plotting boxes on a picture in the last cell of the jupyter notebook. That's why it was SO messed up ;-)

You're welcome, I'm glad it's working now!


Changing clangd format options?

Hello, I'm not sure how to edit the clangd format options on coc-nvim :Format. Clangd defaults to LLVM format guidelines. Is there a way to modify this setting to my liking as done by clang-format?

clangd will respect formatting options specified in a .clang-format file in any of the parent directory of a file. you can also use fallback-style flag, but it will only take affect when there's no .clang-format file at all.


New architecture

Hello. Are there any plans to update this library to support TurboModules? If so, is there anything we can help with?

I updated this package, can we merge this PR? @Junina


Set InstanceRoleName output from IAMRole resource

If the InstanceRoleName parameter (optional) is unset, the CloudFormation stack generates a name for you based on the stack name.
This commit updates the output to extract the name from the resource directly which accounts for both cases, and should be more reliable going forward.
Follow up to https://github.com/buildkite/elastic-ci-stack-for-aws/pull/421. Addresses https://github.com/buildkite/elastic-ci-stack-for-aws/pull/421#issuecomment-392137428

Ugh, apologies, I should have caught this in code review of #421 😓

Updated! The !GetAtt syntax is pretty cool :+1:

Yup, been meaning to convert all of our templates to use it. We convert them to JSON as part of the build process. I only recently figured out how to support the !Blah syntax (it's a bit of YAML hack).

Thanks @dblandin!


Right sidebar shows the "Next" item from the bottom right.

Describe the bug
So, we use Read the Docs hosting service, and use the wonderful Sphinx Book theme. I call the theme in my conf.py with this:
html_theme = "sphinx_book_theme"
This problem didn't exist until now. All of a sudden my right side bar started showing the "Next" item. See the screenshot below:

It looks like this is a new bug? My local builds use the same Sphinx Book theme but it is at least 3 months old (I downloaded to my Mac) and it does not show this problem. See the screenshot from my local build:

Does anyone have a suggestion how I can fix this? Thanks!

Hey @rajk-cerebras just to clarify, what version of sphinx-book-theme does this occur for, I assume the newest v0.0.42?
Also do you have a link to the page where it occurs?
@choldgraf, @DrDrij possibly another side-effect of https://github.com/executablebooks/sphinx-book-theme/commit/76fe7e77883d8029f8f253387b92f28d202fb59e 😬 ?

Hey @chrisjsewell ahh, actually not sure what the version number is. But my guess is it is the newest? I say that because I am thinking when I did the build just a few minutes ago on Read The Docs, it fetched the newest version of the theme, because I do not have that theme placed in my docs repo. So RTD must've gotten it from the latest version repo.
I was afraid you'd ask for the page link. This is a private page and we are a bit paranoid about not sharing private docs pages links. Can I send you that link privately instead of posting it here?

Yeh no problem, I'm at<EMAIL_ADDRESS>

Awesome, just sent it.

Yep thanks, so this appears to be only an issue on a page with sphinx-panels (I guessing thats what you are using for the panels?)
sphinx-panels uses bootstrap which I imagine is conflicting with https://github.com/executablebooks/sphinx-book-theme/commit/76fe7e77883d8029f8f253387b92f28d202fb59e.
It is what I thought was the reason for #303, although @DrDrij thought it was due to the lack of a right ToC, but maybe a mix of the two.

Nah, I am not using sphinx-panels. I am using a raw html code like this in my index.rst file:
.. raw:: html

...

Nah, I am not using sphinx-panels. I am using a raw html code like this in my index.rst file:

Ah fair, well its HTML very reminiscent of what sphinx-panels creates anyhow, so would probably generate a similar issue.
In fact, I created a page for another of my projects with sphinx-panels, very similar to yours: https://aiida.readthedocs.io/projects/aiida-core/en/latest/ 😄

In fact, I created a page for another of my projects with sphinx-panels, very similar to yours: https://aiida.readthedocs.io/projects/aiida-core/en/latest/ 😄

Looks awesome! So you don't have the problem I am facing, woe is me, why? :- ( I guess you don't use the latest version of the theme?

Ah well I'm currently using the pydata-sphinx-theme, which this theme derives from. Was planning to switch once @choldgraf releases a v0.1.0 😜

Is there a way I can set some variable in conf.py to use a specific version of the theme? I am sure this is not the first time I will run into a situation like this.

@chrisjsewell I keep wanting to release v0.1.0 but we keeping finding (or introducing 😬) bugs!

You would pin the install version in your requirements.txt (sphinx-book-theme==0.0.41) or however you are installing the packages to use for RTD

thanks, let me try that, that's a way to stay one version behind :- )

So I pinned the version to 0.0.39 and the problem went away. Thanks a lot, @chrisjsewell, any such quick help is a ton of help!

I don't know the protocol, should I close this issue, or one of you will do it? Either way is fine with me.

thats fine thanks @rajk-cerebras we will close it once we've found a fix 👍

I can't seem to replicate or find an occurrence of this bug.
I don't think there should be any changes from the mobile-menu PR effecting the next button (it sits in the main content div).
@chrisjsewell Happy to attack/troubleshoot the bug if I can find an example

I believe this is now resolved but please ping here if the error persists on the latest book theme


fix rust-toolchain of flake

Alright, this time it should work. rust-analyzer is added in the flake itself so you will not install rust-analyzer if you don't use the dev-shell from the flake.

Okay I won't do anything funky stuff this time around. I will test this throughly and then only I will merge this.

Looks fine. Merging.


best may to get x-linked variants

I'm new to using gemini, and used it to identify de novo, and autosomal recessive variants in our ~200 families. In your documents you mention to you can get X chromosome variants using the gt-filter tool, but is there a may to identify variants that are hemizygous in the child and heterozygous in the mother.
Thanks for your time, and great tool (which has saved we lots of time!).
Andrew

We have X-linked recessive and dominant coming in the next release. The logic and tests are implemented in the inheritance module, we just need to incorporate into gemini. I think this will be in the next couple of weeks.

great, thanks for letting me know.
Andrew

x-linked dominant, recessive, and de novo are now in master. They'll be in the next release.