Question: Applying migrations fails during CI - file not found.

Question:
Is the .surrealdb configuration file required?
Issue:
As I was writing this, I realized I also had this issue when trying to invoke the CLI from an initialization script. For some reason, whether I have a configuration file specified or not, the test panics with the following during CI (Github actions):
CI Panic:  'Failed to run migrations.: Path does not directory'
Note, this works when invoking cargo test but breaks when Github actions fires so I'm sure it's some sort of relative path thing, but I'm not entirely sure.
File: tests/health_check.rs
pub async fn configure_database(config: SurrealdbConfiguration) -> color_eyre::Result<()> {
    SurrealdbMigrations::new(config)
        .up()
        .await
        .expect("Failed to run migrations.");

    Ok(())
}

Repo:

zero2axum - working through Zero to Production w/Axum & SurrealDB (or at least trying to)

Hi @snarkipus
The .surrealdb configuration file is not required unless you want to put your migration project inside a nested folder. In that case, specifiying the path is only possible with the configuration file. I can see you created your migration project at the root of your repository so you don't have to write a configuration.
About the failure on the CI, can you make sure you are running the cli on the root folder? (where the Cargo.toml file is)

I made a very simple repro project using the blog template and running integration test in the GitHub Actions CI. It works.
I will try to look into your code but right now, I do not see what can go wrong.

The error Path does not directory is an error from the fs_extra. So clearly, there is something wrong with the filesystem. But the question is still why?

Well, ok, it seems that the events folder does not exist because it is not created on the Git filesystem. Will make a fix asap.

Thanks @Odonno ! I'll report back if there's any more issues.


Ability to position model external to glb/gltf content

It would be helpful to be able to translate and rotate content external to the model file so that it loads in a desired position. Perhaps through additional tag attributes.
A good use case for this is when loading content from a 3rd party that you don't have edit control over but want to load in a specific orientation.

This has come up in other discussions we have had. I think a top-level rigid transform is not out of the question for us, and has been demonstrated to be desirable for some use cases.
That said, a lot of cases can be handled by the attributes we already have: camera-orbit and camera-target. These attributes let you position the camera relative to the model, which will have the same end result in many cases. Have you tried using those attributes?

Yes, camera positioning will definitely work in some cases but a common use case for us (Smithsonian) would be more about relative positioning of objects from different source files (which I know is also not yet supported) which would need rigid transforms.

@gjcope thanks for confirmation!
For the multi-object use case, the only current idea we have floated is to enable that via 3DOM, peusdocoded:
<model-viewer src="some.glb">
  <script type="experimental-scene-graph-worklet" allow="node-hierarchy; node-transforms; load-models">
addEventListener('model-change', () => {
  // Translate the model loaded from 'some.glb'
  model.scene.nodes[0].setTranslation(...);

  loadModel('other.glb').then((otherModel) => {
    // Translate the model loaded from 'other.glb'
    otherModel.scene.nodes[0].setTranslation(...);
    // Append the other model to some model's scene graph
    model.scene.appendModel(otherModel);
  });
});
  </script>
</model-viewer>

Although this theoretically gives you the capability to do what you describe, I wonder if there is a corresponding declarative form to accomplish something similar (e.g., put one model inside another, and apply transformations to both).
At any rate, I think a top-level rigid transform may make sense for us to add.
cc @elalish for additional commentary.

A top-level transform attribute sounds like a good idea. Even for the single-object case, you want it to be oriented properly relative to the world. Otherwise, for a model that is flipped on its side, camera controls would get confusing.


Build failure: mariadb_114, mariadb_110 and mariadb_106

Steps To Reproduce
Steps to reproduce the behavior:

build mariadb_114

Build log
nix log /nix/store/vkvm2kwvcf25drcanfl6l68rrwfdmh3l-mariadb-11.4.3.tar.gz.drv
warning: The interpretation of store paths arguments ending in `.drv` recently changed. If this command is now failing try again with '/nix/store/vkvm2kwvcf25drcanfl6l68rrwfdmh3l-mariadb-11.4.3.tar.gz.drv^*'

trying https://downloads.mariadb.com/MariaDB/mariadb-11.4.3/source/mariadb-11.4.3.tar.gz
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
^M  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0^M  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
curl: (22) The requested URL returned error: 404
error: cannot download mariadb-11.4.3.tar.gz from any mirror

Additional context
The mariadb source URL is probably stale or outdated. I can confirm the source tarballs for mariadb_114 , mariadb_110 and mariadb_106 are not available at https://downloads.mariadb.com/MariaDB/. mariadb_105 source tarballs are available and does not return a 404 error.
Notify maintainers

@ajs124 @dasJ @Conni2461 @thoughtpolice
Metadata

system: "x86_64-linux"
host os: Linux 6.6.59, NixOS, 24.05 (Uakari), 24.05.20241104.d063c1d
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.18.8
nixpkgs: /etc/nix/inputs/nixpkgs

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

fixed already on unstable. we havent backported the last two patch releases.

https://github.com/NixOS/nixpkgs/pull/334878
https://github.com/NixOS/nixpkgs/pull/353661

Manually updating the commit in flake.lock did the trick

Should still be fixed on 24.05


Look at adding Etherpad-Lite to the UX mix

With this api we can write a home page for managing EtherPads to the UX capabilities

We know that using an etherpad client in FeatherWeight, we can control access and other issues.
Questions to answer include:

When a user clicks new pad in FeatherWeight's presumed Pad Index, does that automatically open a new pad?

When a pad is created, do we get a link to it such that our PadService database can provide an array of pads to paint on the index?

Note that Etherpad Lite uses the AbiWord system for import and export of pads. Since the API allows us to fetch a pad's content for a given version number, it might be the case that we can deal with content handling directly.
It's true that AbiWord allows to export to PDF, Word, etc, and those are nice, but, in the event that we cannot make use of that feature, it appears that we can get by without it.

Started by adding the Etherpad-Lite client to this repo.
Intention is to explore using the API.
It means that I probably must have Etherpad-Lite "online"
Still exploring

We now have an Etherpad.vue communicating with Etherpad. Next up is to explore opening pads in iframes under control of an Etherpad index page

We know that

Pads belong to a group; creating a group returns a groupId

Pads are created in the context of a groupID; creating a Pad returns a padId

Pads are started by creating a session
See the API


Why is my bluetooth microphone not connecting with the Audiotoggle plugin ?

Issue
When I am using the audiotoggle to change from speaker to bluetooth in Angular wrapped in Cordova . I could speak to the speaker but is unable to speak to the bluetooth microphone .I imported it  in my angular project.
if(Headset is not detected){
      Audiotoggle--->speaker
}
else{
Audiotoggle-->Earpeice
}

Could you tell me if there is a solution ?
@lylepratt @alongubkin

+1


[Improve Best Practice Guide]: Continuous integration (RA) through Jenkins, et. al.

Checked for duplicates
Yes - I've already checked
Best Practice Guide
Continuous Integration
Best Practice Guide Sections

[ ] Use Cases
[ ] Trade Studies
[x] Reference Architectures
[ ] Starter Kits

Describe the improvement
We need a reference architecture diagram for building and deploying a project to Artifactory.
This and another item were reflected previously in the R1 change log but were delayed to address concerns raised during code reviews.

This is completed and merged, all code review comments addressed.
Closing.


Fix typo in bbox_flip

 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.


integrate.quad fails silently on pulse type functions with long zero tails

Looking at a function like a step function but reversed in x (y = 1 if x is less than or equal to 0) integrate.quad has a silent failure. Here is a simple example of the failure.
>>> from scipy import integrate
>>> y = lambda x: 1 if x<=0 else 0
>>> integrate.quad(y, -1, 1)
(1.0, 1.1102230246251565e-14)     # perfect!
>>> integrate.quad(y, -1, 100)
(1.0000000002199108, 1.0189464580163188e-08)     # we lost precision
>>> integrate.quad(y, -1, 10000)
(0.0, 0.0)    # this is not correct

With more and more trailing zero's in the function, it gets harder for quad to find the non-zero values at the start.
This issue might be related to the issue about QUADPACK losing precision when used over large intervals, #5428. Pulse functions are also discussed in #4264, but there they are using integrate.simps, and integration does appear to work with sample-based methods (like .romb and .simps) assuming you correctly samples the pulse.
We may just want to add this limitation to the documentation (along with an example as desired in #7168) rather than try to find a generic way to fix it.

We may just want to add this limitation to the documentation

I think this is the way to go. Expecting a quadrature routine to find the support of the function it's supposed to be integrating is not a reasonable expectation.

Should I add this comment to the notes section of the documentation or as an example?

Maybe an example?
.
It's probably not so easy to give a precise statement on when the method
will work --- the issue is probably not only in finding the support but
also in finding sharp features in general.


UW MAIZSIM additional changes

Since #28 has been already merged, I'm opening a new PR to collect additional changes needed to take care of issues as we find them. Dennis, please don't merge it until it's ready.

Changes from tomyun. I updated gas exchange recently where JW (water flux from soil) is no longer used in the leaf energy balance, this had not been done in tomyun's verison. I resolved the conflict and merged.


Cardiac UDA dataset - no labels for Site R - 73

Thank you very much for organizing this dataset and for your amazing work! I am wondering whether segmentation labels are provided for the Site_R_73 partition. I have not been able to find them after downloading the dataset from the repository: https://www.kaggle.com/datasets/xiaoweixumedicalai/cardiacudc-dataset. Additionally, the number of files contained in other subfolders appears to be slightly mismatched compared to the description (e.g., 97 subjects in Site_G_100), which you might want to double-check. Thank you!

I just noticed one typo in Figure 2 caption: it should be left atrium (LA) instead of "left atrium (RA)".


Change the Sponge DataManipulators to extend the API abstract classes.

With SpongePowered/SpongeAPI#940, the abstract superclasses used in SpongeCommon for the DataManipulators (such as AbstractIntData) were moved to SpongeCommon. However, the old classes still remain in the implementation, and many manipulators still use them. Would it be possible to delete the abstract superclasses from SpongeCommon and change the manipulators to use the API versions of those classes?

One of the major roadblocks we have is that many of the Abstract classes provide caching functionality which the API implementations do not. The is mainly the boolean, enum and catalog typed values, as far as I'm aware.

No. There's a reason why I didn't change their implementations. The API classes don't use any of the reflection utils or caching utils.


How to install ?

Hi ...
awsome work,
Are there any steps on using this ?
Thanks in advance

There is an installation guide in the docs.


Adding maven wrapper

Impact

[ ] Bug fix (non-breaking change which fixes expected existing functionality)
[X] Enhancement/New feature (adds functionality without impacting existing logic)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Adding maven wrapper support.
DOCS notes: fix https://contribute.liquibase.com/code/get-started/env-setup/ to get rid of step 2 and update step 3
Partially fixes https://github.com/liquibase/liquibase/issues/5334 .

also, you'll most likely want to change CI to use the wrapper and document that people can/should use the wrapper (in whichever dev. documentation you currently say that one should manually install maven first - if you even say that anywhere)

Replaced by https://github.com/liquibase/liquibase/pull/5486


Feature request: Right justify value of a key/value pair

Couldn't think of the best way to phrase this request. It would be useful for tools that are showing various values of flags, variables, or other fields, to be able to right-justify the values, so they could get an output like this:
+-------------------------------------+
|IP Address             <IP_ADDRESS> |
|Connection Time           4:32:42 AM |
|Time Remaining    1 hour, 45 minutes |
+-------------------------------------+
|Disconnect Now                   ⌥⌘D |
+-------------------------------------+

This would really help to improve readability, I think, for tools showing output along these lines. Another possibility would instead be to allow showing the values all left justified, but all vertically aligned with one another, like this:
+----------------------------------------+
|IP Address       <IP_ADDRESS>          |
|Connection Time  4:32:42 AM             |
|Time Remaining   1 hour, 45 minutes     |
+----------------------------------------+
|Disconnect Now                       ⌥⌘D|
+----------------------------------------+

hey @shepazon, if you used a fixed width font, could you achieve this by padding with spaces?

hey @shepazon, if you used a fixed width font, could you achieve this by padding with spaces?

@matryer The problem with that idea is that I don't necessarily know the width of the menu, so I can't necessarily get the spacing right just using spaces and a fixed-width font. If the menu were a fixed width all the time, then yes, it could be done. :)

@matryer The problem with that idea is that I don't necessarily know the width of the menu, so I can't necessarily get the spacing right just using spaces and a fixed-width font. If the menu were a fixed width all the time, then yes, it could be done. :)

Thinking it would be interesting to come up with a way to let item titles be attributed strings... not sure how that would work other than possibly by letting you specify them using HTML or RTF and converting, and even that probably wouldn't solve this particular issue.


Added Entity Mention Replacement Transformation

This transformation randomly swaps an entity mention with another entity mention of the same entity type. It can be used as a data augmentation strategy to improve the underlying sequence (NER) model.
Contributor: Usama Yaseen, Siemens AG<EMAIL_ADDRESS>

The build failed due to RandomDeletion tests.

@uyaseen Thanks, all is clear now! Could you please update the readme so it's clear that you gather entitites of the same type from other sentences, and maybe remove the replacement entities from the test JSON? Or would that break the test?

@tuetschek please excuse the delayed response. I have included the example and formatting constraint(s) in the readme. I think the replacement entities should be part of the test JSON as otherwise, tests will break.

Correctness: All checks have passed - please pull "main" once into your branch.

Hi @csinva many thanks for your review!
I have pulled the latest changes from main; strangely RandomDeletion was failing the earlier builds, even though it runs smoothly on my local environment, not sure what's wrong :/. I hope it works this time.

Thanks, looks good to me!

Okay @uyaseen thank you for your changes. I have removed the test cases temporarily. Please recreate a separate PR for that. Thank you very much @mille-s and @csinva for the review.

Okay @uyaseen thank you for your changes. I have removed the test cases temporarily. Please recreate a separate PR for that. Thank you very much @mille-s and @csinva for the review.

@kaustubhdhole Thanks! I have created a separate PR for tests.


chore: updating README.md, adding tests, updating to go 1.23.3 and fixing bugs

…xing some bugs

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Flag
Coverage Δ

unittests
100.00% <100.00%> (+100.00%)
:arrow_up:

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

Files with missing lines
Coverage Δ

main.go
100.00% <100.00%> (+100.00%)
:arrow_up:


BUG. SQLX error[E0658] When building the project

https://github.com/launchbadge/sqlx/blob/e0a1f1633c115bb5653fc7bd54b233c677546b95/sqlx-core/src/raw_sql.rs#L261
error[E0658]: async fn return type cannot contain a projection or Self that references lifetimes from a parent scope
/.cargo/registry/src/github.com-1ecc6299db9ec823/sqlx-core-0.7.4/src/raw_sql.rs:260:10
note: see issue #103532 https://github.com/rust-lang/rust/issues/103532 for more information

The same happening to my project in the new version 0.7.4

Try upgrading rustc.

Fair enough.

To confirm, operational with rustc v1.77.2 & sqlx v0.7.4


Login can not switch another google account

Describe the bug
Amplify.Auth.signInWithWebUI( presentationAnchor: UIApplication.shared.windows.first!)  调用这个方法下次重新登录会展示手动输入账号密码界面 ，从而给用户一次 选择登录方式的机会。但是，调用Amplify.Auth.signInWithWebUI( for: .google, presentationAnchor: UIApplication.shared.windows.first!) 方法指定了google登录就永远不给用户选择其他登录方式的机会了。无论怎么退出登录都不给用户切换登录方式的机会（作为用户当然是希望不管哪种登录都应该在退出后可以有选择其他登录方式的机会）。
假如 用户有2个Google账号，要切换另一个账号，这两种 Amplify.Auth.signInWithWebUI( presentationAnchor: UIApplication.shared.windows.first!)  和 Amplify.Auth.signInWithWebUI( for: .google, presentationAnchor: UIApplication.shared.windows.first!)  调用方法都无法切换Google账号。这就麻烦了。
这是个严重的问题，希望能得到您的帮助，再次感谢。
Steps To Reproduce
Steps to reproduce the behavior:
1. Launching App for ios with aws amplify
2. func performOnAppear() async throws {//这个是iOS端的代码
            do{
                let _ = try await Amplify.Auth.fetchAuthSession()
            }catch let error as AuthError {
                print("Authentication error : \(error)")
                await socialSignInWithWebUI() //google登录
            }
    }

func socialSignInWithWebUI() async {
    do {
        let signInResult = try await Amplify.Auth.signInWithWebUI(for: .google, presentationAnchor: UIApplication.shared.windows.first!)
        if signInResult.isSignedIn {
            print("Sign in succeeded")
        }
    } catch let error as AuthError {
        print("Sign in failed \(error)")
    } catch {
        print("Unexpected error: \(error)")
    }
}

func signOutGlobally() async {
        let result = await Amplify.Auth.signOut(options: .init(globalSignOut: true))
        guard let signOutResult = result as? AWSCognitoSignOutResult
        else {
            print("Signout failed")
            return
        }

        print("Local signout successful: \(signOutResult.signedOutLocally)")
        switch signOutResult {
        case .complete: break
            // handle successful sign out
        case .failed(let error): break
            // handle failed sign out
        case let .partial(revokeTokenError, globalSignOutError, hostedUIError): break
            // handle partial sign out
        }
    }

Expected behavior
我希望Amplify有切换其他账号的功能， Amplify.Auth.signInWithWebUI( for: .google, presentationAnchor: UIApplication.shared.windows.first!) 能切换其他Google账号。
Amplify Framework Version
2.29.1
Amplify Categories
Auth
Dependency manager
Swift PM
Swift version
5.8
CLI version
12+
Xcode version
14.2
Relevant log output
<details>
<summary>Log Messages</summary>

INSERT LOG MESSAGES HERE

```
Is this a regression?
Yes
Regression additional context
No response
Platforms
No response
OS Version
ios 13+
Device
iPhone 8
Specific to simulators
iPhone 14+ iOS15
Additional context
No response

Hi @zhangshouyin. What you are describing is expected, as your app is "sharing" the session with Google.
Logging out of Identity Providers is not supported by Amplify.
If you wish to always show the Google login screen to your users, you can try setting the preferPrivateSession options:
try await Amplify.Auth.signInWithWebUI(
    presentationAnchor: self.view.window!,
    options: .preferPrivateSession()
) {
    ...
}

A similar discussion can be found here: https://github.com/aws-amplify/amplify-swift/issues/3593#issuecomment-2038048857

Hi @zhangshouyin. What you are describing is expected, as your app is "sharing" the session with Google. Logging out of Identity Providers is not supported by Amplify.
If you wish to always show the Google login screen to your users, you can try setting the preferPrivateSession options:
try await Amplify.Auth.signInWithWebUI(
    presentationAnchor: self.view.window!,
    options: .preferPrivateSession()
) {
    ...
}

A similar discussion can be found here: #3593 (comment)

@ruisebas  thank you , I got it.


Error on scanning when a file has no extension

Checking error when a file has no extension
It gives an error when it occurs. Is it possible for the application to handle the error, or at least inform the user? I just found out by accident
These are "Amiga - Commodore" rom files

Thanks for letting me know.
The issue has been fixed in the pre-release version:
https://github.com/jimbob4000/RetroFlow-Launcher/releases/tag/4.0.2


SEGYStream: more generic ReadableByteChannel instead of FileChannel

SEGYParser: now we use a more generic ReadableByteChannel instead of FileChannel
Removed proprietary JAR (sun.generic)
String.format replaced by Formatter (before it was not matching Apache Tika requirements)

I will consider merge this request a bit later because ReadableByteChannel will broke Java 1.6 compatibility, but it's still necessary for legacy applications. I will create separate Java 1.8 version of library to be consistent with existing codebase.

Hi,
thanks for the reply.
Could you please explain me why ReadableByteChannel will break Java 6 compatibility as it was added to Java 1.4?
For the creation of Java 8 version of the library: please note that Sigrun will be referenced by Apache Tika's new SEGYParser (Ticket here: https://issues.apache.org/jira/browse/TIKA-1843).
This parser will be used by our open source search engine Datafari.
So, please could you perform this task ASAP, as it's a blocking issue?
Thanks and best regards,
Giovanni

Yeah, I was confused and thought that it's SeekableByteChannel, sorry. I will merge your request today.
Also, your commit breaks totalSize getter (it's always will show zero). I will remove this getter to not confuse users. And please do not use tabs in your IDE, it breaks indentation in github viewer.

OK,
thanks a lot Mikhail!


Compile-time fixes for Clang 10

Hello,
Would you mind applying these fixes to make the codebase compile with Clang 10 please?
Many thanks!
diff --git a/src/ds/aba.h b/src/ds/aba.h
index 21950ff..60dcc2f 100644
--- a/src/ds/aba.h
+++ b/src/ds/aba.h
@@ -121,6 +121,7 @@ namespace snmalloc
       }

       Cmp(const Cmp&) = delete;
+      Cmp(Cmp&&) = default;
     };

     // This method is used in Verona
diff --git a/src/ds/helpers.h b/src/ds/helpers.h
index b479011..8f5b83c 100644
--- a/src/ds/helpers.h
+++ b/src/ds/helpers.h
@@ -14,7 +14,7 @@ namespace snmalloc
   class Singleton
   {
     inline static std::atomic_flag flag;
-    inline static std::atomic<bool> initialised = false;
+    inline static std::atomic<bool> initialised{};
     inline static Object obj;

   public:
diff --git a/src/ds/mpscq.h b/src/ds/mpscq.h
index d5d5161..a397354 100644
--- a/src/ds/mpscq.h
+++ b/src/ds/mpscq.h
@@ -14,7 +14,7 @@ namespace snmalloc
       std::is_same<decltype(T::next), std::atomic<T*>>::value,
       "T->next must be a std::atomic<T*>");

-    std::atomic<T*> back = nullptr;
+    std::atomic<T*> back{};
     T* front = nullptr;

   public:
@@ -72,10 +72,10 @@ namespace snmalloc
         SNMALLOC_ASSERT(front);
         std::atomic_thread_fence(std::memory_order_acquire);
         invariant();
-        return std::pair(first, true);
+        return std::pair<T*, bool>(first, true);
       }

-      return std::pair(nullptr, false);
+      return std::pair<T*, bool>(nullptr, false);
     }
   };
 } // namespace snmalloc
diff --git a/src/mem/alloc.h b/src/mem/alloc.h
index c58015b..4f8aed8 100644
--- a/src/mem/alloc.h
+++ b/src/mem/alloc.h
@@ -240,6 +240,9 @@ namespace snmalloc
     template<size_t size>
     void dealloc(void* p)
     {
+      if (p == nullptr)
+        return;
+
 #ifdef USE_MALLOC
       UNUSED(size);
       return free(p);
@@ -280,6 +283,9 @@ namespace snmalloc
      */
     SNMALLOC_FAST_PATH void dealloc(void* p, size_t size)
     {
+      if (p == nullptr)
+        return;
+
 #ifdef USE_MALLOC
       UNUSED(size);
       return free(p);
@@ -302,6 +308,9 @@ namespace snmalloc

     SNMALLOC_SLOW_PATH void dealloc_sized_slow(void* p, size_t size)
     {
+      if (p == nullptr)
+        return;
+
       if (size == 0)
         return dealloc(p, 1);

@@ -325,6 +334,9 @@ namespace snmalloc
      */
     SNMALLOC_FAST_PATH void dealloc(void* p)
     {
+      if (p == nullptr)
+        return;
+
 #ifdef USE_MALLOC
       return free(p);
 #else
diff --git a/src/mem/pagemap.h b/src/mem/pagemap.h
index fc3da85..e658cd3 100644
--- a/src/mem/pagemap.h
+++ b/src/mem/pagemap.h
@@ -188,7 +188,7 @@ namespace snmalloc
       {
         PagemapEntry* value = get_node<create_addr>(e, result);
         if (unlikely(!result))
-          return std::pair(nullptr, 0);
+          return std::pair<Leaf*, size_t>(nullptr, 0);

         shift -= BITS_PER_INDEX_LEVEL;
         ix = (static_cast<size_t>(addr) >> shift) & ENTRIES_MASK;
@@ -208,11 +208,11 @@ namespace snmalloc
       Leaf* leaf = reinterpret_cast<Leaf*>(get_node<create_addr>(e, result));

       if (unlikely(!result))
-        return std::pair(nullptr, 0);
+        return std::pair<Leaf*, size_t>(nullptr, 0);

       shift -= BITS_FOR_LEAF;
       ix = (static_cast<size_t>(addr) >> shift) & LEAF_MASK;
-      return std::pair(leaf, ix);
+      return std::pair<Leaf*, size_t>(leaf, ix);
     }

     template<bool create_addr>
diff --git a/src/mem/pooled.h b/src/mem/pooled.h
index 71f8d91..f044a77 100644
--- a/src/mem/pooled.h
+++ b/src/mem/pooled.h
@@ -14,7 +14,7 @@ namespace snmalloc
     friend class MPMCStack;

     /// Used by the pool for chaining together entries when not in use.
-    std::atomic<T*> next = nullptr;
+    std::atomic<T*> next{};
     /// Used by the pool to keep the list of all entries ever created.
     T* list_next;
     std::atomic_flag in_use = ATOMIC_FLAG_INIT;
diff --git a/src/mem/remoteallocator.h b/src/mem/remoteallocator.h
index 4c1f906..6d8ae55 100644
--- a/src/mem/remoteallocator.h
+++ b/src/mem/remoteallocator.h
@@ -19,7 +19,7 @@ namespace snmalloc
     union
     {
       Remote* non_atomic_next;
-      std::atomic<Remote*> next = nullptr;
+      std::atomic<Remote*> next{};
     };

     alloc_id_t allocator_id;
diff --git a/src/mem/slowalloc.h b/src/mem/slowalloc.h
index f2fcba1..0ab1169 100644
--- a/src/mem/slowalloc.h
+++ b/src/mem/slowalloc.h
@@ -63,6 +63,6 @@ namespace snmalloc
    */
   inline SlowAllocator get_slow_allocator()
   {
-    return SlowAllocator{};
+    return {};
   }
 } // namespace snmalloc
diff --git a/src/pal/pal_consts.h b/src/pal/pal_consts.h
index 538f90d..fe9a38d 100644
--- a/src/pal/pal_consts.h
+++ b/src/pal/pal_consts.h
@@ -78,7 +78,7 @@ namespace snmalloc
     /**
      * List of callbacks to notify
      */
-    std::atomic<PalNotificationObject*> callbacks = nullptr;
+    std::atomic<PalNotificationObject*> callbacks{};

   public:
     /**
diff --git a/src/pal/pal_windows.h b/src/pal/pal_windows.h
index ba7fcea..8015827 100644
--- a/src/pal/pal_windows.h
+++ b/src/pal/pal_windows.h
@@ -17,7 +17,7 @@
 #  ifdef NTDDI_WIN10_RS5
 #    if (NTDDI_VERSION >= NTDDI_WIN10_RS5) && \
       (WINVER >= _WIN32_WINNT_WIN10) && !defined(USE_SYSTEMATIC_TESTING)
-#      define PLATFORM_HAS_VIRTUALALLOC2
+//#      define PLATFORM_HAS_VIRTUALALLOC2
 #    endif
 #  endif

Thanks for these. I will look into these later.
I think the free(nullptr) test on Windows can be fixed in a different way.  I assume you were getting an AV on that where it tries to access the page map associated with 0, and fails.

Is the
+      if (p == nullptr)
+        return;
+

Just required to make the compiler happy. Freeing nullptr should work on windows, at least based on my testing. Could you clarify the need here. I would sooner add assumes in the right places for this.

It was causing a runtime crash. caused by a call to ::operator delete(void* p) where the p was nullptr.

Okay. Thanks. Will fix

before c++14,

If expression evaluates to a null pointer value, no destructors are called, and the deallocation function may or may not be called (it's implementation-defined), but the default deallocation functions are guaranteed to do nothing when handed a null pointer.

for delete expression, since c++14:

If expression evaluates to a null pointer value, no destructors are called, and the deallocation function is not called.

For delete functions:

In all cases, if ptr is a null pointer, the standard library deallocation functions do nothing. If the pointer passed to the standard library deallocation function was not obtained from the corresponding standard library allocation function, the behavior is undefined.

After the standard library deallocation function returns, all pointers referring to any part of the deallocated storage become invalid.

for std::free,

If ptr is a null pointer, the function does nothing.

An interesting thing is cppreference seems to have a conflict with iso N4835 on delete expressions, where ISO standards claims:

If the value of the operand of the delete-expression is a null pointer value; it is unspecified whether a deallocation function will be called as described above.

Anyway, delete expression should not be our concern; but I think it is better to let deallocation functions do nothing here to keep consistent with the behavior of standard library.

@aganea was it a call to
::operator delete(void* p, size_t size)

The code path where there is a size assumes it is actually an allocation, but I think calling delete on a T*  where is it nullptr would result in a sized deallocation of nullptr with sizeof(T).

So I didn't need those changes to compile with

Windows with clang-cl
Windows with Clang 10.0.0 with GNU-like command-line
Linux with Clang 10

Could you clarify your build options that require these changes.  Or if there is a branch or something I can use to repro.  I would like to ensure this works for you going forward by adding something to CI, and I can't currently repro the requirement for the changes.

I'm compiling snmalloc as part of LLVM, using clang-cl 10.0, on a "x64 Native Tools Command Prompt for VS 2019" (I think that part is important), with the latest VS2019 16.5.4 and latest Windows Kit 10.0.18362.
A full repro can be found here: https://github.com/mjansson/rpmalloc/issues/150
However after applying D71786, you would need to replace the rpmalloc files by the snmalloc ones, in llvm/lib/Support/CMakeLists.txt:
  set(RPMALLOC_FILES "D:/git/snmalloc/src/override/malloc.cc" "D:/git/snmalloc/src/override/new.cc")

Then follow the instructions in the link above. Please let me know how that goes.

If it is possible to extract the command line for building malloc.cc compilation unit that has the errors that would be super useful.  Otherwise, I will set up a machine to build llvm on Windows tomorrow.

Here it is:
clang-cl.exe  /nologo -TP -DGTEST_HAS_RTTI=0 -DUNICODE -D_CRT_NONSTDC_NO_DEPRECATE -D_CRT_NONSTDC_NO_WARNINGS -D_CRT_SECURE_NO_DEPRECATE -D_CRT_SECURE_NO_WARNINGS -D_HAS_EXCEPTIONS=0 -D_SCL_SECURE_NO_DEPRECATE -D_SCL_SECURE_NO_WARNINGS -D_UNICODE -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS -Ilib\\Support -ID:\\llvm-project\\llvm\\lib\\Support -Iinclude -ID:\\llvm-project\\llvm\\include /GS- /D_ITERATOR_DEBUG_LEVEL=0 -Xclang -O3 -fstrict-aliasing -march=skylake-avx512 -flto=thin -fwhole-program-vtables -fuse-ld=lld /Zc:inline /Zi /Zc:strictStrings /Oi /Zc:rvalueCast /W4  -Wextra -Wno-unused-parameter -Wwrite-strings -Wcast-qual -Wmissing-field-initializers -Wimplicit-fallthrough -Wcovered-switch-default -Wno-noexcept-type -Wdelete-non-virtual-dtor -Wstring-conversion /Gw -flto=thin /MT /O2 /Ob2 /DNDEBUG   -std:c++14 /showIncludes /Folib\\Support\\CMakeFiles\\LLVMSupport.dir\\D_\\git\\snmalloc\\src\\override\\malloc.cc.obj /Fdlib\\Support\\CMakeFiles\\LLVMSupport.dir\\LLVMSupport.pdb -c D:\\git\\snmalloc\\src\\override\\malloc.cc"

I think the majority of the patch you have is due to it being compiled with C++14 rather than C++17.

Yes, probably, I can try with C++17, however that is part of the LLVM community policies. That won't change that on the short term. Do you object supporting clients compiling with C++14?

#182 should make it compile with C++14, but it will have warnings as we use C++17 features. E.g.
c:\\src\\snmalloc\\src\\override/../mem/mediumslab.h(91,10): warning: constexpr if is a C++17 extension
      [-Wc++17-extensions]
      if constexpr (zero_mem == YesZero)

We make quite heavy use of C++17 features because doing so helps with code quality.  Given that we can compile with gcc, clang, and Visual Studio, I don't see a benefit in moving everything to an earlier version of the standard and I do see a cost.
As I recall, LLVM has an option in CMake to build in C++17 mode.  I'd suggest that, for LLVM use, snmalloc be guarded by this flag: don't build it unless you are in C++17 mode.  That would allow people on older platforms to build with the default allocator and people using a compiler from the last 3 or so years to gain the benefit.

Sounds good with me. I'll add a warning to the snmalloc integration into LLVM. Last time I compiled LLVM with C++17 there were a few issues, but I'd rather fix & push them upstream, I hope at some point they will move past C++14 (but that all depends on baseline compiler in the distros they want to support).
Many thanks again for the help!


Add support for running the dashboard as ingress

What does this PR do?
Add support for running the dashboard as ingress
Add documentation for why this may be desired
Add documentation for how to get external ip and loadbalancer ip working in own vm/hardware
Motivation
Add some extra documentation for how to get started with traefik on a bare k8s cluster.
Easier support for ingress access to the dashboard
Implicit documentation of how to use Ingress with attributes (something the traefik docs are quite bad at showing)
More

[x] Yes, I updated the tests accordingly

api.insecure is not linked to port. So this should be put elsewhere.

https://doc.traefik.io/traefik/v3.0/operations/api/#insecure
Well, port and entryPoint is used intermixed in the helm config, but this binds specifically to the traefik entryPoint, and that is configured in ports.traefik. So yes I think it very much belongs here

We have moved all example to EXAMPLES.md, wdyt about add your example in this file ?

Primarily because the main place to read the documentation does not have the examples accessible:
https://artifacthub.io/packages/helm/traefik/traefik

api.insecure is not linked to port. So this should be put elsewhere.

https://doc.traefik.io/traefik/v3.0/operations/api/#insecure Well, port and entryPoint is used intermixed in the helm config, but this binds specifically to the traefik entryPoint, and that is configured in ports.traefik. So yes I think it very much belongs here

Well yes and no ;-)
I would tend to agree with @mloiseleur. api.insecure is not in entrypoints section of traefik configuration and ports tries to stick as much as possible at it  whereas api.insecure deals with routers.

@davhdavh Will you update this PR or should we close it ?


Increase code coverage

Codecov Report

Merging #24 (96b46d7) into main (902dcbb) will increase coverage by 3.54%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #24      +/-   ##
==========================================
+ Coverage   47.85%   51.39%   +3.54%
==========================================
  Files          22       22
  Lines         861      862       +1
==========================================
+ Hits          412      443      +31
+ Misses        444      415      -29
+ Partials        5        4       -1

Files
Coverage Δ

cmd/version.go
100.00% <100.00%> (+78.94%)
:arrow_up:


Rasa X Interactive Learning writes conversation in reverse order

Rasa version: 1.2.5
Rasa X version (if used & relevant): 0.20.2
Operating system (windows, osx, ...): OSX 10.14.5, Chrome 76
Issue: When I use Interactive Learning, the chat appears in reverse order, and the stories are generated in reverse order. See screencast: https://www.dropbox.com/s/jgbx2t2brlg1bz8/RasaX-Error-Reverse-Story.mov?dl=0
Content of domain file (domain.yml) (if relevant):
  - greeting
  - insurance-car
templates:
  utter_start-car:
    - text: What kind of car do you have?
  utter_start-home:
    - text: Great. When was your house built?
  utter_start-business:
    - text: Ok. Is your business incorporated?
  utter_greeting:
    - text: Hi there!
  utter_insurance-type:
    - text: What type of insurance do you need?
actions:
  - utter_greeting
  - utter_insurance-type
  - utter_start-car
  - utter_start-home
  - utter_start-business

Thanks for the issue, @ricwo will get back to you about it soon!
You may find help in the docs and the forum, too 🤗

Hi @intranetchris, thanks for raising this issue. This should have been fixed with release 1.3.0. Would you mind upgrading to that version and reporting if the problem persists?
If this still happens, it would be helpful if you could let me know if this happens when you run rasa X from the command line or using docker-compose. Thanks!

@ricwo We upgraded to 1.3.0 and the behavior is unchanged.
We don't run it from the command line. We're deploying to AWS ECS using your docker image as a starting point.

@intranetchris can you please check the latest release (1.3.2) does in fact fix this?

@amn41 Our dev team has determined that Rasa X 0.20.2 isn't compatible with Rasa 1.3.x. At least in our environment. We reverted to 1.2.8 and the issue is no longer occurring.


[TensorRT-LLM][ERROR] Encountered an error in forward function: [TensorRT-LLM][ERROR] Assertion failed: input_ids: expected 2 dims, provided 1 dims (/app/tensorrt_llm/cpp/tensorrt_llm/runtime/tllmRuntime.cpp:138)

System Info
R760xa
error occurred when running triton server. TensorRT-LLM version 0.7.1 and TritonServer version 23.12
Who can help?
@jasleen
Information

[ ] The official example scripts
[ ] My own modified scripts

Tasks

[ ] An officially supported task in the examples folder (such as GLUE/SQuAD, ...)
[ ] My own task or dataset (give details below)

Reproduction
curl -X POST localhost:8000/v2/models/ensemble/generate -d '{"text_input": "What is machine learning?", "max_tokens": 20, "bad_words": "", "stop_words": "", "pad_id": 2, "end_id": 2}'
Expected behavior
When running inference after launching triton server
It should produce output response
actual behavior
[TensorRT-LLM][WARNING] Step function failed, continuing.
[TensorRT-LLM][ERROR] Encountered an error in forward function: [TensorRT-LLM][ERROR] Assertion failed: input_ids: expected 2 dims, provided 1 dims (/app/tensorrt_llm/cpp/tensorrt_llm/runtime/tllmRuntime.cpp:138)
additional notes
[TensorRT-LLM][WARNING] Step function failed, continuing.
[TensorRT-LLM][ERROR] Encountered an error in forward function: [TensorRT-LLM][ERROR] Assertion failed: input_ids: expected 2 dims, provided 1 dims (/app/tensorrt_llm/cpp/tensorrt_llm/runtime/tllmRuntime.cpp:138)

Hi @JasleenSingh91, I believe this has been fixed in 24.01. Can you try nvcr.io/nvidia/tritonserver:24.01-trtllm-python-py3?

I ran into this too actually. @rmccorm4 ... what version of TensorRT-LLM do we need to build the engines for the provided nvcr.io/nvidia/tritonserver:24.01-trtllm-python-py3 docker container?

Hi @ggsavin ,
For the 24.01 (also tagged as v0.7.2) Triton TRT LLM backend, v0.7.1 of the TRT LLM framework is the officially supported version that should be used for building the engine.
Please let us know if you run into any issues.

@rmccorm4 Thank you! Could you please also let me know where I would be able to look this up (so I could find this out accurately myself)? Is there a support matrix published somewhere?


[Bug]: Passing reveiver comma separated

What happened?
In the past, I used to pass the receivers comma separated as a string, which worked for my SMTP accounts:
Mail::to($receivers)->send(new $class($emaildata));
With laravel-msgraph-mail I got the error message:
HTTP request returned status code 400: {"error":{"code":"ErrorInvalidRecipients","message":"At least one recipient is not valid., Recipient 'xxx@xxx- (truncated...)
To fix this, I pass multiple receivers as an array.
Is this behaviour on purpose?
How to reproduce the bug
Pass the receivers comma separated as a string.
Package Version
1.3
PHP Version
8.1
Laravel Version
10
Which operating systems does with happen with?
Windows
Notes
No response

Thank you for reporting this issue. I understand the confusion, but I need to clarify that this isn't actually a bug - it's working as designed.
The Microsoft Graph API explicitly requires properly formatted individual email addresses and doesn't support comma-separated strings. While this might have worked with SMTP in the past, it was more of an unintended side effect than proper usage.
The correct way to specify multiple recipients is indeed to use an array:
<EMAIL_ADDRESS>'recipient2@domain.com'])->send(new $class($emaildata));

Using an array format ensures proper email validation, matches the Microsoft Graph API expectations, and is more explicit about the intended recipients.
So, I will close this issue for now. Feel free to reopen if you have any questions or need further clarification. Thanks again for bringing this to our attention!


[5.7] Adding RouteRegistrar mixin to Router

REPOST: https://github.com/laravel/framework/pull/26504

I see that we use the @mixin docblock a little in the FW, but it doesn't seem to be an official tag yet. It seems like the Docblock PSR is picking up steam again, maybe we wait until that is finalized?

But it's already supported by the IDE (I'm using PHPStorm) and it's very handy because it shows me all the available methods even when we're accessing other instances given by the __call magic method.

In my opinion it would be better to use @method for the public methods of the RouteRegistrar, as @mixin also adds the protected methods that are definitely not accessible.

The CONS about using @method is going to bloat the classes with docblocks and also you need to maintain it if there is any changes in the base methods (like adding arguments, renaming, typehints ...)


Add test for v1/v2 Annotated discrepancy

It makes sense that model_dump only dumps the actual fields on the model. This test is here to ensure the v1 behavior isn't accidentally introduced again.

Thanks for the contribution


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade" workflow

Codecov Report

Merging #76 (23397ae) into main (4958138) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #76   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            3         3
  Lines           27        26    -1
=========================================
- Hits            27        26    -1

Impacted Files
Coverage Δ

src/backend-stack.ts
100.00% <100.00%> (ø)

src/lambda/process-metrics-function.ts
100.00% <100.00%> (ø)

src/stateful-stack.ts
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Fix header_self_contained test and including ArborX_Config.hpp

I was seeing that we were not taking one of the CUDA-aware MPI paths even though ARBORX_USE_CUDA_AWARE_MPI was defined. This pull request makes sure that every header actually sees ArborX_Config.hpp.
I also noticed that the headers_self_contained test was not working at all. Since we defined ArborX_HEADERS after including the test directory. I added the corresponding tests to the CTest run such that we actually see that they are running. Of course, compiling is sufficient but the overhead appears to be really small.

Thanks! That's fine with me.

Hmm.. For some reason src/details/ArborX_DetailsDistributedSearchTreeImpl.hpp doesn't seem to be detected to require the file.

Hmm.. For some reason src/details/ArborX_DetailsDistributedSearchTreeImpl.hpp doesn't seem to be detected to require the file.

What makes you say that?  I don't think this is true.  The file name was printed before I pushed the fixup and the self containment test failed to compile.

I was testing e525386f343d7b11b4755b0812751e34f6fdc03d and it passed the test even if the header was only included in ArborX.hpp and details/ArborX_DetailsDistributor.hpp.
Sure, that makes sense it is transitively included in details/ArborX_DetailsDistributedSearchTreeImpl.hpp through details/ArborX_DetailsDistributor.hpp.

@dalg24 Do you want to dismiss your review?

@aprokop @Rombur Do you have any objections merging this?

I'm OK with merging it. The only thing I'm curious about is the reason behind @dalg24 's strong rejection of the original approach.


#2484 - Incorporate 'Year of Study' Field into All Offering Names

Added year_of_study to the constraint and excluded the offering type from the constraint conditions(#2567);
Changed blurb according to business confirmation: The old banner should be adapted to display: "The Year of Study will be displayed as part of the offering name. Here's an example of what students will see." and the words just below the Name field should have "Year of Study" removed:

Displaying the Year of Study in the dropdowns:

Added tests for bulk upload, add offering and update offering;

Bulk upload

Add Offering

Update Offering

Observations:

The whole file for education-program-offering.model.ts looks different because of the wrong EOL in the file.
Update-offering-duplication-unique-index.sql has a commit; at the beginning of the file because of an error with the migration: "driverError: error: unsafe use of new value "Scholastic standing" of enum type offering_types". That means that the sims.offering_types was updated but wasn't committed.

Please add the required labels to PR.


Fix commenter model example

Update the commenter example to use the User model
Improve example code formatting

@phpfour Thanks.


[Backport] [2.x] Bump peter-evans/create-pull-request from 5 to 6 (#12724)

Backport of https://github.com/opensearch-project/OpenSearch/pull/12724 to 2.x

@andrross @kotwanikunal sorry folks, may I ask you to reapprove, resolved the conflicts, thanks a lot!


Introductory code for Random Forests.

create a separate sub-repository by the name Random Forest and push the code by name intro_to_random_forest.py or something like that and also write a comment in 9th line about what does n_estimators means (no of trees) as for beginners might not know what it means and i was thinking that we should create a separate branch for each algorithm instead of directly pushing on master. I made the mistake of pushing code directly on master but from now it would be better if we create separate branch for each algorithm. I will update this  in contributing.md

and one more thing in training data add few more cases. Now you have just taken 2 examples only in training data set take atleast  5

@shubham Setia it would be great if you review or suggest changes from here https://github.com/Cybros/Lectures-On-Machine-Learning/pull/3/files.
You will find a green button on the right-hand side.
That's why you can still see the yellow dot in the below-mentioned pic it shows that you haven't reviewed the PR till so far.

Ok, I'll do as you stated.

Please review the PR. I've fulfilled all the requirements.

@ShubhamSetia  if it is fine then I am merging it.


Prevent reordering generated imports by deterministically sorting by alias name

Since the typegen import paths are resolved asynchronously, they add their config to importsMap in a non-deterministic order.  This means that when Object.keys(importsMap) is called, it's not guaranteed that the imports are returned in the same order, which in turn causes the generated file to sometimes re-shuffle imports.  This hampers the ability to check the generated files into version control, since the imports are occasionally re-ordered.
Reproduction

Have more than one file in typegenAutoConfig.sources:
makeSchema({
  ...
  typegenAutoConfig: {
    // at least two source files here
    sources: [
      {
        alias: 'one',
        source: 'path/to/one',
      },
      {
        alias: 'two',
        source: 'path/to/two',
      },
      ...
    ],
  },
})

Run the app multiple times - the generated typegen file will sometimes import one first and sometimes two will be imported first:
// sometimes
import * as one from 'path/to/one'
import * as two from 'path/to/two'

// sometimes
import * as two from 'path/to/two'
import * as one from 'path/to/one'

Fix
This PR sorts the keys of importsMap (the import aliases) so that the ordering will remain deterministic across multiple runs.

Ah, good catch! Thanks for the detailed explanation and PR!

No problem, thanks for the fast merge!


add file

Terraform plan in .

Plan: 2 to add, 0 to change, 0 to destroy.
Terraform used the selected providers to generate the following execution
plan. Resource actions are indicated with the following symbols:
+   create

Terraform will perform the following actions:

  # module.azure_virtual_network.azurerm_virtual_network.main_vnet will be created
+   resource "azurerm_virtual_network" "main_vnet" {
+       address_space       = [
+           "<IP_ADDRESS>/16",
        ]
+       dns_servers         = []
+       guid                = (known after apply)
+       id                  = (known after apply)
+       location            = "eastus"
+       name                = "vnet-mybob220-client-use-prod-005"
+       resource_group_name = "rg-mybob220-client-prod-005"
+       subnet              = (known after apply)
+       tags                = {
+           "CostCenter"  = "03444"
+           "Criticality" = "High"
+           "Entity"      = "JP"
+           "Owner"       = "BobTayara"
+           "env"         = "prod"
+           "stack"       = "mybob220"
        }
    }

  # module.rg.azurerm_resource_group.main_rg will be created
+   resource "azurerm_resource_group" "main_rg" {
+       id       = (known after apply)
+       location = "eastus"
+       name     = "rg-mybob220-client-prod-005"
+       tags     = {
+           "env"   = "prod"
+           "stack" = "mybob220"
        }
    }

Plan: 2 to add, 0 to change, 0 to destroy.

:memo: Plan generated in Terraform Plan #3

📝 Terraform Deployment
→ Resource Changes: 2 to create, 0 to update, 0 to re-create, 0 to delete.
✨ Create
module.azure_virtual_network.azurerm_virtual_network.main_vnet
+ address_space       = [
+     "<IP_ADDRESS>/16",
  ]
+ dns_servers         = []
+ guid                = (known after apply)
+ id                  = (known after apply)
+ location            = "eastus"
+ name                = "vnet-mybob220-client-use-prod-005"
+ resource_group_name = "rg-mybob220-client-prod-005"
+ subnet              = (known after apply)
+ tags                = {
+     "CostCenter"  = "03444"
+     "Criticality" = "High"
+     "Entity"      = "JP"
+     "Owner"       = "BobTayara"
+     "env"         = "prod"
+     "stack"       = "mybob220"
  }

module.rg.azurerm_resource_group.main_rg
+ id       = (known after apply)
+ location = "eastus"
+ name     = "rg-mybob220-client-prod-005"
+ tags     = {
+     "env"   = "prod"
+     "stack" = "mybob220"
  }

Triggered by @ibrt2016, Commit: cd9ad2332113154f87c85a1a7adebfe42408e323


Dismiss Alert When Tapping The Background View

Please add this feature. Thanks.

This is called shouldDismissOnTapOutside in the Objective-C version

@ytakzk add this feature already. but now it's only in the example project, not in the master repo. refer to this [commit]
(https://github.com/vikmeup/SCLAlertView-Swift/commit/59ab8e12e66108a7b428c44f724b80f01882505e)

@chancyWu would you able to make that change and send PR?
I would be great to only have one SCLAlertView.swift file, per your request :+1:

@vikmeup sure, i'll check and send PR later.

Waiting for this to get released...thanks guys!

@chancyWu @vikmeup @jaytrixz I added the code to the master repo too.


Global variables from @types/angular not available when other imports/exports are present

TypeScript Version:  2.0.3
Visual Studio Code Version:  1.5.3
Compilation method: tsc -p .
Code:
https://github.com/joselee/ts-2.0-types-issue
Description:
When I install declaration files using npm install --save @types/angular, VS Code (and tsc) complains about the global variable angular, if there are other import/export statements in the same file. The error given is: TS2686: Identifier 'angular' must be imported from a module.

If there are no imports or exports in the same file where I try to use angular, no complaints:

I tried to work around this issue with import * as angular from 'angular', but this gives the error: TS2307: Cannot find module 'angular'. Without this import, auto-completions work fine in VS code (even though it complains - first image). But once I put this import in, auto-completions stop.

Expected behavior:
Global variables from libraries should be available in all .ts files, even if there are other imports/exports.
Actual behavior:
Global variables are only available when no imports or exports are present. Otherwise, compilation errors occur. Also, adding "types": ["angular"] to tsconfig.json does not help.
Note:
@types/lodash does not seem to be affected the same way. _ can be used and has auto-completions without complaint, even with other imports/exports.

I am also experiencing this.

#8255 is the reason for this.
I think it is actually good behavior because you will indeed not have the module available unless you load its source as a global as well. If you are using modules, you need to commit to them.

@aluanhaddad, I don't believe this is the intended behavior. Reasoning is that other libraries, which also expose global variables (such as lodash and jQuery), don't require you to import anything in order to use them. Angular should behave the same way, because globals should be global.
Here you can see that it is only Angular that shows an error about requiring the import:

Further, I am unable to import the 'angular' module, even if I wanted to commit to using modules:

I've updated the project to include jquery, @types/jquery, lodash, and @types/lodash

@joselee If you specify "moduleResolution": "node", in your tsconfig.json you can write
import * as angular from 'angular';

That said, there are some aspects of how angular.d.ts is written that I find confusing. It beings with
/// <reference types="jquery" /> and contains multiple export as namespace ns clauses. Later on it augments the global jQuery declaration. However the shape of jquery.d.ts is completely different, following the classic, pre-umd TypeScript pattern of declaring a global and an ambient external module. I don't know how these kinds of declarations interoperate.

@aluanhaddad "moduleResolution: "node" allows me to import the 'angular' module. Thanks!
So then, is this an issue with angular.d.ts not exposing globals correctly? Or are we now supposed to explicitly import variables, meaning jQuery and lodash (for example) need to be updated to behave similarly? Either way, they should be made to behave consistently.

@joselee I believe the jQuery definitions need to be updated with
export as namespace jQuery;
export as namespace $;

but I'm not 100% sure. There's a lot of churn in this area. I agree we need a consistent way of doing this, it's causing a lot of headaches.

+1 on this, I'd like to update to TS 2.x but running into this error when doing so

This a duplicate of https://github.com/Microsoft/TypeScript/issues/10178

To get this working until further and better solutions are available, I'm using Webpack with this added to webpack.config.js
    externals: {
        'angular': 'angular',
        'jquery': 'jquery'
    }

I found about it on React & Webpack guide. Even though the guide author said in https://github.com/Microsoft/TypeScript/issues/10178#issuecomment-263638106

The only reason I wrote that in the tutorial is because using externals cuts down on bundle-time. If you use the global React variable without importing, you can't easily switch to modules later on, whereas imports give you the flexibility of using either given your loader.

It still works for now and allows Webpack to bundle TS files without issues when using angular or other globally exposed variables.
My full webpack.config.js
module.exports = {
    entry: './app.ts',
    output: {
        filename: 'app.js',
        path: path.resolve(__dirname, 'build')
    },
    module: {
        rules: [
            {
                test: /\.tsx?$/,
                loader: 'awesome-typescript-loader',
                exclude: /node_modules/,
            }
        ]
    },
    resolve: {
        extensions: ['.tsx', '.ts', '.js']
    },
    devtool: 'inline-source-map',

    externals: {
        'angular': 'angular',
        'jquery': 'jquery'
    }
}


Debris Melts to Debris crash while digging

Latest version of Oxygen Not Included with Spaced Out! DLC- had a random crash while digging on a brand-new colony, the game traced it back to this mod. Here's a relevant portion of the log:
[08:00:40.727] [1] [INFO] Debris Melts to Debris's patched_method SimTemperatureTransfer.DoOreMeltTransition matched in the stack trace, adding to referenced mods list
MissingMethodException: int SimMessages.GetElementIndex(SimHashes)
  at (wrapper dynamic-method) SimTemperatureTransfer.SimTemperatureTransfer.DoOreMeltTransition_Patch1(int)
  at Game.StepTheSim (System.Single dt) [0x0082d] in <6ab6b57553ed449280aa2485c86e9aab>:0
  at Game.UnsafeSim200ms (System.Single dt) [0x000b0] in <6ab6b57553ed449280aa2485c86e9aab>:0
  at Game.SimEveryTick (System.Single dt) [0x00054] in <6ab6b57553ed449280aa2485c86e9aab>:0
  at Game.Update () [0x00082] in <6ab6b57553ed449280aa2485c86e9aab>:0

i also had this exact issue, didn't think to get the stacktrace, but confirming it is an issue. the crash occured as soon as my colony generated though.

This happened to me too. I noticed some CO2 converting to liquid in the "cold area" at the top of the spaced out map where it's around -60C. No idea if it's related, but might be an area to explore?

You have a MissingMethodException. This is because of the recent Sweet Dreams update, which now expanded the element cap to 65000 (from the previous 255). as a result, the new return type of the SimMessages.GetElementIndex method is a ushort; while yours was compiled expecting a byte.
Just give your project a rebuild and it should work.

Wow, Aki fixed my mod! This I did not expect. Love your work.
Rebuilt and uploaded. Let me know if issues persist (once verifying that latest version 1.2 is installed, may need to force update with mod updater)
Appreciate it!

Still crashing (or again)


'Octopress new' broken in 3.0.1: missing config key

Octopress is expecting a source key in the jekyl config, which is not present on initialization.
User receives error as follows:
$ bundle exec octopress new blog -t
New jekyll site installed in path.to/Blog/blog.
ruby/lib/ruby/gems/2.2.0/gems/octopress-3.0.1/lib/octopress/scaffold.rb:21:in `join': no implicit conversion of nil into String (TypeError)
        from ruby/lib/ruby/gems/2.2.0/gems/octopress-3.0.1/lib/octopress/scaffold.rb:21:in `init_path'
        from ruby/lib/ruby/gems/2.2.0/gems/octopress-3.0.1/lib/octopress/scaffold.rb:8:in `initialize'
        from ruby/lib/ruby/gems/2.2.0/gems/octopress-3.0.1/lib/octopress/commands/new.rb:16:in `new'
        from ruby/lib/ruby/gems/2.2.0/gems/octopress-3.0.1/lib/octopress/commands/new.rb:16:in `block (2 levels) in init_with_program'
        from ruby/lib/ruby/gems/2.2.0/gems/mercenary-0.3.5/lib/mercenary/command.rb:220:in `call'
        from ruby/lib/ruby/gems/2.2.0/gems/mercenary-0.3.5/lib/mercenary/command.rb:220:in `block in execute'
        from ruby/lib/ruby/gems/2.2.0/gems/mercenary-0.3.5/lib/mercenary/command.rb:220:in `each'
        from ruby/lib/ruby/gems/2.2.0/gems/mercenary-0.3.5/lib/mercenary/command.rb:220:in `execute'
        from ruby/lib/ruby/gems/2.2.0/gems/mercenary-0.3.5/lib/mercenary/program.rb:42:in `go'
        from ruby/lib/ruby/gems/2.2.0/gems/mercenary-0.3.5/lib/mercenary.rb:19:in `program'
        from ruby/lib/ruby/gems/2.2.0/gems/octopress-3.0.1/bin/octopress:10:in `<top (required)>'
        from ruby/bin/octopress:23:in `load'
        from ruby/bin/octopress:23:in `<main>'

Thanks for the report. I've just released a fix!


⚠ TorrentLeech has degraded performance

In 4b63da4, TorrentLeech ($TL) experienced degraded performance:

HTTP code: 200
Response time: 3578 ms

Resolved: TorrentLeech performance has improved in 5cddc29 after 5 minutes.


Added static required property to Validators class

@alexeagle ?

Was there a question? The validator class lacks a static property called required, as my commit message states.

oh, sorry. he is main maintainer of angular2. I want to get his advice...

This is fine with me as a temporary fix, however the next angular release will overwrite this file and revert your change (see the comment near the top).
We already have an open issue: https://github.com/angular/angular/issues/3175
I'll work on that today so this is fixed more permanently in time for the next release.

Here is the updated file with my pending PR that adds all the missing static members. Want to give it a try and confirm?
https://gist.github.com/alexeagle/52fab280286e1220310b

Hello and thank you for your contribution!
Due to an excessively long queue of pull requests that have become stale, we are "declaring bankruptcy" and closing all PRs opened before May 1, 2016. If you'd still like to merge this code in, please open a new PR that has been merged and rebased with the master branch.
Going forward, we are committing to review or merge all PRs on a regular basis so this bankruptcy will not occur again. We apologize for the incovenience and hope you will continue to contribute to DefinitelyTyped in the future.
Thanks
The TypeScript and DefinitelyTyped teams


Fix uninitialized value $s warning in windows static builds

Fixes: #6826
[extended tests]

Travis red cross appears unrelated.

-1
This isn't the right answer.  There are a number of quotify1(shlib($whatever)), and those will return a quoted empty string where they shouldn't.  The right answer in this case should be to change quotify1(shlib to quotify_l(shlib in windows-makefile.tmpl

No problem, I can change the PR, or did you already have one?

Feel free to go ahead

funny, quotify_l(@{shlib($_)}) did not make a warning, but does not print anything when it should.
I think I need to do my $l = shlib($_); quotify1($l) if defined($l);

okay, next try.

CI is good now.

funny, quotify_l(@{shlib($_)}) did not make a warning, but does not print anything when it should.

Well, @{shlib($_)} expects shlib to return an array reference rather than an array...

It did unfortunately not cherry-pick, I will send a new PR for 1.1.0 in a moment.


[CLOUD-2237] [KEYCLOAK-6399] [CLOUD-1782] Define sso72- image stream and application templates. Also do couple of sso/README.md text updates

[CLOUD-2237] Add definition of sso72-* image stream and application templates
Basic proposed templates sanity testing was performed on local OCP v3.7 instance. All five of the proposed templates seem to be working fine (the underlying RH-SSO 7.2 server instance in pod is running, and able to login as RH-SSO admin via the RH-SSO administrator console).

[KEYCLOAK-6399] sso/README.md updates:

Replace listing of deprecated sso70-* templates with listing of the current sso71-* and sso72-* ones
Restructure the format of the sso/README.md file

[CLOUD-1782] sso.adoc.in sso/README.md Replace the deprecated hardcoded RH-SSO server admin username/password credentials with the currently recommended use of SSO_ADMIN_USERNAME and SSO_ADMIN_PASSWORD environment variables

Signed-off-by: Jan Lieskovsky<EMAIL_ADDRESS>

@rcernich PTAL


Bug/Feature Request: Player Reporting button

In 1.19.2, the new Player Reporting button looks like this with the mod:

So obviously that's a bug, but it would also be nice to be able to toggle the button as well.
See also: https://modrinth.com/mod/nrb

I'm having this issue as well. It's very weird.

When I get around to supporting 1.19.2, should be soon.


Safe crash - Cannot find the property 'X' of undefined

The safe with the issue: https://gnosis-safe.io/app/#/safes/0x6C642caFCbd9d8383250bb25F67aE409147f78b2/balances
Tried others safes and they don't crash so it seems is only that safe that has something wrong
Just entering there will cause the crash, no need to be connected with your wallet to the site
Is on every browser (tried on Chrome, FF, Brave).
Snapshot:

Can't reproduce this error again. Is this still happening or maybe this was fixed while changing other things?

Same was not able to reproduce. Closing until this gets reported again.


Feature Request: Heading styles

As #8 has pointed out, Mardown is one of the most requested features of Evernote official. Evernote does have a similar approach by supporting the below, which is a big step in easy formatting, consistency and clean notes.
* Bullet
* Bullet

1. Number
1. Number

Ctrl+B Bold

Ctrl+I Italics

But it lacks titles support. Either through styles, or shortcut (Ctrl+1, Ctrl+2), or more of a Markdown approach (# Title, ## SubTitle).
Ctrl+1 Heading
Ctrl+2 SubHeading

# Heading
## SubHeading

I can live with the already included formatting shortcuts, with an option for H1, H2, H3 headings. Either this or full blown Markdown achieve the "easy to write without messing around with an editor" approach that I like of Markdown, and easy to read in a consistent manner across all notes.

Thank you for taking the time to suggest the feature : )


build fails on AArch64, Fedora 33

Build for x86-64 only?
[jw@cn05 cratedb]$ ./gradlew compileJava

Task :libs:sql-parser:generateGrammarSource FAILED
Task :libs:pgwire:generateGrammarSource FAILED
Task :libs:shared:compileJava FAILED
Task :libs:guice:compileJava FAILED
Task :libs:es-plugin-classloader:compileJava FAILED
Task :libs:cli:compileJava FAILED

FAILURE: Build completed with 6 failures.
1: Task failed with an exception.

What went wrong:
Execution failed for task ':libs:sql-parser:generateGrammarSource'.

Unable to determine version for JDK located at /home/jw/.gradle/caches/transforms-3/6b50cfadbcfc254743b9d4bc2277cb6e/transformed/linux-16-x64.tar.gz. Reason: Command returned unexpected result code: 126
Error output:
/home/jw/.gradle/caches/transforms-3/6b50cfadbcfc254743b9d4bc2277cb6e/transformed/linux-16-x64.tar.gz/bin/java: /home/jw/.gradle/caches/transforms-3/6b50cfadbcfc254743b9d4bc2277cb6e/transformed/linux-16-x64.tar.gz/bin/java: cannot execute binary file

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.
==============================================================================

2: Task failed with an exception.

What went wrong:
Execution failed for task ':libs:pgwire:generateGrammarSource'.

Unable to determine version for JDK located at /home/jw/.gradle/caches/transforms-3/6b50cfadbcfc254743b9d4bc2277cb6e/transformed/linux-16-x64.tar.gz. Reason: Command returned unexpected result code: 126
Error output:
/home/jw/.gradle/caches/transforms-3/6b50cfadbcfc254743b9d4bc2277cb6e/transformed/linux-16-x64.tar.gz/bin/java: /home/jw/.gradle/caches/transforms-3/6b50cfadbcfc254743b9d4bc2277cb6e/transformed/linux-16-x64.tar.gz/bin/java: cannot execute binary file

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.
==============================================================================

3: Task failed with an exception.

What went wrong:
Execution failed for task ':libs:shared:compileJava'.

Unable to determine version for JDK located at /home/jw/.gradle/caches/transforms-3/6b50cfadbcfc254743b9d4bc2277cb6e/transformed/linux-16-x64.tar.gz. Reason: Command returned unexpected result code: 126
Error output:
/home/jw/.gradle/caches/transforms-3/6b50cfadbcfc254743b9d4bc2277cb6e/transformed/linux-16-x64.tar.gz/bin/java: /home/jw/.gradle/caches/transforms-3/6b50cfadbcfc254743b9d4bc2277cb6e/transformed/linux-16-x64.tar.gz/bin/java: cannot execute binary file

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.
==============================================================================

4: Task failed with an exception.

What went wrong:
Execution failed for task ':libs:es-plugin-classloader:compileJava'.

Unable to determine version for JDK located at /home/jw/.gradle/caches/transforms-3/6b50cfadbcfc254743b9d4bc2277cb6e/transformed/linux-16-x64.tar.gz. Reason: Command returned unexpected result code: 126
Error output:
/home/jw/.gradle/caches/transforms-3/6b50cfadbcfc254743b9d4bc2277cb6e/transformed/linux-16-x64.tar.gz/bin/java: /home/jw/.gradle/caches/transforms-3/6b50cfadbcfc254743b9d4bc2277cb6e/transformed/linux-16-x64.tar.gz/bin/java: cannot execute binary file

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.
==============================================================================

5: Task failed with an exception.

What went wrong:
Execution failed for task ':libs:cli:compileJava'.

Unable to determine version for JDK located at /home/jw/.gradle/caches/transforms-3/6b50cfadbcfc254743b9d4bc2277cb6e/transformed/linux-16-x64.tar.gz. Reason: Command returned unexpected result code: 126
Error output:
/home/jw/.gradle/caches/transforms-3/6b50cfadbcfc254743b9d4bc2277cb6e/transformed/linux-16-x64.tar.gz/bin/java: /home/jw/.gradle/caches/transforms-3/6b50cfadbcfc254743b9d4bc2277cb6e/transformed/linux-16-x64.tar.gz/bin/java: cannot execute binary file

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.
==============================================================================

6: Task failed with an exception.

What went wrong:
Execution failed for task ':libs:guice:compileJava'.

Unable to determine version for JDK located at /home/jw/.gradle/caches/transforms-3/6b50cfadbcfc254743b9d4bc2277cb6e/transformed/linux-16-x64.tar.gz. Reason: Command returned unexpected result code: 126
Error output:
/home/jw/.gradle/caches/transforms-3/6b50cfadbcfc254743b9d4bc2277cb6e/transformed/linux-16-x64.tar.gz/bin/java: /home/jw/.gradle/caches/transforms-3/6b50cfadbcfc254743b9d4bc2277cb6e/transformed/linux-16-x64.tar.gz/bin/java: cannot execute binary file

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.
==============================================================================

Get more help at https://help.gradle.org

Deprecated Gradle features were used in this build, making it incompatible with Gradle 7.0.
Use '--warning-mode all' to show the individual deprecation warnings.
See https://docs.gradle.org/6.8/userguide/command_line_interface.html#sec:command_line_warnings
BUILD FAILED in 5s
13 actionable tasks: 9 executed, 4 up-to-date
[jw@cn05 cratedb]$

See https://github.com/crate/crate/issues/11048#issuecomment-783996173, you'll need to set -Dbundled_jdk_arch=aarch64 so that it picks up the correct JDK

Thanks, that does the trick. However, building 'clean communityEditionDistTar' I get
FAILURE: Build failed with an exception.

What went wrong:
Task 'communityEditionDistTar' not found in root project 'cratedb'.

However, building 'clean communityEditionDistTar' I get

The community edition is gone because the entire code base is Apache 2 licensed again, so we don't have to make that distinction anymore and  the task name is distTar.

[jw@cn05 cratedb]$ ./gradlew --info -Dbundled_jdk_arch=aarch64 build
...

Task :libs:dex:forbiddenApisTest FAILED
Caching disabled for task ':libs:dex:forbiddenApisTest' because:
Build cache is disabled
Task ':libs:dex:forbiddenApisTest' is not up-to-date because:
Task has failed previously.
:libs:dex:forbiddenApisTest (Thread[Execution worker for ':' Thread 7,5,main]) completed. Took 0.009 secs.
...
Task :libs:shared:forbiddenApisMain
Caching disabled for task ':libs:shared:forbiddenApisMain' because:
Build cache is disabled
Task ':libs:shared:forbiddenApisMain' is not up-to-date because:
No history is available.

Scanning classes for violations...
Scanned 47 class file(s) for forbidden API invocations (in 0.60s), 0 error(s).
:libs:shared:forbiddenApisMain (Thread[Execution worker for ':' Thread 43,5,main]) completed. Took 0.613 secs.

FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':libs:dex:forbiddenApisTest'.

No signatures were added to task; use properties 'signatures', 'bundledSignatures', 'signaturesURLs', and/or 'signaturesFiles' to define those!

Try:
Run with --stacktrace option to get the stack trace. Run with --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org

Deprecated Gradle features were used in this build, making it incompatible with Gradle 7.0.
Use '--warning-mode all' to show the individual deprecation warnings.
See https://docs.gradle.org/6.8/userguide/command_line_interface.html#sec:command_line_warnings
BUILD FAILED in 7s
45 actionable tasks: 8 executed, 37 up-to-date
Watching 2091 directories to track changes
[jw@cn05 cratedb]$

We don't really use to build task, please have a look at our docs in https://github.com/crate/crate/blob/master/devs/docs/basics.rst

On '... app:run' I get:
...
Successfully started process 'command '/home/jw/.gradle/caches/transforms-3/5fea77ae35781efc3f703ca603556dc5/transformed/linux-16-aarch64.tar.gz/bin/java''
Exception in thread "main" java.lang.ExceptionInInitializerError
at java.base/jdk.internal.icu.text.NormalizerBase$NFKDMode.getNormalizer2(NormalizerBase.java:305)
at java.base/jdk.internal.icu.text.NormalizerBase.normalize(NormalizerBase.java:457)
at java.base/jdk.internal.icu.text.NormalizerBase.normalize(NormalizerBase.java:461)
at java.base/java.text.Normalizer.normalize(Normalizer.java:159)
at java.base/sun.security.x509.AVA.toRFC2253CanonicalString(AVA.java:987)
at java.base/sun.security.x509.RDN.toRFC2253StringInternal(RDN.java:437)
at java.base/sun.security.x509.RDN.toRFC2253String(RDN.java:417)
at java.base/sun.security.x509.X500Name.getRFC2253CanonicalName(X500Name.java:724)
at java.base/sun.security.x509.X500Name.equals(X500Name.java:416)
at java.base/sun.security.pkcs.PKCS7.getCertificate(PKCS7.java:683)
at java.base/sun.security.pkcs.SignerInfo.getCertificate(SignerInfo.java:269)
at java.base/sun.security.pkcs.SignerInfo.verify(SignerInfo.java:429)
at java.base/sun.security.pkcs.PKCS7.verify(PKCS7.java:567)
at java.base/sun.security.pkcs.PKCS7.verify(PKCS7.java:584)
at java.base/sun.security.pkcs.SignerInfo.getTimestamp(SignerInfo.java:656)
at java.base/sun.security.pkcs.SignerInfo.verify(SignerInfo.java:340)
at java.base/sun.security.pkcs.PKCS7.verify(PKCS7.java:567)
at java.base/sun.security.pkcs.PKCS7.verify(PKCS7.java:584)
at java.base/sun.security.util.SignatureFileVerifier.processImpl(SignatureFileVerifier.java:315)
at java.base/sun.security.util.SignatureFileVerifier.process(SignatureFileVerifier.java:291)
at java.base/java.util.jar.JarVerifier.processEntry(JarVerifier.java:316)
at java.base/java.util.jar.JarVerifier.update(JarVerifier.java:230)
at java.base/java.util.jar.JarFile.initializeVerifier(JarFile.java:758)
at java.base/java.util.jar.JarFile.ensureInitialization(JarFile.java:1029)
at java.base/java.util.jar.JavaUtilJarAccessImpl.ensureInitialization(JavaUtilJarAccessImpl.java:72)
at java.base/jdk.internal.loader.URLClassPath$JarLoader$2.getManifest(URLClassPath.java:873)
at java.base/jdk.internal.loader.BuiltinClassLoader.defineClass(BuiltinClassLoader.java:841)
at java.base/jdk.internal.loader.BuiltinClassLoader.findClassOnClassPathOrNull(BuiltinClassLoader.java:753)
at java.base/jdk.internal.loader.BuiltinClassLoader.loadClassOrNull(BuiltinClassLoader.java:676)
at java.base/jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:634)
at java.base/jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(ClassLoaders.java:182)
at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:519)
at io.crate.types.DataTypes.(DataTypes.java:85)
at org.elasticsearch.common.settings.Setting.simpleString(Setting.java:1186)
at org.elasticsearch.env.Environment.(Environment.java:57)
at org.elasticsearch.node.InternalSettingsPreparer.prepareEnvironment(InternalSettingsPreparer.java:66)
at io.crate.bootstrap.CrateDB.createEnv(CrateDB.java:112)
at org.elasticsearch.cli.EnvironmentAwareCommand.execute(EnvironmentAwareCommand.java:81)
at org.elasticsearch.cli.Command.mainWithoutErrorHandling(Command.java:124)
at org.elasticsearch.cli.Command.main(Command.java:90)
at io.crate.bootstrap.CrateDB.main(CrateDB.java:91)
at io.crate.bootstrap.CrateDB.main(CrateDB.java:84)
Caused by: java.lang.NullPointerException: Cannot invoke "java.io.InputStream.available()" because "is" is null
at java.base/jdk.internal.icu.impl.ICUBinary.getRequiredData(ICUBinary.java:94)
at java.base/jdk.internal.icu.impl.NormalizerImpl.load(NormalizerImpl.java:431)
at java.base/jdk.internal.icu.impl.Norm2AllModes$Norm2AllModesSingleton.(Norm2AllModes.java:274)
at java.base/jdk.internal.icu.impl.Norm2AllModes$NFKCSingleton.(Norm2AllModes.java:290)
at java.base/jdk.internal.icu.impl.Norm2AllModes.getNFKCInstance(Norm2AllModes.java:263)
at java.base/jdk.internal.icu.text.Normalizer2.getNFKDInstance(Normalizer2.java:123)
at java.base/jdk.internal.icu.text.NormalizerBase$NFKDModeImpl.(NormalizerBase.java:181)
... 42 more
:app:run (Thread[Execution worker for ':' Thread 10,5,main]) completed. Took 1.078 secs.
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':app:run'.

Process 'command '/home/jw/.gradle/caches/transforms-3/5fea77ae35781efc3f703ca603556dc5/transformed/linux-16-aarch64.tar.gz/bin/java'' finished with non-zero exit value 1

Try:
Run with --stacktrace option to get the stack trace. Run with --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org

Deprecated Gradle features were used in this build, making it incompatible with Gradle 7.0.
Use '--warning-mode all' to show the individual deprecation warnings.
See https://docs.gradle.org/6.8/userguide/command_line_interface.html#sec:command_line_warnings
BUILD FAILED in 8s
65 actionable tasks: 13 executed, 52 up-to-date
Watching 2016 directories to track changes
(base) [jw@cn05 cratedb]$
'./app/build/install/crate/bin/crate' seems to work.

Which java version are you using?
The run error is related to https://github.com/crate/crate/issues/11102

Java 11 is advised, I read.
Trying Java 16 for 'app:run' fails:

What went wrong:
Execution failed for task ':buildSrc:compileJava'.

java.lang.IllegalAccessError: class org.gradle.internal.compiler.java.ClassNameCollector (in unnamed module @0x2e78b24a) cannot access class com.sun.tools.javac.code.Symbol$TypeSymbol (in module jdk.compiler) because module jdk.compiler does not export com.sun.tools.javac.code to unnamed module @0x2e78b24a

Dear Lutz,
thank you for your report on Java 16. May I ask which version of Gradle you are using now? Support for Gradle 7 is currently still in the making. In particular, issues like https://github.com/gradle/gradle/issues/13774, https://github.com/gradle/gradle/issues/15538 and https://github.com/gradle/gradle/issues/16641 indicate some problems in this space.

Java 11 is advised, I read.

We are constantly trying to keep our documentation updated, but sometimes things are slipping through. May I humbly ask where you read this? Then, I will try to update the relevant spot right away.
With kind regards,
Andreas.

Dear Lutz,
as we want to keep the build process compatible with Java 11 (LTS), we will be investigating the java.lang.ExceptionInInitializerError issue, see also https://github.com/crate/crate/issues/11102#issuecomment-819415099 ff. Thanks again for your reports.
With kind regards,
Andreas.

Closing this one so that we have a single issue to track the :app:run issue with https://github.com/crate/crate/issues/11102

(base) [jw@cn05 cratedb]$ gradle -version

Gradle 6.8.3
Build time:   2021-02-22 16:13:28 UTC
Revision:     9e26b4a9ebb910eaa1b8da8ff8575e514bc61c78
Kotlin:       1.4.20
Groovy:       2.5.12
Ant:          Apache Ant(TM) version 1.10.9 compiled on September 27 2020
JVM:          16 (Red Hat, Inc. 16+36)
OS:           Linux 5.8.6-301.fc33.aarch64 aarch64
(base) [jw@cn05 cratedb]$
Java 11: https://crate.io/docs/crate/tutorials/en/latest/install.html


fix bug in ospf backend script, add neighbor

this has now a good size for a review

add the missing part
Fix a bug in the backend + in the frontend
Tabs: Remove 'OSPF' as we are already on the OSPF diagnostics page
add neighbor to the GUI

@fichtner there is only one backend call left which is not included in the frontend: interface
If this is added, the OSPFv2 diagnostics page should be done.


[WIP] refactor: Centralize all code outputting to stdout

As discussed in #7910
Summary
tbd
Test plan
tbd

Any more thoughts on how to move forward with this? Do we want to make this even more abstract (and in that case how) before we start replacing console and process.stdout, process.stderr? 🙂


about  SATRN

why SATRN training phase is case sensitive while testing phase is case insensitive ？ Why not  case insensitive when you're not training ？

why SATRN training phase is case sensitive while testing phase is case insensitive ？ Why not case insensitive when you're not training ？

We use this setting based on the paper (https://arxiv.org/abs/1910.04396).


updates to fix broken ci jobs

These were tested from another repo to make sure they work in CI.  Fixes include updating package versions based on repo updates, updating the openstack client to fix live migrations, resequencing some package installs, and changing the pyopenssl version to fix an SSL issue.

Build triggered. sha1 is merged.

Build started sha1 is merged.

Build finished.

retest

Build triggered. sha1 is merged.

Build started sha1 is merged.

retest

Build triggered. sha1 is merged.

Build started sha1 is merged.

Build finished.

Build finished.

Build triggered. sha1 is merged.

Build started sha1 is merged.

retest

Build triggered. sha1 is merged.

Build started sha1 is merged.

Build finished.

Build finished.

retest

Build triggered. sha1 is merged.

Build started sha1 is merged.

Build finished.

retest

Build triggered. sha1 is merged.

Build started sha1 is merged.

Build finished.

retest

Build triggered. sha1 is merged.

Build started sha1 is merged.

Build finished.


moved stagingTasks inside of LiveState

This allows for tests and other classes to access stagingTasks.
There is also a new test in here for making sure addTask is called. I'll be adding more tests shortly.

LGTM, feel free to add your final tests and I can merge.

@elingg While working on this I am going to need to refactor most of statusUpdate() I'll send in a separate PR with the new tests and the refactor.

Sounds good. :shipit:


How to run it without docker?

It's tedious to install docker and configure the settings, is there any way to run it directly?
After install lots library missed for django, I meet a problem:
ERRORS:
engine.Task: (auth.E005) The permission codenamed 'view_task' clashes with a builtin permission for model 'engine.Task'.
the script is:
sudo python3 manage.py createsuperuser

Docker is simplest way to build the production server.
Also you can setup the debug environment without docker as described here. Debug server works slower. We haven't described problem in this instruction.

Thanks for quick response.
I have tried with your suggesttion, but found it also needs python3-venv, which was not able to install on my machine because of dependicy problems. So is there any way to run it directly?

Why it needs a special version of a libray? Such like numpy 1.14.2, scipy 1.0.1 .etc
I don't think it's good practice.
Another problem is that it uses VS code to run, is it really necessary?
After serval minutes of searching,I succeed to run it with the script:
python3 manage.py runserver --noreload --nothreading --insecure <IP_ADDRESS>:7000

Why it needs a special version of a libray? Such like numpy 1.14.2, scipy 1.0.1 .etc
I don't think it's good practice.
Another problem is that it uses VS code to run, is it really necessary?
After serval minutes of searching,I succeed to run it with the script:
python3 manage.py runserver --noreload --nothreading --insecure <IP_ADDRESS>:7000

Hello,I read your CSDN article about this,but I met a problem

https://blog.csdn.net/qq_26572229/article/details/83150120
and could you help me?I would be very thankful.

Run the following script:
python3 manage.py runserver --noreload --nothreading --insecure <IP_ADDRESS>:7000
return:
[2021-06-04 06:11:56,566] INFO django.server: "GET / HTTP/1.1" 301 0
INFO - 2021-06-04 06:11:56,566 - basehttp - "GET / HTTP/1.1" 301 0
[2021-06-04 06:11:56,568] INFO django.server: "GET / HTTP/1.1" 301 0
INFO - 2021-06-04 06:11:56,568 - basehttp - "GET / HTTP/1.1" 301 0
[2021-06-04 06:11:56,569] INFO django.server: "GET / HTTP/1.1" 301 0
INFO - 2021-06-04 06:11:56,569 - basehttp - "GET / HTTP/1.1" 301 0
[2021-06-04 06:11:56,570] INFO django.server: "GET / HTTP/1.1" 301 0
INFO - 2021-06-04 06:11:56,570 - basehttp - "GET / HTTP/1.1" 301 0
[2021-06-04 06:11:56,571] INFO django.server: "GET / HTTP/1.1" 301 0
INFO - 2021-06-04 06:11:56,571 - basehttp - "GET / HTTP/1.1" 301 0
[2021-06-04 06:11:56,572] INFO django.server: "GET / HTTP/1.1" 301 0
Why can't I open the web page.


Does this work if DoubleClick wraps a textinput

really interesting question ... admittedly never tried it around a TextInput. Will try and update issue


Clean up samples and fix using block

 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.:x: ganesanarun sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

I addressed all review comments I believe.

Thanks for the cleanup.

You are welcome, I just want to contribute to this project, so I thought let me start with sample clean up first and gradually increase.


Quoting code in code tags looses the language specifier

Describe the problem
Quoting the below in a post loses the cpp language identifier
```cpp
some cpp code here
```

results in:
[quote]
```
some cpp code here
```
[/quote]

See https://forum.arduino.cc/t/automatic-code-highlighting-do-you-want-it-or-not/868319/22 and 23
Suggested correction
Expected behaviour is that language specifier will not be affected when quoting code
Additional context
Additional Reports

https://forum.arduino.cc/t/my-code-is-now-posted-in-monochrome/1117879

Issue checklist

[X] I searched for previous reports in the issue tracker
[X] My report contains all necessary details

A formal bug report has now been submitted for this upstream:
https://meta.discourse.org/t/language-identifier-is-missing-from-fenced-code-block-when-it-is-quoted/327895
This means that no further action is needed from Arduino, since the deficiency is in the Discourse forum framework (or perhaps in one of their dependencies), and the forum will be automatically updated to the version that has the fix once the Discourse developers make one.
Discourse is free open source software, so if any Arduino community members would like to move this forward they are welcome to submit a fix to the relevant codebase. There is information about contributing development work to the Discourse project here:
https://meta.discourse.org/t/contributing-to-discourse-development/3823


[5.1] Added seeJsonKeys and dontSeeJsonKeys to CrawlerTrait

Added methods to the CrawlerTrait which allow you to search
for json keys without caring about content.
This is useful if returned data is long, or the exact value doesn't
really matter.

Minor refactor - squashed to one commit

Seems you could do this easier by just converting the JSON to an array and using array_has?

I'm not sure how that would handle the 'non-strict' version. Strict version would work fine like that.

@taylorotwell Ping on this - 'non-strict' wouldn't work with array_has (from my reading, at least).

Going to hold off on adding this one to core.


Add aggressive caching via C# minification

With this PR, I'd like to suggest a more aggressive caching strategy where the hash is built on minified C# sources. Minification (not uglification) removes extraneous whitespace, comments and region markers from the source code. Since these cannot affect a program's behaviour, indentation/whitespace styling changes and add/removing/updating comments/regions won't constitute as a change that invalidates a cached execution. Otherwise and presently, even a change of a single space to a script or its #load-ed parts will generate a new hash and lead to a cache miss.
The PR uses C# Minifier for minification. It's been released since a year and battle/field-tested in production against thousands of C# programs. C# Minifier is a hand-authored lexer written, ground-up, from the C# specification. It does not use Roslyn so it is extremely lightweight to use and in processing. It only needs to understand the lexical structure and practically makes no heap allocations (no syntax trees). As a result, there should be no noticeable impact on performance. If you're curious, you can try C# Minifier online and in your browser (via Blazor WebAssembly). It shows changes in compression and hash computation in real-time.
This PR maintains 100% backward compatibility, including (with painstaking care) at the public API surface level. It introduces a new enum called CommandScriptCompilationCacheLevel:
public enum CommandScriptCompilationCacheLevel { Default, None, Simple, Aggressive }

Simple represents the present behaviour where any physical change to the source will miss the cache. Aggressive is the goal of this PR that uses minification. Default is defined to be Aggressive. This provides a risk-free approach for the future where should you ever choose to revert, you can simply redirect Default to Simple. The interpretation of Default is done at run-time. If it was done at the level of the enum member (e.g. Default = Aggressive) then a change to the default level could cause binary breaking changes.
I've marked constructors taking a Boolean for caching as obsolete. Since this project is still following the 0.x lineage, we can drop those constructors now or in the future. If you think cache levels is an overkill then we can drop it and maintain just a Boolean flag for caching. Most of the changes in this PR are to maintain the public API compatibility. The actual change to enable caching over minified sources is tiny and limited to the TryCreateHash method of ExecuteScriptCommand.

thanks a lot for the PR - however we have decided that it is not a functionality that we need at the moment - we'd like to stay dependency free on the critical path of script execution, even at the expense of some unnecessary cache-misses.
I find the minifier very interesting though 🙂

I forgot to add that we are considering a plugin system that might potentially be an interesting avenue for this feature.

however we have decided that it is not a functionality that we need at the moment - we'd like to stay dependency free on the critical path of script execution, even at the expense of some unnecessary cache-misses.

Thanks for coming back to me on this PR. I hope you don't take it the wrong way if I express some disappointment about the "we" bits reading like a Jedi council making decisions behind closed doors and just passing down a blunt note of the last and final call. It seems counter-intuitive to the collaboration model of open source projects where you'd expect some discussion in the open. That said, I didn't check in advance if something like this would be welcome so I accept your decision.

I find the minifier very interesting though 🙂

Thanks! 🙂

I forgot to add that we are considering a plugin system that might potentially be an interesting avenue for this feature.

I see, but how does a plug-in system alleviate any concerns about dependencies being on the critical path of script execution?


Plot coloring lost when entering experiment from marker gene search

When entering the experiment page from the gene search page by clicking on a marker gene "k", I get colour plot by "k=3" but the plot is all grey and labels is "not available".
How to replicate:
Search for gene "Lgr6"
First hit from from the gene search results has two links to go to k where gene is a marker of a cluster:
https://www.ebi.ac.uk/gxa/sc/search?symbol=Lgr6&species=Mus musculus
Click on "See cluster 1 for k = 3
Goes to: https://www.ebi.ac.uk/gxa/sc/experiments/E-GEOD-90848/results/tsne?geneId=ENSMUSG00000042793&k=3&clusterId=[1]

This is now broken in a different way, the default for inferred cell types gets on the way:

@lingyun1010 if you don't release on Monday, this is something that could be fixed before a release on public when Alfonso comes back.


Make League Sign-up UI Responsive

End goal is for drivers to be able to sign up using their mobile devices.

I WOULD LIKE TO DO THIS

Hi @nitishapiplani , thank you reaching out. However, Collaboration is only open to community members and not open to the public right now.

Implemented in https://github.com/indianracingcommunity/RLIProject/commit/73df3ac3de442bc4aa17be32f86801d6b6f7082a
Released in v5.1.2


Add black configuration

https://pypi.org/project/black/

Close with 'wont fix', see #110 for the discussion


[DONOTMERGE] Client-side password validation

User is given option to toggle visibility of the password
User is given live feedback on password strength as they enter it
User is given suggestions for how to improve the password strength if it is too weak when the password box is unfocused

https://trello.com/c/Pi2rCjuv/182-improve-password-validation-feedback-for-admin-users

I've intregrated Steph's branch but we've noticed there's a problem with the password confirmation not being enforced.  I'm working on a fix; do not merge this yet.

The password confirmation issue turns out to be pre-existing, so doesn't need to be fixed under the scope of this task (I'll raise a new Trello card for it).  I've made a change in response to Steph's PR feedback; hopefully this will be mergeable now after a quick re-review.


Bindings for macOS

Hello there -- I'd love to be able to use this both on Linux and Mac. Is there anything you'd like from me to make that easier?

the place where I took the nar plugin usage is here
https://github.com/uncomplicate/neanderthal-native/blob/master/neanderthal-mkl/pom.xml
I think we need to add something like this in pom.xml after the linux profile but I have no idea what options to add
<profile>
            <id>Mac OS X</id>
            <activation>
                <os>
                    <name>Mac OS X</name>
                </os>
            </activation>
            <build>
                <plugins>
                    <plugin>
                        <groupId>com.github.maven-nar</groupId>
                        <artifactId>nar-maven-plugin</artifactId>
                        <configuration>
                            <linker>
                                <name>g++</name>
                                <options>
                                    <option>-ffast-math</option>
                                </options>
                                <optionSet>!# add options set that makes sense!</optionSet>
                                <sysLibs>
                                    <sysLib>
                                        <name>graphblas</name>
                                    </sysLib>
                                </sysLibs>
                            </linker>
                            <cpp>
                                <options>
                                    <option>m64</option>
                                </options>
                            </cpp>
                        </configuration>
                    </plugin>
                </plugins>
            </build>
        </profile>

Let me know if this works for you
https://search.maven.org/artifact/com.github.fabianmurariu/graphblas-package/0.1.21-mac/jar