Support MySQL 8 client libraries

Just filing this as a tracker because I'm sure it's already on your list, but as of right now, the libmysqlclient-dev package that got automatically installed on my Ubuntu 20.04 causes the MySQL build of this test to fail. The reason for that is that the default version is now 8.0.29-0ubuntu<IP_ADDRESS>.
I need to manually downgrade to libmysqlclient20_5.7.38-1ubuntu18.04_amd64.deb and libmysqlclient-dev_5.7.38-1ubuntu18.04_amd64.deb to allow make to finish.
Build output when failing was as follows:
~/taobench$ make
[  8%] Building CXX object CMakeFiles/benchmark.dir/mysqldb/mysql_db.cc.o
In file included from /usr/include/superior_mysqlpp/connection_def.hpp:16,
                 from /usr/include/superior_mysqlpp/connection.hpp:8,
                 from /usr/include/superior_mysqlpp.hpp:9,
                 from /home/ubuntu/taobench/mysqldb/mysql_db.h:10,
                 from /home/ubuntu/taobench/mysqldb/mysql_db.cc:1:
/usr/include/superior_mysqlpp/low_level/dbdriver.hpp:957:37: error: ‘MYSQL_OPT_USE_REMOTE_CONNECTION’ was not declared in this scope
  957 |               useRemoteConnection = MYSQL_OPT_USE_REMOTE_CONNECTION,
      |                                     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/usr/include/superior_mysqlpp/low_level/dbdriver.hpp:958:39: error: ‘MYSQL_OPT_USE_EMBEDDED_CONNECTION’ was not declared in this scope
  958 |               useEmbeddedConnection = MYSQL_OPT_USE_EMBEDDED_CONNECTION,
      |                                       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/usr/include/superior_mysqlpp/low_level/dbdriver.hpp:959:33: error: ‘MYSQL_OPT_GUESS_CONNECTION’ was not declared in this scope; did you mean ‘MYSQL_OPT_RECONNECT’?
  959 |               guessConnection = MYSQL_OPT_GUESS_CONNECTION,
      |                                 ^~~~~~~~~~~~~~~~~~~~~~~~~~
      |                                 MYSQL_OPT_RECONNECT
/usr/include/superior_mysqlpp/low_level/dbdriver.hpp:960:29: error: ‘MYSQL_SET_CLIENT_IP’ was not declared in this scope; did you mean ‘MYSQL_SET_CHARSET_DIR’?
  960 |               setClientIp = MYSQL_SET_CLIENT_IP,
      |                             ^~~~~~~~~~~~~~~~~~~
      |                             MYSQL_SET_CHARSET_DIR
/usr/include/superior_mysqlpp/low_level/dbdriver.hpp:961:38: error: ‘MYSQL_SECURE_AUTH’ was not declared in this scope; did you mean ‘MYSQL_DEFAULT_AUTH’?
  961 |               secureAuthentication = MYSQL_SECURE_AUTH,
      |                                      ^~~~~~~~~~~~~~~~~
      |                                      MYSQL_DEFAULT_AUTH
/usr/include/superior_mysqlpp/low_level/dbdriver.hpp:964:44: error: ‘MYSQL_OPT_SSL_VERIFY_SERVER_CERT’ was not declared in this scope; did you mean ‘CLIENT_SSL_VERIFY_SERVER_CERT’?
  964 |               sslVerifyServerCertificate = MYSQL_OPT_SSL_VERIFY_SERVER_CERT,
      |                                            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
      |                                            CLIENT_SSL_VERIFY_SERVER_CERT
In file included from /usr/include/superior_mysqlpp/dynamic_prepared_statement.hpp:18,
                 from /usr/include/superior_mysqlpp/connection.hpp:12,
                 from /usr/include/superior_mysqlpp.hpp:9,
                 from /home/ubuntu/taobench/mysqldb/mysql_db.h:10,
                 from /home/ubuntu/taobench/mysqldb/mysql_db.cc:1:
/usr/include/superior_mysqlpp/prepared_statements/initialize_bindings.hpp: In function ‘constexpr void SuperiorMySqlpp::detail::initializeNullable(MYSQL_BIND&, SuperiorMySqlpp::Nullable<T>&)’:
/usr/include/superior_mysqlpp/prepared_statements/initialize_bindings.hpp:52:48: error: ‘my_bool’ does not name a type; did you mean ‘bool’?
   52 |             binding.is_null = reinterpret_cast<my_bool*>(&nullable.detail_getNullRef());
      |                                                ^~~~~~~
      |                                                bool
/usr/include/superior_mysqlpp/prepared_statements/initialize_bindings.hpp:52:55: error: expected ‘>’ before ‘*’ token
   52 |             binding.is_null = reinterpret_cast<my_bool*>(&nullable.detail_getNullRef());
      |                                                       ^
/usr/include/superior_mysqlpp/prepared_statements/initialize_bindings.hpp:52:55: error: expected ‘(’ before ‘*’ token
   52 |             binding.is_null = reinterpret_cast<my_bool*>(&nullable.detail_getNullRef());
      |                                                       ^
      |                                                       (
/usr/include/superior_mysqlpp/prepared_statements/initialize_bindings.hpp:52:56: error: expected primary-expression before ‘>’ token
   52 |             binding.is_null = reinterpret_cast<my_bool*>(&nullable.detail_getNullRef());
      |                                                        ^
/usr/include/superior_mysqlpp/prepared_statements/initialize_bindings.hpp:52:88: error: expected ‘)’ before ‘;’ token
   52 |             binding.is_null = reinterpret_cast<my_bool*>(&nullable.detail_getNullRef());
      |                                                                                ^
      |                                                                                )
In file included from /usr/include/superior_mysqlpp/dynamic_prepared_statement.hpp:20,
                 from /usr/include/superior_mysqlpp/connection.hpp:12,
                 from /usr/include/superior_mysqlpp.hpp:9,
                 from /home/ubuntu/taobench/mysqldb/mysql_db.h:10,
                 from /home/ubuntu/taobench/mysqldb/mysql_db.cc:1:
/usr/include/superior_mysqlpp/prepared_statements/prepared_statement_base.hpp: At global scope:
/usr/include/superior_mysqlpp/prepared_statements/prepared_statement_base.hpp:325:44: error: ‘my_bool’ has not been declared
  325 |             void setUpdateMaxLengthOnStore(my_bool value)
      |                                            ^~~~~~~
/usr/include/superior_mysqlpp/prepared_statements/prepared_statement_base.hpp: In member function ‘bool SuperiorMySqlpp::detail::PreparedStatementBase<storeResult, validateMode, warnMode, ignoreNullable>::getUpdateMaxLengthOnStore()’:
/usr/include/superior_mysqlpp/prepared_statements/prepared_statement_base.hpp:335:17: error: ‘my_bool’ was not declared in this scope; did you mean ‘bool’?
  335 |                 my_bool result = 0;
      |                 ^~~~~~~
      |                 bool
/usr/include/superior_mysqlpp/prepared_statements/prepared_statement_base.hpp:336:106: error: ‘result’ was not declared in this scope
  336 |                 this->statement.getAttribute(LowLevel::DBDriver::Statement::Attributes::updateMaxLength, result);
      |                                                                                ^~~~~~
make[2]: *** [CMakeFiles/benchmark.dir/build.make:193: CMakeFiles/benchmark.dir/mysqldb/mysql_db.cc.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:76: CMakeFiles/benchmark.dir/all] Error 2
make: *** [Makefile:84: all] Error 2

Actually I am little surprised here, I think Meta internally uses MYSQL for tao backend and I was expecting maximum support/tunings for MYSQL.

While Meta does use MySQL to support TAO, we focus on the TAO workload here, so we don't provide tuning for the underlying storage layer.


Turkey test fix

Replaced all String’s methods toUpperCase() to
toUpperCase(Locale.ENGLISH).
Added turkishLocale profile to check whether app passes “Turkey test”.
#16

rather than hardcoding the locale, should it be part of the CsvPreference-class?

In this case, ENGLISH locale prevents from consequences of using default (system) locales. This is a part of algorithm "how to properly do upper case". If we extract it to configuration maybe some users who do not know what it really means will set it to Turkish locales.
This change does not solve the problem because like I wrote in previous post equalsIgnoreCase solves problem. We can revert this change and everything will be OK. By this change I wanted explicitly  say - "convert it without worrying about local characters".
Please, let me know what do you think about it and how it should be solved properly.

would you mind combining these using  git reset --soft head~3 ? I dont min small commits but these ones are going backwards and forward changing the same lines :) Also we should add a comment on the equalsIgnoreCase() telling its important to ignore the casing for a number of reasons, one being the turkey-issue

Actually I combined these 3 commits into one using your tip(thanks for it!) but I do not know how to override my PR. I've never done this before...

I have found an answer - just add force flag helped ;)


bootstrap controller

This is the bootstrap controller

installs secrets
installs the staging packages to the workload

/approve

/lgtm


Fix secrets cli

Fixes encoding issues when creating gpg secrets using the cli (#150)
Breaking change: removes --backend in secrets subcommand. New format for secret token in cli is prefix with backend name (now referred as type_name) e.g. gpg:path/to/secret1
Refactor of cli code into smaller functions easier to test.

CLA signed. A re-test should return a clean result.

CLA updated


Incorrect Validation Query in DB2DataSourceCreator

When trying to use the validation query while connecting to DB2, an error "Failed to validate a newly established connection" is produced. This is caused by the validation query "VALUES 1". A tested and recommended fix for this would be to change the validation query to "SELECT 1 FROM sysibm.sysdummy1".

This is a duplicate of #187. No changes were made in response to that issue because I could not get confirmation that one validation query is appropriate for all DB2 configurations. VALUES 1 seems to work for some users, including the person who contributed the original code. Is SELECT 1 FROM sysibm.sysdummy1 guaranteed to work for all DB2 configurations? Is this specific to the version of DB2? The change is simple to make, but I'd like to be sure it doesn't help some users while hurting others.
In the interim, a workaround was provided here: https://github.com/spring-cloud/spring-cloud-connectors/issues/187#issuecomment-277292931.

Closing this issue for lack of feedback and a viable workaround.


[Bug] Button slways shows "This interaction failed" even if there is no error raised and the interaction worked.

Excpected Action

Respond to the interaction without an error

Actual Action

Shows under the button "This interaction failed" every time even when no error is raised

Reproducing the action
await ctx.send(embed= em, components=[
    Button(label= "Clear Kicked List", style=4, id="clear_kicked_button"),
  ])

      while True:
        res = await client.wait_for("button_click")
        if res.component.id == "clear_kicked_button":
          print("clicked kicked")
          with open("kicked_users.json", "r") as f:
            kicked_users = json.load(f)

            if kicked_users == "{}":
              em = Embed(title="Aucun utilisateur n'a été kick", color= 0xFF5BF8)
              await ctx.send(embed= em)
            else:
              kicked_users.clear()

              confem = Embed(title= "La liste de Kick a été clear", color= 0xFF5BF8)
              await ctx.send(embed= em)

          with open("kicked_users.json", "w") as f:
            json.dump(kicked_users, f, indent=4)

Environment

Python 3.8.2

Check

[x] Are you using the newest version?
[ ] Does this bug occurs security issues?

I think you didn't responded
edit await ctx.send(embed = em to await res.respond(ephemeral = False, embed = em)

Thank youuuu it solved my problem


[TA] Implementation of 2022-03-01

API change check for com.azure:azure-ai-textanalytics
API changes have been detected in com.azure:azure-ai-textanalytics. You can review API changes here
API changes
-         V3_2_PREVIEW_2("v3.2-preview.2");
+         V2022_03_01("2022-03-01-preview");


[Event Request] - Report 7318

We are using BC 14.0.35570.
In report 7318 (Whse.-Shipment - Create Pick), in the OnAfterGetRecord trigger of the "Warehouse Shipment Line" dataitem, we need the ability to add more filters to the WhseWkshLine variable. An event something like this:
Warehouse Shipment Line - OnAfterGetRecord()
IF Location."Directed Put-away and Pick" THEN
  CheckBin(0,0);

WhseWkshLine.RESET;
WhseWkshLine.SETCURRENTKEY(
  "Whse. Document Type","Whse. Document No.","Whse. Document Line No.");
WhseWkshLine.SETRANGE(
  "Whse. Document Type",WhseWkshLine."Whse. Document Type"::Shipment);
WhseWkshLine.SETRANGE("Whse. Document No.",WhseShptHeader."No.");

WhseWkshLine.SETRANGE("Whse. Document Line No.","Line No.");
OnAfterSetWhseWkshLineFilter(WhseWkshLine, "Warehouse Shipment Line" WhseShptHeader); // NEW
IF NOT WhseWkshLine.FINDFIRST THEN BEGIN

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


DeviceInfo.getModel()  return iPhone, not expected iPhone XR

Bug
my iPhone XR's model is A2107 , does not in the dictionary below
https://github.com/react-native-community/react-native-device-info/blob/master/ios/RNDeviceInfo/RNDeviceInfo.m#L145
that cause DeviceInfo.getModel() return iPhone, not expected iPhone XR，
and then DeviceInfo.hasNotch() return false， because brand and model dose not match devicesWithNotch:
  {
    brand: 'Apple',
    model: 'iPhone XR',
  },

and in  iPhone 11 simulator , model is A2111,  the same reason lead DeviceInfo.getModel()  return iPhone, not expected iPhone 11, then  DeviceInfo.hasNotch() return false

Environment info

React native info output:
System:
    OS: macOS 10.14.6
    CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
    Memory: 1.37 GB / 16.00 GB
    Shell: 5.3 - /bin/zsh
  Binaries:
    Node: 12.3.1 - /usr/local/bin/node
    Yarn: 1.16.0 - /usr/local/bin/yarn
    npm: 6.11.3 - /usr/local/bin/npm
    Watchman: 4.9.0 - /usr/local/bin/watchman
  SDKs:
    iOS SDK:
      Platforms: iOS 13.1, DriverKit 19.0, macOS 10.15, tvOS 13.0, watchOS 6.0
    Android SDK:
      API Levels: 21, 23, 26, 27, 28
      Build Tools: 23.0.1, 26.0.1, 28.0.3, 29.0.2
  IDEs:
    Android Studio: 3.5 AI-191.8<IP_ADDRESS>00203
    Xcode: 11.1/11A1027 - /usr/bin/xcodebuild
  npmPackages:
    react: 16.9.0 => 16.9.0
    react-native: 0.61.2 => 0.61.2

 // paste it here

Library version: ^4.0.1-beta.5
Steps To Reproduce

...
Describe what you expected to happen:

in my iPhone XR,getModel() return iPhone XR, not iPhone, hasNotch() return true
in iPhone 11,  getModel() return iPhone 11, not iPhone, hasNotch() return true

Reproducible sample code

in my iPhone XR, getDeviceId return iPhone11,8
https://github.com/react-native-community/react-native-device-info/blob/7aa82298e3df813a92bb63663be143bac9fbab92/ios/RNDeviceInfo/RNDeviceInfo.m#L262
 if (deviceName) {
        if ([deviceName hasPrefix:@"iPod"]) {
            return @"iPod Touch";
        } else if ([deviceName hasPrefix:@"iPad"]) {
            return @"iPad";
        } else if ([deviceName hasPrefix:@"iPhone"]) {
            return @"iPhone";
        } else if ([deviceName hasPrefix:@"AppleTV"]) {
            return @"Apple TV";
        }

        return deviceName;
    }

so this means getModel() always return iPhone in iPhone device?

Interesting - that code was just merged to master from a PR that moved the logic for the lookup table on iPhone models into the Objective-C code, and it looks like your conclusion via code inspection about this line https://github.com/react-native-community/react-native-device-info/blob/master/ios/RNDeviceInfo/RNDeviceInfo.m#L264 resulting in 'iPhone' every time is correct.
I'm still testing prior to official 4.x release, I'll check this on my physical devices now and patch it up as needed.
Thanks for the report!

Indeed - that logic had an issue, great find. I just fixed it - it will be in the v4 release

4.0.1-rc.1 is out and the migration guide is up, unless bugs are found I will release it tomorrow
https://github.com/react-native-community/react-native-device-info/releases/tag/v4.0.1-rc.1
react-native-community/react-native-device-info/wiki/V3-to-V4-Migration-Guide
Please test it, and edits / improvements to the migration guide are always welcome.
Thanks!

The detail of which iPhone model the result returned from the getDeviceId method corresponds to should be added to the document.  A pointer to the code where it's defined might help though?

Please open a PR to the readme that mentions the list of devices is maintained by the community, PRs are welcome, and points to the definition. That should eliminate the confusion, prompt people to help where it is needed, and direct them right at the location where help may be needed - all with no duplication

For some reason, it still doesn't work for iPhone 12, and iPhone SE. I tried looking into library code, it is taking into account the model, converts it to a name, and then checks with a list of notched devices. I ended up copying those lists into my project (device name, notched devices), and there with the code, I have everything works as expected. I'm looking at the code and can't seem to find where the problem lies. (it is latest version, installed pods, I checked everything I could think off)


Added explanation for directly calling request*

Thanks for this! I'll be moving this stuff around into wiki pages soon.


Fix yargs global pollution

Description of the Change
Yargs exports a global singleton by default. Using it directly will break testing apps using yargs themselves.
Alternate Designs
I followed the best-practice example in yargs' documentation. One test had to be adjusted slightly, using a separate yargs instance which has been reset: this is what yargs does internally and required, because run.builder() calls .positional() which cannot be called on the outer yargs instance.
Benefits
Makes mocha compatible with other projects depending on yargs.
Possible Drawbacks
If we need access to mocha's yargs instance, we'll have to expose it ourselves, instead of relying on the global one.
I think this should be a bug fix (patch release).

Coverage decreased (-0.03%) to 91.675% when pulling 88eecd32284f374a18bc1455d6d75a8614d5cfe6 on inukshuk:fix-yargs-global-pollution into b079d24161ead240b5f7ec8dbfbe7449d0380b3b on mochajs:master.


Remove support for cabal-v1 and cabal-new

Update the README and changelog as well.

We can add the github CI files from streamly and transfer the travis tests there.

We can add the gitHub CI files from streamly and transfer the travis tests there.

Will do it in a separate PR.

@harendra-kumar Can you please review this again? I'm rather skeptical about bash scripts.

Looks good.

Looks good.


[Backport 2.x][Bug Bash Fixes][Event Analytics] some fixes (#248)

markdown doesnt care if there's data

Signed-off-by: Eric Wei<EMAIL_ADDRESS>
empty query issue

Signed-off-by: Eric Wei<EMAIL_ADDRESS>
remove duplicate styles

Signed-off-by: Eric Wei<EMAIL_ADDRESS>
hide config panel for markdown

Signed-off-by: Eric Wei<EMAIL_ADDRESS>
fix panel infinite calls

Signed-off-by: Eric Wei<EMAIL_ADDRESS>
hide availability for event analytics

Signed-off-by: Eric Wei<EMAIL_ADDRESS>
remove error message from search

Signed-off-by: Eric Wei<EMAIL_ADDRESS>
fix tab issue with linting fix

Signed-off-by: Eric Wei<EMAIL_ADDRESS>
remove side group button

Signed-off-by: Eric Wei<EMAIL_ADDRESS>
lift data validation to vis container for linting issue

Signed-off-by: Eric Wei<EMAIL_ADDRESS>
update snapshots and minor changes

Signed-off-by: Eric Wei<EMAIL_ADDRESS>
Signed-off-by: Eric Wei<EMAIL_ADDRESS>(cherry picked from commit df92519c0b813c39c43f323da54eaa3081c0e291)
Description
Manually backport #248 to 2.x.
Issues Resolved
[List any issues this PR will resolve]
Check List

[x] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Addressed in this PR


Provide a way to export a Paket-based project as a .unitypackage

If we have #13, then a natural next step is to let Unity developers choose to share their projects as .unitypackages, which are suitable for import through the editor or uploading to the asset store.

If you're not against using gradle:
https://github.com/wooga/atlas-unity -> gradle unity wrapper
https://github.com/wooga/atlas-paket -> gradle paket wrapper
see also our opinionated release plugin which does exactly that https://github.com/wooga/atlas-release.


Configure path variables accordingly when choosing to build in Docker containers

This PR is a solution to issue #1293
The JRE_PATH and JDK_PATH variables are configured according to the OS of the docker container, not the native host.

In the absence of further input from @johnoliver I'm pushing this through on the basis that it has been verified to solve the problem as per https://github.com/AdoptOpenJDK/openjdk-build/issues/1293#issuecomment-561541357


Update bundle.js and index files to pick-up fix

https://github.com/microsoft/PowerToys/pull/1476 fixed a spelling error, need to update bundle.js ad index*.html files.

Fixed in v0.15.2


README is outdated

Running the suggested command:
docker run -it --rm haskell:7.10

I got:
Unable to find image 'haskell:7.10' locally
Pulling repository haskell
FATA[0005] Tag 7.10 not found in repository haskell

Running:
docker run -it --rm haskell:latest

Works fine

It's not that the README is outdated, but rather that haskell:7.10 hasn't been pushed yet.
See docker-library/official-images#651

7:10 has been pushed. Closing this.


Replacing \n

In vim, you can replace \n with a newline by running :s/\\n/\r.
In ex-mode, you have to use :s/\\\\n/\n.
I'm not sure whether we should follow vim in the replacement patterns (\n = NUL, \r = NL), but using \\\\n to match \n isn't right.

+1
I just encountered that bug when trying to  replace a LaTeX-command. I had to search for \\\\mycommand instead of \\mycommand.

It might be that this is because the substitute-function calls the replace-routine for all matches (line 352 in ex.coffee ), but replace/replaceGroups checks for the regex-match, too, so the expression is double-checked.  But I've never written any coffee-script and it is more a guess than the result of a professional code review, so I might be wrong about it...


feat: Add roadmap and features status in README.md

Close #133 .
Adding roadmap and feature status in main page.
This part is heavily inspired by apache/iceberg-go.

cc @Xuanwo @ZENOTME @Fokko PTAL

cc @Fokko @ZENOTME Any other comments?


Add Support for Classes and Custom Commands in seutp.py AST Parsing

Thanks for logging an issue with ptr. Here are some questions to help try
and pull out as much information possible to help us help you!
How is ptr not doing what you expect?
Unable to run ptr on a project where setup.py includes a custom build command definition.
What is your suggestion to make this better?
Add AST parsing support for custom commands and other class definitions.
Code/Bug example?
root@faa971b550dd:/usr/local/tglib# ptr -k
[2019-08-06 19:50:50,751] INFO: Starting /usr/local/bin/ptr (ptr.py:1073)
Traceback (most recent call last):
  File "/usr/local/bin/ptr", line 10, in <module>
    sys.exit(main())
  File "/usr/local/lib/python3.7/dist-packages/ptr.py", line 1090, in main
    args.venv_timeout,
  File "/usr/lib/python3.7/asyncio/base_events.py", line 584, in run_until_complete
    return future.result()
  File "/usr/local/lib/python3.7/dist-packages/ptr.py", line 965, in async_main
    base_path, stats, run_disabled, print_non_configured
  File "/usr/local/lib/python3.7/dist-packages/ptr.py", line 337, in _get_test_modules
    if target.id == "ptr_params":  # type: ignore

How can a developer reproduce this?
Add a custom command to setup.py using the instructions here.

Resolved, thanks!

Awesome. I better merge the fix 📦  ...


Inline help documentation for make commands.

Summary
Adds inline documentation for make commands via make help.
Ticket Link
n/a
Screenshots

@hmhealey @enahum Please let me know if you see any make commands that are meant to be run directly that aren't documented.

I really like this, but I think we need to cleanup the whole Makefile first, what do you guys think if we spend a few minutes tomorrow to talk about it?
after that we can improve on this PR and I could also update the docs.
let me know

@enahum I think that's a great idea. Am available.


wip - add regression test

wip - do not merge.

@rafal-ch , do you think we can close this PR?

@rafal-ch , do you think we can close this PR?

Certainly. Closing now.


@sweir27: Moving vendor libraries to npm DON'T MERGE

This revives this old PR https://github.com/artsy/force/pull/407 with the updated dependency changes. Putting up to see what CI says, but not ready to merge yet.

Along with moving checked in dependencies to npm and updating our dependencies to the latest, this PR...

Moves out this tiny formatting function from /analytics to /lib to keep app code independent of analytics code
Updates a bunch of client-side tests to also expose the jQuery namespace
Move the deep zoom component inside the artwork app to better co-locate
Updates a bunch of tests to move requiring client-side code after benv.setup (in many cases b/c typeahead was being a pain an expecting certain globals like jQuery to be available at require-time)

😵  😮  💨  😑  😄  Got a 💚  finally!


Conditonal rendering value

Description

Checklist

[x] For UI or styling changes, I have added a screenshot or gif showing before & after
[x] I have added a changeset
[x] I have added to the docs where applicable

Stories

Docs:

This looks great! A couple things to check:

Is the red used for redNegatives the same as the one in DataTable?
I would change the docs to match with the description of other color props in the product (there's a few in the DataTable docs)

Updated red color to same in dataTable --> tableRow.svelte

@hughess in terms of the color props, the most similar prop that behaves the same as Value's would be the color prop in annotations.svelte
annotations color prop:

The Value Color Prop, can take in the same color options "options="CSS name | hexademical | RGB | HSL"
I'm also wondering if we should keep the title "Color" instead of "Colour" since the css and out prop is in American Spelling.
Let me know what you think and i can update the Value prop, and the title spelling in annotations if necessary

@hughess I noticed Red Negative is listed in the data table docs, but missing from its prop list. Can we use Value's redNegatives prop list in Data Table
Datatable proplist

Value prop list:

Color is good for consistency
Ah good catch, yes you can use the same one as Value. But it should go under the Column section of the DataTable page.
Also looks like there might be a hanging closing tag on the DataTable page under the prop listing before scaleColumn

Red negative added to column section in data tables

Removed /> above scaleColumn

Value Docs:

DataTable Docs:

Last thing - I think the rednegatives prop needs to be updated to accept strings for “true” and “false” for consistency with other components. The Column component rednegatives prop should already work like that.
goal with that is to remove the need for user to type curly braces to get Boolean values in the syntax - looks more approachable to data analyst, faster to type.
We should make this consistent across the product where Booleans are used in props

We should make this consistent across the product where Booleans are used in props

I've got this on my ToDo list for a general improvement to all core components


Errors reported by ThreadSanitizer

Original issue: https://charm.cs.illinois.edu/redmine/issues/535

This is a tracking bug for errors reported by the ThreadSanitizer dynamic checker for data races and other violations of standardized thread semantics. Any of the errors reported under this result in undefined behavior. They also represent noise when we try to debug modifications of SMP versions of the runtime, and when users may try to debug shared-memory optimizations of their program.
Sub-issues will be assigned to individual PPLers, as decided in the core meeting 2014-07-28.

Original date: 2014-07-28 22:53:29

I'll post instructions on building and running with ThreadSanitizer shortly.

Original date: 2014-10-14 20:51:10

Could you post a link for instructions to run Thread Sanitizer?

Original date: 2014-10-14 21:02:08

To use thread sanitizer, build charm with the tsan option (e.g. ./build charm++ net-linux-x86_64 smp tsan -j16).  Thread sanitizer should then automatically run when you run an executable built with that charmc.

Original date: 2014-11-04 21:37:18

After numerous failed attempts on other machines, I've found this will compile a tsan build on humility:
./build charm++ multicore-linux64 clang tsan -j16
Older kernels and compiler versions will fail to get past the linker test in configure.
The machine will also need /proc/sys/kernel/randomize_va_space set to 1 for the runtime of tsan to work properly.

Original date: 2015-05-04 19:36:09

The need for ASLR to be enabled is described in this upstream bug report:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59410


Implement coherence delta vs. date plot

Implements
This pull request implements:

#4
#8

Outstanding

Deprecate global variables:

I thought I saw a global variable used, but in fact this is only the case the slider and intensity plots are un-commented. That, said there were "redefinition of global variable" warnings which I've scrubbed out.
At a certain point I was thinking that when we get to the point of being able to switch between rasters as described in #3 and between polygon overlays as per #2 it may be necessary to record the state of the relevant radio buttons to determine what to display, when different points are selected on the coherence plot, and that the way to do this is using dcc.Store. But now I'm thinking it's probably possible to query the state of the radio buttons directly.
Conclusion: I don't foresee any need to share data between components.

#11
#12

I see that some checks were not successful, but I'm pretty sure the errors are all in the setup, not in the code I've pushed.

Any progress on getting pylint to pass?

Re: Coherence timedelta plot gaps. My suggestion is to limit visible scale to a fixed temporal baseline, such as 60-80 days. and allow the user to zoom in/out, pan, or zoom to the full network. This visualization needs to support the general case of coherence networks over long periods and different sensors with different repeat pass intervals. We will also likely see different seasonal trends if we begin to examine volcano hazards outside of Canada.
RE: "Remember that part of keeping this usable is going to be "clearing out" date-pairs that turn out to be useless, like summer compared to winter" - Agreed, we can discard interferograms below a certain coherence threshold but once they have been processed and an average coherence value has been generated, we might as well retain that value so that we know empirically that the coherence was calculated and it was low.

@drotheram, I have fixed the CI/CD linters. We now have a PEP8 check which must pass, and a requirement that the pylint score be high (>9). See https://github.com/Volcano-Risk-Reduction-in-Canada/Volcanic-Interpretation-Workbench/tree/feature/coherence_timedelta/.github/workflows and the green checkmarks above.
I think I've addressed all the changes you've requested.


Weak ref support

I'm looking at how feasible this would be to use in RustPython in order to get cyclic gc support. Overall I'm really impressed with this implementation, though one thing I'm wondering about is whether it would be possible to add weakref support to the Cc/ThreadedCc types. What I was thinking of as a user-facing API is:
// Safety: weaklist() should always return the same weaklist for a given object
// `unsafe`ness may not be necessary depending on internal implementation
unsafe trait WeakReferenceable {
    type Error; // so that it can be Infallible if Self can always be referenced
    fn weaklist(&self) -> Result<&gcmodule::WeakList<Self>, Self::Error>; // like tp_weaklistoffset
    type Weak: WeakRef;
}
unsafe trait WeakRef {
    // maybe this should be like `Linked`?
}
struct WeakList<T: WeakReferenceable> { head: T::Weak } // maybe?

Ideally, there would be an easy way to have the actual WeakCc type be a specific kind of Cc as python does it, but I'm not sure of whether that might be unnecessary overhead for other use cases or what that API would look like. Maybe just WeakRef: From<Cc<WeakReferenceable>>?
This is just from a day or so of mulling over how best to implement weaklist in RustPython based on looking at CPython code, so maybe you have a better idea of how to do. Again, I'm not sure how feasible it is/how willing you would be to actually implement the machinery behind this.
Edit: also, looking more at gcmodule's internals, it would be amazing if AbstractObjectSpace could be publically exposed so that RustPython can e.g. directly embed [Threaded]RefCount into PyObject, and generally use the gc machinery at a lower level.

Hi, thanks for your interest. I'm definitely interested in supporting the RustPython usecase.
For weakref, it is a useful feature so it should be in this library. For its implementation, I prefer simpler and not having unsafe if possible. I think the Rust std::rc::Rc does a good job. Its API surface is simple: Rc::downgrade(&self) -> Weak and Weak::upgrade(&self) -> Option<Rc>. There is no unsafe and the space overhead is usize instead of a list. Since it's not hard to implement and verify, I pushed changes adding it. Let me know if it satisfies the weakref usecase. If so, I can publish a new version.
For AbstractObjectSpace, if I understand correctly, you want something like RawCcBox(T) where T embeds RefCount instead of RawCcBox(RefCount, T). In that case, I don't think it's as simple as utilizing AbstractObjectSpace internals. One important reason that T is separate from RefCount is because we need to be able to drop T while keeping the metadata (RefCount) alive. If you want to have read access to the ref counts, APIs can be added for that.

Hi, thanks for your interest. I'm definitely interested in supporting the RustPython usecase.
For weakref, it is a useful feature so it should be in this library. For its implementation, I prefer simpler and not having unsafe if possible. I think the Rust std::rc::Rc does a good job. Its API surface is simple: Rc::downgrade(&self) -> Weak and Weak::upgrade(&self) -> Option<Rc>. There is no unsafe and the space overhead is usize instead of a list. Since it's not hard to implement and verify, I pushed changes adding it. Let me know if it satisfies the weakref usecase. If so, I can publish a new version.
For AbstractObjectSpace, if I understand correctly, you want something like RawCcBox(T) where T embeds RefCount instead of RawCcBox(RefCount, T). In that case, I don't think it's as simple as utilizing AbstractObjectSpace internals. One important reason that T is separate from RefCount is because we need to be able to drop T while keeping the metadata (RefCount) alive. If you want to have read access to the ref counts, APIs can be added for that.

I'll check it out, thanks! I might fork it and experiment with a method on Cc to decrement refcount with a custom drop function, since we're running into UB with our object layout when dropping objects (RustPython/RustPython#2381) and it's tricky to fix that without a performance hit or having a function like that.

I'll check it out, thanks! I might fork it and experiment with a method on Cc to decrement refcount with a custom drop function, since we're running into UB with our object layout when dropping objects (RustPython/RustPython#2381) and it's tricky to fix that without a performance hit or having a function like that.


Login issues

Running this script as well as running the example code on the readme end up in the app hanging while trying to login. Here's the output of the server in debug mode:
http://pastebin.com/fQjSrN6G
Any ideas?

I am seeing the same thing!


Resized images folder not created in S3

Hello Thaiphan,
new sizes of product images are not being uploaded, ie. folder 64x64 in resized folder is not made.
Bests,
Honza

Hello Thaiphan,
new sizes of product images are not being uploaded, i think issue is cache folder not created may be that's why or else something issue in permission.
Bests,
Honza


Im having trouble building this

Can you teach me

What OS do you use? Are you on Windows?

yes

Awesome, what do you plan to use it for? If you want to use it with Zelda64Recomp then you'll probably have to use Visual Studio to build.
Once you have the installer pulled up, scroll down to Desktop & Mobile, and check Desktop development with C++.
Then, at the top next to Workloads, click on Individual Components. Search for cmake, and make sure C++ CMake tools for Windows is checked. Search again for clang, and make sure C++ Clang Compiler for Windows is checked. Then you can click the purple button in the bottom-right to install Visual Studio with what you need.
Once it installs, open Visual Studio and click Open a local folder on the right side. Find and open your N64Recomp folder.
Once Visual Studio opens, and you see CMake generation finished. at the bottom of the screen, click on Build > Build All.
After that, every exe you need should appear in N64Recomp/out/build/x64-Debug. 😋

no i want to try it on super mario 64

Based on #92 it sounds like you got the repo to build, so closing this issue.


textareas?

Support for textareas?

nevermind; I tried with type="textarea" on AvInput as:
<AvInput type="textarea" name="address" id="address" placeholder="Address" required />

and it worked.
Thanks.

Yeah, type is the key. It also works with type of "select" with option(s) as children

can I add more documentation code for this?

Go for it. Right now the documentation is more by example (showing various inputs in a form) and not really going over the individual inputs. Feel free to add a page dedicated to textarea, showcasing how to default values and all. Same for select if you are up for the task.


cla: add dogpawhat

Current behavior

Proposed changes

Checks

[x] All commits in this Pull Request are signed and Verified by Github.
[x] All commits in this Pull Request follow the Ockam commit message convention.
[x] There are no Merge commits in this Pull Request. Ockam repo maintains a linear commit history. We merge Pull Requests by rebasing them onto the develop branch. Rebasing to the latest develop branch and force pushing to your Pull Request branch is okay.
[x] I have read and accept the Ockam Community Code of Conduct.
[x] I have read and accepted the Ockam Contributor License Agreement by adding my Git/Github details in a row at the end of the CONTRIBUTORS.csv file in a separate pull request to the build-trust/ockam repository. The easiest way to do this is to edit the CONTRIBUTORS.csv file in the github web UI and create a separate Pull Request, this will mark the commit as verified.

@DogPawHat We'll merge this shortly.
Thank you for accepting the CLA and taking time to contribute!!
Welcome to the Ockam Community 🥳
If you have questions, as you explore, join us on the contributors discord channel https://discord.gg/bewvnm6zqS


Is there a way to programmatically assign an Audio Type (e.g. Dialogue, Music, etc) to an audio clip, and apply ducking?

Is there a way to programmatically assign an Audio Type (e.g. Dialogue, Music, etc) to an audio clip, and apply ducking?
Here is the scenario:

I have one audio clip in Track 1, and one audio clip in Track 2.
In the "Essential Sounds" panel, I can assign the clip in Track 1 as Dialogue, and the clip in Track 2 as Music.
I can then apply Ducking to the Music clip, click on the "Generate Keyframes" button, and the Music clip will automatically be ducked against the Dialogue clip.

I can do this with the User Interface, but Is there a way to do this programmatically?
Thanks!

Received contact from Adobe Employee. The answer is no. https://community.adobe.com/t5/premiere-pro/how-to-programmatically-assign-an-audio-type-to-an-audio-clip-amp-duck-music/m-p/12311684#M361823


Error parsing JSON

I encountered the issue Error parsing JSON: Deadline of 60.0s exceeded while calling target function, last exception: 503 failed to connect to all addresses;
And now I have resolved the issue with the help of https://github.com/google/generative-ai-python/issues/117
I have tested it on my computer and it works
As it says that it needs to add transport='rest' on API config genai.config
But I did not find the genai.config on the program
So what should I do if I want to config the transport='rest'

@michsiu thanks for the ticket. I noticed it may be a duplicate of #138 so I'll close this one. I wanted to mention this should be fixed now. Please try and let me know if it is still an issue for you in #138. Thanks


"rtf" typo

Typo:
fontmake -o rtf-interpolatable -m path-to-your-designspace-file
should be
fontmake -o ttf-interpolatable -m path-to-your-designspace-file

fixed. good catch. thanks


Plugin stopped working after running sometime.

Here is my config for mongo auth plugin.

Problem in plugin
I enabled plugin using an Emqtt setting with my custom mongo config. it can able to authenticate a user using username and password which is stored in my collection named "mqtts". But after some time, it will be stopped working, Because of that user can able to log in using wrong username and password in the system. I also checked plugin running or not using the following command

./bin/emqttd_ctl plugins load emq_auth_mongo

it will show that your plugin is already running.
It starts working fine if I restart the emqtt server.

I have a similar problem. I start the plugin but it does not verify the username and password, anyone can log in

This is mostly because of the allow_anonymous setting in emqx.conf, please verify that.


two questions about semantic segmentation with dinov2

Hi, thanks for your great work! I have two question. First, I observed that in your paper, dinov2 pre-trained at high resolution performs better as the resolution goes higher, for instance from 512 resolution to 640 resolution. Does it mean the model can adapt to different resolutions for semantic segmentation? If so, is there any insight behind this phenomenon? It is easy for CNNs but not trivial for ViTs, to my knowledge. Second, how did you resize the input to multiple of 14? I saw that you defined the CenterPadding class in notebooks. So you use the padding rather than resizing? Does it have any impact on performance?

Hi, thanks for your great work! I have two question. First, I observed that in your paper, dinov2 pre-trained at high resolution performs better as the resolution goes higher, for instance from 512 resolution to 640 resolution. Does it mean the model can adapt different resolutions for semantic segmentation? If so, is there any insight behind this phenomenon? It is easy for CNNs but not trivial for ViTs, to my knowledge. Second, how did you resize the input to multiple of 14? I saw that you defined the CenterPadding class in notebooks. So you use the padding rather than resizing? Does it have any impact on performance?

Could you public your custom codebase of mmsegmentation to provide more details?

Resize_pos_embed function is not being used in here. What to do when pre-training resolution is inconsistent with downstream input resolution?


Use the automatic caching by setup-go

Pull Request Functional Test Report for #1989 / 66be1575b59e18ed4089fd2b5820d14939912f65
No tests identified for validation.
Help

Thanks for the discussion and review @bstoll


Fix paste on android

see issue #433

Thanks for the fix, sorry it took a while to get out, SauceLabs tests were being flakey.


regression in type signature of UPath.rename

with the release of v0.2.3 commit 93855d207655239e068da0def5e171d1381c9553 introduced a regression in the type signature of UPath.rename, causing mypy to complain. see: apache/airflow#41723

We'd need https://peps.python.org/pep-0728/ to type this correctly via TypedDicts. Need to introduce a unset default value for maxdepth and recursive to get this working as intended.


Updated with support of PubSub+ 9.4 and later

Issue #, if available:
Description of changes:

Updated with support of PubSub+ 9.4 and later

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@davmayd  - Hey Dave, sorry to bug you: you seem to have contributed to this branch most recently back in April. Would you know who can approve this PR and merge develop to master ? master is broken now if using recent versions of the Solace Event Broker without these changes. Thanks a lot in advance: Balazs

Hi Balazs
I’ll take care of it today. Sorry for the delay.
Vinod
On Fri, Jul 17, 2020 at 8:00 AM Balázs Czoma<EMAIL_ADDRESS>wrote:

@davmayd https://github.com/davmayd - Hey Dave, sorry to bug you: you
seem to have contributed to this branch most recently back in April. Would
you know who can approve this PR and merge develop to master ? master is
broken now if using recent versions of the Solace Event Broker without
these changes. Thanks a lot in advance: Balazs
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/aws-quickstart/quickstart-solace-pubsub/pull/20#issuecomment-660069240,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEYNKKUKI6EJKZM346MSF3R4A4PVANCNFSM4M5ZTACQ
.

Hi Vinod, nice to hear from you 😊
No worries, thanks a lot!
Balazs
From: Vinod Shukla<EMAIL_ADDRESS>Sent: Friday, July 17, 2020 8:28 AM
To: aws-quickstart/quickstart-solace-pubsub<EMAIL_ADDRESS>Cc: Balazs Czoma<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [aws-quickstart/quickstart-solace-pubsub] Updated with support of PubSub+ 9.4 and later (#20)
Hi Balazs
I’ll take care of it today. Sorry for the delay.
Vinod
On Fri, Jul 17, 2020 at 8:00 AM Balázs Czoma <notifications<EMAIL_ADDRESS>wrote:

@davmayd https://github.com/davmayd - Hey Dave, sorry to bug you: you
seem to have contributed to this branch most recently back in April. Would
you know who can approve this PR and merge develop to master ? master is
broken now if using recent versions of the Solace Event Broker without
these changes. Thanks a lot in advance: Balazs
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/aws-quickstart/quickstart-solace-pubsub/pull/20#issuecomment-660069240,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEYNKKUKI6EJKZM346MSF3R4A4PVANCNFSM4M5ZTACQ
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/aws-quickstart/quickstart-solace-pubsub/pull/20#issuecomment-660079804, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEWH4XZYBK26XTHRNNVDHXLR4A7W3ANCNFSM4M5ZTACQ.

Hi Balazs, good to hear from you as well. This is taken care of. I did have to add a couple of commits (https://github.com/aws-quickstart/quickstart-solace-pubsub/commit/e111e33b1be1b1a1f893e478cfdea3e724a3e550, https://github.com/aws-quickstart/quickstart-solace-pubsub/commit/0bc6c40c78b10dcf50e1014d8ca26b83703c852c) to make it pass our testing. Please sync your fork.


fix(types): mapState without getters. fix #581

Fix #581

What kind of change does this PR introduce? (check at least one)

[x] Bugfix
[ ] Feature
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:

[x] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[x] All tests are passing
[ ] New/updated tests are included

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:

Codecov Report

Merging #582 (6fbd575) into v2 (d79ed22) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##               v2     #582   +/-   ##
=======================================
  Coverage   93.17%   93.17%
=======================================
  Files           8        8
  Lines         337      337
  Branches       90       90
=======================================
  Hits          314      314
  Misses          7        7
  Partials       16       16

Impacted Files
Coverage Δ

src/types.ts
100.00% <ø> (ø)

Continue to review full report at Codecov.

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

Thanks for the PR, this will work on the next release


Why you run php-cron in separated container?

Why you run php-cron in separated container but not in same php container?

Good question! The docker container is supposed to have one task and one task only, which translates to running a single binary in real life. To use PHP for serving web requests, we need to run a php-fpm daemon (/usr/sbin/php-fpm7.4 -O, link) and expose it to the Nginx via the port. To use cron, we need to run cron daemon (cron -f, link).
That separation provides us with separate logs per container, which is useful when one is serving web requests, and another is running the scheduled tasks; however, it's not used here as a cron container (supposed) not to write anything to stdout.
There are multiple other benefits of the separation of these two functions into two separate containers, but I can't think of any single one which would be applicable in this case, and I do see the downside of having a duplicate configuration for the php-cron container as it needs to access the same volumes and containers to work.
Having both cron and PHP workers running in the same container would be more confusing to someone who used to run a single function per container; however, it will remove one extra place of complexity. I'll see if I can make it work to preserve the clarity of the setup.


Add a new CLI config "existing-output-dir" that

allows users to pass in a previously built storybook
directory.

I was going to suggest this same exact functionality; is this something that can be merged?

Yes i can merge his if the merge-conflicts have been resolved, Do you think you could resolve them @travisbloom ?

hi all, sorry for the delay. Please let me know if there's anything else needed to get this merged :)

Thanks for this @travisbloom LGTM ✅

Published as 2.2.0


Ability to use non-shaded Pulsar Client with the Spring Boot starter

Feature request:
The current Pulsar client used by the spring-pulsar-spring-boot-starter is a "shaded" jar file, which means all the dependencies are in the same jar and with a modified package name. While shading can be useful to avoid dependency conflicts, it also means users of the jar cannot exclude or override these dependencies through Maven/Gradle. Being able to override dependencies is sometimes desirable/necessary, for example if these dependencies have a known vulnerability. A concrete example: the 2.12.1 version of the client (used by the 0.2 version of the starter contains a vulnerable version of Google Guava.
A non-shaded version of the Pulsar client is available (pulsar-client-original), but I cannot use that to override the client used by the starter, as it throws a ClassNotFoundException due to the fact the shaded package and the non-shaded packages are different.
Possible solutions would include:

have the starter use the non-shaded Pulsar Cliente
give the option of using the non-shaded Pulsar Client (either built-in or with a dependency override)

(PS I could solve the concrete example above, as Pulsar has a 2.12.2 version of the client that does not use the vulnerable Guava version, so I am not looking to solve this particular instance, but more in general.)

Hi @jheijmans
I am glad 2.11.2 solves this particular issue.  I do understand your use case. We will likely always use the shaded version as the default as the client has many dependencies that will clash with user application dependencies. The shading allows the client to use whatever it wants as an internal concern, at the expense of not letting the user manage the dependencies (in your case, exclude a vulnerable lib).
I believe you could exclude pulsar-client-all from the starter and just use the pulsar-client-orignal and pulsar-client-admin-original.

Hi @jheijmans
Were you able to verify if you could do the suggested workaround from above?

I believe you could exclude pulsar-client-all from the starter and just use the pulsar-client-orignal and pulsar-client-admin-original.

Closing this out as "wont fix".


Labels

Instead of a separate text file, rename the images with the label itself

You can write a script to do this conversion : )

You can write a script to do this conversion : )

Sure did :+1:


Add Word impls for signed integer types

Working on an I2S module for nrf-hal using these traits and noticed Word impls for signed integer types are missing, so i added them.

Sounds good to me. Thanks!
bors r+

Try again
bors r+


wrong suggestion for DateTime.now

DateTime.now
 Rails/TimeZone: Do not use DateTime.now without zone. Use DateTime.zone.now instead.

001:0> DateTime.zone.now
NoMethodError: undefined method `zone' for DateTime:Class

I think this is fixed on the master branch.

@bbatsov
Using rubocop 0.32.1 from git://github.com/bbatsov/rubocop.git (at master)

✗ rubocop app/controllers/events_controller.rb
Inspecting 1 file
C

Offenses:

app/controllers/events_controller.rb:38:14: C: Rails/TimeZone: Do not use DateTime.now without zone. Use DateTime.zone.now instead.
    DateTime.now

//cc @palkan

:+1:

This was on the master fixed by #1995, but haven't been released yet.

Using rubocop 0.32.1 from git://github.com/bbatsov/rubocop.git (at master)
✗ rubocop app/controllers/events_controller.rb
Inspecting 1 file
C
Offenses:
app/controllers/events_controller.rb:38:14: C: Rails/TimeZone: Do not use DateTime.now without zone. Use DateTime.zone.now instead.
DateTime.now

@palkan ^^

Yeah, I saw it. But I don't believe it was the latest Rubocop from master)
Here is the code which generates the message and here is the spec.
@stereodenis I think you should run smth like gem update rubocop.

@palkan
bundle exec rubocop app/controllers/events_controller.rb
Inspecting 1 file
C

Offenses:

app/controllers/events_controller.rb:38:14: C: Rails/TimeZone: Do not use DateTime.now without zone. Use Time.zone.now instead.
    DateTime.now
             ^^^

and
gem update rubocop
Updating installed gems
Nothing to update

@stereodenis Do you have "rubocop" in your Gemfile or it's installed globally?

@palkan in my Gemfile
gem 'rubocop', github: 'bbatsov/rubocop', branch: 'master', require: false

Ok. And what revision do you have in your Gemfile.lock?

@palkan
GIT
  remote: git://github.com/bbatsov/rubocop.git
  revision: c52ec398184db338528517827056694d52939fbb
  branch: master
  specs:
    rubocop (0.32.1)
      astrolabe (~> 1.3)
      parser (>= <IP_ADDRESS>, < 3.0)
      powerpack (~> 0.1)
      rainbow (>= 1.99.1, < 3.0)
      ruby-progressbar (~> 1.4)

@bbatsov @palkan bump

@stereodenis Please, provide offensive source code.

Having the same behaviour.
Gemfile:
gem 'rubocop', github: 'bbatsov/rubocop'
Ran bundle, output showed that Rubocop was updated to the Git master branch.
$ rubocop -R:
app/controllers/events_controller.rb:11:52: C: Do not use DateTime.now without zone. Use DateTime.zone.now instead.
    @events = Event.where('end_date > ?', DateTime.now).order(:start_date)
                                                   ^^^
app/models/ability.rb:52:50: C: Do not use DateTime.now without zone. Use DateTime.zone.now instead.
        event.registration_open_date <= DateTime.now && event.registration_close_date >= DateTime.now
                                                 ^^^
app/models/ability.rb:52:99: C: Do not use DateTime.now without zone. Use DateTime.zone.now instead.
        event.registration_open_date <= DateTime.now && event.registration_close_date >= DateTime.now
                                                                                ^^^
app/models/ability.rb:54:50: C: Do not use DateTime.now without zone. Use DateTime.zone.now instead.
        event.registration_open_date <= DateTime.now
                                                 ^^^

You definitely have an outdated Rubocop.
Here is my output:
main.rb:15:52: C: Do not use DateTime.now without zone. Use Time.zone.now instead.
    @events = Event.where('end_date > ?', DateTime.now).order(:start_date)

Sample script.

@Silox bundle update rubocop

[17:16] silox@Gallifrey ~/Documents/Projects/Gandalf ✔ bundle update rubocop
Updating git://github.com/bbatsov/rubocop.git
Fetching gem metadata from https://rubygems.org/............
Fetching version metadata from https://rubygems.org/...
Fetching dependency metadata from https://rubygems.org/..
Resolving dependencies.....
...
Using rubocop 0.32.1 from git://github.com/bbatsov/rubocop.git (at master)
...
Bundle updated!
[17:17] silox@Gallifrey ~/Documents/Projects/Gandalf ✔ bundle exec rubocop -R
Inspecting 175 files
....W........................CCC.C.C.CCC......C................................................................................................................................

Offenses:

app/controllers/events_controller.rb:11:52: C: Do not use DateTime.now without zone. Use Time.zone.now instead.
    @events = Event.where('end_date > ?', DateTime.now).order(:start_date)
                                                   ^^^
...
app/models/ability.rb:52:50: C: Do not use DateTime.now without zone. Use Time.zone.now instead.
        event.registration_open_date <= DateTime.now && event.registration_close_date >= DateTime.now
                                                 ^^^
app/models/ability.rb:52:99: C: Do not use DateTime.now without zone. Use Time.zone.now instead.
        event.registration_open_date <= DateTime.now && event.registration_close_date >= DateTime.now
                                                                                ^^^
app/models/ability.rb:54:50: C: Do not use DateTime.now without zone. Use Time.zone.now instead.
        event.registration_open_date <= DateTime.now
                                                 ^^^
...
175 files inspected, 19 offenses detected

A real mystery. :-)

@bbatsov Aha! If you watch closely, the error message has changed! From:
app/controllers/events_controller.rb:11:52: C: Do not use DateTime.now without zone. Use DateTime.zone.now instead.

to
app/controllers/events_controller.rb:11:52: C: Do not use DateTime.now without zone. Use Time.zone.now instead.

So something has changed updating to master :)

So, then the reported problem is solved. Your problem is different - you want to use another enforced style with the cop - acceptable.


meeting-live-coding nodejs sample fixes

Build status - All builds passed
https://github.com/OfficeDev/Microsoft-Teams-Samples/actions/runs/4829078200


bytescale

Casting an image to UINT8 might produce surprising results if not rescaled to between 0 and 255. Scipy have a neat function to handle this called bytescale(). It also allows users to define min and max values for the rescale (e.g. the percentiles returned by the metadata() methods).
I would suggest using it (or something like it). The example here has copied and pasted bytescale() into rio_tiler.utils and updated array_to_img().
The effect is that, if people display or save the Pillow image created by array_to_img() it should always show something pretty.

👀 I'm not sure to see how bytescale will be different from https://github.com/mapbox/rio-tiler/blob/master/rio_tiler/utils.py#L182 ?

Gosh! Apologies for missing that. Yep, looks like are essentially the same.


Merge vote packet with pbft block packet

Purpose

Work in progress:

Remove pbft block packet
send pbft blocks as optional value with each new vote

How does the solution address the problem

Changes made

prnet cleaned-up successfully


Problem with fancy menu

When I used this mod, the menu that I made in the fancy menu was completely reset. I don’t know if the mod completely disables the ability to customize the menu or if it’s only the first time it’s launched, but I’ve already mentioned the problem.

This doesn't do anything with menus - there are no mixins involved that cause this?


Fix broken examples

We have several broken examples (see easy.html, for instance).

I guess you're referring to the examples in the v4 branch. If that's the case, they have been broken since we started the work on widgets (we changed the format of the viz.json). But also, since we extracted all the widgets related to widgets to the deep-insights.js repo, CartoDB.js is no longer working (the logic to create instances of layer groups in Windshaft and update the layer group model has been removed extracted and hence nobody is setting the tile URLs on the layer group model). I will open an issue explaining this and make sure I mention the examples.
cc: @viddo @rochoa

:+1:


[Backport 2.4] Fixed a bug with doc level trigger creation.

Description
Manually backporting PR https://github.com/opensearch-project/alerting-dashboards-plugin/pull/513.
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #535 (f01abd6) into 2.4 (e36b9b8) will increase coverage by 0.00%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##              2.4     #535   +/-   ##
=======================================
  Coverage   52.73%   52.74%
=======================================
  Files         209      209
  Lines        5452     5451    -1
  Branches      764      763    -1
=======================================
  Hits         2875     2875
+ Misses       2575     2574    -1
  Partials        2        2

see 1 file with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Inexpected double preflight request

My architecture is composed by a frontend html+js in one container: this container contains all contents in your folder; a nginx container as load balancer (on port 9200); N VM as CDN (on port 9500).
This is nginx load balancer config file:
upstream cdn-audio {
        server <IP_ADDRESS>:9500;
        server <IP_ADDRESS>:9500;
        server <IP_ADDRESS>:9500;
   }

upstream cdn-video {
        server <IP_ADDRESS>:9500;
        server <IP_ADDRESS>:9500;
        server <IP_ADDRESS>:9500;
    }

server {

listen 80;
server_name <IP_ADDRESS>;
access_log /var/log/nginx/acces.log main;

location = /LynyrdSkynyrdFreebirdAudio.mp4 {

             proxy_pass http://cdn-video/LynyrdSkynyrdFreebirdAudio.mp4;
#           proxy_pass http://<IP_ADDRESS>:9500;
#           return 301 $scheme://cdn-audio/LynyrdSkynyrdFreebirdAudio.mp4;
            add_header X-Upstream  $upstream_addr;

        }

       location = /LynyrdSkynyrdFreebirdVideo.mp4 {
             proxy_pass http://cdn-video/LynyrdSkynyrdFreebirdVideo.mp4;
#             proxy_pass http://<IP_ADDRESS>:9500;
#             return 301 $scheme://cdn-audio/LynyrdSkynyrdFreebirdVideo.mp4;
              add_header X-Upstream  $upstream_addr;
        }

   }

When, from my mpd, i point to localhost:9200/LynyrdSkynyrdFreebirdVideo-Audio.mp4, i first have an OPTIONS request, because of cross origins request; this request is correctly managed; then i see, from network console in the browser, some redirect response (i want use load balancer with a redirect approach, redirecting request to localhost:9200/LynyrdSkynyrdFreebirdVideo-Audio.mp4 to one of my upstream server. So the browser, after this redirect response, will contact directly the chosen upstream server without communicating anymore with load balancer.
Well, what i see in network console in browser is the following;

some redirect response;

XMLHttpRequest cannot load http: // localhost:9200/LynyrdSkynyrdFreebirdVideo.mp4. Redirect from 'http: // localhost:9200/LynyrdSkynyrdFreebirdVideo.mp4' to 'http: // cdn-video/LynyrdSkynyrdFreebirdVideo.mp4' has been blocked by CORS policy: Request requires preflight, which is disallowed to follow cross-origin redirect.

XMLHttpRequest cannot load http: // localhost:9200/LynyrdSkynyrdFreebirdAudio.mp4. Redirect from 'http: // localhost:9200/LynyrdSkynyrdFreebirdAudio.mp4' to 'http: // cdn-audio/LynyrdSkynyrdFreebirdAudio.mp4' has been blocked by CORS policy: Request requires preflight, which is disallowed to follow cross-origin redirect.

Why there is another request that requires preflight?
How could i solve the problem?

Looks like you stumbled into one of the darker corners of CORS.  I was unaware of this, but according to this stackoverflow thread, CORS used to disallow redirects.  http://stackoverflow.com/a/39728229
It appears that the spec has changed since then, but that browser implementations have not caught up yet.
That stackoverflow thread has some suggestions to work around this issue, but ultimately, we will need to push browser vendors to fix their implementations.
What browser are you using?  What others have you tried?

I've tried with chrome v.55 and firefox. Reading stackoverflow thread, w3 has updated his standard about CORS in August 2016, but browser vendors require time to implement this new standard.
I resolved this problem using google chrome canary browser.

Okay, so to confirm, Chrome Canary (v56) allows redirects cross-origin?

I managed to put together a simple repro page for this.  It passes in Chrome 57 and 58, but fails in Chrome 56.
Will test other browsers soon.

Yes, redirect after preflight request works with canary (v.58)

Complete status of cross-origin redirects with preflight:

Chrome 56 (stable)
Chrome 57 (dev)
Firefox 51 (stable)
Firefox 52 (beta)
Firefox 53 (aurora)
Edge 14
IE 11
Safari 10 (latest)

FAIL
OK
FAIL
FAIL
FAIL
OK
OK
FAIL

Chrome has already fixed it, so you can expect it to be released in Chrome stable in about 6 weeks.  Bugs need to be filed against Firefox and Safari.

I filed a bug against Safari here: https://bugs.webkit.org/show_bug.cgi?id=167521
There was already a bug filed against Firefox here: https://bugzilla.mozilla.org/show_bug.cgi?id=1312864

Great! I appreciated i've found something new about cors redirect.
I hope i've give you some help"

@joeyparrish cors redirect doesn't working on Firefox Developed Edition, v.53.0a2 (2017-02-1) (64bit).
I've read that you have write here that it works in "WebKit nightly r211339". Where can i download this webkit from? Thanks!

Yes, this bug on Firefox is still open as of today: https://bugzilla.mozilla.org/show_bug.cgi?id=1312864  You can expect that CORS redirect will continue to fail in Firefox until after that bug is closed.
WebKit nightly builds for Mac can be found here: https://webkit.org/nightly/

Are WebKit nightly builds available also for Ubuntu? I'd like testing my architecture on a Firefox distro.

No, Apple publishes WebKit nightly for OS X only as far as I can tell.  There seems to be a GTK+ port here: https://webkitgtk.org/  I can't really offer any support for you getting/building WebKit on Linux, though.  Good luck!

@joeyparrish is there some firefox release that supports this cors-redirect? I.e. nightly?

When I checked two weeks ago, the answer was no.  Check the bugzilla link to see if they have fixed it yet.  If not, comment on the bug and let them know it's important to you.

The upstream issue is still open, and nobody is looking into it yet.  I pinged the upstream issue to ask for someone to take a look.

Firefox just marked their issue "fixed" for Firefox 63, which is currently scheduled for beta on 2018-09-05 and stable on 2018-10-23.  Hope that helps!


BatchSize in BatchUpdate and BatchDelete

Hello
I have a question about differences between BatchUpdate and BatchDelete generated sql queries
EF Plus version 3.1.15 (latest one at the moment)
1) UpdateBatch with specified BatchSize:
            var affectedRows = await context.Accounts
                .Where(account => account.DeletedAt.HasValue == false)
                .UpdateAsync(
                    account => new Account { DeletedAt = DateTime.UtcNow },
                    update =>
                    {
                        update.BatchSize = 1000;
                    });

Generated SQL query:
exec sp_executesql N'DECLARE @stop INT
DECLARE @rowAffected INT
DECLARE @totalRowAffected INT
DECLARE @ZZZ_INDEX INT

SET @stop = 0
SET @totalRowAffected = 0
SET @ZZZ_INDEX = 0

SELECT Z_Batch.[Id],
       ROW_NUMBER() OVER (ORDER BY (SELECT 1)) AS ZZZ_INDEX
INTO #ZZZBatch_c1d21b27_ecb8_4604_ab91_4d372a1f63f6
FROM [Accounts] AS Z_Batch

WHILE @stop = 0
BEGIN
    UPDATE A
    SET A.[DeletedAt] = @zzz_BatchUpdate_0
    FROM [Accounts] AS A
    INNER JOIN ( SELECT * FROM #ZZZBatch_c1d21b27_ecb8_4604_ab91_4d372a1f63f6 AS ZZZ_temp
                WHERE ZZZ_temp.ZZZ_INDEX > @ZZZ_INDEX AND ZZZ_temp.ZZZ_INDEX <= @ZZZ_INDEX + 1000
               ) AS ZZZ_Batch ON A.[Id] = ZZZ_Batch.[Id]
    INNER JOIN ( SELECT [a].[Id], [a].[Data10], [a].[Data5], [a].[Data6], [a].[Data7], [a].[Data8], [a].[Data9], [a].[DeletedAt], [a].[FirstName], [a].[LastName]
FROM [Accounts] AS [a]
WHERE [a].[DeletedAt] IS NULL
               ) AS B ON A.[Id] = B.[Id]

    SET @rowAffected = @@ROWCOUNT
    SET @ZZZ_INDEX = @ZZZ_INDEX + 1000
    SET @totalRowAffected = @totalRowAffected + @rowAffected

    IF @rowAffected < 1000
        SET @stop = 1
END

SELECT @totalRowAffected',N'@zzz_BatchUpdate_0 datetime2(7)',@zzz_BatchUpdate_0='2021-01-28 07:18:26.7050807'

2) UpdateBatch with not specified BatchSize
            var affectedRows = await context.Accounts
                .Where(account => account.DeletedAt.HasValue == false)
                .UpdateAsync(account => new Account { DeletedAt = DateTime.UtcNow });

Generated SQL query:
exec sp_executesql N'
UPDATE A
SET A.[DeletedAt] = @zzz_BatchUpdate_0
FROM [Accounts] AS A
INNER JOIN ( SELECT [a].[Id], [a].[Data10], [a].[Data5], [a].[Data6], [a].[Data7], [a].[Data8], [a].[Data9], [a].[DeletedAt], [a].[FirstName], [a].[LastName]
FROM [Accounts] AS [a]
WHERE [a].[DeletedAt] IS NULL
           ) AS B ON A.[Id] = B.[Id]
',N'@zzz_BatchUpdate_0 datetime2(7)',@zzz_BatchUpdate_0='2021-01-28 07:22:27.7440262'

3) DeleteBatch with not specified BatchSize
            var affectedRows = await context.Accounts
                .Where(account => account.DeletedAt < DateTime.UtcNow.AddYears(-1))
                .DeleteAsync();

Generated SQL query:

DECLARE @stop int
DECLARE @rowAffected INT
DECLARE @totalRowAffected INT

SET @stop = 0
SET @totalRowAffected = 0

WHILE @stop=0
    BEGIN
        DELETE TOP (4000)
        FROM    A
        FROM    [Accounts] AS A
                INNER JOIN ( SELECT [a].[Id]
FROM [Accounts] AS [a]
WHERE [a].[DeletedAt] < DATEADD(year, CAST(-1 AS int), GETUTCDATE())
                           ) AS B ON A.[Id] = B.[Id]

        SET @rowAffected = @@ROWCOUNT
        SET @totalRowAffected = @totalRowAffected + @rowAffected

        IF @rowAffected < 4000
            SET @stop = 1
    END

SELECT  @totalRowAffected

A question: Is there are reason that DeleteBatch limits amount of rows to 4000 by default (UpdateBatch doesn't have this limit if BatchSize is not specified - rendered SQL query is simpler).
If I want to delete more than 4000 rows, I have to specify huge BatchSize values like int.MaxVal, and it's a bit strange for me

Hello @SergeyBalanEmpeek ,
The more row you update or delete at once, the more time it will take often exponentially.
For example, delete 4,000,0000 rows will be way longer than executing 1000*4000 operations. That's why there are a default 4000 rows for the delete statement.
The only reason why BatchUpdate doesn't have a default batchSize is probably backward compatibility since the BatchSize for this method has been added recently. We might eventually add one.

If I want to delete more than 4000 rows, I have to specify huge BatchSize values like int.MaxVal, and it's a bit strange for me

Your opinion is valid.
However here is mine: When I use a library, I prefer to use a library that's smart enough to try to do the right thing (limiting the number of rows for better performance). So many people got a bad performance issue with the SqlBulkCopy for example just because they didn't know they had to specify a batch size and was inserting millions row at once.
Let me know if that answers your questions.
Best Regards,
Jon

Hello @SergeyBalanEmpeek ,
The more row you update or delete at once, the more time it will take often exponentially.
For example, delete 4,000,0000 rows will be way longer than executing 1000*4000 operations. That's why there are a default 4000 rows for the delete statement.
The only reason why BatchUpdate doesn't have a default batchSize is probably backward compatibility since the BatchSize for this method has been added recently. We might eventually add one.

If I want to delete more than 4000 rows, I have to specify huge BatchSize values like int.MaxVal, and it's a bit strange for me

Your opinion is valid.
However here is mine: When I use a library, I prefer to use a library that's smart enough to try to do the right thing (limiting the number of rows for better performance). So many people got a bad performance issue with the SqlBulkCopy for example just because they didn't know they had to specify a batch size and was inserting millions row at once.
Let me know if that answers your questions.
Best Regards,
Jon


Making keyword problem

Expected behavior
Actual behavior
Steps to reproduce the behavior
Please include the operating system and CPU architecture. When applicable, provide relevant audio data.
Hi, I'm making some prototype and try to make my keyword like below.
But it's not work with pronunciation error.
Can you suggest the way to resolve this problem or add a keyword "ariot" (not "a riot")??
Please advice about it.
Thanks.
C:\Users\1\Porcupine-master\Porcupine-master>tools\optimizer\windows\i686\pv_porcupine_optimizer.exe -r resources\ -w "ariot" -p windows -o .
[ERROR] could not find the pronunciation for 'ariot'. If this is not a typo please contact<EMAIL_ADDRESS>

can you refer to here?
https://endic.naver.com/search.nhn?sLn=en&query=ariot&searchOption=all

OK. We will add it to next release which is coming next week. Thanks.

added the keyword to new release ...

Hey, I am trying to create a snake game but it doesn't recognize the word "go left"?.


DLL Not found error.. When Protecting exe with confuserEx

ConfuserEx Version: <IP_ADDRESS>
Target Framework: 4.5
Operating System: Windows 10

Steps to Reproduce:
Hi
SampleDIr.zip
I'm attaching Directory to check this at your end. (SampleDir.zip)
This Exe basically use DevExpress Reporting components and print report files.
This directory contain 2 exe's
1) Exe without confuser protection
2) Exe with confuser protection

If you run exe NO. 1, then all type of print works fine
If you run exe no. 2,
Print 1 works fine
**_Print 2 failed dll error._**

Also, attached CrossTabIssue.zip (which contains source code of above exe)
CrossTabIssue.zip

Both the protected and unprotected versions raise the same missing DLLs errors for me. Both applications crash instantly when clicking any button.

Hi There Kindly refer below ticket, which I raised with DevExpress Team.https://supportcenter.devexpress.com/ticket/details/t1035303/could-not-find-dll-error The said issue arises whenYou build exe in VB.NetIn runtime, you append some script which is written in C#    Kindly read above ticket carefully, it will help you trace exact cause. Also, refer below similar ticketshttps://supportcenter.devexpress.com/ticket/details/t416360/problem-with-vb-scripting-and-referenced-obfuscated-dll  I’m ready to help you and repeat the same issue in your pc. For this, I need online access of your PC. Regards,Ritesh Mehta  Sent from Mail for Windows From: github-actions[bot]Sent: 06 January 2022 09:43To: mkaring/ConfuserExCc: RiteshMehtaIndia; AuthorSubject: Re: [mkaring/ConfuserEx] DLL Not found error.. When Protecting exe with confuserEx (#420) This issue needs more information and has not had recent activity. Please provide the missing information or it will be closed in 7 days. Thanks!—Reply to this email directly, view it on GitHub, or unsubscribe.Triage notifications on the go with GitHub Mobile for iOS or Android. You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>
This kind of scripting may be affected by ConfuserEx due to the renaming.
That being said, I think the main issue is that the declarations inside the DLL are broken in the "Confuser" version.
This does not look like anything I've seen before.
Also I found that the two DLLs do not line up. There was something else modifying the assembly beside ConfuserEx. In the confused version there is a AritoCrypto attribute present. This one doesn't exist in the original version.
What is the source of this attribute?

I Changed Project Title to "AritoCrypto". But it might not be issue, since I checked above issue in fresh version also.
You are right regarding inside declarations.
Let me more specific

We have ERP Software which is developed using VB.Net
Recently, We have add-on Tool DevExpress Winforms (https://www.devexpress.com/products/net/controls/winforms/)
In their tool, the provide End-User Report Designing Tool.
In Said tool, they provide facility to write end-user script.
If above end-user script, we write in C# then Confuser is working fine.
But If we write the in VB.Net, then Confuser is creating issue.

I'm ready to repeat issue at your end.
Regards,
Ritesh Mehta

Download supporting dll from below link
https://we.tl/t-yausjP4Lkk
And dump all dll into root folder of my sample project.

You did not understand me.
There is an attribute declared in the non-functioning assembly:
using System;

// Token: 0x02000016 RID: 22
internal class AritoCrypto : Attribute
{
	// Token: 0x06000072 RID: 114 RVA: 0x00002524 File Offset: 0x00000724
	public AritoCrypto(string A_1)
	{
	}
}

Where does this come from?

I can not help much regarding above code snip.
Regarding "AritoCrypto".. as I already said, I downloaded source of your project and did some title modifications.
And changed project title to "AritoCrypto".

Also the versions of the files you just send, do not line up with the versions in the original project. Please recreate a working example of the issue, otherwise there is nothing I can do.

The code snip of the attribute is from your assembly. This attribute is declared in the CrossTabIssue CONFUSER.exe file. It's not present in the CrossTabIssue.exe file. This means something modified the CrossTabIssue CONFUSER.exe aside from ConfuserEx.

Send you directory, in which error repeating
https://we.tl/t-G3JKF6SE0b
Please try it

Recording of error at my end

Okay, so the cause of the error is the visual basic compiler crashing when processing the assembly. This issue is caused by the presence of the constants protection. Obfuscated assemblies aren't working well as reference for an compiler.

Can it be fix in upcoming updates ?

No. The way this protection works, breaks the old visual basic compiler.

The same issue occurs, if we compile internal VB code unit using the Roslyn API instead of CodeDOM ?

It's very unlikely that the Roslyn implementation has the same issue.


multiple selection no X button

I use selectivity over a php generated <select>
I have no cross (X) button in my selected items like on the project page; did I miss an option somewhere?
$('#tag-list').selectivity({     multiple: true,     placeholder: 'Add tags' });
also is it possible for the user to add something not in the list ?

@signmeuptwice it seems like the fa-remove alias was removed between FA 4 and FA 5.
The quickest fix is to copy the template code for multipleSelectedItem from templates.js and change fa-remove to fa-times like so
      var extraClass = options.highlighted ? ' highlighted' : '';
      return (
          '' +
          (options.removable
              ? '' +
                '' +
                ''
              : '') +
          escape(options.text) +
          ''
      );
    }```

You could also patch your selectivity.js file with the change to the FA name


Error: io.socket is undefined

I am trying to integrate this with my angular project but as soon as I make a call inside a controller I get the following error:
 Error: io.socket is undefined
 _get@http://localhost:9000/bower_components/angular-sails-bind/angular-sails-bind.js:179:13

I have confirmed that sails.io.js is being included before angular etc etc, any ideas?

Same issue here too, did you manage to fix it?

Yeah I did, you need to manually include Sails.io.js before loading Angular Sails binds, the plug in does work it just not include the required file.

Yes, sorry for the long delay in answering, I've been really busy at may day-job.
You should include sails.io.js. It gets added by default on sailjs projects. I may need to add that to the readme.

Hi everyone. I think that I've had experienced this issue.
Here is how scripts included:
  <script src="js/dependencies/sails.io.js"></script>
  <!-- build:js(src) scripts/vendor.js -->
  <!-- bower:js -->
......
  <script src="../bower_components/angular-sails-bind/dist/angular-sails-bind.js"></script>
.......
  <script src="../bower_components/angular-socket-io/socket.js"></script>
.......
  <!-- endbower -->
  <!-- endbuild -->

And here is my code:
  /** @ngInject */
  function DataRowsController(..., $scope, $sailsBind, ...) {
...
    $sailsBind.bind("proxy", $scope);
...
  }

If I navigate to some route that not uses sails-bind (so sails.io will connect normally) and then navigate to route that uses it -- it works OK. But if I open app at route that uses sails-bind, it will fail with error:


[ENH] Running on individual files without BIDS structure

Hi there
I make simple change to fix bug of empty variable in case of a non bids structure.
The main disadvantage is that subject ID is set to "TODO".
The best would be to add the subject ID as input argument, but then I could not find how to set the subject ID. From debugging it seems that the subject ID is automatically guess when the pipeline are running.
If I am correct this means one should change the niworkflows package so that subject ID can be an input argument. If you think to an other way I will be happy to test

I changed the title so it is compliant with other PR (and looks good with the automated update of the CHANGELOG).

ok, It doest not pass the test, but seems not related to me ...?
On 06/16/2017 05:45 PM, Oscar Esteban wrote:

I changed the title so it is compliant with other PR (and looks good
with the automated update of the CHANGELOG).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/poldracklab/mriqc/pull/566#issuecomment-309061263,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFWhuhXHOLs_sRAjRcifKP2elvk5cHMOks5sEqL9gaJpZM4N8Qhw.

@oesteban We are very interested in this, let me know if there's anything we can contribute to get this going

@romainVala @jvarada sorry for the delays, I'm really swamped. I will have a look and let you know asap.

I guess the most outstanding issue in this PR is how to pass metadata on. Particularly, a way to inform MRIQC with metadata and a more reliable way to recover when metadata is not available (https://github.com/poldracklab/mriqc/pull/566#discussion_r127627957).


receiver: Expose stats for all tenants

Thanos Receiver supports the Prometheus tsdb status API and can expose
TSDB stats for a single tenant.
This commit extends that functionality and allows users to request
TSDB stats for all tenants using the all_tenants=true query parameter.

[x] I added CHANGELOG entry for this change.
[ ] Change is not relevant to the end user.

@yeya24 this is a follow up on what we discussed here: https://github.com/thanos-io/thanos/pull/5402#discussion_r887562883
Changes
Allow exposing TSDB stats for all tenants in Receiver.
Verification
Manual testing on local machine.

@bwplotka mind taking another look?

Added an update for the docs in the TSDB stats receiver section.


To improve in understanding

Hey there your website it to great but one thing that I noticed and I hope if you do this so your website look more cool that thing you write if the COVID-19 case are greater in past records so you write this line  Today is largest growth in COVID-19 case.

Ok as your wish thanks for read these and reply you answer.🙏🙏


After cutover with error, we need to start the full copy data again. Could the migratekit do the incremental copy after the failed cutover?

Dear Mohammed Naser,
          We use the MigrateKit and very happy with it. But when we try to cutover and the Error occurred, we need to start the full copy data again. In the case that the data storage more than 1 TB, it take time too much.

          Cloud the MigrateKit did solved this issue?

Thank you,
Apiwat P.

Unfortunately, if it fails during the virt-v2v stage, we're not able to reliably do a migration cycle that is differential based because the blocks were modified by virt-v2v.
What kind of issues are you running into, perhaps we can just fix those errors instead?

Dear Mohammed Naser,
              Thank you for your answer.

              There is error about the insufficient memory. We need to start the full copy again and can cutover success properly.

              The Migrate Server had been restart then the logs gone. Sorry about this if we found the logs of this problem again we will show the logs.

Best Regards,
Apiwat P.

Indeed, please open up a new ticket with the logs of the migration the next time around. :)


cli: rename workspace and programs

Addresses #1301

renames workspace and program, same time
expects user to be in root of the workspace
does not rename the workspace top-level directory

hi @paul-schaaf, if you can spare a bit of time, could you please review this? ty ty

Hey @paul-schaaf, I'm revisiting https://github.com/project-serum/anchor/issues/1301 after some time away. I think I've got renaming a program down, but hoping to get some input on what it means to rename a workspace.
First, I'm assuming a workspace is defined solely by Anchor.toml, based on seeing the function below, which was implemented before this PR and is used consistently in the cli.
// with_workspace ensures the current working directory is always the top level
// workspace directory, i.e., where the `Anchor.toml` file is located, before
// and after the closure invocation.
//
// The closure passed into this function must never change the working directory
// to be outside the workspace. Doing so will have undefined behavior.
fn with_workspace<R>(cfg_override: &ConfigOverride, f: impl FnOnce(&WithPath<Config>) -> R) -> R {
    set_workspace_dir_or_exit();

    let cfg = Config::discover(cfg_override)
        .expect("Previously set the workspace dir")
        .expect("Anchor.toml must always exist");

    let r = f(&cfg);

    set_workspace_dir_or_exit();

    r
}

The file contents in Anchor.toml define other tags, but relevant to this topic is the program names and corresponding program IDs. For example:
[programs.localnet]
anchor_rename_test_dir = "Fg6PaFpoGXkYsidMpWTK6W2BeZ7FEfcYkg476zPFsLnS"

So if I were to rename an individual program (using the change in this PR), the cli execution would look like this:

Check workspace root Anchor.toml and rename contents as needed.
Check workspace root Cargo.toml and rename contents as needed.
Check programs/<program_name>/Cargo.toml and rename contents as needed.
Check programs/<program_name>/src/lib.rs and rename contents as needed.
Rename directory programs/<program_name>
Check tests/<program_name>.js and tests/<program_name>.ts and rename contents as needed.
Rename file name tests/<program_name>.js and tests/<program_name>.ts

Now if I were to rename a workspace, cli should still follow the above, because it should check for existing programs of the same name as the workspace.
So is there any difference I'm missing between renaming a program and renaming a workspace? The only difference I'm seeing is the folder name of the workspace directory itself, but wanted to get your thoughts on that if you could.

before you continue, please sign your existing commits and force push (https://docs.github.com/en/authentication/managing-commit-signature-verification/signing-commits)

working on unique cases for certain files right now. from what i've been reading, package names in Cargo.toml can be snake or kebab right? and .rs files are snake only? i'm thinking of forcing casing here so no camel/shouty casing gets inserted in these.

working on unique cases for certain files right now. from what i've been reading, package names in Cargo.toml can be snake or kebab right? and .rs files are snake only? i'm thinking of forcing casing here so no camel/shouty casing gets inserted in these.

To add an example:
in the case that the user renames pasta to pasta-sauce
in src/Cargo.toml currently rename will replace all name = "pasta" to name = "pasta-sauce". from reading template.rs, the name =  under [package] should be pasta-sauce (direct replacement), but the name =  under [lib] should be pasta_sauce (snake case).
from reading The Cargo Book the restriction is alphanumeric characters, -, or _ (so camel case/shouty should work here too?)
as for [lib] is it forced to snake_case in template.rs because it should match pub mod pasta_sauce in lib.rs?
i'm thinking we can run through line by line and check for [package] and [lib] and enforce casing per tag.
anyways just wanted to make sure I understand the .toml format correctly - still new to rust 😅  thanks!


Xcode 11 Support to distribute .xcarchive.zip file instead of .ipa

+1 Be able to upload the application.xcarchive.zip file to HockeyApp
With xcode11, we will now have to deliver to Apple .xcarchive bundle.  We would like to be able to post the same .xcarchive bundle to HockeyApp.  Currently with Xcode 10 and below, we are posting .ipa.

Thanks for the feature request @lilyngabichtranADBE! We're currently in the midst of deprecating HockeyApp by Nov, 2019, so if anything, this feature will be implemented in App Center.
In the meantime, I'll let @derpixeldan and @botatoes share more details.

@derpixeldan HockeyApp used to (and still does) accept a .xcarchive.zip file... Would be cool if that were a feature in AppCenter too.

@blinkansalaryman I do see that the journey goes towards xcarchive.zip files but it requires us to process the contents of the file to serve the right .ipa to the right device when it comes to downloading/installing. On HockeyApp it's not possible to upload xcarchive.zip for iOS apps.

@BinkanSalaryman I do see that the journey goes towards xcarchive.zip files but it requires us to process the contents of the file to serve the right .ipa to the right device when it comes to downloading/installing. On HockeyApp it's not possible to upload xcarchive.zip for iOS apps.

@derpixeldan Ok, I'm not 100% sure what the issue is with it. But as a user, I can confirm that HockeyApp does have support for Xcode Archive files. Take a look here: https://hockeyapp.net/releases/mac/

@BinkanSalaryman that's the HockeyApp macOS app that handles the archive to extract the ipa and symbols and uploads them, however, HockeyApp itself does not support to upload/distribute .xarchive.zip. The .zip in the second screenshot is referring to macOS apps and is just an example. You cannot upload .zip for iOS apps.


Arisilon/issue 18 - [merged]

In GitLab by @arisilon on Oct 24, 2019, 14:10
Merges arisilon/issue-18 -> master

In GitLab by @arisilon on Oct 24, 2019, 14:11
enabled an automatic merge when the pipeline for 38576a27b8870cbc834869daaeb2ce3573a07061 succeeds

In GitLab by @arisilon on Oct 24, 2019, 15:23
merged

In GitLab by @arisilon on Oct 24, 2019, 15:23
mentioned in commit 6e1c84edffa4247f44c13500f335b36f851bce50


Support for Apple Metal (MPS) backend

Please add support for the MPS backend as you do for cuda:
if torch.backends.mps.is_available():
    mps = torch.device("mps")
    model = model_utils.DataParallel(model)
    model.to(mps)

# ... and so on...
x = x.to(mps)

Has anyone looked into implementing MPS for this yet?


OpenCV-Extra: 3.2.0 + examples

I've asked already here but just to check once again: should I make PR wth travis/appveyor changes to pkg.opencv or you will add extra testing branch pkg.opencv-extra?

should I make PR wth travis/appveyor changes to pkg.opencv or you will add extra testing branch pkg.opencv-extra?

I will add pkg.opencv-extra
Testing:

https://ci.appveyor.com/project/ingenue/hunter/build/1.0.2146
https://travis-ci.org/ingenue/hunter/builds/263971372?utm_source=github_status&utm_medium=notification

I see some tests are failing. Like Android build is failed on Linux while correctly build on OS X.
Also some issues on older MSCV.
OSX and iOS are failed as well, but I know that they are building no problem , as I use them in the project with this 3.2 version.
What do we do next? On some builds the log is truncated and I don't see the reason why it actually failed.

I'm testing it in different branch, I will add details when finish.

Passed:

https://ci.appveyor.com/project/ingenue/hunter/build/1.0.2163
https://travis-ci.org/ingenue/hunter/builds/264628853?utm_source=github_status&utm_medium=notification

Uploading:

https://ci.appveyor.com/project/ingenue/hunter/build/1.0.2165
https://travis-ci.org/ingenue/hunter/builds/264653221?utm_source=github_status&utm_medium=notification

Released: https://github.com/ruslo/hunter/releases/tag/v0.19.70


Indexed content dashboard

Create a dashboard to give totals of content indexed in various other services (Fedora, triplestore, Solr), so that end users can compare and make basic sanity checks.  E.g. "I have 100 nodes in Drupal, but do I have 100 nodes in Fedora?".

Since all of these pieces are optional, we'll have to use a plugin like system to ensure we aren't trying to query a triplestore that doesn't exist.

@rosiel @ajstanley Is there any work towards something like this with the RDM project at UPEI?

@dannylamb probably wouldn't be hard to use https://github.com/mjordan/islandora_repository_reports to do this.


Set aks default version to 1.20.13

│ Error: creating Managed Kubernetes Cluster "simphera-sswb-aks" (Resource Group "simphera-sswb-aks"): containerservice.ManagedClustersClient#CreateOrUpdate: Failure sending request:
StatusCode=400 -- Original Error: Code="AgentPoolK8sVersionNotSupported" Message="Version 1.20.7 is not supported in this region. Please use [az aks get-versions] command to get the supported version list in this region. For more information, please check https://aka.ms/supported-version-list"

closed by https://github.com/dspace-group/simphera-reference-architecture-azure/pull/44


📦💻 - Fix ftplugin

The problem was caused by a misconfiguration of NeoVim folders. After remapping the folders from ~/.vim to ~/.local/shared/nvim/site it started working as expected.
I've also added a Symbolic link from that folder to ~/.vim just for practicity.


BUG: Getting error in local dev mode

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
currently we are using  "react-icons": "^4.7.1" and we need  "react-icons": "^4.10.1" to fix this issue
Add ScreenShots

What browser are you seeing the problem on?
Chrome
Record

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

@TechSpiritSS


WIP: Add travis config file

This PR introduces the Travis yaml configuration file. This has been copied from Mirage with a couple of trivial changes.

I need to figure out what else to do to set up Travis.

I spoke with Sara Ogaz, who told me that all we need to do to get Travis to run on the repo is add the .travis.yml file to the repo. So I'll merge this and see what happens.


Transition notes: Simplified inline read path, refactoring RestrictedNotePresence

Who is this PR for?
9th grade counselors
What problem does this PR fix?
The dialog view for editing and viewing transition notes is more complex, and the inline view didn't let them read restricted information.
What does this PR do?
Refactors <RestrictedNotePresence /> to use it inline for reading transition notes.
Screenshot (if adding a client-side feature)

Checklists
Which features or pages does this PR touch?

[x] Student Profile
[x] My notes

Does this PR use tests to help verify we can deploy these changes quickly and confidently?

[x] Included specs for changes
[x] Manual testing made more sense here

selfie


help request: cannot import v3 module

Describe the issue
I want to import fluent-operator v3 on my golang project.
Since the module has v2 major version suffix, I cannot import v3.
go get<EMAIL_ADDRESS>
go<EMAIL_ADDRESS>invalid version: module path includes a major version suffix, so major version must match

go get<EMAIL_ADDRESS>go<EMAIL_ADDRESS>invalid version: go.mod has non-.../v3 module path "github.com/fluent/fluent-operator/v2" (and .../v3/go.mod does not exist) at revision v3.0.0

How did you install fluent operator?
No response
Additional context
No response

This should be fixed ： https://github.com/fluent/fluent-operator/blob/master/go.mod#L1 .