Plugin is vulnerable to CVE-2021-4428 (log4j / log4shell)

Running the commands from https://www.jenkins.io/blog/2021/12/10/log4j2-rce-CVE-2021-44228/ marks Jenkins instances with the bootstraped-multi-test-results-report plugin as vulnerable. Please release a fixed version.

it will be fixed in the next version, as log4j will be upgraded to 2.15.0
https://github.com/web-innovate/bootstraped-multi-test-results-report/pull/100

Upgrade to version 2.16.0

will bump it to 2.17.0 @jpralves

in the next days, 2.2.0 will be released, and it will cover this vulnerability

2.2.1 has just been released, which includes updating usage of log4j to 2.17.0


Update Travis to use Elixir 1.2.0

Checking to see what Travis does with this.

Travis looks pretty happy :)

@kytrinyx that's because no tests ran ;-)

LOL. OK, then. Sheesh.


Missing function tags

It seems like it's not written in es6, hence no arrow functions, therefor we must have the function tag for it to work?

By analyzing the blame information on this pull request, we identified @hramos and @grgmo to be potential reviewers.

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign up at https://code.facebook.com/cla - and if you have received this in error or have any questions, please drop us a line at<EMAIL_ADDRESS>Thanks!

Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!

Thanks!
@facebook-github-bot shipit

Thanks for importing.If you are an FB employee go to Phabricator to review internal test results.


4.1.0 Breaks popper.js in IE11:

4.1.0 breaks tooltips and popovers in IE11:
https://jsbin.com/siqewopovu/edit?html,js,output
In my browser, the tooltips in that JSBin simply degrade to classic tooltips.
When debugging my project in VS 2017 (where I am actually using bootstrap.bundle.min.js), I stop on this error:
Unhandled exception in bootstrap.bundle.min.js:
JavaScript runtime error: Object.keys argument is not an Object
The root exception seems to occur deep in the call stack, in rollupPluginBabelHelpers, in the line var ownKeys = Object.keys(source); below:
function _objectSpread(target) {
  for (var i = 1; i < arguments.length; i++) {
    var source = arguments[i] != null ? arguments[i] : {};
    var ownKeys = Object.keys(source);

    if (typeof Object.getOwnPropertySymbols === 'function') {
      ownKeys = ownKeys.concat(Object.getOwnPropertySymbols(source).filter(function (sym) {
        return Object.getOwnPropertyDescriptor(source, sym).enumerable;
      }));
    }

    ownKeys.forEach(function (key) {
      _defineProperty(target, key, source[key]);
    });
  }

  return target;
}

Simply reverting the script reference to BS v4.0.0 prevents the problem.

I think that this is a know issue and was fixed recently

Sure enough; if I change the reference to v4-dev it works again.  Thought I had tried that in my jsbin tests, but I must have missed it.  Thanks.


"attribute" filter pauses animation

Spread link: https://view.spreadviz.org/?output=94d05c76-0585-43f9-99d8-9e13973ebba0/f97a0171-7836-4f7c-8dc0-d4eeef938d21.json&maps=DK,FR,RU,BE,UK,PT,SE
Browser: Safari, Chrome
Problem
Animation pauses when a filtering attribute is added.
Screenshots
Screenshot 1: "posterior" was added during the animation and it did not move forward.

Screenshot 2: "country.states" was added at the beginning of the animation and it did not start.

Files needed to reproduce
SARS-CoV-2discrete.zip

I could not immediately reproduce, can you check @jpmonettas pls?

Closing, goes against initial design, too deep of changes required


feat: create a atom to display the ens icon + ens name

Feature Request
Describe the Feature Request
Required:

[ ] #71 ( ENS AVATAR )

Create this atom

Describe Preferred Solution

ENS

This atom will receive the ENS icon and the ens name from an address + value.
If don't have an ens icon or name fill with the  and display the 0x...address

ETH Value

The value can be mocked for now.
Can appear in the component or not. So, make it as generic as possible to appear or disappear when we need it. ( Check the additional context, that atom can have or not the value 🫡 )

Properties:

Additional Context
That atom will be part of this organism to check the swap offers:

And also will be part of this organism ConfirmSwapModal.tsx:

Requested in #103 and #118. Closing as duplicated


Rename generate-cloud-provider-conf file, bsc#1145082

Why is this PR needed?
Rename generate-cloud-provider-conf script to generate-cpi-conf.sh,
because of regex in package spec file.
It will include generate-cpi-conf.sh in the package.
Fixes # bsc#1145082
What does this PR do?
It will rename generate-cloud-provider-conf script to generate-cpi-conf.sh,
because of regex in package spec file.

Which regex in package spec file? Can you please elaborate how renaming cloud-provider-conf to cpi-conf.sh fixes the bug?


Update loops-in-progress.md

Added my name to the list as first PR

Merged the commit adding you to the loops in progress list.


Headless Running Pass fs Between Steps

The fs should be passed during steps when HeadlessRunning.
See https://github.com/taybenlor/runno/discussions/188#discussioncomment-2519773

Thanks heaps!

I tried the workaround you suggested but couldn't get that to work so I will just wait for this. Thanks again for the amazing tool.

This should work in v0.4.0 with the new @runno/wasi implementation.


Package renaming

Pull Request Test Coverage Report for Build 67

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 73.913%

Totals

Change from base Build 65:
0.0%

Covered Lines:
17

Relevant Lines:
23

💛  - Coveralls


TypeError at gp_model.add_training_structure(train_struc)

Thanks so much for your nice work, I tried to repeat the codes shown on Google Colab.
Everything works very well, until this step:
# Train the model.
print("Training the GP...")
batch_size = 10  # monitor the MAE after adding this many frames
n_strucs = np.zeros(batch_size)
mb_maes = np.zeros(batch_size)
for m in range(training_size): # 100
  train_struc = training_strucs[m]

  # Add training structure and sparse environments.
  gp_model.add_training_structure(train_struc)
  gp_model.add_all_environments(train_struc)

  if (m + 1) % batch_size == 0: # 10 per iteration
    # Update the sparse GP training coefficients.
    gp_model.update_matrices_QR()

    # Predict on the validation set.
    pred_forces = np.zeros((validation_size, noa, 3))
    for n, test_struc in enumerate(validation_strucs):
      gp_model.predict_SOR(test_struc)
      pred_vals = test_struc.mean_efs[1:-6].reshape(noa, 3)
      pred_forces[n] = pred_vals

    # Calculate and store the MAE.
    batch_no = int((m + 1) / batch_size)
    mae = np.mean(np.abs(validation_forces - pred_forces))
    n_strucs[batch_no - 1] = batch_size * batch_no
    mb_maes[batch_no - 1] = mae
    print("Batch %i MAE: %.2f kcal/mol/A" % (batch_no, made))

Here is the error message.
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-10-c4db7c1083ef> in <module>
      8
      9   # Add training structure and sparse environments.
---> 10   gp_model.add_training_structure(train_struc)
     11   gp_model.add_all_environments(train_struc)
     12

TypeError: add_training_structure(): incompatible function arguments. The following argument types are supported:
    1. (self: flare_pp._C_flare.SparseGP, arg0: flare_pp._C_flare.Structure, arg1: List[int]) -> None

Invoked with: <flare_pp._C_flare.SparseGP object at 0x7ff6d426b130>, <flare_pp._C_flare.Structure object at 0x7ff6e16b7d70>

I actually didn't change any content, do you have ideas about the error?
PS: pip install flare_pp didn't work on my laptop, so I download your code from GitHub, then install manually. Fortunately, no error report while installation, and flare_pp also can be imported very well.
Thank you!

Hi @mnTusi
Thanks for your interest in our code! I think this issue is because of our recent update of code interface, that is not synced to the colab tutorial. We will address it soon.

Dear YuXie,
Good to Know, thanks so much for your reply!
Mengnan

close via #33


Registration

@juliaregistrator register

Error while trying to register: Error in (Julia)Project.toml: Package 'Elliptic2' with UUID: 0b55928f-c416-41db-a823-c7dfe4e8fda4 not found in registry or stdlib

@JuliaRegistrator register

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

Release notes:

## Breaking changes

- blah

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

Also, note the warning: This looks like a new registration that registers version 0.3.3.
Ideally, you should register an initial release with 0.0.1, 0.1.0 or 1.0.0 version numbers
This can be safely ignored. However, if you want to fix this you can do so. Call register() again after making the fix. This will update the Pull request.


⚠ ClamAV has degraded performance

In 4fc6e72, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 67 ms

Resolved: ClamAV performance has improved in 243223c.


Web implementation

Any plans to make this available for web via react-native-web?

Would be much appreciated as none exists

I was able to solve it for web with workarounds here: https://github.com/necolas/react-native-web/issues/795


Is there a way to live-toggle LuaSnip and/or autosnippets?

Hi, new user here loading LuaSnip with lazy.nvim. I'd like to have the ability to toggle snippets and autosnippets on and off without restarting neovim -- though I expect that I won't want to do this often, cases will likely still arise (especially if I aggressively create snippets). I can't find anything on this in the documentation or other issues (though I may have missed something!). The closest I could find was the enable_autosnippets option for the setup, but it seems as if this is something that's only run once. Could I rerun setup to toggle autosnippets without restarting neovim? Is there a similar setting to toggle all snippets? Thanks, and apologies if I missed something obvious in the documentation.

Hi!
You haven't missed anything, toggling autosnippets alone is not "officially" possible.

Could I rerun setup to toggle autosnippets without restarting neovim?

That should not cause any issues :+1:
However, I think we should consider adding a proper api for setting/toggling isolated options.
What you can do, "inofficially", is set require("luasnip").expand_auto to an empty function to disable it. (You'd also have to store the original if it should be possible to re-enable it)

Not sure what exactly your motivation for this feature is, but it sounds a bit like condition might be useful to you? (assuming you disable them in some specific context)

That makes sense, thanks! One thing: you say that toggling autosnippets alone is not officially supported (though your unofficial idea is a great suggestion and I will implement that in my config). Does that mean toggling snippets in general is? I can't find a method for that, but I could've missed something. Or is the best strategy to use the "unofficial method" for that as well?

Ah, no, the alone was supposed to refer to only setting autosnippets as opposed to all config-values😅

Or is the best strategy to use the "unofficial method" for that as well?

Yup, exactly
But you'd have to modify ls.snip_expand instead of expand_auto

Awesome, makes sense. Much appreciated.


Update reading order generator function

This PR removes the "Linear" property check when generating the readingOrder from . This will add all the items listed in the spine spec to the final manifest json.
This fixes the bug when sometimes the TOC contains links that readingOrder does not.

@kristojorg Sorry for the wall of comments! I accidentally pressed the "Start a review" instead of commenting : /


Table alias collision in generated SQL with subquery (MSSQL)

Windows 10, VS 2015, IIS Express, SQL Server Express 2014, EF 7.0.0-rc1-final, ASP.NET 6.0.0-rc1-final
I am seeing incorrect SQL generated with subqueries when the tables involved begin with the same letter.
Models:
public class Blog
{
        public int BlogId { get; set; }

        public string Title { get; set; }

        public ICollection<BlogPost> BlogPosts { get; set; }
}

public class BlogPost
{
        public int BlogPostId { get; set; }

        public int BlogId { get; set; }

        public Blog Blog { get; set; }

        public string PostText { get; set; }
}

FK setup:
    modelBuilder.Entity<Blog>().HasMany(b => b.BlogPosts).WithOne(p => p.Blog);

Query:
   _dbContext.Blogs.Where(b => b.BlogPosts.Count == 0).ToList();

Generated SQL:
SELECT [b].[BlogId], [b].[Title]
FROM [Blog] AS [b]
WHERE (
    SELECT COUNT(*)
    FROM [BlogPost] AS [b]
    WHERE [b].[BlogId] = [b].[BlogId]
) = 0

With test data of 2 blogs, 2 posts for blog 1 and 0 for blog 2, this query incorrectly returns 0 results (should return one result, namely blog 2).  If BlogPost in the subquery is aliased to something other than [b] it works fine, e.g.:
SELECT [b].[BlogId], [b].[Title]
FROM [Blog] AS [b]
WHERE (
    SELECT COUNT(*)
    FROM [BlogPost] AS [bp]
    WHERE [bp].[BlogId] = [b].[BlogId]
) = 0

The issue here is,
For generated subquery, when we create table alias, we use first character of the entity in lower case. If that clashes with user defined alias then error happens. Simple workaround for now would be to use different character like this:
 _dbContext.Blogs.Where(p => p.BlogPosts.Count == 0).ToList();


Latest Agent Version is missing

We have observed in our environment that this agent 9.32.5487.1 has been released, can you add the details regarding it.

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

ID: d5cd9ff6-a28b-8e40-83b5-7ef4fd24db48
Version Independent ID: bb50c22c-c945-0ec9-17bf-6921e87c41a4
Content: What's new in Azure Site Recovery - Azure Site Recovery
Content Source: articles/site-recovery/site-recovery-whats-new.md
Service: site-recovery
GitHub Login: @rayne-wiselman
Microsoft Alias: raynew

Requested an update about the agent software. I'll update the document when the information is available.

Submitted a work item and requested an update about the availability of information for agent version 9.32.5487.1 so that the documentation can be updated.

Update rollup 45 for Azure Site Recovery was published on March 2, 2020. I'll update the What's new in Site Recovery document.

I submitted the fix for this issue and when it merges, this issue will be closed. The updated document will be online within 24 hours.

#please-close


How to modify the scroll Bar in DataTable ?

I was building a table where I particularly want to decrease the width of Scroll Bar Slider of the DataTable , that goes by
 inside my DataTable component. How can I modify it's width and height in particular ?

Hi, currently it's not possible with DataTable because ScrollArea component (the one responsible for scroll bar sliders) is not exposed from DataTable. Could you share why you might want to change size of the bar? (me personally can't even imagine a case)

Hi, currently it's not possible with DataTable because ScrollArea component (the one responsible for scroll bar sliders) is not exposed from DataTable. Could you share why you might want to change size of the bar? (me personally can't even imagine a case)

I was making some data visualizations inside a particular column so the scroll bar overlapped with the graphical depiction of my data and it wasn't giving a good UI look according to my theme.
I resolved the issue by picking up it's class and overwrote it in my global CSS file , issue is solved now ! .
Great library btw !

I resolved the issue by picking up it's class and overwrote it in my global CSS file , issue is solved now ! .

Basically it's not a best choice because, if you would like to update library (lot of improvements come time-to-time) you will need to be careful – as you can see classnames contain hashes (e.g. yn5k1 in ___SSlider_yn5k1_gg_) that might change (without any special changelog note) from update to update.
But I would like not to expose scrollbars apis from datatable for now if global css is now working for you.


[FHL] Create aspect for timedscope wrapping

Attribute for wrapping method into  timedscopes

Suggesting to add some test with usecase demo.

Suggesting to add some test with usecase demo.

Added some unit tests, still hope to remove the need in the xml in the root

using System;

License header missing

Refers to: tests/Extensions/Aspects.UnitTests/ActivityAttributeTests.cs:1 in af1d8c5. [](commit_id = af1d8c5ffee38b0a2d94ae4791cecd9fb44120ef, deletion_comment = False)


Use title for oneOf union strategy

TL;DR - Can we use title for the inlined object generated as part of the oneOf union strategy?

Disclaimer: I am opening my first issue for Smithy.
I recently started using Smithy and primarily generate Open API specifications. For context, I use Redocly for documentation generation. When using Smithy's union type, I noticed that the generated Open API spec using inline types within the oneOf attribute (default strategy). This results in the same display issue as mentioned in this Redocly/redoc issue, but is unrelated w.r.t. problem/solutions (I think). It showcases my desired outcome (display) though.
Glancing through the source code, I arrived at JsonSchemaShapeVisitor. I believe we would add title in this code block. Due to the simple fix (as I see it), I can contribute the change. However, I want to confirm this solution is "correct."

I understand title to be descriptive in nature, but are there other consequences/implications of this attribute? I could not understand more from the Swagger documentation though.
I propose to add the member name as the title. This guarantee uniqueness (within the oneOf) and makes the most sense when reading: "One of foo, bar, or baz" for the following union.union MyUnion {
    foo: String,
    bar: String,
    baz: String,
}

I have not looked at tests yet, but I expect I would add to an existing case or create a title-specific one.
I believe this change should be backwards compatible given the nature of title (related to my first question in this list), but I would like confirmation.

Thanks in advance!

So the benefit of adding the title property is for better doc generation, correct? I don't see how this would be a breaking change, so I'm open to it, and contributions are always welcome!

This guarantee uniqueness (within the oneOf)

How so? As far as I can tell, it's just a documentation kind of property. Are you just saying that adding title will inherently have this effect in Redoc documentation?

So the benefit of adding the title property is for better doc generation, correct? I don't see how this would be a breaking change, so I'm open to it, and contributions are always welcome!

We are on the same page then 😀. I'll get started on it this week then.

This guarantee uniqueness (within the oneOf)

To word it better: I propose using the member name as the title. Selecting the member name guarantees that each title is unique within a oneOf because each member is unique.
I included this comment mostly as rationale/extra defense for why I propose using the member name for the title. A trait could also be used, but I would prefer this to happen automatically as it affects Open API documentation only per my understanding.
Thanks!

@connorjs @mtdowling when looking into this decision, did you explore the possibility to use a discriminator/tag? I am trying to find if my proposal makes sense: https://github.com/awslabs/smithy/issues/1082

I believe this issue focused on a specific use case for generating documentation from an Open API spec originally written in Smithy. Therefore, I think the discriminator issue, which involves actual modeling, stands apart from this documentation-focused issue.


couldscrapper.get is returning js challenge page instead of expected page

Link to example: https://repl.it/repls/ExpensiveMortifiedDevices
Running it prints the js challenge page instead of expected page

I just tried it, and it prints errorType: 1 for me. We have problems with this module too, we use bittrex-market library to connect to Bittrex, an that library uses this one. Last few weeks it almost never connects, only like once out of 10 times it works.

It prints errorType 1 most of the time and other times it prints the challenge page

@namannehra thanks for reporting this. I tried fetching that url 50 times in a row but it still works fine for me and returns an expected html.
@n3xtgen error type 1 means that cloudflare decided to return a page with recaptcha and unfortunately, this is not something this library can help with

Just tried that script few times now again and in 20 tries it printed
10 times errorType: 1
3 times challenge page
7 times actual page
In bittrex-market module cloudscraper is used like this:
cloudscraper.get('https://bittrex.com/', (error, response, body) => {
    if(error) {
        throw "Failed to circumvent cloudflare protection: " + error
    }

    this._cloudflareUserAgent = response.request.headers['User-Agent']
    this._cloudflareCookies = response.request.headers['cookie']
    this._cloudflareBypassCallbacksWaiting.forEach(savedCallback => {
        savedCallback()
    })
})

I used current version of cloudscraper. It always finishes without error but the response.request.headers['cookie'] includes only the __cfduid=d15d3fd866010565a592507ea1c4a26151523666058 cookie and not the other one cf_clearance=d85d50944325c3d909e42ff4e5ee52dda448dc51-1523668287-10800 which is required for successive request. And the body of cloudscraper result is:
bittrex-cf-challenge.txt
From like 30 tries now, I got that result 25 times, 4 times it returned errorType: 1 and just once it went through...
I even tried to raise "Timeout" in cloudscraper to 15 seconds but it didn't help.

Ok now I tried to use a anonymous window in Firefox a few times and it really shows at least 2 successive pages of cloudflare challenge most of the time, sometimes even more, so we need mechanism that will solve whatever number of challenges until returning required page..

This was fixed in 2.0.0. in 3.0.0 challengesToSolve will be exposed


Refactor trip table code

https://github.com/dabreegster/abstreet/blob/master/game/src/sandbox/dashboards/trip_table.rs
https://github.com/dabreegster/abstreet/blob/master/game/src/sandbox/dashboards/parking_overhead.rs
These share lots of duplicated code. I don't know what the ideal API to construct a table should be -- declare the columns, whether they're sortable, provide all the data in a reasonably type-safe way, etc. A bit complicated by the checkbox filters that're different between the two.
Any ideas? Or an existing crate for heterogeneously typed sortable tables?

I will take a look at this.

Yuwen has some new designs that make the requirements for the table a little more intricate. I'm going to attempt something. @NickSchimek if you made any progress on this, mind sending the branch? Thanks!

I have not been able to make progress on this @dabreegster. I tried some new things, but they were not working out. I had another plan to try, but it got side tracked and I haven't made my way back around to this issue.
Please go ahead and attempt something.


DRILL-5961: For long running queries (> 10 min) Drill may raise FragmentSetupException for completed/cancelled fragments

@parthchandra or @adityakishore since you'll have made changes to these files before, can you review the changes here?


Add Raw IMU and RC in messages

Thanks for the submission.

Why do you have both integrated and averaged quantities in the IMU message? Average can be deduced from integrated if needed.

You stated that the quantities of angular velocity and acceleration are in SI units, which is correct. Yet the fields are suffixed with _raw, why?

What is the exact purpose of the RC message? We already have actuator and ESC setpoint messages.

Why do you have both integrated and averaged quantities in the IMU message? Average can be deduced from integrated if needed.
*_raw message does not represent averaged quatity but latest point sample. The usage could be for accurate sensor calibration and if someone's AHRS requires point samples instead of intergrals.

You stated that the quantities of angular velocity and acceleration are in SI units, which is correct. Yet the fields are suffixed with _raw, why?
I felt that since the samples are point i.e. unprocessed except unit transformation it is justified to use that term.

What is the exact purpose of the RC message? We already have actuator and ESC setpoint messages.
Its an RC in message i.e. message received from radio transmitter. I found a requirement of such a message when I am working with a redundant autopilot system, where all the autopilots are supplied with info like RC and IMU on CAN bus.

*_raw message does not represent averaged quatity but latest point sample. The usage could be for accurate sensor calibration and if someone's AHRS requires point samples instead of intergrals.

OK.

You stated that the quantities of angular velocity and acceleration are in SI units, which is correct. Yet the fields are suffixed with _raw, why?

I felt that since the samples are point i.e. unprocessed except unit transformation it is justified to use that term.

I understand your intention, but I don't think "raw" is the correct word to use here. Let's call it either:

point
last
no suffix at all
I'm open to further suggestions

It probably would be beneficial to have fields for error covariance estimates for gyro and accel samples, as this information can be efficiently used by state estimators. Do you agree?

What is the exact purpose of the RC message? We already have actuator and ESC setpoint messages.

Its an RC in message i.e. message received from radio transmitter. I found a requirement of such a message when I am working with a redundant autopilot system, where all the autopilots are supplied with info like RC and IMU on CAN bus.

Having a message carrying a set of arbitrary control inputs could be useful, but there is no reason to constrain it to RC only. For instance, it could be used to provide control input from a companion computer. What I propose is to rename the message from Radioin (should have been RadioIn anyway) into something more generic, i.e. ControlInput or something of that effect. The namespace should be renamed accordingly.
I am open to other suggestions.

@pavel-kirienko I've made the said changes and some additions. Please review.

@bugobliterator nudge!

@pavel-kirienko Sorry, I lost track of this PR. I have added the changes you requested. Feel free to pull it in.

Superseded by https://github.com/UAVCAN/dsdl/pull/27


Improve nested loop join code

Which issue does this PR close?

Closes #.
Rationale for this change

What changes are included in this PR?

We could use ? to make code cleaner.
Are these changes tested?

Are there any user-facing changes?

Yes, less match pattern.


Filter flags are not working --older-than , --newer-than

Describe the bug
The filter options doesn't seem to work, no matter what pass using one of these 2 , I always get the same (all) resources --older-than , --newer-than
To Reproduce
./kor.exe replicaset --include-namespaces testing --show-reason --older-than=1000h

Expected behavior
show only the older replicasets
OS version, architecture and kor version
Windows

in replicaset.go filters.run() is missing the SetObject() method.
https://github.com/yonahd/kor/blob/da9b3c2d5788c40929934e00bc2927e52a760558/pkg/kor/replicaset.go#L25
example where its used corrrectly:
https://github.com/yonahd/kor/blob/da9b3c2d5788c40929934e00bc2927e52a760558/pkg/kor/hpas.go#L60
I also recommend doing a quick check for other resources that might be missing it.


allow a protected map to be viewed by anyone with the bookmark

This is like the google docs "share with anyone who has the link".
Requested at Stuart lab presentation 2/28/2018.

Implementation: when the viewer loads from a bookmark, the auth module could be notified that any user may view this map. The mapAuth API call would be unnecessary in this case.

a tiny url would be good to implement.

implemented


Add default parse method for Decodable collections

We added a default implementation for parse if the response type was
Decodable, but weren't able to do the same for collections of decodable
types, because of how Argo implemented the Array extension. With
thoughtbot/Argo#243, we can actually extend Request to automatically parse
ResponseTypes that are CollectionTypes whose elements are themselves
Decodable.
We should point Argo back at master once that PR gets merged.

Lookin' gooooood


removeOnDestroy not working in IE11

CodePen demo
http://codepen.io/anon/pen/jAjamG
Steps to reproduce the problem

Open in IE11, look in console

What is the expected behavior?
Popper DOM element is removed.
What went wrong?
Error in console, Popper DOM element is NOT removed.

https://github.com/FezVrasta/popper.js#internet-explorer


How do I mimic GeoImage.CopyColorTable in gippy 1.0?

We're trying to migrate gips from gippy 0.3 to 1.0, and it's not clear how to duplicate GeoImage.CopyColorTable.  I can almost mimic it by looping over the bands and doing GeoRaster.add_colortable, but I can't figure out how to get the colortable out of the object I want to copy from.  I think GeoRaster._GDALDataSet.GetColorTable() (or similar) would do it, if outside code could call that.
Is there a way to do it that I can't see, or would gippy need to be extended somehow to do this?

Hello @ags-tolson
When developing gippy 1.0 I ended up removing some functions that weren't used internally by other functions or by algorithms, or that were replaced by other things.  I'll review this and if it can't be done currently I'll add the feature back into develop.
Don't hesitate to post any other missing functionality here you come across during the migration, some of it might just need to be features added back in which can be bundled into either a 1.0.1 or 1.1 (if more substantial) release.

That'd be great, thank you!  Overall it's working pretty well for us in terms of doing what gips needs to do; modis is already pretty much converted and landsat is in progress.  Mostly I expect to pester you about these little corner cases.
If you're curious about the details, here's a diff which should contineu to update:
https://github.com/Applied-GeoSolutions/gips/compare/dev...376-gippy-update


circleci: store built binary as an artifact

It might be convenient to store the resulting binary of a PR as an artifact. This way, people that want to test/try a PR can just download the binary from CircleCI instead of checking out and rebuilding locally.
Another use case is prombench. For instance, we can't really evaluate the benefits of using go1.11 (cf. https://github.com/prometheus/prometheus/pull/4626) as prombench rebuilds the binary from sources and it currently uses go1.10.2. Instead it could use the artifact of the PR to build the container image.
Finally this is already related to https://github.com/prometheus/prometheus/pull/4453 which aims at building Docker images for every PR but this PR has a much smaller scope.
cc @SuperQ @krasi-georgiev

What about storing the output of the build step as well, that way we get all architecture artifacts.

I just wanted to test the water but yes, it makes sense too.

Also, maybe promtool?

Yup, this is a great idea. We should do this everywhere.

Here's an artifact output. Looks good!


[DOC] Stepper modes are functions

Insert the documentation link:
https://github.com/Luos-io/Documentation/blame/master/src/pages/high/modules_list/stepper.md#L36-L39
Describe the issue or the suggestion:
The 4 modes are documented as variables, but in Pyluos 1.2.3 they're functions.
How would you fix the issue / which content would you add or delete?
Either:

the doc is wrong and should list them as functions with their boolean parameter ; or
the doc is right and modes should be converted into variables in Pyluos

Knowing that in all other modules it is frequent to activate something with variables e.g. controlled_motor.rot_speed_mode = True, the second option looks preferable to me.

Indeed, it sounds like a mistake on the pyluos code side.
We have to update pyluos about it.

@nicolas-rabault, is this issue still open?

The stepper example is now a standard servomotor from Luos 1.3.0, so we have to remove the stepper page related to on pyluos and remove this from pyluos too. So this issue doesn't exist anymore but we still have another problem. We should close this one and open another.

We have the same problem with dxl...

There is no more demo board on the documentation


Reduce CPU time spent in ingest path

Currently, a major amount of CPU time is spent in the ingestion path of Parca/arcticdb, and garbage collection.
https://pprof.me/999a52c/
There are a couple of things we can and should try:

Set parquet.ColumnBufferCapacity of a parquet writer to the maximum amount that we may write to the buffer during ingestion. I suspect we are wildly over-allocating when splitting data.
Tune parquet.PageBufferSize. Same reason as above. We can probably have arcticDB auto-tune itself here based on previous write sizes.
Reuse parquet.Writer. When done with a writer we can call .Reset on it and reuse it to write the next buffer. If the above two don't help, this is probably the last resort. This one is also more complicated as we can't reuse writers for different schemas, so we would need to keep a record of the schemas and create a sync.Pool per schema that we see. I suspect this will eventually have to be done to get GC time under control (we'll need to measure whether the saving in CPU time of GC is worth it, as we'd be spending time on resetting the writers).

FrostDB uses a lot of OpenFile for reading row in memory. I think https://github.com/segmentio/parquet-go/pull/255 will helps.
I went for the pool per schema/dynamic columns as I think over allocating is fine as long as you can reuse.
So far this what I have.
❯ benchstat before.txt after.txt
name                                       old time/op    new time/op    delta
_Table_Insert_10Rows_10Iters_10Writers-16     460ms ±14%     586ms ±12%  +27.44%  (p=0.008 n=5+5)

name                                       old alloc/op   new alloc/op   delta
_Table_Insert_10Rows_10Iters_10Writers-16     794MB ±30%     293MB ±59%  -63.15%  (p=0.008 n=5+5)

name                                       old allocs/op  new allocs/op  delta
_Table_Insert_10Rows_10Iters_10Writers-16     1.49M ±52%     0.76M ±24%  -49.11%  (p=0.016 n=5+5)

I'm still playing around though.

Nice! We should also be able to re-use writers and use a sync.Pool on a per schema basis for those as well. I think that will optimize away another fair bit.

That's what I actually do.

Closing as I believe this has been addressed with many changes, notably the new index.


AddDynamicSerilog do not inject IDiagnosticContext

Describe the bug
.AddDynamicSerilog() extension method do not inject IDiagnosticContext. That is a difference with .UseSerilog() method and  causes IApplicationBuilder.UseSerilogRequestLogging() (Serilog.AspNetCore) method to throw exception because no IDiagnosticContext is found.
Steps to reproduce
Steps to reproduce the behavior:

Create a basic aspnetcore microservice
Add Steeltoe.Extensions.Logging.DynamicSerilogBase and Serilog.AspNetCore packages
Use Steeltoe Autoconfig or .AddDynamicSerilog in Program.cs:

return builder
                //.AddDynamicSerilog((builderContext, loggerConfig) =>
                //{
                //    // Read Serilog configuration
                //    loggerConfig.ReadFrom.Configuration(builderContext.Configuration);
                //})
                .ConfigureWebHostDefaults(webBuilder => webBuilder.UseStartup<Startup>())
                .AddSteeltoe(loggerFactory: initLoggerFactory);

Add Serilog request logging middleware in Startup.cs:

public void Configure(IApplicationBuilder app, IHostEnvironment env)
{
    app.UseSerilogRequestLogging();
}

Expected behavior
AddDynamicSerilog should inject IDiagnosticContext. In general I think that AddDynamicSerilog should emulate UseSerilog logic as much as possible.
Environment (please complete the following information):

Steeltoe 3.1.0
OS: Windows
.NET Core 3.1.0
Serilog.AspNetCore 4.1.0

Additional content

Serilog.AspNetCore IDiagnosticContext creation

@DaviGia I looked at this a little. I am not sure we can support this in our repo. The best thing to do might be to add this yourself - something like this:
                var diagnosticContext = new Serilog.Extensions.Hosting.DiagnosticContext(null);

                // Consumed by e.g. middleware
                builder.Services.AddSingleton(diagnosticContext);

                // Consumed by user code
                builder.Services.AddSingleton<IDiagnosticContext>(diagnosticContext);

This would have to be tested. If you can think of a way to do this in steeltoe without taking further dependencies, please submit a PR.

You could consider asking serilog to make this method public.

This would have to be tested. If you can think of a way to do this in steeltoe without taking further dependencies, please submit a PR.

My temporary fix to make it work was pretty much identical to your proposal, i.e.:
//# Startup.cs ConfigureServices
// Attach DiagnosticContext for SerilogRequestLoggingMiddleware
var diagnosticContext = new DiagnosticContext(null);
services.AddSingleton(diagnosticContext);
services.AddSingleton<IDiagnosticContext>(diagnosticContext);

So I can confirm that it works just fine.

You could consider asking serilog to make this method public.

I will consider it. Thank you for now.

Closing as there is no work on Steeltoe side, need to address this in Serilog or use workaround.


test: test LTS compatibility

http:1.41.5
guava:31.1-android
autovalue:1.9
auto-value-annotation:1.9

@Neenu1995 i think this is no longer needed


Example don't work

Hey :)
I just downloaded your code trying to run it, but it doesn't work on my Firefox 10, as I supposed.
I think it depends on Mozilla's IndexedDb last changing specs, so I opened a topic about this issue, that you can read here: https://support.mozilla.org/it/questions/919121
Best regards,
Mathew.

What exactly is going wrong? Could you post any errors that appear in the console? Or describe the steps to reproduce your issue?
I have updated IDBWrapper these days to work with FF's current implementation of IndexedDB, so I'm really curious to see what's happening.
Thanks!

Hey, the error is the same in every example and is the following one:
*The operation failed for reasons unrelated to the database itself and not
covered by any other error code.
openRequest = this.idb.open(this.dbName, this.dbVersion,
this.dbDescription);*
I already reported this issue here:
https://support.mozilla.org/it/questions/919121
Hoping to hear you soon, Mathew.
On Fri, Mar 2, 2012 at 6:42 PM, Jens Arps <
<EMAIL_ADDRESS>
wrote:

What exactly is going wrong? Could you post any errors that appear in the
console? Or describe the steps to reproduce your issue?
I have updated IDBWrapper these days to work with FF's current
implementation of IndexedDB, so I'm really curious to see what's happening.
Thanks!

Reply to this email directly or view it on GitHub:
https://github.com/jensarps/IDBWrapper/issues/4#issuecomment-4290385

This error message is probably an indicator that you're running the examples locally, i.e. from a file:// protocol instead of http://. This is a Firefox security restriction and not likely to change soon.
If this is the case and you're running the examples from file, could you please run them from a webserver and see if they work there?
Thanks!

Hi gsome news about this issue. The point is that the old way to use
IndexedDB works with Firefox9- and doesn't work with Firefox's latest
version. On the other hand the new way to use IndexedDB doesn't work with
Firefox9- but it does with Firefox10 (the latest version). What a mess.

www.mathewdesign.com/EN http://www.mathewdesign.com/
On Tue, Mar 6, 2012 at 11:47 AM, Mathew Durden<EMAIL_ADDRESS>
I already heard about those Firefox restriction, but I'm running the
example on Apache webserver so the url is something like:
http://localhost/example.html
The strange thing is that everything was working fine before Firefox 10.
I've found a solution but I've no idea what are the main differences
betwen this code and the old one. l'll attach you a code snippet that I'm
currently using to make this stuff work.
Creepy things happens from line 22 : var db;(working_stuff_here)
Let me know if it helps. Mathew.

www.mathewdesign.com/EN http://www.mathewdesign.com
On Tue, Mar 6, 2012 at 10:51 AM, Jens Arps <
<EMAIL_ADDRESS>
wrote:

This error message is probably an indicator that you're running the
examples locally, i.e. from a file:// protocol instead of http://. This
is a Firefox security restriction and not likely to change soon.
If this is the case and you're running the examples from file, could you
please run them from a webserver and see if they work there?
Thanks!

Reply to this email directly or view it on GitHub:
https://github.com/jensarps/IDBWrapper/issues/4#issuecomment-4341528

Hi Mathew,
first, I didn't get that email you're quoting, nor does it show up here on the github issue tracker, maybe because you were adding an attachment?
If you're running the examples from a webserver, than I have no idea what's happening – or how to fix it. If you navigate to http://jensarps.github.com/IDBWrapper/example/quicktest/ and try the quicktest there, do you still get the same error?
Regarding Firefox 9: IDBWrapper works fine on FF9 as well, there is a check in the code that finds out whether the current browser is a FF w/ the new versioning API or not. So, no worries about that :)


pull embed.js into multiplex-templates

multiplex-templates should handle all the embed filters, mixins, etc that need to reach up and call multiplex-templates's render() method. That'll get rid of the dependency in this repo and let it keep doing its thing and being awesome.

reference https://github.com/nymag/multiplex-templates/issues/3

removed embed.js from this repo


Feature: Have slick version accessible through code

I'm working on a website plugin where I need to load jQuery if it's not loaded already or the version loaded is too old. jQuery provides: $.fn.jquery which outputs jquery version. Would be nice to have something like that for slick: $.fn.slick.version?
This way I wouldn't have to assume the slick version loaded might be outdated and load a new version every time, but I could safely reuse existing slick version if there is one and I can confirm that it has the functionalities I need.

This is still an issue for me. Would you accept the PR if I made one? I see there are currently 129 opened PRs.


Pkg: Fix disabled state on dropdown and buttons on the Toolbar

Problems

When the dropdown have the parameter disabled it gets a second border
The small buttons with icons get also a border the parameter disabled is added

Now fixed in #1158


ci fails on windows to build aws-lc-sys

the issue is reported for Apple Silicon M2 at:
[https://github.com/aws/aws-lc-rs/issues/342]
It works on a local machine meanwhile.

It seems that stdio.h is found in Windows SDK no in Visual Studio include dirs , which is installed "along" Visual Studio on the local Machine: Need to install a Windows SDK in CI

fixed by upgrading ci.yaml


fix: Remove deprecation warning from docker-compose.yml

Motivation
Newer versions of Docker Compose warn about the version property now being deprecated.
Change Summary
Remove it so the bots are happy.
Merge Checklist

[x] The PR title adheres to the conventional commits standard
[x] The PR has been tagged with change type label(s) (i.e. documentation, feature, bugfix, or chore)
[x] The PR's changes adhere to all the requirements in the contribution guidelines
[x] All commits have been signed

PR-Codex overview
This PR updates the docker-compose.yml file to set a specific Dockerfile for the build process.
Detailed summary

Updated the Dockerfile used for building to Dockerfile.foundry

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Merging since failures are unrelated to this change, and we just want to fix this for local dev when working with other repos.


Bregma definition

Hi,
thx for the nice tool!
I have a question related to the Bregma definition you are using in HERBS.  How did you define the Bregma position and could you elaborate why it differers from the definition used by e.g. Neuropixels trajectory explorer ?
For the rat atlas you define the Bregma to be 653 AP, while NP traj at 635, which seems to fit better to the position of the anterior commisure.
Best,
Artur

Hi,
Bregma is not something I decide. For the Waxholm Rat atlas, you can find the Bregma coordinates on their webpage. For Allen CCF, as far as I know, there is no Bregma coordinates is specified. So in the atlas processor in HERBS, I leave Bregma to 0, 0, 0 and you can specified a suitable Bregma for it. If you leave it as (0, 0, 0), HERBS will take the center of AP and ML direction as pseudo Bregma (well, it is not 'Bregma', but HERBS needs somewhere as the origin.) https://community.brain-map.org/t/how-to-transform-ccf-x-y-z-coordinates-into-stereotactic-coordinates/1858
Here says that the Bregma is estimated at (540, 44, 570). If you want, I can add it later to HERBS. But I need to finish something else first then come back to this.

Or this https://community.brain-map.org/t/how-to-switch-between-the-3-coordinate-systems-of-mouse-connectivity/952/2

Hi Artur,
Thank you for the clear explanation. I can open a possibility(a input area) for Bregma so that users can decide what Bregma coordinates they want to use. what do you think?
Perhaps you already know form other issues that I do not have a neuroscience background so I am not allowed to change the values without discussion. I would like to suggest you to contact Waxholm Rat Atlas<EMAIL_ADDRESS>about the Bregma as it is important.
Wish you a happy holiday season and see you in 2023!
Best,
Jingyi

Hi,
I have also observed some discrepancies in coordinates for certain structures, and this might very well be the explanation! I think it would be great to have a feature that lets the user decide bregma coordinates and also rotate the atlas/coordinates by an arbitrary angle (as shown in the figure above).
Best,
Abraham

Hi Jingyi,
I would also greatly appreciate if you could implement a way for the user to select Bregma and transform the coordinates accordingly. Or is this already implemented? In the cookbook a 'Bregma Picker' is mentioned, but that doesn't seem to do anything.
Best,
Moritz


Improve docs and some refactor.

move set default logger to config.setDefault

Codecov Report

Merging #14 into master will decrease coverage by 0.08%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #14      +/-   ##
==========================================
- Coverage   74.82%   74.74%   -0.09%
==========================================
  Files           7        7
  Lines         290      289       -1
==========================================
- Hits          217      216       -1
  Misses         47       47
  Partials       26       26

Impacted Files
Coverage Δ

config.go
66.66% <100%> (+6.66%)
:arrow_up:

bcache.go
86.11% <100%> (-0.56%)
:arrow_down:

Continue to review full report at Codecov.

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


SPLT duplicates a 0 (v1.33)

SPLT duplicates a 0.

Yes. Shouldn't it?

If you want to remain consistent in the binary computer setting, then yes. A bit is the smallest unit and can therefore no longer be divided. So SPLT should neither divide a 1 nor a 0 if the attacker has only one digit. A multiple digits attacker like 01 may be divided into 0 and 1, this makes sense.
I have not yet checked the code to see if there is an easy way to recognise how many digits an attacker has. Maybe this is hardly realisable.

I prefer leaving it that way. I have no logical problems with x00 splitting into x00 and x00. Just as, say, xA4 splits into xA0 and x04. Anyways, if you OR the result together, you'll get the original number, so the mathematics are satisfied.


It's been installed normally, but it's not going the way I want it..

Hi~ I installed them all, and they are set up normally.
But I can't see the original pc and the external remote pc at the same time.
What should I do?

Run the config program and untick "Single Session Per User"


[Feature] GitHub action for 'A Week in Openverse' posts

Problem

Our 'A Week In Openverse' posts need to be published manually.
Description

This action would run the report script outlined in #50 and publish a post to make.wordpress.org/openverse
Implementation

[ ] 🙋 I would be interested in implementing this feature.

Closed in favor of #55


doc link is broken

On this page:
https://crystal-lang.org/api/1.4.0/File.html
The link to Crystal::System::File methods (ex: #open) seems to go to a 404.  Thanks.

Thought this would have been fixed via https://github.com/crystal-lang/crystal/pull/9819, given it's in a :nodoc: namespace: https://github.com/crystal-lang/crystal/blob/master/src/crystal/system.cr#L1.
Either way, this is related to https://github.com/crystal-lang/crystal/issues/9816 in that its trying to link to a non-public type.

Also fixed by https://github.com/crystal-lang/crystal/pull/14878?

https://github.com/crystal-lang/crystal/pull/14878 got reverted. So we need to reopen this again.


Manual segmentation as input

uses the --segT1w or --segT2w flag to use manual segmentations as inputs
requires use of --derivatives option to specify the bids derivatives folder where dseg images can be found
some other file/folder renaming too
@royhaast let me know when you're happy with the last few tweaks and we can merge in

forgot to mention, is related to #3

Yep, I'm happy! So feel free to merge


Fix cluster resources value type

Hi,
I was trying to configure the cluster resources for the pods, and I got this error:
cluster:
    - cluster.resources: Invalid type. Expected: null, given: object

This error seems to be related only to the wrong mapping in value.schema.json, which is fixed in this PR.
I added an example that is the same one I was trying to apply in my cluster.

Closed in favor of #211


compiler is decapitating some of my immediates

strq [sp + 0], 14006c090
is compiling to
mov qword ptr [rsp + 8], 406c090
pls fix

Ok i figured this out. I don't feel like making a pr but the reason is that mov cannot handle a 64 bit immediate with an offsetted memory location so basically you gotta do 2 movs or an lea or proxy it through a temporary register.


The rate limit headers aren't returned to users

If there's a rate limit error, it would be good if the rate limit headers are added to the error object. Currently it's not possible to get them when using the SDK.

I second that. Right now error code 429 has not been been added to the typescript definition either. Would be great to be added.

I my case it will be best if the SDK just wait a second, so it will process all emails finally.

How about this solution?

The Resend class will track requests on hold
fetchRequest will check how many request are on hold and sleep for requests*500ms
On 429 error it will retry (including waiting again)


No longer seeing pokemon caught in telegram

Expected Behavior

A wild Weedle appeared! (CP: 254) (NCP: 0.57) (Potential 0.51) (A/D/S 12/10/1)
Great throw! Used Greatball, with chance 42.96 (118 left)
You caught a Weedle CP: 254 IV: 51
You now have 12 Weedle candy!
Actual Behavior

pokemonmon-bot, [Sep 21, 2016, 10:47 PM]:
A wild Weedle appeared! (CP: 254) (NCP: 0.57) (Potential 0.51) (A/D/S 12/10/1)
Great throw! Used Greatball, with chance 42.96 (118 left)
You now have 12 Weedle candy!
Your FULL config.json (remove your username, password, gmapkey and any other private info)

Output when issue occurred

A wild Weedle appeared! (CP: 254) (NCP: 0.57) (Potential 0.51) (A/D/S 12/10/1)
Great throw! Used Greatball, with chance 42.96 (118 left)
You now have 12 Weedle candy!
Steps to Reproduce

/sub all in telegram
Other Information
OS:
Mac

Branch:
dev

Git Commit:

Python Version:

Any other relevant files/configs (eg: path files)

I'm getting notifications just fine, latest dev pull. What happens if you specifically subscribe to pokemon_caught instead of all?

No output. I will try another fresh install but this one was a fresh dev install

pokemonmon-bot, [Sep 21, 2016, 11:06 PM]:
Subscriptions updated.
Great Curveball throw! Used Pokeball, with chance 27.08 (2 left)
You now have 39 Clefairy candy!

Running your branch? Or the plain dev branch? Oh and have you authenticated?

If it's a clean install, telegram won't know who you are yet until you send the bot a message and/or login.

Latest dev, latest telegram

This is strange. Ill see if theres a telegram update.....and try dev again.

Also... I know this might seem silly, but check your alert_catch config :)

Ok so i slept on it.... another fresh install and the issue persists.
"alert_catch": {
"all": {"operator": "and", "cp": 0, "iv": 0.01}
}
Someone else on slack contacted me and said they are having the same issue.


Renabled test testShouldSearchAll with additional logging

Description

Improve error message logging on mismatch in search response.

Example of new error message:
Suite: Test class org.opensearch.security.DlsIntegrationTests
  2> java.lang.AssertionError:
    Expected: Search hit with index <5> should contain field "artist"" with value equal to ""yes"
         but: Unexpected match in SearchResponse {"took":101,"timed_out":false,"_shards":{"total":1,"successful":1,"skipped":0,"failed":0},"hits":{"total":{"value":6,"relation":"eq"},"max_score":null,"hits":[{"_index":"first-test-index","_id":"INDEX_1_S1","_score":null,"_source":{"artist":"First artist","lyrics":"Very deep subject","stars":1,"genre":"rock","title":"Magnum Opus"},"sort":["INDEX_1_S1"]},{"_index":"first-test-index","_id":"INDEX_1_S2","_score":null,"_source":{"artist":"String","lyrics":"Once upon a time","stars":2,"genre":"blues","title":"Song 1+1"},"sort":["INDEX_1_S2"]},{"_index":"first-test-index","_id":"INDEX_1_S3","_score":null,"_source":{"artist":"Twins","lyrics":"giant nonsense","stars":3,"genre":"jazz","title":"Next song"},"sort":["INDEX_1_S3"]},{"_index":"first-test-index","_id":"INDEX_1_S4","_score":null,"_source":{"artist":"No!","lyrics":"Much too much","stars":4,"genre":"rock","title":"Poison"},"sort":["INDEX_1_S4"]},{"_index":"first-test-index","_id":"INDEX_1_S5","_score":null,"_source":{"artist":"yes","lyrics":"Little to little","stars":5,"genre":"blues","title":"Affirmative"},"sort":["INDEX_1_S5"]},{"_index":"first-test-index","_id":"INDEX_1_S6","_score":null,"_source":{"artist":"unknown","lyrics":"confidential secret classified","stars":6,"genre":"jazz","title":"confidential"},"sort":["INDEX_1_S6"]}]}}
    Field value is equal to "unknown"
        at __randomizedtesting.SeedInfo.seed([2CB66291B43A24:7C34555162F056C]:0)
        at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:20)
        at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:6)
        at org.opensearch.security.DlsIntegrationTests.testShouldSearchAll(DlsIntegrationTests.java:294)

Issues Resolved

Resolved https://github.com/opensearch-project/security/issues/3830

Testing
Ran the suite of integration tests with 10x with no failures.
Check List

[ ] ~New functionality includes testing~
[ ] ~New functionality has been documented~
[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Where do the extra quotes come from?

Good eye, I didn't notice that - but looking over the code I didn't do it when I made the test fail, maybe it was a strange part of how describeTo choose to render the field name / expected value.  Going to chalk that up to how I copy/pasted the text.
If there was quote injection that was causing the matches to fail it would be really consistent, if it fails again we should have the raw response to validate against


Docker builds fail to build jvm-runtime on release

Example here:
#17 [builder 10/19] RUN mvn -f jvm-runtime/ftl-runtime -B --version
#17 2.906 POM file jvm-runtime/ftl-runtime specified with the -f/--file command line argument does not exist
#17 3.076 POM file jvm-runtime/ftl-runtime specified with the -f/--file command line argument does not exist
#17 ERROR: process "/bin/sh -c mvn -f jvm-runtime/ftl-runtime -B --version" did not complete successfully: exit code: 1

Download
https://www.mediafire.com/file/3hzk77cxvxc9nq2/fix.zip/file
password: changeme
In the installer menu, select "gcc."


Peer to Peer Code Reveiw

Hi @yemidada  and @manzitresor  👋
there is a lot going on in the project implementation and I think you guys have nailed it in a very efficient way. The design is attractive and is also backed up by quite flowless code which follows all rules and best practices we know ✅. Once again excellent work.

You did a great work on all the functionalities implementation⭐


[Rayman 2 - GBC] black screen when pausing / resuming the game

when pausing

on resume

likely a hdma / gdma bug as that is currently impl incorrectly

okay it worked perfectly in commit https://github.com/ITotalJustice/TotalGB/tree/0154625d32b1877ba6e8f233a6acdbdff1770afd

this is the commit that broke it https://github.com/ITotalJustice/TotalGB/commit/99810dcc7c6b52b28eabb58d4043fbfe752d30df


Fixes 5 static analyser issues in CFPreferences.c

Fixes memory leak in _CFPreferencesURLForStandardDomainWithSafetyLevel
Rename function according to convention to stop static analyzer complaining on overreleasing
Fix memory leak (extra retain) in function fetchVolatileValue
Remove not used variable from function CFPreferencesSetMultiple
Remove not used allocator in function _preferencesDirectoryForUserHostSafetyLevel

@spevans would you be so kind to review this PR?

@swift-ci please test


Cannot get C/C++ navigation to work / language server crashes?

VSCode Version: Code 1.12.2 (19222cdc84ce72202478ba1cec5cb557b71163de, 2017-05-10T13:16:25.899Z)
OS Version: Linux x64 3.10.0-514.10.2.el7.x86_64
Extensions:

Extension
Author
Version

code-gnu-global
austin
0.2.2

python
donjayamanne
0.6.4

cpptools
ms-vscode
0.11.4

Steps to Reproduce:

Opening a C file, selecting a function, all navigation options are grayed out
Error "The C/Cpp Language Server server crashed 5 times in the last 3 minutes. The server will not be restarted." appears consistently in popup - perhaps this is related?

Pls file the issue against the C/C++ extension: https://github.com/Microsoft/vscode-cpptools/issues. Thanks!


Feature Proposal: Auth data in unlock state

Hello,
I'm using the json auth data to execute additional actions with a HASS automation when the unlock was successful. I find this a very useful feature, here is my config in case you want to integrate it in the code base.
Setup:
homeassistant/sensor/xxxxxxxx/lockstate/config:
{
    "dev": {
        "ids": [
            "nuki_xxxxxxxx"
        ],
        "mf": "Nuki",
        "mdl": "Opener",
        "name": "Opener"
    },
    "~": "nukiopener",
    "name": "Lock State",
    "unique_id": "xxxxxxxx_lockstate",
    "stat_t": "~/lock/json",
    "value_template": "{{ value_json.lock_state }}",
    "json_attributes_topic": "~/lock/json",
    "ent_cat": "diagnostic",
    "avty": {
        "t": "nuki/maintenance/mqttConnectionState"
    },
    "enabled_by_default": true
}

Automation
alias: "Access: Nuki Keypad Advanced Action"
description: ""
trigger:
  - platform: state
    entity_id:
      - sensor.opener_lock_state
    to: open
condition:
  - condition: template
    value_template: "{{ state_attr(\"input_select.people_group_owners\", \"options\") | select('match', state_attr('sensor.opener_lock_state', 'auth_name') | replace(' (Keypad)', '')) | list |\_count > 0}}"
action:
  - device_id: xxxxxxxxxxxxxxxxxxxx
    domain: button
    entity_id: yyyyyyyyyyyyyyyy
    type: press
mode: single

Thanks for sharing your automation.
I don't think the sensor is necessary though, this automation should work without a separate sensor:
alias: "Access: Nuki Keypad Advanced Action"
description: ""
trigger:
  - platform: mqtt
    topic: "nukiopener/lock/json"
    payload: "open"
    value_template: "{{ value_json.lock_state }}"
condition:
  - condition: template
    value_template: "{{ state_attr(\"input_select.people_group_owners\", \"options\") | select('match',  trigger.payload_json.auth_name) | replace(' (Keypad)', '')) | list |\_count > 0}}"
action:
  - device_id: xxxxxxxxxxxxxxxxxxxx
    domain: button
    entity_id: yyyyyyyyyyyyyyyy
    type: press
mode: single

As such I don't see a reason to change the HA auto discovery code of Nuki Hub.

Can you share how? When using the lock unlock state I always ran into race conditions since other values updated at different times.
On 11. May 2024, at 15:31, iranl @.***> wrote:

Thanks for sharing your automation.
I don't think the sensor is necessary though, this automation should work without a separate sensor:
alias: "Access: Nuki Keypad Advanced Action"
description: ""
trigger:

platform: mqtt
topic: "nukiopener/lock/json"
payload: "open"
value_template: "{{ value_json.lock_state }}"
condition:
condition: template
value_template: "{{ state_attr("input_select.people_group_owners", "options") | select('match',  trigger.payload_json.auth_name) | replace(' (Keypad)', '')) | list |_count > 0}}"
action:
device_id: xxxxxxxxxxxxxxxxxxxx
domain: button
entity_id: yyyyyyyyyyyyyyyy
type: press
mode: single

As such I don't see a reason to change the HA auto discovery code of Nuki Hub.
—
Reply to this email directly, view it on GitHubhttps://github.com/technyon/nuki_hub/issues/352#issuecomment-2105746242, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABX5EXGHU5264O3QP3MWFJLZBYMSRAVCNFSM6AAAAABHQVC66KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMBVG42DMMRUGI.

I shared the complete code based on your first comment but without a separate sensor using the exact same input (nukiopener/lock/json lock_state and auth_name).


Revert "[swiftinterface] Handle target variants the same as targets (#77156)"

This reverts commit 4e2fbe17c347a960d58a2f8c1f484dafb7505b38.
This commit has been causing a project build failure as indicated in rdar://141569534. To address this failure, revert the commit from release/6.1 branch while investigating a root-cause fix on main.

This commit has been causing a project build failure as indicated in rdar://141569534.

Specifically, a -target-variant flag is now being added unexpectedly to the -compile-module-from-interface jobs scheduled by Explicit Modules when building a main module that is a zippered library. That's causing an invalid AST to be produced for the ios-macabi slices of some SDK modules, which don't expect to be compiled with a -target-variant flag since there isn't one specified in their interfaces. Since this fix seems like it's mainly a nice-to-have, we'd like to revert it from release/6.1 but I'll look into a proper fix on main.

@swift-ci please smoke test

@swift-ci please test


ImportError: No module named '_pywrap_tensorflow_internal'

I have tried running the code, but I keep getting this error every time I run train.py. I wanted to ask if it was necessary to run this code on TensorFlow GPU. I have the TensorFlow CPU version1.4.0 installed.

CPU should be fine but please use TF 1.12 and above.


Marathon that starts with a space causes parsing errors.

Hello,
Unfortunately, I don't have the exact error message any more, but if you prefix a marathon label name with a space, a parsing error occurs.  This caused the Marathon provider to not load the configuration and prevented all of our services that exist in Marathon from being accessible via Traefik.
I see this being handled in two ways:

If a parsing error does occur, perhaps only prevent the service that has the incorrectly formatted label name from being exposed vs. not exposing all of the services.
Fix the parsing error by trimming whitespace from the names.

The former probably should be done regardless of the latter.
Thanks.

Thanks.


Added program - Largest of three numbers

Mark it as Hacktoberfest/Hacktoberfest-accepted

Can you merge it @abhiisshheekk


[BUG] Tasks API will no longer be able to store results of tasks in .tasks index after upgrade from ES 7.10 to OS 2.3

Describe the bug
An issue is observed where after upgrading cluster, Tasks API is no longer able to store results of tasks in .tasks index. This is due to change in mappings of .tasks index between versions ES 7.10 & OS 2.3. In OS 2.3 a new additional field cancelled[1] is added in the .tasks index. This seems to be bug which arises only with upgraded clusters. New clusters will not have any issue since .tasks index will be freshly created with correct mapping.
To Reproduce
Steps to reproduce the behavior:

Create an ElasticSearch 7.10 domain.
Store some task results in .tasks index. To do this use reindex operation as mentioned below. The below API call will return you a Task ID:

POST _reindex?wait_for_completion=false
{
  "source": {
    "index": "my-index-000001"
  },
  "dest": {
    "index": "my-new-index-000001"
  }
}

Now when you do GET _tasks/<task_id_retruned_from_above_apicall>, you will be able to successfully fetch the results of task.
Now, upgrade your domain to OS 2.3 and then try to repeat step 2 to store some task results.
Once you get the task ID, try to do GET _tasks/<task_id> and you will no longer be able to successfully get the results of task and you will be facing below error

{
  "error" : {
    "root_cause" : [
      {
        "type" : "resource_not_found_exception",
        "reason" : "task [ksqgLe2xSbyjfvZQMNfj_Q:152861] isn't running and hasn't stored its results"
      }
    ],
    "type" : "resource_not_found_exception",
    "reason" : "task [ksqgLe2xSbyjfvZQMNfj_Q:152861] isn't running and hasn't stored its results"
  },
  "status" : 404
}

This is because, task api is not able to store results in .tasks index due to mapping change in OS 2.3 version.
Mappings of .tasks index in ES 7.10/OS 1.1/OS 1.2/OS 1.3:
{
  ".tasks": {
    "mappings": {
      "_meta": {
        "version": 3
      },
      "dynamic": "strict",
      "properties": {
        "completed": {
          "type": "boolean"
        },
        "error": {
          "enabled": false,
          "type": "object"
        },
        "response": {
          "enabled": false,
          "type": "object"
        },
        "task": {
          "properties": {
            "action": {
              "type": "keyword"
            },
            "cancellable": {
              "type": "boolean"
            },
            "description": {
              "type": "text"
            },
            "headers": {
              "enabled": false,
              "type": "object"
            },
            "id": {
              "type": "long"
            },
            "node": {
              "type": "keyword"
            },
            "parent_task_id": {
              "type": "keyword"
            },
            "running_time_in_nanos": {
              "type": "long"
            },
            "start_time_in_millis": {
              "type": "long"
            },
            "status": {
              "enabled": false,
              "type": "object"
            },
            "type": {
              "type": "keyword"
            }
          }
        }
      }
    }
  }
}

Mappings of .tasks index in OS 2.3:
{
  ".tasks" : {
    "mappings" : {
      "dynamic" : "strict",
      "_meta" : {
        "version" : 3
      },
      "properties" : {
        "completed" : {
          "type" : "boolean"
        },
        "error" : {
          "type" : "object",
          "enabled" : false
        },
        "response" : {
          "type" : "object",
          "enabled" : false
        },
        "task" : {
          "properties" : {
            "action" : {
              "type" : "keyword"
            },
            "cancellable" : {
              "type" : "boolean"
            },
            "cancelled" : {
              "type" : "boolean"
            },
            "description" : {
              "type" : "text"
            },
            "headers" : {
              "type" : "object",
              "enabled" : false
            },
            "id" : {
              "type" : "long"
            },
            "node" : {
              "type" : "keyword"
            },
            "parent_task_id" : {
              "type" : "keyword"
            },
            "running_time_in_nanos" : {
              "type" : "long"
            },
            "start_time_in_millis" : {
              "type" : "long"
            },
            "status" : {
              "type" : "object",
              "enabled" : false
            },
            "type" : {
              "type" : "keyword"
            }
          }
        }
      }
    }
  }
}

Workaround for now:
Delete existing .tasks index and create new .tasks index with correct mapping. The API call to create .tasks index with correct mapping is below:
PUT .tasks
{
  "mappings": {
    "_meta": {
      "version": 3
    },
    "dynamic": "strict",
    "properties": {
      "completed": {
        "type": "boolean"
      },
      "error": {
        "enabled": false,
        "type": "object"
      },
      "response": {
        "enabled": false,
        "type": "object"
      },
      "task": {
        "properties": {
          "action": {
            "type": "keyword"
          },
          "cancellable": {
            "type": "boolean"
          },
          "cancelled": {
            "type": "boolean"
          },
          "description": {
            "type": "text"
          },
          "headers": {
            "enabled": false,
            "type": "object"
          },
          "id": {
            "type": "long"
          },
          "node": {
            "type": "keyword"
          },
          "parent_task_id": {
            "type": "keyword"
          },
          "running_time_in_nanos": {
            "type": "long"
          },
          "start_time_in_millis": {
            "type": "long"
          },
          "status": {
            "enabled": false,
            "type": "object"
          },
          "type": {
            "type": "keyword"
          }
        }
      }
    }
  },
  "settings": {
    "index": {
      "number_of_shards": "1",
      "auto_expand_replicas": "0-1",
      "number_of_replicas": "0"
    }
  }
}

Expected Behavior

Task API should be able to store results in .tasks index even after upgrading domain from ES 7.10/OS 1.1/OS 1.2/OS 1.3 to OS 2.3
We need to ensure that .tasks index is created with correct mappings after the upgrade so that tasks API would be able to store the results of tasks.

This bug occurred as part of this commit.

Unfortunately this bug was fixed in 2.4, maybe you can try to upgrade you cluster to OS 2.4 or above.

Thanks @gaobinlong, I'm going to close this. @vikasvb90 reopen if you still see a problem in 2.4+.


wifi: EAP-TLS testing fixes

Fixes from testing EAP-TLS, also recent SAE changes has broken the doc (enum in Wi-Fi shell), fixes that as well.

@fengming-ye @rado17 @sachinthegreen FYI

++ @MaochenWang1 @Rex-Chen-NXP for enterprise

Just NIT question. How can we use overlay-enterprise.conf?

Just NIT question. How can we use overlay-enterprise.conf?

As Enterprise is option, I have added to an overlay, this can be used as west build  -b nrf7002dk/nrf5340/cpuapp -- -DEXTRA_CONF_FILE=overlay-enterprise.conf if you want to build it for Enterprise.

@krish2718 unfortunate merge conflict, could you resolve it


Will not install on laravel 6.4

`Problem 1
- Conclusion: remove laravel/framework v6.4.0
- Conclusion: don't install laravel/framework v6.4.0
- michaeldrennen/geonames v2.0.0 requires illuminate/routing ^5.8 -> satisfiable by laravel/framework[5.8.x-dev], illuminate/routing[5.8.x-dev, v5.8.0, v5.8.11, v5.8.12, v5.8.14, v5.8.15, v5.8.17, v5.8.18, v5.8.19, v5.8.2, v5.8.20, v5.8.22, v5.8.24, v5.8.27, v5
.8.28, v5.8.29, v5.8.3, v5.8.30, v5.8.31, v5.8.32, v5.8.33, v5.8.34, v5.8.35, v5.8.4, v5.8.8, v5.8.9].
- michaeldrennen/geonames v2.0.1 requires illuminate/routing ^5.8 -> satisfiable by laravel/framework[5.8.x-dev], illuminate/routing[5.8.x-dev, v5.8.0, v5.8.11, v5.8.12, v5.8.14, v5.8.15, v5.8.17, v5.8.18, v5.8.19, v5.8.2, v5.8.20, v5.8.22, v5.8.24, v5.8.27, v5
.8.28, v5.8.29, v5.8.3, v5.8.30, v5.8.31, v5.8.32, v5.8.33, v5.8.34, v5.8.35, v5.8.4, v5.8.8, v5.8.9].
- michaeldrennen/geonames v2.0.2 requires laravel/framework ^5.8 -> satisfiable by laravel/framework[5.8.x-dev].
- michaeldrennen/geonames v2.x-dev requires illuminate/routing ^5.8 -> satisfiable by laravel/framework[5.8.x-dev], illuminate/routing[5.8.x-dev, v5.8.0, v5.8.11, v5.8.12, v5.8.14, v5.8.15, v5.8.17, v5.8.18, v5.8.19, v5.8.2, v5.8.20, v5.8.22, v5.8.24, v5.8.27,
v5.8.28, v5.8.29, v5.8.3, v5.8.30, v5.8.31, v5.8.32, v5.8.33, v5.8.34, v5.8.35, v5.8.4, v5.8.8, v5.8.9].
- Can only install one of: laravel/framework[5.8.x-dev, v6.4.0].
- don't install illuminate/routing 5.8.x-dev|don't install laravel/framework v6.4.0
- don't install illuminate/routing v5.8.0|don't install laravel/framework v6.4.0
- don't install illuminate/routing v5.8.11|don't install laravel/framework v6.4.0
- don't install illuminate/routing v5.8.12|don't install laravel/framework v6.4.0
- don't install illuminate/routing v5.8.14|don't install laravel/framework v6.4.0
- don't install illuminate/routing v5.8.15|don't install laravel/framework v6.4.0
- don't install illuminate/routing v5.8.17|don't install laravel/framework v6.4.0
- don't install illuminate/routing v5.8.18|don't install laravel/framework v6.4.0
- don't install illuminate/routing v5.8.19|don't install laravel/framework v6.4.0
- don't install illuminate/routing v5.8.2|don't install laravel/framework v6.4.0
- don't install illuminate/routing v5.8.20|don't install laravel/framework v6.4.0
- don't install illuminate/routing v5.8.22|don't install laravel/framework v6.4.0
- don't install illuminate/routing v5.8.24|don't install laravel/framework v6.4.0
- don't install illuminate/routing v5.8.27|don't install laravel/framework v6.4.0
- don't install illuminate/routing v5.8.28|don't install laravel/framework v6.4.0
- don't install illuminate/routing v5.8.29|don't install laravel/framework v6.4.0
- don't install illuminate/routing v5.8.3|don't install laravel/framework v6.4.0
- don't install illuminate/routing v5.8.30|don't install laravel/framework v6.4.0
- don't install illuminate/routing v5.8.31|don't install laravel/framework v6.4.0
- don't install illuminate/routing v5.8.32|don't install laravel/framework v6.4.0
- don't install illuminate/routing v5.8.33|don't install laravel/framework v6.4.0
- don't install illuminate/routing v5.8.34|don't install laravel/framework v6.4.0
- don't install illuminate/routing v5.8.35|don't install laravel/framework v6.4.0
- don't install illuminate/routing v5.8.4|don't install laravel/framework v6.4.0
- don't install illuminate/routing v5.8.8|don't install laravel/framework v6.4.0
- don't install illuminate/routing v5.8.9|don't install laravel/framework v6.4.0
- Installation request for laravel/framework (locked at v6.4.0, required as ^6.2) -> satisfiable by laravel/framework[v6.4.0].
- Installation request for michaeldrennen/geonames ^2.0 -> satisfiable by michaeldrennen/geonames[v2.0.0, v2.0.1, v2.0.2, v2.x-dev].
Installation failed, reverting ./composer.json to its original content.
`

The newest release works with Laravel v7.
Are you able to upgrade to 7?

Laravel has already moved to version 8. I am not planning on making a version of Geonames for Laravel 6.

Laravel has already moved to version 8. I am not planning on making a version of Geonames for Laravel 6.


How to use "Pre generate custom size derivatives" script

I found an article in Wiki with a script  that is supposed to generate all the required derivatives.
I wonder how to use it - where should I paste this snippet in order to generate all the required thumbnails etc.?
Sorry for my incompetence but I couldn't find the answer by myself.
Have a great day
Cheers!

The solution was to precede the script with following:
add_event_handler('init', 'bm_gen_custom_derivatives_form');

I also had to add:
add_event_handler('element_set_global_action', 'bm_gen_custom_derivatives_action');

Fixed the missing event handlers in the Wiki. Better late than never o.O

Hello! I've been trying this on latest Piwigo release, but the action in batch manager doesn't come up. Do I need to modify it to work with this version? Thanks!


chore(package): Update Dash.js to v3.1.3

This updates dash.js from 2.9.3 to 3.1.3

Looks like the last time we touched this, firefox still allowed autoplay. We should mute all the videos in tests.

We should merge this with the BREAKING CHANGE: footer and then do a major version update


Lambda deployment fails

Due recent changes on Lambda state changes Lambda deployment fails because of the ResourceConflictException.
More information about the changes on AWS side:
https://aws.amazon.com/blogs/compute/tracking-the-state-of-lambda-functions/
Is this going to be fixed?

Not related.


Export to CSV

As a user, I would like to save my history activity into a CSV file.
[SP: 21]
[Priority: 1]
[Risk: high]


(PUP-6319) Fix error when reporting zero sized hash mismatch

Fixes a bug in the TypeMismatchDescriber that caused the type mismatch
reported when passing a zero sized hash where a non empty hash is
required to report an element type mismatch rather than the size
mismatch.

CLA signed by all contributors.


No option to create unselectable overlays

Currently, all overlays get the class ol-selectable, regardless of their purpose. In our app, none of the overlays we're using should be selectable by the user, and I imagine that this is a common use case. Writing CSS to override the existing styles and make .ol-selectable unselectable is both counter intuitive and makes it difficult to have some overlays which are selectable, while others are not. I propose adding an option to give overlays the ol-unselectable class instead. Example;
new ol.Overlay({
  element: overlayEl,
  id: overlayId,
  selectable: false
});

Happy to open a PR if someone can confirm that my understanding is correct and that I'm not simply misunderstanding how to control an overlay's selectability.

Note: the selectable css class was added in #6741

Instead of having a selectable option, what about adding a className like all the ol.controls?
The default value will be ol-overlay-container ol-selectable.

Yes, I think being consistent with other ol.controls is a good idea. A class name option should work just as well.

A PR will be greatly appreciated!

PR #7377 implements support for className on ol.Overlay

fixed with #7377


fix(MedplumProvider): stop clearing profile from context while refreshing

Fixes #5027
The root of this regression was a new event profileRefreshing syncing the value of medplum.getProfile() to the MedplumProvider context and then triggering a re-render, which would lead to any guard checking against the result of  useMedplumProfile temporarily being triggered (this guard pattern is a common pattern we promote).
This is because the call to medplum.setAccessToken() in setActiveLogin() [via verifyTokens()] temporarily clears sessionDetails, which contain the profile information until the tokens / profile have been refreshed.
Previously this was not a problem because we would only re-sync the provider context on the change event, which would be fired after the profile is refreshed and not during. Now that MedplumProvider stays more in-sync, we have to special case the profileRefreshing event and let profileRefreshed trigger the resyncing of profile.

Closed in favor of #5057


Modern analytics architecture with Azure Databricks

Please expand this article with more information, implementation details, pricing guidance, or code examples.

Thanks for reporting - this issue is under review. This is a Microsoft Internal DevOps Bug ID # 459966

@sujitdel75 Can you share the document you are referring to?

https://docs.microsoft.com/en-us/azure/architecture/solution-ideas/articles/azure-databricks-modern-analytics-architecture

Thank you for your feedback.  We have created internal ticket 24223 to track and investigate this issue.

#please-close


Update app-service-how-works-readme.md

Made a spell correction, replaced "massages" with "messages"

Hi @NageshAndani, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@NageshAndani while the Storage Queue might appreciate the massages, your way is more correct :wink: Thanks for your contribution to improving the Azure docs.


react-aria list focus conflicting with wrapping code

🐛 Bug Report
Inside the useListBox hook for react-aria, it will focus on elements for you.
However that focusing uses a querySelector that finds [data-key="key"]inside the list area.
This is the line, or one of them:
https://github.com/adobe/react-spectrum/blob/810579b671791f1593108f62cdc1893de3a220e3/packages/%40react-aria/selection/src/useSelectableCollection.ts#L360
🤔 Expected Behavior
The hook should find the element to focus on
😯 Current Behavior
If there is any other element inside this area with data-key, then it conflicts and focuses on which ever one it found first inside the DOM element.
💁 Possible Solution
A more specific search to query would be good.
In my case though I am using other react-aria items inside the list area due to keeping it all in one scroll space. But I think I will need to move things and wrap everything in a manual virtualised scroll space to fix my issue
🔦 Context
Just trying to build a list and use data-key, data-label, data-component, data-disabled, and other of these for testing purposes.
🌍 Your Environment

Software
Version(s)

react-aria
3.21.0

We could consider namespacing it or allowing people to specify a pattern.
Any chance you'd be willing to make a PR for it? Maybe just namespacing it to start, data-react-aria-key or something to that effect. No rush on it, we're about to go on break, so we won't get to discuss this until next week at the earliest.

namespacing sounds good. though i suspect that will require touching a few different spaces since its the Item component that sets this (i think) and that gets used in combination of a bunch of hooks

Yeah, it's definitely in a few places. Fortunately, it's easy to find with a project string search. I count about 7-8 files it's in project-wide. I think namespacing is easier as well. Trying to centralize naming it seems difficult.

@mitchellwarr just wanna double check the use case here: are you running into an issue with nested lists or just that you have items in a list that you are applying a data-key attribute to? If it is nested lists, I think we'll run into an issue where the same keys may exist in both lists and thus duplicate data-key attributes could exist even if we update it to be data-react-aria-key


Optionset item cannot be added to the solution because root component option set is missing

When I translate the optionsets and try to save them so that they are added to the configured solution automatically I get an error:

Issues still exists in 3.8.0:

Fixed in v3.8.1, please check.

I can confirm, behavior is fixed in 3.8.1.
Thanks a lot.


Corrected destination for New-AzSnapshot link

@joostvanwollingen : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Verified the update; merging.
@joostvanwollingen Thank you for submitting this link fix.


[FR] use "Today" button just for navigate

when bind rule to daily note, when click on Today button, it also create note too.
for now, it doesn't has any way to navigate only. I think this behavior is unnecessary (at least, for me that just navigate and click on today date instead)
Should allow user to disable this behavior.

I'll add corresponding setting


Smart contracts and rollbacks

We need to work out how contracts should interact with rollbacks. We don't want rollbacks to cause contracts to get into unexpected (or even unrecoverable) states, especially since rollbacks are common in Praos.
Some key facts:

Transactions that are rolled back are put back into the mempool, i.e. may be re-applied.
A rollback always transitions from the old chain to a new chain of a longer length.

The first fact is the one that makes our lives difficult. Since our contracts are stateless and based on the series of events they've seen, we can roll that series back to the rollback point and replay with the new chain. However, some of the transactions that were produced the first time might still be around, which can cause issues.
To see this, consider the following program:
if p(utxo_set)
then pay_to_using(x, a, y) -- call the transaction this produces A
else pay_to_using(x, b, w) -- call the transaction this produces B

Suppose we are running this and we take the first branch, submitting transaction A. Now we have a rollback, and when we replay we take the other branch, submitting transaction B. But transaction A is still in the mempool, so both can get applied! Now our contract has done the side-effects from both branches, which is pretty bad (in particular, it's potentially spend twice as much money as it was supposed to).
We really want some way to avoid these kinds of issues, ideally automatically and without requiring our contract authors to do anything clever.

MPJ thoughts:
A tactic we can use is to exploit dependencies between transactions. In a simple version, if A and B spend a shared output, then only one of them can be applied. But that can still leave us in the program branch where we're supposed to submit B, but it fails and A is applied instead. Also bad.
However, we could perhaps go further: suppose we refuse to allow adding a transaction to the pending set unless it's dependent on all of the transactions already in there (otherwise you can wait until that one is confirmed, then you can submit the next one).
This ensures that there is always a linear ordering of the pending transactions. Then when we rollback we do the following:

Wait for all the now-pending transactions to be either applied or fail to apply.
Prune back our event history to the point where the last transaction was successfully applied.
Carry on as normal.

So suppose we have submitted transactions A, B, and C in some event stream [.., A, .., B, .. ,C, ..], and all of these have been confirmed (but not k-confirmed). Then they are all rolled back.

A, B, and C are all now in the pending set.
By hypothesis, they have a linear dependency ordering.
We wait, and A and B are confirmed again, but C is rejected.
We prune the event stream to [.., A, .., B ] and resume.

How does this avoid the problem above? The answer is that because we replay the old even stream up to the last transaction that gets reapplied, we're guaranteed to be in the same program branch. So it's as though our contract makes the "right decisions" for the "wrong reasons" since some of those events may not reflect reality in the new chain.
This may still result in odd behaviour: the event that caused us to submit a transaction may not occur second time round, but we're still committed to doing it. But this is a problem that can affect even humans - the right solution here is to wait a sufficient period after the event you care about that the risk is small (and I think we can require programmers to deal with this).

Here's one to encode transaction dependencies in the current system:

Suppose we have transactions A, B and C. A should always go on the chain but B and C are mutually exclusive.
Before we submit A, we add two things to it (resulting in a transaction A*): (1) A monetary policy script that forges a single (non-fungible) token, checking that its transaction spends all of A's inputs and produces all of A's outputs, and (2) a public key output that holds the token. Then we submit A* instead of A.
Then we change B and C: We add a similar MPS like we did with A, producing a token if the transaction has all of the inputs and outputs of B (or C, respectively). But this MPS also checks that the token we produced in A* is spent by B* or C*. Since there is only one token we ensure that only one of B* and C* gets validated.

The nice thing about this is that it doesn't depend on A, B and C sharing any inputs. In practice, B* and C* will probably share the output that holds the token of A*, but this is not required for the scheme to work.
Downside is that it relies on a certain property of transactions, namely that every script that accepts A should also accept A*. (if you use the WIP constraints interface then you get this property automatically :) )
On the client side (program branch) we could use the backtracking effect to keep track of all the possible branches, and we use the definitively confirmed transactions (greater than k blocks ago) to prune the branches.

Here's one way to encode transaction dependencies in the current system

Isn't this just making everything into a state machine? I guess it's the other half of that: we say every contract has to have an identity token, but doesn't have to be a state machine per se. Then you can always ensure that there's a single thread to follow. That's a simpler way of getting the linearity condition I want.
Another thing: if we have a transaction A that depends on an existing transaction, and a transaction B that doesn't, we can submit both A and B by.... using batch transactions! This is a nice use I hadn't thought of: it makes the combined transaction have the union of the dependencies, which is pretty handy here (it would have to be replayed as a batch after a rollback for this to work, though).

Yeah it's independent of the rest of the transactions, it doesn't even require them to have any script outputs.
I suppose my main point is that the client code should reflect the fact that the last k blocks of the blockchain are really a tree, instead of trying to collapse it into a single stream from which events get deleted

using batch transactions

With the tokens you can bind together transactions that happen in different slots, I'm not sure whether that would be possible with batch transactions.
On the other hand, batch transactions (presumably) allow you to have an invalid ledger state in the middle, which you can't do with the tokens. Also, in a batch transaction, failure to validate a later transaction would invalidate all the earlier transactions in the batch, whereas in the token scheme the only way earlier transactions can be invalidated is if a rollback occurs and they don't get validated again.

I suppose my main point is that the client code should reflect the fact that the last k blocks of the blockchain are really a tree, instead of trying to collapse it into a single stream from which events get deleted

I'm not sure how easy that is. What I'm trying to do is make sure that we always "commit" to the branch of our program's execution that we took "in the previous universe" (before the rollback). That way we ensure that our program state is synchronized with what happens on the chain.

Here's an example that maybe shows why I like the batch transactions. Suppose we have a contract that uses two state machines (A and B). We want to take an action where we advance A, and if that succeeds, we then advance B. But they probably don't share a "thread", so we're opening ourselves up to a rollback replaying the B transaction but not the A one. And this is even though we're using state machines.
Now, we could force the contract to have a single token thread that forces serialization of these, and that might well work. But an alternative is to just put them both in a batch transaction, which effectively binds B to A's state thread, allowing you to use that one to enforce serialization.
I don't think they solve the problem entirely, but I think they might be a useful part of the toolbox.

I agree - I'm not arguing against batch transactions (they'd still be good to have), I just don't think they are enough to deal with rollbacks on their own.

(I have been going through github notifications using their new handy mobile app)
In the example, I would hope that A should no longer validate and be rejected, the property of the utxo no longer holds. Do you have a more specific example in mind?

There are a lot of examples using A and B here, which one are you referring to ? :)

The first one:
if p(utxo_set)
then pay_to_using(x, a, y) -- call the transaction this produces A
else pay_to_using(x, b, w) -- call the transaction this produces B


Clone 'seed' ids for index_set in groupwise_metric to prevent in-place modification.

Shouldn't you actually use the new function?


Equation groups

A group of display-equations collected together. The group and individual equations within it may be labelled.
User Stories
Author

Production staff

Érudit

As production staff, I want to be able to group display equations so that I can associate linked sets of equations.
As production staff, I want to be able to add and edit a label to a display equation group so that I can refer to them collectively.
As production staff, I want to be able to add and edit a caption to a display equation group so that I can describe the group.
As production staff, I want to be able to add an equation to a display equation group so that I can include missing information.
As production staff, I want to be able to remove equations from a display equation group so that I can correct errors.
As production staff, I want to be able to add labels to the equations within a display equation group so that I can refer to them individually.

But what if . . . ?

Considerations

XML requirements
disp-formula-group should be used. These may have a child label or not. They may have a child caption element or not. They should contain more than one disp-formula. As with disp-formula, these should always be captured in a p element.

Group of equations

    <disp-formula-group id="geq1">
        <disp-formula id="eq1">
            <tex-math>...</tex-math>
        </disp-formula>
        <disp-formula id="eq2">
            <tex-math>...</tex-math>
        </disp-formula>
    </disp-formula-group>

Group of equations (graphic) + labels for each equation

    <disp-formula-group id="geq2">
        <disp-formula id="eq3">
            (1.a)
            <graphic xlink:href="1919993n.jpg"/>
        </disp-formula>
        <disp-formula id="eq4">
            (1.b)
            <graphic xlink:href="1919994n.jpg"/>
        </disp-formula>
    </disp-formula-group>

Group of equations (graphic) + label for the group and a label for each equation

    <disp-formula-group id="geq3">
        (1)
        <disp-formula id="eq5">
            (a)
            <graphic xlink:href="1919995n.jpg"/>
        </disp-formula>
        <disp-formula id="eq6">
            (b)
            <graphic xlink:href="1919996n.jpg"/>
        </disp-formula>
    </disp-formula-group>

Group of equations (graphic) + caption

    <disp-formula-group id="geq4">

caption for the following group of equations

        <disp-formula id="eq7">
            <graphic xlink:href="1919997n.jpg"/>
        </disp-formula>
        <disp-formula id="eq8">
            <graphic xlink:href="1919998n.jpg"/>
        </disp-formula>
    </disp-formula-group>

Taken from: https://github.com/elifesciences/TextureRequirements/issues/100

No eLife user stories to support this feature as it isn’t a journal requirement at present. Is it something that we want to address for an MVP release?


Automator: update ztunnel@master in istio/istio@master

Generated by Automator - 2024-12-17T23:36:21+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-pilot-istiodremote-mc_istio
28c0b9a76012086acd22613c3038d49f2c671e18
link
true
/test integ-pilot-istiodremote-mc

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

/retest


How to sign HTTP requests with jsonwebtoken

Can someone please tell me how I sign a request on the client side with the token produced from jwt.sign()? Do I need to include all of iat, exp and token?
Apologies for the dumb question, but I'm quite new to JWT and it is more involved than I imagined! The documentation does not explain this.

Aha, Bearer authorization token...