CHI cleanup

removing all experimental code

were you intending to leave experimental.py and its various references in?
also we should ditch get_task_workers_sampled and get_task_workers_for_download from serializers/task.py

Not really, can you point where those are? I thought I removed all.
Will remove sampled methods

Oh you mean serializer and viewset, gotcha thanks


RuntimeError: dictionary changed size during iteration

Hi, I've got this error below when using websocket for streaming market depth.
Traceback (most recent call last):
  File "inago-simulation-BTCUSD-realtime-linux.py", line 118, in <module>
    if len(depth_cache.get_bids()[:5]) > 0:
  File "/home/soichiro_tsuda/miniconda3/lib/python3.6/site-packages/binance/depthcache.py", line 75, in get_bids
    return DepthCache.sort_depth(self._bids, reverse=True)
  File "/home/soichiro_tsuda/miniconda3/lib/python3.6/site-packages/binance/depthcache.py", line 114, in sort_de
pth
    lst = [[float(price), quantity] for price, quantity in vals.items()]
  File "/home/soichiro_tsuda/miniconda3/lib/python3.6/site-packages/binance/depthcache.py", line 114, in <listco
mp>
    lst = [[float(price), quantity] for price, quantity in vals.items()]
RuntimeError: dictionary changed size during iteration

My code was pretty simple --- read depth_cache every 1s
dcm = DepthCacheManager(client, 'BTCUSDT', refresh_interval=0) # disable refresh
depth_cache = dcm.get_depth_cache()

while True:
    if len(depth_cache.get_bids()[:5]) > 0:
        curr_bids = depth_cache.get_bids()[:5]
    else:
        curr_bids = ""
    if len(depth_cache.get_asks()[:5]) > 0:
        curr_asks = depth_cache.get_asks()[:5]
    else:
        curr_asks = ""
    time.sleep(0.6)

I've got the above error on Windows (7 or 10) as well as on Linux (GCP), both with Anaconda Python 3.6. python-binance was installed via pip (ver 0.6.4).
As a workaround, I changed vals.items() to list(vals.items()) in depthcache.py as follows:
lst = [[float(price), quantity] for price, quantity in list(vals.items())]

I don't have the above error message any more after this modification, although I'm not 100% sure if it's the right way as I'm still a noob.

a little update: I still got the error even after the modification in the code... although it was significantly less to see the error.


StudentMinimizeScheduleHoles criterion

I enabled StudentMinimizeScheduleHoles criterion by adding org.cpsolver.coursett.criteria.additional.StudentMinimizeScheduleHoles to General.AdditionalCriteria
then adding distribution preference same student to group of classes and run the solver
but still have long time between classes on the same days
And Student class distance field in solver page show question mark value

it seems not to work!
how can i use StudentMinimizeScheduleHoles criterion with same student distribution preference?

It appears you have no students in the problem. The StudentMinimizeScheduleHoles criterion needs some form of student course demands in order to work. These could be in a form of last-years student course enrollments, curricula, the actual pre-registrations, or a combination of these -- see the Student Course Demand parameter of the Solver for more details about these options.
If you use distribution preferences to indicate which classes cannot overlap in time, you can also use distribution preferences to further restrict these classes during the day. For instance, see the Max Hours A Day, Work Day, Max Holes, or Max Break distributions from the Additional Distribution Constraints.

Can I use student group reservations instead of same student distribution preference to indicate which classes cannot overlap in time
and if I do, will StudentMinimizeScheduleHoles criterion be work ?

Thank you


Introducing Altair Guru on Gurubase.io

Hello team,
I'm the maintainer of Anteon. We have created Gurubase.io with the mission of building a centralized, open-source tool-focused knowledge base. Essentially, each "guru" is equipped with custom knowledge to answer user questions based on collected data related to that tool.
I wanted to update you that I've manually added the Altair Guru to Gurubase. Altair Guru uses the data from this repo and data from the docs to answer questions by leveraging the LLM.
In this PR, I showcased the "Altair Guru", which highlights that Altair now has an AI assistant available to help users with their questions. Please let me know your thoughts on this contribution.
Additionally, if you want me to disable Altair Guru in Gurubase, just let me know that's totally fine.
Summary by Sourcery
Introduce the Altair Guru on Gurubase.io, an AI assistant that leverages data from the Altair repository and documentation to answer user questions. Update the README.md to include a badge linking to the Altair Guru.
New Features:

Introduce Altair Guru on Gurubase.io, providing an AI assistant to help users with questions related to Altair.

Documentation:

Add a badge to the README.md linking to the Altair Guru on Gurubase.io.

Hi everyone,
Have you had a chance to check this out?
Additionally, I’d like to update you on the release of the Maintainer Panel feature on Gurubase. With this panel, you can add, remove, or update data sources, as well as change the logo and more. You can find the details here.
In the near future, I’m planning to include analytics insights in this panel, such as the number of questions asked, the most frequently asked ones, and more. I’ll be sure to update this thread once it’s available. However, in case I miss it, I highly recommend joining our Discord channel to stay updated.
If you’d prefer that I don’t update this thread anymore, please let me know.


enhancement add bvar stating cached io

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall


T3K Multi-Chip Trace Failures

Branch: asaigal/async_trace_multichip
Trace Unit Test on this branch, running a chain of ops has the following failure modes:
T3K Async Failures

T3K 4 chip hang w/o all-gather (after first trace iteration)
T3K 4 chip assert w/ all-gather  (event id order issue after first trace iteration, similar to https://github.com/tenstorrent/tt-metal/issues/8474)
T3K Regular Mode Failures
T3K 8 chip hang w/ all-gather (first trace iteration)
T3K 8 chip hang w/o all-gather (first trace iteration)
The following configurations pass:
Same tests (w/o all-gather) pass on 8 chip GS
T3K 4 chip w/o async and w/ all-gather
All tests without trace enabled on T3K,
Regular mode test:
pytest -svv tests/ttnn/unit_tests/test_multi_device.py::test_multi_device_multi_op_trace
Async mode test:
pytest -svv tests/ttnn/unit_tests/test_multi_device_async.py::test_multi_device_multi_op_trace
Instructions to toggle configurations are in the test file. This is a dev branch, where we set the device id to match the global trace ID, but that should be functional for single-trace workloads.

Updated Test Status, since picking up prefetch stall commit:

All single chip trace tests are stable on GS and WH (this includes activations with different shapes and multi-trace).
T3K: All tests on a remote chip hang in the first trace execution iteration (we are missing something on the R-Chip path here).
T3K: Multi-trace tests on 4 Chips (no R-Chip here) hanging with specific input shapes and all-gather (these tests have been disabled on CI).

The hanging tests pass without trace.
Branch: https://github.com/tenstorrent/tt-metal/tree/asaigal/ttnn_trace_rebased
Multichip test:
pytest -svv tests/ttnn/unit_tests/test_multi_device_trace.py

These failures should be resolved on main as of d44016b.


Kafka and NATSS were vendoring eventing for impl. Fixed.

Proposed Changes

Do not use the vendored eventing/ code for controllers for NATSS and Kafka

Release Note

NONE

/assign @matzew

/cc @grantr

/hold

/hold

/close
done in another pr.


Add Leap 16 images

Relevant PR https://github.com/os-autoinst/opensuse-jobgroups/pull/489
Progress Ticket: https://progress.opensuse.org/issues/164141

Thank you @foursixnine !


Extension blocks all incoming sound/conversation

Some time ago, I noticed that Google Meet was not sounding other peoples voices. More specifically, when I test audio from the meet settings, the dummy song is getting played and I can hear the music. When someone speaks in the meet, I can see their avatar is getting blue highlight and blue pulse animation, but I can't hear peoples voices.
When I disabled this extension, I could hear peoples voices again with no trouble.
I'm not sure if it's related to macOS update but, I have upgrade from Ventura to Sonoma and it might have an effect

I have no idea how this would be possible. Is this still an issue?

Well, I have disabled it of course but I can re-activate and observe it

That would be great. I'm actively using the extension, and I have not experienced that issue.

Just tried joining a meet and seems like the issue still persist 😞

I finally experienced the issue. Refreshing the page and rejoining the meeting always resolves it though. I'm still wondering if it's unrelated to my extension. I don't see how my extension could interfere with that. For me it only happens sometimes, and I don't need to disable or uninstall the extension to resolve it, so it's difficult to know what's actually causing the issue.

I see. Well, I have tried refreshing, reopening the page even in a new tab didn't worked. So the last resort was to disable it.
I'll try to debug it on my end, and see if I can find the root cause

I'm not sure, but when I added a timeout timer here, the issue is no longer persists. Maybe Chrome's framework is getting triggered before it's getting ready state. I'm not talking about the DOM-ready, but it can be a framework issue.
Having a little delay seems to sort this issue out, however, I'll be observing it for the next days

Nice! I'll give that a try too.

@seahindeniz How has it been with the delay?

So far no trouble at all. Works like a charm

Alright cool, I'll push a new update with the delay

I made some other changes. Would you mind pulling them and testing before I push the next update to everyone?

I pushed them to everyone.

Sorry for delaying, I was dealing with a time taking personal matter. I can pull and test it if you still need it

No worries! I was feeling confident it would work this time, so I decided to go for it. Yeah, that would be helpful. You can pull from the repo or use the one in the store. They are the same now.

I just joined to a meeting and the sound issue is still there 😔

So you chose to use the one on the store? Did it get updated to version 3.1.0 for you? If not, then it would definitely have the same issue.

Yeah, I currently have 3.0.0 the local version and 3.1.0 the store version. Enabling store version blocks audio completely. If I switch back to local version 3.0.0, working fine. However, when I enable it the first time yesterday, the local version was also interrupted audio too but when I rejoin the meeting again, issue got solved. I think it must be insufficient countdown that I added for 500ms

I added more of a delay into the repo version. Can you test again please?


Flow crashes when seeing an invalid package.json

See it in action on a real package:
$ yarn init
$ yarn add<EMAIL_ADDRESS>$ yarn add<EMAIL_ADDRESS>$ touch .flowconfig
$ node_modules/.bin/flow
Launching Flow server for /home/jdscheff/projects/browserify-flow-bug/foo
Spawned flow server (pid=26513)
Logs will go to /tmp/flow/zShomezSjdscheffzSprojectszSbrowserify-flow-bugzSfoo.log
Monitor logs will go to /tmp/flow/zShomezSjdscheffzSprojectszSbrowserify-flow-bugzSfoo.monitor_log
Lost connection to the flow server (3 retries remaining)
Launching Flow server for /home/jdscheff/projects/browserify-flow-bug/foo
Spawned flow server (pid=26540)
Logs will go to /tmp/flow/zShomezSjdscheffzSprojectszSbrowserify-flow-bugzSfoo.log
Monitor logs will go to /tmp/flow/zShomezSjdscheffzSprojectszSbrowserify-flow-bugzSfoo.monitor_log
Lost connection to the flow server (2 retries remaining)
Launching Flow server for /home/jdscheff/projects/browserify-flow-bug/foo
Spawned flow server (pid=26557)
Logs will go to /tmp/flow/zShomezSjdscheffzSprojectszSbrowserify-flow-bugzSfoo.log
Monitor logs will go to /tmp/flow/zShomezSjdscheffzSprojectszSbrowserify-flow-bugzSfoo.monitor_log
Lost connection to the flow server (1 retry remaining)
Launching Flow server for /home/jdscheff/projects/browserify-flow-bug/foo
Spawned flow server (pid=26574)
Logs will go to /tmp/flow/zShomezSjdscheffzSprojectszSbrowserify-flow-bugzSfoo.log
Monitor logs will go to /tmp/flow/zShomezSjdscheffzSprojectszSbrowserify-flow-bugzSfoo.monitor_log
Lost connection to the flow server (0 retries remaining)
Out of retries, exiting!

$ echo "[ignore]" > .flowconfig
$ echo ".*/node_modules/module-deps/test/invalid_pkg/package.json" >> .flowconfig
$ node_modules/.bin/flow
Launching Flow server for /home/jdscheff/projects/browserify-flow-bug/foo
Spawned flow server (pid=26763)
Logs will go to /tmp/flow/zShomezSjdscheffzSprojectszSbrowserify-flow-bugzSfoo.log
Monitor logs will go to /tmp/flow/zShomezSjdscheffzSprojectszSbrowserify-flow-bugzSfoo.monitor_log
No errors!

Here is what that ignored file looks like:
"just a string"

Seems that the same thing happens when you have any file named package.json containing something like that.
I'm also going to submit a PR to module-deps to get them to stop putting tests in their npm package, but I figure it's a more general problem that you guys might want to look into.

a workaround is to add the conflictive file to the [ignore] section in the .flowconfig file:
<PROJECT_ROOT>/node_modules/browserify-middleware/.*/test/.*

That's what I did, for the file that was causing me problems. But it's very poor usability to force users to do that, especially when the error message doesn't tell you what file (or even what dependency) is causing the problem. It's not easy to find the offending file if you have many dependencies.

Looks like this was fixed in https://github.com/facebook/flow/commit/f942d9676d174d3469ed532770934063518ce4f5


Load all translations from a central location

Is your feature request related to a problem? Please describe.
I'm in the process of upgrading to v8 and I've got everything going but I'm wondering if there's a better way to load translations. I have a pretty small project so there is no need for dedicated namespaces for each page. I'd rather load all translations at once and not have to worry about it on each individual page. This worked out of the box with v7 from _app.tsx but now I have to add getServerSideProps or getStaticProps to each page, which is kind of a hassle.
Is there a way around this already or will I really have to add loading of translations to each page individually? Loading them manually from _app.tsx doesn't work I assume since custom apps don't support getServerSideProps or getStaticProps.

Hi @overdub60 – you will always need to pass serverSideTranslations into each of your page components. That is just how next-i18next@8 works.
There were similar patterns/requirements at the page level in previous versions, as well.
I suppose we could consider a config option to omit namespaces entirely, and always pass all translation content to every page. Is that what you're requesting? I'd be hesitant to do that, as it encourages non-optimal patterns.

I suppose we could consider a config option to omit namespaces entirely, and always pass all translation content to every page. Is that what you're requesting? I'd be hesitant to do that, as it encourages non-optimal patterns.

@isaachinman Thanks for getting back to me! Yeah, I think what you're suggesting is actually what we would like. Like I said, we only have about 100 lines of translations so it seems like overkill to separate all of these into namespaces and I think there is a use case for it for smaller projects.

You don't have to separate anything out into namespaces if you don't want to! I've just added "send all namespaces" behaviour with #984.
Outside of that, serverSideTranslations is needed as it is how we pass the actual translation data into pages. The only other option would be what is being proposed in #993.


BigDecimal floor() throws ArithmeticException if current exponent is negative

Describe the bug
https://github.com/ionspin/kotlin-multiplatform-bignum/blob/main/bignum/src/commonMain/kotlin/com/ionspin/kotlin/bignum/decimal/BigDecimal.kt#L1445
    fun floor(): BigDecimal {
        return roundSignificand(DecimalMode(exponent + 1, RoundingMode.FLOOR))
    }

but DecimalMode doesn't accept neither 0 (with roundingMode != RoundingMode.NONE) nor negative precision.
To Reproduce
Steps to reproduce the behavior:
call floor() on a BigDecimal with negative exponent
Expected behavior
A clear and concise description of what you expected to happen.
Platform
JS
If JS (please complete the following information):
chrome
Additional context
bignum version 0.3.4

A workaround is to instead call roundToDigitPositionAfterDecimalPoint(0, RoundingMode.FLOOR)


Feedback related to Project 1

Congratulations on coming up with a smooth UI and data flow.
UI seems very minimal and intuitive.
Below is the resultant data:-

There are some points I will which we want to bring to your knowledge:-

You tried well to persist the data to db for any search before returning it to the user. But It seems some of the time durations have huge data that it's not getting retrieved properly because of the MongoDB limitations. Maybe you can compress the lat-long data to already compressed data and then store to DB. So that merging documents won't reach the limit.
Docker Images can be reduced in size using alpine os images of Node, python and Java(Not sure about this)
Build time can be removed by pushing images on Docker Hub

Review Done by Team Neo(https://github.com/airavata-courses/neo)

Thanks for your review !

Yes, thanks for reporting bug caused by large data set. We have opened an issue #71
Planning to include docker hub in out next sprint


Add Conceptual 12M

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Looks like your dummy_data.zip file is not in the right location ;)
datasets/datasets/conceptual_12m/dummy/default/0.0.0/dummy_data.zip
->
datasets/conceptual_12m/dummy/default/0.0.0/dummy_data.zip


Add C# support for server SSL certificate reloading.

What version of gRPC and what language are you using?
gRPC v1.13.1 NuGet package (Latest Stable) in C#
What operating system (Linux, Windows, …) and version?
Windows 8.1 and 10
What runtime / compiler are you using (e.g. python version or version of gcc)
Visual Studio 2015 and 2017
What did you do?
Attempt to use dynamic server SSL certificates as per #12644
What did you expect to see?
API integration for facilitating this feature in C#, similar to #13188 in Python
What did you see instead?
There does not seem to be any integration for this feature in C#. I searched through the existing and resolved issues, and the open/closed pull requests and have not found mention of this so far. If I have missed something, then simply point me in the right direction and I will close this right away.
Anything else we should know about your project / environment?
We are currently leveraging gRPC for a number of interprocess and local network communications. Due to the nature of our product, we need to be able to rotate certificates to ensure that they are kept up to date and to be able to revoke certificates as needed. This is not as important on the client level, as those processes are generally short-lived. The server however tends to be long-running, so restarting it to refresh the SSL certificates is less than ideal.
I found that the ability to refresh certificates was added to the core in a previous set of pull requests, and support has been added to Python as well as others (Go, I think). I cannot find any information regarding this feature being added to C# or it being planned, and it would be of great benefit.

Thanks for reporting this. Currently we don't have plans to work on this, but we'd be happy to accept a PR that adds this feature.

How about with c++? Is it supported now or not ?  Hope your reply.

Thanks for reporting this. Currently we don't have plans to work on this, but we'd be happy to accept a PR that adds this feature.

How about with c++? Is it supported now or not ? Hope your reply.

How about with c++? Is it supported now or not ? Hope your reply.

I believe it should be supported in c++, as per https://github.com/grpc/grpc/pull/12644
I haven't personally tested it, but that was the base upon which the support in python was added, so it should work.

How about with c++? Is it supported now or not ? Hope your reply.

I believe it should be supported in c++, as per #12644
I haven't personally tested it, but that was the base upon which the support in python was added, so it should work.

I search some functions mentioned with that ticket, but got nothing.
Hope you give relative function interface for grpc_server if possible.

We currently do not support credential reload in C++ (it is only supported in C-core #12644).

+1 to this.
Have any plans formed to work on this, since?

@jtattermusch I would like to implement the API integration for this feature. Would you be so kind as to give me some code pointers. I would be especially glad for a previous pull request which added C# API integration for some other feature. I am sorry if this is not the proper forum for this discussion. I have also posed the same question over at google groups: https://groups.google.com/forum/#!topic/grpc-io/C0Xt4qFQdm4

Let's keep this open.

Since Grpc.Core is now in maintenance mode (see https://grpc.io/blog/grpc-csharp-future/), we don't plan to add any new features.


ToastBackgroundTask vs MyToastNotificationActionTrigger

I am wondering why "ToastBackgroundTask " is checked for creating "MyToastNotificationActionTrigger".

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 0a268967-4b29-b53c-10a5-fd42ea97d8c8
Version Independent ID: a6b72ed5-1cdb-9fe8-d05b-658cc4801c5a
Content: Send a local toast notification - UWP app developer
Content Source: windows-apps-src/design/shell/tiles-and-notifications/send-local-toast.md
Service: windows
GitHub Login: @anbare
Microsoft Alias: mijacobs

@anbare
The following code from the document checks whether "ToastBackgroundTask" has been registered:
// If background task is already registered, do nothing
if (BackgroundTaskRegistration.AllTasks.Any(i => i.Value.Name.Equals(taskName)))
    return;

If it is not registered, the following code from the document seems to create a background task named "MyToastNotificationActionTrigger":
// Otherwise request access
BackgroundAccessStatus status = await BackgroundExecutionManager.RequestAccessAsync();

// Create the background task
BackgroundTaskBuilder builder = new BackgroundTaskBuilder()
{
    Name = "MyToastNotificationActionTrigger",
};

My questions: why does the code check "ToastBackgroundTask" for creating "MyToastNotificationActionTrigger"?
Perhaps I miss some fundamental understanding of how the background task works.

@zipswich thanks for clarifying.
Background task registrations are persisted by the system. There's no need to re-register a background task that's already registered (for example, if your app was already launched once and registered the background task, there's no reason to register the background task again when you get launched the second time).
Therefore, we check if the background task is already registered.
If the background task is already registered, we're all good, and our app's background task will be fired when the user clicks a background activation button on a toast.
If the background task hasn't been registered yet (first time app has ever launched, or the OS potentially had a critical error and lost registrations), then we need to register the background task so we can receive background toast activations.

Thank you for the clarification, @anbare
Actually, I understood the reason for checking and registering as you described.  My question is much simpler than what you tried to explain.  Let me try to ask my question with a bit font styling effect to clarify it:
Why does the code check task
ToastBackgroundTask
while registering
MyToastNotificationActionTrigger?
ToastBackgroundTask
has different spelling than
MyToastNotificationActionTrigger
These two works also have different lengths.
Let me know if my question is clear.  If not, I will draw a picture to clarify it.  I know that I probably miss something fundamental.

Oh, apologies, there was a typo in the code example - that's what you were trying to point out!
Indeed, both sections of code should be using the string "ToastBackgroundTask", I've submitted a pull request to fix this, the correct code is below.  The docs should be updated in a few days. Thank you zipswich!!
const string taskName = "ToastBackgroundTask";

// If background task is already registered, do nothing
if (BackgroundTaskRegistration.AllTasks.Any(i => i.Value.Name.Equals(taskName)))
    return;

// Otherwise request access
BackgroundAccessStatus status = await BackgroundExecutionManager.RequestAccessAsync();

// Create the background task
BackgroundTaskBuilder builder = new BackgroundTaskBuilder()
{
    Name = taskName
};

// Assign the toast action trigger
builder.SetTrigger(new ToastNotificationActionTrigger());

// And register the task
BackgroundTaskRegistration registration = builder.Register();


[Numerics] update CSparse

The version of CSparse in Numerics is very old (2006). This commit updates it.
Nothing much to note, except:

they change their indices to csi, which is defined to ptrdiff_t if we
are not compiling to MATLAB. I kept the name csi when dealing with it
in order to ease the transition if this gets renamed
cs_lusol has its signature (order is now the first arg)
the BIGGEST issue is in the css struct, where they use define lnz and
unz as double, but use them as pointers in the code. By looking at the
function cs_cumsum, it looks like they do that to prevent some
overflow ... well the real fix should be to use size_t, but then the
logic at the end of cs_schol is broken. I left them as csi (which is
like we had before). I don't think we will ever overflow in our case.
Feel free to give our opinion.

conflicts + compil
https://github.com/bremond/siconos/tree/xhub-CSparse-update

Thanks for the review and update. I pushed your branch to master directly to avoid 2 (meaningless merges)


PAPP-30822: Updated requests,lxml modules

According to this thread we can ignore the prodsec failure


Map needs more information

Now we have one problem: while applying map to some sequence, we sometimes need the information that elements of the sequence (say s) to which we are trying to apply our map are really in s. In other words, if we have f : forall x, x \in s -> T we can't just apply it to s via map. Also we can't just check on each step "if the element x is actually in s", because it leads to  O(n^2) complexity after extraction.
We faced this problem in relation.v : https://github.com/volodeyka/event-struct/blob/0c1de9689b54391059204b89b40077735b19182c/relations.v#L45
and in regmachine.v: https://github.com/volodeyka/event-struct/blob/9e64017e0ce3d03fd7f88e0d270d259b26f905ac/regmachine.v#L128
Where it was decided to check if x \in s on each step as a temporary solution.
So think we can try to deal with this problem here. I have got a hand-made solution:
Definition  mem_map {T} {S : eqType} (s : seq S)
  (f : forall x, x \in s -> T) : seq T :=
  let fix mem_map_aux (s' : seq S):=
    match s' with
    | h :: t => fun (sub : subseq (h :: t) s) =>
      f _ (mem_subseq sub (mem_head h t)) ::
      mem_map_aux t (subseq_trans (subseq_cons t h) sub)
    | [::] => fun=> [::]
    end in
  mem_map_aux s (subseq_refl s).

But I suppose it possible to reuse something from mathcomp and not reinvent the wheel or just come up with a nicer solution.

PS. if we decide yo do something like mem_map, we can prove the lemma like:
Lemma mem_map_map T (S : eqType) (s : seq S)
(f : forall x, x \in s -> T) d :
  mem_map s f =
  map (fun x =>
    if x \in s =P true is ReflectT pf then
      f _ pf
    else d) s.

The name mem_map clashes with the lemma from ssreflect/seq.v

The name mem_map clashes with the lemma from ssreflect/seq.v

map_mem then? Or map_in...

I'll comment on this issue a bit later, but one minor observation is that it looks a lot like OCaml's List.mapi function which takes an element and its index within the input list (the index corresponds to the membership proof).

The name mem_map clashes with the lemma from ssreflect/seq.v

map_mem then? Or map_in...

I like map_in, but check for the name clash in vanilla stdlib and ssreflect.

I'll comment on this issue a bit later but one minor observation is that it looks a lot like OCaml's List.mapi function which takes an element and its index within the input list (the index corresponds to the membership proof).

You mean that it is semantically similar, or that we can get the desired behavior using mapi?

So, as @anton-trunov said, using the function with dependent types in big proofs can cause many problems, we should think of another way to deal with this problem. I have some proposals on what we can do in relations.v and regmachine.v:

relations.v

We safely can redo relations.v on map_in because now we use definitely the same thing there as in map_inE lemma: https://github.com/volodeyka/event-struct/blob/90f874c05c9dd5f17d7794d7102cc5890ec790fb/relations.v#L44-L48
map_in will only be here a quick implementation of the code we have now. When we want to prove something, we will simply use map_inE and write the same proof we have now.
As far as the problem is in the casing on x \in ys when we just need x < n, let's case on x < n:if x < n =P true is ReflectT pf then
     s_up_set x
else

if we can decide x < n or not quickly (in natural numbers, for example, we can), this calculation will not lead to O(n^2) on   each step of recursion
But comparing strings or something like that is not so fast and can cause a low speed
We can use Fix. There we have recursion on acc-predicate and perhaps it is quite effective(?)

regmachine.v
The main problem here is that when we want to add something to our event structure, we need to build the add_label record which contains:
https://github.com/volodeyka/event-struct/blob/90f874c05c9dd5f17d7794d7102cc5890ec790fb/transitionsystem.v#L73-L78
Then in regmachine.v we must in the given event structure find special Write-events and put them in add_label.  To statisfy add_write_in_dom in es_seq
https://github.com/volodeyka/event-struct/blob/90f874c05c9dd5f17d7794d7102cc5890ec790fb/regmachine.v#L175-L187
we have to write something like that:
https://github.com/volodeyka/event-struct/blob/90f874c05c9dd5f17d7794d7102cc5890ec790fb/regmachine.v#L155-L156
Where we decide whether for each element y from dom holds y \in dom -- it is a redundant check that leads to the O(n^2) speed.

We can store ffpred and ffrf not as functions from the dom elements to their predecessors (which are in dom too), but as  functions from the dom elements to indexes of their dom's predecessors. Because it is easy either check if given natural number n is the index of an element in dom or to transform it to the index of an element in dom.
In this case, it is quite easy to write write_seq function that by given event structure returns all indexes of it's events satisfying some property in O(n)
Another way to implement write_seq is first to filter dom by wpred x and then to transform it into fset and implement es_seq not as a mapping from the sequence of all events adaptable to the corresponding event structures, but as a set-to-set mapping where we can obtain all needed information from elements type.
BUT as the output we have set of exec_event_structures, so we need to prove that it is a choiceType (it seems like a "try hard")
one more but I don't know if the transformation from seq to fset takes not more then O(n)


[DX Issue] For Headless CLI commands make it easier for users to find the schemas

Amplify CLI Version
10.6.1
Question
If you want to use the headless input for amplify add auth or any other categories the schemas are  not in the docs, there is no  "help" command to see it using the CLI tool.
To find the schemas I had to walk through the code until I found them
https://github.com/aws-amplify/amplify-cli/blob/dev/packages/amplify-headless-interface/schemas/auth/2/AddAuthRequest.schema.json

This is a lot of labour to find basic information. Please uplift this information

40 mins jsut trial end error, examples would be appreciated in the docs...
{
  "version": 2,
  "resourceName": "cruddurauth",
  "serviceConfiguration": {
    "includeIdentityPool": false,
    "serviceName": "Cognito",
    "userPoolConfiguration":  {
      "signinMethod": "EMAIL",
      "requiredSignupAttributes": ["EMAIL"],
      "aliasAttributes": ["EMAIL"],
      "userPoolName": "crudduruserpool",
      "userPoolGroups": [{
        "groupName": "customers"
      }],
      "readAttributes": ["EMAIL"],
      "writeAttributes": [],
      "autoVerifiedAttributes": [{
        "type": "EMAIL"
      }]
    }
  }
}

Hey @omenking :wave: thanks for raising this and apologies for the rough experience! There was a PR addressing the help command that was recently merged and should be available in an upcoming release https://github.com/aws-amplify/amplify-cli/pull/11667


FINERACT-1724: Add disallowExpectedDisbursements flag in Loan account response

Description
Add disallowExpectedDisbursements flag in Loan account API response
FINERACT-1724.
Checklist
Please make sure these boxes are checked before submitting your pull request - thanks!

[ ] Write the commit message as per https://github.com/apache/fineract/#pull-requests

[ ] Acknowledge that we will not review PRs that are not passing the build ("green") - it is your responsibility to get a proposed PR to pass the build, not primarily the project's maintainers.

[ ] Create/update unit or integration tests for verifying the changes made.

[ ] Follow coding conventions at https://cwiki.apache.org/confluence/display/FINERACT/Coding+Conventions.

[ ] Add required Swagger annotation and update API documentation at fineract-provider/src/main/resources/static/legacy-docs/apiLive.htm with details of any API changes

[ ] Submission is not a "code dump".  (Large changes can be made "in repository" via a branch.  Ask on the developer mailing list for guidance, if required.)

FYI our guidelines for code reviews are at https://cwiki.apache.org/confluence/display/FINERACT/Code+Review+Guide.

@josehernandezfintecheandomx Please check the failing tests!


Ship TransactionFetcher panel in Grafana dashboard and fix InsertedTransactions panel bug

Adds a new panel to the reth dashboard in Grafana to track the hashes currently in circulation in the TransactionFetcher. Adds observability to re-fetch logic that is stable since alpha 18.

Fixes units and graph mismatch for InsertedTransactions. Units were fixed at ops/s whereas the graph was moving with __$rate_interval, which can at minimum be 5s. (explanation of bug: https://github.com/paradigmxyz/reth/pull/6650#discussion_r1493768136)

Improves observability of InsertedTransactions by removing override for InvalidTransactions. Grafana already offers the focus change on clicking on graphs in the legend. A zoomed-in version of InvalidTransactions is already available due to this and must not be an override. Removing override also makes proportions of Inserted Transactions and Invalid Transactions visible.

Alternatively, we can keep ops/s by using expressions like $A / 60, and hiding the original graph. However, I can't manage to get rid of the instance and job from the name...json, yay. wdyt?

grafana dashboard version control is a pain

last bug is in tests not in lib will fix in other pr

Alternatively, we can keep ops/s by using expressions like $A / 60

this will not fix it, it would divide every point by 60 which is not correct since it assumes each point represents 60 seconds which is not always the case
i would use prometheus' rate function together with $__rate_interval

Alternatively, we can keep ops/s by using expressions like $A / 60

this will not fix it, it would divide every point by 60 which is not correct since it assumes each point represents 60 seconds which is not always the case
i would use prometheus' rate function together with $__rate_interval

this was already done https://github.com/paradigmxyz/reth/pull/6646/commits/0d419a4acd6334195aee181ce61e92b0619a4c7d


Update release versions for all readme and build files.

@rghetia any idea why build is failing?

@rghetia any idea why build is failing?

C:\projects\opencensus-java\examples\src\main\java\io\opencensus\examples\grpc\helloworld\HelloWorldClient.java:60: error: method usePlaintext in class ManagedChannelBuilder<T> cannot be applied to given types;
            .usePlaintext(true)
            ^
  required: no arguments
  found: boolean
  reason: actual and formal argument lists differ in length
  where T is a type-variable:
    T extends ManagedChannelBuilder<T> declared in class ManagedChannelBuilder
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
1 error

Codecov Report

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

@@            Coverage Diff            @@
##             master    #2028   +/-   ##
=========================================
  Coverage     83.06%   83.06%
  Complexity     2251     2251
=========================================
  Files           319      319
  Lines         10529    10529
  Branches       1048     1048
=========================================
  Hits           8746     8746
  Misses         1446     1446
  Partials        337      337

Continue to review full report at Codecov.

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

@rghetia any idea why build is failing?

C:\projects\opencensus-java\examples\src\main\java\io\opencensus\examples\grpc\helloworld\HelloWorldClient.java:60: error: method usePlaintext in class ManagedChannelBuilder<T> cannot be applied to given types;
            .usePlaintext(true)
            ^
  required: no arguments
  found: boolean
  reason: actual and formal argument lists differ in length
  where T is a type-variable:
    T extends ManagedChannelBuilder<T> declared in class ManagedChannelBuilder
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
1 error

Since grpc v1.26.0, the deprecated API ManagedChannelBuilder.usePlaintext(boolean) is removed. I have removed removed arguments, hopefully build should pass now.


feat: add clientIdentifier option to createServer

This PR is a follow-up to #190. It fixes a couple of issues that were present on that branch, cleans up the commit history, and removes the added package.json file.

Pull Request Test Coverage Report for Build 974909872

15 of 16 (93.75%)  changed or added relevant lines in 1 file are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.09%) to 90.016%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/server.js
15
16
93.75%

Files with Coverage Reduction
New Missed Lines
%

lib/cache.js
2
95.45%

Totals

Change from base Build 972789947:
-0.09%

Covered Lines:
1166

Relevant Lines:
1259

💛  - Coveralls

@Apollon77 Could you maybe have a quick look on this one? :) Maybe it could be integrated into the 0.25.0 release.

Sure, I ran a bit outt of sync with some tasks ... will check tonight

will check tonight

Great, thank you!

@JKRhb Basically ok, I left you one comment for the test question ... what you think?

@JKRhb Basically ok, I left you one comment for the test question ... what you think?

Thank you :) I replied with a naming suggestion :)

accepted ... lets formally have test running and then have fun merging it :-)

accepted ... lets formally have test running and then have fun merging it :-)

Awesome :) Thanks for the review!

Thanks for the review!

yw


merge from arlac77/template-arlac77-github,arlac77/template-esm-only

package.json

chore(package): add ^7.1.1 remove ^7.1.0 (devDependencies.c8)

.travis.yml

chore(travis): (jobs.include)
chore(travis): add 14.1.0 (jobs.include.node_js)

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


PR failing due to style issue: PR uses single quotes yet Brew Cask CI wants double quotes

Used latest cask-repair (v. 0.40.22) to submit an update for cask anki and the PR CI checks are failing due to single quotes and tabs being used.
Not sure whose style to go by here - there's or cask-repair's.
PR submitted: https://github.com/Homebrew/homebrew-cask/pull/86658/files

Pasted diff of the PR as well
--- a/Casks/anki.rb
+++ b/Casks/anki.rb
@@ -1,19 +1,19 @@
-cask "anki" do
-  version "2.1.28"
-  sha256 "c4eec38d80804fcd9e82d24a4ee5d4fbc581b8f5de1e1f08056f80b4bef60a2d"
+cask 'anki' do
+  version '2.1.29'
+  sha256 'b3fc61553b62c014d979ad738c8eca1713a9fc9dcc8108203063b181ace4ba31'

   # github.com/ankitects/anki/ was verified as official when first introduced to the cask
   url "https://github.com/ankitects/anki/releases/download/#{version}/anki-#{version}-mac.dmg"
-  appcast "https://changes.ankiweb.net/README.md"
-  name "Anki"
-  homepage "https://apps.ankiweb.net/"
+  appcast 'https://changes.ankiweb.net/README.md'
+  name 'Anki'
+  homepage 'https://apps.ankiweb.net/'

-  depends_on macos: ">= :sierra"
+  depends_on macos: '>= :sierra'

-  app "Anki.app"
+  app 'Anki.app'

   zap trash: [
-    "~/Library/Application Support/Anki",
-    "~/Library/Application Support/Anki2",
-  ]
+               '~/Library/Application Support/Anki',
+               '~/Library/Application Support/Anki2',
+             ]
 end

cask-repair uses the same command to fix the style, so this will eventually sort itself automatically.


docs: simplify neovim glimpse.menu shortcut

Hey @MarcoIeni! I found this to work, and it seems simpler.

Thanks!

Thanks @MarcoIeni!


Create a contact info block

Add links to social networks (LinkedIn, Facebook, Instagram), phone number, email, Telegram username, etc.

[contact info]https://github.com/AdonisJr/homepage/pull/11/commits/6533f3870288839f63fa50a9c676d663b10d165d


Link to Linux documentation 404s.

If you click the link to Linux documentation, it is a dead end that results in a 404 notification. The URL it points to that fails is:
https://docs.microsoft.com/en-us/microsoft-365/security/defender-endpoint/microsoft-defender-atp-linux

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: e914c02a-e905-fc32-439c-feb1fafa09b8
Version Independent ID: 1a6a15e9-717a-33c1-b90e-96925c680228
Content: Microsoft Defender for Endpoint for non-Windows platforms
Content Source: microsoft-365/security/defender-endpoint/non-windows.md
Product: m365-security
Technology: mde
GitHub Login: @mjcaparas
Microsoft Alias: macapara

It looks like all of the links to documentation for Linux, MacOS, Android and IOS are ALL BROKEN.

PR already checked in to fix this.


Transparent background not working for some icons

Describe the bug
I would like to use the icons with transparent background like this:
	<SocialIcons
		network='github'
		fgColor='blue'
		bgColor='rgba(0,0,0,0)
	/>

For most of the icons it works! But for stackoverflow it does not work.
	<SocialIcons
		network='stackoverflow'
		fgColor='blue'
		bgColor='rgba(0,0,0,0)
	/>

Github and stackoverflow next to each other:

Is there another way to make the background transparent?
Or is this a real bug with this icon?

Hi Máté,
Sorry for the late reply.  I think this issue comes from the way the icon is defined.  If I swap fgColor and bgColor (for stackoverflow only), I get this:

Is that what you want to achieve?
This is the Svelte code:
<SocialIcons
network='github'
fgColor='blue'
bgColor='transparent'
/>

<SocialIcons
network='stackoverflow'
fgColor='transparent'
bgColor='blue'
/>

Thanks for taking an interest in the project.  Let me know if can help more.

Thank you for the help.
This is what I needed.


fix: body font not being picked up by preview components in DSC app FUI-1432

Review Guidance
📓   Related JIRA
FUI-1432
🤔   What does this PR do?

updates the body font value to the correct value used for Roboto Font family everywhere

🚀   Where should the reviewer start?

review changes in src/styles/design-tokens.json

📑   How should this be tested?

npm run bootstrap in client directory
npm run configure-design-system in client/web director
Once you have the DSC app running check if the Roboto family font is being applied as shown in above image.

npx -y @genesislcap/genx@latest init prtest -x --ref YOUR-BRANCH-NAME

✅   Checklist

[x] I have tested my changes.
[ ] I have added tests for my changes.
[ ] I have updated the project documentation to reflect my changes.

Looking good:


When doing a research, able to scroll horizontally with results (no need to go to previous page)

In front office. When we do a research (example), results appear and if we click on one of the book (example), it would be practical to access the book before and after the current one with left and right arrows. You can find a picture to illustrate my idea, hope it helps.

decided to test opening in a new tab by default


[CHANGE ME] Re-generated apis/Grafeas.V1 to pick up changes in the API or client library generator.

This PR was generated using Autosynth. :rainbow:
Synth log will be available here:
https://source.cloud.google.com/results/invocations/82b58ea7-174f-4260-9a09-b2b4e1762539/targets

[ ] To automatically regenerate this PR, check this box.

Closing - we don't want this; we need to tweak the scripts so it's not included.


How Do I Contact the Official Crypto.com Customer Support Number?

How can I recover my Crypto.com account?
If you're looking to recover your Crypto.com account, you can reach out to their customer support team by calling +1-(206)-683-0106 or +1-(206)-683-0106 (USA) for help with the account recovery process.
How to contact Crypto.com Manager?
To connect with a Crypto.com manager, you can use the live chat feature by selecting the "Contact Us" option ☎+1-(206)-683-0106 in the Help Center, and then choosing "Chat."
For urgent concerns, Crypto.com provides phone support as well. Simply call them at ☎+1-(206)-683-0106 for a quick response and assistance. This is one of the fastest ways to speak to someone from Crypto.com.
Does Crypto.com offer live chat?
Yes, Crypto.com offers a live chat option for customer support. To use the live chat:
Visit Crypto.com’s Help Center on the platform.
Click on the "Contact Us" ☎+1-(206)-683-0106 option.
Select "Chat" to begin a conversation with a live support agent.

Alternatively, you can call Crypto.com at ☎+1-(206)-683-0106 (available 24/7) or +1-(206)-683-0106 (USA) for immediate support.
You can also reach Crypto.com support by calling ☎+1-(206)-683-0106 to speak directly with a human representative.

Closed as spam issue.


Bumps version to 1.1.0

@plus3it/watchmaker tested the new yum repo for salt 2018.3.4 and it is working. this is ready to release...


Update dependencies

Purpose
Update dependencies using ] up
Content
Solution implemeneted:

Updated dependencies using ] up

[x] I have read and checked the items on the review checklist.

bors r+


Don't add fullname tag in 2GP package.xml

Fixes #3415

@leboff I'm ... very surprised that this works, but I should have long since given up being surprised by the need for such tweaks!
A few questions for you, as I think I will probably need to discuss this change with the Packaging team to fully understand the impact and expected behavior.

What kind of 2GP package are you building? Managed, unlocked, org-dependent? Namespaced?
When you built the package with this change, what happened? Did you obtain a fully viable/installable package version, with the problem component (a Quick Action, I believe?) included?
Have you ever promoted a 2GP package version built with this change?

@davidmreed thanks for looking into it. Answers below

What kind of 2GP package are you building? Managed, unlocked, org-dependent? Namespaced?
Managed + Namespaced

When you built the package with this change, what happened? Did you obtain a fully viable/installable package version, with the problem component (a Quick Action, I believe?) included?
Yes

Have you ever promoted a 2GP package version built with this change?
Yes, I use a forked version of CCI with this change and have close to 10 2GP packages promoted and in many production environments with this change applied

@leboff Fascinating. I'll check in with my colleagues on Packaging to help me understand what's happening here. Using <fullName> naively should be correct. I'll also compare and contrast with SFDX. On a quick review it appeared to me that SFDX was using it.
There is a related, known bug where packaging a CustomObject subentity in a 2GP that is owned by another package fails. I wonder if this behavior is related, and the distinction you're seeing is an emergent interaction between a couple of different edge cases!
One last question - are your repos in SFDX metadata format?

Sorry, further last question: have you ever tried to upload a package version with this structure using SFDX directly? If so, what behavior did you see?

Sorry, further last question: have you ever tried to upload a package version with this structure using SFDX directly? If so, what behavior did you see?

I think that's how I found the issue. Comparing the zip sfdx CLI used (which worked) and the zip CCI used (which did not)

Thank you again for the exceptional detail and research here!

Please fix this. While nice to name your packages, it completely breaks any ability to deploy tweaks or updates to metadata that is in a package.
Interestingly, the same bug/limitation also affects change sets.

I am currently running into an issue where this is preventing me from creating new 2GP Unlocked package without a namespace. My package has a quick action in it that references a namespaced (2gp unlocked also) lwc and it fails with
Can’t add component MED_Translate_Interaction because it’s associated with metadata from a managed 2GP package. Managed 1GP packages can’t depend on  managed 2GP packages.
Please merge and fix this

Fixed in #3636

@jstvz sorry for commenting on this closed PR. We are currently experiencing this issue as well and it is citically blocking our progress while developing 2GP packages.
Context:

2GP package A contains: Apex, Custom Metdata Types and Records, Layouts and Triggers
2GP package B references package A and some of its content
While creating a beta package for package B (flow: release_2gp_beta, task create_package_version) we get:
Can’t add component xyz because it’s associated with metadata from a managed 2GP package. Managed 1GP packages can’t depend on  managed 2GP packages.
When I create it via sfdx, it works without issues

We understand there might be larger implications of @leboff 's fix, but we are not able to move forward while the issue persists.
We would really appreciate if you could take another look at this, especially considering @leboff 's mentioned track record of successful package creation and deployment.
Thanks in advance!
mentioning @winterrk and @yippie to check-in if they still experience the issue as well.

@pasres I've already opened #3748 as I'm still experiencing the issue as well. I currently use a forked version to get around the issue in our packages.

@leboff thanks for your answer, the initial effort to figure it out and the new PR! Really appreciate it.


Initial pass at forEach and forEachPar in Async

This is an initial pass at adding forEach and forEachPar variants into the Async module in order to more closely resemble the api of effect. This would be a breaking change.
Not expecting this to be a final draft but I wanted to get your thoughts before I spent any more time on it.

Seems reasonable! I wanted to add a note, the reason Async is encoded initially (as a tagged union of primitives) is to have stack-safety but after encoding it I realised it is possible to get safety by using microtasks to suspend potentially recursive calls in a final encoding in the same way as I did in fp-ts https://github.com/gcanti/fp-ts/pull/1591
As a background Async exists mostly for bundle-size constrained environments where Effect may be overkill and it may be good to "re-encode" the type to be final so we aren't forcing all the primitives in the bundle

Thanks for the note! I convinced my team (they would probably say forced) to use @effect-ts in our project and are trying to keep it to just Sync and Async on the frontend. I am a huge fan of this project and Zio as well and would love to help it move forward in whatever way I can.
I actually took a look at your fp-ts PR as well. I personally think there is value in the initial encoding beyond stack safety. I think there is a clearer implementation of effect interruption and environment. Primitives that can be run in sync are done so without putting each within a microtask. And the entire implementation is only a few hundred lines of typescript as well. We are talking about a pretty small difference in final bundle size.

I was unclear, what I meant is using final encoding preserving environment & interruption, I would agree with you on the style, initial encodings are nicer but they do take space, Async is a few hundreds lines of code this is true but adds up to roughly 15/20k where Effect itself if not used with managed & layers is just shy of 35k if we encode it finally I believe we could get it down to a few kb.
Lovely to hear you are adopting :) and looking forward to build it up together!

The initial encoding of Async by itself is only ~4.5kb minified (non-gzipped) as it stands today. import-cost in VSCode actually tells me 3.7kb when I import only runPromiseExitEnv which would include all the primitives. This PR would reduce it slightly by removing the IAll primitive and corresponding runtime code in runPromiseExitEnv.
I have the entire, non-treeshaken Async module sized at 27.4KB before compression.
Smaller would obviously still be better though :)

Then my brain has fused at some point... It was way bigger I must have optimized and forgot... :) well ignore.

All good, just didn't want you to spend time trying to improve things at the margin. My guess is you have bigger fish to fry.

Shall we stage this? I think it's a good basis to start


adding friends spec

Is the name friends fixed in stone? Seems something like 'user notes' might be a more intuitive name given the dominance of the facebook style mutual friendship system.

@elwoodpd We anticipate growth of the Friends aspect of the Gazelle beyond the functionality outlined in the spec in the future.

Be sure to check the brainstorm Trello board for approved friends features that aren't currently in Gazelle.

@guitarman0831 He doesn't have access. I already looked at it. It needs to be finalized and will be included in the future.

@ajax-wcd ah, I thought this was the BT flaming. Apologies.

@guitarman0831 it is me ;)


Focal Manipulator can't craft Projectile Foci

Hi! Eventough i have all the requirements, i still can't craft Projectile foci.

You have zero levels of XP.  The gauge says one but you will likely need 3 or 4 levels.

It's the same when i have levels.

Okay, next make sure your projectile has an elemental focus.  In the picture, you are only creating a projectile, but there's no focus telling it what that projectile should do.  Should it burn the target, freeze it, ignore armor, break blocks, create a rift, or what?

I have this same problem, but I'm doing everything correctly.

Check out this mod:
https://www.curseforge.com/minecraft/mc-mods/thaumic-fixes

Can confirm that the levels are updated when crafting more complex foci.  Looks like a cool fix mod, hope it gets even more traction and fixes.


Use LazyFormattedBuildEventArgs in ProjectImportedEventArgs

This ensures that as few strings are allocated as possible

@dotnet-bot test this please

@dotnet-bot test this please


#71: Merge architecture interoperability and compatibility into interoperability

This closes #71.

Reopened the PR for merging


Angular JS 1.3.20 doesn't work

It seems this module won't work with 1.3.20, I didn't receive any error but the slides property won't get filled. So I always ended up with an empty carousel.
I tried several things also to log from the module but it seems slides array remains always empty.

Same here. In the demo codepen, if one replaces the Angular with this https://cdnjs.cloudflare.com/ajax/libs/angular.js/1.3.19/angular.min.js
it is not initialized.

I can confirm there is a problem with AngularJS < 1.4
I just update my AngularJS to 1.4.14 and the ui-carousel now works fine.

@michelem09 Fixed on 822763c

Thanks but I can't test it anymore. I'm closing it.


Make git status output cleaner

Added useful .gitignore lines, plus .gitmodules ignore=isdirty for googletest submodule, which generates files within. These make my git status only include lines that are things I've modified.
Additions

Removals

Changes
.gitignore and .gitmodules changes
Testing
[No product code changes]
Screenshots
Notes

Todos

Checklist

[X] PR has an informative and human-readable title
[X] Changes are limited to a single goal (no scope creep)
[X] Code can be automatically merged (no conflicts)
[X] Code follows project standards (link if applicable)
[X] Passes all existing automated tests
[X] Any change in functionality is tested
[X] New functions are documented (with a description, list of inputs, and expected output)
[X] Placeholder code is flagged / future todos are captured in comments
[X] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)
[X] Reviewers requested with the Reviewers tool :arrow_right:

Testing checklist (automated report can be put here)
NA
Target Environment support

[X] Linux

sigh . Comedy of errors here. The change to CMakeLists.txt is still sitting in my working copy. Also I think I branched poorly. But particularly I misdiagnosed why the "Build and Run NGEN" action had failed... local RHEL has Python 3.6.8, the container has 3.8.6 (transposition). I will push the change still because it should fix the build on RHEL7 mainline. I think it should still merge fine if merged after #274 .

sigh . Comedy of errors here. The change to CMakeLists.txt was still sitting in my working copy. Also I think I branched poorly. But particularly I misdiagnosed why the "Build and Run NGEN" action had failed... local RHEL has Python 3.6.8, the container has 3.8.6 (transposition). I pushed the change still because it should fix the build on RHEL7 mainline. I think it should still merge fine if merged after #274 .

Just as a note: First Time contributors need a Maintainer to run the workflows for them. Please check the code in the review for changes to workflows and then run them for new folks when they come in.


Unc0ver isnt supporting my device.

Describe the bug
Opening the app displays an unsupported screen yet I was told my device type and version was supported.  Nothing works!
I have an A8 Core, Im on 12.1.4, im on a relatively new ipad, so why isnt it supported??
To Reproduce
Steps to reproduce the behavior:

Open Uncover on my device type and version

Screenshots

Device (please complete the following information):

iOS Version: 12.1.4
iOS Device: iPad Mini 4
unc0ver Version: 3.3.2

Place an "x" between the brackets if true:

[maybe] this is a bug others will be able to reproduce
[x] this issue is present with all tweaks uninstalled(except for default packages) or disabled
[x] this issue is present after a rootfs restore
[x] this issue is present on the latest version of unc0ver

Logs
If applicable, add logs or error messages here.

Yea I'm on the iphone xr and I'm on version 12.1.4 and the same thing happened to me

For unc0ver not support for A7-A8/A8x and A12.
sockpuppet exploit for now can support for A9-A11 and it's not bug.
Pwn& team has been working for fix sockpuppet to work on A7-A8/A8x.

is the iphone xr supported because thats what I'm on and its saying unsupported

@Qweeezye read again my comment "it's not support for A12 devices"
A12 devices = XS/XR/XS Max

are the A12 devices ever going to be supported

are the A12 devices ever going to be supported

No, unless unc0ver takes the same exploit of Chimera.
I am sorry but you can only use Chimera for A12 now. 😕


Project loading should happen asynchronously

Primarily, calling scarb metadata should be an asynchronous process that does not block project model updates like it does currently.
Requirements

This is a good opportunity to drop the detect_crates_for method and replace it with a proper abstraction. See the description for old issue #6124 for more clues.
Implement this in such a way, that when doing #6132 we won't have to change a lot here. Primarily, this means that this system should escape async world as early as possible. Use spawn_blocking like we attempt to do in with_db method.
Avoid calling project refreshing process (scarb metadata, cairo_project.toml parsing) if another instance is already running (but if one is running: always schedule at most 1 extra run to pick any changes that could happen while the previous run was running).
Gracefully deal with projects which loading fails. In such cases, avoid clearing relevant salsa inputs to preserve as much cached queries as possible. For example, a syntax error in the manifest file (this will make scarb metadata fail) shouldn't cause wiping of all analysis state. Check out what RA does here.
Handle detached files well (i.e. one without matching Scarb.toml/cairo_project.toml well). Do the same as Rust Analyzer does.

Gracefully deal with projects which loading fails. In such cases, avoid clearing relevant salsa inputs to preserve as much cached queries as possible. For example, a syntax error in the manifest file (this will make scarb metadata fail) shouldn't cause wiping of all analysis state

@mkaput  this is no longer the case, no? We just try to init unmanaged corelib in case metadata fails now

This is more about recovering from breakage when metadata did succeed already. Take a following scenario as an example:

You open OpenZeppelin/cairo-contracts
It analyses successfully.
You open Scarb.toml and make a syntax error.
scarb metadata will now refresh and it will fail

I don't want to lose all the analysis information due to this problem because it is easily recoverable: if we know something about the code already, and this information is reliable, let's persist that information instead of wiping it.


Pathfinding

Basic pathfinding has been implemented. Needs refactoring though and there are still some bugs to iron out/cases that can't be handled.


LocalCollection issue

When I changed a field in the review database in Firestore (and obviously changed it in the code as well) the app crashes. In the log window it says: unable to initialize type Review with dictionary [...] (in this dictionary it still shows the old field (although it does not exist in the database anymore!). I'm really stuck..Seems like the app is storing the old data somewhere locally and it is not pulling out the new data. Can someone help me please?

Hey, there's a number of things that could be happening here.

Firestore caches values locally, so if you've changed the validation logic the cached values will become unserializable and LocalCollection will crash. You can fix this by reinstalling the app.
Firestore is downloading a value from the backend that's unserializable and causing LocalCollection to crash. You can fix this by deleting the bad keys/values in the database.

LocalCollection checks for the presence of all the keys it needs to initialize a Review object, so any extra keys are ok and will be ignored. Can you post the error message?

Thank you for your response Morgan. I don't think reinstalling the app is a good idea although I think it might work and here is why: let's suppose I launch the app and I decide to change some stuff in the database, I can't ask the user to reinstall the app when that happens....
The error is: Unable to initialize type Review with dictionary ["restaurantName": Paris, "restaurantID": 0, "UserInfo": {
Name = Steph;
PhotoURL ="https://firebasestorage.googleapis.com/v0/b/restaurant-31c3f.appspot.com/o/eater.png?alt=media&token=7e95c1fb-f402-4611-90d4-cb9c2362fbff";UserID = X1;
}, "Opinion ": "Love it"]
and this is the old data that was in Firestore since I changed "Opinion " by "feedback"

You're right, the LocalCollection class is deliberately simplified for codelab purposes. In a real app, you'd want to relax the validation logic to not crash on bad data and instead just discard it. You can do so by modifying the logic here.

I'm sorry but I did not understand how to resolve this...Should I change something in the code?

For example, you can change the declaration of models to this:
let models = snapshot.documents.map(T.init(document:))
    .filter { $0 != nil }
    .map { $0! }

This will ignore any values that cannot be serialized instead of crashing. To improve this code, you might consider printing any dictionary that fails to serialize into a review.

So I did this and I printed models to kinda see what is inside and it is empty...I feel I'm not getting the data from Firestore :(((((

Are you seeing any Firestore errors in the logs? You can print the raw data from Firestore by adding a print(snapshot.documents) before the let models declaration.

when I did that I have this in the logs: [<FIRQueryDocumentSnapshot: 0x600003a14e60>]

Looks like you do have a valid snapshot there, so you're successfully downloading data from Firestore. It's not being serialized because the keys it expects are different on the client.
What's your goal in changing the name of the key in Firestore?

I think I'm getting there..Thanks to you Morgan. Is it ok if the result of the print(snapshot.documents) is [<FIRQueryDocumentSnapshot: 0x600001246530>] and the result of print(models) is [Restaurant.Review(Here it has the data from the database)]? Is it how it is supposed to be or does the print(snapshot.documents) need to be something else?

Yep! You can print the data in the Firestore snapshot with
print(snapshot.documents.map({ $0.data() })

to see what the keys/values are.

It works!

Awesome, glad I could help!

Hi Morgan, it's happening again(the old data problem....) I just implemented the "write a review" feature and its working nicely but if I go in Firestore and delete some review documents, the deleted ones are still showing in the app...

Can you put a breakpoint in the snapshot listener to see if it's being invoked? The line responsible for updating the app data is here.

It is being invoked.
Here is the code That I'm using:

    let models = snapshot.documents.map(T.init(document:))
    .filter { $0 != nil }
    .map { $0! }

    self.items = models
    print(models)

    self.documents = snapshot.documents
    print(snapshot.documents)
    self.updateHandler(snapshot.documentChanges)

Looks good. Once the Firestore changes are synced to the LocalCollection, it's up to the LocalCollection's update callback to push those changes to the view layer. Your RestaurantDetailViewController.swift should look like this.
  private var query: Query? {
    didSet {
      if let query = query {
        localCollection = LocalCollection(query: query) { [unowned self] (changes) in
          if self.localCollection.count == 0 {
            self.tableView.backgroundView = self.backgroundView
          } else {
            self.tableView.backgroundView = nil
          }
          self.tableView.reloadData() // Updates the table view
        }

        dataSource = ReviewTableViewDataSource(reviews: localCollection)
        localCollection.listen()
        tableView.dataSource = dataSource // Connects the table view to the LocalCollection
      } else {
        localCollection.stopListening()
        dataSource = nil
        tableView.dataSource = nil
      }
    }

yes exactly like that... I really don't know why I'm still getting the new data... I don't know how to fix it. I looked online they said add this :
let settings = FirestoreSettings()
settings.isPersistenceEnabled = false
db.settings = settings
But I'm not sure where to add it... I tried the app delegate. swift but it is not working...

Can you try putting a breakpoint here and seeing if the method is invoked?

It is being invoked...

Are you getting any errors in console?

The app is not crashing but I get these 2:
Listen for query at Reviews failed: The query requires an index. You can create it here:
and
Error fetching snapshot results: Error Domain=FIRFirestoreErrorDomain Code=9 "The query requires an index. You can create it here:

I added a composite index and it's working!

Awesome, glad you got it working. I'll close this issue again, but feel free to continue commenting or create a new issue if you run into any more hangups.

Good morning! I am a beginner trying to participate in an app development. Downloaded your app and added some changes, but receive the above message. I am also receiving the above message, but could not implement the corrections stated. Mine is crashing at the RestaurantsTableViewController site. This is how it originally looked:

I also printed out the above array and I am getting back the new data from the database, so it is fine.
I tried to rewrite it to one of the above solutions, so that it does not crach:

But this is not where it crashes anyway.
Any help would be very much appreciated.
Thank you!

@patayg76 Firestore will cache data from a previous database load, so if you've downloaded bad data once the app will continue to crash as it loads the bad data from cache. It looks like the dictionary is malformed, so once you've confirmed the data in the backend is correct you should delete (uninstall) the app from simulator or device to clear the cache.

Thank you! Deleted the app both on the device and the simulator. Rebuilt the app and got the same results. Even deleted the database, so that I could start with repopulate. No luck...

I found the problem and I would like to find a way around. Since I am trying to rewrite the database, whenever I introduce new data, I need to delete everything. Would it be possible to not allow to get ALL fields into the query, just the ones that I need? Where could I set the fields? Thank you!

You can't selectively download just a few fields from each document, but you can ignore the fields on the client once they're downloaded. See the initializer here--any fields outside of the ones defined in the initializer will be ignored.

Thank you! I will try to implement it


Delete unused servlets

PR Checklist
Please check if your PR fulfills the following requirements:

[x] PR has a descriptive title
[x] PR contains closely related changes only (should not have many unrelated changes)
[x] STEP pod members have been added as reviewers
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[x] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Deletes all unused servlets.

Please also remove configuration for /sample that is in the web.xml file.


Export type DefineComponent

What problem does this feature solve?
Most Vue 2 third-party libraries do not provide types for components. Therefore, they cannot profit type checking and intellisense provided by Volar. It may be good to export type DefineComponent just like in Vue 3, so that users can manually shim component types or create PR to add TypeScript support for those Vue2 libraries without changing source code written in Vue <= 2.6.
What does the proposed API look like?
To manually define type of a component in Vue 2.7, we can:
import type { DefineComponent, VNode } from 'vue'

declare module '@vue/runtime-dom' {
  export interface GlobalComponents {
    MyComponent: DefineComponent<
      Props,
      {
        $scopedSlots: {
          slotName: ({args: {propName: propType}}) => VNode[]
        }
      },
      {},
      {},
      Methods
    >
  }
}

Does this have a solution?


Remove hard-wired use of NC_MAX_DIMS

Replace static array dimensioned to NC_MAX_DIMS with dynamic memory allocation of correctly sized array.

I was in the process of making the same change, but this is equivalent, so we can
use it.

Thanks, hope I didn't waste too much of your time.  I saw this while looking at the NX_MAX_DIMS issue and decided that it was probably a non-controversial change and might buy me some karma to use later ;-)
..Greg
--
"A supercomputer is a device for turning compute-bound problems into I/O-bound problems"
From: DennisHeimbigner <notifications<EMAIL_ADDRESS>Reply-To: Unidata/netcdf-c <reply<EMAIL_ADDRESS>Date: Wednesday, November 11, 2015 at 2:08 PM
To: Unidata/netcdf-c <netcdf-c<EMAIL_ADDRESS>Cc: "Sjaardema, Gregory D" <gdsjaar<EMAIL_ADDRESS>Subject: [EXTERNAL] Re: [netcdf-c] Remove hard-wired use of NC_MAX_DIMS (#150)
I was in the process of making the same change, but this is equivalent, so we can
use it.

Reply to this email directly or view it on GitHubhttps://github.com/Unidata/netcdf-c/pull/150#issuecomment-155881056.

Looks good so far; running regression tests via Docker unidata/nctest images to rule out any unforeseen impacts on the fortran, C++ or python interfaces.


Hope to add features: one-click exit aria2c and close ariang page

After the download is complete, you can exit aria2c and close the ariang page with one click.
Exit button can be placed on top toolbar

I think this feature is not a widely used feature and should not be added in the AriaNg. You can modify it by yourselfif you need.


Remove line breaks from sample data

In order to be able to see the real layout and not some weird line breaks when testing the platform.

Coverage remained the same at 100.0% when pulling 0537c3efe822d8b3d24ab83d73ac087691027507 on cleanSampleData into 760a5036dd76e4a287707a07a8fa478227c795b3 on dev.

Coverage remained the same at 100.0% when pulling 7520177b0ba12b2ff1c80dc6778aa293362792b9 on cleanSampleData into e058c575315973e91984fa066e8b79ffcc7d5106 on dev.

I didn't know that @cmfcmf. Id still tend to this version, because people can turn soft wraps on in their editor. With the other one I'd have to decide for everyone how long the lines should be. But very interesting for other cases!


[improve][cli] pulsar cli: use return code 1 for general error

Return code -1 is generally considered to be an invalid code. https://tldp.org/LDP/abs/html/exitcodes.html
Replace return code -1 with 1 which is commonly used as a general error.
Fixes #18945
Motivation
There are a lot of cases in the pulsar cli tools where a -1 is returned as the return code when there is an error executing the command.  This is generally considered an invalid return code and the code should be an integer between 0 and 255, with 1 commonly used as a general error code when specific error codes are not defined.  Because -1 is invalid, running a pulsar command which fails will actually return a 255 because 255 is used to indicate that the return code is out of range.
Modifications
Updated the CLI commands to return 1 instead of -1 when there is an error.
Verifying this change

[X] Make sure that the change passes the CI checks.

This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:
If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[ ] The REST endpoints
[ ] The admin CLI options
[ ] Anything that affects deployment

I'm not sure if there are other applications which depend on the current specific return codes, but it's doubtful that there are many cases.  Normally, scripts just check for a non-zero return code and don't look specifically for a 255.  If there are a significant number of users that depend directly on the 255 return code, then we might have to make this change more gradually.
Documentation

[ ] doc
[ ] doc-required
[X] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository: https://github.com/pgier/pulsar/pull/7

/pulsarbot run-failure-checks

Codecov Report

Merging #18946 (faf59dc) into master (3011946) will decrease coverage by 12.29%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             master   #18946       +/-   ##
=============================================
- Coverage     46.75%   34.45%   -12.30%
+ Complexity    10514     6594     -3920
=============================================
  Files           703      623       -80
  Lines         68858    59106     -9752
  Branches       7383     6147     -1236
=============================================
- Hits          32193    20365    -11828
- Misses        33065    36016     +2951
+ Partials       3600     2725      -875

Flag
Coverage Δ

unittests
34.45% <0.00%> (-12.30%)
:arrow_down:

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

  All the tests passed in my forked repo.


Describe the bug
When using "tns run android" after making a change to the project, livesync causes errors both on an emulator and in an android device. Although the errors are different, they appear to be related. After livesync on a device, the ECONNREFUSED error occurs. On the emulator, I receive:
"System.err: java.io.IOException: Broken pipe
System.err:     at android.net.LocalSocketImpl.writeba_native(Native Method)
System.err:     at android.net.LocalSocketImpl.access$500(LocalSocketImpl.java:36)
System.err:     at android.net.LocalSocketImpl$SocketOutputStream.write(LocalSocketImpl.java:144)
System.err:     at android.net.LocalSocketImpl$SocketOutputStream.write(LocalSocketImpl.java:131)
System.err:     at com.tns.NativeScriptSyncServiceSocketImpl$LiveSyncWorker.closeWithError(NativeScriptSyncServiceSocketImpl.java:424)
System.err:     at com.tns.NativeScriptSyncServiceSocketImpl$LiveSyncWorker.run(NativeScriptSyncServiceSocketImpl.java:187)
System.err:     at java.lang.Thread.run(Thread.java:764)"
Both cause webpack to stop running.
To Reproduce
I do not know how to reproduce this error.
Expected behavior
Livesync should work to refresh the project.
Sample project
This is a very large project, and would be difficult to post to the playground.
Additional context

Upon further investigation, I discovered the typescript version I was using was one version too high for the latest release of Angular. I also found that the nativescript version 5.2.0-next-2018-12-20-172434-01 was not compatible with some plugins. I downgraded typescript to 3.1.1 and nativescript to 5.1, and have successfully saved code changes without livesync stopping. Before closing this issue, I'd like to test some more.

Back on the emulator, livesync appears to work as long code changes are minor. For example, changing code as to how an event is handled caused no error. However, after adding a component or an element, for example, I just added a @ViewChild and ElementRef requiring the imports in the component to change resulted in "Sync operation in progress..."

I have simplified and added this issue to the Nativescript Angular issues repository.


Update os-lib_2.13 to 0.7.4

Updates com.lihaoyi:os-lib_2.13 from 0.6.3 to 0.7.4.
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 = "com.lihaoyi", artifactId = "os-lib_2.13" } ]

labels: library-update, semver-minor

Superseded by #23.


TTNN enable fused RELU in RN50 (new api version)

Currently its being inserted as additional unary op.

resolved: https://github.com/tenstorrent/tt-metal/pull/8919


Plot method ignores upper and lower error bar arguments

This seems to be specific to Pandas as opposed to matplotlib.  Worked as desired as of a few months ago when I last updated from master.
This is the desired behavior in matplotlib:
barh([0,1], [.3, .5], xerr=[[.1,.2], [.3,.4]],
     color = ['red','green'],
     ecolor='black')

In the Pandas call, it seems to ignore the second error bar argument, both the upper and lower bounds of the error bar are the same:
s = pd.Series([.3,.5])
s.plot(kind='barh', xerr=[[.1,.2], [.3,.4]],
       color = ['red','green'],
       ecolor='black')

closing as stale. pls reopen if still an issue.


Switch from structopt to clap

Not intended to review the code, but I have a question: what's the motivation to change structopt to clap. I know little about them, and they seem to have the effects to me.

Not intended to review the code, but I have a question: what's the motivation to change structopt to clap. I know little about them, and they seem to have the effects to me.

It is essentially just an upgrade. The structopt crate was deprecated and its functionality was merged into clap v3. The specific reason to upgrade now is that clap has a new feature that will help with https://github.com/fermyon/spin/pull/444


Can't compile simple example

bot1.ts:
import { Bot } from "grammy";

» tsc bot1.ts
../../node_modules/@types/express-serve-static-core/index.d.ts:584:18 - error TS2430: Interface 'Response<ResBody, Locals, StatusCode>' incorrectly extends interface 'ServerResponse'.
  Property 'req' is optional in type 'Response<ResBody, Locals, StatusCode>' but required in type 'ServerResponse'.
584 export interface Response<
                     ~~~~~~~~

../../node_modules/@types/express/index.d.ts:58:55 - error TS2344: Type 'Response<any, Record<string, any>>' does not satisfy the constraint 'ServerResponse'.
  Property 'req' is optional in type 'Response<any, Record<string, any>>' but required in type 'ServerResponse'.
58     var static: serveStatic.RequestHandlerConstructor<Response>;
                                                         ~~~~~~~~

../../node_modules/grammy/out/core/client.d.ts:1:23 - error TS2688: Cannot find type definition file for 'node-fetch'.
1 /// <reference types="node-fetch" />
                        ~~~~~~~~~~
Found 3 errors.

» tsc -v
Version 4.4.4

» node -v
v17.0.1

What's wrong with my config?

My fault. This fixed the issue:
npx tsc --init
npx tsc


yt-dlp.exe (2023.06.22 release) detected as trojan

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm asking a question and not reporting a bug or requesting a feature
[X] I've looked through the README
[X] I've verified that I'm running yt-dlp version 2023.06.22 (update instructions) or later (specify commit)
[X] I've searched known issues and the bugtracker for similar questions including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Please make sure the question is worded well enough to be understood
Microsoft Defender is detecting https://github.com/yt-dlp/yt-dlp/releases/download/2023.06.22/yt-dlp.exe as Trojan:Win32/Wacatac.B!ml.
Provide verbose output that clearly demonstrates the problem

[ ] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[ ] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
No response

from pinned issues thread:

It's a false positive. Complain to your vendor. There is nothing we can do

Duplicate of #25


"ERROR: failed to load model" for complex-modeling

Hi.
While running the following command for complex modeling,
python network/predict_complex.py -i filtered.a3m -o complex -Ls 218 310
The message comes as:
Using backend: pytorch
ERROR: failed to load model
Please suggest as how we can solve this error.
Thanks in advance!

I had the same question. Did you find out how to solve this?

same


ijwb stores entire bazel workspace as single IntelliJ project

ijwb's use of a single project to represent the entire bazel workspace can result in disagreements between the IDE and bazel as to whether the code compiles properly, which results in more than a little confusion for us internally.
One easily reproducible issue is that in a workspace with two projects, only one need declare a dependency on an external jar and both projects will see the jar as a dependency according to IntelliIJ, though bazel will fail to build. I posted a simple example workspace (bazel doesn't build, but ijwb says everything is fine) on github that illustrates the issue. There's also separate, related issue that we're experiencing internally, where IntelliJ will resolve dependencies in a particular file to jars that are dependencies of an entirely unrelated project that shares no dependencies (transitive or otherwise). Bazel finds the right jars while building and running.
Are there any plans to have more granular classpaths for bazel packages in ijwb?
edit: typos

More context:
We're migrating a large-ish codebase (300+ projects) from maven to bazel. In some cases, different top-level projects depend on different versions (or "flavors" of the same version) of an open source library. For example, we have one project that depends on guice-4.1, and another that depends on guice-4.0-no_aop.
Concretely, what we have is:
Foo → guice-4.1.jar
Bar → guice-4.0-no_aop.jar

The two projects that depend on those jars are unrelated; there's no dependency leakage or anything. Bazel builds cleanly, and correctly; bazel query to examine the dependencies of each project individually show that they're correct.
However, if we open a file in Foo that depends on an API that only exists in guice-4.1.jar, IntelliJ marks the API usage in red, and says that it can't find that API. I suspect that this is due to the ordering in the idea workspace.xml file, because I can fix it (or break it) by twiddling that.
While I was playing around with trying to come up with a reduced test case, I also discovered that it's possible to get into situations where IntelliJ says everything is fine, but bazel doesn't build. This is an easier case to repro, so I posted that one to github.

We have no plans for multiple IntelliJ modules per project. At the moment, the best workaround is to use multiple IntelliJ projects per Bazel workspace.
You can set the scope of the IntelliJ project to whichever set of Bazel targets constitute a sensible module, in the project view file (Bazel > Project > Open project view file).

@brendandouglas If we do that, how would we use IntelliJ to refactor? Currently we load all of our Java code into one workspace, so when we refactor, IntelliJ finds all call sites and updates them; if we split that up into multiple projects, refactorings in one project won't find call sites in the other projects, right?
This bug seems really bad to me. Refactoring is one of the top features people use IDEs for. The whole point is to do the refactor and verify in the IDE that the whole project compiles without errors. If there's only one classpath for the entire workspace, then there's no way to automatically make changes that affect multiple projects and be sure that they safely compile.

The problem is that there's a fundamental mismatch between the build units in IntelliJ and Bazel.
In IntelliJ, you can only configure settings like java language level on a per module basis. This implies one Bazel target == one IntelliJ module.
However Bazel targets operate on lists of files, IntelliJ modules on recursive directories.

I don't think I understand your point. I agree there is some mismatch between Bazeld and IntelliJ, but that mismatch is not relevant to this bug.
Just to clarify, let's walk through the test case Robert posed. The WORKSPACE declares two maven_jar rules: aopalliance and guice_4_1. There are only two BUILD files: project1/BUILD and project2/BUILD (no nested packages in subdirectories).
The project2 build file declares a simple java_library rule:
java_library(
    name='project2',
    srcs=glob(["src/java/**/*.java"]),
    deps=[
        "@aopalliance//jar",
        "@guice_4_1//jar",
    ],
)

//project2:project2 successfully builds both in the bazel CLI and also in ijwb.
The project1 build file is a copy of project2, except the dependency on @guice_4_1//jar is commented out. The Module1.java file imports a Guice class, so //project1:project1 fails the Bazel CLI build, as you'd expect.
But the project1 module compiles without errors in ijwb, because project1 has access to all of the jars in the WORKSPACE, including jars that were only declared as dependencies of project2.
Your point about recursive directories is interesting but is it relevant to this bug? If project1 were a subdirectory of project2, I can see how this would not be something that ijwb could reconcile.
Since project1 and project2 are totally separate directories, AFAIK nothing about IntelliJ's module system should prevent these modules from having separate classpaths (or separate Java language levels, if it came to that).

I agree there are cases where Bazel and IntelliJ's build units can be compatible. My point is that there are also many cases where they're not.
Any time you have build rules in the same package with conflicting dependencies or build flags, there's no way to correctly include them both in a single IntelliJ project.
As things stand, it's pretty much infeasible to autogenerate intellij modules from a bazel project. The only sensible build unit is a bazel target, but:

there are typically multiple targets in a single directory
typical projects have many, many targets, and IntelliJ scales poorly with number of modules (O(n^2) performance)

We could do something like introduce a special kind of bazel rule corresponding to a compilation module, and have the bazel plugin use those to generate modules (would likely require buy-in from the bazel team). We could also allow users to manually specify modules in some other way (e.g. in the .bazelproject file).
I suspect we'll eventually implement something along these lines, but not anytime soon (it's a decent amount of work, and there's little demand for it internally).
In the meantime, some possible workarounds:

use the same version of a given dependency, wherever possible
if practical, have the entire workspace open in a separate project, and use that for refactoring
otherwise, have multiple projects open, and do the same refactoring in each (this is what we do -- it's not pleasant, but I don't find it overly painful)

Maybe people don't need a tool to automagically map targets into modules. It would be nice to have a rule like:
intellij_java_module(
  name = "foo",
  srcs = glob(["src/**/*.java"]),
  deps = [...],
  tests = glob(["test/**/*.java"]),
  testdeps = [...],
)

Which would map into a intellij module. Dependencies would be handled as any jars.
This plugin, as it is right now, is impractical for any real use case scenario, IMHO.

I think the above is a harsher position than I'd take (having used the plugin at YouTube, a year ago, to great effect).
I think there are some possible ways out of this, including possibly a code analyzer that is much more aware of dependency relationships between targets, and can trigger the symbol-not-found behavior.

I also think that "little demand for it internally" made a lot of sense when the comment was made, but at this point, those of us considering bazel adoption outside of Google need to know if our use-cases will be understood and taken on board by Google, so I'm really hoping that this question can be re-prioritized in that newer light, and that the "internal" (inside Google) lack of motivation for a thing doesn't always result in it being de-prioritized.

To add our comments, we wouldn't either mind configuring the modules we want to avoid the problem with mismatch between bazel and a intellij project
Even better would be to be able to configure X projects with Y modules as Intellij gets a bit slow if running with to many modules

Naive comment: What about using tags to identify what the user considers a "module" in Intellij? Something like intellij-module=module-name and then running a query like: bazel query "attr(tags, intellij-module=*, //...:*)

Some promising news from the IntelliJ team, at https://blog.jetbrains.com/idea/2019/12/intellij-platform-roadmap-for-2020/ (emphasis mine):

Project Model Redesign
The project model is how the IDE represents the structure of your project – which files belong to the project, how they depend on each other, which libraries are used, and so on. IntelliJ IDEA’s project model has served us well over the years, however, it has certain limitations. First, it doesn’t support arbitrary mixing of projects of different types. For example, AppCode can open an Xcode project and Rider can open a Visual Studio solution, but there is no way to open a Gradle project and an Xcode project in the same IDE frame. Second, the project model works on the level of directories, not files, and it can’t represent different files in the same directory having different dependencies. This makes it difficult to integrate build systems such as Bazel into the IDE, and presents challenges in other scenarios.
The redesigned project model, which we internally call “workspace model”, will remove these limitations. It also brings additional benefits such as better performance during project opening, smoother synchronization with Maven and Gradle, and a nicer programming model. We’ll start with changing the internal implementation to the workspace model, and once this is stable, we’ll proceed to adding user-visible features such as combining projects of arbitrary types in the same IDE frame.

We have no plans for multiple IntelliJ modules per project. At the moment, the best workaround is to use multiple IntelliJ projects per Bazel workspace.
You can set the scope of the IntelliJ project to whichever set of Bazel targets constitute a sensible module, in the project view file (Bazel > Project > Open project view file).

@brendandouglas curious how you would have multiple projects for a given Bazel workspace.  When I attempt to create a new project with a different .bazelproject file I get an .ijwb already exists error.

same question when I use multiple IDEs from JetBrains for open 1 bazel project like Aqua and Intellij


Split SwiftSDKGenerator into multiple files

SwiftSDKGenerator.swift has grown quite large even at this early stage of supporting only macOS as a host platform and a few Linux distributions as target platforms. It's time to split it up for better readability and easier navigation.

@swift-ci test

@swift-ci test


Beta: ES6: Create an Export Fallback with export default

Challenge create-an-export-fallback-with-export-default has an issue.
User Agent is: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/59.0.3071.86 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
The code below is what the test is currently expecting, but default exports cannot be used with var, let, or const
Ref here under 'using default export'

export default const subtract = (x,y) => {return x - y};

assert(code.match(/export\s+default\s+const\s+subtract\s+=\s+\(x,y\)\s+=>\s+{return\s+x\s-\s+y;}/ig))

confirmed. Submitted a fix.


[test] Port GPUDirect check to Tiger

Will fix UES-580
Changes needed to run on tiger, is there a better way to do it ?

@jgphpc To answer your question "is there a better way to do it ?"
I don't think you can  do it differently  for the moment, but in the future we can think of attaching attributes to systems, that we can later  check in the tests. That would decouple the test further.

https://github.com/eth-cscs/reframe/pull/966#issuecomment-542143143
Would it be possible to detect that we are on a Cray system (/opt/cray/ or /etc/opt/cray/release exist) then we could use stg like:
if self.current_system.name in {'dom', 'daint', CRAY}:
instead of
if self.current_system.name in {'dom', 'daint', 'tiger'}:
?


vue-dropzone appends s3ObjectLocation to form data when s3 is not enabled

https://github.com/rowanwins/vue-dropzone/blob/1edc99efeda7208d22e3da809573b7f85e51df20/src/components/vue-dropzone.vue#L259-L271

Good pickup @caleb-allen , I'll hand this over to @vrajroham as he looks after the s3 stuff.

@caleb-allen Could you confirm if this issue is resolved?


Feature: Classroom mode

It would be handy to be able to gather people within a classroom. You would end up with something like :class/:bin. The :class would provide links to those :bins and make it easier to see what's going on (are people coding, are they stuck etc.).

Good idea :-)