[WIP] Optimize is_card_number to use array

What does this PR do?
A brief description of the change being made with this pull request.
Motivation
What inspired you to submit this pull request?
Additional Notes
Anything else we should know when reviewing?
How to test the change?
Describe here in detail how the change can be validated.

Codecov Report
Attention: Patch coverage is 91.66667% with 1 line in your changes missing coverage. Please review.

Project coverage is 71.79%. Comparing base (ebcd997) to head (850a88a).

Additional details and impacted files
@@                         Coverage Diff                          @@
##           ban/stabilize-credit-cards-bench     #678      +/-   ##
====================================================================
- Coverage                             71.80%   71.79%   -0.01%
====================================================================
  Files                                   271      271
  Lines                                 40992    41000       +8
====================================================================
+ Hits                                  29434    29436       +2
- Misses                                11558    11564       +6

Components
Coverage Δ

crashtracker
20.28% <ø> (-0.06%)
:arrow_down:

crashtracker-ffi
9.07% <ø> (ø)

datadog-alloc
98.73% <ø> (ø)

data-pipeline
91.92% <ø> (+0.06%)
:arrow_up:

data-pipeline-ffi
0.00% <ø> (ø)

ddcommon
82.73% <ø> (ø)

ddcommon-ffi
69.12% <ø> (ø)

ddtelemetry
59.10% <ø> (ø)

ddtelemetry-ffi
22.13% <ø> (ø)

dogstatsd
88.86% <ø> (ø)

dogstatsd-client
79.77% <ø> (ø)

ipc
82.65% <ø> (ø)

profiling
84.30% <ø> (ø)

profiling-ffi
77.46% <ø> (ø)

serverless
0.00% <ø> (ø)

sidecar
36.09% <ø> (ø)

sidecar-ffi
0.00% <ø> (ø)

spawn-worker
50.36% <ø> (ø)

tinybytes
94.77% <ø> (ø)

trace-mini-agent
72.45% <ø> (ø)

trace-normalization
98.25% <ø> (ø)

trace-obfuscation
95.77% <91.66%> (+0.03%)
:arrow_up:

trace-protobuf
77.67% <ø> (ø)

trace-utils
93.08% <ø> (ø)


Fees missing when using GetTransactionsAsync

When I get a Send transaction using GetTransactionsAsync then there isn't anywhere information about the transaction fee, the amount has the fee included instead.
According to the https://docs.cdp.coinbase.com/coinbase-app/docs/api-transactions there is transaction_fee within the network object.
To Reproduce
get send transactions which contained fee using GetTransactionsAsync
Expected behavior
I would expect at minimum e.g. a propery like TransactionFee under Network

When I get a Send transaction using GetTransactionsAsync then there isn't anywhere information about the transaction fee, the amount has the fee included instead.
According to the https://docs.cdp.coinbase.com/coinbase-app/docs/api-transactions there is transaction_fee within the network object.
To Reproduce get send transactions which contained fee using GetTransactionsAsync
Expected behavior I would expect at minimum e.g. a propery like TransactionFee under Network

When I get a Send transaction using GetTransactionsAsync then there isn't anywhere information about the transaction fee, the amount has the fee included instead.
According to the https://docs.cdp.coinbase.com/coinbase-app/docs/api-transactions there is transaction_fee within the network object.
To Reproduce get send transactions which contained fee using GetTransactionsAsync
Expected behavior I would expect at minimum e.g. a propery like TransactionFee under Network

Hey there @michalkovy,
To address the issue of missing transaction fee details in GetTransactionsAsync, ensure the TransactionFee property is properly implemented within the Network object, as specified in the Coinbase API documentation. This would allow fees to be distinctly reported instead of being included in the amount.

Hi, it looks like, according to the docs, that property is available after February 7

I think the fee was available even before in the API just it was represented differently. I used this https://github.com/bchavez/Coinbase NuGet before (which was not updated since 2021) and I had the fee data there


Cost?

Is there any rough estimate of what this lambda-selenium approach costs on a production workload? I'm wondering if reserved instances would be significantly less expensive.

If you compare raw compute time used, i.e. 1hr of billed Lambda compute time at max settings vs ec2, they are pretty close to each other. However in my situation, it reduces build times immensely since we can run hundreds of concurrent tests, so there are some compute savings due to our infrastructure setup. What gets interesting is for team productivity, which is harder to quantify. In terms of Selenium/Chrome, it has better stability since we only use the function for a test run and  thrown it away after, cutting down on all the mystery errors you get reusing grid nodes. If you are going to run a large ec2 based selenium grid, I highly recommend https://github.com/groupon/Selenium-Grid-Extras.

Selenium-Grid-Extras is unfortunately unmaintained. I had been looking at https://github.com/zalando/zalenium possibly comboed with https://aws.amazon.com/blogs/aws/aws-fargate/
lambda-selenium is interesting though. I'll probably try it out in the future.

Yeah, it seems so. It was nice while it lasted. Fargate looks really promising. We'll be posting some better examples soon with the full version of Chrome/Firefox now that we have some of their issues with Lambda ironed out.


[Tabs] Don't animate scroll on first render

[ x] I have followed (at least) the PR section of the contributing guide.

close: #25314

Did you have a look at the failing test case?

Did you have a look at the failing test case?

i have take a look at ci/circleci and did a yarn test:karma but no i have found no hint on how to solve this.
Do you got any idee how i can solve this. and for futre pull request

I had a look at the tests, it was simply an issue with the setup. In JSDOM we were forcing a specific configuration that, once rendered in the browser, does no longer hold true. Instead, I set the right width style property.

nice to hear thanks for the update. do you got any suggestions how i could test localy and find this thing for future pull requests. i like doing contributions for material ui

do you got any suggestions how i could test locally

@manziEric It should be covered in https://github.com/mui-org/material-ui/blob/next/test/README.md


Better JavaScript completion thru JSDOC

It would be very interesting to make the intellisense, completion and documentation popup (for a method for instace) better thru the ability of VSCode to better understand standard jsdoc like @description and such to give some text description to how to use a function, not only @param and parameter typing.

@dbaeumer Is there already a feature request for this?

If so then in the TS land. I will move the issue since the feature request needs to be implemented there.

This issue was moved to Microsoft/TypeScript#9518


Layout should not be called if there are coordinates in Extended SMILES

Steps to Reproduce

Open Ketcher
Paste CC1CCCC(C)C=1N1C([C@@H](Cl)C)=CC=C1 |a:7,&1:10,r,wD:7.6,10.11,(13.73,-9.95,;14.59,-9.45,;16.07,-8.70,;16.18,-6.82,;17.23,-7.80,;16.33,-9.45,;17.19,-9.95,;15.46,-9.95,;15.46,-10.95,;14.76,-11.67,;13.77,-11.53,;13.40,-10.60,;13.16,-12.32,;15.23,-12.55,;16.22,-12.38,;16.36,-11.39,)|

Actual behavior
Layout is called when Extended SMILES have coordinates
Expected behavior
Layout should not be called if there are coordinates in Extended SMILES
Screenshots

Desktop (please complete the following information):

OS: Windows 10
Browser Chrome
Version 112.0.5615.138 (Official Build) (64-bit)

Ketcher version
Version 2.15.0-rc.4
#186288932

Pre-tested. Looks like bug is fixed.


JSON schema coercing empty string to nil

Using the json schema, empty strings are being coerced to nil.
class MyContact < Dry::Validation::Contract
 json do
   optional(:value).maybe(:string)
 end
end

MyContract.new.call({ value: '' })
#<Dry::Validation::Result{:value=>nil} errors={}>

Since this is an optional maybe string, it should not coerce to nil.
Affects my production application: NO
Ruby version: 2.6.2
OS: MacOS 10.15.2

This seems to be unique to the JSON processor:
schema_json = Dry::Schema.JSON do
  optional(:value).maybe(:string)
end

schema_json.call(value: '')
#<Dry::Schema::Result{:value=>nil} errors={}>

schema = Dry::Schema.define do
  optional(:value).maybe(:string)
end

schema.call(value: '')
#<Dry::Schema::Result{:value=>""} errors={}>

@rawburt it works (and should work) this way with Params too

yeah plain schema has no coercions, that's why there's a difference in behavior

I guess the solution would be to check in maybe if the type spec points to a str? predicate, and if yes, then the type should be set to the nominal string, instead of params string.

It turned out this is coercion behavior from dry-types so it should be fixed there:
(byebug) type_registry[:nil]
#<Dry::Types[Constructor<Nominal<NilClass> fn=Dry::Types::Coercions::JSON.to_nil>]>
(byebug) type_registry[:nil]['']
nil

ref https://github.com/dry-rb/dry-types/issues/396


More SEO Friendly Section links

Fix #500

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Hello @ladhadha,Thanks again for the PR! It's HACKTOBERFEST! We want to show our appreciation by sending                                              you some special Hacktoberfest swag. If you have not already, could you please fill out                                              this form so we can send it to you? Thanks!                                             Team SendGrid DX


feat: Support schema inference for external tables

Adds support for infering schema of an external table. Fixes #1537
Co-authored-by: Mark Starc<EMAIL_ADDRESS>Test Plan

[ ] acceptance tests

[X] unit tests

References

Fixes #1537

@kalfonso What happened to this PR? Why was it closed? For me it looks like a very nice feature and I am very interested in using it.


How to read previous messages?

Hi,
I'm trying to read previous messages as well, would you suggest a way to do so?
Thank you

working on this


Replace #[inline(always)] by #[inline] due to the later being conside…

…red harmful
Signed-off-by: Daniel Egger<EMAIL_ADDRESS>

("being considered harmful" is not a very descriptive commit message / PR title, IMO.)
For posteriority, could you please add a comment (or a link) with the rationale about the change to this PR?
@homunkulus r+

:pushpin: Commit 73cd404 has been approved by japaric

:hourglass: Testing commit 73cd404 with merge 895722b...

Well, according to https://rust-lang-nursery.github.io/rust-clippy/master/index.html#inline_always it is considered harmful: It increases compile time and may increase size as well as decrease performance. I'm afraid I can't come up with a better label than "considered harmful".

:sunny: Test successful - status-appveyor, status-travis
Approved by: japaric
Pushing 895722b6ec85cd7dae5d37eb9622a3e665ba6a27 to master...


I think the function "reorder_pts" is problematic

According to the definition of the function reorder_pts, It is entirely possible that, to make sure the order of the four edge center points(tt, rr, bb, ll), using np.argmax/np.argmin will make two of their indexes(l_ind, r_ind, t_ind, b_ind) have same value. In other words, one points may be both the leftmost and the uppermost at the same time. So if carrying on these operation, you may get repetitive points.
I think if you draw some rectangle boxes with random orientation on one paper, you may find that there will always has two edge center points (x1, y1) & (x2, y2) that meet the condition: x1 <= x2 and y1 < y2. Thus we can determine the top point tt(x1, y1) and the bottom point bb(x2, y2), then the left point ll(x3, y3) and the right point rr(x4, y4) meet the condition: x3 < x4 and y3 >= y4.

This operation only happens when theta in [-90.0, -0.0, 0.0].


[TASK]: Add to privacy and terms pages

Task Summary
Add privacy and terms of use info

Go for it

How should I get assigned to this?

@BOSE-git is this something you wanted to work on still?

Sure, why not!

Hello, Is this issue or can I work on it??

@TyrinH i'm new to open source, i'd like it fix this current issue.


Improved Dockerfile layer caching

Description
All dependency installation happens before copying code. This ensures that simple code changes, without dependency changes, result in the cached dependency layers being reused during image rebuilds.

🔬 How to test

Build the image.
Change some code, but no dependencies.
Build the image again.
Marvel at the speed.

📸 Cypress Screenshots (if applicable)

Changes include

[ ] Bug fix (non-breaking change that solves an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Breaking change (change that is not backwards-compatible and/or changes current functionality)
[ ] Documentation
[ ] Release
[x] Other

@itsojon are you open to contributions? If so can you take review this and my other outstanding reviews? If not, can the README be updated stating as much?


[FLINK-12612][coordination] Track stored partition on the TaskExecutor

Based on #8630, #8654 and #8680.
What is the purpose of the change
With this PR TaskExecutors keep track of which partitions are stored locally, and use this information to
a) delay the termination of their jobmanager connection until all these partitions are released
b) release all partitions for a given job if the jobmanager connection is terminated unexpectedly.
To track the partitions a JobAwareShuffleEnvironment was introduced, which is an alternate version of the ShuffleEnvironment interface. This environment is aware of the concept of jobs, something that was refactored out of the shuffle environment over the past weeks.
The default implementation simply wraps a ShuffleEnvironment, and adds additional book-keeping for partitions.
The book-keeping is primarily kept up-to-date via listeners that are introduced into the ResultPartitionWriters by wrapping the writers returned by the backing environment.
General Idea
To explain how things work, let's imagine we have just submitted a job:
Upon connecting to a JobManager the TaskExecutor uses #markJobActive to inform the environment about jobs it should keep partitions for. If this flag is not set for a given job, and partition that finishes will be automatically released. See the special-cases section for an explanation.
Tasks are being submitted, and their ResultPartitionWriters are being created. The JobAwareShuffleEnvironment wraps the writers to introduce listeners that keep it in the loop about lifecycle updates for the partitions: whether they were setup, finished and/or failed.
Tasks start running, ResultPartitonWriters are being setup, and the environment adds the created partitions to the book-keeping as in-progress partitions.
If a task fails it will fail the ResultPartitonWriters (which releases the partition), and the environment removes the failed partitions from the book-keeping.
if a task finishes it will finish the ResultPartitonWriters and the environment now considers these partitions are finished. From this point on the TaskExecutor is responsible for issuing the release call.
At some point an external actor will issue call to release partitions to the TaskExecutor, which forwards these to the environment. Partitions are released and removed from book-keeping.
If no more partitions and slots are held for a given job the TaskExecutor terminates it's connection with the JobManager.
Abnormal events / Special cases
The big special case is losing the connection to the jobmanager. In this case the TaskExecutor will cancel all tasks, mark the job as inactive, and release all finished partitions it has at this point in time.
Partitions that are in-progress will NOT be cleaned up by the TaskExecutor to not interfere with the normal shutdown procedure of tasks. Since the task cancellation happens asynchronously releasing in-progress partitions would likely result in undefined and misleading exceptions.
In a normal cancellation case the task will fail all of it's partitions and they will be cleaned up automatically.
However, it can happen that a task finishes after the job has already been canceled, in which case it will actually finish the partitions. This is where the isActive flag for jobs comes in. This flag is unset right away when the job is canceled; any partition that is finished for a job that isn't active is automatically released by the JobAwareShuffleEnvironment.
The why-is-it-implemented-like-this section
What about concurrency?
The low-level data structure for keeping track of partitions is the ResultPartitionTable, which is thread-safe. This class is accessed by Task threads via listeners, and via TM threads when release calls have been issues. This is where the majority of concurrent accesses will take place.
In the JobAwareShuffleEnvironment there is one additional point of concurrent access which is the set of active jobs, which is queried by task threads when finishing partitions and updated by the TM thread when establishing/terminating jobmanager connections.
No additional locks have been introduced, and as such there should not be any risk for deadlocks.
Is it safe to have the Task thread release partitions if the job is inactive?
Yes, as this is equivalent to failing a partition, which Task threads already do.
Could we not handle this on the TaskExecutor level without wrappers?
Potentially, but the problem here is that the TaskExecutor is quite far away from what is actually happening in the task, and keeping the book-keeping in sync introduces a few challenges.
For example, there is no good point at which one can add partitions to the book-keeping. Partitions are setup within the tasks run() method, at some point after it was started.
Without introducing a callback in the Task you inevitable end up in an inconsistent state, and have to ensure that whatever happens in task this situation is rectified eventually. This introduces several new edge-cases that we could do without. (For example, a task can be canceled right away, in which case it neither sets up partitions nor issues any callback regarding it's final state)
Could we not handle this inside the ShuffleEnvironment?
Yes, we certainly could. But this would require re-introducing the concept of jobs into the environment that were removed recently, and additionally require of every implementation to implement this logic correctly, even though it is be the same across all implementations.

Note that the commit about maintaining the jobmanager connection must not be merged until the release logic on the JM is in place, as without these the TM would never terminate the connection.

Rebased the PR now that the shuffle environment is in master.

@zentol I have finished reviewing the whole PR, thanks for the updates!

rebase on #8789.


Bug(?): LS(LC(LALT)) type mappings in zmk_keycode_map stopped working

Hey @caksoylar , been a minute!
Any idea when / why this stopped working? https://github.com/minusfive/zmk-config/blob/15c42330e30aaa12ecbbe742116d809d283ec65a/keymap_drawer.config.yaml#L613
It used to output the defined icon, now it just outputs the inner mod (LALT). Been wrecking my brain trying to sort it but haven't been able to figure it out.
Much appreciated.

Hey, hope all is well!
This was a breaking change in v0.16 when I added special parsing for modifier functions. Have a look at the release notes and the workaround there: https://github.com/caksoylar/keymap-drawer/releases/tag/v0.16.0
Let me know if it works out!

Gotcha, thanks! Right, so in my case I'm using them as pure mods in my HMRs on all 3 layers, e.g.
#define MEH LS(LC(LALT))

/ {
  keymap {
    compatible = "zmk,keymap";

    layer_1_colemak_dh {
      bindings = <&hrml MEH G &hrmr MEH M>;
    };

//    layer_1_colemak {
//      bindings = <&hrml MEH D &hrmr MEH H>;
//    };

//    layer_1_qwerty {
//      bindings = <&hrml MEH G &hrmr MEH H>;
//    };

    layer_2 {
      bindings = <&hrml MEH N0 &hrmr MEH LEFT>;
    };

    layer_3 {
      bindings = <&hrml MEH F10>;
    };
  };
};

(You can replace the &hrml/rs with &mts, I'm using positional HRMs. Don't know why I'm telling you this 🤣).
So I'm not sure that case is supported, right? Like, using mod combinations as pure modifiers themselves? I'mma play with it for a bit, see if I can figure it out.
Here's the YAML output BTW: https://github.com/minusfive/zmk-config/blob/4db79477174f552576476c38ec68e39103e51906/img/corneish_zen.yaml#L21-L22

Yeah, I don't think the modifier fn configs would work for that. You can add each of those to raw bindings map but as I suspected, it will be inconvenient.
I'd like to reinstate the previous behavior, I will have a look when I am in front of a computer (sometime this week).

That's awesome, thanks! But yeah, no rush — workaround in place 😉:

https://github.com/minusfive/zmk-config/blob/7d1972ec749abfe14e522e591ba849143aaeaee5/keymap_drawer.config.yaml#L422-L443
https://raw.githubusercontent.com/minusfive/zmk-config/7d1972ec749abfe14e522e591ba849143aaeaee5/img/corneish_zen.svg
https://github.com/minusfive/zmk-config/blob/7d1972ec749abfe14e522e591ba849143aaeaee5/img/corne.yaml#L21-L22

Thanks!

I'm OK to close this issue, unless you want to keep it open to work on it?

Glad you got the workaround at least. Happy to see you updating 😄
Yeah, let's keep it open for now.


Hyperlink component

Currently our Hyperlink component is a button.
With latest version, Hyperlink implementation will send url and target properties as well.
So, if you receive an url and optional target property, use a real hyperlink instead of a button!
In Safari, the popup will be blocked if you use a button. In Chrome and Firefox, this is working!

Is there a test available? I've checked in the demo's components screen but there is no url or target.

The components work screen uses Hyperlink, but it doesn't change the url and doesn't use target.

New screen: Hyperlink is available

looks the same as before and now is a "" element

please test with safari

link looks right, but you still send the action event?

This is not a bad thing but the server component shouldn't send you a link back.

Yes, I thought I should try to use every event the button also sent, like focusGained and focusLost aswell for example

Works in my environment.


The v0.1.2 file is empty.

It's just a 425 Bytes empty file and contains no skins.

@Miraakz9 Thanks for the issue. The problem is fixed now. Have a good day


0x801c044f not listed...

Cause and mitigation please?

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

ID: 6b4f9133-887f-17a6-95fc-3712de8a3fa1
Version Independent ID: 09d34ce8-1590-8202-3e70-e6299dadbb65
Content: Windows Hello errors during PIN creation (Windows 10)
Content Source: windows/security/identity-protection/hello-for-business/hello-errors-during-pin-creation.md
Product: w10
GitHub Login: @DaniHalfin
Microsoft Alias: daniha

It seems the solution is "wait". It started working again before support was available.

samehere 0x801c044f

We also got this error, and it seems to be something regarding MFA not working or maybe canceled by user

@officedocsbot assign @e0i

Hello, @spankin @robertrenner @jesoj  Thank you for submitting feedback.
In our understanding the error code that you have noted might actually have surfaced from other services rather than Windows Hello itself.
Find below a quoted reply from our Content Developer:
DaniHalfin wrote:

For error codes unlisted here, in most cases those are transient error codes
being surfaced by other services in the Windows Hello dependency chain.

We are working on something to improve the user experience in that regard,
but that will not solve the issues you are experiencing.

If we find a reoccurring error codes that isn't covered here and should be
(meaning, in most cases, it is coming from Windows Hello and not other
components) we'll make sure to add it to this page.

Also note that the feedback section that you have used to submit this issue is intended for product documentation issues only.
Please consider consulting to Feedback Hub for any of your suggestions, questions or feature requests to help Microsoft improve your experience. Find the link to Feedback Hub below:
Microsoft Windows 10 Feedback Hub
We are closing this issue but feel free to re-open it if you feel that we might have missed something.
Have a nice day.

@officedocsbot close


Add panyarat high school to the repo

add panyarathighschool.txt to the repository
http://www.panyarathighschool.ac.th/

@sukritws40 Pull request merged. Thank you!


Need support for MultiheadAttention op

Describe the bug
Need support for MultiheadAttention layer for PETR model
Torch link: https://pytorch.org/docs/stable/generated/torch.nn.MultiheadAttention.html
Model card: https://github.com/tenstorrent/tt-metal/issues/14296
Torch example:
multihead_attn = nn.MultiheadAttention(embed_dim, num_heads)
attn_output, attn_output_weights = multihead_attn(query, key, value)

device: WH N150

@keerthana-r-mcw please add details and proper description to this issue, add torch examples ...

@punithsekar FYR

Closing this issue, since it's implemented


Service account resource

Added a resource for managing service accounts.
Vendor changes were put in a separate commit and are still a WIP.  That commit can be ignored. Once the necessary api changes land in production, I'll update the PR to include the correct vendor changes from the cockroach-cloud-sdk.
Also, added here is a commit to add some logging of api invocations.
Commit checklist

[x] Changelog
[x] Doc gen (make generate)
[x] Integration test(s)
[x] Acceptance test(s)
[x] Example(s)

This change is

@rgcase I rebased this off main and also updated it to not overwrite the description if its not set in the resource.  Let me know what you think.


Sidebar not showing on deployed version

In vitebook:dev mode it works fine but the deployed version with vitebook:build it shows an error + the sidebar is not showing. There is also a css media query that makes it invisible.

Looking into it now 👍

hydratable compiler option was missing, add this to your config to resolve. Let me know if issue persists.


Refactor/change package export

Codecov Report

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

@@         Coverage Diff          @@
##           master   #12   +/-   ##
====================================
  Coverage      90%   90%
====================================
  Files           1     1
  Lines          40    40
====================================
  Hits           36    36
  Misses          4     4

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 d3640a5...11e303a. Read the comment docs.

Codecov Report

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

@@         Coverage Diff          @@
##           master   #12   +/-   ##
====================================
  Coverage      90%   90%
====================================
  Files           1     1
  Lines          40    40
====================================
  Hits           36    36
  Misses          4     4

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 d3640a5...11e303a. Read the comment docs.

#11


brpc 2.5.1 Demo cannot run.

I use protobuf-2.5 to generate ECHO.java file，this process is right. But when i open the RpcClientTest.java and RpcServerTest.java etc. There are many errors in these files. So what's problem with them, and what should i do to solve these problems?

@MrKingHH you need to run local_build.sh to generate proto files.

ok , i will try it in my linux operating system.  I  used it in windows before.

please use mvn clean install -DskipTests under window.

Build project successfully. But there are still existing some errors. The red sections seem to indicate that there are not these methods in class RpcClientOptions.

And i check it out in RpcClientOptions.java. There indeed not existing these methods.

@MrKingHH you need install Lombok plugin.

ok. when i have installed lombok plugin in IDEA, the project can run successfully. Thanks a lot!

But i think it would be useful to add some more specific informations about how to start this project, and it will be beneficial for brpc-java's better development.


update docs

Please, consider updating the official site, as it created the impression that the project is dead (latest news several years ago, outdated genomes are used by default). And please give examples with fasta and bed files as "bb" is not well known format

I think this project is indeed dead. It's publish and kill it effect.


Trying to figure out how to use firstly{}

PromiseKit 6.5
Here's what I'm trying to do:
@IBAction func AddElectionEvents(_ sender: UIButton) {
        return firstly {
            deleteEvents()
        }.map {
            createEvents()
        }
    }

The user clicks a button, which causes deleteEvents() to trigger first, and then, once it's finished, createEvents() is triggered.
Of course, Swift throws the error: "Cannot convert value of type '() -> ()' to expected argument type '() -> Guarantee<_>'"
Based on the troubleshooting page, I thought that using .map instead of .then would fix this. However, the example on the troubleshooting page included more complex functions then I'm using.
My respective functions delete events in a user's calendar, and creates them. Nothing is returned. Very cut and dried.
So, what do you think?

You have no return on your function.

Closing due to stagnation. Reopen if you still need assistance.


look at html().length instead of children().length

With a plain medium editor (not using medium-editor-insert-plugin), I can set the content to plain text without any html tags, and everything just works.  Here is an example of the plain medium editor working in this way: http://codepen.io/anon/pen/EZrgYQ However, when I try to do this with a medium editor that has the insert plugin enabled, the insert plugin clears out my text. On line 311 of core.js, if you change children() to html(), it will fix this issue.

Good catch. Thanks for the fix

This fixes #392
I made the mistake of commenting earlier.


Xcode 10.1 error when compiling C++ files

I have a file abc.pyx that starts:
# distutils: language=c++

(because I am using vector).
This file compiled fine until I upgraded to Xcode 10.1 (10B61). I now get the following error:
warning: include path for stdlibc++ headers not found; pass '-std=libc++' on the command line to use the libc++ standard
      library instead [-Wstdlibcxx-not-found]
abc.cpp:636:10: fatal error: 'ios' file not found
#include "ios"
         ^~~~~
1 warning and 1 error generated.
error: command 'gcc' failed with exit status 1

All of my pyx files that are compiled using C instead of C++ are compiling correctly.
I am using Cython 0.29 in an Anaconda environment with Python 3.6.2 on macOS Mojave 10.14.1.
Python 3.6.2 |Continuum Analytics, Inc.| (default, Jul 20 2017, 13:14:59)
[GCC 4.2.1 Compatible Apple LLVM 6.0 (clang-600.0.57)] on darwin

My setup.py looks like:
setup(ext_modules=cythonize[(abc.pyx)])

I am building using the following command:
python setup.py build_ext --inplace
python setup.py install

Any suggestions would be greatly appreciated. Is there any way I can pass a compiler option to just those files?
This StackOverflow response is close, but I don't want to compile all of my pyx files into C++. I tried that, but got other errors. It would be simplest if I could just resolve the issue of this single file.

This issue was resolved by upgrading to the Xcode Command Line Tools 10.1, released on November 2, 2018.

This issue has not been resolved. I'm using Command Line Tools 10.14 (released Nov 2) with Mojave version 10.14, and I still get the same problems as before.

I am running Mojave 10.14.2 and no longer have this issue with the latest version of Command Line Tools (Nov 2). There might be some other issue with your environment, though I don't know what it might be.

I tried completely uninstalling the command line tools, along with my python distribution (Anaconda with python 3.7), and reinstalling everything from scratch, and it gives the same problems. Only thing I can think of if it's not a cython issue is that it's somehow an anaconda issue (or something with python versions). Either way, using cython via Anaconda (with python 3.7) on Mojave doesn't appear to work.

I am not sure why this was closed. There are two problems, it seems. One is with OSX LLVM setup, the other with Cython:

LLVM (on OSX) no longer supports libstdc++. So in your Cython setup script, just add '-stdlib=libc++ in the extra_compiled_flags option, and that should compile.

The linker is still broken and requires linkage to libc++ instread of libstdc++.

The first problem can be fixed in the setup script. The second is more involved. I think it should be fixed in Cython proper.

Cython does not change anything in the flags and options of your C compiler or linker. The usual ways how these options enter the game are:

provided by the user, e.g. by setting CFLAGS
inherited from the original build flags of CPython itself in distutils
platform/deployment specific configuration
AFAICT, none of the effects described in this ticket suggest a problem in Cython.

Originally posted by @TruongKhang in https://github.com/coccoc/coccoc-tokenizer/issues/5#issuecomment-504321367_
The setup.py file should be changed to something like:
from Cython.Distutils import build_ext
from distutils.core import setup
from distutils.extension import Extension

import os
os.environ['CC'] = 'g++'
os.environ['CXX'] = 'g++'

ext_modules = [
    Extension(
        name="CocCocTokenizer",
        sources=["CocCocTokenizer.pyx"],
        extra_compile_args=["-Wno-cpp", "-Wno-unused-function", "-O2", "-march=native", '-stdlib=libc++', '-std=c++11'],
        extra_link_args=["-O2", "-march=native", '-stdlib=libc++'],
        language="c++",
        include_dirs=["."],
    )
]

setup(
    name="coccoc-tokenizer", ext_modules=ext_modules, cmdclass={"build_ext": build_ext}
)

Originally posted by @TruongKhang in coccoc/coccoc-tokenizer#5 (comment)_
The setup.py file should be changed to something like:
from Cython.Distutils import build_ext
from distutils.core import setup
from distutils.extension import Extension

import os
os.environ['CC'] = 'g++'
os.environ['CXX'] = 'g++'

ext_modules = [
    Extension(
        name="CocCocTokenizer",
        sources=["CocCocTokenizer.pyx"],
        extra_compile_args=["-Wno-cpp", "-Wno-unused-function", "-O2", "-march=native", '-stdlib=libc++', '-std=c++11'],
        extra_link_args=["-O2", "-march=native", '-stdlib=libc++'],
        language="c++",
        include_dirs=["."],
    )
]

setup(
    name="coccoc-tokenizer", ext_modules=ext_modules, cmdclass={"build_ext": build_ext}
)

@joshlk Thank you for this. Works perfectly on my mac but shows
g++: error: unrecognized command line option ‘-stdlib=libc++’ error: command 'g++' failed with exit status 1
when I try to compile it on Linux(Ubuntu),

Cython does not change anything in the flags and options of your C compiler or linker. The usual ways how these options enter the game are:

provided by the user, e.g. by setting CFLAGS
inherited from the original build flags of CPython itself in distutils
platform/deployment specific configuration

AFAICT, none of the effects described in this ticket suggest a problem in Cython.

It can be configured to work with manually added flags for sure. But the default setting only works with gcc; it does not work with llvm/clang. It would be nice to have cython to detect the compiler and choose proper compiler/linker flags.

https://github.com/pandas-dev/pandas/issues/23424#issuecomment-446393981
I think this comment explains the root reason of the problem. I think such a fix, requiring target macos>10.9, inside cython would be nice to have, given that xcode/clang has such weird behavior and that might be harder to fix.


Scan invoice data from TransferPage

When scanning invoice data from TransferPage app automatically back HomePage
https://github.com/encointer/encointer-wallet-flutter/assets/76556278/554cb44a-c93f-45c3-851a-c8103d5c72a7

Good catch! Needs to be resolved!


Should displayed tx value include fee?

I find this strange: I send 1 NIM. Now it says in big numbers "1.01242". I would expect it to rather be "1 NIM \n plus 0.01242 NIM fee".

@sisou let's definitely exclude the fee


TypeError: chalk.stripColor is not a function

Apologies uprfront (I am a newbie). The newest version of snazzy (7.1.0) has caused a breaking change for me.  I get the following error TypeError: chalk.stripColor is not a function. I have no troubles if I fall back to 7.0.0

Hey, I had the same error, had to fall back to 7.0.0 as well

You're right -- my bad. This is fixed in 7.1.1


Update HNO file name

Updated HNO file name pattern.  Checked HNO scraper to make sure file name will continue to be named this way.
cc @mcarans

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 93.333%

Totals

Change from base Build<PHONE_NUMBER>6:
0.0%

Covered Lines:
1414

Relevant Lines:
1515

💛  - Coveralls


[code quality] Unjustified use of **kwargs in get_ssm_parameter_tags()

Switched from kwargs to 3 arguments initialized to None

I made the fix described in my previous comment on this issue, made the commit to this repo & will now close this issue.

Fixed in commit 8fa2fadf61443e84e5ebecc506ad88bb9aadd9ea


Cannot sign in with 2fa enabled.

I have confirmed that sign in is possible with 2FA enabled
$ mas signin<EMAIL_ADDRESS>==> Signing in to Apple ID<EMAIL_ADDRESS>Password: <******>
An Apple ID verification code is required to sign in. Type your password followed by the verification code shown on your other devices.

At this point a verification code appears on my phone, I then attempt to sign in again and append the verification code to the end of the password
$ mas signin<EMAIL_ADDRESS>==> Signing in to Apple ID<EMAIL_ADDRESS>Password: <******><verification code>

I am also adding a --dialog flag in #44 to allow fallback to the OS authentication window if needed.


Not able to exclude transitive dependencies from a scan?

Hi,
I am using the ./gradlew dependencyCheckAggregate command to generate the dependency report HTML file for one of my Gradle projects. It works fine and lists both direct and transitive jar dependencies which are vulnerable as per the NVD database. However, I tried multiple ways but failed to figure out a way wherein I want to only list the direct dependencies? Is there any way through which I can list the direct dependencies only? Below is the build.gradle file for reference –
File snippet -
plugins {
    id 'org.owasp.dependencycheck' version '6.5.3'
}
repositories {
    mavenCentral()
}
subprojects {
    apply plugin: 'org.owasp.dependencycheck'

    dependencyCheck {
        outputDirectory = 'security-report'
        failOnError = true
    }
}

Is this some feature that is missing from this plugin? or something else? Please confirm.

IMHO the whole point of a tool like ODC is to scan all dependencies... if you want to scope dependencies you schould add a scope to them such as "provided" and use that to limit the scan

thanks, @mprins but I think dependency check is a widely accepted tool by the technology community and I think having a feature like where scan could only list direct or transitive or both dependencies could be great. at least a classification on the generated report could be an advantage where it is clear to the developers. sometimes I would only focus on the direct dependency fixes and not fix the transitive one for a while.

I won't disagree that when using the maven or gradle plugin we could do better about indicating where in the dependency tree the vulnerable component is - however, how much one should worry about the transitive dependencies is complicated.

One can have Used but undeclared dependencies (see an example dependency-analysis report
The risk actually affects your application even if it is only a transitive dependency; which could have happened with the recent log4j event.

Yes, upgrading transitive dependencies can be difficult and sometimes impossible unless you are willing to put in a PR to the direct dependency to help them upgrade. However, just ignoring them is likely not the best option.

thanks, @jeremylong for answering the question. 2 questions -

how can I generate the dependency-analysis report can you help with the documentation to generate the report?
I agree with your opinion however I was thinking it more from a functionality perspective where sometimes I would only want to generate the report containing the vulnerabilities for direct dependencies or sometimes indirect dependencies or sometimes even both. I am hoping if a flag could manage that or something else. Any luck with that?

p.s. - just exploring the different possibilities with this tool.

@beena-yatin-kanyal
regarding 1) The report Jeremy referenced is one that is out-of-the-box available as a report in maven (https://maven.apache.org/plugins/maven-dependency-plugin/analyze-report-mojo.html).
For gradle there seems to be nothing like it (there is a gist that wraps the maven plugin , used by someone else in a plugin that breaks the build on undeclared used dependencies/declared-unused dependencies) they might give you some pointers on how to generate a similar report for gradle.
Regarding 2) I'm with Jeremy that allowing to only list the vulnerabilities of direct dependencies would defeat the goal of this project. I see no sense in distinguishing reports on direct versus transitive dependencies (reporting might be improved to help you asses how the vulnerable transitive dependency was brought into your dependency tree, but it should always be reported. Because in the end your project is to be considered vulnerable unless proven to be unaffected due its vulnerable classes on your project classpath)

@aikebah I have been trying to come up with an easier way for Maven and Gradle to display the dependency tree for transitive deps.  Something like adding a row above related dependencies that shows the path:
Hierarchy
org.junit.jupiter:junit-jupiter-api -> org.apiguardian:apiguardian-api


memchr does not declare #![no_std]

Version of ssd1306 in use (if applicable):  fork from recent git  (15e47c135b )
MCU/other hardware in use: bluepill and other

Description of the problem/feature request/other
I forked to check how difficult it will be to convert rtic examples to recent release cortex-m-rtic v1.0.0. (I'm expecting some updating will be needed so if anyone has already done this then please point the way.) The very first thing I've hit is a problem which must be trivial but I don't understand. Changing only Cargo.toml line cortex-m-rtic = "0.5.6" to cortex-m-rtic = "1.0.0" I get
$ cargo build  --example rtic_dvd
  Compiling memchr v2.3.4
error[E0463]: can't find crate for `std`
  |
  = note: the `thumbv7m-none-eabi` target may not support the standard library
  = note: `std` is required by `memchr` because it does not declare `#![no_std]`

Cargo update after the change gave
    Updating crates.io index
      Adding aho-corasick v0.7.15
      Adding atomic-polyfill v0.1.5
      Adding bare-metal v1.0.0
      Adding bit_field v0.10.1
      Adding cfg-if v1.0.0
    Updating cortex-m-rtic v0.5.9 -> v1.0.0
    Updating cortex-m-rtic-macros v0.5.3 -> v1.0.0
      Adding critical-section v0.2.5
    Updating hash32 v0.1.1 -> v0.2.1
    Updating heapless v0.6.1 -> v0.7.9
      Adding lazy_static v1.4.0
      Adding lock_api v0.4.5
      Adding proc-macro-error v1.0.4
      Adding proc-macro-error-attr v1.0.4
      Adding regex v1.4.6
      Adding regex-syntax v0.6.25
      Adding riscv v0.7.0
      Adding riscv-target v0.1.2
    Updating rtic-core v0.3.1 -> v1.0.0
      Adding rtic-monotonic v1.0.0
    Updating rtic-syntax v0.4.0 -> v1.0.0
      Adding scopeguard v1.1.0
      Adding spin v0.9.2

so it looks like there is no change to memchr. If I add [dependencies] memchr = { version = "2", default-features = false } it makes no difference. The memchr documentation suggests it is no_std, and in any case I do not see why the change in version of cortex-m-rtic affects this.
Any hints?

I've reproduce this locally which is good. Looks like tinybmp and it's older nom dependency is the culprit:
memchr v2.3.4
└── nom v6.2.1
    └── tinybmp v0.3.1
        [dev-dependencies]
        └── ssd1306 v0.7.0 (/Users/james/Repositories/ssd1306)

I'll do some fiddling in tinybmp and see what I can do there.

Looks like the dependency issue is fixed by using edition = "2021" in Cargo.toml. I want to do the same with tinybmp anyway, so this would be a good opportunity to upgrade (and open the door to some shinier new features perhaps).

Ok, that solves the dependency issue. FWIW, copied below is example rtic_dvd with the gross syntax/renaming/etc of cortex-m-rtic v1.0.0. It gets as far as locking issues and de-referencing problems that are a bit beyond me. I've possibly added some unneeded things and maybe clobbered things that are needed.
//! Bounce a DVD player logo around the screen
//!
//! Like this, but with no color changing: https://bouncingdvdlogo.com/
//!
//! For best results, run with the `--release` flag.

#![no_std]
#![no_main]

use panic_halt as _;
use rtic::app;

#[rtic::app(device = stm32f1xx_hal::pac,   dispatchers = [TIM1_UP, EXTI0])]

mod app {

    use ssd1306::{mode::BufferedGraphicsMode, prelude::*, Ssd1306};

    use systick_monotonic::*;
    // secs() and millis() methods from https://docs.rs/fugit/latest/fugit/trait.ExtU32.html#tymethod.secs

    use fugit::TimerDuration;

    const MONOTICK: u32 = 100;
    const CLOCK: u32 = 8_000_000;

    use stm32f1xx_hal::{
        delay::Delay,
        gpio,
        pac::{self, SPI1},
        prelude::*,
        spi::{self, Mode, Phase, Polarity, Spi},
        timer::{CountDownTimer, Event, Timer},
    };
    use tinybmp::Bmp;

    type Display = Ssd1306<
        SPIInterfaceNoCS<
            spi::Spi<
                SPI1,
                spi::Spi1NoRemap,
                (
                    gpio::gpioa::PA5<gpio::Alternate<gpio::PushPull>>,
                    gpio::gpioa::PA6<gpio::Input<gpio::Floating>>,
                    gpio::gpioa::PA7<gpio::Alternate<gpio::PushPull>>,
                ),
                u8,
            >,
            gpio::gpiob::PB1<gpio::Output<gpio::PushPull>>,
        >,
        DisplaySize128x64,
        BufferedGraphicsMode<DisplaySize128x64>,
    >;

    use display_interface_spi::SPIInterfaceNoCS;
    use embedded_graphics::{
        geometry::Point,
        image::Image,
        pixelcolor::{BinaryColor, Rgb565},
        prelude::*,
        primitives::{PrimitiveStyle, Rectangle},
    };

   #[shared]
   struct Shared {
        display: Display,
        timer: CountDownTimer<pac::TIM1>,
        top_left: Point,
        velocity: Point,
        bmp: Bmp<Rgb565, 'static>,
    }

    #[local]
    struct Local {}

    #[monotonic(binds = SysTick, default = true)]
    type MyMono = Systick<MONOTICK>;

    #[init]
    fn init(cx: init::Context) -> (Shared, Local, init::Monotonics) {
        let dp = cx.device;
        let core = cx.core;

        let mut flash = dp.FLASH.constrain();
        let mut rcc = dp.RCC.constrain();

        let clocks = rcc
            .cfgr
            .use_hse(8.mhz())
            .sysclk(72.mhz())
            .pclk1(36.mhz())
            .freeze(&mut flash.acr);

        let mut afio = dp.AFIO.constrain(&mut rcc.apb2);

        let mut gpiob = dp.GPIOB.split(&mut rcc.apb2);
        let mut gpioa = dp.GPIOA.split(&mut rcc.apb2);

        // SPI1
        let sck = gpioa.pa5.into_alternate_push_pull(&mut gpioa.crl);
        let miso = gpioa.pa6;
        let mosi = gpioa.pa7.into_alternate_push_pull(&mut gpioa.crl);

        let mut delay = Delay::new(core.SYST, clocks);

        let mut rst = gpiob.pb0.into_push_pull_output(&mut gpiob.crl);
        let dc = gpiob.pb1.into_push_pull_output(&mut gpiob.crl);

        let spi = Spi::spi1(
            dp.SPI1,
            (sck, miso, mosi),
            &mut afio.mapr,
            Mode {
                polarity: Polarity::IdleLow,
                phase: Phase::CaptureOnFirstTransition,
            },
            8.mhz(),
            clocks,
            &mut rcc.apb2,
        );

        let interface = display_interface_spi::SPIInterfaceNoCS::new(spi, dc);
        let mut display = Ssd1306::new(interface, DisplaySize128x64, DisplayRotation::Rotate180)
            .into_buffered_graphics_mode();

        display.reset(&mut rst, &mut delay).unwrap();
        display.init().unwrap();

        // Update framerate
        let fps = 20;

        let mut timer = Timer::tim1(dp.TIM1, &clocks, &mut rcc.apb2).start_count_down(fps.hz());

        timer.listen(Event::Update);

        let bmp = Bmp::from_slice(include_bytes!("dvd.bmp")).unwrap();

       let mono = Systick::new(cx.core.SYST, CLOCK);

        (Shared { timer, display, top_left: Point::new(5, 3),velocity: Point::new(1, 1), bmp },
         Local {},
         init::Monotonics(mono))
    }

    // just guessing on capacity
    #[task(shared = [timer, display, top_left, velocity, bmp], capacity=2)]
    fn update(cx: update::Context) {
        //   let update::Resources {
        //       display,
        //       top_left,
        //       velocity,
        //       timer,
        //       bmp,
        //       ..
        //   } = cx.resources;

        let bottom_right = *cx.shared.top_left + cx.shared.bmp.bounding_box().size;

        // Erase previous image position with a filled black rectangle
        Rectangle::with_corners(*cx.shared.top_left, bottom_right)
            .into_styled(PrimitiveStyle::with_fill(BinaryColor::Off))
            .draw(cx.shared.display)
            .unwrap();

        // Check if the image collided with a screen edge
        {
            if bottom_right.x > cx.shared.display.size().width as i32 || cx.shared.top_left.x < 0 {
                cx.shared.velocity.x = -cx.shared.velocity.x;
            }

            if bottom_right.y > cx.shared.display.size().height as i32 || cx.shared.top_left.y < 0 {
                cx.shared.velocity.y = -cx.shared.velocity.y;
            }
        }

        // Move the image
        *cx.shared.top_left += *cx.shared.velocity;

        // Draw image at new position
        Image::new(cx.shared.bmp, *cx.shared.top_left)
            .draw(&mut cx.shared.display.color_converted())
            .unwrap();

        // Write changes to the display
        cx.shared.display.flush().unwrap();

        // Clears the update flag
        cx.shared.timer.clear_update_interrupt_flag();
    }
}

I think this has been resolved.


Validators relay

Have Validators relay other peers before using seed services. This is there, but for testnet using seed nodes for reliability during update periods.

This is no longer needed due to beacon service and due to adj pool.


Fixed map sometimes not loading bug

fixes ENG-517

Not sure why commits from other branches were on this PR but this should be fixed now.


yarn?

>yarn add js-Logger

yarn add v1.22.4
[1/4] Resolving packages...
error An unexpected error occurred: "https://registry.yarnpkg.com/js-Logger: Not found".

>yarn add js-Logger

yarn add v1.22.4
[1/4] Resolving packages...
error An unexpected error occurred: "https://registry.yarnpkg.com/js-Logger: Not found".

yarn add js-logger

Sorry folks I'm not sure what you are asking for here 🐱


[Bug]: using tls in home network still tries to resolve certificate via lets encrypt

Error Message and Logs
hi,
i am using coolify self-hosted in my home network and want to use a wildcard domain which i own (setup A record which points to the machine where i installed coolify). for testing purposes i created one wildcard certificate using mkcert. i followed the instructions on this page https://coolify.io/docs/knowledge-base/traefik/custom-ssl-certs/ for adding the custom certificate.

under server -> configuration -> general i configured my domain
the instance setting is also configured to my domain.

however traefik still tries to resolve the certificate via lets encrypt and i think due to coolify.yaml under "dynamic configurations".
i have commented the certificatesresolvers under proxy - configuration and added the label "traefik.http.routers.traefik.tls=true"
i already have one server in my home network configured with traefik and tls for a wildcard domain which i own and wanted to achieve the same with cooilfy.
my next step would be to try different solutions in coolify.yml with removing certresolver: letsencrypt.
does anyone already have working solution for using coolify with tls wildcard domain in home network setup?
maybe for the future there could be a different option whether to use coolify on a cloud vps or in home network where lets encrypt is not used because of the http-challenge
thanks so far
greetings
Steps to Reproduce
see error message and logs description
Example Repository URL
No response
Coolify Version
v4.0.0-beta.364
Are you using Coolify Cloud?
No (self-hosted)
Operating System and Version (self-hosted)
Ubuntu 24.04.1 LTS
Additional Information
No response

obsolete, various services are using my own certificate created by mkcert. lets encrypt is not used.
i was thinking that instead of hostname of the server, it can be reached via the base domain. but this is sufficient.
thanks for the great product


do we get retweets as well ?

@Jefferson-Henrique Does this search give Retweets by the user  or just the original tweets ?

Hi @Jefferson-Henrique  and @swatisaoji1 ,
Jefferson, thanks very much for doing this wonderful project - it is so helpful!
I have found that you only get original tweets and no retweets with this search. Just thought I would add something to this question to say that I have edited the Exporter script slightly in order to get retweets. There is a hidden argument for the Twitter advanced search - if you include " filter:nativeretweets" you get only retweets and if you include " -filter:nativeretweets" you get no retweets.
I have therefore added a few lines of code to both Exporter.py and TweetManager.py such that, if the user inputs --retweetsinclude, the search runs first for non retweets and then subsequently for only retweets. @Jefferson-Henrique, if you would be interested in including this option then please let me know and I can help out

@peterblair Thats great info , I am curious how did you find it out. ?

Just google!

I used the command below to get only retweets by a specific user;
python Exporter.py --querysearch "from:@username filter:nativeretweets"

Hi, I used the query as suggested by @stanupab : the url seems to work fine (I paste and copy it on the browser and the result is as desired), but still GOT doesn't recover the tweets. Am I missing somenthing?

Unfortunately for me the python script is having all kinds of issues. However the Java script works just fine. Is there an option to get the retweets in the Java script?

Hi @peterblair, I'm really interested in retweet data and am wondering if @Jefferson-Henrique ever incorporated your code? I'm super new to coding so I'm not sure how to go about adding that functionality myself. Thanks!!

@peterblair I'm doing a project on this and would like to get retweets included. If you could specify which lines of code you altered, or post files or something, I would greatly appreciate it!

@wuqui it doesn't show all 4000 retweets of the original tweets bc it only gets a random 1% sample of tweets. It's entirely possible to get a couple of the retweets but not the original tweet. There definitely are retweets included tho, they just don't always have the handle in the text for some reason..

@peterblair
Hey Peter, thanks so much for posting your reply!! I am also in need of the retweets and I tried changing the code but for the life of me, I could not!
Would you be able to share the specific modifications you did to the code on TweetManager.py and an example call to the script including --retweetsinclude??
->> "I have therefore added a few lines of code to both Exporter.py and TweetManager.py such that, if the user inputs --retweetsinclude, the search runs first for non retweets and then subsequently for only retweets." <--
I've spent over 20 hours on this, you help would be extremely appreciated! You can reach me at<EMAIL_ADDRESS>if that's easier. Thanks a million, Peter.

Hi @Jefferson-Henrique and @swatisaoji1 ,
Jefferson, thanks very much for doing this wonderful project - it is so helpful!
I have found that you only get original tweets and no retweets with this search. Just thought I would add something to this question to say that I have edited the Exporter script slightly in order to get retweets. There is a hidden argument for the Twitter advanced search - if you include " filter:nativeretweets" you get only retweets and if you include " -filter:nativeretweets" you get no retweets.
I have therefore added a few lines of code to both Exporter.py and TweetManager.py such that, if the user inputs --retweetsinclude, the search runs first for non retweets and then subsequently for only retweets. @Jefferson-Henrique, if you would be interested in including this option then please let me know and I can help out

Hey @wuqui , did you get this figured out? I am hurting here, lol! Went through the code and made a few edits but could not get it work... Were you able to ever come up with a solution to this that you could share?

Hey, unfortunately not, no. I'd still be interested and glad if you could post it here if you find something!

Thanks @wuqui! @peterblair Peter, we need you, man! I'll buy you a beer!! Really need some help with this! Thanks everyone.

Hey @caryshin, were you ever able to figure out exactly which lines to change? Thanks so much.. really appreciate any tips you can offer to get the retweets

Hi, really sorry but this was ages ago and I don't think I still have the code! Hopefully some of what I remember will be useful though!
From what I remember, I think it was a pretty trivial change to the code... If I remember right, GetOldTweets uses the advanced search on Twitter, so if you go to...
https://twitter.com/search-advanced?lang=en-gb&lang=en-gb       (i.e. the advanced search)
and then in the top search bar you type
'try this search filter:nativeretweets'
It comes up with only retweets
But if instead you search
'try this search -filter:nativeretweets'
It comes up with only non-retweets
So wherever in the code the search arguments are pasted together, I think you just need to paste in either filter:nativeretweets to get just retweets or -filter:nativeretweets to filter them out! Then if you want to do both (as I did), then adding in an extra call to the function that does the searching should do it!
Hope that's helpful!

THANK YOU @peterblair I will give this ago. Really appreciate you taking the time to answer, have a great weekend!


Yarn start in app-backend/dynamodb does not start dynamodb on port 8000

Running yarn start in app-backend/dynamodb as simply stated in the docs does not start the dynamodb on port 8000 w/ access to it at 8000/shell.
Dynamodb at 8000/shell only works when starting from app-backend/appsync/dynamo
For the graphql playground at :3000 AND the dynamodb to be running at the same time, I need to have run yarn start from both dynamo directories (app-backend/dynamodb && app-backend/appsync/dynamo)
Am I missing something?
A snippet of what  I'm seeing output in the terminal from app-backend/dynamodb is a steady stream of this:
":\"deprecationReason\",\"description\":null,\"args\":[],\"type\":{\"kind\":\"SCALAR\",\"name\":\"String\",\"ofType\":null},\"isDeprecated\":false,```

solved with this:
https://github.com/99xt/serverless-dynamodb-local/issues/225#issuecomment-498023248


Build libcontainer's container on Instance::new to be spec compliant

Fixes #158 and #326
The issue behind #326 is that we are asking youki to create and start the container on the start call.
We don't do any meaningful work on the create call.
This missmach means that containerd is assuming that extra setup has happened successfully. but youki hasn't done it yet. When youki fails to do it at start, we are breaking those assumptions (e.g., being able to provide a PID for the init process).
The solution is to build (create) the youki container on the create call (i.e., the new method of runwasi's instance).
This also requires that we change the signature of new to return a Result<Self, Error>, which was previously mentioned in #158.
I haven't added any test for this here as testing it is rather involved.
The sideeffect of this issue on the first invocation is that we leave a dangling process, and on a second invocation is a hang.
I'm open to suggestions on how we could test it. Thanks @jsturtevant for suggesting testing approach.

this is the side affect of this change?

No, this is what I observe on linux before this change. With this change I do not observe any dangling process and no hanging.

Would it be something like:

Yes, that's 100% correct, thanks!. I'll add test points for all shims.

I gave the test a go. The right place for it would be in the containerd-shim-wasm crate, as there's where the logic is implemented.
However, that test uses the containerd-shim-wasm-test crate. Adding that crate as a dependency creates a cycle which causes some weird interaction with types being doubly defined (e.g., there's a crate::sandbox::Instance and a containerd_shim_wasm::sandbox::Instance, and they are not the same trait for the compiler)
A solution is bringing the testing to the containerd-shim-wasm-test functionality to the containerd-shim-wasm behind a feature flag. Doing that significantly increases the number of files touched by this PR, but shouldn't add much complexity.

I recommend reviewing the commits separately :-)

@jprendes this needs a rebase, but lgtm after that.


[bug] current_monitor() always returns the external monitor (Ubuntu 22.04)

Describe the bug
Calling current_monitor() on a window just built with the WindowBuilder always returns the external monitor even if the window is clearly opened on the built-in monitor.
Reproduction
I made a repro here on branch wrong-monitor: https://github.com/matthme/menu-stuck-tauri-macos/tree/wrong-monitor
It prints out the monitor of the window.
Expected behavior
Return the monitor where the window is currently residing in.
Platform and versions
Environment
  › OS: Ubuntu 22.04 X64
  › Node.js: 16.11.1
  › npm: 8.0.0
  › pnpm: Not installed!
  › yarn: 1.22.19
  › rustup: 1.25.1
  › rustc: 1.64.0
  › cargo: 1.64.0
  › Rust toolchain: stable-x86_64-unknown-linux-gnu

Packages
WARNING: no lock files found, defaulting to npm
  › @tauri-apps/cli [NPM]: 1.1.1 (outdated, latest: 1.2.3)
  › @tauri-apps/api [NPM]: Not installed!
  › tauri [RUST]: 1.2.1,
  › tauri-build [RUST]: 1.2.1,
  › tao [RUST]: 0.15.6,
  › wry [RUST]: 0.22.6,

App
  › build-type: bundle
  › CSP: unset
  › distDir: ../src
  › devPath: ../src
package.json not found

App directory structure
  ├─ .git
  ├─ src-tauri
  ├─ src
  └─ .vscode

Stack trace
No response
Additional context
No response

could you give me the output of:
dbg!(window.current_monitor().unwrap().unwrap().name());
dbg!(window.primary_monitor().unwrap().unwrap().name());
dbg!(window.available_monitors().unwrap().map(|m|m.name()).collect<Vec<_>>());

and also the output of
xrandr --query

I see what causes the issue now, unfortunately it will take some time to fix. For now, you can workaround this by, creating the window in one command, then run the logic that depends on the current monitor in another command.


mosquito: system_user.json: Permission denied

Hello,
I have strange error when running HA mosquito addin on my Centos 7 AltArch:
/run.sh: line 38: /data/system_user.json: Permission denied
Looks like I don't have /data/ folder in docker image homeassistant/armv7-addon-mosquitto:5.1.
It is an issue with mosquito addin or I have something wrong with my configuration?

@ #1137


maintainers: remove candyc1oud

Description of changes
This is my last PR. I'm not going to be using NixOS after that. I've been using NixOS for a few months, but I'm still not comfortable with a non-FHS Linux distribution.
BTW, I updated lux to 0.17.2

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

Good bye! And thanks for the expressions!


PageFeedback /docs/clouds/aws/guides/vpc/

Submitted On: 10-17-2023
Comment(s): deprecated

Not actionable / unclear feedback.


Codelens does not work if there is a space in the path

Codelens does not work if there is a space in the path.
This happens with macOS and Windows
I can not compile the project....?!?
ORIGINAL
const buildCommand =
        os.platform() === 'win32'
            ? `powershell -ExecutionPolicy ByPass -File build.ps1 -script ${fileName} -target \"${taskName}\" -verbosity=${runConfig.verbosity}`
            : `./build.sh --script ${fileName} --target=\"${taskName}\" --verbosity=${runConfig.verbosity}`;

FIXED
const buildCommand =
        os.platform() === 'win32'
            ? `powershell -ExecutionPolicy ByPass -File build.ps1 -script \"${fileName}\" -target \"${taskName}\" -verbosity ${runConfig.verbosity}`
            : `./build.sh --script=\"${fileName}\" --target=\"${taskName}\" --verbosity=${runConfig.verbosity}`;

@EifelMono thank you for taking the time to report this!  This should be corrected in the next release of the Visual Studio Code addin for Cake.

@EifelMono 0.11.3 has been pushed to the marketplace.  Can you please take this for a spin, and let us know if this has fixed the problem that you are seeing.

@gep13
MAC:
Codenles does not work on a Mac because the equal char is missing between the script and the fileName
bug: ./build.sh --script "${fileName}" ......
fix: ./build.sh --script="${fileName}" ......
If I add the equal char in the command line the path with blank works.
Windows:
On Windows it works.
FYI the verbosity switch on windows has also an equal char but it works.

@gep13 @jrgcubano any reason why Code lens action is using bootstrapper when executing Cake? Wouldn't it be wise to just kick-off cake.exe or dotnet cake.dll instead?

@EifelMono can you please try again with 0.11.4 and let me know if you are still running into problems?  Thanks

@gep13
Code lens "run task" now works on macOS and windows. It works also with a blank in the path.
But "debug task" does not work on macOS and Windows.
It downloads the Cake.CoreCLR starts the debugger and than ????
I have attached two videos macOS and Windows about that.
https://github.com/EifelMono/ReportedBugs/tree/master/cake-build/cake-vscode/94

I haven’t checked the video, but can you provide a link to the project you are trying to debug? It could be that you are using something that isn’t compatible with Cake.CoreClr which is what debugging is using by default. In this instance, you would need to modify the Cake settings to use mono as the debugger rather than .net core.

I will try to look at this tonight, and I will let you know what I find.


Service Now > dedicated format for time field

Hello David,
We need a dedicated format for all the time fields that we send to Service Now (1970-01-01 00:00:00).
What is the best way ?

Create a dedicated field in this bundle ?
Create a dedicated Services/ServiceNowSubmissionRender with a Transformer in the Submission Bundle ?
Other idea ?

Thanks !

Without looking at the code I would try to fit this in the model of a submission. A dedicated serviceNowDateTime object. That will always return the correct form when cast to a string. The model handles the transformation. Then the servicenow submission handler could instantiate this new model whenever a DateTime class is passed in the formdata. Does something like that seems possible?

So this would go in the submission bundle as we only care for this format after/on submit

Yep, sounds good. I look how to do something like that.
Thanks.


Added the swagger docs for PATCH users/self route

Added the swagger docs for PATCH users/self route. Fixes #77

Thanks @Rucha1499 , Awesome work 🔥 👍🏽


#162 Switch to license expression

Closes #162

Thanks for making the changes, other than keeping the reference to the file lgtm.

Keeping the reference will result in the license file also being included in the package.

thanks for reverting back @thompson-tomo

Keeping the reference will result in the license file also being included in the package.

Is there anything wrong with it? I thought it was a best practice.
I'm not sure about .NET standards, I'll defer to @baywet and/or @andrueastman the final decision.

If we want to diff our license file against the template, here is the source https://spdx.org/licenses/

I just ran a diff, they are identical except for formatting and the fact that Red Hat is in the copyright (should it?)

Red Hat is in the copyright (should it?)

I probably took it from another RH project, I'm fine removing the attribution.

Red Hat is in the copyright (should it?)

I probably took it from another RH project, I'm fine removing the attribution.

Since we made a point of creating a separate organization, project, etc... it'd make sense to align licensing as well.
Since we don't have a legal entity behind this project, I think our only option would be to set individuals.
And I don't want to "take credit for work you've mostly done", but for a "hit by a bus reason" I think it'd make sense to have multiple individuals here.
What do you think?

No strong opinions, I would be fine leaving it empty but feel free to proceed as you think is the best.

This should be the answer: https://www.apache.org/licenses/LICENSE-2.0#apply
To me it can be:

Copyright 2023 std-uritemplate

Optionally with a link to: https://github.com/orgs/std-uritemplate/people
But, again, I'm really not fussy about this.

I think you just found the perfect solution!

@baywet do you mind to recap the next steps?
additional note: I'm extracting the C# implementation to a separate repo: https://github.com/std-uritemplate/std-uritemplate-csharp so I'll probably ask to direct this PR there sometime soon.

sure:

[ ] remove the property group block that includes the license file from the csproj
[ ] update the license file to set the copyright to Copyright 2023 std-uritemplate

CC @thompson-tomo

@baywet ok, the changes requested are done.

Superseeded by #165 and https://github.com/std-uritemplate/std-uritemplate-csharp/pull/2


fix bigfiles

closes https://github.com/abzcoding/lvim/issues/107

we don't need this, since we have this in lunarvim core
  lvim.builtin.bigfile = {
    active = true,
    config = {},
  }


improve parallelism capabilities of arangorestore

https://github.com/arangodb/arangodb/pull/14010

Please incorporate --envelope false into the fast cluster restore article and script.
feature/improve-restore-performance/3.8/programs-arangorestore-fast-cluster-restore.md

IMHO scripts like "fast cluster restore" shouldn't even exist.
It should be fast out of the box, without the requirement to run custom scripts and take extra steps.
Apart from that, arangorestore does not have the --envelope option, but only arangodump has. arangorestore can pick up whatever dump exists and use it.

I don't disagree but it still seems to have relevance as it talks to multiple Coordinators instead of one and there's something about a temporary replication factor of 1. I added a remark about the non-enveloped format.


Add .NET 8 Runtime as Build Target for NSwag

.NET 8 projects require Microsoft.Extensions.DependencyInjection.Abstractions version 8.0.0, but NSwag's current dependencies do not support this version, leading to build conflicts. To enable compatibility with .NET 8, please consider updating NSwag's build targets and dependencies.
Reproduction:

Create a .NET 8 project.
Add NSwag.AspNetCore (e.g., version 13.17.0-preview-20221021-0844).
Build fails with NU1107 error due to version conflict on Microsoft.Extensions.DependencyInjection.Abstractions.

Expected:
NSwag should build without conflicts in .NET 8 projects.
Suggested Fix:

Update NSwag's dependencies to support Microsoft.Extensions.DependencyInjection.Abstractions version 8.0.0.
Add .NET 8 as a target framework in NSwag's project files.

Environment:

NSwag version: 13.17.0-preview-20221021-0844
.NET version: 8.0

This seems to be a returning issue with each new version of dotnet. At least, I remember experiencing something similar last year.
Strange thing is that this somehow doesn't get caught when trying out the previews and release candidates. Somehow this version requirement doesn't count for previews?
Anyway, couldn't this dependency requirement be relaxed somehow so this doesn't come back each year?
Why is this requirement necessary? The NSwag packages seem to be the only packages (in quite a long list of packages I use) that exhibit this behavior.

Preview 14.0.0-preview009 solves this problem. (I have not checked what state the preview releases are in or if the are safe for prodction)

This might need changing to the 8.0.0 RTM version now?
https://github.com/RicoSuter/NSwag/blob/a7cea5815aa208ef66f2e6dc5e4ec49020f54e3a/src/NSwag.AspNetCore/NSwag.AspNetCore.csproj#L20

@Numpsy #4584

Preview 14.0.0-preview009 solves this problem. (I have not checked what state the preview releases are in or if the are safe for prodction)

It generates completely broken clients for me, so I'd say it's not safe for production.

I know this will be fixed when v14 releases, but seeing it's still in preview with some issues listed in https://github.com/RicoSuter/NSwag/issues/4524 I'm not sure how long it will be until a stable version is ready.
Would it be a lot of work to create a release for NSwag.AspNetCore v13 supporting .NET 8? Is there anything we can do to help get, let's say version 13.21.0 released?
We currently don't want to use a preview version in production, but are eager to upgrade to .NET 8 for a variety of reasons.
Btw, not trying to come across as impatient here; I really appreciate the effort put into NSwag. In the meantime, I will do some testing with v14 preview and report back in https://github.com/RicoSuter/NSwag/issues/4524 if I run into any issues. 🙂


Optimistic ConcurrencyException thrown if doc is stored and then updated in dirty tracked session

When using optimistic concurrency combined with dirty tracked sessions, you cannot make any changes to document after it is initially stored, but before .SaveChanges() is called.
Sometimes it would be nice to be able to get the auto generated Id of the object which is set on Store, and then make subsequent changes to the doc prior to saving changes.
Failing test (using code in optimistic_concurrency.cs):
        [Fact]
        public void can_store_and_update_same_document_with_optimistic_concurrency_and_dirty_tracking()
        {
            var doc1 = new CoffeeShop();
            using (var session = theStore.OpenSession(tracking: DocumentTracking.DirtyTracking))
            {
                session.Store(doc1);
                doc1.Name = "New Name";
                session.SaveChanges();
            }
        }

FYI, Adding a SaveChanges() after Store() and before the Name update prevents the exception as a workaround, but then you are committing twice.

@barryhagan The Id is assigned upon the call to Store(). You'd have the identity from the document you just passed in.

Right, but I cannot make any other subsequent changes to the doc unless I call SaveChanges() first.
The real use case for me was to call Store() to get the Id, then save a file in a storage location where I need the doc Id (it's part of the storage path), then update the list of files on the document with the new file info.  Trying to update the document after it is stored in the same session is what causes the issue.


find_clusters is not used anywhere ?

Maybe it will be used in later examples?

Yes that's correct.


Dart + IntelliJ view type doesn't work, how can I make it work?

The Dart/Flutter plugins for IntelliJ seems to disable/not support View Type?
I.e. when I press Ctrl + Q I should get a type, but this doesn't work in my IntelliJ for dart files, it works for other files.
Can I fix this?

/cc @devoncarew I am tentatively marking this as area-intellij because I don't know what is the better label.

That works; we'll triage - most issues will likely end up in the analyzer area, or moving to the JetBrains issue tracker.
@jwren for triage


Add support for Hive current_user function

Thanks @wmoustafa for quick merging! Are you planning on doing a release soon. From what I see the latest version uploaded to maven central is 1.0.12.

Thanks @wmoustafa for quick merging! Are you planning on doing a release soon. From what I see the latest version uploaded to maven central is 1.0.12.

Thanks @wmoustafa for quick merging! Are you planning doing a release soon? From what I see the latest version uploaded to maven central is 1.0.12.

@wmoustafa kind ping on this one :)

@losipiuk Currently this is blocked on an issue being discussed here https://github.com/linkedin/coral/pull/36. We are trying to resolve that as soon as possible.

@losipiuk Latest version is now published to Maven Central. https://repo1.maven.org/maven2/com/linkedin/coral/.

Thanks @wmoustafa . I will prepare PR to update dependency in Trino.


CONSEP QA/Testing activities (2nd priority) low-fidelity screen validation with users

Describe the task
Reviewing new low-fidelity screens with Tree Seed Centre staff for feasibility - confirm necessary functionality still exists, information architecture is intuitive
Screens to review:

[ ] Testing request report
[ ] Germination species average
[ ] Seedlot location history
[ ] (Possibly) Family lots

SME/User Contact
Victoria L
Additional context

any additional details that could not be captured above

Scheduled for Thursday Dec 12 @ 1 PM


how to create VM from a custom image based on marketplace image using Azure Powershell

Section Using Powershell
When creating a VM from an image based on a marketplace image, plan info must be supplied, or you fail with
"Provisioning failed. Creating a virtual machine from Marketplace image or a custom image sourced from a Marketplace image requires Plan information in the request."
The simplified parameter set for New-AzVm includes the -Image parameter but does not include parameters for plan info. Nor does it permit piping in a vm, nor does it include a -VM parameter, which frustrates using Set-AzVmPlan.
Can powershell scripting be used to create a vm from a custom image that is in turn based on a marketplace image? If so, how?
Thank you.

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

ID: c95cf419-74ae-f6d1-5c3e-01f988fdd9c4
Version Independent ID: 6dfe7480-2c45-2be9-3f3d-8ee97f62cc62
Content: Create VM from a managed image in Azure - Azure Virtual Machines
Content Source: articles/virtual-machines/windows/create-vm-generalized-managed.md
Service: virtual-machines-windows
GitHub Login: @cynthn
Microsoft Alias: cynthn

@HughDevlin Thanks for the feedback! I think the short answer is "No, you can't use the simplified parameter when you want to pass in plan information."
In order to pass in the plan information, you have to create the VM using a VM config file and passing the plan info in using Set-AZVMPlan.
There is a topic for passing in plan information when using Shared Image Gallery images, but it should be similar for a managed image: https://docs.microsoft.com/en-us/azure/virtual-machines/marketplace-images#create-the-vm
Of course, instead of passing in information from the image version (using a variable) you would need to pass the plan information in directly or use a variable derived in another way (like if you have another VM that was created from the same Marketplace image, you can using something like $planVM = Get-AZVM to get the VM info then pass the plan info in using that variable).
Let me know if that doesn't make sense or if you need more help!

@HughDevlin Thanks for the feedback! I think the short answer is "No, you can't use the simplified parameter when you want to pass in plan information."
In order to pass in the plan information, you have to create the VM using a VM config file and passing the plan info in using Set-AZVMPlan.
There is a topic for passing in plan information when using Shared Image Gallery images, but it should be similar for a managed image: https://docs.microsoft.com/en-us/azure/virtual-machines/marketplace-images#create-the-vm
Of course, instead of passing in information from the image version (using a variable) you would need to pass the plan information in directly or use a variable derived in another way (like if you have another VM that was created from the same Marketplace image, you can using something like $planVM = Get-AZVM to get the VM info then pass the plan info in using that variable).
Let me know if that doesn't make sense or if you need more help!

Thank you very much for your quick reply and your offer of help.
We understand how to determine the plan info, that is not a problem, thanks, our issue is with how to supply the plan info to VM creating cmdlets.
Thank you for your suggestion of the "Supply Azure Marketplace purchase plan information when creating images" topic. This topic is very close to our use case, except that we are not using a shared gallery as our image is project-specific.
In the script in the "Create the VM" section of the topic, Set-AzVMSourceImage takes an image id from an image definition of a shared gallery image as retrieved by New-AzGalleryImageDefinition. Will Set-AzVMSourceImage also accept an image id as retrieved by Get-AzImage?
thanks again

Thank you very much for your quick reply and your offer of help.
We understand how to determine the plan info, that is not a problem, thanks, our issue is with how to supply the plan info to VM creating cmdlets.
Thank you for your suggestion of the "Supply Azure Marketplace purchase plan information when creating images" topic. This topic is very close to our use case, except that we are not using a shared gallery as our image is project-specific.
In the script in the "Create the VM" section of the topic, Set-AzVMSourceImage takes an image id from an image definition of a shared gallery image as retrieved by New-AzGalleryImageDefinition. Will Set-AzVMSourceImage also accept an image id as retrieved by Get-AzImage?
thanks again

@HughDevlin Can you tell me which Marketplace image you are using as a base?
I want to actually walk through this and make sure it works, in case I need to rework one of the existing articles or create a new one. I've been focusing on Shared Image Galleries, so it is good to revisit managed images.
Thanks!

@HughDevlin Can you tell me which Marketplace image you are using as a base?
I want to actually walk through this and make sure it works, in case I need to rework one of the existing articles or create a new one. I've been focusing on Shared Image Galleries, so it is good to revisit managed images.
Thanks!

a Windows with Java pre-installed
$planPublisher = 'azul',
$planProduct = 'azul-zulu13-windows-2019',
$planName = 'azul-zulu13-windows2019',

a Windows with Java pre-installed
$planPublisher = 'azul',
$planProduct = 'azul-zulu13-windows-2019',
$planName = 'azul-zulu13-windows2019',

Yes, Set-AzVMSourceImage will accept an image id as retrieved by Get-AzImage.
Thank you for directing me to the "Supply Azure Marketplace purchase plan information when creating images" topic. I was able to get this working by modify this approach, without using Shared Galleries.

Yes, Set-AzVMSourceImage will accept an image id as retrieved by Get-AzImage.
Thank you for directing me to the "Supply Azure Marketplace purchase plan information when creating images" topic. I was able to get this working by modify this approach, without using Shared Galleries.

In scripting VM creation from a managed image, I read the New-AzVM doc and saw the parameter set with the -Image parameter and ran with it. This so-called "simplified" parameter set does a great job of simplifying VM creation through reasonable defaults. However it was not clear to me it did not offer enough flexibility for my use case. It seems to me creating a VM from a marketplace image, or a custom image based on a marketplace image, must be very common use cases. I wasted some time on a blind alley. I feel the New-AzVM doc could be more clear on the limitations of the simplified parameter set.

In scripting VM creation from a managed image, I read the New-AzVM doc and saw the parameter set with the -Image parameter and ran with it. This so-called "simplified" parameter set does a great job of simplifying VM creation through reasonable defaults. However it was not clear to me it did not offer enough flexibility for my use case. It seems to me creating a VM from a marketplace image, or a custom image based on a marketplace image, must be very common use cases. I wasted some time on a blind alley. I feel the New-AzVM doc could be more clear on the limitations of the simplified parameter set.

Yeah, the simplified parameter set is basically the counterpart to the CLI az vm create. It uses the same defaults and has a very small set of required parameters. Before the simplified parameter set, creating a VM using PowerShell was ALWAYS a very long set of commands to create every separate piece (networking, storage, vm config etc).
The simplified set is nice for creating simple VMs for testing and development, but you really do need to create a configuration file (and everything that goes with it) if you want to create a production-ready VM.
I'll have a talk with the Azure PowerShell writer and see if there is a way to give some better guidance on when you can get away with using the simplified parameter set, and when you need to step-up and create a VM config.
Thank you for all of your detailed feedback! This is a really great issue to bring up :)

Yeah, the simplified parameter set is basically the counterpart to the CLI az vm create. It uses the same defaults and has a very small set of required parameters. Before the simplified parameter set, creating a VM using PowerShell was ALWAYS a very long set of commands to create every separate piece (networking, storage, vm config etc).
The simplified set is nice for creating simple VMs for testing and development, but you really do need to create a configuration file (and everything that goes with it) if you want to create a production-ready VM.
I'll have a talk with the Azure PowerShell writer and see if there is a way to give some better guidance on when you can get away with using the simplified parameter set, and when you need to step-up and create a VM config.
Thank you for all of your detailed feedback! This is a really great issue to bring up :)

This article in particular helped send me down the wrong path. I feel this article among others oversells the simplified parameter set by not giving adequate weight to the significant limitations:
Create a VM using a generalized image
https://docs.microsoft.com/en-us/azure/virtual-machines/vm-generalized-image-version-powershell

This article in particular helped send me down the wrong path. I feel this article among others oversells the simplified parameter set by not giving adequate weight to the significant limitations:
Create a VM using a generalized image
https://docs.microsoft.com/en-us/azure/virtual-machines/vm-generalized-image-version-powershell

Thank you for your reply.
You asked, "when you can get away with using the simplified parameter set"
Anytime you need to supply plan info, anytime you try create a VM from a marketplace image, or from an image based on a marketplace image, it seems you may not use the simplified parameters, what use cases are left?
Isn't Microsoft going to require plan info anytime you try to create a Windows VM?

Thank you for your reply.
You asked, "when you can get away with using the simplified parameter set"
Anytime you need to supply plan info, anytime you try create a VM from a marketplace image, or from an image based on a marketplace image, it seems you may not use the simplified parameters, what use cases are left?
Isn't Microsoft going to require plan info anytime you try to create a Windows VM?

@HughDevlin You'd think so , but nope. I've created hundreds VMs from images and snapshots and never had to pass in plan information. I suspect it's because I usually use the vanilla Microsoft supplied iamge for Windows 2016 or 2019. As far as I know, this only happens with some 3rd party Marketplace images. I've yet to confirm why it happens with some, but not others, and I think it is based on whether you need to accept "offer terms" for that image.
I think I might rewrite all of the imaging articles to include the full syntax you need for images that require plan information. I just need to figure out how to keep it readable :)

@HughDevlin You'd think so , but nope. I've created hundreds VMs from images and snapshots and never had to pass in plan information. I suspect it's because I usually use the vanilla Microsoft supplied iamge for Windows 2016 or 2019. As far as I know, this only happens with some 3rd party Marketplace images. I've yet to confirm why it happens with some, but not others, and I think it is based on whether you need to accept "offer terms" for that image.
I think I might rewrite all of the imaging articles to include the full syntax you need for images that require plan information. I just need to figure out how to keep it readable :)

I've created a work item to rework how we cover plan information, as well as look at where we should stop giving the simplified parameter set as an example. I'm going to close this issue for now. Thanks again!
#please-close


fix(kselect): bg color issue

Addresses the issue that is part of:
https://konghq.atlassian.net/browse/KHCP-5396
Summary
Before:

After:

PR Checklist

[ ] Does not introduce dependencies
[ ] Functional: all changes do not break existing APIs and if so, bump major version.
[ ] Tests pass: check the output of yarn test
[ ] Naming: the files and the method and prop variables use the same naming conventions as other Kongponents
[ ] Framework style: abides by the essential rules in Vue's style guide
[ ] Cleanliness: does not have formatting issues, unused code (e.g., console.logs, debugger), or leftover comments
[x] Docs: includes a technically accurate README, uses JSDOC where appropriate

@aanchalm01 you can likely close this one in favor of #922

This issue is fixed in this PR:
https://github.com/Kong/kongponents/pull/922


g3-icap-client: update handle of IMAP APPEND

Codecov Report
Attention: Patch coverage is 4.30108% with 89 lines in your changes missing coverage. Please review.

Project coverage is 31.33%. Comparing base (74f0409) to head (106a601).

Files with missing lines
Patch %
Lines

lib/g3-icap-client/src/reqmod/imap/append/mod.rs
0.00%
48 Missing :warning:

...cap-client/src/reqmod/imap/append/bidirectional.rs
0.00%
14 Missing :warning:

lib/g3-icap-client/src/reqmod/imap/mod.rs
0.00%
14 Missing :warning:

...icap-client/src/reqmod/imap/append/recv_request.rs
0.00%
10 Missing :warning:

lib/g3-h2/src/body/to_chunked_transfer.rs
0.00%
2 Missing :warning:

lib/g3-h2/src/ext/request.rs
0.00%
1 Missing :warning:

:exclamation:  There is a different number of reports uploaded between BASE (74f0409) and HEAD (106a601). Click for more details.
HEAD has 2 uploads less than BASE

Flag
BASE (74f0409)
HEAD (106a601)

lib
1
0

g3proxy
1
0

Additional details and impacted files
@@             Coverage Diff             @@
##           master     #463       +/-   ##
===========================================
- Coverage   54.31%   31.33%   -22.99%
===========================================
  Files        1173      714      -459
  Lines      108334    55875    -52459
===========================================
- Hits        58845    17507    -41338
+ Misses      49489    38368    -11121

Flag
Coverage Δ

g3bench
31.33% <4.30%> (-0.03%)
:arrow_down:

g3proxy
?

lib
?

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Change dotplot labels from EnsemblID to geneID

[ X] I have checked that this issue has not already been reported.
[ ] I have confirmed this bug exists on the latest version of scanpy.
[ ] (optional) I have confirmed this bug exists on the master branch of scanpy.

I have an AnnData object:
print(adata)
AnnData object with n_obs × n_vars = 77430 × 1988
obs: 'CONDITION', 'input.path', 'experiment', 'Sample type', 'BiOmics Sample Name', 'PatientID', 'SampleID', 'Response', 'Respond', 'Response2', 'Adjuvant', 'CIT', 'CIT2', 'Lesion2', 'Lesion', 'Stage', 'Fresh', 'CD3IHC', 'CD3IHC_RICZ', 'Mutation2', 'Mutation', 'Site', 'Age', 'Gender', 'PBMCs', 'PBMCs2', 'Seq samples', 'Quality', 'n_counts', 'n_genes', 'percent_mito', 'n_cPg', 'n_cPg2', 'batch', 'louvain'
var: 'symbol', 'n_cells'
uns: 'louvain', 'louvain_colors', 'neighbors', 'pca'
obsm: 'X_pca', 'X_umap'
varm: 'PCs'
To label the dotplot with gene symbols instead of ensemblID (index column) I use the gene_symbols parameter:
sc.pl.dotplot(adata=adata, var_names = ['ENSG00000104814','ENSG00000043462'], gene_symbols='symbol')
But I get the following error:
Error: Gene symbol 'ENSG00000104814' not found in given gene_symbols column: 'symbol'
TypeError                                 Traceback (most recent call last)
 in
4     sc.pl.dotplot(adata, myg, groupby=condition,dot_min=0,dot_max=0.2,vmin=0,vmax=0.2, save=title+'_'+myg_geneID+'.png')
5 if type(myg_geneID_orig) == list:
----> 6     sc.pl.dotplot(adata, myg, groupby=condition,dot_min=0,dot_max=0.2,vmin=0,vmax=0.2, gene_symbols='symbol', save=title+'_multiple_genes'+'.png')
/pstore/apps/bioinfo/scseq/modules/software/Scanpy/1.4.1-foss-2018b-Python-3.7.1-2018.12/lib/python3.7/site-packages/scanpy-1.4.1-py3.7.egg/scanpy/plotting/_anndata.py in dotplot(adata, var_names, groupby, use_raw, log, num_categories, expression_cutoff, mean_only_expressed, color_map, dot_max, dot_min, figsize, dendrogram, gene_symbols, var_group_positions, standard_scale, smallest_dot, var_group_labels, var_group_rotation, layer, show, save, **kwds)
1383         var_names = [var_names]
1384     categories, obs_tidy = _prepare_dataframe(adata, var_names, groupby, use_raw, log, num_categories,
-> 1385                                               layer=layer, gene_symbols=gene_symbols)
1386
1387     # for if category defined by groupby (if any) compute for each var_name
TypeError: cannot unpack non-iterable NoneType object
My understanding is that it should search for 'ENSG00000104814' in the index column and return the corresponding value in 'symbols', but it seems that is directly searching 'ENSG00000104814' in the 'symbols' column.
Thanks for helping, find below the version I am using:
Versions
scanpy==1.4.1 anndata==0.6.22.post1 numpy==1.15.4 scipy==1.1.0 pandas==0.25.2 scikit-learn==0.20.1 statsmodels==0.9.0 python-igraph==0.7.1 louvain==0.6.1

My understanding is that it should search for 'ENSG00000104814' in the index column and return the corresponding value in 'symbols', but it seems that is directly searching 'ENSG00000104814' in the 'symbols' column.

If you add the gene_symbol parameter, scanpy will look for the var_names in this column instead of looking into  adata.var index. It will not map between index and the gene_symbol
I you want to do the mapping of labels, you can do something like follows:
# set show=False to get the axes dictionary.
ax_dict = sc.pl.dotplot(adata, myg,  groupby=condition)

# get ensembl ids and map them to gene symbol. Although you can directly map `myg`,
# the following method will work in any case, including `sc.pl.rank_genes_groups_dotplot`
# This method also works for `sc.pl.matrixplot`
ticklabels = [adata.var.loc[x]['gene_symbol'] for x in ax_dict['mainplot_ax'].get_xticklabels()]

# replace ensembls ids by gene symbol in plot
_ = ax_dict['mainplot_ax'].set_xticklabels(ticklabels)

Thanks for your answer and the code suggestion.
When I do:
ax_dict = sc.pl.dotplot(adata, myg,  groupby=condition, show=False)
The plot is still shown and ax_dict is:
GridSpec(2, 5, height_ratios=[0, 10.5], width_ratios=[0.7, 0, 0.2, 0.5, 0.25])
How do I have to set show=False to get the axes dictionary?
Thanks

Please update to the latest scanpy release (1.6)

hello,
reporting the same issue using '1.7.2'
And, I could not get around with the above suggestion as the object becomes a list:
[Text(0, 0.5, 'ENSMUSG00000021565'), Text(0, 1.5, 'ENSMUSG00000097971'), Text(0, 2.5, 'ENSMUSG00000048905')],
Somehow I could not convert the gene id because of this obstacle in the above list 'Text' object is not iterable.
Can someone fix the problem or provide a feasible solution?
Thanks!