Update vulnerabilities

Related command
N/A
Description
Fixes vulnerabilities in mcr.microsoft.com/azure-cli:2.49.0. Affected packages are:

cryptography 38.0.4 (pip)
requests 2.26.0 (pip)
openssl 3.1.0-r4 (apk)
binutils 2.40-r6 (apk)

Testing Guide
N/A
History Notes
N/A

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Thank you for your contribution! We will review the pull request and get back to you soon.

requests is bumped in https://github.com/Azure/azure-cli/pull/26571
openssl and binutils will be updated once there is new version in Alpine repo.
The reason of using old version cryptography is in https://github.com/Azure/azure-cli/pull/25690. We are working on Windows 64-bit MSI package in #26640, once it's finished, we'll bump it.


Support Incremental Annotation Processing in Gradle 4.7+

Support Incremental Annotation Processing in Gradle 4.7+
https://docs.gradle.org/nightly/userguide/java_plugin.html#sec:incremental_annotation_processing

Any update on this?

Anything new?

+1


Submaster slider

Hello!
I'm not shure, that it bug. Submaster slider follows by Intensity level of functions.
Steps to reproduce the behavior:

Add fixture, for example, par or strobe with intensity channel. HTP channel mode
Add sequence with intensity channel 255 level
Add frame with submaster slider and button with sequence function
Set any non-zero level on submaster
activate sequence with button
deactivate sequence with

I expect, that submaster slider will not change, but at deactivation sequence submaster slider sets to zero

OS: Linux Mint 21.1
QLC+ Version 4.12.7

It' bug...
Steps for repeat ar not correct. Right way:

Add fixture, for example, par or strobe with intensity channel. HTP channel mode
Add sequence with intensity channel 255 level
Add frame
Add slider, assign playback mode and sequence, click OK
Enter to slider properties, go to submaster tab, activate this mode, click ok
Add button with sequence function
Set any non-zero level on submaster
activate sequence with button
deactivate sequence with button

Add sequence with intensity channel 255 level
Sequence or Scene?

It doesn't matter.

Should be fixed now. Please check one of the automated builds when done thanks


ABI generation fails with lifetimes

Similar to #972.
#[near_bindgen]
impl MyType {
    pub fn slice<'a>(data: &'a String) -> &'a str { ... }
}

$ cargo build --features near-sdk/__abi-generate
   Compiling adder v0.1.0 (/Users/miraclx/git/near/near-sdk-rs/examples/adder)
error[E0261]: use of undeclared lifetime name `'a`
  --> src/lib.rs:3:17
   |
 3 |     pub fn slice<'a>(data: &'a String) -> &'a str {
   |                 -                          ^^ undeclared lifetime
   |                 |
   |                 help: consider introducing lifetime `'a` here: `<'a>`

For more information about this error, try `rustc --explain E0261`.```

Just a note that the existing lifetime support is limited to top-level lifetimes. For example this does not compile:
pub fn slice<'a>(data: Cow<'a, [u8; 32]>) -> Cow<'a, [u8; 32]> {
    data
}

I think lifetimes in argument position is pretty much invalid form. As part of the return type, which is more reasonable, it compiles just fine.

@miraclx

I think lifetimes in argument position is pretty much invalid form.

Can you elaborate on why you think it is invalid form? The foo example above seems reasonable to me.

Well, really, my thought was: in the context of the code-generated contract API, reference to data that was deserialized just for you doesn't seem very useful. But yeah, the scope extends larger than this. Enough to be generalized, so it could technically be called by another method in the contract that only has a reference to pass to it, so that makes sense. You can ignore the original comment.
But, in that case, we should make an effort to resolve the issue you highlighted in https://github.com/near/near-sdk-rs/issues/974#issuecomment-1330048298.
Two options I've experimented with locally use TokenStream sanitization as an alternative to #982, like in #1001;

The first approach strips all lifetime patterns from type signatures.

&'a Cow<'b, 'c, [u8; 32]> -> &Cow<[u8; 32]>

While the second approach replaces all lifetime patterms with the implicit lifetime.

&'a Cow<'b, 'c, [u8; 32]> -> &'_ Cow<'_, '_, [u8; 32]>

.. both of which are equivalent to the compiler, and are sufficient for ABI generation. I'll submit both PRs for evaluation in a sec.

But yeah, the scope extends larger than this. Enough to be generalized, so it could technically be called by another method in the contract that only has a reference to pass to it, so that makes sense. You can ignore the original comment.

Honestly, I take that back. I think deserialized inputs to methods of the exported contract API should be received with full ownership. And if you need a method that you can either call from inside the contract or outside of it; you can keep the main impl private and just export a proxy that consumes ownership and passes a reference to the main impl.
We're already doing this implicitly, and it works for the trivial cases, but reconciling a complete solution to this issue with that implementation would be difficult.
This, for example, is how the input deserializer looks from the exported contract API:
#[serde(Deserialize)]
struct Input {
    other: Cow<'a, [&'b str; 32]>,
}

some issues I can point out here are;

first, the lifetimes are undeclared so this is invalid
second, serde doesn't know how to reconcile lifetimes by default. So every field would need to be annotated with #[serde(borrow)]


Unable to freeze graph with Elmo Embedding layers with graph_util.convert_variables_to_constants

Problem
Tensorflow serving throws the below specified error when freezing a trained graph
This has been referenced and noticed that the error is thrown only when using the tensorflow hub character embedding Elmo. Works alright for Universal encoder.
Error on the server
'elmo/module_apply_tokens/bilm/embedding_lookup' expects to be colocated with unknown node 'elmo/module_apply_tokens/bilm/embedding_lookup/Read/ReadVariableOp'

Code to freeze the model,
saved_model_dir - Path that contains all the checkpoints
output_node_names - All the output_nodes from the graph + 'init_all_tables' (idea got from here)
def freeze_model(saved_model_dir, output_node_names, output_filename):
    # We retrieve our checkpoint fullpath
    checkpoint = tf.train.get_checkpoint_state(saved_model_dir)
    input_checkpoint = checkpoint.model_checkpoint_path
    print(input_checkpoint)

    # We precise the file fullname of our freezed graph
    absolute_model_dir = "/".join(input_checkpoint.split('/')[:-1])
    output_graph = absolute_model_dir + "/frozen_model.pb"

    # We clear devices to allow TensorFlow to control on which device it will load operations
    clear_devices = True

    # We start a session using a temporary fresh Graph
    with tf.Session(graph=tf.Graph()) as sess:
        # We import the meta graph in the current default Graph
        saver = tf.train.import_meta_graph(input_checkpoint + '.meta', clear_devices=clear_devices)

        # We restore the weights
        saver.restore(sess, input_checkpoint)

        gd = tf.get_default_graph().as_graph_def()

        for table_init_op in tf.get_collection(tf.GraphKeys.TABLE_INITIALIZERS):
            print(table_init_op.name)
            output_node_names.append(table_init_op.name)

        # We use a built-in TF helper to export variables to constants
        output_graph_def = tf.graph_util.convert_variables_to_constants(
            sess, # The session is used to retrieve the weights
            gd, # The graph_def is used to retrieve the nodes
            output_node_names # The output node names are used to select the usefull nodes
        )

        tf.train.export_meta_graph(
                filename=output_graph,
                graph_def=output_graph_def,
                collection_list=[tf.GraphKeys.TABLE_INITIALIZERS])

Code to load the frozen graph
def load_frozen_graph(export_dir, frozen_graph_filepath):
    tf.reset_default_graph()
    with tf.Graph().as_default() as graph:
        tf.train.import_meta_graph(frozen_graph_filepath)
        with tf.Session(graph=graph) as sess:
            try:
                sess.run(graph.get_operation_by_name('init_all_tables'))
            except KeyError:
                pass
            inputs = {node.name: sess.graph.get_tensor_by_name('{}:0'.format(node.name))
                               for node in sess.graph_def.node if node.op=='IteratorGetNext'}
            outputs = {'class_ids': sess.graph.get_tensor_by_name(
                                   'dnn/head/predictions/class_ids:0')}
            builder = tf.saved_model.builder.SavedModelBuilder(export_dir)
            signature = predict_signature_def(inputs=inputs, outputs=outputs)

            builder.add_meta_graph_and_variables(sess, [tf.saved_model.tag_constants.SERVING],
                                                 signature_def_map={'predict': signature},
                                                  main_op=tf.tables_initializer())
            builder.save()

This is the line that causes the error.
tf.train.import_meta_graph(frozen_graph_filepath)
The same code works well in case of using Universal sentence encoder.

@vbardiovskyg do you know of a way to prune / quantize a graph that uses elmo? Currently running into this same error as well.


Manifest file to store options

Is your feature request related to a problem? Please describe.
Currently, each push of a new app requires you to specify configuration options in the push command. It would be nice to offer a way to use something similar to the CF manifest.yml.
Describe the solution you'd like
I would like to have all of the configuration options stored in a file (likely manifest.yml or epinio.yml) that gets read when pushing an app.
Options specified on the command line would override options in the manifest.
The configuration options needed:

name
deployment template name
services to bind
environment variables
routes (including path and ingress class when needed)

We also need to include the ability to generate the manifest from a running application.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

@thardeck Please read. I believe my thoughts, especially in the code section are important to your work on #643 also.
Thinking .... (Always a good idea :wink:)

The configuration options needed:

name
deployment template name
services to bind
environment variables
routes (including path and ingress class when needed)

For the initial PR I would say only that name, services, and env variables are relevant.
The latter dependent on having #643 done. Templating and routes depend on things not on our schedule, IIRC. When they are added and implemented adding them to the manifest would be part of their acceptance criteria.

I would like to have all of the configuration options stored in a file (likely manifest.yml or epinio.yml) that gets read when pushing an app.

My prefered name is epinio.yml. Makes the association clear. Should also prevent mis-identification of CF, kubecf, etc. manifests by epinio as for its own use. Any support for such (and their different yaml spec) can be done as separate enhancements, if actually necessary.

Options specified on the command line would override options in the manifest.

Sensible.

Trivially done for app name. Although it makes the name argument of push optional.
Services (and EVs): Data are lists. The easy specification for override here is to go for total replacement. If there are any options for services (or EVs) the relevant part of the manifest is dropped entirely. No attempt at merging the two.

We also need to include the ability to generate the manifest from a running application.

Doable.

Name of application is always available, whether the app is active or at rest (i.e. no workload).
EVs are also always available Stored separately, and the app env commands can attach EVs to apps at rest.
Services ... Can be bound currently only to active apps. We do not support pre-binding of services to apps at rest.

The last point does not prevent generation of a manifest from an app. It only means that generating it when the app at rest will not be able to put service information into it.
Supporting pre-binding is out of scope for this ticket IMVHO. A separate ticket is needed.
Code
App Creation has an in-memory structure, see ApplicationCreateRequest (short: ACR). Could be extended to hold the services and EVs ...
Or not. Because of the limitation for services to be bound only to an active app. The binding currently happens during push as a separate binding API call ServiceBindingCreate, with all the service names.
For the manifest it would be better to support pre-binding, and then simply pass everything in through the AppCreation. The deploy step can then do the binding, and env vars server side, and the binding call can be dropped from push.
At that point it also makes sense to extend the app create command with support for --bind and the equivalent option for EVs @thardeck is working on for push. Note, I believe that his work should also extend the ACR to pass in the EVs. The server side already knows how to deal with pre-set EVs when the app is deployed.
Conclusions
I believe it might be best for me to put this back ito the TODO and work on supporting pre-binding of services to apps at rest,
plus changing push to move the binding operation to the server side when the app is deployed, like is done for EVs already.


Time profiler analysis based performance improvement

Based on time profiler analysis, a minor performance updgrade. SimpleMapTest performance changes from around 28000 to 33000 on local Mac OS X.

:+1:


Fix issue with duplicate subject labels in aggregations

Consider the following API request: https://api.wellcomecollection.org/catalogue/v2/images?source.subjects.label=Horses&aggregations=source.subjects.label
This is the aggregation:
        {
          "data": {
            "id": "zssqcytq",
            "label": "Horses",
            "concepts": [],
            "type": "Subject"
          },
          "count": 364,
          "type": "AggregationBucket"
        },
        {
          "data": {
            "label": "Horses",
            "concepts": [],
            "type": "Subject"
          },
          "count": 150,
          "type": "AggregationBucket"
        },
        ...
        {
          "data": {
            "id": "jrnfsbrf",
            "label": "Horses",
            "concepts": [],
            "type": "Subject"
          },
          "count": 12,
          "type": "AggregationBucket"
        },

Because the phrase "Horses" appears three times, it appears as three different entries in the front-end filter; consider https://www-stage.wellcomecollection.org/images?source.subjects.label=Horses
For aggregations it's probably sufficient to bin the IDs when we're aggregating by label, but this might point to a broader issue for subjects later on.

You can see the same issue here for works: https://api.wellcomecollection.org/catalogue/v2/works?aggregations=subjects.label&query=horses

Yeah this is a definite bug, at least in the "Do What I Mean" sense. Aggregating by label should do just that, it shouldn't also take IDs into account.
I guess we just pick the first one for returning in the aggregation, as to then go on and filter by label should return results with any of the IDs.
Once we filter by IDs, then yes we probably need to aggregate by IDs too. Concept merging should hopefully fix this problem, though it would be good to confirm why this is occurring. Are these concepts from multiple sources or are we giving an identity to strings without de-duping?

I guess we just pick the first one for returning in the aggregation, as to then go on and filter by label should return results with any of the IDs.

I think we should take a slightly different approach, and remove the identifiers for the purpose of aggregations – so in this case we'd have a single, unidentified aggregation with 364 + 150 + 12 = 526 entries for "horse". This is what we used to have, and it got broken in some recent index restructuring.

Are these concepts from multiple sources or are we giving an identity to strings without de-duping?

Multiple sources.
Work IDSierra IDMARC source value

bvt9jdzf
 b2498582x650  0 Horses.|0sh 85062160 (650 ind 2 = 0 ⇒ LCSH)
 y9dyqe8mb11318430650  2 Horses.|0D006736 (650 ind 2 = 2 ⇒ MeSH)
uzen4bbab15439306650  2 Horses.

That sounds like a much better approach, then when we have identified filters and aggregations those can return the identified concept.
Interesting example, thanks. Shows that as part of https://github.com/wellcomecollection/docs/pull/83 we do need to match and merge concepts.

Fixed by the latest reindex.


dubbo-parent是头文件不包含依赖。替换成dubbo, 同时增加zk连接的依赖。

What is the purpose of the change
文档不正确，运行时会缺少依赖
Brief changelog
增加dubbo的pom文件。zookeeper/curator-framework的依赖。因为dubbo-dependencies-bom当中声明curator-framework时去除了zookeeper的版本，所以外部还需要额外机上zookeeper的依赖包。
Verifying this change
XXXXX
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GITHUB_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[ ] Run mvn clean install -DskipTests=false & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.
[ ] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #3516 into master will increase coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #3516      +/-   ##
============================================
+ Coverage     64.08%   64.09%   +0.01%
  Complexity       71       71
============================================
  Files           664      664
  Lines         28618    28618
  Branches       4771     4771
============================================
+ Hits          18341    18344       +3
+ Misses         8055     8050       -5
- Partials       2222     2224       +2

Impacted Files
Coverage Δ
Complexity Δ

.../dubbo/remoting/transport/netty4/NettyChannel.java
61.17% <0%> (-4.71%)
0% <0%> (ø)

...pache/dubbo/registry/support/AbstractRegistry.java
80.31% <0%> (+1.18%)
0% <0%> (ø)
:arrow_down:

.../apache/dubbo/remoting/transport/AbstractPeer.java
63.04% <0%> (+4.34%)
0% <0%> (ø)
:arrow_down:

...onfig/spring/extension/SpringExtensionFactory.java
85.36% <0%> (+4.87%)
0% <0%> (ø)
:arrow_down:

Continue to review full report at Codecov.

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

we will refresh README when we release 2.7.1, stay tuned.

merge it in for now, since it helps user to add appropriated zk dependency, again, we will refresh README when we release 2.7.1


Create Resource Validation (according to schema)

add "subAttrKey" to schema attribute. This attribute is used to locate the complex sub-object inside an array. It is useful when checking element mutability inside an array. (i.e. group.members)


Hitting  PowerShell DSC resource timeout

We're receiving
"/Stage[main]/Profile::Win_iis/Dsc_windowsfeature[install_iis]/ensure: change from absent to present failed: Catastrophic failure: PowerShell DSC resource timeout (300000 ms) exceeded while executing"
when we trying to install IIS with all the sub-features.
Our manifest looks like:-
  dsc_windowsfeature { 'install_iis':
    dsc_ensure               => 'present',
    dsc_name                 => 'Web-Server',
    dsc_includeallsubfeature => 'True',
  }

I think we can probably just remove the sub-features and install the ones we really need individually... but is there any way to increase the timeout?

Thanks for the feedback @philipwigg  - we've previously discussed this internally and it's something we will be doing in the future.
Please follow https://tickets.puppetlabs.com/browse/MODULES-3180 for updates.  I'm going to close this GH issue in favor of our JIRA ticket.
Thanks!


Bootstrapping milli

Currently, upgrading from one breaking version of milli to another requires creating a dump and re-importing it. This is not the best user experience, and it would be desirable to improve it.
Bootstrapping milli would mean automatically triggering a reindexing when a breaking version change is detected. On milli's side there are three invariants that need to hold for it to be possible:

the document database needs to be stable. This doesn't seem to be an issue, since the database hasn't changed in a while, and probably never will.
The field id to field name mapping. This one hasn't changed in a while either, so we can probably assume its stability.
The settings stability, this one isn't as clear. The settings are stored in a polymorphic database, and its stability is not straightforward.

A solution would be to store user settings in a serialized structure rather than having each separate field in a database. This struct could embed a version number that would allow us to break in the future, while permitting us to add new settings without breaking anything, replacing missing fields with the default value from one version to the other.
From the looks of it, it doesn't seem like a very complicated work, and it would definitely bring value.

I agree with your point of view regarding the lack of ease of upgrading of the engine. I will just comment some points here:

the document database needs to be stable. This doesn't seem to be an issue, since the database hasn't changed in a while, and probably never will.

The documents database, which is currently a raw LMDB database, is one of the biggest databases that we have right now and if we want to reduce the disk usage of Meilisearch we will probably change the format of it by using a grenad segment oriented data structure. By doing so we will be able to compress the raw user documents without impacting the search time as a search typically returns 20-40 documents at a time.

For a 80m songs database
As you can see, the documents database is 11.11 GiB for 80 million and 100 documents. It is the biggest database here.
The main database weigh:
	total key size: 516 B
	total val size: 130.44 MiB
	total size: 130.44 MiB
	number of entries: 24
The word-docids database weigh:
	total key size: 28.45 MiB
	total val size: 1.60 GiB
	total size: 1.63 GiB
	number of entries: 3276632
The word-prefix-docids database weigh:
	total key size: 37.22 KiB
	total val size: 2.86 GiB
	total size: 2.86 GiB
	number of entries: 10896
The docid-word-positions database weigh:
	total key size: 5.61 GiB
	total val size: 2.75 GiB
	total size: 8.36 GiB
	number of entries: 674516404
The word-pair-proximity-docids database weigh:
	total key size: 999.45 MiB
	total val size: 6.73 GiB
	total size: 7.71 GiB
	number of entries: 80185705
The word-prefix-pair-proximity-docids database weigh:
	total key size: 444.52 MiB
	total val size: 9.70 GiB
	total size: 10.14 GiB
	number of entries: 43943290
The word-position-docids database weigh:
	total key size: 100.74 MiB
	total val size: 2.14 GiB
	total size: 2.24 GiB
	number of entries: 8916197
The word-prefix-position-docids database weigh:
	total key size: 6.67 MiB
	total val size: 5.03 GiB
	total size: 5.04 GiB
	number of entries: 950716
The field-id-word-count-docids database weigh:
	total key size: 90 B
	total val size: 178.49 MiB
	total size: 178.49 MiB
	number of entries: 30
The facet-id-f64-docids database weigh:
	total key size: 0 B
	total val size: 0 B
	total size: 0 B
	number of entries: 0
The facet-id-string-docids database weigh:
	total key size: 30.70 MiB
	total val size: 3.26 GiB
	total size: 3.29 GiB
	number of entries: 1851821
The field-id-docid-facet-f64s database weigh:
	total key size: 0 B
	total val size: 0 B
	total size: 0 B
	number of entries: 0
The field-id-docid-facet-strings database weigh:
	total key size: 4.70 GiB
	total val size: 2.73 GiB
	total size: 7.43 GiB
	number of entries: 353564264
The documents database weigh:
	total key size: 305.18 MiB
	total val size: 10.82 GiB
	total size: 11.11 GiB
	number of entries: 80000100

A solution would be to store user settings in a serialized structure rather than having each separate field in a database. This struct could embed a version number that would allow us to break in the future, while permitting us to add new settings without breaking anything, replacing missing fields with the default value from one version to the other.

You don't need to add a version number in the settings, we can use the version of Meilisearch for that and we already have access to it. However, using JSON instead of split settings could help a lot for retro-compatibility.

Having milli auto-reindex the database would be extremely helpful when embedding milli, as I currently need to manually write migration code and bundle every version of milli alongside my library.
Related issues:

https://github.com/GregoryConrad/mimir/issues/9
for the settings struct https://github.com/meilisearch/milli/issues/551
https://github.com/meilisearch/meilisearch/issues/2570

Closed in favor of https://github.com/meilisearch/meilisearch/issues/2570


How to look at bbox predictions after training?

Hi, I finetuned a pretrained Faster RCNN model.
I used the instance segmentation Mask RCNN pytorch tutorial as a guide.
I finished training and can't figure out how to look at bbox predictions.
For segmentation prediction, the guide used the following to display the mask
Image.fromarray(prediction[0]['masks'][0, 0].mul(255).byte().cpu().numpy())
I tried
Image.fromarray(prediction[0]['boxes'].mul(255).byte().cpu().numpy())
But this doesn't work.
link to tutorial I followed : https://colab.research.google.com/github/pytorch/vision/blob/temp-tutorial/tutorials/torchvision_finetuning_instance_segmentation.ipynb#scrollTo=5v5S3bm07SO1
I used the following to train on a custom dataset.

def get_faster_rcnn_model(num_classes):

model = torchvision.models.detection.fasterrcnn_resnet50_fpn(pretrained=True)

num_classes = 2  # 1 class (person) + background

in_features = model.roi_heads.box_predictor.cls_score.in_features

model.roi_heads.box_predictor = FastRCNNPredictor(in_features, num_classes)

return model

Hi,
I hope you have found how to visualize the boxes. The prediction[0]['boxes'] corresponds to the [x0, y0, x1, y1] coordinates of the pixels in the image, so you can use matplotlib.pyplot.Rectangle to plot it


Clean up SegmentMetadataImpl

Remove the constructor with OfflineSegmentZKMetadata because we should never construct SegmentMetadataImpl with ZKMetadata
Fix an issue where RealtimeSegmentImpl numDocsIndexed should be transient, remove redundant counters
Let ValidationManager directly work on SegmentZKMetadata, reducing the overhead of converting it to SegmentMetadataImpl

Codecov Report

Merging #2439 into master will decrease coverage by 0.06%.
The diff coverage is 86.02%.

@@            Coverage Diff             @@
##           master    #2439      +/-   ##
==========================================
- Coverage    69.3%   69.23%   -0.07%
==========================================
  Files         846      846
  Lines       39437    39381      -56
  Branches     5310     5307       -3
==========================================
- Hits        27331    27265      -66
- Misses      10326    10332       +6
- Partials     1780     1784       +4

Impacted Files
Coverage Δ

...manager/realtime/HLRealtimeSegmentDataManager.java
77.89% <100%> (ø)
:arrow_up:

.../pinot/core/segment/index/SegmentMetadataImpl.java
80.3% <100%> (-2.29%)
:arrow_down:

...not/common/metadata/segment/SegmentZKMetadata.java
77.77% <100%> (-0.16%)
:arrow_down:

.../pinot/core/realtime/impl/RealtimeSegmentImpl.java
67.05% <80%> (+0.1%)
:arrow_up:

...pinot/controller/validation/ValidationManager.java
81.33% <84.61%> (+0.06%)
:arrow_up:

...a/manager/realtime/RealtimeSegmentDataManager.java
80% <0%> (-20%)
:arrow_down:

...kedin/pinot/core/util/SortedRangeIntersection.java
83.82% <0%> (-7.36%)
:arrow_down:

...impl/dictionary/DoubleOnHeapMutableDictionary.java
68.88% <0%> (-6.67%)
:arrow_down:

...r/helix/SegmentOnlineOfflineStateModelFactory.java
56.38% <0%> (-6.39%)
:arrow_down:

...nsport/netty/PooledNettyClientResourceManager.java
85.1% <0%> (-4.26%)
:arrow_down:

... and 16 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 31b1dcb...61d714c. Read the comment docs.


llama-stack-client-swift

Getting error while build after installing the llama-stack-client-swift package in Xcode.
Value of type 'Operations.post_sol_agents_sol_turn_sol_create.Output.Ok.Body' has no member 'text_event_hyphen_stream'
File remoteAgents in LlamaStackClient

+1

+1

This should be fixed after https://github.com/meta-llama/llama-stack-client-swift/commit/9586d1accebc87dfd63bf25a6509ebec4b044f9a – please re-open if you're still experiencing this


[chart] Echarts time-series chart does not have x-axis control

nvd3 line chart has x-axis control:

echart version does not have x-axis control:

Could we add a-axis format in eChart version?
@junlincc @villebro

Echarts 5.0 automatically cut the ticks optimally based on the data time range and time grain setting, that's why we are seeing "inconsistent" format on the x-axis.

We can certainly add a formatter in the Echarts, but the question is, do we want them to work exactly as nvd3 charts?
Can we schedule a session with your users to understand the request better before diving in any implementation. @graceguo-supercat @zuzana-vej

Echarts 5.0 automatically cut the ticks optimally based on the data time range and time grain setting, that's why we are seeing "inconsistent" format on the x-axis.

We can certainly add a formatter in the Echarts, but the question is, do we want them to work exactly as nvd3 charts?
Can we schedule a session with your users to understand the request better before diving in any implementation. @graceguo-supercat @zuzana-vej

do we want them to work exactly as nvd3 charts?

No. We only want to have a control to format x-axis, for example to show only month, or month/year etc.

Right now is nothing user can choose.

do we want them to work exactly as nvd3 charts?

No. We only want to have a control to format x-axis, for example to show only month, or month/year etc.

Right now is nothing user can choose.

we probably can do something simple for now. we don't have much planed around Echarts in Q1. do you have a timeline? @graceguo-supercat

we probably can do something simple for now. we don't have much planed around Echarts in Q1. do you have a timeline? @graceguo-supercat

Not urgent at all. At least we have nvd3 line chart to use. If this request can be added to the long term roadmap will be great, at least before fully deprecating nvd3 line chart. Thank you @junlincc

Not urgent at all. At least we have nvd3 line chart to use. If this request can be added to the long term roadmap will be great, at least before fully deprecating nvd3 line chart. Thank you @junlincc

Hi all,
I'm searching for echarts version of line chart in chart list, but I'm not able to find it. git version of today.

Hi all,
I'm searching for echarts version of line chart in chart list, but I'm not able to find it. git version of today.

@zufolo441, it is a time series chart

@zufolo441, it is a time series chart

@junlincc  yes, I think x-axis control and customization should be useful

@junlincc  yes, I think x-axis control and customization should be useful

I believe @maloun96 is currently working on adding x-axis formatting support as a byproduct of another feature, so this may be fixed sooner rather than later.

I believe @maloun96 is currently working on adding x-axis formatting support as a byproduct of another feature, so this may be fixed sooner rather than later.

@eugeniamz
thanks!

@eugeniamz
thanks!

confirmed this feature is available in TimeSeries Echarts in master now. thank you @maloun96 for the enhancement!
@graceguo-supercat @zuzana-vej
https://user-images.githubusercontent.com/67837651/105637127-ada26e80-5e20-11eb-85b7-7d8e431af996.mov

confirmed this feature is available in TimeSeries Echarts in master now. thank you @maloun96 for the enhancement!
@graceguo-supercat @zuzana-vej
https://user-images.githubusercontent.com/67837651/105637127-ada26e80-5e20-11eb-85b7-7d8e431af996.mov


DomainMapping does not work with local gateway "mesh" mode

When I add the entry "local-gateway.mesh: mesh" into config-istio ConfigMap to indicate using mesh mode, DomainMapping does not work.
The request sent to the custom domain will get the response: no healthy upstream.
I think this is because the destination of the VS of the DomainMapping is the k8s service of the target ksvc. When setting "local-gateway.mesh: mesh", the k8s service of the target ksvc  will not be ExternalName service. Instead, it is just a headless service. So the destination of the VirtualService of the DomainMapping will become invalid.
/cc @julz

@ZhiminXiang is this the case before/after/both-before-and-after https://github.com/knative-sandbox/net-istio/pull/512?

I can confirm that it happens after that change. I haven't tried to test it before the change. but I will test it.

Thanks @ZhiminXiang thatd be helpful. What's the correct way to reference a service from outside the mesh in mesh mode?

I think essentially the issue is in the mesh mode, the cluster local gateway is not needed and can be removed, while the DomainMapping feature depends on the cluster local gateway for routing requests.
I think the fix could be:

we need to explicitly ask users to not remove the cluster local gateway if they want to use mesh mode.
we may want to provide an extra field in the config-istio CM to set the cluster local gateway service used by DM when users want to use mesh mode.

If the route service is ExternalName service, then we just use the ExternalName service as the destination service.
If the route service is headless service which means users use mesh mode, then we fall back to the cluster local gateway service defined in the config-istio CM.

Another possible option is we may be able to always keep cluster local gateway service and set the route service as ExternalName service.
But for the mesh mode, I am not sure for the internal service to service scenario, if the requests will be sent to the cluster local gateway (i.e. client envoy -> cluster-local-gateway -> destination) or directly to the destination (client envoy -> destination).
If it will have the extra hop (cluster-local-gateway), then this is not an option.
/cc @JRBANCEL this is related to https://github.com/knative-sandbox/net-istio/issues/549

I have tested the case that 1) Istio sidecar injection enabled and 2) no entry "local-gateway.mesh: mesh" in the config-istio CM 3) dummy knative-local-gateway service (i.e. the service label selector selects dummy labels that map to no pods). And the service to service communication still works.
So I think the entry "local-gateway.mesh: mesh" is not necessary. The service to service requests won't go to cluster local gateway when sidecar is injected in the client side as long as the service knative-local-gateway service exists.

So I think the solution to handle the "mesh" mode could be:

We explicitly require users to install local gateway service and keep the cluster local gateway entry in the config-istio CM if users want to use DomainMapping feature no matter if users want to use "mesh" or "no mesh".
We should not recommend users to use entry "local-gateway.mesh: mesh" in the config-istio CM and to remove the cluster local gateways service because 1) there is no difference in terms of the mesh behavior when using or not using entry "local-gateway.mesh: mesh", 2) the overhead of keeping cluster local gateway service is small (just an extra k8s service and Istio Gateway)
We will still support "local-gateway.mesh: mesh" at least for the short term. For the long-term, we may want to deprecate it.

Hi @ZhiminXiang, why this property was removed? Now with this, the v1/Serivce is created by istio with type ExternalName and then istio does not added as a cluster, so the communication from pod to pod using svc.cluster.local does not work anymore. That is what I see in my current implementation. With the removal of this property, does knative support pod to pod communication without going to the gateway?


#[serde(alias = "name")]

Support it. Make sure we can handle multiple of them and unit test it!

I think this should be tied into #190


Help users to migrate to new tasks.json version 2.0.0.

VSCode Version: 1.1.0

To make use of the task execution in the terminal users need to migrate to version 2.0.0 of tasks.json. Support this migration path with a automatic conversion of version 0.1.0 to 2.0.0.

https://github.com/shanalikhan/code-settings-sync/blob/master/.vscode/launch.json
https://github.com/shanalikhan/code-settings-sync/blob/master/.vscode/tasks.json
Can you tell me what i have to change in order to switch to 2.0 ?

In this simple example change version to 2.0.0

Moving to April. We added API for task in this milestone and we need to wait at least one release before we can finalized the API. Until then we shouldn't finalized the 2.0 version of tasks.

also docs have no mention of 2.0.0 either.

Agree, reason being that this is not the default yet. It got mentioned in the release notes.

Any idea why a problem matcher under version 0.1.0 correctly finds a single error in task output but when I switch the tasks.json to 2.0.0 the same problem matcher reports the single error multiple times?

@rkeithhill nothing has changed with problem matchers and the terminal engine is around since 2 releases. Do you have steps to reproduce (best a workspace to clone from). Then I will look into this.

Just install the PowerShell extension, execute "PowerShell: Open Examples Folder", edit the file Tests\PathProcessing.Tests.ps1 - change line 77 from:
        $files[1] | Should Be "$WorkspaceRoot\SampleModule.psd1"

to
        $files[2] | Should Be "$WorkspaceRoot\SampleModule.psd1"

That will introduce a Pester test error.  Then run the task named test.  That should be enough so you see the multiple errors for the one error.  If you change the tasks.json version back to 0.1.0, reload the window and run the test task again, you should see just the single error.

Just tried this again in today's Insiders build and the problem is still there.  In fact, it seems worse (even more duplicate problem listings).

I looked into it today and I can reproduce this in two ways: no problem is recognized and a lot of problems are recognized. From a first investigation this comes form the 'progress overlay' the runner is showing and the fact that lines reported multiple times by the underlying terminal. Need to come up with a test case to forward this to the terminal implementer.

I debugged this further and the problem is that Pester when run in a 'terminal' (a tty/pty is attached) outputs lines n times. This depends on the progress overlay and the height of the terminal.
When run in the  output panel we simply spawn a child process without a tty and then no progress or ANSI control characters are emitted.
I looked in Pester and if I can disable this 'stylish' output but wasn't successful. @daviwil @rkeithhill do you know such an option.
I can also look into de-duping this in VS Code until we have https://github.com/Microsoft/vscode/issues/29840

Cool!  Yes, we do have a way to eliminate the progress output.  I can change the Pester task args from this:
            "args": [
                "Write-Host 'Invoking Pester...'; Invoke-Pester -PesterOption @{IncludeVSCodeMarker=$true};",
                "Invoke-Command { Write-Host 'Completed Test task in task runner.' }"
            ],

to this:
            "args": [
                "Write-Host 'Invoking Pester...'; $ProgressPreference = 'SilentlyContinue'; Invoke-Pester -PesterOption @{IncludeVSCodeMarker=$true};",
                "Invoke-Command { Write-Host 'Completed Test task in task runner.' }"
            ],

Long term, it would be nice if the tasks/terminal/problems combo could handle these progress overlays but I can appreciate the difficulties in doing so.
@daviwil A VSCode-based UI for progress would also eliminate this issue ... if only PowerShell progress reporting wasn't so complex.

@dbaeumer BTW now that this problem has a workaround, I see another issue.  If the terminal isn't wide enough, the output that the $pester problem matcher looks at changes.  That is, what should be a single line with filename and line number on it, gets wrapped to two lines.  The problem matcher no longer matches and the problem (test failure) doesn't get flagged.  Perhaps I can adjust the problem matcher to handle this but I have to think other folks will run into this issue.

@rkeithhill I think we should be able to determine when a line is wrapped by PowerShell eventually and unwrap it. I just added this recently when copying wrapped lines https://twitter.com/Tyriar/status/875044402867163137
You could verify whether this method should work in PowerShell by trying to copy and paste a wrapped line in the latest Insiders.

@Tyriar Just tried on today's Insider build and it doesn't seem to work for PowerShell.  :-(
Open PowerShell in the terminal and try executing dir in a narrow terminal window and copy/paste the wrapped output.
The difference may be due to PowerShell's formatting & output engine.  PowerShell examines the host's width and adjusts its output - inserting newlines (or truncating with ...) and even placing subsequent lines of data in the appropriate columns e.g.:
-a----         9/9/2015   8:45 PM           2435 UIAutomation_record
                                                 ing_short_9_9_2015_
                                                 8_35_PM.ps1
-a----         6/6/2017   9:33 PM             10 unicode.txt
-a----         7/4/2017   9:03 PM             31 Untitled-1.ps1
-a----         7/6/2017   9:54 AM            183 _lesshst

In the case of Bash, is it the "terminal" that inserts the newline to force wrapping or does the newline come from Bash?  I suspect it is the former, which would make PowerShell behave fundamentally different than Bash in this regard.

@rkeithhill yeah maybe that won't work then, for bash it's the pty that does this which is at a higher level than bash. For Windows it works by running PowerShell through https://github.com/rprichard/winpty which allows us to plug in using Unix-like APIs, there are many issues that come out of that though as the Windows APIs around this are not particularly compatible.
This is particularly problematic as there are spaces padding it out:
-a----         9/9/2015   8:45 PM           2435 UIAutomation_record
                                                 ing_short_9_9_2015_
                                                 8_35_PM.ps1

You would either need to tell PS how to format (preferable) or the problem matcher would need to know about how PS is formatting these things (less preferable).
On the topic of scroll bars, it would be possible but I'm not sure we would want to go down that route as it's a lot of work/complexity for not much payoff imo. Plus it would further diverge how the terminal behaves on Windows which was always one of the core benefits of VS Code's terminal (how you configure/use it the same on all OS's).

You would either need to tell PS how to format (preferable)

Well if this particular command wrote to the PowerShell output stream I could do that but no, it writes directly to the host so it can "color" the text.

@Tyriar just to clarify for me: the terminal itself is no longer inserting new lines into the emitted data to wrap lines. This is a pure visual thing. Things are different in PowerShell since there the engine itself formats to the width of the screen (not simple line wrapping)

@dbaeumer yeah I think that's right and not sure what we can do about it, apart from somehow getting PS not to output like that (some script arg?) or getting tasks to understand this layout better.

The 2.0.0 task system is the default for quite some time. I will close the issue since we don't plan to provide an automatic conversion right now.
If you have problem with the 2.0.0 runner please open a new issue so that we can track it separately. Having a global item will not help addressing them.


Bump aenum requirement from aenum==3.1.11 to aenum>=3.1.11 or aenum==3.1.12

I'm using Home Assistant in a docker environment with this image:
https://hub.docker.com/r/linuxserver/homeassistant
In this image, aenum has been updated to version 3.1.12; the integration requires version 3.1.11.
Unfortunately, due to some permission issues, Home Assistant cannot downgrade the package in this docker image, see:
https://github.com/linuxserver/docker-homeassistant/issues/74
There is a solution using a docker mod:
https://github.com/linuxserver/docker-mods/tree/universal-package-install
but it would be easier /  neater if the integration requirement could be changed in:
aenum>=3.1.11
or
aenum==3.1.12

have the same issue, how did you fix this until req changed ?

@tismij if you're using the same Docker image, you can use:
https://github.com/linuxserver/docker-mods
Basically, you should add these environment variables to your container configuration:
DOCKER_MODS=linuxserver/mods:universal-package-install
INSTALL_PIP_PACKAGES=aenum==3.1.11

noticed in manifest.json it says:  "requirements": ["aenum==3.1.11"]
When I have more time will try changing in here and seeing if that is enough.

Hey @Veda82 @tismij  I changed the requirements to >= 3.1.11 hopefully it will fi the issue for you.

Great! I will test and let you know!

Just tested the update, everything seems fine to me...
Thanks for the support!


Incompatible with BetterEmotes

BetterEmotes 1.4.2 is broken for clients (all with same mod list/identical configs) with TooManyEmotes. For reference, MoreEmotes 1.3.3 does not reproduce this bug.
Clients loading into the lobby report their camera bugging out and frenetically bobbing uncontrollably. Included is the relevant log info
Thanks!
BetterEmotes_TooManyEmotes_critical_conflict.txt

Thanks to the BetterEmotes dev, they have figured out it was somehow being caused by a conflict with the Motion Tracker mod.  Given this information, I'm going to close this and report the issue directly to that dev for further investigation.
Cheers


Add start the service github-authorized-keys automatically on an instance restart

What

Add start the service github-authorized-keys automatically on an instance restart

Why

The service should be started after an instance will be restarted

Plan

this fix included in https://github.com/cloudposse/terraform-aws-ubuntu-github-authorized-keys-user-data/pull/2


added ability to register custom buttons (issue 29)

This PR adds ability to register custom buttons, as discussed in issue 29
This is a quite tricky problem to solve, and this implementation is not that great. Using eval always feels dirty, and having to make the js file publicly available and fetching it with ajax also feels kinda gross, butt hopefully it will get the ball rolling and at least we have a starting point.
To register custom buttons. create a js file containing a function that receives a froala editor instance as its first parameter and make it publicly available on the server.
example:
// /public/js/alert.js
(FroalaEditor) => {
    FroalaEditor.DefineIcon('alert', { NAME: 'info', SVG_KEY: 'help' });
    FroalaEditor.RegisterCommand('alert', {
        title: 'Hello',
        focus: false,
        undo: false,
        refreshAfterCallback: false,
        callback: function() {
            alert('Hello!');
        }
    });
}

Then, in your froala config file, add the following:
'customToolbarButtons' => [
    'alert' => '/js/alert.js',
],

And finally - add the button to your toolbarButtons array like any other button
That's all.
What do you think?

@slavarazum any response? :)

Hi,
Thanks for the implementation. Agreed with you that it's not enough clean as I'd want, but it may help to make new thoughts about solving this issue.

So what's the alternativs?
We can't get them in at build time, so they have to be picked up runtime.
Using imports would be optimal, but i guess we cant rely on all the browsers to support that. Or can we? https://caniuse.com/#feat=es6-module-dynamic-import
IE and edge would not be supported then.
We could also output the button-logic into the dom to be picked up by plugin.

For now dynamic imports are using to include Froala plugins, maybe we can use it for custom buttons.

Alright. I'll look into that


Python: modules

Description 📜
I have explained what are Python modules, why modules are used, examples and syntax for declaring and calling the modules.
Fixes #578

Type of change 📝

[x] Audio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)
[ ] Vi#deo (Animations, screen-recordings, presentations and regular explanatory films are all possibilties etc)
[ ] Doc#umentation (Content Creation in the form of codes or tutorials)
[ ] Other (If you choose other, Please mention changes below)

Domain of Contribution 📊

[ ] Android Dev #(Flutter)
[ ] Android Dev #(Java)
[ ] Android Dev #(Kotlin)
[ ] Backend Dev #(Java)
[ ] Backend Dev #(.NET)
[ ] Backend Dev #(PHP)
[ ] Backend Dev #(Python)
[ ] C/#CPP
[ ] Competitive #Programming
[ ] Cyber #Security
[ ] DS#A
[ ] Data#base
[ ] Datascience with #Python
[ ] Datascience with #R
[ ] Frontend Dev #HTML/CSS/JS
[ ] Frontend Dev #React/Angular/Vue
[ ] Go#lang
[ ] Interview #Prep
[ ] Java_#Domain
[ ] JavaScript_#Domain
[ ] ME#RN
[ ] Machine #Learning
[ ] Open #Source
[x] Python_Domain
[ ] Ru#st
[ ] Statis#tics
[ ] UI/#UX

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

@Randrita
Please verify it!


Modify onChange() example

func arrow missing

this change should actually be made in the javascript file itself, as these reference docs are generated from code:
https://github.com/howdyai/botkit/blob/master/packages/botkit/src/conversation.ts#L486

fix source comment instead


How do I trigger in linux?

I'm trying everything I can to trigger this, but I'm not on a mac. I'm in linux with a standard us keyboard.
What is the equivalent for cmd+shift+P?

ctrl+shift+P should work for linux.
I probably ought to put that in the readme instead of just the macos binding.


feat(design-system): modify default styles in the suggestion-group

This will allow the last element of a suggestion group to go to the end of the container.
EX-7845

Should we add an example without setting the width to 200px? To show the alignment behaviour in the showcase. Good job! 👌

I think that its not necessary, by default its positioning the button at the end.


Export buttons in dashboard don't work

Both export buttons don't work:

I can see the URL is wrong, we may not be passing the backendUrl variable correctly to that method!


Problem

Hello, I apologize for my English, I'm not very good at GIthub yet, please tell me where I can find all possible combinations for mathematical symbols

I'm not sure I fully understand. If you mean the mathematical symbols for LaTex, which is what this program uses, you can find those here:
https://www.cmor-faculty.rice.edu/~heinken/latex/symbols.pdf
Also this is a great resource for finding the code for symbols if you are still having trouble finding them:
https://detexify.kirelabs.org/classify.html

Stale


Incomplete list of sites in Forcedown for US opportunistic sites.

Fixes #9814
Status
not-tested
Description
Adding all the T3_US_ opportunistic sites in the FORCEDOWN list in the wmagent deployment script.
Is it backward compatible (if not, which system it affects?)
YES
Related PRs
External dependencies / deployment changes

Superseeded by https://github.com/dmwm/WMCore/pull/9844 , thanks for providing the initial patch, Todor.


Fixed commit lint

Type: Refactor

Done:

[x] updated commitlint, in order fix the error "formatted.join is not a function"
[x] Moved script 'commitmsg' to husky.hook,  in order to remove the warning.

@priscilawebdev in order to track this could yo document what is exactly the error? any other ticket reference that explain the origin of that warning

@juanpicado it's not in progress ...I will describe the error..


Misc typo

Uptaking the Application app
should be ??
Updating the Application app
[Enter feedback here]

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

ID: 7c934418-1448-eb40-9d9a-11c8c435f491
Version Independent ID: a788c5c0-0e20-14f8-d4f4-a5e0eb646589
Content: The Microsoft_Application.app File - Business Central
Content Source: dev-itpro/developer/devenv-application-app-file.md
Service: dynamics365-business-central
GitHub Login: @SusanneWindfeldPedersen
Microsoft Alias: solsen

Hello, thanks for your feedback. It is actually meant to say Uptaking the Application app - uptaking in this context means building on (top of) - we use the same terminology in other documentation topics. Thanks.


Logout button repositioning in mobile view & Logout confirmation dialog

🚀 Feature Request
Is your feature request related to a problem? Please describe.

I think the logout button in the mobile menu doesn't look good also it creates an uneven gap between the menu & the button

Describe the solution you'd like

I think we should move the logout button below the user name and also add a confirmation dialog.
I would like to work on this task

@SubxX Assigned you


Feature request for Disentangled Attention from DeBERTa

Paper in case you missed it.
https://arxiv.org/abs/2006.03654

@hadaev8 I think the novelty in DeBERTa is mainly in their positional encoding, and I'm deliberating whether to create a separate repository where I aggregate all the positional encoding solutions (and then make them pluggable into x-transformers)

@lucidrains
Please let me know then it happens!

@lucidrains I second that especially if we are starting to discuss 2D and 3D specific ones.


Browser crashes

If the audio is too large, the browser will crash.

audio size is 775MB,audio length is 70 minutes, format is wav
Is this problem related to computer configuration or browser?

It's a known browser issue.

Is there any way to solve it?restricting specific computer configurations or browser restrictions?
my current solution is to cut too long audio into several small pieces of audio,
but I don't think this is a good solution.
so can you give me some suggestions for solving it?
thank you very much.

What version of wavesurfer are you using?

In 2018, maybe we can look at this issue again. A quick google search yielded this interesting gist: https://gist.github.com/revolunet/e620e2c532b7144c62768a36b8b96da2

@213zhaoqi you can get the wav data json file from this api I created and use backend: "MediaElement" while showing the full waveform.
I'll be increasing the upload size soon, it's currently set to 20 MB as I'm waiting for users to test it. #1183
https://waveform.al


Fix #60 - Use Union type instead of enum with Align

Fixes the error in #60 .
Change from enum to union type for Align prop. Now it works with TypeScript in the way the documentation explains it.

Nice!

Thanks for merging so soon!


add pyvcf

Hi,
I am wondering if pyvcf could be added to the docker file?
Thanks

Added to #56

If the package doesn't get added, I highly recommend creating your own flavour of the notebook, it's really easy to do:
FROM bgruening/docker-ipython-notebook:latest
RUN pip install pyvcf

I know, but considering that this such a common package, I just though
about suggesting it...
On 1 September 2015 at 17:28, Eric Rasche<EMAIL_ADDRESS>wrote:

If the package doesn't get added, I highly recommend creating your own
flavour of the notebook, it's really easy to do:
FROM bgruening/docker-ipython-notebook:latestRUN pip install pyvcf
—
Reply to this email directly or view it on GitHub
https://github.com/bgruening/docker-ipython-notebook/issues/57#issuecomment-136890807
.

--
"Democracy matters more than any currency arrangement." - Paul Krugman

I will look at the dependencies for this package and if they are not so large I will add it in the next version.
Thanks!


Always getting "Error Loading JSON File... Exiting"

I created a simple JSON file as example.json  in my C:\projects directory and when executing python file, when prompted for json file, I tired with giving input as example.json, C:\projects\example.json, and c:\projects\example.json and I get "Error loading JSON file.. Exiting"
[{
"name": "John",
"age": 30
}]

can you try with the JSON file in the same directory as the Python script?

I tried it and got that working but here is another problem, in the below block,   file_size = os.path.getsize(file_name)  is not working. If I hard code the path for file_size, it is able to detect the json, else it does not.
file_name = input('filename: ')
f = open(file_name)
file_size = os.path.getsize(file_name)
data = json.load(f)


Generate password using given text.

💥 Proposal
Generate a password using the given text.
You can include the special keyword in between or the start or at the end.
You can capitalize some alphabets in text.
No whitespaces should be there.
You can add numbers in the text.

Can you please assign me this issue as a part of Cross Woc. I would be very happy to do it

Can you please assign me this issue as a part of Cross Woc. I would be very happy to do it

Yes @Akash20x , you can work on it.
I am assigning you this task.
If you have any problem while doing it, you can reach me on Discord.
You are given 5 days to complete this task.

Ok

Please check my pull request

Please check my pull request

Can you check my comment on your PR?


Update background colors

The colours in the designs have changed once more since https://github.com/silverstripe/silverstripe-framework/pull/4940.
Now looks like

Thanks. :D


Scrambled text, possibly during presence of Emoji on the same line

In the attached gif, I'm moving the cursor from the beginning of the line to the end that has an Emoji character. The scrambled text can be noticed. Although not evident in the GIF, the text scrambling can be seen if I have emoji characters in the status line (using airline plugin).

I've opened macvim in terminal mode with the following command:
mvim -v -N -u NONE ~/.tmux.conf

The terminal emulator used for the video recording is iTerm2 running tmux, but this also happens on OSX's Terminal.app without tmux.
OSX version: El Capitan 10.11.2
iTerm2 version: 2.9.20151117-nightly
MacVim version: Github master branch compiled today using Homebrew

MacVim can't handle the terminal rendering. You should ask it in iTerm2 community.
If MacVim GUI doesn't render the emoji properly, please let me know the UCS-2 code.

Thanks for the pointer. I forgot to test it in MacVim GUI before opening this issue :facepalm: Checking on Terminal.app and seeing the issue there too suggested that it was independent of the terminal emulator. But I was wrong.


bug: false alerts about Script injections

Describe the bug
The warning about a </style> tag in the additionalCSS property is shown every time the extension is used, even if the property is not set in either user settings or workspace settings.
To Reproduce
Steps to reproduce the behavior:

ensure the zipViewer.ZipEdit.additionalCSS property is not set in the user settings.json or the workspace settings.json.
open a zip file using the Zip Viewer Editor.

Expected behavior
No error message should be shown.
Environment
What type of VS Code were you using when you encountered the problem?
Mac OS Sonoma 14.5
Visual Studio Code Version: 1.91.0
Zip Tools v3.17.0

Taking a quick look, this seems to be the issue:
https://github.com/AdamRaichu/vscode-zip-viewer/blob/64b380d0c37c70b5364dbf486ec0071d59215806/src/ZipEdit.js#L56-L63
The warning should be inside the if condition.

heavy face palm
I'll get that fixed immediately, and sorry for the delay, I was taking a break this month.

Well I thought I fixed this, but apparently not....

Sorry for the extra notification. I actually did fix it, I just had the test line in my user settings.


Is there an inline mode

Didn't tried it but I tested the demo and I find it very clean. I'm currently looking at what exists on github for this type of components. Is it possible to use it directly without any trigger or input?

Hey, thank you for your question. It's possible to use directly but one mode at the same time only. You can look at it on the demo already.
<mat-clock-dials [selected]="date"></mat-clock-dials>

<mat-time-inputs [(selected)]="date"></mat-time-inputs>

I've fixed some issues for that as well. Fixes will be available in 12.1.0

Think I can close this issue. If you have any other questions or find bugs feel free to create a new issue.


Middle name not being edited

Hello!
I have an issue that my middle name is not being replaced. It may have something to do with the fact that my deadname's middle name is now my current last name? So, it looks like it's repeating my last name twice. It could also be how my school's website printed my name because it does have weird extra spaces. However, I tried adding spaces to the fields themselves with no luck. I would really appreciate it if you could look into this bug.
Thank you so much!
Using: Firefox Extension

Would like to bring this back to someone's attention. I am also using Firefox and noticed that the middle name is not being changed.
Thank you!


[v1.8] RTL is 'on' in menu but is an available app in web interface

Installing RTL might have gone wrong but it shouldn't be shown as installed in the menu when you connect via ssh and be an available app in the web interface.

I reinstalled RTL in previous versions via the menu so now I tried to reinstalled it via the webinterface.
Successfully.

TODO: Add for all apps that are available via the WebUI a final install check before setting on in config.

App instals have now a fatpack option - that sjhould fix most oif install problems for regulat users. But sctipts to be improved in the future - so pushing this to v1.10

@adevoss Can you check again on 1.9.0? Should be fixed AFAIK.
Closing this for now, if it still happens, please re-open.

:ok_hand:


[Mint] Add timestamps to promises and proofs_used tables

Allows better tracking of mint balance and auditing.

@callebtc This is already done, is it not?
#332

Yes, you're right again! Thank you


type 503 warning

what does meaning "type 503 there is no valid location provider available". Some of devices with android os 9 cannot get location.   RequestPermission returns 'granted' but  when device's location  turned off, this message returns and can not get location.

 The library tells you this so you can handle it. If you really need to location, I recommend you tell the user why and tell them how to enable location services again.

How can I get hold of that message {type: 503, message: 'there is no valid location provider available'} so that I can test it?

Is there any way to handle this error, I havnt found anything in the Read me

Any update on  how to catch the error ?

Why did you close this issue when it's still not resolved ?!
I am getting this warning despite of having try and catch ...Any help ?


Allow sorting attributes/namespace declarations (for diffing)

XML documents with identical structure and content, but different ordering of attributes (e.g. xmlns declarations) for an element, show differences, e.g. in xmq-meld. It would be great to have a choice to ignore/suppress these differences (since attribute order is insignificant according to the XML spec).

Good idea! I added --sort-attributes so now you can write:
xmq-meld --sort-attributes alfa.xml beta.xml
Please test!

A (very brief) test with the file pair I used yesterday (slide files within a PPTX file) worked beautifully: with --sort-attributes xmq-meld now considers them identical. Thanks a lot!
NB: Some open source software (authors, I should probably say) never cease(s) to amaze me! Sometimes, you just submit an idea or bug report in a hurry, only to find that the world has become a better place the very next day — you just made my day!


Some URLs cause parsing error for app-link

Describe the bug
I have noticed that app-link URLs like: https://company.okta.com/home/amazon_aws/0obm6u4wyuVYgbLdJ0x7/172 cause parsing error in okta_auth_config.py at line 45 (version 0.4.4).
It is caused by an underscore character '_' that should be encoded prior verification like '%5F'.
Traceback (most recent call last):
  File "/Users/nowam1/Library/Python/3.7/bin/okta-awscli", line 8, in <module>
    sys.exit(main())
  File "/Users/nowam1/Library/Python/3.7/lib/python/site-packages/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "/Users/nowam1/Library/Python/3.7/lib/python/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/Users/nowam1/Library/Python/3.7/lib/python/site-packages/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/Users/nowam1/Library/Python/3.7/lib/python/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/Users/nowam1/Library/Python/3.7/lib/python/site-packages/oktaawscli/okta_awscli.py", line 115, in main
    aws_auth, okta_profile, profile, verbose, logger, token, cache, refresh_role
  File "/Users/nowam1/Library/Python/3.7/lib/python/site-packages/oktaawscli/okta_awscli.py", line 18, in get_credentials
    okta = OktaAuth(okta_profile, verbose, logger, totp_token, okta_auth_config)
  File "/Users/nowam1/Library/Python/3.7/lib/python/site-packages/oktaawscli/okta_auth.py", line 25, in __init__
    self.app_link = okta_auth_config.app_link_for(okta_profile)
  File "/Users/nowam1/Library/Python/3.7/lib/python/site-packages/oktaawscli/okta_auth_config.py", line 45, in app_link_for
    if not validators.url(app_link):
  File "<decorator-gen-29>", line 2, in url
  File "/Users/nowam1/Library/Python/3.7/lib/python/site-packages/validators/utils.py", line 83, in wrapper
    value = func(*args, **kwargs)
  File "/Users/nowam1/Library/Python/3.7/lib/python/site-packages/validators/url.py", line 148, in url
    result = pattern.match(value)
TypeError: expected string or bytes-like object
➜  ~ cat /Users/nowam1/Library/Python/3.7/lib/python/site-packages/oktaawscli/okta_auth_config.py

To Reproduce
Steps to reproduce the behavior:

Specify app-link URLs in the form specified above.
Run octa-awscli
See error

Expected behavior
A clear and concise description of what you expected to happen.
Output
Output of your okta-awscli run, using the --debug flag.
Environment (please complete the following information):

OS: MacOS
okta-awscli 0.4.4

Additional context
Workaround:

comment out line 45 - 47
- OR -
encode underscore or other invalid URL character with proper sequence e.g. '%5F'

@machiev I'm not able to reproduce this on version 0.4.4. The validators library shouldn't need to have it's input url encoded.
Specifying the app-link URL that you provided doesn't produce that error for me. The URL validates ok, but I get an expected auth failure. If I then change the URL to an obvious invalid version (such as putting a parenthesis in the hostname), the URL fails validation with a clear error message.
To help diagnose further, can you please paste the following information?

The full output of okta-awscli, not just the stack trace.
The profile block in .okta-aws that you're using, with sensitive data removed of course.
Output of pip freeze

@jmhale Hmm, it's really strange - now I cannot reproduce it either - just works encoded or not. It makes me think that maybe there was a non-ASCII character when I copied the link. I have copied it from some chat window. I think we can close it and wait for the next occurrence.

Ok, thanks for confirming @machiev! Actually, I think that we should keep this issue open, because pasting in non-ASCII characters is a valid, albeit rare failure scenario, so it makes sense to catch that TypeError exception and print something useful.


Which combination of dependency versions is supposed to work?

I was just trying out the glyphs2ufo command and got the following error:
Traceback (most recent call last):
  File "/usr/local/bin/glyphs2ufo", line 9, in <module>
    load_entry_point('glyphs2ufo==1.0', 'console_scripts', 'glyphs2ufo')()
  File "build/bdist.linux-x86_64/egg/glyphs2ufo/__main__.py", line 56, in main
  File "build/bdist.linux-x86_64/egg/glyphs2ufo/glyphslib.py", line 88, in build_masters
  File "build/bdist.linux-x86_64/egg/glyphs2ufo/glyphslib.py", line 79, in write
  File "/usr/local/lib/python2.7/dist-packages/robofab/objects/objectsRF.py", line 471, in save
    glyph = self[glyphName]
  File "/usr/local/lib/python2.7/dist-packages/robofab/objects/objectsBase.py", line 184, in __getitem__
    return self.layers.getDefaultLayer()[glyphName]
AttributeError: 'RFont' object has no attribute 'layers'

This comes from the ufo3k branch of robofab. In a rather recent issue about this error: https://github.com/robofab-developers/robofab/issues/53 @typesupply states:

The R* objects in the ufo3k branch is incomplete and is not going to be completed. UFO 3 support is going to be accomplished by a rewrite. The ufo3k branch objects were an experiment that failed.

I was using ufo3k because https://github.com/LettError/MutatorMath/blob/master/README.md claims its dependencies as:

The UFO processing tools in MutatorMath need some additional libraries. Note that anticipating the move to UFO3, MutatorMath already uses the UFO3 branches for Defcon, FontMath and UFOLib. While these modules have received a lot of testing, these are not the main branches yet.
| Library  | Author             | Branch | URL                                              |
| -------- | ------------------ | ------ | ------------------------------------------------ |
| Defcon   | TypeSupply.com     | ufo3   |  https://github.com/typesupply/defcon            |
| FontMath | TypeSupply.com     | ufo3   |  https://github.com/typesupply/fontMath          |
| Robofab  | RoboFab Consortium | ufo3k  |  https://github.com/robofab-developers/robofab   |

Based on Tal's comments it would seem that those MutatorMath readme notes are invalid. Personally, I've been using the master branch of RoboFab (and everything else) and that works fine.

In the ufo3k branch, /ufoLib moved away from /robofab into its own package. I'm about 100% certain that the MutatorMath note is referencing the ufoLib package, not the robofab package. Anyway, the UFO 3 branch is an official example implementation of the UFO spec:
https://github.com/unified-font-object/ufoLib

On Jan 12, 2016, at 7:35 PM, James Godfrey-Kittle<EMAIL_ADDRESS>wrote:
Based on Tal's comments it would seem that those MutatorMath readme notes are invalid. Personally, I've been using the master branch of RoboFab (and everything else) and that works fine.
—
Reply to this email directly or view it on GitHub https://github.com/googlei18n/glyphs2ufo/issues/8#issuecomment-171111429.

@typesupply indeed I had not installed the ufoLib since it became stand alone, however, it did not solve the problem.
Having installed the master branches of all dependencies I can now successfully export the Glyphs-masters. Exporting instances is failing though:
$ glyphs2ufo -g Mirza\ 2\ Masters.glyphs -m masters -n instances
[…]
>>> Writing masters
Traceback (most recent call last):
  File "/usr/local/bin/glyphs2ufo", line 9, in <module>
    load_entry_point('glyphs2ufo==1.0', 'console_scripts', 'glyphs2ufo')()
  File "build/bdist.linux-x86_64/egg/glyphs2ufo/__main__.py", line 59, in main
  File "build/bdist.linux-x86_64/egg/glyphs2ufo/glyphslib.py", line 101, in build_instances
  File "build/bdist.linux-x86_64/egg/glyphs2ufo/interpolation.py", line 43, in interpolate
  File "build/bdist.linux-x86_64/egg/glyphs2ufo/interpolation.py", line 83, in add_masters_to_writer
KeyError: 'com.google.glyphs2ufo.weightValue'

However, that's rather another issue.

The last issue should be fixed as of ddebad2605c7fb2ef1cb9f9d0c1aeccbd799784b. Can we close this?


Add example project to nightly release tar

The editor creates an example project if no directory is passed.
When inside the git repository, examples/hello-world is copied to a new folder called new-roc-project.
When using the roc binary, this example is not available. For this reason it would be nice to have an example included.
Having an example app is also useful for those getting started with roc.

Editor is no longer being worked on, and has been removed from the repo. Closing.


update license

Might want to update the license file information from the defaults:
https://github.com/fireeye/capa/blob/master/LICENSE.txt#L190

thanks!


Tome changes & IP Dclone changes

For Tome - user don't need to start with Full tome anymore
For Dclone IP - user need to key in the reqion IP to target & the hot IP, once current game ip and hot ip matches the bot will wait for user to key in Enter to stop the bot

Will fork new for PR


Fix encoding problem

didn't test on python2, sorry.

Hi, what seems to be the problem you were trying to solve? afaik the README doesn't have any utf-8 characters in it.

Verbose output from each command is available in the environment’s build.log file, which is appended to with each operation. Any errors are logged to build.err.
I think ’ cause UnicodeDecodeError.
>>> open("README.rst").read()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
UnicodeDecodeError: 'cp949' codec can't decode byte 0xe2 in position 2831: illegal multibyte sequence

Nice catch! The apostrophe in environment is a utf-8 one not standard ANSI '
Will push a fix shortly.
On 12 Jul. 2017 08:42, "EcmaXp"<EMAIL_ADDRESS>wrote:

Verbose output from each command is available in the environment’s
build.log file, which is appended to with each operation. Any errors are
logged to build.err.
I think ’ cause UnicodeDecodeError.

open("README.rst").read()
Traceback (most recent call last):
File "", line 1, in UnicodeDecodeError: 'cp949' codec can't decode byte 0xe2 in position 2831: illegal multibyte sequence

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/sjkingo/virtualenv-api/pull/36#issuecomment-314592672,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAdFGKJ3Zzr24kOxYbwKzbZ37T4rznp6ks5sM_pagaJpZM4OSamL
.


PushConsumer

and use {@link ResourceManager#setConsumerProperties(String, KeyValue)} to modify.

ResourceManager#setConsumerProperties  not exist

Thanks for your report, we will resolve it soon~


[Specific PSXPC_N] Cant use Debugger in Visual Studio as opposed to normal Debug builds due to a room crash

I can play perfectly fine using MAIN.EXE after building Debug Win32 in VS but when I go to "Run Local Windows Debugger" I get room crashes where I definitely didn't before.
Step 1. run Debug MAIN.EXE
Step 2. Select The Colosseum
Step 3. Run forward to next room and watch it not crash
Step 4. run Run Live Windows Debugger in Visual Studio
Step 5. Select The Colosseum
Step 6. watch it assert on room code

Side note: Colosseum is inconsistent. Jumping in Trajan Market reproduces this far more reliably.

From Trajan jump


Remove nightly build

It's failing right now, and probably doesn't give us much extra value

Coverage remained the same at 66.126% when pulling d563c116a6d4c62d6509b9f036bf1091f8437317 on remove-nightly into 2ae89940028c242aeec6d35c03dff12b5f632bdc on develop.


Faster texture flipping + Disable some image formats

Maybe we could keep BMP for live texture editing with Paintbrush - the others are unlikely to be supported by JS
https://en.wikipedia.org/wiki/Comparison_of_web_browsers#Image_format_support

Do we get a significant speed-up by not using stb flipping api?

150ms on Z3


Fix joint rotations not working with XR SDK

Overview
Currently, there's a bug in the hand tracking implementation that's preventing hand joints from having a correct rotation. The if (x || y) statement short circuits the y call, and so rotation is never actually read. 😳

/azp run

😳😳😳😳😳😳😳😳😳😳😳😳😳😳😳😳😳😳😳😳😳😳😳😳😳😳😳😳😳

😳😳😳😳😳😳😳😳😳😳😳😳

😳😳😳😳

😳

/azp run

/azp run


LogCat upload

What
Debug logs for https://github.com/openfoodfacts/smooth-app/issues/3490
01-05 10:32:33.043  6163  7008 D MediaScannerConnection: Scanned /data/user/0/org.openfoodfacts.scanner/cache/86febcb0-4b12-4e9d-81ba-595ee57330918543385504186919683.jpg to null
01-05 10:32:33.084  6163  6163 E SurfaceSyncer: Failed to find sync for id=0

@monsieurtanuki do I need to build from source to see all logs ? Does the version from GPlay internal show all logs ?

Ok, I can't upload on product addition either @monsieurtanuki
https://world-fr.openfoodfacts.org/produit/3263854938111/uu-yu


Update guidance around --remote-connections-limit-enabled for private & public permissioned networks

Description
Presently in the docs we have some guidance around how to use --remote-connections-limit-enabled and --remote-connections-max-percentage in the case of public networks. Unfortunately, there isn't any guidance at present for private or public-permissioned networks.
Recently while running a node on a public permissioned network we observed that we weren't able to maintain a full peer list, in spite of our max-peers being set to a number well below the size of the network. On checking the logs, we found a lot of connection thrashing. This appeared to be happening at least in part because we weren't accepting new remotely initiated peer connections.
I think that most users are likely willing to accept having a peer list that isn't always full in return for resilience to eclipse attacks, but this trade-off isn't immediately apparent, which causes the instability of the peer count to seem like a bug. Furthermore, in our case it might have been a better approach to raise the value of --remote-connections-max-percentage from 60 to 80. Higher values still might also be acceptable, provided that we quickly disconnect remotely-initiated peer connections that don't have permission to join the network (I don't know whether we do this or not, but I'd assume that we do).
Acceptance Criteria

Documentation is updated to give guidance for setting the values of --remote-connections-limit-enabled and --remote-connections-max-percentage in the context of private, and public/permissioned networks.
Documentation sets users expectations for how these settings might impact their nodes' ability to fill their peer lists.
Documentation should explain that the value chosen for --remote-connections-max-percentage will affect how densely connected the network is, depending on the number of nodes in the network and the max peer limit set.

Versions (Add all that apply)

Software version: 1.4.5

Additional Information

Moving this to besu-docs repo

Issue moved to hyperledger/besu-docs #409 via ZenHub


Create navbar web component

I could be mistaken, but I think Raciel implemented this awhile ago.

Yeah, there are a lot of issues opened that have been resolved but not closed. I'll get around to closing them eventually


odysee.com is missing "Show mature content" option.

the lbry desktop app and lbry.ty have the option, but odysee.com is missing it.
Maybe this shouldn't be here, but I can't find odysee.com github page (maybe it is this page).

There's an issue filed on the lbry-desktop repo (which is the Odysee front end) for this. We'll probably link to the desktop app, as mature content (mainly pornography) is not allowed on Odysee.

as mature content (mainly pornography) is not allowed on Odysee.

😭😭😭 BUUUuuu 😭😭😭
(this message was not written to be taken seriously)


Example for working with Twilio for streaming?

Proposed changes

There's currently a python and a rust version it'd be nice if we could see a Node version.

Context

It would help me easily determine how to work with Twilio live transcriptions.

@cmaycumber Thanks for opening this issue. We've added a task to our backlog to get an example working with Node and Twilio.

I made a working example using Twillio and DeepGram with node here:

https://github.com/vinny-888/twilio-media-stream-live-transcription-node

I get the first utterance but the connection to DeepGram drops right after the first utterance for some reason.

@cmaycumber does the example @vinny-888  help you at all?


[Event Request] Codeunit 7322 Create Inventory Pick/Movement

Describe the request
The following event is required for this object:
    [IntegrationEvent(false, false)]
    local procedure OnBeforeNewWhseActivLineInsertFromJobPlanning(var WarehouseActivityLine: Record "Warehouse Activity Line"; var JobPlanningLine: Record "Job Planning Line"; var WarehouseActivityHeader: Record "Warehouse Activity Header"; var RemQtyToPickBase: Decimal)
    begin
    end;

Additional context
For all other sources this publisher is available

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. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 13909.


feat: remove tailscale extension from pre-installed packages (#891)

This could be installed by users later via GNOME extensions but feels really out of place in the default installation.

Tailscale is considered a major feature for Bluefin! Sorry this is the intended experience.  Here are the docs for that if you want to turn it off: https://universal-blue.discourse.group/docs?topic=166


Support workspace env

What does this PR do?
Adds support for workspaceEnv as in devfile 1.0 via the attribute workspaceEnv on components. Environment variables can be added to all containers in a DevWorkspace by specifying e.g.
components:
  - name: my-component
    attributes:
      workspaceEnv:
        TEST_ENV_1: TEST_VAL_1
        TEST_ENV_2: TEST_VAL_2

What issues does this PR fix or reference?
Closes https://github.com/devfile/devworkspace-operator/issues/530
Is it tested? How?
Test cases are added. Can be tested directly using the DevWorkspace
cat <<EOF | kubectl apply -f -
kind: DevWorkspace
apiVersion: workspace.devfile.io/v1alpha2
metadata:
  name: theia-next
spec:
  started: true
  template:
    attributes:
      controller.devfile.io/storage-type: "ephemeral"
    components:
      - name: tooling
        attributes:
          workspaceEnv:
            TEST_ENV_1: TEST_VAL_1
            TEST_ENV_2: TEST_VAL_2
        container:
          image: quay.io/wto/web-terminal-tooling
          args: ["tail", "-f", "/dev/null"]
      - name: theia
        plugin:
          uri: https://che-plugin-registry-main.surge.sh/v3/plugins/eclipse/che-theia/next/devfile.yaml

PR Checklist

[ ] E2E tests pass (when PR is ready, comment /test v7-devworkspaces-operator-e2e, v7-devworkspace-happy-path to trigger)

[ ] v7-devworkspaces-operator-e2e: DevWorkspace e2e test
[ ] v7-devworkspace-happy-path: DevWorkspace e2e test

This PR is a little half-baked since GitHub submitted it while I was writing the description :/

Needs to be reworked, will reopen once it's actually done right :wink:


TimeLine on the top

Hi! Is there any way to make time line on the top? Something like this:

Or can you provide this like a new feature?

Currently it is impossible, please create a feature request in https://github.com/tradingview/lightweight-charts for this.


ROX-17183: Migrate persistent data #1 - scanner definition

Description
Add:

Large objects utilities in addition to the Gorm
Add migration to move scanner definition to blobstore with test.

Checklist

[ ] Investigated and inspected CI test results
[x] Unit test and regression tests added
~[ ] Evaluated and added CHANGELOG entry if required~
~[ ] Determined and documented upgrade steps~
~[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)~

If any of these don't apply, please comment below.
Testing Performed

Add unit test
Run a migration test to add a scanner bundle and then upgrade and then verify.
Run a migration without scanner bundle and upgrade and then verify.

In addition to reviewing your code, reviewers must also review your testing
instructions and make sure they are sufficient.

The discussions around why use Gorm was documented here. https://docs.google.com/document/d/1OLMylIIoPvqD0yqOdF6fJGF4kSCPznPjTse1pBjdkJE/edit#heading=h.bupciudrwmna
The scenarios are the same. Migrator needs to have backward compatibility to earlier versions while the schema and utilities evolving. The target is to have the schema stable while we still have the freedom to innovate and optimize the central without keep looking back.
Due to lack of facilities and experience, we are still copying codes of stores in addition to the schema in migrations which is cumbersome which IMHO are all feasible with Gorm. What is missing here is to have enough examples and utilities to show people how to create gorm-style migration. We only need to have a frozen schema instead of the whole store.
In this migration, large object is not supported by Gorm but it is easy to add it. This can be used in the future if any other migration may need it.

The discussions around why use Gorm was documented here. https://docs.google.com/document/d/1OLMylIIoPvqD0yqOdF6fJGF4kSCPznPjTse1pBjdkJE/edit#heading=h.bupciudrwmna
The scenarios are the same. Migrator needs to have backward compatibility to earlier versions while the schema and utilities evolving. The target is to have the schema stable while we still have the freedom to innovate and optimize the central without keep looking back.
Due to lack of facilities and experience, we are still copying codes of stores in addition to the schema in migrations which is cumbersome which IMHO are all feasible with Gorm. What is missing here is to have enough examples and utilities to show people how to create gorm-style migration. We only need to have a frozen schema instead of the whole store.
In this migration, large object is not supported by Gorm but it is easy to add it. This can be used in the future if any other migration may need it.

If you intend to use this as an example of a Gorm migration, it might be worth updating the README and pointing this out as an example.

/test ocp-4-10-qa-e2e-tests
/test ocp-4-13-qa-e2e-tests
/test gke-nongroovy-e2e-tests


[Add] Yes & No button

Details
want to add yes and no ,when button turn of it will show cross ❌and button turn on it will show right ✔
Screenshots
No response
Type of Contribution

[ ] Update to an existing Animated Button
[X] Adding a new Button
[ ] Resolving a bug
[X] Proposal to the Repository
[ ] Changes related to documentation or README.md
[ ] Other Changes

Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue
[X] I am a GSSoC'23 contributor

hey...could you please assign me this ?


Support Azure azurerm_virtual_network (TF)

Description
Add Azure support for azurerm_virtual_network (TF)
Sources

Terraform Resource Documentation

LGTM!
0.16 detecting a manually created azurerm_virtual_network:
Found resources not covered by IaC:
  azurerm_virtual_network:
    - /subscriptions/7bfb2c5c-7308-46ed-8ae4-fffa356eb406/resourceGroups/driftctl-sj-tests/providers/Microsoft.Network/virtualNetworks/manual-network
Found 1 resource(s)
 - 0% coverage
 - 0 resource(s) managed by terraform
     - 0/0 resource(s) out of sync with Terraform state
 - 1 resource(s) not managed by Terraform
 - 0 resource(s) found in a Terraform state but missing on the cloud provider
Scan duration: 2s
Provider version used to scan: 2.75.0. Use --tf-provider-version to use another version.

0.16 working with an azurerm_virtual_network managed by terraform:
Found resources not covered by IaC:
  azurerm_virtual_network:
    - /subscriptions/7bfb2c5c-7308-46ed-8ae4-fffa356eb406/resourceGroups/driftctl-sj-tests/providers/Microsoft.Network/virtualNetworks/manual-network
Found 2 resource(s)
 - 50% coverage
 - 1 resource(s) managed by terraform
     - 0/1 resource(s) out of sync with Terraform state
 - 1 resource(s) not managed by Terraform
 - 0 resource(s) found in a Terraform state but missing on the cloud provider

Then also working with a virtual network created by a module:
Found resources not covered by IaC:
  azurerm_virtual_network:
    - /subscriptions/7bfb2c5c-7308-46ed-8ae4-fffa356eb406/resourceGroups/driftctl-sj-tests/providers/Microsoft.Network/virtualNetworks/manual-network
Found 3 resource(s)
 - 66% coverage
 - 2 resource(s) managed by terraform
     - 0/2 resource(s) out of sync with Terraform state
 - 1 resource(s) not managed by Terraform
 - 0 resource(s) found in a Terraform state but missing on the cloud provider


[in_app_purchase] moved android iap unit tests

Moved in_app_purchase unit tests.
As they were on the wrong place there was a misconception that they were missing.
Fixes flutter/flutter#78538 (as there were already quite some tests).
TextUtils and Logs had to be mocked as well to make the tests succeed in the new project.
Pre-launch Checklist

[X] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[X] I read the Tree Hygiene wiki page, which explains my responsibilities.
[X] I read and followed the Flutter Style Guide and the C++, Objective-C, Java style guides.
[X] I signed the CLA.
[X] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[X] I listed at least one issue that this PR fixes in the description above.
[ ] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[ ] I updated CHANGELOG.md to add a description of the change.
[X] I updated/added relevant documentation (doc comments with ///).
[X] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test exempt.
[X] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

all the log says is running java-test script/tool/lib/src/main.dart: Warning: Interpreting this as package URI, 'package:flutter_plugin_tools/src/main.dart'. All Java tests successful!
Is there a way to make sure the tests did actually run? Maybe try to modify a test to intentionally fail and see if ci fails?

@cyanglaz where is the location of the tests set for the CI? I couldn't really find it

@renefloor It's this https://cirrus-ci.com/task/6742052697276416

https://github.com/flutter/plugins/pull/3742 has landed. we should be able to rebase and run tests in this PR.

Yes I tried to merge this with master yesterday, but this PR makes that close to impossible: https://github.com/flutter/plugins/pull/3744
I'll just recreate this today.


python38Packages.scp: 0.13.6 -> 0.14.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python38Packages.scp is: "SCP module for paramiko"
meta.homepage for python38Packages.scp is: "https://github.com/jbardin/scp.py"
meta.changelog for python38Packages.scp is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 0.14.0 with grep in /nix/store/q0vf3xs9mwap4lr87drdk45i1hyn4ahs-python3.8-scp-0.14.0

found 0.14.0 in filename of file in /nix/store/q0vf3xs9mwap4lr87drdk45i1hyn4ahs-python3.8-scp-0.14.0

directory tree listing: https://gist.github.com/b0f5742915e961f32d1688672f672269

du listing: https://gist.github.com/8ecb872d519237e99e7c8211cd0c5347

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

68 total rebuild path(s)

17 package rebuild(s)

17 x86_64-linux rebuild(s)
17 i686-linux rebuild(s)
17 x86_64-darwin rebuild(s)
17 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
ansible
ansible-lint
ansible_2_10
ansible_2_11
azure-cli
python38Packages.ansible-base
python38Packages.ansible-core
python38Packages.ansible-lint
python38Packages.junos-eznc
python38Packages.robotframework-sshlibrary
python38Packages.scp
python39Packages.ansible-base
python39Packages.ansible-core
python39Packages.ansible-lint
python39Packages.junos-eznc
python39Packages.robotframework-sshlibrary
python39Packages.scp

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/q0vf3xs9mwap4lr87drdk45i1hyn4ahs-python3.8-scp-0.14.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/q0vf3xs9mwap4lr87drdk45i1hyn4ahs-python3.8-scp-0.14.0
ls -la /nix/store/q0vf3xs9mwap4lr87drdk45i1hyn4ahs-python3.8-scp-0.14.0/bin

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

2 packages failed to build:

azure-cli
python38Packages.ansible-lint

11 packages built:

ansible (ansible_2_11 ,python39Packages.ansible-core)
ansible-lint (python39Packages.ansible-lint)
ansible_2_10 (python39Packages.ansible-base)
python38Packages.ansible-base
python38Packages.ansible-core
python38Packages.junos-eznc
python38Packages.robotframework-sshlibrary
python38Packages.scp
python39Packages.junos-eznc
python39Packages.robotframework-sshlibrary
python39Packages.scp

Maintainer pings
cc @xnaveira for testing.

Result of nixpkgs-review pr 137119 at 92257f63 run on aarch64-linux 1

1 package failed to build:

azure-cli

12 packages built successfully:

ansible (ansible_2_11 ,python39Packages.ansible-core)
ansible-lint (python39Packages.ansible-lint)
ansible_2_10 (python39Packages.ansible-base)
python38Packages.ansible-base
python38Packages.ansible-core
python38Packages.ansible-lint
python38Packages.junos-eznc
python38Packages.robotframework-sshlibrary
python38Packages.scp
python39Packages.junos-eznc
python39Packages.robotframework-sshlibrary
python39Packages.scp

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement. Does something look off? Please file an issue or reach out on IRC.

On nixos x86_64:
builder for '/nix/store/dd9acwd7g8dgazy4jb35wxk7786130na-python3.9-azure-cli-2.26.1.drv' failed with exit code 1; last 10 log lines:
  Requirement already satisfied: azure-mgmt-hdinsight~=8.0.0 in /nix/store/0j386vchjivqh57jdfmivryrhcialqmm-python3.9-azure-mgmt-hdinsight-8.0.0/lib/python3.9/site-packages (from azure-cli==2.26.1) (8.0.0)
  Requirement already satisfied: azure-graphrbac~=0.60.0 in /nix/store/3asp89zc5mxsgmnkdb802i63r12rzb16-python3.9-azure-graphrbac-0.61.1/lib/python3.9/site-packages (from azure-cli==2.26.1) (0.60.0)
  Requirement already satisfied: azure-mgmt-consumption~=2.0 in /nix/store/23sv2w5gjyxbbq4yvpicyldyxryvj5pl-python3.9-azure-mgmt-consumption-8.0.0/lib/python3.9/site-packages (from azure-cli==2.26.1) (2.0.0)
  Requirement already satisfied: azure-synapse-accesscontrol~=0.5.0 in /nix/store/v6fscmgdihnaldha84y0s1f2pprf7507-python3.9-azure-synapse-accesscontrol-0.7.0/lib/python3.9/site-packages (from azure-cli==2.26.1) (0.5.0)
  Requirement already satisfied: azure-mgmt-authorization~=0.61.0 in /nix/store/lhx5ad4nyxiyw9pi9p9mnzvm3hfhj60z-python3.9-azure-mgmt-authorization-1.0.0/lib/python3.9/site-packages (from azure-cli==2.26.1) (0.61.0)
  Requirement already satisfied: azure-mgmt-sql~=0.29.0 in /nix/store/ml7hi152bnccqbbzddnwgrgzipxndpap-python3.9-azure-mgmt-sql-3.0.1/lib/python3.9/site-packages (from azure-cli==2.26.1) (0.29.0)
  Requirement already satisfied: azure-batch~=10.0.0 in /nix/store/jsdg5mww92pzvmjspa8wm0ixkm0kzdn8-python3.9-azure-batch-11.0.0/lib/python3.9/site-packages (from azure-cli==2.26.1) (10.0.0)
  Requirement already satisfied: azure-mgmt-trafficmanager~=0.51.0 in /nix/store/f2ppjk1mmqq3qblmf5slzkh8dhcvs60b-python3.9-azure-mgmt-trafficmanager-0.51.0/lib/python3.9/site-packages (from azure-cli==2.26.1) (0.51.0)
  ERROR: Could not find a version that satisfies the requirement scp~=0.13.2 (from azure-cli) (from versions: none)
  ERROR: No matching distribution found for scp~=0.13.2

@jonringer could you take care of azure-cli?

Yea, but traveling right now

I'll take care of this in https://github.com/NixOS/nixpkgs/pull/137461