Fix (very) minor typo in docker installation instructions

The trailing slash in the curl params has no effect, but it threw me off
This PR goes hand in hand with https://github.com/Start9Labs/documentation/pull/322

I don't think I've ever seen this line cause issues... What behavior does this fix, exactly?

The main difference between using the -o- option with curl and not using it is that the -o- option downloads the script and outputs it to the standard output before piping it to bash, while not using the -o- option pipes the script directly to bash without outputting it to the standard output.
Neither option saves the script to the local file system, and because we're only interested in installing Docker and won't be processing the script further, it's safe to omit the -o- option.
@gStart9 @kn0wmad Let me know if Dev documentation won't be using -o- and I'll merge it.

Sounds good to me


Adding JCL (Job Control Language)

Description
Adding JCL (Job Control Language) - the grammar was already present in https://github.com/spgennard/vscode_cobol/ so I have only added it as a new language
Checklist:

[x] I am adding a new language.

[x] The extension of the new language is used in hundreds of repositories on GitHub.com.

Search results for each extension:

https://github.com/search?q=extension%3Ajcl+EXEC+DD+JOB&type=Code

[x] I have included a real-world usage sample for all extensions added in this PR:

Sample license(s): JCL isn't exactly unique as there's only so many ways to write it but all samples here are my own

[ ] I have included a syntax highlighting grammar:
[ ] I have updated the heuristics to distinguish my language from others using the same extension.

Whoops, just noticed you've not provided a colour. This means a boring dull grey will be used. Do you want a colour or are you happy with the boring grey?

Oops, I missed this (and the reply). There's no "official" colour as such so I've gone with  #d90e09


Update order of resources in the support.md

GitHub issues should not be the last thing we mention in the support doc after community resources.
This PR re-orders the resources in the order that we expect people to use

/azp check-enforcer reset

/check-enforcer evaluate

/check-enforcer override


Direct users to use https://github.com/GoogleCloudPlatform/cloud-trace-java

Would be nice if the README for google-cloud-trace package directed users use the Stackdriver provided libraries at https://github.com/GoogleCloudPlatform/cloud-trace-java. Also a pointer showed up in the Maven artifacts generated so someone searching for and viewing the maven artifact
https://mvnrepository.com/artifact/com.google.cloud/google-cloud-trace
would be directed to use Stackdriver SDK instead
https://mvnrepository.com/artifact/com.google.cloud.trace/core
Thanks

cc @omaray

I'd like to follow this conversation.

The short response to this issue is that I don't think we want to simply do a redirect. We have an internal discussion on how to reduce the confusion of multiple client libraries for trace. I'm going to close out this issue for now; when we come to a decision, we'll get it sorted out on github.

Just wanted to say that this is all very confusing from the perspective of someone trying to setup tracing. The docs all seem to put in different directions, with varying amounts of guidance on actually using this to capture traces.


demo: BAYC gini example

Adding bayc gini demo, adds demo folder, updates requirements and sets stage for the future...

Tests passed, I'm impressed


Alluxio Metrics Don't Report Bytes Written

Alluxio Version:
1.8.1, 2.0.0-RC2
Describe the bug
When we perform write operations using the CLI (or any other means), the Alluxio metrics for Bytes Written and Bytes Written UFS never get incremented in the master at /api/v1/master, /metrics/json as well as ./bin/alluxio fsadmin report metrics
To Reproduce
using MUST_CACHE as the write type:
./bin/alluxio-start.sh local
./bin/alluxio fs copyFromLocal LICENSE /LICENSE
./bin/alluxio fs persist /LICENSE
./bin/alluxio fsadmin report metrics

The metrics report will report no bytes written to Alluxio or the UFS
Expected behavior
The command-line metrics report and the REST endpoints should be reporting the bytes written to Alluxio and/or the UFS. Currently, it always seems to be 0.
Urgency
minor
Additional context
Reported via slack community

This is because persist is implemented through the job service which write the data through the job worker (as if it were a separate client) and not the Alluxio worker.
To fix this I think we would need to have metrics for job worker tasks and is more of a feature-request than a bug-fix.

Thanks for looking into this!
I would disagree in that as it stands, the metric name implies all bytes written to the UFS. It doesn't have any caveats saying it is worker-only vs job worker. I think this is misleading to the user. Under the current definition of the metric it seems like more of a bug.
Either way, thanks for clarifying this. To reiterate in my own words, in order for bytes written to the UFS to be reported, the user must use CACHE_THROUGH or THROUGH write types in order for the metrics to show up?
I'm not sure exactly how you tested either, but my "Bytes written to Alluxio" (not the UFS) also always remained at 0. Were you able to get that number to change?

Also, another note I forgot- when I tested I used the v1.8.1 tag that doesn't include the job service. Persist operations were still not incrementing the counter.

I can confirm that using write with CACHE_THROUGH updates metrics correctly for the UFSs (verison 1.8.1).

I think we both agree the metrics should be tracked, the question is whether there should be a separate set of metrics for the job service or if the job service should report metrics back to the master.
Yes, currently the metrics only track data that is written through the Alluxio worker.
I don't see a bytes written to Alluxio metric, but I didn't have any Alluxio write metrics because all my writes to Alluxio were short circuit.

Closing the issue - @calvinjia please open another issue to provide job service metrics.


Bootstrap 3 conflicts

Hi,
Thanks for the project, very cool!
I used it in combination with Bootstrap 3, and had to add the following CSS rules to get it to show correctly:
pre, code {
    background: #f8f8ff;
    font-family: monospace;
}

The background color here is not really important (it's the github theme), the thing is bootstrap applies a background to the <pre> and highlight.js to the <code>, so we can see a bit of the <pre> and its different background color.
I didn't do a pull request because maybe that has already been discussed.

i meet the issue also .....


only use the diversification nonce after authenticating a packet

This PR is wrong. We need the diversification nonce to unpack the packet containing the SHLO, so we can't try decrypting it without setting it before.


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-main" workflow

Codecov Report

Merging #788 (fbbcef8) into main (cc417de) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@           Coverage Diff           @@
##             main     #788   +/-   ##
=======================================
  Coverage   95.69%   95.69%
=======================================
  Files           1        1
  Lines         395      395
  Branches       53       53
=======================================
  Hits          378      378
  Misses         17       17


Choose vault for new note in a multi-vault environment

Is your feature request related to a problem? Please describe.
Currently a new note is created in the same vault as the currently open note. Thus I need to pay attention the current vault when adding notes.
Describe the solution you'd like
I'd like to be asked for the vault to use for a new note from the currently defined vaults. The current vault should be the default. In addition I'd like to have a config option to turn this behavior off.
Describe alternatives you've considered
Be more mindful of the currently active vault :)

shipped in 0.20


Video track publishing issues with VP9

Describe the bug
Issue Description
I was testing video codecs in livekit and I have encountered some problems with the video quality when using the VP9 codec. As you know, VP9 is an improved version of VP8, which offers better compression efficiency, quality, and support for high-resolution video. However in my tests livekit does not seem to work well with VP9, as I have experienced worse latency and FPS than with VP8.
In the first video below, you can see the result of one of the tests.
There are 4 clients in this test (3 web and 1 Android named "Android Me")
And the recorded video is from the web client called "Web Me".
In this test, two other web clients are in the same system and are sharing same camera device, but one with vp9 codec and the other with vp8 codec are publishing that video.
As you can see, the video received from the vp9 client has lower fps and more latency than the vp8 client.
https://github.com/livekit/client-sdk-js/assets/47795908/29e9ebe5-155a-4dae-8320-a42be68dc76b
You can also see the result of this test in the "Android Me" client:
https://github.com/livekit/client-sdk-js/assets/47795908/fa8f2a7d-04bd-4218-84af-7cbf37792fc2

android sdk version: 2.0.0 (origin/main)

server version(golang): 1.5.2

components-js: commit hash: 5fa2d6b33c591a5cf511c015d066546c983bf2aa) Update to livekit-client v2 (#733)

publish options for "vp9" client:

options: { publishDefaults: { videoCodec: 'vp9' } }

publish options for "vp8" client:

options: { publishDefaults: { videoCodec: 'vp8' } }

Reproduction
To reproduce this problem and check, just connect up to 2 web clients to the same camera and publish one with vp9 and the other with vp8.
Logs
No response
System Info
Web clients: Linux

Severity
annoyance
Additional Information
We performed a similar test for two Android clients, one publishing vp9 and the other vp8. The result was that we did not observe a similar problem. Therefore, we can probably conclude that this problem is from the js client or from the server side.

vp9 generally still needs more system resources to do the encoding. This might be related to bad encoding performance on linux as well, but hard to tell. If you don't experience the same issue when publishing from an android device it sounds definitely like the issue is more likely to be found on the publishing client's system where it performs worse.

we did some optimizations in https://github.com/livekit/client-sdk-js/pull/1069 as part of v2.0.7.
Closing this issue, but feel free to re-open if you still run into issues even after applying that update


On Dispatcher mode agent fails to load with undefined cordon definition.

agentConfigs='agent110config' without a uncordon/cordon setting fails to load in dispatcher.

@dmatch01 closing this old issue, please try your setup with the latest MCAD version.


Adds optional combinator, changes pipeline to required

Fixes #23.

if we're making a major change, this one seems worth including to me. Are you interested in getting the conflicts fixed and getting it up and running again?

Sorry, started looking into this, but didn't we already do https://github.com/BrianHicks/elm-csv/pull/30? This is sort of the same thing?

Yeah, that’s fair. Thanks for pointing that out. I was just looking over open PRs and liked the optional/required distinction here.
Ok to close then?

Yeah let’s close


Runtime assets seem not to work with DocumentHead

Hi Robi,
first I would like to thank you for this great contribution. I was thinking about building a translation library for Qwik myself, but then found yours.
Currently I face the issue that the title and the description of the DocumentHead seems not to be translated. I added the runtime.json to the runtimeAssets in the config, but it stills shows the placeholder.
Do I miss something here?
One side note I always hated about ngx-translate that it is not type-safe. I was thinking about what to add to qwik-speak to make that feasible. I have had an own solution for Angular that uses a distributed-translate-loader-approach. Maybe we could customize the useTranslate hook to give back a function t that expects a dot-seperated string which matches the keys from the json that is used in the current scope. Especially for large projects that might be a better solution. Drawback is that we could only use the default language, but we could implement an unittest that checks wether the structure of the jsons is equal over all languages.
What do you think?
Sebastian

Hi,
about the problem with DocumentHead, you can find a detailed step by step in tutorials, and in this repository a working example app: did you update the keys in DocumentHead to point your runtime file? Try to verify, and if it still doesn't work, please post the relevant code.
About your second question, I'm not sure I understand. Could you open a discussion  describing the problem and how it could be solved?
Thanks!

Hi Robi,
thanks for your answer.
My head looks like this:
export const head: DocumentHead = {
    title: 'runtime.page.largeCompany.title',
    meta: [
        {
            name: 'description',
            content: 'runtime.page.largeCompany.description'
        }
    ]
};

My config looks like this:
import type { SpeakConfig } from 'qwik-speak';
export const config: SpeakConfig = {
    defaultLocale: {
        lang: 'de-DE',
        currency: 'EUR',
        timeZone: 'Europe/Berlin'
    },
    supportedLocales: [
        { lang: 'de-DE', currency: 'EUR', timeZone: 'Europe/Berlin' },
        { lang: 'en-US', currency: 'USD', timeZone: 'America/Los_Angeles' }
    ],
    assets: [
        'app' // Translations shared by the pages
    ],
    runtimeAssets: [
        'runtime' // Translations with dynamic keys or parameters
    ]
};

Do I miss something here? I was looking into your example before. But I was not able to tell if I forgot to add anything.
Thanks
Sebastian

That's correct. Check also router-head.tsx and the route property of your  runtime file.

He thanks. It worked. I was missing that first part. To my other request. I am also not sure how we could make that work. But the useContext hook also infers the type. We could maybe somehow include the jsons in TypeScript to infer their type only in the dev mode and then use something like this:
https://www.typescriptlang.org/play?#code/C4TwDgpgBACghsAFgZwCoHsDKwBOBLAOwHMYd0xkAeVAGliggA9gICATZKZXQogbQC6UALxRBAPhFRUDZqw5cexAFBQoAfliqoALmmyW7TugBGAKwgBjYNs0BvbWrV8A0lEJQAosxxxrlAGsIEHQAM2k6bnxicQE9VFchJkMFUwtrDVgEFAxsaJIyCmpEuj4AOgqYOhcBST1yyuqBAG5HKABfPm9cP2BA4LCIxXzY7XqW5WVQSCgAKXRCagN5TijeQToAEWWjYd5JYW0ZZJWxIU0CCAA3CBxdI52FPkJQ26gAMXOP+7VjuV3ngRXnd3nQKmUXm8AErnbTvR6rJRETIAAwAJHZ3u0MZtsXZ5otur5-FDIkiJFtxO0UbooJcbncdNo1sRWlNwNAALIgTYQCBgADy5iswCkDjUoXQ6D04qcJjgOBlbTU8oAXnoWURWk4OtrddoAI4AV0YegIRoAtiZbnrEBAADb26VQWVqMgmPAapF69qtX2TabQXn8lzBKQEgiUeBINBYJGkchUbnBwXC6ziOgAIjKmfErSAA
Will leave that open so that you see my answer.

Moved to #56


Update MPI

As a patient
I want change my personal data in cabinet
So that information will be actual
business processes
specs

[ ] it should be forbidden to change first_name and last_name in cabinet (as they where compered with first_name and last_name in DS during sign-up)

[ ] 501 on trying to change number for a number that is not verified

{
    "meta": {
        "url": "http://api-svc.il/api/cabinet/persons/689a0f3e-65da-4955-b40e-25804c03a9de",
        "type": "object",
        "request_id": "218cf1d7-1542-40d7-a700-39056556ca40#393",
        "code": 501
    },
    "error": {
        "type": "internal_error"
    }
}


fix #82

Add support for NotNan and OrderedFloat in crate ordered-float.
fix #82

Any updates from maintainers? @sebcrozet @brendanzab


Missing translations

Issue Description:
Missing translations:
What happens:
The log reports missing translations when launching the mod.
What you expected to happen:
Clean launch.
Steps to reproduce:

Just launch any instance of the mod.
Watch at the log.
...

Affected Versions (Do not use "latest"):

EnderIO: EnderIO-1.10.2-<IP_ADDRESS>_beta
EnderCore: EnderCore-1.10.2-<IP_ADDRESS>-beta
Minecraft: 1.10.2
Forge: Any

Your most recent log file where the issue was present:
http://pastebin.com/uBULUb4H

unfinished software ins unfinished

Got you ;)

I'll probably just disable that warning. those keys don't really need a translation.


how to generate an RSA private key and a matching JWK?

Hello,
I am working to implement LTI 1.3 and I need help with the RSA key and JWK.
how to generate an RSA private key and a matching JWK for LTI 1.3 integration?

Did you look at https://github.com/IMSGlobal/lti-1-3-php-library#creating-a-jwks-endpoint?

Thanks.
Should I always generate a new RSA key for every new LTI 1.3 integration?
How can I generate an RSA key for LTI 1.3 integration?
Can you please confirm?

I wonder if providing a JWS URL is part of lti-php or not? On the one hand, lti-php supports LTI 1.3, and the chapter "Initialising a tool" in the documentation explains where a RSA key needs to be placed. On the other hand it seems that one needs something like https://gist.github.com/kmuenkel/37ead59ff01f56ee881744c11360b00c to tell the platform the public key. Is that correct or did I miss an important feature of lti-php?


How to inspect shapes of tensors

I'm troubleshooting a number of different problems in an experimental network implemented in keras with a theano backend. I'm relatively new to keras and theano (I did some small experiments a few months ago and are returning now, having forgotten most of what I learned then). Lots of my problems would be more transparent if I could inspect the shapes of tensors but try as I might I never find a human readable representation of shapes.
For instance, I have some issues of the "Exception: Input 0 is incompatible with layer c_out: expected ndim=2, found ndim=3" type. While I know that this is a problem with the number of dimensions rather than the size of them, knowing the sizes of the dimensions would help me understand why there is a mismatch and what each dimension contains. Basically, I'm saying that my mental model of tensors and how computations flow through keras/theano is still pretty fuzzy and I need all the help I can get :)
If I print a TensorVariable.shape attribute I get the string "Shape.0", this seems to be an lvector of subtensors but I can't seem to access the values. There is supposed to be a function keras.backend.int_shape(x) which returns the shape as a tuple of integers and/or Nones but when I try it I get  "AttributeError: module 'keras.backend' has no attribute 'int_shape'". The input to the c_out layer comes from a merge layer called merge_c, but when I call merge_c.output_shape or merge_c.get_output_shape_at(0) I get "AttributeError: 'TensorVariable' object has no attribute 'get_output_shape_at'".
Can anyone recommend a way to access a human readable representation of tensor shapes?

@Falkenjack Use keras.backend.shape(x) to get the shape of a tensor or use model.summary() to print the shapes of all of the layers in your model. Please ask usage questions on stackoverflow, slack, or the google group.
Cheers,
Ben

Looks like there are several methods? What is the difference?
	class_weights_list = [1.0]
	class_weights_numpy = np.array(class_weights_list)
	class_weights_keras = K.variable(class_weights_numpy)
	print('class_weights_keras.get_shape', class_weights_keras.get_shape)
	print('K.shape(class_weights_keras)', K.shape(class_weights_keras))

('class_weights_keras.get_shape', <bound method Variable.get_shape of <tf.Variable 'Variable:0' shape=(1,) dtype=float32_ref>>)
('K.shape(class_weights_keras)', <tf.Tensor 'Shape:0' shape=(1,) dtype=int32>)

Note the two ways of accessing shape via shape and int_shape. The first one gives the rank and the second one gives the dimension of the tensor.

k=Input(shape=(29,))
import keras.backend as bk
print(bk.int_shape(k))
(None, 29)

You can get your tensor shape as follow:
H, W, n_ch = tensor.shape.as_list()[1:]


[Darwin] Followup PR for missed code review comments

Fixes following issues:  #32438, #32443, #32439

Fast tracking as this is Darwin Only change


Replace Picard MarkDuplicates with ADAM

Include process that runs on Apache Spark for further analysis at CoFest 2019; work in progress.

Thanks @heuermh ! We need to get you added to the contributors too please 👍
https://github.com/nf-core/nf-co.re/issues/3#issuecomment-499076248

Thank you @drpatelh, created https://github.com/nf-core/nf-co.re/pull/142


swiftKuery to ver 1.0.0

Codecov Report

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

@@           Coverage Diff           @@
##           master      #37   +/-   ##
=======================================
  Coverage   73.42%   73.42%
=======================================
  Files           3        3
  Lines         444      444
=======================================
  Hits          326      326
  Misses        118      118

Flag
Coverage Δ

#SwiftKueryPostgreSQL
73.42% <ø> (ø)
: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 95b5378...732e2b3. Read the comment docs.

@Andrew-Lees11 This looks good, thanks for taking the time to make this change. One minor change before we can merge the PR: can you rename your branch to issue.oldKuery? That way we will have travis ci kick off a build for your PR and ensure it builds just fine on both platforms.


plugins 4xx错误

Same here. I cannot get the submission beat ratio neither.

@yunweiyuan @zhangyanj  可以先用这个 https://github.com/kadaliao/leetcode-cli-plugins/blob/fix-leetcode-cn/plugins/leetcode.cn.js 替换一下

@yunweiyuan @zhangyanj
You can use my workaround to replace the original code temporarily。

Download the file from here and rename it to leetcode.cn.js
cd into you file path
run leetcode plugin -i leetcode.cn.js

@yunweiyuan @zhangyanj
You can use my workaround to replace the original code temporarily。

    problem.fid = Number(fid) || fid
                         ^

ReferenceError: fid is not defined


Add Rust binding links

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Coverage remained the same at 99.256% when pulling 84b584c7bb67bcd88569bdf603718f18ffef7717 on rustyconover:master into dee9b9649664ed32df27317f65544341eeeae4a1 on uber:master.

Thanks for contributing! Can you explain the difference between the two repositories, and possibly include this in either the links or the repo READMEs?

@nrabinowitz Thank you.  I'm glad to help out.
The difference between the two repos is the Rust norm of having a -sys crate that has contains all of the unsafe Rust functions.  Unsafe functions are functions that call the C/C++ library directly.  The libh3 crate contains safe Rust functions, these functions call into libh3-sys but users call them from safe Rust code and don't have to worry about handling all of the vagaries of calling into the C library.
For the difference between safe Rust and unsafe Rust please see:
https://doc.rust-lang.org/nomicon/meet-safe-and-unsafe.html

I've also updated the module readme files.

Great, looks good! Thanks for the explanation and README updates - possibly this is so idiomatic that Rust developers would find it obvious, but I think it's quite helpful.


Support new architecture

Blocked by https://github.com/expo/expo/pull/28829

should be unblocked by now and I wonder if you have any plans to move towards fabric/jsi anytime soon? 👀

I'm planning on doing this soon. Last time I checked it wasn't working well, but I'll give it another shot soon. Also feel free to give it a shot if you'd like to contribute!

I'm actually planning to contribute and was wondering how you were doing. The only thing that holds me back a little bit is the fact that writing Fabric Native Components in Swift is not considered a first class citizen atm. You're expected to expose your Swift code (UIView etc.) over to ObjC/C++ (XXX-Swift.h) and basically do bindings back and forth manually. You also shouldn't rely on bridge backwards-compact layer. So, I'm trying to figure out a comfortable way of writing Swift views and exposing them as Fabric Native Components. If I can get there and have something to offer, I'll come back here :)

@0xFA11  Expo Modules (what this package is using) already supports the new architecture. We would just need to turn on new arch and update anywhere that's relying on didUpdateReactSubviews() since it no longer exists.

@0xFA11 Update on this. I have it somewhat working in the new arch.. will open a PR for at least some of the components once I finish


Menu. Support arbitrary key shortcuts

Here we implemented shortcuts for Window Menu.
Currently we support only mainstream shortcuts [Ctrl]+[Shift]+[Alt]+[Meta]+Key
Investigate how we can support arbitrary shortcuts that works even if we press some key outside of the our application:

Key1 + Key2 (order doesn't matter)
Key1 + Key2 (order is matter)
LeftCtrl + Key1
Multiple shortcuts for the same action (i.e. LeftCtrl + Key1 and RightCtrl + Key1 or Ctrl-Shift-Z and Ctrl-Y for Redo).

To do that, we need to:

Make a public API for KeyShortcut. One of the possible solutions:

class KeyShortcut(val sets: List<KeySet>)

class KeySet(private val keys: Set<Key>) : Set<Key>

Usage:
KeyShortcut(Key.CtrlLeft + Key.C, Key.CtrlRight + Key.C)

a. Observe all keys at the global application level, even if application is minimized (there can be a security issue though)
Swing doesn't allow it (Toolkit.getDefaultToolkit().addAWTEventListener only works if application in focus)
There is a native library https://github.com/kwhat/jnativehook. We can implement something similar in skiko.
b (alternative). instead of observing keys at the global level we can ask OS to get the current state of the Key.
P.2 is needed because users can press part of the shortcut outside of the application window (Ctrl, for example)

Hi, are there any updates on this issue?
Is my understanding correct that to create a global shortcut (when window is out of focus, e.g. minimized), the only way is to use the mentioned library?

For api reference, see QShortcut and QKeyReference https://doc.qt.io/qt-5/qshortcut.html
QKeyReference has a constructor that between 1-4 keys, so no need for vector or set.

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


downloadURL missing for plain uploaded files

Seems PyDrive relies on the downloadUrl meta property on files but that property has been removed in Google Drive API v3.
Doing .GetContentFile() raises "pydrive.files.FileNotDownloadableError: No downloadLink/exportLinks for mimetype found in metadata"
However, the file metadata does come with webContentLink property which seems to work the same way at least for plain uploaded drive files.
I was able to get around the issue by doing:
  file_obj.FetchMetadata(fetch_all=True)
  file_obj.metadata['downloadUrl'] = file_obj.metadata.get('webContentLink')
  file_obj.GetContentFile('/tmp/test.csv')

..but obviously that's a hack.
Any guidance on cleaner way to solve this?
Thanks
Pete

Hi Pete! Thank you for raising this here. I had a look into the problem and can confirm that standard files have the downloadURL present when downloading. I do, however, remember running into this problem at some point.
Could you provide the exact steps to reproduce this issue?
Further, PyDrive is still operating on API v2 (unfortunately), so it may make sense to handle this kind of problem only when the transition to v3 has been finished.

It happening to me also


Publish to npm

You should consider publishing to npm so your addon can be found over on: http://www.emberaddons.com/

definitely will just as soon as it's actually useable ;)


cmd/vet: build tags in raw string literals should be ignored

main.go:
package main
const foo = `
//+build ignore
`

Running vet produces:
main.go:3: +build comment must appear before package clause and be followed by a blank line

This also contributes to noise in the vet output for core, e.g.
./src/crypto/x509/root_darwin_arm_gen.go:181: +build comment must appear before package clause and be followed by a blank line
./src/crypto/x509/root_darwin_arm_gen.go:182: +build comment must appear before package clause and be followed by a blank line
./src/crypto/x509/root_darwin_arm_gen.go:183: +build comment must appear before package clause and be followed by a blank line

Build tags detection by the go build system is by design line oriented to avoid unnecessary parsing. The vet tool does the same thing, which seems correct to me.

@cznic That is a fair point, but in this case the other tools are not going to recognize the +build comment, which is what cmd/vet is saying anyhow.  In a case like this it seems worth skipping raw string literals if not too complicated.

This bug is actually a duplicate of #12269, I guess no one spotted that. That bug was closed by @robpike with the given reason "Not worth the trouble to fix."
I've hit this bug myself (here - tmthrgd/go-bindata#4), it's nothing all too major of course. Although it would be nice if go vet wasn't throwing up this false positive.
Is there any chance this bug might get fixed or is it still "not worth the trouble"?

Yes, it's probably not worth doing.  Closing.

Now that vet is on by default, and we are aiming for 100% accuracy on such checks, and we’ve gotten a handful of dups of this recently, I think we should consider fixing this. Yes, parsing is more expensive than line-oriented searches, but compared to the rest of vet, which does full typechecking, I doubt it matters much, particularly since we only need parse up to the package clause.

Scratch the comment about only parsing to the package clause.  Still shouldn’t be too expensive.

Only lexing (scanning) is needed, no parsing.

I would think the right fix is a complete rewrite of the module that looks at the comment-decorated parse tree, which we have. The parse will honor valid build comments but disregard bad ones, which would still be present as comments in the tree.


git-fleximod requires an ssh password for cprnc

I believe you had said that git-fleximod would address this problem.  I'm unable to test this as it remembers my password being provided from ages ago, so I don't know if you've recently fixed this.
@jtruesdal reported that he needed to type in his password for ssh.  Any idea on why this might need to be done?
Successfully checked out parallelio /project/amp/jet/collections/CAM_flexi/libraries/parallelio
          parallelio up to date.
Recursively checking out submodules of cime /project/amp/jet/collections/CAM_flexi/cime https://github.com/jedwards4b/cime
Enter passphrase for key '/home/jet/.ssh/id_ed25519':
e    CIME/non_py/cprnc not checked out, out of sync at tag None, expected tag is v1.0.6
Successfully checked out CIME/non_py/cprnc /project/amp/jet/collections/CAM_flexi/cime/CIME/non_py/cprnc

You can test by doing mv ~/.ssh ~/save.ssh first then running tests then putting back the .ssh directory.
I did find and fix a case where this wasn't working.


404 page not found for hyperlink on Documentation Page

Under the Configuring user provisioning to GitHub section of the page, there is a hyperlink for "provisioned to a GitHub organization via SCIM" that leads to https://docs.github.com/en/free-pro-team@latest/rest/reference/scim#provision-and-invite-a-scim-user , which no longer exists, apparently.

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

ID: 61e856ff-405a-62ab-20d6-b91a336c674f
Version Independent ID: a54166e0-1a4a-b9ef-1c41-9a9f61dc6af9
Content: Tutorial: User provisioning for GitHub - Azure AD
Content Source: articles/active-directory/saas-apps/github-provisioning-tutorial.md
Service: active-directory
Sub-service: saas-app-tutorial
GitHub Login: @twimmers
Microsoft Alias: thwimmer

@RooneyMcNibNug
Thanks for your feedback! We will investigate and update as appropriate.

@RooneyMcNibNug
Thank you for pointing this out! I've created a PR to update the documentation and it's currently waiting for the author to review and sign off.
Old URL - https://docs.github.com/en/free-pro-team@latest/rest/reference/scim#provision-and-invite-a-scim-user
New URL - https://docs.github.com/en/rest/scim#provision-and-invite-a-scim-user
I will now close out this issue. Please allow some time for the author to review and the changes to reflect.
Thank you for your time and patience!

thanks, @JamesTran-MSFT !


Instance lists do not get updated with IE

Instance lists do not get updated with IE. Instances created before session was started show up, new instances can be created and existing instances can be deleted, but the changes are not reflected in the UI. Refreshing the page (F5) does not help. If one brings up the developer console (F12), things start magically working, also no errors are logged in console.

fixed by #364

Reintroduced when switching to nginx as the ssl terminating proxy


Import Error: No module named bilm.training when I try to incrementally train ELMo

I downloaded bilm-tf-master from https://github.com/allenai/bilm-tf.
When I try to run bin/restart.py, I get an import error.
File "bin/restart.py", line 6, in
from bilm.training import train, load_options_latest_checkpoint, load_vocab
ImportError: No module named bilm.training

My python environment needed to be set up correctly.


API to delete a build scan

We'd like to use build scans more in our CI flows, but are concerned about them being publicly available if links are discovered. It would be ideal if we could save the created scans and then delete them on some scheduled cadence. Could Gradle add an API for this?

Hi @ZacSweers,
It is already possible to delete build scans on scans.gradle.com via the UI, I'm afraid we don't have any current plans to make an API for this.  Perhaps not ideal, but it would be possible to create some automation around this using Selenium or similar.
Alternatively, Gradle Enterprise addresses these kind of data privacy concerns by being installed on-premises.  There are also other features you don't get with scans.gradle.com such as aggregate views, build scan comparisons, flaky test insights, and distributed testing to name just a few.  I only mention in case you were not aware of this, you can request a free trial here if it's of interest https://gradle.com/enterprise/trial/

We would be happy to go to Gradle Enterprise, but it still does not support the type of authentication we need to use on premise. CC @Rooz-sf


chore: Remove local file support for 'manifest'  and 'defaultCR'

Description
Changes proposed in this pull request:

follow-up to https://github.com/kyma-project/modulectl/issues/53
manifest and defaultCR in module config must be links to openly accessible files, not file paths to local files

Related issue(s)

PR on hold. We need to re-discuss the idea of mandating file links. This makes sense for module releases that are put into module-manifests repo. The teams first create the module release, than update the config in the repo...
However, is it necessary for module teams to have the local file option during development of the module? We for instance also use the local file option when testing locally or in the E2E test pipeline. Module teams may do the same to quickly push a working state into a local registry. We need to clarify before we merge this.

basically, module team should not care about ModuleTemplate, modulectl is a tool design and used by submission process only, there is no needs for module team to create ModuleTemplate for local development, to verify the functionally of module just need to deploy raw manifest yaml directly, like what we offered in quick install guide. Additionally, introducing the option for a local file reference may cause confusion, as it requires the team to have separate configurations for local testing and the submission pipeline, supporting such usage would also increase our effort in terms of documentation and clarification.
Anyway, we can remove this option to simplify the usage first, if later do have the needs to reference local file, we can also support accordingly.


Rename 003-depenedency-manager.md to 003-dependency-manager.md

Fix Typo for "dependency"

Codecov Report

Merging #926 (009d8c0) into develop (180e53a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #926   +/-   ##
========================================
  Coverage    80.16%   80.16%
========================================
  Files           36       36
  Lines         1104     1104
  Branches       238      238
========================================
  Hits           885      885
  Misses         209      209
  Partials        10       10

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 5835178...009d8c0. Read the comment docs.


invalid link in error to ranking rules

/Users/masukomi/.rbenv/versions/3.2.0/lib/ruby/gems/3.2.0/gems/meilisearch-0.22.0/lib/meilisearch/http_request.rb:131:in `validate':
400 Bad Request - Invalid value at `.rankingRules[0]`: `title` ranking rule is invalid.
Valid ranking rules are words, typo, sort, proximity, attribute, exactness and custom
ranking rules.. See https://docs.meilisearch.com/errors#invalid_settings_ranking_rules. (MeiliSearch::ApiError)

The link provided is a 404.
I think the correct link is https://www.meilisearch.com/docs/reference/errors/error_codes#invalid_settings_ranking_rules

Hi team, I just moved this issue, since it comes from the engine (confirmed from a cURL request) on Meili v1.1.0.


Correction for Assignment 2

Do not consider '?' as missing values when implementing my_NB.py.
Treat that as one of the possible values X can take.
This should make Assignment 2 easier and last time I checked, sklearn.naive_bayes.CategoricalNB also does not consider missing values.

The "?" does not provide useful information about the features. Wouldn't this introduce an error into the model?

@rohaan2614
You are right. “?” Should be treated as missing values in this dataset. However, this assignment tries to match the categorical naive bayes classifier on sklearn, which does not have a missing value handling mechanism.

Can we remove the rows that contain "?" before calculating the
probabilities? Also, I noticed there are only 200 records (including "?"
rows). Is this sufficient for reliable probabilities?
On Mon, 5 Sept 2022 at 22:33, Zhe Yu @.***> wrote:

@rohaan2614 https://github.com/rohaan2614
You are right. “?” Should be treated as missing values in this dataset.
However, this assignment tries to match the categorical naive bayes
classifier on sklearn, which does not have a missing value handling
mechanism.
—
Reply to this email directly, view it on GitHub
https://github.com/hil-se/fds/issues/106#issuecomment-1237594232, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AGKAIEYHNNP3HIEKBPKNXMDV42UQLANCNFSM6AAAAAAQDQVT6M
.
You are receiving this because you were mentioned.Message ID:
@.***>

@rohaan2614
No. The goal of this assignment is not to predict as accurate as you can. But to implement the naive bayes classifier. Do not modify the A2.py file.


Prefixing

Same as the prefixing for the install role.
Tried to keep the naming style the same.

What's up with the line endings? Do you want them with non-Unix CRLFs?

What's up with the line endings? Do you want them with non-Unix CRLFs?

No , they should be Unix type.... :confused:
Hmm. I'll have to check the Atom config :suspect:

@aalaesar Have converted all files to Unix LF.
Lots of line changes, but hopefully only once :+1:


Update Parsleyjs to current release (2.0.7)

Coverage remained the same at 99.17% when pulling 5ea2a5c8c7b1030f651ee24a1f387dabc8d8fb01 on blueyed:update-parsley into 03fbb0e116a47bb0b10edf560ad587d71951bbb7 on agiliq:master.

Coverage remained the same at 99.17% when pulling 5ea2a5c8c7b1030f651ee24a1f387dabc8d8fb01 on blueyed:update-parsley into 03fbb0e116a47bb0b10edf560ad587d71951bbb7 on agiliq:master.

Coverage remained the same at 99.17% when pulling 5ea2a5c8c7b1030f651ee24a1f387dabc8d8fb01 on blueyed:update-parsley into 03fbb0e116a47bb0b10edf560ad587d71951bbb7 on agiliq:master.

Coverage remained the same at 99.17% when pulling 8700d140d4b4e400b6b276c6e80ed20c62aebe9b on blueyed:update-parsley into 03fbb0e116a47bb0b10edf560ad587d71951bbb7 on agiliq:master.

Coverage remained the same at 99.17% when pulling 8700d140d4b4e400b6b276c6e80ed20c62aebe9b on blueyed:update-parsley into 03fbb0e116a47bb0b10edf560ad587d71951bbb7 on agiliq:master.

Coverage remained the same at 99.17% when pulling 8700d140d4b4e400b6b276c6e80ed20c62aebe9b on blueyed:update-parsley into 03fbb0e116a47bb0b10edf560ad587d71951bbb7 on agiliq:master.


Youtube default poster/play visible if user skips before playing

Current Behavior:
If the user first skips to a specific moment in the video before actually pressing play, the Youtube default play button is visible along with the stretched poster image.
Expected Behavior:
Ideally it would show just a paused video, like it does if the video has already been played and then the user is skipping through.
Example where it's happening..
https://codepen.io/wplit/pen/PorbYjJ
See the video below showing it happening on the above example.. (skip through the video by pressing on the time-slider and don't press play to reproduce the problem)
Video showing issue - https://komododecks.com/recordings/0e7qB6AyJw5TFfPuaLRz
(i think if the player could make sure the video is played and then quickly paused at least once when the user skips, youtube would remove the poster/play and so this would never be seen, without the play Youtube covers with the poster image)

an alternate solution would be for the player to match what Youtube does, so keeping the poster image visible until the video is played rather than it being removed when the user skips. this would also keep the youtube play button hidden.


Parcel doesn't work

Describe the bug
Trying to use nodePackages.parcel in either nix develop or nix-shell fails with
Error: Got unexpected undefined
    at nullthrows (/nix/store/w1synbrvnn297pzwi77jid372j4n372x-parcel-2.12.0/lib/node_modules/parcel/node_modules/nullthrows/nullthrows.js:7:15)
    at Object.<anonymous> (/nix/store/w1synbrvnn297pzwi77jid372j4n372x-parcel-2.12.0/lib/node_modules/parcel/node_modules/@parcel/node-resolver-core/lib/builtins.js:66:38)
    at Module._compile (node:internal/modules/cjs/loader:1358:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1416:10)
    at Module.load (node:internal/modules/cjs/loader:1208:32)
    at Module._load (node:internal/modules/cjs/loader:1024:12)
    at Module.require (node:internal/modules/cjs/loader:1233:19)
    at require (node:internal/modules/helpers:179:18)
    at Object.<anonymous> (/nix/store/w1synbrvnn297pzwi77jid372j4n372x-parcel-2.12.0/lib/node_modules/parcel/node_modules/@parcel/node-resolver-core/lib/Wrapper.js:21:41)
    at Module._compile (node:internal/modules/cjs/loader:1358:14) {
  framesToPop: 1
}

Steps To Reproduce
> nix-shell -p 'nodePackages.parcel'
$ parcel

Expected behavior
I'm just starting out with parcel, so I'm expecting at least a friendly "no input files" style error, or else have it package a Hello World HTML file. No use of parcel so far has done anything but produce the above backtrace.
Additional context
An older issue (#278462) mentions this same backtrace, and when I try to use parcel in nix-shell I see the WARNING: cannot pinpoint dependency messages @hariroshan mentioned. I'm not clear on how to implement a fix though.
Notify maintainers
@SuperSandro2000 maybe?
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.6.52, NixOS, 24.05 (Uakari), 24.05.20240930.1719f27`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.5`
 - channels(root): `"home-manager-21.05.tar.gz, nixos-24.05, nixpkgs-24.05"`
 - channels(judson): `"home-manager-21.11, nixpkgs-21.11.337609.685d243d971"`
 - nixpkgs: `/nix/store/s9hbmmf7hgg7imnm5q6ny7gznbh0amfy-source`

Add a :+1: reaction to issues you find important.

No, I am definitely not a maintainer of that. You probably want to repackage it with buildNpmProject instead of node2nix.

Hey @nyarly,

To solve this, look for optionalDependencies in the package-lock.json, copy the correct versions of the packages to the package.json and run node2nix.

The answer was in this comment https://github.com/NixOS/nixpkgs/issues/278462#issuecomment-1880419394

Okay, but: I don't have a package-lock.json or even a package.json. I'm just trying to run parcel from nixpkgs, and it's busted, seems like. From what I can tell, it's in nixpkgs via adding it to a list of packages to be generated automatically? I'm looking at pkgs/development/node-packages/node-packages.json which seems to be how parcel comes in.
@SuperSandro2000 I'm sorry to bug you with this, but you're the closest thing I can find to a maintainer here. Is there not a proper npm maintainer/team?
I guess I could try doing my own node2nix or buildNpmProject attempt with parcel; I mostly just wanted to try it out - I do a little front-end work, but I'm not specialized enough to keep up with the NPM ecosystem. I'd hoped Parcel would live up to it's promise of being a zero-config build pipeline.

Have you made any progress on this? I'm in the exact same situation, hoping Parcel will make doing front-end stuff a breeze without having to keep up with the whole js/npm ecosystem. If not, I might have to try out using buildNpmProject, though was hoping I wouldn't even need a package.json file, and could just use Parcel.

Is there not a proper npm maintainer/team?

There are people which take care of the infrastructure but not of every individual package in it.


python311Packages.losant-rest: 1.19.2 -> 1.19.3

Diff: https://github.com/Losant/losant-rest-python/compare/v1.19.2...v1.19.3
Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 277289 run on x86_64-linux 1

4 packages built:

python310Packages.losant-rest
python310Packages.losant-rest.dist
python311Packages.losant-rest
python311Packages.losant-rest.dist


Jenkins

testing.py adds and sets the jenkins test runner so that it can generate a report.

I can see what this PR does, but it'd be nice if the title and description described why it was doing it :wink:

@r4vi can you merge this? Simple enough

yep trivial change


Page Feedback component requirements document

📇 User story

As a GCDS Components developer, I want to have the requirements for building and implementing the new page feedback component, so that all stakeholders can clearly understand the needs and functionality of the developed component

✅ Definition of Done / Outcomes

[ ] Done when requirements are written
[ ] Done when requirement doc is reviewed by a fellow developer (optional)
[ ] Done when requirement doc is reviewed by a designer (optional)
[ ] Done when requirement doc has final review by tech lead (optional)
[ ] Done when requirement doc has final review by design lead (optional)

📜 Acceptance criteria
Add here the list of assumed steps that will be needed to complete the task

[ ] Contains the data model
[ ] Contains properties
[ ] Contains a11y considerations (optional)
[ ] Contains design contributions (optional)

Build will be done by Pierre
Need a design ticket
Need an epic and accompanying tickets (content)

Lana stewart is the PO for the page feedback tool, we should collaborate.

Initial requirements documentation has been started in this document: https://docs.google.com/document/d/1TmKBmlBmx9QfgSPlYefOl_8gVD1QGtfObMRseu7UmVU/edit
I'm hoping we can collectively review and collaborate on further revisions as needed. I will ask a review from another dev this week, and touch base with @SmartMouthWords midweek for a review and feedback as well.

Requirements to be worked in collaboration with DTO and taking in consideration also first work proposed by Pierre in this PR #638.
Looking to have alignment conversation :

1 with Pierre to help align dev for requirements
1 with Lana (PO of feedback tool) and design

Met with Lana to discuss some points in the requirements document
Met with Pierre to get the conversation going, he has some work he's done on the requirements document on his side. We have asked him to input his work into the google doc so we can collaborate together and review.

We met with Pierre on Oct 23 to discuss some changes to the requirements documentation. We agreed to work async on rethinking some of the names for the properties and slots.
One thing we are starting to agree on is to call the component Feedback instead of specifically PageFeedback. @SmartMouthWords I'm tagging you here for visibility. We may want the design team to look into this component soon.


Menu item text way too long, overflows, cannot read the important word

It seems there are two different labels for the same notion: in a different part of the same feed:

fixed in #628


fix: [STO-7190]: Update STO SLIs and fix-up existing release notes

Thanks for contributing to the Harness Developer Hub! Our code owners will review your submission.
Description

Please describe your changes: __________________________________
Jira/GitHub Issue numbers (if any): ______________________________
Preview links/images (Internal contributors only): __________________

PR lifecycle
We aim to merge PRs within one week or less, but delays happen sometimes.
If your PR is open longer than two weeks without any human activity, please tag a code owner in a comment.
PRs must meet these requirements to be merged:

[ ] Successful preview build.
[ ] Code owner review.
[ ] No merge conflicts.
[ ] Release notes/new features docs: Feature/version released to at least one prod environment.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66a93c7d10de9981a76649e0--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66a9415611884284bc23465c--harness-developer.netlify.app


WIP

I will implement exercise say

Thanks for your interest. It seems like it would be best to open this on https://github.com/exercism/xc, would it not? Since the goal is to inform other people who may implement say in C that someone else (you) is already doing so, and the information is most effective if placed where contributors to the C track would look.


Joindin 535 removed custom dateformat function, switched to twig filter

Removed custom function and changed the template to use twig date filter

Can one of the admins verify this patch?

ok to test


Compress the media and images to improve the website's performance

Hello Contributers, I have noticed that the images used in this project makes the websites slower, also they load only when they are called, i want someone to compress the images and convert into webp format, and load the images as the website loads,
Ask me if you have any question...

hi I can help with this if you'd like!

hello @eKeiran, I have assigned you the issue
You may work on it now


warning: Calling of [-RCTUIManager setFrame:forView:] which is deprecated

Description
After I upgrade RN from 0.38 to 0.42.0 I got this waring:
Calling of `[-RCTUIManager setFrame:forView:]` which is deprecated

Reproduction
Just after I upgrade RN, I got this warning. In remote debug mode. I only test in iOS platform.
Solution
[empty here]
Additional Information

React Native version: 0.42.0
Platform:  IOS
Operating System: MacOS

same issus here

yes , same warning here .

@jacobcyl : I guess something was changed recently in v0.42.0. Can you confirm that this is not occuring on v0.41.0

Which Xcode version are you using?

@hramos I'm getting the same error with react-native v0.42 and XCode 8.2.1

@hramos as bennygenel

Phew, I'm glad I'm not the only one -- I thought I broke something!  For me, this started occurring after upgrading to 0.42, and Modals not working only happens when remote debugging is enabled.  I also start to get strange crashes after trying to open Modals.

Try to open modal.  Nothing happens.
Try to open other modal.  Then:

Try to open modal.  Nothing happens.
Navigate to other tab.  Then:

Find same issue, maybe can be a related to remote js debugger, stop remote debugging Modal is working for me.

I can confirm that when the debugger is closed, the Modal is working fine

Same issue here, SplashScreen hide stop working when remote js debugger is open.

Same issue. I upgraded from 0.41.2 to 0.42.0 and now the app runs fine when remote debugger is disabled. When it's enabled I get this warning: Calling of '[-RCTUIManager setFrame:forView:]' which is deprecated.

Found same Issue

GitHub added reactions recently - if you are running into the same issue but have no additional data to provide, you can simply add a reaction to the original post at the top. One advantage of this is that it allows us to filter by issues with a large number of reactions, making it easier to identify issues that people care about the most. Thanks!

With the same problem when i upgraged rn to 0.42

same issue, blank screen after upgrading to 0.42

Quick git-blame-fu shows the warning was introduced in https://github.com/facebook/react-native/commit/f33f84e75f1957f381e34bc99a902b9c324db6fa by @shergin with the following description:

Deprecating/removing setFrame, setLeftTop, and co.
Summary:
Motivation:

RCTShadowView's frame property actually represents computed layout of the view. We must not use it as a setter for yoga node styles;
Using frame and setLeftTop in existing way actually works only for view with absolute positioning, so it is super rare and special case;
Internally, setting frame only make sense to RootView, and in that case there we always must not change origin we are introducing setSize method.

Changes:

[-RCTShadowView setFrame:] was removed, frame property is readonly now;
[-RCTShadowView setLeftTop:] was removed; no replacement provided;
[-RCTShadowView size] read-write property was added;
[-RCTUIManager setFrame:forView:] was deprecated, use (just introduced) setSize:forView: instead;
[-RCTUIManager setSize:forView:] was added.

If you are still need some of removed methods, you are probably doing something wrong. Consider using setIntrinsicContentSize-family methods,
setSize-family methods, or (in the worst case) accessing yogaNode directly.

Seems like some internal classes are still dependant on [-RCTUIManager setFrame:forView:]?

@Etheryte Thank you!
Nope, the internal classes do not depend on setFrame:forView:, the proof: https://github.com/facebook/react-native/search?utf8=✓&q=setFrame%3A.
Most likely, one of external native libraries still depends on [-RCTUIManager setFrame:forView:]. The fix should be pretty trivial though (see current deprecated implementation).

[-RCTUIManager setFrame:forView:] which is deprecated."
I changed the code,remind by Xcode to
[self setSize:frame.size forView:view];

@shergin I've found [-RCTUIManager setFrame:forView:] in RCTModalHostView.m
https://github.com/facebook/react-native/blob/v0.42.0/React/Views/RCTModalHostView.m

For those whose modals aren't working in debugger, check this out: https://github.com/facebook/react-native/pull/12744
If you want to make the changes yourself, go to your Libraries/ReactNative/YellowBox.js file in your node_modules and add this line const elevation = Platform.OS === 'android' ? Number.MAX_SAFE_INTEGER : undefined;, then change the 3 elevation: Number.MAX_VALUE in your styleSheet to elevation: elevation.

@binchik Oh, my fault. Branch cut splits my changes across versions. 😞  We have to cherry-pick https://github.com/facebook/react-native/commit/233015c93ee6f986cf1267f143fdfadbf9c6831d for 0.42.
Sorry about that. 😿

I've cherry picked 233015c93ee6f986cf1267f143fdfadbf9c6831d into 0.42-stable. We may still need to deploy a new package to npm. cc @ericnakagawa @grabbou

I'll investigate that as soon as I get home after Conf
On Tue, Mar 14, 2017, 11:34 Héctor Ramos<EMAIL_ADDRESS>wrote:

I've cherry picked 233015c
https://github.com/facebook/react-native/commit/233015c93ee6f986cf1267f143fdfadbf9c6831d
into 0.42-stable. We may still need to deploy a new package to npm. cc
@ericnakagawa https://github.com/ericnakagawa @grabbou
https://github.com/grabbou
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/facebook/react-native/issues/12777#issuecomment-286518137,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACWcxqLxKLle6fkcwYsnMIekUsl-uUbUks5rlt3BgaJpZM4MWTUt
.

Version bumped, new package is getting deployed. I am going to close that issue, please update to 0.42.2 when it gets released to npm (next hour).
Apologies for incovnience caused.

@jacobcyl can you please provide a solution to your template?


Fix the GzippedWhisperReader

The GzippedWhisperReader would attempt to use a private method in
python. Since methods starting with __ are treated specially in python
by prepending _$class before attempting to invoke the method, this
code didn't work for accessing the private module method __readHeader
that exists in the whisper module.

Closing this to change the base branch. We need to make further modifications and use the master branch and I don't want to confuse this PR by adding different issues into the same PR.


Build fails: swift: error: manifest parse error(s):

I set up the swift toolchain according to the instructions at swift-android-toolchain.
Running macOS 13.5.2, Xcode 15.0 (also tried with Xcode 14.2, same result), Apple silicon CPU
$ ./gradlew assembleDebug
Starting a Gradle Daemon (subsequent builds will be faster)

> Configure project :app
Warning: The 'kotlin-android-extensions' Gradle plugin is deprecated. Please use this migration guide (https://goo.gle/kotlin-android-extensions-deprecation) to start working with View Binding (https://developer.android.com/topic/libraries/view-binding) and the 'kotlin-parcelize' plugin.

> Task :app:installSwiftTools
Installing Swift Build Tools v1.9.7-swift5.4
Swift Build Tools v1.9.7-swift5.4 installed

> Task :app:swiftBuildArm64Debug
Swift PM flags: --configuration debug -Xswiftc -DDEBUG
/Users/redacted-username/src/swift-weather-app/android/app/src/main/swift: error: manifest parse error(s):
error: fatalError
/Users/redacted-username/src/swift-weather-app/android/app/src/main/swift/Package.swift:3:8: error: no such module 'PackageDescription'
import PackageDescription
       ^

> Task :app:swiftBuildArm64Debug FAILED

FAILURE: Build failed with an exception.

Hi
The current version of Swift Weather works only with 5.4
That's why you need XCode 13 as it was mentioned in the readme

[IMPORTANT] Install XCode 13.0 and make it default in Command Line

I'm working on supporting Swift 5.7 it will be compatible with Xcode 14.2


possible vulnerability in Logger.js

In Logger.js file it does not do any sanity check for file-path, file or arg. Even though the file is not directly accessible from the client side, it's always a good practice to sanitize any inputs. If external input somehow was able to influence any of these variables, it could lead to command injection attacks.

Thanks ! logger.js is just a demo, it is not used and it is not a full code. Need to update it?


fix chop last char issue, #4

ccls also formatted the whole file

Actual changes: https://github.com/moledoc/pw/issues/4#issuecomment-2171444403
Expected behavior demo: https://github.com/moledoc/pw/issues/4#issuecomment-2171445715


suggestion about SendEventToActor function

GameplayTag
GameplayTag must be a sub-tag of event,otherwise RPGAbilityTask_PlayMontageAndWaitForEvent will not receive event.
Payload(GameplayEventData)
GameplayEventData have OtionalObject parameter.Through a UObject to send more data.
In blueprint,you can use "Construct Object".Instantiate an UObject asset as input.
And When an event is received.You can get this UObject instance.
I think this is very useful.And hope you can add to document.

The PlayMontageAndWaitForEvent AbilityTask can receive any tag. The implementations in the ARPG sample project have a filter of Event.Montage on them as a parameter on the Blueprint node. Change that EventTags parameter in Blueprint and you'll see it receive all tags that come from gameplay events.
I already mention using optional objects when passing data through GameplayEvents for sharing arbitrary data. https://github.com/tranek/GASDocumentation#concepts-ga-data

Use the two Optional Object variables for arbitrary data that does not fit any of the existing variables.

Thank you for your feedback.

Thanks for your reply.


change jedi user id in containers

Description
Perhaps this should have been a bugfix branch...
@kbhargava ran into an interesting bug in the containers.
In order to avoid running MPI as root in the Docker containers, we added a user called jedi.   Normally this causes no problems.    However, when a user enters a singularity container with singularity shell or singularity exec, singularity takes the user id on the host and copies into the /etc/passwd file in the container.   This can lead to a problem if the numerical value of the user id is the same on the host as it is for the jedi user in the container.
For example, consider my AWS ubuntu node.    There the last line in my /etc/passwd file shows the user ubuntu as user number 1000.
ubuntu@ip-172-31-9-105:~$ tail -n 1 /etc/passwd
ubuntu:x:1000:1000:Ubuntu:/home/ubuntu:/bin/bash

Now, look at the last two lines of the same file in the container:
ubuntu@ip-172-31-9-105:~$ singularity exec -e jedi-tutorial_latest.sif tail -n 2 /etc/passwd
jedi:x:1000:1000::/home/jedi:/bin/bash
ubuntu:x:1000:1000:Ubuntu:/home/ubuntu:/bin/bash

So, singularity has not changed the user id: it is still 1000.  But, there are now two users with a user id of 1000, namely jedi and ubuntu.  System commands such as whoami or id choose the first one:
ubuntu@ip-172-31-9-105:~$ id
uid=1000(ubuntu) gid=1000(ubuntu) groups=1000(ubuntu),4(adm),20(dialout),24(cdrom),25(floppy),27(sudo),29(audio),30(dip),44(video),46(plugdev),108(lxd),114(netdev),999(docker)
ubuntu@ip-172-31-9-105:~$ singularity exec -e jedi-tutorial_latest.sif id
uid=1000(jedi) gid=1000(jedi) groups=1000(jedi),4(adm),20(dialout),24(cdrom),25(floppy),27(sudo),29(audio),30(dip),44(video),46(plugdev),108(lxd),114(netdev),999(docker)

This caused problems for @kbhargava for ssh git authentication.   The container is looking in /home/jedi/.ssh for the ssh key instead of $HOME/.ssh.
Issue(s) addressed
Acceptance Criteria (Definition of Done)
The jedi user in the singularity container has (in all likelihood) a different user id than the general user so conflicts like the git ssh authentication example referred to above are avoided.
Dependencies
Impact
If accepted, I will also include these changes in my current feature branch in the containers repo.  So, the change will also be incorporated in the intel containers but there is no need for another PR in that repo.
Test Data
None

Thanks @rickgrubin - unfortunately we cannot change it once the container is built.  Different people will be using the container with different uids.  I figured I'd just avoid 1000 which is the default on ubuntu, so pretty common.   The max on ubuntu is something like 65k, as you said.
Another way to handle this is to delete the jedi user when we build the singularity container from the docker container.    But, this is easier for now and will solve pretty much all problems, I think.


Invariant errors with Tooltip overlays in Chrome

Was: GitLab #46
Just in Chrome, React throws invariant errors when you mouse over the (enabled) undo/redo buttons.
I also wanted to add tooltips to layout switch buttons in the new ui branch, but there it seems to stop Javascript processing with the same errors in Chrome.
In Firefox all is fine and I'm not sure when this bug was introduced.

I can't reproduce this. Where do you see the "invariant errors"

I think it's since I did the update to React 0.13 (not sure though). Did you prune node_modules and npm install.

The errors are in the console.

Did prune node_modules and React is now 0.13.1. Still don't see the error in either Chrome or Firefox.

I cannot reproduce it on my laptop, will try to reproduce from my desktop later (not sure where I reported this from originally).

Not sure what was going on here, closing it for now.

Now resurfaced on the zoom buttons.
Maybe related to this? facebook/react#2410


Can not build an Extension on Mac when pysnmp library is used

Describe the bug
I have the following setup.py:
from setuptools import setup

setup(name="xyz",
      version="0.0.12",
      description="xyz",
      author="lukasz.koniecki",
      packages=["xyz"],
      python_requires=">=3.10",
      include_package_data=True,
      install_requires=["dt-extensions-sdk", "pysnmp", "pycryptodomex"],
      extras_require={"dev": ["dt-extensions-sdk[cli]"]},
      )

The Extension does not build on Mac because of the following error:
Could not find a version that satisfies the requirement pycryptodomex (from xyz) (from versions: none)\nERROR: No matching distribution found for pycryptodomex

The command used to build it by the VSCode Ext.
<EMAIL_ADDRESS>-m pip download -d extension/lib --only-binary=:all: --platform linux_x86_64

pip downloads a pycryptodomex but in the pycryptodomex-3.19.0-cp35-abi3-macosx_10_9_universal2.whl version not the pycryptodomex-3.19.0-cp35-abi3-manylinux_2_17_x86_64.manylinux2014_x86_64.why which is required to build the package for Linux.
I can build the Extension manually using the following command:
dt-sdk build --extra-platform manylinux_2_17_x86_64 xyz

Steps to reproduce
Steps to reproduce the behavior:

Include pysnmp into your Extension 2.0 python project
Invoke the "Dynatrace extensions: Build" command

Expected behavior
I should be able to develop and build an Extension on Mac.

Thank you for raising this issue.
The root cause is how we build the python wheels
So that the extension functions on both Linux and Windows platforms, we will build wheels using the linux_x86_64 and win_amd64 platforms as extras in the pip command.
However, some packages out there (e.g. pycryptodomex, grpcio, etc.) have adopted PEP600 style of referencing linux platform in a format like yours - manylinux_2_17_x86_64
We cannot guess this in advance. So, to allow you to override the platforms, I will add a settings parameter for the workspace that allows you to specify your platforms.

Setting will support a list of strings, each being a platform to build for. Will look like this in the UI:

This should be used per workspace, as it will overwrite our default logic.
Example usage inside .vscode/settings.json at workspace level:
{
    "dynatraceExtensions.pythonExtraPlatforms": [
        "manylinux_2_17_x86_64",
    ]
}

These platforms will be added as options to flag -e for the pip download command that's internally used to fetch wheels.


Unable to use as a client

This may simply be a request that the readme be clarified.  I'm not sure.
It appears that to use this as a client (ignoring the proxy capabilities), you would do the following:
var net = require('net-browserify');

var socket = net.connect({
    host: 'google.com',
    port: 80
});

socket.on('connect', function () {
    console.log('Connected to google.com!');
});

However, the net object that gets returned by require does not have a connect() method.  The value being returned by require('net-browserify') apparently is not an equivalent of that returned by require('net').
I could very well have misunderstand the instructions, but following what I understood to be the way to use this as a client (as shown above) makes it look like there is a bug related to requiring net-browserify directly.

The object returned by require('net-browserify') is an equivalent for net only if used with browserify (see https://github.com/emersion/net-browserify/blob/master/package.json#L11). If you want to use net-browserify with Node, you could require('net-browserify/browser').

Thanks


Updating readme to include documentation for enabling HTTPS access.

This PR is being raised to add HTTPS related documentation in the readme.
HTTPS access is required for features like GPS location access through browser and this information will help testing volunteer's to setup HTTPS enabled test environment.

I think a better solution without modifying the application itself would be to use Nginx as a reverse proxy and do the SSL termination on Nginx

I think a better solution without modifying the application itself would be to use Nginx as a reverse proxy and do the SSL termination on Nginx

This involves installing Nginx and doing Nginx configurations.  The presented documentation was presented in slack #testing channel and I could easily incorporate this into my testing environment.
This is primarily for people like me, without much web/UI background, who are volunteering for testing.  The intent is to reduce overhead so that testing volunteers can quickly ramp up and contribute in testing.

I agree with @MansoorMajeed . Use nginx as a reverse proxy while running the django app using uWSGI which will fork the application to multiple processes each with their own threads. Also uwsgi protocol is significantly faster and nginx natively speaks uwsgi. You can point a uwsgi socket file for nginx to access and uwsgi will take care of load balancing. Hope this helps

@sourceflakes There was a PR with docker-compose, setting up Nginx would be quite easy if that one is merged.

Also, OT: How do I join the slack workspace?

@sourceflakes Why is this required? I never used to run ssl server on local and everything worked fine. I didn't notice this PR when it was merged.

@Biswas was required for working with some features.  Example: getting GPS
coordinates from browser only worked over HTTPS.
On Fri, Sep 7, 2018, 11:49 AM Biswas B<EMAIL_ADDRESS>wrote:

@sourceflakes https://github.com/sourceflakes Why is this required? I
never used to run ssl server on local and everything worked fine. I didn't
notice this PR when it was merged.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/IEEEKeralaSection/rescuekerala/pull/298#issuecomment-419482737,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABtcJ3sMnSXYSSG1OAsnYULYWaQ_SshBks5uYpWmgaJpZM4WBAoU
.

We are using GPS js api even now, it's working fine. Do you mean the Google text based place picker ?
We have a google map based placepicker working fine without ssl now

I am not aware of the present situation, Google APIs had deprecated non
secure transfer of GPS location from browser.
Just wondering why transfer location and other data in a no secure way.
Maybe I am misunderstanding.  Anyways the documentation was done to unblock
testers after discussions in the testing slack channel.  You can refer that
for details.
On Fri, Sep 7, 2018, 1:50 PM Biswas B<EMAIL_ADDRESS>wrote:

We are using GPS js api even now, it's working fine. Do you mean the
Google text based place picker ?
We have a google map based placepicker working fine without ssl now
—
You are receiving this because you were mentioned.
Reply to this email directly , view it on GitHub
https://github.com/IEEEKeralaSection/rescuekerala/pull/298#issuecomment-419516397,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABtcJw0ADcBztJD0JL9YsMxebqvOXSfEks5uYrHlgaJpZM4WBAoU
.


MD5 will generate identical outputs for the different input parameters

MD5 encryption might got problem
Recently, several exploits were developed which proved that MD5 will generate identical outputs for the different input parameters. As a consequence, somebody can break your encryption or log on to your website. For example: someone might be able to log on to your service with two different passwords, if their MD5 hashes are identical.

TODO(edony): fix the hashlib.md5 method for password encrypytion and replace it with hashlib.sha512 method.

P.S.
Check usage of exploitable MD5 hashes

Safety problem from MD5 encryption has been solved by SHA512 encryption.