python3-maturin-native and python3-orjson give compilation error in clean build

As title, currently these two recipes give an error on a clean build.

After investigating it has something to do with inherit and the order of inherits it seems in the python3-orjson recipe.
inherit cargo pypi python_setuptools3_rust python_pyo3
will not build correctly, but will somehow configure the recipe correctly such that afterwards you can call the recipe again with
inherit cargo pypi python_pyo3 and it will compile, but only hand out a .so file instead of a python package.
Then after trying another time (without a clean between) runnin with
inherit cargo pypi python_setuptools_build_meta python_pyo3 I seem to get a correct package.


Update docs: didRender is defined on the card card, regardless of editing context

See bustlelabs/mobiledoc-dom-renderer#31

thanks @joshfrench !


stats by URL path

currently there are HTTP connection manager level stats but no URL path level stats AFAIK.
it would be nice to have by path stats out of the box

The is no reasonable way to have generic path stats, since Envoy doesn't know what a path is exactly with prefix routing.
Currently, we have the virtual cluster feature: https://lyft.github.io/envoy/docs/configuration/http_conn_man/route_config/vcluster.html
This might be enough. Another thing I would be open to is having dynamic stats on a per route basis, which the user opts in to.

thanks for help.
we're proxying gRPC via envoy, and would like to have some generic per method stats from envoy.
i looked at virtual cluster and it seems have to be configured for every method, and the suggested dynamic stats on a per route may also need to do this.
i am a bit confused by the

Envoy doesn't know what a path is exactly with prefix routing

not familiar with Envoy internals, but i thought the HTTP URL path (only base path without query string) is already in the request ?
also, support for grpc-status from Trailer would be a bonus for gRPC.

Regarding per path stats, the issue is basically that the stats can become unbounded so we can't do it by default (even if we strip the query string). I'm open to an optional feature here that is off by default if you want to do a design proposal and PR.
BTW, as a short term workaround, the HTTP/1.1 bridge filter will collect gRPC per method stats: https://github.com/lyft/envoy/blob/master/source/common/grpc/http1_bridge_filter.cc#L19. It will do this even when not bridging for HTTP/2. This might be good enough for what you need for now.
I know that @fengli79 and @lizan have some thoughts on what we want to do medium term with better gRPC stats and whether to put them in the router directly of have them be in a different filter. Perhaps they can weigh in here.

thanks, you saved my day.
wonder if it's possible to add Timer  to the existing stats. if this doesn't involve too much Envoy internals, i may find some time to do it.

@jwfang it should be pretty easy to do timers also. See https://github.com/lyft/envoy/blob/master/source/common/http/codes.cc#L83 for how this might work.

gRPC-Web filter also do method level stats.
About http2 proxying for gRPC payload, I think envoy probably need to know more about the payload than just treat it as normal http2, as the status of gRPC is in the grpc-status header/trailer instead of the http2 status. A filter may help.

Closing for now. Let's make a new issue if there is a particular feature request.


Use debianoid_omnibus bootstrap for Debian 9

Coverage remained the same at 90.153% when pulling 27925a117f6b88d183f21ec49fdd416cc39c370b on reidab:patch-1 into 5e83b0a765dd49eb63761ba9c632e0d639f7e17f on matschaffer:master.

Hey thanks! And happy new year :tada:


Display GTFS in ArcGIS

I am working on ArcMap 10.1 and I am trying to use Display GTFS in ArcGIS, but then error is occurring and its about some module. Could anyone please check if we can solve it or has it already been solved? Also, could anyone tell me what is the logic behind getting the routes shapefile.

Hello.  The numpy module is part of python itself.  It should have been installed with python when you installed ArcGIS.  If you're getting that error, it looks like your python installation is messed up.  Talk to your systems administrator or whoever is in charge of ArcGIS installations at your organization, or reinstall ArcGIS and see if that fixes it, or call Esri tech support.
Regarding the logic behind creating the route shapes: The tool reads the lat/lon coordinates from the [GTFS shapes.txt file] (https://developers.google.com/transit/gtfs/reference/shapes-file) and connects the points with straight line segments.

Hi. Much thanks. I am indeed very curious about using this tool. Already
raised the request to the system engineers, hope to get it done in a day or
2.
Alright, regarding my query about how to spatially locate the routes-
Suppose I am working on the Non-GTFS to GTFS conversion thing, I do not
create the shapes.txt file since its an optional file, so I am more
concerned about that.
Regards,
Shivam
On Thu, Aug 25, 2016 at 8:43 PM, Melinda Morang<EMAIL_ADDRESS>wrote:

Hello. The numpy module is part of python itself. It should have been
installed with python when you installed ArcGIS. If you're getting that
error, it looks like your python installation is messed up. Talk to your
systems administrator or whoever is in charge of ArcGIS installations at
your organization, or reinstall ArcGIS and see if that fixes it, or call
Esri tech support.
Regarding the logic behind creating the route shapes: The tool reads the
lat/lon coordinates from the GTFS shapes.txt file
https://developers.google.com/transit/gtfs/reference/shapes-file and
connects the points with straight line segments.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Esri/public-transit-tools/issues/35#issuecomment-242423839,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APVOERVFCG_jlXyAhTRHxhyet7moMssSks5qjbEJgaJpZM4Jsxpq
.

--
Warm Regards,
Shivam Parashari

If you don't already have a shapes.txt file, you can create one using the Generate GTFS Shapes tool.


binding.gyp not found in math-base-special-ln

Description
Encountered an error when installing @stdlib/math-base-special-ln via npm on macOS. I updated python and node-gyp, and tested that node-gyp works on my machine by making a simple dummy project, and it does indeed compile. This does not appear to be an issue on my linux machine, only macOS.
On a brand new project, if I use npm i @stdlib/math-base-special-ln, the install seems to fail with  gyp: binding.gyp not found (cwd: /Users/matt/Documents/temp/test/node_modules/@stdlib/math-base-special-ln) while trying to load binding.gyp. I'm not too familiar with node-gyp or how binding.gyp is generated, so I'm not entirely sure if this is the issue.
 System Darwin 22.5.0
494 error gyp ERR! command "/usr/local/Cellar/node/21.7.1/bin/node" "/usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
494 error gyp ERR! cwd /Users/matt/Documents/temp/test/node_modules/@stdlib/math-base-special-ln
494 error gyp ERR! node -v v21.7.1
494 error gyp ERR! node-gyp -v v10.0.1
494 error gyp ERR! not ok
495 verbose exit 1
496 timing npm Completed in 11671ms
497 verbose unfinished npm timer reify<PHONE_NUMBER>793
498 verbose unfinished npm timer reify:build<PHONE_NUMBER>776
499 verbose unfinished npm timer build<PHONE_NUMBER>777
500 verbose unfinished npm timer build:deps<PHONE_NUMBER>777
501 verbose unfinished npm timer build:run:install<PHONE_NUMBER>783
502 verbose unfinished npm timer build:run:install:node_modules/@stdlib/math-base-special-ln<PHONE_NUMBER>783
503 verbose code 1
504 error A complete log of this run can be found in: /Users/matt/.npm/_logs/2024-04-04T18_31_26_518Z-debug-0.log

Related Issues
N/A
Questions
No.
Demo
No response
Reproduction
- on macos, start a new project with npm init
- `npm i @stdlib/math-base-special-ln`

Expected Results
Installation Completes

Actual Results
See attached log.

Version
latest
Environments
Node.js
Browser Version
No response
Node.js / npm Version
npm 10.5.0, node 21.7.1
Platform
macOS
Checklist

[X] Read and understood the Code of Conduct.
[X] Searched for existing issues and pull requests.

Thank you @Speuce for this bug report. We have been aware of this problem, caused by a change in how npm determines whether to attempt building native add-ons, but didn't get around to pushing a fix out yet. We just published a patch release for affected packages.
Would you be so kind to please rerun your test with the new version and whether the problem is now addressed:
npm init -y
npm i<EMAIL_ADDRESS>
Please let us know what npm and Node.js versions you are running in case you still encounter this issue.

<EMAIL_ADDRESS>
Issue resolved. Thanks!


Views

All the project requirements have been implemented.
Project requirements

Implement the design from the sneak peek wireframes, as shown here
Styling is nice to have but focus on making sure everything functions first.
Use methods that you have created in the Blog App - processing data in models.
Hint: In order to see some posts or comments, you might need to add some data to your database as described here.

Thanks for your recommendations and suggestions. All have been corrected.


🔧  Fixing the GBD region percentage aggregate error

....but skipping it for risk factor datasets. Also attempting to make the aggregating function a little bit more modular and less horrible.

Hey @veronikasamborska1994 - I think this is an okay solution to the regional aggregates issue in the GBD code. Curious to hear what you think, but no rush!

Admin

hey @spoonerf! It looks like this code would only work for gbd_cause dataset. Maybe a cleaner way would be just to add an argument that would be false by default and only run the add_regions_to_percent if you set it to true in the gbd_cause script? I think adding the weighted averages functionality would take quite a bit of thought as we'd probably just want to build it into the geo module so probably best if we go with your solution for this one.

@veronikasamborska1994 - That is a very good idea! I've added that to the code. If it passes CI okay I'll merge shortly 🤞🏼


Make new release to match readme

Hi! When I run kappa deploy, it doesn't actually run as the latest release is from June, while the package has since been updated significantly (50 or so commits). Would be great to have a released version with all the great new stuff out :)

Alright I can take care of this after lunch!

Done!


[Chat] / Chrome fails to "Ask" for notification privileges

If you haven’t already, check out our contributing guidelines for onboarding!

Platform - version:  v1.0.1-488 web on Chrome
Action Performed (reproducible steps):

Navigate to https://expensify.cash/#/signin on web on Chrome browser
Click into the padlock in the URL bar and set notifications to default (see image below)
Reload page
Sign in
Sign out via /settings
Observe that activeClients2 has no value in localStorage whatsoever (see image below)
Sign in again
Observe that activeClients2 still has no value
Leave a comment on a report that is not currently in view from another session as a shared user (this should trigger the prompt to appear)
Observe that no prompt appears because the notification is skipped and this log displays:
[LOCAL_NOTIFICATION] Skipping notification because this client is not the leader

Expected Result:
The Ask for notifications prompt should display reliably.
Actual Result:
No prompt appears because the notification is skipped because Skipping notification because this client is not the leader
Notes/Photos/Videos:

In addition to posting a proposal here, please make sure to submit via this Upwork job too https://www.upwork.com/jobs/~01b3cfaf1cd42f3481

Hi,
Here is the solution.
In SignInRedirect.js we have to keep current activeClients value remain when user logs out from the system.
In SignIn Redirect.js we can get active clients with this,

let activeClients2;

Onyx.connect({
    key: ONYXKEYS.ACTIVE_CLIENTS,
    callback: (val) => {
        activeClients2 = !val ? [] : val;
    },
});

then In redirectToSignIn method, we can set active clients after clearing the onyx storage.
        if (activeClients) {
            Onyx.set(ONYXKEYS.ACTIVE_CLIENTS, activeClients);
        }

@tylerkaraszewski @marcaaron @tgolen any comments on @SameeraMadushan's proposal above?

I don't have enough background to know if this works, deferring to @tgolen and @marcaaron.

@SameeraMadushan I am a little bit confused about your proposal. I see both activeClients2 and activeClients, but I think they should all be references to activeClients2, is that correct?
I think overall it's a pretty good and simple solution.
I think another possible solution would be to specify a list of keys that should not be cleared out when Onyx.clear() is called. These keys could be registered as part of Onyx.init() and then a small refactoring of Onyx.clear() would be necessary. What are your thoughts on that approach, Marc?

@tgolen Thanks.
activeClients2 will be initiate when the first time user loads the page or on a refresh of the page.
When I access activeClients2 after clearing the Onyx, then it will be empty. Because It's pass by reference.
That's why I have to define a new const name as activeClients and get the previous value.

OK, thanks for explaining. I think we'll need to work on the naming convention a little bit, but the proposal makes more sense now.

I think the onyx library doesn't support defining items that should not clear right?
Seems it's related to the expensify/react-native-onyx library improvement.

That's correct. You'd need to implement a change in that library first and then update the version of react-native-onyx in this repo. Still curious for @marcaaron to chime in about that though. I know we've talked about it in the past, but maybe it would be more harmful than good.

@tgolen Yes, I will change the convention. 😅
Let's get Marc's thoughts as well then. 😃

I think another possible solution would be to specify a list of keys that should not be cleared out when Onyx.clear() is called. These keys could be registered as part of Onyx.init() and then a small refactoring of Onyx.clear() would be necessary. What are your thoughts on that approach, Marc?

I think this could be a good future improvement, but not necessarily something we need right now. Looking at the cases where we want to save some data after an Onyx.clear() and most of them are conditional anyway. If we have an error, if we have a reportID then persist after the clear etc.
https://github.com/Expensify/Expensify.cash/blob/c3a6aa86d34404599f5239a1dbd84f96505f0e30/src/libs/actions/SignInRedirect.js#L47-L52
This is the only case where we want a key to persist unconditionally so I think we can wait for the optimization.

Cool, thanks Marc. I am fine with that. Let's just use the local variable route to reset the Onyx value after clear() then.

nice work @SameeraMadushan - we'll issue payment on 18 March if there are no regressions reported 👍🏽


Notched search

This PR adds support for multiple precursor isolation windows.
MORE ACCURATELY, right now if the spectrum has multiple precursors, sage uses the first one for the search and disregards the rest. With this PR, it will score candidates in all of the annotated precursors.
In my experiments (as expected) it has no effect on the results for files that have a single isolation window.
The main use I have in mind for this feature is for pseudo-generated spectra from DIA, where the assigned precursor might be ambiguous. In that case I could just annotate it as having both precursors and let them fight it out within the search engine.
TODO: add testing to make sure all of them are used + make sure it does not screw up open/wide window search, there might be the need to simplify the overlapping ranges.

Are there mzMLs in the wild that have multiple precursors annotated like this?
What is the performance impact of this for "normal" searches?  I suspect it is non-zero (allocations in initial_hits, doubling of PreScore memory requirements), but it should be benchmarked.
For your use case - why not duplicate the entire spectrum and assign unique precursors to each? My hunch is that it be more efficient search-time and FDR wise.

I am not sure! I know that for some MS3-TMT experiments the precursor is sometimes from multiple MS2 peaks and therefore the MS3 will have multiple precursors annotated (but in those cases, the MS3 is only used for the reporter ions and not for search). I can imagine some hacky methods that might make use of it but definitely in the experimental realm. Having said that, based on some of the issues I see in the repo, a non-negligible amount of people are using the project with self-packed mzml and mgf files, so I can imagine that someone else might make use of this feature if present.

On my system using a human proteome, closed search and a random .d file it is pretty negligible. I believe it would increase the allocations of initial_hits and prescore BUT it would increase it by 1 per thread, since they are aggregated per spectrum.

# Master
        21.50s real             1m20.74s user           7.95s sys
<PHONE_NUMBER>  maximum resident set size
                   0  average shared memory size
                   0  average unshared data size
                   0  average unshared stack size
             1210225  page reclaims
                  33  page faults
                   0  swaps
                   0  block input operations
                   0  block output operations
                  14  messages sent
                  18  messages received
                   0  signals received
               16047  voluntary context switches
              456758  involuntary context switches
<PHONE_NUMBER>73  instructions retired
<PHONE_NUMBER>74  cycles elapsed
<PHONE_NUMBER>  peak memory footprint

# feature/notched_search
        21.64s real             1m21.62s user           7.93s sys
<PHONE_NUMBER>  maximum resident set size
                   0  average shared memory size
                   0  average unshared data size
                   0  average unshared stack size
             1200829  page reclaims
                  20  page faults
                   0  swaps
                   0  block input operations
                   0  block output operations
                  14  messages sent
                  19  messages received
                   0  signals received
               16104  voluntary context switches
              469975  involuntary context switches
<PHONE_NUMBER>51  instructions retired
<PHONE_NUMBER>71  cycles elapsed
<PHONE_NUMBER>  peak memory footprint

I believe it would lead to some FDR issues I am not totally satisfied with ... since the poisson distribution and the number of scored candidates would represent the candidates scored for that notch and not the total number for that spectrum. (I have not done an entrapment to make sure it has an undesired effect, but 'it feels right').

LMK what you think! If you feel like it is not within the scope of the project I can maintain a fork with the feature for my needs (I try to be good at maintaining my PRs but ultimately you are the maintainer of sage)

I'm not opposed to adding features to support homebrewed mzMLs, but I try to keep Sage "zero-cost" for running standard searches - e.g. those features shouldn't impose a non-trivial cost on the other 95% of searches.

I changed the implementation and now it uses a run-length encoded approach to store the precursor information (precursors are stored in order, the number of hits per precursor are stored). This currently makes it ~2% slower on open search using my system (+- 100da, human proteome) but closed search more than that (~8 in my tests) ... there might be some additional optimization calculating the precursor masses.
I guess another alternative is to have an option to disable the feature, which should be actually 0-cost (binary might be a bit larger, if I "understand" what the compiler will do).

Made this guy a draft for now! I don't think its worth the current overhead (clear in open search, pretty negligible in closed). Might revisit it later if I see a strong reason for it.


Complete Recipe Generator component and add CSS to the header, footer, homepage and recipe generator components.

Recipe Generator can be used multiple times
Recipe Generator will tell you when nothing is found
CSS added to header and footer
Mission carousel scales to 80% vh before scaling to just width height.

Looks good!


Implement OData v4 extension for data aggregation

This (partially) implements the OData Extension for Data Aggregation as described here. Due to complexity of the extension only the transformations filter, aggregate and groupby are supported yet.
The extension can be used either strictly typed e.g.
var productsGroupedByCategory = _client
                .WithExtensions()
                .For<Product>()
                .Apply(b => b.GroupBy((x, a) => new
                {
                    x.Category.CategoryName,
                    AverageUnitPrice = a.Average(x.UnitPrice)
                }))
                .OrderBy(x => x.AverageUnitPrice)
                .FindEntriesAsync();

or dynamically e.g.
var x = ODataDynamic.Expression;
var b = ODataDynamicDataAggregation.Builder;
var a = ODataDynamicDataAggregation.AggregationFunction;
var productsGroupedByCategory = _client
                .WithExtensions()
                .For(x.Product)
                .Apply(b.GroupBy(new
                {
                    x.Category.CategoryName,
                    AverageUnitPrice = a.Average(x.UnitPrice)
                }))
                .OrderBy(x.AverageUnitPrice)
                .FindEntriesAsync();

This pull request should fix #490 and probably #748, #415

Thank you for this PR. These extensions look very well thought and properly implemented. Special thanks for unit tests.


Duplicate declaration of field in R.java

$ bazel build //...
INFO: Analyzed 8 targets (55 packages loaded, 1551 targets configured).
INFO: Found 8 targets...
ERROR: /Users/p/Code/examples/bazel-kt-databinding-crash/BUILD:17:22: Compiling Java headers libr-classes-hjar.jar (1 source jar) failed: (Exit 1): java failed: error executing command external/remotejdk11_macos_aarch64/bin/java -Xverify:none -XX:+UseParallelOldGC -XX:-CompactStrings '--patch-module=java.compiler=external/remote_java_tools_darwin/java_tools/java_compiler.jar' ... (remaining 35 argument(s) skipped)

Use --sandbox_debug to see verbose messages from the sandbox
com/example/databinding/lib/R.java:4757: error: duplicate declaration of field: Spinner_android_entries
    public static int Spinner_android_entries = 0;
                      ^
INFO: Elapsed time: 43.073s, Critical Path: 11.77s
INFO: 298 processes: 43 internal, 254 darwin-sandbox, 1 worker.
FAILED: Build did NOT complete successfully

This is after applying the patch in #14 . Files at this PR, this commit

Issue on my end.


How to write the analytical derivatives of the Cartesian velocity of a given point?

Hi guys,
I am not able to properly compute the analytical derivatives of a body-fixed point velocity which is aligned with the world, i.e. LOCAL_WORLD_ALIGNED velocity (named as Cartesian velocity as well). With derivatives I refer to these Jacobians: dv_dq and dv_dv. To be precise, I have troubles to compute dv_dq.
To make things easier, in the following code, I have written two functions: one for analytical derivatives, another one for numdiff. However, before going to the code, I have the following question:

where can I find the mathematical demonstration of the translation of Jacobians? (note that my code is inspired by some internal routines of Pinocchio, but I don't fully understand them)

import pinocchio as pin
import example_robot_data as robex
import numpy as np
import copy

np.set_printoptions(suppress=True, linewidth=1000)

def analytical(model, q, v, frameId):
  # Allocate data
  data = model.createData()
  # Compute the joint velocity derivatives in the world coordinates
  jointId = model.frames[frameId].parent
  pin.forwardKinematics(model, data, q, v)
  pin.computeForwardKinematicsDerivatives(model, data, q, v, np.zeros_like(v))
  dv_dq, dv_dv = pin.getJointVelocityDerivatives(model, data, jointId, pin.WORLD)
  # Translate the dv_dq, dv_dv Jacobians
  jMf = model.frames[frameId].placement
  M = data.oMi[jointId] * jMf
  for k in range(model.nv):
    m_dq = pin.Motion(dv_dq[:, k])
    m_dv = pin.Motion(dv_dv[:, k])
    dv_dq[:3, k] -= np.cross(M.translation, m_dq.angular) # THIS DOESN'T MATCH WITH NUMDIFF
    dv_dv[:3, k] -= np.cross(M.translation, m_dv.angular)
  return dv_dq, dv_dv

def numdiff(model, q, v, frameId, eps=1e-8):
  # Allocate data
  dv_dq = np.zeros((6, model.nv))
  dv_dv = np.zeros((6, model.nv))
  data = model.createData()
  q_plus = copy.copy(q)
  v_plus = copy.copy(v)
  # Compute current cartesian velocity
  jointId = model.frames[frameId].parent
  jMf = model.frames[frameId].placement
  pin.forwardKinematics(model, data, q, v)
  fjMo = pin.SE3(data.oMi[jointId].rotation.T, jMf.translation)
  v0 = fjMo.actInv(data.v[jointId])
  # dv_dq
  for k in range(model.nv):
      q_plus[k] += eps
      pin.forwardKinematics(model, data, q_plus, v)
      fjMo = pin.SE3(data.oMi[jointId].rotation.T, jMf.translation)
      dv_dq[:,k] = (fjMo.actInv(data.v[jointId]) - v0).vector / eps
      q_plus[k] -= eps
  # dv_dv
  for k in range(model.nv):
      v_plus[k] += eps
      pin.forwardKinematics(model, data, q, v_plus)
      fjMo = pin.SE3(data.oMi[jointId].rotation.T, jMf.translation)
      dv_dv[:,k] = (fjMo.actInv(data.v[jointId]) - v0).vector / eps
      v_plus[k] -= eps
  return dv_dq, dv_dv

model = robex.loadTalosArm().model
frameId = model.getFrameId("gripper_left_fingertip_1_link")

q = pin.randomConfiguration(model)
v = np.random.rand(model.nv)

dv_dq_nd, dv_dv_nd = numdiff(model, q, v, frameId)
dv_dq_an, dv_dv_an = analytical(model, q, v, frameId)

#assert(np.linalg.norm(dv_dq_an - dv_dq_nd) < 1e-5) # THIS FAILS
assert(np.linalg.norm(dv_dv_an - dv_dv_nd) < 1e-5)

print "Numdiff"
print "dv_dq:"
print dv_dq_nd
print
print "dv_dv:"
print dv_dv_nd

print
print "Analytical"
print "dv_dq:"
print dv_dq_an
print
print "dv_dv:"
print dv_dv_an

Thanks,
Carlos

For clarity of future readers, my proposed code works well. The problem was in the numdiff routine that needs to consider the frame translation. In any case, #1255 proposes a function to retrieve it for you, i.e. getFrameVelocityDerivatives.
I will close this issue! @jcarpent thank you again for the code and explanations in #1255.


Server side zod schema

Hello,
I've encountered an issue when using server actions with Next.js and attempting to validate phone numbers on the server side.
Currently, the documentation/code suggests using:
import { isValidPhoneNumber } from "react-phone-number-input";
However, this import is client-side oriented and might not be suitable for server-side validations due to its dependency on the react-phone-number-input package, which is primarily designed for client-side React applications.
For server-side validations within Next.js server actions, it's more appropriate to use:
import { isValidPhoneNumber } from 'libphonenumber-js';

Hi,
I think this could be mentioned in the readme for the time being.


Interfaces Overview: DHCPv6 Reload / Release / Renew buttons missing

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[x] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Describe the bug
When the IPv6 configuration type of an interface is set to DHCPv6, but the IPv4 configuration type is not set to DHCP, no 'DHCP Reload / Release / Renew' buttons are shown in 'Interfaces: Overview'.
To Reproduce
Steps to reproduce the behavior:

Go to 'Interfaces: WAN', set 'IPv4 Configuration Type' to DHCP and 'IPv6 Configuration Type' to DHCPv6, save & apply.

See result: Both IPv4 and IPv6 addresses get released.
Go to 'Interfaces: WAN', set 'IPv4 Configuration Type' to 'Static IPv4' or 'None', save & apply.
Go to 'Interfaces: Overview', see error: 'DHCP Reload / Release / Renew' buttons are missing.

Expected behavior
DHCPv6 reload / release / renew feature should be available for interfaces without DHCPv4.
Additional context
There is code in status_interfaces.php to display dedicated buttons for DHCPv6:
https://github.com/opnsense/core/blob/f609a9cbb1a814023fc4d019a8abc65955a02d8e/src/www/status_interfaces.php#L141-L153
But there is no code in get_interfaces_info() which would actually return 'dhcp6link' information.
Environment
OPNsense 23.1.a_102 (amd64, OpenSSL)
Hyper-V Gen2 Ver10.0

Caveat as per commit message 0592276. We could move the thing "up" to the status and just show a link type thing (dhclient and dhcp6c up indicator). SLAAC also missing. Maybe you want to give it a try.

Thanks @fichtner, mostly works. Except for the DHCPv6 "Release" button not switching to "Renew" after a release.  Might have to find another way to determine DHCPv6 state.
Of course it would be nice if we could reload / release / renew DHCPv4 and DHCPv6 independently, but that's probably quite a bit of work.

@maurice-w I can offer f3b51b3 for the status mismatch
In theory interface_bring_down() could split between IPv4 and IPv6 but might end up tearing the other one down as well as a side effect if one is connected over the other. For interface_configure() I think we should stay clear of partial reconfiguration so we would end up restarting the other one as well anyway even if it wasn't killed before.
According to the code here we could make the IPv4/IPv6 individual "up" stay but simply move the buttons to the "link" state in order to manage expectations about what would happen. I'll think about it some more.
Cheers,
Franco

@fichtner, may I suggest maurice-w/core@a57f179?

@maurice-w LGTM, want to create a PR or shall I cherry-pick this directly?


Fix iOS stuttering

WIP
Fixes the stuttering issue on iOS.
Android is still having issues so I've disabled it entirely until we sort those out.
Manual Browser Testing

[x] iOS 9 Safari
[ ] iOS 9 Chrome
[ ] iOS 8 Safari
[x] Android (formatting disabled)
[ ] IE9
[ ] IE10
[ ] IE11
[ ] Edge
[ ] Mac Chrome
[ ] Mac Safari
[ ] Mac Firefox
[ ] Mac Opera
[ ] Windows 10 Chrome
[ ] Windows 10 Firefox
[ ] Mac Firefox 17

Tests

[x] Unit tests
[ ] Integration tests
[ ] Audit unit tests to see if additional coverage needs to be added/adjusted

There's something wrong. In this example:

I typed 2a and expected it to render as --2xaz, but it rendered as --2xxz instead.

Fixed https://github.com/braintree/restricted-input/pull/10#issuecomment-243603466

Superseded by https://github.com/braintree/restricted-input/pull/13


Best way to convert other formats to cooler

Hi, I have several matrices stored in various format like hic format, and output from mirnylib *.hm, and plain text matrix. What's the best way to convert these matrices to cooler? I can easily read these matrices into memory but it seems that the cooler.io.create() is not functioning now.
One other issue is that these matrices may already be balanced. Can the cooler store counts other than int32, like float32?

If you have a fragment dataset from hiclib, cooler can read it directly to make a cooler file.

Thanks, but what if I don't have hdf5 fragment dataset? Some dataset we only have *.hm file available.

Hi Nan,
For hic files, you can use @carlvitzthum's hic2cool.
The cooler.io.create should be functioning, but the API has changed from what is shown in the tutorial. For now, if you have a whole genome .hm file that fits in memory, you can load that as a numpy array and use the cooler.io.DenseLoader to load it into a cooler. Make sure you know the exact set of chromosomes, their order and the resolution.
Here's an example using test data in the repo:

binsize = 2000000
chromsizes = pandas.read_csv(
    'tests/data/hg19-chromsizes.select.txt',
    sep='\t',
    names=['name', 'length']).set_index('name')['chr1':'chrX']['length']   # only contains chr1-22+X
bins = cooler.binnify(chromsizes, binsize)
heatmap = np.load('tests/data/IMR90-MboI-matrix.2000kb.npy')

iterator = cooler.io.DenseLoader(heatmap)
cooler.io.create('/tmp/output.2000kb.cool', chromsizes, bins, iterator, assembly='hg19')

Currently, cooler.io.create coerces the count column to integer dtype, as it is intended to hold the raw counts (with optional additional columns storing transformed data, but this currently has to be done manually). If you have only corrected data and have lost the original raw counts, you can hack cooler to store counts as floats by monkey patching the constant cooler._writer.COUNT_DTYPE = np.float32 or np.float64 before following the above recipe.
If you have our old byChr files, there is no direct way to convert them yet. If they aren't too large, you could concatenate them into one numpy array and use the method above. We can also add a new matrix loader for that file type if necessary.

Thanks Nezar! Very informative.


Create Afterpay.md

forgot to add company file the first time.

@all-contributors please add @whysogreen for documentation


Conflicts with mootools

Running .$slider.trigger('setPosition', []); causes mootools setPosition method to be called (http://mootools.net/core/docs/1.6.0/Element/Element.Dimensions#Element:setPosition)

Just add https://cdnjs.cloudflare.com/ajax/libs/mootools/1.6.0/mootools-core.min.js as a source and run to see the error
More Details

Which browsers/versions does it happen on? Chrome
Which jQuery/Slick version are you using? 1.6.0
Did this work before? before what?

My recollection is that $slider.slick('setPosition') is how that works, but it's been a minute since I've used it.

I've just faced same issue.
method unslick

I have the same problem. As soon as you have slick and mootools both loaded the system crashes. Changing the line in slick to  $slider.slick('setPosition') does not work.

I have the same issue, how can you solve this?

I could not find any way :(

I "solved" it by changing the name of the function setPosition in mootools-core.js to setPosition1 (for example) so that it won't conflict with slick/jquery. Of course you have to be sure you're not using that function with mootools...

This is not very convenient... :/

I found the same problem in Joomla. @sonosergio changed the name of the setPosition method inside slick on this issue https://github.com/kenwheeler/slick/pull/2191, but I still don´t understand why mootools triggers

Worked out well for me after changing 'setPosition' to 'setPosition1' in all the places inside slick js file

Is any other solution to solve this problem without hacks one of fw?


Change DAYZ_111 to DAYZ_1_11

https://github.com/ralian/eai/blob/cad0310af165f13aa354e67d1c17bfc9f0a067a7/Scripts/4_World/eAI/Classes/eAIActionManager.c#L184

Fixed, thanks


[6.1] Sema: Prioritize @_exported imports over local non-public imports

Update the logic selecting the most restrictive import for a given reference to account for @_exported imports from the local module. We should always prioritize @_exported imports from the local module over more restrictive same file imports. Only if an import from the same file is also public we prefer it as it's more useful for diagnostics and generally recommended to locally declare dependencies.

Scope: Affects users of access-level on imports also using @_exported imports from Swift code.
Risk: Low, this change allows to use more imported types in API so it only leads to less errors.
Reviewed by @tshortli
Cherry-pick of #78005
Resolves rdar://140924031

@swift-ci Please test

@swift-ci Please test Linux


Use ldap authentication in vsftpd

When I use ldap authentication in vsftpd, I found nss-pam-ldapd and linux-pam installation package, no pam_ldap.conf file configuration, and pam authentication can not be used, I have been tested on debian docker, please tell me how to do, thank you very much .

Can you give some more debug information so we can reproduce (Dockerfile, installed packages, etc.)?

FROM alpine

RUN set -ex \
    && apk add --no-cache vsftpd nss-pam-ldapd linux-pam

tree /etc/pam.d/
/etc/pam.d/
├── base-account
├── base-auth
├── base-password
├── base-session
├── base-session-noninteractive
├── other
└── su

FROM debian

RUN set -ex \
    && apt-get update \
    && apt-get install -y libnss-ldapd vsftpd

tree /etc/pam.d/
/etc/pam.d/
|-- chfn
|-- chpasswd
|-- chsh
|-- common-account
|-- common-auth
|-- common-password
|-- common-session
|-- common-session-noninteractive
|-- cron
|-- login
|-- newusers
|-- other
|-- passwd
|-- runuser
|-- runuser-l
|-- su
|-- su-l
`-- vsftpd

no pam_ldap.conf file configuration in alpine, and pam authentication can not be used


Refactor affine_transform: use general function apply on coordinates

In pygeos, we have a general apply method to apply a function on the coordinates and recreate geometries from those transformed geometries (we probably need to rename it to transform, but that's another issue).
This speeds up the affine transforms again a bit (which saw a slowdown in https://github.com/Toblerity/Shapely/pull/983). The specific example of https://github.com/Toblerity/Shapely/pull/983#issuecomment-708276648 is now again at the same speed as with released Shapely:
In [1]: from shapely.geometry import Polygon

In [2]: poly = Polygon([(0,1), (1, 2), (2, 1), (1, 0), (0, 1)])

In [3]: %timeit poly.minimum_rotated_rectangle
275 µs ± 6.98 µs per loop (mean ± std. dev. of 7 runs, 1000 loops each)

Now, for single geometries, I don't think we need to expect speed-ups, certainly not compared to the cython code (that I removed here).

Note that the tests are still failing because pygeos.apply doesn't support 3D coords, yet (working on that)

Coverage increased (+0.7%) to 76.03% when pulling 8199b6d0dbe0b8b3c60f2132b9d06f0dda8e888d on jorisvandenbossche:refactor-affine-transform into 4f28db93b4514de2a64cfd21059bac65ec9449bc on Toblerity:shapely-2.0.


Unable to pause the stream

Expected result: The stream should pause on using the 'p' command.
Actual result: The stream continues even after using the 'p' command.
Steps to reproduce:

Install rainbowstream through pip.
Run rainbowstream.
Pause stream with the 'p' command and wait until refresh timer expires.

Environment: Fedora 32 x64, bash 5.0.17

Yes, the same thing with my installation. It's not really a big deal for me, I never pause the stream anyway but although it says that it's paused it carries on regardless. Arch Linux x86_64:  zsh 5.8 (x86_64-pc-linux-gnu) Tilix

@johnpiers Ordinarily, that would be fine for me too, except the behaviour where a tweet I'm in the middle of composing is sent prematurely and incomplete when a stream update is displayed.

@Moodyloaf True, didn't really consider that. Then I guess that would most definitely be an irritant. :-)

This happened to me also.

I'm having the same problem, is there any way to set this pause on the config file?

This is still happening to me and yes it's irritant

I even tried to edit HEARTBEAT_TIMEOUT to 1 and STREAM_DELAY to 9999 but nothing works.


Entity condition position

Hey I'm on a pretty big smp/anarchy server based around the origins mod.  Many people are making custom origins and one  of the dev's is trying to make a power that activates when the player is above a certain y coordinate.  Currently the only idea I have is checking the block they're on and getting the block y.  However, they want this to be available when the player isnt touching a block - since the origin includes alot of levitation and stuff.  Is there an existing condition for comparing entity position - aka y position.

You can use in_block instead of on_block - that will also work when not
standing on a block.
On Thu, 11 Mar 2021, 09:20 JDParker714, @.***> wrote:

Hey I'm on a pretty big smp/anarchy server based around the origins mod.
Many people are making custom origins and one of the dev's is trying to
make a power that activates when the player is above a certain y
coordinate. Currently the only idea I have is checking the block they're on
and getting the block y. However, they want this to be available when the
player isnt touching a block - since the origin includes alot of levitation
and stuff. Is there an existing condition for comparing entity position -
aka y position.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/apace100/origins-fabric/issues/192, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALWSQSWXXFOFEL7ZAW6BDZTTDBVNLANCNFSM4Y7ZREGQ
.


Avoid unnecessary mkdir /app in the Dockerfile.

This is needed in order to remain compatible with an upcoming change to govuk-ruby-base.

you could move up line 22 to 18 in the dockerfile

Good idea — I'll do that in a follow-up (rationale in https://github.com/alphagov/collections/pull/2973#issuecomment-1257736038)


[WIP ] Update Kubernetes to v1.10 version

ToDo List:

[ ] Add NVIDIA docker support.
[ ] Add CoreDNS support.
[ ] Update CNI plugins.
[ ] Change approve bootstrap from manually to automatically.
[ ] Improve roles.

hi, thanks you firstly. But , when to suppose v1.10 ??


NEW DEVICE: Thermo Fisher's QuantStudio PCR

Our facility is Elterman Center for Womens and Mens health,  CLIA 14D0984062
Contact: Ivan
Tim tagged in support box

Device was actually the 3b Blackbio TRUPCR - found out after swapping emails with Ivan and Anastasia at this facility. Device has been added to SR


tree-sitter grammar

Hi,
Seeing as the projet already has a syntax definition for vim it would make sense to also have a tree-sitter grammar which could be used by a number of editors (e.g. neovim).
I guess a good starting point would be the markdown grammar, but personally I'm having a hard time understanding how it works.
Looking forward to using djot in the future!

I don't know anything about tree-sitter, but this would be a great project for someone who does.

Made a start on this in the tree-sitter directory.

OK, it looks like we'd need to define some external scanners to make this work, since the regexes tree-sitter allows don't include any assertions. (We won't be able to distinguish open- and close-emph delimiters, for example.)  But tree-sitter does allow you to define external scanners in C, so maybe we could do this.

https://tree-sitter.github.io/tree-sitter/creating-parsers#external-scanners

I'd like to point out that this treesitter repository for commonmark is now recommended for use over the repository linked before.

Oh, thanks, that one is much easier to understand!

An interesting option for nvim would be to use the djot lua library itself to do the lexing, and call https://neovim.io/doc/user/api.html#nvim_buf_add_highlight()
directly to add the highlighting.
This would be 100% accurate.

That function wants to have (line, column) locations, but column is byte indexed. We could convert our current byte offsets to (line, column-byte) pairs by counting newlines.  Perhaps we should change the Lua library so that it can efficiently produce this output as well as absolute byte offsets.

This would be 100% accurate (but not incremental, which might be an issue on large files).

One of the reasons I'm really interested in djot is because editing CommonMark documents becomes a pain when the documents become fairly large. It'd be awesome if editing djot documents doesn't feel that way as well. 😅

Interesting.  Well, we could try the thing I suggest above and see how it works.  The parser is very fast, so maybe it will be okay.  It would require a few changes in the Lua library, but then it should be fairly straightforward.

While using the lua engine directly might be a good solution for highlighting, having a treesitter grammar has other advantages. In neovim, treesitter is also used for editing (selecting, deleting etc.) and navigation. Basically, as a common language abstraction, it makes integration between arbitrary plugins and arbitrary languages easier, resulting in a more homogeneous/higher level experience. So that would be a pro for a treesitter solution. Also, treesitter is supported by many editors (emerging ones like helix and zed), while a lua solution would need some integration code for each editor. (Pretty excited by the future of djot btw.)

Just coming here to say I'm so happy I saw a tree-sitter discussion already! Hoping to try out djot in Helix, so a tree-sitter parser would be phenomenal.

I accidentally wrote a Tree-sitter grammar that should be feature complete here: https://github.com/treeman/tree-sitter-djot
I also started a discussion about it: https://github.com/jgm/djot/discussions/280


[Candidate 2v2 Lion Heart] Title text is not concise

Issue Description
Title says: "Creator of this map: \n Adriane [Deathcryngthe]"
Expected behavior
Title says: "Map created by Adriane"
Screenshots

Fixed with https://github.com/xezon/GeneralsRankedMaps/commit/bdf1faea39459ae850e8e89e263550d7cf3436f1


No release for Ubuntu mantic

Hi,
I am on Ubuntu 23.10 mantic. When I followed the installation steps, I am seeing the following error with apt update.
 E: The repository 'https://packages.adoptium.net/artifactory/deb mantic Release' does not have a Release file.

When I checked the repository, I couldn't see any release for mantic. However Ubuntu 24.04 noble seems to be there for example.
curl -s https://packages.adoptium.net/artifactory/deb/dists|grep -i mantic; echo $?
1
curl -s https://packages.adoptium.net/artifactory/deb/dists|grep -i noble && echo $?
noble/     14-May-2024 14:26    -
0

I wonder if I am missing anything or there is a particular reason for it.
Thanks,

We don't always add the STS versions in, although since the packages are identical across all versions, feel free to just switch mantic for noble or any of the other ones as a workaround if you still need to use 23.10.

Closing on the basis that

Mantic (23.10) is now out of support
We don't currently have other STS versions included explicitly in the installer repositories
There is an easy workaround


Documentation of safety invariants

MmapMut::map_mut is an unsafe function, but there is not any documentation about why it is unsafe and which invariants the user have to hold.

#13 ?

https://docs.rs/memmap2/0.2.1/memmap2/struct.MmapMut.html#safety


Improve Side Bar Explorer-Outline visbility

1) In vscode
Current VSCode, Outline is below explorer,
not best to jump/check quickly methods/functions : need to navigate 2 explorer:

2) In pycharm
In Pycharm, outline of methods are directly below the file name in Explorer,
much easier to go through the methods quickly

This is a real time saver...

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Swaps: re-enable sponsored swaps with current swap provider

When the improved version is ready, lets re-enable.
It will help users who are otherwise stuck after bridging to aBTC or sUSDT using a bridge because they can't cover the gas fee.

My impression is that we disabled since the SDK functionality was buggy. cc @fbwoolf

My impression is that we disabled since the SDK functionality was buggy. cc @fbwoolf

Alex asked us to disable it.

On app.alexlab.co they fixed issue and re-enabled sponsored transactions. When the SDK is upgraded we can reintroduce it too. They will update us when it is ready.
This is from Chan at ALEX:

The current version of sponsored service on our app is different from those available on our SDK (which wallets use to integrate ALEX apps), and we are looking to upgrade the SDK in the near future.


"ResponseFactory::collection" returns an empty array without the resource key if an empty collection is provided.

I think it's not really logical to return an empty array in case if an empty collection is provided to "ResponseFactory::collection". I think the right way would be to return ["data" => []] assuming that no meta data is provided and the default resource key is used.

It already does this. Was fixed in a02e506faec7eb7043b7d606ee94e0c9847a4f17 and is available since 0.8.2.


How to automatically  generated code

Hi：
I want to submit PR , When I am finished modifying the code, the automatically generated code has an error， Do I need to regenerate the code when an error occurs ？，If needed then what are the steps ？
For example, I modify the struct parameter in types.go ,At this time, the unit test is run and the compilation fails，What to do when this happens ？

@wxuedong Just run make gen

@wxuedong Just run make gen

Thank you for your answer，
After executing make run  other problems occurred.
"Warning: could not find nor generate a final Conversion function for sigs.k8s.io/descheduler/pkg/api.StrategyParameters -> sigs.k8s.io/descheduler/pkg/api/v1alpha1.StrategyParameters"
"The following fields need manual conversion:"
-NodeAffinityType
-RemovePodsViolatingNodeTaints
"NodeAffinityType" and "RemovePodsViolatingNodeTaints" are new fields,How to solve this problem ?

You should add these new fields to both sigs.k8s.io/descheduler/pkg/api/types.go and sigs.k8s.io/descheduler/pkg/api/v1alpha1/types.go

You should add these new fields to both sigs.k8s.io/descheduler/pkg/api/types.go and sigs.k8s.io/descheduler/pkg/api/v1alpha1/types.go, and run make gen again

Two files have been synchronized .
Console output :
ls: _output/bin/golangci-lint: No such file or directory
./hack/update-generated-conversions.sh
./hack/update-generated-deep-copies.sh
./hack/update-generated-defaulters.sh
./hack/update-toc.sh
zz_generated_xxxx.go  files is not updated , the file is still old

You should add these new fields to both sigs.k8s.io/descheduler/pkg/api/types.go and sigs.k8s.io/descheduler/pkg/api/v1alpha1/types.go, and run make gen again

Two files have been synchronized .
Console output :
ls: _output/bin/golangci-lint: No such file or directory
./hack/update-generated-conversions.sh
./hack/update-generated-deep-copies.sh
./hack/update-generated-defaulters.sh
./hack/update-toc.sh
zz_generated_xxxx.go files is not updated , the file is still old

The old file is deleted, and no new file is created by re-make

@wxuedong what version of Go are you using?
/kind support

@wxuedong what version of Go are you using?
/kind support

go version go1.14.4

@wxuedong the descheduler master branch is currently using Go 1.16.1. Try using a newer Go version. I'm not sure this will fix the problem, but this is the only thing I can think of.

@wxuedong the descheduler master branch is currently using Go 1.16.1. Try using a newer Go version. I'm not sure this will fix the problem, but this is the only thing I can think of.

I have upgraded to version 1.16.3 and still have no solution ,  I am thinking if it has something to do with my local environment configuration

I have upgraded to version 1.16.3 and still have no solution , I am thinking if it has something to do with my local environment configuration

Were you able to figure out the cause? I'm experiencing this with go 1.17.

I have upgraded to version 1.16.3 and still have no solution , I am thinking if it has something to do with my local environment configuration

Were you able to figure out the cause? I'm experiencing this with go 1.17.

The issue was https://github.com/kubernetes-sigs/descheduler/pull/316


[Spanish] localize 'Site Reliability Engineering'

Describe your changes
Translated site-reliability-engineering.md in spanish
Related issue number or link (ex: resolves #issue-number)
resolves #1774
Checklist before opening this PR (put x in the checkboxes)

[X] This PR does not contain plagiarism

don’t copy other people’s work unless you are quoting and contributing it to them.

[X] I have signed off on all commits

signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

Thanks for your contribution!
Just some suggestions to update the concepts already localized adding /es/

Applied the suggested changes

@Azanul Spanish translations must target to dev-es branch. Maybe you'll require to redo this PR using dev-es branch as a base instead of main

For now, I've force pushed to resolve it.

@ramrodo @krol3 There's something wrong here. I didn't change any of those. Someone else has also faced the same issue. #1855

This error is probably due to a rebase problem. We should rebase our branch and keep it updated against the main branch. With that, the PR/branch should not include other changes.

@ramrodo Resolved conflicts


Windows 11 window buttons broken

On a fresh install of Windows 11, in addition to the missing application icon at the top left, the window buttons are broken.
In the screenshot below, you can see that the button icons themselves aren't aligned by the hitbox. Clicking on minimize or close don't appear to do anything. Clicking on the restore (middle) button "freezes" the app, as if there is an invisible dialog that makes everything unclickable, but doesn't actually cause the app to go into Not Responding.

for me also

I've seen this, but not sure how to solve it. custom-electron-titlebar package is used for this. Works fine on OSX and I don't have easy means of testing it on windows. Even when I did once I don't get why it crashes and looks broken in the first place. Any help would be appreciated.

any news on this topic? I'm having this problem too

Not yet, and won't be able to for a couple of weeks. And then I need to find a windows laptop 🤷

also for me in windows 11 and ubuntu desktop 22.04

I found a good alternative:  https://github.com/qishibo/AnotherRedisDesktopManager

This is also happening on Windows 10.


Iris: Fixed an issue where the send button would be disabled when a step execution error appeared.

Checklist
General

[ ] I tested all changes and their related features with all corresponding user types on a test server.
[x] This is a small issue that I tested locally and was confirmed by another developer on a test server.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Client

[x] Important: I implemented the changes with a very good performance, prevented too many (unnecessary) REST calls and made sure the UI is responsive, even with large data.
[x] I followed the coding and design guidelines.
[x] Following the theming guidelines, I specified colors only in the theming variable files and checked that the changes look consistent in both the light and the dark theme.
[ ] I added multiple integration tests (Jest) related to the features (with a high test coverage), while following the test guidelines.
[ ] I added authorities to all new routes and checked the course groups for displaying navigation elements (links, buttons).
[ ] I documented the TypeScript code using JSDoc style.
[x] I added multiple screenshots/screencasts of my UI changes.
[ ] I translated all newly inserted strings into English and German.

Changes affecting Programming Exercises

[ ] I tested all changes and their related features with all corresponding user types on Test Server 1 (Atlassian Suite).
[ ] I tested all changes and their related features with all corresponding user types on Test Server 2 (Jenkins and Gitlab).

Motivation and Context

When the execution step fails and a detailed error message is displayed at the bottom of the chat window, the user cannot use the send button and clicking the retry button does not remove the error message. The error message disappears and the send button is enabled only when the user reloads the page.
Description

The error information of step execution is not stored in the state storage, but the detailed error information is bound to the specific step and displayed below the step.
Steps for Testing

Prerequisites:

1 Instructor
1 Programming Exercise with Code Editor Iris enabled, you can use exercise 10

Log in to Artemis
Navigate to "edit in editor" of an exercise
Chat with Iris, when a step failed, the error should be displayed below the step and the send button should not be disabled.

Testserver States

[!NOTE]
These badges show the state of the test servers.
Green = Currently available, Red = Currently locked

Review Progress

Performance Review

[ ] I (as a reviewer) confirm that the client changes (in particular related to REST calls and UI responsiveness) are implemented with a very good performance
[ ] I (as a reviewer) confirm that the server changes (in particular related to database calls) are implemented with a very good performance

Code Review

[ ] Code Review 1
[ ] Code Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Exam Mode Test

[ ] Test 1
[ ] Test 2

Test Coverage

Screenshots

Before

After

The send button is still disabled when an error appears, in this case that there is no API token:

So: Not working as expected on ts9

The send button is still disabled when an error appears, in this case that there is no API token:
So: Not working as expected on ts9

This is not included in this PR and it's the expected behavior

For me it always executes successfully. Is this intended?

Yes, this is the expected behavior of the feature and this PR is to improve the way errors are handled when execution fails. If it's successful, then it can not be tested.


Implement IsPlayingUpdate for ELinuxVideoPlayer/VideoPlayerPlatform

Overview
This PR is to enable updates sent over the event channel back to dart for changes in playing and pausing. This allows VideoEventType.IsPlayingStateUpdate to be triggered (this event type was added recently by the video_player_platform_interface).
Help
Help needed: I was unable to find a way to test this. Inside of the main.dart of example, given the VideoPlayerController, how do I receive events? Or do I have to make changes inside the controller implementation to receive these events in the UI layer?
Other
Please let me know if there was another implementation that you had in mind :)

First of all, do you agree to delegate all of rights related to this pull-request (your change, code) to Sony?

Help needed: I was unable to find a way to test this. Inside of the main.dart of example, given the VideoPlayerController, how do I receive events? Or do I have to make changes inside the controller implementation to receive these events in the UI layer?

Please verify it locally in debug and release flavors.

First of all, do you agree to delegate all of rights related to this pull-request (your change, code) to Sony?

Yes I do.

cleaned up PR and ready from my side 👍🏾

Basically, please check the differences between current plugin version and a new one when updating flutter official plugin's version. In addition, please update pubspec.yaml and CHANGELOG.md too.

@HidenoriMatsubayashi sorry for the delay, I have just resolved your comments. I have added to Changelog as well. Please let me know if there is anything else you need.


Is it possible to add context menus to each widget?

I currently edited the eww code to convert a json array to render the photo's context menu.

Pretty cool. Care to publish the dotfiles or the relevant parts at least? :)

@MartyMcFlyInTheSky I do this

Edit the eww code to add the menu attribute, it converts a json to a gtk menu. The idea is that elkowar can make any widget able to carry this attribute to add context menus anywhere, but in a more beautiful way. Like this
(box
  :menu (menu
    (option
      :label "Show weather"
      :actionn "${EWW_CMD} update show_weather=true"
    )
    (option
      :label "Submenu"
      :submenu (menu
        (option
          :label "Item 1"
        )
      )
    )
  )
)


I'm concerned about bitvec dependency

See: https://github.com/zkcrypto/ff/issues/91

The author responded, there has been some progress. Nothing actionable anymore.


Add new executable flag to check if main or release version

What does this Pull Request accomplish?
This PR adds new command line functionality by allowing clients to see if their executable version is from a main or release branch.
Why should this Pull Request be merged?
To be able to easily determine if clients' version of ni_grpc_device_server is from a main or release branch
What testing has been done?
When the branch is from main (name doesn't begin with 'releases')

When it is a release version (begins with 'releases')

Closes #828

@bkeryan The validate_python step failed here for not having black locked to 23.3.0.
https://github.com/ni/grpc-device/actions/runs/7731440933/job/21237217346?pr=1038#:~:text=Using version ^24.1.1,45
From the other PR I completed to resolve this issue, that line of highlighted text doesn't appear, making me assume that PR used 23.3.0. Even though that PR is merged, does it take a while for new PR's to use the proper version?

@bkeryan The validate_python step failed here for not having black locked to 23.3.0. https://github.com/ni/grpc-device/actions/runs/7731440933/job/21237217346?pr=1038#:~:text=Using version ^24.1.1,45
From the other PR I completed to resolve this issue, that line of highlighted text doesn't appear, making me assume that PR used 23.3.0. Even though that PR is merged, does it take a while for new PR's to use the proper version?

Oh, I'm pretty sure the GitHub actions just run off the code in the PR branch. So you'll have to merge main into this branch and push up to get the PR build to pass I think.
Unlike how we have AzDo set up where I think it'll merge the PR branch with main before running any of the checks. We might've considered doing that for this repo at some point (I can't recall) but it was never high enough value to investigate.

@astarche @maxxboehme @reckenro Could y'all please approve this PR if there are no further changes?


improve Rust close-account fn performance

This adds a minor performance improvement to the close_account function in the Shank example Counter Program utils file. Instead of filling the account with zeros, a more efficient way to close accounts is to reassign to the system program and then realloc the length to zero. This is how Anchor does it, for example.

Nice, thanks for improving the program template!
It looks like we currently don't import the system-program crate though so we get the following error when we try and cargo check the generated program.
Should we add this to the program's Cargo.toml? Or should we just inline the ID here?

Nice, thanks for improving the program template!
It looks like we currently don't import the system-program crate though so we get the following error when we try and cargo check the generated program.
Should we add this to the program's Cargo.toml? Or should we just inline the ID here?

The Cargo.toml already has the solana_program crate which is where system_program module comes from so I just needed to add it to the utils.rs import statement. The commit I just pushed should fix the issue.

Oh I see! Awesome, just waiting for CI to pass and then I'll merge and publish a new release.


discovery: true and authKey

When I set both discovery: true and the authKey in the options, I get a bunch of errors about wrong authKey.  It seems that maybe setting discovery:true causes the driver to try to read conf information before properly connecting using the authKey.    Using exactly the same options, but without discovery:true, and everything works fine.
Any ideas?

What are the errors? Stackstrace?

It's not a stack trace.  The error just says that the auth key is incorrect.

Just tried with 4 servers, it works fine now.
How many servers do you have? How do you run them? Typically are you running them inside containers? If yes do you start them with --bind ...?

I have tried with 1 server and with 3 servers, using both RethinkDB 2.0.4 and 2.1 and in all cases I get the same behavior (see log below).    I use the bind=all option in my /etc/rethinkdb/instances.d/default.conf (that's the only option except for join for some nodes).
If I remove discovery:true, but keep the other (hosts) option the same, then everything works fine.
I'm running the VM's as Google Compute Engine instances running Ubuntu 15.04 (very clean minimal install).
Creating a pool connected to db0:28015
Creating a pool connected to db1:28015
Creating a pool connected to db2:28015
Creating a pool connected to <IP_ADDRESS>:28015
Creating a pool connected to <IP_ADDRESS>:28015
Creating a pool connected to <IP_ADDRESS>:28015
Removing pool connected to: db0:28015
Draining the pool connected to db0:28015
Removing pool connected to: db2:28015
Draining the pool connected to db2:28015
Entering slow growth mode
Fail to create a new connection for the connection pool. Error:{"message":"Server dropped connection with message: 'ERROR: Incorrect authorization key.\n'."}
Fail to create a new connection for the connection pool. Error:{"message":"Server dropped connection with message: 'ERROR: Incorrect authorization key.\n'."}
Fail to create a new connection for the connection pool. Error:{"message":"Server dropped connection with message: 'ERROR: Incorrect authorization key.\n'."}
Fail to create a new connection for the connection pool. Error:{"message":"Server dropped connection with message: 'ERROR: Incorrect authorization key.\n'."}
Fail to create a new connection for the connection pool. Error:{"message":"Server dropped connection with message: 'ERROR: Incorrect authorization key.\n'."}
Fail to create a new connection for the connection pool. Error:{"message":"Server dropped connection with message: 'ERROR: Incorrect authorization key.\n'."}
Fail to create a new connection for the connection pool. Error:{"message":"Server dropped connection with message: 'ERROR: Incorrect authorization key.\n'."}
Fail to create a new connection for the connection pool. Error:{"message":"Server dropped connection with message: 'ERROR: Incorrect authorization key.\n'."}
Fail to create a new connection for the connection pool. Error:{"message":"Server dropped connection with message: 'ERROR: Incorrect authorization key.\n'."}
Fail to create a new connection for the connection pool. Error:{"message":"Server dropped connection with message: 'ERROR: Incorrect authorization key.\n'."}
Fail to create a new connection for the connection pool. Error:{"message":"Server dropped connection with message: 'ERROR: Incorrect authorization key.\n'."}
...

The line Creating a pool connected to <IP_ADDRESS>:28015 above looks suspicious since why is it trying to connect again to a node that is already there?
Here's what one of the entries of the server_status table looks like (it does have size 3):
{"id":"680f661a-445e-4213-9eff-88e9db2b830e","name":"db1","network":{"canonical_addresses":[{"host":"<IP_ADDRESS>","port":29015},{"host":"<IP_ADDRESS>","port":29015}],"cluster_port":29015,"hostname":"db1","http_admin_port":8080,"reql_port":28015,"time_connected":"2015-08-16T22:22:14.360Z"},"process":{"argv":["/usr/bin/rethinkdb","--daemon","--config-file","/etc/rethinkdb/instances.d/default.conf","--runuser","rethinkdb","--rungroup","rethinkdb","--pid-file","/var/run/rethinkdb/default/pid_file","-directory","/var/lib/rethinkdb/default/data"],"cache_size_mb":1264.615234375,"pid":2196,"time_started":"2015-08-16T22:15:22.703Z","version":"rethinkdb 2.1.1+1+g989a97~0vivid (GCC 4.9.2)"}}

Any hints or ideas?

Are <IP_ADDRESS>, <IP_ADDRESS> and <IP_ADDRESS> the ips of db0, db1 and db2?
Do here what happens is the driver connects to db0, db1, and db2.
It then retrieve the ips of the servers from the table server_status.
To match a connection to a server, we check a match for : but not for :
So because we don't find a match, we kill the pool connected to db* and just use the pool connected to 10.240.*.*.
I opened https://github.com/neumino/rethinkdbdash/issues/134 to use the hostname to match a pool with a server too.
My guess right now is that one of the 10.240.*.* ip host a RethinkDB instance with another auth key.

I just pushed 2.1.6 with the fix for #134. You should stay connected to db0, db1, db2 and probably won't get the error anymore.
If you still have an issue let me know. My best guess is still that one of the ips to which rethinkdbdash connects, hosts another instance of RethinkDB with a different key.
I tried to set up a cluster with an authkey and connect via a hostname that RethinkDB doesn't know to trigger the same behavior and I have no issue reconnecting to the cluster.

Closing, if you still have the issue with 2.1.6, feel free to re-open


Websocket APIs

hey @jjxtra !
Do you habe any plans regarding a suuport for websocket APIs? For instance, Bittrex and Gdax have them. That would be super useful.
cheers,
Wojtek

I may do this in the future if there is enough demand.
-- Jeff
On Wed, Dec 6, 2017 at 6:20 AM, Wojtek<EMAIL_ADDRESS>wrote:

hey @jjxtra https://github.com/jjxtra !
Do you habe any plans regarding a suuport for websocket APIs? For
instance, Bittrex and Gdax have them. That would be super useful.
cheers,
Wojtek
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jjxtra/ExchangeSharp/issues/5, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABNoGKxaIC1T1Gn8HYpHoM9jWdVDLh7hks5s9pSOgaJpZM4Q35dO
.

Which web socket API are your highest priority?

@jjxtra We can split the work, I was playing recently with gdax, https://github.com/wklm/gdx-ws.
I could start with proposing an interface for all websokcet-based apis, and implement it for a few.
Nice thing them is that they're usually open, so we don't need to deal with authentication, rate limits etc
If this is fine would get back to you with a PR in ~ 2 weeks.

Awesome. Let's nail down the interface first then we can start coding it up.

Has anything come of this yet?
I've added this myself today before seeing your message. I haven't spent any time really at all considering what the best interface should be, but it looks something like this:
`ExchangePoloniexAPI polo = new ExchangePoloniexAPI();
polo.StartOrderBook();
        Console.WriteLine("POLO:");
        Console.WriteLine(polo.bid["LTC-BTC"] + " " + polo.ask["LTC-BTC"]);
        Console.WriteLine(polo.bid["ETH-BTC"] + " " + polo.ask["ETH-BTC"]);

        ExchangeGdaxAPI gdax = new ExchangeGdaxAPI();
        gdax.StartOrderBook();

        Console.WriteLine("\nGDAX:");
        Console.WriteLine(gdax.bid["LTC-BTC"] + " " + gdax.ask["LTC-BTC"]);
        Console.WriteLine(gdax.bid["ETH-BTC"] + " " + gdax.ask["ETH-BTC"]);`

Which results in this:
`POLO:
0.01879998 0.0188
0.04901001 0.0490103
GDAX:
0.01894 0.01895
0.04914 0.04915`
It starts a new thread when you call StartOrderBook(), which manages the bid/ask prices so that they are always up-to-date using the websocket API.

Web socket is planned and will probably be an interface property on the IExchangeAPI interface, like an IExchangeWebSocketAPI that will handle the connections. @wklm is looking into proposing an interface.


Investigate how to deploy MongoDB Docker image

Might need to look at the following files

docker-compose.yaml
mongo.env
maybe the config.py in flask/app


When clause .drone.yml

I find it strange that the when clause in the publish block of the .drone.yml prevents a build, rather than just a publish.
What was the reasons for this? Would a when clause for both build & publish not make sense?

a when clause in the publish section does not impact the execution of a build


fix: Fix proc-macro server not using the supplied span in Ident::new

This makes the hack introduced by https://github.com/rust-analyzer/rust-analyzer/pull/10899 obsolete.
For async-trait specifically, (unfortunately, but technically correct) due to how async-trait works, the self local now renders white, as it now resolves to the __self local introduced by the attribute.

bors r+

bors retry


Component Inspector not expanding some components

Version
6.0.0-beta.17
Browser and OS info
Chrome 93 / MacOS 11.5.2
Steps to reproduce
Launch app (in none production build) running Vue 2.6.14
Open Chrome DevTools panel
Click Vue tab
Note load time of Vue.js DevTools plugin is increased from beta.15
Note there may be some components that are not expandable in the Component Inspector list
I will try to include some context to this bug, as a re-produceable repo is difficult to provide given the size and complexity of the project I'm experiencing this issue with.
We have a fairly large SaaS platform that runs on Vue 2.6.14, along with Vuex 3.6.2, Vue Router 3.5.2 and the UI framework is being provided using Vuetify 1.5.24.
For a long while (prior to beta.15 the only reliable experience I've been able to get is to use Firefox with the v5 version of Vue.js DevTools extension (Chrome with the same version of the DevTools extension would simply bomb out).
When beta.15 of v6 came out (with support for Vue.js v2) I was pleased to be able to use Vue.js DevTools in Chrome regularly with no performance issues, and I can honestly say it was a great experience again, so thanks for this.
However, when launching Chrome with the beta.17 version of Vue.js DevTools this morning I can report that for some reason the performance (load-time) has probably doubled, and also I'm experiencing that some components are not expandable in the Component Inspector. This makes the Launch in Editor feature useless for any components nested within the non-expandable component.
The particular component that isn't expandable happens to be v-content component of Vuetify v1.5.x. As a result that makes the majority of the nested components properly un-inspectable.
I will try to reproduce this with a simplified codesandbox example when I get the chance.
What is expected?
All components in the Component Inspector are expandable
What is actually happening?
Certain components are not expandable

Same as https://github.com/vuejs/devtools/issues/1563

This issue has since mutated (?), now the component that I couldn't previously expand is simply not displayed in the Component Inspector. This is using beta.18.
There's also an additional warning in the console:

DevTools failed to load source map: Could not load content for chrome-extension://ljjemllljcmogpfapbkkighbhhppjdbg/build/fast-copy.js.map: HTTP error: status code 404, net::ERR_UNKNOWN_URL_SCHEME

@Akryum I've finally managed to get a repo where the problem is happening.
When DevTools (v6.0.0-beta.18) is open, go to Component Inspector and expand:
<Root>  >  <app>  >  <v-app>  >  <v-content>  >  <PageDashboard>  >  <PageWrapper>
You'll see that <PageWrapper> can't expand further. I hope this helps.

Just wanted to say thanks @Akryum - I'm able to use beta.19 quite reliably in Chrome. Will feedback separately if I come across anything else.


Laravel 5.7

Hello,
Here‘s a PR to update composer.json to support Laravel 5.7:
https://github.com/squareboat/sneaker/pull/25
Regards

Yeah, thank you. Would be nice if it soon gets accepted.


Add rewrite_polyfills to custom builds tutorial

Custom builds tutorial got left out of the recent compiler update PR, so this amends it. While I'm at it, I've also added use_types_for_optimization, which will help reduce build size.

Thanks for spotting this @probins.

ok, removed references to use_types_for_optimization (I can also remove it from my own configs :-))

thanks @probins


No option to list schemes

Xcodebuild has the -list option to list schemes, but there is no corresponding option in xctool.

We would be happy to review PR!

Closing since we aren't going to work on this request in the nearest future. We will be happy to review your PR though!


EnhancementRefactoring maxCompute catalog using Storage API.(#40225 , #40888 ,#41386 )

bp #40225 , #40888 ,#41386
Among them, #40225 is the new api of mc,
#40888 is used to fix the bug when reading null between the new and old apis,
#41386 is used for compatibility between the new and old versions
Proposed changes
Issue Number: close #xxx

run buildall

run buildall

run buildall


Improvements/topics

Improvements on communication of each topic because of some fragments presented inside of topics.

First off, thanks for the PR @kaiquewdev! However, as a native english speaker some of the updates you are suggesting don't actually make sense in English.
I'd like to leave them as is for now, but respectfully, thank you for taking the time in putting together this PR. 😄 ❤


Remove invalidcommitmsg check

Remove invalidcommitmsg plugin for ocp-power-automation repo

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: mkumatag
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [mkumatag]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/cc @bpradipt @yussufsh

@mkumatag: GitHub didn't allow me to request PR reviews from the following users: bpradipt.
Note that only ppc64le-cloud members and repo collaborators can review this PR, and authors cannot review their own PRs.

In response to this:

/cc @bpradipt @yussufsh

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/cc @bpradipt

@mkumatag: GitHub didn't allow me to request PR reviews from the following users: bpradipt.
Note that only ppc64le-cloud members and repo collaborators can review this PR, and authors cannot review their own PRs.

In response to this:

/cc @bpradipt

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: bpradipt, mkumatag
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [mkumatag]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@mkumatag: Updated the plugins configmap in namespace prow at cluster default using the following files:

key plugins.yaml using file config/prow/plugins.yaml

In response to this:

Changes made:

Remove invalidcommitmsg plugin for ocp-power-automation repo
Club the rule for entire ocp-power-automation org

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


Is it possible to create more than one Sensey instance in an activity listening on different views

It seems like only one instance of Sensey can be created per activity (or context to be precise). Is there any way to create and attach different Sensey listeners to more than one view in a single activity?

@c0state the reason Sensey uses a singleton , is because it access sensor data which are specifically hardware that you have access to. The data stream you would usually get would also be one, as the hardware is going to be one. Hence even if you had multiple listeners you would still be accessing the same data stream. In this scenario all your listeners would still fire up even if you used multiple Sensey instances.
As of current state, Sensey cannot get data stream from multiple sensors of the same type in the device, which is when this would be useful, wherein Sensey could access 2 different sensors and respond on each as the data changes.
Let me know what are your thoughts on this if this makes sense to you.

That makes sense, thanks for the detailed information.


Celo:

Describe your issue here.
Environment

Elixir Version: 1.9.0

Erlang Version: 22

BlockScout Version: 330e57bb88472a655cb1b67d70131d8ecef56e49

User Agent: Mozilla/5.0 (Linux; Android 10; SM-A716V) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/85.0.4183.127 Mobile Safari/537.36

Steps to reproduce
Tell us how to reproduce this issue. If possible, push up a branch to your fork with a regression test we can run to reproduce locally.
Expected Behaviour
Tell us what should happen.
Actual Behaviour
Tell us what happens instead.

Sent celo to zrx address. Can anyone reverse this transaction or send it back to where it came from.
Zrx wallet - 0x5FFA0139E782aee0EeA9e2D8d0a591628cd4dFF7
Celo wallet-
0x7492Cf837E5DD398e05BF438c3EcE18D642034D6

Sent celo to zrx address. Can anyone reverse this transaction or send it back to where it came from.
Zrx wallet - 0x5FFA0139E782aee0EeA9e2D8d0a591628cd4dFF7
Celo wallet-
0x7492Cf837E5DD398e05BF438c3EcE18D642034D6


Document excluded users functionality

It is possible to exclude users from this baseline policy, for example when you want to have a "fallback" or "Disaster recovery account" to use in adverse situations. This account can have a very strong password, stored in  vault.
Request:

Document this functionality on this page

Currently the exclusion works, you can select an account. But after saving you can no longer see which accounts are excluded.
Request:

Document how you can view which accounts are excluded (It looks like it's impossible to view this, so might be a feature request actually).

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

ID: 53ea4e70-2931-8691-e9cc-13c72b514d9c
Version Independent ID: c1aa4a8f-16b2-79ac-edb1-8a87f51a3f7b
Content: Conditional Access baseline policies - Azure Active Directory
Content Source: articles/active-directory/conditional-access/concept-baseline-protection.md
Service: active-directory
Sub-service: conditional-access
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

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

This is not possible. Customization is not possible with baselines or security defaults. If you want to customize policies you must use Conditional Access.
https://docs.microsoft.com/azure/active-directory/fundamentals/whats-new#replacement-of-baseline-policies-with-security-defaults
https://docs.microsoft.com/azure/active-directory/fundamentals/concept-fundamentals-mfa-get-started

#please-close

Thanks for the info, good to know baselines are being replaced by defaults. However, even today I can customize the baselines to exclude certain users or groups, so at least in our tenants this was/is possible. I hope the same will be possible for security defaults. Not every tenant has P1 / conditional access.

@valentijnscholten no, security defaults do not have the ability to be customized or excluded from users. Exclusions were removed in July from baseline policies unless you already had them configured.

There really should be a way to exclude break glass accounts.  Microsoft has documented that a best practice is to use break glass accounts.  But with the MFA Baseline policy, we can't exclude these accounts.  Sounds like the team who decided not to have exclusions aren't abiding by Microsoft's best practice recommendations.


Does getRequests() capture all the 'GET' calls in the network log?

I don't see getRequests() returning certain endpoints that I see in Chrome devtools. Is this function designed to return all the network calls with a get method?

The service will capture any requests (e.g. put / post / get) made using the window's fetch or XMLHttpRequest. If the page under observation caches the reference to those objects during load, any requests made through the cached objects cannot be observed.
Can you provide additional information about the requests being made that are not being captured?

Additionally, if the requests are due to native browser handling (e.g. loading the URL found in an <image>'s src attribute), those will not be captured either.


TactileButton not working correctly with pulldown

Button currently only works with InputPull_Up (or physical pull up) and not with pulling down.

Fixed with last commits


security_monkey is support China AWS of BJS ??

security_monkey is support China AWS of BJS ??

Security Monkey will try to slurp from all regions.
I don't have access to cn-north-1 or to the US Govcloud regions, so I have not tested it on them.
Older watchers ignore exceptions from "trouble regions":
https://github.com/Netflix/security_monkey/search?utf8=✓&q=trouble+regions&type=
Newer watchers should record the exception in the database table and move on.
So, I can't be sure if the China regions will work, but I'd love to find out.

@ @I have installed a security monkey, but the web UI interface is anonymous landing, and the error message is the API call failed 503 gateway error, I very much hope that we can solve this problem, thank you very much for your reply. BTW，arn:aws-cn:(AWS account): role / SecurityMonkeyInstanceProfile, https: // (AWS account) .signin.amazonaws.cn / console, but abroad is a completely different ARN @ @

@monkeysecurity @ @I have installed a security monkey, but the web UI interface is anonymous landing, and the error message is the API call failed 503 gateway error, I very much hope that we can solve this problem, thank you very much for your reply. BTW，arn:aws-cn:(AWS account): role / SecurityMonkeyInstanceProfile, https: // (AWS account) .signin.amazonaws.cn / console, but abroad is a completely different ARN @ @

Oh the ARNs contain aws-cn instead of just aws.  That's going to break a few things like the regex in the ARN class.
Your 503 error:
That could happen if there are no healthy instances behind the ELB, OR if gunicorn is not running.
sudo supervisorctl status

I presume it will say that securitymonkey is not running.
You could try manually starting the python server:
monkey run_api_server

And see if it starts up or errors out.

@monkeysecurity                  Error 503 gateway has been resolved, but the new problem again, I am in the security monkey web 'interface Create Account  Add AWS account, China AWS is no Canonical ID meaning, only one account ID, add AWS account is not enough to save, the problem is quite trouble, ha ha,
2029-04-27 04:13:26 UTC LOG: could not receive data from client: Connection reset by peer
2022-04-27 04:13:26 UTC LOG: unexpected EOF on client connection with an open transaction

Canonical ID explained here:
http://docs.aws.amazon.com/general/latest/gr/acct-identifiers.html
It's the replacement for the "s3 name" since AWS is deprecating the s3 name.
I believe you can put a dummy value in for the canonical ID and then run:
monkey fetch_aws_canonical_ids

You can also add an account from the CLI:
https://github.com/Netflix/security_monkey/blob/develop/docs/quickstart.md#add-your-awsgcp-accounts
usage: manage.py add_account_aws [-h] -n NAME [--thirdparty] [--active]
                             [--notes NOTES] --id IDENTIFIER
                             [--update-existing]
                             [--canonical_id CANONICAL_ID]
                             [--s3_name S3_NAME] [--role_name ROLE_NAME]


fix: Adjust dropdown width only on resize

Description
Alternative fix to https://github.com/cloudscape-design/components/pull/1669, without caching anything.
Ticket: AWSUI-22613
Note: dropdowns do not expand beyond the trigger width when virtual scroll is enabled. This is a known issue: AWSUI-22357
How has this been tested?

Successful dry-run build to same failing package as in https://github.com/cloudscape-design/components/pull/1669. The build time is similar (11m 47s then, 12m 35s now)
Manually tested against existing page dropdown/width (which also has integration tests)

Review checklist
The following items are to be evaluated by the author(s) and the reviewer(s).
Correctness

Changes include appropriate documentation updates.
Changes are backward-compatible if not indicated, see CONTRIBUTING.md.
Changes do not include unsupported browser features, see CONTRIBUTING.md.
Changes were manually tested for accessibility, see accessibility guidelines.

Security

If the code handles URLs: all URLs are validated through the checkSafeUrl function.

Testing

Changes are covered with new/existing unit tests?
Changes are covered with new/existing integration tests?

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

The correct fix would be to fix this condition in the first place: https://github.com/cloudscape-design/components/blob/0c5e691a1bed9e316c38a4dc446dc9c7f92710b3/src/internal/components/dropdown/index.tsx#L107
The current fix is a patch that may explode in any moment, because there are two connected pieces of logic live in very different files and lifecycles


[soundcloud repost]

Soundcloud Repost
Hello @kranack !
It's is possible to add the option to follow Reposts from users with Soundcloud Bridge ?
Thanks for your work !!

Ping @Roliga as second maintainer.

Hi!
https://github.com/soundcloud/api/issues/2
Nope. The soundcloud api does not support reposts. So either rebuild the bridge from scratch (by crawling their website) or wait until they may introduce the feature. I don't know if soundcloud moved to github at the end of 2020 or so, but the text says "Issue found of: September 27th, 2016", so I wouldn't hold my breath for this.
I would close this isse as I dont think it will be fixed.


Limit to the preemptive optimistic caching behaviour

Looking good, promising project.
Wondered if you'd think it was worth including a capacity to either have a counter or duration for 'idle' timeout.
Use Case
I'd like to cache bearer token oauth2 introspections, and the idea of pre-warming that and saving myself 100ms latency hit on my backend gin middleware sounds appealing - but if I couldn't set how long a cache value is kept warm for I'd just eventually cause a spiralling amount of calls to my oauth2 provider.
Currently I'm just using Redis directly to cache these, but obviously they expire after x seconds (60s for me currently) and next hit after that incurs a cold start penalty to populate the cache before it's then kept for the next 60s.
Would be good if there's only the first penalty of the first login of the day and then set an 8 hour window to keep re-validating that session permission data every x seconds, but after that y hour window elapses the cache stops trying to pre-warm the data.
Just thinking out loud of a feature I don't currently have but would like, currently the only other option I have is extending the ttl value and serving more stale data than I'd like (for when permissions claims on the token change).
Conversely could just take an int value as a multiple, so it will refresh z times in the background multiplied by the expiry time subtracting the revalidation time.

hello @danielloader, it's  indeed an interesting usecase, which would require a cronjob/scheduler to solve. In my opinion, I think this is better handled outside of this package, since you anyway can customize the updater, or even the store.  So you have the hooks available at almost all stages.
p.s: I assume you already found a solution since you closed the issue already.

Yeah I realised the same as you did this is best handled via some hooks.
Thanks for replying


Rails 6 - Exception: uninitialized constant RailsWarden::Authentication

As mentioned I get an Uninitialized Constant error with rails 6 when I try to include the concern.

The latest release on RubyGems is v0.6.0, released in Feb 2018, whereas this module wasn't added until March 2018, hence the errors.
Until a new release is made, you can use the following in your Gemfile:
gem 'rails_warden', git: 'https://github.com/wardencommunity/rails_warden.git'

@kaelumania Did this fix you? I am getting the same error, but already running from git.

I can't see your code, but the most obvious question is whether you're using classic or zeitwerk mode, and loading the constant in an initializer but referencing it elsewhere. Rails 6+ will unset constants that were set during initialization that would normally get autoloaded, but Zeitwerk will do a better job of this.

Oof, Sorry I don’t remember how I fixed but am long past it. I’d say this can be closed


Improved View

table image view improved a bit.

Thanks for this - much appreciated. Unfortunately there are merge conflicts, are you able to rebase your changes?

Your welcome Colin.. I have rebased my changes... Let me know if anything else is required.