[BUG] cannot properly save and load the  TF Retrieval model

Bug description
When we save Two-Tower model we get the following warnings for every feature.  To reproduce the following errors, please run the  05-Retrieval-Model.ipynb example. and then save and reload the model with the following scripts.
model.save('two_tower')
INFO:tensorflow:Unsupported signature for serialization: (({'user_id': TensorSpec(shape=(None, 1), dtype=tf.int64, name='inputs/user_id'), 'item_id': TensorSpec(shape=(None, 1), dtype=tf.int64, name='inputs/item_id'), 'item_category': TensorSpec(shape=(None, 1), dtype=tf.int64, name='inputs/item_category'), 'item_shop': TensorSpec(shape=(None, 1), dtype=tf.int64, name='inputs/item_shop'), 'item_brand': TensorSpec(shape=(None, 1), dtype=tf.int64, name='inputs/item_brand'), 'user_shops': TensorSpec(shape=(None, 1), dtype=tf.int64, name='inputs/user_shops'), 'user_profile': TensorSpec(shape=(None, 1), dtype=tf.int64, name='inputs/user_profile'), 'user_group': TensorSpec(shape=(None, 1), dtype=tf.int64, name='inputs/user_group'), 'user_gender': TensorSpec(shape=(None, 1), dtype=tf.int64, name='inputs/user_gender'), 'user_age': TensorSpec(shape=(None, 1), dtype=tf.int64, name='inputs/user_age'), 'user_consumption_2': TensorSpec(shape=(None, 1), dtype=tf.int64, name='inputs/user_consumption_2'), 'user_is_occupied': TensorSpec(shape=(None, 1), dtype=tf.int64, name='inputs/user_is_occupied'), 'user_geography': TensorSpec(shape=(None, 1), dtype=tf.int64, name='inputs/user_geography'), 'user_intentions': TensorSpec(shape=(None, 1), dtype=tf.int64, name='inputs/user_intentions'), 'user_brands': TensorSpec(shape=(None, 1), dtype=tf.int64, name='inputs/user_brands'), 'user_categories': TensorSpec(shape=(None, 1), dtype=tf.int64, name='inputs/user_categories')}, False), {'feature_context': <tensorflow.python.framework.func_graph.UnknownArgument object at 0x7f8d32cff040>}).

Then when we load back the saved model we get the following error:
reloaded = tf.keras.models.load_model('two_tower')

--------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
Input In [16], in <cell line: 1>()
----> 1 reloaded = tf.keras.models.load_model('two_tower')

File /usr/local/lib/python3.8/dist-packages/keras/utils/traceback_utils.py:67, in filter_traceback.<locals>.error_handler(*args, **kwargs)
     65 except Exception as e:  # pylint: disable=broad-except
     66   filtered_tb = _process_traceback_frames(e.__traceback__)
---> 67   raise e.with_traceback(filtered_tb) from None
     68 finally:
     69   del filtered_tb

File /usr/local/lib/python3.8/dist-packages/keras/saving/saved_model/load.py:1000, in revive_custom_object(identifier, metadata)
    998   return revived_cls._init_from_metadata(metadata)  # pylint: disable=protected-access
    999 else:
-> 1000   raise ValueError(
   1001       f'Unable to restore custom object of type {identifier}. '
   1002       f'Please make sure that any custom layers are included in the '
   1003       f'`custom_objects` arg when calling `load_model()` and make sure that '
   1004       f'all layers implement `get_config` and `from_config`.')

ValueError: Unable to restore custom object of type _tf_keras_metric. Please make sure that any custom layers are included in the `custom_objects` arg when calling `load_model()` and make sure that all layers implement `get_config` and `from_config`.

Expected behavior

Environment details

Merlin version:
Platform:
Python version:
PyTorch version (GPU?):
Tensorflow version (GPU?): TF 2.8.0

Using merlin-tensorflow-training:22.05 docker image with the latest main branches pulled.

This is also required in this RMP https://github.com/NVIDIA-Merlin/Merlin/issues/271

It looks like we've got some custom objects (custom metrics in this case) that need to be specified when calling load_model.
custom_objects = {
  "RecallAt": mm.RecallAt,
  "NDCGAt": mm.NDCGAt,
}
reloaded = tf.keras.models.load_model('two_tower', custom_objects=custom_objects)

We could potentially create a load_model helper that adds all the known custom objects automatically.

Update. The saving of TwoTowerModel is now working. (22.08)
However, loading is still not working. Due to an unbound query variable in ItemRetrievalScorer when model is reloaded. And trying with the example in the ticket description should now give a new error with a message similar to this:
AssertionError: Found 1 Python objects that were not bound to checkpointed values, likely due to changes in the Python program. Showing 1 of 1 unmatched objects: [<tf.Variable 'query:0' shape=(None, 4) dtype=float32, numpy=array([[0., 0., 0., 0.]], dtype=float32)>]

There is work on-going in #633 that is taking us toward a place where we can replace this retrieval scorer with a new implementation that won't have this issue. Aiming for the next release 22.09

PR #790 introduces the definition of TwoTowerModelV2 that can be saved and loaded correctly


Add rand() and fix memoization bug. Change that extensions can't be changed

later is a fairly breaking change but otherwise id look ups are cache misses

going to merge this and make a new PR for break/continue


Allow reading specific Cop documentation from command-line

Is your feature request related to a problem? Please describe.
There are a few situations where you have only the Cop name but you are not able to figure out what it is enforcing. Here are a few examples:

CI fails
Someone disabled a cop inline on the source code
A new issue is discussing enabling/disabling a specific Cop

Describe the solution you'd like
It would be useful to have a command on the command line that allowed me to paste the cop and get a simple documentation. (so it can be offline friendly).
Here is a minimal implementation:
$ rubocop show Lint/SafeNavigationConsistency

This cop check to make sure that if safe navigation is used for a method call in an && or || condition that safe navigation is used for all method calls on that same object.

An improved implementation:
$ rubocop show Lint/SafeNavigationConsistency

This cop check to make sure that if safe navigation is used for a method call in an && or || condition that safe navigation is used for all method calls on that same object.

 - Enabled? Yes/No (default: Yes)
 - Safe? Yes
 - Supports autocorrection? Yes

Configurable attributes:
  - AllowedMethods: Array (default: [present?, blank?, presence, try, try!])

Describe alternatives you've considered
An alternative would be to link to the external documentation pointing to the right page, for example: https://www.rubocop.org/en/latest/cops_lint/#lintsafenavigationconsistency (not offline friendly)
Additional context
Add any other context or screenshots about the feature request here.

We already have --show-cops on the command line, which does at least show the current configuration. For example:
$ rubocop --show-cops Lint/SafeNavigationConsistency
# Supports --auto-correct
Lint/SafeNavigationConsistency:
  Description: Check to make sure that if safe navigation is used for a method call
    in an `&&` or `||` condition that safe navigation is used for all method calls on
    that same object.
  Enabled: true
  VersionAdded: '0.55'
  VersionChanged: '0.77'
  AllowedMethods:
  - present?
  - blank?
  - presence
  - try
  - try!

This might be a good place to start, and see what other information would be good to include?

A potential problem here is that we'd have to generate the yarddoc for the cop on the fly, which is a bit slow. I'm thinking about how to implement this.


New Features

A more effective way of organizing the blogposts, time, author etc

I could merge it automatically but i want u to see it


Merge LoRA model before initializing DS inference in text-generation example

What does this PR do?

As per title.
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Force VDC "master" files to use .nc extension

The various *create command line utilities (vdccreate, wrfvdcreate, etc.) should probably force the resulting file to use the .nc suffix. It makes life easier for the vapor gui file browser to select master files.
This is an issue we should discuss as there may be better alternatives.

Could we consider changing the official extension to .vdc to avoid confusion with regular netCDF files?

We should discuss.

On Dec 19, 2018, at 1:37 PM, StasJ<EMAIL_ADDRESS>wrote:
Could we consider changing the official extension to .vdc to avoid confusion with regular netCDF files?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

John Clyne
National Center for Atmospheric Research
303.497.1236 (w),<PHONE_NUMBER> (c)
<EMAIL_ADDRESS>


Prometheus monitoring fails because of the corrupted metrics response

Bug Description
Looks like the metrics response is invalid, and prometheus cannot collect data from the metrics endpoint.
If I have understood right, the metrics log output comes now from two sources called py_metrics and rust_metrics. Between those two metrics group, there's no newline character, which makes prometheus incapable to parse and collect the data.
I'm not sure why the trailing newline has been removed from rust_metrics, but I think it causes the issue.
https://github.com/bytewax/bytewax/blob/d3b1e441e9b624473fd49da63c72d68e5fd77323/src/webserver/mod.rs#L69-L72
However, I'm not able to verify that as I couldn't set up the dev environment for my data pipeline.
Python version (python -V)
Python 3.11.8
Bytewax version (pip list | grep bytewax)
bytewax 0.19.1
Operating System version (uname -morp)
6.6.22-linuxkit aarch64 unknown GNU/Linux
Relevant log output
...
...
# HELP otel_scope_info Instrumentation Scope metadata
# TYPE otel_scope_info gauge
otel_scope_info{otel_scope_name="bytewax"} 1
# HELP target_info Target metadata
# TYPE target_info gauge
target_info{service_name="unknown_service"} 1# HELP python_gc_objects_collected_total Objects collected during gc
# TYPE python_gc_objects_collected_total counter
python_gc_objects_collected_total{generation="0"} 557.0
python_gc_objects_collected_total{generation="1"} 211.0
python_gc_objects_collected_total{generation="2"} 40880.0
...
...

Steps to Reproduce
Not exact steps as my environment is a bit complicated to set up, but here are rough instructions:

Start up bytewax with BYTEWAX_DATAFLOW_API_ENABLED=true env variable.
Check that bytewax metrics are exposed

curl localhost:3030/metrics
... prints metrics ...

Start up Prometheus (example conf)

global:
  scrape_interval: 10s

scrape_configs:
  - job_name: bytewax
    honor_timestamps: true
    metrics_path: /metrics
    scheme: http
    static_configs:
      - targets:
          - localhost:3030

Prometheus says the target is down. (http://localhost:9090/targets?search=)

strconv.ParseFloat: parsing "1#": invalid syntax while parsing: "target_info{service_name=\"unknown_service\"} 1#"

When I upgraded to Bytewax v0.19 and recognised the issue, the version of Prometheus was 2.49.1. I also tested with the newest one 2.51.2, but the parsing error still exists with the same message.
If it matters, my environment is running on Docker, and it uses bytewax:0.19.1-python3.11 as a base image.


Roll Plugins from 5c11747ae143 to 897ec59cd73d (5 revisions)

https://github.com/flutter/plugins/compare/5c11747ae143...897ec59cd73d
2022-11-04<EMAIL_ADDRESS>[ci] Removes unnecessary overrides. (flutter/plugins#6669)
2022-11-03<EMAIL_ADDRESS>Roll Flutter from 0e98194681fd to e6300da2c39f (46 revisions) (flutter/plugins#6668)
2022-11-03<EMAIL_ADDRESS>Roll Flutter (stable) from 6928314d505d to e99c9c7cd9f6 (2 revisions) (flutter/plugins#6667)
2022-11-03<EMAIL_ADDRESS>[video_player]add integration test for inverted aspect ratio bug (flutter/plugins#6662)
2022-11-02<EMAIL_ADDRESS>[ci]Temporarily disable starqlteue on firebase device lab (flutter/plugins#6657)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-plugins-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Keep scrolling normally after sections?

Hello,
The plugin works great, but after all the sections I have a footer, would it be possible to scroll to that footer normally after the last section?

Hi @vk011,
Sorry for the late reply, I didn't really got your problem, can you explain in detail? Thanks.

I am sorry I somehow posted this in the wrong place.


Size preview iframes to fit contents

When we create preview iframes, we always use the same height.  It would be cool if we could tell the iframe to resize to fit its contents (to a maximum threshold).  There are some libraries to do this, which require you to inject some JS into the document within the iframe and then send a message back to the host.  Not trivial, but doable.
https://chatcraft.org/c/humphd/AsI25XEORVC57AKsPI-rF is a good example of where I want it.

https://github.com/davidjbradshaw/iframe-resizer and https://github.com/davidjbradshaw/iframe-resizer-react are interesting.

Fixed in #267


[Doc] add a migration guide for @azure/abort-controller v2

API change check
API changes are not detected in this pull request.


Add lambda lifter

Lifts both lambas and lets to supercombinators (top-level functions in this case).
It also renames variables and new supercombinators
This PR makes several changes to the grammar.
Example:
f x = let double = \y. y+y in (\x. x+y) 4;

turns into

f x_0 = sc_1 4;
double_3 y_4 = y_4 + y_4;
sc_1 x_2 = x_2 + y

Need to fix the interpreter before merge!

Interpreter is fixed, and I have started on testing in the branch 'testing' but i needs types to generate correct programs so it will have to wait.


Separate local and global variables in block palette

Resolves #1636.

This PR will still need a bit of work, but here is a first draft, to prompt code and design discussion.
Design question
Which labels should show up? Here are the two main options to consider:

Given there is a global variable, only show the "For all sprites" label when there is also at least one local variable.
Make labels show up according to whether there are any variables of the particular scope. That is, if there are any local variables, show the label "For this sprite only", and if there are any global variables, show the label "For all sprites".
(Alternatively, always show both labels, alongside the text "None" when there are no variables of a particular scope, or never show either label, as in Scratch 1.4.)

Those (first) two options basically combat each other in a crucial way. The first takes the perspective of, "Don't expose the user to potentially confusing/overwhelming behavior." The second takes, "Do expose the user to potentially interesting/useful/educational behavior." To make these clear, see a case of a beginning user creating a variable. They click on "Make a Variable", type a name (like Fruit), and leave "For all sprites" selected, since it's the default one and they aren't sure what the difference between the options is. Now assume we go with option 2 above. They will, immediately and throughout working on their project, see the label "For all sprites". They might respond in one of three ways:

Confusion: Why did I make this sprite "for all sprites"? Does that matter? Should I be worried? Do I have to know the difference between "for all sprites" and "for this sprite only"?
Interest: All the same questions as above, but in an "I want to learn" / "I am curious" perspective!
Disregard: Just ignoring it.

I'm inclined towards the second option, because I think the vast majority of users will either ignore the label or be interested in the difference between "for all sprites" and "for this sprite only". I suppose a few might get confused or overwhelmed by the new label (maybe users with experience in 2.0 who never even paid attention to the local/global option). But it's not really that much of an extra load in the UI - that is, to the human using the UI - and it has a fairly obvious meaning, seeing as it's identical in language to the choice selected in the "make a variable" prompt.
As I said, I think option 2 is best here, but I'm interested in other opinions, if you have one.
Definitely TODOs

[ ] Figure out how to deal with the "For this sprite only" and "For all sprites" messages, regarding localization. These are already defined in scratch-gui, but I'm not sure how to access that same translation from scratch-blocks. Temporarily, I've added some messages to msg/messages.js.
[x] Clean up and separate the code for building the variable blocks, in such a way that it's easy to...
[x] Also do all this for list blocks.
[ ] In the screenshot, the gap between "Global B" and "For this sprite only" seems to be smaller than the one between "Local B" and "set Global A to". This should probably be the other way around.

Thanks so much for putting in all this thought on the issue! We used your demo as a starting point for discussion, just like you mentioned, and here is what we talked about:

We need to do a bit of work in understanding how people are actually using the local variables, and to see how the confusion comes up. Since it is a fairly high-ceiling feature, scratchers often seem to get around it by naming their variables something like "local ___".
We have in the past decided against adding more hierarchy to the toolbox (e.g. trying to avoid having nested categories), and this style would introduce another layer of hierarchy into this generally flat and physically small interface.
Would introducing a distinction require an extra distinction for cloud variables?
Other alternatives could be indicating the difference on the reporter blocks themselves, or together in the text on the block.

With those things in mind, I think I'll close this PR, repost some of this discussion/screenshot to the original image, and then schedule a feature time to look more into this.


Fix lint errors

Current coverage is 99.29%

Merging #40 into master will not affect coverage as of ffeb9c4

@@            master     #40   diff @@
======================================
  Files            2       2
  Stmts          141     141
  Branches         0       0
  Methods          0       0
======================================
  Hit            140     140
  Partial          0       0
  Missed           1       1

Review entire Coverage Diff as of ffeb9c4

Powered by Codecov. Updated on successful CI builds.


Update FAQ about migrating from PostgreSQL or MySQL

Fixes #2136

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/ecf5e9dde948f0faf097d34071ea96c26ce5f383/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/eb34497ff431af97b569cf86d41d282e96ad0022/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/e9ad6a37b8edb73d432438f0a5be5e2aa20c51dd/


Allow routes generated by addRoute to be listed in the plugin-content-docs sidebar

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Description
Currently, when a plugin/theme uses addRoute to register dynamic routes, these routes cannot be used in the sidebar. Besides, when accessing them directly, they have no wrapping layout (header/sidebar).
As we have an autogenerated sidebar item type, we could have a specific type to list all the sub-routes of a base route.
Has this been requested on Canny?
https://docusaurus.io/feature-requests/p/allow-routes-generated-by-addroute-to-be-listed-in-the-plugin-content-docs-sideb
Motivation
For plugin developers, using addRoute is way more powerful than performing an fs.createFileSync with a raw template, due to the following reasons:

A React component may be used instead of a raw component generator
This React component may be a part of the theme, and also be swizzled or overridden
There is no need to manage generated files/folders

This resource would be extremely useful for producing API Reference, such as documenting all the React components in a project, or all the TS typings, in a extensible scalable way.
API design
Add a new type to SidebarItemConfig to list all the sub-routes within a route:
export type SidebarItemRoutes = SidebarItemBase & {
  type: 'routes';
  label: string;
  baseRoute: string;
};

Real-life example:
A custom plugin generating dynamic routes (simplified version):
async contentLoaded({ content, actions }): Promise<void> {
  components.forEach(async component => {
    const componentData = await createData(`${component}.json`, JSON.stringify(component));

    addRoute({
      path: `/docs/react/${component.displayName}`,
      component: '@theme/ReactComponent',
       modules: {
         data: componentData,
       },
    });
  });
});

A Docusaurus configuration:
presets: [
  [
    '@docusaurus/preset-classic',
    /** @type {import('@docusaurus/preset-classic').Options} */
    {
      docs: {
        sidebarPath: require.resolve('./sidebars.js'),
      }
    }
  ]
],
plugins: [
  [
    'my-plugin',
      {
        src: ['../src/components/**'],
        tsconfig: '../tsconfig.json',
      },
  ],
],

And finally, a docusaurus sidebar configuration:
module.exports = {
  sidebar: [
    {
      type: 'routes',
      label: 'React components',
      baseRoute: '/docs/react'
    },
];

Have you tried building it?
As this affects docusaurus-plugin-content-docs, I'm afraid it can't be customized outside it.
Also it seems that content-docs has its own way to determine if a route should be wrapped by the layout or not.
Self-service

[X] I'd be willing to contribute this feature to Docusaurus myself.

As a general rule, plugins work in isolation: isolation of data, isolation of routes (they shouldn't shadow/overlap with each other). I believe you should enhance the docs plugin instead if you find that you need to reuse a lot of the layout and data.

Do you mean that the docusaurus-plugin-content-docs should still unaware of the routes, just physical files?
And then the consumer plugin should find a way to register the sidebar entries, and also add sidebar: 'sidebar' to the routes manifest? Is it possible?

Do you mean that the docusaurus-plugin-content-docs should still unaware of the routes, just physical files?

Yes, the docs plugin shouldn't know about anything outside the routes it generated by itself.
Typically, we assume that each plugin renders its data without touching other plugins' data. So if you find yourself needing to generate extra data for the docs page to consume, you should enhance the plugin instead.

Ok, big thanks for the clarification! I read the issue carefully and ran the project locally. But I still can't figure out how to inject the sidebar prop to the generated routes file. Could you help me?
Anyway, we can close the issue.

But I still can't figure out how to inject the sidebar prop to the generated routes file.

The sidebars data are loaded in the loadContent lifecycle, so if you managed to load all the data that you need, then you just need to add an extra addRoute call in the contentLoaded lifecycle. Have you checked the source code? The docs plugin is quite convoluted especially due to supporting versioning

The sidebar is a concept of the docs plugin. Even though the blog plugin also displays a sidebar, it's a totally different implementation.
If you implement your own plugin with a sidebar, you have 2 choices:

Create your own sidebar: a JSON tree that you can pass as a prop/data to your own UI. Eventually, you could re-use some existing theme components but be aware we may refactor them a bit.
Use or extend the docs plugin so that you have exactly the same UX as regular docs

You can leverage the docs plugin in 2 ways:

Have a build step, generate md files in website/docs folder at build time
Extend the docs plugin (quite hacky atm, see https://github.com/facebook/docusaurus/issues/4138#issuecomment-771042127) so that you can add/modify content of each docs version at your will (this may be complex)

We don't have a good public API for plugin extension so for now the best is to either implement your own UI or generate docs at build time.
Example: https://github.com/tgreyuk/typedoc-plugin-markdown/tree/master/packages/docusaurus-plugin-typedoc

Thanks! After deep-diving into the code, this issue, and plugins, I realized these two choices.
Just for curiosity, my actual contribution/ambition is to render docs for all the React components within a project:
https://github.com/atomicpages/docusaurus-plugin-react-docgen-typescript/pull/7
After all, it seems to me it's right to have a sidebar for each docgen tool (OpenAPI, Typedoc, react-docgen, etc), so I lost some interest in injecting routes into the sidebar.
Last, but not least, I still think that the docs plugin should be route-based instead of file-based, but I'm sure there are several implications, such as it would break the complete isolation perspective.

Last, but not least, I still think that the docs plugin should be route-based instead of file-based, but I'm sure there are several implications, such as it would break the complete isolation perspective.

Not sure what you mean here. For now routes are only generated using the files of the docs folder, but in the future we'd like to provide an API so that you could fetch docs and sync with a CMS, the files wouldn't necessarily need to exist on disc as long as the CMS is able to provide md string and eventually some frontmatter

Actually, all the routes registered by addRoute will be present in the routes.js file, not just the docs one.
As of a common web application, every route is a candidate to be listed in the sidebar.
However, if we think that the sidebar is exclusive to the docs plugin, then it doesn't make sense to list routes

Only docs-related routes appear in the docs sidebar. That wouldn't make any sense to have a blog post appear in the blog sidebar, nor our showcase page.

As of a common web application, every route is a candidate to be listed in the sidebar (at least is up to the developer to list a route or not).

I don't believe it's true. If you look at CMS-built site there's generally some kind of item category so that pages are tagged and appear in a menu, but nothing is automatic and has to be implemented.


Bug: ForBraceChecker doesn't see braces.

object Foo {
  def bar(): Unit = {
    // Use braces in for comprehensions
    for (b <- dort) {
      thing()
    }
  }
}

I misunderstood. It wants me to use for {b <- dort}. Is this intended? I think it should only check if there is more than 1 thing in the for comprehension or if there is a newline. The syntax for (i <- 0 until 10) is quite common.

Duplicate of #142


make welcome app "hub" editable

So we can disable the welcome app so it won't get added

And we can change the "Welcome Application is enabled" to "Welcome Application is not enabled". And the text below it when disabled... ermmm... "By enabling this we will prepare a set of examples to let you directly start playing with your new application..." perhaps?


Skip visiting classses generated for services in NodeFinder

Purpose
Fixes an issue with how classes generated for services were skipped in NodeFinder.
Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[x] Added necessary tests

[x] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

#29312 should also be fixed by this


🐛 SplitNetworkTransport conforming to updated NetworkTransport protocol.

For subscription we needed to install the sub-spec Apollo/WebSockets. This meant the SplitNetworkTransport needed updating to match the NetworkTransport protocol I originally changed (for the ability to pass through headers).
The headers aren't actually be used by the subscription, so I could have made an extension specifying a method without it, but since we're going to refactor this completely, I didn't bother with such details.

Looks good to me, do we have the tech debt of not needing this fork tracked anywhere?

Just created one - https://imaginecurve.atlassian.net/browse/FT4-592


Pull magic value regular expressions out into modules

@othiym23: This would be a semver-minor release for newly exposed API. The idea here is to make it possible to require() just the regular expressions for values like SEE LICENSE IN LICENSE.md and UNLICENSED. Those are npm-specific magic values, so they seem to belong in an npm-specific package.
I'd end up using these in npm/newww#996

Sorry. This may not have been necessary. We have this package returning flags on the result object for magic values.


Support async methods.

It could be nice to extend the API to async methods "resolves" & "rejects".
and use it like this
interface Test {
    method(): string;
}

const testStub: any = stubInterface<Test>();
testStub.method.resolves('stubbed');  //return fulfilled promise
testStub.method.rejects(New Error("some error");  //returns rejected promise

Hey,
you can use Promise.resolve(result) / Promise.reject(reason) as a return values f.e.:
interface Test {
    method(): string;
}

const testStub: any = stubInterface<Test>();
testStub.method.returns(Promise.resolve('stubbed'));  //return fulfilled promise
testStub.method.returns(Promise.reject(new Error("some error"));  //returns rejected promise


Feature request: Adding timing for text

Hi again. Would you consider adding a way to add timing information for overlay text? For example if I'm making a gif of a single back-and-forth conversation, it would be nice to have the first line displayed for x seconds, then the second line is displayed for y seconds.
I was thinking a possible way to implement it in the gui, perhaps something like the attached file.

Labels 1 and 2 point to the fields --similar to how you implemented it for selecting clip start and duration--  for the displayed text.
Label 3, which points to the plus button, is saying if you want to add a second line of text with different timing. Perhaps hitting the plus button adds a similar field right below it, or pops up a dialogue, like for selecting fonts. I could mock that up too, if what I'm saying is not clear.
Maybe this adds too much complexity to the program and is beyond the intended scope, but if not, it would be a cool feature to add. Thanks again, lovely program to use.

Hello @iuqiddis,
Thank you for the feature request.
Multiple time based text overlays are currently in the works. Below is a mock up demonstration.

You'll be able to click Add and it will display another text overlay. For each text overlay, you can set the font, size, left/right/up/down position, and the start and duration time.
This feature and being able to see the text in the preview are slated for the next major release.
I'll update this issue once the feature is finished.
Thank you for the kind words and I'm glad you're enjoying Gifcurry.
:+1:

awesome. looks great.
cheers.

Hello @iuqiddis,
Wanted to give you an update regarding the multiple time based text overlays feature.
It is almost complete and I'll be uploading a pre-release AppImage soon.

:+1:

Hello @iuqiddis,
You can now try out the pre-release AppImage for version <IP_ADDRESS>.
:+1:

The feature works great. I tried it on three different clips, easy enough to move timing and placement. thanks a lot!
can i ask about some other features introduced in version <IP_ADDRESS> in this thread? they are not related to the text overlaying interface. Or should i start a new thread for other comments or requests. thanks again!

cool.

(this is is related to the text overlay) Is there a possibility of adding a parameter for outline weight?

Previously the output was as .mp4 and the quality seemed a lot better (when going for the higher quality). The webm version, even at the 'insane' quality seems pretty low quality in the clips i've tried. I can reply later with a comparison between version 3 and 4 and upload it back with the source file you shared in a different thread (I assume its not some bug on my computer). BUT, coming to my request, would you consider having output options for both h264 and webm (also because safari doesn't support webm unfortunately)

Lastly (so sorry for the length), is it possible to implement importing embedded subtitles with timing and load them as pre-loaded text? This might be an edge case, but a lot of times I cut 2-3second scenes with dialogue (hence the timing for text) and often my sources have embedded subtitles. It would be cool if that can get atomically imported.

cheers.

Hello @iuqiddis,

Sure. Would this increase and decrease the size of the text outline? I could add it among the current text controls.

It would be great to add it as a configurable option but that just isn't tenable at the moment unfortunately. :disappointed:  Gifcurry will have to stay with the royalty free codec VP9 inside the WebM container for now. Even though Gifcurry does not distribute, implement, and/or link (neither statically or dynamically) with an implementation of any royalty-burdened (or any for that matter) codec, it is safer to stick with the WebM VP9 combination.
That said, the VP9 codec is very comparable to what you might find used in MP4 and in some cases it is better. The good news is I have increased the quality of the output video considerably to that of 1080p HD video. I've also increased the insane quality level by removing all color count restrictions. When saving as a video, Gifcurry now bypasses the GIF creation step and goes straight to creating the video after the annotation step.
Input video: https://svs.gsfc.nasa.gov/vis/a010000/a012000/a012034/Ice_Edge_UHD_YouTube.mp4
Sample output video (900 width @ Insane): https://streamable.com/zg03z
Sample output GIF (900 width @ High):

You can try out the new settings using the pre-release <IP_ADDRESS> AppImage.
In the future, Gifcurry will switch to the AV1 codec which should be supported by Safari since Apple joined the consortium.
For now, you can pass Gifcurry's output to Curlew or in the terminal just type ffmpeg -i gifcurry.webm gifcurry.mp4.

I could offer a load option for a Structured Subtitle Format (SRT) file that would then automatically create all of the texts. Or are you saying it would read the subtitles directly from the input video file? This may be difficult depending on how many languages are stored with the video. If they are "burned in" then it would not be possible to read them from the video file.

:+1:

yes, exactly. just decreasing the size of the outline. thanks.

I tried the app image link with one of my sources and yup, the insane quality is much nicer now (I'd consider it good/great, whereas the previous output was pretty poor).

For this particular scene, the previous <IP_ADDRESS> was outputting both 'insane' and 'low' webm at ~160 kB, which besides being strange, resulted in poor output quality in both cases. Now the 'insane' setting with AppImage download results in a ~1.0MB high quality output which is awesome; for comparison the 'medium' setting GIF is 24MB and a lot poorer in quality (but still usaable i suppose). I'm using 720p resolution as output to play around with the clips, thus the high GIF size.

Version
Quality
Size (kB)
Qualitative Assessment

new
insane
1121
high/great

new
high
1121
high/great

new
medium
937
maybe slightly worse than above two

new
low
833
slightly worse. still perfectly adaquate

old
insane
158
blocky/not good

old
high
158
blocky/not good

old
low
165
blocky/not good

For a different clip, I get similar ratios between insane and low (~83%):

Version
Quality
Size (kB)
Qualitative Assessment

new
insane
770
high/great

new
high
770
high/great

new
low
636
slightly worse. still perfectly adaquate

I was going to suggest maybe playing around with the settings to only have insane, medium and low, where the low setting generates a file that is approximately 50-60% in size of the insane output? Based on my meager testing on two action-based clips, the the difference between insane and high webm doesn't seem to be much (as seen in the above tables).
As for going for webm, yeah, I can understand only having the option for the royalty-free option. I think you're playing it safe, but that's your call :) . I'll just stick to ffmpeg for h264 output, as you suggested.

I was thinking more along just loading the embedded subtitles from the source clip to start off with. I assumed this would exclude burned-in subtitles. Maybe default to the first embedded track, or look for a particular language (English in my case, though of course that would vary by the user's wants or needs).

Thanks again for listening. Really appreciate your responses. Cheers.

Hello @iuqiddis,

I have updated the pre-release AppImage to include an outline size selection. I also added text rotation.

I went ahead and tweaked the settings reducing the quality options to only High, Medium, and Low.

Input video: https://svs.gsfc.nasa.gov/vis/a010000/a012000/a012034/Ice_Edge_UHD_YouTube.mp4

Start second: 60
Duration seconds: 1

Video output at 1280x720:

High: 904 kB
Medium: 246 kB
Low: 182 kB

I could do something similar that I do for Movie Monad where it will list the available languages, you would select one, and then it would import them for you.

:+1:

Hello @iuqiddis
You can now import SRT subtitles in version <IP_ADDRESS>.
:+1:


Adds org.apache.commons:commons-lang3:3.4

Lots o' other PRs first.

all comments done, merging.
and yes your theory on comments sticking around is correct.


Spectral Emulator Documentation and Examples

While the spectral emulator does a fair amount of heavy lifting inside the code, there isn't much in the documentation about setup and use.
As @gully and @kgullikson88 noted, the documentation between the grid_tools.py and the emulator code is outdated and confusing. 1c6481caef2acd70faa471fe8a7a86e546630073 is a first effort towards fixing this, but more work is needed, especially regarding the emulator.

Just an update here: While writing the documentation, I'm taking this opportunity to revamp the design of the spectral emulator. Right now the Gaussian Processes (one for each eigenspectrum) are optimized independently, contingent on how well they reproduce the weights for that given eigenspectrum.
Ideally, the GP hyperparameters should be optimized contingent on how well the linear reconstruction of  eigenspectra reproduces the original synthetic spectrum, including an error ("nugget") term to account for the truncation of the eigenspectra basis. We're working on implementing the approach in Habib et al. 2007.

So I would say that PR #28 will close this issue.  There is now an example section in the Documentation.  This is very similar to the cookbook, but with a little bit more explanatory material.  With this new addition and the detailed paper Appendix I think the docs are good enough, unless someone raises another issue.


Document raw query support for MongoDB

Update $queryRaw and $executeQuery docs to include MongoDB
Relevant ticket: https://github.com/prisma/prisma/issues/8270

Impacts following docs:
https://www.prisma.io/docs/concepts/components/prisma-client/raw-database-access#queryraw
?
?

This has now been done - see PR #2778


updated inventory in ansible.cfg to full filename.

Signed-off-by: Bryce<EMAIL_ADDRESS>Description
Please provide a description for what this PR is for.
Updated inventory parameter to full filename to fix the error below when running ansible-playbook:
[WARNING]: Unable to parse /home/pi/src/k8s-cluster-installation/ansible/inventory as an inventory source
[WARNING]: No inventory was parsed, only implicit localhost is available
[WARNING]: provided hosts list is empty, only localhost is available. Note that the implicit localhost does not match
'all'

Checklist

[ ] All commits are signed off. See the section Developer’s Certificate of Origin
[ ] Provide a title or release-note blurb suitable for the release notes.
[ ] All commits contain a well written commit description including a title, description and a Fixes: #XXX line if the commit addresses a particular GitHub issue.
[ ] All workflow validation and compliance checks are passing.

Issue Ref (Optional)
Which issue(s) this PR fixes (optional, using fixes #(, fixes #<issue_number>, ...) format, will close the issue(s) when the PR gets merged): Fixes #
Notes
Add special notes for your reviewer here.

Thanks for your contribution and helping fix this @BryceAshey


[W10M] MediaElement Hangs on full screen

Starting from Microsoft.UI.Xaml V2.2.X update every time I try to bring media element to the full screen view I get a black screen and phone will going into a hanged state. sometimes it reboots automatically sometimes I have to hold power key to soft reset.
I tried this on W10M build 15254 on Microsoft Lumia 950XL DS RM-1116
A very simple MediaElement even run in this issue

    <Grid>
        <MediaElement x:Name="MediaVideo" AreTransportControlsEnabled="True" AutoPlay="False"  Source="https://www.radiantmediaplayer.com/media/bbb-360p.mp4">
            <MediaElement.RenderTransform>
                <CompositeTransform/>
            </MediaElement.RenderTransform>
            <MediaElement.TransportControls>
                <MediaTransportControls IsPlaybackRateButtonVisible="True" IsPlaybackRateEnabled="True"
                                            IsStopEnabled="True" IsCompact="True" IsZoomButtonVisible="False" IsVolumeButtonVisible="False"/>
            </MediaElement.TransportControls>
        </MediaElement>
    </Grid>

Finally with helps from @chingucoding I find a temporary workaround for this issue.
The main issue is coming from the MediaTransportControl style overriding of MS.UI.Xaml
My workaround is like this hope can help other Devs.
<MediaElement.TransportControls>
                        <MediaTransportControls IsPlaybackRateButtonVisible="True" IsPlaybackRateEnabled="True"
                                            IsStopEnabled="True" IsCompact="True" IsZoomButtonVisible="False" IsVolumeButtonVisible="False">
                            <MediaTransportControls.Style>
                                <Style TargetType="MediaTransportControls"/>
                            </MediaTransportControls.Style>
                        </MediaTransportControls>
                    </MediaElement.TransportControls>

This will ignore the MS.UI.Xaml styles

We are not investing in MediaElement or Windows 10 Mobile.  For future media usage, I recommend MediaPlayerElement, as it is the replacement for MediaElement - though I don't know if it's available on mobile.

We are not investing in MediaElement or Windows 10 Mobile.  For future media usage, I recommend MediaPlayerElement, as it is the replacement for MediaElement - though I don't know if it's available on mobile.


{DRAFT-DO_NOT_REVIEW} [BANDWIDTH_THROTTLING] Quota Enforcer and QuotaSource implementations for CapacityUnit quotas.

The scope of this PR is to introduce the new QuotaEnforcer and QuotaSource for CapacityUnit quota, and make changes in other code so that the overall behavior remains unchanged.
Note that new QuotaManager changes are out of scope for this PR. The only changes made to QuotaManager is to enable the current QuotaManager to continue working unchanged. Changes in QuotaManager and other components will come in subsequent PRs.

Codecov Report

Merging #2029 (6c717d3) into master (68e91ba) will decrease coverage by 58.49%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             master    #2029       +/-   ##
=============================================
- Coverage     73.58%   15.09%   -58.50%
+ Complexity    10001     2288     -7713
=============================================
  Files           715      711        -4
  Lines         54714    54910      +196
  Branches       6785     6815       +30
=============================================
- Hits          40263     8287    -31976
- Misses        12303    46140    +33837
+ Partials       2148      483     -1665

Impacted Files
Coverage Δ

...main/java/com/github/ambry/config/QuotaConfig.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../com/github/ambry/config/VerifiableProperties.java
42.46% <0.00%> (-30.00%)
:arrow_down:

...ain/java/com/github/ambry/quota/QuotaResource.java
0.00% <0.00%> (-83.34%)
:arrow_down:

...c/main/java/com/github/ambry/quota/QuotaUtils.java
0.00% <0.00%> (-21.06%)
:arrow_down:

...java/com/github/ambry/quota/AmbryQuotaManager.java
0.00% <0.00%> (-78.36%)
:arrow_down:

...java/com/github/ambry/quota/MaxThrottlePolicy.java
0.00% <0.00%> (-96.78%)
:arrow_down:

...ambry/quota/capacityunit/AmbryCUQuotaEnforcer.java
0.00% <0.00%> (ø)

...uota/capacityunit/AmbryCUQuotaEnforcerFactory.java
0.00% <0.00%> (ø)

...ub/ambry/quota/capacityunit/JsonCUQuotaSource.java
0.00% <0.00%> (ø)

...y/quota/capacityunit/JsonCUQuotaSourceFactory.java
0.00% <0.00%> (ø)

... and 569 more

Continue to review full report at Codecov.

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

Closing the PR as refactored and smaller PRs will be released.


broken images

https://docs.ens.domains/dns-registrar-guide all of the images seem to be broken here.

Thanks, fixed.


BUG 2135830: csi: add setmetadata and clustername flag to omap

Description of your changes:
The omap generator sidecar is the one who sets the pvc name and clustername in the metadata if the pv is re-attached to a new pvc or if the clustername changes, for that reason we need to set these flags on the omap generator sidecar also.
Signed-off-by: Madhu Rajanna<EMAIL_ADDRESS>(cherry picked from commit ffdfeb5becb1181ab6c6401d117fa0ef478d2dbd)
Which issue is resolved by this Pull Request:
Resolves #
Checklist:

[ ] Commit Message Formatting: Commit titles and messages follow guidelines in the developer guide).
[ ] Skip Tests for Docs: If this is only a documentation change, add the label skip-ci on the PR.
[ ] Reviewed the developer guide on Submitting a Pull Request
[ ] Pending release notes updated with breaking and/or notable changes for the next minor release.
[ ] Documentation has been updated, if necessary.
[ ] Unit tests have been added, if necessary.
[ ] Integration tests have been added, if necessary.

/assing @travisn


ESP32 M5

is there a way to control the tape from an esp32 like the atom lite m5
Version of the custom_component

Configuration

Add your logs here.

Describe the bug
A clear and concise description of what the bug is.
Debug log

Add your logs here.

I've made several attempts to install an sp110 from an atom m5 but it always gives an error.
It was one of the things that was great, being able to control several tapes remotelyv


Issue No. 2127: User can no longer change offer reqest tab and only one tab will show up

What does this pull request aim to achieve? 💯
I have removed the secondary tab when the user goes to edit the post. If the post is a first time creation this update will have no effect on the form fields
Please be concise and link any related issue(s):
#2127

How do i fix this Terraform error?

How do i fix this Terraform error?

Hi @kamarenmccord , thanks for working on this, you may need to create PR starts with feature/... to trigger the review build, check our How to create PR wiki page for more details

Hi @kamarenmccord , thanks for working on this, you may need to create PR starts with feature/... to trigger the review build, check our How to create PR wiki page for more details

Hi @kamarenmccord , thanks for working on this, you may need to create PR starts with feature/... to trigger the review build, check our How to create PR wiki page for more details

sorry I misunderstood the branch name, your branch name has no problem, please ignore my previous comment 😂

Hi @kamarenmccord , thanks for working on this, you may need to create PR starts with feature/... to trigger the review build, check our How to create PR wiki page for more details

sorry I misunderstood the branch name, your branch name has no problem, please ignore my previous comment 😂


Use erlef/setup-beam and prettify yaml file

List of changes:

Remove the extra redirection. See https://github.com/erlef/setup-beam/issues/20
Prettified using prettier -w .

💜

:partying_face: :partying_face: :partying_face: :partying_face: :partying_face:


Document more details on client apps support for Dynamics 365

Section - Require approved client app
List of client apps that support this setting.
It is listed "Microsoft Dynamics 365" , but it is not clear which one is it? Customer Service? Business Central ? etc.
Please be more specific about which exactly mobile application from Dynamics 365 portfolio supports, and which do not support.
I guess, that Dynamics 365 Business Central is not supported, but Dynamics 365 Customer Service is supported.

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

ID: bd934939-4dbb-3a92-f996-56b9557f49ac
Version Independent ID: 3cda6b81-26e8-c9ac-e768-d912636a461b
Content: Grant controls in Conditional Access policy - Microsoft Entra ID
Content Source: docs/identity/conditional-access/concept-conditional-access-grant.md
Service: active-directory
Sub-service: conditional-access
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

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

@hkusulja
The Dynamics 365 mobile apps, including the Dynamics 365 Customer Service app and the Dynamics 365 Business Central app, do support the "Require approved client app" setting in Conditional Access policies. The list of client apps that support this setting can be found in the following Microsoft documentation: https://docs.microsoft.com/en-us/azure/active-directory/conditional-access/concept-conditional-access-conditions#client-apps. I hope this clears up any confusion. Let me know if you have any other questions.

@hkusulja
We are going to close this thread as resolved but if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.

@SaibabaBalapur-MSFT the Dynamics 365 Business Central app is not supported.
Please write exact names of the apps in a list.

@hkusulja
Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can take a look at it accordingly.

@MicrosoftGuyJFlo
Can you please check and add your comments on this doc update request as applicable.

This would need to be asked of the Dynamics application team. As noted in the doc approved client app is being retired in March 2026.
For app protection policies we support Microsoft Dynamics 365 for Phones and Microsoft Dynamics 365 Sales.
#please-close


Update package version.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Outdated.


bring back the save accounts to file code commented out in the webcomponents PR

we're saving to a cfg file, but we really should also export to the solana cli format, and ammanrc


Vim mode

It will be very nice to enable a vim mode, in fact, if you use a vim mode extension for the browser and visit notion.so you got it.
I think the implementation should be very similar to browser extensions, with classics vim keys, modes and "sneak mode" ,'f' key in Vimium included.

I can't make this. I've never used Vim, so honestly it just doesn't make any sense to me (especially since most of the shortcuts appear to be just normal characters - does not that e.g. interfere with typing?).
If you or someone else wants to try this, the developer documentation and contributor guidelines are there, go ahead.

I can't make this. I've never used Vim, so honestly it just doesn't make any sense to me (especially since most of the shortcuts appear to be just normal characters - does not that e.g. interfere with typing?).
If you or someone else wants to try this, the developer documentation and contributor guidelines are there, go ahead.

Vim uses different modes, Normal mode (shortcuts and navigation), insert (typing) or visual (selection) so there is no interference.
Yes seems little bit harder to implement, maybe I could try it when I have free time. Here I leave the request in any case.

As a vim user this would be my one feature request which I would be happy to work on if no one else is yet. I don't know if the "sneak mode" is necessary as notion has built in functionality to change pages with the keyboard which could either be remapped for vim users or just kept as is. Anyway if no one else is working on it I can.


WIP: Doc update

Description

Updated Notices and Disclaimers based on new Legal guidelines
Converted details and description of batch processing for Moments of Low order
Added a link to DAAL API

Testing updates in CI with this PR as well.

/azp run docbuild

/intelci: run


DFU in SuperMini

How do you enter DFU in SuperMini? shorting gnd to rst once or twice is not doing it. I read a drive in windows is supposed to pop up. Thanks.

How do you enter DFU in SuperMini? shorting gnd to rst once or twice is not doing it. I read a drive in windows is supposed to pop up. Thanks.

have a look at https://github.com/joric/nrfmicro/wiki/Bootloader

So i guess I have bad board. Blue led is blinking twice as fast as the red one. No constant light, even after shorting gnd to rst. Can connect to the board via BT from nRF Connect on iOS though.
I might need to try RaspberryPI route. Will it work or are the boards trash?

Was able to revive one board with RPi.
The only question I have is which bootloader should I really use?
Supermini nrf52840 says to be nice!nano clone, has the same pins on pcb. Logic says I should use nince!nano bootloader from adafruit (which worked in my case and allowed me to flash Meshtastic) but maybe it doesn't matter and I can use generic pca10056 or nfrmicro?
Sorry for dumb question but is bootloader, apart from everything else, responsible also for which pins are exposed? I understand Meshtastic has different variants of boards and now I wonder if good working order depends on bootloader/firmware combo or just pins assigned in the meshtastic firmware.
Thanks.

Defective board. Double reset should work.


No Properties?

Bug Report
Issue: No Properties
Reproducing: Basically, I do not see anything about data char except configuration, I do not have Axis'es etc. (Scott.Plot does not work because I use winforms)
// a code sample may improve communication
var formsPlot1 = new FormsPlot();
            formsPlot1.MajorGrid(Color.FromArgb(100, Color.Black));
            formsPlot1.XAxis.MinorGrid(true, Color.FromArgb(20, Color.Black));
            formsPlot1.YAxis.MajorGrid(2,LineStyle.Dash, Color.Magenta);

System Details

ScottPlot Version: 4.1.52?
Operating System: Windows 8.1
Application Type: Winforms
.NET Version: NET Framework 4.6.1

Hi @dx7v2, try formsPlot1.Plot.MajorGrid,  formsPlot1.Plot.XAxis, etc...
Let us know if that doesn't do the trick! Good luck 🚀

It says there is no MajorGrid function in Plot property.
            this.formsPlot1.Plot.MajorGrid(Color.FromArgb(100, Color.Black));

It says there is no MajorGrid function in Plot property.

Thanks for the correction!
formsPlot1.Plot.XAxis.MajorGrid(Color.FromArgb(100, Color.Black));

more examples in https://scottplot.net/cookbook/4.1/category/advanced-axis-features/#log-scale-tick-density

This happens when I wrote the code you show.


Fix scrollbar dragging on touch devices

Tested that it works on iOS. Sorry about the unrelated styling changes my editor added!

Hi Kevin! Nice to see you again. Thanks for this, gonna do some quick testing on my end.


fix calculation error for new orders on shopware 5.7.x

This Pullrequest provides a probably dirty fix for calculation errors that prevent shpware 5.7.x from saving an order.
There are several cases in the issuetracker that will work again with this fix.
https://issues.shopware.com/issues/SW-25905
https://issues.shopware.com/issues/SW-26125
https://issues.shopware.com/issues/SW-26134

hey @christiankohrn
thanks for bringing this up, my colleague is already working on https://jira.shopware.com/browse/SW-26134
so this will be fixed with the next Shopware 5.7 version
Best regards from Schöppingen
Michael Telgmann

Thanks, is there any way for people to see if you are working on something like this? Your jira system seems to be an internal tool.
We could not find any information and we just spent four days analysing this issue for nothing... :(
And this is not the first time..

sorry, I linked the issue wrong!
It is the third issue, which you initially posted. Have a look at the status to see where it stands right now.
A special problem here was, that the issue occurred in the plugin, but needs a fix in the Shopware core


Updating the charm icon

The design team at Canonical would like to update charm icons to fit within the new circular format, as displayed in the current GUI.
This will improve how charms are displayed in the store, search results and on charm details pages.
A new version of the charm icon is attached.
Please could you send an email to  when this charm is next updated.
Thanks

(by karlwaghorn-moyce)

(by paulgear)
Merged into master; will push a build to cs:~nrpe-charmers/nrpe candidate channel shortly.

(by paulgear)
Released to candidate channel as https://jujucharms.com/u/nrpe-charmers/nrpe/13 - new icon displays correctly.


Find out the max number of entries that is possible to put into the advidce columns given a certain K

The expected maximum number of usable columns for the univariate grand sum circuit is 2^K-6.
https://github.com/summa-dev/halo2/blob/main/halo2_proofs/src/plonk/circuit.rs#L2349
https://github.com/summa-dev/halo2/blob/main/halo2_proofs/src/plonk/prover.rs#L310
However, the observed number is much lower.

The current maximum number of users that can be accommodated is $2^K - 2^{16} - 6$ in the advice column, given K. The deduction of 6 rows for blind factors, and $2^{16}$ rows represents the number of values occupied in the fixed column. These are constant values ranging from 0 to $2^{16}$, utilized for the range check chip. This configuration leads to an issue where the UnivariateGrandSum circuit cannot be fully populated with $2^K - 6$ entries in the advice columns, despite there being $2^{16}$ empty rows in these columns.
The table structure is illustrated as follows:

Number of Row
Username
Currency_1
Currency_2
...
range_u16

1
dxGaEAii
11888
41163

0

...

65536
MBlfbBGI
67823
18651

65535

65537
lAhWlEWZ
18651
2087

0

65538
nuZweYtO
22073
55683

0

...

...

$2^K - 2^{16} - 6$
RZNneNuP
31699
35479

0

$2^K - 2^{16} - 5$

0

...

...

2^K

...
0

The root cause of this issue stems from the constraint_constant method, which is used to verify the zero value in the range check chip. This method performs two primary operations: it assigns a constant value to the fixed column (denoted as range_u16 in the table) and then mapping this value(internally called, permutation.copy) to the advice column. Essentially, the function assigns values to the next available row after the last used row each time it is called, meaning it populates the next empty row in the fixed column.
At row $2^K - 2^{16} - 5$, the range check chip cannot assign 0 values to range_u16 anymore, as the column is already filled with other values.
Below is the actual trace log for reference:
1834972 DEBUG synthesize:region{name="load range check table of 16 bits"}:assign_fixed{name="assign cell in fixed column" column=Column { index: 0, column_type: Fixed } offset=65535}: layouter: Entered
1834973 DEBUG synthesize:region{name="load range check table of 16 bits"}:assign_fixed{name="assign cell in fixed column" column=Column { index: 0, column_type: Fixed } offset=65535}: position: row=65535
1834974 DEBUG synthesize:region{name="load range check table of 16 bits"}:assign_fixed{name="assign cell in fixed column" column=Column { index: 0, column_type: Fixed } offset=65535}: assigned: value=Trivial(0x0
1834974<PHONE_NUMBER>0000000000000000000000000000000000000000000000000ffff)
1834975 DEBUG synthesize:region{name="Perform range check on balance 0 of user 0"}:assign_advice{name="zs_0" column=Column { index: 2, column_type: Advice } offset=0}: layouter: Entered
1834976 DEBUG synthesize:region{name="Perform range check on balance 0 of user 0"}:assign_advice{name="zs_1" column=Column { index: 3, column_type: Advice } offset=0}: layouter: Entered
1834977 DEBUG synthesize:region{name="Perform range check on balance 0 of user 0"}:assign_advice{name="zs_2" column=Column { index: 4, column_type: Advice } offset=0}: layouter: Entered
1834978 DEBUG synthesize:region{name="Perform range check on balance 0 of user 0"}:assign_advice{name="zs_3" column=Column { index: 5, column_type: Advice } offset=0}: layouter: Entered
1834979 DEBUG synthesize:region{name="Perform range check on balance 0 of user 0"}: constrain_constant: cell=Cell { region_index: RegionIndex(2), row_offset: 0, column: Column { index: 5, column_type: Advice } }1834979  constant=Trivial(0x0000000000000000000000000000000000000000000000000000000000000000)
1834980 DEBUG synthesize:region{name="Perform range check on balance 0 of user 0"}:assign_advice{name="zs_0" column=Column { index: 2, column_type: Advice } offset=0}: layouter: Entered
1834981 DEBUG synthesize:region{name="Perform range check on balance 0 of user 0"}:assign_advice{name="zs_0" column=Column { index: 2, column_type: Advice } offset=0}: position: row=0
1834982 DEBUG synthesize:region{name="Perform range check on balance 0 of user 0"}:assign_advice{name="zs_0" column=Column { index: 2, column_type: Advice } offset=0}: assigned: value=Trivial(0x000000000000000001834982<PHONE_NUMBER>0000000000000000000000000000000000000)
1834983 DEBUG synthesize:region{name="Perform range check on balance 0 of user 0"}:assign_advice{name="zs_1" column=Column { index: 3, column_type: Advice } offset=0}: layouter: Entered
1834984 DEBUG synthesize:region{name="Perform range check on balance 0 of user 0"}:assign_advice{name="zs_1" column=Column { index: 3, column_type: Advice } offset=0}: position: row=0
1834985 DEBUG synthesize:region{name="Perform range check on balance 0 of user 0"}:assign_advice{name="zs_1" column=Column { index: 3, column_type: Advice } offset=0}: assigned: value=Trivial(0x000000000000000001834985<PHONE_NUMBER>0000000000000000000000000000000000000)
1834986 DEBUG synthesize:region{name="Perform range check on balance 0 of user 0"}:assign_advice{name="zs_2" column=Column { index: 4, column_type: Advice } offset=0}: layouter: Entered
1834987 DEBUG synthesize:region{name="Perform range check on balance 0 of user 0"}:assign_advice{name="zs_2" column=Column { index: 4, column_type: Advice } offset=0}: position: row=0
1834988 DEBUG synthesize:region{name="Perform range check on balance 0 of user 0"}:assign_advice{name="zs_2" column=Column { index: 4, column_type: Advice } offset=0}: assigned: value=Trivial(0x000000000000000001834988<PHONE_NUMBER>0000000000000000000000000000000000000)
1834989 DEBUG synthesize:region{name="Perform range check on balance 0 of user 0"}:assign_advice{name="zs_3" column=Column { index: 5, column_type: Advice } offset=0}: layouter: Entered
1834990 DEBUG synthesize:region{name="Perform range check on balance 0 of user 0"}:assign_advice{name="zs_3" column=Column { index: 5, column_type: Advice } offset=0}: position: row=0
1834991 DEBUG synthesize:region{name="Perform range check on balance 0 of user 0"}:assign_advice{name="zs_3" column=Column { index: 5, column_type: Advice } offset=0}: assigned: value=Trivial(0x000000000000000001834991<PHONE_NUMBER>0000000000000000000000000000000000000)
1834992 DEBUG synthesize:region{name="Perform range check on balance 0 of user 0"}: constrain_constant: cell=Cell { region_index: RegionIndex(2), row_offset: 0, column: Column { index: 5, column_type: Advice } }1834992  constant=Trivial(0x0000000000000000000000000000000000000000000000000000000000000000)
1834993 DEBUG synthesize:region{name="Perform range check on balance 0 of user 0"}: assign_fixed: name="Constant(0x0000000000000000000000000000000000000000000000000000000000000000)" column=Column { index: 0, col1834993 umn_type: Fixed } row=65536
1834994 DEBUG synthesize:region{name="Perform range check on balance 0 of user 0"}:positioned: copy: left_column=Column { index: 0, column_type: Fixed } left_row=65536 right_column=Column { index: 5, column_type1834994 : Advice } right_row=0

To resolve this issue, we must consider assigning the zero value used in the range check chip to a more suitable location, potentially within the advice column, rather than the fixed column.

$2^K-6$. Fixed in #276.


Add "allowImplicitAnyImports" compiler option for usage with "noImplicitAny"

TypeScript Version:  2.3.2
Code
import _ = require('lodash')
// with 'noImplicitAny' compiler option and without '@types/lodash' package dependency

Desired behavior:
The compiler emits no error.
Actual behavior:
The compiler emits this error:
Could not find a declaration file for module 'lodash'. '/project/node_modules/lodash/lodash.js' implicitly has an 'any' type.

Motivation:

I want to use lodash (or any non-TS package) without types.
I want to have noImplicitAny turned on for my own code.
I want to not depend on @types packages, which for many JS packages do not exist. (And unfortunately some of the ones that do exist are poorly maintained.)
I want to not add .d.ts files for every non-typed third-party JS package I need to use.
I want to use TS's import-require (not var-require).

#6615 should help.

More docs available at http://www.typescriptlang.org/docs/handbook/modules.html#shorthand-ambient-modules

Thanks. From those resources, the recommendation seems to be "Add one file to your project that contains a declare module 'xyz' line for each module xyz you want to import as any."
This is not ideal, but it works. I'd rather not have the extra file, and I'd especially rather not have to add a line to that file for each any module I need, but it's not as invasive as I feared from my first readings of related issue threads and the above-linked resources over the past few days.
Still, it would be much better to have the compiler option. But if it's already been reviewed and decided against by the team, that's fair.

There's also declare module "*"; that will make unknown modules any by default.

Thanks, I missed that. I thought it was going to make all modules any.
However, with declare module '*' I see that I lose module resolution's ability to provide an error when it can't find the module. This moves a very convenient compile-time check to a potentially confusing set of one or more runtime errors.
What's the rationale against this compiler option? That this issue was tagged "Question" instead of "Suggestion" when I was aiming for the latter implies one exists, I think.
From the discussions I've read in the issues, and from the current lack of a usage pattern that is both non-invasive and exactly equivalent, it seems like the rationale is that its benefit to the user doesn't outweigh the cost to develop it. But if that were true, this should be tagged "Suggestion" instead of "Question", since I'm assuming one measure of user benefit is how many users say they would benefit from it.
Would something in TS break or become inconsistent if this compiler option existed?

Could someone please remove the Question label and add the Suggestion label?
I'm asking because this issue was drafted as a suggestion, not as a question, and the suggested functionality is not fully provided by the (still very helpful) workarounds given by the team here and in other issues' discussions.
For other TS users out there who (a) have encountered this same deficiency in TS, (b) want this compiler option, and (c) are browsing and searching the issues, the labeling should indicate that this issue is a valid suggestion --- unless it isn't, but that hasn't been explained (or even asserted).
Thanks.

Marking as suggestion. but i do not think we will be doing this any time soon.
The compiler has a default behavior of treating all imports with no associated declaration files as any. this allows you to skip the definition file search and acquisition step. But it is rightfully flagged under --noImplicitAny.
Allowing the any's from module imports to propagate defeats the whole point of using --noImplicitAny. --noImplicitAny at its core gives you safe refactoring guarantees; this is built on the fact that the compiler knows the types of everything explicitly. allowing module imports to leak implicit anys into your program breaks that assumption.
The language also has affordances for declaring modules, so you can declare specific modules that you do not have declaration files for as declare module "jQuery"; or as @andy-ms mentioned earlier go all the way to declare module "*";. So you already have 1. treat it as any if it exists, or 2. trust me I know what i am doing, having yet another third state adds complexity, reduces safety and not sure i see it adding much value.

@mhegazy Thanks for replying (and re-labeling).

Allowing the any's from module imports to propagate defeats the whole point of using --noImplicitAny. --noImplicitAny at its core gives you safe refactoring guarantees; this is built on the fact that the compiler knows the types of everything explicitly. allowing module imports to leak implicit anys into your program breaks that assumption.

This isn't in doubt at all. I'm with you 100% on this.
However, when you use an externally packaged module that is simply not actually (or correctly) typed, you have no choice but to deal with any and its leakages. However, that fact should not prevent you from using the benefits of noImplicitAny in the rest of your code.
The good news is that we have declare module, as you and @andy-ms have already explained (which I appreciate immensely).
The bad news is

declare module '*' causes TS to not flag missing modules as errors, and
declare module 'xyz' for each module xyz is verbose.

I feel like we're talking past each other and that this really comes down to a disagreement over whether "bad news 2" is actually bothersome.

However, when you use an externally packaged module that is simply not actually (or correctly) typed, you have no choice but to deal with any and its leakages.

this seems like one or n modules, and hopefully not all modules you deal with all the time ; and for that adding a declare module "foo"; seems like an appropriate fix untill you have the time and desire to write foo.d.ts.

However, that fact should not prevent you from using the benefits of noImplicitAny in the rest of your code.

what i am arguing here is that you are not using the benefits of noImplcitAny here. the flag's value is not that most of your code has types, it is that all your code has types.

declare module 'xyz' for each any-typed module xyz is verbose.

and this is the intention. the idea is you made a conscious decision to treat some modules as any, and you are aware of the implications.

the flag's value is not that most of your code has types, it is that all your code has types.

I think I'll have to defer to your experience on this one, because I can only offer the single data point of my own project as counter-evidence. My colleagues and I have definitely benefited from noImplicitAny even though we use a few any-typed modules. The leakage of any has been easy to isolate where it occurs.
 From my own viewpoint though, they leave a small gap where there's actually a valuable user story. I have to admit I can't argue that the value is non-negligible at this point, though.
Thank you both for the help and insight.

the flag's value is not that most of your code has types, it is that all your code has types.

I think that's too strong of a statement, the flag is just about preventing implicit anys.
What seems strange to me is that variables and consts can be marked with any easily while imported symbols need this quite unintuitive solution with a new .d.ts file and declare module '...' for each untyped package in it.
For reference, import foo: any from "module"; has been proposed in https://github.com/microsoft/typescript/issues/3019#issuecomment-98851781. I'm not saying this specific syntax is great but it's conceptually much closer to what I'd expect. Also the allowImplicitAnyImports compiler option proposed here makes sense to me.
For me, it comes down to usability: in the world of various npm modules, untyped imports are quite common and it's currently not intuitive how to deal with it easily. I've seen people just doing require() instead of import which is a strange workaround and maybe suggests that there is some slight usability problem in TypeScript in this area.


CREATE TABLE IF NOT EXISTS should not throw an exception when the table exists.

"IF NOT EXISTS" implies that nothing should be done if the table already exists.
Currently an exception is thrown:
Caused by: ru.yandex.clickhouse.except.ClickHouseException: ClickHouse exception, code: 253, host: xxx.xx.xx.xx, port: xxxx; Code: 253, e.displayText() = DB::Exception: There was an error on ip-xxx-xx-xx-xx.ec2.internal: Code: 253, e.displayText() = DB::Exception: Replica /clickhouse/tables/13/table_name/replicas/hostname-us-east-1 already exists., e.what() = DB::Exception, e.what() = DB::Exception

Hello,
This is the server who throws the exception, while the driver just
translates it into Java exception.
As I see in the error message the thing is not in table itself, but in
replica.
I guess you are trying to create a replicated table with zookeeper path
that is already used in another table.
вт, 21 авг. 2018 г., 17:29 Hunter Sherman<EMAIL_ADDRESS>
"IF NOT EXISTS" implies that nothing should be done if the table already
exists.
Currently an exception is thrown:
Caused by: ru.yandex.clickhouse.except.ClickHouseException: ClickHouse
exception, code: 253, host: xxx.xx.xx.xx, port: xxxx; Code: 253,
e.displayText() = DB::Exception: There was an error on
ip-xxx-xx-xx-xx.ec2.internal: Code: 253, e.displayText() = DB::Exception:
Replica /clickhouse/tables/13/table_name/replicas/hostname-us-east-1
already exists., e.what() = DB::Exception, e.what() = DB::Exception
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/yandex/clickhouse-jdbc/issues/226, or mute the thread
https://github.com/notifications/unsubscribe-auth/AK_lWtHBn1wJfjKBFab_RP-F760r4bHpks5uTBlTgaJpZM4WF5y9
.

I guess you are trying to create a replicated table with zookeeper path
that is already used in another table.

This is because the table already exists.


[Event Request] Codeunit 87 Blanket Sales Order to Order - OnRun

We would like to request the following publisher so we can have our own calculation if contract is exceeding, and also add possibility for user to ignore if the contract is exceeding. There is a simular request for purchase #6256.
                if SalesLine.FindSet then
                    repeat
                        if (SalesLine."Document Type" = SalesLine."Document Type"::"Return Order") or
                           ((SalesLine."Document Type" = SalesLine."Document Type"::"Credit Memo") and
                            (SalesLine."Return Receipt No." = ''))
                        then
                            QuantityOnOrders := QuantityOnOrders - SalesLine."Outstanding Qty. (Base)"
                        else
                            if (SalesLine."Document Type" = SalesLine."Document Type"::Order) or
                               ((SalesLine."Document Type" = SalesLine."Document Type"::Invoice) and
                                (SalesLine."Shipment No." = ''))
                            then
                                QuantityOnOrders := QuantityOnOrders + SalesLine."Outstanding Qty. (Base)";
                    until SalesLine.Next = 0;
                /// START >>
                SkipError := FALSE;
                OnSkipDoContractExceedingCheck(BlanketOrderSalesLine, QuantityOnOrders, VAR SkipError);
                IF NOT SkipError THEN
                /// END <<
                  if (Abs(BlanketOrderSalesLine."Qty. to Ship (Base)" + QuantityOnOrders +
                        BlanketOrderSalesLine."Qty. Shipped (Base)") >
                      Abs(BlanketOrderSalesLine."Quantity (Base)")) or
                     (BlanketOrderSalesLine."Quantity (Base)" * BlanketOrderSalesLine."Outstanding Qty. (Base)" < 0)
                  then
                      Error(
                        QuantityCheckErr,
                        BlanketOrderSalesLine.FieldCaption("Qty. to Ship (Base)"),
                        BlanketOrderSalesLine.Type, BlanketOrderSalesLine."No.",
                        BlanketOrderSalesLine.FieldCaption("Line No."), BlanketOrderSalesLine."Line No.",
                        BlanketOrderSalesLine."Outstanding Qty. (Base)" - QuantityOnOrders,
                        StrSubstNo(
                          Text001,
                          BlanketOrderSalesLine.FieldCaption("Outstanding Qty. (Base)"),
                          BlanketOrderSalesLine.FieldCaption("Qty. to Ship (Base)")),
                        BlanketOrderSalesLine."Outstanding Qty. (Base)", QuantityOnOrders);
                SalesOrderLine := BlanketOrderSalesLine;
                ResetQuantityFields(SalesOrderLine);
                SalesOrderLine."Document Type" := SalesOrderHeader."Document Type";
                SalesOrderLine."Document No." := SalesOrderHeader."No.";
                SalesOrderLine."Blanket Order No." := "No.";
                SalesOrderLine."Blanket Order Line No." := BlanketOrderSalesLine."Line No.";

Many thanks!

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Opens the websocket connection before closing it

This is a workaround for #4009
Gunicorn only fails, when the websocket connection is rejected before opening it. So this helps.
@FinnStutzenstein If we merge this the client maybe has to be changed. This is only relevant for the anonymous user when anonymous user is deactivated. Before this PR, the connection was not opened. Now, it is opened but closed immediately.
@jsaalfeld could you test, if the big mode works again with this workaround?

@jsaalfeld ping


Removed flatten from test

Codecov Report

Merging #1109 (5a3a5a2) into main (ecffdff) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1109   +/-   ##
=======================================
  Coverage   57.72%   57.72%
=======================================
  Files          73       73
  Lines        2316     2316
  Branches       72       72
=======================================
  Hits         1337     1337
  Misses        979      979

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 ecffdff...5a3a5a2. Read the comment docs.


no luck

On a rock64 board (arm), when I do this:
curl -sSf https://moncho.github.io/dry/dryup.sh | sudo sh
it starts downloading and then I get this:
~ $ curl -sSf https://moncho.github.io/dry/dryup.sh | sudo sh   % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current                                  Dload  Upload   Total   Spent    Left  Speed 100    10  100    10    0     0     19      0 --:--:-- --:--:-- --:--:--    19 aar sh: 233: CFG_CPUTYPE: parameter not set

ok - found out that I can just download the bin file, do:
sudo chmod a+x <FILE_NAME>
then:
sudo ./<FILE_NAME>

Hi @CaptainSkywave,
can you run uname -m and copy the ouput here, please?
Cheers

aarch64

Thanks!!
Issue #98 has been created to support Arm64.

Thanks.

Just FYI... this change doesn't seem to have made it into https://moncho.github.io/dry/dryup.sh yet.  Not sure if that's on purpose for now.
Using https://raw.githubusercontent.com/moncho/dry/development/dryup.sh does work fine for aarch64 though.
Thanks!

Thanks for reporting @Codelica, I had forgotten to update the gh-pages branch, which has its own copy of the script. It should work now.


Tycho compatibility with appbundle ?

Hi,
This is more a question than an issue,
do you think your plugin is compatible with tycho project  ?
Best
SR

I'm not familiar with Tycho. Sorry.
I have no plan to make my plugin compatible with eclipse plugin.


[ signup ][ login ] Password Visibility Toggle Eye Icon Behaving Opposite

The eye icon used to toggle the visibility of the password field is behaving in the opposite manner than expected. When the icon is clicked, the password visibility state is not changing correctly.

please assign it to me

Hi @shivamgaur99  assign me this task under GSSOC 2024 . I am confident that I can fix this bug.

@shivamgaur99 i dont think so there is a problem.. it is working fine ..will u please elaborate the issue

@Muskansahuincredible when you enter any password eye toggle should off like in this image

Please assign me this issue under GSSOC'24 @Muskansahuincredible

I am interested in the issue, can you please assign me.

@Muskansahuincredible  assign me this issue i will fix it tommorow only.


Is this plugin right for me?

Hi, I've been in search of a replacement of sonarr-episode-trimmer ever since upgrading Sonarr to v4.
Basically, it works on a by-series basis, where I just say what TV series I want to have a maximum number of episodes, regardless of watched state.
Can Media Cleaner behave the same way for me? It looks like this plugin applies the same criteria to all episodes or movies
Thanks!

Hi,
Unfortunately, it won't meet your requirements, as the rules are applied
globally and can't be customized for individual shows. I don't anticipate
this changing anytime soon, sorry.
сб, 19 окт. 2024 г., 10:20 thereddestdog @.***>:

Hi, I've been in search of a replacement of sonarr-episode-trimmer
https://gitlab.com/spoatacus/sonarr-episode-trimmer ever since
upgrading Sonarr to v4.
Basically, it works on a by-series basis, where I just say what TV series
I want to have a maximum number of episodes, regardless of watched state.
Can Media Clear behave the same way for me? It looks like this plugin
applies the same criteria to all episodes or movies
Thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/shemanaev/jellyfin-plugin-media-cleaner/issues/59,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAICN2MVXIZHLOKV3O5Q7E3Z4IB2HAVCNFSM6AAAAABQHHBIOOVHI2DSMVQWIX3LMV43ASLTON2WKOZSGU4TQOBUGYZTKMI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

OK thanks for your reply


Fill in the middle

Hi!
Do you support fill in the middle technique in pretrain pipelines? If yes, do you have some documentation about this?
Thanks!

No, we don't have support FIM at this time.

Going ahead and closing this. You can of course construct FIM style data yourself and use LLM Foundry to train on that data.


Cmake can’t find TensorRT

Hello,
Thank you for sharing this repository. After installing the requirements, I cloned it and built it . However, I received this error:
Could NOT find TensorRT (missing: TensorRT_LIBRARY) (found version "..")
Call Stack (most recent call first):
/home/safebot/.local/lib/python3.8/site-packages/cmake/data/share/cmake-3.27/Modules/FindPackageHandleStandardArgs.cmake:600 (_FPHSA_FAILURE_MESSAGE)
cmake/FindTensorRT.cmake:73 (FIND_PACKAGE_HANDLE_STANDARD_ARGS)
libs/tensorrt-cpp-api/CMakeLists.txt:22 (find_package)
Which is strange considering I used Debian to install TensorRT and the library gets installed by default on
set(TensorRT_DIR /usr/lib/x86_64-linux-gnu/)
And I checked that as well.  I also used cmake gui to build it and I got this error after specifying /usr/lib/x86_64-linux-gnu as library directory :TensorRT_NVONNXPARSER_LIBRARY-NOTFOUND
I would appreciate it if you could help me identify the source of this issue when building the project.
Kind regards
Nima

Issue solved. Thank you

Can you please explain how you solved the issue so that if others have the same issue they know what to do.

Yes sure. Basically, the problem is the Debian installation of tensor RT. The problem will be solved simply by installing with the tar file of tensor RT not using Debian installation. Cmake can’t find tensorRT in Debian default path .


Angular standalone output still generates NgModule

I am interested in helping provide a fix!
No
Which generators are impacted?

[ ] All
[X] Angular
[ ] HTML
[ ] Qwik
[ ] React
[ ] React-Native
[ ] Solid
[ ] Stencil
[ ] Svelte
[ ] Vue
[ ] Web components

Reproduction case
N/A
Expected Behaviour
The standalone option for the Angular output target should prevent an NgModule from being generated.
Actual Behaviour
When the standalone option is provided to the Angular output target, the Angular code still generates an NgModule.
As per the Angular docs:

Components, directives, and pipes can now be marked as standalone: true. Angular classes marked as standalone do not need to be declared in an NgModule (the Angular compiler will report an error if you try).

Additional Information
No response

Thanks for filing. We want to only run the following line if options.standalone !== true: https://github.com/BuilderIO/mitosis/blob/637ecbddf6d8082c1fbde34a8ba0c245a32874b7/packages/core/src/generators/angular.ts#L571

I am on it!!


[Bug] start-image-builder.js using deprecated AppStream image

In service-workbench-on-aws/scripts/appstream
The start-image-builder.js is now using a deprecated AppStream image 'AppStream-WinServer2019-10-08-2021' which is stopping the builder from running.
This should be replaced with the latest version 'AppStream-WinServer2019-07-12-2022'.

Hi @rikkibar , thank you for bringing this to our attention. We will test with the new image and put up a PR shortly.

Fix has been merged.


qemu: remove multidev in qemu/fsdev parameter on arm64

backport this from kata-containers.
As the current qemu of arm64 is so old, the new multidev parameter
in 9pfsdev is not supported on arm64, so disabled it temporarily.
Fixes:#2868
Signed-off-by: Jianyong Wu<EMAIL_ADDRESS>@devimc @jodh-intel

/test-ubuntu


Implement most of the Data.Array API

See https://github.com/funkia/list/pull/27
I've included the license notice from -arrays as the API and documentation are copied from it.
I still think it's a good idea to include some example usage, but I didn't want to copy those. It'd make sense to include algorithmic complexity as well for each function.

Is updateAtIndices and modifyAtIndices the only functions missing from the Data.Array API?

I still think it's a good idea to include some example usage, but I didn't want to copy those. It'd make sense to include algorithmic complexity as well for each function.

Definitely. I can go over the functions and add running time and adapt examples from the List documentation?

Is updateAtIndices and modifyAtIndices the only functions missing from the Data.Array API?

I believe so. I wanted to discuss whether you wanted to include them or not. They seem fairly specialized and wouldn't be useful for anything other than very large lists IMO.

Definitely. I can go over the functions and add running time and adapt examples from the List documentation?

Great!

I believe so. I wanted to discuss whether you wanted to include them or not. They seem fairly specialized and wouldn't be useful for anything other than very large lists IMO.

I agree. They seem like a small optimization mainly as they can utilize internal mutations. But, I think they're important to have just so we can say "Implements most of the Data.Array API".

I'll merge this 😄
I'll add updateAtIndices and modifyAtIndices when I get the time.
If you'd like to help maintain the package @gabejohnson let me know and I'll give you access to the repository.


943 Navigation Tab for Event Page

We added a new event_nav page to files and used it as a template for the event view pages.
Resolves #943

Updated the branch and fixed the crashing error. The event title now displays correctly.

We've removed the header and renamed the file to your liking. Thanks be to Andonson.

Ready for review.


Add support for non-public OIDC clients

Description
This PR adds support for non-public OIDC Clients.
Changes proposed in this pull request:

add optional clientSecret everywhere where clientId is mentioned
add clientSecret to tests
Updated documentation

Related issue(s)
Resolves #1261

I added the PR tag to frontend and backend deployment, not 100% sure if it is correct for the tests, let me know if i need to change it

/test all

/retest


SAK-43768 SAMIGO: Randomize question pools with option: A student's questions are randomized once for all submissions

https://sakaiproject.atlassian.net/browse/SAK-43768

Well done @PabloArrz ! Clean and neat.

@mpellicer I have improved the documentation and include the brackets.

@josecebe WDYT? Ready to go?

@josecebe WDYT? Ready to go?

It works nice, but we will test it deeply after it gets merged, I saw there are more seed references in the code, but maybe is not necessary to modify those seeds to keep it working properly.

Thanks for the feedback


Fix BUILD_ONLY build error caused by a punctuation mistake

#808

Two other options would be to

not append core to the end of each dependency in get_dependencies_for_sdk
remove core from the end of each dependency, and append it to the dependency list in get_dependencies_for_test

Thanks for catching this, but it has been fixed, will be published soon.
On Wed, Feb 21, 2018 at 9:21 PM jmsundar<EMAIL_ADDRESS>wrote:

Two other options would be to

not append core to the end of each dependency in
get_dependencies_for_sdk
remove core from the end of each dependency, and append it to the
dependency list in get_dependencies_for_test

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/aws/aws-sdk-cpp/pull/809#issuecomment-367570985, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADTsCHiIcAXwlEbn9WMGJSPw8Kb9jCA9ks5tXPlzgaJpZM4SOwOh
.

--
Best Regards!
Liang.
Software Engineer
Game Developer


while loop in StrictLineReader.java

      while (true) {         out.write(buf, pos, end - pos);         // Mark unterminated line in case fillBuf throws EOFException or IOException.         end = -1;         fillBuf();         // Try to find LF in the buffered data and return the line if successful.         for (int i = pos; i != end; ++i) {           if (buf[i] == LF) {             if (i != pos) {               out.write(buf, pos, i - pos);             }             pos = i + 1;             return out.toString();           }         }       }
I dont understand the while means what,If in is not changed in fillBuf(),it will be an endless loop,
I find it when I try to translate the java file to kt file for kotlin

thx


loraplus_lr_ratio: 1 doesn't match lora behavior

Please check that this issue hasn't been reported before.

[X] I searched previous Bug Reports didn't find any similar reports.

Expected Behavior
When setting loraplus_lr_ratio: 1 behavior should be similar to vanilla Lora, however this is not the case and loss values are significantly different.
Current behaviour
loraplus_lr_ratio: 1 doesn't match lora behavior.
Loss is higher, same behavior for different ratio values.
Steps to reproduce
Running training with and without loraplus_lr_ratio: 1
lora config used:
sample_packing: true

adapter: lora
loraplus_lr_ratio: 1
lora_r: 512
lora_alpha: 512
lora_dropout: 0.00
peft_use_rslora: false
lora_target_linear: true

gradient_accumulation_steps: 2
micro_batch_size: 4
num_epochs: 2
optimizer: adamw_torch
lr_scheduler: cosine
cosine_min_lr_ratio: 0.2
learning_rate: 0.00005
max_grad_norm: 1

Config yaml
No response
Possible solution
No response
Which Operating Systems are you using?

[X] Linux
[ ] macOS
[ ] Windows

Python Version
3.10
axolotl branch-commit
0.5.0
Acknowledgements

[X] My issue title is concise, descriptive, and in title casing.
[X] I have searched the existing issues to make sure this bug has not been reported yet.
[X] I am using the latest version of axolotl.
[X] I have provided enough information for the maintainers to reproduce and diagnose the issue.

Looks like it is related to deepspeed.
Adding this for stage 1 fix it for me. Closing.
"optimizer": {
        "type": "AdamW",
        "params": {
            "lr": "auto",
            "weight_decay": "auto",
            "torch_adam": true,
            "adam_w_mode": true
        }
    }


Build failed with an exception.

Hello, I updated to the latest version (7.0.4) and I started to give problems when starting the application, I tried to change to the previous version and they kept giving me these problems...
Here is the screenshot of the error:

I'm having the same issue

@elaishane
please try
flutter clean
flutter update

for your log i can see its point to two different versions of the same plugin

Whenever I get this pub cache issue. I generally try to clear the cache by doing flutter pub cache repair

@elaishane
please try
flutter clean
flutter update

for your log i can see its point to two different versions of the same plugin

I think you replied to the wrong person. He is @esthefany23777 and you have replied to @elaishane i.e. me

I'm having the same issue

Hello, it's already solved, I just put
 flutter clean
and ready...


chore(main): release 0.4.2

:robot: I have created a release beep boop
0.4.2 (2024-08-12)
Bug Fixes

improve assertion message for toBe assertion (bdc520e)
improve assertion message for toBeABoolean assertion (7045117)
improve assertion message for toBeAFloat assertion (0b45434)
improve assertion message for toBeAnArray assertion (3ab4a15)
improve assertion message for toBeAnInstanceOf assertion (c18e754)
improve assertion message for toBeAnInteger assertion (da88321)
improve assertion message for toBeAString assertion (b00a0a2)
improve assertion message for toBeEmpty assertion (9a3926c)
improve assertion message for toBeFalse assertion (95461db)
improve assertion message for toBeGreaterThan assertion (9504538)
improve assertion message for toBeGreaterThanOrEqual assertion (9217d4a)
improve assertion message for toBeInBetween assertion (1e63613)
improve assertion message for toBeLessThan assertion (fec4d53)
improve assertion message for toBeLessThanOrEqual assertion (364162e)
improve assertion message for toBeNegative assertion (5ccd76f)
improve assertion message for toBeNull assertion (ff52e23)
improve assertion message for toBeTrue assertion (b057761)
improve assertion message for toContain assertion on arrays (7e978ac)
improve assertion message for toContain assertion on strings (a09d228)
improve assertion message for toContainAnyOf assertion (fce14c7)
improve assertion message for toEndWith assertion (b654d7d)
improve assertion message for toHaveALengthOf assertion message (51ac2e9)
improve assertion message for toStartWith assertion (c083dc0)
improve assertion message forBePositive assertion (db605b1)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v0.4.2
:sunflower: