OpenSource : 4.1 Explain everything related to the MLH fellowship

Description :

Explain everything related to the fellowship

Note :

Changes should be made inside the Open_Source/ directory & Opensource branch.
Issue only for GWOC'21 contributors.
Issue will be assigned on a first come first serve basis, 1 Issue  ==  1 PR == 1 Contributor
For Contributor please don't create the same issue again, before creating any issue do visit OpenSource Milestone
Doc name should same as the issue title

Must Follow : Contributing Guidelines & Code of Conduct before start Contributing.
Happy Contributing 🙌

Can I work on this?

Can I work on this !

Can I work on this?

Sure @mayankkuthar , Assigning this to you, Happy Documenting 🎉🎉

Can I work on this !

Creating more issues wait 😂

@siddhi-244  Till then complete your assigned ones

@siddhi-244 Till then complete your assigned ones

1 is done.😅

Can I have the honour of handling this PR, please?


Make sure terminated containers are considered as already drained

As @univ0298 points out in Slack, the drain script support from #1302 is still incomplete:
When the main process of a container exits, then the preStop script will not gain back control because there is nothing left to do: the container is already stopped. This is "documented" in Add e2e to verify blocking behavior of preStop hook and PreStop process is not completed when within termination grace period.

PreStop doesn't block a container from exiting or crashing. If the main process exits, that's game over.

My suggestion to work around this issue is to explicitly set the stamps for all terminated containers explicitly before counting how many drain scripts have finished. Proof of concept code:
#!/usr/bin/env bash

NS=kubecf
POD=diego-cell-0

JSON=$(kubectl get pods -n ${NS} ${POD} -o=json)
CONTAINERS=$(echo "${JSON}" |
                   jq -r '.status.containerStatuses[]
                        | select(.state.terminated).name')

for CONTAINER in ${CONTAINERS}; do
    STAMP=$(echo "${JSON}" |
                  jq -r ".spec.containers[]
                       | select(.name==\"${CONTAINER}\").lifecycle
                       | select(.preStop).preStop.exec.command[]
                       | capture(\"(?<a>/mnt/drain-stamps/.*)\n\").a")
    if [ -n "${STAMP}" ]; then
        touch ${STAMP}
    fi
done

This generates a list of all containers that have a state.containerStatuses.terminated field. I then scrape the actual stamp filename out of the preStop commands and make sure the stamp file exists. It is creating stamps for those containers that didn't get a chance to do it themselves.
I've tested the script by replacing the condition above with state.running and used echo instead of touch:
$ ./drain-stamp.sh
/mnt/drain-stamps/garden
/mnt/drain-stamps/loggr-forwarder-agent
/mnt/drain-stamps/loggr-syslog-agent
/mnt/drain-stamps/loggr-udp-forwarder
/mnt/drain-stamps/loggregator_agent
/mnt/drain-stamps/netmon
/mnt/drain-stamps/prom_scraper
/mnt/drain-stamps/rep
/mnt/drain-stamps/route_emitter
/mnt/drain-stamps/silk-daemon
/mnt/drain-stamps/vxlan-policy-agent

I know that we still have other issues with drain scripts, but this should get us one step closer.
@manno What do you think? It feels quite hacky, but on the other hand, it should be pretty straight-forward to add; everything else I can think of is much more complicated.
Remaining issues: It looks like we don't currently include kubectl in the images, so need to figure out how to make that part work.
And I haven't checked if the service account has a role binding that allows get pod access from within the containers.

set the stamps for all terminated containers explicitly before counting how many drain scripts have finished

What I mean by this is inserting the logic from the script above in the waitExit function at this line just after the sleep 5 statement.

I would prefer if the drain scripts were patched to behave better on Kubernetes.
These problems come to mind:

We would need to modify the stemcell to include jq/kubectl.
The script needs to check for binaries, in case it's running with an old stemcell that doesn't have kubectl. We might need a fallback behaviour?
Would need more privileges for pod list/get. Which includes alot. (I didn't check, but I really hope the jobs don't have access to the cluster).
kubectl doesn't report the real status fields, as far as I can remember it's an aggregate over several fields. There is some logic in kubectl, regarding the pod status field.
https://kubernetes.io/docs/concepts/workloads/pods/pod-lifecycle/

The drain script is already doing a lot and it feels wrong to put more logic there. We could implement this in Go, to get rid of the dependencies on jq/kubectl. However, how would that work, we add another sidecar container (with the operator image) and run the new pre stop hook there?

I don't think we can securely add get/list pods RBAC to the service account used by the pods.
If we need to check cluster state, we will need to add this to the operator somehow (i.e. a pod with more access).
First thing that comes to my mind is switching to http PreStop hooks. We’d need a small API server to implement the checks.

This approach does not work because kubernetes never switches a container status to terminated when the main process in the container exits while the PreStop hook is still running. Instead the hook will be killed and exits with code 137, but the container status remains running until the grace period expires, even though there are no processes left running inside the container.
We'll look into an alternate approach by modifying container-run to not exit until it receives a SIGTERM signal.

Yes that’s a good description of why this type of exit will not work thanks


Make clear that CLOPS runs take longer than 10min

The CLOPS readme currently says it will take around 10min.  However, actual execution of the script is ~56min.  Perhaps the readme should be updated to say runtime is ~1hr

@andrewwack ?

I also found it took fairly long to run so it was a bit confusing to me as well

Hmm, let me look.  It shouldn't be that long, but may be longer than 10 minutes.

Thanks, I'll close.


Middleware not logging bodies if unhandled exception occurs downstream

Hi,
thanks a lot for this very helpful extension!
I already use it in multiple projects, but I am missing on crucial feature: Logging of request bodies of failing requests (500 responses).
The middleware does not log any request or response bodies if any unhandled exception occurs downstream.
But for us especially in these cases we would love to have at least the request body available in application insights for further debugging.
The extension currently registers a middleware that does not catch and handle all exceptions.
For the feature I am missing I would like to implement a different behavior, so that if other middleware handlers or downstream processing fails the error is caught, the request telemetry created and rethrown.
This might create unwanted side effects for users who already have some error handling middleware in place, therefore this feature would be disabled by default.
In hope that this is a feature not only helpful for me, I will try to prepare a corresponding PR for discussing this further.

Hi, and thank you for the feedback and pull request. Always appreciated.
I think the app insights logging middleware would need to run as early as possible in the pipeline, just like the ExceptionHandlerMiddleware, so that it can catch unhandled exceptions.
I'll take a closer look and extend your PR.

I can also have a look into putting this behind a guard clause.
Should be an easy fix.
Do you also want to change the way the service registers the middleware? The ensure that is registered as one of the first middlewares? Currently this would be controlled by the user and when the user decides to register the appinsights logger.

I can also have a look into putting this behind a guard clause.
Should be an easy fix.

Great! Always happy to receive PRs

Do you also want to change the way the service registers the middleware? The ensure that is registered as one of the first middlewares? Currently this would be controlled by the user and when the user decides to register the appinsights logger.

How would you solve this?

I can also have a look into putting this behind a guard clause.
Should be an easy fix.

Great! Always happy to receive PRs

I will try to add this today, any preferences on the name the configuration parameter for this?

Do you also want to change the way the service registers the middleware? The ensure that is registered as one of the first middlewares? Currently this would be controlled by the user and when the user decides to register the appinsights logger.

How would you solve this?

To be honest I would not solve it at all and keep the order to the user.
Idea-wise it might be possible to hook into host startup and reorder them there. But from my perspective this introduces too much fairy dust and magic into the startup process. This could lead so some very hard to debug errors for users who have complex setups.

Thank you for the pull request. I'll do some unrelated cleanups and then prepare a release!


Node 8.10 not supported on add auth

Resource Name: UserPoolClientLambda (AWS::Lambda::Function)
Event Type: create
Reason: The runtime parameter of nodejs8.10 is no longer supported for creating or updating AWS Lambda functions. We recommend you use the new runtime (nodejs12.x) while creating or updating functions. (Service: AWSLambdaInternal; Status Code: 400; Error Code: InvalidParameterValueException; Request ID: 66cdc39e-b2fb-4b4b-897d-ecb0cb9a4097)
amplify auth add
amplify push

Please update to the latest version of the CLI  and your lambda functions should be auto updated to Node 10.x on running any Amplify command.


What can fill the Security analysis section in the quality widget?

Is it only HP Fortify or can we use other tools to find in that section?

@onepartcode  eratocode scan and Fortify


Testfairy upload timeout

Checklist

[x] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[x] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.

Motivation and Context

While in Australia, I could not upload my app to TestFairy because the request would often timeout. With this parameter, I can upload my apps even if I'm in the middle of no where with a bad connection.

Description

I've added a timeout parameter to the TestFairy Action. This will allow users who are very far away from TestFairy servers to actually upload their apps before the timeout kicks in.

I used Charles Proxy to throttle my connection so that I could see my timeout value being used.

I signed it!

CLA signed!

Thanks @joshdholtz! Made the changes you asked for.

I'll take this, that's code I added. Strange things happening there.


Initializing BosonSite to a thermal state

Hi,
First of all, thank you so much for building this awesome package.
Could you please provide instructions on how to initialize BosonSite to a thermal state?
I found that from_lat_product_state 's argument p_state can take a list of 1D arrays, so my guess is that each 1D-array element can be used to specify the initial state of each BosonSite.
However, from the documentation and code, it wasn't clear to me exactly how each 1D array is converted to a bosonic state. Should I just naively input the diagonal of the thermal state's density matrix?
Thanks again!

What exactly do you mean with a "thermal state"?
I'd usually write a thermal density matrix rho = exp(-beta H ) / Z with Z = tr(exp(-beta H), but MPS typically represent pure states.
One common approach (besides e.g. METTS, which is currently not implemented in TeNPy) would be a purification of the density matrix with the tenpy.networks.purification_mps.PurificationMPS.
There's some basic examples in examples/purification.py.
The idea is to start at infinite temperature and then cool down to finite temperatures with imaginary time evolution.
There is a subtlety with bosons, however: at infinite temperature (PurificationMPS.from_infiniteT), the boson number can be anything (in the grand canonical ensemble), giving you an infinite-dimensional local Hilbert space, while MPS need to truncate to finite boson numbers to be able to represent it with finite-dimensional tensors....
If you work with finite systems, you can get around this by using a canonical ensemble instead (PurificationMPS.from_infinite_T), I think things should work in principle (assuming that you have a particle-conserving H).

Thanks for this kind explanation! I am new to these simulations and I wasn't aware that MPS should represent pure states.


Number of passed tests can't refresh after run tests

testup version：Version 2.3.1.rbz

Do you have "Run in Console" checked?

If that's the case then the webdialog doesn't update with results.

It would make sense that such button grays out the or otherwise hides the content of the dialog, and add a message describing why.

this is my settings.

@linying1991 what version of SketchUp are you using?

@thomthom
SketchUp: 19.0.685
Ruby: 2.5.1
TestUp: 2.3.1
Minitest: 5.11.3
Platform: platform_win
Locale: zh-CN

This is strange...
hmm.. @Eneroth3 - are you using the latest TestUp? Does it work for you? (Wondering if there's something on my development machine that cause me not to see the issue.)

I'm on TestUp 2.2.0, with the old UI.

@linying1991 - are you able to test v 2.2? If that works it would give us some clues to start working off.

@thomthom I think I've found the reason. Before installing Testup2, I installed Minitest-5.11.3.
Now, i uninstall Minitest-5.11.3, Testup2 works normally.
thanks very mush~~~

@thomthom v2.2 can't work on my computer...

Before installing Testup2, I installed Minitest-5.11.3.

Ah! That's interesting. TestUp bundles 5.4.3 within the RBZ to allow it to be installed without internet connection or have the lag it takes to download and install.
I wonder if I can make it try to load that specific version... However, I guess we'd also run into issues if a newer version was loaded before TestUp. So maybe we need a detection in place to inform the user of a clash.
I'm reopening this issue for that purpose.
That you for your report and investigation.


Minor fixes to README for issue #170

Fix broken links, add syntax highlighting and slight changes to the bad_attributes_reject example referenced in issue #170

Thank you for contribution.


DAPP Store

DAPP Store screen
https://zpl.io/aRKx4vr
There should be no "Add Dapp from a file/Uninstall Dapp" functionality on mobile yet.

Please change icon BeamX DAO and change signature  "LOCK SOME BEAM" to "Governance, staking and voting"
in masternet:

in design:

Checked


Is there any any to set trainable weights in loss function

I have set a custom loss function as below:
def cross_entropy_with_penalty(y_true, y_pre):
    penalty = K.constant([1, 5, 5])
    return -tf.reduce_sum((y_true * penalty) * K.log(y_pre + 1e-7)) - tf.reduce_sum(
        ((1 - y_true) * penalty) * K.log(1 - y_pre + 1e-7))

and it works. However I hope the penalty  can be trained, is there any way for it?

@busyyang
Moving this issue to closed status as there has been no activity, in case you still face the error please create a new issue.


eslint crashes after adding a bunch of more packages

Adding 8 dummy packages seems to be enough to trigger an OOM (with default memory settings)
@acme/package8:lint: FATAL ERROR: Reached heap limit Allocation failed - JavaScript heap out of memory

Logs: zcesur/eslint-oom-repro / CI / build-lint
Repro: zcesur/eslint-oom-repro
Relevant issue: typescript-eslint/typescript-eslint#1192

Looks like a limitation of using typescript-eslint in large multi-package monorepos. But since I found a workaround that seems to work well with t3-turbo, I thought I'd share

I fixed this by changing the root package.json file "lint" script to this:
"lint": "NODE_OPTIONS=--max_old_space_size=4096 turbo lint --continue --concurrency 1",

The concurrency thing may or may not be necessary for all cases.

Adding more memory does resolve the OOM ofc, but it's still extremely slow and seems to scale poorly with the number of packages you have. Plus you might still run into the same issue as you add more packages.
Here are the numbers with your tweak (without the concurrency part):
Elapsed (wall clock) time (h:mm:ss or m:ss): 0:48.72
Maximum resident set size (kbytes): 2359660

To contrast, here are the numbers with parserOptions.project = true as in #415:
Elapsed (wall clock) time (h:mm:ss or m:ss): 0:08.46
Maximum resident set size (kbytes): 417172

@zcesur Thanks for the tip, I made the change and it was a huge improvement to our lint CI


关于unimodal1和unimodal2的问题

请问能否提供Table 1中unimodal1和unimodal2的代码呢？或者你能否告诉我你是如何训练得到这些结果的呢？我自己写的代码在CREMAD中训练出来的结果只有visual 30% ; audio<EMAIL_ADDRESS>

Hi,
I would like to request the same, I tried the unimodal ones for AVE with the given code by running first mp4_to_wav and then video_preprocessing and the results I got from the unimodal networks VClassifier and AClassifier are almost random. I believe there must be something in the unimodal training pipeline that we are missing. Could you please upload the rest of the code?
Thanks!

请问能否提供Table 1中unimodal1和unimodal2的代码呢？或者你能否告诉我你是如何训练得到这些结果的呢？我自己写的代码在CREMAD中训练出来的结果只有visual 30% ; audio<EMAIL_ADDRESS>
I don't know the specific reasons for why you cannot get the uni-modal results. I use 3 frames for each video clips for CREMA-D. Do you just use one frame for that?

Hi,
I can replicate and even surpass the unimodal in audio but the 3 frames per clip do not really get improved while training. I have tried several things, changing the model, optimizers and hyperparams, and the transformations but nothing flourished till now. Could you possibly provide a notebook with dependencies on your code to show its working?
Thanks!

I am coming back to this one because I noticed something strange. I trained the unimodal vision model with 3 frames a bit further till it reached 100 epochs without improvement (~ 250 epochs in total) and the accuracy went up to ~74%. That confused me a bit since the rest of the numbers match the ones from the paper. Any intuition?

Could you upload also the script tuning_uniclassifier that appears in your bash file?


cli help

The cli tool has subcommands, similar to git or npm, and should support command-specific help.
$ nv assign --help

This should be moved to the new independent CLI project:
https://github.com/anvilresearch/connect-cli

Moved into anvilresearch/connect-cli#1


Synchronous mode with AsyncHttpClient

Is there a way to handle responses synchronously with AsyncHttpClient? My handler is created within the main thread, which has a message looper as required, but it still cannot be used with AsyncHttpClient in synchronous mode.
If the only way is to use the SyncHttpClient, then I don't understand why there is a "setUseSynchronousMode" method in AsyncHttpResponseHandler - the response handling strategy could simply be infered by the client instance being used.
What I'd really like is to always use AsyncHttpClient and take advantage of the handler's "SynchronousMode" flag to configure on which thread the response handling will occur.
Does that make any sense?

setUseSynchronousMode purpose is to allow single implementation of AsyncHttpResponseHandler for both sync and async use-cases, you can see that in usages of the method
This is already handled on per-request basis, so if you provide ResponseHandler instance with useSynchronousMode set to true to AsyncHttpClient, it will be handled synchronously, SyncHttpClient is just simplification of the process.


Update CloneOpenApiDocument to use copyConstructors instead of writing and reading new document.

The current cloneOpenApiDocument is very inefficient.  The new copyConstructors in OpenApi.NET 1.4 should significantly improve the performance and reduce memory usage.  (Using copy constructors causes the clone to share the instances of the string. This means no new strings need to be created for the clone)  Related to #1142

Blocked on the release of OpenApi.Net 1.4.0-preview3 that will contain this fix https://github.com/microsoft/OpenAPI.NET/issues/850

Dependent on a fix for this https://github.com/microsoft/OpenAPI.NET/issues/995

@MaggieKimani1 could you check with @irvinesunday  why copyConstructors is failing?

Fixed by https://github.com/microsoftgraph/microsoft-graph-devx-api/pull/1411 and https://github.com/microsoftgraph/microsoft-graph-devx-api/issues/956. We are now reading the Open API document from the metadata repo and caching different copies of the Open API document for each style.


Hatsune Miku: Project DIVA F 2nd [PCSE00434]

App summary

App name: Hatsune Miku: PJ DIVA F 2nd
App serial: PCSE00434
App version: 1.00

Vita3K summary

Version: v0.1.6
Build number: 3004-97f8bb0d
CPU backend: Dynamic
GPU backend: Vulkan

Test environment summary

Tested by: Jackxon
OS: Windows 10
CPU: Intel(R) UHR Graphics 620
GPU: NVIDIA GeForce MX150
RAM: 4 GB

Issues

The problem is similar to Project Diva X which is the opening.avc and other avc files from the "movie" folder won't play, but it is solvable by following the instructions from this "https://github.com/Vita3K/compatibility/issues/270"
However, the visual problem exists following the screenshot but still the game can play normally

Screenshots

Recomended Label
Ingame +

App summary
App name: Hatsune Miku: PJ DIVA F 2nd
App serial: PCSE00434
App version: 1.00
Vita3K summary
Version: v0.2.0
Build number: 3570-aa2c1cb0
Source: AUR bin
CPU backend: Dynamic
GPU backend: Vulkan
Test environment summary
Tested by: @akatsukinoyami
OS: SteamOS
CPU: AMD APU 4c/8t Zen2
GPU: Integrated RDNA2 8CU + 2Gb Ram
RAM: 14 GB
Device: SteamDeck OLED
Screenshots

Game playble with stable FPS. It has 30FPS lock. With FPSmod background PV become laggy. But pressings still works correctly.

App summary

App name: Hatsune Miku: Project Diva F 2nd
App serial: PCSE00434
App version: 1.00
Status: Playable

Vita3K summary

Version: v0.2.0
Build number: 3664-93759d9c
Source: Official website

Test environment summary

Tested by: @ZGStuff
OS: Windows
CPU: Intel Core i7-1355U
GPU: Intel Iris Xe Graphics
RAM: 16 GB
Device: Lenovo IdeaPad Slim 3i

Issues
Minor lag on loading screens, otherwise fully playable
Stable FPS, locked to 30 frames though
Screenshots

Log
vita3k.log


Unit Tests

@jcbeyler @caoman @RyRose Started adding unit tests to the project.

The testing was the biggest change but there are some changes from cpplint, changes to make code more testable, and a bug fix.


Fix bug in TM_Inspiration_ArcanePathway.LetterText.get when dealing with custom classes with only one degree

Use first degreeDatas entry for both labels. Also put traitDef into a variable since EnabledMagicTraits can technically change between lines.

Solves Issue #23

Accepted


how to serve json data during ng serve?

Hi,
I have created an angular 2 app and i am trying to create a service to send json data on UI.
Structure

app
app/data/countries.json

[
{ 'id': '1', 'name': 'United Kingdom' },
{ 'id': '2', 'name': 'United States' },
{ 'id': '3', 'name': 'India' },
{ 'id': '4', 'name': 'Pakistan' }
]

factory/factory.service

import { Injectable } from '@angular/core';
import { Http, Response} from '@angular/http';
import { Observable } from 'rxjs/Observable';
import { IDictionary } from '../shared/dictionary';
@Injectable()
export class FactoryService {
private countryUrl = "../data/countries.json";
constructor(private http: Http) {

}

getCountry(): Observable<IDictionary[]> {
    return this.http.get(this.countryUrl)
        .map((response: Response) => <IDictionary[]>response.json())
        .do(data => console.log("Countries: " + JSON.stringify(data)))
        .catch(this.handleError);
}

private handleError(error: Response) {
    console.log(error);
    return Observable.throw(error.json().error || "500 internal server error");
}

}

IDictionary

export interface IDictionary {
[index: string]: string
}

factory/factory-form.component.ts

export class FactoryFormComponent implements OnInit {
private model: Factory;
countries;
factoryStatuses;
productTypes;
risks;
private errorMessage: string;
private submitted = false;

constructor(private factoryService: FactoryService) {

}

ngOnInit(): void {
    this.countries = this.factoryService.getCountry()
        .subscribe(countries => this.countries = countries, error => this.errorMessage = error);
    this.factoryStatuses = this.factoryService.getFactoryStatusDummy();
    this.productTypes = this.factoryService.getProductTypeDummy();
    this.risks = this.factoryService.getFactoryRiskDummy();
}

}

I have noticed when i open the app in chrome, its missing the data folder although i put it in "app/data" in system.config.js

Error:

do i need to do some extra stuff to make data folder as part of my app?

update:
I changed the path to "app/data/countries.json" but and I can see something like below when debugging

private extractData(res: Response) {
let body = res.json();
return body.data || {};
}

I am not sure how to access private field _body. any idea?

It was not an issue with Angular-cli, it was issue with my code and solution is:
Replace below code:

this.countries = this.factoryService.getCountry()
.subscribe(countries => this.countries = countries, error => this.errorMessage = error);

with

this.factoryService.getCountry()
.subscribe(countries => this.countries = countries, error => this.errorMessage = error);

or

this.countries = this.factoryService.getCountry().error(...);

and use async pipe on html


[EN] Feedback for: /web/tools/chrome-devtools/profile/evaluate-performance/timeline-tool?hl=en

omg，today I updated to 49.0 ,Timeline shows :

omg, the doc has not been updated ?
Frankly, look at the document can't completely let me this useless timeline before people understand now how to debug

Duplicate of your existing iss #2475.
This is being worked on, but updating the Timeline docs is a major undertaking.

Hey @riskers, the doc has been updated!
Thanks for your feedback

@kaycebasques
Select js profile and paint seems to have no effect


Failed to delete OAuth session

Everything seems fine until it tries to delete the KV entry
  const res = await kv.atomic()
    .check(oauthSessionRes)
    .delete(key)
    .commit();

at which point it throws
Failed to delete OAuth session

I am running with --unstable and --allow-all.
No idea why this is failing. My keys and domain are set for Auth0 and Auth0 is configured to point everything to my localhost.

Can you please share the full snippet?

I created an Auth0 account and successfully signed in without issue. I wasn't able to reproduce the issue.
I can't say, with confidence, why the commit within getAndDeleteOAuthSession() is failing. Unfortunately, KV doesn't yet provide much insight into why commits fail, but there's work in progress that should fix that.
Can you please share your deno --version output? Also, has this only happened once or does this happen repeatedly?

But yes, please let me know how you go on Deploy and the other machine.

Looks like it is an issue with my setup on my windows environment.
Just tried it on my debian laptop with the same exact code (and deno version) and it ran fine, and my deploy to deno.deploy is working just fine as well.
Not sure what my issue is on windows.
Appreciate your help @iuioiua !

Oh, that's good to hear. Thanks for bringing this up! Glad I could be of assistance.


Run wycheproof tests against Caliptra

ECDSA verify tests can be run using Caliptra's ECDSAVerify command
HMAC do not currently have a mechanism for external testing of the block. But might be needed for FIPS anyway? Otherwise, could build a test mailbox command for testing.

Just to clarify: the issue is only about running the Known Answer Tests in the wychenproof repository against the hardware implementations? Do you guys imagine this as fully integrated system tests (i.e. cargo test -- wychenproof)?

Completed

Yep, thanks!


Documentation for example module

Relates to https://github.com/paritytech/substrate-developer-hub/issues/44

@bkchr I've pushed a commit that moves the docs into lib.rs based on review comments, and the docs for the staking module that's been approved. In the docs you can show/hide the template by clicking the "> Details" link as shown in below screenshot.
You can build the docs with cargo doc --package srml-example --open
Should I remove the "Implementation Details" and "Extensibility" sections?
Screenshot:

On hold, pending Assets module https://github.com/paritytech/substrate/pull/1945


ISSUE-31 update on window resize and path change

I ended up implementing an interval as one was required to check for path changes anyway, it also handily debounces the resize.
onHashChange doesn't always work e.g. https://www.oliverbonas.com/ is a SPA that updates the path, not hash. Set it at 500ms which seems like a reasonable compromise.

@simonhearne still need to add unit tests :smile:

fixes #31

Verified ok

@MalcolmSmithUK ready for review


Installed HACS on Hassio 0.98.5.  Not loading, halts Hassio

Version of HACS
.13.3

Describe the bug
A clear and concise description of what the bug is.
Fresh install of HACS locks up hassio.  Check config fails.  Reboot doesn't even load HA.
Debug log

2019-09-16 12:31:54 WARNING (MainThread) [homeassistant.loader] You are using a custom integration for hacs which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you do experience issues with Home Assistant.
2019-09-16 12:33:41 ERROR (MainThread) [aiohttp.server] Error handling request
Traceback (most recent call last):
  File "/usr/local/lib/python3.7/site-packages/aiohttp/web_protocol.py", line 418, in start
    resp = await task
  File "/usr/local/lib/python3.7/site-packages/aiohttp/web_app.py", line 458, in _handle
    resp = await handler(request)
  File "/usr/local/lib/python3.7/site-packages/aiohttp/web_middlewares.py", line 119, in impl
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/real_ip.py", line 40, in real_ip_middleware
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/ban.py", line 73, in ban_middleware
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/auth.py", line 231, in auth_middleware
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/view.py", line 128, in handle
    result = await result
  File "/usr/src/homeassistant/homeassistant/components/config/core.py", line 29, in post
    errors = await async_check_ha_config_file(request.app["hass"])
  File "/usr/src/homeassistant/homeassistant/config.py", line 807, in async_check_ha_config_file
    res = await check_config.async_check_ha_config_file(hass)
  File "/usr/src/homeassistant/homeassistant/helpers/check_config.py", line 107, in async_check_ha_config_file
    integration = await async_get_integration_with_requirements(hass, domain)
  File "/usr/src/homeassistant/homeassistant/requirements.py", line 46, in async_get_integration_with_requirements
    await async_process_requirements(hass, integration.domain, integration.requirements)
  File "/usr/src/homeassistant/homeassistant/requirements.py", line 67, in async_process_requirements
    if pkg_util.is_installed(req):
  File "/usr/src/homeassistant/homeassistant/util/package.py", line 45, in is_installed
    return version(req.project_name) in req
  File "/usr/local/lib/python3.7/site-packages/pkg_resources/__init__.py", line 3128, in __contains__
    return self.specifier.contains(item, prereleases=True)
  File "/usr/local/lib/python3.7/site-packages/pkg_resources/_vendor/packaging/specifiers.py", line 703, in contains
    item = parse(item)
  File "/usr/local/lib/python3.7/site-packages/pkg_resources/_vendor/packaging/version.py", line 31, in parse
    return Version(version)
  File "/usr/local/lib/python3.7/site-packages/pkg_resources/_vendor/packaging/version.py", line 200, in __init__
    match = self._regex.search(version)
TypeError: expected string or bytes-like object

There is nothing with this I can do 🤷♂
Your system have issues installing dependencies if that halts HA you have more issues then the installation of HACS.
Try again later.

Same trouble on:
core-ssh:~# hassio ha info
arch: armv7
boot: true
custom: false
image: homeassistant/raspberrypi3-homeassistant
ip_address: <IP_ADDRESS>
last_version: 0.98.5
machine: raspberrypi3
port: 8123
ssl: false
version: 0.97.2
wait_boot: 600
watchdog: true

@martoq
Any errors on this page http://hassio.local:8123/developer-tools/info ?

@martoq ?

Apologizes, was out of pocket for work.  I found the problem.  I had Alexa Media package installed under custom components.  I am running 0.99.2 HA.  When booting it would traceback on that package because of the change from MEDIA_PLAYER_SCHEMA to PLATFORM_SCHEMA.  I manually updated media_player.py line 53 to reference PLATFORM_SCHEMA and now HACS loads correctly.  Why HACs would get hung up with HA on this I am uncertain but I figured I would pass along what I found.


Allow popups to live outside the map div

This is a copy from OL2 trac ticket #1727:

In the case where users have a small map div, it would be nice if they could have their popups go beyond the restrictive limits of the map div... ie live outside the openlayers div itself.
The complication of this is that it will be more difficult for the popup to move with the map as it is panned.
To get a quick solution working, we can make the default behavior for these popups be to disappear when the map is panned (see #1726)

The unsettled question is what to do with popups if the referred marker is moved out of the map entirely. This alone makes the whole feature questionable; however it would still be useful in certain cases (e.g. if the map viewport is constant). Alternatively, as proposed in the original ticket, the popup could just disappear if the map is panned or zoomed.
As I'm interested in this feature as a user (designers always design popups that way!), I'm curious to hear from developers side:

Is this desired at all, taking into account the open question above; i.e. would you accept a pull request?
What would be the rough sketch to implement this?

Want to back this issue? Place a bounty on it! We accept bounties via Bountysource.

Removing overflow: hidden from the .ol-viewport works


Update sbt-scalajs, scalajs-compiler, ... to 1.14.0

About this PR
📦 Updates

org.scala-js:sbt-scalajs
org.scala-js:scalajs-compiler
org.scala-js:scalajs-library
org.scala-js:scalajs-library_2.13
org.scala-js:scalajs-test-bridge
org.scala-js:scalajs-test-bridge_2.13

from 1.7.1 to 1.14.0
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
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!

🔍 Files still referring to the old version number
The following files still refer to the old version number (1.7.1).
You might want to review and update them manually.
doc/changelog/0.10.0.md

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-js" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scala-js" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, old-version-remains, commit-count:1

Superseded by #325.


Enhancement: Add Ability to Modify AI Providers/Models in Running LibreChat Instance

What features would you like to see added?
Currently, to add or modify models in LibreChat, one must:

Modify Environment Variables: Some models require changes in the .env file.
Edit Configuration Files: Certain configurations need to be set in librechat.yaml.
Redeploy the Application: After making these changes, the entire application needs to be redeployed to use the new models.

This process is quite cumbersome and not user-friendly:

User Experience: It disrupts workflow as users need to stop the service, change configurations, and then restart or redeploy the application.
Flexibility: Users cannot dynamically adjust or test new models without going through this cycle.

Proposed Feature:
I propose adding functionality where:

Frontend Configuration: Users can add or modify AI models directly from the LibreChat frontend interface without needing to dive into configuration files or environment variables.
Persistent Changes: These modifications should persist through redeployments, ensuring that the settings are not lost when the application is updated or restarted.

Benefits:

Improved UX: Allows for real-time adjustments to AI providers and models, enhancing experimentation and usability.
Simplification: Reduces the technical barrier for users who might not be familiar with server-side configurations or deployments.

More details
Possible Implementation:

New Config Table in DB: Introduce a new config table in the database. When this table is empty:
Use the existing settings from .env and librechat.yaml files.
Automatically populate this table with the current configurations from these files.
Prioritize Config Table: If the config table is not empty, the application should use the settings stored in this table for AI providers and models, overriding any settings in .env or librechat.yaml.

Which components are impacted by your request?
No response
Pictures
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Something like this is planned, closing in favor of https://github.com/danny-avila/LibreChat/issues/3137


MWPW-147913: Prevent Users From Uploading When They're Signed In

Resolves: MWPW-147913
Test url:

https://main--dc--adobecom.hlx.page/acrobat/online/compress-pdf?martech=off
https://mwpw-147913--dc--adobecom.hlx.page/acrobat/online/compress-pdf?martech=off

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 89.48%. Comparing base (471242a) to head (8f6e2ce).

Additional details and impacted files
@@            Coverage Diff             @@
##            stage     #696      +/-   ##
==========================================
+ Coverage   89.45%   89.48%   +0.03%
==========================================
  Files          37       37
  Lines        4589     4594       +5
==========================================
+ Hits         4105     4111       +6
+ Misses        484      483       -1

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


Paginate the Conversations to Prevent Memory Issues

What is causing memory issues? Is it exhaustion? If so, where / what is getting exhausted - is it the model inference or web client

No issue yet but in the future with large enough conversations it will based on how we are appending the conversation.

Would it make sense to think more about managing conversations in plural, than one long memory busting dialogue?


Open source builds do not support MacCatalyst target

The open source toolchain distributions do not have the appropriate libraries to support the MacCatalyst target.
For example, trying to build a catalyst app using the 5.7 release toolchain downloaded from swift.org results in:
ld64.lld: error: ../../third_party/swift-toolchain/usr/lib/swift/macosx/libswiftCompatibilityConcurrency.a(CompatibilityConcurrency.cpp.o) has platform macOS, which is different from target platform macCatalyst

The above is an excerpt from Chromium's catalyst buildbot: https://logs.chromium.org/logs/chromium/buildbucket/cr-buildbucket/8801593085733033281/+/u/compile/raw_io.output_text_failure_summary_
Environment (please fill out the following information)

OS:  all
Xcode Version/Tag/Branch: Xcode14

I'm a little confused by how this ever worked, even before the concurrency/compat libraries.
We don't ship catalyst swiftmodules or zippered libraries in the public toolchains.
There's not really a reason we can't, but I'm not sure how this is a new regression.

Thank you!!!!


support alternate signature when using current route result

Allows skipping null argument when generating url from current route with different substitutions.
@weierophinney
If you believe it to be not useful at all, pls close it at once, otherwise i will go on and add some tests.
kind regards

@pine3ree it is useful from a convenience standpoint, but the added flexibility does take away from the explicitness of the current behavior. It's clear now to anybody using the helper that the arguments call for an optional route and optional params. Allowing the first param to be either/or may add confusion and could be prone to errors.
I'm going to close this, but if you feel strongly about it please feel free to continue the discussion.

@michaelmoussa
No ,thank You, it's ok! :-)


spurious error for font-weight

Given the SCSS of
@use "@carbon/type";
...
.keyword,
.function {
  font-weight: type.font-weight("semibold");
}

I'm getting the error that:
✖  Expected carbon type token, mixin or function for "font-weight" found "type.font-weight("semibold")".  carbon/type-token-use

Presumably, that shouldn't be flagged, because type.font-weight() is a carbon mixin.

In version 2 this is as expected, set option acceptCarbonFontWeightFunction to true to allow font weight functions.
const defaultOptions = {
  // include standard type properties
  includeProps: ["font", "/^font-*/", "line-height", "letterSpacing"],
  acceptValues: ["/inherit|initial/"],
  acceptsScopes: ["type"],
  acceptCarbonFontWeightFunction: false, // permit use of carbon font weight function
  acceptCarbonTypeScaleFunction: false, // permit use of carbon type scale function
  acceptCarbonFontFamilyFunction: false, // permit use of carbon font family function
  carbonPath: undefined // allows a different path for node_modules (supports monorepo with multiple carbon versions),
   e.g. packages/proj1/node_modules/@carbon
  carbonModulePostfix: undefined, // optional for use in conjunction with `carbonPath` to where a Carbon module has been renamed e.g. `-10` with a carbonPath of `node_modules/@carbon` will use `node_modules/@carbon/type-10`
  enforceScopes: false
};


Add hexo-filter-nofollow to plugins list and news

https://github.com/hexojs/hexo-filter-nofollow/

So, what should we includes in the news? Just hexo released a new official plugin and hexo-nofollow has been deprecated?

---
title: Official plugin hexo-filter-nofollow released
---

We released a new official plugin [hexo-filter-nofollow].

Then followed by the readme.
No need to mention hexo-nofollow since it was never an official plugin.

#1082

https://github.com/hexojs/site/pull/1082 merged


Add ability to nest components

Add ability to nest components with their own view/model/template within existing Tungsten.js applications.

Closed with #122


navmesh file load error-fix

don't try to read 0 length buffers

Yup - found it, this is exactly what I did.


After the line feature layer label is set along the line layout, part of the text is displayed overlapping

Describe the bug
After the line feature layer label is set along the line layout, part of the text is displayed overlapping。
To Reproduce
Steps to reproduce the behavior:

Create a line layer
Add labels along the line
When the curvature of the line segment is large, the labels will overlap

Expected behavior
Labels are displayed continuously and do not overlap。
Solutions
1.Solve it yourself
2.Provide interface for setting word spacing

You have several options to change this behaviour. The simplest is setting a lower maxAngle of the text style. The default is 45 degrees (Math.PI / 4). If you set that e.g. to Math.PI / 8, labels will be placed in straighter segments of the line.
Another option is to add letter spacing with a function similar to the one used in ol-mapbox-style: https://github.com/openlayers/ol-mapbox-style/blob/5872d352ac71ae96aa6d13f3bf832523072f9d36/src/util.js#L46-L61.

Okay, got it, thank you


Hubhop - Merge CJ4 after AAU1 includes WT mod by default

Is your feature request related to a problem? Please describe.
We have two times CJ4 - one the old mod from WT and the new AAU1 version.
Describe the solution you'd like
Since there is only the built-in version, we want only one CJ4 preset set.

I think the current situation of having two CJ4 is confusing. The old CJ4 has some deprecated events, and the AAU1 version only a subset of the new updated events. None of the two versions has a complete set of events, so merging the two seems like the lesser evil.
I suggest to:

All newer "CJ4 AAU1" events include the text "AAU1" in the description.
All older events "CJ4" pre-AAU1 include the text "legacy" in the description.
Merge the two versions under the standard vendor: Asobo, and aircraft: CJ4.
The community will have to delete deprecated events as they are found.  There is probably one deprecated event in the legacy group for each new AAU1 event, but I can't think of a way to automatically flag which ones they are.

This double airplane issue originated from wanting to support and publish the new events for an updated beta aircraft, several months before becoming the official version.  The question remains how to avoid this issue, while still being able to support new beta versions that may come out in the future.


Add release-related rules and GitHub Actions workflow

I implemented some Bazel rules (bzlrelease) that work with a GitHub Actions workflow to create releases for my Bazel rule repositories. In short, it generates the release notes along with a workspace snippet, creates the GitHub release, and creates an auto-merge PR to update the README.md with the latest workspace snippet.
All of the pre-requisites are already being loaded in this repository. I would add a release Bazel package that would contain a BUILD file, README.md file, and a workspace snippet template (i.e., contains the load statements that need to be added to client workspaces). I would update the repository's README.md to include two markers as markdown comments to indicate where to insert/replace the generated workspace snippet. Releases can be created using a Bazel run command or via the GitHub Actions web interface.
If you would like to see an actual implementation, here is the release package, the GitHub Actions workflow and a release generated by this process for cgrindel/bazel-starlib. I could add this pretty quickly if you are interested.

sounds nice to me!


chore(main): release remnote-plugin-template-react 0.2.2

:robot: I have created a release beep boop
0.2.2 (2024-11-30)
Bug Fixes

test (0e58cf2)
test version (18e1bd5)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

remnote-plugin-template-react-v0.2.2
:sunflower:


Hitting a precision limit in cupy.sparse.csr_matrix when using multiply.

I've come across a bug in cupy.sparse.csr_matrix which seems to happen only some of the time. I have created a minimally working example below. This example is quite contrived in order to get the error message to happen every time, and the matrix is actually fully sparse. I am coming across the same bug in my research but it doesn't happen consistently, and in those cases I am using a sparse matrix.
In the MWE we create a huge dense matrix, and multiply it by a vector. The multiply method is actually turning the csr_matrix into a coo_matrix. This happens fine for the first iteration, but on a second iteration this breaks with the following error message.
It seems that the col array hits the precision limit for an int32 and then contains values that are extremely large and negative, or extremely large, and this causes the error message. However, the array is not large enough that it should hit that precision limit.

Conditions (you can just paste the output of python -c 'import cupy; cupy.show_config()')

OS                           : Linux-4.12.14-<IP_ADDRESS>10720-nasa-x86_64-with-glibc2.10
Python Version               : 3.8.5
CuPy Version                 : 9.4.0
CuPy Platform                : NVIDIA CUDA
NumPy Version                : 1.19.2
SciPy Version                : 1.7.1
Cython Build Version         : 0.29.24
Cython Runtime Version       : None
CUDA Root                    : /home4/jimartin/.conda/envs/gpu_hack
nvcc PATH                    : None
CUDA Build Version           : 11000
CUDA Driver Version          : 11000
CUDA Runtime Version         : 11000
cuBLAS Version               : (available)
cuFFT Version                : 10201
cuRAND Version               : 10201
cuSOLVER Version             : (10, 6, 0)
cuSPARSE Version             : (available)
NVRTC Version                : (11, 0)
Thrust Version               : 100909
CUB Build Version            : 100909
Jitify Build Version         : <unknown>
cuDNN Build Version          : None
cuDNN Version                : None
NCCL Build Version           : None
NCCL Runtime Version         : None
cuTENSOR Version             : None
cuSPARSELt Build Version     : None
Device 0 Name                : Tesla V100-SXM2-32GB
Device 0 Compute Capability  : 70
Device 0 PCI Bus ID         : 0000:16:00.0

Code to reproduce

from cupy import sparse
import cupy as cp

def run():
    A = sparse.csr_matrix(cp.ones((2048**2, 25)))
    return A.multiply(cp.random.normal(size=2048**2)[:, None])
run()
run()

Error messages, stack traces, or logs

---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-3-ec9775ede022> in <module>
----> 1 run()

<ipython-input-1-1d6522e0f920> in run()
      4 def run():
      5     A = sparse.csr_matrix(cp.ones((2048**2, 25)))
----> 6     return A.multiply(cp.random.normal(size=2048**2)[:, None])

~/.conda/envs/gpu_hack/lib/python3.8/site-packages/cupyx/scipy/sparse/csr.py in multiply(self, other)
    321             return multiply_by_scalar(self, other)
    322         elif _util.isdense(other):
--> 323             self.sum_duplicates()
    324             other = cupy.atleast_2d(other)
    325             return multiply_by_dense(self, other)

~/.conda/envs/gpu_hack/lib/python3.8/site-packages/cupyx/scipy/sparse/compressed.py in sum_duplicates(self)
    776         # TODO(leofang): add a kernel for compressed sparse matrices without
    777         # converting to coo
--> 778         coo = self.tocoo()
    779         coo.sum_duplicates()
    780         self.__init__(coo.asformat(self.format))

~/.conda/envs/gpu_hack/lib/python3.8/site-packages/cupyx/scipy/sparse/csr.py in tocoo(self, copy)
    436             indices = self.indices
    437
--> 438         return cusparse.csr2coo(self, data, indices)
    439
    440     def tocsc(self, copy=False):

~/.conda/envs/gpu_hack/lib/python3.8/site-packages/cupy/cusparse.py in csr2coo(x, data, indices)
    928         _cusparse.CUSPARSE_INDEX_BASE_ZERO)
    929     # data and indices did not need to be copied already
--> 930     return cupyx.scipy.sparse.coo_matrix(
    931         (data, (row, indices)), shape=x.shape)
    932

~/.conda/envs/gpu_hack/lib/python3.8/site-packages/cupyx/scipy/sparse/coo.py in __init__(self, arg1, shape, dtype, copy)
    157                 raise ValueError('row index exceeds matrix dimensions')
    158             if col.max() >= shape[1]:
--> 159                 raise ValueError('column index exceeds matrix dimensions')
    160             if row.min() < 0:
    161                 raise ValueError('negative row index found')

ValueError: column index exceeds matrix dimensions

I believe this is a duplicate of https://github.com/cupy/cupy/issues/3513 . Could you confirm this is the same issue and we don't still have a solution for that @leofang @takagi ?

No, unfortunately I didn't have the bandwidth to work on a fix. I believe it's still an open issue.

Actually I think it's my mistake. It isn't the same issue, I probably mixed things up. However, I executed run() in a loop for 10k times and wasn't able to reproduce the issue, from the example in the description it seems like it should fail much more frequently, could you confirm that @christinahedges ?
For reference, this is CuPy's info on my side:
$ python -c 'import cupy; cupy.show_config()'
OS                           : Linux-4.15.0-76-generic-x86_64-with-glibc2.10
Python Version               : 3.8.12
CuPy Version                 : 9.4.0
CuPy Platform                : NVIDIA CUDA
NumPy Version                : 1.21.2
SciPy Version                : None
Cython Build Version         : 0.29.24
Cython Runtime Version       : 0.29.24
CUDA Root                    : /datasets/pentschev/miniconda3/envs/ucx-111-112-21.10.210921
nvcc PATH                    : None
CUDA Build Version           : 11020
CUDA Driver Version          : 11020
CUDA Runtime Version         : 11020
cuBLAS Version               : (available)
cuFFT Version                : 10400
cuRAND Version               : 10203
cuSOLVER Version             : (11, 0, 2)
cuSPARSE Version             : (available)
NVRTC Version                : (11, 2)
Thrust Version               : 101000
CUB Build Version            : 101000
Jitify Build Version         : <unknown>
cuDNN Build Version          : None
cuDNN Version                : None
NCCL Build Version           : None
NCCL Runtime Version         : None
cuTENSOR Version             : None
cuSPARSELt Build Version     : None
Device 0 Name                : Tesla V100-SXM2-32GB
Device 0 Compute Capability  : 70
Device 0 PCI Bus ID          : 0000:06:00.0
Device 1 Name                : Tesla V100-SXM2-32GB
Device 1 Compute Capability  : 70
Device 1 PCI Bus ID          : 0000:07:00.0
Device 2 Name                : Tesla V100-SXM2-32GB
Device 2 Compute Capability  : 70
Device 2 PCI Bus ID          : 0000:0A:00.0
Device 3 Name                : Tesla V100-SXM2-32GB
Device 3 Compute Capability  : 70
Device 3 PCI Bus ID          : 0000:0B:00.0
Device 4 Name                : Tesla V100-SXM2-32GB
Device 4 Compute Capability  : 70
Device 4 PCI Bus ID          : 0000:85:00.0
Device 5 Name                : Tesla V100-SXM2-32GB
Device 5 Compute Capability  : 70
Device 5 PCI Bus ID          : 0000:86:00.0
Device 6 Name                : Tesla V100-SXM2-32GB
Device 6 Compute Capability  : 70
Device 6 PCI Bus ID          : 0000:89:00.0
Device 7 Name                : Tesla V100-SXM2-32GB
Device 7 Compute Capability  : 70
Device 7 PCI Bus ID          : 0000:8A:00.0

It's possible this is a bug that was fixed sometime after CUDA 11.0, since I'm running 11.2. If you believe this is an issue that should be reproducible in fewer than 10k iterations, would it be possible for you to try and match the versions I'm running above to see if the issue is resolved?

I've confirmed that cupy-cuda110==9.4.0 and cupy-cuda111==9.4.0 reproduce the error, while cupy-cuda112==9.4.0 doesn't.

That's interesting, so it may have been a CUDA bug prior to 11.2. Would you be able to test somewhere with CUDA 11.2 @christinahedges ?

Hi all thanks for getting back to me about this. It will take me a while to get this tested on 11.2 (I run CUDA on a remote computer and need to check my permissions to update CUDA and get the right version). @dmargala was our mentor on our CUDA project. @dmargala, are you able to run this check on CUDA 11.2?

@christinahedges I tested the example code on the Perlmutter system at NERSC using the cudatoolkit modules available on that system. I can confirm this fails in 11.0 and seems to work starting in 11.2 (there does not seem to be a cudatoolkit module compatible with 11.1).

cudatoolkit module
cupy package
result

cudatoolkit/21.3_11.0
cupy-cuda110==9.4.0
fail

cudatoolkit/21.3_11.2
cupy-cuda112==9.4.0
pass

cudatoolkit/21.9_11.0
cupy-cuda110==9.4.0
fail

cudatoolkit/21.9_11.4
cupy-cuda112==9.4.0
pass

cudatoolkit/21.9_11.4
cupy-cuda113==9.4.0
pass

cudatoolkit/21.9_11.4
cupy-cuda114==9.4.0
pass

Thanks @dmargala that looks fantastic!

Let me close this issue as it seems to work on newer CTK. Feel free to reopen if anything happens again.


[Dashboards] ref OpenSearch Dashboards 1.3.0

Description
Update reference to point to OpenSearch Dashboards 1.3.0 manifest to point to "1.3" branch
Signed-off-by: Kawika Avilla<EMAIL_ADDRESS>Issues Resolved
n/a
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #1725 (2365a4f) into main (acc3d5f) will decrease coverage by 0.14%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main    #1725      +/-   ##
============================================
- Coverage     94.78%   94.63%   -0.15%
  Complexity       17       17
============================================
  Files           169      169
  Lines          3525     3525
  Branches         26       26
============================================
- Hits           3341     3336       -5
- Misses          181      186       +5
  Partials          3        3

Impacted Files
Coverage Δ

src/system/temporary_directory.py
83.87% <0.00%> (-12.91%)
:arrow_down:

src/system/os.py
92.30% <0.00%> (-7.70%)
:arrow_down:

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 acc3d5f...2365a4f. Read the comment docs.


Added in some basic information about Elixir.

Complete with links to more detailed information.

@Egogre Thanks so much for the PR. I have added a few minor comments.
@kytrinyx are we including "ABOUT.md" in the standard list of docs? I was looking for an issue related to this and found https://github.com/exercism/x-common/issues/69 which does not list that file.

Ah, shoot. That issue is out of date. I'll fix it. Yeah, ABOUT.md is what is showed at the very top of the language track page on exercism.

@parkerl, I have changed things around a bit per your request. Let me know what you think. Your quick response and helpful suggestions are greatly appreciated.

Thank you for the comments, @kytrinyx. Is this closer to what you would like, @parkerl? I certainly think it flows better based upon both of your help!

I think this looks really good now. @parkerl what do you think?

@parkerl want me to squash all the commits?

@egogre that would be awesome. Sorry I have been awol on this. Out sick.

squashed. I hope you are feeling better, @parkerl. Thank you and @kytrinyx for being very proactive and helping me commit something we all feel good about.

@egogre this is great. Thanks so much. :tada:


Rename VirtualMachineCRCErrors to VMStorageClassWarning

What this PR does / why we need it:
Pending https://github.com/kubevirt/ssp-operator/pull/875
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[ ] Design: A design document was considered and is present (link) or not required
[ ] PR: The PR description is expressive enough and will help future contributors
[ ] Code: Write code that humans can understand and Keep it simple
[ ] Refactor: You have left the code cleaner than you found it (Boy Scout Rule)
[ ] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
[ ] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
[ ] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
[ ] Community: Announcement to kubevirt-dev was considered

Release note:

Rename VirtualMachineCRCErrors to VMStorageClassWarning

/hold

actually this needs to get merged before https://github.com/kubevirt/ssp-operator/pull/875, so E2E tests pass there
/unhold


Update KMS client setup keys page

Added Windows Server, version 1803 client keys. Please check and update the client keys for Windows 10, version 1803.

@jaimeo - Do you have the client keys so we can add those prior to publishing this to the server library?

Merging now - will add client keys in private.


fixed issue : #159

fixed by draging down the select langauge

aligined the text to center

Great Job @gurramkarthiknetha .


How to query across several relations?

Assume we have:
A hasMany B
B hasMany C
C hasOne D

Now we have A object. And I want to query D with limit 10 using innerJoin.
something like:
let ds = a.$relatedQuery('bs.cs.d').limit(10)

But it doesn't work.
How can I do it? Thanks.
ATM, here is my best try:
let cs = await a.$relatedQuery('bs')
  .joinRelation('cs.d')
  .eager('cs.d')
  .offset(offset)
  .limit(limit)
  .pluck('cs')
let ds = []
for (let c of cs) {
  ds.push(c[0].d)
}
console.log(ds)

Assume that particular A has then B's and B has 10 C's and each C has one D then the the query:
let cs = await A.query()
  .findById(a.id)
  .joinRelation('bs.bs.d')

will have 100 rows. Offsetting and limiting will not work as you'd expect.
Why are you joining here? You are not filtering on the joined rows in any way? Would eager be enough? Leaving out joinRelation alltogether should work.

Maybe this is what you want?
let ds = await A.query()
  .findById(a.id)
  .select('bs:cs:d.*)
  .joinRelation('bs.cs.d')

that should kind of work. It will return instances of A, but with D's fields. I should actually implement a feature that lets you select the model class of the result rows.

As I am writing methods in the A model, do i would expect to query D directly using A relations. I think ORM should all depend on relation instead of querying from the top model class using static methods.
in the current implementation, it's hard to query the cross relations model directly and applying limit and offset there. I have to query all B and the all C and then all D step by step. whould there be a better way in the future to accomplish this?

Since I don't know what you want, here's a couple of suggestions you can try out. Maybe one of them has the results you want.
    // With this one you need to understand what an SQL join means.
    // The result set will be the cartesian product of the relations.
    let ds = await this.$relatedQuery('bs')
      .joinRelation('cs.d')
      .select('cs:d.*)
      // Eager won't work here since it will load relations for
      // all the numB * numC * numD rows.
      .offset(offset)
      .limit(limit);

    // This will limit and offset Bs and not Ds.
    let bs = await this.$loadRelated('bs.cs.d')
      .offset(offset)
      .limit(limit);

  let ds = bs.map(b => b.cs.map(...)))

@koskimas For my previous comment, pool should be this, just a typo mistake. And for your suggest, the first approach is what I want. I will try that, thanks. Objectionjs is the best tool I have used I believe.

@koskimas I have tried select, just I noticed that the result json will be parsed using A.$parseDatabaseJson instead of using D.$parseDatabaseJson, so my attributes conversion will be failed. Any ideas to fix this?


Cherry pick a number of improvements and cleanup

Codecov Report

Merging #31 (35111e2) into master (7d27e40) will decrease coverage by 1.98%.
The diff coverage is 52.38%.

@@            Coverage Diff             @@
##           master      #31      +/-   ##
==========================================
- Coverage   84.33%   82.35%   -1.99%
==========================================
  Files          21       23       +2
  Lines         945     1003      +58
==========================================
+ Hits          797      826      +29
- Misses        148      177      +29

Flag
Coverage Δ

unittests
82.35% <52.38%> (-1.99%)
:arrow_down:

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

Impacted Files
Coverage Δ

supergsl/plugins/sbol/sbol_output.py
25.00% <25.00%> (ø)

supergsl/core/parts/part.py
84.21% <83.33%> (-1.51%)
:arrow_down:

supergsl/core/parts/prefix_part.py
82.25% <88.88%> (+1.90%)
:arrow_up:

supergsl/core/constants.py
100.00% <100.00%> (ø)

supergsl/plugins/sbol/__init__.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


luarocks add sqlite

luarocks-check-hook: init
luaPackages.sqlite: init at v1.2.2-0

Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

I've run nixpkgs-review for a while which helped me find an issue. after that it looked ok.


Simplify request ip and port retrieval logic

This change also ensures that cases where transport stream is already closed is handled gracefully.
The previous change to handle IPv6 (#1053) did not account for the case where the the stream was already closed and self.transport.get_extra_info('socket') returns None. This works around the issue by avoiding the need for socket retrieval.
Note that the exception was only triggered in a scenario where debug was enabled and the request ip was accessed when uvloop threw a RuntimeError.

@abn thanks for this - can you add a unit test to make sure we don't end up with a regression somewhere?

@sjsadowski sure thing

Codecov Report

Merging #1353 into master will increase coverage by 0.22%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1353      +/-   ##
==========================================
+ Coverage    81.2%   81.42%   +0.22%
==========================================
  Files          17       17
  Lines        1681     1674       -7
  Branches      321      319       -2
==========================================
- Hits         1365     1363       -2
+ Misses        248      244       -4
+ Partials       68       67       -1

Impacted Files
Coverage Δ

sanic/request.py
75.53% <100%> (+1.68%)
:arrow_up:

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 5cc12fd...087af64. Read the comment docs.


Stats regarding time spent on each project

Is there any (easy) way to see stats concerning the time spent on each project ? I was curious to know if we can have access to such data, to mainly see how I manage my schedule.

Hi Mathias, yes it should be possible and not too complicate, there's supposed to be an API to the site.
I'll look into it this week, I need to give that website some attention anyway!

So by downloading the csv from the small icon on the tickets list page you should be able to look at the time spent (via  commit ac790e4)

if it's enough for your needs, you can close this


IE Error : SCRIPT1003: Expected ':'

I am trying to use this to count the number of pages in a tiff document. This works fine in Chrome & FF but IE gives an error. When I googled I found that the object notation should be proper({key: value}) for IE. The IE debugger points to this line:
const tags = {
standard,
exif,
gps
};
Can this be changed to :
const tags = {
standard: standard,
exif: exif,
gps: gps
};

The README says that the library is written in ES6, so you'll need to transpile the source to make it work in older browsers.


Resolve comment rendering escaped strings in email notification

Fixes #11143 Resolve comment rendering escaped some characters in email notification by updating the updated_comments.txt template.
Output in Terminal After Changes
Hello rohit,

Testuser has updated comments on "Bread and Circuses".

New replies:

  New replies to: "good job"
   - "<>'"

You can edit the page here: http://localhost:8000/admin/pages/68/edit/

Edit your notification preferences here: http://localhost:8000/admin/account/#tab-notifications

Please check the following:

[x] Do the tests still pass?[^1]
[x] Does the code comply with the style guide?

[ ] Run make lint from the Wagtail root.

[ ] For Python changes: Have you added tests to cover the new/fixed behaviour?
[ ] For front-end changes: Did you test on all of Wagtail’s supported environments?[^2]

[ ] Please list the exact browser and operating system versions you tested:
[ ] Please list which assistive technologies [^3] you tested:

[ ] For new features: Has the documentation been updated accordingly?

Please describe additional details for testing this change.
[^1]: Development Testing
[^2]: Browser and device support
[^3]: Accessibility Target

Thankyou. I think something like this should also have a unit test added, would you be ok for that @rohitsrma ?

@lb- Sure, could you please let me know in which file should I add unit tests?

Haha. I know you'd ask that.
I'm not.100% sure - maybe others on the core team will chime in.
Maybe https://github.com/wagtail/wagtail/blob/main/wagtail/tests/test_comments.py - might need a new class added there.
You can see an example of a test that checks the contents of an outgoing (outbox) notification here. https://github.com/wagtail/wagtail/blob/59b07f777f3468b5c4413c2e60a60af15587c5bb/wagtail/admin/tests/pages/test_moderation.py#L314 (moderation is being removed but it's still a good reference for checking the outbox.

@lb- I have made an effort to write unit tests for creating a comment and for adding a reply to the comment. However, I faced an issue while attempting to verify the email notification functionality (not getting anything in mail.outbox). I have included the code for the email notification test below:
def test_email_notification_on_reply(self):
        revision = self.page.get_latest_revision()

        # Create the main comment with the main user
        comment = self.create_comment(revision, self.moderator_user)
        self.assertIsNotNone(comment, "Comment not created successfully")
        self.assertEqual(Comment.objects.count(), 1)  # Check if the comment is created in the database

        # Add a reply to the previous comment with a different user
        reply_comment = self.create_comment(revision, self.reply_user, parent_comment=comment)

        # Check for the email notification
        self.assertEqual(len(mail.outbox), 1)  # Check if one email is sent
        self.assertEqual(mail.outbox[0].to<EMAIL_ADDRESS> # Check if the email is sent to the correct recipient
        self.assertIn(comment, mail.outbox[0].subject)  # Check if the subject contains the comment text

I would greatly appreciate any suggestions or insights you may have to help me resolve this issue.

Merged in via 42eb8ad82f6273a7670e29a96fe2048e6f475866 (5.2) and 6ef8da04eb9141d8bc43bcbbce65d9d80f4dac2d (main)

https://github.com/wagtail/wagtail/issues/11192 raised for better testing of non-escaping (or investigating a different approach).

Thanks @lb- I'll do some research and try to work on adding unit tests for this!


Update afl-replay.c && aflnet-replay.c

check the exists of the packet file in afl-replay.c && aflnet-replay.c

Thanks @Curator-Kim for your contribution to AFLNet! Your PR is now merged.


build error - unknown project.Context field 'ComposeFile' in struct literal

go build .

_/compose2kube
./main.go:47: unknown project.Context field 'ComposeFile' in struct literal

go version
go version go1.5.3 linux/amd64

This is due to docker/libcompose changes

Now i get
# _/home/hoeghh/Development/compose2kube
./main.go:88:3: error: range clause must have array, slice, string, map, or channel type
   for _, port := range service.Ports {
   ^
./main.go:57:2: error: range clause must have array, slice, string, map, or channel type
  for name, service := range p.Configs {
  ^
./main.go:57:6: error: invalid type for range clause
  for name, service := range p.Configs {
      ^
./main.go:57:6: error: invalid type for range clause


alternative mapping for react-native

I'm trying to use this with react-native, but with additional shims for fs, dgram, dns.js and net. Right now I'm using a fork of this repo (and a fork of webpack because peerDependencies doesn't like git urls), but I thought maybe you had a better idea.

maybe that can be an option in webpack config - the ability to provide an alternate node-libs-browser?

You can provide alternatives via resolve.alias option.


Added my Custom Environment tutorial

As I understand this is just self-promotion for a completely new video that wasn't discussed/requested earlier, which (from a cursory glance) itself has some bad practices w.r.t. the code? I think this can be closed.

@RedTachyon You assume it was not discussed before when in fact I was asked to pull this request by PseudoRnd

A PR description would be useful then. Point still stands that the tutorial doesn't follow best practices, so it can't be accepted.

at least tell me what's so wrong about this tutorial that it can't be accepted and how can I fix it

@GravermanDev Its awesome that you are into Gym and video development!
Here are a few concrete problems with the video:

Lack of Pythonic variable names
Using random.randint instead of self.np_random
Not seeding the environment
Ignoring the return_info argument in reset and ignoring metadata

Overall, the video is not quite up to the professional standards of the other tutorial/gym in general.
Keep learning and you'll get there :)

@trigaten thanks for the feedback, I don't have much time right now but I will address those criticism and make another pull request with better video


Fixup Jackson 2.8 deprecations

There remains a couple more deprecations in the yaml package where Jackson's MarkedYAMLException and Mark have been deprecated in favor of the underlying snakeyaml implementation (see https://github.com/FasterXML/jackson-dataformat-yaml/issues/63), but Jackson still throws the deprecated exceptions. Hence not all deprecations have been removed as they are still needed.
cc @cowtowncoder

Coverage remained the same at 82.276% when pulling ca4d5f7d794a32710582a83e1a2d208420f626fb on nickbabcock:jackson-2.8-deprecations into 89ab17b88345e76f5b35eb1fc526679235e2dc6b on dropwizard:master.


build: add tsickle as explicit dependency

Adds tsickle as explicit dependency. This is necessary because
we access it from node_modules/tsickle in a Bazel target that
wraps tsc with tsickle.
In the past we just relied on the transitive dependency of
@angular/bazel being hoisted, but the hoisting is not guaranteed
if the lock file changes (e.g. when running yarn upgrade). In general,
dependencies we explicitly access, should be added as explicit dependencies.

@crisbeto The lock file doesn't change. This is a pure package.json change that ensures that tsickle is hoisted in node_modules. The lock file doesn't update as the version range already matches the version range of tsickle brought in by @angular/bazel.
<EMAIL_ADDRESS>


Consider the case that hook.params maybe undefined or null

Refering to https://github.com/feathersjs/feathers/issues/1001
https://github.com/feathersjs/authentication-client/blob/5bcf78e3c96ea9f7d56d1ba5df5d473ce845a912/lib/hooks/populate-access-token.js#L9

This should not happen with the latest installed version (https://github.com/feathersjs/feathers/issues/1001#issuecomment-423228192)


Make a Variable cursor focus

I finally had to log this because it's been bugging me so much...
Sometimes when you click to Make a Variable and the dialog pops up, it doesn't capture the cursor focus, so I end up typing my variable names into the Toolbox Search bar instead!

Should be fixed in beta with https://github.com/microsoft/pxt/pull/7448 ~

fixed

fixed


Collapsible JSON Viewer

Contents
The PR promotes the json extension to a React project and installs the react-json-tree as the default viewer and resolves #1566 based on the acceptance criteria listed within it.
NOTE This PR does not attempt to combine the functionality of the Bot State viewer since it would present too large of a work unit. This work will be done in a separate PR.
Themes
Themes are supported using the built in base16 theming and have small differences in color treatment as a result of the way this theming system groups the colorization of data types.
Light

Dark

High Contrast

A11y
a11y, including keyboard navigation, is implemented manually via mutation observers since there does not appear to be out of the box support with the component dependency.
For Reference: this DOM structure was duplicated for this control.

Although I am indifferent to expanding all nodes by default. Most tools showing a hierarchy do not expand all nodes by default. In the spirit of improving UX with the introduction of this control, we may not have enough data to make this decision.
I can say that the authors of the widget chose not to. If I were to speculate, this could be because most users prefer to see only the immediate child nodes and a sensible default configuration took this into consideration.

This is purely anecdotal, but all the JSON viewers I've used tend to initially show all nodes collapsed by default.

First 3 results from google using the followng JSON:
{
  "outer1": {
    "inner1": {

    }
  },
  "outer2": {
    "inner2": {
      "inner-inner1": {

      }
    }
  }
}

Should the focus border color in High Contrast be orange to match the rest of the app?

having the children initially collapsed sounds reasonable. if we get feedback we can always revisit.

Theme names were coming in as sentence case but the css was expecting lowercase. I made the correction.
Thanks @tonyanziano for the good catch!


support uvicorn reload functionality

Autoreload is a pretty handy feature when developing. I changed the server.py to include this functionality and added a dev script in the Makefile

Codecov Report

Merging #1841 (3cd3fb6) into main (7f0c713) will decrease coverage by 6.41%.
Report is 5 commits behind head on main.
The diff coverage is 16.66%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main    #1841      +/-   ##
==========================================
- Coverage   55.49%   49.08%   -6.41%
==========================================
  Files          47       48       +1
  Lines        7300     7328      +28
==========================================
- Hits         4051     3597     -454
- Misses       3249     3731     +482

Files Changed
Coverage Δ

lnbits/server.py
41.30% <16.66%> (-2.60%)
:arrow_down:

... and 20 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Open With Function

Whenever I double click on the HTML file, it opens up Safari. Me, being annoyed by this, decided to right click --> Open With --> Other. I then choose Brackets, and there is an option to always open files like this using Brackets so I check the box, and the next time I double click the HTML file it takes me to Safari again. Is this a Mac problem?
Thanks,
CEOofApple

I believe the "Always Open With" checkbox only affects that one file, index.html.  If you want to open all HTML files in Brackets, I think you need to right click > Get Info > Open with > Change All.  See "Changing the Default App" in this tutorial.

Thank you very much. It worked


Add SSL setup to shell and web

ssl-pico.key.vault_encrypted should obviously be encrypted by ansible vault

Note this presumes an existing SSL cert, this does not cover #22 or #156


Add Typescript typings

I am not able to call spotifyApi functions. Maybe missing typings I think.
Need help with this urgently
Thanks in advance

This works fine in TypeScript...
import { SpotifyWebApi } from "spotify-web-api-node";

const spotify = new SpotifyWebApi();
const searchResult = spotify.searchArtists('Pete Tong');

I will change this issue title so it shows that this is a feature request rather than a bug.

Was poking around in the referenced issues; this one can probably be closed since this was published?

@ARoundSquare thanks for cleaning up. I'll close the issue.

Can we access the interaces too ? I am currently not able to do this with ts v3.8

@thibleroy the interfaces are not exported from the type definition file. That's a shame, I would like to use them too.

@thibleroy the interfaces are not exported from the type definition file. That's a shame, I would like to use them too.

I can access the interfaces by the namespace SpotifyApi

@thibleroy Thanks! Indeed I can access interfaces such as SpotifyApi.TrackObjectFull without importing anything else!


Dev

After creating your pull request, tick these boxes if they are applicable to you.

[ ] I have tested my changes against the review branch (the latest developmental version), and this pull request is targeting that branch as a base
[ ] I have tested my changes on Python 3.8 or higher
[ ] I have ensured my code is formatted using Black

Description
Related issues (if applicable)

Incorrect merge request...


Toggle to show avatars in individual chats

No content

Issue created by BlueBubbles Bot in the BlueBubbles commnity on behalf of @zach in #requests.

Make it togglable

This is going to be an option for all chats

This shouldn't be too hard, but it touches a few parts.

Add the settings toggle
Add the condition to the widget
Make sure everything is still aligned correctly


Unable to export an ES6 getter

The following file works under advanced compilation with the class and the method foo() properly exported (e.g. shadowed):
goog.provide("namespace.SomeClass");

/**
 * @export
 */
namespace.SomeClass = class {

  /**
   * @param foo {string}
   * @export
   */
  constructor(foo) {
    /**@private @type {string} */
    this._foo = foo;
  }

  /**
   * @returns {string}
   * @export
   */
  foo() {
    return this._foo
  }
};

However, if the foo method is changed to a ES6 getter as such:
goog.provide("namespace.SomeClass");

/**
 * @export
 */
namespace.SomeClass = class {

  /**
   * @param foo {string}
   * @export
   */
  constructor(foo) {
    /**@private @type {string} */
    this._foo = foo;
  }

  /**
   * @returns {string}
   * @export
   */
  get foo() {
    return this._foo
  }
};

The compilation will fail with the following errors:
SomeClass.js:6: ERROR - @export only applies to symbols/properties defined in the global scope.
convergence.Other = class {
                    ^

SomeClass.js:21: ERROR - @export only applies to symbols/properties defined in the global scope.
  get foo() {
      ^

2 error(s), 0 warning(s), 94.2% typed

I am not sure it is related, but when using the getter syntax, with the output set to ES5, the code that is transpiled for the Class makes use of Object.defineProperties.  So the foo method is now generated inside code like this:
function k(e){this.b=e}
Object.defineProperties(k.prototype,{a:{configurable:!0,enumerable:!0,get:function(){return this.b}}});

Here k is the renamed SomeClass.  'this.b' is the renamed 'this._foo'.  Object.defineProperties is creating 'a' on the prototype and setting the getter to return 'b' (_foo).  It seems like when the code is transpiled we lose the ability to shadow the property for an export.  I would have expected to see something like:
k.prototype.foo = k.prototype.a;

That error message is not great, to say the least. @export is allowed there, if you pass --export_local_property_definitions to the compiler.

With that flag, this should work. We have tests for it, even: https://github.com/google/closure-compiler/blob/master/test/com/google/javascript/jscomp/IntegrationTest.java#L1713-L1747

@MatrixFrog it feels like error message should be changed; I stumbled here only after googling it.


Can not refresh if there are no events

Description of issue
If contentful has no data, the app gets into a state where the only way to retrieve events is to remove the app and install again. The app just shows the loading text on the event listing screen.
Expected behaviour
Can refresh events.
Steps to reproduce
Load the app when contentful has no data. Try to refresh...
OS / Browser / Device model
Tried on Android, but I am pretty sure this affects all platforms.

Related to #110.
Did we decide how refresh should work in general?

Will cover in https://trello.com/c/VLDK9GUW/196-refreshing-content


floem found no Unfilled circle

When I attempt to add plotters, I need Unfilled circle, which is not found in floem. When I add stroke_arc in the local floem stroke, this function can be realized, but I have no permission to push it to the main warehouse

You can create a rounded rectangle and set the color to transparent and set a border or outline to get the same effect

This solution can achieve the same effect, if you want to complete the circular progress bar, it may be difficult, kurbo library stroke arc can be solved very well

but I have no permission to push it to the main warehouse

What do you mean by this?

I want to push code to floem

You can create a Pull Request if you want to add any changes
https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/creating-a-pull-request

Thank you very much


Add support for Python 3.12

I've run tests in py312 tox env locally.

Fixes #26

It seems mypy failed :/


TRAIN_FULL.md Part 3, Step 3

Following the instructions, I tried the command:
python -m tools.run_nearest_neighbours \
    --config-file configs/LABEL-Verification/dino_label_verification.yaml \
    --num-gpus 2 \
    --eval-only \
    --opts \
    DATASETS.DT_PATH "('checkpoints/coco/faster_rcnn/faster_rcnn_R_50_FPN_ft_all_30shot_aug_ftmore_dropout/inference/coco_instances_trainval_results_score_max10_min08_full_all.json',)" \
    DATASETS.NN_DSET "('coco_trainval_all_30shot',)" \
    QUERY_EXPAND.KNN 10 \
    OUTPUT_DIR "checkpoints/coco/faster_rcnn/faster_rcnn_R_50_FPN_ft_all_30shot_aug_ftmore_dropout"

The difference from the original command of Part 3, Step 3 in TRAIN_FULL.md is "--num-gpus 2".
Then, I got the error:
ValueError: Type mismatch (<class 'list'> vs. <class 'str'>) with values ([] vs. ('checkpoints/coco/faster_rcnn/faster_rcnn_R_50_FPN_ft_all_30shot_aug_ftmore_dropout/inference/coco_instances_trainval_results_score_max10_min08_full_all.json')

So, I modified the following three points.
(1) I modified the command as:
python -m tools.run_nearest_neighbours \
    --config-file configs/LABEL-Verification/dino_label_verification.yaml \
    --num-gpus 2 \
    --eval-only \
    --opts \
    DATASETS.DT_PATH "('checkpoints/coco/faster_rcnn/faster_rcnn_R_50_FPN_ft_all_30shot_aug_ftmore_dropout/inference/coco_instances_trainval_results_score_max10_min08_full_all.json',)" \
    QUERY_EXPAND.NN_DSET "('coco_trainval_all_30shot',)" \
    QUERY_EXPAND.KNN 10 \
    OUTPUT_DIR "checkpoints/coco/faster_rcnn/faster_rcnn_R_50_FPN_ft_all_30shot_aug_ftmore_dropout"

The difference is "DATASETS.NN_DSET" -> "QUERY_EXPAND.NN_DSET".
(2) I modified configs/LABEL-Verification/dino_label_verification.yaml as
DT_PATH: ('checkpoints/coco/faster_rcnn/faster_rcnn_R_50_FPN_ft_all_30shot_aug_ftmore_dropout/inference/coco_instances_trainval_results_score_max10_min08_full_all.json',)

Here, I added the comma and the closing parenthesis ",)".
(3) I modified lvc/config/defaults.py as
_CC.DATASETS.DT_PATH = ()

Here, I changed "[]" -> "()".
Are these modifications correct?

Yes this is correct. I will update the scripts.


host_nsupdate returns 2

We are updating to an AD DNS server. The first time this cookbook runs it always errors out with a code of 2.  on subsequent runs it is successful, as the DNS actually was updated the first time.  I have searched high and low and cannot find any lists of error codes.  Any Ideas of what is going on?  Do we need to allow a return code of 0,2 ?

Could you please share the client-run output too?

@vkhatri this problem is very hard to reproduce and its a bit random. I'll try to do it again and paste here as soon as i occur

Sure.
A quick fix would be to add another attribute retries and set the default value to 2?
Unless, it turned out to be a code fix, this will at least fix the random errors.

Sorry, have changed companies. So can't reproduce now. retries will probably be sufficient fix for now.   I suspect that there are multiple AD Servers and that one of them does not allow for unsecured DDNS.


Deg user

Modify a view of deg user

Coverage remained the same at 100.0% when pulling 6ebb20e24cceee5a0748a5d2e3a1a3a1e2017ee3 on deg_user into f40cff2cf67fe36361ba022b774cb4e68e30e13e on development.


Add fork & star buttons to header

Add fork & star ob github buttons under the logo. The buttons should be similar to the github buttons.

Should these buttons be corresponding to the eventBuddy github repo or ?

@crod93 Please checkout other issues. Thanks :)


AMPI topologies should find neighbor ranks during construction

Original issue: https://charm.cs.illinois.edu/redmine/issues/1086

Currently, AMPI finds the number of neighbors and their ranks within each neighborhood collective call. This is repetitive work for the common case of creating a topology once and reusing it many times for communication, and should be avoided. Instead, create a list of neighbors during topology construction, and use that during the communication routines. This is relevant for MPI_Cart and MPI_Graph (and MPI_Dist_graph once we implement that).

Original date: 2016-05-31 20:53:28

Fix is here: ~~https://charm.cs.illinois.edu/gerrit/#/c/1230/~~ https://github.com/UIUC-PPL/charm/commit/95e5ac0910e44b716fcf962b793fd11bd78f3027
Additional future work:

Replace CkVec's with std::vector's in AMPI
Improve error handling for virtual topologies/communicators:  tests/ampi/mpich-tests/topol/{dims,cart,cartorder} all fail currently
Refactor virtual topologies/communicators into virtual classes of a common type, rather than just having their members be members in all ampiCommStruct's


Failing to plot a simple List with 8760 points as a line chart

Hello guys,
I just came across this library, this is fantastic, thanks for the work! I've succeeded in creating really complex graphs with so few lines of code that now I want to convert all my graphs! Here goes some examples:

and this is all the code I wrote:
<ApexCharts.ApexChart TItem="Zone"> <ApexCharts.ApexPointSeries                                 TItem="Zone"                                 Items="building.Zones.Where(x => x.REVITOccupation > 0)"                                 SeriesType="ApexCharts.SeriesType.Pie"                                 XValue="@(e => e.REVITSpaceKey)"                                 OrderBy="@(e => e.X)"                                 YAggregate="@(e => e.Sum(e => e.REVITOccupation))"                                 />                         </ApexCharts.ApexChart>
That's incredible. My previous library would require me to write code in WASM and in C#.... Thanks again for the porting.
But the reason I'm writing is that I can't seem to be able to create the most simple kind of graph, a List of doubles. I am trying to do something like this:

and i was expecting the code to be as short as the last one, so I wrote the following:
<ApexCharts.ApexChart TItem="double"> 						<ApexCharts.ApexPointSeries TItem="double"<EMAIL_ADDRESS>												SeriesType="Apex.SeriesType.Line" 													YValue="@(e => e)" 													Name="Space Heating" /> 						</ApexCharts.ApexChart>
where the Item supplied is of type List<double>. I can't compile it, says that my YValue needs to be of decimal type.
I've looked in the documentation, but there are no examples of how to deal with a large dataset of 10.000 points, for example. Any help would be appreciated.

Hello,
Thank you for the feedback.
For a simple list question, you have to create a class thank you can use to populate the chart.
for example:
    private class ValuePoint     {         public decimal Value { get; set; }         public int Position { get; set; }     }
Just populate a list of ValuePoint to the chart.
You an easily convert an double to decimal.
For a performance example please check: https://apexcharts.github.io/Blazor-ApexCharts/features/performance
Please note that Apexchart is SVG based so each datapoint will be an SVG dom element. With many datapoints the browser will start to struggle.

Hello again,
There seems that I had a problem with the async part of the code, still trying to figure out how to fix that.
But for now I did a workaround, and the graph is taking 60 seconds to load with 8760 points. That is VERY different than the very interesting webpage you sent, where 10.000 points were loaded in under 2 seconds. Any idea why that is?

Hello,
What is included in the 60 seconds? Is this also taking into account loading the the data from an external source. In my example is when we already have the data in memory.

Hi Joadan,
Still running some tests here. This is what I'm trying to execute in the code:

3 graphs that are updating based on a selection from the table on the left. Each graph 168 points, and currently it's taking me around 5 seconds to refresh the graph. This is how I set up the stopwatch for performance evaluation:

So I divided the timer into three categories, data fetching, graph data reloading and option reloading. The performance is the following:

3 ms data fetching
3152 ms series updating
948 ms option updating

This is another test than the one I described above (that test does not exist anymore), but the results are very much in line with the performance I reported previously. 3 seconds data loading has quite some impact in the UX, and I see your example loading 20000 points in less time than mine, so I'd like to understand why that is. Any thoughts?

Hello,
In your sample you are measuring the time to update all three charts, so you should divide the number by 3.
It's hard to give any advice with the information you give me, for example what is going on in ConvertToTypicalWeekPoints?
If possible try to create a simple repo that illustrate the problem and I can have a look.

hey joadan,
My example is indeed dividing by 3, but I only had 168 points. If I would have 8760 points, with the same performance, it would be around 9 seconds per graph, which is still way too far than the performance.
I have the feeling that the problem is because I was using a listed numeric entries, with X-values ranging from 1 to 8760. When I passed it as a date, it sped up. I've seen that behavior, but I didn't have the time to run some tests. Will report as soon as I do it. And I will set up an example based on your code, to rule out any problems with the code on my side.

Hey there joadan,
It took me quite some time but, as promised, I got back to you, and this time with all the sauce so you can test it on your side.
I've created a little repo showing a page taking never less than 30 seconds to load a line graph with 8760 points. Here is the repo:
https://github.com/sweco-bepema/ApexChartTest

Hello,
Thank you for the repo.
The bulk of the time is for the browser to render the chart. it has nothing to do with the wrapper.
Apexchart is svg based this means that every object is an SVG node and if the number of nodes is high then it takes a long time for the browser to render the chart. I turned of XAxis labels and ticks in your sample and was able to get a much better response.
However I think you should really consider if you have to render so many points at the same time. It
hard for anyone to make something out of that chart. Would it be an option to group the data?


bump lib2p version

Updated libp2p and all related libraries to latest versions.

Closes #306 and #656