d2vpkr instead of dota2.com/jsfeed

fixes #5
Generates abilities.json entirely from d2vpkr instead of dota2.com/jsfeed. This gives us more control over what data we want, and how to format it.
Will add more than just abilities, but I figured I'd start with this and see what people have to say

I'ts also worth noting that abilities.json now contains neutral abilities too, so maybe we should remove the neutral_abilities.json file?

Sure, if you've confirmed that all of the neutral abilities are there.


Use class name when determining import handler

According to the site documentation, the import handler is the name of the class with Handler appended to the end of it.  This does work IF the current class has Import at the end of it.  Otherwise, the code looks for a class named ImportHandler.
This change makes the documentation actually work, regardless if the class ends in Import or not.

Not actually sure why that pull request merge is included in this.  Feel free to let me know if you'd like a modified PR.

Thanks.
Can you also include this fix for the NewExcelFile?

There you go!


Why I can not compile???

$ rebar3 compile
===> Verifying dependencies...
===> Fetching kafka_protocol ({pkg,<<"kafka_protocol">>,<<"0.7.3">>})
===> Download error, using cached file at c:/Users/Net/.cache/rebar3/hex/default/packages/kafka_protocol-0.7.3.tar
===> Fetching supervisor3 ({pkg,<<"supervisor3">>,<<"1.1.3">>})
===> Downloaded package, caching at c:/Users/Net/.cache/rebar3/hex/default/packages/supervisor3-1.1.3.tar
===> Fetching snappyer ({pkg,<<"snappyer">>,<<"1.1.3-1.0.4">>})
===> Downloaded package, caching at c:/Users/Net/.cache/rebar3/hex/default/packages/snappyer-1.1.3-1.0.4.tar
===> Compiling supervisor3
===> Compiling snappyer
MAKE Version 5.2  Copyright (c) 1987, 2000 Borland
Error makefile 15: Command syntax error
Error makefile 18: Command syntax error
Error makefile 20: Command syntax error
*** 3 errors during make ***
===> Hook for compile failed!

Looks like make 5.2 from Borland can't read our Makefile. Can you try to install http://gnuwin32.sourceforge.net/packages/make.htm and see if it works? Don't have a windows PC at hand, but I'll try to figure something out if this does not work.
Also, could you provide more details? Windows version, whether you use cygwin, etc.

Here is the on-windows guide from erlang.mk:  https://erlang.mk/guide/installation.html#_on_windows
Loading nif from dll can't be supported without a significant change to Makefile and source code.
On the other hand, we should probably support a global setting to enable/disable snappy.
when disabled, kafka_protocol should not download the dependency and not compile the snappy compression/decompression clauses either.

Thank you for your help, I try again

@yunnet Did it work out for you?
I have tried to make snappyer dependency optional in Makefile and rebar.config.script of the library kafka_protocol, which worked for erlang.mk and rebar but not rebar3 (because it downloads dependencies recursively from hex).
If you do not need snappy compression in your project, you can try to manually remove the dependency, (also delete the entry from your project rebar.lock).


Scoring board should stay with Research board

When going small it's requiring it's own space, messing the UI

Probably linked to #35 , but the map shouldn't be on the same svg

Is it online?
This is what I'm getting on the iPhone X

It's not online, but you can check in the browser, in dev tools they allow you to switch to mobile mode: https://developers.google.com/web/tools/chrome-devtools/device-mode/

It's online, and I just tested.


Contact form | Email notifications

As a site admin, when someone uses the contact form, I would like there to be an email sent to me so that I don't have to keep remembering to check this.
Acceptance Criteria

[x] When a contact form is filled in, an email is sent to an email address with the contents of the form (sender, subject and message)
[ ] The content of this email needs to be readable - i.e. Formatted (line breaks for example) as it was entered by the user
[ ] Emails can be sent to multiple admin recipients

See https://github.com/cul-2016/site/issues/42 for research already done on this

Have verified the project email here on SES and we are now able to send emails using the npm module sendemail.
Next steps

Test sending email to multiple recipients
Stop apostrophe from submitting the form data if there was an error sending the email.
Ensure that line breaks are displayed correctly in the email


Update bcpkix-jdk15on to 1.70

About this PR
📦 Updates org.bouncycastle:bcpkix-jdk15on from 1.69 to 1.70
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.bouncycastle", artifactId = "bcpkix-jdk15on" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.bouncycastle", artifactId = "bcpkix-jdk15on" }
}]

labels: library-update, commit-count:1

I'm going to update to bcpkix-jdk18on instead of bcpkix-jdk15on


[MM-17850] Add missing currentTeamGroupdConstrained input to getProfilesNotInTeam function call

Summary
This PR fixes an issue where pagination breaks when adding users to a team.  At some point, the currentTeamGroupConstrained value was added to the getProfilesNotInTeam() function. There are two calls to getProfilesNotInTeam() within the add_users_to_team.jsx component file, but only one was adjusted to include this new input.  This PR adds the value to the other function call.
The issue resulted in the USERS_PER_PAGE value (50) getting passed into getProfilesNotInTeam() as the page number value when a user clicks the next button.
Ticket Link
https://mattermost.atlassian.net/browse/MM-17850

Creating a new SpinWick test server using Mattermost Cloud.

Mattermost test server created! :tada:
Access here: https://mattermost-webapp-pr-3529.test.mattermost.cloud

Account Type
Username
Password

Admin
sysadmin
Sys@dmin123

User
user-1
User-1@123

Took a quick spin on the test server and no issues found. Removing QA Review label.

Test server destroyed

@jfrerich
Error trying doing the automated Cherry picking. Please do this manually
+++ Updating remotes...
Fetching upstream
Fetching origin
+++ Updating remotes done...
+++ Creating local branch automated-cherry-pick-of-#3529-upstream-release-5.15-1566635989
Branch 'automated-cherry-pick-of-#3529-upstream-release-5.15-1566635989' set up to track remote branch 'release-5.15' from 'upstream'.
+++ Downloading patch to /tmp/3529.patch (in case you need to do this again)

+++ About to attempt cherry pick of PR. To reattempt:
  $ git am -3 /tmp/3529.patch

Applying: Add missing currentTeamGroupdConstrained input to getProfilesNotInTeam function call
Using index info to reconstruct a base tree...
M	components/add_users_to_team/add_users_to_team.jsx
Falling back to patching base and 3-way merge...
No changes -- Patch already applied.

+++ I'm about to do the following to push to GitHub (and I'm assuming origin is your personal fork):

  git push origin automated-cherry-pick-of-#3529-upstream-release-5.15-1566635989:automated-cherry-pick-of-#3529-upstream-release-5.15

+++ Creating a pull request on GitHub at mattermost-build:automated-cherry-pick-of-#3529-upstream-release-5.15

+++ Returning you to the master branch and cleaning up.

Test server destroyed


⚠ ClamAV has degraded performance

In 133248d, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 70 ms

Resolved: ClamAV performance has improved in 12dbb81 after 7 minutes.


Fix: Navbar docs link

What this PR does / why we need it:

The docs links should use Documentations struct instead of string. Previously, I assumed we only use JSON string for passing docs link but it should be properly parsed as Documentations struct.

LGTM


Some image matchings are no need

Many matchings in robigo.py should be replaced by keyboard operations which are lot more robust and compatible, such as setRoute() to Robigo / Sothis can be replaced by newly-implemented session.setTargetSystem() and so on.
Odyssey support will be provided to robigo.py and entire autopilot soon.

Also, some extra and useless templates will be purged soon.


cbuild  reports "bad pack name"

Describe the bug
Error unexpected with this project Hello.zip
cbuild Hello.csolution.yml --context-set --update-rte --packs --context Hello.Debug+AVH-CM0plus
Hello.cbuild-set.yml - info csolution: file is already up-to-date
E: bad pack name: it should be one of the following formats: Vendor.Pack, Vendor.Pack.x.y.z, Vendor.Pack.x.y.z.pack, or Vendor.Pack.pdsc

To Reproduce
All required packs are installed and up-to-date on local computer.
Error also appears when cpackget is called with:
cpackget add " "

Therefore it is likely that even when no packs are missing cpackget is called.
Expected behavior
Project builds correct.
Environment (please complete the following information):

CMSIS-Toolbox 2.5.0-devint5
Windows

Its seems a side effect of extending --context-set to all the commands
when cbuild calls list packs with -S it tries to generate cbuild-set. As a result of which we
`cbuild list packs -c X.y+Z-S return
C:/Sandbox/opensource/Hello/Hello.cbuild-set.yml - info csolution: file is already up-to-date
<EMAIL_ADDRESS>when parsing the output. The first entry is not a valid pack name.

VS Code extension issues exactly this command. IMHO this needs to be addressed before release

Agreed. Sourabh is on it already.


Add NixOS/Nix installation instructions

Awesome. Thank you!! Do you have to manually update the packages, or does it auto-update? Just curious.

It doesn't auto-update, it's a manual process. But tools like nix-update and nixpkgs-review make the process a breeze. The result is a PR like this one: https://github.com/NixOS/nixpkgs/pull/174083 that will be reviewed and merged by other contributors.

Cool. Thanks for the explanation, and thanks for maintaining the package.


Check if the errorCallback is defined

When an error occurs during the connection ng-stomp correctly reject the promise but also try to call the undefined function errorCallback (line 37). Before calling that function should be checked if it is defined or not.

Resolved by #12. Thank you for the contribution.


Use Tiptap WYSIWYG Editor instead of Quill

You should use Tiptap WYSIWYG Editor instead of Quill, it's more extensible and it's working better with Vue.js.
Tiptap Werbsite: https://tiptap.scrumpy.io/

Hi @octoper,
thanks for the suggestion. Quill has been working great for us, but I know Tiptap is a very interesting alternative. In Twill 1.2.2 we improved our Quill powered form field, but we are still looking for more improvements, so will definitely keep Tiptap in mind.

Desperately hoping that Quill 2 is released sometime soon as the lack of table support is really a problem for me. I had an embarrassing incident where I just assumed that some basic copy/paste table support would work in Twill so told a client the same...  yet it doesn't work due to a total lack of table support in Quill :(
Pasting from things like MS Word/Excel are a working world reality for many users of a CMS.


[SKIP CI] Document interop with vSphere features

This PR documents vDVS volume behavior with vSphere features.
Please refer to issue #270 for different vSphere features.

@govint Have addressed all your comments. Please take a look.


Mining dues not being handled correctly

Raised by bitcointalk user 'essexsend' in this thread:
https://bitcointalk.org/index.php?topic=1085503.msg11697446#msg11697446
A log has been uploaded - labelled with #626 in CrashReporter

Stacktrace:
"stack_trace": "org.bitcoinj.core.ScriptException: Cannot cast this script to a pay-to-address type at org.bitcoinj.script.Script.getToAddress(Script.java:336) ~[multibit-hd.jar:na] at org.bitcoinj.script.Script.getToAddress(Script.java:318) ~[multibit-hd.jar:na] at org.multibit.hd.core.services.WalletService.calculateOutputAddresses(WalletService.java:637) ~[multibit-hd.jar:na] at org.multibit.hd.core.services.WalletService.adaptTransaction(WalletService.java:453) ~[multibit-hd.jar:na] at org.multibit.hd.core.services.WalletService.getPaymentDataSet(WalletService.java:250) ~[multibit-hd.jar:na] at org.multibit.hd.ui.views.screens.payments.PaymentsScreenView.initialiseScreenViewPanel(PaymentsScreenView.java:113) ~[multibit-hd.jar:na] at org.multibit.hd.ui.views.screens.AbstractScreenView.getScreenViewPanel(AbstractScreenView.java:112) ~[multibit-hd.jar:na] at org.multibit.hd.ui.views.DetailView.onShowDetailScreen(DetailView.java:90) ~
"message": "Cannot cast this script to a pay-to-address type",
"stack_trace": "org.bitcoinj.core.ScriptException: Cannot cast this script to a pay-to-address type at org.bitcoinj.script.Script.getToAddress(Script.java:336) ~[multibit-hd.jar:na] at org.bitcoinj.script.Script.getToAddress(Script.java:318) ~[multibit-hd.jar:na] at org.multibit.hd.core.services.WalletService.calculateOutputAddresses(WalletService.java:637) ~[multibit-hd.jar:na] at org.multibit.hd.core.services.WalletService.adaptTransaction(WalletService.java:453) ~[multibit-hd.jar:na] at org.multibit.hd.core.services.WalletService.getPaymentDataSet(WalletService.java:250) ~[multibit-hd.jar:na] at org.multibit.hd.ui.views.screens.payments.PaymentsScreenView.initialiseScreenViewPanel(PaymentsScreenView.java:113) ~[multibit-hd.jar:na] at org.multibit.hd.ui.views.screens.AbstractScreenView.getScreenViewPanel(AbstractScreenView.java:112) ~[multibit-hd.jar:na] at org.multibit.hd.ui.views.DetailView.onShowDetailScreen(DetailView.java:90) ~

The transaction that is problematic is a COINBASE transaction.
If you search in the CrashReporter for 'COINBASE' you'll find it.
(not included here for privacy reasons).

I've pushed some changes that harden the code that gets toAddress.
Still needs testing yet

This has been tested successfully against the OP's wallet
Ready for review and closing.

Code review looks fine and code works fine with existing wallets.
Closing.


default DRY as to the cocurrent handlers of the cosumer

It seems that the default RDY for the cosumer connection is always 1, in spite of the "cocurrency" that set by the "AddConcurrentHandlers" method, consequently, the messages can only be sent to one handler goroutine at the same time, other goroutines has to wait, until the "FIN" is sent for the read message.
To resolve this situation, it has to manually call the "ChangeMaxInFlight" method, then all of the handler goroutines can receive messages (if they're in the queue ) at the same time.

@chaosue You are correct. MaxInFlight (which controls the RDY count each individual connection to a nsqd instance gets and the total number of messages you can have outstanding at any point in time) is independent of the concurrency control.
This is by design.
Since nsqd operates on a push basis (nsqd pushes messages to clients), MaxInFlight allows you to oversubscribe to eliminate latency. If for example you had MaxInFlight=2 but concurrency=1, while processing 1 message nsqd could send the "next" message over the wire so it's ready to be processed instantly after the first one is completed. Without oversubscribing you would have to wait a round trip time between client finishing, sending the response and then getting the next message. This would undesirably limiting throughput in some situations, especially high latency ones.

I'm going to close this issue, but if you have more questions please ask.


[WIP] Upgrade dependencies

References

React Navigation v5: upgrading-from-4.x

TODO

[x] upgrade deps except react-native or @ledgerhq related modules
[x] create ScreenName, NavigationName const
[x] create navigation components
[x] import ScrollView, FlatList… from react-native instead of react-navigation
[x] import SafeAreaView from react-native-safe-area-context
[x] create useNavigationInterceptor
[ ] replace string screen/navigator name with obj from src/const/navigation.js
[ ] replace static navigationOptions
[ ] replace getParams with route.params
[ ] delete old navigation logic files

src/navigation.js
src/screens/OnboardingOrNavigator.js

[ ] delete old navigation deps

react-navigation
react-navigation-stack
react-navigation-tabs

[ ] fix styling issues

Styling issues

[ ] header height

[ ] header margin

[ ] side padding

[ ] side padding

[ ] tab icon

Current outdated dependencies

[ ] 🐞After upgrade react-native, it gets build from xcode but not from react-native cli

Bugs

[ ] Settings -> General -> Password lock
[ ] Settings -> Help -> Set up device
[ ] Transfer -> Buy crypto assets
[ ] Settings -> Debug -> Mock & Test
[ ] No response: Portfolio/Accounts -> Add accounts/Import desktop accounts

Header style bugs

[ ] Password lock -> Set password

[ ] Android build failed

Bug found
iOS - Password lock ON
When user open the app and submit the password form (Valid or incorrect), he is asked for the OS native password. Only then the LL password check is done.
PS: On the screen record below the password input is blank but there is actually a password that is entered.
How to reproduce

Ref

https://github.com/oblador/react-native-keychain/issues/316#issuecomment-602581214


Combining the Jane Street Core and basic-project examples

Hi!
I'm an OCaml newbie, trying to combine your basic project (in the examples/ directory) and Jane Street Core (from the documentation's ‘quick start guide’) examples:
lib/jbuild:
(jbuild_version 1)

(library
 ((name        ocameel)
  (public_name ocameel)
  (libraries (core))
  (preprocess (pps (ppx_jane)))
))

lib/ocameel.ml:
let print =
   Sexp.to_string_hum [%sexp ([3;4;5] : int list)]

bin/jbuild:
(jbuild_version 1)

(executables
 ((names (cli))
  (libraries (ocameel))))

(install
 ((section bin)
  (files ((cli.exe as ocameel)))))

lib/ocameel.ml:
open Core

let () =
   Ocameel.print ()
   |> print_endline

Unfortunately, I'm getting:
ocameel master ❯ jbuilder build @install
    ocamldep bin/cli.depends.ocamldep-output
    ocamlopt .ppx/ppx_jane/ppx.exe
         ppx lib/ocameel.pp.ml
    ocamldep lib/ocameel.depends.ocamldep-output
      ocamlc lib/ocameel.{cmi,cmo,cmt} (exit 2)
(cd _build/default && /usr/local/bin/ocamlc.opt -w -40 -g -bin-annot -I /Users/ec/.opam/system/lib/base -I /Users/ec/.opam/system/lib/base/caml -I /Users/ec/.opam/system/lib/base/shadow_stdlib -I /Users/ec/.opam/system/lib/bin_prot -I /Users/ec/.opam/system/lib/bin_prot/shape -I /Users/ec/.opam/system/lib/core -I /Users/ec/.opam/system/lib/core_kernel -I /Users/ec/.opam/system/lib/core_kernel/base_for_tests -I /Users/ec/.opam/system/lib/fieldslib -I /Users/ec/.opam/system/lib/jane-street-headers -I /Users/ec/.opam/system/lib/num -I /Users/ec/.opam/system/lib/ppx_assert/runtime-lib -I /Users/ec/.opam/system/lib/ppx_bench/runtime-lib -I /Users/ec/.opam/system/lib/ppx_compare/runtime-lib -I /Users/ec/.opam/system/lib/ppx_expect/collector -I /Users/ec/.opam/system/lib/ppx_expect/common -I /Users/ec/.opam/system/lib/ppx_expect/config -I /Users/ec/.opam/system/lib/ppx_hash/runtime-lib -I /Users/ec/.opam/system/lib/ppx_inline_test/config -I /Users/ec/.opam/system/lib/ppx_inline_test/runtime-lib -I /Users/ec/.opam/system/lib/sexplib -I /Users/ec/.opam/system/lib/sexplib/0 -I /Users/ec/.opam/system/lib/sexplib/unix -I /Users/ec/.opam/system/lib/spawn -I /Users/ec/.opam/system/lib/stdio -I /Users/ec/.opam/system/lib/typerep -I /Users/ec/.opam/system/lib/variantslib -I /usr/local/lib/ocaml -I /usr/local/lib/ocaml/threads -no-alias-deps -I lib -o lib/ocameel.cmo -c -impl lib/ocameel.pp.ml)
File "lib/ocameel.ml", line 2, characters 3-21:
Error: Unbound module Sexp

I hope my error isn't super-obvious; the documentation on “how to build your first OCaml package” has been a little scattered, and I've been through about a bajillion Google searches, two build systems (!!), and three paper books (!!!), trying to figure out how the heckage one does this!
Any response is appreciated; also, while I'm here, I'd like to suggest that this particular situation be blessed with its own examples/ example-project? Perhaps even with a link from the README?  Maybe the things I'm reading are out-of-date, but it seems like Core is the go-to in the modern OCaml / ReasonML world; and if you're new to OCaml, like me, coming from Real World OCaml / ReasonML's documentation or similar … the very first thing you're probably going to try to do is create a new project with Jbuilder that you can start typing examples from RWO into (at least, I'd assume I'm not alone in this particular need!) It'd be really great to have a shortcut for beginners like myself!

Ayeee, after half a day banging my head against this, I hit submit about five minutes too soon!
Well, foolish beginner mistake (that I knew it would be) aside, go ahead and consider this a request for improved very-beginner documentation! ^_^


Incorrect code formatting for ternary operator

When formatting the code, ternary operators are not formatted correctly. It is formatted as follows:
(responseTime - dateValue) >= 0?(responseTime - dateValue):0
This should instead be formatted as:
(responseTime - dateValue) >= 0 ? (responseTime - dateValue) : 0

Plugin is being re-written and will be resolved in the new version.


#1865 bug fixs

Fixes #ISSUSE_ID.
Changes proposed in this pull request:

Pull Request Test Coverage Report for Build 6697

14 of 16 (87.5%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.005%) to 73.578%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sharding-core/src/main/java/org/apache/shardingsphere/core/parsing/antlr/filler/impl/dql/SelectItemFiller.java
10
12
83.33%

Totals

Change from base Build 6695:
0.005%

Covered Lines:
10688

Relevant Lines:
14526

💛  - Coveralls


reserved identifier violation

I would like to point out that identifiers like "__LUNA_AST__" and "__LUNA_TOKEN__" do not fit to the expected naming convention of the C language standard.
Would you like to adjust your selection for unique names?

I vote for something like LUNA_AST_H

Thanks for your improvement of affected include guards.


Fix RowManagerTest.testJoinDocUri test

So we can address your issue, please include the following:
Version of MarkLogic Java Client API
develop branch (post (5.2.0)
Version of MarkLogic Server
10.0 nightly
Java version
Java 9
OS and version
Linux
Input: Some code to illustrate the problem, preferably in a state that can be independently reproduced on our end
Run RowManagerTest.testJoinDocUri method
Actual output: What did you observe? What errors did you see? Can you attach the logs? (Java logs, MarkLogic logs)
java.lang.AssertionError: unexpected count of result records expected:<2> but was:<0>
	at org.junit.Assert.fail(Assert.java:88)
	at org.junit.Assert.failNotEquals(Assert.java:834)
	at org.junit.Assert.assertEquals(Assert.java:645)
	at com.marklogic.client.test.RowManagerTest.testJoinDocUri(RowManagerTest.java:500)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.base/java.lang.reflect.Method.invoke(Method.java:564)
	at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
...

Expected output: What specifically did you expect to happen?
Test runs without asserts.
Alternatives: What else have you tried, actual/expected?
None.

Interesting.  For me, that test passes locally.


Code refactor + re-design inference ui

I have added two sub-tabs within the main infer tab with each section; Individual Inference & Batch Inference. I also added the title to the interface.
Some translations are missing but I don't know those languages.

It's a small mistake and I'll fix it


Use a thread local kakfa consumer client for each thread in the Kafka…

…Source. Cap the avg fetch time used for binpacking to 100 milliseconds. This is to avoid bursty topics from having a large negative impact on the binpacking.

@ibuenros, please review.


Get vanilla tests passing in Java 9

We have two choices regarding the @Generated annotation: omit it entirely henceforth, or conditionally exclude it in Java 9+. The former may be the best bet long-term, as we don't want to be generating non–Java 9–compliant libraries from older compilers, but while Java 9 is still under development, I'm happy just giving users the ability to compile with it for now.
Included in this PR:

Omit the @Generated annotation if it's not available]
Upgrade to Gradle 3.2 for Java 9 compatibility
Configure Travis to install oraclejdk9

@Generated is a source-only annotation, so you'll be generating Java 9 compliant code anyway, right? Like, @FreeBuilder doesn't need to be on the classpath in order to run FreeBuilder properly

You make an excellent point.


Right segments not rendering

I'm having an issue where the right segments are not rendering. I'm definitely doing something wrong

.
You can see it in action here: https://catalincodes.vercel.app/posts/test-driven-development-in-an-angular-world-part-1
I copy pasted the code form the setup and changed a few properties 🤔.
I also tried disabling all my global styles. Not sure what else to try.

I identified that if I remove this empty tag from the head, the issue is fixed

it's an empty tag, though.

Found the issue. It was this line in my GlobalStyles.


Rule proposal: no layout thrashing

I've seen a lot of seemingly simple code lead to layout thrashing and performance problems that could have been statically detected and avoided. It would be nice to have a rule that could flag some of these scenarios.
I think it might make sense to take Paul Irish's list of what causes layout/reflow, and write a rule that flags when these happen synchronously in places like componentDidMount, render, and componentWillReceiveProps, etc.
Example:
class Foo extends React.Component {
  constructor(props) {
    super(props);

    this.state = {
      windowHeight: window.innerHeight, // <-- this forces layout
    };
  }
  // ...
}

Thoughts?

If there’s a way to do it that minimizes false positives, that sounds like a great idea.


How to specify remote host's port for Safari browsers

Hello ! Thanks for that great tool, works like a charm.
I have a question about Safari, which doesn't seem to work due to the port used by the launcher.
In the browserstack documentation, there are limitations on the ports that safari can listen on on both OS X and iOS : https://www.browserstack.com/question/664
My question is: is it possible to define a port or a port range while launching the tests ?
By default, it seems that the default port used is 7357, but can't find any reference of that value in the code.

Sorry, that was an unrelated issue.


Firehose support for multiple filter and rollups in a single instance

What are you trying to achieve?
Combining multiple sets of filter/rollup combinations to a single instance of firehose caused unexpected data outputs. Due to this it was necessary to tee flow from one instance of firehose to a second instance allowing each instance to utilize it's own filter set and output rollups as required.
The use of two instances effectively doubles the expected CPU/Memory required to implement the solution
If each firehose instance is limited to only a single --filter then any additional rollups that might require a slightly different filter set would require yet another instance to be tee'd and instantiated increasing the footprint further
The request here is to implement a single firehose instance that can take multiple sets of filter criteria and matching rollup criteria optimizing for resources to limit the required compute footprint required.
What is your current work around? (optional)
tee and spawn a new instance of firehose for each --filter --rollup pair required for reporting purposes
cc @kentik-rbarnes please provide examples of sets filter + rollup that can be used for testing.

Want to allow something like this:
    -filters "string,custom_str.dst_network_bndry,==,external"
         -rollups s_sum,sum_bytes_out,in_bytes,custom_str.src_subscriber_id
         -rollups s_sum,sum_bytes_out,in_bytes,custom_str.output_provider
    -filters "string,custom_str.src_subscriber_id,!=,''"
         -rollups s_sum,sum_bytes_out,in_bytes,custom_str.output_provider

Where the indented rollups are calculated after each distinct filter set.

Thoughts on this syntax to link filters and rollups:
   -filters "string,custom_str.dst_network_bndry,==,external,sum_external"
        -rollups s_sum,sum_external,in_bytes,custom_str.src_subscriber_id

This is adding an optional 3rd argument to filter definitions, which is a rollup_name. If there's a rollup_name defined, this rollup will not get applied globally but rather only to values before the named filter is calculated.
rollup_name is the 2rd argument in the rollup definition, after the type value. Here it is sum_external and it can be any string.
This works equally well for flags and using the config file.

how will this work with AND and OR expressions? Do we need to apply the 3rd rollup_name to all instances?
-rollups s_sum,sum_bytes_in,in_bytes,custom_str.input_provider,device_name,input_int_desc,input_int_alias
-filters "string,custom_str.src_network_bndry,==,external and string,custom_str.src_connect_type,==,free_pni and string,custom_str.dst_subscriber_id,!=,''"

becomes
-filters "string,custom_str.src_network_bndry,==,external,sum_bytes_in and string,custom_str.src_connect_type,==,free_pni,sum_bytes_in and string,custom_str.dst_subscriber_id,!=,'',sum_bytes_in"


3 http client

Created a HTTP Client that can send get() and post() requests using the python httpclient library.
Changes:

Implemented get, post and request methods in the http
Implemented exponential backoff (it will retry when request fails)
Added test coverage for methods, backoff and proprogate kwargs

https://app.zenhub.com/workspaces/cogs-dd-cms-61e83b63053eb70011d37b4f/issues/gh/gss-cogs/dp-python-tools/8


Added create comment and fix title empty string

Fixes #71
Changes you made

Added create comments tests
Fixed empty sting in title of comment, team, task

Look good. I'll approve it once there will be another peer review.
One general comment is that I would expect to see some more negative tests, like comment by non-exist user, for non exist task and so on. No need for a full QA process, but the function should know how to handle these cases.


Automated trigger for Integration tests

Is your feature request related to a problem? Please describe.
OpenSearch Benchmark has automated way to run unit tests, however, no such support exists for integration tests. The goal of this issue is to build an automated trigger to run integration tests.
Describe the solution you'd like
This script is currently used to run unit tests on every pull request. We can extend the same to provide support for integration tests.
Reference : Instruction on running integration tests can be found here.
Describe alternatives you've considered

Create automation using a separate jenkins infrastructure

Closed with https://github.com/opensearch-project/opensearch-benchmark/pull/130


make graph screenshots accessible from monitoring channel

Describe the proposal
Grafana graphs are used to show longhaul-nightly runs that and are currently being forwarded to localhost. Thus, the graph screenshots being sent in alert notification to discord channel is not accessible by anyone.
This needs to be merged to master and have a universal username password for everyone to be able to access these graphs.

/assign

@artursouza : Could you please assign tis to me?


I use LRS2 dataset to train the wav2lip model, but the moving of mouth shape is too small

I tried the author's wav2lip training pipeline with the whole LRS2 datasets as the author did, but I got a very small mouth moving. In contrast, I download the author's wav2lip_gan.pth and test the results, it has a regular mouth moving and it's bigger than mine.
So I want to ask you if you have any suggestion about this question.
Looking forward to your reply, thanks!

i meet the same problem

it seems like the author train with multiple resource, also wav2lip depends on your mouth shape to predict the size of open mouth

i meet the same problem
Have u sove this problem?


KHV002 - proposed solution too invasive

What happened
After getting the error mentioned in the title, I looked into how to fix it and the solution proposed at the moment (completely disable the debugging handlers) seems too far reaching.
Expected behavior
A solution as in https://github.com/kubernetes/kubernetes/issues/84040#issuecomment-585915168 is documented as a solution instead, which only disables this single endpoint for unauthenticated access. Even disabling anonymous authentication would be better than completely disabling debugging handlers (but could lead to issues if load balancers are involved).

@MarkusTeufelberger How can this be resolved? I want to work on this. Thanks.

Thanks @MarkusTeufelberger for reporting this.

I think we need to reconsider this "vulnerability". if KH is running outside the cluster then this is indeed an "information disclosure" issue, but if KH is running as pod maybe it's not an issue. We are still thinking about it (and appreciate any thoughts)
In the meantime, I'm happy to improve the documentation with information from that linked thread (@avats-dev).

The resolution would likely entail the one outlined in that thread (removing the version endpoint from the list of publicly viewable ones), not disabling a whole API.
I'd expect this solution (and its caveats) to be documented, not the one that disables the debugging handlers (which is also not documented too well, I'd expect for example a way to set this in kubeadm and a discussion of what else will be disabled, not just a simple "Add this flag to kubelet" without any more background info given).

Hello,
Here is my workaround :
apiVersion: rbac.authorization.k8s.io/v1
kind: ClusterRole
metadata:
  name: system:public-info-viewer
  labels:
    kubernetes.io/bootstrapping: rbac-defaults
  annotations:
    rbac.authorization.kubernetes.io/autoupdate: "true"
rules:
  - nonResourceURLs:
      - /healthz
      - /livez
      - /readyz
    verbs:
      - get

What I cannot solve, unfortunately, is that upgrades - I use kubespray for my clusters - overwrites this ClusterRole, and I couldn't find out where to set this sustainably.
That said, I apply my patch every time I run kubespray, and "/version" path is not publicly available.


Add code of conduct

Adding Contributor Covenant CoC.
http://contributor-covenant.org

+1 LGTM

:+1:


Fix typo in docs/api/Field.md

Codecov Report

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

@@          Coverage Diff           @@
##           master   #2864   +/-   ##
======================================
  Coverage     100%    100%
======================================
  Files          67      67
  Lines        1395    1395
======================================
  Hits         1395    1395

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 2b14b8f...1323d64. Read the comment docs.


Update readme

Updates the readme. See individual commits for more information.

All looks good :+1:


Track last event for when in Shadow DOM

Fixes https://github.com/facebook/lexical/issues/2119. In the Shadow DOM, window.event returns undefined. In this case, we need to use the last known event that the Lexical event system knows of in order to get some heuristic working. I'm unsure of if this will work in all cases, so leaving as as draft for now.

window.event is marked as Deprecated, why not just replace it?

Holding off on this, as seems there's far more work to get Shadow DOM support working with Selection.


GPS info in XMP doesn't use GPSCoordinate format

Exiv2 and exiftool incorrectly report the GPS information in the XMP sidecar.  In my testing they would ignore the GPSLatitudeRef and GPSLongitudeRef such that all coordinates were E and N.
It seems to be related to the format of the EXIF GPS data when in the XMP sidecar.  The spec seems to indicate that the GPSLatitude and GPSLongitude should be in GPSCoordinate format:  https://www.exiv2.org/tags-xmp-exif.html
The best explaination I found of the GPSCoordinate format is at:  www.cipa.jp/std/documents/e/DC-X010-2017.pdf.  This doc also indicates that the GPSLongitudeRef an GPSLatitudeRef are no longer used.
A photo tagged at Niagara Falls, Canada...
The incorrect format:
<exif:GPSLongitudeRef>W</exif:GPSLongitudeRef>
<exif:GPSLongitude>79.0770034790039</exif:GPSLongitude>
<exif:GPSLatitude>43.07640075683594</exif:GPSLatitude>
<exif:GPSLatitudeRef>N</exif:GPSLatitudeRef>

% exiftool DSCF0034.JPG.xmp | grep GPS
GPS Longitude                   : 79 deg 4' 37.21" E
GPS Latitude                    : 43 deg 4' 35.04" N
GPS Latitude Ref                : North
GPS Longitude Ref               : East
GPS Position                    : 43 deg 4' 35.04" N, 79 deg 4' 37.21" E

The correct format:
<exif:GPSLongitude>79,4.620208740234375W</exif:GPSLongitude>
<exif:GPSLatitude>43,4.58404541015625N</exif:GPSLatitude>

% exiftool DSCF0034.JPG.xmp | grep GPS
GPS Longitude                   : 79 deg 4' 37.21" W
GPS Latitude                    : 43 deg 4' 35.04" N
GPS Latitude Ref                : North
GPS Longitude Ref               : West
GPS Position                    : 43 deg 4' 35.04" N, 79 deg 4' 37.21" W

exif:GPSLongitude79,4.620208740234375W</exif:GPSLongitude>
exif:GPSLatitude43,4.58404541015625N</exif:GPSLatitude>
can this be correct? Does it mean 79 degrees, 4 minutes,<PHONE_NUMBER>34375 seconds? Seconds should have decimal point somewhere?

It's expressed in DDD,MMM.mmm where
DDD = number of degress
MMM = number of minutes
mmm = fractions of minutes
So in this case, it's 79 degress and 4.620208740234375 minutes.
To get seconds, it would be 0.620208740234375 x 60 = 37.212.. seconds

Ahh right it was DDM and not DMS, which I thought was more common. Thanks for fast answer and sorry for bothering!


sql: add optional AS OF SYSTEM TIME with CREATE STATISTICS

PR: https://github.com/cockroachdb/cockroach/pull/32643
From release notes:

Added support for AS OF SYSTEM TIME with the CREATE STATISTICS statement. [#32643][#32643] {% comment %}doc{% endcomment %}

Addressed by #4308


Missing dependencies

Apparently the package is using other packages that aren't listed in its dependencies; at the same time, the readme is not mentioning any of these and so when you do even the most basic application you receive a crash as a welcome.

The dependencies are bundled in the package using zeit/ncc. Can you post an example where you are seeing a crash because it shouldn't be related to the dependencies?

It works after installing the types.

Thanks for the update.

You might want to mention it in the readme though


API for receiving shared with a user

Includes the ability to create shares blocking tenant inheritance.

https://www.pivotaltracker.com/story/show/131005257
https://www.pivotaltracker.com/story/show/131005349
https://www.pivotaltracker.com/story/show/131005337

Euwe Backport details:
$ git log -1
commit 6ba273683985b38e0424a3e9f77c738ab1e8cd66
Author: Gregg Tanzillo<EMAIL_ADDRESS>Date:   Thu Oct 13 20:38:01 2016 -0400

    Merge pull request #11887 from chrisarcand/receive-in-tenant-shares

    API for receiving shared with a user
    (cherry picked from commit 7ead6dfbe6f90857b85e386d31794a8fbf9207a7)


Where is my model file when I use saveNativeModel()?

I am using LightGBMClassifier to train model and all is ok. But I can't find my model file even if I use saveNativeModel(). My train dataframe as follows:
train.printSchema()

root
 |-- label: integer (nullable = true)
 |-- features: vector (nullable = true)

My code:
from mmlspark.lightgbm import LightGBMClassifier,LightGBMRegressor

cate_list = [
        'city_id'
        ,'minute_num'
        ,'weekday'
        ,'hour'
        ,'level'
    ]

model =LightGBMClassifier(boostingType='gbdt',
                          objective='binary',
                          baggingFreq=5,
                          learningRate=0.1,
                          numIterations=50,
                          earlyStoppingRound=10,
                          featuresCol = 'features',
                          categoricalSlotNames = cate_list,
                          numLeaves=30,
                          labelCol="label").fit(train)

model.saveNativeModel("mymodel.txt")

So how can I save my model in local path?
At the same time, I want to ask how to set categorical feature. I assemble all feature columns to generate features column in train. Is it right way for me to set categoricalSlotNames = cate_list? What should I do if it wasn't this way?
Can anybody help me? Thanks in advance.

@giser-yugang might the model be saved to dbfs or wasb storage?  You can save to local path by using file:/ prefix.
" I assemble all feature columns to generate features column in train"
are you using string index or our value indexer on them beforehand?  They would then have metadata on them and it would propagate to VectorAssembler, which we detect, so you wouldn't need to do anything else.  If the metadata is not on them, you can pass the slot names (categoricalSlotNames) or indexes (categoricalSlotIndexes) in the features vector column.  Indexes are the most reliable way since slot names may not be propagated in sparse case sometimes.

@imatiach-msft Thanks for your response.
About  model save
But I tried several ways to specify a path that could not save the model.

I meet the error when I use model.saveNativeModel("file:///data1/alge/mymodel.txt").

Caused by: java.io.IOException: Mkdirs failed to create file:/data1/alge/mymodel.txt/_temporary/0/_temporary/attempt_20190826144259_0013_m_000000_3 (exists=false, cwd=file:/data1/yarn/nm/usercache/root/appcache/application_1542447824018_5473532/container_e18_1542447824018_5473532_01_000028)

I meet the error when I use model.saveNativeModel("/data1/alge//mymodel.txt").

Caused by: org.apache.hadoop.ipc.RemoteException(org.apache.hadoop.security.AccessControlException): Permission denied: user=root, access=WRITE, inode="/":hdfs:supergroup:drwxr-xr-x

I can't save my model to hdfs. The size of model file is 0 when I use model.saveNativeModel("hdfs:///user/root/mymodel.txt")

hadoop fs -ls hdfs:///user/root/

drwx------   - root supergroup          0 2019-08-26 06:02 hdfs:///user/root/.Trash
drwxr-xr-x   - root supergroup          0 2019-08-26 06:26 hdfs:///user/root/.sparkStaging
-rw-r--r--   3 root supergroup<PHONE_NUMBER>-04-25 07:19 hdfs:///user/root/esri-geometry-api-2.0.0.jar
drwxr-xr-x   - root supergroup          0 2019-04-23 09:53 hdfs:///user/root/extra_py
drwxr-xr-x   - root supergroup          0 2018-08-30 13:00 hdfs:///user/root/file.csv
-rw-r--r--   3 root supergroup<PHONE_NUMBER> 2019-04-23 13:58 hdfs:///user/root/inst_yg_py2.zip
-rw-r--r--   3 root supergroup   78539755 2019-05-23 10:03 hdfs:///user/root/map_construction.jar
drwxr-xr-x   - root supergroup          0 2019-08-26 06:26 hdfs:///user/root/mymodel.txt

About  categorical feature
I'm not sure I used string index or value indexer on them beforehand. I can share my code.
from pyspark.ml.feature import VectorAssembler
from pyspark.ml import Pipeline

# train dataframe have 'city_id','minute_num','weekday','hour','level','value1','value2' columns.

assembler = VectorAssembler(inputCols=['city_id','minute_num','weekday','hour','level','value1','value2'], outputCol="features")
pipeline = Pipeline(stages=[assembler])
pipelineModel = pipeline.fit(train)
train = pipelineModel.transform(train)
test = pipelineModel.transform(test)

selectedCols = ["label", 'features']
train = train.select(selectedCols)
test = test.select(selectedCols)

Then I print schema
train.printSchema()

root
 |-- label: integer (nullable = true)
 |-- features: vector (nullable = true)

I'm sorry I don't know how to pass the slot names (categoricalSlotNames) or indexes (categoricalSlotIndexes) in the features vector column.
Should I set categoricalSlotNames = ['city_id','minute_num','weekday','hour','level'] or other way?
Look forward to your kind reply.

@imatiach-msft
Thank your help. It's really a folder.
hadoop fs -ls hdfs:///user/root/mymodel/

Found 2 items
-rw-r--r--   3 root supergroup          0 2019-08-27 02:06 hdfs:///user/root/mymodel/_SUCCESS
-rw-r--r--   3 root supergroup       3193 2019-08-27 02:06 hdfs:///user/root/mymodel/part-00000-0b10704c-33b1-44a3-95d2-a876b994d656-c000.txt

I can pull it to my local path now.
I tried to use categoricalSlotIndexes=[0,1,2,3,4] instead of categoricalSlotNames = ['city_id','minute_num','weekday','hour','level'].
But num_cat always equals to 0 in model file.
...
Tree=1
num_leaves=30
num_cat=0
split_feature=...
...
Tree=2
num_leaves=30
num_cat=0
split_feature=...
...

Is this normal?

@imatiach-msft
No, all trees num_cat always equals to 0.
It looks like categorical cols appears in split_features when I set categoricalSlotIndexes=[0,1,2,3,4]. One of the trees is as follows:
Tree=8
num_leaves=30
num_cat=0
split_feature=3 3 0 6 3 3 0 0 0 0 3 0 0 0 0 0 0 0 3 3 1 6 3 6 3 0 3 0 0
split_gain=478.104 258.01 189.769 129.973 103.058 115.921 63.9315 61.8046 52.9455 67.2287 91.9456 52.7235 51.3641 43.6299 46.7856 42.4851 45.8561 153.979 42.3115 42.1639 42.0908 40.3401 38.7992 38.5578 37.1213 35.5186 91.1962 65.1637 53.4649
threshold=262.50000000000006<PHONE_NUMBER>0000003 55.000000000000007 11.500000000000002<PHONE_NUMBER>0000011<PHONE_NUMBER>0000011 65.500000000000014 30.500000000000004 7.5000000000000009 8.5000000000000018<PHONE_NUMBER>0000011 17.500000000000004 40.500000000000007 17.500000000000004 14.000000000000002 17.500000000000004 19.500000000000004 22.500000000000004<PHONE_NUMBER>0000011<PHONE_NUMBER>0000011 3.5000000000000004 5.5000000000000009<PHONE_NUMBER>0000006 4.5000000000000009<PHONE_NUMBER>0000011 23.500000000000004<PHONE_NUMBER>0000011 27.000000000000004 36.500000000000007
decision_type=2 2 2 2 2 2 2 2 2 2 2 2 2 2 2 2 2 2 2 2 2 2 2 2 2 2 2 2 2
left_child=1 7 3 4 5 11 20 -1 -7 19 15 -2 -9 14 22 18 -17 -18 -11 -10 -4 -14 -5 -3 -15 26 -19 -27 -29
right_child=2 23 6 13 -6 8 -8 12 9 10 -12 -13 21 24 -16 16 17 25 -20 -21 -22 -23 -24 -25 -26 27 -28 28 -30
leaf_value=0.051634413620236276 0.096010840038412523 -0.022761394495521101 0.070948174274542031 0.10004132231776995 0.07372192479049243 0.055754191283526169 0.094293895655561843 -0.025165424669015342 0.070627434835935921 0.11792799647130557 0.071086462507774908 0.042911979763515386 0.1004866089259658 -0.014600343739506716 0.21437546550888764 -0.038473538517489858 0.3480416930873903 0.35065661607323217 0.0085297991829141277 -0.042681209059590164 -0.02754721430168848 0.006113952682283613 0.011651658542238865 -0.042638151778643642 0.049028707149292361 -0.043894952663256173 0.019897539973642853 0.12971543673227243 0.010216514647875815
leaf_count=2715 1235 17234<PHONE_NUMBER> 1389 250 2468<PHONE_NUMBER> 4529 491 8271 55 2247 78 71 509 3166 11175 938 2930 27750 479 2138<PHONE_NUMBER>
internal_value=0 -0.240404 0.169942 0.250887 0.357259 0.282676 -0.189669 0.233905 0.151702 0.0894825 0.182815 0.573818 -0.00039888 0.0212736 0.254211 0.0644772 -0.0045828 0.10931 0.576269 -0.306727 -0.236793 0.400078 0.208722 -0.348544 -0.0899014 0.0532624 0.843142 -0.00283213 0.196797
internal_count=100000 51596 48404 36776 24724 21882 11628 6612 16118 14729 11256 5764 3897 12052 3302 9493 8625 6378 868 3473 11378 1429 3247 44984 8750 6300 438 5862 3724
shrinkage=0.1

thanks @giser-yugang for reporting the issue, sent PR here:
#685
it looks like the categorical indexes need to be on the dataset instead of the booster parameters for lightgbm. I've opened a doc defect for lightgbm repo as well to clear up which parameters belong where.

@giser-yugang closing as fix has been merged
https://github.com/Azure/mmlspark/pull/685


go: download go1.22 for darwin/arm64: toolchain not available

Go version
go1.22
Output of go env in your module/workspace:
go: downloading go1.22 (darwin/arm64)
go: download go1.22 for darwin/arm64: toolchain not available

What did you do?
rm -rf /usr/local/go/
rm -rf ~/go

Then used the installer.

Then ran the command go version.
go: downloading go1.22 (darwin/arm64)
go: download go1.22 for darwin/arm64: toolchain not available

go env also returns this result.

What did you see happen?
go: download go1.22 for darwin/arm64: toolchain not available, not much more
What did you expect to see?
go1.22

Thanks for reporting. What's the output of which go and GOTOOLCHAIN=local go env?

I've installed Go with homebrew and am on an Intel mac.
go: downloading go1.22 (darwin/amd64)
go: download go1.22 for darwin/amd64: toolchain not available

$ which go
/usr/local/bin/go

$ GOTOOLCHAIN=local go env
GO111MODULE=''
GOARCH='amd64'
GOBIN=''
GOCACHE='/Users/adam/Library/Caches/go-build'
GOENV='/Users/adam/Library/Application Support/go/env'
GOEXE=''
GOEXPERIMENT=''
GOFLAGS=''
GOHOSTARCH='amd64'
GOHOSTOS='darwin'
GOINSECURE=''
GOMODCACHE='/Users/adam/code/pkg/mod'
GONOPROXY='github.com/moov-io/*,github.com/moovfinancial/*'
GONOSUMDB='github.com/moov-io/*,github.com/moovfinancial/*'
GOOS='darwin'
GOPATH='/Users/adam/code/'
GOPRIVATE='github.com/moov-io/*,github.com/moovfinancial/*'
GOPROXY='https://proxy.golang.org,direct'
GOROOT='/usr/local/Cellar/go/1.21.6/libexec'
GOSUMDB='sum.golang.org'
GOTMPDIR=''
GOTOOLCHAIN='local'
GOTOOLDIR='/usr/local/Cellar/go/1.21.6/libexec/pkg/tool/darwin_amd64'
GOVCS=''
GOVERSION='go1.21.6'
GCCGO='gccgo'
GOAMD64='v1'
AR='ar'
CC='cc'
CXX='c++'
CGO_ENABLED='1'
GOMOD='/Users/adam/code/src/github.com/moov-io/irs/go.mod'
GOWORK=''
CGO_CFLAGS='-O2 -g'
CGO_CPPFLAGS=''
CGO_CXXFLAGS='-O2 -g'
CGO_FFLAGS='-O2 -g'
CGO_LDFLAGS='-O2 -g'
PKG_CONFIG='pkg-config'
GOGCCFLAGS='-fPIC -arch x86_64 -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -ffile-prefix-map=/var/folders/5c/9jlzxrmd1_7_fw3d05nb62dc0000gn/T/go-build2824011357=/tmp/go-build -gno-record-gcc-switches -fno-common'

@adamdecaf Since your Go toolchain was installed using a different tool, it's hard to know if it's the same problem or a different homebrew-specific problem. Let's wait to hear back from @EwenQuim or someone who's able to reproduce this the problem that affects the Go installer from go.dev/dl/.

which go : /usr/local/go/bin/go

GOTOOLCHAIN=local go env

GO111MODULE=''
GOARCH='arm64'
GOBIN=''
GOCACHE='/Users/ewenq/Library/Caches/go-build'
GOENV='/Users/ewenq/Library/Application Support/go/env'
GOEXE=''
GOEXPERIMENT=''
GOFLAGS=''
GOHOSTARCH='arm64'
GOHOSTOS='darwin'
GOINSECURE=''
GOMODCACHE='/Users/ewenq/go/pkg/mod'
GONOPROXY=''
GONOSUMDB=''
GOOS='darwin'
GOPATH='/Users/ewenq/go'
GOPRIVATE=''
GOPROXY='https://proxy.golang.org,direct'
GOROOT='/usr/local/go'
GOSUMDB='sum.golang.org'
GOTMPDIR=''
GOTOOLCHAIN='local'
GOTOOLDIR='/usr/local/go/pkg/tool/darwin_arm64'
GOVCS=''
GOVERSION='go1.22.0'
GCCGO='gccgo'
AR='ar'
CC='clang'
CXX='clang++'
CGO_ENABLED='1'
GOMOD='/dev/null'
GOWORK=''
CGO_CFLAGS='-O2 -g'
CGO_CPPFLAGS=''
CGO_CXXFLAGS='-O2 -g'
CGO_FFLAGS='-O2 -g'
CGO_LDFLAGS='-O2 -g'
PKG_CONFIG='pkg-config'
GOGCCFLAGS='-fPIC -arch arm64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -ffile-prefix-map=/var/folders/5g/q9_gydg96zzd59zgh0hscvqh0000gn/T/go-build2955048168=/tmp/go-build -gno-record-gcc-switches -fno-common'

I don't see anything wrong here, but it might be only my local setup

is this in a module (if yes, what's the go directive in there)?
what's the GOTOOLCHAIN env set to?
what's in /Users/ewenq/Library/Application Support/go/env?

@EwenQuim Thanks.
Can you post more complete steps for how to reproduce it, in an empty directory?
For example, does it reproduce if you do the following sequence?
cd $(mktemp -d)
go version

Is the GOTOOLCHAIN variable set in your environment?
I can only reproduce this if I set GOTOOLCHAIN to explicitly to "go1.22" (which is not a Go toolchain that exists):
$ cd $(mktemp -d)
$ GOTOOLCHAIN=go1.22 go version
go: downloading go1.22 (darwin/arm64)
^C

Otherwise it doesn't reproduce:
$ cd $(mktemp -d)
$ echo $GOTOOLCHAIN

$ go version
go version go1.22.0 darwin/arm64

@seankhliao

is this in a module (if yes, what's the go directive in there)?

No

what's the GOTOOLCHAIN env set to?

Nothing

what's in /Users/ewenq/Library/Application Support/go/env?

OK the problem is solved after I deleted the /Users/ewenq/Library/Application Support/go/env file, thank you very much !
Sorry for taking some of your time, I will explain the cause of the bug below if you're interested, but no need to search more.

Thanks. It's good to understand that the GOTOOLCHAIN=go1.22 line in the go/env file was causing this, and that it's not a problem with the Go installer.
A problem like that is fixable, as you've done, by unsetting the unwanted value with GOTOOLCHAIN=local go env -u GOTOOLCHAIN or by manually editing (or deleting) that file.

The invalid version of GOTOOLCHAIN that was causing me the same issue again and again even after deleting and reinstalling the /usr/local/go directory.

Maybe that updating the go.dev page with more information can be a good idea to prevent future issues like this one ?

Fixed it by editing the go.mod file
~go 1.22~
go 1.22.0
Then Quit and Reopened VS Code

Thanks for that last comment
I think it's very counter intuitive for go version to not output the actual binary version but instead start doing something else that depends on files in the current directory. I can't think of any other unix binary doing that (there is also no debug or verbose I can't see that explains that behavior, like "using go version from go.mod" output somewhere, after printing the actual current binary version
seems you have to do crazy stuff like
DIR=$(mktemp -d); cd $DIR; go version
go version go1.21.8 darwin/arm64

to get the actual go version instead of
$ go version
go: downloading go1.22 (darwin/arm64)
go: download go1.22 for darwin/arm64: toolchain not available

also probably should say "go.mod has go 1.22 which isn't semantic versioning, please correct to 1.22.0"
or fix the behavior so 1.22 means... like it used to... "latest release of 1.22" instead of failing?

you have to do [...] DIR=$(mktemp -d); cd $DIR; go version

If you'd like to determine the bundled Go toolchain's version, it's possible to do it with GOTOOLCHAIN=local go version. This is documented in the last paragraph of https://go.dev/doc/toolchain#select. The rest of that document provides useful context for the current behavior.

also probably should say "go.mod has go 1.22 which isn't semantic versioning, please correct to 1.22.0"
or fix the behavior so 1.22 means... like it used to... "latest release of 1.22" instead of failing?

@ldemailly You're right that this can be improved. This is now tracked in issue #66175. Please feel free to follow it for updates.
Since this issue is closed and there's another open one to track this problem, I'll lock this to avoid further replies pinging many people. If someone would like to report a different problem, please file a new issue at go.dev/issue/new. Thanks.


UX improvements

As discussed here:

[ ] have the currently-focused dataset auto-open
[ ] the inspector does not always update correctly (e.g. node positions) - is it due to scijava (ab)use, or is it a swing issue, or...?
[x] the sidebar folding currently works in a dumb manner, so it doesn't look fully folded on some platforms.

I'm removing the 1.0.0 tag.

The sidebar stuff has been fixed
Auto-open the current IJ dataset is a nice-to-have
There are some other inspector issues in the works for 1.0.0 so I suspect the attribute updates will be addressed by those.


Added support for OCaml source code

What the caption says, I made it track .ml (OCaml source) and .mli (OCaml interface) files.

I think I removed the base repository, so this can't even be merged anymore. Time to move on.


[GHSA-gpw4-7mcw-m8vx] Cross-site Scripting in Jenkins Package Version Plugin

Updates

Affected products
CVSS
Description
Severity

Comments
Update CVSS scoring according to https://www.jenkins.io/security/advisory/2022-06-22/#SECURITY-2784

Automation is borked on the PR for some reason, but got you updated 👍


Missing consistent order of Javadoc tags

According to Oracle, the Javadoc order of tags should be the following:

@author
@version
@param
@return
@exception / @throws
@see
@since
@serial
@deprecated

Other than fixing the existing order, we should have an automated way to enforce it, maybe via Spotless or the maven-javadoc-plugin.

Hey! I've started working on this issue on this PR here, and have mentioned something on the PR and will mention it here just to see where people stand.
In the Oracle Javadoc, it mentions a convention around multiple @throws tags to be in alphabetical order - order of multiple tags. Is this wanting to be implemented? Perhaps it should be a separate issue if it's wanting to go ahead, but figured I'd get some thoughts on it from more knowledgeable folk than myself before deciding what to do with it.

@PhilipIngham thanks for jumping on it!

In the Oracle Javadoc, it mentions a convention around multiple @throws tags to be in alphabetical order - order of multiple tags. Is this wanting to be implemented?

Sounds sensible to me.
Do you already know how to enforce these rules in an automated way?

Do you already know how to enforce these rules in an automated way?

Not really, no. At the moment I'm just going through everything manually using regex to fix the stuff that's out of place in the tags, but I'll definitely need some help and/or time to figure out how to get the automation down.

I was checking what Checkstyle offers and spotted the following:

https://checkstyle.org/checks/javadoc/index.html
https://checkstyle.org/checks/javadoc/atclauseorder.html

Checked this out and looks really useful, I will try and implement it :) Thanks for the assistance


Fade completed challenges in map

Because the text for completed and not completed challenges is the same color and the checkmark is much closer to the text, it is hard to see where you are at a glance.
I suggest either reusing the faded/not faded styling or making the checkmarks more distinct from the text (by color or placement) for easier distinction.

I agree, this would be a good thing to add back in.

Closed by https://github.com/FreeCodeCamp/FreeCodeCamp/pull/6354. Thanks for posting.


Inference Code

Would you consider add the inference code in the future, just like you did in another project https://github.com/Turoad/lanedet  ?
I tried the detect.py in that project, it works.
But when I try detect.py in this project, it failed, and I have no idea how to modify the code in order to fit in this project.
And sorry for reopening an issue with the same kind of problem.

Currently, I'm busy for other projects and I have no plan for adding the inference code.
If you are interested in making a PR, I would be happy to provide some guidance.


left-join destroys row-sort

When I call tablecloth left-join then it adds rows that are not in the right dataset into the end.
I guess this is a bug in tablecloth.
Because left-join should not really affect the sort order.
My example:
calendar: dataset sorted by :date (it only has :date column)
ds-bars: dataset sorted by :date (columns: :open :high :low :close :volume :date)
(defn align-to-calendar [calendar bars]
  (-> (tc/left-join calendar bars :date)
      (tc/order-by [:date] [:asc]) ; this should not be necessary!
      (tc/set-dataset-name (-> bars meta :name))))

:date
:open
:high
:low
:volume
:close

2023-09-01T00:00
137.49
137.5000
134.8500
16185949
135.66

2023-09-05T00:00
135.37
136.4200
134.5801
14948394
135.75

2023-09-06T00:00
136.02
136.5299
133.6650
14601212
134.46

2023-09-07T00:00
133.57
135.5800
132.9500
14062620
135.27

2023-09-04T00:00

135.27

2023-09-08T00:00

135.27

2023-09-11T00:00

135.27

2023-09-12T00:00

135.27

2023-09-13T00:00

135.27

The above table shows September 4 (which does not have a bar) gets added below. So all rows that
do not have the row in the right dataset are in the end of the dataset.

I'll take a look at that today. One question: which version of TMD is on your classpath?

left-join on single column delegates action to tech.v3.dataset.join/left-join so I think it is a problem on TMD side.

Ok, verified in the TMD source code. It looks like expected behaviour. Nothing to do here.


[trivial] Add findSplitBefore example with an element using only()

Also remove typo.
This example is taken from the docs for std.range.only, but is more useful here.

Auto-merge toggled on


How to navigate to the root route?

I'm trying to figure out how to navigate to the root route / and I can't seem to understand how I would do that?

Is router.navigate('/') working for you?

It was adding another / to the end of the current URL. I actually fixed this with router.navigate('/',true)


Backport key redirection test from master branch

Checklist

[x] tests are added or updated

This is a backport of the engine redirection test from #4503

This needs another reviewer.

Merged.  20fe5052baaaba7a0dff02471ab484b576587898
Thank you


Upgrade Thrift to 0.13.0

In Apache Thrift all versions up to and including 0.12.0, a server or client may run into an endless loop when feed with specific input data.
Some of the classes Accumulo relied on (ie AutoExpandingBufferWriteTransport) are no longer public so the utility methods had to be rewritten. Unfortunately there is no TTransport implementation with the same functionality AutoExpandingBufferWriteTransport provided so I had to switch to TMemoryBuffer as a substitute. However this can not be reset and a new object have to be created for each serialization which is less then optimal. I considered adding a custom implementation, but the changes in the deserialize method should compensate for the the loss of performance.

Thanks for the PR, @BukrosSzabolcs . Thrift upgrades are always highly disruptive for us, as they involve (at minimum):

Updates to pom.xml files
Updates to our build scripts to generate and place generated Java code
Remove any obsoleted workarounds we've added due to bugs in earlier Thrift releases
Regenerate and check in new generated Thrift code (and remove any old stuff) (in a separate commit)
Fix / workaround any new bugs caused by changes in Thrift libraries
Ensure upgrades work across Accumulo versions that change the Thrift version
Test, test, test

I think your PR is a good start, but there's lots more to do to be confident about a Thrift version bump, and I'm not sure whether we're going to want to do that for 2.1 yet or not. When we are ready to update, we will certainly include your changes. However, I wanted to offer an explanation in case you don't see this merged right away.

@ctubbsii I fully understand. Thank you for the explanation.

There has been some recent discussion about following through with a thrift upgrade. It seems since work has been done on this PR, thrift 14.1 has been released. Does this seem like an improvement we should make at this time?

There has been some recent discussion about following through with a thrift upgrade. It seems since work has been done on this PR, thrift 14.1 has been released. Does this seem like an improvement we should make at this time?

If somebody does the work required to get us to the newer version, then yes, we should apply it. Such work would supersede this PR.

FYI there is another vulnerability in 0.13.0. So if we are going to upgrade, we will probably want to go to 0.14.0. See https://github.com/apache/accumulo/security/dependabot/pom.xml/org.apache.thrift:libthrift/open

Since this was opened - there have been a couple of thrift release 0.14.2 on 6/17 and more recently 0.15.0 on 9/11.

@BukrosSzabolcs unless you have already done the work, I will take a look at migrating to 0.15.0

I have quickly run into an issue between the versions. The type TFramedTransport was dropped in version 0.14.0. We currently use this class to set the frame size in ThriftUtil and TServerUtils

I was able to get Accumulo to compile with Thrift 0.15.0, with the exception of TFramedTransport, as I mentioned earlier. This class is a problem because it is directly tied to a configuration property, general.server.message.size.max. I emailed the Thrift user list to see if I could get any suggestions as to a replacement or class with similar behavior.

I was able to get Accumulo to compile with Thrift 0.15.0, with the exception of TFramedTransport, as I mentioned earlier. This class is a problem because it is directly tied to a configuration property, general.server.message.size.max. I emailed the Thrift user list to see if I could get any suggestions as to a replacement or class with similar behavior.

It was moved to org.apache.thrift.transport.layered.TFramedTransport so it shouldn't be a problem.

@BukrosSzabolcs I will go ahead and merge your changes here since they get us up to 0.13.0. Then I will make a PR to take us the rest of the way to 0.15.0

@milleruntime Thank you for the merge!

Since this change was not a complete fix, #2273 that takes us to 0.15.0 will need to be a blocker, or else we'll need to revert this. This change alone breaks my ability to work on the Thrift improvements I was making for sending table IDs instead of table names over our RPC, because I can't rebuild the project with -Pthrift without additional changes.


train with multiple gpus

Hi! When I run the code with 2 or 3 gpus, the program will turn into Sl+ process. Do you know the reason?

Hello! I also trained on multiple gpus, but never had this issue. Can you please elaborate on the Sl+ process? I am not familiar with it. Thanks!

The program is stopped and waiting for an event to complete for Sl+ process . Do you use the conmmand such as  CUDA_VISIBLE_DEVICES=0,1 python main.py --model srVAE --network densenet16x32 --prior RealNVP to run the code with cifar10 or celeba? Thanks!

As the model (by default in the main.py file) is wrapped in the nn.DataParallel module, by simply running python main.py --model srVAE --network densenet16x32 --prior RealNVP should activate all available cuda devices (on /src/utils/args.py the program checks automatically if any cuda devices are available).
However, I did use the command CUDA_VISIBLE_DEVICES=1 python main.py --model srVAE --network densenet16x32 --prior RealNVP to specifically choose only one gpu.
Please try the two above commands and check if they work as expected 🙂.

Hi! I want to know if I change the scale_factor=4, do I need to change it to in_channels *= 4 in the red box below? And is there anything else in the code that needs to be changed?
Thanks!
------------------ 原始邮件 ------------------
发件人:                                                                                "ioangatop/srVAE"<EMAIL_ADDRESS>发送时间: 2020年7月16日(星期四) 下午2:54
收件人<EMAIL_ADDRESS>抄送: "コナンの嫁"<2435142508<EMAIL_ADDRESS>主题: Re: [ioangatop/srVAE] train with multiple gpus (#2)
Closed #2.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Add managed plugins

Fixes #61 Make Calico work with experimental docker plugins support in 1.12.
TODO - swarmkit support
TODO - testing
TODO - add to plugin registry
Swarmkit: Calico uses the KV store from the docker flag --cluster-store. But the swarm mode doesn't need a KV store and swarm mode is also incompatible with "--cluster-store".
Building
Network plugin
To build and enable the main network plugin execute:
   cd pluginv2-network
   make dist/calico-network
   sudo make pluginv2

sudo is needed because exporting the plugin rootfs requires root privileges.
The Docker calico-ipam plugin is required in tandem with the calico-network plugin.
IPAM plugin
To build and enable the IPAM network plugin execute:
   cd pluginv2-ipam
   make dist/calico-ipam
   sudo make pluginv2

Usage
Create network
    docker network create --driver=calico-network:latest --ipam-driver=calico-ipam:latest net1

Create workloads
     docker run --net net1 --name workload-A -tid busybox
     docker run --net net1 --name workload-B -tid busybox

Test connectivity
     docker exec workload-A ping -c 4 workload-B.net1
     docker exec workload-B ping -c 4 workload-A.net1

Closing in favor of https://github.com/projectcalico/libnetwork-plugin/pull/146


Not an issue but a question v2

Would this work for linux?

Follow this tutorial : https://tecadmin.net/setup-selenium-chromedriver-on-ubuntu/


Change collection creation behavior again

@danielmewes brought up that a user restoring from a backup would have an unusable horizon database because the tables would have different UUIDs that what was stored in hz_collections.  In order to avoid this, we are dropping UUIDs from the hz_collections table, and modifying the creation process to:

attempt to insert in the hz_collections table
if the insert was successful, create the table

With this process, we should never have a name conflict barring certain (rather obscure) cases, but if one does happen, we will be unable to serve those collections until an operator intervenes.
This branch also addresses issue #708.

This change is

Reviewed 5 of 5 files at r1.
Review status: all files reviewed at latest revision, 3 unresolved discussions.

server/src/metadata/metadata.js, line 15 [r1] (raw file):

const create_collection = (db, name, conn) =>
// TODO: if the row was already there, this should appear to be success

old todo comment

server/src/metadata/metadata.js, line 225 [r1] (raw file):

                  if (collection) {
                    collection._update_table(change.old_val.id, null, this._conn);
                    if (collection._can_be_removed()) {

ok_to_remove vs. can_be_removed

server/src/metadata/metadata.js, line 352 [r1] (raw file):

      collection._on_ready(done);
    }).catch((err) => {
      if (collection._ok_to_remove()) {

vote for ok_to_remove

Comments from Reviewable

Reviewed 2 of 2 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Review status: :shipit: all files reviewed at latest revision, all discussions resolved, all commit checks successful.

Comments from Reviewable


Fix typo in README.md

This PR fixes a typo in the README.md.

Sorry, I missed that this project is "unmaintained". Closing.


Stamen Terrain basemaps broken

Description
Title says it all.  Per the announcement from Stamen, the old Fastly CDN URLs have been shut off for a bit. We have a migration guide that covers the necessary steps to get this back up and running.
TL;DR steps:

Change the URL pattern to something like https://tiles.stadiamaps.com/tiles/stamen_terrain_background/{z}/{x}/{y}{r}.png (note that these tiles have ben updated and support retina/hidpi now!)
Update attribution / license information as noted (Stamen, Stadia Maps, and OpenMapTiles need attribution + links now).

As this is an OSM community public service project, we can sponsor the usage. No need for anyone at Meta to create an account; we'll get the rapideditor.org domain  into our system sometime next week.
I am also happy to open a PR to do the implementation if you'd like.
Screenshots

Version
2.1.1
What browser are you seeing the problem on? What version are you running?
Brave 1.60.118
The OS you're using
macOS 14.1.1
Steps to reproduce

Open the editor
Select "Stamen Terrain" from the background tab

The browser URL at the time you encountered the bug
https://rapideditor.org/edit#background=stamen-terrain-background&datasets=fbRoads,msBuildings&disable_features=boundaries&map=19.00/37.56595/126.97522
The auto-detected useragent string for your browser (leave blank if you're manually filling this form out)
No response

Hey @ianthetechie - we use the editor-layer-index project to build our imagery sources- so instead of making the change here in Rapid, you can make it there and we'll pick the change up whenever the index gets rebuilt.
I believe the file in question is https://raw.githubusercontent.com/osmlab/editor-layer-index/e66a8077d3ace8a6dcdbd9060db4f5199d74e95f/sources/world/StamenTerrainBackground.geojson - unsure if there are others that need modifying

Thanks @ianthetechie for the heads up!  🙌
I fixed the Stamen Terrain Background layer by adding these lines to manual_imagery.json - this overrides the imagery that we are pulling from Editor Layer Index with the updated URL and Attribution.
https://github.com/facebook/Rapid/blob/f6b58ab998805a560eea343acd2c53fb34936aeb/data/manual_imagery.json#L50-L63


internal: move diagnostics to hir

Hmm yes it goes in the direction, but I wouldn't have expected the diagnostics to just move to hir, rather than all the way to ide? I think my vision is basically how the field shorthand diagnostic is already implemented, actually. But there would have to be an interface for the various kinds of diagnostics in hir, of course :thinking:

Argh, I think I shifted one time to much. This moves diagnostic infra to hir_ty. I want to move it to hir as the end state, not to the ide. The fixes will leave in the ide though.
Not sure where most of diagnostic tests should live: I kinda want all tests to belong to crates actually emitting the specific diagnostics, but it certainly is easier to test at the level where you have generic infra.

I thought I wrote that badly :sweat_smile: I'd have expected diagnostics infra to move to ide in the end. I'd see testing for when diagnostics are emitted in the respective crates, and testing diagnostic descriptions and quickfixes in ide.

So what I think we want in the end is

hir_xxx emit raw diagnostics, something like DefDiagnostic which has local id in it as a public filed.
in hir, we re-formulate that it terms of hir-types. We go from (potentially local) ids to global unique ids. This step doesn't involve traits -- just translating one struct into another
in ide_diagnostics, we have have traits like DiagnosticWithFix and Diagnostic, which we implement for hir types. We might even end-up not using traits there, as there's no real need (and definitelly no dynamic dispatch)
ide_diagnostics also implements a bunch of lints which do not need privileged access to compiler data structures

In terms of diff with this PR, the type definitions in hir::diagnostic stay, while impl Diagnostic for go away.

Agree, I think we probably don't need the traits.

bors r+


about dataset

Thank you for release the data . I notice that there are ’title‘ and ’content' in the dataset. do you use  both and cnoncate them during training or just use only one of them?

During training, I randomly select the text from "title" and "content".

During training, I randomly select the text from "title" and "content".

During training, I randomly select the text from "title" and "content".

During training, I randomly select the text from "title" and "content".

I see. Thx

During training, I randomly select the text from "title" and "content".

what is the sample ratio of them.


No bootable device found

This app is not working on my MacBook Air (2013 model) with Windows 8.1 installed. It just gives me an error after restart "No bootable device found. Press any key to restart." Booting into Windows works fine when choosing as startup disk in Mac OS or when holding down option key at startup.

Having this issue too, but don't think it is bootchamp.  If I choose the window partition from start disk and reboot I get the same message.  If I hold down the option key I don't see the windows volume just EFI boot, choosing that successfully boots to Windows...


Test Data Gathering

Find out where to get test data?

Where to get possible training/testing data for the LLM of choice?

avoindata.fi

ChatGPT prompt:
I am creating an LLM-based text classifier. The purpose of this project is to input text-based data and have the program analyse and classify themes and other things from the data. The program will run in cloud and will be created using Python, and will also use Docker.
What kind of data would be best for testing such a program? What format should the data be in? How much data is enough for tests?
Answer:
For testing your LLM-based text classifier, selecting the right data, format, and quantity is crucial for evaluating its performance. Here are some guidelines:
1. Types of Data for Testing
The best testing data depends on the nature of your text classification tasks, but here are some commonly used types:

Labeled Dataset: You'll need a dataset with clear labels that correspond to the themes or categories you are classifying. For example, if you're classifying news articles by topic (e.g., sports, politics, entertainment), each text sample should have a corresponding label.
Diverse Topics: The data should cover a variety of themes to test the versatility of your classifier. Ensure it includes various domains such as articles, emails, social media posts, etc., depending on your project's focus.
Balanced Data: Ensure that your dataset is balanced, meaning each category has a sufficient number of examples to prevent model bias toward any specific class.
Real-World Data: Try using real-world, unstructured text data to see how well the model handles the noise, such as typos, slang, or abbreviations, common in user-generated content.

Sources:

Public datasets: Datasets like AG News, IMDB reviews, 20 Newsgroups, or Sentiment140 can be good starting points.
Custom data: Collect domain-specific data relevant to your use case (e.g., customer reviews, product descriptions, research papers).

2. Format of the Data
The most common format for text classification tasks is plain text files with corresponding labels. Here are a few formats:

CSV: Each row contains a text sample and a label.
Example:
text,label
"The new smartphone model has excellent features.", "tech"
"The stock market saw significant growth today.", "business"

JSON: Useful for more complex structures where you may need to include additional metadata.
Example:
[
  {"text": "The new smartphone model has excellent features.", "label": "tech"},
  {"text": "The stock market saw significant growth today.", "label": "business"}
]

TXT: Plain text files where each line contains the text followed by its label, often used for simple applications.

Hugging Face Dataset Format: If you're using libraries like Transformers by Hugging Face, you can directly load their datasets in a structured format.

Recommendation: Use CSV or JSON for initial testing, as they are human-readable and easy to manipulate in Python.
3. How Much Data Is Enough for Tests?

Small-Scale Testing: For initial testing of your infrastructure and code, even a small dataset (500–1,000 samples) can help you evaluate whether the system works.
Benchmark Testing: To properly evaluate your classifier’s performance, you'll need a moderate dataset (5,000–50,000 samples) to measure accuracy, precision, recall, and F1 scores.
Stress Testing: For scaling tests or production-level deployment, larger datasets (50,000+ samples) are beneficial to assess how well the classifier handles large volumes of data.

General Rule:

Start small for testing functionality.
Gradually increase the dataset size to assess performance, efficiency, and accuracy.

4. Balancing Data for Testing
Ensure you have an even distribution of samples across all categories to avoid skewing model performance toward any single class. For instance, if you're classifying text into five categories, aim for a roughly equal number of examples in each category. If your dataset is imbalanced, you might consider techniques like oversampling or using metrics like weighted accuracy during evaluation.
Conclusion

Data Type: Labeled datasets with diverse themes relevant to your classification task.
Format: CSV or JSON formats are preferred for ease of use.
Data Size: Start with a small dataset for debugging, but aim for at least 5,000 samples for meaningful performance tests, and 50,000+ samples for stress testing.

Data that could be good for software testing:
https://www.avoindata.fi/data/fi/dataset/eduskuntavaalien-2015-ylen-vaalikoneen-vastaukset-ja-ehdokkaiden-taustatiedot (13MB)
https://www.avoindata.fi/data/fi/dataset/helsingin-vapaa-aikakyselyn-vastaukset (2,4MB)
https://www.avoindata.fi/data/fi/dataset/turun-kaupungin-turvallisuuskysely (1,3MB)

This for larger datasets: https://www.kaggle.com/datasets


Good and bad ending

Describe your problem here.
I want to make a good or bad ending based on the mises of the song like sky, using the stage file
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.


Switch statements

Made one question and answer for switch statements and would accept some feedback. Also changed the numbers on the .md file for Structs practice and answers because I was using them for reference. Probably just needs a merge on those two.

Merged.


validate usernames

Fixed in: https://github.com/liam-middlebrook/open-container/commit/fb798585eb1aaad0737d130ba001a8d7e2869e18


SoapySDR::Device::setGain has undesired behavior for SDRPlay

SoapySDR assumes that gains are additive in dB. When Device::setGain is called without a name, it automatically distributes the requested gain between all available gain stages:
https://github.com/pothosware/SoapySDR/blob/master/lib/Device.cpp#L281
Unfortunately, SDRPlay gains are unconventional. There are two gains exposed via SoapySDR, called "RFGR" and "IFGR".

"IFGR" stands for "intermediate frequency gain reduction". It takes a value between 20 and 59. Contrary to what SoapySDR expects, the greatest amplification is obtained with the smallest numerical value. The IFGR has units of dB.
"RFGR" stands for "radio frequency gain reduction". It takes a value between 0 and 9. As with the IFGR, the greatest amplification is obtained with the smallest value. In addition, the RFGR does not have units of dB! The actual gain reduction is nonlinear and frequency-dependent, but tends to be about 7 dB per count.
(a) Note that RFGR is also settable via the "rxgain_sel" menu item.
(b) RFGR attenuation tables in dB are available in the SDRPlay API documentation, section 5.3.
(c) Most SDRPlay technical documents refer to the RFGR as the LNA state. I am not sure which terminology is preferred.

When we put these unconventional gain definitions together with SoapySDR's default gain distribution algorithm, disaster follows. Nearly any call to
void SoapySDR::Device::setGain(const int, const size_t, const double)
will force the RFGR to 9 (more than 60 dB of attenuation!) before allocating the remainder of the requested gain to the IFGR. Now you might argue that SDRPlay users should always set their gains by name -- but unfortunately, certain unenlightened applications do not expose the individual named gains, and there may be no way to avoid a call to the automatic gain distribution algorithm.
Proposed solution 1: quick and easy

Override the default SoapySDR implementation of SoapySDR::Device::setGain(const int, const size_t, const double). Rather than attempting to distribute gain between stages, the requested gain will be assigned directly to the IFGR and the RFGR will not be changed.
For consistency, also override SoapySDR::Device::getGainRange(const int dir, const size_t channel) to exclude the RFGR.
SDRPlay's automatic gain control works in much the same way -- it varies the IFGR but does not touch the RFGR.

Proposed solution 2: fix the abstraction, break some existing configurations

Remove RFGR from the list of available gains. It will remain available through the "rfgain_sel" menu item (which perhaps should be renamed to "LNA_state").
Invert the sign of IFGR in SoapySDR, so that it spans from -20 to -59 rather than +20 to +59. This will match the SoapySDR sign convention of a bigger number representing more amplification, while preserving a clear correspondence with the numerical IFGR values present in SDRPlay documentation.
With these abstractions fixed, setGain and getGainRange will now do the right thing without any further changes.

I am very new to SoapySDR (and SDR in general, for that matter). I would be happy to send in a pull request, but first I'd appreciate some feedback from the maintainers regarding the preferred approach.
Huge thanks to SDRPlay support for helping me understand what is going on in the hardware... responding in 15 minutes at midnight on Saturday!

I am very new to SoapySDR (and SDR in general, for that matter). I would be happy to send in a pull request, but first I'd appreciate some feedback from the maintainers regarding the preferred approach.

Sounds great, thats exactly what overriding setGain is for. A PR would be very helpful, and gives us something specific to discuss.

@guruofquality Thanks for the response! I've opened #61 with the larger set of changes to bring SoapySDRPlay in line with the SoapySDR gain abstraction. I think this is the best long-term solution.
A simpler change would be to keep the non-standard RFGR and IFGR, and override setGain to ensure that RFGR is never changed. I think this solution is more confusing in the long term, and it's not consistent with the behavior of other SoapySDR hardware interfaces.
https://github.com/pothosware/SoapySDRPlay/compare/master...dlaw:master
The argument I see for the simpler solution would be concern over breaking existing configurations. But if you agree, I'd prefer to move ahead with the changes as sketched out in #61.


Archive one bibeasy repos

I also have a bibeasy repos-- we need to make sure that one of the repos is archived so as to avoid parallel dev

Done


In markdown mode, auto linkify URLs as in GitHub

Version
2.5.1
Development Environment

OS: Ubuntu
Browser Chrome
Version Version 87.0.4280.141 (Official Build) (64-bit)

Current Behavior
When adding any URL is added to the markdown editor, it needs manual markdown tags addition
Expected Behavior
Not expected, but better to make it like the GitHub where if the URLs are not linkified through markdown tags, the link is automatically linkified.

@rrjanbiah I think the feature you are looking for is referring to the extendedAutolinks option. Would you like to check it out?
const editor = new Editor({
  el: document.querySelector('#editor'),
  extendedAutolinks: (content) => {
    const matched = linkify.match(content);
    if (matched) {
      return matched.map(matches => {
        const { text, url, index, lastIndex } = matches;
        const range = [index, lastIndex];
        return { text, url, range };
      });
    }
    return null;
  }
});

@seonim-ryu Thanks for sharing the new feature.
With the above code, I get Uncaught ReferenceError: linkify is not defined error. (I used the latest CDN https://uicdn.toast.com/editor/latest/toastui-editor-all.min.js)
I then stumbled upon this documentation https://github.com/nhn/tui.editor/blob/d3b7bf0f53364211ebff96950d0e6331da05565b/apps/editor/docs/extended-autolinks.md and based on that I tried extendedAutolinks: true. It is working good for getHtml(), but NOT for getMarkdown()


Add test case test_migration_with_unscheduled_replica

Signed-off-by: Chris Chien<EMAIL_ADDRESS>Add test case test_migration_with_unscheduled_replica

@chriscchien Please backport to 1.3 and 1.2. Thanks.


Using syscalls and ld script

I made a projet that produces 159-bytes ELF64 AMD64 binary:
https://github.com/cbytensky/helloworld_tiny

@cbytensky Looks nice! If you could add some more information to the README about the limitations (OS/portability, etc) and some details about why this reduces binary size (no std or libc bindings etc) I'll add your repo as a reference.

Get rid of main. Not the Rust’s main but libcs main`. It’s achived by:

In main.rs instead of fn main():

pub extern fn _start() {

In .cargo/config file in project dir:

[build]
rustflags = ["-C", "link-arg=-nostartfiles"]

If there are custom build script in your project or in any crates that your project uses, you shoud also add this line to [build] section:
target = "x86_64-unknown-linux-gnu"

So .cargo/config will be:
[build]
target = "x86_64-unknown-linux-gnu"
rustflags = ["-C", "link-arg=-nostartfiles"]

Some examples you could see in https://github.com/cbytensky/helloworld_libc

2. Use custom build script.

3. Using syscalls instead of libc
Example here https://github.com/cbytensky/helloworld_syscall

@cbytensky This is great. Could you add a central README that summarizes this (or a gist) that I could link too that discusses all of your techniques and perhaps has other links to your various subprojects.
Ideally, I'd like to see a small discussion about the tradeoffs/limitations of these methods as well.
When that's done, I'd be happy to add a link to this in the references section of this repo.


Saved houseprint can't be unpickled by python3

For testing, we have a saved housprint object /tests/test_saved_hp.hp.  In python2, this can be unpickled without problem, in Python3 this leads to a unicodedecode error.  I tried to modify the unpickling as follows:
    with open(filename, 'rb') as f:
        if sys.version_info.major == 3:
            hp = pickle.load(f, encoding='bytes')
        else:
            hp = pickle.load(f)

It loads, but the resulting object is not correct and the subsequent tests fail.
I don't have a python3 machine configured to test.  @JrtPec maybe you can solve this?  Else we need to get rid of the pickle thing and use a json format for saving/loading houseprint objects?

I suspect it might be because you have pickled the object using Python2 and tried to unpickle it using Python3? Have you tried it with an object that was pickled in Python3?

You are probably right.  I don't have an object pickled with python3
however.  I'll try to create one and test
On Thu, Nov 19, 2015 at 7:26 PM, Jan Pecinovsky<EMAIL_ADDRESS>wrote:

I suspect it might be because you have pickled the object using Python2
and tried to unpickle it using Python3? Have you tried it with an object
that was pickled in Python3?
—
Reply to this email directly or view it on GitHub
https://github.com/opengridcc/opengrid/issues/78#issuecomment-158147025.

I solved the issue the hard way: by using jsonpickle instaed of pickle.  Works perfectly!


Solves Issue #2

This is a bunch of changes to the install script.

Change apache2.conf to accept .htaccess override.
Create a database and user for Drupal in MySQL.
Use Drush to download and install Drupal 8.

Pretty much solves issue #2 as far as I can tell.

Looks great! Closes issue #2


Create johnsmith.toml

Description
All previous genesis validators should name their PRs "Update {validator_alias}.toml for tesntet-15" and provide links to previous PRs merged.
If this is an UPDATE for a previous genesis validator
Checks are made against your net_address. If this has changed since the previous testnet, make sure you provide links of previous prs merged from your previous git username.
Thanks in advance!
Checklist before merging

[ ] Only one toml is added in this PR
[ ] The file being added is indeed a .toml file
[ ] The toml being added is to the correct folder, and only to the correct folder
[ ] The eth_hot_key and eth_cold_key are present
[ ] The email, discord, elements telegram, and twitter fields are present and valid

This PR was not done correctly. Please open a new pr with only one transactions.toml

I would suggest reforking or pulling before pushing

yep I have submitted https://github.com/anoma/namada-testnets/pull/3098 which seems like have been accepted


Support for #2 - nextTick / setImmediate

Added helper to use nextTick, setImmediate, or fall back on setTimeout.

v1.1.0 :tada:


cert-manager ServiceMonitor picks up pods that shouldn't

Due to the labels and selector we are using, the ServiceMonitor included with the module is picking up other pods that should not be scrapped for metrics:

Pods "CA Injector" and "Webhook" should not be scrapped for metrics.

What do you think of adding one more label, like component: { controller, cainjector, webhook } and add component: controller to the ServiceMonitor selectors?

yeap, that should be the way to do it.
But, I'm pretty sure that we are not following upstream labels for some reason and just setting global labels with kustomize, I'd like to take a moment a compare with upstream and if this does not happen with their version maybe align our manifests with them.


Bugfix release V0.20.2

Context: This PR fixes a bug observed with the adjoint Jacobian method and a missing explicit kernel, where a decomposition was not performed.
Description of the Change: An explicit kernel for the CY gate is added.
Benefits: Fixes an issue when using the CY gate as part of the IsingYY operator with the adjoint method.
Possible Drawbacks: None
Related GitHub Issues:

NOTE: do not update this branch against master!


Ensure ContextSwitch has the same memory layout on all platforms.

Fix issue when using Windows debug UI with Linux service by
rearranging class members and defining the enum type.

LGTM!


[NTR] lateral ganglionic eminence derived neuron

Please check that the term does not already exist by using the ontology search tool OLS:
https://www.ebi.ac.uk/ols4/ontologies/cl
Preferred term label
lateral ganglionic eminence derived neuron
Synonyms (add reference(s), please)
Definition (free text, with reference(s), please. PubMed ID format is PMID:XXXXXX)
Neuron derived from precursor cell in the lateral ganglion eminence.
https://en.wikipedia.org/wiki/Ganglionic_eminence#Lateral_ganglionic_eminence_(LGE)
Parent cell type term (check the hierarchy here https://www.ebi.ac.uk/ols4/ontologies/cl)
Anatomical structure where the cell type is found (check Uberon for anatomical structures: https://www.ebi.ac.uk/ols4/ontologies/uberon)
Your ORCID
Additional notes or concerns
Needed as general grouping term.

taking this to test out workflow on a VM with ODK installed :)


T&S link is broken.

Issue is not actually in the app, see https://github.com/threefoldtech/home/issues/1191
Next app update we'll update link to https://library.threefold.me/info/legal/#/

Fixed in settings, not fixed in wizard. I assume wizard is a separate deploy? Will we make sure we don't forget this?

WIzard is indeed seperate deploy


ByBit Websocket Support

Can you implement this in the current code?
https://bybit-exchange.github.io/docs/inverse/#t-websocket

Yes, but it will take a while to complete it, I will @you

Hey can you implement this ?

any updates ?


SQLServiceAccount

Details of the scenario you tried and the problem that is occurring
I am trying to set the service account and it fails with he following exception occurred while trying to enumerate the collection: "The RPC server is unavailable. (Exception from HRESULT: 0x800706BA)".
PowerShell DSC resource MSFT_SqlServiceAccount  failed to execute Test-TargetResource functionality with error message: System.Exception: The DatabaseEngine service on Node.Nodename\MSSQLSERVER
could not be found.
The firewall is temporarily off
Verbose logs showing the problem
The RPC server is unavailable. (Exception from HRESULT: 0x800706BA)".
PowerShell DSC resource MSFT_SqlServiceAccount  failed to execute Test-TargetResource functionality with error message: System.Exception: The DatabaseEngine service on Node.Nodename\MSSQLSERVER
could not be found.
Suggested solution to the issue
The DSC configuration that is used to reproduce the issue (as detailed as possible)
# insert configuration here

SqlServiceAccount SetServiceAccount_User
            {
            ServerName     = 'Node.Nodename'
            InstanceName   = 'MSSQLSERVER'
            ServiceType    = 'DatabaseEngine'
            ServiceAccount = $Credential
            RestartService = $true
            }

#### SQL Server edition and version the target node is running
2016 developer

#### SQL Server PowerShell modules present on the target node
Import-DscResource –Module PSDesiredStateConfiguration
    Import-DscResource -Module SqlServerDsc
    Import-DscResource -Module xPSDesiredStateConfiguration
    Import-DscResource -Module xSQLServer
    Import-DscResource -Module xNetworking
    Import-DscResource -Module xCertificate

#### The operating system the target node is running
windows 2016

#### Version and build of PowerShell the target node is running
5.1

#### Version of the DSC module that was used ('dev' if using current dev branch)

These service are enabled and running Windows Management Instrumentation, Remote Procedure Call (RPC), TCP/IP NetBIOS Helper.

Oops my bad it was a typo in serverName

Oops my bad it was a typo in serverName


Improved README

I've run into several issues when I was configuring this component in our project.
This should save some time to future generations ;-)
Cheers!

:+1:

thanks


What should the specificity for tRNA methylation be?

to capture the processes of
https://github.com/geneontology/go-ontology/issues/17665
I'd be happy with a general term like
tRNA guanine methylation
and
tRNA cytosine methylation
although these dont exist. However these do:
tRNA (guanine-N7)-methylation
tRNA C5-cytosine methylation
tRNA N1-guanine methylation
tRNA N2-guanine methylation
etc

@ukemi Would it make sense for this to be done with
https://github.com/geneontology/go-ontology/issues/17665
by Harold?

I meant to do that!

This seem ok; see ticket #17665