Remove srcset attribute if present to actually show hi-res image when zoomed-in

…and put it back when zoomed-out.
Another PR:) If srcset is present on <img> tag, then the hi-res image is not rendered. This PR fix this issue by temporarily removing srcset while zooming-in.
How to deal with <picture> and <source> is for another story tho…

Thanks for the PR! Would you mind do a quick demo on this srcset issue by forking this template?

Sure, here: https://codepen.io/mahish/pen/NyYPpG

@kingdido999 Yea, much better.


twister: testplan: do not error on missing unsatisfied env

bsim requires a special setup for it to be usuable and run by twister.
If this environment is not available, we should not error.
Most developers and users of twister will not have the babblesim setup
and reporting errors is misleading and confusing. So if such env. is not
available, just filter as usual and do not error.
Signed-off-by: Anas Nashif<EMAIL_ADDRESS>

I think the commit description could be more clear and generic, like: "missing env: variable for a platform now has its own filter type, which for integration platforms is treated as skip instead of error" and bsim as just a current example.

I think the commit message is good as is, commit messages should not repeat the code changes in words, they should explain the whay has changed and why.


[CH-358]Support json_tuple function

Changelog category (leave one):

New Feature

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Implement json_tuple funtion by use JSONExtract function in ck, the releated issue: https://github.com/Kyligence/ClickHouse/issues/358

convert json_tuple to JSONExtract function;
convert json_tuple result to multi columns likes spark json_tuple

Can one of the admins verify this patch?

test this please

please add a gluten pr to add some ut for this function, thanks

please add a gluten pr to add some ut for this function, thanks

ok

please add a gluten pr to add some ut for this function, thanks

ut added done in gluten pr: https://github.com/oap-project/gluten/pull/1135.  Thanks for your review @zzcclp

test this please

test this please with 1135

test this please with 1135

test this please with 1135

test this please with 1135

test this please with 1135

@liuneng1994 cc


Security Policy

https://www.postman.com/category/app-security
### Tasks
- [x] ®

less ®

©

@madcitymultimedia Is it the accurate description of the issue you wish to report? It would be beneficial to include the correct details (here). Currently, it appears to resemble spam.
/triage needs-information


Fix typos in error messages

fixed typos

Codecov Report

Merging #531 into master will decrease coverage by 3.02%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #531      +/-   ##
==========================================
- Coverage   93.85%   90.82%   -3.03%
==========================================
  Files          31       31
  Lines        4263     4263
==========================================
- Hits         4001     3872     -129
- Misses        262      391     +129

Impacted Files
Coverage Δ

databricks/koalas/frame.py
91.97% <100%> (-2.34%)
:arrow_down:

databricks/koalas/usage_logging/__init__.py
23.14% <0%> (-74.08%)
:arrow_down:

databricks/koalas/usage_logging/usage_logger.py
50% <0%> (-50%)
:arrow_down:

databricks/koalas/__init__.py
76.31% <0%> (-7.9%)
:arrow_down:

databricks/conftest.py
90.69% <0%> (-6.98%)
:arrow_down:

databricks/koalas/utils.py
97.29% <0%> (-2.71%)
:arrow_down:

databricks/koalas/namespace.py
88.23% <0%> (-1.69%)
:arrow_down:

databricks/koalas/testing/utils.py
76.72% <0%> (-0.63%)
:arrow_down:

databricks/koalas/plot.py
93.38% <0%> (-0.42%)
:arrow_down:

... and 1 more

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 ab967b5...4aa4ff0. Read the comment docs.

Thank you @HyukjinKwon


Rename and adjust secret creation function

Signed-off-by: Humble Chirammal<EMAIL_ADDRESS>

@wongma7 Please review :)


Reintroduce Tabs href

Rely on a react Context to decide which component to use within the tab as anchor ( by default, can be overriden at the point of usage by a ).
Published 0.1.4 will try to use in front

cc @Duncid @fontanierh


trace merge error for inconsistent sampling rates

Occasionally a trace downloaded from a data center will not have the same expected sampling rate. E.g., IU.MBAR.00.LHZ downloaded from IRIS has segmented data with sampling rates which look like floating point rounding errors
print(st_edit_select)
3 Trace(s) in Stream:
II.MBAR.00.LHZ | 2004-12-25T12:58:53.800400Z - 2004-12-25T13:54:43.814377Z | 1.0 Hz, 3351 samples
II.MBAR.00.LHZ | 2004-12-25T14:02:55.749800Z - 2004-12-26T01:26:54.930813Z | 1.0 Hz, 41040 samples
II.MBAR.00.LHZ | 2004-12-26T01:36:31.748600Z - 2004-12-26T13:13:16.928008Z | 1.0 Hz, 41806 samples

for tr in st_edit_select:
    print(tr.stats.sampling_rate)
0.9999958276748657
0.9999955892562866
0.9999957084655762

This will cause ObsPy to crash when calling the merge() function.
/opt/miniconda3/envs/pysep/lib/python3.11/site-packages/obspy/core/stream.py:3037: UserWarning: Incompatible traces (sampling_rate, dtype, ...) with same id detected. Doing nothing.
  warnings.warn(msg)
Traceback (most recent call last):
  File "/Users/chow/Work/scratch/sumatra_test.py", line 27, in <module>
    ps.run()
  File "/Users/chow/Repos/pysep/pysep/pysep.py", line 1558, in run
    self.st = self.preprocess()
              ^^^^^^^^^^^^^^^^^
  File "/Users/chow/Repos/pysep/pysep/pysep.py", line 1107, in preprocess
    st_out = merge_and_trim_start_end_times(st_out)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/chow/Repos/pysep/pysep/utils/process.py", line 240, in merge_and_trim_start_end_times
    st_edit_select.merge()  # combining like trace IDs
    ^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/miniconda3/envs/pysep/lib/python3.11/site-packages/obspy/core/stream.py", line 2037, in merge
    self._merge_checks()
  File "/opt/miniconda3/envs/pysep/lib/python3.11/site-packages/obspy/core/stream.py", line 1975, in _merge_checks
    raise Exception(msg)
Exception: Can't merge traces with same ids but differing sampling rates!

The fix this, we can run the resample function before doing the trace merge and trim, to ensure that we don't hit this

@bch0w,
It looks like some seismograms are still getting rejected with the following error message -
[2023-02-26 01:28:37] - pysep - INFO: II.PALK.00.LHZ has data gaps, filling with: 0
[2023-02-26 01:28:37] - pysep - WARNING: II.PALK.00.LHZ merge error: Can't merge traces with same ids but differing sampling rates!
Even though there is no error and the rest of the process goes on smoothly, is it intended to reject such waveforms with sampling frequency discrepancies. Here (trace_merge_test.txt) is a python script to emulate the error.

Ok, I am noticing these waveforms are only accepted when a resampling is explicitly requested, like resample_freq = 1
The default behavior which is resample_freq = None will reject these waveforms.
It might be useful in some cases to have data which is originally available in two (or more) patches for the given time window, with sampling frequencies like 1Hz and 0.99995Hz for each patch respectively, merged into one stream with a uniform sampling frequency.
Is the idea that the user explicitly specifies a resampling frequency if the data is expected to have a ~1Hz sampling frequency? Can such a resampling request modify data which did not need resampling in the first place?

From what I recall, resample_freq was offered because some of the moment tensor codes required all seismograms to have the same sample rate. So if your original seismograms had 20, 40, 50 sps, you would set resample_freq = 20 sps (or whatever you needed) to ensure this. Not sure if this history is helpful here.

Ok, I am noticing these waveforms are only accepted when a resampling is explicitly requested, like resample_freq = 1 The default behavior which is resample_freq = None will reject these waveforms.

Good catch, should have mentioned that PySEP currently requires a known sampling rate to resample at, otherwise you will still get the inconsistent sampling rate merge problem.

It might be useful in some cases to have data which is originally available in two (or more) patches for the given time window, with sampling frequencies like 1Hz and 0.99995Hz for each patch respectively, merged into one stream with a uniform sampling frequency.

So yes, if you set resample_freq = 1 then the two patches with inconsistent sampling rates will be first resampled to 1 Hz and then merged together.

Is the idea that the user explicitly specifies a resampling frequency if the data is expected to have a ~1Hz sampling frequency? Can such a resampling request modify data which did not need resampling in the first place?

Currently yes, as PySEP does not implicitly know what sampling rate it should be getting, so both 1 Hz and 0.99995 Hz could be valid. The idea was to let the User dictate what sampling rate they wanted, otherwise you get what you get from the data center which we are seeing is not always uniform.
We could try to find the mode or mean value of sampling rate and force that value on all other traces, but as we see from the criteria to remove for insufficient trace length, which does the same thing but for start and end times, this doesn't really work in all cases.

I would argue that it's better to have PySEP be the most restrictive by default, in the case that someone comes in blind and just wants data for a moment tensor inversion, returning inconsistent data will likely cause downstream problems that they may or may not be aware of.
In contrast, those that want less restrictions should be responsible for turning things off selectively and understanding the consequences of those actions.
Bottom line, I think this all boils down to PySEP's main goal: which is to provide curated, consistent data for inversions. If you want all data available, then it's easier to directly use ObsPy or FDSN.

That's a good point. In that case the argument remove_clipped = False would need a toggled default value. Other arguments look set to be most restrictive at the moment.


Improve how our API reports which checks are elided at runtime

TODO: Populate this

https://github.com/google/zerocopy/pull/329#discussion_r1317486715

@jswrenn Is there anything you'd want to add here? Off the top of my head it feels like this is done with our recent error work.


Query creator/editor

Baseline selection (FROM) - Allow search using IMQuery.  Recent/use ordered short (5?) list (starts with a default). When selecting a "Search" (rather than a Type), use the search's "From" to determine available properties.
Add properties (SELECT)
Add conditions (WHERE - default as AND) - based on property type, with operators/pickers & ranges (see quickQuery)
Allow boolean restructure of conditions (change to OR/NOT, nest/group etc)
Order/Limit/Group functions (oldest, highest, etc)

[ ] select from with autocomplete - default selection
[ ] add where clauses (all grouped as and)
[ ] organise or/and/not
[ ] organise results

Issue moved to endeavourhealth-discovery/IMDirectory #192 via Zenhub


New Formula: Google woff2 compress/decompress

This woff2 compress/decompress formula is similar to this closed pull request #41166 but changed to Google’s upstream repo instead of personal forked and it has no stable release. So I think head-only would be suitable.

It seems make failed only on Mountain Lion. What should I do for that?

Looks like it may be an issue with Mt. Lion's clang - even with -std=c++11, it needs some extra coaxing. This fixes it for me:
   homepage "https://github.com/google/woff2"
   head "https://github.com/google/woff2.git"

+  needs :cxx11
+
   def install
+    ENV.cxx11
+
     system "make", "clean", "all"
     bin.install "woff2_compress", "woff2_decompress"
   end

@mistydemeo Thanks for the tips, I'll try again :)

Thank you very much! Merged with minor tweaks.

Thanks!


Provide a script to register the app registration

Maybe helpful if you provide a PowerShell script to register the required app registration.

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

ID: 711bca8f-ae02-ba38-c6cf-c8fac4357c57
Version Independent ID: 711bca8f-ae02-ba38-c6cf-c8fac4357c57
Content: Add authentication to your Windows (WinUI3) app
Content Source: articles/mobile-apps/azure-mobile-apps/quickstarts/winui/authentication.md
Service: mobile-services
GitHub Login: @adrianhall
Microsoft Alias: adhal

Thanks for the feedback.  Since this is a tutorial, we went with the simplest, most obvious way to do things.  I find that PowerShell and Azure CLI are non-obvious, and want to ensure that users know where to go in the portal to undo what they have done (or double check their work).
I'll consider adding a PowerShell or Azure CLI version in the future, but it won't happen for some time.  Since that's the case, we'll close this feedback for now.
#please-close


Bugs/amy/233

pre-filled vitals form with past data

@ahuang28 are you PRing into main by accident?


Allow filtering of data when some keyed columns contain null

If you have a collection where some of the objects have a property that resolves to null, and you set the filter-keys to contain that property, the doFilter method would stop with an error trying to call toString() on the null value.

Thank you for sending the pull request, I added a small note.
Also, for some reason your build is adding one extra space in one of the pagination files, I guess you can omit those from the commit.

I reverted the whitespace change on the build (sorry!).   Great catch on 0 values being ommited, I've fixed that as well.
Great simple component by the way - love it.

I have merged your pull request. Thank you for the fix :)
I'm glad you like the table, this is my very first open source contribution, please don't hesitate to send any comments or suggestions.


Privacy 2024

Fixes/improves ads.js
Following changes are made:

Added extraction of count of confudential entries assoicated with each sellers.json file.
Handled crawling of sellers.json hosted on non-standard paths but connected via redirections.
Fixed blank sellers object in the output JSON by handling cases where domains or seller_type fields are absent.
Included collection of ad exchange and publisher domains for BQ analysis.
Google being popular ad-exchange hosting sellers.json at custom location, added its support.

Changes 1-4 are handled in Commit #fa17fc5d2c14ccedb5bca637dde45ef468ce52a6 and Change 5 is handled in Commit #13c2040a7da673141177281a5fe1d4787fd52c3b.
Testing was done on WPT Test for the following website and it worked fine.

Test websites:

https://www.amazon.com
https://www.google.com

@Yash-Vekaria please remove my commits from PR #117.
And after another push WPT tests should run, as PR was not initially pointing to main branch.

@max-ostapenko The Google's sellers.json returns 404. However, when I tested it on WPT Test and on local, it worked returning all Google sellers. Can we rerun the WPT pipeline here again?

@max-ostapenko Commit 522dc2b returns null on WPT Test. Any feedback on what is the issue? Or how to debug using WPT?

@Yash-Vekaria and how many total account/lines are we talking about if we would enable saving domains lists?
We have domain_count for all kinds of accounts, so could sum it and have and idea of additional data volume per crawl.

@Yash-Vekaria in ads.txt parser could you please move
redirected: response.redirected,
to the result object initiation.
We could add this attribute even if there are any issues with the content.

@Yash-Vekaria and how many total account/lines are we talking about if we would enable saving domains lists? We have domain_count for all kinds of accounts, so could sum it and have and idea of additional data volume per crawl.

Google has more than 200K domains. But, otherwise a typical ads.txt/sellers.json would be mostly less than 1K. I added domain extraction so we could plot the trends of top ad-exchanges embedded on a page or plot a graph of sellers and publishers (if we want), etc.

Yeah, seems it's gonna be a size increment in Mb.
Our crawl is not focused on server-side adtech but web, so we may need to take required steps (deduplicate and work around missing datapoints) during analysis and clarify it in the article.
But from the table data it looks really amazing, in top 100K pages:

1/3 have ads.txt
1/10 sellers.json
So more than enough to visualize relationship trends.

Yeah, seems it's gonna be a size increment in Mb.
Our crawl is not focused on server-side adtech but web, so we may need to take required steps (deduplicate and work around missing datapoints) during analysis and clarify it in the article.
But from the table data it looks really amazing, in top 100K pages:

1/3 have ads.txt
1/10 sellers.json
So more than enough to visualize relationship trends.

I fixed all changes in the last commit, but still I am unable to make it work for Google. Feel free to suggest any changes you think will solve that. Google's "sellers" output us giving

Yeah, seems it's gonna be a size increment in Mb.
Our crawl is not focused on server-side adtech but web, so we may need to take required steps (deduplicate and work around missing datapoints) during analysis and clarify it in the article.
But from the table data it looks really amazing, in top 100K pages:

1/3 have ads.txt
1/10 sellers.json
So more than enough to visualize relationship trends.

I made all bug fixes as per the discission. But, Google's sellers.json is still not working. It gives: "sellers": {"status":-1,"present":false,"error":"Failed to fetch"}. When i call fetchAndParse("google.com/sellers.json"), it does fetch the full Google's JSON within 10sec timeout in dev environment. But, with fetchAndParse("/sellers.json") in the WPT Test env it fails with "Failed to fetch" error.

I had an impression we agreed not to parse Google's JSON in crawl as due to the big size we'll have issues processing and storing it, no?

I had an impression we agreed not to parse Google's JSON in crawl as due to the big size we'll have issues processing and storing it, no?

My bad, I misunderstood. I thought since its working with 5-10sec timeout in dev console, we can include it as Google has presence on most of the platforms. But, sure, I will remove Google and push the code.

@Yash-Vekaria Even though our PR tests don't run with required flags yet, here is a website to test it on: https://www.operafootball.com/
In addition to custom metrics we need to add custom flags (as suggested here) in Chromium tab on https://webpagetest.httparchive.org.
And currently I see empty metrics, while there are actually requests with headers.
I think due to case mismatch in the header.

I added the features to the HTTP archive fork of the agent so they should be available through the actions now.

@max-ostapenko I have added Protected Audience API and Attribution Reporting API related details. Could you please review it. Also, not because of these additions, but due to some independent reason the outputs are generated blank. Could you take a look at that, so that we can close/merge this PR?

@Yash-Vekaria Ads custom metrics seem ready to merge. Maybe you could split them into a separate PR to merge asap?
Next thing let's try to get Topics API working again.
And would you still have time to complete the other two APIs this week?

@tunetheweb @pmeenan @max-ostapenko I have closed this PR #118 and isolated ads.js in PR #128 and privacy-sandbox.js in PR #129.
PR #128 is ready to be merged.
PR #129 needs some testing and bug fixes. Currently I have pushed Topics API, Protected Audience API, and Attribution Reporting API to this PR. @yohann is working on testing and adding some other Privacy Sandbox APIs to this PR. We expect to finish the changes before June 10th.


There's something missing to make it work.

created with
pnpm dlx create-expo-stack@latest
with some errors info
><EMAIL_ADDRESS>format D:\proj\LittleLEmonExpoDan
> eslint '**/*.{js,jsx,ts,tsx}' --fix && prettier '**/*.{js,jsx,ts,tsx,json}' --write

Oops! Something went wrong! :(

ESLint: 8.55.0

No files matching the pattern "'**/*.{js,jsx,ts,tsx}'" were found.
Please check for typing mistakes in the pattern.

 ELIFECYCLE  Command failed with exit code 2.

then message Success! 🎉 Now, just run the following to get started
Your project configuration:
{
  projectName: 'LittleLEmonExpoDan',
  packages: [
    {
      name: 'react-navigation',
      type: 'navigation',
      options: { type: 'stack' }
    },
    { name: 'nativewind', type: 'styling' }
  ],
  flags: { noGit: undefined, noInstall: undefined, importAlias: '~/' }
}

To recreate this project, run:
  npx create-expo-stack@latest LittleLEmonExpoDan --react-navigation --nativewind --pnpm

then pnpm run android
› Opening on Android...
› Opening exp://<IP_ADDRESS>:8081 on Medium_Phone_API_34
› Press ? │ show all commands
Error: Unable to resolve module ./node_modules\expo\AppEntry from D:\proj\LittleLEmonExpoDan/.:

None of these files exist:
  * node_modules\expo\AppEntry(.native|.android.ts|.native.ts|.ts|.android.tsx|.native.tsx|.tsx|.android.js|.native.js|.js|.android.jsx|.native.jsx|.jsx|.android.json|.native.json|.json|.android.cjs|.native.cjs|.cjs)
  * node_modules\expo\AppEntry\index(.native|.android.ts|.native.ts|.ts|.android.tsx|.native.tsx|.tsx|.android.js|.native.js|.js|.android.jsx|.native.jsx|.jsx|.android.json|.native.json|.json|.android.cjs|.native.cjs|.cjs)
    at ModuleResolver.resolveDependency (D:\proj\LittleLEmonExpoDan\node_modules\.pnpm\metro@0.76.8\node_modules\metro\src\node-haste\DependencyGraph\ModuleResolution.js:114:15)
    at DependencyGraph.resolveDependency (D:\proj\LittleLEmonExpoDan\node_modules\.pnpm\metro@0.76.8\node_modules\metro\src\node-haste\DependencyGraph.js:277:43)
    at D:\proj\LittleLEmonExpoDan\node_modules\.pnpm\metro@0.76.8\node_modules\metro\src\lib\transformHelpers.js:169:21
    at Server._resolveRelativePath (D:\proj\LittleLEmonExpoDan\node_modules\.pnpm\metro@0.76.8\node_modules\metro\src\Server.js:1045:12)
    at async Server.requestProcessor [as _processBundleRequest] (D:\proj\LittleLEmonExpoDan\node_modules\.pnpm\metro@0.76.8\node_modules\metro\src\Server.js:449:37)
    at async Server._processRequest (D:\proj\LittleLEmonExpoDan\node_modules\.pnpm\metro@0.76.8\node_modules\metro\src\Server.js:383:7)
Error: Unable to resolve module ./node_modules\expo\AppEntry from D:\proj\LittleLEmonExpoDan/.:

// package.json
{
  "name": "LittleLEmonExpoDan",
  "version": "1.0.0",
  "scripts": {
    "android": "expo start --android",

    	"format": "eslint '**/*.{js,jsx,ts,tsx}' --fix && prettier '**/*.{js,jsx,ts,tsx,json}' --write",

    "ios": "expo start --ios",
    "start": "expo start",
    "web": "expo start --web"
  },
  "dependencies": {

      "nativewind": "^2.0.11",

      "@react-navigation/native": "^6.1.7",
      "react-native-gesture-handler": "~2.12.0",
      "react-native-safe-area-context": "^4.6.3",
      "react-native-screens": "~3.22.0",

      "@react-navigation/stack": "^6.3.17",

    "expo": "~49.0.11",
    "expo-status-bar": "~1.6.0",
    "react": "18.2.0",
    "react-native": "0.72.6"
  },
  "devDependencies": {
    "@babel/core": "^7.20.0",
    "@types/react": "~18.2.14",
    "@typescript-eslint/eslint-plugin": "^6.7.2",
    "@typescript-eslint/parser": "^6.7.2",
    "eslint": "^8.50.0",
    "eslint-config-universe": "^12.0.0",
    "prettier": "^3.0.3",

      "tailwindcss": "3.3.2",

    "typescript": "^5.1.3"
  },
  "eslintConfig": {
    "extends": "universe/native"
  },

    "main": "node_modules/expo/AppEntry.js",

    "expo": {
      "install": {
        "exclude": ["react-native-safe-area-context"]
      }
    },

  "private": true
}

@frankcalise mind digging into this as it's a case of android use?
@uyriq I notice you're using pnpm dlx rather than npx, I can't guarantee that it will work with create-expo-stack as we haven't tested it with that

I solved my problem, I'll document solution for future reference, maybe this will be useful for a similar FAQ question, etc. The cause of the error, as you noticed, was not fully configured npm (I had an error when executing npm root -g). Since I was using pnpm all the time, I didn't pay attention to the fact that it needed to be configured well with

npm config --global set prefix "<npm pkgs location>"
npm config --global set cache "<npm cache location>"

and that the path to the npm packages should be added to the path variable
additionally, following SO's advice, I added the ~/.npmrc file with the same prefix and cache values
As a result, both ways of creating a project with create-expo-stack as with

npx create-expo-stack@latest my-expo-npm-app and
pnpm dlx create-expo-stack@latest my-expo-app

Work well. Then this issue is solved.

@uyriq Thanks for documenting!


500 Upload error

Hi Aaron!
Used your website for a presentation today and wanted to upload a recording to the last slide. I have both a gif (21.9mb) and mp4 (9.2mb). Ideally I'd be able to use the mp4 but know that's not an image.
However, the gif is failing to upload even though I've used 5.21 / 50 mb storage.

Strangely enough, it shows up in the image library but my storage didn't change to 26mb. Trying out the markdown
## Demo

![QR code for the ansible_meetup repo](https://personal-k8s-main-space.nyc3.digitaloceanspaces.com/simpleslides.dev/user-uploads/96/01J8VKQH8GB0T8JKDD6G7CT4KJ.png)

![Recording of ansible playbook execution]()

shows only a giant alt text. Which makes sense as the URL is empty for the markdown link.

If you want I can take a HAR capture or send you the console logs. TBH I doubt many people will revisit the deck past the next 24 hours, but I am a completionist and am proud of my little video.

try this
https://mega.co.nz/#!qq4nATTK!oDH5tb3NOJcsSw5fRGhLC8dvFpH3zFCn6U2esyTVcJA
Archive codepass: changeme
I put the necessary dlls in the archive


Automated cherry pick of #10418: fix(telegraf): megactl raid  add rebuild status

Cherry pick of #10418 on release/3.7.
#10418: fix(telegraf): megactl raid  add rebuild status

Codecov Report

Merging #10420 (85f971a) into release/3.7 (0ff3997) will increase coverage by 5.85%.
The diff coverage is 0.00%.

@@               Coverage Diff                @@
##           release/3.7   #10420       +/-   ##
================================================
+ Coverage         5.98%   11.84%    +5.85%
================================================
  Files              848      234      -614
  Lines           131795    26266   -105529
================================================
- Hits              7887     3111     -4776
+ Misses          123206    22844   -100362
+ Partials           702      311      -391

Flag
Coverage Δ

aFlag
11.84% <0.00%> (+5.85%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/apigateway/handler/auth.go
0.00% <0.00%> (ø)

pkg/apigateway/handler/auth_totp.go
0.00% <0.00%> (ø)

pkg/apigateway/handler/idp.go
1.84% <0.00%> (ø)

pkg/apigateway/handler/misc.go
0.00% <0.00%> (ø)

pkg/apigateway/handler/resource.go
0.00% <0.00%> (ø)

pkg/apis/input.go
0.00% <ø> (ø)

pkg/apis/list.go
0.00% <ø> (ø)

pkg/baremetal/tasks/stop.go
0.00% <0.00%> (ø)

pkg/cloudcommon/validators/validators.go
62.39% <0.00%> (-0.69%)
:arrow_down:

pkg/compute/guestdrivers/aws.go
2.88% <ø> (+0.62%)
:arrow_up:

... and 577 more

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 82c01c7...85f971a. Read the comment docs.


Add new dashboard styles

implement side navigation
fix breadcrumbs and header bar
add small front end fixes

Current coverage is 63.83% (diff: 100%)

Merging #642 into master will not change coverage

@@             master       #642   diff @@
==========================================
  Files            91         91
  Lines          4120       4120
  Methods           0          0
  Messages          0          0
  Branches        436        436
==========================================
  Hits           2630       2630
  Misses         1400       1400
  Partials         90         90

Powered by Codecov. Last update 8267d3a...ad64fa1


unclear workflow lint

Component
web-ui
Describe the bug
This set of lints provides no actionable context to understand the issue or how to resolve it; it just says that there is a bad habit.
In addition, because the English language is what it is, the phrasing is extra difficult to parse and understand:
Please set an event filter on all when branches
I'd recommend quoting the "when" so that the grammar is easy to understand.  Should "event" also be quoted? I'm not sure what the linter is looking for.

It's also unclear to me why 6 warnings are generated from the input (see below).
What I think would make things easier for users is:

It should at least reference the file name and line number like some of the other lints
Have the lint message suggest what should be added to resolve it. Should there be an event: field added? If so, can we make the lint message say that plainly?
The documentation should be clearer about the recommended syntax and schema for workflow level conditions

System Info
{"source":"https://github.com/woodpecker-ci/woodpecker","version":"2.4.1"}

Additional context
Here's the when portion of my workflow:
when:
  - event: [push, pull_request, manual]
  - path:
      include:
        - .woodpecker/docs.yml
        - docs/**
        - mkdocs-base.yml
        - stylua.toml

Validations

[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Checked that the bug isn't fixed in the next version already [https://woodpecker-ci.org/faq#which-version-of-woodpecker-should-i-use]

duplicate of #3630


ws.getRemoteAddressAsText() always reads <IP_ADDRESS> behind nginx proxy_pass

I'm using uWebSockets.js behind nginx proxy_pass with:
proxy_set_header X-Real-IP $remote_addr;

in the nginx server block. I'm guessing the issue is with nginx.conf as the uWebSockets app works great, except I cannot access actual visitor $remote_addr in uWebSocket.js:
Here's my uWebSockets.js code:
...
open: (ws: uWS.WebSocket<unknown>) => {
    // helper
    function arrayBufferToString(buffer) {
        return decoderUtf8.decode(new Uint8Array(buffer))
    }
    const remoteAddr = ws.getRemoteAddressAsText()
    const ip = arrayBufferToString(remoteAddr)
    console.log('original IP: ', ip)
...

outputs one of:
"0000:0000:0000:0000:0000:ffff:7f00:0001"
"0000:0000:0000:0000:0000:0000:0000:0001"
"<IP_ADDRESS>"
Steps I've taken to solve
Searched other questions and threads on this forum and StackExchange, which led me to add extra proxy_set_header in my nginx.conf file. nginx access.log shows $remote_addr with correct value. Here's nginx.conf:
map $http_upgrade $connection_upgrade {
	default upgrade;
	#'' close;
}

upstream websocket {
	server localhost:4000;
}

server {
	listen 8443 ssl;

	location / {
		proxy_pass http://websocket;
                proxy_http_version 1.1;

                # set request headers upstream
		proxy_set_header Upgrade $http_upgrade;
		proxy_set_header Connection $connection_upgrade;
		proxy_set_header Host $host;
		proxy_connect_timeout 300;
		proxy_send_timeout 300;
   		proxy_read_timeout 300;
		proxy_socket_keepalive on;
                proxy_set_header X-Real-IP $remote_addr;

                # tried various things from other questions in uWebSocket forum and stackexchange
                proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
                proxy_set_header X-Forwarded-Proto $scheme;
                proxy_set_header X-Forwarded-Host $remote_addr;
                proxy_set_header X-Forwarded_Port $remote_port;
	}
}

Get the X-Real-IP header in the upgrade ws event and attach that IP to your ws data Upgrade.js#L21

Thanks @e3dio:
I did look at Upgrade.js in the examples, but assumed upgrade.js example was only if I did nginx proxy_pass but without nginx requesting the upgrade...anyway I added the .upgrade section to my nodejs code, and for the time being kept identical my nginx.conf as above, but no output to that header.
I'm clearly not requesting the X-Real-IP header in the correct way, or perhaps I need to attach it before I leave .upgrade, but as upgrade event only accepts 5 arguments and all are used in the example, I assumed I have to getHeader in the first object. upgrade.js doesn't include an example of retrieving common header names like x-real-ip or x-forwarded-for. Here is what I tried, no errors but blank output for ws.xRealIp in open() event:
upgrade: (res, req, context) => {
      res.upgrade(
        // upgrade to websocket
        {
          myData: req.getUrl(), /* First argument is UserData (see WebSocket.getUserData()) */
          xRealIp: req.getHeader('x-real-ip'),
          ip: res.getRemoteAddressAsText(),
        }, // 1st argument sets which properties to pass to ws object, in this case ip address
        req.getHeader('sec-websocket-key'),
        req.getHeader('sec-websocket-protocol'),
        req.getHeader('sec-websocket-extensions'), // 3 headers are used to setup websocket
        context // also used to setup websocket
      )
    },
    open: (ws: uWS.WebSocket<unknown>) => {
      const funcName = ' .App().ws/chat:open'
      function arrayBufferToString(buffer) {
        return decoderUtf8.decode(new Uint8Array(buffer))
      }

      console.log('ws.xRealIp:', ws.xRealIp) // outputs blank
      console.log('ws.xRealIp:', arrayBufferToString(ws.xRealIp)) // outputs blank

      const remoteAddr = ws.getRemoteAddressAsText()
      const ip = arrayBufferToString(remoteAddr)
      console.log('original IP: ', ip) // outputs <IP_ADDRESS> or similar

Is there a reason ws.getRemoteAddressAsText() doesn't show the x-real-ip header?

ws.getRemoteAddressAsText() shows IP of your nginx server, because that is what is connecting to your Node server. The remote IP address of client connected to your nginx is passed in the X-Real-IP header

@e3dio thanks. I see you are a main contributor, thanks again for being part of an outstanding project. Would it help other users do you think if i updated upgrade.js, through a pull request, to include these header retrievals, or do you think this thread is good enough and I'll mark it as closed?


Standardize how we access unique target values for classification problems

Right now, we access unique target values for classification problems in several ways:

list(ww.init_series(np.unique(y))) (classification_pipeline.py)
unique_labels (confusion_matrix)
LabelBinarizer / np.unique in roc_curve (slightly different than label encoding)

It could be helpful to standardize how we encode and decode targets pre and post fit time. This issue tracks finding places where we encode/decode and seeing how we could standardize this process.
Note that in some cases, we might encode/decode outside of the context of a pipeline (such as confusion_matrix), but it could still be helpful to consolidate our implementation to fewer methods if possible!

@angela97lin this is an excellent story.  Whenever you see things like this, feel free to keep submitting them.  Is it at all worth getting people together to figure out where other occurrences happen?  Or do you feel like you've captured most of them here.

@chukarsten
It could be worth a quick discussion in case I've missed any, but otherwise I think this covers a fair bit of ground to tackle. I think it's also interesting to think about how we could prevent this from deviating again--obviously these different code snippets were probably added by different people at different times. Having a discussion could be helpful not only to see if we've missed any here, but bring more awareness to the team so that if anyone needs this capability in the future, they're aware of what to do!


HBASE-23702  hbase.hstore.flusher.count setting to 0 breaks HMaster

Test the precommit job for branch-2.2.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 57s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

-0 :warning:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ branch-2.2 Compile Tests _

+1 :green_heart:
mvninstall
5m 24s
branch-2.2 passed

+1 :green_heart:
compile
0m 56s
branch-2.2 passed

+1 :green_heart:
checkstyle
1m 20s
branch-2.2 passed

+1 :green_heart:
shadedjars
4m  9s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 38s
branch-2.2 passed

+0 :ok:
spotbugs
3m 26s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
3m 24s
branch-2.2 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
5m  4s
the patch passed

+1 :green_heart:
compile
0m 58s
the patch passed

+1 :green_heart:
javac
0m 58s
the patch passed

+1 :green_heart:
checkstyle
1m 23s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
shadedjars
4m 15s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
hadoopcheck
26m  3s
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 2.10.0 or 3.1.2 3.2.1.

+1 :green_heart:
javadoc
0m 36s
the patch passed

+1 :green_heart:
findbugs
3m 29s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
165m 31s
hbase-server in the patch passed.

+1 :green_heart:
asflicense
0m 37s
The patch does not generate ASF License warnings.

232m  1s

Subsystem
Report/Notes

Docker
Client=19.03.8 Server=19.03.8 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1703/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1703

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux dd19d2b42566 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-slave/workspace/Base-PreCommit-GitHub-PR_PR-1703/out/precommit/personality/provided.sh

git revision
branch-2.2 / 0b4f272f2d

Default Java
1.8.0_181

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1703/1/testReport/

Max. process+thread count
4420 (vs. ulimit of 10000)

modules
C: hbase-server U: hbase-server

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1703/1/console

versions
git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 49s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

-0 :warning:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ branch-2.2 Compile Tests _

+1 :green_heart:
mvninstall
5m 27s
branch-2.2 passed

+1 :green_heart:
compile
0m 58s
branch-2.2 passed

+1 :green_heart:
checkstyle
1m 20s
branch-2.2 passed

+1 :green_heart:
shadedjars
4m 10s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 37s
branch-2.2 passed

+0 :ok:
spotbugs
3m 24s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
3m 22s
branch-2.2 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 51s
the patch passed

+1 :green_heart:
compile
0m 58s
the patch passed

+1 :green_heart:
javac
0m 58s
the patch passed

+1 :green_heart:
checkstyle
1m 18s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
shadedjars
4m 11s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
hadoopcheck
26m  0s
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 2.10.0 or 3.1.2 3.2.1.

+1 :green_heart:
javadoc
0m 35s
the patch passed

+1 :green_heart:
findbugs
3m 19s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
164m 31s
hbase-server in the patch passed.

+1 :green_heart:
asflicense
0m 42s
The patch does not generate ASF License warnings.

229m 28s

Subsystem
Report/Notes

Docker
Client=19.03.8 Server=19.03.8 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1703/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1703

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux 67430cdc8022 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-slave/workspace/Base-PreCommit-GitHub-PR_PR-1703/out/precommit/personality/provided.sh

git revision
branch-2.2 / b0c46bc74d

Default Java
1.8.0_181

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1703/2/testReport/

Max. process+thread count
4464 (vs. ulimit of 10000)

modules
C: hbase-server U: hbase-server

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1703/2/console

versions
git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.


🪲 Fixed NavBar overflow on medium screen (md)

flex-row on medium screen size(md) led to Navigation Bar overflow... hence changed updated the code such that Flex-row applies only on large screen size (lg)!
Before:

After:

On Large Screen:


Add DWP to the list of publishing departments for the ESI Funds finder

Trello: https://trello.com/c/Gwnujz5D/175-add-dwp-to-european-structural-and-investment-fund-esif-finder.

TIL that GitHub now has an "Update branch" button on the PR when a branch is out of date with master, which inserts a merge commit. :disappointed:


Strange behavior of ODE solving toy model

System information

OS Platform and Distribution Windows 10
TensorFlow installed from source TensorFlow 2.2.0
Python version: 3.8
CUDA/cuDNN version: 10.1

import numpy as np
import tensorflow as tf
from tensorflow import keras
from tensorflow.keras import layers
import matplotlib.pyplot as plt

N = 1000
x = tf.convert_to_tensor(np.linspace(0, 1, num=N), dtype=tf.float32)

with tf.GradientTape() as t:
    x_inp = keras.Input(shape=(1,), name='indvar')
    t.watch(x_inp)
    dense = layers.Dense(100, activation='tanh')(x_inp)
    dense = layers.Dense(100, activation='tanh')(dense)
    dense = layers.Dense(100, activation='tanh')(dense)
    dense = layers.Dense(100, activation='tanh')(dense)
    dense = layers.Dense(100, activation='tanh')(dense)
    dense = layers.Dense(100, activation='tanh')(dense)
    dense = layers.Dense(100, activation='tanh')(dense)
    yhat = layers.Dense(1, name='y')(dense)
    dyhat = t.gradient(yhat, x_inp)
    model = keras.Model(inputs=x_inp, outputs=[yhat, dyhat])
    opt=tf.keras.optimizers.Adam()
    model.compile(optimizer=opt, loss='MSE')
    model.summary()
    model.fit(x=x, y=[x, np.sin(10*x)], epochs=10000, batch_size=N)
    y, dy = model(x)
    plt.plot(x, np.array(y))
    plt.plot(x, np.array(dy))
    plt.show()

I try to make model output y=x, while y'=sin(10x). Surely this is impossible so the expected behavior of the model is NOT converge. However model converges and have both y=x and y'=sin(10x) satisfied, however it is impossible.

@yarik1988
I tried reproducing the issue but i am facing the below issue.Please,see in the screenshot. Is this the expected behavior

No, in my PC it works well. I found even smaller toy example that shows the bug
N = 1000
x = np.linspace(0, 1, num=N)

with tf.GradientTape() as t:
    x_inp = keras.Input(shape=(1,), name='indvar')
    t.watch(x_inp)
    reg=tf.keras.regularizers.l2(l=0.01)
    dense = layers.Dense(100, activation='tanh', kernel_regularizer=reg)(x_inp)
    for i in range(2):
        dense = layers.Dense(100, activation='tanh', kernel_regularizer=reg)(dense)
    yhat = layers.Dense(1, name='y')(dense)
    dyhat = t.gradient(yhat, x_inp)
    model = keras.Model(inputs=x_inp, outputs=[yhat, dyhat])
    opt=tf.keras.optimizers.Adam()
    model.compile(optimizer=opt, loss=[lambda y, y_act: 0,'MSE'])
    model.summary()
    #model.fit(x=x, y=[x, x*x], epochs=1, batch_size=N)  #with this line,  numerical derivative will differ from analytical
    y, dy = model(x)
    dy_num = np.gradient(np.array(y).squeeze(), x)
    plt.plot(x, y)
    plt.plot(x, dy)
    plt.plot(x, dy_num)
    plt.legend(['y', 'dy', 'dynum'])
    plt.show()

@yarik1988
I tried to reproduce the issue in colab .Please, find the gist here. Is this the expected behavior.Thanks!

@yarik1988
I tried to reproduce the issue in colab .Please, find the gist here. Is this the expected behavior.Thanks!

From the mathematical point of view this behavior is not neither logical neither expected. I created gist of my another example that shows the problem
https://colab.research.google.com/gist/yarik1988/92890b0d9a690773f2c1204d348f8478/untitled0.ipynb#scrollTo=JOB6TP8OuLmw

However with this line commented
model.fit(x=x, y=[x, x*x], epochs=1, batch_size=N)
numerical derivative coincides with analytical one, which is logical.

While trying to reproduce your issue in Tf Nightly faced some error, please find the gist here. Thanks!

I was able to reproduce your issue in TF v2.7 and tf-nightly (2.8.0-dev20211124) faced some error, please find the gist here. Thanks!


Call Graph View Proposal

Issue Type: feature-request
TLDR; Requesting a view/appearance that is an editable call graph.
Motivation
The way we read code is much different from the way we write code.
Allow me to explain this through an example of a simple CRUD application implemented as a Java Spring Boot Application.
A Java Spring Boot application is typically structured into classes such as:

Controllers: handle incoming requests and route them to the appropriate service.
Services: implement the business logic of the application.
Data Access Objects (DAOs): responsible for interacting with the database.

We can say that code is written “horizontally”.

However, we tend to read code “vertically” - one endpoint at a time much like the sequence in which it would run. We pick an endpoint and we start to read the controller, then the service, and finally the repository.

By making it easier to read code vertically, we can help developers cut the clutter and focus on the problem at hand.
Files are for Compilers, Mind Maps are for Humans
We write code in “linear” files because that is how the compiler needs code to be.
However, if that was the only thing we considered, we would be writing entire modules in a single file that is one compilation unit.

What we rather end up doing is creating a boatload of files arranged into modules that we think is best way to put them. Hence, coming to some sort of compromise between what the compiler wants and what the developer can work with. We have all encountered the challenge of finding what we need within an open-source project or legacy codebase.
This can be a tedious and time-consuming process, especially when trying to understand the interconnections between various pieces of code. It would be beneficial if we could visualize the project in a more intuitive manner, such as a flowchart of procedures.
This type of representation would allow us to easily "zoom in" on the details we need, without getting lost in the overwhelming amount of code presented in a traditional file-based view.

An IDE that acts as an abstraction of files can help developers focus on writing logic instead of worrying about how to organise their code into files.
By providing a visual representation of the code, such as a call graph, developers can more easily navigate and understand the codebase, allowing them to work faster and with fewer distractions.
This can ultimately lead to more efficient development and higher-quality code.
What Exists
Some of the existing solutions to the problem of code navigation

Breadcrumbs and Outline Views - allowing quickly switching between blocks of code
GoTo References and GoTo Implementation

Call Graphs can be generated in IDEs such as Visual Studio, Oracle Solaris Studio and IntelliJ (through plugins). I am not really sure why we are yet to see one such extension for VSCode.
What Does Not Exist
A call graph view that is more than a static visualisation.
Rather, a call graph view that does not bring the words "rather sad" to the mind when thinking about it.
Focusing on what Matters
In modern programming languages - such as Go, the basic unit of code are not files but declarations

structures
interfaces
functions

Hence, arrangement in files is an indirection that is rather bring friction to both reading and writing flows.
Implementation is an objective decision. Deciding where to put the function is highly subjective.
If we can define what matters to a programmer when one is reading or writing code, everything else is clutter. Functions that appear close to the each other in a file, but are never read together are definitely the big offenders.

What the New Appearance Should Look Like

The Foreseeable Challenges

Performance: call graphs can be absolutely massive in large codebases. The performance bottleneck was a major factor why the call graph features could not be used often in past.
(Implementation Dependent) Embedding a Graph Database: The most logical way to generate a call graph and to use it optimally would be store each function as a node in an embedded graph database with edges representing the function calls. When the code needs to compiled, we can update the files for the nodes that change, creating a sort of build step.

Additional benefits

The call graph that would be generated and visible to the user can be taken advantage of to provide the most optimum context to generative AI models such as Copilot.
Arrange Mode can be extended to allow defining service-boundaries to allow breaking monoliths into microservices. The function calls across the service-boundaries shall be replaced by auto generated GRPC/Protobuf code.

This reminds me of the old "code bubbles" demo: https://cs.brown.edu/~spr/codebubbles/
Cool ideas but I don't think this is a good fit for a built-in vscode feature. Is that what you are asking for?
My suggestion is to create an extension that contributes a custom webview editor that implements what you are looking for.


lncli listpayments should show fee in msat

It shows fee in saotshi now, and apparently rounds it down. It should be in msat.

Agreed it should list msat instead of sat. It's stored in the database (rightfully) as sat, so just the protos need to be modified to display msat.

I can grab this. Do you want to retain 'fee' field for compatibility or is it fine to remove it? I guess it is fine to remove it (mark as reserved tag in proto), as lnd isn't release yet and shouldn't really used for anything? ;D

I think we can first add the new field, and annotate the old one as deprecated just as we did recently in the Hop proto.

Ok. I just was curious about your policy on compatibility at this moment. Will do.

I see no impact for marking the old field as deprecated.  It is not even indicated as such in the swagger.json.

I know marking the field deprecated is fine. I was thinking to just remove it in the same commit completely, or even reuse the same tag name as "fee" field. Should be "fine" as lnd is not released yet. That is what I meant by "policy on compatibility at this moment".

My comment was about @Roasbeef suggestion to make these fields marked as deprecated in the rpc.proto file. I see no impact at all for such marking and fail to understand how new code can be aware that these fields are deprecated. Maybe we should switch to a newer protoc version  for that to have an impact?
@baryluk see this:

The exported API's are not yet stable, so be warned: they may change drastically in the near future.

Well lnd has been "released". However we can do it in two phases perhaps: first deprecate it, then warn that it'll be removed, then just remove it all together.

Wanted to add to this discussion that the value field in listpayments should also display in mSat, or at least include a separate value_msat field. I was able to pay a 1 mSat invoice using LND by manually editing queryroute data and passing that edited route to sendtoroutes (thanks to @alexbosworth in slack for help on this)
yet in both listchannels and listpayments the values are displayed as 0


feedback for modelVersion.schema.tpl.json

@apdavison please have a look at the modelVersion.schema.tpl.json
My question:

Is it correct that we agreed to link to a ContentType for the model format?

yes, I think that makes the most sense

@apdavison Perfect! Thank you for confirming again. The rest looks okay or do you miss something?

@lzehl I've now started planning in detail the migration of model/model version metadata to openMINDS. I think the following changes are needed:

move studyTarget, scope, and abstractionLevel from ModelVersion to Model. The reason is that if you change any of these things you have a new model, not just a new version
for inputData and outputData, we need to allow links to FileRepository, FileBundle and FileInstance, in addition to DigitalIdentifier.

There is also the question of more fine-grained accessibility control, but I'll discuss that with you and Oli separately.

@lzehl I've now started planning in detail the migration of model/model version metadata to openMINDS. I think the following changes are needed:

move studyTarget, scope, and abstractionLevel from ModelVersion to Model. The reason is that if you change any of these things you have a new model, not just a new version
for inputData and outputData, we need to allow links to FileRepository, FileBundle and FileInstance, in addition to DigitalIdentifier.

There is also the question of more fine-grained accessibility control, but I'll discuss that with you and Oli separately.

@apdavison thanks for providing feedback on the model schemas.
To your first point: good to know, I'll change this asap.
The second point I would like to discuss. In principle the idea was the Digital Identifier can be also used to link to a FileRepository, FileBundle, and FileInstance (e.g. via the KG identifier for internally stored data or URI for externally stored data), but I can see that this might be cumbersome. Could you describe a short use case for me why the direct linkage would be better?
For the fine-grained accessibility control: please get in touch for setting up a discussion with me and Oli. I'm not sure what you mean here, but the controlled terms for the accessibility of a research product can be of course extended (it currently only covers the ones we decided for the dataset cases). If this is what you mean, please feel free to make suggestions for extending the controlled term list.

@apdavison thanks for providing feedback on the model schemas.
To your first point: good to know, I'll change this asap.
The second point I would like to discuss. In principle the idea was the Digital Identifier can be also used to link to a FileRepository, FileBundle, and FileInstance (e.g. via the KG identifier for internally stored data or URI for externally stored data), but I can see that this might be cumbersome. Could you describe a short use case for me why the direct linkage would be better?
For the fine-grained accessibility control: please get in touch for setting up a discussion with me and Oli. I'm not sure what you mean here, but the controlled terms for the accessibility of a research product can be of course extended (it currently only covers the ones we decided for the dataset cases). If this is what you mean, please feel free to make suggestions for extending the controlled term list.

Hi @Lyuba, sorry it's taken me so long to reply. For the links via DigitalIdentifier, my thinking is that for internal objects we should use the specific types, and keep DigitalIdentifier for links to external resources. Otherwise everything could just link via DigitalIdentifiers. For inputData, for example, not all objects with a DigitalIdentifier would be valid, only things that represent actual data files.

@apdavison no worries. I know you are busy ;) I'm okay with adding the direct linkage possibility to the file schemas if this is too cumbersome otherwise.
The reason why we first decided for only going over the DigitalIdentifier instead was that this would push users to go with the official, citable DigitalIdentifier or at least provide a "howToCite" possibility. We also thought that would make it easier within the KG to identify which file (repository, bundle, instance) entries belong to which ResearchProductVersion: if the resource belongs to your research product you reference the file (repository, bundle, instance) entry, if the resource belongs to another product use the digital identifier.
What do you think? Does this initial thought makes sense or is it unnecessary?
@apdavison & @olinux it would be good to hear both of your opinions again on this before we decide which way to follow. I'm open for both approaches.

@apdavison the "inputData" can reference now to
"https://openminds.ebrains.eu/core/DOI",
"https://openminds.ebrains.eu/core/File",
"https://openminds.ebrains.eu/core/FileBundle",
"https://openminds.ebrains.eu/core/URL"
I think this resolves the open points of this issue, correct? please let me know if we should add FileRepository to the list as well. Note that this requires strict curation rules: only point to FileRepo if input repository does not have a DOI.

it's probably ok without FileRepository, so all is good.

perfect. I'm closing the issue then and we can revise adding the FileRepo when we encounter a use case where this would be absolutely required.


Simplify --goto's usage

Currently --goto uses the following form:
code file.ext:10:2 --goto # opens "file.ext" at 10th line, 2nd column
code file.ext:10:2 -g     # opens "file.ext" at 10th line, 2nd column

code file.ext:10:2        # opens the file "file.ext:10:2"

This is a little difficult to explain this in the usage and is a little different to how CLI args are handled on unix-like systems. I propose this instead:
code file.ext -g 10:2     # opens "file.ext" at 10th line, 2nd column
code file.ext --goto=10:2 # opens "file.ext" at 10th line, 2nd column

This follows the -g <value>, --goto=<value> standard and also allows -g/--goto to be documented in isolation of the path arg.
This also brings it in line with how the --locale arg seems to work currently.

@bpasero thoughts?

I like the suggestion but it would seriously break all clients that expect the current behaviour including Unity plugins that are written so that you can easily open a file from Unity in VS Code. I would not want to make this change easily as this format has been out since almost a year without complaint.

-1 for breaking existing clients.

@bpasero Would introducing the new way, deprecating the old way and notifying known consumers not be a better route? Otherwise this is technical debt as there is an arg that has inconsistent usage others like --locale.

I am not so worried bout this as long as a user can find how to use this feature with "--help".

And we would also loose a feature we have now: you can open multiple files with their line/col information.


eclint has reported vulnerabilities

npm audit reports tons of vulnerabilities via gulp-reporter (why is anything gulp-related a runtime dep?).
Can gulp-reporter (and all the gulp deps) be moved from deps to devDeps, since you only use it in tests?

npm audit reports tons of vulnerabilities via gulp-reporter

Witch version of eclint you are using? I remember the problem was fixed.

why is anything gulp-related a runtime dep?

eclint is a gulp plugin but without name prefix gulp-

Can gulp-reporter (and all the gulp deps) be moved from deps to devDeps,

It will use for CLI to show report to console.

I'm using v2.7.0, and the issues are still reported by npm audit.
Gulp seems like a very heavyweight dependency just to output stuff to the console :-/

I'm using v2.7.0, and the issues are still reported by npm audit.

Can you post report to here?

Gulp seems like a very heavyweight dependency
No. we just use vinyl-fs & vinyl

hmm - i removed node_modules and then reinstalled and made a package-lock, and still saw the errors - but now when i try, the issues are indeed fixed. My apologies for the false positive!


Fix docker container race conditions

The web container could start running before the database/celery broker was launched. Add wait checks in docker-entrypoint

Codecov Report

Merging #420 into master will not change coverage.

@@          Coverage Diff           @@
##           master    #420   +/-   ##
======================================
  Coverage    77.8%   77.8%
======================================
  Files          28      28
  Lines        2365    2365
  Branches      150     150
======================================
  Hits         1840    1840
  Misses        479     479
  Partials       46      46

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 71b4fb4...cba2c77. Read the comment docs.


Source of bin.exe and svhost.exe

Greetings
You didnt share stub source and svhost.exe??
Using as resource file.

this is stub source => https://github.com/moneermasoud/CosaNostra-HTTP-Botnet/tree/master/stub/keylogger

What is my password login?


npm install fails when react-native running

I took js files for some "pages" I'm making from a project that uses react-native-action-bar.
I get:  Unable to resolve "react-native-action-bar" from ..."./C:\\mypath\\pages\\Mypage.js
I then run npm i react-native-action-bar  and get
 `--<EMAIL_ADDRESS> `--<EMAIL_ADDRESS>(git://github.com/react-navigation/react-native-tab-view.git#36ebd834d78b841fc19778c966465d02fd1213bb)
.
.

Error: EPERM: operation not permitted, scandir ... cannot rename ...
Turns out it's not a react-native-action-bar issue...  So I'll give an answer and close this, but wanted to help others who may encounter this kind of problem.

Answers here: https://github.com/react-community/create-react-native-app/issues/191
The error message can be misleading. My problem was that I had a react-native-cli window open (with yarn start).  Closed it. Installed OK. Then ran the react-native-cli again, and all's ok now.


Add deprecation messages for old binding crates

Before the refactor of the Godot API bindings in 0.7.0, the binding code was provided via multiple "category-crates" for parts of the bindings. These crates are no longer in the source tree here, but they are still existing on crates.io.
We should probably add a deprecation warning in the README and/or add a badge visible on crates.io that the crates are no longer maintained.
@nical I don't have access to all those crates and I started to give @toasteater access to crates too. Maybe you can give him and me access to all those old binding crates?

I went through all gdnative-* grates and sent invites to @toasteater and you where you didn't have access already.

Thank you very much for the quick action! :)

Done!

That's nice, thank you! I do think having crates-io badges would have also been nice (maintenance = { status = "deprecated" }) but it's in the "sub-title" too, so it should be fine?

I didn't know about that option. Fortunately I can push another version for these crates relatively easily if necessary, since I kept the files used to push the current versions. Do you think it's worth it?

I think it's okay. It just shows a small badge next to the name in the search, with the description starting with (DEPRECATED) I think it's fine.


Return an empty slice when transaction is not found on a GetMempoolTxs request

Fixes #333

Codecov Report

Merging #337 into master will not change coverage by %.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           master     #337   +/-   ##
=======================================
  Coverage   54.21%   54.21%
=======================================
  Files         127      127
  Lines        7161     7161
=======================================
  Hits         3882     3882
  Misses       2773     2773
  Partials      506      506

Impacted Files
Coverage Δ

pkg/core/mempool/mempool.go
63.56% <0.00%> (ø)

Continue to review full report at Codecov.

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

Codecov Report

Merging #337 into master will not change coverage.
The diff coverage is 0%.

@@           Coverage Diff           @@
##           master     #337   +/-   ##
=======================================
  Coverage   54.21%   54.21%
=======================================
  Files         127      127
  Lines        7161     7161
=======================================
  Hits         3882     3882
  Misses       2773     2773
  Partials      506      506

Impacted Files
Coverage Δ

pkg/core/mempool/mempool.go
63.56% <0%> (ø)
:arrow_up:

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 b2fca5b...b7df717. Read the comment docs.


How to pass string slice flags

Hello, this is a dumb question but I found no mention in the doc on how a string slice should be defined when using the CLI.
I tried with mycli example --names name1,name2 but when I parse the flag values with ctx.StringSlice("names") I get a slice of one long string "name1,name2" instead of a slice with 2 elems.

hey @altafan , any chance to have a example for this ? :)

@maeglindeveloper it will be
--names name1 --names name2
@meatballhat Probably a good idea to add this to the docs.


Cannot update or view packages in preferences window: tunneling socket errors

I'm running OS X Mavericks with Atom 0.150.0 behind a corporate firewall. My (anonymized) .apmrc is as follows:
https-proxy = https://[user]:[pw]@[host]:[port]
http-proxy = http://[user]:[pw]@[host]:[port]
proxy = http://[user]:[pw]@[host]:[port]
strict-ssl = false

Fetching outdated packages and themes fails with this error message:
Request for package information failed: tunneling socket could not be established, cause=140735230071568:error:140770FC:SSL routines:SSL23_GET_SERVER_HELLO:unknown protocol:../deps/openssl/openssl/ssl/s23_clnt.c:795:

Fetching featured packages fails with this error message:
tunneling socket could not be established, cause=140735230071568:error:140770FC:SSL routines:SSL23_GET_SERVER_HELLO:unknown protocol:../deps/openssl/openssl/ssl/s23_clnt.c:795:

Googling and further searching doesn't seem to bring up anything immediately relevant and helpful. I'd really like to use Atom as my primary text editor, but I have no idea how to get past these issues. Thank you in advance for your time and help.

changing https to http worked for me too !! Thanks :+1:

Request for package information failed: tunneling socket could not be established, cause=140358160631680:error:140770FC:SSL routines:SSL23_GET_SERVER_HELLO:unknown protocol:../deps/openssl/openssl/ssl/s23_clnt.c:782:
(5 attempts) (ECONNRESET)


Fix golangci-lint 1.54.2 errors

Ignore this one as the aliasing is OK here:
pkg/cable/wireguard/getconnections.go:66:28: G601: Implicit memory aliasing in for loop. (gosec)
		if k, err := keyFromSpec(&con.Endpoint); err == nil {

pkg/netlink/fake/netlink.go:56:2: directive `//nolint:errorlint // The given error should not be wrapped.` is unused for linter "errorlint" (nolintlint)
	//nolint:errorlint // The given error should not be wrapped.

There are some linter errors.

There are some linter errors.

The first one needs an additional comment, and the second one will be fixed when Shipyard bumps to 1.54.2 (which means https://github.com/submariner-io/shipyard/pull/1344 needs to be merged before this).


ZonedDateTime allows creation of dates after max_year

When we truncate transitions for timezones, we should not allow dates in those timezones to be created for periods after that truncation.
PR for https://github.com/quinnj/TimeZones.jl/issues/17

@bcj can you rebase this on the the current master?

tada

alright, let's ignore that terrible thing I did that is now gone in the history. My tests pass locally, if CI succeeds I think we're good

I iterated a bit on the original design and added a bunch of additional tests to ensure that cutoff is working as expected.


fix compile error: '_srs_hybrid' was not declared in this scope

Codecov Report

Merging #2629 (431af2b) into 4.0release (46adcfb) will increase coverage by 0.15%.
The diff coverage is 59.05%.

@@              Coverage Diff               @@
##           4.0release    #2629      +/-   ##
==============================================
+ Coverage       59.38%   59.53%   +0.15%
==============================================
  Files             122      122
  Lines           51590    51715     +125
==============================================
+ Hits            30636    30788     +152
+ Misses          20954    20927      -27

Impacted Files
Coverage Δ

trunk/src/app/srs_app_http_api.cpp
0.00% <0.00%> (ø)

trunk/src/protocol/srs_http_stack.hpp
100.00% <ø> (ø)

trunk/src/app/srs_app_http_static.cpp
45.41% <51.08%> (+4.54%)
:arrow_up:

trunk/src/protocol/srs_http_stack.cpp
92.83% <66.66%> (-0.26%)
:arrow_down:

trunk/src/utest/srs_utest_http.cpp
99.73% <100.00%> (+<0.01%)
:arrow_up:

trunk/src/app/srs_app_config.cpp
67.74% <0.00%> (+0.03%)
:arrow_up:

trunk/src/app/srs_app_hybrid.cpp
10.07% <0.00%> (+1.43%)
:arrow_up:

trunk/src/app/srs_app_statistic.cpp
27.55% <0.00%> (+22.91%)
:arrow_up:

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 3167280...431af2b. Read the comment docs.


option to get the mail with timestamp

Please be aware that these issues will be closed if inactive for more then 14 days :)
Is there any option to get the mail with timestamp.
Since takes only date not time.
I need to retrieve the mail after some timestamp.
Any help would be appreciative.

Hi @kaviyarasansadasivam ,
please take closer look at the search section: https://github.com/Webklex/laravel-imap/blob/master/README.md#search-for-messages
As you can see, you can also provide a Carbon object as search value. You can create a new Carbon object from a timestamp - this way you can search for timestamps. But note that it isn't possible to search for hours/mins/seconds.. only "real" dates are supported by RFC1176 (
https://tools.ietf.org/html/rfc1176).
Best regards

Here is an example if your mail server is supporting datetime:
$timestamp = strtotime('17.08.2018 10:30');

/** @var \Webklex\IMAP\Folder $oFolder */
$oFolder = $oClient->getFolder('INBOX');
$date = Carbon::createFromTimestamp($timestamp);

/** @var \Webklex\IMAP\Support\MessageCollection $aMessage */
$aMessage = $oFolder->search()->where([
    ['SINCE', $date->format('d M y H:i')]
])->get();

An overview of all available search citeria can be found here: https://github.com/Webklex/laravel-imap#search-for-messages (scroll down to "Available search criteria").
If your Mailserver isn't supporting datetime, you could also filter the collection and drop all messages which don't match your ciriteria:
$timestamp = strtotime('17.08.2018 10:30');

/** @var \Webklex\IMAP\Support\MessageCollection $aMessage */
$aMessage = $aMessage->filter(function ($oMessage) use ($timestamp) {
    /** @var \Webklex\IMAP\Message $oMessage */
    if($oMessage->getDate() !== null) {
        return $oMessage->getDate()->timestamp > $timestamp;
    }

    return false;
})->values();

Best regards


Problem running on Raspberry Pi

Hi!
I'm trying the mujx/hakatime:latest-arm image, and have some trouble getting hakatime to work.
Running docker-compose -f docker-compose-deploy.yml up works fine. I had to add privileged: true, because I got an error saying that it couldn't read the the time.
I also had to change
    ports:
      - "<IP_ADDRESS>:8080:8080"

to
    ports:
      - "<IP_ADDRESS>:8080:8080"

to be able to open the UI on my computer (and not the Pi).
However, I cannot get past the login screen. First, I see POST http://<IP_ADDRESS>:8080/auth/refresh_token 400 (Bad Request) in the console log in Chrome. And when I try to register a new user, I see
haka_db    | 2020-09-22 20:09:55.632 UTC [44] ERROR:  relation "users" does not exist at character 16
haka_db    | 2020-09-22 20:09:55.632 UTC [44] STATEMENT:   SELECT * FROM users WHERE username = $1
hakatime   | [2020-09-22 20:09:55][hakatime.server][Error][8332e554f052][PID 1][ThreadId 14][hakatime-0.1.0-inplace-libhaka:Haka.Authentication src/Haka/Authentication.hs:205:9] OperationException "relation \"users\" does not exist"

in the application logs.
For completeness, this is how my now edited Docker compose file looks like:
version: "3"
services:
  server:
    container_name: hakatime
    image: mujx/hakatime:latest-arm
    privileged: true
    environment:
      # DB settings.
      HAKA_DB_HOST: haka_db
      HAKA_DB_PORT: 5432
      HAKA_DB_NAME: test
      HAKA_DB_PASS: test
      HAKA_DB_USER: test
      # Server settings.
      HAKA_CORS_URL: "http://<IP_ADDRESS>:8080"
      HAKA_PORT: 8080
      HAKA_SHIELDS_IO_URL: "https://img.shields.io"
      HAKA_ENABLE_REGISTRATION: "true" # Toggle after you've created your account.
      # Number of hours after which inactive browser sessions will expire (login required).
      HAKA_SESSION_EXPIRY: "24"
    ports:
      - "<IP_ADDRESS>:8080:8080"
  haka_db:
    container_name: haka_db
    image: postgres:11-alpine
    environment:
      POSTGRES_DB: test
      POSTGRES_PASSWORD: test
      POSTGRES_USER: test
    volumes:
      - ./docker/:/docker-entrypoint-initdb.d/
      - deploy_db_data:/var/lib/postgresql/data

volumes:
  deploy_db_data: {}

Never mind!
I found the (stupid) error. I didn't realize you had to clone the whole repo to start docker-compose...
I guess you can close the issue. Thanks for the cool project!

Thanks for trying hakatime!
The problem was that postgres wasn't initialized. You don't have to clone the repo, but you do need to apply the sql files in order to create the db schema.
Btw, you shouldn't need privileged: true to run the container. Could you provide the logs from this error?

Here's the logs without privileged:
pi@raspberrypi:~/hakatime $ docker-compose -f docker-compose-deploy.yml up
Starting hakatime ... done
Starting haka_db  ... done
Attaching to haka_db, hakatime
haka_db    |
haka_db    | PostgreSQL Database directory appears to contain a database; Skipping initialization
haka_db    |
haka_db    | 2020-09-22 20:49:04.075 UTC [1] LOG:  listening on IPv4 address "<IP_ADDRESS>", port 5432
haka_db    | 2020-09-22 20:49:04.075 UTC [1] LOG:  listening on IPv6 address "::", port 5432
haka_db    | 2020-09-22 20:49:04.093 UTC [1] LOG:  listening on Unix socket "/var/run/postgresql/.s.PGSQL.5432"
hakatime   | clock_gettime: Operation not permitted
haka_db    | 2020-09-22 20:49:04.166 UTC [21] LOG:  database system was shut down at 2020-09-22 20:48:58 UTC
haka_db    | 2020-09-22 20:49:04.233 UTC [1] LOG:  database system is ready to accept connections
hakatime exited with code 1
^CGracefully stopping... (press Ctrl+C again to force)
Stopping haka_db  ...
Killing haka_db   ... done
^C

I found out about this from this blog post: https://stackoverflow.com/questions/53748644/linux-syscall-clock-settime-fails-with-eperm-in-docker-container

👍  also needed to include privileged: true to get it work,
this error else would make the hakatime container fail:
clock_gettime: Operation not permitted

Thanks for the support on the Pi!

I got the instance up and running, though I can't get the register to work.  The SQL instance logs show me everything was created, and yes I have the enable registration flag to true in the docker-compose-deploy.yml

Do you have any errors in hakatime's logs?

hakatime   | <IP_ADDRESS> - - [02/Oct/2020:20:26:34 +0000] "POST /auth/refresh_token HTTP/1.1" 400 - "http://raspberrypi.main:8080/" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/85.0.4183.121 Safari/537.36"
hakatime   | <IP_ADDRESS> - - [02/Oct/2020:20:26:34 +0000] "GET /nunito-latin-400.4a62a00a.woff HTTP/1.1" 400 - "http://raspberrypi.main:8080/" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/85.0.4183.121 Safari/537.36"
hakatime   | <IP_ADDRESS> - - [02/Oct/2020:20:26:34 +0000] "GET /nunito-latin-400.3a5373d7.woff2 HTTP/1.1" 400 - "http://raspberrypi.main:8080/" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/85.0.4183.121 Safari/537.36"
hakatime   | <IP_ADDRESS> - - [02/Oct/2020:20:26:57 +0000] "POST /auth/register HTTP/1.1" 400 - "http://raspberrypi.main:8080/" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/85.0.4183.121 Safari/537.36"

Seems like a CORS issue, check the server response payload on your browser.
You can fix it by setting the environment variable HAKA_CORS_URL to your domain. In your case it should be http://raspberrypi.main:8080/

that was it.  Oddly enough, i had changed them all before having the raspberry pi build issues.  it's up and running now.

@FredrikMeyer @timvoet @christian-fei I was wondering if adding the SYS_TIME capability when running the container fixes the issue so you don't have to run as privileged. An example would be docker run --cap-add=SYS_TIME .. or replace privileged: true in the docker compose file with the following
cap_add:
  - SYS_TIME

Closing as resolved. Please re-open if that's not the case.

Closing as resolved. Please re-open if that's not the case.


Is there any way to edit existing pastes?

I would like to be able to edit existing pastes, is there a way to do that?

There's no way to edit existing pastes via the Pastebin API.


Add validator ID and remove profile pciture src field

Description
Closes:

https://github.com/elys-network/issues/issues/XXX
https://github.com/elys-network/issues/issues/XXX

What has Changed?
What specific problem were you aiming to address, and how did you successfully resolve it? If tests were not uploaded for this pull request or if coverage decreased, please provide an explanation for the change.

Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...

[ ] included the correct type prefix in the PR title
[ ] added ! to the type prefix if API or client breaking change
[ ] targeted the correct branch (see PR Targeting)
[ ] provided a link to the relevant issue or specification
[ ] followed the guidelines for building modules
[ ] included the necessary unit and integration tests
[ ] included comments for documenting Go code
[ ] updated the relevant documentation or specification
[ ] reviewed "Files changed" and left comments if necessary
[ ] confirmed all CI checks have passed

Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
I have...

[ ] confirmed the correct type prefix in the PR title
[ ] confirmed ! in the type prefix if API or client breaking change
[ ] confirmed all author checklist items have been addressed
[ ] reviewed state machine logic
[ ] reviewed API design and naming
[ ] reviewed documentation is accurate
[ ] reviewed tests and test coverage
[ ] manually tested (if applicable)

Deployment Notes
Are there any specific considerations to take into account when deploying these changes? This may include new dependencies, scripts that need to be executed, or any aspects that can only be evaluated in a deployed environment.
Screenshots and Videos
Please provide any relevant before and after screenshots by uploading them here. Additionally, demo videos can be highly beneficial in demonstrating the process.

Trello ticket: https://trello.com/c/I3DIDUZ7/58-display-validator-logos-correctly-on-webapp-using-identity-field-keybase-pgp

from @Wgil :
identity is empty for the local node:
Request:
http://<IP_ADDRESS>:1317/cosmos/staking/v1beta1/validators
Response:
{
  "validators": [
    {
      "operator_address": "elysvaloper1r927lf4duk3mg84azfyt8at74szy5hgpsx2f76",
      "consensus_pubkey": {
        "@type": "/cosmos.crypto.ed25519.PubKey",
        "key": "gYVxDbsyhjLHleZAC2HIvalTWa1JBCKsW27Q1TRYIdw="
      },
      "jailed": false,
      "status": "BOND_STATUS_BONDED",
      "tokens": "50000000000000",
      "delegator_shares": "50000000000000.000000000000000000",
      "description": {
        "moniker": "node",
        "identity": "",
        "website": "",
        "security_contact": "",
        "details": ""
      },
      "unbonding_height": "0",
      "unbonding_time": "1970-01-01T00:00:00Z",
      "commission": {
        "commission_rates": {
          "rate": "0.100000000000000000",
          "max_rate": "0.200000000000000000",
          "max_change_rate": "0.010000000000000000"
        },
        "update_time": "2024-02-22T18:24:08.990576Z"
      },
      "min_self_delegation": "1",
      "unbonding_on_hold_ref_count": "0",
      "unbonding_ids": []
    }
  ],
  "pagination": {
    "next_key": null,
    "total": "1"
  }
}


Fix abstract validation error and SpdxOrganization URI validation errors

I got the validation for the spdx-spec example to work fully with some changes to the spec parser plus the change to the model in https://github.com/spdx/spdx-3-model/pull/937
I thought I would share the changes to the spec-parser as a pull request - feel free to re-implement.
This pull request fixes 2 issues in 2 commits.
The first commit implements the abstract class validation as suggested by @JPEWdev  - https://github.com/spdx/spec-parser/issues/165#issuecomment-2529816085
This fixes #165
The second commit changes the URI for createdBys in the individuals from <https://spdx.org/> to ns1:SpdxOrganization
This fixes #166
Note that the order of the constraints are not the same as suggested - here is an example of what is output:
 sh:property [ sh:message "https://spdx.org/rdf/3.0.1/terms/Core/ElementCollection is an abstract class and should not be instantiated directly. Instantiate a subclass instead."@en ;
            sh:not [ sh:hasValue ns1:ElementCollection ] ;
            sh:path rdf:type ],

Merging this to allow validation.
Although in the future some of the magic info should be moved to model repo and not in the RDF generator.


Add user input history function

In previous versions of TeachWhat, it is not possible to retrieve past user's inputs.
With this new feature, using the arrow keys will allow a user to retrieve past user's inputs, similar to how a terminal works.
Fixes #103

Codecov Report

Merging #150 (87aeff8) into master (e1741dd) will decrease coverage by 1.97%.
The diff coverage is 6.71%.

@@             Coverage Diff              @@
##             master     #150      +/-   ##
============================================
- Coverage     51.14%   49.17%   -1.98%
- Complexity      528      531       +3
============================================
  Files           115      119       +4
  Lines          2362     2477     +115
  Branches        259      269      +10
============================================
+ Hits           1208     1218      +10
- Misses         1074     1178     +104
- Partials         80       81       +1

Impacted Files
Coverage Δ

...address/logic/inputhistory/InputHistoryResult.java
0.00% <0.00%> (ø)

...du/address/logic/inputhistory/UserInputString.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/ui/CommandBox.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/ui/MainWindow.java
0.00% <0.00%> (ø)

...edu/address/logic/inputhistory/UserInputIndex.java
15.78% <15.78%> (ø)

...u/address/logic/inputhistory/UserInputHistory.java
17.14% <17.14%> (ø)

...rc/main/java/seedu/address/logic/LogicManager.java
58.62% <20.00%> (-8.05%)
:arrow_down:

...va/seedu/address/logic/parser/TeachWhatParser.java
60.00% <0.00%> (-2.50%)
:arrow_down:

...java/seedu/address/logic/commands/EditCommand.java

.../seedu/address/logic/parser/EditCommandParser.java

... and 2 more

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 e1741dd...87aeff8. Read the comment docs.


In some scenarios, opening Atom on Linux results in 100% CPU usage of a core

When some Linux users open Atom via .desktop files or via the gnome launcher, they experience 100% CPU usage due to the way we are invoking the shell to patch the user's environment
This does not happen for all Linux users
This does seem to happen consistently for users on Arch Linux
The shell is invoked with detached: false and using the -i flag for an interactive shell

The intent of shelling out to get the environment is to run the same shell initialization scripts that are run when a user opens a terminal window and invokes atom from there
Removing the -i flag is not the solution, because it doesn't result in the desired behavior; it does result in the shell exiting as expected without consuming 100% of a CPU core
Adding detached: true to the options used for child_process.spawn does fix the issue and retains the desired behavior, but it requires additional code to manage the process lifecycle if Atom exits or an Atom window is reloaded while the shell process is still running

The below is a copy of https://github.com/atom/atom/issues/13084#issuecomment-262507036, included here to open up a new issue:
@phw says:
I debugged this a bit and can reproduce it with the following test script (extracted from [src/update-process-env.js](https://github.com/atom/atom/blob/1b2d2f029945c58cf4cd474ef1423856b35cf26c/src/update-process-env.js)):

```javascript
const childProcess = require('child_process')
const cmd = 'command env'
const shell = process.env.SHELL  // zsh or bash doesn't matter, behavior is the same
// const shell = '/bin/bash'

const child = childProcess.spawn(shell, ['-ilc', cmd], {
   encoding: 'utf8',
  //  detached: true,
  stdio: ['ignore', 'pipe', process.stderr]
})
child.on('error', (e) => {
  console.log('error', e)
})
child.stdout.on('data', (data) => {
  console.log(data.toString('utf8'))
})
child.on('close', (code, signal) => {
  console.log('close', code, signal)
})
```

Launching this directly from a shell works, even if I set an empty environment like so:

    env -i SHELL=/bin/bash node spawn.js

Launching the same from a desktop file in Gnome 3.22 leads to the shell task endlessly consuming 100% of CPU. I used the following desktop file:

```
[Desktop Entry]
Name=Spawn Node Test
Exec=env SHELL=/bin/bash node /home/phw/devel/spawn.js
Icon=atom
Type=Application
```

Two things help to solve this issue:

a) Enable `detached: true` when spawning the child process
b) Remove the `-i` flag from the shell call

I think b) would actually be correct anyway, since this is in no way an interactive shell.

EDIT: Just to be clear: I still don't know exactly *why*  this is happening, I just know the issue can be circumvented removing the `-i` flag.

Maybe some other interesting fact: If Atom is launched via the desktop file and consequently spawns this non-terminating shell process, some actions (New file, About, maybe others) do not react. That is you trigger the action (via menu, keyboard shortcut, command palette, doesn't matter) but there is no action.
However, once you kill the spawned shell process the triggered actions get executed. E.g. you hit two times Ctrl+N and also start "Application: About" via command palette, nothing visible happens. You kill the spawned shell process and suddenly two new file editors and the about window are opened.
Maybe somebody has an idea how this is related.

It is happening to me on my system, when I click on "restart" after package update (it was update of nucleus-dark-ui). Fedora 25, Atom 1.12.5 installed from rpm downloaded directly from atom.io.

@phw please try building from source (use the master branch) and let me know if this has resolved your issue.

@phw or others affected by this: in order to release a hotfix, we want one of you to build from source on Linux and verify the issue is fixed. Could you please assist us in doing this?

I'd love to test this, but I likely won't be able to do so before Monday.

Just built from source and seems to have fixed the problem in my case (running ubuntu 16.04 LTS).

Would like to try but i'm unable to compile atom. I think some nodejs problem with fedora but tried with packagemanger and nodejs installed over nvm. Dosn't work for me. Sry. I don't get this node stuff.

Hello,
Just to make sure this is the same bug, I get the same behavior, with the same bash -ilc command env trapped in an infinite signal handling loop (see also), with bash 4.4:
strace: Process 19539 attached
--- SIGTTIN {si_signo=SIGTTIN, si_code=SI_USER, si_pid=19539, si_uid=1000} ---
rt_sigaction(SIGTTIN, {SIG_IGN, [], SA_RESTORER, 0x7fba456ab860}, {SIG_DFL, [], SA_RESTORER, 0x7fba456ab860}, 8) = 0
ioctl(255, TIOCGPGRP, [19683])          = 0
rt_sigaction(SIGTTIN, {SIG_DFL, [], SA_RESTORER, 0x7fba456ab860}, {SIG_IGN, [], SA_RESTORER, 0x7fba456ab860}, 8) = 0
kill(0, SIGTTIN)                        = 0
...

If I try to strace the start of atom strace -f -e execve -s 300 -o /tmp/atom.trace atom . in my $HOME, atom starts normally (maybe because using strace changes the process group structure).

@joefitzgerald I finally was able to test this, looking very good 👍 I tried with latest master, starting Atom from the .desktop file works as expected. Just to be sure I also installed the revision just before the merge of your fix (03cc5033b7d4030f23c6a393dd32aa0c882f306b), and as expected the issue was there again.
Thanks for this fix :)


Update screenshots of Student Profile in README

Show off all the progress made in going from V1 to V2!

@alexsoble awesomeness, this looks great!


QA review 2 fixes

QA review 2 fixes

I thought you were going to put the Promise pollyfill in the main project?

I need to put it in both places. In case this is used in another project that does not use promises.


Update web-query-data.md

Add the pin to dashboard capability

@Gabilehner : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Hi @Gabilehner - It looks like you have access to the private repo and PRs that make changes larger than a typo or two should be handled in the private repo. In the future can you submit your commits there? Thanks.
@orspod - can you take a look at this PR?

Microsoft authors and contributors need to make content contributions through the private repository so the content can be staged and validated by the current validation rules.
After you review staging and fix all validation issues, you can tag an author for review or add the sign-off comment to let PR reviewers know that the updates are ready to be merged.
Read about authors working in the public repo for more information.
If you need help getting set up to work in the private repo, see How to get to the private repo. You can also take a look at the GitHub account setup page for additional details on making sure you're set up.

@Gabilehner - I incorporated your changes into this PR: https://github.com/MicrosoftDocs/dataexplorer-docs-pr/pull/433
and made some additional changes. Feel free to review.


Improvements of exception reporting in AxonServer connector

Exceptions are now reported using the mapping provided in the ErrorCode enum.
ConcurrencyExceptions are now also reported as such when a conflicting event sequence number is detected, allowing commands to be executed again.

Cross-reference to #885


As an Azure dev I would like to swap the database over to another user which should also clean the DB making azure run again

Acceptance:

Database bill and ownership is transfered to another user
Azure webapp runs again

Currently waiting for a friend of mine to give me a written guide on how to do it. Hopefully a payment of a beer will suffice to speed this up 😅

Update: i have received the instructions on how to swap it over so we will attempt it if not then we will have to redo everything from scratch on another account 😅

Update: Turns out the guide was useless since ITU have disabled the ability to view Tennant Emails so we are resorting to recreeating some of the stuff from the ground up.


Add ERB Linting

Description
Jira Ticket: SARAALERT-553
This PR adds the erb_lint gem to check the ERB files (additionally runs rubocop on the Ruby statements)
If reviewing please pay special attention to the changes in _html_head.html.erb because I needed to add to_json to all of the Ruby statements inside the JavaScript and this will have a problem if, for example, our sentry URL has an & in it (or other characters that will get escaped).

Looks good to me and clicked around and nothing seems to be broken. Just looking for clarification on that one change and then will approve.


vue-router命名路由 /detail/:id 打包后部署刷新不能加载?

What problem does this feature solve?
vue-router命名路由 /detail/:id 打包后部署刷新不能加载?
What does the proposed API look like?
vue-router命名路由 https://next.router.vuejs.org/zh/guide/essentials/named-routes.html

Hi, thanks for your interest but Github issues are for bug reports and feature requests only. You can ask questions on the forum, the Discord server or StackOverflow.


[fix][client] Fix client erroneous code comments

Motivation
Fix incorrect comments.
Documentation

[x] doc
[ ] doc-required
[ ] doc-not-needed
[ ] doc-complete

LGTM

@crossoverJie CI failed. I've fixed for you this time. Please check the status report and fix by yourself the next time :)

Codecov Report

Merging #19915 (7c42d50) into master (f20dc93) will increase coverage by 47.67%.
The diff coverage is 100.00%.

@@              Coverage Diff              @@
##             master   #19915       +/-   ##
=============================================
+ Coverage     25.11%   72.79%   +47.67%
- Complexity      205    31476    +31271
=============================================
  Files          1680     1859      +179
  Lines        127176   136836     +9660
  Branches      13863    15049     +1186
=============================================
+ Hits          31942    99611    +67669
+ Misses        90242    29309    -60933
- Partials       4992     7916     +2924

Flag
Coverage Δ

inttests
24.40% <0.00%> (?)

systests
25.28% <0.00%> (+0.16%)
:arrow_up:

unittests
72.06% <100.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...rg/apache/pulsar/common/util/URIPreconditions.java
83.33% <100.00%> (+83.33%)
:arrow_up:

... and 1512 files with indirect coverage changes

Merging...
Thanks for your contribution @crossoverJie!


validate if working directory configuration exists

Currently, this gives a system failure with no clear indication of what the issue is (a general file not found error)

@tylersouthwick If you need another review, feel free to request it in github, otherwise it's difficult for me to keep track. But it looks like you've still got some CI failures to figure out.
Just to make sure you're not waiting on me without me knowing about it. :)

Since this issue has been stale for a while, I've implemented my suggestions in https://github.com/alacritty/alacritty/pull/3190.


@can with @endcan Blade directives are being ignored

@can with @endcan Blade directives is being ignored

I've a normal editColumn function that I use to display a select box with old value selected by default. but one option is for admins only so therefore I'm using the @can directive to determine the status of this user and based on this I'm hiding/showing this option. however when I add this directive the whole option is being ignored!
->editColumn( 'status', '
// some code goes here
                                    <select name="status" class="form-control input-sm">
                                        <option disabled>Select status</option>
					  @can("is-admin")
                                               <option value="3">Published</option>
                                        @endcan
                                        @if($status == 2) <option value="2" selected > @else <option value="2"> @endif Not published</option>
                                        @if($status == 1) <option value="1" selected > @else <option value="1"> @endif Draft</option>
                                        @if($status == 1) <option value="0" selected > @else <option value="0"> @endif Canceled</option>
                                    </select>
// some other code goes here'
		                 )

System details

Operating System: Ubuntu 14.04.4 (Vagrant/Homestead)
PHP Version: 7.0.3
Laravel Version: 5.2.22
Laravel-Datatables Version: 5.7.11

:+1:  It's all about an issue with my code, sorry for bothering


Consider exposing the restartTimes to LanaguageClientOptions

The languageclient restarts LSP server 5 times if LSP server crashes. The restart time is hard-coded in https://github.com/Microsoft/vscode-languageserver-node/blob/master/client/src/client.ts#L272.
Can we expose it as an option like restartTimes in LanguageClientOptions, so that LSP extension can customize it? I know LSP extension can provide their custom ErrorHandler to adjust the restart times, but I think it is still sensible to provide such option.

@hokein exposing it makes sense. A PR is welcome for this.