How to prevent msw from catching some URLs: improve docs?

Hi,
I looked multiple times at the addon docs but also the main docs and it was not obvious at all from the initialize() function I would be able to let pass some specific URLs thanks to passthrough():
initialize({
  onUnhandledRequest: (request, print) => {
    if (request.url.pathname.startsWith('/node_modules/')) {
      request.passthrough();
    } else {
      // For all others, show a warning
      print.warning();
    }
  },
});

It did not come to mind since https://mswjs.io/docs/api/request/passthrough talks about request, but in the examples https://msw-sb.vercel.app/?path=/story/guides-getting-started--page#configuring-msw it only talks about method and url.
It's maybe not an issue, but I hope it can save some time to others :D
Thank you,

Hey @sneko thank you so much for opening this issue! I'll discuss with @kettanaito about it and I'll see if we can improve the documentation. This sounds like a very interesting use case!

Hey! Thanks for proposing this.
I would like to learn more about what you're trying to do. The code snippet you shared above is not going to work and request.passthrough() is going to do nothing unless it's returned from a response resolver.
I suspect multiple things are mixed here. I will do my best guessing what they are.
How to suppress unhandled request warnings from some resources?
Use the onUnhandledRequest option.
initialize({
  onUnhandledRequest(request, print) {
    if (request.url.pathname === '/some-resource') {
      return
    }

    print.warning()
  }
})

Note that MSW won't do anything by default if you provide a function as the value for this option. If you wish to ignore the unhandled requests' warnings, just do an early return in that function and you're all set.
How to bypass a request only in some cases?
So you have a mock for GET /resource but you actually want it to reach production only in some cases (e.g. if there's a certain URL query parameter). You use req.passthrough() for that:
rest.get('/resource', (req, res, ctx) => {
  if (req.url.searchParams.has('someParam')) {
    return req.passthrough()
  }

  // Otherwise, respond with a mock.
  return res(ctx.text('Hello world!'))
})

I think none of these things belong to the add-on's documentation because this is not add-on-specific knowledge. You can find these APIs in the MSW documentation and I'm currently working hard on improving the docs experience.

It's been some time. I think I just wanted to clarify that in your documentation you deconstructed the input object and at that time I thought it would be useful to see all the possibilities of the parameters request and print.
You can close this if does not make sense 🚀
Thanks for answering 👍

@kettanaito would it be useful from the addon's perspective to provide a default initial handler that does something like:
rest.get('/node_modules/*', (req) => {
  return req.passthrough()
})

To help with possible HMR clash issues and also to improve performance as a positive side effect?

@sneko, no worries! I am thankful for your input.
@yannbf, I think that wouldn't bring many benefits because you don't typically serve node modules. HMR requests are often those to a running WebSocket server that has nothing related to node_modules so the handler above wouldn't catch it anyway.
I would love to ship some defaults not to spam people with warnings on requests that are implementation details of the framework they are using (such as HMR) but the issue is that those requests are precisely that—implementation details. They may change without any notice, and it would be virtually impossible to keep up with how every framework structures their features when it comes to HTTP requests that implement them. But I'm open to suggestions, still.


no disable bit on force-close

lnd doesn't send lnd when making force-close
from the issue in #1664
as i understand it, in 1664 it was the lnd side that force closed it.
regardless my lnd didn't send a disabled message, only saw one from the clighting side in gossip. checked two implementations listening to gossip, none of them recorded my side, only the opposite side.
on a properly disabled channel, both sides should signal disable, right? not just the one that first detected it?
lnd version 0.4.2-beta commit=c48ecb85f69949dc25ce4cba730213d64c20297e
(the lnd involved in the closure, the channel_update below is from an irrelevant version)
 SigBytes: ([]uint8) (len=70 cap=70) {
  00000000  30 44 02 20 55 98 74 3a  94 35 21 c8 b1 21 23 72  |0D. U.t:.5!..!#r|
  00000010  33 5f e5 60 f8 17 26 ca  e0 12 a2 47 d4 92 e3 0b  |3_.`..&....G....|
  00000020  84 a8 c0 85 02 20 1e 00  67 20 ae 9c 5b f1 a5 f6  |..... ..g ..[...|
  00000030  aa 80 51 1c 20 ce 7d 2b  7d 44 3e 7a a6 41 a4 62  |..Q. .}+}D>z.A.b|
  00000040  8c e3 26 d5 7b 15                                 |..&.{.|
 },
 sig: (*btcec.Signature)(<nil>),
 ChannelID: (uint64)<PHONE_NUMBER>39492353,
 LastUpdate: (time.Time) 2018-07-31 22:28:59 +0000 UTC,
 Flags: (lnwire.ChanUpdateFlag) 2,
 TimeLockDelta: (uint16) 14,
 MinHTLC: (lnwire.MilliSatoshi) 1000 mSAT,
 FeeBaseMSat: (lnwire.MilliSatoshi) 1000 mSAT,
 FeeProportionalMillionths: (lnwire.MilliSatoshi) 10 mSAT,
 Node: (*channeldb.LightningNode)(<nil>),
 db: (*channeldb.DB)(<nil>)
})```

as i understand it, in 1664 it was the lnd side that force closed it.

Nope, c-lightning force closed.

on a properly disabled channel, both sides should signal disable, right? not just the one that first detected it?

Nope, only one side will send it out.

Flags: (lnwire.ChanUpdateFlag) 2,

Looks disabled to me...

Closing as we disable when we force close, and would only learn of the remote side force closing if it hit the chain, in which case, the entire network would also detect.


Nvidia toggle is confusing

ujust toggle-nvk is confusing, you have to know what nvk is. And then on top of that it just rebases without asking, which might be a bit disconcerting. What do we think about renaming it toggle-nvidia and then showing the current state and what it wants to do like we do for devmode:

❯ ujust devmode
Developer mode is currently enabled
Enable or Disable developer mode

Enable
Disable

Something like:"

❯ ujust toggle-nvidia
Current nvidia driver: NVK (Open Source)
Enable or Disable Nvidia driver

Enable
Disable

For the status we could say:
NVK (Open Source/MESA)
Nvidia (From Nvidia Inc)
That way it's more clear what image I am on, and which image I am switching to?

im fine with renaming it, you can do that right away.
i can do the logic next week since i am busy this friday+weekend (unless anyone else wants to change it, as the logic has been used elsewhere in ujust)


Possibility to link directly to failing test line

Currently the test output will link to test cases, but I'd like to be able to link directly to the offending line. By running my tests as a post-build step I can currently get failing test lines to appear in my Errors list and link from there to the line.
I've never used the Test Explorer much before, but is this something that could be possible or not?

Hi @alarobric, thanks for your interest.

the usage of the TestExplorer is described here: https://msdn.microsoft.com/library/hh270865.aspx
and Find the Test Case Location in Source

Hope this helps.
Regards,

Hi @alarobric and thanks for your interest in the project.
Currently the adapter does not link directly to the error, it just lists the location of the error. Unfortunately, we cannot interact with the error list but there is a similar interface where we list errors in the test explorer window.
We have looked at other adapters and it seems that it might be possible to emulate your request via the stack-trace feature of the adapter where the first error can be listed in a hyper-link-able stack-trace. Will update this issue as we progress further in this regard.

@alarobric maybe you can try with #144 and give us feedback.


41871

Field
Value

Redmine Ticket Number
41871

App-Dev Road Map #
n/a

Start Date
n/a

Due Date
n/a

App-Dev Priority
UP-Disqueued

Development Story Points
n/a

Peer Review Story Points
n/a

Ticket Author
Alix Pereira

#this pull request is been merged and deployed into the master


Positioning the viewport at start of a system

@npiegdon In discussion #377 it was mentioned the posibility of positioning the vierport at start of a system using system number.
Currently, all methods related to the viewport uses physical units (pixels), e.g. to move the viewport:
Interactor::new_viewport(Pixels x, Pixels y, bool fForceRedraw=true)

When it was decided to use pixels instead of logical units, the point was that the viewport management was mainly related to scrolling.
Deciding the shift amount for an scroll operation once the window width/heigh is known, is just deciding how many scroll steps are convenient for a smooth and fast enough scrolling of a whole window, and dividing the window width/heigh by the chosen number of steps. As applications normally use physical units (pixels) for managing the window (e.g. mouse click points, window size) it was considered simpler and more convenient to manage the viewport also in physical units (pixels). Thus, in current API all methods related to the viewports use pixels. Also, methods related to this, such as determining the total width/heigh of the View for properly positioning scrollbars and their sizes, also provides measurements in pixels, e.g.:
Interactor::get_view_size(Pixels* xWidth, Pixels* yHeight)

After studying the possibility of using logical coordinates as an option, a simple way would be by converting logical units to pixels before invoking viewport related methods, e.g. by using:
void Interactor::model_point_to_device(double* x, double* y, int iPage)
UPoint Interactor::screen_point_to_model_point(Pixels x, Pixels y)

For instance, to position the viewport at start of system iSystem:
GraphicModel* pGM = pInteractor->get_graphic_model();
GmoBoxSystem* pBoxSystem = pGM->get_system_box(iSystem);    //iSystem=0..n-1
LUnits yPos = pBoxSystem->get_origin().y;   //LUnits are relative to page origin
double x(0.0);      //x = 0, place viewport at paper left border
double y(yPos);     //place the viewport at system top. Perhpas some margin should be added
int iPage = 0;      //FreeFlowView only has one page, iPage=0
pInteractor->model_point_to_device(&x, &y, iPage);
pInteractor->new_viewport(Pixels(x), Pixels(y));

The gain in creating a new method is minimal, just one additional invokation to lomse, but as the API is oriented to pixes surely some other methods that currently use pixels will have to be duplicated. Therefore, it seems more convenient not to duplicate methods and keep the API in pixels, since the cost for those who want to use logical units is only one additional call to perform the units conversion.
Let me know if this is acceptable and solves your needs.

Yeah, this looks good.  I'm happy using either coordinate system, whichever is most convenient and makes the most sense for the library.
Regarding //iSystem=0..n-1, how would I find out the value of n?

Regarding //iSystem=0..n-1, how would I find out the value of n?

I have added a new method for this. Please see PR #381

I am closing this issue as it appears to be resolved. If you believe things are still unresolved please let me know and I can reopen if needed.


Add support for multiple media types

repo.read() function should have option for choosing the format of the receiving data. (base64)
For example, if requested target is image file and if you are going to show it on browser then it could be much easier to download content as base64 image so the image url could be easily generated when there is no need to convert it to base64 anymore..
and for receiving / sending commits, pulls, etc.. it would be nice to have an option for choosing diff / patch media type.
https://developer.github.com/v3/media/
https://developer.github.com/changes/2012-12-10-Diff-and-patch-media-types/

Marking as a mid-priority item.

#219 should fix this, just waiting on tests to merge.


Migrate 🇺🇸 5 - Fix #159

@GabrielRF this is made of 2 parts only the second is a long string for now the best way would be to add the line break in part2 (also I'm working on the fix for being able to have 3 parts in concept cards) thanks for the PR


get all user groups from keycloak

each time when keycloak user token is verified all user groups are stored in DB (remov old one and store the current)
during  user authorization (isCredentialActive) the stored user name and all user groups are sent to ranger.

Codecov Report

Merging #11 into master will decrease coverage by 0.36%.
The diff coverage is 89.13%.

@@            Coverage Diff             @@
##           master      #11      +/-   ##
==========================================
- Coverage   92.03%   91.66%   -0.37%
==========================================
  Files          17       17
  Lines         251      288      +37
  Branches       14       16       +2
==========================================
+ Hits          231      264      +33
- Misses         20       24       +4

Impacted Files
Coverage Δ

...c/main/scala/com/ing/wbaa/airlock/sts/Server.scala
0% <0%> (ø)
:arrow_up:

...in/scala/com/ing/wbaa/airlock/sts/api/STSApi.scala
100% <100%> (ø)
:arrow_up:

...n/scala/com/ing/wbaa/airlock/sts/api/UserApi.scala
100% <100%> (ø)
:arrow_up:

...a/airlock/sts/keycloak/KeycloakTokenVerifier.scala
92.85% <100%> (+3.96%)
:arrow_up:

.../wbaa/airlock/sts/service/UserTokenDbService.scala
83.33% <100%> (+0.47%)
:arrow_up:

...irlock/sts/service/db/dao/STSUserAndGroupDAO.scala
93.25% <88.23%> (ø)

Continue to review full report at Codecov.

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

Codecov Report

Merging #11 into master will decrease coverage by 0.36%.
The diff coverage is 89.13%.

@@            Coverage Diff             @@
##           master      #11      +/-   ##
==========================================
- Coverage   92.03%   91.66%   -0.37%
==========================================
  Files          17       17
  Lines         251      288      +37
  Branches       14       16       +2
==========================================
+ Hits          231      264      +33
- Misses         20       24       +4

Impacted Files
Coverage Δ

...c/main/scala/com/ing/wbaa/airlock/sts/Server.scala
0% <0%> (ø)
:arrow_up:

...in/scala/com/ing/wbaa/airlock/sts/api/STSApi.scala
100% <100%> (ø)
:arrow_up:

...n/scala/com/ing/wbaa/airlock/sts/api/UserApi.scala
100% <100%> (ø)
:arrow_up:

...a/airlock/sts/keycloak/KeycloakTokenVerifier.scala
92.85% <100%> (+3.96%)
:arrow_up:

.../wbaa/airlock/sts/service/UserTokenDbService.scala
83.33% <100%> (+0.47%)
:arrow_up:

...irlock/sts/service/db/dao/STSUserAndGroupDAO.scala
93.25% <88.23%> (ø)

Continue to review full report at Codecov.

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


docs: LABEL <value> is not optional

Refer: http://docs.docker.com/engine/userguide/labels-custom-metadata/  ==> "Add labels to images" quotes that the "label value can be optional", which is incorrect. This may need to be fixed in the docs ?
FROM scratch
LABEL com.app.version.is-beta=""  // works
LABEL com.app.version.is-beta=""  // fails

$ docker build -t appengine .
Sending build context to Docker daemon 46.08 kB
Error response from daemon: LABEL must have two arguments

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

$ docker version
Client:
 Version:      1.9.0
 API version:  1.21
 Go version:   go1.4.2
 Git commit:   76d6bc9
 Built:        Tue Nov  3 17:43:42 UTC 2015
 OS/Arch:      linux/amd64

Server:
 Version:      1.9.0
 API version:  1.21
 Go version:   go1.4.2
 Git commit:   76d6bc9
 Built:        Tue Nov  3 17:43:42 UTC 2015
 OS/Arch:      linux/amd64

$ docker info
Containers: 2
Images: 278
Server Version: 1.9.0
Storage Driver: aufs
 Root Dir: /var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 282
 Dirperm1 Supported: false
Execution Driver: native-0.2
Logging Driver: json-file
Kernel Version: 3.13.0-68-generic
Operating System: Ubuntu 14.04.3 LTS
CPUs: 4
Total Memory: 7.689 GiB
Name: xxxxx
ID: Q7CC:FNYP:4OWW:Q4FW:HHVL:KORD:AS6L:6AGN:6I3D:LRZG:NBWO:ZDXC
WARNING: No swap limit support

$ uname -a
Linux xxxxxx 3.13.0-68-generic #111-Ubuntu SMP Fri Nov 6 18:17:06 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux

related to https://github.com/docker/docker/pull/17344

@thaJeztah @runcom was this fixed? Can we close it?

Looks like this was addressed in https://github.com/docker/docker/pull/18707, so closing


--reserve-memory puts service in bad state without errors when not enough memory

Output of docker version:
Client:
 Version:      1.12.0-dev
 API version:  1.25
 Go version:   go1.6.3
 Git commit:   80799a7
 Built:        Tue Jul 26 13:59:26 2016
 OS/Arch:      linux/amd64

Server:
 Version:      1.12.0-dev
 API version:  1.25
 Go version:   go1.6.3
 Git commit:   80799a7
 Built:        Tue Jul 26 13:59:26 2016
 OS/Arch:      linux/amd64

Output of docker info:
Containers: 0
 Running: 0
 Paused: 0
 Stopped: 0
Images: 0
Server Version: 1.12.0-dev
Storage Driver: aufs
 Root Dir: /var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 0
 Dirperm1 Supported: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: overlay null host bridge
Swarm: active
 NodeID: 6rp5wpq558chtj2p582og8d4z
 Is Manager: true
 Managers: 1
 Nodes: 1
 Node Address: <IP_ADDRESS>
Runtimes: runc
Default Runtime: runc
Security Options: seccomp
Kernel Version: 4.4.15-moby
Operating System: Debian GNU/Linux 8 (jessie) (containerized)
OSType: linux
Architecture: x86_64
CPUs: 4
Total Memory: 5.82 GiB
Name: 74fbc52a5887
ID: 7SRD:A64P:DA2U:IQI7:HABR:I7CM:YCTX:NTY5:OISM:IEGK:ACVU:R2CN
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): true
 File Descriptors: 29
 Goroutines: 126
 System Time: 2016-07-26T14:06:55.330252696Z
 EventsListeners: 0
Registry: https://index.docker.io/v1/
WARNING: bridge-nf-call-iptables is disabled
WARNING: bridge-nf-call-ip6tables is disabled
Insecure Registries:
 <IP_ADDRESS>/8

Additional environment details (AWS, VirtualBox, physical, etc.):
Steps to reproduce the issue:

docker swarm init
docker service create --replicas 1 --name nginx --reserve-memory 500G nginx

Describe the results you received:
Logs from daemon
DEBU[0156] Calling POST /v1.25/services/create
DEBU[0156] form data: {"EndpointSpec":{},"Mode":{"Replicated":{}},"Name":"nginx","TaskTemplate":{"ContainerSpec":{"Image":"nginx"},"Placement":{},"Resources":{"Limits":{},"Reservations":{"MemoryBytes":1.2884901888e+10}},"RestartPolicy":{}},"UpdateConfig":{"FailureAction":"pause","Parallelism":1}}
DEBU[0156] Service 9wub4yt2y1v1deqzgt7fkm1p6 was scaled up from 0 to 1 instances
DEBU[0156] No suitable node available for task           task.id=e2x9gk4dyu06zhm6dfianjuh1

$ docker service ls
ID            NAME   REPLICAS  IMAGE  COMMAND
9wub4yt2y1v1  nginx  0/1       nginx

$ docker service tasks nginx
ID                         NAME     IMAGE  NODE          DESIRED STATE  CURRENT STATE            ERROR
e2x9gk4dyu06zhm6dfianjuh1  nginx.1  nginx  74fbc52a5887  Running        Allocated 2 minutes ago

Describe the results you expected:
I expect the CURRENT STATE to not be Running since the task is not running. There should be an error that the task cannot be launched because there is no suitable node. Also, it appears the scheduler is not rescheduling this task as I have yet to see anymore logs from the tick/scheduleTask event loop.
Obviously 500G isn't a real value that should be set, but it is possible when people start to reserve memory and nodes run out of reserved memory from a bunch of 8G or w/e.
(not 100% sure if this should be here or swarmkit so lemme know if i should move the ticket over)
cc @stevvooe

@jhorwit2 This is probably a swarmkit issue, but we can discuss here.
Tasks, such as those that require a massive overcommit, can become "unschedulable". These should probably show as PENDING in the UI. I would say the UX for these "stuck" tasks isn't great.
The nice thing is, if the cluster changes such that they do become schedulable, they will be dispatched. We just need to come up with a way of showing this better.
Here are the task states for reference.

@stevvooe PENDING is better than Running. I think it would be super useful to include a reason why it's pending. Some things that pop into my head could be ports are taken, not enough memory, not enough CPU?
While we are on this subject, imagine the scenario happens where a cluster is full because of reserved memory/cpu, so there are constantly pending tasks. I don't see an easy way to view all the tasks & their states all together. You currently have to look at each service's tasks. I also don't see anything in node inspect that shows reserved resources. I'm focusing on reserved because that's easier to fill up without noticing, because you potentially aren't using all the reserved resources so monitoring the host will make it seem like you aren't using everything.

@jhorwit2 We should probably set a message and display it when a task cannot be scheduled.
While there is not a top-level docker tasks command, you can inspect docker node tasks.
I don't think we have any kind of exportable node resource state. We are discussing the addition of more sophisticated resource allocation, so this may be a natural thing to add.

@stevvooe Ah whoops. Never noticed node tasks.
Is there a ticket for that? or offline?

@jhorwit2 https://github.com/docker/swarmkit/issues/594

Moving this to 1.13, and changed to "enhancement"; improve the UX for obtaining scheduling errors/problems

I would like to add another POV to this issue description. I'm using Docker Swarm Mode (1.12.2) via API and POST /services/create returns no indication that scheduling failed due to low resources (aka a full cluster). Any indication of a problem would be helpful in this case.
And for API usage it would be great to get errors like that in Swarm events (#23827) as well!

@sebgie POST /services/create is going to have no idea whether or not the service is actually schedulable. As long as it is valid, it is going to accept it and try to schedule it when resources are available.
The way to detect this condition is to look for tasks that are stuck in the ALLOCATED or PENDING state.
We are making part of this change in https://github.com/docker/swarmkit/issues/1668#issuecomment-255192396. It might also be a good idea for us to write an message in the TaskStatus for tasks that cannot be scheduled due to resource exhaustion.
@mstanleyjones We might need to have better documentation for this.

We recently merged a PR in swarmkit that annotates unschedulable tasks with the reason they cannot be scheduled: https://github.com/docker/swarmkit/pull/1610

Doc improvements: https://github.com/docker/docker.github.io/pull/297

@stevvooe thank you for the explanation!
Just to make sure I understand that correctly. As service/create doesn't provide a status for successfully scheduled containers I need to repeatedly query the task status and if it is stuck on Allocated/Pending for a certain amount of time I can assume that something went wrong and trigger the appropriate escalation steps (alert ops, start a new node, ...)?
What would you suggest to do if I would like to dynamically scale down other services when the swarm is full?
I'm thinking about the following scenario: I run 20 services with 5 replicas. When starting the 21 service it tells me that it can't be allocated. One thing that comes to mind would be to scale some of the services back to 4 replicas, start the new service and create a new EC2 instance. When the new machine becomes available update the services to return to 5 replicas again.
Am I going to receive events for such conditions once Swarm service events are implemented?
Thank you!

What would you suggest to do if I would like to dynamically scale down other services when the swarm is full?

For a reservation based system, I would recommend inspecting reservation status on the nodes. I don't think we expose this (cc @aaronlehmann), so there may be a gap there we need to address. Right now, we allow the reservation of CPU and memory, so accounting client-side isn't wildly challenging, albeit very annoying. As a start, you can get at total task resource usage with something like this:
docker service ps -f desired-state=running  admiring_knuth | tail +2 |cut -d ' ' -f 1 | xargs docker inspect | jq '.[].Spec.Resources.Reservations'

Then you can get at node resource availability:
$ docker node ls -q | xargs docker node inspect | jq '.[].Description.Resources'
{
  "NanoCPUs":<PHONE_NUMBER>,
  "MemoryBytes":<PHONE_NUMBER>
}

Utilization would be represented by dividing the total task reservation by the total node reported resources.
Without using reservations, which I would actually recommend, I would monitor load average, CPU, IO and network bandwidth against application performance metrics (say, avg response time) and try to understand what linear scaling is necessary to maximize application performance.
The methodology chosen is dependent on your requirements.

Am I going to receive events for such conditions once Swarm service events are implemented?

Yes. This is definitely something we need to add. Our discussions are actually centering around how granular these events should be in practice and what consistency guarantees we can provide. For now, you must poll (sorry :( ).
I hope this helps!

Ran into this today. Perhaps a really simple adjustment that would help would be showing the error message in docker service ps. When a task is "Failed" it shows the error message, like task: non-zero exit (1), but for pending it shows nothing:
ut5oyw95kw7d  foo-test2-bar.1      registry:443/foo/bar     Running        Pending 18 minutes ago

Just adding no suitable node (insufficient resources on 3 nodes) (visible in docker inspect ut5oyw95kw7d) would help immensely. The reason why it doesn't show is because the message is in the "Message" field rather than the "Err" field. The latter seems more appropriate anyway.
File cli/command/task/print.go

is there some direct way we can know whether the resource crunch is caused due to CPU or memory?


Docs: Add link to 2D lights and shadows from CanvasModulate

Closes https://github.com/godotengine/godot-docs/issues/8652.
2D lights and shadows is indeed the best (and only) tutorial for use of CanvasModulate.

Nice!

Thanks!


Add Request

Name: Will Sinatra
IRC Nick: wsinatra
Gitlab: https://gitlab.com/Durrendal
Linkedin: https://www.linkedin.com/in/will-sinatra/
CL Projects: https://gitlab.com/Durrendal/Paraexec
Alpine Linux Package Maintainer: SBCL, Clisp, Fennel, Clojure

Thank you for your add request!
Your entry is live!
I hope this is to your satisfaction.
PS: The Common Lispers list does not (currently) track IRC nicks per se, but since you mentioned it, I'm assuming that's your desired nick, so I assigned you that nick. I would normally have auto-assigned you "durrendal" since that's the one you seem to be using most consistently.

Thanks Jean-Philippe!
I do use Durrendal a lot, but all of my serious FOSS work I do under my own
name. Either way is fine, but I prefer wsinatra so I appreciate it!
On Tue, Feb 18, 2020, 1:50 PM Jean-Philippe Paradis <
<EMAIL_ADDRESS>wrote:

Thank you for your add request!
Your entry is live!
https://common-lispers.hexstreamsoft.com/#Will_Sinatra
I hope this is to your satisfaction.
PS: The Common Lispers list does not (currently) track IRC nicks per se,
but since you mentioned it, I'm assuming that's your desired nick, so I
assigned you that nick. I would normally have auto-assigned you "durrendal"
since that's the one you seem to be using most consistently.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Hexstream/common-lispers.hexstreamsoft.com/issues/16?email_source=notifications&email_token=AIK3CKRMLWJPUBEGTP4L7DLRDQUVZA5CNFSM4KXJGU62YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEMDNBYQ#issuecomment-587649250,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIK3CKSKU36PMAIUAVANX4DRDQUVZANCNFSM4KXJGU6Q
.


MBProgressHUD the latest official version is 1.0.0, and you can rely on the inside of the library to upgrade what?

The dependency is too low somehow, should be upgrade to 1.0.0!!!!

Unfortunately this project seems to be dead

升级MBProgressHUD到1.0.0，SDWebImage到4.0.0，使用方法:
pod "MWPhotoBrowser", :podspec =>'https://raw.githubusercontent.com/y500/MWPhotoBrowser/master/MWPhotoBrowser.podspec'
use this


C++ refactoring: datetime64 and timedelta64 support

The following test fails with Numpy 1.3:
def test_ufunc_sum():
    nparray = np.array(
        [np.datetime64("2021-06-03T10:00"), np.datetime64("2021-06-03T11:00")]
    )
    akarray = ak._v2.highlevel.Array(nparray)

    with pytest.raises(np.core._exceptions.UFuncTypeError):
        akarray[1:] + akarray[:-1]

due to an import error:
tests/v2/test_0835-datetime-type.py:12: in <module>
    import numpy.core._exceptions as np_exception
E   ImportError: No module named 'numpy.core._exceptions'

completely_flatten is now a Content method. (Since it only involves one array, turning it into a method lets us do OOP-style small functions instead of if-elif chains, which were only there because I was in a hurry and wanted to avoid writing C++ code.)

    with pytest.raises(np.core._exceptions.UFuncTypeError):
        akarray[1:] + akarray[:-1]

Relying on a private module of another library is not a good idea, for exactly this reason. (It could disappear again in a future version, without warning.) Checking for TypeError (UFuncTypeError's immediately superclass) would work just as well. That's a public API.

I've changed the title of this PR to better reflect what has been updated. I'd also like to merge this as soon as it passes tests to keep PRs granular. (I've made some code cleanliness fixes, and if I'm working on something else in another PR, I might get confused that they're not already done. Thus, I want to start any PRs I do later today on this one.)
Note that I haven't moved any new tests into tests/v2 and I haven't changed the "skip" status of any that are there. The things you were planning to do with this PR aren't done yet. You'll want to open a new PR, probably with this one's old name, to address those things.

Awesome! Thanks, @jpivarski !


it dosent work on RN

I write the same code for RN, but it dosent work, I cant find any state under window__STATE__, how can I  initail the state that I have persisted

Try this:
class Root extends React.Component {
  state = { persistedState: null }

  componenetWillMount() {
    AsyncStorage.getItem('persistedState').then(persistedState => this.setState({ persistedState }))
  }

  render() {
    const { persistedState } = this.state
    return persistedState === null ? <Loader /> : <App persistedState={persistedState} />
  }
}

Show loader when loading state from AsyncStorage. And after load, rehydrate stores with preloaded state and render entire App

The initialState is just for server side render of react-dom. see the example for rn: https://github.com/pinqy520/mobx-persist/tree/master/examples/rn


fixes #480 - dates are always passed to "date" and "datetime-local" e…

…lements using the standard format (YYYY-MM-DD or YYYY-MM-DDTHH:mm:ss), and "datetime" elements are converted into "datetime-local" (datetime is deprecated/obsolete).

Please check if the PR fulfills these requirements

[x] The commit message follows our guidelines
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Fixes #480 - dates are always passed to the HTML5 "date" or "datetime-local" element in the correct format.  "datetime" is also converted into "datetime-local" as "datetime" is now deprecated/obsolete

What is the current behavior? (You can also link to an open issue here)
When a "format" is not provided, the date/datetime field would be sent a number and throw an exception formatting the value

What is the new behavior (if this is a feature change)?
values are always passed to date/datetime-local field in the correct "YYYY-MM-DDTHH:mm:ss" format as described in the docs

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
It should not

Other information:
Refer to #480

Coverage remained the same at 89.916% when pulling db3413fbd2834f795c460d792788a0e0e7099690 on zoul0813:feature/480-date-format into a48403187b0b557832373ac9f8efa4b3bfed97f1 on vue-generators:master.


How to handle several container scenarios?

Is your feature request related to a problem? Please describe.
I have a use case where I need to be able to manage multiple containers each with specific settings. By settings I mean a different object to inject to the last nested object.
Description
To do so It would be very great if we could instantiate containers this way:
let container1 = new Container();
let container2 = new Container();

And then "attach" the settings object (in my use case) to containers like this:
container1.register("SettingsClass", {
  provider: () => settingsObject1
});
container2.register("SettingsClass", {
  provider: () => settingsObject2
});

So when resolving objects the right settingsObject1 will be injected and successfully get
const myService1 = container1.resolve(MyService);
const myService2 = container2.resolve(MyService);

Maybe there is already a way to do it but I could not find it, could you help me?
PS: I'm a noob in typescript :)

You can create a new container instance with container.createChildContainer(), since the DepedencyContainer class is not exported.
Every container instance has it's own Registry but also holds a reference to it's parent and will use it to resolve depedencies only when a given depedency is not found within it's own registry.
Changing your example to the following will probably solve your problem.
import {container} from 'tsyringe';

const container1 = container;
const container2 = container.createChildContainer();

container1.register("SettingsClass", {
  provider: () => settingsObject1
});
container2.register("SettingsClass", {
  provider: () => settingsObject2
});

console.log(container1.resolve("SettingsClass") === container2.resolve("SettingsClass"));

You could also create two child containers and not use the root container at all.

Ok thank you, it's perfect 👌

Going to leave this open until the documentation for this is in the readme

@Xapphire13 I'm in the process of updating the documentation - what exactly needs to be documented here? That you can have multiple containers, or that you can have containers with fallback using createChildContainer()?


Users can see the kudos they have given

Closed by https://github.com/slashkudos/kudos-web/pull/17


refactor loadClass method

extract method ：cacheWrapperClass
extract method ：cacheAdaptiveClass
extract method ：cacheActivateClass
extract method ：saveInExtensionClasses
extract method ：cacheNames
(names != null && names.length > 0)，is always true, remove it
What is the purpose of the change
XXXXX
Brief changelog
XXXXX
Verifying this change
XXXXX
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GITHUB_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[ ] Run mvn clean install -DskipTests=false & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.
[ ] If this contribution is large, please follow the Software Donation Guide.

all settled? can I review again?

@satansk do you have any further review comment?

all settled? can I review again?

im new to github, i dont know where comes this error: all checks have failed, please help me to deal with it.
thankyou

@beiwei30 Nope :)


[FreeBSD] Fix a few issues

This PR fixes a few issues on FreeBSD 11 / 12

creation of installable .tar.gz package
libswiftCore.so: undefined reference to
__atomic_store __atomic_compare_exchange __atomic_load *
crash of produced executables: dlopen() failed on ./main: (NULL)Abort (core dumped)

* Couldn't find a better way.

@swift-ci please test

The FreeBSD bits look good to me but please wait for explicit approval by somebody on the swift team before merging.

Build failed
Swift Test Linux Platform
Git Sha - 8ffd8933b959b1d78e41b4e4fc2acddb88dac1df

@antonmes your code is failing on Linux, PTAL.

@dcci I think it's not related to the PR, something with swift-corelibs-foundation in regression tests. Swift itself passes all the tests. This additions shouldn't affect any Darwin or Linux build instructions.

You probably need to wait until the failure goes away, rebase on top of master and retry. Some minors, otherwise I think this is fine.

@swift-ci please smoke test

@swift-ci please test

Build failed
Swift Test Linux Platform
Git Sha - 8ffd8933b959b1d78e41b4e4fc2acddb88dac1df

Build failed
Swift Test OS X Platform
Git Sha - 8ffd8933b959b1d78e41b4e4fc2acddb88dac1df

If @gparker42 has no further objections then I think this can go in. It's certainly not going to break anyone else.

Looks good. @dcci?

LGTM, thanks!


feat: integrate bolt logging configuration and pipeline builder LogsTo

Closes #103
This makes setting up logging with Bolt a lot easier:
// configure a phase logging sink called "history"
LogsTo(pipelines.BoltLogger[*CheckoutResource]("history")).

The following code wires in a bolt type logger configured by name.
The follow example configuration would cause the pipeline to log to glu.db:
sources:
  bolt:
    history:
      path: "glu.db"

Breaking
Slight adjustment to the builder has lead to even simpler code when composing piplines:
Before:
PromotesTo(func(b pipelines.Builder[*CheckoutResource]) (edges.UpdatablePhase[*CheckoutResource], error) {
    return pipelines.GitPhase(b, glu.Name("production", glu.Label("env", "production")), "checkout")
})

After:
PromotesTo(pipelines.GitPhase[*CheckoutResource](
    glu.Name("production", glu.Label("env", "production")),
    "checkout",
))

I played around with a few ways of shaping it and explicit felt the most consistent (other sources are named after their tech).
I think if we changed the storage in the future, it would be nice to switch over to it explicitly too.
Otherwise, that would be a breaking change.
Is the concern that bolt might seem to niche to some folks / unknown?

Ill merge for now and we can reconsider naming in another PR if needs be


New (Improved) [Fishing] Mod

Crabman works on a new fishing mod, it's very promising, and certainly the future best fishing mod of Minetest !
We don't have many informations to share for the moment, but Crabman still works on his fishing mod...
So, we will add this fishing mod when it will be over.
(maybe youtube testing videos soon)

Work in progress...
https://github.com/crabman77/fishing/commit/ab67ad83b37d05f1ea0febccb9f3dbdac4384c9c

Great work !

Fishing mod finished  :)
https://github.com/crabman77/fishing

Added contest game, demo video  http://youtu.be/8rt8g3F2Gmk

@crabman77
Has I understand, your mod is over. Maybe you can replace our actual [fishing] mod with your [fishing] mod (tweaked and improved) now ?

Before We must find the treasure. https://github.com/crabman77/fishing/blob/master/prizes.lua#L40

Fishing merged, but we must find the treasure stuff before  merge on server.
https://github.com/crabman77/fishing/blob/master/prizes.lua#L40

Now ingame, i closed this issue.
For bugfix, this ticket is opened : https://github.com/MinetestForFun/minetest-minetestforfun-server/issues/169


Enable app-dependent language setting (Android 13)

Avoid duplicates

[ ] This enhancement request has not already been raised before
See below for related requests
[X] Enhancement request is specific for Android only, for general issues / questions that apply to iOS and Android please raise them in CWA-Wishlist
[ ] If you are proposing a new feature, please do so in CWA-Wishlist

Current Implementation
CWA Android is built for API 31 (Android 12) and uses the device's language setting (locale).
Screenshots
Android 13 screenshots with CWA installed showing (as expected) that App Language selection is not available in CWA Android 2.25.0:

Suggested Enhancement
Adapt the CWA Android app to target API 33 (Android 13) and implement Per-app language preferences.
Expected Benefits
More flexible language selection for users in a multi-lingual environment. iOS already has this capability (see https://github.com/corona-warn-app/cwa-app-android/issues/3489#issuecomment-824746939 from @Ein-Tim).
Note that if this is implemented, and there is no change to the Google Exposure Notifications API, then it could mean that messages from the Google API would be output in a different language to the ones output by the CWA app. This might not be a desirable configuration.
Related requests

https://github.com/corona-warn-app/cwa-app-android/issues/2905
https://github.com/corona-warn-app/cwa-app-android/issues/3489
https://github.com/corona-warn-app/cwa-app-android/issues/4980

Dependent on enhancement request https://github.com/corona-warn-app/cwa-app-android/issues/5469

Even Google Maps (from GOOGLE!) can be set to a different language other than the system language, and even on older android versions.
So it is strange to me, why the corona warn app doesn't allow this.
I suppose that the developers follow more some user unfriendly guidelines than usability.
If an app which is only available in a non-English language is installed on a phone that is set to English as system language, it runs with its own non-English language.
So I see not reason why an app which supports multiple languages must "auto-magically" use the system language.
Let the user decide, not an automatic!

There is a YouTube video (5:30 duration) published on Sep 3, 2022 which explains the user story behind the need for in-app language preferences and the steps to implement in Android 13 (Codename: Tiramisu / API 33) and previous Android versions.

Some commits in https://github.com/corona-warn-app/cwa-app-android/tree/feature/13779-Android-13 reference this issue (which does not mean that it will be released though).

Since the pre-requisite of Android 13 support is not planned (see https://github.com/corona-warn-app/cwa-app-android/issues/5469#issuecomment-1371936091), then this enhancement can also not be planned.


ResizingImageProcessor with a referenceSize of .zero causes crash

Check List
Thanks for considering to open an issue. Before you submit your issue, please confirm these boxes are checked.

[x] I have read the wiki page and cheat sheet, but there is no information I need.
[x] I have searched in existing issues, but did not find a same one.
[x] I want to report a problem instead of asking a question. It'd better to use kingfisher tag in Stack Overflow to ask a question.

Issue Description
What
I had run the Kingfisher-Demo, when I tried to follow Processor -> Action -> Processor -> Resizing to resize images, the app crashed.
Finally, I found that .zero was used as referenceSize of  a ResizingImageProcessor processor seems to be the cause.
https://github.com/onevcat/Kingfisher/blob/96c37ef7f8f258eab400f1879391b6bf4e670ba6/Demo/Demo/Kingfisher-Demo/ViewControllers/ProcessorCollectionViewController.swift#L42
Since .zero size makes the context(UIGraphicsGetCurrentContext()) be a nil, which will lead a
"[Kingfisher] Failed to create CG context for blurring image." error and crash.
I wanted to make a PR just to replace that .zero in demo.
but, do you think .zero should be restricted to be used as referenceSize of ResizingImageProcessor?
Thanks!
Reproduce
Install the demo, and follow this:
Processor -> Action -> Processor -> Resizing

Yes, a PR for it is welcome.
I guess we can just fix the example, instead of the actual resizing code. It is an assertionFailure and returning the original image. We want to treat this kind of error as a developer one: the user of the framework should make the promise of passing in a valid size. An assertionFailure will help them to catch this kind of issue earlier and easier.

I see!
Then I will make a PR .


pexpect.pxssh.ExceptionPxssh: password refused

Describe the bug

self.ssh.login(hostname, username, self._ssh_password, port=2222)
raise ExceptionPxssh('password refused')
pexpect.pxssh.ExceptionPxssh: password refused
Setup and configuration
Current version:

My config.toml and environment vars (be sure to redact API keys):

My model and agent (you can see these settings in the UI):

Model:
Agent:

Commands I ran to install and run OpenDevin:

Steps to Reproduce:
1.
2.
3.
Logs, error messages, and screenshots:
Additional Context

Try setting SANDBOX_TYPE=exec in your config.toml or environment. We're working on a longer-term fix

Going to close this as we have a lot of these, but feel free to ping this thread if that doesn't work


velocity function issue

i updated to the latest version of angular2 materialize but when use $ functions it getting the .velocity function issue...
for example....
$('.materialboxed').materialbox();

@chandumunna thanks for reporting this issue.
There's a few existing bugs on this and info on where it comes from (no fix yet, though).
Can you please add your details to the existing bugs and close this one?
Thanks.

html page -->

ts page-->
import { Component, OnInit, AfterViewInit } from '@angular/core';
declare var $:any;
declare var Materialize:any;
import {MaterializeDirective} from "angular2-materialize";
@Component({
selector: 'app-home',
directives:[MaterializeDirective],
templateUrl: 'home.component.html',
styleUrls: ['home.component.css']
})
export class HomeComponent implements AfterViewInit {
constructor() { }
ngAfterViewInit(){
$('.materialboxed').materialbox();
}
}

Need urgency any one help me
thanks in advance...

Sorry, I won't be able to look into it for the next couple of weeks. If you happen to check it yourself and figure it out PR's are welcome. Thanks.
Please close duplicate issues, to help maintain a clear list of issues. Cheers.

Solve with the latest version of angular2-materialize. Notice that the setup process is different and materialize-css needs to be imported separately.
See working example here: https://github.com/InfomediaLtd/angular2-webpack-starter


Server Issue with custom seeds/crops.

Describe the bug
Server doesn't start thanks to custom crops.
To Reproduce
Steps to reproduce the behavior:
Run arclight on 1.20.1 Version, have the mods and config.
Expected behavior
The server to start.
Screenshots / Scripts / Logs
Please add your logs and scripts (if applicable).
Debug and logs, and Config from the server.
debug.log
latest.log
mysticalcustomization.zip
Versions (please complete the following information):

Minecraft: 1.20.1
Forge: 47.2.20
Cucumber: 7.0.8
Mystical Agriculture: 7.0.11
Mystical Customization: 5.0.1
Singleplayer or Multiplayer: Multiplayer, running Arclight

You'll have to disable the automatic recipe generation for your crops docs and add them yourself since the custom server you're using is changing Minecraft internals which breaks the dynamic recipe generation.

If you try with Cucumber version 7.0.10 does this issue still occur?

I'm going to close this due to inactivity. If this is still an issue on the latest version you can reply here.


Support Pandoc's bracketed spans

Cont. to #41 https://github.com/jgm/commonmark-hs/blob/master/commonmark-extensions/test/bracketed_spans.md

To be specific, support id attributes in bracketed spans, e.g., [Practical exercises]{id="practical-exercises"}. (Once supported, my projects would no longer have false positives and I could then take advantage of all the other marksman awesome features.)


Add email to index for user entity

Hey, I just made a Pull Request!
Add user email to the index so that users can be found by their email.
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

@Rugvip anyone need a nudge to review this?


BearerAuth token not getting generated (1.3.11)

          With this change, does it mean that I should be able to set:

System.setProperty("BearerAuth", "abcdef");
when my API Spec contains:
components:
  securitySchemes:
    BearerAuth:
      type: http
      scheme: bearer
      bearerFormat: JWT

I'm not able to get this working.
My understanding is that Specmatic should automatically create a dummy Authorization header on requests without setting any properties / environment variables, but it is not even doing that. I've also tried various ways of setting environment variables, but still doesn't result in any authorization headers getting included on the request.
Any ideas?
I am using 1.3.11 which includes the changes in this PR.
Thanks.
Originally posted by @noelmccrory in https://github.com/znsio/specmatic/issues/1055#issuecomment-2064878462

Hi @noelmccrory, thanks for reaching out.
Looks like you are trying to run contract tests with a specification that has a security scheme defined.
Can you confirm if you are applying this security scheme to the APIs you are trying to test? Here's the documentation on how you can apply the defined security scheme to your APIs.
It would help us if you post a specification that would help us replicate the problem.

Bingo. I was missing the following from my API Spec:
security:
  - bearerAuth: []

After including it, Specmatic started to autogenerate an Authorization header. I was then able to use System.setProperty("bearerAuth", "abcdef"); to set my own token.
It may be worth updating https://specmatic.in/documentation/authentication.html to include some extra detail on setting the security as well as the security scheme.
Thanks!

@noelmccrory Glad you figured it out!
Just curious if you use any linters, if yes, it might have highlighted this issue in the spec in your IDE itself.
For example, in the following screenshot the Swagger Editor is telling me that I have not used the defined security scheme anywhere -

While technically Specmatic is operating correctly based on the OpenAPI spec, what we can do is, add the suggestion of using linters in our documentation to help users find out any spec level issues (e.g. unused security scheme) to get early feedback.
Closing this issue for now. Please feel free to reopen if necessary.

The API Spec is being generated using SpringDoc. I do have Stoplight installed and it didn't flag the unused security scheme 😞  Thanks again.


Updade go-apidiff to fix failing api-diff test

How to categorize this PR?

/area open-source
/kind bug
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #6168
Special notes for your reviewer:
/cc @timebertt
Release note:

NONE

Does this also fix the mentioned second problem?

if apidiff fails, the job should be marked as failed

Locally I have tested it works fine.

@acumino: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:
Test name	Commit	Details	Required	Rerun command
pull-gardener-apidiff	4f62543 link	false	/test pull-gardener-apidiff
Full PR test history. Your PR dashboard. Command help for this repository.

@timebertt it works as expected.


add same logic as glad ranked on integrated alerts download

Overview
Add same logic as the glad ranking widget on the integrated deforestation one.
Demo
If applicable: screenshots, gifs, etc.
Notes
If applicable: ancilary topics, caveats, alternative strategies that didn't work out, etc.
Testing

Include any steps to verify the features this PR introduces.
If this fixes a bug, include bug reproduction steps.

No applicable anymore


Fix: Clarify unclear instructions

Pre-Submission Checklist

[x] Your pull request targets the staging branch of freeCodeCamp.
[x] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[x] You have only one commit (if not, squash them into one commit).
[x] All new and existing tests pass the command npm test. Use git commit --amend to amend any fixes.

Type of Change

[x] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[x] Tested changes locally.
[x] Addressed currently open issue (replace XXXXX with an issue no in next line)

Closes #17624
Description

Helped to clarify the instructions by letting users know the number range is defined by their min and max attributes.

@Vimal-Raghubir, thank you for submitting a PR to fix this issue. Sorry for the delay. We're currently in the process of moving all updates to the curriculum over to a separate repository, so your PR is blocked for now.
Everything LGTM! @raisedadead, myself, or any other members will merge things are ready with the new repo.

Hi @Vimal-Raghubir
Thanks a lot for this pull request. We sincerely appreciate the time and efforts put into this.
However can you please make the same changes once again on the https://github.com/freeCodeCamp/curriculum repository?
I would like to apologise for the inconvenience, but at this point it has become very difficult to carry over commits into the other repo, to which we have moved over the entire curriculum package.
Thanks once again.
Please reach out to us in the Contributors Chat room for any assistance.
Happy contributing!


Create hand drawn "Wasteland Survival Guide" style card deck.

Suggested by user cadpnq.

As a not artist you can at least do a hand drawn Wasteland Survival Guide deck.

Having additional card decks will give us the ability to start on the Caravan card game. This is the kind of art style I can achieve on my own so will make a for a nice starting point.

Update to this, we have a custom model and base texture provided by Yurot, aka WarmocK

Post some screenshots. :)


Separate PoST proof data from metadata

Overview / Motivation
When generating a PoST, the following parameters need to be provided: id, space, difficulty, challenge, numProvenLabels. When validating the proof, the validation is based on these parameters.
At the moment some of these fields are attached to the proof, and some are not. This can lead to a security problem wheres the validator side would forget to assert these values against the expected ones, before validating the proof.
The Task
When generating a PoST, return the proof in addition to a metadata object, which will contain these parameters, and must be sent in addition to the proof, to be used by the validator.
In addition, add assertion to id as part of the activation transaction syntactic validation. It must match the transaction signer public key.

@noamnelke should we close this?

Private case of #1134 - closing


Execute js_binary from the working directory it's called from?

Hey guys,
I'm trying to use js_binary in conjunction with bazel run to build and run an executable that operates on files relative to the current working directory in my terminal.
However, the current working directory (process.cwd()) inside the entry_point script ends up being deep inside the runfiles directory rather than where I ran the program from. Is there a reason for this? It seems it would make sense that the cwd should be where the program is executed from. Or is there a way to make this happen that I'm not seeing in the docs?
Thanks,
Greg

Any bazel run will have the cwd in the runfiles. It looks like BUILD_WORKING_DIRECTORY would be what you're looking for? See https://bazel.build/docs/user-manual

@jbedard Ah, thank you, definitely what I'm looking for! This env var might be a good addition to docs for any other users that might not be familiar.
Thanks again,
Greg


Conversion from TeX to plain text

Is there a way to convert the TeX markup into plain text as much as possible? Just something like pandoc example.tex -o output.txt.
I saw there are conversions to markdown, json, along with other formats in the playground (https://siefkenj.github.io/latex-parser-playground/).
I'm implementing grammar check for TeX documents and I want the TeX markups to be removed because they interfere the checking results.

You can look at the to-markdown plugin for what it looks like to unwrap strings. I don't know if you need line information from the original or not, but it shouldn't be too hard to unwrap everything. You'll have to decide what macro arguments you keep. Like \foo{my thing}{other thing}. Do you want that to turn to my thingother thing? Or something else...

You can look at the to-markdown plugin for what it looks like to unwrap strings.

Thanks, I'll definitely look into to-markdown.
For the macros, I think it really depends. Like \color{red}{my text}, I want it to be my text.
For \usepackage[xxx]{yyy}, I hope it can be entirely ignored.


Add version somewhere

For debugging purposes, we need to somehow add a version to generated code or page

Fixed in #34


mention of avatar in public dir on client

it's possible to set a avatar by putting it in /keybase/public//me/avatar.[png/jpg], but the deskop client does not mention this
it would also be gread if keybase could also use avatars from pgp key.

It doesn't even work for me to put an image in this folder. I tried to drag&drop an image to the folder and getting the message that there is not enough space. And my disk has 34GB still available...

@malgorithms can you copy other files, to any location in your keybase folder? is you avatar > 5mb?
i was using ubuntu 16.04 but i don't know which keybase version it was.
anyway thats what the webapp is mentioning:
Pro tip: With Keybase installed on your computer, you can put a picture up to 5MB in /keybase/public/yellowsir/me/avatar.jpg (or avatar.png) and select it here. ``


Remove vpn from jenkins

Openconnect was being used to connect to the GDS VPN so Jenkins could
access github enterprise.
We've moved all relevant repos from enterprise to github, so the vpn is
no longer required.
PR to remove credentilas from jenkins-vars is here.

 Get rid of GitHub GDS - M


cordova-android 7.0.0 doesn't support API level 27, but the documentation suggests it does

Issue Type
[X] Bug
[ ] Feature Request
Priority
[ ] Minor
[X] Major
[ ] Critical
[ ] Blocker
Environment

OS: Android
Browser: N/A

Description

When reading the documentation on the Android platform support in Cordova, it says that 7.X.X supports API level up to 27.

cordova-android Version
Supported Android API-Levels
Equivalent Android Version

7.X.X
19 - 27
4.4 - 8.1

6.X.X
16 - 26
4.1 - 8.0.0

5.X.X
14 - 23
4.0 - 6.0.1

4.1.X
14 - 22
4.0 - 5.1

4.0.X
10 - 22
2.3.3 - 5.1

3.7.X
10 - 21
2.3.3 - 5.0.2

However, when I tried to run the emulator ("cordova emulate android"), it didn't work, failing with "Cannot read property 'semver' of null" in platforms/android/cordova/lib/emulator.js.
The problem is that "cordova platform add android" (for cordova 8.0.0, the latest version) installs cordova-android 7.0.0, which doesn't include support for API level 27.
According to the documentation, p"every release of cordova CLI pins a version for each platform", which means that the upper limit in the above table is often misleading.

Marked as a bug, for now at least. Will need some further testing on our part.

Thanks! Since the documentation recommends using the latest API Level supported by the version of cordova-android, this seems like something that every new project currently runs into.

Which API levels do work with cordova-android 7.0.0 in your experience or testing @ariofrio?

cordova-android 7.0.0 depends on android-versions 1.2.1

Since android-versions is a bundled dependency, the latest applicable version of it when cordova-android 7.0.0 was published is the one that is always used, unless a new version of cordova-android is built and published

android-versions does not include Oreo MR1 (API Level 27) until version 1.3.0

Makes sense. Could you please practically confirm this? Otherwise some Cordova developer will have to do so.
I guess the table was meant "the latest release of cordova-android 7.x.x supports".
But it might make sense to add the individual releases to the table, right?
Problem: Who volunteers to find out all those data points?

Closing this as the issue is obsolete and won't be corrected.
cordova-android does not use bundled dependencies anymore so moving forward I don't think we will see this issue again.

Closing this as the issue is obsolete and won't be corrected.
cordova-android does not use bundled dependencies anymore so moving forward I don't think we will see this issue again.


Combined services to analyze static files

hey! this is a test comment for the students.

i may have missed it - but i did not see a go.mod or a go.sum file.

Hey team - I am still not seeing the go.mod or go.sum files. Can we make sure those are part of this PR please 😄


Add new  Enhanced Transaction Filtering with Category Filters and fix…

… bug Lack of padding below the expense chart

well done, looks good to me.

Thank you very much. I learned a lot through solving this issue. Hope to be able to participate in many other jobs with you in the future

Fixes: #84


The Humble report for Eclipse Cyclone DDS

@Thijs @joespeed FYI, this is the rebased version of #59, with the conflicts fixed.  I couldn't push to the atolab fork, so I opened this version instead.  We are going to go ahead and merge this one.

@thijs @joespeed FYI, this is the rebased version of #59, with the conflicts fixed. I couldn't push to the atolab fork, so I opened this version instead. We are going to go ahead and merge this one.

Please check the @'s you use, @ thijs is me, but I'm not the one you're looking for.


Find a better place for "anytype" in the docs

Moved anytype into a section about abstract types together with other "any" types in #526


Functionality to verify QuoteNonce

Need to have one source of truth for recreating/verifying the QuoteNonce against the hash thing that SGX SDK is putting into the QE ReportData.

Done in #188


unable to deploy a Node app to Azure website due to a native depenency

I've built an app using 6to5 that I am deploying to an Azure website. When I attempt to use the push to deploy model (as opposed to just uploading the build app), Azure runs npm install and the deployment fails. The problem is discussed here:
http://stackoverflow.com/questions/19039842/node-js-app-with-node-gyp-fails-to-deploy-on-azure-website
I narrowed the depedency down to Chokidar which in turn relies on FSEvents.
My question is: is there away to make this depedency optional?
I can workaround this, but I'd love get back to a push-to-deploy world. :wink:
thanks.

IIRC you can use the 6to5-core package if you're using the API for the build process.

Thanks, I'll give that a shot and report back.

Yeah, you're only using the 6to5 for 6to5/register so installing 6to5-core and using 6to5-core/register will work since 6to5-core doesn't include chokidar as it's only needed for the CLI.

Everything is working. Many thanks!


go generate question

Would it be possible to add the ability to use go 1.4's new generate tool:
//go:generate bambam $GOFILE
So that you can create files using go generate?

I welcome pull requests. Give it a try and let me know if it works.

Generally though, since certain parts of golang are better in the 1.2 and 1.3 series, (namely debugging info for gdb, and calls into C are faster) I wouldn't require the later, slower go1.4 at this point.


[TC-MC-3.2]-KeyPad Input Cluster-In Step 1 TH Fails to sends CEC Settings Keys(0x0A) to DUT

Problem
As per the Test-Plan, when TH sends CEC Settings Keys(0x0A) to DUT,
DUT should responds with success status code(0) but currently from DUT Getting a response as Unsupported key(value 1).

expected behavior:-Success(status code 0)

actual behavior:-UNSUPPORTED_KEY(status code 1)

steps to reproduce:- ./chip-tool keypadinput send-key 10 1 1

system configuration:-
Chip-Tool: RPI
DUT: RPI
Platform : IP
Commit-ID: 99988e5ae1a2271e4eec9e0c296a3fd58358238d
Dut: TV-app
Test-plan reference:-
https://github.com/CHIP-Specifications/chip-test-plans/blob/master/src/cluster/media.adoc#tc-mc-2-1-low-power-mode-verification-dut-as-server
Spec Reference
https://github.com/CHIP-Specifications/connectedhomeip-spec/blob/master/src/app_clusters/media/KeypadInput.adoc#statusenum
PFA log:-
TC-MC-2.1.txt

Facing the same issue in [TC-MC-3.3]
As per the Test-Plan, when TH sends Number 1 to 9 Code to DUT,
DUT should responds with success status code(0) but currently from DUT for numeric input 5, 6, 7, 8 Getting a response as Unsupported key(value 1).
expected behavior:-Success(status code 0)
actual behavior:-UNSUPPORTED_KEY(status code 1)
steps to reproduce:-
./chip-tool keypadinput send-key 5 1 1
./chip-tool keypadinput send-key 6 1 1
./chip-tool keypadinput send-key 7 1 1
./chip-tool keypadinput send-key 8 1 1
Test-plan reference:-
https://github.com/CHIP-Specifications/chip-test-plans/blob/master/src/cluster/media.adoc#tc-mc-3-3-number-keys-verification
PFA log:-
TC-MC-3.3.txt


AnimatedImage not loading gif from assets

I am trying to a gif from assets using AnimatedImage, but it's not loading. Please let me know how I can do that. I am using below mention code.
AnimatedImage(name: "loading.gif", isAnimating: .constant(true))                             .resizable()                             .scaledToFit()

gif from assets

You means, Photos PHAssert library image ?

gif from assets

You means, Photos PHAssert library image ?

I was adding gif in Assets.xcassets. But now I moved it outside in resources folder and it is working fine.

I was adding gif in Assets.xcassets. But now I moved it outside in resources folder and it is working fine.

xcassets does not supports Animation at all. This is limited by Xcode and Apple, not something we can control. Moving gif or webp out of xcassets and use Bundle path to load is OK.

xcassets does not supports Animation at all. This is limited by Xcode and Apple, not something we can control. Moving gif or webp out of xcassets and use Bundle path to load is OK.

Thanks for the explanation.

Thanks for the explanation.


Update name in swagger

Pull Request Test Coverage Report for Build 1522

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

Totals

Change from base Build 1518:
0.0%

Covered Lines:
2937

Relevant Lines:
3710

💛  - Coveralls

:tada: This PR is included in version 1.14.5 :tada:
Your semantic-release bot :package::rocket:


[FEATURE] Re-work exception framework instead of deprecating Falco rules exceptions

Motivation
During today's core maintainer meeting (October 5, 2023) we decided to completely re-work exceptions instead of deprecating them with the primary goals of making them more user-friendly and easier to maintain.
Incorporate this into existing expression language issues work stream, more details TBA @jasondellaluce.
More details TBA.

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

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

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

@poiana: Closing this issue.

In response to this:

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

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.


Monitoring->Utilization view hangs for middleware servers and middleware datasources

Navigating to middleware server details page->Monitoring->Utilization does not display the Capacity & Utilization data. It displays the spinner continuously and does not render the view.
Product: Latest upstream
Steps:

Login to MIQ
Add Middleware Provider
Navigate to Middleware->Middleware Servers
Click on a Server link.
Select Utilization from Monitoring dropdown menu

Also Middleware->Middleware Datasources->Datasource summary page->Monitoring->Utilizaion view hangs

@miq-bot add_label providers/hawkular
cc @abonas

@miq-bot add_label bug

@skondkar does this happen for all server types? in the example you try this on hawkular, does that happen on EAP server as well?

@abonas yes. Tested steps for EAP server.  Monitoring->Utilization view hangs for EAP server as well.

@abonas yes. Tested steps for EAP server. Monitoring->Utilization view hangs for EAP server as well.

ok thanks for clarifying

This issue happens for Hawkular and EAP 7 both.

@chrisarcand is this issue is related to #10088?
I see this exception in the log:
[----] E, [2016-07-31T18:50:45.040755 #11913:36f998] ERROR -- : MIQ(MiqQueue#deliver) Message id: [16274], Error: [undefined method `apply_legacy_finder_options' for LiveMetric:Class]
[----] E, [2016-07-31T18:50:45.040989 #11913:36f998] ERROR -- : [NoMethodError]: undefined method `apply_legacy_finder_options' for LiveMetric:Class  Method:[rescue in deliver]
[----] E, [2016-07-31T18:50:45.041144 #11913:36f998] ERROR -- : /home/abonas/code/manageiq/manageiq/lib/rbac/filterer.rb:442:in `method_with_scope'
/home/abonas/code/manageiq/manageiq/lib/rbac/filterer.rb:202:in `search'
/home/abonas/code/manageiq/manageiq/lib/rbac/filterer.rb:86:in `search'
/home/abonas/code/manageiq/manageiq/lib/rbac.rb:5:in `search'
/home/abonas/code/manageiq/manageiq/app/models/miq_report/generator.rb:277:in `_generate_table'
/home/abonas/code/manageiq/manageiq/app/models/miq_report/generator.rb:176:in `block in generate_table'
/home/abonas/code/manageiq/manageiq/app/models/user.rb:278:in `with_user'
/home/abonas/code/manageiq/manageiq/app/models/miq_report/generator.rb:176:in `generate_table'
/home/abonas/code/manageiq/manageiq/app/models/miq_report/generator/async.rb:34:in `block in _async_generate_tables'
/home/abonas/code/manageiq/manageiq/app/models/miq_report/generator/async.rb:33:in `each'
/home/abonas/code/manageiq/manageiq/app/models/miq_report/generator/async.rb:33:in `each_with_index'
/home/abonas/code/manageiq/manageiq/app/models/miq_report/generator/async.rb:33:in `_async_generate_tables'
/home/abonas/code/manageiq/manageiq/app/models/miq_queue.rb:347:in `block in deliver'
/home/abonas/.rvm/rubies/ruby-2.2.2/lib/ruby/2.2.0/timeout.rb:89:in `block in timeout'
/home/abonas/.rvm/rubies/ruby-2.2.2/lib/ruby/2.2.0/timeout.rb:34:in `block in catch'
/home/abonas/.rvm/rubies/ruby-2.2.2/lib/ruby/2.2.0/timeout.rb:34:in `catch'
/home/abonas/.rvm/rubies/ruby-2.2.2/lib/ruby/2.2.0/timeout.rb:34:in `catch'
/home/abonas/.rvm/rubies/ruby-2.2.2/lib/ruby/2.2.0/timeout.rb:104:in `timeout'
/home/abonas/code/manageiq/manageiq/app/models/miq_queue.rb:343:in `deliver'
/home/abonas/code/manageiq/manageiq/app/models/miq_queue_worker_base/runner.rb:106:in `deliver_queue_message'
/home/abonas/code/manageiq/manageiq/app/models/miq_queue_worker_base/runner.rb:134:in `deliver_message'
/home/abonas/code/manageiq/manageiq/app/models/miq_queue_worker_base/runner.rb:152:in `block in do_work'
/home/abonas/code/manageiq/manageiq/app/models/miq_queue_worker_base/runner.rb:146:in `loop'
/home/abonas/code/manageiq/manageiq/app/models/miq_queue_worker_base/runner.rb:146:in `do_work'
/home/abonas/code/manageiq/manageiq/app/models/miq_worker/runner.rb:334:in `block in do_work_loop'

@miq-bot assign @lucasponce

@lucasponce this is mine. I'll assign back to you when I want your feedback

@abonas this issue is blocker for QE to test utilization. Could you please manage to get it fixed?

@abonas this issue is blocker for QE to test utilization. Could you please manage to get it fixed?

@lucasponce / @kbrock can you please update the status of the fix / ETA? thank you

@abonas sorry - on it

@kbrock any progress/ETA with this issue?

@abonas meeting with gt @ 3:00PM EST, hoping to get merged shortly after that

@abonas and @lucasponce can you let me know if this is resolved?

Hi @kbrock, I have prepared a small PR to adjust a wrapped array into live_metrics.
This solves this issue.
But I expected also that as part of the "args" variable in AAArM.find(*args) we could get the old ext_options[:only_cols] or equivalent data but in different format.
With that LiveMetrics can optimize its calls into the provider and only fetch the info related of the specified columns.
Do you know if that enhancement is on some of the pendings PRs related to AAArM context ?
Thanks.

Re-checked - it works for EAP instance, and still hanging for Hawkulars.
Reopen?

Re-checked - it works for EAP instance, and still hanging for Hawkulars.
Reopen?

I tried yesterday's master branch and it doesn't hang for hawkulars. can you recheck with the latest code please?

Instance I used for check was created today (5 hours ago) through Sprout's 'upstream' option...
Is it the latest one?

Instance I used for check was created today (5 hours ago) through Sprout's 'upstream' option...
Is it the latest one?

I have no idea what Sprout is or what setup does it create

Well, it was re-checked again. It works.
Let be closed.

Verified on latest upstream. This issue does not exist anymore. Monitoring->Utilization view appears for EAP7 and Hawkular middleware servers and datasources.


err_cleanup deletes err_thread_local even if not initialized

I've been debugging a crash in colord-sane which led me to libcrypto (https://bugs.archlinux.org/task/54205).
OPENSSL_cleanup, which is called atexit, runs err_cleanup which cleans up err_thread_local even if the latter has never been initialized and is still 0.
Pthread key 0 is valid and belongs to glib (specifically GSlice's private_thread_memory). Glib is still running in other threads while the main thread is exiting.
Glib's worker thread races with the main thread and, if unlucky, attempts to use the thread-local after libcrypto has destroyed, resulting in a noisy SIGABRT.
libcrypto should not delete a pthread_key_t it never created.
Probably fallout from https://github.com/openssl/openssl/pull/3035.
OpenSSL 1.1.0f
glib 2.52.2
glibc 2.25

Does this patch fix it?
; g diff
diff --git a/crypto/err/err.c b/crypto/err/err.c
index f866f2f..c55f849 100644
--- a/crypto/err/err.c
+++ b/crypto/err/err.c
@@ -122,6 +122,7 @@ static ERR_STRING_DATA ERR_str_reasons[] = {
 #endif

 static CRYPTO_ONCE err_init = CRYPTO_ONCE_STATIC_INIT;
+static int set_err_thread_local;
 static CRYPTO_THREAD_LOCAL err_thread_local;

 static CRYPTO_ONCE err_string_init = CRYPTO_ONCE_STATIC_INIT;
@@ -260,7 +261,8 @@ DEFINE_RUN_ONCE_STATIC(do_err_strings_init)

 void err_cleanup(void)
 {
-    CRYPTO_THREAD_cleanup_local(&err_thread_local);
+    if (set_err_thread_local != 0)
+        CRYPTO_THREAD_cleanup_local(&err_thread_local);
     CRYPTO_THREAD_lock_free(err_string_lock);
     err_string_lock = NULL;
 }
@@ -639,6 +641,7 @@ void ERR_remove_state(unsigned long pid)

 DEFINE_RUN_ONCE_STATIC(err_do_init)
 {
+    set_err_thread_local = 1;
     return CRYPTO_THREAD_init_local(&err_thread_local, NULL);
 }

That indeed seems to fix it; thanks.

I feel that the patch is fixing wrong problem.
It seems to me that the real problem is that macro DEFINE_RUN_ONCE_STATIC is invalid if used more than once in singe file. See: https://github.com/openssl/openssl/blob/master/include/internal/thread_once.h#L58  – the macro relocates actual code to static function named init, then calls this function.
If the macro is used twice in single .c file, then single compilation unit contains two static functions named init, and two calls to this function from static initializer. Usually this means that linker throws one away and calls another twice.
Now take a look at
https://github.com/openssl/openssl/blob/master/crypto/err/err.c
My bet is that  do_err_strings_init gets called twice (leaking a bit), and err_do_init - never.
Not sure what are consequences of uninitialized err_thread_local (apart from openssl cleanup calling pthread_key_delete on value 0, which accidentally happen to be value of uninitialized err_thread_local) but probably there are some.
To fix the problem either macro DEFINE_RUN_ONCE_STATIC should be rewritten to work if called more than once, or err.c split into two files.
PS I just spend a day of debugging tracking mysterious exit error in my code, which finally turned out to be caused by openssl calling pthread_key_delete on key zero which belonged to my library.

As I can't reopen this issue, I raised https://github.com/openssl/openssl/issues/8384


There should be a standard form to display an entity, audit trail

Overview
Currently the framework is able to create an audit trail for any meaningful actions on an entity and or data changes on track properties. However, there is no mechanism to easily display that audit trail.
Acceptance Criteria

Create a standard form that may be displayed in a dialog box to view the audit trail of an entity.
The form should accept 3 parameters:

EntityType - the name of the type of the entity whose audit trail should be displayed.
EntityId - the idea of the entity whose audit trail should be displayed.
IncludeEventsOnChildEntities - If true, the audit trail should also display audited events for child entities.

The details for template should be updated to include a history button on the toolbar which displays the entity audit trail in a dialog box
If an end user clicks on the history button and dialog box should be displayed listing the full audit trail displayed as a Datta list in the following formats.
Events should ordered with the most recent events on top

————— Template for properties updated event———————————
{EventDate} {EventTime} - {user name} updated:

{Property1 changed name} from ‘{prev value}’ to ‘{new value’
{Property2 changed name} from ‘{prev value}’ to ‘{new value’
…
————Template for custom event———————————
{EventDate} {EventTime} - {user name}:
{Custom event description}

Awaiting for a childEntity property that I can use to test with before finalizing the item

Hi @OmolemoBlessingLethuloe, please check requirement 5: Events should ordered with the most recent events on top and update accordingly. Currently, the most recent item is listed at the bottom and old on top


Implement jsx generics

Similar to https://github.com/Microsoft/TypeScript/pull/22415

For reference --cause I'm sure people will come here looking for ways to do this-- right now you have to define dummy components.

Thx @mrkev, that would've never come to my head)

Closing for more descriptive https://github.com/facebook/flow/issues/7672


Using this results in my server-side rendering locking up

I'm trying to use this in the context of react-starter-kit. After integrating apollo-client according to its docs I proceeded to incorporate this similarly straightforwardly. However, for me this causes server-side rendering to lock up (browser times out) without throwing any errors, so I'm somewhat clueless as to the cause (and how to go about debugging it). Any ideas?

Hi,
Thanks for the report. Can you try running your app with debug output enabled (ie DEBUG=apollo-local npm start) and see if anything from this module gets logged? You could also add another debug() call before the try/catch in query() to make sure it's getting called.

Sorry about the trouble: it turns out I was simply using the wrong import syntax instead of the require given in the docs. Things now work as expected, thanks! :-)


Entra ID Authentication for WinGet REST sources

Description of the new feature / enhancement
I want to be able to authenticate to an Entra ID enabled WinGet REST Source.
Related to:

#1918
microsoft/winget-cli-restsource/issues/226

Proposed technical implementation details
No response

I don't know if this is the right place to write this; But the hosted version of winget.Pro supports Entra ID authentication now to restrict access to applications: https://winget.pro/winget-entra-id-authentication/


Add Backup and Restore feature

Closing as duplicate of https://github.com/freeotp/freeotp-ios/issues/20


Tagging Cloudwatch

CloudWatch logs created automatically for chip-n-scale aws resources like the downloadandpredict lambda, but these aren't tagged. We should add tagging to these cloudwatch resources so it's easier to monitor costs.
cc @drewbo

Attempted to add this morning but there are a few blockers that make this more of a challenge:

Cloudwatch logs can't be tagged via Cloudformation (ref: https://lumigo.io/blog/sar-app-to-propagate-cloudformation-tags-to-unsupported-resources/)
Testing shows that the log group for the download and predict lambda isn't created at deploy time but will be automatically creation after one function invocation

Steps to implement:

Add cloudwatch logs for each lambda function to the cloudformation template
Tag in a script


opti: create a custom unsigned_div_rem to reduce step used

Feature Request
Add unsigned_div_rem to kakarot utils and inline where possible.
Remplace every mention of from starkware.cairo.common.math import unsigned_div_rem by the one created.

can i work on this?

@enitrat @obatirou can i create unsigned_div_rem.cairo in kakarot utils folder because of a circular import error in (bytes.cairo and utils.cairo) when i add the unsigned_div_rem function in utils.cairo? or how best to resolve this error?

The idea of this issue is to inline the ˋassert_le. A benchmark needs to be done to know how many steps would be saved. As it is used a lot in the codebase it could be interesting. What is the exact circular import error you get ? You cannot import from utils.cairo to bytes.cairo ? If it is not resolvable, I would suggest to create a new file named math.cairo` to replicate the structure from starkware.

@enitrat @obatirou can i create unsigned_div_rem.cairo in kakarot utils folder because of a circular import error in (bytes.cairo and utils.cairo) when i add the unsigned_div_rem function in utils.cairo? or how best to resolve this error?

Hi, thank you for your interest, unfortunately, issues open to contribution have a specific tag open for contribution


screwy nametags

image might be from spectrum or voyager ! don't know which but it happens the same on both

I was not able to reproduce the completely missing text seen in the image, using either OptiFine 1.19.2 HD U I3 or Iris 1.6. That part in particular may be the result of an outdated version of either OptiFine or Iris.

It cannot be an out of date iris version as I was using the latest iris version at the time of posting the issue Sent from Mail for Windows From: Jacob ErikssonSent: niedziela, 23 kwietnia 2023 15:32To: zombye/spectrumCc: KiloTheNeko; AuthorSubject: Re: [zombye/spectrum] screwy nametags (Issue #4) I was not able to reproduce the completely missing text seen in the image, using either OptiFine 1.19.2 HD U I3 or Iris 1.6. That part in particular may be the result of an outdated version of either OptiFine or Iris.I did reproduce the name background being overly dark, as well as entities and translucent blocks like stained glass always rendering as if in front of that background.—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>


Storage Explorer to be used with IoT Edge blob storage.

Would it be possible to change the storage explorer so it doesn't only go to either Azure or <IP_ADDRESS>, but can also put in the fully qualified address (f.i. http://<ip-address>:<port>/<storage-name>? If we can set the qualified address to point at the IoT Edge it would be possible to access the Azure IoT Edge Blob Storage module (https://docs.microsoft.com/en-us/azure/iot-edge/how-to-store-data-blob).

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

ID: 337c96d2-0b4f-ad70-bb7a-cf8d6736cc3c
Version Independent ID: 09c7c658-16d1-04ac-6bb4-6aa5c1b529a6
Content: Get started with Storage Explorer
Content Source: articles/vs-azure-tools-storage-manage-with-storage-explorer.md
Service: storage
GitHub Login: @cawa
Microsoft Alias: cawa

@gbbiotwesouth AFAIK , it is not possible to change it from a local host to an actual IP address, Ports though can be changed. I'd recommend providing feedback  on this link. Let me know if this helps.
Thanks,
Adam

We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


fix: Existing git connected app shows migration changes related to queries

This reverts commit ccd5c3a6600e91e0daa5008b3999faa52a3d258c.
PR fixes following issue(s)
Fixes #29400

if no issue exists, please create an issue and ask the maintainers about this first

Media

A video or a GIF is preferred. when using Loom, don’t embed because it looks like it’s a GIF. instead, just link to the video

Type of change

Please delete options that are not relevant.

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
Chore (housekeeping or task changes that don't impact user perception)
This change requires a documentation update

Testing

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Also list any relevant details for your test configuration.
Delete anything that is not relevant

[ ] Manual
[ ] JUnit
[ ] Jest
[ ] Cypress

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

Summary by CodeRabbit

Refactor

Improved the export functionality by ensuring creation and update timestamps are not included, leading to cleaner data management for exported objects.

/build-deploy-preview


[PM] Error: Git builder not found

This error is rather frustrating, because I'm running the command to create a git builder, so I don't understand why it would need to find one.
$ rspm create git-builder<EMAIL_ADDRESS>--source=prod-internal-src --build-trigger=commits
Cloning repo and waiting for initial package builds...

This could take some time depending on the size of the repo and the number of package builds...

Error: Git builder not found.

What's also weird is this appears to work randomly. One time I ran a command, it failed, ran the exact same command again, and it worked.
What's also very weird is when I list the git builders I see builders, but they have weird names.
Git Builders:
- B2Ig1sRG1nne
    Source: prod-internal-src
    URL<EMAIL_ADDRESS>    Trigger: commits
    Branch:
    Credential Name: none

Chart version 0.5.13

That is really strange and concerning 😮 Sorry for the trouble here @Almenon !
This honestly may be something to report to the product team - have you submitted a support ticket on this issue yet?
If so, please link here! The helm chart may be involved here, but this definitely seems more like a potential product issue!
Are you using our public docker image? Or your own container?

Public docker image. I submitted a support ticket, the error is extremely weird. More details in the ticket.
#96579

Support recommended that I use a external database rather than a EBS volume. Once I did that the error went away.


Add support for dependency relationships for alpine (apk)

Partially addresses #572

@tgerla I took this out of draft - if you push your changes I can review

Hi @spiffcs (and anyone else interested) -- I think this one is ready to go. Can you please review my last commit, "adjust test conditions"? I think I've updated the test correctly, but I am Go noob.

@tgerla it looks like there may be more things that were not originally in scope that need to be considered as part of this PR. It looks like the https://wiki.alpinelinux.org/wiki/Apk_spec page got a really healthy update over the last few months to help interpret an APK index vs installed DB file, which will be really helpful for determining the missing bits. I'd be happy to pair on this.

This one should not have been squashed 😄


When creating data changes, no components are re-rendering.

Context:
https://github.com/bdougie/open-sauced/issues/255 & https://github.com/bdougie/open-sauced/issues/215 are both instances where data changes on triggering a Mutation. Not that we are fully using hooks, the error is gone. The problem of the data not updating when the mutation finishes now exist.
Things to fix:

[ ] As a user I should create a new goal and see it populate in the list without refreshing the page
[ ] As a user without any open-sacued-data, I should be able to click a "create a goal" button and be presented with blank repo list.

Labeling as a bug, because it should get addressed sooner than later.

@NileshPatel17 do you have any thoughts on this? Or have experience with data stores outside of redux?
I am looking into leveraging this pattern with Apollo caching and useReducer..

Actually, I think the solution is leveraging apollo better with the @apollo/react-hooks.
import gql from 'graphql-tag';
import { useQuery } from '@apollo/react-hooks';

const GET_DOGS = gql`
  {
    dogs {
      id
      breed
    }
  }
`;

function Dogs({ onDogSelected }) {
  const { loading, error, data } = useQuery(GET_DOGS);

  if (loading) return 'Loading...';
  if (error) return `Error! ${error.message}`;

  return (
    <select name="dog" onChange={onDogSelected}>
      {data.dogs.map(dog => (
        <option key={dog.id} value={dog.breed}>
          {dog.breed}
        </option>
      ))}
    </select>
  );
}

In addition to that also add localState management with apollo/react-hooks. I might use this as a guide: Learn how to work with your local data in Apollo Client

Updating local state
There are two main ways to perform local state mutations. The first way is to directly write to the cache by calling cache.writeData. Direct writes are great for one-off mutations that don't depend on the data that's currently in the cache, such as writing a single value. The second way is by leveraging the useMutation hook with a GraphQL mutation that calls a local client-side resolver. We recommend using resolvers if your mutation depends on existing values in the cache, such as adding an item to a list or toggling a boolean.

let me explore apollo-cache-inmemory

I tried this a little last night and did the initial setup. From my exploration a custom hook may be needed to take the OneGraph implementation into account. I could not quite figure it out on first pass and ran out of time.
I have not created a custom hook before and using this article on local caching as inspiration. Let me know if you get farther in your discovery @NileshPatel17

As a user I should create a new goal and see it populate in the list without refreshing the page

This issue has been taken care in #383
** Yet to explore apollo-cache-inmemory

is it possible to deploy code from PR having in draft state??

alright. it is possible. It is deployed at https://deploy-preview-383--open-sauced.netlify.com/
let me know what do you think.

** Yet to explore apollo-cache-inmemory

@NileshPatel17 I left comments on the PR. Regarding the caching, it doesn't have to be addressed in your current PR since it is actually unrelated to this issue. It is related to #262, I made the original suggestion on the wrong issue to begin with.
Thanks for PR, it is very much appreciated.


not sure what this error is about but

Any time script editing is done, this error appears.

Don't install zslint vscode extension, it is out-of-date and unsupported


Use transaction with lock on order

Lock the order by row to prevent completing segments from creating a new
payment + payment source and attempting to complete the order.
Both sections do more or less the same thing so they really need to happen
in isolation as they branch based on whether or not the order is completed.

otherwise looks good

jenkins pls

jenkins pls

jenkins pls

jenkins rebuild this

jenkins rebuild this

jenkins rebuild this

jenkins rebuild this

We've run our regression suite on this as well, in case you were struggling to test it like the other PR, but this test doesn't really check whether the concurrency issues are fixed.

@simonpfry I'm open to suggestions about how to expose the race condition in a spec. Due to concurrent nature of the problem it's difficult to simulate it.
The only thing I can think of doing to try to expose it would be to spawn two threads that independently issue one of the two competing requests, a couple dozen times in a hope that the behavior manifests itself.
I'm quite positive that this should resolve the concurrency issue as both critical sections use a lock on the same record prior to the branching behavior that is dependent on order completion and having a payment.

Ultimately if this does fix the problem, then it should now longer appear in production, and if it doesn't (which is unlikely) there will be no change.
I of course would prefer this to be tested but given the limitations of rspec and ruby and don't think it would be worth the amount of effort required to try to replicate the behavior.

I totally agree @Dkendal, we've tried to spec various concurrency issues we've encountered by spawning multiple threads, but we've never found these tests particularly stable. I'm happy to get this pushed live to our system and see what comes out, I expect you're right that this will fix the issue.

Cool, here's hoping!


error

error with plugin. test with tesseract

12:17:56 [CRITICAL] Could not pass event 'pocketmine\event\server\DataPacketReceiveEvent' to 'NameCorrector v1.0.0#build011': Class pocketmine\network\protocol\Info not found on NameCorrector\NameCorrector
12:17:56 [CRITICAL] ClassNotFoundException: "Class pocketmine\network\protocol\Info not found" (EXCEPTION) in "/src/spl/BaseClassLoader" at line 144

wrong repo? lol