“unmanaged” and “managed” don’t have an intuitive meaning

I always have to check out the documentation to remember what they mean. Also, the fact that the sourceDirectory actually points to the root of the unmanagedSourceDirectories is confusing.
What do you think of the following naming convention instead?
unmanagedSourceDirectories becomes sourceDirectories,
managedSourceDirectories becomes generatedSourceDirectories,
sourceDirectories becomes allSourceDirectories.

@julienrf Thanks for the report.
The term "managed" vs "unmanaged" also comes up in managedClasspath, where it means that dependency manage takes care of some of the JARs files as opposed to the ones that you manually supply in lib/ directory. In the classpath case, it feels a bit more natural because we normally let sbt deal with the JARs. But for sourceDirectories it's a more awkward since humans write the source code by default. Maybe this was a case of consistency where it shouldn't have been. I don't know.
Given the amount of build and plugins that exists in the world based on this unmanaged or managedSourceDirectories, however, I don't know what we can really do here. Renaming unmanagedSourceDirectories to sourceDirectories even in sbt 2.x I feel like would cause more problem than the naming problem it aims to solve.


Include the British English spellcheck extension

Not use if we want to decide on using British English or US spelling of words. There's an extension for including the British English spelling: https://marketplace.visualstudio.com/items?itemName=streetsidesoftware.code-spell-checker-british-english

In order to have both the UK and the US spelling the following settings should be implemented for CSpell.
"cSpell.language": "en,en-GB",
@lwjohnst86 can this be shared via the .Github repo, or should we set it individually?

This could definitely be put into the settings in .github!


Seller adds cover and it reflects to the shops page

Description

Added image upload to the seller form and modified the necessary form schema.
Now this image is reflected in the shops page for respective sellers.
For safety,
NEXT_PUBLIC_BASE_Cover="https://res.cloudinary.com/drz5akzu1/image/upload/v1729718405/mschk1pmmng6nqnlzl32.jpg"
the above link has been added as a default value in the db itself, so that the page doesn't break as it was before.
Please add a similar key value pair to your vercel deployment.

Related Issues

Closes #911

Screenshots / videos (if applicable)

https://github.com/user-attachments/assets/a29bfd33-5691-4202-97d4-7484bfb4f312
https://github.com/user-attachments/assets/89d7ada4-a817-4983-bc79-04283dc915e5

Alright @ShivanshPlays, I will add it to the Vercel deployment. Thank you!


Added MEDICAL_LISENCE test cases

I like the generation of medical licenses with a smaller list by mixing and matching. I also ran your tests and your tests pass (this is good since you're also checking the score so we know everything matches up).
However, you have some commented out code that isn't being used; if you commit and push a change to your branch with some edits it should be reflected here. However, I don't think it's strictly necessary for this assignment, especially since your tests function and such.
I approve of merging this 👍 Let's wait for the final approval from @claesmk and get this new functionality added 😎

Actually I see what's happening.  It matches the regex and then validates the result - once that happens the score increases to 1.0

@ChanningBarnes what are your thoughts on this?

Ran the tests on my machine as well and it looks good.


Build for iOS (aarch64-apple-ios) fails

Hi BLAKE3-team,
I cannot build the library for target aarch64-apple-ios.
I've added a CI/CD workflow to test this:
https://github.com/olliiiver/Blake_test/actions/runs/12350122968/job/34462322857
The error message is:
  = note: ld: warning: object file (/Users/runner/work/Blake_test/Blake_test/target/aarch64-apple-ios/debug/deps/libblake3-2e845a816eca43ce.rlib[6](a1edd97dd51cd48d-blake3_neon.o)) was built for newer 'iOS' version (17.5) than being linked (10.0)
          Undefined symbols for architecture arm64:
            "___chkstk_darwin", referenced from:
                _blake3_hash4_neon in libblake3-2e845a816eca43ce.rlib[6](a1edd97dd51cd48d-blake3_neon.o)
          ld: symbol(s) not found for architecture arm64
          clang: error: linker command failed with exit code 1 (use -v to see invocation)

I think the error lies somewhere here
https://github.com/BLAKE3-team/BLAKE3/blob/master/build.rs#L236
However, don't know how to fix it ...

The warning seems relevant:

note: ld: warning: object file [.../blake3_neon.o] was built for newer 'iOS' version (17.5) than being linked (10.0)

The symbol blake3_hash4_neon is indeed provided by that object file. So we are building it, correctly (because is_aarch64 is true here), but something about how we compiled it is making the linker unhappy. I don't have any experience with iOS, and I don't know how different versions of it differ from a compatibility perspective. Are there flags you need to pass to the C compiler when you compile things for iOS 10? We're just using the default compiler as determined by the cc crate (maybe the $CC env var?), and we don't currently have any iOS-specific logic.

As you can see on those same lines of build.rs, there are currently a couple different ways you can disable the NEON implementation in the blake3 crate. I'm curious whether enabling the no_neon crate feature unbreaks your build (at the cost of some runtime performance). Our Cargo.toml says that those features are considered unstable, but at this point they've been around long enough that it would be hard for us to remove them...

Hi @oconnor663  . Thanks for coming back to me. The build works by using the following workaround:
export CARGO_FEATURE_NO_NEON=1
cargo build --target aarch64-apple-ios

However, strange that the enabled functionality is later eliminated as dead code ... 🤔

The problem is that it's probably using the wrong linker, and you also need to set the deployment target for the C compiler, See here:
https://github.com/olliiiver/Blake_test/pull/2
https://github.com/elichai/Blake_test/actions/runs/12389872402/job/34583767864


Optimize PDE solving in Julia 0.4

I believe 0.4 has changed type-inference and this is affecting the speeds.

This seems to have fixed itself!  Probably adding Domain allowed type inference.


Type of arg 1 to keys must be hash (not hash element)

[root@vm1 graphite-collector]# ./eseries-metrics-collector.pl -n -c proxy-config.conf
Type of arg 1 to keys must be hash (not hash element) at ./eseries-metrics-collector.pl line 334, near "} ) "
Type of arg 1 to keys must be hash (not hash element) at ./eseries-metrics-collector.pl line 336, near "} ) "
Execution of ./eseries-metrics-collector.pl aborted due to compilation errors.
[root@vm1 graphite-collector]# perl -V
Summary of my perl5 (revision 5 version 24 subversion 0) configuration:
On another system with 5.16 no problem, but 5.24 has the problem.
I made some updates like this and it fixed it:
diff eseries-metrics-collector.pl eseries-metrics-collector-v2.pl
334c334
<         foreach my $vols ( keys $met_coll->{$system} ) {

    foreach my $vols ( keys %{$met_coll->{$system}} ) {

336c336
<             foreach my $mets ( keys $met_coll->{$system}->{$vols} ) {

        foreach my $mets ( keys %{$met_coll->{$system}->{$vols}} ) {

If you can fix in master please.  Thanks!

Thanks @dutchiechris
Applied via https://github.com/plz/E-Series-Graphite-Grafana/commit/52759f605e7958f9b7ed31dd599c148f15ceeb48


The user can create the Kafka destination connector with an invalid value "-1"

The user can create the Kafka destination connector with an invalid value "-1" for  the number of acknowledgments required before considering a record written to Kafka ( "acks": "-1" )

Expected result: Error: "message": "failed to create connector: could not init connector: invalid destination config: config is invalid: couldn't parse ack: unknown ack mode: required acks must be one of none, one, or all, not "-1""

Valid values: none, one, all.
GitHub - ConduitIO/conduit-connector-kafka: Provides a source and a destination Kafka connector, for Conduit.

@hariso need to check documentation because -1 is a valid value

We're using Segment client's acks.UnmarshalText method here, which is also able to handle integer values for this parameter. So it's valid to use:

none, one, all (text version)
0, 1, -1 (int version).

We need to update the docs to mention this possibility too.


drive table possession times

Can you please help with the drive table? I have multiple records that overlap Q1/Q2 or Q3/Q4, so although possession times per half add up properly, the quarters don't.
Is this an inconsistency within the gamecenter JSON? A load error?
I am trying to compute percent possession time per quarter, and it's a bit dodgy right now.

Could you help me to understand your problem? The best way to help me understand is to provide some small amount of code that doesn't do what you expect. What have you tried? What is it doing? What do you think it should be doing?
Note that Drive objects have a pos_time field, which has type PossessionTime. A PossessionTime has a total_seconds field which should be useful for adding times up.

Drive can span quarters. I wanted to calculate a possession time percent by
quarter, so I went down to the play level. This is part of the gamecenter
data structure, but it's something that should be documented.
On Wed, Aug 19, 2015 at 5:25 AM, Andrew Gallant<EMAIL_ADDRESS>wrote:

Could you help me to understand your problem? The best way to help me
understand is to provide some small amount of code that doesn't do what you
expect. What have you tried? What is it doing? What do you think it should
be doing?
Note that Drive http://pdoc.burntsushi.net/nfldb#nfldb.Drive objects
have a pos_time field, which has type PossessionTime
http://pdoc.burntsushi.net/nfldb#nfldb.PossessionTime. A PossessionTime
has a total_seconds field which should be useful for adding times up.
—
Reply to this email directly or view it on GitHub
https://github.com/BurntSushi/nfldb/issues/94#issuecomment-132507388.

Well, it's not just "part of the GameCenter data structure." The possession time of a drive, by definition, can span quarters. I don't really see any reason to expect otherwise.
Using play data is probably the right heuristic for possession time by quarter.


[1.10.2] Server Crash

Hi,
i placed down a DR smelter with two tanks and a purifier with a tank. Afterwards i went to my armoma mining world and had a crash, my server where im running it on crashed.

Description: Exception ticking world entities
java.lang.NullPointerException: Exception ticking world entities
at java.util.Objects.requireNonNull(Objects.java:203)
at java.util.Optional.(Optional.java:96)
at java.util.Optional.of(Optional.java:108)
at mcjty.deepresonance.blocks.tank.TileTank.onTileUnloaded(TileTank.java:170)
at mcjty.deepresonance.blocks.tank.TileTank.onChunkUnload(TileTank.java:158)
at net.minecraft.world.World.func_72939_s(World.java:1843)
at net.minecraft.world.WorldServer.func_72939_s(WorldServer.java:620)
at net.minecraft.server.MinecraftServer.func_71190_q(MinecraftServer.java:709)
at net.minecraft.server.dedicated.DedicatedServer.func_71190_q(DedicatedServer.java:387)
at net.minecraft.server.MinecraftServer.func_71217_p(MinecraftServer.java:613)
at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:471)
at java.lang.Thread.run(Thread.java:745)

Full crash log:
crash-2017-05-18_12.24.55-server.txt
Client didn't crash only Server.
Using version:
deepresonance-1.1.x-1.4.7
mcjtylib-1.1.x-2.3.11
Hope the log gives enough informations.
Greetz,
Camrais

When will people stop reporting bugs for a version that is outdated...

Sorry for being annoyed but this is already almost the tenth ticket of this sort even though the fixed version has been out there for some time already

Well i wrote it when i saw that the post from december is still open... Sry for bothering.


Fixed #24561 -- Added support for callables on model fields' choices.

This is a follow up of PR #16943 (ticket-31262). Most of the ground work was done in that PR, this branch removes the system check and adds some tests to ensure the feature works as expected.

Thanks @nessita. Interesting... So I can see that there are a number of potential reasons why you may want to use a callable:

Performing I/O to gather the choices (which could potentially be cached), e.g.

a query of a table in the same database (which is used for associated metadata for the choice values)
a query to a different external database (maybe for integration purposes)
accessing the choices from a static file

Avoiding migration churn for choices that can be added to periodically

e,g. using the available timezones in tzdata (or previously pytz)

Fully dynamic choices as per the example you provided above

If we cannot solve the case for the fully dynamic case, I think it's still worth adding this support for callables for some of the other use case and we can slap in a chunky warning that more dynamic cases are not supported, i.e. you can return additional choices, but not remove them otherwise some things such as the admin will not behave.
Otherwise I don't know whether we can make a form field show the existing value plus the available values somehow and allow the existing value to validate. It starts to get a bit tricky and would require application specific logic. Also there would be challenges around how to display the value as the display label value would be unknown...

I can see that there are a number of potential reasons why you may want to use a callable:

I agree on the uses cases you shared and I can add my own: I have a pet project for recording household expenses, and each expense can have one tag. The tag list is a setting, since that list may vary from family to family, and I explicitly avoided made it an M2M because I don't want to have an ever growing table for tags that can be considered static within one deploy.
So right now my Expense model has an CharField to store the tag, and the choices point to the setting. I had to manually modify the migration introducing this to avoid a new migration every time the setting was changed. If a callable would be allowed, that would have solved my problem.

@ngnpope I have re-worked the docs using your feedback, ideally we can merge this soon so it makes the cut for the feature freeze (which is scheduled for next Monday!).
Let me know your thoughts, thanks!

@felixxm I think you wanted to evaluate if we could use a better/different example for callable choices. I have proposed a different one, no longer using Fibonacci but something more close to a real life case.
I will land following Nick's approval, but if you want me to change the docs, I'm happy to do that in a follow up branch. Thanks!


Remove check for container when generating help message.

Some positional parameters would be listed in the help text and others
would not, when what is desired is that no positional parameters are
listed with the other command options. This change suppresses the help
listing for all positional parameters.
For the following example,
  m_cmd_options->add_options()
    ( "h,help", "Display usage" );

  m_cmd_options->add_options("pipe")
    ( "c,config", "File name containing supplemental configuration entries. Can occur multiple times.",
      cxxopts::value<std::vector<std::string>>() );

    // positional parameters
  m_cmd_options->add_options()
    ( "p,pipe-file", "Input file", cxxopts::value<std::string>())
    ( "extra", "Extra command line args",  cxxopts::value<std::vector<std::string>>());

  m_cmd_options->parse_positional({"pipe-file", "extra"});

The group help for pipe-file would be displayed and the help for "extra" would not. This is because the "extra" command option referred to a container. I propose to remove the test for a container from the code that generates the help text. This will skip all positional parameters without regard for being a container.

This change is

This seems reasonable.


TypeError: undefined is not a function, js engine: hermes

I have error TypeError: undefined is not a function, js engine: hermes. the error pops up when i start writing.
I am using

RN 0.69
React 18
and Hermes Enabled

@pierroo yes, I was unintentionally using onChangeInput to get value, which resulted in the above error, which I was able to resolve by using onChange


Remove unused test service

Why
In a docker development environment, the test are run using the app
service - IE the use the same "image" as the running application - via
bin/dev/test. This service is no longer needed.
Resolves #739
Pre-Merge Checklist
All these boxes should be checked off before any pull request is merged!

[ ] All new features have been described in the pull request
[ ] Security & accesibility have been considered
[ ] All outstanding questions and concerns have been resolved
[ ] Any next steps that seem like a good ideas have been created as issues for future discussion & implementation
[ ] High quality tests have been added, or an explanation has been given why the features cannot be tested
[ ] New features have been documented, and the code is understandable and well commented
[ ] Entry added to CHANGELOG.md if appropriate

How
bin/dev/test reuses the app service's image to spin up a container in which the tests are run. For rspec, the spec_helper already specifies that the application will run in the test environment, rather than the default.
Testing
bin/dev/test should still run the test suite
Next Steps
It might be nice to provide an interface to allow the developer to run a specific, or subset, of examples/specs in the test suite. I know this fuctionality exists via RSpec, but the bin/dev/test script currently assumes that you want to run the Rails and JavaScript test suites - IE test the whole application.

My pleasure, friend!


[Examples Browser]: Video texture doesn't play on Chrome Android

http://playcanvas.github.io/#/graphics/video-texture does not play video on Chrome on Android
No errors in console.

I think there is a typo in adding the video element. That should be a node, rather than string, probably:

How odd that it works on desktop :thi

Had a quick look and it does fix up the DOM but video texture still doesn't play.
The MP4 doesn't show up in the network tab either in devtools which is odd

Ah, it looks like the video is off the end of the page/browser view so the mobile browser doesn't start loading the video


web performance

NOTE: All 1st-time contributors should know Topic approval and PR submission does NOT guarantee your Topic/Article will be published. Our team of Peer Reviewers and Content Moderators will review all PRs that come in to make sure they adhere to the standard of quality we expect from the EngEd community.
We expect all community memebers to go through provided examples, resources, and previously published material before submitting content. As a rule of thumb, please only submit articles (pull requests) that are complete, formatted correctly, and include a fully polished article (ready to be published and error free).
All revisions and edits should be completed on your own forked repo (as to not take up room in the queue). Any PR that is submitted incomplete (meaning not ready to be published as is) will be closed. A PR being closed does NOT mean the article can not be published (or fixed) it simply means that the edits, revisions, and fixes will happen outside of the queue.
NOTE: Any topic suggestion copied/taken/modified from YouTube may Lead to a permanent ban from the community.
Topic Suggestion
Enter topic suggestion here...
Pre-submission advice
By following all our pre-submission advice and reviewing our Resources folder, you will maximise your chances of your topic being approved.
We ask that you please be patient as our team works through approving and publishing all articles/tutorials in a timely manner.
Allow 1-3 days for a topic to be reviewed and/or approved - allow 3-7 days for an articles to be reviewed and/or published (subject to vary depending on volume and/or backlog of articles).
Be sure to visit our Resources Page for tools, resources, and example articles that will help you propose and write a successful article.
Please ensure that you have only one open issue + linked pull request at a time. This will ensure that we complete the article in a timely manner from inception to publishing.)
We tend to stray away or tend not to publish reviews/comparisons of commercial product offerings.
Writing sample(s):
Include any links or writing samples - to help our team better gauge your writing quality.
Proposal Submission
Proposed title of article
Your title should be descriptive of the article/tutorial. Be Specific.
Use keyword research to improve your article's SEO.
Proposed article introduction
We're looking for the first 2-3 paragraphs of your article that appropriately summarize what your article will be about.
We will use this writing snippet to help us assess the overall quality before approval so ensure it has been proofread, spellchecked and grammarchecked.
For example, if the article is based on machine learning, use the following: [Machine learning] Introduction to Machine Learning.
If the article is based on developing an Android application, use the following: [Android] Developing Apps using Android.
ONLY viable topics to pick from:

API
Artificial Intelligence
Blockchain
Containers
Edge Computing
Languages
Machine Learning
Networking
Node.js
Security

Key takeaways
What are the 3-5 most important things the reader should understand or be able to do after reading this article?
Use this area to get your ideas down on the bulk of your article or tutorial.
Article quality
How is your article unique? Tell us what makes your approach different from similar articles that have published on the same topic?
Is yours more in-depth? Does it cover additional topics? Do you provide handy tips or anecdotal advice?
References
Please list links to any published content/research that you intend to use to support/guide this article.
Conclusion
Finally, remove the Pre-Submission advice section and all our blockquoted notes as you fill in the form before you submit.
We look forwarding to reviewing your topic suggestion.
Templates to use as guides

How To Guide Template
Software Review Template
Tutorial Template

         STRUCTURING THE WEB WITH [HTML]

INTRODUCTION
To build websites, you should know about HTML — the fundamental technology used to define the structure of a webpage. HTML is used to specify whether your web content should be recognized as a paragraph, list, heading, link, image, multimedia player, form, or one of many other available elements or even a new element that you define.
HTML stands for Hyper Text Markup Language. It is used to design web pages using a markup language. HTML is the combination of Hypertext and Markup language. Hypertext defines the link between the web pages. A markup language is used to define the text document within tag which defines the structure of web pages. This language is used to annotate (make notes for the computer) text so that a machine can understand it and manipulate text accordingly. Most markup languages (e.g. HTML) are human-readable. The language uses tags to define what manipulation has to be done on the text.
OBJECTIVE
To gain basic familiarity of what is involved with web performance.
KEY TAKEAWAYS
Fundamentals of Web Design
Basic overview of Web Industry
HTML Tags and Attributes
Differences between HTML5 and HTML
CONCLUSIONS
I want to come out with all possible ways to manage the web using the [HTML]

Thank you for your response and we thank you for submitting your topic. After some careful consideration it struck us that this topic may be a bit over saturated throughout other blog sites and official documentations as previously mentioned on your topic form by our content moderator.
We typically refrain from publishing content that is covered widely on the net or other blogs. As we're more interested in original, practitioner-focused content that takes a deeper dive into programming-centric concepts.
We believe this is the best way for students to build a great portfolio (for potential employers) is by building what does not exist and what can provide the most value.
You are more than welcome to pursue another more in depth topic.


Remove extra ';' after member function definition.

This fixes a clang compiler warning about extra semicolons that blocks ANGLE's integration.

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

I have filed #2781 about getting the right flags enabled to prevent this in the future. I will implement it after I get back from the long weekend.


Color Function

The color bar at the bottom doesn't seem to update the color of the compiled fractal.

Hello Dan, to avoid having many redraws that cause the app to slow down as the user pulls the slider, you must use the "update" button in the top portion of the screen. So, to change the color: drag the slider to the desired color, then the "update" button will illuminate yellow, and a click on that will update the color of the fractal.

Thanks, Cody! It wasn’t working earlier today, but it is now, so it may have been a reload issue!
==================
Daniel M Look
Rutherford Professor of Mathematics
Department of Mathematics, Computer Science, and Statistics
St. Lawrence University
https://sites.google.com/view/dmlook/home
Pronouns: he, him, his
From: Dakota Bryan @.>
Reply-To: FractalVoyager/FractalVoyager @.>
Date: Monday, August 28, 2023 at 11:21
To: FractalVoyager/FractalVoyager @.>
Cc: Daniel Look @.>, Author @.***>
Subject: Re: [FractalVoyager/FractalVoyager] Color Function (Issue #3)
CAUTION: This email message originated from outside of St. Lawrence University. Please do not click links, follow instructions or open attachments unless you recognize the sender and know the content is safe.
Hello Dan, to avoid having many redraws that cause the app to slow down as the user pulls the slider, you must use the "update" button in the top portion of the screen. So, to change the color: drag the slider to the desired color, then the "update" button will illuminate yellow, and a click on that will update the color of the fractal.
—
Reply to this email directly, view it on GitHubhttps://github.com/FractalVoyager/FractalVoyager/issues/3#issuecomment-1695889459, or unsubscribehttps://github.com/notifications/unsubscribe-auth/APFWZO3LYAGBM3K4GOELN5DXXSZOVANCNFSM6AAAAAA4BSCEOY.
You are receiving this because you authored the thread.Message ID: @.***>

No problem, yes it's pretty buggy at the moment. Thanks for submitting an issue. I have a list of bugs that I'm working through fixing on the README file; if you find any others, please open more issues!


Support IBM i/PASE systems

This PR is for the support of IBM i/PASE env. The code changes are all covered by IBM i /PASE macros to avoid it affecting any other platforms. Please note, we also upgraded the fmt and fast_double_parser external packages to make it works on our platform. Please help to review it before merge. thanks.

Thanks very much for your proposal.
Before we continue, please provide answers for the following.

How would this patch improve LightGBM?
What is "IBM i/PASE" and why is supporting it important for LightGBM?
Do you work for IBM?
How can we get access to an IBM i/PASE system to test these changes?
Are you or some group you work with (company, research institution, etc.) committed to supporting these changes, e.g. responding in a timely manner if we receive bug reports about this support and @ you?

Links to documentation would be greatly appreciated.

we also upgraded the fmt and fast_double_parser external packages

How did you do that, exactly? I see https://github.com/lemire/fast_double_parser/pull/70 but don't see a related change at https://github.com/fmtlib/fmt.

In the future, I strongly recommend not using master on your fork when making pull requests to open source projects. That makes it very difficult to keep up with upstream changes during the period where your PR is open, and makes it difficult to work on multiple pull requests at the same time.

@microsoft-github-policy-service agree [company="{your company}"]

@microsoft-github-policy-service agree [company="IBM"]

How would this patch improve LightGBM?

What is "IBM i/PASE" and why is supporting it important for LightGBM?

Do you work for IBM?

How can we get access to an IBM i/PASE system to test these changes?

Are you or some group you work with (company, research institution, etc.) committed to supporting these changes, e.g. responding in a timely manner if we receive bug reports about this support and @ you?

Thanks for your comments @jameslamb . I am so sorry for the long time wait , per my first round of Covid-19. This may make it more difficult for us to keep track of the latest changes in *master branch.  Which branch would you suggest for this kind of PR? I may create a new one if you like.
Let me try to answer the questions you raised:
How would this patch improve LightGBM?
A: This patch is for IBM i platform. With the support of this platform, LightGBM could extend the scope of customers. This would give more profit and opportunity to improve its quality and performance.
What is "IBM i/PASE" and why is supporting it important for LightGBM?
A : IBM i/PASE platform is one of the IBM platform, we have millions of customers in financial over the world. With the support of this platform , LightGBM could affect more customers and business.
Do you work for IBM?
A: Yes.
How can we get access to an IBM i/PASE system to test these changes?
A: we would do the test on our system, and we would also make sure the changes would not break or affect lightGBM on any existing platforms.
Are you or some group you work with (company, research institution, etc.) committed to supporting these changes, e.g. responding in a timely manner if we receive bug reports about this support and @ you?
A: For us, the ML/AI on our platform is also in the very beginning phase. We do not have any pressure to commit this kind of changes , although we wish to make it available ASAP, which would encourage our customers to use them.

@jameslamb as for this quesiton:
How did you do that, exactly? I see https://github.com/lemire/fast_double_parser/pull/70 but don't see a related change at https://github.com/fmtlib/fmt.

Yes, we did changes in fast_double_parse project to make it fit for our platform. For the fmt , we really have similar issue on current level of fmt integrated in LightGBM. But with the latest version of fmt(as I updated by this commit), the related code was changed, the original issue disappear, so, that's the reason why we do not have similar PR in fmt project. But just do an version upgrade .

It's been 2+ months without a response from the author, so I'm going to close this.


Something went wrong installing the "sharp" module with serverless offline and watch

When I run serverless offline and change something on a lambda function the next time the app tries to access an endpoint that uses sharp it throws this error:
 Something went wrong installing the "sharp" module

  Module did not self-register: '/Users/stathis/WebstormProjects/xxxx/.esbuild/.build/node_modules/sharp/build/Release/sharp-darwin-x64.node'.

  Possible solutions:
  - Install with verbose logging and look for errors: "npm install --ignore-scripts=false --foreground-scripts --verbose sharp"
  - Install for the current darwin-x64 runtime: "npm install --platform=darwin --arch=x64 sharp"
  - Consult the installation documentation: https://sharp.pixelplumbing.com/install

To Reproduce

run serverless offline
Change something inside a function
Let serverless-esbuild refresh/update
Try to access the lambda that usessharp
Throws the error
restart serverless offline
Try to access the lambda that usessharp
works as it should as long as I do not touch anything that could trigger an esbuild refresh

Please note that if I do not touch the lambda functions then it will never throw an error.
I've monitored the .esbuild/.build/node_modules/sharp/build/Release folder and the sharp-darwin-x64.node is always there between code refreshes.
esbuild config on serverless.yaml:
  esbuild:
    bundle: true
    external:
      - sharp
    minify: true
    sourcemap: true
    packager: yarn
    packagerOptions:
      scripts:
        - SHARP_IGNORE_GLOBAL_LIBVIPS=1 npm_config_arch=x64 npm_config_platform=darwin yarn add sharp
    target: node16
    define: {}
    platform: node
    concurrency: 1
    watch:
      pattern: ["src/**/*.ts"] # match only typescript files in src directory
      ignore: ['.serverless/**/*', '.build', 'node_modules']

If there's more information needed please let me know.
Expected behavior
I should not have to restart the serverless offline command everytime
Versions (please complete the following information):

OS: MacOS Monterey 12.5.1
Serverless Framework Version: 3.22.0
Plugin Version: 1.32.8
sharp version: 0.31.0
node version (nvm): 16.17.0

I have the same/similar issue. It works in lambda environment but not with serverless offline.
and when I check the node_modules/sharp/build/Release/sharp-darwin-arm64v8.node is there
but the issue is that it is not there in .esbuild/.build/node_modules/sharp/build/Release/ folder.
There I only find sharp-linux-x64.node which is the one specified to build in
serverless.yml
  esbuild:
    packagerOptions:
      scripts:
        - rm -rf node_modules/sharp
        - SHARP_IGNORE_GLOBAL_LIBVIPS=1 npm install --arch=x64 --platform=linux --libc=glibc sharp

however I want this setting packagerOptions to only apply when running sls deploy and be ignored when running sls offline.
I think this is the issue. How can this be accomplished?

having the same issue here. there is no issue calling a function. but once i edit some code in lambda and try to call the same function, i get this error :
`  Something went wrong installing the "sharp" module
Module did not self-register: '//.esbuild/.build/node_modules/sharp/build/Release/sharp-darwin-x64.node'.
Possible solutions:

Install with verbose logging and look for errors: "npm install --ignore-scripts=false --foreground-scripts --verbose sharp"
Install for the current darwin-x64 runtime: "npm install --platform=darwin --arch=x64 sharp"
Consult the installation documentation: https://sharp.pixelplumbing.com/install`

I had the same issue with serverless-plugin-typescript and I fixed it by following serverless-offline docs: https://github.com/dherault/serverless-offline#usage-with-other-plugins
plugins:
  - serverless-plugin-typescript
  - serverless-offline # MUST be in last position

Serverless offline must be started with the start argument
serverless offline start


refactor(graphql): strawberry to input

[ ] make sure the docs script is run out of the box https://docarray.jina.ai/advanced/graphql-support/

I don't think there is a way to create a strawberry type from a strawberry object or vice versa even if object seems to be a type under the hood but instantiate differently. There is no method to do it proprely and and did not find a hack by looking at the source code. So I propose to keep one source of truth and create both type and object from it

Why do we need both? is that clear @samsja ?

Why do we need both? is that clear @samsja ?

I am not sure to understand 100% all of the concept but what I understand is that input is more restrictive than  type.
On the core part we can't use the type we need input and on the docarray part they seems to need the type.
So I need to check with docarray team if they really need the strawberry.type but I guess so.
@alaeddine-13 maybe ?

Why do we need both? is that clear @samsja ?

If my understanding is correct, one is what you put into a query, and one is what you get out. So even if, on a technical level, we could get away with using just one for both purposes, we probably shouldn't, because they are conceptually quite different.


Update Edit description in project permission dropdown

Closes #1241
Description

It's a one-liner fix, just the text update.

How Has This Been Tested?

Login to the dashboard as an admin user
Go to any of the projects
Go to the Permissions tab
Click on Add user and open the dropdown to select the permission, check if the description of the Edit selection is updated

Test Impact

Text update, N/A.
Request review criteria:

[x] The commits have meaningful messages (squashes happen on merge by the bot).
[x] Included any necessary screenshots or gifs if it was a UI change.
[x] Included tags to the UX team if it was a UI/UX change.
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has manually tested the changes and verified that the changes work
[x] The developer has added tests or explained why testing cannot be added (unit tests & storybook for related changes)

/assign @kywalker-rh

/assign @andrewballantyne


Ignore vscode files

to: @ryandeivert
cc: @airbnb/streamalert-maintainers
size: small
Background
Add VSCode to Git Ignore

Coverage remained the same at 97.617% when pulling d016a8786d31c6fd97d2c73f796cb8d1886261fe on vscode-git-ignore into 9ae8703869184924afcaafc0374bdc9d97d88943 on master.


[Fix] check for the CUDA version installed on image

Up to now we used nvidia-smi for the CUDA version check, but because of the [1] we should rather use the nvcc tool for such operation.
Also, correct version of CUDA is 12.1 in our images now, see [2].
This is a followup of #1413 [3].
[1] https://issues.redhat.com/browse/RHOAIENG-6673
[2] https://github.com/red-hat-data-services/notebooks/blob/rhoai-2.9/cuda/ubi9-python-3.9/Dockerfile#L29
[3] aae86d286bc8ed7fd8cfb9c0c468b86cf6654764

CI: good - failed test is unrelated issue to this PR

Thank you, Jan!
LGTM


Start of restructuring with blueprints and modules

We've been talking about using blueprints for ages (started in #14), so this is a start in that direction. This moves the room hassle section of the site into a new module. Once we do this with the entire site, it should make everything easier to work with.
I hoping to get comments on this organization structure, and if it's good then I'll continue with the rest of the site. The general structure for all blueprints would be the same as this, so let me know if anything seems complicated/confusing/non-pythonic.

I like it! Seems like everything is working fine, so it should basically be ready to merge.
One thing, though is the naming of the helper function files - I think it would be better to name library files for each module as hasslelib.py, userlib.py, adminlib.py, etc. That way, if multiple modules need to use library functions, there wouldn't be conflicts with multiple modules being named helpers?
I suppose we could always put those kinds of library files in the RuddockWebsite package and then make sure that everything in the helper modules is only needed by that module.

I like it! Seems like everything is working fine, so it should basically be ready to merge.
One thing, though is the naming of the helper function files - I think it would be better to name library files for each module as hasslelib.py, userlib.py, adminlib.py, etc. That way, if multiple modules need to use library functions, there wouldn't be conflicts with multiple modules being named helpers?
I suppose we could always put those kinds of library files in the RuddockWebsite package and then make sure that everything in the helper modules is only needed by that module.

Awesome, thanks for the quick review! I'll merge it now, then, so we can fix the merge conflicts in the other PRs.

I suppose we could always put those kinds of library files in the RuddockWebsite package and then make sure that everything in the helper modules is only needed by that module.

Yeah, I think that's the best way to organize it, and was one of the reasons I named it helpers.py. I think it's best if each blueprint module is isolated from one another. The codebase could quickly become confusing/unmaintainable if we have one blueprint relying on functions from another. So I'm hoping that any library files that are needed across blueprints are put in the RuddockWebsite package, and helpers are only needed within its module. This is why I created a common_helpers.py in the main package, since we had a few functions that were needed across blueprints.

Awesome, thanks for the quick review! I'll merge it now, then, so we can fix the merge conflicts in the other PRs.

I suppose we could always put those kinds of library files in the RuddockWebsite package and then make sure that everything in the helper modules is only needed by that module.

Yeah, I think that's the best way to organize it, and was one of the reasons I named it helpers.py. I think it's best if each blueprint module is isolated from one another. The codebase could quickly become confusing/unmaintainable if we have one blueprint relying on functions from another. So I'm hoping that any library files that are needed across blueprints are put in the RuddockWebsite package, and helpers are only needed within its module. This is why I created a common_helpers.py in the main package, since we had a few functions that were needed across blueprints.

This is now live.
When pulling from master, you'll need to re-init and re-load the submodule (since I moved it).

This is now live.
When pulling from master, you'll need to re-init and re-load the submodule (since I moved it).


Add more explanation about CCSync and its components

Document more about the environment variables

So in this case, what is the difference between origin from the flutter app and CONTAINER_ORIGIN in this container environment?

the origin in the flutter app is a placeholder (atleast for now) so it can be anything but null as we need a parameter.
The CONTAINER_ORIGIN refers to the taskchampion sync server containers' url that you're using to sync across all your devices.

I've added comments about the variables in the index.md in the latest commit.

Title: Getting Errors while setup
Hey @its-me-abhishek ,
I have done setup of ccsync and sync-server.
Also I have done with the installation of taskwarrior
following containers are running
[root@ShGI ccsync]# docker-compose up
[+] Running 3/0
 ✔ Container ccsync-syncserver-1  Running                                                                                0.0s
 ✔ Container ccsync-backend-1     Running                                                                                0.0s
 ✔ Container ccsync-frontend-1    Running                                                                                0.0s
Attaching to backend-1, frontend-1, syncserver-1

but when I add task or sync tasks its returning internal server error
I have also done with the setup of taskwarrior as
[root@ShGI ccsync]# task --version
3.2.0
[root@ShGI ccsync]# task config sync.encryption_secret ZZZZaee9076288e08ea13e3508580a99c1bdc120f8bae8fd5c0afb22ee1bXXXX
Are you sure you want to add 'sync.encryption_secret' with a value of 'ZZZZaee9076288e08ea13e3508580a99c1bdc120f8bae8fd5c0afb22ee1bXXXX'? (yes/no) yes
Config file /root/.taskrc modified.
[root@ShGI ccsync]# task config sync.server.origin http://localhost:8080/
Are you sure you want to add 'sync.server.origin' with a value of 'http://localhost:8080/'? (yes/no) yes
Config file /root/.taskrc modified.
[root@ShGI ccsync]# task config sync.server.client_id d542def4-XXXX-34a4-bd44-27efd5b71XXX
Are you sure you want to add 'sync.server.client_id' with a value of 'd542def4-XXXX-34a4-bd44-27efd5b71XXX'? (yes/no) yes
Config file /root/.taskrc modified.
[root@ShGI ccsync]# task sync init
sync.server.origin is deprecated. Use sync.server.url instead.
Syncing with sync server at http://localhost:8080/

getting following errors in react frontend

I dont know if this is correct place to ask for help as there is no slack channel for ccsync.

sync.server.origin is deprecated. Use sync.server.url instead.

have u tried working on this? as I am unaware if this has recently got deprecated or not...
might be a simple fix if you could work it out on the Docker shell

I have tried both sync.server.origin and also sync.server.url but still Internal Server Error

Sorry, I didn't tried on docker shell instead i was trying on my WSL.
Confused by this :

Sorry about that

for these common issues, I think that the official Slack's #taskwarrior-flutter channel should suffice, until a new channel is created dedicated to CCSync


Drop CentOS 7 from workflows.

This PR drops CentOS 7. See https://github.com/rapidsai/build-planning/issues/23.

@jameslamb @ajschmidt8 @vyasr Feel free to weigh in on this set of matrix changes. I replaced centos7 with rockylinux8 and moved a few other pieces around.

@AyodeAwe Good question. We don't have a chart, but this is the process we used last time we reconfigured the CI matrix: https://github.com/rapidsai/build-planning/issues/5#issuecomment-1973775147

Sorry I wasn't able to get to this yesterday. Changes look good to me 👍🏻


Ensure pip download dir is uncontended.

When pip encounters an existing file during any of its file operations
it prompts for an action if --exists-action has not been set. Pex was
supplying a download directory to pip that was degenerate for
interpreters with the same abi and same major and minor versions (e.g.:
2.7.6 and 2.7.18). We ensure the download directory is unique per
interpreter and further configure the --exists-action to abort to get
fail fast behavior and smoke out regressions or any remaining issues in
this vein.
Fixes #983

It was very unsatisfying but I could not come up with a repro for the original problem. The backtrace + logic + debugging confirmation done in CI re the presence of degenerate interpreters indicates this is the fix though


Added a maxResponseSize option that limits the maximum buffered response size

Added maxResponseSize option, checks for invalid values, backward compatibility and relevant tests plus fix to the test-timeout.js so all the tests pass
PR Checklist:

[Y] I have run npm test locally and all tests are passing.

[Y] I have added/updated tests for any new behavior.

[Y] Relevant documentation

[Y] Full backward compatibility is ensured

PR Description
When using a callback, the response is being buffered either in Buffer or string array.
The size of this buffer however is not regulated and there is no way to explicitly limit it.
The same also applied when the response is compressed, after decompression it can become much bigger leading to out-of-memory problems.
This feature adds an optional limit(maxResponseSize) that can protect you from such issues.
Please see the updated Doc for more information.

I'm happy I stumbled across this, was pretty helpful to my use-case. Thanks!
Hoping it gets merged in - it's a great option to have. Prior to this, I found that reading the length of the Buffer after-the-fact often led to some "Out of memory" heap issues if the buffer was too large. This seems to calculate it by the chunk though, which solves that problem.

Thank you @mivanovaxway, I would be very happy to see it merged!


[terra-paginator] Styling update to support anchor attribute gap in F…

…usion

Summary

What was changed:
A new optional boolean prop(false by default) is added to support the anchor style. When the new prop is set(true), the paginator will be rendered in a gray background color with a borderline at the top.
Why it was changed:
Currently the paginator renders without any borderline and the background color is white. However, MPage PagiantorFooter(utilizing Progressive Paginator) has an "anchor" style, which has a top borderline and the background color is gray. We will want to add a new optional boolean prop(false by default) to support the anchor style from the Terra perspective.
Testing

The default progressive paginator looks like(no top border line and no background color):

The progressive paginator with Fusion Anchor style(top border line and gray background color):

This PR does not contain Accessibility changes.
This change was tested using:

[x] WDIO
[x] Jest
[x] Visual testing (please attach a screenshot or recording)
[ ] Other (please describe below)
[ ] No tests are needed

Reviews
In addition to engineering reviews, this PR needs:

[x] UX review
[ ] Accessibility review
[ ] Functional review

Additional Details

This PR resolves:
UXPLATFORM-9527

Thank you for contributing to Terra.
@cerner/terra

@eawww @sdadn @sycombs This PR is ready for review. Thanks!

Synced with Scott and Terra folks, we believe that the best practice would be that Fusion takes care of its own customized style. Adding a prop for a specific styling usage is very confusing for consumers. Closing this PR out. Will comment on the UXPLATFORM-9527 soon.


Feature/sm modis dynveg4 merra3

PURPOSE: Add an option to use pre-processed inter-annually varying MODIS MCD15A2H.006 LAI and MCD43GF.006 VISDF and NIRDF Albedo for the catchment model (only)
SOURCE CODE CHANGES:

GEOS_surfaceGridComp.F90
GEOS_catchGridComp.F90
GEOSland_GridComp/Shared/CMakeLists.txt

NEW FILES:

GEOSland_GridComp/Shared/process_modis.py
(This script created the 2 GCM input ExtData files. The script will be run by the MERRA pre-processor to update the ExtData files in parallel to MODIS data production)
GEOSland_GridComp/Shared/GEOS_LandGridComp_ExtData.rc
GEOSland_GridComp/Shared/GEOS_LandGridComp.rc

Per email conversation with @smahanam, Rolf Reichle, and Randy Koster, let's wait for analysis of longer-term runs before merging.

As mentioned in the issue https://github.com/GEOS-ESM/GEOSgcm_App/issues/30 , GEOS_LandGridComp.rc  should be in it's own pull request to go along with  https://github.com/GEOS-ESM/GEOSgcm_App/pull/39

@JustinPerket per gatekeeper policy mentioned at https://github.com/GEOS-ESM/GEOSgcm_GridComp/wiki/Pull-requests has 1) @wmputman approved this PR going into develop and has this 2) passed the zero-diff tests?

@JustinPerket per gatekeeper policy mentioned at https://github.com/GEOS-ESM/GEOSgcm_GridComp/wiki/Pull-requests has 1) @wmputman approved this PR going into develop and has this 2) passed the zero-diff tests?

@sdrabenh discussion is still pending. Regardless, this pull request is for feature/ldas-surface, not develop

I see. But has it been verified as zero-diff for the default configuration?

I see. But has it been verified as zero-diff for the default configuration?

Testing has not been done yet. Awaiting review with Rolf and Randy (no git names for them yet)

Okay, awaiting zero-diff status. Please label it appropriately "0 diff" if it passes the 4 mentioned tests.


Fixes #25 Fix docs script

Fixes #25
Create a new script that handles all things docs related:

Used node's rmdirSync instead of rm -rf in order to have better cross-platform support
Move the generated array from guides.Interfaces to api.Interfaces
Delete api.Widgets array

Thanks for this 🚀

@all-contributors please add @michaeltintiuc for Infrastructure


chore(deps): update github.com/savetherbtz/zstd-seekable-format-go/pkg digest to d6cafef

This PR contains the following updates:

Package
Type
Update
Change

github.com/SaveTheRbtz/zstd-seekable-format-go/pkg
require
digest
42610f0 -> d6cafef

[!WARNING]
Some dependencies could not be looked up. Check the warning logs for more information.

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Renovate Ignore Notification
Because you closed this PR without merging, Renovate will ignore this update. You will not get PRs for the github.com/SaveTheRbtz/zstd-seekable-format-go/pkg d6cafef update again.
If you accidentally closed this PR, or if you changed your mind: rename this PR to get a fresh replacement PR.


Provide marginally better debug information for client side auth problems

The correct thing to do is move this string into the status response: this will take a little longer however.
Built on #2777, which needs to be merged first.
Partial fix for #2458.

LGTM waiting for green


'On confirm' event is not responding to callback return value

After writing some Mocha tests utilising Horseman I've come across an issue with Javascript confirmation dialogues.
Following the PhantomJs documentation of dealing with confirmation events. I'm returning true from the on confirm callback. This page was referenced by the Horseman docs.
As explained on the PhantomJs page:

true === pressing the "OK" button, false === pressing the "Cancel" button
};

No matter what I return in my Horseman code it says it says cancel was pressed.
I've created some test files to illustrate this issue.
My test HTML:
<!DOCTYPE html>

    <meta charset="utf-8">
    <title>Confirm check</title>

Confirm?

My test Horseman script:
var Horseman = require('node-horseman');

var browser = new Horseman();
var url = 'file:///Users/user/Desktop/confirm.html';

browser
    .viewport(1920, 1080)
    .on('loadFinished', function(status) {
        console.log('%s Status %s', url, status);
    })
    .on('consoleMessage', function( msg, lineNumber, sourceId ) {
        console.log('Console: %s, %s, %s', msg, lineNumber, sourceId);
    })
    .on('confirm', function( msg ) {
        console.log('Confirm message: %s', msg);
        return false;
    })
    .open(url)
    .click('a#confirm');

browser.close();

The terminal output with DEBUG=horseman enabled:
  horseman .setup() creating phantom instance on +0ms
  horseman .setup() phantom instance created ok. +448ms
  horseman .setup() creating phantom page. +1ms
  horseman .setup() phantom page created ok. +21ms
  horseman .setup() setting viewport. +0ms
  horseman .setup() viewport set ok. +4ms
  horseman .viewport() set +60ms
  horseman .on loadFinished set. +0ms
  horseman .on consoleMessage set. +0ms
  horseman .on confirm set. +0ms
  horseman .opening: file:///Users/user/Desktop/confirm.html +0ms
file:///Users/user/Desktop/confirm.html Status success
  horseman .open: file:///Users/user/Desktop/confirm.html - status: success +22ms
Confirm message: OK or Cancel?
Console: You pressed Cancel, undefined, undefined
  horseman .click() a#confirm +81ms
  horseman .close(). +0ms

I may be doing things wrong but I've not seen any indication that I should be doing things differently to respond to confirmation events.
Any help with this would be appreciated.
Cheers.

Hi @johntitus thanks for following this up with the PhantomJs guys.
Do you have any recommend work-arounds which could be used in Horseman? We chose Horseman as it's powerful but simple enough for our less skilled testing developers to use it would be great if we could keep using it and writing tests before the PhantomJs team come up with a fix.
Thanks again.

Grab 1.5.7. I added a wrapper around the issue, along with tests for clicking both Ok & Cancel to make sure both work.

Hi @johntitus, I am experiencing this issue with PhantomJs 2.0.0 and Horseman 2.8.1.
I took @Unlearn minimal test and I modified to work with promises:
<!DOCTYPE html>

    <meta charset="utf-8">
    <title>Confirm check</title>

Confirm?

And the JS:
'use strict';

var Horseman = require('node-horseman');

var browser = new Horseman();
var url = '/Users/marianocarpentier/development/horsemanMinimal/index.html';

var finished = false;
var promise = browser
    .viewport(1920, 1080)
    .on('loadFinished', function (status) {
        console.log('%s Status %s', url, status);
    })
    .on('consoleMessage', function (msg, lineNumber, sourceId) {
        console.log('Console: %s, %s, %s', msg, lineNumber, sourceId);
    })
    .on('confirm', function (msg) {
        console.log('Confirm message: %s', msg);
        return true;
    })
    .open(url)
    .click('a#confirm')
    .waitForSelector('#result-div')
    .evaluate(function (selector) {
        return ($(selector).text() === "YES");
    }, '#result-div')
    .then(function (res) {
        if (res) {
            console.log("Success");
        } else {
            console.log("Error");
        }
        browser.close();
        finished = true;
    });

var intl = setInterval(function () {
    if (finished)
        clearInterval(intl);
}, 100);

Following this thread, I managed to make it work by adding this workaround code into the actions.js file, line 999 for me:
...
else {
	var pageEvent = "on" + eventType.charAt(0).toUpperCase() + eventType.slice(1);
	self.page[pageEvent] = callback;
	if(pageEvent === "onConfirm") {
		self.page.setFn(pageEvent, callback);
	}
}
...

The if(pageEvent === "onConfirm") {...} is my workaround. If you have a better option, please let me know!
Cheers,

It sounds like we need to consider fixing the other event handlers where PhantomJS expects a return value, e.g., onPrompt.

@marianocarpentier, it looks a very similar line of code used to be there and has been removed. Perhaps @johntitus has a reason for removing it?

Actually, I don't consider this is a fix, I just added that as a temporary solution. I think that was removed for a reason, and since I don't know what that reason is, I didn't add code for onPrompt or made a pull request.

I can confirm I'm experiencing the same issue with the 'confirm' event.  I'm using node-horseman 2.8.2 and phantomjs 1.9.18.  @marianocarpentier's workaround fixed this for me.

I think the code that was there before is the way (or at least a way( to go about fixing it @marianocarpentier. Do you want to chime is as to why said code was removed @johntitus?

I am calling this issue solved in v3. I added an .at action. The return value of the callback is properly given to phantom for confirm, prompt, etc.
The way node-phantom-simple works, the .on callback's return value cannot have an effect.


Fix annotation

There is no Connection class or interface here. From the code I think the correct type here is Link.

Thanks!


white color text clearly visible now

text in light mode is not visible now

text in light mode is not visible now

@himanshujaidka @anushbhatia meant that since you changed the color of the text to white for dark mode, it is now invisible in normal mode

you have to refresh the page
for that

after refreshing we are able to see white text

Why should u refresh for that?

after refreshing we are able to see white text
I am aware of that 😅
This is the current deploy of your changes: https://deploy-preview-50--doc2pen.netlify.app/editor.html

yes please guide me on how I could achieve this without refreshing the page?

yes @anushbhatia you are right user doesn't want to refresh the page so how I could achieve this

Try having state for dark mode text and light mode text or u can di that using jquery by (textarea).removeclass(white-text).addclass(black-text) ; something like this but prefer using css for it using dark mode clour for text.

I have made another PR in this all the bugs which I mentioned above is solved but we have to refresh the page another bug arises 😅

Ok i will take care for this


Remove TUNNEL_MAP lookup

per discussion https://github.com/cilium/cilium/pull/17106#issuecomment-916297787, TUNNEL_MAP lookup is a fallback path for node pod cidr prefix, we could add the pod cidr prefix in IPCACHE and completely remove TUNNEL_MAP

With https://github.com/cilium/cilium/pull/19765 / https://github.com/cilium/cilium/pull/20117 we should be able to remove the requirement to specify the identity.

ok, I will wait then till they are merged

no  stale

Spoke with @joestringer a bit about this history.
Probably good to summarize quickly my notes from this chat.
Ideally, we want to reduce the usage of tunnel map w/r/t my work in IPsec/Encryption moving forward, so these notes lean a bit toward that objective.

Tunnel map specifies prefixes instead of addresses

Maps a prefix to a tunnel endpoint
Maps a prefix to an SPI
Exists to handle cases where IPCache has not been populated yet.

Tunnel map syncs on node creation, while IPCache syncs on pod creation. IPCache therefore has much more turn and higher prop delay.

Tunnel Map could potentially be removed by adding CIDR prefixes into the IPCache map.

Prefix entries will get their own security identity when inserted into the IPCache map.
Policy can be performed on the CIDR themselves once allocated a security ID.
The CIDR entries can act as a 'catch' for traffic from Pods which have

Not been allocated a security ID yet
Has been allocated a security ID but has not been propagated to the node's IPCache.

Tunnel map removal implications for IPsec

IPsec stores the SPI in three locations currently.

NodeID map
IPcache map
Tunnel map

The idea behind having the SPI in multiple areas is:

By default, node to node traffic should not be encrypted, therefore daddr lookup in IPCache for a InternalIP or CiliumInternalIP returns an encryptKey (SPI) of 0
If datapath knows the traffic is already encapsulated for tunneling, lookup for the InternalIP or CiliumInternalIP is done against the NodeID map which maps InternalIP types to their NodeID and SPI values.
Finally, if the traffic was headed to a pod, but an IPCache lookup fails, a more general lookup of the dadd is done against the cidr in the Tunnel map, and an SPI resides there to instruct this egress traffic for encryption.


RCTWebViewManager.m was missing from the macOS target

Please select one of the following

[ ] I am removing an existing difference between facebook/react-native and Microsoft/react-native :thumbsup:
[ ] I am cherry-picking a change from Facebook's react-native into Microsoft/react-native :thumbsup:
[X] I am making a fix / change for the macOS implementation of react-native
[ ] I am making a change required for Microsoft usage of react-native

Description of changes
During the last merge of RN .58 into the Microsoft fork, the RCTWebViewManager.m accidentally got dropped from being compiled into the macOS target.   This results in a RedBox error when a WebView is rendered because the NativeModule is not available.
Focus areas to test
This was found in manual testing of the RNTester app.   It will impact an macOS app that uses a WebView.
Microsoft Reviewers: Open in CodeFlow

How come it didn't break the build?
And are we using webview on mac?  Its being removed from RN as part of lean-core, so if we are not using it, we should maybe remove it?

How come it didn't break the build?
And are we using webview on mac? Its being removed from RN as part of lean-core, so if we are not using it, we should maybe remove it?

Internally we are not using WebView.  And we will remove it when the Microsoft fork merges from the facebook repo and its is removed there.   But in the meantime, RNTester is broken.   There is no automated test to actually show the WebView test panel, so I didn't know it was broken until I manually navigated to it.


Tap action not working

Checklist:

[x] I updated to the latest version available
[x] I cleared the cache of my browser

Release with the issue:
all
Last working release (if known):
i think all
Browser and Operating System:
Safari, iOS App on both iOS and macOS

Description of problem:
When I click on the compass it draws a grey box (screenshot attached) around it but unlike every other card, this does not support tap action nor hold action . Is this wanted behavior?
I wanted to add a radar graph showing the most common wind directions of the day when clicking on it, but i was not able to do it.

OT: I just noticed that your card is the only "custom" I have that shows up in the Lovelace GUI to add new cards: what a cool feature!! I love it!

Javascript errors shown in the web inspector (if applicable):

Additional information:

Good find!
The 'tap and hold' selection box around the card is a leftover from the boilerplate card code I used to start the compass-card.
I removed most of the stuff for that, however I was not able (yet) to find where the selection box came from.
This is on the wishlist as 'Cleanup unused boilerplate code'.
However, seeing that you'd like to use the 'tap and hold' functionality, I might as well label this an enhancement and stop searching for how to remove the selection box and just re-implement the removed functionality.
OT: Thanks for the love, sometimes just reading the documentation pays off. It's not mentioned in great detail however, the following line of example code preview: false, // Optional - defaults to false made me ask myself what would happen if I set that to true. And voila, one line of code, and a far better user experience for everyone.


Postmark Provider

Early PR so we can build together this Provider.
Right now it supports simple HTML emails, no attachments, no optional features from Postmark.

Let's get a bit of a roadmap together. I've reviewed Postmark's API docs, and they seem quite simple - certainly fewer configuration options than Sendgrid, which is great.
I think the following things are achievable.

Full implementation of the email send endpoint. Is there any reason to not always use the Batch Send endpoint?
Full implementation of the Template email send endpoint.
Full handling of all HTTP status codes and API error codes.

You're most of the way there already, so I don't think it should be very much more work to get this provider to a fully finished state. Let me know what you think.

I'm going to close this now as I'm updating the API to Vapor 2. Let me know if you're interested in implementing this again.


Still alive?

Is this project still alive, otherwise I might as well copy paste and and keep it updated under another name. Seems like no pull requests are getting through anymore and no issues are being handled.
Some major issues which have still not been picked up include ES6 support since you can't even properly import it at the moment which is a big no no.

+1

Up you go! +1


Fix capitalization of 'mini' in product names

As discussed here: https://github.com/lmirosevic/GBDeviceInfo/commit/18129e2b08044806b942a60a93ff443c2a6769c3
I only changed the strings, not the enums, as that looked weird and might break peoples builds.

Awesome!
Thanks @ricsantos
🚀 Published as 6.6.0


Use latest version of service module without S3 config

What is this PR trying to achieve?
Simplify service modules!
Who is this change for?
💻
Have the following been considered/are they needed?

[x] Deployed new versions

@kenoir I think this is ready for final review.

@alexwlchan all good.


initial /me/drives implementation

Inital spaces API, following https://github.com/owncloud/ocis/pull/1827
requires https://github.com/cs3org/reva/pull/1678

cc @dragotin

blocked by reva PR https://github.com/cs3org/reva/pull/1802


KcapiHash should not derive Clone

Hi,
Currently KcapiHash derives Clone and that makes the underlying pointer being shared by two objects after cloning. I believe this is leading to use-after-free bug since when the first instance goes out of scope Drop is called and the second one is using already freed instance.
The following test illustrates the problem:
use kcapi::md::KcapiHash;

let hash = KcapiHash::new("sha256")
    .expect("Failed to initialize hash handle");

hash.update([13, 10].to_vec()).unwrap();
let hash2 = hash.clone();
// std::mem::forget(hash);
hash2.update([13, 10].to_vec()).unwrap();
let digest = hash2.finalize()
    .expect("Failed to finalize message digest");

On my computer running this test results in:
---- src/md.rs - md::KcapiHash::finalize (line 240) stdout ----
Test executable failed (signal: 11 (SIGSEGV) (core dumped)).

If I uncomment the std::mem::forget(hash); line (that makes the drop not being called for one instance) the test works.
If Clone implementation is necessary I think it should be done using specialized method that copies the underlying handle to the new instance similarly as it has been done in the openssl crate:
https://github.com/sfackler/rust-openssl/blob/28b3925a329967f3ce1d3a1a7be2db55b75dd5e6/openssl/src/hash.rs#L347-L363

This is a great finding. I don't know why I derivce Clone here. I will make a pull request later this week to fix.

Don't worry I'll push a PR: https://github.com/puru1761/kcapi/pull/11
I wanted your confirmation that I'm not overlooking anything :)
The additional smaller issue is that these structs derive (Partial)Eq and that compares pointers... not sure if (Partial)Eq is needed there but it's not that security-sensitive as use-after-free here.


500 Internal Server Error during registration porecess with specific emails

When i try to register new user with<EMAIL_ADDRESS>email on my hub it returns 500 Internal Server Error
https://ussr.win/register
There webserver logs:
[Thu May 03 14:41:30.707792 2018] [fcgid:warn] [pid 4186] [client ip:port] mod_fcgid: read data timeout in 31 seconds, referer: https://ussr.win/register
[Thu May 03 14:41:30.707837 2018] [core:error] [pid 4186] [client ip:port] End of script output before headers: index.php, referer: https://ussr.win/register

When i try<EMAIL_ADDRESS>email during registration - everything goes ok.
I tried another hubs: with<EMAIL_ADDRESS>https://peacepharm.org/register - returns Not a valid email address
https://dacha.xmpp.moe/register - everything goes ok, this hub accepted this email, i received email and deny registration
and tried some other hubs - it was ok...
I tried enable/disable phpmailer - 1.1 plugin, it did not change anything.
Why my server returns error on some specific emails during registration?

As it turns out PHPMailer has a built-in timeout of 5 minutes. I just tried to connect to the SMTP port at yandex.ru and could not get a protocol greeting after waiting for about 10 minutes.
Not certain the best approach to take here. If we can't deliver mail reliably, the account can't validate. The application probably shouldn't fail, but this is the first time we've been made aware that this timeout exists.

@zotlabs I tried to track this error and found out that hangs on the image to the function z_dns_check(). And specifically on: dns_get_record()
I began to check on my server by making a file test.php with the call of this function:
<?php
var_dump(dns_get_record("gmail.com"));
?>

It returns array with DNS records...
Then i try dns_get_record("notregisteredomain83765.com") - it returns small array without any NS records.
Finally i tried dns_get_record("yandex.ru") - and result 500 Internal Server Error after timeout...
It is very interesting that after trying dns_get_record("ya.ru") (ya.ru is the short domain of russian yandex) - it reterned array with records, but dns_get_record("yandex.com") and dns_get_record("yandex.ua") hangs too (as yandex.ru). I checked - all these yandex domains use different DNS. I can not explain this behavior in any way.
It's Debian 9 server and system nslookup command works fine with all of this domains. Problem only with php dns_get_record().
Also i noticed article If you see an error during database setup that DNS lookup failed in /install/INSTALL.txt file, and add App::$config['system']['do_not_check_dns'] = 1; to .htconfig.php - after that registration for the domain yandex has earned.
This problem is not unique to my server. For example, on this hub, the check does not hang, but simply returns the false result:
https://peacepharm.org/register/email_check.json?f=&email=anyemail%40yandex.ru
https://peacepharm.org/register/email_check.json?f=&email=anyemail%40yandex.com
https://peacepharm.org/register/email_check.json?f=&email=anyemail%40yandex.ua
and correct result with ya.ru like my server:
https://peacepharm.org/register/email_check.json?f=&email=anyemail%40ya.ru
Probably, this problem is typical for other servers and for other domains. So just turning off dns checking on my server is not the best solution...
So i suggest using the function checkdnsrr() in z_dns_check(): checkdnsrr($domain) or checkdnsrr($domain,"MX"). On my server it works perfectly with any domains.
If you agree, i will make an appropriate pull request.

Also, i now found that the mandatory indication of the second parameter does not cause the function dns_get_record() to hanging:
var_dump(dns_get_record("yandex.ru", DNS_MX)); - returns the corresponding array. This is another way to solve this problem.

We used to check MX records for email domains but this causes issues for BSD platforms. Their PHP libraries only support  looking for 'any' DNS response and not specific DNS attributes. So perhaps this should be a site configuration. Thew problem we have is that some DNS checks are done during system installation before there is any way to set a configuration; so we opted to just look for the existence of any DNS record so that BSD would work. Otherwise Hubzilla could not be used at all on BSD platforms.
So perhaps we need to go the other way - provide system installation using any DNS response, but have a configuration which allows sites with working DNS to probe just MX records. Or as you've suggested, check for any response first, and then check for an MX record if that fails. You still might get a timeout, but the next query would go through. that would probably work for the BSD folks, since they'll get a valid response on the first query and won't try the second - which would give them a failure.

@zotlabs I made pull request witl little additions https://github.com/redmatrix/hubzilla/pull/1187 ... I found that checking for the second parameter is not done (commented out), although the reference from the problem function goes exactly with this second parameter. The call to a function z_dns_check() with the second parameter only occurs from function validate_email(). So the added condition is logically suitable for this check.
Such dns check processing should not change any existing functionality, while it solves the problem on my server, and maybe on peacepharm.org and some others...

@zotlabs Could you please check the work of z_dns_check() with this commit on MacOS / FreeBSD?
If the problem remains, then I think this will work in any case:
	if ($check_mx)
		return(checkdnsrr($h));
	else
		return((@dns_get_record($h) || filter_var($h, FILTER_VALIDATE_IP)) ? true : false);

With this commit https://github.com/redmatrix/hubzilla/commit/cd4454100b41f384148d0c1ebdaf6e64d629b0e1 everything works fine, thanks.


regex validator always shows error if required property is always is set to false

The validator for the following setting in binding.json will always throw, however if the required property is set to true then it works as expected.
{
	"name": "consumerGroup",
	"value": "string",
	"defaultValue": "$Default",
	"required": false,
	"label": "Event Hub consumer group",
	"help": "The name of the Event Hub consumer group to receive events from.",
	"validators": [
	    {
	        "expression": "(^[a-z0-9]$|^[a-z0-9][a-z0-9-_.]{0,48}[a-z0-9]$)|^\\$Default$",
	        "errorText": "Your consumer group name"
	    }
	]
}

related issue https://github.com/Azure/azure-webjobs-sdk-templates/issues/251

Reopening since i can still repro this


Child View Blur at Move or Zoom

From<EMAIL_ADDRESS>on November 20, 2013 10:45:00
Thanks for your jar.
I am fetching one issue when I will zoom or move, Child objects get blur for some milliseconds.
May you help me on that pls?
Original issue: http://code.google.com/p/android-zoom-view/issues/detail?id=6

thanks for your reply , i have also same problem
Now its working fine by using zoomView.setAnimationCacheEnabled(false);


Add log_msg extern function to pna.p4 include file?

This exists in the v1model.p4 include file, here: https://github.com/p4lang/p4c/blob/main/p4include/v1model.p4#L705-L711
You can find a few examples of its use in this v1model architecture program that runs on BMv2 simple_switch/simple_switch_grpc here: https://github.com/jafingerhut/p4-guide/blob/7b60e42d8aa3e7afe11d630d7f3357bff3064bb9/ptf-tests/registeraccess/registeraccess.p4
Like assert and assume (see https://github.com/p4lang/pna/issues/65 ) it is useful for testing and development on software switch environments, and would typically be treated as no-op when compiling to hardware targets.

TODO Andy: Discussing this with Cristian Dumitrescu on precisely how these would be implemented on DPDK.
Note that they are already implemented on BMv2 for the v1model architecture.  The PSA implementation for BMv2 is only partial as of 2022-Oct, with no one that I know of currently actively working to finish it.  Similarly, I do not know of anyone working on BMv2 implementation of PNA architecture at this time.

Consider adding log_msg to both PSA and PNA, not only PNA.

+1 for me.
I might have HW which supports having these log messages out somewhere so it might be useful to mention that like assert it can be turned into a no-op by some mechanism (like you said on real HW it would be a no-op).

This is more question of how {} are handled, and about the formatting part. Does {0} says the first element of the tuple or is it just {}. How does printing of a header/struct work? For a header, does it print each field and the valid/invalid field. Or is the tuple just supposed to be a tuple of bit/int/bool types (or is bool not supported there).
Or is this just left up to the implementation on what is supported or not supported?
I think defining a standard set of what is the min supported here would be a good idea.

I think the BMv2 implementation of this is fairly bare-bones, where each {} in the format string corresponds to the next value to be printed, and there is no variation supported in the format, e.g. no choice of decimal/hex/binary/etc.  If you have an Ubuntu system where you are willing to install p4c/behavioral-model/etc. open source dev tools, and write little v1model programs to try out different types of values to log_msg to see what it does today, it shouldn't take long, but I can hack up something over the weekend to show some examples too.

I have updated the comments in the p4c test program issue2201-bmv2.p4 with this PR, so that the comments are up to date with the latest behavioral-model/BMv2 simple_switch behavior of its implementation of the log_msg extern function: https://github.com/p4lang/p4c/pull/3637
Note: There is no spec defining its behavior.  It was an implementation created by someone for BMv2, and I'd say does something fairly expedient but useful.  I do not know how many people rely on its current precise behavior.  I would guess not many, but for all I know someone has created automated test scripts that parse the output of simple_switch to recognize these messages and check their contents :-)


tabula muris crashes ingest ui

reported by @ESapenaVentura
https://contribute.data.humancellatlas.org/projects/detail?uuid=e0009214-c0a0-4a7b-96e2-d6a83e966ce0

moving to dev board


Aperture updates related to JSOCOPS-24 (limited to Grism Time Series)

Under this PR NIRCam will deliver the follwoing changes:

F444W field point (several GRISMTS apertures in SIAF): Shift X by -94 LW px (DMS oriented).

NRCA3 and NRCA1 GRISM128 aperture: Shift Y by -8 SW px (DMS orientation)

NRCA3 and NRCA1 GRISM64 aperture: Shift Y by -10 SW px (DMS orientation)

Adjust the NRCA5_TAGRISMTS32_F405N:

Target should move up and right in DMS/Science coordinates

Delta X = +0.165 LW px
Delta Y = +0.16 LW px

This PR is ready for review

Now this is weird...
Every test passes on my Mac, but test_aperture.py::test_idl_to_tel fails here for NIRISS. I noticed the values are very small- 7e-14.  We still get the warnings about 'np.float' being deprecated and that we should switch to 'np.float64' to avoid platform differences. Is that what's going on here?
On my Mac:
`========================================================= test session starts ==========================================================
platform darwin -- Python 3.9.12, pytest-7.1.1, pluggy-1.0.0
rootdir: /Users/dlong/pysiaf, configfile: setup.cfg
collected 32 items
test_aperture.py ......                                                                                [ 18%]`
And then on witserv2:
`========================================================= test session starts ==========================================================
platform linux -- Python 3.9.13, pytest-7.1.2, pluggy-1.0.0 -- /user/dlong/linux/miniconda3/envs/pysiaf/bin/python
cachedir: .pytest_cache
rootdir: /home/dlong/pysiaf, configfile: setup.cfg
collected 32 items
test_aperture.py::test_idl_to_tel FAILED                                                                                [  3%]`
@tonysohn is it safe to ignore these tests failures here, or is something more serious broken?

@Witchblade101 Was that the exact error message? It's hard to tell whether it's safe to ignore or not just from that...

@Witchblade101 Was that the exact error message? It's hard to tell whether it's safe to ignore or not just from that...

No. To see the full error look at the failed checks for PR #261 (which is what these comments refer to. It was a long night, and Mario's NRC work here is completely innocent!)

Deleted comments that were intended for #261 after copying them there.

Complicated conflicts:
remote: Enumerating objects: 31, done.
remote: Counting objects: 100% (31/31), done.
remote: Compressing objects: 100% (5/5), done.
remote: Total 31 (delta 26), reused 30 (delta 26), pack-reused 0
Unpacking objects: 100% (31/31), 2.87 KiB | 133.00 KiB/s, done.
From https://github.com/mgennaro/pysiaf

branch            JWSTSIAF-186 -> FETCH_HEAD
Removing pysiaf/source_data/NIRSpec/nirspec_siaf_detector_parameters.txt
Removing pysiaf/source_data/NIRSpec/nirspec_siaf_ddc_apername_mapping.txt
Removing pysiaf/source_data/NIRSpec/nirspec_siaf_aperture_definition.txt
Removing pysiaf/source_data/NIRSpec/delivery/test_data/apertures_testData/sky_fpa_projectionMSA_Q4.fits
Removing pysiaf/source_data/NIRSpec/delivery/test_data/apertures_testData/sky_fpa_projectionMSA_Q3.fits
Removing pysiaf/source_data/NIRSpec/delivery/test_data/apertures_testData/sky_fpa_projectionMSA_Q2.fits
Removing pysiaf/source_data/NIRSpec/delivery/test_data/apertures_testData/sky_fpa_projectionMSA_Q1.fits
Removing pysiaf/source_data/NIRSpec/delivery/test_data/TA_testDataWithGWATilt/testDataTA_SCA492F140X.fits
Removing pysiaf/source_data/NIRSpec/delivery/test_data/TA_testDataWithGWATilt/testDataTA_SCA492F110W.fits
Removing pysiaf/source_data/NIRSpec/delivery/test_data/TA_testDataWithGWATilt/testDataTA_SCA492CLEAR.fits
Removing pysiaf/source_data/NIRSpec/delivery/test_data/TA_testDataWithGWATilt/testDataTA_SCA491F140X.fits
Removing pysiaf/source_data/NIRSpec/delivery/test_data/TA_testDataWithGWATilt/testDataTA_SCA491F110W.fits
Removing pysiaf/source_data/NIRSpec/delivery/test_data/TA_testDataWithGWATilt/testDataTA_SCA491CLEAR.fits
Removing pysiaf/source_data/NIRSpec/delivery/test_data/TA_testDataNoTilt/testDataTA_SCA492F140X.fits
Removing pysiaf/source_data/NIRSpec/delivery/test_data/TA_testDataNoTilt/testDataTA_SCA492F110W.fits
Removing pysiaf/source_data/NIRSpec/delivery/test_data/TA_testDataNoTilt/testDataTA_SCA492CLEAR.fits
Removing pysiaf/source_data/NIRSpec/delivery/test_data/TA_testDataNoTilt/testDataTA_SCA491F140X.fits
Removing pysiaf/source_data/NIRSpec/delivery/test_data/TA_testDataNoTilt/testDataTA_SCA491F110W.fits
Removing pysiaf/source_data/NIRSpec/delivery/test_data/TA_testDataNoTilt/testDataTA_SCA491CLEAR.fits
Removing pysiaf/source_data/NIRSpec/delivery/positionsSIAFApertures.fits
Removing pysiaf/source_data/NIRSpec/delivery/disperser_MIRROR_TiltY_TA.gtp
Removing pysiaf/source_data/NIRSpec/delivery/disperser_MIRROR_TiltX_TA.gtp
Removing pysiaf/source_data/NIRSpec/delivery/delivery_SCA492toGWA.pcf
Removing pysiaf/source_data/NIRSpec/delivery/delivery_SCA491toGWA.pcf
Removing pysiaf/source_data/NIRSpec/delivery/delivery_F140X_GWA2XanYan.pcf
Removing pysiaf/source_data/NIRSpec/delivery/delivery_F110W_GWA2XanYan.pcf
Removing pysiaf/source_data/NIRSpec/delivery/delivery_CLEAR_GWA2XanYan.pcf
Removing pysiaf/source_data/NIRSpec/delivery/OTE.pcf
Removing pysiaf/source_data/NIRSpec/delivery/Fore_F290LP.pcf
Removing pysiaf/source_data/NIRSpec/delivery/Fore_F170LP.pcf
Removing pysiaf/source_data/NIRSpec/delivery/Fore_F140X.pcf
Removing pysiaf/source_data/NIRSpec/delivery/Fore_F110W.pcf
Removing pysiaf/source_data/NIRSpec/delivery/Fore_F100LP.pcf
Removing pysiaf/source_data/NIRSpec/delivery/Fore_F070LP.pcf
Removing pysiaf/source_data/NIRSpec/delivery/Fore_CLEAR.pcf
Auto-merging pysiaf/source_data/NIRCam/nircam_siaf_aperture_definition.txt
CONFLICT (rename/delete): pysiaf/pre_delivery_data/NIRCam/nircam_roundtrip_PRDOPSSOC-045-003_pre-delivery.txt deleted in fca75c1d51db63aa26ca3d58e558c39cb2c9280c and renamed to pysiaf/pre_delivery_data/NIRCam/nircam_roundtrip_PRDOPSSOC-051_pre-delivery.txt in HEAD. Version HEAD of pysiaf/pre_delivery_data/NIRCam/nircam_roundtrip_PRDOPSSOC-051_pre-delivery.txt left in tree.
CONFLICT (modify/delete): pysiaf/pre_delivery_data/NIRCam/nircam_enhanced_siaf_aperture_definition.txt deleted in fca75c1d51db63aa26ca3d58e558c39cb2c9280c and modified in HEAD. Version HEAD of pysiaf/pre_delivery_data/NIRCam/nircam_enhanced_siaf_aperture_definition.txt left in tree.
CONFLICT (modify/delete): pysiaf/pre_delivery_data/NIRCam/NIRCam_SIAF.xml deleted in fca75c1d51db63aa26ca3d58e558c39cb2c9280c and modified in HEAD. Version HEAD of pysiaf/pre_delivery_data/NIRCam/NIRCam_SIAF.xml left in tree.
CONFLICT (modify/delete): pysiaf/pre_delivery_data/NIRCam/NIRCam_SIAF.xlsx deleted in fca75c1d51db63aa26ca3d58e558c39cb2c9280c and modified in HEAD. Version HEAD of pysiaf/pre_delivery_data/NIRCam/NIRCam_SIAF.xlsx left in tree.
Auto-merging generate/generate_nircam.py
CONFLICT (content): Merge conflict in generate/generate_nircam.py
Automatic merge failed; fix conflicts and then commit the result.

I'll start resolving things by deleting the previous output files.

`<<<<<<< JWSTSIAF-186
# # make figures for JWSTSIAF-186 Jira ticket
selected_aperture_names = [['NRCA1_GRISMTS256','NRCA1_GRISMTS128','NRCA1_GRISMTS64'],
['NRCA3_GRISMTS256','NRCA3_GRISMTS128','NRCA3_GRISMTS64'],
['NRCA5_TAGRISMTS32','NRCA5_TAGRISMTS32_F405N']
]
        # # make figures for JWSTSIAF-242 Jira ticket
        selected_aperture_names = [['NRCB{}_SUB160'.format(sca) for sca in ['5','1','2','3','4']],
                                   ['NRCB{}_SUB320'.format(sca) for sca in ['5','1','2','3','4']],
                                   ['NRCB{}_SUB640'.format(sca) for sca in ['5','1','2','3','4']],
                                   ['NRCB5_SUB64P','NRCB1_SUB64P','NRCB5_FULL','NRCB1_FULL'],
                                   ['NRCB5_SUB160P','NRCB1_SUB160P','NRCB5_FULL','NRCB1_FULL'],
                                   ['NRCB5_SUB400P','NRCB1_SUB400P','NRCB5_FULL','NRCB1_FULL'],
                                   ['NRCB1_SUB64P','NRCB1_SUB160P','NRCB1_SUB400P','NRCB5_SUB64P','NRCB5_SUB160P','NRCB5_SUB400P']
                                  ]

siaf-updates`

I think selected_aperture_names needs to have all of the apertures for both tickets, then we'll be good to go.

yes, you can choose to have them both, or becuase you already produced the figures for 242, you can actually leave just those for 186 and post those to the JWSTSIAF-186 ticket

Right, I see what you're saying. It is just the loop that creates the figures.
I merged them locally, and it worked with no problems, but I think it makes more sense to do the auxiliary files in their separate tickets.

I actually could solve the conflict myself (I probably have write privileges because this is my PR). I chose to go with -186 only, becuase we already have the plots for 242

Sounds good.

This approach is also more similar to the typical sequential flow we use for when we do one ticket at a time. I.e. we modify the siaf-upadtes version of generate_nircam to put the "new" plots in that statement that was giving conflict. We usually do not have conflicts because the feature branch is the one and only feature branch that is branched off siaf-updates, but in this case we had 2 branches branched off at the same point in time from siaf-updates and the merging of the jwstsiaf-242 PR into siaf-updates has caused this minor conflict that indeed was expected.

Uh-oh, we just did something bad. This last PR deleted all of the NIRSpec data:
======================================================= short test summary info ========================================================
FAILED test_aperture.py::test_raw_transformations - FileNotFoundError: [Errno 2] No such file or directory: '/Users/dlong/pysiaf/pysi...
FAILED test_nirspec.py::test_against_test_data - FileNotFoundError: [Errno 2] No such file or directory: '/Users/dlong/pysiaf/pysiaf/...
FAILED test_nirspec.py::test_nirspec_aperture_transforms - FileNotFoundError: [Errno 2] No such file or directory: '/Users/dlong/pysi...
FAILED test_nirspec.py::test_nirspec_slit_transformations - FileNotFoundError: [Errno 2] No such file or directory: '/Users/dlong/pys...
FAILED test_nirspec.py::test_sci_det_consistency - FileNotFoundError: [Errno 2] No such file or directory: '/Users/dlong/pysiaf/pysia...
FAILED test_plotting.py::test_aperture_plotting - FileNotFoundError: [Errno 2] No such file or directory: '/Users/dlong/pysiaf/pysiaf...
==================================================== 6 failed, 26 passed in 24.37s =====================================================

I think I can just try a new branch and pull those in from master...

Holy cow...that is my faults:
https://github.com/spacetelescope/pysiaf/pull/254/commits/569b8d1a40b001d27bdfa765cf4e251b95300955
I was just seeing all these "._" files in my branch and I git rm-ed them, but it looks like i removed also the actual files. OMG I am so sorry.


feature/create-signin-page-ui

Work List

Create a signin page

Design

UI

Left: sign-in page design
Right: sign-in form

Elements of sign-in form

Title
Description
E-mail and Password Input

label, input, placeholder, error state and message

Remember Me checkbox and Forgot Password
Sign-Up page link

Closed by #112


Improvements to kafka_int_tests

We are seeing some issues with kafka in a test setup that are not showing up in our integration tests.
I've got a few ideas as to why it passes in our integration test but fails here:

It could be specific to the java driver, we currently only test against the C/C++ driver.
It could be specific to the way the cluster is setup. Currently we only test against against a cluster of 3 nodes without racks configured.

We should investigate adding the following to our test suite:

6 node cluster across 2 racks.
Add the java driver as a backend to KafkaConnectionBuilder.

Both of these have been completed.


Terms of Use link in VS Code links to blank wiki page

Steps to reproduce

Enable extension and expand the popup message which should show the below text and click the link

We're excited that you're test driving Einstein for Developers. Make sure to read our Terms of Use before you start.

Issue
Expected behavior: The link takes you to the Terms of Use page
Actual behavior: You are redirected to a blank page as shown below as the page doesn't exist (hence the edit page warning)

Extension: salesforce.salesforcedx-einstein-gpt
Version: v0.19.1

Thanks for reporting! We've restored the missing Terms page that was set aside in the shuffle.
Let us know if you have any more questions.


Incorrect use case information for the 'MSIE 10.0; Windows NT 6' user agent string

The use cases listed for the 'MSIE 10.0; Windows NT 6' user agent string is incorrect it says: "IE 10.0 for Windows XP and newer versions of desktop operating system" IE 10.0 isn't available on Windows XP (Only IE 8.0 is), I believe it should say  "IE 10.0 for Windows Vista and newer versions of desktop operating system".

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

ID: 375a5fd8-c881-b6f7-c93f-b8898020c43f
Version Independent ID: 0c58c9d1-cb1f-03b2-9456-3e5550e8d8ab
Content: Configure browsers to use Windows Integrated Authentication (WIA) with AD FS
Content Source: WindowsServerDocs/identity/ad-fs/operations/Configure-AD-FS-Browser-WIA.md
Product: windows-server-threshold
GitHub Login: @billmath
Microsoft Alias: billmath

#please-close


Disable output of Kirki styles - solution

Hello @aristath,
First, I want to congratulate you on the genius touch of 'transport' => 'auto' !
Issue description:
Using 'disable_output' => true was rendering a my-theme_no-kirki-css stylsheet who still was rendering a my-theme_no-kirki-inline-css.
This was affecting the theme original styles, and outputting a weird behavior...
Version used:
I'm actually using Version 2.3.7
Using theme_mods or options?
option_type'   => 'theme_mod'
Quick fix
In functions.php
/**
 * Disable output of kirki styles if the plugin is disabled or removed.
 */
if( ! class_exists( 'Kirki' ) ) {
    function my_theme_remove_kirki_styles() {
        wp_dequeue_style( 'my-theme_no-kirki' );
        wp_deregister_style( 'my-theme_no-kirki' );
    }
    add_action( 'wp_enqueue_scripts', 'my_theme_remove_kirki_styles', 20 );
}

Hope this can help others.
Thanks a lot for your time and all of your efforts.
SYA :-)

First, I want to congratulate you on the genius touch of 'transport' => 'auto' !

Thank you, I really appreciate that! Version 3.0 will be a whole lot better, this is getting a complete rewrite. 😉
As for the extra stylesheet that was getting added, this has already been fixed in version 3.0 so I'll go ahead and close this one 👍

Can't wait to see V 3.0. Actually using Kirki for a theme that will be soon released on wp.org repo, so I have to use and recommend the repo plugin.
Thanks again 👍


Deleting the header in text editor duplicates it

If the header in the text editor is deleted, another copy of the header is placed in front of it.
The header text should be either read only or detect what the user has deleted...

The header should be set to read only

Fixed as of https://github.com/dfst/deepforge/commit/3f0b7e472013fce1ead33eb0137d60c297a9f456


1 minute batching of plays as implemented in #32 is maybe too short for slow games

<johnm> Also, would it be possible to instead of messaging when someone plays, message updates once per minute for a few minutes, then reminding people every hour or so? then during busy times with big numbers of players it'll probably just go straight to the pick a winner step
<grifferz> it sends a play notification one minute after a play is made and collects up all the plays made in that minute
<johnm> grifferz: ah OK. :)
<plett> grifferz: Maybe once per 10 minutes for the play notification would be better in the slow-play version? Or even longer?

Original implementation was discussed in #32.
How about we try the play batching timer being 1/60th of the turnclock? So #debug would remain at 60 seconds (turnclock is 3600 seconds), #bitfolk would become 24 minutes (86400 seconds divided by 60).

Yes, setting it as a fraction of the turn timer sounds like a sensible thing to do.

This seems to be working okay.


Weird message that appears once every few searches

After the upgrade to 0.0.6, straw viewer pops up this message once every few searches
[JSON]: , or } expected while parsing object/hash, at character offset 144465 (before "endscreenClick":+["h...") at /home/user/lib/WWW/StrawViewer/ParseJSON.pm line 34.
It does not seem to affect anything, since playback and search both work fine, but I thought I should mention it.
Straw viewer 0.0.6, youtube-dl 2<IP_ADDRESS>, perl 5.30.3, mpv 0.32 as the player and all that on a machine running debian testing x64.
Since my distro does not package it, I have put the binary inside ~/bin and its libraries in ~/lib and I have this bit in my .bashrc to make it work
PATH=$PATH:$HOME/bin/
PERL5LIB=$HOME/lib;export PERL5LIB

It has been like that since I started using youtube viewer on my machine, 5+ years ago, and it works flawlessly.

I'm not quite sure why this happens. Most likely it's an YouTube issue. I will have to investigate this more deeply to see if this can be fixed in straw-viewer or the warning should just be hidden.
Same as: https://github.com/trizen/straw-viewer/issues/3


Error code 1010

I use the API with OpenHAB to display some values for private use.
Since three days i get "error code 1010" as response. Looks like a Cloudfrare issue: https://support.cloudflare.com/hc/de/articles/360029779472-Cloudflare-1XXX-Fehler-beheben#error1010
Has been added some filtering for non-browser access or is this my "personal" problem?

Does it work now? I disabled the browser integrity check.

The error disappeared today at about 12:20.
But from 14:18 until now I see a lot of other errors, e.g.
Calling
https://api.corona-zahlen.org/districts/03404/history/cases/7
results in
{"error":{"message":"There is a problem with the official RKI API.","rkiError":{"code":499,"message":"Token Required","messageCode":"GWM_0003","details":["Token Required"]},"url":"https://services7.arcgis.com/mOBPykOjAyBO2ZKk/arcgis/rest/services/RKI_COVID19/FeatureServer/0/query?where=NeuerFall IN(1,0) AND IdLandkreis = '03404' AND MeldeDatum >= TIMESTAMP '2021-10-26'&objectIds=&time=&resultType=standard&outFields=AnzahlFall,MeldeDatum,Landkreis,IdLandkreis&returnIdsOnly=false&returnUniqueIdsOnly=false&returnCountOnly=false&returnDistinctValues=false&cacheHint=false&orderByFields=IdLandkreis,MeldeDatum&groupByFieldsForStatistics=IdLandkreis,MeldeDatum,Landkreis&outStatistics=[{\"statisticType\":\"sum\",\"onStatisticField\":\"AnzahlFall\",\"outStatisticFieldName\":\"cases\"}]&having=&resultOffset=&resultRecordCount=&sqlFormat=none&f=json&token="}}
Calling
https://api.corona-zahlen.org/districts/03404
results in
{"error":{"message":"An error occurred.","details":"Cannot read properties of undefined (reading 'map')","stack":"TypeError: Cannot read properties of undefined (reading 'map')\n    at /workspace/src/data-requests/districts.ts:89:35\n    at Generator.next (<anonymous>)\n    at fulfilled (/workspace/dist/data-requests/districts.js:5:58)\n    at processTicksAndRejections (node:internal/process/task_queues:96:5)"}}
Calling the API via my browser produces the same errors. So that might be something different.

Now it's working again.
Thanks for your help!


Bogus characters in 'pending' message

The status of a sent message changes to Pendingâ€¥  before the SuccessResponse is received.

Duplicate of #600 due to problems on waffle.io when creating