GCS gfile operations fail in TF 2.17.0rc0 and 2.18 nightly when not running in GCP

Issue type
Bug
Have you reproduced the bug with TensorFlow Nightly?
Yes
Source
binary
TensorFlow version
2.17.0rc0, 2.17.0-dev20240320, 2.17.0.dev20240528, 2.18.0.dev20240613, 2.18.0.dev20240617
Custom code
No
OS platform and distribution
Debian bookworm, macos 14.5
Mobile device
No response
Python version
3.11
Bazel version
No response
GCC/compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current behavior?
When trying to run GCS operations with tf.io.gfile on nightly 2.17 or 2.18 anywhere outside of a GCP VM the command hangs and eventually fails after 10 retries with the error message as below.
I can't seem to reproduce this issue on either colab or a GCP VM. But it will consistently fail locally on my mac, inside a python:3.11 docker container, on GitHub actions or inside a kaggle notebook. The same code works fine with TensorFlow 2.16 so I don't think this is due to my local setup.
It also seems like other people are running into this with the latest TF nightly: https://github.com/tensorflow/datasets/issues/5360
Would be great to get this fixed before the 2.17 stable release.
Standalone code to reproduce the issue
import tensorflow as tf

tf.io.gfile.exists("gs://tfds-data/dataset_info/mnist/3.0.1/dataset_info.json")

Relevant log output
2024-06-14 15:45:30.081439: W external/local_tsl/tsl/platform/cloud/google_auth_provider.cc:184] All attempts to get a Google authentication bearer token failed, returning an empty token. Retrieving token from files failed with "NOT_FOUND: Could not locate the credentials file.". Retrieving token from GCE failed with "FAILED_PRECONDITION: Error executing an HTTP request: libcurl code 6 meaning 'Couldn't resolve host name', error details: Could not resolve host: metadata.google.internal".
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python3.11/site-packages/tensorflow/python/lib/io/file_io.py", line 290, in file_exists_v2
    _pywrap_file_io.FileExists(compat.path_to_bytes(path))
tensorflow.python.framework.errors_impl.AbortedError: All 10 retry attempts failed. The last failure: Error executing an HTTP request: HTTP response code 301 with body '<meta http-equiv="content-type" content="text/html;charset=utf-8">
<TITLE>301 Moved</TITLE>

301 Moved

The document has moved
here.

'
	 when reading metadata of gs://tfds-data/dataset_info/mnist/3.0.1/dataset_info.json

The only commit that comes to mind that might influence this is a4e3b786447042f4aa6ad5649a7fef9c4b40cee7, but I can't easily test this since tf-nightly wheels on pypi older than this date are not available.

@learning-to-play , this is another regression in TF 2.17 RC0

Please consider filing an issue in TensorFlow IO. To my knowledge there are similar issues already reported to TensorFlow IO.
FYI @yongtang @monicadsong

Please consider filing an issue in TensorFlow IO. To my knowledge there are similar issues already reported to TensorFlow IO.

@learning-to-play I already did at https://github.com/tensorflow/io/issues/2016.
But I'm not sure that it's related to TensorFlow IO since I can only reproduce this issue with TF 2.17 but not TF 2.16 even when both of them use the same tensorflow_io_gcs_filesystem package version.

@aaudiber Could you PTAL and help route to the right person?

This is from tsl layer, not TF-IO.

Indeed, using a wheel from Feb 20 works:
gs://tensorflow-public-build-artifacts/prod/tensorflow/official/release/nightly/linux_x86_cpu/wheel_py311/391/20240220-042013/github/tensorflow/build_output/tf_nightly_cpu-2.17.0.dev20240220-cp311-cp311-manylinux_2_17_x86_64.manylinux2014_x86_64.whl
(there's more wheels in that bucket)
I've been able to reproduce  this in one place, but it does work in a Docker container on a non-VM PC, so it's not necessarily VM-related.

Probably could be "reverted" via copybara

Thanks for looking into it and reproducing the issue. Also thanks for sharing the link to the older tf-nightly wheels, that's very useful to know.

This has been fixed in the latest nightly and on 2.17.0rc1 🎉


Question about the back translations.

Can I not do data augmentation on unlabelled data?

We use back-translation to create paraphrases for unlabeled data and perform consistency training. You could use other ways to generate paraphrases.

So I have to create paraphrases, right? In addition, When I look at the code, I find that only the first 100,000 pieces of data in the data set have been back translated. Do I not need to perform back translation for all the datasets

It depends on the size of the unlabeled data you are going to use.  In this work, we used 100,000 unlabeled data, so we just did back translations on them, not the whole dataset.

Sorry, I'm still a little confused.
When I test with:
python ./code/train.py --gpu 0,1 --n-labeled 10 \ --data-path ./data/yahoo_answers_csv/ --batch-size 2 --batch-size-u 4 --epochs 20 --val-iteration 1000 \ --lambda-u 1 --T 0.5 --alpha 16 --mix-layers-set 7 9 12 \ --lrmain 0.000005 --lrlast 0.0005
The number of unlabeled data per class seems to be 5,000. Do they add up to exactly 100,000?

You could use up to 100,000

10,000

Anyway, the number of data you need to paraphrase only depends on the number of unlabeled data you are going to use.

Are they one-to-one correspondence？

one unlabeled data could be associated with multiple paraphrases. Please refer to the paper/codes for details.


Templates for spag-rs

Not ready for merge yet.

[x] Needs some more test cases
[x] Needs to let me disable the shortcut syntax if I want
[x] Needs to avoid panicking if the environment isn't found with [env].thing syntax

Merge me @Squab.
The error messages are not great in a bunch of places. I didn't want to tests for grabbing stuff from envs (but this works) because I didn't want to read/write files in the unit tests, and since we'll test it later in the python tests
With this code as it is, you can spag request "<untemplate-this>" --with key val --with mini wumbo to test this out. This will not make request but will untemplate the text you pass to the request command. You should be able to do the following:
$ spag env unset --everything
---
{}
$ spag env set mini wumbo
---
"mini": "wumbo"
$ spag env set a.b.c squidward
---
"a":
  "b":
    "c": "squidward"
"mini": "wumbo"
$ spag request {{[].a.b.c}}
key=[] val=[]
untemplated: Ok("squidward")
$ spag request {{[].mini}}
key=[] val=[]
untemplated: Ok("wumbo")
$ spag request {{[].m}}
key=[] val=[]
untemplated: Err("substitution failed")
$ spag request '{{a, [].m}}' --with a A
key=["a"] val=["A"]
untemplated: Ok("A")
$ spag request '{{[].m, mini}}' --with mini wumbo
key=["mini"] val=["wumbo"]
untemplated: Ok("wumbo")
$ spag request '@mini' --with mini wumbo
key=["mini"] val=["wumbo"]
untemplated: Ok("wumbo")
$ spag request '@poo' --with mini wumbo
key=["mini"] val=["wumbo"]
untemplated: Err("substitution failed")
$ spag request '@poo/things' --with poo wumbo
key=["poo"] val=["wumbo"]
untemplated: Ok("wumbo/things")
$ spag request<EMAIL_ADDRESS>key=[] val=[]
untemplated: Ok("squidward/things")

@pglass You still want to merge this wrt the conversations we had earlier about errors?

@Squab Let's see. So the untemplater requires at least some functions (like in load_environment)  to return the error message instead of panicking/terminating so that it can ignore the error and move on to try to substitute the next item in the template list.
Idk. I think I still prefer using Result<> as much as possible. Usually, we probably won't care and could just exit(1). But returning the error message via Result<> is more flexible and makes the code more reusable: e.g. I can decide when I terminate, rather that load_file or whatever terminating for me. And I can append to the error message or replace it or whatever to give better error messages. And then, testing for negative cases is easier when everything gives you back the error message.
We should still have an error(msg) function for the main module so we actually have the right exit codes. But it feels better to me if all of the supporting modules just return error messages.

I can get down with that. :+1:


[packager] Homebrew - Push to a different branch (or disable pushing)

Hi there,
I'm not sure if I'm making something bad here or if there is a bug (or if it's a missing feature).
For Quarkus, we are pushing maintenance branches from time to time. Meaning we have release 2.16.0.Final as the latest but we are still pushing 2.13 releases. It also happens that we release alpha versions of a coming branch.
In both cases, we do not want Homebrew main branch to be updated with this version as it should still be using the latest stable release.
It's not a problem for Chocolatey or SDKMAN as they are published as a different version.
I tried using:
export JRELEASER_BRANCH=${BRANCH}
if [ "$MAINTENANCE" == "true" ]; then
    export JRELEASER_HOMEBREW_GITHUB_BRANCH=${BRANCH}
fi

but it doesn't seem to change anything: the commit is still pushed to main and I have to revert it (see https://github.com/quarkusio/homebrew-tap/commits/main).
FWIW, I have the following warning when running JReleaser:
[WARN]  Auto configure detected '' as default branch but project has '3.0' configured

Not sure if I understood what JRELEASER_HOMEBREW_GITHUB_BRANCH is supposed to be doing.
Another way to solve my issue would be to have an env variable to disable a particular packager.
Thanks!

I can see the following validation in place
https://github.com/jreleaser/jreleaser/blob/5173229ad0a6e820f4ff9151d6c9c3d00f3cee79/core/jreleaser-model-impl/src/main/java/org/jreleaser/model/internal/validation/common/Validator.java#L235-L240
This should match the name you're using: JRELEASER_HOMEBREW_GITHUB_BRANCH. If it's not working as expected then this is indeed a bug.
OTOH the warning message comes from
https://github.com/jreleaser/jreleaser/blob/5173229ad0a6e820f4ff9151d6c9c3d00f3cee79/core/jreleaser-engine/src/main/java/org/jreleaser/engine/context/ModelConfigurer.java#L94-L97
Which makes me think JRELEASER_BRANCH has been set to an empty value (?).
Finally, activating/deactivating packagers via env vars is something we've discusses internally before hand but have not found the use case for it. Now we have 😅 Matter of fact, other properties are also candidates for having their values resolved via env vars, also potentailly system props, the trick is defining the rules for resolution order.

@gsmet do you happen to have the capture of trace.log when the release was made? Alternatively you may run jreleaser config to check if branch value. I'll keep looking in my side.

Released in v1.5.0 -> https://github.com/jreleaser/jreleaser/releases/tag/v1.5.0


Refactor/move tests

Lets rename files with example code form endpoints.txt to endpoints.go.txt
Rename test funcs, from TestEndpointsForCountSvc to TestEndpoints
Move load code out of tests, to something like helper which can recieve

func Generate(service, example, interface string, template *generator.Template) (*bytes.Buffer, error) {
}

service and example here is a file names


[Zed] search recent projects not working

Extension
https://raycast.com/ewgenius/zed-recent-projects
Description
Error:
Error: Error: in prepare, no such column: local_paths
  SELECT local_paths, timestamp FROM workspaces
         ^--- error here

    at /Users/spark/.config/raycast/extensions/715b336d-1cef-4f40-be8a-f740b3f1656a/search.js:11:3767
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

Steps To Reproduce
idk started happening randomly
Current Behaviour
No response
Expected Behaviour
No response

Hello @auspy, thanks for reporting. Most likely you have old version of Zed, could you please check? Workspaces schema was recently updated, see #11995 and #12137

Hello @auspy, thanks for reporting. Most likely you have old version of Zed, could you please check? Workspaces schema was recently updated, see #11995 and #12137

I have the latest version. this issue got automatically resolved when I restarted my mac.

@raycastbot close this issue


Cant get string resources

Cannot get string resources which are located in the resource files. Only resources returned are the Manifest and Version. If we use the RT_String for getting specifically string resources we get no matching key in the dictionary.

You should be able to debug this or at least post something with a repro here (test + binary).

Sure will get back with it.

Hi,
Following is the code and the response. I am also uploading the projects repos and will send link in comments.
`
using (ResourceInfo ri = new ResourceInfo())
{
ri.Load(filename);
          // I get following error at the  ri[Kernel32.ResourceTypes.RT_STRING] because there is no
          // String resources extracted from the executable.
          // An unhandled exception of type 'System.Collections.Generic.KeyNotFoundException
          //occurred in mscorlib.dll
          //Additional information: The given key was not present in the dictionary.

           foreach (StringResource rc in ri[Kernel32.ResourceTypes.RT_STRING])
            {
                Console.WriteLine("StringResource: {0}, {1}", rc.Name, rc.TypeName);
                Console.WriteLine(rc);
            }
        }

`

Hi Following are the links to the sample repos.
Project with the Resource file which we aim to extract.
https://github.com/HassanNiazi/ResourceLibSampleDataProject
Project with the ResourceLib code which should extract the resource file
https://github.com/HassanNiazi/ResourceLibSampleDataExtractionProject

Thanks. Just to set expectations I haven't done much windows development for a long time, so I am not going to be debugging this. I am happy to help with any advice though ;)

@HassanNiazi The ConsoleApplication2 in the ResourceLibSampleDataProject repository doesn't contain any Win32 resources but embedded .NET resources. resourcelib's scope is Win32 resources however.
For accessing embedded resources I recommend to use .NET's ResourceManager.

Thanks for checking it out @thoemmi!

Thanks @thoemmi  :)


Stooge Sort in JavaScript

Stooge Sort
Implementation of stooge sort, a recursive sorting algorithm in JavaScript
Have you read the Contributing Guidelines on Pull Requests?
Yes

/assign


Notification issues [Needs runtime testing]

Description
Following potential solutions at https://stackoverflow.com/questions/25317659/how-to-fix-android-app-remoteserviceexception-bad-notification-posted-from-pac .
If anybody has a Huawei I would appreciate if they could test this PR, specifically connecting to wifi when offline products are waiting to be uploaded.
Related issues and discussion
Fixes #1702

Hey @ross-holloway94,
Your changes look good to me!

@teolemon There is an answer on the article which suggests it could cause the issue

@Karljoones there should be no harm in doing so. I'll leave it up to the team.

Limited connectivity for the next few days (marriage of my best friend). Please take the lead ;)

Merged, thanks @ross-holloway94, if we notice anything up with it we can sort it

@Karljoones great, ping me on Slack if you do.
@teolemon Have a nice time :)


BAH-2758 | Update error messages on registration page [Lite Env]

BAH-2758 requirement was to update the error messages for registration page for lite-env
sample changes are as follows :

Raising PR again due to CLA issue
@binduak @kk-tw

 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.Abhishek seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Geduardo/qrng quickstart

Docs Build status updates of commit 3d4cc26:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/quickstarts/qrng.md
:warning:Warning
View (qsharp-preview)
Details

articles/quickstarts/Bloch.svg
:white_check_mark:Succeeded
View (qsharp-preview)

articles/quickstarts/H.svg
:white_check_mark:Succeeded
View (qsharp-preview)

articles/quickstarts/Measurement.svg
:white_check_mark:Succeeded
View (qsharp-preview)

articles/quickstarts/qrng.md

[Warning] Author megbrow cannot be recognized in GitHub.
Line 17: [Warning] Invalid file link:(~/quickstarts/install).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit e79a828:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/quickstarts/qrng.md
:warning:Warning
View (qsharp-preview)
Details

articles/quickstarts/Bloch.svg
:white_check_mark:Succeeded
View (qsharp-preview)

articles/quickstarts/H.svg
:white_check_mark:Succeeded
View (qsharp-preview)

articles/quickstarts/Measurement.svg
:white_check_mark:Succeeded
View (qsharp-preview)

articles/quickstarts/qrng.md

[Warning] Author megbrow cannot be recognized in GitHub.
Line 17: [Warning] Invalid file link:(~/quickstarts/install).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 2f5dcab:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/quickstarts/qrng.md
:warning:Warning
View (qsharp-preview)
Details

articles/quickstarts/Bloch.svg
:white_check_mark:Succeeded
View (qsharp-preview)

articles/quickstarts/H.svg
:white_check_mark:Succeeded
View (qsharp-preview)

articles/quickstarts/Measurement.svg
:white_check_mark:Succeeded
View (qsharp-preview)

articles/quickstarts/qrng.md

[Warning] Author megbrow cannot be recognized in GitHub.
Line 17: [Warning] Invalid file link:(~/quickstarts/install).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 5353cec:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/quickstarts/qrng.md
:warning:Warning
View (qsharp-preview)
Details

articles/quickstarts/Bloch.svg
:white_check_mark:Succeeded
View (qsharp-preview)

articles/quickstarts/H.svg
:white_check_mark:Succeeded
View (qsharp-preview)

articles/quickstarts/Measurement.svg
:white_check_mark:Succeeded
View (qsharp-preview)

articles/quickstarts/qrng.md

[Warning] Author megbrow cannot be recognized in GitHub.
Line 17: [Warning] Invalid file link:(~/quickstarts/install).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 5353cec:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/quickstarts/qrng.md
:warning:Warning
View (qsharp-preview)
Details

articles/quickstarts/Bloch.svg
:white_check_mark:Succeeded
View (qsharp-preview)

articles/quickstarts/H.svg
:white_check_mark:Succeeded
View (qsharp-preview)

articles/quickstarts/Measurement.svg
:white_check_mark:Succeeded
View (qsharp-preview)

articles/quickstarts/qrng.md

[Warning] Author megbrow cannot be recognized in GitHub.
Line 17: [Warning] Invalid file link:(~/quickstarts/install).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Closing this pull request since we merged the shorter version to master already.


Add globe icon to language dropdown

Proposed Changes

Adds font awesome globe icon to language dropdown to help people realise they can change language

I feel like the contrast is way too overpowering

maybe opacity: 0.6 is better?

@JacksonLaw are we happy to make these changes, can merge straight after. Cheers


add millisecond parsing to strftime parser using %L

Summary of changes
I added a new format specifier %L to the strftime parser, to parse milliseconds from a 3 digit number, for example "2015-05-02 15:11:02.023" with "%Y-%m-%d %H:%M:%S.%L".  I added a milliseconds function to lib/parse/datetime/parsers.ex, the other changes were adding additional cases in the existing functions.

Coverage decreased (-0.005%) to 70.471% when pulling d8bdcb9fd0c481639346850ebac16c80713e04be on kf8a:add_millisecond_parsing into c5cb58dd02dd4fa490af427b688ad59322aad35c on bitwalker:master.

Looks good, thanks!


ChangeIndicatorToMoveStateToVariable Bug

There is a bug in the ChangeIndicatorToMoveStateToVariable function of the main repository file cRefactorFunctionLibrary.pkg.

Do you prefer to fix it first and after that return to the Tokenizer function?

The Tokenizer function has top priority.

Fixed!


[etlas] etlas exec inside of a sandbox fails

According to @JustusAdam:
etlas exec eta — —version
will give
eta: unrecognised flag: --print-global-package-db
in the presence of a sandbox.

Sandboxes are deprecated with the new-build system, so this issue is no longer relevant.


Release 2.1.4

This PR gets out there two more bug fixes in the 2.1 line. Specifically #7529 and #7537.

@bundlerbot merge


[Feature Request] Support specifying group in isend and irecv

Motivation
Currently, isend and irecv do not have a group argument, making it impossible to specify the desired communication group when there are multiple groups.
An example is when using a NCCL-based group for GPU communication, while also using a GLOO-based group for CPU communication.
Solution
Add a group argument to isend and irecv, and pass the parameter down to the underlying torch.distributed methods.
Checklist

[x] I have checked that there is no similar issue in the repo (required)

Good point will take care of vat today unless you want to do it :)

@vmoens I just submitted a PR but have not added tests yet, since I am not familiar with the test environment settings of this project.

I can help with that! Do you mind if I edit your PR directly?


[BUG] Documentation in correct for session state For 3.1

In the  Following the static class helpers that  have been given is actually wrong in .net  core 3.1 the string elements have been removed.
I have .net core 3.1 selection in the drop down menu and the statements are in correct.
Url Source ;
https://docs.microsoft.com/en-us/aspnet/core/fundamentals/app-state?view=aspnetcore-3.1
public static void Set<T>(this ISession session, string key, T value)
{
    session.SetString(key, JsonSerializer.Serialize(value));
}

public static T Get<T>(this ISession session, string key)
{
    var value = session.GetString(key);
    return value == null ? default : JsonSerializer.Deserialize<T>(value);
}

It should be this but even then it complains and gives an error that it cannot convert from readonly type of byte.
    public static void Set<T>(this ISession session, string key, T value) {
        session.Set(key, JsonSerializer.Serialize(value));
    }

    public static T Get<T>(this ISession session, string key) {
        var value = session.Get(key);
        return value == null ? default : JsonSerializer.Deserialize<T>(value);
    }

Error that is given is
Severity	Code	Description	Project	File	Line	Suppression State
Error	CS0411	The type arguments for method 'Helpers.Get(ISession, string)' cannot be inferred from the usage. Try specifying the type arguments explicitly.	ElectricalSurvey.DAL	D:\GitMaster\sybersystemslimited\ElectricalSurvey\ElectricalSurvey\ElectricalSurvey.DAL\Helpers\Helpers.cs	19	Active

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

ID: 76e22150-f1f1-5e4f-0e97-39b1ac089297
Version Independent ID: 687172c3-b7d8-525d-39dc-658b4a0f526f
Content: Session in ASP.NET Core
Content Source: aspnetcore/fundamentals/app-state.md
Product: aspnet-core
Technology: aspnetcore-fundamentals
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

@davidbuckleyni  just tested the code and it works fine. The type must be serializable.


expo-permissions is deprecated and camera permission is not needed

My App Store Connection submission was rejected by Apple because expo-images-picker was asking for camera permissions (with no clear reason why) and I had not specified a custom camera permission string. So I forked the repo and modified the code accordingly. While in the process, Expo also warned me that expo-permissions is deprecated and will soon be removed, so I removed those function calls as well.

Fix "noAssets" sample code. Sample code suggests "noAssets" should be passed in as an object but React FlatList (and VirtualizedList) expects string\component.


calculate_column_height! not defined

It seems recent changes in Oceananigans cause OrthogonalSphericalShellGrids to fail:
Failed to precompile OrthogonalSphericalShellGrids [c2be9673-fb75-4747-82dc-aa2bb9f4aed0] to "/Users/gregorywagner/.julia/compiled/v1.10/OrthogonalSphericalShellGrids/jl_nvamCB".
ERROR: LoadError: UndefVarError: `calculate_column_height!` not defined

I feel if these errors continue we should discuss merging this package into Oceananigans.

After thinking a bit more I think the design flaws go deeper... it seems there is not a clear "separation of concerns" between this package and Oceananigans which is really the root of the issue. Oceananigans apparently contains features that are not universally compatible with all grids (namely the grids in this package). This means that specific aspects of Oceananigans (such as the split explicit free surface) must be amended in order to be compatible with the grids here.
This is a design flaw, because it means that Oceananigans development is deeply entangle with the grids in this package. It also defeats the purpose of maintaining a distinct package for OrthogonalSphericalShellGrids, because it shows that the grids are not general. In order to work with certain physical models, those models have to be extended. Therefore this package is not actually separate from Oceananigans, in practice.

Why don't we make a module of Oceananigans?

I don't completely understand why we don't simply put this whole package into Oceananigans.

Notice that it has to make some pretty serious changes including a new boundary condition ZipperBoundaryCondition. In addition to that, apparently some aspects of the free surface have to be adapted. This is not really some simple auxiliary package, its fairly deeply entangled.
What I can't figure out is what the downside of putting this in Oceananigans is. Clearly there are a lot of advantages, but there have to be some disadvantages to have motivated making this a separate package in ths first place.

I think this is closed

We can probably move this to Oceananigans now that it is a bit more consolidated (still experimental though!). I can do it.


as contributor, would like rx_token split into multiline-statement so its easier to edit.

basically took output of console.log(JSON.stringify(rx_token.source)) and split it into multiple lines.
verified regular expressions are identical:
aa = (
/^((\s+)|([a-zA-Z_$][a-zA-Z0-9_$]*)|[(){}\[\],:;'"~`]|\?\.?|=(?:==?|>)?|\.+|\*[*\/=]?|\/[*\/]?|\+[=+]?|-[=\-]?|[\^%]=?|&[&=]?|\|[|=]?|>{1,3}=?|<<?=?|!(?:!|==?)?|(0|[1-9][0-9]*))(.*)$/
);

bb = new RegExp(
    "^("
    + "(\\s+)"
    + "|([a-zA-Z_$][a-zA-Z0-9_$]*)"
    + "|[(){}\\[\\],:;'\"~`]"
    + "|\\?\\.?"
    + "|=(?:==?|>)?"
    + "|\\.+"
    + "|\\*[*\\/=]?"
    + "|\\/[*\\/]?"
    + "|\\+[=+]?"
    + "|-[=\\-]?"
    + "|[\\^%]=?"
    + "|&[&=]?"
    + "|\\"
    + "|[|=]?"
    + "|>{1,3}=?"
    + "|<<?=?"
    + "|!(?:!|==?)?"
    + "|(0|[1-9][0-9]*)"
    + ")(.*)$"
);

console.assert(aa.source === bb.source);

web-demo of this branch deployed @ https://kaizhu256.github.io/JSLint/branch.rx_token/index.html

Thanks. I've been wanting to do something like this for a long time.


Add support for backend cloned PVC

Describe the problem/challenge you have
There is a requirement that want to provision nfs PV by clone an original backend PVC (the backend PVC CSI should support snapshot/clone, for example, LocalPV lvm or cStor).
Describe the solution you'd like
In the StorageClass yaml file, add dataSource annotation, then provisioner create backend PVC by clone original PVC by dataSource.
apiVersion: storage.k8s.io/v1
kind: StorageClass
metadata:
  name: openebs-rwx
  annotations:
    openebs.io/cas-type: nfsrwx
    cas.openebs.io/config: |
      - name: NFSServerType
        value: "kernel"
      - name: BackendStorageClass
        value: "openebs-hostpath"
      - name: dataSource
        value: "backend-origin-pvc"

Anything else you would like to add:
If we can directly add the snapshot/clone support for nfs PVC, that is fine also.
Environment:

OpenEBS version (use kubectl get po -n openebs --show-labels):
Kubernetes version (use kubectl version):
Cloud provider or hardware configuration:
OS (e.g: cat /etc/os-release):
kernel (e.g: uname -a):
others:

Vote on this issue!
This is an invitation to the OpenEBS community to vote on issues.
Use the "reaction smiley face" up to the right of this comment to vote.

:+1: for "The project would be better with this feature added"
:-1: for "This feature will not enhance the project in a meaningful way"

This would be extremely useful. Right now, if I want to upgrade a node running the NFS node or reboot it, I need to do the following steps:
Create a new NFS PVC.
Find where the PVC is running.
Copy the data over from the active node to the new physical volume.
This can be automated OFC. However, I think this would better be done in this operator directly.


Use minor version for go

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
There is no reason to depend in the patch version. This was a miss during the review of #1740
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?

NONE

🧐


Fixed a typo in examples

Added a missing ")" to a map function in example "Payment provider form rendering".

Thanks!


exec doesnt seem to be working

Node version:
6.2.0
ShellJS version
0.7.0
Operating system:
Mac
Description of the bug:
I am trying to run the following command (that I copied from docs) in my electron console.

Any idea, why this would be having. As I want to return the version number for checks. Other commands such as which are working but just exec
Thank you! :)

@ariporad can you reproduce this? This seems to be related to #471, and seems to be Mac-specific

@jeremyrajan: Can you expand the error messages to see the full stacks? Thanks!

Hi @ariporad , apologies for delay. Following is the full stack trace:
exec('node --version', {silent:true})

/Users/DextorG/Documents/Projects/Github/installer_app/node_modules/shelljs/src/com…:40

exec: internal errorlog @ /Users/DextorG/Documents/Projects/Github/installer_app/node_modules/shelljs/src/com…:40

error @ /Users/DextorG/Documents/Projects/Github/installer_app/node_modules/shelljs/src/com…:68

_exec @ /Users/DextorG/Documents/Projects/Github/installer_app/node_modules/shelljs/src/exe…:256

(anonymous function) @ /Users/DextorG/Documents/Projects/Github/installer_app/node_modules/shelljs/src/com…:269

(anonymous function) @ VM345:1

Thank you :)

Any idea guys? :). Neither of the exec commands are working for me

The only workaround I know might be process.version for the version. Doesn't fix this issue though.

Hmm...
We do seem to have a lot of problems with electron, don't we?
@jeremyrajan: Will you try this with the fix/480 branch, and let me know what the output is? Thanks.

@ariporad Do you need any further information?
Mac OS X 10.11.6
e.message
"Command failed: "/path/to/app/node_modules/electron-prebuilt/dist/Electron.app/Contents/Frameworks/Electron Helper.app/Contents/MacOS/Electron Helper" "/var/folders/p2/v0630kzj4lb8v65d325zn5vh0000gn/T/shelljs_a4b21fa8414ec26b6caf""

e.stack
"Error: Command failed: "/path/to/app/node_modules/electron-prebuilt/dist/Electron.app/Contents/Frameworks/Electron Helper.app/Contents/MacOS/Electron Helper" "/var/folders/p2/v0630kzj4lb8v65d325zn5vh0000gn/T/shelljs_a4b21fa8414ec26b6caf"
    at checkExecSyncError (child_process.js:487:13)
    at Object.execSync (child_process.js:527:13)
    at Object.childProcess.(anonymous function) [as execSync] (ELECTRON_ASAR.js:678:28)
    at execSync (/path/to/app/app/node_modules/shelljs/src/exec.js:106:13)
    at Object._exec (/path/to/app/app/node_modules/shelljs/src/exec.js:264:14)
    at Object.exec (/path/to/app/app/node_modules/shelljs/src/common.js:275:23)
    at file:///path/to/app/app/index.html:14:15

Ohh... I think I know the problem: I think Electron is breaking process.execPath. I'll look into this a bit more.

@giacgbj: So, it would appear that the node binary is built in to the electron executable, and as such we can't run it individually. This is problematic, and I'm really not sure how we can work around it.

@ariporad Suggestion:
var nodePath = isValid(process.execPath) ? process.execPath
             : (which('node') || which('nodejs'));
if (!nodePath)
  throw new Error('Unable to find path to NodeJS executable');

Check the semantics of which() to make sure this will work.

@nfischer: I'm not really sure what isValid would do. Theoretically, it should be totally valid to do:
mv /usr/bin/node /usr/bin/myNodeJSWithAFunnyName
myNodeJSWithAFunnyName script.js

Yes, that's the rationale behind using process.execPath. But you said execPath doesn't work, due to it being part of electron.
Just test if execPath points to something that is part of electron, then fallback to a name we know node usually goes by. If process.execPath doesn't work, and neither do any of the usually names for the node binary, then we can throw an error. It won't handle every case, but wouldn't it fix this issue?

@nfischer: There would be no way to test if it's part of electron. For example, Slack is written with Electron. From Slack's perspective, process.execPath is /Applications/Slack.app/Contents/MacOS/Slack. Atom is also written with slack. From Atom's perspective, process.execPath is /Applications/Atom.app/Contents/MacOS/Atom. So, I'm not sure how we would detect this.
I don't think we could just test if the path ends in node, because that would break for iojs, (or if there are symlinks).
We could have a shell.config.electron, which feels messy. Or, we could not support electron, which I don't like. Or, we could try to detect electron and do something special based on it. (Maybe we could submit a PR to electron which adds a -e option?).
Another thing is that Electron apps are frequently non-developer facing, so it might be bad to require people to have a seperate node instance on their machines.

Hmm... @giacgbj, Good point. @jeremyrajan?

hmm, that works too but is it cross platform compatible because I believe normally we have to execute a bat file for WIN? Or am i totally wrong :)

@jeremyrajan
var ret = child_process.execSync('node --version');

What's the issue with Windows?

Closing this because we've documented a workaround on the wiki. Please comment if this workaround is insufficient.

As per wiki I need to set 'config.execPath', How can I specify that? Any sample example for it?

@HardikDG please see the wiki, it explains how you might find the path to your node binary...

@nfischer I already saw the wiki, from the wiki itself I come to this post, I got the path to the node in my system.
My question is where I need to set this path? it says 'set config.execPath' but where? In my JS file or in my system where?
Can you please show the command/code to set this path
Sorry if it's very stupid question but I don't have deep knowledge of NodeJS, so I don't have that idea.

Set it like any regular variable.
// This is inside your javascript file
var shell = require('shelljs');
shell.config.execPath = 'path/to/node/binary'; // Replace this with the real path

// The rest of your script...

@nfischer  Thanks. It worked in my electron app.

@nfischer  Hi, glad to see there is  a workaround. This got failed throwing an error
        var shell = require('shelljs');
        let nodePath = (shell.which('node').toString());
        shell.config.execPath = nodePath;
        shell.exec('git');

May I know is there a compatible library version to make this work? Thank you

@gayanai does that workaround not work for you? This is still hit-or-miss, and we don't guarantee that workaround (otherwise, we would bake it into shelljs core).

@gayanai well, where is your path to node? For example, run this in the terminal:
$ which node || which nodejs # does this point to something?

Whatever that points to should be your execPath value. This is a best-effort kind of thing. If you don't have a standalone installation of node outside of electron, you'll need to install one for exec() to work.

@nfischer Hi
The path is /usr/local/bin/node and I have referred to that path

@gayanai if that doesn't fix the issue, then you'll need to provide full error output

setting exec to async seems to work.
let command = shell.exec('command', {silent:true, async:true});
command.stdout.on('data', (data) => {
    /* ... do something with data ... */
});
command.stdout.on('finish', (data) => {
    /* ... do something when finished ... */
});

@hazim-j yes, that is another valid workaround (assuming async works for your use-case). Now that we have async/await in some versions of node, maybe this is a better workaround.

node: 10.9.0
electron: 2.0.6
shelljs: 0.8.2
I have set the shell.config.execPath:
import shell from 'shelljs';
shell.config.execPath = '/Users/xuhuiyuan/.nvm/versions/node/v10.9.0/bin/node';

but it product another problem:

@nanjixiong218 thanks for reporting. Yes, it seems this isn't quite working. I'm not sure what that file (shelljs_22ff...) is supposed to be, so I'm not sure what options to recommend.

shell.witch('node') is null after package with electron-forge, then I used child_process instead

Forgive my ignorance... If you're bundling the app, couldn't you create an npm dependency to node@compatible-version and reference that?

If you're bundling the app, couldn't you create an npm dependency to node@compatible-version and reference that?

If you're requesting that ShellJS bundle node as a dependency, that would blow up our module's size. If you're recommending this for authors of electron apps, then this might be reasonable? I don't know enough about electron to comment, but it otherwise seems like it would satisfy ShellJS's requirements.

Thanks to hazim-j above, I got shelljs working within electron like this:
import shell from 'shelljs';

function runCommand(command, timeout) {
  return new Promise((resolve, reject) => {
    if (timeout)
      setTimeout(() => {
        reject(new Error('Command timed out'));
      }, timeout);

    shell
      .exec(command, {
        async: true,
        silent: true
      })
      .stdout.on('data', data => {
        resolve(data);
      });
  });
}


removed the label property from sdk manifest so users wont have colla…

…psing error on the label property

Coverage remained the same at 33.731% when pulling b65731ff7c7c18a58d284e11245fd1e4a091ed27 on label-fix into f4d97a59f59a862878ce93e8ce529cece7627bc3 on development.

Coverage remained the same at 33.731% when pulling e648e69a2ea646d73410e4d113f0effaf3620fe6 on label-fix into f4d97a59f59a862878ce93e8ce529cece7627bc3 on development.

Coverage remained the same at 33.731% when pulling 1f057dcc22e9e29cbe5b632846db0c97bee49646 on label-fix into f4d97a59f59a862878ce93e8ce529cece7627bc3 on development.

Coverage remained the same at 37.445% when pulling ee8bddf17ffe55aae24e7b7c57b6bf09aa803843 on label-fix into 79091590f45104c8229c9e146b119da4bc70fc18 on development.

Coverage remained the same at 37.445% when pulling 827e8a44359f82f5c18fb4143cc293380ac1f9ea on label-fix into 79091590f45104c8229c9e146b119da4bc70fc18 on development.


[flake] CurlRequestTest.WellKnownQueryParametersProjection failed due to DNS/host resolution timeout

seen once on 2020-07-28 in ubsan build: log
==================== Test output for //google/cloud/storage/tests:curl_request_integration_test:
Running main() from gmock_main.cc
[==========] Running 19 tests from 1 test suite.
[----------] Global test environment set-up.
[----------] 19 tests from CurlRequestTest
[ RUN      ] CurlRequestTest.SimpleGET
[       OK ] CurlRequestTest.SimpleGET (1644 ms)
[ RUN      ] CurlRequestTest.FailedGET
[       OK ] CurlRequestTest.FailedGET (1 ms)
[ RUN      ] CurlRequestTest.RepeatedGET
[       OK ] CurlRequestTest.RepeatedGET (4752 ms)
[ RUN      ] CurlRequestTest.SimplePOST
[       OK ] CurlRequestTest.SimplePOST (3258 ms)
[ RUN      ] CurlRequestTest.MultiBufferPUT
[       OK ] CurlRequestTest.MultiBufferPUT (667 ms)
[ RUN      ] CurlRequestTest.MultiBufferEmptyPUT
[       OK ] CurlRequestTest.MultiBufferEmptyPUT (3266 ms)
[ RUN      ] CurlRequestTest.MultiBufferLargePUT
[       OK ] CurlRequestTest.MultiBufferLargePUT (2915 ms)
[ RUN      ] CurlRequestTest.Handle404
[       OK ] CurlRequestTest.Handle404 (2667 ms)
[ RUN      ] CurlRequestTest.HandleTeapot
[       OK ] CurlRequestTest.HandleTeapot (495 ms)
[ RUN      ] CurlRequestTest.CheckResponseHeaders
[       OK ] CurlRequestTest.CheckResponseHeaders (505 ms)
[ RUN      ] CurlRequestTest.UserAgent
[       OK ] CurlRequestTest.UserAgent (1616 ms)
[ RUN      ] CurlRequestTest.WellKnownQueryParametersProjection
google/cloud/storage/tests/curl_request_integration_test.cc:297: Failure
Value of: response
Expected: is OK
Actual: EasyPerform() - CURL error [6]=Couldn't resolve host name [UNAVAILABLE]
[  FAILED  ] CurlRequestTest.WellKnownQueryParametersProjection (4739 ms)
[ RUN      ] CurlRequestTest.WellKnownQueryParametersUserProject
[       OK ] CurlRequestTest.WellKnownQueryParametersUserProject (496 ms)
[ RUN      ] CurlRequestTest.WellKnownQueryParametersIfGenerationMatch
[       OK ] CurlRequestTest.WellKnownQueryParametersIfGenerationMatch (718 ms)
[ RUN      ] CurlRequestTest.WellKnownQueryParametersIfGenerationNotMatch
[       OK ] CurlRequestTest.WellKnownQueryParametersIfGenerationNotMatch (644 ms)
[ RUN      ] CurlRequestTest.WellKnownQueryParametersIfMetagenerationMatch
[       OK ] CurlRequestTest.WellKnownQueryParametersIfMetagenerationMatch (2696 ms)
[ RUN      ] CurlRequestTest.WellKnownQueryParametersIfMetagenerationNotMatch
[       OK ] CurlRequestTest.WellKnownQueryParametersIfMetagenerationNotMatch (1695 ms)
[ RUN      ] CurlRequestTest.WellKnownQueryParametersMultiple
[       OK ] CurlRequestTest.WellKnownQueryParametersMultiple (2581 ms)
[ RUN      ] CurlRequestTest.Logging
[       OK ] CurlRequestTest.Logging (498 ms)
[----------] 19 tests from CurlRequestTest (35853 ms total)

[----------] Global test environment tear-down
[==========] 19 tests from 1 test suite ran. (35853 ms total)
[  PASSED  ] 18 tests.
[  FAILED  ] 1 test, listed below:
[  FAILED  ] CurlRequestTest.WellKnownQueryParametersProjection

 1 FAILED TEST
================================================================================
[1,039 / 1,040] 56 / 57 tests, 1 failed; Testing //google/cloud/storage/tests:object_media_integration_test; 57s processwrapper-sandbox
[1,039 / 1,040] 57 / 57 tests, 1 failed; Testing //google/cloud/storage/tests:object_media_integration_test; 78s processwrapper-sandbox
INFO: Elapsed time: 107.327s, Critical Path: 78.51s
INFO: 86 processes: 86 processwrapper-sandbox.
INFO: Build completed, 1 test FAILED, 58 total actions

also seen on 2020-07-28 in msan: log
This time many tests failed with the error above (Couldn't resolve host name) or (Couldn't connect to server)
FAIL: //google/cloud/storage/tests:curl_request_integration_test (see /h/.cache/bazel/_bazel_kbuilder/a7ecd8237744645c5d189c197108d6d2/execroot/com_github_googleapis_google_cloud_cpp/bazel-out/k8-fastbuild/testlogs/google/cloud/storage/tests/curl_request_integration_test/test.log)
INFO: From Testing //google/cloud/storage/tests:curl_request_integration_test:
==================== Test output for //google/cloud/storage/tests:curl_request_integration_test:
Running main() from gmock_main.cc
[==========] Running 19 tests from 1 test suite.
[----------] Global test environment set-up.
[----------] 19 tests from CurlRequestTest
[ RUN      ] CurlRequestTest.SimpleGET
google/cloud/storage/tests/curl_request_integration_test.cc:51: Failure
Value of: response
Expected: is OK
Actual: EasyPerform() - CURL error [6]=Couldn't resolve host name [UNAVAILABLE]
[  FAILED  ] CurlRequestTest.SimpleGET (4409 ms)
[ RUN      ] CurlRequestTest.FailedGET
[       OK ] CurlRequestTest.FailedGET (2 ms)
[ RUN      ] CurlRequestTest.RepeatedGET
google/cloud/storage/tests/curl_request_integration_test.cc:80: Failure
Value of: response
Expected: is OK
Actual: EasyPerform() - CURL error [6]=Couldn't resolve host name [UNAVAILABLE]
[  FAILED  ] CurlRequestTest.RepeatedGET (3268 ms)
[ RUN      ] CurlRequestTest.SimplePOST
google/cloud/storage/tests/curl_request_integration_test.cc:119: Failure
Value of: response
Expected: is OK
Actual: EasyPerform() - CURL error [7]=Couldn't connect to server [UNAVAILABLE]
[  FAILED  ] CurlRequestTest.SimplePOST (2135 ms)
[ RUN      ] CurlRequestTest.MultiBufferPUT
[       OK ] CurlRequestTest.MultiBufferPUT (2218 ms)
[ RUN      ] CurlRequestTest.MultiBufferEmptyPUT
[       OK ] CurlRequestTest.MultiBufferEmptyPUT (3139 ms)
[ RUN      ] CurlRequestTest.MultiBufferLargePUT
google/cloud/storage/tests/curl_request_integration_test.cc:201: Failure
Value of: response
Expected: is OK
Actual: EasyPerform() - CURL error [7]=Couldn't connect to server [UNAVAILABLE]
[  FAILED  ] CurlRequestTest.MultiBufferLargePUT (4310 ms)
[ RUN      ] CurlRequestTest.Handle404
google/cloud/storage/tests/curl_request_integration_test.cc:217: Failure
Value of: response
Expected: is OK
Actual: EasyPerform() - CURL error [7]=Couldn't connect to server [UNAVAILABLE]
[  FAILED  ] CurlRequestTest.Handle404 (4110 ms)
[ RUN      ] CurlRequestTest.HandleTeapot
google/cloud/storage/tests/curl_request_integration_test.cc:230: Failure
Value of: response
Expected: is OK
Actual: EasyPerform() - CURL error [7]=Couldn't connect to server [UNAVAILABLE]
[  FAILED  ] CurlRequestTest.HandleTeapot (8323 ms)
[ RUN      ] CurlRequestTest.CheckResponseHeaders
[       OK ] CurlRequestTest.CheckResponseHeaders (5874 ms)
[ RUN      ] CurlRequestTest.UserAgent
google/cloud/storage/tests/curl_request_integration_test.cc:277: Failure
Value of: response
Expected: is OK
Actual: EasyPerform() - CURL error [7]=Couldn't connect to server [UNAVAILABLE]
[  FAILED  ] CurlRequestTest.UserAgent (9604 ms)
[ RUN      ] CurlRequestTest.WellKnownQueryParametersProjection
[       OK ] CurlRequestTest.WellKnownQueryParametersProjection (624 ms)
[ RUN      ] CurlRequestTest.WellKnownQueryParametersUserProject
[       OK ] CurlRequestTest.WellKnownQueryParametersUserProject (700 ms)
[ RUN      ] CurlRequestTest.WellKnownQueryParametersIfGenerationMatch
[       OK ] CurlRequestTest.WellKnownQueryParametersIfGenerationMatch (4777 ms)
[ RUN      ] CurlRequestTest.WellKnownQueryParametersIfGenerationNotMatch
google/cloud/storage/tests/curl_request_integration_test.cc:366: Failure
Value of: response
Expected: is OK
Actual: EasyPerform() - CURL error [7]=Couldn't connect to server [UNAVAILABLE]
[  FAILED  ] CurlRequestTest.WellKnownQueryParametersIfGenerationNotMatch (9617 ms)
[ RUN      ] CurlRequestTest.WellKnownQueryParametersIfMetagenerationMatch
[       OK ] CurlRequestTest.WellKnownQueryParametersIfMetagenerationMatch (1616 ms)
[ RUN      ] CurlRequestTest.WellKnownQueryParametersIfMetagenerationNotMatch
[       OK ] CurlRequestTest.WellKnownQueryParametersIfMetagenerationNotMatch (4565 ms)
[ RUN      ] CurlRequestTest.WellKnownQueryParametersMultiple
[       OK ] CurlRequestTest.WellKnownQueryParametersMultiple (4580 ms)
[ RUN      ] CurlRequestTest.Logging
[       OK ] CurlRequestTest.Logging (538 ms)
[----------] 19 tests from CurlRequestTest (74410 ms total)

[----------] Global test environment tear-down
[==========] 19 tests from 1 test suite ran. (74410 ms total)
[  PASSED  ] 11 tests.
[  FAILED  ] 8 tests, listed below:
[  FAILED  ] CurlRequestTest.SimpleGET
[  FAILED  ] CurlRequestTest.RepeatedGET
[  FAILED  ] CurlRequestTest.SimplePOST
[  FAILED  ] CurlRequestTest.MultiBufferLargePUT
[  FAILED  ] CurlRequestTest.Handle404
[  FAILED  ] CurlRequestTest.HandleTeapot
[  FAILED  ] CurlRequestTest.UserAgent
[  FAILED  ] CurlRequestTest.WellKnownQueryParametersIfGenerationNotMatch

 8 FAILED TESTS

A repeat, or at least similar enough.
https://source.cloud.google.com/results/invocations/8ab725e8-3cdb-42cf-a4d1-a191fbaa6a76


Help needed for production push certificates

@bchen-twilio after following the instructions I can receive calls on Sandbox (both on simulator and on a device), but I can't receive calls on a production build installed on a device.
The errors I can see on the console suggest I am using a development certificate for production.

Trying to use a development certificate for a production application or vice-versa.
How can I create the correct VOIP certificate that works on both sandbox and production? Is there such an option?

Hi @fabriziomoscon
It is awesome that you are able to receive incoming call in development environment.
Before going for production, you will have to create another Push Credential SID with the same VoIP Service Certificate only this time leave the sandbox option unchecked. Use this Push Credential SID in the access token in your production application so that Twilio can send push notifications to the production endpoint of Apple's push notification service.
Let me know if this helps.
-bobie

@bchen-twilio I have separate subaccounts for local and production.
Under my production account I have created an APN Push Credential SID - I tried both via API and manually - using the cert.pem and private_key.pem extracted from my VoIP Service Certificate as a .p12 file
My Account SID is: AC34712d10934e857f92d3b4ddc3b07f53
the access token uses the APN push Credential SID.

Hi @fabriziomoscon
You mind sending your production Push Credential SID to me via email? I do have some internal tool to check the details of the credential and see if I can find anything.
Thanks for being patient.
-bobie

Hi @bchen-twilio
I'm having this issue as well :(
I have created Push Credentials for Sandbox and Live with the same VoIP certificate. Is working with development profile, but not when distributed.
Thanks
Esteban

The solution in my case was:

use sandbox = false for anything that is distributed by Apple: TestFlight and App Store
use sandbox = true for any other build, even for the Release build from Xcode

@fabriziomoscon thanks for jumping in. I have a similar arrangement to your solution, thats why I dont understand whats failing. Hopefully @bchen-twilio can help me.

Hi @eabait
Sorry for my delay of replying.
Could you help me check the Programmable Voice Dashboard and see if there is error/warning being logged when you experience the push notification delivery failure in production?
Thanks
-bobie

Hi @bchen-twilio
I have a lot of: APNs device token format is invalid :(
I have double checked the client configuration. I'm trying to figure out if the error is a configuration issue or an issue in our backend.
Thanks in advance!
Esteban

Hi @eabait
Could you provide the detailed error messages from the dashboard?
Also if you could email me the Twilio account SID + push credential SID you are using, I can check out the backend system and see if there is any immediate error.
Thanks

@bchen-twilio sent you an email

Hi @eabait
I've used your account and the two push credential sids and looked up the errors logged for the past week. It looks like most of the errors are caused when trying to send push notifications with the sandbox push credential sid and Apple rejected the requests.
Could it be that the token vending endpoint from you server is having trouble coordinating with the token request from your mobile app (or vice versa)? If that's the case, my suggestion would be to always issue the production push credential by default and only issue the sandbox credential when explicitly requested in the token URL. For example:

default token URL

  https://your.server/accessToken?identity=alice

daily development token URL

  https://your.server/accessToken?identity=bob&push_env=sandbox

-bobie

No activity for a while here. I'm going to close this issue. If there's more questions, let us know and we can reopen. Also, please note we've released more betas since then - check out our latest!


Added C# language support

Added a formatMap entry for the C# syntax of a Hashtable. I chose this over the more common generic Dictionary because additional refactoring would need to be done to account for the types and I didn’t feel it would be worth it.
I also modified the jsonToLang() function to account for double quotes in C# vs singles quotes in the existing languages implemented.
I wasn't sure of the order you would like the supportedFormats to be in, so I just tacked C# on to the end. They were in alphabetical, but I will leave that up to you.
I tested all of the rendered output in a C# application and all worked as expected.
Any feedback appreciated.

Shoudn't a C# dataset use static typing?

@KarolS It could. I explained my reasoning of using a simple Hashtable above.

I was thinking more of always generating Dictionary<string, T> and choosing T depending on what types of objects are in it: T = string for most datasets, T = object for heterogenous datasets (which don't exists here yet) and T = custom_generated_type for currencies dataset.

@vlucas Removed the custom C# logic for quotes and changed to use double quotes by default.
@KarolS For this use case (at least at this point), I think keeping it simple is better. Most use cases (in my opinion) will probably be to do one-off tasks like inserts or something similar. For that, a simple hashtable should work. If you or someone else would like to tackle something a little more complex though, it could be done.

Awesome :+1:


Feature request: add support for final fields

Spring Data Neo4j already allows non-relationship final fields thanks to its @PersistenceConstructor feature. I'd like to see a similar feature incorporated in Neo4j OGM as well for non-Spring projects.
Expected Behavior
Allow final (Java) or val (Kotlin) fields to be (de)serialized the same way as non-final fields.
Current Behavior
Currently final is not supported. You have to use non-final (Java) or var (Kotlin).
Possible Solution
Probably similar to the implementation of Spring Data Neo4j.
Context
I was working on a project in Kotlin with Neo4j OGM and I couldn't incorporate the encouraged immutability concepts provided by data class without introducing a lot of boilerplate code.

Neo4j-OGM with Spring Data uses Spring infrastructure to determine constructions of objects. Thanks for asking, but we're not gonna backport it, because it is - even with Spring integration - incomplete, that is: It doesn't work with relations due to pending restrictions in the mapping framework itself.


[Snyk] Security upgrade eventsource from 1.1.0 to 2.0.2

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `npm` dependencies of this project.

Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

package.json
package-lock.json

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Breaking Change
Exploit Maturity

718/1000  Why? Proof of Concept exploit, Recently disclosed, Has a fix available, CVSS 6.5
Information Exposure SNYK-JS-EVENTSOURCE-2823375
Yes
Proof of Concept

(*) Note that the real score may have changed since the PR was raised.

Commit messages

Package name: eventsource
    The new version differs by 35 commits.

774ed10 2.0.2
53356b5 chore: rebuild polyfill
7ed08fe docs: update HISTORY for v2.0.2
dc89cfd refactor: reuse unsafe header regex pattern
10ee0c4 fix: strip sensitive headers on redirect to different origin
a95ba90 2.0.1
9321d5c Fix: browser compatibility Merge pull request #268 from ajinkyarajput/ajinkyarajput-url-import
a759bc5 Update HISTORY.md
f52ed50 Standard: Make URL is global variable
9f28bd6 Make it again browser compatible
4579961 2.0.0
1f2357a Rebuild polyfill
751aedc Merge pull request #256 from joeybaker/version2
9ce9f65 Doc update history.md for v2.0.0
fa900cc Merge pull request #249 from AlexHladin/remove-original-dependency
1b722b8 Remove the original dependency and switch it to URL module
adc6e49 Merge pull request #239 from pfreixes/avoid-mutliple-buffer-allocation
ef3a4ee Less aggressive timeout
bab41d9 Modify test timeout that checks performance of the chunk parser
dbd7d13 Fix tests and use Buffer.alloc
99714d5 Add description about 256KB rationale
a9edd80 Change the threshold for 256KB
86dc983 Preallocate buffer size when reading data
82e0343 Merge pull request #150 from HonkingGoose/patch-2

See the full diff

Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.

 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.


Who has the Scene Graph of Flickr30K dataset , can you publish it,thanks a lot

here

Have you solved this problem? If so, can you share it?


多做cypress的確認動作

Open "Window Security"
Open "App & Browser control"
Click "Exploit protection settings" at the bottom
Switch to "Program settings" tab
Locate "C:\WINDOWS\System32\vmcompute.exe" in the list and expand it
Click "Edit"
Scroll down to "Code flow guard (CFG)" and uncheck "Override system settings"
Delete all files from C:\Users\AppData\Roaming\Docker
Start vmcompute from powershell "net start vmcompute"
npx cypress verify
npx cypress install

完成


Fixed link in README.md

Made the link to the i3wm website in the description link properly.
Before it wouldn't link to the website, but attempt to link to a nonexistent page at https://github.com/way-cooler/way-cooler/blob/master/i3wm.org

Thanks!


Fix link to crawl dashboard for local install

The link currently points to /banana, which is fine unless you also need to define the port because it's a local install.
Can we get this to work in both cases?

@wmburke, the link points to "/banana" because of the web server configuration where "/banana" is proxied to 8983 port. This means that users should have root access to modify the web server configuration by adding a 1 to 1 mapping.
Theoretically we could tell docker to proxy internal to always use 8983 on host but in that case you only can have one process on that port. I think this is not a critical requirement so I believe we should keep things as is and trust the end proxy configuration.
@wmburke and @sujen1412 please feel free to add your comments and/or close the issue if you agree with my explanation.

hmmm.... is this related to #41 by some chance?


Collabration White Board

Description
Collaborative_white_board is a webapp allow multiple user to interact read & write along on multiple server in Real time.
Fixes # (issue no.)

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Explain the Testing instructions
Test Configuration:

Operating system:
Version:
Text-editors used:

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK

@roy-shivam issue no ?

no there is not any issue

first make a issue and when it assign to u then make PR


Provide endpoint for fusion search integration

This issue is logged to provide an api endpoint which Zeiss SMT team can use to fetch Franklin site data for Fusion search integration.

Already covered via #50 . The index endpoint can be used to fetch details on indexed pages in Franklin by an external system.


chore: Release (0.2.3)

:robot: I have created a release beep boop
0.2.3 (2024-05-21)
Bug Fixes

Change bootstrap commit (3f59841)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/crate-crypto/peerdas-kzg/releases/tag/v0.2.3 :sunflower:


Using limit() and offset() in subqueries not working for offset > 0

I discovered a problem when using limit() and offset() in subqueries in the SQL component (QueryDSL 4.0.6). I am basically doing this:
subQuery = new SQLQuery();
subQuery.from(YYY).where(...).select(YYY.ID).distinct();
subQuery.limit(10);
subQuery.offset(20);

query.from(XXX).where(XXX.id.in(subQuery));

This yields the following SQL snippet (only WHERE clause is shown):
WHERE
a.ID IN
(
    SELECT
        *
    FROM
        (
            SELECT
                a.*,
                rownum rn
            FROM
                (
                    SELECT DISTINCT
                        a1.ID
                    FROM
                        FOO.BAR a1
                    WHERE
                        a1.X_ID = 123 ) a)
    WHERE
        rn > 20
    AND rownum <= 30)

This leads to an exception in my case (Oracle 11g), because the subquery now return more than one value. If I omit the offset, this SQL is generated:
WHERE
a.ID IN
(
    SELECT
        *
    FROM
        (
            SELECT DISTINCT
                a1.ID
            FROM
                        FOO.BAR a1
                    WHERE
                        a1.X_ID = 123 )
    WHERE
        rownum <= 10)

which is correct and yields the expected result.
Is it possible to explicitly return only the columns of the original subquery in the generated container query like this:
WHERE
a.ID IN
(
    SELECT
        a.ID
    FROM
        (
            SELECT
                a.*,
                rownum rn
            FROM
                (
                    SELECT DISTINCT
                        a1.ID
                    FROM
                        FOO.BAR a1
                    WHERE
                        a1.X_ID = 123 ) a)
    WHERE
        rn > 20
    AND rownum <= 30)

Or is there some other way to do what I do?

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

I've stumbled upon the same issue today. Would it be a big deal to add limit for subqueries? We would use it to limit the maximum number of deletions on a table.

I've stumbled upon the same issue today. Would it be a big deal to add limit for subqueries? We would use it to limit the maximum number of deletions on a table.

how to select column using subquery for limit(1) ???


addon downloaded all the mods, but none work in game

your manager created all the folders for the mods, but didnt create a d3d9.dll file anywhere, so i downloaded it from delta and placed it in my bin64 folder, and arcdps now loads on its on, but no other mods load. no keyboard shortcuts will load radial

Was this on the alpha version or version 0.4.5?

GW2 UOAOM v1.0.0 latest

So yeah, the prerelease. It should download the d3d9.dll file from https://github.com/gw2-addon-loader/loader-core. Can I see your config.yaml file?

application_version: v1.0.0
loader_version: v0.1
bin_folder: bin64
isupdate: true
game_path: C:\Guild Wars 2
default_configuration:
d3d9_wrapper: false
version:
d3d9_wrapper: v1.0
arcdps: >
6bd045305a6b689f60c536e72c6f3f4c  x64/d3d9.dll
arcdps_bhud: v0.2.4
gw2radial: v1.2.4_al
arcdps_mechanics: >+
39af47dea0d955e181aea205c0ec6a7c
d912pxy: v1.9h2
installed:
d3d9_wrapper: d3d9_wrapper
arcdps: arcdps
arcdps_bhud: arcdps_bhud
gw2radial: gw2radial
arcdps_mechanics: arcdps_mechanics
d912pxy: d912pxy
disabled:
d3d9_wrapper: false
arcdps: false
arcdps_bhud: false
gw2radial: false
arcdps_mechanics: false
d912pxy: false

Interesting...it says loader version v0.1, so it should have placed a d3d9.dll from the url I linked into the bin64 folder. You could manually use the release from https://github.com/gw2-addon-loader/loader-core/releases and stick it in /bin64/ yourself, or you could try setting this line:
loader_version: v0.1
to
loader_version:
and re-running the manager and hitting 'update'. If that doesn't work, I'll do some testing myself on my machine and get back to you.

that worked, thanks! any chance of some kind of gw2hook functionality, all i want is fog removal

actually, mounts and proxy work, but now arcdps doesnt

did you install arc through the manager? there should be an /addons/arcdps folder with gw2addon_arcdps.dll inside it if you did.

yes, and i deleted my manually downloaded d3d9, removed v0.1 again and updated via the manager, it added d3d9 its self, again, proxy and radial do load, but not arcdps

That's weird; can I see a screenshot of /addons/arcdps?

https://imgur.com/tszrxRh

the arcdps buildpad/templates was working before i decided to try your manager

as the release page says, currently addons dependent on arcdps aren't working. I think if just gw2addon_arcdps.dll is in that folder Arcdps should work.

I'm hoping to get the dependent addons working soon, though - I believe it's a change to plugin searcfh locations that Deltaconnected would have to make in Arcdps itself.

youre right, i removed the buildpad/templates folder and all works fine now, and while buildpad still loads in arcdps, i have lost access to my old templates files. nevermind, i can revert everything, save the old templates into anets new templates, and delete my old backups and have it all running again.
thanks again

No problem - if you feel like trying it it, the last full release, 0.4.5, should probably work with all the things you're using (it still uses traditional chainloading, which most of the development community was using before megai2 released the addon loader).


pdf has a "fainter" line in the table, which is not picked up in lattice mocde

000153061.pdf
Looking at this table I tried using columns, but was unable to figure out what the -c setting would be as I think I must use the "lattice" mode.
But a bigger issue is in lattice mode the last column is never "seen" as I think the vertical column is to faint.
Ideas?
Thanks!
Evan

Also what is the command line argument to say use "lattice" when using tabula from the command line?

Hi @EvanSmithMSMD,
The command-line argument for using lattice mode is --spreadsheet, "spreadsheet mode" being the old name for lattice mode. (I should get around to updating that...)
I'm not seeing the problem you're getting with the fainter line -- that is to say, though I don't have your expert eye, Tabula's reconstruction of the table looks accurate... Lattice mode should pick up all lines, regardless of color. Can you tell me which page you're looking at and send the output you're getting, so I can compare? Happy to continue to help debug this problem.

Hello,
All seems to work now that I understand --spreadsheet mode.
I did modify my copy of your ruby code to replace \r with a " ", as I
found the "" replace to make a mess.
Otherwise, great stuff.
Evan
Jeremy B. Merrill wrote:

Hi @EvanSmithMSMD https://github.com/EvanSmithMSMD,
The command-line argument for using lattice mode is |--spreadsheet|,
"spreadsheet mode" being the old name for lattice mode. (I should get
around to updating that...)
I'm not seeing the problem you're getting with the fainter line --
that is to say, though I don't have your expert eye, Tabula's
reconstruction of the table looks accurate... Lattice mode should pick
up all lines, regardless of color. Can you tell me which page you're
looking at and send the output you're getting, so I can compare? Happy
to continue to help debug this problem.
—
Reply to this email directly or view it on GitHub
https://github.com/tabulapdf/tabula/issues/427#issuecomment-159016501.

--
Evan C. Smith, MS, MD
<EMAIL_ADDRESS>Cell:<PHONE_NUMBER>

Great, glad you were able to get the results you needed! There's a command-line option that should solve the other problem too, --use-line-returns will put in \n (which you can then replace with " " if that's all you want).
The question of what to do with text on multiple lines is unfortunately a really complicated one, since sometimes line-breaks are just long text that didn't fit on one line and sometimes they're meaningful -- but the PDF format doesn't give us much in the way of hints.
Closing this, since it sounds like you've got the problem resolved.

Cool.  Thanks for the help!!

On Nov 23, 2015, at 10:13 PM, Jeremy B. Merrill<EMAIL_ADDRESS>wrote:
Great, glad you were able to get the results you needed! There's a command-line option that should solve the other problem too, --use-line-returns will put in \n (which you can then replace with " " if that's all you want).
The question of what to do with text on multiple lines is unfortunately a really complicated one, since sometimes line-breaks are just long text that didn't fit on one line and sometimes they're meaningful -- but the PDF format doesn't give us much in the way of hints.
Closing this, since it sounds like you've got the problem resolved.
—
Reply to this email directly or view it on GitHub.


Add restart example to the docs.

Add example showing how to restart the timer from the onComplete function and with a new value for timeout. Mention the .attr() method will not work and we must use the .data() method instead. See #20

Turns out it doesn't actually work as reported in #20 . There's a lot of small gotchas and since this is an edge case, I'll close it for now. The "hack" version I posted on #20 is still valid, though. If more people report the need for this feature then I'll revisit a public API.


GH-1795 increment snapshot number when closing branch instead of when flushing

GitHub issue resolved: #1795

@jeenbroekstra this is an alternate direction

Todo
[ ] port the inference test from the other branch
[ ] port any bug fixes from that branch
[ ] run benchmarks
[ ] consider creating benchmarks to test mixed parallel workloads

that test failure isn't looking very promising :(

2020-02-21T08:38:57.7857246Z [INFO] Running org.eclipse.rdf4j.sail.inferencer.fc.SchemaCachingRDFSInferencerIsolationLevelTest
2020-02-21T08:38:58.2094547Z 08:38:58.207 [Thread-426] ERROR Size was 1141. Expected 141 or 2141
2020-02-21T08:38:58.3857342Z 08:38:58.367 [Thread-426] ERROR

So this again allows another transaction to peek between two flushed branches.

I've found a new issue. When creating two datasets they can be created from separate snapshots!

I tried to fix this by adding a check for if the two datasets were in the same snapshot. This works for some of the tests, but others hang. It seems that when using regular SNAPSHOT isolation it stores the snapshot dataset in the SailSourceBranch, meaning that it will return a cached snapshot each time. My solution of "get a new dataset until we have two in the same snapshot" didn't work then.
Because we don't set the snapshot when beginning a transaction it's going to be hard to make sure that we don't create an inferredDataset at a different place than we create an explicitDataset.

I'm very curious as to why the NativeStore doesn't have this kind of issue.

I'm very curious as to why the NativeStore doesn't have this kind of issue.

The nativestore does not work with incrementing snapshot numbers at all. Instead, it keeps a bitmask persisted with each statement that indicates its transactional status: ADDED/REMOVED if added/removed by a currently active transaction (but not yet committed), and an EXPLICIT flag to indicate if the statement is inferred or explicit. This means that regardless of the isolation level, sinks can flush to disk, as filtering on these bit flags will ensure any 'dirty' statement will not be included in a concurrent transaction's read operations yet.

I  believe the memory store uses something along the lines of MVCC.

@jeenbroekstra To summarise the issue now:

T1: get inferred dataset => snapshot 0
T2: commit some changes => increment snapshot
T1: get explicit dataset => snapshot 1

T1 can now read old inferred data. This is correct. T1 can now read new explicit data, this is incorrect.
This is incorrect for all levels starting at READ_COMMITTED since you are reading parts of a transaction.
I don't know how to solve this. Any ideas @jeenbroekstra? I still think the best solution would be remove the inferred/explicit branch approach.

I still think the best solution would be remove the inferred/explicit branch approach.

If you want to start on that idea, go right ahead - but I did a bit of a spike on it earlier and it's a tremendous amount of work.
I'm personally thinking that the simpler fix is to see if we can copy over the bit flag mechanism the native store uses to track state (not literally copy perhaps, but use as inspiration).
By the way there's of course no reason why we can't do both approaches in parallel. My idea is on the level of a bugfix, whereas yours is a more comprehensive redesign (and likely needs a minor or even a major new rleease).

@hmottestad I lost track of this - what's the state according to you? You want to rebase this to get it up to date with the current main branch? What needs to happen to get this in a working (and hopefully, mergable) state?
Looking back at the discussion I think we have two general ideas about how to address the snapshot numbering issue. I like the idea you mentioned (getting rid of the separte explicit/inferred branches) a lot, and given that we are looking at a possible major release, now would be a good time to start planning for that. Do you think you will have capacity to look into that together with me? Is this something we want to add on top of this current branch, or should we just throw this away and start again?

We were not able to fix the issue no. The only real fix I can think of is to add support for reasoning throughout the stack instead of the split approach we have today.

Time wise I am very uncertain. At least not before august.


Nan loss for ResNext backbone trained on cifar 100

Thank you for your work. While trying your code for the Resnext backbone on cifar100, I get nan values for the training loss. As mentioned in the published paper, I use the initial learning rate of 0.1 for SGD with cosine scheduling.

Yes, same here.
Could you please help with this?
Thanks,
Sara


Remove refactored LSPClient target

NOTE: The LanguageServerProtocol dependency in Packages.swift should get proper version after new LSP repo tag is made

Wow, I forgot I didn't address this. You are right the LSP library needs to be tagged first.

Ugh, my mistake I actually forgot to re-look at this. And then someone reported the issue and I just instinctively fixed it. Sorry about that this. You tried! I failed!


⚠ Bofrost.at has degraded performance

In 8e6da31, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 905 ms

Resolved: Bofrost.at performance has improved in b8e7832.


Binding Key Events - prev not working on Chrome

All other bindings seems to work fine, but prev doesn't work. It sends this error to console every time I press the prev button:
Uncaught TypeError: _audioNavigation2.default.setPrev is not a function
at runPrevKeyDownEvent (keydown.js:168)
at runKeyEvent (keydown.js:95)
at HTMLDocument.handle (keydown.js:59)
I'm using latest version (5.0.1) with latest Chrome (Version 75.0.3770.142 (Official Build) (64-bit))

I am able to replicate this on the demo site too: https://521dimensions.com/open-source/amplitudejs
Thanks for your bug report!

This is fixed in the v5.0.2 release. I was able to confirm the behavior here: https://521dimensions.com/open-source/amplitudejs


bump malachite to 0.4.4

Branch 041 of https://github.com/RustPython/malachite-bigint.git has now been deleted.

Close #5107


[SPARK-42612][CONNECT][PYTHON][TESTS] Enable more parity tests related to functions

What changes were proposed in this pull request?
Enables more parity tests related to functions.
Why are the changes needed?
There are still some more tests we should enable.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Modified/enabled related tests.

Merged to master and branch-3.4.


Prestashop 1.7 - MOLPay response information not showing

MOLPay response message on transaction does not shown in Order detail's message panel.

In Prestashop 1.7, AdminOrderController has updated their message variable definition.
To retrieve the message so that it can shown in Message Panel, Prestashop owner is required to modify the following script in AdminOrderController.php:
Steps to display MOLPay response message in Message Panel:

Accessing to AdminOrderController.php via path Domain site/controllers/admin/AdminOrderController.php
Look for $this->tpl_view_vars array , replace message variable in the array with the following details.
Before
'messages' => CustomerThread::getCustomerMessagesOrder($order->id_customer, $order->id)
After
'messages' => Message::getMessagesByOrderId($order->id, true)

Precaution

This issue founded in Prestashop version <IP_ADDRESS>
Prestashop owner should take their own risk for any amendment in Prestashop's script


Update CupertinoTextField example

Related https://github.com/flutter/flutter/issues/72926
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Gold has detected about 61 new digest(s) on patchset 6.
View them at https://flutter-gold.skia.org/cl/github/93738

I'm not sure what's up with the golden test message... I think you can ignore it.


Shall we rename to LastMinuteCoders/ctf-writeups ?

Now that the Reply Cyber Security Challenge is over @GGn0 and I were considering to refactor this public repo, making it more general, reorganizing it like similar repos already available on GitHub.
By doing this we may continue to use it to host our writeups of past and future challenges.
Here is question to all LastMinuteCoders members: How about renaming it to LastMinuteCoders/ctf-writeups ?
Please comment here, thanks

Thanks @alv67. How about you, @gteti?


recipes are duplicated in JEI, as they are loaded everytie a world is loaded

sometimes even a wrong output is created (i'm testing the custom shapes atm)

Which branch is this on?

1.10

Yeah I haven't added the recipe paging code yet. The 1.10 branch is still heavily WIP.


[WIP] Introducing new versioned API structure (task #2589)

Codecov Report

Merging #459 into master will increase coverage by 2.63%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master     #459      +/-   ##
============================================
+ Coverage     26.31%   28.94%   +2.63%
- Complexity      729      739      +10
============================================
  Files            68       69       +1
  Lines          2554     2591      +37
============================================
+ Hits            672      750      +78
+ Misses         1882     1841      -41

Impacted Files
Coverage Δ
Complexity Δ

src/Controller/Api/V1/V0/AppController.php
50.58% <ø> (ø)
33 <0> (?)

...oller/Api/V1/V0/LanguageTranslationsController.php
100% <ø> (ø)
5 <0> (?)

src/Controller/Api/V1/V0/UsersController.php
100% <ø> (ø)
6 <0> (?)

src/Routing/ApiRouter.php
100% <100%> (ø)
10 <10> (?)

src/Event/Controller/Api/IndexActionListener.php
46.77% <0%> (+38.7%)
20% <0%> (ø)
: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 5e20537...4b728a3. Read the comment docs.


Update add.tasks.to.project.yml

Standards checklist:

[x] The PR title is descriptive.
[x] The PR doesn't replicate another PR which is already open.
[x] The PR has self-explained commits history.

[x] The code is mine or it's from somewhere with an Apache-2.0 compatible license.
[x] The code is efficient, to the best of my ability, and does not waste computer resources.
[x] The code is stable and I have tested it myself, to the best of my abilities.
[x] If the code introduces new classes, methods, I provide a valid use case for all of them.

Changes:

[...]

Other comments:
...

What is the point in this change?
And it has an error in path

I welcome the addition of badges - infographics are great! But I also don’t understand how it works with forkflow? Where will this badge be placed? Could you demonstrate the example on your repository?


Prevent user to create label greater than 63 characters

Bug description
UI allows to create label greater than 63 characters, because if you create such a label, elemental node will fail to boot:

Env
Rancher: 2.7.2-rc2
Elemental-UI: 1.0.1
Expected behavior
UI should not allow user to enter label greater than 63.

@aalves08 https://kubernetes.io/docs/concepts/overview/working-with-objects/labels/ - worth noting that the name part is limited to 63 chars - the prefix can be 253 chars in length.

@aalves08 You can use the existing validation in the rke2 editor. Then you can add dynamically the between rule I used already for the  password setting to the fvFormRuleSets as I did there.

@aalves08 https://kubernetes.io/docs/concepts/overview/working-with-objects/labels/ - worth noting that the name part is limited to 63 chars - the prefix can be 253 chars in length.

Indeed, just tested it with a long prefix and name ( < 63 ), total was 71 characters and it worked as expected.

Sucessfully tested in latest elemental-ui version (1.1.0)

We can see the warning and the Create button becomes un-clickable.
I will add a regression test in our automated test suite.


Set hasExitAnimation to true, @react-aria/toast can close.

Provide a general summary of the issue here
If hasExitAnimation is false, we can close the toast by clicking the 'close' button. If we set hasExitAnimation to true, nothing happens when clicking the 'close' botton.
🤔 Expected Behavior?
Close the toast no matter what value hasExitAnimation is set.
😯 Current Behavior
Can not close the toast.
💁 Possible Solution
No response
🔦 Context
No response
🖥 Steps to Reproduce
See https://codesandbox.io/p/devbox/test-nextui-forked-sk748x?file=%2Fapp%2Ftoast%2Findex.jsx%3A68%2C5-68%2C21
Version
3.0.0-beta.7
What browsers are you seeing the problem on?
Chrome
If other, please specify.
No response
What operating system are you using?
Win11
🧢 Your Company/Team
No response
🕷 Tracking Issue
No response

It's OK. I use framer motion.


Scan - receipt thumbnail is not shown due to a cors policy error

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.57-2
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team
Action Performed:

Open a new Gmail account
Click on FAB > request money > Scan > upload a receipt that will not fail
While on the participants page input 2 new email addresses you did not have history with
Confirm the request and wait for the scan to finish
Once the scan has finished open the console and click on the split preview component to open the RHP

Expected Result:
The receipt thumbnail should load
Actual Result:
The receipt thumbnail does not load due to a cors policy error:
Access to fetch at 'https://www.expensify.com/receipts/w_850728b68d65c07f20d99f83e4b91b01ee49d238.jpg.1024.jpg'
from origin 'https://staging.new.expensify.com'/
has been blocked by CORS policy: Response to preflight request doesn't pass access control check: No 'Access-Control-Allow-Origin' header is present on the requested resource. If an opaque response serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/a608af86-1bec-4a0b-8ced-3860e855837d

View all open jobs on GitHub

@kidroca @aimane-chnaif this looks related to our image caching PR - i wonder if there's an isAuthTokenRequired missing somewhere?

Def good questions 🤷  you're right, this could be unrelated

Hmm I can't reproduce this issue

I haven't been able to reproduce and I can see receipt images everywhere at the moment so I think we can call this NAB for now & keep this open a bit in case applause can reproduce

Waiting for applause to re-test to see if this is still a bug or not

Issue is reproducible during KI retests.
https://github.com/Expensify/App/assets/43995119/e59a81bf-5520-4716-834a-81abaced92c7

Waiting for scan to complete. Will update once it does.

i'm gunna guess we can close this


Feature Request

When creating a contract, I've found that it's a pain if I add extra bids on accident. Is there any way to remove bids when creating a market, or will I have to start the market creation process all over again?

Dup of
https://github.com/AugurProject/augur-app/issues/117
Closing


Cannot set input tensor

RuntimeError: There is at least 1 reference to internal data       in the interpreter in the form of a numpy array or slice. Be sure to       only hold the function returned from tensor() if you are using raw data access.
This occurs when running:
    def input_tensor(self):
        """Returns input tensor view as numpy array of shape (height, width, 3)."""
        tensor_index = self.interpreter.get_input_details()[0]['index']
        return self.interpreter.tensor(tensor_index)()[0]

    def set_input(self, frame):
        """Copies a resized and properly zero-padded image to the input tensor.
        Args:
          size: original image size as (width, height) tuple.
          resize: a function that takes a (width, height) tuple, and returns an RGB
            image resized to those dimensions.
        Returns:
          Actual resize ratio, which should be passed to `get_output` function.
        """
        width, height = self.input_size()
        w, h = frame.size
        scale = min(width / w, height / h)
        w, h = int(w * scale), int(h * scale)
        tensor = self.input_tensor()
        tensor.fill(0)  # padding
        _, _, channel = tensor.shape
        tensor[:h, :w] = np.reshape(frame.resize((w, h), Image.ANTIALIAS), (h, w, 3))
        return scale, scale

This doesn't tell me a lot, what input are you feeding into the function and how are you calling it>?

This doesn't tell me a lot, what input are you feeding into the function and how are you calling it>?

Closing this due to lack of activity. Feel free to reopen this thread if the issue still persists.


DifferSpec Fix for Scala 2.13

Fixed a failing test in DifferSpec when builds with scala 2.13.

This has also been pulled into 3.2.x.


Graphical artifacts on screen then plasmashell crashes

Describe the bug
I am running a brand new Bazzite install. Plasmashell is crashing at seemingly random times. Usually when playing a game, but sometimes just browsing through Steam. Graphical artifacts cover the screen, green, yellow, red triangles (looks similar to failing gpu). Then the screen goes black and I have to do a hard reboot. Happens multiple times a day. It happened with Plasma 6, and it is still happening with Plasma 6.1.
What did you expect to happen?
No plasmashell crashes multiple times a day.
Output of rpm-ostree status
State: idle
Deployments:
● ostree-image-signed:docker://ghcr.io/ublue-os/bazzite:stable
                   Digest: sha256:73dabc6ac45bc2c2c8e3b5c04ea33a48d7b15bfae7972f757125a4d461f46b12
                  Version: 40.20240621.0 (2024-06-21T05:11:41Z)
            LocalPackages: lact-0.5.4-0.x86_64

  ostree-image-signed:docker://ghcr.io/ublue-os/bazzite:stable
                   Digest: sha256:ed0d7dae4c45dca10fcca514b9e5e9f30cc8c851599cc021fd649b2441f43f9c
                  Version: 40.20240619.0 (2024-06-20T01:07:01Z)
            LocalPackages: lact-0.5.4-0.x86_64

Hardware
Ryzen 1600, RX 580
Extra information or context
"Crashed Processes Viewer" shows me this for "plasmashell":
poiioo.pdf

Does this happen without LACT installed?

I've just run "rpm-ostree uninstall lact" and restarted the computer, will let you know how it goes!

Does this happen without LACT installed?

Sorry, even without LACT the same problem just happened while browsing Steam! This time I have two crashes in "Crashed Processes Viewer", one for Xwayland and one for plasmashell (logs attached)
plasmashell.pdf
Xwayland.pdf

Xwayland and plasmashell crashed again, posting logs here in case they're helpful.
Should I be using xorg instead? Or maybe rebasing to Gnome? This machine is almost unusable as is :(
plasmashell2.pdf
Xwayland2.pdf

Yea you can give GNOME a try, strange issue since you're all AMD.
I wouldn't bother with X11 at all.

Should I go ahead and ignore the multiple bolded warnings about rebasing to a different desktop environment?

Rebased to GNOME, all is well so far, will let you know if I get any more weird crashes!

No problems at all with GNOME, I guess plasmashell and my computer hate eachother for some reason? Maybe I'll try rebasing back to KDE when Fedora 41 comes out. Thanks for your help!

Alright so it took a long time but I got the same wierd crash using Gnome almost immediately after turning the computer back on from "Suspend". However, in Gnome, it bumped me back to the login screen automatically where I could get back to work fairly quickly. KDE used to just hang indefinitely on a black screen requiring a forced shutdown.
I don't have "Crashed Processes Viewer" on this Gnome version so I can't post a log, unless someone has another alternative?


"error": "invalid parameter: expected an object (filter)"

Everything has been working fine for me but suddenly i start to get this error
{"level":"info","message":"Request info:\nUser message: what are vectors\nStream: true\nIP: ::1\nConversationId: 663fe16f4e7d858ab63c1e83","reqId":"66599a882ba207f3a014a906"} {"level":"error","message":"Responding with 500 status and error message: invalid parameter: expected an object (filter).","reqId":"66599a882ba207f3a014a906"}
I have tried debugging several methods. started the repo afresh and still yet i am getting this error

thanks for bringing this to our attention and apologies for any inconvenience that this issue may have caused you.
this is the same issue as #425.
we have investigated and have a fix in PR #427.
closing this issue. we will provide information about when the fix is live in #425.


Add extension dependency for OPA extension

We use the rego langId to control visibility of the deploy command; that langId is defined by the core OPA extension.

Closed via #22.


Global Reader Mode

Control the use of Reader View at a global level. For example:

Use Reader View on all feeds
Use Reader View on all feeds within this folder
User Reader View on selected feeds. I can only toggle Reader Mode one feed at a time, enable this for a group of feeds at a time.

Reader View is intended for use only when needed. The service we use to provide it is donating the servers and bandwidth to support it. We don't want to abuse this kindness and use Reader View when it isn't really needed.


Battle works fine but doesn't select or scroll support

MAKE SURE YOU TESTED THE LATEST RELEASE.

Did you look at other issues and the Troubleshooting Guide?

Describe the bug
Skills order, NP and battle all works fine
It's just that I can't loop because FGA doesn't select a support
I tried to full screen one or the other or both together and it doesn't select click/scroll support either way
FGA only works in battle if both are not full screen too
Screenshots
If applicable, add screenshots to help explain your problem.
It is recommended that you atleast add one screenshot of FGO running on your device.
These are my settings

Video
I'm the one who clicked on the support myself at the beginning of the video because FGA doesn't
https://www.youtube.com/watch?v=93Tybnk9n3U&ab_channel=SleepyAran
Device Info (please fill atleast 'Device model'):

Device model:  Samsung Galaxy S10 Lite
Screen size: 1080x2400
Android Version: Android 10
RAM: 8 RAM

First line in the issue literally says this: Did you look at other issues and the Troubleshooting Guide?

Hi, sorry. Did I look at the troubleshooting guide? Yeah, that's why I tried to change around the full screen thing like you wrote in there with the Samsung settings and  it does give blue screens at the side but than FGA stops working. The other issues you mainly told them to go read the same guide about the screen detection area / turning on or off the notch detection or download the latest release but I just downloaded it from google play an hour ago so I just assume it's the latest(?).

Can you post a video of blue bars not working with debug mode?

BUG/Problem:
I've got a similar problem, but i think it roots off from FGA detecting the FateGOJP clone (its com.aniplex.fategrandordes) as FGONA.
Under More Options; I've set it back to JP Mode, but it keeps switching back to NA Mode.
It can select the node/ go into battle just fine, but not the Support Selection Screen. It just gets stuck there in support screen.
It was working this morning too.
I haven't changed any settings anywhere, except for Left Padding for the Screen Size (0 for default, 90 when in use). That's for the past few days.
As for my regular FGOJP and FGONA clients, FGA is working fine.
VIDEO: https://youtu.be/0yUvBQuPyJc
Note : I'm the one selecting the classes tab, to get a reaction out of FGA's detections.
The autoskill script is programmed to pick the first servant of MIX.
Device Info :
Phone : Nokia 5 (no notch~)
Screen Size : 720x1280
Android Ver : Android 8.1 (64bit ver.)
RAM : 2GB

@reihunter360 Your issue isn't even remotely close to this issue. And, we don't support modded APKs. Don't post about modded APKs here again or we'll ban you from this repository.