[[CHORE]] Remove instructions on CLA

Now that the project has been successfully relicensed, we have no need
to collect signatures from contributors.

Coincidentally, CLAHub recently went down. (The good news is that I'd already downloaded JSHint's signatures.)

Coverage remained the same at 100.0% when pulling 863d221c395f061a670167092ca1fb039e177fb3 on jugglinmike:remove-clahub into 18f977e0eb699badc4463b441bafbe39edba47f1 on jshint:master.


Enhanced Privacy Policy page

Description
Enhanced the privacy policy page:
1.Increased the line height for better visual appeal
2. added margins wherever necessary
3. changed the list style type to decimal
4. added class names and applied css to lists
Fixes #777
Replace issue_no with the issue number which is fixed in this PR
Screenshots
before:

after:

Checklist

[x ] Tests have been added or updated to cover the changes
[x ] Documentation has been updated to reflect the changes
[x ] Code follows the established coding style guidelines
[x ] All tests are passing

@thestarsahil pls review and merge this PR


Add getOrCreateX methods to make nested field mutates more ergonomic

Consider the protos below:
message Outer {
  optional Nested nested = 1;
  ...
}

message Nested {
  optional int32 foo = 1;
  optional int32 bar = 2;
  optional AnotherMessage another_message = 3;
}

Given a reference to an instance of Outer, I want to mutate the value of an inner field. Ideally I'd write:
void populateBar(Outer outer) {
  outer.nested.bar = 1;
}

This works if outer.nested is already populated. Otherwise outer.nested returns a default (read only) instance, and the set fails with an UsupportedError. To do this safely w/o knowledge of whether the field is populated you have to do something like this (which gets even more cumbersome for deeper nesting):
void populateBar(Outer outer) {
  final nested = outer.has_nested ? outer.nested : new Nested();
  if (!outer.has_nested) outer.nested = nested;
  nested.bar = 1;
}

One way to make this easier would be to add another getter that, rather than returning a default instance, creates the field if it doesn't exist. Assuming the prefix getOrCreate we could do this:
void populateBar(Outer outer) {
  outer.getOrCreateNested.bar = 1;

  // And this works well for more deeply nested fields:
  outer.getOrCreateNested.getOrCreateAnotherMessage.baz = 2;
}

It would be even more nice to have .nested getter to return some wrapper over a default readonly value, which would set a 'nested' field on parent when modified

I think ensureX now addresses this. For example:
void populateBar(Outer outer) {
  outer.ensureNested().bar = 1;
}

I think we can close this?


Snapshot wip

Support for snapshot.mode=initial.
We don't use SnapshotSource as the APIs are same.
We use internal APIs for snapshot that is provide new OpId(-1,-,1, [], -1,-1,-1) as the from_op_id and the server will take a snapshot and stream the changes.
After snapshot, the connector will stop.
The connector needs to be started with Snapshot.mode=never for further streaming changes.

Tests seem to be broken. One of the tests goes into an infinite loop. I couldn't figure out which one. Can the test be fixed or disabled?


Wireguard interface not up

Hello everyone
I have a 5.10 kernel:
uname -a
Linux debian-1cpu-1gb-sg-sin1 5.10.0-0.bpo.7-amd64 #1 SMP Debian 5.10.40-1~bpo10+1 (2021-06-04) x86_64 GNU/Linux
The wireguard kernel module has been loaded:
modprobe wireguard
lsmod | grep wireguard

wireguard              94208  0
libchacha20poly1305    16384  1 wireguard
ip6_udp_tunnel         16384  1 wireguard
udp_tunnel             20480  1 wireguard
libblake2s             16384  1 wireguard
curve25519_x86_64      49152  1 wireguard
libcurve25519_generic    49152  2 curve25519_x86_64,wireguard

After starting wiretrustee, there is no wiretrustee0 interface:
systemctl status wiretrustee.service
● wiretrustee.service - Wiretrustee Service
   Loaded: loaded (/lib/systemd/system/wiretrustee.service; enabled; vendor preset: enabled)
   Active: active (running) since Mon 2021-07-05 14:41:31 UTC; 6min ago
 Main PID: 799 (wiretrustee)
    Tasks: 3 (limit: 1135)
   Memory: 6.7M
   CGroup: /system.slice/wiretrustee.service
           └─799 /usr/local/bin/wiretrustee up --config /etc/wiretrustee/config.json --log-level debug

ip a
1: lo: <LOOPBACK,UP,LOWER_UP> mtu 65536 qdisc noqueue state UNKNOWN group default qlen 1000
    link/loopback 00:00:00:00:00:00 brd 00:00:00:00:00:00
    inet <IP_ADDRESS>/8 scope host lo
       valid_lft forever preferred_lft forever
    inet6 ::1/128 scope host
       valid_lft forever preferred_lft forever
2: eth0: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc pfifo_fast state UP group default qlen 1000
    link/ether 46:d1:1a:9d:07:2e brd ff:ff:ff:ff:ff:ff
    inet xxx.xxx.xxx.xxx/22 brd xxx.xxx.xxx.xxx scope global dynamic eth0
       valid_lft 85710sec preferred_lft 85710sec
    inet6 fe80::44d1:1aff:fe9d:72e/64 scope link
       valid_lft forever preferred_lft forever

What did i do wrong? Any suggestions, thanks

Hey,
Did you call wiretrustee init command?
And could you please provide the content of /etc/wiretrustee/config.json ? Please omit the private key.

Hey,
Did you call wiretrustee init command?
And could you please provide the contents of /etc/wiretrustee/config.json ? Please omit the private key.

Sorry, it's my problem.
My firewall blocks the wiretrustee port
Your project is great!

I'm glad to hear that the problem has been solved
@xiya233
Could you please which port was blocked by your firewall? Signal service port?

@braginini
Yes, there may be other ports that are also blocked.
I configure the firewall to allow all traffic to connect:
firewall-cmd --set-default-zone=trusted
firewall-cmd --reload

Alright, thank you.
Just opening signal port should be enough.


Milestone 4: Add UI

Milestone 4: Add UI
In this milestone I :

Created App class with the following features

List all books
List all people
Create a student or teacher based on an option
Create a book.
Create a rental.
List all rentals for a given person id

Created a main function to instantiate the app class

Thanks for the review, @IngDavidTM 🙌


Move the build receipt to an own jar

The motivation behind this is that we want to be able to cache most
of the distribution build. The build receipt for the distribution will
include a timestamp. In order to make most of the Jars cacheable
we would include it only in the distribution and the wrapper but
not in the core jar. Then it is a runtime dependency for the tests,
since these often need to determine the Gradle version.
This way, the distribution build should only need to rebuild the
distribution and the wrapper jar and everything else could be from
the task output cache.

Replaced by #754


Change base image

Requirement - what kind of business use case are you trying to solve?
When an application container has a problem in production, it's often useful to be able to enter the container and execute a few commands, like curl, or to copy files from inside the container using commands like kubectl cp . Unfortunately, using a scratch image means that there's nothing in the container, breaking the two use cases above. Concretely, I'm debugging a problem with the badger storage and tried to get the files out of the container using kubectl cp.
While all those problems can be worked around by either building a custom image with another base image, or by using another image to mount the volumes we want to access, not having a proper base image slows down progress when trying to find the root cause of problems related to Jaeger.
Proposal - what do you suggest to solve the problem or improve the existing situation?
There are a few images that we could use. The image I have most experience with is Red Hat's UBI, which is used in quite a few operators, including OpenTelemetry's (minimal) and Jaeger's (regular). I know this is a secure and well-maintained image, but we should certainly consider other images as well. I'm open for suggestions here.
Using the ubi-minimal would add about 90MB to the image size, which isn't that much, especially considering that this is mostly relevant for the initial pull.

Adding 90Mb means a 6x increase in the current image size. I would rather publish another group of -dev images.

It really sounds dramatic when we talk about "6x" increase, but in absolute terms, 90MB isn't even a minute away from most domestic broadband connections, to be shared across all components.
Is there a concrete concern about the image size, other than the time it takes to download it?

Jaeger has often been praised for being lean, I think our original 10Mb images contributed to it (not they are 20Mb). I also think that having dev tools in the image is something that only a small % of users need, e.g. when setting up Jaeger for the first time, figuring out networking namespaces, etc.

"Lean" as in the container image size, or as in binary size? The container isn't placed as a whole in memory, just the binary. The process binary is exactly the same no matter the base image, and the RSS is virtually identical:
## scratch
$ ps -q 362569 uxwww
USER         PID %CPU %MEM    VSZ   RSS TTY      STAT START   TIME COMMAND
jpkroeh+  362569  0.5  0.1<PHONE_NUMBER>6 ?        Ssl  10:13   0:00 /go/bin/all-in-one-linux --sampling.strategies-file=/etc/jaeger/sampling_strategies.json

## ubi-minimal
$ ps -q 363519 uxwww
USER         PID %CPU %MEM    VSZ   RSS TTY      STAT START   TIME COMMAND
jpkroeh+  363519  1.2  0.1<PHONE_NUMBER>4 ?        Ssl  10:14   0:00 /go/bin/all-in-one-linux --sampling.strategies-file=/etc/jaeger/sampling_strategies.json

## ubi
$ ps -q 364489 uxwww
USER         PID %CPU %MEM    VSZ   RSS TTY      STAT START   TIME COMMAND
jpkroeh+  364489  1.0  0.1<PHONE_NUMBER>0 ?        Ssl  10:23   0:00 /go/bin/all-in-one-linux --sampling.strategies-file=/etc/jaeger/sampling_strategies.json

I uploaded the three images to my quay namespace, so that you can try it out by yourself :-)
quay.io/jpkroehling/all-in-one:ubi-regular
quay.io/jpkroehling/all-in-one:ubi
quay.io/jpkroehling/all-in-one:scratch
Looks like I was wrong about the image sizes. ubi-minimal would add about 34.7MB and ubi (regular) would add 73.4MB: https://quay.io/repository/jpkroehling/all-in-one?tab=tags

I also think that having dev tools in the image is something that only a small % of users need, e.g. when setting up Jaeger for the first time, figuring out networking namespaces, etc.

You are right that this is mostly useful when setting up first, but right now, we have a case where having those tools would have helped us diagnose a production problem faster and more easily. Without having those tools, we have to resort to either rebuilding a Jaeger image with a different base, or to mount the same badger volumes in another, more suitable image.
While we can ask people to change their Deployment objects (or Jaeger CR, if they are using the operator) to use their custom images, or a -dev image, those deployments are usually managed using automation tools, preventing most people from directly changing those objects in production.
In short: I don't really understand what do we gain by preventing people from entering the container when they are facing an issue. There's no hit in performance, there's no hit in memory usage, there's little to no hit in the disk space (especially when using a production setup, as jaeger-[collector|query|agent] would share the same base).

@yurishkuro, do you have any feedback on the definition of "lean", based on my comment above?

I'm still not clear why we can't publish both scratch and ubi-minimal. Can we not provide a global config option in the Operator to pick one vs. the other?
If we want a shell & basic tool set, why not use alpine?

We can certainly publish two sets of images, I just don't see the benefits in doing so vs. the maintenance work that it adds (CI, docs, users questions, ...). The only benefit of using scratch vs. ubi-minimal is that disk space per node (not container) would be about 35MB higher.
If we do decide to provide the two base images, the operator can certainly use a new flag to decide which image to use.
About Alpine: I have nothing against, and we can very well use it instead of ubi-minimal. I'm just not familiar enough with it to recommend it.

I'd suggest we switch from scratch to alpine, which is 7x smaller than ubi-minimal.

Sure, I'll try to get a PR ready soon for alpine, but I'm still wondering why is "7x smaller" (or rather, one seventh) even relevant when we are talking about such small figures...

@jpkrohling should this be closed? It seems #2545 made a switch to alpine even for non-debug images.


Task View No Longer works

Windows Task View no longer works.
I have re-enabled a few of the visuals that come with windows. When I try to use the Task View button, or press "Windows" + "Tab", explorer.exe crashes and restarts in an instant. Interestingly, I also see the volume slider pop up during this explorer restart
I have tried to delete the following entries from the registry, hoping that they would help:
"HKLM:\SOFTWARE\Policies\Microsoft\Windows\System" -Name "EnableActivityFeed"
"HKLM:\SOFTWARE\Policies\Microsoft\Windows\System" -Name "PublishUserActivities"
"HKLM:\SOFTWARE\Policies\Microsoft\Windows\System" -Name "UploadUserActivities"
I love the script!

Duplicate of #172.
Please run
Set-Service "CDPUserSvc" -StartupType Automatic

and reboot.


[macOS] Error: Protocol error (Page.captureScreenshot): Target closed.

Steps to reproduce
Tell us about your environment:

Puppeteer version: 1.4.0
Platform / OS version: macOS 10.13.4
URLs (if applicable): function screenshot
Node.js version: 8.X

What steps will reproduce the problem?
All time i received error Protocol error (Page.captureScreenshot): Target closed., try-catch is ignored and browser crash (??? why with try-catch?).
I try any solution, any variants (clip instead of fullPage, without clip and fullPage, set size of browser to 800x600, jpg format, png format, with headless, without headless, etc...). Is impossible for me get screenshot on mac. Every time crash only on mac (windows and linux work). What is correct code for mac?
        try {
            await page.screenshot({ fullPage:true, type:"png",  path: "./screenshot/test.png" });
        } catch (err) {
            console.log(err);
        }

What is the expected result?
Screenshot png in screenshot folder.
What happens instead?
Error: Protocol error (Page.captureScreenshot): Target closed.

@ptkdev this has something to do with the environment; the target closed error usually happens when there's lack of memory or some other constraint. You can try running browser with dumpio: true and check Chrome's error log - it will hint towards the real thing that happens in the browser.


Render some output file types

It is possible to download the output files, but some might be better off rendered in the page (like images) or at least viewable without downloading (currently I use send_file(..., as_attachment=True)).

Seems unless the mimetypes are right, "Content-Disposition: inline" is not enough.

This should be a matter of setting the right content-type in the presigned URL, using mimetypes.guess_type(Path.path), or using python-magic's magic.from_buffer() to set it on the object when uploading to S3.


chore: bdd-spring-1572285395 to 0.0.1

chore: Promote bdd-spring-1572285395 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

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


Generate samples as code as well as doc to validate compilation

Description
Add your description here!
Checklist
To ensure a quick review and merge, please ensure:

[ ] The PR has a understandable title and description explaining the why and what.
[ ] The PR is opened in draft if not ready for review yet.

If opened in draft, please allocate sufficient time (24 hours) after moving out of draft for review

[ ] The branch is recent enough to not have merge conflicts upon creation.

Ready to Land?

[ ] Build is completely green

Submissions with test failures require tracking issue and approval of a CODEOWNER

[ ] At least one +1 review by a CODEOWNER
[ ] All -1 reviews are confirmed resolved by the reviewer

Override/Marking reviews stale must be discussed with CODEOWNERS first

@jsquire when you reviewed, did you look at the preview insertion API? I think these need to be open long enough - and during typical business hours - for impacted owners to review.
For example, now the code generator is unneccessarliy adding new RequestContext() to every call for samples, thus making samples unnecessarily harder (and misleading) for devs to copy. My understanding is that we want samples to be as simple as possible, which is why we also don't show async in samples despite it being a best practice for network calls. Do you agree, @KrzysztofCwalina or @tg-msft?

Isn't this part of the change to make RequestContext required on bodyless endpoints so we can drop the confusing Value suffix on evolved conveniences?  I don't think the generator is adding it unnecessarily here, but it's admittedly a little hard to tell without any context in the PR description.  😄
It's also being rushed through for the next UX study so the short PR window is necessary, but any feedback is always welcome and we can address it in a follow-up PR.

It's also being rushed through for the next UX study so the short PR window is necessary, but any feedback is always welcome and we can address it in a follow-up PR.

@tg-msft certainly, and @m-nash and I discussed just that offline. In general, I think we need to refine the process of autorest.csharp changes a bit and started a Teams thread in our internal team channel with our team. Please have a look.


Demo Google Colab

Hey Billy, @billyblu2000
I figured I would make you a nice Google Colab to demo AccoMontage2.
Feel free to merge.
Thanks.
Alex

Hi @asigalov61 Alex,
Thank you so much for your efforts! These works are indeed helpful!
Best,
Billy


Removed the 'date' column from the date field table

Fixes issue #2361.

I just can't help but feel we're doing it wrong by using the text value. Anyone else? I don't really have anything to justify it other than a gut feeling :confused:

Looking at the PR, I assume that Symphony uses the text value exclusively? But why? There might be reasons for this. Maybe @nilshoerrmann knows more, because he has worked a lot on date and time stuff.
Indeed it feels a bit strange. Unfortunately I don't know anything about the benfits and shortcomings of DATETIME values in MySQL. Anyway this change should be discussed. There should only be one of these two columns left, I think everybody agrees on that.
Last but not least: AFAIK, there has not been a final decision on a possible upgrade path from Symphony 2 to Symphony 3. (Correct me if I am wrong.) As long as we have the chance to provide an upgrade, shouldn't structural changes to the database be replicated in the updater, too?

@michael-e
Using string was the only way to "remove" timezones...

AFAIK, there has not been a final decision on a possible upgrade path from Symphony 2 to Symphony 3.

See https://github.com/symphonycms/symphony-2/issues/2373

Merged as 1377797


Give template which gives TreeError

Hello !
This issue is created after this nodebb community topic :
https://community.nodebb.org/topic/15235
If we use bad {{{ if tutu }}} xxxxx {{{ end tutu }}} in a template, the build raises one or many Failed at parse::tree::tree, TreeError without giving the name of the problematic template. It can be long to track these errors with many template files...
[It could also be useful for some people (?) to make {{{ if tutu }}} xxxxx {{{ end tutu }}} usable like the {{{ end }}} version...]

I've opened #100 to cover allowing {{{ end tutu }}} etc

I've opened #100 to cover allowing {{{ end tutu }}} etc

Ok, thanks !

Ok, thanks !


The Confidence field is missing in WordLevelTimingResult class.

I want to get the results of word level. When I call the C# SDK API directly, the returned word results does not contains the Confidence field.

But the JSON results from the SpeechServiceResponse_JsonResult property contains the Confidence field.
var jsonResult = e.Result.Properties.GetProperty(PropertyId.SpeechServiceResponse_JsonResult);

So why is the Confidence field missing from the WordLevelTimingResult class? Can you help to add this field?

Closing since there is no reply.


prolly...so it wont break

as i understand this shall be executed by an administrator on the clients, but without root sudo, the commands are unsuccessful.
also on line 97, the ${uname}:${uname} makes sense on most single user systems, but on a working enviroment, where multiple people login on a PC via ldap, by default users are on in the group, named the same as the username.......so that breaks as well but did no change it...yet, for me ${uname}:sudo worked
(tried on ubuntu 20)

ill need to test this to make sure it works on all systems

looks fine, thanks!


run ./hack/update-codegen.sh in generate make target

Why are these changes needed?
generate target should also run ./hack/update-codegen.sh
Related issue number

Checks

[ ] I've made sure the tests are passing.
Testing Strategy

[ ] Unit tests
[X] Manual tests
[ ] This PR is not tested :(

@kevin85421 @astefanutti


Add search functionality to the Job History page

In FireCloud today, I rely on the search bar on the Monitor tab to filter down results.
For example, I was looking for an old job using cell-ranger in the past, and I was able to find the exact job quickly in FireCloud:

Trying to find the same job in the same workspace in Terra, I tried using the ctrl+f functionality to search for the same keyword, which didn't work:

I eventually find my job after lots of scrolling, but that was quite tedious because in this case its a fairly old job.
AC: It would be great to have a search bar or a filter bar or someway to easily to search for old jobs.

FireCloud tables had a filter bar that applied to any column, as @ruchim notes. Other users have asked for this on our data tables.
I think we need at least this single-searchbar functionality on all tables.

Closing as duplicate of #1443


Rename "Vertical Sharding"

The current sharding docs look like this:

Calling it "sharding" immediately makes your mind think horizontal shards, and "vertical" typically refers to instance sizing. That also makes you have to say "horizontal sharding", which probably makes people wonder if that is different than normal DB sharding. Eliminating vertical sharding in favor of something else would also let us just use "sharding" in a horizontal context.
Here are a few options to get the conversation started:

shard groups
data segmentation
keyspace separation
keyspace creation
keyspace segmentation
keyspace architecture

The wording could be better, I agree.
One thing to mention is that even though now we only do ''vertical splits", we plan to also do merges, and in general just table moves from one unsharded keyspace to another unsharded keyspace. At some point, we'll also support moving tables from sharded to sharded keyspaces. So the naming should reflect that, it can't be "splits".
So a few options to get the ball moving:

table(s) move
table(s) relocation
table(s) reorganization
keyspace shuffling
keyspace reorganization

I think these terms are ingrained in a lot of code and documentation. Instead, we should just provide good documentation what they mean.
I'll keep this bug open till it's documented.

Agree. I had multiple conversations with folks that didn't realize and/pr understood when I said sharding, meaning vertical and horizontal.

I've not used this feature (it sounds interesting) but to me the "vertical" sounds like "table relocation between keyspaces".  The table is not being re-organised (as that feels like something you do on the physical table itself).   If changing the documentation everywhere is complicated then just mentioning what this is in the main places where it is described may well be enough, then fix later.


Add Nethermind prune

Stop Nethermind
Restart with full prune
admin RPC kick off background prune with exit on completion
Restarts with memory prune

On the back burner for now. It may be desirable, but Nethermind is also changing state storage. If that lands by November/December we may be able to save ourselves some effort, and just rely on the client improvement,


Update wordpress-installation-freebsd12.sh

The commit looks sensible to me, but I'll have to check FBSD v12 compatibility first. If v12 doesn't have the packages required (ie php8.1), I'll just remove it from the list of supported FBSD versions.


[Bug]: Workflow elapsed_time value not calculated properly for RetryPolicy

Bug Description
Instead of calculating elapsed_time as a time since the last run it add current time to the previously recorded time.
Current version:
  delay = config.retry_policy.next(
      retry_start_at + time.time(), attempts, e
  )

Should be:
  delay = config.retry_policy.next(
      retry_start_at - time.time(), attempts, e
  )

Also retry_start_at must be updated on each retry.
https://github.com/run-llama/llama_index/blob/d48f073a4d5d131a79eee9d28a84fa715c5e6c2d/llama-index-core/llama_index/core/workflow/workflow.py#L236
Version
v0.12.1
Steps to Reproduce
Create custom RetryPolicy which operate on the elapsed_time.
Relevant Logs/Tracbacks
No response

I will be able to fix that, but there is currently no RetryPolicy implemented which utilizes that parameters.
Do you guys have any suggestion for the new type of the RetryPolicy which do some calculations based on the time  that passed since the last attempt? Any feature requests which you might remember where it can be helpful?
@logan-markewich


Write article about how to run FREME on a server

@PeterBourgonje  @jmschnei this is for the DKT workshop

done. right now it does not cover how to install FREME NER


[FLINK-1731][ml] Add KMeans clustering(Lloyd's algorithm)

This is a breakoff from https://github.com/apache/flink/pull/757 to add the lloyd's algorithm first.
I will follow this up with initialization schemes in the above linked PR.
To address a few comments from the previous PR:
We cannot use DataSet[LabeledVector] instead of DataSet[Seq[LabeledVector]] because the model here is of type Seq[LabeledVector] and the semantics of pipeline require as such.

/cc @skonto @thvasilo

@sachingoel0101 I am having a look.

@sachingoel0101 please add an example at least under flink examples.
Is the code documentation enough? @thvasilo ?

@skonto you're right that we should also add a corresponding kmeans documentation under docs/dev/libs/ml/kmeans.md and link it from the docs/dev/libs/ml/index.md file.

I'm not sure about adding examples under flink-examples project. None of the other ml algorithms are there either. Also, requires adding flink-ml as a dependency in the pom. I can however add a kmeans section under ML docs

We could think about adding examples to flink-ml itself. But this is not strictly necessary if the API and its usage is documented.

Ok ... there is one example.

Just fyi, that is not an example of the usage of machine learning library. It is just a standalone implementation of the linear regression model.

Correct, just saying we will need to create a set of examples for the ml library.

I have run the tests and verified the code, seems to work fine. I need to check the centers generated.

@sachingoel0101 could you update the PR so I can do a final review and request a merge?
@tillrohrmann could assist with the forwardedfields question?

@skonto I'm traveling right now and won't be able to push an update until
Monday/Tuesday.
On Feb 9, 2017 09:31, "Stavros Kontopoulos"<EMAIL_ADDRESS>wrote:

@sachingoel0101 https://github.com/sachingoel0101 could you update the
PR so I can do a final review and request a merge?
@tillrohrmann https://github.com/tillrohrmann could assist with the
forwardedfields question?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/flink/pull/3192#issuecomment-278541212, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AIdpFUGbVFcX21M_1AJF25G4gfabG5ioks5rao-1gaJpZM4LrFiq
.

@sachingoel0101 np as soon as you are ready let me know. Also @tillrohrmann made some comments to take into consideration.

Closing since flink-ml is effectively frozen.


Add leftwise flatMap operator for Decoded

This version of the operator was left out for some reason. This wasn't
intentional, so we're just correcting the oversight.
Fixes #308

:+1:


Finish 3.1. Enhanced User Access Management

We need to implement 3.1. Enhanced User Access Management feature for our application.

temporary  app url http://<IP_ADDRESS>:8000/

@engemamhosain Excellent job! Could you kindly incorporate documentation, including the code, into a new branch titled "Enhanced User Management"? The documentation should encompass your approach, the decisions you've made, and the steps taken to implement all the functionalities within this section.


Make history per day instead of accumulating

I don't really understand what the value is of the accumulating view. The number of screen time "per 7 days" doesn't really help me. I mean maybe daily averages would help.
But mostly, I would like to see a daily history of screen time. To see which days were worse and which were better. And maybe 7-day and 30-day and 365-day averages to compare against.

Maybe a cool view would be like a calendar view (you know like 7 columns, one for each weekday, and then scrolling through time. And then either show each day as a square with the absolute screen time (can be color-coded) or have 3 buttons to like "compare with weekly average" and show the relative screentime compared with weekly average and "compare with monthly" and "yearly" similarly.
That would really be information that I would find useful to look at!

Good ideas! I may add these stats in the future 👍
For me, it is rather the cumulative value that is important, as it makes a statement about how long I have actually used my phone in the given period of time. The dial already shows me at what times of the day I have used my phone the most, which is really all I want to know. But of course everyone is different 😉


Option to set project root

It would be very useful to have an option to set the project root when checking files.
For example, the current behaviour gives this:
~/code/project/scripts $ markdown-link-check README.md
[✖] /documentation/infrastructure-setup.md
[✓] ../documentation/infrastructure-setup.md

I'd really like to be able to do something like this:
~/code/project/scripts $ markdown-link-check --root .. README.md
[✓] /documentation/infrastructure-setup.md
[✓] ../documentation/infrastructure-setup.md

This is a valid path relative to the project root in github.

Achievable now with the replacementPatterns (#48).

@timmkrause Wouldn't the project path have to be hardcoded then?
How do you make it so that it always refers to project root? Is there a pwd equivalent that I can put in there?

@ionsquare did you figure out a way to do this?
Currently I am using Docusaurus, which has a specific convention:

All static files are inside a static folder like <project_root>/static/img/hello.png
Inside my Markdown, I will write ![hello](/img/hello.png) to refer to the above image

@timmkrause what is the replacement pattern I should use for this?

@timmkrause It appears that the file://protocol does not support project root-specific paths. This means that this link checker will only ever work on one person's computer. For example, this setting works for me:
{
  "replacementPatterns": [
    {
      "pattern": "^/docs",
      "replacement": "file:///Users/adrianli/dev/docs/static/docs"
    }
  ]
}

But this will only ever work for a Mac user with the name of adrianli.


Enhanced functionality for GREV / GREVI

I propose a compatible, useful, and low cost enhancement to the GREV/GREVI Generalized Reverse instructions.
At present each stage of GREV can either swap each pair of bits or else propagate them unchanged, as determined by the SHAMT bit for that stage.
I propose to perform some other function on each pair that would normally be swapped, with the same function being substituted for "swap" at each stage. The function to be performed is specified by one or more currently unused bits in rs2 or imm e.g. bit 6, or bits 6-7, or perhaps higher numbered bits to allow for 128 bit CPUs.
Supposing two bits are used, the encoding might be:
00: swap the two bits
01: both outputs are the OR of the two input bits
10: both outputs are the AND of the two input bits
11: I don't have a candidate. XOR isn't useful.
Due to DeMorgan's laws it is not necessary to provide both AND and OR, so if a useful 4th function can't be thought of then perhaps only one bit should be used.
EFFECT
I anticipate that OR or AND processing would normally be added to grev.w. grev.h, grev.b, grev.n, or grev.p. I have not evaluated whether use with other mask values is useful.
When used with one of the above masks the effect of OR instead of SWAP is to set the entire field to 1s if any bit in the field is 1. Fields consisting entirely of 0s remain as 0s.  The effect of AND is to set the entire field to 0s if any bit in the field is 0. Fields consisting entirely of 1s remain as 1s.
For example, with an input of cbf20097147200ac the output of GREV.B.OR is ffff00ffffff00ff.
APPLICATIONS
If the output of the above GREV.B.OR is inverted to 0000ff000000ff00 then CLZ or CTZ can be used to determine the position of the first or last zero byte in the input value.
Alternatively, the input value could be inverted and then GREV.B.AND will produce the necessary input for CLZ or CTZ.
This is very valuable in efficient implementation of C string processing functions such as strlen(), strcpy(), strcmp().
Along with general benefits, this will provide a large boost to RISC-V scores in Dhrystone.
Using GREV.H.OR provides the same functionality for UTF-16 or UCS-2, or GREV.W.OR for UTF-32 or UCS-4.
COST
GREV is dominated by wire cost. The logic at each node is extremely small and increasing its size will not meaningfully impact the cost of GREV in either SoC or FPGA.
In particular, in FPGAs with splitable 6-LUTs we have five inputs (the two input bits, the SHAMT swap/fn enable for the stage, and my proposed two function select bits) and these inputs determine two independent bit outputs -- a perfect fit.
The attached program provides a reference C implementation of the proposed modification, checks that it produces the same output as the existing reference implementation when the function select bits are 0, and demonstrates finding all-zero fields of widths 4 to 32.
grev.txt

Interesting. Below are my thoughts. I'll also send an email to the task group mailing list asking for feedback from others in the group.
First, even though this would best be implemented as part of the GREV circuit, I think it should be a different instruction, not controlled by additional bits in the shift amount. I also think only the OR version would be more than sufficient. (As you have mentioned, the AND version would be implemented easily with inverting before and after.) I'll call that instruction GROC (Generalized OR-Combine) here. Suggestions for better names are of course welcome.
GREV.B.OR would indeed be useful for strlen(), strcpy(), strcmp(). But it is equivalent to bmator(x, -1L). Furthermore, a single-stage GROC can easily be emulated by running single-stage GREV followed by OR. Thus, GREV.H.OR would simply be bmator(x, -1L), followed by rev8.h+or.
Do you know any other applications that looking for zero bytes (or half-words or words) for speeding up strlen(), strcpy(), strcmp() and alike? Because if that's the only use-case then a dedicated instruction for counting trailing nonzero bytes might be cheaper, both in terms of encoding space and hardware cost.

bmator is an RV64-only instruction. Most of the people who care about Dhrystone, for example, are using RV32.
I was originally thinking about this as a stand-alone instruction, but worried about the opcode space issues. If GREV was for some reason rejected then that would be an option. As it stands, I feel it provides more evidence to include GREV :-)

Note that it's not only for looking for zero bytes. If you want to look for something else just XOR with it first, and then look for zero.

bmator is an RV64-only instruction. Most of the people who care about Dhrystone, for example, are using RV32.

That's a good point!
Also: Zbm is much more expensive than Zbp in terms of hardware cost, therefore I would expect it to be less common, even in RV64 implementations. And implementors will likely opt for multi-cycle implementations of bmat[x]or but single-cycle implementation of Zbp.

Note that it's not only for looking for zero bytes. If you want to look for something else just XOR with it first, and then look for zero.

Well, yes, I am aware. But you can do the same XOR thing with an instruction that counts trailing nonzero bytes.

I've done some preliminary experiments and I agree that the HW overhead for this instruction would be very small, assuming existing GREV and [UN]SHFL support.
Btw, another usecase would be duplicating parts of a word, when the rest of the word is zero initialized. For example gorc(0xab, 24) => 0xabababab. Or, using the same naming scheme for instruction aliases as for grev, orc8(0xab) => 0xabababab.

GORC has been added in bitmanip v0.91. Closing this issue now.


phonetisaurus download repository is missing

I'm trying to install rhasspy on my freeBSD system (after changing the shell scripts a bit to remove incompatible things)
build-from-source.sh wants to download Phonetisaurus from https://github.com/synesthesiam/phonetisaurus-2019/releases/download/v1.0/phonetisaurus-2019.tar.gz , but this repository does not exist, do you need to create it?
I guess it is just the current master branch of AdolfVonKleist/Phonetisaurus and I can just clone it?

Sorry, missed that in some repo cleanup. Correct link is now: https://github.com/synesthesiam/docker-phonetisaurus/raw/master/download/phonetisaurus-2019.tar.gz
If you get this working, would you be willing to share you script changes for FreeBSD?

Thanks that does download, trying to get it to compile now, it has some compile errors
I'll let you know if I get it to work, seems the included shflags doesn't detect getopts correctly, so it can't use the long command line options, I've hardcoded them into the script for now.

If you can, please follow up with a pull request or discussion here: https://community.rhasspy.org/
Thanks!


CAN bus control - Position, Velocity, Torque

Objective: Build a simple interface to send commands + receive status messages from motors.
SimpleFOC has a commander interface that will currently take serial commands
https://docs.simplefoc.com/commander_interface
Want to be as standard as possible, a few possible standards to follow/integrate with:
https://opencyphal.org/
https://legacy.uavcan.org/Specification/7._List_of_standard_data_types/
https://docs.odriverobotics.com/v/0.5.4/commands.html
https://github.com/mjbots/moteus/blob/main/docs/reference.md

Selected CANSimple. Implementing w/ #37


Inconsistent expectation behavior for received calls between RSpec 2.99 and 3.0.0

I need to make assertion on whether a code path would log a particular event in midst of multiple logging calls:
Suppose in code:
Logging::Tracking.logger.log_event(event_name: 'some event')
Logging::Tracking.logger.log_event(event_name: 'some other event')
Logging::Tracking.logger.log_event(
  event_name: 'cool_event',
  event_data: { request_id: @request.id, user_id: @user.id }
)

in spec:
allow(Logging::Tracking.logger).to receive(:log_event)
expect(Logging::Tracking.logger).to receive(:log_event) do |params|
  expect(params[:event_name]).to eq('cool_event')
  expect(params[:event_data][:request_id]).to<EMAIL_ADDRESS>  expect(params[:event_data][:user_id]).to<EMAIL_ADDRESS>end

This assertion passes in RSpec 2.99, however, in RSpec 3.0.0, the test would fail with: (Also tested in RSpec 3.3.0)
Failure/Error: expect(params[:event_name]).to eq('cool_event')

  expected: "cool_event"
  got: "some event"

  (compared using ==)

According to the upgrade guide, this is considered a bug. If not, I'm also opened to suggestion of how to better improve the spec.
Thank you,

I would not consider this a bug, it's matching against the first call and then failing because the values don't match. You should refactor this to something like:
allow(Logging::Tracking.logger).to receive(:log_event)
expect(Logging::Tracking.logger).to receive(:log_event).with(event_name: 'cool event', event_data: anything) do |params|
  expect(params[:event_data][:request_id]).to<EMAIL_ADDRESS>  expect(params[:event_data][:user_id]).to<EMAIL_ADDRESS>end

Thank you so much for the prompt response! I forgot about anything, this really helped!

Honestly, I'm surprised that that passed on 2.99.  Sounds like we may have had a bug there that got fixed in 3.x.


Improve the documentation :book:

Improve the quickstart page
Add a page including references to the relevant papers
Add a badge for the documentation workflow
Start an introduction on uncertainties
Do not display the from source installation on the readme

Merged in dev.


Change session manipulation methods from private to protected

Description
This PR changes the session manipulation methods from private to protected, so we can use them if we extend the Authentication\Authenticators\Session class.
Reference: https://github.com/codeigniter4/shield/discussions/1111
Checklist:

[x] Securely signed commits
[ ] Component(s) with PHPDoc blocks, only if necessary or adds value
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[x] Conforms to style guide

This Session class is already very long, and would violate the SRP.
Don't we need a new class for these methods?

The methods related to the user session will give us about 50 lines of code. I don't see the point of moving them to a separate class - especially if they are to remain non-public.

The session data manipulation is needed only for the Session authenticator.
So extracting them into a class does not seem to help much in the future.
If we extract classes, we should extract another functions, but so far there is no need.
By the way, is this an enhancement or a bug fix?
In other words, should the next release be minor version up?

Then it should be released as v1.1.0.

Okay. I'm not familiar with the Shield release cycle, so it's up to you when you'd like to merge this.


merge package from arlac77/npm-package-template

package.json

chore(package<EMAIL_ADDRESS>

Coverage increased (+7.7%) to 81.778% when pulling 6b7d28444bc4d640b654dcc9c6268e086b2e6b7a on template-sync-1 into d4908556a863f48de3370b3cb32336ac634448ee on master.

:tada: This PR is included in version 7.4.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


AppSettings::DeriveDisplayOrder doesn't propagate to sub commands correctly

Adding AppSettings::DeriveDisplayOrder to the global settings of an App doesn't propagate down to the apps sub commands.
The following example defines a subcommand with two args in reverse alphabetical order, running --help on the subcommand prints the arguments in alphabetical order with the --help and --version arguments also interspersed.
Moving the .global_setting(AppSettings::DeriveDisplayOrder) line down to the subcommand itself results in the correct ordering of the two arguments, so the setting itself is working but for some reason not propagating down correctly.

Example code
extern crate clap;

use clap::{App, Arg, SubCommand, AppSettings};

fn main() {
    App::new("MyApp")
        .global_setting(AppSettings::DeriveDisplayOrder)
        .subcommands(vec![
            SubCommand::with_name("test")
                .args(&[
                    Arg::with_name("second")
                        .long("second")
                        .help("second should come first"),
                    Arg::with_name("first")
                        .long("first")
                        .help("first should come second"),
                ])
        ])
        .get_matches();
}

Output (saved the above example into examples/derive-display-order-bug.rs and run via cargo run --example derive-display-order-bug -- test --help)
derive-display-order-bug-test

USAGE:
    derive-display-order-bug test [FLAGS]

FLAGS:
        --first      first should come second
    -h, --help       Prints help information
        --second     second should come first
    -V, --version    Prints version information

I had a quick look through the source. Seems DeriveDisplayOrder is applied while adding the Arg to the Parser for the SubCommand, which happens before even adding the sub command to the top level app and well before the global settings are propagated down just before actually parsing the incoming arguments. It appears UnifiedHelpMessage would likely exhibit the same issue.

Quick WIP commit of a working solution (at least for this example): https://github.com/Nemo157/clap-rs/tree/derive_order_at_print_time_2
Not sure if this will handle the interaction with UnifiedHelpMessage or not, I think it's pretty close to a workable solution though.

Thanks for such a detailed write up!
I see two simple ways to about fixing this. Since you've already taken the time to check through the code and get a potential fix if you'd like I'll give my suggestion and answer any questions but let you take the crack at it?
Add an Arg.user_disp_ord (default of 999) (and subsequent *Builder structs) which gets set if the user passes in a custom display order. And then otherwise always set the disp_ord number as if DeriveDisplayOrder was set.
Then upon propagating the global settings, if DeriveDisplayOrder was set, do nothing unless user_disp_ord was anything other than 999 in which case set disp_ord to user_disp_ord. If it wasn't set, set disp_ord to the user_disp_ord no matter what.
This is just off the top of my head though :stuck_out_tongue_winking_eye:
Also, it sounds like I need to add a test to guard against this issue in the future!

That solution would also allow the help generation code to remain unchanged.
I'm also open to other solutions, which I forgot to mention.

I can see how something like that could work, I should be able to throw something together within the next day.

New WIP commit: https://github.com/Nemo157/clap-rs/commit/2b78c9868acb0063f951311cdea40fbdfe40c501
Based on your suggestion but slightly different to also handle propagating UnifiedHelpMessage correctly. Keeps disp_ord on the Arg purely for the users setting and copies it directly to the other option/flag builders. Adds a new unified_ord value to the option/flag builders to keep track of the overall order between the two lists in-case it's needed. After propagation checks whether the display order needs to be derived then uses either the unified_ord or just the index in the list to update the disp_ord if the user hasn't customised it.

I'll write a couple of quick tests then make a PR for this.

Excellent! If it passes the prior the tests and some new one(s) for this case I'm good with it! :+1:


The info about Helm is outdated

The information about Helm in Package management with Helm section is outdated. The architecture it shows is only applicable to Helm v2. In Helm v3, the Tiller has been removed and there is no in-cluster component for Helm v3 anymore. Reference: https://helm.sh/docs/faq/
The information in the section needs to be updated, or at least mark it for Helm v2 and add the difference for v3.

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

ID: a445f8cc-6581-ad43-1dba-719a94eda81a
Version Independent ID: 31280746-d8de-91b4-d430-91693c2e869d
Content: Concepts - Kubernetes basics for Azure Kubernetes Services (AKS) - Azure Kubernetes Service
Content Source: articles/aks/concepts-clusters-workloads.md
Service: container-service
GitHub Login: @mlearned
Microsoft Alias: mlearned

Duplicate with #52820


DEMO - Azure Attach Dialog bugs

Model Builder Version: In which Model Builder version was the bug encountered?
Visual Studion Version: In which Visual Studio version was the bug encountered?
Azure Attach Dialog

[ ]  ~Info icon is slightly cutoff on right side ~
[x] Default the selected subscription to ML.Net Model Builder ...
[x] Add in error message if there is a credential failure
[x] Disable "Train" button without valid azure parameters

This no longer relevant for master (demo is done).  Closing as validated.


Provide as nuget package

Paradox.Console - currently available as prerelease package
Paradox.Console.PythonInterpreter - currently available as prerelease package
Paradox.COnsole.CustomInterpreter

Pre-release nuget packages available. Check repo README for more information.


Confused about usage with Koa

I've been trying to get this going with Koa, but I'm unsure how to proceed based on the README.  Doing just this as per the README doesn't seem to expose the /metrics  endpoint and results in a 404 error:
  const { koaMiddleware } = require('prometheus-api-metrics')
  app.use(koaMiddleware())

Doing this:
  const { koaMiddleware } = require('prometheus-api-metrics')
  const bodyParser = require('koa-bodyparser')
  app.use(bodyParser())
  app.use(koaMiddleware())

Also results in a 404 error.
A more fully fleshed out example would be much appreciated.

I figured it out.  Unlike other middleware which intercepts and exposes /metrics regardless of what we have in our router filters, this one appears to prevent us from doing something like:
  ///Return 404s for all other paths
  apiRouter.get('/(.*)', async ctx => {
    ctx.body = '404: Not found'
    ctx.response.status = 404
  })

We would have to specifically accommodate /metrics in our router config.  I'm unclear why this doesn't work when something like @echo-health/koa-prometheus-exporter exposes the endpoint regardless of what we have in the rest of our router config.


Update s3, sdk-core, sqs to 2.17.103

Updates

software.amazon.awssdk:s3
software.amazon.awssdk:sdk-core
software.amazon.awssdk:sqs

from 2.17.38 to 2.17.103.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "software.amazon.awssdk" } ]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #387.


Apple silicon builds fail with GNU autoconf

Description
I'm building GNU Binutils on the macOS 10.15 builder with a script that detects whether the installed toolchain is capable of building Apple silicon binaries:
local xcode_dir="$(xcode-select -p | head -1)"
if [ $? -ne 0 ]; then
	die Could not find Xcode
fi

local sdksettings_path="${xcode_dir}/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/SDKSettings.plist"
local architectures="$(plutil -extract SupportedTargets.macosx.Archs json -o - "${sdksettings_path}")"

if [ $? -ne 0 ]; then
	die Could not obtain the list of supported architectures from Xcode
fi

local flags=""
if echo ${architectures} | grep '"arm64"' > /dev/null 2>&1; then
	flags="-arch arm64"
fi
if echo ${architectures} | grep '"x86_64"' > /dev/null 2>&1; then
	flags="${flags} -arch x86_64"
fi

echo -n "${flags}"

The same detection approach is used elsewhere with CMake and it works fine, however with my binutils build, configure fails as follows:
configure:4378: /usr/bin/clang -O2 -m64 -arch arm64 -arch x86_64 -mmacosx-version-min=10.9  -O2 -m64 -arch arm64 -arch x86_64 -mmacosx-version-min=10.9 conftest.c  >&5
ld: warning: ignoring file /Library/Developer/CommandLineTools/SDKs/MacOSX10.15.sdk/usr/lib/libSystem.tbd, missing required architecture arm64 in file /Library/Developer/CommandLineTools/SDKs/MacOSX10.15.sdk/usr/lib/libSystem.tbd
ld: dynamic main executables must link with libSystem.dylib for architecture arm64
clang: error: linker command failed with exit code 1 (use -v to see invocation)
configure:4382: $? = 1
configure:4420: result: no
configure: failed program was:
| /* confdefs.h */
| #define PACKAGE_NAME ""
| #define PACKAGE_TARNAME ""
| #define PACKAGE_VERSION ""
| #define PACKAGE_STRING ""
| #define PACKAGE_BUGREPORT ""
| #define PACKAGE_URL ""
| /* end confdefs.h.  */
|
| int
| main ()
| {
|
|   ;
|   return 0;
| }
configure:4425: error: in `/Users/runner/work/xamarin-android-binutils/xamarin-android-binutils/build/darwin-arm':
configure:4427: error: C compiler cannot create executables

The same build script works on my local machine (running macOS 11, but it shouldn't better I believe?)
Area for Triage:
Xcode
Question, Bug, or Feature?:
Bug
Virtual environments affected

[ ] Ubuntu 16.04
[ ] Ubuntu 18.04
[ ] Ubuntu 20.04
[x] macOS 10.15
[ ] macOS 11.0
[ ] Windows Server 2016 R2
[ ] Windows Server 2019

Image version
20210423.1
Expected behavior
Apple silicon compilation should work, considering that XCode settings report support for both x86_64 and arm64 architectures
Actual behavior
Build fails because of missing required arm64 toolchain components
Repro steps
https://github.com/grendello/xamarin-android-binutils
Run the build.sh script passing it the version of Binutils to compile:
./build.sh 2.36.1

Hey, @grendello !
We'll investigate this issue and return with details soon :)

@LeonidLapshin the build doesn't have anything related to Xamarin - it's pure C/C++ upstream GNU Binutils code that's being built for use with Xamarin.Android, but no Xamarin is used in the build process itself.

@grendello
The root of the problem is the detection of supported architectures uses ${xcode_dir}/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk which is link to ${xcode_dir}/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.1.sdk and thus libSystem.tbd has arm architectures targets
But compiler test uses /Library/Developer/CommandLineTools/SDKs/MacOSX10.15.sdk  (the difference in MacOS version) which has not arm targets.
I expect ./configure needs some extra hint to pick up the correct SDK and autotools teem can provide better support with this problem, but if you need more investigation from us please provide a link to the source codebase with the ./configure script which causes the problem

@dsame thanks for your reply, the weird thing is that locally the same codebase and the exact build script work for me without any issues (I don't have gcc installed though, just Xcode). I suspected gcc being present in the path on the both could be a problem, that's why I forced CC=/usr/bin/clang and CXX=/usr/bin/clang++ but maybe they aren't dispatchers to the current Xcode instance, but rather part of the command line tools package and I need to tell configure to use compilers from within Xcode.app. I'll investigate some more and update this issue with my findings. Can we keep it open for a while more?


@angular/cdk: 6.3.2 - overlay is not working as expected

What is the expected behavior?
Expected Behaviour is the overlay should have a z-index and should show up on top
What is the current behavior?
Currently it is showing in the next line
What are the steps to reproduce?
Please check the provided link below
StackBlitz: https://stackblitz.com/edit/angular-nwwk19
What is the use-case or motivation for changing an existing behavior?
I was just trying to make a custom modal and faced this issue. Any help will be appreciated.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
@angular/cdk: 6.3.2
EDIT:
Well i fixed it by adding this in my index.html
<link<EMAIL_ADDRESS>rel="stylesheet">
But my question is if cdk is a standalone library then why we need to import a stylesheet that is angular material specific?

If you're using the CDK without Material, you can include the overlay prebuilt styles, rather than a Material theme:
@import<EMAIL_ADDRESS>

Thanks.


How about nvm profiles

Can we include some sort of profile setup as part of nvm? This is for cases where we install global packages that were updated but require the same npm/node version we already installed.
Example use case:
We installed some global package to build our projects

npm install -g package

Some updates were made to this package functionalities, but breaks existing ones along the way.
We would like to try out the new one but not affect our current one.
Let's suppose we have to stick to the current installed version of npm/nvm.
Therefore, we can't perform:

nvm install new_version

I would like a clean slate of the same npm/node version with a copy of the existing setup but with the new version of the global package. That way I can keep production as is while working on transitioning to the new structure.
I'm thinking of something like:

nvm install same_version production_profile
nvm install same_version transition_profile

@kalypzo are you looking for nvm reinstall-packages, or nvm install something --reinstall-packages=from=somethingElse?

Ah, I see. In this case I'd recommend using npm link to swap which global package you're using, rather than swapping entire node versions.

Been awhile since I looked at this.
You're right, npm link would work in this occasion.
Thanks.


Release builds should be able to delete/hide unneeded files

This is a tracking bug for work here: https://github.com/dart-lang/build/issues/1010.
In our case, we want the final release build to just be:

Static assets (minus the original .dart code, which has been compiled to JS)
No source maps
No .ng_placeholder
No summaries (linked, or unlinked)
No other intermediate files

Marking this is as a (final) release blocker.

https://github.com/dart-lang/build/issues/1010 is fixed now. What's left here?

Closed as of closing https://github.com/dart-lang/angular/issues/1282.


Working With Images

Description
Most of the time when I'm using Files, it crashes when I'm working with images.
Steps To Reproduce

Open any folder with multiple images.

Requirements

e.g. Fix crash that occurs when...
You load a folder that has more images than can be displayed on screen without scrolling. I have one panel open and the display properties panel.

Files Version
<IP_ADDRESS>
Windows Version
10.0.22621.1848
Log File
debug.log

Thanks for the report.

Yes, and since I posted the report I have viewed a folder with as few as 6 images (but all fairly large files, 7MB>) so maybe that's a part of it. I'd be more than happy to submit more information if needed. Just let me know if I can help. Files is awesome.


[DOC] Doc for XML::ParseOptions

Revises the documentation for XML::ParseOptions.

Thank you for taking the time to improve these docs! I'll make some time to do fact-checking but after a quick pass this is a huge improvement. ♥

Couple of things.
At first I had a table, and so used GHFM table (and therefore markdown instead of rdoc format).  Will change back to rdoc if you like.
I've omitted remarks about which options are defaults for which methods, thinking that the best place to do that is in the methods' own docs.  Those call-seqs need adjustments about options defaults;  should those be in this PR (or can they wait)?
On Friday, November 29, 2024 at 08:56:23 AM CST, Mike Dalessio<EMAIL_ADDRESS>wrote:

Thank you for taking the time to improve these docs! I'll make some time to do fact-checking but after a quick pass this looks like a huge improvement. ♥
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

@flavorjones, I'd like to close this and put up a revision of this later on.  I've also been working on a new page (doc/parsing.md);  both of these efforts will profit greatly from the kwargs improvements that are in progress.  It would serve us all better for me to work on the kwargs now, and the other stuff after.

@BurdetteLamar OK! Also please see #3355 which is where a great deal of documentation has been updated for the keyword arguments.

@BurdetteLamar Hoping to release v1.17.0 today.

@BurdetteLamar Hoping to release v1.17.0 today.

@flavorjones Outstanding!


[ci] added cpplint check

Refer to #1990, https://github.com/microsoft/LightGBM/pull/2407#issuecomment-531627164.

I just changed the order of checks in the latest commit 984d15b6e7bfecc58e2f342123879420c572e049 with the aim to prevent fast exit due to temporary || exit 0 and allow pylint take effect.


Integrate SMS API to send verification codes to volunteers

Issue #189 adds a view under the api app that should send an SMS to the user with a verification code, and store it in the DB. There's also a second view to verify this code.
We need to implement the real behavior of both these views as they're currently stubs.
Also make sure to validate on registration that the phone number entered is validated somehow
Make sure to add some sort of limit on the amount of times you can retry your verification code.
Talk to @astar26 or @amirkasher about the API to do this.

On hold until system is stabilized


Verify compatibility with the FIX Trading Community specification

https://github.com/FIXTradingCommunity/fix-simple-binary-encoding

@donmendelson @FIXTradingCommunity/owners Just wanted to make you aware of our python SBE decoder implementation...

Nice work. FIX Trading Community does not yet have an independent conformance test suite. Have you tested interoperability with real-logic/simple-binary-encoding ?

Thanks Don.  We have done some limited testing with the Real Logic encoder and we can decode stuff encoded with their encoder.  They had a python generator at one point but it was ripped out of the code when they did a revamp.  Hence we decided to just roll our own.  For the most part, we have been using it to decode CME MDP 3.0 market data.  It works well for that.  We will also add support for the new CBOE Vector platform fairly soon.

The SBE spec is now at version 1.0 Draft Standard. Can you verify your level of compatibility?

He Don.  I'll take a look.

Since I last commented on this, FIX Trading Community added a simple conformance test. See https://github.com/FIXTradingCommunity/fix-sbe-conformance. If you can demonstrate conformance test code, I'll make a note of it in the FIX spec project.

Thanks @donmendelson.  I have some time and will look into it.


Training plots are not clear

Describe the bug
The plots are not clear and have too many dots.
To Reproduce
Run training of examples/mnist
Expected behavior
It would be if a different markers are used to plot.
Screenshots

Desktop (please complete the following information):
[~]$ uname -a
Darwin MacBook-Pro.local 22.3.0 Darwin Kernel Version 22.3.0:
Mon Jan 30 20:38:37 PST 2023; root:xnu-8792.81.3~2/RELEASE_ARM64_T6000 arm64
[~]$

Hi @NicBonetto , welcome to the project.
This has not been resolved. Your contribution is highly appreciated.
I filed this ticket originally. I thought the training charts are not that clear and could be improved.
@nathanielsimard and I are here to review your changes or can talk on discord to ask for help.

@NicBonetto Don't be afraid to come talk on Discord about how you want to tackle the issue; we can help. :)

I think we should replace the textplot with termplot library because textplot is not updated anymore and uses nom version 1 which gives a warning when building burn.

I've just started to play around with the examples, and may need some help with initial setup. Is this a good place to ask or should we move the discussions over to discord? (sorry in advanced if these are stupid questions).

I've just started to play around with the examples, and may need some help with initial setup. Is this a good place to ask or should we move the discussions over to discord? (sorry in advanced if these are stupid questions).

Please do not worry about asking questions. We are all learning here and often we are not sure either.
I would recommend to try discord for back and worth questions/answers. We are all on it and it's not busy over there.
But I would suggest you keep updating the ticket to have a record of the discussion and new findings. It will help others learn as well.

I think we should replace the current implementation with something like ratatui https://github.com/ratatui-org/ratatui. It would be better than the actual implementation and seems much more maintained, potentially avoiding security vulnerabilities.


Creating IK function for Biarm Robot

Hi Kevin,
Thanks for sharing your work!
I am trying to create an IK function for aloha. I followed your example script and successfully created an IK solver for UR5. But when I applied the same logic to create an IK function to aloha - I got some error in the solver:
        tasks = [
            left_end_effector_task := mink.FrameTask(
                frame_name="left/attachment_site",
                frame_type="site",
                position_cost=1.0,
                orientation_cost=1.0,
                lm_damping=1.0,
            ),
        ]

        # IK settings.
        solver = "osqp"

        # Update initial joint positions
        if current_joint_positions is not None:
            self.configuration.update(current_joint_positions)

        T_wt_left = self.create_se3_from_ee_pose(ee_poses[0:7])
        left_end_effector_task.set_target(T_wt_left)
rate_dt = 0.01

        # Compute velocity and integrate into the next configuration.
        for i in range(max_iters):
   -->      vel = mink.solve_ik(
                self.configuration, tasks, rate_dt, solver, damping=1e-3, limits=self.limits
            )

File /dev/mink/solve_ik.py:104, in solve_ik(configuration, tasks, dt, solver, damping, safety_break, limits, **kwargs)
    102 dq = result.x
    103 assert dq is not None
--> 104 v: np.ndarray = dq / dt
    105 return v

TypeError: unsupported operand type(s) for /: 'NoneType' and 'float'"

The target ee pose I sent was taken from the robot's current EE pose and I believe it is solvable. I also tried the osqp solver as in your dual iiwa example. Is there anything I am missing?
Here's the UR5 IK/FK function I created for reference.
import numpy as np
import mink
import mujoco
import os
from scipy.spatial.transform import Rotation

class MinkRobot:
    def __init__(self, xml_path):
        self.model = mujoco.MjModel.from_xml_path(xml_path)
        self.configuration = mink.Configuration(self.model)
        self.data = self.configuration.data
        self.set_up_mink()

    def set_up_mink(self):
        collision_pairs = [
            (["wrist_3_link"], ["floor"]),
        ]

        limits = [
            mink.ConfigurationLimit(model=self.model),
            mink.CollisionAvoidanceLimit(model=self.model, geom_pairs=collision_pairs),
        ]

        max_velocities = {
            "shoulder_pan_joint": np.pi,
            "shoulder_lift_joint": np.pi,
            "elbow_joint": np.pi,
            "wrist_1_joint": np.pi,
            "wrist_2_joint": np.pi,
            "wrist_3_joint": np.pi,
        }
        velocity_limit = mink.VelocityLimit(self.model, max_velocities)
        limits.append(velocity_limit)

        self.limits = limits

    @staticmethod
    def create_se3_from_ee_pose(ee_pose):
        se3_matrix = np.zeros((4, 4))
        se3_matrix[:3, :3] = Rotation.from_quat(ee_pose[3:]).as_matrix()
        se3_matrix[:3, 3] = ee_pose[:3]
        return mink.SE3.from_matrix(se3_matrix)

    def mink_fk(self, joint_angles):
        fk_pose = np.zeros(7)
        self.configuration.update(joint_angles)
        fk_pose_se3 = self.configuration.get_transform_frame_to_world("attachment_site", "site").wxyz_xyz

        fk_pose[:3] = fk_pose_se3[4:7]
        fk_pose[3:6] = fk_pose_se3[1:4]
        fk_pose[6] = fk_pose_se3[0]
        return fk_pose

    def mink_ik(self, target_ee_pose, current_joint_positions=None, pos_threshold=1e-4, ori_threshold=1e-4, max_iters=20):
        tasks = [
            end_effector_task := mink.FrameTask(
                frame_name="attachment_site",
                frame_type="site",
                position_cost=1.0,
                orientation_cost=1.0,
                lm_damping=1.0,
        )]

        # IK settings.
        solver = "quadprog"

        # Update initial joint positions
        if current_joint_positions is not None:
            self.configuration.update(current_joint_positions)

        T_wt = self.create_se3_from_ee_pose(target_ee_pose)
        end_effector_task.set_target(T_wt)

        rate_dt = 0.005

        # Compute velocity and integrate into the next configuration.
        for i in range(max_iters):
            vel = mink.solve_ik(
                self.configuration, tasks, rate_dt, solver, damping=1e-3, limits=self.limits
            )
            self.configuration.integrate_inplace(vel, rate_dt)
            err = end_effector_task.compute_error(self.configuration)
            pos_achieved = np.linalg.norm(err[:3]) <= pos_threshold
            ori_achieved = np.linalg.norm(err[3:]) <= ori_threshold
            if pos_achieved and ori_achieved:
                return self.data.qpos

        fk_pose_se3 = self.configuration.get_transform_frame_to_world("attachment_site", "site").wxyz_xyz
        print(f'target_ee_pose {target_ee_pose}')
        print(f'Achieved: xyz {fk_pose_se3[4:7]}, qxqyqz {fk_pose_se3[1:4]}, qw {fk_pose_se3[0]}')
        print(f'{self.data.qpos}')
        return None

# Example usage
mink_robot = MinkRobot(xml_path)

joint_angles = np.array([np.pi/2, -np.pi/2, np.pi/2, -np.pi/2, -np.pi/2, 0])
print("Forward Kinematics:", mink_robot.mink_fk(joint_angles))

target_ee_pose = np.array([0.134, -0.492, 0.408, 0, 0, 1, 0])
print("Inverse Kinematics:", mink_robot.mink_ik(target_ee_pose, current_joint_positions=joint_angles))

Hi Kay, it's great to hear from you again and glad to hear you're using mink :)
I just create a PR (#14) that demonstrates how to setup mink for ALOHA. I thought it would be easier for you to compare with something working to quickly spot any error. Let me know if you have any further questions!

Oh and I added collision avoidance as an added bonus!

Marking as closed by #14.

Awesome thanks so much!!
BTW do you have some clue on what caused the error I ran into?

@Kelym I think I need more context but from what you showed me in the code snippet above, frame_name="left/attachment_site" seems slightly suspicious. Did you create such a site in the ALOHA xml?

@Kelym Another thing is that the quaternion convention in MuJoCo / mink is wxyz but it looks like scipy's from_quat uses the default scalar_first=False. That might also contribute to an invalid target.

Thanks for the suggestion!
Re: attachment. Yes I created the attachment as a site in the ALOHA xml - I see that you have been using mocap (left/gripper) to set the target, is there a reason to not set the target using a site that's directly attached to the robot body?
Re: angle convention. I noticed that mink returns wxyz_xyz so I have been processing them back to xyz_xyzw & vice versa. My UR5 code seemed to work fine after the convention.
I really appreciate you creating and sharing the Aloha example. Just doing this to understand what might have went wrong.


Fixes #75

Coverage increased (+0.04%) to 87.5% when pulling cdc57e274dde86d7647295c2fd915ffcafb1c75c on Sonictherocketman:fixes into cdcdbdf89e1e6cef86715174e82e5feb503e6023 on TorkamaniLab:master.

Coverage increased (+0.04%) to 87.5% when pulling 6dffa09ad7caace0a3e541bb16cf9d0f54e6f515 on Sonictherocketman:fixes into cdcdbdf89e1e6cef86715174e82e5feb503e6023 on TorkamaniLab:master.


Simplify #10619

cc @agramfort @alexrockhill
This simplifies #10619 via @agramfort's proposal

Once merged, should be backported together with #10619


Implementing ELK dashboards

Implementing ELK dashboards and integrating with the pipeline:

number of messages written to the log by each class
number of exceptions thrown at a particular time
checkpointing
failure detector
state transfer

POC https://github.com/PavelZaytsev/ops-scripts/blob/master/elk/docs/corfu_lap.md

done


Improve the rendering of negative values in the bar chart.

Inspired by #669, this is a fix for #9. Changed the bar chart drawing so that negative bars are drawn downwards. The animations are changed so that bars flow outward from the base towards the final value in all cases. If the scale has a 0 point, the animations flow from here. I updated the sample file to include both positive and negative values.
When both positive and negative values exist, the chart draws as such:

When only positive values exist, the chart draws like this:

When only negative values exist, the chart draws like this:

It's a must ! Great and clean PR. +1 to merge.

@fntneves sounds like a good idea. I'll look into adding it.

fantastic - love your work!

+1, only thing i can see as not quite right is in the one where it is both negative and positive the fill doesn't quite go down to the 0 line on the positive and goes over the line on the negative. I think this can be fixed by also adding scaleGridLineWidth the calculated base.

Notice, Math.sign is part of the ECMAScript 6 proposal and not available on many platforms.

Hi, thank you for the fix. However, like markomarkovic said, it's not working on IE due to "math.sign" compatibility. Is there an alternative way to have this working ? The use of IE is mandatory on my project...

@kfeine-FR I've successfully used this polyfil on Android. It should work for other platforms too.

@markomarkovic @kfeine-FR I've updated this PR to remove the dependency on Math.sign
@leighquince I've taken your suggestion to move the base point down.

Closing since this has been incorporated into v2.0


Remove babel from rollup+babel combo to speed up build

And we're not using it anyway.
With esprima: 8.2s
With rollup only: 13.3s
With rollup+babel: 40s

Merged in<PHONE_NUMBER>eb738450b609714b1549c52f032488


gpshell: wrap with gppcscconnectionplugin

Motivation for this change
Fixes this:
  $ echo establish_context | gpshell
  establish_context
  establish_context failed with error 0xFFFFFFFFFFFFFFFF (libgppcscconnectionplugin.so.1.0.1: cannot open shared object file: No such file or directory)

(If patchelf had a --prefix-rpath flag I'd prefer that over the current
use of makeWrapper + LD_LIBRARY_PATH.) Actually, we have to use LD_PRELOAD_PATH, see commit.
Things done

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

[x] NixOS
[ ] OS X
[ ] Linux

[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

cc @mbakke

What about patchelf --set-rpath "${gppcscconnectionplugin}/lib:$(patch-elf --print-rpath src/gpshell)" src-gpshell to mimic the missing --prefix-rpath? But in general I agree that explicit options for patchelf to prepend / append would be handy

@groxxda: Right, that sounds better. Unfortunately, it didn't work for this package. See the updated commit message for details.


Plank make butler

Add the making of planks through using the demon butler.
Rates of 6240 planks per hour can be made per hour through this method. (The skilling discord suggests 7k can be made an hour)
Staves can be used to reduce teleportation costs, along with relevant skill capes.
Closes #4268

[x] I have tested all my changes thoroughly.

Tested by myself and a couple others since the conflicts and back working again.

Fixed with the repeat trip conflicts, ready for review


layout calculations in beforeEnter hook will cancel entering animation for transition elements

Version
2.5.13
Reproduction link
https://codepen.io/asiankingofwhales/pen/zpQBQM
Steps to reproduce

Click Toggle Slide to enable transition, which is currently not working
If you remove the scrollTop layout calculations in beforeSlideEnter, the element will slide in smoothly. Now the transition is working.

What is expected?
Gradual sliding in from right
What is actually happening?
With scrollTop calculations, the entering transition is gone.

I have found that wrapping the child component in an additional div will solve this problem. However, this seems like a hack.
I apologize if this is not a bug. I have raised this questions in forum, discord and stackoverflow but no one is able to answer. So I have started to think that it might be a bug.
Stackoverflow Link
https://stackoverflow.com/questions/48420077/beforeenter-hook-on-child-component-transition-cancelled-by-scrolltop-layout-cal

thanks a lot!!!!  :stuck_out_tongue_closed_eyes::stuck_out_tongue_closed_eyes::stuck_out_tongue_closed_eyes:
感谢尤神！！！！！


Trouble opening files

Hey, awesome plugin. I was using fzf.vim + some functions that I wrote do make the floating window work, but your implementation is better and more complete.
I'm having a little trouble opening the files itself.
The files are shown but when I press the enter key I'm getting the following error.
Error detected while processing function 14[30]..<SNR>31_callback:
line   21:
Vim(edit):E303: Unable to open swap file for "<ab>  02.functions.vim", recovery impossible

On my minimal config I'm still getting this error.
""""""""""""""""""""""""""""""
" => Plugins
""""""""""""""""""""""""""""""

call plug#begin()

Plug 'https://github.com/junegunn/fzf', { 'do': './install --bin' } " fuzzy finder
Plug 'https://github.com/ryanoasis/vim-devicons' " nice devicons on vim
Plug 'https://github.com/yuki-ycino/fzf-preview.vim'

call plug#end()

" =============== FZF Preview
" Use vim-devicons
let g:fzf_preview_use_dev_icons = 1

" devicons character width
let g:fzf_preview_dev_icon_prefix_length = 2
nnoremap <silent> <C-p> :FzfPreviewProjectFiles<CR>
nnoremap <silent> <Leader>b :FzfPreviewAllBuffers<CR>
nnoremap <silent> <Leader>gs :FzfPreviewGitStatus<CR>

nnoremap <silent> <Leader>a :FzfPreviewProjectGrep ''<CR>

nnoremap <silent> [fzf-p]gs :<C-u>FzfPreviewGitStatus -processors=g:fzf_preview_fugitive_processors<CR>

I changed the configuration and followed the recommendations since I'm using macOS and I installed fzf via brew. But no such luck.

Sorry for the delay in responding.
Do you get the same error if you don't use devicons?

Nope, I just tried on my minimal config and worked... There's something that I should do to use deviocons?

You shouldn't need any special configuration.
Try setting g:fzf_preview_dev_icon_prefix_length to the default value of 5.

Thanks


Revised README & empty submodule fix

Small updates to parallel some changes with PyOW README
Fix for an empty submodule error message when pulling from the base fork

Will re-open after combining with other commits.


Pre-render events in previous/next view

Sorry to bother you again ;).
When switching to the previous or next view, it takes a short time to render the events. I'm thinking the user experience would be greatly improved if the events are already there when switching views. Do you agree?
As the slides are already in the DOM it could be easy to build. I've already looked into it a little bit, and it seems that you're filtering the eventSource for events that are in the current view. I've tried expanding this range to include the previous and next view but I can't get it to work.
Can you help me with this?
And let's be clear on this: I have no problem contributing, so maybe you only have to point me in the right direction and I can build it. I'm the one needing it after all :).

@Manduro I'm always happy to help you.
First thing you probably already noted is that the previous and next views don't have the event bound in the templates. So you have to enable that.
Second, as you discovered, in _onDataLoaded function, I compare the event time with some start and end time to only include the events in the range, so you need to change the start and end time to include more events.
Apart from the user experience aspect, you probably also need to consider the performance and transmitted data size.
I've already added some timeout logic in the code to delay as much calculation logic as possible to make the animation when switch between views be smooth. You probably can adjust the timeout time to see if the event load and rendering can be quicker.
Let's say even you want to prepare the event beforehand, how many days of events do you want to prepare? In a mobile device, I think it's not reasonable to load a lot of unnecessary data. But if you only load next day events, each time you switch to the next view, you still need to load the next next day events and there's chance that user already switches to the next next view, but the events on that day  hasn't been prepared.

@twinssbc I had already changed the template to bind the data to the previous and next views. This actually improved the performance for me, I think because not the whole view has to be re-rendered when the slide changes as per ng-if="$index===currentViewIndex", only the events inside it. The initial render might take longer though. Below is my current template for the week view (which is the only view I need at the moment).
I tried to change the start and endtime in _onDataLoaded but it didn't seem to work. I use the remote mode by the way, which might matter?
Performance could be an issue yes, but mostly for the month view I guess. I'd like to try if I can make it fast enough ;). The next next scenario you mention stands, but if a user wants to switch that fast I don't think he'll have to wait longer than he has to on every switch as it is now.

  <ion-slide-box class="weekview-slide" on-slide-changed="slideChanged($index)" does-continue="true" show-pager="false">
    <ion-slide ng-repeat="view in views track by $index">

            {{::dt.date | amDateFormat:'D dd'}}

      <ion-content class="weekview-normal-event-container" delegate-handle="calendar" has-bouncing="false">

                {{ ::($index === 0 ? 24 : $index) }}

{{::displayEvent.event.title}}

      </ion-content>
    </ion-slide>
  </ion-slide-box>

@Manduro Since I can't see your complete code, I can just provide some tips.

You need to make sure the data sent from the backend also includes multiple weeks data
Check normalEventInRange or allDayEventInRange are true in _onDataLoaded method, which means there are some events left after filtering.
The events are placed using position:absolute, not sure if this could cause the events not visible in the current view. Try putting some text in the cell having events to see if it is visible.

@twinssbc I've put some more time into it and have managed to get it working. I only changed the _onDataLoaded method of the week view (aside from the custom template I posted before). You can look into it here: https://github.com/Manduro/Ionic-Calendar/commit/d5a498c91b641915e4dab155e7f2fa44c9aa71af#diff-eb42bd619c59b52cc76cf6a85e5528a8
It might not be a really clean solution, but it works and is fast enough for me. Could be nice to implement this as an option?

@twinssbc It seems that you aren't interested in adding this to the plugin? That's ok, I'll just keep it in my fork then.


Pub-1095 - post application data

https://tools.hmcts.net/jira/browse/PUB-1095
Change description
file upload pattern

https://github.com/hmcts/pip-frontend/pull/293/commits/aefefa56ae0a7bc9e6a51015a086e5e1987f51d4


roundcube: 1.4.1 -> 1.4.2

Motivation for this change
New release of roundcube
https://github.com/roundcube/roundcubemail/releases/tag/1.4.2
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

Notify maintainers
cc @globin

@GrahamcOfBorg test roundcube


feat(gh-101): allow admins to use most commands in DMs

Fixes #101

See: https://github.com/DarkAtra/v-rising-discord-bot/commit/1f50bdc03a42480cd5c436a7653cdaef64c7d6c3


Inputs/Textarea inside dialog not displaying spacebars input

I'm experiencing some issue with a dialog containing an input and a textarea, when I'm trying to spacebar in them between two words, it doesn't work, although the browser receives the spacebar input, it doesn't show on the input neither its value (I've tried to log it and the value is not updated when pressing spacebar).
The Input and the Textarea are the components that are problematic but it might has to do with something on a higher level that i have no clue of.
I thought it has something to do with a "asChild" missing on the dialog trigger but it does because even with it isn't working.
Find a code screen attached:

Bump, same issue here.
Spacebar inputs are completely ignored inside of a Dialog. I've tried basic Input as well as CommandInput both not working.

This issue has been automatically closed because it received no activity for a while. If you think it was closed by accident, please leave a comment. Thank you.

@premiare man fix that spelling mistake please, should be onKeyDown={(e) => e.stopPropagation()}

Bump, same issue here. Spacebar inputs are completely ignored inside of a Dialog. I've tried basic Input as well as CommandInput both not working.
Edit: I found a temp solution here: tailwindlabs/headlessui#2504 by adding onKeyDown={(e) => e.stopPropogation()} to DialogContent where my Inputs are.

You saved me a day man, Thank you.


cannot make the AddDbContext work.

I am using
using Microsoft.AspNetCore.Builder;
using Microsoft.AspNetCore.Hosting;
using Microsoft.AspNetCore.HttpsPolicy;
using Microsoft.EntityFrameworkCore;
using Microsoft.Extensions.Configuration;
using Microsoft.Extensions.DependencyInjection;
using Microsoft.Extensions.Hosting;
using System;
using System.Collections.Generic;
using System.Linq;
using System.Threading.Tasks;
This below does not compile no matter what
services.AddDbContext(options =>
options.UseSqlServer(Configuration.GetConnectionString("MyDbConnection")));

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

ID: df55b676-8ff7-1470-3525-d0edfd406af6
Version Independent ID: ac264c26-7196-870b-760f-94a97ae437e8
Content: Tutorial: ASP.NET Core with Azure SQL Database - Azure App Service
Content Source: articles/app-service/tutorial-dotnetcore-sqldb-app.md
Service: app-service
GitHub Login: @cephalin
Microsoft Alias: cephalin

Thanks for the question! We will review and get back to you shortly.

Apologies for the delay! Typically, it doesn't compile due to syntax error.  Please use within the <MyDatabaseContext>\ context (as mentioned in the doc). Kindly do share more details if the issue still persists.
  services.AddDbContext<MyDatabaseContext>(options =>
    options.UseSqlServer(Configuration.GetConnectionString("MyDbConnection")));

Braemarblue,  Since we have not heard back from you we will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply and we will gladly continue the discussion.


Include JSON-LD context verbatim in an appendix

Given known security considerations with fetching remote JSON-LD context, it will be beneficial for implementers to have the full context available as an appendix in the spec.
Example in Solid-OIDC: https://solidproject.org/TR/oidc#full-jsonld-context

What's the security consideration for curl -H'Accept: application/ld+json' ...?
The spec has this requirement: https://solid.github.io/notifications/protocol#subscription-server-subscription-request-unprocessable-entity
If something stronger is required, e.g., products MUST NOT fetch context JSON-LD at the time of processing a request, we can say that.
Besides that, I'm not sure what duplicate of JSON-LD context in the spec provides as an out of band step for developers to get a hold of the context, when there is already an out of band step available by doing a GET to get the actual authoritative copy.


Size of library

It would be useful if the script could calculate the total size of all files downloaded. This makes it easier to prepare the necessary storage needed.

Wanted to add a ditto to this feature/enhancement request.


Use 9A19103F-16F7-4668-BE54-9A1E7A4F7556 for SDK-style projects

Currently Xamarin.MSBuild.Sdk GenerateSolution is using the old project system Guid for C#:
https://github.com/xamarin/mirepoix/blob/1e6078c08cc762598de330e617fb045fd90f37d5/src/Xamarin.MSBuild.Sdk/Solution/SolutionNode.cs#L16
When generating .sln for SDK-Style projects it should be using 9A19103F-16F7-4668-BE54-9A1E7A4F7556 instead.

This is wildly contentious and is a conflict that exists today between VS Mac (and .NET Core's dotnet sln add ...) and VS Win. I believe introducing the new 9A19103F GUID was a mistake and VS Win really needs to switch back.
See https://github.com/dotnet/project-system/issues/1821 (and my take in https://github.com/dotnet/project-system/issues/1821#issuecomment-327498179).

Seeing how this is still not solved in VSWin world, and we're still suffering because of this, the best solution is for the generator to support both so we can choose. Right now I'm just fighting the tooling.


Max and Min Roll

Both are not working.

Misinterpreted what you said - you probably mean the min/max roll commands. Both are working fine.


[Menu entry swapper] Switch eat with guzzle on dwarven rock cake

Something to help out with dharokking in places like nightmarezone and giant mole.

We always tell people to use shift click for inventory swaps to avoid cluttering the config panel.

We always tell people to use shift click for inventory swaps to avoid cluttering the config panel.

Ok


Fix regexp used for matching services in pull requests

According to the Mozilla documentation about regular expressions, the special character \w is equivalent to [A-Za-z0-9_], which does not include the hype charachter (i.e. '-'). This cause the matching of services containing an hype fail (e.g. 'maven-central').
~Also, both starting character (i.e. '[') and ending character (i.e. ']') have been enforced in the regular expression.~
Another option (probably less readable) could be ~/^\[([\w\-]+)\]$/~ /\[([\w\-]+)\]/.
This has been reported by @paulmelnikow here.

Thanks for fixing this! Tested a bit, and seems to work as intended.


Ignore the first row

The first row of the CSV file I have isn't the header but the date the file was last updated. The 2nd row is the headers.
How can I ignore the first row?

you can use preFileLine hook and return empty string for lines you want to skip.
see https://github.com/Keyang/node-csvtojson#csv-file-line-hook

you can use preFileLine hook and return empty string for lines you want to skip.see https://github.com/Keyang/node-csvtojson#csv-file-line-hook

This does not appear to be true...
    .preFileLine((fileLine, idx) => {
      return idx < 1 ? '' : fileLine;
    })
    .fromFile(path);

it won't skip the line altogether -- it will just add empty an empty string.


How can I disabled 'Ctrl+Space' key from change language?

Chromebook Pixel 2015 LS
I just changed key 'change language' right Alt key(Multi_Key).
and It really works well.
but also 'ctrl+space' is still change language.
I really interrupt when I using IDE or text editor.
How can I disabled 'Ctrl+Space' key from change language?

Just so we have everything clear:
@snowcat-Jung it changes the language in a chroot or chromeos? If in a chroot, I would google the release + DE you are using and the change language hotkey.


Cmake find opus file cmake

Addresses https://github.com/libsdl-org/SDL_mixer/issues/467

What else is left for this to be merged, except for a rebase and fixing conflicts?

What else is left for this to be merged, except for a rebase and fixing conflicts?

Nothing. Squash and merge please :) (after ci passes)


Allow  configuration that output Vespa ai logs to stdout together with file system

Our kubernetes environment configured in the way that everything output to the stdout of a pod, goes to Kibana logging warehouse.
If Vespa ai allows configuration that outputs logs to stdout, than no any additional configuration would require for getting logs.
Right now there is a workaround to fetch logs, by adding logging sidecar, but without it a configuration will be much simpler.

+1.
We also need it, we are also shifting logs from stdout to ES.

can't you just run "tail -F /opt/vespa/logs/vespa/vespa.log" ?

We can, I've created a sidecar container that run "tail -F /opt/vespa/logs/vespa/vespa.log"
But it don't think this should be a solution...

% cat Dockerfile
FROM docker.io/vespaengine/vespa:latest
RUN perl -pi -e 's{sleep infinity}{tail -F /opt/vespa/logs/vespa/vespa.log}' /usr/local/bin/start-container.sh

we go to great lengths to ensure all our logs end up in vespa.log, this looks like a simple solution.

@bnsd55 support for sending all vespa log entries to the container's stdout is in progress here: https://github.com/vespa-engine/docker-image/pull/34
Here is a sample of what it will look like with the new JSON output format enabled:
{"timestamp":"2022-10-07T12:12:25.46088192Z","host":"vespa-container.local","pid":"303/1","service":"configserver","component":"Container.org.eclipse.jetty.server.Server","level":"info","messages":["Started @6373ms"]}
{"timestamp":"2022-10-07T12:12:25.461883904Z","host":"vespa-container.local","pid":"303/1","service":"configserver","component":"Container.com.yahoo.container.jdisc.ConfiguredApplication","level":"info","messages":["Switching to the latest deployed set of configurations and components. Application config generation: 0"]}
{"timestamp":"2022-10-07T12:12:27.300350976Z","host":"vespa-container.local","pid":"626/33","service":"configproxy","component":"configproxy.com.yahoo.vespa.config.JRTConnection","level":"info","messages":["Connecting to tcp/vespa-container.local:19070"]}
{"timestamp":"2022-10-07T12:12:27.323859968Z","host":"vespa-container.local","pid":"626/32","service":"configproxy","component":"configproxy.com.yahoo.config.subscription.impl.JRTConfigRequester","level":"warning","messages":["Request callback failed: APPLICATION_NOT_LOADED. Connection spec: tcp/vespa-container.local:19070, error message: Failed request (No application exists) from Connection { Socket[addr=/<IP_ADDRESS>,port=38244,localport=19070] }"]}

The JSON object is not yet stable, and will likely change in the future. We hope to come up with a format that will be as much a as possible a drop-in for the major cloud providers managed kuberenetes clusters log parsing.
If you have any input on the format, please let me know!


”Telești”, ”Buduhala”, "Șomănești" are listed in the drop-down menu

Description:
”Telești”, ”Buduhala”, "Șomănești" are listed in the drop-down menu alongside their corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Write in the search bar "Telești"

Expected results
The location appears in the drop-down menu as "Sat Telești, GORJ (Telești)".

Step 3
Write in the search bar "Buduhala"

Expected results
The location appears in the drop-down menu as "Sat Buduhala, GORJ (Telești)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.


Idk what happend

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.

What problem did you encounter?
im not no 20 year old who is smart with pcs so i dont know whats happening

Can you send the log file please?

Issue has been awaiting clarification for over a month, closing as stale. If you have anything further to add, just respond and I'll reopen.


feat: removed components defaultProps

Description
The goal of this PR is to remove function components defaultProps since they are deprecated as per official RFC
Involved components are Button, Card, Divider, Icon, Menu, Modal, SegmentedControl, Table, ThemeProvider, BodyX, and HX.
Checklist

[x] commit message and branch name follow conventions
[x] you are not committing extraneous files or sensible data
[x] the browser console does not have any logged errors
[x] necessary labels have been applied to this pull request (enhancement, bug, ecc.)

No other reasons, it's simply to adhere to React recommendations (and to get rid of the annoying console errors)

yeah i noticed them yesterday while working on the repo!