text:
pre-commit running pip-compile across multiple operating systems

I have pre-commit set up using GitHub Actions to run on Linux and Windows. pre-commit is configured to run pip-compile:
- repo: https://github.com/jazzband/pip-tools
  rev: 6.4.0
  hooks:
    - id: pip-compile

The pip-compile documentation suggests running pip-compile for each environment combination (OS+Python):
https://github.com/jazzband/pip-tools#cross-environment-usage-of-requirementsinrequirementstxt-and-pip-compile
"As the resulting requirements.txt can differ for each environment, users must execute pip-compile on each Python environment separately to generate a requirements.txt valid for each said environment."
On Linux, I currently manually run pip-compile and output to requirements.txt and on Windows I have pip-compile output to requirements.win.txt.
How do I get pip-compile to output to the correct requirements file depending on the operating system that pre-commit runs on?

See https://github.com/pre-commit/pre-commit/issues/1455#issuecomment-628019565 for workarounds.

I was able to get it to work like this:
- repo: https://github.com/jazzband/pip-tools
  rev: 6.4.0
  hooks:
    - id: pip-compile
      entry: ./.pre-commit-scripts/pip-compile.py

and then in the custom pip-compile.py:
#!/usr/bin/env python

from subprocess import call  # nosec
from sys import platform

def main():
    if platform == "win32":
        cmd = (r"pip-compile.exe", "--output-file", "requirements.win.txt")
    else:
        cmd = ("pip-compile",)
    return call(cmd)  # nosec

if __name__ == "__main__":
    exit(main())

Thanks for the link!

authors: BTOdell atugushev

repo: jazzband/pip-tools


text:
col-num 属性设置为1时会堆叠在一起

为啥我的col-num 属性设置没效果？始终都是默认的12

col-num 要为number类型

source: gharchive/issue

authors: JoffreyN wxwebfeng yaoxingpu

repo: jbaysolutions/vue-grid-layout

license: mit

license_type: permissive

license_source: bigquery


text:
Avoid ambiguity in indirect constructors

Fixes #159

DRAFT.md needs updating

source: gharchive/pull-request

license: MIT

license_source: github-api


text:
Knex 2.1.0 support

It looks like it should be easy to support with mock-knex.

I'm having issues with knex@2.2.0 when using the current latest version of this library (0.4.11)
It seems that it won't work with any knex version above 2.0 due to a restriction set in the peer dependencies:
"peerDependencies": {
    "knex": "> 0.8 <= 2.0"
}

It works when using npm install --legacy-peer-deps to ignore the range of specified versions, but this is not optimal

Same problem here. The peerDependencies is really annoying.
Normally, If we update knex, We don't do the npm install --legacy-peer-deps

There hasn't been any commit or message on any of the PRs since May. @jbrumwell do you need any help? Maybe a call for maintainers would be a good idea

@jbrumwell I've tested with knex 2.3.0 as well and it works fine.

Removed the requirement for the version range, should not work as expect in 0.4.12 let me know if not. @Fryuni I haven't had a professional need for mock-knex in 4 years, I would be interested in discussing this.  Always bugs me that I don't have time to get to these issues :)

@jbrumwell we ended up migrating to another library for mocking knex since we also needed some extra features (like testing if concurrent transactions are executed in isolation), so I also don't have any professional use for mock-knex anymore.
Although I like this API more for simple applications.

@Fryuni :+1: Thanks for letting me know :)

Removed the requirement for the version range, should not work as expect in 0.4.12 let me know if not. @Fryuni I haven't had a professional need for mock-knex in 4 years, I would be interested in discussing this. Always bugs me that I don't have time to get to these issues :)

I'd be interested to know how you handle the db tests nowadays. Do you use a lite inmemory db ? Or another approach ?

We use docker to test against postgres, MySQL etc

'd be interested to know how you handle the db tests nowadays. Do you use a lite inmemory db ? Or another approach ?

I'm using the library knex-mock-client.
I found its API to be easier to extend to more complex scenarios. It also made easier to implement a feature that both libraries were missing, which is to test interleaving transaction contexts, as you can see here

I always loved the simplicity of not needing a real database when running the test-suite, even though I couldn't find much on the matter when googling. It seems a large majority of people don't go that way. And now even the creator of mock-knex does so. I guess we can't argue with such an overwhelming majority... There must be something wrong with mocking the db adapter.
The thing that annoys me the most is that we need the full docker-compose shebang for unit-testing on alocal dev environment, wich has a price in term of performance / run time :-/

For that, many solution can be found. When I work with mongoDB, we used a mock simulate the entire DB (using mongo-mock : https://www.npmjs.com/package/mongo-mock). In tests, we have isolation of our Database in memory (with documents, ...) and in the tests, we check if the request return correct documents  according to the logic of the request.
The same with the elements in the database as soon as we manipulated it (via insert/update/delete).
For me, the better solution is this : in-memory test database with isolation for each tests (Otherwise, with parallel tests you can quickly have unwanted side effects to check if everything is going as you want).
Unfortunately, I could never find an equivalent library for knex or mysql or sql in general

@Fryuni
I believe that unit-tests should be fast and standalone, hence not require a real database. Testing that "the query is truly working" should be done in integration tests or end to end tests imho.
Also, when you use something like pg-mem, you diminish the "query is really working" thing, as it is not the real targeted db, so the guarantees you get from the tests are not that strong...
All things considered, it looks like an acceptable tradeoff, but for localDev it requires always running the tests in a docker container, which slows things down quite a bit

authors: Fryuni JeremyVanBiljon jbrumwell mlarcher rage28 taybin throrin19

repo: jbrumwell/mock-knex


text:
检测结果可视化得到的图片不清楚

作者辛苦，通过show_result_pyplot生成的图片不高清 且保存后有大量空白区域  无法用作插图 请问如何得到高清的可视化结果

可以尝试使用 BboxToolkit 提供的可视化程序，并选择直接保存，不使用窗口


text:
remove desmos in favor of recharts

What does this PR do and why?
Provide a description of what this PR does. Link to any relevant GitHub issues, Notion tasks or Discord discussions.
Screenshots or screen recordings
If applicable, provide screenshots or screen recordings to demonstrate the changes.
Acceptance checklist

[ ] I have evaluated the Approval Guidelines for this PR.
[ ] I have tested this PR in all supported browsers.
[ ] I have tested this PR in dark mode and light mode (if applicable).

Current dependencies on/for this PR:

main

PR #2502   👈

This comment was auto-generated by Graphite.

repo: jbx-protocol/juice-interface


text:
Get error when restart session

Hi, i get error message when i execute dap.restart(), this is my config https://github.com/jbyuki/one-small-step-for-vimkind/issues/31#issue-1659728432
E5113: Error while calling lua chunk: Vim:E475: Invalid argument: Channel id must be a positive integer

stack traceback:
        [C]: in function 'rpcnotify'
        ...packer/start/one-small-step-for-vimkind/lua/osv/init.lua:55: in function 'sendProxyDAP
'
        ...packer/start/one-small-step-for-vimkind/lua/osv/init.lua:166: in function 'f'
        ...packer/start/one-small-step-for-vimkind/lua/osv/init.lua:631: in function '__index'
        t.lua:13: in main chunk

Is there a use for restart with osv ? It does not support it currently.

repo: jbyuki/one-small-step-for-vimkind


text:
PR for discussion: how to read a YamlNumber either as Int or as Double

In order to do YAML<->JSON conversion, I need to read YamlNumber that can carry either Int or Double, and convert them differently.
After having elaborately studied the source, and its differences with spray.json (which I'm familiar with), I still cannot get this to work.
Would you please suggest a way that works.
Below code shows three different things I've tried. One seems to ignore the Int and it gets converted to a Double. The other ones (if there is no case for YamlNumber(x: Int), claim they cannot match YamlNumber(42).
The code that MoultingYaml itself has uses .num which is not available outside of the library.

I think you're suffering from numeric widening in the function you're supplying to map (i.e. Int gets widened to Double, since the latter is the "larger" of the types you're returning), hence the "unexpected" result.
You can see that the types are accurate inside the YamlNumbers with:
val nums = Seq(YamlNumber(42), YamlNumber(4.2))

"be convertible to an Int as well as a Double" in {
  nums(0).value.getClass.getSimpleName mustEqual "Integer"
  nums(1).value.getClass.getSimpleName mustEqual "Double"
}

You can avoid the numeric widening in this case by explicitly typing the return value of the function you supply to map as Any:
val nums = Seq(YamlNumber(42), YamlNumber(4.2))

"be convertible to an Int as well as a Double" in {
  def conv(v: YamlNumber[_]): Any = v.value
  val Seq(outA, outB) = nums.map(conv)

  outA.getClass.getSimpleName mustEqual "Integer"
  outB.getClass.getSimpleName mustEqual "Double"
}

Thanks for the pointers, @jcazevedo
The problem arises within a pattern match and is better showcased by this StackOverflow issue I did.
It's because of moultingyaml's use of the type parameter for YamlNumber, which spray.json does not need/use. Ideally, I'd like to do something like this:
case YamlInt(x) => JsNumber(x)      // JSON output without digits
case YamlDouble(x) => JsNumber(x)

What works in my (closed source) source is:
      case x: YamlNumber[_] => x.value match {
        case xx: BigDecimal => JsNumber(xx)
      }

That simply looks rather elaborate to my eye and I was looking for a more straightforward solution than two inner pattern matches.

Also this works:
case x: YamlNumber[_] => JsNumber( x.value.toString )

Performance is not top priority to me. The above converts Any (but which we know is a number) to a string, and back to spray.json. If you can provide this without the interim string conversion, that'd be splendid.

I simplified things a bit by removing the type parameter from YamlNumber in 15a3e3629c7b4a0b5c4ca9f29bff41237a69810c. Can you test this and tell me if it suits your needs? I'll publish a stable version of MoultingYAML afterwards.

@jcazevedo Yep, it works perfectly for me now.
case YamlNumber(x) => if (x.isValidInt) JsNumber(x.toInt) else JsNumber(x.toDouble)

or
case YamlNumber(x) if x.isValidInt => JsNumber(x.toInt)
case YamlNumber(x) => JsNumber(x.toDouble)

I hope you didn't break anything by removal of the template param.
It may be a good idea to change the version to 3.1.2-SNAPSHOT now?

Closing since the discussion was done.

@akauppi: I've just released version 0.4.0 of MoultingYAML with this change.

authors: akauppi jcazevedo

repo: jcazevedo/moultingyaml


text:
Added BodyAsString extension method

Added BodyAsString extension method as wished in #34

Awesome 😄👍
Nice work in your first piece of OSS! 🎉🎉

Awesome. Thanks 😄👍
Congrats on your first piece of OSS. 🎉🎉

authors: BugAgain jchannon

repo: jchannon/Botwin


text:
🛑 geb haus soe is down

In ea6a858, geb haus soe (http://www.geburtshaus-soest.de) was down:

HTTP code: 403
Response time: 302 ms

Resolved: geb haus soe is back up in fa75c60.

repo: jcruse-erw/UptimeReport


text:
🛑 audience is down

In e397353, audience (https://www.audience-soest.de) was down:

HTTP code: 0
Response time: 0 ms

Resolved: audience is back up in f0eb5cb.


text:
Updated version and added numba to setup.py

@jdebacker. This PR updates the version to 0.11.13 in setup.py, ogcore/__init__.py, and CHANGELOG.md. I also added numba to the requirements in setup.py and requirements.txt. We can merge your PR as soon as you review and incorporate these changes.

@rickecon Thanks for this. Merging.

authors: jdebacker rickecon

repo: jdebacker/OG-Core


text:
🛑 Home Assistant - DS918P is down

In 3b734b0, Home Assistant - DS918P (https://home.jdh1.duckdns.org/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Home Assistant - DS918P is back up in 9f127f6 after 10 minutes.


text:
View Private Key

Need to make viewing address private key more accessible/obvious in desktop version

Add "Private Key" field to "settings" screen similar to mobile version

View Private Key option added to Address menu in v0.9.10
https://github.com/jdogresorg/freewallet-desktop/releases/tag/v0.9.10

repo: jdogresorg/freewallet


text:
Using fuzzaldrin-plus on the client

I love this library, It's possible to compile this into a js that run on the client?

I got it working by compiling it with browserify and it seems to work

It's possible this library get transferred to atom for maintenance.
But If I end up maintaining it long term i'll surely add a /dist/ folder with a minified and properly packaged for the browser. Probably something that's compatible with the different require style and fallback to a global.
Lastly I'll say fuzzaldrin-plus is specialized around the programming editor use case. If that's what you are building, that's great.  I have another library FuzzySearch that don't get nearly the same exposure, but would be better for natural language kind of fuzzy. (Paint my walls <-> wall painting)

For the moment I'll close as your browserify version seems to cover that need.

authors: jeancroy pdgago

repo: jeancroy/fuzzaldrin-plus


text:
Non train_hook module

Traceback (most recent call last):
File "train.py", line 21, in
from train_hook import check_if_should_pause
ImportError: No module named 'train_hook'
Could not find train_hook anywhere else

I have ran into the same problem.
Also have a problem in utils.py importing box_overlaps *
Throws 'ModuleNotFoundError: No module named 'box_overlaps'.
Also, is  there a purpose for scattered *'s in front of feature_map_shape? They throw syntax errors so I removed them.

@ZiningWang Sorry, and I just add this in 9138855, you can merge this latest commit to your local repo.
@MilPat555 I'm not sure if you are using Python2.x. Since this code can only run under Python3.5+, so this syntax error may due to this.

authors: MilPat555 ZiningWang jeasinema


text:
关于CallEnqueueObservable进行onError分发的时候为什么不返回httpResponsecode的疑问？

我对OkGo结合Rx2的进行了测试，故意修改url使得触发404错误。
我发现框架直接调用了onError，返回给我一个HttpException。但是这个exception的code为0，追查到CallEnqueueObservable里面，我发现是OkGo框架的Response返回了getException而不是基于原生的getRawResponse再封招好code，这是什么考虑呢？🤔

我觉得把http code塞入HttpException是个不错的主意

repo: jeasonlzy/okhttp-OkGo


text:
Typescript 1.1 support

Typescript 1.1.0-1 is out.
A lot of gulp plugins depend on this repo.
Possible to update to 1.1?

I'm interested in getting this working with TypeScript 1.4. Any reason not to try doing this?

@flcdrg that depends on what you intend to do with it. Is there any reason gulp-typescript can't do what you're trying to do? What's your goal?

I'm looking to support building typescript in ember-cli. I'm open to alternatives, but I ended up here via broccoli-typescript
-----Original Message-----
From: "Jed Mao" notifications@github.com
Sent: 26/02/2015 5:37 PM
To: "jedmao/typescript-api" typescript-api@noreply.github.com
Cc: "David Gardiner" david@gardiner.net.au
Subject: Re: [typescript-api] Typescript 1.1 support (#1)
@flcdrg that depends on what you intend to do with it. Is there any reason gulp-typescript can't do what you're trying to do? What's your goal?
—
Reply to this email directly or view it on GitHub.

You can also just require('typescript'), in version 1.4 a public api is exposed.

Yeah, I reckon that might be the easiest path.
On 26 February 2015 at 19:16, Ivo Gabe de Wolff notifications@github.com
wrote:

You can also just require('typescript'), in version 1.4 a public api is
exposed.
—
Reply to this email directly or view it on GitHub
https://github.com/jedmao/typescript-api/issues/1#issuecomment-76141658.

@svallory @flcdrg The TypeScript 1.4 API is reasonable easy to work with - I've updated gavinhungry/typestring to use it, instead of typescript-api.

You are correct - it was pretty straight forward to use. I've managed to get something working in https://github.com/flcdrg/ember-cli-typescript
Thanks for the suggestions!

Thanks @gavinhungry! I tryed that, but the api doesn't expose all I need :/ I've implemented a compilation method which accepts a custom compiler host and it now works even on Rhino :) But to do that I had to stick to compiling my package with the typescript source

authors: bestander flcdrg gavinhungry ivogabe jedmao svallory

repo: jedmao/typescript-api


text:
Remove deprecated user-read-birthdate scope from authentication process

This PR fixes the issue #8 that is causing the application to get the "Illegal scope" message when trying to authenticate against the Spotify API. According to the solution in the issue comment the problem is related with the user-read-birthdate scope.
I've spent some time doing a research about the scope, and it is not available anymore in Spotify's API docs.
I've also found some issues at GitHub repositories and seems that this scope was deprecated and seems that the application does not need this data, so I removed the scope from the authentication function in order to allow the sign-up process to proceed.
https://github.com/spotify/web-playback-sdk/issues/11#issuecomment-538620631
https://github.com/spotify/web-api/issues/1262#issuecomment-524833983

Thank you for your collaboration!

I will prepare new release with your change next week.

authors: devsdmf jedruniu

repo: jedruniu/spotify-cli


text:
Details

I liked those yapb bots but I see that they do not use the silencers, flashlight and night vision
and I see that the bot chatter of the csbot no longer works

Yes, we needed this features on yapb bots. And also add more waypoints like 3000 waypoints or more. Or make it unlimited. 1024 is not enough for some maps. Make a command for them to have the ability to use flashlight or not. And make some bits use silencer. Dont make them always using it

They do use silencers actually. From time to time.
As for night vision and flashlight, it's useless, as engine doesn't give lighting info for given point easy way, so they can use it randomly, if you need it.
Podbotmm has code for calculating lighting info, but it's too cpu-intensive to include it into yapb. Sorry.

I am pretty sure they didnt use silencer. I have played with yapb so many times. And i mean so many times! But never see them use silencers at all.
If you think that u cant add flashlight, than forget about this idea. But i really want it :)
plz add these:
1)make mp_freeforall 1 and 0 works with yapb.
2)yapb bots use silencer. But dont let them use silencer every time. Make it random. Make the percent of using silencer as 50% chances. Or make a command that allow or disallow yapb to use silencer. Here is an example: yb_silencer 0 / yb_silencer 1
3)Third, plz fix issue for low-difficulty bots because they never try to defuse the bomb.
4)the lowest difficulty 0 and 1 bots will not throw grenades, plz let them throw grenades.
5)Let yapb bots use flashlight on dark areas. Add a command : yb_allow_flashlight 1 / yb_allow_flashlight 0. (If u can add this, it will be great)

add 3048 points. Need lots of waypoints for larger maps. If 2048 is the max u can do...than go for 2048 than :)

Can u make bots buy any pistols they want on the first round? They never bought any pistols. They only buy it if they have a rifle :(
You can do this: If yb_economics rounds 0, can buy any pistols on first round. and yb_economics_rounds 1, they wont buy pistols on first round. ( Follow economics cfg).

Or u can make a new cfg for yapb when they use economics and when they do not use economics. Make two cfgs.
8)[Mention on Above ] for yb_difficulty 0 and 1, the bots are already kinda good. For difficulty 2 to 4 is extremely pro like they are cheating. For difficulty 2, can u make them react slower abit? And also make the difficulty 2 bots not to aim so well. Difficulty 2 is already like playing against professional level. It is very hard due to the bots have aimbot and can immediately kill you cuz they always aim for the head. For difficulty 0,1,2,3, can you make them have shaky aim? So that the bots dont play like a aimbot hacker. Maybe you can make them aim below the head or make their aim abit shaky.
BTW, Do u have any social media so that i can talk to u??
I hope u understand these suggestions i gave u because i have played with yapb for almost a year. And i hope to see my suggestions being added :)
Make it on pc and android :D

done
done
need to test
done (but less than hs bots)
will try to add without impact on perfomance, need some time.
2048 added
probably, not sure now
aimbot is cool, however will tweak difficulties in some time.

android build delaying. sorry.

Thanks for making these. But i played on android. Can u try to make it faster for android? :(

@Left4Dad
On what map do you play where you need flashlights?

Hmm it depends on some maps that needed flashlights. For example, Dust l, cs_747 (forgot the name), Nuke, And other custom made maps. Hmm, if its hard, i suggest u dont add flashlights than :)

Flashlight and nvg already working in current build.

You mean for yapb?

Yup

So u added the function?? Cool! I was hoping u can do all those suggestions on android tho. Do it as quick as you can :)

https://yapb.ru/files/android/cstrike/5592/
Google Play version of Xash3D & CS16client should be uninstalled before try. Probably android debugging should be enabled to make those APKs.
It's temporary, just for test. Changes will be published to CS16client with next update.

Wow thanks! I will try it on and tell you if it works perfectly :)

https://yapb.ru/files/android/cstrike/5594/
friendlyfire should be 1, after setting mp_freeforall -> sv_restart 1 should work.
about blood and bullets, dunno actually. this is just a test build, have no time to debug anything atm.

So will all these yapb features be in the official apk cs 16client apk on play store???
About the mp_freeforall. I will try now to see if it works

Sure, changes will be available in official client, when all testing will be finished.

I tried. It didnt work. I go to game and set friendly fire on. Than when i enter a map, the yapb bots automatic spawn in. And i set mp_freeforall 1 and type sv_restart 1. When the match restart, my team still ignore each other and try to kill enemies. They didnt kill their own teammates. Can plz help? Have u tried urself in the apk and does it work?

Tried. And it's works correctly. Show yb version output.
On Wed, May 15, 2019, 20:04 Left4Dad notifications@github.com wrote:

I tried. It didnt work. I go to game and set friendly fire on. Than when i
enter a map, the yapb bots automatic spawn in. And i set mp_freeforall 1
and type sv_restart 1. When the match restart, my team still ignore each
other and try to kill enemies. They didnt kill their own teammates. Can plz
help? Have u tried urself in the apk and does it work?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jeefo/yapb/issues/79?email_source=notifications&email_token=AANS4YXVYZ3ALGPBJJTO2PLPVQ7ABA5CNFSM4HEDO5J2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVPJVZQ#issuecomment-492739302,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AANS4YWSR6UEBDXJYUIQETDPVQ7ABANCNFSM4HEDO5JQ
.

How to show yb version output?

Do you have facebook messenger? Easier to talk using it.

I type yb_version and its 2.10. Only problem is that mp_freeforall 1 didnt work :(
Is there other yapb commands i must use to make them shoot each other??

Ok it works now. Thank u so much bro!!! U helped out alot :). I will close this issue now. Hope u add these features in official apk soon :D

Here are the final things that should be added and fix so that yapb will be more fun and making it look realistic. [YAPB WILL BE BETTER]

When yapb bots zoom using snipers, and when the bot using sniper saw the enemy and when the enemy left their sight, the bots unzoom the scope too fast. Its not realistic. Plz make them zoom abit longer before unzooming. Make it 2secs before unzooming the sniper.

Make afk yapb bots. I know this sounds dumb, but its a cool feature. There is once i played yapb and there is a bot in my team that goes afk. Its so realistic. Maybe its a bug. But can you add this plz??? Its fun. Add this function using a command like for example, yb_afk 0 means all bots will not be afk, and yb_afk 1 means bots sometimes goes afk. Make the max number of 1-3 bots that goes afk. And make it super random and very rare. Hope u can add this. And remember, make it rarely happen.

Make bots balance themselves using all grenades. Yapb bots prefer smoke grenade more. Plz balance it so that they can use other grenades better.

4)Plz add yb_debug 2. The ones that work is yb debug 0 and 1. Can u make another one? Which is yb_debug 2. Make yb_debug 2 only show these features (Name, Health, Armor, Item, Clip, Ammo and money). Thats all. Dont add the viewable waypoints where the yapb will walk to certain places with a line that they will follow just like in yb_debug 1. Add a yb_debug 2 with only those things plzz.
Yb_debug 1 is too complicated.
5)Make the yapb always listen to our commands for [Drop C4, Drop weapon,  and Make double jump] Sometimes yapb will not drop the c4 or weapons when i use those commands. Make them 100% listen to what we want them to do. And also make them come to where i stand and make a double jump. Sometimes they stand at the wrong position for the double jump and sometimes they ignore to do the double jump and walked away.

Make yapb bots look around while in spawn. This can make it look realistic. In cs1.6 normal bots, they will look at different positions while freezing at spawn. But those bots only look at one direction. Thats all. But for yapb, when they spawn, they only look straight forward and all yapb bots stand in a line like a soldier 😂. Make 1 to 4 bots to look around when they spawn. So that they look like humans. Make them look at different places while freezing at spawn. Make a command for this if possible. Make two commands like, one will make them not look around in spawn, another will make them look around the area while freezing at spawn. Make it random. Dont make all 4 bots to look around. Make some bots to look around and not all the bots.

7)Can u make the commands to dissapear when we click these commands? Like Drop c4, drop weapon, double jump and finish double jump. Here is my problem. When i click drop c4 or double jump, the commands 1,2,3,4 is still there. We must click 0 to exit the commands. Can u make it more simplier?? Do it this way: if i clicked on 1 or 2 or 3 or 4, the yapb action commands will disapear by itself so that i dont need to press 0 to exit the yapb action commands . This way is much more faster. Plz do it :)

[This is an optional idea]. Since that all yapb bots has aimbot, can u make two seperate commands for two seperate bot playstyle? For example, yb_aimbot 1 means they will play like the way they play right now. They uses aimbots. And another command is yb_aimbot 0 that will disable them to aimlock at their enemies and they will only shoot the enemy if they see their enemy. Plz make these. Its more realistic. For example like cs 1.6 normal bots, they dont have aimbot. So when we go behind our enemy, they wont see us. But for yapb, they will always know where we are😂. So plz add these two commands whether they will use aimbot or will not use aimbot. [This is optional. If u can do this, it will make us have better gameplay experience with yapb bots]

Make a command for yapb to use or not to use night vision and flashlight. Because i played the apk version that u send me. I see those bots use night vision. But i didnt see them use flashlight yet because i only play for 10 mins. Can u tell me they will use flashlight or not? And have u added any commands for them to allow or disallow to use flashlight and night vision?? Please add commands in the yapb.cfg file or u can add the ability to buy or not to buy night vsion on the general.cfg file.

THANK YOU ONCE AGAIN. YOU HAVE MADE YAPB BETTER EVERY SINGLE TIME!
If u dont understand my requests, feel free to ask me as well. I just want yapb to be realistic and better :)

So can you implement these features?

?? Have u gone on vacation?😅

authors: GhostRander Left4Dad jeefo

repo: jeefo/yapb


text:
配置好没反应

问题描述
跳出网页配置后双击打开软件只有这两行字
2023/10/28 21:29:44 可使用 .\ddns-go.exe -s install 安装服务运行
2023/10/28 21:29:44 监听 :9876 ...
重启软件没用，阿里云后台也没有记录，完全没有反应
DNS服务商
Alidns(阿里云)
是否已搜索同类问题
已搜索，但未能解决
操作系统
Windows
架构
x86_64
版本
5.6.4
安装方式
其它
额外信息
No response

配置截图看下，重要信息打码

配置截图看下，重要信息打码

谢谢，不用了，已经修好了，不好意思我发现忘了填写Domains里的域名而且也没错误提示直接保存成功了（≧﹏ ≦）

配置截图看下，重要信息打码

谢谢，不用了，已经修好了，不好意思我发现忘了填写Domains里的域名而且也没错误提示直接保存成功了（≧﹏ ≦）

谢谢老哥，救了命了，我也是忘了填写域名。
因为直接启动又没问题（忘了配置文件不同），装到docker里和安装成服务就都不行。

authors: jeessy2 maomaokeai wzdxy


text:
Is there a recommended way to run the tests in parallel?

the test suite is awe-inspiring... but obviously it takes awhile to run. Do you have any tricks for running it in parallel?
I just played around with using concurrencytest on a concurrenttest branch. I did not submit a pull request, as I'm not confident that this is the optimal solution (that package does not seem to be updated often, though perhaps that is not a problem). I've only tested on python 2.7.9 (on ubuntu)
It did improve the runtime of:
time python dendropy/test/__main__.py

from:
real    5m26.438s
user    5m14.988s

to:
real  2m30.712s
user  0m0.724s

PS: I'm not sure what is going on with the huge diff in user. I'd guess that the tests are largely IO bound, but I don't know why that doesn't show up in the single-threaded version's accounting. And 0.724 seems too low for the concurrent version.

Hi Mark, that's a good idea. I've tinkered around with trying to break the tests into components, but as with anything that requires on-going curation on a constantly changing base, it quickly lapsed into obsolescence (artifacts remain in the "@" constructs as, unfortunately, are still documented in the README).
The thing about the tests are, that I think they can be organized in a much smarter way. There are so many tests that are reported as a single test but actually run several variants in a loop. In many cases they are useful variants, but in others the variants are just a brute force/dumb way of checking things that, e.g., the instruction to preserve underscores in a Newick source is maintained, by running through cases that really should be invariant with respect to the test, e.g. when reading from a file/stream, file path, or a data string.  This is cruelly and callously accelerating the heat death of the universe due to brutish design. I think the answer to a lot of this is re-architecturing most if not all the tests around using mock fixtures, now the mock module is part of the standard library. I thought, though, that I may want to make the jump the 4 and finalize 4.0.0 first, and address bring the tests into a smarter and kinder-to-the-universe footing.
So now, I just run the tests using "python -m unittest -fc", which exits on the first fail or keyboard interrupt and reports results, or "python setup.py test > ~/Scratch/fails.txt", and watch funny animal videos on youtube or something. Obviously, parallelizing the test execution is a much better approach ...

p.s. Did you commit the changes to this branch? Not seeing anything in the diff.

doh! fixed.

I fixed some shared state bugs and the tests should now run in parallel fine. I would recommend using the pytest-xdist https://pypi.org/project/pytest-xdist/ plugin for pytest
python3 -m pytest -n 4 tests/

authors: jeetsukumaran mmore500 mtholder

repo: jeetsukumaran/DendroPy


text:
feat: added support for connecting via proxy server

Description
Partially Fixes #321
The proposed change allows the user to specify a proxy server ip address and port in the .env file to use for the connection to the internet. In certain networks internet access is restricted and is only allowed via a proxy server.

Testing
Note: Requires proxy server such as squid-proxy to be able to test.
Once a proxy server is available, add the following entries to the .env file:
PROXY_ADDRESS="<IP ADDRESS OF PROXY SERVER>"
PROXY_PORT=<TCP PORT FOR PROXY SERVER>

Finally, if a values is provided for PROXY_ADDRESS and a connection to that address is not possible, the connections will fail with a message similar to the one below
[9:04:30 PM] error :: ✖ [nvidia] nvidia founders edition - net::ERR_PROXY_CONNECTION_FAILED at https://www.nvidia.com/en-us/shop/geforce/gpu/?page=1&limit=9&locale=en-us&categ
ory=GPU&gpu=RTX%203080

Nice, this will be good for those running in the cloud a well!

Can you add the fields for proxy authentication? or a way to feed it a list of proxies via command line like .........................................npm run start --proxy /users/user1/proxylist.txt
then some format for the proxy files for it to use the proxies like http://username:password@61.18.233.35:60121

repo: jef/nvidia-snatcher


text:
Adding Linting and Tests to CI

This PR is meant to be added after Rev0, it will not work until the following are fixed:

CI will not pass until the existing files are passed through the linter
CI will not pass until the failing test case is corrected

new pr at #253

repo: jeff-rey-wang/utrition


text:
giga brain solution (python)

https://www.hackerrank.com/challenges/balanced-brackets/forum/comments/800459

I just realized I did this one recently.
I saw the giga brain solution, pretty smart. Unfortunately, it is much easier to code the giga brain solution in Python than C++

authors: blackwidoq jeffrylew

repo: jeffrylew/add-wrinkles-to-brain


text:
--fold-now is a one-time dealio

For some reason, the --fold-now option works only one time, after which the TUI will display no results, and then when it quits back to console, the terminal is corrupt.

No longer relevant - have removed fold-now option  and using console scripts created at install time to launch TUIs.


text:
year corrected

wake up, America!
Abe was long gone by the year that was.

Total B$. PR #54 2>/dev/null.

authors: diomed theInfectedDrake


text:
在遇到云端日志平台故障或者本地网络故障时，添加日志缓存，方便网络恢复后重试

避免丢失日志。

建议在sdk 做这个缓存复用性比较强

为log4j， log4j2，logback 的 plugin 分别增加错误写入磁盘，并且重试上传的功能。

authors: jemygraw lvheyang

repo: jemygraw/qiniu-logging-plugin


text:
Override infra-bug template

Override infra-bug template
Testing done
Submitter checklist

[x] Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
[x] Ensure that the pull request title represents the desired changelog entry
[x] Please describe what you did
[ ] Link to relevant issues in GitHub or Jira
[ ] Link to relevant pull requests, esp. upstream and downstream changes
[ ] Ensure you have provided tests - that demonstrates feature works or fixes the issue

Sadly it cannot override from organisation but replace everything (like remove the doc)

I will restore https://github.com/jenkinsci/.github/tree/master/.github/ISSUE_TEMPLATE

repo: jenkins-infra/plugin-modernizer-tool


text:
markewaite & darinpope as testng maintainers

Add markewaite as TestNG plugin maintainers
The TestNG plugin is up for adoption.  I (@MarkEWaite) would like to adopt it so that we can modernize it and use that modernization to highlight specific benefits from some of the modernization steps.
Mentioning @nullin in case there is any concern with the adoption request.
Submitter checklist for adding or changing permissions
Always

[x] Add link to plugin/component Git repository in description above

When adding new uploaders (this includes newly created permissions files)

[x] Make sure to @mention an existing maintainer to confirm the permissions request, if applicable
[x] Use the Jenkins community (LDAP) account name in the YAML file, not the GitHub account name
[x] Make sure to @mention the users being added so their GitHub account names are known if they require GitHub merge access (see below).
[ ] All newly added users have logged in to Artifactory at least once

Reviewer checklist (not for requesters!)

[ ] Check this if newly added person also needs to be given merge permission to the GitHub repo (please @ the people/person with their GitHub username in this issue as well). If needed, it can be done using an IRC Bot command
[ ] Check that the $pluginId Developers team has Admin permissions while granting the access.
[ ] In the case of plugin adoption, ensure that the Jenkins Jira default assignee is either removed or changed to the new maintainer.
[ ] If security contacts are changed (this includes add/remove), ping the security officer (currently @daniel-beck) in this pull request. If an email contact is changed, wait for approval from the security officer.

There are IRC Bot commands for it

@darinpope indicated to me that he has logged into repos.jenkins-ci.org, but rather than slow this pull request, I've removed him from the request so that it can be processed.  We can do the livestream session with just me as a maintainer.


text:
Add @stephen-badger as a maintainer of metrics

Description
https://github.com/jenkinsci/metrics-plugin
As suggested by @stephenc
Submitter checklist for changing permissions

Always

[x] Add link to plugin/component Git repository in description above

When adding new uploaders (this includes newly created permissions files)

[x] Make sure to @mention an existing maintainer to confirm the permissions request, if applicable
[x] Use the Jenkins community (LDAP) account name in the YAML file, not the GitHub account name
[x] All newly added users have logged in to Artifactory at least once

Build failed; the context from the latest run is:
Expand to view
	at groovy.json.JsonBuilder.toString(JsonBuilder.java:347)
	at groovy.json.JsonBuilder.toPrettyString(JsonBuilder.java:361)
	at groovy.json.JsonBuilder$toPrettyString$0.call(Unknown Source)
	at io.jenkins.infra.repository_permissions_updater.ArtifactoryPermissionsUpdater$_generateApiPayloads_closure1.doCall(ArtifactoryPermissionsUpdater.groovy:193)
	at sun.reflect.GeneratedMethodAccessor10.invoke(Unknown Source)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at org.codehaus.groovy.reflection.CachedMethod.invoke(CachedMethod.java:93)
	at groovy.lang.MetaMethod.doMethodInvoke(MetaMethod.java:325)
	at org.codehaus.groovy.runtime.metaclass.ClosureMetaClass.invokeMethod(ClosureMetaClass.java:294)
	at groovy.lang.MetaClassImpl.invokeMethod(MetaClassImpl.java:1024)
	at groovy.lang.Closure.call(Closure.java:414)
	at groovy.lang.Closure.call(Closure.java:430)
	at org.codehaus.groovy.runtime.ResourceGroovyMethods.eachFile(ResourceGroovyMethods.java:1070)
	at org.codehaus.groovy.runtime.ResourceGroovyMethods.eachFile(ResourceGroovyMethods.java:1088)
	at org.codehaus.groovy.runtime.dgm$936.invoke(Unknown Source)
	at org.codehaus.groovy.runtime.callsite.PojoMetaMethodSite$PojoMetaMethodSiteNoUnwrapNoCoerce.invoke(PojoMetaMethodSite.java:274)
	at org.codehaus.groovy.runtime.callsite.PojoMetaMethodSite.call(PojoMetaMethodSite.java:56)
	at org.codehaus.groovy.runtime.callsite.CallSiteArray.defaultCall(CallSiteArray.java:48)
	at org.codehaus.groovy.runtime.callsite.AbstractCallSite.call(AbstractCallSite.java:113)
	at org.codehaus.groovy.runtime.callsite.AbstractCallSite.call(AbstractCallSite.java:125)
	at io.jenkins.infra.repository_permissions_updater.ArtifactoryPermissionsUpdater.generateApiPayloads(ArtifactoryPermissionsUpdater.groovy:134)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at org.codehaus.groovy.reflection.CachedMethod.invoke(CachedMethod.java:93)
	at org.codehaus.groovy.runtime.callsite.StaticMetaMethodSite$StaticMetaMethodSiteNoUnwrapNoCoerce.invoke(StaticMetaMethodSite.java:151)
	at org.codehaus.groovy.runtime.callsite.StaticMetaMethodSite.callStatic(StaticMetaMethodSite.java:102)
	at org.codehaus.groovy.runtime.callsite.CallSiteArray.defaultCallStatic(CallSiteArray.java:56)
	at org.codehaus.groovy.runtime.callsite.AbstractCallSite.callStatic(AbstractCallSite.java:194)
	at org.codehaus.groovy.runtime.callsite.AbstractCallSite.callStatic(AbstractCallSite.java:214)
	at io.jenkins.infra.repository_permissions_updater.ArtifactoryPermissionsUpdater.main(ArtifactoryPermissionsUpdater.groovy:317)
[Pipeline] stage (Archive)
Using the ‘stage’ step without a block argument is deprecated
Entering stage Archive
Proceeding
[Pipeline] archiveArtifacts
Archiving artifacts
[Pipeline] archiveArtifacts
Archiving artifacts
[Pipeline] }
[Pipeline] // node
[Pipeline] End of Pipeline

GitHub has been notified of this commit’s build result

ERROR: script returned exit code 1
Finished: FAILURE

Powered by the Comment Logger

Hello, seems like you ignored the template. It is here for a reason :). Can you please check all boxes, and mark possibly N/A the ones that are not applicable to show you've processed the entry. Thanks!

@stephenc

@batmat updated PR description accordingly

@stephen-badger seems like your account could not be found in Artifactory, but you did check the corresponding checkbox:

We need an Artifactory admin like @daniel-beck to verify. Are you sure you logged in to Artifactory, not JIRA?

I'm positive I am logged in to Artifactory:
https://gist.github.com/stephen-badger/2b2dd7841e3b9d9b1489f4ad5f0c5f29#file-sbadger-logged-in-wider-png
I am also logged in JIRA and Confluence too, but I don't imagine that affects anything here?

Probably just the PR build needing a kick.

I'm psychic 😆
Closing & reopening PRs triggers a new PR build and doesn't need Artifactory admin access.

authors: batmat daniel-beck jenkinsadmin oleg-nenashev stephen-badger


text:
cleanup: delete Jenkinsfile_k8s

As the job configuration on infra.ci.jenkins.io is already specifying the Jenkinsfile path, this PR removes the unused simlink from Jenkinsfile to Jenkinsfile_k8s.
https://github.com/jenkins-infra/kubernetes-management/blob/2cd45da2352361481c0b242b4b5bcfae19e9a602/config/jenkins-jobs_infra.ci.jenkins.io.yaml#L417C37-L417C37

Self-merging, node code change, build OK and still executed on both ci.jenkins.io & infra.ci.jenkins.io.

repo: jenkins-infra/stories


text:
文章格式的建议和讨论

参考 微信公众号排版指南篇（全）
相对其他公众号来说，我们的文章格式看起来有些突兀。下面是我测试过的一些格式，建议字体调小同时间距调大一点会更美观。

1.75倍行间距

14px

15px

16px

17px

1倍行间距

14px

15px

16px

17px

嗯，是的，现在的文章在发布时没有做什么编辑（但已经要占用 很多时间）。思考怎么样才能话费较少的时间来完成这个事情。

On May 28, 2019, at 11:07, Yang Zhou notifications@github.com wrote:
参考 微信公众号排版指南篇（全） https://zhuanlan.zhihu.com/p/35408995
相对其他公众号来说，我们的文章格式看起来有些突兀。下面是我测试过的一些格式，建议字体调小同时间距调大一点会更美观。
1.75倍行间距
14px
https://imgchr.com/i/VeyPDU
15px
https://imgchr.com/i/VeyAUJ
16px
https://imgchr.com/i/VeyWrT
17px
https://imgchr.com/i/VeyZCR
1倍行间距
14px
https://imgchr.com/i/Veymgx
15px
https://imgchr.com/i/VeyKKK
16px
https://imgchr.com/i/VeyQbD
17px
https://imgchr.com/i/VeyY8I
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/jenkins-infra/wechat/issues/141?email_source=notifications&email_token=AALCFPLLEPKFR74WOA72FBDPXSOVTA5CNFSM4HP7L7BKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4GWD32ZA, or mute the thread https://github.com/notifications/unsubscribe-auth/AALCFPJKEULCIQVGBP6V3GDPXSOVTANCNFSM4HP7L7BA.

这个可以自动化完成吗，我们可以在明天的会议上讨论一下。

是的，我们可以在会议上讨论下。

On May 28, 2019, at 11:20, Yang Zhou notifications@github.com wrote:
动化完成吗，我们可以在明天的会议

authors: LinuxSuRen arjenzhou

repo: jenkins-infra/wechat


text:
chore: bdd-spring-1576099524 to 0.0.1

chore: Promote bdd-spring-1576099524 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


text:
chore: bdd-spring-1573211311 to 0.0.1

chore: Promote bdd-spring-1573211311 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


text:
chore: bdd-spring-1574450616 to 0.0.1

chore: Promote bdd-spring-1574450616 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


text:
chore(deps): Upgrade kpt

I can't see in the docs that there are any breaking changes for kpt pkg, which as far as I know is the only functionality currently used by Jenkins-X. But I plan to use kpt live to solve jenkins-x/jx#8024 and those commands have breaking changes, so it would be easier to upgrade before starting to use that functionality.

/cc ankitm123 tomhobson babadofar

repo: jenkins-x-plugins/jx-gitops


text:
Optimise  Dockerfile

Optimize Dockerfile to be better with size and security.
FROM cgr.dev/chainguard/wolfi-base:latest

COPY --from=build /jx-cdevents-adapter /jx-cdevents-adapter
	EXPOSE 80

# required for external tools to detect this as a go binary
ENV GOTRACEBACK=all

ENTRYPOINT [ "./jx-cdevents-adapter" ]

Hello, is this an open issue? I would be happy to work on it

Yes i have assigned to you!

Hey, thanks for the assigning the issue. It would be great if you could briefly describe what you mean when you mention "optimized and better security."

I guess no one is working, Can I work on it?

Hey @i-am-yuvi sure no problem

Hey @bradmccoydev, I would like to suggest some changes:

How about using a more lightweight base image golang:1.16-alpine3.14 which will eventually reduce the size of docker image
We can also run the application as the non-root user to reduce the vulnerabilities that could occur if it run as a root user.

# Add a new user
RUN adduser -D myuser

authors: Kalpesh-18 bradmccoydev i-am-yuvi

repo: jenkins-x/jx-cdevents-adapter


text:
[JENKINS-61900] JCasC support for CredentialsProvider extensions

See JENKINS-61900.
/cc @jenkinsci/configuration-as-code-plugin-developers

Looks like I can not add reviewers, so just mentioning here: @timja @jetersen @oleg-nenashev

the other credential providers I've seen

Can you point me to those please? I'm validating this with an non-public extension, but I'd like to do it with at least another one.

https://github.com/jenkinsci/aws-secrets-manager-credentials-provider-plugin/blob/master/src/main/java/io/jenkins/plugins/credentials/secretsmanager/AwsCredentialsProvider.java
https://github.com/jenkinsci/azure-keyvault-plugin/blob/master/src/main/java/org/jenkinsci/plugins/azurekeyvaultplugin/AzureCredentialsProvider.java

OK. I understand now. So those providers do not create credentials entries in Jenkins, the credentials store is just calling the provider to read credentials from the external system (AWS or Azure).
So, yeah, this is for CredentialsProvider extensions which actually store something in Jenkins. In my specific case it's CyberArk, the credentials provider reads from CyberArk but users need to create a credentials entry in Jenkins (just with an ID matching the CyberArk one). This approach has some benefits that are not relevant now.
Thanks for the references!

Build failed for unrelated reasons: Remote call on EC2 (aws) - Ubuntu 18.04 LTS (i-096e07c7f640f5d6a) failed. Re-triggering.

Is this ready for review?

Yes, Jenkins CI is just sick 😓

authors: amuniz jetersen jvz timja

repo: jenkinsci/credentials-plugin


text:
Notary signature missed for 2.46.1-alpine

Docker Notary signature is missed for 2.46.1-alpine image, however it is present for the rest images including 2.46.1.
[root@centos7 ~]# notary lookup docker.io/library/jenkins 2.46.1-alpine

* fatal: No trust data for 2.46.1-alpine
[root@centos7 ~]# notary lookup docker.io/library/jenkins 2.46.1
2.46.1 sha256:c0cac51cbd3af8947e105ec15aa4bcdbf0bd267984d8e7be5663b5551bbc5f4b 4290
[root@centos7 ~]#

So basically all of these tags are signed:
jenkins:2.32.3-alpine
jenkins:alpine
jenkins:latest
jenkins:2.46.1
Just not jenkins:2.46.1-alpine

no longer applies as we are not pushing to jenkins anymore

authors: carlossg rhuddleston roman-vynar

repo: jenkinsci/docker


text:
Fix windows slaves

Fix windows configs on not first position..

@stephenashank Please confirm and merge if its something left after testing..

repo: jenkinsci/google-compute-engine-plugin


text:
Unable to pull logs while build still in progress

What's the preferred way to fetch logs periodically from jenkins?
Most of the clients using response headers to determine if log is complete.
Example here:
https://github.com/silas/node-jenkins/blob/master/lib/build.js#L190
Thanks to that approach logs can be fetched partially. Jenkins also offers another API to retrieve logs using offset.
Java client using simplified endpoint:
https://github.com/jenkinsci/java-client-api/blob/28d9dbf31241a7b7453488f06408b19c387538b2/jenkins-client/src/main/java/com/offbytwo/jenkins/model/BuildWithDetails.java#L358
I would be happy to contribute and build more sophisticated way to retrieve logs, but I just want to know general target for this client, before doing it.

:+1:

It would be great to offer more ways to fetch logs or better if those can be fetched partially for the users of the API..If you like to contribute that would be great idea...

Awesome. I think we can do this in many ways some propositions bellow. I would create PR soon.
h4. Fetching logs asynchronously.
BuildWithDetails build=...
build.poolLogs(new LogUpdateListener(){
  function String onData(){...}
  function String onFinish(){..}
}

h4. Fetching logs with markers and additional metadata.
BuildWithDetails build=...
LogDetails details = build.fetchLogs(...)
details.finished()
details.delta(); // delivers delta

h4. Fetching with specifying limit
BuildWithDetails build=...
LogDetails details = build.fetchLogs(0,1000) //  start and offset

What would be preferred approach?

To be honest I have no preferred approach. Start with that one you like most...may be you can offer multiple ways ;-) ?

build.details().getConsoleOutputText()？

@doterme This is for finished builds when you do not need to deal with the partial console log.

@wtrocki ,you should get a build number at first,then get the console log in a timer,this is not for finished,it is in real time

@doterme - Sorry for confusion. In fact I implemented getConsoleOutputText method to resolve this issue. Obviously this ticket is resolved so I'm going to close it. See referenced PR for more details.

authors: doterme khmarbaise matzew wtrocki

repo: jenkinsci/java-client-api


text:
How to get job status when job was in queue but not start to run

Could you please let me know if there is a way to get job status when it was in queue but not started running?
Currently, I can't get the job status when it was in queue. Always show me the job was running although the job was really in queue still.

`JenkinsServer jenkinsServer = new JenkinsServer(newURI("http://yourJenkinsServer:8080/"));
JobWithDetails smokeTestJob = jenkinsServer.getJob("YourJobName");
Map<String,String> jobParams = new HashMap<String,String>();
QueueReference queueReference = smokeTestJob.build(jobParams);
do {
Thread.sleep(2000L);
queueItem = jenkinsServer.getQueueItem(queueReference);
log("Job still in queue"+queueItem.getExecutable());
} while (queueItem.getExecutable() == null);
Build build = jenkinsServer.getBuild(queueItem);`
When queueItem.getExecutable()!=null, the job was still in queue, no node was available for this job, but build.details().isBuilding() is always true.


text:
Add support for Compress Build Log plugin

https://wiki.jenkins-ci.org/display/JENKINS/Compress+Build+Log+Plugin

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests

authors: jenkinsadmin misery

repo: jenkinsci/job-dsl-plugin


text:
Generate AWS X-ray compatible trace IDs

What feature do you want to see added?
It would be very useful when using AWS X-Ray to have the plugin be able to generate X-Ray compatible trace IDs. The awsxrayexporter chokes when it encounters non-xray ids. I'm not aware of other ways to convert or process non-xray IDs in flight at the otel-collector.
`2024-05-24T10:14:53.225Z	debug	awsxrayexporter@v0.78.0/awsxray.go:114	Error translating span.	{"kind": "exporter", "data_type": "traces", "name": "awsxray", "error": "invalid xray traceid: b47fb1759d3adef02fac9043e6d0354b"}`

I appreciate that vendor specific support might not be desirable but AWS might be a large enough of a use case to be considered, I'm not aware of other vendors using their own ID formats.
Upstream changes
The AWS distribution for the Java agent patches in support for generating X-ray IDs, I wonder if this could be a toggle based on a configuration option in the Jenkins plugin.

https://github.com/aws-observability/aws-otel-java-instrumentation/blob/v1.32.0/awsagentprovider/src/main/java/software/amazon/opentelemetry/javaagent/providers/AwsTracerCustomizerProvider.java
AWS Java Agent - how it works

Are you interested in contributing this feature?
No response

I think I've found the answer, X-Ray started supporting W3C trace IDs ~8 months ago so no need to generate X-Ray format at the source. ADOT Collector >= v0.34 is required.

https://aws.amazon.com/about-aws/whats-new/2023/10/aws-x-ray-w3c-format-trace-ids-distributed-tracing/

Thanks @hhtpcd . Can we then close this issue?

Sure can - I've tested the plugin with the ADOT Collector and it works like a charm. Thanks for the plugin! ❤

authors: cyrille-leclerc hhtpcd

repo: jenkinsci/opentelemetry-plugin


text:
Add direct geospatial index support

Since #371 was raised back in 2014 I’ve wanted a way to improve and simplify adding geospatial indexes.
For a long time now I've been returning to that issue as a basic guide on how to implement geospatial queries.
There's plenty of other issues out there about how to use geospatial features.
This pull request adds geospatial support by adding a geospatial index function to the Schema builder, and updating the documentation to help others how this can be done.
Affected issues

#55
#297
#299
#371
#708
#853
#886
#965

@benargo thanks for the great contribution

Thanks @benargo! 🍺 !

authors: arubacao benargo jenssegers

repo: jenssegers/laravel-mongodb


text:
Approving updated config hangs

When choosing "y" when prompted to approve an updated config, the process hangs. I have a feeling that this is related to https://github.com/jenterkin/vim-autosource/commit/68223a12264cdcea53e6532f458b8a32c0aa3999.
If anyone is hitting this you can fix this in the meantime by rming the associated file in g:autosource_hashdir.

fixed by https://github.com/jenterkin/vim-autosource/commit/7ef554d0bcbad52fc82abd020c5ffbc9666b320f
I have absolutely no idea how

repo: jenterkin/vim-autosource


text:
🛑 new_pressmedia is down

In da31190, new_pressmedia (http://13.125.155.96) was down:

HTTP code: 0
Response time: 0 ms

Resolved: new_pressmedia is back up in d23c745.


text:
Radar chart tooltip label incorrect

I see yout live demo, found that radar chart tooltip labels were incorrect.
Like the live demo page, the label should be "Thing 1", "Thing 2", but it appear the data number.
I tried the code in my project it caused the same problem, and I use the origin chart.js is fine.
Thanks!

Would you mind attaching/referencing some code so I can investigate this further?

I found the live demo site is broken, so I try a simple code about radar chart.
const labels = ['label1', 'label2', 'label3', 'label4', 'label5'];

const radarData = {
  labels: labels,
  datasets: [
    {
      label: 'radar',
      fill: true,
      cubicInterpolationMode: 'monotone',
      backgroundColor: 'rgba(66,129,164,0.2)',
      borderColor: 'rgba(66,129,164,1)',
      pointBorderColor: '#fff',
      pointBackgroundColor: 'rgba(66,129,164,1)',
      data: [1,2,3,4,5]
    }
  ]
};

const radarOptions = {
  maintainAspectRatio: true,
  responsive: true,
  legend: {
    labels: {
      fontColor: 'rgba(248, 184, 82, 1)',
      fontSize: 16
    }
  }
};

class App extends Component {
  render() {
    console.log(radarData)
    return (

            <Radar
              data={radarData}
              width={400}
              height={350}
              options={radarOptions}
            />

    );
  }
}

export default App;

And I screenshot the radar graph, I thought the label text must be "label4"?

The "# of Votes" text that is appearing is set here. This is the intentional behavior of chart.js, an example of this can be seen here.
If it is desired to change what is appearing in the tooltip, that is functionality that can be changed in the label callback portion of the options.

okay it works. Thanks a lot!

authors: Holychung jhonnold


text:
False Positive on jackson-mapper-asl

Just recently, all the CVEs for jackson-databind started to be reported for jackson-mapper-asl.
While jackson-mapper-asl is largely the Jackson 1 equivalent to jackson-databind for Jackson 2, I don't know to what extent all the "before version 2.x" flaws would apply.
False positive on library jackson-mapper-asl-1.9.13.jar - reported as cpe:2.3:a:fasterxml:jackson:1.9.13:*:*:*:*:*:*:*
<dependency>
  <groupId>org.codehaus.jackson</groupId>
  <artifactId>jackson-mapper-asl</artifactId>
  <version>1.9.13</version>
</dependency>

I'm guessing it's a change in the upstream data. Perhaps they changed the CPE but didn't add any lower bounds?

I moved this from a FP report to a question after running the test myself. The findings being reported are from OSS Index - a vulnerability source that has information beyond what is available within the NVD.  If you look at one of these in the OSS Index (CVE-2017-7525 - you may need to register/login to view the component information) you will see that the OSS Index is saying that pkg:maven/com.fasterxml.jackson.core/jackson-databind is vulnerable to CVE-2017-7525.

@jeremylong you mean it's saying that jackson-mapper-asl is vulnerable? I'll raise it over there.

It appears I was mistaken.

authors: OrangeDog jeremylong

repo: jeremylong/DependencyCheck


text:
CentralAnalyzer.fetchMavenArtifacts failing to fetch files from https://search.maven.org

Our CVE scanning got lots of errors from CentralAnalyzer when fetching pom files from https://search.maven.org. It looks like a transient error. I have a few questions:

I know I can disable CentralAnalyzer 1 with property analyzer.central.enabled=false or CLI flag --disableCentral, but could someone help me understand what the analyzer does and the consequence of disabling it. A concrete example would be helpful.
It is possible to get the artifacts from a local cache? our scanning should avoid external dependencies/connections.
Is there a backup site for search.maven.org?

[ERROR] Error retrieving https://search.maven.org/remotecontent?filepath=org/eclipse/persistence/jakarta.persistence/2.2.2/jakarta.persistence-2.2.2.pom; received response code 502; Bad Gateway
[ERROR] Analysis task was cancelled.
[ERROR] One or more fatal errors occurred
[WARN] An unexpected error occurred during analysis of '/workspace/bigtop/presto_340-1_all/usr/lib/presto/plugin/presto-pinot/larray-0.2.1.jar' (Central Analyzer): java.lang.InterruptedException: sleep interrupted
[ERROR]
org.owasp.dependencycheck.analyzer.exception.UnexpectedAnalysisException: java.lang.InterruptedException: sleep interrupted
	at org.owasp.dependencycheck.analyzer.CentralAnalyzer.fetchMavenArtifacts(CentralAnalyzer.java:355)
	at org.owasp.dependencycheck.analyzer.CentralAnalyzer.analyzeDependency(CentralAnalyzer.java:229)
	at org.owasp.dependencycheck.analyzer.AbstractAnalyzer.analyze(AbstractAnalyzer.java:131)
	at org.owasp.dependencycheck.AnalysisTask.call(AnalysisTask.java:88)
	at org.owasp.dependencycheck.AnalysisTask.call(AnalysisTask.java:37)
	at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
	at java.base/java.lang.Thread.run(Thread.java:829)
Caused by: java.lang.InterruptedException: sleep interrupted
	at java.base/java.lang.Thread.sleep(Native Method)
	at org.owasp.dependencycheck.analyzer.CentralAnalyzer.fetchMavenArtifacts(CentralAnalyzer.java:352)
	... 8 common frames omitted

We had a full unprecedented production outage on thursday 12.January because of this issue.
https://serverfault.com/questions/1120168/access-denied-to-search-maven-org
https://issues.sonatype.org/browse/MVNCENTRAL-7785
it looks like a DOS attack filter on the gateway to the search.maven.org...
or some configuration errors in the server cluster?
So far the only solution was to disable the dependencyCheck in our CICD system.
Not a pretty workaround at all!
I am trying to use the nexus analyzer but without success so far.

@uvwildos Disabling just the centralAnalyzer --disableCentral as documented on the CLI configuration page should work to get dependency-check finishing successfully. It may however may trigger more False Positives (hence disabling it is discouraged for CLI-users) that you would have to suppress locally if dependency-check is configured build-breaking on vulnerabilities in your pipeline.
For libraries published on Maven Central that don't package the maven coordinate information (pom) CentralAnalyzer helps establish their proper G/A/V coordinates which help the logic to better match and reduce FP rates as many suppressions use the maven coordinates to select the applicable suppression-rule.
The Nexus analyzer AFAIK is coded to work with Nexus v2 only, so if you're trying against a Nexus v3 instance I expect it will indeed fail to work.

Isn't it possible to handle a timeout exception here in case CentralAnalyzer is down?
In the incident we had the timeout was basically given by the CI job execution limit of 1 hour.
I presume that if after let's say 1 minute, dependency-check cannot use CentralAnalyzer to fetch pom files and what else it needs from https://search.maven.org/, it would be good to skip and not block/ hang execution for a very long time.

@uvwildos Disabling just the centralAnalyzer --disableCentral as documented on the CLI configuration page should work to get dependency-check finishing successfully. It may however may trigger more False Positives (hence disabling it is discouraged for CLI-users) that you would have to suppress locally if dependency-check is configured build-breaking on vulnerabilities in your pipeline.
For libraries published on Maven Central that don't package the maven coordinate information (pom) CentralAnalyzer helps establish their proper G/A/V coordinates which help the logic to better match and reduce FP rates as many suppressions use the maven coordinates to select the applicable suppression-rule.
The Nexus analyzer AFAIK is coded to work with Nexus v2 only, so if you're trying against a Nexus v3 instance I expect it will indeed fail to work.

thanks for this important detail I did not find

@uvwildos Disabling just the centralAnalyzer --disableCentral as documented on the CLI configuration page should work to get dependency-check finishing successfully. It may however may trigger more False Positives (hence disabling it is discouraged for CLI-users) that you would have to suppress locally if dependency-check is configured build-breaking on vulnerabilities in your pipeline.
For libraries published on Maven Central that don't package the maven coordinate information (pom) CentralAnalyzer helps establish their proper G/A/V coordinates which help the logic to better match and reduce FP rates as many suppressions use the maven coordinates to select the applicable suppression-rule.
The Nexus analyzer AFAIK is coded to work with Nexus v2 only, so if you're trying against a Nexus v3 instance I expect it will indeed fail to work.

thanks a lot for this important config detail I did not get from the documentation!
So I save wasting my time on trying to get this to work!!!

There is an argument of readtimeout which at facevalue, looks like it would timeout any requests after initial connection is made. I am yet to confirm this https://jeremylong.github.io/DependencyCheck/dependency-check-cli/arguments.html

@uvwildos Disabling just the centralAnalyzer --disableCentral as documented on the CLI configuration page should work to get dependency-check finishing successfully. It may however may trigger more False Positives (hence disabling it is discouraged for CLI-users) that you would have to suppress locally if dependency-check is configured build-breaking on vulnerabilities in your pipeline.
For libraries published on Maven Central that don't package the maven coordinate information (pom) CentralAnalyzer helps establish their proper G/A/V coordinates which help the logic to better match and reduce FP rates as many suppressions use the maven coordinates to select the applicable suppression-rule.
The Nexus analyzer AFAIK is coded to work with Nexus v2 only, so if you're trying against a Nexus v3 instance I expect it will indeed fail to work.

Thanks a lot for this important config detail for disable Central I did not get from the documentation! So I save wasting my time on trying to get this to work!!!
FROM SEARCH

Sets whether the Nexus Analyzer will be used (requires Nexus v2 or Pro v3). You can.....

so the documentation thinks it should work with v3

It may have worked for the early v3 beta's, but I think already quite early in the process the legacy API was removed. In any case NexusAnalyzer is attempting to access <nexusURL>identify/sha1/<sha1 of jar>, which back in the days of Nexus v2 was https://<nexushost>/service/local/identify/sha1/<sha1 of jar> If the identify-by-sha1 XML-over-HTTP service from back in v2 is still offered on v3 it should work, but to my knowledge (and according to all Google results I get) the services were replaced by a new JSON REST-API.

authors: aikebah alexandrumd functicons ryandutton uvwildos


text:
[FP]: CVE-2022-28111 on pagehelper-5.3.2.jar

Package URl
pkg:maven/com.github.pagehelper/pagehelper@5.3.2
CPE
cpe:2.3:a:com.github.pagehelper:pagehelper:5.3.2:::::::*
CVE
CVE-2022-28111
ODC Integration
{"label"=>"Maven Plugin"}
ODC Version
8.3.1
Description
No response

Security researchers of Sonatype disagree with the NVD and Github security listing on whether it was fixed.
DependencyCheck properly reports that, according to the OSSIndex this version is vulnerable to the mentioned CVE.
https://ossindex.sonatype.org/vulnerability/CVE-2022-28111?component-type=maven&component-name=com.github.pagehelper%2Fpagehelper
https://ossindex.sonatype.org/component/pkg:maven/com.github.pagehelper/pagehelper@5.3.2

authors: aikebah dashontsai


text:
Significant Increase in CI Workflow Duration After Dependency-Check Upgrade

Following the upgrade of the dependency-check tool from version 8.4.3 to 9.0.2, we have noticed a considerable increase in our CI workflow duration. The workflow, which runs on GitHub Actions, now takes approximately 50 minutes to complete.
To perform the dependency-check, we are using the following command in our Maven build:
mvn --no-transfer-progress clean package -DskipTests -DskipITs -Dmax.cvss.score=8 org.owasp:dependency-check-maven:check -l ${{github.workspace}}/mvn-output.txt
We need assistance in understanding whether this is a bug introduced with the newer version or if there's something we might have overlooked in our codebase. Thanks in advance

It is not a bug - rather a not well documented implementation procedure. We are working on the documentation (see https://github.com/jeremylong/DependencyCheck/pull/6220).
Even prior to 9.x it would have been advisable to keep a copy of the data directory between executions. With 9.x it is almost necessary.

This change was not because we wanted to change - but more that the original data source at the NVD is being deprecated and replaced with an API that is not as fast.

oh - and yes, the API key is really important as well. Thanks @TobiX

Thanks for the prompt response, guys. I've requested the nvdApiKey. Can you confirm if the following workflow is correct? Additionally, do you have any guidance on how I should configure the cache for it? Would cache: 'maven' in the Set up JDK 17 step be sufficient?
name: Vulnerability Scan

on:
  pull_request:
  workflow_dispatch:

jobs:
  owasp-scan:
    if: github.actor != 'dependabot[bot]'
    runs-on: ubuntu-20.04
    steps:
      - uses: actions/checkout@v4

      - name: Set up JDK 17
        uses: actions/setup-java@v3
        with:
          java-version: 17
          distribution: 'adopt'
          server-id: github
          server-username: MAVEN_USERNAME
          server-password: MAVEN_PASSWORD
          cache: 'maven'

      - name: Build & scan
        id: scan
        run: |
          mvn --no-transfer-progress clean package -DskipTests -DnvdApiKey=${{secret.nvdApiKey}} -DskipITs -Dmax.cvss.score=8 \
            org.owasp:dependency-check-maven:check -l ${{github.workspace}}/mvn-output.txt
        env:
          MAVEN_USERNAME: ${{ secrets.MAVEN_USERNAME}}
          MAVEN_PASSWORD: ${{ secrets.MAVEN_PASSWORD}}

I would highly recommend using the actions/cache@v3 action to cache the data directory. Possibly something like:
 - name: Cache ODC Data
        id: odc-cache
        uses: actions/cache@v3
        with:
          path: ~/.m2/repository/org/owasp/dependency-check-data/
          key: odc-cache
          restore-keys: |
            odc-cache

Note that the above has not been tested yet.

Thanks a ton for your help on the issue! Your guidance was awesome, and I really appreciate it. You rock!

@atilsensalduz did you use the actions/cache@v3 - is what I wrote working?

Hey @jeremylong, apologies for the delay. I gave your suggestion a shot, but the workflow still took around 20 minutes. I tweaked the cache settings as follows, and now it's down to 2 minutes
name: Vulnerability Scan

on:
  pull_request:
  workflow_dispatch:

jobs:
  owasp-scan:
    if: github.actor != 'dependabot[bot]'
    runs-on: ubuntu-20.04
    steps:
      - uses: actions/checkout@v4

      - name: Set up JDK 17
        uses: actions/setup-java@v3
        with:
          java-version: 17
          distribution: 'adopt'
          server-id: github
          server-username: MAVEN_USERNAME
          server-password: MAVEN_PASSWORD
          cache: 'maven'

      - name: Get Date
        id: get-date
        run: |
          echo "datetime=$(/bin/date -u "+%Y%m%d%H")" >> $GITHUB_OUTPUT
        shell: bash

      - name: Restore cached Maven dependencies
        uses: actions/cache/restore@v3
        with:
          path: ~/.m2/repository
          # Using datetime in cache key as OWASP database may change, without the pom changing
          key: ${{ runner.os }}-maven-${{ steps.get-date.outputs.datetime }}-${{ hashFiles('**/pom.xml') }}
          restore-keys: |
            ${{ runner.os }}-maven-${{ steps.get-date.outputs.datetime }}
            ${{ runner.os }}-maven-

      - name: Build & scan
        id: scan
        run: |
          mvn --no-transfer-progress clean package -DskipTests -DnvdApiKey=${{secrets.nvdApiKey}} -DskipITs -Dmax.cvss.score=8 \
            org.owasp:dependency-check-maven:check -l ${{github.workspace}}/mvn-output.txt
        env:
          MAVEN_USERNAME: ${{ secrets.MAVEN_USERNAME}}
          MAVEN_PASSWORD: ${{ secrets.MAVEN_PASSWORD}}

     - name: Cache Maven dependencies
        uses: actions/cache/save@v3
        if: always()
        with:
          path: ~/.m2/repository
          key: ${{ runner.os }}-maven-${{ steps.get-date.outputs.datetime }}-${{ hashFiles('**/pom.xml') }}

thanks for confirming. We need to add this as an example on https://github.com/jeremylong/DependencyCheck/blob/main/src/site/markdown/data/cacheh2.md

authors: atilsensalduz jeremylong


text:
Pass by value

Enabling registration of free functions, function objects, member functions, etc. which can return instances of registered classes by value increases the applicability of Selene.
Those values should actually be copied over to Lua such that a Lua script can use them like any other value. This also reduces the risk of having objects in Lua referring to things living in C++ without being able to tell whether those things are still alive.

Work in progress at https://github.com/AnonymousProgrammerLandau/Selene/tree/PassByValue

repo: jeremyong/Selene

license: Zlib


text:
maybe open the SwipeTableViewCell's panGestureRecognizer

so I can change the isEnabled to temporary disable the gesture??

Hi @awuu - I'm interested to know why you would need direct access to the gesture recognizer?  Swiping will be disabled if you return nil from the editActionsOptionsForRowAt method in SwipeTableViewCellDelegate.

oh I return nil and it temporary disable now
forget me thanks

I'm running into a similar issue: I have a cell I use in a couple of places around my app. In some view controllers, I provide actions to the swipe cell, but not in all. In a special case, a table view controller is held in a page controller, and the pan gesture recognizer for the cell is interfering with the page swipes. Now, this case is one in which I don't provide actions for the cell.
I see that you told @awuu that returning nil from editActionsOptionsForRowAt will disable swiping, but, checking theSwipeTableViewCell code, I see that the gesture recognizers are not turned off or on based on this return value.
Is there any way of solving this without modifying the SwipeTableViewCell code? I thought of adding a property that will toggle both gesture recognizers on or off in a cell-by-cell basis.
Thanks!

authors: awuu jerkoch nameghino

repo: jerkoch/SwipeCellKit


text:
Text node

How hard would it be to do a text node only text shows like an annotation in Vizio or just putting clear text next to notes for descriptions.

I could make a node with a textarea. And class transparent or color of note.
You might have a problem with the height of the node. You can see a solution here:

https://github.com/jerosoler/Drawflow/issues/250

Is it possible to have a different type of object for text and not a node ?  There is a line object - is that not different ?
It could be a small div with a transparent BG ... does that fit into this architecture ?

They are different yes, but they are in the core of the library.
There is no easy way to add a new element.
But a node is still a div. It can be made transparent with classes and without inputs and outputs.
You can see in the demo how there is a node without inputs and outputs. And another node with another type of color.

Yep - I created a simple node that has a transparent BG with no connectors ... just need to eliminate the border and it will do the job.
Thanks !

authors: jerosoler pkasson

repo: jerosoler/Drawflow


text:
Deeplake vectore store

Added support for deeplake Vector store. Also provided Jupyter notebook with example on how to use deeplake with Llama index

sweet! will take a look soon :)

Thx a lot!, will address your comments soon

authors: adolkhan jerryjliu


text:
fix: Connection Session Management on Postgres Vector Store

Description
Improve connection session management on Postgres Vector Store
Fixes #6821
Type of Change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[X] Added new unit/integration tests
[X] Run existing unit test
[ ] Added new notebook (that tests end-to-end)

Suggested Checklist:

[X] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[X] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes

Hi @Disiok
Honestly since I'm not sure what actually happened, since I'm not an expert in python nor SQLAlchemy.
But when I played around with the PostgrestVectorStore, I found this #6821 bug, and the change I made here solved the bug.
My first assumption is the bug caused by dirty session management. As I explain on the bug page, SupabaseVectorStore, so I follow their connection management and implemented it here. And it solves the bug.
https://github.com/supabase/vecs/blob/095b3c3bea872786f6666877de3f2184bb03e6a7/src/vecs/collection.py#L292

Oh also, the delete function on the PostgreVectorStore is actually failing after the ref_doc_id implementation.
Probably no one noticed this before because the use test will be skipped if no RDS database is supplied when executing the test.
This PR also solves this delete function issue.

The linting failed because the line is too long, but I don't think there's a way make it shorter
f"DELETE FROM public.data_{self.table_name} where (metadata_->>'doc_id')::text = '{ref_doc_id}' "

Any idea @Disiok, or can we ignore this for now?

@rendyfebry actually I was just going to ask about this line
How is it working? Since the metadata does not store a doc_id attribute. Instead, the schema should probably be specifically storing node.ref_doc_id for this to work 🤔

@rendyfebry For some reason I can't push to the PR branch, but you can just split that line into two lines
                stmt = sqlalchemy.text(
                    f"DELETE FROM public.data_{self.table_name} where "
                    f"(metadata_->>'doc_id')::text = '{ref_doc_id}' "
                )

Can you also update the changelog? I'll merge once that is done 👍🏻

Thank's @logan-markewich It works. Feel free to check again.

authors: logan-markewich rendyfebry


text:
ModuleNotFoundError: No module named 'util_io'

Thank you very much for this project. Unfortunately, I am unable to reproduce the results.
I'm running the script in the following way:
python image_translation_infer.py --model_path="/Documents/weights/256" --image_hw=256 --input_tensor_name="sources_ph" --output_tensor_name="custom_generated_t_style_source:0" --input_image_path="/Documents/art/photo.png" --output_image_path="/Documents/art/"
and get the following error:
Traceback (most recent call last):
File "image_translation_infer.py", line 35, in
import util_io
ModuleNotFoundError: No module named 'util_io'
Could you please look into that?
Python 3.6.4. Anaconda, tensorflow 1.10, keras 2.2.2

I believe this issue is similar to #3 where system PYTHONPATH is not set correctly. The path is added as sys.path.append('./') so you'll need to run the infer script from the TwinGAN directory.
If you do the following it should work. Let me know if it doesn't. Thanks!
# Assume you're in TwinGAN/inference/
cd ..  # Go to the TwinGAN directory.
python inference/image_translation_infer.py --YOUR FLAGS...


text:
Does snapshot work with native functions?

If I add a native C function wrapper to the global JS scope then load and execute a snapshotted function that invoke the said native function wrapper inside its function body, will it work?

Yes, since the functions (bindings) are identified by names. Hence if a snapshot call f() function, the resolving of f happens at runtime.

Nevermind the problem was the buffer size is too small.

repo: jerryscript-project/jerryscript


text:
target: mbedos5: release jerry_value_t

JerryScript-DCO-1.0-Signed-off-by: Marko Fabo mfabo@inf.u-szeged.hu

Please, give a more descriptive commit message:

the patch clearly does not deal with releasing values only (it seems that there was also a bug in arg checks), and
the commit message does not tell why the releasing of f is necessary (e.g., what issue rose from it not being released).

If you do not start the eventloop in mbed, just call the jerry_cleanup function at the end of the application, you got this error message:
ICE: Assertion 'ECMA_STRING_IS_REF_EQUALS_TO_ONE (string_p)' failed at ../../jerry-core/ecma/base/ecma-literal-storage.c(ecma_free_string_l. Error: ERR_FAILED_INTERNAL_ASSERTION
@akosthekiss I am sorry about the empty message, but I thought it is a simple bug, and my PR is correct and clear solution for that. Probably i was wrong, and there is an other way to fix this error.
I'm working on the solution, and i will update the PR soon.

@akosthekiss @LaszloLango
updated

@akosthekiss
If you delete the jerry_release_value from dropCallback, you get the same error.
My new update possibly make the code more clear, please check.

authors: akosthekiss fbmrk


text:
[Draft] Hack fix to wayland key grab

Instead of using rdev's simulate function to pass through mouse button presses when not preforming gestures. I am instead sending the mouse keycode to a modified python script I ripped from the libinput repo. The script uses evdev library to create a virtual mouse and then presses the gesture key. Since the device is created after mouse-actions starts to grap input, mouse-actions is not aware of the event.
This allows me to use mouse-actions on wayland and the gesture button is not stolen.
I don't intend for this pull request to be accepted. I only want to show a workaround in case other people want it.
There exists rust bindings for evdev which could be used for a more clean solution. I might implement that at one point, but I have never coded in rust before.
If you want to try my hack you have to build my pull request, have python-evdev installed, and move the script play-event.py to a folder in your PATH.

This is a good proof of concept for the proposed (rust based) resolution to https://github.com/jersou/mouse-actions/issues/8
But the big problem we have on Wayland is that all of the Wayland input handling apps take the same approach of exclusively grabbing the evdev input devices, meaning we can only use ONE input handling app: https://github.com/jersou/mouse-actions/issues/17 ... So even if we fix mouse-actions we're still totally screwed if we need to use anything else as well.

@pallaswept yeah the situation is pretty bad. I think libei is supposed to be a solution to emulated input in wayland. I don't know how ready libei is or if anyting have implemented it.

@pallaswept yeah the situation is pretty bad. I think libei is supposed to be a solution to emulated input in wayland. I don't know how ready libei is or if anyting have implemented it.

Thanks very much for the tip!! I hadn't heard of it at all so I'm going to read into it.
And thanks again for this PR, it's nice to have a working POC.

authors: Spaceslug pallaswept

repo: jersou/mouse-actions


text:
Run tests on CircleCI

💁 Tests are great, especially for dynamic programming languages. These changes add an initial build configuration for CircleCI using the "latest and greatest" available versions of Clojure and Ruby.

@sgerrand: How did I never see this?! I've just updated this project so that it runs again and I'm looking at getting it live again.

Live to give. ™

Closing this (a million years later), as I adapted what you wrote via GitHub Actions. 🙇🏻

I'll take that as a win.

@sgerrand: Hopefully we get another win on Saturday

Sunday night should be a big one.

authors: jesse-c sgerrand

repo: jesse-c/Brewfile