DBZ-4590 Test the MS SQL Server Plugin with Transparent data encryption (TDE)

Context
Transparent Data Encryption (TDE) encrypts the entire database and transaction log at rest, ensuring unauthorized file access remains unreadable. It operates automatically and transparently, requiring no code changes, and interactions through JDBC remain unaffected. For this reason, the Debezium connector should work fine even with a database protected by TDE.
Expected Behavior:
All the tests should works fine also with a ms SQL Server encrypted with TDE
Changes
We add a new test container with an initial script that activate the encryption to database level. The test container is added in TestInfrastructureHelper in order to be used in DebeziumSqlServerConnectorResourceIT. The nature of TDE is to be trasparent to application level and my suggestion is to leave activated for all the tests. Add a new set of tests for only this feature can grow the pipeline execution time.
closes https://issues.redhat.com/browse/DBZ-4590

@kmos Applied, thanks!


Frontend not validating fields or blocking actions before graphql request

The following two errors are recurring in our graphql error logs:
[x] is not a valid phone number

Cannot create multiple queue entries for the same patient

These can both be (mostly) avoided by performing some quality checks on the frontend prior to sending a request

It seems that google-libphonenumber and disabling the submit test button has resolved these failures


ValueError: Columns must be same length as key  when len(date_cols) > 0

This error is from pandas
throw at

This happened when len(date_cols)>0  .
I found  important_features doesn't equal to old_important_features :
In [4]: 'reg_date_hour' in important_features
Out[4]: True

In [5]: 'reg_date_hour' in old_important_features
Out[5]: False

In [6]: len(date_cols)
Out[6]: 2

In [14]: len(important_features) == len(old_important_features)
Out[14]: False

In [15]: len(important_features)
Out[15]: 460

In [16]: len(old_important_features)
Out[16]: 471

Oh , you create some new feature for date col reg_date_hour , such as
'reg_date_hour_hour',
'reg_date_hour_dayofmonth',

saved in old_important_features , and try to update to dataname[important_features] which doesn't contain the new created features.

Hi @eromoe 👍
You are working with a slightly older version. This problem has been found and fixed. Please upgrade to the latest version via:
!pip install featurewiz --no-deps --ignore-installed --upgrade

thanks
AutoVimal

I installed from conda forge which fixed in featurewiz  0.1.87 .


[Bug]: Desktop version, exporting result set failed

ODC version
odc432
OB version

What happened?

What did you expect to happen?
Export result set successfully
How can we reproduce it (as minimally and precisely as possible)?
Desktop version export result set
Anything else we need to know?
No response

same issue  as  https://github.com/oceanbase/odc/issues/3393


Set function requires check if guest exists

Hi,
I know that you need to do a create before a set but when I created vms I forgot for one vm and the set command created the folder.
I expected an error but I didn't get any.
I tried a destroy and remove and yes the destroy deleted the vm but the remove didn't delete the folder.
If I try to do it manually in /iohyve off course it's said the device is busy because it's mounted. So I had to umount it.

Set function requires check if guest exists


veeam: 'str' object has no attribute 'fetch_json'

This issue respects the following points:

[X] This is a bug, not a question or a setup/configuration issue.
[X] This issue is not already reported on Github (I've searched it).
[X] I use the latest release of the Monitoring Plugins (https://github.com/Linuxfabrik/monitoring-plugins/releases).
[X] I agree to follow Monitoring Plugins's Code of Conduct.

Which variant of the Monitoring Plugins do you use?

[ ] Compiled, from repo
[X] Source Code, from GitHub

Bug description
since the removing the "3" suffix
the veeam and the huawei plugins are throwing exceptions
lib/huawei.py", line 43, in get_creds
result = base.coe(url.fetch_json(
AttributeError: 'str' object has no attribute 'fetch_json'
lib/veeam.py", line 43, in get_token
success, result = url.fetch_json(
AttributeError: 'str' object has no attribute 'fetch_json'
Steps to reproduce - Plugin call
x
Steps to reproduce - Data
No response
Environment
5.10.0-21-amd64 #1 SMP Debian 5.10.162-1 (2023-01-21) x86_64 GNU/Linux
Plugin Version
git
Python version
No response
List of Python modules
No response
Additional Information
No response

Fixed in https://github.com/Linuxfabrik/lib/commit/498d6fdd03c37d75926b65e9cd7650d741227d45. Please test.


Getting the message type

I'm writing a receiveMiddleware and curious if there is a way to determine if a message is an instance of EnterMessage within my script.
Inspecting the context.response.message I can see the difference between a normal message and an enter message, but wondering if there is a  built in way to test this.
Enter message:
message: {
  user: [Object],
  done: false,
  room: 'testing'
}

Normal message:
message: {
  user: [Object],
  text: 'this is a normal message',
  rawText: 'this is a normal message',
  rawMessage: [Object],
  id: '1441831148.000012',
  done: false,
  room: 'testing'
}

The main problem I see with checking for existence of certain keys in the response, is that other events also don't have a text or rawText (leaving a channel for example)

robot.enter and robot.leave for reference.


ArrayIndexOutOfBoundsException in FunctionResource#fromName

patch by kamlesh_ghoradkar, Berenguer Blasi; reviewed by Berenguer Blasi for CASSANDRA-16977
Co-authored-by: kamlesh_ghoradkar<EMAIL_ADDRESS>Co-authored-by: Berenguer Blasi<EMAIL_ADDRESS>

CI 3.11 lgtm

CI 3.11


Major refactor, new syntax

Not sure if you'll want to accept such a massive change, but I stopped the Pyfire class from being stateful, moving the query stuff to a separate class, and returning objects instead of strings.
I didn't incorporate the .child() method since it seems simple enough to just build the correct path and pass that in.

This looks good, thanks 👍  - I'll merge


[WIP] oneMKL migration: Temporary deselection of tests

This PR deselects the tests that are failing in oneMKL migration job:
http://intel-ci.intel.com/ef7a857f-11b3-f179-ab7e-a4bf010d0e2d

Checklist to comply with before moving PR from draft:
PR completeness and readability

[ ] I have reviewed my changes thoroughly before submitting this pull request.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have updated the documentation to reflect the changes or created a separate PR with update and provided its number in the description, if necessary.
[ ] Git commit message contains an appropriate signed-off-by string (see CONTRIBUTING.md for details).
[ ] I have added a respective label(s) to PR if I have a permission for that.
[ ] I have resolved any merge conflicts that might occur with the base branch.

Testing

[ ] The unit tests pass successfully.
[ ] I have run it locally and tested the changes extensively.

Performance

[ ] I have measured performance for affected algorithms using scikit-learn_bench and provided at least summary table with measured data, if performance change is expected.
[ ] I have provided justification why performance has changed or why changes are not expected.

@Vika-F please attach the job, that makes the CI green
current branch + OneDAL oneMKL migration branch

@samir-nasibli I cannot provide the job you are requesting until #2063 is merged. Because the mentioned job would require the changes from #2063.

@samir-nasibli I cannot provide the job you are requesting until #2063 is merged. Because the mentioned job would require the changes from #2063.

#2063 is merged. Let's check this one

Link to the rerun with fixes:  http://intel-ci.intel.com/ef7b39b8-64f0-f1ec-91a0-a4bf010d0e2d

Link to the CI job with the requested changes: http://intel-ci.intel.com/ef7b5b7d-beee-f1af-b697-a4bf010d0e2d


Unable to install ssd1306 library

Bug Report
micropython-ssd1306 fail to install
Just followed the tutorial point by point on a fresh Win11.
After Thonny installation, tried to install as requested by tutorial the "micropython-ssd1306" library but...
Installation fails. Get this error:
"gz does not appear to be a Python project: neither 'setup.py' nor 'pyproject.toml' found.
Process returned with code 1"
Other libraries install correctly, but impossible to proceed without ssd lib!

Fixed, for some reason I had to run Thonny as Admin, doing this, fixed the error!

Glad you were able to sort it out. I haven't used Windows 11 and Thonny myself, so I don't know if this is a new issue. Might be worth checking with the Thonny developers to see if there's something that can be done to work around that.


qhub destroy -c qhub-config.yaml

Hi all,
wouldn't it be a good idea to also have a destroy command for the qhub cli?
qhub destroy -c qhub-config.yaml
What do you think?
Best,
-Filippo

I am closing this since it will be discussed in #172


Consider merging Request and ContextRequest

If we take a look at the types of ContextRequest and Request, we can see a isomorphic relationship between
ContextRequest[F[_], Unit] and Request[F[_]]
Perhaps ContextRequest is the type we really need and we can then have a type alias
type Request[F[_]] = ContextRequest[F, Unit]
We are using the ContextRequest to pass quite a bit of information decoded out of the request and also enriched with data from the database.

Interesting.  I haven't used ContextRequest, but the isomorphism seems correct.
Cons:

That extra field in the Request case might cost some memory request.  I doubt anybody notices.
Type aliases make error messages and code browsing more intimidating to new users.

Pros:

I bet some middleware that takes a Request[F] input now could be generalized.

Random ponderings:

ContextRequest is an awkward shape if everything has a context: (A, (Method, Uri, Headers, Attributes, Stream[F, Byte])).  Can context be flattened into the existing type?
Can context subsume attributes?  Probably not?  attributes could subsume context, but there's a lot of Option.get down that road.
I think @ChristopherDavenport has been interested in a Request[A], where A might be the decoded body.  I think that's a more radical idea

For our own part at WORK we need to have the type exposed in the signature to make sure we have authorization and whatever else we have extracted and enriched from diverse sources.
Having it in attributes hides this fact, and I`m not really a fan of that.
We can indeed flatten the context into the existing Request type, this was just to express what we want.

So yeah, I don't thinking making the context an attribute of the vault will work.  I wonder whether we could make attributes one way of expressing a context, but I think this would break things, not limited to:

the path info caret
the current web socket model

The context is good at surfacing explicit metadata in the type, and the vault is good at smuggling optional metadata.  Maybe we really do need both?


react-native run-ios not working

[Ueno RNS] Building environment config
[Ueno RNS] Using .env
Reading env from .env
Wrote to /Users/dmitrigrossman/projects/my-new-app/ios/build/Build/Products/GeneratedInfoPlistDotEnv.h
/Users/dmitrigrossman/projects/my-new-app
Copied GeneratedInfoPlistDotEnv.h and GeneratedDotEnv.m to ./node_modules/react-native-config/ios
[Ueno RNS] Generating ./src/config.env.js
[Ueno RNS] Config built successfully

** BUILD FAILED **

The following build commands failed:
	PhaseScriptExecution [CP]\ Check\ Pods\ Manifest.lock build/Build/Intermediates.noindex/MyNewApp.build/Debug-iphonesimulator/MyNewApp.build/Script-DE29A3C61EA94606C7B30F25.sh

(1 failure)

Installing build/Build/Products/Debug-iphonesimulator/MyNewApp.app
An error was encountered processing the command (domain=NSPOSIXErrorDomain, code=22):
Failed to install the requested application
The bundle identifier of the application could not be determined.
Ensure that the application's Info.plist contains a value for CFBundleIdentifier.
Print: Entry, ":CFBundleIdentifier", Does Not Exist

Command failed: /usr/libexec/PlistBuddy -c Print:CFBundleIdentifier build/Build/Products/Debug-iphonesimulator/MyNewApp.app/Info.plist
Print: Entry, ":CFBundleIdentifier", Does Not Exist

After installing all according to the docs I'm getting the following error when trying to run react-native run-ios.
Dependencies:
react-native-cli: 2.0.1
react-native: 0.56.0

Looks like you are on the development branch.
Did the rename script complete successfully, looks like Info.plist is either corrupt or non-existent.
You can try reporting some of your findings from this thread: https://github.com/facebook/react-native/issues/12737

@scopsy, I had the same error when I was picking up react native after some time. My fix was to update xcode to latest version.
Perhaps you also need to update cocoapods:
pod install --repo-update
# or if that fails for some reason like it did for me
pod repo remove master
pod setup
pod install # inside ios dir

Please re-open the issue if you are still encountering errors.


4.2 commandBuffer flush slow

hi Mark,
i am trying latest snapshot.
The com.lambdaworks.redis.protocol.CommandHandler.flushCommands()impl is slow

My guess that the copy and remove pattern is inefficient.
Unfortunately the implementation does not have a drainTo Method, so I propose to do the following
                    queuedCommands = new ArrayList<>(commandBuffer.size());
                    RedisCommand<K, V, ?> cmd;
                    while ((cmd = commandBuffer.poll()) != null) {
                      queuedCommands.add(cmd);
                    }

this requires command buffer to be Deque
  public final static <T> Deque<T> newConcurrentDeque() {
    return PlatformDependent.newConcurrentDeque();
  }

sounds good for you? then i can make a pr. otherwise I am looking forward to a better solution :)

Thanks for your research. Have you profiled your proposal using poll()? I'm currently removing Guava from the code so there is a lot of refactoring going on (see https://github.com/mp911de/lettuce/blob/issue/217/src/main/java/com/lambdaworks/redis/internal/LettuceFactories.java). Code looks good but I'd propose to wait couple of days until I complete my work on the branch and then feel free to create a PR.

I will do so next, but the profiles are hard to compare right now the flush method is the most expensive in a 10 minute profile, so thats why i had a look.
i assume my proposal is faster because it is doing a single pass over the collection rather than two (including equals checks)

I can confirm my change is better. hard to get reliable numbers. happy to provide y patch once your work is done, or you directly incorporate it.

Thanks a lot. Let me finish the Guava refactoring, then I'll apply your code.

Implemented.


Memory Leak

Hi Guys, I have this behavior:
When I'm uploading a media, the memory never released, and I have to restart the service for that. in visual studio we can see that as well.
I have a simple .net core application.

app.UseTus(httpContext => new DefaultTusConfiguration
{
Store = new TusDiskStore(@"c:\tusfiles\uploads"),
UrlPath = "/files"
});

Interesting, thanks for reporting! Does the memory continue to increase over time or does it go flat at ~883 MB?

Interesting, thanks for reporting! Does the memory continue to increase over time or does it go flat at ~883 MB?

In my deployed environment, after a hard night of upload media, the memory reaches 12GB, I waited for 3 hours checking if at some point the memory it is stabilized, but that not happened, still using 12GB until I had to restart the service.

Not great... I'll do some profiling and see if I can find anything. Just to remove any other source of this issue, are you running anything else in the same server or is it just tusdotnet?

Only the tusdotnet, I was running a transcoding service but I turned off it and I had the same behavior.

Just an update: Spent some time yesterday profiling and I think the issue is how the read and write buffers are initialized and used. I have started on a fix but won't be able to work on this anymore this week. Current status is that the memory leak seems to be gone and memory consumption is down but the files written to disk are corrupted. Glass half full I guess ;)

TL; DR: A new version will be out once this issue has been fixed for real. As a workaround for now set buffer sizes to 40960 bytes (40 KB) to prevent this issue. Note that this value should be used for both the read and write buffer.
return new DefaultTusConfiguration
            {
                UrlPath = "/files",
                Store = new TusDiskStore(@"C:\tusfiles\", deletePartialFilesOnConcat: false, bufferSize: new TusDiskBufferSize(40960, 40960))
            };

Longer version:
Basically the problem is that we use a MemoryStream with an initial capacity as the write buffer. The MemoryStream is backed by an array. Once the MemoryStream's backing array gets near the size of the initial capacity the array will be resized into double its size. The default write buffer size is 50 KB meaning that the doubled size will be 100 KB. Objects larger than ~83 KB are placed by .NET in the Large Object Heap (LOH, https://docs.microsoft.com/en-us/dotnet/standard/garbage-collection/large-object-heap). These values are not GCed and compacted the same way as smaller objects, causing the memory leak. A new version is underway where we use array pooling instead allowing us to re-use arrays and not initiate new ones.

Thanks, I'm going to set the buffer size while the new version comes out.

New version published on nuget: https://www.nuget.org/packages/tusdotnet/2.2.2
Release notes: https://github.com/tusdotnet/tusdotnet/releases/tag/2.2.2


SKK /= I in chapter 4

I am working through your wonderful "Write you a Haskell" tutorial, thanks a lot for creating it in the first place.
I have downloaded and compiled the code for chapter 4 (the untyped lambda calculus).
Contrary to the explanations in the text, SKK does not reduce to I in my case, instead I get:
Untyped> (\x y z. x z (y z)) (\x y . x) (\x y . x)
  => \x y z . (x z (y z))
  => \x y . x
 => \x y z . (x z (y z)) (\x y . x)
 => \x y . x
<<closure>>
Untyped>

Recall, that according to the text on page 50 I should see something along the lines of:
Untyped> (\x y z. x z (y z)) (\x y . x) (\x y . x)
  => \x y z . (x z (y z))
  => \y z . ((\x y . x) z (y z))
    => \x y . x
    => \y . z
   => z
 => \z . z
\z . z

I wonder what's going on? Cannot be a huge thing, as the data structures / code base
is relatively small at this point.
Nevertheless, as SKK=I is supposed to be a kind of litmus test, I would like to
get this fixed.
A few more explanations for what's going on in detail in Eval.hs would also be nice
(VClosure, Scope, type Eval = WriterT [Step](State EvalState) a, inc, red, etc)
Thanks.

In this case you get a closure whose value is an identity function (\x -> x).

OK, thanks.
Stephen Diehl<EMAIL_ADDRESS>writes:

In this case you get a closure whose value is an identity function (\x -> x).


Keep recently loaded chunks in a deque with a maximum length.

Vastly improves the speed of blockAt/setBlockAt and friends since the AnvilChunk isn't released immediately.
Feel free to increase the deque's maxlen.
As mentioned in https://github.com/Khroki/MCEdit-Unified/issues/334#issuecomment-80725139.

Is this the thing that you said you were going to backport to MCEdit 1.x?

Oh no, I'm still working on that. This was a simple change so I pushed it first.

Thanks a lot for helping out on the project codewarrior!


Add H5FD_http_finalize function and call on hdf5 finalize

Fixes Unidata#2617

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

H5FD_http_finalize is added to unregister the class - the terminate call in the class structure is still there, just it doesnt need to do anything in this case
It looks like the terimnate call is called as class shoytsdown, which is normally when HDF5 closes. In a case where netcdd AND hdf5 are closing at the same time, this is ok.
In the case like octave where H5 is still around after netcdf has closed, it is a problem and the h5 library still has references to classes from netcdf.
Hence the new function intended on just unregistering the class that was regisitered via H5FD_http_init

But wouldn't it work to add the unregister to H5fD_http_term?

But wouldn't it work to add the unregister to H5fD_http_term?

Possibly? But you would also need to call H5fD_http_term from within hdf5dispatch and provide it as an external function within libhdf5/H5FDhttp.h
That also assumes that calling unregister from that function when its being as terminate() on a shutdown in the class is also ok.

I did a quick test and calling unregister from H5FD_http_term causes a crash unless it is removed  from being a reference in the H5FD_http_g struct.
If you would rather keep the function as that and call it from dispatch, but remove from the class as terminate, I can change it


How to use?

I thought it was plain JavaScript which I can simply link and use.

Installed Node
cd binance-master
npm install

I have never used node but I am good with JavaScript. Can you list down in few steps that how to use/run this?

in your directory "npm install --save binance"

It looks like maybe you've checked out the repository.
You typically should only need to do that if you're looking to make changes to the repository.
If you just want to use this in another project, do the following:
You may first want to have the package.json file: npm init
Then install the binance project: npm install binance --save
Then use it in your code via const binance = require('binance')

Thanks, I was trying to use it from this repository. Installing from package repository using npm install binance --save worked.


"BEAUTY ADVISER FULL TIME (8H/ZI) - IULIUS MALL, TIMISOARA" doesn't not found to "Sephora" site

URL: https://peviitor.ro/
browser: Microsoft Edge
device: Computer
OS: windows

Steps to reproduce

1.Open the company recruitee page: https://jobs.sephora.com/Balkans/search/?createNewAlert=false&q=&locationsearch=romania&optionsFacetsDD_customfield1=&optionsFacetsDD_customfield2=&optionsFacetsDD_shifttype=
2.Pay attention that are showed ten jobs
3.Open peviitor URL
4.Put the name "sephora" in the search field
5.Press the button "Caută"
6.Pay attention that is displayed eleven jobs.
7. Press the button "Vezi postul" from job "BEAUTY ADVISER FULL TIME (8H/ZI) - IULIUS MALL, TIMISOARA"
8. You observe that this job doens't exist

Expected results

It should't displayed on peviitor.ro site the job "BEAUTY ADVISER FULL TIME (8H/ZI) - IULIUS MALL, TIMISOARA" at Sephora company.

Actual results

It is displayed on peviitor.ro site the job  "BEAUTY ADVISER FULL TIME (8H/ZI) - IULIUS MALL, TIMISOARA" at Sephora company.

Retest.


Some site design - ideas

To make the site visually more clean, there are a couple of options.
My ideas :

use Bootstrap as a base
use a free template as a base, like https://www.free-css.com/free-css-templates/page292/simply-amazed
...

I agree that the site looks very basic currently, though I also like the clean simple look and the instant loading.
To especially keep the latter, I don't want to use any large Javascript or CSS libraries, or external fonts, all of which would also make maintenance a lot harder and more time-consuming.

See my new proposal for something more simple (that is, just some css rules)
https://github.com/LorcanaJSON/LorcanaJSON-website/pull/4
Don't hesitate to ask me questions, or propose a view of what a "Better site design" (your own words) can be for you.

That proposal looks great, thanks!
There's a few small things I'm not sure about, like the question mark in front of optional fields, the icons in the 'Downloads' table before links, and how it's hard to see that the JSON link and MD5 link are separate links, but those are minor things. It looks a lot better than what I made!
The official Lorcana site has a Resources page with, among other things, this color guide, which would be fun to implement.
Would you mind if I accept this proposal and then change some things about it, like those colors and the download links? Or would you rather we discuss the changes together, since it's your design?

Hi,
Happy to hear that, you're welcome !
I just pushed another commit :

I removed the question marks
I put back "md5" on the next line
And I applied the template to changelog page, with some specific styles for this page

Dynamically generated TOC of the versions on the left
Permalinks put in a box on the right
version format shown has badges

I used the Lorcana colors and added the logo

Of course, you can improve it ! (You may even add commits on the branch of my fork, if I correctly understand the Github help pages)
Some of your remarks are about ideas I put to see what it looks like, and be improved. Typically :

the question marks were here as a tentative to make optional fields visually different. I'll try other things.
the download icons are meant to visually enhance the download links
So you can tell me what you like / dislike, and/or make the changes yourself. It's your project.

:)

Oh, by the way, one last thing : as I'm color-blind, color associations may seem awful for one who isn't. Don't hesitate to tell me !

By the way, in the last minute I tried something for the links, but I'm not fan of it.

Sorry for being quiet for a while, I suddenly got very busy with verifying and publishing the cards for Set 4.
But that's now done!
In the process I did break your merge request by adding a changelog entry and updating the index page though, sorry.
But the new colors look good, thanks for the update.
On the changelog page, the list of each changelog entry is a good idea. The links to the index page sections look a bit weird there though, instead of the whole list just a 'Back to main' link would be better.

Thanks for set 4; I'll merge your commit into my MR.
For the menu, well... My opinion was rather to consider that you have the full menu on every page of the site (like a menu on every site)
But it's your site, so I'll get the "back to main" item back in place of the menu.
Le 19 mai 2024 à 15:30, à 15:30, Didero @.***> a écrit:

Sorry for being quiet for a while, I suddenly got very busy with
verifying and publishing the cards for Set 4.
But that's now done!
In the process I did break your merge request by adding a changelog
entry and updating the index page though, sorry.
But the new colors look good, thanks for the update.
On the changelog page, the list of each changelog entry is a good idea.
The links to the index page sections look a bit weird there though,
instead of the whole list just a 'Back to main' link would be better.
--
Reply to this email directly or view it on GitHub:
https://github.com/LorcanaJSON/LorcanaJSON-website/issues/2#issuecomment-2119239381
You are receiving this because you authored the thread.
Message ID:
@.***>

That's done !
You can check, and merge (I suggest you squash the commits when doing the merge, there is no real need to pollute the history with back and forth)

Sorry I've been quiet for so long again, I was fighting with parsing the cards from 'Deep Trouble'.
I've just merged your redesign. I'll make some small changes, since I just noticed that the footer doesn't show up.
Again, thanks very much for your work, it really elevates the website!!

I made some small changes, and some much-needed updates to the actual texts, since quite a few of it was a bit behind, whoops.
I also just put the redesign live, so now everybody can admire your work!
I put the old design here for now, for comparison


Defender for DNS enablement

The deprecated Defender for DNS plan shows this on the tooltip.
"Applies to all Azure resources connected to the Azure default DNS resolvers."
As of August 1, customers with an existing subscription to Defender for DNS can continue to use the service, but new subscribers will receive alerts about suspicious DNS activity as part of Defender for Servers P2.
The enablement of Defender for Servers P2 implies only Azure server resources (VMs, Arc, etc).
Does that mean the new DNS plan only supports servers or did the deprecated plan only support servers in the past as well?

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

ID: 608b5811-b62d-d7b0-8b70-99253c49a0ee
Version Independent ID: 56cedb4d-d286-c9f7-fe9b-7678f2967f74
Content: Microsoft Defender for DNS - the benefits and features - Microsoft Defender for Cloud
Content Source: articles/defender-for-cloud/defender-for-dns-introduction.md
Service: defender-for-cloud
GitHub Login: @dcurwin
Microsoft Alias: dacurwin

@applefacts,
Thanks for your feedback! We will investigate and update as appropriate.

@applefacts can you please clarify the question? What needs to be added to the documentation?

#label:"backlog-item-created"


Fix workflow for Kotlin DSL capturing samples

I noticed the new workflow to verify the capturing samples using the Kotlin DSL subproject was referencing a build.gradle file instead of build.gradle.kts. It was also using Groovy DSL (apply from: ...) instead of Kotlin DSL (apply(from = ...)) to apply the capturing script.
I also update the steps to use the working-directory property to make it consistent with the surrounding steps.

Good question.
Here is a Build Scan comparison between groovy and kotlin for a build without these changes.
In both builds, you can see the Gradle Enterprise plugin version did apply, so the verification did "work". However, the kotlin DSL run executed using the build.gradle file, not the build.gradle.kts file.

For completeness, here is a Build Scan comparison between two builds with these changes which shows the kotlin DSL file is being properly updated.

Ah, i see now, the groovy projected was being executed and not the kotlin one as well! Thanks for fixing that!


Oddity with Encoding Raw Strings

I've been fixing and expanding the Finch implementation of the Tech Empower Benchmark.
For the plain text test, I have a router:
  val plaintext: Router[String] = Get / "plaintext" /> "Hello, World!"

When I run Tech Empower's tests this yields an error saying that the response has 2 more bytes than expected. When I look at the response I see that there are 2 quotes in the body I don't think should there:
HTTP/1.1 200 OK
Content-Type: application/json;charset=utf-8
Date: Wed, 17 Jun 2015 18:09:55 GMT
Content-Length: 15
Server: Finch 0.7.0

"Hello, World!"

Are these quotes supposed to be in the response and if so how do I get rid of them? Or is this a bug?

Hm. Interesting. I believe these quotes happened after #283. We should probable write a tase case and try to fix it.

I hope this doesn't mean that we have the same issue with JSON encoding...

It doesn't seem to. I've cleaned up the json test too and it passes their test with flying colors.

Just double-checked the demo project. The output JSON is not wrapped with quotes:
TTP/1.1 200 OK
Content-Type: application/json;charset=utf-8
Content-Length: 221

[{"id":0,"name":"Tom Paine","tickets":[]},{"id":1,"name":"Benjamin Franklin","tickets":[{"id":2,"label":"PHL -> CDG"},{"id":3,"label":"PHL -> LHR"}]},{"id":4,"name":"Betsy Ross","tickets":[{"id":5,"label":"PHL -> DCA"}]}]

Maybe @yanana can quickly spot the error?

Yeah, so far it just seems to be strings. I've pushed my current work here. Might be a good starting put. I'll continue to look at it, but help other sets of eyes are appreciated.

Argonaut provides a codec for strings that quotes them, and that's probably what's getting picked up here. As a workaround you could provide the instance in EncodeResponse explicitly. In the longer term we may want to consider parametrizing EncodeResponse on the contentType (at the type level).

Ok, so it seems I wasn't importing io.finch.response.EncodeResponse._, which means I didn't have the encodeString implicit on scope. Importing that package fixes the issue. I find this more than a little concerning because I don't know what implicit conversions are making the other behavior happen. I think we probably need to note this somewhere.

You shouldn't need to import EncodeResponse._ in normal usage—the compiler knows to look for the EncodingResponse[String] instance there. The issue is that io.finch.argonaut also provides an EncodeResponse[String], and that one takes priority (until you import the original one explicitly).
More generally, it makes sense to want to have different EncodeResponse instances for the same type depending on the content type. This wouldn't be difficult, but it would be a pretty big API change.

Exactly as @travisbrown said, an EncodeResponse which converts String to JSON is used as Content-Type: application/json;charset=utf-8 suggests.

I don't think it has anything to do with argonaut. Especially seeing as the plain text endpoint doesn't use it. I just removed both EncodeResponse._ and all references to argonaut in that file and I still get the original behavior.

Which original behavior do you mean?
Here's an example of how the Argonaut instance can change things:
scala> import io.finch.response.EncodeResponse
import io.finch.response.EncodeResponse

scala> implicitly[EncodeResponse[String]].apply("foo")
res0: com.twitter.io.Buf = ByteBuffer(3)

scala> import io.finch.argonaut._
import io.finch.argonaut._

scala> implicitly[EncodeResponse[String]].apply("foo")
res1: com.twitter.io.Buf = ByteBuffer(5)

The second Buf is two characters too long.

Disregard that last message, you're right it is argonaut causing the issue.

Ok, so then this isn't a bug with code, but more of an issue of me not being aware of all of the EncodeResponses on scope. We probably need to address this. Do you want to close the issue and open a more general one for parameterizing EncodeResponse or keep everything here?

@travisbrown nice analysis!
@yanana good catch of the application/json content-type (I somehow missed), which indicates that a wrong instance of EncodeResponse has been used.
For now it's good to know a workaround: an explicit import of the implicit conversion raises its priority. But we definitely have to think about type-level EncodeResponse.

Thanks for figuring this out @travisbrown and @yanana! I'm going to close this, as this particular issue is fixed.

@rpless maybe we should keep this opened until we introduce the new EncodeResponse API? Although, I'm totally fine with opening a new issue for that.

@vkostyukov My feeling is that this issue is rather specific and the idea of a new EncodeResponse API is only introduced about halfway through the thread. A more generic issue might better represent the problem without the noise from this issue (it can certainly reference this issue though). But I'm fine with leaving it here. It's your call.

I think the way to go on this is a type member representing the content type inside EncodeResponse—that way most usage is unchanged, but you can ask for an encoder for a specific content type when you want one.
I'll take a stab at an implementation this evening.


Csse layout 550 error

Description

[x] see changelog

Changelog description

Status

[x] Code base linted
[x] Ready to go

Requesting review @davidbrownell, @krachwal, @rfievet, @ketanbj, @varun646, @jyoung3131


[SPARK-34779][CORE] ExecutorMetricsPoller should keep stage entry in stageTCMP until a heartbeat occurs

What changes were proposed in this pull request?
Allow ExecutorMetricsPoller to keep stage entries in stageTCMP until a heartbeat occurs even if the entries have task count = 0.
Why are the changes needed?
This is an improvement.
The current implementation of ExecutorMetricsPoller keeps a map, stageTCMP of (stageId, stageAttemptId) to (count of running tasks, executor metric peaks). The entry for the stage is removed on task completion if the task count decreases to 0. In the case of an executor with a single core, this leads to unnecessary removal and insertion of entries for a given stage.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
A new unit test is added.

cc @squito

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/136174/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/40756/

Cc @squito @wypoon  Could you review this change? Thanks!

lgtm gentle ping @dongjoon-hyun @cloud-fan for reviews

@Ngone51 I added a unit test for the changes. The unit test is to verify the stage entry and its peak metrics will be kept in the current heartbeat interval even if the active task count is down to 0. The goal is to avoid peak metrics missing.

Test build #136419 has started for PR 31871 at commit 27fef30.

cc @tgravescs @mridulm @attilapiros

And please apply the change requested by @Ngone51 too:
https://github.com/apache/spark/pull/31871#discussion_r600302467
Thanks!

Test build #136473 has started for PR 31871 at commit 2751c75.

I give time for others to review and if no issue comes up I'll merge it in the end of next week (assuming it's still passing CI and no review is in progress).

Sounds good, thanks for the review! @Ngone51 @attilapiros

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/136473/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41057/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41057/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/41057/

LGTM too. Thanks for fixing the bug!

retest this please

Test build #136679 has started for PR 31871 at commit 2751c75.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41261/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41261/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/41261/

Test build #136679 has finished for PR 31871 at commit 2751c75.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/136679/

@wypoon Thanks for pointing it out! I will update the PR.

@wypoon I updated the PR description fixed typo in the title.

@Ngone51 Could you retest this pr? I think the test failures are not related to the changes of this pr. Thanks!

retest this please

Test build #136826 has started for PR 31871 at commit 2751c75.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41406/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41406/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/41406/

Test build #136826 has finished for PR 31871 at commit 2751c75.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/136826/

Merged to master

Is this a regression introduced in Spark 3.0? If so, I think we should backport the fix to 3.1/3.0 as well.

@Ngone51 Yes, this was introduced in spark 3.0.

So, could you help submit a separate PR to backport to 3.1 & 3.0?

@Ngone51 this is not a bugfix but an improvement (see the updated PR description and https://github.com/apache/spark/pull/31871#issuecomment-809805138) and we are not backporting improvements

I know it's an improvement based on ExecutorMetricsPoller. But, is it a regression compares to Spark 2.4?

The whole ExecutorMetricsPoller was introduced in Spark 3.0 by https://issues.apache.org/jira/browse/SPARK-26329 which was definitely an improvement (even without this PR) on Spark 2.4.

Ok, I saw executorUpdates is totally a new field in Heartbeat that introduced in Spark 3.0. So it shouldn't be a regression and don't need to backport @baohe-zhang .

Yeah, it makes sense.


Iox #624 add nodiscard keyword to expected

Pre-Review Checklist for the PR Author

[x] Code follows the coding style of CONTRIBUTING.md
[x] Tests follow the best practice for testing
[x] Branch follows the naming format (iox-#123-this-is-a-branch)
[x] Commits messages are according to this guideline

[x] Commit messages have the issue ID (iox-#123 commit text)
[x] Commit messages are signed (git commit -s)
[x] Commit author matches Eclipse Contributor Agreement (and ECA is signed)

[x] Update the PR title

Follow the same conventions as for commit messages
Link to the relevant issue

[x] Relevant issues are linked
[x] Add sensible notes for the reviewer
[x] All checks have passed (except task-list-completed)
[x] Assign PR to reviewer

Notes for Reviewer
The [[nodiscard]] or when possible the strong __attribute__((warn_unused)) was added to cxx::expected. This causes some restructuring of the code, tests and of the examples. Please do not comment on anything which is not strictly related to this since unit test and code refactoring should be part of another PR.
This PR only introduced IOX_NO_DISCARD.
Checklist for the PR Reviewer

[ ] Commits are properly organized and messages are according to the guideline
[ ] Code according to our coding style and naming conventions
[ ] Unit tests have been written for new behavior
[ ] Public API changes are documented via doxygen
[ ] Copyright owner are updated in the changed files
[ ] PR title describes the changes
[ ] Check QNX build

Post-review Checklist for the PR Author

[ ] All open points are addressed and tracked via issues

References

Closes #624

Great work, thanks! Friendly-reminder, as we're close to the v1.0-rc1: Please build it manually on QNX to see if QCC 5.4.0 is able to understand warn_unused.

Great work, thanks! Friendly-reminder, as we're close to the v1.0-rc1: Please build it manually on QNX to see if QCC 5.4.0 is able to understand warn_unused.

Both [[no_discard]] and [[gnu::warn_unused]] keywords are only activated when:

gcc >= 5. or clang >= 4

The [[no_discard]] keyword is only activated in Windows.
This means there is no change for QNX.

Great work, thanks! Friendly-reminder, as we're close to the v1.0-rc1: Please build it manually on QNX to see if QCC 5.4.0 is able to understand warn_unused.

Both [[no_discard]] and [[gnu::warn_unused]] keywords are only activated when:
* gcc >= 5. or clang >= 4

The [[no_discard]] keyword is only activated in Windows.
This means there is no change for QNX.

Would be a pity if it's not used even if it would work. Is there anybody with QNX access who could check this?


resolve frc46_token ambiguity

          Why does it have the _official_ token whilst in the other datacap states we have a copy-pasted one? Can we always use the crate?

Originally posted by @LesnyRumcajs in https://github.com/ChainSafe/fil-actor-states/pull/244#discussion_r1478366611

I'm not sure why we did have the copy-pasted one in the first place, but the datacap token is modified to cater for Lotus RPC compat.
What needs to be done is:

Use the crate instead.
Introduce TokenExt in forest to cater for RPC compat.

There's a discussion here: https://chainsafeio.slack.com/archives/C01SZBSC6KZ/p1707717786883299.
I appreciate that this is a private link, but still keeping it here should we need to find the thread fast.

@ruseinov keep in mind that older release used older crate versions. It might make sense to introduce several flavours of this dependency, just like for FVM.

@ruseinov keep in mind that older release used older crate versions. It might make sense to introduce several flavours of this dependency, just like for FVM.

I'm unsure why we should bother with that. Why don't we just update latest fil-actor-states as discussed, release it and integrate into latest forest with TokenStateExt. Older release depended on older crate versions, that is fine. Why several flavours?

We should bother with that if there are breaking changes between the crate releases, same reason why we have FVM2, FVM3, and FVM4. It might not be needed now for the token crate, but it's something we should keep in mind.

We should bother with that if there are breaking changes between the crate releases, same reason why we have FVM2, FVM3, and FVM4. It might not be needed now for the token crate, but it's something we should keep in mind.

I'm still unsure why. FVM versioning is a really bad example, it's horrible to deal with and I hope we never have to resort to that kind of thing. I'm pretty sure that if we had control over it - it could have been done much nicer, at least the compatibility layer.
I'm unclear on why we can't introduce a breaking change in a new release of the crate and then if people want to use the old version - they can. In any case this consideration warrants a valid use-case, before this materialises - I think we're safe to proceed in a way that we did so far.
I would also argue that before resorting to the FVM approach we should carefully weigh all the pros and cons, I'd rather have several codepaths and versioning WITHIN the crate, than having to deal with 3 versions of the same crate and trying fit them into the same product. I know Lotus isn't the best example, but so far they have managed to maintain actors compatibility without resorting to anything as extreme as that.

I'm not sure what are you even referring to. Let me rephrase. We may need a dependency like frc46_token_9 = 9.0.0, frc46_token_10 = 10.0.0 and so on, if there are breaking changes between those two. For now, there doesn't seem to be such need. If it's still unclear, let's just drop it.

I'm not sure what are you even referring to. Let me rephrase. We may need a dependency like frc46_token_9 = 9.0.0, frc46_token_10 = 10.0.0 and so on, if there are breaking changes between those two. For now, there doesn't seem to be such need. If it's still unclear, let's just drop it.

I wasn't sure what you were referring to, I thought we are talking about fil-actor-states. I'm not sure why we are discussing this now as it's currently irrelevant. When it becomes relevant - let's discuss it again.


Newer versions of DIC have problem analysing cd projekt.exe

Version
All recent MPF versions with DIC are crashing, the MPF 3.2 does not. Seems like DIC in this release was released around 2021-11-03.
Describe the bug
Cannot continue dumping process.
This should be the file.
cd projekt.zip
From this disc: http://redump.org/disc/35110/
(Currently trying to do verification.)

Can you upload all logs created by DIC?

DESPERADOS.zip
This should be it.

Try to use the latest version. https://github.com/saramibreak/DiscImageCreator/releases/tag/20240101

It helped, thx.


Update requirements.txt

pinning pygments version, using Pygments==2.3.1 on wrangler caused an issue w/ loading EntryPoints

Also added curlify to the requirements.txt


How to ignore nan when using cross_value?

Or specify a default value as result when compare a nan.

Here is my use case. The first 6 rows of CrossDown80 expected value should be false, but true here.

@GF-Huang,
Right.. RSI Crossings. I recommend checking out: https://github.com/twopirllc/pandas-ta/issues/406. Perhaps that might address the issue? 🤷🏼♂  Lemme know.
KJ

The ta.xsignal seems not meet my use case, which if I only need the entries, without exits.

@GF-Huang,
Sorry for the delay.
Unfortunate that xsignals, it does need some work and assistance on it would be great. 😎
Regarding cross_value, perhaps @lluissalord may have suggestions as well. 🤷🏼♂
KJ


HDH-159

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[ ] Feature
[x] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[x] Tested Locally
[

] Optional Screen Shoot.

Visually button looks great. Can we just call every link "Subscribe" without the module name? Also for the PR, maybe have to bring the Chaos file up to speed. This PR might have more commits in it than the button.

  42 | </ul>

> 43 |   </TabItem>

     |   ^

  44 |   <TabItem value="Early access">

  45 | <p>{`This release does not include early access features.`}</p>

  46 |   </TabItem>

[ERROR] Client bundle compiled with errors therefore further build is impossible.

error Command failed with exit code 1.

info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.


[executor] change all subsequent txn statuses to RETRY after a valida…

…tor set change txn
Motivation
@zekun000 's request to set all the txns in a block after a validator-set-change-txn with a new
transaction status Retry.  https://github.com/libra/libra/issues/2485
Have you read the Contributing Guidelines on pull requests?
Y
Test Plan
Existing tests

Are we planning to implement the logic for reporting the retry signal back to the mempool in this PR or in a separate one?

@runtian-zhou separate pr

Would be good to add some tests?

test added @wqfish

:umbrella: The latest upstream changes (presumably 95b4cfeb79bb41233f7315186685f2362482daa5) made this pull request unmergeable. Please resolve the merge conflicts.

:umbrella: The latest upstream changes (presumably 85c5d2d1d8d596061f38a548d8fa9cc57462b360) made this pull request unmergeable. Please resolve the merge conflicts.

@bors-libra r=zekun000

:pushpin: Commit 8b26c7a has been approved by zekun000

:hourglass: Testing commit 8b26c7abf3b50f3131608d778b1ccc841c41592e with merge b29eeb8469ea8ac3c9a105f922faa47caac9460f...

:broken_heart: Test failed - checks-circle_commit_workflow

@bors-libra retry

@bors-libra r=zekun000

:pushpin: Commit 5e24181 has been approved by zekun000

:hourglass: Testing commit 5e24181e503e20268e38dedb96f5dd3b0752d94b with merge b1e240e99a6f40c859c286cdacbd13f4be60fc63...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: zekun000
Pushing b1e240e99a6f40c859c286cdacbd13f4be60fc63 to master...


Problems with the greek translation

Hi, as a native greek speaker, I have to point out there are some problems with the greek translation. I wonder if automatic translation was used? There's some inaccurate and/or unnatural wording used throughout, and I am afraid the two paragraphs in lines 29-40 are simply unreadable.

Hi, @pankaramp . Please, just send a pull request that I will approve it.


Evaluation for the Rope3d

Thank you for your excellent work, I have some questions for you.
**1. Whether the evaluation of Rope3d is filtered according to the mask provided by the official?
**2. Can you provide the code of bevdepth and bevformer for the Rope3d?
Looking forward to your reply

Thank you for your attention.
**1. Yes, please visit "https://github.com/liyingying0113/rope3d-dataset-tools/tree/main/eval_tools" for more details.
**2. We are working on a unified code base supporting bevformer, bevdepth baselines and other roadside datasets which will be open source.

Thank you for your attention.
**1. Yes, please visit "https://github.com/liyingying0113/rope3d-dataset-tools/tree/main/eval_tools" for more details.
**2. We are working on a unified code base supporting bevformer, bevdepth baselines and other roadside datasets which will be open source.


Use err directly when errorMessage missing

Solves #448

as this is a small fix, I'd like to merge it - is the PR ready to go and can be converted from draft to a regular PR?

I'll leave it to the experts :)


Error when trying to run report for source freshness

When I run edr report, I have a test in my source set up like this:
sources.yml:
version: 2

sources:
  - name: mysource

    schema: mydataset
    tables:
      - name: table1
        loaded_at_field: "CAST(consumer_etl_date_time AS TIMESTAMP)"
        freshness:
          warn_after: {count: 3, period: hour}
          error_after: {count: 3, period: day}
        meta:
          elementary:
            timestamp_column: "CAST(consumer_etl_date_time AS TIMESTAMP)"
        tests:
          - elementary.freshness_anomalies
          - elementary.all_columns_anomalies:
              column_anomalies:
                - null_count
                - missing_count
                - zero_count

I get the error in the report:
elementary_source_freshness_anomalies_ingestion_layer_table1_ (models/ingestion_layer/sources.yml) dbt was unable to infer all dependencies for the model "elementary_source_freshness_anomalies_ingestion_layer_table1_". This typically happens when ref() is placed within a conditional block. To fix this, add the following hint to the top of the model "elementary_source_freshness_anomalies_mysource_table1_": -- depends_on: {{ ref('monitors_runs') }} > in macro test_freshness_anomalies (macros/edr/tests/test_freshness_anomalies.sql) > called by test elementary_source_freshness_anomalies_mysource_table1_ (models/mysource/sources.yml)
Where should I exactly add
-- depends_on: {{ ref('monitors_runs') }}
Is there a way to not have to do that? The documentation does not mention that.
 I can't get in

@erikzaadi can you or someone help out here? looks like all of the depends_on hints are added by default on the elementary macros. What else should I do on my side? This happens with a lot of other tests.
  To fix this, add the following hint to the top of the model "elementary_schema_changes_table1_":

  -- depends_on: {{ ref('schema_columns_snapshot') }}

  > in macro test_schema_changes (macros/edr/tests/test_schema_changes.sql)

Hey @jpnalvim,
As the error states, you should add -- depends_on: {{ ref('monitors_runs') }} at the top of your table1_.sql model.
You can further read here on depends_on expression.

What is your version of dbt-core and dbt-elementary? We're getting pretty much the same error when we upgraded dbt version to 1.7.*, but with any previous dbt versions it ran smoothly and without any "depends_on" errors.

Hi all,
Closing this issue since it has no activity for over 3 months, but if it's still relevant please feel free to re-open!
Thanks,
Itamar


User

It was a bad idea to have an id and a buck_id with which to identify a user. This moves everything to just use buck_id.
While I was at it, the migrations were condensed. Since there isn't a production database yet, combining some migrations makes sense

Codecov Report

Merging #283 into rails will increase coverage by 0.07%.
The diff coverage is 76.92%.

@@            Coverage Diff             @@
##            rails     #283      +/-   ##
==========================================
+ Coverage   96.69%   96.76%   +0.07%
==========================================
  Files          52       52
  Lines        1209     1206       -3
==========================================
- Hits         1169     1167       -2
+ Misses         40       39       -1

Impacted Files
Coverage Δ

app/models/discipline_action.rb
100% <100%> (ø)
:arrow_up:

app/models/password_reset_request.rb
100% <100%> (ø)
:arrow_up:

config/routes.rb
100% <100%> (ø)
:arrow_up:

app/models/user_challenge.rb
100% <100%> (ø)
:arrow_up:

app/controllers/users_controller.rb
72.22% <50%> (+3.8%)
:arrow_up:

app/models/user.rb
96.11% <75%> (-1%)
: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 e7657db...817ba30. Read the comment docs.


Problem with aarch64 for Jetson compiling, memory alignment issue

If you are reporting any crash or any potential security issue, do not
open an issue in this repo. Please report the issue via emailing
<EMAIL_ADDRESS>where the issue will be triaged appropriately.
Title: Cannot run the image binary on Jetson Xavier
Description:

I'm trying to run "envoyproxy/envoy-dev:9bdb13e831da2f3a1dfbd0ffb60e1be8aab681c9" or "envoyproxy/envoy:v1.19-latest". There is an issue with memory paging and alignment. It looks like the problem comes from "tcmalloc". I have seen there is a patch issued to address this issue: https://www.gitmemory.com/issue/envoyproxy/envoy/15235/850516622

Repro steps:

You can try to pull the images mentioned above on Jetson Xavier and try to run them with this command:
docker run --rm envoyproxy/envoy:v1.19-latest --version

Note: The Envoy_collect tool
gathers a tarball with debug logs, config and the following admin
endpoints: /stats, /clusters and /server_info. Please note if there are
privacy concerns, sanitize the data prior to sharing the tarball/pasting.

Admin and Stats Output:

Include the admin output for the following endpoints: /stats,
/clusters, /routes, /server_info. For more information, refer to the
admin endpoint documentation.

Note: If there are privacy concerns, sanitize the data prior to
sharing.

Config:

Include the config used to configure Envoy.

Logs:

external/com_github_google_tcmalloc/tcmalloc/system-alloc.cc:550] MmapAligned() failed (size, alignment)<PHONE_NUMBER><PHONE_NUMBER> @ 0x5578d6a960 0x5578d5cc80 0x5578d5c6c0 0x5578d45ef0 0x5578d598f0
external/com_github_google_tcmalloc/tcmalloc/arena.cc:34] FATAL ERROR: Out of memory trying to allocate internal tcmalloc data (bytes, object-size) 131072 48 @ 0x5578d6ac70 0x5578d45f6c 0x5578d598f0

Note: If there are privacy concerns, sanitize the data prior to
sharing.

Call Stack:

If the Envoy binary is crashing, a call stack is required.
Please refer to the Bazel Stack trace documentation.

If it is beneficial, I can grant access to the device. Also, I'm willing to help to apply the patch if I get the instruction on how I should apply the patch.

This looks like a dupe of https://github.com/envoyproxy/envoy/issues/15235, let's followup there.


No struct definition in swagger generate

Problem statement
I am generating the swagger json by running below command
swagger generate spec -m -o ./doc/swagger.json
It generates the json, however it does not include the fields in the definition and only includes
"definitions": {
  "MyStruct": {
    "x-go-package": "github.com/myorg/my-service-repo/model/"
  }
}

I have the struct with swagger comments as below
// swagger:model MyStruct
type MyStruct struct {
	Field1 string `json:"field1" binding:"required"`
	Field2 string `json:"field2" binding:"required"`
}

Steps to reproduce
Generate swagger from root directory
swagger generate spec -m -o ./doc/swagger.json
Environment
swagger version
version: v0.30.5
commit: ab6a7885723430004f1d7db6395369b6e7f3370b

go version
go version go1.20.5 darwin/amd64

sw_vers
ProductName:            macOS
ProductVersion:         14.4.1
BuildVersion:           23E224

by setting env variable export SWAGGER_GENERATE_EXTENSION=false it eliminates x-go-package but still keeps the definitions empty
"definitions": {
  "MyStruct": {
  }
}

I confirm that the issue is with the binary built. It works fine with installing the go module
go install github.com/go-swagger/go-swagger/cmd/swagger@latest

Please release the proper binary.


question: possibility to force data type ?

Hello mbleron,
Many thanks for your great work, excelGen saved (and still saves) us a lot of time !
Here's what I could need your help on :
My use case is calling addSheetFromQuery with a SQL request I can’t modify, which returns text, dates & number, all in text format.
I’d like to know if there is a way to force a type to be applied on columns, even if not present in the source SQL.
e.g. I'd like to be able to call :
sql_request := 'select ''test'', to_char(sysdate) from dual';
sheet1 := ExcelGen.addSheetFromQuery(ctxId, sheetName, sql_request);
and still have a way to force sysdate to be recognized as a date in generated xls file, without modifying sql_request
Thanks !

Hi Michel,
I understand what you're asking but I'm afraid that is beyond the scope of ExcelGen right now.
If you know which columns need to be re-cast, you may build a new query on top of the original one with the necessary transformations in the SELECT.
I can provide some sample code for that if you like.
Marc.

hello,
Thanks for the quick answer :)
This is exactly what I was trying to avoid, our request generation is a bit complex - but if there's no choice, well...
And thanks again for the help proposition, but I'll be able to manage this on my side.
Michel


Standardize API for dataflow and node versioning

Currently, there are at least 3 sets of features that rely on dataflow and node versioning:

the CLI version command and its underlying logic (see here) which outputs a dictionary / JSON of nodes and dataflow versions. Node versioning is based on the hash of the originating function using graph_utils.hash_source_code(). The dataflow version is derived from the dictionary of {node_name: node_version}.
the h_diskcache.DiskCacheAdapter also uses graph_utils.hash_source_code() to version the node's originating function. The cache key is derived from (node_hash, input_1, input_2, ..., input_n)
the h_experiments.hook.ExperimentTracker preceeded the two other features. Therefore, it has its own h_experiments.hook.graph_hash() function. It creates a hash from the dictionary of node originating functions. It should leverage the same logic as the CLI.

Solution design

does it make sense to have hash_source_code() under graph_utils? If yes, we could add hash_node and hash_dataflow()
it could make sense as properties for graph_types.HamiltonNode and graph_types.HamiltonGraph
disambigue terms around "hash", "version", "node", "function", "originating_function"

I don't think people should be using graph_utils as part of the API -- I like having properties on the graph_types to compute these in a "blessed" way.


bouncing balls

Add an extension for bouncing balls game

Everytime I seeing this types of issues instead of raising it via feature request.
@mahek0620 Kindly explain this.

Creating an advanced bouncing balls game using HTML, CSS, and JavaScript involves incorporating various features such as multiple ball types, collision detection, physics simulation, interactive controls, scoring and levels, visual and sound effects, responsive design, obstacles, and customization options.


Orbital Laser Drill Not Turning On

Version 1.10.2-1.0.17
I've looked through the documentation, and I have everything set up perfectly. The multiblock is formed, there is a lens in the GUI, there are no blocks underneath the laser, it is facing the planet, I have a lever on the control block, and nothing happens. The energy input blocks and the GUI both show that there is plenty of power, yet nothing happens.

Restarting the game fixed it. Weird.


add content types

this adds the the official apache promoted content types to allow for clients to distinguish between compact and binary protocols. This is of use to me and hopefully of use more generally.

Current coverage is 75.68% (diff: 100%)

Merging #685 into master will not change coverage

@@             master       #685   diff @@
==========================================
  Files            30         30
  Lines           625        625
  Methods         600        600
  Messages          0          0
  Branches         25         25
==========================================
  Hits            473        473
  Misses          152        152
  Partials          0          0

Powered by Codecov. Last update 08f2cde...0f8042c

@crispywalrus this looks interesting. In which way do you use these?

for some services inside a finagle environment where the thrift is transported over http rather than the normal socket connection. The other side of this conversation uses the content type to understand how to deserialize when compact vs binary are both supported.


2230 - OneDrive list

OneDrive list

Adds 'onedrive list' command. Closes #2230

Thank you @vipulkelkar ! We will review it soon 😊

Thanks for the review @waldekmastykarz , will try to be more mindful of those little adjustments in the next one,

Merged manually. Thank you! 👏


Project Approval page for system administrators

User Story
As a System Administrator, I need to carefully enter critical information to confirm the approval of a project, so that I can be sure I haven't made any mistakes before I send it through to complete the setup for the Data Sponsor.
Context
Following up on the early work in #443 and #475, we now need a refined way for System Administrators to approve projects in the Presentation Layer. While the Collection ID from MediaFlux is the key marker of a project's existence, it is not the only bit of information we need to record. Crucially, a System Administrator needs a way to alter the Project Directory and Storage Capacity, as well as to fill out the provenance information. We'll have to streamline this to scale up, but for now, my vision is for assisted manual input: pre-fill everything but the Collection ID and the Event Note, but allow them all to be edited.

Completion Criteria

[ ] The page is titled "Project Approval: " with the title of the project appended
[ ] The first section is "Metadata Highlights," which displays the following project metadata fields for the System Administrator's reference:

[ ] Data Sponsor
[ ] Affiliated Department(s)
[ ] Project Directory (requested)
[ ] Title
[ ] Project ID
[ ] Storage Capacity (requested)
[ ] Submission

[ ] The section is "Project Approval Confirmation Information," which contains the following fields:

[ ] "Confirm Project Directory"

[ ] Allows the System Administrator to change the approved directory from what is given by the request form
[ ] Displays "/tigerdata/" followed by two subfields shown for data entry convenience, segmented by "/"
[ ] The first subfield is pre-filled with the value for the super-project directory (generated automatically from the DataSponsor's NetID)
[ ] The second subfield is pre-filled with the value entered by the Data Sponsor in the request form
[ ] The value for the full path is then constructed as "/tigerdata/[subfield1]/[subfield2]"

[ ] The value as approved gets the attribute "Approved"

[ ] "Confirm Storage Capacity"

[ ] Allows the System Administrator to change the approved capacity from what was given by the request form
[ ] There is one field, which allows a positive integer only, followed by a displayed unit of "GB"
[ ] The field is pre-filled with the appropriate value from the request form (in GB units)

[ ] E.g., if the request form entered "100 TB" it is translated to 100000 GB

[ ] The value is stored as a string constructed from the integer and the unit (e.g., "100000 GB" or "500 GB")

[ ] The value as approved gets the attribute "Approved"

[ ] "Collection ID"

[ ] This field is not pre-filled
[ ] A value is required, and it must be a positive integer

[ ] "Approver"

[ ] Prefilled with the NetID of the System Administrator who opened the page
[ ] Can be edited, but the value must be the NetID of an eligible System Administrator
[ ] Maps to the provenance field "Submission" :  "ApprovedBy"

[ ] "Approval Date-Time"

[ ] Pre-filled with the ISO 8601 timestamp at the moment the page was opened in US Eastern time
[ ] Can be edited, but must conform to ISO 8601 (with the time zone)
[ ] Maps to the provenance field "Submission" :  "ApprovalDateTime"

[ ] "Event Note"

[ ] Optional and not pre-filled
[ ] Two subfields:

[ ] "Type": given by a dropdown menu with the possible values of "Quota", "Tier", and "Other" (no selection by default)
[ ] "Message": a free-text field (blank by default)

[ ] Maps to the provenance field "Submission" :  "EventNote", with the following subfields:

[ ] "NoteBy" filled by the NetID of the System Administrator who opened the page
[ ] "NoteDateTime" filled by the ISO 8601 timestamp at the moment the "Confirm Approval" button is clicked
[ ] "EventType" filled by the "Type" selection
[ ] "Message" filled by the "Message" box entry

[ ] There are two action buttons at the bottom of the page:

[ ] "Confirm Approval" first performs a validation check when clicked

[ ] If any fields are invalid, they are highlighted on the page with no further action taken
[ ] If all fields are valid, then the project is approved

[ ] A provenance event is logged with the approver and approval date-time as entered, as well as the Event Note, if present
[ ] The provenance "Status" is updated to "Approved"
[ ] A background job to confirm the connection with MediaFlux is kicked off (see #553)
[ ] The user is taken to the "Project Approval Received" page (#575)

[ ] "Cancel" returns the System Administrator to their dashboard without making any changes

This ticket should be evaluated to be broken out into smaller, workable tickets.

Hey team! Please add your planning poker estimate with Zenhub @bess @hectorcorrea @JaymeeH @jrgriffiniii @kelynch @leefaisonr


Q: What are the performance/power implications of using this?

Hi,
thanks for this nice project. It surely fills a need I didn't know I had. 😉  But now I find it really nice to be able to declutter my home dir.
But before I go along and throw it at every possible "intruder": do you know or have an estimate about the impact on the performance of the programs run within boxxy? And does it have any noticeable impact on the battery life of mobile devices?
Would be nice to know. :)

afaik the impact is literally zero -- it should be exactly as much impact as unshare -rm unless you're using the ptrace features.


Create issue

Description of changes:
Support to create github issue defined repository if the chart validation passed.
It expect as input a yaml file. See example in create-issue-test.ts and its used file

Closing this as is included in #3


Update scripts that pull compose file for name change

See https://github.com/edgexfoundry/edgex-compose/issues/70
On hold until above issues has a PR ready to be merged.

TAF compose files do not contain the release name, so no changes are required.

Fixed by PR #387.


Question about the calibration board size

Hi. I want to downsample the fisheye images and need to recalibrate the intrinsic. So I would appreciate it if you can provide the size of the calibration board in the calibration sequence.

Hi, it's 10cm squares.


Use a JSON form generator

Utilize a JSON form generator, such as https://github.com/brutusin/json-forms .
In turn, use this to eliminate some of the modals in DJ Controls; use a single blank modal for forms and use the form generator to make, validate, and process the forms.

[ ] Implement

[ ] Test

Using jsonform instead; json-forms does not have any easy form submission mechanism... though I really like how it uses array inputs (jsonform array inputs are ugly).


[QUESTION] How do I start this?

Yes, I read that I have to run the main.py script. But I can not. Is it, because it is not on my main drive?
I can not use the cd command on the command prompt to go to the directory.
HELP PLS
edit: Well, I installed Python now. I can start the script, it opens, and then it closes again. can not see anything - so if there are any problems shown in that cmd prompt that opens for a millisecond, then I can sadly not tell you them.
What do I have to do now? The bots in dota behave normally - all but anti-mage do the regular valve bot stuff...

you probably don't have the necessary python modules installed as listed in requirements.pip
Install all of them using "pip install " in a command shell.


more user friendly result for outgoing-dialstatus-BUSY, -CHANUNAVAIL, -CONGESTION?

Are we handling busy signals appropriately? Do we need to play the signal? Are we just hanging up?
when we fail an outgoing call, give an informative technical difficulty message, then prompt user for keypress to leave a message for the operator.
Which are the failed statuses? BUSY CHANUNAVAIL CONGESTION? At least one of these mean a busy signal, which we don't want to change.
we get BUSY when twilio is throttling us (when we exceed 1 outgoing call per second per something). I thought this meant we got a busy signal, maybe it's also used for that? Can we tell which is which?
Log from calling with the same twilio everything from prod and a dev vm:
    -- Executing [s@macro-dial:2] Dial("SIP/667-0000000c", "SIP/+15038419766@twilio-termination,,g") in new stack
  == Using SIP RTP CoS mark 5
    -- Called SIP/+15038419766@twilio-termination
    -- Got SIP response 486 "Busy Here" back from <IP_ADDRESS>:5060
    -- SIP/twilio-termination-0000000d is busy
  == Everyone is busy/congested at this time (1:1/0/0)
    -- Executing [s@macro-dial:3] Goto("SIP/667-0000000c", "s,BUSY") in new stack
    -- Goto (macro-dial,s,12)
    -- Executing [s@macro-dial:12] NoOp("SIP/667-0000000c", "") in new stack
    -- Executing [s@macro-dial:13] NoOp("SIP/667-0000000c", "") in new stack
    -- Executing [s@macro-dial:14] NoOp("SIP/667-0000000c", "") in new stack
    -- Executing [s@macro-dial:15] Macro("SIP/667-0000000c", "metric,outgoing-dialstatus-BUSY") in new stack
    -- Executing [s@macro-metric:1] AGI("SIP/667-0000000c", "metric.agi,outgoing-dialstatus-BUSY") in new stack
    -- Launched AGI Script /opt/asterisk/var/lib/asterisk/agi-bin/metric.agi
 metric.agi,outgoing-dialstatus-BUSY: CALLERID(number)=+15034448759, UNIQUEID=1484973089.12, CHANNEL=SIP/667-0000000c, name=outgoing-dialstatus-BUSY
    -- <SIP/667-0000000c>AGI Script metric.agi completed, returning 0
    -- Executing [s@macro-dial:16] Hangup("SIP/667-0000000c", "") in new stack

at least for twilio, we are not getting that status for a circuit busy response, so we should be able to retry/notify on it.
Could do the same tryagain thing we do for CHANUNAVAIL/CONGESTION
   -- Executing [s@macro-dial:2] Dial("SIP/667-00000005", "SIP/+18002064972@twilio-termination,,g") in new stack
  == Using SIP RTP CoS mark 5
    -- Called SIP/+18002064972@twilio-termination
    -- SIP/twilio-termination-00000006 is making progress passing it to SIP/667-00000005
[Jan 21 02:59:41] WARNING[10585][C-00000003]: app_dial.c:1546 wait_for_answer: Unable to write frametype: 2
  == Spawn extension (macro-dial, s, 2) exited non-zero on 'SIP/667-00000005' in macro 'dial'
  == Spawn extension (macro-filterdial, s, 5) exited non-zero on 'SIP/667-00000005' in macro 'filterdial'
  == Spawn extension (outgoing-dialtone,<PHONE_NUMBER>, 1) exited non-zero on 'SIP/667-00000005'

for busy signal, we hear the tones, then if user does not hang up, we get CONGESTION, which tries again.
Do we want to try again on CONGESTION?
This probably means that BUSY is OK to try again and then notify.

this seems to say that congestion is a call setup problem, busy is when callee cannot accept the call. This makes it seem like we should try again for congestion but never for busy.
http://www.voip-info.org/wiki/view/Asterisk+variable+DIALSTATUS
If so, that's bad, because we can't tell a twilio ratelimit from callee busy. Maybe we need a rate limited call queue on our end.

twilio is not giving us BUSY, they are giving us this
<--- SIP read from UDP:<IP_ADDRESS>:5060 --->
SIP/2.0 500 Trunk CPS limit exceeded
CSeq: 103 INVITE
Call-ID: 2aef9630759570ef5a97764a51df5b66@<IP_ADDRESS>:5060
From: <sip:+15034448759@<IP_ADDRESS>>;tag=as18f05dd8
To: <sip:+15038419766@futel.pstn.twilio.com>;tag=98481305_6772d868_e2db0826-6257
-4734-9bbc-675258008c96
Via: SIP/2.0/UDP <IP_ADDRESS>:5060;received=<IP_ADDRESS>;branch=z9hG4bK3cbe4af6;rpor
t=61247
Server: Twilio
Contact: <sip:<IP_ADDRESS>:5060>
X-Twilio-Error: 32001 Trunk CPS limit exceeded.
Content-Length: 0

<------------->
[Jan 24 06:41:04] VERBOSE[9489] chan_sip.c: --- (10 headers 0 lines) ---
[Jan 24 06:41:04] VERBOSE[9489][C-00000c27] chan_sip.c:     -- Got SIP response
500 "Trunk CPS limit exceeded" back from <IP_ADDRESS>:5060

we want to try again and do the failure things on CHANUNAVAIL CONGESTION, any other?
we should also delay 1s before trying again
we don't want to try again or do the failure things on BUSY, we may need to play tones, maybe not?


chore(main): release 2.7.2

:robot: I have created a release beep boop
2.7.2 (2024-03-06)
Performance Improvements

♻ 使用环境变量实现接口域名切换 (b571d42)
♻ 更新用户目录名称 (c2b7f5e)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/viarotel-org/vite-uniapp-template/releases/tag/v2.7.2 :sunflower:


Blueprint creation should have name and description fields on top

Also, name and description are required fields but no message pops up until you enter a value for the first time, a message box can pop up telling user the same


Calculate time between actions for a User Journey and plot in a table/Sankey

Added !PU(timeonpage)


inference.py outputs ground truth video rather than the generated one.

I've notice that vico_challenge_baseline/PIRender/inference.py outputs the ground truth video rather than the generated one. It should be a problem with write2video() function. After I replace this function with the original one in vico baseline, the output seems to be correct. Please consider fixing this.

Thank you for kindly remind! @hzwer Please check this


[3pt] East Fork of Carson River Showing Significant inundation with very low flows (Vlab #87308)

From Tim Bardsley:
"Hi, The East Fork of the Carson River nr Gardnerville NV (USGS 10309000) has been flowing well below median, and the RFC has steady flow for the 5 day forecast, but the 5 day replace and route max inundation extent has some roads and neighborhoods impacted."
Reference Vlab Ticket for screenshots.

HUC: 16050201
Some NWM featureids: 8920165, 8920193, 8921831, 8921899, 8921907

HUC: 16050201
Some NWM featureids: 8920165, 8920193, 8921831, 8921899, 8921907

I produced the FIM <IP_ADDRESS> MS (calb) outputs for this area using the flows from the NWM RnR flows copied from WaterView on 1/29/21. The FIM3 results look very similar to those in FIM2. Likely causes for the over-estimation are missing bathy and large Manning's n.

I produced the FIM <IP_ADDRESS> MS (calb) outputs for this area using the flows from the NWM RnR flows copied from WaterView on 1/29/21. The FIM3 results look very similar to those in FIM2. Likely causes for the over-estimation are missing bathy and large Manning's n.

Nice find, Ryan. I've added you as the assignee on Vlab. Feel free to respond to Tim when ready.

Nice find, Ryan. I've added you as the assignee on Vlab. Feel free to respond to Tim when ready.

This issues addressed in this card are slated to be addressed by: #32 (bathymetry prototype solutions), #116 (Manning's global calibration), & #212 (Manning's local calibration override)


docker stack restart option is ignored

Steps to reproduce the issue:

Create next Dockerfile in "test" directory:
FROM debian
CMD echo "test"

Go to "test" directory and execute next command (build image): "docker build -t test:latest ."

Create docker stack file with name "docker-cloud.yml":
version: "3"
services:
test:
image: test:latest

Execute next command: "docker stack deploy -c docker-cloud.yml services"

Wait several seconds and execute "docker ps -a".

Expected:
One exited container "test:latest" because default restart option value is "no" according documentation.
Actual:
Several exited containers with "test:latest" tag.
Docker version:
Client:
Version:      17.03.0-ce
API version:  1.26
Go version:   go1.7.5
Git commit:   60ccb22
Built:        Thu Feb 23 10:54:03 2017
OS/Arch:      linux/amd64
Server:
Version:      17.03.0-ce
API version:  1.26 (minimum version 1.12)
Go version:   go1.7.5
Git commit:   60ccb22
Built:        Thu Feb 23 10:54:03 2017
OS/Arch:      linux/amd64
Experimental: false`
Docker info:
Containers: 5
Running: 5
Paused: 0
Stopped: 0
Images: 16
Server Version: 17.03.0-ce
Storage Driver: overlay
Backing Filesystem: extfs
Supports d_type: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
Volume: local
Network: bridge host macvlan null overlay
Swarm: active
NodeID: 06qm3mlo2dtnnrt0lebbfqkiz
Is Manager: true
ClusterID: yonkvlxzljjjztpxzcuc3fnrn
Managers: 1
Nodes: 1
Orchestration:
Task History Retention Limit: 5
Raft:
Snapshot Interval: 10000
Number of Old Snapshots to Retain: 0
Heartbeat Tick: 1
Election Tick: 3
Dispatcher:
Heartbeat Period: 5 seconds
CA Configuration:
Expiry Duration: 3 months
Node Address: ...
Manager Addresses:
Runtimes: runc
Default Runtime: runc
Init Binary: docker-init
containerd version: 977c511eda0925a723debdc94d09459af49d082a
runc version: a01dafd48bc1c7cc12bdb01206f9fea7dd6feb70
init version: 949e6fa
Security Options:
seccomp
Profile: default
Kernel Version: 3.10.0-514.10.2.el7.x86_64
Operating System: CentOS Linux 7 (Core)
OSType: linux
Architecture: x86_64
CPUs: 2
Total Memory: 3.695 GiB
Name: q4de3csye47
ID: 2FXY:XP3H:UWFL:3R7U:CF5H:THXJ:XC5K:SZTS:U4LV:AY7D:3JNV:FOJF
Docker Root Dir: /docker
Debug Mode (client): false
Debug Mode (server): false
Http Proxy: ...
Https Proxy: ...
No Proxy: ...
Registry: https://index.docker.io/v1/
Experimental: false
Insecure Registries:
...
<IP_ADDRESS>/8
Live Restore Enabled: false

@aleskurlovich The reference links is for Docker cloud "defaults" (https://docs.docker.com/docker-cloud/apps/stack-yaml-reference/). It is not the case when you use docker stack deploy, the defaults are the same as swarm mode — and in swarm mode, the default is restart always. If you want to make sure it does not restart, you shouldn't rely on defaults value but be explicit about it.
That being said, it feels a little bit confusing having https://docs.docker.com/compose/compose-file/ and https://docs.docker.com/docker-cloud/apps/stack-yaml-reference/ — and we might need to update documentation to be more clear 👼

/cc @mstanleyjones @dnephin @thaJeztah

Thank you! I use right documentation and it works.

@londoncalling do you think the Compose docs could be improved?

@mstanleyjones probably yes. I'll look at this now.

@pkennedyr @KickingTheTV @ManoMarks
Can you all scan through the discussion in this issue?
The gist of it is confusion between the definition for restart in the Cloud stack YAML reference here, and restart policy settings in Compose file v3 here.
@aleskurlovich needed the latter, which worked after he tried it.
The larger issue is confusion between the two files I think. I have several questions:

Does the Cloud stack YAML need updates per the latest Compose v3 file (or are they completely independent?
Should we include several notes in Cloud stack YAMl to redirect users to the latest Compose file? (Both at the top of the file, and at each config option?)
Is there further confusion that could result from competing concepts of Cloud stack YAML and docker-stack.yml (a type of Compose file for v3)? If so, what's the best way to address this?

Let me close this ticket for now, as it looks like it went stale.


Show All Buttons Comment/Request Changes/Approve when there is enough space

Currently it looks like this:

I think it would be nice if it looked like this when there is enough horizontal space:

I think I lose some mental energy when I want to approve, but I have to first click on the drop down of "Request changes".

@hbons in case you also have thoughts on this!

It's a bit funny because I actually just removed these in the full sized view: https://github.com/microsoft/vscode-pull-request-github/issues/5334

@bpasero you recently requested that I move the three buttons into the dropdown menu in the full editor webview in https://github.com/microsoft/vscode-pull-request-github/issues/5334. Now, I've gotten the request to expand the dropdown out into 3 buttons when space permits in the sidebar webview. If I make this change in the sidebar, I would also move the 3 options back into 3 buttons in the editor webview, since space always permits there. Do you have any opinion on this?

I still like and prefer the dropdown for reducing UI complexity.

@benibenj also noted that it is difficult to find the approve button.

I still like and prefer the dropdown for reducing UI complexity.

I agree that it reduces UI complexity, but I would argue that this comes at the cost of increased mental complexity.
"Approve" and "Request Changes" are not suboptions of "Comment", they are primary actions with opposite effects.
Especially when "Request Changes" is pre-selected, it causes mental resistance to click on its drop-down to select "Accept", as they have opposite effects.

To verify:

Install the pre-release version of GitHub Pull Requests.
Open a folder that contains a GitHub repository and some pull requests.
Check out a pull request that isn't yours
Open the active pull request view (this is the webview in the sidebar) and drag it larger and smaller. Verify that when it's "small" you get one button with the drop down caret for completing your review. Verify that when it's "big enough" you get all three buttons in a row (comment, approve, request changes).


@Lazy is not respected in resolvers

Describe the bug
If I declare a @Component @Lazy resolver (extends GraphQLQueryResolver), its constructor is called in application startup and them all beans dependencies of this resolver are also resolved.
To Reproduce
Declare a GraphQLQueryResolver with @Lazy.
Expected behavior
Should work as others lazy beans.
Desktop (please complete the following information):

OS: Windows 10
Java 8
graphql-spring-boot-starter 5.10.0

You're on a very old version of graphql-spring-boot-starter. We released version 8.0.0 today! Update that version too.
The starter does not create the beans themselves, the Spring ecosystem is responsible for creating these beans as you've defined them, so don't see how this is caused by this library.

My project is large and currently has incompatibilities with new versions of this library and graphql library which I was not able to resolve yet.
The Spring alone does respect @Lazy. Note that there is an .graphqls file referencing the resolver and seems the problem come from this relation.

Knowing Spring I'd advise to try to move away from those @Lazy annotations entirely, or put a layer in between somewhere to get more control. The fact that you're experiencing it using this library in your large project, doesn't mean it's caused by this library.

I do not use @Lazy, but I'm looking to use as startup is becoming slower and slower as I'm adding graphql resolvers.
Sorry if it's not problem of this library. The architecture of it is not very clear for me of what is its dependencies and what is the job of each one.

In the stack trace, it's graphqlSchema who is creating my lazy beans.


npm or yarn selection

Add another step to the cli to have the user pick their prefered package manager

Would also like to see pnpm in the selection menu


FullScreen: The button design includes distinct colors for both text and icon.

Marin said it is intentional

Marin said it is intentional

That's correct, but I haven't implemented it in my code.

WORK IN PROGRESS

Related PR posted!


Revert "Enable group suspension by default [run-systemtest]"

Reverts vespa-engine/vespa#16537

Suspecting that this broke some systemtests.


I still cannot run a cpp file

I followed the instructions in this article:

I cloned a github repository which contains a lot of c++ programs (each in its own subfolder).
I opened the main repository folder using "Open Folder". It appeared in my Solution Explorer tab.
I created a CppProperties.json file in the main repository folder, with a single configuration "x64-Debug" (automatically generated by Visual Studio).
I opened one of the cpp files and clicked "Ctrl+F5", but nothing happened.

What else should I do in order to run cpp files in Visual Studio?

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

ID: 315efb2e-e70d-1117-4916-522a5e9f629f
Version Independent ID: 796872b1-bba1-f7d8-5df9-d3f1c3ea905d
Content: Open Folder projects in Visual C++
Content Source: docs/ide/non-msbuild-projects.md
Product: visual-cpp
GitHub Login: @mikeblome
Microsoft Alias: mblome

@erelsgl You need to configure a launch.json file and tasks.json file. The document shows how to do that, at least for basic scenarios.

I tried to read it, but the number of options is overwhelming. All I need is to be able to run single CPP files (for demonstration in class). Is there a simpler guide for this need?

@erelsgl Sorry that it's confusing for you. We are working to make the workflow simpler in future versions of VS. In the meantime we now have this new tutorial: https://docs.microsoft.com/en-us/cpp/build/get-started-linux-cmake?view=vs-2017
If you still have questions, a great place to ask is the C++ VS Developer Community forum. There are several experts active on that site. Good luck!


pythonPackages.lxml: fix build on darwin

Motivation for this change
Update in #107408 broke it for Darwin.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[x] macOS (aarch64)
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

I believe this needs to be rebased on staging-next (i.e. some previous commits removed) and retargeted against staging-next.

I believe this needs to be rebased on staging-next (i.e. some previous commits removed) and retargeted against staging-next.

git rebase --onto=staging --onto=staging-next --onto=master HEAD~1

@SuperSandro2000 your command seems to rebase onto master:
$ git rebase --onto=origin/staging --onto=origin/staging-next --onto=origin/master HEAD~1
Current branch ivan/lxml-darwin is up to date.

$ git log -2
commit fd04a5e15401f1ae17791c5e0ac66cc423f0f70f (HEAD -> ivan/lxml-darwin)
Author: Ivan Babrou<EMAIL_ADDRESS>Date:   Wed Mar 17 13:32:48 2021 -0700

    pythonPackages.lxml: fix build on darwin

commit ea68396a78b6143fc97c8319ba29c408a80faf9f (origin/master, origin/HEAD)
Merge: 3643876a3c1 59617f748e5
Author: Sandro<EMAIL_ADDRESS>Date:   Thu Mar 18 01:41:03 2021 +0100

    Merge pull request #101510 from omasanori/libressl-3.2.2

    libressl_3_2: init at 3.2.5, libressl_3_1: 3.1.4 -> 3.1.5

It may be easier to ask the bot to do the right thing.

/rebase staging

staging-next, not staging?

@SuperSandro2000 your command seems to rebase onto master:

🤔 might be. Using the bot is not a bad idea.

/rebase staging-next


Partial mock for private setter method crash

I used OCPartialMockObject to verify whether some method invoked.
such as
id mock = [OCMockObject partialMockForObject:[DataProvider class]];
[mock updateSearchTerm:searchTerm]
[[mock expect] invokedMethod]

In updateSearchTerm method, the test code works well as I expected. But there also has an async block inside updateSearchTerm which is a black box for test code, so I don't know when the block will be invoked. Sometimes when test runs successfully, but async block will call back and crash with the following callstack:
2020-03-05T10:27:06.357Z [E]uncaught exception: -[DataProvider ocmock_replaced_setCurrentSearchTerm:]: unrecognized selector sent to instance 0x60000166a9a0 StackTrace :
(
	0   CoreFoundation                      0x000000011eb4c6fb __exceptionPreprocess + 331
	1   libobjc.A.dylib                     0x000000011dd7bac5 objc_exception_throw + 48
	2   CoreFoundation                      0x000000011eb6aab4 -[NSObject(NSObject) doesNotRecognizeSelector:] + 132
	3   CoreFoundation                      0x000000011eb51443 ___forwarding___ + 1443
	4   CoreFoundation                      0x000000011eb53238 _CF_forwarding_prep_0 + 120
	5   CoreFoundation                      0x000000011eb534cc __invoking___ + 140
	6   CoreFoundation                      0x000000011eb50a45 -[NSInvocation invoke] + 325
	7   OCMock                              0x000000013dda4ba8 -[OCPartialMockObject forwardInvocationForRealObject:] + 216
	8   CoreFoundation                      0x000000011eb511a1 ___forwarding___ + 769
	9   CoreFoundation                      0x000000011eb53238 _CF_forwarding_prep_0 + 120
	10  HostApp                             0x00000001065c5ddb __54-[DataProvider updateSearchTerm:traceInfo:]_block_invoke + 235
	11  libdispatch.dylib                   0x0000000120b3accf _dispatch_call_block_and_release + 12
	12  libdispatch.dylib                   0x0000000120b3bd02 _dispatch_client_callout + 8
	13  libdispatch.dylib                   0x0000000120b42720 _dispatch_lane_serial_drain + 705
	14  libdispatch.dylib                   0x0000000120b43261 _dispatch_lane_invoke + 398
	15  libdispatch.dylib                   0x0000000120b4bfcb _dispatch_workloop_worker_thread + 645
	16  libsystem_pthread.dylib             0x0000000120f18611 _pthread_wqthread + 421
	17  libsystem_pthread.dylib             0x0000000120f183fd start_wqthread + 13
) AXPContext:788

currentSearchTerm is a private property of DataProvider. it random fails and I have tried by calling "stopMocking", which is no use.
So how does OCPartialMockObject release, and why it can not find  private setter method?
OCMockito version: 5.1.2
OCMock version: '3.4.3'.

Please ask questions regarding the use of OCMock on StackOverflow using the ocmock tag.


sql: add new timestamp format for Sequelize

PR: https://github.com/cockroachdb/cockroach/pull/13159

We have closed this issue because it is more than 3 years
old. If this issue is still relevant, please add a comment and reopen
the issue. Thank you for your contribution to CockroachDB docs!


BLE SupportedServices issue

According to description BluetothLe.SupportedServices should have return list of services, but if we check output from this component with component "is a list?" it will never return value tru. Its the output from example:
The operation is in list? cannot accept the arguments: , [00001812-0000-1000-8000-00805f9b34fb], [00001801-0000-1000-8000-00805f9b34fb Unknown Service,00001800-0000-1000-8000-00805f9b34fb Unknown Service,0000180a-0000-1000-8000-00805f9b34fb Unknown Service,00001812-0000-1000-8000-00805f9b34fb Unknown Service,]
As we can see im trying to check if on services list is service with UUID 00001812-0000-1000-8000-00805f9b34fb and such service exists in list, but cant be found because service UUID is appended with 'Unknown Service' string.

The SupportedServices operation actually returns a string, not a list (ref: Javadoc). If you would prefer to lookup a UUID, you could use the DeviceServices property instead, which returns a list of lists that can be used with the lookup in pairs block.

Thanks.


Compile code for theme development

I am trying to update the daux theme for my dauxIO documentation, basically all I want to do is use the default daux theme but just change the colors. I run the following command to update and compile the less files
gulp watch

I then edit the "theme-blue.less" file and just change some of the colors and everything looks like its working, Gulp seems to be running the appropriate tasks, but no css changes are made, I just get the same css. Is there another Gulp command I need to run?

Hello, no there is no other command to run, can you check if there is anything different in the destination file ?

@onigoetz Silly mistake, I didn't realize that I had created a new theme and had not added the less resources to the gulpfile.js. I added them and they are working as they should now.


Clean up exporting of unconnected ports

Anything with the LCD's MISO not-connected causes compiler errors. Datalogger and high-side switch (lights controller) also has issues with unconnected CAN block pins, though is hacked around with explicit dummy block connections.
Solutions might include non-local compiler operations, or generating dummy blocks (either in the frontend or compiler).

New solution sketch: explicit not-connected - which is analogous to NC flags in schematics.
Implementation in IR: dummy block with port initialized to nop-defaults. Possibly some TBD syntactic sugar to generate these blocks - either as part of the port (eg, DigitalBidir.makeDummy()), or with infrastructural support (DummyBlock(DigitalBidir) - but where would this block definition live, automatically generated for all ports, for some ports, explicitly, or ???).
Alternatives considered:

Remove unused exports - but requires chasing through the export and link chain, and would require deleting (or ignoring) elements from the IR that are not local to the block with the disconnected port. Likely difficult to implement and the result can be confusing.
Automatically generate dummy blocks: requires core compiler support / refactoring (more complexity in an already somewhat-complex system), also wouldn't catch unintentionally disconnected ports (one of the more useful aspects of schematic ERC).


Ensure training pipeline is deterministic

In order to decrease sources of error we want to make sure the training pipeline creates a deterministic ordering of the dataset batches and other elements regarding initialisation. This will help to track down the sources of loss spikes i.e. malformed samples.
See this experiment for more information. We might want to check out this.

In the current non-nightly releases of PyTorch the setting above causes training to hang and is a known issue https://github.com/huggingface/transformers/issues/22363, going to try torch.backends.cudnn.deterministic = True.

Using torch.backends.cudnn.deterministic = True and model.eval() to counteract dropout the loss for each of the first N batches became more similar but not completely consistent.