text:
Flat Checks don't follow automatic success and failure rules for DCs of <=1 or >=21

Player Core page 405 states "In the rare circumstance that a flat check has a DC of 1 or lower, skip rolling; you automatically succeed. Conversely, if one ever has a DC of 21 or higher, you automatically fail."
Currently a natural 20 succeeds on a flat check with a DC of 21 and a natural 1 fails a DC 1.

Hesitant to act on this since it would lock out critical success or failure results merely by the DC being out of that range.

Sounds like that's the intent for flat checks specifically to me.

I feel it's more likely that they saw an opportunity to invite the reader to skip a roll and didn't stop to think it through.

It could be that they didn't think the 4 degrees of success mattered for flat checks, then forgot about that when it came time to do inventor.
I agree with the hesitancy.

Or, hell, dying

We are more interested in what is affected than we are in what isn't.

authors: CarlosFdez stwlam websterguy


text:
Restore blank select option to weapon sheet range dropdown

Closes #14698

I don't think this fixes the bug, it seems like this is a deviation from behavior still. I brought up the original select and this is what it was:
{{#select item._source.system.range}}
    {{#unless mandatoryRanged}}<option value="">{{localize "PF2E.Item.Weapon.NoRangeMelee"}}</option>{{/unless}}
    {{#each weaponRanges as |label range|}}
        <option value="{{range}}">{{localize label range=range}}</option>
    {{/each}}
{{/select}}

It seems that mandatoryRanged weapons should not have a blank entry. I suspect what isn't actually working is the disabled on the entire select.

I'm in a game right now and probably shouldn't have done this right before. I think this is correct now?

Looks correct!

source: gharchive/pull-request

license_type: permissive

license_source: github-api


text:
Colon in input box becomes black if written after \u07EE

A strange issue for sure, but if you write "\u07EE :", the colon becomes black (doesn't matter if you put space in between). For copy:
߮ :

This seems to be the case for other "special" characters aswell, like ; - + # ~ $ etc. Also numbers.
Letters seem to be fine.
Additionally I need to press backspace twice to delete the character.

cannot replicate anymore in nightly
@pajlada close?

Does this still happen? I tried it but I can see the : maybe it had to do with an old qt version or something

yeah seems to be solved somehow

@TETYYS close please

source: gharchive/issue

license: mit

license_source: bigquery


text:
Specify system emoji fonts for emoji code-points

Mac OSX & Ubuntu in particular have bugs relating to the proper display of emojis in browsers. Explicitly specifying locally-installed system emoji fonts as fall-backs at the end of the font stack fixes the issue! 🤷
Documented here: https://www.client9.com/css-color-emoji-stack/

A big thank you @barneycarroll

authors: barneycarroll foxdonut

repo: foxdonut/meiosis


text:
Not working in streaming app

I believe i installed it correctly, whenever i click on the stream it turns green and shows buttons are working in preview but i get no input in the actual stream?

This is a common theme that is happening to other people but i am just unable to replicate this issue, are you using the Xbox App or Xbox Beta App?

Usually it's that the "new controller" attached isn't the active controller in the game because the user started with a physical controller attached, etc.  If that's the issue, XboxKeyboardMouse's guide button (by default the ` key) will still be bringing up the guide where input will start working.  Then Switch profile -> select your gamertag, and boom... this virtual controller is now the active controller for your gamertag.
Hopefully that helps.  This issue is stale though, so I'm closing it.  (PS there's a new version available with bugfixes.)

repo: fqlx/XboxKeyboardMouse


text:
Remove image-size logic from cellpose_segmentation task

We have this block of code in cellpose_segmentation.py:
    # Extract image size from FOV-ROI indices
    # Note: this works at level=0, where FOVs should all be of the exact same
    #       size (in pixels)
    FOV_ROI_table = ad.read_zarr(f"{zarrurl}tables/FOV_ROI_table")
    list_FOV_indices_level0 = convert_ROI_table_to_indices(
        FOV_ROI_table,
        level=0,
        full_res_pxl_sizes_zyx=full_res_pxl_sizes_zyx,
    )
    ref_img_size = None
    for indices in list_FOV_indices_level0:
        img_size = (indices[3] - indices[2], indices[5] - indices[4])
        if ref_img_size is None:
            ref_img_size = img_size
        else:
            if img_size != ref_img_size:
                raise Exception(
                    "ERROR: inconsistent image sizes in "
                    f"{list_FOV_indices_level0=}"
                )
    img_size_y, img_size_x = img_size[:]

This was probably a leftover from previous implementation, and the image size information is not used in the rest of the task.

Will be closed by https://github.com/fractal-analytics-platform/fractal-tasks-core/pull/306/commits/6b345ca1a5e303e71bbd0bb3b8caf89e39632ae0, as part of #306


text:
Shall we keep the "info" page?

The current Homepage may be dropped as part of https://github.com/fractal-analytics-platform/fractal-web/issues/242.
I'd be in favor of keeping an "Info" page, with a small (one-paragraph) description of what the user may want to do, or where to get support. Or, otherwise, we can simply drop the "info" page.

Good idea, I'd be in favor of that.
Contents for the page:

Links to all relevant documentation
Links to the splash page (like the one in bottom left)
Some extra explanation text that can be set per deployment (i.e. giving a different text on billing, a different support link or email depending on institution)

Recap (also merging #242 into this issue):

As soon as a user is logged-in, there should be a redirect towards the Projects page.
The Home and Info pages should be merged into a single page (I suggest to name it Home).
The Home page should be visible to anybody, not only logged users.
Remove Fractal organization page link and Fractal splash page link from the footer; I guess there is no reason to always show them, and it's enough to show them in the home page.
Should we also remove the fractal-server version from the footer, and only show it in the home page? For now, I'd be in favor of keeping it (maybe with a more explicit phrasing, like "fractal-server X.Y.Z").

Contents of the home page:

A small (one-paragraph) description of typical actions the user may want to do. This includes a link to the login page, which should be made extremely visible.
A sentence about where to get support (this has to be an env variable, since it may change for each Fractal instance).
Links to relevant documentation pages, and to the Fractal splash page.
Abything else?

Sounds good!

Should we also remove the fractal-server version from the footer, and only show it in the home page? For now, I'd be in favor of keeping it (maybe with a more explicit phrasing, like "fractal-server X.Y.Z").

Let's keep this at least for the time being, it's quite useful to see what version things run when we run into issues :)

A sentence about where to get support (this has to be an env variable, since it may change for each Fractal instance).

Very nice!

Anything else?

I think that should cover it for the moment :) We can eventually add the Fractal logo there, once we have one.

authors: jluethi tcompa

repo: fractal-analytics-platform/fractal-web


text:
is there any plans of developing material design search pattern?

[] bug
[x ] enhancement
[x ] feature-discussion (RFC)

is there any plans of developing  material design search pattern?
I think we need the two types

Persistent search
Expandable search

If not plans I can create a plugin for that

Not at the moment. Would be great to see plugin from you ;)

Issue is closed because of outdated/irrelevant/not actual
If this issue is still actual and reproducible for latest version of Framework7, please create new issue and fill the issue template correctly:

Clearly describe the issue including steps to reproduce when it is a bug.
Make sure you fill in the earliest version that you know has the issue.
Provide live link or JSFiddle/Codepen or website with issue

license: MIT


text:
MissingInputException in line 145 of /home/yw/desktop/metaGEM/Snakefile:

Dear Francisco,
Hi!
In order to solve errors cauesd by snakemake version, I donwloaded a file "snakemake-5.10.0.tar.gz". Then I replaced the original snakemake file in "metaGEM/envs/metaGEM/snakemake" by new snakemake file in "snakemake-5.10.0.tar.gz". When I tried to run a commond {bash metaGEM.sh -t fastp -j 2 -c- 2 -m 20 -h 2}, it showed errors as followed:
Unlocking snakemake ...
Building DAG of jobs ...
MissingInputException in line 145 of /home/yw/desktop/metaGEM/Snakefile:
Missing input files for rule qfilter:
/home/yw/desktop/metaGEM/dataset/testdata1/testdata_R2.fastq.gz
/home/yw/desktop/metaGEM/dataset/testdata1/testdata_R1.fastq.gz
Looking forward your reply,
Kind regards,
yw

Hi Yuwei,
Thanks for opening an issue, in the future please open issues in the main metaGEM repo.
Could you please check that the short read files are present and properly named in the input folder? e.g.
ls -la /home/yw/desktop/metaGEM/dataset/testdata1/

Please make sure that the files are present in this folder, and also that they are named as expected by the metaGEM output, i.e. testdata_R1.fastq.gz & testdata_R2.fastq.gz. For more on this please refer to the quickstart page in the wiki.
Best wishes,
Francisco

authors: franciscozorrilla tan-yuwei


text:
🛑 SIU Guarani is down

In f669266, SIU Guarani (https://siufai.uncoma.edu.ar/informatica) was down:

HTTP code: 500
Response time: 16771 ms

Resolved: SIU Guarani is back up in de8579b.


text:
Fix race condition in execute Do method

Hi, Detected race condition when execute Do concurrency.
WARNING: DATA RACE
Write at 0x0000035980d0 by goroutine 47:
  github.com/me/myproject/vendor/github.com/franela/goreq.Request.Do()
      /home/koya/repos/src/github.com/me/myproject/vendor/github.com/franela/goreq/goreq.go:311 +0x3bf
    ~~~~
    ~~~~
    ~~~~

Previous write at 0x0000035980d0 by goroutine 40:
  github.com/me/myproject/vendor/github.com/franela/goreq.Request.Do()
      /home/koya/repos/src/github.com/me/myproject/vendor/github.com/franela/goreq/goreq.go:311 +0x3bf
    ~~~~
    ~~~~
    ~~~~

So I changed coping DefaultClient entity into local memory from global memory.
Thank you.

Please review this. 🙇  @marcosnils

repo: franela/goreq


text:
Wrong characters in the object. [""]

Unfortunately the adapter generates the characters [""] and I cannot process the filtered number any further.
How can I remove the characters?

https://forum.iobroker.net/topic/8672/neuer-adapter-systeminfo-auf-git-und-npm/63

As mentioned in the forum already, the "" are generated because this is a string and not a number. The reason is because javascript cannot parse European numbers with , as decimal point, you nee to use . for decimal point. To convert this string to a number you could write Number(_string_.replace(",","."))

authors: capitaenz frankjoke


text:
C++20 Module Support

How about support for modules?
At least for Clang, C++ modules (given as '.cppm' files) need to be precompiled using a '--precompile' option. This produces a '.pcm' file that regular compilation needs.
So we need an additional precompile phase whose output should go into a modules folder. That folder has to be provided to the regular compilation phase using the '-fprebuilt-module-path' option.
GCC has something similar but I haven't tried it. I think a general precompile phase for certain files (ie 'cppm' or some user specified type) will cover both compilers. Perhaps inputs for module folder and module specific switches as well?

I will wait until c++23 is fully implemented in gcc and msvc.


text:
Presence of zero values deforms pie charts

Actual Behaviour

Steps to Reproduce:
{
	"height": 350,
	"title": "Fatturati: confronto Network (apr 2018)",
	"type": "pie",
	"colors": ["#7ED321", "#007bff", "#fc0", "#FDA5EE", "#9c6d1e", "#c7b6f7"],
	"data": {
		"labels": ["Blunet", "Bluvacanze", "Cisalpina Tours", "Primarete Viaggi e Vacanze", "Travelbuy", "Uniposte"],
		"datasets": [{
			"name": "apr 2018",
			"values": [5270381, 0, 0, 3249212, 1033141, 377]
		}]
	}
}

Frappé Charts version: 1.1.0 (iife)

I'm having the same issue but with small values!! 😞

@eyewritecode : its not due to the small values. the graph behaves unpredictable when you have values subtotal less that 100.

Fixed: https://github.com/frappe/charts/pull/181

authors: Eyewritecode kedar9444 massimo-cassandro scmmishra

repo: frappe/charts


text:
What's the module "upt" in inference.py?

I try to run the inference.py to to test the model and get error ModuleNotFoundError: No module named 'upt'. After running pip install upt and executing the inference.py again, the program prints error: ImportError: cannot import name 'build_detector' from 'upt' (/opt/conda/envs/pvic/lib/python3.8/site-packages/upt/__init__.py).
I think I misunderstand the source of upt. And I find a UPT model from UPT: Unary–Pairwise Transformers. Is this source right? If so, how to combine it with your project?

Your guess is correct. You need to read the readme in the UPT carefully.

Hi @musicBoxes,
I have been a bit busy lately, and have not cleaned up the code for this work. I will most likely wrap it up by the end of the month. In the meantime, I think you could just change the name of the module to pvic and it should work.
Fred.

Hi @musicBoxes,
The codebase for inference and visualisation has now been updated.
Cheers,
Fred.

authors: fredzzhang musicBoxes yaoyaosanqi

repo: fredzzhang/pvic


text:
Beta: Can't view other campers' public profile

I'm curious how I (or other campers) would view the 'public profile' of other campers
In the current live FCC I can view other campers' progress and projects by going to the url/campers name (not super obvious, but findable) - In beta, to view a profile, I have my id in the url which is a giant string of random characters, which makes impossible to view other profiles

A few things:

The user profiles will now become available at /portfolio/user-name
The long id which you are talking about is actually your username and is very much customiszable, by the user.
When rolling over beta to production your old username will be retained, you will not see that long username if not already set to one of your choice.
Only the new signups will have a unique UUID like username, which again is Point 2 above.

Finally we never had discoverability other than in the chat rooms and forums where users would share their profiles with their fellow campers.
We would keep it that way, the user profile belongs to the user and it will be highly customizable, and can only be made known to others, when the choose to.

I have closed this because there is not much to be done here in terms of the UX. The settings page already has all the options that new users will be able to use and customize their profiles.
Returning campers with a username will not face this and even they can update the username now, if they wish to (and a ton of other stuff in their bio, if they choose to).

repo: freeCodeCamp/freeCodeCamp


text:
Cloned repo but can't run 'npm run unpack'

Describe your problem and - if possible - how to reproduce it
I am running a local repo of freeCodecamp and am on a new branch but when I run 'npm run unpack' I am getting error messages.
The Error says that it can't find /Desktop/FreeCodeCamp/freeCodeCamp/seed/unpack.js'
I am trying to fix the html on one of the challenges but can't seem to access the challenges this way.
Any reason this is the case?
Add a Link to the page with the problem
Tell us about your browser and operating system

Browser Name:
Browser Version:
Operating System: OSX

If possible, add a screenshot here

We have refactored our repositories to be a mono repo now. Please re-clone this repo and make a fresh installation

authors: JamesStaffordCoyte raisedadead


text:
Test failing in: Applied Visual Design: Create a Horizontal Line Using the hr Element

Describe your problem and - if possible - how to reproduce it
Second part of test is failing, incorrectly stating that "The hr tag should come between the title and the paragraph."  although actually  tag is already placed and displayed in results view as it should be.
Add a Link to the page with the problem
https://learn.freecodecamp.org/responsive-web-design/applied-visual-design/create-a-horizontal-line-using-the-hr-element
Tell us about your browser and operating system

Browser Name: Chrome
Browser Version: Version 67.0.3396.99 (Official Build) (64-bit)
Operating System: Mac OS

If possible, add a screenshot here

I can confirm I am having this problem too. Been talking about it on the forum here - https://forum.freecodecamp.org/t/create-a-horizontal-line-using-the-hr-element-is-this-bug/212450/6
Seems like a recent problem...

Same  issue for me too.

Here a link
https://learn.freecodecamp.org/responsive-web-design/applied-visual-design/create-a-horizontal-line-using-the-hr-element/
About browser:
Browser Name: Chrome
Browser Version: 67.0.3396.99 (Official Build) (32-bit)
Operating System: Windows 10
Here is screenshot

Same Here

Same here.

Same issue. I have seen references to a fix but have been unable to find. Anyone with resources here?

same issue, using chrome.

i am also having same issue

I am also having the same issue. It made me feel better to see I wasn't going crazy with this simple task.

Same issue in Chrome

Same here, tried it a bizillion times, good to see I'm not the only one.

Try to switch you  tag like this.. This work for me..

Hope it helps, for someone who want to pass this test..

@asheavenist That solution worked for me! Thanks!

@asheavenist That solution worked for me as well, which is funny since I threw the emphasis tag around the paragraph in Use the em Tag to Italicize Text and it made me move them inside . Thanks

@asheavenist  thanks its work for me :+1:

@asheavenist Awesome. It works.

 and  switch worked!

@asheavenist thanks for bringing the solution, while i still have motivation to learn fcc

Thank you very much!  It worked, and now I can sleep.
On Sun, Jul 29, 2018 at 9:19 PM, asheavenist notifications@github.com
wrote:

Try to switch you

tag like this.. This work for me..*

[image: fix solution]
https://user-images.githubusercontent.com/36536058/43372953-878b6872-93d0-11e8-9b42-246145e359b3.JPG
Hope it helps, for someone who want to pass this test..*

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/freeCodeCamp/freeCodeCamp/issues/17944#issuecomment-408720872,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AnP_W2VSYM2weLkFR471YxKKkFT17e03ks5uLl8KgaJpZM4Vk-Xj
.

Did not work for me. My em and p were in the correct order, to begin with.

Switching the  tags to being outside of the  worked for me too!

Switching the 'em' tag to enclose the 'p' tag worked for me too

@asheavenist thanks bro! U r a genious 💃

I moved the em to before the p tag and all is working now thank you

Getting it to pass is one thing, but is that really the intention?
Users are going to get stuck here, and it can be very demotivating when it will say it is wrong when it is correct. Maybe can you swap the tags for the default script when starting this challenge?

@povilasraciunas I am attempting to solve this issue. I anticipate finding a solution tonight. Atm, I am stumped as to how swapping the em and p tags could pass the test as that does not fit the regex at all. This would be my first pull request and with the changes made recently to the structure of the repos ( moving challenges from /seed to the curriculum repo) without updating the contributing instructions is giving me some confusion as well.

The contributing of curriculum can be found in docs of curriculum repo. Here’s the link

https://github.com/freeCodeCamp/curriculum/blob/dev/docs/CONTRIBUTING.md

Hi Joe, thank you! Yeah, that "trick" of putting em tags outside to make
the test pass was interesting :)
Of course, it is not a big deal for me as I move on with other challenges,
but still wanted to report it as fcc is really great place for such
beginners like myself and I want to contribute at least by reporting bugs :)
On Wed, Aug 1, 2018 at 2:06 PM Joe Andersen notifications@github.com
wrote:

@povilasraciunas https://github.com/povilasraciunas I am attempting to
solve this issue. I anticipate finding a solution tonight. Atm, I am
stumped as to how swapping the em and p tags could pass the test as that
does not fit the regex at all. This would be my first pull request and with
the changes made recently to the structure of the repos ( moving challenges
from /seed to the curriculum repo) without updating the contributing
instructions is giving me some confusion as well.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/freeCodeCamp/freeCodeCamp/issues/17944#issuecomment-409536371,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANPcbM5Jx7jWBQPIRRo5LuMedKoSgJY1ks5uMYvJgaJpZM4Vk-Xj
.

--
Pagarbiai
Povilas Račiūnas

Awesome! For those completists that don't want a skipped lesson on their conscience! Me...

thanks to you, it works....

Still an issue on 08/06/2018; asheavenist fix helped move one. Thanks!

Thank you @asheavenist!

Thank you @asheavenist it works!

FINALLY! Thank you 👍 @asheavenist

https://forum.freecodecamp.org/t/bug-in-applied-visual-design-create-a-horizontal-line-using-the-hr-element/212910/4
The above post brought me here. Definitely frustrating and hope it gets fixed soon! FCC is so beginner friendly that this contradicting bug makes it difficult to trust the course content. With that said, this has been such a great resource that I'm merely commenting here to report that I ran into the same issue and felt frustrated.
Thanks for all you do!

I wanted to create a PR for this but it looks like is already fixed on staging, can someone confirm?

@goncy Yep, the issue was fixed here: https://github.com/freeCodeCamp/curriculum/pull/189.
The reason the issue still occurs is because the UI has not been updated to use the latest curriculum version. They were broken up into separate components and the UI needs to import the curriculum in order to use it. The fix for this issue is in the curriculum version 3.1.2, while the UI is still using version 3.1.1.

exact same thing still happening to me.

Tried the solution from a comment above saying to put the  tags outside of the  tags, and it worked.  Thank you! :]

I can confirm also that the workaround is to switch the tags 'p' and 'em' like below:
` Alphabet

Google was founded by Larry Page and Sergey Brin while they were Ph.D. students at Stanford University.`
despite the fact that the right solution should be as follow:
`Alphabet

Google was founded by Larry Page and Sergey Brin while they were Ph.D. students at Stanford University.`

Yep! Switching tags 'p' and 'em' is working :)  Thanks guys!

Switching tags word for me too .....

Thank you! Switching the tags 'p' and 'em' worked for me too!
Glad I follow the direction: "When you get stuck, remember: Read-Search-Ask"
read the documentation or error
Search Google
Ask for help
search google is definitely the way to go!!! yeah

@asheavenist Thanks!! it worked for me

How did someone figure out that switching other tags would fix this? Glad it's a workaround and I found it.  There is nothing more annoying to me than having that one incomplete problem in the lists of problems

@asheavenist Thank you

@asheavenist - would you mind explaining how you figured out this solution?

Hey guys wanted to let everyone know you are doing it right, but you have to switch the em element with the p element. I put ** buy the corrections so you guys can see. Scroll down just past your
.cardText {
margin-bottom: 30px;
}

Alphabet

      ****Google was founded by Larry Page and Sergey Brin while they were Ph.D. students at Stanford University.****

Larry Page
Sergey Brin

Why is no-one pointing out that the “workaround” is invalid HTML? How many new coders will now think that it’s perfectly OK to nest p tags within em tags because it was suggested here and passed the “test” on here?

Why is no-one pointing out that the “workaround” is invalid HTML? How many new coders will now think that it’s perfectly OK to nest p tags within em tags because it was suggested here and passed the “test” on here?

They are addressing it, see here:
https://github.com/freeCodeCamp/curriculum/pull/189

Why is no-one pointing out that the “workaround” is invalid HTML? How many new coders will now think that it’s perfectly OK to nest p tags within em tags because it was suggested here and passed the “test” on here?

They are addressing it, see here:
freeCodeCamp/curriculum#189
Valid point though.

Thank you for the link, I'd missed that one. It's nice to see the updates.

Looks like this just got fixed. It passes for me now with the correct code. :)

Can someone close this issue as it has been fixed?

authors: BryantCabrera Corneliaclassic Crazypari Doministic FoxNeo HansFernandez Hinageshichan Lluran Lninn LuckyStephy13 Nirajn2311 RaspberryLime Theloudneon adarshnanwani adonis8441 alwayzup asheavenist avidDIY bpkaur brewbart courseiam daldoy dhanush13-c goncy gxcad huyenltnguyen joeslost joncartee kathleeng77 kc-taylor lsumner marthehe nhankiat pepperH philheb povilasraciunas puyanwei rolandjo saneki26 slavo7 veryfunkymunky yohanisgobai


text:
CSS cascade and inheritance not introduced

While looking into #36108 I realised that inheritance and the cascade are not defined in the curriculum.  Some of the concepts are introduced at various points, but they aren't formally defined or really tied together.
Should we add some challenges to cover these concepts and, if yes, where?

I think the only way to properly do it is to introduce the basic ideas of both at the very beginning of the Introduction to Basic CSS and then rewrite (and possibly reorder) some of those challenges so that it is clear how each relates to the concepts.  Some of the existing challenges gloss over specificity which is directly related to the cascade concept.   This could end up being a major undertaking to pull this off in the current curriculum.  It might be best to introduce all of these concepts in a project of the new curriculum.  I don't see any project in that repo which would cover these concepts.

That might be the best idea.  One problem is Basic CSS is all very practical and fairly light on technical details, so a raft of definitions wouldn't really fit stylistically.

As we tell others on the forum, FCC is not meant to be comprehensive.  We teach just enough so users can get things going, but they will still be required to learn much more and asking many more questions outside of FCC to be well rounded.

As we tell others on the forum, FCC is not meant to be comprehensive.  We teach just enough so users can get things going, but they will still be required to learn much more and asking many more questions outside of FCC to be well rounded.

On reflection, since the curriculum is going to be reworked anyway, a big change to the existing curriculum seems premature.  Closing this on that basis.

authors: RandellDawson ojeytonwilliams


text:
Add a Visual Snapshot testing tool for visual tests

After hearing about Percy and giving it a try, I found it really powerful especially when used with Cypress. But I found some limitations with Percy and thought that I can overcome those obstacles, so I tried creating my own Visual Snapshot Testing tool.
So after a few days, now I have a working PoC up and running.
Here is a small preview of it's capabilities -
If this two images are given as input (think of the first image as the current visual state in the master branch and the second image as the requested changes in a PR.) it would mark the visual diff among them like the third image.
First Image (master branch)

Second Image (PR branch)

Detected Visual Diff

Syntax & Cypress Integration
As I said before, visual snapshot testing gets really powerful when used with Cypress. So I've tried to keep the Cypress integration simple yet powerful.
Let's say you want to take a visual diff of the landing page. Just type the following line in the Cypress test file -
cy.snap("Landing page");
Then run the visual snapshot testing tool -
visual_diff "npm run e2e:dev:run"
Advantages over Percy
These are advantages this homemade tool has over Percy -

Run visual tests locally in your PC with a single command (You can't do this with Percy as Percy only shows visual diff in CI)
No subscription fee

When to expect -
All this is quite exciting but I still have to iron out some rough spots before I can send you all the preview of the tool. I'll probably need a day or two get the preview ready. 😓

First off, this looks amazing!  Great work @Twaha-Rahman.
The main question is about the Cypress integration - what's the plan for showing off the diffs?  If a PR changes how something looks, how do I see that diff?

Going to close this as stale for now @Twaha-Rahman, if you have any follow up feel free to reopen.

authors: Sembauke Twaha-Rahman ojeytonwilliams


text:
Typo in css property name - RWD beta course

Affected page
https://www.freecodecamp.org/learn/2022/responsive-web-design/learn-basic-css-by-building-a-cafe-menu/step-76
Expected behavior
In the instructions,

Changing the bottom-margin to 5px looks great.

bottom-margin is an invalid css property. It should be margin-bottom.
Alternatively, we could remove the code quotes and replace it with bottom margin.
Screenshots

System

Device: Laptop
OS: Windows 10
Browser: firefox
Version: 98.0.1 (64-bit)

Additional context

I would like to contribute, am I eligible? This would be my first PR here


text:
[ Full Stack Cert ] - update description for video compilation lab

Let’s update the intro text to the following:
        "intro": ["For this lab, you will create a video compilation web page. This lab will give you the opportunity to practice working with the iframe element."]

Here is the file you need to edit
https://github.com/freeCodeCamp/freeCodeCamp/blob/main/client/i18n/locales/english/intro.json#L1731-L1734

This has been opened for contribution. The first comprehensive PR created will be reviewed and merged. We typically do not assign issues to anyone other than long-time contributors.
If you would like to contribute and have not read the contributors docs, please do so here: https://contribute.freecodecamp.org/#/
If you have any issues with contributing, be sure to join us on the contributors channel, or on the contributors sub-forum
Happy coding


text:
fix(common): Fix for loopProtect undefined error

Pre-Submission Checklist

[x] Your pull request targets the staging branch of freeCodeCamp.
[x] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[x] You have only one commit (if not, squash them into one commit).
[x] All new and existing tests pass the command npm test. Use git commit --amend to amend any fixes.

Type of Change

[x] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[x] Tested changes locally.
[x] Addressed currently open issue (replace XXXXX with an issue no in next line)

Closes #16260
Description

Edited the file so that the loopProtect code, which is in the head of
the iframe on www.freecodecamp.org, is in the same script tag as the
base challenge content that get's rendered to the page. While the
loopProtect code is not rendered to the page like the rest of the
challenge code, it's within the same scope and seems to function
normally.
BREAKING CHANGE: None that I know of, but will keep checking.

@scissorsneedfoodtoo Awesome! Thank you for doing this so quickly.
@BerkeleyTrue could you take a look at this code and this approach, and QA it if you think it's a sound approach?

Great work on this! Was able to run my new changes to ES6 curriculum and get the LoopProtect error to go away 👍

@BerkeleyTrue @Ethan-Arrowood Thanks for taking a look at this and evaluating its approach.
I can confirm that it works great on my end.
@scissorsneedfoodtoo thank you so much for this pull request!

No problem! I'm glad it worked. Was kind of just throwing things at the wall, but was happy to find a solution.


text:
added Go books section

including 2 Go books.  One for absolute beginner and the other one is about good practice in Go programming.

[ x] I have read freeCodeCamp's contribution guidelines.
[x ] My pull request has a descriptive title (not a vague title like Update index.md)
[x ] My pull request targets the master branch of freeCodeCamp.
[x ] None of my changes are plagiarized from another source without proper attribution.
[ x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

@tbushman Please QA.

Hi @qrius . Congrats on your first pull request (PR)! 🎉
Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 📝

authors: Manish-Giri Nirajn2311 qrius


text:
Update index.md

Added additional Terraform integration information.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Hi,
Thanks for this pull request (PR).
Unfortunately, we are marking this PR invalid for hacktoberfest. We believe it does not follow the recommended guidelines:

Quality Standards
In line with Hacktoberfest value #2 (Quantity is fun, Quality is key), we have provided examples of the quality standards we encourage. This applies mainly to beginners.

PRs that are automated e.g. scripted opening PRs to remove whitespace / optimize images.
PRs that are disruptive e.g. taking someone else's branch/commits and making a PR.
PRs that are regarded by a project maintainer as a hindrance vs. helping.
Something that's clearly an attempt to simply +1 your PR count for October.

Learn more at https://hacktoberfest.digitalocean.com/details

authors: RandellDawson tomkirish


text:
updated comment.md

I have rephrased some expressions.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

I have rephrased some expressions.

@suruihuang
Thank you for your contribution!


text:
Grammatical fix on line 1

Fixed sentence #2 under ##Output by adding "for"

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Congrats @nikhilrwadekar on your first pull request (PR)! 🎉
Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 📝

authors: RandellDawson nikhilrwadekar


text:
Edit wording/examples on PHP array documentation

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

Closed and Reopened this PR to attempt to resolve a specific Travis build failure.

Hi @exiam
It seems that similar changes have already been accepted earlier for this article you're editing, sorry about that. 😓
If you feel you have more to add, please feel free to open up a new PR.
Thanks again! 😊

If you have any questions, feel free to reach out through Gitter or by commenting below. 💬

Congrats @exiam on your first pull request (PR)! 🎉
Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 📝

authors: RandellDawson exiam


text:
Replaced translated CSS code in adjust-the-width-of-an-element-using-the-width-property.arabic.md

Replaced translated CSS, code shouldn't be translated.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

✨ Awesome! Congratulations on your first contribution to freeCodeCamp! ✨
Thank you for submitting a PR for this issue, @4ssil  . We're all really looking forward to your next contribution.


text:
Fixed grammar

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Thank you for opening this pull request.
This is a standard message notifying you that we’ve reviewed your pull request and have decided not to merge it. We would welcome future pull requests from you.
Reason: Since your PR was create, the master branch content has updated making some of your changes unnecessary.  Please re-sync your fork so you are looking at the latest version of master and then create a new PR if you find any changes warranted.
Thank you and happy coding.

authors: RandellDawson leecri


text:
Fix errors and expanded content

Made clear that an infinite decimal expansion does not mean irrational, then added some history, context for interest and explained that in most computing languages their existence can (ironically) be ignored.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

@BTmathic 👏👏👏

authors: BTmathic paulywill


text:
Deleting typo

'increment' instead of 'increament'

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Thanks for the contribution, @VedranaR

authors: VedranaR farisoft


text:
feat(api, client): integrate sentry for log reporting

[x] Remove Rollbar (everywhere)
[x] Add Sentry to API
[ ] Add Sentry to client (not a priority)

It's getting there.  Debugging stuff is now in its own commits.
Still need to

[ ] report useful errors in development
[ ] decide which errors should not be reported
[ ] remove rollbar etc

Note: At the minute, I am using the status/ping as a http monitor on https://status.freecodecamp.org for actively monitoring the API. I can change that to something else if we want this to be used with Sentry for debugging purposes.

@raisedadead if you want to test locally, it's ready to review as is.  However, before you deploy it to staging, I'd drop the DEBUG: log locally, break client ajax commit.
The /sentry/error and /sentry/wrapped endpoints might be useful for QA (I removed the /status ones, so we don't accidentally alarm the status page).

authors: ojeytonwilliams raisedadead


text:
Fix: Update video link for Relationship Building video challenge.

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Closes #39041

Travis says the build has passed but it hasn't reflected here

authors: beaucarnes thecodingaviator


text:
feat(client): Typescript migration frame runner

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Is part of #43778

Migrated File
/client/src/client/frame-runner.js

✅ rebased
✅ conflicts fixed
❓ Tests passed

conflicts fixed again - this should be good to go 🤞


text:
fix: simplify savedChallenges update

Give this a good review - I finished up kind of late.
I focused on the database writes, I didn't want to mess with the request responses or anything on the client - so there may be some questionable looking code.
It appears that this and Oliver's PR make it so the database doesn't write all that extra "stuff". That seems to be the case anyway - so that's good. Don't know why.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #46024

It appears that this and Oliver's PR make it so the database doesn't write all that extra "stuff". That seems to be the case anyway - so that's good. Don't know why.

:magic_wand:

I finished up kind of late.

and here I am with coffee in hand, still couldn't do it. Good job, I learned a lot from this PR and Oliver's PR.

authors: Sboonny moT01 ojeytonwilliams


text:
fix(client): import Tfunction from i18next

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Ref #49800
TFunction is local type in react-i18next, so this aims to import it from where it's valid

Different files, I should name these PRs better, but I was focused on finishing it fast


text:
fix(curriculum):Updated lessons in Number sorter project to better show how inputValues works

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #54434

@Ksound22 Thank you for the review! I will make the changes


text:
fix(api): use generic error message if CSRF error

fix: show generic error if CSRF error
test: update tests to reflect new response
refactor: generic500 -> genericError
fix: use default to serialize all generic errors
fix: provide correct schema for save-challenge
fix: allow for CSRF errors as well as other 403s

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

There's probably a better way to serialize 'default' errors, but I haven't found it. So, for now, I've gone with sharing the default error schema and using unions when the endpoint has a custom 500.
In the future, we probably won't need many 500 errors, since it should mean "something unexpected occurred" and so we're unlikely to be able to tell the user anything more specific than that. For now, I think this slightly messy approach will have to do.

tested locally, all tests pass.

authors: Sembauke ojeytonwilliams


text:
feat: add forum to news redirects

This PR adds redirects for 21 articles to see how traffic and SEO are affected by moving some of the most popular forum posts to /news.

Its my understanding that we will need to manage this redirection on the forum itself as well like we do for Ghost, because of its internal routing.

Can we revisit these?

@raisedadead, so sorry, I missed this notification. Yes, let's close this and try to handle it on the forum itself.

I think the point was missed, we still need to handle the redirects on the NGINX as the primary way to intercept these routes and add client side rounting on the forum as a fallback for the internal navigation.

authors: raisedadead scissorsneedfoodtoo

repo: freeCodeCamp/nginx-config


text:
Add support for mill

This is patterned after @grebe's work on dsptools. It's not completely baked - the current release of mill is missing some features:

doesn't handle symbolic links
doesn't restrict source files for Java/ScalaDoc.

There is a fix in the master branch for the former, and I'm submitted a PR for the latter.
In the meantime, I've set make to fetch the mill executable from my fork of the repo.
This shouldn't interfere with sbt usage - I've added a Module definition that reflects our sbt layout.
The new make targets are:

mill.build - compile and package the firrtl.jar for the default (first) crossVersion,
mill.build.all - compile and package firrtl.jar for all crossVersions,
mill.test - compile and test the default (first) crossVersion,
mill.test.all - compile and test all crossVersions,
mill.publishLocal - compile and publishLocal the firrtl.jar for the default (first) crossVersion,
mill.publishLocal.all - compile and publishLocal the firrtl.jar all crossVersions,

These targets include dependencies on mill and antlr-4.7.1-complete.jar and they will be downloaded and installed in the $HOME/bin and $HOME/lib directories respectively (once). If $HOME/bin isn't in your PATH, you'll need to add it.
Check out mill (or my fork) for documentation and examples.

Thank you for doing this!
I'm in the process of fixing the Travis regressions, once https://github.com/freechipsproject/firrtl/pull/812 is merged I'll merge master and I'll stop killing your builds :)
I think we should probably include this in the Travis regressions as well?

@ucbjrl is this still in play?

Very much so. I'm going to bring dsptools back into the fold (at which point all of the BIG6 will support building with mill). @jackkoenig  is working on rocketchip support and the ability to switch between source and library dependencies.

@ucbjrl Has this been superseded by other mill work? If not can conflicts be resolved and I will review.

@ucbjrl any update on the status of this or the fixing of conflicts

I've updated this branch. We were waiting on two items:

@jackkoenig's work with rocket-chip,
switching to the official mill release

before merging this.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.6 out of 16 committers have signed the CLA.:white_check_mark: seldridge:white_check_mark: jackkoenig:white_check_mark: albertchen-sifive:white_check_mark: aswaterman:white_check_mark: colin4124:white_check_mark: albert-magyar:x: ucbjrl:x: codelec:x: debs-sifive:x: chick:x: alonamid:x: jwright6323:x: nhynes:x: azidar:x: johnsbrew:x: felixonmarsYou have signed the CLA already but the status is still pending? Let us recheck it.

This branch currently has a bunch of commits that are already on master, it seems that the history has diverged in a serious way. Given the CLA comment above, we should probably rebase this branch to remove those unnecessary commits.

I've been using the prototype-mill branch and have some updates there as well. Maybe we can just combine all the changes into one big squashed commit and open it as a new PR to avoid this history divergence?

Closing in favor of #1083 given the serious divergence in this branch.

authors: CLAassistant chick edwardcwang jackkoenig ucbjrl

repo: freechipsproject/firrtl


text:
Update README to have known issues

Fix #6 - other sections of the README could use fleshing out, particularly the usage (once #9 is resolved).

Err looks like there's a ton of other stuff on this branch - I'll just add straight to master.

repo: freedomjs/freedom-social-wechat


text:
Epoll模式错误

main.xml中改成Epoll模式，执行PurenessScopeServer。提示如下:
[ 10:16:18.672993|140657353844512][CServerManager::Init()]... i=[0].
ACE_Dev_Poll_Reactor::open failed inside ACE_Dev_Poll_Reactor::CTOR: No such file or directory
[ 10:16:18.673788|140657353844512][CAceReactorManager::AddNewReactor]New [0] ReactorTxype = [4] nThreadCount = [1]. pAceReactor=[0x789030]
[ 10:16:18.673824|140657353844512][CServerManager::Init]AddNewReactor REACTOR_CLIENTDEFINE = Reactor_DEV_POLL.
[ 10:16:18.673850|140657353844512][CServerManager::Init()]... i=[1].
ACE_Dev_Poll_Reactor::open failed inside ACE_Dev_Poll_Reactor::CTOR: No such file or directory
[ 10:16:18.674584|140657353844512][CAceReactorManager::AddNewReactor]New [1] ReactorTxype = [4] nThreadCount = [1]. pAceReactor=[0x789560]
[ 10:16:18.674618|140657353844512][CServerManager::Init]AddNewReactor REACTOR_CLIENTDEFINE = Reactor_DEV_POLL.
[ 10:16:18.674643|140657353844512][CServerManager::Init()]... i=[2].
ACE_Dev_Poll_Reactor::open failed inside ACE_Dev_Poll_Reactor::CTOR: No such file or directory
[ 10:16:18.675030|140657353844512][CAceReactorManager::AddNewReactor]New [2] ReactorTxype = [4] nThreadCount = [1]. pAceReactor=[0x789b00]
[ 10:16:18.675060|140657353844512][CServerManager::Init]AddNewReactor REACTOR_CLIENTDEFINE = Reactor_DEV_POLL.
[ 10:16:18.675084|140657353844512][CServerManager::Init()]... i=[3].
ACE_Dev_Poll_Reactor::open failed inside ACE_Dev_Poll_Reactor::CTOR: No such file or directory
[ 10:16:18.675470|140657353844512][CAceReactorManager::AddNewReactor]New [3] ReactorTxype = [4] nThreadCount = [1]. pAceReactor=[0x78a090]
[ 10:16:18.675500|140657353844512][CServerManager::Init]AddNewReactor REACTOR_CLIENTDEFINE = Reactor_DEV_POLL.
[ 10:16:18.675523|140657353844512][CServerManager::Init()]... i=[4].
ACE_Dev_Poll_Reactor::open failed inside ACE_Dev_Poll_Reactor::CTOR: No such file or directory
[ 10:16:18.675899|140657353844512][CAceReactorManager::AddNewReactor]New [4] ReactorTxype = [4] nThreadCount = [1]. pAceReactor=[0x78a620]
[ 10:16:18.675928|140657353844512][CServerManager::Init]AddNewReactor REACTOR_CLIENTDEFINE = Reactor_DEV_POLL.
[ 10:16:18.675952|140657353844512][CServerManager::Init()]... i=[5].
ACE_Dev_Poll_Reactor::open failed inside ACE_Dev_Poll_Reactor::CTOR: No such file or directory
[ 10:16:18.676326|140657353844512][CAceReactorManager::AddNewReactor]New [5] ReactorTxype = [4] nThreadCount = [1]. pAceReactor=[0x78abb0]
[ 10:16:18.676356|140657353844512][CServerManager::Init]AddNewReactor REACTOR_CLIENTDEFINE = Reactor_DEV_POLL.
......................
[ 10:16:19.988129|140657353844512][CServerManager::Start]Begin.
Segmentation fault (core dumped)
我用的是ACE6.3.0版本，按照说明文档已经把/etc/security/limits.conf文件各项设置6000，并已经重启系统生效。用的TCPtest例子测试的。
用select，poll模式都测试了，正常运行，只有epoll出现问题。
请帮忙看下这是什么问题导致的，谢谢

我找到原因了。 main.xml中  MaxHandlerCount="10000"，而我系统设置的最大是6000，超过了系统的上限值。把系统的上限值改大或者改小MaxHandlerCount，就不会出错误了。

repo: freeeyes/PSS


text:
feat(nd): support returning per_curiam key

This is just a small change to collect "per_curiam" now that we can ingest it in Courtlistener.
Changes the scraper and the test files

also we can use the same trick you suggested before if we change the regex to
regex = r"(?P<volume>20\d{2})\s(?P<reporter>ND)\s(?P<page>\d+)"

This is good but there a few problems.

we can improve the citation extraction which I pointed out above.
There are a tremendous number of duplicates on their page.  I count roughly 25/100.  They appear to list the same case every time even when consolidated.

one 4 times - another 16.  The only way I see being able to do this is to extract the title from the document.  or ... use the first one you get and then extract if it you can.  Its actually relatively easy to get most of hte names but when they issue corrections the PDFs are total junk

I will take advantage of this more complete refactor to implement a backscraper and address a gap #1098

authors: flooie grossir

repo: freelawproject/juriscraper


text:
Missing token icons for Mozilla, ProtonMail, Epic Games

Mozilla icon is for Bugzilla token
Mail or email icon or ProtonMail icon (nothing showed in search
Epic Games Store icon

Missing icons will need to be added into FontAwesome before they can be made available for use in FreeOTP
https://fontawesome.com/support#fa-requests

authors: Githubuxer justin-stephenson

repo: freeotp/freeotp-ios


text:
SRX-GUPKQB - argocd baseurl argument

clean up unused Argocd arguments
add Argocd baseurl argument
create getConfig rpc

@medo-freiheit please remove installing the ArgoCD client in Dockerfile

@medo-freiheit please remove installing the ArgoCD client in Dockerfile

@mouhsen-ibrahim @hannesg  I think we still need it for syncing, right? here

@medo-freiheit please remove installing the ArgoCD client in Dockerfile

@mouhsen-ibrahim @hannesg I think we still need it for syncing, right? here

We don't use the sync endpoint in any team, please remove it. @medo-freiheit

repo: freiheit-com/kuberpult


text:
Avvisbar melding

📥 Proposed changes
Legg til mulighet for å lukke/avvise meldinger, på samme måte som i AlertMessage
☑ Submission checklist

[x] I have read the CONTRIBUTING doc
[x] yarn build works locally with my changes
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation (if appropriate)

💬 Further comments
Legger også til visuell regresjonstesting for komponenten

Det begynner å bli veldig mye delt kode mellom MessageBox og AlertMessage nå. Kan det være en idé å samle kodebasene? Man kan fortsatt eksponere det som ulike komponenter gjennom en faktory f.eks.

repo: fremtind/jokul


text:
Finding CNVs that do not intersect/merge with reference set

Hi
I've been using SURVIVOR to merge sets of SVs (i.e. compute the intersection).
Now I have a slightly different problem. Given 2 sets of SVs, say set A (study) and set B (reference), I would like to compute the set C=A-B. That is I'd like to find all SVs that are novel in A.
Can Survivor be used for this purpose? At the moment, my (imperfect) strategy is as follows

Survivor(A,B) -> merged set (ie A-intersect-B)
bedtools intersect -v -a A -b (A-intersect-B)

Which is of course sub-par for several reasons (bedtools doesn't know if an interval is a deletion or insertion, for starters)

Sure. What I usually do is to merge as normal.
then look at the SUPP_VEC= . Identify the pattern that matches your query. E.g. first sample = A, second sample =B you want to grep 'SUPP_VEC=10;' or something similar.
Cheers
Fritz

Thanks Fritz! Worked fine when filtering for ''SUPP_VEC=01'.

authors: fritzsedlazeck ggstatgen

repo: fritzsedlazeck/SURVIVOR


text:
Support Laravel Octane

This PR aims to support Laravel Octane by avoiding the injection of the IoC container.
The container instance injected in the class HandleCors is not removed in this PR to avoid potential breaking changes, however the injected instance won't be called in HandleCors.
Facades are used instead as they always resolve up-to-date instances out of the latest version of the container.

Closing this as CORS is now included using https://github.com/fruitcake/php-cors and the Laravel core middleware

repo: fruitcake/laravel-cors


text:
Fix API documentation

Best PR ever…

For things like this: just do the merge already ;)

Ok :-).

authors: Hywan evert

repo: fruux/sabre-dav


text:
Update csp tests

https://sourceclear.atlassian.net/browse/FLAT-400
https://sourceclear.atlassian.net/browse/FLAT-398

:ok:

authors: AlekseyYeremeyev RamilGilmanov

repo: fs/security-headers


text:
Remove .NET Core version of Project Cracker

The ProjectCracker.dll and ProjectCrackerTool.exe currently have .NET Core versions.
These are for supporting "old" F# project formats, pre .NET SDK 2.0.  I don't see the ned to continue to maintain and deliver .NET Core versions of these, since .NET Core tooling normally only understands "new" F# project file formats for .NET SDK 2.0
@ncave @Krzysztof-Cieslak @rneatherway @matthid Can you chime in with your thoughts on this?  thanks

@dsyme +1 for using the new project file format.
On a related note, are upstream changes planned to come into FCS? Is there a plan to fold FCS back into upstream? I know, so many questions...

On a related note, are upstream changes planned to come into FCS? Is there a plan to fold FCS back into upstream? I know, so many questions...

I'm trying to work this out now.
We integrated very large chunks of fcs back to Microsoft\visualfsharp, all the core code and enough to enable a CI build of the .NET Framework version of FSharp.Compiler.Service.dll.  That's good. In https://github.com/Microsoft/visualfsharp/pull/3480  I've done the same for the .NET Framework versons of the other tools (ProjectCracker, FSharp.Compiler.Service.MSBuild.v12.0.dll). However the  .NET Core/Standard build of FSharp.Compiler.Service.dll was not included in that, nor were the docs, nor .NET Core testing and so on.
The .NET Core/Standard build of FSharp.Compiler.Service is a bit of a pain to build, because of the manual codegen done in build.fsx etc.  I don't yet fully understand why all that is needed but there must be rationales for it.
Anyway, even after https://github.com/Microsoft/visualfsharp/pull/3480  this has left us in a messy situation where we have one build of FSharp.Compiler.Service.dll that uses on build system (the one from Microsoft/visualfsharp), and one here that uses another (Paket, FAKE etc.).  I really want to just have one build system with ultra-simple integrations.

The ProjectCracker.dll and ProjectCrackerTool.exe currently have .NET Core versions.
These are for supporting "old" F# project formats, pre .NET SDK 2.0

the .net core version doesnt make sense, should be removed.
Is useless to parse old fsproj
Projectcracker use msbuild autiomation, so as library.
The old fsproj/csproj expect full .net msbuild, the .net core version of msbuild is more restricted.
An example: some target file can embeed c# code, this is not supported by .net core msbuild, so will not work correctly
doesnt work with new sdk fsproj
why new project can be msbuild 15 without sdk style, these are .net full, so anyway, useless

authors: dsyme enricosada ncave


text:
Revert PR #518 to re-enable test cases

PR #518 disabled some test cases due to an issue with Mono 4.2.x. The bug was fixed later in the 4.2.x builds. The current stable mono build is now 4.8.x, and anyone who's working with this repo (and running the test cases) should be running at least 4.6.x, so let's re-enable these test cases for parity with the Microsoft/visualfsharp repo.

I restarted the failed job (I think the failure was unrelated)

The macOS job with mono-latest failed again. Both times it was due to an error in travis-ci's internal scripts, where the script wasn't able to fetch the mono package.

@jack-pappas Thanks for this change

authors: dsyme jack-pappas

repo: fsharp/fsharp


text:
Library refs break when project name contains a dot

In latest Fable (0.4.4), I'm experiencing a problem when building a project that references another project. If the project being referenced has a dot in the name, then Fable is generating incorrect paths in the require calls. Before upgrading Fable, I did not have the issue.
Imagine you have a solution with three F# projects:Library, Library.Another, and Main. Main references both Library and Library.Another. Now assume the two library projects have already been compiled by Fable to out/library and out/library-another.
Now, if you compile the Main project using this fableconfig.json...
{
  "module": "commonjs",
  "outDir": "out/main",
  "projFile": "Test/Main/Main.fsproj",
  "refs": {
    "Library": "../library",
    "Library.Another": "../library-another"
  }
}

...you will get incorrect require calls in the generated output whenever a module from the Library.Another project is referenced (modules referenced from Library work fine). For example:
var _SomeModule = require("../library/SomeModule");

var _AnotherModule = require("../library-another/Library.Another/AnotherModule");

The first require call works as expected. However, the second require call has Library.Another/ in its path for some reason, which causes a runtime error when trying to resolve the reference.
A similar problem occurs when using the AMD module system.
Note: In my case, the references from Main to the libraries are project references made within the same solution (as opposed to an external DLL) -- not sure if that makes a difference or not.

Hmm, I'm not sure the dot is the problem because we have a similar reference in the project containing the tests. Is your project public or can you upload a minimal project structure which can reproduce the problem? Is by any chance your directory structure similar to the one below?
/Main/Main.fsproj
/Library/Library.fsproj
         SomeModule.fs
/Library.Another.fsproj
/Library.Another/AnotherModule.fs

Thanks for the report!

My project isn't public, but I created a quick repo with similar structure to show the problem I'm having:
https://github.com/funlambda/fable-project-ref-example
If you clone it and run build-and-run.sh, you should see the module ref run-time error. The problematic path is in out/main/Program.js:
var _AnotherModule = require("../library-another/Library.Another/AnotherModule");

Thanks for taking the time to prepare the structure to reproduce the issue! It was indeed the dot :) I've released fable-compiler@0.4.6 with the fix. Could you give it a try?

Works perfectly now -- thanks for the fix and the insanely fast turnaround!

authors: alfonsogarciacaro funlambda

repo: fsprojects/Fable


text:
paket install fails on extraction if path is longer than 260 characters

I'm having an issue with paket when I execute paket in a really long directory name. When I try to run paket update or paket install in my directory with the long name (near or over 260 characters), I get this error message:
Paket failed with:
        Error during extraction of C:\j\workspace\really-long-path-name\another-really-long-directory-name\third-really-name-path-name\last-really-really-long-name\packages\nuget-bot.Microsoft.VisualStudio.Services.Common\nuget-bot.Microsoft.VisualStudio.Services.Common.12.0.21005.1.nupkg.
Message: The specified path, file name, or both are too long. The fully qualified file name must be less than 260 characters, and the directory name must be less than 248 characters. In rare cases a firewall might have blocked the download. Please look into the file and see if it contains text with further information.

My paket.dependencies has a source line plus this package in it:
nuget nuget-bot.Microsoft.VisualStudio.Services.Common

Microsoft has this information about prefixing paths with "\?" to resolve this: https://msdn.microsoft.com/en-us/library/aa365247(VS.85).aspx#maxpath. I'm not sure if this can be applied to paket.
Is this an issue with paket that can be fixed or do I need to make sure our path names in our projects are always shorter than this limit?
Edit:
There was a ticket with using directories with more than 260 character but I think the issue is different: https://github.com/fsprojects/Paket/issues/949

Currently this is not supported. We need to use a different api for this,
but I'm not sure when we have it.
On Dec 27, 2015 07:51, "jschroed" notifications@github.com wrote:

I'm having an issue with paket when I execute paket in a really long
directory name When I try to run paket update or paket install in my
directory with the long name (near or over 260 characters), I get this
error message:
Paket failed with:
Error during extraction of C:\j\workspace\really-long-path-name\another-really-long-directory-name\third-really-name-path-name\last-really-really-long-name\packages\nuget-botMicrosoftVisualStudioServicesCommon\nuget-botMicrosoftVisualStudioServicesCommon120210051nupkg
Message: The specified path, file name, or both are too long The fully qualified file name must be less than 260 characters, and the directory name must be less than 248 characters In rare cases a firewall might have blocked the download Please look into the file and see if it contains text with further information
My paketdependencies has a source line plus this package in it:
nuget nuget-botMicrosoftVisualStudioServicesCommon
Microsoft has this information about prefixing paths with "?" to resolve
this:
https://msdnmicrosoftcom/en-us/library/aa365247(VS85)aspx#maxpath I'm
not sure if this can be applied to paket
Is this an issue with paket that can be fixed or do I need to make sure
our path names in our projects are always shorter than this limit?
—
Reply to this email directly or view it on GitHub
https://github.com/fsprojects/Paket/issues/1331.

Isn't it basically this one?

authors: eggapauli forki jschroed

repo: fsprojects/Paket


text:
Upgrade tokio to 1.0 and futures to 0.3.9

Fixes: #4
Signed-off-by: Tim Zhang tim@hyper.sh

@dpeckett Please have a look for this pr

@dpeckett Please have a look for this pr

@shanecurran Can you please merge it and cargo publish it? thank you very much.

@shanecurran Can you please merge it and cargo publish it? thank you very much.

@Tim-Zhang Unfortunately I don't have write permissions on the repo, but your changes look good to me. Hopefully @mxpv can approve and merge shortly!

@Tim-Zhang Unfortunately I don't have write permissions on the repo, but your changes look good to me. Hopefully @mxpv can approve and merge shortly!

@dpeckett PTAL

@dpeckett PTAL

Just wanted to bump this @mxpv @dpeckett
Thank you!

Just wanted to bump this @mxpv @dpeckett
Thank you!

@shanecurran @dpeckett has transferred this repo to the new org rust-vsock. Welcome to join the org, I have invited you.
I plan to merge this and publish 0.3.0 next week

@shanecurran @dpeckett has transferred this repo to the new org rust-vsock. Welcome to join the org, I have invited you.
I plan to merge this and publish 0.3.0 next week

It's ready to merge now, I will merge it tomorrow and release the 0.3.0

authors: Tim-Zhang mxpv shanecurran

repo: fsyncd/tokio-vsock


text:
CLI / wasm

How do I compile to wasm?  Does cyber CLI have menu? Tried -h, --help, didn't work. The CLI section in doc is very lacking.
thanks

It's cyber help, but I suppose -h and --help should be aliases too. I will also update the CLI docs.
For building wasm, see the build.md. That is for standalone wasm which won't have any interop with web. zig build web-lib will produce the wasm with interop like the playground. In the future it will contain JS promise to Cyber futures and maybe DOM access, so I'd check back in a week or so.

I have added CLI instructions to the docs. Closing, thanks.

authors: fubark mationai

repo: fubark/cyber


text:
纯粹研究技术，想知道在树莓派上的详细使用方法！

RT,我下载了树莓派的版本，但是无法安装，想请教各位大大，如何在树莓派上使用这款软件？

How to install this software on Raspberry?

Please how to install this software on Raspberry Pi (version:raspberry pi 3b)?

哈哈哈哈哈哈，哥们儿你太逗了，发英文属实没必要啊，你觉得能是老外开发的这东西吗？哪得是一种什么精神啊？

哈哈哈哈哈哈，哥们儿你太逗了，发英文属实没必要啊，你觉得能是老外开发的这东西吗？哪得是一种什么精神啊？

大公无私的国际共产主义精神，类似前辈白求恩。
[doge]

哈哈哈哈哈哈，哥们儿你太逗了，发英文属实没必要啊，你觉得能是老外开发的这东西吗？哪得是一种什么精神啊？

大公无私的国际共产主义精神，类似前辈白求恩。
[doge]

authors: SamirDuran hupo376787 wang-bailin

repo: fuck-xuexiqiangguo/Fuck-XueXiQiangGuo


text:
problem about inference code

Hi, I used my own  .jpg as the reference image and encounted the following bug, what should I do?

this is the reference .png picture

besides, I used another picture but do not encountered the bug, I do not know what happended? More than that, I found out that the result vedio is not good enough, how can I improve the quality of the generated vedio?
https://github.com/user-attachments/assets/5014cf7f-29ad-4123-af3f-e02c1987f8bc

After reading more papers, I think I now understand why this happens. It's because your work lacks a special module to enhance identity alignment. Anyway, thank you all for your open work!

The bug persists. However, after reading additional papers, I believe I now understand the cause of the poor face generation. It seems your work lacks a specific module to enhance identity alignment. Regardless, I appreciate your openness and the valuable work you've shared!

repo: fudan-generative-vision/champ


text:
Add StyleGuide.md

Changes
Add StyleGuide.md as the reference for Japanese localization
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[ ] Includes tests (if functionality changed/added)
[x] Includes docs (if user facing)
[x] Commit messages follow commit message best practices

See the contribution guide
for more details.

書かれている内容については必要十分だと思います。

記号類は全角で表記ということですので、現在の messages_ja.jsonについて、GUIに表示される部分の記号が全角になっていることを確認する必要があります(別途指摘しています)。

体裁について指摘を数点コメントしました。

成果物と直接関係ないですが、、

http://10.125.71.194:3000/issues/9232 の説明欄に、内部レビュー用PRのURLを記載するところがあります。本PRのURLを貼っておいていただけますか。
9232-wkと9232-prのブランチが古いかもしれません(最新コミットではmessages_ja.jsonにparamaterが消えているはず)。もしかしたらコミュニティ版にPR発行するときにコンフリクトが起きるかもしれません。起きそうであれば対処しましょうか(別途相談)。

authors: Martsings kobaji

repo: fujitsu-cdf/dashboard


text:
プライバシー保護3次元視線ターゲット検出のための、上半身姿勢に基づく視線推定

タイトル: プライバシー保護3次元視線ターゲット検出のための、上半身姿勢に基づく視線推定
リンク: https://arxiv.org/abs/2409.17886
概要:
視線検出、つまり外部視点から人がシーン内のどこを見ているかを判断することは、特に3D空間においては困難な課題です。既存の手法は、主に顔に焦点を当てて人の外観を分析することに大きく依存しており、視線目標を予測しています。本論文では、人の上半身の姿勢と利用可能な深度マップを利用して3D視線方向を抽出し、多段階またはエンドツーエンドのパイプラインを用いて視線目標を予測するという、この問題に取り組むための新しいアプローチを提案します。正確に予測されれば、人体の姿勢は頭部の姿勢に関する貴重な情報を提供することができます。これは視線方向の良い近似値であり、また、人が行っている活動や注目している可能性のある物体と関連付けられる腕や手の位置に関する情報も提供します。その結果、3Dでの視線推定に加えて、視線目標検出も同時に行うことができます。私たちは、人の顔の画像を必要とせずに、最も包括的な公開されている3D視線目標検出データセットにおいて最先端の結果を示しており、様々なアプリケーションコンテキストにおけるプライバシー保護を促進します。コードはhttps://github.com/intelligolabs/privacy-gtd-3Dで公開されています。

論文要約
論文要約:

従来の視線検出技術は、プライバシー上の懸念がある顔画像に依存していた。
本論文は、顔画像を使わずに上半身の姿勢情報から視線を推定する新しい手法を提案。

人間の姿勢は、頭部の向きだけでなく、腕や手の位置から対象物に関する情報も提供してくれる。
これにより、3D空間での視線方向の推定だけでなく、視線の対象物も特定できる。

提案手法は、公開されているデータセットを用いた実験で、従来手法に匹敵する精度を達成。
顔画像を必要としないため、プライバシー保護の観点からも優れている。

→ つまり、この論文は、プライバシーに配慮した新しい視線推定技術を提案し、その有効性を示したと言える。

@yukihiko-fuyuki が以下のラベルを提案し、適用しました：

eye-tracking
privacy-enhancement
3D-scene-representation

repo: fulfulggg/Information-gathering


text:
🛑 Waddle Penguins Island - Game Server is down

In 9f069b1, Waddle Penguins Island - Game Server ($GAMESERVER_URL) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Waddle Penguins Island - Game Server is back up in 09f2044 after 13 minutes.

repo: fullmoondev/status


text:
Custom Favicon

Hey guys,
Awesome repo!
Question: Would it be possible to include a flag to provide a custom favicon? :) Or is there another way to use a custom one?
Thanks!

👍 yes please, good idea !

At this time, there is no option to use a custom favicon. But I'm going to have a look at this possible feature and come back with either an implementation or an answer

@Steerlin @duckchip this feature has landed in version 5.4.0. Please, be aware that a custom favicon can be applied only in build mode. See readme

authors: Ge11ert Steerlin duckchip

repo: funbox/blueprinter


text:
.tj domains

Problem
There's no whois server registered to the IANA Route Database Zone for all .tj domains.
Conclusion
That makes all .tj domains fails.

Please report to http://www.nic.tj/whois.html for whois lookup

repo: funilrys/funceble


text:
Filename of failing validation

Consider a schema that has a boolean for draft:
class Page(Templated, Content):
    in_menu: bool = True
    call_to_action_text: str

Then this file:
title: Home
in_menu: False
---

We give high-performing companies the tools they need to compete in the digital age, from skills development and automation to communications and strategy expertise.

Will give the following traceback
Traceback (most recent call last):
  File "/home/janis/digital_site/venv/bin/gilbert", line 11, in <module>
    load_entry_point('gilbert==0.3.4', 'console_scripts', 'gilbert')()
  File "/home/janis/digital_site/venv/lib/python3.7/site-packages/gilbert/cli.py", line 94, in main
    args.func(args, site)
  File "/home/janis/digital_site/venv/lib/python3.7/site-packages/gilbert/cli.py", line 39, in handle_render
    site.render()
  File "/home/janis/digital_site/venv/lib/python3.7/site-packages/gilbert/site.py", line 22, in _inner
    return method(self, *args, **kwargs)
  File "/home/janis/digital_site/venv/lib/python3.7/site-packages/gilbert/site.py", line 184, in render
    self.load_pages()
  File "/home/janis/digital_site/venv/lib/python3.7/site-packages/gilbert/site.py", line 22, in _inner
    return method(self, *args, **kwargs)
  File "/home/janis/digital_site/venv/lib/python3.7/site-packages/gilbert/site.py", line 198, in load_pages
    self.pages.load(self.pages_dir)
  File "/home/janis/digital_site/venv/lib/python3.7/site-packages/gilbert/collection.py", line 53, in load
    self._items[name] = self.load_file(item, name=name)
  File "/home/janis/digital_site/venv/lib/python3.7/site-packages/gilbert/collection.py", line 67, in load_file
    obj = self.default_type.create(name, self.site, data=data, meta=meta)
  File "/home/janis/digital_site/venv/lib/python3.7/site-packages/gilbert/content.py", line 52, in create
    return klass(name, site, data=data, meta=meta)
  File "/home/janis/digital_site/venv/lib/python3.7/site-packages/gilbert/content.py", line 27, in __init__
    super().__init__(**meta)
  File "/home/janis/digital_site/venv/lib/python3.7/site-packages/gilbert/schema.py", line 160, in __init__
    setattr(self, name, value)
  File "/home/janis/digital_site/venv/lib/python3.7/site-packages/gilbert/schema.py", line 133, in __set__
    raise ValueError(f'{instance.__class__.__name__}.{self.name} does not accept value {value !r}')
ValueError: Page.in_menu does not accept value ['False']

I'm thinking that it may be helpful here to report on the filename that contained the non-validating metadata.

Also here's the in_menu.yaml file:
content_type: Collection
filter_by:
  all:
    - attr: ['in_menu']
---

I've just tried to replicate this locally, and did not trigger an error.

created new project
put your Page class into plugins.py
created pages/fail.yml
ran "gilbert render"

$ gilbert render
Searching /.../gilbert/plugins for plugins...
Loaded plugin: markdown
Loaded plugin: collection
Loaded plugin: yaml
Loaded plugin: scss
Loaded plugin: asset_bundle
Loaded local plugins.
Found 0 content objects.
Found 1 pages.
Rendering fail.yml ...
Template for default.html not found: ['default.html']

Which version of Gilbert are you using?
[also, overridding Page wasn't something I had thought of - neat!]

So it turns out that the issue here was that the loader was passing the field in_menu: False over unprocessed as {in_menu: 'False'} and that was causing the schema to fail validation since it was expecting a boolean but getting a string.
I fixed this over on the loader in https://github.com/shuttle1987/gilbert-frontmatter-markdown/pull/2
Given the source of this appeared to be an issue with my loader this issue may be invalid, unless there's some error reporting you'd like to add in here?
I'm not exactly sure how I should be making a test case for this though... (and I'd like to make a couple of tests)

I think this issue came up with this exact commit version: https://github.com/funkybob/gilbert/pull/29/commits/57614be95207cab7495e93263cd63ca381ee4a21
I had to work off that commit since it lets me use the alternative loader.

Can we consider this issue resolved now?

I believe this is not a bug in Gilbert but rather was in my loader, I'll reopen if not

repo: funkybob/gilbert


text:
Two groups with same name, but different color

My use-case for TM is to have tabs for approx 60-70 different URLs to servers which all would look identical if not for TM. One very important dimension is production vs test. Naturally, I would use color (red/yellow) to separate the two environments - but the same names. That works fine for TM rules, but not for groups (understandable, since group concept is pretty new).
The issue is this:
Tab Rule 1: "S1" (red) - (URL fragment "prodserver.com")
Tab Rule 2: "S1" (yellow) - (URL fragment "testserver.com")
Group 1: "G" (red);
Group 2: "G" (yellow)
TabRule1 is placed in Group 1
TabRule2 is placed in Group 2
If you open the URL "prodserver.com" - Group 1 will be displayed. After that, you open "testserver.com" and both! tabs will be added to Group 2 (yellow). If you open "prodserver.com" again, all three tabs will be added to Group 1...and so on.

Thanks for your feedback. I've implemented a fix, which will only work if the colors for the same group name are different. This will resolve the issue in release 1.0.9.
To avoid mixing tabs between groups, ensure each group has a unique color. Even if groups share the same name, distinct colors will keep them separate. This way, your "prod" and "test" environments won't get confused in the tab management.

authors: mortensimon sebastienfontaine

repo: furybee/chrome-tab-modifier


text:
CSSModules issues

I have played around with the react-example and CSSModules and found some issues
I have added this alias.
alias: {
    styles: '~/styles',
    images: '~/images'
},

However, it seems alias is not working when in a css file for exampe:
@value blu from 'styles/colors.css';

Another thing is that when running in dev mode and i make any change it will result in
SyntaxError: Unexpected token o in JSON at position 1
    at Object.parse (native)

Seems it adds css to the main js file?

This only happens when sourceMaps: true
This is my playground project.
https://github.com/Fjandin/react-example
PS! I'm very new to fuse-box so might be because i'm doing something wrong. If that's the case... sorry :P

Hi :)
You almost got it right, CssModules needs to come after PostCss plugin

HI
I tried movin CssModules after PostCss, but I still am having the same issues. Both with the alias not working and sourcemaps corrupting the main js file.

@Fjandin aliases are related only to javascript code, it has nothing to do with CSS.
You need to utilize some PostCSS modules that handles the replacement.
We have it in Sass, for example, http://fuse-box.org/plugins/sass-plugin#macros but that's because Sass supports it out of the box

ah ok. Fair enough 👍
But about the other issue... any idea what could cause this?
This is the output i get from fusebox:
https://gist.github.com/Fjandin/9ba060988b5a21a22f99b5fd5b08fab3
basically as soon as i change any file i get a SyntaxError: Unexpected token o in JSON at position 1 error.

Well, it's kind of because your PostCSS spits and error that breaks fusebox, that shouldn't happen in fact, and will be fixed soon.

Error is only thrown if sourceMaps: true because JSON.parse({}) is called. The bug seems to be provided by the CSSPlugin that handles souceMaps: {} in the file configuration

It's weird that i can't find anything on this. Am i the only one using css modules together with postcss? :P + wants sourcemaps

This configuration works :
const {FuseBox, PostCSSPlugin, CSSPlugin} = require('fuse-box')
const fuse      = FuseBox.init({
	homeDir    : 'sources',
	output     : 'built/$name.js',
	log        : true,
	sourceMaps : true,
	plugins    : [
		[
			'style/**.css',

			PostCSSPlugin([require('autoprefixer')(), require('precss')()], {
				sourceMaps: true,
			}),
			CSSPlugin({ group: 'app.css', inject: (file) => `built/${file}`, outFile: 'built/app.css' })
		],

		CSSPlugin({ group: 'vendor.css', inject: true }),
	],
	useTypescriptCompiler : true
})

fuse.dev({ root: './', port: 8081, devServer: false })
fuse.bundle('vendor').instructions('~ index.ts').watch().hmr()
fuse.bundle('app').instructions('!> [index.ts]').watch().hmr()

fuse.run()

fusebox master ✗ 51e87b9 ✖ △ ➜ tree sources
sources
├── index.ts
└── style
    └── app.css

fusebox master ✗ 51e87b9 ✖ △ ➜ cat sources/index.ts
import './style/app.css'

fusebox master ✗ 51e87b9 ✖ △ ➜ cat sources/style/app.css
body {
	background: black;
}

I don't know why but it's very redundant to repeat the outDir configuration into the CSSPlugin:outFile and CssPlugin:inject object members in order to make HRM working...

authors: Fjandin cmizzi nchanged

repo: fuse-box/fuse-box


text:
Build API for fetching Raspiblitz log

https://github.com/cstenglein/raspiblitz-web/issues/49

Endpoint is in, but not tested yet on a real Raspiblitz. Access via http://127.0.0.1:8000/latest/system/get_debug_logs_raw. Make sure to pass a valid JWT token.

Sorry for the confusion, the next release will change the file home.admin/XXdebugLogs.sh to https://github.com/rootzoll/raspiblitz/blob/dev/home.admin/config.scripts/blitz.debug.sh


text:
Add endpoint to open a lightning channel

As a user I would like to open lightning channels to others.

@rootzoll important issue

will be fixed by PR #82

authors: cstenglein rootzoll


text:
Removed Filter div from fleet location list page Closes #53 .

Dipping my toes into the angular code with some low hanging fruit.

@BWortman requested changes checked in.


text:
[requires.io] dependency update on master branch

This change is

Current coverage is 100.00%

Merging #27 into master will not affect coverage as of 688d4c1

@@            master     #27   diff @@
======================================
  Files            7       7
  Stmts          263     263
  Branches        23      23
  Methods
======================================
  Hit            263     263
  Partial          0       0
  Missed           0       0

Review entire Coverage Diff as of 688d4c1

Powered by Codecov. Updated on successful CI builds.

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from the review on Reviewable.io

authors: codecov-io mithrandi

repo: fusionapp/fusion-index


text:
componentWillUnmount called without calling componentDidMount

Type of issue
Bug
Current behavior
componentWillUnmount is called but componentDidMount is not
Expected behavior
componentWillUnmount should only be called if componentDidMount is called on a component.
Steps to reproduce
Create a component that defines componentWillUnmount and componentDidMount.
Your environment

fusion-react version: 1.2.1 (it worked properly in 1.1.0)

And disabling SSR via https://github.com/fusionjs/fusion-react/issues/95#issuecomment-399980523 did not fix the issue

This should no longer be an issue due to https://github.com/fusionjs/fusion-react/pull/160 landing.  We should no longer be calling componentWillUnmount, please re-open or file a new issue with additional context if you're still seeing this.

authors: KevinGrandon supersonicclay

repo: fusionjs/fusion-react


text:
Add Initial implementation

Resolves #1

!merge


text:
🛑 fashionunited.nl is down

In 55cc840, fashionunited.nl (https://fashionunited.nl) was down:

HTTP code: 0
Response time: 0 ms

Resolved: fashionunited.nl is back up in 2e0c34f after 7 minutes.


text:
Permissions

please add perms!

Please tell me more specifically…

Yea i also noticed it, texter.command.txt should be set to op not true

If you only want to use op, you can configure it in Config

This issue closed because there is no reply from the proposer. Please reply if there is anything else

authors: Kauso8507 dearminder fuyutsuki

repo: fuyutsuki/Texter

license: NCSA


text:
gitchat - bugstack虫洞栈

https://fuzhengwei.github.io/gitchat.html
沉淀、分享、成长，专注于原创专题案例，以最易学习编程的方式分享知识，让自己和他人都能有所收获。目前已完成的专题有；Netty4.x实战专题案例、用Java实现JVM、基于JavaAgent的全链路监控、手写RPC框架、DDD专题案例等。

君子知至学之难易，而知其美恶,然后能博喻。能博喻，然后能为师。


text:
.nz domain updates

Add .nz 1st level suffix
Adds missing cri.nz 2nd level suffix
Fix typos in parliament.nz and "New Zealand"
Add link to .nz details

I see that CONTRIBUTING.md says

Avoid changing existing sets of data. Some developers use Faker with seeding for unit tests ; changing the data makes their tests fail.

Is the above OK by this restriction?

Well, since you fix a typo in the same time (parliment => parliament), yes ;)

Thanks!

authors: fzaninotto xurizaemon

repo: fzaninotto/Faker


text:
enhance WindowRenderCapturer

maybe enhance WindowRenderCapturer, s.t. it knows more info?
for example: "i_j" means ith plain-old frame, jth winodw.render submission

[ ] also need to support: match golden file on disk

firstly, enhance binding to expose frame number

Question: what query (expect call) will happen?

fully match - this is easy
only part of it

Question: which api is better - expectAndReset per frame? expect-all in the end?


text:
Request against X2 and retrieve graphs

The first step is to run:

node_match?id=Taro
edge_match?label=person

Against:

x2neo
x2ora

Done with b380c115485116e6721157ae96f60ffac3876ecc. Thank you!
The queries can be taken with:

?query=node_match?node_ids[]=Taro
?query=edge_match?edge_labels[]=person


text:
Documentation about toml script to create commands

We need documentation about how to use these toml script to create commands:

[x] : Explain what can be achieve with it. (command, startup, ...)
[x] : Document the current toml that exists
[ ] : Small internal documentation how it works

https://github.com/g8os/core0/tree/0.12.0/docs/config

@zaibon i don't get what kind of internal documentation you need. can u check the provided link and tell me if u think we can add more.

authors: muhamadazmy zaibon


text:
document rationale for sample id being associated with ReadGroup

See #652 https://github.com/ga4gh/schemas/pull/652#discussion_r70047549

After discussing this with you and @mbaudis I was able to quickly convince myself that multi-sample BAMs are a common use case. For example, GATK includes the argument sample_name.
--sample_name -sn	NA	Name of single sample to use from a multi-sample bam
Or in samtools the -I flag will Include only listed read group or sample name. To me, this fairly conclusively suggests that multisample BAM files are a common occurrence in practice.
A quick search for "how do I split my BAM into multiple samples" or "how do I combine my samples into one BAM" turns up a convincing subset of results.
I suppose one could construe an analysis for discovering differential expression for many samples contained in a single BAM file. Having the ability to have a Read Group Set host multiple samples recreates the same process in GA4GH.

In the world of cancer genomics, BAMs with multiple  samples split by read groups is exceedingly rare. While there is are probably good use cases, a good design will be explicit about this, rather then leave the reader guessing from a command line of an unrelated program.
This ticket doesn't suggest changing anything, only documenting the rationale for a uncommon use case to ensure it remains supported.

We can close this by pointing to the BAM documentation of the SM tag.

This ticket was intended to note something for inclusion in the
GA4GH API documentation, which was suppose to be independent of
existing file formats.  It was noted that it's a common assumption
that there is one sample per BAM, hence the need to clear explain
the conceptual model of GA4GH.
David Steinberg notifications@github.com writes:

We can close this by pointing to the BAM documentation of the SM tag.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.*

Generally:
// A Biosample refers to a unit of biological material from which the substrate
// molecules (e.g. genomic DNA, RNA, proteins) for molecular analyses (e.g.
// sequencing, array hybridisation, mass-spectrometry) are extracted. Examples
// would be a tissue biopsy, a single cell from a culture for single cell genome
// sequencing or a protein fraction from a gradient centrifugation.

So in my understanding (of my own writing, alas) a BAM containing reads from several original samples (or host + virus etc.) still references one biosample, which contains this mixed material.
What we need is a general provenance system for tracking biosamples ("derived from", "contains" ...) which then addresses this problem.
An alternative approach would be the [Biosample] => Experiment => ... approach, which however would not solve inherent mixes.

In BAM, the sample id is associated with the ReadGroup.  If the read group
has multiple samples, it should have a pool sample id.  There isn't
a global BAM sample id
See page 3 of https://samtools.github.io/hts-specs/SAMv1.pdf
SM is under a readgroup.  The rationale is not explain.
Michael Baudis notifications@github.com writes:

So in my understanding (of my own writing, alas) a BAM containing reads from
several original samples (or host + virus etc.) still references one biosample,
which contains this mixed material.

authors: david4096 diekhans mbaudis


text:
🛑 Revista is down

In 05dc596, Revista (https://pensamientopenal.com.ar/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Revista is back up in dc77bd9.


text:
🛑 Revista is down

In f7df389, Revista (https://pensamientopenal.com.ar/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Revista is back up in 3905580.


text:
Update package.json to be used with our npm account

We plan on having this package on our npm account temporarily. Hopefully we'll get our change approved on the original project, and at that time we can remove our own version and this whole repo.

Sorry, I didn't mean to open this up here.

repo: gabidavila/react-usa-map


text:
🛑 Trends is down

In 8d77f26, Trends (https://trends.gab.com) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Trends is back up in 43d6542 after 53 minutes.

repo: gadmln/gabstatus


text:
Replace lodash's mapValues

From #112's discussion, this replaces lodash's mapValues to reduce the number of lodash dependencies.

Great job, thanks!

authors: frankychung gaearon

repo: gaearon/redux


text:
sprint 4 proposal

Melody Mates
Our project aims to foster connections in the classroom based on common tastes in music. We hope to create an interactive, user-friendly program in which students can share their top five songs and be matched with their peers based on their song choices.
Repositories:

https://github.com/gaheerab/sprint4_backend.git
https://github.com/gaheerab/sprint4_frontend.git

Features

Login Page: Prompts users to login before they can access their friend page
Friend Page: Where users can see their matches and current friends, as well as accept matches.
Chat Page: Allows users to post comments on their public profile about their music choices to share with their friends
Music Choice Page: Users update their top 5 song list, which is used to identify others with corresponding song choices.

Trio 1

Scrum Master: Gaheera- APIs and Database
Developer 1: Rhea- Chatroom
Developer 2: Hannah- Music Choice Page

Trio 2

Assistant Scrum Master: Brandon- APIs and Database
Developer 1: Rowan- Login and Friend Page
Developer 2: Carson- Login and Friend page

I like that you considered different pages, that is the highlight of your issue.   I think you are missing song page, where people communicate about a song.  You may consider genre and perhaps top 20 songs in a genre to make matching more likely.  You can probably get top 20 songs from an API.   You need to consider managing the titles as in central place as you can't have that typed by an individual user as they will never match.
.88


text:
Git clone failed

http://pastebin.com/gqYf04ju
Cloning into 'LuaUI/images/scenedit/resources'...
fatal: reference is not a tree: 56fb60d58943c1445cc52c83d6a950d62b5be3c3
Unable to checkout '56fb60d58943c1445cc52c83d6a950d62b5be3c3' in submodule path 'LuaUI/images/scenedit/resources'

Whoops. Forgot to push that commit...
Truthfully I think you'll run into a number of other issues before I release the next version (https://github.com/gajop/Scened-Core/milestones/0.5). You can still try it out and if you can give me any opinions or suggestions I might incorporate them, but I think it's fair to warn you of the instability of current git version.
Fixed with https://github.com/gajop/Scenario-Editor-Resources/commit/56fb60d58943c1445cc52c83d6a950d62b5be3c3

authors: gajop sprunk

repo: gajop/Scened-Core


text:
Remove/Add random cards in throw-confidence example

Hi my friend. Very nice job! :)
I try remove cards in throw-confidence example, without use angular, but not have sucess.
Has one method like "stack.remove(cardElement)" ?
Don't have too much experiente in Angular, i'm a newbie in javascript...rsrs
I'm Brazilian.  Sorry for my bad english language.

You have to call

You have to call $scope.$digest() after changing/removing the scope cards.

thank's my friend

authors: barryvdh cezarant

repo: gajus/angular-swing


text:
ci/test: improve CI jobs and tests

Motivation
Closes #1991
Closes #1978

Solution
TODO

[x] ethereum/feature tests

[x] Fix live/goerli tests
[x] foundry-rs/foundry#4431
[x] fix windows build

[x] docs

[x] #2187
[x] #2203

[x] examples

[x] #2207

Not planned for this PR (#2191):

Feature checks: another PR because it requires massive feature changes. see: https://github.com/gakonst/ethers-rs/actions/runs/4266867552/jobs/7427873846
WASM: same as feature checks. Doesn't work with --all-features because of tokio: https://github.com/gakonst/ethers-rs/actions/runs/4267048448/jobs/7428242514

PR Checklist

[x] Added Tests
[x] Added Documentation
[ ] Updated the changelog
[ ] Breaking changes

@gakonst ready for review :pray:
Remaining failing actions:

deny (licenses): will open an issue later / tomorrow
windows: I have no clue what is wrong... It fails to compile a random example every time, which I cannot reproduce on my own machine
examples: fixed by #2207

OK. send it

🫡

authors: DaniPopes gakonst

repo: gakonst/ethers-rs


text:
vm.expectRevert fail reason parsing custom errors as strings

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.1.0 (64f779a 2022-03-20T00:11:33.074266889+00:00)
What command(s) is the bug in?
forge test
Operating System
Linux
Describe the bug
Encountered this error while converting require string reverts into Solidity 0.8.4 custom errors:

It seems like forge is trying to parse the expected error (custom error's selector) in the first fail as a string and it ends up with those replacement characters.
Below is an example contract on how to reproduce this error. Using forge-std and ds-test just for the Vm interface and address.
// SPDX-License-Identifier: MIT
pragma solidity ^0.8.4;

import "ds-test/test.sol";
import "forge-std/Vm.sol";

contract Contract {
    error CustomError();

    function requireStringFail() public pure {
        require(false, "Require string");
    }

    function customErrorFail() public pure {
        revert CustomError();
    }
}

contract TestContract is DSTest {
    Vm internal vm = Vm(HEVM_ADDRESS);
    Contract internal extContract;

    function setUp() public {
        extContract = new Contract();
    }

    function testExpectRevert_customError() public {
        // works as intended
        vm.expectRevert(Contract.CustomError.selector);
        extContract.customErrorFail();

        // reverts with a string while expecting a custom error
        // Reason: Error != expected error: 'Require string' != '   ���'
        vm.expectRevert(Contract.CustomError.selector);
        extContract.requireStringFail();
    }

    // this is ok
    function testExpectRevert_string() public {
        // works as intended
        vm.expectRevert("Require string");
        extContract.requireStringFail();

        // reverts with a custom error while expecting a string
        // Reason: Error data != expected error data: 0x09caebf3 != 0x5265717569726520737472696e67
        vm.expectRevert("Require string");
        extContract.customErrorFail();
    }
}

I'll look into this after REVM is merged, the fix is probably not too hard. I'll keep you posted.

hey @DaniPopes you should be able to account for custom errors using something like:
bytes customError = abi.encodeWithSignature("CustomError()");
vm.expectRevert(customError);
extContract.customErrorFail();

We now fallback to hex if it's not a valid UTF8 string, so in your case it would look like 'Require string' != '0x09caebf3'

hey @DaniPopes you should be able to account for custom errors using something like:
bytes customError = abi.encodeWithSignature("CustomError()");
vm.expectRevert(customError);
extContract.customErrorFail();

leaving a note for posterity here, if you're already importing the contract w its custom errors might as well use the selector:
        VM.expectRevert(
            abi.encodeWithSelector(
                ImportedContract.CustomError.selector,
                arg1
            )
        );
        ImportedContract.customErrorFail();

authors: DaniPopes onbjerg wminshew zaqk

repo: gakonst/foundry


text:
Update deployments list for the multi-task setup

Accommodate the UI for changes in https://github.com/galaxyproject/cloudlaunch/pull/85.

I feel this is ready go be merged unless there are comments?

repo: galaxyproject/cloudlaunch-ui