Font sizes inconsistent in response delivery radio buttons [API v2 QA]

Description
The hint text for download is smaller than the hint text for email and API delivery methods.
Steps to reproduce

Go to Settings
Select Protected B
Select Download

See discrepancy

Details

Browser: Edge
Operating system: Windows
Environment: Staging
Language: EN / FR
Viewport: Tablet

Expected behaviour
Consistency in typography, design system
Screenshots or videos

Additional context
#4395 #4721

Covered under
https://github.com/cds-snc/platform-forms-client/pull/4754


airflow backfill fails when specifying a PythonOperator task instance and sending DAG pickle to worker

airflow backfill -t opc -s 2012-01-02 -e 2012-01-02 -i one_day fails with error:
[2015-09-22 11:01:04,101] {models.py:902} INFO - Executing <Task(PythonOperator): opc01> on 2012-01-02 00:00:00
[2015-09-22 11:01:04,101] {models.py:959} ERROR - 'PythonOperator' object has no attribute 'params'
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/site-packages/airflow/models.py", line 904, in run
    context = self.get_template_context()
  File "/usr/local/lib/python2.7/site-packages/airflow/models.py", line 1001, in get_template_context
    if 'tables' in task.params:
AttributeError: 'PythonOperator' object has no attribute 'params'
[2015-09-22 11:01:04,110] {models.py:995} ERROR - 'PythonOperator' object has no attribute 'params'

but airflow backfill -t opc -s 2012-01-02 -e 2012-01-02 -i -x one_day succeeds.
Offending library code:
tables = None
 if 'tables' in task.params:
     tables = task.params['tables']

Why is task.params sometimes set but not in this case?

what's the type of params['tables']?

Heh, not sure. I was hoping you could give insight since the code throwing the error is from Airflow's library, specifically https://github.com/airbnb/airflow/blob/master/airflow/models.py#L1015

Looks like the problem is that the pickle is incomplete, it should have a params attribute but doesn't. There are no codepaths that I know that can cause a task to not have a params attribute

I have an open issue #406 that our DAG is a bit large so it has to be stored in MySQL as a longblob. Could the pickling be failing silently due to its size?
FWIW, running a backfill on the whole DAG (without specifying a task filter) does succeed, and that probably pickles the DAG since CeleryExecutor is specified.

Does this happen consistently? It seems very odd to me.
At Airbnb we have many DAGs that can't be pickled because they use macros that aren't picklable, so we tend to use backfill --local which forces a local executor, does that work for you as a fallback?
Otherwise to debug it you'd have to fire up an ipython shell, pickle the DAG by hand, unpickle it and compare the two objects, try to understand why the one that comes back has missing params. That's pretty involved...

I've had a similar problem, that pickles are too big. My response was to add a config option to disable pickling (it previously existed as a flag). Accordingly, if not running locally, all celery workers must have the same code available to them.


Solution architecture guide - update ASP.NET 5 an EF 7 to their core names

AT https://github.com/HTBox/allReady/wiki/Solution-architecture there are a few places that still mention ASP.NET 5,  ASP.NET MVC 6 and Entity Framework 7, these should be renamed to the .NET Core names.
Entity Framework 7 = Entity Framework Core
ASP.NET 5 = ASP.NET Core 1.0
ASP.NET MCVC 6 = ASP.NET Core MVC 1.0

I would like to volunteer to take this task on. Safe to assume that the download links should be updated as well?

Looks like this was is fixed and has already been merged.


Files can't be downloaded after being uploaded

low priority but when i'm on the project details page and I upload a file, it shows up in the list but you can't download it.  If there was an existing file you can download that one and you are left with one downloadable and one not.  A page refresh makes the file downloadable again.

This was happening to me locally, but not when I tried it on the azure site.
I wonder if this happens with larger files/slow upload speed?

Not sure but let’s update the azure site and try again, maybe it’s because of more recent code.

From: Jason Sylvestre @.>
Sent: Thursday, July 29, 2021 12:39:59 PM
To: ucdavis/Harvest @.>
Cc: Scott Kirkland @.>; Author @.>
Subject: Re: [ucdavis/Harvest] Files can't be downloaded after being uploaded (#386)
This was happening to me locally, but not when I tried it on the azure site.
I wonder if this happens with larger files/slow upload speed?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/ucdavis/Harvest/issues/386#issuecomment-889407989, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AABRQANGQ2FRVG5NLJP6ASLT2GVA7ANCNFSM5BG7RCNA.

I Uploaded 3 files, first and last are clickable, middle one isn't:

@wongband Ok, figured it out and Jason was right, it seems to happen with larger files.  Try uploading something that's a few MB, like a large picture or a big PDF.  I think what is happening is that the file is loaded with a link while it is still uploading, and then when uploading is finished the file info is reset and the link disappears.  If you have trouble reproducing it let me know, I can reproduce it just about every time now.

@srkirkland Okay I will try it out


Behavior w.r.t. backpressure

We've seen some situations where gof3r is using unexpectedly large amounts of memory (e.g., hundreds of megs of RSS). This seems odd, since there's nothing that's inherently memory-intensive in its functionality as I understand it. I thought the issue might be buffering: does gof3r provide backpressure when reading stdin is a lot faster than the S3 upload? Does it provide backpressure when writing to stdout is a lot slower than the S3 download?

Is this uploading or downloading? From your description it sounds like uploading. If it's occurring downloads, that could indicate an unknown memory leak. On uploads, gof3r tries to be smart (probably too smart) about doubling buffer sizes periodically to allow larger maximum upload size given the 10,000 part S3 limitation.
With the current code, the best workaround is to increase initial part size and decrease concurrency to limit memory usage. The buffer size will only be increased if necessary.
I'm open to a better solution.
The buffer size logic is here: https://github.com/rlmcpherson/s3gof3r/blob/ed07ab7f4c26c61b927fc5e51e80e2d3b40f4470/putter.go#L152

Related discussion in #33 as well.

You know, I think we might be using a version older than the fixes you'd put in there--I'm going to close this for now and reopen if I still see problems with a recent version. I'd looked at the docs, but hadn't looked at the closed issues Thanks!

Yeah, the memory usage was higher and less predictable before the switch to byte slices from bytes.Buffer so hopefully that will be sufficient.


Add 's' at the end of import yoga-helpers

The documentation does not have the letter 's' at the end, so importing the library(@gympass/yoga-helper) will not work.
That way, you must add the 's' at the end of the import for the library(@gympass/yoga-helpers) to work.
Before

After

Thanks for your contribution. Indeed, we don't have any other occurrence of yoga-helper in our codebase.

@all-contributors please add @matheushenrique1  for doc


MAR-4055-frontend-change-brief-to-opportunity

There are some instances of briefs found under
BriefDownloadResponses.js
BriefSubmitted.js

This branch has some conflicts.  @Revit can you please fix this conflict. Let me know if you have any issues.


typo in request.html

It is "label" not "lable":

<input type="checkbox" name="pwchange" value="pwchange">Password Change Request</lable>

Imported from trac ticket #62,  created by ahelsing on 10-24-2013 at 10:42, last modified: 10-24-2013 at 19:04

fix pushed to develop
Trac comment by ahelsing on 10-24-2013 at 11:16


fix style for quarter

Description
Linked issue: #4219
Problem
Quarter Picker with "excludeDates" is not getting inactive styles.
Please check the issue for details.
Changes

Include "excludeDates" as a condition for calling the Quarter Picker's disable judge function.

Contribution checklist

[ ] I have followed the contributing guidelines.
[ ] I have added sufficient test coverage for my changes.
[ ] I have formatted my code with Prettier and checked for linting issues with ESLint for code readability.

Sorry...
https://github.com/Hacker0x01/react-datepicker/blob/main/src/year.tsx#L370
I have modified the call condition again to match the above.

Sorry for repeating myself.
"excludeDate" and "includeDate" tests added.
The "filterDate" test has not been added to match MONTH.

@asada-no4 please fix the prettier error so we can merge this

Sorry.
I ran "yarn run prettier".


缺少依赖啊。

WARNING: Makefile 'package/feeds/small8/linkease/Makefile' has a dependency on 'ffmpeg-remux', which does not exist
WARNING: Makefile 'package/feeds/small8/luci-app-store/Makefile' has a dependency on 'luci-lib-taskd', which does not exist
WARNING: Makefile 'package/feeds/small8/luci-app-xray/Makefile' has a dependency on 'kmod-nft-tproxy', which does not exist

一样
Create index file './feeds/small.index'
WARNING: Makefile 'package/feeds/small8/linkease/Makefile' has a dependency on 'ffmpeg-remux', which does not exist
WARNING: Makefile 'package/feeds/small8/luci-app-store/Makefile' has a dependency on 'luci-lib-taskd', which does not exist
WARNING: Makefile 'package/feeds/small8/luci-app-xray/Makefile' has a dependency on 'kmod-nft-tproxy', which does not exist

一样 Create index file './feeds/small.index' WARNING: Makefile 'package/feeds/small8/linkease/Makefile' has a dependency on 'ffmpeg-remux', which does not exist WARNING: Makefile 'package/feeds/small8/luci-app-store/Makefile' has a dependency on 'luci-lib-taskd', which does not exist WARNING: Makefile 'package/feeds/small8/luci-app-xray/Makefile' has a dependency on 'kmod-nft-tproxy', which does not exist

解决了吗？从别的仓库下载过来也不行～头疼～

WARNING: Makefile 'package/feeds/small8/linkease/Makefile' has a dependency on 'ffmpeg-remux', which does not exist
WARNING: Makefile 'package/feeds/small8/luci-app-store/Makefile' has a dependency on 'luci-lib-taskd', which does not exist
WARNING: Makefile 'package/feeds/luci/applications/luci-app-webdav/Makefile' has a dependency on 'webdav-hacdias', which does not exist
WARNING: Makefile 'package/feeds/small8/luci-app-xray/Makefile' has a dependency on 'kmod-nft-tproxy', which does not exist
缺的越来越多了！

一样，哈哈

WARNING: Makefile 'package/small-package/luci-app-xray/Makefile' has a dependency on 'kmod-nft-tproxy', which does not exist
我用的是lede源码编译的，提示缺少依赖


Generate basic report with out meme comparison

Hi! I am using tf-modisco lite on data generated using chrombpnet. I generated the modisco.h5 file and want to now generate the report. Is it still possible to generate the report using the command modisco report -i modisco_results.h5 -o report/ -s report/ without the -m meme.txt input or has that option been omitted (I am getting an error message alerting me that the pipeline is missing the meme.txt input)?  Thank you in advance for your help!

Yes, you shouldn't need the MEME file if you don't want to map your learned motifs to a database. Have you checked to make sure you are using the latest version? Can you post the entire command you used and the error message you got?

Hi Jacob - thank you for your quick reply. I actually moved all of my work to a different PC and the error
doesn't seem to be occurring anymore! However, the motif.html that it generated is not able to load any of the cwm images...

The above report was for a specific region on using the chromBPNet tutorial model. I just tried using tfmodisco-lite on my own chrombpnet model (still using the tutorial bias model), but the report isn't showing any motifs this time. Any idea why this may be happening?

Are you running the report command in the same directory you're running the motifs command? What's in the report/ directory after running it?

Yes, I am running both commands in the same directory. The report folder contains an empty trimmed_logos folder as well as a motifs HTML file.

I generated another run with a different region and got 4 patterns, however, the same issue is occurring where the images are not fully showing up in the HTML despite being in the trimmed_logos folder.

Hi @jmschrei - I am checking in to see if you have any suggestions to resolve the above-mentioned issues... I've been running other regions through the pipeline, all of which are successfully creating pred_bw files that can be viewed through IGV, and a lot of them are not generating any motifs. Thank you in advance!

Hi @kaillahs. Unfortunately, issues like this are usually challenging to debug remotely. But what do you mean the report is generated for a specific region? Are you running tfmodisco on a single example? The report should be generated genome-wide. I do not believe that there are over 100 real seqlets in a single example.

@jmschrei  - thank you for your reply. I've been using the chrombpnet contribs_bw pipeline which requires you to enter a specific region of the genome you want to run through the command. I've been then using the output_prefix.profile_scores.h5 and the output_prefix.counts_scores.h5 for the -i argument of the modisco motifs command. Please correct me if this is the wrong input.

Sorry, but I'm actually unfamiliar with how the chrombpnet repo works. @austintwang do you happen to know?

@kaillahs 3000 bp is a very small region to get deepshaps and summarize seqlets using modisco. What are you trying to do?
Modisco is a summarization tool used to get a list of motifs found in all your peaks for example.

@panushri25 I am trying to get transcription factors across specific peak regions in the genome. These regions tend to be fairly small... Would it be best to just extend the region to around 300,000 bp? I've done this once and it seems to work just fine so I'm assuming the issues I was running into were simply size-related.

Hi! I am noticing that my match q-values are far lower for the counts score modisco reports than for the profile scores. Is there a reason for this? Also, should there be a difference in the quality of the modiso motif predictions depending on how far out I extend the regions? I seem to be getting around 8 predictions regardless of whether I extend my region by 360,000bp or 150,000bp, 4 of which are present in both for the count predictions and 1 of which is present in both for profile prediction.


dart issue

Analyzer Feedback from IntelliJ
Version information

IDEA AI-191.8<IP_ADDRESS>00203
2.7.0-edge.bbe2ac28c9ab5bfec4e13d5df9360e7aa691cfe6
AI-191.8<IP_ADDRESS>00203, JRE 1.8.0_202-release-1483-b03x64 JetBrains s.r.o, OS Windows 10(amd64) v10.0 , screens 1366x768

Exception
Dart analysis server, SDK version 2.7.0-edge.bbe2ac28c9ab5bfec4e13d5df9360e7aa691cfe6, server version 1.27.4, FATAL error: Failed to handle request: {id: 3, method: analysis.setAnalysisRoots, params: {included: [E:\Flutter Projects\fouad_app], excluded: [E:\Flutter Projects\fouad_app\.idea, E:\Flutter Projects\fouad_app\.dart_tool, E:\Flutter Projects\fouad_app\.idea, E:\Flutter Projects\fouad_app\.pub, E:\Flutter Projects\fouad_app\build]}, clientRequestTime:<PHONE_NUMBER>464}
FileSystemException: Directory watcher closed unexpectedly, path = 'E:\Flutter Projects\fouad_app'
#0      _FileSystemWatcher._listenOnSocket.<anonymous closure> (dart:io-patch/file_patch.dart:309:11)
#1      _ExpandStream._handleData (dart:async/stream_pipe.dart:248:23)
#2      _ForwardingStreamSubscription._handleData (dart:async/stream_pipe.dart:164:13)
#3      _rootRunUnary (dart:async/zone.dart:1136:13)
#4      _CustomZone.runUnary (dart:async/zone.dart:1029:19)
#5      _CustomZone.runUnaryGuarded (dart:async/zone.dart:931:7)
#6      _BufferingStreamSubscription._sendData (dart:async/stream_impl.dart:336:11)
#7      _BufferingStreamSubscription._add (dart:async/stream_impl.dart:263:7)
#8      _SyncStreamControllerDispatch._sendData (dart:async/stream_controller.dart:764:19)
#9      _StreamController._add (dart:async/stream_controller.dart:640:7)
#10     _StreamController.add (dart:async/stream_controller.dart:586:5)
#11     new _RawSocket.<anonymous closure> (dart:io-patch/socket_patch.dart:1384:35)
#12     _NativeSocket.issueReadEvent.issue (dart:io-patch/socket_patch.dart:890:18)
#13     _microtaskLoop (dart:async/schedule_microtask.dart:41:21)
#14     _startMicrotaskLoop (dart:async/schedule_microtask.dart:50:5)
#15     _runPendingImmediateCallback (dart:isolate-patch/isolate_patch.dart:116:13)
#16     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:173:5)

For additional log information, please append the contents of
file://C:\Users\Amr-Gouda\AppData\Local\Temp\report1.txt.

Thanks for reporting this issue! This was previously reported at https://github.com/dart-lang/sdk/issues/39180; please follow along there.

Duplicate of https://github.com/dart-lang/sdk/issues/39180


Consumer loses connection to router

I have as environment Enmasse 0.15.3 and IoT Hub running in the same namespace of k8s cluster.
My health-check consumer looses sometimes connection to Qpid for several minutes. In the log I cannot see any reasons, except DnsNameResolver problem. This occurs after HealthCheck tries to execute reconnect to Qpid, but the Qpid pod is not ready.
The attached file contains log output of several pods (router, ragent, health-check and others).
I can see several "Detected Restart of Router Node Router.qdrouterd-798fd68c97-qsk9z", but I not see a reason.
The message from weave-npc is interesting: "EVENT UpdatePod" for qdrouterd pod. But we don't execute any k8s updates on this time. And "ragent info retrieved link routes for Router.qdrouterd" several times.
all.log

is obsolete


config-reloader: Remove readiness check

This PR is part of larger redesign of config-reloader alerting and SLI pipeline discused internally.
TL;DR: To denoise our internal alerts and SLI we remove the config-reloader readiness check. There is no benefit in checking readiness only at the container start compared to trying to reload and fail reload HTTP request straight away. Furthermore, we don’t check readiness after the initial check (target component can restart anytime), so we increment reloader_reloads_failed_total anyway (contradicting initial rationale).
Removal will give us:

Consistency and predictable failure mode for config reloader (all problematic error cases will consistently increment reloader_reloads_failed_total).
We stop restarting the container due to a broken target.
No extra readiness configuration item to maintain and set.

Thanks, responded in the document, let's keep discussion in one place.

NOTE: We are discussing the next steps internally.

Changing the front as discussed in the proposal. We propose to upstream this sidecar functionality to help not only ourselves, but also wider community: https://github.com/prometheus/prometheus/issues/9783#issuecomment-1669341273

Since then we went back - we might need sidecar anyway. This PR is still relevant to avoid unnecessary restarts which only blurs what's happening. Let's only increment errors.

I'm not sure if this is the right solution. I would have preferred either:
To go the other way, where we use the readyness check on restarts (in addition to startups).

I explained how this is non trivial to implement (race conditions and also expensive) in our design doc.

Keep initial readiness check to prevent the initial error total from incrementing at startup but use a metric that only increments once per-error (since current reloader_reloads_failed_total is noisy).

Hm, is it really helpful though? reloader_reloads_failed_total is noisy, so it's the restart signal these days (on healthy Prometheus we restart at least one config-reloader (!)). With this change we at least move to one noisy metric and we can iterate later further.
The only benefit of pre check is it to treat reloader_reloads_failed_total metric as situation when we "definitely" configured Prometheus well, just it's not healthy (because it responded once) - not sure if this is helpful for us though.
Also, any other metric will require new alert and new plumbing too.
Let's think about it more, moving to draft.

We didn't get consensus on this, so closing for now.


Design — add sign in button to header

Add the sign in button as a subcomponent in the header:

[x] in EN library
[ ] update EN poster
[ ] in FR library
[ ] update FR poster

Decision to put on hold until all typography and header alignment work is complete.


Update readthedoc setting

fix requirements value fixes #283

I'll PR again after configuration is fixed.


Pass global variables

Cannot find a way how to pass global variables to a script. Same issue as I mentioned in #45 about CancellationToken, but could be useful in general for other cases. Compilation accepts type of globals here, but cannot pass an actual value and access it from script.
After looking into Roslyn Scripting API, tried to pass an instance of Globals as object[0] to the entry point and access Foo from script:
public class Globals
{
    public string Foo { get; set; }
}

When I run host in-process, long-running scripts print output to all running scripts, because they channel the output via static ObjectExtensions.Dump. With a global variable we could provide dump channel per a script and use only Dump() instead of Console.Write (which is redirected to a custom TextWriter globally in AppDomain) to avoid printing output in all running scripts.

Hi Victor, I'm not sure I quite understand the scenario. Since RP only runs a single script in a host process at a time, there's no issue with Dump being globally routed.
Where would you provide the instance of the globals in RP?

Roslyn 2.3 works great with its scripting API for execution. Couldn't find a way to pass globals to RP's manual compiler.

This scenario seems to be what I am currently trying to do. I am working on an application that uses RoslynPad to write scripts. In the application I have some objects that I would like to pass to the RP as globals. Using https://github.com/aelij/RoslynPad/issues/129 I was able to get my custom classes to show up in the editor. But how can I pass global objects to the editor?

@quasar-pankaj please see my answer on SO.

@ quasar-pankaj请在SO上查看我的答案。

This can only be used for known types.
What if it's a dynamically generated variable?


Cannot read property 'layout' of undefined

Tried new spa mode and get this error message "Cannot read property 'layout' of undefined" with npm run dev on all pages except index.
Any idea what this can be? Thanks

Hi @husayt. Are you using the router in hashed mode? Any reproduction code or repo would be so nice.

Hi @husayt @pi0, not sure if this will help you... but I had exactly same issue in SPA mode. Issue is gone after removing @vuejs/vuex-router-sync from the project

Indeed @pi0 I am using hash mode. I will try to share reproducible code. Maybe the reason is that I have multiple layouts?
@kepek  no vuex-router-sync is not the cause for this one.

Could you create a reproduction repo for this @husayt please an re-open the issue when done?

I can confirm, that spa mode with hash routing indeed results in this issue. The index page gets loaded properly, but if you go to another page, and refresh there, the error message appears.
Repro: https://github.com/simplesmiler/nuxt-spa-hash-issue

@simplesmiler It should be fixed with #1532 with next release :)


Completion of error handling

Would you like to add more error handling for return values from functions like the following?

malloc ⇒ multibloom_new
pthread_create ⇒ brubeck_backend_run_threaded

Thanks! Fixed upstream!

Where do corresponding software updates appear?

I find that there are still a few update candidates left over since your commit a moment ago.
How do you think about to improve the source code a bit more?


st-term

My running ends up with the "stlinky magic not found in sram :(" error. The cause is that the "if (STLINKY_MAGIC == READ_UINT32_LE(sl->q_buf))" check never succeeds. The hard-coded STLINKY_MAGIC value is 0xDEADF00D. My printf() code finds out that the sl->q_buf value on my stm32f207vg production board is 0x09e25030. I replaced 0xDEADF00D with 0x09e25030 and ran again. The same error occurred.
I am have two questions:
What is the purpose of the st-term utility? Is it for logging the printf() messages from the stm32 board to minicom on stlink designated tty node?
How to make the STLINKY_MAGIC match READ_UINT32_LE(sl->q_buf) happen, in order to run st-term successfully?

An example of st-term which works would be great, the code appears to not be currently maintained.

A more robust way is to use the ARM Semihosting which is fully supported by OpenOCD and libc of https://launchpad.net/gcc-arm-embedded.

I currently don't see the point why the focus shifted to ARM Semihosting here as the question targeted st-term (now obviously st-util)...

Closed by #507.


init.lua#21
mt_teams.teams = {{color=mt_teams.colors.orange,
>       owner='singplayer',
        name='base',

Should that be 'singleplayer'?

yes, but it should cause any issues(yet)


x-ui panel with docker

Hello, I have installed the FranzKafkaYu panel and I have a question, how can I access the x-ui panel through Docker? I tested everything I thought.
In the end, when I executed Docker and entered the bash environment and gave the x-ui command, this happened:

please help me ,thanks

There is no such thing as x-ui panel. In FranzKafkaYu Panel, the x-ui panel is a shell script that allows you to control the x-ui service, such as install, uninstall and update. In docker, it's the x-ui binary. You do not need to worry about how to interact with the x-ui service.
But you can still use x-ui help to use the subcommand supported by the x-ui binary

How about the other two panels? Isn't there a panel for them in the Docker environment?
And what do you suggest to install two panels in one server?

No, there is no such panel because it's not needed at all.
It's using host network so if you want to install two panels you need to change the docker-compose.yaml file like using bridge network but you need export the port you want to use to run the xray or v2ray by your self.

i install with this command
mkdir x-ui1 && cd x-ui1
docker run -itd --network=host
-v $PWD/db1/:/etc/x-ui1/
-v $PWD/cert1/:/root/cert1/
--name x-ui1 --restart=unless-stopped
enwaiax/x-ui:alpha
and i add a number for every dir and name
can you tell me What else should I change?

Try to learn something about docker network model. What is host network and what is bridge network.
It makes no difference how many panel you install.

thanks for reply ,
have nice night


ffi(sdk): replace HashMap<Arc<EventId>, Arc<Timestamp>> with Vec<NegentropyItem>

Add NegentropyItem
Fix swift bindings compilation

Fixes https://github.com/rust-nostr/nostr/issues/360

Thanks for the fix.
It compiles now for Swift binding.
How about removing the Eq and Hash for EventId now that it's useless?


新增虚拟机栈抛出错误描述。

What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[x] Feature
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

The description of the PR:
增加了 OOM 错误的描述。
Other information:
参考文档

@paperant 👍


Stop using deprecated mysql.DialPassword

It came up here: https://github.com/db-operator/db-operator/pull/64
We need to migrate to other auth methods for google mysql
Link to a comment from the PR: https://github.com/db-operator/db-operator/pull/64#discussion_r1374452210

Closing, since gsql instances are deprecates


Add some kind of compatibility matrix with supported Zig versions

Since Zig sometimes introduces breaking changes in its build process (see e.g. #4), we should state explicitly which Zig versions are currently handled.

For now we will only support Zig 0.11.0, closing this since it will be indicated in the docs.


Add support for Ruby 3.0

I made a pull request because Ruby 3.0 release.
I'd be happy if you could merge it. 😃
Have a happy holidays again this year! 🎉

Hehe, happy holidays again to you too!


[GitHub Loader] Add support for loading specific folder and branch of a github repository

🚀 The feature
Users want to load a specific folder from a github repository. Moreover, they want to load the data from a specific branch and not the default branch.

Motivation, pitch
Requested by a user on discord community: https://discord.com/channels/1121119078191480945/1125758905310519327/1204150824868126790

@deshraj Can I pick this up?

@Dev-Khant sure go for it.

Hi @deshraj,
Here to get data for repo, branch and for specific folder I think using get_repo function from Github library would be easier compared to the current approach of cloning the repo and then traversing the tree. For extracting specific file we can directly use get_contents.
Docs:

get_repo: https://pygithub.readthedocs.io/en/latest/examples/Repository.html#get-all-of-the-contents-of-the-root-directory-of-the-repository
get_contents: https://pygithub.readthedocs.io/en/latest/examples/Repository.html#get-a-specific-content-file

I have previously worked around this approach: https://github.com/Dev-Khant/Analyze-Github-Code/blob/main/LLM/scrap.py#L33
Making a change to this will only affect the query with type=="repo".
Let me know if I can move ahead with this approach.

Yeah this seems like a reasonable approach to me as well. Please proceed with this approach.

@deshraj  Here do we need to store data from results because currently data variable is already getting replaced by self._get_github_repo_data. Let me know if we want to add data from results or just the content of repo.

Ah good catch. This seems like a bug and should be fixed. Can you please fix it in your PR?

Yes I can fix it. But do we have to add results to data or just the repo contents?

@deshraj I have raised the PR.


Add makefile/gruntfile

Needs to uglify and compile scss before pushing to appengine.

Apparently it already makes minified versions of js


Generator task for Rails controller with Phlex views

As mentioned here, it seems desirable to be able to emulate the Rails controller generator:
rails generate controller Home index about
This PR provides a single generator that creates a Rails controller with routes, actions, and the set of Phlex views. Each action renders the view, and the default content of the view matches that of the default ERB template for an action:
rails generate phlex:controller Home index about
This generates:
# app/controllers/home_controller.rb
class HomeController < ApplicationController
  def index
    render Views::Nice::Index.new
  end

  def about
    render Views::Nice::About.new
  end
end

# app/views/home/index.rb
module Views
  class Home::Index < Phlex::HTML
    include ApplicationView

    def template
      h1 { "Home#index" }
      p { "Find me in app/views/home/index.rb" }
    end
  end
end

# app/views/home/about.rb
module Views
  class Home::About < Phlex::HTML
    include ApplicationView

    def template
      h1 { "Home#about" }
      p { "Find me in app/views/home/about.rb" }
    end
  end
end

I hope this is useful, please let me know if there's anything that could be improved. Did you want a commit updating the docs too?
If #279 gets merged I'll have to come back and check that view file tests are created alongside the controller.
(I ran the tests btw, but I'm getting the same two failures before and after these changes)

@barrywoolgar this is great! Should we use the phlex:page generator for the views?

Yes, that would be a lot better than me half-duplicating the functionality - though I do think telling people where to find the view file is helpful. I'll have a look this evening at calling that code instead.

Thanks. I believe the test errors are unrelated to your change and caused by a gem update. I haven't tracked down which one yet. Maybe we should commit a Gemfile.lock.

I've sorted out the Rubocop errors, and had a look at using phlex:page. I feel like I should be able to define a hook_for somewhere but I'm not quite seeing it yet - will keep looking.
It seems like the phlex:controller code gets run even with a plain rails g controller, but that will trying to generate ERB templates unless you override the template_engine generator config to :phlex.
Is your preference for people to use the phlex: commands over the standard Rails generators?
Also, I've added in the Layout to the view files, but there's nothing to guarantee this class will exist. Do you think the phlex:install command should create a default Layout class file too?

It seems like the phlex:controller code gets run even with a plain rails g controller, but that will trying to generate ERB templates unless you override the template_engine generator config to :phlex.

Phlex isn't really an ActionView template engine — it's an ActionView replacement, so I don't think we can do that. I’m not sure the best way to stop Rails from generating ERB views but it might be best to keep the genreator namespaced under phlex: for now.

Also, I've added in the Layout to the view files, but there's nothing to guarantee this class will exist. Do you think the phlex:install command should create a default Layout class file too?

That's complicated. If you've installed Phlex in a new project, you'll probably want to generate a layout and use that from your page views. But if you're bringing it into a new project, you'll probably want to keep your ERB layout and use the phlex:view generator instead. (see https://www.phlex.fun/rails/migrating/)

Should we use the phlex:page generator for the action views since that will include the layout?

I figured out how to do that now, but I've used phlex:view instead - I'm not sure of the intended use for the various phlex:* generators is, but it feels more Rails-conventional to use phlex:view in a controller context.

it might be best to keep the generator namespaced under phlex: for now

Agreed. I think I had some "spare" code in my test app that was making me think this Phlex controller generator was running on a normal rails g controller. Sorry for the confusion.

That's complicated.

Good point about Layout being optional. I've added a naive check for app/views/layout.rb, but ideally we'd look for a view file that has Phlex::Rails::Layout as an included module. I wasn't able to get that working though, perhaps due to lazy class loading in development. If a layout is found it uses the wrapped markup, and if not the example Layout wrapper is commented out as example code.
I think the USAGE files are bit out of sync, but I'll update them once you're happy the code is working as desired.

I updated the USAGE file to use articles as the example becuase I think the more common use-case is a plural resource.


[Feature request] Added support for copying and pasting webpage content and retaining the layout format and style of the original webpage content

Hello friends!
Copying webpage content and pasting it into a Markdown editor cannot retain the source format of webpage content, such as hyperlinks, emoticons, emoticons, bold, italic, underline, image resources, etc. The original format and style of webpage content.
Because so far, choosing VSCode to edit Markdown files has been unable to preserve the original formatting and style of content copied from web pages. This question is very disturbing. Looking forward to your further implementation of this feature. Thank you so much for all your hard work! 👍😜
OS: Windows10
Version: VScode 1.81
Version: 1.9.1690302206

The reason why this function is requested is that when you rely on network content for Markdown document editing, then it is very useful at this time, and so far the VSCode extension center cannot find an extension with similar functions. If the vscode-runme extension can add this feature, it will be a boon for users who often rely on web content for Markdown document editing. 😜

@zhouxinghong thanks for raising the issue.
Unfortunately Runme (in this context) is "only" a VS Code extension and the feature you are requesting is currently out of scope for us. It should be ideally be implemented as part of VS Code or as an individual extension.

OK! Thank you very much for your reply!
I raised this issue with VSCode but they said that this functionality should be implemented by an extension...
Although this function is beyond the scope of your current work, considering that it can make runme have a better user experience, I still hope that you and your excellent team can seriously consider developing this function as a sister extension with runme . Because all professional Markdown editing tools have this function, and runme is the best Markdown editing tool in the VSCode community. If the runme user experience is getting better and better and the functions are getting more and more perfect, it will also be very helpful for runme to accumulate more users. Although other professional Markdown editing tools have this function, they are always commercial software. Therefore, as the functions of runme become more and more perfect, I believe that most of them should choose to use runme in VSCode to edit work and study documents like me. 😜


maybe: show better error msg if windows visual studio git outputs linux paths

So, ultimately, I think this is a PATH misconfiguration problem on my end, and fixing it has fixed my issue with GitInfo.
I'm running windows 10 + GitInfo latest 2.1.2, visual studio 2019 community edition.
I installed via Nuget and got a cascade of weird errors (like not being able to find the 'System' namespace)
My .sln file was in a directory, let's call it D:\projects\project1\
The clue was this error message here:
Severity	Code	Description	Project	File	Line	Suppression State
Error		The specified task executable "cmd.exe" could not be run. System.IO.DirectoryNotFoundException: The working directory "D:\d\projects\project1\" does not exist.
   at Microsoft.Build.Tasks.Exec.GetWorkingDirectory()
   at Microsoft.Build.Utilities.ToolTask.GetProcessStartInfo(String pathToTool, String commandLineCommands, String responseFileSwitch)
   at Microsoft.Build.Utilities.ToolTask.ExecuteTool(String pathToTool, String responseFileCommands, String commandLineCommands)
   at Microsoft.Build.Tasks.Exec.ExecuteTool(String pathToTool, String responseFileCommands, String commandLineCommands)
   at Microsoft.Build.Utilities.ToolTask.Execute()

The path is incorrect. Should be:
"D:\projects\project1\"
Instead, it's:
"D:\d\projects\project1\"
In the script, gitinfo uses git rev-parse --show-toplevel to get the path.  If I run the visual studio version of git, I get this correct output:
"C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\Common7\IDE\CommonExtensions\Microsoft\TeamFoundation\Team Explorer\Git\mingw32\bin\git.exe" rev-parse --show-toplevel

gives correct output of:
"D:\projects\project1\"
But what was happening was I had mingw64 version of git.exe (likely... incorrectly) in my path, and it was returning the unix-style path:
git rev-parse --show-toplevel output was /d/projects/project1
Somewhere in GitInfo it was attempt to marry /d/project/project1 and D:\ and produced an incorrect path of:
"D:\d\projects\project1\"
Then it all failed.  Fixing my path fixed the issue and now everything's fine.

I don't know if this bug is worth trying to figure out, or if it's better to say 'silly user, please fix your busted path'.
Either way, it might be nice to at least detect the condition and either convert to a windows-style path.
Or at least display some kind of error message to give a hint about what to fix.  Something like "We detected your version of git using unix-style paths, we need you to fix your PATH so it has a windows-like path".
You could do it by looking for patterns like:
"{someuppercaseletter}:\{thesamebutlowercaseletter}\" to catch strings that look like this: "D:\d\[whatever]"
Anyway, otherwise the plugin looks awesome, was just a little confused tracking down the issue as a first time user.
Thanks!
 :(
Yeah, I think the targets could check that the path separator matches the detected OS. Might not be trivial to do it in all relevant places.
Maybe you could take a shot at sending a PR?
Thanks in advance, I'll keep it open as an enhancement in case someone from the community wants to take a shot at it.

Ksu, if I have time, I'll give a pull request a shot that at least drops a breadcrumb pointing to the solution if a user hits this issue
Totally unrelated to this issue: I just realized I think I was using your ObservableDictionary gist in a project I was messing around with recently. Small world :)

Haha, awesome!! 😍


textfields in form emitting multiple UPDATE_QUERY actions, causes repainting

by removing the need to have the QueryPickerItem concurrent with the Query Form q field, we no longer need to do these multiple query updates. It is likely that this is fixed now, will confirm


Memory leak fix on JPClassHints

Adds a clean up routine to JPClassHints.
Fixes #654

Codecov Report

Merging #655 into master will increase coverage by 0.03%.
The diff coverage is 75.00%.

@@             Coverage Diff              @@
##             master     #655      +/-   ##
============================================
+ Coverage     85.07%   85.10%   +0.03%
  Complexity      170      170
============================================
  Files           118      118
  Lines          8718     8723       +5
  Branches       3682     3684       +2
============================================
+ Hits           7417     7424       +7
+ Misses          769      767       -2
  Partials        532      532

Impacted Files
Coverage Δ
Complexity Δ

native/common/jp_classhints.cpp
77.77% <75.00%> (+0.22%)
0.00 <0.00> (ø)

jpype/_jcustomizer.py
82.56% <0.00%> (+0.16%)
0.00% <0.00%> (ø%)

native/common/jp_exception.cpp
76.31% <0.00%> (+1.05%)
0.00% <0.00%> (ø%)

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 d63a6e5...bb3e2e8. Read the comment docs.


auto-Localization issue with xdai.io

Xdai.io several times in different devices is booted with the new hebrew translation localisation as default. that's cool for the users using that as their primary language
but as this is not an alphabet-based language it's a rather-confusing experience for everybody else!
Issue occurred on several devices and on chrome browser.
local language of the device was Portuguese, so expected behaviour was a default english-based localization.
i think it's might be an issue related to the recent hebrew PR :) link here : https://github.com/austintgriffith/burner-wallet/pull/118
good luck finding the bug!

I initialized a wallet today and this happened. https://nimb.ws/1qY9u5

@tomislavmamic may i ask what's your device current language setting and OS?
My empiric-based guess for this bug is that if the device is not setup to one of the main localization
language, (EN-ES-FR-DE-CA it'll set it to Hebrew as some "default-language".
@austintgriffith

@austintgriffith may make sense to have EN as default-language if @marczeller is on the right path (and I think he is 👍)

@mkosowsk that's the hard part then bacause the fallback is set to EN :

so i'm really clueless here.

@marczeller really got me scratching my head too 🤔
It must be some type of override then I guess, but where? 🤔

https://github.com/austintgriffith/burner-wallet/commit/1a004c92ae90876292b7f15d247d32e17730745d
that commit didn't worked 😂

This is still happening. I was with a vendor today and his wallet initiated with the Hebrew language.
It's quite urgent to solve this as we have an event happening in 2 days. Can you please resolve this?


Dependency conflict while installing

Hey @jwparktom , I tried to get this run locally, but ran into errors when doing npm install, looks like there are some dependency package conflicts, but I'm new to this so not sure how best to fix it. I'm attaching the log and shell output here so you can see my npm version as well. Hopefully it's just a matter of relaxing/narrowing some package versions, thanks!
eresolve-report.txt
2021-11-22T13_50_00_748Z-debug.log

Yes, I also faced that error. Since we plan to use React 17, you can solve it with the --force flag. So run
npm install --force

I will update the README.md to reflect the command.

Thanks @jwparktom , I added the --force flag, however now it's prompting something else:

Have you run into these? If not I'll just follow the steps to see if I can get it fixed, thanks!
PS, I updated the README with some links to get npm, feel free to edit as you see fit!

That's weird 🤔. I've just removed the lottie-react package since it's known to cause the dependency error between React versions. So running these commands (line by line) should make the app work now!
git pull
rm package-lock.json
npm install
npm start

Unfortunately	that doesn't appear to work, so I followed the instructions in

Steps 1-4 didn't work and I didn't want to mess with yarn, but following step 7 shows that two of the libraries have different requirement toward babel-loader and I think that's the culprit:

I'm not sure if that's something you can fix in package.json
Since it looks like you could ignore this error, so I added the .env file to skip the check, and it worked
BTW, it took a solid 5-10 sec to load the page, is it normal? When using Jekyll, it's like instant, but probably because it's static.
Thanks!!!

I will try to find what's wrong with the dependencies, but I think you are right. The upside of node/npm is the richness of dependencies, and the downside is the hassle of linking the old and new ones together.
Yes, the first load of npm start always takes some time since it's building the whole source code. Once that's done though, the hot reloading should happen almost immediately. Jekyll and other static frameworks (like Hugo or Gatsby) are built to be fast and simple :)

ah I see, thanks Jongwon! I'll close this now, but if you fix the dependencies in the future, it'll be great to tag this issue, thanks!


Embedded tool script vanishing at the scene opening in the editor if it contains an init method awaiting arguments

Godot version:
V3.2.2.stable.official
OS/device including version:
Debian 10 Buster Cinnamon
Issue description:
When I open a scene in the editor to which the following (minimal) embedded script is attached, it disappears :
tool
extends Control

func _init(value : int) -> void:
	pass

The following error appears in the output :
modules/gdscript/gdscript.cpp:127 - Condition "r_error.error != Variant::CallError::CALL_OK" is true. Returned: __null
Steps to reproduce:
Simply add the content given above to an embedded script and open it in the editor.
Minimal reproduction project:

Duplicate of #28986


Cannot Build Spin from Released Source Code

Building Spin from a Spin release source code fails for all releases from v1.2.0 and onwards due to required Git information needed in build.rs:
fn main() {
    vergen::EmitBuilder::builder()
        .build_date()
        .build_timestamp()
        .cargo_target_triple()
        .cargo_debug()
        .git_branch()
        .git_commit_date()
        .git_commit_timestamp()
        .git_sha(true)
        .fail_on_error()
        .emit()
        .expect("failed to extract build information");
...

This was added in https://github.com/fermyon/spin/pull/1492/files in order to populate more information to plugins. We should set empty defaults for these values if the do not exist.
To repro
curl -LRO https://github.com/fermyon/spin/archive/refs/tags/v1.3.0.tar.gz
tar -xvf v1.3.0.tar.gz
cd spin-1.3.0
cargo build

Output
error: failed to run custom build command for `spin-cli v1.3.0 (/Users/kagold/Programs/temp/spin-1.3.0)`

Caused by:
  process didn't exit successfully: `/Users/kagold/Programs/temp/spin-1.3.0/target/release/build/spin-cli-022ecc32c217a5a5/build-script-build` (exit status: 101)
  --- stderr
  thread 'main' panicked at 'failed to extract build information: not within a suitable 'git' worktree!', build.rs:28:10
  note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
warning: build failed, waiting for other jobs to finish...
error: failed to compile `spin-cli v1.3.0 (/Users/kagold/Programs/temp/spin-1.3.0)`, intermediate artifacts can be found at `/Users/kagold/Programs/temp/spin-1.3.0/target`

This probably has been an issue since https://github.com/fermyon/spin/pull/1209 as the Rust SDK requires the git commit sha information too for proper versioning


[BACKLOG-40978] 10.2: Default driver class used is org.gjt.mm.mysql.driver instead of recommended com.mysql.jdbc.Driver

BACKLOG-40978 10.2: Default driver class used is org.gjt.mm.mysql.driver instead of recommended com.mysql.jdbc.Driver

 Note:

Frogbot also supports Contextual Analysis, Secret Detection, IaC and SAST Vulnerabilities Scanning. This features are included as part of the JFrog Advanced Security package, which isn't enabled on your system.

🐸 JFrog Frogbot

:warning: Build finished in 19m 51s
Build command:
mvn clean verify -B -e -Daudit -Djs.no.sandbox -pl core

:no_entry: Failed Tests
:cloud_with_lightning_and_rain: 3 test(s) failed:
org.pentaho.di.connections.vfs.provider.ConnectionFileProviderTest.testGetChildren (click to expand)
${result.errorDetails}

org.pentaho.di.connections.vfs.provider.ConnectionFileProviderTest.testGetFile (click to expand)
${result.errorDetails}

org.pentaho.di.connections.vfs.provider.ConnectionFileProviderTest.testGetFileWithDomain (click to expand)
${result.errorDetails}

Tests run: 1282, Failures: 3, Skipped: 1    Test Results

:information_source:  This is an automatic message

 Note:

Frogbot also supports Contextual Analysis, Secret Detection, IaC and SAST Vulnerabilities Scanning. This features are included as part of the JFrog Advanced Security package, which isn't enabled on your system.

🐸 JFrog Frogbot


support urllib3-dev

A user complained about this this weekend. Open to review / comments.

Feature request revoked


Add functions that modify privacy settings

In Settings, if you go to Privacy > General, these functions can disable and enable the
"Let websites provide locally relevant content by accessing my language list"
"Let Windows track app launches to improve Start and search"
and
"Show me suggested content in the Settings app"
options. I was not sure how I should name these functions, and phrase their comments, so please change them (or suggest a change) before merging this PR. Their locations in the script may also need to be changed.
So far, I have tested this in Windows 10 Pro 1709

"Let websites provide locally relevant content by accessing my language list" -> Shame there isn't a GPO for that. About the name... hmmm... DisableWebLangList ?
"Let Windows track app launches to improve Start and search" -> This is mostly harmless, so I'm not sure if it's even worth to have it there. Either I would squeeze it into existing DisableAppSuggestions (In a way it is a start menu suggestion) or have it separate as DisableAppLaunchTracking. I think I like the former more.
"Show me suggested content in the Settings app" -> Definitely add to existing DisableAppSuggestions. It already has a few friends there.


Feature/add frozen trustlines status

Closes: https://github.com/trustlines-protocol/clientlib/issues/241
closes: https://github.com/trustlines-protocol/clientlib/issues/242
Add information as to whether a trustline is frozen. Since it was not too much work, also add ability to freeze a trustline with prepareUpdate()
See https://github.com/trustlines-protocol/relay/pull/313 for updated relay api with frozen TL info
See https://github.com/trustlines-protocol/contracts/pull/233 for new CurrencyNetwork TL update function signatures with isFrozen on updateTrustline()

Codecov Report

Merging #243 into master will decrease coverage by 27.85%.
The diff coverage is 0%.

@@             Coverage Diff             @@
##           master     #243       +/-   ##
===========================================
- Coverage   61.44%   33.58%   -27.86%
===========================================
  Files          18       18
  Lines        1040     1045        +5
  Branches      200      203        +3
===========================================
- Hits          639      351      -288
- Misses        344      670      +326
+ Partials       57       24       -33

Flag
Coverage Δ

#integration
33.58% <0%> (-0.17%)
:arrow_down:

#unit
?

Impacted Files
Coverage Δ

src/typings.ts
50% <ø> (-50%)
:arrow_down:

src/CurrencyNetwork.ts
26.66% <ø> (-71.12%)
:arrow_down:

src/Trustline.ts
21.73% <0%> (-56.43%)
:arrow_down:

src/Transaction.ts
41.37% <0%> (-58.63%)
:arrow_down:

src/Event.ts
19.17% <0%> (-53.43%)
:arrow_down:

src/utils.ts
28.69% <0%> (-44.35%)
:arrow_down:

src/signers/Web3Signer.ts
25% <0%> (-40.91%)
:arrow_down:

... and 5 more

Continue to review full report at Codecov.

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


Error with ResourceDictionary

Hi all.
When I add a new XAML file with ResourceDictionary to a WPF project, I get an error : " ... packages\Obfuscar.2.2.15\build..\tools\obfuscar.console.exe " obfuscar.xml " with code -1073741819

Shipped 2.2.17 with another BAML implementation to fix the issue completely.


image doesn't fit preload window, scrolled splash

First of all, thank you for making such an app. It works well on my Ubuntu 16.04 platform.
Splash screen or preload window image size unproportionate, so it scrolls the image to fit the window.
https://ibb.co/h1HAn6
I wonder if it's specific to my platform.
Regards.
Kenn

Added my own issue to this : also under Ubuntu 16.04 (Kubuntu) , no splash screen is been loaded at all.


Migrate to Swift 5

Migrate to Swift 5

@juvs Any idea if this will be merged soon?

Hey, sorry about the delay on this! I haven't had nearly enough time to give this stuff the attention it deserves. I think this all looks good with a couple of caveats:

master is, unfortunately, in a semi-finished state right now. I started a fairly major refactor a long time ago and haven't had time to finish it up. So if the intent here is to get the shipping version onto Swift 5, we'll probably have to approach this differently. I still might merge this PR since it makes sense for master to be on Swift 5, but it's worth thinking about.
I'm still unclear on how swift version bumps like this affect our versioning scheme. I think that the safest way forward here is probably to bump the shipping version to Swift 5 and then give it a major version bump? Does anyone have any thoughts/opinions on this?

@gfontenot Hey! Thanks for such a quick response! Those are both good points.
It would be ideal if the shipping version supported Swift 5 just to make installation/development easier, but I understand this may not be as easy as merging this PR and creating a new release.
In terms of the versioning scheme, I was expecting any Swift 5 updates to cause a major version bump anyways.
What are my current options for using Argo on a brand new project targeting Swift 5?

Ok so here's the plan:

I'm going to merge this (after verifying locally on master that no additional changes need to happen)
I'll update the current shipping version to 5.0 locally
I'll release that as v5.0

I'll post back here once all of that is done.

Ugh, actually, this is going to take me a minute because I have to fix the Package.swift files as well for the new tool versions. I'll spend some time on this tonight though.

Oh! Turns out: I've gone through this once already. Version 4.2 of Argo adds support for Swift 5 (via the Swift 4.2 tooling), so I think that should work fine? I'll still cut a new release based off that one that adds support for 5.0 and 5.2 explicitly, but it's good to know there's already a solution. I'm also going to close this PR in favor of a more comprehensive PR that updates our dependencies and adds better SPM support.

Version 5.0 has been released which explicitly supports Swift 5, and I've also added these changes (along with some others) to master.

@gfontenot Thanks mate! That's great news

@gfontenot Will the CocoaPod Specs Repo be updated to point at 5.0? It seems it still points at 4.2.0

Ah crap, sorry about that. It's up now!


Highlight linked issue comment

Highlight linked issue comments with :target and modified color. Hope this is what you expected @bfred-it.
Test comment : https://github.com/sindresorhus/refined-github/pull/988#issuecomment-357531152

Close #1164

Not sure whether related, But reaction avatars of linked comments are back to default GitHub style 😕
Test comment: https://github.com/sindresorhus/refined-github/issues/1122#issuecomment-370383235

It should probably include all the rules from GitHub, to make sure it works in every case. We can just pick the coloring properties and drop the rest. Their code is:
.timeline-comment-wrapper .timeline-comment.unread-item::after,
.timeline-comment-wrapper .timeline-comment.unread-item::before {
    position: absolute;
    top: 11px;
    right: 100%;
    left: -16px;
    display: block;
    width: 0;
    height: 0;
    pointer-events: none;
    content: " ";
    border-color: transparent;
    border-style: solid solid outset
}

.timeline-comment-wrapper .timeline-comment.unread-item::after {
    margin-top: 1px;
    margin-left: 2px;
    border-width: 7px;
    border-right-color: #fffdef
}

.timeline-comment-wrapper .timeline-comment.unread-item::before {
    border-width: 8px;
    border-right-color: #e5d999
}
.timeline-comment.unread-item {
    border-color: #e5d999
}

.timeline-comment.unread-item .timeline-comment-header {
    background-color: #fffdef;
    border-bottom-color: #e5d999
}

.timeline-comment.unread-item .timeline-comment-label {
    border-color: #e5d999
}

.timeline-comment.unread-item .previewable-comment-form .comment-form-head.tabnav {
    color: #e5d999;
    background-color: #fffdef;
    border-bottom-color: #e5d999
}

This is how linking my own comments look like:

And this is what it should look like (screenshot made with .unread-item)

@bfred-it Wouldn't using JS enable the functionality from GitHub where the custom styling disappears after a bit?

I don't see why that would be desirable; the comment should be colored as long as the :target points to it, like the rest of the site (e.g. permalinked code)

Updated with suggested modifications. But few questions about it,

Should we modify the color of border around reactions ?
Can't find a selector to change color for ::before with .current-user. Current selector affects all the comments by user on thread.

I posted the whole code, there's a couple ::before/::after selectors there that I don't see in your commits. All of that needs to be in there or else we're just playing whack-a-mole.

I'm extremely sorry for overlooking ::after, Will make the necessary modifications.

Each selector needs :root at the start otherwise it's not strong enough to override the .current-user style.

Screenshot for readme

Thanks for your PR, @salmanulfarzy!
By the way this is particularly useful when opening notifications because now the least recent unread comment will be highlighted (because the notifications link to it)

@salmanulfarzy thank you so much! this has been a real help for me!


loadDataWithBodyRetry(): Set maxRetries to 3 and add incremental delay

Ticket Link
GROUPINGS-1537
Notes

Used Math.log function so that the first retry has no delay (0 milliseconds)

log(maxRetries / retries) = 0 (Where maxRetries = 3 and retries = 3 on the first retry)

2000 was picked to give a value of ~800 ms on second retry and ~2000 ms on the third

Test Checklist

[x] Exhibits Clear Code Structure:
[x] Project Unit Tests Passed:
[x] Project Jasmine Tests Passed:
[x] Executes Expected Functionality:
[x] Tested For Incorrect/Unexpected Inputs:

Love it!


Innate spellcasting shows incorrect proficiency modifier

I created an elf monk with stats 4str, 2dex, 0con, 1int, 2wis, 0cha.
Took the spellcasting ancestry feat otherworldly magic to get a cantrip.
Under the proficiency tab on the character sheet, under spells, spell attack modifiers & DCs are marked as trained at +1.  It should say +3.
When I changed the spellcasting ability on the spellcasting tab, to strength, the number on the proficiency tab did not change to +7.

As spellcasting abilities can use different attribute modifiers, the proficiencies tab just shows your proficiency bonus.

That's fair. But it also doesn't even show that. The bonus is+3 not+1

Innate spells are normally cast with trained proficiency, but that does not give one a general spellcasting proficiency. If anything, nothing should be shown in the proficiencies tab.

Would you mind posting the character? There's not enough information to go off of here.


Current implementation of "leave" in Calc is insufficient

Consider a function with nested loops, where inside the innermost loop we want to leave the function on a certain condition. Just "leave x" will exit the innermost loop, but not the outer one(s), and certainly won't exit from the function.
Java handles this with a "named break" where any statement can have a label, as well as a "return" statement.  So, maybe "return" in addition to "leave" would be simplest....

I think I want to implement this as follows:

Add optional "label" syntax to "loop" and "while"
Add optional ": label" syntax to "leave" as in "leave : loop1 123"
Allow "label" to refer to a function name as well as the loop/while labels.

Changing to leave loop1: 123 instead (it repeats the label format itself and seems to flow more nicely).
Note that "leave" inside a function has to work differently too: either there is a label on the "leave" and it matches the function name, or there is no label either way. The only way it doesn't get caught is if there is a label and it doesn't match the function name. Also, the outer level catch has to produce an error if there was a label on the "leave" (meaning nobody matched on the way up).

I briefly considered adding (optional) labels to next as well as leave, but I don't think it would solve any pressing needs, and would be confusing since next can be used inside case, but not inside a function, so the parallel to leave is not perfect. But (obviously) we could reconsider this possibility in a separate issue if the need / utility ever arises in the future.


run in background

i'm use verdaccio in linux,
i want to use in background verdaccio
/usr/local/node/bin/verdaccio /usr/local/log/verdaccio.log &

Hi @bhabgs,
You can use pm2 or any other solution to run verdaccio, also you can always send processes to the background on Linux (using &, screen and more).

Thank you, I'm use forever in linux success.


Faq page looks weird on Netlify deployment

Maybe wrapping the Faq component in Bootstrap's Col and Row will fix the issue.

Still not working: https://deploy-preview-43--stoic-noether-907ad0.netlify.app/faq
@ubien if we commit and push changes to the fix-faq branch, it gets automatically added to the #43 PR.

Ok thanks for trying I'll take a look

Implemented Bootstrap accordion which looks better.  Never figured why styled-component was applying these had to override:


File upload using multi curl?

What is the expected behavior?
To upload multiple files using multi curl to speed up the uploading of multiple files!

Thanks for the suggestion!

Here's an example https://github.com/php-curl-class/php-curl-class/blob/master/examples/multi_curl_upload_file.php


SBG Venue Deployment

SBG Venue Deployment
Venue: SBG
Project: development
Venue identifier: sbg/development
Acceptance Criteria

SPS deployment (Hysds or Airflow -  see dependencies SPS)
Data Bucket (DS)
Data Collections (DS)
Jupyter (ADS)
SDAP (AS)
Venue API Gateway (CS)
Cognito groups, users (SBG)
Lambda Authorizer (CS)

Work Tickets

[ ] https://github.com/unity-sds/sbg-workflows/issues/12

Dependencies

[ ] https://github.com/unity-sds/unity-project-management/issues/114

@galenatjpl, @ngachung, @mcduffie, @jjacob7734 please create and/or link your work tickets that cover the acceptance criteria shown above

@GodwinShen I added two work tickets for these above.


The window does not close when you open the success stories

The only way to close the window would be to go back on the browser.

Is this still an issue? I tried to reproduce on the site on desktop, and mobile was having trouble reproducing.

@kento726 Which browser?

@ambethia Chrome, I think it's working now

But I suggested to Justin that we could close the window on an off window click.

The site is probably going to be getting a full rebuild/redesign in the
very near future.
On Mon, Oct 14, 2019 at 11:11 AM Kento<EMAIL_ADDRESS>wrote:

But I suggested to Justin that we could close the window on an off window
click.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/suncoast-devs/web/issues/272?email_source=notifications&email_token=AAAACKRZ77XQLFY6TNVLZG3QOSD23A5CNFSM4I2P6TEKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBFEF3I#issuecomment-541737709,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAACKRXSUQ5BAJYUBVDUBLQOSD23ANCNFSM4I2P6TEA
.

--
Jason L Perry
CEO at Suncoast Developers Guild
(727) 201-2012
<EMAIL_ADDRESS>

@ambethia would love to help! I have been studying Gatsby a ton.


Is there any way to use this project with Spring Cloud Gateway?

Hey, first of thank you for developing this. I have an issue, I'm trying to get contextual logging working with a gateway application. I've tried several different solutions and failed since its either resource intensive (Spring Sleuth) or it can produce misleading results (https://www.novatec-gmbh.de/en/blog/how-can-the-mdc-context-be-used-in-the-reactive-spring-applications/.  ( Correlation ids showing up in logs that were not triggered by my request but rather some background process that uses WebClient))
.
Ive wired up your solution and it does seem to add a correlation_id to the reactive context but the QudiniLogEvent always has an empty context.  Below is an example of a log line generated in the HttpWebHandlerAdapter. Notice theres nothing in the MDC
{"timestamp":"2021-11-02T21:43:06.058Z","level":"DEBUG","message":"[9d56bc30-2, L:/0:0:0:0:0:0:0:1:8085 - R:/0:0:0:0:0:0:0:1:51859] HTTP GET \"/services/unified/v1/authn/.well-known/webfinger?resource=iampsrtenant-robin3&tenantAlias=IAMPSRTENANT288\"","build_version":"unknown","thread":"reactor-http-nio-6","logger":"org.springframework.web.server.adapter.HttpWebHandlerAdapter","env":"unknown"}
Do you know how to get this working for a project based of spring cloud gateway?
I've gotten it to where it will output
{"correlation_id":"dca04dc2-a4f1-4242-9932-dd1971948bda","timestamp":"2021-11-03T14:06:45.714Z","level":"INFO","message":"TESTTESTTEST","build_version":"unknown","thread":"reactor-http-nio-6","logger":"com.something.gateway.filters.CorrelationFilter","env":"unknown"}
However that is when the log statement is inside Log.thenMono. Is there any way to do this throughout the whole request chain? Starting at the httphandler?
https://spring.io/projects/spring-cloud-gateway#samples

Hey,
The correlation id will be available only if the log has been produced from within one of the Log#... static methods (see https://github.com/qudini/qudini-reactive/tree/master/qudini-reactive-logging#comqudinireactivelogginglog).
Any log produced outside one of these (e.g. just like what HttpWebHandlerAdapter does) won't have the MDC populated, because it doesn't know the reactive context holds it: we need to programmatically populate it each time, hence the need of these Log#... static methods (see https://github.com/qudini/qudini-reactive/blob/master/qudini-reactive-logging/src/main/java/com/qudini/reactive/logging/Log.java#L290 for the actual implementation).


Remove unused code

There was no case when cache_dir would == "".

Thanks for the PR! This is currently used:
λ XYZ= python -c 'import os; print(repr(os.environ["XYZ"]))'
''

Seems like if the environment variable is set to an empty string, it could be empty.


Remove vscode devcontainer

This feature is not actively used and has caused some confusion with
new contributors trying to run the full test stack in the dev
container, which we don't expect to work.
Signed-off-by: Russell Bryant<EMAIL_ADDRESS>

I think it should be removed from the https://docs.nexodus.io/development/tooling/ doc too

I think it should be removed from the https://docs.nexodus.io/development/tooling/ doc too

Good catch!  Removed. I searched for "devcontainer" but not "dev-container" !


Add pyproject.toml for Custom Node Registry

Hey! My name is Robin and I'm from comfy-org! We would love to have you join the Comfy Registry, a public collection of custom nodes which lets authors publish nodes by version and automate testing against existing workflows.
Eventually, the registry will be integrated as a backend for the UI-manager where all nodes will go through a security scan. Nodes that pass these checks will have a verification flag (✅) beside their name on the UI-manager. Feel free to read up more on the registry here
Action Required:

[ ] Go to the registry. Login and create a publisher id (everything after the @ sign on your registry profile).
[ ] Add the publisher id into the pyproject.toml file.
[ ] Merge the separate Github Actions PR, then merge this PR.

If you want to publish the node manually, install the cli by running pip install comfy-cli, then run comfy node publish
Otherwise, if you have any questions, please message me on discord at robinken or join our server!

Thanks a lot.
As I now develop on the "devel" branch, I will make the changes.


An error with training on multiple GPUs

Hello author,
Thanks for this research, I want to train vitlarge_clip_k400 on 4 V100
but when I use the command with "bash tools/dist_train.sh configs/recognition/vit/vitclip_large_k400.py 3 --test-last --validate --cfg-options model.backbone.pretrained=openaiclip work_dir=work_dirs_vit/k400_vitlarge/debug".
It get an error with below
/home/u1043795/anaconda3/envs/AIM/lib/python3.7/site-packages/torch/distributed/launch.py:186: FutureWarning: The module torch.distributed.launch is deprecated
and will be removed in future. Use torchrun.
Note that --use_env is set by default in torchrun.
If your script expects --local_rank argument to be set, please
change it to read from os.environ['LOCAL_RANK'] instead. See
https://pytorch.org/docs/stable/distributed.html#launch-utility for
further instructions
FutureWarning,
Traceback (most recent call last):
File "/home/u1043795/anaconda3/envs/AIM/lib/python3.7/site-packages/torch/distributed/run.py", line 564, in determine_local_world_size
return int(nproc_per_node)
ValueError: invalid literal for int() with base 10: '--validate'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/u1043795/anaconda3/envs/AIM/lib/python3.7/runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "/home/u1043795/anaconda3/envs/AIM/lib/python3.7/runpy.py", line 85, in _run_code
exec(code, run_globals)
File "/home/u1043795/anaconda3/envs/AIM/lib/python3.7/site-packages/torch/distributed/launch.py", line 193, in
main()
File "/home/u1043795/anaconda3/envs/AIM/lib/python3.7/site-packages/torch/distributed/launch.py", line 189, in main
launch(args)
File "/home/u1043795/anaconda3/envs/AIM/lib/python3.7/site-packages/torch/distributed/launch.py", line 174, in launch
run(args)
File "/home/u1043795/anaconda3/envs/AIM/lib/python3.7/site-packages/torch/distributed/run.py", line 709, in run
config, cmd, cmd_args = config_from_args(args)
File "/home/u1043795/anaconda3/envs/AIM/lib/python3.7/site-packages/torch/distributed/run.py", line 617, in config_from_args
nproc_per_node = determine_local_world_size(args.nproc_per_node)
File "/home/u1043795/anaconda3/envs/AIM/lib/python3.7/site-packages/torch/distributed/run.py", line 582, in determine_local_world_size
raise ValueError(f"Unsupported nproc_per_node value: {nproc_per_node}")
ValueError: Unsupported nproc_per_node value: --validate

The command looks good to me. I am not sure why it is interpreting --validate as the number_of_gpus. Could you directly try the provided script run_exp.sh ?


Silence aux tools

This set of commits switches command line argument parsing to use getopt(3).
Arguments such as directory, image extension, and output are now handled using command line flags (guess what -d, -e, and -o do), which makes them position insensitive.
I've added some helpful sensible defaults, eg. if you ask to use a nonexistent font for keograms, you get a warning about that, and the program proceeds to use the SIMPLEX font. Keogram font color is now specified as a single hex value string, like colors in HTML, leading # is optional, so you can say something like -C ff0000 or --font-color '#ff0000', whatever you prefer. These don't appear to be used in endOfNight.sh
A help screen is also available with -h or --help. If argument errors are detected, the utility exits with error code 3 (as before), but if the help screen was requested then it exit with error code 0. It is not an error, after all, to ask the program how to correctly invoke it.
From there, I added a -v flag which is used to enable/suppress extra verbose output, such as that emitted for every file read by keogram or startrails. The default is not verbose.
Finally, endOfNight.sh was updated to take advantage of these new command line flags. Apologies for vscode being overly enthusiastic about normalizing all the spaces and tabs.

Chris, in keogram.cpp, shouldn't this line:
std::cout << "-r | --r <float> : number of degrees to rotate image, "
be
std::cout << "-r | --rotate <float> : number of degrees to rotate image, "
I thought one reason for keogram to display all the lines was to aid in debugging, if keogram stopped part way through the user could look at the log file (that's now removed in endOfNight.sh) to see where it stopped.  Is that no longer needed?
Ditto for startrails.  And didn't its output lines include a brightness level to aid the user in debugging problems?
When giving usage examples, I've been changing "ex:" in the scripts to "example:" to help users where english isn't their native language.

Wait'll you see what else I've got in the pipeline. 🧑💻
Anyway, I'm going with "it's probably no longer needed to emit verbose messages by default." I was reprocessing a bunch of startrails and keograms today, and found a few reasons why OpenCV would crash; the logs weren't particularly helpful. In fact, I just looked for the directories with missing keograms or startrails, then ran the program under GDB to find and fix the line of code that triggered the offending assertion. Previously you might want to know what file caused the failure so that you could delete it ... now you don't have to delete it, the programs just skip it and continue.
As for the summary statistics that used to emit - it still does that by default. I never found any value in the file-by-file brightness; the summary statistics were very helpful though, in picking some good settings, and those statistics aren't lost in all the rest of the spew.


VSCodium doesn't download

Script is trying to download the dmg and the dmg.sha1
Adding  && ! /sha1/  to the end of the curl command fixes the issue.
downloadURL=$(curl -fs "https://api.github.com/repos/VSCodium/vscodium/releases/latest" | awk -F '"' "/browser_download_url/ && /dmg/ && ! /sig/ && ! /CLI/ && ! /sha256/ && ! /sha1/ { print \$4 }")

This is fixed in version 9, the current dev branch. The label was updated to use Installomator's native GitHub support. Could you test and confirm?

@acodega Thanks. I can confirm this is fixed in version 9. Much appreciation.


Automatic Fisher doesn't support modded rods properly

Version: 1.9.5
Using a modded fishing rod such as the flux infused fishing rod from redstone arsenal will cause the rod to be deleted after one use, i'm assuming it's you check for durability and it doesn't use that so it's "one use".

Just realized this was reported before... In 1.10 i had put in a quickfix to at least stop it from deleting item, but it still didn't drain RF.  https://github.com/PrinceOfAmber/Cyclic/issues/388
Ill look into doing it the right way instead, meaning

check if the fishing rod item uses durability or RF and drain the right thing

which means pulling in the RedstoneFluxAPI, which i was bound to do eventually anyway
https://mods.curse.com/mc-mods/minecraft/270789-redstone-flux

Do you happen to know the item ids of the fishing rods that use energy instead of durability?
Is it ONLY from this mod or are there others?
https://minecraft.curseforge.com/projects/redstone-arsenal

https://minecraft.curseforge.com/projects/cyclic/files/2501283


Installed binary with duplicating name not added to path: case of npm.

Describe the bug
Having installed nodejs in my nix-profile environment, a newer version of - installed - npm is not available in path.
To Reproduce
Having added https://nixos.org/channels/nixpkgs-unstable to my channels with nix-channel add [url]

install nodejs with nix-env -i nodejs-10.16.3 -> puts node, npx, npm in path.
Verify npm version with npm --version -> gets 6.9.0
Install newer version of npm with nix-env -iA nixpkgs.nodePackages.npm -> installs version 6.11.3
Verify npm version with npm --version -> still 6.9.0

Expected behavior
The newer version, i.e installed later, should be found, i.e. first.
Metadata
 - system: `"x86_64-linux"`
 - host os: `Linux 4.19.74, NixOS, 19.03.173522.021d733ea3f (Koi)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.2.2`
 - channels(joe): `"nixpkgs-20.03pre196679.65d7bf58f11"`
 - channels(root): `"nixos-19.03.173522.021d733ea3f"`
 - nixpkgs: `/home/joe/.nix-defexpr/channels/nixpkgs`

workaround
There are 3 workarounds I've been using.

Manually add the installed npm binary to path.
Not install npm in nix-profile. Start a nix shell with nix-shell -p nodePackages.npm

However, npm now uses its own dependency node and not the one linked to by ~/.nix-profile/bin/node. Which entails another workaround: setting npm config scripts-prepend-node-path to true. Nix-shells also have other adverse-effects making this an undesirable choice.

Not install npm in nix-profile. In each project, install npm locally with npm install --save-dev npm. Then run npm via $(npm bin)/npm ... or add it to the path manually.

Another neater workaround is to install nodejs-slim (which is basically nodejs --without-npm). Then install, separately nodePackages.npm. Npm will still depend on its own copy of node, so the

setting npm config scripts-prepend-node-path to true

still applies.

still


Hyprland is not smooth on intel-iGPU

Hyprland Version
 System/Version info
Hyprland, built from branch HEAD at commit<PHONE_NUMBER>014997776e091ff8903acfb311dd2f  (props: bump ver to 0.38.0).
Date: Mon Apr 1 18:30:37 2024
Tag: v0.38.0

flags: (if any)

System Information:
System name: Linux
Node name: nitro5
Release: 6.8.2-arch2-1
Version: #1 SMP PREEMPT_DYNAMIC Thu, 28 Mar 2024 17:06:35 +0000

GPU information:
0000:00:02.0 VGA compatible controller [0300]: Intel Corporation TigerLake-H GT1 [UHD Graphics] [8086:9a60] (rev 01) (prog-if 00 [VGA controller])
0000:01:00.0 VGA compatible controller [0300]: NVIDIA Corporation GA107M [GeForce RTX 3050 Ti Mobile] [10de:25a0] (rev a1) (prog-if 00 [VGA controller])

os-release: NAME="Arch Linux"
PRETTY_NAME="Arch Linux"
ID=arch
BUILD_ID=rolling
ANSI_COLOR="38;2;23;147;209"
HOME_URL="https://archlinux.org/"
DOCUMENTATION_URL="https://wiki.archlinux.org/"
SUPPORT_URL="https://bbs.archlinux.org/"
BUG_REPORT_URL="https://gitlab.archlinux.org/groups/archlinux/-/issues"
PRIVACY_POLICY_URL="https://terms.archlinux.org/docs/privacy-policy/"
LOGO=archlinux-logo

plugins:

Bug or Regression?
Bug
Description
So, i just installed hyprland on a multi-gpu laptop. The first thing i  noticed is that , the it has very occasional and inconsistent framedrops.
My display is 144hz. And this is my monitor Config. monitor=,highrr,auto,1.
So to give you a better understanding. This is what is the case when i first login to hyprland session.

But after opening a few windows or moving them around, basically just working the hyprland animations. This is what i get.

So, basically the issue is that , there is always inconsistent framedrops. It just is not smooth.
If anyone knows a fix or solution, please reply.
To make sure everything is rendered on the iGPU, i ran this
OpenGL renderer string: Mesa Intel(R) UHD Graphics (TGL GT1)

and
Graphics:
  Device-1: Intel TigerLake-H GT1 [UHD Graphics] driver: i915 v: kernel
  Device-2: NVIDIA GA107M [GeForce RTX 3050 Ti Mobile] driver: nvidia
    v: 550.67
  Device-3: Chicony HD User Facing driver: uvcvideo type: USB
  Display: server: X.org v: <IP_ADDRESS> with: Xwayland v: 23.2.5 driver: X:
    loaded: modesetting,nvidia dri: iris gpu: i915 resolution: 1920x1080
  API: EGL v: 1.5 drivers: iris,kms_swrast,nvidia,swrast
    platforms: gbm,surfaceless,device
  API: OpenGL v: 4.6.0 compat-v: 4.5 vendor: mesa v: 24.0.4-arch1.2
    note: incomplete (EGL sourced) renderer: llvmpipe (LLVM 17.0.6 256 bits),
    NVIDIA GeForce RTX 3050 Ti Laptop GPU/PCIe/SSE2, Mesa Intel UHD Graphics
    (TGL GT1)

How to reproduce
Install Hyprland on a Multi GPU system with integrated intel GPU, and dedicated NVIDIA GPU.
Crash reports, logs, images, videos
No crash reports

could be #5426?
Have you checked if this happens on an older version, e.g. 0.36?

Yes , I have checked. I'm not sure about the version. But I had the same issue 4 months ago. So I decided to stay off until it hopefully got fixed. But unfortunately, it's still there.

generally due to how wlroots' frame scheduler works, if your gpu is unable to render a frame within the frametime window, the framerate will halve.
Though I don't see why opening a few windows in the background would be putting any significant additional strain on the gpu.

Exactly. I don't understand as well. When I start the session everything works fine. But then if I open something like thunar, and just by moving windows i can see it's not smooth, feels more like 60 or 70 fps, when it should be 144fps.
If there's anything you like to know, like a log, or anything you'd like me to check out, please do tell me.

if you are moving windows at the same time as the test is running, it's normal it will tank - rendering the movement of the window is a lot of pixels.

if you are moving windows at the same time as the test is running, it's normal it will tank - rendering the movement of the window is a lot of pixels.

Okay, that's expected. But I think the same happened even when the test wasn't running. But please allow me  to confirm it once I get back to my system.
Is there any possible way I could see an fps monitor, so I can actually determine the extent of performance loss.

set debug:overlay to true https://wiki.hyprland.org/Configuring/Variables/#debug

yeah odd. check 5426 as I feel like this could be the same issue.
Try reverting to 0.36-ish and check if it happens there too

Yeah , I'll do that

I tried out version 0.36 , installed it using cmake.
Hyprland, built from branch HEAD at commit 1c460e98f870676b15871fe4e5bfeb1a32a3d6d8  (props: bump ver to 0.36.0).
Date: Wed Feb 28 00:32:40 2024
Tag: v0.36.0

flags: (if any)

System Information:
System name: Linux
Node name: nitro5
Release: 6.8.2-arch2-1
Version: #1 SMP PREEMPT_DYNAMIC Thu, 28 Mar 2024 17:06:35 +0000

GPU information:
0000:00:02.0 VGA compatible controller [0300]: Intel Corporation TigerLake-H GT1 [UHD Graphics] [8086:9a60] (rev 01) (prog-if 00 [VGA controller])
0000:01:00.0 VGA compatible controller [0300]: NVIDIA Corporation GA107M [GeForce RTX 3050 Ti Mobile] [10de:25a0] (rev a1) (prog-if 00 [VGA controller])

os-release: NAME="Arch Linux"
PRETTY_NAME="Arch Linux"
ID=arch
BUILD_ID=rolling
ANSI_COLOR="38;2;23;147;209"
HOME_URL="https://archlinux.org/"
DOCUMENTATION_URL="https://wiki.archlinux.org/"
SUPPORT_URL="https://bbs.archlinux.org/"
BUG_REPORT_URL="https://gitlab.archlinux.org/groups/archlinux/-/issues"
PRIVACY_POLICY_URL="https://terms.archlinux.org/docs/privacy-policy/"
LOGO=archlinux-logo

plugins:

Unfortunately, the frame drops are still there. Even a small animation like just moving a window, drops the fps from 144fps to weirdly exactly 72 fps.
But one thing that's different from before , is once i don't do anything, i mean with animations, the fps returns to 144 fps, which it did not with version 0,38(where it just stayed at low fps)..

read my previous comments. That's expected.

I wasn't talking about running glxgears.
I was saying , without any test running, and just only hyprland, I monitored the fps using debug: overlay and it's dropping from 144fps to 72fps.
I just don't feel like it's expected since, I have a low end laptop with amd igpu and a 60hz display, and the fps is pretty stable.

please help us over there by bisecting this if you have a bit of spare time.

Will definitely do it tomorrow.
Thank you for your time and efforts.


🐛 Fixed temporary CSV generated by the DuckDBToSQLServer flow not b…

Summary

Fixed temporary CSV generated by the DuckDBToSQLServer flow not being cleaned up
Checklist

This PR:

[ ] adds new tests (if appropriate)
[x] updates docstrings for any new functions or function arguments (if appropriate)
[x] updates CHANGELOG.md with a summary of the changes

Test to be added in a separate PR.


Smooth scroll

after zooming and scrolling up and down it scrolls very slowly.

can you provide more information about your issue?  what steps are necessary to recreate it?  does it happen regardless of state (scale, scroll position, zoom level)?  does this happen just on one device, or all devices, or just on an emulator (if so, which emulator)?  etc.

any update on this?


ci: update dependabot configuration

This Pull Request fulfills the following requirements:

[x] The commit message follows our guidelines.
[x] Tests for the changes have been added if needed.
[x] Does not affect documentation or it has been added or updated.
[x] Does not introduce cycles into the flow of execution.

Resolves #

Pull Request Test Coverage Report for Build 928391587

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 809447975:
0.0%

Covered Lines:
76

Relevant Lines:
76

💛  - Coveralls


Critical Error: ERROR: the renter is idle

goroutine 175 [running]:
runtime/debug.Stack(0x445687, 0x0, 0xc43ba2bc28)
#011/usr/local/go/src/runtime/debug/stack.go:24 +0x79
runtime/debug.PrintStack()
#011/usr/local/go/src/runtime/debug/stack.go:16 +0x22
github.com/NebulousLabs/Sia/build.Critical(0xc43ba2bd80, 0x2, 0x2)
#011/home/team/Go/src/github.com/NebulousLabs/Sia/build/critical.go:16 +0xa7
github.com/NebulousLabs/Sia/persist.(*Logger).Critical(0xc420fa6b80, 0xc43ba2bd80, 0x2, 0x2)
#011/home/team/Go/src/github.com/NebulousLabs/Sia/persist/log.go:37 +0xe0
github.com/NebulousLabs/Sia/modules/renter.(*Renter).managedDownloadIteration(0xc420d56540, 0xc43ba2bf20)
#011/home/team/Go/src/github.com/NebulousLabs/Sia/modules/renter/download.go:369 +0x5be
github.com/NebulousLabs/Sia/modules/renter.(*Renter).threadedDownloadLoop(0xc420d56540)
#011/home/team/Go/src/github.com/NebulousLabs/Sia/modules/renter/download.go:632 +0x37e
created by github.com/NebulousLabs/Sia/modules/renter.newRenter
#011/home/team/Go/src/github.com/NebulousLabs/Sia/modules/renter/renter.go:208 +0x3ae
Critical error: ERROR: the renter is idle , but tracking active pieces: 7
Please submit a bug report here: https://github.com/NebulousLabs/Sia/issues
I'm building from source against master.

did you ever find out how to fix this? i am having the same problem

@jestes11 I did not figure out the root cause.  The best I could come up with is removing the renter/ directory.  You lose all contracts, but the value of those contracts will be refunded at the end of the contract period.

This is a duplicat this issue already exists let me find it.

https://github.com/NebulousLabs/Sia/issues/1591
I believe this is the same issue.

should be fixed in v1.3.0


No verbose return status

The utility returns 0 either it found a match or it did not.
It makes it pretty unusable in bash scripts where the status used as an indication either a pattern is present in a binary or not.

Yeah, that would probably help!
I haven't touched this project in a while. But if you want to fix it, I'd be happy to merge it!


Fix EditAssessment position

Also closes #178

What should be different on mobile?
On Sun, Mar 19, 2023, 4:21 PM Yunfan Yang @.***> wrote:

@.**** approved this pull request.
Looks good. Expecting a fix for mobile screens too.
—
Reply to this email directly, view it on GitHub
https://github.com/techstartucalgary/lifeline/pull/234#pullrequestreview-1347590628,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQZPOEUAD3GKP2DQUTIYTT3W46BGNANCNFSM6AAAAAAVXB66PQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


Fix For Installation

This seems to be working for me now.   I have go through the install process a few times and it works for me.

These are a lot of different changes together and I can't merge them like that.

Ok, no worries. How would you like me to update ?

I need to make some structural changes to the repo first, considering making main default branch. But I also see a lot of changes in your pull request that are unrelated or incorrect.

Ok.  I don't believe there are any errors as yaml complains, esp32 doesn't compile it when there are errors, so it wouldn't be working for me and I did follow the ESPHome documentation  https://esphome.io/guides/creators.html
I do make use of the secrets.yaml.. which could cause some confusion.
I was just trying to help.

I understand that and your help is certainly appreciated ❤
But for now I'll "park" your pull request and go through it again later to see if there are really changes that are useful.

Ok.   I think there have been some changes to the installation process.  Connect a device, it configures it as a basic YAML, then paste, then YAML, and it works.  Maybe that's the biggest change since the initial implementation.
Thank you for doing this project, its been awesome for me.

I have tested what I said above.   Your YAML works fine. LOL!   It's the installation process in the existing readme that confused the hecc out of me :-)   No need to copy it to the ESPHome Folder.

New ESP32 goto ESPWeb and flash with ESPHome, connect to wifi.
Edit the config, copy and paste your config over the top
Update Wifi Details.
Install
Download the BIN file.
Go back to the ESPHome Web and flash it on
Home Assistant Detects it as version 3.1.0


Create Set-SCOMMonitorMaintenanceMode.ps1

#sign-off

It doesn't work like that here unfortunately


Better settlement flow for staking rewards

Design needed: True
Relates to: Staking Rewards
When a user has recently traded for iETH or iBTC (applies to all inverse synths) they might need to reclaim or settle their inverse trade before they can stake their tokens.
The current flow blocks users from being able to claim rewards and perform other actions if this is true. We need something less intrusive.

Closed as iETH, iBTC rewards are now being deprecated for shorts


SOF-1500: Add three features for variants as infinites

Codecov Report

Merging #258 (7a784bc) into develop (c9e11f1) will decrease coverage by 0.10%.
Report is 19 commits behind head on develop.
The diff coverage is 58.33%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the GitHub App Integration for your organization. Read more.
@@             Coverage Diff             @@
##           develop     #258      +/-   ##
===========================================
- Coverage    68.13%   68.04%   -0.10%
===========================================
  Files          280      281       +1
  Lines        34136    34342     +206
  Branches      4574     4586      +12
===========================================
+ Hits         23259    23368     +109
- Misses       10414    10502      +88
- Partials       463      472       +9

Files Changed
Coverage Δ

packages/job-worker/src/events/handle.ts
22.19% <0.00%> (-0.13%)
:arrow_down:

...job-worker/src/ingest/syncChangesToPartInstance.ts
22.01% <0.00%> (-0.07%)
:arrow_down:

packages/job-worker/src/playout/actions.ts
62.54% <0.00%> (ø)

...ckages/job-worker/src/playout/timeline/generate.ts
69.32% <0.00%> (ø)

meteor/lib/Rundown.ts
32.30% <3.44%> (-8.87%)
:arrow_down:

...ackages/job-worker/src/playout/timeline/rundown.ts
71.19% <15.78%> (-2.81%)
:arrow_down:

...ueprints/context/syncIngestUpdateToPartInstance.ts
22.76% <33.33%> (+0.78%)
:arrow_up:

...kages/job-worker/src/blueprints/context/setNext.ts
35.48% <35.48%> (ø)

meteor/server/api/rest/rest.ts
85.10% <50.00%> (+0.57%)
:arrow_up:

packages/job-worker/src/ingest/generation.ts
71.10% <50.00%> (-0.06%)
:arrow_down:

... and 12 more

... and 1 file with indirect coverage changes


[UX][Publisher] Add help text for API definition URL Pattern

Description:
Add help text which includes URL Pattern explanation and an example.
Suggested Labels:
UX Improvement
Affected Product Version:
2.5.0

Jaggey based publisher app is no longer in use. Please reopen if this improvement is needed in the latest version.


Trouble with seeing collectible from artworks.json

I just started using the buyer-mints-artworks, and I'm having some trouble seeing the collectibles on localhost. I've also tried the same with simple-nft-example, and the console returns that the collectibles were minted and injected correctly, but none show up on the app. Any ideas of what I am missing here?

Adding some error messages, maybe second one is relevant?
Warning: Each child in a list should have a unique "key" prop.

Check the render method of `Ramp`.  If anyone has any idea on what's happening here, please let me know! Am closing since it's irrelevant to this repo.

How did you resolve the issue? I'm also not seeing images when loading the page.


Added 1) ResultDataModel and it's REST API services 2) ActiveExamData…

Added 1) ResultDataModel and it's REST API services 2) ActiveExamDataModel(  you will need to make it Tempropary Table) and REST API services For them

i didnt add table mapping in this please add them according to the requirement and do the necessary changes


Bumping versions

@FDelporte Can you check why it doesn't build? Maybe we can't upgrade the ant-plugin just yet?

Fixed, seems we can't bump this one:
<ant-contrib.version>20020829</ant-contrib.version>


Websocket port to Zewo 0.13.x (Swift 3.0 Release)

Summary

Upgraded Websocket, WebsocketServer, WebsocketClient to Zewo 0.13.x (Swift 3.0 Release)
added WSServer and WSClient example apps (a Websocket server and a Websocket client)
tested on macOS and Linux

Excellent! I'll review this asap 👍

I'll fix the issues I noted. Thanks for the pr :)

Current coverage is 65.35% (diff: 6.63%)

No coverage report found for websocket at 6f29ae4.

Powered by Codecov. Last update 6f29ae4...bc2292d

Thanks for the review! Once CI passes I'll merge.


Fix installer/file layout

https://jrsoftware.org/isinfo.php

https://www.youtube.com/watch?v=jPnl5-bQGHI

https://pyinstaller.org/en/stable/ ?

I am testing this. I have not had great luck so far, I'd like to get this in by v0.9.0. This will be the next big thing I work on.

I do not like this route because it is dependent on Windows OS. Windows users should have to install dependencies themselves. I can make it easy for them by documentation and the requirement.txt script.
The install will remain as it is in v0.8.5.


Feature bbox for geonames

support bbox parameter when geocoding with proximity
support bbox property when using method details

Thanks, however this is the Leaflet/Google bbox format and not the GeoJSON bbox format.
We could easily support, the GeoJSON bbox is Array of 4 numbers [west, south, east, north] => [-76.353915, 44.962733, -75.2465979, 45.5375801].
https://github.com/DenisCarriere/geocoder/blob/master/geocoder/base.py#L325

interesting. Yes that would be trivial to support both.
Before I do it, would you be ok to add a page in the doc about bbox format ?

the differences that exist (between google format and geojson to start with)
and more important: how geocoder is supposed to handle

My idea is to provide the user with the description of a consistent experience (i.e how you want geocoder to handle all this).
I will push a commit adapting to your description ;)

:) Very trivial I know... Maybe we can rename the northeast & southwest as bounds instead of bbox.
So the bounds (northeast, southwest) formats derives from Google Maps (Leaflet uses that as well).
https://developers.google.com/android/reference/com/google/android/gms/maps/model/LatLngBounds
And the GeoJSON spec uses bbox, so maybe making a difference between both.
I'll look into adding this to the docs before we move ahead.

sounds good 👍