Set without expanding the Expire time

Its not clear to me how to ".Set()" and not start the expire time over again.
Can someone point to reference or doc or advise ?

I think it's not possible. As a workaround, you could get the item and reset it with the modified ttl value.
var ttl time.Duration = cache.DefaultTTL

if previousItem := cache.Get(key); previousItem != nil {
      ttl = previousItem.ExpiresAt().Sub(time.Now())
}

cache.Set(key, value, ttl)

At present not updating an existing expiration time isn't possible, though it might a good feature for one of the future versions. We could add a new ttlcache.PreviousTTL that would indicate that the previous TTL should be set after an update.

Implemented in #121


Add optional password secret

Reworked the solution as suggested in PR71.
Still WIP. With the current implementation (using HELM's built-in htpasswd function), login with configured credentials is not possible. I suspend this to be a result of HELM's htpasswd not using bycrypt.

I got auto-tagged as a codeowner reviewer, but the latest comment says this is still a WIP. Let me know if / when you'd like me to review ;)

Hi,
I’ve added a bit of context to the PR in the p3r Slack as there’s room for discussion

Am 17.08.2021 um 07:13 schrieb David Young @.***>:
I got auto-tagged as a codeowner reviewer, but the latest comment says this is still a WIP. Let me know if / when you'd like me to review ;)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/portainer/k8s/pull/72#issuecomment-899999046, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABVPBPNYFVDWHBMM4TT5M5TT5HVV7ANCNFSM5CH3563A.


fix types and modernize some devices

@keybase/react-hackers this removes all implicit any types from devices and switches the paper key screen to hooks
also adds the hoist non react statics so we can have better acting hocs (if any)

@keybase/react-hackers bump


add new CUAV Nora board

Rebased version of  #14272.
@CUAVcaijie can you test this? I added some changes in abe16d7242d686ec11627b8a32eee6699cc1ccad.
TODO:

[ ] add to CI
[ ] adis16470 driver
[ ] icm20649 driver

Hi @bkueng You guys got a plan to add docs for this board?

重新定位的版本 ＃14272。
@CUAVcaijie你可以测试吗？我在abe16d7中添加了一些更改。
去做：

[ ]  添加到CI
[ ]  adis16470驱动程序
[ ]  icm20649驱动程序

Ok

重新定位的版本 ＃14272。
@CUAVcaijie你可以测试吗？我在abe16d7中添加了一些更改。
去做：

[ ]  添加到CI
[ ]  adis16470驱动程序
[ ]  icm20649驱动程序

when I use CUAV Nora to debug ADIS16470, when I use int SPI :: transferhword (uint16_t * send, uint16_t * recv, unsigned len) function, the program is stuck.

重新定位的版本 ＃14272。
@CUAVcaijie你可以测试吗？我在abe16d7中添加了一些更改。
去做：

[ ]   添加到CI
[ ]   adis16470驱动程序
[ ]   icm20649驱动程序

when I use CUAV Nora to debug ADIS16470, when I use int SPI :: transferhword (uint16_t * send, uint16_t * recv, unsigned len) function, the program is stuck.

hard fault?

Rebased version of #14272.
@CUAVcaijie can you test this? I added some changes in abe16d7.
TODO:

[ ]  add to CI
[ ]  adis16470 driver
[ ]  icm20649 driver

when I use CUAV Nora to debug ADIS16470, when I use int SPI :: transferhword (uint16_t * send, uint16_t * recv, unsigned len) function, the program is stuck.

重新定位的版本 ＃14272。
@CUAVcaijie你可以测试吗？我在abe16d7中添加了一些更改。
去做：

[ ]    添加到CI
[ ]    adis16470驱动程序
[ ]    icm20649驱动程序

when I use CUAV Nora to debug ADIS16470, when I use int SPI :: transferhword (uint16_t * send, uint16_t * recv, unsigned len) function, the program is stuck.

hard fault?

It is possible that the H7 library does not yet support 16-bit SPI transmission

The board has been added already. Can this be closed?

The board has been added already. Can this be closed?

Yes, although one missing piece in master is still supporting all 14 PWM. Let's bring that in separately.


minor fix: update the new interface name

Commit Message: minor fix: update the new interface name
Additional Description:
Rename createFilterServerFactoryFromProto to createFilterFactoryFromProto.
#27621 introduced the createFilterServerFactoryFromProto method. But The Factory in the method is means factory of filter, not Factory in the FactoryContext or ServerFactoryContext. So the previous naming is little weird.
Risk Level: low.
Testing: n/a.
Docs Changes: n/a.
Release Notes: n/a.
Platform Specific Features: n/a.

cc @tyxia

/assign @tyxia

didn't found a perfect naming orz. At least this naming has no ambiguity and the users of factory still could use the unified createFilterFactoryFromProto.

@wbpcode loosk like gcc build is unhappy.
/wait

Still hope we can finally resolve the naming problem.
Another problem is that in current impl, if the users want to use the per route extension matcher, the users need implement two version createFilterFactoryFromProto explicitly, even they only use some shared methods of server context and factory context.
We still need to find the a way to simplify it just like the DualFactoryBase simplifying the development of dual http filter. But this will need more time and will be a big PR, so just pending it to my TODO list for now. orz.
At least this minor patch eliminate the ambiguity.

At least this minor patch eliminate the ambiguity, we can land it first. hope the ci is happy.

Still hope we can finally resolve the naming problem.
Another problem is that in current impl, if the users want to use the per route extension matcher, the users need implement two version createFilterFactoryFromProto explicitly, even they only use some shared methods of server context and factory context. We still need to find a way to simplify it just like the DualFactoryBase simplifying the development of dual http filter. But this will need more time and will be a big PR, so just pending it to my TODO list for now. orz.

Sure, that is because currently in Envoy creating from factoryContext is default approach every http filter needs to have , even though it can just be a no-op (return nullptr) like the test filter I added.
During the development of this feature. I do see lots of room of clean-up and reducing the tech debt. Right now there are just too many contexts in Envoy and probably they are in diamond or unnecessary-complicated inheritance


Send different completion item kind

Motivation

There are no such things as functions in Ruby.
Test plan

Do you have any thoughts on how to write an automated test for this?
Should I have a way to assert this in the completion results?
I think it dictates what icon VS Code uses to show completion items, and my Vim
plugin also shows the text Function or Method.

Do you have any thoughts on how to write an automated test for this? Should I have a way to assert this in the completion results?

Some thoughts:
obj.f # complete: Method foo, Constant fuzz, ...


EDU-912 Mark tiles match found in memory plugin

Closes https://educandu.atlassian.net/browse/EDU-912

Looks very nice!


Introduction to the Extension component-library

Description
Create an educational video to introduce the extension component-library
Technical Details
https://www.notion.so/Extension-component-library-7dc2115bfcf74a7db37cd322f231e675?pvs=4
Acceptance Criteria

After watching an engineer should be able to effectively and efficiently use the component-library in the extension

Hey team! Please add your planning poker estimate with Zenhub @Akatori-Design @brianacnguyen @garrettbear

This has been completed and can be found here
https://www.notion.so/Extension-Engineer-8da34c7887fe4f2182305a5d576d199c?pvs=4
https://user-images.githubusercontent.com/8112138/236995848-302dffaa-4180-4508-aa1f-4e414ac956c7.mov


Add a strictly depth calculator

Look at how mosdepth does this with bitvecs and apply it with par_io.

Done, see v0.4.0


Valorant watcher throwing errors

Hey im getting this error,
🔗 Streamer:  https://www.twitch.tv/winghaven
(node:2828) UnhandledPromiseRejectionWarning: TimeoutError: Navigation timeout of 30000 ms exceeded
at Promise.then (/home/pi/Valorant-watcher/node_modules/puppeteer-core/lib/LifecycleWatcher.js:142:21)
-- ASYNC --
at Frame. (/home/pi/Valorant-watcher/node_modules/puppeteer-core/lib/helper.js:111:15)
at Page.goto (/home/pi/Valorant-watcher/node_modules/puppeteer-core/lib/Page.js:672:49)
at Page. (/home/pi/Valorant-watcher/node_modules/puppeteer-core/lib/helper.js:112:23)
at viewRandomPage (/home/pi/Valorant-watcher/app.js:175:20)
(node:2828) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:2828) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

#19 #41


Duplicate targets in FLTPLANGO

Howdy. Here's a video that best exhibits the issue I'm having: https://youtu.be/Cqjrd-AVHYw
The screen grab is from my Galaxy S4 but the same exact glitch happens in my Ipad 2 3g , but I don't know if that's sufficient evidence to say the stratux is culprit.
Basically, if I'm not in range of a GBT and picking up raw 978 from airborne aircraft, I'll have 2 targets for each plane following/right on top of each other, one will show whatever the plane is squawking (ie 1200) and the other the actual tailnumber for the aircraft. Haven't tested with 1090es yet (trying to figure how to enable that still pardon the noobness).

I don't recognize the software on the S4, is the same package as on your iOS?  It could be a fault
of the display software since stratus just passes the messages it receives through wifi to the tablet.
There is a known glitch in ADSB in systems where if you do not have ADSB out, you may see two
targets for yourself.  Also, if you have both 978 and 1090 running, each actual target should have the
same ICAO identifier and thus the display software should merge the two data streams to a single
target

Yes. It's fltplango, just like in the Ipad, same glitch.

Just tested it out with your data @PM-ME-YOUR-EMPENNAGE.
When I tap on the targets (there were 10 in your file) and click "clear breadcrumbs", it actually clears the trail. Their traffic icons are way too big, so most of them overlap. In the settings menu there is "Toggle breadcumbs - Turns the breadcrumb trail on/off" which doesn't specify if checked is on or off, but in either position it displays the breadcrumbs - so that is obviously broken.
Their "Clear/remove breadcrumbs" doesn't work either. Not sure the difference between clearing and removing breadcrumbs, but there are breadcrumbs still displayed when using that feature from the menu.
Looks like some unimplemented features in FltPlan Go.
If you look on Google Play, their changelog is "•added ADS-B traffic" as of Sept 7, looks like they're just working out the bugs. I'll send a bug report.

"Hi Chris,
Thanks for letting us know.
The traffic on our Android Go app is a work in progress.
There are many features that need to be added and many bug fixes to be made.
Look for continuing enhancements over the next weeks and months.
Hope this explains."

Looks like fltplango pushed an update a couple days ago. Appears that the breadcrumbs are not turned on by default. The traffic icons are still way too large to be useful and sometimes when there are too many traffic targets, the app crashes.


test(cli): add test for delete-project extension

Description of changes:
Increase test coverage of amplify-cli package.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

I'd include tests for remove-env-from-cloud.ts and path-manager.ts into this PR for Codecov.

just saw this is marked as WIP. Feel free to @ me when it's ready and I can take another look.
In the future, you can create a draft PR if it's not ready to be reviewed yet.

@edwardfoyle Thank you for comment! This PR is ready to ship and I'll add more tests related to the test code I put into this PR separately.


fix: reenable pdm lock check hook for all packages

pdm lock check doesn't run for any of the packages after the monorepo
move
only the root level pdm lock check (which only has pylint and pre-commit
in it) actually runs.
keeping only one single top-level pdm.lock file at the repo root level
may work for simple development tasks (e.g. pdm sync from repo root),
but any pdm functions that depend on more granular pdm.lock input (e.g.
pdm export) will break when run from an individual package folder if the
individual pdm.lock files in each package folder is not kept in sync as
well.
adding all packages back into the .pre-commit config fixes this
a new dedicated Github Action step to force a pdm.lock check on every PR
also has been added.

Looks good, although I think the datajunction-query block needs to be uncommented. And good catch!

The pdm lock check for datajunction-query will be enabled in a separate change.

LGTM thanks @cwegener! Just one comment. Also just to confirm, if I wanted to locally run these checks to make sure the CI will pass, that would just require changing into the datajunction-(server|query|reflection)/ directory and running:
pdm lock --check --project datajunction-(server|query|reflection)

Is that right? Might be useful to add to the contributing guide.

That's a great question.
For now, the most straight-forward answer would be for a developer to perform the exact same test that the dedicated GitHub action runs, which is:
pre-commit run pdm-lock-check --all-files

The next most important thing would be to strongly encourage all new developers to make sure that the pre-commit hook is installed in their local dj repo by running pre-commit install, which can only be done through proper documentation and education.
On a bigger picture, there is the overall question of how the tooling should integrate across the different execution layers (pdm, git hooks, Github runners, generic task runners e.g. make)
Trying to simplify all these different execution layers and make the configuration of them easy to maintain is still a bit of a challenge and I don't have a good answer at the moment. Using make as a generic task runner for abstraction is not really my favourite option (as previously mentioned) since designing and maintaining a set of fully featured makefiles with all the bells and whistles needed is not easily achieved. I'm still investigating and trialing some other options as a generic task runner, mainly https://taskfile.dev
Once I have a proven baseline for a generic task runner that I'd be comfortable with recommending, I'll be making that recommendation. :smile:


update docs surrounding IAM permissions, add terraform example

This PR updates some documentation, and adds a docs/ folder with an example of how to create the required minimum permissions policy and IAM user for AWS to run this tool!

One thing I wanted to add here @jpetrucciani, maybe I can just add it in myself, is the build-args. I am building the container like this: docker build --build-arg AWS_ACCESS_KEY_ID=<ACCESS_KEY> --build-arg AWS_SECRET_ACCESS_KEY=<SECRET_KEY> -t ec2instances.info .
Is that what you're doing?

Sorry, didn't get a notification for this!
Currently I've been just running with locally configured creds with the minimum user like in this example, with the env configured through nix!
If you wanted to use this through docker, I'd probably say we might want to rethink the dockerfile a bit - iirc, build args will leave traces in the container, potentially exposing your creds. This is tricky though since the docker image looks to want to include the data pulled in the final image. We could make use of multi-stage builds for that, if that's the case - although I'd probably want to split out the actual environment used there (into somewhat of a 'cache' image), pull the data + render into one stage, then build the resulting container with the rendered HTML/CSS/JS. Or, if you want to run it locally just to pull the data, you could build the dockerfile to just be the environment, and have it pass in env vars that can be used to pull at runtime.
Although at that point it becomes a question of how you want to host things - if it can all be statically built out (perhaps on a cronjob?), you may just want to drop it into s3 + cloudfront

That's a great point about build-args, hadn't thought of that. Environmental variables are the way to go with docker. I was looking at #610 and I realized that we can set the host via envvars as well.
Anyway this is good to merge I think. Let me just add a little info from this conversation to the README and I'll put it through tonight. Thanks a lot for the additional details.

That's a great point about build-args, hadn't thought of that. Environmental variables are the way to go with docker. I was looking at #610 and I realized that we can set the host via envvars as well.
Anyway this is good to merge I think. Let me just add a little info from this conversation to the README and I'll put it through tonight. Thanks a lot for the additional details.

I am actually doing the samthing.
docker build --build-arg AWS_ACCESS_KEY_ID=$AWS_ACCESS_KEY_ID --build-arg AWS_SECRET_ACCESS_KEY=$AWS_SECRET_ACCESS_KEY  -t ec2instances.info .

Yep! That's what my command looks like. Perhaps then I give folks that option but give a warning saying it is insecure https://pythonspeed.com/articles/docker-history/

BTW, I notice http port doesn't work once I run  docker run -d -p 8080:8080 ec2instances.info. Seems the it's the binding issue? Did you guys see the same problem? When I docker exec into the container and curl 8080 and it works.

Yes, by default the python server works over <IP_ADDRESS>, see #610. Happily you can override this with an environmental variable passed during docker run. You can do docker run --env HTTP_HOST=<IP_ADDRESS> to host over the loopback interface which will then expose the app on http://localhost:8080.

@EverettBerry Thanks for the pointer. It works!  Should we update the doc or update default HTTP_HOST to <IP_ADDRESS>?

I think update the docs because setting the port to <IP_ADDRESS> by default may cause the local site to not load for folks not using docker or on mac and windows.


Source code fix for modern version of Python and scikit-learn

Dear project maintainers,
crab is a very good starting point for machine learning researchers that would like to implement recommender system in Python. However, there are a couple of issues in the source code that do not let the code working out-of-the-box.
There are people already discussing about these issues.
I fixed the code in order to work with current versions of Python and scikit-learn. I tested on my machine (Mac OSX El Capitan, 10.11.4) with Python 2.7.10 and it works.
I hope you are going to accept this pull request.
Bests,
Vincenzo

I believe the project is no longer maintained, so you fork could be a great starting point for updating it.

Thanks Hiyorimi. Unfortunately, I am not in the recommendation system area,
I just needed for a specific case. Bests, Vincenzo
On Thu, Jun 16, 2016 at 11:00 PM, Hiyorimi<EMAIL_ADDRESS>wrote:

I believe the project is no longer maintained, so you fork could be a
great starting point for updating it.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/muricoca/crab/pull/108#issuecomment-226612259, or mute
the thread
https://github.com/notifications/unsubscribe/AK9djVV_-8M-HS0dO1CwiF-nVWYPfbgxks5qMblSgaJpZM4IuUxh
.

--
Vincenzo Manzoni
Head | Control and Analytics on Industrial Processes | Tenaris
President | Information Engineering Committee | Ordine degli Ingegneri di
Bergamo
<EMAIL_ADDRESS>| vincenzomanzoni.com |
linkedin.com/in/vincenzomanzoni


page.overlay.append - AttributeError: 'property' object has no attribute 'append'

Flet 0.22
Windows 10
The code is from the documentation
Have the following code and and i am receiving  AttributeError: 'property' object has no attribute 'append' on the following line
page.overlay.append(pick_files_dialog)
import flet as ft
def main(page: ft.Page):
def pick_files_result(e: ft.FilePickerResultEvent):
selected_files.value = (
", ".join(map(lambda f: f.name, e.files)) if e.files else "Cancelled!"
)
selected_files.update()
pick_files_dialog = ft.FilePicker(on_result=pick_files_result)
selected_files = ft.Text()

page.overlay.append(pick_files_dialog)

page.add(
    ft.Row(
        [
            ft.ElevatedButton(
                "Pick files",
                icon=ft.icons.UPLOAD_FILE,
                on_click=lambda _: pick_files_dialog.pick_files(
                    allow_multiple=True
                ),
            ),
            selected_files,
        ]
    )
)

ft.app(target=main)

It should certainly because the code is wrongly formatted(indentation?).
I reformated it as below and it works(Flet v0.22.0):
import flet as ft

def main(page: ft.Page):
    def pick_files_result(e: ft.FilePickerResultEvent):
        selected_files.value = (
            ", ".join(map(lambda f: f.name, e.files)) if e.files else "Cancelled!"
        )
        selected_files.update()

    pick_files_dialog = ft.FilePicker(on_result=pick_files_result)
    selected_files = ft.Text()

    page.overlay.append(pick_files_dialog)

    page.add(
        ft.Row(
            [
                ft.ElevatedButton(
                    "Pick files",
                    icon=ft.icons.UPLOAD_FILE,
                    on_click=lambda _: pick_files_dialog.pick_files(
                        allow_multiple=True
                    ),
                ),
                selected_files,
            ]
        )
    )

ft.app(target=main)

If you still face issues, please report with your Flet version and virtual environment details.

Thanks, I'll check it out.
On Mon, Apr 22, 2024 at 10:52 AM TheEthicalBoy @.***>
wrote:

It should certainly because the code is wrongly formatted(indentation?).
I reformated it as below and it works(Flet v0.22.0):
import flet as ft
def main(page: ft.Page):
def pick_files_result(e: ft.FilePickerResultEvent):
selected_files.value = (
", ".join(map(lambda f: f.name, e.files)) if e.files else "Cancelled!"
)
selected_files.update()
pick_files_dialog = ft.FilePicker(on_result=pick_files_result)
selected_files = ft.Text()

page.overlay.append(pick_files_dialog)

page.add(
    ft.Row(
        [
            ft.ElevatedButton(
                "Pick files",
                icon=ft.icons.UPLOAD_FILE,
                on_click=lambda _: pick_files_dialog.pick_files(
                    allow_multiple=True
                ),
            ),
            selected_files,
        ]
    )
)

ft.app(target=main)
If you still face issues, please report with your Flet version and virtual
environment details.
—
Reply to this email directly, view it on GitHub
https://github.com/flet-dev/flet/issues/3073#issuecomment-2068848772,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXYIYQER7XIBIXKZ6KRSTPTY6TFV7AVCNFSM6AAAAABGRRNH5GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANRYHA2DQNZXGI
.
You are receiving this because you authored the thread.Message ID:
@.***>


Colours are given with too many digits for CMYK

Colours are defined in exports like this:
\definecolor{color0}{rgb}{0.941176470588235,0.501960784313725,0.501960784313725}

This causes ! Number too big. when using such colours in a CMYK context, e.g., like this:
\documentclass{standalone}

\usepackage{xcolor}
\selectcolormodel{cmyk}
\definecolor{color0}{rgb}{0.941176470588235,0.501960784313725,0.501960784313725}

\begin{document}\end{document}

The reason for this is simply that the numbers have too many digits – which are irrelevant for application anyway.

This looks like a bug in xcolor/cmyk. You can work around it by setting, e.g., float_format="{:.10g}" on save. I've reported this to Uwe Kern, the maintainer of xcolor.


Truncate a string

Challenge Truncate a string has an issue.
User Agent is: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/58.0.3029.110 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

function truncateString(str, num) {
  // Clear out that junk in your trunk
  if (isNaN(num)) {
    return str;
  }
  var newStr="";
  var endPos=0;
  if (num > 3) {
	num -= 3;
  }
  //else
  //{
//	num -= 1;
 // }
  //return num;
  newStr = str.slice(0,num);
  //return newStr;
  newStr = newStr.concat("...");
  return newStr;
}

truncateString("A-tisket a-tasket A green and yellow basket".length);

My Code is failing this test, which is incorrect. How can the function return a String when it is being passed a number?
truncateString("A-tisket a-tasket A green and yellow basket", "A-tisket a-tasket A green and yellow basket".length) should return "A-tisket a-tasket A green and yellow basket".

@migscay have you tried sharing your work with the HelpJavascript chatroom, for tips and guidance in solving this challenge - https://gitter.im/FreeCodeCamp/HelpJavaScript
However it sounds to me like you need to review the instructions:

Truncate a string (first argument) if it is longer than the given maximum string length (second argument). Return the truncated string with a ... ending.

ThanK you! I misread the instruction.


error on install app

Oh hi, yes we have some problems for now. But you can use developer installation as well.

Hi for now we just removed native app support.


[github-actions] add border router tests

This commit adds Border Router tests in Github Actions.

@jwhui
I guess the Github Actions of this project are blocked too.

@jwhui
I guess the Github Actions of this project are blocked too.

I have submitted another GitHub support ticket.


Build latest Caffeine on CI

Fixes #1110
We can see how this goes for some time before making it a required job.

This is pretty cool!
BTW we have a similar job in NilAway that posts the diff of diagnostics produced by NilAway on stdlib on HEAD vs main for each PR: https://github.com/uber-go/nilaway/blob/main/.github/workflows/golden-test-run.yml
Example: https://github.com/uber-go/nilaway/pull/281#issuecomment-2351988905
We have generally found this to be very useful (although it's partly because NilAway is less mature)

This is pretty cool!
BTW we have a similar job in NilAway that posts the diff of diagnostics produced by NilAway on stdlib on HEAD vs main for each PR: https://github.com/uber-go/nilaway/blob/main/.github/workflows/golden-test-run.yml
Example: uber-go/nilaway#281 (comment)
We have generally found this to be very useful (although it's partly because NilAway is less mature)

Very cool!  I think once this bakes a bit we could implement some similar logic here.  We would just have to grep out the NullAway warnings from the Gradle build output, though possibly we could just use NullAway's serialization support (thus far used only for NullAway Annotator) to make things more robust.

We would just have to grep out the NullAway warnings from the Gradle build output

You can use problem-matcher.json to annotate the pr. Caffeine’s build should fail unless you decide to mark the gha step as continue-on-error.

We would just have to grep out the NullAway warnings from the Gradle build output

You can use problem-matcher.json to annotate the pr.

Very cool!  From what I'm reading, problem matchers help to annotate source code in the PR.  But here, we have a PR on NullAway, and the errors / warnings we would be looking for would be reported on Caffeine's code.  Would this approach still work?

Caffeine’s build should fail unless you decide to mark the gha step as continue-on-error.

Ah ok, so NullAway is running at error level now, cool!  That makes things easier, we can just look for a job failure.

Would this approach still work?

I believe it would simply be unlinked to the file but still be an annotation attached to the pr. It might look like the ones you have currently wrt the pending ubuntu upgrade.


Hyperdrive: local development docs

Blocks on https://github.com/cloudflare/workers-sdk/pull/4900
This PR:

[x] Documents the new HYPERDRIVE_LOCAL_CONNECTION_STRING in the wrangler docs
[x] Adds a new /hyperdrive/configuration/local-development/ page to document how to test and use Hyperdrive in local dev
[x] Documents the precedence over localConnectionString in wrangler.toml

cc @Maddy-Cloudflare


clear stack on re selecting menu option

Hello,
first of all you have done awesome work.
i want to use your library and find it very useful.
i need one modification as in Instagram
when user opens application and landed to Main home tab here user select option and go to home subpage1 and again select option from here and go to home subpage2
now if user re select home tab again user should landed to MainHome tab and subpage1 and subpage2 should automatically remove from fragment stack
waiting for your kind response
thanks

Hey @rizwan321 ,
Thanks for the kind words and sorry for my late response.
Basically, when the user clicks for example Home twice, should be navigated to the starting screen of Home, correct?
I was thinking on implementing this functionality as well, but haven't had the time. I'll do it really soon, but this is pretty trivial, you can probably do it yourself. In any case I will address this issue when I implement it.
Regards,
Stefan

@smihajlovski thank you for your assistance.
i have done it myself.
i just replace Bottom Navigation Menu with TabLayout.
in the method on onTabReselected i just added below lines
// if user reselect tab then clear all fragments and pop last fragment on top
while (stacks.get(currentTab).size() > 1) {
stacks.get(currentTab).pop();
}
    Fragment fragment = stacks.get(currentTab).elementAt(stacks.get(currentTab).size() - 1);
    removeFragment(getSupportFragmentManager(), fragment, currentFragment);
    assignCurrentFragment(fragment);

This has been now implemented in cff1de992ad9fc8e92717e38ffa588e3ffff9128 commit.


Add new phone operator code

hi there,
can u add a new phone operator code for Kyrgyzstan.
new code 50
+996 50X XXX XXX

what about my issue


added stylecheck on new commits

Which issue this PR addresses:
Adds stylecheck as TODO was suggesting. Only enabling on commits coming after the parent commit of that PR.

I changed the style check options slight because :

they were not correct (did not do what they were supposed to)
these checks are according too opinionated according to staticcheck.io . So I set it back to the default

This option sets which checks should be enabled. By default, most checks will be enabled, except for those that are too opinionated or that only apply to packages in certain domains.

There is something fishy with golangci-lint in the pipeline. It does not seem to take the config file into account. It works fine on my laptop with my own version of golangci-lint (v 1.42 while the one vendored is 1.32) though. I think we should update it


Update flightplan.py

Added function plot_usurf to help plot the surface wind speed, which is another quantity of interest when flight planning.

I think this is a nice addition! 👍
Could you please run ruff format to bring the python syntax to pass our check?


Deduplicate results

(disclaimer: Hopefully you don't mind all the issues I've been filing. I really enjoy this project so I'm trying to list the things I'm hitting/wishing for :smile: )
This could be a bug or feature-request depending on how you look at it.
Is your feature request related to a problem? Please describe.
navi duplicates results. I.e. I have a personal repository of .cheat files, as well as a shared repository at work of .cheat files. There are .cheat files I have in my personal repo, that are also applicable to my work/co-workers, however I don't want to "move" my personal cheat files into the work repository since I may not be at that job forever, and co-workers can also make changes that my not fit my personal flows.
However, having the same cheats in two places means I get duplicate results cluttering the navigation window and taking time to figure out if they really are duplicates, or just similar (we have a lot of similar cheats as well, that look like duplicates at first glance).
Describe the solution you'd like
Results are filtered. Exact duplicates are only displayed once.
Describe alternatives you've considered
Only alternatives are manually curating cheat files, or living with duplication.
Additional context
Implementation wise, I'd prefer each cheat line to be looked at independently, and not cheat files as a whole. I.e. my personal git.cheat isn't an exact duplicate of my work's git.cheat, however many cheats inside that file are.
I would imagine hashing the the lines would solve this and would be fast enough that it wouldn't impact performance. I'm sure there's some fashion in which the cheats could be lazily hashed as well, (since for example there may be 10,000 cheats, but only 100 or so can fit on a page at once).
I'd be happy to take a look at the impl for this given a starting point of where to look.

You could create a hashset here https://github.com/denisidoro/navi/blob/master/src/cheat.rs#L124 and if the line is already there, just continue.
I won't be able to implement it today, but feel free to do so and I'll review and merge it :+1:
I won't require any advanced benchmarks, but to whoever implement this, please make sure there's no noticible lag for, say, 1500 lines + there's no substantial increase in RAM.

Actually the hashset should be create outside this function because we want to maintain the values from previous calls of this function.

I'll take a stab at it after work today. I'll dig around too and see the flow and where the best place to do some hashing may be, I just needed a jumping off point :wink:

Maybe consider: https://docs.rs/bloom/0.3.2/bloom/

The problem with a bloom filter is you want the opposite guarantee it provides. A false positive in navi terms would be not adding a cheat to the displayed list, when in there are actually no duplicates. Having the occasional false negative (adding a duplicate by accident) would be a better outcome. I try and look at some benchmarks to see how big of a difference it makes

Don't worry about performance too much, though


changing use of qPCR to cDNA

edited 'qPCR primers' to 'cDNA primers' throughout. to close #17.
qPCR primers doesn't make sense, as we do qPCR also on genomic DNA

And updated AgamPrimer to 0.5.9


Fix an issue when generating ExternalEntity from ExternalNode

Use group/apiversion for OwnerReference when creating ExternalEntity from ExternalNode
Add unit tests for ExternalNodeController

Signed-off-by: wenyingd<EMAIL_ADDRESS>

/test-all

The unit test failure is a known issue on main branch: https://github.com/antrea-io/antrea/issues/4294, no need to rerun.


MH-13023 Let default value fulfill requirement

This fixes a problem where a required metadata field is not considered filled even though there is a default value set.

@staubesv Fixed, thanks for noticing!


Support shared libraries

This supports the new iCloud shared libraries.
Example usage:
for library_name, library in api.photos.libraries.items():
    print(f'Library: {library_name}')
    for photo in library.all:
        print(f'{photo.asset_date} {photo} {photo.filename}')

@pneff How do the proposed changes align with original pyicloud lib? There is a desire to remove the fork and go with upstream pyicloud (https://github.com/icloud-photos-downloader/icloud_photos_downloader/issues/179) and I am wondering if this change may further diverge the fork if it does not match with similar changes in upstream. WDYT?

@pneff a chance something changed in the last month or so as Apple rolled out their new icloud website? Consistently now getting errors with your patch eg. --list-libraries
# /usr/bin/icloudpd --list-libraries
2023-03-02 23:00:21 DEBUGAuthenticating...
iCloud Password:
2023-03-02 23:00:26 INFO Two-step/two-factor authentication is required!
0: SMS to ********XX
1: Enter two-factor authentication code
Please choose an option: [0]: 1
Please enter two-factor authentication code: XXXXXX
2023-03-02 23:00:39 INFO Great, you're all set up. The script can now be run without user interaction until 2SA expires.
You can set up email notifications for when the two-step authentication expires.
(Use --help to view information about SMTP options.)
Traceback (most recent call last):
File "/usr/bin/icloudpd", line 33, in <module>
sys.exit(load_entry_point('icloudpd==1.7.2', 'console_scripts', 'icloudpd')())
File "/usr/lib/python3.10/site-packages/click/core.py", line 722, in __call__
return self.main(*args, **kwargs)
File "/usr/lib/python3.10/site-packages/click/core.py", line 697, in main
rv = self.invoke(ctx)
File "/usr/lib/python3.10/site-packages/click/core.py", line 895, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/usr/lib/python3.10/site-packages/click/core.py", line 535, in invoke
return callback(*args, **kwargs)
File "/usr/lib/python3.10/site-packages/icloudpd-1.7.2-py3.10.egg/icloudpd/base.py", line 293, in main
libraries_dict = icloud.photos.libraries
File "/usr/lib/python3.10/site-packages/pyicloud_ipd/services/photos.py", line 275, in libraries
libraries[zone_name] = PhotoLibrary(
File "/usr/lib/python3.10/site-packages/pyicloud_ipd/services/photos.py", line 154, in __init__
request = self.service.session.post(
File "/usr/lib/python3.10/site-packages/requests/sessions.py", line 635, in post
return self.request("POST", url, data=data, json=json, **kwargs)
File "/usr/lib/python3.10/site-packages/pyicloud_ipd/base.py", line 100, in request
self._raise_error(code, reason)
File "/usr/lib/python3.10/site-packages/pyicloud_ipd/base.py", line 122, in _raise_error
raise api_error
pyicloud_ipd.exceptions.PyiCloudAPIResponseError: INTERNAL_ERROR (INTERNAL_ERROR)

Is there any plans to fix this PR and merge it?

With no apparent changed, shared library API calls started working again for me on May 26th.

Any chance this will get merged in the near future?

Any chance this will get merged in the near future?

I don't see reasons for merging: pyicloud library is no longer used in main icloudpd project, so merging this PR will have no affect on icloudpd (will not add shared libraries support on its own).

I think this may be the only open PR anywhere (right now) that has a working POC for icloud shared libraries.


Prevent Error when override express form controller

Prevent Error when express form controller override from application.

Hi,
my commit fix this issue, can anyone please take a look ?

Why don't you simply declare the __construct method in your custom class that overrides it?
Something like this:
<?php
namespace Your\Namespace;

use Concrete\Core\Application\Application;
use Concrete\Core\Block\BlockController;
use Concrete\Core\Express\Entry\Notifier\Notification\AbstractFormBlockSubmissionNotification;

class YourClass extends AbstractFormBlockSubmissionNotification
{
    public function __construct(Application $app, BlockController $blockController)
    {
        $this->app = $app;
        $this->blockController = $blockController;
    }
}

Agreed with @mlocati – another approach would be to make sure your custom express form block controller extends the express block controller in the core, which is the way it was kind of intended to work in terms of extending a block controller.


How to interpret the depth map here?

I visualized the depth map and it showed something like below:

Does anybody know why is it different from the common depth maps, such as those from Kinect, with all these waves on top of the view?

Thanks.

@kevinzakka @andyzeng @yenchenlin If you are convenient. Sorry to bother🙏

@HaFred The 7-scenes depth maps are saved as 16-bit PNG images in mm. You should cast to float32 and set depth values equal to 65535 (invalid) as 0. Check the demo.py file to see how it’s done.

Thank you for your reply. I've got the point now. Initially, I started with this one and I figured how come there is no depth information for all the whitened stuff.

Then I found that it was due to the range of depth_im, so I normalize it and it makes sense now.


bug: inserted at changed to be epoch, test to see if it matches either

resolved in https://github.com/seanbreckenridge/listenbrainz_export/commit/f3305e04327e17000d33e2537f2dd98fd125fca9


Multiple assignments don't work

This is different from supporting multiple returns in functions.
Test case:
-- this works
function f()
   local x = 1
   local y = 2
end
-- this doesn't
function g()
   local x, y = 1, 2
end

This is not even contemplated in the parser yet, so I assume it is considered a enhancement.

WIP in #104

Implemented by @mascarenhas!


Possible to have StackOverflow icon in header?

Is this possible?

Yeah, you can follow https://github.com/samarsault/plainwhite-jekyll/issues/63.

@samarsault thank you!!


Provide sh-stdlib installer script

sh-stdlib must be install-able independently of other projects.
The current suggestion is to procvide an install.sh script that:

installs sh-stdlib in $HOME/.local/lib/shell/sh/;
creates a $HOME/.bashrc.d/;
add evaluating $HOME/.bashrc.d/*.sh to $HOME/.bashrc;
places proper environment variables setup in a script  within $HOME/.bashrc.d/;
warns about the need to refresh the environment.

This task is to discuss this with @veren-damyanov, possibly with others, have this fully specified and create a task for implementation (or update and use this one).

Done. all changes merged to main, closing.


Expose dead_letter_target_arn variable from lambda module

Is your request related to a new offering from AWS?
No
Is your request related to a problem? Please describe.

I'm unable to configure a dead-letter queue for the lambda function
Describe the solution you'd like.

I'd like to be able to pass a dead-letter queue arn into the module. The child module - terraform-aws-lambda - support this.
Describe alternatives you've considered.

Additional context

The AWS config NCSC Cloud Security Principles conformance pack checks for the presence of a dead-letter queue for lambda functions.

This should be a relatively simple addition if anyone wants to work on it.
If not, I put it in the priority queue since you are my GitHub Sponsor, and I will make it during the week!

This issue has been resolved in version 5.4.0 :tada:


Upgrade Go sdk  to v0.4.0

What type of PR is this?
/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area plugins

/area registry

/area build

/area documentation

What this PR does / why we need it:
We had to bump the Go SDK because the issue we addressed by https://github.com/falcosecurity/plugin-sdk-go/pull/54
This PR just upgrades all plugins to use the latest Go SDK version.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leogr
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 [leogr]

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

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jasondellaluce, leogr
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 [jasondellaluce,leogr]

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


How to call a function?

https://github.com/shalzuth/DeepRockGahactic/blob/main/SDK/SDK/FSD.cs
line 8562
    public class FSDSteamUtils : BlueprintFunctionLibrary
    {

        public Object GetPlayerSteamName() { return Invoke<Object>(nameof(GetPlayerSteamName)); }

    }

I want to call GetPlayerSteamName().and print.
                var sb = new StringBuilder();
                sb.AppendLine("Shalzuth's Helper Tool");
                sb.AppendLine("FPS : " + esp.MeasuredFps.ToString("0.00"));
                sb.AppendLine("Steam Name: " + Sname);  // <--print

I need an example. Thank you.

Need to either:

Find a reference to FSDSteamUtils somewhere else
Create an instance of it.

"FSDSteamUtils" is not reference anywhere.

Can't I call this function?


Inquiry about Default Video Buffer Format (I420) in DSL and Its Impact on Memory Efficiency Compared to RGBA

Hello
I am currently working with the DeepStream Service Library (DSL) and have a few questions regarding the default video buffer format used in DSL

Reason for Defaulting to I420: Could you provide insights into why I420 is chosen as the default video buffer format in DSL? Is DSL following this same default format due to some specific advantages or compatibility reasons? Is it just that the deepstream adopts I420?

Memory Efficiency of I420 vs. RGBA: I am particularly interested in understanding the memory efficiency of using I420 compared to RGBA. Is there a significant difference in memory usage between these two formats in practical applications within DSL?

Format Conversion for Primary GIE: It's my understanding that the primary GIE in DeepStream supports formats like RGB, BGR, and GRAY. Does this mean that there is an automatic format conversion at the input and then a reversion back to the base buffer format at the output?

Thank you for your time and assistance.
Best regards,

@youngjae-avikus the default is actually NV12
#define DSL_VIDEO_FORMAT_I420                                       L"I420"
#define DSL_VIDEO_FORMAT_NV12                                       L"NV12"
#define DSL_VIDEO_FORMAT_RGBA                                       L"RGBA"
#define DSL_VIDEO_FORMAT_DEFAULT                                    DSL_VIDEO_FORMAT_NV12

Keep in mind that the NVIDIA Streammux plugin only excepts NV12 and RGBA buffers

Also, the nvv4lsdecoder and other Source plugins default to NV12.  Selecting NV12 as the DSL default avoids a conversion on input in most cases.
Conversion on input to the infer-plugin can be done if it is something you need, however, color conversion requires GPU overhead.

@rjhowell44
https://github.com/prominenceai/deepstream-services-library/blob/master/docs/api-source.md#dsl-video-format-types
Ah, I checked the API document, but it doesn't seem to have been updated...

@rjhowell44
It seems I was mistaken. I thought that the input for the Primary-GIE must always be converted to RGBA because it accepts color, but are you suggesting that it can also internally handle COLOR_Format from NV12?

@youngjae-avikus thanks for pointing out the incorrect docs... I will update this.
This statement from the nvidia docs seems a little contradictory to me...
The plugin accepts batched NV12/RGBA buffers from upstream. The NvDsBatchMeta structure must already be attached to the Gst Buffers. The low-level library (libnvds_infer) operates on any of INT8 RGB, BGR, or GRAY data

@rjhowell44
I'm wondering if you might have the opportunity to reach out to NVIDIA for clarification on this matter.


RichText - toolbar persists when control loses focus, when rendered inside Panel or Modal

Category
[ ] Enhancement
[X] Bug
[ ] Question
Version
3.15.0
Expected / Desired Behavior / Question
When I place a RichText control inside a Modal or Panel, I would expect that when the control loses focus, the toolbar should automatically hide.
Observed Behavior
The RichText toolbar is on screen permanently, no matter which element on the page has focus.
Steps to Reproduce
Use YO to setup a new Web Part project, add @pnp/spfx-controls-react to the project.  Modify default React class component to add something like this to the render function:
<Panel isOpen={true} type={PanelType.medium} >
  <TextField value={this.state.textField} onChange={(event, value) => { this.setState({textField: value}); }} />
  <RichText value={this.state.richTextField} onChange={(text) => { this.setState({richTextField: text}); return(text); }} />
</Panel>

Observe that when clicking inside the RichText control, then clicking in the plain text field above it - the RichText toolbar remains visible.

Sit the same two text edit controls outside of the Panel (i.e. in the main body of the web part), and observe that focus/blur works fine.

Any fix yet? Having same issues

@stevesuk did you find any solution?

Wasn't able to find a fix for the reported bug.  In the end we changed our solution to use the TinyMCE richtext editor instead (which was a minor battle in itself - but we got there).

@michaelmaillot any help please?

I have the same problem. in addition when I close the panel and reopen the extenden Dialog won't open any more


Update timepicki.js

Line 37 breaks display option when minutes between 0 and 9 (will show 15:8 instead of 15:08)

@senthilraj please pull
Fixes #36
Fixes #26


When using git scm with push=false only the last target is left committed

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
When using multiple targets the changes get lost as the clone is reset to .spec.branch every time, when --push=false is set.
Expected Behavior
All pipeline changes should be present.
Steps To Reproduce
Showing git not working

Apply the following patch:

diff --git a/e2e/updatecli.d/success.d/json.yaml b/e2e/updatecli.d/success.d/json.yaml
index b784b97..a00af52 100644
--- a/e2e/updatecli.d/success.d/json.yaml
+++ b/e2e/updatecli.d/success.d/json.yaml
@@ -1,5 +1,12 @@
 name: Basic Json Example

+scms:
+  default:
+    kind: git
+    spec:
+      url: https://github.com/updatecli/updatecli.git
+      branch: main
+
 sources:
   local:
     name: Get value from json
@@ -56,6 +63,7 @@ targets:
     name: Test value from json
     kind: json
     sourceid: local
+    scmid: default
     spec:
       file: pkg/plugins/resources/json/testdata/data.json
       key: firstName
@@ -65,9 +73,9 @@ targets:
     name: Test value from json
     kind: json
     sourceid: local
+    scmid: default
     spec:
       files:
-        - pkg/plugins/resources/json/testdata/data.json
         - pkg/plugins/resources/json/testdata/data.2.json
       key: firstName
       value: John

Execute test

updatecli apply --config ./e2e/updatecli.d/success.d/json.yaml --push=false

Check results:

git log --oneline -n 3
c48ce4d0 (HEAD -> main) chore: Update key "firstName" from file "pkg/plugins/resources/json/t...
9765d1db (origin/main) fix: jenkins plugin comment
58c43036 Add `spec.files` for Condition and Target of `kind` `yaml` (#650)

Environment
- OS: Ubuntu-20.04 (WSL)
- updatecli: v0.43.0

Anything else?
No response

Very early in the lifecycle of Updatecli we needed both workflow

Being able to commit directly to a head branch
Being able to commit to a working branch

Back then, we decided to use the git scm type for the case one and github for the case two.
That decision didn't age very well.
Both @dduportal and I want to solve this but we took two different approaches
Mine was to add a new boolean disableworkingbranch to all scm
-> https://github.com/updatecli/updatecli/issues/465
At the same time @dduportal started working on https://github.com/updatecli/updatecli/issues/465
With FOSDEM happening this weekend, we'll have the opportunity to talk about to see how we can finish
One rule we try to follow as much as possible is never introducing breaking changes.

With the above decisions in mind, I think there's potentially still a bug with "Being able to commit to a working branch" with multiple targets as changes for the first commit don't get retained.

Definitely there is a bug here. Because Updatecli should commit and push changes after each target

Regarding Gitlab integration, I opened https://github.com/updatecli/updatecli/issues/738 to track the effort so feel free to add a +1
It's a bit of work but not really difficult as I leverage the go-scm library like I did for gitea

The branch is reset here: https://github.com/updatecli/updatecli/blob/main/pkg/plugins/utils/gitgeneric/main.go#L266-L269 on each target checkout, in the changes are not pushed, then this will reset back to origin.

Thanks for  pointing this out.
I am wondering if we should disable this behavior by default, and only enabled if the flag such as --reset=true is specified. Or just remove it.
I don't think exposing this parameter via the manifest would make sense
@dduportal has been looking at the scm refactoring so he may have some opinion here

The parameter makes sense, but I was having a think on this, and context probably changes the need, for a source and condition the reset is useful (and possibly a must). On target it causes the issue.

The parameter makes sense, but I was having a think on this, and context probably changes the need, for a source and condition the reset is useful (and possibly a must). On target it causes the issue.

It's a very good point that it's useful for the shell resource, and in fact it would even solve some of the problem that I am facing with shell target executed in DRY_RUN

I have been thinking about this issue over the night.
I am leaning toward:

enabling/disabling the git reset from the git checkout operation via a scm spec parameter, we want to disable for the local scm by default and enable it by default at the beginning of an Updatecli run
After each source/condition/target execute this git  reset operation but only in dry-run mode using updatecli diff. This is an additionnal safeguard and usefull for the shell resource


Stochaistic Electricity profiles for Zurich case

@martin-mosteiro  we talked about this on the phone but the issue was never open. It relates to the script of Markov chains we got from Diego. I talked to adam, who made similar model.
here is the link:
http://www.tandfonline.com/doi/abs/10.1080/19401493.2014.888595#.VzLRtMemA-I
Could you take a look and open the discussion about models in here and consider the possibility to implement it for June 13th?
@gabriel-happle this might be of interest for us too.

Hey! Indeed, since we talked about this we switched to more or less working 100% on the test run, so I haven't looked into it any further :)
The script we got is based on this paper:
http://www.sciencedirect.com/science/article/pii/S037877880700031X
...which I started reading but haven't been able to go through yet. After the test run I'll spend more time researching this, I'm pretty sure it would be very useful for us.

Based on @JIMENOFONSECA's feedback, this method seems to have no advantage, so it will not be pursued any further.


No types for generate

Describe the bug
I was looking to trying this fork out, but I am not able to use it programmatically at all.
This PR https://github.com/nicolas-chaulet/openapi-typescript-codegen/pull/82 seems to have changed the released files and types. And currently it only includes the config types, not for the actual generate-function.
There also seems to be some issues with the released files, as I cannot import this in my ts-node project with moduleResolution nodenext and esm: true. This is probably an issue with my setup though, related this now being type: module.
Reproduction
import OpenAPI from '@nicolas-chaulet/openapi-typescript-codegen'
// or
import * as OpenAPI from '@nicolas-chaulet/openapi-typescript-codegen'
Only UserConfig and defineConfig seems to be exported
Logs
No response
System information
Mac, ts-node. Node v18.12.1

Severity
annoyance
Additional information
No response

@Birkbjo oh yes, you're correct, I changed that. Before I add the types back, can I ask why you're generating types programatically?
You're also correct on the ESM modules, are you able to share what kind of error message(s) you're seeing?

@Birkbjo I just tried it and it works for me in a file like this, but I am also using "type": "module" in my package.json
import { generate } from '@nicolas-chaulet/openapi-typescript-codegen'

generate()

I will fix the missing types, thanks for flagging! Let me know more details about your execution if you need help with that too

Thanks for the quick resolution!
I started looking into migrating to txs as well, need to cleanup my tsconfig etc.
As for the use-case - I use the codegen in a wrapper too to clean up the results. I need to do some quite specific transformations of the resulting types, like resolve refs (I see that this now includes this, but I dont think it will work for me because the schema im working doesn't directly reference the component...), and some renames etc. Most of this is because the schema I'm working with isn't really ideal.
My project is here if you're interested https://github.com/Birkbjo/dhis2-open-api-ts

@Birkbjo this is fixed in v0.27.33 🎉
I added a section on programmatic setup to README – https://github.com/nicolas-chaulet/openapi-typescript-codegen?tab=readme-ov-file#installation
Here's how I use it
import { createClient } from '@nicolas-chaulet/openapi-typescript-codegen'

const client = await createClient()

You can pass config to createClient(), but it also tries to read config from a file so it's optional.

I need to do some quite specific transformations of the resulting types, like resolve refs (I see that this now includes this, but I dont think it will work for me because the schema im working doesn't directly reference the component...), and some renames etc

We can discuss this more if you want, I think there's some potential to unlock. I am now returning client from createClient() as opposed to not returning anything previously. I don't recommend using it as that's an internal API, but it's a good start as now you can have access to the same information openapi-ts has when generating your client. In theory, you could even run const client = await createClient({ write: false }) to skip writing any files and re-implement the write operation yourself!


Update to 1.8, Still need to add rabbits to Animals

Hey, I can give you contributor status on this project if you want. I'm busy IRL with a few more projects and it may take up to a couple months before I can focus on my Minecraft work!

That's fine. I can keep the project maintained and updated on Bukkit for the users.


Added function to check for whitespace on display name and trim

Purpose
Users were able to put display names of " ", "                  ", "          test        ".
The proposed changes removes the ability to have a whitespace as a display name as well as trimming the names to not have space on either side.
Solution
Before creating the profile to be saved as a change, the name goes through a new function to check for empty whitespace as well as trim the sides of the name for any excess whitespace.
Change summary

Moved the creation of profile to inside the try/catch block
Made function to check for any unnecessary whitespace that is called before profile creation

Steps to Verify

Try to make a name with only spaces and verify the error popup shows up
Make a name with spaces on the sides and check that they are trimmed correctly

looking good. try adding a little test case.

Alright I'll try making some tests and the necessary changes to get past the checks


Fix client portion of contributing to Galaxy for 22.05.

  Liquid Exception: Could not locate the included file 'topics/dev/tutorials/core-contributing/AnalysisRouter.js_diff.md' in any of ["/home/runner/work/training-material/training-material/"]. Ensure it exists in one of those directories and, if it is a symlink, does not point outside your site source. in topics/dev/tutorials/core-contributing/tutorial.md

that looks potentially relevant

Rebased with a fix for that - thanks @hexylena !


Fix Postgres container

Postgres connector does not start up anymore, after mocing to obi.liferay.int
So currently for the users there is no Postgres available in CloudDB.

Postgres 9.4 online again - though it's a completely new container.


HIVE-26431: Use correct schema for iceberg time travel queries

What changes were proposed in this pull request?
For time travel queries we currently always use the latest schema in IcebergSerDe, however since schemas are now versioned, we could use the schema which was active at the time in the past.

Why are the changes needed?

Does this PR introduce any user-facing change?
No

How was this patch tested?
Unit test, manual test

@lcspinter , I wonder, is that PR abandoned?
Can we take it over?
Can you summarise what are the necessary next steps?
Thank you,
Zsolt


打包报错 Cannot create property 'name' on string 'import'

Describe the bug
Cannot create property 'name' on string 'import'
error during build:
TypeError: Cannot create property 'name' on string 'import'
at normalizePlugins (file:///D:/%E9%A1%B9%E7%9B%AE/vant3/myvue3tsmap/node_modules/rollup/dist/es/shared/node-entry.js:25933:25)
at getInputOptions (file:///D:/%E9%A1%B9%E7%9B%AE/vant3/myvue3tsmap/node_modules/rollup/dist/es/shared/node-entry.js:25920:5)
at async rollupInternal (file:///D:/%E9%A1%B9%E7%9B%AE/vant3/myvue3tsmap/node_modules/rollup/dist/es/shared/node-entry.js:25860:72)
at async build (file:///D:/%E9%A1%B9%E7%9B%AE/vant3/myvue3tsmap/node_modules/vite/dist/node/chunks/dep-e8f070e8.js:46471:18)
at async CAC. (file:///D:/%E9%A1%B9%E7%9B%AE/vant3/myvue3tsmap/node_modules/vite/dist/node/cli.js:812:9)
Reproduction
https://github.com/jianjiandanda01/bmapvue3vitts
Steps to reproduce
npm run build
System Info
Cannot create property 'name' on string 'import'
error during build:
TypeError: Cannot create property 'name' on string 'import'
    at normalizePlugins (file:///D:/%E9%A1%B9%E7%9B%AE/vant3/myvue3tsmap/node_modules/rollup/dist/es/shared/node-entry.js:25933:25)
    at getInputOptions (file:///D:/%E9%A1%B9%E7%9B%AE/vant3/myvue3tsmap/node_modules/rollup/dist/es/shared/node-entry.js:25920:5)
    at async rollupInternal (file:///D:/%E9%A1%B9%E7%9B%AE/vant3/myvue3tsmap/node_modules/rollup/dist/es/shared/node-entry.js:25860:72)
    at async build (file:///D:/%E9%A1%B9%E7%9B%AE/vant3/myvue3tsmap/node_modules/vite/dist/node/chunks/dep-e8f070e8.js:46471:18)
    at async CAC.<anonymous> (file:///D:/%E9%A1%B9%E7%9B%AE/vant3/myvue3tsmap/node_modules/vite/dist/node/cli.js:812:9)

Used Package Manager
npm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

@antfu looks like it is failing to import unplugin-vue-components

Repro with only the plugin import
Also failing with vite 4.2.3
Have you seen this before?

I think the config is invalid. It's passing a [string, object] to the plugins.
https://github.com/jianjiandanda01/bmapvue3vitts/blob/3cb899ac001098602712a9bd15dbc382a1cb2e90/vite.config.ts#L58-L65

Yes, thanks for the comment sapphi. I removed that part and got to the error above while debugging, forgot to add a note on that.

The error doesn't give much clues:
failed to load config from /home/projects/vitejs-vite-jnubgu/vite.config.js
error when starting dev server:
SyntaxError: Unexpected token '.'

It works with unplugin-vue-components 0.24.0 so it is something that changed recently there.

This may be an issue with StackBlitz, I can't replicate it locally. I'll report it to them. Closing this issue, the config in the repro is wrong as Sapphi mentioned.


Use pypsg to interact with PSG API

I will have to add instructions for how to set up your API key.


Cannot extend theme Input baseStyle in dark mode

Description
Cannot extend theme Input baseStyle in dark mode, in the example on snack link below, the input border should be red.500 in dark theme
CodeSandbox/Snack link
https://snack.expo.dev/@foxeyerinx/cannot-mofify-inputborder-darktheme
Steps to reproduce

Click on "Toggle" button
See error

NativeBase Version
3.4.23
Platform

[X] Android
[ ] CRA
[X] Expo
[ ] iOS
[ ] Next

Other Platform
No response
Additional Information
No response

Hey @Foxeye-Rinx ,
Thanks for reporting. We will look into this


generating c-code from a new model

Is it possible to generate c-code from a new model using your repo?

Hi
What do you mean by new model?

I want to make my own code_gen.cpp file, and then generate a new model inside mode/codegen

Hi
I am still unable to understand this. I will let the other authors of this repo to take a shot at it.
But it will be good, if you explain what is exactly your use-case? What application?

I just want to regenerate the c-code you have inside mpc_acado/model/codegen. How to generate these files?


potential issue with UniProt connection

Hello,
Thanks for the great package. I think there may be intermittent issues with the UniProt connection, I received this error today:

But as you can see, there is a UniProt entry for this gene: https://www.uniprot.org/uniprot/Q8NBP7
Oddly, grabbing amino acid sequences worked fine for me yesterday. I appreciate any tips!

I cannot solve the issue on my end, which leads me to believe that this is a problem with the UniProt REST API. I contacted their team and will keep you updated. Thank you for bringing this issue to my attention!

Great, thank you!

On May 25th, UniProt released version 2022_02. They added version number identifiers for Ensembl IDs as part of this release. This would not be a problem, except currently, the API does not seem to accept any Ensembl IDs, neither with nor without a version number. It looks like they are still working on updating the API, but it will hopefully happen soon.

On May 25th, UniProt released version 2022_02. This version introduced a requirement for Ensembl ID version numbers. However, this requirement is not true (yet?) for all IDs. As far as I can tell, it seems to be true for most homo sapiens IDs. I adjusted gget accordingly. Please upgrade to the latest gget version, and your query should work again.
Please note that due to the ongoing changes to the UniProt API, there might be more issues in the coming days. Please let me know if you notice anything else. In the meantime, I am working with the UniProt team to get to a consistent solution.


Refactor

[ ] Make the log type names a bit sensible.
[ ] Add Identity Service Sequence diagram in readme.
#88

assign me


feat(transfer): add exchangeData method

Types of changes

[x] New feature
[ ] Bug fix
[ ] Enhancement
[ ] Component style change
[ ] Typescript definition change
[ ] Documentation change
[ ] Coding style change
[ ] Refactoring
[ ] Test cases
[ ] Continuous integration
[ ] Breaking change
[ ] Others

Background and context

https://github.com/arco-design/arco-design-vue/issues/1564
Solution

How is the change tested?

Changelog

Component
Changelog(CN)
Changelog(EN)
Related issues

Checklist:

[x] Test suite passes (npm run test)
[x] Provide changelog for relevant changes (e.g. bug fixes and new features) if applicable.
[x] Changes are submitted to the appropriate branch (e.g. features should be submitted to feature branch and others
should be submitted to main branch)

Other information

@LadyChatterleyLover hi，🥰 感谢 pr

看到 commit 的信息有提交 #1448 可以调整下 commit。
可以补充一下 pr 的描述信息

🤔 #1564 功能还在讨论中

好的


Update sdk references to newer versions of the Azure SDK for Python

This does not change the API version of any of the SDKs. It only bumped the version of the SDK reference for SDK packages where the only change was to build with the azure_bdist_wheel extension.
This checklist is used to make sure that common guidelines for a pull request are followed.
General Guidelines

[N/A] The PR has modified HISTORY.rst with an appropriate description of the change (see Modifying change log).

Command Guidelines

[N/A] Each command and parameter has a meaningful description.
[N/A] Each new command has a test.

(see Authoring Command Modules)

Codecov Report

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

@@           Coverage Diff           @@
##           master    #3031   +/-   ##
=======================================
  Coverage   63.17%   63.17%
=======================================
  Files         484      484
  Lines       27574    27574
  Branches     4275     4275
=======================================
  Hits        17420    17420
  Misses       9000     9000
  Partials     1154     1154

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 27b63e2...3bc39cb. Read the comment docs.

I approved but also think there should be a CI task that verifies that all command packages use the bdist_wheel extension correctly. (Maybe check if the extension file exists and is correct and check setup.py for cmdclass or something).


cluster: release reloading with delay.

/cc @dim
See comment for details.

@dim ping

Sorry, busy recently :) I'm no a big fan of artificial delays but the only alternative solution I can think of is a background channel-select loop, where a c.reloading <- struct{}{} channel is triggering a reload.

How that would help? The problem here is that sequence

start processing command
get hash slot
slot moved - reloading
finish processing

is not synchronized and we can't tell if we need to reload slots or there was a race between reloading and command processing.

Sorry, I thought the problem was that:

start processing command #1
get hash slot, MOVED
trigger reloading
start processing command #2
get hash slot, MOVED
trigger reloading again

I don't quite understand what you mean by:

there was a race between reloading and command processing

The sequence is:

cmd1: start processing command
cmd1: get hash slot
cmd1: MOVED
cmd1: trigger reloading
cmd2: start processing command #2
cmd2: get hash slot
cmd1: reloading finished
cmd2: MOVED
cmd2: trigger reloading

So there is a race between cmd1: reloading finished and cmd2: trigger reloading.

@dim ping

I think adding a 1s delay will cause more problems than it solves. You could have a genuine slot reallocation by redis in the meantime. I don't see the 'race' being much on an issue, it will trigger another slot sync, but is this really causing much overhead? I don't believe we will even end up in a spinning state. If you are really keen to avoid it, why don't compare you compare addr returned by isMovedError against  the stored address for that slot? Then, you would only trigger lazyReloadSlots if they are different.
As said, I really don't like artificial delays, they always tend to cause not trivial issues downstream.

@dim done. PTAL

@dim gentle ping. I think I addressed your comment.

@vmihailenco really sorry, been pretty busy, LGTM now


New version: pizzaboxer.Bloxstrap version 2.3.0

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.4 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

I see the Bloxstrap icon in the taskbar, looks like it didn't run silently? Can I see what window it's showing just in case it's an error or something?

I see the Bloxstrap icon in the taskbar, looks like it didn't run silently? Can I see what window it's showing just in case it's an error or something?

Unfortunately the Automatic Validation only provides 1 screenshot per run. I'm guessing this is due to the dependency.

@wingetbot waivers Add Validation-Unattended-Failed


3.23.0 is missing a changelog entry

Thanks for maintaining this library and staying up-to-date with Zod!

Nice catch! I'll add it for the next release.


[5.0.0] addon-info does not render code snippet markdown given on the text parameter

Describe the bug
After upgrading to 5.0.0, addon-info does not render any code snippets given via the text key on the info parameter on the addParameter() function. It successfully displays all the given text but fails and does not display the given code under the ~~~ ~~~ notation, i.e:
\~~~js
  function something(a) {
    return a;
  }
\~~~

To Reproduce
Steps to reproduce the behavior:

Go to any story, put any string with aforementioned ~~~js ~~~ markdown notation
View it on the storybook
See that the supposed code snippet does not show at all.

Expected behavior
The supposed code snippet should show correctly as before
Screenshots

Code snippets

System:

OS: MacOS Mojave, also does not work on netlify deployment
Device: Macbook Pro 2017
Browser: Chrome
Framework: React
Addons: addon-info
Version: 5.0.0

Additional context
none

I am also experiencing this with the js  syntax.

I see this issue when using individual story parameters as well.

It does, however, interpret HTML. I had to use <code>, <, > to make do.

This needs to be merged ASAP.

I also would appreciate that fix to be released

Here are some clues I found:
This line should maybe changed to:
const Code = (props) => {
  const { language, code } = props

  return (
    <SyntaxHighlighter
      bordered
      copyable
      language={language}
    >
      {code}
    </SyntaxHighlighter>
  );
};

But then this line (and other lines will probably too) results in:
Cannot read property 'bar' of undefined

Which tells me components consuming the theme aren't up to date with the latest themes found here.
May anyone pick it up from here?

Is there any motion on this issue? Still seeing it in 5.0.10 this morning.

Yes the issue still exists..

For the sake of helping to weight this issue and keep it fresh:
Having the same issue upgrading from storybook 4 to 5.
I even tried the markdown example from the README and it is also not working.
https://github.com/storybooks/storybook/tree/master/addons/info#markdown

Hi all, still no news about this issue? It would be great to have it fixed...

Fix this plx!

This is a rather urgent problem that I feel deserves more attention, so bumping a bit.

There is a relatively easy workaround in this linked issue:
https://github.com/storybooks/storybook/pull/6016

@whengely Just be nice if it supported the syntax laid out in the documentation. The same syntax this addon supported prior.

bump

Boo-yah!! I just released https://github.com/storybookjs/storybook/releases/tag/v5.2.0-alpha.29 containing PR #6016 that references this issue. Upgrade today to try it out!
You can find this prerelease on the @debug NPM tag.
Closing this issue. Please re-open if you think there's still more to do.

Boo-yah!! I just released https://github.com/storybookjs/storybook/releases/tag/v5.2.0-alpha.29 containing PR #6016 that references this issue. Upgrade today to try it out!
You can find this prerelease on the @debug NPM tag.
Closing this issue. Please re-open if you think there's still more to do.


Power On/Off Commands?

Heya!
Just wondering reading through the commands, is there a Power On/Off command?
Thanks!

PWM_ON and PWM_OFF

Ah, thanks for clarification!
But I think this is a tiny bit confusing even? Cause the Z906 has a built in speaker test function, and at first I thought the "PWM" was meant as this testing function. Maybe calling it "PWR" instead would be easier understandable :)

Ah, thanks for clarification!
But I think this is a tiny bit confusing even? Cause the Z906 has a built in speaker test function, and at first I thought the "PWM" was meant as this testing function. Maybe calling it "PWR" instead would be easier understandable :)

can be a bit confusing but this is how class D amplifiers work,
I have left out the speaker test function yet but it is implemented in the DSP firmware.

Okay that makes sense then.
Still hope someone can someday implement this into a ESP-friendly firmware, know there is another project currently building upon this research (I think?) but sadly no progress yet. Would love to implement the Z906 directly into my Smart Home :) Would even donate a few bucks if someone does it.
Thanks for the research work and reversing btw!

I opened up the console last year and produced this documentation.
I've started on an ESP32-S2 application that will make this possible but I don't know when it'll be finished. Currently all it does is relay the communication between the two devices.


Added remedies into the model object

Added a new property "remedies" into the model object

Remedies added in predictions page are working fine.


Build warnings in 0.7.0

Hi,
Related to my response in #34, I noticed that building Dobby @0.7.0 results in this warning (from Xcode or via Carthage integration):
ld: warning: linking against a dylib which is not safe for use in application extensions: /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/Library/Frameworks/XCTest.framework/XCTest.tbd
ld: warning: linking against a dylib which is not safe for use in application extensions: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib/swift/iphonesimulator/libswiftXCTest.dylib

Looking into this further, Dobby sets APPLICATION_EXTENSION_API_ONLY = YES via the Configuration/Base/Targets/Frameworks.xcconfig configuration file.
// Disallows use of APIs that are not available
// to app extensions and linking to frameworks
// that have not been built with this setting enabled.
APPLICATION_EXTENSION_API_ONLY = YES

While this probably makes sense for most frameworks, does it make sense for Dobby? The warning seems to stem from the fact that Dobby links/requires XCTest which itself doesn't seem to have been built with this setting enabled.
I've tested that overriding that setting for the Dobby-iOS target removes the warnings. I kind of think that Dobby as a unit-test framework that links against XCTest shouldn't disallow the use of APIs that are not available to app extensions. Thoughts?
I can open a PR with this change if you agree: https://github.com/trivago/Dobby/compare/master...mdarnall:dont-require-app-extension-apis

Thanks for looking into this, @mdarnall. After checking the origin of this setting and comparing with Quick, I do agree we should change this. Please open a pull request!


Update svelte

Hey! Thanks for this project, I learned about it on Twitter today. I noticed the Svelte version was out of date (our template syntax changed in v2) so I updated it and made a couple of tweaks to make it more idiomatic.

The size of the JS is really impressive: https://wc-todo.firebaseapp.com/svelte/

Any way to minify that to make it smaller?

My bad it is already minified! I was confused because of the source map. thanks!

Svelte has the smallest bundle size: https://twitter.com/julienrenaux/status/1040739084803735552
👏


x@ should return a map. Currently, it has to be converted to a map.

From spec:
xml-attributes-expr := expression @

Returns the attributes map of a singleton xml value, or nil if the operand is not a singleton xml value.
The result type is map<string>?


Nested exposure doesn't work

This code:
class User < Grape::Entity
  expose :photo do |p|
    expose :large do
      p.photo.url(:large)
    end
  end
end

Returns:
NoMethodError (undefined method `expose' for #<API::User:0x00000023cdec68>):

grape-entity: v0.4.8
grape: v0.13.0

Did you try this on HEAD? I think the fixes and support for nested exposures hasn't been released yet. We have tests for this, https://github.com/ruby-grape/grape-entity/blob/2baf103f5a0574a4d83a7453b2fbeec998a6a46d/spec/grape_entity/entity_spec.rb#L88.
cc: @idyll @marshall-lee maybe time to release 0.5.0 ...

I saw it, but haven't tried, as we are depending on https://github.com/ruby-grape/grape-swagger here. Thought it's available for 0.4.x as well, but the 0.5.x has better implementation of the nested exposures. Looks like I need to work on https://github.com/ruby-grape/grape-swagger/issues/314 first :)

@dmitry Have you managed to get this to work? I think we are currently running into the same issues here.

@namxam you can upgrade to 0.5, it should be fixed there. I've solved it by not using nested exposures :)

I believe this is fixed, closing.


Parse.User.current() doesn't update local storage.

Parse.User.current() updates on save() in the current session - but if you do a browser refresh (F5/CMD+R) the content is lost. The only way to update it is to force a logOut() and then logIn() again. Parse.user.current().fetch() is supposed to bring the localStorage up to date - but that is not happening.
Let me know if you need any more to reproduce.

User is being logged out everytime he quits the app. Let me know, If I can get help from localStorage () function?

Under the hood, we update an in-memory cache of the current user, but we don't save back to disk on each successive server save.
The solution is to special-case the save of a ParseUser object, checking to see if it is the current user; if it is, we can save back to localStorage before resolving the outermost Promise.

Is this a regression in the latest version? Because; I have 1.5.0 in one of my projects where saving the current parse user saves the object to localStorage as well, but on the other project where I'm using 1.6.7 I'm having this issue; the object doesn't persist in localStorage.
Same for fetch() the latest version only saves the current user to memory not localStorage but the earlier version I'm using updates localStorage as well.

Just pushed a fix. We will release 1.6.9 later today with these issues resolved.

Is 1.6.9 still being released? I see the tag in this repo, but site and CDN only have 1.6.7. Really need this fix in. Thanks!

Hi, just checking in to ask if 1.6.9 will be available soon? The CDN is still at 1.6.7 and I am also eager to distribute the fixed version.

The CDN is at 1.6.14 at time of writing, the (recommended way of getting this aka the) npm module is at 1.6.14, this is fixed and should be closed.


How to add KML as layer vector

Currently trying to add KML file to map as follows:
`const kml = JSZipUtils.getBinaryContent(URL, function (err, data) {
if (err) {
throw err; // or handle err
}
                        const zip = new JSZip(data);

                        const kmlFile = zip.file(/\.kml$/i)[0];
                        console.log(kmlFile);
                        console.log(kmlFile.asText());
                        console.log(URL.createObjectURL(new Blob([kmlFile.asArrayBuffer()])));
                        return URL.createObjectURL(new Blob([kmlFile.asArrayBuffer()]));
                    });

                    return <RLayerVector url={kml} format={new KML()}></RLayerVector>;`

But not seeing any results, any help appreciated

found out that const kml never has a value so need to do it with useeffect/state or smthng along those lines


glab: 1.40.0 -> 1.41.0

Description of changes
Diff: v1.40.0...v1.41.0
Changelog: https://gitlab.com/gitlab-org/cli/-/releases

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@jonochang please include link to change log and or diff in PR description and commit message

@jonochang please include link to change log and or diff in PR description and commit message

Done.


Difficult to reproduce paper results

What is the reason why the Image-level AUROC metric and the Pixel-level pAUROC metric are particularly low, only 0.575 and 0.872 respectively, when running the screw category with default settings?

Try setting resnet34 as an encoder for the screw category, or setting num_layer = 4.


Fix regression in #6980, add functional tests

The logging refactor in #6980 should have been a class variable. This fixes that.
Also adds functional tests that would catch that and the original #6980 bug.

There's a specs failure in AppVeyor that needs to be fixed

This change should fix the appveyor use case.


Game doesn’t launch from Game mode

Hey there,
Just installed Ubisoft connect and Prince of Persia. The game is installed on SSD and launch well from Ubi Connect.
The game automatically appears in the non steam  category but it does not start from here.
I tried switching to proton experimental but not working either.

Does it help ?

@sysmoon14 @windboy91 there's an extra Steam in there!

Sorry I don’t get it 😅
This is my first time.

@windboy91 it says Steam twice in the target and Startin directory it should just be Steam/steamapps/ so just delete "Steam" on both of them.. I just fixed this just now in the script.... however your scanner needs to be updated so RUN Nonstealaunchers.desktop again and it will update your .py

Oh nice ! That’s working thanks 😃
another question: is it normal that it launches Ubi Connect when launching the game ?
Amazing script by the way!

@windboy91 yes its normal that's what it's supposed to do. Make sure to run the .desktop file again though so you can get the fix! There is also a "Stop NSLGameScanner" that will restart the scanner for you with the latest fixes.


[BugFix] mkdir $STARROCKS_HOME/meta in bin/start_fe.sh

Why I'm doing:
When I run ./bin/start_fe.sh after running build.sh, it reports an error:
2024-07-25 16:57:39.027Z ERROR (main|1) [MetaHelper.checkMetaDir():166] meta dir /data/starrocks/output/fe/meta does not exist
2024-07-25 16:57:39.031Z ERROR (main|1) [StarRocksFE.start():185] StarRocksFE start failed
com.starrocks.common.InvalidMetaDirException: null
        at com.starrocks.leader.MetaHelper.checkMetaDir(MetaHelper.java:167) ~[starrocks-fe.jar:?]
        at com.starrocks.StarRocksFE.start(StarRocksFE.java:120) ~[starrocks-fe.jar:?]
        at com.starrocks.StarRocksFE.main(StarRocksFE.java:83) ~[starrocks-fe.jar:?]

This is because we have not create meta directory.
What I'm doing:
Add mkdir $STARROCKS_HOME/meta command in bin/start_fe.sh
What type of PR is this:

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[x] Yes, this PR will result in a change in behavior.
[ ] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[x] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[ ] This is a backport pr

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto-backported to the target branch

[ ] 3.3
[ ] 3.2
[ ] 3.1
[ ] 3.0
[ ] 2.5

 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.


maintainers/teams: vim -> neovim

Things done
Replace the legacy lib.teams.vim with lib.teams.neovim which mirrors the GitHub NixOS/neovim team.
cc @PerchunPak @mrcjkb @khaneliman

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Doesn't seem to be happy in eval.


We do not always enable (and sometimes disable) gzip for unrewritten resources

I'm doing this with a debug install of mod_pagespeed using
/mod_pagespeed_example.

1) Put mod_pagespeed in passthrough:

    ModPagespeedRewriteLevel PassThrough
    ModPagespeedDisableFilters add_instrumentation

2) Take a look at content type, http status, and url in mod_pagespeed example,
and whether the results are gzipped when we ask for it.  A script:

for f in $(cd $HTDOCS_PATH && find mod_pagespeed_example -name '*.js') ; do
wget http://$HOST:8080/$f --header='accept-encoding: gzip, deflate' -S -O
/dev/null |& egrep 'Content-Encoding|--2015|HTTP/|Content-Type:'; done

What we see is that gzip is not being enabled for .css and .js files that are
being passed through.  .html is being gzipped.

3) Attempt the same thing, with gzip switched on for text/css and
application/javascript.

This appears to work in my configuration, but we've had reports of it failing
when compression worked with ModPagespeed off.  But really we should enable
compression for any content type we don't think we should explicitly blacklist
(eg the image types and woff / woff2).

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 23 Feb 2015 at 6:24

If you're seeing some resource where:

With PageSpeed on it's served without gzip
With PageSpeed unplugged or not installed it's served with gzip

Please give a link and reopen.

Check here
https://www.finpower.co.in/


Gravity generator rewrite

See commits
:cl: PJB

tweak: Gravity generator now needs ~100 seconds to charge up/down when turned on/off. This means spamming the power button won't immediately do anything.
tweak: Gravity generator UI now looks very fancy.

Fixes #4758
Fixes #4645

pic of the UI?

(note: client side UI not yet rewritten)


Create all-in-one "installer" for a ZIO CLI application

The installer or package should do the following:

Download the GraalVM native image or Docker image
Add the CLI application to the path
Add completions for bash and zshell

Inspiration could be taken from Fury which seems to do a nice job with this.

Hi, I would like to try to tackle this together with @JakDar 🙂

@Porcupine96 Awesome!
Fury's installer script was actually moved into a runner script: https://github.com/propensive/fury/blob/main/fury
I like this idea, and I think our own "installer" script could actually be a runner, too, to simplify the process of running the application. Then the first part of the runner would check and see if the app is installed, and if not, it could first download and install it, before proceeding to run it.
Now this installer would look different right now versus after #4 is completed. Before #4 is completed, the "installer" script could be more like the Fury script, being able to download the user's CLIApp from a remote ZIP/JAR, and then also being able to run the CLIApp using Graal (if possible, otherwise defaulting to JVM options allowing fast start).
I would take inspiration from the Fury script and try to make something that "just works" with an existing CLIApp that's been packed up somewhere online. As a test, you could create a Github repository, build a simple CLIApp, and upload it as a Github Release file. Then test that if you point the installer script to that CLIApp, it is capable of downloading and installing it and running it appropriately, and that it doesn't download it again every time if necessary.
Warning: Knowledge of shell / bash probably required for this ticket!

Hi @jdegoes !
We made a POC which can be tested by setting the following variables in installer.sh.
appName="zio_cli_test"
version="0.0.1"
downloadUrl="https://github.com/JakDar/zio-cli-test/releases/download/0.0.1/zio-cli-test-0.0.1.jar"

The runner works as follow:

It downloads the JAR file from $downloadUrl
2a) If a user has Graal with native-image installed it builds and saves a native image from JAR
2b) If not, it runs the JAR with java -jar
It executes the CLIApp passing the proper options/args

Additionally if ENSURE_NATIVE_IMAGE is set, the runner downloads Graal and installs native-image.
(We thought it will make the tool more pleasant to use. If you think that this feature does not make sense we can remove it. 🙂)
The runner makes sure not to download CLIApp / Graal if they were previously installed.

We have a question about ZIP / TAR.GZ archives. Should we assume that the archive contains a single JAR file?

Let us know what you think! Thanks!

Script with filled variables is also here, one can download example app using this script via:
wget https://raw.githubusercontent.com/JakDar/zio-cli-test/master/installer.sh -O ~/.local/bin/zio-cli-test &&  chmod u+x ~/.local/bin/zio-cli-test

# And (provided you have ~/.local/bin in PATH):
zio-cli-test /etc/passwd # or any other existing file ;)


on_unmount doesn't work

Describe the bug
as you view on rx.cond
Note: is_upload = False (but use ~ on cond)
True: mount -> _upload
False: mount -> rx.fragment

when i upload a file, it  trigger on_drop, so is_upload change to True
Now the rx.cond should render the rx.fragment, as my understand this unMount the _upload so it should call his on_unmount calling the rx.redirect to the path.
But, in prod mode this doens't call the on_unmount in _upload
To Reproduce
Steps to reproduce the behavior:
class Upload(rx.ComponentState):
    _data: bytes | list[bytes]
    is_upload: bool = False

    async def upload_single(self, upload_files: list[rx.UploadFile]):
        if upload_files:
            upload_data = await upload_files[0].read()
            print(upload_files[0].filename)
            self._data = upload_data
            self.is_upload = True

    async def upload_multiple(self, upload_files: list[rx.UploadFile]):
        if upload_files:
            upload_data = [await file.read() for file in upload_files]

            self._data = upload_data
            self.is_upload = True

    @classmethod
    def get_component(
        cls,
        id: str,
        select_files: Literal["single", "multiple"],
        redirect_path: str = "",
    ) -> rx.Component:
        _upload = rx.upload(
            rx.vstack(
                rx.text("Drag and drop files here or click to select files"),
            ),
            id=id,
            on_drop=getattr(cls, f"upload_{select_files}")(
                rx.upload_files(upload_id=id)
            ),
            multiple=True if select_files == "multiple" else False,
            on_unmount=rx.redirect(redirect_path),
        )

        new_upload = rx.box(
            rx.cond(
                ~cls.is_upload,
                _upload,
                rx.fragment(
                    "Redirecting",
                ),
            )
        )

        return new_upload

cond calls the redirect on load the page.
class Upload(rx.ComponentState):
    _data: bytes | list[bytes]
    show_upload: bool = True

    async def upload_single(self, upload_files: list[rx.UploadFile]):
        if upload_files:
            upload_data = await upload_files[0].read()

            self._data = upload_data
            self.show_upload = False

    async def upload_multiple(self, upload_files: list[rx.UploadFile]):
        if upload_files:
            upload_data = [await file.read() for file in upload_files]

            self._data = upload_data
            self.show_upload = False

    @classmethod
    def get_component(
        cls,
        id: str,
        select_files: Literal["single", "multiple"],
        redirect_path: str = "",
    ) -> rx.Component:
        _upload = rx.upload(
            rx.vstack(
                rx.text("Drag and drop files here or click to select files"),
            ),
            id=id,
            on_drop=getattr(cls, f"upload_{select_files}")(
                rx.upload_files(upload_id=id)
            ),
            multiple=True if select_files == "multiple" else False,

        )

        new_upload = rx.box(
            rx.cond(
                cls.show_upload,
                _upload,
                rx.fragment(
                    "Redirecting", on_mount=rx.redirect(redirect_path, external=False)
                ),
            )
        )

        return new_upload