Installation guide

Metanorma PR checklist

[ ] Breaking changes (list related PRs)
[ ] Documentation update required (create task for this)
[ ] External dependency introduced (documentation update need)
[ ] Gem with native library introduced

Decided not to merge because of the following reasons:

The installation guide on main has already been extensively reworked.


Advanced network query system

Provide an alternative to networks built from the Ontology based gene mapping, similar to functionality at http://csbio.cs.umn.edu/cob.
Allow for a list of genes to be provided and build a network from these. This will also include all first neighbors of query genes. Include interactions among neighbors also.
We will implement this instead of the second network that includes all global genes. If you wanted to see the local / global neighbors of a gene, you can use this instead.
Tasks

[ ] Generate network from genes in query box
[ ] Include first neighbors of all query genes
[ ] Apply force directed layout instead of polywas layout
[ ] Color query nodes different from neighbor nodes
[ ] Default to 0 degree filter (I want to see if queries are not connected to anything)
[ ] Disable parameter boxes for FlankLimit and WindowSize

IF you have time, could you maybe give me a quick example use case so I can get an idea of the work flow?

Sure.

Vlad cares about getting big and strong. He wants to make sure that he is getting enough vitamin A so he can one day be strong enough to overpower his enemies. Vlad is working on a secret project to increase the amount of vitamin A in his favorite power food, Maize. He has a list of genes that he generated using super secret experiments that he thinks control Vitamin A levels in maize grain. Vlad heard about using co-expression to validate whether or not lists of genes maybe be functionally related. He downloads and install cob so he can see any interactions. He doesn't see vitamin A on any of the pre-loaded ontologies so he opts to use the custom search feature in cob. He enters his list of genes into the query box on COB and generates a network. The network contains all the genes in his list but also any other genes that connect to two or more of his query set. He sees that there are quite a few direct interactions between his original gene set, but to his surprise, he discovers a couple new genes that are highly connected to his original query set. In fact, without these key hubs, his original set does not look like it would be that interesting at all. These could be interesting new candidates to explore! Maybe they are master vitamin A regulators! Jackpot! :moneybag:  Using this information, Vlad adds those extra couple of genes to his Vitamin A list using co-expression as evidence of related Vitamin A function. Vlad is happy. He celebrates by cooking an extra large steak for dinner. :cow2:

New tasks now that I have a version working reasonably wel!

[ ] Report genes that do not appear in the selected network
[ ] Tailor gene tables to current graph

#38 Adds this! With the possible exception of the table optimizations

Per your gchat and talking to Meesh, lets implement the following algorithm for loading a sane number of neighbors (Its the one we used in the original COB website):

Generate a subnetwork of query genes including neighbors
Calculate local degree for all genes
Render all query genes and top X neighbors (default: 50)
Report all genes in the table

Include a "Rendered" column in the table
If table row for unrendered gene is clicked, render and add it to the network

Does this sound feasible? It will add some pretty complex logic but is an implementation that I've been wanting for a long long time.

Another thing the original COB implemented for advanced querying is using aliases. Can your query system also search an alias list and map the GRMZM number, the list is here:
http://ftp.maizesequence.org/release-5b/named-genes/ZmB73_5a_named_genes.txt

For aliases, since that is implemented into refgen in camoco, shouldn't we just piggyback off that so the code is only in one place?

Yes, I forgot about that. Use the code from RefGen for sure!

I added some super advancements to the tasks list above.

Okay so working on the algorithm I cam up with an issues, when I query the 4 default genes and retain all of the neighbors, I end up with 897 neighbors and 152,195 edges, which takes forever to pack up and process, how did you deal with this in your implementation?

New general paradigm as reminder from chat:
Have one request to get the top X genes and edges and build the graph from that, have a second server channel to just get the name of all of the genes that are connected and populate the table with that.
With that working, update click logic to update the graph by issuing a request to the server to pull that gene and its connections to genes already in the graph and re-render, do not run a whole new query.

Done except for ontology queries which will get its own issue, in next PR.

What will happen is done in #50


Provide a link to the PowerShell command that performs the same actions.

It would be helpful to have a direct link to the PowerShell command that performs the same actions. Another option is the ability to quickly switch between CLI and Powershell from a menu option or dropdown on the page.

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

ID: c910113e-ccf1-6c81-400c-40d2e00c09d7
Version Independent ID: e9baac80-b846-074c-dc34-c68f5313e19b
Content: az bot authsetting
Content Source: latest/docs-ref-autogen/bot/authsetting.yml
Service: bot-service
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

route to CXP team

@yonzhan / @PramodValavala-MSFT , this request has been added to work item 327968 and I'll discuss it with @mikefrobbins, the Azure PowerShell doc's lead.


Update create2 lookup to support multiple internal transactions

#321  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @tremarkley and the rest of your teammates on  Graphite


Token lifetime

Summary
This PR adds the ability to define a token lifetime in minutes using the --lifetime argument instead of defining expiry as a UNIX timestamp which can be impractical when writing the command out by hand.
It's probably possible to calculate exp in the shell using date and bc/expr to add the desired number of minutes to the current timestamp but that would quickly turn ugly/shell flavor specific.
Not a huge fan that I had to change the signature of from_payloads, so if you would prefer to do argument matching inside of encode_token, please let me know.
Also, this was the first time I read/wrote any Rust code so please let me know if I am using some anti-patterns and I'll fix accordingly. I tried to follow the project conventions to the highest possible extent.
Preflight checklist

[x] Code formatted with rustfmt
[x] Relevant tests added
[x] Any new documentation added

@kkrauth Thanks for this PR! I've been thinking about this for a couple of days after my initial review, and I think adding another argument to the CLI adds too much burden to users when making a JWT. Additionally, it only works in minutes, when the user may want seconds, days, or months.
What would you think instead of making expires more generic? i.e. you could pass a unix timestamp, or a more friendly string like -e 1h, -e '2d 12h', -e '1m 30s', etc. To me this is much friendlier, and the code to differentiate between the friendly string and a unix timestamp. There's even a crate to do the parsing for you.
What are your thoughts?

I like it! I thought about extending the capability of the -e argument but wasn't sure whether you'd consider mixing of argument data types an acceptable tradeoff for offering this capability. I'll play around with it and submit an update. Thanks for the crate suggestion!

Changed it around as suggested. A few thoughts...
I like how the --exp argument is now more explicit as its default value is displayed in the help. Additional bonus is that the default value can be defined as a valid systemd formatted string (+30 min) so the implementation is more elegant as it can take advantage of that fact.
As for the doubts, I've had a few head scratching moments during implementation, especially around validating whether the exp argument is either an integer or a parsable systemd.time string. The way I did it in the end didn't quite sit right with me so please let me know if there's a more elegant way to achieve the same. Of course happy to take input on other parts too.
Thanks!

@kkrauth This is looking great! The expiry parsing is a little much, like you mentioned, but also kind of necessary. I'm not a rust guru by any means, so the way you approached it looks good to me and feels rust-y and fp-y.
Your suggestion to extend it to nbf would be great for consistency. Once that's in, I would love to merge this 😁

@kkrauth This is looking great! The expiry parsing is a little much, like you mentioned, but also kind of necessary. I'm not a rust guru by any means, so the way you approached it looks good to me and feels rust-y and fp-y.
Your suggestion to extend it to nbf would be great for consistency. Once that's in, I would love to merge this 😁

Scope creep! Spoken like a true product owner. Should have kept my mouth shut. 😄 Let me add it in.

Done. I'm confident this is not  the most idiomatic way to achieve this in Rust, but it works. :)
Specifically, calling the function to normalize the duration twice, once for each parameter, is probably not the best way to do it. I'm sure there is a way to iterate over an array of claims for which duration parsing should be performed, and then modify those on the fly, but this is where I hit a wall with my lack of experience using Rust language capabilities.
Open to suggestions and let me know if you see any corner cases I've missed!

I think it's ok to call it twice—I wouldn't say "idiomatic rust" would dictate a loop or anything like that. One thing I think would be more standard would to add a process_duration method to the PayloadItem struct, which operates on self. You could even then call it in the encode_token function.
impl PayloadItem {
  fn process_duration(&self, now: i64) {
    ...
  }
}

let expires = PayloadItem::from_string_with_name(matches.value_of("expires"), "exp")
    .process_duration(now);

True, if I normalize the value closer to source, I can reduce the amount of mapping/filtering I'm doing currently. Let me take another stab at it with that approach.

This looks a lot cleaner to me now.
I tried really hard to find a way to define a global constant for Utc::now().timestamp() but alas Rust would not accept that as a constant value no matter what. Based on googling, it appeared possible to create a macro that would have allowed me to do it but that was way more ceremony than simply passing it as a function parameter to where it's needed.
A few places where I also had to use a lot of .to_string().as_str() calls but that seems like a necessary evil, not necessarily a sign I'm doing something fundamentally incorrect.
Let me know how this looks to you!

@kkrauth Thanks again so much for all your work and patience on this. I'm working through publishing it to all channels are 3.1.0 😁

Turned out much better than the very first proposal! :) Have to frequently generate tokens with a max lifespan of 15 minutes so this will come in really handy. Cheers!


Implement daily publishing file number limits to SOAR

the GFTS server can only handle a certain number of new files per day.
the upper limit is 40.000
for the initial data load but also after possible reprocessing we would easily have more files to upload.
Implement a configurable daily (per run) limit of maximum files to publish

closed in https://github.com/i4Ds/STIXCore/pull/334


Changes to check

simplification
remove check on file name

@daniele-proverbio, can you check so that we can make #24 work?

I removed the check as I don't see the value in it. Basically, any file can be uploaded with any name. The web interface will rename the file to input-data.xlsx and store it in the input directory. So the name of the input file itself does not matter.

I agree than any wrong file will not pass the subsequent checks, but I would nonetheless alert a ministry employee that he\she's using a wrong file. Their pipeline is to always update this specific file with this specific name (which was agreed upon with the Taskforce); if they change it by accident, I believe they should be notified

Okay fair point - I will put it back then

Ok, let me know when to pull :)

I added it now as a first check and then a warning before moving the file. That way, the file handling/moving/renaming is done as part of the pipeline itself.


Deprecated middleware classes support

Hi again @mongkok
I have a proposition about old Django MIDDLEWARE_CLASSES support.
Because Django will be deprecated this old middleware style support in 2.0 version (check in link below)
Django 2.0 deprecation timeline
And still, many Django apps are used only Django 1.11
So when we want to use django-graphql-jwt, we must change MIDDLEWARE_CLASSES to MIDDLEWARE, but then few of other older apps, which still don't support Django 2.0 don't work.
I found a simple solution to this stalemate in django-cors-headers app.
check files:
https://github.com/ottoyiu/django-cors-headers/blob/master/corsheaders/middleware.py
and especially
https://github.com/ottoyiu/django-cors-headers/blob/master/corsheaders/compat.py
What do you think about adding this support?
Thank you for your great work :)

Hi again @maledorak and thanks for the suggestion, I also think it's a good idea to support the old middleware for Django 1.11.

done!
pip install django-graphql-jwt==0.1.8

@mongkok Thank you :)


Document the update format

The device/firmware expects content updates to be packaged in a specific way. We should document this to make it easy for someone to write their own updater.

Some initial work has been done in https://github.com/inseven/statuspanel/blob/main/architecture.md, but this needs to be updated to include details of RLE, and perhaps some pseudo code examples.

Done.


fix handling of SPLUNK_ACCESS_TOKEN to check for empty string

I've been trying to add a test for this in otel_overrides_test.rb but it thinks the access token is abcd. Been fighting to figure out how things aren't being reset after the tests in otel_test.rb and figured I'd just open this at this point.

Codecov Report
Base: 97.67% // Head: 97.67% // No change to project coverage :thumbsup:

Coverage data is based on head (bffff82) compared to base (0688337).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #114   +/-   ##
=======================================
  Coverage   97.67%   97.67%
=======================================
  Files          27       27
  Lines         517      517
=======================================
  Hits          505      505
  Misses         12       12

Impacted Files
Coverage Δ

lib/splunk/otel.rb
96.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Error: test: RUM response from Rack middleware(Splunk::RumRailsTest): LoadError: cannot load such file -- nokogiri/nokogiri

Classic nokogiri...


Expand isn't working or throwing errors

Future<void> getDrills() async {
  // tags is a multi-relation, progression_of is a single-relation
  final x = await pb.collection('drills').getFullList(
        fields: 'id,title,tags,progression_of',
        expand: 'tags,progression_of',
      );
  print(x.first.data);
  print(x.first.expand);
  //
}

data: {progression_of: zmjveahr61ltn40, tags: [d1e52n7bdxv3foz, zqb10h1ggft3ni9, yafqui8bh5so6dq, xbeq20vt6mb0dmk], title: Mozambique}
expand: {}

user has access to List and View on all tables in the database
I have three expectations here and none are working:

the multi-relation should expand
the single-relation should expand
if expand points to something user doesn't have access to, or doesn't exist, an error should be thrown

Anything I might be missing?
The rows I'm trying to expand on this table:

ok I think I got it, I need to specify the fields on the expanded items!
  final x = await pb.collection('drills').getFullList(
        fields: 'id,title,expand.tags.*',
        expand: 'tags',
      );


Add missing info in API doc

As a few methods were removed and replaced it is a good idea to describe
which method(s) can reproduce the same functionality. In addition add a
bit of extra comments for various methods/structs.
JerryScript-DCO-1.0-Signed-off-by: Peter Gal<EMAIL_ADDRESS>

Update done and also added an extra comment for the parse option bits.

There were a lot of api doc changes nowdays, please rebase.

Rebased, as per request.


Don't use Wordplate.... o_O

At the end of the installation process after composer update, I get
Package wordplate/plate is abandoned, you should avoid using it. No replacement was suggested.
Perhaps just funny, but I don't know, what exactly that means....

Plate - it is about plate, not about wordplate.
Plate is a plugin that was provided by the developers of WordPlate to eg. decluter the admin interface and provide a couple of other simplifications to WordPress.
https://www.producthunt.com/posts/administration-ui this was recommanded instead for a while - but seems like Vincent does not support it anymore.

[ ] remove plate from dependencies
[ ] remove https://github.com/NANU-Wandel-Wege-entdecken/NANU-website/blob/master/public/themes/nanu/inc/Plugins/Plate.php

Would be great to find an alternative or solve this inline.
Plate was a good tool for get rid of unnecessary stuff.

@ibes Will you remember that or shall we create an issue to find a replacement?


Fix SyntaxError: unterminated string meets end of file

This is meant to be a deprecation warning, but currently any code that loads this file will raise a SyntaxError.

@ryanfb Thanks for the fix.  I'll release a 3.0.1 with this fix today.


Fix example source file name in TESTS.md

I'm no Objective-C expert, but this looked like a typo to me! Feel free to close the PR if I'm wrong :)

thanks!


Support selections using basic aggregations

Problem
As of today, Prisma only allows selecting aggregations using count but doesn't allow for the other basic aggregations listed below:

SUM
MIN
MAX
AVG

Putting it in the context of someone building an application involving products, this prevents from implementing use cases like computing an average rating for products, getting the minimum or maximum price of a set of products, getting the sum of an invoice based on its individual line items' prices...
The workaround is to select the entire dataset and perform the aggregation at application level, which can turn into poor performance.
Note: this problem is separate from the ability to use an aggregation within a where clause (e.g. selecting products which have an average rating >= 4) which should be approached separately.
Suggested solution
Provide similar functions like count, following the same behavior, for SUM, MIN, MAX, AVG.
This would come with the same limitations:

No support for Group By (cf. prisma/prisma-client-js#1)
No support in filter conditions
No combination with distinct

Next steps

[x] Define the final API surface definition
[x] Define the internal query engine implementation
[ ] Provide documentation examples

Alternatives
prisma/prisma-client-js#5 attempts to solve the problem more broadly with a generic aggregate approach. This could be considered whenever looking at filter conditions or field selection
Additional context
N/A
EDIT: removed MEDIAN as not being evenly supported by database providers.

Note: median doesn't exist as a simple, straight-forward aggregate on all databases and will not be in the API.

This is implemented in dev - you can have a look in https://github.com/prisma/prisma/blob/master/src/packages/client/src/tests/runtime-tests/aggregations/index.js to see how to use this feature

This has been released with 2.2.0 and can be tested this way:
generator client {
  provider = "prisma-client-js"
  experimentalFeatures = ["aggregateApi"]
}

Then you have a new API abailable:
const result = await prisma.user.aggregate({
  where: { age: { gt: 5 } }, // optional
  count: true,
  avg: {
    age: true,
  },
  max: {
    age: true,
  },
  min: {
    age: true,
  },
  sum: {
    age: true
  }
})

Please let us know if this works for you and if we can promote this feature to production.

@janpio Was wondering why my aggregations aren't working. I tried experimentalFeatures & it gave a warning Error: Schema parsing Invalid feature flag: aggregateApi.
Then checked Prisma's version using npx prisma -v & it turned out nexus is using an old version of prisma ,i.e, v2.1.3.
Any way I can use v2.2.0 or do I have to wait for a nexus release?

Indeed, nexus needs to release an update to incorporate the new version.

Aah, wish they would've not combined packages then as this will happen every time prisma releases new features :(

You can use yarn resolution if you want to use the latest version right now.
Read more about it here: https://nexusjs.org/meta/architecture#dependency-philosophy


Important Git information seemingly missing from INSTALL files

After cleaning my PATH, I was really struggling to build Vim. After much frustration I eventually discovered that both Git's bin and usr\bin directories need to be in the PATH along with the path to Mingw. This is pretty important, but I've looked in the INSTALL files in Vim's src directory and I think it's nowhere to be found. Am I missing something?

What do you mean with git's bin and usr\bin directory? /usr/bin should always be in your path, if not you'll encounter various other problems, of which building Vim is probably the most uninteresting. I mean, binaries you are using should always be in the path, whether it be /bin, /usr/bin or even /usr/local/bin or /opt/bin. I don't think this is essentially a Vim problem you are seeing here.

Understood. I just thought it would be worthwhile to note, but sorry if I wasted your time.

Okay. However, improvements to the documentation is always appreciated. So if you have concrete examples, what needs to be improved, let us know.

Thanks. I just thought that if somebody did what I did and decided to clean their PATH (there were many non-existent directories there), it would be helpful. After all, I was able to successfully build Vim once I added both Git directories back to the path.


RecorderDemo (iOS) on device produces very low volume playback

On simulator, everything works as expected; but when running on a device the playback is heavily muted.

I bet its coming out of the ringtone speaker, not the actual speakers.

The earpiece is a speaker as well. Is it coming out of that?  I looked at your example project code and its definitely a style I haven't seen implemented before.  The core team and I are at NAMM this week and I'll try to have a couple of us look at this issue, and the other similar ones.

@aure ah yes indeed it is coming out of the earpiece.

I think was already addressed:
https://github.com/audiokit/AudioKit/issues/662
do {
//  try AKSettings.setSession(category: .playAndRecord, with:  AVAudioSessionCategoryOptions.defaultToSpeaker)

let session = AVAudioSession.sharedInstance()
try session.setCategory(AVAudioSessionCategoryPlayAndRecord)

if !AKSettings.headPhonesPlugged {
    try session.overrideOutputAudioPort(AVAudioSessionPortOverride.speaker)
 }

} catch {
print("Errored setting category.")
}


Add "Sematext Cloud" to Observability & Analysis

Project or Product name:  Sematext Cloud
Is it open source? (if you are tempted to say partially, please answer no): No
What are the best URL(s) for it? Repo and homepage (if different) for open source, product homepage for closed source.:  Homepage: http://sematext.com/kubernetes/  GitHub: https://github.com/sematext/sematext-agent-docker
What category does it belong in? Observability & Analysis
Sematext Docker Agent collects metrics, logs, and events. The product would fit in Logging, Monitoring, and Tracing. Is there a "general" section for an all-in-one solution?
What's the URL for the best logo (and it needs to include the name)? https://sematext.com/wp-content/uploads/2017/09/octi-square-logo-200x200-2.png
Anything else we need to know?:
Sematext Docker Agent is Docker Certified and Red Hat Certified. Sematext is Red Hat and Kubernetes partner.

https://kubernetes.io/partners/
http://blog.kubernetes.io/2016/11/kubernetes-containers-logging-monitoring-with-sematext.html
https://access.redhat.com/containers/?tab=overview#/registry.connect.redhat.com/sematext/sematext-agent-docker
https://store.docker.com/images/sematext-agent-monitoring-and-logging

Hi @megastef, we are including Sematext in the Logging section for the upcoming version, releasing soon! Thanks

That's great.  Note that Sematext provides Infrastructure Monitoring (including containers) and APM, too.

Understood @otisg, however we generally limit to 1 category. Closing as we placed in best fit - Logging, please reopen if you'd prefer a different location. Thanks


RuntimeException: The process has been signaled with signal "5". (OS X 10.6.8/MAMP)

After installing gitlist and trying to browse a local repository that I'd just created, the following error message was displayed:
RuntimeException: The process has been signaled with signal "5".

in /Applications/MAMP/htdocs/gitlist/vendor/symfony/process/Symfony/Component/Process/Process.php line 332
at Process->wait() in /Applications/MAMP/htdocs/gitlist/vendor/symfony/process/Symfony/Component/Process/Process.php line 200
at Process->run() in /Applications/MAMP/htdocs/gitlist/vendor/klaussilveira/gitter/lib/Gitter/Client.php line 71
at Client->run(object(Repository), 'branch') in /Applications/MAMP/htdocs/gitlist/vendor/klaussilveira/gitter/lib/Gitter/Repository.php line 164
at Repository->getBranches() in /Applications/MAMP/htdocs/gitlist/src/GitList/Util/Routing.php line 44
at Routing->parseCommitishPathParam('master', 'test') in /Applications/MAMP/htdocs/gitlist/src/GitList/Controller/TreeController.php line 22
at GitList\Controller{closure}('test', 'master')
at call_user_func_array(object(Closure), array('test', 'master')) in /Applications/MAMP/htdocs/gitlist/vendor/symfony/http-kernel/Symfony/Component/HttpKernel/HttpKernel.php line 126
at HttpKernel->handleRaw(object(Request), '1') in /Applications/MAMP/htdocs/gitlist/vendor/symfony/http-kernel/Symfony/Component/HttpKernel/HttpKernel.php line 66
at HttpKernel->handle(object(Request), '1', true) in /Applications/MAMP/htdocs/gitlist/vendor/silex/silex/src/Silex/Application.php line 538
at Application->handle(object(Request)) in /Applications/MAMP/htdocs/gitlist/vendor/silex/silex/src/Silex/Application.php line 515
at Application->run() in /Applications/MAMP/htdocs/gitlist/index.php line 32

This appears to be an issue with Symfony and the DYLD_LIBRARY_PATH set by MAMP. To confirm what this was set to I used this code:
var_dump(getenv('DYLD_LIBRARY_PATH'));

string(31) "/Applications/MAMP/Library/lib:"

If I edit /gitlist/vendor/symfony/process/Symfony/Component/Process/Process.php and add
putenv("DYLD_LIBRARY_PATH=''");

at the start of the script (after the namespace and use declarations) the error message isn't displayed and I can browse the repository without any issues.
My server configuration is OS 10.6.8, MAMP 1.9.6 and PHP 5.3.5.

Seems to be a duplicate of #512.


Remove /tmp/MariaDB4j at the end to enable easy multiple system users support

If both temporary directories (base and data) are removed, maybe we could also remove the parent directory (/tmp/MariaDB4j). This would avoid the following issue:

run as user1

ok

run as user2 (in my case, that's the jenkins user on a local Jenkins installation)

expected: no error
actual: java.io.IOException: Unable to create directory /tmp/MariaDB4j/base

cause: /tmp/MariaDB4j exists and it's only writable by user1 (rwxr-xr-x)

Hello!
Thank you for your interest in this project.
You raise a fair point. If you would like to contribute to this project, please raise a Pull Request about this.
M.

FTR while (finally) updating the CHANGELOG today, I remembered that #604 added something related to this. That allows to "manually" do what this suggests - I guess the goal here would be for someone to "automate" it. Go for it and raise a PR!

@membersound @jzillmann @A248 thanks for your tumbs up (above) on this issue; FYI I'm cleaning up old issue, and as part of that closing this one which hasn't seen any community engagement in ~1 year; any future readers, please open a new issue, or (better) a PR with a fix!


clean.txt import error

Traceback (most recent call last):
  File "H:\Deepfacelab\Voice\fish-speech\tools\webui.py", line 23, in <module>
    from tools.api import decode_vq_tokens, encode_reference
  File "H:\Deepfacelab\Voice\fish-speech\tools\api.py", line 35, in <module>
    from tools.llama.generate import (
  File "H:\Deepfacelab\Voice\fish-speech\tools\llama\generate.py", line 19, in <module>
    from fish_speech.text import clean_text, split_text
ImportError: cannot import name 'clean_text' from 'fish_speech.text' (H:\Deepfacelab\Voice\fish-speech\fish_speech\text\__init__.py)

Describe your procedure.


Changes done in about page and make it visible.

Title :[Feature Request]: About Section Not Working on Myntra Clone .
Issue No. : #866
Code Stack : Css
Close #866 >

Video (mandatory)
https://github.com/user-attachments/assets/60df74d8-859a-4cfc-922c-9cca76528484
Before:

After:

Checklist:

[X ] I have mentioned the issue number in my Pull Request.
[ X] I have commented my code, particularly in hard-to-understand areas
[X ] I have created a helpful and easy to understand README.md
[X ] I have gone through the  contributing.md file before contributing

Additional context (Mandatory )
Are you contributing under any Open-source programme?

[ ] I'm a GSSOC'24 contributor
[X ] I'm a SSOC'24 contributor

@anuragverma108 please review it and merge it

@anuragverma108 please review it and merge

@apu52 please review this and merge it


Return error on v2pull in case of a digest mismatch

There seems to be a regression from Docker 1.7.1, where we were verifying the digests of individual layers and returning an error. Somehow, 1.8 does not have that code anymore.
Me @jlhawn and @dmcgowan are currently working on creating some integration tests to make sure this doesn't happen again.
Signed-off-by: Diogo Monica<EMAIL_ADDRESS>

LGTM

Integration tests on this fork PR: https://github.com/diogomonica/docker/pull/2

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "error_on_v2pull_hash_mismatch" git@github.com:diogomonica/docker.git somewhere
$ cd somewhere
$ git rebase -i HEAD~4
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

This PR is going to be superseded by one from @jlhawn

@jlhawn please add a link here once that PR is up, thanks!

Sorry, It's been up since this one closed. Here it is: https://github.com/docker/docker/pull/15222

No problem, thanks!


Update quick-start.md

Avoid “current URL string parser is deprecated” warning by setting useNewUrlParser to true

Thanks for the contribution @unkn0wnU53r !


Fix Korean translation (grammar, naturalness)

I recently returned on FFXIV and I found xivanalysis using FFlogs. While using the service, I found some misspellings and unnaturalness of the sentence, and then corrected it to promote it.
Thank you for making a good service, and I hope my contribution helps.
My main job is Monk, so I only worked with content that has the mnk tag. I will try to contribute to the translation of other jobs.

@Fleta Thanks for your interest!  We do our translations through Crowdin, at https://crowdin.com/project/xivanalysis - if you can add them to the dictionaries there, we'll get them picked up in our next translation merge pass.


Generate notification when mentioning a user

Change-type: minor

Relies on https://github.com/product-os/jellyfish-worker/pull/1647 to be testable, as currently the mentions fields are only populated if the event was created in the UI.

@balena-ci retest

@balena-ci I self-certify!


[DEVOPS-464] stack2nix: Pin down hackage db.

https://iohk.myjetbrains.com/youtrack/issue/DEVOPS-464
Currently hitting https://github.com/input-output-hk/stack2nix/issues/66, should be tested.
Fixes https://github.com/input-output-hk/stack2nix/issues/52

Works locally for me with caveats:

cabal2nix 2.7 is not in nixpkgs yet, so it had to be built manually (this blocks this PR until we have nixpkgs with it)
lots of debugging output ended up in stdout, so I had to strip it out
stack2nix changed hashes:

       mkDerivation {
           pname = "Cabal";
           version = "<IP_ADDRESS>";
-          sha256 = "0chhl2113jbahd5gychx9rdqj1aw22h7dyj6z44871hzqxngx6bc";
+          sha256 = "6c99fe6cc71f868308f946fa76a0105c05895b4e1d32ff4a836ac91182a01
032";
           libraryHaskellDepends = [
             array
             base

Invoked
$ PATH=/nix/store/7cbhi0yjm185cb987d0vbsnwz410179r-cabal2nix-2.7/bin:$PATH /nix/store/16zdkwsxh181ilcjmmmws8ydbssdifk8-stack2nix-<IP_ADDRESS>/bin/stack2nix --hackage-snapshot 2017-10-30T12:18:35Z . > stack2nix.nix

The latest commit switches to using cabal2nix 2.7 as a library, eliminating the need for the runtime binary dependency.

@jmitchell seems we have a bit more impurity. Since you added happy to stack, it didn't generate the package for it.
Also I had to apply following diff:
diff --git a/default.nix b/default.nix
index 6940458..4117e62 100644
--- a/default.nix
+++ b/default.nix
@@ -6,8 +6,7 @@ with pkgs.haskell.lib;
   overrides = self: super: {
     stack2nix = justStaticExecutables super.stack2nix;

-    # https://github.com/input-output-hk/stack2nix/issues/51
-    Cabal = pkgs.haskell.packages.ghc802.Cabal_2_0_0_2;
+    Cabal_2_0_0_2 = super.Cabal;

     # https://github.com/NixOS/cabal2nix/issues/146
     hinotify = if pkgs.stdenv.isDarwin then self.hfsevents else super.hinotify;

@jmitchell also we'll have to use threaded runtime now

Done.

@jmitchell it was confirmed this fixes https://github.com/input-output-hk/stack2nix/issues/66

@jmitchell I think if we pin hackage-db while checking stack2nix output we're good to merge this as a stop-gap.


Add debug-logging

It would be useful in developing and debugging to add some kind of logging-engine with different levels that is configurable for users with an argument or environment-variable.

Ideally some structured logging, so it can be ingested and parsed properly.


Consider using xUnit as testing framework

It seems like xUnit would be a better choice for testing, especially since other main .NET repos are using (e.g. corefx, iot, etc.).  Would it be possible to convert before too many tests are created?

@shaggygi Could you please share why xUnit would be better, may be share some benefits over what we have today? That would help us make a educated decision.

@amanbha This probably comes down to my experience while working on projects (both for employer and GitHub), so don't let me sway if the core dapr team already has experience/decision to use the current test framework.  I've also used NUnit, but as mentioned, it appears xUnit has a good traction in the wild and is used by many of the core .NET repos like Roslyn and such.
I'm sure they all have similar features.  One thing I like about xUnit is it includes its own analyzers to help improve tests.
I'll keep reviewing and reading the docs.

@shaggygi I don't have strong preference for xunit or mstest, If xunit is more preferable by community and makes the life easier then sure, lets use it.

@amanbha thank you for being flexible.  If you don't mind, let me add the xunit packages while I'm working on #121.  I don't think there should be any conflicts with both.  This would allow you to review the PR and if all good, I can circle back and update the current tests with xunit and remove the older package dependencies.  Thoughts?

@amanbha thank you for being flexible. If you don't mind, let me add the xunit packages while I'm working on #121. I don't think there should be any conflicts with both. This would allow you to review the PR and if all good, I can circle back and update the current tests with xunit and remove the older package dependencies. Thoughts?

Would there be an issue with 2 different Adapters? I believe it should be fine, You can go ahead and make the changes to all.


fix(query): type return of useMutation query

Status
READY
Description
Fix #1195
Using orval in a repository where the tsconfig moduleResolution property is not node results in https://github.com/anymaniax/orval/issues/1195 when producing useMutation queries.
As per the issue, looks like it needs to be fixed properly upstream but in the meantime, this PR puts in a fix which should silence that error for users using orval with moduleResolution set to something newer than node.
Todos

[ ] Tests
[ ] Documentation
[ ] Changelog Entry (unreleased)

Steps to Test or Reproduce
Outline the steps to test or reproduce the PR here.
> git pull --prune
> git checkout <branch>
> grunt jasmine

Run yarn build

This LGTM but going to let @anymaniax review.

@anymaniax could I get your eyes on this please? Thank you 🙏

Seems good. Why only for react query?

Seems good. Why only for react query?

Hi @anymaniax,
Sorry for the delay in getting back.
I am using orval with react-query that's why. I am not sure about the other libraries.

@anymaniax,
Updated the mutation queries for react-query, svelte-query and vue-query


CI error message: The virtual environment found in  seems to be broken.

In GitHub workflow logs, poetry doesn't seem to be able to used the virtual environment that is restored from cache and outputs the following messages:

The virtual environment found in folder seems to be broken.

Example workflow run: https://github.com/RasaHQ/rasa/pull/6211/checks?check_run_id=876354752
It might affect all our builds. If we manage to fix it it could speed up build times (it seems that it doesn't happen in Rasa X).

Closing as can't reproduce anymore


A similar css has led to some problems

A List Page and a Detail Page.
I wrote a function to generate a class prefix so I not need to think a unique class, but this made a problem。
Look at the 2 gif, one is refresh then List into Detail, and other is refresh then Detail into List.

I think because the two page has the same css, but the prefix is not the same.

I saw when into a new page, the old style will be deleted. and goBack to the old page, the style not append.
I guess the styled-jsx fail to judge the variable and not append the old style.
Then I modified one of the styles to make them a little different and it work well.

@czy0729 can you put together a sample repo (very minimal) with the necessary code to reproduce the issue? Looking at some code and try it would make it easier to understand what is going on.

I try to write the code in codepen , but I don't know how to import styled-jsx in codepen (；′⌒`)
I make a function to generate unique class like
function generateUniqueId() {
   return `styles-${uniqueId++}`;
}

So I can get different class in different js to pass to the third-party component because I hate to wrap a unnecessary div.
const prefixCls = generateUniqueId();

<CountDown
    className={`${prefixCls}__count-down`}
    start={beginTime}
    end={endTime}
    current={current}
/>

<style jsx global>{`
    .${prefixCls}__count-down {
        position: absolute;
        top: 0;
        left: ${Styles.wind};
    }
`}</style>

But this solution causes me to appear in two pages with exactly the same code and resulting in the following:
Page List

and goto the Page Detail

now goBack to the Page List

styles-2__count-down has lost, so the left padding is 0

now I made two style a little different like one page is
<style jsx global>{`
    .${prefixCls}__count-down-b {
        position: absolute;
        top: 0;
        left: ${Styles.wind};
    }
`}</style>

and became well

@giuseppeg is there some config can disable the performance of auto delete historical  styles ?


[DBZ-4809] Add task id and partition to the logging context...

...for multi-partition connectors.
Co-authored-by: Sergei Morozov<EMAIL_ADDRESS>-- Resolves https://issues.redhat.com/browse/DBZ-4809

@gunnarmorling here's a pr for updating the logging context with task id/partition names.

This LGTM, thanks, @jribera-sugarcrm! @morozov, any thoughts from your side? The test failure should be unrelated, it's the first time I'm seeing it (the test itself was added rather recently), so I'll log an issue in case it shows up again.

Looks good to me too.

Merged; thanks a lot, @jribera-sugarcrm!


API: Investigate Relative Time

The last 30 chat messages which appear in the internal endpoint have a relative timestamp of when they were sent.
I need to find out whether this is always "x minutes ago" or whether it changes to a different format once it's longer. (Want to start reading this information into the app, and storing it for the new messages, in order to display a richer chat experience)

@runmyrobot Maybe you could answer this?
I've just noticed it does "x seconds ago" but it does that at 79 seconds. Where is the boundary for that? 111 seconds ago -> 2 minutes ago?
Is there any boundary above minutes ago? Hours ago? Days ago? etc?

Low priority, skip for now.

@runmyrobot How difficult would it be to return the absolute time rather than relative time? Makes my life a lot easier than having to account for all these situations (and is just more powerful honestly)


dragging support

it will be nice if you add dragging support !!

If you are talking about dragging a pane, it is out of the scope of the library.
Related: #88

Why is it out of the scope of the library? what can be done to enable this

Why is it out of the scope of the library?

Please refer the related comment:

Tweakpane has an option container to specify its parent element and users can locate the pane to anywhere with that option. So I think a position of the pane is out of scope of the library.
https://github.com/cocopon/tweakpane/issues/88#issuecomment-692647065

The scope of Tweakpane is as far as specifying a container element. The position and the size of the pane should be handled by CSS since it's a matter of appearance. If you want to implement a draggable pane, you should make a container element draggable.

Makes sense. Thank you for the explanation!

If case anyone else finds this issue and is interested in a draggable Tweakpane, I have an implementation in a Tweakpane x Svelte" wrapper I wrote, svelte-tweakpane-ui.
This particular implementation is pretty reliant on Svelte, but the code still might be helpful.


ci: Use windows-latest in github actions

CI Vulkan-Tools build queued with queue ID 8565.

CI Vulkan-Tools build # 576 running.

CI Vulkan-Tools build # 576 passed.


[Snyk] Security upgrade golang from 1.22.5-bullseye to 1.23.0-bullseye

Snyk has created this PR to fix 4 vulnerabilities in the dockerfile dependencies of this project.
Keeping your Docker base image up-to-date means you’ll benefit from security fixes in the latest version of your chosen image.
Snyk changed the following file(s):

Dockerfile

We recommend upgrading to golang:1.23.0-bullseye, as this image has only 172 known vulnerabilities. To do this, merge this pull request, then verify your application still works as expected.
Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Link Following SNYK-DEBIAN11-GIT-6846200
  714

Link Following SNYK-DEBIAN11-GIT-6846200
  714

CVE-2019-1387 SNYK-DEBIAN11-GIT-537128
  614

Arbitrary Code Injection SNYK-DEBIAN11-GIT-5461943
  614

Directory Traversal SNYK-DEBIAN11-GIT-5461949
  614

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Directory Traversal

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 75.45%. Comparing base (6745eff) to head (3aec36d).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##           master      #52   +/-   ##
=======================================
  Coverage   75.45%   75.45%
=======================================
  Files          16       16
  Lines        1422     1422
=======================================
  Hits         1073     1073
  Misses        326      326
  Partials       23       23

Flag
Coverage Δ

unittests
75.45% <ø> (ø)

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.


Surprising operator overloads for Coin

The impls of std::ops arithmetic operator traits for Coin have a Result type as the output. This somewhat defeats the purpose of operator overloading by encumbering the notation with mandatory result checking, and violates an API guideline on operator overloading acting predictably.
A more conventional solution for checked math would be methods on Coin, similar to the checked_* methods found on built-in integer types.

it's not unpredictable to add a result here, just like the checked api provide you with an option. My take is that, the guideline really talk about using operator to do something not even related to their operation (for example concating with +).
Now whether we come up with a better way, we could look into it eventually

The user is supposed to be able to do e.g. a + b + c.
As a compromise, it may be possible to hide the possible failure in an intermediate result type like CoinExpression and have that interoperate with Coin operands. Then, the only explicit check to perform would be at the final assignment to a Coin value.
Custom method-based APIs can be anything because the method use is self-evident and can be looked up in documentation if necessary. Not so for overloaded operators.

Case in point: instead of code like this:
        let total = iter
            .fold(Ok(Coin::zero()), |acc, v| {
                acc.and_then(|acc| acc + v)
            })?;

It should be possible to write like this:
        let total = iter
            .fold(Coin::zero_acc(), |acc, v| acc + v)
            .try_into()?;

Only the final assignment of CoinExpression to a valid Coin would need an explicit conversion method.


Update README.md

Observed a few changes. Suggesting the same. Kindly check.

The change from "logo" to "logos" is incorrect. The remainder are purely style changes that are inconsistent with the remainder of the Tomcat documentation. We use "open source" not "open-source". We don't use oxford commas.


Update defender-for-containers-enable.md

Docs Build status updates of commit d785160:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/defender-for-cloud/defender-for-containers-enable.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@ElazarK
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@mayaherskovic we will have to remember this for post ignite and it in then

#please-close


Introduced MessageExample, fixed SingleMessage.exmaples and and MessageTrait.examples

Fixed broken spec rendering with examples for Message/MessageTrait.

Hopefully finished - it should fix #174. Once merged I could update tapir to fix spec generation from endpoint definitions.

Thank you!


Collapse nodes in time-scaled tree with low bootstrap support

Examining fasttree2 output, it is a bifurcating (binary) tree but many of the internal branches with low bootstrap support have near zero branch lengths.  We should add a step that converts these into actual polytomies and then use the --keep-polytomies option in treetime to prevent these from being resolved.

Made the following changes in dev:
-def fasttree(fasta, binpath='fasttree2', seed=1, gtr=True):
+def fasttree(fasta, binpath='fasttree2', seed=1, gtr=True, collapse=True):
     """
     Wrapper for FastTree2, passing FASTA as stdin and capturing the
     resulting Newick tree string as stdout.
@@ -39,7 +39,15 @@ def fasttree(fasta, binpath='fasttree2', seed=1, gtr=True):

     p = Popen(cmd, stdin=PIPE, stdout=PIPE)
     stdout, stderr = p.communicate(input=in_str.encode('utf-8'))
-    return stdout.decode('utf-8')
+    nwk = stdout.decode('utf-8')
+
+    # collapse low support nodes into polytomies
+    if collapse:
+        phy = Phylo.read(StringIO(nwk), format='newick')
+        phy.collapse_all(lambda x: x.confidence is not None and
+                                   x.confidence < 0.5)
+        nwk = phy.format('newick')
+    return nwk

 def treetime(nwk, fasta, outdir, binpath='treetime', clock=None, verbosity=1):
@@ -71,7 +79,9 @@ def treetime(nwk, fasta, outdir, binpath='treetime', clock=None, verbosity=1):
             '--aln', alnfile.name, '--dates', datefile.name,
             '--outdir', outdir, '--verbose', str(verbosity),
             '--plot-rtt', 'none',  # see issue #66
-            '--clock-filter', '0']  # issue #245
+            '--clock-filter', '0',  # issue #245
+            '--keep-polytomies'  # issue #339
+    ]
     if clock:

Using the following script to test:
import json
from batch import parse_args
from covizu.utils.progress_utils import Callback
from covizu import treetime

cb = Callback()
args = parse_args()

cb.callback("Loading JSON data")
by_lineage = json.load(open("data/by_lineage.json"))

lineages = {}
with open(args.lineages) as handle:
    for line in handle:
        taxon, lineage = line.strip().split(',')
        lineages.update({taxon: lineage})

cb.callback("Identifying representative genomes")
fasta = treetime.retrieve_genomes(by_lineage, known_seqs=lineages, ref_file=args.ref, earliest=True)

with open("repgenomes.fasta", 'w') as outfile:
    for h, s in fasta.items():
        outfile.write(">{}\n{}\n".format(h, s))

nwk = treetime.fasttree(fasta, binpath=args.ft2bin)

with open("repgenomes.ft2.nwk", 'w') as nwkfile:
    nwkfile.write(nwk)

treetime.treetime(nwk, fasta, outdir="issue339")

Treetime output before and after collapsing polytomies (above code):

Zooming in on B.1.617.*
Before:

After:


Refactor CRC interfaces and implementations

This PR:

Creates a templated CRCChecker interface class, as well as CRC8 and CRC32 aliases which can be used as interfaces.
Removes inclusion of the Pufferfish/HAL/STM32/CRC.h header from driver files so that they can compile without the STM32 libraries.
Updates the CRC32C-based driver classes to take the CRC32 interface in their constructors, rather than an STM32-specific CRC32C class.
Generalizes the CRC8 software implementation to work for CRCs of any size in the templated SoftCRC class in Pufferfish/HAL/CRCChecker.h. This class subclasses CRCChecker and is usable both for any CRC8 and for CRC32C. It uses a table lookup-based algorithm rather than the naive bit-by-bit algorithm. Because it implements the CRC32 interface, it should be possible to use the SoftCRC32 as a drop-in replacement for the hardware-based HALCRC32 in unit tests by dependency injection, though I haven't written a unit test confirming this.
Defines a templated CRCParameters struct for initializing SoftCRC objects in the SFM3019 driver and other Sensirion drivers.
Simplifies the interface of SensirionDevice by passing the CRC implementation in the constructor, rather than passing in CRC parameters with every method call.
Creates Catch2 unit tests for CRC8 (with SDP3019 parameters) and CRC32C, matching the example inputs which were used in a unit test in the backend (though that test was deleted in #90 because we moved to a third-party library for CRC in python).
Adds clang-tidy checks for the Catch2 unit tests to the Github Actions workflow.
Fixes clang-tidy issues for the Catch2 Nonin OEM III unit tests written by Hemanth.

As discussed with Renji on Slack, this PR appears to work, so I will merge this PR in first. For records-keeping:

This project is licensed under Apache License v2.0 for any software, and Solderpad Hardware License v2.1 for any hardware - do you agree that your contributions to this project will be under these licenses, too? Yes
Were any of these contributions also part of work you did for an employer or a client? No
Does this work include, or is it based on, any third-party work which you did not create? No


gcp-project should be an optional flag

We should auto detect the project from gcloud config

Fixed with #172 and #176.


refactor(mock-factory): change the return type of mock factory (#40)

Codecov Report

Merging #48 (d214d28) into master (c1422d5) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master       #48   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           20        20
  Lines          226       230    +4
  Branches        22        22
=========================================
+ Hits           226       230    +4

Impacted Files
Coverage Δ

src/class-processor.ts
100.00% <100.00%> (ø)

src/class-reflector.ts
100.00% <100.00%> (ø)

src/factories/mock-factory.ts
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Update test_power and test_takeoff functions.

Well done sir, it's working like a charm.
Merging PR


Add ability to use source reference as a fall back

Currently when the source reference is returned over the debugger protocol, the source is retrived via the debug adapter (ie remote source is always used).
Would it be possible for vs code to check if the file exists locally, if not then fall back to using the source reference.
This would greatly simplify remote debugging , at least in Python. This way user doesn't need to configure any path (local or remote) mappings.
Here's the work flow:

In the attach request we'll (extension in vscode) send current work space as the default local root
On the remote side when attach request is received, we'll map the local root (received) to the current directory (serves as remote root)
We return mapped paths along with a source reference (in stack traces)
VSCode will check if the file exists locally, if yes then use that else use the source reference.

If on the other hand the user provides the path mappings, the remote debugger will not return source reference (number). Hence vscode will always look for those files locally.

No longer required


Couple of simple detections

Fixes #
Proposed Changes

New TI Detection (IP) against App Services HTTP Logs
New App Services AV Detections based on new feature (https://azure.github.io/AppService/2020/12/09/AzMon-AppServiceAntiVirusScanLogs.html)

@shainw  i updated the 3 files per feedback.  Thanks!

/azp run

/azp run


v14 Changelog

This is an automatically generated pull request.
Please review and merge the changes.
Context
Updated changelog for v14 release
Brief Changelog

Updated main changelog with on-chain and off-chain changes

closing as this was only opened for debugging purposes


Is there any refresh, reload or destroy method?

Currently No but i will add it for you.
I want more information
For what purpose you want this
Thanks

can i  have some methods to jump to special index's image?
such as  method  jump(3) , and it will slide to third image.

at last i find it in source code :  moveTo(i) .

I wanna create dynamic slider. So i must reload slider.

@musaay you can use ngFor or extend carouselComponent to add some methods

so what was the solution to slide to a specific image in the carousel @icepoint0 @sheikalthaf

I think these methods should be added. Because it can be needed

I have already included the method to slide to the particular point number. moveToSlide is an input which accepts the number to slide.
I will push the update tomorrow with details. wait for the tomorrow update
Thanks for the suggestion

I have released the new build 1.3.0. read the changelog file

For example, you have a filter. I am searching and there are 20 products. Then I search with new criteria and no results are found. In this case, slider should destroy. How can i do that ?

@musaay I will try to implement this for you. but I cant tell the timeline of the implementation

@sheikalthaf Also really need the ability to "reset" whole carousel (in ngOnChanges) so it behaevs like if I just started it for the first time. I am using the carousel in modal so my component doesn't get destroyed and nor does the carousel.
Would really appreciate if you could implement something asap.

Hello Sheik,
I have simple fix to close this dynamic add values to the carousel. we need to call the transform function under buttoncontrol
if(this.userData.grid[this.data.deviceType] >= this.carouselItems.length){
this.setStyle(this.leftBtn, 'display', 'none');
this.setStyle(this.rightBtn, 'display', 'none');
        this.transformStyle(0);
    }


Earth occultation in the image deconvolution (ScAtt binning)

I added a functionality to consider the earth occultation in the coordinate conversion matrix.
The earth occultation can be included in the analysis of the image deconvolution with the scatt binning + local coordinate.
I want to emphasize that it is limited to the case that the satellite always points to the zenith. So, it is valid for the DC2 but not for general situations.
Also, I changed the codes additionally as follows. All of them are minor changes.

mask a part of the sky if such a region has zero exposure time.
add a parameter mininum_flux in deconvolution_algorithm_base in order not to make pixels with minus flux. The default value is 0.
modified the implementation of the function that calculates the acceleration parameter in the RL algorithm.
use hp.smoothing for the Gaussian image smoothing instead of preparing a large matrix.

I compared the reconstructed spectrum of 3-month Crab data with/without considering the earth occultation effect.
The reconstructed flux becomes more consistent with the injected model, especially in the higher energy band where gamma rays can pass through the bottom BGO shield more easily. For example, the difference changes from -26% to 0.3% at the highest energy band.

Hi @hirokiyoneda. Thanks for working on this. I'm OK with all the changes you listed, but I'm on the fence about adding the Earth occultation code itself.
For this workaround, specific to DC2, we don't need to change the code at all, just change the detector response file. Since this new code is not meant to be definitive, and the same can be achieved with a simpler solution --that doesn't involve people getting a new release-- I thought it was cleaner to leave the code as is and save us from more code that needs to be tested and maintained. What do you think?

That's a good point. I wanted to merge it into the development branch because:

The change in the coordinate conversion matrix does not harm the DC2. While I added a new parameter, earth_horizon_angle, the old notebooks published for DC2 should work since earth_horizon_angle is not a mandatory parameter.
To consider the earth occultation effect in a general way, I understood that we need information about the satellite altitude and position in the orientation file. So, I think that we also need to modify the orientation. I think that this code can be a good example to understand and discuss it.

But I agree that it is cleaner to keep it as it is.
One concern is that I made minor changes following the earth occultation function. Actually, my current other works depend on them. I am happy if I can merge them into the develop branch. Should I do PR including these minor changes but without the earth occultation functions? Is it a good way to remove the commit regarding with the earth occultation, probably using git revert?

Depending on your situation, you can use either git rebase --onto (see this) or git cherry-pick (here). I'll follow up on this over slack.
You can create a branch called e.g. earth_occ_DC2 from the commit that has the Earth occultation code, but before you made these other changes, and maybe open a draft PR with it. I agree we should keep it around as an example and discuss it when we work out the actual code that works for the general, but no need to merge it now.

Thanks a lot. I am closing this PR.  I have made a draft PR by removing unrelated changes as #184.


Add documentation how CSS Custom Property handling internally works

We need to clarify the CSS Custom Property (aka CSS Variables) handling within the elements. It's confusing as we use a mix of built-time and run-time logic as well as default values.
The description should be targeted mainly towards contributors, but it's also important for developers working with the elements to have a solid understanding how CSS Custom properties are processed internally.
Necessary points we need to do:

Add a new entry CSS Variables within the Doc section in Storybook.
Describe the advantages of using CSS Custom Properties.
Describe the possibilities for the developer point of view.
Describe the possibilities for the contributor point of view.
Add some CSS examples for two or three components for a developer using the elements.

should be documented in our developer docs


Feature Request: using Env vars in .fresh config or load from .envrc file

Or is this possible already, couldn't find anything?
This may be something I can help out with, if desired.

Sure, absolutely! I'll look into both issues during this week


Parking lot

This is a holding place for issues that arise in working toward 1.0, but aren't worth a separate issue that will likely become stale for a while. If you want to work on one of these tasks, feel free to open a new issue dedicated to that task. I want to avoid having a large number of small issues open, making the project look less maintained than it is.

[ ] When running Platform.sh e2e tests, if the CLI is not authenticated the test hangs on the first run. A subsequent run catches the not authenticated error message. Not obvious to troubleshoot quickly, because it's only some runs that hang. Problem likely in platform_sh/tests/e2e_tests/utils.py:

def check_logged_in():
    """Check that user is currently logged in to Platform.sh through CLI."""
    print("\nVerifying logged in to Platform.sh CLI...")
    auth_info_output = make_sp_call("platform auth:info --quiet", capture_output=True)
    if "LoginRequiredException" in auth_info_output.stderr.decode():
        msg = "\n----- Error: Not logged in through CLI -----"
        msg += "\nPlease log in to the Platform.sh CLI and then run the e2e test."
        msg += "\n  You can log in with the command: platform login"
        msg += "\n-----\n"
        print(msg)

        exit_msg = "Please run `platform login` and then run e2e tests."
        pytest.exit(exit_msg)

[ ] Separate private utils from public utils. Public utils are utils that are meant to be used by plugins.

Simplifying plugin_utils
I don't really like how many of the plugin_utils functions require the sd_command argument. It's boilerplate, and not entirely clear. I'm accepting it for now because I like plugin utility functions much better than a set of methods on Command that the plugin needs to call. I think it's better for plugin authors to read through a single module of functions they can use, than to have to look into the simple_deploy.py listing with all its internals.

[ ] Make a plugin_utils.init() function that register required resources such as sd_command.stdout as a global. This is not ideal, with all the issues that globals bring about.
[ ] Create a config module or object containing the elements of state that need to be shared, and pass that around instead of a reference to the command itself. (This seems most appealing!)

[ ] Support consistent approaches across all platforms, such as confirming before overwriting.

[ ] Consider an "all" response, if would like permission to overwrite all files found.
[ ] Make sure approach to selecting from multiple resources such as orgs and project names is consistent.

[ ] In plugin_utils.modify_settings_file(), consider just accepting a string for the modified block. The template would be part of core, since it's the same across platforms?

[ ] This really depends on how likely it is for platform-specific templates to have unique entries in the context dict. Fly passes one, but it isn't used in the template. How likely are platforms to want this?
[ ] If so, plugins can call plugin_utils.get_template_string() for this (or any file!).
[ ] Final reason for not implementing this now (#354): it's rather helpful to see a template that looks like an actual settings.py file when considering how to modify settings. The templates as they are have the same indentation structure as a modified settings.py file.

[ ] Root conftest.py should look for all installed plugins, and discover their tests.

Misc

[ ] Write a stability policy. #214

Testing

[ ] Revisit how plugins are handled, ie in integration_tests/conftest.py.run_simple_deploy(), also manage_sample_project.py.


Index out of bounds crash

pressing 'm' more than 8 times causes the game to crash due to index out of bounds error

This issue has been fixed in the latest version.


Recommendation: The predicted rating is garbled on the Evaluate page.

System Information (please complete the following information):
Windows OS: Windows-11-Enterprise-21H2
ML.Net Model Builder 2019&2022: <IP_ADDRESS>5501 (Main Build)
Microsoft Visual Studio Enterprise: 2019(16.11.18); 2022(17.3.0)
.Net: 5.0 & 6.0
Describe the bug

On which step of the process did you run into an issue:
The predicted rating is garbled on the Evaluate page for Recommendation scenario.

TestMatrix
https://testpass.blob.core.windows.net/test-pass-data/recommendation-ratings.csv
https://testpass.blob.core.windows.net/test-pass-data/iris.mdf
To Reproduce
Steps to reproduce the behavior:

Select Create a new project from the Visual Studio start window.
Choose the C# Console App (.NET Core) project template.
Add model builder by right click on the project.
Select Recommendation scenario.
On Data page, select the data source and completed the training.
On the Evaluate page, click "Predict" button and check the evaluate results.

Expected behavior
The predicted rating is accurate and has no garbled on the Evaluate page.
Screenshot

Additional context:

can't reproduce in latest main

The bug has been fixed on the latest main branch: <IP_ADDRESS>5801


Always expire status keys

My proposed solution to deal with what's happening in https://github.com/Shopify/spaces/issues/370#issuecomment-123894122.

:+1:


support plugins as a target type

You should be able to manage and deploy plugins with Mantle.

Endpoints:

Create plugin: POST https://data.roblox.com/Data/Upload.ashx?allowComments=False&type=Plugin&name=&groupId=&assetId=0&isPublic=True&description=
Configure sales: POST https://itemconfiguration.roblox.com/v1/assets//release

Body: { "price": 0, "saleStatus": "OffSale" }

Create icon: POST https://publish.roblox.com/v1/assets//thumbnail

Standard multi-part formdata with request.files image

Adding this to milestone v1.0.0 because I would like the targets API to be stable before v1.0.0 so that once we reach it we can add more target types without breaking changes.


What meaning of @ in defaults.yaml?

Can someone tell me the meaning of @ in defaults.yaml？ I can't found its role.
Thanks!

The @ sign is used to indicate specific configuration sections to be loaded or referenced from other parts of the system. This means that a particular key or section is assigned to a specific module or component.
e.g.
<EMAIL_ADDRESS>means that train_datasets should be configured according to the settings defined in DATASETS.TRAIN.
The @ sign allows you to configure your system hierarchically and modularly. For example, you can have a general configuration in one file and detailed, case-specific configurations in others, and combine them using @.


Not an issue, but help.

Hello,
I am trying to use this tool for backuping data to gcloud.
I have added some lines in alfresco-bart.sh :

case $BACKUPTYPE in
"s3" )
export AWS_ACCESS_KEY_ID="${AWS_ACCESS_KEY_ID}"
export AWS_SECRET_ACCESS_KEY="${AWS_SECRET_ACCESS_KEY}"
DEST=${S3FILESYSLOCATION}
PARAMS="${GLOBAL_DUPLICITY_PARMS} ${S3OPTIONS} ${NOENCFLAG}"
;;
"gcloud" )
export GS_ACCESS_KEY_ID="${GS_ACCESS_KEY_ID}"
export GS_SECRET_ACCESS_KEY="${GS_SECRET_ACCESS_KEY}"
DEST=${GCLOUD_FILE_SYS_LOCATION}
PARAMS="${GLOBAL_DUPLICITY_PARMS} ${NOENCFLAG}"
;;
"ftp" )
if [ ${FTPS_ENABLE} == 'false' ]; then
DEST=ftp://${FTP_USER}:${FTP_PASSWORD}@${FTP_SERVER}:${FTP_PORT}/${FTP_FOLDER}
PARAMS="${GLOBAL_DUPLICITY_PARMS} ${NOENCFLAG}"
else
DEST=ftps://${FTP_USER}:${FTP_PASSWORD}@${FTP_SERVER}:${FTP_PORT}/${FTP_FOLDER}
PARAMS="${GLOBAL_DUPLICITY_PARMS} ${NOENCFLAG}"
fi
;;
"scp" )
DEST=scp://${SCP_USER}@${SCP_SERVER}/${SCP_FOLDER}
PARAMS="${GLOBAL_DUPLICITY_PARMS} ${NOENCFLAG}"
;;

I think, I need more configuration or something.
I need help with backup to gcloud bucket.
Thanks.

Hello.
Is gcloud the Google Cloud Storage?
As this option is not listed as supported by this project, you have 2 options: implement it by yourself, or raise an issue asking for a new feature, but in this case, you will have to wait until someone decides to go ahead and implement it.
Judging by the piece of code you shared, it seems you are in the right path, but for sure, that is not enough.
We have just merged some major changes into the code, so I would like to suggest to you to grab the brand new code before trying to implement it.
And if you want to have an idea about how to finish the feature you need, take a look at how I created the new SFTP option in this last PR that has just got merged. It will give you some ideas.
And remember that you will have to install new dependencies in order to have duplicity talking with Google Storage/gcloud.
I have made a quick research, and found this https://medium.com/google-cloud/how-to-make-ubuntu-backups-using-duplicity-and-google-cloud-storage-849edcc4196e
I hope you find this answer useful.

Yes. thank you for your help.
I have not tried it on latest update but somehow it worked.
Thanks, Hopefully we'll get this in easier way.


Moved phpunit to dev dependencies

Problem
PHPUnit was declared as a regular dependency, which meant that any
client consuming this composer package would end up getting phpunit
installed. More importantly, the version requirements for phpunit
imposed by this package would have to be adhered to by clients that
consumed this package.
This causes particular issues with projects that require stable
dependencies, because the phpunit version required by this package
is not stable.
Fix
I moved the phpunit dependency to the dev dependencies.

Paging @TaysirTayyab to review the back-end work in this pull request!
I couldn't recognize who these files were relevant to:
composer.json
composer.lock

!reassign back-end @goodevilgenius

Paging @goodevilgenius to review the back-end work in this pull request!

That did it. Now composer update medology/flexible-mink properly updates to the most recent version.


Inconsistent examples for Yarn

The examples under Key input and Restore keys suggest adding yarn as one of the restore keys.
- task: Cache@2
  inputs:
    key: 'yarn | "$(Agent.OS)" | yarn.lock'
    restoreKeys: |
       yarn | "$(Agent.OS)"
       yarn
    path: $(YARN_CACHE_FOLDER)
  displayName: Cache Yarn packages

This caused our builds to fail when we had multiple platforms as the CLI packages get built and cached for a specific OS and don't work when the cache is moved to a different OS.
The example in Node.js/Yarn section only has yarn | "$(Agent.OS)" which gets around this, as it ensures there is a cache entry for each OS and its platform-specific packages. I think the other two examples should be updated to match this.
steps:
- task: Cache@2
  inputs:
    key: 'yarn | "$(Agent.OS)" | yarn.lock'
    restoreKeys: |
       yarn | "$(Agent.OS)"
    path: $(YARN_CACHE_FOLDER)
  displayName: Cache Yarn packages

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

ID: 77c20289-dab2-7a6e-4f63-4b30eaff2c71
Version Independent ID: d75cc196-90c1-8291-9907-ff405bde4edf
Content: Pipeline caching - Azure Pipelines
Content Source: docs/pipelines/release/caching.md
Product: devops
Technology: devops-cicd-release
GitHub Login: @ramiMSFT
Microsoft Alias: rabououn

@sachinahya Thank you for your feedback!


Feature: Provide internationalization for formatting of numbers

The changes in this PR are to provide internationalization when creating formatted prices. Solves #21
Note: this is built on top of #20 to avoid remerging later

@ChrisBrownie55  Wow! I'm going to look at this later today, could you advice me on the best way to merge? Should I merge #20 first or should I be able to merge this directly?

I would merge #20 first and then this one

@dayhaysoos It should work fine if you do that, I'm not an expert on merging behavior by any means

Cool! I'll take a look at them today, I appreciate you so much. Please excuse me while I try to do all-contributor stuff lol

@all-contributors please add @ChrisBrownie55 for code


Better performance in rgb for SAM demo, but worse in 3d slicer

First of all, thank you very much for your excellent work.
But I noticed an interesting phenomenon that when we load a medical image with 3d slicer the segmentation we get is not good, but if I use the demo from the SAM website to do it with a screenshot of the medical image the result is very good.
Can you please explain the reason for this? This will help us to improve it, thanks!

I am not sure the reason but perhaps it has something to do with the system glitch, which should be improved after the changes in: https://github.com/bingogome/samm/commit/5b63c7dfa120c39066215fb766f8b246be3e2414
Theoretically they should be exactly the same if you put the same prompts. However that won't be the case because of scaling, intensity, coding of screenshot etc. End of the day they should be similar because there is not change in the network design.


Decorators positioning is hard-coded

getDecoratorStyles() on line 635 should have a way to be overwritten. I cannot position decorators on another position then the cases that have been set here.

getDecoratorStyles() is only used for the parent . You can use any style you want in your decorators.
It does seem like a somewhat odd design though.

All this privatizing of code is an illusion of power, there is always a way around it:
class MyNukaCarousel extends NukaCarousel {
  constructor(props) {
    super(props);

    this.getDecoratorStyles = function(position) {
      //put whatever position styles you want here
      return {
        position: 'absolute',
        top: 0,
        left: 0
      };
    };
  }
}

Not pretty, but like , it gets the job done.  Ideally there would be a better way, or just fork, extend, repeat.


HV-1113-1115 Assorted fixes to the XML parsing

https://hibernate.atlassian.net/browse/HV-1113
https://hibernate.atlassian.net/browse/HV-1114

@gunnar wondering if we should have a more comprehensive form of LocalNamespace in validation-api? It could include the MappingXmlParser.SCHEMAS_BY_VERSION information too.
Something similar to https://github.com/hibernate/hibernate-orm/blob/master/hibernate-core/src/main/java/org/hibernate/boot/jaxb/internal/stax/LocalSchema.java without the 3 static methods.

Jenkins, retest this please

Jenkins, retest this please

@gunnar wondering if we should have a more comprehensive form of LocalNamespace in validation-api? It could include the MappingXmlParser.SCHEMAS_BY_VERSION information too.

Dunno, would we gain that much from it? Actually I'm not even fully convinced of the enum itself, seems we only use it once for doing the mapping, i.e. simple constants would be as good.

Rebased and applied. Thanks, @gsmet!

@gunnarmorling well, it would be more useful if we add the schema paths and versions and if we include it in validation-api, it would provide an unified way to access the schemas and the namespace.


[SONARMSBRU-177] Fix capability for invalid SARIF from RTM Roslyn.

Also attempt to pick up NuGet packages automatically using AppVeyor

SonarQube analysis reported 4 issues:

 4 major

Watch the comments in this conversation to review them.
Note: the following issues could not be reported as comments because they are located on lines that are not displayed in this pull request:

 Change the type of parameter 'hostUrl' of method 'Utilities.HandleHostUrlWebException(WebException, string, ILogger)' from string to System.Uri, or provide an overload of 'Utilities.HandleHostUrlWebException(WebException, string, ILogger)', that allows 'hostUrl' to be passed as a System.Uri object.

SonarQube analysis reported 4 issues:

 4 major

Watch the comments in this conversation to review them.
Note: the following issues could not be reported as comments because they are located on lines that are not displayed in this pull request:

 Change the type of parameter 'hostUrl' of method 'Utilities.HandleHostUrlWebException(WebException, string, ILogger)' from string to System.Uri, or provide an overload of 'Utilities.HandleHostUrlWebException(WebException, string, ILogger)', that allows 'hostUrl' to be passed as a System.Uri object.

Closed for some minor changes.

@HSAR note that you can update a pull request by pushing further commits to your branch. such new commits will re-trigger AppVeyor builds. Then, you can rewrite your branch's git history, and merge all these technical commits into just 1 for instance, prior to merging the PR into the master.

Thank you  :)


config for Adam

I'm trying to use the implementation of the adam optimizer available in the optim package with the line
-- not used -- x, batchlowerbound = optim.adagrad(opfunc, parameters, config, state)
x, batchlowerbound = optim.adam(opfunc, parameters, config, state)
It does not seem to converge or even change much with any of the configurations I've tried.
Could you suggest a config which works please?
Thank you.

Update - the adam optimizer was fixed yesterday - it works now, with default parameters on the Rosenbrock test problem.

Great! Curious to see your result.

Hi Joost,
unfortunately I have'nt managed to get any convergence yet with adam, over a range of different config parameters?
Looking at figure 4 a) of the adam paper, it seems that
beta_1 = 0.1
beta_2 = 0.0001
alpha = 0.002
epsilon = 1e-8
lambda = 1 - 1e-8
with the model they state,
dim_hidden = 50
hidden_units_encoder = 500
hidden_units_decoder = 500
should get convergence after after 10 epochs, but I can't reproduce their results?

Clearly the result is much better after 100 epochs (4 b) so those figures do not indicate convergence. It shows the value of the negLL after a set amount of epochs for different learning rates (x-axis) and illustrates the necessity of the bias-correction factor.
I am not sure exactly how many epochs are necessary with Adam, I never tested that.

Hi, yes sorry I was sloppy with my language, applogies.
Basically I've tried all the grid points they mention in their paper, i.e. the bias correction terms beta_1 and beta_2  and learning rate - but I still get the negLL after 10 epochs to be about
-4 e+155
i.e. basically -infinity. Maybe you want to give it try? If you pull the latest  optim module
luarocks install optim
I think its then just a task of
i) changing your while loop to a for loop, running for 10 epochs, and writing the last negLL to a table
ii) Constructing a small grid and iterating your above code over then.
If you really want to be fancy, there's a Bayesian optimization pakage on git called spearmint, which uses a fancy iterative Gaussian process scheme for continuous hyper-parameter optimization. I'm trying it now.
I'm guessing if we both can't get it to work, it might be a problem with the optim.lua code?

I'm not sure adam.lua code in the optim package works.
Maybe it's best to use Dirk Kingma's adam implementation here,
https://github.com/dpkingma/nips14-ssl/blob/master/adam.py
with a theano implementation?

I have tried adam before (in Theano) and it works nicely, so this does indeed sound like a bug, either in my code or in the adam implementation.
I will take a look at it tomorrow.

Fixed by setting negative learning rate (gradient ascent)


Implement Widget on &BarChart or &mut BarChart

Problem
Widget currently cannot be implemented on &BarChart nor &mut BarChart because it mutates its state in a destructive way.
Solution
Change BarChart implementation to not mutate or in a non destructive way.
Additional context
See https://github.com/ratatui-org/ratatui/pull/833#discussion_r1465312072 for some context.

This is fixed by https://github.com/ratatui-org/ratatui/pull/903


Suggestion: add warning when using BertForSequenceClassification without special [CLS] token

Thank you for the awesome package!
As I understand it right now, it is the user's responsibility to add the special CLS and SEP tokens. People who haven't read the paper might miss this detail.
It would be nice to issue a warning in the tokenizer or the model itself if the input is missing these tokens. An alternative is to include it in a more prominent location in the docs.
I understand that the package needs to be flexible so multiple architectures and tokenization procedures can be implemented, so I'm unsure where would be the best place for such a notification.
In short, my suggestion:
GIVEN
A text sequence without the CLS tag
WHEN
BertForSequenceClassification is called
THEN
A warning is logged

I understand the issue but I'm not sure this would be very easy to implement as we would like to keep the model and tokenizer separated one from the other.
Do you have a solution in mind?
Otherwise, I'll guess people will have to continue to read the paper before using the model... 😉

I totally understand the need for separation between tokenizer and model.
But, the specific BertForSequenceClassification does have the assumption and expects the input to be in a specific form. Even the docs mention "look at the preprocessing logic".
I'd suggest extending the documentation for the model - sth like https://github.com/huggingface/pytorch-pretrained-BERT/pull/480


Update to a more recent Plutus repo version

This is the same commit currently being used by the node top level.

bors merge


Type Error in PostNewChatInput

See https://github.com/unipile/unipile-node-sdk/blob/develop/src/types/input/input-messaging.ts#L58, I think we'd want
attendees_ids: string[];
FYI I didn't have the permissions to contribute by pushing a fix for this. Not sure if it's intentional.

Hi Guillaume,
Thanks for pointing this out! We've fixed the issue and it will be included in the upcoming release.
Regarding the permissions, it is intentional to control changes through pull requests for better tracking. Feel free to submit one in the future!


[BEAM-5621] Fix unorderable types in Python 3

This is is part of a series of PRs with goal to make Apache Beam PY3 compatible. The proposal with the outlined approach has been documented here: https://s.apache.org/beam-python-3
This PR suggests another fix to solve the problem of unorderable types in Python 3, as already discussed in #6602.
Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---
---
---
---
---

Java

Python

---

---
---
---

R: @tvalentyn @robertwb @splovyt

cc: @markflyhigh

LGTM, thanks, @RobbeSneyders.

retest this please

Pushed a new change to always raise a BeamAssertException instead of returning False. This seems to be the behavior expected by tests using equal_to.

Rebased on fixed master.

Updated to try original sorting method first and fall back to O(n²) method if needed.
PTAL


Libnd4j/SameDiff Bincount array offset error when calculating shape

When trying to import Bincount from tensorflow I get:
o.n.l.c.n.o.NativeOpExecutioner - Failed to calculate output shapes for op bincount. Attempted to execute with 3 inputs, 0 outputs, 0 targs and 0 iargs. Inputs: [(INT,[],c), (INT,[],c), (INT,[0],c)]. Input var names: [in_0, mul, Const_2]. Output var names: [Bincount] - Please see above message (printed out from c++) for a possible cause of error.

To reproduce:
@Test
public void testBincount(){

    BinCount c = new BinCount();

    c.addInputArgument(Nd4j.createFromArray(8), Nd4j.createFromArray(7), Nd4j.createFromArray(new int[]{}));
    c.calculateOutputShape();
}

My suspicion is that it has to do with the difference between tensorflow's c++ interface and the python interface, wher I think Libnd4j follows the python version while the tensorflow export follows the c++.
The arguments 8, 7, [] make more sense as arr, size, weights (c++) than arr, min, max (python/libnd4j).

Confirmed still an issue

cc @shugeo ^^^


[rancher-python]

Hello,
I have a question about rancher python api.
I need functionality to launch launcher app from catalog.
For now i am using below code.
`
launchAppRequestData = {
        "projectId": projectID,
        "name": appName,
        "prune": False,
        "type": "app",
        "targetNamespace": namespace,
        "externalId": "catalog://?catalog=" + catalogue.lower() + "&template=" + appName.lower() + "&version=" + appVersion}`

requests.post(self.rancherServerAddress + '/projects/' + projectID + '/apps', data=launchAppRequestData, auth=self.accessToken, verify=self.checkSSLFlag).ok
I am trying to dig in into rancher python api.
Is it possible to run app somehow similar to above code through api client in python itself?
Or is it neccesary to use raw requests?

There is operation available but i cannot get which method if there is one is responsible for that in api and on what resource. Could someone help me ?

I believe the same goes for replicaSets DaemonSets Jobs Workloads , is it possible to do it via API?

Why this action to create app isn't listed in projects action list?

Hey @RRadziejewski, yes you can do this through the python client. Anything that exists in rancher can be done through the python client as it is auto generated based off the schema of the server you are using it with.
For the apps, you need a project client as they live under a project. The same is true for the other resources you mentioned as well.
project_url = project.links.self + '/schemas'
project_client = rancher.Client(url=project_url, verify=False, token=<api_token>))
project_client.create_app(...)

@Razikus Creating an app is not a project action, actions affect the resource they live on. Check here for more info.

Thank you for your answer dramich.
I have also a question how to access v3-public api to login with username and password through the api?

You can access it through v3-public/localProviders/local?action=login based off just local auth. If you enabled a provider you can't do it through the API as we redirect to the provider for login. If using the python client you should have an api token to use with it as we don't support the login flow when it's not being done through the UI.

@RRadziejewski if you have any more questions, please open a new issue.


Add debian to the list of supported deb distributions

Signed-off-by: Fathi Boudra<EMAIL_ADDRESS>

I'll resubmit after cleanup


Script becoming unresponsive on Firefox with large dataset

Issue reported by user stating they receive a error message stating that marker cluster has become unresponsive. This issue occurs on a 8,822 point dataset, while running on 3.2GHz i3-550.

I'm going to steal the leaflet contributing guide.
https://github.com/Leaflet/Leaflet/blob/master/CONTRIBUTING.md


Change the tag helpers FileVersion property to something clearer

We have a few tag helpers (Link, Script, and the upcoming Image) that have a FileVersion property that indicates whether to add a "version" to the URL to help with cache busting.
The property name is a bit confusing: it looks like it is a file version, as opposed to saying whether to append a file version.
I recommend changing the property to be AppendVersion or similar because that'll be a lot clearer about what it does.

@DamianEdwards thoughts?

Sure

Any objection to me tackling this one on the weekend?

Yeah that would be great. Let's settle on a name. Here are a few ideas:

AppendFileVersion / AddFileVersion
AppendVersion / AddVersion
AppendFileHash / AddFileHash
AppendHash / AddHash
CacheBust

I do kind of like my original suggestion of AppendVersion because it's concise and reasonably clear about what it does.
Thoughts?

I can live with AppendVersion. I also like AddFileVersion.

I like AppendVersion

Is this about just the property name?  Note that's irrelevant in every scenario except unit testing.  The attribute name (what users see in the Razor source) is currently asp-file-version.
If we're going to change the attribute as well, would the name there be asp-append-version?

Yup.

@dougbu yes it's just the property name and yes we'd rename the tag helper attribute name.
Sounds like everyone wants to go with AppendVersion!

:+1:

3fe0490

The web app template in Visual Studio still uses asp-file-version, looks like it wasn't updated.

/cc @rustd

https://github.com/aspnet/Templates/issues/142

I get a 404 on that link; is it a private repo?

Yeah, sorry about that. Removing the link to avoid further confusion.


skibidi rizzler

this works?

Skibidi toilet kaicenat Kai cenat john pork yapanese Ice spice is womp womp goofy ahh Quandale dingle let him cook ishowspeed when the bro glazing grimace shake grimace shake Is not roblox am I only in ohio thinks he yap +1000 aura 🗿🍷 Gooning english or spanish