add Franz

Closes https://github.com/caskroom/homebrew-cask/issues/19368.

To solve common stylistic issues (which this has), run brew cask style --fix {{your_cask_file}} prior to submission.

Thank you for the contribution. It was merged directly as a3650e7ac84472e8c88939df6d98d141ac4111ca to keep commit history cleaner. Your contribution is still credited to you.


fbcat: init at 0.5.1

Motivation for this change
Closes #106771
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

CC @SeraphyBR for testing

Build and works for me :smile: ,
Result of nixpkgs-review pr 107172 1
1 package built:
fbcat

print of my tty, just copied ./results/fbgrab/* to / tmp to run


Fix typos in README.md

comand ==> command
awsome ==> awesome
precendence ==> precedence
Description:
Fix typos in README.md
Checklist:

[x] Does your PR pass tests?
[x] Have you written new tests for your changes? Not Applicable
[x] Have you lint your code locally prior to submission?

😄 thank you!


Add specs

Add some basic tests for functionality.
Fixes #13.

One thing to note about this, npm test is gone, since any meaningful testing would need atom/apm anyway. As long as that is fine with you merge away :wink:.

Adding it back is a small PR away if this is an issue, merged in.

So now I just run 'apm test', if I'm correct?
Good work 👍

Yep, apm test will test the functionality of the plugin itself. (npm run lint is unchanged).


Full Reset on any state changes in React Codesandbox

Hey, so i have a react app inside that im calling this codesandbox, anytime any state changes, the code resets, I am not sure is this a bug, or by design is there a work around to it.
I understand re render when the state changes, but is there is no memory implementation or something not sure, can someone provide pointers if this is a bug or not.
Issue: Say i have wrote many lines in that editor and some state changes, the code resets to initial
Minimal Reproduction
function E2() {
    const [seeTarget, setSeeTarget] = useState(false);

    return (

                <Sandpack />

            {seeTarget ? (

                    <Canvas>
                        <ambientLight />
                        <pointLight position={[10, 10, 10]} />
                        <Cube />
                        <OrbitControls />
                    </Canvas>

            ) : null}

 setSeeTarget((x) => !x)}>
                    <FiTarget />

    );
}

export default E2;

ok i found the solution, just put the Sandpack  into a different file and call it, then this issue goes away, basically any usestate changing value in the same file that has this compnent causes the issue. Looking back, i think this might not have been a bug but a basic react principle. Sorry guys.


MS CRM additional login information

MS CRM need additional fields for CRM login. Currently scribe only takes userId and password. Add meta CRM info object for additional data provision into scribe URLs

Provide CRM account information using [CRM-MetaObject-Info] in Scribe request
http://localhost:8080/scribe/object/contact?[CRM-MetaObject-Info]

Replace [CRM-MetaObject-Info] with following string after relacing all '[]' values with your CRM credentials
MetaObject.CrmUserId=[Your-CRM-User-Id]&MetaObject.CrmPassword=[Your-CRM-Password]&MetaObject.CrmType=NS&MetaObject.CrmAccountId=[Your-CRM-Account-Id]

MetaObject.CrmUserId need your CRM user id
MetaObject.CrmPassword is your CRM password
MetaObject.CrmType is your CRM type, which is 'NS' in case of NetSuite CRM
MetaObject.CrmAccountId is your NetSuite CRM account id


Number parsing failing on Linux

It looks like parsing numbers into JSON is failing under linux. I'm not sure what the deal is, but you can see the failure in this REPL

Not sure why, but I can't load that REPL:


No Button Templates in the Repository

Is your feature request related to a problem? Please describe.
This repository contains website templates but lacks templates  for custom buttons
Describe the solution you'd like
Button templates; so that people can just use it directly in their websites without sweating a lot on the designs

@Diptenusarkar Assign this issue to me, I will add some button templates to the repository


Feature/pos penalties

Added penalties to invalid block proposers
Votes

Regular votes = voting for a valid block
Evidence votes = voting against an invalid block
/conensus/number/${blockNumber}/${blockHash}/vote/${address} { block_hash, stake, is_against, offense_type }

Proposals

offenses field introduced that counts how many offenses the address has made per type ({ [address]: { [offense_type]: number } })

Blocks

evidence field introduced ({ [address]: [{ offense_type, transactions, block, votes }] })

_blacklist native function

updates /consensus/blacklist and stake lockup duration upon a proposal with the offenses field

Took me a while to fix all the bugs & pass the integration test (still couldn't get the restart test case to work, so it's now commented out..) Extra thanks to @platfowner for taking the time to review this PR! 🙌 Really appreciate all your comments & thoughts.

Since I'm using the timestamp of the block that the transaction is included in as blockTime, it's null when the tx is executed on node's db.

I see. Would it make sense to use the timestamp of the previous block, which is available most cases (in db, just use lastBlockTime() like lastBlockNumber())? A value not too far from now and deterministic in the network would suffice? Wondering your thoughts.

Since I'm using the timestamp of the block that the transaction is included in as blockTime, it's null when the tx is executed on node's db.

I see. Would it make sense to use the timestamp of the previous block, which is available most cases (in db, just use lastBlockTime() like lastBlockNumber())? A value not too far from now and deterministic in the network would suffice? Wondering your thoughts.

Hmm, I think we should still try to use the timestamp of the block that tx is in if possible, to reduce confusion. But using lastBlockTime() for executing on node.db sounds good to me! I'll try to add the changes to my future PR! Please let me know if I'm not understanding correctly.


Install of keybase_amd64.deb on Ubuntu 20.04 Focal Fossa ISO boot fails

For some reason, when booting the Ubuntu 20.04 Focal Fossa ISO and selecting "Try Ubuntu", installation of Keybase fails. Doing so on an installed system running Ubuntu 20.04 Focal Fossa works fine.
Steps to reproduce

Boot a Ubuntu 20.04 ISO
Select "Try Ubuntu"
Open a Terminal
sudo apt install curl
curl --remote-name https://prerelease.keybase.io/keybase_amd64.deb
sudo apt install ./keybase_amd64.deb

Expected results
Keybase would install on a stock Ubuntu LTS ISO
Actual results
ubuntu@ubuntu:~$ curl --remote-name https://prerelease.keybase.io/keybase_amd64.deb
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  123M  100  123M    0     0  5617k      0  0:00:22  0:00:22 --:--:-- 5899k
ubuntu@ubuntu:~$ sudo apt install ./keybase_amd64.deb
Reading package lists... Done
Building dependency tree
Reading state information... Done
Note, selecting 'keybase' instead of './keybase_amd64.deb'
Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:

The following packages have unmet dependencies:
 keybase : Depends: libappindicator1 but it is not installable
           Depends: libgconf-2-4 but it is not installable
E: Unable to correct problems, you have held broken packages.

What it looks like on an installed Ubuntu 20.04 system for comparison
user@myhost:~$ sudo apt install ./keybase_amd64.deb
Reading package lists... Done
Building dependency tree
Reading state information... Done
Note, selecting 'keybase' instead of './keybase_amd64.deb'
The following additional packages will be installed:
  gconf-service gconf-service-backend gconf2-common libappindicator1 libdbusmenu-gtk4 libgconf-2-4
Suggested packages:
  indicator-application
The following NEW packages will be installed:
  gconf-service gconf-service-backend gconf2-common keybase libappindicator1 libdbusmenu-gtk4 libgconf-2-4
0 upgraded, 7 newly installed, 0 to remove and 4 not upgraded.
Need to get 909 kB/130 MB of archives.
After this operation, 483 MB of additional disk space will be used.
Do you want to continue? [Y/n] y

A possible reason might be you need to apt update first.


Change SNP to Gene mapping

It should be strongest, not what it is now.

Make this match the default in Camoco

This option works now, can choose either, pending camoco PR


Packages for 2024-2.1 envs

Updates:

[x] bluesky v1.12.0+
[x] blop - renamed from bloptools (to be released with the new name)
[ ] #38
[x] pipx
[x] MX packages by @JunAishima
[x] ipydatagrid (example: https://github.com/bloomberg/ipydatagrid/blob/main/examples/CellEditing.ipynb)

For CHX:

[x] python-pptx (https://pypi.org/project/python-pptx/ and https://anaconda.org/conda-forge/python-pptx)
[x] fpdf2 (https://anaconda.org/conda-forge/fpdf2)
[x] pyobjcryst (https://anaconda.org/conda-forge/pyobjcryst)
[x] diffpy.* packages
[x] pyfai v2024.2.1+

Added most of the requested packages, besides pyarrow (have opened an issue #38 for that).


Custom request

Add a custom request protocol to iroh
It sends a Bytes and expects a GetRequest and the data associated to that GetRequest.
What exactly to do in the Bytes is completely up to the user.
Look at the test_custom_request_... tests for inspiration how to use it.

Closing this since it is unmergeable. Gonna do a new branch.


Rewrite BrokerServiceTest#testBrokerStatsTopicLoadFailed

The BrokerServiceTest#testBrokerStatsTopicLoadFailed is disabled by #20494.

Remove the test case and if the coverage is necessary, file a new ticket to add it back.

Originally posted by @tisonkun in https://github.com/apache/pulsar/pull/20494#pullrequestreview-1462643509
The test BrokerServiceTest#testBrokerStatsTopicLoadFailed introduced in #19236 is invalid.

It modifies the current shared state and that causes other tests to fail.
The test uses Awaitility with a 2 minute timeout. That is too long for unit tests.

https://github.com/apache/pulsar/pull/22580 is trying to fix this issue


Bug: CURL call returns always code 200 in case de server uses HTTP/2

When making a curl call using the CURLRequest class and the server is IIS using the HTTP 2.0 protocol, the first header starts with "HTTP/2 "
The file CURLRequest.php contains this regex parsing on line 598:
preg_match('#^HTTP/([12].[01]) ([0-9]+) (.+)#', $header, $matches);
This fails to parse. It would work if the server used HTTP/2.0 but unfortunately, IIS says HTTP/2.
As a result, no status code is retrieved and CodeIgniter probably somewhere down the way decides it's 200.
CodeIgniter 4 version: 4.0.4
Affected module(s): CURLRequest
*Context: * IIS

I did have a look at this and did some testing. This what I believe is the correct regex expression:
#^HTTP/(12?) ([0-9]+) (.+)#

Thanks! Would you submit a Pull Request with your update regex?

I will try. Right now I'm quite under pressure from the actual commercial project and when I look at the process of correctly submitting a pull request, it takes more time than I have with the required test cases and so...

I totally get it. Do what you can. I think in this case you could probably just edit the files right in GitHub: submit the updated regex and then add to https://github.com/codeigniter4/CodeIgniter4/blob/develop/tests/system/HTTP/CURLRequestTest.php a copy of testResponseHeaders() with the shortened header.


SignatureHelp UI is still open when completionlist ui open

From @BleyChen on January 26, 2016 9:47
Repro steps:

type c# code as following to trigger the SignatureHelp UI:
Console.Beep(
backspace four characters ,signaturehelp ui still exists when completionlist ui display

Copied from original issue: Microsoft/vscode#2274

@jrieken -- how would this be an extension issue?

Signature help works like this in VS Code.

you register a trigger character, like (
when that is hit the language service is asked for signature help on basically each subsequent character being typed
signature help stops/hides when the language service return something falsy (undefined  et al)

got it -- thanks!

This was fixed by #107


[Feature request] Custom format replacements in custom modules

I'm currently writing a custom module to monitor free space on my SSD. However, custom modules that return a JSON can only return a text field containing already formatted text. I would like to return different values (free space, used space, total space, usage) and let the format be specified in my waybar config file.
Here is an example of what I would like to return with my module:
{"mount": "/home",
 "total": "871",
 "used": "86",
 "free": "740",
 "usage": "9.8",
 "tooltip": "",
 "class": "ok"}

And I would like to define the format this way in my config file:
"format": "{mount}: {used}G/{total}G"
That would make custom modules better by dissociating data and format.

It'll be possible once #404 get finished :)

OK. I will be waiting for this update then :smile:

Did this happen? I just spent a day trying to output my own json fields but alas... Now I feel a bit silly as it doesnt seem to be supported ;) LOL #1118  https://github.com/rileyrg/linux-init/blob/master/directories/bin/sway/waybar-ip-info-json

I don't think extending format like in the example  is that important, since you can just format in your external helper.
It would be nice to be able to pass more fields from custom modules though.

Formatting in the external helper is exactly what I wanted to avoid and indeed previous comments in the thread agreed . Certainly it seemed (lol!) obvious to me that adding in {fields} would just work if I returned them in json object.  That said, yes, I can separate the json object collection and the formatting into different scripts. Anyway, I just format in the help now as you say. cheers!


Docs: Add link to SpringArm3D tutorial

Links to the tutorial added in https://github.com/godotengine/godot-docs/pull/10156.

Thanks!


Overlap between folders when using conditions

Issue: when you switch background color from color to gradient, the options displayed conditionally for gradient will overlap the bottom spacing folder.
See: https://codesandbox.io/s/wonderful-sun-qpgznn?file=/src/App.js

Hopefully should be fixed in #412


Ubuntu 20.04 Support

Hello,
Does gamma-desk work on Ubuntu? I see there is a linux_config.json located here:
https://github.com/thocoo/gamma-desk/tree/main/gdesk/config
But when pip install per the README this file is not present in /gdesk/confg. I can manually copy it, but I get several errors. I see the defaults_linux.json hasn't been updated in ~2 years, so I just wanted to check:
1.) Do you expect gamma-desk to work out of the box on Linux?
2.) If so is there a reason defaults_linux.json isn't installed when installing via pip?
Thanks,
Denver

Hi Denver,
Good to hear from you.
We do have it running on linux but the default_linux.json file being missing is still something that needs to be fixed. What kind of errors are you getting when you manually add it? On which python and pyside version are you?
Regards,
Lou

Hi Lou,
Good to hear from you as well :)
It looks like my above issues were user error (running via SSH, so it was a bit of a mess). I ended up just transitioning to mac and noticed there was no system support for Darwin. I added the bare minimum needed to get gamma-desk working on mac and the PR is here:
https://github.com/thocoo/gamma-desk/pull/32
Thanks,
Denver


Create Top Contributor Badge

We recently announced our 2018 Top Contributors.
This award is based loosely off of Quora's Top Writers award. And we want to similarly recognize freeCodeCamp's Top Contributors with a badge on their profile:

I can think of no better icon to represent Top Contributors than GitHub's Pull Request Icon, which happens to be MIT licensed.
Here's what we need to do:

Add a Top Contributor attribute to the user object. Something like this:

"yearsTopContributor": [
  "2018",
  "2019"
]

Then, if a user is a Top Contributor, render the badge on their public potfolio:

Note: we will want to include just the SVG for the individual pull request icon - not the entire icon library.

Note: we will want to include just the SVG for the individual pull request icon - not the entire icon library.

We already have the FA library available to us. No changes are necessary to the packaging.

I'd like to give this a try!
@raisedadead I couldn't find the pull request icon in font awesome so I will just download the icon in the link that Quincy provided and use it in the user profile.

I think we could use https://fontawesome.com/icons/trophy-alt?style=solid (we have a pro licensing for font awesome)

Alright, I've got nearly everything down, but I can't seem to get font awesome to work. I tried to go the traditional i tag approach, then I found the FontAwesome component in one of the files and tried that, but no luck. Can you help? @raisedadead

Yes of course. Please throw in a PR (with whatever you have), I'll be advising further on the code on that thread.

AFAIK the new site is built using react, and there's a better approach available than using the traditional i tags. I can help out with creating the relevant react components for the same!


Sets aren't comparable

I was wanting a unionAll function:
unionAll : Set (Set comparable) -> Set comparable

I went to define it as such:
import Set as S

unionAll xs = S.foldl S.union S.empty xs

a = S.fromList [1, 2, 3]
b = S.fromList [3, 4, 5]
xs = S.fromList [a, b]

main = asText <| unionAll xs

Receiving the following compile error:
Type error on line 3:
Expecting something comparable such as an
Int, Float, Char, String, or a list or tuple of comparables.

        S.union

   Expected Type: S.Set comparable
     Actual Type: comparable

Compiled on share-elm with master/HEAD

Closing in favor of #1008 which tracks a bunch of issues in this ballpark. Idea is that we centralize the feedback in one place so we can have a better answer.

I was wanting to keep track of "collections" of data I am subscribing to over websockets.
e.g., I might be subscribed to channel namefriends::user:12345
This is what I wanted:
type Collection
    = UserFriendsCollection String
    | SomeOtherCollection String
    | ...

type alias model =
    { collections : Set Collection
    , ...
    }

I am trying to hack out something using tuples, but a union type would have fit nicely here.

I was trying to do the opposite of the OP. I wanted to make a function powerSet:
powerSet : Set comparable -> Set (Set comparable)

The power set of a set is a common function of sets in mathematics, and it is semantically incorrect to represent that as a List or Array of Sets.

Thinking about this more, I can't say I understand why Sets can only have comparable members. The only thing one needs to be able to do with members of a set is compare their equality, not their order.
So I would approach this problem differently and say instead of allowing Sets themselves to be comparable, we could simply remove the constraint that Sets contain only comparable types.

I posted about this on Discourse. https://discourse.elm-lang.org/t/why-must-set-members-be-comparable/3205/3 The answer makes sense, so I'm back to the OP's position. If a Set has a deterministic ordering of comparable elements, it should make sense for the set itself to be comparable, just like tuples of comparable types are also comparable.


There is no 'cross_entropy_sigmoid_loss'.

fastreid/modeling/losses/__init__.py line 7:
from .cross_entroy_loss import cross_entropy_loss, log_accuracy, cross_entropy_sigmoid_loss

But these is no cross_entropy_sigmoid_loss in cross_entroy_loss.py.

fixed


Mistral Models ct2 support

Feature request
ctranslate2 doesn't support mistral model conversion out of the box
https://github.com/OpenNMT/CTranslate2/issues/1501
currently when using ct2 it automatically converts the model on the fly.
can we allow support to convert model to ct2 offline? that way i  can convert mistral model offline and then use it with TGI.
Motivation
mistral support
Your contribution

Aren't the models converted by CT2 reusable and offline? Like my one here.


Support multiple directories for migrations

Consider the following folder structure:
migrations
migrations/base
migrations/dummy_data
Now, depending on some input either only migration files in base are to be executed or also the ones that insert some "dummy data".
From the top of my head there are two options to solve a situation like that:

Either make https://github.com/Keats/dbmigrate/blob/master/dbmigrate-lib/src/files.rs#L94 recursive
Allow multiple folders to be passed to the function

The workaround is copying everything from base also to dummy_data which is not really nice.
I wanted to discuss this, before starting any work.

I don't think it really make senses, fixtures are not migrations and are not meant to be handled by dbmigrate. You can load fixtures directly with psql or other cli tool if it's in a sql format

Point taken.
Generally splitting up into multiple different folders probably doesn't make sense due to the numbering which has to be kept in sync over multiple directories then.


"Roșiori" is listed in the drop-down menu

Description:
"Roșiori" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Roșiori" in the search bar.

Expected results
The location is listed in the drop-down menu as "Roșiori, Brăila".


Error starting bot

Please tick all applicable boxes.

[x] I am using Python 3.5.3 or higher (run python --version on the command line)
[x] I have followed the official guides to install the bot for my system
[x] I have updated my dependencies to the latest version using the appropriate update script

Which version are you using?

[x] The latest master version (release-260419)
[x] The latest review version

What type of issue are you creating?

[x] Bug
[ ] Feature request
[x] Question

Description of issue
im getting the error that the music bot cannot start "Error starting bot"
Steps to reproduce

musicbot.log
Log file
Please attach your MusicBot log file (located at logs/musicbot.log) to this issue. You can do so by dragging and dropping the file here. If you do not include your log file, you WILL be asked to provide one.

Duplicate of #1997


After upgrade 4.24, wekan not working

After auto upgrade 4.24, wekan is not working!
Then i restart mongodb use follow command:
$ snap restart wekan.mongodb
And i view the log use the follow command:
$ snap logs -f wekan.mongodb
like this:
2020-08-31T11:52:14Z mongod.27019[10250]: [initandlisten] MongoDB starting : pid=10250 port=27019 dbpath=/var/snap/wekan/common 64-bit host=hbhly_87_150
2020-08-31T11:52:14Z mongod.27019[10250]: [initandlisten] db version v3.2.22
2020-08-31T11:52:14Z mongod.27019[10250]: [initandlisten] git version: 105acca0d443f9a47c1a5bd608fd7133840a58dd
2020-08-31T11:52:14Z mongod.27019[10250]: [initandlisten] OpenSSL version: OpenSSL 1.0.2g  1 Mar 2016
2020-08-31T11:52:14Z mongod.27019[10250]: [initandlisten] allocator: tcmalloc
2020-08-31T11:52:14Z mongod.27019[10250]: [initandlisten] modules: none
2020-08-31T11:52:14Z mongod.27019[10250]: [initandlisten] build environment:
2020-08-31T11:52:14Z mongod.27019[10250]: [initandlisten]     distmod: ubuntu1604
2020-08-31T11:52:14Z mongod.27019[10250]: [initandlisten]     distarch: x86_64
2020-08-31T11:52:14Z mongod.27019[10250]: [initandlisten]     target_arch: x86_64
2020-08-31T11:52:14Z mongod.27019[10250]: [initandlisten] options: { net: { bindIp: "<IP_ADDRESS>", port: 27019 }, storage: { dbPath: "/var/snap/wekan/common", journal: { enabled: true } }, systemLog: { destination: "syslog", quiet: true } }
2020-08-31T11:52:14Z wekan.mongodb[9523]: Error parsing command line: unrecognised option '--bind_ip'
2020-08-31T11:52:14Z wekan.mongodb[9523]: try 'mongo --help' for more information
2020-08-31T11:52:14Z systemd[1]: snap.wekan.mongodb.service: main process exited, code=exited, status=2/INVALIDARGUMENT
2020-08-31T11:52:14Z systemd[1]: Unit snap.wekan.mongodb.service entered failed state.
2020-08-31T11:52:14Z systemd[1]: snap.wekan.mongodb.service failed.
2020-08-31T11:52:14Z systemd[1]: snap.wekan.mongodb.service holdoff time over, scheduling restart.
2020-08-31T11:52:14Z systemd[1]: Stopped Service for snap application wekan.mongodb.
2020-08-31T11:52:14Z systemd[1]: start request repeated too quickly for snap.wekan.mongodb.service
2020-08-31T11:52:14Z systemd[1]: Failed to start Service for snap application wekan.mongodb.
2020-08-31T11:52:14Z systemd[1]: Unit snap.wekan.mongodb.service entered failed state.
2020-08-31T11:52:14Z systemd[1]: snap.wekan.mongodb.service failed.
How can i do?

Try rebooting with:
reboot

Because sometimes systemd gets confused with too fast changes.

Also check you have enough free disk space.

I still don't understand. I faced the same issue today, I tried:

snap stop wekan
systemctl stop snapd snapd.socket
systemctl daemon-reload
systemctl restart snapd.socket
snap start wekan

What does a reboot more than that?

@Glandos
What OS you have at server? I recommend Ubuntu 20.04 64bit.

I'm forced to used CentOS7, no choice here :/

If you have problems with Snap, and Snap does not start, you could try making Backup or rescuing from raw MongoDB files and then using Wekan with Bundle version - plain node+mongodb without Docker and Snap.
If Snap does not work well enough, what update method would be best? Do you update often with RPM packages, or something else?

I do think that if Snap does not work well enough on CentOS, then:
a) Some fix for Snap on CentOS should be made, so updating works well enough
b) Some alternative update method should be made

Using bundle is just too many steps to Install, I think.

@Glandos
Can you give me ideas how to make installing Wekan simpler? For example:
a) Should RPM package automatically create service files, etc?
b) Should there be some step-by-step install script?
c) Should configuring Wekan be made in webbrowser?
d) Have you used some other CentOS software that is easier to install and configure? How does that software install and work easier?

@Glandos
What databases you use mostly at CentOS, for other software? Would something be easier, if Wekan would support some other database than MongoDB ?

I just want to make it clear: rebooting the machine solved the issue, but I was wondering why (I edited my comment).
Snapd was installed from EPEL, and is at version <IP_ADDRESS>. I don't know if it's recent enough, but maybe the issue is systemd itself then (version 219 is quite old now).
To answer your last questions:
a) A RPM package can't ask question, and in my opinion, it should create all needed files to be able to launch the service.
b) Avoid step-by-step install script if possible. Maybe a few command like snap set wekan port='3001' are needed anyway
c) Yes, it is great to have a web based configuration UI, since wekan is a web UI. However, in order to start, it should have basic configuration first (e.g. hostname)
d) I usually install software using RPM. And for web applications, I tend to prefer scripts provided by web servers, like a2ensite for Apache, if they exists
For databases, postgresql is my favorite, because it has all the feature you need. But MongoDB is not an SQL database (as far as I know), so I'm not sure it can be switched with postgresql…

Yes, MongoDB is NoSQL database, with Javascript query language. Current Wekan does queries with Javascript. I'm just thinking about some future version of Wekan, doing some little experiments and tests.

Moved to https://github.com/wekan/wekan/issues/5562


Coverflow effect doesn't work on MacOS Safari 16.5

Check that this is really a bug

[X] I confirm

Reproduction link
https://codesandbox.io/s/funny-framework-69yj35?file=/index.html
Bug description
Creating a simple carousel with Swiper Element works on Chrome and Safari 16.1, but not on Safari 16.5
Expected Behavior

Actual Behavior

Swiper version
9<EMAIL_ADDRESS>Platform/Target and Browser Versions
MacOS Safari 16.5
Validations

[X] Follow our Code of Conduct
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.
[X] Make sure this is a Swiper issue and not a framework-specific issue

Would you like to open a PR for this bug?

[ ] I'm willing to open a PR

Duplicate #6650


[Add] : Add SuperSimpleDev Channel for Learning Frontend Development Course in Simplify Way

Site URL
https://www.youtube.com/@SuperSimpleDev/videos
Description of the above provided site

Add SuperSimpleDev Channel for Learning Frontend Development Course in Simplify Way.
Tech Stacks included in this Course is likely HTML, CSS, JavaScript and Other Basic Backend Course Videos
This Youtuber recently uploaded JavaScript Course is Really Awesome and His Teaching Style is Just Outstanding

Category
Youtube
Sub Category
web-development
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (optional)
[X] I am a GSSoC'23 contributor

Hi, @NarendraKoya999 are you working on this issues?

Hi, @NarendraKoya999 are you working on this issues?

This Issue is Already Merge in One PR. Please Closed.

Hi, @NarendraKoya999 are you working on this issues?

This Issue is Already Merge in One PR. Please Closed. Only 3 Issues are Pending Unassigned No : #1384 #1385 and #1458

Please read contributing guidelines. When your PR is merged or closed then its YOUR responsibility to close the issue

@rupali-codes. It's Tough Activity from My Side. Because of Issues are Locked. Not Possibility to Close that Issues. I have Mentioned Here #1340 #1341 #1342 #1343 #1344 #1345 #1346 #1347. I am Sure these All Issues are Linked to 2 PR's No : #1388 #1389

@rupali-codes. It's Tough Activity from My Side. Because of Issues are Locked. Not Possibility to Close that Issues. I have Mentioned Here #1340 #1341 #1342 #1343 #1344 #1345 #1346 #1347. I am Sure these All Issues are Linked to 2 PR's No : #1388 #1389 are Merged already.

can u please close all those issues?


DOT Token needed

@ymherklotz In one of the past commits, you deleted DOT, however NUMBER DOT SPACE(>=1) form an ordered list
1. 5
2.    4
5. 44
6.5

5
4
44
6.5

@ThunderMikey I can add a DOTNUM Token which would only match when there is a number, a dot and then a space. This would fix it. Otherwise I have no problem adding back the DOT Token, but then that wouldn't fix the issue above. Tell me what you prefer.

By adding DOT Token, I can match pattern NUMBER :: DOT :: SPACE _ :: toks' to be the case for ordered list in my program.


Streaming data ingestion support in Pinot for Redpanda.

Redpanda is a Kafka-compatible streaming data platform. This issue starts the community discussion around supporting Redpanda for streaming data into Pinot.

Its already support redpanda. Recently, I setup pinot with redpanda

Its already support redpanda. Recently, I setup pinot with redpanda

Thanks for mentioning. Since Readpanda is Kafka API compatible, it would be working.
question, did you set property streamType  as Kafka? Can you please share a sample config?

all configs for kafka are same for redpanda.
[Here}(https://redpanda.com/blog/streaming-data-apache-pinot-kafka-connect-redpanda) you can find details

@Jackie-Jiang / @xiangfu0, is it okay to follow the same config as Kafka or change something related to Redpanda, like streamType as RedPanda.
Also do we want to add the test cases and integration tests related to RedPanda like we are doing for Pulsar or Kinesis?

Seems like RedPanda is Kafka compatible, so I'd suggest keeping it simple and just follow the same configs. +1 on adding an integration test on RedPanda if it doesn't pull in too many dependencies.
@kelseiv Should we add some documentation for RedPanda?

@Jackie-Jiang, please assign this to me. I will work on this. Thanks

Hi @abhioncbr, are you still working on this ? If not I can give it a try too
@Jackie-Jiang my simple understanding on this is we just have to add some documentations on this as Redpanda uses the same Kafka protocol, similar to this doc https://docs.pinot.apache.org/basics/data-import/pinot-stream-ingestion/import-from-apache-kafka :thinking: ?

@dttung2905 you can work on this. Thanks

Thanks @abhioncbr . I will work on this during this weekend


Optimize GMC e2e test in CI and reduce the test duration

Since GMC e2e is moved to KIND environment. The testing always takes 18+ minutes to finish. We should figure out some possible optimizations to reduce the test duration in order to improve efficiency

The GMC e2e on xeon has been reduced to 12 minutes. The test log is here: https://github.com/opea-project/GenAIInfra/actions/runs/9834081498/job/27145358124

The GMC e2e on Xeon has been reduced to 10+ minutes. The test log is here: https://github.com/opea-project/GenAIInfra/actions/runs/9834247322/job/27145780072

Fixed in this PR: https://github.com/opea-project/GenAIInfra/pull/160


Fix pagination bugs on Flow page

Fix pagination bugs on Flow page:
-Next and Previous buttons don’t work when clicked
-Invalid pagination is generated in some cases. For example:

if the number of executions yields 3 pages in total, when the last page(3) is clicked the pagination displayed will be -1 0 1 2 3.

It also separates the implementation of Flow Trigger tab from that of Executions tab by creating a Backbone Model and View for the former.

@ypadron-in  please check  build failure

Pull Request Test Coverage Report for Build 4342

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.07%) to 36.785%

Totals

Change from base Build 4340:
0.07%

Covered Lines:
10903

Relevant Lines:
29640

💛  - Coveralls


Use PEP8 in tests and fix existing PEP8 errors

All our other projects enforce PEP8; we should do the same here for consistency.

Glad to see that all of my recent changes were causing problems. :+1:


linux/stdenv: Fixups to accomodate boostrap-tools changes and ssl in fetchurl

This is the second patchset for #8081. This adds support for the new boostrap-tools changes and also adds ssl verification support into fetchurl.

@copumpkin @shlevy Let me know what needs to be done to add ssl verification support to the darwin stdenv. It looks like there is already a working curl with https in the bootstrap tools. Would it make sense to add cacert to the darwin bootstrap-tools and then enable ssl verification throughout?

I added support to the darwin stdenv in both of these PR's, let me know if it works fine.

+1, having tls support in fetching is important as we don't know how users get the hash and they might use an incorrect sha. If we decide to use insecure fetchurl, we should fix it's message not to report the hash, but to rather instruct to use nix-prefetch-url.

I don't really see the point of this. I mean, the hash is verified by Nix, so what's wrong with using --insecure?

For end users yes this is true, but if developers use the workflow of
changing the package version / invalidating the hash + use nix-build to
tell them the new hash they won't be using ssl since they are relying on
fetchurl. As someone who is guilty of almost always doing this I think it
should at least have ssl verification behind it.
On Tue, Jul 7, 2015, 06:28 Eelco Dolstra<EMAIL_ADDRESS>wrote:

I don't really see the point of this. I mean, the hash is verified by Nix,
so what's wrong with using --insecure?
—
Reply to this email directly or view it on GitHub
https://github.com/NixOS/nixpkgs/pull/8082#issuecomment-119202508.

Yeah, as @domenkozar said, it's better to use nix-prefetch-url for that.

Sure, but it isn't always obvious what that URL is if you aren't directly
calling it (fetchFromGitHub and family) or the calls in Ruby gems. On top
of that it is too easy for other developers to follow the same practice. I
don't think the right answer is to not include this because users shouldn't
do it.
On Tue, Jul 7, 2015, 11:00 Eelco Dolstra<EMAIL_ADDRESS>wrote:

Yeah, as @domenkozar https://github.com/domenkozar said, it's better to
use nix-prefetch-url for that.
—
Reply to this email directly or view it on GitHub
https://github.com/NixOS/nixpkgs/pull/8082#issuecomment-119284564.

I always use the following hacky shell function around nix-prefetch-url:
prefetch-url-from() {
    local url=$(nix-instantiate --eval-only --xml --strict "${2:-.}" -A "$1".urls ${system:+--argstr system $system} | grep 'value=".*"' | sed 's^.*value="\(.*\)".*^\1^' | head)
    if test -n "$url"; then
        echo "fetching $url"
        nix-prefetch-url "$url"
    fi
}

To use:
$ cd nixpkgs
$ prefetch-from-url hello.src

will download the source of hello and print the hash on stdout.

Would still love to have this!


Why was node bumped to version 18?

Hi, thank you for this module. May I ask why node has been bumped to version 18? This is a breaking change for many devs.

Sorry if the latest release broke your development flow.
The reason is to support the latest release of marked https://github.com/markedjs/marked/compare/v4.3.0...v5.0.0

Thanks for the clarification, can you bump this:
https://github.com/gmsgowtham/react-native-marked/blob/5972610d5e0ffdef41f5deafcd833f69870bdf97/package.json#L169
to be 18.16.0?
Thanks

thanks @gmsgowtham, keep it up!

It was my bad. v18.16.0 has an issue with Jest, so blindly restricted it to v18.15.0 😅
v5.0.2 Addressed it


talk: clicking into a DM does not redirect/link to correct DM

Describe the bug, optionally with screenshots
I'm encountering a rather serious(?) issue in which I see that an urbit is DMing me, it has a blue unread dot, and when I click into it to read it, I'm redirected to a completely unrelated DM. Watch this video to see what I mean:
https://user-images.githubusercontent.com/1195363/216106463-e64cba9d-aa54-46cc-b516-13335b41ae21.mov
I click to chat with ~solfer-mafed, and am redirected to a DM with ~simfur-witwed, when I leave this latter chat, and try to access the first DM, I'm redirected to a chat with ~wicdev. I'm unable to click into the initial DM and view it.
Expected behavior
I expect to click on a DM and be brought to the DM in question, not an unrelated one
Additional context:

Ship: planet — ~fabled-faster
Instance type: Tlon-hosted
MacOS Ventura — 13.0.1 (22A400)
Safari — Version 16.1 (186<IP_ADDRESS>.12)
Chromium — Engine Version 109.0.5414.87

This is also happening for me since yesterday.  Happens when clicking on any non-group dm, I'm on hash blu1o for %talk


Add type attribute to A_CONST nodes.

This fixes an issue where BitString's cannot be deparsed properly
because they cannot be distinguished from normal strings.
Example:
SELECT B'0101'

and
SELECT '0101'

This patch adds a type which outputs the integer node tag so that
callers can be certain of the type of the constant given the limitations
of the serialization format (JSON). This also fixes a bug where
BitStrings were being dumped as invalid JSON:
{ "val": b0101 }

Looks good!
I'll do some quick testing with the Ruby library, to make sure this doesn't confuse any callers - looking to merge this later today.

@lfittl one possible thing to consider with the bitstrings is if the leading char should be stripped.
Current PR:
{
    "A_CONST": {
        "location": 7,
        "type": 654,
        "val": "b0001"
    }
}

The lib could either return "val": "b0001" or "val": "0001" with the leader stripped since it's implicit from the type. Not sure which is better, just thought I'd bring it up!

@zhm Good point - lets maybe do the version without b, since you're right, its already in the type.
Also, looking at the output, do you think we can output the type as a readable name instead? (might make parsing more reliable across versions)

:+1: for readable type names!

I updated the PR to use strings instead of enums for the type names.
We now have:
integer
float
string
bitstring
null

Also I think we need to leave the b prefix on bitstrings. It turns out there's yet another syntax for bitstrings. Both of the following are returned as bitstrings:
SELECT B'0101`

> b0101

SELECT X'EFFF`

> xEFFF

Also note that I don't think bitstrings are particularly useful to most people, just trying to get some random SQL scripts to parse/deparse ;)
I suppose it's possible that Postgres could introduce new value types, so maybe this all makes sense anyways.

@lfittl also, are there any other scenarios where the caller would care about the type of the Value struct? I had originally put the type info in the _outValue function but I noticed the Value node is used for very trivial things like column names and such. Is there anything other than A_CONST where a bitstring/integer/float would come up?

@zhm Patch looks good!
From my quick grep-ing through the node definitions, it looks like A_CONST is the only one that really uses different types - all others seem to be strings.
If we wanted to put it into all value nodes, we could consider only outputting the type if its not string (and maybe also if its not null), to avoid bloating the tree. But I'll merge this as-is, we can revisit later if needed.


Disable throttling

I'm trying to disable throttling at all in the config, but it seems that it's not possible.
I've tried to set
    monitor: {
      throttling: undefined
    }

and other variations, but when running push I get:
> preparing all monitors
> creating all monitors
x Error
   > Failed to save or update monitor. Configuration is not valid: monitor(prod-uk login)
       Invalid value "undefined" supplied to "throttling,download" | Invalid value "undefined" supplied to "throttling,upload" | Invalid value "undefined" supplied to "throttling,latency"

   > Failed to save or update monitor. Configuration is not valid: monitor(prod-uk matching page)
       Invalid value "undefined" supplied to "throttling,download" | Invalid value "undefined" supplied to "throttling,upload" | Invalid value "undefined" supplied to "throttling,latency"

Thanks for catching this.
Seems to be a bug in the Synthetics agent CLI and also on the Kibana API level. Would require a new release and also have to be done on the new elastic stack releases.

I believe @justinkambic post FF tested this as part of another issue.

++ I tested this feature e2e.


The generated command to run tests on Typescript is wrong

Package version
1.2.0
Describe the bug
When running npm init japa@latest and choosing Typescript or Typescript ESM, at the end of the setup these lines are printed:
✔ Configured japa successfully
❯ Run "node bin/test.ts" to execute tests

if the user directly runs the command node bin/test.ts it will give them an error. They will have to go search in the docs how to correctly run tests with Typescript ...
Reproduction repo
No response

Hello,
Same here ! I've been created a fresh install of japa with typescript. I added expect as assertion library
But the type of TestContext it's not resolve as expected.
It's possible to fix this issue ?
As a can see the PR linked is out of date.

This issue was about Japa 2. The problem has been solved with Japa 3. In any case, I've just tried it and I've had no problems, Testcontext and Expect are correctly typed.

@Julien-R44 thank for you're reply...
I'm actually on monorepo with turborepo
so i can share some file
Node version: v20.5.1
Turborepo: latest
// Package.json
  "scripts": {
    "test": "node --loader ts-node/esm --enable-source-maps bin/test.ts"
   },
  "devDependencies": {
    "@japa/api-client": "^2.0.0-1",
    "@japa/expect": "^3.0.0",
    "@japa/runner": "^3.0.0-9",
    "@types/node": "^20.8.2",
    "ts-node": "^10.9.1",
    "typescript": "^5.2.2"
  }

import { test } from "@japa/runner";

test.group("Maths.add", () => {
  test("add two numbers", ({ expect }) => {
    // Test logic goes here
    expect(1 + 1).toBe(2);
  });
});

// japa config
import { configure, processCLIArgs, run } from '@japa/runner'
import { expect } from '@japa/expect'
import { apiClient } from '@japa/api-client'

processCLIArgs(process.argv.splice(2))
configure({
  files: ['tests/**/*.spec.ts'],
  plugins: [expect(), apiClient('http://localhost:3333')],
})

run()

And i've try to debug, so as i understand the type definitions from @japa/expect doesn't work
// the error is from here => '@japa/runner/core' does not exist
declare module '@japa/runner/core' {
  interface TestContext {
    expect: Expect
  }
}

If you want more tell me :)
Best regards
Vincent

well, i would say it's probably something caused by turborepo and the way it handles packages, since it works totally fine with a "simple" repo. If it's possible for you to create a minimal reproduction repo with turborepo, i would be happy to take a look !
what package manager are you using?

Closing since original issue has been fixed
@vincent-vade Feel free to open another issue for that ( perhaps with a reproduction ? ). To me, it looks like a different issue.


GitHub Action: Check for broken links

(How it would work: Github workflow that just runs that part of the build script, red x when it fails and green checkmark if it is happy)

Full thread: https://prisma-company.slack.com/archives/C5Z9TH6N9/p1592476676060100

Can I see the Full thread if possible

Unfortunately not (as people did not expect this to become public), but I can paraphrase:

We have a build step that outputs warnings in the Netlify console about broken links:

This issue is about the idea to turn that into a Github action that produces a check on all PRs that would fail if there are broken links, and pass if there are not.
The script or build step can probably be extracted and modified to not only warn, but fail on errors.
That of course also requires the codebase to be really broken link free when we start.

Does that give you enough context to understand what we meant?

That will be enough @janpio, No problem at all, Well what I understand if we fix the warning of broken links, I think we can fix this
and I already did that on my local deployment, Just waiting for some responses from @mhwelander to confirm about  non-linking of few warning links, then I will make pull request with the fix.

Removed some of the  anchor links for now and used a nearby heading (needs must) - feature request #508 here. There are 0 now (ignore Netlify - there's an issue: #506)
Big thanks @rahul3v for all the work you've put into the link checker 🚀

But we do not have an action that runs automatically and fails PRs via GH Action yet, right?

Closing this since we do not have a Github Action for broken links but rather a plugin which outputs the broken links as errors during the build process.


Jenkins Integration

Bug Report or Feature Request (mark with an x)
- [ ] bug report -> please search issues before submitting
- [ ] feature request
- [x] question

Versions.

@angular/cli: 1.0.4
node: 7.9.0
os: darwin x64
@angular/animations: 4.1.3
@angular/common: 4.1.3
@angular/compiler: 4.1.3
@angular/core: 4.1.3
@angular/flex-layout: 2.0.0-rc.1
@angular/forms: 4.1.3
@angular/http: 4.1.3
@angular/material: 2.0.0-beta.5
@angular/platform-browser: 4.1.3
@angular/platform-browser-dynamic: 4.1.3
@angular/router: 4.1.3
@angular/cli: 1.0.4
@angular/compiler-cli: 4.1.3

Repro steps.

Created a project with angular CLI. Created a project on Jenkins using following shell execution steps:
npm install --silent
./ng test --code-coverage --single-run
# I created a symlink for ng to the version installed by npm, see below

The log given by the failure.

n/a
Desired functionality.

I would like to integrate a project using Angular CLI onto our Jenkins server.
Mention any other details that might be useful.

I don't have a lot of Jenkins configuration experience, but from how it looks like it is not that simple out of the box. We are running compass on our Jenkins server, we have NPM version 4.6.1 on our Jenkins server. However, it looks like I need to adjust the configuration quite a bit more. Like changing Chrome to PhantomJS, convert the istanbul reports to cobertura reports etc.
I even created a symlink for angular cli in the project, so we don't have to install it globally on the server using: ln -s node_modules/@angular/cli/bin/ng ng.
I feel like I am missing something obvious and all these steps shouldn't be necessary. Unfortunately I am not finding any documentation about CI integration here or somewhere else (google...).
Could you give me some pointers/hints please?

E.g. Is there a karma.conf template for jenkins?

Thinking about it, my question is probably not that clear and might be more Angular 4 related than Angular CLI...
I also looks to me like PhantomJS is currently not supporting ES6, but maybe I misread that. I will look into this more tomorrow.

I have a Jenkins CI job setup that runs and builds angular...
As a part of the Jenkins job we do not install the CLI globally on our build machines at all, but instead call
npm install on each git pull down that kicks off the job. This will install whatever version of the CLI that the package.json specifies for that particular revision.
We also do testing with ng test --single-run --code-coverage (which typically works fine) we also test on chrome and not PhantomJS.
I hope that this is helpful but if you need anything more specific I'd be happy to help you out.

@BustyLoli-Chan thanks a lot for your answer. That's very helpful. For some reason I thought I'd need a headless browser, but I am having issues getting it working with PhantomJS.
Also, I was hoping I could put the browser onto the package.json – as a dependency – , however as far as I know this is only possible with PhantomJS.
But I will give it a try with Chrome and report back.
One question: Did you also use a symlink to use ng – since you didn't install it globally?

Package.json
"karma-chrome-launcher": "~2.1.1",
It wasn't necessary to make a symlink since running an npm install just downloads and puts the "binaries" in the local directory.

@BustyLoli-Chan Thanks!
Hmm.. that's odd.. in our installation it wasn't able to find 'ng', even after running npm install. I will investigate that. Thanks for pointing that out.

we are windows machines. I don't know if ./ng would fair better for you since most linux machines don't have their current directory added to their path.
Most of our ng jobs are simply executed as bat 'npm run something'
and in our package.json we would define
"scripts": {
    "something": "ng do something or other",
  },

I realize this actually different from what I first told you because technically it's running npm which is installed globally and npm is doing the intermittent lookup work to find the CLI and run its stuff

@BustyLoli-Chan Oh okay! That makes sense though. Thanks for the explanation!
I think I figured it out. I will close this issue now and add another comment explaining what I did as soon as I verified that everything works the way I want it.

Thanks again for your help. So there were several things that confused me:

I didn't know how easy it was to have istanbul to create a cobertura report (alternatively we could have installed a different plugin for istanbul)
I didn't know PhantomJS is not supporting ES6
I didn't realize that I need to "turn on" polyfills

What I ended up doing was:

turn on all polyfills that applied (we activated all of them in the src/polyfills.js-file)
added intl, phantomjs-prebuilt, karma-phantomjs-launcher and karma-junit-reporter to package.json

This is how our karma.conf.js ended up looking like:
// Karma configuration file, see link for more information
// https://karma-runner.github.io/0.13/config/configuration-file.html

module.exports = function (config) {
  config.set({
    basePath: '',
    frameworks: ['jasmine', '@angular/cli'],
    plugins: [
      require('karma-jasmine'),
      require('karma-chrome-launcher'),
      require('karma-jasmine-html-reporter'),
      require('karma-phantomjs-launcher'),
      require('karma-coverage-istanbul-reporter'),
      require('@angular/cli/plugins/karma'),
      require('karma-junit-reporter')
    ],
    client:{
      clearContext: false // leave Jasmine Spec Runner output visible in browser
    },
    files: [
      { pattern: './src/test.ts', watched: false }
    ],
    preprocessors: {
      './src/test.ts': ['@angular/cli']
    },
    mime: {
      'text/x-typescript': ['ts','tsx']
    },
    junitReporter: {
            outputDir: "test-results",
            outputFile: "unit-test-results.xml"
    },
    coverageIstanbulReporter: {
      reports: [ 'html', 'lcovonly', 'cobertura', 'text-summary' ],
      'report-config': {
        cobertura: {
          file: 'cobertura.xml'
        }
      },
      fixWebpackSourcePaths: true
    },
    angularCli: {
      environment: 'dev'
    },
    reporters: config.angularCli && config.angularCli.codeCoverage
              ? ['progress', 'junit', 'coverage-istanbul']
              : ['progress', 'kjhtml'],
    port: 9876,
    colors: true,
    logLevel: config.LOG_INFO,
    autoWatch: true,
    browsers: ['PhantomJS'],
    singleRun: false
  });
};

Hi all, I just set everything up to use PhantomJS in my Angular4 project. I've uncommented all the polyfills, added intl as a dependency, configured my karma.conf.json as @skofgar commented before and targeted to es5 on tsconfig.spec.json. I'm having this problem on and on though:

Does anyone know how would be happening?
Thanks a lot in advance,
Cristóbal

@cbeldacap: As stated above phantomjs doesn't support es6, so it will choke on any 'const' variables in your test-code.
Your non-test files should be transpiled before the tests start so you just need to avoid const in your testfiles and you should be good (use var instead)

I could finally solve it importing some polyfills! (and some others not) :)
in my case I ended up having those ones:
/** IE9, IE10 and IE11 requires all of the following polyfills. **/
import 'core-js/es6/symbol';
import 'core-js/es6/object';
// import 'core-js/es6/function';
import 'core-js/es6/parse-int';
import 'core-js/es6/parse-float';
import 'core-js/es6/number';
import 'core-js/es6/math';
import 'core-js/es6/string';
// import 'core-js/es6/date';
import 'core-js/es6/array';
// import 'core-js/es6/regexp';
// import 'core-js/es6/map';
// import 'core-js/es6/set';

Already solved and totally working!

can anyone help here https://stackoverflow.com/questions/47315228/deploying-an-angular-application-using-continuous-integration-from-git-or-gitlab


[SPARK-19112][CORE][FOLLOW-UP] Add missing shortCompressionCodecNames to configuration.

What changes were proposed in this pull request?
Spark provides four codecs: lz4, lzf, snappy, and zstd. This pr add missing shortCompressionCodecNames to configuration.
How was this patch tested?
N/A

Test build #91157 has started for PR 21431 at commit 877d270.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3586/
Test PASSed.

Test build #91157 has finished for PR 21431 at commit 877d270.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

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

How was this patch tested?

BTW, I also assume you manually tested zstd works as documented?

Yes. I have tested with —conf spark.io.compression.codec=zstd.

Merged to master and branch-2.3.


Add equality minus epsilon comparison for Doubles

@millieSavalia the reason why I did not add you as a reviewer is because you are not part of the members for this repo.  We use it as a collection of generic utilities. I just invited you, so you should have an invite in your inbox.


Schema validation error when using enum in two tables

I have following schema:
create type user_opinion as enum ('pro','cons');
create table user_view(id serial primary key,view user_opinion);
create table argument(id serial primary key,side user_opinion);

When I am trying to start postgraphql it gives me error:
Error: Schema must contain unique named types but contains multiple types named "UserOpinion".
    at invariant (/usr/lib/node_modules/postgraphql/node_modules/graphql/jsutils/invariant.js:19:11)
    at typeMapReducer (/usr/lib/node_modules/postgraphql/node_modules/graphql/type/schema.js:207:29)
    at /usr/lib/node_modules/postgraphql/node_modules/graphql/type/schema.js:234:22
    at Array.forEach (native)
    at /usr/lib/node_modules/postgraphql/node_modules/graphql/type/schema.js:225:37
    at typeMapReducer (/usr/lib/node_modules/postgraphql/node_modules/graphql/type/schema.js:236:7)
    at /usr/lib/node_modules/postgraphql/node_modules/graphql/type/schema.js:234:22
    at Array.forEach (native)
    at /usr/lib/node_modules/postgraphql/node_modules/graphql/type/schema.js:225:37
    at typeMapReducer (/usr/lib/node_modules/postgraphql/node_modules/graphql/type/schema.js:236:7)
    at Array.reduce (native)
    at new GraphQLSchema (/usr/lib/node_modules/postgraphql/node_modules/graphql/type/schema.js:104:34)
    at schema (/usr/lib/node_modules/postgraphql/dist/graphql/createSchema.js:25:39)
    at /usr/lib/node_modules/postgraphql/dist/createGraphqlSchema.js:32:57
    at next (native)
    at step (/usr/lib/node_modules/postgraphql/dist/createGraphqlSchema.js:17:191)

Schema validator works using reference comparison(===), so that can be fixed by using cache by name for enum types in https://github.com/calebmer/postgraphql/blob/master/src/graphql/getColumnType.js#L113.

Fixed in 1.2.2 with #35, thanks so much!


fail to assign a function to a field, then call it:  "You can't overwrite static expression"

[] > xx
  cage > tmp
  [nothing] > xtmpFuntheTest
    stdout "xtmpFuntheTest" > @
  [] > xtheTest
    cage > xcallme
    seq > @
      xcallme.write (xtmpFuntheTest)
  seq > @
    stdout "x\n"
    xtheTest.@
    tmp.write ((((xtheTest).xcallme)) ("None"))
    TRUE

You have to do it this way:
[] > xx
  cage > tmp
  [nothing] > xtmpFuntheTest
    stdout "xtmpFuntheTest" > @
  [] > xtheTest
    cage > xcallme
    seq > @
      xcallme.write (xtmpFuntheTest)
  seq > @
    stdout "x\n"
    xtheTest.@
    tmp.write ((((xtheTest).xcallme.@)) ("None"))
    TRUE

I just added a test, which passes: 06d12264


 Search for Beautify and click on it.
Expected results
The icon looks correct on github.

Actual results
The shadow is a solid color and obviously incorrect.

Thanks to faustinoaq for letting me know that VS Code no longer supports SVG icons. I'll just make a png.


fix: dark mode icon doesnt change

dark/light mode text/icon now changes dependent on state

Closed in favour of other PR


Bump Bundler to 2.4.15

New version has a working bundle lock --update --bundler command, so we can bump the Bundler version locally without having to actually install any gems.

Actually we should let @deivid-rodriguez merge/deploy this one, as he will best know if any bugs have cropped up preventing it...

Both 2.4.15 and 2.4.16 had some regressions over previous versions, so I'd go straight to 2.4.17. If that passes CI, I think it's good to go as is, since 2.4.17 did not have any regressions reported.


Extract variables to replace by environment var.

Issue #, if available:
Description of changes:
for a better understanding, Extract variables to replace with environment var. like my Cloudformation Stack name
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Same in this file :

https://github.com/aws-samples/eks-workshop/edit/master/content/x-ray/role.md

Better Fix with this PR https://github.com/aws-samples/eks-workshop/pull/229


OpenStack infra node destroy should use SupportFeaturesMixin instead of validate_

This issue was raised in this rPR https://github.com/ManageIQ/manageiq/pull/11500

@miq-bot  move_issue manageiq-providers-openstack


Add channel.live flag for user defined filters.

Describe your issue

There is currently no way to check if a channel is live via filters, I wanted to create a filter to block online chat, as I didn't want to see it. (Just close it 4Head)
I personally think this filter system along with some imagination and more values could work wonders!
https://github.com/Chatterino/chatterino2/blob/770b9f263b947b7fc1b293bcd29878463e97d534/src/controllers/filters/parser/FilterParser.cpp#L65-L84
I propose channel.live, a boolean value showing whether the channel is live.
OS and Chatterino Version

All verisons please 👍

I had someone ask me if we had the functionality for something similar to this in the past few days, so I think it is something that would be wanted.

Doesn't look too bad to implement.


Use pflag

Looks good to me.
richard   pflag  ~  projects  improbable-eng  156-pflag  1  ./bin/manager -h
Usage of ./bin/manager:
      --backup-tmp-dir string    The directory to temporarily place backups before they are uploaded to their destination. (default "/tmp")
      --enable-leader-election   Enable leader election for controller manager. Enabling this will ensure there is only one active controller manager.
      --metrics-addr string      The address the metric endpoint binds to. (default ":8080")
      --version                  Print version to stdout and exit
pflag: help requested

vs
 richard   master  ~  projects  improbable-eng  etcd-cluster-operator  2  ./bin/manager -h
Usage of ./bin/manager:
  -backup-tmp-dir string
    	The directory to temporarily place backups before they are uploaded to their destination. (default "/tmp")
  -enable-leader-election
    	Enable leader election for controller manager. Enabling this will ensure there is only one active controller manager.
  -kubeconfig string
    	Paths to a kubeconfig. Only required if out-of-cluster.
  -master --kubeconfig
    	(Deprecated: switch to --kubeconfig) The address of the Kubernetes API server. Overrides any value in kubeconfig. Only required if out-of-cluster.
  -metrics-addr string
    	The address the metric endpoint binds to. (default ":8080")
  -version
    	Print version to stdout and exit

This makes it so that only the --option is supported, which I like.
The original version silently accepted both the -option and the --option variants...which I find confusing.
richard   master  ~  projects  improbable-eng  etcd-cluster-operator  1  ./bin/manager --help
Usage of ./bin/manager:
  -backup-tmp-dir string
    	The directory to temporarily place backups before they are uploaded to their destination. (default "/tmp")
  -enable-leader-election
    	Enable leader election for controller manager. Enabling this will ensure there is only one active controller manager.
  -kubeconfig string
    	Paths to a kubeconfig. Only required if out-of-cluster.
  -master --kubeconfig
    	(Deprecated: switch to --kubeconfig) The address of the Kubernetes API server. Overrides any value in kubeconfig. Only required if out-of-cluster.
  -metrics-addr string
    	The address the metric endpoint binds to. (default ":8080")
  -version
    	Print version to stdout and exit
 richard   master  ~  projects  improbable-eng  etcd-cluster-operator  2  ./bin/manager --enable-leader-election
2020-02-12T17:35:36.132Z	INFO	setup	Starting manager	{"version": "v0.1.0-39-gb6cbfaa"}
2020-02-12T17:35:36.540Z	ERROR	setup	unable to start manager	{"error": "unable to find leader election namespace: not running in-cluster, please specify LeaderElectionNamespace"}
github.com/go-logr/zapr.(*zapLogger).Error
	/home/richard/go/pkg/mod/github.com/go-logr/zapr@v0.1.1/zapr.go:128
main.main
	/home/richard/projects/improbable-eng/etcd-cluster-operator/main.go:64
runtime.main
	/home/richard/sdk/go1.13/src/runtime/proc.go:203


EQ 13 New Relic integration

What
This change install the New Relic agent programmatically. We need to do it this way as we're deploying into Elastic Beanstalk.
How to test

Create a new relic account
Follow the instructions on how to generate a newrelic.ini file - New Relic
Move the file to the top level directory of this repository (or point the EQ_NEW_RELIC_CONFIG_FILE to the location of that file).
Run the server
Browse to the health check
Check new relic is receiving information.

Who can review
Anyone apart from @warrenmethods

LGTM - Do you think we'll ever add the newrelic.ini file to source control? If so is it worth adding it to gitignore, but including it in ebignore - so that a local ini file is deployed but never added to git?

The newrelic.ini file contains the license key, so it shouldn't be in GIT. I can add it to the .gitignore. I think we'll have to manually copy the file to the Jenkins box for deployment.

That works for me - just need to make sure we add it to .ebignore with a ! so that it is included when you run eb deploy

@warrenbailey @dhilton could we put this key/value into VAULT later?

There is more than just a K/V in the settings file - we possibly could extract the license ID / key and just retain the settings - would need to look if the newrelic python lib supports environment variables.


[Question]: Does "COMMIT" with "N" mean to write to /local?

Is your question not already answered in the FAQ?

[X] I made sure the question is not listed in the FAQ.

Is this a general question and not a technical issue?

[X] I am sure my question is not about a technical issue.

Question
Currently COMMIT == N (default) will write to /storage.
Is this logic reverted?

Got it. COMMIT might need a better name. :-)
BTW, STORAGE itself is an env and can be pointed to other places
for docker commit.


Index shows in minimal_package toc but not others

Apparently if the toc is empty, it shows the toc of the document itself. But I have figured out that index can now be included directly in a toc.


[1417] remove opted_in from provider

Context
All providers are now opted in
Changes proposed in this pull request
Remove opted_in column from the provider table
Guidance to review
rails db:migrate
Checklist

[ ] Make sure all information from the Trello card is in here
[ ] Attach to Trello card
[X] Rebased master
[X] Cleaned commit history
[X] Tested by running locally

 Post-transition clean-up of provider opt-in code


Example inputs

JOSS Review 2514
In multi_am_radius_2_rk_opt.m there is a missing fgets that causes an error when using the example input file given in the comments (or the empty line before the first separation line in the example input needs to be removed).
--- a/examples/multi_am_radius_2_rk_opt.m
+++ b/examples/multi_am_radius_2_rk_opt.m
@@ -157,6 +156,9 @@ function [s,k,p,coeffs_monomial]= read_data(read_fid,class)
 %
 % Read number of stage and order of the method

+    % Read white line
+    tline = fgets(read_fid);
+
     % Read separation line
     tline = fgets(read_fid);

There also a missing ; earlier in the file.
--- a/examples/multi_am_radius_2_rk_opt.m
+++ b/examples/multi_am_radius_2_rk_opt.m
@@ -64,7 +64,7 @@ for i_stab_poly = 1:number_stab_poly
         % The order of accuracy is set to one to re-use completely the
         % functions implemented in the RK-coeff-opt to impose just the
         % linear order conditions.
-        h_p = 1
+        h_p = 1;

         % Polynomial coefficients indicies
         poly_coeff_ind = 2:s;

It would be helpful to include the example input in the directory so a user can run the example without having to create the input file first.
For multi_stab_fun_2_rk_opt.m what is parsed by read_data didn't seem to match what is described in the comments and what is used in the multi_stab_fun_2_rk_opt function. I made the following changes
--- a/examples/multi_stab_fun_2_rk_opt.m
+++ b/examples/multi_stab_fun_2_rk_opt.m
@@ -29,7 +29,7 @@ function multi_rk = multi_stab_fun_2_rk_opt(input_file_name,class,objective,vara
 for i_stab_poly = 1:number_stab_poly

     % Read number of stage and order of accuracy of the RK scheme
-    [s,p,fp,d]= read_data(read_fid);
+    [s,p,fp,poly_coeff_val]= read_data(read_fid);

     % Set tall tree numbers (indices) and tall tree values that will be
     % used to enforce the stability coefficients. Indeed, when "s" is
@@ -52,14 +52,15 @@ for i_stab_poly = 1:number_stab_poly
     end

     % Polynomial coefficients indices
-    poly_coeff_ind = p+1:s;
-
-    % Polynomial coefficients values
-    poly_coeff_val = d(7+p+1:length(d));
+    if (fp > 0)
+        poly_coeff_ind = p+fp:s;
+    else
+        poly_coeff_ind = [];
+    end

     % Call to rk_opt
     rk = rk_opt(s,p,class,objective,'poly_coeff_ind',poly_coeff_ind,...
-                'poly_coeff_val',poly_coeff_val,varargin{:})
+                'poly_coeff_val',poly_coeff_val,varargin{:});

 end

@@ -105,8 +106,8 @@ end

 % =========================================================================

-function [s,p,fp,d]= read_data(read_fid)
-%function [s,p,fp]= read_data(read_fid)
+function [s,p,fp,poly_coeff]= read_data(read_fid)
+%function [s,p,fp,poly_coeff]= read_data(read_fid)
 %
 % Read number of stage and order of accuracy of the method

@@ -127,6 +128,9 @@ p = floor(d(2));
 % the order of accuracy of the scheme for nonlinear problems.
 fp = floor(d(3));

+% Polynomial coefficients values
+poly_coeff = d(4:length(d));
+
 end
 % =========================================================================

and ran the example with with the following input file:
stability poly
1

stages order free params. poly coeff

5      4     1            1 1 0.5  0.166666666666667 0.041666666666667 0.009615384615385

I'm sure it this is the intended usage but it seemed to work as expected. I think it would be helpful in include a sample input file for this example as well.
For both of the above examples it would also be helpful to include some comments about the expected output or maybe have an example output file for a user to compare against.
I also tried out the examples given in the opt_poly_bisect documentation as well as other setups from KA12. I've included the commands I ran below in case anyone else also wants to try reproducing plots from KA12. I'm not sure if it was intentional but I noticed the gap size is fixed at 20 when using the 'gap' option in spectrum.
%% Doc example
lam = spectrum('realaxis',500);
s = 10; p = 2;
[h,poly_coeff] = opt_poly_bisect(lam,s,p,'chebyshev')
plotstabreg_func(poly_coeff,[1])

%% KA12 - Fig 3 top
lam = spectrum('realaxis',6400);
s = 20; p = 4;
[h,poly_coeff] = opt_poly_bisect(lam,s,p,'chebyshev')
plotstabreg_func(poly_coeff,[1])

%% KA12 - Fig 3 bottom
lam = spectrum('realaxis',6400);
s = 20; p = 10;
[h,poly_coeff] = opt_poly_bisect(lam,s,p,'chebyshev')
plotstabreg_func(poly_coeff,[1],[-140 1 -10 10])

%% KA12 - Fig 4 left
lam = spectrum('imagaxis',3200);
s = 7; p = 1;
[h,poly_coeff] = opt_poly_bisect(lam,s,p,'rotated chebyshev')
plotstabreg_func(poly_coeff,[1])

%% KA12 - Fig 4 right
lam = spectrum('imagaxis',3200);
s = 7; p = 4;
[h,poly_coeff] = opt_poly_bisect(lam,s,p,'rotated chebyshev')
plotstabreg_func(poly_coeff,[1])

%% KA12 - Fig 6 left
lam = spectrum('disk',500);
s = 8; p = 3;
[h,poly_coeff] = opt_poly_bisect(lam,s,p,'monomial')
plotstabreg_func(poly_coeff,[1])

%% KA12 - Fig 6 right
lam = spectrum('disk',500);
s = 15; p = 4;
[h,poly_coeff] = opt_poly_bisect(lam,s,p,'monomial')
plotstabreg_func(poly_coeff,[1])

%% KA12 - Fig 8 (size of gap is hard coded to 20)
lam = spectrum('gap',500);
s = 6; p = 1;
[h,poly_coeff] = opt_poly_bisect(lam,s,p,'chebyshev')
plotstabreg_func(poly_coeff,[1],[-50 2 -10 10])

%% KA12 - Fig 10 top
lam_func = @(kappa) spectrum('rectangle',100,kappa,1)
[h,poly_coeff] = opt_poly_bisect(lam,10,1,'chebyshev','lam_func',lam_func)
plotstabreg_func(poly_coeff,[1])

%% KA12 - Fig 10 bottom
lam_func = @(kappa) spectrum('rectangle',100,kappa,10)
[h,poly_coeff] = opt_poly_bisect(lam,20,1,'chebyshev','lam_func',lam_func)

Regarding the examples directory: thanks for going over this so carefully and debugging this code!  In fact, I had forgotten about this code completely; it was written by someone who left the project long ago and hasn't been touched since.  My feeling is that these examples are needlessly complicated and the scripts are useful only in a rather specific situation, so I will remove them.  I think simple examples that just call rk_opt once (like those available in the documentation) are more helpful.
The list of inputs to reproduce the KA12 paper is great; I will add that to the project.


finish all processes spawned in the test suite

This PR fixes a test case to ensure all spawned processes are finished. In tests/test-rules.c a shell process is created which itself spawns the sleep-executable. The latter isn't killed after the test run which confuses some automated test suites an leads to timeout issues (see https://bugs.gentoo.org/836790).

It looks like this will fix https://github.com/VirusTotal/yara/issues/1667 too.


Is this still working?

Hi Matt,
Is your script still working? For some reason mine is no longer collecting any data?
http://mynestreport.com/
Just wondering, thanks!
Roch

Mine is working but I needed to update the Nest library #10 .  I've merged my PRs here if you'd like to try it: https://github.com/costerwi/Nest-Extended

Thanks, i'll try this week to update the script.


[New Port Request] webview

Library name: webview-webview
Library description: Tiny cross-platform webview library for C/C++/Golang. Uses WebKit (Gtk/Cocoa) and Edge (Windows)
Source repository URL: https://github.com/webview/webview
Anything else that is useful to know when adding (such as optional features the library may have that should be included):
Renamed form zserge-webview

We already have port zserge-webview, we can update the existing port if necessary.


electrum: 3.3.6 -> 3.3.7

Motivation for this change
https://github.com/spesmilo/electrum/blob/master/RELEASE-NOTES
Things done

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

[x] NixOS
[ ] macOS
[x] other Linux distributions

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

Does this work at all without https://github.com/NixOS/nixpkgs/pull/64118 ..?

Should be okay now, thanks for the heads up.


fix:  uneven indentation on karmor install

Fix the uneven indentation on karmor uninstall script as shown below

Hi,
Can I take up this issue? if nobody's working on it.
thank you

Hi, Can I take up this issue? if nobody's working on it. thank you

@EswaramoorthyS Thank you for your interest to working on this and yes you can take this up.


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_4blec0qjtq/clone/recipe/meta.yaml:24: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_4blec0qjtq/clone/recipe/meta.yaml:25: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_4blec0qjtq/clone/recipe/meta.yaml:27: missing_wheel: For pypi packages, wheel should be present in the host section
WARNING: /tmp/abs_4blec0qjtq/clone/recipe/meta.yaml:32: python_build_tool_in_run: The python build tool setuptools is in run depends
ERROR: /tmp/abs_4blec0qjtq/clone/recipe/meta.yaml:39: missing_pip_check: For pypi packages, pip check should be present in the test commands
ERROR: /tmp/abs_4blec0qjtq/clone/recipe/meta.yaml:49: http_url: http://github.com/chimpler/pyhocon/ is not https
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_9cuhy86etd/clone/recipe/meta.yaml:24: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_9cuhy86etd/clone/recipe/meta.yaml:25: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_9cuhy86etd/clone/recipe/meta.yaml:27: missing_wheel: For pypi packages, wheel should be present in the host section
WARNING: /tmp/abs_9cuhy86etd/clone/recipe/meta.yaml:32: python_build_tool_in_run: The python build tool setuptools is in run depends
ERROR: /tmp/abs_9cuhy86etd/clone/recipe/meta.yaml:39: missing_pip_check: For pypi packages, pip check should be present in the test commands
ERROR: /tmp/abs_9cuhy86etd/clone/recipe/meta.yaml:49: http_url: http://github.com/chimpler/pyhocon/ is not https
Errors were found


Make ResultDisplay messages more user-friendly

[x] Edit all messages to fit within 3 lines

[x] job commands
[x] person commands
[x] general commands

[x] For messages that cannot fit (like the email one), enable scrolling
[x] Edit the language of the messages. Some were clearly not designed to be read by the user like "Expected ... string"
[x] Edit tests to pass with the new messages

Codecov Report

Merging #214 (93798b0) into master (dd50858) will decrease coverage by 1.32%.
The diff coverage is 56.86%.

@@             Coverage Diff              @@
##             master     #214      +/-   ##
============================================
- Coverage     65.09%   63.77%   -1.33%
+ Complexity      596      592       -4
============================================
  Files           111      112       +1
  Lines          2484     2490       +6
  Branches        262      262
============================================
- Hits           1617     1588      -29
- Misses          755      791      +36
+ Partials        112      111       -1

Impacted Files
Coverage Δ

...rc/main/java/peoplesoft/commons/core/Messages.java
0.00% <ø> (ø)

...n/java/peoplesoft/logic/commands/ClearCommand.java
100.00% <ø> (ø)

...in/java/peoplesoft/logic/commands/ExitCommand.java
100.00% <ø> (ø)

.../java/peoplesoft/logic/commands/ExportCommand.java
0.00% <0.00%> (ø)

...in/java/peoplesoft/logic/commands/HelpCommand.java
100.00% <ø> (ø)

...a/peoplesoft/logic/commands/job/JobAddCommand.java
41.66% <0.00%> (ø)

...eoplesoft/logic/commands/job/JobAssignCommand.java
15.38% <0.00%> (ø)

...esoft/logic/commands/person/PersonListCommand.java
100.00% <ø> (ø)

...a/peoplesoft/logic/parser/DeleteCommandParser.java
0.00% <0.00%> (ø)

...a/peoplesoft/logic/parser/ExportCommandParser.java
0.00% <0.00%> (ø)

... and 25 more

Continue to review full report at Codecov.

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


Allow Laravel v9 and Symfony v6

Update the package to allow it to be included in a Laravel v9 application that uses Symfony component v6.
Also, updated the CI tests to run on PHP 8.1
And bump shivammathur/setup-php to 2.16.0, since version 2.1.0 did not want to install PHP 7.3

Not sure if it's possible to have a Test App that is installable on both PHP 7.3 and PHP 8.1 with appropriate dependencies.

Would it help if we dropped support for Laravel 6?

I have updated the test app to a version (Laravel 8) that should be installable on everything from PHP 7.3 to PHP 8.1.
But it is actually one version of Laravel that is tests, I don't know if that is an issue?

What needs to be done to merge this PR? Would like to have Laravel 9 support 😄

The thing that fails now in CI is PHPCS can not run because of this:
Composer detected issues in your platform: Your Composer dependencies require a PHP version ">= 8.0.2". You are running 7.4.12.

I'm not sure if I upgrade some dependencies that do not need an upgrade, or if phpcs needs to be updated.

Alright, I believe the issue is regarding the composer.json since Laravel 9 has dropped support for all PHP versions below PHP 8. So to fix the issue, we must update the PHP version in composer.json. Laravel version 6 also supports PHP 8 so Laravel version 6 can still be supported, I suppose with the following change: "php": "^8.0",

Can we get this merged soon?

Can we get this merged soon?

@mnapoli is there something we are waiting for before we can merge?

It's possible to merge this one soon?

@omarkdev please calm down, this is becoming rude.

Thanks @hmazter!


Add id validators for client generated ids.

Add id validators that can check for existence and format of a client generated id, as per the JSON-API spec for client generated ids.

Moving this to 0.7.0 as this feature is required for current work.

Closing this as the package is deprecated. The Laravel package has an incomplete test for client generated ids, so that will remind me to implement it there.


Persist prometheus storage

All the prometheus metrics are currently wiped when restarting commit boost. We should have the option to persist those to disk

Retention of metrics is usually handled by prometheus itself and commit-boost should probably start with no context/be stateless.
The usual way is to write prometheus queries around this to handle restarts, for example: if you want total queries done to relayers in the last 3 days and you only have a counter that is initialized at 0 at startup and commit-boost restarts 5 times, it's totally fine, you can do a sum(metric...) in prom ; in general better be stupid around metrics and work around them using promql.
This approach is handy especially if you work at scale: chances are you'll have N replicas of commit-boost that can all receive queries  (to have HA, be able to upgrade commit-boost without missing blocks, etc), so each commit-boost is independent and has its own metrics and via PromQL you can still craft queries to have the global view.


Patch for lowering

This change removes global initialization of lower_extensions
with one functions and replaces with a list of functions.
This change made in commit 947b407 by @reazulhoque.
@AlexanderKalistratov have thoughts to remove this code because with TargetDispatcher it is not effective.

Need to redesign:

Make lowering.lower_extensions not a global variable but a part of context.
Until this redesign it could be a patch in our conda-recipe.


Remove the badge from the $GITHUB_STEP_SUMMARY

I don't think having the badge setup in each and every summary makes sense: it's something the user needs once and then never more. I'd suggest moving it to the README of this Action https://<url>/<owner>/<repo>/etc and / or moving it to the Actions output to still help users that want to get it.

The user needs the badge only once per workflow, that is true. But I think every other location to have this is unintuitive. Also the user has to piece together the URL then, because it is not generated automatically.
Since it is the last line also I think it does not really obfuscate any info or hinder readability.

I use summaries a lot and this clutters that up with unneeded info (after adding the badge to the repo's README). Would it be an idea to add an input for this that defaults to true, but when false that badge info is not rendered at all?

If we make it as an optional functionality we could go even one step further and also allow disabling the graph and disabling the total metrics.
So effectively a step with three input variables that all default to true but can be turned off.
I see people also finding the graph maybe not too crazy helpful after a while or when inspecting an extremly long run.
I can see this feature making more sense now. Thanks for the input!
If you want to provide a PR with the possible flags for "get-measurement" and "display-results" we are happy to take it. Otherwise feel free to leave the issue and i will put a dev on it likely end of next week.
I see the options as follows:

get-measurement

display-stats: True
send-data: True

display-results

display-stats: True
display-graph: True
send-data: True
display-badge: True

@rajbos
I have updated the action to take a few more optional flags:

display-table
display-graphs
display-badge

Each of them are true by default, and meant to be used in the final `display-results' step and hides the corresponding output from being displayed in the github_step_summary. Hope this helps!


Arrows elements are polluted

Problem
When using arrows (arrows: true), elements used in nextArrow and prevArrow are polluted. Their className is changed entirely. Used Chrome DevTools to check the class list.
Expected behaviour
Either:
a) embed the passed element in some div or
b) make the library append required classes (slick-arrow, slick-next) instead overwriting whole class list (the same would be with other fields possibly(?)).

Workaround
Embed the passed element in dummy div. By the way, React Fragment won't do ('<></>`).
Tags: [Not critical], [Quality of Life]

Also, there are warnings/errors from React while using custom arrows: See https://github.com/akiran/react-slick/issues/1238#issuecomment-872816250


react-hot-loader disable-warnings

As I checked the code, there is no possibility to disable react-hot-loader warnings.
What Next.js team does? Ignore them? Thank you.
https://github.com/gaearon/react-hot-loader#disable-warnings

Duplicate of #3109


Add importance in  XGBoost report

As far as I can tell this is not straightforward, as the feature importances do not appear accessible through the  XGBoost.jl wrapper MLJ uses.

Yes, fortunatley I am wrong: https://github.com/dmlc/XGBoost.jl/issues/81#issuecomment-573548332


Pen tablet stylus cannot perform left drag input in editor

Tested versions

Reproducible in 4.3 dev 1
Reproducible in 4.3 dev 2
Reproducible in 4.3 dev 3
Reproducible in 4.3 dev 4
Reproducible in 4.3 dev 5
Reproducible in 4.3 dev 6
Reproducible in 4.3 beta 1
Reproducible in 4.3 beta 2
Not reproducible in 4.2.2

Probably regressed by #92424
System information
Godot v4.3.beta2 - Windows 10.0.19045
Issue description
When using a pen tablet (in my case a Wacom Intuos Pen and Touch), the tip no longer works for dragging.
Steps to reproduce
In the editor, try to drag anything, e.g. reorder nodes in the scene tree or select text in the script editor.
Minimal reproduction project (MRP)
N/A
CC @bruvzg

Oops sorry, I just realized that #92424 was merged after beta 1, so it is actually not the regressor. I can reproduce it as early as 4.3 dev 1, may need to do a bisect.


[FEATURE] allow custom amount ZAP

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Currently, it only allow zap 500 1000 1500.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Allow custom amount ZAP, or add smaller denomination 50, 5
Bounty (in Bitcoin sats) offered for the implementation
Incentivize developers to work on your feature. Describe clear milestones to claim payment.
Additional context
Add any other context, video, or screenshots about the feature request here.

You can click and hold in the Zap button to change amounts :)

It might be nice to add a button to the zap popup for zapping that amount one time without adding it to the list, WDYT?

Great idea, @maxmoney21m. That will definitely help people being aware of it.

added on https://github.com/vitorpamplona/amethyst/pull/319


Can the ccache build instructions be simplified?

Description
https://reactnative.dev/docs/build-speed has detailed instructions for enabling ccace for xcode, I think this can be simplified
What is the problem?
The current instructions indicate the following code should be added to the Podfile

installer.pods_project.targets.each do |target|
      target.build_configurations.each do |config|
        # Using the un-qualified names means you can swap in different implementations, for example ccache
        config.build_settings["CC"] = "clang"
        config.build_settings["LD"] = "clang"
        config.build_settings["CXX"] = "clang++"
        config.build_settings["LDPLUSPLUS"] = "clang++"
      end
    end

How can we address it?
I think this can be replace by uncommenting the :ccache_enabled argument to  react_native_post_install
e.g.
    react_native_post_install(
      installer,
      config[:reactNativePath],
      :mac_catalyst_enabled => false,
      :ccache_enabled => true # this is normally commented out
    )

Why is it important?
RN now supports caching and from a quick look at the code does a better job at it. A pod update even recommends enabling ccache_enabled. So the docs aren't in line with the tooling.
Who needs this?
ios developers that want build peed improvements
When should this happen (use version numbers if needed)?
I would suggest updating the docs for all versions of RN that support the ccache_enabled flag
Other
I'd be happy to create a PR for the changes if the above is correct.

I'll defer to @cipolleschi here

Hi @johnf, could you open a PR with the change?

@cipolleschi Happy to!
Quick question about versioning. This change was implemented in 0.74. Does that mean I need to create versions in website/version_docs using the existing /docs for 0.72 and 0.73 and then update the various config files to support more versions?

@johnf If the change has been introduced in 0.74, I think it's fine to just update website/version_docs for 0.74, 0.75 and the docs ones. We can leave the older versions as they are.


fix: move from deprecated LFGBrowseFrame to LFGListFrame

related issues:

#181
#190

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

#27  👈
#26
#25
#24
#23
#22
#21
#20
#5
#19
#2 : 1 other dependent PR (#3 )
working-trunk

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

Join @juemrami and the rest of your teammates on  Graphite

Merge activity

May 18, 6:50 PM EDT: @juemrami started a stack merge that includes this pull request via Graphite.


Can SEW ever be set greater than ELEN

Can you confirm/clarify that if SEW is set to be greater than ELEN that vtype,vill be set. I think this is implied but it would be clearer to be more explicit about this.

ELEN is by definition the maximum size of a single vector element in bits (i.e. maximum possible SEW). Then, section 6.1 says:
If the vtype setting is not supported by the implementation, then the vill bit is set in vtype, the remaining bits in vtype are set to zero, and the vl register is also set to zero.
An attempt to set SEW greater than ELEN falls into this category, because it is not supported by the implementation.

I believe the spec is clear here so closing this issue.


Feature/shadcn

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


lookup columns, checkboxes, formulae?

Thanks for creating this, and all you did with dataview!
I'm not a notion user, but have extensively used coda.  Three of the best features there are lookup columns which enable the creation of on the fly relational databases (and dramatically simplify maintaining lists of field values), checkboxes for interacting with boolean fields, and simple formulae, which allow for things like logging changes in boolean fields on the fly...  I know this can sorta be done now in obsidian by stitching together code and plugins, but it gets awkward and laggy fast...
I looked at your (impressive) roadmap and could't tell if you were thinking along these lines. +1 if you are...  Would love to ditch coda and consolidate on obsidian for tasks and the more complex relational stuff that I do...

Checkboxes and formula will be present (as custom columns). This also implicitly means lookup columns will be available (since you can run subqueries inside of a query). I'm not super confident in the performance of lots of lookup columns in big tables but I've generally found rendering takes 90%+ of total query time so perhaps it may just work great!

awesome.
on the performance side, any chance you will be implementing a manual
update option, or some sort of cache for formula results?  e.g. lets say
all my notes have a type field that has in total maybe 20 values and I want
to be able to select a value and filter a table on it.  Ideally I don't
burn performance by constantly updating the select list when the search
requires the whole vault, but the values update infrequently.  I've run
into versions of this problem with dataview which can bring obsidian to its
knees- e.g. 200 notes sorted by group in dataview js can bog down when
viewing and get completely stuck if updating a value in the table.  I
imagine that if there was some way to limit the refresh these problems
would go away.  but maybe this is a limitation in my understanding of
javascript?
thanks for building these plugins. they are great.
On Fri, Sep 15, 2023 at 1:18 AM blacksmithgu @.***>
wrote:

Checkboxes and formula will be present (as custom columns). This also
implicitly means lookup columns will be available (since you can run
subqueries inside of a query). I'm not super confident in the performance
of lots of lookup columns in big tables but I've generally found rendering
takes 90%+ of total query time so perhaps it may just work great!
—
Reply to this email directly, view it on GitHub
https://github.com/blacksmithgu/datacore/issues/11#issuecomment-1720870147,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACPOSM52X6PLCGQJ66ETVKLX2QFMRANCNFSM6AAAAAAZJPO2MA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Datacore does paging by default which limits the number of visible results, which is a big performance bump. Currently refreshing is controlled via a delay (the higher delay means the longer the plugin will wait before refreshing; negative delay means do not refresh). Views only update if a relevant page in the search has actually changed (using some diffing logic that was not present in dataview).
My hope is to avoid needing "manual refresh" by making the default refresh functionality sufficiently fast.


JpgEncoder: options

Ability to choose options (quality and monochromaticism) when initializing video streaming added for JS and Interpretation versions.

Current coverage is 37.58%

Merging #1704 into master will increase coverage by +11.36% as of 8ccd5b8

@@            master   #1704   diff @@
======================================
  Files         1872     558    -1314
  Stmts        79017   24453   -54564
  Branches         0       0
  Methods          0       0
======================================
- Hit          20725    9191   -11534
  Partial          0       0
+ Missed       58292   15262   -43030

Review entire Coverage Diff as of 8ccd5b8

Powered by Codecov. Updated on successful CI builds.

осталось только помержить с мастером


Comments are not aligned with the post details

Essentially, the comments are not currently aligned with the post details. They are not wide enough, so it does not look very unified.
Expected Behavior
What should happen is, the comment replies for the post should be the same width as the post itself
Current Behavior
Currently, they have their own size and it is not relative to the post details
Possible Solution
Tweak the CSS or Containers in the code
Steps to Reproduce
N/A
Context (Environment)
The current problem is that it doesn't look unified. The comments look like they are separate from the post details

Detailed Description
N/A
Possible Implementation
N/A

I approve


Unable to install devle using brew on OSX

I tried to use brew to install delve following the instructions, but see "not found" error:
~$ brew install -v go-delve/delve/delve
Updating Homebrew...
Error: No available formula with the name "go-delve/delve/delve"
==> Searching for similarly named formulae...
Error: No similarly named formulae found.
==> Searching taps...
Error: No formulae found in taps.

Is it still eligible to install delve using brew now?

dont use brew, production branch version isnt compatible with the OSX sierra (aka macOS)
you will have to follow instruction on this issue ticket to get it working.
https://github.com/derekparker/delve/issues/645

So why you cant modify your documentation for this? I spend about 2 days because documentantion.

@freezlite: the better question would be: what is missing to get it into the production branch and how can I help you doing this?
Also you could contribute to the docs if you are not satisfied with them.

Should be resolved: https://github.com/derekparker/delve/issues/702.


[BUG] Wrong "Created" date for templates

Describe the bug
In the templates search menu, all templates are shown as "Created:" with today's date.

not sure if swate or swobup bug @Zerskk

This is an issue we have with database persistancy. Due to our service-on-demand approach the database can be build/rebuilt with ease but it is difficult for us to persist information between these database states.
You might also notice that all "Times used" counter where reset to 0.
Me and Marcel are already discussing solutions to this issue, but none of them are very clean. So this was kinda pushed to the backlog.

Will add this as Swobup issues and close this here.