text:
Update zio-sbt-ci, zio-sbt-ecosystem, ... to 0.4.0-alpha.19

About this PR
📦 Updates

dev.zio:zio-sbt-ci
dev.zio:zio-sbt-ecosystem
dev.zio:zio-sbt-website

from 0.4.0-alpha.6+40-c57e764c-SNAPSHOT to 0.4.0-alpha.19
📜 GitHub Release Notes
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "dev.zio" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "dev.zio" }
}]

labels: sbt-plugin-update, early-semver-pre-release, semver-spec-pre-release, commit-count:1

Superseded by #261.

repo: zio/zio-crypto


text:
Update sbt to 1.9.1

About this PR
📦 Updates org.scala-sbt:sbt from 1.9.0 to 1.9.1
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-sbt", artifactId = "sbt" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scala-sbt", artifactId = "sbt" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, version-scheme:early-semver, commit-count:1

Superseded by #986.

source: gharchive/pull-request

repo: zio/zio-json

license_type: permissive

license_source: github-api


text:
Update zio, zio-test, zio-test-sbt to 2.0.0-RC6

Updates

dev.zio:zio
dev.zio:zio-test
dev.zio:zio-test-sbt

from 2.0.0-RC5 to 2.0.0-RC6.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "dev.zio" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequest = { frequency = "@monthly" },
  dependency = { groupId = "dev.zio" }
}]

labels: library-update, early-semver-pre-release, semver-spec-pre-release, commit-count:1

Superseded by #129.

repo: zio/zio-morphir


text:
Add infrastructure for testing against Oracle

We will soon need to run tests against Oracle. To prepare for this, we can modify the CircleCI continuous build to install Oracle (on supported platforms such as Windows, Linux, or Solaris), and possibly add a helper script(s) to install Oracle locally so developers who run locally tests can have a local install of the database for testing purposes.

[x] Have helper SBT command that can install Oracle locally on developer machines
[ ] Modify CircleCI build to install Oracle

This ticket will be complete when a small ZIO Test is added into a oracle project that verifies a local Oracle database can be connected to. This will be enough functionality to begin developing test suites against Oracle.
Note: Oracle cannot be installed on Mac outside a VM.

We're switching to Testcontainers. This should be implemented analogically to how PostgresModuleTest is done.

Closing this. At this point it duplicates https://github.com/zio/zio-sql/issues/70

source: gharchive/issue

authors: jczuchnowski jdegoes

repo: zio/zio-sql


text:
Missing 'skip login success page' option for native app redirect settings

Preflight Checklist

[X] I could not find a solution in the documentation, the existing issues or discussions
[X] I have joined the ZITADEL chat

Version
1.2.0
ZITADEL Version
No response
Describe the problem caused by this bug
When creating native app, in redirect settings there is an option 'Skip Login Success Page', but it looks like it missed in tf provider.
To reproduce

Define zitadel_application_oidc resource in terraform
skip_login_success_page is missing

Screenshots

Expected behavior
No response
Relevant Configuration
No response
Additional Context
No response

Hi @dmytrum
Thanks for adding this.
I do not really consider this as a bug, as each attribute has currently to be added manually to the terraform provider when something changes in the api, i see it as an improvement.
I will put it to the small issues/bug fixing state.
Meanwhile, if you're interested in implementing it yourself, we also welcome pull requests.

@hifabienne here is another problem I discovered - as you can see 'Skip Login Success Page' is checked. When I adding with terraform 'Redirect URIs' - it return back to unchecked (default state)

authors: dmytrum hifabienne

repo: zitadel/terraform-provider-zitadel


text:
Toastmessages Change

As an user of ZITADEL console I want to see the errors as easy as possible, and they should not disappear after I while so If i missed it I don't know what happend.
Acceptance Criteria

[ ] Critical error messages do not disappear after a while (for example errors on forms (missing inputs, wrong format, etc..))
[ ] Critical errors disappear only when I close them or when I redo the action (e.g resend the form)
[ ] The error message is shown next to the input that is wrong

will close this issue at the moment, has never been mentioned by a customer

authors: hifabienne juergrinaldi

repo: zitadel/zitadel


text:
docs:change titles

Definition of Ready

[ ] I am happy with the code
[ ] Short description of the feature/issue is added in the pr description
[ ] PR is linked to the corresponding user story
[ ] Acceptance criteria are met
[ ] All open todos and follow ups are defined in a new ticket and justified
[ ] Deviations from the acceptance criteria and design are agreed with the PO and documented.
[ ] No debug or dead code
[ ] My code has no repetitions
[ ] Critical parts are tested automatically
[ ] Where possible E2E tests are implemented
[ ] Documentation/examples are up-to-date
[ ] All non-functional requirements are met
[ ] Functionality of the acceptance criteria is checked manually on the dev system.

I would only change the titles on the page, and not also of the sidebar.
This can be achivied by adding the "sidebar_label", and not only having the "title"

authors: dakshitha hifabienne


text:
Observable transits

This branch adds 3 new functions that work with Population objects:

find_observable_transits: creates a transit table (astropy table) for a given population of 1 object
find_all_observable_transits: creates a transit table for a given population of 'n' objects
make_airmass_plot: creates an airmass plot for a given transit table. Plots transit ingress,midpoint, and egress as well as civil ,nautical, and astronomical twilight

Awesome, I'm so excited about this @MiriCaradonna ! I'll take a look tomorrow!

@MiriCaradonna , I added a bunch of comments trying (at least a little bit; I realize I'm definitely not explaining everything carefully enough) to show the what/why of the changes I'm making to fold your lovely code into exoatlas. Please feel free to hit Resolve conversation on these comments once you've read through them! (It should make those comments disappear, just like resolving a comment in a Google Doc.)

@MiriCaradonna , as discussed above, I'm trying not to include notebook outputs directly inside the github repository, but here is an example of what our newly added planning.ipynb documentation notebook looks like when I try to run it on my computer. I see a couple of things in here:

For what I think should be valid inputs to find_observable_transits, I get an empty table. This might just be me not exactly understanding what I should expect as the output. Is looking for 1 transit from this 1 planet and finding nothing saying that the single next transit that will happen isn't observable from our site, or is this saying that this planet would never be observable from this site? Is it effectively saying "the transit that happens at time t = t_0 + P*n (where t_0 is the transit midpoint and P is the period) isn't observable for n=1?" or is it saying "it's not observable for any value of n?"
For putting in a population of multiple planets, find_all_observable_transits throws an error about n_transits being an int that isn't subscriptable. Do you get the same behavior on your computer? Maybe this is just a problem with the versions of things I have installed?

@MiriCaradonna , please:

Pull down the changes that I've made onto your computer, which might look something like

git checkout observable_transits
git pull

from the command line. You should see your transits.py move from its previous location in the main exoatlas directory into the new one of exoatlas/observing/transits.py, and you should also see a docs/planning.ipynb appear.
2. Edit planning.ipynb with an example of your own that runs on your computer. If you need to change things inside your functions, change them inside exoatlas/observing/transits.py. As long as you're using the Developer Installation, the changes you make to the files should be reflected every time you restart the kernel with which you run planning.ipynb.
3. Clear the outputs of planning.ipynb, save it, commit the changes, and push it back up to this branch!

Hi Zach, thank you for the comments!! As to your questions about the empty table, I believe that means that the transit that happens at 'x' time isn't observable for n=1 next transits, given the constraints (I believe the default constraint is the transit ingress has to occur after civil twilight). Perhaps try entering in 'n=30' or something to that order, and see if any of those transits are observable. I hope that helps! I'll be sure to add that when I fill out the planning.ipynb.

Hmmm that's odd that it says that n_transits isn't subscriptable as an int. Did you enter it as 'n_transits=some integer number'?

authors: MiriCaradonna zkbt

repo: zkbt/exoatlas

license: MIT


text:
allow numerical operators in filters for numerical variables

It would be desirable to filter numerical variables like sample size using numerical operators. e.g., a filter to plot only studies with sample size >30

I resolved this issue using sliders instead of inequalities. So now if you click on a column filter text box, it should bring up a slider so the user can select a range. However, this does create a slightly annoying bug where the first time you select a filter box, it resets the position of the table on the page. I'll open a separate issue to fix this later on, but the main functionality should work as expected afterwards.


text:
infcover.c: suppress false positive from clang-11 --analyze

Works around #593 until https://bugs.llvm.org/show_bug.cgi?id=46314 is fixed.

Can you put your last GH comment along with the link as a comment in the code?

The static analyzer warning isn't altogether wrong.
Checking the return value of realloc against NULL may be incorrect when reallocating to size zero
on platforms that return NULL from malloc(0).
(For instance, System V and QNX 4.0 may have behaved this way,
https://stackoverflow.com/questions/1986538/how-to-handle-realloc-when-it-fails-due-to-memory
mentions that very old Solaris behaved this way, and
NetBSD 5.0 and QNX Neutrino may behave this way if MALLOC_OPTIONS
contains 'V'.)
See also "MEM04-C. Beware of zero-length allocations" at CMU and elsewhere.

Thanks

authors: Dead2 dankegel nmoinvaz

repo: zlib-ng/zlib-ng

license: Zlib


text:
Does not work when the canvas is not in Screen Space - Overlay

This issue is due to how you are testing for the mouse position.
Here is the modification from my variant that corrected the problem
private static bool GetLocalMouse(RectTransform rt, out Vector2 result)         {             RectTransformUtility.ScreenPointToLocalPointInRectangle(rt, Input.mousePosition, Camera.main, out mousePosition);             result.x = Mathf.Clamp(mousePosition.x, rt.rect.min.x, rt.rect.max.x);             result.y = Mathf.Clamp(mousePosition.y, rt.rect.min.y, rt.rect.max.y);             Debug.Log(rt.rect.Contains(mousePosition));             return rt.rect.Contains(mousePosition);         }
Note I have also changed most of the code at this point to remove any local declrations to variables within funcitons that can be called on an update loop such as the GetLocalMouse function hince the new private static mousePosition that use as opposed to var mp as it was in your code branch.

Thanks for your fix there. Unfortunately when i use ScreenPointToLocalPointInRectangle, the mouse input handling breaks for Canvases with Screen Space - Overlay Render Mode. Also the code breaks if no camera is tagged as MainCamera. We can either:
Detect the Render Mode for the parent canvas and the attached Render Camera / Event Camera.
Supply the camera to the picker script in some way.
Let the user supply the mouse positition.
More?

Yes my usual preference is to on start of the GO see if a camera was provided by the dev e.g. in a Camera field if not I set the field to main camera if that is null I go looking for the top most event camera.
As to handling events you could get around that by leveraging the event data already coming from the UI controls which represent the various inputs e.g. the UnityEngine.UI.Textures for the saturation and hue where in you would handle drag event on each repsectivly and where that was meaningful you would pass that change up to the parent color picker to process. The advantage here is that you spliting that logic out to each respective element thus its a bit easier to modify for the dev without impacting the picker its self the draw back is its yet anotehr behaviour script to maintain but considering at current its all in one thats not to bad.

Ah forgot to comment on use when in screen space overlay.
Personally I rarely if ever use that mode but to support it you could simply look up to the canvas your on and if its in that mode use a different function for handling the input than if its not ... thats just off the top of my head I'm sure there is a more elegant way but as I basicly never use overlay I dont know :)

Any chance of the complete edited code to work in World space? I'm having trouble getting your code snippet above to integrate with the existing code.

authors: LodenDarkStar theboyrossy zloedi

license: mit

license_source: bigquery


text:
Segementation fault when no target ports are provided (e.g. ICMP)

Describe the bug
When using a probe module which does not require port numbers, e.g. ICMP , a segmentation fault happens.
CLI Arguments
zmap -M icmp_echo_time -O json 8.8.8.8 -f "sent_timestamp_ts,sent_timestamp_us,timestamp_ts,timestamp_us,recv_timestamp_ts,recv_timestamp_us" > res.json
Cause
https://github.com/zmap/zmap/blob/50c3a2736c645972348c4604185f339ca2218ff8/src/zmap.c#L665-L669
Inside parse_ports, the strtok function modifies the string it tokenizes, and since line 668 provides a literal, the segmenation happens.
Suggested Fix
	if (args.target_ports_given) {
		parse_ports(args.target_ports_arg, zconf.ports);
	} else {
		char line[] = "0";
		parse_ports(line, zconf.ports);
//		parse_ports((char *) "0", zconf.ports);
	}

Example Target IP
8.8.8.8
Expected behavior
ICMP echo request probe is sent.
Environment:

OS: Ubuntu Server 18.04
Version: Github HEAD

Thanks (and apologies, that's a real 🤦🏽 on our part). We'll get fix merged in today.

repo: zmap/zmap


text:
modpython socket creation does not persist

I am attempting to open a telnet socket using the example socket code from https://wiki.znc.in/Modpython - the socket creates and connects ok, but immediately disconnects and shuts down without ever triggering OnReadLine().  OnReadData triggers, despite EnableReadLine() having been used prior.  If I put Write() statements immediately after Connect() the data is transmitted (and received) but the connection then closes.  OnSockError() sees nothing (assuming this is exposed in modpython).
How do I get the socket to persist?

Sorry, should have included the code.  The host in question is an eggdrop bot, FYI.
# telnettest.py
import znc

class telnettest(znc.Module):
    description = "Telnet Test Module"

    def OnRaw(self, rawline):
        self.PutStatus(rawline.s)
        return znc.CONTINUE

    def OnModCommand(self, cmd):
        #any command to start the test
        self.sock = self.CreateSocket(telconn, "localhost", 30200)
        return znc.CONTINUE

class telconn(znc.Socket):
    def Init(self, host, port):
        self.Connect(host,port,timeout=60)
        return znc.CONTINUE

    def OnReadLine(self, line):
        self.GetModule().PutStatus("OnReadLine: {0}".format(line))
        return znc.CONTINUE

    def OnConnected(self):
        self.GetModule().PutStatus("connected")
        self.GetModule().PutStatus(self.GetRemoteIP())
        return znc.CONTINUE

    def OnReadData(self, data):
        #self.GetModule().PutStatus("OnReadData: {0}".format(data))
        self.GetModule().PutStatus("OnReadData: {0}".format("".join(map(chr, data))))
        return znc.CONTINUE

    def OnDisconnected(self):
        self.GetModule().PutStatus("disconnected")
        return znc.CONTINUE

Output to *status is:
<*status> connected
<*status> 127.0.0.1
<*status> OnReadData: ÿû
<*status> Botname  (Eggdrop v1.8.4 (C) 1997 Robey Pointer (C) 2010-2018 Eggheads)
<*status> Please enter your nickname.
<*status> disconnected

Can you check znc -D?
ÿû� looks concerning. Does the module stay loaded after the disconnect?
My suspicion is that it can't parse the line as UTF-8 (the default encoding), and reports the error via closing it.

OnReadData triggers, despite EnableReadLine() having been used prior

Correct. OnReadData triggers for all data. OnReadLine is triggered for lines only, after parsing them from raw bytes with the specified encoding.

Can you check znc -D?

I think you're onto something - znc -D shows:
[2021-07-28 22:06:31.886761] python socket failed in OnReadLine: UnicodeDecodeError: 'utf-8' codec can't decode byte 0xff in position 0: invalid start byte\x0A

ÿû� looks concerning. Does the module stay loaded after the disconnect?

Yes, the module stays loaded.

Correct. OnReadData triggers for all data. OnReadLine is triggered for lines only, after parsing them from raw bytes with the specified encoding.

So possibly I'm never getting to that point as it's erroring and closing the socket when the decode fails.

Either don't use EnableReadLine(), so that it doesn't try to decode the bytes at all, or specify the encoding via SetEncoding(), which supports some fallback options between UTF-8 and something else (which I should document somewhere...)

Either don't use EnableReadLine(), so that it doesn't try to decode the bytes at all, or specify the encoding via SetEncoding(), which supports some fallback options between UTF-8 and something else (which I should document somewhere...)

Awesome, thanks.  Using SetEncoding('utf8') gets me past that point - will soldier on - thanks a lot!

authors: DarthGandalf arfNZ

repo: znc/znc


text:
Error loading the adapter

error: A hook (orm) failed to load!
error: TypeError: Cannot read property 'config' of undefined
at validateModelDef (/usr/lib/node_modules/sails/node_modules/sails-hook-orm/lib/validate-model-def.js:108:84)
at /usr/lib/node_modules/sails/node_modules/sails-hook-orm/lib/initialize.js:218:36
at arrayEach (/usr/lib/node_modules/sails/node_modules/lodash/index.js:1289:13)
at Function. (/usr/lib/node_modules/sails/node_modules/lodash/index.js:3345:13)
at Array.async.auto._normalizeModelDefs (/usr/lib/node_modules/sails/node_modules/sails-hook-orm/lib/initialize.js:216:11)
at listener (/usr/lib/node_modules/sails/node_modules/sails-hook-orm/node_modules/async/lib/async.js:605:42)
at /usr/lib/node_modules/sails/node_modules/sails-hook-orm/node_modules/async/lib/async.js:544:17
at _arrayEach (/usr/lib/node_modules/sails/node_modules/sails-hook-orm/node_modules/async/lib/async.js:85:13)
at Immediate.taskComplete (/usr/lib/node_modules/sails/node_modules/sails-hook-orm/node_modules/async/lib/async.js:543:13)
at processImmediate [as _immediateCallback] (timers.js:383:17) [TypeError: Cannot read property 'config' of undefined]
error: Could not load Sails.
error: Are you using the latest stable version?
just installing at the latest sailsjs version and using the example

Could it be an unrelated issue? Is Sails working properly with another Waterline adapter?

Yup. I have elastic search adapter and it's working fine in the same project

I need to see some code to try and debug. Can you post a gist or paste here in the issue?

Installed it on a new installation of SailsJS and it worked. Will close this issue, and thanks for the response . will check what could be the cause of the conflict and will update you if it has something to do with it. Many thanks

No worries. Happy to look into things if something is broken in sails-rest itself

authors: abdulhamid-alattar zohararad

repo: zohararad/sails-rest


text:
Automate deployment

Currently any changes require manual deployments to be run by the developer via these scripts, https://github.com/zooniverse/text-editor/blob/0316f4753724a1cd48357e7a95c8bc680778667e/package.json#L52-L53
This codebase requires the ability to autodeploy the changes to our static app targets (azure blob store / s3). We need to add CI and CD via similar work to https://github.com/zooniverse/anti-slavery-manuscripts/pull/323

This is done.

authors: camallen srallen

repo: zooniverse/alice


text:
Tests and filters

Понадобилось через плагин дать возможность отфильтровывать часть лишних трейсов по условию, и для этого поправить напрямую код.
Заодно докидал немножко тестов, и поправил кодостиль.
Если чо не так - пишите, поправлю :)

Просьба вынести новую логику в отдельный коммит. Сейчас она кажется в codestyle гдето.

Новая логика в коммите 524e5c21bcd2d2801c7f12e3dd16be86bfaa1453

@chorry Спасибо!

authors: chorry xtrime-ru

repo: zoonru/pyrospy


text:
make executable

should resolve https://github.com/ix-plugin-hub/iocage-plugin-index/issues/163 :smile:

I don't know what solution you used, I found git update-index --chmod=+x ./post_install.sh

By the way : Thank a lot !!!

I don't know what solution you used, I found git update-index --chmod=+x ./post_install.sh

I'm working from my (Linux) computer, not from the GitHub website
These are the steps I took:
First I made a fork of your repository.
Then, from my computer I ran the following commands
git clone https://github.com/tprelog/iocage-plugin-movienight.git
cd iocage-plugin-movienight
chmod +x post_install.sh
git commit -am "make executable"
git push

Finally I made pull-request from my fork back to you

By the way : Thank a lot !!!

No problem, I'm glad this helped :)

I don't know what solution you used, I found git update-index --chmod=+x ./post_install.sh

I'm working from my (Linux) computer, not from the GitHub website
GOt it, my problem is I'm working from an Windows computer, so i don't have any chmod.
These are the steps I took:
First I made a fork of your repository.
Then, from my computer I ran the following commands
git clone https://github.com/tprelog/iocage-plugin-movienight.git
cd iocage-plugin-movienight
chmod +x post_install.sh
git commit -am "make executable"
git push

Finally I made pull-request from my fork back to you
I definitely know the steps you went through. I mean I'm no sys-admin but I know about chmod.
I was just thinking that Git didn't carry the ...

By the way : Thank a lot !!!

No problem, I'm glad this helped :)

authors: tprelog zorglube

repo: zorglube/iocage-plugin-movienight


text:
Fix word splitting issue

By performing word splitting with ${=spec} instead of the previously
used flags, I got rid of some anoying error messages when not using
en_US.UTF-8 in LC_MESSAGES.
Fixes #419

I tried you fix on my computer where LC_MESSAGES is set to fr_FR.UTF-8 and I still have the #419 issue.

I tried you fix on my computer where LC_MESSAGES is set to fr_FR.UTF-8 and I still have the #419 issue.

I'm sorry to hear that :( It fixed it for me in the past, now I'm not using it anymore because this project got abandoned.

I tried you fix on my computer where LC_MESSAGES is set to fr_FR.UTF-8 and I still have the #419 issue.

I'm sorry to hear that :( It fixed it for me in the past, now I'm not using it anymore because this project got abandoned.

authors: dadav smumu

repo: zplug/zplug


text:
Use stricter eslint configuration

This solves #10. While working on this, I discovered the current eslint glob for file patterns was missing quite a few errors. That is fixed as well.
This uses my personal eslint config for js, mocha, and react. It is very strict. I understand if you wish to use something else; doing this also allowed me to test my rules on more code.

@rcoy-v can you resolve the merge conflicts on this?

fixed

authors: rcoy-v zpratt

repo: zpratt/react-tdd-guide


text:
Add option to run only on current files in the repo

Going back in time to check every single commit can be a slow process on big repositories. It'd be nice if we could add an option to check only the files that exist currently on the master branch. Maybe adding a --only-master-files option.
What do you think?

--depth=1 seems to do the trick.
It would be nice though to specify in the manual which values are accepted by --depth. I can't understand what --depth=0 does exactly

@nightspotlight I thought --depth=1 would check the last commit in the repo but I might be wrong here.
@zricethezav Can you clarify, please?

that is correct @camaya, --depth=1 will inspect the changes from the most recent commit. If your repo only has one commit then you would be checking all the current files in the repo but only for that case.

Today at work I've tried --depth=1, and in the report it showed results for more than just the most recent commit.

@zricethezav is there (will there be) an option to scan just one specific branch?

There is a key --branch=YOUR_BRANCH_NAME, goes all over the history.
--only-master-files would be a super-useful flag. As mentioned, going back in history is very nice but slow and might not be optimal. Secrets could have been already changed and rotated, not much value going back.
In this case, we want to know if current state of the repository, the latest (not the commit but all the latest files) have secrets. --only-master-files option should make GitLeaks going via the latest fileset to identify is there are any secrets right now right here in the repository.

This feature should have been there by default. Why would I want to fail my pipeline because of some historical secret checkin which does not exists anymore.

--files-at-commit=latest will do the trick

authors: Kamleshpa avishnyakov camaya nightspotlight zricethezav

repo: zricethezav/gitleaks


text:
[BUG] deleteVertex deletes all of it's neighbors from the inEdge Map, even when that neighbor still has Vertex's outgoing to it

Describe the bug
Mostly what the title says.
Here is a simple reproduction
const graph = new DirectedGraph();
graph.addVertex('A');
graph.addVertex('B');
graph.addVertex('C');

graph.addEdge('B', 'A');
graph.addEdge('C', 'A');

// 'Incoming to A should contain ['B','C']
console.log(graph.incomingEdgesOf('A').map((e) => e.src)); //  ['B','C']

// Now delete B, which has no direct link to C, only that C -> A.
graph.deleteVertex('B');

// Now if we do the same call to incoming edges for A we should get only ['C']
console.log(
    graph.incomingEdgesOf('A').map((e) => e.src), // []
);

// but it only shows an empty array, since we deleted all of `A's edges, not just the one to `B`.

// If we check C, it correctly shows A as an outgoing edge,
// even though A no longer has any knowledge of C linking to it.
console.log(graph.outgoingEdgesOf('C').map((e) => e.dest));

Unless I'm fundamentally misunderstanding how this should work, deleting B should not wipe out all incoming edges to A, only the edges pointing at B.
I tracked the issue here https://github.com/zrwusa/data-structure-typed/blob/main/src/data-structures/graph/directed-graph.ts#L264, which I believe should be deleting an edge inside A's set of edges, not all of A's edges.

Changing that line from this._inEdgeMap.delete(neighbor); to this.deleteEdgeSrcToDest(vertex, neighbor); appears to fix the issue.

repo: zrwusa/data-structure-typed


text:
Syntax highlighting Command Output Text

It would be incredibly helpful to have the regex selection of a text output in terminal be able to be highlighted.
In the case of an output log steam, having any 'error' or 'warning' keywords highlighted would be helpful.
Exmaple regex for finding 'error' in a commands output: 'error.*?([;\n$])'
Is this possible to achieve utilizing the code from this repo?
How can we achieve this?

No, zsh-syntax-highlighting only handles the zsh command line. Command output does not pass through zsh at all. I'd recommend seeing if your terminal emulator has such a feature.

For your specific use-case, just pipe to egrep --color '^|error'.  You might need to add stdbuf(1) on the LHS of the pipe.  For the general case, expect(1) might be a good place to start.  Feel free to ask further on zsh-users@.

authors: Erik-Koning danielshahaf phy1729

repo: zsh-users/zsh-syntax-highlighting


text:
Allow manually setting the folder name

Currently, the location of the config folder is determined by the Description property of the DebuggerVisualizerAttribute. The end-assembly passes the visualizer type into the config provider, which then reads all instances of DebuggerVisualizerAttribute, and chooses the single distinct description.
This forces end visualizers to use the same description across all uses of DebuggerVisualizerAttribute. If multiple uses have different descriptions, the code will fall, because it's using LINQ Distinct then Single to get a single value.
There may be other use cases where the folder should be set manually.
It would be enough to provide an overload of LoadConfigFolder that takes a string as the subfolder name.

Fixed in 9e4959af

repo: zspitz/Periscope


text:
你好，编译py_api的时候报错

CMake Error at CMakeLists.txt:50 (add_subdirectory):
The source directory
/home/demo/fastllm/third_party/pybind11

does not contain a CMakeLists.txt file.
CMake Error at CMakeLists.txt:60 (pybind11_extension):
Unknown CMake command "pybind11_extension".
请大佬readme里面说明下需要的python版本和必要库，感谢~
此外c++版本的ip端口不支持修改

需要加载pybind11，这个在根目录的readme里面没有写。
在根目录执行：
git submodule update --init --recursive

再执行cmake步骤。

authors: Be-carefull TylunasLi


text:
bug: Header 导航栏中，添加 Menu.item 至 user 登出菜单时，被 numberCard 遮挡

./src/components/Layout/Header.js 中添加如下代码：
<SubMenu style={{
            float: 'right',
          }} title={< span className={styles.button}> <Icon type="user" />
            {user.username} </span>}
          >
            <Menu.Item ic key="changePassword">
              <Icon type="key" />
              修改密码
            </Menu.Item>
            <Menu.Divider />
            <Menu.Item key="logout">
              <Icon type="poweroff" />
              Sign out
            </Menu.Item>
          </SubMenu>

这时由于 Menu 的属性  styles.rightWarpper 中 display: flex，导致 Sign out 子菜单被遮挡。去掉该属性后，修改密码和 sign out 子菜单都可以正常显示，但是与 mail 图标div 的排版又有问题。
google 发现这个问题好像是 antd-design 本身的 bug https://github.com/ant-design/ant-design/issues/2830 。
我目前的解决办法是把 mail div 去掉，把 rightWarpper 的 display 属性也去掉。

我是在Recharts发现的这个问题，菜单被Show More的按钮挡住了，我通过对Header设置z-index解决了问题，并且已经发了PR，你可以试一下

@kuanger 已采用，非常感谢

authors: jzoe kuanger

repo: zuiidea/antd-admin


text:
Show current organization's name and icon atop main menu

This is a followup to:

#1035

As the design in Figma shows, we'd like the main menu to identify the current organization, using its name and icon:

To do that, we'll need to track the realm name and realm icon in our data model. For this issue, though, we don't need it in the persistent database — so we get to skip the full complexity of #1036. Instead we just need to have the values on PerAccountStore, and keep them updated there.

Hi, I've started working on this issue and would like to claim it

Welcome @sand-storm-17, this is a follow-up after #1035 is resolved. Until then, I would recommend looking at some other issues to work on. Please take a look at our documentation on Picking an issue to work on to get started.

repo: zulip/zulip-flutter


text:
Fix channel navigation when tapping recipient header areas

This PR fixes the tap target behavior in the recipient header of channel messages. Previously, tapping above or below the channel name would navigate to the topic view instead of the channel view.
The fix adds behavior: HitTestBehavior.opaque to the GestureDetector which makes the entire area (including padding spaces) respond to taps.
The fix improves the header's usability by ensuring the entire channel section responds correctly to taps, making navigation more intuitive.
Issue: #1179
Screencast:

Thanks. Before we can review this, it will need a test — see our README: https://github.com/zulip/zulip-flutter#submitting-a-pull-request


text:
Uncaught Syntax Error

Issue Summary
Just got the below when using the mobile app on chat.zulip.org.
Thought it would be worth raising.
Issue Screenshot

Versions
Zulip Mobile Version: v13.4.88
OS: Android
Phone: Samsung Galaxy S7 Edge
Extra Information
Let me know if you need any more information, or to grab any logs if you can show me we're to find them.

Thanks @adambirds for the report! Folding this into #2505, which appears to be the same issue.

authors: adambirds gnprice

repo: zulip/zulip-mobile


text:
deps: Upgrade to RN v0.66!

Fixes: #5231

Here are the RN commits that touch the RN template app that are in v0.65.2 but are not in v0.66.4. They're ordered from first to last (oldest to newest).
I.e., they're the commits in the output of git log --oneline --reverse v0.64.2..v0.65.2 -- template, when I run it in a clone of the facebook/react-native repo.
Along with whether, how, and when we propagate the changes to our app.
We shouldn't forget to update this, if necessary, to follow any changes made during code review and merge.

RN commit
When resolved
Comment

facebook/react-native@6386f691f Deploy Flow v0.150.0
main commit
Flow 150 adds a format.bracket_spacing option; see facebook/flow@2d7614f38. The RN template app sets that to false, but I think we're fine to stick with its default, true.

facebook/react-native@7252798ab Deploy Flow v0.151.0 to xplat
main commit

facebook/react-native@726e68926 Deploy Flow v0.152.0
main commit

facebook/react-native@547b4c92e Gradle 6.9, Android Gradle Plugin 4.2.1 (#31593)
53ed19500
This was backported to RN v0.65 in facebook/react-native@7599593b3 and we processed it for that upgrade.

facebook/react-native@70da64094 remove jcenter (#31609)
1f80f4b6c
This was backported to RN v0.65 in facebook/react-native@9a923be89 and we processed it for that upgrade.

facebook/react-native@5d01110b5 bump buildToolsVersion to 30.0.2 (#31627)
N/A
This was backported to RN v0.65 in facebook/react-native@37e9f1d36 and we processed it for that upgrade.

facebook/react-native@b3a715f6e removal of textColor from base application theme (#31487)
4fc4e9681

facebook/react-native@80927791e Update _gitignore - Remove .hprof (#31679)
N/A
The commit doesn't help me understand why an .hprof file might appear, and I've never seen one appear. Let's wait till we see one, then maybe we can get a better idea of why it appears, and we can act on that.

facebook/react-native@06c33e9ab Bump Android deps to 0.93 (#31675)
7b4be5d33
This was backported to RN v0.65 in facebook/react-native@d8b115afa and we processed it for that upgrade.

facebook/react-native@b176617d1 Release v0.153.0 to xplat
main commit

facebook/react-native@aa43aab77 Bump NDK to 21.4.7075529 (#31731)
N/A
Instead of pinning explicitly, we defer to the Android Gradle Plugin's choice of NDK version; see https://chat.zulip.org/#narrow/stream/243-mobile-team/topic/Pin.20to.20explicit.20NDK.20version.3F/near/1132815

facebook/react-native@36318b677 Deploy v0.154.0 to xplat
main commit

facebook/react-native@ca440b910 Upgrade Metro to v0.66.1
this PR

facebook/react-native@d6ed1ff58 Allow configuring ndk build architectures (#31232)
N/A
We're skipping this for now; see https://chat.zulip.org/#narrow/stream/243-mobile-team/topic/RN.3A.20.22Allow.20configuring.20ndk.20build.20architectures.22/near/1377512

facebook/react-native@efd4dafc5 Update xplat to Flow v0.155.0
main commit

facebook/react-native@1e6fddbe0 Deploy Flow v0.155.1
main commit

facebook/react-native@cfdc4fed0 Upgrade metro to 0.66.2
this PR

facebook/react-native@a67799fc4 Deploy 0.156.0 to xplat
main commit

facebook/react-native@41f45a77a Bump Flipper to 0.99.0 (#31896)
this PR

facebook/react-native@d27288044 Handle OSS renderers in sync script
72583b13c
This was backported to RN v0.65 in facebook/react-native@5a1dc1b6b and we processed it for that upgrade.

facebook/react-native@d583c2f15 Fix Undefined symbol: _swift_FORCE_LOAD$_swiftFileProvider when building with Xcode 13
N/A
This gets cleanly reverted in facebook/react-native@ec083f914; ignore.

facebook/react-native@83b16292c Deploy 0.157.0 to xplat
main commit

facebook/react-native@ae494e7ce Bump AGP to 4.2.2
this PR

facebook/react-native@ec083f914 Back out "Fix Undefined symbol: _swift_FORCE_LOAD$_swiftFileProvider when building with Xcode 13"
N/A
This cleanly reverts facebook/react-native@d583c2f15; ignore.

facebook/react-native@eb9388630 Include Swift lib in LIBRARY_SEARCH_PATHS
N/A
The RN maintainers announced a simpler fix to the build failure this seems to be aimed at: just don't include the LIBRARY_SEARCH_PATHS section at all. With one requirement: take an easy CocoaPods version bump. We did all that in 80a86212f. See there, and also https://github.com/facebook/react-native/commit/eb9388630#commitcomment-58407458

facebook/react-native@b48c3d1d4 Fix irregular indent in template (#29871)
N/A
This tiny indentation error doesn't appear in our project.

facebook/react-native@06e31c748 fix AGP 7 compatibility (#32030)
N/A
This was backported to RN v0.65 in facebook/react-native@8b430e00d and we processed it for that upgrade.

facebook/react-native@b55201c53 Deploy 0.158.0 to xplat
main commit

facebook/react-native@ab829005b Move react-native-codegen to root
main commit

facebook/react-native@a0429ebb5 (tag: v0.66.0-rc.0) [0.66.0-rc.0] Bump version numbers
N/A

facebook/react-native@ea5109fc0 OSS: add Xcode 12.5 + M1 machines CocoaPods post_install workaround
main commit

facebook/react-native@038cdda23 Switch order of search libraries to fix M1 build error
N/A
This is downstream of facebook/react-native@eb9388630, which we decided not to follow; see there.

facebook/react-native@d552362f8 (tag: v0.66.0-rc.1) [0.66.0-rc.1] Bump version numbers
N/A

facebook/react-native@614a370d8 (tag: v0.66.0-rc.2) [0.66.0-rc.2] Bump version numbers
N/A

facebook/react-native@ab50c6ee2 (tag: v0.66.0-rc.3) [0.66.0-rc.3] Bump version numbers
N/A

facebook/react-native@d47fd4a32 (tag: v0.66.0-rc.4) [0.66.0-rc.4] Bump version numbers
N/A

facebook/react-native@09a21f0f3 (tag: v0.66.0) [0.66.0] Bump version numbers
N/A

facebook/react-native@d48ed4a4b (tag: v0.66.1) [0.66.1] Bump version numbers
N/A

facebook/react-native@6c19dc326 (tag: v0.66.2) [0.66.2] Bump version numbers
N/A

facebook/react-native@3b5e4463a (tag: v0.66.3) [0.66.3] Bump version numbers
N/A

facebook/react-native@1b31d6bb5 (HEAD, tag: v0.66.4, origin/0.66-stable) [0.66.4] Bump version numbers
main commit

Thanks for the review! Revision pushed.

Before merging, let's let a release with RN v0.65 go out to all users.

Before merging, let's let a release with RN v0.65 go out to all users.

(That way if there are issues that seem related to the upgrade, we have a head start on narrowing them down and don't have to wonder whether they were introduced by RN v0.65 or v0.66.)

Meanwhile, this branch all looks good! So once we have a release out to prod with RN v0.65, this will be ready for merge.

Great! Thanks for the review!

authors: chrisbobbe gnprice


text:
types: Switch to TsFlower for react-navigation types

(Marking this as draft only because it's stacked atop #5396.)
This replaces the type definitions we've had for react-navigation,
which derive from the flow-typed project with some changes and fixes
of our own, with new definitions based more directly on the
TypeScript type definitions found in the library itself upstream.

The Flow type definitions are first generated from the upstream TS
type definitions using TsFlower, a tool I recently wrote for this
purpose:
https://www.npmjs.com/package/tsflower
Then a stack of local patches is applied.  There are 24 of these,
including one added for react-native-safe-area-context in the
previous PR.  They comprise:

9 patches for things that could be fixed upstream, e.g. by adding
readonly.  These are places where the TS definitions are less
accurate than they could be, but TS apparently takes the
distinctions less seriously than Flow does.

8 patches for things that could be fixed in TsFlower.

5 "irreducible" patches, where we fundamentally need a local
patch like this.

3 patches are because TS doesn't have a way to express a
distinction that becomes important in Flow: exact object types,
or covariant type aliases.

2 patches are because the types use a TS feature that there's
no way to translate into Flow: conditional types.  As a result,
our patched-in versions are imperfect approximations, but I
think good enough.

1 patch that just reformats a few definitions for readability.
This was useful in developing some of these changes.

1 patch (the one at the end) that fixes an issue I haven't fully
debugged.

The best way to read these patches is probably to use the command:
$ tools/tsflower unpack -s

in order to unpack them into a Git branch, and then read that branch
in the usual way.  (The -s just makes the command faster, by not
rerunning TsFlower.)

Meanwhile, because these upstream-based type definitions are quite
different from the ones we had based on flow-typed, our own code
that interacts with react-navigation needs a number of small changes.
All of these changes are purely in the types, with no runtime
changes -- after all, we're using the same underlying library JS
code at runtime, just switching from one type-level description of
its API to a different one.
Many such changes are pulled out as prep commits, for changes that work already with the old definitions.  Others happen simultaneously with the change of type definitions, in the main commit.
Fixes: #5391

Thanks for the review! Replies above, and revision pushed.

And pushed another revision where the type definitions are auto-formatted -- atop #5399, with the patches rebased over the auto-formatting using the technique described there. (As discussed at https://github.com/zulip/zulip-mobile/pull/5396#issuecomment-1146491215 .)

Marking ready, as the preceding PRs are merged.

Thanks! I've just noticed that in some cases, the object type for a React component's props is inexact, meaning Flow lets you pass nonsense props like foo={123}. This is better than letting you pass a wrong type for an expected prop, but I wonder if it's something we should try to fix. I've noticed it on <Stack.Navigator … /> and on <NavigationContainer … /> so far.
For <Stack.Navigator … />, the following gives me the "property foo is missing" error I'd been hoping for:
diff --git src/__flow-tests__/nav-test.js src/__flow-tests__/nav-test.js
index ba7921277..b0f5f5ba4 100644
--- src/__flow-tests__/nav-test.js
+++ src/__flow-tests__/nav-test.js
@@ -61,7 +61,7 @@ function testNavigatorTypes() {

   const Stack = createStackNavigator<NavParamList, NavParamList, NavigationProp<>>();

-    <Stack.Navigator>
+    <Stack.Navigator foo={123}>
       {/* Happy case is happy */}
       <Stack.Screen name="Profile" component={Profile} />
       {/* FAIL - Should error but doesn't! on mismatch of name with route prop */}
diff --git types/@react-navigation/core/lib/typescript/src/types.js.flow types/@react-navigation/core/lib/typescript/src/types.js.flow
index b1232ee31..e5baf28c0 100644
--- types/@react-navigation/core/lib/typescript/src/types.js.flow
+++ types/@react-navigation/core/lib/typescript/src/types.js.flow
@@ -24,7 +24,7 @@ import {
 export type DefaultNavigatorOptions<
   ScreenOptions: { ... },
   ParamList: ParamListBase = ParamListBase,
-> = DefaultRouterOptions<string> & {
+> = DefaultRouterOptions<string> & {|
   children: $tsflower_subst$React$ReactNode,
   screenOptions?:
     | ScreenOptions
@@ -41,8 +41,7 @@ export type DefaultNavigatorOptions<
         options: ScreenOptions,
         ...
       }) => ScreenOptions),
-  ...
-};
+|};

 export type EventMapBase = {
   +[key: string]: {|
diff --git types/@react-navigation/routers/lib/typescript/src/types.js.flow types/@react-navigation/routers/lib/typescript/src/types.js.flow
index 780a9b08f..6ddd25ca6 100644
--- types/@react-navigation/routers/lib/typescript/src/types.js.flow
+++ types/@react-navigation/routers/lib/typescript/src/types.js.flow
@@ -84,10 +84,9 @@ export type ActionCreators<Action: NavigationAction> = {
   [key: string]: (...args: any) => Action,
   ...,
 };
-export type DefaultRouterOptions<RouteName: string = string> = {
+export type DefaultRouterOptions<RouteName: string = string> = {|
   initialRouteName?: RouteName,
-  ...
-};
+|};
 export type RouterFactory<
   State: NavigationState<>,
   Action: NavigationAction,

…but it also gives me 59 other errors.

I've just noticed that in some cases, the object type for a React component's props is inexact, meaning Flow lets you pass nonsense props like foo={123}.

Yeah, that's the TypeScript semantics.
I believe TS goes on to try to mitigate that by attempting some kind of warning or error when you pass extra properties, even though they don't recognize the exact/inexact distinction in the type system and so it's impossible to do that soundly. I'm not sure of the details of that.

This is better than letting you pass a wrong type for an expected prop, but I wonder if it's something we should try to fix. […]
For <Stack.Navigator … />, the following gives me the "property foo is missing" error I'd been hoping for:
[…]
…but it also gives me 59 other errors.

Yeah. You're now intersecting two exact object types that have different sets of properties, so there are no objects that belong to both sides of the intersection, meaning there are none that belong to that intersection type. (In particular, any such object must have a children property to satisfy the second member of the intersection; but by the first member, it can't have any properties other than initialRouteName.) Which isn't the intended result.
The Flow idiom here is to use spreads instead of intersections, and then those work nicely with exact as well as inexact object types. That does make the patch a bit more invasive, though.
I think it could be reasonable to try to add a patch that makes the props types exact. Because these props types are constructed from complicated expressions, the simplest way for that here might be to add an $Exact just inside the React.ComponentType in TypedNavigator, rather than to make all the object-type literals that go into it be exact and convert all the intersections to spreads.

That does make the patch a bit more invasive, though.

I'm excited to get more use out of TsFlower's "push button, receive types" service, and I think this is a fine place to let the output sit as it is. We can always add to the patch in the future if we end up getting burned by this lack of coverage.

Thanks for the review! Done.


text:
UI: Organization logo does not change with the color scheme

Hi,
I've just set my Display settings > Theme > Color scheme > sync with computer on my zulip client and noticed that the orgaganization logo (we have a day and night logo) does not swich back to the "day" logo when my client turns back to the light theme.
I noticed this happens both on the desktop client and the browser client.
It then updates properly once I refresh the page/reload the organization.

Happens to us too.

repo: zulip/zulip


text:
Scroll selected PM conversation into view

As reported on CZO:

When we narrow to a private message conversation, we aren't scrolling it into view in the left sidebar, like we do when you narrow to a stream. I think this is just an oversight -- previously they were pinned to the top, so this would have been a noop.

The main reproducer that feels bad here for me is that you hit the p keyboard shortcut to go to a private message conversation, and the conversation you just navigated to isn't in view. (Nor are others if you want to see what next one you might get to by hitting p again)

We should implement scrolling to a PM conversation when it's selected, as we do for stream conversations, and test on CZO.

@zulipbot claim

@zulipbot claim

@zulipbot claim

Hello, yes I am working on this issue. The discussion is happening on the CZO linked here.

@zulipbot claim

@zulipbot abandon


text:
In the Deactivated users panel, align filters better in narrow windows

In the Deactivated users table, when the filters don't fit next to the headings, we should:

Put them on the left below the heading.
Leave more horizontal space between the filters and the table. We might need less vertical space between the heading and the filters.

The same might apply to other settings panels (perhaps in other languages), but "Deactivated users" is a long heading, so it's easy to see the issue there.
We should probably wait on this PR to merge to avoid conflicts:

#30268

https://github.com/zulip/zulip/issues/29894 might be related and worth fixing along with this.

@zulipbot claim

@zulipbot claim

@zulipbot claim

@zulipbot claim

authors: NickeSteenen NotVishesh alya amenu4aie2 timabbott userAdityaa


text:
Zulip server crashing daily

Summary of issue

Users are unable to use Zulip
Restarting rabbitmq fixes the problem
Problem seems to occur daily now (or every two days)
Users have to login again (and receive a new login email)
All users appear offline
New information not present in the chat: Seems like our CPU is spiking to 100% when this issue occurs

Other info

Version: 1.6.0
Hosted on DigitalOcean: 2gb ram + 40gb disk
First deployed 2017-07-13

The following is copy pasted from chat.zulip.org https://chat.zulip.org/#narrow/stream/issues/topic/Zulip.20server.20crashing
Hey we've been running our Zulip server a a 2gb DO box without problems for about three weeks. We only have about 5 active users and lately we've been experiencing a "Woops something went wrong, please drop us a line."
I follow some of the steps on
http://zulip.readthedocs.io/en/latest/prod-troubleshooting.html
And restarting rabbitMQ via fixes the problem
service rabbitmq-server restart
However I'm finding that it's happening more frequently -- three days in a row now.
Can anyone possible point me towards some logs I can potentially look at or maybe something I can do to debug this problem next time it happens?

After a followup with @gnprice and investigating all the logs in /var/log/rabbitmq
Seeing this over and over again

Disabling disk free space monitoring on unsupported platform:

Here's an error in the logs
=ERROR REPORT==== 16-Aug-2017::14:04:08 ===
** Generic server rabbit_disk_monitor terminating
** Last message in was update
** When Server state == {state,"/var/lib/rabbitmq/mnesia/zulip@localhost",
50000000,37808316416,100,10000,
#Ref<0.0.1.20127>,false,true}
** Reason for termination ==
** {{unparseable,[]},
[{rabbit_disk_monitor,parse_free_unix,1,
[{file,"src/rabbit_disk_monitor.erl"},{line,204}]},
{rabbit_disk_monitor,internal_update,1,
[{file,"src/rabbit_disk_monitor.erl"},{line,173}]},
{rabbit_disk_monitor,handle_info,2,
[{file,"src/rabbit_disk_monitor.erl"},{line,145}]},
{gen_server,try_dispatch,4,[{file,"gen_server.erl"},{line,615}]},
{gen_server,handle_msg,5,[{file,"gen_server.erl"},{line,681}]},
{proc_lib,init_p_do_apply,3,[{file,"proc_lib.erl"},{line,240}]}]}
and here's the the paired down log

Here's a reduced version of the logs that has more events and errors: https://gist.github.com/uri/5563db5e1248b204bdaa60b744fff9c4

Not sure if these are related but after restarting rabbitMQ I have to login again and I get an email saying there has been a new login.
And another thing: all users appear offline even though they are not

It sounds like RabbitMQ is crashing, perhaps due to OOM kills (aka the server is running out of memory).  Can you check /var/log/syslog to see if the system is having OOM kills invoked?

This turned out to be a memory leak due to some buggy supervisord configuration.  Fixing now.  Huge thanks for the help debugging this @uri!

authors: timabbott uri


text:
Support more than just left-anchored (prefix) matches in stream search UI

Currently if i have a stream like fhir-infrastructure, I won't find it by typing q, infra -- because stream search anchors to the left. It would be nice to find internal terms, too.

This one is sorta interesting, in that doing a prefix match does make life better for cases like this.  I wonder if we're doing per-word prefix matching already but just not treating - as a word-break... if not, that's probably what we should be doing.

authors: jmandel timabbott


text:
tests: Use users in send_*_message.

This commit mostly makes our tests less
noisy, since emails are no longer an important
detail of sending messages (they're not even
really used in the API).
It also sets us up to have more scrutiny
on delivery_email/email in the future
for things that actually matter.  (This is
a prep commit for something along those
lines, kind of hard to explain the full
plan.)

Merged, and added 5835023021087650435b649677ea7345b82f3ce6 on top as something related I've wanted to do for a long time.
Thanks for doing this @showell!  I think the next thing on my list of structural test API changes is making the api_get bundle of functions take UserProfiles rather than emails.

authors: showell timabbott


text:
wip: replace Favicon.png generated files with favicon.svg.hbs

The positioning of the numbers will be fixed, but the intention of this wip is to demonstrate an implementation approach. If this is acceptable, the next step would be to make a switch case statement with 99 numbers and data paths.
Testing Plan:
GIFs or Screenshots:

I tried to address your feedback Anders, this is a simpler and better approach, thanks for the review.

Here's how it looks now, with the previous version to compare... I don't really have ideas for making it appear better, reducing the white stroke makes it very blury and difficult to see.

Fixed

I'm excited about getting rid of that giant pile of image files!

Do we still have the same plan of waiting one week before deleting the files?

I'm not sure if this is the right way to load the font since this is using the google api in the url, but I couldn't find a similar example in the project or docs

Is the choice to have this be black-on-blue intentional?  It looks a bit odd to me when testing this locally.

I’ve rewritten this as #16582 for improved visual design, better display for large counts, and an integrated PM dot (no more flashing).

authors: YashRE42 andersk timabbott


text:
help: Add gear menu icons to relative links.

As discussed in , https://github.com/zulip/zulip/pull/26313#issuecomment-1646130071 this PR adds icons to the following relative help links:
{relative|gear|manage-streams}
{relative|gear|settings}
{relative|gear|organization-settings}
{relative|gear|stats}
{relative|gear|keyboard-shortcuts}
{relative|gear|message-formatting}
{relative|gear|search-filters}
{relative|gear|integrations}
{relative|gear|plans}
{relative|gear|billing}

Screenshots and screen captures:

Notes:

I also tested not including the icon as part of the link, but I think it looks better the other way around as shown above.

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[x] Automated tests.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Links.

@laurynmm This PR is ready for review.
@zulipbot add "help center review"

@drrosa - In general, these changes look good to me. I agree that including the icon in the relative link version looks better than not.
I guess one question I have about adding these is the possibility of us missing that these need to be updated if the icon/design of a gear menu option changes. I feel like a change in text would definitely get flagged for a help center update, but a change in the icon might not. And not having to update the help center docs for minor design changes seems somewhat important. Thoughts?

@laurynmm That's a good point. I agree that not having to update the help center docs for minor design changes is important, but I think this is more of a concern when it comes to UI screenshots or SVG images. I feel like we've already established a consistent pattern for including icons in the help center docs, which would make us likely to remember flagging a change in the icons for a quick help center update. Also, looking at the changelog, it seems like changes to the icon/design of a gear menu option are very infrequent. I think it's worthwhile and a nice touch to include these sort of icons in the instructions.

@drrosa Cool. Yeah, that all makes sense. And there are already icons in other relative help center link text, so I think these updates look good. Marking for integration review.

This looks great, merged, thanks @drrosa and @laurynmm!

authors: drrosa laurynmm timabbott


text:
Use an iterator when updating caches, so we don't load all of them at once

Fixes:

Screenshots and screen captures:

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Functions that only iterate their argument once can accept Iterable[T], which is a supertype of both List[T] and QuerySet[T], to avoid the need to spend extra queries converting from one to the other.

authors: alexmv andersk


text:
travis: Enable zulipbot build notifications.

PRs labeled with the "travis updates" label will receive build notifications from zulipbot.
Usage:
Comment @zulipbot label "travis updates" on a pull request to label the pull request with the "travis updates" label. When the Travis build of the PR is complete, zulipbot will post a comment on the build status.
Note: It is highly recommended that a "travis updates" label be created in the repository (if else, zulipbot will automatically add a default-colored "travis updates" label).

@zulipbot label "area: misc"

Merged, thanks @synicalsyntax!

authors: synicalsyntax timabbott


text:
Refactor people/search_suggestion, add diacritic ignoring in search bar

This includes a refactor of people.js so that there is a people.person_matches_query function that will match people against a query, ignoring diacritics in the name.
search_suggestion was updated to use this function instead of its own person_matches_query function, so now it has the advantage of being able to ignore diacritics in names.

Automated message from Dropbox CLA bot
@cory2067, it looks like you've already signed the Dropbox CLA. Thanks!

Codecov Report

Merging #5531 into master will increase coverage by 0.04%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #5531      +/-   ##
==========================================
+ Coverage   86.64%   86.68%   +0.04%
==========================================
  Files         432      432
  Lines       53567    53566       -1
  Branches     2092     2091       -1
==========================================
+ Hits        46413    46434      +21
+ Misses       7154     7132      -22

Impacted Files
Coverage Δ

frontend_tests/node_tests/search_suggestion.js
97.53% <100%> (+0.34%)
:arrow_up:

static/js/search_suggestion.js
100% <100%> (ø)
:arrow_up:

static/js/people.js
100% <100%> (ø)
:arrow_up:

zerver/tornado/ioloop_logging.py
93.33% <0%> (+46.66%)
:arrow_up:

Continue to review full report at Codecov.

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

Merged, thanks @cory2067!

authors: codecov-io cory2067 showell smarx


text:
issue 7909 fixed

clicking on the stream hides the search box and the issue seems to be fixed. :)

@Redarcher9 can you explain why this change is correct?  We're not interested in PRs that don't explain the reasoning.  I'm closing this; if you'd like to do this, please submit a new PR :)
Check out the Zulip commit message guidelines for more details: https://zulip.readthedocs.io/en/latest/contributing/version-control.html#commit-messages


text:
invite: minimal UI for invite modal

This removes all the streams listed in the invite modal. The users are invited to default streams by default.
If the user wishes to edit the streams to invite to, then it can be edited.

@rishig @hackerkid
I think we can use pills like styling instead of tags. I have updated the pull request accordingly. Please review it and let me know if any changes are required.

@hackerkid  I have changed 'Add as administrator' to 'Invite as administrator' and appended '#' infront of stream names as suggested by Brock. Please let me know if any other changes are required.

@hackerkid @brockwhittaker

Closing in favor of #9992.  Thanks for your work on this @AasthaGupta!

authors: AasthaGupta timabbott


text:
issues getting the close-button to work

sorry for the spam, but I can't seem to quite get the close-button working.
application.js
//= require foundation
//= require turbolinks

$document.on('page:load ready', function() {
  $(document).foundation();
})

haml file for rending a flash message as a callout
- if notice
  .success.callout{ data: { closable: '' } }
    = notice
    %button.close-button{ aria: { label: 'Dismiss alert' }, data: { close:  '' }, type: 'button' }
      %span{'aria-hidden' => 'true'} ×
- if alert
  .alert.callout{ data: { closable: '' } }
    = alert
    %button.close-button{ aria: { label: 'Dismiss alert' }, data: { close:  '' }, type: 'button' }
      %span{'aria-hidden' => 'true'} ×

I got the navbar working so I don't think it's a JS issue, but perhaps I'm missing something? Any advice would be appreciated.
Thank you,

It doesn't look like an issue related to the gem, you should post your issue in this repo: https://github.com/zurb/foundation-sites
I also tried to make closable components work but without success, there is probably a bug (or something missing in the documentation).

I believe this is because the Motion UI CSS isn't loaded. We don't currently have a gem for Motion UI (just npm and Bower packages), but we could publish one.
We also need a better fallback if Motion UI isn't loaded.

Closing this since it is in the Foundation Sites repo

I just added the motion-ui classes to my application.sccs file and it worked.

Awesome!

Great to hear, we decided to keep Motion UI out of the Gem since it is an add-on to Foundation for Sites.

great, thank you!

I think you should include the motion ui css in the gem, so that changes
are kept in sync.
On 1/14/2016 4:20 PM, Arthur wrote:

Great to hear, we decided to keep Motion UI out of the Gem since it is
an add-on to Foundation for Sites.
—
Reply to this email directly or view it on GitHub
https://github.com/zurb/foundation-rails/issues/138#issuecomment-171783231.

@weiserma Agreed, we could publish a separate gem but it might be easier to bundle them together.

authors: Deckluhm arthurzurb gakimball shicholas weiserma zurbchris

repo: zurb/foundation-rails


text:
[Tabs] inconsistent use of active class

The Tab component use .is-active, however the Menu component use .active
Need consistent naming of 'active' class.  I had a scenario where i needed to toggle the active state of both a tab and menu item at the same time.  But, because they use different classes for active, I had to it in two lines instead of one.  Please, update the Tabs component to use 'active' instead of 'is-active'.
Example:
$('my-tab-item, my-menu-item').toggleClass('active');
instead i had to split it up:
$('my-tab-item').toggleClass('is-active');
$('my-menu-item).toggleClass('active');

I think we should actually move everything to .is-active, as these are state classes. Would accept a PR moving menus to .is-active.

Is this still an open issue? I can look into correcting it.

I can look into correcting it.

@jmp1548 .... Sure, why not!

I believe @jmp1548 took care of this with #9757, so I'm going to close this out.

authors: IamManchanda aldees colin-marshall jmp1548 kball

repo: zurb/foundation-sites


text:
This is version 2.1.5 released on December 19, 2011

Don't live in the past :-)

Noted (and fixed) :)

repo: zurb/foundation


text:
Error in the settings.scss file

I am running Foundation 5 with Grunt and I after compling the scss file, the error is
Running "compass:dev" (compass) task
error sass/main.scss (Line 320 of sass/foundation/_settings.scss: Invalid CSS after "...cordion-classes": expected selector or at-rule, was ": $include-html...")
in the settings.scss file, I have uncommented out the $include-html-accordion classes: $include-html-classes;..
I am using Grunt 0.4.5
Grunt Watch 0.5.3
Foundation Version 5.5.1
What could be the problem be?

The Foundation forum would be a great place to seek some help with troubleshooting what might be going on here. If it turns out to be a defect, you'll have some more info to provide here which will help prioritize and get it fixed.
We're trying to keep the issue list focused on confirmed defects in Foundation, so any help everyone can provide to differentiate between usage / troubleshooting and actual defects is appreciated!


text:
hair home page

@babboe1 You are to work on this

ok Boss
I will get on it


text:
[Request] Offline mode during installation (required for Nix packaging)

Hello,
I'm currently integrating private-gpt into Nix and creating a NixOS module as part of this PR. I noticed that when running private-gpt, some requests are made to the internet (hence the creation of the tiktoken_cache directory).
NixOS builds and tests modules in a completely isolated environment, without network access or environmental variables, making it challenging to run tests effectively.
I'm opening this request to see if it would be possible to enable some kind of offline mode and avoid doing external request by default. I don't know how it could be done, but since we're relying on configuration on YAML files, maybe we could add a flag in there to disable access to the network?
There are already multiple issues open about this: https://github.com/zylon-ai/private-gpt/issues/467, https://github.com/zylon-ai/private-gpt/issues/1151, https://github.com/zylon-ai/private-gpt/issues/1179, https://github.com/zylon-ai/private-gpt/pull/1467, https://github.com/zylon-ai/private-gpt/issues/1497, https://github.com/zylon-ai/private-gpt/issues/1766, https://github.com/zylon-ai/private-gpt/issues/1861
What do you think about this idea? Could we consider including this feature in the next release? What are your thoughts?

There needs to be a tokenizer downloaded, given the RAG implementation (using LLamaIndex) needs to be able to count tokens.
LlamaIndex defaults to tokenizer https://docs.llamaindex.ai/en/stable/module_guides/models/llms/?h=tiktoken#a-note-on-tokenization
So there is no way to avoid downloading the tokenizer at some point. Think of it as any other dependency.
I think there could be a couple options:

Add the download of the tokenizer as part of the packaging process.
Add the file itself to the cache folder directly (it is less than 2MB), just like in this solution https://stackoverflow.com/a/76107077

That worked very well, thank you !!!

authors: drupol imartinez

repo: zylon-ai/private-gpt


text:
Bug about the numpy version

Hello, pls what is the version of numpy, I got an error when I reproduced the code. Specifically, I used the latest numpy-1.26.2 as well as mega-nerf's numpy-1.21.2 and both got an error. Hope to hear back from you.

The version of numpy is also 1.21.2 in our setting.


text:
Super Tabs is breaking when I am trying to update the tabs dynamically.

Fyi: I am trying to fetch the tab title and the related pages based on an API call.

I am just updating the model with tabs title and relevant page info.
Ex: Earlier it was like - this.feed = ['You', 'Healthy Reads']; after the user removes it from the options then I am just updating the model this.feed to ['You'].

Please ignore the above query. was able to solve this via the solution mentioned in #46


text:
Allow multiple URLs as input to the spiders

Description:
The current ecommerce spider in the repository accepts a single input URL. This means that crawling different categories in a website requires the creation of multiple spiders. However, this approach is impractical for several reasons:

Handling a large number of inputs becomes tedious and challenging to monitor.
Merging datasets requires additional effort.
Running numerous small spiders is inefficient in resource utilization.

Proposed Solution:
Implementing this feature can be achieved through various methods

Allow multiple URLs in the url field using a separator like |.
Enable the use of an HTTPs URL to a CSV file with one input URL per line. This extensible to other protocols like S3, or SFTP.
Integrate with external services such as Google Spreadsheets or Airtable.

My prefered solution is number 2 as is usually the simpler and more versatile of all of them. It allows to update the URLs set on the file without having to re-configure the spider and there is a huge number of options to host such files without getting handcuffed into a specific provider.

1 is friendlier with a no-code approach.
If we are OK with only supporting HTTP/HTTPS start requests, we could automatically interpret input URLs with a different protocol as links to lists of input URLs, which would allow to support both 1 and 2 with a single spider parameter. Or we could follow an approach similar to that of pip for VCS, e.g. if the protocol is prefixed with urls+ we treat it as a link to a list of input URLs, supporting all protocols, even HTTP or HTTPS, to point to a URL list.

What if we add a checkbox to indicate the URL is a input file instead ? It seems easier to use than having to prefix urls+.

What if we had a separate input_urls field, in addition to urls, to specify the URLs that point to a list of URLs? We get the flexibility of urls+ without a custom schema, and having urls + some boolean option does not seem that different from having 2 URLs options, in the end is 2 fields in both cases.

The downside is that now you have to validate that people is not using url and ìnput_urls at the same time.
Also, I think they could easily be confused, resulting in a bad user experience.

Alternatively, we could have a single field and decide what to do with it based on the headers responses from a HEAD request:

If response is a txt file, treat it as input_urls
If responses is a html, treat it as current url

The downside is that this method requires one additional requests, and there could network / bans issues that affects its outcome.

Yeah, then I think the boolean option is the way to go. Any naming suggestions?

https://github.com/zytedata/zyte-spider-templates/pull/38
https://github.com/zytedata/zyte-spider-templates/pull/41

authors: Gallaecio VMRuiz

repo: zytedata/zyte-spider-templates


text:
ValueError: Model cascade_rcnn_vgg16_pruned_voc is not supported. Available options are listed ...

why i run the code following the instruction, it shows "cascade_rcnn_vgg16_pruned_voc" is not supported ??

@sheldon606  Please check you mxnet version first, and make sure the version is 1.3.x.  Compile the latest MXNet 1.3.x  will probably solve most of those problems.
If not, please post the operations or commands you did, and the errors related.  And comment below.Thanks


text:
Error:(2, 0) Plugin with id 'com.jfrog.bintray' not found.

ide是android studio ,  加入这个库之后sync出错，请问应该如何解决？

以源码的形式导入吗? 去掉build 中的apply plugin: 'com.jfrog.bintray'  这行再试试看

project 目录下的buil.gradle 的dependencies里加入
 classpath "com.jfrog.bintray.gradle:gradle-bintray-plugin:1.7.3"
对应的最新版本可以这里查看

年代久远的issues

来自两年后的回答。。。

authors: ahjsrhj crazycodepeng zzz40500


text:
请问push的时候出现push error怎么办

ERROR - git@github.com:wqyzstql/PicBed.git: invalid privatekey: [B@2acf6e9c

应该是git的私钥有点问题，你可以尝试用http的方式提交

authors: wqyzstql zzzzbw

repo: zzzzbw/gitPic


text:
Create blank.yml

your welcome

your welcome

@16CentAstrology

done


text:
library assembly rewritten to esbuild

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

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


text:
feat!: 💥 ✨ v1 of aireadi.org

📝 Changed routes:

/404
/contact

(view full size)
(view full size)

/cookies/content
/cookies

(view full size)
(view full size)

/
/modules/data

(view full size)
(view full size)

And 13 other routes:

/api/contact
/modules/ethics
/modules
/modules/pedp
/modules/skills
/modules/standards
/modules/teaming
/modules/tools
/privacy/content
/privacy
/publications/content
/publications
/team

4 deleted routes:

/api/contact
/comingsoon
/terms/content
/terms

Commit ae6fc78b5281cdd3180efe52c70d475cc82070f4 (https://aireadi-1aevx6bej-fairdataihub.vercel.app).


text: Typo in Widget Description

repo: AKORA-Studios/Calq


text:
Fix test-assets Folder Reference in README

Noticed that benchmarks CI is failing.
7c209e7 should solve it in the next PR

repo: ARK-Builders/ark-rust


text:
update dependencies

Closing in favor of a new PR.


text:
Modify exit message. Fixes #27.

Codecov Report

Merging #74 (f9bcd1b) into master (5884997) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #74   +/-   ##
=========================================
  Coverage     71.73%   71.73%
  Complexity      417      417
=========================================
  Files            72       72
  Lines          1288     1288
  Branches        134      134
=========================================
  Hits            924      924
  Misses          328      328
  Partials         36       36

Impacted Files
Coverage Δ

...java/seedu/address/logic/commands/ExitCommand.java
100.00% <ø> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

LGTM, merged


text:
Update find feature in user guide

Codecov Report
Base: 73.94% // Head: 73.94% // No change to project coverage :thumbsup:

Coverage data is based on head (1d85202) compared to base (1cd8cef).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #139   +/-   ##
=========================================
  Coverage     73.94%   73.94%
  Complexity      627      627
=========================================
  Files           100      100
  Lines          1842     1842
  Branches        215      215
=========================================
  Hits           1362     1362
  Misses          417      417
  Partials         63       63

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

authors: codecov-commenter leirdas


text:
Update descriptions for images in "Navigating the application" section

Codecov Report
Base: 69.90% // Head: 69.90% // No change to project coverage :thumbsup:

Coverage data is based on head (9a81924) compared to base (0f26557).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #399   +/-   ##
=========================================
  Coverage     69.90%   69.90%
  Complexity      717      717
=========================================
  Files           106      106
  Lines          2429     2429
  Branches        322      322
=========================================
  Hits           1698     1698
  Misses          610      610
  Partials        121      121

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

authors: codecov-commenter tsammeow


text:
Replace all instances of Addressbook with TruthTable

Closed by #172


text:
Fix DG

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

lgtm


text:
#862je6338[IN REVIEW]

Task linked: CU-862je6338 Build static elemnts

repo: Acme-Admin/DISNEY-PROJECT


text:
Feat/components init

fixes #15

repo: Adeeshaj/carvestor-visualizer


text:
is it updated flutter 2.8.0 ?

Any update on flutter 2.8.0?

Yes, major update on Camera

Our SDK use native API to access the camera, I think It is not affected.

Okay. But did you look up 2.8 updates?

Okay. But did you look up 2.8 updates?

Im using 2.8, and 4.0.7 seems to work just fine

authors: LichKing-2234 mirrorlink sahmirzali

repo: AgoraIO/Agora-Flutter-SDK


text:
一种sleep的备选方案：不使用sleep而一直检查当前时间，防止线程长时间休眠被杀死

测试过了吗？

测试过了吗？
去Actions里面Run一次workflow看看运行正不正常呗，正常就合了

手动运行workflow是可以的。

authors: AiRanthem SmartPolarBear

repo: AiRanthem/xmu-daily-report


text:
Custom Chip - Laser Engraved

ecb0d1a

repo: AlexHedley/poker-recording


text:
Fixing RiboDetector rules

thanks @maggimars

repo: AlexanderLabWHOI/eukrhythmic


text:
feat: daily&frp&encrypt

ok


text:
new pull

this is a new think


text:
Monolithic headers fail packaging in non development configurations

fixed

I hope I got all of them.

repo: Amir-BK/unDAW


text:
Fix number length, check  start, end

Thank you for this pull request

Can you explain to me exactly why the regex should support 00 after the number?  I want to know if it is necessary to implement for other regexes as well.  And if you don't need to delete the changes related to that 00 from your request, I will merge your request!

Can you explain to me exactly why the regex should support 00 before the phone number?
I want to know if this is really necessary, implement it for other regexes as well.  And if you don't need to delete the changes related to that 00 from your request, I will merge your request!

Can you explain to me exactly why the regex should support 00 before the phone number?
I want to know if this is really necessary, implement it for other regexes as well.  And if you don't need to delete the changes related to that 00 from your request, I will merge your request!

Dear amir thank you for your response 🙏
Sure i can explain that adding support for 0098 can have mutiple benefits and reasons why
1- 0098 is country code the international dialing code for iran so its can be good for global audiences
2-can has some user experience benefits is some users want to store numbers with 0098 wothout problem
3-phone numbers this way can store formated based on international standards
However +98 is same as 0098 but this way is more comprehensive and versataile
Im glad to see that you want to merge the pr
Feel free if you want to remove it or edit it ❤🙏thnk you

Oh, thank you very much for the explanation.
I didn't look for this issue at all.
but now that you said, I will add it for the others and change the examples as well.

Oh, thank you very much for the explanation. I didn't look for this issue at all.
but now that you said, I will add it for the others and change the examples as well.

You're welcome
thank you for merging this PR its an honor to participate in this repository and contribute
wish you all the best

Thank you friend. this repo is for all iranaian programmers
بقول ما ایرانیا، بفرمایید خونه خودتونه 😁

Thank you friend. this repo is for all iranaian programmers بقول ما ایرانیا، بفرمایید خونه خودتونه 😁

به به قربونت دمت گرم واقعا ❤🙏

authors: AmirMahdyJebreily mhrnnoi

license: Unlicense


text:
Dep

This pull request introduces several enhancements and new features to improve platform-specific configurations and requirements handling in the project. The main changes include adding platform-specific requirements and imports, updating the pyproject.toml file, and modifying various Python files to support these new configurations.
Platform-specific enhancements:

Added platform-specific requirements (requirements_linux, requirements_macos, requirements_win) and imports (run_imports_linux, run_imports_macos, run_imports_win) in pyproject.toml [1] [2].
Updated oxt/___lo_pip___/basic_config.py to include new properties for platform-specific requirements and imports [1] [2] [3] [4].
Updated oxt/___lo_pip___/config.py to include new properties for platform-specific requirements and imports [1] [2].

Configuration and setup updates:

Modified .devcontainer/devcontainer.json and .devcontainer/lo_sdk/devcontainer.json to include new extensions (charliermarsh.ruff, tamasfe.even-better-toml) [1] [2].
Updated pyproject.toml to include new URLs for Python versions and added platform-specific sections.

Functional improvements:

Enhanced oxt/___lo_pip___/install/pkg_installers/install_pkg.py to handle platform-specific requirements during package installation.
Improved oxt/___lo_pip___/install/requirements_check.py to validate platform-specific requirements and imports [1] [2] [3].
Updated src/processing/json_config.py to load platform-specific requirements and imports from configuration.

Documentation:

Added a new link to the README.md for the LibrePythonista extension.

repo: Amourspirit/python-libreoffice-pip


text:
Write up the minidataapi spec

In-progress here: https://github.com/AnswerDotAI/fasthtml/pull/294

repo: AnswerDotAI/fastsql


text:
Btn header

ok.


text:
Commit

I Just commented

repo: Anulakshmikk/github-slideshow


text:
Create Leap year

approved

repo: AromalPS/Hacktoberfest


text:
No tabs in circle ci config

@rgardler,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

authors: msftclas rgardler

repo: Azure/acs-demos


text:
Merge main to live 08-28-2022

Docs Build status updates of commit 03c259a:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: GitBranchDeletedOrForcePushed]

Cannot sync git repo to specified commit because ref +refs/pull/2352/merge has been deleted or has been force pushed.
 If this is pull request, please ensure it is mergeable (no merge conflict) and open (not closed).

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

repo: Azure/azure-docs-sdk-java


text:
KeyVault: Secrets Rotation

Item out of date, closing as there are no plans in the future for this

authors: RickWinter gearama

repo: Azure/azure-sdk-for-cpp


text:
[Post-patch-release] Merged changes back to main branch after auto-patch release

API change check
API changes are not detected in this pull request.

authors: azure-sdk mssfang

repo: Azure/azure-sdk-for-java


text:
[AzureMonitorDistro] Update Readme with advanced configuration

API change check
API changes are not detected in this pull request.

authors: azure-sdk rajkumar-rangaraj

repo: Azure/azure-sdk-for-net


text:
[Containerregistry] Update min dependence

/check-enforcer override

repo: Azure/azure-sdk-for-python


text:
Summarize beta.8 changes

LGTM

authors: anjulgarg prprabhu-ms

repo: Azure/communication-ui-library


text:
新前端主题文件

感谢你提交新主题😄，我会晚点审查代码后合并，建议在下次提交pr时at我一下，否则我很难察觉。

好的，谢谢

authors: cabbagelol karma-kuiming

repo: BFBAN/bfban-website


text:
您好，我是哈工大的研究生，我们组对你们的研究非常感兴趣，请问有训练好的模型吗？我在代码中没有找到。

+1，跪求模型

模型会在近期整理后上传，感谢关注

好的，感谢您的回复
---原始邮件---
发件人: @.>
发送时间: 2021年11月25日(周四) 晚上9:35
收件人: @.>;
抄送: @.@.>;
主题: Re: [BIT-DA/I2V-GAN] 您好，我是哈工大的研究生，我们组对你们的研究非常感兴趣，请问有训练好的模型吗？我在代码中没有找到。 (Issue #1)
模型会在近期整理后上传，感谢关注
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

We currently offer a pretrained traffic model, please download from the links.

@BingfengHan-BIT 请问一下预训练文件应该放在哪里？另外能提供一下训练的shell脚本吗？

authors: BingfengHan-BIT DreamWaterFound IncubatorShokuhou UUUUUUsername


text:
Create Herbarium_Project_Outline

@drago1234 thanks! can you open a PR with content on https://github.com/BU-Spark/ml-herbarium instead


text:
fixed NME trying to switch to webgpu but failing

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/15716/merge
Test environment:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15716/merge/index.html
To test a playground add it to the URL, for example:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15716/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/15716/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/15716/merge
https://gui.babylonjs.com/?snapshot=refs/pull/15716/merge
https://nme.babylonjs.com/?snapshot=refs/pull/15716/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/15716/merge#BCU1XR#0

Visualization tests for WebGPU (Experimental)
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15716/merge/testResults/webgpuplaywright/index.html

WebGL2 visualization test reporter:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15716/merge/testResults/webgl2playwright/index.html


text:
fix: / isURL and keep exists files named with hash untouch

Thank you for your contribution!

This should be published in v2.2.6!

authors: BjornTheProgrammer mindon

repo: BjornTheProgrammer/bun-plugin-html


text:
Numerous axis labels result in slow performance when resizing window

Added fix to not deal with axis labels if they aren't being displayed.
More to come...

repo: Blake-Madden/Wisteria-Dataviz


text:
Update from 1.21.0 to 1.21.30

Btw can an update to the extension be pushed, so that it includes these schema changes?

Will do


text:
Graph Analytics - Missing property usage (percentage))

See the mocks

authors: dhaneshnm samuel-kerrien

repo: BlueBrain/nexus


text:
task/AN-4999/update-android-studio

@nitindasari-BO given that this is just a git submodule, we can just merge this in. And then in the HiRoad repo, we need a PR to update the gitmodule for the main branch

authors: marcoatblueowl nitindasari-BO

repo: BlueOwlDev/analytics-android


text:
BamlImage should be pydantic compatible

Fixed by #1062

authors: aaronvg hellovai

repo: BoundaryML/baml


text:
Fix select micropython neopixel

Codecov Report

Merging #137 (05f9370) into main (c180818) will decrease coverage by 0.17%.
The diff coverage is n/a.

:exclamation: Current head 05f9370 differs from pull request most recent head 855c1ee. Consider uploading reports for the commit 855c1ee to get more accurate results

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main     #137      +/-   ##
==========================================
- Coverage   70.52%   70.35%   -0.17%
==========================================
  Files          38       38
  Lines        1693     1697       +4
  Branches      377      378       +1
==========================================
  Hits         1194     1194
- Misses        423      428       +5
+ Partials       76       75       -1

Flag
Coverage Δ

unittests
70.35% <ø> (-0.11%)
:arrow_down:

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

Impacted Files
Coverage Δ

belay/cli/select.py
26.00% <ø> (-0.05%)
:arrow_down:

... and 2 files with indirect coverage changes

authors: BrianPugh codecov-commenter

repo: BrianPugh/belay


text:
[PAB-32] refactor signup OTP

review lại

repo: CNDvn/ParkingApp-BE


text:
Formal verification of interrupt detection

@CharlyCst You can already review & merge if you want. I still need to make sure that the detected interrupt is the same. Currently I only check if they detects equivalently but I don't check the returned type

@CharlyCst I rebased the commit and introduced a new function in Virt-sail which allows us to model check the entire virtualisation of the interrupts at once :)

repo: CharlyCst/miralis


text:
Added Dashboard Menus and info/ Changed font

merging without review, any blame goes to the requestee


text:
feat: new projects page

close #7


text:
Add donum to monorepo

https://github.com/CivMC/Donum

repo: CivMC/Civ


text:
GISS-E2-1-G pr

missing model output in GCS and therefore not including in final model list


text:
setPage on a relationmanager does not work and causes the test to fail

This might actually be a Filament bug.

repo: CodeWithDennis/filament-tests


text:
Hw1

lgtm

authors: timkpaine yd-wu


text:
Show number of missing classifications per stations

This might already be fixed backendwise (#171).
If you want an explicit number and not sum up the returned patients, lmk and I will provide that to you.

repo: CombiRWTH/medical-staff-assessment


text:
Add draft report for new-starter sign off

 Prototype: update reports page for new sign off process


text:
Create a timeline

Add a list of all your activities, achievements, work experience. Add as much info as you can remember. In case you don’t have working experience tell about your pet-projects or student projects. Don’t be shy :)


text:
feat: title enhanced and cards animated

@Nagendra88688 Thanks for this amazing work, love the shadows man, Greate job. If you still want to work on more, I will assign more issues to you. Thank you very much for your contribution

repo: DeveloperAspire/developers-fact-card


text:
fix: Fix key access to possibly non-existent location

:tada: This PR is included in version 0.4.1-dev.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

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


text:
Initiate mult-frequency support in models

Will be superseeded by new work


text:
Use Everest::Date/Serial/Thread where applicable

@a-w50 can you check if this still compiles/works. should be able to merge by now

repo: EVerest/everest-core


text:
Meta MMLU

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

repo: EleutherAI/lm-evaluation-harness


text:
Feature flag directive

AC

[ ] Should be able to show/hide a feature in a template using a directive
[ ] Should be based on feature flags configuration

repo: Energinet-DataHub/greenforce-frontend


text:
Adding a first draft of the JSON creation of widget, via WidgetCanvas

There is a list of documentation error you need to fix before merging this PullRequest :

Sparkle/src/widget/spk_widget_canvas.hpp:28
Compound spk::WidgetCanvas is not documented.

Sparkle/src/widget/spk_widget_canvas.hpp:39
Compound spk::WidgetCanvas::Geometry::Value is not documented.

Sparkle/src/widget/spk_widget.hpp:207
Member findChild(const std::string &p_name) (function) of class spk::Widget is not documented.

Sparkle/src/math/spk_vector2.hpp:91
Member IVector2(const spk::JSON::Object &p_object) (function) of interface spk::IVector2 is not documented.

Sparkle/src/math/spk_vector3.hpp:90
Member IVector3(const spk::JSON::Object &p_object) (function) of interface spk::IVector3 is not documented.

Sparkle/src/widget/spk_widget.hpp:207
Member findChild(const std::string &p_name) (function) of class spk::Widget is not documented.

Sparkle/src/widget/spk_widget.hpp:207
Member findChild(const std::string &p_name) (function) of class spk::Widget is not documented.

Sparkle/src/widget/spk_widget.hpp:207
Member findChild(const std::string &p_name) (function) of class spk::Widget is not documented.

Sparkle/src/widget/spk_widget.hpp:207
Member findChild(const std::string &p_name) (function) of class spk::Widget is not documented.

Sparkle/src/widget/spk_widget.hpp:207
Member findChild(const std::string &p_name) (function) of class spk::Widget is not documented.

Sparkle/src/widget/spk_widget.hpp:207
Member findChild(const std::string &p_name) (function) of class spk::Widget is not documented.

Sparkle/src/widget/spk_widget.hpp:207
Member findChild(const std::string &p_name) (function) of class spk::Widget is not documented.

Sparkle/src/widget/spk_widget.hpp:207
Member findChild(const std::string &p_name) (function) of class spk::Widget is not documented.

Sparkle/src/widget/spk_widget_canvas.hpp:31
Member Instanciator (typedef) of class spk::WidgetCanvas is not documented.

Sparkle/src/widget/spk_widget_canvas.hpp:85
Member WidgetCanvas(const std::filesystem::path &p_canvasFilePath, Widget *p_parent=nullptr) (function) of class spk::WidgetCanvas is not documented.

Sparkle/src/widget/spk_widget_canvas.hpp:86
Member WidgetCanvas(const std::string &p_name, const std::filesystem::path &p_canvasFilePath, Widget *p_parent=nullptr) (function) of class spk::WidgetCanvas is not documented.

Sparkle/src/widget/spk_widget.hpp:207
Member findChild(const std::string &p_name) (function) of class spk::Widget is not documented.

Sparkle/src/widget/spk_widget_canvas.hpp:32
Member classInstanciatorLambda (variable) of class spk::WidgetCanvas is not documented.

Sparkle/src/widget/spk_widget_canvas.hpp:42
Member Type (enumeration) of struct spk::WidgetCanvas::Geometry::Value is not documented.

Sparkle/src/widget/spk_widget_canvas.hpp:59
Member Value(const spk::JSON::Object &p_object) (function) of struct spk::WidgetCanvas::Geometry::Value is not documented.

Sparkle/src/widget/spk_widget_canvas.hpp:53
Member active (variable) of struct spk::WidgetCanvas::Geometry::Value is not documented.

Sparkle/src/widget/spk_widget_canvas.hpp:54
Member type (variable) of struct spk::WidgetCanvas::Geometry::Value is not documented.

Sparkle/src/widget/spk_widget_canvas.hpp:55
Member value (variable) of struct spk::WidgetCanvas::Geometry::Value is not documented.

Sparkle/src/widget/spk_widget_canvas.hpp:56
Member formulaValue (variable) of struct spk::WidgetCanvas::Geometry::Value is not documented.

repo: EreliaStudio/Sparkle


text:
Updating ArcGIS Maps SDKs name

Thanks @noahmulfinger ! @alaframboise the awesome have to be shorted alphabetically 😅

repo: Esri/awesome-arcgis-developers


text:
Expose filter options when creating a persistent subscription.

There are no tests checking the filtering feature.

authors: YoEight oskardudycz

repo: EventStore/EventStoreDB-Client-Java


text:
feat: update action input type

What are these changes for?

What are these changes for?

somehow my local always changes to this when I do git commit.

What are these changes for?

somehow my local always changes to this when I do git commit for this repo.

I was asking about the PR as a whole.

authors: actong mtruongsa

repo: Evoke-Platform/evoke-sdk


text:
That's FLAIS, folks 🥁

Not so FLAIS yet..

repo: FINTLabs/fint-geointegrasjon-adapter


text:
Implement staffSuppress for bibs. closes #145. closes #206

@bltravis  @branchedelac  any of you up for a code review?