Query inventory without internet connection

It's not possible to offline query inventory (without sku details). querySkuDetails variable (line 249 in file  gdx-pay/gdx-pay-android-openiab/src/com/badlogic/gdx/pay/android/openiab/PurchaseManagerAndroidOpenIAB.java ) is always set up to true. According to this comment http://stackoverflow.com/questions/14231859/how-to-best-save-inapp-purchase-status-locally#comment41999027_15569448 it could be fixed by changing querySkuDetails to false. I know that this variable cannot always be false, but maybe it could be parametrized earlier.

Oh. I have set querySkuDetails in one of the latest pull requests to true,
because otherwise we did have any prices. But your case is also important.
I guess we should make it with two requests, you are right. The first one
with "false", then call observer methods, then if the first call was
successful, call queryInventory with "true" value.
You can make a pull request with the fix ;)
3 авг. 2015 г. 19:19 пользователь "Berobasket"<EMAIL_ADDRESS>написал:

It's not possible to offline query inventory (without sku details).
querySkuDetails variable is always set up to true. According to this
comment
http://stackoverflow.com/questions/14231859/how-to-best-save-inapp-purchase-status-locally#comment41999027_15569448
it could be fixed by changing querySkuDetails to false. I know that this
variable cannot always be false, but maybe it could be parametrized earlier.
—
Reply to this email directly or view it on GitHub
https://github.com/libgdx/gdx-pay/issues/49.

What do you think about this idea: put querySkuDetails variable into PurchaseManagerConfig and user will decide what he want. User could query inventory without prices (querySkuDetails variable set to false) for the first time and then he could manualy query once again (with changing querySkuDetails in PurchaseManagerConfig to true). I don't need more detailed information about product and I don't know if querying list of products two times (always without user need) will be goog idea.

I'm holidays currently, it's hard to take a look, sorry. But I think in
this case we will break the basic idea of config: setup once. So it will
make the API not so clear for the user.
I would better go for something like PurchaseConfig.autoFetchInfo variable.
It is better to set it to true by default, because it could be not clear
for the user, why there is PurchaseSystem.getInformation method, which is
always return null.
Additionally we could have a method PurchaseSystem.fetchInformation() (or
with other name), which fetches info for all skus.
What does @noblemaster think?
4 авг. 2015 г. 7:51 пользователь "Berobasket"<EMAIL_ADDRESS>написал:

What do you think about this idea: put querySkuDetails variable into
PurchaseManagerConfig and user will decide what he want. User could query
inventory without prices (querySkuDetails variable set to false) for the
first time and then he could manualy query once again (with changing
querySkuDetails in PurchaseManagerConfig to true). I don't need more
detailed information about product and I don't know if querying list of
products two times (always without user need) will be goog idea.
—
Reply to this email directly or view it on GitHub
https://github.com/libgdx/gdx-pay/issues/49#issuecomment-127499806.

I agree with alex-dorokhov to keep it simple.
A) As suggested by alex-dorokhov, we could add a "autoFetchInformation" option to PurchaseManagerConfig. In that sense we would fetch all the information during install for people that prefer that. I would not add a fetchInformation() method though. Having it in install(...) seems the more straightforward solution!?
B) Depending on the platform, it might be a lot of information to fetch? It's not just Android, but also includes iOS, Desktop markets, you name it. As far as I know no images are currently fetched. But some stores might add that in the future. In order to reduce memory consumption we could modify the getInformation(identifier) method and instead use a callback such as:
void getInformation(Callback<Information> information, String identifier).
In that sense we deprecate the current getInformation(identifier) method and instruct everyone to use the new method with callback.
Comments? Other ideas/suggestions?

You're right. @noblemaster could you add this in the near future? For me more important is to first fetch inventory with querySkuDetails set to false (method getInformation() could be changed later).

@noblemaster sounds good to me.
I also had in mind the idea from point B.
4 авг. 2015 г. 9:10 пользователь "Christoph Aschwanden" <
<EMAIL_ADDRESS>написал:

I agree with alex-dorokhov to keep it simple.
A) As suggested by alex-dorokhov, we could add a "autoFetchInformation"
option to PurchaseManagerConfig. In that sense we would fetch all the
information during install for people that prefer that. I would not add a
fetchInformation() method though. Having it in install(...) seems the more
straightforward solution!?
B) Depending on the platform, it might be a lot of information to fetch?
It's not just Android, but also includes iOS, Desktop markets, you name it.
As far as I know no images are currently fetched. But some stores might add
that in the future. In order to reduce memory consumption we could modify
the getInformation(identifier) method and instead use a callback such as:
void getInformation(Callback information, String identifier).
In that sense we deprecate the current getInformation(identifier) method
and instruct everyone to use the new method with callback.
Comments? Other ideas/suggestions?
—
Reply to this email directly or view it on GitHub
https://github.com/libgdx/gdx-pay/issues/49#issuecomment-127515050.

OK, going with point B (using a callback for getInformation(...)) and not querying the SKU-details during install(...) makes the most sense.
Also, I don't use the getInformation. I would definitely be in favor of not querying the SKU-details during install as well! Would someone mind submitting a pull request? Sorry, I am little bit busy at the moment to implement this myself.

Sorry, I have thought more and tried to go with option B, but it requires to many changes and also complexifies the usage of the library. It was much easier to go with option A.
I would suggest to do not overcomplicated getInformation method without real need.

Oops, will do the pull request without the conflict

Fair enough!


parse parameterized Type failed

I use asn1c to  generate C code from an asn file, but I got such warning and errors:
FATAL: Type ScmsPdu-Scoped expects specialization from ScmsPdu-Scoped at line 33 in ./test3.asn
FATAL: Cannot find type "<Type>" in constraints at line 38 in ./test3.asn
FATAL: Type ScmsPdu-Scoped expects specialization from ScmsPdu-Scoped at line 33 in ./test3.asn
FATAL: Cannot find type "<Type>" in constraints at line 38 in ./test3.asn
FATAL: Type ScmsPdu-Scoped expects specialization from ScmsPdu-Scoped at line 33 in ./test3.asn
FATAL: Cannot find type "<Type>" in constraints at line 38 in ./test3.asn

And these is my asn content:
TEST3
DEFINITIONS AUTOMATIC TAGS ::= BEGIN

  Uint8  ::= INTEGER (0..255)                  -- (hex)

  RaAcaCertRequest ::= OCTET STRING(SIZE(32))
  AcaRaCertResponse ::= OCTET STRING(SIZE(48))

  AcaRaInterfacePdu ::= CHOICE {
    raAcaCertRequest   RaAcaCertRequest,
    acaRaCertResponse  AcaRaCertResponse,
    ...
  }

  ScmsPdu ::= SEQUENCE {
    version  Uint8 (1),
    content  CHOICE {
      aca-ra  AcaRaInterfacePdu,
      ...
    }
  }

  ScmsPdu-Scoped {Pdu} ::= ScmsPdu (WITH COMPONENTS {
    ...,
    content (CONSTRAINED BY {
      Pdu
    })
  })

  ScopedCertificateRequest ::= ScmsPdu (
    ScmsPdu-Scoped {
      AcaRaInterfacePdu (WITH COMPONENTS {
        raAcaCertRequest
      })
    }
  )

END

These asn can be parse well by other asn to c compiler, such as  ossasn1c and eclipse asn compiler.
Thank you very much and look forward to your reply!

hey, bro. have you solved it? I'm having the same trouble, any advice would be greatly appreciated.

Hello, i have the same trouble. Some solution?


[Feature Request]: Add support for different similarity/distance method for collection.query()

Describe the problem
I can not find the documentation saying what the similarity method is used, or any API doc for using my own distance method.
Describe the proposed solution
Has an option to allow specify distance calculation method, eg.:
collectoin.query(similarity_fn = cosine_similarity,
    ...
)

Alternatives considered
No response
Importance
nice to have
Additional Information
No response

this is a little convoluted right now - though we are cleaning it up
you can configure it like this:
 collection = api.create_collection(
        name="test_index_params",
        metadata={"hnsw:space": "cosine", "hnsw:construction_ef": 20, "hnsw:M": 5},
    )

you can use "l2", "ip", or "cosine".

@jeffchuber
Thanks for prompting that. Sorry I'm not familiar with HNSW, may I know how should the fields represent for?

hnsw:space for the algorithm? Candidate option: cosine, l2, ip?
hnsw:construction_ef stands for __?
hnsw:M stands for __?

yes, I just want this feature....

@8rV1n

Thanks for prompting that. Sorry I'm not familiar with HNSW, may I know how should the fields represent for?
* hnsw:space for the algorithm? Candidate option: cosine, l2, ip?

* hnsw:construction_ef stands for __?

* hnsw:M stands for __?

See here: https://github.com/nmslib/hnswlib/blob/master/ALGO_PARAMS.md

Added to the docs, thank you @Russell-Pollari


apply PEP8 style to python files

set up clang-format for PEP8 if possible
apply style to existing code
remove unused functions

PEP8 is reporting, but we aren't doing anything about it.

@mattldawson do you know the state of this now?

#353 adds a github action to do this. pep8speaks used to work but appears to have gone down (https://github.com/pep8speaks-org/pep8speaks/issues/189). There are new maintainers (https://github.com/pep8speaks-org/pep8speaks/issues/191), but still the service appears to be down. We are abandoning it in favor of a pep8 PR


Backstage CSS broken

There seems to be a problem with the backstage CSS
http://dl.dropbox.com/u/28489257/TSIssues/backstageSpaceBroken.png
http://dl.dropbox.com/u/28489257/TSIssues/backstageUserBroken.png

I think this was an oversight in 49e2524e13b56092852f27af52d35fcfe4598515
The _account and _space apps share some of the css definitions that the html serialization does

jup. has full width again.


Mark notifications as read when snoozing, ignoring, archiving tasks

Fixes #55. This will mark any associated notification on GitHub as read when you take an action on a task such as snoozing, archiving, or ignoring it. This way, you won't have to do double work by managing tasks in the snoozer app and also clearing out your notifications on GitHub for the same issues and pull requests you've already dealt with.
It works by fetching notifications first, then searching issues/PRs by the selected filter. It associates notifications to tasks based on their API URL. If the notification is for the same subject (issue/PR) as a task being displayed, then when you archive/ignore/snooze the task, a request is sent to mark the notification as read.
Bonus:

Adds a new default filter, My Repositories, that shows all issues/PRs opened by others on the user's own repos. I found this useful to keep track of pull requests others were submitting to my personal projects.

@probablycorey and @summasmiff, this is ready for review now! 🎉

@cheshire137, got some failing tests!

Tests are passing now!

I paired on this with @probablycorey a bit to improve function names and make testing getNotifications a bit nicer.


Mixer + Proxy integration test

Mixer and Proxy should have an integration test that exercises an verifies a few features.

Ratelimit
Metrics -- verification using prometheus backend.
k8s pod info attributes generation

This is being done in an end to end test.
https://github.com/istio/istio/blob/master/tests/kubeTest.sh#L214
We generate load on the cluster using wrk and metrics are being fetched.
We need to apply rate limits between those calls to wrk and verify that fetched metrics look correct.

we have basic integration tests now that exercise a few mixer aspects / features. these include:

rate-limiting
denials
adding new metrics

They run with the k8s attribute generation preprocess adapter enabled (though they do not currently validate output from that attribute generator).
There is some obvious work that can beef them up (adding prometheus to the deployment and verifying metrics via promql as opposed to from /metrics, checking the source and target dimensions more carefully for metrics to verify correct k8s attribute generation, etc.).
That being said, can we close this for alpha and file a different issue for enhancing integration testing?


Icon for .Rmd files

how do I have the icon of .rmd in vscode? now it appears as the unknown file type.

Originally posted by @fdq09eca in https://github.com/Ikuyadeu/vscode-R/issues/295#issuecomment-622164109

I submitted the PR to the original seti icon project that is referred by vscode.

@Ikuyadeu That was quick, thank you!

OK, the PR is merged. Maybe, the new feature will be enabled from the next vscode update.

Confirmed that .Rmd files have a proper icon in VS Code 1.45.0. Thanks @fdq09eca and @Ikuyadeu!


Links not recognized

Hello,
I'm using neovim with the orgmode plugin and treesitter. When opening the treesitter playground, it looks like the links are not recognized, they just appear as "expr" inside a paragraph. Is it a problem on my machine or is it not implemented?
If I wrote on the wrong place, then I'm sorry, but it's kind of hard to understand which packages are used...

That's intended, objects are parsed outside of tree-sitter to allow customization and simplify the parser

Ok, thank you for the information!


use druid Datasource failed startup for: sql injection violation, comment not allow

What version of Flyway are you using?
4.0.3
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin, SBT plugin, ANT tasks)
Java API
What database are you using (type & version)?
mysql
What operating system are you using?
ubuntu
What did you do?
I use Druid database connection pool, the fault seems to it's default setting don't allow comment in sql execution.
What did you expect to see?
What did you see instead?
Caused by: org.flywaydb.core.internal.dbsupport.FlywaySqlScriptException:
Script failed
-------------
SQL State  : null
Error Code : 0
Message    : sql injection violation, comment not allow : CREATE TABLE `db`.`table_version` (
    `installed_rank` INT NOT NULL,
    `version` VARCHAR(50),
    `description` VARCHAR(200) NOT NULL,
    `type` VARCHAR(20) NOT NULL,
    `script` VARCHAR(1000) NOT NULL,
    `checksum` INT,
    `installed_by` VARCHAR(100) NOT NULL,
    `installed_on` TIMESTAMP NOT NULL DEFAULT CURRENT_TIMESTAMP,
    `execution_time` INT NOT NULL,
    `success` BOOL NOT NULL,
    -- Add the primary key as part of the CREATE TABLE statement in case `innodb_force_primary_key` is enabled
    CONSTRAINT `table_version_pk`PRIMARY KEY (`installed_rank`)
) ENGINE=InnoDB

Please raise a defect with them as this is perfectly valid SQL with no risk of SQL injection.


Modern POSIX style CLI for SQLCMD - Part 2

This is the 2nd part for the modern-CLI.  See PR165 for first part:

https://github.com/microsoft/go-sqlcmd/pull/165

This PR moves the file-layout for the modern CLI to conform to the idiomatic golang layout, and focuses on test and code coverage, there is no new functionality in this PR.  The Unit test coverage for the modern sqlcmd.exe now 95%+.
The formatting of the MIT license file was part of @shueybubbles original commit, I presume we want to keep these.
TODO:

A follow up PR will at the ./cmd/modern folder (that builds the new sqlcmd.exe) to the build pipelines.
Right now, three tests are skipped, TestQuery, TestNegUninstallNoInstanceToUninstall and TestUnicodeInput, I am debugging these now

we should commit .editorconfig so everyone gets the same behavior. the go tooling handles most of it ok but if there are options the go tools don't enforce automatically we should use this file for consistency.

Refers to: .gitignore:27 in 3db8294. [](commit_id = 3db829431465689d481d049b4b091272c20168f3, deletion_comment = False)

Done

In reply to:<PHONE_NUMBER>

Refers to: .gitignore:27 in 3db8294. [](commit_id = 3db829431465689d481d049b4b091272c20168f3, deletion_comment = False)

the structure looks ok to me. I don't see much value in unit tests that exercise a function with no assertions about the behavior. A unit test should serve as a specification (code > English) for a method, but without assertions there's no definition of correct behavior other than "it doesn't panic".  With the dependency structure you have established, unit tests should have an easy ability to inject an interface implementation into the dependencies and verify that the method under test utilized that dependency correctly, much the way C# tests do with Moq.  That way the unit test establishes the relationship between a given method and its dependencies and can truly prevent regressions in the future.


vlm.velocity.tofile export the velocity values?

Hi Velocyto team,
Thanks for providing so cool tool.
Just want to make sure that the vlm.velocity.tofile function will export the velocity values for each the cells and genes? Right? And the row of the output corresponding to the genes ID and the col corresponding to the Cell ID? Does this correct?
Thanks so much!

Any guys has the same situation? Thanks!


Script create_embedding_bio_sp.py fails with 'float' object is not subscriptable error

Describe the bug:
When I attempt to run the create_embedding_bio_sp.py script, it fails with a TypeError: 'float' object is not subscriptable. The error seems to be due to the script's inability to handle 'float' values in the data.
To Reproduce:
Here are the exact commands that I ran:

Set up the Python environment and install dependencies.
Run the tokenise_bio.py script:

source activate myenv && python "tokenise_bio.py" -i "<your_path>/storage/FASTA_small/smallfasta1.fasta" -t "<your_path>/storage/tokenizer-001.json"

Run the create_dataset_bio.py script:

source activate myenv && python "create_dataset_bio.py" "<your_path>/storage/FASTA_small/smallfasta1.fasta" "<your_path>/storage/FASTA_small/smallfasta2.fasta" "<your_path>/storage/tokenizer-001.json" -c 200 -o "<your_path>/storage/dataset-bio-003"

Run the create_embedding_bio_sp.py script:

source activate myenv && python "create_embedding_bio_sp.py" -i "<your_path>/storage/dataset-bio-003/train.csv" -t "<your_path>/storage/tokenizer-001.json" -o "<your_path>/storage/embeddings-bio-002"

Expected behavior:
Expected create_embedding_bio_sp.py script to process the dataset and generate the embeddings without errors.
Error message:
TypeError: 'float' object is not subscriptable
Suggested fix:
Unclear. Issue appears to be in utils.py at line 895, in <lambda>:
sp = data[column].apply(lambda x: x[1:-1].replace("'", "").split())

Dear @schultzjack , thanks for your informative error report!
I just pushed a new commit - please try again now with the latest version and let me know if it works. If so, I'll push a new conda build as well.
I believe the issue was incorrect logic in the create_embedding_bio_sp.py where labels were incorrectly passed.

Thank you for your prompt response, @tyronechen!
I appreciate your efforts in fixing the issue. I pulled the latest changes and re-ran the script. Unfortunately, I am still encountering the same error.
It seems like the error arises from the embed_seqs_sp() function in utils.py. The function appears to process columns of the CSV file as if they are strings. The TypeError is raised when the function tries to apply string operations to a float value, possibly suggesting that somewhere in the function, a value is not being correctly processed as a string.
Based on the analysis of the error traceback and the code, it seems that the embed_seqs_sp() function might be trying to process a column that doesn't exist in the input data or a column that contains non-string values.
Could it be possible that the function is expecting a different format or type of data than what is being provided in the train.csv file? Any insights you could provide would be greatly appreciated.
For reference, I am passing in -i (infile path), -t (tokeniser path), and -o (output_dir) when running the script.

Thank you for your prompt response, @tyronechen!
I appreciate your efforts in fixing the issue. I pulled the latest changes and re-ran the script. Unfortunately, I am still encountering the same error.
It seems like the error arises from the embed_seqs_sp() function in utils.py. The function appears to process columns of the CSV file as if they are strings. The TypeError is raised when the function tries to apply string operations to a float value, possibly suggesting that somewhere in the function, a value is not being correctly processed as a string.
Based on the analysis of the error traceback and the code, it seems that the embed_seqs_sp() function might be trying to process a column that doesn't exist in the input data or a column that contains non-string values.
Could it be possible that the function is expecting a different format or type of data than what is being provided in the train.csv file? Any insights you could provide would be greatly appreciated.
For reference, I am passing in -i (infile path), -t (tokeniser path), and -o (output_dir) when running the script.

Thanks for the info @schultzjack ,
Based on your feedback, theres a few possibilities i can think of but ill need to investigate these 1 by 1:

Depending on your read length, the token array may be truncated to [[1 ... 1]] (you can inspect a long entry in your train.csv to rule this out - where ... would be interpreted as str. (this is a known bug and caused by the default numpy array display limits)
I may be iterating over the data at an incorrect level, ie [] instead of [[]]
I was having an issue with the way datasets encodes ClassLabels earlier, I cannot rule this out yet (but its probably less likely than the other options)

Unfortunately I will only be able to look at this in detail probably next week, but if any of the above helps, or if you can think of other options please let me know (and feel free to submit a pull request if you choose to do so ٩(◕‿◕)۶)

Hello.
I am facing the same issue. I do not think it is truncation, as this is fixable with  np.set_printoptions(threshold=np.inf) which I have set at the top utils.py, resulting csvs that are no longer truncated.

Unfortunately I will only be able to look at this in detail probably next week, but if any of the above helps, or if you can think of other options please let me know (and feel free to submit a pull request if you choose to do so ٩(◕‿◕)۶)

Sorry all, i might have to postpone this until late August.

Hello,
I believe I may have figured out how to fix it, although I am unsure if it is correct. It seems to be working but I'm no expert, and I'm not exactly sure what I am looking for except no errors and an embeddings.csv.
In create_embeddings.py, I modified the following code before the first call to Word2Vec  (in the if model == None:   statement
 all_kmers = itertools.chain()
        for i  in kmers:
            all_kmers = itertools.chain(all_kmers, i)

real_kmers = itertools.chain()
        for i, j in all_kmers:
            all_kmers = itertools.chain(real_kmers, [i])

 model = Word2Vec(
            sentences=real_kmers,
            vector_size=w2v_vector_size,
            window=w2v_window,
            min_count=w2v_min_count,
            sg=w2v_sg,
            workers=njobs
            )

I think the error was coming from word2vec trying to create a vocabulary for the label (1 or 0)
I obtain a file (kmers_embeddings.csv) with one line per kmer starting with the label, the string, and then 100 float values that I assume are the embeddings. It would be great if you could confirm that this is the expected behaviour.
Thank you for this really cool project :)

I think the error was coming from word2vec trying to create a vocabulary for the label (1 or 0)

I took a closer look and it does seem to be the case, the aim of that step is to parse out the embeddings. It appears the tokens are not generated correctly, which is likely the cause of the float not subscriptable error everyone is getting (it is trying to index a NaN).
Interestingly, I was unable to replicate the fix by the method you specified, but I narrowed down the most likely low-level cause of the issue now.

I obtain a file (kmers_embeddings.csv) with one line per kmer starting with the label, the string, and then 100 float values that I assume are the embeddings. It would be great if you could confirm that this is the expected behaviour.

Yes, this is the intended behaviour: label(int),kmer(str),embedding(float)
0,AAAAA,-0.122,-0.313,...
...

Thank you for this really cool project :)

Hope you find it useful, if you see any other bugs please do not hesitate to let us know

Interestingly, I was unable to replicate the fix by the method you specified, but I narrowed down the most likely low-level cause of the issue now.

Nevermind, this was one component of the fix, a second component was that an intermediate file was being parsed incorrectly (column frameshift caused by an unintended null column)
This should work as expected now, ran a few small tests and no issues so far. v2.4.4 is building and will be online shortly.
Please install with:
mamba install -c tyronechen -c conda-forge genomenlp==2.4.4

In case there are still any bugs related to this, I will leave this issue open until the end of the month and close it if there are no further bug reports.

I think the error was coming from word2vec trying to create a vocabulary for the label (1 or 0)

I took a closer look and it does seem to be the case, the aim of that step is to parse out the embeddings. It appears the tokens are not generated correctly, which is likely the cause of the float not subscriptable error everyone is getting (it is trying to index a NaN).
Interestingly, I was unable to replicate the fix by the method you specified, but I narrowed down the most likely low-level cause of the issue now.

Thank you for posting your thoughts regarding this issue and what you're saying aligns with what I was seeing with how it was trying to index a NaN. I thought it was more related to how pandas handles missing values and empty strings and how, when reading a CSV file, pandas treats both as NaN. Given what you said about the tokens not getting generated correctly, it seems like the issue is less about simply preserving empty strings when loading DataFrames, so strategies like using the fillna function to convert NaN values to empty strings won't actually resolve the core issue (even if they eliminate that particular error).
I'm not doing further testing on this issue at the moment as I'm focusing on running the tokenise_bio.py script with the human genome fasta featured in your preprint. I'll use a separate issue for that.
Thank you for your creating this very interesting project and for your help with investigating these issues!

No problem @schultzjack , in that case I will close it for now, but if this resurfaces let me know and I will reopen to investigate.


Aliphatic

By using this template, you can ensure that your feature request provides all the necessary information for the project maintainers to understand your proposal clearly. The more precise and well-defined your request is, the easier it will be for the developers to assess its feasibility and potential impact on the project.
Happy to contribute here 😁🤩

@ALIPHATICHYD Its better to raise an issue first and then raise a PR . Please follow the code of conduct of this project.

The more precise and well-defined your request is, the easier it will be for the developers to assess its feasibility and potential impact on the project.

I have created an Issue for the PR


Unable to compose up

I try to compose up but i am facing the below issue

docker-compose up
WARNING: Found orphan containers (srsenb) for this project. If you removed or renamed this service in your compose file, you can run this command with the --remove-orphans flag to clean it up.
Pulling icscf (docker_kamailio:)...
ERROR: The image for the service you're trying to recreate has been removed. If you continue, volume data could be lost. Consider backing up your data before continuing.
Continue with the new image? [yN]N
ERROR: pull access denied for docker_kamailio, repository does not exist or may require 'docker login': denied: requested access to the resource is denied

Please re-read the README. You need to compile docker_kamailio image first


Xamarin.Forms 2.2+ breaks support for Android

Xamarin.Forms 2.2+ introduces a new set of Android support libraries which will break compatibility with libraries that use older versions of Xamarin.Forms. The resolution is to update all projects to add support for 2.2.

Are you saying there is a resolution that makes NControls work with Forms 2.2?
Or are you saying Xamarin needs to creation a resolution for all their libraries to be backwards compatible with various NuGets such as NControls?  I'm unclean by what you're statement is saying.
I have a rather exhaustive post on this on the Xamarin forums if anyone has a solution they can detail here and/or there.
http://forums.xamarin.com/discussion/68085/xamarin-forms-2-2-support-libraries-breaks-some-android-nugets-such-as-ncontrol-and-others/p1?new=1

Looks like the root of this issue is Xamarin Forms 2.2 Hard dependency on 23.3 android support libraries.. don't look like that is going to be fixed anytime soon.
Would be awesome if NContol.Controls could update their dependencies to the newer released android support libraries & Xamarin Forms 2.2+

Ah, you want MORE than XF 2.2? Like 2.3 to solve this?
On Wed, Aug 3, 2016 at 11:43 AM, Christian Falch<EMAIL_ADDRESS>wrote:

Just spent some time upgrading to XF 2.2:
Release <IP_ADDRESS>
https://github.com/chrfalch/NControl.Controls/releases/tag/<IP_ADDRESS>
Should this solve the issue?
On Wed, Aug 3, 2016 at 10:46 AM, Malcolm Jack<EMAIL_ADDRESS>wrote:

Looks like the root of this issue is Xamarin Forms 2.2 Hard dependency on
23.3 android support libraries.. don't look like that is going to be fixed
anytime soon.
Would be awesome if NContol.Controls could update their dependencies to
the newer released android support libraries & Xamarin Forms 2.2+
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/chrfalch/NControl.Controls/issues/29#issuecomment-237179534,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA1a9KmjdaIY_-SQivIm9jQdY-tCdF6eks5qcFVigaJpZM4IeX84
.

Just spent some time upgrading to XF 2.2:
Release <IP_ADDRESS>
https://github.com/chrfalch/NControl.Controls/releases/tag/<IP_ADDRESS>
Should this solve the issue?
On Wed, Aug 3, 2016 at 10:46 AM, Malcolm Jack<EMAIL_ADDRESS>wrote:

Looks like the root of this issue is Xamarin Forms 2.2 Hard dependency on
23.3 android support libraries.. don't look like that is going to be fixed
anytime soon.
Would be awesome if NContol.Controls could update their dependencies to
the newer released android support libraries & Xamarin Forms 2.2+
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/chrfalch/NControl.Controls/issues/29#issuecomment-237179534,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA1a9KmjdaIY_-SQivIm9jQdY-tCdF6eks5qcFVigaJpZM4IeX84
.

... it should... will be checking this evening.

yup - seems like its all good if using the ><IP_ADDRESS> NControls :)


Remove unused DynamicForm

I think this may have been a precursor to DDF, not used now, removing.

:tada: This PR is included in version 0.11.65 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Redundant backslash in 1.4.3 breaks emergency mode

There are 2 items in 1.4.3, they add sudo authorization check to emergency and rescue modes. This is done using lineinfile module in RHEL7-CIS/tasks/section1.yml.
Expected Behavior after the role is executed: emergency or rescue mode asks root authorization before loading. (same as default for my RHEL7 by the way).
Observed Behavior: emergency mode never loads because the /usr/lib/systemd/system/emergency.service file is corrupted.
The reason: backslash is not required, but added (see lines below). In my case it is added to emergency.service causing corrupted file. The issue is quite serious because it's quite hidden: you never observe this until you really need emergency/rescue mode. I went there, so I know )
Current line (i.e. 5489b4f on Apr 1 2020):
line: 'ExecStart=-/bin/sh -c \"/usr/sbin/sulogin; /usr/bin/systemctl --fail --no-block default\"'
Correct line (in my opinion):
line: 'ExecStart=-/bin/sh -c "/usr/sbin/sulogin; /usr/bin/systemctl --fail --no-block default"'
Unfortunately I don't have all info right now about ansible version I used, it was quite fresh, I can clarify if it's important. Described behavior was the same from RHEL ansible management point and from cygwin ansible on WINdows management point in other system.

by the way I had to edit my description to make those backslashes shown )). Markdown syntax showed both lines same initially. (just a funny observation)

Hello,
I wanted to reach out and let you know that this issue is being closed. We have re-worked the role and want to start with a fresh issues list with this latest version. There was a post in the Ansible-Lockdown google group (https://groups.google.com/g/ansible-lockdown) with the details of the changes that are coming. Please checkout the thread titled RHEL 7 CIS and STIG Changes for all of the details, I also have the message pasted at below.
Please as you use the latest version and open issue tickets as you find them, it is the best way for us to improve the role for everyone. Thank you for being part of the community and providing awareness of problems or advice on improvement. Reporting is a huge part of improving this project.

Hello,
Thank you to everyone in the Ansible-Lockdown community who has contributed to RHEL7 STIG/CIS. Our team at MindPoint Group has been working with the entirety of the Ansible-Lockdown project, and we have some significant updates for both RHEL 7 STIG and CIS. With these updates, some larger changes have been made. I have these changes/updates outlined below.
Testing:

CI/CD - We have implemented some automated testing pipelines to test pull requests into the devel and main branches. With the current workflow, the community will PR into the devel branch (never the main branch) for review by the administrators. When your PR is created, the first check will remain the DCO check. The second check is a functional testing pipeline that will automatically perform a functional test of the branch the PR is initiated from. Once both tests pass, someone from the Administrator Team will review the changes and merge them into the devel branch. From there, an additional review is completed before the devel branch is merged into the main branch. Only the Admin Team will perform PRs/merges into the main branch. There is also an automated pipeline for PRs from devel to main. Please do not edit the .github/workflows files since those are used as part of the pipeline.
Compliance Checking – MindPoint Group has been working to create our own compliance audit scan tool. The tool uses a goss framework executable to run custom checks that we have created. The goal is to provide a more thorough check for control compliance and decrease the number of false positives/negatives. For example, it will check the configuration file related to the control and as well as checking if that configuration is active. With a smarter scan, we can hopefully identify attempts to trick scanners as well (for example stacking a parameter in a config file where the first instance is enabled and second disabled. Most audit tools search for the first instance but the application might look for the last instance of the parameter, thus making the scanning tool think it's enabled). In testing, we have found that our audit scan runs significantly faster than other audit tools, reducing audit times. Our audit tool and profiles will have their own repositories in the Ansible-Lockdown org, but within the remediation role there will be an integrated way to incorporate the audit. Keep an eye out for the audit tools as they are released. We plan on developing a goss audit profile for each current remediation role. Going forward, we plan to release a remediation role and goss audit tool profile simultaneously.
Role Updates:
RHEL 7 STIG/CIS – We have re-written much of the RHEL 7 STIG and CIS roles to increase clarity and readability and address some functionality items. We performed these updates while creating our goss testing framework for each of these roles. We plan on pushing our update to the devel and main branches. We will move the current devel and master branches to a devel_stable_ and master_stable_ branch in the respective repositories. Accordingly, community members who rely on the current version can still use that version going forward; this process will not remove what is currently there. The latest versions of the roles have also been updated to comply with the latest benchmarks.
Role Architecture – All roles will change with regard to the structure in the tasks folder. Taking CIS as an example, there will be a folder per section and yaml files for each sub-section. For example control 1.2.1 in CIS will be located in RHEL7-CIS/tasks/section_1/cis_1.2.x.yml. The cis_1.2.x.yml file will contain all controls related to section 1.2.x. This will hopefully make updates to roles a bit easier with less risk. This matches the architecture of our audit tool, creating consistency across remediation and audit platforms. The end goal is to repeat this architecture (the best we can) on STIG roles, but we are starting with CIS.
Existing PRs and Issues – With all of these changes comes the task of cleaning up existing PR’s and issues. Our plan is to close all of the existing PR’s and issues because of the re-work. Our team is growing and should be able to stay on top of the new issues and PRs as they come in.
Again, I would like to thank the community for your involvement in this project. The input and work from the community has contributed significantly to the success of this project. Please keep an eye out for these changes, which will be rolling out in the coming weeks.


[SPARK-16843][MLLIB] add the percentage ChiSquareSelector feature

What changes were proposed in this pull request?
Now, there is only numTopFeatures Param in ChiSquareSelector. In practice, it is convenience to use the percentage as the Param.
We add the percentage Param for ChiSquareSelector in this PR.
How was this patch tested?
add scala ut

Can one of the admins verify this patch?

I'm not sure it's worth a whole other API method for this. If you want to select 10% of features, you can trivially ask for 0.1 * numFeatures features from the selector.

Hi @srowen, thanks for your comment.
I agree for your comment, user can get the number of features without percentage method. For the user experience, sometimes the percentage method seems better.
In scikit-learn, there are both  SelectKBest and SelectPercentile APIs: http://scikit-learn.org/stable/modules/classes.html#module-sklearn.feature_selection

Hi @srowen , I also plan to submit some PR about feature selection methods based on univariate statistical test, like the methods in scikit-learn:  SelectFpr (using false positive rate),  SelectFdr ( using false discovery rate), and SelectFwe (family wise error ).
http://scikit-learn.org/dev/modules/feature_selection.html
Now, the ChiSqSelector in Spark only can select the topNumFeatures.  Do you think Spark should support other feature selection methods?

Percentage is a useful addition to ChiSquareSelector, it is a common and intuitive param to data scientists and statistician as scikit-learn has, but it may be not worthy a whole other API in MLlib indeed. @srowen I suppose it could be implemented by adding a new Param in ML.ChiSqSelector?

If anything it could be a Param in the ml version (that can be discussed on the JIRA perhaps). But I'm pretty ambivalent about even that since as Sean says it's pretty easy to just specify it. Overall I just don't think it's worth it.

As for other feature selection methods, feel free to create a JIRA to discuss. Some work has been done outside of Spark in packages - e.g. https://github.com/sramirez/spark-infotheoretic-feature-selection. Generally I think this is a good place for that kind of work to start - I don't think it necessarily must be in Spark itself. If usage and performance is high, it can always be considered for inclusion later on.

I'd like to close this in favor of the changes in https://github.com/apache/spark/pull/14597 because I think it would actually lead towards making this functionality trivial to expose from the model class.


Compability with new release of ObjectMapper

Hi there,
I am wondering if you could please use ObjectMapper 2.2.1, thought they had some bugs fixed.
Thanks for your help.

@bss-duy Sure, I will consider to update it

Update to ObjectMapper to 2.x. Check the latest release: 1.1.1.


www/caddy: Allow bind to default loopback interface and non standard ports

Fixes: https://forum.opnsense.org/index.php?topic=40435.msg200960#msg200960
Fixes: https://github.com/opnsense/plugins/issues/4065
Adds following options:

http_port: https://caddyserver.com/docs/caddyfile/options#http-port
https_port: https://caddyserver.com/docs/caddyfile/options#https-port
default_bind: https://caddyserver.com/docs/caddyfile/options#default-bind

http_port and https_port can be chosen freely.
default_bind is a checkbox that enables bind to <IP_ADDRESS> and [::1], which should ensure that there won't be service startup problems, since lo0 should be always there on boot.
Allows a configuration like this:
root@OPNsense:~ # sockstat -ln | grep -i 8080
0        caddy      86521 16 tcp4   <IP_ADDRESS>:8080        *:*
0        caddy      86521 18 tcp6   ::1:8080              *:*
root@OPNsense:~ # sockstat -ln | grep -i 8443
0        caddy      86521 25 tcp4   <IP_ADDRESS>:8443        *:*
0        caddy      86521 26 udp4   <IP_ADDRESS>:8443        *:*
0        caddy      86521 27 tcp6   ::1:8443              *:*
0        caddy      86521 28 udp6   ::1:8443              *:*

This should satisfy advanced usecases without causing regressions.

@AdSchellevis If you have time, I would love some feedback on this idea. Offering bind to the default loopback interface should be safe, right?


Swift 4

Related Issues:

AGIOS-555 - Set deployment target to iOS 9 on aerogear-ios-oauth2
AGIOS-551 - [Swift 4] update aerogear-ios-oauth2

Going to send a new one to the correctly branch


Invalid API Calls Error handling

Hi, thanks for your hard work ! I really apreciate it ! I amworking on a script that will interactively look for some data on a list of tickers. I've donwloaded the .csv list for the digital currencies and i've noticed that several of the currencies are returning an error from the API, i have reported to Alpha but i think that i would be nice to implement and api error handling and return a code in case of api error to avoid that the scprit don't halt.
if you look at their digital_currency_list.csv avilable in the docs the first item in the list is 1ST but if i call:
data, meta_data = cc.get_digital_currency_daily(symbol='1ST', market='EUR')
i have
ValueError: Invalid API call. Please retry or visit the documentation (https://www.alphavantage.co/documentation/) for DIGITAL_CURRENCY_DAILY.

Thanks Riodda!
Hmm... It looks like this is a missing ticker to bring up to AV support!
The error you're getting is the error message. If you want your script to not halt, you can just add a try catch. Something like:
from alpha_vantage.timeseries import TimeSeries
symbols = ['AAPL', 'GOOG', 'TSLA', 'MSFT', 'asldkfjalskdjfaskdljflasdfjlasdflkajsdfjlasdf']
results = []
ts = TimeSeries()
for symbol in symbols:
    try:
        results.append(ts.get_quote_endpoint(symbol)[0]['05. price'])
    except:
        pass
print(results)

This will just keep going down the list, and if an error occurs, it will not break

Feel free to follow up with questions.


PA2 Level 6 Reduce()

Summary
Is it allowed to use recursion to solve level 6 reduce() ?
Description
I have solved level6 reduce with a gcd method along with a helper method to solve reduce(), but the helper method uses recursion. It seems like codecrunch did not flag me for this, I'm not sure if it is allowable since level2 says no recursion. Thanks!

I think should be fine as long as there are no specific constraints mentioned i.e. level 2 explicitly mentions no recursion, maybe prof can clarify this? 😅

I believe the way it has been is that codecrunch will be the judge, so if it's not flagged then you should be fine! (checkstyles are not graded for PAs anyway, unless Prof decides that codecrunch should be more stringent on that)

The no recursion constraint seems to only apply for the add/sub... methods

Hi, I think it should be okay. For reduce(), the question paper didn't explicitly state that recursion was not allowed.
Probably recursion is not allowed only for the earlier methods like add etc
Hope this helps

i guess the only way to check if its banned is via CC


Automated Package Upgrade

:tada: This PR is included in version 25.6.118 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


500 Internal Server Error on API Request to Pirate Weather

Describe the bug
A 500 Internal Server Error occurs when making an API request to Pirate Weather using the forecast URL. The error happens during the setup process in Home Assistant, preventing the integration from retrieving weather data.
Failed setup, will retry: 500, message='Internal Server Error', url='https://api.pirateweather.net/forecast/API KEY/LAT,LONG?units=si&extend=hourly&version=2'

Expected behavior
The API request should return the weather data as per the location coordinates and API call parameters (units=si, extend=hourly, version=2), allowing the Home Assistant integration to process the data successfully.
The integration should successfully retrieve forecast data from Pirate Weather, without a 500 error, and update the sensors in Home Assistant.
Actual behavior
Instead of receiving the expected weather data, the API request results in a 500 Internal Server Error, causing the integration to fail the setup process. Home Assistant attempts to retry but continues to receive the same error from Pirate Weather's API.
Home Assistant version
2024.9.1
Integration version
1.5.8
Other details
API endpoint: https://api.pirateweather.net/forecast/API KEY/LAT,LONG?units=si&extend=hourly&version=2
The issue could be related to server-side problems on Pirate Weather’s API.
The error persists across multiple attempts and appears to be unrelated to rate limits or syntax errors in the request URL.
A test of the API key and URL in external tools also results in the same error.
Troubleshooting steps

[X] I have updated my Home Assistant installation to the latest version.
[X] I have updated the Pirate Weather Integration to the latest version.
[X] I have gone through the documentation before opening this issue.
[X] I have searched this repository and API Repository to see if the issue has already been reported.
[X] I have restarted my Home Assistant installation.
[X] I have queried the API in my browser to confirm the issue is not with the API.
[X] I have written an informative title.

When my "everything is down" alarm sounds, I assume it's an issue with (in order):

Datetimes
Node memory
Node Memory

And onwards. There are a ton of fallbacks and checks in the code base at this point, which usually avoids major issues... but not this one. Of all things, it was my healthcheck- turns out a misformed request in the healthcheck code worked before, but not with the new error responses in V2.3. The fallback then failed because (you guessed it), the healthcheck also marked it as unhealthy.
In any case, really sorry for this downtime, and everything is back up and running now. Error handling improvements have been turned off, bumping to V2.3.1

@cloudbr34k84 You included your API and location in your message which I edited to remove but for some reason it's not letting me remove the revision. When including API URLs next time be careful to remove your API key from the URL.
Either way since the API is back up I will close this issue.

Awesome man, no need to apologise, these things happen :)


General warning fixes for v3.1.2

Good to hear that you are going to remove the survey deprecated code soon, The reason behind this PR is to eliminate/silence all the warnings from mixpanel code. Since the project I am involved in is treating warnings as errors (and hence those prevents compiling and running). Plus no warnings is a good thing :)
Anyways, I updated this PR to only include general warnings fixes.

Thanks for doing this @hassanain-idris! We're planning on removing surveys from our product this month, so I'd rather not change any of that code. If you want to make a different PR with just the other warning fixes, that would be much appreciated!

thanks for doing this!


Map on "get-involved" page missing countries

The map of the World (translations.svg) in the get-involved page (http://beta.elementaryos.org/get-involved) is missing a number of countries and thus looks a little strange.
For example, New Zealand and various Pacific islands, Indonesia, PNG, Philippines, Japan, Taiwan, Madagascar, The Falklands, Iceland, Sicily, the UK and Northern Island plus whatever that boomerang shaped island is off the north of Russia, Serverny Island, that's it.
Tasmania is also missing from Australia. I mean I can understand leaving off New Zealand, but Tasmania!? :-)

And the island off the coast of eastern Canada (where I'm from) :O


Prevent max_unknown_words < 1

Description:

max_unknown_words < 1 in the IntentClassifierDataAugmentationConfig makes the data augmentation crash, this should not be allowed

Codecov Report

Merging #711 into develop will increase coverage by 0.06%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop     #711      +/-   ##
===========================================
+ Coverage       88%   88.06%   +0.06%
===========================================
  Files           65       65
  Lines         3809     3795      -14
  Branches       721      716       -5
===========================================
- Hits          3352     3342      -10
+ Misses         348      345       -3
+ Partials       109      108       -1


cognito-at-edge is not able to redirect to IdP

I am using federated cognito, where the user is redirected to SAML IdP. Cognito-at-edge does not expose the "scope" and "idp_identifier". this is based on the documentation from the aws site https://docs.aws.amazon.com/cognito/latest/developerguide/authorization-endpoint.html
What happened: When using this in lambda, user is not redirected to the IdP
What did you expect to have happen: User must be redirected to the IdP
How to reproduce this (as precisely and succinctly as possible): Create a new Cognito Pool and use a third party  identity provider (like SAML). Set the properties from the pool
const authenticator = new Authenticator({
// Replace these parameter values with those of your own environment
region: 'us-west-2', // user pool region
userPoolId: 'ddffff', // user pool ID
userPoolAppId: 'xxxxx', // user pool app client ID
userPoolDomain: '.auth.us-west-2.amazoncognito.com', // user pool domain
});
Anything else you think we should know?
Environment:

version of cognito-at-edge being used: Latest version

node version of code base which uses cognito-at-edge:16.x

other:

Hi @nagendrakumar02 ,
I understand that when using cognito-at-edge's Authenticator.handle method to handle your CloudFront's distribution viewer request your website's visitors are not being redirected to your SAML identity provider. Would you mind describing the behavior that you're seeing instead?

Hi @jeandek,
I was able to get it working by forking the code and added 2 optional parameters (idprovider and scope). If the 2 parameters are set, it is appended to the redirect. This has fixed my issue.

Hi @jeandek,
I was able to get it working by forking the code and added 2 optional parameters (idprovider and scope). If the 2 parameters are set, it is appended to the redirect url. This has fixed my issue.

Are you able to make a PR for these changes? I need this functionality as well.

I forked the code from the main branch to support redirecting to IdP.
https://github.com/nagendrakumar02/cognito-at-edge

Nagendra Dattatreya
CTO
973.960.3951
zappyride.com https://www.zappyride.com/
https://www.zappyride.com/
[image: LinkedIn] https://www.linkedin.com/company/27054560[image:
Twitter] https://twitter.com/zappyride
On Wed, Feb 15, 2023 at 12:02 PM Theodore Rogers @.***>
wrote:

Are you able to make a PR for these changes? I need this functionality as
well.
—
Reply to this email directly, view it on GitHub
https://github.com/awslabs/cognito-at-edge/issues/56#issuecomment-1431691341,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AY4N7ZMNI35PICOQ5ZJUMC3WXUD2PANCNFSM6AAAAAATUHB3CQ
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

2 new parameters are added
_idpIdentifier: Name of the IdP
_scope: scope
To read more about this follow this article
https://docs.aws.amazon.com/cognito/latest/developerguide/authorization-endpoint.html

Nagendra Dattatreya
CTO
973.960.3951
zappyride.com https://www.zappyride.com/
https://www.zappyride.com/
[image: LinkedIn] https://www.linkedin.com/company/27054560[image:
Twitter] https://twitter.com/zappyride
On Sat, Feb 18, 2023 at 9:40 AM Nagendra Dattatreya @.***>
wrote:

I forked the code from the main branch to support redirecting to IdP.
https://github.com/nagendrakumar02/cognito-at-edge

Nagendra Dattatreya
CTO
973.960.3951
zappyride.com https://www.zappyride.com/
https://www.zappyride.com/
[image: LinkedIn] https://www.linkedin.com/company/27054560[image:
Twitter] https://twitter.com/zappyride
On Wed, Feb 15, 2023 at 12:02 PM Theodore Rogers @.***>
wrote:

Are you able to make a PR for these changes? I need this functionality as
well.
—
Reply to this email directly, view it on GitHub
https://github.com/awslabs/cognito-at-edge/issues/56#issuecomment-1431691341,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AY4N7ZMNI35PICOQ5ZJUMC3WXUD2PANCNFSM6AAAAAATUHB3CQ
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

@nagendrakumar02 was this ever merged into this package? I need this feature too.

Looks like the PR was closed and not merged: https://github.com/awslabs/cognito-at-edge/pull/73


Updates to support an explicit home directory

When github mounts container images, they do a number of things to ensure the home directory maps, unfortunately it is not enough and other scripts and programs do not follow their lead. This enables the user to override the environment variable to resolve that issue.

https://github.com/webfactory/ssh-agent/issues/20#issuecomment-739097276

Thank you @adamrogas for this suggestion! I like the change better than the suggestion in #20.
However, could you help us to figure out (over at #20) why this is necessary in the first place? What's so different inside the container that we need some special handling at all?

Does the changed home dir detection from #17 make a difference here?


Car interfaces: remove hard-coded signal definitions

@deanlee the car interfaces would look much cleaner once we start parsing all the signals :). Do you think performance is good enough to use now?

the new benchmark looks good:
https://github.com/commaai/opendbc/pull/795#issuecomment-1492864190
need to continue improve the code and do more testing, but surely, I think it's ok to remove them all after https://github.com/commaai/opendbc/pull/795  is ready.

and vw! (sorry comments)

If you strike down my comments, I shall become more verbose than you can possibly imagine.
...
j/k :) this is a good change.

@sshane before merging this, I'd like a prereq PR that explicitly copies out any fields that . otherwise, it's easy to merge a DBC update that silently changes behavior, especially since those strings won't be grep-able.

@adeebshihadeh I'll look over everything tomorrow and should be good if your opendbc PR is ready.

Ran test_models locally a few times on master and this PR to test the time to parse CAN messages in each platform's interface and carstate. The two tests ran were test_car_interface and test_radar_interface with 4 jobs, all rlogs cached.
master:
430 tests
03:26
03:24
03:22

what-if-no-signals:
430 tests
03:40
03:42

@jyoung8607 @incognitojam if there's any comments this removes for signals, make sure they're in the DBC. Then you can add to where they're used if how they're used isn't obvious


state machine impl

and first iteration of a server

Looks like a great improvement in clarity on first glance. Might be useful to document the state machine in a diagram.

That's the plan. I was kinda hoping to automate that though :)

Would also be great to put type annotations on various fields where possible.

This now has a few (integration) tests via a mock server and properly typed structs.
State machine:

There's also the question whether we care about Julia versions older than 1.3 here. I can just switch to Downloads.jl if not.

We are encountering following error https://gist.github.com/bmharsha/590067fb52899d520146bd8ea3e08001#file-error1-md when we tried using PkgAuthentication.jl from this PR. Pkg server was set to pkg.juliahub.com when we encountered this error. This server requires end-users to be authenticated inorder to download packages.

I made some tweaks to this and asked some questions in a PR to this PR: https://github.com/JuliaComputing/PkgAuthentication.jl/pull/9. Let's address the questions (marked with "QUESTION") and then we can figure out whether to merge my changes or not. I'm especially interested in the expiration logic. Is the key ever expiry rather than expires or expires_at or expires_in? If both expires_at and expires_in are set, I think the best thing to do is to take the minimum of expires_at and the locally computed expiration time based on the current time and expires_in. Ideally the server should be sending both expires_at and expires_in. The reason for this is so that if there's significant clock skew between the client and the server the worst case that can happen is that the client just refreshes their token every time, which is slow but works. Otherwise if there's clock skew the client could end up refreshing too infrequently, leading to auth failures.

Left a review over at #9 that should address your questions.
Adding another toml["expires_in"] + mtime > time() check totally makes sense, yes.

Alright, pushed a commit that addressed your concerns and improved the expires_in/expires_at situation.


Heading Level Indent stopped working with latest Minimal theme

Is the bug present when using the default Obsidian theme?
No
Is the bug present when snippets and plugins are disabled?
Yes
Minimal theme version
7.0.5
Describe the bug
Since the version  7.0.5 of the Minimal theme, the plugin Heading Level Indent stopped working.
This bug report on the plugin side is: https://github.com/svonjoi/obsidian-heading-level-indent/issues/9
Thanks in advance,
Aurélien
Debug info
SYSTEM INFO:
Obsidian version: v1.3.5
Installer version: v1.3.5
Operating system: Windows 10 Enterprise 10.0.22621
Login status: not logged in
Insider build toggle: off
Live preview: on
Legacy editor: off
Base theme: light
Community theme: Minimal v7.0.7
Snippets enabled: 1
Restricted mode: off
Plugins installed: 9
Plugins enabled: 9
1: Tasks v4.2.0
2: Banners v1.3.3
3: Dataview v0.5.56
4: Homepage v3.3.2
5: Force note view mode v1.2.0
6: Minimal Theme Settings v6.3.2
7: Emoji Shortcodes v2.2.0
8: Heading Level Indent v1.0.3
9: Janitor v1.0.7
RECOMMENDATIONS:
Custom theme and snippets: for cosmetic issues, please first try updating your theme and disabling your snippets. If still not fixed, please try to make the issue happen in the Sandbox Vault or disable community theme and snippets.
Community plugins: for bugs, please first try updating all your plugins to latest. If still not fixed, please try to make the issue happen in the Sandbox Vault or disable community plugins.

Hi,
I disabled 'Readable line length' in Obsidian today (Editor -> Display).
That made the Heading Level Indent plugin work again.
Hope this helps pinpoint where the issue is.
Aurélien


Update Bernd's profile and regenerate TOC

Docs Build status updates of commit 49e6548:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

advocates/bernd-verst.yml
:x:Error

Details

advocates/index.html.yml
:warning:Warning

Details

advocates/map.yml
:warning:Warning

Details

advocates/toc.yml
:warning:Warning

Details

advocates/index.yml
:warning:Warning

Details

advocates/bernd-verst.yml

Line 15, Column 3: [Error-yaml-syntax-error] While scanning an anchor or alias, found value containing disallowed: []{},

advocates/index.html.yml

Line 10, Column 8: [Warning-file-not-found] Invalid file link: './map'.
Line 12, Column 8: [Warning-file-not-found] Invalid file link: './tweets'.
Line 200, Column 8: [Warning-xref-not-found] Cross reference not found: 'advocates.bernd-verst'.

advocates/map.yml

Line 196, Column 8: [Warning-xref-not-found] Cross reference not found: 'advocates.bernd-verst'.

advocates/toc.yml

Line 45, Column 10: [Warning-xref-not-found] Cross reference not found: 'advocates.bernd-verst'.

advocates/index.yml

Line 10, Column 8: [Warning-file-not-found] Invalid file link: './map'.
Line 12, Column 8: [Warning-file-not-found] Invalid file link: './tweets'.
Line 130, Column 8: [Warning-xref-not-found] Cross reference not found: 'advocates.bernd-verst'.
Line 372, Column 8: [Warning-xref-not-found] Cross reference not found: 'advocates.jeremy-likness'.
Line 376, Column 10: [Warning-file-not-found] Invalid file link: 'media/profiles/jeremy-likness.png'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit ae26cad:
:warning: Validation status: warnings

File
Status
Preview URL
Details

advocates/index.html.yml
:warning:Warning
View
Details

advocates/bernd-verst.yml
:white_check_mark:Succeeded
View

advocates/map.yml
:white_check_mark:Succeeded
View

advocates/media/profiles/bernd-verst.png
:white_check_mark:Succeeded
View

advocates/toc.yml
:white_check_mark:Succeeded
View

advocates/index.html.yml

Line 10, Column 8: [Warning-file-not-found] Invalid file link: './map'.
Line 12, Column 8: [Warning-file-not-found] Invalid file link: './tweets'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

@MaximRouiller please merge :)


chore(ci): increase Lighthouse CI max_timeout

Motivation
Currently Lighthouse CI test fails of on the initial PR deploy because build take more than 5 minutes:

Build time: 7m 58s. Total deploy time: 8m 21s

This PR increased this time to 600 seconds, so 10 minutes.
Unfortunately this is not the best way to solve the issue, but it is the best what can be achieved right now.
The other problem with the Lighthouse check is that pushing another commit to the PR will cause Lighthouse check to run immediately on the site deployed from the previous build, because it only depends on 200 response from the content located under deploy URL and deploy URL do not change between commits (do not include commit hash).
To fix this properly the website deployment should be a part of GitHub workflows, and lighthouse check should depends on that. Then the needs job parameter could be used to prevent the issue described above.
Another way could be modifying Netlify or/and Lighthouse setup/config files in a way which would change the deploy URL between the commits or Netlify status can be hooked into Lighthouse workflow somehow.
Have you read the Contributing Guidelines on pull requests?
Yes.
Test Plan
Run the CI.
Related PRs

#3863

Thanks, will try to figure out a solution to this problem but increasing the timeout is good enough for now.
We should find a way to make this lighthouse score more reliable too because currently it's quite random 🤪


galaxykit: use main branch, instead of feature branch

now that rbac is merged, we can use galaxykit main again :).
(#2239 & ansible/galaxykit#39)

Ugh, merging right away, since merging the galaxkit PR removed the original branch


Add counters parameter to firewall function

Feature Request

**Is your feature request related to a problem?
N/a
Describe the solution you'd like
Rocketcyber added a counters parameter to their v3 firewall endpoint
Describe alternatives you've considered

Added with the 2.1.0 update

Added with the 2.1.0 update


Automatic update did not provide warning period before updating

see https://openziti.discourse.group/t/updates-for-zedw/599/4  user was USING ziti via ZDEW when 2.0.0 was published and did not get a warning and could not control when the upgrade happened. this is a poor user experience and we need to at least give them time to close out what they are doing. better yet is to let them delay the update by continually snoozing the update etc

This appears to be continuation/similar to #474


[BUG] Identify and access > Roles > selected role > Rules - unnecessary scroll bars

Describe the bug
There are scroll bars that are not necessary on the 'Identify and access > Roles > selected role > Rules' screen.
To Reproduce
Steps to reproduce the behavior:

Go to 'Identify and access > Roles'
Click on a role, then on the 'Rules' tab.

Expected behavior
There's no need for scroll bars on this page.
Screenshots

Desktop (please complete the following information):

OS: Windows 10 Pro x64
Browser: Chrome
Version: 80.0.3987.116 (Official Build) (64-bit)

Additional context
None

@vladimirpetrov made them automatic now, not show if not necessary. Please retest master.

Confirmed, closing the issue.


Move internal header contents to public headers

What
Where/Who

JIRA Issue
JIRA_TICKET_ID

People Involved
@colleague1, @colleague2

Notes on PR descriptions
Background
This PR is an example for solving the problem with Xcode 12.5 and Swift Package Manager. A more detailed explanation is on the Swift Forums: https://forums.swift.org/t/potential-issue-with-obj-c-class-extensions/47808
In short, methods found in LPActionContext-Internal.h and LPVar-Internal.h are publicly available in Xcode 12.4, but private in Xcode 12.5.
To reproduce the issue, create a new Xcode project, include the SDK as an SPM package and try calling one of the APIs found in the files mentioned.
Note: I'm unsure if the intention was to keep these methods and properties internal, or to expose them to customers? If I add the cocoapods version to a project, they certainly are public (even though the filenames suggest otherwise). This PR does not add any documentation to the methods and properties, it's just a PoC at this time.
Implementation
As a proof of concept, I just took the contents of the header files named xx-internal.h and added them to their respective class header files.
Testing steps
Updated our project to use a patched version of the SDK, and the compiler errors disappear.
Is this change backwards-compatible?
They are certainly API stable, but I can't say anything about binary stability at this point.

Closing this PR. We will create another one and will move only few methods now from Internal classes


Fix method Trie::doesWordExist()

Method Trie::doesWordExist() return true when word is complete otherwise false

Codecov Report

Merging #175 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master   #175   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files         134    134
  Lines        2485   2486    +1
  Branches      415    416    +1
=====================================
+ Hits         2485   2486    +1

Impacted Files
Coverage Δ

src/data-structures/trie/Trie.js
100% <100%> (ø)
:arrow_up:

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 5eb1195...a8929a0. Read the comment docs.

@iRipVanWinkle
Олежка, спасибо за фикс!


As a user, I want a self-service way to set up automated deployment

As noted in https://github.com/18F/cg-docs/pull/262#issuecomment-221426769, it isn't currently possible for non-admins to create deployer credentials for themselves. This is a barrier to users setting up automated deployment, which is a behavior we want to encourage.

As noted in Slack, creating tokens is preferable to creating dedicated users, since passwords expire.

Token conversation broken out to https://github.com/18F/cg-docs/issues/286.

To be taken up in future using UAA OAuth tokens now that it's possible, but closing for now.


Is there a way to add a default page convention that looks at the folder name

From @JohnGoldsmith on July 12, 2018 13:25
Hi,
Is there a way to add a routing convention for Razor Pages that allows you to use the parent folder name as a default route?
For example, if I have Pages/Products, folder structure I'd prefer to have a default page named Products.cshtml and for Pages/Services, Services.cshtml (as opposed to having to use Index.cshtml in both cases).
This is just to make things clearer in code ie the model would then be name ProductsModel and all open tabs in Visual Studio wouldn't have to say "Index.."
Thanks
John
Copied from original issue: aspnet/Home#3320

Thanks for contacting us, @JohnGoldsmith.
@kichalla, can you please answer the above question?

Here's the part that calculates routes for "Index.cshtml" Razor pages - https://github.com/aspnet/Mvc/blob/release/2.2/src/Microsoft.AspNetCore.Mvc.RazorPages/Internal/PageRouteModelFactory.cs#L67-L79. You could do something similar looking at the path in a route model convention. Alternatively, you could define absolute routes for your page e.g. @page /Products to avoid having to write a convention.

That's great.  Thanks very much @pranavkm
I've gone for the directive approach for the time being, as it seems to be the lightest touch and I'm not dealing with lots of pages.
For anyone else who (like me) wants more details on @page directive you can find it over here:

Thanks again Pranav.


interact - continuous update

from gitter chat: https://gitter.im/ipython/ipywidgets

Awesome, thanks!

It would be nice to strip the output of notebooks before committing it.

Why output should be removed? Do you having testing for these notebooks?
On Wed, Jan 6, 2016, 11:11 AM Sylvain Corlay<EMAIL_ADDRESS>wrote:

It would be nice to strip the output of notebooks before committing it.
—
Reply to this email directly or view it on GitHub
https://github.com/ipython/ipywidgets/pull/289#issuecomment-169392284.

Output it adds a lot of noise on git since it may change from execution to execution.

@SylvainCorlay it's okay, I'm going to add a PR that will add a few more things to these docs, I'll strip the output there.

OK, I'm adding the nested example as well. Continuous update as a second
button with True and False options.
On Wed, Jan 6, 2016, 11:17 AM Jonathan Frederic<EMAIL_ADDRESS>wrote:

@SylvainCorlay https://github.com/SylvainCorlay it's okay, I'm going to
add a PR that will add a few more things to these docs, I'll strip the
output there.
—
Reply to this email directly or view it on GitHub
https://github.com/ipython/ipywidgets/pull/289#issuecomment-169393708.

@denfromufa @jdfreder thanks!


[Bug]: egg-pterodactyl-arma3 - Headless Clients and localClient is getting overwrited

Panel Version
1.11.7
Wings Version
1.11.13
Service
arma3
Modified
No, I did not modify the egg
Expected Behavior
Have an Variable for headlessClients config Entry and for localClient Config Entry.
Actual Behavior
Both are always getting overwritten with { "<IP_ADDRESS>" }
Steps To Reproduce
Download the Egg
Change the Values in the server.cfg
Start the Server
Check server.cfg
Install logs
https://ptero.co/savarikemo

This is expected behavior of the Egg and it will not be adjusted. The Egg supports a feature where you can run Headless Clients in parallel with the server within the same container. Therefore, the server must be configured to look for them locally using the loopback address. Hosting Headless Clients externally (ie. at a different address) can introduce latency and diminishing returns, so it is not generally recommended. However, if you still wish to do this, you can simply edit the Egg by removing the following two lines from the "Configuration Files" section:
+ "maxPlayers": "maxPlayers = {{env.MAX_PLAYERS}};"  <- Remove trailing comma
- "headlessClients": "headlessClients[] = { \"<IP_ADDRESS>\" };",
- "localClient": "localClient[] = { \"<IP_ADDRESS>\" };"


⚠ Bofrost.lu has degraded performance

In 64c73a7, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 880 ms

Resolved: Bofrost.lu performance has improved in 3933cbd after 39 minutes.


Ensure random ads code branch is exercised by the load generator

Changes

Add a new product in a new category, "Books"
Update load generator script to fetch the new product, as well as request ads in the new book category, and request ads without any category, in order to test the random ad delivery code path
This is in preparation for adding custom business metrics to the ad service.

Merge Requirements
For new features contributions please make sure you have completed the following
essential items:

[x] CHANGELOG.md updated to document new feature additions
[x] Appropriate documentation updates in the docs folder

Maintainers will not merge until the above have been completed. If you're unsure
which docs need to be changed ping the
@open-telemetry/demo-approvers.

Hi I wonder if 'This is in preparation for https://github.com/open-telemetry/opentelemetry-demo/issues/73.', what metric do you want to provide if this pr is merged?

Hi I wonder if 'This is in preparation for #73.', what metric do you want to provide if this pr is merged?

I was thinking of an ad requests counter with two dimensions: request type of either targeted or "not targeted", and response type of either targeted or random.

Maybe it was mentioned in a SIG meeting that I missed, but can we get a detailed plan in the #73 issue? I'm ok with adding another product and category, but if this is in reference to that issue, I would like to understand what the bigger picture looks like.

Do we have permission to use this image from National Geographic?

Hmm, good point.  Any tips on where to get images where licensing won't be an issue?  Alternatively, we don't strictly need to add a new product in order have the load generator exercise the random ads code branch since the load generator is hitting the ads API directly.

Maybe it was mentioned in a SIG meeting that I missed, but can we get a detailed plan in the #73 issue? I'm ok with adding another product and category, but if this is in reference to that issue, I would like to understand what the bigger picture looks like.

Sorry, I didn't realize this would be appropriate fodder for the SIG meeting.  I'm happy to bring it up at the next meeting.

Do we have permission to use this image from National Geographic?

Hmm, good point. Any tips on where to get images where licensing won't be an issue? Alternatively, we don't strictly need to add a new product in order have the load generator exercise the random ads code branch since the load generator is hitting the ads API directly.

I've updated the new product to use a creative common licensed image.

Sorry, I didn't realize this would be appropriate fodder for the SIG meeting. I'm happy to bring it up at the next meeting.

Providing color in issue #73 itself would be fine as well. I just want to understand the big picture.

Sorry, I didn't realize this would be appropriate fodder for the SIG meeting. I'm happy to bring it up at the next meeting.

Providing color in issue #73 itself would be fine as well. I just want to understand the big picture.

I've added a couple of paragraphs describing my general proposal in #73, let me know what you think.

to be merged


Refactor required config params

Right now the way we handle required config params and defaults is a bit inconsistent. For example, the controller_count config param is required at the lokoctl level but optional with a default of 1 at the Terraform level:
https://github.com/kinvolk/lokomotive/blob/a6c6db6a6d3284635396c668ca46d8ff51fdc36f/pkg/platform/packet/packet.go#L58
https://github.com/kinvolk/lokomotive/blob/a6c6db6a6d3284635396c668ca46d8ff51fdc36f/assets/lokomotive-kubernetes/packet/flatcar-linux/kubernetes/variables.tf#L37-L41
I propose we rethink the way we handle required parameters. Here are some points to consider:

We should make a parameter required only if the user really needs to decide something about it. The more required params, the trickier it becomes for newcomers to deploy a cluster. We should prefer to use sane defaults as much as possible and allow "advanced users" to override them if they so desire. Keeping the configuration surface narrow should be a goal.
We should consider enforcing required/optional params in one place - lokoctl or Terraform. To me personally it makes more sense to enforce in lokoctl because we can fail faster this way, do our own validations (rather than use the Terraform ones) and provide meaningful errors.

We should consider enforcing required/optional params in one place - lokoctl or Terraform. To me personally it makes more sense to enforce in lokoctl because we can fail faster this way, do our own validations (rather than use the Terraform ones) and provide meaningful errors.

:+1: for lokoctl to be place for defaults. That way we can control the errors better. For e.g. if we rely on terraform to set default then errors will trickle up from terrform to lokoctl and that generally degrades the UX for user.

:+1: for lokoctl

@johananl
would something like this work
https://github.com/kinvolk/lokomotive/blob/imran/refactor-approach-no-8/pkg/platform/packet/packet.go#L101-#L116

@johananl
would something like this work
https://github.com/kinvolk/lokomotive/blob/imran/refactor-approach-no-8/pkg/platform/packet/packet.go#L101-#L116

@ipochi I'm not sure I understand. I'm seeing a NewConfig() function which returns a config struct with hardcoded values in some of the fields. Could you explain your intention?

@ipochi I'm not sure I understand. I'm seeing a NewConfig() function which returns a config struct with hardcoded values in some of the fields. Could you explain your intention?

@johananl, @ipochi idea is to have a single function, which will return the Platform default configuration. While decoding HCL, user configuration will be decoded on top of this custom configuration.

As long as the right params (and only them) are required and the rest have sane defaults, I don't have a strong opinion on how to implement that.


single press of delete key deletes whole word on linux

[x] I am on the latest Hyper.app version
[x] I have searched the issues of this repo and believe that this is not a duplicate

OS version and name: Ubuntu 14.04
Hyper.app version: Hyper 2.0.0-canary.8 (stable)
Link of a Gist with the contents of your .hyper.js:
Relevant information from devtools (CMD+ALT+I on macOS, CTRL+SHIFT+I elsewhere):
The issue is reproducible in vanilla Hyper.app:

Issue
On linux, a single press of the delete key deletes the entire next word, rather than the next letter. This is due to a typo in the linux keymap: https://github.com/zeit/hyper/blob/canary/app/keymaps/linux.json#L44

Thanks for reporting this @transcranial ❤ Fixed in #2539 so it will be in the next canary release.

Hi,
Sorry to reopen this, but I'm having the same problem on windows on the same version of Hyper and there seems to be the same type on the windows keymap here.
Thanks.


Remove  from config table

Closes #1041

Coverage remained the same at 86.608% when pulling 9a83df6045395c075f87f4708bd1e13bce602fa1 on na3d:remove-config-th into 269b5d6fbf342a1ee34dd57161a9f179088c6271 on linode:master.


Created static programming for the web page

If it's just one page it probably doesn't need to have its own TOC page.

Deleted the index page

Deleted the index page

LGTM


unittests?

what would be useful unittests?
any simply generatable test signals?

I think including some samples is a good idea. I have a few that I have created in the past, and I also just created an artificial test file to trigger the overflow that you discovered. Assuming GitHub allows (they're big) I will create a folder for them and include them.

@dbry : in case the files are really big, a separate "audio-stretch-data" repository might be appropriate

@hayguen I only made the samples big so that I could also use them for performance testing. I think I can easily make short versions for all other testing.

@dbry : then, i'd suggest short files .. and amending a loop count option as command line argument.

it's not a unittest .. but meanwhile having the build and test scripts as well as the audio sample files.


Automated cherry pick of #5459: add bucket connection test

Cherry pick of #5459 on release/3.0.
#5459: add bucket connection test

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/3.0@2c35de7). Click here to learn what that means.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             release/3.0   #5461   +/-   ##
=============================================
  Coverage               ?   6.87%
=============================================
  Files                  ?     535
  Lines                  ?   91144
  Branches               ?       0
=============================================
  Hits                   ?    6262
  Misses                 ?   84361
  Partials               ?     521

Flag
Coverage Δ

#aFlag
6.87% <0.00%> (?)

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 2c35de7...a4ef41c. Read the comment docs.


Roll Engine from d3427ddb8546 to fdc457d34289 (2 revisions)

https://github.com/flutter/engine/compare/d3427ddb8546...fdc457d34289
2020-06-04<EMAIL_ADDRESS>Revert "onDisplayPlatformView JNI (#18786)" (flutter/engine#18826)
2020-06-04<EMAIL_ADDRESS>onDisplayPlatformView JNI (flutter/engine#18786)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md

Commit queue failed; closing this roll.


Mapping  table valued function with entity framework code first

I'm trying to execute a table valued function and get a list of entities .
[DbFunction("APIDBContext", "GetMyEntities")]
public virtual IQueryable<MyEntity> GetMyEntities(Nullable<int> userId)
            {
      return ((IObjectContextAdapter)this).ObjectContext.CreateQuery<MyEntity>(string.Format("[{0}].{1}", GetType().Name,
                        "[GetMyEntities](@userId)"), userIdParameter).AsQueryable();

    }

Seems like it I need to have same names for the columns i'm returning from the database even though I have specified column mappings in the model builder.
I'm getting the exception
Invalid column name 'MyColumn'
But I have mapped this column in the model builder ,
Property(e => e.MyColumn).HasColumnName("My_Column");
Is this not supported yet in entity framework code first ?

@jayanthakgjls Why is this closed? Is it fixed, duplicate? I got same issue.

@bogatykh - This repo is for EF Core. The original issue posted was for EF6. If you are having issue with EF6 then please file a bug report at https://github.com/aspnet/EntityFramework6

@smitpatel Is this issue fixed in EF Core?

Code posted above doesn't compile in EF Core. If you are using EF Core, then please file a new issue with the code details and error you are getting.

@bogatykh the only way to use TVFs currently in EF Core is to write a SQL query that performs a SELECT and pass it to the FromSql() method:
var q = context.Products.FromSql("SELECT * FROM MyTVF({0}), argument);

There is no first class support for mapping a TVF in the model. That said, SELECT queries passed to FromSql() are composable using LINQ and at execution they go through the same mapping as any other queries, e.g. the TVF needs to have the same column names as the corresponding table in the model.
So, I guess you can say that this issue does not exist in EF Core.

@divega Can you expand on your example a bit more?  Is Products a DbQuery?

also, are WHERE clauses not possible on a TVF unless it's transcribed in the string provided to FromSql ?

=>In Context File
public DbQuery Products{ get; set; }
var productList = context.Products.FromSql("SELECT * FROM MyTVF({0}), argument).ToList();


static file delivery in foxx with to verbose error messages

As hello foxx documents:

/* no special code required, the manifest contains

  "files": {
    "/": "files"
  },

*/```
can be used to deliver static files. This in term will report the full disk path to the webbrowser, which it shouldn't for security reasons. Local disk paths should only be exposed to the ArangoDB error log, the UI needs to only report back the queried URL to be non existant in error case.

closed via 6c25aa764835716b66829fe854bf2bf85a3bee2e


Introducing ReplyMarkup.php entity

I've wrote this class to implement easily interfaces with reply markup.
It support Keyboard, Hidekeyboard and ForceReply.
I've also attached some new function where is possible to see how use this class into the commands.
The code passed the test described in the comtributing.md

@MBoretto why closed?

I've merger ReplyKeyboardMarkup ReplyKeyboardHide and Force Reply in one class. Maybe it's better to build three differet classes.
Then i build a custom function to generate json string instead of use the one inherited from Entities. I woul like to rewrite it. That's all!

Ok :) I appreciate your help


Explicitly list the names of plugins

Just adding in the plugin names explicitly in the docs to help clarify the name that needs to be passed in to the loader line.

Hi,
Sorry I missed this one when you first posted it! Thank you for this - it's something I'd meant to done but must have overlooked :)


++!Here’s How To Watch Germany vs Canada, Live Free World Junior Hockey Championship Game on 29 December 2024

On Sunday, December 29, 2024, Canada vs Germany will face a thrilling Hockey showdown that promises to be nothing short of spectacular. Both teams have been putting on impressive performances this season, and this game is bound to be a memorable clash between two formidable contenders.
🅻🅸🆅🅴🔴▶ CLICK HERE TO WATCH LIVE ONLINE
🆆🅰🆃🅲🅷🔴▶ CLICK HERE TO LIVE STREAM
Event Details:
2025 World Junior Championships 2024
Date: Sunday, December 29, 2024
Time: In Progress
Stream: Canada vs Germany Live On Demand (Free Trial)
Canada vs Germany have established themselves as powerhouse programs in 2025 World Junior Championships within their region. Their rich histories and impressive track records have made them perennial contenders in local competitions.
Canada vs Germany, known for its tenacious spirit, boasts a proud Hockey tradition. Their commitment to excellence has led them to multiple championships and a reputation for producing top-tier athletes. The team’s dedication to skill development, teamwork, and sportsmanship has cemented their place among the elite.
Toledo , on the other hand, represents 2025 World Junior Championships with consistent High School-level competition. Their journey to success has been defined by discipline, hard work, and a strong sense of community.
Every great Hockey matchup is defined by its star players. Both Canada vs Germany have their standout athletes who are expected to play pivotal roles in the game.
Both teams have had impressive journeys leading up to this showdown. Let’s take a closer look at their respective patHigh School to the big game.
The Canada vs Germany matchup started the season with a strong showing in non-conference play, showcasing their offensive firepower and defensive tenacity. They have built a reputation for their fast-paced style of play, often outscoring opponents by a significant margin.
As they entered conference play, Canada vs Germany faced tough competition, but their resilience and teamwork prevailed. Close victories against their rivals added to the anticipation surrounding this matchup with each other.
Both teams have been equally impressive on their journey to the showdown. Their well-rounded team has consistently demonstrated a commitment to defense and ball movement, resulting in several dominant victories.
The success of Canada vs Germany can be attributed to their ability to adapt to different opponents’ styles of play. They have faced tough challenges throughout the season, and each game has been a learning experience, preparing them for the ultimate test against each other.
The buzz surrounding the Canada vs Germany matchup is palpable. The anticipation for this game has reached a fever pitch, with fans and community members eagerly awaiting the tip-off.
Local media outlets have covered the game extensively, with previews, player interviews, and expert analysis. The excitement has spilled over into social media, where fans from both sides have engaged in friendly banter and predictions.
2025 World Junior Championships is more than just a sport; it’s a community event that brings people together. Both Canada vs Germany have enjoyed unwavering support from their fans, families, and local businesses.
The Canada vs Germany faithful have been a sea of High School colors at every home game, creating an electric atmosphere that fuels the team’s performance. Similarly, both communities have rallied around their teams, turning their gymnasiums into fortresses of support.
One of the intriguing aspects of this matchup is the clash of playing styles. Both teams’ up-tempo, run-and-gun offense contrasts with each other’s disciplined, half-court approach. The battle between speed and precision promises to be a spectacle for Hockey purists.
Both teams’ High School-scoring games have often relied on fast breaks, quick passes, and sharp shooting from beyond the arc. The Canada vs Germany methodical approach involves patient ball movement, strong defense, and High School-percentage shots.
The coaching strategies will play a significant role in determining which style prevails. Both coaches have been preparing their teams meticulously, studying each other’s strengtHigh School and weaknesses to devise winning game plans.
🅻🅸🆅🅴🔴▶ CLICK HERE TO WATCH LIVE ONLINE
🆆🅰🆃🅲🅷🔴▶ CLICK HERE TO LIVE STREAM
In Hockey, momentum can shift in the blink of an eye. The ebb and flow of the game will be critical, and key runs by either team can swing the outcome. The first quarter will set the tone, with both teams vying for an early advantage.
Both Canada vs Germany will look to establish their tempo early, pushing the pace and capitalizing on fast-break opportunities. On the other hand, they will focus on controlling the game’s tempo, limiting turnovers, and executing their half-court offense efficiently.
Rebounding and interior play will be another pivotal aspect of this matchup. Both teams have dominant big men and women who can impact the game on both ends of the floor.
The Canada vs Germany is known for its prowess in the paint, grabbing offensive rebounds and converting second-chance opportunities. As the game unfolds, expect both teams to make defensive adjustments to counter their opponent’s strengtHigh School. Canada vs Germany may employ a full-court press to disrupt the half-court offense, while the area might switch to a zone defense to limit the fast breaks.
The ability to adapt and execute defensively will be crucial, as it can lead to turnovers and transition baskets – factors that often decide the outcome of High School-stakes Hockey games.
In games of this magnitude, it’s often the unexpected heroes who step up and make a difference. These X-factors can be role players coming off the bench, making timely three-pointers, grabbing crucial rebounds, or playing lockdown defense.
Both bench depth has been a strength all season, with several players capable of providing a spark when needed. The Canada vs Germany has shown resilience and composure in close games, which can be a valuable asset in pressure situations.
As the game enters its final moments, the tension will be palpable. Fans from both sides will be on the edge of their seats, and every possession will carry immense weight.
The crowd noise will reach a crescendo as the players execute their final plays. The players themselves will rely on montHigh School of preparation, countless hours of practice, and their unyielding determination to secure victory.
While predictions are always a tricky business in sports, one thing is certain: the Canada vs Germany showdown promises to be an unforgettable Hockey spectacle. Regardless of the final score, both teams have already earned the respect and admiration of their communities for their hard work and dedication.
2025 World Junior Championships is more than just a game; it’s a showcase of young talent, a testament to teamwork, and a celebration of community spirit. The Canada vs Germany matchup on December 27, 2024, exemplifies the essence of High School sports – the pursuit of excellence, the thrill of competition, and the unity of communities rallying behind their teams.
As Canada vs Germany take the court, they carry with them the hopes and dreams of their High School, their families, and their fans.


Image upload handler improvements

File upload view changes required for better drag-n-drop support.

Include original image URL on AJAX file upload
Move ajax_upload view out of ClipboardAdmin to allow to import it

(Please, do not merge yet. Would go into the release after the next current one, which would probably be 1.1.1.)

LGTM :+1: