Json Output: Respect Column Filter

Currently there is no way to exclude columns if JSON output is used.
This is a problem because on a list resource command every resource needs to be decrypted which may timeout depending on the amount of resources.

Hi @speatzle,
I think it's good, that you want to tackle this. This improvement was also already in my mind.


eslint process returning incorrect exit/error codes

Version
3.4.1
Reproduction link
https://github.com/ffxsam/repro-errors-passing-build
Environment info
  System:
    OS: macOS 10.14.3
    CPU: (12) x64 Intel(R) Core(TM) i7-8700B CPU @ 3.20GHz
  Binaries:
    Node: 10.15.0 - ~/.nvm/versions/node/v10.15.0/bin/node
    Yarn: 1.13.0 - /usr/local/bin/yarn
    npm: 6.8.0 - ~/.nvm/versions/node/v10.15.0/bin/npm
  Browsers:
    Chrome: 72.0.3626.119
    Firefox: 65.0.1
    Safari: 12.0.3
  npmPackages:
    @vue/babel-helper-vue-jsx-merge-props:  1.0.0-beta.2
    @vue/babel-plugin-transform-vue-jsx:  1.0.0-beta.2
    @vue/babel-preset-app:  3.4.1
    @vue/babel-preset-jsx:  1.0.0-beta.2
    @vue/babel-sugar-functional-vue:  1.0.0-beta.2
    @vue/babel-sugar-inject-h:  1.0.0-beta.2
    @vue/babel-sugar-v-model:  1.0.0-beta.2
    @vue/babel-sugar-v-on:  1.0.0-beta.2
    @vue/cli-overlay:  3.4.1
    @vue/cli-plugin-babel: ^3.4.0 => 3.4.1
    @vue/cli-plugin-eslint: ^3.4.0 => 3.4.1
    @vue/cli-service: ^3.4.0 => 3.4.1
    @vue/cli-shared-utils:  3.4.1
    @vue/component-compiler-utils:  2.6.0
    @vue/preload-webpack-plugin:  1.1.0
    @vue/web-component-wrapper:  1.2.0
    eslint-plugin-vue: ^5.2.2 => 5.2.2
    vue: ^2.6.6 => 2.6.8
    vue-eslint-parser:  5.0.0
    vue-hot-reload-api:  2.3.3
    vue-loader:  15.7.0
    vue-style-loader:  4.1.2
    vue-template-compiler: ^2.5.21 => 2.6.8
    vue-template-es2015-compiler:  1.9.1
  npmGlobalPackages:
    @vue/cli: 3.4.1

Steps to reproduce

yarn
yarn build

What is expected?
I expect an error return code (so the CI build would fail)
What is actually happening?
Return code 0 (success) is returned, so app deploys

https://cli.vuejs.org/config/#lintonsave

@sodatea The behavior with lintOnSave: 'error' is still broken.

yarn serve: eslint warnings cause the app to fail
eslint warnings cause yarn build to fail

This is not handled properly. Warnings are just that: warnings.
According to the eslint docs (emphasis mine):

"off" or 0 - turn the rule off
"warn" or 1 - turn the rule on as a warning (doesn’t affect exit code)
"error" or 2 - turn the rule on as an error (exit code is 1 when triggered)

Umm…
Though, you can customize it with eslint-loader config
There must have been a design tradeoff or implementation mistake here but I'm too sleepy to further investigate into this. Will have a look at this later.

Related code: https://github.com/vuejs/vue-cli/blob/dev/packages/%40vue/cli-plugin-eslint/index.js#L49-L50
Maybe we should use failOnWarning /failOnError instead of emitWarning/emitError

Thanks for taking a look. I'll look at that webpack config.
But besides that, I'm speaking of the vue-cli developer experience out of the box. The way eslint behaves is incorrect and shouldn't require any config for it to work the way it should.

The implementation has a few pounds of historic baggage from the old days of the webpack template, and some issues in making both eslint-loader and friendly-errors-webpack-plugin play nice together.
Additionally, the default of not emitting any errors was chosen so the linting errors doesn't halt HMR updates during development (See: https://github.com/webpack-contrib/eslint-loader#emitwarning-default-false)
But I think we could try and improve the behaviour for production builds.

Ok, so right now we either:

emit everything as warnings (lintOnSave: true)
emit everything as errors (lintOnSave: 'errors')

A third options would make sense that does what would be the default:

emit warnings as warnings and errors as errors (lintOnSave: 'normal' or what should it be called?)

Do we need a 3rd option or can we change behaviour of one of the existing ones? I say we can't, as that would be a breaking change. We could however switch to better API (where Option 3 is the default for true) with the next semver major release.

@LinusBorg Thanks! I think the new default of lintOnSave: true which would emit warnings as warnings, and errors as errors makes sense, for both yarn build and yarn serve (if I have a rule to be flagged as an error, I would expect it to stop my app from building during local development).
I tried your suggested workaround and noticed a few things:

It doesn't matter what I set emitWarnings and emitErrors to, it has the same behavior either way. They appear to have no effect.
Adding this config.module code for eslint causes strange behavior upon running yarn build, for example:

/Users/samh/granted/src/App.vue
  5:7  error  Attribute ":two" should go before "one"  vue/attributes-order

It's complaining about this code:
<div
  one="one"
  :two="two"
  three="three"
/>

There's no rule in vue/attributes-order about v-bind attributes coming before static attributes. In fact, the example on that page shows this as being good code:
  <div
    prop-one="prop"
    :prop-two="prop"
    prop-three="prop">
  </div>

It doesn't matter what I set emitWarnings and emitErrors to, it has the same behavior either way.

Likely because I misspelled the options: emitWarning / emitError (singular)

the mere presence of the code you suggested does make errors get emitted properly.

No idea.

There's no rule in vue/attributes-order about v-bind attributes coming before static attributes. In fact, the example on that page shows this as being good code:

Also no idea. Please provide code to actually run.

@LinusBorg That's my thinking, that yarn build is using a different eslint version than the rest of the vue-cli.. somehow.
Go ahead and (in the same repo) checkout branch webpack-manual-config. Steps to reproduce:

yarn lint (notice there are no errors or warnings, which is correct)
yarn build (console.log error is expected; vue/attributes-order error is not expected - conflicts with the official rule specs)

As a side note, I noticed if I run yarn build a second time, it ignores any warnings and errors and builds with a success result. Is this intended behavior? It's apparently checking if the bundled filename hash has changed, and if it hasn't, it doesn't lint the file.

To avoid breaking changes I think it's better to adjust it to the following:

"default": emit warnings and warnings and errors as errors (what @ffxsam wants)
"warnings": emit everything as warnings (avoid errors to pop up in overlay during development)
"errors": emit everything as errors
true: same as "warnings" (also same as current behavior so it doesn't have to be breaking, can be deprecated in next major)

@ffxsam Hm, looking at the implementation, there'S a reference to an older issue about two versions of eslint, and supposedly, this commit was meant to fix it:
https://github.com/vuejs/vue-cli/commit/077343ba6accb3eec9334c0b0f023d89d704b9f9
I'll check out your repro later tonight, but it seems like a bug unrelated to the improvement disucssed in this issue here, so we should maybe open a new one for the bug.

Submitted a PR to add the 2rd option.
@ffxsam Also installed your repro and only receive one error: that a bout the console. Can you delete the lockfile and /node_modules/.cache and reinstall? maybe some dependency mixup...
If the problem persists for you, we should investigate (in a separate issue), but right now i can't reproduce what you are xperiencing, and the commit I previously linked to should have fixed it  ...

@LinusBorg Thanks! I'm happy to wait till this PR is merged and published, then I can open a new issue if the problem still persists.

@sodatea @LinusBorg I'd suggest this be the default behavior in vue-cli projects. I noticed it was not, but once I put this in vue.config.js:
module.exports = {
  lintOnSave: 'default'
}

Then it behaved as expected. I imagine if someone wants eslint to throw errors, they'd want it to prevent their build from succeeding.

In case this helps others who happen upon this thread from Googling:
Setting lintOnSave to default did not work for me as I still got an exit code of 0 even though the linter found errors that it auto-fixed as part of the pipeline.
So I ended up running the lint command with these flags on the pipeline to get the appropriate error code:
vue-cli-service lint --max-warnings 0 --no-fix
Hope that helps :)


Create terminal session in cloned directory

Similar to how https://asciinema.org/ drops you into a new shell session, then when completed gives options such as upload/cancel.
Not sure if this is possible with node. I've tried spawning bash with no luck. Will require some research.

This would be for situations where you need to run multiple/more complex commands before running the code in a cloned repo.
e.g. build, make, run migrations/seeds

Something like https://github.com/dthree/vorpal might work.

https://github.com/ialpert/vorpal-as-default


[1.16.5] Filling Jerrycan from Immersive Engineering with water causes client crash

Description of the issue:
Holding a Jerrycan and right-clicking while looking at water causes a client crash - The server the client is connected to does not crash. Apotheosis is mentioned in the log so I thought I'd put the issue here in addition to IE's tracker.
Issue on IE's tracker: https://github.com/BluSunrize/ImmersiveEngineering/issues/4672
Crashlog:
https://gist.github.com/NillerMedDild/3f26127eb12afc0a76133417dc3086a5
Versions & Modlist

ImmersiveEngineering-1.16.5-4.2.1-131
Forge-1.16.5-36.0.40
Apotheosis-1.16.3-4.4.1 (Also mentioned in crash report)

Modpack is Enigmatica 6 v0.4.0: https://www.curseforge.com/minecraft/modpacks/enigmatica6/files/3214669
Modlist: https://github.com/NillerMedDild/Enigmatica6/blob/master/MODLIST.md

This code has been deleted and made into a safer event implementation in Placebo, so my involvement in this stacktrace is resolved™


Option to sample transaction creation

The current transaction_sample_rate does not seem to cause fewer transactions to be captured:

We still record overall time and the result for unsampled transactions, but no context information, labels, or spans.

The tuning docs are misleading on this topic and should possibly be reworded:

Sample rate is the percentage of requests which should be recorded and sent to the APM Server.

For a specific HTTP application of ours, which already does not emit any spans (because there are no instrumented methods, automatically or otherwise), transaction_sample_rate did not result in any fewer events being sent to the APM server (because they were already all transactions). We are still interested in the http.request.* metadata but we could deal with having it for e.g. 10% of the incoming traffic.
This application has fairly high traffic and our small-ish Elastic cluster is having issues as a result.
How can we configure the APM agent so that fewer transactions will be sent to the server?

Thanks. The agent handles this slightly differently for different server versions. Prior to 8.0, the transaction data is sent but drops contextual information to reduce the transfer and storage size of the trace. From 8.0 the transactions should be dropped by the agent and no transfer should happen.
If the transactions are dropped entirely for pre-8.0 servers, the statistics and charts would display incorrect data because they aren't prepared for that, they use the non-dropped transactions for the stats. I wouldn't recommend doing so. If upgrading to the latest server version is not possible and you nevertheless want to try your agent with dropping those transactions, it's perfectly possible for you to fork the agent or add a community plugin which can force that (eg changing this method)
You're right the documentation is a little misleading for pre-8.0 server, I'll change it to make it clearer

The agent handles this slightly differently for different server versions

Oh, OK, I did not realize this could be a thing! Yes it probably warrants some mention in the docs.
Are there other differences in agent behavior depending on the target APM server?

All related to comms protocol, statistics, or improved/additional server features like span compression

Thanks @jackshirazi.


browser compatibility

Hi,
What´s the browser compatibility list ?
Does it work in IE ?
Thanks

@mehtabhaumik Thank you, Circliful didn't work in Safari but with that piece of code it does. Great! :D

For those looking to support ie8 with excanvas, in additional to the animation fix mentioned above (which you only need if you turn animation on), you also need to add the following line right before "var context = canvas.getContext('2d');":
        if (typeof(canvas.getContext) == "undefined" && typeof(G_vmlCanvasManager) != "undefined") {
            canvas = G_vmlCanvasManager.initElement(canvas)
        }


[Project question] convert *glib.Value to int

I have a *gtk.ListStore with 2 values glib.TYPE_INT, glib.TYPE_STRING
I can get the string using GetString method but I cant find a way to get the int

You've probably found this by now, you have to get a goValue, and then assert that as an integer.
goValue, _ := value.GoValue()
key := goValue.(int)


Check if Incremental Regen is enabled before using it.

.

Decided it wasn't worth it, no reason to do this, Jekyll has a grip on it.


Update Draw.js

fix remove last point in draw interaction

Hi,
Thanks for contributing. I'm sorry but this will need a bit more context on what you're trying to achieve with this change!

Hi, removeLastPoint function is not working properly, next to last item in sketchLineCoords_ array but last not updated

the last two coordinates are repeated

I believe this only happens on mobile

Sorry, that's still not clear enough. Please refer to the contributing guide as to what your PR shoud look like. Thanks!


Brpc write buffer fail when querying with countDistinct()

SQL:
select count(distinct(version))
from test.client
where toDate(timestamp)<'2023-10-04'  AND event >= '1000' AND  event <= '1060';

Error:
DB::Exception: ExchangeSource: BrpcReciver[xxxxx.xxxxx:xxxx]@xx.xxx.xxx.xxx:xxxx fail to receive data: BrpcRemoteBroadcastReceiver: already has been finished code: 9014 exception: Segment: 1, exception:Code: 2010, e,displayText() = DB::Exception: [Proxy]BrpcSender with keys: xxxxxx.xxxxxxxxx
fail to send data: Code: 2007, e.displayText() = DB::Exception: Write stream-8589940585 buffer fail, io buffer size is bigger than 671088640 current is<PHONE_NUMBER> SQLSTATE: HY000,. ()

@kevinthfang

Could u increase the timeout and retry  --receive_timeout=6000 --max_execution_time=3600 --exchange_timeout=3600000 --exchange_enable_multipath_reciever=0
There might be some issues on worker, could u check the log of workers.


Improve contrast ratio of toolbar buttons

Currently, the buttons in the toolbar in the drink and liquor activities are not visible for some photos,
since they are white buttons on a white background.
I've added a scrim at the top of the CollapsingToolbarLayouts in activity_drink and activity_liquor to make the buttons more visible.

Thank you for this :+1: LGTM

Thank you for this :+1: LGTM


[Feature Request] Index Page Implementation

Overview
Since we cannot implement a search feature at this time, an index is needed as a quick way to find articles. This page should be a simple list that contains all of the articles in the survival guide in alphabetical order. Since we aren't sure how long the MVP will last without a search feature, the index should be easily scalable to contain a large amount of articles. A link to the index should also be placed in common locations that a user would expect a search bar.
Proposal
In the index page itself, there should be:

Page title
Top bar, footer, and sidebar that are consistent with the rest of the site
Two columns of links to all articles in the website (desktop)
Endless scrolling (desktop)
One column of all links to all articles in the websites (mobile)
No more than 25 rows of links (mobile)
Links should be listed in alphabetical order and left aligned

As for the rest of the website, the index page should be:

Linked where a search bar would be in the top bar
Accessible from every page on the website

Supplementary Information
Here's an example of an index from Fandom.
Our version definitely doesn't have to be this complex, but this contains the main elements we need.

Closing as duplicate of #47, which has been implemented.


Turn the whole message thread into a click-area for input focus

The better chat clients out there behave in a way where a click somewhere onto the message thread will focus the input so you can easily add a reply (most likely the primary action apart from reading/scrolling).
http://en.wikipedia.org/wiki/Fitts's_law

Why the whole message thread? Is the input area not enough?

The larger the click-area is the easier it is to perform the action. If you've tried chat clients that support this behaviour it is frustrating to go back to ones that don't.
An even better behaviour could be to just allow typing at any time (whether input is focused or not) whether the input field is focused or not. This might make some potential keyboard combinations impossible but imho the use-case of writing a reply is so fundamental to chat that it is worth the trade-off.

I think that goes beyond the scope of this project. The aim is to provide Messenger on desktop, not to reinvent, or derive from it. I do see merit in your suggestions, but it will probably be up to the Messenger team to provide such functionalities.


MINOR: Add test for ConsumerNetworkClient.trySend

add test for the change of ConsumerNetworkClient.trySend in #6264,
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

retest this please


Problem_Statement_44

We have an array of integers, nums, and an array of requests where requests[i] = [starti, endi]. The ith request asks for the sum of nums[starti] + nums[starti + 1] + ... + nums[endi - 1] + nums[endi]. Both starti and endi are 0-indexed.
Return the maximum total sum of all requests among all permutations of nums.
Since the answer may be too large, return it modulo 109 + 7.
Example 1:
Input: nums = [1,2,3,4,5], requests = [[1,3],[0,1]]
Output: 19
Explanation: One permutation of nums is [2,1,3,4,5] with the following result:
requests[0] -> nums[1] + nums[2] + nums[3] = 1 + 3 + 4 = 8
requests[1] -> nums[0] + nums[1] = 2 + 1 = 3
Total sum: 8 + 3 = 11.
A permutation with a higher total sum is [3,5,4,2,1] with the following result:
requests[0] -> nums[1] + nums[2] + nums[3] = 5 + 4 + 2 = 11
requests[1] -> nums[0] + nums[1] = 3 + 5  = 8
Total sum: 11 + 8 = 19, which is the best that you can do.
Example 2:
Input: nums = [1,2,3,4,5,6], requests = [[0,1]]
Output: 11
Explanation: A permutation with the max total sum is [6,5,4,3,2,1] with request sums [11].
Example 3:
Input: nums = [1,2,3,4,5,10], requests = [[0,2],[1,3],[1,1]]
Output: 47
Explanation: A permutation with the max total sum is [4,10,5,3,2,1] with request sums [19,18,10].

created PR for this issue #80

Hello, I'm Aishwarya Menon from CCOEW (SY EnTC).
I'd like to work on this issue.

Hello, I'm Aishwarya Menon from CCOEW (SY EnTC). I'd like to work on this issue.

We allow to make only 2 PR's in our repo and you have are already done with it. Explore other repositories now

I am Mayuree Sawal from B.Tech IT CCOEW. I would like to work on this issue

You can work on this issue. Submit your solution within 2 days @MayureeS


Feature/eventmanager convenience api

This fulfills story #21 on the board:

Allow passing a priority "hint" in attachAggregate()
Allow attach() to proxy to attachAggregate() when the first argument is a ListenerAggregate
Allow detach() to proxy to detachAggregate() when the argument is a ListenerAggregate

To clarify: the second two points are to have a symmetry between $events->attach($aggregate) and $aggregate->attach($events).

Merged following review from PadraicB.


Use the shadcn scrollarea component to make the UI cleaner

Shadcn provides Scroll-area component to make the scrolling sections more clear and go hand in hand with the theming.
Here is how it looks with the shadcn Scroll-area -

Current implementation -

I'd like to know if you like using the shadcn Scroll-area. If so, I can help you with a PR.
Thank you 😊

thanks! This repo was just a throw-away demo, didn't expect it to pick up traction! but if you have a fix I'd be open to learning from how to do things better. This was my first project with tailwind

Thanks for the suggestion, I've implemented this, and it looks much nicer!

https://github.com/Georgegriff/react-dnd-kit-tailwind-shadcn-ui/commit/ec6ded122bfb1245dbde3070bf3e5116f74dc20e


remove drag/drop handlers in forceFallback mode

drag/drop handlers should be disabled in forceFallback mode to avoid unwanted behaviors when dragging elements

You're right, thank you.


feat: decoupled buttons from showButtons

Remove showButtons boolean value and changed it to an array so the user can add its buttons on the array instead of disabling all buttons.

It fixes #40


Implement Path helper module

Summary by CodeRabbit

New Features

Introduced functions for advanced path manipulation, including combining paths, checking rooted paths, and extracting file extensions and filenames, enhancing file system interactions.

The test code should be run directly on the host machine. We should create another project specifically used for testing


[Python] Array.str shows misleading output for timestamp types with time zone set

The output is being shown with UTC interpretation

In [13]: arr = pa.array([0, 1, 2], type=pa.timestamp('ns', 'America/Los_Angeles'))

In [14]: arr.view('int64')
Out[14]:
<pyarrow.lib.Int64Array object at 0x7fd1b8acdd70>
[
  0,
  1,
  2
]

In [15]: arr.type
Out[15]: TimestampType(timestamp[ns, tz=America/Los_Angeles])

In [16]: arr
Out[16]:
<pyarrow.lib.TimestampArray object at 0x7fd1b8ad5ec0>
[
  1970-01-01 00:00:00.000000000,
  1970-01-01 00:00:00.000000001,
  1970-01-01 00:00:00.000000002
]

Reporter: Wes McKinney / @wesm
Note: This issue was originally created as ARROW-9525. Please see the migration documentation for further details.

Hi, I wanted to bump this issue. Do you think __str__ could implement the timezone-aware timestamp?
In [1]: now = pd.Timestamp.now(tz=zoneinfo.ZoneInfo('America/New_York'))

In [2]: now
Out[2]: Timestamp('2023-03-06 14:16:41.385865-0500', tz='America/New_York')

In [3]: arr = pa.Array.from_pandas(pd.Series([now]))

In [4]: arr
Out[4]:
<pyarrow.lib.TimestampArray object at 0x7faee1489600>
[
  2023-03-06 19:16:41.385865000
]

One potential workaround is to cast to string.  Though I agree, changing the __str__ would be nice too:
>>> now = pd.Timestamp.now(tz=zoneinfo.ZoneInfo('America/New_York'))
>>> now
Timestamp('2023-03-06 17:48:12.283474-0500', tz='America/New_York')
>>> arr = pa.Array.from_pandas(pd.Series([now]))
>>> import pyarrow.compute as pc
>>> pc.cast(arr, pa.string())
<pyarrow.lib.StringArray object at 0x7f7ae2160580>
[
  "2023-03-06 17:48:12.283474000-0500"
]


feat(events): update resource last modification date event

resolves DSP-1592

@subotic Thank you for reviewing this.


Dashboard: Add copy story URL and open story menu action items

Summary
Adds two new context menu items to a story menu

Copy the story URL to the clipboard "Copy Story URL"
Open the story URL in a new tab "Open in new tab"

Relevant Technical Choices

Added two new menu items in the same style as other context menu items.
Tweaks to story menu item constant as well as popover menu to allow list items to be rendered as anchors.
Moves ToastProvider up to encompass <AppContent /> not just the ToasterView now that we're not just using it for state errors generated from context. This allows us to easily wire up any kind of toast.

To-do
N/A
User-facing changes

2 new action items in the story context menus of the dashboard to copy story url or open story in new tab.
Alert 'X' dismiss button is a bit larger to identify it from message body text.

Testing Instructions
Go to dashboard and see the two new options.

validate that the copy link copies the link (you will also get a success toast)
validate that open in new tab opens a new tab with the story

Addresses #4101

However because the second menu item is an a-href the last-child margin for the li items is applying to the first menu item. We might need to change that logic so the margin applies to the whole group.

Thanks! It looks like common js is applying a 6px margin-bottom to list items by default. I have removed this and set the separators to have margin top/bottom so nothing is getting inherited for this spacing. The separators will come up again soonish for a11y since we need to remove their ability to be focused on in the menu for keyboard usage.


Build fails on M1 macos at libssl

[ 98%] Linking CXX shared library ../../../libtdclientshared.dylib
ld: warning: ignoring file /usr/local/opt/openssl/lib/libssl.dylib, building for macOS-arm64 but attempting to link with file built for macOS-x86_64
ld: warning: ignoring file /usr/local/opt/openssl/lib/libcrypto.dylib, building for macOS-arm64 but attempting to link with file built for macOS-x86_64

Hi @Bellavene - I don't see this problem on my M1 macOS.
Could you share the entire build output on terminal, perhaps attach as a txt file, or at least the output from cmake?
On my M1 homebrew installs (arm64) packages under /opt/homebrew/opt, so when cmake runs it detects openssl there:
-- Found OpenSSL: /opt/homebrew/opt/openssl/lib/libcrypto.dylib (found version "3.0.2")
-- Found OpenSSL: /opt/homebrew/opt/openssl/include /opt/homebrew/opt/openssl/lib/libssl.dylib;/opt/homebrew/opt/openssl/lib/libcrypto.dylib

But it looks like it detects it wrongly on your system.
Btw, are you using brew, and have installed openssl using it?

Hi @Bellavene - I don't see this problem on my M1 macOS.
Could you share the entire build output on terminal, perhaps attach as a txt file, or at least the output from cmake?
On my M1 homebrew installs (arm64) packages under /opt/homebrew/opt, so when cmake runs it detects openssl there:
-- Found OpenSSL: /opt/homebrew/opt/openssl/lib/libcrypto.dylib (found version "3.0.2")
-- Found OpenSSL: /opt/homebrew/opt/openssl/include /opt/homebrew/opt/openssl/lib/libssl.dylib;/opt/homebrew/opt/openssl/lib/libcrypto.dylib

But it looks like it detects it wrongly on your system.
Btw, are you using brew, and have installed openssl using it?

Hello @d99kris
I am a bit busy last days, will come back with that. Honestly I don't remember, but most likely yes, installed through brew. Will check that. Maybe there is some leftover from intel brew, because I have migrated the system last time. Thank You.

https://send.vis.ee/download/5946b153b4069ce6/#tkIOhDiCKj-SKtPDprE-TA

Looks like I was too slow, the above link shows:
"This link has expired"
Anyway, I believe Github issues allow attaching files directly. It should be possible to just drag and drop onto this web page form.

nchat make.txt
Thank you, did not knew that.

Thanks! Yeah, the Github issues UI could perhaps be improved with a paperclip button to make it more obvious it's supported.
So from the log you shared I see this:
-- Found OpenSSL: /usr/local/opt/openssl/lib/libcrypto.dylib (found version "1.1.1k")
-- Found OpenSSL: /usr/local/opt/openssl/include /usr/local/opt/openssl/lib/libssl.dylib;/usr/local/opt/openssl/lib/libcrypto.dylib

which indicates openssl was found in /usr/local/opt/openssl, which is the default location for x86_64 installation.
The default location for arm64 is /opt/homebrew/opt/openssl. Could you check if you have that dir, and perhaps also that the installed openssl there is arm64? One can use the following command:
file /opt/homebrew/opt/openssl/bin/openssl
/opt/homebrew/opt/openssl/bin/openssl: Mach-O 64-bit executable arm64

Assuming your system does have openssl arm64 installed, perhaps you could try to edit the nchat source file lib/tgchat/CMakeLists.txt and remove the following line:
list(APPEND OPENSSL_ROOT_DIR /usr/local/opt/openssl)
If this works, I will try prepare a proper fix in nchat. Thanks!

Thank you! That worked flawlessly

Thanks for the feedback! In the above commit I've implemented a proper fix to support M1 systems which have both arm64 and x86_64 installations of openssl, by prioritizing the native (arm64) openssl search path.


Loading Indicator

Is there anyway to have a loading indicator between when an image is selected and the image is displayed in the img-cropper ?
Thanks!!

not with this tool. this repo is deprecated, cause ctefanche is not able to maintain it anymire, so i took over and now it is available as ngx-img-cropper
your requested feature is not there yet, but maybe one day... ;)


Adds custom allocator

This change allows specifying acustom memory allocator (e.g. RM_Alloc) via the use of defines.

@itamarhaber care to rebase this and make sure it has no conflicts? It conflicts with a bugfix I just merged.

@dvirsky rebased - please review the PR and (fingers crossed) merge it.


There should be a "Verify ssl certificate" option for Graphite checks

HTTP checks have a "Verify ssl certificate" option that allows you to
"Set to false to allow not try to verify ssl certificates (default True)"
However, Graphite checks do not have this option.
It would be nice to be able to use Graphite servers that have not yet been configured with a proper certificate (for example, in a development environment).

I think this should be an environment option rather than on the graphite check - would be happy to see a PR for this.


fixes solution dir value definition

Hey @MIchaelMainer sorry for the subsequent PRs here. I had not noticed the conditional variable at the beginning, hopefully this one is the right one.

Actually don't merge that just yet, still having issues.

@MIchaelMainer this is good to merge as https://github.com/microsoftgraph/MSGraph-SDK-Code-Generator/pull/272 finally builds!


Publish 1.1 release on GitHub & tag

Add 1.1 release to 'releases' section on GitHub
Add binary artifacts
tag git repo with version label

@mandy-chessell Since we are ready to release 1.1, we should tag the version in git
The current tags are: V.0.1, V0.2, V1.0
I propose a new tag 'V1.1'
We also have the option of creating a lightweight tag, or a full tag object. The latter can be signed with gpg key, contains name/email, and a tagging message.
We could create a lightweight tag for 1.1, and I can raise an issue for 1.2 to see if we can augment the release process pipelines to get the git code tagged at the point we push from staging to maven central?
I'm unsure if I have permission to tag but can try if you agree with the above.

I have created a V1.1 lightweight tag
Release has been created at https://github.com/odpi/egeria/releases/tag/untagged-ae705f972b47aef6f148 - DRAFT
Add sourced artifacts using

a clean clone
checkout of egeria-release-1.1 (same as V1.1 tag)
git archive --format=tar --prefix=egeria-1.1/ -o ../egeria-1.1.zip V1.1
and similar for tar, then gzip'd

This seems to be the format used last time
@mandy-chessell please can you review the files/notes.

Note the files are named slightly differently - need to figure out what was done before

The maven central artifacts are now published (note https://github.com/odpi/egeria/issues/1969 ). closing


Avoid excessive retries in tracing test

Fixes https://github.com/istio/istio/issues/16985
Both cases should sample at 100%. This PR limits the number of request to ensure that the sample rate is applied.

@bianpengyuan: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_istio
06f472e8b3b00416d799e7def19153bede5762e0
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@bianpengyuan: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_istio
06f472e8b3b00416d799e7def19153bede5762e0
link
/test lint_istio

integ-telemetry-k8s-tests_istio
06f472e8b3b00416d799e7def19153bede5762e0
link
/test integ-telemetry-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@bianpengyuan: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-security-k8s-tests_istio
40cc226459a429a897158d802435382dbe46db99
link
/test integ-security-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-security-k8s-tests_istio

@bianpengyuan: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

pilot-e2e-envoyv2-v1alpha3_istio
40cc226459a429a897158d802435382dbe46db99
link
/test pilot-e2e-envoyv2-v1alpha3_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test pilot-e2e-envoyv2-v1alpha3_istio


Find me

Description 📜
A fully responsive web app to find a word in a sequence of jumbled words.
Fixes #1283

Type of change 📝
Please delete options that are not relevant. In order to tick the check box just put x inside them for example [x] like this

[x] Add a new project

Domain of Contribution 📊
Please delete options that are not relevant. In order to tick the check box just put x inside them for example [x] like this

[x] Project using HTML, CSS & JS

Checklist ✅
Please delete options that are not relevant. In order to tick the check box just put x inside them for example [x] like this

[x] I have performed a self-review of my own code or work.
[x] I have commented on my code, particularly in hard-to-understand areas.
[x] My changes generate no new warnings.
[x] I'm GSSoC'22 contributor

Screenshots / Gif (Recommended) 📸
https://user-images.githubusercontent.com/82095877/167262424-afce4cf9-92e2-4623-8aa0-5466c17d6ba5.mp4

@gurjeetsinghvirdee pls bhai ji


Group Permissions

thank you for this library but how i can make the group request permission like this group
Manifest.permission_group.STORAGE

please check out http://stackoverflow.com/a/30720740/1627904


Update gmaven_rules to drop the dependency on Maven

@slinzner I've updated gmaven_rules to use Bazel's own HTTP fetching mechanism to download AARs, instead of using Maven. This should improve reliability and reduce flakiness in our CI.

LGTM


Use tonic-build compile_fds to build file descriptor set

Updates the example of usage with tonic-build.

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

Project coverage is 92.67%. Comparing base (2df8867) to head (008851a).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #87      +/-   ##
==========================================
+ Coverage   92.63%   92.67%   +0.04%
==========================================
  Files          19       19
  Lines        2430     2430
==========================================
+ Hits         2251     2252       +1
+ Misses        179      178       -1

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

Thanks!


Fix build error for aggregates of exactly 80 elements

FUZZTEST_INTERNAL_BIND_AGGREGATE_ skips x78 meaning that building fails when you have an aggregate of exactly 80 elements. For example:
#include "fuzztest/fuzztest.h"

struct MyStruct79 {
    std::array<char, 79> data;
};

struct MyStruct80 {
    std::array<char, 80> data;
};

struct MyStruct81 {
    std::array<char, 81> data;
};

// OK
void TestMyStruct79(MyStruct79 s) {}
FUZZ_TEST(ExampleTest, TestMyStruct79);

// error: type 'std::array<char, 80>' decomposes into 80 elements, but only 79 names were provided
// error: static assertion failed due to requirement '80 > 80'
void TestMyStruct80(MyStruct80 s) {}
FUZZ_TEST(ExampleTest, TestMyStruct80);

// OK
void TestMyStruct81(MyStruct81 s) {}
FUZZ_TEST(ExampleTest, TestMyStruct81);

Thank you so much! Merging.


Add animated class on animated.less

Hi Dan
First of all thanks a lot for this less version of animate.css. Exactly what i want! :)
But i have to tell you that you should update your animate.less file by adding the following:
  .animated{
     .animated;
  }

By adding this you can now hook the html elements by class, not only by css.
Sorry about my english!! :

This is a huge change and is for a different repo—this is not the animate.less project, which I don’t maintain.

I don´t understand! What is an huge change ? Add a class and run a mixin inside it?


Added Blood Sugar Calculator

Fixes Issue🛠
Closes #1583
Description👨💻
I've added a new calculator, Blood Sugar Calculator which inputs the age and medical test result. It then provides the range in which your test results lie. I've added

index.html
style.css
script.js
README.md

Type of Change📄

[ ✅] New feature (non-breaking change which adds functionality)

Checklist✅

[✅] I am an Open Source contributor
[ ✅] I have performed a self-review of my code
[✅ ] My code follows the style guidelines of this project
[✅ ] I have commented on my code, particularly in hard-to-understand areas

Screenshots/GIF📷

@vidhiisaxena  You should add your calc in main index.html file of project in alphabetical order.

@Nihal4777 Kindly review it once and if everything is fine, then please merge my PR.

@vidhiisaxena Why background image is repeating on x axis; You can have background-size: cover; property added in body tag.

I've already added the following tag.

Removed the background as it was not responsive.

@Nihal4777 @Rakesh9100 Kindly check and merge my PR.

@vidhiisaxena Don't worry It will be reviewed & merged very soon.

In the meantime if are facing difficulty in adjusting background then you can contact me on Discord @nihalrajpal


Sort comments newest/oldest on top

Hi,
I would like to make a feature request. May you please add the possibility to sort the comments within a task to have the newest or oldest ones on top? As in #236. If you have a long list of comments you wouldn't need to scroll down till you get to the latest answer. Could this also be made permanent as a setting.
Thanks in advance.
Cheers
Alwin

Sorry, I may have not clarified my intent enough, I thought about a user setting &/ some clickable sort feature on the "Comments" headline.
Cheers
Alwin

Duplicate with #236


Fix: handle nonexistent vimtex.root

It can happen that vimtex.root is not a proper directory (for example with plugin gitsigns.nvim, the diff window has vimtex.root as something like gitsigns:///dirpath/.git), leading to errors in vimtex#paths#pushd(b:vimtex.root).
It seems that not starting the parser in such buffers works alright, but I'm not sure if it is the right/optimal solution.

Thanks for the PRs. I would never have caught this issues.


OpenSSL 1.1.0 X25519 implementation

I'm trying to develop an application that uses X25519 (or ECDH over Curve25519), which is supposed to be supported since version 1.1.0 (issue #309). But, when looking for examples, I realised that in the code this curve is not tested under test/ecdhtest.c. Literally:
     * Skipped for X25519 because affine coordinate operations are not
     * supported for this curve.
     * Higher level ECDH tests are performed in evptests.txt instead.

In test/evptest.c it is only tested that the generated secret key is what was expected. There is no key generation.
Another test that I tried to do was to run openssl speed ecdhx25519, which crashes:
ECDH failure.
139633730975552:error:100AE081:elliptic curve routines:EC_GROUP_new_by_curve_name:unknown group:crypto/ec/ec_curve.c:3100:
139633730975552:error:100AE081:elliptic curve routines:EC_GROUP_new_by_curve_name:unknown group:crypto/ec/ec_curve.c:3100:
OpenSSL 1.1.0  25 Aug 2016
built on: reproducible build, date unspecified
options:bn(64,64) rc4(16x,int) des(int) aes(partial) idea(int) blowfish(ptr)
compiler: gcc -DDSO_DLFCN -DHAVE_DLFCN_H -DNDEBUG -DOPENSSL_THREADS -DOPENSSL_NO_STATIC_ENGINE -DOPENSSL_PIC -DOPENSSL_IA32_SSE2 -DOPENSSL_BN_ASM_MONT -DOPENSSL_BN_ASM_MONT5 -DOPENSSL_BN_ASM_GF2m -DSHA1_ASM -DSHA256_ASM -DSHA512_ASM -DMD5_ASM -DAES_ASM -DVPAES_ASM -DBSAES_ASM -DGHASH_ASM -DECP_NISTZ256_ASM -DPOLY1305_ASM -DOPENSSLDIR="\"/usr/local/ssl\"" -DENGINESDIR="\"/usr/local/lib/engines-1.1\""  -Wa,--noexecstack
                              op      op/s
 253 bit ecdh (X25519)   0.0000s      inf

It works perfectly with whichever another curve. I've tried to run the same with two versions (1.1.0 and 1.1.0c) and on two different machines, with identical result.
So, is X25519 100% supported in 1.1.0? If so, where is my mistake?

Due to the way X25519 is defined in various standards it behaves differently from other curves. It is not accessible through the EC API for example. It is treated as a distinct "top level" algorithm that supports key generation and key agreement. So to generate a key you create a new EVP_PKEY_CTX with NID_X25519 and call keygen on it instead of via EVP_PKEY_EC. Via the command line: "openssl genpkey -algorithm X25519" does the trick.

@snhenson Thanks for your reply. But, as far as I understand, the openssl speed command uses the API. Since there is already a sub-command for this curve, does it imply that an API-accessible version will be released in future versions?

I know this is closed but for info the speed bug you mention was fixed in the master branch by the following pull request:
https://github.com/openssl/openssl/pull/1658
This fix was not backported to the OpenSSL 1.1.0 branch, I think as it was considered a change that added a 'feature' i.e. it moved the ECDH speed tests to use the higher level EVP API rather than the low level EC API. Released branches will only take bug and security fixes not new features.
Hope that helps!


[docs] auto create commands documentation

The Issue
Commands must be maintained in two places, the source and afterwards the docs and docs therefor are not always up to date.
How This PR Solves The Issue
There is the cobra/docs package to auto generate the docs from the command definitions.
Manual Testing Instructions
Automated Testing Overview

Related Issue Link(s)
Release/Deployment Notes

@mattstein a first draft of auto generating the docs for the commands. I did not do much so far because I like to discuss this topic with you. At the moment one file per command is created in contrast to one doc for all commands at the moment.
Please have a look and give me some feedback so I know how to continue on this PR...
Thanks!

@mattstein is going to be very happy with you.

I’m a fan of this @gilbertsoft, nice work!
If the docs can be generated from code comments that seems like a win for everybody. In my experience, the efficiency comes with the tradeoff of making it more challenging to customize for the reader. But I’ve also never spent any real time with Cobra (or Golang 😅).
Questions:

Is there an option to keep everything to one page? I don’t much care what the URL structure looks like, but a benefit of the one long page right now is discovery; as a new user I can scroll through all the commands and see how they’re used. With a command per page, I worry that the experience could be more cumbersome and fragmented for someone browsing. (Barring any UI work to address that.)
What does support look like for documentation-specific thinks like anchors and ::: tip formatting? The ddev blackfire command uses both, for example:

I’m a fan of this @gilbertsoft, nice work!
If the docs can be generated from code comments that seems like a win for everybody. In my experience, the efficiency comes with the tradeoff of making it more challenging to customize for the reader. But I’ve also never spent any real time with Cobra (or Golang 😅).
Questions:
1. Is there an option to keep everything to one page? I don’t much care what the URL structure looks like, but a benefit of the one long page right now is discovery; as a new user I can scroll through all the commands and see how they’re used. With a command per page, I worry that the experience could be more cumbersome and fragmented for someone browsing. (Barring any UI work to address that.)

We should be able to change it to one page, yes. At the moment I've only copied the implementation from the GH cli to have a first draft to discuss. 15 minutes of work so far.

2. What does support look like for documentation-specific conventions like anchors and `::: tip` formatting? The [`ddev blackfire` command](https://ddev--5016.org.readthedocs.build/en/5016/users/usage/commands/#blackfire) uses both, for example:

Guess this should also be possible but have to check the details how we can implement it.

3. Would you take a similar approach to automating the [Config Options](https://ddev--5016.org.readthedocs.build/en/5016/users/configuration/config/) reference? One of the few benefits of doing this all by hand right now is that a new feature tends to make it into both the command and config reference when that’s relevant.

Yes, I've already discussed this some weeks ago with Randy. But this needs a bit more preparation and we first have to cleanup and optimize the config stuff before. It's on the list though.
Thanks for your input. Will go ahead and change it to a one page as next step.

This is a phenomenal initiative, but of course it's gone way stale and has inadequate information about what it does and where it's going. I sure hope we can pick it up again! And of course we always appreciate your massive contributions to DDEV. Closing for now.


Needs an update to use GHA

I believe the travis build is not happening anymore.
This repo also may require an update to use GHA.
The only tricky thing would be the secrets configuration : Github Actions doc: https://docs.github.com/en/free-pro-team@latest/actions/reference/encrypted-secrets
It is not a rds file like you have currently.
But it seems rtweet is no more using this mechanism: https://docs.ropensci.org/rtweet/articles/auth.html

Thanks! Finally I learned a little more about GHA. It still feels too complicated and "raw" to me. Not sure if I got it correct: https://github.com/yihui/twitter-blogdown/blob/master/.github/workflows/twitter-blogdown.yaml
I base64 encoded the rds file to plain text and added it to secrets. Then base64 decode it. rtweet still uses the rds file for authentication.


Take BufRead as arguments

First, see https://github.com/daac-tools/vibrato/pull/31#discussion_r956692630.
Here, I consider the two options:

Keep Read and removes the Notes section.
Fix csv crate to take the BufRead argument and use BufRead through all arguments.

It seems easy to modify the csv crate to use BufRead.
I think the second option is better, and I will do that.
@vbkaisetsu What do you think?

The first option would be better for convenience.

modify the csv crate

Is this possible?
Narrowing the trait boundaries is unacceptable to many users.

Is this possible?

This can be done if I fork csv and update the implementation.

Narrowing the trait boundaries is unacceptable to many users.

Does it mean Read is easier to use than BufRead (for users)?

Ofcourse Read is easier for users because the BufRead trait requires Read.

I don't see the benefit of forking the csv crate just to make the argument BufRead.

I think that taking both Read and BufRead is not a bad option.

I don't see the benefit of forking the csv crate just to make the argument BufRead.

I fear that handling Read and BufRead in the same function may be a bit confusing (https://github.com/daac-tools/vibrato/pull/31#discussion_r956844555m). Even if we replace C and P with BufRead in Dictionary::from_readers, there remains the fact that there is no need to wrap S and U of Read in a buffered reader. To apply BufRead to S and U, it is necessary to fix the csv crate.

I think that taking both Read and BufRead is not a bad option.

I think the first option that keeps Read is the best because the API is easy to understand.


apex-3.4.0 can not found

http://apache.cs.utah.edu//apex/apache-apex-core-3.4.0/apex-3.4.0-source-release.tar.gz

Mee to，  http://apache.communilink.net/apex/

I am also running into the same issue. Could you provide the source code link or the maven repo link ?

Looks like there are only apex 3.6.0 and 3.7.0 in the Utah artifacts. You may manually change the version to unblock yourself.


G200 missing from flake8-logging-format

Heya, I noticed that the majority of flake8 logging rules were ported thanks to the work done on here https://github.com/astral-sh/ruff/pull/2150, however I'm still missing G200 and am hoping it can be implemented ✌

Related to https://github.com/astral-sh/ruff/issues/7248


Consider adding missing wrappers for FunC's store_something() functions from the stdlib.fc

store_maybe_ref() as Builder.storeMaybeRef():

https://github.com/tact-lang/tact/blob/779b0fe736d423d5047222d2443d1ebe7a5dff1e/stdlib/stdlib.fc#L458-L461
As mentioned in Tact chat

store_builder() as Builder.storeBuilder():

https://github.com/tact-lang/tact/blob/779b0fe736d423d5047222d2443d1ebe7a5dff1e/stdlib/stdlib.fc#L638-L639

This is definitely useful for people who wish to go low-level. It would be great to also think of some higher-level constructions which in the end solve the Y problem.

Well, we have .toCell() and .fromCell() now, which are great win in terms of DX and, potentially, gas optimizations in the future.
And if there's a need for manual parsing/construction of cells with very specific bit sizes, my P.S. comment by the end of https://github.com/tact-lang/tact/issues/472 may be it.
We'll definitely think about it deeper when the Tact 2.0 starts fledging out :)


Try to detect DMX value changes and output a minimal changeset first, as...

... it will be quicker
I have a fairly cheap clone of the uDMX hardware, and it turns out that it's quite slow to output a full DMX universe of 512 channels. I can only get about 3 or 4 frames per second out of it. This is too slow for smooth fades.
I find that if I make this change then my fades become a lot smoother when there's only a few fixtures affected, as the output range is smaller, so it outputs quicker.
I doubt this code is particularly well implemented so far, but it serves my purpose, and hopefully someone will be able to comment on how it can be improved.

More than one year an no activity.
Closing this now.


flutter app building issue

Error:

Flutter doctor -v

Hi, as the error mentions, in the code super.addAllowedPointer(event) event is an instance of PointerEvent class. But, addAllowedPointer() accepts an instance of PointerDownEvent class instead. Pls make the necessary changes in your code to solve this.
If this doesn't help, pls paste here the complete code where the error is happening. Please refrain from pasting screenshots, and paste your code and error messages in text. Use this for formatting code and this for formatting large logs and errors.


Aurora leads to BSOD with Asus AURA sync

I'm getting some full blue screens when I launch Aurora, even went as far as to get a memory glitch where it consumed up to 4gb of RAM for no apparent reason, followed by the same bsod (stop code: SYSTEM_THREAD_EXCEPTION_NOT HANDLED  // What Failed: stornvme.sys). Unfortunately I thought reinstalling would be a good solution, so the logs got lost, but the same odd behaviour is permanent:
I install aurora, it detects my hardware (Aura Sync MoBo, Razer BlackWidow Elite, Logitech G502 Proteus Spectrum), but I get no GUI and no traybar icon, so I try to /taskkill /f /pid [PID] on admin, nothing happens so I go over to task manager and try to kill it there, but it gives an access denied error, I then try to relaunch aurora, and that's when the blue screen strikes... Here are the three latest logs, all of which triggered the BSOD
2020-03-29 19.33.07.log
2020-03-29 19.30.41.log
2020-03-29 19.34.59.log

Is this only when installing Aurora or permanently?
Sounds critical.

That behaviour happens when Aurora is present on the system, not just when installing... I've had it uninstalled since then, for obvious reasons, but that always happened after reboots, untill it would be uninstalled

I got the EXACT same BSOD as yours... but with AuraConnect, a different software with a similar goal: use AuraSDK to control Asus RGB hardware through 3rd party software. The goal with AuraConnect is to control Aura Sync with Razer Synapse 3.
Long story short, multiple RGB software sometimes cause a mess on the system. Sometimes I got BSOD with just Aura and iCue installed.
And, to be honest, I tryed Aurora after don't know how many SSD wipes untill I finally left the Synapse Auraconnect combo. I did this this morning, actually...
My guess is that sometimes one software crashes with another, sometimes don't. You can try uninstalling EVERYTHING RGB related and installing again. Or, before that, try switching Aura with Armoury Crate, or vice versa. Worked for me...
One last thing... are you using a Kingston A2000 m.2 NVME SSD? If so, it's the same as mine. And I've read a few posts about people complaining that this particular is have some issues that causes BSOD with some RGB software...
Hope you get your problem solved!

Funnily enough I do have that very same ssd, but it's a secondary drive that windows doesn't even mount... You know, multi-boot things haha. My main drive, the one that boots windows, is a Corsair Force MP510... Currently I'm just using the native app for every peripheral I have...


sass module syntax support is incomplete

So it seems the sass module system (@use/@format) should be supported, but the support seem incomplete
The following examples do not work
@forward 'library' with (
  $black: #222 !default,
  $border-radius: 0.1rem !default
);

or
@use 'library' with (
    $black: #222 !default,
    $border-radius: 0.1rem
);

it seems support is missing for:

@forward with configuration
@use with !default in the configuration (@use with configuration without !default works fine)

Version: 1.43.0 (system setup)
Commit: 78a4c91400152c0f27ba4d363eb56d2835f9903a
Date: 2020-03-09T19:47:57.235Z
Electron: 7.1.11
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Windows_NT x64 10.0.18362
(i can also provide extensions but I don't think it really matters in this case)
Steps to reproduce

Open a .scss file
copy paste this
@forward 'library' with (
$black: #222 !default,
$border-radius: 0.1rem !default
);

Expected result : no problems
Actual result:
{
"resource": ".../sass/all.scss",
"owner": "generated_diagnostic_collection_name#0",
"code": "css-unknownkeyword",
"severity": 8,
"message": "unknown keyword",
"source": "scss",
"startLineNumber": 3,
"startColumn": 20,
"endLineNumber": 3,
"endColumn": 24
}

Also !default is missing from the suggestions


docs: Move to kubevirt/developer-guide

What this PR does / why we need it:
This is moving the /docs directory to an external repository (current proposal: kubevirt/developer-guide, as shown here: https://github.com/fabiand/developer-guide).
Primary reasons:

Don't trigger CI for docs/ changes
Have a repository with a clear purpose: Educate developers

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
The extracted docs can be found in https://github.com/fabiand/developer-guide.
I did consider to just propose to skip CI for changes to docs/.
In the end I did not like it because a) it adds an exception for CI b) if docs/ would require CI , then it might be hard to catch or solve.
Release note:

NONE

@lyarwood @acardace @xpivarc @aburdenthehand @vladikr @dhiller my proposal: Let's pull out the docs/ dir into a separate repository to
a) don't trigger ci on doc changes
b) shape up this new repository to be developer focused. AKA a repository with more focus and clear prupose.

For completeness:
community repo also contains developer documentation/contributing guides.
we could
community/design-proposals --> repo:enhancements (as proposed by @lyarwood )
community/developer guide stuff --> repo:developer-guide (No pr)
kubevirt/docs --> repo:developer-guide (this PR)
This would move developer dcumentation to one central place.
Similar to how we moved release data from community to sig-release

Good idea!
docs/metrics.md gets auto-generated, I guess we need to keep that one.
Also let's create and populate the new repo before merging this.
/hold

Interesting idea.
I would expect to keep these files in this repo:
developer-certificate-of-origin.md
fossa.md
reviewer-guide.md
and maybe these ones as well (but I can see the argument for them moving):
getting-started.md
release-branch-backporting.md
release-procedure.md
release.md
Also, how much of this content belongs in the user guide rather than moved to a new repo?

Interesting idea. I would expect to keep these files in this repo: developer-certificate-of-origin.md fossa.md reviewer-guide.md
and maybe these ones as well (but I can see the argument for them moving): getting-started.md release-branch-backporting.md release-procedure.md release.md
Also, how much of this content belongs in the user guide rather than moved to a new repo?

Some of the docs you mentioned might be part of https://github.com/kubevirt/community/issues/235

/close


Commission should be optional on the order model

Making the commission optional on the order base model since it's not returned in this endpoint.

Also I fixed that poetry CI bug - try pulling from master so that we can run the CI


A error report when building apex and maskrcnn-benchmark

❓ Questions and Help
After the command python setup.py install --cuda_ext --cpp_ext
Raises error: /usr/local/cuda/lib64/libcudnn.so.7/include: Not a directory
however, I do not understand what kind of file libcudnn.so.7 is, which seems like not a folder
I found out that the CUDNN_PATH in .bashrc is  /usr/local/cuda/lib64/libcudnn.so.7
Then, I did the operation of modifying it, just change libcudnn.so.7 to some random characters which has no meaning, and the building procedure successfully finished.
I was wondering what is the problem? Could anybody help me?

I think, you should set CUDNN_PATH as /usr/local/cuda/ instead of /usr/local/cuda/lib64/libcudnn.so.7

@anl13 yeah thanks!


How to find out if pivot table is filtered?

I'm using pivottable.js in the open source project project Koia, thanks for sharing this amazing library!
In my project, the CellClickHandler  class is responsible for showing a raw data dialog when the user clicks on a table cell. In order to query accurate data for total cells also, I need to know if the pivot table data has been filtered by the user.
Is there a way to automatically find out if the pivot table is currently filtered?

I guess you could check the inclusions / exclusions of the config object, as demonstrated in this demo: https://pivottable.js.org/examples/onrefresh.html

Thanks Mike , this is what I was looking for!


add refactor at executor instance frame

What this PR does:
Which issue(s) this PR fixes:

Fixes #396
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Codecov Report

Merging #397 (d87d454) into master (334879e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #397   +/-   ##
=======================================
  Coverage   43.25%   43.25%
=======================================
  Files         123      123
  Lines        7146     7146
=======================================
  Hits         3091     3091
  Misses       3830     3830
  Partials      225      225

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


How to disable cache in development?

const dev = process.env.NODE_ENV !== 'production'
...
cacheableResponse({
   enable: dev
})

Check the unavatar example:
https://github.com/Kikobeats/unavatar/blob/master/src/index.js#L13
It should be enough don't load it into development 🙂


Calculate distance between ioBroker and tracker

@Scrounger
As changes to this repository are currently NOT possible, adapter tractive-gps hase been moved into iobroker-community adapters area
https://github.com/iobroker-community-adapters/ioBroker.tractive-gps
Please create a PR containing your changes at this repo so that we can include it into a new release of this adapter.
Thanks in advance.
mcm1957


Google Inbox Support

Hi!
I was unable to use this tool with Google Inbox. The mail gets added to the draft folder but it will not open it in the compose window of Inbox. It will only open a browser window to the inbox of Inbox.
It would be great if you could add this feature.

If anyone who uses inbox wants to provide a patch I'd be happy to accept it.

Inbox user here! Definitely needed.

Is there any work outs? Seems only redirect the link to inbox.google.com/draft.. I tried to look into the inbox behavior but seems no work outs for direct link of a specific draft

Google inbox is no more.


[BUG] Incorrect Logic in Option#isDefault

Summary of Issue:
According to the the javadoc comment for Option#isDefault (jd reference), the method should return true only if all the properties of that option are null (or default values).
However, in all the implementations I checked, the method returns true when all the properties are not default.
I'll use Fill as an example option.
   public boolean isDefault() {
        return isEnabled() != null &&
                getType() != null &&
                getColor() != null;
    }

This return true if enabled is was initialized to a boolean, a type was specified, and a color was specified (meaning all the properties were set).
Indirect Bugs Caused by This
Trying to use the Options builder results in an incorrect options instance because the builder checks each time if an option is default and if so will set the field to null. Hence, when all properties of an option are set (thus Option#isDefault will return true), the option field will be set to null in the builder and not be applied when built.
Expected Behavior
I expect the method to return true if all the properties of that option are null. I believe the logic should be changed to something like:
   public boolean isDefault() {
        return isEnabled() == null &&
                getType() == null &&
                getColor() == null;
    }

So sorry for the delay in fixing this. :3 It's been a rough last couple months for me IRL.

No worries! Thank you for fixing this and hope things get better soon!


Added module title column to store, controller, and updated admin UI to update those

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

#1878
#1891  👈
#1884
main

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

Join @tommylau523 and the rest of your teammates on  Graphite

Merge activity

Apr 29, 2:53 PM EDT: Graphite rebased this pull request after merging its parent, because this pull request is set to merge when ready.


Invite user

Feature:
Users should be able to invite other users to their lakeFS installation.
Description:
We should add the functionality for users to invite other users to their lakeFS installation using an email address, invited users will be in a "pending" state until they approve their accounts using an email verification mail we'll send upon invitation. This invitation will include a link to the respective lakeFS organization/installation registration form.
Purpose:
Adding user invitation functionality to lakeFS.
User problem:
Users depend on their lakeFS admin to create a user for them and provide them credentials.
User value:
Users will now be able to asynchronously register to the system regardless of the lakeFS admin availability (besides the invitation itself), also, users can pick their own password (and credentials) without someone else knowing what they are (more secured).
Assumptions:
Invitations should have an expiration (let's say 12 hours).
Registration is protected by the invitation's token.
Not doing:
Adding "resend" invitations functionality, for now, deleting/inviting again a user should do the same trick.
Acceptance criteria:
Users can register to lakeFS upon invitation via email.
Depends On:
#3090 #3089 #3103

#3199 closes this issue


Avoid type promotion in MultiVector.__invert__

Also adds better tests for checking for type promotion.

In what I can only describe as shocking, the azure builds passed!


Print strings for tick labels

Maybe I missed it somewhere, but the examples certainly don't have it.
In the original matplotlib you can use custom strings as tick labels on the axes, such as this x axis:

In matplotlib there is a set_xticklables for this but in the plotpy I cannot find anything along these lines.

Hi,
I've implemented this feature.
See the second example from here: https://docs.rs/plotpy/latest/plotpy/struct.Plot.html
For instance, the following snippet changes the tick labels:
// set tick labels
let mut plot = Plot::new();
let ticks: Vec<_> = (0..vegetables.len()).into_iter().collect();
plot.add(&img)
    .set_rotation_ticks_x(45.0)
    .set_ticks_x_labels(&ticks, &farmers)
    .set_ticks_y_labels(&ticks, &vegetables);

Cheers


0/1 Knapsack using Tabulation method

@smv1999  Okay I will take care of it in future . Actually I followed the contributing guidelines there It is not mentioned to create to a issue for alternate approach . Issue is mentioned only for new problem.

@smv1999 Okay I will take care of it in future . Actually I followed the contributing guidelines there It is not mentioned to create to a issue for alternate approach . Issue is mentioned only for new problem.

Thanks for pointing out, I will add that information.

@RAUNAK-PANDEY , I have received information from the DevIncept team that PRs can be merged even before the actual coding period starts i.e., 10th July.
So, proceed and make the suggested changes.
Give your alt solution as a separate file.

@smv1999 Please check alt solution for knapsack. I have added a separate file.


Connect to the Internet to check for updates

Hi, we are currently visiting Canada and I keep getting the below message on my comma ai. I connected to my mobile hotspot and then it lets me drive however, once I restart the car and it is no longer connected to my mobile hotspot it presents the same message again.
dongle id: B4c3c4a8ff4b49f1
Immediately connect to the Internet to check for updates. If you do not connect to the Internet open pilot want to engage in 0 days.

The updater only runs while the car is off, so you'll need to keep it off and connected to internet until you see the update prompt. We allow the device to go onroad the first time after boot, which is why you it works sometimes for you. The latest release has some updater controls which should help your situation next time.

I have connected it while the car is off as well as ACC mode but still nothing happens.

It's downloading a large update. You can uninstall and reinstall openpilot if you want to watch the progress.

Do I need to do this in the car or can I bring the phone unit into the house and do the update while it is disconnected from the car?


Paging links do not contain all selected options for MultipleDropdownFilter

What version of Unfold are you using?
0.40.0
What version of Django are you using?
4.1
What browser are you using?
Brave 1.70
Did you check changelog/commit history, if the bug is not already fixed?
Yes
Did you searched other issues, if the bug is not already fixed?
Yes
Did you check documentation?
Yes
Are you able to replicate the bug in the demo site?
I do not see this filter being used in the demo source code
Describe your issue
When using MultipleDropdownFilter, the paging links do not contain all selected values - they only include one. Example code below:
class ResponseCodeFilter(MultipleDropdownFilter):
    title = "Response Code"
    parameter_name = "response_code"

    def lookups(self, request, model_admin):
        unique_response_codes = (
            AuditLog.objects.order_by("response_code")
            .values_list("response_code", flat=True)
            .distinct()
        )
        return [
            (response_code, response_code) for response_code in unique_response_codes
        ]

    def queryset(self, request, queryset):
        # Get the selected values from the request's GET parameters
        if self.value() not in EMPTY_VALUES:
            return queryset.filter(response_code__in=self.value())
        return queryset

With the above in place, the filtering works fine i.e. I can see records with the various response codes that I had selected.
The first page result URL looks like this:
/admin/webhooks/auditlog/?response_code=200&response_code=401
But the URL for page 2 looks like this - it's only including the last selected option:
/admin/webhooks/auditlog/?p=2&response_code=401

This appears to be due to my django version (4.1).  I implemented this locally on the formula repo and it works fine.


Automated cherry pick of #1135: fix disk delete task base

Cherry pick of #1135 on release/2.9.0.
#1135: fix disk delete task base

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.9.0@1a7f933). Click here to learn what that means.
The diff coverage is n/a.

@@               Coverage Diff               @@
##             release/2.9.0   #1137   +/-   ##
===============================================
  Coverage                 ?   7.75%
===============================================
  Files                    ?     334
  Lines                    ?   53822
  Branches                 ?       0
===============================================
  Hits                     ?    4174
  Misses                   ?   49267
  Partials                 ?     381

Flag
Coverage Δ

#aFlag
7.75% <ø> (?)

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 1a7f933...fa0462b. Read the comment docs.

/lgtm
/approve


Improve docker build and caching

@hkjn,
This PR achieve the following:

Cleanup the docker image during make clean
Make efficient use of docker cache system by building the tools inside the Dockerfile and the use of multi stage build
docker-build-go is only building (no build if docker cache hit) and copying the build output to the host's mapped folder

This make sure that modifications only to tooling for example does not rebuild the middleware.
If no file changes then docker-build-go is near instantaneous.

The docker rmi does not clean the cache, seems I need docker rmi $(docker images -a --filter=dangling=true -q) But this will delete the cache of all images on the system :(

The docker rmi does not clean the cache, seems I need docker rmi $(docker images -a --filter=dangling=true -q) But this will delete the cache of all images on the system :(

Yeah, I think between those two options it's better to just remove the final image and not the cached layers used to produce it for make clean.. if there are issues that requires cached layers to be removed as well, users could in that case manually take steps wipe out all their untagged images / all the Docker data.
(Optional) Possibly this limitation could be mentioned as a TODO in the make clean rule?

Added a comment

@hkjn btw, the cool stuff of this Dockerfile also, is that it is multi arch. You can build it on arm just fine as well!

@hkjn btw, the cool stuff of this Dockerfile also, is that it is multi arch. You can build it on arm just fine as well!

Good point, and I can confirm; I just tested golang:1.12.5-stretch on aarch64 as well as armv7l, and it worked on both.


[SPARK-11561][SQL] Rename text data source's column name to value.

LGTM

Build triggered. sha1 is merged.

Build started sha1 is merged.

Build finished. No test results found.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45246/

Test build #1998 has started for PR 9527 at commit 0e0959e.


Pgweb hangs on rows with bytea columns

I have a table with one bytea column where I'm storing small images. Unfortunately, if I'm having like more than 10 rows there, pgweb totally hangs when I want to see the rows and it's unresponsive.

Yeah, this is the sad default behavior. I guess we could just show some kind of placeholder with the column size instead of loading binary data if the size exceeds the threshold.

@sosedoff Sounds reasonable. I'm out of time right now but in a month I should have more free time so I could take a try on this. Of course, if someone would like to take this faster then just do it :smiley:

Im closing this stale issue.


I can't open my build of the game outside of export/windows/bin

Describe your problem here.
For some reason my build of psych engine just doesn't open if I try and open it outside of the path export/windows/bin
I think I might be exporting it wrong, I've tried copy and pasting the bin folder to a new folder and it just doesn't open, then I tried compacting it in WinRar and then de-compacting in the new folder and it also didn't work
Might be a dumb question but I'm a beginner and have no ideia what's causing this
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
Yes, but only small changes: file name, discord rich presence, added a difficulty and used the letterY kickstarter intro for the title screen

It would be useful if you provide logs of the game. You can do that by launching the game in a terminal.
(I don’t remember exactly whether the game use the current directory, or the directory the executable is located in. Try going into the bin folder before executing the game, if it isn’t what you already did)


2 or 3 peerings

Text still talks of 3 peerings, but that's obsolete, there are only two now (as shown in picture)

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

ID: fe411166-7001-5d28-c286-def540fa690d
Version Independent ID: 224e81f0-7093-2f50-cf57-92470f77055a
Content: Azure ExpressRoute circuits and routing domains
Content Source: articles/expressroute/expressroute-circuit-peerings.md
Service: expressroute
GitHub Login: @cherylmc
Microsoft Alias: cherylmc

@jtuliani Thanks for the feedback! We are currently investigating and will update you shortly.

@jtuliani Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

We're updating the article this week. Thanks, @jtuliani! We've taken a doc bug. #please-close

@jtuliani We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Destination states based on logic?

Hey does Transporter support switching destination states based on logic in a function?

Hey!
It depends on how you define logic or a function =) Transporter has ability to switch to destination state based on Event, which itself can have some logic, and has shouldFireEvent block, that can be used to determine whether Event should be fired.
Basically, i try to stay true to original concept of state machine and keep it very simple. Does that answer your question?

A great answer. Thanks! Made me see things slightly differently, and more cleanly :)


Docs removed

I'm actively developing an integration and noticed the docs disappeared.
This seems to be intentional?
https://github.com/Commvault/cvpysdk/commit/ff8f4ee2aa76be65b35be118a7c872e6414f4a01
As a result links like this do not work anymore
https://commvault.github.io/cvpysdk/cvpysdk/commcell.html
I can pull down the old sha and self host but I think you want to fix this.

Hi @mattmorganpdx
We had a few issues with the documentation and we were reconfiguring it. Please check the updated documentation now


[5.1] PostgresConnector set application_name for a connection

In PostgresSql you can set the application name of a connection, the name is shown in monitoring through pg_stat_activity and logs. Useful for monitoring and debugging.
Added optional application_name configuration to PostgresConnector which when provided will be used to the the application name for the connection.

+1


Command failed: gradlew.bat app:installDebug -PreactNativeDevServerPort=8081

Environment:
OS-Wondows 10
Processor i7-7700 CPU @3.60GHz 3.60GHz
RAM- 16GB

I am stucked with this issue and not able to proceed for further development.

Unless you provide more information, there is no chance we can help you. Ideally provide a reproducer project. At least explain what you are trying to do and provide the stacktrace of the error.

FAILURE: Build failed with an exception.

What went wrong:
A problem occurred configuring root project 'OneApp'.

Could not resolve all artifacts for configuration ':classpath'.
Could not resolve com.android.tools.build:gradle:3.4.2.
Required by:
project :
Could not resolve com.android.tools.build:gradle:3.4.2.
Could not get resource 'https://dl.google.com/dl/android/maven2/com/android/tools/build/gradle/3.4.2/gradle-3.4.2.pom'.
Could not GET 'https://dl.google.com/dl/android/maven2/com/android/tools/build/gradle/3.4.2/gradle-3.4.2.pom'.
Remote host closed connection during handshake

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 5s
error Failed to install the app. Make sure you have the Android development environment set up: https://facebook.github.io/react-native/docs/getting-started.html#android-development-environment. Run
CLI with --verbose flag for more details.
Error: Command failed: gradlew.bat app:installDebug -PreactNativeDevServerPort=8081
FAILURE: Build failed with an exception.

What went wrong:
A problem occurred configuring root project 'OneApp'.

Could not resolve all artifacts for configuration ':classpath'.
Could not resolve com.android.tools.build:gradle:3.4.2.
Required by:
project :
Could not resolve com.android.tools.build:gradle:3.4.2.
Could not get resource 'https://dl.google.com/dl/android/maven2/com/android/tools/build/gradle/3.4.2/gradle-3.4.2.pom'.
Could not GET 'https://dl.google.com/dl/android/maven2/com/android/tools/build/gradle/3.4.2/gradle-3.4.2.pom'.
Remote host closed connection during handshake

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 5s
at checkExecSyncError (child_process.js:630:11)
at execFileSync (child_process.js:648:15)
at runOnAllDevices (C:\Girish\OneApp\OneApp\node_modules\react-native\node_modules\@react-native-community\cli-platform-android\build\commands\runAndroid\runOnAllDevices.js:94:39)
at buildAndRun (C:\Girish\OneApp\OneApp\node_modules\react-native\node_modules\@react-native-community\cli-platform-android\build\commands\runAndroid\index.js:179:41)
at C:\Girish\OneApp\OneApp\node_modules\react-native\node_modules\@react-native-community\cli-platform-android\build\commands\runAndroid\index.js:133:12
at processTicksAndRejections (internal/process/task_queues.js:97:5)
at async Command.handleAction (C:\Girish\OneApp\OneApp\node_modules\react-native\node_modules\@react-native-community\cli\build\index.js:182:9)

Please help me ..i am struggling since yesterday.

Here is Package.json:
{
"name": "OneApp",
"version": "0.0.1",
"private": true,
"scripts": {
"android": "react-native run-android",
"ios": "react-native run-ios",
"start": "react-native start",
"test": "jest",
"lint": "eslint ."
},
"dependencies": {
"@react-native-community/cli-platform-android": "^2.9.0",
"react": "16.9.0",
"react-native": "0.61.5"
},
"devDependencies": {
"@babel/core": "7.8.4",
"@babel/runtime": "7.8.4",
"@react-native-community/eslint-config": "0.0.5",
"babel-jest": "24.9.0",
"eslint": "6.8.0",
"jest": "24.9.0",
"metro-react-native-babel-preset": "0.56.4",
"react-test-renderer": "16.9.0"
},
"jest": {
"preset": "react-native"
}
}
============================================
Build.gradle:
// Top-level build file where you can add configuration options common to all sub-projects/modules.
buildscript {
ext {
buildToolsVersion = "28.0.3"
minSdkVersion = 16
compileSdkVersion = 28
targetSdkVersion = 28
}
repositories {
google()
jcenter()
}
dependencies {
classpath("com.android.tools.build:gradle:3.4.2")
    // NOTE: Do not place your application dependencies here; they belong
    // in the individual module build.gradle files
}

}
allprojects {
repositories {
mavenLocal()
maven {
// All of React Native (JS, Obj-C sources, Android binaries) is installed from npm
url("$rootDir/../node_modules/react-native/android")
}
maven {
// Android JSC is installed from npm
url("$rootDir/../node_modules/jsc-android/dist")
}
    google()
    jcenter()
    maven { url 'https://jitpack.io' }
}

}
=======================================
gradle-wrapper.properties:
distributionBase=GRADLE_USER_HOME
distributionPath=wrapper/dists
distributionUrl=https://services.gradle.org/distributions/gradle-5.5-all.zip
zipStoreBase=GRADLE_USER_HOME
zipStorePath=wrapper/dists

Unless you provide more information, there is no chance we can help you. Ideally provide a reproducer project. At least explain what you are trying to do and provide the stacktrace of the error.

Hi, I'hv done a setup on new machine with given configuration and trying to run a sample project but facing this issue.

Hi @girishgadad,

Could not get resource 'https://dl.google.com/dl/android/maven2/com/android/tools/build/gradle/3.4.2/gradle-3.4.2.pom'.

This points to some kind of environment/network problem.  Either that is being blocked or the Java install you're using is unable to authenticate that server.   If you can download that URL from a browser, I would try installing a newer version of Java if you can.

Hi @big-guy ,
I am still facing the same issue after installing new version of java

Hi @girishgadad,

Could not get resource 'https://dl.google.com/dl/android/maven2/com/android/tools/build/gradle/3.4.2/gradle-3.4.2.pom'.

This points to some kind of environment/network problem. Either that is being blocked or the Java install you're using is unable to authenticate that server. If you can download that URL from a browser, I would try installing a newer version of Java if you can.

Hi @big-guy ,
I am still facing the same issue even after installing the latest version of java.

someone please help me...I am facing this issue since a week.

Is there any solution since then?

Is there any solution since then?


Add SumatraPDF v3.2

Microsoft Reviewers: Open in CodeFlow

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: drproteus sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

@drproteus did you install Sumatra on your local machine with the manifest. I got an error that looks like a "remove:" failed. This may be the difference in user mode vs. admin mode. It looks like it did install, but I'd like to make sure we're providing the best experience we can for users.

@denelon I must be too used to apt land because I immediately thought I was doing something wrong when I got the error in a user terminal, so I ran it in an elevated prompt. It seems like winget is trying to delete the temp installer before its done being executed. Not sure if its at all related to https://github.com/microsoft/winget-cli/issues/189.

This is blocked on Issue https://github.com/microsoft/winget-cli/issues/189

The command line is incorrect the fileNAME-install.exe is the trigger and the silent option was /s (small) it will likely be -s in the next release but there are other considerations such as allow the multi user administrator to specify /d (destination will become -d) the current default is LocalAppdata


function scaling up is unresponsive while same function is still terminating

I noticed if I allow the 10 second linger to expire for a function that was deployed as a single instance, and then I immediately send an event (via /request) so that it scales to 1 again, the reply from that request seems delayed until just a moment before the terminating function pod's status shows 0/2. If instead I wait until that status is 0/2 or the "Terminating" function pod disappears altogether, then a new event (via /request) is handled much more quickly.

one idea is that the terminating function's sidecar is hanging on to its Kakfa consumer connection

The function-sidecar was not responding to SIGTERM signals which occur when K8s terminates Pods: https://kubernetes.io/docs/concepts/workloads/pods/pod/#termination-of-pods
This has been fixed via: https://github.com/ericbottard/function-sidecar/commit/b1cf1888efe2f96dc8f0afef7105b41a4d42d1fd


Rework database-type and database-version setup in config.yaml/ddev config

In preparation for

#2187

it's going to be important to rework how we describe database type and version.
Currently we have the very awkward "mysql_version" and "mariadb_version", whose use implies the database type. It's really ugly.
I recommend a change to this format:
db_server: mariadb:10.3
or
db_server: postgres:14
It will be ugly trying to upgrade current stuff, but maybe it will be possible. But I don't think #2187 is really possible without this change.

This was completed in https://github.com/drud/ddev/pull/3539


Increasing need to store emojis in a single location

Remaining instances of this issue to rectify

[x] Prestige emojis are currently installed on the application, and we require the CDN URL of the emoji for our prestige embed thumbnail. Currently not possible because the API endpoints for this PR hasn't been implemented yet. See the comment below as a workaround has been made to this issue.

Is your feature request related to a problem? Please describe.
When I update an emoji, which I had to do multiple times in a recent commit, I have to find every usage of the old emoji and replace it with the new one. No need to explain why this is a problem.
Describe the solution you'd like
Store all the emojis in a json file?
Describe alternatives you've considered
Use it in a botvar, perhaps.
Additional context
Add any other context or screenshots about the feature request here.

Some say to use a dictionary to store emoji name as key and the markdown as the value, some say to do this in a seperate JSON file. I understand both and will consider later down the road.

Some use enums for this.

Put on hold until we can do something like this:
{em.name: em.id for em in self.bot.application_emojis}

The relevant PR for app emojis is still a WIP.

After making some progress on this, decided it wasn't worth it. The chances of needing to change all the emoji usages now after the application emojis introduction are really low, and even then the bot hasn't grown and most likely will never grow at a scale to cause a major pain to rewrite.
As for the task associated with this issue, a better workaround has already been implemented, one that does not require making an HTTP request.
The overhead involved in fetching the application emojis via HTTP and getting it via a stored dictionary does not feel right. So, for now, will continue leaving things with emojis as-is, even if it reduces the degree of maintainability by some extent.

Incorrectly labelled the issue closure as completed


Returning earlier instead of throwing a locally caught error

Summary
Looking at the Sentry issue Failed setCurrentUserStatusOffline - I found that an error is thrown and caught locally if the  current user is not found while trying to execute the setCurrentUserStatusOffline action.  This action is called by the WS onWebsocketClosecallback.  I'm not sure if this line should be throwing an error at all... since the WebSocket manager isn't using the error.
Release Note
NONE

The reason for this throw is so it is logged in the catch part. The real problem here is why this function is being called with no current user for the server.

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!

closing


Add PropertyInfo.GetHandledValue

There are 40 tests failing. Would you be able to check? Also, did you see my last comment about the overload method that accepts the argument 'entityType'v

Where can I see the failing tests. RepoDb.UnitTests are working, RepoDb.IntegrationTest is not possible for me because I have no MSSqlServer. Do you have a buildserver?

I think I commented just right after your PR. By that time, the tests are still executing. But upon visiting now, they're all green.

What does that mean?

It means that all the tests were passing. In every project page on Github (ie: Core, SqlServer, MySql, SqLite, PostgreSql and SqlServer.BulkOperations) has their own Unit and Integration Tests at App Veyor. You can click the icon shields to visit the test. Here is the link for the Core's integration tests.

I can accept your PR now, but will you still do the overload method? I can also do it right after if you wish.

I added already the overload or did I do it wrong?
public static object GetHandledValue(this PropertyInfo propertyInfo, object entity) { ... }
public static object GetHandledValue(this PropertyInfo propertyInfo, Type declaringType, object entity) { ... }

Perfect! Thanks!

@davidrot - made a fine review today and added some XML comments. I liked your coding standard. I hope you will contribute more soon.
Btw, I notice you're not a part of the contributors list, I do not why. It might be because you're using different email address (I supposed).

Hi @mikependon,
thanks for the feedback. I think I will contribute in the future. It looks like I'm using RepoDb in a bigger project.
I've added my commit email address to the github profile, now I'm listed in the contributors list :)


Fix build

I know it's not used anymore, but having the build working on stable helps to better test the new upcoming release :)

Thank you very much! :)


injected License object attribute unitOfMeasure not valid when liceseType is regKey

submitted declaration License object:
        "licenseKey": {
            "licenseType": "regKey",
            "class": "License",
            "regKey": "BUKNP-EAXJD-AKKVF-IDSYT-FAYLEUR",
            "overwrite": false
        },

produces an output declaration object with License object:
            "licenseKey": {
                "licenseType": "regKey",
                "class": "License",
                "regKey": "BUKNP-EAXJD-AKKVF-IDSYT-FAYLEUR",
                "overwrite": false,
                "unitOfMeasure": "monthly"
            },

which when validated against:
https://raw.githubusercontent.com/F5Networks/f5-declarative-onboarding/master/schema/base.schema.json

Generates a validation error: 'Property unitOfMeasure is not allowed'
All injected default values MUST conform to schema validation.

AT-212 for internal tracking

Resolved in 1.5.0


Fixes CVE-2023-48795

chore: Fixes CVE-2023-48795 - golang.org/x/crypto Authentication Bypass by Capture-replay

Description

How Has This Been Tested?

Merge criteria:

[ ] The commits are squashed in a cohesive manner and have meaningful messages.
[ ] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[ ] The developer has manually tested the changes and verified that the changes work

/retest


Sort order in lists

With reference to https://github.com/trimstray/the-book-of-secret-knowledge/pull/87 and after going through some of the other longer lists (e.g. https://github.com/trimstray/the-book-of-secret-knowledge#black_small_square-network) I think it would be of great benefit to users if the list of utilities were sorted alphabetically. Currently most of the lists don't have any sort order.

That's a great idea, each category can be sorted alphabetically.

HealthofSoulHeart&Mind=Ninpo=Chi+Seal=Meditation=Energy=ConvertWork2Exercise=free012131410

I agree to that suggestion, it would be awesome ;)

I agree to that suggestion, it would be awesome 😁

It's been a year since this was raised and I see a tag "Review Needed" which as per my understanding is - a review of the suggestion is needed.
No, I don't think any such review is needed to move a list to an alphabetical order. The only reason a maintainer can keep pushing a list with bizarre ordering is - he has personal bias towards some entries and wants to highlight those. Nothing wrong with that but I believe such kind of lists should have a disclaimer that says - the order is based on personal preference so visitors are not influenced by the order.


Add convenience script for metal platform

Add a script that does everything we want in the various places this is
to be used in the metal platform, with the first argument specifying
which:

--all for the installer bootstrap
--pxe for the metal3 Deployment
--image-build for the image-customization-controller

Also create symlinks to the existing ironic-python-agent.* filenames so
that no code in the installer or cluster-baremetal-operator has to
change to reference the right files.
Add support for the IP_OPTIONS environment variable to customise the kernel command line when extracting the ISO.

/cc @elfosardo

/lgtm
I'd love to see this written in Python/Go/anything that is not bash :)

/override ci/prow/e2e-metal-ipi-ovn-ipv6

/override ci/prow/e2e-metal-ipi-serial-ipv4


Extend graph merge

While so.merge() provides some initial merge functionality, it would be nice to be able to combine / merge graphs more extensively (see image for an example).

We would need to think about the interface, but it would make the construction of custom graphs by combining subgraphs much easier.

After a number of useful additions on this issue (see https://github.com/scailable/sclblonnx/pull/19), it seems to make sense to first flesh out in a bit more detail what the desired behavior should be of (additions to) the merge function. Useful ways to proceed seem two-fold:

Create (additional) tests for the merge function (in test/test_merge.py to start with) such that any new functionality can be done using test driven development.
Create some "jargon" to easily discuss possible additions to the merge function.

Let me give 2. a try below (but please do suggest changes / improvements).
Discussing graph merge variations
It seems that for all merge purposes a graph can be defined by its inputs and outputs (two lists). Assuming we coin "graphs to be merged" subgraphs sg, we could write sg1{[in1, in2, in3, ...], [out1, out2, out3, ...]} to identify a graph. From that starting point it seems we can provide meaningful descriptions of merge features. For example, I could detail the current functionality generally as:

Given two subgraphs sg1{[in1, in2, in3, ...], [out1, out2, out3, ...]}, sg2{[in1, in2, in3, ...], [out1, out2, out3, ...]}, an ordered list of outputs of sg1, [out1, out2, ...] called outputs and an ordered list of inputs of sg2, [in1, in2, ...] called inputs, create a new graph g in which sg1 is deemed the parent of sg2 and which has inputs sg1[in] + (sg2[in] - inputs) and outputs (sg1[out] - inputs) + sg2[out].

This leads to some practical examples:

Suppose we have a simple subgraph that adds two tensors x1 and x2, and outputs their sum. Next, we have a subgraph that checks whether the input is less than some threshold and outputs check. If we now want to merge the two we thus have:

sg1{[x1, x2], [sum]}  # first subgraph
sg2{[input], [check]}. # second subgraph
outputs = [sum]  # the outputs vector of sg1 to be matched with
inputs = [input]. # the inputs vector of sg2

# result:
g{[x1, x2], [check]}

Which is the simplest use case the function was designed to handle.
Next steps.
If the above "jargon" is useful we can propose new functionality using it, such that we are precise whenever suggesting functionality.
Suggested additions that are clearly useful:
In https://github.com/scailable/sclblonnx/pull/19 we have at least two things that seem useful as is:

Being able to (re) name the internal nodes of the graph g when doing a merge; this is useful for repeated merges on the same graph etc. Thus, I would suggest accepting the append_unique_string proposal.
Being less strict on the checking of the match of inputs and outputs. This is useful whenever one would like to merge more than two subgraphs in sequence, where the intermediate result might not itself be a fully valid graph yet. Actually, it seems that this would allow behavior such as the desired behavior at the start of this issue.

Any suggestions on changing the signature of the function to make it flexible (and intuitive) are very welcome!

A number of these ideas are implemented in 44c5078 (and thanks to all the discussion in https://github.com/scailable/sclblonnx/pull/19). I will close this issue for now, but please raise additional issues if you find any problems in the new solution using merge(), join(), split(), and concat().


import cv2 on codespaces

importing the cv2 throws
ImportError: libGL.so.1: cannot open shared object file: No such file or directory,
when I started working on notebooks/201-vision-monodepth/201-vision-monodepth.ipynb
I was able to solve this by installing opencv-python-headless. Although I was using the GitHub codespace im not sure if that is the reason, so just wanted to report this. I followed the installation guide  Ubuntu

Hi @fajemila
Is your issue resolved?

Closing this, I hope previous responses were sufficient to help you proceed. Feel free to reopen and ask additional questions related to this topic.