How to train the translation model?

Hi,
I use the following command for model training.
mkdir /tmp/nmt_model
python -m nmt.nmt
--src=mn --tgt=zh
--vocab_prefix=/tmp/nmt_data/vocab
--train_prefix=/tmp/nmt_data/train
--dev_prefix=/tmp/nmt_data/test
--test_prefix=/tmp/nmt_data/test
--out_dir=/tmp/nmt_model
--num_train_steps=12000
--steps_per_stats=100
--num_layers=2
--num_units=128
--dropout=0.2
--metrics=bleu
However, the output error message：
ValueError:vocab_file '/tmp/nmt_data/vocab.mn 'does not exist.
I am newbie of nmt.I do not know what is the command that generates the source language (target language) vocabulary?
Looking forward to your advice or answers.
Best regards,
yapingzhao

Use this script:
build_vocab.zip
with the following command:
python path_to_script/build_vocab.py --data=path_to_corpus/corpus_name --save_vocab=save_path/vocab_file_name --size=50000
you can change the vocab size to anything you want, it will create a vocab with the first n most common words in the corpus and add the    tags

thank you very much!

Hi @yapingzhao .. were you able to solve this error? I am getting an Assertion Error if I use the build_vocab.py to generate the vocab


Add the song "Theivam Neerae"

Title: Theivam Neerae
Alternate Title: Theivam Neere
Author:
Album: Iraivaa

Not going to work on this now


Update error handling in AccountEntity.swift

This ticket is about to use of ZcashError. How to do it is described in #361.

Closed with #966


Update test dockerfile for testing for rotating keys

Docker images have been updated. Fixing should no longer be necessary.


How to know if one extension is not loaded correctly?

Related command
az alias xxx
Is your feature request related to a problem? Please describe.
Sometimes I succeeded to install an extension. But the extension is not correctly loaded. Check the commands in my side. The install did not fail but when I run the command az alias -h, it shows an warning and fail to load it.

Describe the solution you'd like
This issue is not asking to fix the broken extension. I need a way to know some extension is not correctly loaded. Such as a command az extension status to show information like this:
loading exteions:
account: success
acrtransfer: success
alias: fail

Describe alternatives you've considered
Maybe the installation can fail if extension can not be loaded.
Additional context
I am using this information to generate the Docs Page(https://docs.microsoft.com/en-us/cli/azure/alias?view=azure-cli-latest) . If there is possibility that an extension is installed successfully but fail to be loaded. Our Docs Pipeline may keep running without awareness, then publish the new content. That could finaly delete the commands releated to the extension.
Current work-around is, I am using az help and check the output to see if there is any warning like "Unable to load extension 'xxx"

Extensions


Is there a way to disable camelcase within function params?

I currently can't change the backend to allow the params to use camelcase and I'd rather not disable the warning about camelcase for everything.
Is there a way to disable the check just for the allowed_qualities and preferred_qualities?
$.get('config/general/saveAddShowDefaults', {
    allowed_qualities: anyQualArray.join(','),
    preferred_qualities: bestQualArray.join(',')
});

According to the official ESLint documentation for the camelcase rule, no.
However, you can add per-line ESLint ignore directives to your code.

I was hoping this project had possibly added something to work around that.
Guess I'll have to stick with ignore lines.

Ignore lines can be a good thing anyway; they clarify the intent of the code. :)
Feel free to close unless there is something else.

You can also turn off the rule globally or per file. See the rules option. Though I would recommend preferin inline ignore comments.

@sindresorhus that main reason I didn't want to use inline ignores is we use ajax and get requests quite a bit meaning most files contain 10 - 20 ignore lines just for those. If we could disable it just for certain functions like $.get that'd be a lot easier.


Header Group broken when Row Expand Indicator is enabled

Hi,
when using the Row Expand Indicator option in conjunction with the header grouping, the header columns end up being wrongly set up. Have a look at the minimal example broken code here with the typical products.
class BrokenTable extends React.Component {
  render() {
    return (
      <BootstrapTable data={products}
                            expandableRow={(r) => {return true;}}
                            expandComponent={(r) => {return (hello world);}}
                            expandColumnOptions={ { expandColumnVisible: true } }>
        <TableHeaderColumn row='0' rowSpan='2' dataField='id' isKey>ID</TableHeaderColumn>
        <TableHeaderColumn row='0' colSpan='2'>Product</TableHeaderColumn>
        <TableHeaderColumn row='1' dataField='name'>Name</TableHeaderColumn>
        <TableHeaderColumn row='1' dataField='price'>Price</TableHeaderColumn>
      </BootstrapTable>
    );
  }
}

const products = [];

function addProducts(quantity) {
  const startId = products.length;
  for (let i = 0; i < quantity; i++) {
    const id = startId + i;
    if (i < 3) {
      products.push({
        id: id,
        name: 'Item name ' + id,
        price: 2100 + i,
        expand: [ {
          fieldA: 'test1',
          fieldB: (i + 1) * 99,
          fieldC: (i + 1) * Math.random() * 100,
          fieldD: '123eedd' + i
        }, {
          fieldA: 'test2',
          fieldB: i * 99,
          fieldC: i * Math.random() * 100,
          fieldD: '123eedd' + i
        } ]
      });
    } else {
      products.push({
        id: id,
        name: 'Item name ' + id,
        price: 2100 + i
      });
    }
  }
}

addProducts(5);

And the resulting table looks as follows:

I'm actually having the same issue and was just about to post, is there a possible workaround?

I'll fix it in next version, thanks @PeterMerkert your good example to reproduce 👍

Fixed on v3.4.2, please have a look on it and let me know if the problem still remain. thanks

Just update my project to 3.4.2. Works smoothly! :+1:
Massive thanks for the quick fix @AllenFang !


Notifications won't load (very large list)

https://staging.civicdb.org/users/6/notifications
I have more than >16k notifications, which I don't anticipate to be common, but my notifications won't load.

We think that this is related to the filters since you have so many entities related to your notifications. We are planning on reducing the number of entities available to filtering to the top 10 or so, which should hopefully make this load faster (see #271).

This might actually be unrelated to #271 after all. In my own notifications I get a "Cannot return null for non-nullable field EventSubjectWithCount.subject" graphql query. I expect this is the same error KK might be encountering.

I just get a spinning wheel.

Errors in dev tools window

This should be working now, but let us know if you run into any other issues!

works now!


[Feature Request] Move script file to CRUD for cross-theme use

Everything here.

Awesome! Thanks @maurohmartinez . Heads-up, I've moved the general scripts and styles files from resources/views/crud/inc to resources/views/ui/inc. I think it makes more sense there because:

the files aren't really CRUD-specific, they're about the UI;
that way each file can individually override that file, if needed;

@tabacitu great! Anything else from needed here? Just making sure.

Nope, all good from my perspective. The only changes needed in order to merge all of these are in https://github.com/Laravel-Backpack/CRUD/pull/5075

Nope, all good from my perspective. The only changes needed in order to merge all of these are in Laravel-Backpack/CRUD#5075

@tabacitu I think I made all changes, should be fine by now :)


Tuple[(int, ...)] cannot be parsed on Python 2.7

Having a type comment like Tuple[(int, ...)] cannot be parsed on Python 2.7:
.../pytypes/typecomment_parser.py", line 225, in _funcsigtypesfromstring
    argTp = eval(argString, globals)
  File "<string>", line 1
    (Tuple[(int, ...)])

First note that Tuple[(int, ...)] is no valid Python 2.7 code and pytypes assumes that type comments contain code that is compatible with the currently running interpreter (try to process test_type = Tuple[(int, ...)] in ordinary Python 2.7 code). I thought this is in line with PEP 484, because they demand all imports and locals/globals (e.g. type variables) required by code in type comments. Type comments are like usual code, just in a special position.
That said, I would actually like this to work anyway. I'm thinking of some text-based preprocessing hack that allows this to work on Python 2.7. I cannot promise now that I will be able to come up with something totally reasonable. We'll see...
(So I will label this an enhancement rather than a bug...)

What would be an equivalent of Tuple[(int, ...)] in PYthon 2.7?

I'm not sure. It might depend on the version of typing. From typing 3.5.3 onwards it might be
Tuple[(int, Ellipsis)]

Before that it might not be possible as a one-liner:
t = Tuple[(int,)]
t.__tuple_use_ellipsis__ = True

I did not yet find time to investigate this in more detail, so this answer is sort of preliminary. We will need some version-bridging utility method in pytypes for this. The old variant is still relevant as it's what is bundled with Python 3.5.

Tuple[int, ...] seems to work perfectly fine in Python 2.7. Is there a special need for Tuple[(int, ...)] rater than Tuple[int, ...]?
Note that _issubclass_Tuple does not check regarding ellipsis case, but I am right now working to fix that, so ellipsis support for tuples should be available soon.
Given that Tuple[int, ...] works, I suppose we can close this issue... Thoughts?

Ellipsis support was established a while ago. Given that Tuple[int, ...] syntax works fine I'll close this issue now. We can reopen if a good reason is presented. I'm also open to accept PRs that tweak the syntax parsing in typecomment_parser to support this. However, I consider it low priority at best...

I also do not need it anymore.


feat(pkg/clusteragent/admission): enable automatic lib injection

What does this PR do?

Add functionality to inject the APM libraries into all Kubernetes deployments with a single opt-in deploy-time configuration option, apm.instrumentation_enabled. This feature is in beta.
Deployments can be opted out of this feature by reusing the admission.datadoghq.com/enabled label, setting it to false.
The Kubernetes deployment name is used as the APM service name by default if no other service names are provided on the pod in order to provide a good default experience to the user.
Existing features
Today, there is the "manual library injection" feature which relies on the following settings:

admission.datadoghq.com/enabled: pod label to opt into the cluster agent mutating the pod. Note that the library annotations (eg. admission.datadoghq.com/java-lib.version: v1.16.0) need to be added as well.
clusterAgent.admissionController.mutateUnlabelled: agent-level setting to eliminate the need for admission.datadoghq.com/enabled on every pod.

With this change
The apm.instrumentation_enabled feature now takes precedence over mutate_unlabeled.

[ ] Ignore namespace of the cluster agent itself to avoid circular issues
[ ] Allow library overrides via annotations (eg. when admission.datadoghq.com/java-lib.version: v1.16.0 is specified then that version of the java library should be injected, rather than the default with the instrumentation_enabled setting).

Motivation

Provide users with an easy-to-configure (single flag) way to instrument all their Kubernetes deployments.
Possible Drawbacks / Trade-offs

There is the potential for some collision/confusion with the existing admission_controller. mutate_unlabelled setting. mutate_unlabelled  was added to relax the constraint on local (annotation-based) injection requiring the admission.datadoghq.com/enabled: true label on every deployment.
The default is mutate_unlabelled: false which, by name, implies that mutations won't occur to unlabelled pods but this feature needs to override that setting as all pods will be mutated. This needs to be well documented to avoid confusion.
Additional Notes
The name apm.instrumentation_enabled is chosen to match what is being used to do a parallel feature in the Core Agent installer for Host & Docker opt-in automatic injection.
Describe how to test/QA your changes

Integration tests are added to assert that the flag enables injecting all the libraries and that any existing configuration is respected.
Integration testing with the tags.datadoghq.com/service label is not done here but will need to be done as well to ensure that setting is used over the default.
Richer integration tests will be added to https://github.com/datadog/system-tests to confirm the feature enables APM libraries and trace data is produced.
Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

Superseded by https://github.com/DataDog/datadog-agent/pull/19504, https://github.com/DataDog/datadog-agent/pull/19595 and https://github.com/DataDog/datadog-agent/pull/19714


Remove global options from ruff server subcommand

Currently, the ruff server subcommand includes the global options:
$ ruff server --help
Run the language server

Usage: ruff server [OPTIONS]

Options:
      --preview  Enable preview mode; required for regular operation
  -h, --help     Print help

Log levels:
  -v, --verbose  Enable verbose logging
  -q, --quiet    Print diagnostics, but nothing else
  -s, --silent   Disable all logging (but still exit with status code "1" upon detecting diagnostics)

Global options:
      --config <CONFIG_OPTION>  Either a path to a TOML configuration file (`pyproject.toml` or `ruff.toml`), or a TOML `<KEY> = <VALUE>` pair (such
                                as you might find in a `ruff.toml` configuration file) overriding a specific configuration option. Overrides of
                                individual settings using this option always take precedence over all configuration files, including configuration
                                files that were also specified using `--config`
      --isolated                Ignore all configuration files

The "Log levels" and "Global options" aren't required for the server as it's never used internally and is actually provided via the server initialization options.

This isn't possible today (https://github.com/clap-rs/clap/discussions/5578). It would require doing some major changes like moving the GlobalConfigVars to each subcommand struct, flattening it, and then updating all of the references to use the values from the subcommand struct. I don't think it's worth the effort right now to do this change.


bkt version incorrect in Cargo.lock

In 0.8.1, updating the version in the Cargo.lock file must have slipped through the cracks, now building with --locked fails.

Ugh, I've forgotten that before too 🙄 Fixed, and added --locked to the CI/CD steps.
Is it a bad idea to just bump the 0.8.1 tag to b7b07299, or should I cut a 0.8.2 to address it?

The package maintainer in me prefers cutting 0.8.2 to keep the tags clean


Design and Define Attributes for Order and Related Models

Goal
This issue focuses on defining the attributes and relationships for the Order, OrderItem in the project. The goal is to ensure a clear and consistent structure for handling order data within the system.
Class Diagram
The following class diagram visually represents the Order model and its relationships with OrderItem and Product. This design serves as the foundation for implementing the necessary attributes and associations as shown on the image above

Updated version:


SpiceyPy 2.0.0 N66 upgrade

Upgrade spiceypy to N66
expand build matrix on travis/appveyor to cover 2.7, 3.4-3.6
remove 32-bit builds

I am not really sure if 2.0.0 is the correct version increment for this, but it makes sense to increment major versions on new releases of spice

Coverage remained the same at 97.522% when pulling 467fc98d11c9c48748036933f15db9488d29a1d0 on n66_upgrade into 5e066109aac237133d6335322a951de6cad80ae5 on master.

Coverage remained the same at 97.522% when pulling 467fc98d11c9c48748036933f15db9488d29a1d0 on n66_upgrade into 5e066109aac237133d6335322a951de6cad80ae5 on master.

Coverage remained the same at 97.522% when pulling 467fc98d11c9c48748036933f15db9488d29a1d0 on n66_upgrade into 5e066109aac237133d6335322a951de6cad80ae5 on master.

Coverage remained the same at 97.522% when pulling 141cc1f13b0cbfb97cec27df01dcba78567852a5 on n66_upgrade into 5e066109aac237133d6335322a951de6cad80ae5 on master.

Coverage remained the same at 97.522% when pulling 141cc1f13b0cbfb97cec27df01dcba78567852a5 on n66_upgrade into 5e066109aac237133d6335322a951de6cad80ae5 on master.

Coverage increased (+0.004%) to 97.526% when pulling c5e5b31a557be224a985023f2723aac51261c1ff on n66_upgrade into 5e066109aac237133d6335322a951de6cad80ae5 on master.

Coverage increased (+0.004%) to 97.526% when pulling c5e5b31a557be224a985023f2723aac51261c1ff on n66_upgrade into 5e066109aac237133d6335322a951de6cad80ae5 on master.

Coverage increased (+0.004%) to 97.526% when pulling c5e5b31a557be224a985023f2723aac51261c1ff on n66_upgrade into 5e066109aac237133d6335322a951de6cad80ae5 on master.

Coverage decreased (-0.3%) to 97.252% when pulling f663237de111aa912bee70b1c2525920000daf4e on n66_upgrade into 5e066109aac237133d6335322a951de6cad80ae5 on master.

Coverage decreased (-0.3%) to 97.252% when pulling 2b2c151c517fd8b1f1104afe609db26b05b6f080 on n66_upgrade into 5e066109aac237133d6335322a951de6cad80ae5 on master.

Coverage decreased (-0.3%) to 97.252% when pulling 2b2c151c517fd8b1f1104afe609db26b05b6f080 on n66_upgrade into 5e066109aac237133d6335322a951de6cad80ae5 on master.

Coverage decreased (-0.3%) to 97.252% when pulling 9ed70dca60434d9fdbfa88102af9600dbc797d83 on n66_upgrade into 5e066109aac237133d6335322a951de6cad80ae5 on master.

Coverage decreased (-0.3%) to 97.252% when pulling b5c4f24fd30efca8f49d454d005a43f5d84e1a0a on n66_upgrade into 5e066109aac237133d6335322a951de6cad80ae5 on master.

Coverage decreased (-0.3%) to 97.252% when pulling cf0bfbfa78a295c65bfcf58a11e02afed45568fe on n66_upgrade into 5e066109aac237133d6335322a951de6cad80ae5 on master.

Coverage decreased (-0.3%) to 97.256% when pulling d99270ffbe88a816167ac51522e5c7197c53e0d4 on n66_upgrade into 5e066109aac237133d6335322a951de6cad80ae5 on master.

Coverage decreased (-0.3%) to 97.182% when pulling 1c5bf46304e012b11f678fc6bb77aebe79520fc6 on n66_upgrade into 5e066109aac237133d6335322a951de6cad80ae5 on master.

Coverage decreased (-0.005%) to 97.516% when pulling 80a9820a7a855dfe20de9ab418c14ca7c4b93bbe on n66_upgrade into 5e066109aac237133d6335322a951de6cad80ae5 on master.

Coverage increased (+0.003%) to 97.525% when pulling 600c5d092c6ff5346d617c7a98b20769a697697d on n66_upgrade into 5e066109aac237133d6335322a951de6cad80ae5 on master.

Coverage increased (+0.003%) to 97.525% when pulling 600c5d092c6ff5346d617c7a98b20769a697697d on n66_upgrade into 5e066109aac237133d6335322a951de6cad80ae5 on master.

Coverage increased (+0.003%) to 97.525% when pulling 600c5d092c6ff5346d617c7a98b20769a697697d on n66_upgrade into 5e066109aac237133d6335322a951de6cad80ae5 on master.

Coverage increased (+0.003%) to 97.525% when pulling 2341c68f9d94e9c50a601b36f5e978a0c5351888 on n66_upgrade into 5e066109aac237133d6335322a951de6cad80ae5 on master.

Coverage increased (+0.07%) to 97.588% when pulling f6a660ebbe372e7bd0a5426088d4eb8c686e9ffc on n66_upgrade into 5e066109aac237133d6335322a951de6cad80ae5 on master.

Coverage increased (+0.07%) to 97.592% when pulling 4b461e6f18cdbb6b1b9c94d412fbd506fde74afa on n66_upgrade into 5e066109aac237133d6335322a951de6cad80ae5 on master.

Coverage decreased (-0.3%) to 97.243% when pulling 84f0701a2b989d3d81a9b8f26efc903e666003c7 on n66_upgrade into 5e066109aac237133d6335322a951de6cad80ae5 on master.

Coverage increased (+0.3%) to 97.831% when pulling 330e71ca237b187b70232a2187adf87520815f9b on n66_upgrade into 5e066109aac237133d6335322a951de6cad80ae5 on master.

Coverage increased (+0.03%) to 97.552% when pulling 330e71ca237b187b70232a2187adf87520815f9b on n66_upgrade into 5e066109aac237133d6335322a951de6cad80ae5 on master.

Coverage increased (+0.3%) to 97.822% when pulling 84fddeb57c83da1d781c3091170a39de95bec119 on n66_upgrade into 5e066109aac237133d6335322a951de6cad80ae5 on master.

Coverage increased (+0.2%) to 97.673% when pulling c303d48f7393122c2607160563f50f37076a3d2a on n66_upgrade into d088ee7680426106715715bfdfc2b8b02f94bb0c on master.

Coverage decreased (-0.1%) to 97.374% when pulling 02cd23cfc9ca497986e551fc7417ddf1814198a9 on n66_upgrade into d088ee7680426106715715bfdfc2b8b02f94bb0c on master.

Coverage decreased (-0.2%) to 97.334% when pulling 64dee133e8f10313dcf8a07a8814d9850ab1e62d on n66_upgrade into d088ee7680426106715715bfdfc2b8b02f94bb0c on master.

Coverage decreased (-0.2%) to 97.343% when pulling 4b7c74e07be8948b2f3e4680ff24b2c8caf69c5f on n66_upgrade into d088ee7680426106715715bfdfc2b8b02f94bb0c on master.

Coverage decreased (-0.2%) to 97.343% when pulling df74ffc8bd447e2f2cd6dcd71ed4d50451ccb391 on n66_upgrade into d088ee7680426106715715bfdfc2b8b02f94bb0c on master.

Coverage decreased (-0.2%) to 97.343% when pulling df74ffc8bd447e2f2cd6dcd71ed4d50451ccb391 on n66_upgrade into d088ee7680426106715715bfdfc2b8b02f94bb0c on master.

Coverage decreased (-0.09%) to 97.432% when pulling 82206deee518c8b6804d3aec281dc997e358ba65 on n66_upgrade into d088ee7680426106715715bfdfc2b8b02f94bb0c on master.

Coverage decreased (-0.07%) to 97.451% when pulling ddc520c614e1aeff438a299c54dabb7b4667a15a on n66_upgrade into d088ee7680426106715715bfdfc2b8b02f94bb0c on master.

Coverage decreased (-0.07%) to 97.451% when pulling 40598734d7dada68acc110563e3b82e3c45be91f on n66_upgrade into d088ee7680426106715715bfdfc2b8b02f94bb0c on master.

Coverage increased (+0.3%) to 97.824% when pulling efe88fe67bffe22546d7f96b350cf3caf78b35e2 on n66_upgrade into d088ee7680426106715715bfdfc2b8b02f94bb0c on master.

Coverage increased (+0.3%) to 97.825% when pulling 00be9b585206d7cbf126c47874219eb1bbd149e0 on n66_upgrade into d088ee7680426106715715bfdfc2b8b02f94bb0c on master.

Coverage increased (+0.3%) to 97.824% when pulling 061a15b365556b0c906a8d7d02ec23c09ae726aa on n66_upgrade into d088ee7680426106715715bfdfc2b8b02f94bb0c on master.

Coverage increased (+0.3%) to 97.824% when pulling 56988ee5a33564de709d585d62e253c0c73f78aa on n66_upgrade into d088ee7680426106715715bfdfc2b8b02f94bb0c on master.


Update error code Python script

Currently we have this Python script that pulls in error codes from the prisma-engines repo and formats them for our Error Reference page.
At a minimum it would be nice to have a forked version of this script that is nicely commented with informative variable names, so it's easy to read and maintain.
There was also some discussion in this old PR (which I never finished, oops) about some information that is currently missing from the script and getting manually added, so if this is also easy to automate it would be good to include it.
Relevant old discussion: https://prisma-company.slack.com/archives/C5Z9TH6N9/p1646209993731929
Some instructions on running the script here

Yay for code comments. ;-)

Can we tweak the script to make it pull in PDP error codes, which are in private repo.

The error code script has been rewritten. Closing this issue.


Update sklearn-random-forest-example.ipynb

Add !pip installation

Codecov Report

Merging #152 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #152   +/-   ##
=======================================
  Coverage   93.23%   93.23%
=======================================
  Files          16       16
  Lines        1049     1049
  Branches      176      176
=======================================
  Hits          978      978
  Misses         42       42
  Partials       29       29

Flag
Coverage Δ

#unittests
93.23% <ø> (ø)

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 0bbd560...c644dd2. Read the comment docs.

When i tried to load this in Jupyter, I got
Unreadable Notebook:
/root/hummingbird/notebooks/sklearn-random-forest-example.ipynb
AttributeError('metadata')

so i'm adding the empty metadata tag to match the rest of the NB. will merge soon, thanks!

Thanks Karla! Apologies for the miss on that.
Sent from Joe's iPhone

On Jun 18, 2020, at 11:47 AM, Karla Saur<EMAIL_ADDRESS>wrote:

When i tried to load this in Jupyter, I got
Unreadable Notebook:
/root/hummingbird/notebooks/sklearn-random-forest-example.ipynb
AttributeError('metadata')
so i'm adding the empty metadata tag to match the rest of the NB. will merge soon, thanks!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Revise FAQ

Will BITNATIONS IDs allow me to cross borders.
I'd like to suggest that this part be edited.
Currently the BITNATION ID is compatible with EU regulations. It can and has been used to enable Stateless people access to private services. It is possible that as BITNATION gains significant market adoption and as it becomes the norm, more people will be able to use the BITNATION ID while traveling. The Pangea Jurisdiction brings governance choices to your home, (in your phone or on your computer!), regardless of what country you were born in, where you live, or what passport you hold. Furthermore, there are individuals, currently working as BITNATION citizens, ambassadors, and nations, to gain mass adoption of BITNATION IDs as well as documentation of property such as sailboats and automobiles.

Thanks, I'll update it. In the future please highlight the changed/ added or deleted words or sentences in bold or italic so it's easier to review.


Today Widget Route Directions

Expected behavior
At a glance look at Today widget bookmarks and know which stop is which.
Actual behavior
Have to click the stop to see which direction bus is traveling at that stop.
Steps to reproduce the behavior
Screenshot of the issue (if applicable)

If the direction (north or south in this case) were listed first rather than last, user can quickly discern the difference in the stops.
I'll note I do use larger text so perhaps that's why the direction gets truncated on the widget.
possibly related #1387

Hi Michael - thanks for the report! The today widget displays the custom name that you give each bookmark displayed in it. I recommend renaming these to be something like “NB route x to work” so that it’s clear to you which is which.

Where do you edit that?

Nvm found it.


debug piggyback does not detach when request is complete

I ran a single piggyback request, and now each push is sending the debug info again.
istiod shows: istio.io/debug/syncz: PUSH for node:shell-7854df9c5-c6rjd.default-2 resources:3 size:191B
proxy shows: tap response "istio.io/debug/syncz" arrived too late; discarding

I've started looking at this.
Investigating https://github.com/istio/istio/blob/master/pilot/pkg/xds/ads.go#L200 which was intended to make sure the debug requests never make it far enough to remember the debug request.


tf.train.match_filenames_once() under Windows

I use  tf.train.match_filenames_once() as follows:
tf.train.match_filenames_once("./*.py")
I mean , I want to get all the py file in current directory.
But, the result is I also get subdirectory py file under Windows.
How can I get the specified files only in current directory?
Thank you！
Have I written custom code : No
OS Platform and Distribution: Windows
TensorFlow installed from :pip command
TensorFlow version: 1.4
Bazel version: N/A
CUDA/cuDNN version: CUDA8.0
GPU model and memory:  GTX1080 8G
Exact command to reproduce： tf.train.match_filenames_once("./*.py")

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
TensorFlow installed from
TensorFlow version
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce

Thank you for your interest in TF. This question is better suited for StackOverflow, which the TensorFlow team also monitors under the TensorFlow tag. Github is strictly for bugs and feature requests.

Nagging Awaiting Response: It has been 14 days with no activityand the awaiting response label was assigned. Is this still an issue?

Nagging Awaiting Response: It has been 14 days with no activityand the awaiting response label was assigned. Is this still an issue?

Nagging Awaiting Response: It has been 14 days with no activityand the awaiting response label was assigned. Is this still an issue?

It has been 14 days with no activity and the awaiting response label was assigned. Is this still an issue?

It has been 14 days with no activity and the awaiting response label was assigned. Is this still an issue?

It has been 29 days with no activity and the awaiting response label was assigned. Is this still an issue?

It has been 152 days with no activity and the awaiting response label was assigned. Is this still an issue?


Build failure: lokinet

Steps To Reproduce
Steps to reproduce the behavior:

insert lokinet in configuration.nix file
sudo nix-channel --update
sudo nixos-rebuild switch --upgrade  --show-trace

Build log
/build/source/llarp/util/thread/queue_manager.cpp: In member function 'void llarp::thread::QueueManager::abortPushIndexReservation(uint32_t, uint32_t)':
/build/source/llarp/util/thread/queue_manager.cpp:491:33: error: 'popIndex' was not declared in this scope
  491 |       uint32_t loadedPopIndex = popIndex().load(std::memory_order_relaxed);
      |                                 ^~~~~~~~
/build/source/llarp/util/thread/queue_manager.cpp: In member function 'size_t llarp::thread::QueueManager::size() const':
/build/source/llarp/util/thread/queue_manager.cpp:509:56: error: 'pushIndex' was not declared in this scope
  509 |       uint32_t combinedPushIndex = discardDisabledFlag(pushIndex());
      |                                                        ^~~~~~~~~
/build/source/llarp/util/thread/queue_manager.cpp:510:35: error: 'popIndex' was not declared in this scope
  510 |       uint32_t combinedPopIndex = popIndex();
      |                                   ^~~~~~~~
/build/source/llarp/util/thread/queue_manager.cpp: In member function 'bool llarp::thread::QueueManager::enabled() const':
/build/source/llarp/util/thread/queue_manager.cpp:543:33: error: 'pushIndex' was not declared in this scope
  543 |       return !isDisabledFlagSet(pushIndex().load());
      |                                 ^~~~~~~~~
make[2]: *** [llarp/CMakeFiles/lokinet-util.dir/build.make:202: llarp/CMakeFiles/lokinet-util.dir/util/thread/queue_manager.cpp.o] Error 1
make[2]: *** Waiting for unfinished jobs....
[ 37%] Building CXX object external/oxen-mq/CMakeFiles/oxenmq.dir/oxenmq/proxy.cpp.o
[ 37%] Building CXX object external/oxen-mq/CMakeFiles/oxenmq.dir/oxenmq/worker.cpp.o
make[1]: *** [CMakeFiles/Makefile2:567: llarp/CMakeFiles/lokinet-util.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
[ 38%] Linking CXX shared library liboxenmq.so
[ 38%] Built target oxenmq
make: *** [Makefile:136: all] Error 2
error: builder for '/nix/store/aj291y7nzjkr6gwcscl23ayfg1vynvv7-lokinet-0.9.11.drv' failed with exit code 2

Notify maintainers

Metadata
nix-shell -p nix-info --run "nix-info -m"                                                                                255 ↵ ──(Sat,Feb03)─┘

system: "x86_64-linux"
host os: Linux 6.1.74, NixOS, 24.05 (Uakari), 24.05pre578631.b8b232ae7b8b
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.18.1
channels(root): "nixos, nixos-unstable, nixpkgs, nixpkgs-unstable"
channels(k): ""
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

Add a :+1: reaction to issues you find important.

https://github.com/NixOS/nixpkgs/pull/281069

Fixed in #287709


Add candidate websites

Adds candidate websites to cards

Everything looks good here, & this is a great addition! Except the absolute black links, pls change that to a grey from our palette.


doc issue about develop support

https://github.com/eth-cscs/stackinator/blob/5f161107f51c8ff9c1cf8989e9f5ccb5e0d7ba65/docs/configuring.md?plain=1#L45-L51
In addition to the repeated "the following", IMHO the bullet points list is not straightforward to understand. Some doubts I have:

it probably refers to spack env activate --with-view <name> and not spack env --with-view
first point: I would specify that the second part of the sentence refers to the environment configuration
second point: are you referring to <name> with "positional argument"?
third point sounds like a bit unlinked. I think doc meant that --with-view <name> applies starting from 0.21 (and so also develop included). right?
fourth point: are you referring to --develop? in case, I might argue that it would be beneficial to keep it there for a while. Two arguments in support of this argument:

what if you remove it and then you have to re-add a develop option because another similar workaround is needed?
develop sounds general enough, so it applies to all possible workarounds needed. otherwise we might end up having different flags for different workarounds of this kind and drop just the ones not used anymore

I really hope that spack will allows us to completely forget about this option sooner rather than later.

I agree that this could be better explained.

second point: are you referring to  with "positional argument"?

This is how I interpret it.

third point sounds like a bit unlinked. I think doc meant that --with-view <name> applies starting from 0.21 (and so also develop included). right?

I think the doc is describing https://github.com/eth-cscs/stackinator/pull/164. The spack_version is defined here
https://github.com/eth-cscs/stackinator/blob/5f161107f51c8ff9c1cf8989e9f5ccb5e0d7ba65/stackinator/recipe.py#L206-L222
So if you are not using 0.20 or 0.21, develop or unknown is returned.
The string comparison in #164 looks dangerous (it evaluates to True with both develop and unknown) and should be fixed, but what the doc is likely trying to convey is that if you are not using a specific version it's up to the user to trigger the condition with --develop.

fourth point: are you referring to --develop?

It should only refer to the specific spack env activate --with-view <name> workaround (see https://github.com/eth-cscs/stackinator/pull/152 for the removal of an old workaround, when it became unnecessary). --develop will need to stay around until 1.x, I think.
I don't recall if it's clearly stated in the documentation, but the idea is to support the last two minor versions of Spack, plus develop. So right now we should be able to work with v0.20, v0.21 and develop.


about window，how to control the position , hide, display, get focus

about window，how to control the position , hide, display, get focus

Nothing of that is supported.

Proton Native depends on libui, which doesn't have support for any of those.

but these features are important

Yes, they are. Proton-native is built on libui-node which is in turn a wrapper around libui. These features would have to be implemented in libui itself.
Window focussing is tracked in this issue (uiWindowRequestAttention): https://github.com/andlabs/libui/issues/309
Window position can't be easily implement cross platform: https://github.com/andlabs/libui/issues/351

If only on windows and macos?

The entire point of Proton Native is that it is cross platform. Once you start making stuff that only works on one platform, it gets very confusing and defeats the purpose. This all depends on how libui implements this and when they do.


Handle no date found in changelog

Change-type: patch
Signed-off-by: Amit Solanki<EMAIL_ADDRESS>

@balena-ci I self-certify!


Feature Request: Get parameter position

While you can retrieve GET parameters using req.query I'm not familiar with a method to retrieve a parameter's position.
Let me clarify the use case: I'm building a server that filters a database by given filter parameters, in the same order they are passed. Therefore I need to determine the arrangement of filter parameters.
You probably wouldn't like to add the position directly to the parameter, e.g. req.query.x.pos. Therefore my idea is to have a separate array with the positions, e.g. req.queryPosition.x.
Example:
/?test1=true&test2=false
req.queryPosition.test1
// => 0

req.queryPosition.test2
// => 1

To me this seems like something a third party library should do, not express core.  You have direct access to the full url already, so parsing the querystring in this way should be simple.

I'm doing this already, but it's uncomfortable

I'm doing this already, but it's uncomfortable

How so? The whole point of Express is to extend it, so if extending it is uncomfortable, we have failed you :) How can we make it more comfortable to extend Express functionality? Can you share what the code you are using is and elaborate on what aspects of it make you feel uncomfortable?

You can use regexp to simply detect '&' symbol.

I'm going to close this because it seems like the conversation has stalled, with no follow up on the questions to move forward.


Venue Maps

EIther interactive maps or pictures

@casassg @zakum42 @ani300 @AsLogd let's keep discussion here

Moved to separate repo -> https://github.com/hackupc/hackupc-maps


Add Python 3.11 Support

Adding python 3.11 support and testing. This resolves  #96

Codecov Report

Merging #98 (09a427a) into main (93fee3b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #98   +/-   ##
=======================================
  Coverage   82.23%   82.23%
=======================================
  Files          45       45
  Lines        1880     1880
=======================================
  Hits         1546     1546
  Misses        334      334

Flag
Coverage Δ

unittests
82.23% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


docs: Add MkDocs

chore: Delete sphinx-related configs
docs: Add basic mkdocs structure
docs: Add API reference in documentation
refactor: Remove griffe warnings
Bump version: 0.2.0 → 0.2.1

I've bumped the package version so doc changes show up on PyPI - I'll bump the PyPI version as soon as this gets merged

One thing that is still missing is the "Getting Started" section from the README, so I'm accepting any and all suggestions on that.


Add new user level that has access to the "admin only job creation"

During a conversation with @sejsong and @adswafford they brought up that it would probably be a good idea to have a new user level that could handle sequence processing requests. Otherwise, admins would have to handle these requests or new trainees would need to be given "admin" access to handle these requests.
We'll also probably want to rename the endpoints, templates, etc to not refer to "admin..." but that should be rather minor.

Thank you @ElDeveloper. Just to assess the priority of this feature request:  how many non-admin trainees users are you planning to add in the future? Perhaps having a list for the next: 3 months, 6 months, 1 year, 3 years would be helpful.

That's a good question. I think in 3 months we would expect to 3-5 users, 6 months 5-7 users? And generally not have many more people than that. Se Jin or Austin any expectations/projections?

On Sep 15, 2021, at 9:17 AM, Antonio Gonzalez @.***> wrote:
Thank you @ElDeveloper. Just to assess the priority of this feature request: how many non-admin trainees users are you planning to add in the future? Perhaps having a list for the next: 3 months, 6 months, 1 year, 3 years would be helpful.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

OK, thank you. Let's see what @sejsong and @adswafford reply. Also, it would be beneficial to know which exact commands the trainees would be able to run and which should not; for example, assigning and placing sequences into new studies should be admins (so they can access sandbox/private studies) but other commands might not.

Yoshiki and I will be going over the process next week since I missed code
review, and after that we should have a better idea which commands are I
needed to enable process of getting from a sample treatment sequences to a
biom table.
Number of users would be small (1-2) from my group, but if we can have the
same procedures you access granted to about the same number of individuals
and the other groups that closely collaborate with the night lab and
deposit their data on to barnacle post-sequencing from either the MiSeq or
IGM that should help future proof us.
On Wed, Sep 15, 2021, 10:26 AM Antonio Gonzalez @.***>
wrote:

OK, thank you. Let's see what @sejsong
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_sejsong&d=DwMCaQ&c=-35OiAkTchMrZOngvJPOeA&r=14FDY_dEWAszbl41dECf7h-ud-C0AZVTVBL_Sfs_BIc&m=qn-GWfwkCc4htlZ5Qd6p3OjdJw9baONAFAsIKiIwakQ&s=KWFWBRERgfFOzoO39DWiL_dauV5agA9erYT5qW2MHqc&e=
and @adswafford
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_adswafford&d=DwMCaQ&c=-35OiAkTchMrZOngvJPOeA&r=14FDY_dEWAszbl41dECf7h-ud-C0AZVTVBL_Sfs_BIc&m=qn-GWfwkCc4htlZ5Qd6p3OjdJw9baONAFAsIKiIwakQ&s=Ped1nSIm9GF-ujlJQ6cl4fJDfRKfDQO4JwbtMaUVcrg&e=
reply. Also, it would be beneficial to know which exact commands the
trainees would be able to run and which should not; for example, assigning
and placing sequences into new studies should be admins (so they can access
sandbox/private studies) but other commands might not.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_qiita-2Dspots_qiita_issues_3147-23issuecomment-2D920171081&d=DwMCaQ&c=-35OiAkTchMrZOngvJPOeA&r=14FDY_dEWAszbl41dECf7h-ud-C0AZVTVBL_Sfs_BIc&m=qn-GWfwkCc4htlZ5Qd6p3OjdJw9baONAFAsIKiIwakQ&s=ZavsTSNwxzyh10ZitAXuF1ToPv0cRoseI9Lf2nioLzU&e=,
or unsubscribe
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AGOEDBUUPZ3CE3P3Q3CGE2LUCDCKRANCNFSM5ECZ5KUA&d=DwMCaQ&c=-35OiAkTchMrZOngvJPOeA&r=14FDY_dEWAszbl41dECf7h-ud-C0AZVTVBL_Sfs_BIc&m=qn-GWfwkCc4htlZ5Qd6p3OjdJw9baONAFAsIKiIwakQ&s=lEIAU2vdp1u_YFebjXGDht_tB6VzcSoLOx0IDbT-KBg&e=
.
Triage notifications on the go with GitHub Mobile for iOS
https://urldefense.proofpoint.com/v2/url?u=https-3A__apps.apple.com_app_apple-2Dstore_id1477376905-3Fct-3Dnotification-2Demail-26mt-3D8-26pt-3D524675&d=DwMCaQ&c=-35OiAkTchMrZOngvJPOeA&r=14FDY_dEWAszbl41dECf7h-ud-C0AZVTVBL_Sfs_BIc&m=qn-GWfwkCc4htlZ5Qd6p3OjdJw9baONAFAsIKiIwakQ&s=2DCjIfoy6f5bBb1npnFNLszzvZCOQiAedCjlS4H-pFA&e=
or Android
https://urldefense.proofpoint.com/v2/url?u=https-3A__play.google.com_store_apps_details-3Fid-3Dcom.github.android-26referrer-3Dutm-5Fcampaign-253Dnotification-2Demail-2526utm-5Fmedium-253Demail-2526utm-5Fsource-253Dgithub&d=DwMCaQ&c=-35OiAkTchMrZOngvJPOeA&r=14FDY_dEWAszbl41dECf7h-ud-C0AZVTVBL_Sfs_BIc&m=qn-GWfwkCc4htlZ5Qd6p3OjdJw9baONAFAsIKiIwakQ&s=d_uT5e1_DkFEl2TYu4bxFgG3UVvMXgJVOU9YiWJqgTM&e=.

I just realized that this was done here: https://github.com/qiita-spots/qiita/pull/3157


fix: Mobile pop out menu hidden incorrectly

Closes #777
✅ Checklist

[x] I have followed every step in the contributing guide (updated 2022-10-06).
[x] The PR title follows the convention we established conventional-commit
[x] I performed a functional test on my final commit

Changelog
Changed the max-width property of the media-query from 767px to 768px

Screenshots

💯

Please remove the changeset, we only need it for CLI changes

So, I have to remove the changeset and push it again?

I'll look into it @juliusmarminge

The reason for the 767 value is that the breakpoint is at 768, so we want this behaviour until 1 pixel before that.

The reason for the 767 value is that the breakpoint is at 768, so we want this behaviour until 1 pixel before that.
Was this PR in response to a specific issue in the UI?

#777 but that issue seems very flaky

The issue was that at 767 px the popout wasn't being shown.
It was being shown at 766px and 768px but not at 767px

The reason for the 767 value is that the breakpoint is at 768, so we want this behaviour until 1 pixel before that.

Looks like the menu doesn't pop up only @ 767 x 767 dimensions. What if we set the max-width to 767.5 instead of 767 / 768?

It is actually working correctly when max-width is 767.5px.

https://user-images.githubusercontent.com/67912316/202848877-37a97b7b-add1-4195-ba38-43837167d2d8.mp4

Create.T3.App.-.Brave.2022-11-19.17-08-11.mp4

what you are showing here seems like the correct behaviour - the menu exists until 767 px, and doesn't exist at 768+ px because then the "Docs" etc items appear in the navbar instead

It is actually working correctly when max-width is 767.5px.

Can you please push the changes, so that we can get a live preview?

I still can't reproduce the original issue though, it works as expected with breakpoint on 768 for me on both Safari, Chrome and Firefox.

I still can't reproduce the original issue though, it works as expected with breakpoint going from 767 to 768 for me on both Safari, Chrome and Firefox.

I can reproduce the issue on brave on windows

I'm out currently, I'll push the changes in an hour.


[Tangram R&D] Tangram-carto fails with geometries conditionals

#layer['mapnik::geometry_type'=1] {
  marker-width: 7;
  marker-fill: #FFB927;
  marker-fill-opacity: 0.9;
  marker-line-color: #FFF;
  marker-line-width: 1;
  marker-line-opacity: 1;
  marker-placement: point;
  marker-type: ellipse;
  marker-allow-overlap: true;
}
#layer['mapnik::geometry_type'=2] {
  line-color: #3EBCAE;
  line-width: 1.5;
  line-opacity: 1;
}
#layer['mapnik::geometry_type'=3] {
  polygon-fill: #374C70;
  polygon-opacity: 0.9;
  polygon-gamma: 0.5;
  line-color: #FFF;
  line-width: 1;
  line-opacity: 0.5;
  line-comp-op: soft-light;
}

The above code fails because the ccss code include static properties under conditions.

Acceptance

Get the branch
yarn start
go to the url
paste this url in the input https://flopez2.carto.com/builder/2db72448-1867-11e7-bc6c-0e3ebc282e83/
click go
look that the points are loaded

Followed the instructions for acceptance and it seems to work as expected, so 👍


update conftest for backwards compat and new api for to parquet call

For cudf version 22.10.00+ the to_parquet api has changed. It no longer support the chunk_size call. It was replaced with row_group_size_rows, which can take minimum of 5000 rows. We have updated conftest call to reflect this change while maintaining backwards compatibility. We need this for all future versions.

For cudf version 22.10.00+ the to_parquet api has changed. It no longer support the chunk_size call. It was replaced with row_group_size_rows

Hmm - I may be mistaken, but I don't think chunksize was ever a supported argument to cudf.to_parquet. I suspect that older cudf versions were  just silently ignoring chunk_size=1000.


Implement #57 (Add ItemBuilder)

This is supposed to be a simple ItemBuilder suggested in #57 that isn't all too bloated with stuff you are not going to use.
I am not gonna add player skulls and custom skulls because the newer versions of Minecraft break that version inter compatibility completely and I don't want to get my hands dirty with reflection stuff.
TODO:

[ ] Add more stuff
[ ] Add JavaDoc
[ ] Test stuff

Any ideas on what also to add are welcome (but not too specific I'd say)

I think, item builder shouldn't be just 1 class, for example when a user want to create a banner, there should be a BannerItemBuilder class,

I would recommend you to that item builder library
https://github.com/portlek/BukkitItemBuilder


optgroup selected ignored

when using optgroup, if an element is selected before plugin init, the first <option> keep getting selected instead
example :
    <select tabindex="0">
      <option>Select with optgroup</option>
      <optgroup label="Group 1">
        <option>Option 1.1</option>
      </optgroup>
      <optgroup label="Group 2">
        <option selected="selected">Option 2.1</option>
        <option>Option 2.2</option>
      </optgroup>
      <optgroup label="Group 3" disabled="">
        <option disabled="">Option 3.1</option>
        <option disabled="">Option 3.2</option>
        <option disabled="">Option 3.3</option>
      </optgroup>
    </select>

this will show Select with optgroup as the selected item, with class="selected" on it.

This problem is caused by the selector for the selected option assuming the option is a direct child of the <select>, but with an optgroup it isn't a direct child. Furthermore, the code determined what the selected option is by determining its index. But this fails because it uses jQuery's .index() which returns the index of the option within the optgroup. I have a pull request that should be coming soon to fix this.

thanks


SPI big endian swap bug fix, spi_get_settings()/spi_set_settings()

Bugfix and some new functionality:
static const spi_settings_t my_settings = {
    .mode = SPI_MODE0,
    .freq_divider = SPI_FREQ_DIV_4M,
    .msb = true,
    .endianness = SPI_LITTLE_ENDIAN,
    .minimal_pins = true
};
....
spi_settings_t old;
spi_get_settings(1, &old); // save current settings
spi_set_settings(1, &my_settings);
// work with spi here
....
spi_set_settings(1, &old); // restore saved settings

This looks very nice, I like the idea of a settings data type. Just out of curiosity, what SPI hardware are you testing this with?
@foogod , I'm good to merge this if you are.

MFRC522 - 13.56 MHz RFID reader/writer
MAX7456 - TV OSD
logic analyzer

And I plan to make a PR with example of using MFRC522 over SPI, something like wifi door lock.


Is there a way to get the last part of the crumb

Getting the last part of the crumb only would be helpful for me

Hi @AirTechWick ! Great idea!


Run2 Sample List

List of samples: https://docs.google.com/spreadsheets/d/1C3wC3vG5VHEX0-bk-s6qdhGR0Hy7KcVLkG9efDYlN6Q/edit#gid=952387652

[ ] MiniAODv2 - 2016APV
[ ] MiniAODv2 - 2016
[ ] MiniAODv2 - 2017
[ ] MiniAODv2 - 2018
[ ] NanoAODv9 - 2016APV
[ ] NanoAODv9 - 2016
[ ] NanoAODv9 - 2017
[ ] NanoAODv9 - 2018
[ ] Legacy 2016

#34


Broken Link for RLS-Auto-Enable.sql

The document refers to tao the link and has a mention of the helper script.
https://techcommunity.microsoft.com/t5/sql-server/apply-row-level-security-to-all-tables-helper-script/ba-p/384360
The link to the helper script is broken in the attached Article.
http://rlssamples.codeplex.com/SourceControl/latest#RLS-Auto-Enable.sql
This script seems useful and would help a lot of Developers implementing RLS including me. I am still searching from the Script. So, can the link be updated to point to a correct location?
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: ab2082be-56d9-6438-0568-045af14348b2
Version Independent ID: 8d9779bb-e935-692f-39ef-8034a3cea327
Content: Multi-tenant apps with RLS and elastic database tools - Azure SQL Database
Content Source: articles/azure-sql/database/saas-tenancy-elastic-tools-multi-tenant-row-level-security.md
Service: sql-database
Sub-service: scenario
GitHub Login: @VanMSFT
Microsoft Alias: vanto

Hi @VanMSFT Can you please provide your inputs.

Hi @pdabhiargus - Samples from blogs are not something the Docs team handles, and we don't have a direct replacement of what is mentioned in the blog. Here's the closest that I can find regarding RLS in our samples repository:
https://github.com/microsoft/sql-server-samples/blob/master/samples/databases/wide-world-importers/sample-scripts/row-level-security/DemonstrateRLS.sql
You can check out other samples here: https://github.com/microsoft/sql-server-samples/find/master
#please-close

For anyone that might still look for it, the script can be found using archive.org in this link: https://web.archive.org/web/20210629135047/https://codeplexarchive.blob.core.windows.net/archive/projects/rlssamples/rlssamples.zip


Adds release notes for v4-preview5

I see now I never actually merged this. Resolved comments and merging now. Please create an issue if you have more suggestions!


[Enhancement] clarify expectation around schemaValidation flag and .spec.schemaValidation

Description
As far as I understand schemaValidation doesn't serve a purpose anymore. Both as a command flag and as a the .spec. schemaValidation field in the a policy.
You can still find reference about that field in the documentation that hints it is a valid workaround for solving problems related to missing CRD.
Example: https://kyverno.io/docs/writing-policies/tips/#validate.
And this workaround makes sense. As you might expect kyverno validation to be dynamically typed.
I certainly did.
This issue is a follow up to that thread: https://kubernetes.slack.com/archives/CLGR9BJU9/p1719931376972489
In there you can find the statement "That's correct, a CRD is always required.". I think this is a very clear statement that I wish would be directly available in the documentation.
recommendation:

state clearly in the documentation that CRD is always required
cleanup references to schemaValidation that are not about deprecation.

Slack discussion
https://kubernetes.slack.com/archives/CLGR9BJU9/p1719931376972489

/assign


Maryland Statewide Centerlines

Maryland DoIT has split their centerlines into several different feature classes by type.

[x] Interstate see #8
[ ] US Routes
[ ] MD Routes
[ ] UMD Roads (University of Maryland roads)
[ ] Local and Other Roads

#10 addresses the rest of these

closed with #12


Group chatting Functionality

Can you Add group chatting functionality this project.

Thanks for the feature suggestion @rajkumar73443!
We are considering adding a chat feature to the app. We're still discussing it, so I don't think the feature will be added until 2021.
In the meantime, I wrote a blog post about how to use DataTracks to add simple chat functionality to this app. You can see the blog post here: https://www.twilio.com/blog/open-source-video-app-reactjs-datatrack-snack-chat
And you can see the code that the blog post references here: #317
Note: The "Snack Chat" feature that is described in the blog post uses DataTracks to send chat messages to other participants in a room. The messages sent over DataTracks are ephemeral. This means that you will be unable to see any messages that were sent before you joined the room. Next year, we will likely use the Twilio Conversations API to add more robust chat functionality that will allow users to see messages that were send before they enter a room.

We are considering this feature, but it's not currently on our roadmap. I will close for now.
Thank you!


Setup maven publishing plugin

For publishing to a maven artifact repository, setting up the publishing plugin is a required step.

Maven Publish Plugin
Using the maven publishing plugin
Publishing overview
Sonatype Publishing

Closing after #5 was merged.


Encountered an error while adding the hunt_scanner.py and hunt_methodology.py in Burp

Attached files are error logs while adding hunt_scanner.py and hunt_methodology.py. I also attached an image of burp extensions options.

hunt_methodology_error.txt
huntscanner_error.txt
Thank you!

@bladeism I'm not seeing any attachments. Mind posting them again?

I am also getting error as in the screenshot attached

@swagnetow Done updating it. Thanks!

@bladeism Try unsetting "Folder for loading modules".

@swagnetow Same error sir.
Burp Suite Pro 1.7.16
Jython -stand alone 2.7.0
Win 7 SP1 64bit

Burp Suite Pro 1.7.12
Jython -stand alone 2.7.0
Win 7 64
@swagnetow      Being given

Got it working by entirely copying the hunt folder contents inside the folder where burp is installed.  I think the error is because of os.getcwd(retrieving the current directory in "os.getcwd() + os.sep + "conf" + os.sep + "checklist.json". os,getcwd is giving the path where burp is installed.

@connectlazyguy I'll try this when I got home. Thanks!

@swagnetow   According to your approach, still did not solve the problem.

I'm not sure why os.getcwd() is not getting the current working directory of where the HUNT files are for you guys. I just tried the plugin as it currently is in master in Windows 10 and was able to run it fine in Burp Suite. I'll investigate further.
Windows 10
Burp Suite v1.7.26
Jython Standalone 2.7.0

I don't have an error regarding issues.json
but got this instead
UnicodeDecodeError: 'utf-8' codec can't decode bytes in position 526-528: invalid data

@bladeism Mind creating a separate issue with more details so I can debug? Thanks.

@bladeism Do you mind creating another issue for your Unicode issue? Looks like there were two separate issues here and everything got jumbled up.

@swagnetow I am also getting the same error.

Same error.
java version 1.8.0_141
Burp Suite Pro 1.7.13
Jython Standalone 2.7.0

same here
Traceback (most recent call last):
File "/root/Burp Suite Professional 1.7.17/HUNT/hunt_methodology.py", line 41, in run
self.runner()
File "/root/Burp Suite Professional 1.7.17/HUNT/hunt_methodology.py", line 47, in init
self.view = View()
File "/root/Burp Suite Professional 1.7.17/HUNT/hunt_methodology.py", line 207, in init
self.data = Data()
File "/root/Burp Suite Professional 1.7.17/HUNT/hunt_methodology.py", line 169, in init
self.set_checklist(None)
File "/root/Burp Suite Professional 1.7.17/HUNT/hunt_methodology.py", line 176, in set_checklist
file_name = os.getcwd() + os.sep + "conf" + os.sep + "checklist.json"
IOError: [Errno 2] No such file or directory: '/root/conf/checklist.json'
Traceback (most recent call last):
File "/root/Burp Suite Professional 1.7.17/HUNT/hunt_methodology.py", line 47, in init
self.view = View()
File "/root/Burp Suite Professional 1.7.17/HUNT/hunt_methodology.py", line 207, in init
self.data = Data()
File "/root/Burp Suite Professional 1.7.17/HUNT/hunt_methodology.py", line 169, in init
self.set_checklist(None)
File "/root/Burp Suite Professional 1.7.17/HUNT/hunt_methodology.py", line 176, in set_checklist
file_name = os.getcwd() + os.sep + "conf" + os.sep + "checklist.json"
IOError: (2, 'No such file or directory', '/root/conf/checklist.json')
at org.python.core.Py.IOError(Py.java:226)
at org.python.core.io.FileIO.<init>(FileIO.java:102)
at org.python.core.PyFile.file___init__(PyFile.java:177)
at org.python.core.PyFile$exposed___new__.createOfType(Unknown Source)
at org.python.core.PyOverridableNew.new_impl(PyOverridableNew.java:12)
at org.python.core.PyType.invokeNew(PyType.java:494)
at org.python.core.PyType.type___call__(PyType.java:1706)
at org.python.core.PyType.__call__(PyType.java:1696)
at org.python.core.OpenFunction.__call__(__builtin__.java:1725)
at org.python.core.PyObject.__call__(PyObject.java:461)
at org.python.core.PyObject.__call__(PyObject.java:465)
at org.python.pycode._pyx1.set_checklist$21(/root/Burp Suite Professional 1.7.17/HUNT/hunt_methodology.py:180)
at org.python.pycode._pyx1.call_function(/root/Burp Suite Professional 1.7.17/HUNT/hunt_methodology.py)
at org.python.core.PyTableCode.call(PyTableCode.java:167)
at org.python.core.PyBaseCode.call(PyBaseCode.java:153)
at org.python.core.PyFunction.__call__(PyFunction.java:423)
at org.python.core.PyMethod.__call__(PyMethod.java:141)
at org.python.pycode._pyx1.__init__$20(/root/Burp Suite Professional 1.7.17/HUNT/hunt_methodology.py:170)
at org.python.pycode._pyx1.call_function(/root/Burp Suite Professional 1.7.17/HUNT/hunt_methodology.py)
at org.python.core.PyTableCode.call(PyTableCode.java:167)
at org.python.core.PyBaseCode.call(PyBaseCode.java:307)
at org.python.core.PyBaseCode.call(PyBaseCode.java:198)
at org.python.core.PyFunction.__call__(PyFunction.java:482)
at org.python.core.PyFunction.__call__(PyFunction.java:476)
at org.python.core.PyInstance.__init__(PyInstance.java:124)
at org.python.core.PyClass.__call__(PyClass.java:193)
at org.python.core.PyObject.__call__(PyObject.java:445)
at org.python.core.PyObject.__call__(PyObject.java:449)
at org.python.pycode._pyx1.__init__$28(/root/Burp Suite Professional 1.7.17/HUNT/hunt_methodology.py:216)
at org.python.pycode._pyx1.call_function(/root/Burp Suite Professional 1.7.17/HUNT/hunt_methodology.py)
at org.python.core.PyTableCode.call(PyTableCode.java:167)
at org.python.core.PyBaseCode.call(PyBaseCode.java:307)
at org.python.core.PyBaseCode.call(PyBaseCode.java:198)
at org.python.core.PyFunction.__call__(PyFunction.java:482)
at org.python.core.PyFunction.__call__(PyFunction.java:476)
at org.python.core.PyInstance.__init__(PyInstance.java:124)
at org.python.core.PyClass.__call__(PyClass.java:193)
at org.python.core.PyObject.__call__(PyObject.java:445)
at org.python.core.PyObject.__call__(PyObject.java:449)
at org.python.pycode._pyx1.__init__$5(/root/Burp Suite Professional 1.7.17/HUNT/hunt_methodology.py:47)
at org.python.pycode._pyx1.call_function(/root/Burp Suite Professional 1.7.17/HUNT/hunt_methodology.py)
at org.python.core.PyTableCode.call(PyTableCode.java:167)
at org.python.core.PyBaseCode.call(PyBaseCode.java:307)
at org.python.core.PyBaseCode.call(PyBaseCode.java:198)
at org.python.core.PyFunction.__call__(PyFunction.java:482)
at org.python.core.PyMethod.instancemethod___call__(PyMethod.java:237)
at org.python.core.PyMethod.__call__(PyMethod.java:228)
at org.python.core.PyMethod.__call__(PyMethod.java:223)
at org.python.core.Deriveds.dispatch__init__(Deriveds.java:19)
at org.python.core.PyObjectDerived.dispatch__init__(PyObjectDerived.java:1112)
at org.python.core.PyType.type___call__(PyType.java:1713)
at org.python.core.PyType.__call__(PyType.java:1696)
at org.python.core.PyObject.__call__(PyObject.java:445)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:498)
at burp.m1d.<init>(Unknown Source)
at burp.llg.a(Unknown Source)
at burp.lub.run(Unknown Source)
at java.lang.Thread.run(Thread.java:748)

solved it...just moved my conf file to root folder ..and it worked...great work keep it up

@sonahri     Is moved to the root directory of HUNT?

check line 10...IOError: [Errno 2] No such file or directory: '/root/conf/checklist.json'
Traceback (most recent call last):
just moved conf folder to root...and it worked

Anyone solved the unicode error?

i solve this issue by copy "conf" directory in HUNT master, to where burp working directory / installed.


fix: emit current folder

When selecting the current folder in the location picker, an empty array has been emitted. We've fixed this by emitting the current folder.
Short note: this is our first feature with integration tests 🥳

fixes #39

Changes look good, I would've loved to have both possibilities in emitSelectedResources() covered by test cases but not strictly necessary 😉

Read this too late. Yes, even more test coverage would be awesome of course :-)

I would've loved to have both possibilities in emitSelectedResources() covered by test cases but not strictly necessary

Location variant will be fully covered via integration tests here #46


Rollup of backup commits

What problem does this PR solve?
Problem Summary: Cherry pick recent bug fixes about backup & restore
Check List
Tests

Unit test
Integration test

Release note

Support AWS IAM web identity.

1d44cf3  has a typo in the commit message and failing the DCO.

/run-all-tests

typo still not fixed in c6a9ff6.

/run-all-tests

/run-unit-test

/merge


Initial test API

Mentoring @spences10 and @amandeepmittal by adding some tests to this project for an API using TDD.

@MarcL Thanks!

@amandeepmittal - I've refactored it a bit and I'll add some more of the API + tests to show you a bit more if you like. Will try and get it done one night this week.

Thanks @MarcL 👍

Take your time @MarcL  👍


Added function to clone a repository recursively

It is similar to git clone --recursive. The current implementation is very
simple and limited as it does not allow to customize the fetch and checkout
tasks and submodules are processed sequentially (i.e. no option like --jobs
to process them concurrently).
Addressing the limitations described above requires some thought:

FetchOptions and CheckoutBuilder are not Send.
Should different submodules have different options?
Are new options and callbacks needed?

I ended up creating repositories with recursive submodules and thought a simple implementation was better than none for now, especially for build scripts. RepoBuilder can be extended in the future to allow customizing the recursive cloning operation.
Note: One improvement I may be able to do now to this PR is updating submodules concurrently. I would need to add a dependency on rayon or crossbeam to git2. Just let me know if you want me to do it.
Thank you.

Thanks! Out of curiosity, is it safe to do this concurrently? I'd imagine that may muck with internal libgit2 state that wouldn't be safe to happen concurrently.
Also mind keeping the update_submodules method private for now?

To do it concurrently, the idea would be to get Repository objects from Submodule objects using git_submodule_repo_init and process these repositories concurrently. My understanding is that separate Repository objects are safe to use concurrently (libgit2 thread safety).
The reason I made update_submodules public is that it's actually useful in build scripts. Let's say you have a sys crate with a submodule for the C lib and the build script is in charge of updating the submodule. In this case, when the build script runs, the repository has already been cloned and the script just needs to update the submodules of this repo (similarly to the libgit2-sys crate whose build script updates its submodule). The build script could just get the Submodule object and update it (assuming there's only one direct submodule), but the repo the submodule points to may itself contain submodules, now or in the future, and the build script should not need to know. I am working on a crate that has just this scenario: Rust crate repo has C lib repo as a submodule and the C lib repo has a submodule of its own.
Please let me know if the purpose is still unclear. How do you want me to proceed?

Given the above explanations, would you agree that it is useful to have update_submodules public too? If not, do you have another design in mind for the use case I described?
Do you want me to look into updating submodules concurrently? It helps in the case of more than one submodule because fetching is done in parallel.

Thank you.

I'm not sure if it's clear from my previous message: Repository::update_submodules is roughly equivalent to git submodule update --init --recursive, which I feel is useful to update all the submodules of an already-cloned repository recursively, for example in build scripts that update and build a foreign submodule (which could have its own submodules).

Thanks for the explanation! Makes sense to me. For now I'd prefer to keep this conservative as it doesn't expose all the flavors of options for how to update submodules. I figured if we added parallel support (which I'm sort of hesitant to do) we'd want to expose all the options instead of hardcoding the fetch options. Having a helper though as a constructor on Repository seems fine to me!

I made update_submodules private as requested. Let me know if there's anything else you would like me to do.

Thanks!


xmlrpc assignNextUnassignedForState: filter on set propertyFilters

This probably can be solved a bit cleaner or maybe even the in-SQL filter stuff could be fixed, but for now this works and is deployed on the c3voc instance...

FTR: This broke the recording-scheduler script, because Record.StartedBefore did not match anymore.


Potential for a discrepancy in hash and size when artifact is store with dvc

In dvc branch both .mlem/model/rf.dvc and .mlem/model/rf.mlem have hash and size for the same artifact. This feels fragile. One might update one of this independently, say *.dvc file:
cd .mlem/model
dvc pull
<edit> rf
dvc add rf
dvc push
git add rf.dvc && git commit ...

What do you thing @aguschin?

I moved this issue from example-mlem-get-started repo here. The problem you mention @Suor definitely exists.

I think we could issue a warning when MLEM loads a model, but binary have a hash different to what's in .mlem file (and maybe we already do, need to check this).
We could add --check-hash-and-fail-if-it-changed flag to MLEM commands or something with a shorter name

Do you maybe have some ideas also?


a bogus "remoteRoot" should not result in file not found errors

use a simple "hello world" node.js program
create a launch config for it and set a breakpoint
add a bogus remoteRoot path to the launch config, e.g. "remoteRoot": "/foobarxxxxxxx"
press F5

Observe:
this error notification shows up:

A bogus "remoteRoot" path should not result in this type of error because a non-matching "remoteRoot" path should be ignored.
This problem does not occur with node-debug (legacy).
BTW, the value of ${workspaceFolder} is /Users/weinand/MS/Projects/REPROS/bug64332

It seems that node-debug still applies the bogus remoteRoot, it just returns the file as readonly. With legacy if I set the bogus remoteRoot then I can't set BPs in that file but I do hit a debugger statement in a readonly file. The code between the two DAs is basically the same.
https://github.com/Microsoft/vscode-node-debug/blob/master/src/node/nodeDebug.ts#L3959 I think will return a path starting with ...

node-debug will still have this issue


Solve Daily Problem: Maximum Distance in Array

624. Maximum Distance in Arrays
Medium
You are given m arrays, where each array is sorted in ascending order.
You can pick up two integers from two different arrays (each array picks one) and calculate the distance. We define the distance between two integers a and b to be their absolute difference |a - b|.
Return the maximum distance.
Example 1:
Input: arrays = [[1,2,3],[4,5],[1,2,3]]
Output: 4
Explanation: One way to reach the maximum distance 4 is to pick 1 in the first or third array and pick 5 in the second array.
Example 2:

Input: arrays = [[1],[1]]
Output: 0

Constraints:

m == arrays.length
2 <= m <= 105
1 <= arrays[i].length <= 500
-104 <= arrays[i][j] <= 104
arrays[i] is sorted in ascending order.
There will be at most 105 integers in all the arrays.

Solved:
class Solution:
  def maxDistance(self, arrays: list[list[int]]) -> int:
    ans = 0
    mn = 10000
    mx = -10000

    for A in arrays:
      ans = max(ans, A[-1] - mn, mx - A[0])
      mn = min(mn, A[0])
      mx = max(mx, A[-1])

    return ans


Avoid order by [guid column] statement after Include()

Is it somehow possible to avoid the order by statement after including an entity relation?
It hurts the performance for entities with a guid id column and sorting on a guid doesn't make much sense.
Code:
var result = await DbSet.Where(tk => tk.App.Any(a => uuids.Contains(a.AppUuid)))
                        .Include(tk => tk.Translations)
                        .ToListAsync();

Sql:
 SELECT [tk.Translations].[uuid], [tk.Translations].[value]
      FROM [dbo].[translations] AS [tk.Translations]
      INNER JOIN (
          SELECT [tk0].[uuid]
          FROM [dbo].[keys] AS [tk0]
          WHERE EXISTS (
              SELECT 1
              FROM [dbo].[app_keys] AS [a0]
              WHERE [a0].[app_uuid] IN ('uuid') AND ([tk0].[uuid] = [a0].[keys_uuid]))
      ) AS [t] ON [tk.Translations].[keys_uuid] = [t].[uuid]
      ORDER BY [t].[uuid]

In the query plan 80% is used for the sort/order by.

@peterbomers The order-by is required by EF to correctly populate the included collections while streaming in the results from the query.

@ajcvickers How so? EF6 didn't do this AFAIK. Is EF Core now using a sort of "Stream Aggregate"?
Doing an "Hash Aggregate" blocks entities from being reclaimed early but not specifying AsNoTracking() causes this anyway and when calling ToListAsync() this is wasted effort.
I'm just going to quote

I jokingly say that SQL Server is the world’s second most expensive place to sort data – second only to Oracle. Next time someone complains about the $7K per core cost of SQL Server Enterprise Edition, remind them that Oracle is $47K per core. $47K. Hoo boy.
If you need to sort data, try doing it in the app tier instead. Developers are really good about scaling out application work, and their app servers don’t cost $7K per core.

Note for triage: Tested the code below (simple collection Include) with EF6 and generated SQL was:
SELECT
    [Project1].[Id] AS [Id],
    [Project1].[C1] AS [C1],
    [Project1].[Id1] AS [Id1],
    [Project1].[BlogId] AS [BlogId]
    FROM ( SELECT
        [Extent1].[Id] AS [Id],
        [Extent2].[Id] AS [Id1],
        [Extent2].[BlogId] AS [BlogId],
        CASE WHEN ([Extent2].[Id] IS NULL) THEN CAST(NULL AS int) ELSE 1 END AS [C1]
        FROM  [dbo].[Blogs] AS [Extent1]
        LEFT OUTER JOIN [dbo].[Posts] AS [Extent2] ON [Extent1].[Id] = [Extent2].[BlogId]
    )  AS [Project1]
    ORDER BY [Project1].[Id] ASC, [Project1].[C1] ASC

The same LINQ query in EF Core generates:
      SELECT [b].[Id], [p].[Id], [p].[BlogId]
      FROM [Blogs] AS [b]
      LEFT JOIN [Posts] AS [p] ON [b].[Id] = [p].[BlogId]
      ORDER BY [b].[Id], [p].[Id]

EF6 code:
public class Blog
{
    public int Id { get; set; }

    public ICollection<Post> Posts { get; set; }
}

public class Post
{
    public int Id { get; set; }

    public int? BlogId { get; set; }
    public Blog Blog { get; set; }
}

public class BloggingContext : DbContext
{
    static BloggingContext()
    {
        Database.SetInitializer(new DropCreateDatabaseAlways<BloggingContext>());
    }

    public BloggingContext()
        : base(@"Server=(localdb)\mssqllocaldb;Database=Test;ConnectRetryCount=0")
    {
    }

    public DbSet<Blog> Blogs { get; set; }
    public DbSet<Post> Posts { get; set; }
}

public class Program
{
    public static async Task Main()
    {
        using (var context = new BloggingContext())
        {
            context.Blogs.Add(new Blog
            {
                Posts = new List<Post>
                {
                    new Post(),
                    new Post()
                }
            });

            await context.SaveChangesAsync();
        }

        using (var context = new BloggingContext())
        {
            context.Database.Log = Console.Write;

            var blogs = context.Blogs.Include(e => e.Posts).ToList();
        }
    }
}

Such a beautiful SQL. 😍


Refactor Chassis structures

As oneview-redfish-toolkit developer, I should maintain common behavior for all Chassis variants in a single place and still be able to code specific behavior.

Lucas please join this with PR for issue #62

OK @ffdarkpenguin


Feature request: Ability to search for lyrics from track context menu

Hi Jacques,
I just found your foobar component openlyrics and must say: I am very impressed. Keep up the excellent work, plz!!
The plugin seems very comprehensible, there is just one thing I miss: there is no way to add a shortcut enabling users to start a search. This would be especially useful for batch processing a big numer of files unattendedly.
Is there a chance to implement this?
For what its worth: I just donated two books on your buymeacoffee site.
:)
Best to you and this great project!!!

I'm glad you find it helpful, and your support is much appreciated!
For supporting batch lyric download: In what way would it be helpful to download many lyrics at once in a batch? They wouldn't be shown until you played the actual track and they'd be automatically searched/retrieved at that point anyway.

Hey Jacques, thanks for getting back.
My setup is: large library in foobar (mostly lossless) and several lossy libraries on altogether 5 android devices. These lossy libraries change a lot. After having used foobar mobile for a while, I switched to Musicolet, because

this player can display lyrics
it is one of the very few programs that work completely well without any internet connection
in general, I prefer programs to not make use of the internet unless explicitly needed / wanted by me and then I always go for software that lets me decide in which way I expose myself to the digital world

I keep my files well tagged and have decided on going the tag based lyrics route as this makes it far easier handling the main and lossy libraries while still having all tag info in one place.
Apart from this more organisational aspect: lyrics for me sometimes take quite a while to be downloaded. This is another reason why I would very much prefer to do the lyrics searching unattendedly. Inside foobar, I would set up a filter (CUI) to see which files do not yet have lyrics and batch run openlyrics on them.

Just to summarise and check that I understand correctly: You use another media player (Musicolet) doesn't support retrieving lyrics and you want to use openlyrics as a way to quickly attach lyrics to many files such that they can be displayed in Musicolet?

I understand the main purpose of you component. I was just asking if there was any way to use it like it was needed for my purpose. All that is needed, really would be exposing the command to execute inside the shortcut setup menu, just like
view->openlyrics panel.
foobar is my main player. I use it for everything: tagging, replaygain and moving stuff to the right place.
In general: I make sure the files are in proper order, including lyrics tags.
Your plugin works great when it comes to fetching lyrics and displaying them. Unfortunately, every time it searches the internet, I wait between 5 and 10 seconds until the servers have been queried. This makes it a very slow process to make sure all files have lyrics as every file would need to be played for at least 10 seconds.
With the mobile music players I prefer to remain online (as with all apps that do not directly need internet such as browser or mail), that is why I used foobar and (due to a lack of features) I currently use Musicolet.
The scenario here is pretty straightforward: I convert the lossless files to aac, preserving all tags and copying folder art, then I copy the lossy library to the device.
So, the ideal scenario would be that the main library in foobar would have all the lyrics already properly tagged as then I would never be filling the mobile device with any tracks that does not contain lyrics.

Ok fair. I can add a context-menu item to download lyrics for non-playing tracks (as LSP3 does).  Then it'd give you a progress bar and say "we've done 12/25 tracks, current track is ArtistName - TrackTitle" or something.
Two things to note though:

The download for each track would not be any faster than the usual download though. Sure it'd be much faster than leaving the entire playlist to play (in which case each track would take several minutes rather than 10 seconds) but it would obviously take at least as long as each download does serially. Probably a bit longer since I'd want to space it out a little and avoid hammering servers when asked to download many lyrics at once (so make it ensure that it doesn't move faster than 1 track every 10 or 15 seconds or something, I dunno).
This would be following the same process as the existing auto-search (so no ability to modify search params for each track, it searches sources in the order you configured and saves the first result it finds if its new, etc).

Would that cover your use-case?

That sounds absolutely fantastic!
I am fine with a delay for the servers, the main thing is to not have to do all this by hand clicking tracks I do - at that moment - not want to hear. Also, the search parameters need not be altered.
So essentially: YES, that would very nicely cover my use case.
THANK YOU!

@jacquesh I noticed that this is closed but for some reason i can't find the interface for bulk search. Could you please give me a pointer to this?

Hey Jacques,
THANKS so much for addressing my request. I was on holiday when you released it and the mail notification was washed away during my absence. Only now I realised the update was out.
GREAT. Thanks again.
But I have the same problem as Reapor-Yurnero:
I cannot find any bulk-search option. I looked everywhere in context menu,  otpions and also in the context submenu of options to make sure I do not oversee anything.
Did we miss something nonetheles?

EDIT: HECK. Found it
"Search for lyrics" on context menu when more than one file is selected.
I don't know for the life of me why I did not find it before ...
THANKS

@rk-kontur It should show up when only one file is selected as well :)
What do you mean by "sort of gets very lagging?" I know the progress bar doesn't animate smoothly (I'm not sure why, but tbh I wasn't too fussed at the time, I'd like to fix that at some point but eh) and there is intentionally a several-second-long gap between searches for files, but I don't get any actual slowdown (or even a jump in the amount of work being done) on my end. A gif or something of what you're seeing would be useful.
Please create a new issue for that though. That would be a bug, whereas this ticket is a request for a feature :)


Recorded video is not playable

Bug Information
Version Number of Plugin: <IP_ADDRESS>/<IP_ADDRESS> Beta
Device Tested On: Huawei P9 (Android 7.0)
Version of VS: VS 2017 Version 15.7.5
Version of Xamarin: <IP_ADDRESS>7442-pre1
Steps to reproduce the Behavior
Use  CrossMedia.Current.TakeVideoAsync to take a video, after that in the preview options try to open the video and it says video can't be played
Expected Behavior
Video could be played after it has been recorded
Actual Behavior
Video can't be played after it can't be recorded
Code snippet
await CrossMedia.Current.Initialize();
        if (!CrossMedia.Current.IsCameraAvailable || !CrossMedia.Current.IsTakeVideoSupported)
        {
            // DisplayAlert("No Camera", ":( No camera available.", "OK");
            return;
        }

        var file = await CrossMedia.Current.TakeVideoAsync(new StoreVideoOptions
        {

            CompressionQuality = 92,
            SaveMetaData = true,
            DefaultCamera = CameraDevice.Rear
        });

Screenshots
Screenshot of the playback error (native video view with custom renderer, also tested with other apps):

Screenshot of the video codecs

I think there might be an issue with the video codecs, for example with vlc player the video can be played, with the device video app it can't. With the device camera app there is no error.
I hope anybody could help me with this issue. Thanks in advance.

I have the same issue but on Iphone X. The video is playing on vlc media player but on others show codec error


Add text description for maps

Detailed Description
We've had a couple of clients now ask for the ability to have text descriptions associated with each map. This would give people the ability to keep notes about their map, talk about the context in which they created the district, and in general give a longer description of the map than they can with a title. It would also be helpful if they were submitting public testimony about the map to use these notes and aid in organization administrators in understanding the map.

Designs here: https://app.abstract.com/projects/9f988240-1dad-11e8-9a40-cba930331af7/branches/master/commits/latest/files/DCA38482-FC7F-4F48-BECD-44E07D9F7443/layers/1C440CF8-1FE6-4B51-8A94-C46E5AE848D9?collectionId=b88037d0-f61b-4451-a280-66968a028d8e&collectionLayerId=a9249f60-1b7e-4558-8b0b-4e2729d616ec

@jfrankl is there a place we show the description for a map that isn't your own? It looks like the wireframes show editing it, but not how to see it

@maurizi we don't currently have designs for the description being displayed outside of the input field itself. @dmcglone I don't have a clear user story for this feature, do you have any insights?


Download collections in the background, JS part

(JS part to avoid saying ...background, frontend part)
Since #549 the collections download has a new step for downloading all thumbnails. The backend reports "completed" to the frontend when these tasks are queued (not finished) so the thumbnails download continues in the background. But the frontend doesn't have a way to know when the thumbnails are available, so the frontend doesn't refresh. We want to also move the extra metadata stage to the background #650 .
The simplest way would be to monitor the background tasks from the root component and then reload the page. But this has been noted as not very user-friendly in the past. The user might be reading or playing content when suddenly the page is reloaded. So we need a bit of design thinking to communicate when this happens and let the user confirm the page refresh.

I'm deassigning myself from this task.


fix: update example name

Description
Replace this text with a summary of the changes in this PR. Include why the changes are needed and context about the changes. List required dependencies. If there is a Git issue for the change, please link to it.
Types of changes in this PR
No release required

[x] Examples or tests (addition or updates of examples or tests)
[ ] Documentation update
[ ] CI-related update (pipeline, etc.)
[ ] Other changes that don't affect Terraform code

Release required

[ ] Bug fix (patch release (x.x.X): Change that fixes an issue and is compatible with earlier versions)
[ ] New feature (minor release (x.X.x): Change that adds functionality and is compatible with earlier versions)
[ ] Breaking change (major release (X.x.x): Change that is likely incompatible with previous versions)

Release notes content
Replace this text with information that users need to know about the bug fixes, features, and breaking changes. This information helps the merger write the commit message that is published in the release notes for the module.

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

Merge actions for mergers

Merge by using "Squash and merge".

Use a relevant conventional commit message that is based on the PR contents and any release notes provided by the PR author.
The commit message determines whether a new version of the module is needed, and if so, which semver increment to use (major, minor, or patch).

@Aashiq-J Can you run git submodule update --remote --merge to pull in latest git submodule (renovate currently disabled due to ICD issues). This should resolve the failing pipeline

/run pipeline

:tada: This PR is included in version 3.3.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Module Loader fails with --experimental-import-meta-resolve

Describe the bug
In a runtime environment with --experimental-import-meta-resolve enabled, the module loader fails and does not resolve correctly.
To Reproduce
Steps to reproduce the behavior:

Create a program which uses import.meta.resolve to resolve a module.
Start the programm with node --experimental-import-meta-resolve --import ts-blank-space/register ./path/to/program.ts.

The program/loader fails with the following error:
> node --import ts-blank-space/register  --experimental-import-meta-resolve ./src/index.ts "./src/index.ts"

node:internal/modules/run_main:123
    triggerUncaughtException(
    ^
TypeError [Error]: Invalid URL
    at Object.getPackageScopeConfig (node:internal/modules/package_json_reader:132:33)
    at packageResolve (node:internal/modules/esm/resolve:792:43)
    at moduleResolve (node:internal/modules/esm/resolve:908:18)
    at defaultResolve (node:internal/modules/esm/resolve:1038:11)
    at nextResolve (node:internal/modules/esm/hooks:748:28)
    at resolve (file:///xxx/node_modules/.pnpm/ts-blank-space@0.4.1/node_modules/ts-blank-space/loader/hooks.js:7:22)
    at nextResolve (node:internal/modules/esm/hooks:748:28)
    at Hooks.resolve (node:internal/modules/esm/hooks:240:30)
    at MessagePort.handleMessage (node:internal/modules/esm/worker:199:24)
    at [nodejs.internal.kHybridDispatch] (node:internal/event_target:816:20) {
  code: 'ERR_INVALID_URL',
  input: 'input.ts'
}

Node.js v22.8.0

Expected behavior
It should be able to resolve the required module.
Environment (please complete the following information):

Windows with WSL2 (Debian)
Node Version 22.8.0

Without --experimental-import-meta-resolve it does work, but not resolves relative to the parent.  That is expected and not an issue with the loader.


[WIP] Allow login via http basic

I might do the public/private thing in another PR, but with faraday-panoptes I'm closing this PR since it's now just about as easy to just use OAuth's client_credentials flow.


@AllArgsConstructor with generic type not compiling with 13.1.5 intellij

Lombok version: org.projectlombok:lombok:1.14.8
Intellij Plugin: 0.8.8
Intellij: 13.1.5
Java: 8
The following doesn't compile
@AllArgsConstructor
private static final class Entity<K, V> {
  private final K key;
  V value;
}

new Entity<>(key, value)

Error:(58, 15) java: cannot infer type arguments for com.github.dcapwell.collect.mutable.HashMap.Entity<>
  reason: cannot infer type-variable(s) K,V
    (actual and formal argument lists differ in length)

When I build from gradle, works just fine.

Unfortunally I can't reproduce it right now too as the class i needed this combination is now removed and replaced.

Ok, I will close this as invalid. Feel free to reopen if you find another example.


Fix: Long display name not being wrapped

Details

Fixes an issue where our display name wasn't respecting the size of it's container.
Fixed Issues

#3172
Tests

Wrote a long display name (first+last name, or email)
It was trimmed correctly on the chat and on the IOU details modal

QA Steps

Choose a long display name (first+last name, or email)
It should be trimmed correctly on the chat and on the IOU details modal

Tested On

[x] Web
[x] Mobile Web
[x] Desktop
[x] iOS
[x] Android

Screenshots

Web

https://user-images.githubusercontent.com/53711423/124042762-f36eee80-d9df-11eb-80f8-b11877de815b.mov
Mobile Web

https://user-images.githubusercontent.com/53711423/124042766-f5d14880-d9df-11eb-93fe-4a820a2f9c1f.mp4
Desktop

https://user-images.githubusercontent.com/53711423/124042770-f8cc3900-d9df-11eb-96b5-d6a976206a61.mov
iOS

https://user-images.githubusercontent.com/53711423/124042779-fbc72980-d9df-11eb-827b-e3633ead5bc4.mp4
Android

https://user-images.githubusercontent.com/53711423/124042787-fff34700-d9df-11eb-9338-3a23191d2428.mp4

Yeah I agree @Luke9389. This solution will most likely go out of date as soon as we start adding other kinds of report actions to NewDot. Maybe better to pass this in as a prop to ReportActionItemFragment for now and then conditionally set the single line. Then apply it here where we are building the "person" text:
https://github.com/Expensify/Expensify.cash/blob/b330b3da015703a0e3ceacfb3c946bf565019752/src/pages/home/report/ReportActionItemSingle.js#L67-L75

@Luke9389  @marcaaron It's a good point! We would be restricting too much the use cases of our TEXT fragment.
This commit keeps the containerStyle in our ToolTip (we still don't want the content of our TEXT to go beyond the container), and changes the numberOfLines to be a conditional.
I thought it might be better to make the isSingleLine boolean a property of the fragment object instead of a prop of our ReportActionItemFragment, since it's only really relevant for our TEXT and doesn't change the behavior of our component.

OK, these changes look good @rdjuric 👍
Just want to double-check, did you test the most recent commits on each platform? (I know it's a hassle but regressions have been sprouting up a lot lately).

@Luke9389
Yes! Updated the PR with videos testing the most recent commit on all platforms.

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

Just found an regression https://expensify.slack.com/archives/C01GTK53T8Q/p1625174167354500

Thanks, @parasharrajat. Looking at it now.

Unfortunately the deploy comments are not working right now. This was deployed to staging yesterday.


Content within code blocks under list items treated as errors

Hello,
Thanks for these utils!
Code blocks within list items are reporting what I think are false positives.
The problem
Indented code blocks under list items are processed as errors:
1. Run this script in your browser to prevent 'AAA' masking from effecting your local session:

    ```javascript
    window._uxa = window._uxa || [];
    window._uxa.push(["trackConsentGranted"]);
    ```

1. Do something else.

    ```javascript
    console.log('ok');
    ```

Inconsistent list item capitalization: "w" should be "W" eslint(remark-lint-list-item-style)
Inconsistent list item capitalization: "c" should be "C" eslint(remark-lint-list-item-style)
My .remarkrc.js file:

module.exports = {
  settings: {
    "bullet": "-",
    "incrementListMarker": false,
  },
  plugins: [
    "remark-frontmatter",
    // https://github.com/remarkjs/remark-lint/tree/main/packages/remark-preset-lint-markdown-style-guide#rules
    "remark-preset-lint-markdown-style-guide",
    ["remark-lint-first-heading-level", 2],
    ["remark-lint-no-duplicate-headings", false],
    ["remark-heading-gap", 1],
    ["remark-lint-maximum-heading-length", false],
    ["remark-lint-list-item-indent", 'space'],
    ["remark-lint-list-item-content-indent", false],
    ["remark-lint-list-item-spacing", false],
    ["remark-lint-list-item-style", {
      "checkFirstWord": "capitalize",
      "checkPunctuation": false,
    }],
    ["remark-lint-fenced-code-flag", false],
    ["remark-lint-maximum-line-length", false],
    ["remark-lint-emphasis-marker", false],
    ["remark-lint-no-emphasis-as-heading", false],
  ],
};

Nice catch! I'll check this out later today.

@fviolette Should be patched in the latest version. Let me know if the problem persists.

:tada: This issue has been resolved in version 1.0.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket: