Fixes #82

Fixes #82 by checking if the REQUESTS_CA_BUNDLE env variable is set, otherwise use conda certificates.

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe:

License is not an SPDX identifier (or a custom LicenseRef) nor an SPDX license expression.

Documentation on acceptable licenses can be found here.

@conda-forge-admin, please rerender

Seems reasonable to me

Thanks @j-4!

I have tried it and the changes did not fix the problem because the variable is read in the build step, but it needs to be set after "installation".
So the settings should be set post-linkage via a post-linkage script such that git config is written during the installation process.
I am preparing a PR.


get latest master

Can one of the admins verify this patch?

Codecov Report

:exclamation: No coverage uploaded for pull request base (Versioning@ba63316). Click here to learn what that means.
The diff coverage is 100%.

@@              Coverage Diff              @@
##             Versioning    #6108   +/-   ##
=============================================
  Coverage              ?   66.75%
=============================================
  Files                 ?       35
  Lines                 ?     1814
  Branches              ?        0
=============================================
  Hits                  ?     1211
  Misses                ?      603
  Partials              ?        0

Impacted Files
Coverage Δ

sdk/core/azure-core/azure/core/version.py
100% <100%> (ø)

sdk/core/azure-core/azure/core/settings.py
97.61% <100%> (ø)

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 ba63316...b9e3cd7. Read the comment docs.


typo: Double word "is"

@nschonni ... Can we really have enough "is" and "the" tho?!! :smile: lol
Actually, I can tell you now after a few years working on these how this happens. This isn't the first time, and it won't be the last: When working on a topic ... even proofreading it several times ... even when reviewers look ... if a line breaks right where the first word occurrance appears at the end of one line and the next occurrance of the word appears at the start of the next line .. :eye: :boom: .. the human eye can read right past it it. :smile:
Thanks for spotting these and getting them OUT! :rocket:


DynamicModuleLoader's children mounts twice

Due to the following workaround:
https://github.com/Microsoft/redux-dynamic-modules/blob/e1046942f26db108f7dbb6ccc593824c9de716f5/packages/redux-dynamic-modules/src/DynamicModuleLoader.tsx#L142-L147
children of DynamicModuleLoader is mounted twice, once with a <Provider> wrapper and once without the wrapper.
~I think this workaround is not needed anymore since React 16.6.3 and store.getState() gives the correct state on first render.~
Nevermind, it still behaves the same. I forked and unconditionally wrapped the component with the Provider always, which is better than double mounting IMO and has no side effects AFAIK.

@anilanar  thanks for reporting, Can you give a codesandbox with repro?

https://codesandbox.io/s/8kx2jr7762

Sorry I didn't get a chance to look at it yet. I will take a look this week and get back to you.

@anilanar  I am not able to open the sandbox, I get following error. I will try again later today

Perhaps codesandbox had problems when you tried it, I can open it.

I can repro this and this is what I see, the storeState changes  and thus React rerenders children
We need to debug more to see why the storeState is not upto date in first render after the module is loaded. Most probabbly this is another side effect of React's new context API, but for sure this is not a bug in redux-dynamic-modules. It takes a long time to debug react, I will give it a try later in the week if I get sometime.

I have this trouble too. This issue blocks releasing our app (

@navneet-g State not being up to date after updating a store is not a bug in redux-dynamic-modules. But using two different component roots (to overcome aforementioned problem) is a bug in redux-dynamic-modules.

@shockk  in our app we are using following version of react-redux, and redux and don't see this issue. We are using it with React 16.8. Please try these versions for now to unblock the release.
react-redux": "^5.1.1",
"redux": "^4.0.0",

@anilanar which two different component roots are you mentioning? Please elaborate.

First the following:
https://github.com/Microsoft/redux-dynamic-modules/blob/master/packages/redux-dynamic-modules/src/DynamicModuleLoader.tsx#L135-L148
and
https://github.com/Microsoft/redux-dynamic-modules/blob/master/packages/redux-dynamic-modules/src/DynamicModuleLoader.tsx#L116-L125
_getLatestState will be true on the first render and will be false on the second render. Thus, on the first render, it will render something like:
<ReactReduxContext.Provider ...>
  {children}
</ReactReduxContext.Provider>

On the second render, it will just render the {children}. Because the root of the component changes from ReactReduxContext.Provider to children's root component, whatever is inside {children} will mount twice no matter what.

Thanks @anilanar ... @abettadapur can you take a look?

@anilanar @shockk  Sorry for the delay here. Could you guys try this version of the package and let me know if you still have issues?
<EMAIL_ADDRESS>

@abettadapur It works. To understand the effects of strictMode, I created the following sandbox with bunch of console.logs, checking state update order of connected components inside and outside DynamicModuleLoader and also assigning refs to DynamicModuleLoader's children from outside.
https://codesandbox.io/s/73r20x4jzj
The difference seems to be only when mounting DynamicModuleLoader. strictMode changes the order of state updates. When it's turned off, DynamicModuleLoaders children's connected state updates before anyone else.

correct, the only difference should be at mount time. I don't think we can get around this for strict consumers.  Thanks for verifying and the example!
@navneet-g Let's move to publish the new version

Thanks @anilanar  and @abettadapur .
@abettadapur  I left a comment on your PR, let me know you want to address before merging or in a separate PR.


Rename Action.Execute related interfaces

Related Issue
https://github.com/microsoft/AdaptiveCards/issues/3924
Description
We aim to utilize ChannelAdapter abstraction to handle request/response for Dynamic Typeahead in Input.Choiceset. Currently, it is only used for Action.Execute and the interfaces tied up with it, although general in name, are specific to Action.Execute. This PR introduces the changes which make it more generic to be used for other types of requests.
Breaking Changes

activity-request.ts: Renamed interface IActivityRequest to IExecuteRequest
adaptive-applet.ts: Renamed instance methods of class AdaptiveApplet:

onPrepareActivityRequest to onPrepareExecuteRequest
onActivityRequestSucceeded to onExecuteRequestSucceeded
onActivityRequestFailed to onExecuteRequestFailed

Microsoft Reviewers: Open in CodeFlow

@microsoft-github-policy-service agree [company="Microsoft"]

added @dclaux.

This shouldn't be done. An activity request is the "messenger" and carries an Action.Execute, just the same as it could carry a "Data.Query" or whatever else. These renames are not only unnecessary, they are not desirable.

If something needs to change, it is the the action member of IActivityRequest, which as of now is strongly typed to ExecuteAction. It should be updated to allow the activity request to carry a Data.Query

As for the events, again they are fine as they are:

onPrepareActivityRequest prepares an activity request by setting its members etc. including its action member, the piece that needs to change as per my previous comment
The other two events are also fine as they are

Again, an "activity request" is a generic "messenger", and its payload (the "action") is what types it.

@dclaux I understand that 'Activity Request' as its name suggests is meant to be generic for all types of requests. And to maintain that notion, I separated the interfaces to make a clear distinction between Activity Request and an Action.Execute request. The interface which enforces Activity Request has members which are only required for Action.Execute, such as trigger, consecutiveRefreshes, authCode and 'authToken'.
I know that we can make this implementation work for dynamic typeahead via the action property, but I do not feel that it is clean. And adding more properties if needed to this interface for more types of requests in the future does not do a good separation of concerns. It is better for Activity Request to be a base which can be extended by several types of requests and all those requests are handled separately. Makes it more extensible as well.

@SuryanshPrabhat I am sorry but I am not convinced by those arguments. I understand your goal and your intent which make sense, but I do not believe the proposed changes are the best way to achieve those. I have sent you additional thoughts offline, let's continue discussing there.


implement AdvancedPrefsViewController with custom polling

partly implements changes discussed in #37.
This is what the menu currently looks like:

I'm thinking we'd probably want monitor specific polling settings though?
I'd like some input on how that should look, should we use another NSPopupMenu with the friendlyName?
TODO:

[ ] Translations? (can be done through new PR's)
[x] Make it work for each display specific (this is a bit hard to test for me since I only have one display, and I wanted to hear the feedback first)
[x] Add longerDelay option
[x] Add hideOsd option

Please feel free to add/make changes or point out things that still need changing, this is very much a work in progress.

While we're at it, here are some things I'd also like to add to advanced prefs :

[ ] Inputs for the max/min value of brightness/volume for sliders and using keyboard (with default 0 and 100 or max value from ddc)
[ ] Input to change the modifiers for smaller increment (with default alt + shift)
[ ] Reset all prefs button

Sounds like a good idea! I'm currently pretty busy studying but after I finish (which should be somewhere next week) I can continue to work on it.
Quick question, in the advanced prefs for polling, should I use a tableview for displaying the monitors and their settings like we do in DisplayPrefsController or just the way I did it in the screencapture up top but add an extra NSPopup button for each monitor? I have the NSPopup almost implemented but it might be better to use a CollectionView instead?

Hi @JoniVR and good luck with your finals !
Let's try with an NSPopup for now as to not overload this view too much

Something else we could add here would be another NSPopup for trying out different replyDelays to make debugging easier for people experiencing issues.

Some notes:

I went with the tableview approach anyways because it's much cleaner to do in code, especially when trying to update data. It also looks better when there's no connected displays.

I've added the reset prefs button but we still need a way to refresh the menu states when the prefs are reset.

I've separated the logic for holding the displays array.
Since we're using it in several places, it seemed like a better idea to me to move it to a separate DisplayManager. I was thinking about doing this using a singleton but eventually went with a dependency-injection like approach (if that's what you can call it). I'm not sure if you'll agree with this approach or not but let me know.

Adding the specificPollingMode to the whitelist still seems pretty impossible with the current implementation, unless I'm missing something. I think we could just get rid of it once we have the advanced settings implemented tbh.

[ ]  Input to change the modifiers for smaller increment (with default alt + shift)

How would you want to go about this? An NSPopup and then store it in NSUserDefaults?

I think we could start wrapping up this PR (translations and cleanup and such) and perhaps if it's ready, release it and then open an issue with other stuff we might want to add to the advanced prefs as to not make this PR too big and get the most important stuff out there for specific issue fixes. (#37, #105 for example)

I'm thinking of adding a help button which links to a wiki page on this repo where some of these advanced settings could be explained:

@the0neyouseek, Would you be open to enabling the wiki so that I can add some stuff like Polling Count, Longer Delay and Hide OSD to it?

All we need now is testing/review and translations.. I'm not really sure if having so many translations is going to be a good idea long term though.. given that now we'd have to ask everyone to translate this.

Hi @JoniVR I'll review the PR this week-end 😉

Looks good to me 👍

Is there a way to download the new version anywhere with this new feature in it?

@stereonom Version 1.7.0 has been released which includes this :)


use flyteplugins with scheduler

TL;DR
I added k8s scheduler option support to flyteplugins https://github.com/lyft/flyteplugins/commit/209c52d002b4e6a39be5d175bc1046b7e631c153
Type

[ ] Bug Fix
[x] Feature
[ ] Plugin

Are all requirements met?

[x] Code completed
[ ] Smoke tested
[ ] Unit tests added
[ ] Code documentation added
[ ] Any pending items have an associated Issue

Complete description
Importing the new flyteplugin by running
go get github.com/lyft/flyteplugins@209c52d002b4e6a39be5d175bc1046b7e631c153
Tracking Issue
https://github.com/lyft/flyte/issues/
Follow-up issue
NA
OR
https://github.com/lyft/flyte/issues/

I've cut a new release: https://github.com/lyft/flyteplugins/releases
And manually updated lines in this PR by removing extra suffix from the pkg versions. I am not sure it was correct to edit manually as h1 values might be changed now. Is there is a recommended way to automatically generate this PR with the cut release?

Codecov Report

Merging #110 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #110   +/-   ##
=======================================
  Coverage   50.16%   50.16%
=======================================
  Files         129      129
  Lines        8015     8015
=======================================
  Hits         4021     4021
  Misses       3602     3602
  Partials      392      392

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 9e36b42...25fe3f7. Read the comment docs.

@migueltol22 please take another look

@vsbus can you please link this to an issue, so in the description please fill out the linked issue number. This way when we release it helps


Fix typo: 'Form' to 'form' since it is a HTML element

This should actually read <FancyForm/> instead of <Form/>, given the context in the sentence before:

For example, imagine I have a <FancyForm/> component
that enhances an HTML <form>. I need some way to pass all its inputs.

I initially misunderstood the code example, thinking it was the body of the FancyForm component function, which made the previous sentence unclear. Changing "Form" to "FancyForm" clarifies everything. Can I update this pull request to fix the typo correctly (I don't know if that is even a thing), or should I close it and create a new one with the correct fix?

Just updating this PR is great :-)

Done!


[Bug] 8.0 crashes when trying to use combo

Expected behavior
when using combos the app should not crash
Actual behavior
app crashes when using combos
Steps to reproduce the behavior

open beeftext
begin using combos
after an indeterminate amount of combos, the app begins to crash when using combos
restart app
once app has crashed once using a combo it will continue to crash every time a combo is used until the computer is restarted

Beeftext version (e.g. Beeftext v7.0 Portable Edition).
8.0 (standard)
Operating system version (e.g Windows 10)
Windows 10 Enterprise
20H2
19042.746
Windows Feature Experience Pack 120.2212.551.0

Hi.
A couple of questions:

Do you use any other 'productivity tool' on your system (keyboard or clipboard related tools)?
If you change the value of the Advanced ➜ Use legacy copy/paste option in the preferences, does the problem continue?

Hi.
A couple of questions:

Do you use any other 'productivity tool' on your system (keyboard or clipboard related tools)?
If you change the value of the Advanced ➜ Use legacy copy/paste option in the preferences, does the problem continue?

Hi.
A couple of questions:

Do you use any other 'productivity tool' on your system (keyboard or clipboard related tools)?
If you change the value of the Advanced ➜ Use legacy copy/paste option in the preferences, does the problem continue?

I got the same issue just now (was working flawlessly for a week). Using legacy copy/paste fix the crashing

Hi.
A couple of questions:

Do you use any other 'productivity tool' on your system (keyboard or clipboard related tools)?
If you change the value of the Advanced ➜ Use legacy copy/paste option in the preferences, does the problem continue?

I got the same issue just now (was working flawlessly for a week). Using legacy copy/paste fix the crashing

Hi.
A couple of questions:

Do you use any other 'productivity tool' on your system (keyboard or clipboard related tools)?
[BNN]: no, I don't use any other productivity tools
If you change the value of the Advanced ➜ Use legacy copy/paste option in the preferences, does the problem continue?
I'm rolled back to version 7.x -- I'll re-upgrade to see if that fixes things

Hi.
A couple of questions:

Do you use any other 'productivity tool' on your system (keyboard or clipboard related tools)?
[BNN]: no, I don't use any other productivity tools
If you change the value of the Advanced ➜ Use legacy copy/paste option in the preferences, does the problem continue?
I'm rolled back to version 7.x -- I'll re-upgrade to see if that fixes things

Thanks for your feedback. You're not the first to report issues with the new copy/paste code. I definetely need to review this code.

Thanks! Regardless of the issue, this piece of software has been a huge time-saver for me -- I really appreciate the care and support!


Supports export PATH

Supports exports PATH
export PATH=$HOME/.npmbrew/current/bin:$PATH

Update directory stack
.npmbrew
  - current
    - bin
      - npm: ln -s ../npm/bin/npm-cli.s npm
    - npm: current npm/bin/npm symlink

:shipit:


Properly set IsPrecise for SDL scroll events

Continuation of https://github.com/ppy/osu-framework/pull/5186#issuecomment-1132064369.
Tested on a windows precision touchpad, and with a regular mouse.

I guess this method of detection will mean the flag could be incorrect if the first event to arrive is from a precision source but exactly 1.00. Very very unlikely in practice, but just confirming.

Correct, could be false for the first scroll, but I expect the following deltas to be precise.
Also, due to how the debounce is implemented, any scroll within 100 ms of SDL_Init will be considered precise, not worth mentioning as the window is not created/visible in the first 100 ms.

I've tested this on windows and it works as expected (did not test transitioning to mouse from precision, but I think the code is sound enough).


Adding a custom class to the trix-editor via className prop is not possible

I need the ability to pass a custom class name to <trix-editor> because I want to override a css property that is set via the default trix.css. More specifically I want to override min-height because I want the editor to have different heights at different times.
It looks like we could accomplish this through trixEditorOptions.
I'm open to collaborating on this further.

@alejo4373 Sounds good to me. Please feel free to raise a PR. 😊

What do you think of updating storybook? It took me some time to figure how to implement the new story with the older API. See the change in API here for how to add parameters and decorators.

We can log a separate issue for upgrading Storybook. Closing this issue as PR is merged.


Feat: Updating the Hovering Effects on Join Discord and Follow us on linkedin like the mentioned animation below

Describe the feature
We can update the Hovering effect on front page in join discord and follow us on linkedin button in an animated version with having good ui. Kindly assign me this issue, so i can work towards it under GSSoC'24. Thank You
Add ScreenShots
https://github.com/Its-Aman-Yadav/Community-Site/assets/115927008/84269024-db9b-4625-b33f-8fff37ca51d6

Record

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

Hi @Its-Aman-Yadav
My name is Lakshay Rao and I'm a contributor under GSSOC'24 .
I think the animation you have provided in the website of fading text is very nice it just needs a bit of improvements in colors so that it looks visually appealing.
Please assign me this issue under GSSOC'24 .


This adds horizontal scrollbar in responsive mode

Greetings,
Whenever I add this, it adds a slight horizontal scroll to the website, and I cannot figure out why. I can see it on the demo aswell https://hyperstown.github.io/puresnowjs/
Any idea as to why this happens?

Hi,
<body> tag has CSS property overflow-x: hidden.
This property ensures that scrollbar will stay hidden even if some elements are overflowing.
Have you altered in anyway way attached CSS? I would also recommend you to check if you don't overwrite that property in some other file?
If issue also occurs on the demo page then perhaps one of installed extensions is causing this problem.
I've tested on both Chromium and Firefox, mobile and desktop, and it works fine.
You can check in Firefox what element is causing overflow in inspector:

(Screen with overflow-x disabled)

This is true, the scrollbar is hidden, but it's still possible to do a horizontal scroll by tilting the mousewheel on PC or pulling the edges on mobile. This possibility disappears when the script is unloaded for some reason :o

Then perhaps overflow-x: clip could be a solution.
As said in MDN docs:

Like for hidden, the content is clipped to the element's padding box. The difference between clip and hidden is that the clip keyword also forbids all scrolling, including programmatic scrolling. The box is not a scroll container, and does not start a new formatting context.

This possibility disappears when the script is unloaded for some reason :o

That's because individual snowflakes are flowing outside screen border. When script is unloaded there is no element that could cause overflow.

Then perhaps overflow-x: clip could be a solution. As said in MDN docs:

Like for hidden, the content is clipped to the element's padding box. The difference between clip and hidden is that the clip keyword also forbids all scrolling, including programmatic scrolling. The box is not a scroll container, and does not start a new formatting context.

Does not seem to solve the issue either, unfortunately :(

Then perhaps one of those solutions will work for you (You can also try using all of them ):
In index.html replace:
<meta name="viewport" content="width=device-width, initial-scale=1.0">

with
<meta name="viewport" content="width=device-width, initial-scale=1.0, maximum-scale=1.0, user-scalable=0">

Additionally in pureSnow.js you can set max snowflake position like that:
let randomX = Math.random() * 90; // vw (90 vw instead of 100)

Yet another solution would be to set snow div as relative with max-width:

/* style.css | http://<IP_ADDRESS>:5500/style.css */

body {
  /* overflow-x: hidden; */
}

/* Element | http://<IP_ADDRESS>:5500/ */

#snow {
  overflow-x: clip;
  max-width: 100vw;
  position: relative;
}

worked for me


Updated spelling error - Line 72

Added the letter 'p' to deloyment on Line 72.

Thanks for catching that!


design feedback of index page on beta site

Warning: The bullet points are the issues... the rest is my need to need be overly verbose (which in the past has caused some excellent technical documentation)
I have some visual design issues with the first page of the beta site that are something to consider when following best practices. For context purposes, I'm viewing this page from a new iMac 5120 x 2880 Retina display and realize I may just need to beat back since it currently only affects a small % of viewers, please let me know if this is the case.
Note: image examples below may need a min to load & reach top speed:

the landing page is too long, probably because the images are too large for the page and diminishes the textual content they relate to. But they are not quite large enough for those who are vision impaired and/or have older eyes. Consider making them smaller on the page, with a click to overlay a much larger image container or full screen option so one doesn't have to leave the current page...

It's neat that the animated images are running through the steps and screens relating to your text below them as you scroll down, but they are moving waaaaay too fast. Whether or not it's this way on purpose, for me it's the equivalent of blinking banner ads, or scrolling text, and stock prices ticker taping across my screen..   it causes alot of visual confusion and a level of irritation that may cause people to leave the page, it's what I did ...

When my browser is full sized I can see 3 or more of these rapidly moving images at a time, not only does it obscure the text below it (which should be be a touch larger and a few lines lower), but creates an impulse to scroll down further to get away, only to find I'm barraged with more of the same... it makes me want to hack your code to make it work for me, which will be a time sink if I didn't know how. It also risks my losing interest after leaving the planned teaching path you've set out before me.

Consider using a contrasting color or size for headings for the text below the images. I thought the headings were dead links, until I saw the example below. I think the colors are too similar unless the text is larger, then it may be more visible, but realize that blue (or a variation of it) has been the default link color since the beginning of web browsing.

Now for the kudos... fabulous work everyone! I'm so delighted to have found this FREE!!! resource a few months ago.... (thanks GitHub) and am especially excited to know your plan to craft new lessons that formerly went off-site. I was laid off last summer from a 7 year stint, it has turned out to be both a blessing and a curse. The blessing is the overwhelming amount of free resources generous programmers, like you folks, provide to the world to make the hard stuff easier and available at will. The curse for me was my body of work as a web designer for the last 15+ yrs had either vanished from the interwebs or been replaced, leaving me nothing viable, by modern standards, to show for myself in my online portfolio.
Thanks soooo very for helping me get my career back up to speed. You've payed oodles forward and stirred my deep obligation to give back to your efforts if I can. In fact, I can gladly help your beta site in the "look and feel" department, but don't get me wrong, I think your site very charming and nicely accessible across all cultures, I just believe it could even better. I loved the html lessons, but had trouble seeing all the components at the same time, like when instructions were expanded, it moves you away from seeing the code area create live changes as you typed... I would arrange the components with the code and phone centered next to each other filing most of the browser real estate, and instead of having the instructions sliding down when expanding, try popping it out into a text box the can be resized as needed, copied and saved easily or having the text framed either of side of the screen, to maybe scroll, or be spoken if your OS is capable... I know your idea was to only reveal more instruction if it was needed, but the reveal felt clunky and distracted from the experience because I had read it over and over to create continuity and full meaning.
I wish the idea of the "web designer" was not so passe, it's what I've always done best. Just ask any marketing professional how much value there is in landing on a pleasant looking page without any immediate wonder or brain strain. Its a tremendous skill to display text heavy pages of important information in a pleasing way (think first mortgages or filing your taxes). I hope you found my feedback helpful and not taken as criticism, I really just want to help you promote the invaluable resources you've given freely to this digital world full of future coders, leading by example and showing us how to fill up our tanks with good karma... phew!

he landing page is too long, probably because the images are too large for the page and diminishes the textual content they relate to. But they are not quite large enough for those who are vision impaired and/or have older eyes. Consider making them smaller on the page, with a click to overlay a much larger image container or full screen option so one doesn't have to leave the current page...

This is in the works (I believe there is an issue somewhere if not I will open one.)

It's neat that the animated images are running through the steps and screens relating to your text below them as you scroll down, but they are moving waaaaay too fast. Whether or not it's this way on purpose, for me it's the equivalent of blinking banner ads, or scrolling text, and stock prices ticker taping across my screen.. it causes alot of visual confusion and a level of irritation that may cause people to leave the page, it's what I did ...

Definitely needs to be addressed

When my browser is full sized I can see 3 or more of the...

This could be addressed by making this step through instead of scrolling down.

Thanks for all the kind words :blush:
We are working our hardest to ensure that (FCC) Free Code Camp is not only the best resource online to learn to code, but also foster a great community that will make FCC long
cc @QuincyLarson

@gingerkitty wow - thank you for all the feedback. This is very helpful! I don't have strong design sensibilities, and I knew this sort of layout was a big risk, so I'm thankful you looked at it before we released it. Would you have time to do a quick screen share regarding this? I think I have a good idea of what you're saying, but I want to make sure, as redoing all the gifs and potentially breaking the page up into smaller "slides" will take some work, and I want to make sure it's done in the right direction. Do you have Screen Hero? https://github.com/FreeCodeCamp/freecodecamp/wiki/How-to-install-Screenhero I'll also message you in Gitter.

Hi Quincy,
Your welcome! I’d be glad to show you what’s going on using screen sharing, and I even have some ideas about how to break up the page. Unfortunately, I’m up to my tits with work today, could we do this tomorrow afternoon?
I’ve downloaded Screen Hero and will add you as a contact, just let me know what email address to use and we’ll work it out.
gingerkitty

On Aug 12, 2015, at 1:49 AM, Quincy Larson<EMAIL_ADDRESS>wrote:
@gingerkitty https://github.com/gingerkitty wow - thank you for all the feedback. This is very helpful! I don't have strong design sensibilities, and I knew this sort of layout was a big risk, so I'm thankful you looked at it before we released it. Would you have time to do a quick screen share regarding this? I think I have a good idea of what you're saying, but I want to make sure, as redoing all the gifs and potentially breaking the page up into smaller "slides" will take some work, and I want to make sure it's done in the right direction. Do you have Screen Hero? https://github.com/FreeCodeCamp/freecodecamp/wiki/How-to-install-Screenhero https://github.com/FreeCodeCamp/freecodecamp/wiki/How-to-install-Screenhero I'll also message you in Gitter.
—
Reply to this email directly or view it on GitHub https://github.com/FreeCodeCamp/freecodecamp/issues/1728#issuecomment-130220568.

Hi @gingerkitty, I was able to incorporate your feedback to the best of my abilities. Berkeley will deploy this tomorrow and you can take a look at it. All the gifs are much slower, and I created significant visual separation between them. We may address the issue of multiple cards being visible on the page at the same time (I can see how that would be distracting) but I'll just leave it as a single scrolling page. Thanks again for your clear feedback!

 I’ll take a look tomorrow and let you know what I think.
gingerkitty

On Aug 13, 2015, at 1:10 AM, Quincy Larson<EMAIL_ADDRESS>wrote:
Hi @gingerkitty https://github.com/gingerkitty, I was able to incorporate your feedback to the best of my abilities. Berkeley will deploy this tomorrow and you can take a look at it. All the gifs are much slower, and I created significant visual separation between them. We may address the issue of multiple cards being visible on the page at the same time (I can see how that would be distracting) but I'll just leave it as a single scrolling page. Thanks again for your clear feedback!
—
Reply to this email directly or view it on GitHub https://github.com/FreeCodeCamp/freecodecamp/issues/1728#issuecomment-130570503.


fix: Unskip failing test in planetscale related to dates

Close https://github.com/prisma/team-orm/issues/517
Requires https://github.com/prisma/prisma-engines/pull/4422 to be merged and integrated into prisma/prisma before merging

Waiting to update prisma main with latest engines' after merging https://github.com/prisma/prisma-engines/pull/4422, There's a PR open for the integration branch in https://github.com/prisma/prisma/pull/21831 and I'm looking for an (a priori unrelated) failing test

Actually the offending test in https://github.com/prisma/prisma/pull/21831 passes locally, so I will merge this PR as soon as the new version of engines is published and bumped either in main or here (whatever happens before)
❯ pe && cargo build -p query-engine-node-api && p && cd packages/client && PRISMA_QUERY_ENGINE_LIBRARY=/Users/miguel/GitHub/prisma/prisma-engines/target/debug/libquery_engine.dylib && pnpm run test:functional:code  --flavor=js_planetscale 15176
    Finished dev [unoptimized + debuginfo] target(s) in 0.64s

><EMAIL_ADDRESS>test:functional:code /Users/miguel/GitHub/prisma/prisma/packages/client
> dotenv -e ../../.db.env -- node -r esbuild-register helpers/functional-test/run-tests.ts --no-types "--flavor=js_planetscale" "15176"

{
  column: 28,
  file: '/Users/miguel/GitHub/prisma/prisma/packages/engines/dist/index.js',
  length: 2,
  line: 16652,
  lineText: '          } else if (x2 === -0) {',
  namespace: '',
  suggestion: ''
}
Comparison with -0 using the "===" operator will also match 0
 PASS  tests/functional/issues/15176/tests.ts (5.547 s)
  issues.15176 (provider=sqlite)
    ○ skipped should update both updatedAt fields on a model
  issues.15176 (provider=sqlite, js_libsql)
    ○ skipped should update both updatedAt fields on a model
  issues.15176 (provider=postgresql)
    ○ skipped should update both updatedAt fields on a model
  issues.15176 (provider=postgresql, js_pg)
    ○ skipped should update both updatedAt fields on a model
  issues.15176 (provider=postgresql, js_neon)
    ○ skipped should update both updatedAt fields on a model
  issues.15176 (provider=mysql)
    ○ skipped should update both updatedAt fields on a model
  issues.15176 (provider=mysql, js_planetscale)
    ✓ should update both updatedAt fields on a model (97 ms)
  issues.15176 (provider=mongodb)
    ○ skipped should update both updatedAt fields on a model
  issues.15176 (provider=cockroachdb)
    ○ skipped should update both updatedAt fields on a model
  issues.15176 (provider=sqlserver)
    ○ skipped should update both updatedAt fields on a model

Test Suites: 1 passed, 1 total
Tests:       9 skipped, 1 passed, 10 total
Snapshots:   0 total
Time:        5.579 s
Ran all test suites matching /15176/i.

I actually merged the PR, but it was indeed not flaky. 😅
https://github.com/prisma/prisma/pull/21832#issuecomment-1801350061
See the failing test for Planetscale, it's the same error in this PR now that it's sync with main

So technically not blocking this PR, but someone needs to have a look

I'm taking a look.

Ok, fix: https://github.com/prisma/prisma-engines/pull/4426, The reason for the flakeyness, is that there is a lag between the create and the update, of up to a hundred milleseconds. Sometimes that makes it to the next second since epoch and the test passed. There was an underlying bug, tho DateTimes didn't include franctionals in main.
I'm merging this, in the light of https://github.com/prisma/prisma-engines/pull/4426


Lab 4 Test Case 5

I am unsure what is generating this problem. I am able to get the same output as shown in Level 5.

Hi! May I know how you resolved this? Am facing this issue as well, thanks!


machine upgrade does not upgrade vmwarevsphere

machine upgrade machines using the vmwarevsphere driver seems to fail without errors.
docker-machine version 0.6.0, build e27fb87
mbp scripts → machine upgrade d
Waiting for SSH to be available...
Detecting the provisioner...
Upgrading docker...
Stopping machine to do the upgrade...
Upgrading machine "d"...
No default Boot2Docker ISO found locally, downloading the latest release...
Latest release for github.com/boot2docker/boot2docker is v1.10.0
Downloading /Users/paul/.docker/machine/cache/boot2docker.iso from https://github.com/boot2docker/boot2docker/releases/download/v1.10.0/boot2docker.iso...
0%....10%....20%....30%....40%....50%....60%....70%....80%....90%....100%
Copying /Users/paul/.docker/machine/cache/boot2docker.iso to /Users/paul/.docker/machine/machines/d/boot2docker.iso...
Starting machine back up...
(d) Waiting for VMware Tools to come online...
Restarting docker...
mbp scripts → machine ls
NAME      ACTIVE   DRIVER          STATE     URL                           SWARM   DOCKER    ERRORS
d         -        vmwarevsphere   Running   tcp://<IP_ADDRESS>:2376              v1.9.1

cc @frapposelli

Manually (via vSphere-client) replacing the boot2docker.iso in the vm-datastore seems to resolve the problem.

definitely a bug, I'll look into it, thanks for reporting.

Any update on this @frapposelli ?

I also have this problem.
➜  ~ docker-machine -v
docker-machine version 0.7.0, build a650a40
➜  ~ docker-machine ls
NAME      ACTIVE   DRIVER          STATE     URL                        SWARM   DOCKER    ERRORS
node-00   -        vmwarevsphere   Running   tcp://<IP_ADDRESS>:2376           v1.10.2
node-01   -        vmwarevsphere   Running   tcp://<IP_ADDRESS>:2376            v1.10.2
node-02   -        vmwarevsphere   Running   tcp://<IP_ADDRESS>:2376            v1.10.2
➜  ~ docker-machine upgrade node-00
Waiting for SSH to be available...
Detecting the provisioner...
Upgrading docker...
Stopping machine to do the upgrade...
Upgrading machine "node-00"...
Default Boot2Docker ISO is out-of-date, downloading the latest release...
Latest release for github.com/boot2docker/boot2docker is v1.11.1
Downloading /home/philipz/.docker/machine/cache/boot2docker.iso from https://github.com/boot2docker/boot2docker/releases/download/v1.11.1/boot2docker.iso...
0%....10%....20%....30%....40%....50%....60%....70%....80%....90%....100%
Copying /home/philipz/.docker/machine/cache/boot2docker.iso to /home/philipz/.docker/machine/machines/node-00/boot2docker.iso...
Starting machine back up...
(node-00) Waiting for VMware Tools to come online...
Restarting docker...
➜  ~ docker-machine ls
NAME      ACTIVE   DRIVER          STATE     URL                        SWARM   DOCKER    ERRORS
node-00   -        vmwarevsphere   Running   tcp://<IP_ADDRESS>:2376           v1.10.2
node-01   -        vmwarevsphere   Running   tcp://<IP_ADDRESS>:2376            v1.10.2
node-02   -        vmwarevsphere   Running   tcp://<IP_ADDRESS>:2376            v1.10.2

I actually have the same bug while trying to upgrade from 1.11.1 to 1.12 with vsphere :
machine upgrade docker01
Waiting for SSH to be available...
Detecting the provisioner...
Upgrading docker...
Stopping machine to do the upgrade...
Upgrading machine "docker01"...
Copying /Users/sitle/.docker/machine/cache/boot2docker.iso to /Users/sitle/.docker/machine/machines/docker01/boot2docker.iso...
Starting machine back up...
(docker01) Waiting for VMware Tools to come online...
Restarting docker...

then
$ eval $(docker-machine env docker01)
$ docker ps
Error response from daemon: client is newer than server (client API version: 1.24, server API version: 1.23)

I actually use this workaround for now to make it work :
$ export DOCKER_API_VERSION=1.23
$ docker ps

I have the same issue, always replace boot2docker.iso manually, any official methods?

More then a year and still not fixed?


[WIP] [Feature] Add Admin Sales Analytics to OSF [#OSF-5932]

Purpose
With the pageview tracking code that we've added to {{staging.osf.io}} it should be possible to start generating metrics that are interesting from a sales perspective. Blow is a list of metrics we add to OSF admin server:

Average user session length (past month)
Average MAU session length (past month)
Number of users that view 2+ products in the same year
Number of users that view 2+ products in the same month

Changes
Add a new tab: "OSF Sales Analytics" to OSF Admin.
Side effects
No side effects.
Ticket
OSF-5923

Pull request closed. Move to  #5608.


minifier removes script-type-module attribute. it should not.

hi,
test case:
input (test.html):
<script type="module">console.log('hello')</script>

command:
./node_modules/.bin/html-minifier-terser --remove-script-type-attributes test.html

output:
<script>console.log('hello')</script>

as you see, the type="module" part got removed. it should not be removed, because when a script tag has type-module, it's content will be executed differently. also, the documentation for the flag says:
"
--remove-script-type-attributes      Remove type="text/javascript" from script tags. Other type attribute values are left intact.
"
which is not correct with the behavior.

Hi @gabor,
thanks for reporting this. The option at https://github.com/DanielRuf/html-minifier-terser/blob/v5.0.3/src/htmlminifier.js#L556 runs the following checks:
https://github.com/DanielRuf/html-minifier-terser/blob/v5.0.3/src/htmlminifier.js#L557
You can see it at https://github.com/DanielRuf/html-minifier-terser/blob/v5.0.3/src/htmlminifier.js#L67
It checks executableMimetypes at https://github.com/DanielRuf/html-minifier-terser/blob/v5.0.3/src/htmlminifier.js#L169
And this contains more than just text/javascript. So the docs are misleading imo.
https://github.com/DanielRuf/html-minifier-terser/blob/v5.0.3/src/htmlminifier.js#L158-L164
Same at https://github.com/kangax/html-minifier/blob/v4.0.0/src/htmlminifier.js#L163
Also see https://github.com/DanielRuf/html-minifier-terser/pull/29 should probably fix this.

Hi @gabor,
thanks for reporting this. The option at https://github.com/DanielRuf/html-minifier-terser/blob/v5.0.3/src/htmlminifier.js#L556 runs the following checks:
https://github.com/DanielRuf/html-minifier-terser/blob/v5.0.3/src/htmlminifier.js#L557
You can see it at https://github.com/DanielRuf/html-minifier-terser/blob/v5.0.3/src/htmlminifier.js#L67
It checks executableMimetypes at https://github.com/DanielRuf/html-minifier-terser/blob/v5.0.3/src/htmlminifier.js#L169
And this contains more than just text/javascript. So the docs are misleading imo.
https://github.com/DanielRuf/html-minifier-terser/blob/v5.0.3/src/htmlminifier.js#L158-L164
Same at https://github.com/kangax/html-minifier/blob/v4.0.0/src/htmlminifier.js#L163
Also see https://github.com/DanielRuf/html-minifier-terser/pull/29

yes, module should probably not be in that list. as far as i can see it got added with https://github.com/DanielRuf/html-minifier-terser/pull/29 ... i asssume the current logic is that for given type-values it minifies-body and removes the type-attribute. for type=module the body can probably be minified, but the type-attribute should not be removed.

Well, all type=some-valid-js-type are removed and already were in previous versions. The documentation of the option is just outdated.
In my opinion this is not wrong.

Both parts are ~4 years old. I doubt that these are wrong. See https://github.com/kangax/html-minifier/blame/v4.0.0/src/htmlminifier.js#L157-L164

See https://github.com/kangax/html-minifier/issues/694#issuecomment-235342390 and especially https://github.com/kangax/html-minifier/issues/694#issuecomment-236160075

sorry, maybe there is a misunderstanding. what i am saying is that if there is a script tag that looks like this:
<script type="module">....</script>

then it should not be minified to
<script>....</script>

browsers handle the content of the script-tag differently when there is a type="module" attribute.
you can test it easily, like this:
create a html file with this content:
<script>import test from "./test.js"</script>

and open it in google-chrome. open the dev-tools, and you will see the following error-message on the console:
"
Uncaught SyntaxError: Cannot use import statement outside a module
"
if you modify the test file to look like this:
"

"
then it will work. well, it will not work because there is no ./test1.js file, so you will get a http404 error, or a cors-error if you open it from the filesystem, but you can see you get a different error message.
so, in summary, the 'type="module"' part should not be removed from script-tags.

i've checked the code mentioned above, but that's not the code from this module.
this link https://github.com/kangax/html-minifier/blame/v4.0.0/src/htmlminifier.js#L157-L164
contains:
var executableScriptsMimetypes = utils.createMap([
  'text/javascript',
  'text/ecmascript',
  'text/jscript',
  'application/javascript',
  'application/x-javascript',
  'application/ecmascript'
]);

if i search for this variable in the codebase of this module, i find this:
https://github.com/DanielRuf/html-minifier-terser/blob/master/src/htmlminifier.js#L157
var executableScriptsMimetypes = utils.createMap([
  'text/javascript',
  'text/ecmascript',
  'text/jscript',
  'application/javascript',
  'application/x-javascript',
  'application/ecmascript',
  'module'
]);

as you see, the last item in the array, the 'module' is not there in the kangax-version.
it was added in this commit:
https://github.com/DanielRuf/html-minifier-terser/commit/4b9949c288c9a57ea47ff1bfeac781e9b8b5ea99
you can also verify the change in functionality of this program, if you compare version 5.0.2 and 5.0.3.
5.0.2 will not remove type="module", but 5.0.3 will.

it was added in this commit:
4b9949c

Yes, please reread my previous comments in detail =)
new script types are supported through this variable array. This is a fact. And this is used by default to remove any valid script types by html-minifier. The logic did not change. Just a new script type is supported now.
As written at https://github.com/DanielRuf/html-minifier-terser/issues/31#issuecomment-588261173:
See https://github.com/kangax/html-minifier/issues/694#issuecomment-235342390 and especially https://github.com/kangax/html-minifier/issues/694#issuecomment-236160075
This is not a bug - the documentation is just wrong. So wontfix at the moment. I see no wrong behavior here happening in html-minifier and html-minifier-terser.
Also the original part was / is 4 years old.
https://github.com/DanielRuf/html-minifier-terser/issues/31#issuecomment-588260377
As this is a fork of html-minifier with a few changes it uses the same logic.

it was added in this commit:
4b9949c

Yes, please reread my previous comments in detail =)
New script types are supported through this variable array. This is a fact. And this is used by default to remove any valid script types by html-minifier. The logic did not change. Just a new script type is supported now.
As written at https://github.com/DanielRuf/html-minifier-terser/issues/31#issuecomment-588261173:
See https://github.com/kangax/html-minifier/issues/694#issuecomment-235342390 and especially https://github.com/kangax/html-minifier/issues/694#issuecomment-236160075
This is not a bug - the documentation is just wrong. So wontfix at the moment. I see no wrong behavior here happening in html-minifier and html-minifier-terser.
Also the original part was / is 4 years old.
https://github.com/DanielRuf/html-minifier-terser/issues/31#issuecomment-588260377
As this is a fork of html-minifier with a few changes it uses the same logic.

i did check the comments you linked. one of them is this:
https://github.com/kangax/html-minifier/issues/694#issuecomment-235342390
"We delete whatever default value is for

There is definitely that may have changed with the recent html(5) specs regarding that as the code parts were not changed in the last 4 years.
There is currently no distinction between such "classic" and "module" type / mode. And as previously mentioned, it is not just text/javascript that is already removed.
And https://developer.mozilla.org/en-US/docs/Web/HTML/Element/script#attr-type is linked in the comment.
We may have to refactor that.

There is definitely something that may have changed with the recent html(5) specs regarding that as the code parts were not changed in the last 4 years.
There is currently no distinction between such "classic" and "module" type / mode. And as previously mentioned, it is not just text/javascript that is already removed.
And https://developer.mozilla.org/en-US/docs/Web/HTML/Element/script#attr-type is linked in the comment.
We may have to refactor that.

@rarous are you interested to take a look at this and improve it?

I workaround it by disabling removal of type attribute in html-min settings. It's good for my use cases.

@rarous can you share the command that would be needed for the CLI?

@rarous can you share the command or setting file + command that would be needed for the CLI?

@DanielRuf Original issue was solved here https://github.com/DanielRuf/html-minifier-terser/commit/4b9949c288c9a57ea47ff1bfeac781e9b8b5ea99, right?

@evilebottnawi https://github.com/DanielRuf/html-minifier-terser/issues/31#issuecomment-588996133 ;-)

I'm having a little trouble following the evolution of this issue.  Is this still considered a "documentation issue" or is the original problem going to be fixed?
It looks to me like the spirit of the original behavior was to remove type="javascript" and synonyms, because specifying one of those types resulted in the same behavior as omitting the attribute.  type="module" has always modified how the tag is interpreted and removing it is always an error.  The current behavior generates broken output any time an ESM module is included.
If the plan is to "refactor" per https://github.com/DanielRuf/html-minifier-terser/issues/31#issuecomment-589082078 then I can just avoid minifying until it's fixed.  If the package maintains that producing broken output is correct behavior then I will update https://github.com/jantimon/html-webpack-plugin/issues/1434 to request that they stop using removeScriptTypeAttributes by default.

We may have to refactor that.

So far this is still the case. We have to extend the checks in the minifier and introduce a new array which is not removed but includes the supported types.
PRs which implement this are very welcome.
You can also use ignore comment, see https://github.com/DanielRuf/html-minifier-terser#ignoring-chunks-of-markup
At the moment the docs are incomplete regarding this.

Not sure if this is the best solution: https://github.com/DanielRuf/html-minifier-terser/pull/40
I'm not that motivated today so feel free to give constructive feedback / a helpful review.
Not sure if we have to update the documentation.

The changes from https://github.com/DanielRuf/html-minifier-terser/commit/dceec7ddab6c7ff2a74cc1f61e6e356c5131c2bf landed in v5.1.1.


Created InstagramBot Terms Of Service

check out the file here via this link. https://www.dropbox.com/s/78ja2rgsokgkzq4/InstagramBot TOS.docx?dl=0
Please merge immediately.
Thanks.

Codecov Report

Merging #521 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #521   +/-   ##
=======================================
  Coverage   38.62%   38.62%
=======================================
  Files          25       25
  Lines        2058     2058
=======================================
  Hits          795      795
  Misses       1263     1263

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 48507b1...e902204. Read the comment docs.

Thanks. But why do you think this is needed?

You make it sound as if the developers gain access to your account details, which is not the case.

@order4adwriter Can you please give us any example of open source project that uses this types of documents?

Hello @ohld and Everyone,
Of the top of my head I'd say Tensorflow, Invoice Ninja, Flutter, Linode,....  Heck even the paid versions of every major Instagram -based bots out there all have a Privacy Policy..... every serious app, website or business entity needs one on their website.
@basnijholt , I'm sorry if I sounded that way but that was not my intention. I know the bot cannot do that.. .but most people out there may not know this or completely understand how the bot works.
But in all honesty, you'll agree with me that having one serves a dual purpose of assuring your end  users of  the sincerity of your intentions in creating this bot as well, thus encouraging them to use  and spread the word about Instabot, ...  and to also offer some form of indemnity for you guys ...just in case some unforeseen circumstances occur.
All I'm saying is that it is a lot more confidence-boosting to potential users if they see a privacy policy  for Instabot on the website...than seeing nothing at all.
Hope this helps.


lua/hlchunk/utils/indentHelper.lua:28: attempt to perform arithmetic on local 'leftcol' (a nil value) error when terminal width is less than 138 cols

Describe the bug
When I open nvim with hlchunk installed and configured, and if terminal width is less that 138 cols, I'm getting following error on nvim startup:
...vim/lazy/hlchunk.nvim/lua/hlchunk/utils/indentHelper.lua:28: attempt to perform arithmetic on local 'leftcol' (a nil value)

Screenshots

Maybe it is worth to add something like or 0 (or something like that) there?

Well, it seems that it also sometimes appears not even when width is <138 cols.
Just nvim don't wait for I press enter, and boots further.
I just found that is also appears when I editing nvim lua config with full width (236 cols)

(also, happens only when blank module is enabled)


Suggestion: WinCLIP: Zero-/Few-Shot Anomaly Classification and Segmentation

WinCLIP: Zero-/Few-Shot Anomaly Classification and Segmentation
https://paperswithcode.com/paper/winclip-zero-few-shot-anomaly-classification
https://github.com/caoyunkang/WinClip
https://github.com/bychelsea/vand-april-gan
https://sites.google.com/view/vand-cvpr23/challenge?authuser=0

Hi @monkeycc, @djdameln is currently working on an implementation, which is almost complete. We might see a PR quite soon. For more details, you could keep an eye on #1569.


add lightning and all options to payment-methods

fixes #4

I'm happy with the testing.
I've few stuff to add in regards of " getPaymentMethods " by api soon as you suggested
Thank for your contribution.


Add the ability to filter messages based on the account activity and age

Problem Statement
Current anti-scam measures in our bot are effective, but they could be further refined to target a specific behaviour pattern of scammers. Typically, scammers operate through new accounts and are removed from the platform after their first offence. This means their account age and message count is low. By not differentiating between new and established users, our bot may inadvertently affect the user experience of long-standing, legitimate members.
Proposed Solution
I propose introducing two new configurable parameters in the bot:

[x]  account_trigger_age: This setting would allow the bot to consider the age of the user's account. For instance, if set to 30 days, only accounts newer than 30 days would trigger the warning system.

[ ] number_of_messages_threshold: This would set a threshold for the number of messages a user has sent. Accounts below this threshold would be more rigorously monitored. For example, setting this to 50 messages means only accounts with fewer than 50 messages would trigger the warning system.

These parameters would make the bot's anti-scam system more dynamic, focusing on potential scam accounts without impacting the experience of regular, active members.
Alternative Solutions

User Verification: Implementing a user verification system where new users must go through extra steps to verify their identity.
Temporary Restrictions: Placing temporary restrictions on new accounts (e.g., limiting the ability to send messages or use certain features) until they reach a certain age or message count.

Additional Context
Implementing these features would require careful consideration to ensure that legitimate new users feel unrestricted. The goal is to enhance security while maintaining a welcoming environment for all genuine participants.

Feature 1 was added in https://github.com/rickstaa/warnings-discord-bot/pull/23.

Feature 2 can't currently be implemented since https://github.com/bwmarrin/discordgo doesn't have a way to quickly get the message count of a user in a server. You can only get the user message count in a channel.


Include survey links in instructor profile view

When a person logs in to AMY, they can see their own profile with a list of workshops they have taught.  Next to each workshop we would like to see a link to the survey results for that workshop (coming from the links in that event's page).

As a follow up to this - this will typically be just one link because our survey tool creates a single link to view pre- and post-workshop surveys.  Right now this is not the case for Instructor Training, which uses a different tool and has different links for pre- and post- results.
@fmichonneau two questions

When will instructor training use the same system as workshops?
Is it possible to integrate the survey results app with AMY so we don't have to manually enter the link in AMY?

When will instructor training use the same system as workshops?

We are piloting the first instructor training to use the same system with events that are running in September for a full transition by the end of the year.

Is it possible to integrate the survey results app with AMY so we don't have to manually enter the link in AMY?

not easily at the moment.

OK.  For now we should keep both survey link fields in AMY, and change to just one by the end of the year.
We'll revisit integration of the survey results app with AMY at another time.

To be clear:

all events, except for Instructor Training events, should be listed with links to their post-workshop survey results
we're only saving link to the survey itself, so post-workshop survey results link needs to be generated
to generate the link we're using salts; the results should be available at workshop-reports.carpentries.org, correct?

Notes from meeting with @fmichonneau:

Use a new field inside Event for storing survey results link
Show the link to instructors teaching particular workshop in their profile pages.

We are currently recording these urls in the Pre-workshop assessment survey for instructors field in the Event view.  Is there a reason we'd need to create a new field in the Event model?

Note that the survey links should only be visible when the event ROLE is one of the following:

Host
Instructor
Supporting Instructor

Survey links should not be visible for any other role.


enable async/await on handlers

In nodeJs 7.6 async/await was enabled
the newest version is 10.5.0
even aws lambdas now support 8.x.x so i felt comfortable adding async and await to enable this functionality
This is a simple way to enable full async support rather then using wrappers.

Thanks! Because this would be a breaking change, can you reimplement this against the 5.0 branch and open a new PR?

I uploaded non breaking change in a separate PR. i will redo this in 5.0 as well :)

guys, i dont see the router in 5.0... im missing something?

The router code was removed from express and instead resides in the npm module "router".


[WIP] Initial commit for Scheduler

This is an initial commit for the new core component, "the scheduler".
This is a base branch and it should not be merged until all dependent modules are integrated into this PR. I and my colleagues will open separate PRs to add all dependent modules. They need to be merged into this PR with proper reviews from committers.

We will implement modules based on the order defined in this page and add design documents under the page as well:
https://cwiki.apache.org/confluence/display/OPENWHISK/Component+Design
I added the first component design document for the "scheduler" component.
https://cwiki.apache.org/confluence/display/OPENWHISK/Scheduler
Please also refer to this integration plan proposal:
https://cwiki.apache.org/confluence/display/OPENWHISK/Integration+Plan+Proposal
Since this PR would become a huge one and will include many modules, I proposed the plan to integrate it.

I created a new label, "scheduler".
All PRs which are dependent on this PR will be labeled with it.
So it is supposed to not to merge PRs labeled with "scheduler" directly into the master branch.

Since this PR will be used as an intermediate PR to merge subsequent PRs into this, I added this branch in upstream repo instead of creating this PR from my forked-repo branch.
Other guys will open PRs against to this PR.

@style95 I'm not quite sure why we need an "official" branch for this feature in the repo itself. We have never done that to date. Can you not work on this feature on a fork and incrementally merge the bits into master? Merging this once it's done into master is kinda impractical anyway.

@markusthoemmes Thank you for the feedback, I wanted this kind of thing.
Yes, I have thought a few ways to include this kind of big change.
If we agree to accept some components which are not being used in any place at some point, I am happy to do that way.
But I couldn't see much difference.
For example, if I have 18 modules and am willing to contribute all of them with separate PRs(based on the assumption we accept unused codes), reviewers will review each of them and merge them into the master.
And if we have one base branch and merge 18 PRs into the branch one by one, reviewers will also review each of them but just merge them into the base branch instead of the master branch.
We can just avoid the unused codes with the later.

Well if you do it the latter way, you'll have a huge chunk to merge at some point anyway, right?
I'd just push it all to master. That'll ensure it's all continuously tested, non disruptive and everything.

@markusthoemmes
I am ok with either way.
I will open a thread in the dev list to form a consensus to keep some unused codes.
Thank you for the suggestion.

+1 for small prs to master.

@markusthoemmes
FYI, I removed this branch in the upstream repo.
From now on, I will work on this from the forked branch.


flat_hash_map over shared memory

Describe your problem
as titled. The flat_hash_map could allocate its payload on the vineyard shared memory a blob, when rehash, the orignal one would be released and rehash to the new spaces.
Additional context
Add any other context about the problem here.

Closing as bottleneck isn't there and we have already applied optimization to the fragment loader.

Closing, as long term optimization requires further overall design revision.


Write a cli to quickly generate templates

Write a cli to quickly generate templates, automatically download necessary dependencies, and choose the default environment (streamer, samp-node, omp-latest) or raknet, etc

Done


fix(sourcemaps): reduce verbose noise

verbose mode logging generates a lot of noise for non sourcemaps files, this PR changes so we outputs logs only when true .js.map sourcemaps are skipped

Does that check actually do anything? My understanding was that in here we are only getting filenames that are in the sources list, which should always be source filename and not sourcemap names.


JAVA_TOOL_OPTIONS cause version naming clash

I've installed versions 6, 7 and 8 via brew cask install java[x] and have tried to add these to jenv. I've also got   JAVA_TOOL_OPTIONS defined in my .bash_profile, which gives me odd version names (possibly related to #29) which means there's a clash when trying to add a new JDK version, as shown below.
$ env | grep JAVA
JAVA_TOOL_OPTIONS=-Dfile.encoding=UTF8
$ jenv add /Library/Java/JavaVirtualMachines/1.6.0.jdk/Contents/Home
oracle64-Picked added
Picked added
There is already a Picked JDK managed by jenv
Do you want to override (type y to confirm)? y
Picked added
$ jenv add /Library/Java/JavaVirtualMachines/jdk1.7.0_79.jdk/Contents/Home
There is already a oracle64-Picked JDK managed by jenv
Do you want to override (type y to confirm)? n

After doing this, I get two JDK versions listed, despite only adding one JDK.
$ jenv versions
* system (set by /usr/local/opt/jenv/version)
  Picked
  oracle64-Picked

If I remove these versions and then unset JAVA_TOOL_OPTIONS,the version name clash goes away, but I now seem to have 3 versions listed for each JDK I add ...
$ jenv versions
* system (set by /usr/local/opt/jenv/version)
$ unset JAVA_TOOL_OPTIONS
$ jenv add /Library/Java/JavaVirtualMachines/1.6.0.jdk/Contents/Home
oracle64-<IP_ADDRESS> added
<IP_ADDRESS> added
1.6 added
$ jenv add /Library/Java/JavaVirtualMachines/jdk1.7.0_79.jdk/Contents/Home
oracle64-<IP_ADDRESS> added
<IP_ADDRESS> added
1.7 added
$ jenv add /Library/Java/JavaVirtualMachines/jdk1.8.0_60.jdk/Contents/Home
oracle64-<IP_ADDRESS> added
<IP_ADDRESS> added
1.8 added
$ jenv rehash
$ exec $SHELL -l
$ jenv versions
* system (set by /usr/local/opt/jenv/version)
  1.6
  <IP_ADDRESS>
  1.7
  <IP_ADDRESS>
  1.8
  <IP_ADDRESS>
  oracle64-<IP_ADDRESS>
  oracle64-<IP_ADDRESS>
  oracle64-<IP_ADDRESS>

Is this as intended and should having environment variables defined which cause the first line of java -version to report Picked up ... cause the version naming clash?

@vicchi that is as intended. The "canonical" version is oracle64-<IP_ADDRESS>. <IP_ADDRESS> and 1.8 is added for convenience.
In my case, I have 2 versions of Java 8. I want 1.8 to point to the latest version of Java 8
lrwxr-xr-x  1 hanxue  admin  63 Feb  2 16:59 1.8 -> /Library/Java/JavaVirtualMachines/jdk1.8.0_72.jdk/Contents/Home
lrwxr-xr-x  1 hanxue  admin  63 Jun 11  2015 <IP_ADDRESS> -> /Library/Java/JavaVirtualMachines/jdk1.8.0_45.jdk/Contents/Home
lrwxr-xr-x  1 hanxue  admin  63 Feb  2 16:59 <IP_ADDRESS> -> /Library/Java/JavaVirtualMachines/jdk1.8.0_72.jdk/Contents/Home
lrwxr-xr-x  1 hanxue  admin  68 Jun 11  2015 oracle64-<IP_ADDRESS> -> /Library/Java/JavaVirtualMachines/jdk1.8.0_45.jdk/Contents/Home
lrwxr-xr-x  1 hanxue  admin  63 Feb  2 16:59 oracle64-<IP_ADDRESS> -> /Library/Java/JavaVirtualMachines/jdk1.8.0_72.jdk/Contents/Home

With this, I only need to type $ jenv version 1.8 instead of $ jenv version oracle64-<IP_ADDRESS>.

Thanks. I can see that pull request #127 is supposed to resolve this (as well as issue #107) but assuming this is included in v0.4.4, it's still an issue.
I've got 3 JDKs installed on my local machine at the moment and I've manually done a jenv remove on all of them.
$ jenv --version
jenv 0.4.4
$ jenv versions
* system (set by /usr/local/opt/jenv/version)
$ ls /Library/Java/JavaVirtualMachines/
1.6.0.jdk       jdk1.7.0_79.jdk jdk1.8.0_60.jdk

I still have JAVA_TOOL_OPTIONS set as an environment variable, and when I re-add a JDK ...
$ jenv add /Library/Java/JavaVirtualMachines/jdk1.7.0_79.jdk/Contents/Home
oracle64-Picked added
Picked added
There is already a Picked JDK managed by jenv
Do you want to override (type y to confirm)? ^C

If I unset JAVA_TOOL_OPTIONS and re-add ... success, or sort of. The oracle64-Picked "version" smells wrong to me.
$ unset JAVA_TOOL_OPTIONS
$ jenv add /Library/Java/JavaVirtualMachines/jdk1.7.0_79.jdk/Contents/Home
oracle64-<IP_ADDRESS> added
<IP_ADDRESS> added
1.7 added
$ jenv versions
* system (set by /usr/local/opt/jenv/version)
  1.7
  <IP_ADDRESS>
  Picked
  oracle64-<IP_ADDRESS>
  oracle64-Picked

@hanxue @klursh ... any thoughts? I don't want to keep banging on about this as jenv is a vital part of my workflow and I'm really grateful it exists, but this just feels like the wrong behaviour even though unsetting any JAVA_ environment variables is a work-around.

I will try to fix it

Awesome. Thank you!

Thanks @gcuisinier - can I assume this will be wrapped up in v0.4.5, when that gets released?

Interested by this change as well

I will try to find time to check and release if all is ok !

@gcuisinier thanks :)

still not solved ?

still not fixed


Add missing return types

This PR adds missing return types to several methods:

clearAll(): self
clearScreen(): self
die(): void
html(): self

Thanks!


Control doesn't become read-only when the form becomes read-only

I am using javascript to change the forms state (Active -> Inactive) or (Inactive -> Active).
Xrm.WebApi.updateRecord('entity_name', primaryControl.data.entity.getId(), {
    statecode: 2, // Inactive
    statuscode: 2 // Inactive
}).then(() => {
    formContext.data.refresh(true);
});

On success I refresh the form, all of the other controls on the page change to the appropriate edit mode (Editable or Read-only) but this control does not.  If I refresh the page using F5 the control then has the correct edit mode.
This is happening in v2.5 (I saw Issue #4 said this was fixed in 2.2 but still seems to be an issue.)

Thank you for opening an issue for this.
I will have a look at this in the next days.

It seems that the Read-only flag got broken in V2.5. I fixed this today - Please update to current solution V2.6.

Dankeschön Chris, I can confirm that it is working now.


In order to save on space as a backend dev, I want quantum computing

Please

Du er en quantum computer ;)


Transactional shorthand until we get the 2.12 Java Lambda interop

graph.transactional { graph =>
  graph.addVertex(...)
  graph.addVertex(...)
  ...
}

does all actions in ACID

LGTM

That's useful for now but I think once we have 2.12 and ditched 2.11 (pfiou) that method will not have anything better than .tx.submit()

agreed. by then we can drop a number of helper methods from the package object.


XIONE-9937: CEC bus busy write failures

Reason for change:
CEC bus busy write failures
Ping prits tracking
Test Procedure: None
Risks: Low
Change-Id: Ic8e4a5a62082401db88a515e091da1c14bc5af3b Signed-off-by:Anooj Cheriyan<EMAIL_ADDRESS>

wrong patch


Is it RESTful to have chirps and chirp routes?

I'm not an expert in that field, but AFAIK a RESTful API should use a single endpoint for all matters regarding a resource. In particular I think you should use

GET /chirps to get all chirps
GET /chirps/<id> to get the chirp with the given id

You're right. I wanted it to do the way the GitHub API is modeled, but got confused with the way the GitHub web UI build URLs:
Web UI:

https://github.com/britter/reactive-chirps/commits - List of all commits
https://github.com/britter/reactive-chirps/commit/:sha - single commit

API:

https://api.github.com/repos/britter/reactive-chirps/commits - List of all commits
https://api.github.com/repos/britter/reactive-chirps/commits/:sha - single commit

I pushed #2 as proposal to fix this.


Is clientjs an idled project?

Is it abandoned?
As the code has not been touched for a year, is it recommended to use in production, in terms of bug reports followup?
Will it have any future major release?

If you are looking for a different library, there is the FingerprintJS library, which has an accuracy between 40% and 60%, but it is fully open source.
ClientJS itself has been built upon Valve/fingerprintjs, which then evolved into fingerprintjs/fingerprintjs.

ClientJS is not built on FingerprintJS,but you should def use FingerprintJS! It's professionally maintained, and does provide good accuracy for an open source, client side only product.

I apologize: I was "quoting" the project page, which says Built Upon: and shows three links, including https://web.archive.org/web/20200714191004/https://github.com/Valve/fingerprintjs.
I did not mean this library has code that comes from that library, but that the project page has links to other library, which I take as meaning These are other libraries with a similar purpose. If I were to look to alternatives, I would check those three links.

https://github.com/fingerprintjs/fingerprintjs just change its license so it can't run in production anymore

3.x and lower versions are still licensed under MIT.

is this true? I can use it on production?

I can use it on production?

Changes are not retroactive. The code in version4 is still readble, but not open source anymore! By version3 you will be stuck on an immobile creature!

Yes, Branch 3.x is still licensed under MIT. I guess they will not back-port to branch 3.x the changes they will introduce in branch 4.x, though.


[6.x] JoinClause::on $second also supports Expression

While it might seem it supports anything which is supported by \Illuminate\Database\Query\Builder::where, in fact it doesn't. E.g. you can't pass literal values (string or int values); string is strictly interpreted as column name.
However expression is correctly accepted to work this around and pass a literal integer like this:
            ->join('other_table', function (JoinClause $join) use ($someId): void {
                $join
                    ->on('other_table.my_table_id', '=', 'my_table.id')
                    ->on('other_table.some_id', '=', DB::raw($someId));

This should work:
            ->join('other_table', function (JoinClause $join) use ($someId): void {
                $join
                    ->on('other_table.my_table_id', '=', 'my_table.id')
                    ->where('other_table.some_id', '=', $someId);

🤦♀
@dkulyk thanks, guess you're right! Should have paid more attention.
Should we close it then as being an unnecessary implementation detail that Expression in fact does work on on or is this still a valid PR?


WeatherMap sort order

Beta 4.
1> Adjust the order in console->management->weathermaps, and the order of the actual display pages remains unchanged.
2> In console->management->wathermaps, click move map up/down to move one line, and after refreshing, it is found that two lines have been moved.

So, you need to restart what is wrong more precisely.  The order does work on the management page, so what is wrong now from your perspective.

After adjusting the order, the order on this page does not change.

click move map up/down to move one line, and after refreshing, it is found that two lines have been moved.

Okay.  In the back end, you sequencing is mangled up with more than one of the maps sharing the same sequence.
We had this at one time in cacti, and we switched the algo.

I made a few updates to help you diagnose this.

So how should I test now? I downloaded your updated version. The sorting is still unsuccessful. It seems that no matter how the value of Order changes, the value of ID is always used to sort.

It looks like the sorting is correct on the Management page, but not good on the Top Tab Weathermap page.

Okay, fixed now.


Allow ranges to be list members of valid_values constraint

Description of changes:
This PR allows ranges to be list members of valid_values constraint.
e.g.
type::{
  type: decimal,
  valid_values: [
    range::[-20d0, -10d0],
    0d0,
    range::[10d0, 20d0]
  ]
}

Test:
https://github.com/amzn/ion-schema-tests/pull/8

Changes:

if valid_values is a range then the implementation wraps it into IonList and stores the range with buidlRange.
Otherwise, it uses buildRange to either build a range or keep the value as it is for validation.
Stores all the values in builtValues which later is used in validation rather then building range for every value in validation.


Do not switch to the next code segment after pressing Enter

When I enter two spaces and press Enter again, I find that the original style does not change, for example.

hi,
try to use config option useLineBreak: true

你好，
尝试使用配置选项useLineBreak: true
或\和
在标题和文本之间添加空间行

Adding  empty line is feasible, but useLineBreak: true does not work.

oh, sorry, about useLineBreak: true - my mistake, this parameter works different.
yep, just use empty line after a header


Docker bash completion

https://docs.docker.com/docker-for-mac/#/installing-bash-completion

A bit of work, and i'm not using docker at the moment. Closing.


Battery.ipynb no longer reporting

As described on Facebook foxess owners page the app tries to load soh which my V1 ECS batteries don't have the capability to report. My batteries are 4x ECS 2900 BMS is version 1.017 slaves are all version 1.12 the app was working fine until the other day.

Please ignore above issue, apparently is now working properly again tvm

Thanks.
I pushed an update eariler and another one just now. Although it no longer crashes, there is a chance it does not report the battery capacity or residual correctly. I've also added an estimated SoH based on the data Fox returns.
If you can, please can you run the script again (using v1.7.5) and see what it reports. Please can you also run this additional cell below and paste a screen shot of the contents. I'm particularly interested to see the first 7 characters of the BMS serial number and the BMS firmware version as this will help to identify the correct residual handling.
f.batteries

V1.7.5 now not reporting anything again. f.batteries script reports as follows

V1.7,5 reports as follows

OK, thanks for posting the info. Clearly, the problem here is that Fox are returning the ratedCapacity of your battery as 0. HV2600 BMS v1 and v2 both return a value for that. I can check for this to stop the error.
Out of interest, what do you see when you look at the battery at foxesscloud.com? The rated capacity is shown as 'Nominal Capacity' here:

It's also shown in Fox App v2 if you tap the Battery, select Detailed Parameters and then look at Capacity.

I've pushed v1.7.6. Hopefully, this will resolve the problem of ratedCapacity being returned as 0 and causing the divide by zero error. Please give this a go and confirm if it now works you when you get a chance.


Update Netron to 5.0.9

null
Microsoft Reviewers: Open in CodeFlow

What? How is it possible? 🤔😕

PRs with deleted branches can also be merged; since it's stored in the database and not completely deleted.


new(tests): EOF - EIP-6206: clarify "non-returning instruction"

🗒 Description
Recently some confusion was reported whether what are "non-returning instructions" mentioned in the test. It looks like a mental shortcut, and can also raise questions like "is RETURNCONTRACT or RETURN one?". This clarifies this and also adds some coverage for such instructions (which are basically "terminating but not RETF or JUMPF-to-returning").
Also fixes a typo in that file's name.
🔗 Related Issues
NA
✅ Checklist

[ ] All: Set appropriate labels for the changes.
[ ] All: Considered squashing commits to improve commit history.
[ ] All: Added an entry to CHANGELOG.md.
[ ] All: Considered updating the online docs in the ./docs/ directory.
[ ] Tests: All converted JSON/YML tests from ethereum/tests have been added to converted-ethereum-tests.txt.
[ ] Tests: A PR with removal of converted JSON/YML tests from ethereum/tests have been opened.
[ ] Tests: Included the type and version of evm t8n tool used to locally execute test cases:  e.g., ref with commit hash or geth 1.13.1-stable-3f40e65.
[ ] Tests: Ran mkdocs serve locally and verified the auto-generated docs for new tests in the Test Case Reference are correctly formatted.

I'll go ahead and merge here as there are no fw changes involved


Fixing test failure in linux and mac agents and addressing previous c…

…omment.
Test was failing in Linux and mac agents because of alphabet casing.

Interestingly, test is failing on PR build but its passing on the build with topic branch..


Calculus comes first before Linear Algebra?

Hi.  I am starting out the course.  My minor gripe though was calculus 1 and 2 came first before linear algebra.  I thought it should be the other way around?  Please verify.  Thank you.

Calculus actually does come before linear algebra in a standard curriculum.
I do question why calculus comes after mathematics for computer science though as it's listed as a direct prerequisite if you look at http://ocw.mit.edu/courses/electrical-engineering-and-computer-science/6-042j-mathematics-for-computer-science-fall-2010/Syllabus/

I took both clasess when I was doing the Bioengineering degree, Calculus was before Linear Algebra.

Thank you guys for your collaboration.
I'll move the Calculus section and it will come before Discrete Math.
So we will have the following sections:
Calculus
Discrete Math
Algorithms
...


perlPackages.WWWCurl: fix build

Include patch from AUR to fix compatibility with newer libcurl.
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[X] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@GrahamcOfBorg build perl530Packages.WWWCurl perl528Packages.WWWCurl

Fixed darwin build issue.

@GrahamcOfBorg build perl530Packages.WWWCurl perl528Packages.WWWCurl

@GrahamcOfBorg build perl530Packages.WWWCurl perl528Packages.WWWCurl


How to set focus?

How to set focus on search view, when i open activity?

you can use         mSearchView.setSearchFocused(true);


Missing from registry?

What happened?
A user reports not finding this package in pulumi-registry which I can confirm. https://github.com/pulumi/pulumi-yaml/pull/576#issuecomment-2105022819
Example
N/A
Output of pulumi about
N/A
Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

Also do note that the str package contains the following functions

And of those trimSuffix, trimPrefix, replace are all present in the std package. Even the replace from the regexp module is present in the std package. I am not sure if the replace and match from the regexp module is present or not.