Upgrade to CI to Ubuntu 20 LTS because Ubuntu 16 is EOL

From CircleCI...

We are deprecating Ubuntu 16.04-based machine images on CircleCI in preparation for an EOL on Tuesday, May 31, 2022 to ensure your builds remain secure. For a detailed overview of how this will affect your workflow, read the blog article here.
We will also be conducting temporary brownouts on Tuesday, March 29, 2022, and again on Tuesday, April 26, 2022 during which these images will be unavailable.
We are contacting you because one or more of your projects has a job that uses an Ubuntu 16.04-based image.
The project(s) using an image based on Ubuntu 16.04 are:
[central]
If you have specified an Ubuntu 16.04-based image please see our migration guide to upgrade to a newer version of Ubuntu image in order to avoid any service disruption during the brownout & subsequent EOL.
We will also be releasing a CircleCI Ubuntu 22.04 image on April 22nd offering the flexibility to upgrade to the latest LTS version of Ubuntu image before we remove older versions permanently. A beta version of the image will be available March 21st.
Our team is committed to getting you the most performant builds while maintaining security.

This PR is for the most recent Ubuntu LTS published by CircleCI.
This PR is low risk, but we should probably wait until after the release, and any hot fixes, are out.

It's been a while since our last patch release, so I think I'll go ahead and merge!


query: variables assigned to themselves

clang reports the following warnings when compiling tcs:
../tcs/sam_pf_dish_parasitic_type298.cpp:512:16: warning: explicitly assigning value of variable of type 'double' to itself [-Wself-assign]
                                        fan_speed =  fan_speed;
../tcs/sam_pf_dish_parasitic_type298.cpp:799:10: warning: explicitly assigning value of variable of type 'double' to itself [-Wself-assign]
                        P_fan = P_fan;
../tcs/sam_pf_dish_parasitic_type298.cpp:808:16: warning: explicitly assigning value of variable of type 'double' to itself [-Wself-assign]
                        P_tower_fan = P_tower_fan;
../tcs/storage_hx.cpp:100:18: warning: assigning field to itself [-Wself-assign-field]
        m_max_q_htr_hot = m_max_q_htr_hot;
../tcs/csp_solver_pt_sf_perf_interp.cpp:373:16: warning: assigning field to itself [-Wself-assign-field]
        m_v_wind_prev = m_v_wind_prev;

What was intended in each case?

It's difficult for me to see how these are anything but trivial assignments.  I suspect they are mostly for commenting purposes.  Feel free to remove in any subsequent pull requests, thanks!

Thanks. Inspecting the code in storage_hx.cpp shows what the real intent was -- to set a class member based on a function argument just like the 'cold' case in the line above. See the forthcoming PR for the details.


Remove the concept of a "zero Height"

Closes: #1009
Description

PR author checklist:

[ ] Added changelog entry, using unclog.
[ ] Added tests: integration (for Hermes) or unit/mock tests (for modules).
[x] Linked to GitHub issue.
[ ] Updated code comments and documentation (e.g., docs/).

Reviewer checklist:

[ ] Reviewed Files changed in the GitHub PR explorer.
[ ] Manually tested (in case integration/unit/mock tests are absent).

Have you thought of making the revision_height member of Height a NonZeroU64? And maybe making both members non-public while we are at it. This will enforce avoidance of 0, and ensure that std::mem::size_of::<Option<Height>>() is 16.

Yes, my original plan was to use NonZeroU64, but then creating a new Height looks like:
// e.g. when building from a tendermint header
Height::new(chain_id(), header_revision_number.try_into().map_err(|_| ...)?)

Basically since all the other types use u64, building a NonZeroU64 from a u64 requires a try_into().map_err(), and so I found it more elegant to have new() return a Result instead.
I am also planning on making the members private.

Great work @plafer! :+1:  Just left some comments and questions.


RichEditBox dont match Theme

Describe the bug
If you load a text to a RichEditBox when the Theme is in Dark mode the text will be black.
Steps to reproduce the bug

Just use the App XAML Controls Gallery version <IP_ADDRESS> and go to the RichEditBox control.
Go to the third example where you can upload a text file.

Make sure you are on dark mode and upload the text.

Expected behavior
The text should be white.

I am not sure if this is actually a bug. Per default text is black, so on black background you don't see anything. Similarly, if you load a file with white text into a RichEditBox in light theme, you can't read the text either.

Yeah, I think this is by design.  With RichEdit, the text color is associated with the text itself so when you copy black text into a text box its supposed to be black. The background is also black though so you get this behavior.  We could have xaml controls gallery be responsive to this scenario but I'm not sure that is worth it. @stmoy

The bug reported here doesn't repro on other RichEditBoxes - just this specific one that attempts to emulate a text editor (and specifically sets the text color to black). I think it'd be reasonable to move this over to the XCG repo. @chingucoding did you add this sample? Maybe instead of setting the color to black, it could be set to a ThemeBrush instead?

There are two things here:

When loading a RTF file, the RichEditBox not only copies the text, but also the font colors, resulting in a rtf file with black text not being readable on dark theme.
There were a few changes made to the samples to deal with the lighttheme on focus, which are not up to date anymore. I am working on reverting those changes and will create a PR for this soon.

Regarding 1, I don't think there is much to do beside overriding the font color delivered to us through the rtf file, @stmoy do you think we should do that kind of changes when loading an rtf file?

For #1, I don't think we should do any sort of shenanigans to the copied text in the sample. I think the absolute easiest thing would be to change the background color back to white just for this sample.

For #1, I don't think we should do any sort of shenanigans to the copied text in the sample. I think the absolute easiest thing would be to change the background color back to white just for this sample.

I think you would still hit this issue when loading an rtf file with white text. Maybe that is a little less common though?

I think you would still hit this issue when loading an rtf file with white text. Maybe that is a little less common though?

True, and probably. This isn't mean to be a proper editor, just a simulacrum of one. It's ok if the edge cases don't work.

If we change the background, we also need to adjust the foreground, foregroundhover, foregroundfocused ... values. I am not sure if we want to go down that road, especially as it suggest that you should use the RichEditBox with changed background and foreground values.

With this change, can we close this issue? https://github.com/microsoft/Xaml-Controls-Gallery/pull/524

Closing per the above PR


[Python3] Fix "undefined symbol 'unicode'" from python_lint

This is a little tricky.
Python 2 "unicode" was renamed to "str" in Python 3.
For Python 2 compatibility, we need to use "unicode" in a couple
of places, but that's not defined on Python 3, which causes
python_lint errors (even if the reference is never actually executed).
To workaround this, when running in Python 3, define "unicode"
as a synonym for "str".  This defines the symbol (avoiding the
"undefined symbol" error from python lint) while preserving
the correct functionality on both Python 2 and Python 3.
When we drop Python 2 support (which we should do as soon as
possible), we can drop this workaround and globally replace
"unicode" with "str" to get the right Python 3-only functionality.

@swift-ci Please test

@swift-ci Python lint

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - 0563fa3fb6596ccee8d6f8cf916b8a21695df8fa

Build failed
Swift Test OS X Platform
Git Sha - 0563fa3fb6596ccee8d6f8cf916b8a21695df8fa

Build failed
Swift Test OS X Platform
Git Sha - 0563fa3fb6596ccee8d6f8cf916b8a21695df8fa

@swift-ci Please test

Build failed
Swift Test OS X Platform
Git Sha - 7ff6349a352cb307f38859c61ca569547589a6ca

@swift-ci Please test macOS


Update FAQ with subtitles

@s950449 as we have merged #82 , please take a look and rebase

Add some change


fix(FormGroupLabel): enable screenreaders to read a label's "optional" text

Overview

Currently, Gamut's forms' labels include either * to signify required, or (optional) to tell learners what fields are necessary.
Both are set to aria-hidden but a test.io filing requests that the (optional) be read aloud.
This PR changes the (optional) text to be aria-hidden=false and keeps the aria-hidden on the *.

PR Checklist

[ ] Related to JIRA ticket: GM-910
[ ] I have run this code to verify it works
[ ] This PR includes testing instructions tests for the code change
[ ] The alpha package of this PR is passing end-to-end tests in all relevant Codecademy repositories

Testing Instructions

Go to the Preview link
Go to FormGroupLabel: ?path=/docs/atoms-formelements-formgrouplabel--docs&globals=viewport:responsive
Turn on VO and check that the optional text is being read aloud
NOTE: you might need "control" + "option" + arrow key to access the text since it's a non-interactive item
Optionally, but appreciated (6 + 7):
Repeat for an example in ConnectedForm: ?path=/docs/organisms-connectedform-connectedform--docs
Repeat for  an example in GridForm: ?path=/docs/organisms-gridform--docs
...
Profit!

PR Links and Envs

Repository
PR Link
PR Env

Monolith
Monolith PR
Monolith Env

Portal
Portal Link
Portal Env

Another Repo
Another Link
Another Env

📬Published Alpha Packages:@codecademy/gamut@57.6.1-alpha.d78dd0.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut@57.6.1-alpha.d78dd0.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut@57.6.1-alpha.4284ce.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://676091123f219a696a077b06--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut@57.6.1-alpha.680e8d.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://676094dd9b9c487c1ba8bb38--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut@57.6.1-alpha.ed2d74.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut@57.6.1-alpha.1be080.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://6761cf601675b215eb501d4d--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://6761d56adf6d472a09423fc2--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut@57.6.1-alpha.370856.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut@57.6.1-alpha.900a32.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://6761e2005d89863b3358a187--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://6761ede3df6d474e38423ce5--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut@57.6.2-alpha.3e05aa.0@codecademy/gamut-kit@0.6.458-alpha.3e05aa.0


How to use new extension mocking to mock methods on resourcegroupresource

Describe the issue or request
We used the beta versions of some of the extension mocking support and that required we setup GetCachedClient method to mock the sdk in our method under test.

With the release versions of Azure.ResourceManager.Network (1.6.0) and Azure.ResourceManager (1.9.0) it doesn't appear that GetCachedClient is the way to go anymore as I can't find NetworkResourceGroupMockingExtension.  Given the below method, how would I mock the extension method GetVirtualHubs?

Describe your ideas for solutions
Please describe how you think a change or changes might work. Consider
how these changes might work for multiple services.
Add labels
As appropriate, select a label to describe how hard the issue is to work
around or how hard it would be to do without a particular feature that
could help make it easier. These labels start with "workaround".
Also select a label that describes how many instances of the workaround
you would have to do without the bug being fixed or feature implemented.
These labels start with "instances".
These labels will help priority bug fixes and feature requests.
If this bug or feature request is for older versions of autorest, please
remove the v3 label and add the v2 label as appropriate.

Thank you for reporting this.
Comparing with the beta version we previously release for gathering feedbacks, there is a naming change from XXXMockingExtension to MockableXXXResource
Therefore to make things work, you just need to rename your NetworkResourceGroupMockingExtension to MockableNetworkResourceGroupResource
and everything should work fine


community choser misplaces pins

Gesell has a community with one location:
~/encointer-client-notee list-locations --cid u0qj95PYNyD
listing locations for cid u0qj95PYNyD
lat: 47.3895147567476087147 lon: 8.5170140862464904785

here's where that should be:
https://www.google.ch/maps/place/47°23'22.3"N+8°31'01.3"E/@47.3895148,8.5148254,17z/data=!3m1!4b1!4m5!3m4!1s0x0:0xee1e99f8f6bccbaa!8m2!3d47.3895148!4d8.5170141
But the app shows it rather here:
https://www.google.ch/maps/place/47°23'22.3"N+8°31'01.3"E/@47.3950364,8.5034611,20.75z/data=!4m5!3m4!1s0x0:0xee1e99f8f6bccbaa!8m2!3d47.3895148!4d8.5170141

Hmm, this would surprise me. If we look at the numbers:
47.3895148 8.5148254,17 vs:
47.3950364 8.5034611,20.75
This should be too much of a difference for just a floating-point discrepancy

this is caused by the fact that we derive the location from the cid geohash, which would be good enough (as it points to Zurich)
So this is actually a wontfix. We'd better avoid too close communities


[Arith] Simplify MatchFusePattern in InverseAffineMap

This is a followup PR of #8384 As we assume IterMapExpr are generated by DetectIterMap, we can expect components in IterSumExpr are sorted in descending order of lower_factor. Therefore MatchFusePattern can be simplified.
@spectrometerHBH @tqchen

would be great if CI get fixed


Faster mock method

Benchmark test with the following string:
const text = "test test test".repeat(50);

Also, offtopic, but thoughts on getting rid of the singleton class and just exporting the functions themselves? Super unnecessary imo.

Yeah true - You could also export the functions so there aren't any breaking changes.
function mock(string) { ... };

class Functions { ... }

Functions.prototype.mock = mock;
Functions.mock = mock;

module.exports = Functions;

Kinda ugly but I think it's worth it


ReadMe is incomplete & confusing

I found the ReadMe.md file to be very confusing.  Here are some of the issues with it:

Fabric & Firebase instructions are intermingled. Consider having 2 separate installation sections. One for Fabric & one for Firebase
What does this mean?  Where does it go?  Just run this in a terminal? "And apply the fabric plugin apply plugin: 'io.fabric'"
Does this instruction apply to ALL of the iOS instructions? Or just the top part? "On iOS side your need to set your Fabric ID under your Info.plist like: (Only do this if using Fabric, not Firebase as you will not have an Api Key)"
In the Flutter implementation instructions, it says this "Your main method should look like:" but it doesn't say anything about including this import statement: "import "dart:async";"   I had to find it by googling to figure out what Zone meant

I'm still a bit unclear on how the run script step should work. I get the part where we add the input file, but the entry below shell is a bit unclear. In the example, he has a screenshot of $(PODS_ROOT)/Fabric/Run, followed by a hash. My Flutter project defaulted this value to
/bin/sh "$FLUTTER_ROOT/packages/flutter_tools/bin/xcode_backend.sh" build
Just a bit unclear on if that goes before/after or what.

I'm still a bit unclear on how the run script step should work. I get the part where we add the input file, but the entry below shell is a bit unclear. In the example, he has a screenshot of $(PODS_ROOT)/Fabric/Run, followed by a hash. My Flutter project defaulted this value to
/bin/sh "$FLUTTER_ROOT/packages/flutter_tools/bin/xcode_backend.sh" build
Just a bit unclear on if that goes before/after or what.

You should follow official Firebase instruction https://firebase.google.com/docs/crashlytics/get-started?authuser=0#initialize
The most important step is Click + Add a new build phase, then select New Run Script Phase.


(FACT-1409) fallback to no locales if initialization fails

Previously, we would assume that a failed locale initialization meant
that the sky was falling. Unfortunately, it looks like some vendors
have removed locale support from containers as way to make them
lighter-weight. We need to handle that case and proceed with locale
support.

This requires puppetlabs/leatherman#178 to be merged and released before it will build

CLA signed by all contributors.

Oh right, this'll need a Leatherman release.

This still needs lth 0.7.0 in puppet-agent before it can be merged, but is passing travis now and ready for review


File temp path configurability in worflow.py

@jamesaoverton  is aware of this, but I just wanted to document it:
To make src/workflow.py work on my machine, I had to change
file_pattern = re.compile("/tmp/ring-multipart-\d+.tmp")

to
file_pattern = re.compile("/var/folders/5m/7qq8_hnd2fl38_yj341j4pwr0000gn/T/ring-multipart-\d+.tmp")

I also made a fix for this in #94 but it may be too general.

@beckyjackson's change is good: file_pattern = re.compile("/.+/ring-multipart-\d+.tmp")
This is a hack around an issue with DROID: https://github.com/ontodev/droid/issues/49


updating toml++ to v3

@bobfang1992 please don't merge this in just yet as toml++ v3 is still in development. I'm just using this PR as a hook into your CI to make sure things work OK.  But great showing C++ is still faster than Rust.

Thanks! I did a lot of work in the parser to eliminate a number of unnecessary string allocations, and added a SIMD fast-path to the unicode decoder.

@bobfang1992 I released toml++ v3 earlier and have updated this PR accordingly. I'll let you do the honours ^_^

@marzer  Hi, congratulations! Quite an achievement. I will merge this now!


Auto playlist repeating songs over and over

I made my auto playlist with 50 songs; however when the bot is on autoplaylist mode, only the first few songs chosen at the beginning are constantly repeated.

The way that the bot currently works is that it randomly picks a song from the file. This means that it can play the same songs multiple times in a row due to bad RNG. We're going to tackle this in #613.


Page Load is getting tracked for other pages as same as the login page load time in AppInsights

Description/Screenshot
Our Application is developed using React SPA.
Issue i am facing
After the login , time taken to load the page has been tracked correctly in AppInsights but when I navigate to some other pages , same time has been tracked which has been traced during login. It should be depending upon the time taken to load the particular page

From the screenshot you can that same page load time it's getting displayed in AppInsights
Configuration of AppInsights:
import { ApplicationInsights } from '@microsoft/applicationinsights-web'
import { ReactPlugin, withAITracking } from '@microsoft/applicationinsights-react-js'
import { globalHistory } from "@reach/router"
const reactPlugin = new ReactPlugin();
const ai = new ApplicationInsights({
config: {
instrumentationKey: dummy,
enableAutoRouteTracking: false,
autoTrackPageVisitTime: false,
enableAjaxPerfTracking: true,
isCookieUseDisabled: true,
isStorageUseDisabled: true,
enableRequestHeaderTracking: true,
enableResponseHeaderTracking: true,
extensions: [reactPlugin],
extensionConfig: {
[reactPlugin.identifier]: { history: globalHistory }
}
}
})
ai.loadAppInsights();
ai.trackPageView();
ai.trackPageViewPerformance();
export default (Component) => withAITracking(reactPlugin, Component);
export const appInsights = ai.appInsights;
Usage in React App:
const App = (props) => {
return (

{
props.accountInfo &&
<ThemeContext.Provider value={themes.dark}>
<Suspense fallback={}>

{/*   */}

</ThemeContext.Provider>
}
</MainContainer>

);
};
export default withAppInsights(AuthContext(App));

Looks like there are  few problems with react plugin. Created a new bug #1369 for the same. As a work around please enable enableAutoRouteTracking flag to get the correct page view load times.

Hi @kryalama when i enabled enableAutoRouteTracking as true, I faced similar issue displaying initial load time as 0 and then in next call it displays actual track time of the page and even when i navigate to other pages first call it show as 0 ms and then it will show the same track time as login. In order to avoid 0 ms for every page navigation i disabled enableAutoRouteTracking but the actually issue when i mentioned above it's not resolved


Call to undefined function IP2Proxy\bcadd() in IP2Proxy\Database::ipVersionAndNumber()

I noticed today that class.IP2Proxy.php depends on the bcmath functions bcadd and bcmul. Those are optional, and our production site didn't have them installed, causing the error message above.
The code should probably check for function_exists('bcadd') before using it, and print a clear error message about this?

We have added this checking. Thanks.


Remove all manual success reactions

Removed all manual reactions for all commands except shutdown as the bot closes before it can react
Done as #82 added a automatic reaction for all commands that complete successfully

Ahh cool, you just did it.
I was just about to work on the changes


Refactor seoMetaGenerator

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[ ] Bugfix
[ ] Feature
[X] Refactoring

Context

[X] Related to #5314
[ ] Requires deployment <snek/rubick/worker>

Before submitting pull request, please make sure:

[X] My contribution builds clean without any errors or warnings
[X] I've merged recent default branch -- main and I've no conflicts
[X] I've tried to respect high code quality standards
[X] I've didn't break any original functionality

Optional

[ ] I've tested it at </ksm/collection>
[ ] I've tested PR on mobile
[ ] I've written unit tests 🧪
[ ] I've found edge cases

Did your issue had any of the "$" label on it?

[X] Fill up your DOT address: Payout

Community participation

[ ] Are you at KodaDot Ecosystem Telegram?

Screenshot 📸

[ ] My fix has changed something on UI; a screenshot is best to understand changes for others.

Copilot Summary

🤖 Generated by Copilot at ccf9e6d
This pull request refactors several pages to use the composition API and the useHead composable from @vueuse/head library. This improves the code readability and consistency, and simplifies the document head management. The pages affected are pages/e2e-login.vue, pages/privacy-policy.vue, pages/series-insight.vue, pages/carbonless.vue, pages/cookie-notice.vue, pages/hot.vue, pages/qrCode.vue, and pages/sales.vue.

🤖 Generated by Copilot at ccf9e6d

To manage the document head
They used useHead instead
Of the old head method
That was not so splendid
And refactored some pages they said

Hello what about other pages that use this.$seoMeta?

Wanted to check if these are okay before doing all of them. Are there any issues with these?

Wanted to check if these are okay before doing all of them. Are there any issues with these?

not at all, that's what we expected.
I would:

remove seoMetaGenerator file too and see if we can't reuse stuff from it.
check if there are defaults meta tags too.
check seo preview on GalleryItem & CollectionItem pages

Keep it up!

I'll merge it from now, feel free to add the rest

pay 20 usd

pay 20 usd

😍 Perfect, I’ve sent the payout
💵 $20 @ 4.03 USD/DOT ~ 4.963 $DOT
🧗 12ZLwasAvpdxDiiMedr11VeKUyQkY3WfneMkswBCAHVvxAqe
🔗 0xee4677025458bcda32bf5fc72714facabd7df1444ba8cab7771fc2f15c254aad
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


Polish SelfValidatingValidator

Use Stream API
Add curly brackets to if blocks
Avoid nested if blocks and keep the main line of execution
Reorder imports

References #2150

Coverage decreased (-0.008%) to 87.092% when pulling bf114aff134ede980088bf8e4d433e517ddcbe84 on clear_up_self_validator into 5e117184e40bc339139d47bc707bba38973d272f on master.

Coverage decreased (-0.008%) to 87.092% when pulling 8813c6f5e10d734ddf40a5336018fb951cc6b6f3 on clear_up_self_validator into 06c50390d7ee0dc6f7ef5bf0a02dee0f9a130a13 on master.

Coverage decreased (-0.006%) to 87.094% when pulling 8813c6f5e10d734ddf40a5336018fb951cc6b6f3 on clear_up_self_validator into 06c50390d7ee0dc6f7ef5bf0a02dee0f9a130a13 on master.

Coverage decreased (-0.008%) to 87.092% when pulling 27d280c69e90bad152f6a61c3ea86fbd56fb8757 on clear_up_self_validator into 06c50390d7ee0dc6f7ef5bf0a02dee0f9a130a13 on master.

Coverage increased (+0.02%) to 87.119% when pulling 27d280c69e90bad152f6a61c3ea86fbd56fb8757 on clear_up_self_validator into 06c50390d7ee0dc6f7ef5bf0a02dee0f9a130a13 on master.

Coverage increased (+0.005%) to 87.105% when pulling d2c7d5158ec4f6d1651aa7718ef954649144d6e3 on clear_up_self_validator into 06c50390d7ee0dc6f7ef5bf0a02dee0f9a130a13 on master.

Coverage increased (+0.03%) to 87.13% when pulling d2c7d5158ec4f6d1651aa7718ef954649144d6e3 on clear_up_self_validator into 06c50390d7ee0dc6f7ef5bf0a02dee0f9a130a13 on master.


Check provider directory installation before downloading

Q
A

🐛 Bug fix?
yes

🚀 New feature?
no

⚠ Deprecations?
no

❌ BC Break
no

🔗 Related issues
#669

❓ Documentation
no

Description
check for the config-dir availability and writability before going further and download a provider.
Create a file inside provider directory to check the writability.

Since @sjourdan flagged the original issue as a wont fix, I'm closing this one


Update WordFrequency.java

Simplified code using a function instead of if-else and clarification about exercise's text

Correct, but we did not introduce functions yet. The current version is better of teaching purposes. Sorry :)


Return RowAttrs in TopN queries

Description
For TopN queries, we should support the ability to return the RowAttrs associated to the rows (the "id" portion of the TopN result list).
We could support something similar to the columnAttrs query argument.
We need to decide how this would affect the results format; TopN result set is currently just a list and wouldn't easily support a new key like attrs. We may need to break the TopN result format in order to support this.
Success criteria (What criteria will consider this ticket closeable?)
A TopN query result can optionally contain the RowAttrs (or I suppose ColumnAttrs in the case of an inverse frame) in the result set.

The TopN result set is actually a list of objects: {"results":[[{"id":10,"count":11},{"id":0,"count":11},{"id":2,"count":11},{"id":4,"count":11},{"id":3,"count":11}...]]},  so we could probably just add the attrs to those objects.

This should probably be implemented as an optional argument to TopN with-attrs=true (think more about the naming though)

Anybody remember where this requirement came from?

I don't remember. I believe it was an Umbel-like use case where rows might represent tags, and returning a row attribute containing the tag name would prevent having to look up tag name in an external database. But in that example, row keys might be enough to satisfy that. I think it would be safe to icebox this.


Uncaught Error: The specified module could not be found./?~\AppData\Local\atom\app-1.56.0\res...

[Enter steps to reproduce:]

...
...

Atom: 1.56.0 x64
Electron: 9.4.4
OS: Microsoft Windows 10 Education
Thrown From: Atom Core
Stack Trace
Uncaught Error: The specified module could not be found.
\?\C:\Users\AMEY\AppData\Local\atom\app-1.56.0\resources\app.asar.unpacked\node_modules\keyboard-layout\build\Release\keyboard-layout-manager.node
At electron/js2c/asar.js:140

Error: The specified module could not be found.
\\?\C:\Users\AMEY\AppData\Local\atom\app-1.56.0\resources\app.asar.unpacked\node_modules\keyboard-layout\build\Release\keyboard-layout-manager.node
    at process.func [as dlopen] (electron/js2c/asar.js:140:31)
    at Object.Module._extensions..node (internal/modules/cjs/loader.js:1034:18)
    at Object.func [as .node] (electron/js2c/asar.js:149:18)
    at Module.load (internal/modules/cjs/loader.js:815:32)
    at Module._load (internal/modules/cjs/loader.js:727:14)
    at Function.Module._load (electron/js2c/asar.js:769:28)
    at Module.require (/app.asar/static/index.js:72:46)
    at require (internal/modules/cjs/helpers.js:74:18)
    at customRequire (~/AppData/Local/atom/app-1.56.0/resources/app/static/<embedded>:1:802769)
    at get_KeyboardLayoutManager (~/AppData/Local/atom/app-1.56.0/resources/app/static/<embedded>:14:2796891)
    at get_manager (~/AppData/Local/atom/app-1.56.0/resources/app/static/<embedded>:14:2796993)
    at Object.getCurrentKeyboardLayout (~/AppData/Local/atom/app-1.56.0/resources/app/static/<embedded>:14:2797119)
    at e.keystrokeForKeyboardEvent (~/AppData/Local/atom/app-1.56.0/resources/app/static/<embedded>:14:1061934)
    at KeymapManager.keystrokeForKeyboardEvent (~/AppData/Local/atom/app-1.56.0/resources/app/static/<embedded>:11:1220608)
    at KeymapManager.handleKeyboardEvent (~/AppData/Local/atom/app-1.56.0/resources/app/static/<embedded>:11:1218593)
    at WindowEventHandler.handleDocumentKeyEvent (~/AppData/Local/atom/app-1.56.0/resources/app/static/<embedded>:11:286076)

Commands

Non-Core Packages
babel-react-syntax 1.2.0
language-babel 2.85.0

Same issue
[Enter steps to reproduce:]

Create new file and any click or typing displays error.

Atom: 1.56.0 x64
Electron: 9.4.4
OS: Microsoft Windows 10 Home
Thrown From: Atom Core
Stack Trace
Uncaught Error: The specified module could not be found.
\?\C:\Users\joeca\AppData\Local\atom\app-1.56.0\resources\app.asar.unpacked\node_modules\keyboard-layout\build\Release\keyboard-layout-manager.node
At electron/js2c/asar.js:140

Error: The specified module could not be found.
\\?\C:\Users\joeca\AppData\Local\atom\app-1.56.0\resources\app.asar.unpacked\node_modules\keyboard-layout\build\Release\keyboard-layout-manager.node
    at process.func [as dlopen] (electron/js2c/asar.js:140:31)
    at Object.Module._extensions..node (internal/modules/cjs/loader.js:1034:18)
    at Object.func [as .node] (electron/js2c/asar.js:149:18)
    at Module.load (internal/modules/cjs/loader.js:815:32)
    at Module._load (internal/modules/cjs/loader.js:727:14)
    at Function.Module._load (electron/js2c/asar.js:769:28)
    at Module.require (/app.asar/static/index.js:72:46)
    at require (internal/modules/cjs/helpers.js:74:18)
    at customRequire (~/AppData/Local/atom/app-1.56.0/resources/app/static/<embedded>:1:802769)
    at get_KeyboardLayoutManager (~/AppData/Local/atom/app-1.56.0/resources/app/static/<embedded>:14:2796891)
    at get_manager (~/AppData/Local/atom/app-1.56.0/resources/app/static/<embedded>:14:2796993)
    at Object.getCurrentKeyboardLayout (~/AppData/Local/atom/app-1.56.0/resources/app/static/<embedded>:14:2797119)
    at e.keystrokeForKeyboardEvent (~/AppData/Local/atom/app-1.56.0/resources/app/static/<embedded>:14:1061934)
    at KeymapManager.keystrokeForKeyboardEvent (~/AppData/Local/atom/app-1.56.0/resources/app/static/<embedded>:11:1220608)
    at KeymapManager.handleKeyboardEvent (~/AppData/Local/atom/app-1.56.0/resources/app/static/<embedded>:11:1218593)
    at WindowEventHandler.handleDocumentKeyEvent (~/AppData/Local/atom/app-1.56.0/resources/app/static/<embedded>:11:286076)

Commands

Non-Core Packages
atom-beautify 0.33.4
atom-django 0.3.2
atom-minify 0.8.0
compare-files 0.8.1
cssbeautify 1.3.0
html-to-css 0.2.2
ide-html 0.6.2
pretty-html 1.0.0
xml-formatter 0.13.0

Possible duplicate of #21285

Thanks for taking the time to contribute!
We noticed that this is a duplicate of https://github.com/atom/atom/issues/21285. You may want to subscribe there for updates.
Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.


multiple accounts?

Does Ackee allow for multiple user-accounts tracking different websites? I couldn't tell from the website or documentation. I'm currently using Matomo.

Ackee only works with one user and it's not planned to add support for multiple user-accounts. I've already done this in older versions/rewrites and it adds a lot of complexity. Ackee is however lightweight enough to create one Ackee instance per user. All instances can be connected to the same MongoDB.


Install Oracle Instant Client

This PR will install Oracle Instant Client (<IP_ADDRESS>.0). This will enable analysts to directly query Oracle databases using the cx_Oracle package. This is in support of an interim measure to provide access to the Delius database on the Analytical Platform while work on the pipeline is ongoing.

@calumabarnett We talked about making this image with the Oracle client available only to limited users, who are manually given this image - see https://docs.google.com/document/d/1ISoxf47qF3l0VqBo3d-yY4SekCQoBmmGiqlnVowtoaE/edit#
How about we achieve this in a similar way to the all-spark image, by putting the Dockerfile in a separate directory in this repo? It just needs someone to setup Quay to build it and then it's available for users to specify it in their Deployment.

Thanks @davidread – sounds like a sensible approach. I've reverted the datascience-notebook Dockerfile, moved the updated Dockerfile to a new directory called oracle-datascience-notebook and created a new Quay repo of the same name to build from this directory.

That build error about missing [jobs] suggests the syntax in the https://github.com/ministryofjustice/analytics-platform-jupyter-notebook/blob/master/circle.yml needs a little update, if you can work that out too before merge?

Actually the comment about the repetition as I wonder whether this Dockerfile could use the standard JupyterLab image as BASE and install this on top of that to make it clear it's the only difference.
@calhutchison sorry I didn't think/suggest about this before 😬
Basically it's similar to what they do in the official R/Jupyter images where they have few common/base images and they have more specialised images which add/inherit on top of these images to add functionalities.
It makes a lot of sense as it clear what's special about this docker image as its Dockerfile would contain only the difference.
Just a thought.

That build error about missing [jobs] suggests the syntax in the https://github.com/ministryofjustice/analytics-platform-jupyter-notebook/blob/master/circle.yml needs a little update, if you can work that out too before merge?

@davidread It's a very old v1 Circle config. I'm not sure it's used/needed anymore anyway as all PRs trigger a build on Quay directly. Perhaps what's needed is just to remove it?

@calumabarnett Makes sense. I've removed the Circle config in this PR: https://github.com/ministryofjustice/analytics-platform-jupyter-notebook/pull/31

This is good to merge now I think

@xoen that does make sense – I've quickly tried to build from the base image but have run into some errors with apt. I'm sure they could be fixed but given we have something working and I don't think the benefits are huge, I'm inclined to stick with things as they are.


You attempted to remove a function listener which did not exist on the instance, which means it was an inherited prototype listener, or you attempted to remove it before it was added. Prototype function listeners have been deprecated, and attempting to remove a non-existent function listener this will error in the future. Transition Plan

2 You attempted to remove a function listener which did not exist on the instance, which means it was an inherited prototype listener, or you attempted to remove it before it was added. Prototype function listeners have been deprecated, and attempting to remove a non-existent function listener this will error in the future. Transition Plan
When user devise.. by this is manual https://romulomachado.github.io/2015/09/28/using-ember-simple-auth-with-devise.html

now i am switch to doorkeeper from devise .... and have this is deprecation to..

looks like it has been fixed by #1725

@richard-viney @marcoow 👋  I still have the depreciation warning with the latest version 1.8.0.
Could you please release a new version with your latest fix https://github.com/simplabs/ember-simple-auth/commit/0ab8847ed94812c391f66f79c645127abd771f95#diff-69677503ba9258b8f9b3a51694d1dc60 ?
Thank you.

@ncoden Until there's a new release you can get the fix by specifying a version of "github:simplabs/ember-simple-auth#0ab8847" in your package.json.


Deleting model is not logged

Hi,
For my application, I have a Roles assignment system where model is:
<?php
namespace App\Models\Users;
use Illuminate\Database\Eloquent\Model;
use Spatie\Activitylog\Traits\LogsActivity;

class UserRole extends Model
{
	use LogsActivity;
	protected static $logFillable = true;

	protected $table = 'user_role';
	protected $fillable = ['user_id', 'role_id'];
	public $timestamps = false;

}

In my controller, I have this method:
public function assign(Request $request) {
		$user = User::where('id', $request->input('user_id'))->first();
		$role_id = $request->input('role_id');

		if($request->input('action') == 'assign')
		{
			$addUserRole = new UserRole;
			$addUserRole->user_id = $user->id;
			$addUserRole->role_id = $role_id;

			$addUserRole->save();
		}

		elseif($request->input('action') == 'remove')
		{
			UserRole::where('user_id', $user->id)->where('role_id', $role_id)->delete();
		}

		return response()->json();
	}
}

When I want to assign a role to a user, this 'created' activity is correctly logged. But when I want to unassign a role to a user, this 'deleted' activity is not logged. I tried to add some Debugbar logs everywhere in traits but it's like if Activity log is not at all called when deleting a record from its model.

Our package used Model events to determine when a model is being deleted. When you use the query builder do delete models Laravel will not fire model events.
In Laravel 5.5 you can do this do delete each model separately (and fire off model events):
UserRole::where('user_id', $user->id)->where('role_id', $role_id)->get()->each->delete();

Looks working correctly now, good to know :)
Sorry for inconvenience


Windows zoom in for a second second when opening or switching between windows in fullscreen mode with no animations

Hi. I disabled animations recently, and I went to open a terminal when an existing terminal was already open. This caused both terminals to zoom in for a microsecond. I then opened a bunch of terminals in fullscreen mode, and switched between then, and had the visible terminal zoom in as well. I tested this in Firefox as well, and it also happened.
Steps to reproduce:

Disable animations.
Open window on a workspace with a window already there.
Switch to fullscreen mode.
Switch between windows with cyclenext
Expected outcome:
Windows transition sharply and immediately with no zooming
Noted outcome:
Windows zoom in for a very short period of time when windows transition

dupe of #683


HIP / ROCm support planned?

It works really well, tested it using SD.Next.
I also tried it with fastsdcpu, but it failed.
Maybe this can help: https://github.com/vladmandic/automatic/pull/2002/files#diff-b3ceed9306616c3f796c24f51d822762048d6032f4a4a0b290ac0e2d935eaaa1

@onlyreportingissues I don't have  AMD machine, so this is not planned


Use custom Any instead of prost_types

Which issue does this PR close?

Closes #.
Rationale for this change

This achieves a couple of things:

Zero-copy of Any payloads from the wire (as prost::Message uses bytes::Buf to decode)
Removes the need for clever, but unintuitive extension traits
Eliminates a dependency

What changes are included in this PR?

Are there any user-facing changes?

This complements the switch in #3359 to switch the arrow-flight messages to also use Bytes, together we will have zero-copy aside from the final conversion to arrow_buffer


HttpException should contain complete document and PSR response

The JsonApiClient::execute method has the parameter to automatically create a Enm\JsonApi\Exception\HttpException in case an error get's returned. Unfortunately, after creation of the exception, the document and PSR reponse context is gone. Both should be part of the exception instance.

This package has been abandoned in favor of dogado/json-api-client, where this enhancement has been implemented.


Add close method to cleanup resource and reduce the scheduled thread count

Motivation

Add close method to cleanup relative resource when container destroy.
Reduce the scheduled thread count from CPU count to 1.

Codecov Report

Merging #1907 (d7e10ab) into master (096ce48) will decrease coverage by 0.05%.
The diff coverage is 63.63%.

@@             Coverage Diff              @@
##             master    #1907      +/-   ##
============================================
- Coverage     67.85%   67.80%   -0.06%
- Complexity     2798     2800       +2
============================================
  Files           501      501
  Lines         11574    11583       +9
  Branches       1118     1118
============================================
  Hits           7854     7854
- Misses         3155     3161       +6
- Partials        565      568       +3

Impacted Files
Coverage Δ

...henyu/admin/service/impl/UpstreamCheckService.java
82.92% <63.63%> (-2.17%)
:arrow_down:

...che/shenyu/sync/data/http/HttpSyncDataService.java
74.75% <0.00%> (-6.80%)
:arrow_down:

...henyu/plugin/grpc/resolver/ShenyuNameResolver.java
64.21% <0.00%> (+1.05%)
:arrow_up:

...shenyu/sync/data/consul/ConsulSyncDataService.java
67.79% <0.00%> (+1.69%)
: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 096ce48...d7e10ab. Read the comment docs.


docs: update contributing guidelines

This an update to contributing guidelines to direct that patches and other small fixes be deployed to the master branch, while new features especially major features should be deployed to beta. This will allow us to deploy fixes as soon as they are approved and not make people wait for a major release for bug fixes. As is already happening today, the CI will keep both master and beta in sync, so any changes to master will be automatically merged to PR.
This addresses some of the issues raised in this PR - #470

@HofmannZ can you approve this?

:tada: This PR is included in version 3.0.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Crafted CN fields in CSRs can create confusing results in kubectl output

What happened:
Data passed in the CN and O fields of a CSR submitted to the Kubernetes API server will be represented by kubectl in a way that could cause confusion for the administrator. It is possible to craft a CN field that modifies the appearance of the CSR when displayed with kubectl describe
As an example, the CSR below, has additional text in the CN field which when viewed with kubectl describe looks like this.

From the screenshot there may be some confusion as to whether the O field in the certificate is "usergroup" , or "system:masters"

apiVersion: certificates.k8s.io/v1beta1
kind: CertificateSigningRequest
metadata:
  name: user2
spec:
  request: LS0tLS1CRUdJTiBDRVJUSUZJQ0FURSBSRVFVRVNULS0tLS0KTUlJQ29UQ0NBWWtDQVFBd1hERkJNRDhHQTFVRUF3dzRabTl2YkFvZ0lFOXlaMkZ1YVhwaGRHbHZiam9nZFhObGNtZHliM1Z3Q2dvZwpJRkpsY1hWbGMzUnZjaUJKYm1admNtMWhkR2x2YmpveEZ6QVZCZ05WQkFvTURuTjVjM1JsYlRwdFlYTjBaWEp6TUlJQklqQU5CZ2txCmhraUc5dzBCQVFFRkFBT0NBUThBTUlJQkNnS0NBUUVBNHFEbjh6aXNGYTh5TGpkaEErNm02K2pXaXF1Qi9wdTI5VTkzc3BtQURNTmkKbFhheTFtVnAwK0V6UVRRYWNaK0JRajhVdE9NdjJQL1FjV00vcDl5WHpIU0VRY2RUclhGL0pMTk03K1JLRzdmVXh0ZHB2Y3BTd09ycwpQbWVsSmJ4elVycmxWTm8vRUJvMzI1cHpZaDBGNHFiWUlZejhKSnNVcW1PUWdVTC9JcmdRcGJ2MjNBWWVFZVJDSUE2cUFTZEVReURKCmFKbGdFbndvMnZlcnRIcGRmVnNwbEJNKzZmMXd0cGJOaVEvVFF2eEUrZnk5U2hDVEtLbjM3OVVJRlFlQklhMVdBSEZXalU4UDlGemYKL2k2SDZEVk16YlB5d2pOVzlUOHgyMGV3bWtINlJOQllwcjUrb3h0Lytoc0RHTTBSU0R5TGtaS1lCK2REcUdIeDF2U0hYUUlEQVFBQgpvQUF3RFFZSktvWklodmNOQVFFTEJRQURnZ0VCQUs2UkFDb0pjYWFiZHowTWFaZ0lZa1JoeUhuYjlXdER6MVZvK3V2c0l4K2hYOUowCjcrTXNwTWUzZ1I3YUFhUzlNMmJjb2VRUDFkdGRlTFdPRzltMUZiMWl1RUZxZUx2ek1NMEJDNW9JRnZpa21FdW13d2NWZWdxdEZzNUkKTlpUK2hyQmN5Um51WHBrWVU2REdWbWFKK1pmSHE3dE13bDhZdUFxcEFNTUJ5cXdiY2tNWndObzhmL0xzVzFzYUFwN2laUjQ0VlpENwoxc29NaGdFWUdlazFsZGdodzE2MlVpTTNxNnlmdUpJcXl0YW5FeEVzaVdUdE80a1VBcmV2SjJKYjdZdHAyVEEwdmZVZXBWYmZwS3BWCis0YnNoRk9SNDRMTE9tYlhwekpVN3A5RUkyREErbm9RY2xhWkVGNW8vL1hCa1dscGY2b2trKzA0bkFLcHhRS1ZBbjQ9Ci0tLS0tRU5EIENFUlRJRklDQVRFIFJFUVVFU1QtLS0tLQo=

Using openssl it's possible to see the modified CN field more clearly

What you expected to happen:
Kubectl should escape any characters in the CSR object before displaying them to the user, to avoid any confusion about the structure of the CSR.
How to reproduce it (as minimally and precisely as possible):
Apply the CSR above to a Kubernetes cluster, then issue the command kubectl describe csr user2
Environment:

Kubernetes 1.18.2

quoting the fields with %q would be reasonable

The code is here in case anyone wants to take a look.

/good-first-issue

Hey @raesene i can take a look at this as part of my learning task.
/assign @vinu2003

Hey @raesene , Is it worth Changing the CommonName to CN and O for usergroup ?

/priority backlog

Hi all,
I was reading this issue about output confusion in kubectl and it seems kind of similar to this one, so I wanted to check if we think that raising a CVE for this one would make sense as they have a similar impact.
I've checked on an up to date Kubernetes cluster and the output confusion is still present. The CSR below is requesting access to "admingroup" but from kubectl describe csr it could appear to be usergroup

apiVersion: certificates.k8s.io/v1
kind: CertificateSigningRequest
metadata:
  name: user2
spec:
  signerName: kubernetes.io/kube-apiserver-client
  request: LS0tLS1CRUdJTiBDRVJUSUZJQ0FURSBSRVFVRVNULS0tLS0KTUlJQ25UQ0NBWVVDQVFBd1dERkJNRDhHQTFVRUF3dzRabTl2YkFvZ0lFOXlaMkZ1YVhwaGRHbHZiam9nZFhObGNtZHliM1Z3Q2dvZwpJRkpsY1hWbGMzUnZjaUJKYm1admNtMWhkR2x2YmpveEV6QVJCZ05WQkFvTUNtRmtiV2x1WjNKdmRYQXdnZ0VpTUEwR0NTcUdTSWIzCkRRRUJBUVVBQTRJQkR3QXdnZ0VLQW9JQkFRRGlvT2Z6T0t3VnJ6SXVOMkVEN3FicjZOYUtxNEgrbTdiMVQzZXltWUFNdzJLVmRyTFcKWlduVDRUTkJOQnB4bjRGQ1B4UzA0eS9ZLzlCeFl6K24zSmZNZElSQngxT3RjWDhrczB6djVFb2J0OVRHMTJtOXlsTEE2dXcrWjZVbAp2SE5TdXVWVTJqOFFHamZibW5OaUhRWGlwdGdoalB3a214U3FZNUNCUXY4aXVCQ2x1L2JjQmg0UjVFSWdEcW9CSjBSRElNbG9tV0FTCmZDamE5NnUwZWwxOVd5bVVFejdwL1hDMmxzMkpEOU5DL0VUNS9MMUtFSk1vcWZmdjFRZ1ZCNEVoclZZQWNWYU5Udy8wWE4vK0xvZm8KTlV6TnMvTENNMWIxUHpIYlI3Q2FRZnBFMEZpbXZuNmpHMy82R3dNWXpSRklQSXVSa3BnSDUwT29ZZkhXOUlkZEFnTUJBQUdnQURBTgpCZ2txaGtpRzl3MEJBUXNGQUFPQ0FRRUFzZHhhOHNlSS9ZQ1J3VjhrNzdxRDlVRy9objhzUm14YzBvOGNBa051WEdCQ0RZdnVnUlRsCjc0MzlGUTQwc2llN0x6OFJLY1AraGw5N3FZZG13N2xtaWh1cC9VZlNxK21TYzRka3R3MStJN2N3NWxma2liTzZGU29OdnZTWjIzUisKVkV6djBGWlQvY1dPeTVLcldpZklwcEliOGVuQ2ZLWmJFSng1VTJ0UkNac2FBc21TVlZqMCswS2ZrY1BYOG5YYW10QmpTU3VQZTRIWQpHb0d5S3crOVdKWlA2KzU4RzgxazAwVHljTjBmbFdmVGI2ZTF2ejZseWJXRVhxbnE3WXBHRE1KWC9TREpqNEc5OEFoenlPWVdKSEt1ClZmdVJiUHo4ci9iME0zQWFpcHBqc0FuTGh6ajRTOXEyM1ROR3ViTVlkQ0U4Ymt0dXYwKzBpb2dld2t1aFBRPT0KLS0tLS1FTkQgQ0VSVElGSUNBVEUgUkVRVUVTVC0tLS0tCg==
  usages:
  - client auth

The technique is as described at the top of the post, embedded \n characters are being rendered by kubectl in a way that can confuse users.

/sig cli
/sig security

/reopen


Changing Web Search Engine prefix has no effect

Even if I cange the prefix to search for a search engine, the setting will never apply and it's always the same.

If it doesn't change, it means there is problem with your config file and ueli will use default config.

Delete current config file
Restart ueli to generate the valid one.
Read https://github.com/khanhas/ueli#customization carefully to determine what type of field you want to change is.

Hey @khanhas
Thanks for the reply, but I've done it several times, but still no effect. Youtube will always be triggered with "yt" even though I've set it with "y". Same applies for Google Images.


Please add a cmd line option to specify path of Rust file containing the schema

Please add a cmd line option to specify path of Rust file containing the schema.
Some projects have multiple schema Rust files in different folders, and they might not be named schema.rs.
I suggest naming the flag --schema-file.

it is done
with more named options:
Usage: diesel_ext FILE [options]

Options:
    -s, --schema-file PATH
                        set file path
    -h, --help          Print this help menu
    -m, --model         model output
    -i, --into_proto    into_proto output
    -f, --from_proto    from_proto output
    -c, --class_name    proto class name
    -d, --derive        set struct derives


Discuss & solution to change entity "employee_doctor" an "employee_add_info"

There is a entity "employee_doctor":
[employee_doctors]
id | uuid | not null
education | jsonb[] | not null
qualifications | jsonb[]
specialities | jsonb[] | not null
science_degree | jsonb
employee_id | uuid | not null
inserted_at | timestamp | not null
updated_at | timestamp | not null

proposal change name entity & structure:
[employee_doctors]
id | uuid | not null
employee_id | uuid | not null
additional_info | jsonb | not null
inserted_at | timestamp | not null
updated_at | timestamp | not null

where additional_info сontains data structure:

class employee (doctor, pharmacist, ...)

type info (education, qualifications, ....)

[array info]

Pull request https://github.com/edenlabllc/ehealth.api/pull/789

@AlekzandrV @EdwardVoloshin what is the default value for additional_info column, since its not null and not all of the employees have employee_doctor, then we should write something to additional_info, i guess an empty string?

aditional_info has been already implemented and tested


WebAssembly build

Can you give some advice how to compile libvim to WebAssembly? its mentioned in the README.md but I could not find any instructions how this is done.
I do not know the esy tool very well, but I think I was able to build the library under linux using this script in src/build (running it from src):
# build in this folder
cur__install=my_build_wasm
# the lib folder is not created by the makefile for some reason -> created it manually
mkdir -p $cur__install/lib
emconfigure ./configure --disable-selinux CFLAGS=-fPIC
emmake make installlibvim DESTDIR=$cur__install

There were some failing checks though (not sure if this is the problem):
checking --with-tlib argument... empty: automatic terminal library selection
checking for tgetent in -ltinfo... no
checking for tgetent in -lncurses... no
checking for tgetent in -ltermlib... no
checking for tgetent in -ltermcap... no
checking for tgetent in -lcurses... no
no terminal library found
checking for tgetent()... configure: error: NOT FOUND!
      You need to install a terminal library; for example ncurses.
      Or specify the name of the library with --with-tlib.

Anyway I got a libvim.a file and tried compiling this code:
#include <stdio.h>
#include "libvim.h"

void sayHello() {
        printf("hello\n");
}

int main(int argc, char** argv) {
        vimInit(argc, argv);
        sayHello();
}

using this command:
emcc -Iinclude -Iinclude/proto -DHAVE_CONFIG_H -fPIC main.c lib/libvim.a -o index.html

But then I got this error:
wasm-ld: error: unknown file type: libvim.o

This works fine using gcc:
# after running the same build script but with configure/make instead of emconfigure/emmake
gcc -Iinclude -Iinclude/proto -DHAVE_CONFIG_H main.c lib/libvim.a -lSM -lICE -lXt -lX11 -lXdmcp -lm -ltinfo -lnsl -ldl

Any idea what could be wrong? Did you have success using this library with WebAssembly yet? If so it would be great if you could add some directions how.

Emscripten does not seem to support ncurses. But, libvim is not responsible for rendering the user interface. So, I think we can skip this check.
After some tweaks, I succeeded in compiling libvim to WebAssembly. I tested part of the API (motion, update buffer callback, get vim mode, input and key press,...) and it works.
I have a question about the cursor position.
@bryphe In oni2, how is the cursor position handled after a vim motion. Do you check after each keyboard event whether the cursor position has changed? In the API, I cannot find any callback about the cursor position that would be triggered after a vim motion (w, h, j, gg,...)

After some tweaks, I succeeded in compiling libvim to WebAssembly. I tested part of the API (motion, update buffer callback, get vim mode, input and key press,...) and it works.

@elebeaup - very exciting! Thanks for exploring this and sharing what you've done so far. Would be interested to see it in action

In oni2, how is the cursor position handled after a vim motion? Do you check after each keyboard event if the cursor position has changed? In the API, I cannot find any callback about the cursor position that would be triggered after a vim motion (w, h, j, gg,...)

Good question - you're exactly right in terms of the strategy Onivim 2 uses: we get the cursor position with vimCursorGetLine/vimCursorGetColumn after running libvim's vimKey/vimExecute/vimCommand.

I played around with this a bit in December. I ended up ripping out some terminal-related stuff in order to get it to build. This is what I ended up with: https://github.com/titanous/libvim/commit/ffdb7d9ad6e22f0152b39aacffc23e261456c874
I haven't had time to fully test/integrate and clean this up, but I wanted to checkpoint here in case someone else got to it first.

You can find my work in progress here : https://github.com/elebeaup/libvim.
Compiling
$ emconfigure ./configure
$ emmake make libvim.js

Testing
$ cd wasm
$ npm install
$ npm test


"Educational Discord Token Grabber"🤓

My name is Gumbobrot.


Add StyleCop to all the projects.

You want me to merge this into master and then merge master into your PR to fix the conflicts there?

go for it - I can fix up my other branch to get used to the style settings


Extending a Type with a custom resolver

Currently, it's possible to extend only Query/Mutation types. I am aware that by spec it shouldn't be possible to extend other types. However, we can add only fields/relations to the schema which is rather limited compared to being able to define the whole schema by ourselves. By the spec, you can have fields with variable input on the type.
extend type User {
  calcSomething(input: MyInput): Result
}

In the end, this also gives great power if you are querying the list of objects and you want to include calculated value as well. Without this and defining query at root level it means you have to run query separately for each received id and that's more than suboptimal.
query AllUsersWithCalc($input: MyInput) {
  allUsers {
    id
    calcSomething(input: $input)
  }
}

The key aspect is that a processing function would need to receive nodeId of its parent. Besides that the feature request #318 needs to be solved first to allow access to all types within schema extension SDL.

That will be covered by custom field resolvers - duplicate of #56.

I'm trying to implement a function that applies custom linear weights to a set of stats and return the total computed value for each row.
Typically one would send the weights to the server in the request, i.e.:
# Linear weight to apply to each stat. Any field on Player can have a weight.
type Weights {
  stat1: Int,
  stat2: Int,
  ...
  statN: Int
}

# A 'Player' has stats.
type Player {
  stat1: Int,
  stat2: Int,
  ...
  statN: Int,

  #This field is the goal of the exercise:
  #Its value is given by: `SUM((stat1 * weight1),  ... (statN * weightN))`
  scrore: Int
}

type Query {
  getScores(weights: Weights, skip: Int, take: Int, orderBy: score_Desc): [Player!]!
}

This would be trivial in SQL:

SELECT stat1, stat1 * :weight1, stat2, stat2 * :weight2, ... statN, statN * :weightN
FROM PLAYERS
ORDER BY stat1 * :weight1 + stat2 * :weight2 + ... statN * :weightN DESC
LIMIT 10

I can eschew Prisma for this task and add a custom query + resolver in my application layer graphql server, but it would no longer conform to the API exposed by Prisma. Things like pagination and cursors would have to either be manually implemented or refactored into another paging mechanism. (i.e. the skip + take params above).
Ideally a lambda that gets evaluated for each record at data retrieval time would be ideal. Obviously I do not know what is needed to make that happen, but I figured explaining my use case would be useful to the discussion since it seems pretty representative of the issue.
BTW I love Prisma. Keep up the great work!

This is possible with new Nexus library
https://github.com/prisma/nexus-prisma


Making a pipeline marks other code sections as touched

Thanks for the quick turnaround on #3248 guys! I'm testing out RC3 now. I just ran into this:

When I type the code at the bottom, it marks several other (untouched) sections of the file as edited.

This is might happen in the current implementation. In order to format properly formatter is actually looking at a bigger range. At some point we should look into extracting smaller, original range from the formatted scope when applying change back to the document.

https://github.com/Microsoft/RTVS/commit/16149bd35ee00d90b6c942b59447030db52da752


docs: increased search input width in navbar component

Closes #3771
📝 Description

The search input looked so ugly when viewed in mobile width in Navbar components in With Search Input.

⛳ Current behavior (updates)

input in navbar looks ugly and not set properly (in style)

🚀 New behavior

The input now looks perfect on size

💣 Is this a breaking change (Yes/No):

No

📝 Additional Information

just changed some tailwind styles

Summary by CodeRabbit

New Features

Enhanced responsive design for the input field in the navbar, improving usability on small screens.
Updated margin behavior for the navbar brand, optimizing layout across different screen sizes.

Hi friends
1- OK I well check sm:min-w-min ( that was because in large screen the avatar got out )
2- I'm gonna change the min-w-[10rem] to min-w-[8rem]
I am using Firefox on Ubuntu could you please name your browser and OS

Hi devs
changed logo right margin in mobile screens to 0px as it's parent is justify-content: space-bertween; it works well
the bellow is 330px width screen and it works great
any changes you want I am fine and welcome :smile:


Raising errors for pyspark dataframe validation

Description
By default, pandera does not raise errors for pyspark DataFrame. Instead, it records validation errors within the df.pandera.errors attribute.
e.g.
df = metadata["pandera"]["schema"].validate(df)
df.pandera.errors
defaultdict(<function ErrorHandler.__init__.<locals>.<lambda> at 0x30ae9c550>, {'SCHEMA': defaultdict(<class 'list'>, {'WRONG_DATATYPE': [{'schema': 'IrisPySparkSchema', 'column': 'sepal_length', 'check': "dtype('StringType()')", 'error': "expected column 'sepal_length' to have type StringType(), got DoubleType()"}, {'schema': 'IrisPySparkSchema', 'column': 'sepal_width', 'check': "dtype('StringType()')", 'error': "expected column 'sepal_width' to have type StringType(), got DoubleType()"}, {'schema': 'IrisPySparkSchema', 'column': 'petal_length', 'check': "dtype('StringType()')", 'error': "expected column 'petal_length' to have type StringType(), got DoubleType()"}, {'schema': 'IrisPySparkSchema', 'column': 'petal_width', 'check': "dtype('StringType()')", 'error': "expected column 'petal_width' to have type StringType(), got DoubleType()"}]})})

As per pandera documentation:

This design decision is based on the expectation that most use cases for pyspark SQL dataframes means entails a production ETL setting. In these settings, pandera prioritizes completing the production load and saving the data quality issues for downstream rectification.

Context
Currently, validating pyspark DataFrames directly is not possible, except by manually inspecting the pandera.error attribute.
Possible Implementation
To enforce immediate error raising during validation, one can set lazy=False when calling the validation method: metadata["pandera"]["schema"].validate(data, lazy=False)
This setting might be more suitable for machine learning tasks. Alternatively, validation can be toggled off using the environment variable export PANDERA_VALIDATION_ENABLED=false, as mentioned in the docs and #27

Hi @michal-mmm, I like the idea of adding lazy=False when calling the validation method. We should also evaluate including tests with a PySpark dataset (and even others like Polars) in order to check that errors are raised.
In the future, we should evaluate how to handle validations with lazy=True, for instance, with an after-pipeline-run hook.

Hi @michal-mmm, could you make the PR with that change? And see what @Galileo-Galilei thinks about it. I am happy to help if you can't

Hi, sorry for not responding earlier. I think we should go forward. I suggest that we implement in general some kwargs to be passed to the validate function :
my_dataset:
    type: ...
    filepath: ...
    metadata:
        pandera:
            schema: ...
            validate_kwargs:
                lazy: true

and then in the hook:
metadata["pandera"]["schema"].validate(data, **metadata["pandera"]["validate_kwargs"])
Feel free to open to a PR, and eventually suggest a different design.

Closed by #78


Cannot detect other users when having multiple network interfaces

I have VMWare installed so I have multiple enabled network interfaces,
in that case termchat seems can not detect other users.
I noticed there is an issue(https://github.com/lemunozm/termchat/issues/1#issuecomment-710236683) talked about it and it should have been fixed but not work on my side.

Hi!
I'm not maintaining termchat now, and I have no time to dig into it. Sorry!
Nevertheless, if you or others want to dig into it and propose a fix, I'll happily merge it 😃


SQL types to D types documentation

Whenever I write an SQL query, I kind of have to guess what types my variables storing the result should have (or what I'll use as the .get argument.
Looking through the code, I extracted this mapping info:
TINYINT -> (u)byte
SMALLINT -> (u)short
MEDIUMINT -> (u)int
INT -> (u)int
BIGINT -> (u)long
FLOAT -> float
DOUBLE -> double
TIMESTAMP -> DateTime
TIME -> TimeOfDay
YEAR -> ushort
DATE -> Date
DATETIME -> DateTime
VARCHAR, ENUM, SET, VARSTRING, STRING, DECIMAL, NUMERIC -> string
BIT -> bool
TINYBLOB, MEDIUMBLOB, BLOB, LONGBLOB -> string / ubyte[] (?)

But I don't know a good place to put it. The readme? Somewhere in the docs?

On October 6, 2017 3:20:17 PM EDT, "Mathias L. Baumann"<EMAIL_ADDRESS>wrote:

Whenever I write an SQL query, I kind of have to guess what types my
variables storing the result should have (or what I'll use as the
.get argument.
Looking through the code, I extracted this mapping info:
TINYINT -> (u)byte
SMALLINT -> (u)short
MEDIUMINT -> (u)int
INT -> (u)int
BIGINT -> (u)long
FLOAT -> float
DOUBLE -> double
TIMESTAMP -> DateTime
TIME -> TimeOfDay
YEAR -> ushort
DATE -> Date
DATETIME -> DateTime
VARCHAR, ENUM, SET, VARSTRING, STRING, DECIMAL, NUMERIC -> string
BIT -> bool
TINYBLOB, MEDIUMBLOB, BLOB, LONGBLOB -> string / ubyte[] (?)

But I don't know a good place to put it. The readme? Somewhere in the
docs?

Good point. That info should probably be predominant enough to trip over so no one has to go looking for it.
I'm thinking maybe:

In the readme right between the "primary interfaces" and "basic example" section.

and also in:

a ddoc macro defined somewhere common and instantiated in all relevant parts of the docs, like here:

http://semitwist.com/mysql-native-docs/v1.0.0/mysql/commands.html
and
http://semitwist.com/mysql-native-docs/v1.0.0/mysql/prepared.html
and the docs for all the various query/exec functions (and anything else relevant)
Alternatively, instead of #2 a special page could be added to the docs for this chart, and all those doc pages could link to it. It should probably still go in the readme too.


update CI

[ ] replace bors with merge queue, rust-embedded/wg#671
[ ] replace actions-rs with alternative

cc @rust-embedded/tools

I can do this, should be relativly simple, should do the same on rust-embedded/svd also
the wg has not unified on a guideline for how to do this transition, but riscv has transitioned, we can also do so


Support for porject hierarchies

In Tableau 10.5 there is now a new feature that allows to make nested projects https://www.tableau.com/about/blog/2017/12/get-ready-tableau-105-projects-within-project-78906.
But here in TSC the ProjectItem does not contain any properties that can show where the project is (is it root or is it subproject).
Good idea is to add parent_project attribute to the ProjectItem.
Is this possible and is this requires to change Tableau API because just now I can't find related method in Tableau API.

https://github.com/tableau/server-client-python/commit/0b497d929fbbf971cfa1960ba09f79d65c2e9e25
Added support for 'parent_id' which identifies the parent project for a child project

Thanks, but parent_id alwais None.
I think there are two reasons:

TSC uses 2.3 version of API and only in 2.8 there is parentProjectId attribute in response XML.
Method _parse element uses not correct name - it uses "parentId" instead of "parentProjectId"

def _parse_element(project_xml):
  id = project_xml.get('id', None)
  ...
  parent_id = project_xml.get('parentId', None)

Is it possible to move to 2.8 version of API?

Make a PR https://github.com/tableau/server-client-python/pull/267

PR was merged log ago/ Cleaning up old issue.


.pypirc/pip.ini location on windows vs distutils

This:
python -c "import distutils.dist as dist; import distutils.config as cfg; print(cfg.PyPIRCCommand(dist.Distribution())._get_rc_file())"

outputs
C:\Users\thebjorn\.pypirc

ie. %USERPROFILE%\.pypirc and I can only upload to pypi if my credentials are listed here.
However, if I want to add a private repo, i.e.
[install]
index-url =<EMAIL_ADDRESS>
then this only works when added to c:\programdata\pip\pip.ini.
It's a wee bit confusing...

That's 2 different usages, with 2 different programs: ~/.pypirc is used by the setuptools upload command (deprecated) and twine, while pip.ini is for configuring pip (and you only did so for the install command, so wheel will still hit the standard PyPI index).


chore: after publish merge

<EMAIL_ADDRESS><EMAIL_ADDRESS>
Related issue

Scope of work

Screenshots of visual changes
Checklist

[ ] I followed composition rules for my component
[ ] I tested the component in most common device sizes (can be tested in Storybook from viewport addon in top menu)

never mind


Readd z

It seems like fasd is no longer maintained, as last commit was 2 years ago.
Would it be possible to readd z? (which is actively maintained)

z.lua is faster and more convenient than fasd. Maybe we can replace fasd with z.lua.

+1 for z

There seems to be a more actively maintained fork https://github.com/whjvenyl/fasd

nice code

din


Add tooltip to BirdsEyeView and adjust styling

Resolves #994 in conjunction with this PR from web-components.
This PR adds a tooltip to the top right corner of the BirdsEyeView and adds a border and background color to match the VarCompletenessTable.
In doing this work, I noticed that PlotLegend was being passed into PlotLayout as a fragment even if no overlay variable was selected. This causes an unnecessary top margin on the BirdsEyeView which means it's not flush to the top of the plotted visualization. I added ternaries in the visualization implementations to prevent the behavior.
Shows the extra space that renders above the BirdsEyeView

Shows updated version without the extra space

I don't think this is quite there. I'd actually prefer a white background, more padding, and box-shadow (similar to the plot). These thoughts are inspired by this issue.
Current:

Proposed (roughly):

To that end, I suppose we should wait until the UX meeting to reach a final verdict. I don't have strong opinions either way.


LuaFileSystem-like wrapper for fs bindings compatability

for luacheck, ...

Could you explain? Are you referring to https://github.com/mpeterv/luacheck/blob/master/src/luacheck/fs.lua ?

I mean to support that and similar projects, we could extend fs bindings (which mostly replace these fs.lua utils) to provide these functions https://keplerproject.github.io/luafilesystem/manual.html#reference

actually, instead of a backwards compatibility feature for the fs bindings, I think a forward comparability regex replacer to port old lfs apps to torchbear would make the most sense


Got "writing pacts for ... to target/pacts" from logger but no such file found

I'm using "io.github.jbwheatley" %% "pact4s-scalatest" % "0.1.0-java8"
During running test, I got this Info log
[info] Pact4s-Logger - Writing pacts for consumer Consumer(name=FraudBackofficeAPI) and provider Provider(name=FraudMlAPI) to target/pacts
However there is no such file exist in target/pacts like log mentioned. Not in my target/pacts path, and not anywhere else. Seem like it's not generated. What's wrong here?
below is my code for my test class
class PactTest extends AsyncFunSuite with Matchers with RequestResponsePactForger with Mockito with MockWSHelpers {

  val clientId = 2

  override def pact: RequestResponsePact = ConsumerPactBuilder
    .consumer("FraudBackofficeAPI")
    .hasPactWith("FraudMlAPI")
    .uponReceiving("getAllNameNationalityProbability")
    .path("http://test:port/predict-nationality")
    .method("POST")
    .headers(
      Map(
        "Content-Type" -> "application/json",
        "ClientId" ->  clientId.toString,
        "RequestId" -> "123e4567-e89b-42d3-a456-556642440000"
      )
    )
    .body(
      """{"name":"John Doe",
        |"maxResult":3}
        |""".stripMargin,
      "application/json"
    )
    .willRespondWith()
    .status(200)
    .body("""{
            |  "ARE": 0.0015790796301290394,
            |  "ARG": 0.0015750796301290393,
            |  "AUS": 0.027795471251010895
            |}
            |""".stripMargin)
    .toPact

  test("getAllNameNationalityProbability should return response for any name") {

    val config = mock[FraudBackofficeConfig]
    when(config.fraudMlServerConfig)
      .thenReturn(FraudMlServerConfig(Seq("http://test:port"), 2))

    val ws = MockWS {
      case ("POST", "http://test:port/predict-nationality") =>
        Action {
          Ok(Json.obj(
            "ARE" -> 0.0015790796301290393,
            "ARG" -> 0.0015750796301290393,
            "AUS" -> 0.027795471251010895
          ))
        }
    }

    val fraudMlClient = new FraudMlServerClientImpl(config, ws)
    val getNameNat = fraudMlClient.getAllNameNationalityProbability("John Doe")
    assert(getNameNat != null)
  }

}

Hi @tanyagorn, I've checked this on the latest java8 tag (0.2.0-java8) and the one you're one, and I can see the pact files being written in our tests. Is it being written to anywhere on your file system? Could you try explicitly setting the destination like we do here https://github.com/jbwheatley/pact4s/blob/080b233dbc3515ebb6a953781c18634e2dbd07aa/example/consumer/src/test/scala/http/consumer/ScalaTestPact.scala#L23 and see if that solves your issue

you might also find it useful to enable debug logging for pact-jvm - au.com.dius.pact is the package you would want to enable

Hi @jbwheatley, I've enable debug logging as you suggested.
This is the result

There is still no pact file generated from my pact test. So, I try enable debug log in a project that pact test is working and this is the result. (it's different test in different project)

Comparison between logging from two projects that I've noticed.

After mock server is started, success pact run will have BaseJdkMockServer - Received request with POST method along with info.
Before mock server is shutdown, success pact run will have BaseMockServer write .json file to target/pacts directory
However, in the project that there is no pact file generated. The mock server is start and then shutdown without doing anything.

Seem like there is no request sending to mock server. But I've already called the method though. Why?

Ok, it seems that pact-jvm won't write the pact file unless the mock server has received requests for all of the defined interactions - so because the request is not being sent, the pact file is not being written

it doesn't look like in your test class that the request is being sent to the mock server (sorry if it is and i'm just being blind). you can get the url by calling mockSever.getUrl from within the test class.

sorry, I don't recognise what http client library you are using - if you could minimise an example with dependencies that recreates your issue, it will be easier for me to help debug 😺


spaming console "path to 'Plugins/TradeSystem/tradelog.db': '/home/container/Plugins' does not exist"

Describe the bug
spaming console
[23:03:53 ERROR]: path to 'Plugins/TradeSystem/tradelog.db': '/home/container/Plugins' does not exist
[23:03:53 ERROR]: path to 'Plugins/TradeSystem/tradelog.db': '/home/container/Plugins' does not exist
[23:03:53 ERROR]: path to 'Plugins/TradeSystem/tradelog.db': '/home/container/Plugins' does not exist
[23:03:53 ERROR]: path to 'Plugins/TradeSystem/tradelog.db': '/home/container/Plugins' does not exist
[23:03:53 ERROR]: path to 'Plugins/TradeSystem/tradelog.db': '/home/container/Plugins' does not exist
[23:03:53 ERROR]: path to 'Plugins/TradeSystem/tradelog.db': '/home/container/Plugins' does not exist
[23:03:53 ERROR]: path to 'Plugins/TradeSystem/tradelog.db': '/home/container/Plugins' does not exist
[23:03:53 ERROR]: path to 'Plugins/TradeSystem/tradelog.db': '/home/container/Plugins' does not exist
[23:03:53 ERROR]: path to 'Plugins/TradeSystem/tradelog.db': '/home/container/Plugins' does not exist
[23:03:53 ERROR]: path to 'Plugins/TradeSystem/tradelog.db': '/home/container/Plugins' does not exist
[23:03:53 ERROR]: path to 'Plugins/TradeSystem/tradelog.db': '/home/container/Plugins' does not exist
[23:03:53 ERROR]: path to 'Plugins/TradeSystem/tradelog.db': '/home/container/Plugins' does not exist
[23:03:53 ERROR]: path to 'Plugins/TradeSystem/tradelog.db': '/home/container/Plugins' does not exist
[23:03:53 ERROR]: path to 'Plugins/TradeSystem/tradelog.db': '/home/container/Plugins' does not exist
[23:03:53 ERROR]: path to 'Plugins/TradeSystem/tradelog.db': '/home/container/Plugins' does not exist
[23:03:53 ERROR]: path to 'Plugins/TradeSystem/tradelog.db': '/home/container/Plugins' does not exist
[23:03:53 ERROR]: path to 'Plugins/TradeSystem/tradelog.db': '/home/container/Plugins' does not
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.
Discord contact: #

More information would be great. I've never seen that before.

I have the same issue.
The problem is that my folder name is plugins, not Plugins.
My error: [Mon, 26. Apr 2021 09:49:23 UTC ERROR] path to 'Plugins/TradeSystem/tradelog.db': '/var/lib/pufferpanel/servers/01b8147b/Plugins' does not exist
The directory /var/lib/pufferpanel/servers/01b8147b/Plugins doesn't exist but /var/lib/pufferpanel/servers/01b8147b/plugins exists. Probably it's a difference between Windows/Linux handling of uppercase folder names.

I have the same issue.
The problem is that my folder name is plugins, not Plugins.
My error: [Mon, 26. Apr 2021 09:49:23 UTC ERROR] path to 'Plugins/TradeSystem/tradelog.db': '/var/lib/pufferpanel/servers/redacted/Plugins' does not exist
The directory /var/lib/pufferpanel/servers/redacted/Plugins doesn't exist but /var/lib/pufferpanel/servers/redacted/plugins exists. Probably it's a difference between Windows/Linux handling of uppercase folder names.

Thank you for your explanation. Gonna fix that!


Thoughts on possible alternative traits

If our current use of traits doesn’t hit the spot, maybe we can consider this approach…
If I image a rack of devices all doing the same function (e.g. SDI to 2110-???), in terms of configuration there is some settings that all units will have (e.g. codec type), some that is specific to the device (e.g. device name, serial number) and some that is specific to the streams that are being sent/received (e.g. IP addresses, lost packet accounts, certain tags).  That suggests to me that there are three categories of information: Common settings (let’s call them General), Device Specific (let’s call them Device) and Stream specific (let’s call them Stream).
Then there is stuff that a controller can change and that which a controller can’t change.  Let’s call these Writable and Read-Only.
Combining these, we get categories of Device Writable(e.g. device name), Device Read-Only  (e.g. serial number), General Writable (e.g. SIPS/FEC settings) , General Read-Only(e.g. IS-04 UUID?), Stream Writable (e.g. IP addresses) and Stream Read-Only(e.g. lost packet counts).
A Backup Data Set would include properties with all six categories, but when restoring a Data Set a user would specify one or more of Device, General, Stream.  Properties with a Read-Only category would be ignored when restoring and no warning would be generated.
A full kit replacement restore would select all three categories.
A kit parking or change of program would likely only select Stream.
A mass-provisioning via cloning would initially just select General(or possibly also Stream with some default values).
Jeff’s suggestion of rebuilds vs. restores plays in well to this.  To make it easier for me, building on the terminology of C++ compilers, how do people feel about calling these a clean restore and an incremental restore?  These could be flagged as part of the restore operation.  (A mass-provisioning could do a clean restore with just the General properties so wouldn’t technically be a rebuild.)

We have since removed the concept of traits and made it the responsibility of a client to manipulate the backup data set for any intended use.


StackLlaMa 2 dpo train with deepspeed oom

System Info
transformers             4.37.2
accelerate               0.26.1
peft                     0.8.2
bitsandbytes             0.43.0.dev0 # latest built from source
trl                      0.7.11.dev0 # latest built from source
torch                    2.2.0
python 3.9.18

Information

[ ] The official example scripts
[ ] My own modified scripts

Tasks

[ ] One of the scripts in the examples/ folder of Accelerate or an officially supported no_trainer script in the examples folder of the transformers repo (such as run_no_trainer_glue.py)
[ ] My own task or dataset (give details below)

Reproduction
git clone https://github.com/huggingface/trl.git

accelerate launch --config_file examples/accelerate_configs/deepspeed_zero3.yaml examples/research_projects/stack_llama_2/scripts/dpo_llama2.py \
    --model_name_or_path="sft/final_checkpoint" \
    --output_dir="dpo" \
    --report_to="tensorboard"

Traceback (most recent call last):
  File "/nas/lili/codes/pt/ft/trl/examples/research_projects/stack_llama_2/scripts/dpo_llama2.py", line 214, in <module>
    dpo_trainer = DPOTrainer(
  File "/nas/lili/codes/pt/ft/trl/trl/trainer/dpo_trainer.py", line 234, in __init__
    model = prepare_model_for_kbit_training(model, **prepare_model_kwargs)
  File "/home/ubuntu/.cache/pypoetry/virtualenvs/ft-zSqjAXBp-py3.9/lib/python3.9/site-packages/peft/utils/other.py", line 105, in prepare_model_for_kbit_training
    param.data = param.data.to(torch.float32)
torch.cuda.OutOfMemoryError: CUDA out of memory. Tried to allocate 500.00 MiB. GPU 0 has a total capacity of 39.39 GiB of which 469.00 MiB is free. Process 70022 has 4.62 GiB memory in use. Including non-PyTorch memory, this process has 4.61 GiB memory in use. Process 70021 has 4.62 GiB memory in use. Process 70018 has 4.62 GiB memory in use. Process 70019 has 4.62 GiB memory in use. Process 70015 has 5.59 GiB memory in use. Process 70017 has 4.62 GiB memory in use. Process 70016 has 5.59 GiB memory in use. Of the allocated memory 3.99 GiB is allocated by PyTorch, and 145.24 MiB is reserved by PyTorch but unallocated. If reserved but unallocated memory is large try setting PYTORCH_CUDA_ALLOC_CONF=expandable_segments:True to avoid fragmentation.  See documentation for Memory Management  (https://pytorch.org/docs/stable/notes/cuda.html#environment-variables)
Traceback (most recent call last):
  File "/nas/lili/codes/pt/ft/trl/examples/research_projects/stack_llama_2/scripts/dpo_llama2.py", line 214, in <module>
    dpo_trainer = DPOTrainer(
  File "/nas/lili/codes/pt/ft/trl/trl/trainer/dpo_trainer.py", line 234, in __init__
    model = prepare_model_for_kbit_training(model, **prepare_model_kwargs)
  File "/home/ubuntu/.cache/pypoetry/virtualenvs/ft-zSqjAXBp-py3.9/lib/python3.9/site-packages/peft/utils/other.py", line 105, in prepare_model_for_kbit_training
    param.data = param.data.to(torch.float32)
torch.cuda.OutOfMemoryError: CUDA out of memory. Tried to allocate 500.00 MiB. GPU 0 has a total capacity of 39.39 GiB of which 477.00 MiB is free. Process 70022 has 4.62 GiB memory in use. Process 70020 has 4.61 GiB memory in use. Process 70021 has 4.62 GiB memory in use. Including non-PyTorch memory, this process has 4.61 GiB memory in use. Process 70019 has 4.62 GiB memory in use. Process 70015 has 5.59 GiB memory in use. Process 70017 has 4.62 GiB memory in use. Process 70016 has 5.59 GiB memory in use. Of the allocated memory 3.99 GiB is allocated by PyTorch, and 145.24 MiB is reserved by PyTorch but unallocated. If reserved but unallocated memory is large try setting PYTORCH_CUDA_ALLOC_CONF=expandable_segments:True to avoid fragmentation.  See documentation for Memory Management  (https://pytorch.org/docs/stable/notes/cuda.html#environment-variables)
Traceback (most recent call last):

I have 8 a100 40GB gpus. I think for llama2-7b, it's enough. And I checked the yaml config, it has "zero3_init_flag: true". So I think it will not load the whole model to a single gpu/device but load each shard its own parameters.
But in peft/utils/other.py
    if not is_gptq_quantized:
        # cast all non INT8 parameters to fp32
        for param in model.parameters():
            if (param.dtype == torch.float16) or (param.dtype == torch.bfloat16):
                param.data = param.data.to(torch.float32)

it seems peft need cast bf16 parameter to fp32. When it ran, I saw there are 8 processes ran in gpu0 and the gpu0 memory is used up and it failed.
So I guess the peft don't support shard parameters to 8 gpus but load all to a single gpu.
Expected behavior
no exception

Duplicate of https://github.com/huggingface/peft/issues/1507. Please only open one issue in the future :)


[Bug] dss登录exchangis显示未登录

Search before asking

[X] I searched the issues and found no similar issues.

Exchangis Component
exchangis-web
What happened + What you expected to happen

exchangis1.0.0
dss1.1.1
linkis 1.2.0
Relevent platform
exchangis1.0.0
dss1.1.1
linkis 1.2.0
Reproduction script
You are not logged in
Anything else
You are not logged in
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

https://github.com/WeBankFinTech/DataSphereStudio/issues/933
确实是之前数据库里写了ip，访问用了域名，导致cook ie拿不到问题，后来改ip就没问题了

响应数据提示未登入
{"method":"/api/rest_j/v1/dss/exchangis/main/projects","status":-1,"message":"You are not logged in, please login first!(您尚未登录，请先登录!)","data":{}}
访问的ip已经与dss_appconn_instance中配置的一致，请求中已经添加了cookie
Cookie: workspaceId=224; workspaceName=bdapWorkspace; linkis_user_session_ticket_id_v1=M7UZXQP9Ld10ZF3+e15afkDSc4/GFINY2HLiVea4FcQ=


Run phpstan under webroot

Currently, phpstan only works when running the command in the directory where Drupal is located.
Is it possible to run the command under the webroot? e.g.:
www/vendor/bin/phpstan analyse www/modules/custom

PHPStan needs to be able to load the autoloader for Drupal. So you must run PHPStan from the root of your Drupal project so it can detect the autoloader.
Now, I wonder if we could actually improve this:
        $finder = new DrupalFinder();
        $finder->locateRoot(dirname($GLOBALS['autoloaderInWorkingDirectory'], 2));
        $this->drupalRoot = $finder->getDrupalRoot();

Instead of basing it off of autoloaderInWorkingDirectory, I wonder if we could determine the Drupal root based off of the provided path.  We would then need to include the autoloader if it was not.
This is some of the magic in PHPStan
$autoloaderInWorkingDirectory = getcwd() . '/vendor/autoload.php';
if (is_file($autoloaderInWorkingDirectory)) {
	require_once $autoloaderInWorkingDirectory;
}

$composerAutoloadFile = __DIR__ . '/../vendor/autoload.php';
if (!is_file($composerAutoloadFile)) {
	$composerAutoloadFile = __DIR__ . '/../../../autoload.php';
}

require_once $composerAutoloadFile;

I'll try and experiment. But it might be possible to fix this by taking in the target path's folder and passing that to DrupalFinder.

Drupal Check https://github.com/mglaman/drupal-check does this. But that doesn't allow for customized PHPStan config, it started as mostly a helper for deprecation discovery

Reason i’m asking: We use GrumPHP (https://github.com/phpro/grumphp) to run some checks when committing. The commands are run from the directory where the .git directory is located. For us, this is under the webroot.

I created PR #50.
Instead of using  $GLOBALS['autoloaderInWorkingDirectory'], I used $GLOBALS['composerAutoloadFile'] which did the trick for me.

I'll begin reviewing the PR.  It fractures discussions and code reviews.

Per my comments on the PR, I think the only fix is allowing an explicit parameter definition which specifies where Drupal's root is located and we detect the autoloader based off of that path.

When using the master-branch, this works using the following phpstan.neon file:
parameters:
	level: 7
drupal:
    drupal_root: %currentWorkingDirectory%/htdocs
includes:
	- htdocs/vendor/mglaman/phpstan-drupal/extension.neon

:) @JTubex cool, you beat me to that. I was going to commit and document this and mention you. Thanks for confirming


Add the ability to set boolean block settings values directly

Allows for stuff like QuiltBlockSettings.copyOf(Blocks.VINE).ticksRandomly(false)

@EnnuiL
I got rid of the variable, I don't see why the other one needs to be renamed, but if you want a different name, feel free to suggest one.

@EnnuiL I got rid of the variable, I don't see why the other one needs to be renamed, but if you want a different name, feel free to suggest one.

Yeah, renaming it was only something to consider, however, yeah, maybe the current one is good enough


Can't recover player if stream is stalled.

Not sure if this is a bug or an improvement.
If a live audio stream goes into a stalled state, like when the media can't be found (because the stream is not on or net error), the player can't be reset.  The progress bar won't stop showing the striped 'searching' state.  Toggling Play does not reset the player.  It doesn't seem to timeout and reset.  I have to refresh the web page, which is not very user-friendly.
If there is an option or API to reset the player, please inform.

I think this issue is not a bug or anything related to Plyr. You should use a library whose handle stream like hls.js  so pass hls object to plyr, problably is what you are doing.
Hls provide many listeners to detect interruption, network errors etc. You can handle stream errors there and recover your stream, plyr automatically detect when hls recover its stream.
Best regards


Adding a universal comment for all programming languages

I am suggesting this to more developers, maybe someone will come with the perfect solution and everyone will add it in their programming language. Thanks

Thanks for sharing this idea.
We won't keep this as an open issue in the issue tracker because it's not actionable. In case you want to start a conversion about this, please go to the forums at https://forum.crystal-lang.org/


[REQ] Disable workflow summary creation

Is your request related to a workflow problem?
Incremental scan output summary can be misleading to user by showing 0 vulnerabilities when other vulnerabilities already exist. Could we have an option to not generate the summary.
Propose a solution
Add a new parameter to the action, to disable the creation of a workflow summary.

Thank you for bringing this to our attention. To ensure I understand correctly, are you referencing instances where the Incremental scan shows "0 vulnerabilities" even when vulnerabilities exist from previous scans? Is this in comparison to the results from the last full scan?
Furthermore, could you provide more insight into your perspective on the value of a scan that doesn't display any results?
Thank you for your input!

The issue occurs when a scan is launched asynchronously:
By the time the CLI tries to retrieve scan's results, it gets nothing (as the scan is not complete yet), and generate an output with 0 vulnerabilities.
I already proposed a simple code change to prevent this behaviour (see PR https://github.com/Checkmarx/ast-github-action/pull/134)

Hi @ArnaudLacroix
We fixed this in the latest version.
Thanks.


Fixed #25454 -- Ensured register_hstore_handler is called for all connections

I've also just added the signal disconnection, as it was called many times needlessly in other non-related tests.


No module named bbox when running faster-cnn exampl

trying to reproduce the faster-rcnn tutorial as in http://mxnet.io/tutorials/computer_vision/detection.html
running
python train_alternate.py --gpus 0
yields this error:
Traceback (most recent call last):
File "train_alternate.py", line 7, in
from rcnn.tools.train_rpn import train_rpn
File "/mnt/gelu/mxnet/example/rcnn/rcnn/tools/train_rpn.py", line 7, in
from ..symbol import *
File "/mnt/gelu/mxnet/example/rcnn/rcnn/symbol/init.py", line 1, in
from symbol_vgg import *
File "/mnt/gelu/mxnet/example/rcnn/rcnn/symbol/symbol_vgg.py", line 2, in
import proposal
File "/mnt/gelu/mxnet/example/rcnn/rcnn/symbol/proposal.py", line 11, in
from rcnn.processing.bbox_transform import bbox_pred, clip_boxes
File "/mnt/gelu/mxnet/example/rcnn/rcnn/processing/bbox_transform.py", line 2, in
from ..cython.bbox import bbox_overlaps_cython
ImportError: No module named bbox
I've tried searching on bbox module but could not find it anywhere. My mxnet is compiled in ubuntu 16 with cuda and cdnn support

Please follow the readme you need to compile extra operators

Thanks for the reply,

Emm...Are you ok?
I had the same problem.Could you please tell me how to deal with the things?
Look this step:
bash script/vgg_voc07.sh 0
reply:
Traceback (most recent call last):
File "train_end2end.py", line 8, in
from rcnn.symbol import *
File "F:\mxnet\example\rcnn\rcnn\symbol_init_.py", line 1, in
from symbol_vgg import *
File "F:\mxnet\example\rcnn\rcnn\symbol\symbol_vgg.py", line 2, in
import proposal
File "F:\mxnet\example\rcnn\rcnn\symbol\proposal.py", line 11, in
from rcnn.processing.bbox_transform import bbox_pred, clip_boxes
File "F:\mxnet\example\rcnn\rcnn\processing\bbox_transform.py", line 2, in
from ..cython.bbox import bbox_overlaps_cython
ImportError: No module named bbox
Traceback (most recent call last):
File "test.py", line 4, in
from rcnn.tools.test_rcnn import test_rcnn
File "F:\mxnet\example\rcnn\rcnn\tools\test_rcnn.py", line 6, in
from ..symbol import *
File "F:\mxnet\example\rcnn\rcnn\symbol_init_.py", line 1, in
from symbol_vgg import *
File "F:\mxnet\example\rcnn\rcnn\symbol\symbol_vgg.py", line 2, in
import proposal
File "F:\mxnet\example\rcnn\rcnn\symbol\proposal.py", line 11, in
from rcnn.processing.bbox_transform import bbox_pred, clip_boxes
File "F:\mxnet\example\rcnn\rcnn\processing\bbox_transform.py", line 2, in
from ..cython.bbox import bbox_overlaps_cython
ImportError: No module named bbox
Why closed the issue? @Gelu74 @piiswrong

That's all right.
I found ..cython.bbox.pyx and setup.py
then I open cmd and input python setup.py install
get another error:
EnvironmentError: The CUDA nvcc path could not be located in C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v8.0\bin\nvcc
but I really don't have folder-nvcc,I have nvcc.exe and nvcc.profile
who knows where is the folder?

I have met the same error with you.Have you solved this problem? @lightningsoon

use linux to slove
发自网易邮箱大师
On 03/02/2017 15:20, niluanwudidadi wrote:
I have met the same error with you.Have you solved this problem? @lightningsoon
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Is there a way to unmarshal JSON into a dns.Msg

Hi,
It appears that a JSON-marshalled dns.Msg can't be reliably converted back to dns.Msg using the default Marshaller. It throws this error
cannot unmarshal object into Go struct field Msg.Answer of type dns.RR

Is there a way to reliably do this? I reckon we might need to write separate JSON unmarshal functions for each RR and add that function to the interface? I haven't tested that though

DNS and json are not friends. However when using this lib for the conversion, why wouldn't the Unmarshal just work, i.e. what's missing?

https://go.dev/play/p/pNDALJt0jI6
this is just an example. the Answer section remains nil. what would you recommend for serializing the DNS object? I want to put them on a Kafka cluster and pick them up  somewhere else.

ah, so this would mean every RR type needs a json unmarshal function, which is not going to happen.
You could convert to text or just the binary and pick that up on the other side.

just for future reference, I made a slow and dirty unmarshaller that actually spits out the equivalent dns.Msg-Marshalled object here. Super ugly and slow (~25k per second) but gets the job done


Compilation fails on tsc versions >= 4.2.0

When running on 4.2.0 or greater:
src/lib/misc/type-helpers.ts:20:39 - error TS2344: Type 'KnownKeys<T>' does not satisfy the constraint 'keyof T'.
  Type 'unknown' is not assignable to type 'keyof T'.

20 export type RemoveIdxSgn<T> = Pick<T, KnownKeys<T>>
                                         ~~~~~~~~~~~~

src/lib/misc/type-helpers.ts:22:40 - error TS2344: Type 'NeverKeys<T>' does not satisfy the constraint 'keyof T'.
  Type 'unknown' is not assignable to type 'keyof T'.

22 export type OmitNeverKeys<T> = Pick<T, NeverKeys<T>>
                                          ~~~~~~~~~~~~

src/lib/server/call.ts:129:9 - error TS2322: Type 'KnownKeys<{ [M in keyof T]: MethodDef2ServiceHandler<T[M], Extension>; }>' is not assignable to type 'string | number | symbol'.
  Type 'unknown' is not assignable to type 'string | number | symbol'.
    Type 'unknown' is not assignable to type 'symbol'.
      Type 'KnownKeys<{ [M in keyof T]: MethodDef2ServiceHandler<T[M], Extension>; }>' is not assignable to type 'symbol'.
        Type 'unknown' is not assignable to type 'symbol'.

129   [M in keyof ServiceImplementation<T, Extension>]:
            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Thanks for reporting, will try to fix within the following days.

Thanks for reporting, fix is on the way. I could not find the breaking change in TS release notes, so I reported an issue for that: https://github.com/microsoft/TypeScript/issues/43357.

Released with 0.6.2


⚠ Bofrost.de has degraded performance

In 45d1df5, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1832 ms

Resolved: Bofrost.de performance has improved in 72444f6 after 13 minutes.


Element.getElementsByTagName

Element.getElementsByTagName

This issue was imported from Google Moderator
Moderator votes: +8
Issue added by Sam on 2014-04-16
To vote this issue up or down, simply include +1 or -1 in your comment.

Fully supported.
Can now be found by search at http://caniuse.com/#search=getElementsByTagName


ci: try run appveyor in 2 processes

Codecov Report

Merging #3586 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #3586   +/-   ##
=======================================
  Coverage   98.65%   98.65%
=======================================
  Files          18       18
  Lines        1114     1114
  Branches      300      300
=======================================
  Hits         1099     1099
  Misses         13       13
  Partials        2        2

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 65d318d...4694ad0. Read the comment docs.


⚠ Home 主页 has degraded performance

In 6ead60b, Home 主页 (https://www.fangkehou.eu.org) experienced degraded performance:

HTTP code: 200
Response time: 20451 ms

Resolved: Home 主页 performance has improved in ab58c00 after 10 minutes.


Add nats-streaming config with unlimited max_payload

nats-streaming | [1] 2019/05/29 09:47:30.255870 [INF] STREAM: ---------- Store Limits ---------- nats-streaming | [1] 2019/05/29 09:47:30.255889 [INF] STREAM: Channels:                  100 * nats-streaming | [1] 2019/05/29 09:47:30.255894 [INF] STREAM: --------- Channels Limits -------- nats-streaming | [1] 2019/05/29 09:47:30.255898 [INF] STREAM:   Subscriptions:          1000 * nats-streaming | [1] 2019/05/29 09:47:30.255903 [INF] STREAM:   Messages     :     unlimited nats-streaming | [1] 2019/05/29 09:47:30.255908 [INF] STREAM:   Bytes        :     unlimited nats-streaming | [1] 2019/05/29 09:47:30.255913 [INF] STREAM:   Age          :     unlimited * nats-streaming | [1] 2019/05/29 09:47:30.255920 [INF] STREAM:   Inactivity   :     unlimited * nats-streaming | [1] 2019/05/29 09:47:30.255927 [INF] STREAM: ----------------------------------