Add service discovery for IONOS Cloud

This change adds a new service discovery for IONOS Cloud based on the previous work for Hetzner and Scaleway. The new discovery allows users to create scrape targets based on servers deployed in a IONOS Cloud datacenter.
This is my first PR to this project and only read the contributor guideline, therefore I'm happy to be pointed to any resources I've missed and should have read next.
I've found the implementation (thanks to prior work) straight forward and impact on other things relative small, thus I didn't start a discussion on the mailing list.
Please be aware that this has only been deployed on non-production environments yet.
P.s. Thanks for the awesome work on Prometheus and Ecosystem.

@roidelapluie gentle ping. Did you have a chance to test the implementation?

I am waiting news from IONOS to enable my account

Thanks! I will have a deeper look on monday

Thanks for a first review and improvements. Looking forward to the next one :)

The labels still seem confusing to me:
ip addresses should not be in lists.
can one nic belong to multiple lans? it does not seem so.
I would:
Remove _meta_ionos_server_ip (it's equivalent to address)
Remove __meta_ionos_server_lan_id
Change "__meta_ionos_server_nic_ip_d09cabed_4fef_425c_8dce_a7f6c8fbe84e" from a list to a single addr.
Remove __meta_ionos_server_primary_ip
Remove __meta_ionos_server_primary_lan_id
If a server does not have a address, it should be dropped

can one nic belong to multiple lans? it does not seem so.

No, IONOS allows users to attach multiple NIC's to the same LAN.

If a server does not have a address, it should be dropped

Indeed, it makes sense to drop servers without a address.

Remove _meta_ionos_server_ip (it's equivalent to address)

The __address__ label is not the same as __meta_ionos_server_ip, but __meta_ionos_server_primary_ip + :port, where as __meta_ionos_server_ip is a list of all IP addresses assigned to a server. Therefore, I would prefer keeping the __meta_ionos_server_ip label and remove __meta_ionos_server_primary_ip.
What is the reason to change __meta_ionos_server_nic_ip_<nic_id> to a single value, I'm asking as IONOS allows users to assign multiple IPs to a single NIC?

I've seen that it's possible to provide NIC's with a name.
From a users perspective it makes sense to change __meta_ionos_server_nic_ip_<id> to __meta_ionos_server_nic_ip_<name>. Otherwise, operators which want to set the address` label to the IP address of a specific NIC would have to match UUIDs it's current ID.
The only problem is that the creation of a NIC doesn't require users to set a name for it, therefore leaving it empty by default. This could be resolved by either dropping the dynamic label for NICs with an empty label or defaulting to a NICs ID.
I think it would be fair to drop the dynamic label for NICs without a name, as the creator of the NIC would otherwise have given it a name to be able to make it easier to refer to it later on.

While adding a test case for a server with multiple NICs, I've noticed that the NICs do not maintain their order between server stop/start. This means that a server with multiple NICs is likely to change it's value for the __address__ label.
I'm not sure if this is an issue or not, but also haven't found a NIC attribute to sort on. Both deviceNumber and pciSlot will be reassigned when a NIC is deleted and a new one is created.
Additionaly, when a new NIC is added it will be prepended on the list of NICs.
I'm not sure if this is a problem and if it needs to be solved, but wanted to shared my findings.

Additionaly, when a new NIC is added it will be prepended on the list of NICs.

Then we probably should invert the list.

I've seen, I've misread your question regarding the possibility to attach multiple LANs to a single NIC. :facepalm:
It's not possible to attach a single NIC to multiple LAN's, but as above multiple NIC's to a single LAN.

To add to my comment above regarding NIC ordering.
Today, I've noticed that the Data Center Designer provides users the option to sort NICs by either Device number (default) or PCI slot. This ensures that the order of NICs remains the same across instance restarts and when a new NIC is added, as long as previously no NIC has been deleted as the device number would be reassigned.

With the current changes I've

added dropping of targets with no IP address
removed the __meta_ionos_primary_ip label
removed the __meta_ionos_server_lan_id and __meta_ionos_server_primary_lan_id labels
changed the dynamic label __meta_ionos_server_nic_ip_<nic_id> to __meta_ionos_server_nic_ip_<nic_name> by grouping NIC IPs with the same NIC name. NIC's without a name are grouped under unnamed.
inverted the list of NICs before labeling
added test cases for instances with multiple NICs

Thanks! I have reviewed this and tested with IONOS cloud, and it looks good to me!

Thanks for the review and help getting this landed. :)


marker directive handler memory leak fix

The mouse handlers need to be properly unregistered to not cause retention leaks. This change vastly improves the over time retention when removing and adding markers.

jshint warnings


fix: split on comma

What does this PR do?
The code was splitting on a space char when it needed to be a comma.
What issues does this PR fix or reference?
@W-12490382@
https://github.com/forcedotcom/cli/issues/1912

QA Notes

before:
 ➜  sfdx package version list  -p 0Ho3i000000GmjfCAC,0Ho3i000000GmjVCAS,0Ho3i000000GmjpCAC
Error (1): The Package Id: 0Ho3i000000GmjfCAC,0Ho3i000000GmjVCAS,0Ho3i000000GmjpCAC isn't defined in the sfdx-project.json. Add it to the packageDirectories section and add the alias to packageAliases with its 0Ho ID.

after:
 ➜  sfdx package version list  -p 0Ho3i000000GmjfCAC,0Ho3i000000GmjVCAS,0Ho3i000000GmjpCAC
=== Package Versions [6]

 Package Name Namespace Version Name Version Subscriber Package Version Id Alias Installation Key Released Validation Skipped Ancestor Ancestor Version Branch
 ──────────── ───────── ──────────── ─────── ───────────────────────────── ───── ──────────────── ──────── ────────────────── ──────── ──────────────── ──────
 orgInDep               ver 0.1      <IP_ADDRESS> 04t3i000002OUFTAA4                  false            false    false              N/A      N/A
 testWR                 ver 0.1      <IP_ADDRESS> 04t3i000002OUFnAAO                  false            false    false              N/A      N/A

❓:
 ➜  sfdx package version list  -p "0Ho3i000000GmjfCAC, 0Ho3i000000GmjVCAS, 0Ho3i000000GmjpCAC"
Error (1): The id [ 0Ho3i000000GmjVCAS] is invalid. It must start with "0Ho".

seems like we could fix in the flag validation to trim() then validate 🤷♂


Storybook for data (moved from https://github.com/swissgeol/ngm/pull/753)

This reverts commit feda1d22ddd99ed86eaeb977b46b3b45ce5365de.

@gberaudo storybook updates break CI. Can you please have a look?

@gberaudo storybook updates break CI. Can you please have a look?

I updated timeout


Fast Break

Fast Break is a game built on the NBA Top Shot contract. Players of Fast Break submit moments in their Top Shot collection to play the game. Each Fast Break contains n FastBreakStatistic the wallet must match or exceed.
When a player submits a lineup to Fast Break, a game token is minted to their Fast Break collection. This game token contains their submission data and links to the game. As NBA games are played, the metadata of the token (points & win) are updated.
A Fast Break Run is a block of Fast Breaks. Winning a Fast Break puts you on the Fast Break Run leaderboard.
The resource FastBreakDaemon is reserved for the game oracle. The oracle is responsible for bringing trusted NBA data into its platform, computing points & wins, and updating the blockchain with the derived data. The FastBreakDaemon keeps the on-chain & off-chain games in sync.
Custodial submissions to Fast Break will be made by the game oracle in batch. Non-custodial submissions can be made by any wallet with Top Shots.

Please correct me if I am wrong, but it looks like the actual NFTs that you are defining here are just meant to only be used in a single game and are not used any time afterwards? If that is the case, it doesn't really make sense to make them the NFTs and it would make a lot more sense to make the NFT represent the player instead of the game, so it can be used indefinitely across multiple different games, can be moved to a new address if the player wants to for whatever reason, can be traded or sold on marketplaces, and can contain meaningful metadata about the player' history.
In addition, you'll also need to have this contract implement ViewResolver and use metadata views for the NFTs and the contract

Yes, the intention is to create a game token that represents a single play of Fast Break. I chose this design because it maximizes the flexibility other contracts can have building on-top of Fast Break.
Having said that, I really like the idea of having a FastBreakPlayer resource or NFT. I think it adds a ton to the game. I started off creating it as a resource over here, but I am thinking it might be better as an NFT in its own contract. Would appreciate advice or collaboration on this PR on the FastBreakPlayer concept.
https://github.com/dapperlabs/nba-smart-contracts/pull/237

@joshuahannan, I want to make the Player an NFT in a new contract and started working on it in the PR mentioned in my comment. Having said that, I do think a game token adds more to the goals of the game, is more experimental, and more composable, and I am planning to stick with this concept. However, I do like the idea of a player NFT as well, but correct me if I am wrong, that would need to be in a different contract.
Either way, the core functionality I want in this contract is present in this PR. Have made all the suggestions Bastian made afaik. Thanks again for your time.
https://github.com/dapperlabs/nba-smart-contracts/pull/229#issuecomment-1871637570

oh sorry, I think I missed that last part of your comment last week. I disagree that you are able to implement the player as a resource in a different contract. You're already tracking players by address in this contract, so if you were to try to graft a resource from a separate contract onto this as a player, it wouldn't work IMO because it can never be reconciled with the address tracking that you're doing here. It is a fundamental enough part of the game that you would need to track players by only their resource IDs and remove address tracking completely. That is also a more Cadence-best practices way to do it.

Hey @joshuahannan, sorry for the delay as I am working on this over weekends. I implemented the Player resource and removed wallet data in favor of a resourceId. I like the concept a lot and feel like this really improved the contract. Thanks for pushing me to do this. Would appreciate your re-review.


Would it be possible to get the ability to add images?

It seems like the javascript is deleting my img tags. They don't even show up in the DOM.

Thanks for the suggestion -- yes, this is not currently supported. If you add  in the editor then it would get escaped to
<img src="foo.jpg">

in the HTML (and rendered as ).
Maybe it would be good if Markdown was supported as well as plain text. Would support for Markdown images be good enough for your use case? I was thinking of supporting only a small subset of Markdown (e.g. lists, links and images) for now, and this would be available by setting an attribute data-enable-markdown-based-syntax=true.
Unfortunately I can't really predict when I might be able to get this done, but I hope I will have time to work on this project again in the near future.

If you upgrade to v2.0.0 and switch to the Markdown-based syntax, then you should be able to use img tags now.
I haven't tested this extensively, if you do give it a try and run into any problems then please feel free to open another issue (or reopen this one).


Left align text on startup window buttons

Accepted

BButton is hardcoded to center align only. Would require a custom BButton subclass just for this. So not doing so at this time. (Effort vs. reward)


CLARITY CSIS Configuration

Adapt compose file and configuration for CSIS CLARITY setup.
See also https://github.com/clarity-h2020/csis/issues/1

done


[DO NOT MERGE] Refactor knn

Draft PR for discussion purposes.
KNN does not fit the traditional Spark ML Estimator + Model paradigm.  You don't really need to "train" it, and there really isn't any "model state" to be stored in the Model (other than k).  That said, we currently have an Estimator implementation that is "fit" on a DataFrame, which really just returns a Model with a reference to that DataFrame.  Then, the Model has APIs to compute nearest neighbors between that stored DataFrame to another DataFrame passed into the function arguments.  While this does work, it also leads to issues with saving the associated Estimator and Model instances, since the DataFrame reference is not serializable.
So, one option is to find a way to serialize a DataFrame reference cleanly.
This PR proposes another option which removes the Estimator/Model base classes entirely, and just represents KNN as a simple class with stateless APIs.  This is most similar to the classes in stats.py.
I also investigated basing on the Transformer class, but that API generally just "transforms" a single DataFrame at a time, so once again, we'd need to store a secondary DataFrame reference somewhere.
Anyhow, I've also staged the PR in multiple commits to simplify the review (with the first commit being the most important).  Please feel free to comment (and/or reject this entirely), since I'm not sure if we're losing anything important with this change.

Seems this PR has changed the signature of the kneighbors, so in that case, seems we don't need to store the item_df: DataFrame in KNearestNeighbors, which means we may directly return KNearestNeighborsModel in the _fit function of KNearestNeighbors. And this seems to conform to the spark estimator/model style. Please correct me.

Would it make sense to throw exceptions when save/load functions are called? Fit runs super fast (this is different from other algorithms), so it seems no need to store an estimator or model.

Yeah, after all the comments (thanks, everyone!), and reviewing the sklearn and cuml APIs a bit more, I think this is the best solution.  Basically, leave as-is and just disable save/load.


Add footer for copyright claim of the game

Prerequisites

[X] I am an Open Source Contributor
[X] I checked to make sure that this issue has not already been filed

Description
I want to add a footer to the site for the copyright claim
Screenshots

Code of Conduct

[X] I follow Contributing Guidelines of this project.

It's not needed @radhika141


feat(extensions): extensions enable/disable

What this PR does 📖

allows enable/disable extensions and save their status in persisted state

Which issue(s) this PR fixes 🔨

Resolve #

Special notes for reviewers 🗒
Additional comments 🎤

Just one small question, the state of toggle will not change after navigating away from Settings 👍  but typing indicator behavior is the same with toggle on or off. Tried new accounts and cargo update/clean
https://user-images.githubusercontent.com/93608357/206759865-1de716ae-1762-4a1a-ad6e-afc5305c8e5c.mov

Just one small question, the state of toggle will not change after navigating away from Settings +1 but typing indicator behavior is the same with toggle on or off. Tried new accounts and cargo update/clean

Thats because the the indicator is enabled by default while this PR makes it so you can enable and disable the extension. In this case, when you enable the extension a button pops up on the right hand side of the text area that allows you to enable or disable the typing indicator.

I will make a quick comment (since I forgot to do it when approving the PR) that we should probably default back to the default setting of say the indicator rather than for it to stick, but i assume that it would probably have its own configuration in the future that it would rely on.

Thanks for clarifying Darius!


add scripting with JS or Lua

Use eg https://bellard.org/quickjs/ or Lua to add scripting of app UI, Midi, patch editing etc

That would be cool, then I could add some skins. Designs inspired by famous retro synthies would be cool :)
And one could have a drum machine layout instead of keys or something, right?

Potentially yes, it's should be possible to have completely custom controls, though it will likely be a while yet before I have a chance to work on this feature.


Sample Scene Bot, CharacterConfig Updates

Updates the existing bots to use the new characterConfig type (rather than characterType). Also adds a new SampleScene bot that is used in the tutorial.

@abeizer back to you!


Mem ptr implementation

Hello!  Pretty useless for the most part but it does help pass almost all of the tests in Woody's MemPtr and Flag tests. I can't seem to get Bit n, (HL) to pass though no matter what. lol. I'll leave it to you to work that one out - if you care!

@ArjunNair oof, I missed PR, sorry! I'll review it and merge it on this week

@ArjunNair  Thanks for PR, I fixed formatting to pass CI and going to merge it


Flutter Video Player Plugin doesn't support m3u8 files on Android

The flutter videoplayer plugin doesn't play m3u8 video formats. But the underlying exoplayer2 for android which is used supports it.
Currently the plugin uses the ExtractorMediaSource of the exoplayer2 for all videos.
https://github.com/flutter/plugins/blob/master/packages/video_player/android/src/main/java/io/flutter/plugins/videoplayer/VideoPlayerPlugin.java#L87
But exoplayer2 specifies that you need use the HlsMediaSource for playing HLS /dash video files.
As given here in this example.
https://github.com/yusufcakmak/ExoPlayerSample
The videoplayer plugin should allow to pass a parameter which will decide if its a HLS video and use the HSLMediaSource so that m3u8 videos could be supported.
Steps to Reproduce
This is a flutter app that shows the error.
https://gist.github.com/pyros2097/b839209899482d63b22ac396d213916f
Logs
[+1916 ms] I/System.out(11421): Keeping screen on
[ +112 ms] I/ExoPlayerImpl(11421): Init 26ce7c7 [ExoPlayerLib/2.8.0] [generic_x86, Android SDK built for x86, Google, 27]
[  +16 ms] D/NetworkSecurityConfig(11421): No Network Security Config specified, using platform default
[+1259 ms] E/ExoPlayerImplInternal(11421): Source error.
[        ] E/ExoPlayerImplInternal(11421): com.google.android.exoplayer2.source.UnrecognizedInputFormatException: None of the available extractors (MatroskaExtractor, FragmentedMp4Extractor, Mp4Extractor, Mp3Extractor, AdtsExtractor, Ac3Extractor, TsExtractor, FlvExtractor, OggExtractor, PsExtractor, WavExtractor, AmrExtractor) could read the stream.
[        ] E/ExoPlayerImplInternal(11421):      at com.google.android.exoplayer2.source.ExtractorMediaPeriod$ExtractorHolder.selectExtractor(ExtractorMediaPeriod.java:931)
[        ] E/ExoPlayerImplInternal(11421):      at com.google.android.exoplayer2.source.ExtractorMediaPeriod$ExtractingLoadable.load(ExtractorMediaPeriod.java:853)
[        ] E/ExoPlayerImplInternal(11421):      at com.google.android.exoplayer2.upstream.Loader$LoadTask.run(Loader.java:317)
[        ] E/ExoPlayerImplInternal(11421):      at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1162)
[        ] E/ExoPlayerImplInternal(11421):      at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:636)
[        ] E/ExoPlayerImplInternal(11421):      at java.lang.Thread.run(Thread.java:764)

Analyzing limitless...
No issues found! (ran in 2.3s)

[!] Flutter (Channel beta, v0.4.4, on Linux, locale en_US.UTF-8)
    • Flutter version 0.4.4 at /home/pyros2097/Applications/flutter
    • Framework revision f9bb4289e9 (4 weeks ago), 2018-05-11 21:44:54 -0700
    • Engine revision 06afdfe54e
    • Dart version 2.0.0-dev.54.0.flutter-46ab040e58
    ✗ Downloaded executables cannot execute on host.
      See https://github.com/flutter/flutter/issues/6207 for more information
      On Debian/Ubuntu/Mint: sudo apt-get install lib32stdc++6
      On Fedora: dnf install libstdc++.i686
      On Arch: pacman -S lib32-libstdc++5

[!] Android toolchain - develop for Android devices (Android SDK 27.0.3)
    • Android SDK at /home/pyros2097/Android/Sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-27, build-tools 27.0.3
    • ANDROID_HOME = /home/pyros2097/Android/Sdk
    • Java binary at: /home/pyros2097/Android/android-studio/jre/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b01)
    ! Some Android licenses not accepted.  To resolve this, run: flutter doctor --android-licenses

[✓] Android Studio (version 3.1)
    • Android Studio at /home/pyros2097/Android/android-studio
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b01)

[✓] Connected devices (1 available)
    • Android SDK built for x86 • emulator-5554 • android-x86 • Android 8.1.0 (API 27) (emulator)

! Doctor found issues in 2 categories.

Corresponding PR here.
https://github.com/flutter/plugins/pull/607

Closed the PR 607 and improved on it to support more video formats not just .m3u8
Here is the current PR for tracking.
https://github.com/flutter/plugins/pull/613

https://github.com/flutter/plugins/pull/613
PR Merged. Closing now.


CI: output log of each test target

There seems only one target tensorflow_addons/ubuntu/py2/presubmit in the detail page, check out:
https://source.cloud.google.com/results/invocations/bcda13af-4a13-41c6-a7db-76ff8e3ec504/targets/tensorflow_addons%2Fubuntu%2Fpy2%2Fpresubmit/log
@av8ramit
Could we give all the details for each test target, like tensorflow/tensorflow? That would be greatly helpful for review.

Hi @facaiy unfortunately for reasons decided unbeknownst to me, the way the system internally is set up is that you can only run predefined bazel targets if you want the targets to be split up. I think this is for security reasons. Since the script is modified a lot externally, I've made the internal invocation a simple shell script call. We're discussing on what the best way to move forward is.

@av8ramit Thank you for the detailed reply, Amit. I start to understand why we have only one presubmit target here. It has been good enough for me. Thanks again, we wouldn't have gotten here without your work. Please update the issue when new information becomes available :-)

No problem @facaiy glad I was able to help!


Add resource param to signinSilent request

Closes/fixes #920 and probably #655
Checklist

[X] This PR makes changes to the public API
[X] I have included links for closing relevant issue numbers

thanks for contributing


bootlanderCreate FUNDING.yml

Thanks for your PR!
We are in the process of archiving this repo and are no longer accepting new PRs. Please see https://github.com/kubernetes/org/issues/1563 for more details. Thanks again!


Make a first time setup script that generates the spreadsheets + worksheet names and rows

Merge it with generate_keys.py

Maybe also create the channels and roles stated in config.yaml

Use: await guild.create_text_channel('cool-channel') for creating new text-channel
And await guild.create_role(name="role name") for new role

Also renames the config_example.yaml to just config.yaml


Please provide PowerShell examples

We received this message yesterday and we are very much concerned.
https://techcommunity.microsoft.com/t5/Microsoft-SharePoint-Blog/Delivering-SharePoint-modern-experiences/ba-p/315023
How to disable it through PowerShell for SharePoint Online sites?
I found this article; https://techcommunity.microsoft.com/t5/SharePoint-Support-Blog/How-to-disable-the-modern-experience-in-SharePoint-2019/ba-p/303649 it seems it is applicable for SharePoint 2019, not Online versions.
and we have several sites that has both classic list and modern list. What happens if we disable at site collection level? Does it change the existing modern experience lists to classic or ignores it?
Expected behavior is it should ignore because we have explicitly defined as Modern experience on those lists/libraries.

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

ID: 4f1bd4ab-c640-1db8-05bb-e249129dfa46
Version Independent ID: 4ceb7bc8-39c0-61fb-b31c-381cb7a5018b
Content: Change the default list and library experience
Content Source: SharePoint/SharePointOnline/change-default-list-and-library-experience.md
Service: sharepoint-online
GitHub Login: @kaarins
Microsoft Alias: kaarins

Hello, please see this page for information on how to opt out a site collection or site use PowerShell:
Opting out of the modern list and library experience
If you enable this feature to opt out a site collection, and then you enable modern on a list or library inside that site collection, then that list or library will use modern mode.  The lowest-level setting wins.

Thank you @ldemaris . Please include that information in the document. "Lowest-level setting wins". You shall put it under FAQs too.

@ldemaris And I tried this PnP example at site collection level. Nothing happens.
Here are the details;

Site collection is Classic team site
Few list are modern and few are classic
Executed both Disable and Enable scripts

Nothing happens. No change in the list and libraries.
I have created a new custom list after Disable and Enable.
It is still creating a Classic custom list.
Am I missing anything?

The PowerShell feature can not be used for enabling modern in a site collection.  It can only be used for disabling modern.  Turning the feature on in a site collection will ensure that every list inside that site collection stays classic, unless that list specifies a modern setting.

@ldemaris I'm confused here. I see this below line in the documentation
And again enable modern lists and libraries at site collection level
#Disable-PnPFeature -Identity E3540C7D-6BEA-403C-A224-1A12EAFEE4C4 -Scope Site
But you are saying that this command will not work !!!!!
Please clarify.

Thank you for the feedback, the wording in this article could definitely be improved.  Enabling the feature will cause that site collection's lists to stay in classic, unless those lists are configured to use modern.  Disabling the feature will not force all libraries into modern; it will cause those libraries instead to defer to the tenant-wide setting.

Thanks @ldemaris
So, if I try it on or after April 1, 2019, we could see the differences. Am I right?
Because according to this announcement, tenant's opt out setting will be switched to Modern.
(Currently my tenant setting is "Classic")

@kaarins Any updates on this? Thanks!

Updated article with the removal of the organization-level setting.


Issue with batch_ell matrix when solving with non zero per line > 1

Hello,
I am actually using Ginkgo in order to solve a batch of linear systems. I have some issues on the execution of the solver (batch BiCgStab ).
In fact the building of the whole structure is Ok. If i take a diagonal matrix(ie nonzero per raw =1 )  the results are good.
If i take  number of non zeros components per line > 1, the obtained result is false..
My question is more about the storage format. What i understood from test files for batch ell,  is that  it loops over lines and stores columns index. Is this assumption ok ? In the other hand, i have read in the code that it uses a column major storage (which is supposed to store line indexes )
Thank you for help

I think you might be misunderstanding the ELL format slightly. All sparse matrix formats in Ginkgo are row-based (i.e. they store all column indices for each sparse row), but the ELL format itself internally arranges these entries in a column-major order for better vectorization. To be fair, row-major and column-major are less useful concepts in sparse matrices than they are in dense matrices, which is where some of the confusion might be coming from.
If you want to assign the column index or value to the nz th nonzero of row row, you need to use the storage location nz * stride + row where stride usually defaults to num_rows.

Thank you for the answer,
For the stride i took the number of non zero per line instead of the matrix size eg:
ell_index= row*non_zero_per_row+ j   (it is usual when you want to "linearize" 2d array )
where j refers to the j-th non zero compoenent of the line.

Thanks a lot for your advices, it is more clear and it works for me !

That's great to hear, good luck with your experiments ;)


[0.8.x] more ci related backports

Same as #1289
Test run: https://github.com/buildpacks-community/kpack/actions/runs/5661303030

Codecov Report

Merging #1290 (c83542f) into release/v0.8.x (b4089a3) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@               Coverage Diff               @@
##           release/v0.8.x    #1290   +/-   ##
===============================================
  Coverage           67.76%   67.76%
===============================================
  Files                 123      123
  Lines                7280     7280
===============================================
  Hits                 4933     4933
  Misses               1937     1937
  Partials              410      410


[Build process] Error fetching deb

We are seeing a lot of apt related failures when building Docker, especially on CI:
07:19:41 E: Failed to fetch http://httpredir.debian.org/debian/pool/main/p/python3-defaults/python3-minimal_3.4.2-2_amd64.deb  Error reading from server. Remote end closed connection

I'm opening this to explore alternatives.

One unsatisfactory earlier attempt was to move all 3rd party dependencies to a base image (#20665).
Should we try some of the other solutions described here? They will surely be inferior in some aspects, but maybe we should optimize for reliability here.

If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
For more information about reporting issues, see CONTRIBUTING.md.
You don't have to include this information if this is a feature request
(This is an automated, informational response)

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

I'm not sure it is possible (couldn't find it with a quick search), but if we could have apt-get fail over to a backup mirror (if the one from httpredir fails) that might be slower, but more reliable then that seems like the better approach.
Another idea is to install our own apt-cacher, so that we can can cache the packages we use close to us, and limit the impact of a bad mirror. see https://help.ubuntu.com/community/Apt-Cacher-Server for more details.

https://docs.docker.com/engine/examples/apt-cacher-ng/

yeah so its an apt problem, we could set a different mirror or like ken and @thaJeztah said

I'm thinking maybe we can use a --build-arg to override the Debian mirror at build-time, so that:

Our contributors around the world could continue benefiting from geo-aware httpredir as the default.
Our CI servers could use a specific mirror depending on their location (which should be pretty much all US, except ARM in France).

What do you think?

Seems like an OK solution here.
For the general issue, I'm thinking it might be time to switch the Debian base images off httpredir since it appears to be in some sad period of mourning (this is the third independent report I've heard of issues like this, which is more serious than the usual Hash Sum Mismatch).  I've got high hopes for http://deb.debian.org, but it's currently stretch-only, and isn't supported by debootstrap (yet), so we'd have to hard-code cdn-fastly.deb.debian.org for now, and I'm not sure how well-supported that'll stay without an official announcement that it ought to be used on debian-devel-announce. :disappointed:


Add new version of updateBricks which takes a custom DiffUtil.Callback

Fixes https://github.com/wayfair/brickkit-android/issues/195

Codecov Report

Merging #196 (03a1714) into dev (d04a912) will increase coverage by 0.02%.
The diff coverage is 85.71%.

@@             Coverage Diff              @@
##                dev     #196      +/-   ##
============================================
+ Coverage     87.95%   87.97%   +0.02%
- Complexity      311      312       +1
============================================
  Files            32       32
  Lines           888      890       +2
  Branches        133      133
============================================
+ Hits            781      783       +2
  Misses           55       55
  Partials         52       52

Impacted Files
Coverage Δ
Complexity Δ

...in/java/com/wayfair/brickkit/BrickDataManager.java
84.84% <85.71%> (+0.06%)
132.00 <0.00> (+1.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 d04a912...03a1714. Read the comment docs.

Codecov Report

Merging #196 (03a1714) into dev (d04a912) will increase coverage by 0.02%.
The diff coverage is 85.71%.

@@             Coverage Diff              @@
##                dev     #196      +/-   ##
============================================
+ Coverage     87.95%   87.97%   +0.02%
- Complexity      311      312       +1
============================================
  Files            32       32
  Lines           888      890       +2
  Branches        133      133
============================================
+ Hits            781      783       +2
  Misses           55       55
  Partials         52       52

Impacted Files
Coverage Δ
Complexity Δ

...in/java/com/wayfair/brickkit/BrickDataManager.java
84.84% <85.71%> (+0.06%)
132.00 <0.00> (+1.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 d04a912...03a1714. Read the comment docs.


Extension causes high cpu load

Issue Type: Performance
Extension Name: tabnine-vscode
Extension Version: 3.2.17
OS Version: Windows_NT x64 10.0.18363
VSCode version: 1.45.1

:warning: Make sure to attach this file from your home-directory:
:warning:C:\Users\asus\TabNine.tabnine-vscode-unresponsive.cpuprofile.txt
Find more details here: https://github.com/microsoft/vscode/wiki/Explain-extension-causes-high-cpu-load

Hi @Cessiia , where did you install the extension from? can you please update to the latest version (3.4.3)?

你说的是什么版本
What version are you talking about？

Hi @Cessiia , where did you install the extension from? can you please update to the latest version (3.4.3)?

I downloaded it from the vscode extension store and I'll try it out。

I have recently installed TabNine and faced the same issue today while coding an app. My laptop's cooling fans were going crazy. When I checked the processes in the Windows 10 Task Manager, I found TabNine processes utilizing around 27% of the CPU.
This is a case of very high CPU usage.
Please find the screenshot below:

I have downloaded the extension directly from the VS Code extension marketplace.


Crash in slice::from_raw_parts

No repro yet
thread 'rayon-10' panicked at 'unsafe precondition(s) violated: slice::from_raw_parts requires the pointer to be aligned and non-null, and the total size of the slice not to exceed `isize::MAX`'
core/src/panicking.rs:219
stack backtrace:
   1: re_crash_handler::callstack_from
             at re_crash_handler/src/lib.rs:223:21
   2: re_crash_handler::install_panic_hook::{{closure}}
             at re_crash_handler/src/lib.rs:30:25
   3: <alloc::boxed::Box<F,A> as core::ops::function::Fn<Args>>::call
             at alloc/src/boxed.rs:2077:9
      std::panicking::rust_panic_with_hook
             at std/src/panicking.rs:799:13
   4: std::panicking::begin_panic_handler::{{closure}}
             at std/src/panicking.rs:656:13
   5: std::sys_common::backtrace::__rust_end_short_backtrace
             at std/src/sys_common/backtrace.rs:171:18
   6: rust_begin_unwind
             at std/src/panicking.rs:652:5
   7: core::panicking::panic_nounwind_fmt::runtime
             at core/src/panicking.rs:110:18
      core::panicking::panic_nounwind_fmt
             at core/src/panicking.rs:120:5
   8: core::panicking::panic_nounwind
             at core/src/panicking.rs:219:5
   9: re_renderer::allocator::uniform_buffer_fill::create_and_fill_uniform_buffer_batch
  10: re_renderer::renderer::point_cloud::PointCloudDrawData::new
             at re_renderer/src/renderer/point_cloud.rs:271:43
  11: <re_space_view_spatial::visualizers::points3d::Points3DVisualizer as re_viewer_context::space_view::visualizer_system::VisualizerSystem>::execute
             at re_space_view_spatial/src/visualizers/points3d.rs:265:13
  12: re_viewport::system_execution::run_space_view_systems::{{closure}}
             at re_viewport/src/system_execution.rs:44:19

Cause: https://github.com/purpleprotocol/mimalloc_rust/issues/128

We've worked around this on main, and have also update mimalloc


plt.xlim() doesn't work in dependence_plot() when plotting interaction values

Hi,
First of all, I want to say this is a great library for model interpretability. It can help add more insights into the black-box model, especially for business. Thanks for all the work.
The issue I'd like to raise here is about the dependence_plot().
If I want to plot the interaction values with dependence_plot:
shap.dependence_plot(
    ("Age", "Age"),
    shap_interaction_values, X.iloc[:2000,:],
    display_features=X_display.iloc[:2000,:],
    xmin = 0,
    xmax = 40
)

xmin and xmax won't work here. I think it is because xmin and xmax are not passed into the recursive function here. (https://github.com/slundberg/shap/blob/aff7ee81f33899f0d22972f1e8afff98f0ee7ef3/shap/plots/dependence.py#L91-L112)
There is a workaround:
shap.dependence_plot(
    ("Age", "Age"),
    shap_interaction_values, X.iloc[:2000,:],
    display_features=X_display.iloc[:2000,:],
    show=False
)
plt.xlim(xmin=0,xmax=40)
plt.show()

Hello! Unsure if it's because I'm using shap v. 0.35.0, but I am finding that ymin similarly does not work on dependence_plot().
I'm not smart enough to investigate if the error is similar to that identified for xmin, but at the very least, the workaround for xmin above seems to work for ymin in my case.
fyi in case not yet addressed in the latest version. :) Brilliant, brilliant idea, this SHAP thing. :) thanks so much!


Deprecating TLS 1.0 and 1.1 on NuGet.org - Stage 2

As announced in the blog post here, it's time to permanently remove support for TLS 1.0, 1.1.
Exact date in April is TBD and will be announced several weeks in advance.

Resources by environment:
DEV

[ ] Gallery

[ ] Cloud service
[ ] App service

[ ] Search
[ ] APIM
[ ] CDN

[ ] apidev.nugettest.org
[ ] globalcdn.dev.nugettest.org

[ ] Symbol server
[ ] Licenses app service
[ ] Status app service

INT

[ ] Gallery

[ ] Cloud service
[ ] App service

[ ] Search
[ ] APIM
[ ] CDN

[ ] apiint.nugettest.org
[ ] globalcdn.int.nugettest.org

[ ] Symbol server
[ ] Licenses app service
[ ] Status app service

PROD

[ ] Gallery

[ ] Cloud service
[ ] App service

[ ] Search
[ ] APIM
[ ] CDN

[ ] api.nuget.org
[ ] globalcdn.nuget.org

[ ] Symbol server
[ ] Licenses app service
[ ] Status app service

Has the support for TLS 1.0, 1.1 been removed today?
Maybe this is unrelated but since this morning we have troubles with Install-PackageProvider nuget. The changes as described in the post seems to work.

Production environment support was not removed. Qualys' SSL test still reports it as enabled: www.nuget.org, api.nuget.org.
DEV and INT environments however are in progress of disabling TSL 1.0 and 1.1 currently. Are you using those by any chance?

Production environment support was not removed. Qualys' SSL test still reports it as enabled: www.nuget.org, api.nuget.org.
DEV and INT environments however are in progress of disabling TSL 1.0 and 1.1 currently. Are you using those by any chance?
What exactly you were trying to do and what errors did you get?

How can i check if im using the DEV or INT environments. I guess i'm not but i want to be sure.
I try to run the powershell command Install-PackageProvider -Name NuGet but i receive this warning: Unable to download the list of available providers.  The command was working before friday.
Just like this thread: https://answers.microsoft.com/en-us/windows/forum/windows_7-performance/trying-to-install-program-using-powershell-and/4c3ac2b2-ebd4-4b2a-a673-e283827da143

@jorismathijssen , from the thread here https://answers.microsoft.com/en-us/windows/forum/windows_7-performance/trying-to-install-program-using-powershell-and/4c3ac2b2-ebd4-4b2a-a673-e283827da143 I see that the error you are getting is from https://onegetcdn.azureedge.net/. This domain doesn't belong to nuget.org. TLS 1.0 must have been deprecated by them as well.

@jorismathijssen , from the thread here https://answers.microsoft.com/en-us/windows/forum/windows_7-performance/trying-to-install-program-using-powershell-and/4c3ac2b2-ebd4-4b2a-a673-e283827da143 I see that the error you are getting is from https://onegetcdn.azureedge.net/. This domain doesn't belong to nuget.org. TLS 1.0 must have been deprecated by them as well.

 You are definitely right. Thank you for the information. It’s good to know that the upgrade works perfect.
Thank you!


Handel Codepipeline --> Git Hub Actions

This PR removes Handel Codepipeline and replaces the functionality with GitHub Actions. To test the process, a new environment (gha) was setup and updated successfully.  The "gha" environment variables are left remaining everywhere but the github action. This is so we will be able to standup another environment as a test or experiment if needed.

I removed references to the 'gha' environment but added a note in the ReadMe about how to add an environment option.


Hide all toolbars when taking screenshots

Are you requesting a feature or reporting a bug?
feature request
What is the current behavior?
all toolbars are displayed when a screenshot is taken
What is the expected behavior?
remove all toolbars (run chrome from application mode)
Specify your

operating system: OS-X 10.11.6
testcafe version: 0.14.0
node.js version: 7.9.0

Hi,
Thanks for the report, we should think out it thoroughly. If we will create screenshots without browser toolbars it can be a breaking change for someone.
As a workaround you can run chrome in application mode via browser arguments:
testcafe "chrome --kiosk" test.js
(quit the running chrome before try it)

The problem with application mode is that debugging is difficult (if not impossible). I don't believe there is a way to open dev tools or toggle application mode if a breakpoint is added.
This issue seems related to https://github.com/DevExpress/testcafe/issues/1357.
The use case for me is for visual regression testing. Since the address is different for each test, there will always be a regression. Perhaps a flag that removes the testcafe UI bar and does not include the address bar on screenshots...

Also, curious if you know of anyone that has added on a visual regression feature to testcafe? Looking at using https://github.com/mapbox/pixelmatch to compare the images. Writing a few scripts now to incorporate into the testing workflow.

@mjhea0 We have #1207. We even had had a long in person discussion about original proposal and decided to drastically change proposed approach afterwards. Discussion should have been documented by @kirovboris. However, unfortunately I can't find any notes 😞 . You can add 👍 to the #1207 so we'll know that there is a demand for the feature.


Memory Leak

I have a view with 3000 editable-textboxes inside a ng-repeat. When I replace the model of the ng-repeat alternately with two arrays of the same lenght, memory consumpion is increasing and after about 20 iterations chrome crashes.
It  seems that some event handlers are not unbound on $destroy.

+1
The simplest example shows a memory leak:
http://embed.plnkr.co/WE7XFvtMhrRzPtpxqcc0/preview

Is this issue fixed now that #489 has been merged?

Yes, works for me.


management: Run log_email_config_errors in send_test_email command.

Follow-up to @alexmv comment https://github.com/zulip/zulip/pull/21053#discussion_r802137646
Regarding:

Seeing that code makes me also wonder if WARN_NO_EMAIL should get set in this case, or should be extended to not just be a bool to cover this case.

Hmm, I think that might be a bit confusing because that variable gives "email sending is not configured at all" kind of warnings - which is not the case in the "config error" scenario and might make the admin think that none of their setting changes are actually being applied?

Follow-up to @alexmv comment #21053 (comment)
Regarding:

Seeing that code makes me also wonder if WARN_NO_EMAIL should get set in this case, or should be extended to not just be a bool to cover this case.

Hmm, I think that might be a bit confusing because that variable gives "email sending is not configured at all" kind of warnings - which is not the case in the "config error" scenario and might make the admin think that none of their setting changes are actually being applied?

Sorry, all I was trying to say was that we doul extend WARN_NO_EMAIL to be more general, and use the same style of system to show various configuration problems -- not just "not configured," but add the "misconfigured" message that this was addressing.
I think the log message is fine for now, particularly with it in send_test_email.


proposal: x/crypto/argon2: add password hashing/verification wrapper

I propose that we extend x/crypto/argon2 to include a wrapper for password hashing and verification, similar to what currently exists in the x/crypto/bcrypt package. This would provide convenient methods for developers to generate hashed passwords and compare them, using Argon2's state-of-the-art password hashing scheme.
Background
x/crypto/bcrypt package provides two crucial functions: GenerateFromPassword and CompareHashAndPassword. These make handling passwords quite straightforward for developers while still ensuring a high level of security.
However, as Argon2 is the reigning winner of the Password Hashing Competition, it would be beneficial for developers to have an equally convenient and familiar way of handling passwords using Argon2 within the Go standard library. The x/crypto/argon2 package provides an interface to the Argon2 functionality, but it lacks the same developer-friendly methods for password hashing and verification.
Additionally, NIST 800-63B recommends using a secret value of at least 112 bits. Using secret is implemented in deriveKey, but the Key and IDKey functions pass a nil value when calling it.
Current Usage
There is currently a well-written wrapper implementation with https://github.com/alexedwards/argon2id (329 ⭐) that is used in ~370 .go files across public Github. However, it doesn't support using a NIST-recommended secret value, and given that it exists outside of the core Go packages, its long-term maintainability cannot be guaranteed.
bcrypt.GenerateFromPassword is used across ~18.8k files, and bcrypt.CompareHashAndPassword is used across ~16.9k files. Of these, I suspect there is a fair number of developers who would rather be using Argon2, but they instead opted for bcrypt simply because the wrappers didn't exist or because they didn't want to rely on a third-party library for cryptography.
We're currently using a custom fork of x/crypto/argon2 at my job to accomplish these goals, but I would much rather be using something from the core packages.
Proposal
I propose adding three exported functions to the x/crypto/argon2 package:
// IDKeyWithSecret derives a key from the password, salt, secret (also known as "pepper"),
// and cost parameters using Argon2id returning a byte slice of length keyLen that can be used as
// cryptographic key. The CPU cost and parallelism degree must be greater than
// zero.
func IDKeyWithSecret(password, salt, secret []byte, time, memory uint32, threads uint8, keyLen uint32) []byte {}

// GenerateFromPassword returns the hash of the password using Argon2id with the given optional secret
// and parameters for `keyLength` (length of the generated key), `time` (number of iterations over memory),
// `memory` (in kibibytes), and `threads`. If 0 is passed for keyLength, time, memory, or
// threads, they will be replaced by default values keyLength=128, time=3, memory=65536, threads=4. A random salt
// of length 16 bytes will be generated.
//
// Example usage:
// hashedPassword := argon2.GenerateFromPassword([]byte("mypassword"), nil, 3,  8*256, 2, 0)
// hashedPasswordSec := argon2.GenerateFromPassword([]byte("mypassword"), []byte("optionalsecretpepper"), 3,  8*256, 2, 0)
func GenerateFromPassword(password, secret []byte, time, memory uint32, threads uint8, keyLength uint32) ([]byte, error) {}

// CompareHashAndPassword compares an Argon2 hashed password and an optional secret with a possible
// plaintext equivalent. Returns nil on success, or an error on failure.
func CompareHashAndPassword(hashedPassword, plaintext, secret []byte) error {}

Benefits

Increases usability and developer-friendliness of the x/crypto/argon2 package
Encourages the use of a more modern and secure password-hashing algorithm
Offers a standardized way to handle Argon2 hashed passwords in Go, reducing the risk of implementation errors

[cc: @FiloSottile, we chatted about this at a high level at GothamGo on 2023-06-09]

CC @golang/security

High-level wrappers are already approved in #16971, but we didn't discuss adding support for the secret input.


Personal goals

Only u

Change project selection to autocomplete, where you could select a user instead of a project
Add a boolean field to project that shows that it's personal
Disable and pre-fill assignee field
Hide personal projects from search, dashboard, etc
Limit project visibility to current user and project owner
E2E


Support refresh=wait_for

See https://www.elastic.co/de/blog/refreshing_news.
Should also deprecate Refresh(bool).

Done by the first steps on release-branch.v5.


Fix typo in cuisine

Hi Sean,
I love your project, and I just noticed this typo. I think this fixes a mistake.

Thanks!

Cool! Thank you!


Add self liked feed

@t1maccapp thank you very much for pull request more welcome, but I need to reject this.
Feed needs to be under client/v1/feeds directory and should be done same way as other feeds!
But don't worry
https://github.com/huttarichard/instagram-private-api/commit/e22965cb0edaae76a2da2d4caddce8425f908ecf :)

@huttarichard Hooray, I waited for working solution for node.js since 1 Jun. Thank you so much for your work!


Save image info in json

I think the output json should completely specify the LW++ view, which means it needs to know what the image being worked on is. Can we include the image filename and/or URL in the json, so that the whole thing is loaded in?

Agree, including the image file name and/or URL should be necessary. I will work on it asap.

Thanks! :-) I guess also the json filename could be related to the image
name, too - if I upload he0435.png, the json I make should be he0435.json.
I'm not sure if the json's should be named uniquely (to allow tracking of
model evolution) but that would be a separate issue anyway :-)
On Sun, Sep 11, 2016 at 8:50 PM, Nan Li<EMAIL_ADDRESS>wrote:

Agree, including the image file name and/or URL should be necessary. I
will work on it asap.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/linan7788626/LensWranglerPlusPlus/issues/2#issuecomment-246237571,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AArY99dBwzTrneSIsiPrW8a2Ba5bmJSRks5qpMv7gaJpZM4J5PmQ
.

http://lenszoo.files.wordpress.com/2013/12/asw0009cjs-zoomed.jpg
Hi Phil,
I tried to load the image above and save the URL and NAME of the image in the JSON file, it seems working. You can upload the URL in the textbox, construct your own model, then press the button "save the model" to export a JSON file named "($imageName).JSON".  I also attached an example of the output JSON file below, please let me know if you have any questions or suggestions.
Thanks
Nan Li

{
"name": "asw0009cjs-zoomed",
"src": "http://lenszoo.files.wordpress.com/2013/12/asw0009cjs-zoomed.jpg",
"components": [
{
"plane": "source",
"size": 1.183,
"ell": 2,
"ang": 22.135,
"x": 0.068,
"y": 0.405
},
{
"plane": "lens",
"theta_e": 2.077,
"ell": 2,
"ang": 31.842,
"x": -0.54,
"y": 0.811
}
]
}

Looks good to me! I can't test it yet though, the example must be using code from a different branch.

I just pushed the newest version to the web page of the example. I believe you can test it now.
Thanks.
http://linan7788626.github.io/pages/LensWranglerPlusPlus/index.html

Excellent! I too can make a nice JSON file :-)


Metrics-Endpoint returns 404

The alertmanager's metrics-endpoint /metrics exposed via -web.listen-address returns a 404-error on http-request.

Metrics were added after 0.1.0-beta1. What revision are you using.

The most recent pre-release. Building from git fixes this, you say, @fabxc?
alertmanager, version 0.1.0-beta1 (branch: master, revision: 26cfefd)
build date:       20160108-19:32:30

Works from HEAD, nevermind :)


Add description for all Stripe GraphQL Fields

Stripe has excellent description for all their fields in their API. We should copy all those descriptions and add them to the Stripe GraphQL API.
Here's an example:
Stripe:
https://github.com/stripe/stripe-node/blob/master/types/2022-08-01/Customers.d.ts#L12
GraphQL API:
https://github.com/nhost/nhost/blob/main/packages/stripe-graphql-js/src/schema/customer.ts#L13
That code should be updated to:
id: t.exposeString('id', {
  description: `Unique identifier for the object.`
}),

This is to be repeated for all fields in the Stripe GraphQL API.

hello, could I work on this issue?

Hello @piromsurang ,
Let us know if you need further details to help you in your PR!

could you please correct me if I am wrong. Basically, this task is to add all description in schema/ folder right?

That's correct. I recommend starting with one file in the schema/ folder and creating a PR so we can ensure you're on the right path. Happy hacking! 💪

@piromsurang That's perfect!

I'm re-opening this issue as there are more fields to add descriptions to.

I will continue and raise a new PR.

Please help review.


can I use rasa_nlu in non english languages?

hi.
this is a very good bot engine.
can I use rasa_nlu in non english languages?
for example can I use it to handle persian texts?

http://rasa-nlu.readthedocs.io/en/stable/languages.html

thank you a lot.


#240 added my YouTube and Spotify Playlists

#240 I have added my personal favourite playlist of Spotify and Youtube...
Thank u  for letting me do that:)

Thanks for contributing :)


Expose network bandwidth comsumption in metrics.

Mesos masters and slaves hold the remaining amount of network bandwidth
resource. We want to expose it in the exporter.

@aboten


Embedded struct not created in table

I am trying to create a schema with an embedded struct in a table. However when the table gets created there is no reference to the embedded struct and as such any query against it fails.
Code Extract:
type User struct {
	Id     int64
	Email  string
	// ...
}

type AccessToken struct {
	Id         int64
	User       User
	Token      string  `sql:"size:64;unique"`
}

func main() {
	// ... connection setup ..

	db.CreateTable(User{})
	db.CreateTable(AccessToken{})

	user := User{}
	db.Where("email = ?", "test@example.com").First(&user)

	token := AccessToken{
		User: user,
		Token: "access token",
	}

	db.Debug().Create(&token)
}

Table as created by db.CreateTable:
                                    Table "public.access_tokens"
   Column   |           Type           |                         Modifiers
------------+--------------------------+------------------------------------------------------------
 id         | bigint                   | not null default nextval('access_tokens_id_seq'::regclass)
 token      | character varying(64)    |
 created_at | timestamp with time zone |
Indexes:
    "access_tokens_pkey" PRIMARY KEY, btree (id)
    "access_tokens_token_key" UNIQUE CONSTRAINT, btree (token)

Database: PostgreSQL 9.3
Environment: Mac OS X 10.9.3
Go Version: 1.3
Gorm Revision: 90106d82c51d1befbb998144864652968db4df3c

Is it possible to automatically create a one-to-many relationship automatically if the foreign key is not called Id, and it isn't an int64?


multiprocessing failing on spyder when function contains a class on the same file

Problem Description
The following code executes properly from the command line but not within spyder. The program hangs and the command window show the  traceback pasted below. Placing the class code (everything before if __name__ == "__main__") in a separate file and importing the class solves the problem - but it is not clear why this code does not work as it does when executing from the command line.
What steps reproduce the problem?
#!/usr/bin/env python3
# -*- coding: utf-8 -*-
from multiprocessing import Pool
class myClass():

    def __init__(self,q_seed=0,*args,**kwargs,) :
        self.q_seed = q_seed
        print(self.q_seed)

def simulateLocPool(kwargs) :

    m = myClass(**kwargs)
    return m

#%%
if __name__ == "__main__":

    nboots = 10
    nprocs = 10

    # construct list to pass to processes
    iterlist = []
    kwargs = {'q_seed' : 0}

    # change seeds
    for bootn in range(nboots):
        newkw = kwargs.copy()
        newkw['q_seed'] = kwargs['q_seed'] + bootn
        iterlist.append((newkw))

    #spawn nboots processes, nprocs at once
    pool = Pool(processes=nprocs)
    allmodels = pool.map(simulateLocPool,(iterlist))
    pool.close()
    pool.join()

Output
The code hangs with the following traceback

Exception in thread Thread-23:
Traceback (most recent call last):
  File "/Users/moroa/opt/miniconda3/lib/python3.9/threading.py", line 954, in _bootstrap_inner
    self.run()
  File "/Users/moroa/opt/miniconda3/lib/python3.9/threading.py", line 892, in run
    self._target(*self._args, **self._kwargs)
  File "/Users/moroa/opt/miniconda3/lib/python3.9/multiprocessing/pool.py", line 576, in _handle_results
    task = get()
  File "/Users/moroa/opt/miniconda3/lib/python3.9/multiprocessing/connection.py", line 256, in recv
    return _ForkingPickler.loads(buf.getbuffer())
AttributeError: Can't get attribute 'myClass' on <module 'spyder_kernels.console.__main__' from '/Users/moroa/opt/miniconda3/lib/python3.9/site-packages/spyder_kernels/console/__main__.py'>

Expected output
A printout of digits from 0 to 9
Versions

Spyder 5.1.5|Python 3.9.5 64-bit | Qt 5.12.10 | PyQt5 5.12.3 | Darwin 20.6.0
Operating System name/version: MacOS 11.5.2

Hi @andreamoro-git,
Thanks for opening this issue, I could reproduce the problem with the snippet you provided. @jitseniesen, @ccordoba12 do you know why this is happening?

fwiw, I have always directed people towards using "run in external console" when using multiprocessing due to a number of issues. Chiefly among them is that stdout is not always correctly handled (no child print statements in ipython console). The way to debug things like this is then to start spyder like so: python -m spyder.app.start, and all child output will go to the terminal, while main process output goes to the ipython console.
Doing this shows that the code is being run in the child, so __main__ is clearly importable in the child, and confirms what the error message suggests (go figure....) that the error occurs when un-pickling themyClass instances sent back from the children. I also took a look at sys.modules in various scenarios, and while I don't necessarily understand the difference... __main__ shows up in two different ways:
'__main__': <module '__main__' (built-in)>
or
'__main__': <module '__main__' from 'C:\\Users\\aaron\\Documents\\scripts\\python\\untitled0.py'>

So, @andreamoro-git and @athompson673, if you try to run the snippet of code in an ipython console, the error appears?

That's where it appears, on spyder's ipython console. If you run it using python's command line (outside of spyder) there is no error.

I don't know why this is not working, but

If you're running your code in an unsaved file (as @athompson673 seems to be doing) could be related to this because multiprocessing could have problems pickling code in that situation.
The work @impact27 is doing in PR #14025 could help at least to debug these problems.

The file is saved

So I checked this issue. I can reproduce with iPython. If you:

import multiprocessing.spawn
then run the file
You will get: AttributeError: Can't get attribute 'myClass' on <module '__main__'>

This might be a python error?
Desktop % ipython3
Python 3.8.6 (default, Oct  8 2020, 14:06:32)
Type 'copyright', 'credits' or 'license' for more information
IPython 7.29.0 -- An enhanced Interactive Python. Type '?' for help.

In [1]: import multiprocessing.spawn

In [2]: %run untitled1.py
0
1
Exception in thread Thread-3:
2
Traceback (most recent call last):
  File<EMAIL_ADDRESS>line 932, in _bootstrap_inner
3
4
5
6
7
8
9
    self.run()
  File<EMAIL_ADDRESS>line 870, in run
    self._target(*self._args, **self._kwargs)
  File<EMAIL_ADDRESS>line 576, in _handle_results
    task = get()
  File<EMAIL_ADDRESS>line 251, in recv
    return _ForkingPickler.loads(buf.getbuffer())
AttributeError: Can't get attribute 'myClass' on <module '__main__'>

Thanks @impact27 for your help with this!
This will be fixed in our next version (5.2.2), to be released next week.


chore(templates): add templates @ionic-app-generators

Short description of what this resolves:
Add templates @ionic-app-genetators
Changes proposed in this pull request:

Ionic Version: 2.x

Hi @ramonornela,
These templates are located here:
https://github.com/driftyco/ionic/tree/master/scripts/templates
Am I missing something that you've noticed? I am trying to understand the intent and context.
Thanks for contributing! Sorry the project is in so much flux right now as we get ready for Beta 12!
Thanks,
Dan

@danbucholtz ionic-cli@beta reference the path node_modules/ionic-angular/templates.
Perhaps the problem in tasks of the gulp not copy to npm publish.

@ramonornela,
Yep, you are correct. I forgot to push my commit with copying the templates over to the bundle. Been really scattered lately working on too many different parts of Ionic.  Thanks for letting us know.  I'm going to close this for now - thank you for being an active community member
Thanks,
Dan


export without -u, -p, -o, -r arguments

When GitReleaseManager run from directory with git repository, for export read-only public access is enough. GitHub repository - from remote repositories for current (in current directory) repository. As https://github.com/GitTools/GitReleaseNotes...

@sergey-s-betke I will have to check whether Octokit allows the creation of an anonymous API call.  I will check, and let you know.


Make it possible to just read current_user

I have a use case, where I want to do A if there is a user, and B if there is no user.
So I need to read the current_user without doing anything if it isn't there.
I know you're in the midst of version 2.0, but just wanted to get this idea in here.

It should definitely be possible to do this. I'll make sure I look at this asap (in the week if everything goes well). Sorry for the delay.
Thanks for contributing! :+1:

This seems like a great solution to me! :+1:
The authentication mechanism will change in the v2 but I'll make sure it is still possible to do this.
I'll be happy to merge this if we can add a few things:
:wrench:

[ ] Add some tests (We probably need to create another controller in the dummy test app for this)
[ ] Replace split(' ') by split as the default delimiter is already a whitespace
[ ] Fix conflicts with master

:book:

[ ] Update the CHANGELOG.md
[ ] Update the README.md

Just let me know if you don't think you'll have time for this and I'll create another PR.
Sorry it took me a bit of time to review!
Cheers

Hi @nsarno ,
You should be able to check off everything on your list now. :)
I did something slightly different for the split(' '), though, to be more specific.
That's probably how I would have written it, if I had written it in the first place.
I hope you agree that it is better. Otherwise I can change it to split in a... you guessed it... split second.

Awesome! We could probably squash this into one commit before I merge it.

I'll do that. Just a sec.
ons. 30. dec. 2015 kl. 14.28 skrev Arnaud Mesureur<EMAIL_ADDRESS>
:

Awesome! We could probably squash this into one commit before I merge it.
—
Reply to this email directly or view it on GitHub
https://github.com/nsarno/knock/pull/24#issuecomment-167998784.

Uhm... I'm not sure how to do that.
The fixed conflict makes me trip over my git-skills.
Any suggestions?

No worries. Master should probably have been rebased instead of merged, then you could have squashed the commits without trouble. I created a new branch, cherry-picked your commits and squashed them (cf #31). Should be all good now :+1:
I'll release v1.4 soon with all the recent changes including this, thanks! :)

No worries. Master should probably have been rebased instead of merged, then you could have squashed the commits without trouble. I created a new branch, cherry-picked your commits and squashed them (cf #31). Should be all good now :+1:
I'll release v1.4 soon with all the recent changes including this, thanks! :)


passport-linkedin

Hi,
I don't seem to be able to get passport-linkedin working. It redirects back with the following response below. I copied it while it was in the address bar. Upon redirect however it doesn't authenticate and no entry is made in the DB.
Any help would be appreciated. Does anybody have a sample config or how-to?
linkedin: {
name: 'Linkedin',
protocol: 'oauth',
strategy: require('passport-linkedin').Strategy,
options: {
consumerKey: 'mykey',
consumerSecret: 'mysecret'
},
scope:["r_basicprofile"],
state:true
}
http://localhost:1337/auth/linkedin/callback?oauth_token=yy--7c1e3f37-bbb6b-aaa6-94cf-5dc41447c16d&oauth_verifier=78888

I have also tried the passport-linkedin-oauth2 with no luck either.

I have tracked the issue down to the user.username or user.email never being set.
Any suggestions on why this is?
// If neither an email or a username was available in the profile, we don't
// have a way of identifying the user in the future. Throw an error and let
// whoever's next in the line take care of it.
if (!user.username && !user.email) {
console.log("error username");
return next(new Error('Neither a username nor email was available'));
}

For those that are interested. It was a config issue. Here is where to put the profileFields attribute in options. This will set username and email address.
linkedin: {
name: 'Linkedin',
protocol: 'oauth',
strategy: require('passport-linkedin').Strategy,
options: {
consumerKey: 'key',
consumerSecret: 'secret',
profileFields: ['id', 'first-name', 'last-name', 'email-address', 'headline']
},
callbackURL: 'http://localhost:1337/auth/linkedin/callback',
state:true,
scope:['r_basicprofile','r_emailaddress'],
}


Address issues picked up by clazy/clang-tidy

There are a lot of autogenerated files submitted as this PR. Please edit the commit (reset HEAD~) and add only source files.

@piggz: looks good by me, now have to wait for OBS build results. Would you want to supplement this with one commit with reformatting whole source?


refactor: rename component infra to requests and handlers

Stack:

#198 ⬅
#185
#181
#180
#167
#166
#161
#160

⚠ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

This change is

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 65.37%. Comparing base (e051e80) to head (545d52c).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #198   +/-   ##
=======================================
  Coverage   65.37%   65.37%
=======================================
  Files          25       25
  Lines         930      930
  Branches      930      930
=======================================
  Hits          608      608
  Misses        295      295
  Partials       27       27

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


[swift-generator] Skip more unhandled types

These types currently cause a crash. I actually have an upcoming fix to
handle the types instead of just skipping them, but I want this in first
in case I need to back-port a "fix" to older release branches without adding
functionality.

lgtm


feat: Reenable native fee payment

Had been disabled after #6403
Builds on top of #6403

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#6571  👈
#6403
master

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

Join @spalladino and the rest of your teammates on  Graphite

Benchmark results
Metrics with a significant change:

proof_construction_time_sha256 (16): 711 (-23%)
tx_pxe_processing_time_ms (2): 57,420 (-18%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Proof generation
Each column represents the number of threads used in proof generation.

Metric
1 threads
4 threads
16 threads
32 threads
64 threads

proof_construction_time_sha256
5,719
1,552 (+1%)
:warning: 711 (-23%)
767 (-10%)
777 (-5%)

L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
64 txs

l1_rollup_calldata_size_in_bytes
1,412
1,412
1,412

l1_rollup_calldata_gas
9,464
9,476
9,452

l1_rollup_execution_gas
616,105
616,117
616,093

l2_block_processing_time_in_ms
1,343 (-3%)
4,996 (-3%)
10,124

l2_block_building_time_in_ms
45,672 (-3%)
180,725 (-3%)
367,221 (-2%)

l2_block_rollup_simulation_time_in_ms
45,506 (-3%)
180,097 (-3%)
365,987 (-2%)

l2_block_public_tx_process_time_in_ms
24,960 (-3%)
101,540 (-3%)
210,494 (-2%)

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
3 blocks
5 blocks

node_history_sync_time_in_ms
9,909 (-1%)
15,160 (-2%)

node_database_size_in_bytes
14,458,960
21,336,144

pxe_database_size_in_bytes
18,071
29,868

Circuits stats
Stats on running time and I/O sizes collected for every kernel circuit run across all benchmarks.

Circuit
protocol_circuit_simulation_time_in_ms
protocol_circuit_witness_generation_time_in_ms
protocol_circuit_proving_time_in_ms
protocol_circuit_input_size_in_bytes
protocol_circuit_output_size_in_bytes
protocol_circuit_proof_size_in_bytes
protocol_circuit_num_public_inputs
protocol_circuit_size_in_gates

private-kernel-init
158 (-2%)
3,626
22,121
19,990
61,798
86,720
2,643
1,048,576

private-kernel-inner
597 (-2%)
4,175
47,393 (-3%)
88,862
61,798
86,720
2,643
2,097,152

private-kernel-tail
521 (-2%)
3,004 (+6%)
35,165
86,653
76,130
10,624
265
2,097,152

base-parity
6.51
1,273 (-5%)
2,922 (+1%)
128
64.0
2,208
2.00
131,072

root-parity
49.3 (-1%)
53.3 (+9%)
44,058
27,084
64.0
2,720
18.0
2,097,152

base-rollup
771
2,289 (-1%)
74,236
119,578
756
3,648
47.0
4,194,304

root-rollup
114 (-1%)
65.7 (-3%)
19,673 (+2%)
25,297
620
3,456
41.0
1,048,576

public-kernel-app-logic
517 (-2%)
2,839
47,265
103,405
84,766
116,832
3,584
2,097,152

public-kernel-tail
1,177 (-2%)
25,510
155,030
393,850
7,490
10,624
265
8,388,608

private-kernel-reset-small
571 (-2%)
2,214 (-4%)
26,652 (+8%)
118,173
61,798
86,720
2,643
1,048,576

merge-rollup
28.5 (-1%)
N/A
N/A
16,534
756
N/A
N/A
N/A

public-kernel-setup
631 (+1%)
N/A
N/A
103,405
84,766
N/A
N/A
N/A

public-kernel-teardown
534
N/A
N/A
103,405
84,766
N/A
N/A
N/A

private-kernel-tail-to-public
N/A
9,600 (+1%)
83,076 (+1%)
N/A
N/A
116,832
3,584
4,194,304

Stats on running time collected for app circuits

Function
app_circuit_proof_size_in_bytes
app_circuit_proving_time_in_ms
app_circuit_size_in_gates
app_circuit_num_public_inputs

SchnorrAccount:entrypoint
16,128
46,023 (-1%)
2,097,152
437

Token:transfer
16,128
46,221 (+6%)
2,097,152
437

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
1 leaves
16 leaves
64 leaves
128 leaves
512 leaves
1024 leaves
2048 leaves
4096 leaves
32 leaves

batch_insert_into_append_only_tree_16_depth_ms
11.3 (-1%)
18.5
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_count
16.7
31.8
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_ms
0.656 (-1%)
0.569
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_32_depth_ms
N/A
N/A
52.4 (-1%)
81.0
258 (-3%)
505
969 (-3%)
1,964
N/A

batch_insert_into_append_only_tree_32_depth_hash_count
N/A
N/A
95.9
159
543
1,055
2,079
4,127
N/A

batch_insert_into_append_only_tree_32_depth_hash_ms
N/A
N/A
0.534 (-2%)
0.500
0.468 (-3%)
0.472
0.460 (-3%)
0.469
N/A

batch_insert_into_indexed_tree_20_depth_ms
N/A
N/A
62.3 (-1%)
118 (-2%)
372 (-3%)
740 (-1%)
1,451 (-3%)
2,942 (-1%)
N/A

batch_insert_into_indexed_tree_20_depth_hash_count
N/A
N/A
106
208
692
1,363
2,707
5,395
N/A

batch_insert_into_indexed_tree_20_depth_hash_ms
N/A
N/A
0.539 (-1%)
0.532 (-1%)
0.505 (-2%)
0.509 (-1%)
0.503 (-3%)
0.512 (-1%)
N/A

batch_insert_into_indexed_tree_40_depth_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
66.6 (-1%)

batch_insert_into_indexed_tree_40_depth_hash_count
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
107

batch_insert_into_indexed_tree_40_depth_hash_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
0.591 (-1%)

Miscellaneous
Transaction sizes based on how many contract classes are registered in the tx.

Metric
0 registered classes
1 registered classes

tx_size_in_bytes
82,348
664,768

Transaction size based on fee payment method
| Metric |  |
| - |  |
Transaction processing duration by data writes.

Metric
0 new note hashes
1 new note hashes
2 new note hashes

tx_pxe_processing_time_ms
11,596 (+4%)
4,170 (-1%)
:warning: 57,420 (-18%)

Metric
1 public data writes
2 public data writes
3 public data writes
4 public data writes
8 public data writes

tx_sequencer_processing_time_ms
3,175 (-2%)
2,148 (+1%)
6,078 (+1%)
2,315
2,608


Feature/docker params

Implementing a generic way of passing in docker params the same way marathon does it.
references #536
cc/ @strzelecki-maciek

One tiny comment, but it otherwise looks great. Once you update the PR, I'll merge it.

Thanks @brndnmtthws for having a look. I dropped the double newlines, but I kept a single newline as a visual separator. I hope that's what you meant.

Yep, looks good!

@brndnmtthws, @gkleiman any chance to get a new release with this feature? It's been a while since 2.4.0.

Same here, I'd really like to use a released version with this feature.

scheduler/jobs doesn't return the custom container parameters

@vivekkothari is it because this feature hasn't been released?

@m1k3,  which release is this feature available in?

AFAIK it wasn't released yet as the latest release 2.4.0 was released before this feature was merged.

Until it's released one workaround is to start you docker container by yourself with a chronos command, then you can use any specific config or param.

v2.5.0 has been released with the changes in this PR.
https://github.com/mesos/chronos/releases/tag/2.5.0


Set $TERM_PROGRAM and $TERM_PROGRAM_VERSION by default

os: ubuntu 16.04
Many terminal emulator sets $TERM_PROGRAM and $TERM_PROGRAM_VERSION by default or $VTE_VERSION if VTE based. Settings this helps Alacritty integrate with other tools.

Many terminal emulator

Which ones? Termite doesn't. There's $TERM, which probably does the same thing as $TERM_PROGRAM with the difference of being more widespread. I can't imagine any use for a $TERM_PROGRAM_VERSION variable except for user information, which is usually accomplished via a command line flag like -v. For anything else, there's terminfo.

iTerm2.app, Apple Terminal and Hyper does set these env variables.
I had a look at terminfo through infocmp but it doesn't tell me anything unique about Alacritty. The tool I'm working on is changing cursor shape in vim (via vimscript). I would be happy if there is something that says alacritty or something that uniquely identifies it so I can trigger the desired behaviour.

I think best practice is to look for if the terminal implements the features you require. That information should be in terminfo. Otherwise you might want to add it there.

Great feature request! Neovim, among others, use these environment variables, e.g.
neovim/neovim/blob/f050aaab/src/nvim/tui/tui.c#L206-L211 to address capabilities and bugs.

@rafi If possible, neovim uses the info from that is provided in termcap/terminfo to see what the terminal supports.
Sadly many terminals lie and says that they are xterm but do not support all the features xterm does. It has gotten better recently, so there are only a few cases where neovim has to work around this by looking for non standard env variables. (Or make sure that the terminal really IS xterm)
As we can see from neovim, the only terminal that is detected via $TERM_PROGRAM is iTerm. So I do not think this method is that widely adopted. The other env variables are for konsole and libvte terminals.

Okay I'll have a look at neovim and see what I can pick out thanks.

Hm.. is this feature request can be opened again ? although not a standard, many terminal emulators on macOS did this. My use case is to detect if alacritty is in used, then it will launch tmux, since no tabs in alacritty, it is my preference though. This is an example ~/.zshrc:
if [ "$TERM_PROGRAM" != "Apple_Terminal" ] && [ "$TERM_PROGRAM" != "iTerm.app" ]; then
    [[ $- != *i* ]] && return
    [[ -z "$TMUX" ]] && exec tmux
fi

this will be easily readable:
if [ "$TERM_PROGRAM" = "Alacritty" ]; then
    [[ $- != *i* ]] && return
    [[ -z "$TMUX" ]] && exec tmux
fi

taken from here


Multicast from TS

Hi, maybe help with create command for extraction or analyze this situation:
source: satellite signal using Multistream 21
stream: multicast stream on pid 515
`root@osmio4kplus:~# dvbsnoop -s pidscan
Transponder PID-Scan...
PID found:    0 (0x0000)  [SECTION: Program Association Table (PAT)]
PID found:   16 (0x0010)  [SECTION: Network Information Table (NIT) - actual network]
PID found:  515 (0x0203)  [SECTION: DSM-CC - private data section  // DVB datagram]
PID found:  802 (0x0322)  [SECTION: Program Map Table (PMT)]
PID found: 2097 (0x0831)  [SECTION: User private]
PID found: 2098 (0x0832)  [SECTION: User private]
PID found: 5632 (0x1600)  [unknown]
root@osmio4kplus:~# tsp -v -I dvb --device-name /dev/dvb/adapter0 --delivery-system DVB-S2 --fec-inner 8/9 --frequency 10810000 --polarity horizontal --modulation QPSK --roll-off auto -s 30000000 -P tables -a -d -p 515 -b test -O drop

tables: creating test.zip
Error: dvb: error opening /dev/dvb/adapter0/frontend0: Device or resource busy

root@osmio4kplus:~# tslsdvb -a 0
/dev/dvb/adapter0 (Availink AVL6261, DVB-S, DVB-S2)`

Hi @EnoSat
The "device busy" error seems to indicate that the tuner is already used, typically by some program receiving a TS.
Some additional remarks:

You name the output binary file of the tables plugin "test.zip". The name seems to imply that you expect a zipped file. That won't be the case. The binary file contains raw sections as described in section 2.2.1 of the TSDuck user's guide.
You do not specify any terminating condition. Your command will run forever (since a DVB reception has no "end of file") and sections accumulate forever in the binary file. You should specify some time or packet limit using a plugin until.

Judging by the command line and tuner type, he's probably trying to run TSDuck on a Linux satellite receiver running Enigma2 OS (Edision OS Mio 4K). It probably can't work because of tuner limitations in these boxes. This was discussed before: https://github.com/tsduck/tsduck/issues/173

@rezna11, thanks for the hint. This type of device is typically used for illegal access to PayTV content and consequently out of scope for TSDuck which is a lab tool for professional and hobbyists (needless to say that hobbyists and wannabe-hackers are distinct types of population).


Disable failing didReceiveRemoteNotification tests(#1451)

See #1451 for why this workaround is needed.

Will merge on approval and green travis.


CORE-338 - Refactor FTE Image Processing

Refactoring the FTE image upload and retrieve process.

Obviously this brings about a breaking change in how images are stored and retrieved. It might be that we have to manually enter the file names of the images in production.

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


Add training data

@erasche do we want to have a separate file for taining data? For training data we do not need some columns.

@bgruening yeah I'd be in support of that.

As-is this breaks the column conventions - would empty values work?

We found a better way for this imho. We getting doi's as seen here: https://github.com/bgruening/training-material/tree/master/Exome-Seq


[Draft] Add font objects

See #3488 and #3331.

[X] Add font type (implicitly convertible from string)
[ ] Add per-font OpenType features – will be a lot more complicated because text::features assumes that text will use the same OpenType features regardless of which font is selected.
[ ] Add per-font Unicode ranges

Unresolved questions:

Should we deprecate OpenType feature-related options on text?

Should we deprecate OpenType feature-related options on text?

In my opinion yes. These features are fundamentally tied to each specific font.

In my opinion yes. These features are fundamentally tied to each specific font.

Would that only apply to alternates, stylistic_set, and features, or others like kerning, ligatures, and number_type?

In my opinion yes. These features are fundamentally tied to each specific font.

Would that only apply to alternates, stylistic_set, and features, or others like kerning, ligatures, and number_type?

All of them. But Laurenz may have a different opinion.

I have a feeling that it'd still be nice to be able to configure those independently of the font (while also offering the possibility of having per-font configuration).

I have a feeling that it'd still be nice to be able to configure those independently of the font (while also offering the possibility of having per-font configuration).

"But Laurenz may have a different opinion."
That is: The font-specific features should be move from field of text element to that of font object, but not necessary for those are not font-specific. For example, it is possible not meaningful to share stylistic_set, but can be meaningful to share ligatures.

"But Laurenz may have a different opinion."

Yeah! Just wanted to share mine too :)

That is: The font-specific features should be move from field of text element to that of font object, but not necessary for those are not font-specific. For example, it is possible not meaningful to share stylistic_set, but can be meaningful to share ligatures.

Sure, but how could we implement those shared properties other than through the current design?
I guess we could make font() foldable in order to have some default-font property usable within set rules, or something. Not sure though - I feel a bit uneasy about the current design (I wonder if this is sufficiently appealing to new users).

Right now, I have font-specific features always take precedence over features set in text. I think it would be better to have whatever setting is innermost take precedence, but I’m not sure how this would be done.

I left a few comments in the Fonts forge on Discord.

I think unfortunately this needs more design work before we can proceed with the implementation, so I'll close this for now. :/ Thank you for the work you put into this!


Does GLM-130 B support [sMASK] for sentence generation?

I'm developing a chat bot on top of GLM-130B.
Currently I'm using "[MASK]" at the end of dialogue for bot's response generation.
[gMASK] is too slow for me on my 8xV100 server.
Your GLM repo https://github.com/THUDM/GLM reports [sMASK] could be used for sentence generation.
But I didn't find any doc in this repo. Does GLM-130 B support [sMASK] for sentence generation?
Do you have any plan to export more API of GLM-130 B ? Such as compute LM perplexity / Multiple choice selection or any other features? Since you have already test the model on Few-CLUE, there must be ways to utilize those features.

@lomizandtyd Hi,
Thanks for your interest in GLM-130B!

Currently I'm using "[MASK]" at the end of dialogue for bot's response generation.

That's cool! Looking forward to your application online, and please contact as we can also help to promote it for you.

Your GLM repo https://github.com/THUDM/GLM reports [sMASK] could be used for sentence generation.
But I didn't find any doc in this repo. Does GLM-130 B support [sMASK] for sentence generation?

Unfortunately, we didn't implement [sMASK] in GLM-130B since it harms the efficiency of pre-training. However, we are planning to train GLM-130B for a few steps on [sMASK] objectives to enable this feature. Please wait for a few days as we are working on it : )

Do you have any plan to export more API of GLM-130 B ? Such as compute LM perplexity / Multiple choice selection or any other features? Since you have already test the model on Few-CLUE, there must be ways to utilize those features.

That sounds great. But it may not be our first priority since we are now having a hard time sparing any machines for such a goal. Thanks for your suggestions.

Could you provide some guidelines about how to compute PPL/logprobs on GLM-130 B?

Hello @lomizandtyd, we have recently implemented preliminary code to test PPL on tasks such as WikiText103 and PTB, you can check out the eval-lm branch and the code will be merged into the main branch after further testing. If you have any questions, please feel free to ask!

Thanks, that's what I want.


No "dotenv" thing.

Error:
<"path"\RVC1006Nvidia>python infer-web.py
Traceback (most recent call last):
File ""path"\RVC1006Nvidia\infer-web.py", line 3, in
from dotenv import load_dotenv
ModuleNotFoundError: No module named 'dotenv'

did you install python-dot or requirements.txt?

I have them all installed and its still is throwing a fit saying I don't have the module installed. Actually infuriating

@SeranaZ
that's weired. I mean did you activate your environment correctly?(aka conda venv whatever) I use vscode. vscode sometimes use default environment if you didn't specify environment. then It can't find specific module what you installed.

I feel like its not recognizing 3.12 of python. Thats where dotenv is
On Sun, Mar 10, 2024, 11:16 PM fabyday @.***> wrote:

@SeranaZ https://github.com/SeranaZ
that's weired. I mean did you activate your environment correctly?(aka
conda venv whatever) I use vscode. vscode sometimes use default environment
if you didn't specify environment. then It can't find specific module what
you installed.
—
Reply to this email directly, view it on GitHub
https://github.com/RVC-Project/Retrieval-based-Voice-Conversion-WebUI/issues/1871#issuecomment-1987550309,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A25Z3DG2VVZ2727QY7QIXZLYXUO2FAVCNFSM6AAAAABEJNXPL6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBXGU2TAMZQHE
.
You are receiving this because you were mentioned.Message ID:
<RVC-Project/Retrieval-based-Voice-Conversion-WebUI/issues/1871/1987550309
@github.com>

@SeranaZ
strongly agree with you. how about checing the python path is correct?
'''which python'''
and then find dot-env in libray/site-package

Im not sure how. Tips on linux syntax for the command to check?
On Mon, Mar 11, 2024, 11:57 PM fabyday @.***> wrote:

@SeranaZ https://github.com/SeranaZ
strongly agree with you. how about checing the python path is correct?
'''which python'''
and then find dot-env in libray/site-package
—
Reply to this email directly, view it on GitHub
https://github.com/RVC-Project/Retrieval-based-Voice-Conversion-WebUI/issues/1871#issuecomment-1990157674,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A25Z3DES7VU2QBWSQ67ZH5LYXZ4KJAVCNFSM6AAAAABEJNXPL6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOJQGE2TONRXGQ
.
You are receiving this because you were mentioned.Message ID:
<RVC-Project/Retrieval-based-Voice-Conversion-WebUI/issues/1871/1990157674
@github.com>

oh, I'm sorry I use cygwin or wsl. so I can use command mixed.

It's activated, I checked. i got it to recognize 3.12. The issue: I can only run it on linux, not from go-web application AND it thinks I don't have the package i18n. This is the buggiest release I have ever seen, and 3.12 also has its fair shar of bugs, but a downgrade didn't get different results

@SeranaZ
could you show me your entire error message?

I have all packages installed, I already checked. But I will send over the screenshots

heres when I run on linux versus batch file. I am mor successful on linux, I have the package from the first screenshot. it is only when I remove it trying to call it as a package that it functions but because unix has no GUI it never opens it, which is on me. However, running batch would open on previouu RVC updates so idk what the issue is with this update specifically
here is how it runs when I remove .i18n

runs completely fine, there may be a bug outside of using python engine that they need to fix

I was able to get a virtual environement going without heavy code modification, the only issue now is still no gui will open. I had to modify run because i could not find .venv because my actiavte syntax was no .venv, another coding issue they might need to fix

@SeranaZ
nice description. thanks.
ok... i18n is existed on this project. you might know that.
I think it was caused by some reason.
if project doesn't consider i18n as module, what about add init.py to i18n folder.
just add empty init.py.
like this.

I commented like here for detail.
https://docs.python.org/3/tutorial/modules.html
I think that i18n recognition is tiny problem.

Deleting the .i18n causes no issues. I got everything working just fine
now. Calling .i18n is redundant because since it is already in RVC there is
no point in some instances. Vocal separation is clear, training is good,
GPU detected and it tells me that i18n is there and working whenever I
start it up. I got it to work through tinkering with the code, downgrading
to version 1.23 of numpy (python is dumb and decided to get rid of np.float
and just make it float, causing several issues with librosa) and using a
combo of linux and windows. The fact I had to jump through so many hoops
just to get this working is crazy, mostly because python cant decide what
it wants to do with its updates. Its not necessarily an RVC issue, besides
the fact that it favors windows set up which is an issue in more ways than
one when you dont account for debugging for linux (not sure about mac,
since I dont use it but seeing as they get their own shell files I'd guess
its dumb dumb proof)
All in all, a terrible experience. I cant explain why what I did works, it
just does. The life of a programmer in a nut shell. So note to anyone
having issues-if it throws i18n.i18n does not exist, try getting rid of the
second i18n and make sure you do NOT use anything about python 3.10 unless
you want to deal with numpy, getting yelled at about pytorch
incompatibility, and overall bash file hell.
I spent so much time on this I legit am confident I am qualified to help
trouble shoot rvc, thats how much time I have spent on this project
dubugging and understanding how to use it across multiple platforms. I
fixed or diagnosed several issues on my own, it was insane. It was like IT
role play how intense these issues absorbed my life.
Anyway, rant over. Its working, and I'm happy.
I think it was caused by some reason.
if project doesn't consider i18n as module, what about add init.py to
i18n folder.
just add empty init.py.
like this.
8AFA253F-5FFC-490D-A30C-64B9F0190286.png (view on web)
https://github.com/RVC-Project/Retrieval-based-Voice-Conversion-WebUI/assets/28583613/15b08f44-9d9d-429d-82c3-a8b88c8596a9
I commented like here for detail.
https://docs.python.org/3/tutorial/modules.html
I think that i18n recognition is tiny problem.
—
Reply to this email directly, view it on GitHub
https://github.com/RVC-Project/Retrieval-based-Voice-Conversion-WebUI/issues/1871#issuecomment-2017121113,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A25Z3DHY4BA6755JNFMZCEDYZ6G27AVCNFSM6AAAAABEJNXPL6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMJXGEZDCMJRGM
.
You are receiving this because you were mentioned.Message ID:
<RVC-Project/Retrieval-based-Voice-Conversion-WebUI/issues/1871/2017121113
@github.com>

hahah. you were so mad. It is usally happended on this field.
But I think you finally found your own solution.
yes. I thought too.🤭

What was the solve here? I am also facing this issue on mac

Mac will have different issues so I can not help you
On Sun, Mar 31, 2024, 10:00 AM Daniel Raban @.***>
wrote:

What was the solve here? I am also facing this issue on mac
—
Reply to this email directly, view it on GitHub
https://github.com/RVC-Project/Retrieval-based-Voice-Conversion-WebUI/issues/1871#issuecomment-2028755250,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A25Z3DGVCRPH2243QG2PHJTY3AJIJAVCNFSM6AAAAABEJNXPL6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMRYG42TKMRVGA
.
You are receiving this because you were mentioned.Message ID:
<RVC-Project/Retrieval-based-Voice-Conversion-WebUI/issues/1871/2028755250
@github.com>


Loading samples and have default errors

Describe the bugs
Loading samples and have default errors - setting Allow interruptions to be true solves the issue.
This issue exists in AskingQuestionsSample, ControllingConversationFlowSample, and ActionSample.
AskingQuestionsSample

To Reproduce

Expected behavior
Should not show default errors when loading a sample bot.

@hibrenda and I found this issue when she was showing me how to use the dynamic choice prompt of Composer last week. The new feature has been added to this doc.


Feature request for Dashboard template for ontaps3 metrics

Is your feature request related to a problem? Please describe.
I re-enable the ontap s3 metrics on 24.02.0 release by uncommenting those ontaps3 metrics templates. It'd be good if we have some dashboard templates for those ontaps3 metrics.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Thanks for the feature request @gaow1423.
Have you built an s3 metrics dashboards already or have any input on which metrics are most important?

See also https://discord.com/channels/855068651522490400/1265265157479862335/1275400044769579132

Default ontap_s3_svm  template is disabled and dashboard look like this,


Using msal-angular with Angular9 and IE11 causes build error.

Core Library
MSAL.js v2 (@azure/msal-browser)
Core Library Version
2.14.2
Wrapper Library
MSAL Angular (@azure/msal-angular)
Wrapper Library Version
2.0.0
Description
We're using IE11 with Angular9 in our project, but having some trouble while introducing msal-angular into it.
When we install msal-angular and build the project, we get the following error.
ERROR in getInternalNameOfClass() called on a non-ES5 class: expected MsalService to have an inner class declaration
It's clear that it is just a compilation issue of msal-angular which is built with ES2015 has not downgraded properly by Angular9 IVY compiler,
We've already got around this problem by using locally built msal-angular package targetting at es5, and put it directly into the resource, but I wonder if there's any better workaround.
MSAL Configuration
No response
Relevant Code Snippets
It's fairly easy to reproduce using an official sample.

1. download Angular9 sample
https://github.com/AzureAD/microsoft-authentication-library-for-js/tree/dev/samples/msal-angular-v2-samples/angular9-v2-sample-app

2. change build target to ES5
[tsconfig.json]
"target": "es5",

3. add some pollyfills
npm -i core-js@latest
[pollyfill.ts]
import 'core-js';

4 npm run build

Identity Provider
Azure B2C Custom Policy
Source
External (Customer)

I can reproduce this, and will close as a duplicate of #3277. It seems Angular 9 does not perform the needed transpiling by default, while Angular 11 does, so I'll see if I can figure out what can be done about that.