Add Multiple Sentiment/Hate Speech Classification Datasets

Checklist for adding MMTEB dataset
Reason for dataset addition: Increase multilingual coverage of the MMTEB benchmark by adding 3 new datasets about Sentiment/Hate Speech Classification spanning 8 languages (3 new).
Specifically, the following 3 datasets have been added to this PR:

German Politicians Twitter Sentiment
Hate Speech Portuguese
Tweet Sentiment Multilingual

I have evaluated the models on the 3 datasets as specified in the checklist, and added the results in the PR.
Concerning the points, I computed them as follows: 6 for the German dataset (2 + 4 for the new language), 6 for the Portuguese dataset (2+4), and 20 (2*7 + 6) for the multilingual dataset with 8 languages (1 new). I'm not sure about the calculations, please correct me if I'm wrong.

[x] I have tested that the dataset runs with the mteb package.
[x] I have run the following models on the task (adding the results to the pr). These can be run using the mteb run -m {model_name} -t {task_name} command.

[x] sentence-transformers/paraphrase-multilingual-MiniLM-L12-v2
[x] intfloat/multilingual-e5-small

[x] I have checked that the performance is neither trivial (both models gain close to perfect scores) nor random (both models gain close to random scores).
[x] If the dataset is too big (e.g. >2048 examples), considering using self.stratified_subsampling() under dataset_transform()
[x] I have filled out the metadata object in the dataset file (find documentation on it here).
[x] Run tests locally to make sure nothing is broken using make test.
[x] Run the formatter to format the code using make lint.
[x] I have added points for my submission to the points folder using the PR number as the filename (e.g. 438.jsonl).

Concerning the points, I computed them as follows: 6 for the German dataset (2 + 4 for the new language), 6 for the Portuguese dataset (2+4), and 20 (2*7 + 6) for the multilingual dataset with 8 languages (1 new). I'm not sure about the calculations, please correct me if I'm wrong.

I believe german, portugese and  is already covered by clf for the multilingual dataset you only get 2 points for the dataet and 4 pr. new language. I believe this makes the points 2+2+(2+4) (what is the new language in the dataset)

Concerning the points, I computed them as follows: 6 for the German dataset (2 + 4 for the new language), 6 for the Portuguese dataset (2+4), and 20 (2*7 + 6) for the multilingual dataset with 8 languages (1 new). I'm not sure about the calculations, please correct me if I'm wrong.

I believe german, portugese and is already covered by clf for the multilingual dataset you only get 2 points for the dataet and 4 pr. new language. I believe this makes the points 2+2+(2+4) (what is the new language in the dataset)

Oh, I see. Thanks a lot for the clarification, Kenneth! Then it is 2+2 for the German and Portuguese datasets; I checked if these languages had specific subfolders in the Classification folder without checking the languages covered by the multilingual datasets.
The new language for the multilingual dataset was Spanish, but I just checked and it is included in other multilingual datasets. I made the multiplication 2*7 only because I saw it in PR #512, but if it should not be multiplied, the total is 2+2+2=6.

Thanks, I have added a comment on #512 (generally it has not been the case though).

Thanks, I have added a comment on #512 (generally it has not been the case though).

Perfect! :D


undefined for authResult when passed into auth0.parseHash

I'm seeing undefined for authResult after authenticating by Auth0
Below is my callback component:
import { Component } from '@angular/core';
import { AuthService } from '../auth.service';

@Component({
  template: ``
})
export class CallbackComponent {

  constructor(private authService: AuthService) {
    this.authService.handleAuth();
  }
}

Below is my AuthService handleAuth() method:
handleAuth() {
    // When Auth0 hash parsed, get profile
    this.auth0.parseHash((err, authResult) => {
      console.error(authResult); // responds "undefined"
      if (authResult && authResult.accessToken && authResult.idToken) {
        window.location.hash = '';
        this._getProfile(authResult);
        this.router.navigate(['/']);
      } else if (err) {
        this.router.navigate(['/']);
        console.error(`Error: ${err.error}`); // responds "Error: invalid_token"
      }
    });
  }

Any idea why this might be happening? How can I further debug this?

I'm having this issue too. However, my callback isn't working. My handleAuthentication function is called from the app.component.ts and also from callback component.

same issue. any luck?


[Bug] Segmentation Fault when using a directory map without definition

#load_all "project:lib"
use lib

use core { * }

main :: () {
    println(lib.NAME)
}

./onyx run  scratch.onyx

[!CAUTION]
[1]    13675 segmentation fault  ./onyx run scratch.onyx

./onyx run --map-dir project:. scratch.onyx

[!NOTE]
Onyx

I think a good way would be show an error message saying that map-dir/collection was not defined.

Thanks for the bug report! I can't believe I didn't test the case where the mapped directory didn't exist... This will be fixed in the next release.

I can't seem to reproduce this on my machine on the latest master branch. Can you post some more details about all the files in the compilation and what system you are on?

macOS Monterrey: 12.7.4
Onyx toolchain version v0.1.13
Runtime: ovmwasm

Added sample project
mapdir-bug.zip


Don't clear the terminal after every message

Either output logs somewhere or don't clear the terminal screen to allow us a chance to read what exactly got executed and if any be able to read error messages.
I would also appreciate if you could omit the first nag messages on script start.
Thanks for your work.

@iMouath We are working on the loggings..
Please explain more about the 'nag' messages..

It takes more than 30+ seconds to get to the actual app.
I understand I might be impatient and I believe there's some actual setup
happens during this time(haven't looked at the code) I apologise for coming
out angry.
But this happens all the time you launched the script or even change
something. And on the way out.
If it isn't necessary for the workflow then it shouldn't be there. For
technical help messages either provide them in a README file or create a
submenu where a person who is looking for them can go and read them.
Otherwise it's taking time and cannot even read them before the screen gets
cleared.
Just a suggestion
Thanks.
On Wed, Dec 12, 2018, 9:19 PM Avinash Reddy<EMAIL_ADDRESS>wrote:

@iMouath https://github.com/iMouath We are working on the loggings..
Please explain more about the 'nag' messages..
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/EnergizedProtection/block/issues/50#issuecomment-446828822,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AH1f2V7x3PfUgP4sR2a7kkilSmkfu4_Wks5u4cc9gaJpZM4ZQ2bO
.

--
Mouath Ibrahim
Mouath.com

@iMouath I understand your situation.
Do have a look at the code and send us some suggestions via PR.


Example required

Hi, It is very interesting tool. Can you add examples to the repo touching different aspects of the tool?

Hi - I have already provided the examples in the readme.md on the mail page. I have also included test cases in the test folder. Is there something specific you are looking for?


Stream freezes when cycleCamera is called without delay multiple times

Describe the bug
Stream freezes when cycleCamera is called without delay multiple times
To Reproduce
Steps to reproduce the behavior:

Call cycleCamera twice in a row

Minimal reproducible example:
import android.Manifest
import android.content.pm.PackageManager
import android.os.Bundle
import android.view.ViewGroup
import android.widget.Toast
import androidx.activity.ComponentActivity
import androidx.activity.result.contract.ActivityResultContracts
import com.opentok.android.Publisher
import kotlinx.coroutines.Dispatchers
import kotlinx.coroutines.GlobalScope
import kotlinx.coroutines.Job
import kotlinx.coroutines.delay
import kotlinx.coroutines.launch

class CycleCameraProblem : ComponentActivity() {

    private val requestPermissionLauncher =
        registerForActivityResult(ActivityResultContracts.RequestPermission()) { isGranted: Boolean ->
            if (isGranted) {
                onCameraPermissionGranted()
            } else {
                Toast.makeText(this, "Cannot run without permission", Toast.LENGTH_LONG).show()
            }
        }

    private lateinit var publisher: Publisher
    private lateinit var container: ViewGroup

    private var job: Job? = null

    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        setContentView(R.layout.main)

        container = findViewById(android.R.id.content)
    }

    override fun onResume() {
        super.onResume()
        if (checkSelfPermission(Manifest.permission.CAMERA) == PackageManager.PERMISSION_GRANTED) {
            onCameraPermissionGranted()
        } else {
            requestPermissionLauncher.launch(Manifest.permission.CAMERA)
        }
    }

    override fun onPause() {
        super.onPause()
        job?.cancel()
        job = null
    }

    private fun onCameraPermissionGranted() {
        if (!::publisher.isInitialized) {

            publisher = Publisher.Builder(this).build()
            publisher.publishVideo = true

            container.addView(publisher.view)
        }

        runJob()
    }

    private fun runJob() {
        job = GlobalScope.launch(Dispatchers.Main) {
            delay(2000)

            publisher.cycleCamera()
            publisher.cycleCamera()
        }
    }
}

Expected behavior
Camera cycles multiple times
Screenshots
Video attached:
CycleCamera.webm
Device (please compete the following information):

sessionId, if applicable: N/A
Android SDK version: 34
OS and version: Emulator API 34 or Samsung S23 Ultra
com.opentok.android:opentok-android-sdk:2.27.1

Additional context
cycleCamera should either be synchronous, or a Future, so that the API users know when the process of switching camera is finished and is ready to be switched again. Note that Publisher.CameraListener.onCameraChanged is firing before cycleCamera is complete. It cannot be used as a reliable source of information about when cycleCamera is ready to be called again.

@Doctoror, I'm going to close this since you're already working with Support: https://jira.vonage.com/browse/VIDCS-2099.


[GOBBLIN-964] Add the enum JOB_SUCCEEDED to org.apache.gobblin.metrics.event.EventName

Dear Gobblin maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[X] My PR addresses the following Gobblin JIRA issues and references them in the PR title. For example, "[GOBBLIN-964]] My Gobblin PR"

https://issues.apache.org/jira/browse/GOBBLIN-964

Description

[X] Here are some details about my PR, including screenshots (if applicable):
In the notifyListeners method of AbstractJobLauncher, it is unable to convert the TimingEvent.LauncherTimings.JOB_SUCCEEDED successfully into a valid EventName with this line below

EventName.getEnumFromEventId(timerEventName)
We miss an enum for JobSucceededTimer in org.apache.gobblin.metrics.event.EventName
Without this enum, EventMetadataGenerator cannot work correctly to generate additionalMetadata for the event
Tests

[X] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Commits

[ ] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Codecov Report

Merging #2817 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master    #2817      +/-   ##
============================================
- Coverage     45.56%   45.56%   -0.01%
+ Complexity     8954     8953       -1
============================================
  Files          1900     1900
  Lines         71213    71214       +1
  Branches       7856     7856
============================================
- Hits          32451    32450       -1
+ Misses        35780    35778       -2
- Partials       2982     2986       +4

Impacted Files
Coverage Δ
Complexity Δ

...va/org/apache/gobblin/metrics/event/EventName.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...lin/util/filesystem/FileSystemInstrumentation.java
92.85% <0%> (-7.15%)
4% <0%> (ø)

...ava/org/apache/gobblin/fsm/FiniteStateMachine.java
73.48% <0%> (-3.04%)
18% <0%> (-3%)

.../apache/gobblin/runtime/api/JobExecutionState.java
79.43% <0%> (-0.94%)
24% <0%> (ø)

.../org/apache/gobblin/cluster/GobblinTaskRunner.java
65.27% <0%> (+0.46%)
28% <0%> (ø)
:arrow_down:

...main/java/org/apache/gobblin/util/HadoopUtils.java
30.87% <0%> (+0.67%)
25% <0%> (+1%)
:arrow_up:

...lin/restli/throttling/ZookeeperLeaderElection.java
72.22% <0%> (+2.22%)
13% <0%> (ø)
:arrow_down:

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 e24bc26...6522f30. Read the comment docs.


BUG: About Us page is navigating to the wrong page

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
When clicking on the About Us link, the website navigates to an incorrect page instead of the intended About Us page. This affects the user experience by not providing the expected information.
https://github.com/anuragverma108/SwapReads/assets/168854094/c81b37cc-cd38-4b4b-96fc-6769f75906b7
It must navigate to

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@huamanraj @anuragverma108 Sir please assign this issue to me.


Change generated Dockerfile base image

Purpose

Describe the problems, issues, or needs driving this feature/fix and include links to related issues in the following format: Resolves issue1, issue2, etc.

Goals

Describe the solutions that this feature/fix will introduce to resolve the problems described above

Approach

Describe how you are implementing the solutions. Include an animated GIF or screenshot if the change affects the UI (email<EMAIL_ADDRESS>to review all UI text). Include a link to a Markdown file or Google doc if the feature write-up is too long to paste here.

User stories

Summary of user stories addressed by this change>

Release note

Brief description of the new feature or bug fix as it will appear in the release notes

Documentation

Link(s) to product documentation that addresses the changes of this PR. If no doc impact, enter “N/A” plus brief explanation of why there’s no doc impact

Training

Link to the PR for changes to the training content in https://github.com/wso2/WSO2-Training, if applicable

Certification

Type “Sent” when you have provided new/updated certification questions, plus four answers for each question (correct answer highlighted in bold), based on this change. Certification questions/answers should be sent to<EMAIL_ADDRESS>and NOT pasted in this PR. If there is no impact on certification exams, type “N/A” and explain why.

Marketing

Link to drafts of marketing content that will describe and promote this feature, including product page changes, technical articles, blog posts, videos, etc., if applicable

Automation tests

Unit tests

Code coverage information

Integration tests

Details about the test cases and coverage

Security checks

Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines? yes/no
Ran FindSecurityBugs plugin and verified report? yes/no
Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets? yes/no

Samples

Provide high-level details about the samples related to this feature

Related PRs

List any other related PRs

Migrations (if applicable)

Describe migration steps and platforms on which migration has been tested

Test environment

List all JDK versions, operating systems, databases, and browser/versions on which this feature/fix was tested

Learning

Describe the research phase and any blog posts, patterns, libraries, or add-ons you used to solve the problem.

Codecov Report

Merging #247 into master will increase coverage by 0.06%.
The diff coverage is 66.66%.

@@             Coverage Diff              @@
##             master     #247      +/-   ##
============================================
+ Coverage     61.48%   61.55%   +0.06%
- Complexity      116      117       +1
============================================
  Files            13       13
  Lines           605      606       +1
  Branches         60       60
============================================
+ Hits            372      373       +1
- Misses          201      202       +1
+ Partials         32       31       -1

Impacted Files
Coverage Δ
Complexity Δ

...allerinax/docker/generator/DockerGenConstants.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...org/ballerinax/docker/utils/DockerPluginUtils.java
37.14% <0%> (-2.26%)
5 <0> (ø)

...allerinax/docker/generator/models/DockerModel.java
79.22% <100%> (-0.27%)
33 <2> (ø)

...erinax/docker/generator/DockerArtifactHandler.java
50% <100%> (+1.17%)
17 <0> (+1)
:arrow_up:

...g/ballerinax/docker/DockerAnnotationProcessor.java
75.65% <100%> (ø)
15 <0> (ø)
:arrow_down:

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 d937017...310980f. Read the comment docs.


Code Inspection: Tslint on python

await-promise

lib/feature/domain/pythonDependenciesFeature.ts:38: (error) Invalid 'await' of a non-Promise value.
lib/feature/reportersAgainst.ts:77: (error) Invalid 'await' of a non-Promise value.
lib/feature/spring/springBootVersionFeature.ts:47: (error) Invalid 'await' of a non-Promise value.
lib/setIdeals.ts:47: (error) Invalid 'await' of a non-Promise value.

max-line-length

lib/routes/features.ts:78: (warn) Exceeds maximum line length of 150
views/org.tsx:114: (warn) Exceeds maximum line length of 150

no-console

lib/analysis/offline/persist/PostgresProjectAnalysisResultStore.ts:191: (warn) Calls to 'console.log' are not allowed.
lib/analysis/offline/spider/github/GitHubSpider.ts:113: (warn) Calls to 'console.log' are not allowed.
lib/analysis/offline/spider/local/LocalSpider.ts:59: (warn) Calls to 'console.log' are not allowed.
lib/feature/domain/pythonDependenciesFeature.ts:59: (warn) Calls to 'console.log' are not allowed.
lib/feature/repoTree.ts:87: (warn) Calls to 'console.log' are not allowed.
lib/routes/orgPage.ts:206: (warn) Calls to 'console.log' are not allowed.
views/org.tsx:26: (warn) Calls to 'console.log' are not allowed.

no-duplicate-imports

lib/routes/features.ts:36: (error) Multiple imports from '@atomist/sdm-pack-fingerprints' can be combined into one.

no-empty-interface

views/sunburstQuery.tsx:3: (error) An empty interface is equivalent to {}.

no-floating-promises

lib/setIdeals.ts:51: (error) Promises must be handled appropriately

no-object-literal-type-assertion

lib/routes/orgPage.ts:208: (error) Type assertion on object literals is forbidden, use a type annotation instead.
test/spider/github/GithubSpider.test.ts:57: (error) Type assertion on object literals is forbidden, use a type annotation instead.

no-shadowed-variable

lib/feature/reportersAgainst.ts:89: (error) Shadowed name: 'fp'
lib/feature/reportersAgainst.ts:100: (error) Shadowed name: 'fp'
lib/feature/reportersAgainst.ts:100: (error) Shadowed name: 'fp'
lib/feature/reportersAgainst.ts:112: (error) Shadowed name: 'fp'
lib/feature/reportersAgainst.ts:112: (error) Shadowed name: 'fp'
lib/routes/api.ts:67: (error) Shadowed name: 'workspaceId'
lib/routes/api.ts:134: (error) Shadowed name: 'workspaceId'
lib/routes/features.ts:104: (error) Shadowed name: 'branchCount'
lib/routes/orgPage.ts:211: (error) Shadowed name: 'displayIdeal'

no-var-requires

lib/analysis/offline/persist/FileSystemProjectAnalysisResultStore.ts:35: (error) require statement not part of an import statement

radix

lib/routes/wellKnownReporters.ts:62: (error) Missing radix parameter
lib/routes/wellKnownReporters.ts:83: (error) Missing radix parameter

triple-equals

lib/element/codeOfConduct.ts:64: (error) == should be ===

typedef

lib/element/codeOwnership.ts:69: (error) expected member-variable-declaration: 'displayName' to have a typedef
lib/element/codeOwnership.ts:71: (error) expected member-variable-declaration: 'name' to have a typedef
lib/element/codeOwnership.ts:73: (error) expected call-signature: 'apply' to have a typedef
lib/element/codeOwnership.ts:79: (error) expected member-variable-declaration: 'fp' to have a typedef
lib/element/codeOwnership.ts:81: (error) expected call-signature: 'derive' to have a typedef
lib/element/codeOwnership.ts:91: (error) expected call-signature: 'relevanceTest' to have a typedef
lib/element/codeOwnership.ts:95: (error) expected call-signature: 'necessityTest' to have a typedef
lib/feature/DefaultFeatureManager.ts:74: (error) expected call-signature: 'features' to have a typedef
lib/feature/domain/TsLintFeature.ts:32: (error) expected member-variable-declaration: 'displayName' to have a typedef
lib/feature/domain/TsLintFeature.ts:34: (error) expected member-variable-declaration: 'name' to have a typedef
lib/feature/domain/TsLintFeature.ts:36: (error) expected call-signature: 'apply' to have a typedef
lib/feature/domain/TsLintFeature.ts:42: (error) expected member-variable-declaration: 'fp' to have a typedef
lib/feature/domain/TsLintFeature.ts:44: (error) expected member-variable-declaration: 'p' to have a typedef
lib/feature/domain/TsLintFeature.ts:61: (error) expected parameter: 'path' to have a typedef
lib/feature/domain/TypeScriptVersionFeature.ts:33: (error) expected member-variable-declaration: 'displayName' to have a typedef
lib/feature/domain/TypeScriptVersionFeature.ts:35: (error) expected member-variable-declaration: 'name' to have a typedef
lib/feature/domain/TypeScriptVersionFeature.ts:37: (error) expected call-signature: 'apply' to have a typedef
lib/feature/domain/TypeScriptVersionFeature.ts:43: (error) expected member-variable-declaration: 'fp' to have a typedef
lib/feature/domain/TypeScriptVersionFeature.ts:45: (error) expected call-signature: 'derive' to have a typedef
lib/feature/domain/TypeScriptVersionFeature.ts:59: (error) expected call-signature: 'toDisplayableFingerprintName' to have a typedef
lib/feature/repoTree.ts:40: (error) expected call-signature: 'without' to have a typedef
lib/feature/repoTree.ts:58: (error) expected call-signature: 'fingerprintsChildrenQuery' to have a typedef
lib/routes/api.ts:114: (error) expected call-signature: 'resolveFeatureNames' to have a typedef
lib/routes/orgPage.ts:256: (error) expected parameter: 'req' to have a typedef
lib/routes/wellKnownReporters.ts:153: (error) expected call-signature: 'mapping' to have a typedef
lib/setIdeals.ts:31: (error) expected call-signature: 'setIdeals' to have a typedef
lib/tree/partitioner.ts:45: (error) expected call-signature: 'rate' to have a typedef
test/spider/github/GithubSpider.test.ts:123: (error) expected call-signature to have a typedef
test/spider/github/GithubSpider.test.ts:123: (error) expected parameter: 't' to have a typedef
test/spider/github/GithubSpider.test.ts:123: (error) expected parameter: 'q' to have a typedef
test/spider/github/GithubSpider.test.ts:134: (error) expected call-signature to have a typedef
test/spider/github/GithubSpider.test.ts:134: (error) expected parameter: 't' to have a typedef
test/spider/github/GithubSpider.test.ts:134: (error) expected parameter: 'q' to have a typedef
test/spider/github/GithubSpider.test.ts:153: (error) expected call-signature to have a typedef
test/spider/github/GithubSpider.test.ts:153: (error) expected parameter: 't' to have a typedef
test/spider/github/GithubSpider.test.ts:153: (error) expected parameter: 'q' to have a typedef
test/spider/github/GithubSpider.test.ts:177: (error) expected call-signature to have a typedef
test/spider/github/GithubSpider.test.ts:177: (error) expected parameter: 't' to have a typedef
test/spider/github/GithubSpider.test.ts:177: (error) expected parameter: 'q' to have a typedef
test/tree/treeBuilder.test.ts:68: (error) expected call-signature: 'mapping' to have a typedef
test/tree/treeBuilder.test.ts:68: (error) expected parameter: 'ars' to have a typedef

[atomist:code-inspection:python=@atomist/atomist-sdm]

Issue closed because branch python was deleted.

Issue closed because branch python was deleted.


1348/partners listing search

Pull Request Template
Issue Overview
This PR addresses #1348

[x] This change addresses the issue in full

Description
This re-implements search on Detroit Partner's and refactoring it to utilize the new AG table and AG Pagination components.
How Can This Be Tested/Reviewed?
This PR can be tested by going to Detroit Partners, logging in, and typing in the search bar (at least 3 characters). The listings should automatically filter to match that string. Additionally, test all other existing partners table features. This includes sorting by column, ordering the columns, and logging out and logging back in to see your most recent table configuration.
Checklist:

[x] My code follows the style guidelines of this project
[x] I have added QA notes to the issue with applicable URLs
[ ] I have performed a self-review of my own code
[x] I have reviewed the changes in a desktop view
[ ] I have reviewed the changes in a mobile view
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules
[x] I have assigned reviewers
[ ] I have run yarn generate:client and/or created a migration if I made backend changes that require them
[ ] I have exported any new pieces added to ui-components
[ ] My commit message(s) is/are polished, and any breaking changes are indicated in the message and are well-described
[ ] Commits made across packages purposefully have the same commit message/version change, else are separated into different commits

Reviewer Notes:
Steps to review a PR:

Read and understand the issue, and ensure the author has added QA notes
Review the code itself from a style point of view
Pull the changes down locally and test that the acceptance criteria is met
Also review the acceptance criteria on the Netlify deploy preview (noting that these do not yet include any backend changes made in the PR)
Either explicitly ask a clarifying question, request changes, or approve the PR if there are small remaining changes but the PR is otherwise good to go

On Merge:
If you have one commit and message, squash. If you need each message to be applied, rebase and merge.

Good Catches @emilyjablonski! Those issues should be resolved now. Also, noticing that the centering issue is present in core as well so I can make those component updates once this PR is finalized.


Parquet Writer Support

Is your feature request related to a problem? Please describe.
The project team has received asks about Parquet support over the past few months. We don't have a need for Parquet internally, but if someone else is interested in adding it, then it's worth adding.
Describe the solution you'd like
This should be added in the Sink file wrapper. I'm not very opinionated about supporting custom options aside from custom compression settings.
Describe alternatives you've considered
N/A
Additional context
There are multiple implementations of Parquet in Go, but all have positives and negatives. parquet-go appears to have the most development velocity, but would be complex to implement (the code in Benthos is a good model).

Thanks @jshlbrd This is a very useful feature. Parquet and other columnar formats can increase query performance and efficiency once logs are loaded by query engines.

Closing this -- if anyone wants Parquet support, then they could try using AWS Glue to convert from JSON, otherwise a Python script would likely suffice. If support did exist, then Substation would likely produce many small files which can cause problems. We wouldn't want to add this without some large scale testing.


Invoke-WebRequest not responding to «Ctrc+c»

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
I am experiencing a problem with this comandlet.
I started it to downloading and turned off the router.  Comandlet hanged and hangs for more than 10 hours.
I think it should be breaked after I push «Ctrl+c», but it continue hanging.
Sorry my English.
Expected behavior
«Ctrl+c» – and Oll Korrect.

Actual behavior
Nothing changes.

Error details
I can not start the next command.

Environment data
Name                           Value
----                           -----
PSVersion                      7.1.4
PSEdition                      Core
GitCommitId                    7.1.4
OS                             Microsoft Windows 10.0.19043
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals
I prepared the window of PowerShell and reduced the speed of the router to minimal:

I pushed the «Enter»:

I turned off the router:

Now I am trying unsuccessfully to stop it.

I can reproduce: start downloading an ISO and then disconnect Wi-Fi - PowerShell hang.

If there is the VPN- or proxy-program between PowerShell and Internet, that program is returning the connection error and PowerShell by itself (without external intervention) is continue after 1-2 minutes with «$_.Exception.HResult -EQ -2146232800», «$_.Exception.InnerException.ErrorCode -EQ 10053» and «$_.Exception.InnerException.Message -Like "Unable to read data from the transport connection: *"».

Even though we have StopProcessing() implemented, because we have HttpClient.SendAsync() being run synchronously.. I think it is blocking the cancellation.
https://github.com/PowerShell/PowerShell/blob/7cf1ef8c8599a90cd9579c7bb7b7676e097c4938/src/Microsoft.PowerShell.Commands.Utility/commands/utility/WebCmdlet/Common/WebRequestPSCmdlet.Common.cs#L1358


Sarah Lean Profile Update

Updating my profile and contact details

Docs Build status updates of commit 94d9b52:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

advocates/sarah-lean.yml
:white_check_mark:Succeeded
View

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


fix usagestats bar sizing, unspent calculation

See #727. This PR fixes an edge case where the spending bars could overflow their container, and changes Sia-UI to use the API's unspent instead of doing its own unspent calculation.

LGTM but Travis is failing

Should be all set now.


Dynamic DNS response control

Please describe your feature request:

https://github.com/projectdiscovery/interactsh/issues/364#issuecomment-1223680939
Describe the use case of this feature:

DNS Rebind, Easy SSRF like https://nip.io/ usage ...

This DNS Rebinder defines a nice syntax that could be implemented

It would be very nice to integrate DNS rebinder into interact.sh. Currently interactsh and DNS rebinder can't run on the same host / interface due to the DNS port 53 collision.


Add false positive negative command

add false positive and false negative command
deprecate abnormal-security-submit-inquiry-to-request-a-report-on-misjudgement
change image

LGTM - do you mind explaining how the is able to resolve the url path (ex. /detection360/reports)

hmm the client has the base_url which is https://api.abnormalplatform.com so we just add on /detection360/reports to the base url to get the path we want


powertrain_binary and powertrain_thermo failures

ALIGN throws assertion for powertrain_binary and powertrain_thermo examples. Reproducible on master branch.

@soneryaldiz
The terminal of the module is illegal. The terminal name here is:
module powertrain_binary ( on_d[0], on_d[1], on_d[2], on_d[3], on_d[4], on_d[5], vout );
on_d[0], '[' is illegal here.
Yaguang

@stevenmburns
Hi Steve,
In Soner's case, an illegal terminal name appears in the verilog file. Should we make that illegal name illegal or just change the terminal name to other illegal name?
module powertrain_binary ( on_d[0], on_d[1], on_d[2], on_d[3], on_d[4], on_d[5], vout );
on_d[0], '[' is illegal here.
Yaguang

@stevenmburns
Sorry for the typo. I mean, should we make that illegal name legal or just change the terminal name to other legal name?
Yaguang

This is easy to fix in the Verilog parser. I’ll add it to the PR.
From: Yaguang Li<EMAIL_ADDRESS>Sent: Wednesday, April 22, 2020 6:07 PM
To: ALIGN-analoglayout/ALIGN-public<EMAIL_ADDRESS>Cc: Burns, Steven M<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [ALIGN-analoglayout/ALIGN-public] powertrain_binary and powertrain_thermo failures (#423)
@stevenmburnshttps://github.com/stevenmburns
Sorry for the typo. I mean, should we make that illegal name legal or just change the terminal name to other legal name?
Yaguang
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/ALIGN-analoglayout/ALIGN-public/issues/423#issuecomment-618117779, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AALRI7X2KB5S6XS4NHNPUHDRN6IC7ANCNFSM4MOSEQ5Q.

@stevenmburns
Thanks!
Yaguang


The 'md' 'lg' properties do not work in the Col component

I made an update of the module to the last version, and the properties md and lg do not work in the Col component.
<Col xs={12} sm={4} md={4} lg={2}>
 <ItemList {...process} />
</Col>

The problem seems to be affected by the .sort() function in this commit; https://github.com/LoicMahieu/react-styled-flexboxgrid/commit/7089c3f421a12ab42b5408b921435c157272f76b
I can remove this function but no idea why this sort was added, any ideas?

Thanks for reporting!
Sort has been introduced for fix #44 . It would like that prop order does not impact rendering. I appears that dimensions iteration was from lower to higher instead of the inverse.
PR #56 should fix the problem.

<EMAIL_ADDRESS>

Still not working.. Can you confirm this?

Not working, too

Indeed... Brokes my project too... Grrr... Working on it

<EMAIL_ADDRESS>
Sorting wasn't correct. Now it should be ok.

It works correctly in my project, thanks!


feat(plugins/plugin-client-common): top-level tabs are not allowed to be closed or added in a readonly client

Fixes #7746

Description of what you did:

My PR is a:

[ ] 💥 Breaking change
[ ] 🐛 Bug fix
[ ] 💅 Enhancement
[x] 🚀 New feature

Please confirm that your PR fulfills these requirements

[x] Multiple commits are squashed into one commit.
[x] The commit message follows Conventional Commits, which allows us to autogenerate release notes; e.g. fix(plugins/plugin-k8s): fixed annoying bugs
[x] All npm dependencies are pinned.

/lgtm


Clicking on Repo icon shows "There is no active source control providers"

Issue Type: Bug
Have VS code installed and got the latest version update. Was trying it but when I click on repo icon don't get any source control providers. Though the documentation says that Git is built-in. Not able to clone a repository. Yes, when I open a file which is part of a Git repo, SCM comes automartically but that's looks real buggy to me.
VS Code version: Code 1.23.0 (7c7da59c2333a1306c41e6e7b68d7f0caa7b3d45, 2018-05-03T16:44:55.614Z)
OS version: Windows_NT x64 10.0.16299

Ahh!! finally...
Did gone through few existing post/bug like below but didn't helped. But finally, I created a workspace, saved it and then when I try, could see the git icon and can as well clone a repo. But honestly, this is no where mentioned in documentation clearly. So it still feels like a real buggy application.
https://github.com/Microsoft/vscode/issues/35741
https://stackoverflow.com/questions/46609255/git-missing-in-vs-code-no-source-control-providers/46610954#46610954
** This bug can be closed now.


srv broken on official servers

To reproduce:
use srv to log in
Observed behaviour:
[00:17:15][WARN] screeps_api::websocket::parsing: unparsed data in screeps typed channel update: 1.decorations
[00:17:15][ERROR] srv::net: Error occurred: handling room update for shard2:E56N7: creating 5f014d486dee62141ea49979 with data {"_id":"5f014d486dee62141ea49979","hits":215000,"hitsMax":250000,"nextDecayTime":25859020,"notifyWhenAttacked":true,"room":"E56N7","store":{"energy":430},"storeCapacity":2000,"type":"container","x":26,"y":16}: missing field `energyCapacity` (Context { context: "handling room update for shard2:E56N7", source: Context { context: "creating 5f014d486dee62141ea49979 with data {\"_id\":\"5f014d486dee62141ea49979\",\"hits\":215000,\"hitsMax\":250000,\"nextDecayTime\":25859020,\"notifyWhenAttacked\":true,\"room\":\"E56N7\",\"store\":{\"energy\":430},\"storeCapacity\":2000,\"type\":\"container\",\"x\":26,\"y\":16}", source: Error("missing field `energyCapacity`", line: 0, column: 0) } })
[00:17:15][ERROR] panic: thread 'unnamed' panicked at 'Error occurred: handling room update for shard2:E56N7: creating 5f014d486dee62141ea49979 with data {"_id":"5f014d486dee62141ea49979","hits":215000,"hitsMax":250000,"nextDecayTime":25859020,"notifyWhenAttacked":true,"room":"E56N7","store":{"energy":430},"storeCapacity":2000,"type":"container","x":26,"y":16}: missing field `energyCapacity` (Context { context: "handling room update for shard2:E56N7", source: Context { context: "creating 5f014d486dee62141ea49979 with data {\"_id\":\"5f014d486dee62141ea49979\",\"hits\":215000,\"hitsMax\":250000,\"nextDecayTime\":25859020,\"notifyWhenAttacked\":true,\"room\":\"E56N7\",\"store\":{\"energy\":430},\"storeCapacity\":2000,\"type\":\"container\",\"x\":26,\"y\":16}", source: Error("missing field `energyCapacity`", line: 0, column: 0) } })': src/net.rs:112
[00:17:23][ERROR] panic: thread 'main' panicked at 'already borrowed: BorrowMutError': /rustc/94d346360da50f159e0dc777dc9bc3c5b6b51a00/src/libcore/cell.rs:878

Also, there are some build issues, which I will file a PR to partially fix.

This looks like something that will need to be fixed in rust-screeps-api.
I think I have an old in-progress update on my hard disk which I started a while ago but never finished. I've got some free time, so I'd be down to either see if I can finish that today/tomorrow, or otherwise I'd upload the code if you want to see what you can do with it.
Would you be interested in fixing/updating screeps-game-api?

I had a lot of trouble trying to understand why some of the very strange compiler errors were happening given an intersection of wrong dependency versions and use of old preview functionality on a new compiler. I can poke at it I guess- depends on how much time I have in the near future.

Alright
I've got some experience upgrading/dealing with the weird future versions, and I know how annoying it can be to work with the old versions when all the docs point to the newer ones, so I can at least try to resolve those errors today. We'll see how it goes from there, I guess?
If you find time and end up inspired, fixing the login errors in rust-screeps-api shouldn't be too complicated. The logging is set up to complain about exactly what it finds wrong, and the solution usually ends up being either adding new fields to a few structs in rust-screeps-api, or making existing fields optional, and then just copying the JSON data from the logged warning/error in to make a new parsing test.

Alright - I've updated screeps-game-api and srv to the latest futures, as well as associated crates (tokio, hyper, etc.), and I've pushed all of the in-progress protocol updates to screeps-game-api.
There are still outdated protocol structs in screeps-game-api, so it isn't fully working. I probably won't be fixing those anytime soon. However, all the dependency issues should be resolved.
I've also written up some notes on doing the protocol updates at https://github.com/daboross/rust-screeps-api/blob/master/CONTRIBUTING.md, if you're interested in getting it fully working.


Better Bones mod

you might want to have a look at this Bones Redo cos I was very happy with it.
Bone generation 99% guaranteed, fits all your items, waypoints.
I love it.
https://github.com/OgelGames/bones/tree/master

Currently evaluating on the test server.

Installed.


Deployment steps to heroku

Steps to deploy rendertron to heroku

Sorry, there was another PR to document Heroku already.
Please check the pending PRs and consider commenting on issues to get them assigned to you.


Queries extremely slow with mysql 5.6.43

I have not been able to find the root cause yet but I noticed when running the same queries against mysql 5.6.43 they run extremely slowly for even basic queries. However when run against mysql 8 there are no issues. Both instances of mysql are being run inside docker containers. I am using sqlx 0.3.4 with the tokio runtime.
I am running queries like the following
                let query = "select * from users where email = ? and status != 'DELETED'  and DELETED_AT = '1000-01-01 00:00:00' limit 1;";
                trace!("checking for duplicate emails");
                 match sqlx::query_as::<sqlx::MySql,User>(&query).bind(email).fetch_one(mysql).await {
                     Ok(_u) => {
                    trace!("user email {} is already present in the database", email);
                    Ok(true)
                    },
                    _ => Ok(false)

Note that mysql variable I am passing to fetch is a MySqlConnectionPool and that the query being called is using an index on the table
I noticed some performance improvements when using fetch instead of cursors. When I used cursors I noticed a lot of sleeping connections in mysql but queries were not being executed.
I have a few theories about the issue but would appreciate guidance or recommendations from others.

Step 1 is narrowing this down to a tiny program we can run to test against when profiling or debugging the database.
I'd like to see the execution time of the mysql command line for both versions compared to the SQLx execution time of both versions.

I will try to post a simple example soon, as far as timings my integration test suit runs in 19 seconds on mysql 8 and 28 minutes on 5.6.43 the schema and codebase were not changed between runs. Not sure what would account for such a massive change. We will need to run on 5.6 in production and upgrading the shared production database is most likely not on the table

That's a bit ridiculous of a difference. Unfortunately we've got two vectors here. Either SQLx itself is slower on 5.6 (totally possible, I test on 8 a lot more locally) or 5.6 is just that much slower for that query (still possible). To rule out the latter is why I want to see timings using the mysql command line.

One thing to note is MySQL 5.6 sends (mostly useless) EOF packets (that have since been deprecated and are no longer used in v8) after several operations which is a fairly significant increase to bandwidth.
Another thought is you could be experiencing network latency here as well if we're talking a whole test suite and not just one query. A dozen queries is going to execute significantly slower against a remote database.

I don't think mysql is self is the issue
running:
select * from users where email =<EMAIL_ADDRESS>and status != 'DELETED'  and DELETED_AT = '1000-01-01 00:00:00' limit 1;

mysql 5.6.43
| Duration |
+----------------------+----------+
| starting             | 0.000111 |
| checking permissions | 0.000008 |
| Opening tables       | 0.000020 |
| init                 | 0.000039 |
| System lock          | 0.000007 |
| optimizing           | 0.000016 |
| statistics           | 0.000164 |
| preparing            | 0.000015 |
| executing            | 0.000002 |
| Sending data         | 0.000032 |
| end                  | 0.000003 |
| query end            | 0.000006 |
| closing tables       | 0.000008 |
| freeing items        | 0.000026 |
| cleaning up          | 0.000015 |
+----------------------+----------+
mysql 8
+--------------------------------+----------+
| Status                         | Duration |
+--------------------------------+----------+
| starting                       | 0.000126 |
| Executing hook on transaction  | 0.000007 |
| starting                       | 0.000011 |
| checking permissions           | 0.000009 |
| Opening tables                 | 0.000067 |
| init                           | 0.000011 |
| System lock                    | 0.000015 |
| optimizing                     | 0.000026 |
| statistics                     | 0.003186 |
| preparing                      | 0.000034 |
| executing                      | 0.000051 |
| end                            | 0.000006 |
| query end                      | 0.000005 |
| waiting for handler commit     | 0.000011 |
| closing tables                 | 0.000010 |
| freeing items                  | 0.000030 |
| cleaning up                    | 0.000037 |
+--------------------------------+----------+
Also I am running both locally via docker-compose so I don't think latency is the issue

Here is a simplified version that shows the issue, running this on mysql 8 works on 5.6.43 it hangs the connection since I set the pool size to 1:
#[allow(unused_imports)]
use log::{debug, info, trace};
use chrono::{DateTime, Utc};
use sqlx::mysql::MySqlQueryAs;
use std::time::Duration;

#[derive(sqlx::FromRow)]
struct User {
    id: u64,
    email: Option<String>,
    deleted_at: Option<DateTime<Utc>>,
}

pub async fn email_is_duplicate(test_email: Option<&str>, mysql: &sqlx::MySqlPool) -> Result<bool, sqlx::Error> {
match test_email {
    Some(email) => {
        let query = "select user_id as id, email, deleted_at from users where email = ? and DELETED_AT = '1000-01-01 00:00:00' limit 1;";
        trace!("checking for duplicate emails");
            match sqlx::query_as::<sqlx::MySql,User>(&query).bind(email).fetch_one(mysql).await {
                Ok(_u) => {
            trace!("user email {} is already present in the database", email);
            Ok(true)
            },
            _ => Ok(false)
            }
    }
    None => Ok(false),
}
}

#[tokio::main]
async fn main() -> Result<(), sqlx::Error> {
    env_logger::init();
    let user = User {
        id:<PHONE_NUMBER>4,
        email: Some("someone@somewhere.com".to_string()),
        deleted_at: None
    };

    let database_url = "mysql://USERNAME:PASSWORD@HOST:PORT/DATABASE";
    let pool = sqlx::MySqlPool::builder()
            .max_size(1)
            .min_size(1)
            .connect_timeout(Duration::from_secs(30))
            .build(database_url).await?;

    email_is_duplicate(user.email.as_ref().map(|e| &**e), &pool).await?;
    email_is_duplicate(user.email.as_ref().map(|e| &**e), &pool).await?;
    email_is_duplicate(user.email.as_ref().map(|e| &**e), &pool).await?;
    email_is_duplicate(user.email.as_ref().map(|e| &**e), &pool).await?;
    email_is_duplicate(user.email.as_ref().map(|e| &**e), &pool).await?;
    email_is_duplicate(user.email.as_ref().map(|e| &**e), &pool).await?;
    email_is_duplicate(user.email.as_ref().map(|e| &**e), &pool).await?;
    email_is_duplicate(user.email.as_ref().map(|e| &**e), &pool).await?;
    email_is_duplicate(user.email.as_ref().map(|e| &**e), &pool).await?;
    email_is_duplicate(user.email.as_ref().map(|e| &**e), &pool).await?;
    Ok(())
}

I also used a database schema that only has the users table with 3 fields and an index on id , email + deleted_at

@kmyers608 for completeness, what are the types of the columns on users?

+--------------------------+----------------------------------------------------------+------+-----+---------------------+----------------+
| Field                    | Type                                                     | Null | Key | Default             | Extra          |
+--------------------------+----------------------------------------------------------+------+-----+---------------------+----------------+
| id                  | bigint(20) unsigned                                      | NO   | PRI | NULL                | auto_increment |
| email                    | varchar(255)                                             | YES  | MUL | NULL                |                |
| deleted_at               | datetime                                                 | NO   |     | 1000-01-01 00:00:00 |                |
+--------------------------+----------------------------------------------------------+------+-----+---------------------+----------------+

I have been tracing the issue through this crate it seems that queries after the first hang at
sqlx-core/src/mysql/stream.rs in the read fn at line 139:
    let header_enum = self.stream.peek(4_usize).await;

if I set test_on_acquire to false in the pool, otherwise it hangs on ping but the connection is still open but sleeping according to sql:
| 18 | USER | IP | DATABASE | Sleep   |  826 |       | NULL                  |
I am omitting the specifics since this is not an open source project

Thank you :heart: for the reproduction. I'll dig into this tonight (PST).

I was unable to debug the issue further so we moved to the mysql_async crate instead. You are welcome to close this issue if you want.

Sorry we didn't circle back to this quickly. If you don't mind, I'd like to keep this open until we figure this out.

Please try master if you can. I get very good performance on MySQL 5.6.x with the posted query.


psw: 0.1.2 -> 0.2.0

Motivation for this change
Update psw to latest version.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109593 run on x86_64-linux 1

1 package built:

psw

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109593 run on x86_64-linux 1

1 package built:

psw


Rename package to co.ello.ElloApp

ello.co.ello was not a good package name 😏

ping @jayzes

LGTM!  Though I would try it without the co.Ello.ElloApp. prefix on R.

Good catch @colinta, I hadn't noticed that those had the package prefix. Removed now.

PR reviews FTW!

:metal:


cdc panic when kafka sink rolling restart

What did you do?

TiDB cluster with 3 CDC deployed (32C 64G each)
There are 2 kafka changefeed, one for single big table, the other for 4k small tables. the lag is normal for both changefeed
rolling restart the kafka sink (3 instances)

What did you expect to see?
cdc should not panic
What did you see instead?
cdc panic seen
[root@bogon bigCluster]# kubectl  --kubeconfig kubeconfig.yml -n cdc-kafka-big-cluster-tps-1712340-1-428 logs -p tc-ticdc-2
[WARN] TiCDC server data-dir is not set. Please use `cdc server --data-dir` to start the cdc server if possible.
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x10 pc=0x12b2609]

goroutine 74332 [running]:
github.com/Shopify/sarama.(*partitionProducer).newHighWatermark(0xc0c18b9140, 0x3)
        github.com/Shopify/sarama@v1.36.0/async_producer.go:620 +0x1a9
github.com/Shopify/sarama.(*partitionProducer).dispatch(0xc0c18b9140)
        github.com/Shopify/sarama@v1.36.0/async_producer.go:564 +0x537
github.com/Shopify/sarama.withRecover(0xc1ea43e580?)
        github.com/Shopify/sarama@v1.36.0/utils.go:43 +0x3e
created by github.com/Shopify/sarama.(*asyncProducer).newPartitionProducer
        github.com/Shopify/sarama@v1.36.0/async_producer.go:515 +0x1ea

Versions of the cluster
cdc version: https://github.com/pingcap/tiflow/pull/9010
Release Version: v7.1.0
Git Commit Hash: 9b1497c7fba1d290443011f1d7d1e4305a125e1d
Git Branch: heads/refs/tags/v7.1.0
UTC Build Time: 2023-05-22 10:16:54
Go Version: go version go1.20.3 linux/amd64
Failpoint Build: false

This issue is a sarama bug: https://github.com/Shopify/sarama/issues/2322
It was fixed by: https://github.com/Shopify/sarama/commit/237925756e46ac948e583fceb81e8738c9f1e04b
We need to bump cdc's sarama dependency to 1.38.1 to avoid this issue.

/severity critical

/remove-label affects-6.1


Add line breaks to instructions  (#40425)

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Closes #40425

Congrats @ianturner88 on your first pull request (PR)! 🎉
Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 📝

Congrats @ianturner88 on your first pull request (PR)! 🎉
Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 📝


Swift compatibility

Hi, I'm trying to use this library on swift but it has been nearly impossible due to the name of the delegate method. Basically, any delegate method name beginning in "DKScrollingTabController: ..." will mess the type "DKScrollingTabController" leaving it unusable even for the controller parameter in the delegate method.
Also fyi: if anyone else is trying to implement this library in swift you should follow this order
1- Visual configuration of the controller
2- Adding the subview to the parent
3- Assign the frame
4- Assign tab items
Any other orders (at least the ones I've tried) had visual issues when the view is displayed

:confused:
so you were able to use it or no? will take a look

I looked into this, the DKScrollingTabControllerDelegate method name cannot start with DKScrollingTabController.. will update the library


[Question]: Chat assistant not retrieving any chunks

Describe your problem
I have added a document to a new KB and chucked it using the 'General' method. When I do retrieval testing on the KB, it retrieves the correct chucks. I have also created a chat assistant on my knowledge base, when I ask the same question as I did during retrieval testing it returns no results. I have tried all different types of configuration on the assistant but it doesn't seem to work. Is there any way I can see what the query the assistant is sending?

I think it's the thing about LLM lack of capability to under stand the relevance between your question and the retrievals.
There's a lamp up on the assistant output. You could click it to see the prompt send to LLM.

I see, thank you very much.


Make template for thank-you page

Make the page more informative, for example:

display some Event fields
Facebook widget
add link 'back to Events' / 'back to main page'

Made the template and now it shows the reference number. Great.
Won't close the issue before implementing some more information to the template (to suffice osakunta/django-sivusto project's ad-hoc needs)

Has the thank-you message of the event and payment details. Sufficent ad hoc so closing.


MediaStream.stop() deprecated function fix

Once MediaStream.stop() function was deprecated, the code was fixed to work in Chrome 47+.


Make equivalent default feature

I think this is how it is meant to be:
https://github.com/rust-lang/hashbrown/blob/e25e6bb02e4fe4d58835f525d60f86091f41d50f/src/lib.rs#L140-L142
CC @cuviper

@bors r+

:pushpin: Commit f93727d9f3bbb16590c2eb9f81ec6f92a17e550d has been approved by Amanieu
It is now in the queue for this repository.

:hourglass: Testing commit f93727d9f3bbb16590c2eb9f81ec6f92a17e550d with merge a34158ca5528a72668b8ca116c11ccda7df0b9fa...


[Question] How can selenide switch to the new window or new tab in browser?

As title, if a new window appear after click a web element ? How can I switch to the new window or new tab in browser?

You can do it multiple ways:
import static com.codeborne.selenide.Selenide.*;

// by index
switchTo().window(1);

// by name or title or handle
switchTo().window("Title of the window");

// for switching to default window
switchTo().defaultContent();

@anilreddy Thank for the answer!
@yisake exactly, use switchTo() method.

@anilreddy Many thanks.
@asolntsev I think many people like me are new to selenide need more example with  many detail for selenide. ~~

@yisake Absolutely agree! Good point!
Don't you want to help us?
Source of selenide.org is also in github.com/codeborne/selenide, but in branch "gh-pages".
Feel free to send pull requests there!

just wanted to say that when I started to use Selenide i had the same problem. When I found out that there are lots of examples at https://github.com/codeborne/selenide/tree/master/src/test/java/integration testing, everything  became much easier. Please look at those examples if you are not doing it already

@thasherwin Good point, thank you!

Hi, Andrei (@asolntsev)!
I have just start to use this very helpful library and really love it, Thank you very much for sharing it.
I have seen your videos in youtube about using the library and read most of main pages at project website too. I read and try to use eclipse IDE "code proposals" to start creating tests and realise that to use them they ALL should to be to imported to the test (java) file. Being a newbie I should to ask a question. Is there a selenide libraries FULL list to be imported in the tests or there is another better way to get code (suggestions) proposals?
List should look like this:
import static com.codeborne.selenide.Selenide.;
import static com.codeborne.selenide.CollectionCondition.size;
import static com.codeborne.selenide.Condition.appear;
import static com.codeborne.selenide.Condition.appears;
import static com.codeborne.selenide.Condition.visible;
import static com.codeborne.selenide.Condition.attribute;
import static com.codeborne.selenide.Condition.cssClass;
import static com.codeborne.selenide.Condition.hidden;
import static com.codeborne.selenide.Condition.disappear;
import static com.codeborne.selenide.Condition.disappears;
import static com.codeborne.selenide.Condition.text;
import static com.codeborne.selenide.Condition.value;
import static com.codeborne.selenide.Condition.exist;
import static com.codeborne.selenide.Selenide.$;
import static com.codeborne.selenide.Selenide.$$;
import static com.codeborne.selenide.Selenide.open;
import static com.codeborne.selenide.Selenide.screenshot;
import static com.codeborne.selenide.Selectors.;
import static com.codeborne.selenide.WebDriverRunner.;
import com.codeborne.selenide.WebDriverRunner;
import com.codeborne.selenide.;
Thank you for an answer in advance.
P.S. I saw the selenium 3.2 was published out there. Selenide maven repository uses the lower version 3.0.1 right now.

@Cotola you can use like above or can use like this:
// here '*' imports all static methods in that class
import static com.codeborne.selenide.Selenide.*;

import static com.codeborne.selenide.Condition.*;

Thank you @anilreddy!
Sorry in my comments asterisk was not shown properly.
These two lines you mention don't cover com.codeborne.selenide.CollectionCondition.size method.

@Cotola I have just gave you 2 examples.

Thank you!
I see that I need more Java core and eclipse knowledge. :)

i can't seem to switch to an angular MatDialog, does someone know how?

@NickMeert-dev can you share the example so that I can help you with that.


Setting Typescript types for returned duck-typed objects.

Motivation
As far as I understand, the only way to return an object with a duck-typed interface from Rust to JS for the time being is to build it by reflection and return it as a JsValue. The generated Typescript definition rightfully gives the returned type as any, which does its job, but it fails to convey the author's intentions. A better solution would be to add a way to declare a Typescript type for a returned JsValue.
Proposed Solution
The proposed solution would be to add a wasm_bindgen attribute, perhaps called something like return_typescript_type, containing a Typescript type. Any named references contained in the type must be defined elsewhere in a typescript_custom_section, although parsing the TS to ensure that might not be a priority.
#[wasm_bindgen(js_name = jpegImageSize, return_typescript_type = {width: number, height: number}]
pub fn jpeg_image_size(image: &[u8]) -> JsValue {
   let (width, height): (u32, u32) = unimplemented!();
   let ret = Object::new();
   Reflect::set(&ret, &JsValue::from_str("width"), &JsValue::from_f64(width as f64);
   Reflect::set(&ret, &JsValue::from_str("height"), &JsValue::from_f64(height as f64);
   ret
}

Alternatives
Hopefully sometime in the future, wasm_bindgen will offer a way to return duck-typed interfaces from Rust without reflection, which would provide the necessary types for the TS declaration to be reasonably complete by default.

Seems like a reasonable feature to me!

Currently you can just return a Rust struct, like so:
#[wasm_bindgen]
pub struct Foo {
    pub width: f64,
    pub height: f64,
}

#[wasm_bindgen]
pub fn jpeg_image_size(image: &[u8]) -> Foo {
   let (width, height): (u32, u32) = unimplemented!();
   Foo { width, height }
}

This also generates correct TypeScript types. The only issue is it creates a JS class, so I think it would be cool to have a plain_object or similar attribute that would cause it to instead generate a plain JS object:
#[wasm_bindgen(plain_object)]
pub struct Foo {
    pub width: f64,
    pub height: f64,
}

(Obviously methods wouldn't work on it, and there's some subtlety around free)

Any progress?

Any progress?

I'd moved on from this long ago, following @Pauan's suggestion I believe. I'd tackle this now that you've reminded me, but I'm not familiar with the wasm-bindgen code or have a lot of time to invest in it.

@andreubotella Thanks for ur advice, this is a good entry for me to contribute to wasm-bindgen, I'll try to handle this.

Since the plain-object won't work with Vecs, I'd rather having something like
struct Bar { ... }
struct Foo {
  bars: Vec<Bar>
}
#[wasm_bindgen]
fn foo() -> JsValue<Foo> {
  //
}

interface Bar {
 ...
}
interface Foo {
  bars: Array<Bar>
}

For me, the first-prize solution would be for us to be able to declare the a struct can be marshalled to Javascript in one direction, only.
For example:
#[wasm_bindgen(serialize-only)]
struct DataPack {
   anumber: i32,
   anarray: Vec<i32>,
   aboxedslice: Box<[(i32, i32, i32)]>,
}

This would have the same effect, conceptually, as making the struct serializable with serde and marshalling it via JsValue (preferably via the serde-wasm-bindgen crate because that's cheaper) except that it would allow wasm-bindgen to do all the leg work for us and allow us to use readable types in our methods:
   pub fn get_the_data(&self) -> DataPack {
   ...
   }

Additional benefits:

No need to worry about arrays, vectors or collections because anything iterable can be marshalled to a plain array if one-way marshalling can be assumed.
No need to worry about anything that isn't state because there's no way to call a method on something that marshals one way.
Covers 100% of the friction that I, personally, experience with wasm-bindgen.

For the reverse case, using an extern to expose a Javascript type to Rust is a tonne easier than this serde -> JsValue time-sink.
My project runs on Rust in the background, providing a TypeScript U.I. This means that 99% of my marshalling needs are one-way: TypeScript needs to read something in order to draw it or update a control.

wasm-bindgen can automatically create strong TypeScript definitions for the types.
No more JsValue in signatures.

I appreciate @clearloop's contribution to this issue. The problem, however, is that we still cannot use #[wasm_bindgen(typescript_type = "...")] for structs that use features not allowed by wasm-bindgen, like the one mentioned in https://github.com/rustwasm/wasm-bindgen/issues/2407:
#[wasm_bindgen(typescript_custom_section)]
const TS_APPEND_CONTENT: &'static str = r#"
type DatasourceDbURL
  = { static: string }
  | { env: string }
"#;

#[wasm_bindgen(typescript_type = "DatasourceDb")]
pub enum DatasourceDbURL {
    Static(String),
//  ^^^^^^^^^^^^^^
//  only C-Style enums allowed with #[wasm_bindgen]

    Env(String),
}

I'd like to return a Set

A decent workaround to this is to skip automatically emitting the type and add it manually. This is error-prone but not too much boilerplate.
// Define TypeScript interfaces for your serializable types.
#[wasm_bindgen(typescript_custom_section)]
const TYPES: &str = r###"
	interface Position {
		x: number,
		y: number,
	}
"###;

#[wasm_bindgen]
pub struct Step(...);

#[wasm_bindgen]
impl Step {
	// Apply skip_typescript to the methods that return JsValue.
	#[wasm_bindgen(skip_typescript)]
	pub fn trace(&mut self) -> JsValue {
		serde_wasm_bindgen::to_value(&self.0.trace()).unwrap()
	}
}

// Manually add the method to the interface.
#[wasm_bindgen(typescript_custom_section)]
const STEP_TYPES: &str = r###"
	export interface Step {
		trace(): Position[];
	}
"###;

Hopefully we can get proper support for emitting proper types for these "JSON" values soon.


mtrand error: 'NoneType' object cannot be interpreted as an index

hey, hoping someone can help me :)
After this code:
python gen_w2v.py /home/ba/MatchZoo/data/toy_example/ranking/word_dict.txt /home/ba/MatchZoo/data/toy_example/ranking/glove.840B.300d.txt /home/ba/MatchZoo/data/toy_example/ranking/embed_glove_d300
I am getting this error:
`2196017it [9:41:20, 62.96it/s]
load word vectors ...
Loading vectors from /home/ba/MatchZoo/data/toy_example/ranking/word_dict.txt
100%|████████████████████████████████████████████████████████████████| 212644/212644 [00:00<00:00, 1206949.69it/s]
Traceback (most recent call last):
File "gen_w2v.py", line 129, in
embeddings = load_word_embedding(word_dict, w2v_file)
File "gen_w2v.py", line 90, in load_word_embedding
curr_embed = (2.0 * np.random.random_sample([dim]) - 1.0) * alpha
File "mtrand.pyx", line 861, in mtrand.RandomState.random_sample
File "mtrand.pyx", line 167, in mtrand.cont0_array
TypeError: 'NoneType' object cannot be interpreted as an index`
Word Vectors were successfully loaded before, then this error appears when trying to load the word dict. I tryed to reinstall numpy and some other packages but nothing helped.
The word_dict file was produced by the preprocess.py and looks like this (Its just a little part of the Dict, it has 212644 lines):

extrastress 190601
aceton 13407
autosom 5845
chemoradiotherapeut 14713
polarprob 71802
93,134 118825
opn/ca-125 203195
procavia 160607
pmit+ 135356
143,991 204032
noxious 20925
choedochoscop 163663
hhai. 203721
edelfosin 160238

Thanks for any help :)

The order of parameters for gen_w2v.py is mistaken. For you case, it should be like this:
"python gen_w2v.py /home/ba/MatchZoo/data/toy_example/ranking/glove.840B.300d.txt  /home/ba/MatchZoo/data/toy_example/ranking/word_dict.txt /home/ba/MatchZoo/data/toy_example/ranking/embed_glove_d300".

Thanks for the help, that fixed it i guess. Now im getting

Killed

After nothing happening for a few minutes. Any suggestion on that?

Kern.log File says: "Out of memory: Kill process ....".
I got 12gb Ram, should this not be enough? Or is there any way around this Error?
Thanks in advance

This happened in generating the word embedding map file ? How many words in you word_dict.txt?

Yes, 212644 words

You can try to reduce the vocabulary size. Specifically, you can remove low frequency words.

Even getting killed when only using a size of 10 words, and i looked at the memory monitor wich fills up to 100% and shortly after the killed happens

Strange

So your memory is not sufficient to load embedding

So how much memory do you guys have to load the glove.840B.300d.zip vectors?

Thanks for your question, I'll close it for now. If you have further questions, please feel free to re-open it or create a new issue. Best.
We'll think a smarter way to load embeddings in the next version, thanks!


Powertoys settings won't open in Windows 11 Preview Build 22621.1

Microsoft PowerToys version
"0.58.0"
Running as admin

[ ] Yes

Area(s) with issue?
Settings
Steps to reproduce

Upgrade to Windows insider Preview 22621.1
Try to right click on tray icon to open settings, or run PowerToys from start menu.
No settings window appears.

✔ Expected Behavior
The Settings Window to appear
❌ Actual Behavior
No Settings Window appeared
Other Software
No response

BTW. 0.57 works, but of course it prompts me to upgrade. When I upgrade 0.58 doesn't work

See Issue #18015 . Could be RivaTuner Statistics Server that also is getting installed by MSI Afterburner. Did you install one of those two applications?

I have the same issue after update Windows to Insider 22621.1.
I don't have MSI Afterburner or RivaTuner Statistics Server
PowerToysReport_2022-05-13-17-54-08.zip

Thanks, I tried the RTSS setting and that's totally what it was on my system. Of course, I still think it shouldn't crash because of that ;)


Duplicate styles applied

When using the stylesheets from the dist folder in selectize, the themed ones (e.g. bootstrap3) are actually compiled less stylesheets which include the base styles. Including selectize.css also just doubles the size of the css getting imported.
Notice how large selectize.bootstrap3.css is, and how similar it is to selectize.css.
https://github.com/brianreavis/selectize.js/blob/master/dist/css/selectize.bootstrap3.css

On further investigation, it seems like perhaps those styles are needed, even though they seem very much like duplicates. I will close this issue for now. Might be something I should take up with the maintainer or selectize.

If you ever open any issue, please let us know.

On further investigation, the selectize.xx.css files do include the base styles.
I'm opening up a PR for this.


release: v3.14.10

[v3.14.10] (June 13, 2024)
Fixes

Replaced non-existent onlight-05 with onlight-03 as per product designer's recommendation.
Added the onClose event to the modals inside of the FileViewer components.

/bot create

/bot create ticket

/bot create ticket


Sync with master

PR to sync up with upstream/master that already includes test fixes needed to get tests to pass for firefox images


[TA-Patch-Feb] Prepare for the Feb patch release

Monthly patch release for Feb

API changes have been detected in com.azure:azure-ai-textanalytics. You can review API changes here


fix logic for others len in pivot_longer_spec

PR Description
Please describe the changes proposed in the pull request:

fix if statement that fails in pivot_longer_spec due to an empty list

This PR resolves #1405 .

Please tag maintainers to review.

@ericmjl

🚀 Deployed on https://deploy-preview-1406--pyjanitor.netlify.app

@samukweku I'm going to merge. Please do the honors of cutting the new release!


Support "priority" and add 2 templates for InsERT-software-generated invoices

templates: pl: add templates for InsERT's software issued invoices

InsERT is one of the most popular Polish accounting software company.
They have two very common softwares:
1. Subiekt nexo
2. Subiekt GT

Add 2 templates to parse invoices generated by above softwares. Those
templates are software-specific so they have priority set to 3.

This allows parsing a lot invoices issued in Poland.

Regex: support more grouping functions

In case of multiple matches we only had an option to return a sum of
numeric values. Add more functions.

Add "priority" support for templates

In case of multiple templates matching given invoice - choose the one
with the highest "priority" value. To provide proper support for
prioritizing AND existing templates (backward compatibility) the default
value 5 is assumed in case "priority" property is missing.

This feature can be used for writing more generic as well as more
specific templates. So far all templates were assumed to be
company-specific. With this change we can have:
1. Invoice-generating software specific templates
2. In-company varying templates

This feature may be very useful for:
1. Countries with just few very popular accounting software applications
2. Big companies with multiple departments adding some invoice details

Thanks for this very interesting PR!
Will need some time to test it.
I like the idea of creating generic templates, even selecting on the accounting software or library that generated it.
At some point it might be interesting to look / filter on the meta data passed in the invoicefile.
Just a thought..
Will the use of prioritys have a big impact on precessing time?
Either way, being able to parse multiple invoices and minimize the writing write specific templates is a big plus.

I'm happy to get a positive respose to this. Take your time to review / test this.

Will the use of prioritys have a big impact on precessing time?

I think at loading templates takes the most time. This part doesn't get changed with this pull request.
With this change matches_input() gets called for every loaded template. Previously it was called until the first match. matches_input() is a simple function that just uses re.search(). So this pull request shouldn't affect perfroamnce much.

Since you mentioned perfromance I think we should check why loading templates takes so much time. Is that file access or parsing YAML? Maybe we could optimize / cache?
(Just a topic for another discussion)

Made the same performance update as on the 'old code' in #470
Tests will pass after the flipkart template has been updated from that pr.

btw, I run this trough the profiler. No noticable difference on the performance.

Just reverted the performance changes. As we are discussing them in #470.
This one is now independent from that one.
When tests are green, we can merge this one.


Unify python Socrata Open Data API tooling?

I happened to notice some SODA query/API helpers here @swo. Since this is a common need across CFA (there are R helpers within forecasttools and @AFg6K7h4fhy2 is thinking about recreating them in forecasttools-py ), we should discuss how not to duplicate effort and create robust tools.
It's a shame that https://github.com/xmunoz/sodapy is no longer maintained. I see one attempt to fork it and continue development, but it has just the one dev: https://github.com/digital-nirvana/sodapy2
We could:

Build our own very lightweight helper package on CDCGov
Fork the original sodapy for ourselves
Contribute to sodapy2.

Explicitly stating agreement with above DHM comment.
Conversation here seem like it could easily reduce wasted motion.
My preference is (1) but this might change were I less ignorant concerning sodapy and  sodapy2 internals.

I like option 1.
Now that you're here asking about this, I prefer option 1 to option zero (status quo). I've written Socrata queries by hand a few times and re-implemented this kind of interface at least once. Pagination is new but I think that's important.
sodapy was a pretty simple set of wrappers around requests. It included support for things I don't know if we need now (e.g., uploading data) and also made some API choices that I think were suboptimal (e.g., get_all() iterates over the records one by one). I don't know that there's a lot there we need to rescue, which makes options 1 and 2 less appealing.


Fix object keys names

Hello,
Prior to change, downloading the sites files would not work and the generated zip file would be empty.
Also, if you can please shed some light on how to link this project to the primo project. What i managed to do is clone the builder repo and add it to the primo project using npm install --save ../builder but for each change i need to run
npm run build, surely there's a better way.
Thanks again

To kind of answer my own question
First cd to the builder project and run npm run package-watch then run npm run dev in primo project

Hey there - looks like you're including the 'dist' folder. Mind removing that so I can merge this? Thanks!

I'll close this one since i had no idea that further commits on my part in this repo would be included in the original pull request that only included file src/lib/views/modal/Deploy.svelte


Gulp error

I'm running the following:
Vanilla 2.3
NPM 3.10.10
Node v6.10.3
When I build or watch I get: TypeError: themes.restore is not a function
Any ideas?

I'm unable to replicate this using Node.js 7 and npm 4. Are you using the latest master?

It's the one downloaded from the themes site ...
On 20 May 2017 21:18, "Kasper Isager"<EMAIL_ADDRESS>wrote:

I'm unable to replicate this using Node.js 7 and npm 4. Are you using the
latest master?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kasperisager/vanilla-bootstrap/issues/199#issuecomment-302896309,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA-XMO7TvVnXBoqntSFqUxGO2kix9Kg7ks5r70qWgaJpZM4Ngftj
.

Alright, that's likely why in that case. You can try this one instead: https://github.com/kasperisager/vanilla-bootstrap/archive/master.zip

Thanks Kasper. That's worked perfectly :)


Updated python-hypothesis to 6.36.2

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[x] The toolchain has been rebuilt successfully (or no changes were made to it)
[x] The toolchain/worker package manifests are up-to-date
[x] Any updated packages successfully build (or no packages were changed)
[x] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[x] All package sources are available
[x] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/tools/cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[x] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[x] All source files have up-to-date hashes in the *.signatures.json files
[x] sudo make go-tidy-all and sudo make go-test-coverage pass
[x] Documentation has been updated to match any changes to the build system
[x] Ready to merge

Summary

Updated python-hypothesis to 6.36.2 for 2.0
Change Log

Updated python-hypothesis to 6.36.2

Does this affect the toolchain?

NO
Test Methodology

local build

The tests did pass, with RUN_CHECK=y enabled.


feat(Makefile): Dev Environment container inception

Enable users to bootstrap the development environment from a container.
Use case: A developer has their tools inside a container and mounted
this directory from the host in it.

OK. This is not a little, but completely crazy. Fact is, I'm experimenting with a completely remote containerised development environment over a coreOS machine in a droplet.
Has its benefits, the whole glide operation takes 2'20'' to complete there, as opposed to an eternity using my 4Mbps home connection :sleeping:. You just have to mount your docker socket and play from it. The problem starts when your inside container router directory (ex: /root/dev/router) differs from the outside container router directory (ex:/home/core/dev/router), then, docker would not be able to mount your directory, stopping you from the joy of getting all glide dependencies fast.
Using this variable I added, DEV_ENV_HOST_DIR, fixes that and enables this little inception.

I've got to think about this a little bit.  This seems like an extreme outlying edge case.

@hermanjunge I don't quite get it. Are you spinning up a container in a pod and then ssh-ing to that container or something?

@arschles he has a container running on a remote host that has make, docker (in docker), and kubectl.  So while I have containerized go and such, he's containerized everything.  (Of course, your local machine still needs a docker client.)  While this is neat, I think the "inception" nature of it complicates things a good bit.  I think I'd want t see some sort of evidence that many others want to work in this way before adding any explicit support for it.

I think this is a neat idea. Though difficult (and messy) to do now... wouldn't that be the dream? From any machine spin up your dev environment per project (think rbenv but where your build environment is remote on AWS, GCE, etc.).
Right now it makes more sense to me to just spin up a virtual machine in a cloud provider to do this (and avoid the docker-in-docker), but that requires quite the setup and backing code to do so.

To be clear, I don't want to stop anyone from doing this, but I'm leaning towards not explicitly supporting it.

Makes sense, @krancour. To me it feels very much like what Wercker is headed toward, but with the idea of a whole development workflow baked in.

I agree with @krancour that this is a long shot, and with @sgoings that is way easier to just do everything in a remote machine well configured to do that instead of a container. Actually I started with an ubuntu virtual machine, and then with a container over coreOS inside a droplet.
This PR has more an intention to share the finding in the weird case something else needed a heads up on this particular. So is perfectly reasonable to close this PR, and maybe just add the note in some sort of "Special Use Case" document, with a disclaimer of no support, of such.

@krancour @hermanjunge right, but you still have to write code, right? do you write it inside the container?

@arschles No. Besides the Dockerfile, and the modification in this PR, no other code is needed to make this proposal work. You can edit the code of deis/router outside or insider the container, it does not make any difference, since you are mounting it as a volume. (If I understood your question well).

yes, I think you understood right. so you're using the VM's filesystem to store your code and mounting it? If so, fwiw I think this is a fine idea.
Your mileage may vary, but from where I'm standing, this is a huge time saver.

I'm :+1: on the time saver point also.
@krancour @sgoings perhaps this only changes 2 vars in the makefile. perhaps we could move those changes to under a build target so we can make this work w/o altering the current workflow?

As we're getting down to the Alpha milestone freeze, would someone mind weighing in on whether this needs to go into Alpha?

It does not.

I'd close the issue. As @krancour stated, it represents an extreme edge case. Anyways, It is good enough as a reference for anyone working on something similar.


Document breaking change

This change is breaking and should be added to changelog.
I also suggest keeping backward compatibility, constructor can look like:
public function __construct(array $optimizers, $executeFirst = false, LoggerInterface $logger = null)

Which would not break all project that depends on this package.
I upgraded (1.0.6 => 1.1.2) and errors started popping.

Hi,
Yes, this change is breaking, it is in the changelog. I am not sure if reverting this BC breaking change is the right way: it will break when someone updates from 1.1.x to the newest version as well, especially when 1.1.0 version is almost one year old.

it is in the changelog.

Where?

it will break when someone updates from 1.1.x to the newest version as well, especially when 1.1.0 version is almost one year old.

I am not sure if reverting this BC breaking change is the right way.

What is the downside? BC break was not needed in order to bring this functionality with this minor release. https://github.com/psliwa/image-optimizer/pull/48 proves that.

@psliwa Ping ^.

All three points in the changelog describe BC-break, maybe it is not clear so I merged yours suggestion. Thanks :)


[3599] Use variable template for stacks-typescript-ssr

📲 What
Move the top level variable out of the main pipeline file into a separate file.
AB#3599
🤔 Why
The Stacks CLI currently performs a search and replace on the template file to change the values based on the user input to the CLI. This is brittle because the original value could change and the CLI would not replace the value correctly.
By making this a separate file the CLI is able to build up the file as needed, without the need for any text replacements.
🛠 How
All of the variables are written to the variable template file, azuredevops-vars.yaml. This is included in the main build file, e.g.
variables:
    template: azuredevops-vars.yml

👀 Evidence
The builds are working as before
🕵 How to test
Notes for QA
✅ Acceptance criteria Checklist
Code peer reviewed?
Documentation has been updated to reflect the changes?
Passing all automated tests, including a successful deployment?
Passing any exploratory testing?
Rebased/merged with latest changes from development and re-tested?
Meeting the Coding Standards?

/azp run

/azp run

/azp run


Spi speedup 1

Added fixes for corner condition  ( clock divider set to 0)

This breaks my build when fineDelayBits is left at its default value of 0. The creation of the register is not conditional.
[error] Caused by: java.lang.IllegalArgumentException: requirement failed: RegField width must be > 0, not 0
[error] 	at scala.Predef$.require(Predef.scala:277)
[error] 	at freechips.rocketchip.regmapper.RegField.<init>(RegField.scala:141)
[error] 	at freechips.rocketchip.regmapper.RegField$.apply(RegField.scala:177)
[error] 	at sifive.blocks.devices.spi.SPITopModule.<init>(TLSPI.scala:137)

This breaks my build when fineDelayBits is left at its default value of 0. The creation of the register is not conditional.
[error] Caused by: java.lang.IllegalArgumentException: requirement failed: RegField width must be > 0, not 0
[error] 	at scala.Predef$.require(Predef.scala:277)
[error] 	at freechips.rocketchip.regmapper.RegField.<init>(RegField.scala:141)
[error] 	at freechips.rocketchip.regmapper.RegField$.apply(RegField.scala:177)
[error] 	at sifive.blocks.devices.spi.SPITopModule.<init>(TLSPI.scala:137)

I see in rocket chip the 'require' statement indeed has ">= 0" comparison, meaning register width '0' is allowed. However, your error signature shows '> 0' comparison. Is your rocket-chip module up-to-date?


move #295 DNS Egress Proxy <IP_ADDRESS>/19 out of modules into a tfvar in environments - even though it is static

https://github.com/GoogleCloudPlatform/pbmm-on-gcp-onboarding/blob/main/modules/20-partner-interconnect/main.tf#L78
see #295
and
#286
#249

20240406: Closing issue during retrofit/rebase of this TEF V1 based/modified repo to TEF V4 standards
This issue may participate in the LZ refactor after rebase
Query on all issues related to the older V1 version via the tag
https://github.com/GoogleCloudPlatform/pbmm-on-gcp-onboarding/labels/2024-pre-tef-v4


disable border on specific space

With some appliations, drawing a border causes performance problems. Unfortunately,

kwmc config space 0 8 border focused off

does nothing. Would it be possible to add this? Disabling an application border via a blacklist rule appears to work inconsistently because it produces a ghost border.

It's highly improbable that drawing the border is causing performance issues, I think the real lag you may notice is caused by focus follow mouse, you can try to disable that and see if that is the real bottleneck

No, it's indeed drawing the border in some capacity. Disabling focus follows mouse globally does not fix the issue, disabling borders globally does.

Need more information to be able to debug this. I have not had any performance problems caused by the border.

Okay, it turns out that this is actually expected behavior with kodi, which detects when the window is "in the background" and lowers its own framerate to save resources. I suppose drawing a border somehow triggers that behavior. The question then is if you would consider adding the feature of disabling borders for a specific space or even application. Obviously that makes this a feature request and not a bug. :)

I've decided that this is not something I will be doing, partly due to how borders work (code-wise).


Support for creating new Event Logs

Would be great if I could use this resource to create new event logs and event sources.
For example today I would call New-EventLog like this to get this done:
### Create log and source
New-EventLog -LogName BigLog -Source TheHorsesMouth

My lame workaround is to use the Script resource:
Script MyEventLog
{
    GetScript  = {
        return @{LogName='MyEventLog'}
    }
    TestScript = {
        try{
            Write-Verbose "Getting EventLog 'MyEventLog' with Source 'MySource'"
            Get-EventLog -LogName MyEventLog -Source MySource -Newest 1 | Out-Null
            return $true
        }
        catch{
            Write-Warning 'Log not found'
            return $false
        }
    }
    SetScript  = {
        Write-Verbose "Creating EventLog 'MyEventLog' with Source 'MySource'"
        New-EventLog -LogName MyEventLog -Source MySource -Verbose
    }
}

I'd also like this to be implemented.

I've ran into this problem as well and I'm working on a PR to add it.
While looking at using this as a basic layout for implementing it I've discovered a small problem:

The test block runs and find that the LogName doesn't exist (or it's sources don't match)
The set block runs and creates the LogName with the various sources specified
The test block runs again and can't find the LogName because it doesn't have any log entries in it.

So my current solution is that as part of the Set block it will write a log entry for each Source defined. This will populate the log to actually create it but also populate the Source entries allowing further Test blocks to run correctly.
Does anyone see any issues with this? Other than having a few spurious event log entries that might flag in monitoring solutions.

Submitted PR #14 which adds this functionality.


Add Rust extension

The solution is based on https://github.com/build-server-protocol/build-server-protocol/pull/306

Is this PR ready for review?

No, this PR is not ready. I must've missclicked somehow in IDE, sorry

Superseded by #557


d20200331_PEICHL

Metadata, upload link sent.

to Avni we submitted only CH4 flux data from our 4 dark chambers located in natural vegetation which are the ones we consider for our CH4 fluxes. The chambers also measure Reco but the data is not included since Avni only had asked for CH4. There is a readme tab in the datafile that explains which chamber is clear/dark and dark on the trenched bare peat plots, but in summary:
Ch1,4,7,10 = NEE (clear)
Ch2,5,8,11 = Reco (dark) + CH4 flux
Ch3,6,9,12 = Rh (dark, trenchend plots)

This dataset would be a good candidate for data/ancillary restructuring in #197


Post-deploy cleanup for Suite

[x] Add google analytics
[x] Improve logging across the site, including to engine-light
[x] Fix weird alert placement in opp site
[x] Fix #40
[ ] Fix #44

[ ] Add title filter to NIGP categories


Restrict tinyrpc to version 0.9.4 as newer versions break python2 support

Tinyrpc dropped python2.7 support in verison 1.0.0.
Version 0.9.4 is the last version to support python2.7, therefor ryu should use that version until it itself drops python2.7 support.

Coverage remained the same at ?% when pulling e3ebed794332ca23a0f67580fd3230612d9d7b07 on freundTech:master into c776e4cb68600b2ee0a4f38364f4a355502777f1 on osrg:master.

Coverage remained the same at ?% when pulling e3ebed794332ca23a0f67580fd3230612d9d7b07 on freundTech:master into c776e4cb68600b2ee0a4f38364f4a355502777f1 on osrg:master.

thanks


ciao-cli: Fix instance list template ordering

The commits that sorted the result of the instance list command and
added template support for this command conflicted.  The automerge
done by github was insufficient and resulted in an unsorted slice
being passed to the user supplied template.  This commit simply
re-arranges the code a little bit to ensure that the instances processed
by the user template are sorted.
Signed-off-by: Mark Ryan<EMAIL_ADDRESS>

Coverage decreased (-0.02%) to 66.597% when pulling 761f0a9524d189803c6a58e68ee70ea900e89b0a on markdryan:master into d4bdb87fb5302c03741faa7a8b1cc2506a40540d on 01org:master.


lowercase terms with parentheses

this adds a function to decided when to lowerCase the term
the current rules we implement are:

term start with UpperCase eg: Word

the new suggested rule:

term start with ( parentheses followed by UpperCase eg: (Test

needs discussion:

term with UpperCase no matter the position

as the analyzer shows the results based on the tokens we produce we can be excluding results from showing up
example:
dreamWorks will produce [dreamWorks] as the  sole token that returns this results
if we search for dearmworks we will not get results from the latter
@adamretter  what do you think?

@yamahito

I have to admit, I'd assumed it was already doing the "no matter the position" one;  what do you think the risks are of implementing that?
Perhaps DreamWorks --> [dreamWorks][dreamworks][DreamWorks] would hedge our bets so that an exact match is higher, but the other terms would all at least provide some match?


Case of multi-container app with project references

One important topic is missing in this article. Real-world applications often don't contain all business logic inside single aspnet project. So my question is: how to make this example work if there's third project in solution - class library - and one of, or both, deployable projects reference that library?
Say, we have ./MyBusinessLogic/MyBusinessLogic.csproj alongside ./MyWebApi/MyWebApi.csproj and ./WebFrontEnd/WebFrontEnd,csproj. And, for example, MyWebApi references ../MyBusinessLogic/MyBusinessLogic.csproj and uses some method from some class defined in MyBusinessLogic.
Are there other ways to achieve this other than having dockerfiles for each deployable project on root level, where .sln file is located?

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

ID: 7aee46f8-9db1-33ef-b637-fc8e8881fab9
Version Independent ID: 6c469f9f-9751-669f-3dd4-c4fb49c9ae19
Content: Work with multiple containers using Docker Compose - Visual Studio (Windows)
Content Source: docs/containers/tutorial-multicontainer.md
Product: visual-studio-windows
Technology: vs-container-tools
GitHub Login: @ghogen
Microsoft Alias: ghogen

Another good example would be Blazor app. We have WebApi (container 1), which we might want to have several instances of, BlazorApp (container 2), which will be hosted as static content, and library with requests and responses shared between them.

@lucky-ly -- Dmitry, thank you for your feedback and suggestions.  Please consider these resources:

Visual Studio on Microsoft Q&A
Visual Studio MSDN Forum
Stack Overflow
Reddit

@ghogen -- Gordon, please look into this issue.

@lucky-ly
You would have a Docker Compose project, a class library project, and the WebApi project as well as the front end, maybe Blazor or Razor. The WebApi project and the front-end project would have Dockerfiles. I don't know if your proposed structure makes the situation much more complicated than the scenario used in the article, but I guess I can try that scenario. Did you run into problems?


Uncaught TypeError: Cannot read property 'arrayToDataTable' of undefined

Exception in dashboard.js line 134.  This is only observed in the development branch.

Fixed.