Redesign Profile Implementation

Background: A few things broke in our testing pipelines when working with the Network Activity class schema, it turns out the class now includes 5 profiles (“cloud”, “domain_security”, “host”, “malware”, “user”), which basically add the attributes as defined in the profile definitions to the rendered schema by default. These definitions then define requirements for the attributes within. For example, this is how the malware profile looks like.
With this example in mind, now if you query for a network_activity class schema in the browser, you’ll get a JSON schema that includes all the attributes defined in the profile, which includes additional required fields that you don’t really require to normalize a network activity log. With the malware profile, it adds disposion_id & disposion as required fields (side note - there appears to be a spelling mistake in this attribute, assuming it is disposition_id)
Thus, even if, I as a user do NOT want malware profile for my network_activity class schema, I am forced with the requirement to populate/map disposion_id, else my data will technically be non-compliant with OCSF, which doesn’t make sense.
Additionally, there is a disconnect between browser view and the actually rendered JSON schema, the JSON seems to by default include all profile attributes regardless of what you choose in the browser via the checkboxes. So the checkboxes give false sense of control to the user.
In summary, profiles appear to have become a compulsive factor instead allowing for originally intended freedom.

From my understanding, it boils down to how profiles are defined in the event_class (as includes in the JSON definitions of the event_class)
I think the way it should be implemented is,
You shouldn’t define profiles in the JSON class definitions at all, instead the profile selection should an api parameter where you can explicitly mention which profiles you want for any desired  event_class. The same can be translated in the browser in the form of checkboxes.
If no profiles are mentioned the user must be presented with the vanilla definition of the event_class both in the browser & in the actual JSON schema. Profiles should only be added (i.e. the attributes defined in the profiles), when explicitly selected in the api/browser.

Rajas, is there anything else to do for this issue?

We should be good with the API enhancements, especially considering we are sticking the profile definition in the event class schema.


Warning: validateDOMNesting(...):  cannot appear as a descendant of .

I get this console warning when pulling data from Sanity to a React/Gatsby app:
react_devtools_backend.js:3973 Warning: validateDOMNesting(...):
 cannot appear as a descendant of
.
    at p
    at normal (webpack-internal:///./node_modules/@portabletext/react/dist/react-portable-text.mjs:132:14)
    at PortableText (webpack-internal:///./node_modules/@portabletext/react/dist/react-portable-text.mjs:165:10)

Data looks like this (passed as value prop in <Portable Text value={data}/>:
{
  "_key": "422b2df8e2a1",
  "_type": "block",
  "children": [
    {
      "_key": "33d2dcb9b87f0",
      "_type": "span",
      "marks": [],
      "text": "Some text here."
    }
  ],
  "markDefs": [],
  "style": "normal"
}

Happens on all block content. This might not be judged as an error as things do work, but annoying to always have it in the console.

What does your rendered html structure look like?

What does your rendered html structure look like?

Sorry, should have thought about including the output html to get a view on the nested
's:
When exanimating this I see that it is MUI's Typography component that's making the trouble. I thought this was just a div, but it too renders a p, so combined with Portable text there will be a nested paragraph element.
So this:
                    <Typography variant="body1">
                      {content ? (
                        <PortableText
                          // Pass in block content straight from Sanity.io
                          value={content}
                          components={{
                            types: { image: ImageComponent },
                          }}
                        />
                      ) : `[${getLang('noContent')}]`}
                    </Typography>

Leads to this:

Test text

My bad. Sorry. I'll close the issue. :-)


fix(fix form max character length validation): fix form max character validation

Fix form max character length validation
Add maxLength attribute to html text area tag
Jira Ticket: https://pins-ds.atlassian.net/jira/software/c/projects/ASB/boards/166?modal=detail&selectedIssue=ASB-448
You can use a character count website like: https://wordcounter.net/
Local Project URL: http://localhost:9004/register/myself/check-answers
I would like to have someone pulling it and checking if it works for you as well.
Caveat: you won't see any errors if you input more than 65234 characters in the text area because its maxLength attribute is set to 65234.
To see the error and bypass it you need to inspect the element and manually increase it.

:tada: This PR is included in version 1.3.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


TAJO-1885: Simple query with projection should be supported.

+1 LGTM.
And it would be better if you commit this PR to 0.11.0 branch and master branch. :)


Remove outdated comments

Since variables n, d, p are no longer there, it seems this comments should be removed...

Ping @richsalz and @levitte

Thanks for the nudge; merged!


Upgrading to AlphaFS 2.1.3

Are you updating the code to the new AlphaFS 2.1.3

the website dosent list the new version yet but i did notice there was a new release on github....did some quick tests and found some methods that were supposed to be there missing so not sure whats going on ...but will update when i am sure of the version.

on second thought I dont see any major gains in updating to 2.1.3 so will wait for the next version.


Fix the data sync pull "path" value for transition_production

If this doesn't match its push counterpart, then the script won't
find the data in the S3 bucket.

https://trello.com/c/l7dNI7Th/1411-configure-the-postgres-data-sync-policy-via-hieradata-and-ensure-it-is-functioning

Thanks Dean. I added a link in the PR description.


Remove trailing hash value from demangled Rust symbol names

When demangling Rust symbol names, binutils addr2line removes the trailing hash value. The rustc_demangle crate allows Demangle to be formatted with or without the hash. I think this crate should match the behavior of binutils.
From rustc-demangle:
// With hash
assert_eq!(format!("{}", demangle("_ZN3foo17h05af221e174051e9E")), "foo::h05af221e174051e9");
// Without hash
assert_eq!(format!("{:#}", demangle("_ZN3foo17h05af221e174051e9E")), "foo");

cc @fitzgen @philipc

Published 0.8.0. Had to be a major release because object is part of the public API.


Update Package.toml: Change Repo URL

Fix repo URL

Is it possible to get this merged? @giordano (sorry again if I'm tagging the wrong person, please feel free to point me to someone else!)


Add a python helper to download circleci artifacts

Status
Ready
Related Issues
(related to future PR on sane-pdf-reports where we will need to fetch the latest artifacts from circleci).
Description
This is a python helper utility script everyone can use for downloading CircleCI artifacts.

We will move to the specific directory.


mentions are not persisted when ENABLE_NOTIFY_MAIL=false

Gitea version (or commit ref): 1.14.0
Description
When ENABLE_NOTIFY_MAIL in app.ini is set to false, mentions in issues / pulls are not persisted (so not listed in the "Mentioning You" filter on the dashboard).

https://github.com/go-gitea/gitea/blob/ac88b0ee839bfbfae5759e211d0f9a69abe4d8f7/services/mailer/mail_comment.go#L21-L29
This logic does not belong in mail_comment.go

https://github.com/go-gitea/gitea/blob/ac88b0ee839bfbfae5759e211d0f9a69abe4d8f7/services/mailer/mail_comment.go#L21-L29
This logic does not belong in mail_comment.go

I'm working on a PR, combined with #14013.

I'm working on a PR, combined with #14013.

I would not be at all surprised if that had the same root cause

I would not be at all surprised if that had the same root cause


Add a way to control the treatment of the Http2 system frames in the …

…http2-to-http1 flow
Motivation:
Http2StreamFrameToHttpObjectCodec is used to convert objects in the pipeline from HTTP/2 classes to their HTTP1 counterparts so that the existing HTTP1 pipeline could be reused. There are some frames however that are not being converted. E.g. Http2ResetFrame or Http2WindowUpdateFrame. The existing behavior is for those frames to be sent through the channelRead() pipeline, which may be breaking the logic of pipelines designed to only support HTTP1 requests. This goal of this change is to allow clients to configure the treatment of the HTTP/2 "leftover" frames.
Modification:
A new processSystemFrame() method is added to the Http2StreamFrameToHttpObjectCodec class. The method is called for all Http2Frame objects that are not processed by the Codec itself.
The default implementation is to send the frame to the channelRead() pipeline.
Result:
A client now has a way to implement a custom way of handling HTTP/2 system frames.

Another variant of this change is submitted as #8889.

@mosesn @bryce-anderson @ejona86 @carl-mastrangelo @trustin PTAL and let me know which one of the PRs you prefer

@netty-bot test this please

I don't understand the point of this. If the default is to pass the frames further through the pipeline, then couldn't you use another handler to throw them away?
I thought Http2ResetFrame or Http2WindowUpdateFrame aren't read on child channels... but it looks like that changed since my initial PR. Http2WindowUpdateFrame is supposed to get converted into Netty's flow control; that's right, that's still a TODO for the API. Even if Http2WindowUpdateFrames remain on the child channel for advanced use-cases, they need to be user events because you need to receive them even with AUTO_READ==false.
Http2ResetFrame, was just going to be a user event. You can see it still says that in the documentation of Http2MultiplexCodec:

The child channel will be notified of user events that impact the stream, such as {@link Http2GoAwayFrame} and {@link Http2ResetFrame}, as soon as they occur.

@ejona86 I think this patch may still be useful as the Http2StreamFrameToHttpObjectCodec may also be used with the Http2FrameCodec only (so no multiplex codec at all). IMHO it's a bit "overkill" to add just another handler to the pipeline to filter out everything else.
How we we handle the other frames when the Http2MultiplexCodec is in place is a different discussion imho.
Let's close this anyway and continue in https://github.com/netty/netty/pull/8889 as this is the preferred implementation it seems


Improve documentation for custom cgroups (--cgroup-parent)

The --cgroup-parent option was proposed in https://github.com/docker/docker/issues/8551, and implemented in https://github.com/docker/docker/pull/11428 and (for docker build) in https://github.com/docker/docker/pull/11844
The documentation for this feature is currently very limited, and even missing in some places (for example, in the docker run command line reference). In addition, there's no example on how to use this feature, which might help users understand on how to use this.
Obviously, care should be taken to not to explain cgroups in itself, but I found a nice short walk-through here; http://linuxaria.com/article/introduction-to-cgroups-the-linux-conrol-group)

+1 an example for this would be very useful

@thaJeztah There are some related issues:
https://github.com/opencontainers/runc/issues/141
https://github.com/opencontainers/runc/pull/336
https://github.com/docker/docker/issues/16681
https://github.com/opencontainers/runc/pull/339
When we work them out, I can help improve the documentation. So #debs :)

@thaJeztah: This option is in-flux and currently being discussed for systemd cgroups. Meanwhile, here's something for cgroupfs.
cgroups subsystems are individually mounted under cgroup mount /sys/fs/cgroup. When cgroups are managed by cgroupfs, a cgroup is created per container for every subsystem, under the default parent cgroup, "docker". Typically hierarchy is /sys/fs/cgroup//docker/containerID, where subsystem list is derived from /proc/cgroups.
The --cgroup-parent option can be used to override the default parent cgroup. A container that is run with a specific non-default parent is nested under the parent's directory in the cgroup fs hierarchy.
Start container c1.
docker run -it busybox sh
This creates container with ID, c1ID under default parent "docker" and a dir such as /sys/fs/cgroup//docker/c1ID is created.
Start container c2 under the same cgroup as c1 above.
docker run --cgroup-parent=c1ID busybox sh
This creates c2's cgroups dir nested under c1 such as /sys/fs/cgroup//docker/c1ID/c2ID
Another way to confirm that the child is nested under the parent, is to peek into the container's /proc/self/cgroup file.

Thanks @anusha-ragunathan I'm good with waiting until things stabilize a bit; this may be a nice subject for a guest writer to write an article on, like we had for the storage driver section
/cc @moxiegirl (just thinking out loud here :smile:)

@thaJeztah Guest writer or internal --- yes,  I agree that the cgroups material is just not where it needs to be. @anusha-ragunathan gives us a starting point (thank you helpful smart person) of the technical details.  I"d like to go further:

why should I care about cgroups as a Docker user
what kinds of things do cgroups impact and how can I adjust or control tune them
if I had a problem in an application and that problem was related to cgroups what would I see
how do I troubleshoot or fix the problem.
are there alternatives or similar things to cgroups

Some of that I'm sure is in those tickets....

Yes, I think a section like that could be really helpful

@aditirajagopal  Here is a very interesting challenge.  WDYT?

@moxiegirl thank you! this sounds like a good place to start :smile: -- will follow-up on IRC!

@moxiegirl been trying to catch you on IRC - when is a good time to chat about this?

https://github.com/opencontainers/runc/pull/336 and https://github.com/docker/docker/pull/18612 implement the "--cgroup-parent" option implementation on systemd. Feature is ready for documentation.

@anusha-ragunathan I think I got that on #18379 wdyt?

@moxiegirl just saw the text you wrote for https://github.com/docker/docker/pull/18379, 😍. After that is completed/merged, perhaps we should consider moving it to a central location so that we can link to it from the docker daemon, docker run and docker create docs? For example, to here; https://docs.docker.com/engine/reference/run/#specifying-custom-cgroups, that would reduce duplication. wdyt?

@anusha-ragunathan do you mean in readhat 6?
in my 7u test, the results is not what you had descripted.

@thaJeztah I am more thinking we need a whole article just on cgroups --- rather than just adding to run or some other place.  Once we have a central article, we can link to the central article rather than attempting to coordinate the whole deal from various files.

@moxiegirl oh, that would be great. I referred to the section in "run reference", because we're describing how to constraint resources in a container. cgroups are part of that (and, in fact, some of the other options are actually just wrappers for setting cgroups)

@thaJeztah Yep, there is a whole thing around setting up the daemon and then overriding...it is really central and we are handling it poorly right now.  So much to do so little time. Some days...

Awesome!! Haha!

@aditirajagopal Are you still interested in doing this material?  The reason I ask is we have activity around this on another PR.  I know it is a big commitment so if you would like to take a pass and do something smaller, I can find you another.
What do you think?

@moxiegirl yeah I noticed. Maybe I can hold off working on this till it gets fleshed out more?

Hello Docker team,
This is also related to https://github.com/docker/docker/issues/16681, but since it's closed i report here.
Description of problem:
I am trying to use cgroup-parent option while creating some containers in order to limit resource usage for a group of containers (not a single one). I provide an existing cgroup (nested properly into kernel's subsystems i am interested on), but the result is to get a newly created cgroup prefixed with the name i provided as input (for cgroup-parent option)
docker version:
Client:
Version:      1.9.1
API version:  1.21
Go version:   go1.4.2
Git commit:   a34a1d5
Built:        Fri Nov 20 12:59:02 UTC 2015
OS/Arch:      linux/amd64
Server:
Version:      1.9.1
API version:  1.21
Go version:   go1.4.2
Git commit:   a34a1d5
Built:        Fri Nov 20 12:59:02 UTC 2015
OS/Arch:      linux/amd64
docker info:
Containers: 2
Images: 17
Server Version: 1.9.1
Storage Driver: aufs
Root Dir: /var/lib/docker/aufs
Backing Filesystem: extfs
Dirs: 21
Dirperm1 Supported: true
Execution Driver: native-0.2
Logging Driver: json-file
Kernel Version: 3.16.0-4-amd64
Operating System: Debian GNU/Linux 8 (jessie)
CPUs: 4
Total Memory: 494 MiB
Name: debian8-sandbox-1
ID: MONT:SGPY:7CEE:572J:TDLW:7E32:ARNI:ZVQL:4OR4:AKFI:J6EF:3OLV
uname -a:
Linux debian8-sandbox-1 3.16.0-4-amd64 #1 SMP Debian 3.16.7-ckt11-1+deb8u5 (2015-10-09) x86_64 GNU/Linux
Environment details (AWS, VirtualBox, physical, etc.): -
Steps to Reproduce:

Create an arbitrary cgroup like cluster-1. E.g.:

> lscgroup | grep cluster-1
cpuset:/cluster-1
cpu,cpuacct:/cluster-1
memory:/cluster-1
devices:/cluster-1
net_cls,net_prio:/cluster-1
blkio:/cluster-1

Run a container inside cluster-1.

docker run --cgroup-parent cluster-1 -it --name redis-1 --hostname redis-1.internal redis:3.0

Actual Results:
The cgroup i intended to use is ignored. Instead a new cgroup is created under system.slice. The truncated output of systemd-cgls is:
├─system.slice
....
│ ├─cluster-1-1b1a1b5d22a60a012ee785565b99830abb9b2b1c840abc233a5f42b9b69de578.scope
│ │ └─4422 redis-server *:637
....

Expected Results:
I expected to group (>=1) containers into pre-existing cgroups. Maybe, i have misunderstood the semantics of cgroup-parent option, but i think that documentation is too terse at this point.
Additional info:
What is the proposed way of allocating resources for groups of containers? Should i reside on plain cgrules.conf functionality ?

@drmalex07 Unfortunately --cgroup-parent can't have the save semantic for both "fs cgroup" and "systemd cgroup", you can see discussion here https://github.com/opencontainers/runc/pull/336
That's something we need in docs.
And I think the latest upstream docker will satisfy your scenario, because it'll use "fs cgroup" as the default cgroup manager.

Hello @hqhq, Ok, i undestand now.
When you say "latest upstream" i you mean 1.9.1 or master?. My results above are tested on 1.9.1 and still docker doesn't seem to add (or reuse) a cgroupfs entry. Basically it always creates a scope (named as docker-<cid>.scope) under system.slice.
Let's assume i want to only manage cgroups by systemd means (i.e define slices and assign services to them). Is there a way to group running containers (wrapped as systemd service units) into a certain slice ? I think it would be sufficient to somehow support moving the docker-<cid>.scope (which should remain as a way to set per-container quotas) into a existing slice (e.g. memory/cluster-1.slice/docker-<cid>.scope)

@drmalex07 "latest upstream" I mean master.
If you want to mange cgroup by systemd, you can start daemon with:
docker daemon --exec-opt native.cgroupdriver=systemd --cgroup-parent system.slice
Then all containers will have a default parent "system.slice", you can also change it when running a container with:
docker run -ti --cgroup-parent my.slice ubuntu bash

@hqhq
That seems to work fine! The 1.10.0-dev daemon (from master.dockerproject.org) correctly places container scopes inside given slices.
Do we know when version 1.10 will be released and when will be available via apt.dockerproject.org repositories?

@thaJeztah This should be done. And I think should be done by Docker Inc.

Hey Guys, @moxiegirl @anusha-ragunathan @aditirajagopal @hqhq ,
is there any news about this years after? The documentation is still very short.
Can I use this:

Start container c1.
docker run -it busybox sh
This creates container with ID, c1ID under default parent "docker" and a dir such as /sys/fs/cgroup//docker/c1ID is created.
Start container c2 under the same cgroup as c1 above.
docker run --cgroup-parent=c1ID busybox sh
This creates c2's cgroups dir nested under c1 such as /sys/fs/cgroup//docker/c1ID/c2ID

to set a general limits for all containers involved?
And can I use it in some way in a compose file?


Implement base ui elements from our design.

We should have designs to work with on Thursday.

Stuff that should be useful
https://github.com/airbnb/react-native-maps
https://github.com/jaysoo/react-native-parallax-scroll-view
https://github.com/react-native-training/react-native-elements (pretty components)
https://github.com/jayesbe/react-native-cacheable-image (automatically cache images from a url)
And a bunch more here
http://www.awesome-react-native.com/#components

We need to build

The Gallery page including a grid of images -> clicking an image launches a new mural detail view
The Map page rendering a map with some testing markers
The Contact page with accordion style list


Clarify command results and break out into clearer list

I noticed on my version caff 15 2 actually does 2 hours and 15 minutes. So I'm tweaking the docs to reflect the hour / minute swap.

Closing this in favor of fixing the bug so that it actually works the way initially described.


Updated property 'isOpen' in popover.directive.ts

Updated property 'isOpen' into 'show' and 'hide' methods. Through this is possible to use 'toggle' method together with 'show' and 'hide' methods.
Example: if you want to toggle popover by using a button and close it also by click another button inside the popover.

Codecov Report

Merging #1765 into development will increase coverage by 0.05%.
The diff coverage is 100%.

@@               Coverage Diff               @@
##           development    #1765      +/-   ##
===============================================
+ Coverage        85.41%   85.47%   +0.05%
===============================================
  Files               85       85
  Lines             2290     2292       +2
  Branches           300      300
===============================================
+ Hits              1956     1959       +3
+ Misses             235      234       -1
  Partials            99       99

Impacted Files
Coverage Δ

src/popover/popover.directive.ts
100% <100%> (+2.7%)
:white_check_mark:

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 4a7719b...b69505e. Read the comment docs.

@valorkin it's possible to merge this? Thanks! 😄

thanks for help! ;)


Test issue with test_quadpack and libm.so as a linker script

On Fedora Rawhide (F24):
ERROR: test_typical (test_quadpack.TestCtypesQuad)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib64/python2.7/site-packages/numpy/testing/decorators.py", line 146, in skipper_func
    return f(*args, **kwargs)
  File "/builddir/build/BUILDROOT/scipy-0.16.0-1.fc24.x86_64/usr/lib64/python2.7/site-packages/scipy/integrate/tests/test_quadpack.py", line 42, in setUp
    self.lib = ctypes.CDLL(file)
  File "/usr/lib64/python2.7/ctypes/__init__.py", line 360, in __init__
    self._handle = _dlopen(self._name, mode)
OSError: /lib64/libm.so: invalid ELF header

This is because libm.so is a GNU ld script:
# cat /usr/lib64/libm.so
/* GNU ld script
*/
OUTPUT_FORMAT(elf64-x86-64)
GROUP ( /lib64/libm.so.6  AS_NEEDED ( /lib64/libmvec.so.1 ) )

Some background: http://www.gossamer-threads.com/lists/python/dev/941581

Thanks @opoplawski. Looks like it's just the test that needs a tweak. I'll send a fix for simply catching OSError.


Create window with physical dimensions

With the change to LogicalSize there's no way to create a window with desired physical dimensions without checking DPI and resizing after it's created.

This is related to winit, not glutin. Feel free to re-report there.
This is discussed here: https://github.com/tomaka/winit/issues/591#issuecomment-403113701

My bad, didn't realize WindowBuilder was a winit re-export.


NSCalendar identifiers and components updated for iOS8

Was getting compiler warnings for NSCalendar related identifiers and components in iOS8

Can we get this pulled?

:+1:

Hey all,
As part of a large refactor, this code has moved to our iOS SDK at https://github.com/stripe/stripe-ios, which now has a class called STPPaymentCardTextField with similar behavior to PTKView. That class already uses updated NSCalendar constants, so I'm going to close this PR in favor of that. I'd also encourage you to migrate your apps to use our SDK instead of this library (it should take just a few minutes, and we've written a brief guide at https://github.com/stripe/paymentkit#migration). Of course, if that doesn't work for you, or you have any trouble migrating, reply here and I'll be happy to help. Thank you for opening this PR, though, and I'm sorry we won't be able to get it merged.
Cheers!


[1.41.x backport] KOGITO-9463: Create a new guide describing the Job Services Quarkus Extensions (#408)

…

KOGITO-9463: Create a new guide describing the Job Services Quarkus Extensions

Code review suggestions I

Code review suggestions II

Code review suggestions III

Code review suggestions IV

Code review suggestions V

JIRA:

Description:

Please make sure that your PR meets the following requirements:

[X] You have read the contributions doc
[X] Pull Request title is properly formatted: KOGITO-XYZ Subject
[X] Pull Request title contains the target branch if not targeting main: [0.9.x] KOGITO-XYZ Subject
[X] The nav.adoc file has a link to this guide in the proper category
[X] The index.adoc file has a card to this guide in the proper category, with a meaningful description

How to backport a pull request to a different branch?

In order to automatically create a backporting pull request please add one or more labels having the following format backport-<branch-name>, where <branch-name> is the name of the branch where the pull request must be backported to (e.g., backport-7.67.x to backport the original PR to the 7.67.x branch).

NOTE: backporting is an action aiming to move a change (usually a commit) from a branch (usually the main one) to another one, which is generally referring to a still maintained release branch. Keeping it simple: it is about to move a specific change or a set of them from one branch to another.

Once the original pull request is successfully merged, the automated action will create one backporting pull request per each label (with the previous format) that has been added.
If something goes wrong, the author will be notified and at this point a manual backporting is needed.

NOTE: this automated backporting is triggered whenever a pull request on main branch is labeled or closed, but both conditions must be satisfied to get the new PR created.

@radtriste @ricardozanini Guys, this is the backport of the jobs-service documentation in the 1.41.x branch, it was my fault to not mark it to be backported  at the time we merged the documentation in master :angry:.
the peer backport into 1.40.x is here https://github.com/kiegroup/kogito-docs/pull/444
the peer backport into 1.40.x-prod is here https://github.com/kiegroup/kogito-docs/pull/445
the peer backport into 1.42.x is here https://github.com/kiegroup/kogito-docs/pull/446

Guys @radtriste @ricardozanini , I'll merge this PR to be sure we don't forget it and fall into the same situation again. B
Note that:

I'll be on PTO for some weeks.
It is a backport that is simply adding content that was already reviewed in the original PR, so we don't need for more greens.


update package to include babel-polyfill as dependency

I just realised that the es5 compatibility that I added a few months ago is broken for anyone who does not have babel-polyfill in their package.
This is a simple fix to remedy that problem. Sorry for the silly mistake.

I appreciate the PR! But I'm not super enthusiastic about including those deps in my projects, since I don't use es5. Is there any way we can do this differently?

I completely understand. I was kind of surprised that babel expects such dependency after already transpiling the code - seems poor practice. I did a bit of research on how to avoid it and could not find anything obvious, but let me have another look. Perhaps now node 6 is LTS we could just increase es5 to es6 which I don’t think would require the polyfill for promises, etc.


successfully rendered qusetions from db to html

Added another path that displays the questions asked that are stored in db

Amazing job!
from polls.models import Question

q = Question.objects.create(question_text="What is the quetion?", pub_date=datetime.now())

The requirements.txt is missing a few packages. Try repopulating it and updating the file


Fix typo in config file

Hello Marcel,
This fixes a typo in the config file.
Thanks,
Nicolas

Hello Marcel,
Not sure why some of the tests didn't pass, but you can safely merge this into master.
Thanks,
Nicolas


Autosave for just .excalidraw files

Hi!
This extension is litreally a life saver, thanks so much!
Maybe, a suggestion, a way to toggle on autosave just for .excalidraw files? Like, when you draw a new component, resize something, etc, it auto saves.
Thanks!

^ Also, please bump the excalidraw package to 0.7.0

I think you can get this feature by using the files.autoSave setting. Mine is set to: onFocusChange.

Would this work for just 1 type of file though?

Sadly no :/ -> This setting does not support per-language configuration.

Sad :(
Do you know if it is possible to write autosave into the extension just for the .excalidraw files?

Yes it is pretty easy to achieve. I had it in a previous version, but it did feel like an hack (the dirty indicator was blinking on file change), so I removed it.

Yes it is pretty easy to achieve. I had it in a previous version, but it did feel like an hack (the dirty indicator was blinking on file change), so I removed it.

Oh, I think maybe make it configurable, would be so helpful. Maybe also open an issue on microsoft/vscode asking about how you do it? (Removing the top left unsaved files count going from 1 to 0)

I think the issue could be resolved by directly writing to the file instead of creating a workspace edit.

I think the issue can be resolved by directly writing to the file instead of creating a workspace edit.

Yes, that'd work!

fixed in 8110d4f3908bd9a29d8c7ae7de109b6f05150866


mod_proxy_html filter's ProxyHTMLURLMap environment variables interpolation caused startup warnings

mod_proxy_html filter's ProxyHTMLURLMap environment variables interpolation causes startup warnings.
It used v2.2 ${var} notation style to reference environment variables instead of v2.4 notation style %{ENV:var} (like mod_rewrite syntax)
So it correctly caused startup warnings like "AH00111: Config variable ${var} is not defined" because, under v2.4,
the ${var} syntax is reserved for variables defined by a DEFINE directive or for environment variables existing before the server was started.
See:

https://httpd.apache.org/docs/2.4/en/configuring.html
https://bz.apache.org/bugzilla/show_bug.cgi?id=58467

Before:
RewriteRule ^/path/(foo|bar)/(.*)$ - [E=mypath:$1]
<LocationMatch "/path/(foo|bar)/.*">
	SetOutputFilter proxy-html
	ProxyHTMLInterp On
	ProxyHTMLURLMap ^/a/path/to/change/(.*)$ /newpath/${mypath}/images/$1 ecRiLV
</LocationMatch>

After:
RewriteRule ^/path/(foo|bar)/(.*)$ - [E=mypath:$1]
<LocationMatch "/path/(foo|bar)/.*">
	SetOutputFilter proxy-html
	ProxyHTMLInterp On
	ProxyHTMLURLMap ^/a/path/to/change/(.*)$ /newpath/%{ENV:mypath}/images/$1 ecRiLV
</LocationMatch>

This might be a breaking change for people who used to live with the startup warning.
Thank you for considering the change.

It would be nice to tolerate the old syntax so it can be backported to 2.4.x
An alternate way to do this would be to allow the second arg to be an an actual expression. Other modules do this cautiously by looking for an expr= prefix on the argument.

I totally agree with you regarding the backport to 2.4, but I guess that requires changes to core.c to not complain if this directive uses it wrongly.
https://github.com/apache/httpd/blob/382aba75811fc71c1cf91afe992383413de3b7eb/server/core.c#LL1417C1-L1425C18
Unfortunately I am by far not experienced enough to implement this 😔
Regarding the expr: There is a forth argument to ProxyHTMLURLMap: cond

A cond is evaluated by the Expression Parser. In addition, the simpler syntax of conditions in mod_proxy_html 3.x for HTTPD 2.0 and 2.2 is also supported.

With this expr the map can be enabled or disabled. But inside the from-pattern or to-pattern currently no expr can be used to return a value.

With my additional commit the old syntax should still work. (But will continue to emit a startup warning, just as before)

Is there anything else I can do to get this merged? Thx.

@covener What do I need to do to get this merged? Thank you!


cmake

cc @neonichu @abertelrud @compnerd
this is what I managed to pull together to get a cmake build, imitating what we do in SwiftPM's bootstrap. I am not 100% sure how CI for TSC is managed so not sure if we need to change something there to invoke this ala bootstrap

CI for TSC is basically identical to SwiftPM, using the same build-script presets.
I am not sure how the build-script helper stuff is used,  probably a question for @shamishal.

@shahmishal can you provide info on how TSC is built exactly in toolchain CI, as this change could potentially impact that.


rustls feature

Hi!
Could you add a rustls feature to use rustls for outgoing http requests?

nevermind. just saw that this is up to the user.


Add Show instances, printCSSLength and printFontSize

With this PR every attribute type now has:

An Eq instance
A Show instance
A printX function that produces what is needed by SVG

The documentation for Show says "While not required, it is recommended that for any expression x, the string show x be executable PureScript code which evaluates to the same value as the expression x." This is true for all types that have Show instances except CSSLength and FontSize. Fixing CSSLength and FontSize will be a (minor) breaking change.
This fixes #22 and I'll add a separate issue to track changes to CSSLength and FontSize.

I'm not sure we need this comment below to reappear multiple times.
{- The documentation for Show says "While not required, it is recommended that
for any expression x, the string show x be executable PureScript code which
evaluates to the same value as the expression x." The instance of Show above
predates the use of printFontSize and does not follow that recommendation. This
may be changed in a future version and `show` when generating SVG is
depreciated. Use printFontSize instead.
-}

Rather, I think it should be stated more generally in the Attributes.purs file as a module documentation. Something like

Each attribute has two functions: show from the type class Show
and printX where X corresponds to the type. When printing the value
for SVG attributes, use the printX function. For example
... example code block...
When debugging, use either show or printX.

Otherwise, this looks good to me!

Can you add the module documentation comment? Then I'll merge this PR.

Sorry about the passive-aggressive CHANGELOG commit. It suddenly occurred to me and I didn't see that you'd already commented.
I've put that Attributes level comment in but I've left smaller comments on CSSLength and FontSize to call out that they are different and are likely to change. Turns out similar comments on MarkerUnit and Orient are also required.

Thanks!


passport representatives

The passport representatives s_1,s_2,s_3 of https://beta.lmfdb.org/Belyi/6T3/[6%2C2%2C2]/6/222/2211/g0/a
do not satisfy s_3s_2s_1=1. We have s_2s_3s_1 =1 but the function BelyiMap([s_1,s_3,s_2]) produces the error that it has failed a sanity check, but (s_2s_3s_1)^-1 = 1 and BelyiMap([s_1^-1, s_2^-1, s_3^-1]) works fine.

Please help:

sigma := [Sym(6) | (1,2,3,4,5,6), (1,4)(2,3)(5,6), (1,5)(2,4)];
sigma[3]*sigma[2]*sigma[1];
Id($)

Seems OK to me.

There is an issue with a different triple in the passport.

sigma := [Sym(6) | (1, 2)(4, 5),
(1, 4)(2, 6)(3, 5),
(1, 6, 2, 4, 3, 5)
];
sigma[3]sigma[2]sigma[1];
Id($)
BelyiMap(sigma);
Chebyshev spherical Belyi map
Local Sanity Check Failed for p=9721:
sigma =
[
(1, 2)(4, 5),
(1, 4)(2, 6)(3, 5),
(1, 6, 2, 4, 3, 5)
].
supp(phi) =
[
Place at (0 : 1),
Place at (7086 : 1),
Place at (2635 : 1),
Place at (9720 : 1),
Place at (4861 : 1),
Place at (4860 : 1),
Place at (1 : 1)
]
[ 2, 2, 2, -1, -2, -2, -1 ].
supp(phi-1) =
[
Place at (1 : 0),
Place at (9720 : 1),
Place at (4861 : 1),
Place at (4860 : 1),
Place at (1 : 1)
]
[ 6, -1, -2, -2, -1 ].
Curve over Rational Field defined by
0
(16x^6 - 24x^4 + 9x^2)/(16x^6 - 24x^4 + 9x^2 - 1)
Runtime error: Failed sanity check, so the Belyi map is wrong.  This shouldn't
happen.

The sanity check fails but not because the map is wrong: it looks like it wants the ramification at a different place, so the usual S_3 issue.

Yeah it looks like I implemented the local version of the sanity check
without a lax (up to S_3 action) option. I see 3 options:

redo the hardcoded spherical map in nonhyperbolic.m to match the triple.
add ability to test lax isomorphism in sanity_check_local.m
for spherical maps it should be OK to use "regular" (non-local) sanity
check which has optional lax := true argument.

Mike
On Tue, Apr 21, 2020 at 1:15 PM John Voight<EMAIL_ADDRESS>wrote:

There is an issue with a different triple in the passport.
sigma := [Sym(6) | (1, 2)(4, 5),
(1, 4)(2, 6)(3, 5),
(1, 6, 2, 4, 3, 5)
];
sigma[3]
sigma[2]sigma[1]; Id($) BelyiMap(sigma); Chebyshev spherical Belyi map
Local Sanity Check Failed for p=9721: sigma = [ (1, 2)(4, 5), (1, 4)(2,
6)(3, 5), (1, 6, 2, 4, 3, 5) ]. supp(phi) = [ Place at (0 : 1), Place at
(7086 : 1), Place at (2635 : 1), Place at (9720 : 1), Place at (4861 : 1),
Place at (4860 : 1), Place at (1 : 1) ] [ 2, 2, 2, -1, -2, -2, -1 ].
supp(phi-1) = [ Place at (1 : 0), Place at (9720 : 1), Place at (4861 : 1),
Place at (4860 : 1), Place at (1 : 1) ] [ 6, -1, -2, -2, -1 ]. Curve over
Rational Field defined by 0 (16x^6 - 24x^4 + 9x^2)/(16x^6 - 24x^4 + 9x^2

Runtime error: Failed sanity check, so the Belyi map is wrong. This
shouldn't
happen.
The sanity check fails but not because the map is wrong: it looks like it
wants the ramification at a different place, so the usual S_3 issue.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/michaelmusty/BelyiDB/issues/13#issuecomment-617295461,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABZHRU7FJTCUXZCOYYSR6GDRNXIBZANCNFSM4MMX6VVQ
.

Yes sorry I meant a different triple in the passport. The examples I came across where sigma[3]*sigma[2]*sigma[1] is not the identity but BelyiMap() works are:
s1:=[ Sym(3) |
(1, 3, 2),
(2, 3),
(1, 2)
];

&Reverse(s1);
(1, 2, 3)
BelyiMap(s1);
Curve over Rational Field defined by
0
2/(4x^3 - 3x + 1)
s2:=[ Sym(6) |
(1, 3, 2)(4, 6, 5),
(1, 4)(2, 6)(3, 5),
(1, 5)(2, 4)(3, 6)
];
&Reverse(s2);
(1, 2, 3)(4, 5, 6)
BelyiMap(s2);
Curve over Rational Field defined by
0
-4x^3/(x^6 - 2x^3 + 1)
s3:=[ Sym(6) |
(1, 6, 2, 4, 3, 5),
(1, 4)(2, 6)(3, 5),
(1, 2)(4, 5)
];
&Reverse(s3);
(1, 2, 3)(4, 5, 6)
BelyiMap(s3);
Curve over Rational Field defined by
0
-1/(16x^6 - 24x^4 + 9x^2 - 1)
s4:=[ Sym(12) |
(1, 9, 2, 7, 3, 8)(4, 12, 5, 10, 6, 11),
(1, 4)(2, 6)(3, 5)(7, 10)(8, 12)(9, 11),
(1, 11)(2, 10)(3, 12)(4, 8)(5, 7)(6, 9)
];
&Reverse(s4);
(1, 2, 3)(4, 5, 6)(7, 8, 9)(10, 11, 12)
BelyiMap(s4);
Curve over Rational Field defined by
0
-4x^6/(x^12 - 2*x^6 + 1)


export_bioimageio() generates a default dependency file not required by bioimage.io

The default behavior of export_bioimageio() is to generate an environment.yaml file, documenting the environment in which the function is executed. While this feature could prove beneficial to certain users, I believe it should be an optional aspect, especially considering that bioimage.io does not mandate it for TensorFlow models.
Currently, a workaround for me involves specifying "dependencies": None in the overwrite_spec_kwargs dictionary. However, the existence of a default config file led me to take it for granted initially, resulting in unwanted reuploads and DOI records on Zenodo, and unnecessary installation of environments for bioimage.io's CI on GitHub. Additionally, even if an update is made without the environment.yaml file, the file persists and must be manually deleted.
To streamline the export process and enhance clarity for new users, I suggest adding a parameter, such as generate_default_deps=False, allowing users to opt out of this feature by default while retaining the option for those who find it valuable.
PS: A bug in generated environment.yaml file has been reported into #252

Let's discuss this in #254

Fixed by #254


Working with tables: Split and freeze

Hello everyone,
I'm currently working with the version 0.9.0-RC1 of the odfdom-java artifact (still bound to Java 1.8, so I can't use the  1.0.0-BETA1).
In OpenOffice you'll find "Split" and "Freeze" in the "Window" menu.
I went up and down the API (OdfTable, OdfTableColumn, OdfTableRow, OdfTableCell), but couldn't find any method related to that. I also created a speadsheet in OpenOffice, loaded it via the ODF Toolkit and analysed all the styles, but also couldn't find any information or clue there.
Is this a bug? A missing feature? Not in the standard of the ODS format, but in some unstandardised fields only OpenOffice knows? 🤔 Or is the problem in the front of screen? 😏

Hello Gerrit,
I was not sure myself, therefore I tested it.
Let's start making sure we talk about the same things:

The document format ODF, which contains the to bytes "shock-frozen"
information of your run-time ODF application.
A run-time ODF application with WYSIWG editor. ODF has its origin
with StarOffice, where *OpenOffice *was the open-source branch. *LibreOffice
*forked before Oracle donated the OpenOffice sources to Apache. After
IBM left the Apache OpenOffice developers it is kind of quiet there AFAIK.
The ODF Toolkit contains ODFDOM, which is an ODF run-time model but
without WYSIWYG.

Furthermore, there are three kind ODF features:

Those being part of the ODF standard, you may find the latest ODF 1.3
XML feature described by:

http://docs.oasis-open.org/office/OpenDocument/v1.3/OpenDocument-v1.3-part3-schema.html
and
https://tdf.github.io/odftoolkit/docs/odf1.3/OpenDocument-v1.3-cs01-schema-rng.html
2. Those being new and currently evaluated by an ODF application,
already serialized, but not yet defined by OASIS. The ODF app named them as
"extended" standard. In general, those are already in the issue tracker of
OASIS, on its way to standardization.
3. Those being saved in the settings.xml not being part of the standard
and not (yet ?) on the list.
Let's continue below...
Am Sa., 27. Juni 2020 um 11:42 Uhr schrieb Gerrit Hohl <
<EMAIL_ADDRESS>
Hello everyone,
I'm currently working with the version 0.9.0-RC1 of the odfdom-java
artifact.
In OpenOffice you'll find "Split" and "Freeze" in the "Window" menu.

I found a description for your WYSIWYG feature:
https://wiki.openoffice.org/wiki/Documentation/OOoAuthors_User_Manual/Getting_Started/Freezing_rows_and_columns
As I installed today myself a LibreOffice 7.0 Beta to test ODF 1.3, I found
your desired choices in LO7 when selecting, a row and using the context
menu.
I added this feature saved and reopened the document and the feature still
existed in the ODS. Therefore I created a simple ODS and saved the file as
a second file when I enabled the freeze feature (find WithoutFreeze.ods and
WithFreeze.ods attached).
The first *2 rows are being freezed *and are not moving when scrolling down.
I opened and indented the XML using JEdit (archive & XML plugin) and
compared them.
Your feature is in the third ODF feature XML category:
In the settings.xml I found the following change from 0 to 2:
<config:config-item config:name="VerticalSplitPosition" config:type="int">
2</config:config-item>
I went up and down the API (OdfTable, OdfTableColumn, OdfTableRow,

OdfTableCell), but couldn't find any method related to that. I also
created a speadsheet in OpenOffice, loaded it via the ODF Toolkit and
analysed all the styles, but also couldn't find any information or clue
there.

ODFDOM is focusing on features from the OASIS standard.
But you might add your feature on top of the DOM of the settings.xml file.

Is this a bug? A missing feature? Not in the standard of the ODS format,
but in some unstandardised fields only OpenOffice knows? 🤔 Or is the
problem in the front of screen? 😏

Unstandardised that only OpenOffice / LibreOffice knows.
Have a nice weekend, Gerrit!
Svante

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tdf/odftoolkit/issues/51, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGJNW7EXYZSLRRTFS3MKY3RYW5HZANCNFSM4OJ6ODJQ
.

So from my perspective, you have the following three options:

Assign this feature to yourself and implement it
Find someone who implements this issue for you & assigns this feature request to this someone
Close it ;-)

The full implementation would be consisting of:

Implement it for ODFDOM (via the settings.xml as an intermediate solution)
Gathering support at the LibreOffice Dev list to change the ODF XML from settings.xml to content.xml & styles.xml
Raise an enhancement request at LO (perhaps there is one already)
If there is an agreement write a JIRA issue for the OASIS TC
Finally, change the XML support from settings.xml to content.xml & styles.xml according to the final specification. (At that stage I would be much more interested to assist.)

It is quite some work and I am not overtaking the issue as I am riding against my own windmills ;-)
But if you like to do it, I would assist and guide you...

Seems GitHub is not supporting attached ODF files, therefore I had to zip them and add that instead:
FreezeTestDocuments.zip

Thank you very much, @svanteschubert , for your very long and in-detail reply. 😲
I hope it didn't take too much of your precious time. Especially as this line would have been enough information for me. 😏🙇

Unstandardised that only OpenOffice / LibreOffice knows.
In that case I simply will close the case as it is bearable for my case not having that feature.

Anyway thanks for your reply.

@gerrit-hohl
Don't worry, I was curious myself and I took the time to explain why and how I did this analysis so we are able to point to this issue, in case a similar question arises and let the questioner do this test. ;-)


Implement hash truncation for DSA

Extracted of https://github.com/google/end-to-end/issues/337. Similarly to recent ECDSA fix (https://github.com/google/end-to-end/pull/317) we need to truncate the hash to q length in DSA. Both FIPS 186-4 and https://tools.ietf.org/html/rfc4880#section-5.2.2 require that.

Nice. I'm going to recompile and build tonight, thanks for the quick fix


Watcher TestWatcherWaypoints fails on fast dispatch remote device

Issue appears to be due to #5604 (has a workaround) and #6159 (no workaround as of now)


updatebot issue. Can not import a multi module maven project.

Hello,
I have a problem importing an existing maven project. Import fails with the issue:
[ERROR] Failed to execute goal io.jenkins.updatebot:updatebot-maven-plugin:1.1.60:plugin (default-cli) on project platform-parent: Execution default-cli of goal io.jenkins.updatebot:updatebot-maven-plugin:1.1.60:plugin failed: Cannot invoke "org.apache.maven.model.Plugin.getVersion()" because "plugin" is null -> [Help 1]
I don't need the updatebot, and as far I can see in the older version it was the possibility to disable it.
Can you suggest me where to dig?

@jstrachan Can you please help me here? In new version there are no option to disable updatebot and in multi module maven project it's failing.

The option --no-maven-fix has now been added to jx project import


Establish trust between each Dataverse installation and the OpenDP App

What is the mechanism to establish trust (certificates) between each Dataverse installation and the OpenDP App? Currently, the Dataverse installation is determined based on the external tools siteUrl parameter which may be spoofed.

doi authority info: https://github.com/IQSS/dataverse/issues/6807#issuecomment-780756997
- code used:
- https://github.com/data-exp-lab/girder_ythub/blob/master/server/rest/ythub.py

replaced by #127


Approach of conversion for array in query to avoid breaking change

related to: https://github.com/vuejs/vue-router/issues/1232

Sorry if I wasn't clear enough in my requests. I updated the part, could you check to see if I'm missing something?

Could you please revert that last commit? It's easier to keep the example simple to make easy to match every one case. If you want to use a computed property in the example, please explain why

@posva Explained in the review of your commit.

@posva And I intended using watcher, which is really effortless in resolving the problem caused by such a breaking change during migration. And I still think it's the best practice.

@KingMario 👍 Updated some lines to polish it

@posva Thx

Still think adding watcher 一劳永逸
:(

@KingMario
You can send the pull request :)

The watcher idea relied on modifying the $route object which hours be
avoided because it's a snapshot. While it made it easier to adapt, it will
teach people a pattern we want to avoid
On Tue, 14 Mar 2017, 01:35 kazuya kawaguchi<EMAIL_ADDRESS>wrote:

@KingMario https://github.com/KingMario
You can send the pull request :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/vuejs/vuejs.org/pull/817#issuecomment-286286499, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAoicYm370mxc9jXEanVCrVLBOvVZmosks5rleDBgaJpZM4MZE8v
.


Double workspace trust prompt, after making a decision

Open a trusted workspace
File > Open File and pick a file outside the workspace
You'll get a dialog asking Do you trust the authors of these files?, choose Open in Restricted Mode

🐛 A new window comes up with a dialog asking pretty much the same in a different way:

this is being tracked in this issue https://github.com/microsoft/vscode/issues/155227 which while closed still needs work


Helper readme for running benchmarks

@mlange05 I sadly just noticed that numa  still attempts to use the old Propagator :(
Also, there's no simple work around to the fact that due to auto-tuning, the effect of numa will anyway be totally unpredictable, so...
The right and only solution is one MPI rank per socket. I vote to drop numa.

Merged.


Cannot save "Mail Setttings" when set the sending method to "SES(AWS)"

Environment
Host

item
version

OS

GROWI
4.2.5

node.js
14.15.3

npm
6.14.9

Using Docker
- (Using GROWI.cloud)

Using growi-docker-compose
- (Using GROWI.cloud)

(Accessing https://{GROWI_HOST}/admin helps you to fill in above versions)
Client

item
version

OS
Windows

browser
FireFox 85.0 (64 bit)

How to reproduce? (再現手順)

Prerequisites

mail settings are empty

Set the sending method to "SES (AWS)" in the "Mail Settings" section
Enter the "From address", "Access key ID", and "Secret access key", and click "Update" button
Reload the browser

What happens? (症状)

When I reload the browser, the "Access key ID" and "Secret access key" that I entered are empty
(After pressing the "Update" button, a toaster will appear to show that the update was successful)

What is the expected result? (期待される動作)

When I reload the browser, the "Access key ID" and "Secret access key" that I entered are remain
(Ofcourse after pressing the "Update" button, a toaster will appear to show that the update was successful)

Note

I was able to set the sending method to "SMTP" (It was able to send mail)
In version 4.1.13, I was able to set SES(AWS)

Also, after setting SES(AWS) in 4.1.13, I changed to 4.2.5 and the setting was still entered (It was also able to send email)

I think the bug is that values entered in the mail settings are not saved in the DB

Close because the issue has been resolved by PR #3378.


Final Step [Apply Migrations] for Mac Instructions missing

The final step in this tutorial [Apply Migrations] says that migrations are not required for SQLite, but this just seems more lazy than true. I looked at the database with DB Browser for SQLite and it showed that the Title and Rating column did not have 'Not Null' constraint against them.
So I rebuilt the database with the following commands:

dotnet ef database drop
dotnet ef migrations remove
dotnet ef migrations add initial
dotnet ef database update

Afterwards the DB Browser for SQLite showed all the constraints/validation in this part of the tutorial.
This should be added to the VS code VS for Mac Tab to complete the tutorial.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 39b6d0f3-9bc7-d091-0d45-9fc1e2e7aac8
Version Independent ID: 20e7eb53-215b-09ab-205d-d20177221278
Content: Add validation to an ASP.NET Core Razor Page
Content Source: aspnetcore/tutorials/razor-pages/validation.md
Product: aspnet-core
Technology: aspnetcore-tutorials
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

@D-BowdreyR  thanks. I'll get this updated.
If you have more than one migration you'll need to delete the migrations folder.

@tdykstra  I like the  dotnet ef migrations remove step. We could add, if you have two migrations, run dotnet ef migrations remove twice. Alternatively, delete the migrations folder.

Afterwards the DB Browser for SQLite showed all the constraints/validation in this part of the tutorial.

@D-BowdreyR  was there any change in the char length like SQL?

@Rick-Anderson I haven't seen multiple remove commands in a row work, have you?  Deleting the Migrations folder always works.

Looks like this was fixed.


jetbrains-runner: init at 3.0.4

https://github.com/alex1701c/JetBrainsRunner
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 348382 run on x86_64-linux 1

1 package built:

jetbrains-runner


DCOS-40469: add status code to next event

BREAKING CHANGE: the output format of the next event changed. Instead of
request(...).subscribe({next(response) {...}}) you can now write
request(...).subscribe({next({ code, response }) {...}})

As discussed offline, let's please add message as well so that the interface is consistent across the abord

@nLight @Poltergeist fixed 👍


dragen typescript-expression tools used by the v4.0.3 dragen pipeline

Added a typescript expression list for dragen typescript expression tools,
Comprises functions similar to those found throughout the 3.9.3 dragen tools.
Also added in some miscellaneous js handling functions
cwl-ica ts logic
cwl-ica commands used.
Create typescript expressions templates
cwl-ica create-typescript-expression-from-template \
  --typescript-expression-name dragen-tools \
  --typescript-expression-version 4.0.3

...
typescript edited and then some time later
...
Validate typescript expressions

This can take some time

cwl-ica typescript-expression-validate \
  --typescript-expression-dir typescript-expressions/dragen-tools/4.0.3

Waiting for merge of #190 first before triggering sync command


Error messages from requesting api

What is it and who's it for?
Displaying informative error messages to people using the requesting service.
The requesting service API can respond with an error under several circumstances. At present we just tell the user that the request failed and ask then to try again.

The possible errors can be found here
Should we be displaying the specific error message returned from the API and, if so, are they descriptive enough or do we want to change the message?
For example, a 403 error which occurs when the provided API key or bearer token does not permit access to the resource method has the message 'Forbidden'.

Probably worth changing the messaging where we can to make them more descriptive.
Is there a list of messages that the errors will show, like the 'Forbidden' one? I couldn't see them in the link above.

https://en.wikipedia.org/wiki/List_of_HTTP_status_codes#4xx_client_errors

These are the errors you'd be most likely to see in normal use:

"You are at your account limit and you cannot request more items"
"Item $itemNumber is on hold for another reader"
"You cannot request $itemNumber"

@DominiqueMarshall  - can you look at the most commonly seen ones of these (cc @alexwlchan for current list)

Okay, these are the current messages you'd be likely to see:
Item a222wwjt is on hold for another reader
You are at your account limit and you cannot request more items
You cannot request a222wwjt
(There are other errors, but if we show them to users something has gone extremely wrong.)

I would consider editing the first one:
Item a222wwjt is on hold for another reader - should "reader" be "library member" seeing as we have more items than just books?

We currently use "Item is in use by another reader" when we show a note against an item that can't be requested. So if we change it, we should change how we refer to "readers" elsewhere in the journey.
My only other thoughts are just around the formality of the language use (e.g. cannot) - again, I'm not clear what our approach or style is on these.

So am I changing it, or leaving as-is?

Change please as per Dom's suggestion 'reader' > 'library member'
Formality i've asked for a steer from Danny.

Also confirmed, can we change instances of 'cannot' in these to 'can't'
So these 3 errors become:
Item a222wwjt is on hold for another library member
You're at your account limit and you can't request more items
You can't request a222wwjt

Done, deployed.

Okay, so this is deployed in the API, but we still need to pull through the messages in the front end – which should fix https://github.com/wellcomecollection/wellcomecollection.org/issues/7451

This is the relevant component: https://github.com/wellcomecollection/wellcomecollection.org/blob/f075c49e435e4211a4be5a7dd842ed9d21edf886/catalogue/webapp/components/ItemRequestModal/ItemRequestModal.tsx#L387-L400


MSS and WMS crashes sometimes if no coastline is present in the selected area

Similar to the problem described here
https://stackoverflow.com/questions/59855017/determining-if-a-coastline-is-present-when-using-basemap
I think this is an issue with basemap and/or matplotlib, but we can handle it in the proposed manner with a simple try/except ourselves.

@jgrooss


Pixelated Text

System Details

OS: Windows 11
Thorium Version M117.0.5938.157

Problem
Whenever I open a new tab or stay on a tab, the text gets weird and becomes pixelated. I'm not sure if this is correlated but sometimes there are artifacts that show up on images and text too, second screenshot is an example of that.
Screenshots

Mark as duplicate


Fixed it with the request. We need to test it ;)

yeah my bad its an easy fix but until later i will fix it

You dont have the newest version ;( On my version it will function @hedges7

You dont have the newest version ;( On my version it will function @hedges7

Maybe not I guess. Go ahead and merge it if it works.

It should work XD


Allow proper use of etcd proxies.

Resolves #319.
This pull request adds a --no-discover flag that applies only when using the etcd backend. This flag prevents confd from discovering the whole cluster, and so ensures that it only uses the proxy.
This change may be incomplete: right now with the change in this form you need to provide the etcd authority in the form http://host:port when using the --no-discover flag. If that's problematic, we can copy go-etcd's Client.createHttpPath method, which can add a scheme to the front of the authority if needed. Thoughts?

I've actually just gone ahead and added some code that automatically prepends the scheme.

I think with etcdctl they call this --no-sync so it might make sense to rename it as such.

Seems reasonable to me.

Fixes #354 for me.

+1

This should be fixed with the move to the new etcd client libs. Please feel free to reopen if that is not the case.

This is not resolved. I'm still hitting that issue. Any URL options I need to pass? I don't see any relevant CLI flags. Using 0.11

The new etcd client lib is included in the 0.12 pre-releases. 0.11 still
uses the old Library. Can you please test this with one of the 0.12
releases ?
Jerome Gravel-Niquet<EMAIL_ADDRESS>schrieb am Di., 27. Sep.
2016, 22:57:

This is not resolved. I'm still hitting that issue. Any URL options I need
to pass? I don't see any relevant CLI flags. Using 0.11
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/kelseyhightower/confd/pull/329#issuecomment-249996378,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQFc84Rbce19twL4XdZBJPf1ehGtH2Fyks5quYNFgaJpZM4FxpsV
.

@HeavyHorst oh, you're right. I just tried it and it seems to work, mostly.
The initial lookup works, as well as the subsequent ones when watching. Which is what troubles me given my logs are being spammed with:
2016-09-27T18:09:29-04:00 jgn.local confd[78591]: ERROR client: etcd cluster is unavailable or misconfigured
2016-09-27T18:09:37-04:00 jgn.local confd[78591]: ERROR client: etcd cluster is unavailable or misconfigured
2016-09-27T18:09:45-04:00 jgn.local confd[78591]: ERROR client: etcd cluster is unavailable or misconfigured
2016-09-27T18:09:53-04:00 jgn.local confd[78591]: ERROR client: etcd cluster is unavailable or misconfigured
2016-09-27T18:10:01-04:00 jgn.local confd[78591]: ERROR client: etcd cluster is unavailable or misconfigured
2016-09-27T18:10:09-04:00 jgn.local confd[78591]: ERROR client: etcd cluster is unavailable or misconfigured
2016-09-27T18:10:17-04:00 jgn.local confd[78591]: ERROR client: etcd cluster is unavailable or misconfigured

It works though, which is conflicting with those messages.
It's worth noting I current use a 2 proxy setup, through https, I provide a client-ca-keys argument, a username and a password.

I noticed that the Error occurs every 8 seconds. The etcd-watcher uses http
long polling to track changes. Is it possible that one of your proxies had
a connection timeout of 8 seconds configured and closes the connection
because of that ?
Jerome Gravel-Niquet<EMAIL_ADDRESS>schrieb am Mi., 28. Sep.
2016, 00:23:

@HeavyHorst https://github.com/HeavyHorst oh, you're right. I just
tried it and it seems to work, mostly.
The initial lookup works, as well as the subsequent ones when watching.
Which is what troubles me given my logs are being spammed with:
2016-09-27T18:09:29-04:00 jgn.local confd[78591]: ERROR client: etcd cluster is unavailable or misconfigured
2016-09-27T18:09:37-04:00 jgn.local confd[78591]: ERROR client: etcd cluster is unavailable or misconfigured
2016-09-27T18:09:45-04:00 jgn.local confd[78591]: ERROR client: etcd cluster is unavailable or misconfigured
2016-09-27T18:09:53-04:00 jgn.local confd[78591]: ERROR client: etcd cluster is unavailable or misconfigured
2016-09-27T18:10:01-04:00 jgn.local confd[78591]: ERROR client: etcd cluster is unavailable or misconfigured
2016-09-27T18:10:09-04:00 jgn.local confd[78591]: ERROR client: etcd cluster is unavailable or misconfigured
2016-09-27T18:10:17-04:00 jgn.local confd[78591]: ERROR client: etcd cluster is unavailable or misconfigured
It works though, which is conflicting with those messages.
It's worth noting I current use a 2 proxy setup, through https, I provide
a client-ca-keys argument, a username and a password.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kelseyhightower/confd/pull/329#issuecomment-250016967,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQFc80n6zmpoeiib-IxSIrvYI_ZS1tknks5quZdvgaJpZM4FxpsV
.

I believe I'm hitting this issue: https://github.com/coreos/etcd/issues/3405
I've looked at the proxies' logs and all I see are those premature closed connections (at the same interval you mentioned):
2016-09-28 11:38:38.350727 I | proxy: client <IP_ADDRESS>:51677 closed request prematurely
2016-09-28 11:38:46.351517 I | proxy: client <IP_ADDRESS>:52204 closed request prematurely
2016-09-28 11:38:54.352242 I | proxy: client <IP_ADDRESS>:52714 closed request prematurely
2016-09-28 11:39:02.353108 I | proxy: client <IP_ADDRESS>:53205 closed request prematurely

Could be an issue with compose.io though (that's where my etcd is.)

Looks like this may be related too: https://github.com/coreos/etcd/issues/3894 (which is fixed by: https://github.com/coreos/etcd/pull/4254)
:thinkingface:

@jeromegn i can reproduce the error with a newly created compose.io etcd cluster.
i also tested this with github.com/HeavyHorst/remco, its working fine there.
I think it should be sufficient to bump the etcd client to the latest version.


fix: bump rust builder image to 1.78-bookworm

What kind of change does this PR introduce?
Bump rust builder image to 1.78-bookworm

cc @laktek


Make dynamic table name passing safe

Should be escaped and verified before the query runs

Fixed thanks to https://github.com/citizenwallet/indexer/pull/38. Shoutout to @pjol


cas-107 Delay proposal start by 1 hour in production env (UI ONLY)

cas-107

@dbslone updates done, this is ready for new review now


Revert "Import/export improvements (#573)"

This reverts commit 33c86d640078fde6c22fdc326d67f8239530c5b7.
It appears that this broke the loading of 3D models on the Quest

moving to draft while Andy decides if it's a real problem