Dependencies

https://github.com/yangclaraliu/COVID19_NPIs_vs_Rt/blob/f71a74575722678dcf4171a2b0daff31c17963a3/1_Descriptives.R#L2
Minor point. In the previous script used pacman to install packages here uses library which doesn't. Some central list of what packages are needed would likely really help reproducibility as would not depending on the tidyverse and instead calling the packages used.

4e3161f82de142880eb75bd8f744d4dfd874e6f5


Is it args for data_path(Train / Validation Set)?

Hi, First thanks for sharing your results.
I found your repo a few hours ago.
I'm trying to train my custom datasets.

How to set Train / Validation data_path?
I want to split datasets for cross-validation.

├─dataset
│  ├─train
│  │  ├─images
│  │  └─labelTxt
│  └─val
│      ├─images
│      └─labelTxt
I checked dataset_dota.py but i don't know exactly.
I just found difficulty in DOTA format but it seems not option for k-fold.
(I am not familiar with the DOTA format. My datasets label format is geojson.)
If not in code, I probably need to write code.
p.s. I recently used efficientDet, yolov5, DETR,
so I have some format_change code except for dota format.

How to use your pretrained weights?

I also checked this point,
is it '--resume' args?
(It's like resume, but I think that it seems like a purpose to learn again in the middle.)

Some wrong help in parser

I found that '--resume' and '--dataset' helps are same.
parser.add_argument('--resume', type=str, default='model_44.pth', help='weights to be resumed')
parser.add_argument('--dataset', type=str, default='dota', help='weights to be resumed')
And '--phase' is also same others.
parser.add_argument('--data_dir', type=str, default='../Datasets/dota', help='data directory')
parser.add_argument('--phase', type=str, default='test', help='data directory')
parser.add_argument('--wh_channels', type=int, default=8, help='data directory')
Not long after checking the code, I added it to the issue without making pull requests right away.
If I succeed in using the custom dataset later, I will contribute.
Thanks!

You can write a class that inherits BaseDataset in datasets/base.py, and arrange data in your way but need to provide the images and annotations, for example, write function of load_image (return 3 channels image) and load_annotation (return a python dictionary that contains keys of ['pts': [bl, tl, tr, br],'cat':category index], you can take dataset_hrsc.py as an example

--resume is for general usage, you may not need it in train.py.  It will not raise error if you don't put any resume weights

My fault. I will correct it. Thanks for your help!

Sure, thanks

Thanks.
I will try your advices!


Use logrusctx instead of logrus on task

The modifications introduced with this pull request can be used to provide detailed operation status, to avoid a a “The Cloudformation” experience where users have no insight on the operation progress.
Uses github.com/0x2b3bfa0/logrusctx, which is in turn based on github.com/juju/zaputil/zapctx, mentioned by @tasdomas in a casual conversation, unsuspecting of my implementation impetus.
Use case
logger := logrus.WithField("session", sesssionID)
logger.SetOutput(&buf)
ctx = logrusctx.WithLogger(ctx, logger)
tsk.Create(ctx)
// buf will contain the real-time progress of the operation

I have a couple of questions.

why do we need this change?
can you show or say how the behavior is being changed as a result of this?
Is this something that could be accepted upstream to lorgrus itself?
perhaps using a logging library where context is supported already?

As stated on the pull the body, to provide [service] users with detailed operation status instead of a laconic and opaque “doing stuff, will take time” message. We don't need this change, but it seems like a nice addition to me.
Behavior will stay the same after merging this pull request, but it will be really easy to alter it so logs for individual requests can be stored or sent individually to requesters.
No, logrus is in maintenance mode and does not accept new features. See the official repository for more information.
Yes, that would be nice but would require us to rewrite more code, and we'd risk breaking other unorthodox use cases of the logger like the command-line tool.

See https://github.com/iterative/terraform-provider-iterative/pull/423#pullrequestreview-901140497 for context (pun intended) on the introduction of logrus to this project.

As stated on the pull the body, to provide [service] users with detailed operation status instead of a laconic and opaque “doing stuff, will take time” message. We don't need this change, but it seems like a nice addition to me.

I'm still missing this, I did read the body. To me looks like this lets the logging write out to some buffer that we could then print out to a user via an API call (this is to help enable that correct?), but I see no changes that actually enable this, all is see is %s/ logger.Info(string)/logger.Info(ctx, string)/gc  with no other changes

Behavior will stay the same after merging this pull request, but it will be really easy to alter it so logs for individual requests can be stored or sent individually to requesters.

Should this just go with the changes which utilize the feature?[^1]

No, logrus is in maintenance mode and does not accept new features. See the official repository for more information.

More for validity Q4?

Yes, that would be nice but would require us to rewrite more code, and we'd risk breaking other unorthodox use cases of the logger like the command-line tool.

Maybe I'm missing some hacky-ness somewhere but I think that swapping out the logger should be the easiest update/rewrite/refactor that we could do for this codebase?
[^1]: see response to Q1

I think this is a step in the right direction. The internal task package is going to be used (hopefully) in several different contexts: in the CLI tool, in TPI and in the server. These all have different logging requirements. By adding context support to logrus we will be able to use different log writers in different setups.

I'm still missing this, I did read the body. To me looks like this lets the logging write out to some buffer that we could then print out to a user via an API call (this is to help enable that correct?), but I see no changes that actually enable this, all is see is %s/ logger.Info(string)/logger.Info(ctx, string)/gc with no other changes

Yes, this pull request just enables the possibility of injecting a custom logger into the context, but doesn't introduce any example.

Should this just go with the changes which utilize the feature?1

I would explicily like to separate both changes for pull requests like https://github.com/iterative/terraform-provider-iterative/pull/716 (this) and https://github.com/iterative/terraform-provider-iterative/pull/710, so they can be easily cherry-picked to the main branch and separated from commits related to the LEO server implementation.

Maybe I'm missing some hacky-ness somewhere but I think that swapping out the logger should be the easiest update/rewrite/refactor that we could do for this codebase?

Sure! What could possibly go wrong? 😅


Use PSR-1 for namespacing

As stated in PSR-1 section: Namespaces and Classes. In PHP code written in PHP 5.3 or after must use formal namespaces.
PHP-FIG: PSR-1, section: Namespaces and Classes
PSR-1 also states that code written for PHP 5.2 and below must follow the pseudo naming convention, which is now used throughout the code base:
class Google_Service_Resource

Since the codebase is written for PHP 5.4 and after, the project must follow the formal standard.
This problem is present in all branches of this repository.

Hi,
Although I understand your point, I don't see why (from a user perspective) we would have to break existing code "just" to introduces namespaces instead of pseudo naming convention such as Google_Service_Resource. That's a lot of work (in the API here, and in users scripts) for a small benefit.
And if you really want to stick to PSR-x, I think it's only recommandations ; by the way, from your excerpt :

"code written for 5.2.x and before SHOULD use..."

"SHOULD" is not "MUST" ;-)
Let's see what @bshaffer and others users thinks of this "issue".

I agree that namespaces are great and we eventually want to use them.
This is not something we have any near plans of doing, as @net-tools mentions this is not really practical and creates no real problems for our users.
Adding to v3 milestone, however, so we don't forget about it.

I understand your choice, though I would like to remind you that if you want to eventually do it it might be better to do it early so less code gets broken.

Duplicate of #692


uvc-camera for blobs-undo-bad

Commits from https://github.com/OSVR/OSVR-Core/pull/492
Not ready for merging, but this may be used as a reference.

OK #587 is this pr cleaned up.


CefDownloadHandler

suggest add CefDownloadHandler
Thanks!

Hi @Signin-or-Signup  thanks for your suggestion, but could you open an PR to contribute this? Welcome your contribution.

https://github.com/CefView/CefViewCore/commit/170a08d09668ae65e8ec0b4d34ae765fafaf5dd2
Added


Fix FxCop violations

CA1037 Use StringComparison.Ordinal for StartsWith
CA1063 Seal classes with trivial Dispose methods
CA1305 Use InvariantCulture for ToString and string.Format
CA1309 Use ordinal comparison
CA1507 Use nameof
CA1801 Review unused parameters
CA1823 Remove unused fields
CA1825 Avoid zero length array allocations
CA2007 Call ConfigureAwait

See #162
This isn't all of them, but it's a good chunk.


vscode does not run npm script names with whitespaces

VSCode Version: 1.52.1
OS Version: Windows 10 10.0.19042 build 19042

Steps to Reproduce:

Create a new node project
Add a script in the package.json with whitespaces in the key name
Try to run it from the "scripts" panel

Does this issue occur when all extensions are disabled?: Yes
I have these scripts defined  in the package.json
 "scripts": {
        //...
        "[test] unit": "jest src/.*.unit.test.tsx?",
        "[test] unit --watch": "jest src/.*.unit.test.tsx? --watch",
        "[test] integration": "jest __tests__/.*.int.test.tsx? --silent",
        "[test] integration --watch": "jest __tests__/.*.int.test.tsx? --watch --silent",
        //...
    },

If I run yarn "[test] unit --watch" or npm run "[test] unit --watch" it works
But if I click on the play button near the script name in the left panel it calls yarn run [test] unit --watch, and throws a Command "[test]" not found.
Link to the SO question
Is this a vscode bug?

@alexr00 I would like to contribute to resolving this issue.

@alexr00 I would like to contribute to resolving this issue.

@swapnilmisal thanks for the interest! However, there is already a PR that fixes this issue that is almost ready to merge: https://github.com/microsoft/vscode/pull/113840

@swapnilmisal thanks for the interest! However, there is already a PR that fixes this issue that is almost ready to merge: https://github.com/microsoft/vscode/pull/113840

@alexr00 yeah sure,  I'll find some other issue to work on.

@alexr00 yeah sure,  I'll find some other issue to work on.


Feature: add option to change grid size

Tracking bug for grid size changes:

User should be able to change photo and video sizes on the grid.

The icons are not the best. Asked a FR, maybe they add better onces: https://github.com/ionic-team/ionicons/issues/1277

I'm on the latest edge version, the option is visible in the NavBar, if I change the option the icon changes also, but there is no change in the actual grid. Is this already working or still in development?

Yes it should work. Can you reproduce on the demo page.
Please send a screenshot of the issue
-- Sorry for being brief, sent from my phone.
On Wed, 13 Sept 2023, 11:30 mrtnbr, @.***> wrote:

I'm on the latest edge version, the option is visible in the NavBar, if I
change the option the icon changes also, but there is no change in the
actual grid. Is this already working or still in development?
—
Reply to this email directly, view it on GitHub
https://github.com/bpatrik/pigallery2/issues/716#issuecomment-1717278234,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABZKA5QTGNMSERNLDDGO7CTX2F4LPANCNFSM6AAAAAA4TNZCWE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Right, figured it out. I  tried it on a page only containing Albums, but this option is for previews.


Regression in many places due to change of nxsem_timedwait_uninterruptible with nxsem_tickwait_uninterruptible

It seems that several timeout functions have broken recently due to this commit:
commit 1fb8c13e5e80ee4cd5758e76821378162451c2fa
Author: Xiang Xiao<EMAIL_ADDRESS>Date:   Tue May 10 09:32:31 2022 +0800
Replace nxsem_timedwait_uninterruptible with nxsem_tickwait_uninterruptible

Signed-off-by: Xiang Xiao<EMAIL_ADDRESS>
For example: in net/utils/net_lock.c the change looks like this:
--- a/net/utils/net_lock.c
+++ b/net/utils/net_lock.c
@@ -97,27 +97,15 @@ _net_timedwait(sem_t *sem, bool interruptible, unsigned int timeout)

   if (timeout != UINT_MAX)
     {
-      struct timespec abstime;
-
-      DEBUGVERIFY(clock_gettime(CLOCK_REALTIME, &abstime));
-
-      abstime.tv_sec  += timeout / MSEC_PER_SEC;
-      abstime.tv_nsec += timeout % MSEC_PER_SEC * NSEC_PER_MSEC;
-      if (abstime.tv_nsec >= NSEC_PER_SEC)
-        {
-          abstime.tv_sec++;
-          abstime.tv_nsec -= NSEC_PER_SEC;
-        }
-
       /* Wait until we get the lock or until the timeout expires */

       if (interruptible)
         {
-          ret = nxsem_timedwait(sem, &abstime);
+          ret = nxsem_tickwait(sem, MSEC2TICK(timeout));
         }
       else
         {
-          ret = nxsem_timedwait_uninterruptible(sem, &abstime);
+          ret = nxsem_tickwait_uninterruptible(sem, MSEC2TICK(timeout));
         }
     }

In the original code, the _net_timedwait alwas went to sleep if the timeout is > 0. In the changed code, if MSEC2TICK(timeout) rounds to 0, the nxsem_tickwait doesn't sleep at all.
I noticed this problem when i2c driver broke for stm32f7 board, and fixed only this in PR #8303
There are several alternatives to fix this problem:

In all of the places be changed into rounding up the MSEC2TICK / USEC2TICK return value, for example "MSEC2TICK(timeout + (MSEC_PER_TICK / 2)  - 1).
Pros:

Doesn't break anything unknown

Cons:

The rounding creeps in to almost everywhere where the macros are used

We could change the functionality of MSEC2TIC/USEC2TICK to round up by default (actually some OS's do this, see e..g linux jiffy calculation).
Pros:

Cleanest code, fixes all the places at once

Cons:

May affect off-tree/private code of nuttx users
The functionality of current MSEC2TIC/USEC2TICK has been as it is since 90's

We could revert the change
Pros:
-Fixes the original problem
Cons:

The original change greatly simplifies the code
Would probably lead to conflicts in revert; subject to new breaks

This problem only happen when the sleep smaller than one tick?

I think the best way is just to always just round up, not round to nearest as what is currently done.
The problem now happens always; e.g. if  tick is 10ms and user wants to sleep 19ms, the code only sleeps 10ms. Previously it did 20ms. As this is mostly used as a timeout, it is very large error.
If user wants to sleep 9ms, the code doesn't sleep at all, which breaks more or less all the i2c drivers at least.
So if we want to change the *SEC2TICK macros, it would be something like:
#define NSEC2TICK(nsec)       (((nsec)+(NSEC_PER_TICK/2))/NSEC_PER_TICK) /* Rounds */
#define USEC2TICK(usec)       (((usec)+(USEC_PER_TICK/2))/USEC_PER_TICK) /* Rounds */

#if (MSEC_PER_TICK * USEC_PER_MSEC) == USEC_PER_TICK
#  define MSEC2TICK(msec)     (((msec)+(MSEC_PER_TICK/2))/MSEC_PER_TICK) /* Rounds */
#else
#  define MSEC2TICK(msec)     USEC2TICK((msec) * USEC_PER_MSEC)          /* Rounds */
#endif

into
#define NSEC2TICK(nsec)       (((nsec)+(NSEC_PER_TICK - 1))/NSEC_PER_TICK) /* Rounds */
#define USEC2TICK(usec)       (((usec)+(USEC_PER_TICK - 1))/USEC_PER_TICK) /* Rounds */

#if (MSEC_PER_TICK * USEC_PER_MSEC) == USEC_PER_TICK
#  define MSEC2TICK(msec)     (((msec)+(MSEC_PER_TICK - 1))/MSEC_PER_TICK) /* Rounds */
#else
#  define MSEC2TICK(msec)     USEC2TICK((msec) * USEC_PER_MSEC)          /* Rounds */
#endif

But I would like to see that before decision like this is done, there are more eyes looking into it

@davids5 : Related to this, I see that PX4 changed all the boards to have 1ms systick lately in the main branch. It causes a huge CPU load overhead, and I am not able to use that in some internal configurations of ours; maybe this issue is the reason why it was done?

Your proposal look reasonable, let's wait more feedback from other.

@davids5 : Related to this, I see that PX4 changed all the boards to have 1ms systick lately in the main branch. It causes a huge CPU load overhead, and I am not able to use that in some internal configurations of ours; maybe this issue is the reason why it was done? All the i2c drivers are basically broken if you have 10ms systick atm.

We have always run at 1 mS since day one. So it is un related.

We have always run at 1 mS since day one. So it is un related.

That's right, sorry I confused something.

I decided to make a PR #8371 of the proposed change


Use an indicator to show current level in git gud levels

Right now, we explicitly state the current level, but this output isn't very clean and is somewhat verbose. It would be nice to add an indicator for the current level. This would be a part of #185, but it is already so big, it would make sense to make it a separate issue.

@sahansk2 You could probably take this on

I'll tackle this issue.

I'll go ahead and implement this:
$ git gud levels

2. Rampup (rampup)
├─◇ 1. Detaching HEAD (detaching)
├─◈ 2. Relative References I: Using (^) (relrefs1)
├─◇ 3. Relative References II: Using (~) (relrefs2)
└─◇ 4. Reversing Changes in Git (reversing)


tally command line tool to sanity check vm

compile it and run with any address:

./tally -db state.sql -acc sm1qqqqqqpy8svgfxfhh2w42ujhrynsxgwsrrzgplss9t0lv
address=sm1qqqqqqpy8svgfxfhh2w42ujhrynsxgwsrrzgplss9t0lv
layer=27658
earned=1549272967127595
rewards=0
spent=1075931911672603
tally=473341055454992
account=473341055454992

@dshulyak do ineffective transaction have a status different from types.TransactionSuccess?

all effective transactions should have a result. transaction ineffective if it is included in the block and block was applied, but has no result (e.g field in db is null)

select distinct id, tx, header, result
from transactions
left join transactions_results_addresses on id=tid
where result is not null

all effective transactions should have a result. transaction ineffective if it is included in the block and block was applied, but has no result (e.g field in db is null)

select distinct id, tx, header, result
from transactions
left join transactions_results_addresses on id=tid
where result is not null

Thanks for the explanation.

Closing as this seems abandoned. @dshulyak feel free to re-open if needed 🙂


Incorrect translation on Reddit and some websites.

When using the Lingva module, some Reddit community names are translated as "Unified" in the Firefox for Android browser.

Thanks for your report, please add some data about your environment, to help me debug the problem.

What translator you use?
Whats versions of android and firefox? What model of your device?

1- Lingva
2- Android 10 DivestOS - Firefox 113 (Mull)

@rsssxd where i can find a Lingva translator code? Did you use custom translator with this code https://github.com/translate-tools/linguist-translators/blob/c826016314f5f3f68b69d83b9e3182618b991086/translators/LingvaTranslator.js ?

:)

As workaround, to solve the problem, we can try to detect URLs and never translate strings that contains only URL.
Hovewer, this approach will not fix a bug for cases when string contains urls like "Hey Jeff, did you tried a https://google.com? I really like this site!"


Prep for localization

Moving to @Cosifne who will take this forward.

Close this because of https://github.com/dotnet/vscode-csharp/pull/5990


SNS/SQS PubSub: missing sqsDeadLettersQueueName info

What content needs to be created or modified?

Support for SQS Dead Letter Queue was introduced in dapr 1.4 but the documentation is lacking that information in terms of "Spec metadata fields" missing the new sqsDeadLettersQueueName field and information about the dependency of setting this metadata field and messageReceiveLimit.
Describe the solution you'd like

add description for the missing sqsDeadLettersQueueName metadata field spec
add note about messageReceiveLimit that becomes mandatory if sqsDeadLettersQueueName is set
add note about  messageReceiveLimit meaning in the context of specifying the sqsDeadLettersQueueName as it will be affecting the number of times a message is fetched before it is moved to the DeadLetters Queue, unlike when there is no DeadLetters Queue specified

Where should the new material be placed?

https://docs.dapr.io/reference/components-reference/supported-pubsub/setup-aws-snssqs/
The associated pull request from dapr/dapr, dapr/components-contrib, or other Dapr code repos

Additional context

@amimimor - Is this the PR for this issue https://github.com/dapr/components-contrib/pull/1066? We need to help here with the docs. Would you be able to submit a docs PR for this?


Msg Queue

What does this pull request do? Explain your changes. (required)
implements #378
Specific updates (required)

this implements 2 main methods

queue.pop() to get the next msg

queue.emit() to push a msg into the queue

@iameli @ya7ya What do you think about moving processed events to the separate 'history' table? SELECT for events scans all the existing events (it will use index, but index will be growing and growing...)?


Boxes3DVisualizer does not account for rotations when computing bounding boxes

Describe the bug
The visualizer ignores the rotations component when computing the bounding box of the batch of boxes.
https://github.com/rerun-io/rerun/blob/05853c6f408594db12af9ba668e0f8fe24ac7217/crates/viewer/re_space_view_spatial/src/visualizers/boxes3d.rs#L79-L85
To Reproduce

Open an example with rotated boxes, such as cargo run --bin snippets -- box3d_batch
Click on “Show bounding box”
Observe the boxes escaping the box.

Screenshots

Rerun version
Commit 05853c6f408594db12af9ba668e0f8fe24ac7217

Good find!
Taking this over - rotation is going to be removed as part of #6831 but we then need to make sure that the new transform / out of tree transform is handling the bbox correctly


Thank you for sharing your code

Few days ago I was trying to reverse engineer your project to see how it works :) now I can look at the code.
By the way what kind of obfuscator you used there? It didn't seem to be Agile.Net or .NET Guard, modified versions of those?

I don't remember, but I think it was a private obfuscator of a friend.


[help wanted] Is there any alternative way to check whether the session is valid or not.

I'm using the get_timeline_feed() method to check whether the session is valid. But this code takes too much time. Also, other API calls are taking too much time though I'm using a paid proxy. For example to search user by username took almost 30-35 seconds.

getting timeline feed will take less than 5 sec to execute ,
so if it take 30 sec for u to run , i guess it relate to your proxy

@SaeidB can you suggest me some good proxies, please?

@SaeidB can you suggest me some good proxies, please?

Most proxies will work without problem, just change your provider or send them ticket and ask for help.


Update GitHub action

Q
A

Is bugfix?
✔

New feature?
❌

Breaks BC?
❌

Tests pass?
?

Fixed issues
#17865

Merged current things fixed. Thank you very much for it.


Question about the normalization in attention

Dear authors,
Nice work! I have a few questions regarding the normalization in the implementation of the RetNet and would like to consult your ideas about them:
Here,
qk_mat = qk_mat / qk_mat.detach().sum(dim=-1, keepdim=True).abs().clamp(min=1)

why there is a normalization for attention? Is it mentioned in the paper anywhere? Why do you choose abs()?
Here,
value_inner_decay = mask[:, -1] / mask[:, -1].sum(dim=-1, keepdim=True)

for the decay term, why do you normalize the mask here? Isn't the unnormalized mask correct?
Here,
scale = mask.sum(dim=-1, keepdim=True).sqrt()

what is the role of this scale variable and why is it divided both in the inner_mask and query?
Thank you very much in advance!

It's a little bit complicated. The key idea is to align the normalized computation to be identical to parallel representation. Due to fp16 data range limitation, in chunkwise forward, we have to normalize data range in different part. So, after these modification, the output will be same as totally naive version.
We mention this problem in Retention Score Normalization of our paper.

Thanks a lot!


add client packages

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 65.62%. Comparing base (d6cdaf3) to head (4c49d30).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

:exclamation:  There is a different number of reports uploaded between BASE (d6cdaf3) and HEAD (4c49d30). Click for more details.
HEAD has 3 uploads less than BASE

Flag
BASE (d6cdaf3)
HEAD (4c49d30)

4
1

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #382       +/-   ##
===========================================
- Coverage   77.78%   65.62%   -12.17%
===========================================
  Files          37       37
  Lines        2804     2804
===========================================
- Hits         2181     1840      -341
- Misses        623      964      +341

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


GSF IEF Meeting Agenda 2024-01-18

Time 10:30am (GMT) / 4:00pm (IST) - See the time in your timezone

Lead – @srini1978 (Microsoft)
Co-Lead @navveenb (Accenture)
Co-Lead – @jawache (GSF)
PM - @jmcook1186 (GSF)

Antitrust Policy
Joint Development Foundation meetings may involve participation by industry competitors, and it is the intention of the Joint Development Foundation to conduct all of its activities in accordance with applicable antitrust and competition laws. It is therefore extremely important that attendees adhere to meeting agendas, and be aware of, and not participate in, any activities that are prohibited under applicable US state, federal or foreign antitrust and competition laws.
If you have questions about these matters, please contact your company counsel or counsel to the Joint Development Foundation, DLA Piper.
Roll Call
Please add 'Attended' to this issue during the meeting to denote attendance.
Any untracked attendees will be added by the GSF team below:
Agenda

[ ] Mock backend demo
[ ] Sprint progress report
[ ] Discussion about units/param handling
[ ] Discussion on exceptions and error handling
[ ] AOB

Attended

Attended

Attended

Attended

Attended

Attended

Attended

Attended


There's no need to check if the production isn't null a second time in the parse() method

Yes there is, because there may not be any productions.


Put exlusive lock on application lock file

I don't see a point in adding a library that does what the other 2 lines of code already did.

So after talking with michael bishop, what I understand is that to implement this, I need to use filelock library which uses FileLockEx on Windows and that will safely put a exclusive lock on a lock file in Windows.


Hermite interpolator may cause ringing

Hi there,
I'm quite likely to use your library on a current project. However, I noticed that your implementation of the Hermite interpolation uses the typical approach found in literature. This approach causes ringing for orbit data, where the interpolation is visibly poor. A good test is to implement this test: https://github.com/nyx-space/anise/blob/master/anise/src/math/interpolation/hermite.rs#L218.
The solution is to use the exact same algorithm as in SPICE. Here is the re-implementation of that algorithm in Rust, as part of ANISE: https://github.com/bradsease/oem/blob/fd85a67a96cd03ec38ab837a909b996fb96c6a79/oem/interp.py#L23 . For the details on the tests, you can look at this huge PR, or specifically this commit.
You could also simplify the Lagrange interpolation by using directly the relatively simple SPICE algorithm: https://github.com/nyx-space/anise/blob/master/anise/src/math/interpolation/lagrange.rs#L15 . I would expect this function to be faster than the numpy implementation, but I'm not sure since numpy is pretty well optimized. From my literature review a couple of years ago, I believe that the Lagrange interpolation commonly found matches the SPICE version, so I don't expect there to be any issue in the Lagrange interpolator. My OEMs use Lagrange so this ticket doesn't preclude me from using your library.
Cheers

Here's a write-up of the algorithm if that's of interest: https://nyxspace.com/anise/reference/mathspec/interpolation/hermite/ .

Thanks for the feedback! I'll try to get this into the next version


🚀 RELEASE: v0.0.37

Just release a v0.1 already 😂

haha yeah I know I know...I'll create a milestone for it as a part of our conversations in https://github.com/executablebooks/meta/issues/149


⚠ AZ-Launcher has degraded performance

In 2a10368, AZ-Launcher (https://az-launcher.nz/) experienced degraded performance:

HTTP code: 200
Response time: 1316 ms

Resolved: AZ-Launcher performance has improved in 3b53de8 after 11 minutes.


Observable demo does not work

See https://observablehq.com/@domoritz/vega-with-omnisci-core-transform
@jheer any idea why the transform is not being called? https://vega.github.io/scalable-vega/ works fine.

Ugh. The breakdown is due to redundant imports of Vega. It looks like we're getting two different versions: one loaded by the OmniSci plugin and the other by embed.
To make the queries run, I can hack the prototype chain like so: QueryCore.prototype.__proto__ = embed.vega.transforms.collect.prototype.__proto__;
However, downstream rendering is still buggy because the tuples ingested by the query transform use a different Symbol for the tuple id. If I set an explicit data join key on the rect mark (e.g., `"key": "bin_start") then the join skips the symbol-keyed id and rendering is fixed.
We need a way to ensure that different Vega versions are not loaded side-by-side. There are parts of the system that could be made more robust to prototype chain issues, but that still wouldn't solve the issue of having different Symbol instances used to set/get Vega tuple ids.
I don't have good insight into Observable's require implementation to diagnose why it is not caching / reusing the require('vega') calls that both libraries make. Maybe @mbostock has an idea?

This is arguably a caching bug on jsDelivr’s side. These two URLs resolve differently:
<EMAIL_ADDRESS>(resolves to 5.3.1)
https://cdn.jsdelivr.net/npm/vega/package.json (resolves to 5.3.2)
The first one comes from vega-embed’s peer dependency on vega@*. The second one comes from vega-transform-omnisci-core, which does not declare a vega dependency in its package.json, but since it does require vega at runtime, it resolves to vega with no version range, which is equivalent to vega@latest (and slightly different than vega@*).
So, the first thing I would do is to declare the dependencies consistently, say by adding vega@* as a peer dependency here.
The second thing is that you can purge the jsDelivr cache after releasing new software by replacing “cdn” in the link above with “purge”. This way, even though they are loaded with different URLs, they will resolve to the same value and not be loaded twice.
And lastly, you can again workaround these issues using require.alias to force them to resolve to the same value:
require.alias({vega: 'vega<EMAIL_ADDRESS>
Or better, to pin major versions so that it won’t break when a new release is available:
https://observablehq.com/compare/1c142edfbb839048...cc4f77bed1274eee

Thanks @mbostock! Both the purge and require.alias tricks were new to me!

FYI, I purge'd vega@* and the notebook is now working. But we should still declare our dependencies in a systematic fashion.

Thank you for finding the issue. I declared the peer dependency and so I think it should work better now.


When I have a draft with an author, then I should be able to clear out their name and save it as a draft

Currently, I can delete the only author name and save the draft, but that information is still saved and is visible on the item view page and is present in the form when I reload it make further edits.


Can't add multiple classes

Hello,
Thank you for this great fork.
If I instanciate toastr like this :
const toastr = new Toastr({
    toastClass: 'class1 class2',
})

I get this error :
Uncaught DOMException: Failed to execute 'add' on 'DOMTokenList': The token provided ('class1 class2') contains HTML space characters, which are not valid in tokens.

(it works with the jquery version since $.addClass() accepts space separated classes).
You could do :
$toastElement.classList.add(...options.toastClass.split(' '), iconClass);
Could be nice to have this behaviour for position and icon classes too.
Best,

Thanks for your issue. I will try to fix this asap :)

@sydney-d should be fixed in v3.0.0-alpha.9


Improve creative resizing logic

When using native-render.js, after the ad is rendered, we take the ad height and send it to Prebid for it to be set on the frame's container:
https://github.com/prebid/prebid-universal-creative/blob/0d0ef0466cda24a6db14f18255da7c2d77b5c22e/src/nativeAssetManager.js#L381
This is because GPT will fix the safeframe's height based on the contents it knows about, and it does not know that we are dynamically generating it. However, this approach has problems (see https://github.com/prebid/prebid-universal-creative/pull/190#issuecomment-1308640825):

at the time we requestHeightResize, we do not necessarily know the true height of the creative, because images and scripts may still need to be loaded. We should try to wait until page load is complete before requesting the resize.
it is possible to tell GPT to allow height: auto safeframes (allowPushExpansion), in which case we should not fix the height ourselves.

allowPushExpansion support still needed


[86bxyvukh][utils] fixed focusLock hook for multiple portals

Motivation and Context
I've added capture: true for focusLock event listeners for fix their work in multiple portals.

How has this been tested?
I've tested it manually

Screenshots (if appropriate):
Types of changes

[X] Bug fix (non-breaking change which fixes an issue).
[ ] New feature (non-breaking change which adds functionality).
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected).
[ ] Nice improve.

Checklist:

[X] My code follows the code style of this project.
[X] I have updated the documentation accordingly or it's not required.
[X] Unit tests are not broken.
[X] I have added changelog note to corresponding CHANGELOG.md file with planned publish date.
[ ] I have added new unit tests on added of fixed functionality.

Closing in favor of https://github.com/semrush/intergalactic/pull/1218


It should be possible to reset password using phone number

In the forgot password flow, allow using the phone number to identify the user. Currently only email or username are allowed.

Closing since a duplicate of https://github.com/ekstep/Common-Design/issues/499


Is it possible to load multiple configuration files?

I wonder if it's possible to load multiple configuration files e.g i want to keep interface processors and bgp processors in a separate files?

I'm afraid that is not supported


Call StateHasChanged on global config options updates

At least for the properties:

NewestOnTop
PreventDuplicates
MaxDisplayedToasts
PositionClass

Done in version 0.3.0


Ubuntu 22.04 Serve Install issue

Environment details:

Ubuntu Server 22.04

Steps to reproduce the issue:

N/A

Provide the actual results and outputs:
vboxuser@ubuntu:/opt/openwhisk/tools/ubuntu-setup$ sudo ./all.sh
+ JAVA_SOURCE=open
+ SOURCE=./all.sh
++ dirname ./all.sh
+ SCRIPTDIR=.
+ echo '*** installing basics'
*** installing basics
+ /bin/bash ./misc.sh
+ export DEBIAN_FRONTEND=noninteractive
+ DEBIAN_FRONTEND=noninteractive
+ sudo apt-get update -y
Hit:1 http://ca.archive.ubuntu.com/ubuntu jammy InRelease
Hit:2 http://ca.archive.ubuntu.com/ubuntu jammy-updates InRelease
Hit:3 http://ca.archive.ubuntu.com/ubuntu jammy-backports InRelease
Hit:4 http://security.ubuntu.com/ubuntu jammy-security InRelease
Hit:5 https://download.docker.com/linux/ubuntu jammy InRelease
Reading package lists... Done
+ sudo apt-get install -y ntp git zip unzip tzdata lsb-release npm
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
lsb-release is already the newest version (11.1.0ubuntu4).
zip is already the newest version (3.0-12build2).
npm is already the newest version (8.5.1~ds-1).
ntp is already the newest version (1:4.2.8p15+dfsg-1ubuntu2).
git is already the newest version (1:2.34.1-1ubuntu1.10).
tzdata is already the newest version (2024a-0ubuntu0.22.04).
unzip is already the newest version (6.0-26ubuntu3.2).
The following packages were automatically installed and are no longer required:
  python-pkg-resources python-setuptools
Use 'sudo apt autoremove' to remove them.
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.
+ sudo tee /etc/timezone
+ echo Etc/UTC
Etc/UTC
+ sudo dpkg-reconfigure --frontend noninteractive tzdata

Current default time zone: 'Etc/UTC'
Local time is now:      Tue Mar  5 00:58:34 UTC 2024.
Universal Time is now:  Tue Mar  5 00:58:34 UTC 2024.

+ sudo service ntp restart
+ sudo ntpq -c lpeer
     remote           refid      st t when poll reach   delay   offset  jitter
==============================================================================
 0.ubuntu.pool.n .POOL.          16 p    -   64    0    0.000   +0.000   0.000
 1.ubuntu.pool.n .POOL.          16 p    -   64    0    0.000   +0.000   0.000
 2.ubuntu.pool.n .POOL.          16 p    -   64    0    0.000   +0.000   0.000
 3.ubuntu.pool.n .POOL.          16 p    -   64    0    0.000   +0.000   0.000
 ntp.ubuntu.com  .POOL.          16 p    -   64    0    0.000   +0.000   0.000
+ echo '*** installing python dependences'
*** installing python dependences
+ /bin/bash ./pip.sh
+ sudo apt-get install -y python-pip
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
The following packages were automatically installed and are no longer required:
  libjs-jquery libjs-sphinxdoc libjs-underscore libpython3-dev libpython3.10-dev python3-dev python3-wheel python3.10-dev zlib1g-dev
Use 'sudo apt autoremove' to remove them.
Recommended packages:
  python2-dev
The following packages will be REMOVED:
  python3-pip
The following NEW packages will be installed:
  python-pip
0 upgraded, 1 newly installed, 1 to remove and 0 not upgraded.
Need to get 0 B/1,021 kB of archives.
After this operation, 2,153 kB disk space will be freed.
(Reading database ... 107521 files and directories currently installed.)
Removing python3-pip (22.0.2+dfsg-1ubuntu0.4) ...
Selecting previously unselected package python-pip.
(Reading database ... 106837 files and directories currently installed.)
Preparing to unpack .../python-pip_20.3.4+dfsg-4_all.deb ...
Unpacking python-pip (20.3.4+dfsg-4) ...
Setting up python-pip (20.3.4+dfsg-4) ...
Processing triggers for man-db (2.10.2-1) ...
Scanning processes...
Scanning linux images...

Running kernel seems to be up-to-date.

No services need to be restarted.

No containers need to be restarted.

No user sessions are running outdated binaries.

No VM guests are running outdated hypervisor (qemu) binaries on this host.
+ sudo pip install argcomplete
sudo: pip: command not found

Additional information you deem important:
*** pip command will obviously will not found because pip3 is working

And when when I removed pip3 and install python-pip still it says pip command not found**

vboxuser@ubuntu:/opt/openwhisk/tools/ubuntu-setup$ sudo apt install python-pip
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
python-pip is already the newest version (20.3.4+dfsg-4).
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.
vboxuser@ubuntu:/opt/openwhisk/tools/ubuntu-setup$ sudo ./all.sh
+ JAVA_SOURCE=open
+ SOURCE=./all.sh
++ dirname ./all.sh
+ SCRIPTDIR=.
+ echo '*** installing basics'
*** installing basics
+ /bin/bash ./misc.sh
+ export DEBIAN_FRONTEND=noninteractive
+ DEBIAN_FRONTEND=noninteractive
+ sudo apt-get update -y
Hit:1 http://ca.archive.ubuntu.com/ubuntu jammy InRelease
Hit:2 http://security.ubuntu.com/ubuntu jammy-security InRelease
Hit:3 http://ca.archive.ubuntu.com/ubuntu jammy-updates InRelease
Hit:4 http://ca.archive.ubuntu.com/ubuntu jammy-backports InRelease
Hit:5 https://download.docker.com/linux/ubuntu jammy InRelease
Reading package lists... Done
+ sudo apt-get install -y ntp git zip unzip tzdata lsb-release npm
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
lsb-release is already the newest version (11.1.0ubuntu4).
zip is already the newest version (3.0-12build2).
npm is already the newest version (8.5.1~ds-1).
ntp is already the newest version (1:4.2.8p15+dfsg-1ubuntu2).
git is already the newest version (1:2.34.1-1ubuntu1.10).
tzdata is already the newest version (2024a-0ubuntu0.22.04).
unzip is already the newest version (6.0-26ubuntu3.2).
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.
+ sudo tee /etc/timezone
+ echo Etc/UTC
Etc/UTC
+ sudo dpkg-reconfigure --frontend noninteractive tzdata

Current default time zone: 'Etc/UTC'
Local time is now:      Tue Mar  5 01:10:25 UTC 2024.
Universal Time is now:  Tue Mar  5 01:10:25 UTC 2024.

+ sudo service ntp restart
+ sudo ntpq -c lpeer
     remote           refid      st t when poll reach   delay   offset  jitter
==============================================================================
 0.ubuntu.pool.n .POOL.          16 p    -   64    0    0.000   +0.000   0.000
 1.ubuntu.pool.n .POOL.          16 p    -   64    0    0.000   +0.000   0.000
 2.ubuntu.pool.n .POOL.          16 p    -   64    0    0.000   +0.000   0.000
 3.ubuntu.pool.n .POOL.          16 p    -   64    0    0.000   +0.000   0.000
 ntp.ubuntu.com  .POOL.          16 p    -   64    0    0.000   +0.000   0.000
+ echo '*** installing python dependences'
*** installing python dependences
+ /bin/bash ./pip.sh
+ sudo apt-get install -y python-pip
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
python-pip is already the newest version (20.3.4+dfsg-4).
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.
+ sudo pip install argcomplete
sudo: pip: command not found
vboxuser@ubuntu:/opt/openwhisk/tools/ubuntu-setup$

I think using K8s deployment is a better way.


Incorrect loading order of overrides.

Im not sure why its happening but my .vimrc.local and .vimrc.bundles.fork seem to be loading before the bundles themselves.
http://www.pastebin.ca/2329442
I am including a few new Bundles in .vimrc.bundles.local:
Bundle 'joonty/vdebug.git'
And i need to override an option or two:
let g:vdebug_options['break_on_open'] = 0
let g:vdebug_features['max_children'] = 128 " this is only available in dev branch
My understanding was that i could do this in either .vimrc.local (which should be the last file parsed) or .vimrc.bundles.fork which is where you seemingly indicate it should be done most cleanly (makes sense).
Both fail with the following message onload:
Error detected while processing /Users/michaelfavia/.vimrc.bundles.fork:
line    4:
E121: Undefined variable: g:vdebug_options
line    5:
E121: Undefined variable: g:vdebug_features
line    8:
Seemingly as a result of the dictionary not being defined yet (e.g. the bundle/plugin not laoded yet). My pastebin from :scriptnames above seems to confirm this.
Am I doing this wrong or is this some sort of odd bug? Im happy to chase it down anyway you need. Thanks, -mf

closing for lack of response. reopen if you can you can reproduce with latest spf13-vim


Expose error str in Error struct

My crate compiledfiles takes a dependency on the ability to inspect which error occurred in order to determine what to do new.
https://github.com/schultetwin1/compiledfiles/blob/0452bc2756ca01f31e4df06bbf38540ced37c1fb/src/lib.rs#L216-L222
Would you be open to making the internal &str in the Error struct public?
I'm new to Rust, so if this is not the proper way to do error handling let me know.

You definitely shouldn't be matching on the string. There is no guarantee that it will remain the same, even if we exposed it.
The current API of this crate doesn't allow you to distinguish between error reasons. A common way to distinguish between errors in Rust is to use an enum for the errors, but I've been reluctant to do that because the API that exposes is that any function can return any error, which is too broad and bloated. We could maybe add an error enum just for File::parse though.
Another option would be to split File::parse into two functions: one to detect the file format, and one to parse as a specific file format.
Also, reconsider whether distinguishing between these errors is required. File format detection can have both false positives and false negatives, so maybe you want to try all possible parsers regardless of the error.

Thanks for all the great advice! Not matching on strings makes sense.

Also, reconsider whether distinguishing between these errors is required. File format detection can have false positives, so maybe you want to try all possible parsers regardless of the error.

I think I'll take this approach for now. Closing this issue.


routes promo component

Trello card
https://trello.com/c/PBVFgRtv/7046-design-and-build-a-find-your-route-promo
Context
Changes proposed in this pull request
Guidance to review

This looks great 👍
Don't forget to add some documentation to docs/content.md. In the future we could consider how to generalise the Adviser Promo CTA with the Routes CTA - but lets leave that for now.


Feature: set excel BIG HEADER

As an excel user, I want to set BIG HEADER(Or First Header), so that other people can notice  the core information。

Accept criteria:

can set font style
can set the amount of cells to merge

done


refactor: move traditional CRDs back to original group name so we dont need to upgrade CRDs, they havne't changed so can keep the same api version number

/approve


[11.0] ISPN-12397 Upgrade to WildFly OpenSSL 1.0.11.Final

https://issues.redhat.com/browse/ISPN-12397

integrated! thanks @tristantarrant !


feat: improve UX when loading images

Currently when adding images in the conversation, if the image is big and is not that clear, clicking in the image will attempt to download the image.
This PR attempts to improve the UX a bit by just opening the image in a new window (aka popup) to see the image larger. It has a download option on this new popup as well to preserve the original download behavior, but now the UX is a bit better

thanks @dokterbob for helping review and approve this and my last change as well 👍


BUG: get_structure_parent doesn work for Mouse Atlas

Using get_structure_parent with ABA25Um fails because:

E           AttributeError: 'ABA25Um' object has no attribute 'structures_ids'

I'll try to work on brainatlas tonight, updating the human atlas and testing stuff out, but stuff like this error should have been caught by the testing..

Similarly:
E       AttributeError: 'ABA25Um' object has no attribute 'structures_acronyms'


Implement MyStudentChart

Time category: https://its-pro.ucsd.edu/browse/MA-79
This is done when:

[x] MyStudentChart links to WebView / responsive site

Deployed in Build 51 (v7.0)


ComfyUI API By URL uses wrong model name

If you want to work with a remote ComfyUI the name of the locally installed model is used instead of the remote model name.
I installed StableSwarmUI on my Mac and want to access a remote ComfyUI on my Linux box using ComfyUI API By URL. As soon as I want to generate an image ComfyUI on my Linux box is executed but I get the following error:
Failed to validate prompt for output 9:
* CheckpointLoaderSimple 4:
  - Value not in list: ckpt_name: 'OfficialStableDiffusion/sd_xl_base_1.0.safetensors' not in ['sd_xl_base_1.0.safetensors', 'sd_xl_refiner_1.0.safetensors']
Output will be ignored
invalid prompt: {'type': 'prompt_outputs_failed_validation', 'message': 'Prompt outputs failed validation', 'details': '', 'extra_info': {}}

So it uses the local path to the model from StableSwarmUI instead of the models installed on the remote machine.

Correct, it assumes the path is the same on all backends. I'm keeping the feature request of expanding it to support remote models open as #36


fix(@ngtools/webpack): give higher priority to ivy-specific entry-points

When Ivy is enabled in an app, the app's dependencies are processed by ngcc (the Angular compatibility compiler), which will generate new Ivy-compatible entry-points in each package. To allow webpack find those entry-points, it will add references to them in package.json, named after the original property with the _ivy_ngcc suffix. So, for example, es2015 and module will become es2015_ivy_ngcc and module_ivy_ngcc.
Previously, the mainFields passed to webpack would give higher priority to an Ivy entry-point compared to the corresponding non-ivy entry-point but not compared to other entry-points. For example, es2015, module would becode es2015_ivy_ngcc, es2015, module_ivy_ngcc, module. This could mean that es2015 would be preferred over module_ivy_ngcc, even though the former is probably not Ivy-ready. Generally, if es2015 exists and has a higher priority than module, then es2015_ivy_ngcc would be generated before (or instead of) module_ivy_ngcc, but this can be changed using an ngcc configuration file (ngcc.config.js).
This commit fixes this by ensuring that any _ivy_ngcc entry-point is considered before any of the original entry-points in the mainFields list.
NOTE:
While it is possible that a format is Ivy-compatible without bhaving an _ivy_ngcc suffix (e.g. if the user does a manual ngcc pass without --create-ivy-entry-points), it is quite unlikely to happen and even if it does, choosing a different format should be OK.

Why question would be why would module be converted with NGCC and es2015 wouldn't?

if es2015 exists and has a higher priority than module, then es2015_ivy_ngcc would be generated before (or instead of) module_ivy_ngcc

I am not following why es2015 should be generated instead of module. They are 2 different formats and used differently in the CLI based on which compilation target the user has set and are not always interchangeable.

es2015 and module were just an example. Typically, CLI uses compileAllFormats: false, which means that it will pass ngcc a list of format properties in order of preference and ngcc will process the first that is finds for each package.
So, if CLI passes es2015, browser, module, main ngcc will process the first that it finds. This means that if a package has all format properties in its package.json.

However, it is possible that an app developer or library author includes an ngcc.config.js file that instructs ngcc to ignore a format property. For example, say that a library's es2015 format is in some way incompatible with ngcc and trying to process it will result in a build error. The developer can work-around that by instructing ngcc to ignore the es2015 format property, so then ngcc will move to the second-best property (e.g. browser).
In that case, we want webpack to also pick up browser_ivy_ngcc (which is Ivy-ready) and not the (non-Ivy-ready) es2015.

BTW, this usecase is more likely to come up with the main property, where the formats can vary a lot. So, it happens that the main property of some (often old and unmaintained but still used) libraries cannot be processed by ngcc (see for example angular/angular#36558). This is problematic when building for SSR, since main is the first preference.
One way to fix this would be for the app developer to configure ngcc to ignore the main property and thus fal back to the second-best: module
In that case, we want webpack to pick module_ivy_ngcc and not the (unprocessed) main property.
Does that make sense?

Make sense.

BTW, this usecase is more likely to come up with the main property, where the formats can vary a lot. So, it happens that the main property of some (often old and unmaintained but still used) libraries cannot be processed by ngcc (see for example angular/angular#36558). This is problematic when building for SSR, since main is the first preference.

In most cases at least since version 8, the SSR's preferred entrypoint is es2015 and not main.

In most cases at least since version 8, the SSR's preferred entrypoint is es2015 and not main.

Oh, I didn't know that. That's great!
(main can still be an issue with old, unmaintained libaries, since many of them do not have es2015 bundles.)

Also merged to 9.1.x patch branch.


Update scalafix-rules to 0.9.19

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

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "ch.epfl.scala", artifactId = "scalafix-rules" } ]

labels: library-update, semver-patch

Duplicate of #55


The project needs a better readme

The project readme is currently a little lacking, and should probably be expanded or partially rewritten
The following is a list of things that would be good to have in the readme, feel free to expand, remove, or comment below:

[ ] Installation instructions
[ ] Screenshots
[ ] Before and after examples
[ ] Badges, including version information and further links (such as links to the betterdiscord page for this style or links to another repo such as this one which reverts even further)

Mostly lacking currently because README.md is automatically generated by GooseThemeBundle - only providing basics inside of it atm.

Mostly done.


Add ConsoleZ to terminal emulation.

This might be considered a duplicate of ConEmu since both projects seem to share a lot of features, but none the less I think it's a great mention.

 I'll fix that, sorry.


Error no end time for subtitle

vobsub is now supported (read-only) by subparse in the format-vobsub branch.
I tried to test it on some files from the internet, but I'm only getting no end time for subtitle errors. Are there .sub files that have no timing information, or do I use this crate wrong? I attached a file that shows this behavior (sub.zip).

Thank you for the sample file! I'll check it out.
Theoretically, subtitles are supposed to include a start time and an end time. But it wouldn't surprise me if there were some pretty weird subtitles out there. I'll take apart the file and have a look.

OK, I'm looking at this file this morning. I'll let you know what I find!

OK, I've tracked down the problem. It looks like this file really doesn't have end times for subtitles. This is going to require some API changes and some internal trickery to use the start time of the next subtitle as a default whenever possible. I'll try to get something out shortly.

OK, that was a rather weird subtitle file. :-) It was poorly compressed with large blocks of unnecessary 00 bytes, and it was missing the end_time fields. I came up with a design that should work:

If the subtitle missing an end_time is before another subtitle, keep it on screen until just before the next subtitle is shown. We might also arguably want to add a maximum duration here. What do you think?
If the subtitle missing an end_time is the last subtitle, just arbitrarily assume that it should be 3 seconds long.

Theoretically, I could also leave end_time blank, but that would just be obnoxious for users of the library, especially since this is a rare case.
If you have a chance, please feel free to test this change locally on more of your files using a vobsub = { git = "https://github.com/emk/subtitles-rs" version = "*" } in your Cargo.toml, and let me know if it handles all the files you can find.

If the subtitle missing an end_time is before another subtitle, keep it on screen until just before the next subtitle is shown. We might also arguably want to add a maximum duration here. What do you think?

mpv handles this by showing every subtitle until the next begins, except the last subtitle which is shown for exactly 1 minute. I think exposing a variable to the library user is unneeded complexity, just taking an arbitrary length should be okay.
Another 2 bugs(?) have appeared, which might have to do with the current issue. Please feel free to redirect me to a new issue if I'm wrong.
The problem is this code, which takes about 10 seconds with 100% cpu with the provided subtitle (is this a problem with the format?):
let lines = vobsub::subtitles(b)
            .collect::<vobsub::Result<Vec<VobSubSubtitle>>>()
            .map_err(|e| Error::from(e))?;

The other problem is that many lines don't take the end time from the next subtitle but have the end time "5". This happens with almost all subtitles in the beginning of the provided subtitle file.

The problem is this code, which takes about 10 seconds with 100% cpu with the provided subtitle (is this a problem with the format?):

Make sure you compile in --release mode when benchmarking Rust programs. If you compile in debug mode, then iterators and some of low-level byte-parsing code will be poorly optimized. Also keep in mind that if you're processing a movie, you need to decompress about 1,000 compressed images.
Also, I don't recommend trying to collect vobsub::Subtitle objects in a Vec<vobsub::Subtitle>. Again, this may involve loading 1,000 images into memory all at once. Sometimes, these images are relatively small, but in some cases, they may be full-screen images. So instead, it's better to iterate over the subtitles and to extract any information you need.
Decoding the images is a little tricky; you need to call to_image and pass it the palette you got from the idx file (or IFO file, or container metadata, if you're trying to parse a closely related format). This will give back an image::Image object, which can then be saved to disk in a variety of formats, or otherwise used.

The other problem is that many lines don't take the start time from the next subtitle but have the end time "5".

 This should now be fixed in vobsub 0.2.1. Thank you for finding this bug!

Make sure you compile in --release mode when benchmarking Rust programs. If you compile in debug mode, then iterators and some of low-level byte-parsing code will be poorly optimized. Also keep in mind that if you're processing a movie, you need to decompress about 1,000 compressed images.

Yes, that was it. I'm (again) surprised by the large difference between debug and release...

Also, I don't recommend trying to collect vobsub::Subtitle objects in a Vecvobsub::Subtitle. Again, this may involve loading 1,000 images into memory all at once. Sometimes, these images are relatively small, but in some cases, they may be full-screen images. So instead, it's better to iterate over the subtitles and to extract any information you need.

Fixed that, thank you for this information!

Decoding the images is a little tricky; you need to call to_image and pass it the palette you got from the idx file (or IFO file, or container metadata, if you're trying to parse a closely related format). This will give back an image::Image object, which can then be saved to disk in a variety of formats, or otherwise used.

Currently I'm not interested in extracting the image data (I only want a common interface for timestamps, lines, and eventually styling information; but I'm contemplating over single-format specific "extensions"). Nonetheless I will keep that in mind for when I need it.

 This should now be fixed in vobsub 0.2.1. Thank you for finding this bug!

Works like a charm! That resolved the last issue before putting my code on master! The subparse on master now supports 5 different formats (.sub for MicroDVD and VobSub are new).


Bushey Meads Secondary School, UK

Add Bushey Meads Secondary School which is based in Bushey, Hertfordshire
https://busheymeads.org.uk/

@aadilchoudhury First stage of verification passed, official domain is busheymeads.org.uk as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.


feat: Add implementation for Sponsor deletion #939

Fixes #939
Changes:
Adds a delete icon to sponsor item, which deletes the Sponsor item from the server and the local database upon confirmation
Screenshots for the change:

@iamareebjamal Please review.

I really like the UI of the sponsor component which you have made. This is a lot better than other components in the app. I like the placement of the contextual buttons as well, but a little concerned about the overflow. I mean, the name of the sponsor could be huge and that won't look nice. Also, this doesn't allow the ability to delete multiple sponsors at once. So I think contextual menu on long click would be more practical.

No do both of them in different PRs


Suggestion for FAQ: note on performance

I was testing out CNL in a bit of code I already had written using raw ints, and I was shocked to see my execution time go up by about 100x after writing the same code with CNL. This was with LLVM 12 on MacOS.
Eventually I realized I was in the debug build, and turning on optimizations gave equal performance. Would it make sense to include a note in the FAQ that compiling without optimizations enabled may cause significant performance penalties?

Hi. It's increasingly common for C++ libraries to require inlining in order to perform well. You could argue that this is the domain of modern C++ user -- rather than CNL user in particular. That said, it's an especially acute problem for a library like this.
Can I ask what docs your read first. Did you read the README.md? Did you read the front page of the documentation? In particular, did you look through the FAQ first for advice? If I was going to mention performance, it would help to know where users would find it.
Thanks,
John


Compatibility with notebook

This PR enable jupyterlab_chat in Notebook.
Fixes #105

[ ] Add integration test to ensure the chat is opening in side panel and as single document widget

Bump CI

Bump CI again


Visual Studio Online refresh

Update references to Visual Studio Online to Azure Repos, Azure Test Plans, Azure DevOps, etc.

@erickson-doug FYI found some pretty outdated VSO references in the asp.net docs. I didn't find any VSTS references though.

Thank you, @gewarren!


Add data-uri support

Add support for the following syntax:
<isvg src="data:image/svg......." />

Although xmlhttprequest in Google Chrome & Firefox supports data-uri, the one in Safari doesn't. This pull request skips the ajax part if an SVG data-uri is given in the src property.

Awesome, thanks! I've merged in your first two commits and added some code to support non-base64 data-uris too.

:+1: Thanks


Missing dependencies for analysis

When running the final cell of the roms_marbl example notebook:
BGC_DS=xr.open_dataset(roms_marbl_case.caseroot+'/output/MARBL_bgc.20120103120000.nc')

---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
Cell In[29], line 2
      1 ######################################################
----> 2 BGC_DS=xr.open_dataset(roms_marbl_case.caseroot+'[/output/MARBL_bgc.20120103120000.nc](http://localhost:8888/output/MARBL_bgc.20120103120000.nc)')
      3 var='DOC'
      4 BGC_DA=BGC_DS[var].isel(s_rho=99)

File [~/miniconda3/envs/cstar_environment/lib/python3.12/site-packages/xarray/backends/api.py:569](http://localhost:8888/lab/tree/examples/~/miniconda3/envs/cstar_environment/lib/python3.12/site-packages/xarray/backends/api.py#line=568), in open_dataset(filename_or_obj, engine, chunks, cache, decode_cf, mask_and_scale, decode_times, decode_timedelta, use_cftime, concat_characters, decode_coords, drop_variables, inline_array, chunked_array_type, from_array_kwargs, backend_kwargs, **kwargs)
    566     kwargs.update(backend_kwargs)
    568 if engine is None:
--> 569     engine = plugins.guess_engine(filename_or_obj)
    571 if from_array_kwargs is None:
    572     from_array_kwargs = {}

File [~/miniconda3/envs/cstar_environment/lib/python3.12/site-packages/xarray/backends/plugins.py:197](http://localhost:8888/lab/tree/examples/~/miniconda3/envs/cstar_environment/lib/python3.12/site-packages/xarray/backends/plugins.py#line=196), in guess_engine(store_spec)
    189 else:
    190     error_msg = (
    191         "found the following matches with the input file in xarray's IO "
    192         f"backends: {compatible_engines}. But their dependencies may not be installed, see:\n"
    193         "https://docs.xarray.dev/en/stable/user-guide/io.html \n"
    194         "https://docs.xarray.dev/en/stable/getting-started-guide/installing.html"
    195     )
--> 197 raise ValueError(error_msg)

ValueError: found the following matches with the input file in xarray's IO backends: ['netcdf4', 'h5netcdf']. But their dependencies may not be installed, see:
https://docs.xarray.dev/en/stable/user-guide/io.html
https://docs.xarray.dev/en/stable/getting-started-guide/installing.html

This is probably because it looks like netcdf4-python is not installed.
Generally we should probably just ship a full conda environment that includes all the necessary python dependencies.

would this be an environment.yml in the top-level? or add dependencies that aren't directly needed by C-Star (e.g. xarray here to ci/environment.yml ?

There are multiple ways we could imagine this working in future (e.g. dask allows you to do pip install "dask[complete]"), but for now I think we should just add these python dependencies to ci/environment.yml, because we are probably going to need xarray for testing purposes anyway.


[Feat]: Add Option to upload code from local files

Is your feature request related to a problem? Please describe.
Sometimes a user may have a particular file to modify.
Describe the solution you'd like
Add Option to upload code from local files
Are you willing to contribute to this issue? [Yes/No]
Yes
Describe alternatives you've considered
User can add files from local storage for the chosen Language and it should have the same extension for a file to be uploaded
(We can also add upload from google drive as an option).

Hi @jainaayush01  , First of all, thank you so much for taking out the time to create this issue #57
Currently, the upload from the local feature is not available, Hence as you aforesaid, it would be great if this feature is also added.
For the MVP design of this feature, it would be great if we can add the " Upload from Local Storage " feature first and then continue on from fetching from the Drive feature.
It would be great if after uploading from the local storage, the system detects the extension of the code file and automatically switches to that language for further editing of the code on the platform.
Please go forward to work on this issue.  raised_hands I am assigning this Issue to you +1
And last but not least,
Keep Contributing prayfire

Okay sure I will try to implement " Upload from Local Storage " feature first.
Thanks for your time

Hi @jainaayush01 , First of all, thank you so much for taking out the time to create this issue #57
Currently, the upload from the local feature is not available, Hence as you afore-mentioned, it would be great if this feature is also added.
For the MVP design of this feature, it would be great if we can add the " Upload from Local Storage " feature first and then continue on for fetching from the Drive feature.
It would be great if after uploading from the local storage, the system detects the extension of the code file and automatically switches to that language for further editing of the code on the platform.
Please go forward to work on this issue.🙌 I am assigning this Issue to you 👍
And last but not least,
Keep Contributing 🙏🔥

True, these points are really impressive. 🙌
Detecting a file language by its extension and changing the file accordingly would really improve UX.  💫
Great to go @AyushShri  🚀

@jainaayush01 any updates?

Oh no, I totally forgot about this issue.
Okay,
Where should we provide this option.

@Nishant127 I have made the Pull Request. Thanks again for reminding me.


Error: Number of users is different than the number of passwords

Hi,
i get this error every time i run the script "The number of users is different than the number of passwords! This is most likely due to a regex mismatch".
I ma running Win7 in a VM with only 1 User on the system and latest Google Chrome.
I ran the script in 32bit powershell.
What is the issue here?

Hi there, the most likely cause for this issue is that the regex I use to
try and parse out the credential information didn't manage to capture
either the username or the credential information. When that happens, the
total number of usernames and passwords have different counts, and
therefore are no longer paired to one another. This means that you might
see a password for a different username in the results. Unfortunately this
method is prone to regex mismatches, despite my best efforts to try and
make it as accurate as possible. You could try to open up the sqlite
database yourself with a Nirsoft utility, open a raw (hex) version, and
compare the output of BrowserGather to figure out which username/password
the mismtach occurred on (you'll know because the usernames/passwords will
all be mismatched right after it occurs). I am unfortunately too busy at
the moment to keep adjusting the regexs for every edge case, but I hope you
found this informational helpful. Good luck!
On Wed, Jul 26, 2017 at 9:02 AM, Drazen Mokic<EMAIL_ADDRESS>wrote:

Hi,
i get this error every time i run the script "The number of users is
different than the number of passwords! This is most likely due to a regex
mismatch".
I ma running Win7 in a VM with only 1 User on the system and latest Google
Chrome.
I ran the script in 32bit powershell.
What is the issue here?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/sekirkity/BrowserGather/issues/3, or mute the thread
https://github.com/notifications/unsubscribe-auth/AP2Hl7oAjC1kxGGRDyS7ufr3XlDRUIg1ks5sRzj4gaJpZM4Oj4-i
.

It seams there is mistake in second regex. Try to use (?<=\x0D\x0D\x0D[\s\S]{3}\x68\x74\x74\x70)... instead of (?<=\x0D\x0D\x0D[\s\S]{2}\x68\x74\x74\x70)

Sometimes there is 3 bytes after x0D x0D x0D in second regex. '(?<=\x0D\x0D\x0D[\s\S]{2,3}\x68\x74\x74\x70) instead of '(?<=\x0D\x0D\x0D[\s\S]{2}\x68\x74\x74\x70) should work

I have the same issue - I have a single Chrome password stored<EMAIL_ADDRESS>with password "Password.1" on Facebook, and I get the same error:
Get-ChromeCreds : The number of users is different than the number of passwords! This is most likely due to a regex mis
match.
At line:1 char:1
+ Get-ChromeCreds | format-list *
+ ~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Write-Error], WriteErrorException
    + FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorException,Get-ChromeCreds

Any possibility to fix it?

For anyone else coming to this thread looking for answers, I can confirm @hannajs fix worked just fine. Changing the user regex from {2} to {2,3} fixes the problem.

I've figured out that when you press "Do not store password for this website" (or smth like that)  in Chrome, it saves website address w/o password. It causes "The number of users is different than the number of passwords!" error. Just ignore it :)


Videos are not showing to be downloaded in the recent update 2.9.0.

The download button in videos is not showing.

v2.9.0 updated the video player UI to fit better on narrower screens (#127), so the download button is now under the menu. Please can you confirm if it appears there on your device?

Yeah, the download button is under the menu, but the downloading video
format/extension is .m3u8, can't be played on Mobile vlc.
On Tue, 15 Jun 2021, 3:08 pm Jonjo McKay, @.***> wrote:

v2.9.0 updated the video player UI to fit better on narrower screens (#127
https://github.com/jonjomckay/fritter/issues/127), so the download
button is now under the menu. Please can you confirm if it appears there on
your device?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jonjomckay/fritter/issues/146#issuecomment-861349296,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQWOELJFEU32SGBFWHNNXTTTS4NRFANCNFSM46SAPAZQ
.

This problem is for some videos not all.
On Tue, 15 Jun 2021, 6:46 pm raj kishore choudhury, <
@.***> wrote:

Yeah, the download button is under the menu, but the downloading video
format/extension is .m3u8, can't be played on Mobile vlc.
On Tue, 15 Jun 2021, 3:08 pm Jonjo McKay, @.***>
wrote:

v2.9.0 updated the video player UI to fit better on narrower screens (
#127 https://github.com/jonjomckay/fritter/issues/127), so the
download button is now under the menu. Please can you confirm if it appears
there on your device?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jonjomckay/fritter/issues/146#issuecomment-861349296,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQWOELJFEU32SGBFWHNNXTTTS4NRFANCNFSM46SAPAZQ
.


add .desktop file to open it from menu

it would be good to add a desktop file and icon.
so it will be accessible from menu.

Hi, I've made a PR #5 for this one, please review.


[Distributed][Core] Support Py39 and Py38 for PP

Resolves case when:
(_RemoteRunner pid=3675275) ERROR 07-03 12:23:07 async_llm_engine.py:53] RuntimeError: Task <Task pending name=‘Task-89’ coro=<RayGPUExecutorAsync._driver_execute_model_async.<locals>._run_task_with_lock() running at /tmp/ray/session_2024-07-03_12-20-07_947853_3662522/runtime_resources/working_dir_files/_ray_pkg_cac8e03ca973e3a0/vllm/executor/ray_gpu_executor.py:354> cb=[_gather.<locals>._done_callback() at /data/youkaichao/miniconda/envs/vllm/lib/python3.9/asyncio/tasks.py:767]> got Future <Future pending> attached to a different loop

cc: @youkaichao

This isn't the cleanest solution... ideally there's a better way. Basically occurs because pp_locks is created in a different event loop.

Works on Py38 as well

This isn't the cleanest solution... ideally there's a better way. Basically occurs because pp_locks is created in a different event loop.

@andoorve  Sorry to interrupt, I am wondering why pp_locks could be in different event loop. As I understand it, the constructor only has one process and one thread, so it only has one loop?

Hi @HarryWu99 it's difficult to tell why this happens, I did not investigate in that level of detail. However, based on the error message we know the construction and use must be in 2 different loops.
There could be a default event loop set somewhere else, one guess is uvicorn although this is not for sure


update mongodb dependency

Could you please update the mongodb dependency to a newer version? I am currently getting an upstream dependency conflict as i already use a newer version in my project

Can't confirm your issue.
~/Tests
➜ mkdir typeorm-storage

~/Tests
➜ cd typeorm-storage

~/Tests/typeorm-storage
➜ npm install @satont/grammy-typeorm-storage typeorm mongodb --save

added 92 packages, and audited 93 packages in 14s

9 packages are looking for funding
  run `npm fund` for details

found 0 vulnerabilities

~/Tests/typeorm-storage via ⬢ v16.10.0 took 15s

Can you provide more information how i can reproduce it?


Fix error thrown for nullable field resolving null

Closes #22

Seems cool, but I think the commits are not conform to semantic release, so they won't trigger a release. I would count this as a simple fix, so there could be a single commit like this fix: error thrown for nullable field resolving null

@DanielMSchmidt ok, can't you squash while merging and rename as sematic wishes?

Done 👍

:tada: This PR is included in version 3.0.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Torch] Various updates for PyTorch frontend

This PR adds various updates to the PyTorch frontend, to fully support the recent transformer based object detection model from facebook, DETR https://github.com/facebookresearch/detr. After this PR, DETR runs on TVM and gets the correct results. TVM with auto scheduled GPU conv2d and batched matmul is 1.4x faster than PyTorch in my environment.
Also added various missing ops reported by hummingbird projects. @interesaaat

Adds support for ops requested by hummingbird project: logical_and, masked_select, sort and argsort. masked_select requires VM to run.
Adds support for cumsum and masked_fill op, to enable importing DETR https://github.com/facebookresearch/detr. cumsum is also requested by hummingbird.
Fix negative indexing using mode="wrap" in Relay take op. Without this, the result doesn't match with DETR.
Remove explicit broadcast_to before batch_matmul. Without it, memory usage blows up during constant evaluation of DETR. This is the same issue as https://github.com/apache/tvm/pull/6616, please see the explanation there.
Add  torch.nn.Transformer to the tests.

Since torch.sort returns both sorted values and indices while the Relay one doesn't, torch.sort conversion is not efficient, especially for multidimensional input (currently does both sort and argsort!). Suggestions for a better implementation are welcome. UPDATE: fixed
please review @siju-samuel @jwfromm @kevinthesun @t-vi

@t-vi Thanks, I think I tried gather before but for some reason I got wrong results, so I gave up gather. I tried again now, and it worked :) I don't know what I was doing, but I'm happy now.

@t-vi Thanks, I think I tried gather before but for some reason I got wrong results, so I gave up gather. I tried again now, and it worked :) I don't know what I was doing, but I'm happy now.

Thanks @siju-samuel @t-vi

Thanks @siju-samuel @t-vi


Git Graph article missing description

See: https://ardalis.com/git-graph-visualizes-branches-in-vs-code-for-free/#sq_hdnhaitmzw

This query might be able to help us find where the null descriptions happen:
    query {
          site {
            siteMetadata {
              siteUrl
            }
          }
          allMdx(filter: { frontmatter: {description: {eq:null}}}){
            nodes {
                fields { slug }
                frontmatter {
                  title
                  description
                }
              }
          }
        }

This can be run locally in GraphiQL:

Run gatsby clean.
Run gatsby develop.
Assuming Step 2 runs cleanly, go to  http://localhost:8000/___graphql
Copy the query above into the query window.
Select Run.


add confirmation message when delete is clicked

Issue:
Delete button is really dangerous #391

would love to see this merged cuz... agreed. no confirmation is a bit dangerous.

+1

could use some help getting a docker image with these changes working...
I'm not very good with node... I manage to build the docker image, but when I run it, I get
Missing devDependencies to compile resource cache, please reinstall or run `npm install --only=dev` inside the project directory

hey. so, I got @Whiterid's code built using npm/node installed locally on an x86_64 ubuntu 18.04 machine and was running it successfully off a mounted exfat drive. I've never built something in node before so it was a bit of a trick to figure out how to get all the dependencies installed on my machine, but it wasn't too hard to figure out.
I then got a raspi 4B+ (armv7) and copied the resulting repo (dist and some other such build files that aren't tracked) over to the pi. I swapped the header in the Dockerfile to be arm32v7/node and successfully built the docker image with this PR's contribution and can confirm it works flawlessly on the raspberry pi. (all that is to say I didn't need node installed on the pi).
For anyone else coming by... you can avoid building by pulling my docker images, here are the resulting paths I pushed to:
mathematicalmichael/droppy:x86 (latest)
mathematicalmichael/droppy:arm32v7
thank you so much @Whiterid !!!

also, despite the README saying that silverwind/armhf-droppy supports ARMv7, I found that I couldn't get it working on the pi 4. I got it running and logged in, but there was a perpetual loading screen on the homepage after login. I would suggest building from the arm32v7/node base if you're shooting for the 4. perhaps the arm32v6 base would support the older models.

@silverwind pinging you because this adds value to the project and the delete button is really scarry.

Hi @mathematicalmichael !
I can't pull your docker image.
i try: docker pull mathematicalmichael/droppy
I got this message:
Using default tag: latest Error response from daemon: manifest for mathematicalmichael/droppy:latest not found
i'm on linux debian 9.
Ty for your response. And ty to your job too.

@Whiterid oh that's my bad
https://hub.docker.com/r/mathematicalmichael/droppy/tags
at this time it looks like I only pushed my ARM images. let me go ahead and push the x86 one. apologies.

@mathematicalmichael  you rocks !

@silverwind any thoughts about the confirmation for delete option??

Please add this function. Without it is way to dangerous to use droppy on a prod system.


perform js inspector on tracing task

Describe the feature
inspecting js tracer is currently performed in sync which is blocking
https://github.com/paradigmxyz/reth/blob/0147e5033001069f258aa1e03fa1a7166f989e4c/crates/rpc/rpc/src/debug.rs#L292-L292
it should be moved to a tracing task like:
https://github.com/paradigmxyz/reth/blob/0147e5033001069f258aa1e03fa1a7166f989e4c/crates/rpc/rpc/src/debug.rs#L246-L249
FYI @tcoratger
Additional context
No response

Will have a look!

@mattsse Are you sure that this change is desirable? I understand the purpose to migrate the execution to an async case but it looks like the sync execution was done on purpose with the mentioned comment:
https://github.com/paradigmxyz/reth/blob/4ab924c5d361bbfdcdad9f997d16d67b4a1730b7/crates/rpc/rpc/src/debug.rs#L267-L268
With a simple migration to something like:
let mut inspector = JsInspector::new(code, config, to_db_service)?;

let (res, env, inspector) = self
  .inner
  .eth_api
  .spawn_with_call_at(call, at, overrides, move |db, env| {
      let (res, _) = inspect(db, env, &mut inspector)?;
      Ok((res, env, inspector))
  })
  .await?;

let result = inspector.json_result(res, &env)?;
Ok(GethTrace::JS(result))

we will get an error like:
error[E0277]: `NonNull<boa_gc::internals::gc_box::GcBox<boa_engine::script::Inner>>` cannot be sent between threads safely
   --> crates/rpc/rpc/src/debug.rs:296:26
    |
296 |                         .spawn_with_call_at(call, at, overrides, move |db, env| {
    |                          ^^^^^^^^^^^^^^^^^^ `NonNull<boa_gc::internals::gc_box::GcBox<boa_engine::script::Inner>>` cannot be sent between threads safely

due to the fact that JSTracer and all JS types are not Send, as mentioned in the comment, what do you think?

I think it works if you move

let mut inspector = JsInspector::new(code, config, to_db_service)?;

inside the closure


Updating CLA link from cla2 to cla

cla2.dotnetfoundation.org link is returning a 404.  Changing this to cla.dotnetfoundation.org takes it to a page about the CLA.  Found this while trying to figure out how to commit to this repo.  Figured this probably should be fixed.

@sadukie Thank you so so much for finding this problem and correcting it!


User can Scroll in Windows leading to bad render

The middle mouse button scroll in Windows causes glitches in render. I have attached screenshots below from one of my presentations.
Moreover, I can scroll indefinitely in either axes using the same middle mouse scroll when the rotate transforms are used (only in Firefox).
Wheel scroll also causes slides to rearrange on the display.
This issue is not reproduced in Linux using Firefox.
Normal view (Chrome 38 on Windows):

After scrolling using wheel (Chrome 38 on Windows):

Firefox keeps the backgrounds intact, though.

X-Ref: #262, #263.

Apologies if this is the wrong place to ask. I was wondering if there is an ability (planned maybe) to support zoom in/zoom out behaviour for mousewheel/touchpad so that panning and zooming is possible independently from the steps order. Couldn't find a plugin that does this, but maybe I'm not looking in the right places. Thanks for any help!

Hi @jorisvanzundert
The right place to ask would be to just file a new issue

Thnx @henrikingo I've done so. Would be happy to help with implementation, but I'm not up to speed with the code base yet at all.

It's fixed. We can close this issue, although I think that I might want to implement such a zoom plugin


3074: Improve non observers show on StandardFormReport.

The system was fetching all users to show them on the form. This was
making the system extremely slow when it had a lot of users on the
database. I created one query to fetch the users limiting how many
we are getting and used SQL_CALC_FOUND_ROWS to get the total count
without hitting the db again.

Looks great! Thanks for changing it into a query. Nice work. Just a few refactor suggestions in my comments.

Looks great, please merge.


watchdog: Prevent reload if a file gets closed without any changes

Ignores the new closed_no_write event type fired by watchdog 5.x that is causing unnecessary reloads.
fixes https://github.com/pallets/werkzeug/issues/2945

Test case added
Requirements updated
Changelog added

Will send another, properly tested PR


FileSelNative::Execute() fix: default filename was some trash on saving

BUG + FIX: FileSelNative shows default filename as some trash characters [message #58798]

Uhm, those NULL characters added to the list are for multiple filenames.
https://docs.microsoft.com/en-us/windows/win32/api/commdlg/ns-commdlg-openfilenamew
You are supposed to separate files with NULL, then end the list with double NULL...

Fixed with slightly modified code...

Thanks!


Add guides to explain how the contract extensions can be used

Description:
A new section to explain how Semaphore can be used for specific use-cases by using the contract extensions.
Tasks:

[ ] Voting app
[ ] Whistleblowing app

Closing this as there're already other ongoing issues to write articles about extensions.


JabRef GSoC projects

Jabref Blog

Sound good already! Regarding JabRef-Account, I would add a short sentence, like: More on what's behind a JabRef-Account will be revealed in later blog posts.

So far, looks good to me. Now only @btut's contribution is missing ;-)
Coming soon!

As I don't have the rights to push to @mohit038, I created https://github.com/mohit038/blog.jabref.org/pull/1.


lightning: adjust lightning additional routes

What problem does this PR solve?

Issue Number: ref https://github.com/pingcap/tiflow/issues/3340
What is changed and how it works?
After lightning supports additional routes in https://github.com/pingcap/tidb/issues/37797, dm should adjust this field for inline lightning.
Check List
Tests

Integration test

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
None

/run-dm-integration-tests

/run-dm-integration-tests

/run-dm-integration-tests

@GMHDBJD PTAL

/run-all-tests

/merge

Codecov Report

Merging #7467 (aa58f4c) into master (6b48846) will decrease coverage by 6.1267%.
 Consider uploading reports for the commit 0bd27db to get more accurate results

Additional details and impacted files

Flag
Coverage Δ

cdc
67.4312% <ø> (-0.2066%)
:arrow_down:

dm
52.0259% <50.0000%> (?)

engine
61.0874% <ø> (-0.0963%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
@@               Coverage Diff                @@
##             master      #7467        +/-   ##
================================================
- Coverage   65.9877%   59.8610%   -6.1267%
================================================
  Files           564        804       +240
  Lines         52337      92394     +40057
================================================
+ Hits          34536      55308     +20772
- Misses        15129      32274     +17145
- Partials       2672       4812      +2140

/run-verify

/run-verify


Support angularCompilerOptions schema in tsconfig.json

Command
config
Description
Can the team support angularCompilerOptions type in tsconfig.json with schema, just like $schema defined in angular.json:
{
    "$schema"<EMAIL_ADDRESS>}

so user can change angularCompilerOptions with autocomplete feature and no typo there
Describe the solution you'd like
No response
Describe alternatives you've considered
No response

Also, .browserslistrc.template line 9 should trim the extra two spaces before npx browserslist


Unable to import module 'wsgi'

After testing the plugin with the project very much similar to the one in example, in which local tests work correctly, after deployment to AWS I get the following message on call:
{
  "errorMessage": "Unable to import module 'wsgi'"
}

Detailed log:
START RequestId: 6b36272e-920d-11e6-9aed-0d5b9d25097b Version: $LATEST
Unable to import module 'wsgi': No module named datastructures

END RequestId: 6b36272e-920d-11e6-9aed-0d5b9d25097b
REPORT RequestId: 6b36272e-920d-11e6-9aed-0d5b9d25097b	Duration: 0.41 ms	Billed Duration: 100 ms 	Memory Size: 1024 MB	Max Memory Used: 15 MB

It seems that the requirements are not correctly visible to WSGI. Everything in the deployment package seems to be in the right place.
Thanks for the help.

If you look inside the zip file created by serverless, du you see a .requirements directory?

Yes, everything is in place and the "missing" werkzeug is in the folder. I think the problem might be with the python path adding, but I cannot check it, at least easily, as wsgi.py, which is the main point of entry, does not load.

I just tried deploying the example from the README, using this serverless.yml file:
service: example

provider:
  name: aws
  runtime: python2.7

plugins:
  - serverless-wsgi

package:
  exclude:
    - node_modules/**

functions:
  api:
    handler: wsgi.handler
    events:
      - http:
          path: cats
          method: get
          integration: lambda-proxy
      - http:
          path: dogs/{id}
          method: get
          integration: lambda-proxy

custom:
  wsgi:
    app: api.app

Producing the following Lambda bundle:
https://dl.dropboxusercontent.com/u/313905/example.zip
Could you check how it compares to your bundle?

OK I've found that the datastructures.py  from werkzeug. The reason was that in my 'serverless.yml' I had exclude as follows:
exclude:
    - data

This meant to exclude my data folder; when I changed it to - data/ only the folder is excluded correctly. I was not aware that this form of exclude works as a global wildcard.
Nonetheless, I get a new error now:
u'requestContext': KeyError Traceback (most recent call last): File "/var/task/wsgi.py", line 34, in handler script_name = '/
{}
'.format(event[u'requestContext'].get(u'stage', '')) KeyError: u'requestContext'

It may be related with the fact I' using CORS in Serverless:
events:
      - http:
          path: task
          method: get
          integration: lambda-proxy
          cors: true

Strange, enabling CORS does not cause any issues for my example (it shouldn't, as the CORS setting will just add an OPTIONS method to APIGW).
According to AWS docs, requestContext should always be available for LAMBDA-PROXY integrations. Could you check in API Gateway, that your endpoint is indeed configured with a LAMBDA-PROXY integration? Also, I assume that you're seeing that error in the logs after invoking the Lambda through a HTTP call to API Gateway, right?

The last one did the trick, though, for some reason, serverless.yml setting as above, i.e.:
events:
      - http:
          path: task
          method: get
          integration: lambda-proxy
          cors: true

doesn't change the Integration Request: I had to change it manually. Also I've tested it on a vanilla environment and it still doesn't trigger the integration setting. But after setting it manually, Serverless doesn't change the setting back, so it's fine for now at least.
I also tested it adding two new endpoints foo and bar, with and without CORS, respectively. In both cases the lambda-proxy setting hasn't been added.
Do you think this is related with your project or should I report it to Serverless directly?
Thank you very much for the help!

Ok, great that you found the issue. It definitely sounds like a Serverless bug, and you should be able to reproduce regardless of using this plugin. I'll close this issue and it would be cool if you could report directly to Serverless!

Just leaving a note: the issue with setting integration was related to older version of Serverless I had, namely, 1.0.0-rc2; after updating to 1.0.2 the issue dissipated.


updating the Funny System again (1.2.0)

Your Discord Name
(no need to do this anymore)
Mod uniqueName
O32.FunnySystem
Mod name
The Funniest System
GitHub repository URL
https://github.com/RealOrbital32/funny-system
Author display name
Orbital 32 (a.k.a. cigtu)
Parent uniqueName
xen.NewHorizons
Utility mod
No response
Alpha Mod
No response

and yes, i did update the version number, so i'm 90% sure you can close that merge thing
is there a more effecient way to ask to update a mod?

Will update automatically, no need for an issue


Fix: webm -> wav for iOS Safari

Resolves #2836.

Any thoughts why?

It has to do with audio encoding probably. Or the fact that we load a blob. We need to experiment with different options to see what works on iOS.


Initializes VmMgr

The recommended way to review this PR is to review each commit separately.

I assume the next step is to move all the memory syscalls into the new VmMgr, right?

Correct.

@ultimaweapon In which struct should the map of the addresses to the allocations be?