Help converting examples from nightly to stable

Is your feature request related to a problem? Please describe.
We've had many requests over time to switch from using the nightly feature and nightly compiler as the default for examples and templates to using stable by default, to make it easier to understand for newcomers and to avoid giving the impression that Leptos only works on nightly or is particularly experimental or unstable.
The use of nightly everywhere also causes annoying issues in CI from time to time.
It's been a low-priority item for a while, and it would be great to have community help to get there.
Describe the solution you'd like
Remove the nightly feature on all examples and adjust the syntax accordingly, typically just switching the function-call syntax for signals to use .get() or .set().

Just curious, wouldnt it be better to have both? Personally I always use nightly because I like not having the .get() or .set() syntax (and I can use stuff like the parallel frontend in the compiler) but I understand why its a good idea to show that it works fine in stable, so why not keep both?
But yeah,  I think I can take care of the hackernews examples and maybe some of the counter ones

Personally I always use nightly because I like not having the .get() or .set() syntax (and I can use stuff like the parallel frontend in the compiler)

Me too, which is why it's been the default until now.

but I understand why its a good idea to show that it works fine in stable, so why not keep both?

My main concern is actually CI-related. We have not had many problems until fairly recently, but the instability of nightly APIs means that occasionally, things break and need to be fixed in CI in ways that can be confusing for people trying to use the examples.
One small case study that prompted me to open this: A few months ago, we found that rkyv was throwing clippy warnings on new nightly versions, which was breaking our CI during cargo check on leptos_reactive featurs that pulled in rkyv.  I probably should've investigated more deeply at the time (it turns out just disabling the copy feature on rkyv would've fixed it) but we just pinned to a nightly version that worked, instead.
Then some time in the last few weeks, an Axum patch update was released that was only compatible with more recent nightly compilers — but rkyv still wasn't. This meant we were in an uncanny valley where there was no nightly version of the compiler that could actually cause our CI to pass. I ended up spending a few hours of my day yesterday just dealing with the consequences of that combination and updating to a new nightly.
Ultimately this stuff is just a pain, and it will continue to be a pain as long as we have nightly examples that we're running CI against — hence why I'd rather replace them than just add more stable examples.

But yeah, I think I can take care of the hackernews examples and maybe some of the counter ones

Thanks!

Okay, I'll work on error_boundary.

My main concern is actually CI-related. We have not had many problems until fairly recently, but the instability of nightly APIs means that occasionally, things break and need to be fixed in CI in ways that can be confusing for people trying to use the examples.
One small case study that prompted me to open this: A few months ago, we found that rkyv was throwing clippy warnings on new nightly versions, which was breaking our CI during cargo check on leptos_reactive featurs that pulled in rkyv. I probably should've investigated more deeply at the time (it turns out just disabling the copy feature on rkyv would've fixed it) but we just pinned to a nightly version that worked, instead.
Then some time in the last few weeks, an Axum patch update was released that was only compatible with more recent nightly compilers — but rkyv still wasn't. This meant we were in an uncanny valley where there was no nightly version of the compiler that could actually cause our CI to pass. I ended up spending a few hours of my day yesterday just dealing with the consequences of that combination and updating to a new nightly.
Ultimately this stuff is just a pain, and it will continue to be a pain as long as we have nightly examples that we're running CI against — hence why I'd rather replace them than just add more stable examples.

Yeah, CI breaking because small issues at random due to breaks in nightly can become a huge pain when piled on, I get why you would want to move it and after converting a few examples, aside from having to move errors into closures tbh it wasnt that different

I'll work on counter_url_query

Thanks to everyone for contributing! I think this is all of them. Much appreciated :heart:


Add debug mode to print HTTP API calls

This is a small PR which adds a debug mode for the API. When debug = true is passed into the API, it will log any HTTP requests ran by the package.
The main use for this is to be able to debug cases where unnecessary API calls are performed to optimize on data usage and rate limits applied by the instance. Here are some example debug prints:
2023-12-04T11:45:56.785005: HttpMethod.get /site
2023-12-04T11:45:56.785683: HttpMethod.get /community/list
2023-12-04T11:45:57.096583: HttpMethod.get /user
2023-12-04T11:45:57.101890: HttpMethod.get /community/list
2023-12-04T11:45:57.124396: HttpMethod.get /post/list
2023-12-04T11:45:57.292789: HttpMethod.get /private_message/list

The only things that are printed out are the HTTP method and path! I think those generally don't contain any sensitive outputs since it's not the request or response body.

Perfect!


Great job

Great job, looking forward to your code!

Hi, the code of 4DRadarSLAM is released in July already:
https://github.com/zhuge2333/4DRadarSLAM

Okay, may I ask if the code for 4DRT-SLAM will be open source? I am very interested in this job

Currently, we don't have plans to open source the code of 4DRT-SLAM. But it is possible in future.

Currently, we don't have plan to open source the code of 4DRT-SLAM. But it is possible in future.


Separate console logs from different runs

Scenario:
Run training in one process, with Task.init().
Run evaluation in another process and write to the same task with Task.get_task(task_id='abcd1234').
Current outcome:
Console logs of both processes come out interlaced in Trains.
Possible desired outcome:
Separate console logs, one for each process.

@michaelitvin the issue is verified, and already fixed on the master branch.
The bug itself is that Task.get_task should not auto-log the console output (only Task.init should do that)
If you want to report logs to the "external" task , you can always do:
external_task = Task.get_task(task_id='abcd1234')
external_task.get_logger().report_text('hello from another process')

You can verify it was fixed, by installing directly from the git:
pip install -U git+https://github.com/allegroai/trains.git

Cool!
So mentioning #160 in this context.

Hi @michaelitvin
Just updating here, the latest RC contains the fix.
You can install with:
pip install trains==0.15.2rc0

Does it come with a way to restart automagical logging?

@michaelitvin what do you mean by "restart automagical logging"?
When calling Task.get_task there should not be any automagical logging, this was the bug :)

That thing we discussed in another thread, that sometimes a training task stops (e.g. another user killed it, or I decided to pause it for some reason), and I want to restart it later on and continue logging loss, learning rate and console from the point it stopped.
So this bug was actually a feature to me:)

Sure, you mean #160 :)
Let's close this issue and leave #160 open, as it is now a new feature (which hopefully will be pushed soon :)


Elm package is broken

Hey! Firstly thank you for your package, I am an avid user :) I assume you have changed your account name or moved the package recently as I am getting this error:
I downloaded the source code for tricycle/elm-imgix 1.4.0 from:
--
02:08:58.849 | https://github.com/tricycle/elm-imgix/zipball/1.4.0/
02:08:58.849 | But it looks like the hash of the archive has changed since publication:
02:08:58.849 | Expected: 7f9015ad6d29a00ba468020d4baa8101ac30a611
02:08:58.849 | Actual: 80e3d28dc3eaa41d8202c1a9b788d22844e371a4
02:08:58.850 | This usually means that the package author moved the version tag, so report it
02:08:58.850 | to them and see if that is the issue. Folks on Elm slack can probably help as
02:08:58.850 | well.

I made a fork and a convenience release until then in case anyone needs it:
https://package.elm-lang.org/packages/raen79/elm-imgix/latest/

Thanks for reporting @raen79 - a long overdue organisation rename has us scrambling today to resolve all of our open-source Elm packages, and unfortunately it seems the only way to address the rename is to re-publish the packages as 1.0.0 under the new organisation name :-/ which we've done https://package.elm-lang.org/packages/bellroy/elm-imgix/latest/


Don't copy parameters on setting a key in libssl

Whenever we set a private key in libssl, we first found the certificate that matched the key algorithm. Then we copied the key parameters from the private key into the public key for the certficate before finally checking that the private key matched the public key in the certificate. This makes no sense! Part of checking the private key is to make sure that the parameters match. It seems that this code has been present since SSLeay. Perhaps at some point it made sense to do this - but it doesn't any more.
We remove that piece of code altogether. The previous code also had the undocumented side effect of removing the certificate if the key didn't match. This makes sense if you've just overwritten the parameters in the certificate with bad values - but doesn't seem to otherwise. I've also removed that error logic.
Due to issue #13893, the public key associated with the certificate is always a legacy key. EVP_PKEY_copy_parameters will downgrade the "from" key to legacy if the target is legacy, so this means that in libssl all private keys were always downgraded to legacy when they are first set in the SSL/SSL_CTX. Removing the EVP_PKEY_copy_parameters code has the added benefit of removing that downgrade.
Once the above change had been made I found that the tests started to fail. This was due to an unrelated bug that was exposed because we were using provider based keys for the first time in libssl. The legacy_asn1_ctrl_to_param implementation of
ASN1_PKEY_CTRL_DEFAULT_MD_NID calls EVP_PKEY_get_default_digest_name() which returns an mdname. Previously we were using OBJ_sn2nid/OBJ_ln2nid to lookup that name in the OBJ database. However we might get an mdname back that only exists in the namemap, not in the OBJ database. In that case we need to check the various aliases for the name, to see if one of those matches the name we are looking for.

Ping for review?

Ping for review?

This pull request is ready to merge

This pull request is ready to merge

Pushed.

Pushed.


Check metric reports for updated age ranges

Please review all reports metrics that refer to age and the coding to be sure that the additional age categories (30-34, 35-39, 66-70) to the de-identified age variable (934298480) are included and will populate in the reports. For categories where we have zero participants in it they should still show in the report as ‘0’ instead of not appearing at all.
Reports possibly impacted:
-Operations Report
-Biospecimens Report
-Weekly Surveys
-Weekly Logs
-anything else?

@robertsamm  This is all set! It will be reflected starting on the 19th as the next automation update is the 16th. In the Operations report, tables 59 and 62 have been updated as well as the weekly logs. The Weekly CCC Module Metrics report and Weekly Biospecimen Report don't reference this age variable.

I've also reviewed the module metrics and confirmed that nothing there needs to be changed. The age variables are not limited to 40-69 (ex: age at diagnosis, age starting to use certain products, age at first/last period, LIFE questions, ect.). I will, however, update the Module 3 QC to make sure people aren't answering the LIFE questions past their current age or providing say an age of using a product past their current age

Hi @KELSEYDOWLING7 thanks this looks good! I see that Marshfield already has someone in the '66-70' age range verified for 'site reported age at time of recruitment'. @mnataraj92 @brotzmanmj @kmazzilli is this expected? Do you recall discussing this with MF already?

I don't remember the situation exactly because it was some time ago but there was someone we agreed to let stay in the study that was outside of the age range at the time because they were already verified and they would be eligible with the new age ranges.

@robertsamm  Hi Amelia, does anything else need to be looked into for this or am I good to close the issue?

Hi @KELSEYDOWLING7 I confirmed with Michelle she was aware of the MF participant in the 65-70 age group so nothing more needed. Closing issue.


More prominent information on running Raku

The problem
Command line options and very basic information on running perl6 should be on the "programs" page.
(which is described as "...A collection of documents describing how to run the Perl 6 executable program...", but I don't see anything there about how to run the Perl 6 executable.
Suggestions
Even if Rakudo specific, add "perl6" itself to https://docs.perl6.org/programs.html

Several issues here. The file (and thus URL) is called environment_variables. Although the title says otherwise (Running Rakudo), it's not totally clear this is the only way of running Raku right now, and it includes command line options, which is not obvious from its title.


Fixed climb slow down on both retract

Codecov Report

Merging #47 into master will decrease coverage by 0.14%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master      #47      +/-   ##
==========================================
- Coverage   85.11%   84.96%   -0.15%
==========================================
  Files          20       20
  Lines        1350     1350
  Branches      133      133
==========================================
- Hits         1149     1147       -2
- Misses        171      172       +1
- Partials       30       31       +1

Impacted Files
Coverage Δ

components/climb.py
71.01% <0%> (ø)
:arrow_up:

utilities/pure_pursuit.py
88.19% <0%> (-1.39%)
: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 85fe0ca...3180bb0. Read the comment docs.


static mandatory string component should go outside CWL Parameter References

glob: $(inputs.outputVcf).tbi is better than glob: $(inputs.outputVcf + ".tbi")
Many CWL runners (including cwltool) will be able to parse the first way without calling out to a Javascript engine

This might not be worth doing here, as it is cleaner to match the structure of the source WDL for string interpolation. Lets keep this for post-processing in cwl-format


Caching

Integrate caching to all our fetch calls so we don't get blocked by github

Is this open to be taken up?

Is this open to be taken up?

Yes. Do you want to work on it?

Is this open to be taken up?

Yes. Do you want to work on it?

is this still open ? I want to work on this .

I fixed this issue .


SystemError in tf.ensure_shape and tf.compat.v1.ensure_shape when dtype of shape is tf.uint64 and its value is too large.

Issue type
Bug
Have you reproduced the bug with TensorFlow Nightly?
Yes
Source
binary
TensorFlow version
tf 2.15
Custom code
No
OS platform and distribution
Linux Ubuntu 20.04
Mobile device
No response
Python version
3.10
Bazel version
No response
GCC/compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current behavior?
When the dtype of shape in tf.ensure_shape or tf.v1.ensure_shape is tf.uint64 and its value is close to 2^64,SystemError and OverflowError will be triggered.
Taking shape = tf.constant([18446743219011059112, 1], dtype=tf.uint64) as an example, in eager mode, these APIs will trigger a SystemError when it is called. I think it is a bug.
Standalone code to reproduce the issue
https://colab.research.google.com/drive/1sy4yzSx-n6HqS9oMvPv46WasMtMsCU_R?usp=sharing

Relevant log output
SystemError: <built-in function isinstance> returned a result with an exception set

@xl1043237213 Could you use a smaller integer data type like int32 for the shape tensor if the maximum re-presentable value is sufficient as per the needs?
Thank you!

Thanks for your response.
shape = tf.constant([18446743219011059112, 1], dtype=tf.uint64) is an illegal input. The shape input should be 'A TensorShape representing the shape of this tensor, a TensorShapeProto, a list, a tuple, or None'. However, tf.ensure_shape and tf.compat.v1.ensure_shape do not handle this error cases correctly, which may affect the stability of the code at runtime.

@xl1043237213 Before using tf.ensure_shape, could you explicitly validate the shape argument. You can check if it's a tf.TensorShape object, a list/tuple of integers, or None.
Thank you!

Thanks for your answer, I'd still hope that tensorflow would fix the bug if it could.

@SuryanarayanaY  I was able to replicate the issue reported here.
Thank you!


Scope (jwt 'scp' attribute) returned in VisualStudioCredential and in Microsoft Graph Explorer are different.

Describe the bug
I have an Azure function that uses Managed Identity to access Graph API's (Sharepoint sites and files), in my development environment i want to rely on my VisualStudioCredential (VSC).  In my tests i have noticed that the Scopes that are returned for the same User identity in Graph explorer and using VSC are different.
Expected behavior
Scopes (scp) returned for VSC should be same as the one in Graph explorer.
Actual behavior (include Exception or Stack Trace)
scp value of jwt token returned in graph explorer:
Files.ReadWrite.All Mail.ReadWrite Notes.ReadWrite.All openid People.Read profile Sites.ReadWrite.All Tasks.ReadWrite User.Read User.ReadBasic.All User.ReadWrite email
scp value returned as part of VSC:
Application.ReadWrite.All email openid profile User.ReadWrite.All
To Reproduce
Azure C# function:
var credential = new ChainedTokenCredential( new VisualStudioCredential(), new ManagedIdentityCredential(), new AzureCliCredential());
var token = await credential.GetTokenAsync(new Azure.Core.TokenRequestContext(new string[] {
	"https://graph.microsoft.com/.default"

	//"00000003-0000-0000-c000-000000000000",
	//"https://graph.microsoft.com//sites.readwrite.all",
	//"https://graph.microsoft.com/Files.ReadWrite.All"
}));
accessToken = token.Token;

I have tried using different scope values while calling TokenRequestAsync(), with the first two scopes the results returned are same (expected), however while using the other two the request fails.  Is it possible to request for multiple scopes so as to have parity with explorer?
Environment:

Azure.Identity 1.5.0
Microsoft Graph 4.10.0
Azure SDK Functions 3.0.13
Azure function 3.1 (dotnet), Consumption plan
IDE and version : Visual Studio 2019 16.11.5

Thank you for your feedback.  Tagging and routing to the team members best able to assist.

Hi @spmanjunath The reason for this difference is the permissions requested for the application being used by Graph Explorer are different than the ones being used by these credentials. If you are able to admin consent an application in your directory, you could register an application with all these permissions for that application and specify that clientId for the credential.

/unresolve
How is Graph explorer able to get additional scopes using the same credential as VisualStudioCode?  I tried monitoring the network traffic in the browser (for graph explorer api requests), but couldn't figure out how it achieves these additional scope.  The endpoint that Graph explorer calls is same as that called by Azure.Identity libraries.  Is there a way that i could request additional scopes in my request for the token?  The parity between the access obtained from Graph explorer and VisualStudioCode is critical for better developer experience.  I would prefer to avoid having Client credential if there are better alternatives.  Thanks.

is Graph explorer able to get additional scopes using the same credential as VisualStudioCode

/unresolve How is Graph explorer able to get additional scopes using the same credential as VisualStudioCode? I tried monitoring the network traffic in the browser (for graph explorer api requests), but couldn't figure out how it achieves these additional scope.

It does so by using a different client application.
The endpoint that Graph explorer calls is same as that called by Azure.Identity libraries. Is there a way that i could request additional scopes in my request for the token?
Yes, you can request any number of scopes for each token request. However, you can only be granted scopes that the application has been consented for.


SelectTransformBuilder doesn't handle subqueries. Cannot make any kind of subquery projection

Given this class:
class User
{
    public Guid Id { get; set; }
    public List<Guid> RoleIds { get; set; }
}

var session = store.QuerySession();
session.Query<User>(x => new { x.Id, FirstRoleId = x.RoleIds.FirstOrDefault() });

This query does not work and throws an exception, it appears to be only expecting sub-member expressions from the RoleIds collection which doesn't work because it's a collection of scalar values.
PR with failing unit test coming.

This is closer to what Marten generates:
SELECT
       jsonb_build_object('Id', d.data->>'Id',
       'RoleId', (SELECT id.data->'RoleIds'->>0 FROM mt_doc_user AS id WHERE id.id = d.id LIMIT 1))

Beating this for now w/ the ability to use array indexers and calling that good enough.

I'm closing this one. We're not going any farther for now.


add wechat oauth2.0 login，support both QR code and Wechat embedded browser login

add wechat oauth2.0 login，support both QR code and Wechat embedded browser login.
Wechat uses different scopes and authorization endpoints in normal browsers and WeChat embedded browsers.The current implementation determines which one to use the authorization endpoint based on the scope passed in by the client.
Scope
snsapi_login：For web pages, a QR code will be displayed when the user opens the web page. user can use the  WeChat App to scan the QR code to confirm.
snsapi_base，snsapi_userinfo：For WeChat embedded web pages，This authorization endpoint url needs to be opened inside WeChat。snsapi_base does not require user secondary confirmation, but only gets openid.
https://open.wechat.com/cgi-bin/newreadtemplate?t=overseas_open/docs/web/login/login#login_login

Merged. Thanks again for your contribution.

I completed the documentation: https://github.com/pac4j/pac4j/commit/e338a57681f46f3be32df6f09cca43958770e5e2


Image controller not recognizing latest image when using pre-release

We have docker tags like v{semver}-{commit-offset}-{gitHash} generated by git describe. The issue is once stable tags of the format v{semver} get pushed to the repository,  tags that should be more recent do not get recognized.
v0.0.0-1-abcd (first tag)
v0.0.1 (second recognized tag)
v0.0.1-1-abcd (not recognized even if latest)

Here is an example of the imagePolicy spec:
  policy:
    semver:
      range: '>=0.0.0-0'
  filterTags:
    pattern: '^v.*'

Let me know if i'm doing something wrong or need to provide more details.
I've also tested the case with the "github.com/Masterminds/semver/v3" package here is a link to the go playground: https://play.golang.org/p/M3UfEk1O8tZ

Something interesting to note is that having  pattern: '^v(.*)-(.*)' recognizes prerelease tags as latest

Check out the semver spec: https://semver.org/
TLDR: v0.0.1 > v0.0.1-1-abcd you need to bump the patch version e.g. v0.0.2-1

You are right this is what we ended up doing. Thank you!


Master

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 3 committers have signed the CLA.:white_check_mark: doraemonext:x: liuyujie-tx:x: TwwyYou have signed the CLA already but the status is still pending? Let us recheck it.


Implement client that sends hmac-signed request

Probably using servant-client library.

Blocked by #9


Go v1.21 support

I use the new slog package coming with go v1.21.
# compile for 'linux' in Docker container
fyne-cross linux \
-pull \
-app-build 1 \
-name ucrc \
-arch amd64 \
-icon ./Icons-2/refresh_116834_blau_512x512_whiteround.png
[i] Target: linux/amd64
[i] Checking for a newer version of the docker image: docker.io/fyneio/fyne-cross-images:linux
[✓] Image is up to date
[i] Cleaning target directories...
[✓] "bin" dir cleaned: /app/fyne-cross/bin/linux-amd64
[✓] "dist" dir cleaned: /app/fyne-cross/dist/linux-amd64
[✓] "temp" dir cleaned: /app/fyne-cross/tmp/linux-amd64
[i] Checking for go.mod: /Users/klaustockloth/go/src/ucrp.vwg/ucrc/go.mod
[✓] go.mod found
[i] Packaging app...
benchmark.go:5:2: package log/slog is not in GOROOT (/usr/local/go/src/log/slog)
note: imported by a module that requires go 1.21

error building application: exit status 1
[✗] could not package the Fyne app: could not package the Fyne app: exit status 1

% fyne-cross version
fyne-cross version v1.4.0

Questions:

Is there an ETA for go v1.21 support in fyne-cross?
Is there any workaround to use fyne-cross with go v1.21?

Is there an ETA for go v1.21 support in fyne-cross?

We always wait for at least one point release before upgrading. It needs to be stable before we force everyone onto a new version.
I don’t think there is any workaround but I might be wrong.

As for the workaround, you can modify the Dockerfile of fyne-cross-image and generate the image locally in the mean time.


Resource Leak of android.database.sqlite.SQLiteCursor seems false positive within isClosed() branch

Please make sure your issue is not addressed in the FAQ.
Please include the following information:

The version of infer from infer --version.
v1.1.0
Your operating system and version, for example "Debian 9", "MacOS High Sierra", whether you are using Docker, etc.
Ubuntu 20.04
Which command you ran, for example infer -- make.
infer run --pulse -- ./gradlew build
The full output in a paste, for instance a gist.
resource of type android.database.sqlite.SQLiteCursor acquired by call to query(...) at line 99 is not released after line 110.

              }

          } finally {

            if (!cursor.isClosed()) {

                  cursor.close();

              }

If possible, a minimal example to reproduce your problem (for instance, some code where
infer reports incorrectly, together with the way you run infer to reproduce the incorrect
report).
isClosed is the Android API：
https://developer.android.com/reference/android/database/Cursor#isClosed()

Can we avoid this FP by set the .inferconfig options ?
thanks.

#679 related , it is still open


"npm run build" failed - TypeError: Ajv is not a constructor

Followed instruction to build project and got  error
https://github.com/aws-samples/amazon-timestream-simple-visualizer/blob/main/README.md
C:\github\amazon-timestream-simple-visualizer>node --version
v16.13.1

C:\github\amazon-timestream-simple-visualizer>npm run build

><EMAIL_ADDRESS>build
> react-app-rewired build

C:\github\amazon-timestream-simple-visualizer\node_modules\mini-css-extract-plugin\node_modules\schema-utils\dist\validate.js:66
const ajv = new Ajv({
            ^

TypeError: Ajv is not a constructor
    at Object.<anonymous> (C:\github\amazon-timestream-simple-visualizer\node_modules\mini-css-extract-plugin\node_modules\schema-utils\dist\validate.js:66:13)
    at Module._compile (node:internal/modules/cjs/loader:1101:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1153:10)
    at Module.load (node:internal/modules/cjs/loader:981:32)
    at Function.Module._load (node:internal/modules/cjs/loader:822:12)
    at Module.require (node:internal/modules/cjs/loader:1005:19)
    at require (node:internal/modules/cjs/helpers:102:18)
    at Object.<anonymous> (C:\github\amazon-timestream-simple-visualizer\node_modules\mini-css-extract-plugin\node_modules\schema-utils\dist\index.js:6:5)
    at Module._compile (node:internal/modules/cjs/loader:1101:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1153:10)

thanks @sethusrinivasan for this, I'm not able to reproduce this issue I've tried to build with v16.13.1 and v14.19.0 but both works on my side. Are you able to run it in debug (npm start)?

closing this. Open a new issue in case of outstanding problems


[New] react/jsx-wrap-multilines: add never option to prohibit wrapping parens on multiline JSX

Fixes #1035
First PR here so if I'm missing anything please let me know!
(Also I'm aware this particular option is very controversial, please don't hate me 😇)

Relates to #1039 as well, i think.


Setup server on GCP, AWS or Azure without open 25 port.

Hello,
I am trying to setup an email relay server to approximately 15 million emails a day.
However, the place that is possible to use is on GCP, Azure or AWS, and both do not allow traffic on port 25.
how to conf server to send mail without port 25?

Your using of port 25 won't be the problem here, you will likely find your Postal unable to connect to other servers on the standard ports.
I do not know much about GCP and Azure but on AWS you can request permission to send emails from your servers.
Other server providers are available so its down to you to ask if they can allow email sending.

Thank you very much for responding.
You are consulted by GCP and Azure, and they cannot use the same.
I know that not all email servers will have this port 587, but the big ones will.
But the question is, is it possible for the postcard to send e-mails only using port 587?

It may be possible but if your server provider isn't allowing email, they will likely have that port blocked as well.


wheelRotate: false still rotates the element on wheel move

When passing wheelRotate: false in options, moving the wheel still rotates the element.

Can't reproduce in Firefox or Chrome. Please re-open if you can give me a browser / OS combination where this is broken.

Happens in Chrome - Latest version, Ubuntu Linux 16.04

Have same problem
Chromium Version 52.0.2743.116 Built on Ubuntu , running on LinuxMint 17.2 (64-bit)


uv sync fails for workspace if workspace member is declared to come from a named index

I ran into an issue with the new named indexes feature and workspaces.
Here is a minimal reproducible example (I'm on uv 0.4.24)

Start with the albatross-virtual-workspace example (https://github.com/thorbenk/uv-workspaces-question/commit/af0e74139f464c625dfe779cbdef0ebdebd8fb81)
Move the source declarations to the root (https://github.com/thorbenk/uv-workspaces-question/commit/48aad9e8ddd0c3ca5fb723ecf37dd77b3a74b8dc, as discussed in https://github.com/astral-sh/uv/issues/6786 this is OK and indeed, does NOT break uv sync)
But declaring seeds to come from a named index breaks uv sync (https://github.com/thorbenk/uv-workspaces-question/commit/869f602b75f759a1da3a8c8ee9d8b5c9777fc57b)

(It's probably not important if that package actually exists on that index; in my original private repository that was the case)

The error message is:
error: Failed to generate package metadata for `bird-feeder==1.0.0 @ editable+packages/bird-feeder`
  Caused by: Failed to parse entry for: `seeds`
  Caused by: Package is not included as workspace package in `tool.uv.workspace`

So you have a workspace member named seeds, but you're trying to define an alternate source for it? I don't think this is "allowed" right now: workspace members have to come from workspace sources.
(@konstin -- is that right?)

I actually think this isn't possible to support because we lock the full workspace together, so we'd always be trying to resolve the workspace member and the from-the-index member at the same time, which can't succeed.

OK, it looks like I may have been misusing this behavior for my use-case then.
The following setup worked for me before I tried to switch to named indexes:

The workspace members are independent python projects, each with it's own git repository, uv.lock file and associated CI pipeline. They depend on each other through released package versions in their pyproject.toml files.
These packages are all published to a private index.
For development purposes, I have created a "meta" repository to tie multiple of these packages together. The workspace members are git submodules. When hacking on a feature that spans multiple of these packages, I found the workspace setup very useful: I have overwritten the sources in the meta repository to declare the packages as workspace members, even though they themselves still reference each other through released package versions on some index. Everything is installed as editable packages inside a single workspace and it is easy to test that big new feature. Once everything works, I can still publish each workspace member separately as its own independent project.
Only after I tried porting to the new named indexes, I ran into the above problems

Probably I'm missing something, surely there has to be a better way to support such a development workflow?
(when I was using poetry before I used to constantly switch between path dependencies and released package versions manually in the pyproject.toml, which was a really bad way of doing things).

Editing dependencies in remote repositories is one of those problems we want to build a dedicated workflow for, but don't have good support yet.
Workspaces aren't well suited for this: A workspace assumes that it's the only source of truth for all packages it includes, so if the root says all of these packages are local path dependencies, the members can't say they are path dependencies. As a rule of thumb, if a member has it's own lockfile, it's probably not suited as a workspace member.
For now, using the submodules as editable path dependencies instead of as workspace members is likely the better approach.

@konstin Thanks for your explanation. I found the documentation in that regard a bit confusing (https://docs.astral.sh/uv/concepts/dependencies/#path), as it explicitly advises

However, it is recommended to use workspaces instead of manual path dependencies.

So, is the following setup something you would recommend doing: https://github.com/thorbenk/uv-path-dependencies-test/tree/main (it's a minimal example)?

Yeah we can improve the wording: https://github.com/astral-sh/uv/pull/8478
Submodules are an odd case because they are kinda part of the repository and kinda not; overall i hope to properly solve your use case with a dedicated workflow.


Unfocus active control after exterior click

This is especially important and visible for input box - when you
click outside of it, you expect the focus to be gone. On mobile devices
this also makes the virtual keyboard disappear when tapping outside of
a focused element.

On desktop this is definitely undesired, so cant merge as is.

Not sure why this is undesired on desktop.
<reduz> punto: actually this one i would like your input too, as you implemented this on ios
<reduz> as is, for desktop its undesired
<reduz> but may make sense on mobile
<reduz> dont really know
<reduz> may also not be the right way to do it on mobile
<Hinsbart> Why is it undesired on desktop? For me it kinda makes sense to lose focus after clicking somewhere else
<punto> doesn't the text input notify the os about the focus lost for the keyboard?
<reduz> yeah.. i guess that should remove the focus from the entry instead
<reduz> i dont know, can you comment on the issue?

I tested this on desktop, too, and the behaviour didn't make sense before the change, in my opinion. Clicking outside of a text box should remove focus, but it didn't. I'm not sure I understand @reduz's concern. Could you elaborate?

This makes sense, but probably needs discussion. As mentioned above, Godot editor works better with this behavior turned on. It also makes sense that this behavior works by default on mobile. So, should we:

Make this behavior the default on mobile (when has_touchscreen flag is true), but not on desktop?
Make this behavior the default, and make the editor change it?

What do you think?

I am in the camp that expect focus to be gone if you click outside of the control, so probably 2. If there are real good reasons for not going with 2, then 1, as it's clearly needed for mobile.

I think when it comes to Editor there is no solution "one fits them all", in one situation you wan't to loose focus on widget in other don't...

It seems this behavior is not standard everywhere. We could make it an option in the project settings with an override for mobile

Closing as no activity happened. I don't mind changing this making the setting (in project settings) optional, so it can be overriden per platform


use usvg to further simplify flags before waving

Another proof of concept.
With this I can wave all the 253 flags, no more warp_errors.log from wave_ninja.py.
yeah I know, it feels a bit like cheating. Maybe it could just be a temporary hack to get us going? :)
NOTE: this requires one to have usvg binary on PATH, which is acargo install usvg away.
We could even make usvg into a pre-built python wheel (using tools like maturin) and publish on PyPI (with resvg's author permission) so one can just pip install usvg (so that a full rust toolchain is not required to install).

It's good to know it's possible to get to 100% but ... how far away from working w/o this are we?

only one flag away (GS.svg) and this one is hard https://github.com/rsheeter/warp/issues/14#issuecomment-815987531


Add Hugging Face fields in metadata-api

Populate metadata with HF URL and last modified/updated.
https://discuss.huggingface.co/t/help-finding-creation-dates-for-hugging-face-models/35176

spraakbanken.gu.se currently calls Hugging Face to see if a resource exists, and uses this to display a HF link.
That is enough for now.


Rancher upgrade fails with no healthy host found error

I have one AWS instance for development purposes and I am using Rancher server and agent both in this server. The MySQL is not inside rancher server (it is another container in same host). There are a few services (dockerized) running on this server and the first time I set up the rancher everything works fine and services are deployed all fine.
The problem happens when I need to upgrade service for any reason and from that moment all my upgrades (even on other services start to fail). I tried many solutions I found and I checked the error logs but nothing meaningful found.
There is something interesting though. When "Scheduling" set to "Automatically pick hosts for each container matching scheduling rules" upgrade fails but if I set it manually to "Run all containers on a specific host" it doesn't fail and upgrade succeed. I think the error is totally misleading as it implies something is wrong with the host but it seems the scheduler is having some problem.
I would appreciate any help to set this right cause I was just started to like Rancher!
Thanks
In case it helps the OS is Ubuntu 16:04
What I see in Rancher UI: someservice (Failed to allocate instance [container:1i2917]: Bad instance [container:1i2917] in state [erroring]: Allocation failed: No healthy hosts meet the resource constraints: [809:80/tcp portReservation, instanceReservation: 1]) | Image:
What I see in Rancher container logs:
2017-09-06 03:43:54,827 ERROR [:] [] [] [] [cutorService-87] [.e.s.i.ProcessInstanceDispatcherImpl] Unknown exception running process [service.upgrade:46003] on [26], canceled by [State [removed] is not valid for process [instance.stop:46012] on resource [2736]]
time="2017-09-06T03:54:52Z" level=error msg="Waiting for 1s21 failed: Failed to allocate instance [container:1i2754]: Bad instance [container:1i2754] in state [erroring]: Allocation failed: No healthy hosts meet the resource constraints: [805:80/tcp portReservation, instanceReservation: 1]"
2017-09-06 07:18:56,639 ERROR [:] [] [] [] [utorService-105] [.e.s.i.ProcessInstanceDispatcherImpl] Unknown exception running process [service.upgrade:48154] on [21], canceled by [State [removed] is not valid for process [instance.stop:48168] on resource [2904]]
time="2017-09-05T09:50:25Z" level=error msg="Error Action [upgrade] not available on [&{1s22 service map[account:http://localhost:8080/v2-beta/projects/1a5/services/1s22/account consumedbyservices:http://localhost:8080/v2-beta/projects/1a5/services/1s22/consumedbyservices consumedservices:http://localhost:8080/v2-beta/projects/1a5/services/1s22/consumedservices instances:http://localhost:8080/v2-beta/projects/1a5/services/1s22/instances networkDrivers:http://localhost:8080/v2-beta/projects/1a5/services/1s22/networkdrivers stack:http://localhost:8080/v2-beta/projects/1a5/services/1s22/stack storageDrivers:http://localhost:8080/v2-beta/projects/1a5/services/1s22/storagedrivers self:http://localhost:8080/v2-beta/projects/1a5/services/1s22 serviceExposeMaps:http://localhost:8080/v2-beta/projects/1a5/services/1s22/serviceexposemaps serviceLogs:http://localhost:8080/v2-beta/projects/1a5/services/1s22/servicelogs containerStats:http://localhost:8080/v2-beta/projects/1a5/services/1s22/containerstats] map[rollback:http://localhost:8080/v2-beta/projects/1a5/services/1s22/?action=rollback remove:http://localhost:8080/v2-beta/projects/1a5/services/1s22/?action=remove finishupgrade:http://localhost:8080/v2-beta/projects/1a5/services/1s22/?action=finishupgrade removeservicelink:http://localhost:8080/v2-beta/projects/1a5/services/1s22/?action=removeservicelink addservicelink:http://localhost:8080/v2-beta/projects/1a5/services/1s22/?action=addservicelink setservicelinks:http://localhost:8080/v2-beta/projects/1a5/services/1s22/?action=setservicelinks]}] in upgrading service 1s22"
-- | --

Useful
Info

Versions
Rancher v1.6.7 Cattle: v0.183.5 UI: v1.6.17

Access
localauth admin

Orchestration
Cattle

Route
stacks.index

@ippbuilder how big is that host, spec-wise? Is it doing well on resources? Any errors in the docker logs that you can share?

When you select the "run all containers on a specific host" option, we force the containers to that host, even if there is a scheduling conflict.
When you don't select that option, we let the scheduler decide. The logs indicate that you want port 805, but no hosts have that port available.

how big is that host, spec-wise? Is it doing well on resources? Any errors in the docker logs that you can share?

The instance has 2 CPU and 8 GB memory which is more than enough to host a few small web app. I experienced the same setup with ECS and Docker Cloud which never ran our of resources.

When you don't select that option, we let the scheduler decide. The logs indicate that you want port 805, but no hosts have that port available.

I did realize the problem but my question is if there is no host available with that port then it should not work (whether I ignore scheduler or not).

Same problem here. Dont know how to fix it.

Hi guys, I jus started using rancher and I get into the same problem:
Rancher 1.6.14
Docker: 17.12.1-ce, build 7390fc6
I have Ranches in a VM with mysql in the host (the same host serve as node too) and another VM wich I use for the tests, I tried deploying in different ports and always get the same error:

if the container have no port opened to the world it works just fine:

There is no service in the host using such ports (these are the ones I tried):

Also checked the endpoint via the api for that particular host:

Actually I executed the docker-compose.yml and it worked just fine.
In the environment is only that stack (and the default created by rancher)
is there any other information I can give you to help here?
Regards.

Please let us know in a comment if you can still reproduce the issue, and we'll reopen it


Implement Payload::incoming_resource_estimate for Message

This PR removes the default implementation of the incoming_resource_estimate method in Payload trait and provides a dedicated one.
It sets the estimate to 1 for GetRequest::Deploy, GetResponse::Deploy and all request with tags that are used in the fast sync process:

Block
BlockAndMetadataByHeight
BlockHeaderByHash
BlockHeaderAndFinalitySignaturesByHeight
Trie

Closes https://github.com/casper-network/casper-node/issues/2644

This comment needs to be clarified before proceeding:
https://github.com/casper-network/casper-node/issues/2644#issuecomment-1029173071

Still unsure about deploy_responses, though. I wonder whether 0 would actually be safer?

bors merge


Add dark mode styling

When working on this, consider removing background colour from the graphs

Happy to give this one a go


implement nested run sequences

implement nested run sequences and arbitrary FORTRAN functions, as in this config:
https://github.com/NOAA-EMC/DATM-MOM6-CICE5/blob/develop/parm/nems.configure.medcold_atm_ocn_ice.IN

need to figure out how many sequences can be nested per-sequence, and how many layers deep the nesting can go

it seems that nested sequences run with even multiple time intervals of their containing sequence (i.e. a 30 minute sequence will run twice inside a 1 hour sequence)

pull request #15 implemented custom functions as members of a defined mediator object


b-table-column refers to wrong $parent

Overview of the problem
this line
No longer referrers to b-table when calling this.$parent
Buefy version: [DEV Branch Latest]
Vuejs version: [2.5.2]
OS/Browser: CHROME
Steps to reproduce

Npm link to dev version
Copy example from website on using tables
this.$parent will be the root component that is calling b-table-column

Expected behavior
this.$parent  refers to b-table
Actual behavior
this.$parent refers to root component

"this.$parent refers to root component" is miss leading, its just the top level component for that template.
`
      <b-table-column field="Title" label="Title">
      {{ props.row.Title }}
      </b-table-column>

      <b-table-column field="DateCreated" label="Created">
      {{ fmtTimeAgo(props.row.DateCreated) }}
      </b-table-column>

      <b-table-column field="DateModified" label="Last Changed">
      {{ fmtTimeAgo(props.row.DateModified) }}
      </b-table-column>

    </template>
  </b-table>
</div>

`

Looks like its specific to me, but I never had this problem until I reinstalled node_modules.
The tree is actually broken

If i install straight from NPM the issue doesn't arise, so I don't think its my dom tree, i just double checked it and all seems right

Solved! I was using an old version of vue-template-compiler in my main project and because it was npm linked it was causing issues.


When I rotate screen with drawer open and my finger keep touch drawer , some problem happen

@eseidelGoogle Please to verify.

This is no longer a reproduceable bug.


Missing docstring for make_dataset method

📚 Documentation
Public method make_dataset does not have a docsting:
https://github.com/pytorch/vision/blob/0467c9d74c9b34f91df905ed8cf8433de48d7fa5/torchvision/datasets/folder.py#L35-L42
It would be nice to add one.
cc @pmeier

I'm willing to pick that up if needed?

Fixed via https://github.com/pytorch/vision/pull/2879


Please add pAVAX token

C-chain checksummed address:  0x6ca558bd3eaB53DA1B25aB97916dd14bf6CFEe4E
Token logo 200X200 PNG:

Or just you know.. read the instructions and update it yourself by forking the repo..


[BUG] Webhook is installed in the downstream cluster post rancher backup/restore roll back on a migrated cluster

Information about the Cluster
Rancher Server Setup

Rancher version: v2.7.0 upgraded to v2.7-head e2410e rolled back to 2.7.0
Installation option (Docker install/Helm Chart): Helm Chart

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc): RKE1, v1.24.6, 1.4.0 Hardened

Proxy/Cert Details: Self-signed

Information about the Cluster

Kubernetes version: v1.24.10 non-hardened RKE1 cluster and a hardened RKE2 cluster v1.24.10+rke2r1
Cluster Type (Local/Downstream): Custom  3 worker, 1 etcd, 1 cp RKE1 and custom 1 worker, 1 cp and 1 etcd RKE2

Describe the bug
On a rancher server upgrade and roll back to 2.7.0 webhook version v0.3.2-rc19 is installed in the downstream clusters. Local cluster is migrated from a hardened RKE1 to non hardened RKE1 cluster.
To Reproduce

Install rancher on version 2.7.0
Create a custom RKE1 and a custom hardened RKE2 cluster with the specs mentioned above.
Install rancher backup/restore chart version 3.0.0 in the local cluster with config set to S3
Take a backup from the rancher backup/restore page and the backupfile gets saved in the s3.
Upgrade rancher server to v2.7-head
Create a new rke1 standalone cluster
Bring down the nodes of current rancher local cluster
Migrate rancher to v2.7.0 by following instructions here https://ranchermanager.docs.rancher.com/v2.6/how-to-guides/new-user-guides/backup-restore-and-disaster-recovery/migrate-rancher-to-new-cluster

Result
Once the migration is successful, and rancher is installed if we navigate to downstream clusters, we see the webhook is installed downstream.
Expected Result
Webhook is not installed on downstream clusters.
Screenshots

Release Note
To enforce new security boundaries, rancher's webhook will now be installed in all downstream clusters. Users who rollback from a version of rancher >= v2.7.2 to a version < 2.7.2 will experience an issue where these webhooks will remain in downstream clusters. Since the webhook is designed to be 1-1 compatible with specific versions of rancher, this can cause unexpected behaviors to occur in downstream clusters. The rancher team has developed a script which should be used after rollback is complete (meaning after rancher version < 2.7.2 is running) to remove the webhook from the affected downstream clusters.

Verified on a migration from v2.7.0 to v2.7-head cf555aa back to v2.7.0 with byo-valid certs and self-signed certs helm rancher installation and docker rancher installation:

Install rancher on version 2.7.0
Create a custom RKE1 and a custom hardened RKE2 cluster with the specs mentioned above.
Install rancher backup/restore chart version 3.0.0 in the local cluster with config set to S3
Take a backup from the rancher backup/restore page and the backupfile gets saved in the s3.
Upgrade rancher server to v2.7-head
Create a new rke1 standalone cluster
Bring down the nodes of current rancher local cluster
Migrate rancher to v2.7.0 by following instructions here https://ranchermanager.docs.rancher.com/v2.6/how-to-guides/new-user-guides/backup-restore-and-disaster-recovery/migrate-rancher-to-new-cluster
Run the script as mentioned here.
Verified from downstream - imported rke1, rke2 and rke1 node driver clusters the webhook has been removed.
Created a 3 users and added them as cluster owner and cluster member in the downstream clusters.
Verified the  user creation was successful
Login as the user1 - CO and verified edit on the cluster and delete cluster works as expected
Login as user2- CM and verified edit/delete of cluster fails as expected.

Sorry for a potentially silly question, but does this issue address CVE-2023-22651 as mentioned in the release notes? Or perhaps some other issue is addressing that CVE?
For context, I am just looking where exactly in the rancher code the fix is located.

Hi @zpavlinovic 👋🏻 We just updated the advisory CVE-2023-22651 and added a reference section with the link to the commit that fixed the CVE.

Thank you!


Consolidate and simplify list views (continued)

Continue on from 7eb10f2b26a593c17dfcc21f506a7fa451fc426d , consolidating Compact and Card views too.
Also, consider consolidating recent Talks/Pubs/Posts and featured Talks/Pubs into either generic Recent and Featured widgets or consolidating into one combined recent/featured widget for each content type.
Todo:

[x] Compact view
[x] Card view
[ ] Consolidate widgets as per above?

Breaking changes

abstract_short field dropped in publications and talks in favour of summary field.

I've been using the talk section as a list of talks I can present at conferences but not an actual schedule of talks. so I've used the custom overrides of the partials for talk_li_* to remove the date/time, event name, etc fields.
now that you've consolidated to reuse generic li_* templates, I cannot do this. Any suggestion on how to handle this edge case situation?

@AlanBarber it's possible to override the new single layout file similar to your previous setup. The additional step in this case would be to modify the {{ if eq .Type "talk" }}" talk block according to your preferences.


[INFRA] Enable shellcheck on CI

Why are the changes needed?

Fix #2062
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

Thanks, merged to master

I'm pleasure that

@xiaoyuandajian I just noticed that[1]

ludeeus/action-shellcheck@master is not allowed to be used in apache/kyuubi.

I opened the https://issues.apache.org/jira/browse/INFRA-24064 to ask the ASF INFRA team to evaluate this plugin and hope it can be added to the whitelist.
[1] https://github.com/apache/kyuubi/actions/runs/3856320011

yes, I hope so


Regulate positive-negative values in the generated data

Description

I updated the logic of the filter module, fixed the issue where positive and negative values were not correctly applied during sampling, and added a test package synthesizer for establishing functional tests related to synthesizers.
Additionally, I founded that a series of logical issues during ctgan training; I refactored this part to make its logic clearer and added some comments explaining its background.
Moreover, I believe that some built-in filter can be integrated, such as the range value filter.
Motivation and Context

Related:

https://github.com/hitsz-ids/synthetic-data-generator/issues/231
https://github.com/hitsz-ids/synthetic-data-generator/issues/189

How has this been tested?

See tests/synthesizer/test_ctgan_synthesizer.py
Types of changes

[ ] Maintenance (no change in code, maintain the project's CI, docs, etc.)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

I'm not too familiar with the original design of this piece, but it looks OK.
cc @MooooCat

@Wh1isper I have made many attempts, but I don't know why the CI/CD test cases are failing. Can you help me take a look? It works fine on my local machine. 😩

@Wh1isper 🫠 Yeah, what should I do？

I reproduced it locally by logging. It seems to be caused by class variable pollution.
For some unknown reason, it is the variable pollution caused by the same field names of NonValueTransformer and OutlierTransformer, and further reasons are still being explored.
2024-11-07 11:33:12.401 | INFO     | sdgx.data_processors.transformers.nan:fit:71 - NonValueTransformer Fitted.
2024-11-07 11:33:12.401 | INFO     | sdgx.data_processors.transformers.nan:fit:74 - NonValueTransformer get column list from metadata: ['int_id', 'pos_int', 'neg_int', 'pos_float', 'neg_float', 'mixed_int', 'mixed_float'].
2024-11-07 11:33:12.401 | INFO     | sdgx.data_processors.transformers.nan:fit:75 - Metadata int columns: {'int_id', 'pos_int', 'mixed_int', 'neg_int'}.
2024-11-07 11:33:12.401 | INFO     | sdgx.data_processors.transformers.nan:fit:76 - Metadata float columns: {'pos_float', 'mixed_float', 'neg_float'}.
2024-11-07 11:33:12.401 | INFO     | sdgx.data_processors.transformers.nan:fit:77 - Self int_columns: {'hours-per-week', 'educational-num', 'fnlwgt', 'capital-loss', 'age', 'capital-gain'}.
2024-11-07 11:33:12.402 | INFO     | sdgx.data_processors.transformers.nan:fit:93 - NonValueTransformer get int columns: {'pos_int', 'hours-per-week', 'educational-num', 'mixed_int', 'fnlwgt', 'capital-loss', 'neg_int', 'age', 'capital-gain'}.
2024-11-07 11:33:12.402 | INFO     | sdgx.data_processors.transformers.nan:fit:102 - NonValueTransformer get float columns: {'pos_float', 'mixed_float', 'neg_float'}.

@Wh1isper
For _clear(): This is just a temporary solution. In fact, I don't know why this situation occurs; when I run this test case individually, there are no issues, but when running all cases at once, the problem arises. Basically, each processor should be new and there shouldn't be any "old data." I only added this _clear in fit to make it pass the test. Do you have any idea on this?
For your suggestion: I'm not quite sure where to appropriately call this after setting it to public based on your suggestion. My thought is to call it when obtaining processors ( sdgx/synthesizer.py#fit()-line-296 ) when starting the fit. How do you think?

I recommend checking all inspect and transforms
As for the clean method, it can be implemented as optional if Synthesizer doesn't use it. Users may be able to implement Synthesizer themselves to use these methods.

@Wh1isper

I recommend checking all inspect and transforms
As for the clean method, it can be implemented as optional if Synthesizer doesn't use it. Users may be able to implement Synthesizer themselves to use these methods.

Okay. I think we should conclude this PR here. As for the issue with the variable class in the processor, it's a refactoring issue, so let's address it in a new PR. Let's merge this one first; what do you think?

As for the issue with the variable class in the processor, it's a refactoring issue

Acturally it's a bug and no conflict with the original design, can you do it in this pr? If not, we cloud leave it in next pr, but this pr should not include clear func as a temporary measure to get tests passed. To be clear, I agree that clean is not necessary which should be removed(or revertd)

Given that we'll be fixing the problem immediately, a temporary failure of the CI is acceptable, in which case I agree to merge this pr first.

As for the issue with the variable class in the processor, it's a refactoring issue

Acturally it's a bug and no conflict with the original design, can you do it in this pr? If not, we cloud leave it in next pr, but this pr should not include clear func as a temporary measure to get tests passed. To be clear, I agree that clean is not necessary which should be removed(or revertd)

🧐 Okay, The ❌ of CI/CD is annoying. I would have try if     def __init__(self): self.x = [] works.

class A:
   x: list

   def __init__(self):
       self.x = []

   def callx(self):
       self.x.append("1")

a = A()
a.callx()
print(A.x)  # AttributeError: type object 'A' has no attribute 'x'
print(a.x)  # [1]

@Wh1isper The approach you mentioned works out. and I have already submitted it. Could you help me review it?
Furthermore, I suddenly realized this is a basic question that I couldn't figure out. 😅


Add no-unnecessary-type-check rule

Rule Suggestion
Is your rule for a general problem or is it specific to your development style?
General problem
What does your suggested rule do?
Warns you if you compare against null or undefined unnecessarily.
List several examples where your rule could be used
const x = "my string"; // definitely not undefined or null

if (x !== undefined) { // should warn
    console.log(x);
}
if (x !== null) { // should warn
    console.log(x);
}

function getStringOrUndefined() {
    return Math.random() > 0.5 ? "string" : undefined;
}

const y: string | undefined = getStringOrUndefined();
if (y !== undefined) { // should not warn
    console.log(y);
}

Additional context
Checking if a variable is undefined unnecessarily is a problem for two reasons. Either the variable can actually never be undefined - in which case the check is unnecessary and probably confusing to a reader. Or, the variable can be undefined (i.e. the code-writer knows something that the compiler doesn't) - in which case the typings should be updated to reflect that the variable might be undefined.
In the former case, this rule more more for style. But in the latter (and imo more important) case, this rule can prevent bugs - i.e. since the type is wrong, there could be a variable of that type somewhere that could be undefined but neither the person using the variable nor the compiler know this.
Submitting a PR with this rule shortly

🤔 this seems like a subset of strict-type-predicates: https://palantir.github.io/tslint/rules/strict-type-predicates/
Tempting to close as a duplicate and already resolved... Thoughts @zachkirsch?

Oh man! I guess i should have made the issue first..
Well it was a good learning experience :)


Restore v4 docs on readthedocs

I'm still using v4 in a project, and it'd be a pain to upgrade right now. Since those docs are no longer on the documentation site, I'm having to use archive.org to read them. Could you restore them please?

You can switch to tag 4.* (e.g. https://github.com/guzzle/guzzle/tree/4.2.3) and get the docs from the appropriate directory (e.g. https://github.com/guzzle/guzzle/tree/4.2.3/docs)

Oh, I didn't realize github would actually format the *.rst files!


Modulo Simplification

Is your feature request related to a problem? Please describe.
I'm always frustrated when I see a bunch of shifts, division, logical and other arithmetic for the modulo operation.
Describe the solution you'd like
The operations to be simplified into the modulo operation.
Describe alternatives you've considered
Opening up the program in IDA to see what is really going on. I am guilty of doing exactly this as it still beats breaking out a pencil and paper to figure it out.
Additional context
int mod6(int i) {
    return i % 6;
}

Here are some xml debug function decompilation outputs. (This one is guilty of not automatically appending the xml file extension)
mod2.xml.txt
mod3.xml.txt
mod4.xml.txt
mod6.xml.txt
See, I do know what the modulo operator is...

So you want to undo peephole optimization. Is there any way besides hardcoding a list of heuristics? Sounds like an interesting project.

There has been a lot of progress with deoptimizing/recovering division by constants, this should be pretty similar, I think.

So you want to undo peephole optimization. Is there any way besides hardcoding a list of heuristics? Sounds like an interesting project.

This was compiled with -O1. I understand why it would be difficult to detect but it's extremely common. If it didn't obfuscate the decompilation results then I wouldn't complain about it.

Like @Swyter mentioned, work has been done for division by constants (see #1718, #668). Something like 2463cad should be doable eventually?

Thank you! @caheckman


CCCS-65-Start-button-Tutorial

Completed the Start/Splash page for the project. Added a working demo tutorial so the user isn't completely lost. Currently unskippable at this time. Doesn't use time.

tutorial character cutout thing is broken. seems like it only alligns at one specific window size. please make them align, at least on most typical window sizes.

@Tartarus6 Fixed for the first window. Let me know if it works on your machine so I can do the other ones.

that fixed it, might want to make the boxes a bit bigger that the actual thing youre highlighting, now it sorta looks like the thing is just bright. just a suggestion, might not be worth the effort.

Don't care. Also fixed resizing.

some things no longer allign. the graph doesnt line up anymore, as well as the PC.

@Tartarus6 Fixed for graph.

UTD with main. Awaiting merge.

Pretend I looked at it. I'm tired so lets send this into main.


Fixed is_array property for loggers

Related Issue:
N/A
Description of changes:
This was a simple fix discovered via testing through enhancements to the new metaschemas from https://github.com/ocsf/ocsf-schema/pull/736 that cover dictionary.json files. is_array is meant to be a boolean field (true) but in this case is defined as a string instead "true".

Tagging @pagbabian-splunk from the original PR.


error.handling.codes doesn't overwrite

I try to wrap IllegalArgumentException with
error.handling.codes.java.lang.IllegalArgumentException=ILLEGAL_ARGUMENT as is written in documentation, but result is still tomcat error page not JSON response. My test is to put illegal simbols like 1'||( on Integer @RequestParam.
Do you have any idea how I can turn it to generic JSON respons, without trace information, like the others?
SpringBoot 2.7.6 with 3.3.0 error-handling

Can you read https://wimdeblauwe.github.io/error-handling-spring-boot-starter/current/#handle-non-rest-controller-exceptions and see if this applies to what you are trying to do?

When I test this, I see I get this by default:
{
  "code": "ARGUMENT_TYPE_MISMATCH",
  "message": "Failed to convert value of type 'java.lang.String' to required type 'java.lang.Integer'; For input string: \"1'||(\"",
  "expectedType": "java.lang.Integer",
  "property": "integer",
  "rejectedValue": "1'||("
}

Using this endpoint:
    @GetMapping("/request-param")
    void test(@RequestParam Integer integer) {

    }

This is because of this default mapping:
error.handling.codes.org.springframework.web.method.annotation.MethodArgumentTypeMismatchException=ARGUMENT_TYPE_MISMATCH

If you can send me a test case, I can have another look.

This is exactly what I want but in mine case I got Tomcat error. Here is mine test endpoint
@GetMapping(path = "/test", produces = MediaType.APPLICATION_JSON_VALUE)
public ResponseEntity<?> testEndPoint(@RequestParam("page") Integer page) {
    return ResponseEntity.ok().build();
}

with or without error.handling.codes maps the result doesn't change and I can't figure out where I'm going wrong

Could you try to create a small test program that shows the problem so I can debug it further?

test.zip
here it is

When I run your application and get the request using the IntelliJ HTTP client, I get this:
{
  "code": "ARGUMENT_TYPE_MISMATCH",
  "message": "Failed to convert value of type 'java.lang.String' to required type 'java.lang.Integer'; nested exception is java.lang.NumberFormatException: For input string: \"1'||(\"",
  "expectedType": "java.lang.Integer",
  "property": "page",
  "rejectedValue": "1'||("
}

When I change the application.yaml to:
error:
  handling:
    codes:
      java.lang.IllegalArgumentException: ILLEGAL_ARGUMENT
      org.springframework.web.method.annotation.MethodArgumentTypeMismatchException: ILLEGAL_ARGUMENT

Then I get:
{
  "code": "ILLEGAL_ARGUMENT",
  "message": "Failed to convert value of type 'java.lang.String' to required type 'java.lang.Integer'; nested exception is java.lang.NumberFormatException: For input string: \"1'||(\"",
  "expectedType": "java.lang.Integer",
  "property": "page",
  "rejectedValue": "1'||("
}

So I am afraid I don't see any issue.

Can you try with
http://localhost:8888/api/test?page=|

Same:
{
  "code": "ILLEGAL_ARGUMENT",
  "message": "Failed to convert value of type 'java.lang.String' to required type 'java.lang.Integer'; nested exception is java.lang.NumberFormatException: For input string: \"|\"",
  "expectedType": "java.lang.Integer",
  "property": "page",
  "rejectedValue": "|"
}

I also tested in the browser and same result. And you get a different result using this test application as well? Or are you only testing in your actual application?

Apparently the problem is local only for me, but I don't understand why. Both my real application and test app are like that. Anyway, thanks for your time, I'll look for the reason

Closing as the issue does not seem reproducible for me.


Export Date state then import back convert into Strings

When export a state contains a date object, then import the same state back, the object is stored as string instead of date

I'm experiencing this issue when my dates are meant to be moment objects, but just come back as strings and then none of the state loads.

@holyman2k, @JemarJones in order the extension to handle unserializable  by JSON.stringify data (like dates, regexes, undefined, error objects, symbols and functions) just set serialize parameter to true:
const store = Redux.createStore(reducer, window.__REDUX_DEVTOOLS_EXTENSION__ && window.__REDUX_DEVTOOLS_EXTENSION__({
   serialize: true
}));

See Troubleshooting and API for more details.
Note that the extension implementation are completely different from the repository you were filing the issue, thus the confusion.

Oh, and here i was thinking it didn't have anything to do with the extension 😅. Also i seem to have something slightly different than what you have there, i have:
const store = createStore(
  appReducer,
  compose(
    // The logger MUST be last (other than DevTools)
    applyMiddleware(routerMiddleware(browserHistory), thunk, logger),
    window.devToolsExtension ? window.devToolsExtension() : f => f
  )
);

Should i just do:
const store = createStore(
  appReducer,
  compose(
    // The logger MUST be last (other than DevTools)
    applyMiddleware(routerMiddleware(browserHistory), thunk, logger),
    window.devToolsExtension ? window.devToolsExtension({
        serialize: true
    }) : f => f
  )
);

?

@JemarJones yes, your snippet looks good to me. Just keep in mind that window.devToolsExtension will be deprecated in favour of window.__REDUX_DEVTOOLS_EXTENSION__.


Add support for defining streams

Related: https://github.com/microsoft/typespec/issues/154

:x: There is undocummented changes. Run chronus add to add a changeset or click here.
The following packages have changes but are not documented.

:x:@typespec/compiler
:x:@typespec/events
:x:@typespec/http-server-csharp
:x:@typespec/http
:x:@typespec/openapi
:x:@typespec/openapi3
:x:@typespec/rest
:x:@typespec/sse
:x:@typespec/streams

Show changes

You can try these changes here

🛝 Playground
🌐 Website
📚 Next docs

https://github.com/Azure/typespec-azure/pull/1615 addresses the e2e test failures.


Add a new API (empty implementation) in Alfred for token revocation

This PR add a new API in Alfred for token revocation feature. Currently it is an empty implementation and only return 503.
It also adds a new scope (permission) for this new API: token:revoke
This new scope is not in the same place as other scopes because scopes are usually defined in protocol package, which is out of server package. Not sure if it should be moved there. Or we can move it later in the future.
The API url is like this: ${alfred_url}/documents/${tenantId}/document/${documentId}/revokeToken
This API can be invoked only if the token include the new scope: token:revoke.

Let's put the new scope (token:revoke) to the same place as existing scopes.


feat: improve headers

This pull request improves the definition of default headers.
As fix for the wrong naming of X-device-Type and X-device-OS I added absolute references to the corrected version. To check effectiveness after merging we need to look at https://apis.zalando.net/apis/af983b04-fc6f-11ec-b939-0242ac120002/ui.

👍

:+1:

:+1:


clip example crashes with WGPU

cargo run --example clip --features "render-wgpu"
produces this error:
wgpu error: Validation Error

Caused by:
    In Device::create_bind_group
      note: label = `create_clip_mask_bind_group_no_texture`
    Number of bindings in bind group descriptor (0) does not match the number of bindings defined in the bind group layout (1)

note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

After debugging, I found that it produces WgpuShader::Simple with StandardShaderVariant::ClippingMask, but the clip_texture is None, so it uses clip_mask_layout as there would be a clip_texture, but there is no actually.

Should be fixed as of b555d1c84e858c9c6388517cfd17bda6d6729bc5 - problem was it just wasn't updating the clip mask in the state, and the shader wasn't using the right coordinates.


[move-prover] Eliminate fall through and make all control jumps explicit

Motivation
In an earlier PR #3656, I attempted to make all control jumps explicit by eliminate all sources of fall through.   I missed one case however which I am fixing with this PR.  As a result, the implementation of get_successors is tightened.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
Existing tests
Related PRs
#3656

@bors-libra r=wrwg

:pushpin: Commit 70ec783 has been approved by wrwg

:hourglass: Testing commit 70ec7836dbcd25fcf92f9d71b273b34f8b297e63 with merge 668bc020315479369a2d251fd46e261d4823411b...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: wrwg
Pushing 668bc020315479369a2d251fd46e261d4823411b to master...


Masks of the same class predicted as if being 1 object

Hello,
I managed to train on a custom dataset and get very good prediction results on instance segmentation, but all the masks on the image appear inside 1 box, as if being the same object.
Is there a parameter to change for each mask to be recognised as a different object? (in the annotations, there is 1 mask per object).
In 1 target, I have for instance 5 segmentations, and 1 bbox. Could this be the issue? Are the different masks supposed to be in different targets? I followed the instructions on this post to annotate my images so I don't understand why would the annotations be wrong but...
Thank you in advance for any insight on this.

Uhh I'm not sure what you mean by that because in COCO, there's one mask per bbox. I haven't used labelme myself, but check the resulting JSON to see if it makes sense. You can do this by opening the json using the json package in python, then check the 'annotations' list. That should have as many annotations as you have segmentations, so check if the length of that list is appropriate.
Then you should also check the bbox for those annotations to check if they surround the mask. Your bounding boxes should surround just the mask that's being predicted and include no padding around that, since that could confuse the network.

Sorry, it turns out it was an issue with the labels.
When using labelme, the different objects on the frame must have names differentiated by -#, for instance object-1, object-2 and not object1, object2.

@melodie-r
How to label a single category with labelme? After looking at the user-defined data set, there is no annotation naming method. For example, there is only one category in a graph: cat, cat, cat or CAT1, cat2, Cat3 or cat-0, cat-1, cat


Update weekly meeting notes from March-April 2017

what is the change?:
We had few things to note each week and were deferring updating the
meeting notes.
why make this change?:
Based on issue #1193
it sounds like the meeting notes will provide signal that the project is
still maintained.

nice, thanks


set_error_handler doesn't handle string rejections

Not sure if this is a bug or an expected limitation.
In express I use express-promise-router to add this. Would be nice to see this built into hyper-express.
const HyperExpress = require('hyper-express');
const webserver = new HyperExpress.Server();

// Works
webserver.get('/works', () => {
    throw new Error('Hello World');
});

// Never responds
webserver.get('/not-working', async () => {
    await new Promise((_resolve, reject) => {
        return reject('Hello World');
    });
});

// This is only called for the thrown error but not the rejection
webserver.set_error_handler((_request, _response, error) => {
    console.log('error', error);
});

webserver.listen(80)
.then(() => console.log('Webserver started on port 80'))
.catch(() => console.log('Failed to start webserver on port 80'));

Interesting, I thought that https://github.com/kartikk221/hyper-express/blob/29e81dfb92abd3e771f2b3ad42b9deba48e43441/src/components/Server.js#L461 would catch any Promise rejections from the route handler. Could you try forking and utlilizing this older wrapper https://github.com/kartikk221/hyper-express/blob/2b2252f89cc7f0962cdcede6b6fb4fa4a444f902/src/components/Server.js#L474 to see If that catches your specific rejection?

I'll give that a go today and let you know.

Oh found the issue.
// Never responds
webserver.get('/not-working', async () => {
    await new Promise((_resolve, reject) => {
        return reject('Hello World');
    });
});

// Responds
webserver.get('/working', async () => {
    await new Promise((_resolve, reject) => {
        return reject(new Error('Hello World')); // The new Error() is required otherwise it never finishes the request
    });
});

This issue has been fixed in the recently released v6.2.4 of HyperExpress. Let me know If you encounter any other issues.

Thank you so much, this works perfectly. 🥳


Flag mode as bool instead of unit

Describe the feature
My model has a bool column to indicate the soft-delete, not a uint.
It will be great if this plugin can support bool flag.
Motivation
Trying this plugin raises error:
"sql: Scan error on column index 7, name \"Deleted\": converting driver.Value type bool (\"false\") to a uint: invalid syntax"

when reading from DB bool column to model soft_delete.DeletedAt field.
Related Issues

I've just read about https://gorm.io/docs/serializer.html
Going to try it. Don't know if it will work with this plugin.


Have job duration inside Job selector

Description
A piece of new information should be present inside the job selector: the job duration.
Acceptance Criteria

[ ] Ensure new code is unit tested, and check code coverage is at least 90%.
[ ] Create related issue in taipy-doc for documentation and Release Notes.
[ ] Check if a new demo could be provided based on this, or if legacy demos could be benefit from it.
[ ] Ensure any change is well documented.

Code of Conduct

[X] I have checked the existing issues.
[ ] I am willing to work on this issue (optional)

Blocked by: #1544


@ buzz GA Release

@jelicacomo app launch outline here.
ETA launch date:
Cross Stores Checklist

[x] The name of you app (should be 30 characters or less to comply with both stores)
[ ] Secure and activate an @sign for your app
[x] A logo for your app that can be used as an Icon in both stores
[x] Privacy policy + URL (we recommend GitHub pages) @nickelskevin @tinashe404
[x] Website landing page for your app @rtilney
[ ] A wavi.ng page for your app @rtilney

Apple App Store Checklist
To get set up in the Apple App store, you will need:

[x] Name of your app (30 characters, reviewed by Apple before it is made available)
[x] Subtitle for your app (30 characters, reviewed by Apple before it is made available)
[x] App Store Icon
[ ] Category selections for your app @nickelskevin
[x] Content Rights selection
[x] Age Rating selection
[x] License Agreement selection (your own or Apple’s Standard License Agreement)
[x] Pricing information
[ ] Availability (by country) selection
[x] Privacy Policy URL (required) @nickelskevin @rtilney
[x] User Privacy Choices URL (optional)
[x] Data Collection Type
[ ] What’s new in the version to be released
[x] Up to three App Preview Videos @rtilney
[x] Up to 10 screenshots for iPhone 6.5" display @rtilney @tinashe404
[x] Up to 10 screenshots for iPhone 5.5" display @rtilney @tinashe404
[x] Up to 10 screenshots for iPad Pro (3rd Gen) 12.9" display @rtilney @tinashe404
[x] Up to 10 screenshots for iPad Pro (2nd Gen) 12.9" display @rtilney @tinashe404
[ ] Promotional Text
[x] A description of your app, detailing features and functionality
[ ] Keywords for search optimization
[x] Support URL for your app (could be a mailto://)
[x] Marketing URL for your app (AKA landing page) @rtilney
[x] Copyright details

Google Play Store Checklist
To get set up in the Google Play store, you will need:

[x] Name of your app (up to 50 characters, reviewed by Google before it is made available)
[x] Short description (up to 80 characters, reviewed by Google before it is made available)
[ ] Full description (up to 4,000 characters, reviewed by Google before it is made available)
[x] App icon @rtilney
[x] Feature graphic @rtilney
[ ] Feature YouTube video
[x] App or Game selection for your app
[x] Category selections for your app @nickelskevin
[ ] Tags for search optimization (up to 5 from their list)
[x] Privacy Policy URL (required) @nickelskevin @rtilney
[x] Ads Policy
[x] App access policy (for testing) @nickelskevin
[x] Content ratings
[x] Target audience and content
[x] Pricing information
[x] 2 to 8 phone screenshots @rtilney
[x] Up to 8 screenshots for 7" tablet display @rtilney @tinashe404
[x] Up to 8 screenshots for 10" tablet display @rtilney @tinashe404
[x] Marketing URL for your app (AKA landing page)

@VJag or @murali-shris can you please add an update here with regard to any blockers or major bugs still to be resolved? Also what is the status of the Apple store blocker? If you can include timing of resolving issues that would be great. TY
SDK release pending (Thursday 8/19 target)


Make the URL clickable

A simple contribution to make the URL clickable instead of copying and pasting....
This code has a problem.... The URL opens in a new window inside electron.
I don't know how electron works, but maybe this can help you guys:
$(document).on('click', 'a[href^="http"]', function(event) {
    event.preventDefault();
    shell.openExternal(this.href);
});

or
require("shell").openExternal("http://www.google.com")

Thanks for the great tool! :)

Haven't tested, but this should work.

Looks good, thanks!


support additional attributes for various objects in Payload

The API docs are pretty clear that it's ok to send non-standard attributes in the item JSON, but rollbar-java 1.0.0 has removed this capability. (It could be done for some, but not all, objects in 0.5.4.)
Would you consider re-adding this capability?
I find it useful to record things like the response code, the timestamps at the beginning and the end of the request, extra person attributes, etc.
I'm sorry I wasn't able to get this feedback to you before 1.0.0 was released!

I assume you mean for the objects where we say:

// Can contain any arbitrary keys. Rollbar understands the following:

Yeah we should add this back, will figure out the best way to do this.

Yes, that's what I meant.
Thank you!

We should allow these on Person, I agree.


[ADAM-1513] Strandedness for FeatureRDDs

This was the lightest change that added the most flexibility for the future.

Coverage decreased (-0.07%) to 82.728% when pulling 2377e9dfe7154df05b0342b6edb19617f2f2dd8e on devin-petersohn:issue#1513strandedness into b7762c2b38b2bae00ca3e67bcf04041db312f2b3 on bigdatagenomics:master.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/2079/Build result: FAILURE[...truncated 15 lines...] > /home/jenkins/git2/bin/git fetch --tags --progress https://github.com/bigdatagenomics/adam.git +refs/pull/:refs/remotes/origin/pr/ # timeout=15 > /home/jenkins/git2/bin/git rev-parse origin/pr/1555/merge^{commit} # timeout=10 > /home/jenkins/git2/bin/git branch -a -v --no-abbrev --contains 0eb3bcb4d7c660f505f886d16e07dcd7451842d8 # timeout=10Checking out Revision 0eb3bcb4d7c660f505f886d16e07dcd7451842d8 (origin/pr/1555/merge) > /home/jenkins/git2/bin/git config core.sparsecheckout # timeout=10 > /home/jenkins/git2/bin/git checkout -f 0eb3bcb4d7c660f505f886d16e07dcd7451842d8First time build. Skipping changelog.Triggering ADAM-prb ? 2.3.0,2.10,1.6.1,centosTriggering ADAM-prb ? 2.3.0,2.11,2.0.0,centosTriggering ADAM-prb ? 2.6.0,2.10,2.0.0,centosTriggering ADAM-prb ? 2.6.0,2.11,1.6.1,centosTriggering ADAM-prb ? 2.6.0,2.10,1.6.1,centosTriggering ADAM-prb ? 2.3.0,2.11,1.6.1,centosTriggering ADAM-prb ? 2.6.0,2.11,2.0.0,centosTriggering ADAM-prb ? 2.3.0,2.10,2.0.0,centosADAM-prb ? 2.3.0,2.10,1.6.1,centos completed with result FAILUREADAM-prb ? 2.3.0,2.11,2.0.0,centos completed with result FAILUREADAM-prb ? 2.6.0,2.10,2.0.0,centos completed with result FAILUREADAM-prb ? 2.6.0,2.11,1.6.1,centos completed with result FAILUREADAM-prb ? 2.6.0,2.10,1.6.1,centos completed with result FAILUREADAM-prb ? 2.3.0,2.11,1.6.1,centos completed with result FAILUREADAM-prb ? 2.6.0,2.11,2.0.0,centos completed with result FAILUREADAM-prb ? 2.3.0,2.10,2.0.0,centos completed with result FAILURENotifying endpoint 'HTTP:https://webhooks.gitter.im/e/ac8bb6e9f53357bc8aa8'
Test FAILed.

Coverage decreased (-0.07%) to 82.728% when pulling 094f1e75a444acc418b8bddeee7fba66b054578a on devin-petersohn:issue#1513strandedness into b7762c2b38b2bae00ca3e67bcf04041db312f2b3 on bigdatagenomics:master.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/2080/
Test PASSed.

Coverage decreased (-0.08%) to 82.713% when pulling 1f0870c9c3a3728d362f958c8f7d20329c0a756f on devin-petersohn:issue#1513strandedness into b7762c2b38b2bae00ca3e67bcf04041db312f2b3 on bigdatagenomics:master.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/2081/
Test PASSed.

Coverage decreased (-0.6%) to 82.573% when pulling f42238b4b19a974bfc1c433a15db8d99da4cefca on devin-petersohn:issue#1513strandedness into ad5ae6df68c87e34caf42d7877be3064fea95a4f on bigdatagenomics:master.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/2098/
Test PASSed.

Coverage decreased (-0.4%) to 82.713% when pulling 71c22475de74e9ec8ce37b96408e990e1df4271c on devin-petersohn:issue#1513strandedness into ad5ae6df68c87e34caf42d7877be3064fea95a4f on bigdatagenomics:master.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/2099/
Test PASSed.

@devin-petersohn can you rebase this and clear the conflicts?

Also, shouldn't the stranded parameter propagate through to the flattenByReferenceRegion method and all the joins?

Yes, it should propagate. I also need to add unit tests for stranded = true.

Coverage decreased (-0.8%) to 82.573% when pulling b0bd8968f583c7ac699a360e73b7470d3ebbf7d5 on devin-petersohn:issue#1513strandedness into 152a8ad17509919f25a4f5b7c04161c5b313fc33 on bigdatagenomics:master.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/2103/
Test PASSed.

Closing as WontFix.  Feel free to create a new PR after rebasing against git head.


Without source code it shows 404 after build and start

Example name
https://github.com/zeit/next.js/tree/canary/examples/custom-server-koa
Describe the bug
404 error inhttp://localhost:3000/
To Reproduce

npm i
npm run build
Delete pages folder ,because I only want compiled files,like .next folder.
npm run start     (Note:set NODE_ENV=production && node server.js)

Expected behavior
The page should be available.
Screenshots

System information

Win 7 x64
Node 8.12
Npm 6.4.1

I'm assuming the reason is that it actually booted in dev mode because NODE_ENV=production was not set correctly or something 🤔

Good!It should be this set NODE_ENV=production&& node server.js.Remove space after production.


ci, swtpm: Pin to a supported version ubuntu 20.04

The CI is currently running with the latest ubuntu version (22.04) The latter does not support the use the swtpm package currently used. In order to allow the CI to work, temporarilly downgrading the ubuntu version to the former one 20.04.
This does not affect the actual CI testing as they are run on the RHCOS VM.

workaround to https://github.com/RHsyseng/rhcos-slb/issues/84

CI will not pass until PR is merged. merging.


Ryuk container don't destoy testcontainer with database

Module
PostgreSQL
Testcontainers version
1.17.2
Using the latest Testcontainers version?

I have a lot of text nodes, they are launched sequentially, but the launch order may be different. Each text node works within its own object lifecycle
@TestInstance(TestInstance.Lifecycle.PER_CLASS)
class ControllerTest extends WebTestBase {
...
}

I have configured the test container so that it is reused. At the first launch, it creates a database using a script. Then, in each test node, the database will be cleared after all the tests of one node are completed.
@ActiveProfiles("test")
@SpringBootTest(webEnvironment = SpringBootTest.WebEnvironment.RANDOM_PORT)
@AutoConfigureTestDatabase(replace = AutoConfigureTestDatabase.Replace.NONE)
@ContextConfiguration(initializers =   WebTestBase.DockerPostgresDataSourceInitializer.class)
public abstract class WebTestBase {

    static PostgreSQLContainer<?> postgresSQLContainer;

    static {
        postgresSQLContainer = new PostgreSQLContainer<>("postgres:14")
                .withUsername("web")
                .withPassword("web")
                .withInitScript("sql/init.sql")
                .withDatabaseName("name_table")
                .withReuse(true);

        postgresSQLContainer.start();
    }

    public static class DockerPostgresDataSourceInitializer
            implements ApplicationContextInitializer<ConfigurableApplicationContext> {

        String jdbcUrl = "spring.datasource.url=" + postgresSQLContainer.getJdbcUrl();
        String username = "spring.datasource.username=" + postgresSQLContainer.getUsername();
        String password = "spring.datasource.password=" + postgresSQLContainer.getPassword();

        @Override
        public void initialize(@NotNull ConfigurableApplicationContext applicationContext) {

            TestPropertySourceUtils
                    .addInlinedPropertiesToEnvironment(applicationContext, jdbcUrl, username, password);
        }
    }

    @Test
    void contextLoads() throws SQLException {

        ResultSet resultSet = performQuery(postgresSQLContainer);
        resultSet.next();

        int columnIndex = 1;

        int columnAmountExpected = 1;

        int result = resultSet.getInt(columnIndex);
        assertEquals(columnAmountExpected, result);

        Assertions.assertThat(postgresSQLContainer.isRunning()).isTrue();
    }

    private ResultSet performQuery(@SuppressWarnings("rawtypes") PostgreSQLContainer postgreSQLContainer)
            throws SQLException {

        String query = "SELECT 1";

        String jdbcUrl = postgreSQLContainer.getJdbcUrl();
        String username = postgreSQLContainer.getUsername();
        String password = postgreSQLContainer.getPassword();

        Connection conn = DriverManager.getConnection(jdbcUrl, username, password);
        return conn.createStatement().executeQuery(query);
    }

}

When I run the tests, all the text nodes work out correctly.  When the project is delivered to Github, then during assembly on the CI/CD pipeline, the tests also work and the project is deployed in Kubernetes.
But when I run the tests again on the local PC, I get an error that says it is not possible to create a table since it already exists. I executed the command
docker ps

in the console and saw that the ryuk container was deleted, but the test container with postgresql remained active in the processes. I have to manually delete it after each test run.
How can this be fixed ?
Relevant log output
Caused by: org.postgresql.util.PSQLException: ERROR: relation "name_table" already exists

Additional Information
No response

This works as expected when using withReuse(true) and having reusable mode enabled. Consider disabling reuse if you want containers to be cleaned up after tests.

so the point is to import the scheme according to which all test nodes work once and use the same container with the database. and after the tests are completed, delete it. This is how I understand the meaning of using the withReuse(true) method. Creating a testcontainer for each node is an expensive operation.

    static {
        postgresSQLContainer = new PostgreSQLContainer<>("postgres:14")
                .withUsername("web")
                .withPassword("web")
                .withInitScript("sql/init.sql")
                .withDatabaseName("table_name")
                .withReuse(false);

        postgresSQLContainer.start();
    }

.withReuse(false);  - really solved the problem, but then I don't understand its use in the value -> true.

The purpose of withReuse(true) is to decouple containers created by Testcontainers from the usual JVM process lifecycle. Per default, Ryuk will remove all remaining Testcontainers containers. However, when withReuse(true) is set and reusable mode enabled, containers will survive the end of the JVM process and can be reused by the next JVM process, i.e. test run.


Updating Version for PhyloPickMe

@JuliaRegistrator register

Registration pull request created: JuliaRegistries/General/115381
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v1.0.2 -m "<description of version>" ac387d47cb17c75dd66dbd3fc143a255a468a4e4
git push origin v1.0.2

Also, note the warning: Version 1.0.2 skips over 1.0.1
This can be safely ignored. However, if you want to fix this you can do so. Call register() again after making the fix. This will update the Pull request.

Thanks I corrected the version number.
@JuliaRegistrator register

Registration pull request created: JuliaRegistries/General/115424
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v1.0.1 -m "<description of version>" 4310b2bb83073857fa5f8f1d3c6ec0a0a1200b04
git push origin v1.0.1

I updated the release number to match.
@JuliaRegistrator register

Registration pull request updated: JuliaRegistries/General/115424
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v1.0.1 -m "<description of version>" 4310b2bb83073857fa5f8f1d3c6ec0a0a1200b04
git push origin v1.0.1

Registration pull request updated: JuliaRegistries/General/115424
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v1.0.1 -m "<description of version>" d3c3632710c135b0412c0c35d3869fd730115c85
git push origin v1.0.1

I eliminated the using Sparse Arrays and using CSV lines
@JuliaRegistrator register

Registration pull request updated: JuliaRegistries/General/115424
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v1.0.1 -m "<description of version>" cd35205bca2a35cb9d74df9cb4e51c3b7149e2a4
git push origin v1.0.1

I eliminated the remaining standard libraries from the dep [compat] sections
@JuliaRegistrator register

Error while trying to register: Version 1.0.1 already exists


UNR-3100 Added unit tests for LayeredLBStrategy

Description
Tests for the LayeredLBStrategy
Release note
Shoudn't need any?
Tests
This PR is entirely tests.

Overall looking really good! A few nits and a couple of structural things, but I'm super excited at this level of test coverage.

Closed as this PR was superseded by https://github.com/spatialos/UnrealGDK/pull/2218


JS error when hide running carousel

When carousel is running, and elements are hidden with v-if, the setTimeout code still continues to run and throws errors:
[Vue warn]: Error when evaluating expression "function slider() {
		      return this.$el.querySelectorAll('.item');
		    }": TypeError: Cannot read property '$el' of null
warn @ vue.common.js:1019
Watcher.get @ vue.common.js:3176
Watcher.evaluate @ vue.common.js:3356
computedGetter @ vue.common.js:8035
nextClick @ vue-strap.js:903
(anonymous function) @ vue.common.js:216

Respectively another error follows:
vue-strap.js:903 Uncaught TypeError: Cannot read property 'length' of undefined
nextClick @ vue-strap.js:903
(anonymous function) @ vue.common.js:216

I'm also receiving the above errors when I remove an array holding the data for the carousel (removing the carousels from the page).


Feature Suggestion

Feature_explanation.txt describes it.

I don't think a PR is the best way to propose a feature. Please post this in django-cms-developer group https://groups.google.com/forum/#!forum/django-cms-developers