[ALLUXIO-1837] Removed extra empty lines to fix the build failure

Can one of the admins verify this patch?

ok to test

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/9046/
Test PASSed.

Merged build finished. Test PASSed.

LGTM. @Slyrich thanks for the improvement. Going forward, you don't have to close and reopen a new PR, but push a commit to an existing PR.


Allow floating-point density for arbitrary SVG render size

First, thanks for an awesome library! I'm using it to create a tool for rendering PNG icons for React Native Android and iOS mobile applications from SVG source files. In general, everything goes smoothly until the SVG source file happens to be the wrong size relative to the desired output size. This of course feels a bit silly for Scalable Vector Graphics images.
The proper way to resize vector images with the sharp library seems to be to set the density constructor parameter when reading a vector image. Unfortunately the value is an integer, which means that it isn't possible to produce arbitrary sized SVG files. For example, it is not possible to render a 100x100 sized SVG image as a 72px x 72px PNG image. The target density would have to be
desiredOutputSize / inputSize * inputDensity = 100 / 72 * 72 = 51.84
which is an invalid value for the density parameter. It produces the error Error: Invalid density (1 to 2400) 51.84
The only workarounds I can think of is to either load the SVG initially at a higher resolution and the scale down the resulting raster to the desired size with the resize function, or alternatively edit the SVG source width and height attributes on the fly before passing the file to sharp. Both solutions have their issues, with the first possibly causing resizing artefacts in the resulting image, and the second requiring additional logic to parse XML and handle absolute SVG element dimensions with correct viewBox attribute.
Thus, I would like to ask if there is a way to support more flexible SVG resizing in sharp. I guess just passing floating point numbers as the density parameter probably won't be a good idea, since it has the possibility of rounding errors. Maybe it would be more appropriate to be able to pass a fixed width or height of the canvas the SVG image should be rendered on. I suppose passing both would be superfluous, since there shouldn't be a need to change the image aspect ratio.
Unfortunately though, it seems that libvips svgload only seems to support defining the rendered image size using density and scale values. Seems to support floating point numbers though, so maybe sharp could expose that API in some way then to allow a bit more control of the resulting image size.

Hello, the density parameter in sharp originates from *magick support where only integer values were allowed. As you point out, librsvg rendering can handle floating point values for this parameter so we should at least remove the integer-only limitation.
Happy to accept and/or help with a PR if you're able.

Okay, that seems like a reasonable start for this. I'll make a PR for supporting floating point numbers. Then I can try around and see how well it would work for my use cases. If the rounding errors seem improbable, then this change would probably be enough.

Created a PR for this at https://github.com/lovell/sharp/pull/1362.

v0.20.8 now available with your fix, thank you!

Awesome, thanks! Seems to produce correct size images in my test cases. Of course very small or large source images will hit the allowed density range of 1-2400, but that should be a less frequent issue though.


[Question] Height and width of an MRI?

fslinfo reports the following for my data:
data_type       INT16
dim1            176
dim2            256
dim3            256
dim4            1
datatype        4
pixdim1         1.000000
pixdim2         1.000000
pixdim3         1.000000
pixdim4         3.200000
cal_max         0.000000
cal_min         0.000000
file_type       NIFTI-1+

Then, what would be the height and width for this?

Width and height are with respect to the selected slicing plane (axial (default), coronal or sagittal). Now which dimensions in your data correspond to what direction depends on the orientation in which the data is stored (which is not reported in the output above, but is part of the header, or can be determined via the affine matrix). Maybe use FreeSurfer mri_info to get more information (specifically the orientation). Most likely the first dimension is Left or Right.

I used this manually, couldn't have any luck with unequal dim1, dim2, dim3

Anyways, here is mri_info you asked for:
Volume information for sub-1001_ses-01_desc-Xc_T2w.nii.gz
          type: nii
    dimensions: 176 x 256 x 256
   voxel sizes: 1.000000, 1.000000, 1.000000
          type: FLOAT (3)
           fov: 176.000
           dof: 0
        xstart: -88.0, xend: 88.0
        ystart: -128.0, yend: 128.0
        zstart: -128.0, zend: 128.0
            TR: 3200.00 msec, TE: 0.00 msec, TI: 0.00 msec, flip angle: 0.00 degrees
       nframes: 1
       PhEncDir: UNKNOWN
       FieldStrength: 0.000000
ras xform present
    xform info: x_r =   1.0000, y_r =   0.0000, z_r =   0.0000, c_r =     0.5000
              : x_a =   0.0000, y_a =   1.0000, z_a =   0.0000, c_a =     0.5000
              : x_s =   0.0000, y_s =   0.0000, z_s =   1.0000, c_s =     0.5000
Orientation   : RAS
Primary Slice Direction: axial

voxel to ras transform:
                1.0000   0.0000   0.0000   -87.5000
                0.0000   1.0000   0.0000  -127.5000
                0.0000   0.0000   1.0000  -127.5000
                0.0000   0.0000   0.0000     1.0000

voxel-to-ras determinant 1

ras to voxel transform:
                1.0000   0.0000   0.0000    87.5000
                0.0000   1.0000   0.0000   127.5000
                0.0000   0.0000   1.0000   127.5000
                0.0000   0.0000   0.0000     1.0000

Yes, for training FastSurfer we used conformed images which are 256 cubed. You should be able to conform them with the conform.py script or FreeSurfer's mri_convert -c if you want to.

if you want to.

Oh I didn't seem to have a choice. If that is indeed the case, the height and width parameters should be removed from the line I referenced.

Hi @m-reuter and @LeHenschel, any suggestions on how to un-conform the output (such that the new output is in the same space and dimensions as the original input image)?

Hello @isukrit , my name is Tashrif and I am an engineer, unrelated to this project, working with the kind of problem you mentioned. Although you haven't directed the question to me, I can help you with that.

how to un-conform the output (such that the new output is in the same space and dimensions as the original input image)

Are you familiar with ANTs tool for registration? antsRegistrationSyNQuick.sh and antsApplyTransforms are two commands that can be used in succession to do that. See these two lines for example.
For your second comment, can you reference relevant lines so I can possibly second/refute your understanding?

Hi @isukrit , no need to install Ants. This FS command:
mri_vol2vol --mov aparc+aseg...  --out aseginorigspace.nii.gz --targ originalinput.nii.gz --regheader --nearest
should do the trick.

@m-reuter and @tashrifbillah thanks for the info. I am using mri_vol2vol and this works as needed.

@m-reuter and @tashrifbillah thanks for the info. I am using mri_vol2vol and this works as needed.


SDP-915 Improve error messages during receiver registration process

What
Return better error messages during receiver registration.
Why
Help the receiver troubleshoot registration issues.
Known limitations
[TODO or N/A]
Checklist
PR Structure

[ ] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[ ] This PR does not mix refactoring changes with feature changes (split into two PRs otherwise).
[ ] This PR's title starts with the name of the package, area, or subject affected by the change.

Thoroughness

[ ] This PR adds tests for the new functionality or fixes.
[ ] This PR contains the link to the Jira ticket it addresses.

Configs and Secrets

[ ] No new CONFIG variables are required -OR- the new required ones were added to the helmchart's values.yaml file.
[ ] No new CONFIG variables are required -OR- the new required ones were added to the deployments (pr-preview, dev, demo, prd).
[ ] No new SECRETS variables are required -OR- the new required ones were mentioned in the helmchart's values.yaml file.
[ ] No new SECRETS variables are required -OR- the new required ones were added to the deployments (pr-preview secrets, dev secrets, demo secrets, prd secrets).

Release

[ ] This is not a breaking change.
[ ] This is ready for production.. If your PR is not ready for production, please consider opening additional complementary PRs using this one as the base. Only merge this into develop or main after it's ready for production!

Deployment

[ ] Does the deployment work after merging?

stellar-disbursement-platform-backend-preview is available here:SDP:      https://sdp-backend-pr69.previews.kube001.services.stellar-ops.com/healthAP:       https://sdp-ap-pr69.previews.kube001.services.stellar-ops.com/healthFrontend: https://sdp-backend-dashboard-pr69.previews.kube001.services.stellar-ops.com


Pt 162474517 plant hard fork

Clean up usage of macros for genesis/protocol versions.
Introduce an unconditional hard fork at height 1 (will need to change before release

https://www.pivotaltracker.com/story/show/162474517

@lucafavatella I can spend some time on this today. It seems like a reasonable approach.


CC0105 breaks multiply locals

Before:
bool recordsProcessedRaised = false, employeeProcessedRaised = false;

After:
var recordsProcessedRaised = false;
var employeeProcessedRaised = false;

@sm-g 'var' declarations doesn't allow multiple variables.

So maybe this rule should not warn such usage?

@sm-g Make sense.

@carloscds notice that the problem also happens for CC0001. Both should be fixed.


Make an Office Add-on

The VSTO format seems interesting.
It's unfortunate that a whole new project would probably be required.
Will look into it once I grab a copy of Microsoft Office.

Low priority -- I stopped using Office when I re-installed my Windows (advice not taking from FuckingClippy)


YAML validation breaking with certificates

certificate: |
  -----BEGIN CERTIFICATE-----
  MIIDWzCCAkOg...
  -----END CERTIFICATE-----

The first 3 --- are treated as delimiters, resulting in cannot unmarshal !!str --BEGIN....
This even happens when commented out!
Also the problem again the 3 hyphens,
certificate: "-----BEGIN CERTIFICATE-----
  MIIDWzCCAkOg...
  -----END CERTIFICATE-----"

certificate: '-----BEGIN CERTIFICATE-----
  MIIDWzCCAkOg...
  -----END CERTIFICATE-----'

runs into found unexpected end of stream
This worked in earlier versions, can't say exactly when it became an issue.

Hello @twolf-adc,
The issue is identified and resolved in PR #331.
Greetings,
@quintush


Investigate MRTK3's compatibility with Pico

Any update?

https://github.com/Phantomxm2021/PicoMRTK3 May be a reference


Q: how to link azure-storage-cpp

I apologize for possibly stupid question, but how do you link azure-storage-cpp library using cmake (on Linux)? find_package(azurestorage CONFIG REQUIRED) doesn't work and no cmake wrappers are exported to installed/<triplet>/share directory

Found a way:
# Azure stuff
find_package(cpprestsdk CONFIG REQUIRED)
find_path(WASTORAGE_INCLUDE_DIR was/blob.h)
find_library(WASTORAGE_LIBRARY azurestorage)
find_package(Boost REQUIRED COMPONENTS log)
find_library(UUID_LIBRARY uuid)
find_package(LibXml2 REQUIRED)

target_include_directories(tools PRIVATE ${WASTORAGE_INCLUDE_DIR})
target_link_libraries(tools PRIVATE ${WASTORAGE_LIBRARY} ${UUID_LIBRARY} ${LIBXML2_LIBRARIES} cpprestsdk::cpprest Boost::log)

vcpkg is an awesome idea... But it is so bleeding edge that the amount of PITA is reaching ludicrous levels sometimes :-)


Empirical correlation for DSI data

Compute empiricial correlation for a DSI dataset.
Dipy provides a DSI data set, taiwan_ntu_dsi, which could be used.
See http://nipy.org/dipy/examples_built/reconst_dsi.html
Resample this onto a multi-shell pattern, with a large number of bvals and the same large number of bvecs for each shell. Use bvecs provided by Hans Knutsson or Dipy's function for spreading out charged particles?

I can't find information about small and big delta for the NTU dataset referenced above. How to compute q-values then?

The provider of the NTU data does not know big and small delta. Anyway, it seems that the NTU is too small to be of real use. Unless we can find DSI data with more samples I'll consider this closed.


[Propose] [Need Discussion] Allow user to manually add calculated columns

What problem does this feature solve?
Users may found helpful when having additional calculated columns in which they could specify calculation formula by themselves.
What does the proposed API look like?
Have a calculated column adder which allows user to input the formula (e.g. referencing current stage data, referencing current row data, etc,.) and display the calculated column accordingly.

With the new v4 frontend, such feature shall be more feasible for implementation, but still unknown for user stories to support such feature with evidence. Leaving untouched for now and still open to discussion.


Feature Request - something like GRDB's ImmediateValueObservationScheduler

I hacked up my toy image board viewer app to use Blackbird, and compared it to a version that uses GRDB.
Blackbird branch: https://github.com/jackpal/HD/commits/Blackbird
GRDB branch: https://github.com/jackpal/HD/commits/main
The main UX difference between the two versions is that the Blackbird-based list views start out blank, while the GRDB-based list views start out filled in.
I think this is due to GRDB's value change publisher having a default "immediate" mode that fetches the value immediately upon creation.
It makes a big difference in the perceived speed of the app.
So I would like to request similar Blackbird functionality.
FWIW the Blackbird version of my app was less code and ceremony than the GRDB version. Good job!
But, the GRDB version does more. It uses foreign keys with cascading deletes to update the local database when posts and threads are deleted on the service. I haven't tried implementing similar functionality in the Blackbird version yet.

Thanks for the feedback!
 I'm still brainstorming how I might do them without adding a level of complexity that I don't want in Blackbird.
I downloaded your app and tested it out to see the immediate-vs.-blank initial state. If I understand correctly, the perceived-speed difference is because Blackbird's SwiftUI wrappers load asynchronously, rather than blocking the main thread until an initial load has completed. Is that what you're referring to? (Your app seems to be making network requests for the second-level views, making it longer than just database access, right?)
Assuming so, that's a design choice that I don't think I want to make an escape hatch or exception for. Maybe this is just me always fighting the last war, but Overcast to date (via FCModel) has serialized ALL database access onto the main thread, and it causes a bunch of little performance bottlenecks and problems throughout the app.
I designed Blackbird as the opposite extreme: blocking the main thread as little as possible, and forcing client access to be async.
The intention with the SwiftUI wrappers is to use their nil/.didLoad states to detect a "loading" state and display a placeholder UI for the (hopefully very brief) time that it's waiting for the database access. In my usage so far, it's extremely difficult to visually catch the UI in this state since it's usually so fast.


SharePoint library not visible

Describe the bug
SharePoint library not visible to select.
Component (please tell us which flow or app you are experiencing issues with):

Nurture component
Version [e.g. 1.40]
App or Flow [e.g. Maker Assessment | Import starter data cloud flow]
Environment Variable - Maker Assessment SharePoint Library"

To Reproduce
Steps to reproduce the behavior:

Import Nurture component
While importing solution update Environment Variable
Scroll down to 'Maker Assessment SharePoint Library'
Can't see library.

Expected behavior
When trying to select "Maker Assessment SharePoint Library", the new site we created  does not show up in the list of sites. Checked with permission, user is owner as well as Admin for SharePoint site. Also created library, which is not displayed within the Maker Assessment | Import starter data cloud flow.
Screenshots

Additional context
Add any other context about the problem here.

We have moved to a new dedicated repository for the CoE Starter Kit, and issues here will no longer receive attention. Please raise your issues here from now on:
https://github.com/microsoft/coe-starter-kit/issues
For this particular issue, you are facing a product issue and not an issue with the toolkit and so unfortunately we cannot support.


plot_kicad_sch not working because of non englisch language setting

Hello,
If the KiCad language is not English, the keyboard shortcut alt+f will not open the file menu but maybe another menu.
Therefore, ploting the schematic fails without any error message or info.
Maybe you can have a section about the KiCad language requirements in the Readme.md?

Hi @worldwidemv
Can you show me the output of this command?
locale -a

Can you check if you can launch the English version (with the English shortcuts) by doing this:
Note that the language you are going to use (English) has to be in the list above.
LANG=en_US.utf8 kicad

# or
LANG=C.utf8 kicad

# or
LANG=C kicad

Unfortunately, the language overrides are ignored by KiCad / eeschematic, probably because the have there own language system via Preferences-> Language.
Or maybe Python ignores that?
I would be ok, mentioning it in the install instructions. I am not sure, but the person installing this needs to add the en_EN locale anyway by hand ....
Changing the language in KiCad works.
Here is the output of locale -a after I had to fight with it to work properly again...
>$ locale -a
C
C.utf8
de_AT.utf8
de_BE.utf8
de_CH.utf8
de_DE
de_DE.iso88591
de_DE.utf8
de_IT.utf8
de_LI.utf8
de_LU.utf8
deutsch
en_AG
en_AG.utf8
en_AU.utf8
en_BW.utf8
en_CA.utf8
en_DK.utf8
en_GB.utf8
en_HK.utf8
en_IE.utf8
en_IL
en_IL.utf8
en_IN
en_IN.utf8
en_NG
en_NG.utf8
en_NZ.utf8
en_PH.utf8
en_SG.utf8
en_US.utf8
en_ZA.utf8
en_ZM
en_ZM.utf8
en_ZW.utf8
german
POSIX

Another idea was to override the hotkey file user.hotkeys.
But Alt+f seems to be a system shortcut since it is not in there...

Can you check again if Kicad opens with the other language when telling Kicad to do so from the command line?
For me, it works.
I am forcing it to use my mother language here

For you, check please if kicad opens in your language when you do this
LANG=de_DE.utf8 kicad

Can you also show me the output of the locale command?
This is mine.
➜  ~ locale
LANG=en_US.UTF-8
LANGUAGE=
LC_CTYPE="en_US.UTF-8"
LC_NUMERIC="en_US.UTF-8"
LC_TIME="en_US.UTF-8"
LC_COLLATE="en_US.UTF-8"
LC_MONETARY="en_US.UTF-8"
LC_MESSAGES="en_US.UTF-8"
LC_PAPER="en_US.UTF-8"
LC_NAME="en_US.UTF-8"
LC_ADDRESS="en_US.UTF-8"
LC_TELEPHONE="en_US.UTF-8"
LC_MEASUREMENT="en_US.UTF-8"
LC_IDENTIFICATION="en_US.UTF-8"
LC_ALL=

Here I installed the german since I did not have it on my system and then launched it setting the same variable

What is your operating system by the way?

I pushed a fix, it may work for you by default.

I will try it again tomorrow. You probably have the KiCad language set to Default. I will try that tomorrow. If it is set to German or English, then the LANG variable has no effect.
OS is ubuntu 22.04

Indeed, in mine, the language is the default. But why do you use the system in one language and kicad in another language? It does not make much sense to me.

I changed KiCad to English to get it to work.
And it is also sometimes easier because tutorials and so on are in English which makes following much easier....

Sure, then use the OS in English too.

Yes, sorry I thought this is not important, since the base system is Ubuntu22.04  and this is bash anyway.
Anyway, LC_ALL=C does the trick. With and without export.

In this case, it is important since it is not the default window manager. Nice it has worked, thanks.


Hopefully proper scheduler fix

I think this handles the loops introduced by backtracking, but I'm also not sure the backtracking is right in the first place.

Nice


improve app logging

Summary
We need improve and standardize format of logs records because current implementation has few issues which can do problem with integration of log collector:

"Main error" reason for terminate of application is printed with standard fmt.Println() when other errors use zap logger.
https://github.com/ironcore-dev/libvirt-provider/blob/main/provider/cmd/main.go#L18
Duplicated "main error" logs:Error: failed to initialize machine class registry: unable to unmarshal machine classes: error converting YAML to JSON: yaml: line    2: did not find expected alphabetic or numeric character
failed to initialize machine class registry: unable to unmarshal machine classes: error converting YAML to JSON: yaml: line 2: did     not find expected alphabetic or numeric character

We should use production configuration as default. So i recommend disable development level for zap logger as default settings because if you need it, you can use argument --zap-devel
Disable print of usage for every "main error" which terminated application. We can print usage, if users use wrong argument or required arguments are missing, but it doesn't make sense print usage for application runtime error.

Motivation
Easier integration log collectors with our application

@lukasfrank do you have any objections or advice before implementation? Replace of fmt.Println() will be little complicated, another points are small changes

@lukasfrank do you have any objections or advice before implementation? Replace of fmt.Println() will be little complicated, another points are small changes

In order to solve the main issue that the log-collector can't process an unformatted log , I would propose to init a setup-logger in the main.go like here:
https://github.com/ironcore-dev/ironcore/blob/main/poollet/machinepoollet/cmd/machinepoollet/main.go

@lukasfrank do you have any objections or advice before implementation? Replace of fmt.Println() will be little complicated, another points are small changes

In order to solve the main issue that the log-collector can't process an unformatted log , I would propose to init a setup-logger in the main.go like here: https://github.com/ironcore-dev/ironcore/blob/main/poollet/machinepoollet/cmd/machinepoollet/main.go

i tested you solution and really cannot work. Default controller-runtime logger is dummy logger which discard logs (no op) so it will hide errors, if you use wrong arguments or you don't use required arguments.
✘  10:15 ~/work/libvirt-provider [ 85-improve-app-logging-v2 | ✚ 1 …4 ] $ ./cmd -y
✘  10:15 ~/work/libvirt-provider [ 85-improve-app-logging-v2 | ✚ 1 …4 ] $


Funnel system to work as intended

distinction between independent funnel want and dependant funnel want

from start to end: if previous room independently wants funneled, then then the consecutive one will dependently want funneled.
From end to start: if the previous room wants independently funneled, then the following one will register as dependent. else if the previous room wants dependently funneled, the room will register dependently funneled.

Problem

when the first room reaches its funnel threshold only when other rooms have already done so (let's say alternative rooms incidentally have higher energy incomes or priorities) then the funneling will always end up split between multiple rooms
problem example

if the first room wants to be funneled but

Solution

the start must be contiguous, and dependent
n = no, i = independent want, d = dependent want, o = overdrive want
pre state: {n, n, n, n, n, i, n}
post state: {d, n, n, n, n, n, n}
final state: {f, n, n, n, n, n, n}
the independent want of any funnel target triggers the independent and actionable want of the first room while disregarding the want of the actuator

psuedocode
if (the first target wants independent funnel) then set it so and continue to next target(s) to potentially create a contiguous set of wanters
if (any independent funnel wanter) then set first funnel target to override funnel want

If anyone has ideas on how to do this system well, ideas are much welcome. Thanks.


Experiment divide&conquer on ruptureit

First try by separating the alphabet of the secret (and reflection), the alphabet of the context, and the alphabet of the padding in our zero-noise endpoint.

Why doesn't divide & conquer work?

Depends on #111


Add scenarios (/ scenario group) to extdata

This should probably be merged as its used to build produciton

if it's in production, let's merge it, but I dislike it

https://github.com/epidemics/covid/issues/405

I dont understand what you mean. It was already like this, I just added the name and description fields.

You are right, it's apparently late enough for me, I understood it wrong...

I am going to merge it as I am going to work on https://github.com/epidemics/covid/issues/399 so would like to prevent conflicts.


[automation] Update go release version 1.18.4

What
Bump go release version with the latest release.
Further details
See changelog for 1.18.4
Automatically generated from https://apm-ci.elastic.co/job/apm-shared/job/bump-go-release-version-pipeline/

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2022-07-25T21:18:36.996+0000

Duration: 10 min 59 sec

Test stats :test_tube:

Test
Results

Failed
0

Passed
219

Skipped
0

Total
219

:robot: GitHub comments
To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.


revised the contact form button without conflicts

Last night i added expandable contact form button which had conflicts, doing it again without conflicts.
Added a "Contact Us" Button at the footer.
Once the button is clicked, it expands into a collapsable in-page form.
The contact us form is basically the requests of the users who want reviews of new places which isnt already included in the page.
The Button when its collapsed

The Button when its expanded

thanku for contributing


implement ApplyBQSR on dataflow

requirement for this ticket is to replicate the current gatk4 functionality (and make tests pass). We'll work from text-based recalibration tables for now.

@jean-philippe-martin can you describe the current status?

Works with local files, and a PR for cloud functionality (complete with passing test) is under review (it's PR #595). The one thing left to do is to adapt to the skeleton once that's merged in.

is this done?

This issue was moved to broadinstitute/hellbender-dataflow#4


Hacktoberfest 2022 Leetcode solution

Accepted Solution
Problem Link
https://leetcode.com/problems/sliding-window-maximum/

Please give the hacktoberfest accepted label also if accepted


Added function to determine if a window is fullscreen or not

I had to make a custom implementation for macOS, because for some reason SDL2 has issues with fullscreening in general on macOS.
I didn't add a setFullscreen function either, because it appears to be unstable across platforms.
This is mainly to address onivim/oni2#1462. This will allow us to differentiate between the window being fullscreen (i.e. taking up an entire desktop non-windowed) and maximized (i.e. taking up an entire desktop windowed).

Tested this out, and it works great - just what we need to solve https://github.com/onivim/oni2/issues/1462 ! 💯


Space does not let save using "&:"

I can't save the File when is like that:

.div {
  &: after {}
}

or:

.sample {
  &: hover {
    color: red
  }
}
Should Be:

.div {
  &:after {}
}

.sample {
  &:hover {
    color: red
  }
}
A space is created and does not let save the document.  (just usign &:)

I think we have this as a bug report already, but I'll double check

This does not repro for me.  I've tested on the website ( ) and this remains unchanged:
<style>
    .div {
        &:after {}
    }
</style>

I'm going to move this to 1.6.0 for further review, but I'm pretty sure it is fixed and working correctly in 1.5.4.


Not possible to use shorthand route handlers if RestSerializer is used?

Hi folks,
I already posted this issue on ember discuss and stackoverfow, but I really would like to receive as many answers/opinions as possible, because I have to find the truth. 🔮
I set up a simple Ember Twiddle to show you my error that is occurring when trying to update a model.
According to the docs, I created a shorthand route that should handle the PUT request.
It does, but with the error: Your handler for the url /api/shops/1 threw an error: Cannot convert undefined or null to object
When using the JSONAPISerializer, everything is working with shorthands (mirage/config.js) and I'm able to update models, but in my case I have to use the RESTSerializer with serialized IDs in the responses.
The request payload when I'm sending the model's attrs are without Id at the end of the property name, f.e.:
// attrs object in PUT request
{
  name: "Shop 1",
  city: "1" // belongsTo relationship,
}

Now Mirage is trying to find those properties on the respective database model that has to be updated, but cannot find it, because in the database it's cityId and not just city...
Yeah, I also found this solution in this issue report and it's working, but I was hoping I could avoid something like this, because as far as I can remember, in previous versions of ember-cli-mirage (v0.1.x) it was also not needed to override the normalize method in the serializer to be able to make use of the RestSerializer with serializedIds...
My question is: Is there a way to stick to shorthand route handlers only, or do I really have to override the normalize method, writing a helper or other custom solution only because I have to use the RestSerializer?
That would be really sad, but at least I would know then.
Thanks for your support!

Answered on StackOverflow as well, but pasting my response here as well for future reference for other folks:

Short answer: it looks like you need the custom serializer for now until the bug fix for it is merged.

Long answer: that issue looks to be an issue that occurred in the 0.2 -> 0.3 upgrade for Mirage, likely because of underlying DB changes made in Mirage. It'll probably get fixed, but for now you'll need to work around it.

I'll go ahead and close this issue, feel free to speak up if you think it needs to be re-opened.


choco install of openjdk arm not obvious

installing the arm version of openjdk is not obvious.  can you please assure that
choco install openjdk

installes the correct binary?
currently it downloads the x64 version for both, openjdk, and microsoft-openjdk:
Downloading openjdk 64 bit
  from 'https://download.java.net/java/GA/jdk21/fd2272bbf8e04c3dbaee13770090416c/35/GPL/openjdk-21_windows-x64_bin.zip'

Downloading microsoft-openjdk 64 bit
  from 'https://aka.ms/download-jdk/microsoft-jdk-21.0.4-windows-x64.msi'

@soloturn - This really is a request for the chocolaty team. It's not something that the Java Engineering Group at Microsoft has capacity to support I'm afraid!


Route events to multiple slack channels

Motivation
Background:
One of the most common rules we have alerts on is Terminal shell in container. We really like this rule as it is an audit log of when a user accesses a k8s container.
While this rule is not  "security alert" which requires action, it is more of an "audit alert". E.g. Terminal shell in container is not the same severity as say Delete Bash History or Unexpected outbound connection destination
And at the moment all audit logs and severity event logs are going to a dedicated falco channel on our slack as it can be  lot of noise. In theory, I would like to be able to route specific events to go just to events-security slack channel instead or infra-audit channel.
Feature

Have the ability to route events to multiple slack channels based on (either) rule name or severity of alert

Hello,
This is a request we already had. I understand the need but it means a lot of changes.
Here my answer last time
A solution could be to use one the available FaaS Outputs (Kubeless, OpenFaas, Knative (Cloud Events), AWS Lambda) and code yourself the logic.

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle rotten

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close


Add Firebase analytics

Add firebase analytics
Sensor count up to 25
More than 25

Changes made on Android side:
added_tags - exact number of added sensors
claimed_tags - exact number of sensors claimed by user
offline_tags - not claimed and not shared to user (Bluetooth only sensors)
signed_in - (true/false) user signed in to cloud or not

Moving to done


Create faker generator to double and float

Summary by CodeRabbit

New Features

Introduced DoubleFaker for generating random double values for various measurements and rates.

Improvements

Updated generateValue function to use propertyName for improved consistency.
Enhanced value generation logic for String, Int, Double, and Float types using faker classes.

Refactor

Moved constants to dedicated constants package for better organization and maintainability.

Tests

Added DoubleFakerTest to verify double data values against predefined ranges.

@coderabbitio full review


fix issue #66 about pointer arithmetic with + - += -=

Fix issue #66 about pointer arithmetic.
It fixes the behaviour of the following cases:

pointer - integral
pointer - pointer
pointer += integral
pointer -= integral

and simplifies processing of:

pointer + integral

When void* types are involved, it warns the user and replaces it by char* (GCC extension)

Very cool! I'll manually merge your work! Thanks!

Implemented with some test cases --> b205d5247680552637968d9f9ffad908bbeb19ba

Codecov Report

Merging #81 into master will increase coverage by 0.03%.
The diff coverage is 81.52%.

@@            Coverage Diff             @@
##           master      #81      +/-   ##
==========================================
+ Coverage   78.49%   78.52%   +0.03%
==========================================
  Files         344      344
  Lines       44956    45082     +126
==========================================
+ Hits        35286    35400     +114
- Misses       9670     9682      +12

Impacted Files
Coverage Δ

ppci/cli/link.py
82.14% <ø> (ø)

ppci/lang/c/utils.py
96.80% <ø> (-0.07%)
:arrow_down:

ppci/lang/c/semantics.py
83.18% <73.19%> (-1.56%)
:arrow_down:

ppci/lang/c/context.py
83.88% <87.50%> (+0.16%)
:arrow_up:

ppci/lang/c/codegenerator.py
95.13% <94.59%> (+0.21%)
:arrow_up:

ppci/arch/arm/thumb_instructions.py
85.39% <100.00%> (+0.20%)
:arrow_up:

ppci/lang/c/nodes/types.py
99.45% <100.00%> (ø)

ppci/lang/c/preprocessor.py
88.97% <100.00%> (+0.02%)
:arrow_up:

ppci/arch/x86_64/instructions.py
81.23% <0.00%> (+0.70%)
:arrow_up:

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 d024ae9...9e2d6a6. Read the comment docs.

Issue fixed


incorporate data logger code within waterchem

Added loggerScripts branch and this contains new loggers subpackage. Added docstrings to loggerScripts.py, but still need to document the individual toolkits.


mlflow in the browser : some errors

Hello,
I found your lib and I like it.
I tried to use it in the browser (using webpack), but I'm getting this error
Uncaught (in promise) Error: TypeError: Failed to execute 'fetch' on 'Window': Illegal invocation

do you know how to fix this ?
I would think using isomorphic-fetch as you are doing may make it possible to use your lib both in node and the browser

I'm sorry, I've just noticed this issue right now.
Unfortunatelly mlflow api server doesn't provide any CORS headers on responce at this time, that's why it doesn't work.
We must prepair a proxy server to go with.
I'll maybe write this down into README. Thanks.


fix: estimateFeeBulk include skipValidate in accountInvocationsFactory

Motivation and Resolution
Fix: https://github.com/starknet-io/starknet.js/issues/1029
RPC version (if applicable)
Usage related changes
Development related changes
Checklist:

[ ] Performed a self-review of the code
[ ] Rebased to the last commit of the target branch (or merged it into my branch)
[ ] Linked the issues which this PR resolves
[ ] Documented the changes in code (API docs will be generated automatically)
[ ] Updated the tests
[ ] All tests are passing

LGTM


Messages are not sent to Group members in communicator

The Group membership is not populating while sending messages to Groups.

This is happening since last friday.

Hi @PersistentIDC,
Thank you for raising the query, can you please let me which version of the app you're using, also do you see any exceptions in the app insights, if so can you share the same for us to troubleshoot the issue.

we are using V4.0. We do not see any log as such. But the client secret had expired and we had created the new key and put it in configuration of the app. Post that this specific issue started. Before replacing the key the error was permission needed. contact your IT.

Hi @PersistentIDC,
Currently v5.0 released for CC, if you want to upgrade to latest version here is the link
In case you don't want to upgrade, seems the client secret expiration causing the issue, have you restarted the apps after updating the configurations.

Closing this issue due to inactivity, please open new issue if you've any other query.


(session) fix NereidsTracer shouldLog always true after set enable_nereids_trace from true to false

Proposed changes
Issue Number: close #xxx
when set enable_nereids_trace = true, NereidsTracer init and set shouldLog = true. ater set enable_nereids_trace = false, NereidsTracer.shouldLog always is true, it need set false when enable_nereids_trace = false
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


pre-training data about UniXcoder

In the paper, C4 dataset is used to pre-training UniXcoder. Which subset of C4 is used in paper? en?

How many pieces of data do you have in your training set?

from datasets import load_dataset
dataset = load_dataset("c4","en")["train"]


Terraform scan for sadcloud needs investigation

It appears like checkov is not running properly for this repo.

Issue raised with checkov - https://github.com/bridgecrewio/checkov/issues/295

This is resolved now


Infer the use of SSL

As suggested in issue #424, this PR implements inferring the use of SSL instead of using the UseSSL parameter

[X] Change details added to Unreleased section of changelog.md?
[X] Added/updated documentation and descriptions in .schema.mof files where appropriate?
[X] Examples updated for both the single server and small farm templates in the examples folder?
[X] New/changed code adheres to Style Guidelines?
[X] Unit and Integration tests created/updated where possible?

This change is

@ykuijs Since this is a breaking change, I'm going to hold off on it for a moment, pending what we come up with in the discussion about v2.0 stuff in #456 - I suspect we'll need to merge this to a new branch though.

What do you mean with a breaking change? I thought that you meant you planned to have them fixed before v2

@ykuijs By removing the UseSSL property we will break peoples schemas, there is no way around that, so this needs to be held off until we are ready to commit to releasing things as v2.0 - and what I raised as a point in that discussion is that it might be worth grouping all the breaking changes we have pending in issues in to one big release rather than releasing them one and a time.

Hello. Can I suggest we add a tag 'breakingchange' so it is easy to track these down?

@ykuijs Can you please re-open this one to merge in to the "preview-2.0" branch? We'll park any breaking changes there until we drop 2.0. We can still review this now and merge it in though.


bulk insert did not return and fill models primary key

bulk insert did not return and fill models primary key,  primary key is used in some situations

This has been discussed before and os not really possible. See #24 for a short discussion regarding how this would even be possible.
I'm not sure if and if so how gorm tackles this issue in v2.0 but the latest release do have support for bulk insert so you should try to use that release and raise any issues in the gorm repository


Publish workflow warnings

[CD (16.x)](https://github.com/m-h-c-t/mh-helper-extension/actions/runs/3358478715/jobs/5565500376)
Node.js 12 actions are deprecated. For more information see: https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/. Please update the following actions to use Node.js 16: actions/checkout, actions/setup-node, actions/upload-artifact, mikepenz/release-changelog-builder-action, actions/setup-node, actions/checkout

[CD (16.x)](https://github.com/m-h-c-t/mh-helper-extension/actions/runs/3358478715/jobs/5565500376#step:10:11)
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

For convenience.
https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
For even more convenience:
A workflow using save-state or set-output like the following
- name: Save state
  run: echo "::save-state name={name}::{value}"

- name: Set output
  run: echo "::set-output name={name}::{value}"

should be updated to write to the new GITHUB_STATE and GITHUB_OUTPUT environment files:
- name: Save state
  run: echo "{name}={value}" >> $GITHUB_STATE

- name: Set output
  run: echo "{name}={value}" >> $GITHUB_OUTPUT

this needs to be checked/fixed across all github actions, not just publish.

Closed via #367


fix: 'invalid order function for sorting' when nothing running

If nothing running:
Error executing Lua callback: ...s.nvim/lua/telescope/_extensions/tasks/picker/finder.lua:77: invalid order function for sorting
stack traceback:
	[C]: in function 'sort'
	...s.nvim/lua/telescope/_extensions/tasks/picker/finder.lua:77: in function 'get_tasks_ordering'
	...s.nvim/lua/telescope/_extensions/tasks/picker/finder.lua:108: in function 'order_tasks'
	...s.nvim/lua/telescope/_extensions/tasks/picker/finder.lua:25: in function 'available_tasks_finder'
	...sks.nvim/lua/telescope/_extensions/tasks/picker/init.lua:10: in function 'available_tasks_telescope_picker'
	...sks.nvim/lua/telescope/_extensions/tasks/picker/init.lua:38: in function 'picker'
	...telescope-tasks.nvim/lua/telescope/_extensions/tasks.lua:23: in function <...telescope-tasks.nvim/lua/telescope/_extensions/tasks.lua:21>
	...share/nvim/lazy/telescope.nvim/lua/telescope/command.lua:193: in function 'run_command'
	...share/nvim/lazy/telescope.nvim/lua/telescope/command.lua:253: in function 'load_command'
	...ocal/share/nvim/lazy/telescope.nvim/plugin/telescope.lua:108: in function <...ocal/share/nvim/lazy/telescope.nvim/plugin/telescope.lua:107>

I don't think this is the correct solution. Your fix would cause that only tasks that are currently running would be sorted, but we want to sort all tasks based on the timestamp of their latest run.
I think a better solution would be the following:

Replace:

  -- finder.lua:94
    if a_timestamp ~= nil and b_timestamp == nil then
      return true
    elseif a_timestamp == nil and b_timestamp ~= nil then
      return false
    end
    return a_timestamp == nil and b_timestamp == nil
        or a_timestamp > b_timestamp

With

    if a_timestamp ~= nil and b_timestamp == nil then
      return true
    elseif a_timestamp == nil and b_timestamp ~= nil then
      return false
    elseif a_timestamp == nil and b_timestamp == nil then
      return a < b
    end
    return a_timestamp > b_timestamp

I think that would solve the issue, as I think the following return causes it:
    return a_timestamp == nil and b_timestamp == nil
        or a_timestamp > b_timestamp
-- If both timestamps are nil, this would still try to compare timestamps, which are nil

return a_timestamp == nil and b_timestamp == nil
    or a_timestamp > b_timestamp

-- If both timestamps are nil, this would still try to compare timestamps, which are nil
If a_timestamp == nil and b_timestamp == nil then lua not eval a_timestamp > b_timestamp, no problem at all.

I change the code following your suggestion.


hi i am getting issue while creating react app please help anyone

C:\Project\testv\react-microsite>npm start

<EMAIL_ADDRESS>start C:\Project\gitlab-live\testv\react-microsite
react-scripts start

'react-scripts' is not recognized as an internal or external command,
operable program or batch file.
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>start: react-scripts start
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      C:\Users\prem\AppData\Roaming\npm-cache_logs\2020-03-26T08_37_51_325Z-debug.log
C:\Project\testv\react-microsite>npm install<EMAIL_ADDRESS>npm WARN deprecated<EMAIL_ADDRESS>Please update: there are crash fixes
npm WARN registry Unexpected warning for https://registry.npmjs.org/: Miscellaneous Warning SELF_SIGNED_CERT_IN_CHAIN: request to https://registry.npmjs.org/@babel%2Ftraverse failed, reason: self signed certificate in certificate chain
npm WARN registry Using stale package data from https://registry.npmjs.org/ due to a request error during revalidation.
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\react-scripts\node_modules\fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"x64"})
npm ERR! Unexpected end of JSON input while parsing near '...pCb9kW4Y0MFDCLX9mNCf/'
npm ERR!      C:\Users\prem\AppData\Roaming\npm-cache_logs\2020-03-26T08_38_46_468Z-debug.log

Update typescript.

already installed typescript in my project

Update it to the latest version


Add High Power Feature for RFM69HCW

#4 Add High Power
Datasheet

Added tx_level and tx_level_high_pwr methods to the rfm69 struct

Added enable_boost_mode and disable_boost_mode methods to enable and disable the high power transmit settings during transmissions where the tx level is >= 18

Please feel free to provide any feedback. I just wanted to get your input sooner than later, I tested it with two RFM69HCW radios and an RP Pico, and it seems to be working great. I'm not sure if there is a way to ID if the radio is a high power model or not, but if so it might be worth returning an err if the user tries to call tx_level_high_pwr on a radio that doesn't support high pwr.

As for the detection, I have unfortunately no idea if it's possible. I wonder what happens if you set wrong power level on non-high power RFM. Would it destroy the chip? If that might be the case maybe we should mark the high power function as unsafe and add warning to the documentation.


Text generation final touches

This simplifies a little the sequence of operations to better align on the transformers user experience, and adds the missing guide for generative models.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Client needs to validate if responses actually belong to topic

When alternate bucket is read, or when when there is no next update, the client will get back random data that doesn't decode as expected.
libpdb/Topic needs to maintain a checksum or otherwise know when a read is 'successful', and not advance state in that case.

This begins to be addressed in 57f39b9798dacc84c13fd04c38cbf6ac971b23af, where messages are signed, allowing subscriptions to validate that a message is part of the expected log.

This is now merged to master. new items in a log are signed by the writer, and validated by readers before they move on to the next sequence number.


Oracle Cloud Infrastructure and Python 3.8 End of Support

Describe the bug
Unable to upgrade the python version for the Oracle Cloud Infrastructure function app. Changing the stack version above Python 3.8 generates errors.
To Reproduce
Steps to reproduce the behavior:

Go to the OCI logs function app
Under Settings, select Configuration
In Configuration, select General Settings
Change the Python Version from Python 3.8 to 3.9.
Save the configuration
See error:
Result: Failure
Exception: ModuleNotFoundError: No module named '_cffi_backend'
Stack:   File "/azure-functions-host/workers/python/3.9/LINUX/X64/azure_functions_worker/dispatcher.py", line 505, in _handle__invocation_request
call_result = await self._loop.run_in_executor(
File "/usr/local/lib/python3.9/concurrent/futures/thread.py", line 58, in run
result = self.fn(*self.args, **self.kwargs)
File "/azure-functions-host/workers/python/3.9/LINUX/X64/azure_functions_worker/dispatcher.py", line 778, in _run_sync_func
return ExtensionManager.get_sync_invocation_wrapper(context,
File "/azure-functions-host/workers/python/3.9/LINUX/X64/azure_functions_worker/extension.py", line 215, in _raw_invocation_wrapper
result = function(**args)
File "/home/site/wwwroot/AzureFunctionOCILogs/main.py", line 46, in main
stream_client = oci.streaming.StreamClient(config, service_endpoint=MessageEndpoint)
File "/home/site/wwwroot/.python_packages/lib/site-packages/oci/streaming/stream_client.py", line 66, in init
signer = Signer(
File "/home/site/wwwroot/.python_packages/lib/site-packages/oci/signer.py", line 253, in init
self.private_key = load_private_key(private_key_content, pass_phrase)
File "/home/site/wwwroot/.python_packages/lib/site-packages/oci/signer.py", line 49, in load_private_key
backend = default_backend()
File "/home/site/wwwroot/.python_packages/lib/site-packages/cryptography/hazmat/backends/init.py", line 14, in default_backend
from cryptography.hazmat.backends.openssl.backend import backend
File "/home/site/wwwroot/.python_packages/lib/site-packages/cryptography/hazmat/backends/openssl/init.py", line 6, in
from cryptography.hazmat.backends.openssl.backend import backend
File "/home/site/wwwroot/.python_packages/lib/site-packages/cryptography/hazmat/backends/openssl/backend.py", line 113, in
from cryptography.hazmat.bindings.openssl import binding
File "/home/site/wwwroot/.python_packages/lib/site-packages/cryptography/hazmat/bindings/openssl/binding.py", line 14, in
from cryptography.hazmat.bindings._openssl import ffi, lib

Expected behavior
Function should run without error. Reverting back to Python 3.8 resolves the issue, however, we've received emails from Azure that community support for Python is ending 14-Oct-2024 and to transition to Python 3.9+.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Windows 11
Browser: Edge
Version: 123.0.2420.81 (Official build) (64-bit)

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Hi @DavidT8777 , Thanks for flagging this issue, we will investigate this issue and get back to you with some updates by 16-04-2024. Thanks!

Hi @DavidT8777,
Working on replicating the issue, will update you.
Thanks

Hi @DavidT8777,
We tried to replicate the issue,We deployed function app with manual deployement and also through arm template.
After that we changed python version from 3.8 to 3.9 ,but not getting this error - Exception: ModuleNotFoundError: No module named '_cffi_backend'.
Please provide details steps about how to configure function app and also provide python_packages folder.you will get python_packages folder with below steps

Go to storage account
Search with your functionapp name
Data storage -> Containers ->scm-releases
download file - scm-latest-functionapp
share downloaded file

Thanks

This function was originally installed via the Sentinel Content Hub with version 3.0.0. We've upgraded to version 3.0.1 due to issues we reported in #8313. We attempted the upgrade to python 3.9 from the General Settings tab on the function app.  Looking at the scm-releases container in the storage account, it's empty.

The function app is running and we have current logging from the function.
Thanks!

Hi @DavidT8777,
We updated function app packages, please follow below steps for configuration.

Go to Setting section of your function app

Click on Environment variables

Under the App setting click on WEBSITE_RUN_FROM_PACKAGE

Replace the value with below url (Copy existing value and paste it in notepad)

https://github.com/Azure/Azure-Sentinel/raw/vrusraut/OCIDataConnector/Solutions/Oracle Cloud Infrastructure/Data Connectors/OCILogsConn.zip

Apply changes

Go to Configuration section -> General settings

Change python version from 3.8 -> 3.9

Click on Save.

Restart function app

Check log in Monitor Section

Hi, @v-rusraut,
I've made the changes you requested to the function app Configuration\Application settings (I don't see an "Environment variables" in the portal) and restarted the app at approximately 10:10am EST.

I'm showing the following error on the Overview page:

I've had one invocation of the function since the changes were made and nothing since.

I'll revert the changes so that we don't miss any logging.
thanks!
David

Hi @DavidT8777,
Please follow above steps again, update below url in WEBSITE_RUN_FROM_PACKAGE and verify.
https://github.com/Azure/Azure-Sentinel/raw/v-rusraut/OCI/Solutions/Oracle Cloud Infrastructure/Data Connectors/OCILogsConn.zip
Thanks

Hi @DavidT8777,
For this solution supporting packages will be update before 14 Oct 2024, so till that time you can use python 3.8.
Once packages will update, we will notify you. currently closing this issue.
Thanks

Hey @DavidT8777, could you please check once with below package and let me know if error fixed:
Select python 3.9 version and then updated below link in path of WEBSITE_RUN_FROM_PACKAGE -
https://github.com/Azure/Azure-Sentinel/raw/82fbb2afd404176623de5dd9133f72bf8d2a828e/Solutions/Oracle Cloud Infrastructure/Data Connectors/OCILogsConn.zip

hi @v-sudkharat, aside from one error immediately after implementation, all further invocations of the function have been successful using the provided .zip and the python version set to 3.9.

Hey @DavidT8777, It would be great if you could share that one error screenshot with us. Thanks!

@v-sudkharat will do...  I didn't get a screen shot but pulled the information from the logs in the attached .txt file.
10265-Error After Additional Script and Python 3.9.txt
10265-Error After Additional Script and Python 3.9.txt

Thanks @DavidT8777 for sharing logs, Just want to know one more info from you, After updating 3.9 version in invocation tab its show as status Success right like below screenshot? and also logs are receiving into the workspace correct?-


FUNMAN 5: Workflow and drilldown UI

Create workflow operation, funman node, and funman drilldown to support user interactions. Note this does not include the charts and utilities to manipulate funman outputs - these are covered in other tasks. Please check with Jami and Pascale as there may be some subtle design changes.
Design Reference:

https://www.figma.com/file/GkVfEqpItMg67pNgh5oUAd/Terarium---New-%26-Improved?type=design&node-id=15-47982&mode=design&t=lC7kKMEXvYFKCizP-0

@jamiewaese-uncharted @ngraham76 could you update the figma link to point to the latest version as seen in the prototype please? e.g. placing number of samples and tolerance into the 'more options' section, etc. Thanks.

@jamiewaese-uncharted @ngraham76 could you update the figma link to point to the latest version as seen in the prototype please? e.g. placing number of samples and tolerance into the 'more options' section, etc. Thanks.

This Figma link points to the Sept 28th version of the file, is there a newer version? If so, I can't find it - @jamiewaese-uncharted will have to update this link as he worked on this.


could't use fancybox-plus in angular 8

Hi I installed fancybox in an angular 8 project via npm and then try to use it.  There are several version of this lib.  I followed the instructions in the page which forwarded from this project. But it didn't work for me.  Would you please tell how to use this directive in angular 2+ component with a small example. Thanks in advance.

The docu was for angular 1.x  which has a different architecture.
Fancybox-plus is a jquery based plugin. Which modifies the DOM outside of the lifecycle of angular 2.x or higher.
That means, you need to:

include fancy-box and jquery as npm dependency,

use the right  code when your component has been initialized. onAfterViewInit

-destroy the fancybox when the component is destroyed.
Sorry, no time to create a newer example.

The docu from the plain jquery, is for plain html pages.
The docu for angular 1.x has a different architecture. (https://github.com/igorlino/angular-fancybox-plus)
I believe you can use it, but fancybox-plus is a jquery based plugin. Which modifies the DOM outside of the lifecycle of angular 2.x or higher.
That means, you need to:

include fancy-box and jquery as npm dependency,
use the right  code when your component has been initialized. onAfterViewInit

-destroy the fancybox when the component is destroyed.
Sorry, no time to create a newer example.

Hi, thanks for reply. Actually I already did this steps. I want to show an html element in the fancybox. But I could't find any example for that purpose especially foe angular version. Could you please give little info about syntax.


Define a UUID base for drogue

Use UUID base for firmware service
Support resuming firmware update

bors r+


Various modifications

Added

ESM Status
Frequency sensor as configurable values
fixed some typo

This closes #24

Change the base branch to master as well

Will split this into three separate PR's instead.


disable option

for debugging/test it would be useful for the application to be able to disable auth via a configuration via an environment variable.

this is addressed


It has something to do with the CSS that Tracy loads into the page. Not sure if it is worth investigating further.


Blocks - Never place two statements on the same line

[From Perl Best Practices by Damian Conway, O'Reilly]
If two or more statements share one line, each of them becomes harder to comprehend. Give each statement its own line.
For example: var x; x = 1; should tickle jshint.

We will consider adding these kind of things as style guides in future, but not now, sorry.


Remove NSObject inheritance from all protocols.

I was intrigued by the discussion on https://github.com/spotify/HubFramework/issues/70.
This PR solves the main issue with removing NSObject inheritance (by providing an NSObject-agnostic HUBConformsToProtocol function in HUBUtilities). The only protocol that keeps NSObject is HUBSerializable because it needs isEqual.
Quick note: The reason HUBConformsToProtocol first tries to use the object's confromsToProtocol implementation (if it exists) is to make sure we don't break mock classes that override it.
Missing:

Linting.

Current coverage is 93.35% (diff: 93.10%)

Merging #265 into master will decrease coverage by 0.05%

@@             master       #265   diff @@
==========================================
  Files            72         72
  Lines          5019       5023     +4
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           4688       4689     +1
- Misses          331        334     +3
  Partials          0          0

Powered by Codecov. Last update 185da1f...28aef8c

Had a quick look and looks good, I’ll give it a proper review on Tuesday when I’m back in the office. Thank you for the PR much appreciated!

@marmelroy We can add the linting later. Since it’ll require some infrastructure to be added 😄

Thanks @rastersize. Good catch.

I don't see any tests for HUBConformsToProtocol. Can we add some, considering it's now an important part of our codebase?

@cerihughes, just got around to adding a simple test that covers all the cases. Let me know your thoughts.

🎉  shipit

Niiice 😃 🌮 🎉


Fix scroll behind suggestion list when scrolling on suggestion popup

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/27140
PROPOSAL: https://github.com/Expensify/App/issues/27140#issuecomment-1731142643
Tests

Open app
Open a chat with a conversation history spanning to multiple screen pages
Search for mentions in a way that search results should be less then 5
Scroll the mention list
Verify that scrolling over mention list will not scroll behind

[x] Verify that no errors appear in the JS console

Offline tests

Same as tests
QA Steps

Same as tests

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

Mobile Web - Chrome

https://github.com/Expensify/App/assets/75031127/faa4ce72-6a28-4e52-9742-73c2949bf166

Mobile Web - Safari

https://github.com/Expensify/App/assets/75031127/d5d909b0-2f09-4078-90d6-0e275e4405cc

Desktop

https://github.com/Expensify/App/assets/75031127/b986c245-5140-4414-8ec4-b65a4e6d1135

iOS

https://github.com/Expensify/App/assets/75031127/4069f13b-7c82-416e-a93b-083dd51d7bee

Android

https://github.com/Expensify/App/assets/75031127/8a860ec6-d40f-467f-a77b-a52734b83f1d

Select Any chat

Open a chat with a conversation history spanning to multiple screen pages

Reviewer Checklist

[x] I have verified the author checklist is complete (all boxes are checked off).
[x] I verified the correct issue is linked in the ### Fixed Issues section above
[x] I verified testing steps are clear and they cover the changes made in this PR

[x] I verified the steps for local testing are in the Tests section
[x] I verified the steps for Staging and/or Production testing are in the QA steps section
[x] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[x] I checked that screenshots or videos are included for tests on all platforms
[x] I included screenshots or videos for tests on all platforms
[x] I verified tests pass on all platforms & I tested again on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[x] I verified proper code patterns were followed (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I verified that this PR follows the guidelines as stated in the Review Guidelines
[x] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
https://github.com/Expensify/App/assets/2590174/5102fb8f-471b-4c70-9ce5-0d6a948d8713

Mobile Web - Chrome
https://github.com/Expensify/App/assets/2590174/edb4cc87-e599-45f8-9a8d-f92268833a12

Mobile Web - Safari
https://github.com/Expensify/App/assets/2590174/3dbd960a-a2dd-4f42-8431-70210923f49f

Desktop

iOS
https://github.com/Expensify/App/assets/2590174/03a63875-11f1-4934-9d9a-5275ae178258

Android
https://github.com/Expensify/App/assets/2590174/ca243258-e273-4b45-a3e8-bee43aae0058

@cubuspl42 do you want me to change the step to

Open a chat with a conversation history spanning to multiple screen pages

@getusha That's what I meant. If the chat history is short, the fix cannot be effectively verified!

@cubuspl42 Updated thanks!

@NikkiWines friendly bump :)

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/NikkiWines in version: 1.3.79-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/jasperhuangg in version: 1.3.79-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
failure ❌

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/jasperhuangg in version: 1.3.79-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
cancelled 🔪

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/jasperhuangg in version: 1.3.79-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


Github Patchwork Singapore - 6th June 2015

Learning open source, together
(based on Github's PatchWork by @jlord)
Patchwork is a hands on workshop for learning Git and GitHub. There will be snacks for hackin', lots to learn, and new friends to be made. We'll have GitHubbers and other open source contributors as mentors to lend a hand as attendees make their way through the workshop.
When?

Saturday, Jun 6th at 2:30pm - 5:30pm

Where?

IDA Labs Level 3

Tasks:

[ ] Gather attendees and mentors (RSVP)
[ ] Catering
[ ] Venue pictures
[ ] Sponsors logos

cc: @winston, @Learnemy, @Valentine, @spinningarrow, @donal, @jesseplusplus, @ElizabethN

How do we want to do registration? Meetup.com? Eventbrite?

What kind of food do we want for catering? FYI, I am using Mum's Kitchen for RDRC. http://www.mumskitchen.com.sg/

@winston at the moment it looks like @ElizabethN will be handling the registration.
@Valentine will be handling the catering.


update packages

:white_check_mark: Build TheIdServer 3.0.1-fix-dependencies1-0019 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/61274d913d by @)


LIDO-1677

@PrateekshaDubey can you verify the test cases are correct?

@tyleradams I have verified the test cases those are there in the Microsoft support document attached to Linear Ticket (LIDO - 1677), on high level those seems correct, more insight can be provided while executing these on particular functionality.


Add "activation" in the param of recurring payment init, so it will charge 1 immediately upon complete.

Related to issue #51.
I think current recurring payment won't charge immediately (for the first month) upon complete. And I found add a "activation" param to make it happen (am I wrong?). Since it's quite normal to charge immediately for recurring payments, maybe can add this option in the wiki docs?
Paypal::Payment::Recurring.new(
      start_date: SET_YOUR_OWN,
      description: SET_YOUR_OWN,
      activation: {
        initial_amount: SET_YOUR_OWN
      },
      billing: {
        period: :SET_YOUR_OWN,
        frequency: SET_YOUR_OWN,
        amount: SET_YOUR_OWN
      }
    )

Be careful, in our experience paypal sandbox doesn't work exactly as production. Recurring payments are done with a delay (one day or more).
If you use initial_amount in sandbox it works as expected more or less(it charges at the first time) but in production it charges both amounts(initial_amount + amount).
It seems to be the same experience for others users as you can read in this comment:
https://github.com/nov/paypal-express/issues/51#issuecomment-68234257

I've just check our current payment history, so far so good for production use with initial amount. No double charge on the first month. I'll check more detail later.


Tweak encryption terms

Deploy preview for calico ready!
Built with commit 89f04fd72730f3223ad171d93b05a63e5e34268d
https://deploy-preview-3630--calico.netlify.app


[Backport release-22.05] tracee: init at 0.7.0

Description of changes

Backport tracee so it is in the stable release rather than waiting months for the next stable
I think this is how you backport :eyes:
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[X] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[X] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[X] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[X] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[X] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/882

I think this is how you backport

It's a bit more standard to use the GH label which triggers a bot to create a backport PR.  But this is fine, too... I'm not sure whether you even have permissions to add labels yourself.

IDK if there was a backport label for 22.05 at that point

You can add these labels any time.  Even on a PR that's long been merged.

Yes but 22.05 had only just branched off for beta. The actual label might not have existed

Ah, this backport PR is quite old as well, but the bot worked a couple days before that already: https://github.com/NixOS/nixpkgs/pull/174166


Add content filter to list component

The user should be able to filter the content of the list component.
Ideally the content should be filtered by one single textbox, which would be able to find in each table cell.

Done on #15


QueryClientEvaluation in Select doesn't throw when configured to throw

Say I have a Post-Blog model and the Post has a member function ViewCount() (which is not a mapped property or anything like that, just a plain old member function). My Context.OnConfiguring contains:
optionsBuilder.ConfigureWarnings(warning => warning.Throw(RelationalEventId.QueryClientEvaluationWarning));
I would expect the following line to throw a "warning as exception, QueryClientEvaluation". Instead it quietly evaluates ViewCount() on the client:
foreach(var count in ctx.Posts.Select(post => post.ViewCount())) { }
During the Evaluation of ViewCount(), the post is not yet in the context (i.e. just like usual QueryClientEvaluation, and something that should never happen if I disabled QueryClientEvaluation).
Edit: Still no answer? I guess it'll be another Wednesday response of "we discussed this in triage and came to the conclusion that it's by design" :trollface:

@ajcvickers So, can I get an answer? I don't mean to hurry you, I just thought you might have forgotten since the issue is closed and you haven't written in 2 weeks.

@Wain123 Ultimately I don't think that I can answer the question in the way that you are asking. I can see your perspective, but it's just not the way things are implemented right now. As I said above, we have considered changing the warning or when it is generated, but the value of doing so does not seem to outweigh the problems it will cause. In terms of being explicit about when client eval may happen without a warning, then based on your (possibly correct) definition I think it would take a long time to come up with something and my feeling is that even then it probably wouldn't be very useful, certainly to most people. If more feedback in this area indicates that others are hitting similar issues, then we will revisit, but for now I think it is what it is with regard to client evaluation and warnings.

If more feedback in this area indicates that others are hitting similar issues, then we will revisit

There are now 4 issues marked as duplicates of #10471, is that enough for you to revisit?
In particular, I would appreciate a response to this one point I mentioned in the previous post. You said:

The warning was intended to identify when we execute on the client something that might result in more work (e.g. bringing back more data) being done on the client that would be ideal because we cannot translate something to be done on the store

And this seems inconsistent with the example I gave:

In the example above, ctx.Posts.Select(post => post.ViewCount()) is an IQueryable<int>. Therefore, if I enumerate it, I would expect only a bunch of int to be returned from the DB. Instead, the DB gives me a bunch of Post, i.e. a lot more data.

Can you clarify whether the "bringing back more data" part of your statement applies to my example?

Bringing back in terms of size of result set rather than individual result. Suppose each row in database correspond to one row in result on client side, then a client where can get more rows from database which may not appear in result. But a client projection will fetch same amount of rows. It is true that it may fetch more data in each row but that still maintain 1:1 correspondence. Yes, it is more data but user asked us to generate result in particular way which requires fetching extra data per row (EF6 also worked this way).


REDIS_URL: unbound variable error

What happened?
I originally managed to get my project up and running on digitalocean with no problems, however needed to make some changes. Now when I try and get my project up, I am getting a REDIS_URL: unbound variable error. If I hardcode this in my entrypoint file, then I get the same issues with the POSTGRES variables. This would suggest to me that I'm having a problem reading the .env file, but I am a complete loss as to why this is. So postgres and traefik fail with exit code 1, and the other containers are simply waiting for progres. I've asked around on a few forums and been unable to resolve the issue, so hoping that posting here will get me some assistance from people with more specific knowledge!
What should've happened instead?
Project should've run.
Additional details
I believe the above summarises my problem to the extent I am able to understand it! Thanks for any assistance any is able to provide!

What changes did you made? How are you hosting your codebase? Check you path's on top of config/settings/base.py and config/wsgi.py and check READ_DOT_ENV_FILE on top of base.py.

If you are running without docker you should run merge_production_dotenvs_in_dotenv.py to create a .env file. Make sure DJANGO_READ_DOT_ENV_FILE=True is listed in the .env

Nevermind, forget what I said in my comment about the .env since you are running with docker-compose.
Do you run docker-compose with -f production.yml? Did you change production.yml? If so, can you post the contents? Are the .django and .postgres files available in .envs/.production? Sorry for my guessing here, trying to sum up the mistakes I made in the past.

Nevermind, forget what I said in my comment about the .env since you are running with docker-compose.
Do you run docker-compose with -f production.yml? Did you change production.yml? If so, can you post the contents? Are the .django and .postgres files available in .envs/.production? Sorry for my guessing here, trying to sum up the mistakes I made in the past.

No problem at all - I appreciate guessing! I've been guessing for hours over the last few days so having other people's inputs is great. To answer your question, yes - I renamed my production.yml file to docker-compose.yml and yes I run docker-compose with -f docker-compose.yml. And yes the .django and .postgres are both available in the .envs/.production. I have tried taking them out, which then throws an error saying they're not there. When I put them back in, the error disappears. So the project is definitely aware they are there, but is just not reading the values from them for some reason I can't understand.
Separate question - I might just try and avoid the problem by changing my hosting provider - do you know what is recommended for django-cookiecutter docker hosting, or what do you use? I've no experience with Amazon hosting but willing to give it a go if it's going to be quicker than burning more time on this issue haha.

I have the same issue running production.yml locally. Does this help us understand the issue?

Using docker as abstraction layer makes it machine agnostic, so changing the server does not make any sense to me. Instead, try building en running production.yml locally. This shortens the deployment cycle.

Can you post production.yml here? Production depends on a different Dockerfile and uses a different start script. Did you made changes to the production Dockerfile?

Can you post production.yml here? Production depends on a different Dockerfile and uses a different start script. Did you made changes to the production Dockerfile?

I've resolved it. I had made changes to the production.yml file to include selenium, and thought I had not made changes to anything else, however upon comparing the github cookiecutter template with my production file, I had somehow fat fingered an extra letter into one of the image names.
Thanks for your assistance in resolving! Can't express how much I appreciate it.


Add meta data to tools pages

Books, zines, stickers, posters, logos, videos should all have the right meta tags to generate a preview when pasted into Slack, Messages, etc.
Here is a book, not expanding a preview.

See an article page for example meta tags.
For example:
https://crimethinc.com/2020/04/21/whats-worth-dying-for-confronting-the-return-to-business-as-usual
Preview:

Meta tags:

i cannot reproduce this. also, it looks like books, zines, and other tools already have social meta tags

I just try it myself with older and super recent article, and I cannot reproduce.
I'll close for now.


NATS core support added

Bump github.com/nats-io/nats.go v1.30.2 to 1.31.0
NATS sink: added support core nats messaging in addition to jetstream
NATS sink: added timeouts configuration options.  publishtimeout, dialtimeout, reconnecttimeout

I'd still be hesitant to merge this since non-Jetstream topics don't guarantee ordering, do they? 🤔

Ok it seems ordering it available (https://docs.nats.io/reference/faq#does-nats-offer-any-guarantee-of-message-ordering) but the delivery guarantee for non-Jetstream is at-most-once which will still lead to issues (https://docs.nats.io/reference/faq#does-nats-guarantee-message-delivery).

Yes, it's not stream but pub/sub and we have one publisher
Does NATS offer any guarantee of message ordering?
NATS implements source ordered delivery per publisher. That is to say, messages from a given single publisher will be delivered to all eligible subscribers in the order in which they were originally published. There are no guarantees of message delivery order amongst multiple publishers.

In some cases, the only fact of changes makes sense. It's up to the user to decide how to use it.
Yes, for Core pub/sub, it's at-most-once delivery. If there are no subscribers on the topic, the message will not be delivered and will be skipped. For Openfaas CE, I can't use jetstream because it's a Pro and paid functionality. My function reacts to CDC and requests initial data on start, then tracks the CDC events.

Hey @falkolab! Sorry for the long wait.
I'm still hesitant to put this into the core.
I'm thinking about adding plugins to the distribution though. Meaning, it'd be delivered as part of the download, but needs to be specifically enabled. However, that would work on Linux only, due to the plugin system in Go.
Any reason, you can't use it as a plugin?

Hey @falkolab! Sorry for the long wait.
I'm still hesitant to put this into the core.
I'm thinking about adding plugins to the distribution though. Meaning, it'd be delivered as part of the download, but needs to be specifically enabled. However, that would work on Linux only, due to the plugin system in Go.
Any reason, you can't use it as a plugin?

Yes I can. I can even use my forked repository. If one need this functionality then welcome to my fork.

Looking into this again. How would you write an actual test for this sink when you it's at-most-once delivery. While it will work in 99% of the cases, the 1% spurious failure would be bothersome 🤔
My current idea would be to mark sinks are "potentially complicated" (whatever the final term would be), meaning you should only use them when you know what you do. That would generate a warning during startup and may need to be enabled specifically.
Anyhow, it would still require a clean integration test.


Added missing Cairo core implicits to BUILTINS

Closes #2397
Introduced changes

Added missing implicits present in Cairo core but not ignored by forge-runner: RangeCheck96, AddMod, MulMod

Checklist

[x] Linked relevant issue
[ ] Updated relevant documentation
[ ] Added relevant tests
[x] Performed self-review of the code
[ ] Added changes to CHANGELOG.md

Please add tests, implementation wise looks good from my side

It looks good, please merge it so we can profile our code

@integraledelebesgue please add tests in another PR after you come back

https://github.com/foundry-rs/starknet-foundry/issues/2437


CB-15347 Support AWS custom regional RDS root certs

application.yml (redbeams): Add regional AWS RDS root certificates for
af-south-1 (Africa / Cape Town), eu-south-1 (Europe / Milan) and
me-south-1 (Middle East / Bahrain).
DatabaseServerSslCertificateConfig:

Add support for certificate lookups by (cloud platform, region)
pair. If no match is found for (cloud platform, region), the search
will fall back to the query by cloud platform only.
Adapt all client classes to pass the region of the DBStack.
Add validation of cloud platform string format for the certificate
entries received from application.yml.
Add legacy CloudProviderIdentifier and version entries for AWS RDS
regions af-south-1 (Africa / Cape Town), eu-south-1 (Europe / Milan)
and me-south-1 (Middle East / Bahrain).

DatabaseServerSslCertificateConfigIntegrationTest: Verify the new
certificates added to application.yml.
Remove dead code in AllocateDatabaseServerV4RequestToDBStackConverter
and AllocateDatabaseServerV4RequestToDBStackConverterTest.
Fix typos and eliminate code style warnings in several places.
Testing:

Manual test of an old (unaffected) AWS region and all new regions in
local CB.
Added new UT & adapted existing ones.

See detailed description in the commit message.

Please send the PR from another branch, because the merge will delete it.

Please send the PR from another branch, because the merge will delete it.

why? Is this a new feature? I mean I have a plugin which deletes the branch automatically, but I don't remember github doing it by default

Please send the PR from another branch, because the merge will delete it.

why? Is this a new feature? I mean I have a plugin which deletes the branch automatically, but I don't remember github doing it by default

GitHub does this automatically, because we have configured it on the repo. You can disable your plugin.


Statics files are generated in the init container

With the new version of server deployment, the static files are generated in the init container in production but are not present in the server container at the end. We should remove this init container or we should copy those static in the server container once generated

Another way of solving that is to share a volume between the init-container and the final containers. Thus the statics are generated during the init and already there when the container start.


Cannot call method "substring" of undefined

After the last update, I cannot upload any file from Brackets. I receive this message:
"Cannot call method "substring" of undefined"

I uninstalled the extension, then I reinstalled it and now it works normally.

This is happening for me as well. If I restart brackets it will work again after a couple hours of coding and switching repos it will then bring back up this error. Please fix as soon as possible.