Foreign / Primary Key Validation

Hi,
Whilst looking at bug #1009 i tried to replicate but couldnt. When i checked further I noticed that neither foreign key integrity nor primary key uniqueness is validated. It is possible that my code syntax is wrong!
const alasql = require('alasql');

create_string=`CREATE DATABASE Fruits;
      USE DATABASE Fruits;
      CREATE TABLE Fruits (
        fruitid INT PRIMARY KEY,
        fruitname NVARCHAR(MAX),
        price MONEY
      );

      CREATE TABLE Orders (
        orderid INT PRIMARY KEY IDENTITY,
        fruitid INT REFERENCES Fruits(fruitid),
        qty FLOAT
      );
`

alasql(create_string);

insert_obj = {
    fruits : [
        {
            "fruitid" : 1,
            "fruitname" : "Banana",
            "price" : 2
        },
        {
            "fruitid" : 2,
            "fruitname" : "Apple",
            "price" : 1
        }
    ],
    orders : [
        {
            "orderid" : 1,
            "fruitid" : 9,
            "qty" : 7
        },
        {
            "orderid" : 1,
            "fruitid" : 9,
            "qty" : 7
        }
    ]
}

alasql('SELECT * INTO Fruits FROM ?',[insert_obj.fruits]);
alasql('SELECT * INTO Orders FROM ?',[insert_obj.orders]);

console.log(alasql("SELECT * FROM Orders ORDER BY name DESC"));

Output:

[ { orderid: 1, fruitid: 9, qty: 7 },
  { orderid: 1, fruitid: 9, qty: 7 } ]

Should this not error as I am both duplicating primay keys and using an invalid foreign key?

Sounds like a clear bug (or missing feature)
In any case, it should be very clear from the documentation if this is not implemented on purpose.

From bug #1009 it sound like this feature used to work. Looking at the code there is a function that verifies foreign keys / primary ket violation but I dont think this is being called. I will try to have a look further.

I will get hold of a release from before bug #1009 was reported to see if it used to be supported.

Thank you @akumadare - that would be great.


[bazel] allow specifying exec_properties in envoy_cc_test

Commit Message: [bazel] allow specifying exec_properties in envoy_cc_test
Additional Description: In order to pass properties such as sandboxNetwork as we are trying to do in Envoy Mobile here: https://github.com/envoyproxy/envoy-mobile/pull/2022
Risk Level: Low

Thanks @lizan!


--ignore does not work for more than one folder

e.g. fails if you do this
 pipreqs . --ignore dist build

It's comma separated.

Thanks. Not sure if there is a standard but I use setup.py and docopt. In
both of these repeated options are space separated.
On 13 March 2017 at 16:37, Vadim Kravcenko<EMAIL_ADDRESS>wrote:

It's comma separated.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bndr/pipreqs/issues/63#issuecomment-286164700, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABJN6euh4juUhsfFDntYky18hYWt_uM5ks5rlXDKgaJpZM4MbgKa
.

Ah. My mistake. Multiple option args are not possible so commas is the only way.


categorize tokens

https://github.com/jettjaniak/tinyevals/issues/9

[ ] Do not include dependency labels
[ ] Map the categories to all labels and save them as a pickl in a dict mapping it


GitHub Issues are disabled, but README recommends using them?!

Fixes
A short description of what this PR does.
Checklist

[x] I acknowledge that all my contributions will be made under the project's license
[ ] I have made a material change to the repo (functionality, testing, spelling, grammar)
[ ] I have read the Contribution Guidelines and my PR follows them
[ ] I have titled the PR appropriately
[ ] I have updated my branch with the main branch
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added the necessary documentation about the functionality in the appropriate .md file
[ ] I have added inline documentation to the code I modified

If you have questions, please file a support ticket.

Man, this has really gone downhill in the Twilio era 😆


Terminal error and more

I just upgraded to code-server 3.10.2, and I got a TON of errors. This is very important.

The terminal is screwed up, there are lines between the text.
It takes forever, and I literally mean forever, to load. It never happened at Code-Server 3.9.3.
I had to downgrade back. Can you please look up these errors on code-server?

Please fill out a proper bug report and we'll be happy to take a look!

Okay

Check issue at #3530 please


adding dark theme

I am very exited as this my first gssoc .I will be adding dark theme compatibility

Hello @codervivek5, assign the task to me, and I will implement the described issue.


[Hero] Hero image should fade between consecutive images

Hero image on franklin site jumps abruptly to next image
Firefly:
https://github.com/adobecom/firefly/assets/39915804/5cf18a2f-99cb-42cc-a02f-91e1bd9766c0
Franklin:
https://github.com/adobecom/firefly/assets/39915804/c70c64f6-b0bd-4b66-bb3a-b946b451790d

@tom-hightower this is fixed. Please review https://main--firefly--adobecom.hlx.live/.


Remove own-cluster SKR test

Description
The skr-aws-own-cluster-dev test is no longer required.
Changes proposed in this pull request:

Remove skr-aws-own-cluster-dev test

Related issue(s)

Removal in test-infra: https://github.com/kyma-project/test-infra/pull/8221


Update ADM references from public preview to private

After a discussion with David Tepper at MSFT, this is a not a public preview but private for the moment.

@slapointe : Thanks for your contribution! The author, @tfitzmac, has been notified to review your proposed change.

@slapointe - thanks for making this change.

#sign-off


Building with PyPy3.6 7.3.0 fails

My issue is about ...
Reproducing code example:
Building scipy 1.4.1 with PyPy3.6 7.3.0 on linux-x86_64 with gcc 7.3.0, numpy 1.18.1 pybind11 2.4.3 and cython 0.29.15 fails.
Error message:

  creating build/temp.linux-x86_64-3.6/scipy/_lib/_uarray
  compile options: '-I/home/conda/feedstock_root/build_artifacts/scipy_1584945645441/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_place/site-packages/numpy/core/include -I/home/conda/feedstock_root/build_artifacts/scipy_1584945645441/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_place/include -c'
  extra options: '-std=c++14 -fvisibility=hidden'
  x86_64-conda_cos6-linux-gnu-c++: scipy/_lib/_uarray/_uarray_dispatch.cxx
  In file included from /home/conda/feedstock_root/build_artifacts/scipy_1584945645441/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_place/include/Python.h:79:0,
                   from scipy/_lib/_uarray/_uarray_dispatch.cxx:2:
  scipy/_lib/_uarray/_uarray_dispatch.cxx: In member function '{anonymous}::py_func_args {anonymous}::Function::replace_dispatchables(PyObject*, PyObject*, PyObject*, PyObject*)':
  scipy/_lib/_uarray/_uarray_dispatch.cxx:722:8: error: base operand of '->' has non-pointer type '{anonymous}::py_ref'
     if (!PyTuple_Check(res) || PyTuple_Size(res) != 2)
          ^
  scipy/_lib/_uarray/_uarray_dispatch.cxx:734:8: error: base operand of '->' has non-pointer type '{anonymous}::py_ref'
     if (!PyTuple_Check(new_args) || !PyDict_Check(new_kwargs))
          ^
  scipy/_lib/_uarray/_uarray_dispatch.cxx:734:36: error: base operand of '->' has non-pointer type '{anonymous}::py_ref'
     if (!PyTuple_Check(new_args) || !PyDict_Check(new_kwargs))
                                      ^
  error: Command "/home/conda/feedstock_root/build_artifacts/scipy_1584945645441/_build_env/bin/x86_64-conda_cos6-linux-gnu-c++ -DNDEBUG -O2 -march=nocona -mtune=haswell -ftree-vectorize -fPIC -fstack-protector-strong -fno-plt -O2 -ffunction-sections -pipe -isystem /home/conda/feedstock_root/build_artifacts/scipy_1584945645441/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_place/include -fdebug-prefix-map=/home/conda/feedstock_root/build_artifacts/scipy_1584945645441/work=/usr/local/src/conda/scipy-1.4.1 -fdebug-prefix-map=/home/conda/feedstock_root/build_artifacts/scipy_1584945645441/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_place=/usr/local/src/conda-prefix -DNDEBUG -D_FORTIFY_SOURCE=2 -O2 -isystem /home/conda/feedstock_root/build_artifacts/scipy_1584945645441/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_place/include -fPIC -I/home/conda/feedstock_root/build_artifacts/scipy_1584945645441/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_place/site-packages/numpy/core/include -I/home/conda/feedstock_root/build_artifacts/scipy_1584945645441/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_place/include -c scipy/_lib/_uarray/_uarray_dispatch.cxx -o build/temp.linux-x86_64-3.6/scipy/_lib/_uarray/_uarray_dispatch.o -MMD -MF build/temp.linux-x86_64-3.6/scipy/_lib/_uarray/_uarray_dispatch.o.d -std=c++14 -fvisibility=hidden" failed with exit status 1

Scipy/Numpy/Python version information:
Building scipy 1.4.1 with PyPy3.6 7.3.0 on linux-x86_64 with gcc 7.3.0, numpy 1.18.1 pybind11 2.4.3 and cython 0.29.15 fails.

Duplicate of https://github.com/scipy/scipy/pull/11300


Specify redux as a peer dependency

Thanks for the suggestion, but I don't think this is necessary. It doesn't depend on redux. It only assumes its signature. It can work with any redux compatible store.


fix: Stringify response headers for intercepted requests

Stringifying the headers was the behaviour before v1.15
Relates to the discussion in #4379

Thanks! Can you please also update documentation to mention this behavior?


Fixed bdist_wheel bug by removing runtime dependency on setuptools_scm

Related Issues
Fixes #7
Proposed Changes

Removed setuptools_scm runtime dependency
Added backport dependency on importlib-metadata for python versions older than 3.8

Checklist

[x] Tests
[x] Coverages
[ ] Documentations

Codecov Report

Merging #41 into master will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master       #41   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           25        25
  Lines          354       353    -1
=========================================
- Hits           354       353    -1

Impacted Files
Coverage Δ

esmigrate/commons/__init__.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Updated to use ScaleIO v2 API

This commit adds the ability to integrate against ScaleIO v2
platforms.

Coverage remained the same at 92.193% when pulling 8f4cb73d14e37aaf641354ea408ab5712792fed8 on clintonskitson:feature/scaleio_v2 into 806dc9be95ad20cfbaeb52869f500047278f52a6 on emccode:release/0.3.3.

Tested with SIO v1.32, and v2.0.

Coverage remained the same at 92.193% when pulling 60a8435314c8bba3f8207eb71dce4a6d14111784 on clintonskitson:feature/scaleio_v2 into 806dc9be95ad20cfbaeb52869f500047278f52a6 on emccode:release/0.3.3.

Hoora! Thanks @oskoss and team!


Fix for cyw43 driver working with SMs other than 0

Fixes #1077
cyw43 driver is currently hard wired to use state machine (SM) 0. If a user program is allocated the same pio and SM 0 as cyw43 then the program will fail. Both cyw43 and user program will fall over each other trying to use SM 0.
This fix updates the cyw43 driver so that it doesn't assume that it has unique access to SM 0.
The cyw43 driver has a nod to playing fairly with the pio and sm but didn't quite make the required changes in cyw43_spi_transfer :)

can you please fix conflicts (presumably re-base onto newer develop)

I've just rebased onto develop. Added a const variable for fDebug flag.


Any idea what am I doing wrong?

Maybe the problem is here:
https://github.com/hauleth/agnoster/blob/master/fish_prompt.fish#L4
set -q $name; or set -g $name $argv

Try removing all those variables, AGNOSTER_SEGMENT_SEPARATOR, etc.
Then try using agnoster as you usually do.

Well, I fixed it by installing segment plugin, though it's not listed as a requirement.

@lburdzy What segment plugin? Do you mean https://github.com/fisherman/segment?

Yep, that's the one. Maybe installing it did not fix my problem, but caused it to be resolved, but it works fine now.

@lburdzy That's crazy, because I don't see Agnoster using segment anywhere in the source code haha.

Maybe I did something else that fixed it, but I definitly didn't touch any env variables.
Sorry about the mess, should I close the issue if it's resolved or should the maintainer do it? I'm pretty new to all that.

No worries, yeah if it's fixed for you go ahead and close it :)

Thanks for help :)


Update foxes.txt

Hi! Could you please change the text to read "Foo bar." and add some explanatory text to your commit message?


Does libp2p not include stun?

I can see code for stunning here. My understanding was that the p2p networking was handled by IPFS's libp2p. Is that correct? Does libp2p not handle stun natively?

I don't think it does. It's also broken atm. The utp lib they use has bugs in it and they are trying to find a replacement.


Fixed Auto away <-> Presence mixed up

Switched autoaway and presence values.
Heating graph max 24h, Don't know why yet but my initial data for the first half day draw the graph to 30h. Hadn't had time to look at the data in combination with the query.

I've got little time this week, so I closed this pull request and you can merge your changes. Didn't see the revert button. The changes I made weren't that big so no problem.
Keep up the good work


Unrevert streaming shadows

Commit Message: Revert the reversion of shadow streaming.
Additional Description: Shadow streaming was implemented in #24365, and rolled back in #25184 due to a bad interaction with retry buffering. This fixes the retry issue.
Risk Level: See #24365
Testing: Regression test for the issue added.

@snowp can you review this?

I believe that the failure is due to an actual bug in the way QUIC downstreams interact with filters that add bodies to header-only requests.

Turns out I just didn't understand how QUIC works :)


Fix exit() warning

Sorry it took me so long to reply to this. I had environment issues :(
Sorry I missed this, I was running different checks when I created the package :p
Everything works fine for me but I'm still getting other warnings:
 checking whether package ‘rhumba’ can be installed ... WARNING
  See below...

❯ checking top-level files ... WARNING
  A complete check needs the 'checkbashisms' script.
  See section ‘Configure and cleanup’ in the ‘Writing R Extensions’
  manual.

❯ checking for missing documentation entries ... WARNING
  Undocumented code objects:
    ‘MAMBA_ADD_PIP_AS_PYTHON_DEPENDENCY’ ‘MAMBA_ALWAYS_YES’
    ‘MAMBA_AUTO_ACTIVATE_BASE’ ‘MAMBA_CHANGE_PS1’ ‘MAMBA_DEV’
    ‘MAMBA_DRY_RUN’ ‘MAMBA_JSON’ ‘MAMBA_KEEP_TEMP_DIRECTORIES’
    ‘MAMBA_KEEP_TEMP_FILES’ ‘MAMBA_NO_PROGRESS_BARS’ ‘MAMBA_OFFLINE’
    ‘MAMBA_ON_CI’ ‘MAMBA_QUIET’ ‘MAMBA_USE_INDEX_CACHE’ ‘install’ ‘list’
    ‘print_context’ ‘set_channels’ ‘set_opt’ ‘set_root_prefix’
    ‘set_verbosity’
  Undocumented data sets:
    ‘MAMBA_ADD_PIP_AS_PYTHON_DEPENDENCY’ ‘MAMBA_ALWAYS_YES’
    ‘MAMBA_AUTO_ACTIVATE_BASE’ ‘MAMBA_CHANGE_PS1’ ‘MAMBA_DEV’
    ‘MAMBA_DRY_RUN’ ‘MAMBA_JSON’ ‘MAMBA_KEEP_TEMP_DIRECTORIES’
    ‘MAMBA_KEEP_TEMP_FILES’ ‘MAMBA_NO_PROGRESS_BARS’ ‘MAMBA_OFFLINE’
    ‘MAMBA_ON_CI’ ‘MAMBA_QUIET’ ‘MAMBA_USE_INDEX_CACHE’
  All user-level objects in a package should have documentation entries.
  See chapter ‘Writing R documentation files’ in the ‘Writing R
  Extensions’ manual.

❯ checking data for ASCII and uncompressed saves ... WARNING
    Warning: package needs dependence on R (>= 2.10)

❯ checking compilation flags in Makevars ... WARNING
  Non-portable flags in variable 'PKG_CPPFLAGS':
    -std=c++17

Don't seem like an issue to me, but if you think we should fix it I can do it.
I'll merge this one, thank you @SylvainCorlay :)


Jc/robust

This PR includes a number of robust tests for executing queries, especially around join queries. And addresses a bug or two along the way.

I'll rebase this one and it'll be much shorter after other PRs are merged

There are 3 tests that fail for me.  They are all similar to this:
dart:isolate  _RawReceivePortImpl._handleMessage

Failed to load "/Users/ross/src/aqueduct/example/templates/default/test/register_test.dart":
Unable to spawn isolate: Unhandled exception:
Could not import "package:wildfire/wildfire.dart" from "file:///Users/ross/src/aqueduct/example/templates/default/test/register_test.dart": No mapping for 'wildfire' package when resolving 'package:wildfire/wildfire.dart'.
null

Weird, where are you running tests from/command you are using?

Those are the templates tests that aren't really meant to run on their own - they get added with the template. Just not sure how they got ran


Error log on client for running after a long time

After running the client for 2 days, the following errors start coming up very frequently. Note that if I restart the client, these errors go away. I am guessing the pool is getting corrupted and every time we are asking for a connection, its creating a new one. This would have unwanted impact / delay in performance. Please advice.
[2015-09-18 10:47:53,419] [pool-706-thread-1] [WARN] [com.orientechnologies.common.log.OLogManager:140] [log] [serviceName:SubscriptionService]=> Caught I/O errors from Not connected (local socket=?), trying to reconnect (error: java.net.SocketException: Socket is closed)
[2015-09-18 10:48:02,601] [pool-434-thread-1] [WARN] [com.orientechnologies.common.log.OLogManager:140] [log] [serviceName:SubscriptionService]=> Connection re-acquired transparently after 10ms and 1 retries to server 'fsasydgrhdb03.foxsports.com.au:2424/subscription-service': no errors will be thrown at application level

Hi Sajid,
we took the issue in charge and we are working on it. We will provide a feedback ASAP.

Hi @sajid2045,
Yes if this happen every request may slow down the client, the reason for this problem could be many, from a network problem to a graph.shutdown problem, will be really helpful if you can take a memory dump of the client when this problem show up again.
Thank you

hi @sajid2045
Any news about this, did it happen again?

I suppose this is solved, closing, reopen it if happen again.


cache_timeout default is a bit of a landmine

The default caching behavior (that is, to have it enabled) was a pretty big source of confusion for me, causing all sorts of odd behavior in production but not in testing. Would you consider disabling all caching by default? I know it's simple enough to pass cache_timeout=0, but this is for the next person!

I'm sorry you were bit by stale filelistings caches. Can I ask what you were doing when this became an issue? Operations that add/remove entries from gcsfs itself should invalidate caches and for a reload when you next list.
I think, that for most other users, the current behaviour is the one wanted: listing directories can be expensive, and those listings are not generally expected to change.

I was using gcsfs in a service that would receive a notification that a set of files have been created on GCS. Since they were created externally and not by the local gcsfs, it wasn't in the cache -- I feel like this is not an uncommon pattern?
I agree that directory listing can be expensive, but I would just expect that caching to be done at the application layer.

Not sure what to say - the caching here and in s3fs was created in response to applications, particularly within Dask, where the file listing was significantly making operations slower. Typically, the data is written once, and then static, or gcsfs itself is doing the writing.
Given that the default has been the way it is for quite some time, I would need a lot of convincing that your situation, with data coming from a third party, is the common one.

Okay, I understand that changing probably has a fairly high bar. I'm just surprised in this day of distributed applications it hasn't come up more!
Anywhoo, thanks for the nice library -- much nicer than the google provided module.


Setting the default language for the app

This is a sub-tasks of Multi-Language support #25. Related to SDK issue: Set locale from within the app: support multi-language app #503
We need to test this from the SDK and give feedback for improvement/feature addition

Closed by https://github.com/opensrp/fhircore/pull/262


activesupport missing from gemfile?

I just tried running this and got: Could not find activesupport-4.2.3 in any of the sources.

Hi @mintcanary can you try running bundle install?

Oops. Yes, that installed it :)


Make IREE thread/synchronization compile on macOS

@benvanik: I'm not sure NULL is the proper attribute for initializing mutex and condition. Let me know what value should be. And this just enables it compiling on macOS. I'm seeing a test failure for ThreadTest.PriorityOverride.

NULL is fine for those!
What's the failure like?

+ /Users/antiagainst/iree/build-debug/iree/base/iree_base_threading_test
[==========] Running 5 tests from 3 test suites.
[----------] Global test environment set-up.
[----------] 3 tests from ThreadTest
[ RUN      ] ThreadTest.Lifetime
[       OK ] ThreadTest.Lifetime (0 ms)
[ RUN      ] ThreadTest.CreateSuspended
[       OK ] ThreadTest.CreateSuspended (151 ms)
[ RUN      ] ThreadTest.PriorityOverride
../iree/base/threading_test.cc:153: Failure
Expected: (nullptr) != (override0), actual: (nullptr) vs NULL
../iree/base/threading_test.cc:156: Failure
Expected: (nullptr) != (override1), actual: (nullptr) vs NULL
../iree/base/threading_test.cc:159: Failure
Expected: (nullptr) != (override2), actual: (nullptr) vs NULL
/Users/antiagainst/iree/build_tools/cmake/run_test.sh: line 33: 73578 Segmentation fault: 11  "$@"
+ cleanup
+ echo 'Cleaning up test environment'
Cleaning up test environment
+ rm -rf /Users/antiagainst/iree/build-debug/iree_base_threading_test_test_tmpdir

I haven't looked into the thread/synchronization magic you've landed recently so I'm not sure how to properly decipher this. :)

Weird that it would fail like that - according to the docs (https://opensource.apple.com/source/libpthread/libpthread-105.1.4/pthread/qos.h.auto.html) I'm calling it right. Maybe it is allowed to return NULL in cases when the arguments are valid. We could make the test gracefully fail (SKIP() or whatever the gtest macro is) if it returns null and only run the rest of the test (actually accessing the values) if it's valid.

SGTM. I can try to SKIP it later.


Added Andrew Schumaker

Added myself to the community folder.

I agree @gonuke, and thanks for doing the build because I deleted my build environment again.

Changes per request

Is the other line required? Whatever it was I removed I figured I didn't need since there was no useful information on it.
Respectfully,
Andrew Schumaker
Office: ERB 443
Email: @.***
Campus ID:<PHONE_NUMBER>

From: Paul Wilson @.>
Sent: Sunday, December 10, 2023 5:08 PM
To: cnerg/cnerg.github.com @.>
Cc: Andrew Schumaker @.>; Author @.>
Subject: Re: [cnerg/cnerg.github.com] Added Andrew Schumaker (PR #230)
@gonuke requested changes on this pull request.
Looks like you removed one line too many

In community/people/aps/index.mdhttps://github.com/cnerg/cnerg.github.com/pull/230#discussion_r1421834551:

@@ -0,0 +1,11 @@
+---
+layout: person
+title: Andrew Schumaker
+role: Graduate Student
+image: Andrew_Github.jpg
+email: @.***
+city_state_zip: "443 ERB, Madison, WI"

github: inquisitor-schumaker

Your last change took out this necessary line
⬇ Suggested change

github: inquisitor-schumaker
+services:

github: inquisitor-schumaker

—
Reply to this email directly, view it on GitHubhttps://github.com/cnerg/cnerg.github.com/pull/230#pullrequestreview-1774116462, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BCDE5TTONCDQJXIX7D4RR2DYIY6GHAVCNFSM6AAAAABAH65UVWVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTONZUGEYTMNBWGI.
You are receiving this because you authored the thread.Message ID: @.***>


Add functions needed for datastore changes

These changes should have no effect.

https://ocd.appscale.com:8080/job/Daily Build/1664/

https://ocd.appscale.com:8080/job/Daily Build/1675/


Dark / Light theme feature

Add dark and light themes feature to website
Tasks for #56

This PR add the dark and light mode button (right-top) in the navigation bar to switch between themes. Default theme is dark.
Tasks

[x] Create a theme switcher component using next-themes
[x] Add the button icon to navbar

Screenshots
Dark

Light

@khalilcodes, there is a file called obsidian_light.png in assets/images. I've added it some time ago when I was testing out different options for the hero. Can we maybe try how it would look like with the light version of both obsidian and our light themed website screenshots? For wrapping the Flowrshow screenshot in the browser frame I used: https://browserframe.com/

@khalilcodes in the doc page you mention that "The theme component from the package can be found in templates/default/pages/_app.js and its default mode can be configured there.". I'd rather add this option to siteConfig.js file (e.g. defaultTheme: 'dark') and allow users to overwrite it in their config.jsfile so that they don't touch_app.js`. What do you think?

@rufuspollock see also my comments to #107 (docs for this feature) as they are also connected to the implementation

@olayway content on the website screenshot is different now in docs/index (ie., Quickstart, getting started, etc) and would be different from the dark version. Should I use the new one ?

@olayway content on the website screenshot is different now in docs/index (ie., Quickstart, getting started, etc) and would be different from the dark version. Should I use the new one ?

Yes, and please retake the screenshot for the dark version as well if you can. You can use https://browserframe.com/ to add a nice frame to them.


Fix for downloading the linkerd2 binaries

Clones https://github.com/alexellis/arkade/pull/308 resolves https://github.com/alexellis/arkade/issues/327
Description
Appends the correct arch to the download URLs for linkerd2
Motivation and Context

[x] I have raised an issue to propose this change (required)

How Has This Been Tested?
Tested in pkg/get/get_test.go
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I've read the CONTRIBUTION guide
[x] I have signed-off my commits with git commit -s
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

[ ] I have tested this on arm, or have added code to prevent deployment

Compiled arkade on Darwin and installed linkred2 for darwin successfully


testutils: add require library

See doc.go for description.
Add a library inspired by github.com/stretchr/testify/require but
with simpler implementation and using generics. The benefit of
generics is that we don't have to cast constants, e.g.
require.Equal(t, a, uint32(2))).
We also have some additional assertions around channel receive and
function error argument unwrapping.

This change is

TFTR!


Joystick not initialized

ETS2Autopilot says joystick not initialized. how do i fix this?

how did you fix this:?


how should I edit settings after deployment?

for example, address of server for external clients.
I set it to a rounter's free ddns address, but it broke last week, so I bought a rounter from another company, so I have to use different address for it. I changed manually for already created clients, but pivpn -add will create ovpn files with old address with it. how can I change that?

You have two options:

Reinstall PiVPN and input the correct name. This is the easiest solution, but will require you to do the whole setup again.
Edit /etc/openvpn/easy-rsa/keys/Default.txt and replace remote old.ddns.com <port> with remote new.ddns.com <port>.

Just a note that /etc/openvpn/easy-rsa/keys/Default.txt is now found here: /etc/openvpn/easy-rsa/pki/Default.txt
See: https://github.com/pivpn/pivpn/blob/df16e8dd591b09913450279f7fad71df2af621da/auto_install/install.sh#L962

Just FYI,I had to sudo to be able to see/edit this file.


further improvements to command logging

hi!
so i'm happily using the logging stuff from recent versions to report what's going on to the user... unfortunately, things are still pretty noisy:
INFO:sh.command:<Command '/usr/bin/git -C /home/anarcat/wikis/anarc.at branch -a --color=never -v'>: starting process
INFO:sh.command:<Command '/usr/bin/git -C /home/anarcat/wikis/anarc.at branch -a --color=never -v', pid 21028>: process started
INFO:sh.command:<Command '/usr/bin/git -C /home/anarcat/wikis/anarc.at branch -a --color=never -v', pid 21028>: process completed
INFO:sh.command:<Command '/usr/bin/git -C /home/anarcat/wikis/anarc.at branch -a --color=never -v', pid 21028>: process completed

First, there's a duplicate line there - not sure why. But then I also feel like a bunch of those could be hidden in the DEBUG level: one line per call is basically what I expected to happen here. having only the last line, like this for example, would be fine:
INFO:sh.command:<Command '/usr/bin/git -C /home/anarcat/wikis/anarc.at branch -a --color=never -v', pid 21028>: process completed

Would you welcome a PR that would tweak that change?

I get significantly more repetitions of these multiple process completed entries in my output log.
2018-07-18T04:26:27.795225Z [info     ] <Command '/usr/local/bin/mongo example --quiet --eval db.test_collection.find( { creation_date: { $gte: ISODate("2018-05-01T00:00:000Z") , $lt: ISODate("2018-06-01T00:00:000Z") } } ).count()'>: starting process [sh.command]
2018-07-18T04:26:27.803997Z [info     ] <Command '/usr/local/bin/mongo example --quiet --eval db.test_collection.find( { creation_date: { $gte: ISODate("2018-05-01T00:00:000Z") , $lt: ISODate("2018-06-01T00:00:000Z") } } ).count()', pid 73087>: process started [sh.command]
2018-07-18T04:26:27.993515Z [info     ] <Command '/usr/local/bin/mongo example --quiet --eval db.test_collection.find( { creation_date: { $gte: ISODate("2018-05-01T00:00:000Z") , $lt: ISODate("2018-06-01T00:00:000Z") } } ).count()', pid 73087>: process completed [sh.command]
2018-07-18T04:26:27.994147Z [info     ] <Command '/usr/local/bin/mongo example --quiet --eval db.test_collection.find( { creation_date: { $gte: ISODate("2018-05-01T00:00:000Z") , $lt: ISODate("2018-06-01T00:00:000Z") } } ).count()', pid 73087>: process completed [sh.command]
2018-07-18T04:26:27.994571Z [info     ] <Command '/usr/local/bin/mongo example --quiet --eval db.test_collection.find( { creation_date: { $gte: ISODate("2018-05-01T00:00:000Z") , $lt: ISODate("2018-06-01T00:00:000Z") } } ).count()', pid 73087>: process completed [sh.command]
2018-07-18T04:26:27.994772Z [info     ] <Command '/usr/local/bin/mongo example --quiet --eval db.test_collection.find( { creation_date: { $gte: ISODate("2018-05-01T00:00:000Z") , $lt: ISODate("2018-06-01T00:00:000Z") } } ).count()', pid 73087>: process completed [sh.command]
2018-07-18T04:26:27.995106Z [info     ] <Command '/usr/local/bin/mongo example --quiet --eval db.test_collection.find( { creation_date: { $gte: ISODate("2018-05-01T00:00:000Z") , $lt: ISODate("2018-06-01T00:00:000Z") } } ).count()', pid 73087>: process completed [sh.command]
2018-07-18T04:26:27.995302Z [info     ] <Command '/usr/local/bin/mongo example --quiet --eval db.test_collection.find( { creation_date: { $gte: ISODate("2018-05-01T00:00:000Z") , $lt: ISODate("2018-06-01T00:00:000Z") } } ).count()', pid 73087>: process completed [sh.command]

If this is a bug it would be nice to get it fixed, if not, I'm really curious to understand what generates these and how I can prevent them. It seriously clutters up the output of scripts executing multiple queries. I like to leave info level logging turned on unless its prohibitive since it can be informative to know what a wrapper tool built with sh is doing, however this level of log output doesn't really make it possible to leave the level set at logging.INFO

Logging has been cleaned up a bit for the next release, including repetitive "process completed" entries and different events being moved to DEBUG.


Tic Tac Toe Game using Python

Description
I would like to create a tic tac toe game using tkinter library in python.
Type of issue

NOTE: These boxes can be checked using [X]

[x] Feature (New Script)

[ ] Bug

[ ] Documentation

Checklist:

[x] I have read the project guidelines.
[x] I have checked all the existing projects, before submitting a new project issue.
[x] I have checked previous issues to avoid duplicates.
[x] This issue will be meaningful for the project.

Please assign me this project

Please assign me this project

Sure @madhavmehndiratta

I would like to work on this , please assign me this project .

I would like to work on this , please assign me this project .

Sorry @Swatichanchal but this issue has been taken already

okay no problem .

also I can change the UI of the game , like playing tic tac toe in canvas . If you want me to do that .

also I can change the UI of the game , like playing tic tac toe in canvas . If you want me to do that .

To change something it needs to created  first, correct me if I am wrong @Swatichanchal

then let me create a different type of tic tac toe game , the concept will be same but the UI and method will be different

then let me create a different type of tic tac toe game , the concept will be same but the UI and method will be different

Having the same thing in different flavours won't be a meaningful contribution - we already have two command line tic tac toes and one GUI based is under progress so I don't think we want another Tic Tac toe because in the end all four of them will have the same functionality. Hope you understand
Thanks


Fixes Heap Allocations for Large alignMask

Uses the posix_memalign C function from the stdlib instead of malloc so that in case the alignMask is larger than what the system (or malloc) supports (on OS X malloc aligns memory on 16 byte boundaries so for larger values) we do not get unnecessary crashes.

The alignMask used by swift::slowAlloc() and the alignment used by posix_memalign() are not compatible. You need to convert them. (I think alignment = alignMask+1 works.)
Calling posix_memalign() for every allocation may be slow. You should measure performance. You may need to call malloc directly when alignMask is within the system's guaranteed alignment.

I could not find anything that showed that swift::slowAlloc() requires an alignment mask + 1 maybe I am missing something, but that should be an easy enough fix. However, we should make sure this is true because posix_memalign requires the alignMask to be a power of 2, or else will fail.
You are right, posix_memalign incurs a significant overhead in order to align memory, so we can use it only if the alignMask exceeds the system's guaranteed alignment. However, after a fair amount of research I've found that on Darwin, malloc promises a 16 byte alignment whereas on other posix systems the glibc only promises 8 byte alignment. There probably is a clean way to work around the platform dependent system size, any suggestions?

The compiler could emit more specific function calls when it knows the alignment needed for an allocation, and only fall back to slowAlloc when the alignment is dynamic. That's how @zarzycki's original allocator entry points were planned to work.


VS Code takes more time to bootup.

Type: Performance Issue
I am facing this issue since yesterday. i was doint my project on unity where it directed me to vs code for c# programming. where i performed some tasks and  soo n after that my VS Code now takess soo much longer time to boot up. wgen i open VS Code it takes that much time that the dialogue box appears to choose whether to close program or wait. and after waiting for minutes it opens up..
and one more issue there is usual error showing ("The connection to terminal's pty host process is unresponsible")
please look into my issue ASAP.
THANK YOU!!
VS Code version: Code 1.77.3 (704ed70d4fd1c6bd6342c436f1ede30d1cff4710, 2023-04-12T09:16:02.548Z)
OS version: Windows_NT x64 10.0.19045
Modes:
Sandboxed: Yes
Remote OS version: Linux x64 5.4.0-1105-azure

System Info

Item
Value

CPUs
AMD Ryzen 7 4800H with Radeon Graphics          (16 x 2895)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
undefined

Memory (System)
7.42GB (2.61GB free)

Process Argv
--crash-reporter-id 879b7e02-0632-4874-91b3-036ea1f9f298

Screen Reader
no

VM
0%

Item
Value

Remote
Codespaces

OS
Linux x64 5.4.0-1105-azure

CPUs
Intel(R) Xeon(R) Platinum 8272CL CPU @ 2.60GHz (2 x 2593)

Memory (System)
3.84GB (2.80GB free)

VM
0%

Process Info
CPU %	Mem MB	   PID	Process
    0	   104	  1484	code main
    0	   124	  1536	   gpu-process
    0	    47	  2336	   utility-network-service
    0	    26	  8560	   crashpad-handler
    0	    84	 10132	fileWatcher [1]
    0	   170	 11228	window [1] (README.md - MYNKcodes [Codespaces] - Visual Studio Code)
    0	    82	 12200	   window
    0	    90	 15404	shared-process
    0	    82	 15600	     ptyHost
    0	    91	 15528	window [2] (Issue Reporter)
    0	   175	 17952	extensionHost [1]

Remote: Codespaces
CPU %	Mem MB	   PID	Process
    0	     0	  2135	remote agent
    0	     0	  2168	   ptyHost
    0	     0	  2475	   extension-host
    0	     0	  3960	     /vscode/bin/linux-x64/704ed70d4fd1c6bd6342c436f1ede30d1cff4710/node /vscode/bin/linux-x64/704ed70d4fd1c6bd6342c436f1ede30d1cff4710/extensions/markdown-language-features/server/dist/node/workerMain --node-ipc --clientProcessId=2475
    0	     0	  2563	   fileWatcher
    0	     0	  4365	   /bin/sh -c /usr/bin/ps -ax -o pid=,ppid=,pcpu=,pmem=,command=
    0	     0	  4366	     /usr/bin/ps -ax -o pid=,ppid=,pcpu=,pmem=,command=

Workspace Info

|  Remote: Codespaces|    Folder (MYNKcodes): 2 files|      File types: md(1)
|      Conf files:;

Extensions (3)

Extension
Author (truncated)
Version

codespaces
Git
1.14.5

codespaces
Git
1.14.5

vscode-pull-request-github
Git
0.62.0

(1 theme extensions excluded)

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscorecescf:30445987
pythondataviewer:30285071
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593cf:30376535
pythonvs932:30410667
cppdebug:30492333
vsclangdf:30486550
c4g48928:30535728
dsvsc012:30540252
pynewext54:30695312
azure-dev_surveyone:30548225
vsccc:30610678
nodejswelcome1cf:30587006
282f8724:30602487
pyind779:30671433
89544117:30613380
pythonsymbol12:30671437
a9j8j154:30646983
showlangstatbar:30672706
vsctsb:30705553
azdwalk:30687957
pythonms35:30701012
pythonclientmv:30713943

Please check https://github.com/Microsoft/vscode/wiki/Performance-Issues on how to provide additional information on performance issues and append them here. Thanks!

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Building a Todo App - Step 22 - Improving destructuring hint

Describe the Issue
Step 22 of build a Todo app can use a better a hint or a short reminder on destructuring if one can be added into the instructions.
If we forget to put the curly braces around the parameter names to destructure them, the hint only says:

You should destructure id, title, date, description as the parameters of the callback of your forEach().

A better hint would be

You should use {id, title, date, description} as the destructured parameters of the forEach() callback function.

I think parameter destructuring was only covered once before this project. (though I may recall that incorrectly) here:
https://www.freecodecamp.org/learn/javascript-algorithms-and-data-structures-v8/learn-modern-javascript-methods-by-building-football-team-cards/step-26
Affected Page
https://www.freecodecamp.org/learn/javascript-algorithms-and-data-structures-v8/learn-localstorage-by-building-a-todo-app/step-22
Your code

taskData.forEach((id, title, date, description) => {});

Expected behavior
Either a better hint or a reminder of parameter destructuring in the instructions would be helpful.
Screenshots
No response
System
N/A
Additional context
No response

Hello. I would like to work on this issue. Please assign this to me.


Fix KGS currency template

Currently, the KGS currency format is coming as сом1,234,567.89 whereas it should be 1,234,567.89 сом.
https://en.wikipedia.org/wiki/Kyrgyz_som

@Rhymond review please

Heyo, thanks and sorry for late review, merging to master and will publish release shortly


ncost argument for 3d CostModelForce - [merged]

In GitLab by @cmastalli on Mar 6, 2019, 16:11
Merges topics/3d-cost-model-force -> devel
As suggested in #132, I introduced the ncost argument in the CostModelForce constructor. It values by default is 6, which correspondence to 6d contacts.
By passing ncost=3, the user is able to model 3d contact forces. General cases need to be tackled later.

In GitLab by @nmansard on Mar 6, 2019, 21:03
mentioned in commit 68da1b1d837f4ee9a96061942b861267054b80be

In GitLab by @nmansard on Mar 6, 2019, 21:03
merged


Update migrating-to-v1.md

Coverage remained the same at 77.17% when pulling bdcee39268e13e5dfd4437c085021897f5820192 on Skn0tt-patch-1 into 55c48934c64720c807e18cb353f13cc0a3b12289 on main.

Coverage remained the same at 77.17% when pulling bdcee39268e13e5dfd4437c085021897f5820192 on Skn0tt-patch-1 into 55c48934c64720c807e18cb353f13cc0a3b12289 on main.


Terms of service.

-I don’t think we need this. It is too intrusive. The user is agreeing to it during wallet creation. And also every time it changes via. release notes.

-verified this issue on staging app v1.4.5(228)


Trivial error cleanup

Done while working on other error code; two trivial cleanups.
This PR gives us a place to debate the current error From impl format. The canonical form is, as far as I understand:
impl From<FooError> for Error {
    #[inline]
    fn from(e: FooError) -> Self { Self::Foo(e) }
}

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

-9 of 10 (10.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 83.951%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

bitcoin/src/crypto/ecdsa.rs
0
3
0.0%

bitcoin/src/crypto/key.rs
1
7
14.29%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
19438

Relevant Lines:
23154

💛  - Coveralls

Started to test, but my local CI is complaining about the nightly thing fixed by https://github.com/rust-bitcoin/rust-bitcoin/pull/2512.

I'm trying to find the problem but I don't have a repro yet. Can you post an error or give me a tip please. just sane is running on master.


Code Sandbox Example Tests: Expect is not Defined

The latest example docs refer to the following code sandbox example for tests, unfortunately the tests fail to run, throwing the error: "expect is not defined".
https://codesandbox.io/s/github/kentcdodds/react-testing-library-examples/tree/master/?from-embed
Chrome 71
Windows 7

The examples have their own repo, let's move this over there https://github.com/kentcdodds/react-testing-library-examples


Defining cops' include/exclude overwrites department configuration (and there's no way to merge them)

Expected behavior
I expect that when I define inherit_mode.merge = ['Exclude'] on my rubocop.yml, the department exclude configuration is merged with the individual cops' Exclude. This should actually happen with Include too.
Actual behavior
When I have an Exclude option configured at the department level, it is always overwritten on individual cops if they define any Exclude themselves.
Steps to reproduce the problem
Given test.rb:
1 + 1

And given a .rubocop.yml:
inherit_mode:
  merge:
  - Exclude

AllCops:
  NewCops: disable

Style/FrozenStringLiteralComment:
  Exclude:
    - foo.rb

Style:
  Exclude:
    - test.rb

When I run rubocop --debug --only Style/FrozenStringLiteralComment test.rb I get:
For /Users/Drowze/workspace/ruby-scripts/rubocop_bug6: configuration from /Users/Drowze/workspace/ruby-scripts/rubocop_bug6/.rubocop.yml
Default configuration from /Users/Drowze/.asdf/installs/ruby/2.7.3/lib/ruby/gems/2.7.0/gems/rubocop-1.18.4/config/default.yml
Inspecting 1 file
Scanning /Users/Drowze/workspace/ruby-scripts/rubocop_bug6/test.rb
Loading cache from /Users/Drowze/.cache/rubocop_cache/f0c6549b3ba85dee6fbdac52c969fbbd0c5e7eeb/71f7dda238ff76ce0d032ce86f7a4921eb79aa1e/fcd33c8d415b48f56d850a02862f251c3520b52b
C

Offenses:

test.rb:1:1: C: [Correctable] Style/FrozenStringLiteralComment: Missing frozen string literal comment.
1 + 1
^

1 file inspected, 1 offense detected, 1 offense auto-correctable
Finished in 0.157611999893561 seconds

RuboCop version
$ rubocop -V
1.18.4 (using Parser <IP_ADDRESS>, rubocop-ast 1.8.0, running on ruby 2.7.3 x86_64-darwin20)

#9952 might have fixed this already (or #9624?)

I don't think it's related to #9952 as the issue is still reproducible using the steps given.
I would like to give a bit more details of our setup and our particular use-case. We currently use:

a gem to centralize our configuration (using inherit_gem to include it)
rubocop-rspec to check our tests styling
rspec to write our automated tests
rswag to write our request tests

rswag uses a DSL written on top of RSpec to describe requests tests. We do not want to run rubocop-rspec on our request tests as there would be a lot of false positives (see https://github.com/rswag/rswag/issues/138).
That being said, what we would like to have (but doesn't currently work) is:
our_gem's rubocop.yml:
RSpec:
  Enabled: true
  Excludes:
    - "spec/requests/**/*.rb"

project's .rubocop.yml
inherit_from: rubocop_todo.yml # line auto-added by rubocop auto-gen-conf

inherit_gem:
  our_gem: rubocop.yml

project's .rubocop_todo.yml (generated by rubocop auto-gen-conf)
# ... some comments added automatically ...
RSpec/LetSetup:
  # note: rubocop auto-gen-conf does not populate Excludes with anything from `spec/requests` (which is expected)
  Excludes:
    - spec/controllers/foo_controller.rb

With this particular setup, if we run bundle exec rubocop --only LetSetup, we would still get dozens of issues on spec/requests/
For this reason, as a temporary workaround we are not defining excludes at RSpec department level anymore: https://github.com/Over-haul/rubocop-overhaul/pull/24

@Drowze I commented on that rswag issue, but it should be possible to use rubocop-rspec now with it's configurable syntax. That's not really the point of this issue, but I thought I'd mention it.
I still think the fix in #9624 may solve your problem. Can you try checking out the branch from that PR and running it against your code?

@dvandersluis checking out to #9624 unfortunately did not fix anything for me.
Using example from first post
$ (9325_merge_exclude_by_default) git branch
* 9325_merge_exclude_by_default
  count-formatter-only-correctable-cops
  master

$ (9325_merge_exclude_by_default) git status
On branch 9325_merge_exclude_by_default
Your branch is up to date with 'jonas054/9325_merge_exclude_by_default'.

Changes not staged for commit:
  (use "git add/rm <file>..." to update what will be committed)
  (use "git restore <file>..." to discard changes in working directory)
        deleted:    .rubocop.yml

no changes added to commit (use "git add" and/or "git commit -a")

$ (9325_merge_exclude_by_default) cat /Users/Drowze/r/rubocop_bug6/.rubocop.yml
inherit_mode:
  merge:
  - Exclude

AllCops:
  NewCops: disable

Style/FrozenStringLiteralComment:
  Exclude:
    - foo.rb

Style:
  Exclude:
    - test.rb

$ (9325_merge_exclude_by_default) exe/rubocop --debug /Users/Drowze/r/rubocop_bug6/test.rb
warning: parser/current is loading parser/ruby27, which recognizes
warning: 2.7.4-compliant syntax, but you are running 2.7.3.
warning: please see https://github.com/whitequark/parser#compatibility-with-ruby-mri.
For /Users/OH-RGIBIN/workspace/rubocop: Default configuration from /Users/OH-RGIBIN/workspace/rubocop/config/default.yml
Inspecting 1 file
Scanning /Users/Drowze/r/rubocop_bug6/test.rb
For /Users/Drowze/r/rubocop_bug6: configuration from /Users/Drowze/r/rubocop_bug6/.rubocop.yml
Loading cache from /Users/OH-RGIBIN/.cache/rubocop_cache/efca944991c69c5a1cb1b4e2d92b6d039ab41014/6d7a3b621ca1730e04accd938619e4bdab66cfb1/82108402598d5a1ba449fe9300886deee1d98b9b
C

Offenses:

/Users/Drowze/r/rubocop_bug6/test.rb:1:1: C: [Correctable] Style/FrozenStringLiteralComment: Missing frozen string literal comment.
1 + 1
^

1 file inspected, 1 offense detected, 1 offense auto-correctable
Finished in 0.482789000030607 seconds

Using inherit_gem/rubocop_todo/etc
$ cat Gemfile | grep -E "\"rubocop\"|\"rubocop-overhaul\""
  gem "rubocop", path: "../../rubocop"
  gem "rubocop-overhaul", github: "Over-haul/rubocop-overhaul", ref: "a80d6eb7"

$ cat .rubocop_todo.yml
RSpec/LetSetup:
  Exclude:
    - 'test.rb'

$ cat .rubocop.yml
inherit_from: .rubocop_todo.yml

inherit_mode:
  merge:
    - Exclude

require:
  - rubocop-rspec

inherit_gem:
  rubocop-overhaul:
    - rubocop.yml
    - rubocop-rspec.yml

$ bundle exec rubocop --debug --cache false --only RSpec/LetSetup spec/requests/foo.rb
warning: parser/current is loading parser/ruby27, which recognizes
warning: 2.7.4-compliant syntax, but you are running 2.7.3.
warning: please see https://github.com/whitequark/parser#compatibility-with-ruby-mri.
For /Drowze/company/project: configuration from /Drowze/company/project/.rubocop.yml
configuration from /Users/OH-RGIBIN/.asdf/installs/ruby/2.7.3/lib/ruby/gems/2.7.0/gems/rubocop-rspec-2.4.0/config/default.yml
configuration from /Users/OH-RGIBIN/.asdf/installs/ruby/2.7.3/lib/ruby/gems/2.7.0/gems/rubocop-rspec-2.4.0/config/default.yml
Default configuration from /Drowze/rubocop/config/default.yml
Inheriting configuration from /Users/OH-RGIBIN/.asdf/installs/ruby/2.7.3/lib/ruby/gems/2.7.0/bundler/gems/rubocop-overhaul-a80d6eb7f493/rubocop.yml
Inheriting configuration from /Users/OH-RGIBIN/.asdf/installs/ruby/2.7.3/lib/ruby/gems/2.7.0/bundler/gems/rubocop-overhaul-a80d6eb7f493/rubocop-rspec.yml
Inheriting configuration from /Drowze/company/project/.rubocop_todo.yml
.rubocop.yml: inherit_mode:merge overrides the same parameter in .rubocop_todo.yml
Inspecting 1 file
Scanning /Drowze/company/project/spec/requests/foo.rb
C

Offenses:

spec/requests/foo.rb:44:9: C: RSpec/LetSetup: Do not use let! to setup objects not referenced in tests.
        let!(:id) { shipment.id }

A problem I see here is that it's not entirely clear what we mean by inheritance in RuboCop.
(1): A cop's parameter overrides the same cop parameter in another file. We reference that other file with inherit_from (for example), the documentation calls this inheritance, and we can use inherit_mode to modify how it works.
(2): A cop's parameter overrides the cop's department's value for the same parameter in the same file or in another file. This is actually not called inheritance in the documentation (as far as I could see), and inherit_mode doesn't affect it.
But maybe it's reasonable to call (2) inheritance too, and to let inherit_mode affect it. It might be hard to get it right, though, as the logic is already complex, so an alternative idea is to just clarify the documentation.

Just chiming in that I got surprised by this as well. I tried adding inherit_mode directly to the department config like so:
Style:
  inherit_mode:
    merge:
      - Exclude
  Exclude:
    - test.rb

But it didn't stick.


Added all Apollo3 boards and modified Example5_LCDDemo for them

Hello Andy,
I prepared something for you...
Best regards
Armin

Excellent!! That was fast. can you please bump library.properties to 1.2.8 so we can get your changes to Example 5 pulled into the Arduino library manager. Thanks again!

Done!


Privacy manifest required by Apple

Hi team,
Starting with Spring 2024 Apple requires third-party SDKs to add a Privacy Manifest file.
https://developer.apple.com/support/third-party-SDK-requirements/
https://developer.apple.com/documentation/bundleresources/privacy_manifest_files
Does this SDK need a privacy manifest? If yes, is there any plans to add a privacy manifest?
Thanks,
Duan

Hi,
Hope to get any updates,

大家好，
从 2024 年春季开始，Apple 要求第三方 SDK 添加隐私清单文件。 https://developer.apple.com/support/third-party-SDK-requirements/  https://developer.apple.com/documentation/bundleresources/privacy_manifest_files
此 SDK 需要隐私清单吗？如果是，是否有计划添加隐私清单？
谢谢， 段

This SDK is not on the list

@ptlCoder Do you mean that this SDK do need the privacy manifest since Apple does not highlighted?

I haven't found any calls to required reasons APIs, so from my point of view Privacy Manifest is not needed for this SDK.


Vintage network policies for CAPI clusters

CAPI clusters should have the same default network policies configured as our Vintage clusters in order to ensure:

the same cluster behavior for applications;
a smooth migration from Vintage to CAPI for AWS and Azure;

- [x] CAPA default network policies
- [x] CAPZ default network policies
- [x] CAPV default network policies
- [x] CAPVCD default network policies

Links

See Vintage network policies issue.

Old issue, this is already done. All of our providers have the network policies aligned.


Potentially breaking broker config change in version 2.8

Describe the bug
The default behavior changed for force deleting a namespace has changed from 2.7 -> 2.8.
https://github.com/apache/pulsar/pull/9819

@codelipenghui @cdbartholomew PTAL

@cckellogg We should disable the namespace force deleting by default, I can add this one in the release note.

I think the default settings should be true to maintain backwards compatibility.
forceDeleteTenantAllowed=true
forceDeleteNamespaceAllowed=true

The current settings are:
# Allow you to delete a tenant forcefully.
forceDeleteTenantAllowed=false

# Allow you to delete a namespace forcefully.
forceDeleteNamespaceAllowed=false

Ping for this issue. It was added to support:
[Issue 9790] Support disabled the tenants/namespaces force deletion in the broker.conf

I believe that is it safer to let this be 'disabled' and we can add something in the release notes.
We are now at 2.10, so if we change the default we will break compatibility with 2.8 and 2.9  users.
We did something wrong in breaking compatibility, we should not do it twice.
@hsaputra you could start a thread in dev@ mailing list

@cckellogg - we should just close this issue for now. It is already exist to 2 versions. Thanks

The issue had no activity for 30 days, mark with Stale label.

Closed as stale. If it comes back, let's open a new issue and refer here.


Add ESLint to the project

Adding ESLint will help maintain sanity in the codebase ensuring that the coding style is consistent. For example, since this is an open source project, there's a keen focus on readable code which means using if...else instead of ternary operators et cetera.

This issue has been met in this pull request


Update text in the Zabbix 3.4 section

I just tried two more things, fix the broken links, but I do fail in that. But I at least tried.
Also some updates and rewrites on the english text, typos. I also tried to be more explicit on where some tasks should take place.

@rouba002 ,
Thank you for taking the time to update; much appreciated!
Regarding broken links I utilize the github-markdown-toc tool which is very useful and generally works well.
I'll merge the PR and perform some minor clean up regarding grammar, etc.
Best Regards,
Andreas

@rouba002, I simplified your notes and fixed some minor syntax and other details to help make the document more readable and understandable.
Thank you again for taking the time and steps to test this in your instance.
I have yet to test your implementation but will plan to do so in the coming months.


Performance regressions from recursive type reference support

The extra work done in #33050 and follow-up PR #33678 is valuable, but maybe it doesn't need to be so expensive.
Setup:

Clone https://github.com/amcasey/material-ui.git
Check out Benchmark
yarn to restore packages
yarn typescript to prebuild and run some TS tests

Repro

tsc -p docs

On a random Mac Mini:
For #33050

10-run avg on https://github.com/microsoft/TypeScript/commit/4ee251b053eb2d419ce5304b309c5e4e1cd7ad1c: 33,685ms
10-run avg on https://github.com/microsoft/TypeScript/commit/250d5a8229e17342f36fe52545bb68140db96a2e: 34,223ms (2% slower)

For #33678

10-run avg on https://github.com/microsoft/TypeScript/commit/7f3c03d4414939bb48758c0f957babed830df48c: 34,403ms
10-run avg on https://github.com/microsoft/TypeScript/commit/7ce793c5b8c621af5ce50af0ca3958c7bd6541bf: 35,488ms (3% slower)

Note: to build old TS commits, you probably need to change const { default: chalk } to const chalk in scripts/build/utils.js.

FYI @ahejlsberg


Problem: block_id stored in a string but should be stored in an int

See https://github.com/bigchaindb/bigchaindb/pull/2163#discussion_r178039081

Note: The docs will also have to be updated in some places to reflect this.


Cannot pickle over 4GB

Hi, Great package, finally found this one that I was able to use for my purposes.  I am running into an issue using 3.7 python and pickle not being able to save over 4GB.  I have forked and updated line 139 of pickle_core.py to use protocol 4:
  pickle.dump(cache, cache_file, protocol=4)

However, I am not sure if earlier version of python would accept this hard coded protocol which is why I didn't do a PR.  I am wondering if you could update your package to include this somehow?
Thanks again for for cachier :)

Hey @jmrichardson ,
Why don't you create a PR with this change, and we will see how it works on all environments and versions on the Travis CI job that will run? :)
Then we can figure out how to condition the pickle protocol on Python version and/or file size.

Solved with release v1.4.3 of changes in PR #33 .

Thank you @jmrichardson !


race condition in land albedo read

This only happens for some specifications of the MPI setup as here. Currently, we get around this using this fix
This needs to be removed once land is fixed.

A fix for this race condition is made in ClimaLSM v0.4.0. #502 updated to use that version and removed this fix in ClimaCoupler, closing this issue


Revert "0.1.3"

Reverts danibene/docite#11

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 11 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-1.6%) to 91.791%

Totals

Change from base Build<PHONE_NUMBER>:
-1.6%

Covered Lines:
86

Relevant Lines:
89

💛  - Coveralls


Ignore 'local' folder for virtualenv

Whenever I use virtualenv, I get a folder called 'local' that I would like to ignore.
For reference, I am using virtualenv version 1.11.4 for Python 2 on Linux Mint 17.
I added [Ll]ocal. I do not know when it would ever be capitalized, but I assume there is such as case based on the other entries in this file.

Does anyone have any blocking concerns? If not, I think this is simple enough that it can just be merged.

@skylerberg we've been asking for extra details on where new rules are coming from as part of reviews, so that future visitors and contributors can follow along. Do you know what this folder represents, and why it can be safely ignored in source control?

@shiftkey Thanks for getting back to me on this. Virtualenv creates directories and files containing python packages to use instead of your system wide packages. As such, these are generated by scripts and contain large amounts of data unrelated to the repository. It is desirable for nothing from a virtualenv to end up in a repository as each user should create a new virtualenv locally.
This local folder in particular is added on Ubuntu machines because virtualenv needs to deal with Ubuntus package installation scheme. Here is the function that creates this folder: https://github.com/pypa/virtualenv/blob/develop/virtualenv.py#L1543. For reference, here is the commit that introduced the change: https://github.com/pypa/virtualenv/commit/285679cfd326c918676e765e06ed142db66efde0.

@skylerberg that's enough for me to refer back to. Many thanks! :heart: :sparkles: :tada:


Directives can be used without registering

Actual behavior
I didn't register v-tooltip but I could still use it:

Expected behavior
According to the docs, the users are required to set
export default defineNuxtConfig({
  modules: ['vuetify-nuxt-module'],
  vuetify: {
    vuetifyOptions: {
      directives: 'Tooltip' // or true
    }
  }
})

Reproduction
https://stackblitz.com/edit/github-fi3r47-mf4yjn

The v-tooltip directive is a wrapper around VTooltip component, and so, when using it the VBtn will try to resolve VTooltip component.


unsupported_grant_type error when trying to obtaint token from token endpoint

If i follow the instructions i get next error:
{
"error": "unsupported_grant_type",
"error_description": "AADB2C90086: The supplied grant_type [authorization_code\r\n] is not supported.\r\nCorrelation ID: 802cba29-bb35-4de8-b84d-08087f266bda\r\nTimestamp: 2020-03-08 15:24:36Z\r\n"
}
I'm using "Sign in" user-flow

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

ID: 836ec1be-7675-d0f1-c861-0520dc3ea01e
Version Independent ID: c6a2a6f6-57fd-20ae-36f7-b6db1987ebe5
Content: Request an access token - Azure Active Directory B2C
Content Source: articles/active-directory-b2c/access-tokens.md
Service: active-directory
Sub-service: b2c
GitHub Login: @msmimart
Microsoft Alias: mimart

example of request:

@alex-1244 Thanks for your feedback! We will investigate and update as appropriate.

@alex-1244 Instead of sending the request body as raw, you need to send it as form-data or x-www-form-urlencoded in below format:
grant_type:authorization_code
client_id:a7eb03a0-c3...
scope:https://tenant.onmicrosoft.com/api/read
code:eyJraWQiOiJjcGltY2...
redirect_uri:https://jwt.ms
client_secret:KuhgfUY...

Please let me know if you have any further question or shall we proceed with closure of this thread.

When the including using Content-Type: application/x-www-form-urlencoded this returns back a 400 bad request:
{"error":"unsupported_grant_type","error_description":"AADB2C90086: The supplied grant_type [authorization_code\r\n] is not supported.\r\nCorrelation ID: 75c409dc-f022-49db-9935-2b7aa05b83c4\r\nTimestamp: 2020-03-09 05:57:48Z\r\n"}

All post request must be in form-data

If you use postman, make sure the application/x-www-form-urlencoded is configure is the right place (under the headers), and not in the body.

I used "Content-Type: application/x-www-form-urlencoded" header in requests
But apparently it's not the same as setting "body type" in postman

So it's working now, sorry for disturbance, thank you.


Your example

I using this example for my contract
https://github.com/OpenZeppelin/zeppelin-solidity/blob/master/contracts/examples/SampleCrowdsale.sol
I`m wrong or this way written the mint function revert because is not executed by the owner.
Upon a purchase I got revert error - when I delete onlyOwner on mint function of the token - everything is correct.
I use your latest code from GitHub

The crowdsale must be the owner of the mintable token if using a mintable token and mintable crowdsale. See the tests for an example: https://github.com/OpenZeppelin/zeppelin-solidity/blob/master/test/crowdsale/MintedCrowdsale.test.js#L22

So you mean after I deploy my contract I must run a transaction to actually own my own token ?
Also, how I`ll go to do this in a real network?

After you deploy your token contract, the deployer is the owner. Only the owner is allowed to mint tokens, so to use a MintableCrowdsale, the crowdsale must be the owner. You can change the owner of the token by doing token.transferOwnership(crowdsaleAddress).
If you are using truffle to deploy, you can make this transaction part of your deploy script

Is there any different to go with your way or create the token contract directly in my crowdsale contract?

The old version of OpenZeppelin's crowdsale created the token as part of the crowdsale, but there were a few issues with it, mainly inflexibility and the increased deployment size.
You can create a Factory contract that

deploys the token with your arguments
deploys the crowdsale with your arguments
transfers the ownership of the token to the crowdsale

all in one transaction if you'd like

OK thanks for your help my problem is resolved


Upgrade to Symfony 2.7 LTS

:sparkles: @DRvanR :sparkles:

[ ] Update Stepup bundles to dev-develop.

:+1:

:bomb: @DRvanR


setUserId incorrect method name in iOS

When calling setUser in analytics in iOS this references the native side method as "setUserId" instead of "setUser". If you attempt to call setUser in javascript this will reject the promise with an error.
This is in file react-native-firestack/lib/modules/analytics.js:34
Currently is:
setUser(id, properties={}) {
return promisify('setUserId', FirestackAnalytics)(id, properties);
}
Should be:
setUser(id, properties={}) {
return promisify('setUser', FirestackAnalytics)(id, properties);
}

@Paul-Todd thanks for this 👍  this has been fixed in the upcoming v3 release of firestack - feel free to check out the v3 branch and use it directly, its stable.
A lot of the missing analytics methods have also been added as well as android support. Just note that v3 now heavily mirrors the firebase web sdk.
See here for the available methods.
Closing.


IndexError: index 1014 is out of bounds for dimension 0 with size 1014

I changed a data set, and the rest is consistent with your code, but  the result above appears. I don't know the reason, please kindly answer for me in your busy schedule, thank you

can you provide more information about the dataset you are using?

I use the data set is more than 1014 characters in a row, but when change the lo to 4056, and an error is as follows: a RuntimeError: size mismatch, m1:32 x [37376], the m2: [8704 x 8704] at/pytorch aten/SRC/TH/generic/THTensorMath CPP: 2070

I want to know what parameter values the CNN model needs to adjust when changing lo value, namely max length of a sample

Hi, it seems that the argument "--l0" sets the max length of samples. Feel free to adjust it to a larger value

@zhijing-jin thx!
@feiyangdezhu issue fixed, try again.


[ModalManager] Prevent overflow: hidden

I have a side Drawer and I don't want my body to be set to overflow: hidden what would be the best way to achieve this?
https://github.com/callemall/material-ui/blob/v1-beta/src/internal/modalManager.js

There is no option to disable the default overflow behaviour. I guess that's something we could support, but I'm wonder if the use case is legitime. Anyway, you can always provide your own implementation of the modalManager without the feature. If you find that appropriate, we will close the issue.

I couldn't find an option for disabling that behavior on the bootstrap project (we follow the same approach). Still, feel free to add an option to the createModalManager, you could call it enableScroll.